Support for Currencies Depending on the Country Code.

I couldn't find a way in the SDK to get prices according to user's locale.
I tried ShopifyCustom to make a query  on products with inContext (Reference), but it didn't return the currency code for the provided country code.
My query
query alProducts @inContext(country: PK) {
  products(first: 5) {
    pageInfo {
      hasNextPage
    }
    edges {
      cursor
      node {
        options(first: 50) {
          id
          name
          values
        }
        variants(first: 250) {
          edges {
            node {
              id
              title
              image {
                altText
                id
                originalSrc
              }
              priceV2 {
                amount
                currencyCode
              }
              compareAtPriceV2 {
                amount
                currencyCode
              }
              weight
              weightUnit
              availableForSale
              sku
              requiresShipping
              quantityAvailable
              selectedOptions {
                name
                value
              }
            }
          }
          pageInfo {
            hasNextPage
          }
        }
        availableForSale
        collections(first: 250) {
          edges {
            node {
              description
              descriptionHtml
              id
              handle
              updatedAt
              title
            }
          }
        }
        createdAt
        description
        descriptionHtml
        handle
        id
        onlineStoreUrl
        productType
        publishedAt
        tags
        title
        updatedAt
        vendor
        images(first: 250) {
          edges {
            node {
              altText
              id
              originalSrc
            }
          }
        }
      }
    }
  }
}

The code snippet you provided did not return the price on my side. I had to update priceV2 to price.
While testing, it did not return the localized price for the country 'PK' only. I tried it with CA, US, NP and ES, it worked everytime.
Country as 'ES'

Country as 'CA'

Country as 'PK'

I can't actually figure out why this is happening. I guess It is coming from the Shopify store config

@imsujan276 Thanks a lot. I will check the store config.
Is there a plan to support multiple currencies within the SDK?

I am not thinking of adding this feature as I am occupied with work.
But PR is welcome

@imsujan276 It was a problem with the store configuration.


Unclear cookie behavior in middleware

First Check

[X] I added a very descriptive title to this issue.
[X] I used the GitHub search to find a similar issue and didn't find it.
[X] I searched the FastAPI documentation, with the integrated search.
[X] I already searched in Google "How to X in FastAPI" and didn't find any information.
[X] I already read and followed all the tutorial in the docs and didn't find an answer.
[X] I already checked if it is not related to FastAPI but to Pydantic.
[X] I already checked if it is not related to FastAPI but to Swagger UI.
[X] I already checked if it is not related to FastAPI but to ReDoc.

Commit to Help

[X] I commit to help with one of those options 👆

Example Code
middleware.py

async def add_user_data_to_request(request: Request, call_next):
    response = await call_next(request)
    user = await is_authenticated(request)
    if user:
        # add user_id to response
        response.set_cookie(
            key="user_id",
            value=user,
            httponly=True,
            max_age=1800,
            expires=60,
        )
        notifications = await MessagesService.get_count(user)
        print(f'notifications: {notifications}')
        if notifications:
            print('not')
            # add notification to response
            response.set_cookie(
                key="event_notifications",
                value=notifications['events'],
                httponly=True,
                max_age=1,
                expires=1,
            ) if notifications['events'] else None
    return response

template.html

{% if request.cookies.event_notifications %}
{{request.cookies.event_notifications}}
{% else %}
0
{% endif %}

Description
I am trying to add a user notification system. To do this I add the values I need to cookies using middleware. Using static html template jinja2 to interact with the user.
The way I see how this should work
Every time the user refreshes the page in the browser, middleware sends a request to the database to get the message count.If the counter has a non-zero value, a new value is added to the cookie using response.set_cookie. The html template checks if this value exists, and if it does, it is output in the interface.
The issue
So I send a message to a user, so notifications = await MessagesService.get_count(user) returns a response to me like this
{'total': 1, 'events': 1}
and I can see it in the console if I print(f'notifications: {notifications}')
But in the template, the value of the message counter is not displayed. But what is interesting and completely confusing is that if I refresh the page twice in one second, the counter starts to be displayed. Apparently this is due to the value of max_age=1. For experiment I tried to increase max_age = 3 and now that the counter is displayed twice to reload the page for 3 seconds. If I reload the page every 4 seconds or more, nothing is displayed.
Operating System
Linux
Operating System Details
No response
FastAPI Version
0.73.0
Python Version
3.9
Additional Context
p/s
I may not have chosen the most appropriate way to implement the notification system. If you have any ideas on how to improve it, I would be very interested to hear about it.

Basically you need 2 things:

deliver information from server to a user that he has new notification(s)
deliver information from user to a server that he saw it and we should stop inform him about it.

With that in mind, cookies are not the best way to go in my opinion, and expires=1 or something like that makes sense only when you want notification alert based on time, which is not usually a case, you user can visit a site after 2 days and want to see notification. And you cant really handle second requirement in that setup where server doesn't care about client and always send this cookies. Except you reset number of notification every time it is more than 0.
Some standard way is to have some kind of counter with "number of notification" that is unique for every user, for example stored as field in database in user table. And you already have something that look like that, notifications = await MessagesService.get_count(user). Now first requirement is simple, you can use cookies (why not), i would use header but this is my taste or even separate GET endpoint where i ask for a number of notifications.
And then your client need a way to reset his "number of notification" field in user table, for example you can make POST request when user click on notification div to a server or something like that and this action set it to 0, in next reload there will be no notifications.

@rafsaf thanks for reply!
I may not have understood you exactly, but I have it  as you described. There is a table in the database where request notifications = await MessagesService.get_count(user). Every time the user reloads the page this request is executed.  There is a functionality which resets this table in the database at the moment of reading the message. But it is not clear why the displaying of the notification counter works so strangely.

Thanks for the help @rafsaf ! ☕
I assume you solved it, thanks for closing the issue @Lepiloff  👍


develop

Pull Request Test Coverage Report for Build 814618598

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

Totals

Change from base Build 814254905:
2.6%

Covered Lines:
297

Relevant Lines:
305

💛  - Coveralls


Remove Indent Guides in Solution Explorer

Please add ability to remove Indent Guides:

settings for regular explorer:
  "workbench.tree.renderIndentGuides": "none",

You can already hide the ident guides (settings -> Workbench -> Appearance -> Tree: Render Ident Guides):

@fernandoescolar
I know that I can do it in regular explorer, I wrote the peace of settings.json code:
"workbench.tree.renderIndentGuides": "none"
but I need it in regular explorer. I want to remove this lines only in solution explorer like in Visual Studio IDE.

