Support local assets in stylesheets and inline styles

Framework does static analysis of HTML and JavaScript to find references to local assets, such as linked images and imported JavaScript modules; however, it doesn’t currently perform static analysis of stylesheets and inline styles, so attempting to reference local images and import other local stylesheets currently breaks. Here are a few examples (that can serve as test cases for this feature).
An inline stylesheet that references a local image:
<style type="text/css">

body {
  background-image: url("image.png");
}

</style>

A linked local stylesheet that references a local image:
<link rel="stylesheet" href="test.css" type="text/css">

Alternatively, via @import (which is currently also unsupported for local stylesheets, and should be considered part of this issue):
<style type="text/css">

@import url("./test.css");

</style>

And the following stylesheet:
body {
  background-image: url("image.png");
}

An inline style that references a local image:
<div class="card" style="background-image: url('image.png');">
  I have a background image.
</div>

Related #415 #416 #423 #467 #474 #786 #788 #1372.

I have been trying to go around this issue for a long time now. I am able to solve it via D3js today.
So, I just simply took the served absolute URL of the image by
const imgURL = await FileAttachment("./exampleImage.jpg").href;

Then by doing the following I am able to put a background image in a div with id divId:
d3.select("#divId").style("background-image", "url("+imgURL+")");

It's possible to work around the issue and host these fonts within your data app, but it requires a little bit of juggling. Here's my current approach:
https://observablehq.observablehq.cloud/framework-example-font-face/?secret=w7WHOz2YV5uW-V0Bml5Cmwvc7hEsVOAfm2RRwuh-vu4
(I'm sharing in case it can unblock someone, but it would be better to fix the issue.)


Today python pollination is not working

URL Result Link Dead: HERE
504 ERROR

Bring back pollinations.ai 1.0.1 please...

Hi! It may have been a temporary issue. Your link seems to be working now. Could you let me know if it occurs frequently?
You could also consider using the flux model instead of turbo as the quality is higher and we have a few more servers allocated to it.

Hi! It may have been a temporary issue. Your link seems to be working now. Could you let me know if it occurs frequently?
You could also consider using the flux model instead of turbo as the quality is higher and we have a few more servers allocated to it.

The problem is solved now, maybe there was a problem since pollination updated the website theme this day to be more attractive. And From what I know, the pollination server often has errors, usually only a few minutes and a maximum of 1 hour. It can be seen when the website cannot request an image feed.


feat(icon): enables treeshaking when used by parcel or webpack

Treeshaking for Icons is not be working. After creating a test project I concluded it's two things adding sideEffects: false to our repo and optimization: {usedExports: true,} to a webpack config for webpack 4. As for Parcel I haven't tested yet but according to their docs just the sideEffects part is needed only.

for ref, docs: https://webpack.js.org/guides/tree-shaking/


Break out conda management

Create a dedicated folder to manage the conda environment and associated lock files.
Beefed up the lock-environment.sh file a bit and added some checks.

I haven't run the script to generate new lock files. There are quite a few more changes other than adding the cookiecutter
Want to create a separate PR to better review them and be able to revert.
Then I would also need someone with a Mac to test the .lock file for that platform

Going to wait until PR #44 is in to test this against the updated GH Actions

Checks for test are failing and include this error:
./scripts/deploy_website.sh: line 24: cookiecutter: command not found

As mentioned above in a comment, I will follow up with a separate PR to update the lock files and add cookiecutter.

As mentioned above in a comment, I will follow up with a separate PR to update the lock files and add cookiecutter.

Sounds good. I think you'd be fine adding the new lockfiles in this PR as well. up to you! merge whenever.

Here another error from the test run:
Setup auth token
  Error: Action failed with "not found deploy key or tokens"

Log an issue?

Error: Action failed with "not found deploy key or tokens"

Looks like GitHub actions checks the step inputs even if the step does not run. I'm surprised it raises an error, if we want to avoid it, we could pass the default token for any steps that run buildbook but do not publish:
    - uses: ./.github/actions/buildbook
      with:
        jb-cache: true
        publish-to-gh: true
        jb-save: true
        token:  ${{ secrets.GITHUB_TOKEN }}

As in https://github.com/uwhackweek/jupyterbook-template/blob/a9d26cfa325291b5b5f1d63cea9d0085de8844d5/.github/workflows/deploy.yaml#L22

Alternatively we could also access the input context via `index syntax':
inputs['token']

This should not raise an error and I can see whey the 'property dereference' syntax throws an error.
https://docs.github.com/en/actions/learn-github-actions/contexts#about-contexts

interesting. wouldn't using the 'index syntax' still raise an error if 'token' doesn't exist? maybe just give the 'token' input an empty string default value: default: ' ' ?

Depends what GitHub is using for interpreting the syntax. Ruby would return a nil value, while Python raises a key error.
I will open a PR and test this.


Make compilation & deployment easier

Provide:

./Makefile
./Dockerfile
./deploy/*.yaml: Kubernetes yaml files to deploy with kubectl apply -f deploy/

I think this is fixed now.


Steam games not detected

Hi, I don't know why, while GOG, Epic and Gamepass games are well detected, Steam games are not detected at all.
There's the right path for Steam in the settings.
I've got Steam games on several SSD, maybe that's why?

As I wrote here (https://github.com/Drommedhar/DlssUpdater/issues/12) this should work.
At least it does on my side and on a friends PC.
Might need to add even more logging to the Steam parser.
Can you attach your logfile, so I can see whats in it?

ok here it is
AppLog.2024-10-06.txt

Okay thats odd. We might be onto something here.
Could you pleasxe check if the following file is present: E:\Steam
\steamapps\libraryfolders.vdf
If it is, could you maybe provide it? There might be something different to it then mine. I ask because the log shows its not even trying to read any steam game. I most definately need to increase Logging for steam tomorrow.

There is a libraryfolders.vdf in E:\Steam\steamapps\
libraryfolders.zip

Thanks.
I actually found a bug because of that.
One library path did not have Apps installed and I incorrectly ignored this entry, which in turn completly stopped parsing games from steam.
This one will be fixed in the next version.

I just pushed an Update (<IP_ADDRESS>) which hopefully fixes your issue.
If not I at least have more data on what exactly might have failed.

Good work, with <IP_ADDRESS> Steam games are detected .


"Table of Contents Anchor" causes Obsidian to get stuck

If there are no other headings under the heading "^toc", Obsidian will get stuck after one second and no clicking, typing or other arbitrary actions will be possible. You can only force close Obsidian and delete "^toc" immediately within one second of opening to avoid the next stuck.

Thanks for submitting this! I fixed this bug in the upcoming version 1.11.0, which I will release later today.


styleguidist?

how to use your component.js with react-styleguidist?

I'm using the library in styleguidist currently, so you should be able to!

I'm not sure, you'd have to reference their documentation


Bump com.diffplug.spotless from 6.15.0 to 6.17.0

Description

Backports #6751  to 2.x

Issues Resolved

N/A

Check List

[ ] New functionality includes testing.

[ ] All tests pass

[ ] New functionality has been documented.

[ ] New functionality has javadoc added

[x] Commits are signed per the DCO using --signoff
[x] Commit changes are listed out in CHANGELOG.md file (See: Changelog)

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

Linked Issue: https://github.com/opensearch-project/OpenSearch/pull/6764
needs to be merged in post this PR merge.


[Feature Request] pipelines create should support Github Enterprise

Is your feature request related to a problem? Please describe.
Yes. I am trying to automate pipeline creation for a client and am unable to do so through the Azure DevOps CLI extension because it does not support Github Enterprise.
Describe the solution you'd like
"ghe" or "githubenterprise" should be an allowed value for --repository-type, and pipeline creation from a Github Enterprise repository should work.
This may require either adding an additional parameter for the github api base url, or parsing the domain from the --repository argument and then appending it and using https://{domain}/api/v3/.
Additional context
Error received when --repository-type is not provided and remotes of local repo point to Github Enterprise:

The following arguments are required: --repository-type. Check command help for valid values.

Error received when --repository-type github is provided along with a --service-connection that resolves to a GHE service connection and a --repository that points to a GHE repo:

Unable to configure a service on the selected GitHub repository. GitHub returned the error 'Bad credentials'.

Thank you for the request and feedback. At present, we are not focusing on extending this to GHE. We are open to contributions on this one - let me know if you need any help or clarification on this.

Upvote at the Developer Community - https://developercommunity.visualstudio.com/t/Add-support-for-githubenterprise-reposit/1684063


branch-2.1 fix error result in auto partition name (#41130)

pick https://github.com/apache/doris/pull/41130

run buildall

run buildall

run buildall


force acled to run in serial mode only, remove acled template for other problems which may possible to use

also, add more time for clean up jobs


Lab3 deliverables

Resume written in html with accompanying css style document

Hello, your code looks really code maybe you can try changing the font sizes.


fixMtmv support row column (#35860)

pick from master: https://github.com/apache/doris/pull/35860
when create mtmv with property 'store_row_column' = 'true',and `select

from mtmv order by xxx limit xxxbe will core,before MTMV not have hidden columnDORIS_ROW_STORE_COL`

Proposed changes
Issue Number: close #xxx

run buildall

run feut


cmd: Improve lock handling with keyring cache

Signed-off-by: Morten Linderud<EMAIL_ADDRESS>

Just saw the rather sparse commit message. Do you mind elaborating how the keyring lock is fixed, while also mentioning why we need the ephemeral (build) and the cache locks.
Splitting the patch one for each lock - root.lock build.lock cache.lock and keyring.lock - would be amazing, although I realise that I may be pushing my luck by suggesting it.


Start charging with different RFID credentials

Hi,
thanks for your work on this node!
we have two different RFID tags and I'd like to start charging with either the first or second RFID tag, depending on which car is connected. Is there a way to do this as I have not found the right parameters on the API page (https://developer.easee.com/reference/post_api-chargers-chargerid-commands-start-charging)
Would be great to have a hint on that.
Thanks!

Hey,
Unfortunately it would appear that the API does not stream RFID tag event, atleast on my device - but I have not activated RFID based charging.
I'll check with my other one when it comes online if i get something on that one.
Jon

I see now that some observations exist that might be interesting for this usecase, I will have to look into what data is returned:
69 | PAIRED USER IDTOKEN | Observed user token when charger put in RFID pairing mode
108 | USER IDTOKEN REVERSED | User ID token string from RFID reading (NB! Must reverse these strings) | String | Event
128 | USER IDTOKEN | User ID token string from RFID reading | String | Event

Thanks for your effort!
In the meantime, I had a bad experience: the charger is operated by reev, that bills the charging for the company car to my company, and ignoring the charching done with our private RFID key. When I start the chargeing over Node-Red, reev doesnt recognize that it should be recorded for the company bill, so I've run a full charging without refund. This wasn't a desaster but I recognized, that it can't be done like that. I contacted reev but they dont offer an API whatsoever... :(


Add circle background for Firefox OS

RFG suggests a circular background with a gloss effect for Firefox OS. There should be a way to do this with JIMP.

I've opened an issue here: https://github.com/oliver-moran/jimp/issues/69. Judging from the docs, I think the only way to do this at the moment would be having an algorithm to set pixel transparency / colour programatically, but we'll see.

Added in<EMAIL_ADDRESS>


help me pls

This dependency was not found:

C:\xampp\htdocs\finals\resources\sass\theme.scss in multi ./resources/js/app.js ./resources/sass/app.scss ./resources/sass/theme.scss

To install it, you can run: npm install --save C:\xampp\htdocs\finals\resources\sass\theme.scss
Asset      Size   Chunks             Chunk Names
/css/app.css   436 KiB  /js/app  [emitted]  /js/app
/js/app.js   1.4 MiB  /js/app  [emitted]  /js/app
\js\sweetalert.min.js  39.9 KiB           [emitted]
ERROR in multi ./resources/js/app.js ./resources/sass/app.scss ./resources/sass/theme.scss
Module not found: Error: Can't resolve 'C:\xampp\htdocs\finals\resources\sass\theme.scss' in 'C:\xampp\htdocs\finals'
@ multi ./resources/js/app.js ./resources/sass/app.scss ./resources/sass/theme.scss /js/app[2]
npm ERR! code ELIFECYCLE
npm ERR! errno 2
npm ERR! @ development: cross-env NODE_ENV=development node_modules/webpack/bin/webpack.js --progress --hide-modules --config=node_modules/laravel-mix/setup/webpack.config.js
npm ERR! Exit status 2
npm ERR!
npm ERR! Failed at the @ development script.
npm ERR! This is probably not a problem with npm. There is likely additional logging output above.
npm ERR!      C:\Users\Ihatepoper\AppData\Roaming\npm-cache_logs\2019-10-05T13_57_25_654Z-debug.log
npm ERR! code ELIFECYCLE
npm ERR! errno 2
npm ERR! @ dev: npm run development
npm ERR! Exit status 2
npm ERR!
npm ERR! Failed at the @ dev script.
npm ERR! This is probably not a problem with npm. There is likely additional logging output above.
npm ERR!      C:\Users\Ihatepoper\AppData\Roaming\npm-cache_logs\2019-10-05T13_57_25_851Z-debug.log

How are you trying to import it in /resources/sass/app.scss?


setup: a safe, no choices to be made setup that safely installs dependencies

@yupferris The idea is to remove choices and sharp edges from install process.
Just run:
sudo etc/setup.sh
Then, to set up environment variables, run:
. env.sh
From ORFS, a local build is then made by running:
./build_openroad.sh --local
Then, after https://github.com/The-OpenROAD-Project/OpenROAD-flow-scripts/pull/928 merged, you can run the commands to set up env vars to yosys and OpenROAD binaries:
. env.sh

looks good and helpful to me at least

Moved everything into https://github.com/The-OpenROAD-Project/OpenROAD-flow-scripts/pull/928


Skodaconnect

Update BaseLib to 1.3.0 (New token handling URLs) and several Improvements

Thx!

Happy new year Dominik.. Just for info. There might be some trouble for the moment with skoda and also with other vw connectors. VW rolled out an update on January 1st but turned back later the day, too much trouble with this update.. I'll keep an eye on that and contribute newer versions which I test at least for 24h before...

Thank you very much for sharing that information. I tested fhempy with Skodaconnect the first time at December 24th and was happy when i saw the SOC of my Citigo the first time. But when I started to implement all other required modules in FHEM on January 1st, half the day I tried to make fempy and Skodaconnect work again without any chance.
At the end I throw all away...
But looking again yesterday in the evening, everything works fine again.


Make use of django_storage

Currently the django_storage adapter tightly coupled with django_chatterbot. By introduce this fix, others can reuse storage adapter.
@gunthercox please provide your comments/suggestions on this PR.

@gunthercox Master, any plans to release chatterbot minor version this week?

Yes, possibly this weekend.

@gunthercox Any updates?

@vkosuri Apologies. I haven't had a chance to get around to drafting a new release yet. There is still a few possible issues that I want to investigate with the new SQL Alchemy adapter before doing the release.


VentoyVlnk.exe -s the save as file path.

Closes #XXXX

Relates to #XXXX

[ ] I have read the Contributing Guide.

There are conflicts

The tab error happened again.


OSSM-1975 Use actual Kubernetes version when rendering charts (#1012)

Do we need to update 2.0 and 2.1 branches of operator?

@jwendell WDYM? This is a cherry-pick of #1012 that I'm now applying to all previous branches (see also #1030 and #1031).

I meant.. it's unlikely we will release a new 2.0.x operator, right? People should use only the latest (2.2.x ATM) version of the operator, right?

Ah, I see what you mean. Yes, I wasn't sure, so I submitted PRs for all previous versions.


Add counters for failed/succeded transactions broken down by sender oracle

This doesn't work and is not needed anymore!


Add clone method

In some cases making a deep copy of an object instance is useful.

Add a class Clone and implement it for other classes.

IdlValue: public Clone {
    IdlValue clone() noexcept {}
}

:link: zboto Link

no necessary at the moment.
Can be re-opened later.


[feature request] local variables in v-for

Here is a nested v-for case

0">

 {{y}}

fallback content

</su-tab>

where calcYs is a method. As you can see, for every x, calcYs(x) will be called twice (if not empty) which is not necessary. It would be more efficient if we can bind the result of calcYs(x) to some local variables inside v-for.

This complicates the template quite a bit with very little benefits. I don't think it's worth it.

Could this be accomplished with a custom directive? eg,

0">

 {{y}}

fallback content

I have a similar problem. I would like to do something like this v-for="project.relations().tasks()".
What would the recommended approach be? A new component just to list task names seems like overkill.

Please disregard my previous post. It was working all along.

Code below generates "active table". It would be nice to have a local v-for variable for current index: (r-1)*10 + (c-1).

    {{ (r-1)*10 + c}}

@AlexandreBonneau
As a quick hack you can try this way:

    <template v-for="scope in [{ subtotal: findSubtotal(payment.term) }]">

{{ scope.subtotal.shortDescription }}

{{ anFloat(scope.subtotal.int) }}

{{ anFloat(scope.subtotal.cap) }}

{{ anFloat(scope.subtotal.ins) }}

{{ an(scope.subtotal.fee) }}

{{ anFloat(scope.subtotal.ter) }}

    </template>

this issue is one year old 😅
You can give https://github.com/posva/vue-local-scope a try too

Sometimes it turns out to be more convenient to move repeating code out to a separate component.

Sometimes it turns out to be more convenient to move repeating code out to a separate component.

It can't cover all use cases, but it was a nice and working advice for me. A small and easy-to-maintain subcomponent.


dev -> master

This change is

Reviewed 72 of 75 files at r1.
Review status: all files reviewed at latest revision, all discussions resolved.

Comments from Reviewable

Review status: all files reviewed at latest revision, all discussions resolved, all commit checks successful.

a discussion (no related file):
Wait on testing

Comments from Reviewable


Player locations error in map

I have completed all the steps but the location marker does not appear
I need a guide

Do you run PUBG-map-hack?
This map need PUBG-map-hack to send player data.


Default device not detected

On the current version, ares-install would ignore my standard device.
Its impossible to choose it specifically as well.

I cannot even call it directly via IP:


CARDS-2118: Add an Unsubscribe link in the footer + CARDS-2119: The unsubscribe page looks sloppy

Includes:

CARDS-2118 - Patient portal: add an Unsubscribe link in the footer
CARDS-2119 - Patient portal: the unsubscribe page looks sloppy

Testing:

start prems
go to /Survey.html without a token - there should be no Unsubscribe link (it only appears when the patient is authenticated)
as admin, create a Patient information form (can be left empty) and a Visit information form with any survey
as patient, go to the link for filling out that survey -> notice the unsubscribe link in the footer
click on Unsubscribe and follow instructions
once unsubscribed, open the Patient information form and check that the "unsubscribed" answer is set accordingly
bonus: start in proms and do similar testing

Reviewing:

commit by commit, ignoring whitespace recommended

Preview:

Question: I think moving Unsubscribe link below TOU would look better. WDYT?

Question: I think moving Unsubscribe link below TOU would look better. WDYT?

I'm not a fan, especially on Form & Review pages - I think it's a waste of vertical space for two very rarely used links that fit just fine on one line.

In proms mode there ar a few issues with this

Since prems is active and proms isn't, I suggest we make a separate task for proms specific fixes so we can roll out the prems improvements asap.

In prems mode things are much better, the only tiny bug is when trying to access without a token, the extra dot at the end of the breadcrumbs is present, moving the ToU link a bit off center.

Addressed in 0613876.

one can access the patient portal without a token, authenticate using the personal information, and then in theory they will be authenticated, but the Unsubscribe link doesn't show up
when accessing with a token, the unsubscribe link can be used before the patient actually validates who they are; that's not really bad, since we already have an unsubscribe link in the email that doesn't require authenticating, but from the user's perspective it may seem wrong that they can do something with their account before they actually "log in" into the account

Actually the more I think about all this the more it make it makes sense the way it is now even for proms. "Unsubscribe" is there because the patient was brought to that screen from their email link. There's no reason to want to unsubscribe if you came here out of your own free will. Anyway, I created https://phenotips.atlassian.net/browse/CARDS-2128 and we can deal with one way or the other it when time comes to wrap up proms.

Actually the more I think about all this the more it make it makes sense the way it is now even for proms. "Unsubscribe" is there because the patient was brought to that screen from their email link. There's no reason to want to unsubscribe if you came here out of your own free will.

You're right.

The ToU link is actually a button, while the Unsubscribe is a link. That makes the two not vertically in line (ToU is 2 pixels lower).

The look pretty well-aligned to me. This is in FF, same in chrome:

The ToU link is actually a button, while the Unsubscribe is a link. That makes the two not vertically in line (ToU is 2 pixels lower)

Replicated and addressed in f33f1ea.


Customizing the context menu

With the removal of shouldShowInContextMenu and contextMenuGroupId from IActionDescriptor in version 0.6.0, how can we edit what commands get shown in the context menu, and how do we include our own custom commands in there?
monaco-editor npm version: 0.6.0
Browser: Chrome
OS: Windows

Sorry about that, we need to bring back that functionality.

Let's track in #175


.interact() tpf is blank

Problem description

When plotting sector 8 TESS data with 'interact()', pixel data does not plot accurately. Potentially due to the change in postage stamp size for sector 8?

Example

import lightkurve
tpf = lightkurve.search_targetpixelfile('L 98-59').download_all()
tpf[2].interact()
# earlier sectors (tpf[0] and tpf[1]) have no issues

Expected behavior

Environment:

platform (e.g. Linux, OSX, Windows): OSX
lightkurve version (e.g. 1.0b6): 1.0.0
installation method (e.g. pip, conda, source): pip

Thank you for reporting this issue @emilygilbert.  You are definitely encountering some unexpected behavior there!
/cc @gully can you have a look?

Hi @emilygilbert,
Thank you for reporting the issue with helpful screenshots to help diagnose the phenomenon you are seeing.  The all-yellow pixels are a result of an imbalanced screen stretch normalization.
The default screen stretch for interact() is usually pretty sensible for most well-behaved data.  Your TPF-of-interest exhibits an extreme dynamic range, apparently due to an instrumental artifact in the middle third of the time window.  Luckily, the screen stretch can be controlled in the bottom right "Screen Stretch (log)" slider bar.  Does updating the screen stretch slider solve your problem?
While investigating your TPF, I noticed that something else seems to be affecting this target pixel file:  The mapping from cadence to the display window seems inaccurate.  Sometimes I observe all-gray pixels---which represent NaN data---even when the vertical red cadence indication bar points to a valid position in the lightcurve.

Quick followup:
The latter issue seems to be related to some unknown source of hysteresis between the mask pixel-selection assignment, screen stretch, and time series plot range.  If you notice some unusual behavior, I recommend a temporary workaround of adding-and-removing pixels in your mask.  You can use shift-click to toggle pixels on-and-off, though watch out for the inability to deselect the most-recently-selected pixel.

Yes, stretching helps! Thanks, that's a good tip!

I tested this on ~50 sector 8 targets, and they all have the same issue. I know sector 8 had some spacecraft anomalies, so I guess that's not too surprising. I can send you an example notebook if that'll help!

Good, glad that the screen stretch resolved the problem!
I suspect all TESS Sector 8 targets will exhibit the same screen stretch default appearance.  What's happening behind the scenes, is that lightkurve computes the 1% and 95% percentile across all finite pixel values in the TargetPixelFile.
vlo, lo, hi, vhi = np.nanpercentile(tpf.flux - pedestal, [0.2, 1, 95, 99.8]) # line 312 of interact.py

Sector 8 is apparently an exception, because 1% of the cadences have extremely low values, and these anomalously-low-values are not automatically masked with quality flags.
We could consider increasing the lo threshold default to, say, 5%.  That threshold might be too high for other, more typical TPFs.  We've found 1% and 95% to be decent threshold defaults for a wide range of Kepler and TESS data.
If you want to over-ride the hardcoded 1% for yourself, you could install the lightkurve developer version alter line 312 of the file interact.py, and relaunch your jupyter-notebook.


Payload HTTP Backend Example

Recipe Name
PlayloadHttpApiNode

Each recipe name must be unique. Use PascalCase.
Recipe Description
A simple API example for the strongly typed HTTP backend payload. Ideally, it would show an API that allows for get all, get by id and insert, at least.
I already implemented something using AVar, and I was wondering whether this is too complicated for a single recipe? The alternative would be using files instead (to keep state across calls), but then it would also complicates matters due to encoding/decoding.
Do you think it would be an useful recipe?
PS: I cannot use StateT because it's not currently possible yet (in payload). I added an issue, and am willing to update this when/if it gets resolved.

I cannot use StateT because it's not currently possible yet (in payload). I added an issue, and am willing to update this when/if it gets resolved.

If you use StateT, it would likely not produce the behavior you want. StateT threads state through a function call. I believe each request-response would start with the original value passed to the StateT computation, not the latest value.

I already implemented something using AVar, and I was wondering whether this is too complicated for a single recipe? The alternative would be using files instead (to keep state across calls), but then it would also complicates matters due to encoding/decoding.

AVar is fine. If we wanted to do this properly, we might use the ReaderT pattern, similar to what I did in the random number game. However, that might get too complicated for little benefit. This recipe highlights how one can build an HTTP server quickly using the library.

PS: I cannot use StateT because it's not currently possible yet (in payload). I added an issue, and am willing to update this when/if it gets resolved.

Issue for reference: https://github.com/hoodunit/purescript-payload/issues/7

We're able to get really great insights into cookbook usability by reviewing your stream of submitting a recipe:

Making request issue: https://www.twitch.tv/videos/721301879?t=1h45m20s
Making request PR: https://www.twitch.tv/videos/721301879?t=2h10m59s
After Package set detour: https://www.twitch.tv/videos/721301879?t=2h29m40s

Kudos for adding payload to the package set.
Also, feel free to suggest ways that we could make the cookbook more user-friendly. For example with our organization, the instructions we're giving to users, available issue templates, or anything else.
Some comments from reviewing the stream:

Github doesn't allow self-assignment without being a collaborator on the repo. isaacs/github#100
If you make a recipe typo, you can simply use scripts/newRecipe.sh to create a new recipe from your misspelled one with the correction, then delete the misspelled directory.
The common commands to build+run recipes are printed by running make list (or just make without arguments). They're also listed under Running Recipes and printed after running newRecipe.sh.
The non-traditional cookbook directory structure does not play nice with most IDE configurations. I'll make an issue for this and a note in the docs.

I'll make a PR with these changes to improve the instructions:

Show how newRecipe.sh can be used to rename recipes.
Note that newRecipe.sh prints relevant commands.
Convert recipe template selection list to a table.
Link to Running Recipes from Contributing.md.

Git Tips:

Create a branch based on master:

git checkout -b my-new-branch master

Update master:

git pull --rebase upstream/master

Nice display of SHAs for all branches (for cherry-pick purposes):

git all with this alias https://github.com/milesfrain/configs/blob/e5111c70c998188a73279f8bec79ba4badac6086/core/.gitconfig#L19

A note on branch naming:
I notice you're naming branches with a forward slash. This unfortunately confuses TryPureScript github path parsing, and prevents posting a demo link for us to preview before the recipe is merged. This is irrelevant for this particular node-only recipe, but swapping the slash for a hyphen would make it easier to review future TPS-compatible recipes.


[Issue]: How to format number in NumberInput

Tremor Version
latest
Link to minimal reproduction
.
Steps to reproduce
I have a NumberInput and I want the German number format. In German, they use comma , instead of . fr their float numbers.
Code

 const handleNumberInputChange = (property: string, value: number) => {

   // Currently, the user enters numbers in this format 1.1 | 2.2
  // However, I want him that format 1,1 | 2,2
    dispatch(updatePanelMeasures({ ...measures, [property]: value }))
  }

 <NumberInput
          icon={TextQuote}
          placeholder='1.1'
          max={1}
          min={0.1}
          step={0.1}
          className='mt-2 min-h-11'
          value={measures.border}
          onValueChange={(border) => handleNumberInputChange("border", border)}
        />

What is expected?
.
What is actually happening?
.
What browsers are you seeing the problem on?
No response
Any additional comments?
No response

This looks interesting.
I am wondering whether to extend this to German number format or make this general. @christopherkindl

Would be awesome if we receive as optional props an "formatter" which represents Intl.NumberFormat constructor
https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Global_Objects/Intl/NumberFormat/NumberFormat
So we separate the render value from the reference value
return (
   <NumberInput
        defaultValue={123456.79}
        formatter={ new Intl.NumberFormat('en-US', { style: 'currency', currency: "USD" }) }
    />
)

  DISPLAY VALUE:  '$123,456.79'
  REF VALUE:   123456.79

I am available to work on this if needed.

Hi! @hikinine, this feature would be awesome! I assigned you to this, let me know if you need any help!

Hi! @hikinine, this feature would be awesome! I assigned you to this, let me know if you need any help!

Wonderful, i'll keep you posted

Hi! @hikinine I would love to have this feature, do you have any updates? Need an extra pair of hands to work on this?

Hi! @hikinine I would love to have this feature, do you have any updates? Need an extra pair of hands to work on this?

Hey, would be good.
I open a pull request  with a initial try to implement it. Take a look. #1111
I got some problems with UX to do it in the best way possible (which is edit the formatted value like editing a number... something like we used to do with regex to represente some number pattern as string)
Implemented some work around which is not the best UX solution, but works.
Lets move the discussion to https://github.com/tremorlabs/tremor/pull/1111


IIS 10 write_image() fails on POpen()

I think I have found the issue of IIS launching the orca server.
In _orca.py ensure_server()
    stderr = DEVNULL if "CI" in os.environ else None  # fix for CI
    orca_state["proc"] = subprocess.Popen(
        cmd_list, stdout=DEVNULL, stderr=stderr
    )

fails with orca_state["proc"] = None
If you change to:
    orca_state["proc"] = subprocess.Popen(
        cmd_list, stdout=subprocess.PIPE,
        stderr=subprocess.PIPE, stdin=subprocess.DEVNULL)

it works as expected.
Can I set stdout, stdin, and stderr in a configuration setting instead of modifying source code?
Thanks

cc @nicolaskruchten @jonmmease


This process does not work with ZOS

Attempting to deploy (and EVM package) using zos following the mainnet recomendations as listed here fails with the error:
Cowardly refusing to execute transaction with excessively high default gas price of 100 gwei. Consider explicitly setting a different gasPrice value in your truffle.js config file. You can check reasonable values for gas price in https://ethgasstation.info/.
We need to set the gas value and gas price. I added an example above, however this is not always enough either and can fail with either:
ProviderError: insufficient funds for gas * price + value
or
Error: Contract transaction couldn't be found after 50 blocks
or
Error: ProviderError: gas required exceeds allowance or always failing transaction
They contract may still be deployed however in the case of the final error example.

Thanks @crazyrabbitLTC


Get-AzResource shows networks and gateways from deleted applications

Azure Portal won't show them, but using the current azure powershell module there are still resources:
PS C:\Users\b.bopp> Get-InstalledModule -Name Az
Version              Name                                Repository
-------              ----                                ----------
1.0.1                Az                                  PSGallery

PS C:\Users\b.bopp> Get-AzResource | Select-Object ResourceType,Name

ResourceType                         Name
------------                         ----
Microsoft.ServiceFabricMesh/gateways ProductionOrderAppGateway
Microsoft.ServiceFabricMesh/networks ProductionOrder_AppNetwork
Microsoft.ServiceFabricMesh/networks ProductionOrderAppNetwork

Might be related to #305
Keep in mind that Networks and Gateways are azure resources independent from Applications, deleting the application won't delete the network or gateways.


Recursive extra dependencies don't compile like pip-tools

Essentially the same issue as https://github.com/jazzband/pip-tools/issues/1685. When you have a recursive extras in a pyproject.toml for a private project, uv pip compile fails. If the project is uploaded to an index, then it looks in the index first, but I'd expect it to prioritize the self-reference in pyproject.toml.
Info

Platform: macOS Sonoma 14.3
uv version: v0.1.10
command: uv pip compile pyproject.toml --extra=dev

Reproducible example (from pip-tools issue)
Minimal pyproject.toml
[project]
name = "privateproject"
version = "0.0.1"
dependencies = [
    "tomli",
]

[project.optional-dependencies]
test = [
    "pep517",
]
dev = [
    "privateproject[test]",
]

With pip-tools, this is the output:
❯ pip-compile pyproject.toml --extra=dev
WARNING: --strip-extras is becoming the default in version 8.0.0. To silence this warning, either use --strip-extras to opt into the new default or use --no-strip-extras to retain the existing behavior.
#
# This file is autogenerated by pip-compile with Python 3.10
# by the following command:
#
#    pip-compile --extra=dev pyproject.toml
#

pep517==0.13.1
    # via privateproject
privateproject[test] @ file:///...
    # via file:///...
tomli==2.0.1
    # via
    #   pep517
    #   privateproject
    #   privateproject (pyproject.toml)

With uv, this is the output:
❯ uv pip compile pyproject.toml --extra=dev
  × No solution found when resolving dependencies:
  ╰─▶ Because privateproject[test] was not found in the package registry and privateproject depends on privateproject[test], we can conclude that
      the requirements are unsatisfiable.

This worked without issue for me:

Ahh this is slightly different, you're passing the pyproject.toml directly, I see.

echo "privateproject[test] @ ./privateproject" | cargo run pip compile - -n works without issue, for reference.

Fixed in the next release, thanks.

Thanks for the quick fix!


Elements isn't tabbable

Hello!
Thank you for really beautiful controls!
I have a problem that the elements isn't tabbable. The role attributes doesn't work for this.
I think that it's very important for Web.
This is because native checkbox element is hidden display: none;
For correct working native element doesn't need to be hidden. It's need to be invisible for good view but doesn't hidden.
For myself I apply opacity: 0; but I'm not sure that it's a good idea... You are a pro, I think you can better :)
.magic-checkbox {
  position: absolute;
  opacity: 0; }

And for visual effect of focus I added the outline
.magic-checkbox:focus + label:before {
  outline: -webkit-focus-ring-color auto 5px;
}

I hope for your fix for everyone. Thank you! All the best!

Thank you Elias, I added the following to your code to fix the issue of not being able to tab to it:
.magic-checkbox {
  position: absolute;
  opacity: 0;
  display: unset;
}

We need to unset their display:none; setting. I know you mention it, but you didn't include it in the CSS. Thanks again though, I was about to dismiss the whole Magick-checkbox package. Shame it's not maintained any more.


[Bug] Misalignment of download button due to hard coded pixels

Description
On some of the resource pages the download button is not aligned to the centre due to the hard coded pixels in resource.html file.

Unnecessary code which needs to be deleted

Instead of this we can use flex properties in style.css
Reproduction

Go to resource page
See error.

Expectation
It should be properly aligned in the centre.
Resolution

[x] I would be interested in resolving this bug.

Good catch. Apparently, the misalignment of download button is only in open data handbook resource and no other.
The misalignment is not due hard coded pixels because if it were it would have been same for all resources.
Thanks to your issue, I had to take a look at it. The reason behind this is a commit from @Keerthivardhan1 . Apparently, he was removed all repo links from resources that were not working. But due to this change in resource.html , the repo link of Open data handbook which was working fine is not displayed. So next to the download button there should be a repo button link, which is missing.
Here's the repo url that was compromised.
If this was intentional, then I would suggest to just remove the repourl from opendatahandbook.html and the issue would be resolved.
I request @possumbilities to take a look into this as the change is live already.

@Xaid-vfx Thanks for the details you provided. I completely agree the misalignment is due to missing repo button but ultimately this needs to be corrected as it is not a good practice and any changes made to the download or repo button will result into misalignment in future.

@Xaid-vfx Updated issue title and description to address the main point.

@Lakshay-Pruthi
under the guidance of @possumbilities I had successfully resolved all the broken links this is one from them.
I think this issue is occurring with only --> open data hand book page.
Clicking on the button previously led to a website that is no longer available on the internet, namely lawrent.com.
we should not include the repository button again.  For additional information, please refer to https://github.com/creativecommons/cc-resource-archive/issues/11.

@Lakshay-Pruthi, you only need to organize the CSS styles, and your exploration in the website is really appreciative.

@Lakshay-Pruthi, you only need to organize the CSS styles, and your exploration in the website is really appreciative.

Hey @Lakshay-Pruthi and @Keerthivardhan1,
We can carry this conversation to #144, as it addresses the issue accurately.

Please assign this issue to me.

@abhishekshah5486 The UI for the page has major improvements done, and the issue has become irrelevant since then.
We can close the issue as completed @possumbilities 🙌

@Murdock9803 Is there a specific PR that targets this best?

@possumbilities The PR #304 targets the UI improvement for Resource Page. The button alignment and everything is taken care of there 🙌
refactored the html and css for resource page


Colored carpets in "Coloring spells" GUI are all white

Hello,
In latest versions of Magic and since 1.13, carpets in the "Coloring spells" GUI are all white, instead of being colored like they should be. I can't find the items linked to these, so I can't correct the issue myself.
Thank you for looking into this.

Fixed in 7.7.3- thanks!


Publish gh release binaries

I'd like my team to use this tool without forcing them all to install the go runtime

When https://github.com/bazelbuild/buildifier/pull/45 lands, I will make a tag and a companion release

Sweet.

I added Mac and linux64 builds -- hopefully that is enough?

Personally, I also need this on Windows.

Mac is perfect for me. Linux will be great for ci validation. Thanks @pmbethe09!

I don't have a good way to make a Windows release -- let me ask the bazel team for suggestions

looks like the links for the OSX release bins are invalid here https://github.com/bazelbuild/buildifier/releases. is this the right place to look?

Not sure how that happened, but I have fixed this.
On Sat, Feb 4, 2017 at 6:11 PM, doug tangren<EMAIL_ADDRESS>wrote:

looks like the links for the OSX release bins are invalid here
https://github.com/bazelbuild/buildifier/releases. is this the right
place to look?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/bazelbuild/buildifier/issues/44#issuecomment-277485017,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ALF-0kt5bXXNAIgawA1wE3CF0m1nvKbQks5rZQWTgaJpZM4LwmyL
.

@pmbethe09 The OSX release worked for me, but the Linux one claims that it does not support the -tables flag. Are you sure you built it from the correct commit?

...  No not sure.  We need a better release process as these were ad-hoc.
I have updated to fresh builds.

@pmbethe09 You might be able to have Circle-CI do the build and publish it for you. That's what we do for Nuclide, and the setup does not appear to be npm/apm specific: https://github.com/facebook/nuclide/blob/master/circle.yml. (Though maybe it is because somewhere we have to provide our credentials to publish on our behalf to npm...)
Also, FWIW, have you just been changing the binaries under the release tag rather than stamping out a new release with its own version number? It feels a little dodgy to just change the binaries in-place.

Yes, but only because I didn't know what I was doing -- no more changes as of now :)

No worries: I'm sure 0.4.4 will go much smoother!


Retry interval #709

I would like to implement retry interval which was discussed in issue #709.
To implement,  I changed _retry parameter as follows.
_retry: num interval constant|exponential
interval is seconds to wait for. (default 0)
constant or exponential set interval increments.(default constant)
if constant is set,  interval is fixed.
if exponential is set, interval increments with interval * 2^ retry_count.
Examples

_retry: 3

Same as original behavior. Retry 3 times without interval.

_retry: 3 10  or
_retry: 3 10 constant

Wait for 10 seconds for each retry

_retry: 3 10 exponential

The retry intervals are  10, 20, 40 for each retry.

Sorry to be late. This is good feature. I will take a look.

The basic logic looks good. What do you think that retry params could be represented as Json object like following? Because the string representation is confused by me.
_retry:
  limit: 3
  interval: 10
  interval_type: exponential

If the value is integer instead of Json object, the value will be treated as retry_limit as usual.
_retry: 3

What do you think?

@muga Thank you for your comment.
At first I also tried to implement with JSON.
But I noticed that I needed to register the new key names to WorkflowCompiler.GROUPING_TASK_CONFIG_KEYS.
(Sorry it may be my misunderstanding because it was so long ago)
Is it ok to increase keys for validation?
If so, I would like to change the implementation to JSON.
Best regards,
You Yamagata

@yoyama got your point and agree with you. I also don't want to register new key name. What do you think about the following idea prototyping roughly, not care of interval things there? to keep backward compatibility of the behavior of _retry, we'd better to keep evaluating _retry value as integer if the value is integer. on the other hand, we can check if the value is json object type or not.
diff --git a/digdag-plugin-utils/src/main/java/io/digdag/util/RetryControl.java b/digdag-plugin-utils/src/main/java/io/digdag/util/RetryControl.java
index cc777a91c2..d0cccde4ca 100644
--- a/digdag-plugin-utils/src/main/java/io/digdag/util/RetryControl.java
+++ b/digdag-plugin-utils/src/main/java/io/digdag/util/RetryControl.java
@@ -1,5 +1,6 @@
 package io.digdag.util;

+import com.fasterxml.jackson.databind.JsonNode;
 import io.digdag.client.config.Config;
 //import io.digdag.core.workflow.TaskConfig;

@@ -17,7 +18,20 @@ public class RetryControl
     private RetryControl(Config config, Config stateParams, boolean enableByDefault)
     {
         this.stateParams = stateParams;
-        this.retryLimit = config.get("_retry", int.class, enableByDefault ? 3 : 0);
+        final JsonNode retry = config.getInternalObjectNode().get("_retry");
+        if (retry != null && (retry.isNumber() || retry.isObject())) {
+            if (retry.isNumber()) {
+                this.retryLimit = retry.intValue();
+            }
+            else { // object
+                this.retryLimit = retry.has("limit") ?
+                        retry.get("limit").intValue() :
+                        (enableByDefault ? 3 : 0);
+            }
+        }
+        else {
+            this.retryLimit = enableByDefault ? 3 : 0;
+        }
         this.retryCount = stateParams.get("retry_count", int.class, 0);
     }

@muga Thank you for your comment.
I agree with your idea. It is very excellent!
I will change the implementation with your idea.
Best regards,
You Yamagata

@yoyama thank you. got it. please let me know if ready to take a look.

@muga Thank you for your comment.  I changed the code according to our discussion.
But CI failed.
I don't know the cause of issue. Please wait a moment until fixing it.
Best regards,
You Yamagata

@muga Thank you so much for your reviewing in detail.
I fixed both coding style issue and adding information to error message.
If there are something other issues, please let me know.
Best regards,
You Yamagata

@yoyama looks good! thank you for improving. merging.


Possible bug in anomaly_detection.py

In anomaly_detection.py line 105, you calculate the standard deviation of the mean value (which is a scalar) and it makes no sense. I think it should be window_std = np.std(window_elts)

Yes, you are right. Need to fix it. Will you be interested in fixing it?

@arunppsg I just made a pull request.

Thanks for it. Will close this issue.


#6 - eo-hamcrest: unit tests assertions

introduce eo-hamcrest v.0.1.5

@yegor256 I have done with eo-hamcrest v0.1.5, please have a look

@Graur thanks!


Highlight matched hints

Highlights matched hints in CSS

@redmunds Done!

Good catch. It looks like same change needs to be made to /src/extensions/default/JavaScriptCodeHints/styles/brackets-js-hints.css.

JavaScript hints uses more than one rules for hints, depending on priority. It will be a nice idea to have @larz0's opinion on the choice of colors.

@sprintr

JavaScript hints uses more than one rules for hints, depending on priority.

Good point. I'll just merge this fix for now.


SPIKE Get an iframe PDf viewable

🔗 Pivotal Tracker ticket
#187289780
Notes in Notion
✍ Description

Add test pdf to /static folder
Add security config to allow same origin frames
Add a new screen to show the inline PDF

DO NOT MERGE
Once we're done referencing this Spike. We should close and delete this branch.

I'm closing this PR as it's no longer needed.


Add support for Search Guard in Elasticsearch connector

Add support for Search Guard in Elasticsearch connector
Extracted from: https://github.com/prestodb/presto/pull/12422

Applied comments.

Gentle reminder (faced same issue)


Incorrect SYN detail

Changed SYN from "Sequence Number" to "Synchronization"

Thank you a lot for your contribution @AnirudhKK ❤


Needs a way to remove skins from the list

Maybe bind it to X or something

just delete the files

as @arceuss said: a temp-fix is that you can just delete the files off your SD card. not a high-priority issue atm


Added Application UI

Signed-off-by: Gaurav Chadha<EMAIL_ADDRESS>Description
Application UI is added.

This PR fixes #3163
Signed commits

[x] Yes, I signed my commits.

Thank you, @Chadha93. Looks great. 👍
@ramrodo, the build failure is orthogonal to the changes in this PR. Are you ok with moving forward and investigating the failed (unnecessary) docs builds separately?

Welcome @leecalcote. Yes, I was also wondering about the docs failure 👀.

@leecalcote @Chadha93 The errors are due to a mismatched syntax in the main file of the docs, I've opened a PR for the same to fix it 👀

Thank you @Chadha93 🚀

Thanks 😊

Thank you, @Chadha93. Looks great. +1
@ramrodo, the build failure is orthogonal to the changes in this PR. Are you ok with moving forward and investigating the failed (unnecessary) docs builds separately?

Yes. Those builds are separated but something is wrong with the Netlify builds. We can move forward if it is urgent. If not, I'll dig in that bug this weekend.

@ramrodo Let's go ahead with this PR and you can fix the docs build over the weekend.

@ramrodo Let's go ahead with this PR and you can fix the docs build over the weekend.

Sure. Thanks.


Tekton controller queue processing time

Hello,
I am scheduling taskruns in a namespace with a resource quota which allow only a certain number of taskruns to schedule their pod at the same time. Each taskrun takes a few minutes to complete.
The taskruns not running are in the queue waiting for free resources for their pod to be scheduled. The following events are reported:
  Normal   ExceededResourceQuota  8m25s (x8 over 8m26s)  TaskRun  TaskRun Pod exceeded available resources: pods "exec-tc-21.6-27832-pod" is forbidden: exceeded quota: tekton, requested: requests.memory=3500Mi, used: requests.memory=213700Mi, limited: requests.memory=211Gi
  Warning  InternalError          8m25s (x8 over 8m26s)  TaskRun  1 error occurred:
           * pods "exec-tc-21.6-27832-pod" is forbidden: exceeded quota: tekton, requested: requests.memory=3500Mi, used: requests.memory=213700Mi, limited: requests.memory=211Gi

Expected Behavior
Tekton controller continuously check if he can process taskrun in the queue. It starts taskrun as soon as resourcequota is free. The number of taskruns running at the same time should be the consistent over time.
Actual Behavior
Taskrun are waiting in the queue even if resourcequota is free to schedule their pods. The number of running taskrun decrease over time.
Steps to Reproduce the Problem

Create a resource quota in the namespace
Create a task with a step using resources requests and limits
Apply taskruns which trigerred the previous task. The sum of resources used by the created taskrun should exceed the quota.

Additional Info

Kubernetes version:

Client Version: version.Info{Major:"1", Minor:"21", GitVersion:"v0.21.0-beta.1", GitCommit:"88e7eba36ac7cc026d064bb6e4a988acc18a144e", GitTreeState:"clean", BuildDate:"2022-01-21T03:47:14Z", GoVersion:"go1.16.6", Compiler:"gc", Platform:"linux/amd64"}
Server Version: version.Info{Major:"1", Minor:"21+", GitVersion:"v1.21.2-1555+9e8f924492b7d7-dirty", GitCommit:"9e8f924492b7d70b0216d20529f3471eed59463c", GitTreeState:"dirty", BuildDate:"2021-11-10T16:00:46Z", GoVersion:"go1.16.6", Compiler:"gc", Platform:"linux/amd64"}

Tekton Pipeline version:

Client version: 0.21.0
Pipeline version: v0.34.0
Dashboard version: v0.24.1

We've heard this issue before, and digging into it I believe the underlying issue is that Tekton's retry logic when Pods don't schedule due to resource quota is to retry with exponential backoff.
So, imagine the scenario where a TaskRun is about to be started, but there aren't enough resources right now, but there will be resources available in a few minutes. The TaskRun fails to create, and retries in 1 second. Still no resources, so backoff and retry in 2 seconds. Again, in 4s, then 8s, 16s, 32s, 1 minute, 2 minutes, 4 minutes, 8 minutes (this is an example). If resources became available right now, the TaskRun in the queue wouldn't check to find out for another 8 minutes (or more, if it had retried more times). When it retries, it may find available resources and try again, but if not, it'll try again in 16 minutes, and so on.
This is mostly conjecture, but I think it's a reasonable guess since we do exponential backoff for other failure scenarios.
If this is indeed the issue, we should see if we can configure our backoff logic to cap out at, say, 1 minute, so it never ends up waiting so long to try again.

Yes this is what you explained me on Slack, I couldn't find by myself what cause the exponential backoff. Then I created this issue with more detailed to get help and find the root cause of that problem.
Is it possible to test and confirm your conjecture ?

I reapplied the taskruns with the controller log level set to debug. I couldn't see any logs reporting Failed to create taskrun pod for taskrun .*
The logs that I see about the ExceededResourceQuota are
{
  "level": "debug",
  "ts": "2022-05-11T15:33:39.250Z",
  "logger": "tekton-pipelines-controller",
  "caller": "taskrun/reconciler.go:323",
  "msg": "Updating status with:   v1beta1.TaskRunStatus{\n  \tStatus: v1beta1.Status{\n  \t\tObservedGeneration: 0,\n  \t\tConditions: v1beta1.Conditions{\n  \t\t\t{\n  \t\t\t\tType:               \"Succeeded\",\n  \t\t\t\tStatus:             \"Unknown\",\n  \t\t\t\tSeverity:           \"\",\n- \t\t\t\tLastTransitionTime: apis.VolatileTime{Inner: v1.Time{Time: s\"2022-05-11 15:30:57 +0000 UTC\"}},\n+ \t\t\t\tLastTransitionTime: apis.VolatileTime{\n+ \t\t\t\t\tInner: v1.Time{Time: s\"2022-05-11 15:33:39.249508293 +0000 UTC m=+9104.073853021\"},\n+ \t\t\t\t},\n- \t\t\t\tReason:  \"ExceededResourceQuota\",\n+ \t\t\t\tReason:  \"Pending\",\n- \t\t\t\tMessage: `TaskRun Pod exceeded available resources: pods \"exec-tc-21.6-22901-pod\" is forbidden: exceeded quota: tekton, requested: requests.memory=3500Mi, used: requests.memory=213500Mi, limited: requests.memory=211Gi`,\n+ \t\t\t\tMessage: \"Pending\",\n  \t\t\t},\n  \t\t},\n  \t\tAnnotations: nil,\n  \t},\n  \tTaskRunStatusFields: v1beta1.TaskRunStatusFields{\n- \t\tPodName:         \"\",\n+ \t\tPodName:         \"exec-tc-21.6-22901-pod\",\n- \t\tStartTime:       nil,\n+ \t\tStartTime:       s\"2022-05-11 15:33:39.236575377 +0000 UTC m=+9104.060920114\",\n  \t\tCompletionTime:  nil,\n- \t\tSteps:           nil,\n+ \t\tSteps:           []v1beta1.StepState{},\n  \t\tCloudEvents:     nil,\n  \t\tRetriesStatus:   nil,\n  \t\tResourcesResult: nil,\n  \t\tTaskRunResults:  nil,\n- \t\tSidecars:        nil,\n+ \t\tSidecars:        []v1beta1.SidecarState{},\n  \t\tTaskSpec:        &{Params: {{Name: \"application_version\", Type: \"string\"}, {Name: \"amos_version\", Type: \"string\"}, {Name: \"test_project\", Type: \"string\"}, {Name: \"test_plan\", Type: \"string\"}, ...}, Steps: {{Container: {Name: \"qftest\", Image: \"image-registry.openshift-image-registry.svc:5000/$(params.regist\"..., Env: {{Name: \"AMOS_VERSION\", Value: \"$(params.amos_version)\"}, {Name: \"APPLICATION_VERSION\", Value: \"$(params.application_version)\"}, {Name: \"TEST_PROJECT\", Value: \"$(params.test_project)\"}, {Name: \"TEST_PLAN\", Value: \"$(params.test_plan)\"}, ...}, Resources: {Limits: {s\"memory\": {i: {...}, s: \"3700Mi\", Format: \"BinarySI\"}}, Requests: {s\"cpu\": {i: {...}, s: \"800m\", Format: \"DecimalSI\"}, s\"memory\": {i: {...}, s: \"3500Mi\", Format: \"BinarySI\"}}}, ...}}}},\n  \t},\n  }\n",
  "commit": "e055233",
  "knative.dev/controller": "github.com.tektoncd.pipeline.pkg.reconciler.taskrun.Reconciler",
  "knative.dev/kind": "tekton.dev.TaskRun",
  "knative.dev/traceid": "5240389b-fbe3-4251-afd4-280d588cc9d6",
  "knative.dev/key": "tekton-run-full-single/exec-tc-21.6-22901"
}

