Inverting stereo information when running a reaction from a SMARTS string

Describe the bug
In some cases, when running a Reaction created from a SMARTS string, the stereo information is inverted next to the reactive center while it should be retained.
To Reproduce
See below a simple way to reproduce the issue. Two reactions have been defined, one failing, one working. The difference between the two is only that the methanol moiety has an explicit H on the failing one.
from rdkit import Chem
from rdkit.Chem.rdChemReactions import SanitizeRxn, PreprocessReaction, ReactionFromSmarts, SanitizeFlags
from rdkit.Chem.rdmolops import AdjustQueryParameters
from rdkit.Chem.Draw import IPythonConsole
IPythonConsole.drawOptions.addAtomIndices = False
IPythonConsole.drawOptions.addStereoAnnotation = True

workingReaction = ReactionFromSmarts("[H:1][#8:2]-[#6:3].[F:9][c:8]1[c:7][c:6][c:5][c:11][n:10]1>>[#6:3]-[#8:2]-[c:8]1[c:7][c:6][c:5][c:11][n:10]1")
failingReaction = ReactionFromSmarts("[H:4][#6:3]-[#8:2][H:1].[F:9][c:8]1[c:7][c:6][c:5][c:11][n:10]1>>[H:4][#6:3]-[#8:2]-[c:8]1[c:7][c:6][c:5][c:11][n:10]1")

reaction = workingReaction
SanitizeRxn(reaction, sanitizeOps=SanitizeFlags.SANITIZE_ALL)
reaction.Initialize()
n_warnings, n_errors, n_reactants, n_products, labels = PreprocessReaction(reaction)

print(n_warnings, n_errors, n_reactants, n_products, labels)

for reactant in reaction.GetReactants():
    reactant.UpdatePropertyCache(False)
    Chem.SetAromaticity(reactant)

reactant1 = Chem.MolFromSmiles('O=C1C[C@H](O)CCN1') # (R) stereocenter
reactant2 = Chem.MolFromSmiles('CC(C)c1ccc(F)nc1C')
print(reaction.IsMoleculeReactant(reactant1)) # True
print(reaction.IsMoleculeReactant(reactant2)) # True
try:
    products = reaction.RunReactants((reactant1, reactant2))
except Exception as e:
    print(e)

prod = products[0][0]
Chem.SanitizeMol(prod)

print(Chem.MolToSmiles(prod))
# workingReaction (R) stereocenter => Cc1nc(O[C@@H]2CCNC(=O)C2)ccc1C(C)
# failingReaction (S) stereocenter => [H][C@]1(Oc2ccc(C(C)C)c(C)n2)CCNC(=O)C1
Chem.Draw.MolsToGridImage((prod, reactant1), subImgSize=(400,250))

Expected behavior
Given my reactant matches the reaction I would expect the stereochemistry to be retained.
Screenshots
Working Reaction and output

Failing reaction and output

Configuration (please complete the following information):

RDKit version: 2022.03.3
OS: Ubuntu 18-04
Python version (if relevant):3.8.12
Are you using conda? yes
If you are using conda, which channel did you install the rdkit from? conda-forge

My impression is you are doing something wrong:
reaction = AllChem.ReactionFromSmarts("[H:4][#6:3]-[O:2][#1:21].[F:9][c:8]1[c:7][c:6][c:5][c:11][n:10]1>>[H:4][#6:3]-[O:2]-[c:8]1[c:7][c:6][c:5][c:11][n:10]1.[F:9][#1:21]")
reaction.Initialize()
reactant1 = Chem.MolFromSmiles('[H][C@@]1(O)CCNC(=O)C1')
reactant2 = Chem.MolFromSmiles('CC(C)c1ccc(F)nc1C')
products = reaction.RunReactants((AllChem.AddHs(reactant1), reactant2))
print(Chem.MolToSmiles(AllChem.RemoveHs(products[0][0])))

Cc1nc(O[C@@H]2CCNC(=O)C2)ccc1C(C)C

Hi @wopozka,
Thank you for your input. I tried with your suggestion of Adding and Removing Hs and indeed it seems to help.
However, it seems incompatible with the SanitizeRxn(reaction, sanitizeOps=SanitizeFlags.SANITIZE_ALL). If I keep this line, I have an error with a pentavalent C (the stereogenic one). If I remove it, all works fine.
Digging a bit more, it seems the error only happens if I use the SanitizeFlags.SANITIZE_MERGEHS flag. I couldn't find exactly what it does. It seem to also aromatize molecules, but it is not relevant for my reactant here. Do you have any insight?

@MaxDNG
It is not incompatible, it is just wrong reaction SMARTS you are using. During handling the reaction some of hydrogens are copied, and if you do not handle it properly (in reaction SMARTS) then are copied to products, and that results in sanitization error. Please proviede the simplest example you meet this problem and then I can comment.

I see. Actually the reaction SMARTS was generated by MarvinJS. Do you mean we need to sanitize it and remove extraneous Hs before?
As a matter of fact, if I use this reaction: ReactionFromSmarts("[H:3][#6:1]-[#8:2][H:6].[#6:4][F:5]>>[#6:1]-[#8:2]-[#6:4]"), it works even when I use the SanitizeFlags.SANITIZE_ALL.

@MaxDNG, Using Smarts from Marvin is not good idea, it is better to create smarts by yourself. As I said, until you show me reaction SMARTS which results in broken molecule (sanitization failed) I am not able to explain why. It is in most cases SMARTS error, in the way, that you do not cover all atoms, and some of them are just copied to final molecule, which results in zanitization error.


Crashes every other run..

gulp.task('sass', function () {
	return sass('source/')
        .on('error', sass.logError)
        .pipe(gulp.dest('dist'));
});

gulp.task('default', ['build', 'serve', 'sass', 'watch']);

gulp.task('watch', function () {
	gulp.watch('source/**/*.scss', ['sass']);
});

So, the first run of my gulp sass crashes. The second time and every other time after that it's ok, but still crashing in between.
osx: 10.6.8
node: 0.12.7
npm: 3.3.4
gulp-ruby-sass: ^1.4.0
sass: 3.4.18

Thanks for the report. Seems like the intermediate directory path needs to
be escaped. I'll see what I can do.
On Tuesday, September 29, 2015, Francisco De La Cruz <
<EMAIL_ADDRESS>wrote:

gulp.task('sass', function () {
return sass('source/')
.on('error', sass.logError)
.pipe(gulp.dest('dist'));
});
gulp.task('default', ['build', 'serve', 'sass', 'watch']);
gulp.task('watch', function () {
gulp.watch('source/**/*.scss', ['sass']);
});
So, the first run of my gulp sass crashes. The second time and every other
time after that it's ok, but still crashing in between.
osx: 10.6.8
node: 0.12.7
npm: 3.3.4
[image: crash]
https://cloud.githubusercontent.com/assets/148098/10161793/7fd36d64-6674-11e5-80aa-d80eece42613.jpg
—
Reply to this email directly or view it on GitHub
https://github.com/sindresorhus/gulp-ruby-sass/issues/266.

--
Rob Wierzbowski
@robwierzbowski http://twitter.com/#!/robwierzbowski
http://github.com/robwierzbowski
http://robwierzbowski.com

Can you install ^2.0.3 and see if the issue persists?

Ok, that seems to fix that issue. It still crashes when run from the watch task, but I guess that's a whole other issue.

What's your watch task? How does it crash?
On Tuesday, September 29, 2015, Francisco De La Cruz <
<EMAIL_ADDRESS>wrote:

Ok, that seems to fix that issue. It still crashes when run from the watch
task, but I guess that's a whole other issue.
—
Reply to this email directly or view it on GitHub
https://github.com/sindresorhus/gulp-ruby-sass/issues/266#issuecomment-144207478
.

--
Rob Wierzbowski
@robwierzbowski http://twitter.com/#!/robwierzbowski
http://github.com/robwierzbowski
http://robwierzbowski.com

gulp.task('watch', function () {
     gulp.watch('source/**/*.scss', ['sass']);
 });

It crashes with the same exact error when I make changes  to my sass file.

Hey @fdelacruz, can you check the version in https://github.com/sindresorhus/gulp-ruby-sass/pull/267 and see if it fixes your issue? I was able to reproduce in an automated test, so it should work.

If it works and the code looks good to you I'll merge and release a new version.

Hi @robwierzbowski, how do I add the changes? Manually?

It is not crashing anymore which is great, but it is attempting to save the file to a ./.css/style.css which doesn't of course exist.

Even the error logging works beautifully..

Hmmm, the gulp.dest step is what handles writing... Can you post your sass
task?
On Wednesday, September 30, 2015, Francisco De La Cruz <
<EMAIL_ADDRESS>wrote:

Even the error logging works beautifully..
[image: error1]
https://cloud.githubusercontent.com/assets/148098/10209599/3d84cdd0-67ab-11e5-9cf7-affeb4d1f45d.jpg
—
Reply to this email directly or view it on GitHub
https://github.com/sindresorhus/gulp-ruby-sass/issues/266#issuecomment-144575547
.

--
Rob Wierzbowski
@robwierzbowski http://twitter.com/#!/robwierzbowski
http://github.com/robwierzbowski
http://robwierzbowski.com

Sure
gulp.task('sass', function () {
    return sass('source/')
        .on('error', sass.logError)
        .pipe(gulp.dest('dist'));
});

Ah, you cannot use a directory source any more. See the new read me. Try a
glob source like source/**/*.scss.
On Wednesday, September 30, 2015, Francisco De La Cruz <
<EMAIL_ADDRESS>wrote:

Sure
gulp.task('sass', function () {
return sass('source/')
.on('error', sass.logError)
.pipe(gulp.dest('dist'));
});
—
Reply to this email directly or view it on GitHub
https://github.com/sindresorhus/gulp-ruby-sass/issues/266#issuecomment-144594024
.

--
Rob Wierzbowski
@robwierzbowski http://twitter.com/#!/robwierzbowski
http://github.com/robwierzbowski
http://robwierzbowski.com

Ok, that fixed the problem.
One small thing, the log keeps saying it is writing to ./style.css which in this at least it's not. It is writing to ./dist/style.css
Shouldn't it say that??

The Sass gem doesn't know the directory gulp.dest will write the files to. It is writing these files to a temporary directory, and the gulp plugin removes the temporary directory from the Sass gem output for cleaner logging. So this part is works as designed.
I may add an error when a user passes a directory as a source. More than one person has missed that change between GRS 1.x and 2.x.
Looks like everything has been addressed. I merged the fix and released as 2.0.4. If there's anything else please reopen or start a new issue, and thanks for working through the bug with me.

Awesome, thank you for all the good work, time and effort @robwierzbowski in looking into this.

No problem.

Hate to open this up again, but I'm having trouble with the updated version!
I am still trying to figure out what is wrong but I get this error:

Here is gulp task
var files = {
    foundation: [
        './bower_components/font-awesome/scss',
        './src/assets/scss/vendors/animate',
        './src/assets/scss/vendors',
        './src/assets/scss'
    ],
    all: [
        './src/assets/scss/**/*.scss'
    ]
};
return $.rubySass('./src/assets/scss/_main.scss', {
                loadPath: files.foundation,
                style: isProd ? 'compressed' : 'nested',
                sourcemap: !isProd ? true : false,
                bundleExec: true
            })
            .on('error', $.rubySass.logError)
            .pipe($.cached('scss'))
            .pipe($.autoprefixer(prefixes))
            .pipe($.remember('scss'))
            .pipe($.concat('exchange-' + pkg.version + '.css'))
            .pipe($.if(!isProd, $.sourcemaps.write('.')))
            .pipe(gulp.dest('./dist/assets/css/'));

If anyone else is having this problem please let me know!

Nevermind, figured it out. Got rid of the _main and turned it into main.scss and that fixed it. Sorry!


Calling ProgressBar.complete() exhausts all available memory

Simple example:
import peschke.console.progressbar.ProgressBar
import scala.concurrent.ExecutionContext.Implicits.global

object Test extends App {
  val pb = ProgressBar(totalCount = 20000)
  pb.complete()
  Thread.sleep(9000000)
}

Expected: Constant memory usage
Actual: Explosion 💥

False alarm: the version of this library I found copypasted in our codebase had this line changed from .takeUntil to .takeWhile. With dire consequences for my sanity.

Yep, that would do it.
I'd highly recommend using JitPack.io to avoid copy-paste errors.
I just pushed version 1.4.0, which switches to using this instead of an SBT dependency internally, and it works really well. The first time sbt update is run for a particular version it'll probably timeout, as it needs to build the jar, but after that there doesn't seem to be a noticeable difference in speed between that and the other dependencies.


No pagination no categories just what you see is what you get.

Hey guys! I upgraded to your latest code and I am using the theme_material theme and everything is great you all do wonderful work. The only problem I have is this. http://sid.dontexist.dynu.com as you can see the user cant view different categories, the menu only has "home" and there is no pagination on the front page its just what you see is what you get. Am I missing something? Also where can I get new themes? I see that in the admin panel I can pick plenty of different themes for the admin but where do I get themes for the client side to see and what exactly is the proper method to select them. I went in config and changed default theme but I was wondering if thats the right way to do it because I had a typo and I had to restore my database from backups! Thanks!

I reverted back to the pure theme and just changed it a bit.


C++-ize slice

Should this be an issue in the gRPC issue tracker?
Yes
What version of gRPC and what language are you using?
Core, C++
What operating system (Linux, Windows, …) and version?
N/A
What runtime / compiler are you using (e.g. python version or version of gcc)
N/A
What did you do?
Read the code for grpc_slice
What did you expect to see?
Clean idiomatic C++ code
What did you see instead?
C-style code with manually-inlined vectors
Anything else we should know about your project / environment?

Reassigning to @arjunroy and I think that he has mostly or completely completed all of this anyway.


Zed on Windows can't connect to LSP

I know this is early days for Zed on Windows but I figured I'd try it out. Honestly it looks so good!

I installed netcat through scoop so that nc.exe is available on path. Here is the error from the console:
[2024-05-28T21:16:22+10:00 ERROR lsp] Cancelled LSP request task for "initialize" id 0 which took over 120s
[2024-05-28T21:16:22+10:00 ERROR project] failed to start language server "gdscript": LSP request timeout
[2024-05-28T21:16:22+10:00 ERROR project] server stderr: Some("")
[2024-05-28T21:16:22+10:00 ERROR util] crates\lsp\src/lsp.rs:782: oneshot canceled
[2024-05-28T21:16:23+10:00 ERROR util] crates\language\src\language_registry.rs:880: server container removal

Caused by:
    The system cannot find the file specified. (os error 2)

It does hit the LSP as I get this in Godot:
[LSP] Connection Taken
[LSP] Disconnected

I don't have a Windows machine, so I am afraid I can't be of any help here, if you find a way to solve it open a PR.

Starting zed from terminal made the executables in PATH accessible from zed, making the LSP work

@mrtheduts @grndctrl I fixed it here > #7

Merged @mrtheduts pull

FYI this seems to work better with ncat.exe than nc.exe
I installed nmap which comes with ncat.exe and that works great. nc.exe used to work but for some reason doesn't anymore.


chore: update dashboard image for apprunner

update image for aws app-runner

@aswanson-nr , Hi, I was asked to see if we can go ahead and push this through, I thought you might be the one to review since you reviewed the main PR. It's only an image change, looks like we had the wrong image for the dashboard.
cc @4demos


Update 3DPrintServer.md

Documentation update to describe how to connect custom 3d printers with the raspberry pi 3 solution

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.:x: mamanian sign nowYou have signed the CLA already but the status is still pending? Let us recheck it.

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

windows-iotcore/extend-your-app/3DPrintServer.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

windows-iotcore/extend-your-app/3DPrintServer.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


Added a Chess AI + Game

The project seems to be nice but, there is an issue when there is a Stalemate the game doesn't show Draw,
Also, can you structure the assets of the game into another folder?
Include proper documentation on the Python topics used in the Game you have created in the Readme file instead of just Installation.

The project seems to be nice but, there is an issue when there is a Stalemate the game doesn't show Draw, Also, can you structure the assets of the game into another folder? Include proper documentation on the Python topics used in the Game you have created in the Readme file instead of just Installation.