Currently this feature is handled for all treeviews and there is no extension point in the vscode-api to do it in a custom treeview :(
https://code.visualstudio.com/api/references/vscode-api#TreeDataProvider


Info button links to endpoint

Provide endpoint information related to the selection of source

For the info button on the available climate data please see this for reference:
https://cds.nccs.nasa.gov/nex/
More details are in the [documentation pdf] (https://cds.nccs.nasa.gov/wp-content/uploads/2014/04/NEX-DCP30_Tech_Note_v0.pdf)
A modified brief could be:
NEX-DCP30: Downscaled 30 Arc-Second CMIP5 Climate Projections for
Studies of Climate Change Impacts in the United States
This NASA dataset is provided to assist the science community in conducting studies of climate
change impacts at local to regional scales, and to enhance public understanding of possible future
climate patterns and climate impacts at the scale of individual neighborhoods and communities. These are derived from General Circulation Model (GCM) runs conducted under the Coupled Model Intercomparison Project Phase 5 (CMIP5) [Taylor et al. 2012] and across the four greenhouse gas emissions scenarios known as Representative Concentration Pathways (RCPs) [Meinshausen et al. 2011] developed for the Fifth Assessment Report of the Intergovernmental Panel on Climate Change (IPCC AR5). We provide three scenarios here: observed or modern (years 1950-2005), RCP 4.5, RCP 8.5 (for comparison of scenarios). Details and contact information are here https://cds.nccs.nasa.gov/wp-content/uploads/2014/04/NEX-DCP30_Tech_Note_v0.pdf

Added grey info button next to raster picker. It's very basic, just links to the API endpoint which displays raster details.
Could we include a link to the technical documentation for each raster layer within the API itself? Then it will show up automatically.
Example of raster information endpoint: https://ecoengine.berkeley.edu/api/rstore/pr_yr_ens-avg_amon_historical-1953-01-01/
http://stamen.github.io/ecoengine/explore/


fix: fix e2e tests for windows

they pass: https://github.com/kili-technology/kili-python-sdk/actions/runs/3872221419


Login/Sign-Up Page UI Update

[ ] I would like to work on this page as this is also an important part for a dynamic website so it should be attractive.
[ ] I've some great UI ideas to implement on this site.
[ ] Suggestion - Login and Sign-Up should be in one page only in different tabs. (UI idea)

@QAZIMAAZARSHAD can you please assign this issue to me as participant of LGM-SOC'21

[ ]  I would like to work on this page as this is also an important part for a dynamic website so it should be attractive.
[ ]  I've some great UI ideas to implement on this site.
[ ]  Suggestion - Login and Sign-Up should be in one page only in different tabs. (UI idea)

@QAZIMAAZARSHAD can you please assign this issue to me as participant of LGM-SOC'21

@AKC068 this issue is already open. See issue number https://github.com/QAZIMAAZARSHAD/Movie-Streaming-Website/issues/13.
Also, that issue has been resolved and will be closed soon.
And if you believe your design is better than the new one then you are welcome to open a new issue to improve it.
Wait till I merge the other PR. Then you can open another issue.

Sure


Add formatter

Codecov Report

Merging #21 (2e3482a) into main (d963acd) will not change coverage.
The diff coverage is 100.00%.

@@            Coverage Diff            @@
##               main      #21   +/-   ##
=========================================
  Coverage     95.68%   95.68%
  Complexity      391      391
=========================================
  Files           101      101
  Lines           950      950
  Branches         42       42
=========================================
  Hits            909      909
  Misses           36       36
  Partials          5        5

Impacted Files
Coverage Δ

...ttrader/candle/api/application/CandleInteractor.kt
100.00% <ø> (ø)

...dle/api/infrastructure/repository/dao/CandleDao.kt
100.00% <ø> (ø)

...pi/infrastructure/repository/entity/CandleQuery.kt
100.00% <ø> (ø)

...r/candle/api/presentation/handler/CandleHandler.kt
100.00% <ø> (ø)

...ient/http/bitflyer/BitflyerHttpPrivateApiClient.kt
100.00% <ø> (ø)

...bclient/http/bitflyer/model/order/OrderResponse.kt
100.00% <ø> (ø)

...nt/http/bitflyer/model/order/OrderSignalRequest.kt
95.45% <ø> (ø)

...rastructure/webclient/http/order/OrderApiClient.kt
100.00% <ø> (ø)

...bittrader/order/api/application/OrderInteractor.kt
100.00% <ø> (ø)

...bittrader/order/api/domain/service/OrderService.kt
100.00% <ø> (ø)

... and 5 more

Continue to review full report at Codecov.

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


add min and max math block

construct min and max math blocks with ability to add arbitrary number of inputs to be compared.

Coordinate with #94 since min and max are list functions


Specified font not working when .Technology = Technology.DirectWrite

It's OK when .Technology = Default, specified font will be displayed.
When .Technology = Technology.DirectWrite, it seems font is reset to default.
do I need to set something else?

Hi,
This seems to be Scintilla issue as the technology is set via direct message to the underlying Scintilla control:
    public Technology Technology
    {
        get
        {
            return (Technology)DirectMessage(NativeMethods.SCI_GETTECHNOLOGY);
        }
        set
        {
            var technology = (int)value;
            DirectMessage(NativeMethods.SCI_SETTECHNOLOGY, new IntPtr(technology));
        }
    }

...
public const int SCI_SETTECHNOLOGY = 2630;

See: https://www.scintilla.org/ScintillaDoc.html#SCI_SETTECHNOLOGY
Ps. Sorry for the late response 🙈

Closed due inactivity.


Make edge server re-validate file before using them

Sets cache control policy with max-age=0 so that the edge servers checks with the source server if there are any new file available before serving them

Closing as not using CDN link in the extension to download LEAN executables anymore


Mark classes as final where applicable

I checked and there should be no conflict with https://github.com/Microsoft/calculator/pull/188

I'm fine with this change, but I noticed that this improvement is limited only to pure C++ classes.  For C++/CX, there is the equivalent sealed keyword and, if we think classes should be final/sealed where possible, then we should also seal the CX classes.


Repo Card Spacing On Mobile View Updated

as im searching for a repo i found out that the repo card is getting out of the screen. here are the two different links your main one and mine(preview)
preview: https://hacktoberfest-projects-six.vercel.app/repos/javascript
Main : https://finder.usmans.me/repos/javascript
please Use Mobile Device. Im using IPhone X device
#121

Can you also share the screenshots of the issues you were facing because it is unnoticeable to me for some reason?
Also reverse back the all contributors and readme files as those will be updated automatically

seems like its not showing again


certificate has expired [again] (IDFGH-5948)

Guys come on, this isn't the first time this has caught you out.
Error: certificate has expired
	at TLSSocket.onConnectSecure (_tls_wrap.js:1497:34)
	at TLSSocket.emit (events.js:315:20)
	at TLSSocket._finishInit (_tls_wrap.js:932:8)
	at TLSWrap.ssl.onhandshakedone (_tls_wrap.js:706:12)
Error opening esp-idf version list file. HTTP/HTTPS Request error https://dl.espressif.com/dl/esp-idf/idf_versions.txt

Hi!
Same issue here. It gets up to loading Ninja in the tools section and then fails. I've had the problem for a couple of days while trying to set up VSCode with esp-idf for the first time. I thought it was just me messing things up.

Same problem

Facing the Same Issue

Hi all, thanks for the reports and apologies for the slow reaction to this issue.
dl.espressif.com was using the following certificate chain, which is recommended by letsencrypt for compatibility:
 0 s:/CN=dl.espressif.com
   i:/C=US/O=Let's Encrypt/CN=R3
 1 s:/C=US/O=Let's Encrypt/CN=R3
   i:/C=US/O=Internet Security Research Group/CN=ISRG Root X1
 2 s:/C=US/O=Internet Security Research Group/CN=ISRG Root X1
   i:/O=Digital Signature Trust Co./CN=DST Root CA X3

Most operating systems which have "ISRG Root X1" certificate in the certificate store will recognize certificate "1" in the chain as a valid root certificate, and will ignore the fact that certificate "2" is signed by an expired root ("DST Root CA X3").
However, as the above letsencrypt page notes,

In some cases, the expiry of the root (and its related expiring R3 intermediate certificate) may causes certificates to be considered untrusted or invalid.

Which seems to be what is happening in your case.
We have now updated the certificate chain served by dl.espressif.com to the following:
 0 s:/CN=dl.espressif.com
   i:/C=US/O=Let's Encrypt/CN=R3
 1 s:/C=US/O=Let's Encrypt/CN=R3
   i:/C=US/O=Internet Security Research Group/CN=ISRG Root X1
 2 s:/C=US/O=Internet Security Research Group/CN=ISRG Root X1
   i:/C=US/O=Internet Security Research Group/CN=ISRG Root X1

which is the "modern" chain recommended by letsencrypt.
Please try the installation process again. If you meet an issue related to the ISRG Root X1 certificate not being trusted, please follow the instructions here, section "Clients (browsers etc)" to add ISRG Root X1 to your OS certificate store.

Awesome, thanks for the quick update.


fix: Fixed Index out of bounds issue with network ordinal

Added Goerli support
Added tests for Goerli
All tests passed (50 of 50 tests passed)

Hey @craigraw
Let me know if this fits. We have the need to use Goerli so I've added support for that. Following this to specify Goerli value

Although I have no need of Ethereum values myself, I've no particular problem adding support for what is defined in the specification.
Quite a few of the changes proposed simply add spaces before brackets, probably due to automatic code formatting - could these be removed to simplify and keep the code style consistent?

Thanks!


haiku: add missing ioctl identifiers to control the TTY

Ref. termios.h

Thanks for the pull request, and welcome! The Rust team is excited to review your changes, and you should hear from @gnzlbg (or someone else) soon.
If any changes to this PR are deemed necessary, please add them as extra commits. This ensures that the reviewer can see what has changed since they last reviewed the code. Due to the way GitHub handles out-of-date commits, this should also make it reasonably obvious what issues have or haven't been addressed. Large or tricky changes may require several passes of review and changes.
Please see the contribution instructions for more information.

@bors: r+

:pushpin: Commit 60254f1151af28d923c547014483c7598f21c3d4 has been approved by gnzlbg

:hourglass: Testing commit 60254f1151af28d923c547014483c7598f21c3d4 with merge a73d6bc00d0a7d7c63028e6b69ed9b46734a2369...

:sunny: Test successful - checks-cirrus-freebsd-11, checks-cirrus-freebsd-12, checks-travis, status-appveyor
Approved by: gnzlbg
Pushing a73d6bc00d0a7d7c63028e6b69ed9b46734a2369 to master...


User story 4 // Add radio buttons to form

Hi Claudia! This looks very good in general. 🚀 I just have one little comment about the keys you use to store the value from the radio button.

One more thing: I'm not a big fan of calling styled components StyledXY. Take this for example:
			<StyledDiv> 					<StyledInput type="radio" value="proud" id="mood2" name="mood" required /> 					<StyledRadioLabel> Proud</StyledRadioLabel> 				</StyledDiv>
Isn't it much better readable when it looks like this?
			<MoodButtonBox> 					<MoodButton type="radio" value="proud" id="mood2" name="mood" required /> 					<MoodButtonLabel> Proud</MoodButtonLabel> 				</MoodButtonBox>


Node.exe path should support relative paths (again)

I upgraded from 1.1 Beta to the 7-15-2015 dev build and reopened my solution and was unable to run it.
I had a relative path in the project settings under "Node.exe path". I use the electron pre-built package which installs the executable under node_modules\electron-prebuilt\dist\electron.exe relative to the project. It used to work but no longer does.
Specifying a relative path is key in allowing us to submit the .njsproj file into source control and having others be able to debug/run the project without having to modify the project file to specify an absolute path on their machines.

I'm pretty sure we never supported this because we haven't really changed anything in this codepath, and the current code would always produce an error if File.Exists returned false on the relative path (running without the debugger attached, however, does work.) In any case, working on a fix that explicitly adds this support.

 I'm not denying that it used to work before... Just saying that I am very surprised if it did :smiley:

Interesting. I checked my the project file history in source control and--surprisingly, I suppose--did not have an absolute path before the upgrade and it surely worked!
<NodeExePath>node_modules\electron-prebuilt\dist\electron.exe</NodeExePath>

Thank you for officially adding this feature! A small change but a major win :)

Thanks @mousetraps! This will be used every day here.

Np, mind trying the latest dev build?


App router with Next 13.5.1 - Build error on preview routes

Hi!
Thanks for your amazing work on this module. I have a little build problem since I updated to Next 13.5.1.
Versions

@prismicio/next: 1.3.4
next: 13.5.1
node: 18.15

Reproduction
Following Prismic documentation for preview, add a /preview route handler in Next.Js App Router with the following content:
export async function GET(request: NextRequest) {
    const client = createClient();

    draftMode().enable();

    await redirectToPreviewURL({ client, request });
}

Steps to reproduce
Run a simple build with Next: npm run build
What is expected?
No build error.
What is actually happening?
Error on checking validity of types:
src/app/api/preview/route.ts
Type error: Route "src/app/api/preview/route.ts" has an invalid export:
  "Promise<void>" is not a valid GET return type:
    Expected "Response | Promise<Response>", got "Promise<void>".
      Expected "Promise<Response>", got "Promise<void>".
        Expected "Response", got "void".

The error appeared since Next.JS changed their expected return type for the route handlers. The redirectToPreviewURL() should have a return type of never to match the call to redirect() from next/navigation. The problem is also present for exitPreview() which should return Response instead of Response | void.
I get that those problems are due to @prismicio/next trying to support both Pages router and App router with the same functions. Maybe it could be interesting to have a parameter on those functions that provides a way to change the function return type conditionnally.

Hi @roux1max, thanks for the detailed report! 🙂
The type error should be fixed as of @prismicio/next v1.3.5. You can find details on the necessary changes in this PR: #85. Your diagnosis was correct; redirectToPreviewURL() needed to return never when we detect the App Router is being used.
Could you update to the latest version and let me know if the build error still happens?
npm install @prismicio/next@latest

Thank you!

The problem is now fixed!
Thanks for the quick answer!


SSL Certificates not applied

I see that there are a few variations of this issue but I wanted to document my own experience. I've been using Kong for a couple of years but without requiring it to manage SSL certificates, hence I haven't encountered this problem until now. So, how I got here:

Installed Kong to Google Cloud via the Bitnami image, works as expected.
Configured Services, Routes, Plugins and Consumers/Credentials without any problems.
Added my wildcard certificate and associated SNIs

Having done all that I find that my Routes are working as desired, sending traffic to two Services which handle different parts of my API. However, I cannot get the SSL to be returned against these routes... so I'm confused as to whether I've made a subtle mistake in setup or if there's a genuine bug in Kong.
If it helps, here are the details of my config:
Services:
{
    "data": [
        {
            "connect_timeout": 60000,
            "created_at":<PHONE_NUMBER>,
            "host": "atomcore-node.c.blockbooker.internal",
            "id": "06695dc9-238b-47c7-b241-00a6b541054f",
            "name": "atomcore-api-node",
            "path": null,
            "port": 3000,
            "protocol": "http",
            "read_timeout": 60000,
            "retries": 5,
            "updated_at":<PHONE_NUMBER>,
            "write_timeout": 60000
        },
        {
            "connect_timeout": 60000,
            "created_at":<PHONE_NUMBER>,
            "host": "atomcore-api-1.c.blockbooker.internal",
            "id": "c6234e05-a6be-486b-a256-13f83134ba58",
            "name": "atomcore-api",
            "path": null,
            "port": 8000,
            "protocol": "http",
            "read_timeout": 60000,
            "retries": 5,
            "updated_at":<PHONE_NUMBER>,
            "write_timeout": 60000
        }
    ],
    "next": null
}

Routes:
{
    "data": [
        {
            "created_at":<PHONE_NUMBER>,
            "hosts": [
                "api.atomco.re"
            ],
            "id": "201fec12-ef34-44e9-8325-335e83799d85",
            "methods": null,
            "paths": [
                "/v2"
            ],
            "preserve_host": false,
            "protocols": [
                "http",
                "https"
            ],
            "regex_priority": 0,
            "service": {
                "id": "06695dc9-238b-47c7-b241-00a6b541054f"
            },
            "strip_path": true,
            "updated_at":<PHONE_NUMBER>
        },
        {
            "created_at":<PHONE_NUMBER>,
            "hosts": [
                "api.atomco.re"
            ],
            "id": "a5734885-7f39-4d31-b8f7-463cea96b535",
            "methods": null,
            "paths": [
                "/v1"
            ],
            "preserve_host": false,
            "protocols": [
                "http",
                "https"
            ],
            "regex_priority": 0,
            "service": {
                "id": "c6234e05-a6be-486b-a256-13f83134ba58"
            },
            "strip_path": false,
            "updated_at":<PHONE_NUMBER>
        }
    ],
    "next": null
}

Certificates:
{
    "data": [
        {
            "cert": "-----BEGIN CERTIFICATE----- ... -----END CERTIFICATE-----",
            "created_at":<PHONE_NUMBER>,
            "id": "03467dba-e40c-4acf-ace0-3f2f8270443d",
            "key": "-----BEGIN RSA PRIVATE KEY----- ... -----END RSA PRIVATE KEY-----",
            "snis": [
                "api-test.atomco.re",
                "api.atomco.re"
            ]
        }
    ],
    "next": null
}

I hope that provides enough context and details for someone to offer a suggestion, it would be much appreciated.

Ping @neilalbrock, are you able to provide any more feedback and details on your configuration?

Closing this as stale. If you are able to provide my data, please let us know and we’ll be happy to take a look. Thanks!


Programmatically specify reporter in .run()

Unless I missed the existing way of doing this.
Like this:
.run({reporter: 'spec'});

If reporter is not found - it would default to dot-matrix.

Good one. Please merge it.

Seconded, I think this would be very helpful.


For DSYM files, don't load any segments except __DWARF

Fix for #23

Thank you very much. Merged.

Minor mess-up: line 536 of mach_o.py should go instead:
raw = (c_ubyte * sec.size).from_buffer(data, sec.offset)


Adds ceres-solver to Indigo.

This is failing CI because the version needs to be a branch. But i'm not sure this is exactly what you're looking for a source entry here can enable CI of this repo (devel jobs), and documentation generation. But if you want to use it as a dependency debian packages need to be made by running it through the bloom release process.


Requires old hugo version

[daniel@reef fluxcd.io ]$ git grep -i hugo netlify.toml
netlify.toml:HUGO_VERSION = "0.55.6"
[daniel@reef fluxcd.io ]$

Using recent hugo
[daniel@reef fluxcd.io ]$ hugo version
Hugo Static Site Generator v0.75.1/extended linux/amd64 BuildDate: 2020-09-15T14:28:18Z
[daniel@reef fluxcd.io ]$ hugo server
port 1313 already in use, attempting to use an available port
Start building sites …
Built in 13 ms
Error: Error building site: TOCSS: failed to transform "sass/style.sass" (text/x-sass): SCSS processing failed: file "stdin", line 10, col 9: malformed URL
[daniel@reef fluxcd.io ]$

Using an older version of hugo
[daniel@reef fluxcd.io ]$ ./hugo version
Hugo Static Site Generator v0.57.0-9B00E647/extended linux/amd64 BuildDate: 2019-08-14T08:12:12Z
[daniel@reef fluxcd.io ]$ ./hugo server
port 1313 already in use, attempting to use an available port

                   | EN
+------------------+----+
  Pages            |  3
  Paginator pages  |  0
  Non-page files   |  0
  Static files     |  8
  Processed images |  0
  Aliases          |  0
  Sitemaps         |  1
  Cleaned          |  0

Total in 169 ms
Watching for changes in /home/daniel/dev/fluxcd.io/{assets,content,layouts,static}
Watching for config changes in /home/daniel/dev/fluxcd.io/config.toml
Environment: "development"
Serving pages from memory
Running in Fast Render Mode. For full rebuilds on change: hugo server --disableFastRender
Web Server is available at http://localhost:38783/ (bind address <IP_ADDRESS>)
Press Ctrl+C to stop

Fixed with #24. Thanks @hiddeco !


Improve linked content with descriptions and optionally an image

Suggestion/Improvement
Currently, our links to external resources for learning are not very descriptive. Improve one or more links by adding a description, and optionally a screenshot or gif that provides more context for readers to the linked content.
Description
For example, a link that says only "Codum" can be improved to provide more information in the link itself such as:
"Codum engineering accountability buddy platform"
Codum runs multiple cohorts a year. They pair individuals who form a team of two. These two define a goal for themselves and hold each other accountable for achieving that goal within the timeframe of the cohort.
Motivation
We have great information on other pages of the knowledge-hub, we should raise the quality of our links to external content, to match other information.
Implementation (if applicable)
Choose one or more links to improve.

I want to contribute to this.

I want to contribute to this.

Thanks @RitaDee ! I've assigned you to the issue.

@RitaDee I pushed some changes today so you may want to update if you started on this at all. :)