I also receive logs about requeing, I don't understand them because the time reported is over 50 minutes but the taskrun where in the queue for less than 20 minutes:
{
  "level": "debug",
  "ts": "2022-05-11T15:36:54.469Z",
  "logger": "tekton-pipelines-controller",
  "caller": "controller/controller.go:562",
  "msg": "Requeuing key tekton-run-full-single/exec-tc-21.6-19485 (by request) after 57m29.542598207s (depth: 0)",
  "commit": "e055233",
  "knative.dev/controller": "github.com.tektoncd.pipeline.pkg.reconciler.taskrun.Reconciler",
  "knative.dev/kind": "tekton.dev.TaskRun",
  "knative.dev/traceid": "f241ae99-6dab-4323-afa4-dee7fcbc58ce",
  "knative.dev/key": "tekton-run-full-single/exec-tc-21.6-19485"
}

That 57m requeue might be due to this code:
https://github.com/tektoncd/pipeline/blob/155179bf2de895b0ab76859b3d8aef142f0206db/pkg/reconciler/taskrun/taskrun.go#L179-L184
This enqueues a reconcile at a time after the TaskRun should have either been completed or timed out -- if it's not done at that time, it's failed as timed-out.
That re-enqueueing happens only if the TaskRun has a start time, and this code is supposed to unset .Status.StartTime if pod creation failed due to resource quotas.