Stalemate seems to be working (tests with https://www.chess.com/blog/JCH2021/the-fastest-stalemate-possible-in-chess). Other changes have been made.

The project seems to be nice but, there is an issue when there is a Stalemate the game doesn't show Draw, Also, can you structure the assets of the game into another folder? Include proper documentation on the Python topics used in the Game you have created in the Readme file instead of just Installation.

Stalemate seems to be working (tested with https://www.chess.com/blog/JCH2021/the-fastest-stalemate-possible-in-chess). Could you mention how to replicate the issue you faced? Other changes have been made.

When all pieces get eliminated. It should be draw due to insufficient pieces. But here we are still able to move both kings.

Ah yes. This isn't stalement and other draw case (such as this one) haven't been accounted for as I simply don't know how to implement a good solution for them :P
Any ideas?

Oh my bad not stalemate but draw. I will have to look into it. Nice work by the way.

Oh my bad not stalemate but draw. I will have to look into it. Nice work by the way.

Thank you <3
If you do find a solution, lmk via<EMAIL_ADDRESS>or you could send a PR to github.com/nnisarggada/NyChess
Thank you for the merge :D


Hide unlisted in recs until LH is sorted out

Issue
Recs are showing unlisted if I'm viewing a similar-named content by the same creator
Change
Filter by tag for now.

Is this happening for newer content or older? Older content still may show, but newer uploads as of yesterday shouldn't. Should be safe to hide also, but would be good to understand.

oooh ... it was old content.


update issue

Changes made by ourselves,whether it be overwritten when the update.thanks

I'm not sure what you're asking here @ChengVincent
Can you clarify, please?


added Lilo and Stitch Ohana quote

"Ohana means family. Family means no one is left behind or forgotten."

@HaileyB0311 your branch contains merge conflicts, pls resolve and pr again.


Add version tag for bower

Hi @dfcreative,
There are missing version on bower due to not tagging releases (same as in  #59)
Could you make appropriate tags, please?
git clone https://github.com/kudago/smart-app-banner.git

git tag -a v1.3.1 -m "Version 1.3.1" bd25a33aa0924cd80d78a5231e01734e0bcf0aeb
git push origin v1.3.1

git tag -a v1.4.0 -m "Version 1.4.0" 3af7a66780d40fbba129d4ca9b9cf790e38a9f65
git push origin v1.4.0

Thank you  :smiley:

Same issue here. I would need features from version 1.5.0, so updating the tags would be nice 👍


Wrap TCP request reading in async_timeout.timeout()

Fixes: https://github.com/robbinjanssen/home-assistant-omnik-inverter/issues/116
There have recently been reports of RuntimeError: coroutine ignored GeneratorExit which are presumably triggered by timeouts on the socket, without being raised as TimeoutError.  This case seems to be mitigated by timing out the request after a default of 10 seconds which doesn't give GeneratorExit - triggered by something yet unknown - enough time to surface.

Converting this to draft as I still need to double-check if the blocking happens in writer.drain() or perhaps even in open_connection.

Stop with force pushing, you make it impossible to review your pull request.


Network Locations or equivalent of This PC

What's the Problem?
When using Files vs File Explorer, I noticed that there is no way to view the "Network Locations". This is where we place all Department Share shortcuts at our enterprise, so having this feature would be amazing. It could drastically increase adoption at our organization.
Solution/Idea
I suggest to add another tab in the sidebar for Network locations.
Alternatives
An alternative would be to add a tab similar to This PC in File Explorer that would show the Network locations on a PC.
Priorities

Capability
Priority

This proposal will allow developers to accomplish W
Must

This proposal will allow end users to accomplish X
Should

This proposal will allow developers to accomplish Y
Could

This proposal will allow end users to accomplish Z
Won't

Files Version
<IP_ADDRESS>
Windows Version
10.0.19044.1348
Comments
No response

Could you show a screenshot of windows explorer showing this?

Hopefully this gives a clear picture of what I am referrring to. It is best practice to use shortcuts instead of mapped drives these days, so we push these shortcuts to all our users for accessing department shares instead of mapping drives.

This issue should be resolved with #7185


Use more compatible flag name

This flag name was changed without comment in 6e1ff781e062b4e01c7da347e7a3f74c60040c2c. It has caused a few problems:

https://github.com/commercialhaskell/stack/issues/3345
https://github.com/fpco/stackage/issues/2755
https://github.com/fpco/stackage/issues/2759
https://github.com/fpco/stackage/issues/2842
https://github.com/haskell/cabal/issues/4686
https://github.com/hvr/cassava/issues/150

Those problems either caused or accelerated some (attempted) changes in Cabal:

https://github.com/haskell/cabal/pull/4654
https://github.com/haskell/cabal/pull/4687
https://github.com/haskell/cabal/pull/4696

Those problems also caused a change in Stack:

https://github.com/commercialhaskell/stack/pull/3349

In short: Cabal never had any trouble with this. Stack did. The current master version of Stack can handle flags like this, but no released versions of it can. It's not clear when the next version of Stack will be released. In the meantime, no Stack users can use this package. This commit changes the offending flag to something that Stack can handle. By using a flag that Stack can handle, Stack users can once again use this package, and it can return to Stackage. There are no apparent downsides to using a more compatible flag name.

Thank you for taking some time on this one 💯

@hvr what can I do to help move this along?

Stack users can once again use this package

I don't think that's true. Stack users can use this package just fine. LTS 9.6 has https://www.stackage.org/lts-9.6/package/cassava-<IP_ADDRESS> wheras Nightly 2017-09-29 has https://www.stackage.org/nightly-2017-09-29/package/cassava-<IP_ADDRESS>

That's... deliberately missing the point. The flag only changed in version <IP_ADDRESS>, so of course Stack users can install versions before that. Stack users cannot install the latest version of this package because the flag name is invalid for all released versions of Stack.
> stack --version
Version 1.5.1 x86_64 hpack-0.17.1

> stack --resolver lts-9.0 build cassava-<IP_ADDRESS>
Error: While constructing the build plan, the following exceptions were encountered:
In the dependencies for cassava-<IP_ADDRESS>:
    bytestring-<IP_ADDRESS> must match >=0.9.2 && <0.10.4 (latest applicable is <IP_ADDRESS>)
needed for unknown reason - stack invariant violated.
Plan construction failed.

> stack --resolver lts-9.0 build --flag cassava:-bytestring--LT-0_10_4 cassava-<IP_ADDRESS>
option --flag: Invalid flag name: bytestring--LT-0_10_4

Why is it important that the flag name be like this?

Let me turn the burden of proof around, and ask why does it take so long for Stack to make a point release even though this bug in Stack has been known for over a month already, when I made it quite clear in https://github.com/commercialhaskell/stack/issues/3345#issuecomment-322395708 that Stack's the place where this ought to be fixed. So I'd suggest you try shaming Stack dev into releasing a point release instead.

I agree that it would be nice for Stack to cut a release that supports these flag names. However I think it's clear that it's easier to change a flag name in Cassava than it is to release a new version of Stack.
Regardless, that doesn't explain why this flag name has to be the way it is. What was wrong with the old flag (or my proposed flag)?

For anyone trying to install cassava-<IP_ADDRESS> with Stack 1.5.1, I released my fork on Hackage as Cassava-<IP_ADDRESS>. You can install it like so:
> stack --resolver lts-9.0 build --flag Cassava:-bytestring-LT-0_10_4 Cassava-<IP_ADDRESS> text-short-0.1.1

Other than the flag name, it's identical to cassava-<IP_ADDRESS>.

@hvr Even if stack fixes this, older versions of stack will still not work with this package.  Not all distributions and users keep up to date.
Your criticism of stack's release cycle seems hilarious considering cabal-install's history of releases.  Even so, I'm not sure we will "fix" this at all.  IMHO there should be a single set of rules for package names and flag names.  Less to remember.  So, IMHO, this is an oversight in cabal's spec and implementation.  In order to make it so that most current build configurations will build with prior versions of stack, I think it makes sense to keep the restriction in stack.
Ideally, to maintain compatibility between the tools, cabal should add the restriction.  I doubt this will break many existing packages, because it is such a strange corner case.
I have seen you give no justification for this flag name.  It seems that its purpose is solely to break compilation with stack.  While at times the relationship is tenuous, I think overall there has been an amicable relationship between hackage / cabal-install and stackage / stack.   From our perspective, this is a petty and childish action on your part which partially compromises future amicable collaboration.  We will gladly continue collaborating with community members who are more reasonable.
Please reconsider your decision to flagrantly disregard the stack use-case.  It is not too late to restore good will.
I realize that you may draw an analogy to many packages not having upper bounds.  However, many intelligent people, perhaps most, will agree there are good reasons to omit upper bounds.  On the other hand, there are no good reasons to have a double hyphen in a flag name.  The only purpose here is to break usage of this package with stack.

After some discussion with others via email and slack, the following is now clear:  @hvr has a tool that generates these flag names, and likely did not initially realize that they would cause problems for stack
So, my attribution of definitive malice to this choice seems to be inaccurate.  Sorry for tossing around accusations.  With the comments here, HVR was making it quite hard to not attribute malice.  From the perspective of a stack developer, stubbornness to just change an inconsequential name seems quite malicious.  At the very least, not very friendly.
FWIW, typical stack configurations are currently quite forwards and backwards compatible.  The very first released version of stack can still build the stack of a few months ago, and vice versa.  It probably works the other way too.  While this certainly isn't guaranteed, it really sucks to have some packages unnecessarily break this property.
I agree that this is a bug in stack.  However, by not working around this bug, you are implicitly saying that you don't care about the users of these packages.  Or at least, not the ones that use older versions of stack.

Programmers :)

Man... was releasing Cassava-<IP_ADDRESS> (note: capital C) really the right move here? Heh. If you had to grab this package in a pinch I'm sure a non-Hackage dependency would have sufficed until you could get this issue sorted out. But now we have two different [cC]assava packages forever. Confused emoji face.

@tfausak says via email, posted w/ his permission:

If cassava fixes the flag name, I'll gladly deprecate Cassava in favor of it. It'll be just like the cabal package.

Its unfortunate that there is an impasse of such massive proportions between the stack devs and @hvr.

Dumb question:
Would changing the flag actually fix anything if stack has to parse the old files anyways?
It seems based on cursory observation that once any version of any package with a flag name that stack didn't understand made its way into the index, it seems stack is just hosed, no?
I'd be happy to be wrong here.


Mathematical expressions are marked correct if they aren't

I have an inequality I'd like the students to simplify, but it does something unexpected.

Something like 5>=x also gets marked as correct, while something as -3>=x gets marked as incorrect.

The question itself: https://numbas.mathcentre.ac.uk/question/84481/test-ongelijkheden/

I think I found my mistake: I have to use the accuracy tab to fix this.
Is it possible to let it check all integers between -10 and 10 (once) though?

Marking inequalities is difficult: the standard marking algorithm checks numerically, which doesn't work particularly well for boolean expressions like this.
You can check a fixed set of values with a custom marking algorithm, redefining the vset note to contain all the values of x you want to check:
vset:
   map(["x": x], x, -10..10)

But that doesn't work when the student has a non-integer on the right hand side of the inequality.
This question is really about the form of the student's answer, so you should use a pattern restriction. A pattern of
x <= (`+- $n);rhs

would check that the student has simplified the inequality as far as you know they can. You can then set Numbas to check only the number part, which will require the student's number to be exactly the same number as in your expected answer.
I've made a copy of your question which does this at https://numbas.mathcentre.ac.uk/question/share/view/e1a6230b-acea-4aea-a57c-68ebf5e296a9

Marking inequalities is difficult: the standard marking algorithm checks numerically, which doesn't work particularly well for boolean expressions like this.
You can check a fixed set of values with a custom marking algorithm, redefining the vset note to contain all the values of x you want to check:
vset:
   map(["x": x], x, -10..10)

But that doesn't work when the student has a non-integer on the right hand side of the inequality.
This question is really about the form of the student's answer, so you should use a pattern restriction. A pattern of
x <= (`+- $n);rhs

would check that the student has simplified the inequality as far as you know they can. You can then set Numbas to check only the number part, which will require the student's number to be exactly the same number as in your expected answer.
I've made a copy of your question which does this at https://numbas.mathcentre.ac.uk/question/share/view/e1a6230b-acea-4aea-a57c-68ebf5e296a9


wait for Quartz to shutdown

also try to interrupt any interruptable jobs

addresses Quartz on 4.1 for #1458

Shouldn't these properties be added (yet commented) in the cas.properties file?

This looks great. One last comment following @leleuj that I think it would be good to explicitly call these properties out in cas.properties, but have them be commented out.

@frett Would you be able to apply the last few changes to this patch today? This is holding up the 4.1.x release.

done

Perfect. Thanks.


feat: Add Embedding Picker to Linear UI

@blessedcoolant ill defer to you for the internal changes in this PR, I'm not sure how to test them (not sure my TIs are actually working 😅 )


Test for preview vercel

Test that preview page has created... And I found it successfully created so that teammates can see the preview version of new website based on their modification..


Move demo docker image to docker hub.

Find place for public demo image. Use docker hub ExaWorks organization. Get credentials.
This is the image used in the pdf2wordlcoud example.

Moved to ExaWorks


Update sharepoint-server.md

Updating with new format.

Docs Build status updates of commit 910c338:
:clock10: Full build: the average full build time is 12 min(s) 3 sec(s), this is based on the last 658 build(s) for this repository.

Docs Build status updates of commit 910c338:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

sharepoint/sharepoint-ps/sharepoint-server/sharepoint-server.md
:white_check_mark:Succeeded
View (sharepoint-ps)

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.

Docs Build status updates of commit 00fce6d:
:clock10: Full build: the average full build time is 11 min(s) 59 sec(s), this is based on the last 702 build(s) for this repository.

Docs Build status updates of commit 00fce6d:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

sharepoint/sharepoint-ps/sharepoint-server/sharepoint-server.md
:white_check_mark:Succeeded
View (sharepoint-ps)

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


pyqtgraph stopped working in Spyder

My pyqtgraph windows suddenly stopped working inside Spyder.
What steps will reproduce the problem?
Inside an ipython console in Spyder do

import numpy as np
import pyqtgraph as pg
pg.image(np.eye(3))

What is the expected output? What do you see instead?
The expected output is a new popup window, holding a pyqtgraph image set to the array.
Instead, a black window with a title bar is shown. When I close the window, an error message pops up, stating, that the:
"Application is not responding."
and giving me two options: Force Quit or Wait (gnome3 default probably).
When I force quit, the embedded Ipython Console gives me a
"Kernel died, restarting"
message.
Please provide any additional information below
To start spyder, I have to
unset QT_API
(which is otherwise set to pyqt and leads to the error "qtpy.PythonQtError: No Qt bindings could be found" when running spyder.)
Versions and main components
Installed via Anaconda:

Spyder Version:  3.2.3
Python Version:  2.7.13
Qt Version:         5.6.2
PyQt Version:    5.6.0
Operating system: Ubuntu 16.10

Dependencies
IPython >=4.0;<6.0:  5.3.0 (OK)
cython >=0.21     :  0.26 (OK)
jedi >=0.9.0      :  0.10.2 (OK)
nbconvert >=4.0   :  5.2.1 (OK)
numpy >=1.7       :  1.12.1 (OK)
pandas >=0.13.1   :  0.20.3 (OK)
psutil >=0.3      :  5.2.2 (OK)
pycodestyle >=2.3 :  2.3.1 (OK)
pyflakes >=0.5.0  :  1.6.0 (OK)
pygments >=2.0    :  2.2.0 (OK)
pylint >=0.25     :  1.7.2 (OK)
qtconsole >=4.2.0 :  4.3.1 (OK)
rope >=0.9.4      :  0.9.4 (OK)
sphinx >=0.6.6    :  1.6.3 (OK)
sympy >=0.7.3     :  1.1.1 (OK)

This has to do with #4349. Within Qt you can only have one instance of QApplication. But Spyder is already a QApplication so this can interfere. You could add some code to check if there is already a QApplication running with the help of:
def run():     if not QApplication.instance():         app = QApplication(sys.argv)     else:         app = QApplication.instance()      app.exec_()
or depending on how you set up your program you could also add it in your init method see Comment on #4349

I am not sure if this really is true. Until this day everything worked fine and I don't see anything changed in my setup over the day. I am not running a QT application inside Spyder. I just merely execute the three given lines on a new started Spyder and it leads to this behaviour.

Now I understand what you mean. I did run it without Spyder and the same problem occurred.

If I run in it in a pure python shell outside Spyder, it works without problems.

Please run this command after starting an IPython console
%gui qt5
then run your code.

Can I ask what %gui qt5 does?

Sure, it activates the Qt5 event loop, so that so you can run Qt apps in the IPython consoles without blocking it and making the apps fully interactive at the same time.

Works for me in an IPython shell. As the python shell is going to be abandonned in future Spyder releases (?) I consider this done.


Refactor createPath method

remove unnecessary else statements for clearer intentions and less complexity

Submitted under https://github.com/Azure/azure-storage-php/pulls


Error during update

Following the recommended update procedure:
~/.zprezto git pull && git submodule update --init --recursive
Already up-to-date.
fatal: reference is not a tree: d0381d859c0eb8d52a6211281e622e89153ef812
Unable to checkout 'd0381d859c0eb8d52a6211281e622e89153ef812' in submodule path 'modules/history-substring-search/external'

Same here. Updating prezto (gfm upstream master) and submodules (gSI) throws the same error here.

When I clone from scratch, I don't have a problem initializing modules. At what commit were you before trying to update; so, that I may attempt to replicate it?

Getting the same issue when trying to clone from scratch:
Cloning into '/users/pzuraq/.zprezto'...
remote: Counting objects: 8495, done.
remote: Total 8495 (delta 0), reused 0 (delta 0), pack-reused 8495
Receiving objects: 100% (8495/8495), 1.69 MiB | 445.00 KiB/s, done.
Resolving deltas: 100% (3760/3760), done.
Checking connectivity... done.
Submodule 'modules/completion/external' (https://github.com/zsh-users/zsh-completions.git) registered for path 'modules/completion/external'
Submodule 'modules/history-substring-search/external' (https://github.com/zsh-users/zsh-history-substring-search.git) registered for path 'modules/history-substring-search/external'
Submodule 'modules/prompt/external/agnoster' (https://gist.github.com/3712874.git) registered for path 'modules/prompt/external/agnoster'
Submodule 'modules/prompt/external/powerline' (https://github.com/davidjrice/prezto_powerline.git) registered for path 'modules/prompt/external/powerline'
Submodule 'modules/prompt/functions/pure' (https://github.com/sindresorhus/pure.git) registered for path 'modules/prompt/external/pure'
Submodule 'modules/syntax-highlighting/external' (https://github.com/zsh-users/zsh-syntax-highlighting.git) registered for path 'modules/syntax-highlighting/external'
Cloning into 'modules/completion/external'...
remote: Counting objects: 2235, done.
remote: Total 2235 (delta 0), reused 0 (delta 0), pack-reused 2235
Receiving objects: 100% (2235/2235), 888.25 KiB | 335.00 KiB/s, done.
Resolving deltas: 100% (1288/1288), done.
Checking connectivity... done.
Submodule path 'modules/completion/external': checked out '68c949470eaa87c5d67080d32fb2b69c8b837eb4'
Cloning into 'modules/history-substring-search/external'...
remote: Counting objects: 261, done.
remote: Compressing objects: 100% (3/3), done.
remote: Total 261 (delta 0), reused 0 (delta 0), pack-reused 258
Receiving objects: 100% (261/261), 78.24 KiB | 112.00 KiB/s, done.
Resolving deltas: 100% (128/128), done.
Checking connectivity... done.
fatal: reference is not a tree: d0381d859c0eb8d52a6211281e622e89153ef812
Cloning into 'modules/prompt/external/agnoster'...
remote: Counting objects: 71, done.
remote: Compressing objects: 100% (62/62), done.
remote: Total 71 (delta 23), reused 38 (delta 8), pack-reused 0
Unpacking objects: 100% (71/71), done.
Checking connectivity... done.
Submodule path 'modules/prompt/external/agnoster': checked out '43cb371f361eecf62e9dac7afc73a1c16edf89c7'
Cloning into 'modules/prompt/external/powerline'...
remote: Counting objects: 22, done.
remote: Total 22 (delta 0), reused 0 (delta 0), pack-reused 22
Unpacking objects: 100% (22/22), done.
Checking connectivity... done.
Submodule path 'modules/prompt/external/powerline': checked out '8e81152340c4beb2d941340d1feb2dc29bbcc309'
Cloning into 'modules/prompt/external/pure'...
remote: Counting objects: 347, done.
remote: Total 347 (delta 0), reused 0 (delta 0), pack-reused 347
Receiving objects: 100% (347/347), 250.28 KiB | 54.00 KiB/s, done.
Resolving deltas: 100% (153/153), done.
Checking connectivity... done.
Submodule path 'modules/prompt/external/pure': checked out 'ef1329909a3c72bb76765ccc519214ce60cedddb'
Cloning into 'modules/syntax-highlighting/external'...
remote: Counting objects: 828, done.
remote: Total 828 (delta 0), reused 0 (delta 0), pack-reused 828
Receiving objects: 100% (828/828), 140.66 KiB | 173.00 KiB/s, done.
Resolving deltas: 100% (440/440), done.
Checking connectivity... done.
Submodule path 'modules/syntax-highlighting/external': checked out '3dc574190071ef59054f6d693132410b37aa3ae4'
Unable to checkout 'd0381d859c0eb8d52a6211281e622e89153ef812' in submodule path 'modules/history-substring-search/external'

I do not know. It works here with the latest git. Try cd into that module then checkout master. Then update it.

My computer is running Mavericks, could that be an issue?

~/.z/m/h/external git:master~1 gco master
Previous HEAD position was 9f9fc7d... README: MacOSX 10.9 users need to bind ^[[A/B too
Switched to branch 'master'
Your branch is behind 'origin/master' by 7 commits, and can be fast-forwarded.
  (use "git pull" to update your local branch)
~/.z/m/h/external git:master git pull
Updating 3c7b079..6d7bc71
Fast-forward
 README.md                        |  8 ++++++++
 zsh-history-substring-search.zsh | 34 +++++++++++++++++++++++++++++-----
 2 files changed, 37 insertions(+), 5 deletions(-)
~/.z/m/h/external git:master cd ../../..
~/.zprezto git:master git pull && git submodule update --init --recursive
Already up-to-date.
fatal: reference is not a tree: d0381d859c0eb8d52a6211281e622e89153ef812
Unable to checkout 'd0381d859c0eb8d52a6211281e622e89153ef812' in submodule path 'history-substring-search/external'

This is not a platform-specific issue.

Fixed in c98da90.


The sync fails with: Unrecognized field "migrated"

Log entry looks like:
2023-03-06 13:35:50,205 INFO [org.bf2.cos.fleetshard.sync.client.FleetShardObservabilityClient] (main) Creating Observability resource
2023-03-06 13:35:50,937 ERROR [io.quarkus.runtime.Application] (main) Failed to start application (with profile prod): com.fasterxml.jackson.databind.exc.UnrecognizedPropertyException: Unrecognized field "migrated" (class com.redhat.observability.v1.ObservabilityStatus), not marked as ignorable (6 known properties: "lastSynced", "lastMessage", "clusterId", "tokenExpires", "stage", "stageStatus"])
at [Source: (BufferedInputStream); line: 1, column: 2510] (through reference chain: com.redhat.observability.v1.Observability["status"]->com.redhat.observability.v1.ObservabilityStatus["migrated"])

This is related with the latest changes in the Observability operator where a new field Migrated was introduced

fixed by https://github.com/bf2fc6cc711aee1a0c2a/cos-fleetshard/pull/612


Support for Dom element as a selector

So I can do this:
var canvidControl = canvid({
selector : document.querySelector(".myelement"),
videos: {
clip1: { src: 'clip1.jpg', frames: 38, cols: 6 },
clip2: { src: 'clip2.jpg', frames: 43, cols: 6, loops: 3 }
},
width: 500,
height: 400,
loaded: function() {
canvidControl.play('clip1');
// reverse playback
// canvidControl.play('clip1', true);
}
});
I have multiple videos on page and all of them have same selectors.

Good idea!

should work now https://github.com/gka/canvid/blob/master/canvid.js#L24


my flutter is crashing

C:\Users\ok\Desktop\try\hello>flutter run
Using hardware rendering with device Android SDK built for x86. If you get graphics artifacts, consider enabling
software rendering with "--enable-software-rendering".
Launching lib/main.dart on Android SDK built for x86 in debug mode...
Initializing gradle...                                                 |
Oops; flutter has exited unexpectedly.          Sending crash report to Google.
Crash report sent (report ID: 7c356d2319e6ecd1)
Initializing gradle...                                                                                \Crash report written to C:\Users\ok\Desktop\try\hello\flutter_02.log;
please let us know at https://github.com/flutter/flutter/issues.

@taend we need the contents of flutter_02.log to help

Without additional information, we are unfortunately not sure how to resolve this issue. We are therefore reluctantly going to close this bug for now. Please don't hesitate to comment on the bug if you have any more information for us; we will reopen it right away!
Thanks for your contribution.


Document Typo

[Enter feedback here]

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

ID: e6fbc978-36aa-d15b-c770-e9140fb89f79
Version Independent ID: ba3e2bc4-d812-c405-15cc-02660b374676
Content: Tutorial: Designer - deploy no-code models - Azure Machine Learning
Content Source: articles/machine-learning/tutorial-designer-automobile-price-deploy.md
Service: machine-learning
Sub-service: core
GitHub Login: @likebupt
Microsoft Alias: keli19

Thanks for the feedback! We are currently investigating and will update you shortly.


post description not shown

Forst of all, awesome work!
But I miss the "description" text, its not shown anywhere..
e.g.
---
layout: post
title: "health-check external disk"
description: "Hard disk health is measured using S.M.A.R.T. (if available) and bad blocks scan."
category: Linux
---

The theme doesn't use description on posts or pages. Use excerpt instead.
sample-post.md
---
layout: post
title: Sample Post
excerpt: "Just about everything you'll need to style in the theme: headings, paragraphs, blockquotes, tables, code blocks, and more."
---

Which will then populate a excerpt in post indexes and in the various meta tags in the <head>.

Great, can I show the excerpt on the post itself as well?

No, only the entire post content is shown. If you want that feature you'll need to hack up the post.html layout to meet your needs.

Awesome, works like a charm, thanks alot!


createChatCompletion endpoint msg duplication

Node is createChatCompletion endpoint is duplicating messages due to _msgid property reuse.

Fixed in v 0.8.0


fix(ui): table data issue in entity table lineage sidebar

Describe your changes:
Fixes #13880

I worked on fixing table data issue in entity table lineage sidebar

https://github.com/open-metadata/OpenMetadata/assets/66266464/8163b4a4-534e-48ed-85e9-90efc5e1800b

Type of change:

[x] Bug fix
[x] Improvement
[ ] New feature
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] Documentation

Checklist:

[x] I have read the CONTRIBUTING document.
[ ] My PR title is Fixes <issue-number>: <short explanation>
[ ] I have commented on my code, particularly in hard-to-understand areas.
[ ] For JSON Schema changes: I updated the migration scripts or explained why it is not needed.

@Chirag Madlani changes made as per comments!


Changes: Set swift compiler version to 4.2 and some fixes (renamings)

Hallo,
I want to use ChartLegends with a Swift 4.2 project, but ChartLegends had the compiler version 4.0. So I have change this in project settings and have also done a little fix in ChartLegendsView.swift (Renaming UICollectionViewScrollDirection to UICollectionView.ScrollDirection).
Best regards
S. Tanneberger

@ivanschuetz  Could we merge it and deploy latest version ASAP? Can't be used with latest xcode/swift :(


Pull harvest out into a separate pkg

Hi!
I’ve begun to use the harvest transform for a bunch of little things — for the purposes of this conversation, let’s say one of these things is a type of runtime debugger.
I started with a version of harvest which was a little more restrictive than the one in Oryx. But now I want to apply my debugger to more complicated programs, with control flow primitives.
now, this just about reaches the limit of what I would be comfortable maintaining separately (judging by the amount of touching internals)
Is there any talk of pulling harvest into JAX directly? (E.g. similar to experimental.checkify)?

We have discussed it and I personally would be open to it. I can re-open the conversation with the other Jax devs.

I can participate in a few days time! Thanks for considering.

I will say — some of the functionality I’ve developed could be useful as part of a general “jax.experimental.debug” package — paired with checkify.
The pattern is simple and likely easy for you to guess — but it involves recording state out and pairing it with relevant code information (location source lines, modules, etc)
I’ve found it to be tremendously useful for debugging some of my generative computations.

I don't think we should put harvest in JAX directly; while it's clearly useful I'm not yet satisfied it's a canonical API. (FWIW jax.experimental isn't meant to be a collection of useful JAX extensions, but rather a temporary staging ground for things that will ultimately either be upstreamed into the main jax namespace or else deleted.)
In practical terms, is the upside just not having to pip install another package / have another package requirement?

Fwiw, I've found some of the function transformations (inverse and ildj) here super useful, and the wheel is only 200kb, so I haven't minded that as a requirement for now.

In practical terms, is the upside just not having to pip install another package / have another package requirement?

Yes, that's basically it. I'm only really relying on harvest -- so seems slightly unnecessary.
There's slightly more advanced reasons -- e.g. later on, I'd like to support Oryx models as an optional plug-in for my system, and I don't want that to complicate my usage of harvest. That might be something I can figure out myself with some clever dependency wizardy.


chore: bdd-gh-1585233872 to 0.0.1

chore: Promote bdd-gh-1585233872 to version 0.0.1

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by:
To fully approve this pull request, please assign additional approvers.
We suggest the following additional approvers:
If they are not already assigned, you can assign the PR to them by writing /assign  in a comment when ready.
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment


multiple api process can be started on the same node on same port

Issue/Feature Description:
One can start multiple api server process and they all are in running state and binding to same port. This can create issue.
Why this issue to fixed / feature is needed(give scenarios or use cases):
Ideally there should be 1-1 port-process binding.
How to reproduce, in case of a bug:
**port binding**
root@aks:~/gopath/src/github.com/sodafoundation# lsof -i :8190
COMMAND    PID USER   FD   TYPE   DEVICE SIZE/OFF NODE NAME
python3 923160 root    5u  IPv6 53514951      0t0  TCP *:8190 (LISTEN)
python3 923160 root    7u  IPv6 53514951      0t0  TCP *:8190 (LISTEN)
python3 924646 root    5u  IPv6 53535517      0t0  TCP *:8190 (LISTEN)
python3 924646 root    7u  IPv6 53535517      0t0  TCP *:8190 (LISTEN)

**PS**
root@aks:~/gopath/src/github.com/sodafoundation/SIM/install# ps -eaf |grep api.py
root      923160       1  0 09:04 pts/0    00:00:01 python3 /root/gopath/src/github.com/sodafoundation/SIM/install/../dolphin/cmd/api.py --config-file /etc/dolphin/dolphin.conf
root      924646       1  0 09:34 pts/0    00:00:01 python3 /root/gopath/src/github.com/sodafoundation/SIM/install/../dolphin/cmd/api.py --config-file /etc/dolphin/dolphin.conf

Other Notes /  Environment Information: (Please give the env information, log link or any useful information for this issue)

it looks like ipv6, did you test ipv4?

Looks to be new feature added in Linux kernal https://lwn.net/Articles/542629/ .
Have tested with mutiple APIS process and both can take requests at same time. works like a load balancer.

closing as per above .


[Arith] Provide tighter ConstIntBounds for special cases

Expressions of the form (A+B)*C < (A*B)*D can occur occur when comparing the number of operations required for two different orderings in which matrix multiplications can be performed. Proving or disproving this conditional allows an optimal order of execution to be selected, even for dynamic argument shapes.
The default behavior of ConstIntBounds assumes that each term in an expression is independent.  For example, the maximum value of (A+B)*C - (A*B)*D is determined by taking the maximum value of (A+B)*C and subtracting the minimum value of (A*B)*D.  This algorithm can be applied in all cases, but can provide a bound that is looser than strictly required.
This commit adds a check for this case in ConstIntBounds, to provide a tighter bound of possible values.  When A, B, C, and D are all positive values, as is the case for tensor shapes, the inequality can be written as 1/A + 1/B < D/C.  If this inequality holds for the minimum values of A, B, and D, along with the maximum value of C, then it holds for all values.

One thing that we would like to consider is the overall efficiency. ConstIntBound is one such case where we are not trying to do complicated pattern matching and relies on the independent property, as a result it can be called extensively in the inner loop of analysis.
I think it is helpful to support some of the more complicated patterns, usually we do that use another form of SubAnalyzer, or proves with stronger strength(https://github.com/apache/tvm/blob/main/include/tvm/arith/analyzer.h#L75) while also make sure they are. not being used in inner loops

Good point on checking the performance.  I did a benchmark, with shown results shown in the plot below.  The x-axis is the time required to run the analyzer with the BoundUsingReciprocal function disabled, and the y-axis is the time with it enabled.  A y=x diagonal line is shown for comparison.

Click for benchmark settings

The benchmark measures the time to run the analyzer itself.  The is the call to analyzer.const_int_bound(expr) for the tests in test_arith_const_int_bound.py, and the call to analyzer.rewrite_simplify(expr) for the tests in test_arith_rewrite_simplify.py.
Tests were performed using pytest-benchmark plugin using default settings.  For each benchmark, the number of iterations was selected to take about 1 second total time, and the plot shows the average per-iteration time.  The number of iterations was at least 5k for all benchmarks, typically around 30-40k.
When disabling PR-16588, the BoundUsingReciprocal function was never entered.
Order of each benchmark was randomized to avoid any systematic bias between groups.

In the majority of cases, there is effectively no performance difference.  In cases where the improved bounds returned from ConstIntBound allow a simplification, the performance is improved as fewer recursive rewrites need to be applied.

My main worry is we open a flood gate of introducing many recursive rewrite patterns to ConstIntBound itself.

Ah, I think I see where I may have miscommunicated.  There isn't actually any recursive rewriting being performed.  The pattern matching are used to extract information, and not to perform any rewrites or allocations.  Stating that (A+B)*C < (A*B)*D could be rearranged into 1/A + 1/B < D/C was for the derivation of the tighter bounds, and isn't actually performed at runtime.
I absolutely agree that ConstIntBound should be a pure analysis pass, and should not perform any rewrites to the argument.

Do you think if it is possible to introduce another subalayzer for this kind of prove given they are specific to LoRA, or have a stronger proof strength

I think it could be moved to the TryProve* methods within the RewriteSimplifier, but hoisting it all the way out to a new subanalyzer probably wouldn't be feasible.  It would need to simplify first, then check for the specific comparison being made, and then simplify again, which seems like a lot of overhead.
My worry with introducing arguments for the proof strength is that it adds code complexity, and requires callers to know an additional argument.  For cases that have a significant computational overhead, that's worth it to avoid surprising a user.  For cases that don't, and would follow the same API, I'd want to avoid having that requirement where possible.
Rather than enabling it by proof strength, what if we start with it it behind an extension flag (link).  Where more computationally expensive simplifications would always require a user to opt-in with a higher proof strength, individually-enabled flags could allow new simplifications to be introduced as opt-in, and only later be enabled by default.

In all cases, we should take a close look at the proves and make sure they are correct, since it is at center of what we are doing

Absolutely agreed.  For this case, I compared the range of output values for several np.arange test cases.  Long-term, I wonder if it would be good to add fuzzing to the unit tests.  As good as the unit tests are at the moment, I always like adding more steps that could catch an incorrect simplification.

If in the end we end up keep things in ConstIntBound, I think we should have a comment saying that this was really an exceptional case, and future changes to ConstIntBound should only be made in exceptional cases

That makes sense, and I agree that it would be good to comment on it.

This PR is now updated to perform the checks for (A+B)*C < (A*B)*D patterns in RewriteSimplifer, gated behind the Extension::kComparisonOfProductAndSum extension flag.  This flag is currently disabled by default, with unit tests explicitly enabling the extension.
The updated behavior is sufficient to unblock https://github.com/apache/tvm/pull/16589.

Thank you @Lunderberg !


Implement Nuxt module

Why?
We want to offer a Nuxt module that simplifies how onyx can be used/installed in Nuxt projects.
Acceptance criteria

[x] the Nuxt module is implemented and published as @sit-onyx/nuxt npm package
[x] the Getting Started docs are updated with instructions for Nuxt users
[ ] the onyx Nuxt module is showcased in the official Nuxt module list
[x] auto imports are supported for onyx components
[ ] options for the createOnyx() Vue plugin can be set as module options inside the nuxt.config.ts file

Out of scope

offer onyx layout components as Nuxt layouts
integrate with the Nuxt i18n module so no additional config is needed for onyx i18n options

Definition of Done

[ ] covered by component tests
[ ] screenshot tests are updated
[ ] follow-up tickets were created if necessary
[ ] updated version + documentation is deployed
[ ] Set task to In Approval

Approval

Deployed Docs
[ ] approved by dev

Implementation details

Nuxt module dev guide

PR to add the onyx module to the Nuxt module list: https://github.com/nuxt/modules/pull/983


TargetError: No such file or directory

STDOUT: /system/bin/sh: cat: /sys/fs/cgroup/devlib_cpuset/big/cpuset.cpus: No such file or directory.

I have checked the target board, there is /sys/fs/cgroup/devlib_cpuset/big/cpus, no cpuset.cpus, it looks like the same issue we have met in Cambridge?

I believe this should have been resolved by https://github.com/ARM-software/devlib/commit/c97618944439c4d71ec4a492dbdf0c83d47c75a3


Added ability to append custom definitions to kustomization.yaml.

To solve the issue with SASWORK location and similar customizations, I made some changes to allow for a custom block to be added to the end of kustomization.yaml.
This fixes #223
Instructions:

Add a folder "customkustomize" to <cluster>/<namespace>/site-config/
Add the files you want to put in the kustomization.yaml file that's not under "resources" to this new folder
Add a file "customkustomize.yaml" to that folder which contains exactly what you want to append in the resulting kustomization.yaml file.

The end result can for example look like this:
$ ls site-config/customkustomize/
change-viya-volume-storage-class.yaml  customkustomize.yaml
$ cat site-config/customkustomize/customkustomize.yaml
patches:
  - path: site-config/customkustomize/change-viya-volume-storage-class.yaml
    target:
      kind: PodTemplate
      labelSelector: "sas.com/template-intent=sas-launcher"

Closed after deleting staging, reopening again

Closing this one as code base has changed and the current work-flow allows the customer to add items in the site-config directory with customer code.


attach fns to exports instead of module.exports

module.exports = exports
We usually use one of them:
module.exports = {
  getSourceQuestion: require('./getSourceQuestion'),
  getTitleQuestion = require('./getTitleQuestion'),
};

Or
exports.getSourceQuestion = require('./getSourceQuestion');
exports.getTitleQuestion = require('./getTitleQuestion');

I prefer the first one style.
Could you make some change to fit the style?


Update the default TLS termination policy to reencrypt

What type of PR is this?

/kind enhancement

What does this PR do / why we need it:

This PR updates the default TLS termination policy of OpenShift Route from passthrough to reencrypt. With this change, OpenShift users could use the default ingress certificates for the argocd server without any explicit modification.
The user/browser/CLI and the Router will communicate using the ingress certificate. The operator relies on OpenShift Service's CA to generate a self-signed certificate in the argocd-server-tls secret which will be used between the Router and the Argo CD server.

Have you updated the necessary documentation?

[x] Documentation update is required by this PR.
[x] Documentation has been updated.

Which issue(s) this PR fixes:
Fixes #?
How to test changes / Special notes to the reviewer:

Jira: https://issues.redhat.com/browse/GITOPS-4674


improvement move load once to index reader to reduce

memory footprint of column reader
Proposed changes
Issue Number: close #xxx

Further comments
If this is a relatively large or complex change, kick off the discussion at<EMAIL_ADDRESS>by explaining why you chose the solution you did and what alternatives you considered, etc...

run buildall

(From new machine)TeamCity pipeline, clickbench performance test result:
the sum of best hot time: 46.7 seconds
stream load tsv:          535 seconds loaded<PHONE_NUMBER>9 Bytes, about 133 MB/s
stream load json:         21 seconds loaded<PHONE_NUMBER> Bytes, about 107 MB/s
stream load orc:          64 seconds loaded<PHONE_NUMBER> Bytes, about 16 MB/s
stream load parquet:          31 seconds loaded 861443392 Bytes, about 26 MB/s
insert into select:          29.0 seconds inserted 10000000 Rows, about 344K ops/s
storage size:<PHONE_NUMBER>8 Bytes

run buildall

run buildall

run buildall

(From new machine)TeamCity pipeline, clickbench performance test result:
the sum of best hot time: 49.1 seconds
stream load tsv:          552 seconds loaded<PHONE_NUMBER>9 Bytes, about 129 MB/s
stream load json:         20 seconds loaded<PHONE_NUMBER> Bytes, about 112 MB/s
stream load orc:          64 seconds loaded<PHONE_NUMBER> Bytes, about 16 MB/s
stream load parquet:          31 seconds loaded 861443392 Bytes, about 26 MB/s
insert into select:          29.2 seconds inserted 10000000 Rows, about 342K ops/s
storage size:<PHONE_NUMBER>7 Bytes

(From new machine)TeamCity pipeline, clickbench performance test result:
the sum of best hot time: 45.77 seconds
stream load tsv:          548 seconds loaded<PHONE_NUMBER>9 Bytes, about 130 MB/s
stream load json:         20 seconds loaded<PHONE_NUMBER> Bytes, about 112 MB/s
stream load orc:          64 seconds loaded<PHONE_NUMBER> Bytes, about 16 MB/s
stream load parquet:          30 seconds loaded 861443392 Bytes, about 27 MB/s
insert into select:          28.8 seconds inserted 10000000 Rows, about 347K ops/s
storage size:<PHONE_NUMBER>0 Bytes

(From new machine)TeamCity pipeline, clickbench performance test result:
the sum of best hot time: 45.88 seconds
stream load tsv:          538 seconds loaded<PHONE_NUMBER>9 Bytes, about 132 MB/s
stream load json:         20 seconds loaded<PHONE_NUMBER> Bytes, about 112 MB/s
stream load orc:          64 seconds loaded<PHONE_NUMBER> Bytes, about 16 MB/s
stream load parquet:          31 seconds loaded 861443392 Bytes, about 26 MB/s
insert into select:          28.7 seconds inserted 10000000 Rows, about 348K ops/s
storage size:<PHONE_NUMBER>1 Bytes


Option to toggle on/off the prompt for discard

Every time I save a recording and I want to close the recording window, I have to go through the prompt asking me if I want to discard the recording.
I would be more convenient if there was a toggle to turn this prompt on or off.

Agree, don't get why do I need to prompt after saving to a file...


fix(version): update fabric8-stack-analysis-ui to 0.1.24

There were the following issues with your Pull Request

Commit: c126cb8d38e8daf393e853f22089adaadedcbb31
Your subject line is longer than 50 characters

Guidelines are available at https://github.com/fabric8io/fabric8-ui

This message was auto-generated by https://gitcop.com


Added documentation for visualiser and named access

Improved the docs for the visualiser to explain how to switch modes
Made the controller the default mode, followed by trajectory. Passive dynamics are now last
Added note in named access docstrings that MuJoCo IDs start at 0.

Codecov Report
Attention: 66 lines in your changes are missing coverage. Please review.

Comparison is base (32a25ed) 25.59% compared to head (9db2647) 25.47%.

Files
Patch %
Lines

src/LibMuJoCo/functions.jl
0.00%
48 Missing :warning:

src/LibMuJoCo/structs.jl
0.00%
16 Missing :warning:

ext/VisualiserExt/visualiser.jl
0.00%
2 Missing :warning:

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@            Coverage Diff             @@
##             main      #85      +/-   ##
==========================================
- Coverage   25.59%   25.47%   -0.13%
==========================================
  Files          22       21       -1
  Lines        5168     5161       -7
==========================================
- Hits         1323     1315       -8
- Misses       3845     3846       +1

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


[3322] Clone for Structs in Generics

Fixes #3322

AppVeyor build fails with an error, which seems to be not related to the pull. Had the same issue locally, it fixed for me after updating Microsoft.Build, Microsoft.Build.Framework and Microsoft.Build.Utilities.Core to the latest version (16.4.0).


Fix timestamp display in UI

Kate - Hi Alex! Jenny looked at the UI and I'm going back over some of her comments/ideas. One thing that we are both seeing is that some of the timestamps are still wonky - when I created a negotiation card in the UI, I get the right date/time. However, a specification I created in code is giving a timestamp from 1970. Here are some screenshots!

Fixed by #363


font-silkscreen not exported from package

Current Behavior
If I try to import the Silkscreen font, via import<EMAIL_ADDRESS>I get the following error:
Module not found: Package path ./css/400.css is not exported from package
> 6 | import<EMAIL_ADDRESS>
Expected Behavior
Expected behaviour is for it to load the font.
Tamagui Version
1.116.14

Platform (Web, iOS, Android)
Web

Reproduction
https://codesandbox.io/p/devbox/elastic-pine-kjnhj7

System Info
No response

Added an export for next version, thanks


Either freeze or no results warning when trying to search videos

So basically, everything is in the title. I've tried removing the configuration files but it didn't work, and it seemed to work fine before. I'm using pipe-viewer-git from the chaotic-aur, and its version is 0.0.9 so the current latest.

I cannot confirm this issue...
Execute pipe-viewer --debug test to see what it's going on.
If there is an issue connecting to m.youtube.com, pass the --invidious argument (or set prefer_invidious to 1 in the config file), which will use an invidious instance to get the YouTube results.


says player id is invalid

everytime i try and use the script i type /giveuniform 2

Try updating with the recent version I just pushed


Falco using tokens past their lifetime in Kubernetes 1.21

Describe the bug

In Kubernetes 1.21, the BoundServiceAccountTokenVolume feature is enabled by default.  This feature replaces the mounted token secret from the past with a projected volume that includes an auto rotated token.  With a default configuration, the Kubernetes API server monitors when tokens are used past their lifetime and annotates requests in the audit log with a stale-token annotation.  Based on my testing, Falco does not load in rotated tokens and uses the first token mounted in the pod forever.
How to reproduce it

Deploy falco to Kubernetes with the Helm Chart
Wait 1 hour for the token to expire
Observe the audit log annotating requests from Falco with the stale-token annotation

Expected behaviour

Falco should load the rotated token instead of caching the initial token.
Environment

Falco version: 0.30.0

System info:

{
  "machine": "x86_64",
  "nodename": "falco-4wlp5",
  "release": "3.10.0-1160.42.2.el7.x86_64",
  "sysname": "Linux",
  "version": "#1 SMP Tue Sep 7 14:49:57 UTC 2021"
}

Cloud provider or hardware configuration:
OS: CentOS 7

Kernel: 3.10.0-1160.42.2.el7.x86_64

Installation method: Kubernetes Helm Chart

Additional context

Example Audit log line:
{
  "kind": "Event",
  "apiVersion": "audit.k8s.io/v1",
  "level": "Metadata",
  "auditID": "a6a52418-b225-43a2-b48b-56e489b9fbab",
  "stage": "RequestReceived",
  "requestURI": "/api/v1/watch/nodes?pretty=false",
  "verb": "watch",
  "user": {
    "username": "system:serviceaccount:services:falco",
    "uid": "146556d9-3753-4f47-922b-cd68f8597d1d",
    "groups": [
      "system:serviceaccounts",
      "system:serviceaccounts:services",
      "system:authenticated"
    ],
    "extra": {
      "authentication.kubernetes.io/pod-name": ["falco-gg6rl"],
      "authentication.kubernetes.io/pod-uid": [
        "760c5c22-73c4-4da0-a21a-ca6e6b57513c"
      ]
    }
  },
  "sourceIPs": ["<IP_ADDRESS>"],
  "userAgent": "falcosecurity-libs",
  "objectRef": { "resource": "nodes", "apiVersion": "v1" },
  "requestReceivedTimestamp": "2021-11-10T23:26:08.128787Z",
  "stageTimestamp": "2021-11-10T23:26:08.128787Z",
  "annotations": {
    "authentication.k8s.io/stale-token": "subject: system:serviceaccount:services:falco, seconds after warning threshold: 1388501"
  }
}

Issues go stale after 90d of inactivity.
Mark the issue as fresh with /remove-lifecycle stale.
Stale issues rot after an additional 30d of inactivity and eventually close.
If this issue is safe to close now please do so with /close.
Provide feedback via https://github.com/falcosecurity/community.
/lifecycle stale

/remove-lifecycle stale

Issues go stale after 90d of inactivity.
Mark the issue as fresh with /remove-lifecycle stale.
Stale issues rot after an additional 30d of inactivity and eventually close.
If this issue is safe to close now please do so with /close.
Provide feedback via https://github.com/falcosecurity/community.
/lifecycle stale

/remove-lifecycle stale

Any update on this? We are seeing this as well.

Any update on this?

Issues go stale after 90d of inactivity.
Mark the issue as fresh with /remove-lifecycle stale.
Stale issues rot after an additional 30d of inactivity and eventually close.
If this issue is safe to close now please do so with /close.
Provide feedback via https://github.com/falcosecurity/community.
/lifecycle stale

/remove-lifecycle stale

Hello @jasondellaluce Any update on this? We are using eks and we have to upgrade eks 1.20 to 1.21 by end of this month, and in eks 1.21 we need this otherwise we have to restart the falco pods everytime before the token is marked as stale.
Thanks

Issues go stale after 90d of inactivity.
Mark the issue as fresh with /remove-lifecycle stale.
Stale issues rot after an additional 30d of inactivity and eventually close.
If this issue is safe to close now please do so with /close.
Provide feedback via https://github.com/falcosecurity/community.
/lifecycle stale

/remove-lifecycle stale

Hi, Do we have any updates here?

Issues go stale after 90d of inactivity.
Mark the issue as fresh with /remove-lifecycle stale.
Stale issues rot after an additional 30d of inactivity and eventually close.
If this issue is safe to close now please do so with /close.
Provide feedback via https://github.com/falcosecurity/community.
/lifecycle stale

/remove-lifecycle stale

Related issue: https://github.com/falcosecurity/falco/issues/2485

The new k8smeta plugin should solve this issue, give it a try with our dedicated helm chart and Falco 0.36.2 https://github.com/falcosecurity/falco/issues/2973#issuecomment-1876880587

This should be solved by Falco 0.37.0! Feel free to reopen if this is still an issue


How to implement async getContent?

I'm trying to include async function calls inside the getContent method for a tooltip as follows:
 <>
      <img
        data-tip=""
        data-for={name}
        data-event="click"
        className="history-button"
        src={isLight ? historyLight : historyDark}
      />
      <ReactTooltip
        className="tool-tip"
        id={name}
        type="dark"
        effect="solid"
        place="top"
        globalEventOff="click"
        getContent={async () => {
          const docRef = doc(db, `completion-history`);
          const snap = await getDoc(docRef);
          return helper;
        }}
        clickable
      />
    </>

but get the following error: Uncaught Error: Objects are not valid as a React child (found: [object Promise]). If you meant to render a collection of children, use an array instead.
What's the proper way to implement a tooltip with async content?

@sabetAI did you solve this? I'm having the same requirement.


OSError: Cannot load native module 'Cryptodome.Cipher._raw_ecb'

I just installed this application locally with Python 2.7.12 and tried to run this.
After running I immediately got this error:
C:\PokeManager>python ".\pogo\demo.py" -a google -u<EMAIL_ADDRESS>-p "mypassword" -l "xx.lat , xx.long"
Traceback (most recent call last):
  File ".\pogo\demo.py", line 14, in <module>
    from api import PokeAuthSession
  File "C:\PokeManager\pogo\api.py", line 10, in <module>
    from gpsoauth import perform_master_login, perform_oauth
  File "C:\Python27\lib\site-packages\gpsoauth\__init__.py", line 4, in <module>
    from . import google
  File "C:\Python27\lib\site-packages\gpsoauth\google.py", line 4, in <module>
    from Cryptodome.PublicKey import RSA
  File "C:\Python27\lib\site-packages\Cryptodome\PublicKey\RSA.py", line 79, in <module>
    from Cryptodome.IO import PKCS8, PEM
  File "C:\Python27\lib\site-packages\Cryptodome\IO\PKCS8.py", line 73, in <module>
    from Cryptodome.IO._PBES import PBES1, PBES2, PbesError
  File "C:\Python27\lib\site-packages\Cryptodome\IO\_PBES.py", line 44, in <module>
    from Cryptodome.Cipher import DES, ARC2, DES3, AES
  File "C:\Python27\lib\site-packages\Cryptodome\Cipher\__init__.py", line 78, in <module>
    from Cryptodome.Cipher._mode_ecb import _create_ecb_cipher
  File "C:\Python27\lib\site-packages\Cryptodome\Cipher\_mode_ecb.py", line 46, in <module>
    """
  File "C:\Python27\lib\site-packages\Cryptodome\Util\_raw_api.py", line 168, in load_pycryptodome_raw_lib
    raise OSError("Cannot load native module '%s'" % name)
OSError: Cannot load native module 'Cryptodome.Cipher._raw_ecb'

Some information about my software:

Python 2.7.12
pip 8.1.2
git version 2.9.2.windows.1
java version 1.8.0_102-b14 64-bit
Windows 10 pro 64-bit

Have you followed the Readme instructions?
Installed the requirements.txt?
Are you using 2 Factor Authorisation?
Have you tried updating python and pip?
Tried updating the requirements.txt (using --upgrade for pip)?

@cglatot
I followed the Readme instructions.txt
I installed the requirements
I am using 2 Faction Authorisation
I fresh installed Python and pip this morning
Tried to update the requirements.txt as well
I don't know if something has to do with it, but according to other github projects and Reddit it is possible that they are busy patching the API's at this moment.

Did you follow what I said on the Readme about 2FA?

@cglatot About the App Password?
I haven't seen that, so I did not follow that.
But, when I follow that link, I get this error (translated) on the Google site:

The setting you are searching, is not available for your account

https://support.google.com/accounts/answer/185833


How to set request header in ajax

Hi i want to set a request header in ajax, normally i am setting header in beforesend function in ajax..
please give me solution for this..
Thanks for the plugins.. i was searching like this plugin for long time..

Hii
i got solution, sorry i did not seen in closed issues..
Thank you


Release version 1.8.12

Hi @holzeis!
This PR was created in response to a manual trigger of the release workflow here: https://github.com/get10101/10101/actions/runs/8045186446.
I've bumped the versions in the manifest files in this commit: 449119e243540f7eec9402ed4b4d0f385f2b2095.
Merging this PR will create a GitHub release!

we have a breaking change in the next version. changing to 1.9.0


The unity build folder has different files for unity 2019 compared to 2020

Can i use this library with 2019 unity? Because the build output for 2019 has different files from what is to be passsed to unity Context.

Ok sorry apparently theres a different documentation for 2019


Getting 403 urn:acme:error:unauthorized: Account creation on ACMEv1

When I add the annotations to my route
metadata:
  annotations:
    kubernetes.io/tls-acme: "true"

I notice the following error on openshift-acme:
403 urn:acme:error:unauthorized: Account creation on ACMEv1
I1029 15:27:01.256284       1 route.go:385] Started syncing Route "mobile-office/mobile-office-lb" (2019-10-29 15:27:01.256276003 +0000 UTC m=+497907.871877433)
I1029 15:27:03.373213       1 route.go:387] Finished syncing Route "mobile-office/mobile-office-lb" (2.116908979s)
I1029 15:27:03.373246       1 route.go:718] Error syncing Route mobile-office/mobile-office-lb: failed to get ACME client: 403 urn:acme:error:unauthorized: Account creation on ACMEv1 is disabled. Please upgrade your ACME client to a version that supports ACMEv2 / RFC 8555. See https://community.letsencrypt.org/t/end-of-life-plan-for-acmev1/88430 for details.
I1029 15:27:03.383454       1 route.go:385] Started syncing Route "mobile-office/mobile-office-lb" (2019-10-29 15:27:03.383445272 +0000 UTC m=+497909.999046703)
I1029 15:27:05.875666       1 route.go:387] Finished syncing Route "mobile-office/mobile-office-lb" (2.492207975s)
E1029 15:27:05.875708       1 route.go:728] failed to get ACME client: 403 urn:acme:error:unauthorized: Account creation on ACMEv1 is disabled. Please upgrade your ACME client to a version that supports ACMEv2 / RFC 8555. See https://community.letsencrypt.org/t/end-of-life-plan-for-acmev1/88430 for details.
I1029 15:27:05.875723       1 route.go:729] Dropping Route "mobile-office/mobile-office-lb" out of the queue: failed to get ACME client: 403 urn:acme:error:unauthorized: Account creation on ACMEv1 is disabled. Please upgrade your ACME client to a version that supports ACMEv2 / RFC 8555. See https://community.letsencrypt.org/t/end-of-life-plan-for-acmev1/88430 for details.
I1029 15:28:13.188061       1 reflector.go:357] github.com/tnozicka/openshift-acme/pkg/cmd/openshift-acme-controller/cmd.go:257: Watch close - *v1.Route total 1 items received

I've used these commands to run acme cluster-wide:
oc create -fdeploy/letsencrypt-staging/cluster-wide/{clusterrole,serviceaccount,imagestream,deployment}.yaml
oc adm policy add-cluster-role-to-user openshift-acme -z openshift-acme

@tnozicka

I tried deleting the instance I had installed that was cluster-wide and retried using the namespace wide and It worked just fine. Does this make any sense? I would prefer to have it running cluster-wide so I don't need to deploy acme for each namespace.

letsencrypt-staging should be letsencrypt-live - staging registrations are already disabled for a while. You probably used the correct "live" for single-namespace.

letsencrypt-staging should be letsencrypt-live - staging registrations are already disabled for a while. You probably used the correct "live" for single-namespace.

@tnozicka Exactly right. Thanks for your support Tomáš. Much appreciated.

yw
/close


Migrates CI to GHA

this is in preparation for the update to 4.2, so merging for now so we can test in the PR


Offline support for ApplicationInsightsTarget

We have successfully integrated our Xamarin Forms application with Azure Application Insights using Microsoft.ApplicationInsights.NLogTarget. It's working perfect in online scenarios. Our app has an offline capability, so how can we enable offline scenarios also with Microsoft.ApplicationInsights.NLogTarget? Is there any samples for the same?

Please post questions about external NLog targets to the intended github-project:
https://github.com/microsoft/ApplicationInsights-dotnet
I have no clue if there is an offline mode in ApplicationInsights. Think there is one for AppCenter. But none of them are controlled by NLog. NLog is just a "thin" wrapper that calls the official API for the external storage.

Seems there is a "disconnected mode". Guess you can check the storage folder whether it is working:
https://devblogs.microsoft.com/premier-developer/application-insights-in-disconnection-mode/


Can't find the register keys

For me

HKEY_LOCAL_MACHINE\SOFTWARE\Microsoft\Windows NT\CurrentVersion\Image File Execution Options\TcXaeShell.exe
HKEY_LOCAL_MACHINE\SOFTWARE\Microsoft\Windows NT\CurrentVersion\Image File Execution Options\devenv.exe
do not exist. I also searched for other devenv.exe entries or for Image File Execution Options at other locations. None of them contain a dpiAweness entry.

Which Windows OS are you on, and which version of TwinCAT are you using? Maybe I can try to reproduce the issue.

Manually adding the devenv.exe/dpiAwareness entry worked! Visual Studio looks like crap now on my screen x)
OS Name	Microsoft Windows 10 Education
Version	10.0.19043 Build 19043

Closing as it looks like this worked for you.


Access to the metric name from the relabelling rules

I'm facing a situation where I would like to add a label to a specific metric at scrape time, but don't seem to be able to as the metric name isn't part of the __ variables exposed to the relabelling rules.
Would love a __metric__ or __metric_name__.

Isn't this __name__?

Huuum, it could be! I'm looking at https://prometheus.io/docs/prometheus/latest/configuration/configuration/#<relabel_config> and there's no __name__ documented there.

It makes more sense to ask questions like this on the prometheus-users mailing list rather than in a GitHub issue. On the mailing list, more people are available to potentially respond to your question, and the whole community can benefit from the answers provided.
Look at metric_relabel_configs.


Drag with scrol view

Hello,
I have a problem with your wonderfull lib.
I have 2 columns .
The first is a scrol div with many dragable item
and the second is the droppable container
when i drag an element on top of with scrol view no problem
But When i drag an element of the bottom on my scrol view ( i scrol so )
when i get inside my droppable container the position of dragable keep the Y of the scrool view
How can i fix this ?
Thx for your reply
Have a nice day

Please provide a CodeSandbox replicating the issue you are facing if you want me to be able to assist you

I assume you're not on the latest version of the library.

All right your the boss dud.
I just update and its work :)
great job
enjoy your weekend


Disabling eval in async functions in non-strict mode

If a var declaration occurs in eval in a non-strict async function the
variable should ideally bind to the inner function. But right now if the
var declaration is shadowing one of the formals then the binding goes
wrong. This changelist makes it a SyntaxError to use eval inside async
functions in non-strict mode. Fix for the binding issue will be addressed later.

@pleath @ianwjhalliday @abchatra @tcare Can you please take a look?

Looks good to me.  We need to make sure this is what we want to do before merging though.

LGTM.

Looks good.

Followed up on the thread https://github.com/Microsoft/ChakraCore/pull/859


Feature: implement backend for the web

Resolves: #1

Hi @DouglasDwyer.
I have gotten very far along on this now, and have been able to get a component model working with strings, results, vecs, etc to go back and forth across the boundary.
However, them implementation ended up rather large, and I had to use unsafe in one place as there was a reentrant state both the static and dynamic borrowing models could not solve, as references were not expressive enough, and RefCells could not model a recursive but exclusive borrowing.
Do you think it would be better to split this into a separate crate instead, and if so, would you prefer if wasm_runtime_layer depended on it like you do with wasmtime, and then forwarding the impls, or should the web runtime crate depend on wasm_runtime_layer and implement relevant traits in the crate itself?
It also goes to ask if you'd like it to be a subcrate in this repo or a separate repo altogether.
Which method do you think would be best for you?
Thanks for all the help :D

Amazing.
I am finishing up and implementing some missing things and todo!()s, as well as writing some tests to ensure it all works.
It didn't seem like you had any tests or CI currently. I tried to add tests but struggled with the nested crates and workspaces, so I ended up making top level tests in a wrapping workspace for both wasm_runtime_layer and wasm_component_layer here: https://github.com/ten3roberts/wasm-scripting-example.

A proposal:
Would we consider adding a name: &str as a first parameter for WasmFunc to make it easier when debugging and in logs to know which export or import was called?
It will work like a debug label and can be used for tracing span generation and for supplementing anyhow errors to know which function failed to be called.
Example:

@philpax, do you mind reviewing this small behemoth 😅

Regarding the tests that you wrote, it looks like you've found a way to add them to the runtime layer. If there are still any in that other crate, I would like to help with getting them into wasm_runtime_layer itself.
On the topic of a debug name: &str, I am a bit reluctant to change the signature of that function (since it would be a breaking change and the signature would no longer be identical to wasmtime). What if we added the debug name string to the FuncType structure instead? We could add methods FuncType::with_name and FuncType::name which would serve the same purpose, but not be breaking.

Thank you for the incredible work on this PR, @ten3roberts! Your contribution is greatly appreciated :)


Add the larger word directories

https://github.com/dustinkirkland/petname/blob/master/usr/share/petname
The OG golang-petname had some weird bash script that used the debian package to source the words. However, that's not very cross plattform.
KISS: Download the files manuallym use regex to generate the go file, stash the regex somewhere for future reuse.

Done


add support for *.csv files import/export

add support for *.csv files import/export

https://www.cnblogs.com/LY-HeroesRebor/p/9359743.html
https://blog.csdn.net/woaixiaoyu520/article/details/78455650
https://baike.baidu.com/item/CSV

fixed by #9
please use V1.2.0 and above


[HUDI-3998] Fix getCommitsSinceLastCleaning failed when async cleaning

Tips

Thank you very much for contributing to Apache Hudi.
Please review https://hudi.apache.org/contribute/how-to-contribute before opening a pull request.

What is the purpose of the pull request
Fix getCommitsSinceLastCleaning failed when async cleaning
When async cleaning
000.commit
001.commit.requested
002.clean
001.commit.inflight
001.commit
003.commit.requested
004.clean

002 cleaning did not include 001 because it has not been completed,So when 004 cleaning , 001 should be included when we calculate the commit after 002.But because 001 < 002,Therefore, 001 is not included. getCommitsSinceLastCleaning returns 0.
So we need to save last completed commit timestamp (000.commit), The last completed commit timestamp is used to calculate how many commit have been completed since the last clean
Brief change log

Add lastCompletedCommitTimestamp in HoodieCleanMetadata  for getCommitsSinceLastCleaning
Add support for async clean in some clean unit test cases

Verify this pull request
(Please pick either of the following options)
This pull request is a trivial rework / code cleanup without any test coverage.
(or)
This pull request is already covered by existing tests, such as (please describe tests).
(or)
This change added tests and can be verified as follows:
(example:)

Added integration tests for end-to-end.
Added HoodieClientWriteTest to verify the change.
Manually verified the change by running a job locally.

Committer checklist

[ ] Has a corresponding JIRA in PR title & commit

[ ] Commit message is descriptive of the change

[ ] CI is green

[ ] Necessary doc changes done or have another open PR

[ ] For large changes, please consider breaking it into sub-tasks under an umbrella JIRA.

good find on the bug.
if my understanding is right, lastCompletedCommitTimestamp actually refers to last CompletedCommit w/o any holes. i.e. before the last completed commit, there should not any inflight commit. either its completed or rolledback.
Also, can you confirm the changes are backwards compatible ?

@nsivabalan  As the comments in the code,lastCompletedCommitTimestamp  is the last completed commit timestamp before clean. There is no guarantee that it is the last one on the whole timeline,because we can't ensure that there is no completed commit after clean,It is bound to clean.
before the last completed commit, there should not any inflight commit. either its completed or rolledback.As for this, I think it is related to the rollback mechanism. As long as we ensure that we will rollback before each commit, this is true
For version compatibility, I verified it locally, it is backward compatible .Because we added a default value for the value of lastCompletedCommitTimestamp  in HoodieCleanerPlan and  HoodieCleanMetadata.In this way, 'lastCompletedCommitTimestamp' uses the default value("") when serializing files of older versions.When 'lastCompletedCommitTimestamp'is an empty string,
numCommits = commitTimeline.findInstantsAfter(lastCompletedCommitTimestamp).countInstants(); returns the number of all completed commits,The subsequent clean operation can run normally

@dongkelun : sorry, I dropped the ball on this. can you rebase w/ latest master. I can review it and we can get it in for 0.12.1.

A high level thought. I appreciate you parametrized most tests for both sync and async cleaning. But do keep in mind that, these are going to add to our test run-time. So, may be take another look to see if we need to make all cleaner tests run for both sync and async. Enable for those where it matters and for others probably running it for sync cleaning should suffice.

Unnecessary asynchronous cleaning test cases have been removed

also, a good practice to follow. whenever you are addressing feedback, try to add it as new commits. Easier for reviewer to re-review just the new changes. if not, I have to review entire patch again. If not, I will click on newer commits and will review only the newly changed code.

also, a good practice to follow. whenever you are addressing feedback, try to add it as new commits. Easier for reviewer to re-review just the new changes. if not, I have to review entire patch again. If not, I will click on newer commits and will review only the newly changed code.
OK, sorry, I thought it was OK to click compare after 'force pushed'. I mistakenly thought that 'force pushed' would look cleaner. I don't know you reviewed it by comparing two commits. I'll pay attention in the future


Fix documentation typo.

I think the example called to the wrong function.
Another typo, I think, is here:
https://github.com/livingdocsIO/editable.js/blob/35e4e2a767db6351810a513bdcec4c69ee583057/src/parser.js#L189-L200
The function should be called lastChild() and not “latest”.  So, if interested I can submit a PR and use a trampoline function to maintain compatibility — if you folks care 😉

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Update Node to 4.4.1 and 5.9.1

Always up to date :-) Thanks.

Good way for me to support your good work!


How to set event level groups?

The migration guide says you can emulate logEventWithGroups by calling setGroup before logEvent, but setGroup will add a user to a group. How could you add an event to a group with this function?

Hi @Mlu456, setGroup will set user to certain group, so later events for this user will be connected to that group. logEventWithGroups was set the group on the event level, so by mean calling setGroup before logEvent it is to emulate the logEventWithGroups. Not quite the same, but it is a way to handle the transition.cc @yuhao900914


[type: feat] Support PostgreSQL && update Readme in English

Description

Support PostgreSQL
Update Readme in English
Fix websocket error( the namespace in websocket url is hardcoded to shenyu in the past)

Note
Now, there is something wrong with the official image 2.5.0 version about pg. You could use my dockerhub image to test.(executed by make build-all-image, based on 115e33accc, yunlongn<EMAIL_ADDRESS>Thu Sep 29 17:09:49 2022 +0800)
helm install shenyu0930 bird-shenyu/shenyu -n=shenyu --create-namespace \
      --set version=latest \
      --set admin.image=aflybird0/shenyu-admin \
      --set bootstrap.image=aflybird0/shenyu-bootstrap \
      --set dataSource.active=pg \
      --set dataSource.pg.ip=<IP_ADDRESS> \
      --set dataSource.pg.port=5432 \
      --set dataSource.pg.username=postgres \
      --set dataSource.pg.password=123456

Tested in Ubuntu 20.04

I think this pr can be merged now, what do you think?
@JooKS-me


Gemfile updates

add  source 'https://rubygems.org' to Gemfile
add bech32 to Gemfile
Without it, I experienced this:

$ irb
irb(main):001:0> require "./nostr-ruby.rb"
<internal:/Users/me/.asdf/installs/ruby/3.1.2/lib/ruby/3.1.0/rubygems/core_ext/kernel_require.rb>:85:in `require': cannot load such file -- bech32 (LoadError)

(Is bech32 the correct gem to use?)

@jholton you are right, bech32 (it is the right gem) is missing from the gemfile.
I included it statically in the my first playground from which I'm derivating the lib, but I installed as gem too and forgot to do a wipe and reinstall test before the relase.
Thank you!