@RitaDee I pushed some changes today so you may want to update if you started on this at all. :)

Thank you for the notification. I will do a rebase.


Readme update for blog

Medium seems to downsample the image no matter what!!! WTF?
https://medium.com/@hossainkhan/source-code-syntax-highlighting-on-android-taking-full-control-b704fd4bd8ee

Set of screenshots taken from the demo app


Add specific option to build or not tools

Hello!
This PR brings the feature of adding a new CMake option to build or not the tools provided by GeographicLib. There are cases where users only want to consume libraries and want to execute a minimal build, including avoiding documents and tools.

CMakeLists.txt already provides the capability to skip the installation of any components, e.g., with -D BINDIR=, -D MANDIR=, etc.
This is probably a better solution than skipping the build of particular components as this PR implements, because if you don't build the tools, (1) the man pages shouldn't be installed, (2) you need to skip the tests.

@cffk All three: tools, man and tests could be excluded from building as well and consume only libraries. Besides that, the same CMakeLists.txt has options to exclude man pages, tests and documentation from building already.

I introduced the various BINDIR, MANDIR, options to the CMake configuration to give packagers the flexibility to make sub-packages for GeographicLib.  At the time, I figured that building the tools even if you didn't want them installed would be a minimal piece of overhead.
So, perhaps you can tell me the scenario in which this overhead needs to be avoided?  The only time the tools will get repeatedly built is if you're modifying the source code for GeographicLib.  Is this your situation?
A cleaner approach to the problem might be to exclude from make all any components which are not to be installed (as signified by the corresponding installation directory evaluating as OFF).
This would appear to be a superior solution.  Setting BINDIR= and SBINDIR= would mean that make all would not build the tools. However make test would cause them to be built.

@cffk Our specific case is packaging Geographic library in Conan (https://conan.io/center/recipes/geographiclib). In the past we added an option in the package to avoid building/packaging tools. But thinking about your last comment, makes sense only skipping the installation step. Thank you for your feedback.

OK, I'll close this PR.  As a result of this discussion, I've made some changes to the CMake configuration:

CMake configuration changes

exclude the tools and doc targets from
the default build (make all) if these components
won't be installed (BINDIR or DOCDIR
set to OFF).

testprograms is now excluded from the default build
and it now depends on tools.  Before running the
test suite, build testprograms<, e.g., make
testprograms test.

added a target libs to build only the libraries.

Thanks for your feedback.

@cffk Thank you!!


Cups filled the drive with naïve « auto » debug log feature

Hi,
Got a workstation which no more started (debian stable / buster) since deamon logs have been flooded.
The relevant log extracts :
Jul 3 23:01:53 poste-02 systemd[1]: Starting Hostname Service...
Jul 3 23:01:53 poste-02 dbus-daemon[538]: [system] Successfully activated service 'org.freedesktop.hostname1'
Jul 3 23:01:53 poste-02 systemd[1]: Started Hostname Service.
Jul 3 23:02:24 poste-02 systemd[1]: systemd-hostnamed.service: Succeeded.
Jul 3 23:15:56 poste-02 cups-browsed[604]: Timeout happened during creation of the queue Canon_xxx_series_shared_by_other, turn on DebugLogging for more info.
Jul 3 23:18:48 poste-02 cups-browsed[604]: Timeout happened during creation of the queue Canon_xxx_series_shared_by_other, turn on DebugLogging for more info.
Jul 3 23:19:48 poste-02 cups-browsed[604]: Timeout happened during creation of the queue Canon_xxx_series_shared_by_other, turn on DebugLogging for more info.
Jul 3 23:21:05 poste-02 cups-browsed[604]: Timeout happened during creation of the queue Canon_xxx_series_shared_by_other, turn on DebugLogging for more info.
Jul 3 23:21:28 poste-02 cups-browsed[604]: Timeout happened during creation of the queue Canon_xxx_series_shared_by_other, turn on DebugLogging for more info.
Jul 3 23:21:28 poste-02 cups-browsed[604]: Max number of retries (5) for creating print queue Canon_xxx_series_shared_by_other reached, skipping it.
Jul 3 23:21:28 poste-02 cups-browsed[604]: Max number of retries (5) for creating print queue Canon_xxx_series_shared_by_other reached, skipping it.
Jul 3 23:21:28 poste-02 cups-browsed[604]: Max number of retries (5) for creating print queue Canon_xxx_series_shared_by_other reached, skipping it.
Jul 3 23:21:28 poste-02 cups-browsed[604]: Max number of retries (5) for creating print queue Canon_xxx_series_shared_by_other reached, skipping it.
Jul 3 23:21:28 poste-02 cups-browsed[604]: Max number of retries (5) for creating print queue Canon_xxx_series_shared_by_other reached, skipping it.

Last lines are repeated soo much that ~33 minutes and 4Go after, the disk has been full.
Maybe, the automation of « turn on DebugLogging » should be done with more cautious.

The version:
# apt show cups-browsed
Package: cups-browsed
Version: 1.21.6-5
Priority: optional
Section: net
Source: cups-filters
Maintainer: Debian Printing Team<EMAIL_ADDRESS>Installed-Size: 379 kB
Depends: libavahi-client3 (>= 0.6.16), libavahi-common3 (>= 0.6.16), libavahi-glib1 (>= 0.6.16), libc6 (>= 2.4), libcups2 (>= 2.0.2), libcupsfilters1 (>= 1.12.0), libglib2.0-0 (>= 2.37.3), libldap-2.4-2 (>= 2.4.7), cups-daemon, lsb-base
Recommends: avahi-daemon (>= 0.6.31-3~)
Breaks: cups-filters (<< 1.4.0~)
Replaces: cups-filters (<< 1.4.0~)
Enhances: cups
Homepage: https://wiki.linuxfoundation.org/openprinting/
Download-Size: 177 kB
APT-Manual-Installed: no
APT-Sources: http://deb.debian.org/debian buster/main amd64 Packages
Description: filtres CUPS OpenPrinting — cups-browsed
[...]

The package:
# grep -v '^#\|^$' cups-browsed.conf
BrowseRemoteProtocols dnssd cups

@fpoulain thanks for pointing out that issue
If I'm able to understand your problem correctly you are saying that, the logs kept on filling over the time of 33 minutes once you turned on the DebugLogging until all of your disk space was used up due to which you could no longer run system, correct me if I have misunderstood your query.

Hum, I whish my globish doesn't pollute a lot the issue.
This OS has been installed by me quite recently. The configuration is the default one. What happened is the following:

the user experienced booting issues
it appears the partition has been flooded by cups-browsed logs
a post mortem log reading let me thinking that cups-browsed automatically set him self with DebugLogging turned on

(maybe I am wrong with this hypothesis)
Actually, the bug is that the sys logs have been flooded with :

Max number of retries (5) for creating print  reached, skipping it.

@fpoulain as per for automatically setting debugging, that has to be enabled by the user first. As you can see in err log that the person was asked for debug logging. However I am quite curious if these logs were present in filename "cups-browsed_log" or some other file?
Also could you send us your cups-browsed.conf and cupsd.conf files. There is something wrong with the maximum number of logs that can be maintained in whichever log file you recieved this output.