I tested the change using two different Pipelines, one created 200 TaskRuns and the other ~ 6'000 in the queue. In my cluster only 65 pods could be scheduled at the same time, leaving all the others TaskRuns with .status.message:
TaskRun Pod exceeded available resources: pods "exec-tc-xxxx-pod" is forbidden: exceeded quota: tekton, requested: requests.memory=3500Mi, used: requests.memory=213700Mi, limited: requests.memory=211Gi

It resulted in a consistent number of Pods running at the same time until all TaskRun in the queue are proceeded.
I observed only a small raise in term of cpu or memory consumption for the controller pods for 6'000 TaksRuns in the queue: <0.300 cpu and < 700MiB ram.
For the test with thousands of TaskRuns in the queue, I saw that the TaskRuns took more than a minute to be completed with .status.conditions.reason|type: Succeeded. The controller seems to operate all the requeuing before moving the another action.

This issue is solved by #4881.
Many thanks to @imjasonh for your detailed explanations and fast support :satisfied:


issues/65: resuse address/port for pprof and redirect servers

fixes: https://github.com/komuw/goweb/issues/65

Codecov Report

Merging #67 (d5fa56b) into main (a7ec7f7) will decrease coverage by 1.04%.
The diff coverage is 28.35%.

:exclamation: Current head d5fa56b differs from pull request most recent head 26c2259. Consider uploading reports for the commit 26c2259 to get more accurate results