@tillkamppeter @fpoulain after going through the code file for cups-browsed, I'm quite sure that these are not debug messages. Debug mode has to be turned on.
@tillkamppeter sir this error is being written to stderr (which if I am correct are stored in /var/logs/cups/error_log)
This is a part of update_cups_queues function. Also having a look at the error log file snapshot posted by user, if we see the timestamp it points out that the function is either being called too many times in a small time frame, or it is getting caught in between some infinite loop.

I have found out what is happening here. Nothing with CUPS' error_log or cups-browsed's cups-browsed_log.
The problem is that cups-browsed was writing two error messages to stderr (and when invoked through a systemd service these go to syslog/journald) and not to its own log. I have spotted the two messages and corrected them in the cups-browsed code, commit aa19fa3f21, please test.

@tillkamppeter but still doesn't the fact holds that even after commit aa19fa3, the length of debug_logging can go lengths if not maintained. Builinding upon your suggestion I have sent you another kind of solution to keep the debugLogging logs in check.

Hi,

Also could you send us your cups-browsed.conf and cupsd.conf files.

The cups-browsed.conf is summarized in this comment and here is the cupsd.conf:
LogLevel warn
PageLogFormat

MaxLogSize 0

Listen localhost:631
Listen /run/cups/cups.sock

Browsing On
BrowseLocalProtocols dnssd

DefaultAuthType Basic

WebInterface Yes

<Location />
  Order allow,deny
</Location>

<Location /admin>
  Order allow,deny
</Location>

<Location /admin/conf>
  AuthType Default
  Require user @SYSTEM
  Order allow,deny
</Location>

<Location /admin/log>
  AuthType Default
  Require user @SYSTEM
  Order allow,deny
</Location>

<Policy default>
  # Job/subscription privacy...
  JobPrivateAccess default
  JobPrivateValues default
  SubscriptionPrivateAccess default
  SubscriptionPrivateValues default

  # Job-related operations must be done by the owner or an administrator...
  <Limit Create-Job Print-Job Print-URI Validate-Job>
    Order deny,allow
  </Limit>

  <Limit Send-Document Send-URI Hold-Job Release-Job Restart-Job Purge-Jobs Set-Job-Attributes Create-Job-Subscription Renew-Subscription Cancel-Subscription Get-Notifications Reprocess-Job Cancel-Current-Job Suspend-Current-Job Resume-Job Cancel-My-Jobs Close-Job CUPS-Move-Job CUPS-Get-Document>
    Require user @OWNER @SYSTEM
    Order deny,allow
  </Limit>

  # All administration operations require an administrator to authenticate...
  <Limit CUPS-Add-Modify-Printer CUPS-Delete-Printer CUPS-Add-Modify-Class CUPS-Delete-Class CUPS-Set-Default CUPS-Get-Devices>
    AuthType Default
    Require user @SYSTEM
    Order deny,allow
  </Limit>

  # All printer operations require a printer operator to authenticate...
  <Limit Pause-Printer Resume-Printer Enable-Printer Disable-Printer Pause-Printer-After-Current-Job Hold-New-Jobs Release-Held-New-Jobs Deactivate-Printer Activate-Printer Restart-Printer Shutdown-Printer Startup-Printer Promote-Job Schedule-Job-After Cancel-Jobs CUPS-Accept-Jobs CUPS-Reject-Jobs>
    AuthType Default
    Require user @SYSTEM
    Order deny,allow
  </Limit>

  # Only the owner or an administrator can cancel or authenticate a job...
  <Limit Cancel-Job CUPS-Authenticate-Job>
    Require user @OWNER @SYSTEM
    Order deny,allow
  </Limit>

  <Limit All>
    Order deny,allow
  </Limit>
</Policy>

<Policy authenticated>
  # Job/subscription privacy...
  JobPrivateAccess default
  JobPrivateValues default
  SubscriptionPrivateAccess default
  SubscriptionPrivateValues default

  # Job-related operations must be done by the owner or an administrator...
  <Limit Create-Job Print-Job Print-URI Validate-Job>
    AuthType Default
    Order deny,allow
  </Limit>

  <Limit Send-Document Send-URI Hold-Job Release-Job Restart-Job Purge-Jobs Set-Job-Attributes Create-Job-Subscription Renew-Subscription Cancel-Subscription Get-Notifications Reprocess-Job Cancel-Current-Job Suspend-Current-Job Resume-Job Cancel-My-Jobs Close-Job CUPS-Move-Job CUPS-Get-Document>
    AuthType Default
    Require user @OWNER @SYSTEM
    Order deny,allow
  </Limit>

  # All administration operations require an administrator to authenticate...
  <Limit CUPS-Add-Modify-Printer CUPS-Delete-Printer CUPS-Add-Modify-Class CUPS-Delete-Class CUPS-Set-Default>
    AuthType Default
    Require user @SYSTEM
    Order deny,allow
  </Limit>

  # All printer operations require a printer operator to authenticate...
  <Limit Pause-Printer Resume-Printer Enable-Printer Disable-Printer Pause-Printer-After-Current-Job Hold-New-Jobs Release-Held-New-Jobs Deactivate-Printer Activate-Printer Restart-Printer Shutdown-Printer Startup-Printer Promote-Job Schedule-Job-After Cancel-Jobs CUPS-Accept-Jobs CUPS-Reject-Jobs>
    AuthType Default
    Require user @SYSTEM
    Order deny,allow
  </Limit>

  # Only the owner or an administrator can cancel or authenticate a job...
  <Limit Cancel-Job CUPS-Authenticate-Job>
    AuthType Default
    Require user @OWNER @SYSTEM
    Order deny,allow
  </Limit>

  <Limit All>
    Order deny,allow
  </Limit>
</Policy>

<Policy kerberos>
  # Job/subscription privacy...
  JobPrivateAccess default
  JobPrivateValues default
  SubscriptionPrivateAccess default
  SubscriptionPrivateValues default

  # Job-related operations must be done by the owner or an administrator...
  <Limit Create-Job Print-Job Print-URI Validate-Job>
    AuthType Negotiate
    Order deny,allow
  </Limit>

  <Limit Send-Document Send-URI Hold-Job Release-Job Restart-Job Purge-Jobs Set-Job-Attributes Create-Job-Subscription Renew-Subscription Cancel-Subscription Get-Notifications Reprocess-Job Cancel-Current-Job Suspend-Current-Job Resume-Job Cancel-My-Jobs Close-Job CUPS-Move-Job CUPS-Get-Document>
    AuthType Negotiate
    Require user @OWNER @SYSTEM
    Order deny,allow
  </Limit>

  # All administration operations require an administrator to authenticate...
  <Limit CUPS-Add-Modify-Printer CUPS-Delete-Printer CUPS-Add-Modify-Class CUPS-Delete-Class CUPS-Set-Default>
    AuthType Default
    Require user @SYSTEM
    Order deny,allow
  </Limit>

  # All printer operations require a printer operator to authenticate...
  <Limit Pause-Printer Resume-Printer Enable-Printer Disable-Printer Pause-Printer-After-Current-Job Hold-New-Jobs Release-Held-New-Jobs Deactivate-Printer Activate-Printer Restart-Printer Shutdown-Printer Startup-Printer Promote-Job Schedule-Job-After Cancel-Jobs CUPS-Accept-Jobs CUPS-Reject-Jobs>
    AuthType Default
    Require user @SYSTEM
    Order deny,allow
  </Limit>

  # Only the owner or an administrator can cancel or authenticate a job...
  <Limit Cancel-Job CUPS-Authenticate-Job>
    AuthType Negotiate
    Require user @OWNER @SYSTEM
    Order deny,allow
  </Limit>

  <Limit All>
    Order deny,allow
  </Limit>
</Policy>

@fpoulain try the commit aa19fa3 should solve the problem and these messages will be redirected now to cups-browsed_log. Please assure us if the fix works for you.
Thanks.

I know that this bug is closed, but unless some other fixes were done, this is a purely cosmetic "fix", which will likely leave cups-browsed with 100% CPU-usage. The fix https://github.com/OpenPrinting/cups-filters/commit/aa19fa3f21d9f00150e188c3cb84b39965c23367 will probably prevent the disk from filling up, but the real problem seems to be that it tried to create more than 50.000 print-queues per second!!!
fmsv030@eichhorn:/var/log$ sudo zgrep -c "Oct 16 19:17:09.*cups-browsed" syslog.4.gz
52683
fmsv030@eichhorn:/var/log$ sudo zgrep -c "Oct 16 19:17:10.*cups-browsed" syslog.4.gz
51595
fmsv030@eichhorn:/var/log$ sudo zgrep -c "Oct 16 19:17:11.*cups-browsed" syslog.4.gz
54502

All these messages read Max number of retries (5) for creating print queue - interesting is, that on various computers within the same network, the number of printers varies. One computer showed this tight loop of error-messages only for 6 printers, another computer showed the error-messages for 17 printers (of 24 total). The messages started shortly after the cups-server lost its network-connection intermittently ("link-flapping"), the messages ended only on logrotate shortly after midnight. On some computers, logrotate was still able to compress the huge syslog, on others - not.


WrapperException ctor now takes const ref.

The previous declaration was non-idiomatic C++.

Thanks for picking this one up @gurry!


⚠ Bofrost.ch has degraded performance