@@            Coverage Diff             @@
##             main      #67      +/-   ##
==========================================
- Coverage   70.91%   69.86%   -1.05%
==========================================
  Files          20       20
  Lines        1530     1573      +43
==========================================
+ Hits         1085     1099      +14
- Misses        408      435      +27
- Partials       37       39       +2

Impacted Files
Coverage Δ

server/server.go
35.29% <0.00%> (-4.27%)
:arrow_down:

server/pprof.go
88.70% <82.60%> (-11.30%)
:arrow_down:

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 a7ec7f7...26c2259. Read the comment docs.


Incorrect file headers

We have This file is part of the PHP CS Fixer. atm, but it should just be This file is part of PHP CS Fixer.

it was that way. there is the one tool

It's incorrect grammar because it's a proper noun.

Good point, thank you.


Fails on Windows with Guava 32.0.0

Run Jenkins 2.407 with the Artifactory plugin containing this code, on a Windows machine. That leads to exceptions because some methods in Guava 32.0.0 simply don't work anymore on Windows. Older versions of Jenkins core contained Guava 31.x and therefore work fine.
Jenkins bug: https://issues.jenkins.io/browse/JENKINS-71375
Guava change describing the non-working behavior: https://github.com/google/guava/commit/3dfe3633eb84ec96ac88c764862b0f8d0c3546fc
The Jenkins core developers recommend removing Guava from Jenkins plugins. I support that opinion.