In 0ba0479, Bofrost.ch (https://www.bofrost.ch) experienced degraded performance:

HTTP code: 200
Response time: 1652 ms

Resolved: Bofrost.ch performance has improved in 4996c76 after 6 minutes.


DB::Exception: Bad get: has UInt64, requested String.

When I set up the distributed tables, there was such a mistake.
`node01 :) create table dummy (p Date, k UInt64, d String) ENGINE = MergeTree(p, k, 8192)
CREATE TABLE dummy
(
p Date,
k UInt64,
d String
)
ENGINE = MergeTree(p, k, 8192)
Ok.
0 rows in set. Elapsed: 0.020 sec.
node01 :) create table distributed (p Date, k UInt64, d String) ENGINE = Distributed(cluster-1, 'default', 'dummy')
CREATE TABLE distributed
(
p Date,
k UInt64,
d String
)
ENGINE = Distributed(cluster - 1, 'default', 'dummy')
Received exception from server (version 19.9.5):
Code: 170. DB::Exception: Received from localhost:9000, ::1. DB::Exception: Bad get: has UInt64, requested String.
0 rows in set. Elapsed: 0.001 sec.
`
I don't know why. I tried many times.

Distributed(cluster - 1

-1 ? It should be a cluster name for example test_shard_localhost
ENGINE = Distributed(test_shard_localhost, default, dummy)
check for available clusters select distinct cluster from system.clusters

@JonLeeCSDN
cluster-1 is interpreted as arithmetic expression. You have to put in in backticks: cluster-1 or use different cluster name.

thank you ,in fact ,code as follow will be true:
`node03 :) CREATE TABLE ontime_all AS ontime_local_2 ENGINE = Distributed('cluster-1', 'h2', 'ontime_local_2', 100);
CREATE TABLE ontime_all AS ontime_local_2
ENGINE = Distributed('cluster-1', 'h2', 'ontime_local_2', 100)
Ok.
0 rows in set. Elapsed: 0.005 sec

@JonLeeCSDN
cluster-1 is interpreted as arithmetic expression. You have to put in in backticks:
`cluster-1`

or use different cluster name.

thank you ,in fact ,code as follow will be true:
`node03 :) CREATE TABLE ontime_all AS ontime_local_2 ENGINE = Distributed('cluster-1', 'h2', 'ontime_local_2', 100);
CREATE TABLE ontime_all AS ontime_local_2
ENGINE = Distributed('cluster-1', 'h2', 'ontime_local_2', 100)
Ok.
0 rows in set. Elapsed: 0.005 sec


Consider adding a driver for OTP 21 counters

OTP 21.2 introduced a mutable counters feature which seems like it would be a good fit for this project. It could be implemented as an alternate driver to ets.
Similarly there's also the persistent term feature although I can't think of a scenario where a persistent term would be preferred over a counter.

It would be neat to have some profiling to compare both implementations, if counters are indeed a good fit it's likely that they would be a better default.

Here are some things to consider when implementing this:

The counters module is accessed by a ref (and optionally an array index), while throttle counters need to be referenced by Scope and Key. This means we need some way to map {Scope, Key} a counter ref. The best way to approach this is probably to have persistent terms to access the counters like:

counters:get(persistent_term:get({throttle, Scope, Key}), 1)

throttle also requires some way to periodically access all keys in a given scope to reset the counters back to zero. We would then need some other structure to track all keys in each scope, perhaps a map in a separate persistent_term.

Note it wouldn't work to have a single persistent_term with a map of all counters, since it would be expensive to write to it when a new key is added to the scope.

There's an alternate structure for keeping the counters, the atomics that seems a better fit for the semantics we need in throttle (e.g. it has an atomic add_get method). counters with write_concurrency are supposed to have faster writes, but we likely don't care about that difference.
It's not clear if there's any difference between having many 1-sized counter arrays vs. fewer counter arrays of a bigger size. We probably don't care for now.

On further analysis, using persistent terms would also be problematic:

tracking the keys in a scope inside a persistent term would require to update the term (and trigger a global GC) every time a new key is added
having individual persistent terms for each scope/key combination would mean an ever-growing number of small persistent terms. Thinking in the case of each key representing something like a distinct IP making a request, it wouldn't be desirable to keep old keys lingering forever. A separate mechanism would be necessary to delete old keys and, again, deleting them too frequently would cause expensive GC work.


Invoice HTML as a string

This is a requirement more than an issue. The InvoiceGenerator, generate method should be able to return the rendered html as a string.

Apologies for the delay in responding to this issue, I've been drowning in work over the past few months and progress on addressing my GitHub backlog has been frustratingly slow.
I've now added a generate_string() method to the InvoiceGenerator class that will return the generated HTML as a string.


Current methodology results in failures during cutover run

Here is another method of approach that may help. This person creates a bash script, and that runs once puppet completes it's run. This avoid puppet killing self halfway through it's current run. It also has logic for only running the clean action once.
https://stuckinadoloop.wordpress.com/2012/02/16/automated-migration-of-systems-to-a-new-puppet-master-server/
Sorry if this isn't the right place to post this, I couldn't find a better place for posting this info.

Is there any specific reason you have for wanting it to work that way? To date I haven't really seen many issues with just blowing things away in the middle and causing some failures. The only real issue is that you end up with no report getting submitted to the old master, and I do suppose that would be nice...but since the report wouldn't contain anything about removing the ssldir anyway, I'm not sure how useful it would really be.
Thoughts, @hypnoz?

I'm going to close this for now because I don't really see much benefit to it, but I'm willing to be convinced otherwise.


⚠ Bofrost.at has degraded performance

In 83170d9, Bofrost.at (https://www.bofrost.at) experienced degraded performance:

HTTP code: 200
Response time: 1485 ms

Resolved: Bofrost.at performance has improved in c88c2a9.


error to import decimal number

Prerequisites

[ ] Able to reproduce the behaviour outside of your code, the problem is isolated to Laravel Excel.
[ ] Checked that your issue isn't already filed.
[ ] Checked if no PR was submitted that fixes this problem.

Versions

PHP version: 7.3.0
Laravel version: 5.7
Package version: 3.1

Description
Hi,
i have my excel with decimal number ex: 105,54
but when i import the sheet the number is rounded to: 106
Steps to Reproduce

Expected behavior:

Actual behavior:

Additional Information
Any additional information, configuration or data that might be necessary to reproduce the issue.

Could you please elaborate with e.g. your code?
You might have a look at this as well: Custom Formatting Values
Thanks!

hi,
this is my code:
namespace App\Models;
use PhpOffice\PhpSpreadsheet\Cell\Cell;
use Maatwebsite\Excel\Concerns\ToModel;
use PhpOffice\PhpSpreadsheet\Cell\DataType;
use Maatwebsite\Excel\Concerns\WithCustomValueBinder;
use PhpOffice\PhpSpreadsheet\Cell\DefaultValueBinder;
use App\Models\Tbc_FlussoSapRiga;
class ImportTest extends DefaultValueBinder implements WithCustomValueBinder, ToModel {
 private $count = 0;
 private $id_testata = 0;

 public function __construct($idTestata) {
    $this->id_testata = $idTestata;
}

public function bindValue(Cell $cell, $value) {
    if (is_numeric($value)) {
        $cell->setValueExplicit($value, DataType::TYPE_NUMERIC);

        return true;
    }

    // else return default behavior
    return parent::bindValue($cell, $value);
}

public function model(array $row) {
    if ($this->count > 0) {
        return new Tbc_FlussoSapRiga([
            'id_testata' => $this->id_testata,
            'importo' => $row['18']
        ]);
    }

    $this->count++;
}

but bindValue not firing:
ps: $row['18'] is my number.
Br
max

Hi @maxdiable ,
Might have to do with the decimal separator not being recognized as a number by is_numeric(). Could you try replacing the , with a .?
For your current implementation, there's a bug reported about the bindValue. We'll be looking into this for a future release.

hi, I decided to use the "\PhpOffice\PhpSpreadsheet\IOFactory" library directly.
In fact, using the library directly, I correctly load my decimal number: "105, 57".
I will wait for the new version of your plugin.
br
Max

Thanks for your quick reply @maxdiable
Closing the issue in that case.


Kauri module implementation

kauri implementation first draft

Codecov Report

Merging #89 (7ef9f23) into master (5196107) will decrease coverage by 2.51%.
The diff coverage is 5.29%.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
@@            Coverage Diff             @@
##           master      #89      +/-   ##
==========================================
- Coverage   62.61%   60.11%   -2.51%
==========================================
  Files          73       75       +2
  Lines        7185     7529     +344
==========================================
+ Hits         4499     4526      +27
- Misses       2382     2705     +323
+ Partials      304      298       -6

Flag
Coverage Δ

unittests
60.11% <5.29%> (-2.51%)
:arrow_down:

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

backend/config.go
74.25% <0.00%> (ø)

cmd/hotstuff/main.go
0.00% <ø> (ø)

cmd/plot/main.go
0.00% <ø> (ø)

consensus/byzantine/byzantine.go
1.78% <0.00%> (-0.14%)
:arrow_down:

hotstuff.go
100.00% <ø> (ø)

internal/cli/root.go
24.19% <ø> (ø)

internal/orchestration/worker.go
65.42% <ø> (+0.25%)
:arrow_up:

internal/proto/hotstuffpb/convert.go
98.08% <ø> (ø)

kauri/tree.go
0.00% <0.00%> (ø)

leaderrotation/carousel.go
3.77% <ø> (ø)

... and 22 more

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


tox-poetry is broken with tox>=4.0.0

tox-poetry is does not work with tox>=4.0.0:
python -m pip install -U "tox<4.0.0" tox-poetry  # 3.27.1 is latest 3.x version
tox -h  # works

python -m pip install -U "tox==4.0.0" tox-poetry
tox -h  # broken:
...
ModuleNotFoundError: No module named 'tox.exception'

I can take a look at this :)

This is breaking my env for all my projects . I am dropping tox-poetry

@codectl and it is right decision, check this https://github.com/tkukushkin/tox-poetry/pull/16#issuecomment-1383044770


Call schema in Liquid

{% jsonLdScript meta, type, tags %} is the shortcode for Nunjucs. Which shortcode should be used for LiquidJS instead?

@xplosionmind I don't use Liquid, but looks like it is the same: https://www.11ty.dev/docs/shortcodes/
Commas seem to be optional.


Independence from the starting directory

When post4 binary is started as ./src/post4, it cannot find the file post4.p4:
./src/post4
post4: post4.p4: No such file or directory
ok

But if we are in the src directory, it works:
./post4
ok

Can post4 obtain its binary image location from the OS and try to find the file post4.p4 from this location?

You could do a make install in which case post4 is copied to /usr/local/bin/ and post4.p4 to /usr/local/lib/post4/.
Alternatively you could:
    ./src/port4 -c ./src/post4.p4

or you could export POST4_PATH=..., eg.
    export POST4_PATH="$(pwd)/src"
    src/post4

The built-in default POST4_PATH can be shown with post4 -V; it can be altered at compile time if the default is not ideal.

Closed given the existing methods to specify word definitions file, post4.p4.


feat(auth): add support for device code grant flow

This PR adds support for Twitch's Device code grant flow (DCF). This makes it possible to (effectively) use tokens for much longer without requiring the user to reauthenticate (assuming the scopes didn't change).
Currently, this uses a client-id I created (THIS MUST BE CHANGED BEFORE A MERGE). Because we use the public client type, the client-id is limited to DCF-only (as far as I understand).
(effectively) closes https://github.com/Chatterino/chatterino2/discussions/5169.
Here's a cool video I took in February when I started this (hasn't changed much):
https://github.com/Chatterino/chatterino2/assets/19953266/259ba236-3f02-4375-9ef9-c5cc455ebdd9

Been running this for a few days and seems solid 👍

Should we validate that stored tokens have all of DEVICE_AUTH_SCOPES? (or perhaps should be tackled in a separate PR)

Should we validate that stored tokens have all of DEVICE_AUTH_SCOPES? (or perhaps should be tackled in a separate PR)

Twitch does that when we refresh tokens (the last time I tested it). The error you get there is pretty bad, though (iirc, it's the same one you get when the token is invalid).


Cannot find module 'app'

I run electron . on mac, seems neovim-e don't want to talk to me, and give me an exception.
Who could help me?
~/repos/neovim-e $ electron .
App threw an error during load
Error: Cannot find module 'app'
    at Module._resolveFilename (module.js:440:15)
    at Function.Module._resolveFilename (/usr/local/lib/node_modules/electron/dist/Electron.app/Contents/Resources/electron.asar/common/reset-search-paths.js:35:12)
    at Function.Module._load (module.js:388:25)
    at Module.require (module.js:468:17)
    at require (internal/module.js:20:19)
    at Object.<anonymous> (/Users/pysnow530/repos/neovim-e/src/launcher.js:4:9)
    at Object.<anonymous> (/Users/pysnow530/repos/neovim-e/src/launcher.js:34:4)
    at Module._compile (module.js:541:32)
    at Object.Module._extensions..js (module.js:550:10)
    at Module.load (module.js:458:32)

After run npm install app, the error changed as follow:
App threw an error during load
TypeError: core.require is not a function
    at Object.<anonymous> (/Users/pysnow530/repos/neovim-e/node_modules/app/lib/App.js:10:18)
    at Object.<anonymous> (/Users/pysnow530/repos/neovim-e/node_modules/app/lib/App.js:356:4)
    at Module._compile (module.js:541:32)
    at Object.Module._extensions..js (module.js:550:10)
    at Module.load (module.js:458:32)
    at tryModuleLoad (module.js:417:12)
    at Function.Module._load (module.js:409:3)
    at Module.require (module.js:468:17)
    at require (internal/module.js:20:19)
    at Object.<anonymous> (/Users/pysnow530/repos/neovim-e/node_modules/app/lib/index.js:3:29)

launcher.coffee:
electron = require 'electron'
app = electron.app
BrowserWindow = electron.BrowserWindow

grunt && electron .

+1 (exactly the same message).
krzysztofantczak solution didn't work for me

@cgarciaq i think it's just more errors, right? different ones, those need to be fixed as well. I had to fix like (afair) 7 files in total to make that work, but it works ;-)

Hmm... I'm not sure, but the error seems very similar, except for some line numbers.
A JavaScript error occurred in the main process
Uncaught Exception:
Error: Cannot find module 'app'
at Module._resolveFilename (module.js:455:15)
at Function.Module._resolveFilename (/usr/local/lib/node_modules/electron/dist/resources/electron.asar/common/reset-search-paths.js:35:12)
at Function.Module._load (module.js:403:25)
at Module.require (module.js:483:17)
at require (internal/module.js:20:19)
at Object. (/home/carlos/neovim-e/src/launcher.js:4:9)
at Object. (/home/carlos/neovim-e/src/launcher.js:34:4)
at Module._compile (module.js:556:32)
at Object.Module._extensions..js (module.js:565:10)
at Module.load (module.js:473:32)

--> uname -a
Linux xxxxx 4.4.0-38-generic #57-Ubuntu SMP Tue Sep 6 15:42:33 UTC 2016 x86_64 x86_64 x86_64 GNU/Linux
--> nvim -v
NVIM 0.1.6-dev
Build type: RelWithDebInfo
Compilation: /usr/bin/x86_64-linux-gnu-gcc -g -O2 -fstack-protector-strong -Wformat -Werror=format-security -Wconversion -U_FORTIFY_SOURCE -D_FORTIFY_SOURCE=1 -O2 -g -DDISABLE_LOG -Wall -Wextra -pedantic -Wno-unused-parameter -Wstrict-prototypes -std=gnu99 -Wvla -fstack-protector-strong -fdiagnostics-color=auto -DINCLUDE_GENERATED_DECLARATIONS -DHAVE_CONFIG_H -D_GNU_SOURCE -I/build/neovim-TW_bde/neovim-0.1.5ubuntu1+git201610061357+2804+21~ubuntu16.04.1/build/config -I/build/neovim-TW_bde/neovim-0.1.5ubuntu1+git201610061357+2804+21~ubuntu16.04.1/src -I/build/neovim-TW_bde/neovim-0.1.5ubuntu1+git201610061357+2804+21~ubuntu16.04.1/.deps/usr/include -I/build/neovim-TW_bde/neovim-0.1.5ubuntu1+git201610061357+2804+21~ubuntu16.04.1/.deps/usr/include -I/build/neovim-TW_bde/neovim-0.1.5ubuntu1+git201610061357+2804+21~ubuntu16.04.1/.deps/usr/include -I/build/neovim-TW_bde/neovim-0.1.5ubuntu1+git201610061357+2804+21~ubuntu16.04.1/.deps/usr/include -I/build/neovim-TW_bde/neovim-0.1.5ubuntu1+git201610061357+2804+21~ubuntu16.04.1/.deps/usr/include -I/build/neovim-TW_bde/neovim-0.1.5ubuntu1+git201610061357+2804+21~ubuntu16.04.1/.deps/usr/include -I/usr/include -I/build/neovim-TW_bde/neovim-0.1.5ubuntu1+git201610061357+2804+21~ubuntu16.04.1/build/src/nvim/auto -I/build/neovim-TW_bde/neovim-0.1.5ubuntu1+git201610061357+2804+21~ubuntu16.04.1/build/include
Compilado por buildd@lcy01-01
Optional features included (+) or not (-): +acl   +iconv    +jemalloc +tui
For differences from Vim, see :help vim-differences

@krzysztofantczak solution didn't work for me too, this question is still opened.

@krzysztofantczak
After I modify some coffee, all is ok now, thank you very much!


Permission denied opening logfile

I have attempted to install prometheus-postfix-exporter as an ubuntu package but I cannot get it to work on 20.04 (permission denied opening the logfile), whereas it does work on another server 22.04 which seems to have the same config.
The documentation doesn't say anything about permissions anywhere but the service is installed as user prometheus which doesn't appear to have any special permissions or be added into any groups. For this reason, the permission denied problem makes sense to me and might simply point to needing some docs to say, "by the way, you have to setup relevant permissions etc." except for the fact that the other server works without these steps!
The docs say that this exporter will truncate the incoming log file which presumably means that the prometheus user needs to have write permissions to the log file. However, this doesn't actually seem to happen, I ran the exporter on the CLI as root and it ran up fine but didn't truncate the logfile (the metrics were all shown on port 9054 as expected)
So as you can see, I have got very confused. Since it does work as root, maybe it is just a permissions issue and maybe it is not actually supposed to truncate the file any more? I got the ubuntu instructions here: https://manpages.ubuntu.com/manpages/focal/man1/prometheus-postfix-exporter.1.html
Thanks

OK, so I found a bug/feature in rsyslog which makes it all confusing! In Ubuntu, rsyslog runs as syslog:syslog by default (not root:root as per Debian), this means that not all of the permission features that you are supposed to have in syslog configurations work as expected.
For example, I tried creating the second exporter log file as prometheus:prometheus so it could be read (and truncated) by the exporter but then realised this didn't work because syslog does not have permission to chown a file owned by someone else. I also noticed that the log file that this package is supposed to configure (under /var/log/prometheus/postfix-exporter) also wasn't create for similar reasons.
Eventually, the only way I could make it work was to let syslog create the files/create them manually and then set their permissions using sudo. I set the second log file to be syslog:prometheus so that syslog can write it and prometheus can read/truncate it. I then set the mail.log under /var/log/prometheus/postfix-exporter to prometheus:adm, which should have had a similar effect.
Anyway, the result is that data is logged to the second logfile that is recommended (since it gets truncated) but it is STILL not truncating after the metrics are read. I'm not sure whether the whole file should be consumed/truncated (as I am expecting) or whether it is more intelligent and will leave some data and delete the rest, which might be what is happening but this is still really hard to set up.
Ubuntu 22.04 btw.


chore(artifacts/gitRepo): git repo artifact

@brian-armory @dorbin a couple of questions:

This feature is only available in 1.17 and I'd like to put a note at the top indicating that. Do we have a style we like for things like that?
For the Artifacts Rewrite version of this doc the table of contents makes the top section look a bit off. It pushes the table down below it. Do you have any suggestions for this? It looks like it only works for other pages because there are fewer sections.

For 1) I haven't noticed a standard style. (A style guide is one of the things the Docs sig wants to come up with.)
I recommend inserting

  Note: This feature is only available in version 1.17 or later.

That'll put it in a box.
You can also use the {: .notice--info} syntax instead.
https://mmistakes.github.io/minimal-mistakes/docs/utility-classes/#notices for more info.

For 1), I had a similar concern for our external configuration page in "Setup". We used a simple notice on that page (we had to explain that the feature came to different Spinnaker services in different versions):
> For Clouddriver, support for external configuration was added in Spinnaker 1.15. For Echo and Igor, support was added in Spinnaker 1.16.

For 2), yes @ethanfrogers I think the TOC only works on other pages because they have fewer sections. I’m not sure of a way around that. I probably would just put up with it (or do something drastic like restructure the content of the page to have more text up front or use fewer subheadings) for now, unless @brian-armory has found anything else we can do using the theme to get around it.

Actually, I think where I saw it before was with an image. In this case you've got a table. See what happens if you refactor that table as Markdown?

Thanks for the input everyone, that was really helpful. @dorbin, I converted the table to Markdown but it didn't help the TOC issue, which, makes sense, since MD is converted to HTML. I ended up changing the headings that were causing the problem to bold instead of actual headings. It isn't exactly consistent with other pages but seemed worth the trade off to get formatting right.


Style: Async method nomenclature, missing ConfigureAwait and CancellationTokens

Fixes https://github.com/microsoft/botbuilder-dotnet/issues/3889

Thanks @johnataylor ! Code analisys detects ConfigureAwait and async names but not cancellation tokens so it's more manual. Appreciate the detailed review, new iteration incoming

    public async Task<T> LoadTypeAsync<T>(Resource resource)

no cancelation token?

Refers to: libraries/Microsoft.Bot.Builder.Dialogs.Declarative/Resources/ResourceExplorer.cs:155 in 3daaf46. [](commit_id = 3daaf46b902e0171778a3f57f3a35e9219651912, deletion_comment = False)


Nuget package restore failing

I am getting the below error when deploying azure web site from local git repository. I was under the impression that with the nuget's new restore flow downloading of missing packages should be automatically handled. Also some of the packages were installed successfully. The below error showed up twice in the log.
This project references NuGet package(s) that are missing on this computer. Enable NuGet Package Restore to download them.  For more information, see http://go.microsoft.com/fwlink/?LinkID=317567.
Failed exitCode=1, command="D:\Windows\Microsoft.NET\Framework\v4.0.30319\MSBuild.exe" "D:\home\site\repository\Some.API\Some.API.csproj" /nologo /verbosity:m /t:Build /t:pipelinePreDeployCopyAllFilesToOneFolder /p:_PackageTempDir="D:\local\Temp\da975b77-8365-46bf-9394-fa5bf6eca016";AutoParameterizationWebConfigConnectionStrings=false;Configuration=Release /p:SolutionDir="D:\home\site\repository.\"
An error has occurred during web site deployment.
d:\Program Files (x86)\SiteExtensions\Kudu\44.40324.1452\bin\scripts\starter.cmd "D:\home\site\deployments\tools\deploy.cmd"

Kudu does do NuGet package restore by default, and this normally works fine. Could you share a test repo that demonstrates what you are seeing? Thanks!

Hi David,
The project is at https://github.com/freetrialbeforebuying/ARIA.API
One thing I noticed is that with my existing project I already had the "packages" folder and it was building fine but I got the error (posted in my issue description) when trying to deploy to azure. When I cloned a fresh project (i.e. without the "packages" folder) visual studio build gave the same error. I am not sure what am I missing ? Shouldn't the new Microsoft.Bcl.Build and nuget 2.7+ take care of restoring the packages automatically.
Thanks

I was not able to access your project (404). In any event, if you see the same issue locally with a clean clone, we at least know it is not Azure/Kudu related!