Suggest the following patch:
diff --git a/build-info-extractor/src/main/java/org/jfrog/build/extractor/clientConfiguration/util/DependenciesDownloaderHelper.java b/build-info-extractor/src/main/java/org/jfrog/build/extractor/clientConfiguration/util/DependenciesDownloaderHelper.java
index f74280ba..a1210ac7 100644
--- a/build-info-extractor/src/main/java/org/jfrog/build/extractor/clientConfiguration/util/DependenciesDownloaderHelper.java
+++ b/build-info-extractor/src/main/java/org/jfrog/build/extractor/clientConfiguration/util/DependenciesDownloaderHelper.java
@@ -1,6 +1,6 @@
 package org.jfrog.build.extractor.clientConfiguration.util;

-import com.google.common.io.Files;
+import java.nio.file.Files;
 import org.apache.commons.io.FileUtils;
 import org.apache.commons.io.IOUtils;
 import org.apache.commons.lang3.BooleanUtils;
@@ -245,7 +245,7 @@ public class DependenciesDownloaderHelper {
     protected Map<String, String> downloadFileConcurrently(final String uriWithParams, long fileSize, final String fileDestination, String filePath)
             throws Exception {
         String[] downloadedFilesPaths;
-        File tempDir = Files.createTempDir();
+        File tempDir = Files.createTempDirectory(null).toFile();
         String tempPath = tempDir.getPath() + File.separatorChar + filePath;
         try {
             downloadedFilesPaths = doConcurrentDownload(fileSize, uriWithParams, tempPath);

As the Guava developer responsible for this, I apologize for the trouble. I endorse the patch above, and I am working to fix this in Guava for anyone who isn't in a position to make such changes.

The Guava fix is released for anyone who needs it: https://github.com/google/guava/releases/tag/v32.0.1

Thanks for the information, @cpovirk.
Are you interested in making a contribution by submitting a pull request to address this problem?


Restore doesn't work

Hello again,
I have a problem concerning the restore.
Here is the screenshot og my german version:

You helped me out once with the same problem.
Thank you,
Dirk

Hi @dslengel Are you sure this is not the same problem as in this issue #276 ?

Yes, you are right.
You've written: "You can try to drop the backup_backup table in the Mysql database and then try to open the Restore tab again. Just make a pre-backup of the table."
Please let me know, wether this here would be the right way:

Thankx

In MySQL WorkBench you need to select a "Trucate Table...", i've descired it in this comment https://github.com/ONLYOFFICE/CommunityServer/issues/276#issuecomment-864454077.

Thanks! I did it.
Why is it so, that I have to truncate this special table?
Liebe Grüße,
Dirk Büchin
Am Do., 19. Aug. 2021 um 18:41 Uhr schrieb Mike Grunin <
@.***>:

In MySQL WorkBench you need to select a "Trucate Table...", i've descired
it in this comment #276 (comment)
https://github.com/ONLYOFFICE/CommunityServer/issues/276#issuecomment-864454077
.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/ONLYOFFICE/CommunityServer/issues/328#issuecomment-902071431,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ATBJ2FOMRKC744VKCQ6TJKTT5UX37ANCNFSM5COGSMKQ
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&utm_campaign=notification-email
.

Because the problem is in one of the backup records that are stored in this table. One of the records could not be read therefore an error occurred.

Ok, thanks!
Mike Grunin @.***> schrieb am Fr. 20. Aug. 2021 um
10:40:

Because the problem is in one of the backup records that are stored in
this table. One of the records could not be read therefore an error
occurred.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/ONLYOFFICE/CommunityServer/issues/328#issuecomment-902534567,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ATBJ2FMUNDCIGUAFPA6XATDT5YIIJANCNFSM5COGSMKQ
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&utm_campaign=notification-email
.

--
Mit himmlischen Grüßen,
Dirk Büchin
DSL-Engel Dirk Büchin
Rathausstr. 4
66125 Saarbrücken
0681-6865005 (Tel)
0681-6865006 (Fax)
www.dsl-engel.de


Added SSIL post processing effect

Implements Screen Space Indirect Lighting (I still need to rename it in the code)
SSIL adds short-range indirect lighting into the scene by sampling much like SSAO. I went with the ASSAO-like approach after months of testing and experimenting with other methods (including full blown SSGI). This method strikes the best balance between performance and quality and does not require Temporal Super Sampling at all.
SSIL is intended to complement VoxelGI or SDFGI and is not a form of GI on its own. It only adds light into the scene and it only takes lighting into account in a short to medium range
As it stands, SSIL is slightly slower than SSAO on my hardware. Results will vary, but for me, it takes between 0.3-1.0 ms.
https://user-images.githubusercontent.com/16521339/128133354-743f3794-cd7f-4220-a4ff-1b6d1a72222b.mp4
TODO:
Before marking this PR as ready to merge I need to:

[x] 1. convert all references to SSGI to SSIL
[x] 2. clean up Screen Space reflections (likely I will just remove the changes to SSR and make them in another PR)
[x] 3. clean up code style (there are lots of places where the code is ugly)
[x] 4. move SSIL uniform sets into the renderbuffers
[x] 5. ~move SSIL after AO in the main scene shader~ (nevermind, this doesn't actually make sense)
[x] 6. write docs :)

After this PR is merged I need to investigate:

merging the SSIL buffer with the main GI buffer (might be nice, but limits flexibility)
generating Screen Space Bent Normals during SSIL pass (and using them for specular occlusion)
generating SSIL at quarter resolution  for performance
moving last frame texture copy to raster (so we avoid jumping back and forth between raster and compute

Lights' Indirect Energy property is not taken into account by SSIL. If this is a limitation that can't be resolved, it should be documented in the class reference.

That's right. I've added this limitation to the docs. It will also need to be reflected in a tutorial style doc

SSIL colors are broken (everything is grayscale) when Half Size is disabled in the Project Settings. Not that it's very practical for performance reasons, but still 🙂

I think I have fixed this. At any rate I can't reproduce on the most recent version

Enabling MSAA while SSIL is enabled may break rendering until you look away from any SSIL'd surface and look back.

I haven't been able to reproduce this. I will keep my eye on it though.

It seems the scene got excessively bright when approaching the material in question. This only happens if roughness is 0 or 0.01. With a roughness of 0.02 or higher, SSIL gets quite flickery but it doesn't turn the whole scene black. Maybe this can be fixed by clamping the material roughness to a minimum value (0.15 looks good from my quick testing).

There is certainly a risk of infinite feedback loops forming between surfaces that face each other. I should put a cap on energy to avoid this.

Okay updated with fixes for all the identified issues. I have solved the issue with propagating INFs and NaNs. It should be ready to go.

After spending more time thinking about this, I think I can polish it a bit more before it is ready to merge.

It seems enabling SSIL creates an SSAO-like effect automatically. Should SSAO be disabled when SSIL is enabled?

Eh, I know that technically SSIL on Ultra is broken (as it actually lights up a lightless scene on higher intensity, and also behaves differently to all the other settings), but I really love the Ultra setting effect, just have to set 0.1 intensity instead of 1 for the scene not to be lit by it without lights in a noticable way...

Just pushed documentation updates to address recent comments. If CI is happy, I think this is ready for final review and merging!

Hmm, Could SSIL maybe use Indirect Lighting by adding an extra step, instead of just using Direct Lighting to contribute to the end result?
I mean the way I see it, there are a few different possible implementations that would allow for Indirect Energy from lights to contribute to SSIL instead of just Energy. Here are some ideas

Compute a Frame where you Determine which pixel is affected by which lights, and then use the Indirect energy to multiply the output of the end result Light contribution by that amount
Compute an Extra Frame where instead of using Energy for the Light's energy, you use Indirect Energy, and mix the two results
Just render a Frame with Indirect Lighting Energy Instead of Direct Lighting Energy, and pass that to SSIL instead of the current Direct Lighting Energy that's passed to it.

I mean this could help with giving more fine-tune control over the intensity of the effect on a Per-Light basis, which would be great for artistic control

In some cases (usually with exagerrated settings) the geometry wireframe becomes visible. I wonder what could be causing this:

It's exagerrated when sharpness is turned up:

Of course this example  uses an extremely rage radius.
It seems to occur only from Medium quality upwards.
On Low:

On Medium:

It seems to occur only from Medium quality upwards.

The wireframe is present on Low too, but it's hard to discern:

BTW, I'm testing what's the limit:
https://youtu.be/O39GyzSdZfc
It seems that if emissive sources are touching the screen border - sudden light energy increase is produced.
Maybe that could be tweaked?

Is the Scene Geometry CSG Nodes? They Try disabling all gizmos and see if the geometry is still visible, or run the project..
I've noticed that it incorrectly uses in-Editor stuff in it's results, such as Skeletons, CSG node wireframes, etc...

No, the scene geometry is imported from a GLTF exported from Blender. No CSG is used.

@unfa The "wireframe" artifacts look like there may be an issue with your mesh normals along triangle edges. What does it look like when you use the "normal buffer" render mode from the perspective menu?

The environment normal pass:

Rebased locally to make sure that it still builds fine.

Thanks!


Spelling error YOu -> You

First paragraph: "YOu can also detect language, entities,"

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

ID: 8ec84065-6739-6626-3983-4e4a63ea04a1
Version Independent ID: c9339023-7753-5d37-1c03-c539c27b77af
Content: Tutorial for calling Cognitive Services APIs in an indexing pipeline - Azure Search
Content Source: articles/search/cognitive-search-tutorial-blob.md
Service: search
GitHub Login: @LuisCabrer
Microsoft Alias: luisca

Thanks for your contribution, I've addressed this issue in PR: https://github.com/MicrosoftDocs/azure-docs-pr/pull/70185.
The changes should be live EOD or tomorrow. We will now close this issue. In future, if you see any minor bug fixes in the doc, feel free to click on the edit button and submit a PR to update. we love to take community contributions.


torch1.9 torch.distributed.launch: the main node keep waiting for the other nodes

❓ Questions and Help
Before asking:

search the issues.
search the docs.

What is your question?
Hi,
I have successfully trained with fairseq-hydra-train on a single node with multiple GPUs.
And then I decided to finetune/pretrain wav2vec2.0 models on several gpu nodes. Each node contains 8 GPU of 3090.
(The torch environment I'm using is 1.9. )
However, when I start the processes on the nodes, the  main node kept waiting for store based barrier, the log on the main node looks like this:
[2021-12-17 16:36:32,821][torch.distributed.distributed_c10d][INFO] - Waiting in store based barrier to initialize process group for rank: 6, key: store_based_barrier_key:1 (world_size=16, worker_count=7, timeout=0:30:00)
[2021-12-17 16:36:32,832][torch.distributed.distributed_c10d][INFO] - Waiting in store based barrier to initialize process group for rank: 3, key: store_based_barrier_key:1 (world_size=16, worker_count=7, timeout=0:30:00)
[2021-12-17 16:36:32,842][torch.distributed.distributed_c10d][INFO] - Waiting in store based barrier to initialize process group for rank: 4, key: store_based_barrier_key:1 (world_size=16, worker_count=7, timeout=0:30:00)
[2021-12-17 16:36:32,872][torch.distributed.distributed_c10d][INFO] - Waiting in store based barrier to initialize process group for rank: 5, key: store_based_barrier_key:1 (world_size=16, worker_count=7, timeout=0:30:00)

and the log on the child node looks like this:
[INFO] 2021-12-17 16:15:54,690 api: [default] starting workers for entrypoint: python
[INFO] 2021-12-17 16:15:54,690 api: [default] Rendezvous'ing worker group
[INFO] 2021-12-17 16:15:54,690 dynamic_rendezvous: The node 'localhost_242249_0' attempts to join the next round of the rendezvous '114514'.
[INFO] 2021-12-17 16:15:57,894 dynamic_rendezvous: The node 'localhost_242249_0' has joined round 2 of the rendezvous '114514' as rank 1 in a world of size 2.
/home/shenyang/anaconda3/envs/py3.8/lib/python3.8/site-packages/torch/distributed/elastic/utils/store.py:52: FutureWarning: This is an experimental API and will be changed in future.
  warnings.warn(
[INFO] 2021-12-17 16:15:58,192 api: [default] Rendezvous complete for workers. Result:
  restart_count=0
  master_addr=localhost
  master_port=54625
  group_rank=1
  group_world_size=2
  local_ranks=[0, 1, 2, 3, 4, 5, 6, 7]
  role_ranks=[8, 9, 10, 11, 12, 13, 14, 15]
  global_ranks=[8, 9, 10, 11, 12, 13, 14, 15]
  role_world_sizes=[16, 16, 16, 16, 16, 16, 16, 16]
  global_world_sizes=[16, 16, 16, 16, 16, 16, 16, 16]

[INFO] 2021-12-17 16:15:58,192 api: [default] Starting worker group
[INFO] 2021-12-17 16:15:58,194 __init__: Setting worker0 reply file to: /tmp/torchelastic_3n9nak5r/114514_0rth0621/attempt_0/0/error.json
[INFO] 2021-12-17 16:15:58,194 __init__: Setting worker1 reply file to: /tmp/torchelastic_3n9nak5r/114514_0rth0621/attempt_0/1/error.json
[INFO] 2021-12-17 16:15:58,194 __init__: Setting worker2 reply file to: /tmp/torchelastic_3n9nak5r/114514_0rth0621/attempt_0/2/error.json
[INFO] 2021-12-17 16:15:58,194 __init__: Setting worker3 reply file to: /tmp/torchelastic_3n9nak5r/114514_0rth0621/attempt_0/3/error.json
[INFO] 2021-12-17 16:15:58,194 __init__: Setting worker4 reply file to: /tmp/torchelastic_3n9nak5r/114514_0rth0621/attempt_0/4/error.json
[INFO] 2021-12-17 16:15:58,194 __init__: Setting worker5 reply file to: /tmp/torchelastic_3n9nak5r/114514_0rth0621/attempt_0/5/error.json
[INFO] 2021-12-17 16:15:58,194 __init__: Setting worker6 reply file to: /tmp/torchelastic_3n9nak5r/114514_0rth0621/attempt_0/6/error.json
[INFO] 2021-12-17 16:15:58,194 __init__: Setting worker7 reply file to: /tmp/torchelastic_3n9nak5r/114514_0rth0621/attempt_0/7/error.json
[2021-12-17 16:16:02,034][fairseq.distributed.utils][INFO] - distributed init (rank 8): env://
[2021-12-17 16:16:02,046][fairseq.distributed.utils][INFO] - distributed init (rank 10): env://
[2021-12-17 16:16:02,070][fairseq.distributed.utils][INFO] - distributed init (rank 9): env://
[2021-12-17 16:16:02,074][fairseq.distributed.utils][INFO] - distributed init (rank 14): env://
[2021-12-17 16:16:02,108][fairseq.distributed.utils][INFO] - distributed init (rank 11): env://
[2021-12-17 16:16:02,162][fairseq.distributed.utils][INFO] - distributed init (rank 12): env://
[2021-12-17 16:16:02,195][fairseq.distributed.utils][INFO] - distributed init (rank 13): env://
[2021-12-17 16:16:02,223][fairseq.distributed.utils][INFO] - distributed init (rank 15): env://

And nothing else happens on the childnode.
I have already setup the sftp connection between two nodes.

What's your environment?

fairseq Version (e.g., 1.0 or main): main
PyTorch Version (e.g., 1.0): 1.9
OS (e.g., Linux): Ubuntu 16.04.6
How you installed fairseq (pip, source): source
Build command you used (if compiling from source):
Python version: 3.8
CUDA/cuDNN version: 11.1
GPU models and configuration: XLSR-53
Any other relevant information:

Best regards,
Yang

OK, I think I have solved the problem. fairseq-hydra-train is incompatible with the torch1.9/1.10 elastic launch. It's not necessary to  use torch.distributed.launch/run to launch the trainers.

I came acrosss this problem when the number of GPUs set by "CUDA_VISIBLE_DEVICES=4,5" (2xGPUS) doesn't match with --nproc_per_node=6   (6xGPUs) when using python -m torch.distributed.launch ,

OK, I think I have solved the problem. fairseq-hydra-train is incompatible with the torch1.9/1.10 elastic launch. It's not necessary to use torch.distributed.launch/run to launch the trainers.

Could you please elaborate on how you were able to resolve this issue? I am using hydra with torch 1.11, and getting the same errors. Waiting in store based barrier to...


Fix NameError when pg is not loaded

Most apps only load a single db adapter gem.
A MySQL app, for example will not load the pg gem.
See https://github.com/airblade/paper_trail/issues/1022

NameError introduced by https://github.com/airblade/paper_trail/pull/1018 ..
.. and we didn't catch it because our test suite loads all three adapter gems. See https://github.com/airblade/paper_trail/issues/1022


keybase: 5.0.0 -> 5.1.1

Motivation for this change
Things done

[ ] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[ ] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nix-review --run "nix-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[x] Fits CONTRIBUTING.md.

Notify maintainers
cc @

Oops, when I searched before, I didn't see this https://github.com/NixOS/nixpkgs/pull/75531.


Add builds for android, ios, windows/arm*, etc.

The complete list is at https://go.dev/doc/install/source#environment

I don't really maintain the fork for purposes of making public releases so this change is useless to me.


Problems with ^{} escape

When I use \^{} to render symbol ^, I get following result: .
Ok, then I added replaceNode for macro ^ and got some interesting things:
$a^b$ // Replaced with replaceNode
a\^{}b // Also replaced with replaceNode

Seems like not expected behaviour. Of course I found fix: check that node doesn't have escapeToken, but this looks bad:
(node: Macro) => {
    if (node.escapeToken !== undefined) {
        return node;
    }
    return s("^");
}

I expect, that ^ in equation is not assumed like macro.

Indeed, the ^ in an equation is considered a macro with escapeToken === "". That allows it to take arguments :-).
If you want to match a macro exactly, you can use match.macro(node, m("^")). Or, if you want to only replace macros in text mode, you can look in the info prop that's passed into the visitor. That contains a flag about whether the macro was found in math mode or not.

I think I need to make PR to allow \^{} work as expected out of the box. Also, I haven't run into issues with escaping other special characters, but those are worth checking too.

I finally fixed all cases with the following macro replacement:
"^": (_: Macro, info: VisitInfo) => {
    if (info.context.inMathMode) {
        return undefined;
    }
    if (info.index === undefined || !info.containingArray) {
        return undefined;
    }
    if (info.containingArray.length <= info.index + 1) {
        return undefined;
    }
    const nextToken = info.containingArray[info.index + 1];
    const content = printRaw(nextToken);
    if (content != "{}") {
        return undefined;
    }
    return s("^");
}

I checked following cases:
\^a   // renders as â
\^{a} // renders as â
\^{}a // renders as ^a

As for now I don't know how to fix this in unified-latex out of the box. @siefkenj can you help me?

I'm a little confused about what you're doing/where you're hoping for things to change. Are you talking about the ligature code for HTML conversion, or are you trying to make a custom processor?


Control car with a joystick

As a user, I want to be able to control the car using a joystick through the android application.
Acceptance criteria

[ ] User should be able to move the car forward, backward, right and left with the joystick.
[ ] The speed of the car should depend on the strength of the forward/backward motion of the joystick
[ ] The angle of the car should be consistent with the angle of the joystick

@sh4r10 removed from this issues to work in issue #69. @arrestoteles added to work with @um33 on joystick.


Style1 to patternfly conversion in Configuration Management

Parent issue: #3139
Before:

After:

@epwinchell please review

@PanSpagetka please update your PR as is described in #4197

Updated, @epwinchell @skateman please review

LGTM

Spacing fixed.


add simple auth to checkpoint ingestion and Rest API client

Description
Added simple auth to rest api client to allow connection to private fullnode
Test plan
How did you test the new or updated feature?

Release notes
Check each box that your changes affect. If none of the boxes relate to your changes, release notes aren't required.
For each box you select, include information after the relevant heading that describes the impact of your changes that a user might notice and any actions they must take to implement updates.

[ ] Protocol:
[ ] Nodes (Validators and Full nodes):
[ ] Indexer:
[ ] JSON-RPC:
[ ] GraphQL:
[ ] CLI:
[ ] Rust SDK:
[ ] REST API:

This whole interface is getting a lot of refactoring done in https://github.com/MystenLabs/sui/pull/20435 and it would be great if we could get that PR in first and then rebase this change on top.

This change is much smaller than #20435 and seems kind of orthogonal, does it make sense to priority invert them like that? Can we land a version of basic auth support to the current version of the REST API and then figure out how it should manifest in the refactored version?

Yes it is much smaller but the code changes themselves are not really the concern, it's more that the interface is going to change from a REST request to a gRPC one and auth may need to be handled differently in that world.


All editor-opened file streams have BOM prepended, breaking coding handling

Found while debugging #354. Have the file:
# -*- coding: ascii -*-
print "foo"

Definitely without a BOM:
& 'C:\Program Files\Git\usr\bin\xxd.exe' .\ascii.py
00000000: 2320 2d2a 2d20 636f 6469 6e67 3a20 6173  # -*- coding: as
00000010: 6369 6920 2d2a 2d0a 7072 696e 7420 2266  cii -*-.print "f
00000020: 6f6f 220a                                oo".

Looking at what happens in the debugger, the BOM gets appended during the reencoding process in ProjectEntry.cs.
This should be fixed to properly handle the incoming stream (not modify it?), or potentially even dropping any special stream buffering to just stick to UTF-8, which we'll have to figure out.

Per conversation with Mikhail


fixed the mobile size styling

This is the max size it can keep for two columns

Here is the "mobile size"

My PR deals with the marketplace styling as well, and hence the layout of that page is completely different. This PR will conflict with the final design, so I don't think we should go ahead with this.
For reference, this is my PR https://github.com/phreid/somethingBorrowed/pull/149


Remove dependence of aries module on VcxStateType

Signed-off-by: Miroslav Kovar<EMAIL_ADDRESS>

Codecov Report

Merging #314 (11dfd8b) into master (3c2165d) will decrease coverage by 4.12%.
The diff coverage is 36.38%.

@@            Coverage Diff             @@
##           master     #314      +/-   ##
==========================================
- Coverage   49.72%   45.59%   -4.13%
==========================================
  Files         156      154       -2
  Lines       19634    18803     -831
  Branches     6205     5890     -315
==========================================
- Hits         9763     8574    -1189
- Misses       5070     6116    +1046
+ Partials     4801     4113     -688

Flag
Coverage Δ

integration
?

unittests
45.59% <36.38%> (+0.05%)
:arrow_up:

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

Impacted Files
Coverage Δ

libvcx/src/api_lib/api_c/credential.rs
27.54% <0.00%> (ø)

libvcx/src/api_lib/api_c/credential_def.rs
34.51% <ø> (ø)

libvcx/src/api_lib/api_c/disclosed_proof.rs
34.60% <0.00%> (ø)

libvcx/src/api_lib/api_c/issuer_credential.rs
28.92% <0.00%> (-0.06%)
:arrow_down:

libvcx/src/api_lib/api_c/proof.rs
44.87% <0.00%> (ø)

libvcx/src/api_lib/api_c/wallet.rs
42.23% <ø> (-1.43%)
:arrow_down:

...aries/handlers/connection/invitee/state_machine.rs
62.55% <0.00%> (-1.95%)
:arrow_down:

...aries/handlers/connection/inviter/state_machine.rs
64.27% <0.00%> (-0.19%)
:arrow_down:

libvcx/src/api_lib/api_handle/issuer_credential.rs
48.80% <10.00%> (ø)

libvcx/src/api_lib/api_handle/disclosed_proof.rs
52.22% <14.28%> (ø)

... and 50 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 3c2165d...11dfd8b. Read the comment docs.

@mirgee the failing tests seems to indicate that the state mapping is a bit off somewhere


Add undo command

Adds undo functionality to commands that change the address book.
This includes: 'add', 'clear', 'delete', 'edit', 'editgame'
'undo' ignores commands not listed above.

Not ready to merge yet, there are currently some bugs relating to how FilteredLists


Setup Luau analysis

Closes (ISSUES HERE).
Put your pull request body here!
Checklist before submitting:

[ ] Added entry to CHANGELOG.md
[ ] Added/updated relevant tests
[ ] Added/updated documentation

Coverage remained the same at 94.783% when pulling 4fbcb212d2ef8a142cc33a7b6e4e547d7c5a832e on ghostnaps:basic-luau-analysis into 29092223a4c5cbe0c303ff07b7d1775abf164fe5 on Roblox:master.


Installation on Redhat 8.4 issue.

Hi brainiak devs!
I am trying to install brainiak on Red Hat Enterprise Linux release 8.4 (Ootpa), which is quite a challenge with the ubuntu instructions.
I managed to get the package requirements:
sudo yum install openmpi-devel python3-devel mpich python3-mpich libgomp python3-pip
sudo yum groupinstall 'Development Tools'

However, when running pip install brainiak, I run into an error in the file brainiak/factoranalysis/tfa_extension.cpython-36m-x86_64-linux-gnu.so when trying to build a wheel for brainiak (mpi4py, theano, and nitime seem to build fine):
  g++ -pthread -shared -Wl,-z,relro -Wl,-z,now -g -Wl,-z,relro -Wl,-z,now -g build/temp.linux-x86_64-3.6/brainiak/factoranalysis/tfa_extension.o -L/usr/lib64 -lpython3.6m -o build/lib.linux-x86_64-3.6/brainiak/factoranalysis/tfa_extension.cpython-36m-x86_64-linux-gnu.so -g0 -fopenmp -lirc -lintlc -DVERSION_INFO="0.11" -std=c++14
  /usr/bin/ld: cannot find -lirc
  /usr/bin/ld: cannot find -lintlc
  collect2: error: ld returned 1 exit status

I tried to find out which library I needed for -lirc and -lintlc but without success.
Is there anyone who knows?
Kind regards,
-Jelle
PS. I am using Python 3.6.8

Oh, weirdly enough, I got it working now after rebooting.


[JOSS review] Lack of community guidelines

JOSS considers it good practice to provide communite guidelines for software users who want to

Contribute to the software
Report issues or problems with the software
Seek support

Please consider adding such guidelines to the documentation.

This is a very good suggestion. I have added such guidelines in the Wiki pages. I am not very familiar of this type of information, I tried to provide the main information. Let me know if you have additional suggestions.

That looks great!


OpenVX/rocAL - remove the call to hipDeviceReset

@LakshmiKumar23 to verify it fixes the rocBLAS error in training

I do not see that rocBlas error either


continue train from binary dataset

https://github.com/microsoft/LightGBM/pull/2252#issuecomment-509637035

I looked through the fixed and comments about this issue (#2252 ) but cannot figure out why need rawdata to continue train. My raw data is really big so has to constructed into binary file, so continue train from binary is needed in my case.

LGBM_BoosterGetPredict is not for prediction for bin dataset. It is only used in training and store the training scores. After loading booster from file, there is not way to predict for bin dataset.


Update firebase commands - update pods and swift files to latest version #39

Updated to match the current Firebase framework and its documentation.

@Lino17

Accaunt welles Fargo benk scaduto

Uw

Please translate your comments into English. Not the whole world can understand Italian.

Can you update this for swift 4.0

@GurpalRajput Who? Me? And what are you talking about? The changes I made to the QuickChat folder are all in Swift 4.x. If you're experiencing issues with the Firebase Framework, you have to update the pods.


docs: Document local-id syntax

Right now, the symbol parser doesn't validate that the rest of the suffix is a simple identifier.
Should we do that too?
Fixes #205
Test plan
n/a

I've updated the parsing logic in Go and Rust to make sure that we only allow simple identifiers.
I noticed that the Rust tests weren't running in CI, so I've tweaked the CI script to fix that.


engine-schema: fix schema change for #6960

Description
Moves schema change from #6960 in the correct upgrade path
Types of changes

[ ] Breaking change (fix or feature that would cause existing functionality to change)
[ ] New feature (non-breaking change which adds functionality)
[x] Bug fix (non-breaking change which fixes an issue)
[ ] Enhancement (improves an existing feature and functionality)
[ ] Cleanup (Code refactoring and cleanup, that may add test cases)

Feature/Enhancement Scale or Bug Severity
Feature/Enhancement Scale

[ ] Major
[ ] Minor

Bug Severity

[ ] BLOCKER
[ ] Critical
[ ] Major
[ ] Minor
[ ] Trivial

Screenshots (if appropriate):
How Has This Been Tested?

Checked PR #6960 LGTM


Update astropy-helpers to v1.3.1

This is an automated update of the astropy-helpers submodule to v1.3.1. This includes both the update of the astropy-helpers sub-module, and the ah_bootstrap.py file, if needed.
This is intended to be helpful, but if you would prefer to manage these updates yourself, or if you notice any issues with this automated update, please let @bsipocz or @astrofrog know!

Coverage remained the same at 73.737% when pulling 135fe42d135c300471c29550c860ddefe048da83 on bsipocz:update-helpers-v1.3.1 into dc589b1e21f634e2d193231251537a3a3703d8f7 on radio-astro-tools:master.


Update case to match parameter name

It looks like a simple typo was made and onsuccess should be OnSuccess

Yes, a typo. Thanks


Upgrade this to new react-native version 0.71

Upgrade this to new react-native version 0.71
Environment:
React: 18.2.0
React native: 0.71.8
react-native-snap-carousel: 3.9.1

Need this...
Is this project maintained anymore ?


[pylint] Implement bad-staticmethod-argument (PLW0211)

Summary
Implement bad-staticmethod-argument from pylint, part of #970.
Test Plan
Text fixture added.

Thanks!


Add: Vigenere cipher utilities

What does the program do ?
learn what is Vigenere cipher and how to find length of key given cipher text
In what programming language it is written?
C

Thank you for participating  in hacktoberfest ❣ if you liked working on this repo you can share it with friends and star this project for helping others


Scrollbars do not re-clip after resize followed by an operation

Repro steps:

Go to http://jsfiddle.net/tZVsS/556/
Zoom multiple times (maybe 5 or 6 times)

Actual: the text inside CodeMirror gets clipped from the bottom
Expected: the text should not be clipped
Video of repro: https://imgur.com/a/pMFhnIm
Browser: Chrome 66.0.3359.181 and 69.0.3450.0
OS: Linux, Mac, possibly windows too
Hello there!
In CodeMirror, there seems to be some "scrollbar clipping" logic that helps ensure that CodeMirror's scrollbar is hidden.  Its "resize" handler seems to throttle itself every 100ms, and sets d.scrollbarsClipped = false to indicate that clipping needs to be updated soon.
In my website, I'd like to start CodeMirror operations within my own window "resize" handler, which seem to interfere with scrollbar clipping.  I'm not very familiar with CodeMirror internals, but perhaps this could be fixed just by setting:
d.scrollbarsClipped = false
inside refresh()?  Refresh already clears caches and forces an update, but it doesn't force scrollbar clipping.
Thanks for taking a look!

onResize was using the last editor size in a way that wasn't reliable. I've made it always recalculate things in attached patch.

That makes sense.  Thank you for the quick fix!


Triage View breaks here = bv.start + offset

Version and Platform (required):

Binary Ninja Version: 3.1.3553-dev
OS: macOS
OS Version: 10.15.7
CPU Architecture: x86

Bug Description:
When Triage view is set to open by default, you cannot use here = bv.start + offset to navigate (even if offset = 0)
Steps To Reproduce:
Please provide all steps required to reproduce the behavior:

Enable triage.preferSummaryView
Open a binary
Enter in the console: here = bv.start
Observe "Address 0x10000000 is not valid for the current view"
Click address in text and observe how navigation totally works

Expected Behavior:
I expected here = bv.start to work in all cases

This happens to be fixed by 7c7ceb899dfd18138584cefc8b79221210cd2e29, in which I did a navigation on the viewFrame, rather than the binary

Can confirm, this is fixed. Thanks!


Style utilities : Section color

AS your theme doc i cannot find the reason why you used yellow color for each box and red color for showing the section ?

Theme related documentation will be provided soon. please wait for further instruction. you can continue the test after you have the theme documentation file.

As of now closing this issue