Sorry I had to delete the repo because connections strings slipped in. It should be up now. Thanks.

 :( In any case, since we established that it is not Kudu related, it becomes more of a NuGet/msbuild issue, and there are better places to discuss that.

thats silly of me.. completely forgot the history. And you are right there are better places to discuss.. but i thought since you are the nuget expert it would be helpful to get some insights. In any case thank you for making azure web sites deployment so easy with project kudu. Its fantastic !

Thanks for the feedback!
While it's not likely that someone had time to catch your connection string, if the data is very sensitive, I would suggest regenerating the key, to be on the safe side.


react: key warnings

I seem to be getting a lot of key warnings when using hyperx with react. Here is an example that generates a key warning but shouldn't really:
const React = require('react')
const hyperx = require('hyperx')
const hx = hyperx(React.createElement)

hx`<div><a href="#">link</a></div>`

If I add a key to the anchor tag the warning goes away - but I shouldn't really need to do that.
Any solutions to this?

I've done a bit more research on this. When you have this JSX:

link

it will output this JS:
  return React.createElement(
    "div",
    null,
    React.createElement(
      "a",
      { href: "#" },
      "link"
    )
  );

The example I gave above:
hx`<div><a href="#">link</a></div>`

Produces this JS:
  return React.createElement(
    "div",
    null,
    [
      React.createElement(
        "a",
        { href: "#" },
        "link"
      )
    ]
  );

Note the last parameter is an array rather than an element. React.createElement can take either an array of elements as the last parameter or a spread of elements. When an array is supplied it warns that you should add key properties to uniquely identify them.
Hyperx seems to create arrays rather than spreading the children which is why we get key warnings.

Any ideas on this? Is anyone using hyperx without all these React warnings?

function createElement(Component, props, children) {
    return React.createElement(Component, props, ...children);
}

const hx = hyperx(createElement);

i think @s9k's solution is the best way to go. documentation patch welcome!


Convert to menuinst v2

{package} {version} {:snowflake:}
Destination channel: {Snowflake | defaults}
Links

{ticket_number}
Upstream repository
Upstream changelog/diff
Relevant dependency PRs:

...

Explanation of changes:

...

Linter check found the following problems:
ERROR conda.cli.main_run:execute(125): `conda run conda-lint /tmp/abs_31yaae27cw/clone` failed. (See above for error)
The following problems have been found:
===== WARNINGS =====

clone/recipe/meta.yaml:25: host_section_needs_exact_pinnings: Linked libraries host should have exact version pinnings.

===== ERRORS =====

clone/recipe/meta.yaml:74: invalid_url: https://jupyter-notebook.readthedocs.io : Not reachable: 403
===== Final Report: =====
1 Error and 1 Warning were found

Linter check found the following problems:
ERROR conda.cli.main_run:execute(125): `conda run conda-lint /tmp/abs_ba411ecx4n/clone` failed. (See above for error)
The following problems have been found:
===== WARNINGS =====

clone/recipe/meta.yaml:25: host_section_needs_exact_pinnings: Linked libraries host should have exact version pinnings.

===== ERRORS =====

clone/recipe/meta.yaml:74: invalid_url: https://jupyter-notebook.readthedocs.io : Not reachable: 403
===== Final Report: =====
1 Error and 1 Warning were found

Linter check found the following problems:
ERROR conda.cli.main_run:execute(125): `conda run conda-lint /tmp/abs_15osra88ja/clone` failed. (See above for error)
The following problems have been found:
===== WARNINGS =====

clone/recipe/meta.yaml:25: host_section_needs_exact_pinnings: Linked libraries host should have exact version pinnings.

===== ERRORS =====

clone/recipe/meta.yaml:74: invalid_url: https://jupyter-notebook.readthedocs.io : Not reachable: 403
===== Final Report: =====
1 Error and 1 Warning were found

Linter check found the following problems:
ERROR conda.cli.main_run:execute(125): `conda run conda-lint /tmp/abs_fdmkts4g2t/clone` failed. (See above for error)
The following problems have been found:
===== WARNINGS =====

clone/recipe/meta.yaml:26: host_section_needs_exact_pinnings: Linked libraries host should have exact version pinnings.
clone/recipe/meta.yaml:75: invalid_url: https://jupyter-notebook.readthedocs.io : Not reachable: 403
===== Final Report: =====
0 Errors and 2 Warnings were found

Linter check found the following problems:
ERROR conda.cli.main_run:execute(125): `conda run conda-lint /tmp/abs_1ft7jm6wab/clone` failed. (See above for error)
The following problems have been found:
===== WARNINGS =====

clone/recipe/meta.yaml:75: invalid_url: https://jupyter-notebook.readthedocs.io : Not reachable: 403
clone/recipe/meta.yaml:26: host_section_needs_exact_pinnings: Linked libraries host should have exact version pinnings.
===== Final Report: =====
0 Errors and 2 Warnings were found


Update manifest.json

Thank you for the excellent integration!
This fixes a warning within HA:
No 'version' key in the manifest file for custom integration 'shairport_sync'. As of Home Assistant 2021.6, this integration will no longer be loaded. Please report this to the maintainer of 'shairport_sync'

Thank you, on both accounts!


Auto-generate the VHDL code that connects layers

The title already tells! But I can try a better template for the network layer by passing parameters layers can be assembled.

This task could be very difficult for me. I will try and prepare materials for the auto-generation. Here is some stuff I have done:

The hand-written network layer with documentation: https://github.com/SuperChange001/HWLib/tree/master/source/fp_network
its corresponding testbench:  https://github.com/SuperChange001/HWLib/blob/master/testbench/fp_network_tb.vhd

[ ] The next step, understand how the code generator works in the creator and try to generate the code block for instantiating components, in my case, two linear layers and two activation functions.
The generated code should look like this:

--------------------------------------------------------------------------------
    -- Instantiate all layers
    --------------------------------------------------------------------------------
    i_fp_linear_1d_0 : entity work.fp_linear_1d_0(rtl)
    port map(
        enable => i_fp_linear_1d_0_enable,
        clock  => i_fp_linear_1d_0_clock,
        x_addr => i_fp_linear_1d_0_x_addr,
        y_addr => i_fp_linear_1d_0_y_addr,

        x_in   => i_fp_linear_1d_0_x_in,
        y_out  => i_fp_linear_1d_0_y_out,

        done   => i_fp_linear_1d_0_done
    );

    i_fp_linear_1d_1 : entity work.fp_linear_1d_1(rtl)
    port map(
        enable => i_fp_linear_1d_1_enable,
        clock  => i_fp_linear_1d_1_clock,
        x_addr => i_fp_linear_1d_1_x_addr,
        y_addr => i_fp_linear_1d_1_y_addr,

        x_in   => i_fp_linear_1d_1_x_in,
        y_out  => i_fp_linear_1d_1_y_out,

        done   => i_fp_linear_1d_1_done
    );

    i_fp_hard_sigmoid_2 : entity work.fp_hard_sigmoid_2(rtl)
    port map(
        enable => i_fp_hard_sigmoid_2_enable,
        clock  => i_fp_hard_sigmoid_2_clock,
        input  => i_fp_hard_sigmoid_2_input,
        output => i_fp_hard_sigmoid_2_output
    );

    i_fp_relu_3 : entity work.fp_relu_3(rtl)
    port map(
        enable => i_fp_relu_3_enable,
        clock  => i_fp_relu_3_clock,
        input  => i_fp_relu_3_input,
        output => i_fp_relu_3_output
    );
    --------------------------------------------------------------------------------
    -- Instantiate all layers
    --------------------------------------------------------------------------------

[ ] - generate instantiation
      (<node_id> : entity work.<module_id>         port map(           <signal_id_0> => <node_id>_<signal_id_0>,           ...           <signal_id_n> => <node_id>_<signal_id_n>         );
for each node in the network.


fix: better display received errors in DisplayStream

Signed-off-by: Thomas Soëte<EMAIL_ADDRESS>

Merged. Thanks.


command pve_exporter not working

Hi,
i was just trying to install everything as it looked great but i am unable to find the config file.
The command: pve_export config troughs me an Error. pve_exporter -h works.
Details on the attached picture.

Thanks ahead for your help.

Please note that [config], [port] and [address] are positional arguments. Specify a path to the pve.yml configuration file as the first positional argument, the other ones are optional. You can find an example for pve.yml in the repository and also in the wiki. Place that example, e.g., in /etc/pve.yml. Then start pve_exporter like this:
pve_exporter /etc/pve.yml

Note that there is an example for a systemd unit in the wiki as well.
HTH

Ok thanks that worked but i ran into a different error.
403 Forbidden: Permission check failed (/, Sys.Audit) - {"data":null}
I am getting this Error when trying to access (my) http://localhost:9221/pve?target=<IP_ADDRESS>
Couldn't find anything about this Error.

Where did you install pve_exporter? On a proxmox host or on another machine?
The URL in the README is an example. Replace localhost with the hostname or IP address where pve_exporter is running and <IP_ADDRESS> with the hostname or IP address of your Proxmox PVE host.

Thanks a had a line double in the configuration file in prometheus.


The collision shape problem of concave object

I have got the urdf model of a hole from solidworks, and also a mesh file of the hole. But when i try to assemble the peg into the hole. I found the collision shape of the hole is cylinder which is solid not contain the hole of the cylinder.
Could you tell me how to confige this stl model or use which of software to config this property
this is that model stl
hole_f.STL.tar.gz

Hi,
This is a problem with the underlying simulator pybullet. When you load a mesh through the World or Simulator class, you have a flag that you can set. Here is the documentation:

@bdelhaisse
Thanks for your answer, I try to use blender with a plugin could construc the model collision model (.obj), but it cannot perform the shape of the hole precisely. I also ask the same question on bullet3 package https://github.com/bulletphysics/bullet3/issues/2531, but i couldn't found those parameters in your load_urdf() function, just in load_mesh() that sim.GEOM_FORCE_CONCAVE_TRIMESH is ok.

@bdelhaisse
And I have another question on the coliision problem.
When the robot (i used UR5) collide with the hole or something else. I will be in chaos, it is out of control. Is that maybe something wrong with the arm joint damping parameters is too small. Sould I change some parameters of the joint or something else?

@bdelhaisse Thanks for your answer, I try to use blender with a plugin could construc the model collision model (.obj), but it cannot perform the shape of the hole precisely. I also ask the same question on bullet3 package https://github.com/bulletphysics/bullet3/issues/2531, but i couldn't found those parameters in your load_urdf() function, just in load_mesh() that sim.GEOM_FORCE_CONCAVE_TRIMESH is ok.
I have the same question，can you solve the question?


After signin in to AAD - Exception: Correlation failed.

Hi,
I've followed the sample, when I run the IIS on localhost and click Sign In then go through the authentication process I end up with:
An unhandled exception occurred while processing the request.
Exception: Correlation failed.
Unknown location
Exception: An error was encountered while handling the remote login.
Microsoft.AspNetCore.Authentication.RemoteAuthenticationHandler.HandleRequestAsync()

Thanks @maszeq for the report! I'll migrate to .NET Core 3.1 and refactor this sample in about two weeks time and will try to repro this issue as well. 🙂

Hia @maszeq the migration has been completed, the authentication libraries upgraded. I wasn't able to repro the aforementioned issue. Can you please re-check and in case the problem still persist, provide mw with more information (what kind of account did you use to log in, etc)?


Added goal Ids for type of business & other type of business screens

CI: No security warnings found


Swapping with offhand causes pickup notification

Title is self-explanatory.

Ah always more edge cases that I have to hard code for. Thanks for reporting, will get to it either tonight or tomorrow

Turns out this isn't really solvable without having a server-side component. This is fixed now in the latest version (1.1.0) but requires installing the mod on the server for multiplayer.
https://modrinth.com/mod/roundaround-pickup-notifications/version/1.1.0


Use Binstar to automatically build condo packages

Originally reported by: Yung-Yu Chen (BitBucket: yungyuc, GitHub: yungyuc)

See http://docs.binstar.org/examples.html#BinstarBuild

Bitbucket: https://bitbucket.org/solvcon/solvcon/issue/117

This is no longer valid; even the link itself is dead.


virtiofs: fix virtiofs crash when cache=none

When virtio_fs_cache is set to none, the mount options for the folder
inside the guest should not contain the dax option else it leads to
invalid address errors and a crash of the daemon on the host.
Fixes: #1907
Signed-off-by: Ganesh Maharaj Mahalingam<EMAIL_ADDRESS>

/test

@jcvenegas @devimc the initrd fails is the one a revert is going to fix, right?

metrics CI seems to be failing with below error.
08:06:18 Error: Error downloading packages:
08:06:18   Curl error (16): Error in the HTTP2 framing layer for https://mirrors.fedoraproject.org/metalink?repo=updates-released-f30&arch=x86_64 []
08:06:18 The command '/bin/sh -c dnf install -y qemu-img parted gdisk e2fsprogs gcc xfsprogs findutils' returned a non-zero code: 1

@chavafg any chance you have seen this in the recent past?

not really, maybe a network issue in the infra? I sent a restart, hopefully it doesn't fail this time


bug: UNSTABLE_shimOnConnectSelectAccount is true and metamask is already open not rejected error

Is there an existing issue for this?

[X] I have searched the existing issues

Current Behavior
i want to error handling connect when already metamask opened.
but when UNSTABLE_shimOnConnectSelectAccount is true no ResourceUnavailable  error throw
i found when UNSTABLE_shimOnConnectSelectAccount is true, only throw UserRejectedRequestError.
why not throw ResourceUnavailable error ?
my code and metamask file code image below

async function connectWallet() {
    try {
      const connectResult = await connectAsync()

      return Promise.resolve(connectResult)
    } catch (e) {

      if (e.name === 'ResourceUnavailable') {
        return
      }

      if (e.name === 'UserRejectedRequestError') {
        // TODO 예외사항처리필요
        return
      }

    }
  }

Expected Behavior
when UNSTABLE_shimOnConnectSelectAccount true and metamask already open throw error
Steps To Reproduce
No response
Link to Minimal Reproducible Example (CodeSandbox, StackBlitz, etc.)
No response
Anything else?
No response

I am unsure what you mean here. We throw a UserRejectedRequestError when a user rejects the connection request. Why would we throw a ResourceUnavailable error? Can you please provide some more details here.

I want to show the user an error saying metamask is already open when metamask is already open.
In Metamask, in that case, the error code is -32002 and the name is ResourceUnavailable.
However, wagmi only throws errors for UserRejectedRequestError,  I can't notify the user of an error when metamask is already on.
So I want do not throw only UserRejectedRequestError, if metamask is already turned on and the error occur , that error throw too @jxom

@tmm thanks!


Ansible GRPC playbook + other ansible tweaks

This adds a grpc playbook to checkout and build grpc from source.
You can override the default repo by setting the grpc_repo variable, and the tag / commit to checkout with grpc_checkout.  Technically you could put a branch name for grpc_checkout, but that will mess with the idempotency check.  When the playbook installs grpc it drops a file at /opt/grpc/.grpc_version_installed{{ grpc_checkout }} to avoid rebuilding after it's already installed.  So if you use a branch name, it won't trigger a rebuild if the branch tip changes.
Also does a few other things:

adds the -sharedDb flag to the local dynamo db service, and fixes the idempotency check on downloading the dynamo jar
moves the apt upgrade task from the transactor playbook to transactor-provisioning.yml b/c it was messing up my vmware box when the kernel got updated
adds some missing dependencies to transactor-client-provisioning.yml (libjpeg-dev and libpng-dev)
adds the new grpc role to dev.yml and transactor-client-provisioning.yml, pinned to the latest commit on master -- be223358795b645416e801727f4d3fcad3d73964

It would be nice if the grpc role could optionally install to a virtualenv, but I didn't want to get carried away :)
Also, the grpc playbook does pretty much everything as root, since it's set to checkout the code to /opt/grpc by default and needs write permissions.  I'd like to just check it out to the home dir instead, but the username varies between vagrant and the deployment env, and I don't know how to abstract that in ansible.  I guess I could add a current_user variable to the playbook, but it seems like that's probably built in to ansible and I just haven't found it.

Can you move the system upgrade in a role that is used by both transactor-provisioning and transactor-client-priviosioning? As it stands right now the client-provisioning playbook is missing the system update!

:+1:


[tvp] Add support for TVP Wiadomości

This is the main state-funded daily news programme in Poland.

Instead of separating extractor just expand _VALID_URL of TvpIE.

@dstftw Thanks for taking a look. I did try that at first but could not come up with one expression to handle both the wiadomosci.tvp.pl URLs and the other ones. Notice that in the former the video ID comes at first, in the latter it comes at the end:
http://wiadomosci.tvp.pl/25169746/24052016-1200
http://www.tvp.pl/there-can-be-anything-so-i-shortened-it/17916176

If you have an idea, it'd be appreciated.

Thanks! :-)


Start Process

Great project you have here! Thanks for your work.
Do you have any plans to add a hook for pm2.start?

@lmangani I'm working on building a better tool on top of pm2-webui with webhooks, auto-deployment and more. Hopefully I'll try to implement that feature there.

@suryamodulus let us know where and we'll join & contribute!


Feature - WebVTT to SRT

Add new function - (void)loadWebVTTSubtitlesAtURL:(NSURL *)url error:(NSError **)error, so ASBPlayerSubtitling.h accepts WebVTT.
Re-install pod to make project compilable.

Hi @autresphere I wonder if this PR is ok now?
I removed the part of PR that change the original API, it should be ok now.

Hi @allenlinli, this looks really good, thanks a lot!
There is now an issue with Travis CI, I guess the test command is too old and should be updated. Would you be kind enough to update it? I don't have time to have a look at it before several weeks.

I probably cannot fix Travis CI when I on work, I want to have a look when I am off work.
I have no experience with Travis CI. Where can I edit the Travis CI shell script?


Allow not having seperate subdirectory for each contest kind and group

In the latest update, cf parse creates the following directory structure $currdirectory/cf/contest/<contest id>/<problem code>. I liked the previous format much better, where the problem would be parsed to $currdirectory/<contest id>/<problem code>.
The new update may be convenient to some, but I prefer you gave an option to omit the sub-directories in the corresponding cf config option. Maybe something like, Enter ~ if you prefer to not have this subdirectory.

Well, this is an embarrassing problem. cf needs directory to detect where the user is. So I made that decision what you described.
But you can enter /cf/contest/ at first. Then cf parse contest 1111 can make you convenient. Because It will only create a 1111/ under the /cf/contest/.

I have to agree with @infixint943 on this one. I don't think that forcing a directory is a good solution. @xalanq maybe you can explain why is the extra folder needed and maybe we could come up with a workaround?

idea: it's possible to allow the user to configure this. For example
"contest_directory": "cf/contest/%d/%d",
"group_directory": "cf/group/%d/%d/%d"

If the user only use cf submit, cf needs to get the extra information from the current path. And cf needs to know what's the type of the current contest (gym, contests, acmsguru or groups? E.g. if the contest id is 100001, it could be a gym contest or group contest.). So I create a workspace cf/ which contains different types of the contests.
The reason is I don't want to create some extra files in any folder to help cf to detect that extra information.

idea: it's possible to allow the user to configure this. For example
"contest_directory": "cf/contest/%d/%d",
"group_directory": "cf/group/%d/%d/%d"

Update: by reading the config file, it appears that the folder name is configurable (is this documented anywhere?), but is that still requires that there are multiple folders instead of encoding it in the folder name.

Contests' rounds are always in the folder {cf}/{contest}, gym's rounds are always in the folder {cf}/{gym}, groups' rounds are always in the folder {cf}/{group}. So you can configure the folders' name {cf}, {contest}, {gym}, {group} by cf config

You can read the source code client/info.go and cmd/args.go to understand why I did that.

Maybe folder prefix could specify the type?
Example folder names:
1234 -> by default to contest 1234
gym123456 -> gym with id 123456
groupXXXXXXXXXX/123456 -> group XXXXXXXXXX,  contest id 123456
acmsguru123 (or acm123) -> acmguru problem 123
(We could probably strip out gym too, as it has 6 digits and groups/acmguru are different). User could also modify these prefixes.
This way we avoid extra folders.

Maybe folder prefix could specify the type?
Example folder names:
1234 -> by default to contest 1234
gym123456 -> gym with id 123456
groupXXXXXXXXXX/123456 -> group XXXXXXXXXX, contest id 123456
acmsguru123 (or acm123) -> acmguru problem 123
(We could probably strip out gym too, as it has 6 digits and groups/acmguru are different). User could also modify these prefixes.
This way we avoid extra folders.

That idea is not bad. But I prefer the workspace haha.
Maybe we can add an option to switch 2 different methods. You can try it and pull a request :)

I'll implement it. Just make it more general.
(I hope the configuration does not become a mini configuration language...)

Suppose your folder structure to be:

parent

cf

group
gym
1302

As a temporary workaround, you can use
"folder_name": {
    "acmsguru": "cf/acmsgurfu",
    "contest": "cf",
    "group": "cf/group",
    "gym": "cf/gym",
    "root": "parent"
  }


Moon connectivity depends on ~zod?

On my moon, whenever I get ~zod not responding still trying, it is almost always followed by ~planet not responding still trying.
If I try |hi from my moon to my planet, or the other way around, it won't go through. On my planet, doing so prompts a ~moon not responding still trying message to print.
Once I get a ~zod is ok, the |hi goes through.
This makes it seem like moons proxy their traffic through ~zod, which is definitely not something we want.

Closing this one as it hasn't gotten any follow-up in over a year.

We don't exactly proxy anything in ames, but we do forward packets in order to (re-)discover peers. And we do a lot of that, because we expire routes pretty quickly. Maybe a moon's route to its planet should not expire as quickly as other routes?
These issues/questions fall under ames tuning and optimization, which has not been attempted.
/cc @belisarius222


add idea testSourceDirs for integration tests

Coverage remained the same at 99.559% when pulling 53d88737f3c96a63c962aade8462e66285cb1864 on jmcampanini:jc/idea-source-sets into 17fa57aad642243d4838c35fe178ba88d377cd46 on srs:master.


Feature: Ability to render the editor in an iframe

I'm trying to render the editor inside an iframe (via a react portal), but it's not working and I suspect this is due to the fact that it's referencing the global window object for selection state, etc. I'd like to be able to pass a custom window object (the iframe window)

One solution is to provide the window object in a react context. It can be passed in via an optional prop to the RichTextPlugin or PlainTextPlugin (or maybe even LexicalComposer) and would default to window.
Alternatively, Slate reads the window from the contentEditable DOM node. So if you have a ref to that DOM node, you can look up it's parent window with something like...
contentEditableNode.current.ownerDocument.defaultView

We can expose this as a helper and replace all references to the global window with the helper like Slate...
https://github.com/ianstormtaylor/slate/blob/main/packages/slate-react/src/plugin/react-editor.ts#L54-L64

I think we can extract it from the contenteditable like Slate does.

this would be useful

Thanks for reporting this. Fixed in #2918 :)

I'm trying to get this working but can't seem to get it to work when it's in the iframe.
I have it reproduced here: https://codesandbox.io/s/aged-frog-iu8ff7
It works when outside of the iframe but not when inside it.
Is there something obvious that I'm doing wrong?

@trueadm can we reopen this? I'm facing the same issue as @darabun.
Interestingly, I can almost make it work if I provide an initial editor state with some text content like this: https://codesandbox.io/s/modern-wave-1ehl2o?file=/src/Lexical.js
But you'll notice the Enter/Return character doesn't work in the iframe, but works outside it. It seems like there are some event listeners missing in the iframe.

@darabun See https://github.com/facebook/lexical/issues/3534

Let me know if this PR helps: https://github.com/facebook/lexical/pull/3573

Definitely an improvement, but there are still selection issues. It seems like we're getting the proper selection part of the time now. Try inserting a line break in the iframe here...
https://codesandbox.io/s/sweet-fire-e9ry1d

How about https://github.com/facebook/lexical/pull/3578?

It works! Thanks @trueadm https://codesandbox.io/s/sweet-fire-e9ry1d