Images are pixelated on 4.1.0 when exporting

Version of Archi, Operating System

4.1.0, Windows 10
Expected Behaviour

Exported images should be crisp and clear at 100%
Actual Behaviour

Images are pixelated, Element borders not rendered
Steps to Reproduce the Behaviour

Open a view
Export view as image
Look at the image in an internal editor

Can you provide an example please?

Here you go...

Both files were exported at 100% to PNG. Archi 4.1.0 is noticably worse.

Do you have the View at 100% in the zoom control in the app? This should be set to 100%.

Yep, both exported at 100%. Scaling up to 400% doesn't restore the box borders. I'm hoping this is user error, I have a 90 page document due tomorrow.
SVG is rendering correctly in Chrome but not in Visio or Gimp.

What machine are you using? Does it have a hi-res screen? If so, can you compare when using another non-hi-res screen machine? Can you compare with Archi 4.0.3?
I'm trying to see if there is a regression in the image export code. 4.1 uses a different version of Eclipse which has deprecated image methods.

I am using a MS Surface with Windows 10 and yes it does have a Hi Res Screen. I will see if I can get Archi installed on another machine at work tomorrow (10pm here). The second image was exported from Archi 4.0.3 with the same settings (100% png). They are different diagrams but the same scale.

The size on the second image (and other images from 4.0.3) is about double what I would expect. The boxes are about 288 pixels wide and 144 pixels high whereas I would expect 144 wide and 72 high. The 4.1.0 image has the correct dimensions but the rendering is poor. Given this screen is scaled at 200% the factor of 2 makes sense.

I've made a new beta build of Archi that rolls back the code for image handling:
https://www.archimatetool.com/dev/beta
Can you test this version?

Latest beta build should fix this:
https://www.archimatetool.com/dev/beta

I will test when I am back in the office in a couple of hours.

This version has image export working as expected. Thanks for the fast response!

Thanks for the feedback. I'll get a maintenance build out soon (I want to look at some image 2x things). In the meantime, you can use the beta version. The only changes are image export fixes and one other Jasper Reports fix.

Fixed in 4.1.1


Add CC BY 4.0 Int license

Similarly to https://github.com/CaptainSifff/paper_teaching-learning-RSE/pull/234.
@CaptainSifff you have a better overview of what else we may need to do. Together with @jlinx (initial commit), I am currently the only contributor in the Git history, so I don't think we need any further approvals.
Note that the website is based on https://github.com/mmistakes/minimal-mistakes, which is distributed under the MIT license. We still mention it in the footer.
This license is, of course, only for the content, not for the (minimal) technicalities.
Closes #7.

OK, I put some thought into it, and I'm wondering whether CC-BY is still a good idea... We are curating the resources of others. Awesome lists recommend CC-0 for that: https://github.com/sindresorhus/awesome/blob/main/awesome.md


[Feature] Smart Substitution for JSON fields in Mongo Form

Fixes #5122

Hi @trishaanand, it seems like request paramas have not been captured for mongo form. Can you please update the request params - either as part of this PR or a new one ?

/ok-to-test sha=7591128e5aab68ae7a15139bb1802f9eee46efe7


Add accept multiple paths in as path argument.

I was using this tool, ca a command line user having to pass in a list of all our csproj would come in handy.

Hello,
I'm using this tool as the the nuget cli tool doesn't work with PackageReference. It is very usefull.
The Usage section on the frontpage is not up-to-date. If you browse the source code you can find that:
[Option(CommandOptionType.NoValue,
    Description = "Recursively search for all projects within the provided directory.",
    ShortName = "r", LongName = "recursive")]
public bool Recursive { get; set; } = false;

With this option you can recursively update all the projects in a directory and all his subdirectories.
If you want to pass in all your projects in a directory and not only a few ones then I think this option can help you.


Updating to ios-sim version 5

ios-sim 5 uses a different approach to specify a simulation target, but we still support all the same previous inputs. As part of this change we also now allow the iOS version to be specified, but if the target/version combination doesn't exist on the server then it will fail.

don't we need a new taco-remote-lib (new entry in mux) for this?

This fix is backwards compatible (and actually addresses bugs in previous versions) so we don't need a separate entry in taco-remote-multiplexer, just need to update it when we get to publishing. I'm waiting to update the package versions until we take the RI from v1.0.2 so we don't have too many conflicts.

:+1:


Create a GUI-Element to control stongswans reuse_ikesa option

Related and tested OPNsense version: 20.1 (production).
As descriped in  https://forum.opnsense.org/index.php?topic=16218.new;topicseen#new
there is an issue trying to connect to an ASA 5540 using IPSec with more than 1 Phase 2 definitions.
This issue can be solved by setting strongswans option "reuse_ikesa=no".
In the test case that was done by creating a file /usr/local/etc/strongswan.opnsense.d/noreuse_ikesa.conf with a line
charon { reuse_ikesa=no }
and restarting ipsec.
Because I was not the only one who had this problem (which was my luck: I got this solution offered!), I think it would be a good idea to create a GUI Element to be able to enable or disable the "reuse_ikesa" option.

Still an issue that is very hard to troubleshoot, Palo Alto firewalls need this setting for IPSec tunnels to be stable and rekey without downtime.


docs(api): update required type accept

Thank you very much @neighborhood999

Thank you very much @neighborhood999


remove asim logging config

We realised that we only started seeing 499/504 errors after this commit https://github.com/uktrade/ip-filter/commit/9a855ba09fa3fda8ca6d881035bb43a4e706c4af . Commenting out the lines adding the ASIM formatter resolved these issues

I'm assuming the app after call to log response wasn't the cause and it was the formatter 🙏

Yes I should've added that we tested that and it wasn't the cause of the issue


[SMALLFIX]Use static imports for standard test utilities in GroupMappingServiceTest.java

Use static imports for standard test utilities in GroupMappingServiceTest.java

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/17195/
Test PASSed.


Feature/multiple return

Add return type to Pandar40P and PandarQT point types.

The different devices handle multiple returns differently so classification of return type must be different
When the sensor is in dual return mode, the user can select to publish only "Strongest" or "Last" (for Pandar 40P) or "First" or "Last" (for PandarQT) with the parameter return_mode
If the device is in a setting that does not allow publishing to match the return_mode parameter (for example "Strongest" requested when the sensor is in "Last" mode) then a warning is issued. Therefore this setting is only useful when the sensor is in "Dual" mode.

Notes:
Tested on Pandar QT and Pandar40P packets data in dual mode.
　Can I start a review?

Sorry I missed this message! Please go ahead and review, thank you :)

LGTM


Remember file extension when exporting plots from interactive

Would be cool if the export plot interface retained the last selected file extension. For me it always reverts back to PDF.

Thanks for the suggestion @janosh! We'll discuss this enhancement in our next triage meeting.


A new decorator: @ignoreignore_exception

The decorator will automatically ignore the exception raised in the decorated function.

Done in pr#13


ffmpeg freeze

I want to use ffmpeg pull rtsp stream  to nginx hls.
I do long time test,  the ffmpeg freeze,  can't  pull stream any more
the ffmpeg command:
for((;;));do
ffmpeg -rtsp_flags prefer_tcp -i rtsp://myipcame-rtsp-url  \
-vcodec copy -an -f flv rtmp://<IP_ADDRESS>/hls/cam;
current_date=date "+%Y-%m-%d %H:%M:%S"
echo "=======================================>$current_date stream disconnect, reconnect it....."
sleep 1;
done
=====
ffmpeg freeze, the process never stop,  If can stop,  my shell script can create a new one to pull stream.
Any one  has the problem?

Nobody face this problem?   I think it easy to produce again.
I find this bug  using exec_static  firstly.
Then I use shell command line reproduct it.

Can you try to stream some image to check that is no network lag?
ffmpeg -re -loop 1 -i ./target/target_image.png -r 24 -vcodec libx264 -f flv "rtmp://<IP_ADDRESS>/hls/cam live=1 timeout=0 buffer=0"

And wait for freeze.

Use this command line,  Not freeze.
But  pull rtsp streaming (tcp),  will freeze
On Wed, Aug 5, 2015 at 12:25 AM, Sergey<EMAIL_ADDRESS>wrote:

Can you try to stream some image to check that is no network lag?
ffmpeg -re -loop 1 -i ./target/target_image.png -r 24 -vcodec libx264 -f flv "rtmp://<IP_ADDRESS>/hls/cam live=1 timeout=0 buffer=0"
And wait for freeze.
—
Reply to this email directly or view it on GitHub
https://github.com/arut/nginx-rtmp-module/issues/659#issuecomment-127664887
.

OK, I think this is a ffmpeg bug.  I  find a simple way to fix ffmpeg, make
sure that if no stream data for 5 seconds, exit ffmpeg.
Now my shell script work fine.
On Thu, Aug 6, 2015 at 9:14 AM, Jim Ying<EMAIL_ADDRESS>wrote:

Use this command line,  Not freeze.
But  pull rtsp streaming (tcp),  will freeze
On Wed, Aug 5, 2015 at 12:25 AM, Sergey<EMAIL_ADDRESS>wrote:

Can you try to stream some image to check that is no network lag?
ffmpeg -re -loop 1 -i ./target/target_image.png -r 24 -vcodec libx264 -f flv "rtmp://<IP_ADDRESS>/hls/cam live=1 timeout=0 buffer=0"
And wait for freeze.
—
Reply to this email directly or view it on GitHub
https://github.com/arut/nginx-rtmp-module/issues/659#issuecomment-127664887
.

Maybe ip-cam just stop streaming? Like motion detection triggered.
Issue closed?

I think change ffmpeg code can meet my need
(pull ip-cam streaming,  when the network problem,  reconnect it ).
So i think i can close this issue.
Anyway, thanks for your reply.


steal-conditional boolean throw an error

@pYr0x commented on Tue May 16 2017
i used the steal-conditional boolean syntax to load a module.
require("~/modules/notice-box#?triwala-vk/modules/notice-box-exists");
reduce the notice-box-exists to the basic the module looks like this:
import $ from 'jquery';
const foo = false;
export default foo;

when i create a production version i get this error

the problem is "jquery" but i dont know why.
@m-mujica any idea why jquery cause an error in steal-conditional
if i import a module that does not use jquery i works.
import '~/util/url';
url.js
exports.setLocation = function (url) {
	var htmlDoc = document.getElementsByTagName('html')[0];
	htmlDoc.classList.add('wait');
	window.location.href = url;
	return false;
};

@pYr0x commented on Tue May 16 2017
i found out that module.exports = !!window.document.querySelector('.notice-box'); is not working too.
window is not defined. is the problem that while build, the loader is using node and there is not window object present?

@m-mujica commented on Tue May 16 2017
@pYr0x are you using the latest version of steal-conditional?
steal-conditional should not eval your condition module (notice-box-exists) during the build
See https://github.com/stealjs/steal-conditional/blob/68283fd641bf97aa969946576fd6c4a05ae0d1cc/conditional.js#L323-L327
I'll try to replicate the jquery issue.

@m-mujica commented on Tue May 16 2017
@pYr0x in this case, since you're using the boolean syntax, steal-conditional should make a bundle of ~/modules/notice-box. https://github.com/stealjs/steal-conditional/blob/68283fd641bf97aa969946576fd6c4a05ae0d1cc/conditional.js#L259

@pYr0x commented on Tue May 16 2017
@m-mujica i used "steal-conditional": "^0.3.2"
notice-box is bundled into a file called index-form-index-bd4d57f2.js
and
notice-box-exists is also bundled into a file
<EMAIL_ADDRESS><EMAIL_ADDRESS>function (require, exports, module) {
    const loader = require('@loader');
    let ex = false;
    if (loader.isPlatform('window')) {
        ex = !!document.querySelector('.notice-box');
    }
    module.exports = ex;
});

and if i use a console.log in the module notice-box-exists the output is shown during the build. so the module will be eval during build

@m-mujica any plan of fixing this bug? do you need a example app?

@pYr0x I haven't looked at this. I'll try to get some time before end of week to fix it. I'll let you know if I run into issues replicating the bug.

@pYr0x I just published<EMAIL_ADDRESS>with the fix, I did a small example to replicate your issue and it build correctly with 0.3.5
Please give it a try when you have some time and let me know if you run into any issue.


merge package template from Kronos-Tools/npm-package-template

Updated standard to latest version

Codecov Report

Merging #144 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #144   +/-   ##
=======================================
  Coverage   98.03%   98.03%
=======================================
  Files           1        1
  Lines         102      102
=======================================
  Hits          100      100
  Misses          2        2

Continue to review full report at Codecov.

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


Workflow and Agent should support custom implementation

Since there are still some flaws in the existing LLM capabilities, it is difficult to implement some complex logic by LLM decision-making, and it will be more stable to implement it through code.

I believe this needs to be analyzed on a case-by-case basis. My perspective is that unless there are very clear accuracy requirements that should be calculated, LLM should be used as much as possible where it can be used. As a CPU, the robustness of LLM is really impressive.

Many times in applications, deterministic results are required, but LLM cannot guarantee to return deterministic results every time, at least this is my experience. What we make is a system. If a user wants to develop an app based on it, but the user just can’t find the appropriate prompt word, you can’t tell the user that you are not capable enough and stop developing it. You should give the user a way to solve the problem.

In version 0.5.2, we are granting more flexibility in how the LLM Process and Agent behaviors are configured. This enhancement should provide users with increased adaptability to suit their specific needs. Thank you for your continued support!


Navigate back after search and replace

When using Poedit, I'm always afraid of using the search and replace, even if it would be useful. That's because I'm losing the editing position. There are no bookmarks anywhere that I could use, and there is no "Continue where you were before" button in the Replace dialog.
Typically, when I want to use the search and replace function, I want to interrupt my main task of translating the whole file, but only for a brief moment, then I want to return.
I think this is a typical workflow, not only for me but also for other Poedit users, therefore I think supporting it would benefit several translators.

I understand the problem (I think; although I'm not sure if this applies to just Replace or Find as well). But I'm not sure how to do UI for this. ..

As in today, you can place bookmarks to some places to better navigate long files. My question is how to clear a set bookmark once set?

It applies to both Find and Replace.
Another interpretation of that idea is to have a "main editing" window, and when I discover an interesting word while translating, I can open a second editing window. In that window, I can filter the messages like with grep, to see only those that contain the interesting word. The second window contains an editor as well, so that I can fix typos or small wording changes. When I no longer need that second window, I can close it.
As long as a single message is not edited in several windows at the same time, this should be doable in terms of UI, as there are no editing conflicts. There would have to be a notification mechanism though, when a message has been modified in one window and is displayed in another window. Also, the filtering needs to be updated in real time.


piv: mark PC/SC connection as exclusive

Since this package holds onto a transaction, ensure other calls to Open exit immediately instead of hanging.
Updates #45
Updates #47

Going to merge this for now since the YubiKey does require an exclusive transaction. If we figure out how to address #47, we can move it back to shared.


Cherry pick Remove unpassable cargo publish check from verify-release-candidate.sh to active_release

Automatic cherry-pick of 898924f

Originally appeared in https://github.com/apache/arrow-rs/pull/882: Remove unpassable cargo publish check from verify-release-candidate.sh

Resolves #948
This bug got introduced in 6.1.0 and I fixed in on master, but I didn't realize people used the release verification script from the tarball itself. Hopefully this will be in the 6.3.0 release
cc @xudong963

Codecov Report

Merging #949 (90333f7) into active_release (311d59f) will not change coverage.
The diff coverage is n/a.

@@               Coverage Diff               @@
##           active_release     #949   +/-   ##
===============================================
  Coverage           82.39%   82.39%
===============================================
  Files                 168      168
  Lines               48668    48668
===============================================
  Hits                40101    40101
  Misses               8567     8567

Impacted Files
Coverage Δ

arrow/src/array/transform/mod.rs
85.33% <0.00%> (ø)

Continue to review full report at Codecov.

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


Need some kind of refresh button for Functions list, fixed #23

other than the 1 comment, it looks good!


Unable to find pin connection for BME280

Would you please guide me where you have defined the pins for BME sensor?

See in app/config.h file


[BUG] Open documentation for selection goes to wrong page

Describe the bug
Extension always opens file:///Applications/Unity/Documentation/en/ScriptReference/30_search.html
To Reproduce
Steps to reproduce the behavior:

Go to a c# files
Right-Click on a class
Select 'Unity Tools: Open documentation for selection'
file:///Applications/Unity/Documentation/en/ScriptReference/30_search.html opens with no params

config
Extension is configured for local documentation : "Applications/Unity/Documentation/en/ScriptReference"
Local documentation viewer = "safari"
Expected behavior
Help for the selected class should open. Eg:
file:///Applications/Unity/Documentation/en/ScriptReference/30_search.html?q=GameObject
VSCode (please complete the following information):

OS: OS X
Version Mojave

To clarify, the extension searches for what you have highlighted. If you don't have something highlighted, it takes you to the blank search page.
Is it the case if you do, between your step 1 and 2, highlight "GameObject" and then right click/search it works?
If that is the case: I think I might need to add a feature to see if I can auto highlight/search the right click selected word rather than open a blank search page. That would probably be a useful feature either way.
If you actually are highlighting a word, and it still doesn't search correctly, that is a bug. Let me know if that is the case, and I can figure out what might be happening.

Hi!
Thanks for reaching me back.
I Confirm I am highlighting a word and that this word is not passed on to safari. Even ?q= is missing.
Cheers!
Mick

Interesting!
Is that also the case if you trigger via Command Palette (⇧⌘P) as well?

You should be also able to trigger via Command Palette with nothing highlighted, and it will prompt you to enter some text. You should be able to enter "Test" or whatever you want, and search.
If that doesn't work, I wonder if this is a bug with passing params like ?q=something into Safari via the open method. I had not tested that since I don't have easy access to a Mac.

Debugging to do:

[ ] Try to trigger via command palette rather than right-click
[ ] Does search work as expected via another browser? (ex. Firefox)
[ ] Does search work as expected when not searching local documentation?

command palette same issue
with chrome nothing happens at all
with online doc it works like a charm

Did you enter "chrome" or "Google Chrome"? If nothing happens at all, probably just not finding the chrome app.
One of the issues with searching local documentation is needing to pass in the browser. This was because of issues I ran into that general "open url" commands were parsing out the ~?=
Hence why I'm suspecting it might be safari that's doing that automatically. But I could be wrong.

You might also want to try "/Applications/Google Chrome.app"

Chrome do open with full path in settings (/Applications/Google Chrome.app)
But then again, it takes no parameter.

Also tried this setting with no luck:
/usr/bin/open -a Safari

Interesting, so like Safari, it opens but the parameters are either being parsed or not assigned.
I think this is related to the main problems of #21 that file:/// is not designed to take parameters like HTTP is. The whole setting viewer thing was a hack in itself.
Tobiah's To-Do:

[ ] Get access to a Mac to test
[ ] Use debugging and/or change "Base" URL to ensure the extension is correctly sending the parameters in the first place.
[ ] Check if open works with parameters on mac without setting app

Other general weird debugging ideas

[ ] What happens if localDocumentationViewer is blank? (May need to be set to something non-null like a space " " to override the default value of "firefox")
[ ] Does firefox work, where chrome/safari don't?

Overall, it may require a rewrite of how local documentation works to do something like suggested here: https://stackoverflow.com/questions/35557531/parameters-are-removed-when-opening-local-url-in-default-browser
Generating an HTML page and loading from that, which redirects to the correct URL with parameters.

Hi,
I've investigated a little more and open will not accept URL params: https://stackoverflow.com/questions/9533720/open-a-web-page-with-url-params-from-the-command-line-locally
this works:
osascript -e 'tell application "Safari" to open location "file:///Applications/Unity/Documentation/en/ScriptReference/30_search.html?q=GameObject"'
Cheers!
Mick

I think that further cements I need to do a system where it launches a local html page that has been prewritten to automatically redirect. Then I can remove the need to pick the browser entirely, and remove open as a dependency for the project

👌let me know when you need me to test it.

Hello
I just started using your extension (great tool btw) v1.2.12 and noticed the same behaviour (appended "/30_search.html").
I'm on OSx Catalina 10.15.7, Unity Editor v2019.4.18f1, VS Code Version: 1.52.1., Safari Version 14.0.1
In my VSCode settings file I have:
"unity-tools.localDocumentationPath" : "Applications/Unity/Hub/Editor/2019.4.18f1/Documentation/en/ScriptReference",
"unity-tools.localDocumentationViewer": "safari"
TO reproduce behaviour I highlight a method in a cs file and cmd+' (shortcut key ... same result with right-click search).
Any luck on this?


Resolve small typo in 06-keyboard.md

Updated properities to properties

These will all be moved to _docs.rb and will use org-mode formatting: https://orgmode.org/manual/


Doc(plugins): Fix documentation typos and errors

Plugins document fixes:

Missing paranthesis, )
MarkDown errors fixed, md007, md031, md032
Typos fixed

Change Summary

Related Issue(s)
Fixes #1237
Component(s) name
arista.avd.<role-name>
Proposed changes

How to test

Checklist
User Checklist

N/A

Repository Checklist

[x ] My code has been rebased from devel before I start
[ x] I have read the CONTRIBUTING document.
[ x] My change requires a change to the documentation and documentation have been updated accordingly.
[ ] I have updated molecule CI testing accordingly. (check the box if not applicable)

Let me know if I have to make any further changes to examples.


gmsh: 4.0.5 -> 4.0.7

Motivation for this change
Keeping 3rd party dependencies for FreeCAD up to date
Things done

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

[ ] NixOS
[ ] macOS
[ ] other Linux distributions

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

https://gitlab.onelab.info/gmsh/gmsh/blob/master/CHANGELOG.txt

@GrahamcOfBorg build gmsh

Keeping 3rd party dependencies for FreeCAD up to date

As far as I can see, gmsh is not an input of freecad.
When I tried to test the GUI it complains that it can not find opencascade (which is not an input of gmsh). Is there maybe a dependency not properly defined?

Keeping 3rd party dependencies for FreeCAD up to date

As far as I can see, gmsh is not an input of freecad.
It is required if you want to run the Finite Element Material Workbench


[📚] Error in Application.java in "Optionally handle background messages" (CHANGE ME)

Hello guys, there is an error in the steps to setup the app to process messages in the background. In the second step, you're asked to create a file called Application.java in /android/app/src/main/java// and it is given the content to put in there, but there is an error in the following section:
@Override
public void registerWith(PluginRegistry registry) {
    GeneratedPluginRegistrant.registerWith(registry);
}

Which should be changed for:
@Override
public void registerWith(PluginRegistry registry) {
  FirebaseMessagingPlugin.registerWith(registry.registrarFor("io.flutter.plugins.firebasemessaging.FirebaseMessagingPlugin"));
}

And the following import should be added at the top:
import io.flutter.plugins.firebasemessaging.FirebaseMessagingPlugin;
Please correct that.

Closing since this is no longer relevant


Vector to columns: added support for splitting arrays of different lengths

Added support to split arrays of different lengths in the vector_col. If
the user does not provide feature names, we pad each array to the
maximum length and split across the maximum possible number of features.
If the user does provide feature names, we truncate/pad the arrays
according to the number of features the user desires. Added dev-check
tests and removed unit test to incorporate this change.
Co-authored-by: Rahul Iyer<EMAIL_ADDRESS>

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/madlib-pr-build/651/

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/madlib-pr-build/655/


RCTAsyncStorage.getAllKey callback does not happen RN 0.31 Android 4.2.2

Android 4.2.2
RN 0.31
RCTAsyncStorage.getAllKeys(function(error, keys) { line in AsyncStorage.js, getAllKeys method never gets to callback when debugging in chrome.
Also, AsyncStorage.js chooses to use sqlite implementation (at least in the chrome debugger).
This issue happen only in debug.

I have replicated this issue on 0.31 and 0.32.0-rc.0.
Haven't isolated the problem, but can confirm that AsyncStorage.getAllKeys works while running on the device normally, but not when debugging JS remotely in Chrome.

I'm also experiencing this issue. It means I haven't been able to use the chrome debugger since upgrading to 0.31. Anyone have any workarounds for now?

@alenia nope, still broken for me. Haven't had a chance to investigate.

I have the same issue, it works as expected in 0.30.
I have downgraded until the cause is determined. I have tested on a Nexus 5 a 6.0.1, and a Nexus 6P at N preview 5

I hit this as well in 0.32.0-rc. I suspect that the commit in 0.31.0 (c89c25f27d115c1928104b06dbe6eabf005995a6) which makes the new bridge the default bridge introduced the issue.

@cjhopman, do you have any thoughts on this issue. Do you think my theory about your commit (c89c25f27d115c1928104b06dbe6eabf005995a6) is accurate?

@rigdern the bridge is definitely the issue. I modified my MainApplication to call setUseOldBridge(true) in the ReactNativeHost subclass, and I'm now able to use AsyncStorage while debugging on Android with 0.31.0
I don't have the time to debug this issue fully beyond that, but for those experiencing this issue, I posted a workaround: https://gist.github.com/kageurufu/79e4aaa1c41cbdb147e936d7e473029d

@rigdern Thanks for looking into it! I'll ping @cjhopman and @mhorowitz on fb@work messenger.

Closing in favor of https://github.com/facebook/react-native/issues/9436


flow in webstorm

why webstorm highlight the flowing code:
type ObjectAlias = {     property: string,     func : (arg?:string)=>number };

eslint is disable

as a real example when I open the parse SDK (https://github.com/parse-community/Parse-SDK-JS) in webstorm I get same error at CoreManger.js

This probably can be closed.
/cc @vkurchatkin


Fix error boundary to correctly handle and log errors

The app's error boundary isn't set up to properly handle errors, they are just re-thrown and no usefull message is shown to users.
handler declared here,
https://github.com/hirosystems/stacks-wallet-web/blob/8788e010b9504f1cfa5c8f2c619af19da240dbc8/src/app/features/errors/error-boundary.tsx#L154-L159
passed as arg here (TS accepts the different fn signature)
https://github.com/hirosystems/stacks-wallet-web/blob/8788e010b9504f1cfa5c8f2c619af19da240dbc8/src/app/features/errors/app-error-boundary.tsx#L52-L65
called here,
https://github.com/hirosystems/stacks-wallet-web/blob/8788e010b9504f1cfa5c8f2c619af19da240dbc8/src/app/features/errors/error-boundary.tsx#L94-L96

Closing this in favour of #3986


get ready for 200.5 release

Description
Description
Updating the version numbers in preparation for the 200.5 release
Checklist:

[x] Set target branch to main (current release) or v.next (next release)
[x] Request a reviewer
[x] Assign a reviewer
[x] Tag reviewer in comment

@mbcoder could you please perform the first review?

@colinanderson could you please do a second review?


notifications: Support tapping a notification to open the conversation

After #122.

It might be useful to study how we do this in zulip-mobile. With

https://github.com/zulip/zulip-mobile/pull/5740

, we simplify the case that was the most opaque in zulip-mobile, because it relied on a low-quality library: on iOS, tapping a notification to launch the app (from closed).


Allow option to create a GitHub release

Currently there is the possibility of creating PyPI and conda releases. Would it be of interested to create a release in GitHub as well?
I didn't find existing functionality and assumed you do this manually currently?

Really sorry I pressed merge on the wrong PR!

Do you mind reopening this PR please.  Sorry about this

@fabge thanks for the very neat PR! We actually have something for this already though, which is our fastai/fastrelease project. But your PR has inspired me to merge it into nbdev, so please watch this space :)

Thank you very much, I will definitely watch it closely. The work from all of you is incredibly inspiring and teaches me a lot!

Here you go @fabge https://nbdev.fast.ai/release.html


Question : how to check if file exists ?

Is it possible to check if some file exists in orbit.yml.
I want to create a conditional task that runs one command if some file exists and another one if not.
Thanks in advance for your help.

Hello @kpym,
AFAIK, there is currently no template method for that. You have to rely on a OS native command 🤔

@gulien Can I recover the result of shell command in template variable in a way to use after that something like {{ if eq $result 0 }} ?

No you can’t, for such requirements you have to rely on a custom script ^^

😢 Bad news. Deciding what to do depending on the file existence is very useful feature.
In any case, thanks for your answer and for your work. 🙏

Yep, there are a lot of features that could be nice to add 👍 I’ll try to find some time in the future !


[C++] Bump bundled Google benchmark version

Describe the enhancement requested
Title says it all.
Component(s)
Benchmarking, C++

This would pick up useful bugfixes such as https://github.com/google/benchmark/pull/1631

@Cerdore Can you post something here so that we can assign the issue to you?


ddev: 1.23.0 -> 1.23.1

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://repology.org/project/ddev/versions.
meta.description for ddev is: Docker-based local PHP+Node.js web development environments
meta.homepage for ddev is: https://ddev.com/
Updates performed

Golang update

To inspect upstream changes

Release on GitHub

Compare changes on GitHub

Impact
Checks done

built on NixOS
The tests defined in passthru.tests, if any, passed
found 1.23.1 with grep in /nix/store/kpb9y6qc8qxvw6nkan0h81296qs0hsbg-ddev-1.23.1
found 1.23.1 in filename of file in /nix/store/kpb9y6qc8qxvw6nkan0h81296qs0hsbg-ddev-1.23.1

Rebuild report (if merged into master) (click to expand)

2 total rebuild path(s)

2 package rebuild(s)

First fifty rebuilds by attrpath

ddev

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/kpb9y6qc8qxvw6nkan0h81296qs0hsbg-ddev-1.23.1 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A ddev https://github.com/r-ryantm/nixpkgs/archive/b8a56384a00ff3036a6d66c38c122e173b14401a.tar.gz

Or:
nix build github:r-ryantm/nixpkgs/b8a56384a00ff3036a6d66c38c122e173b14401a#ddev

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/kpb9y6qc8qxvw6nkan0h81296qs0hsbg-ddev-1.23.1
ls -la /nix/store/kpb9y6qc8qxvw6nkan0h81296qs0hsbg-ddev-1.23.1/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review --extra-nixpkgs-config '{ allowInsecurePredicate = x: true; }' run on x86_64-linux 1

1 package built:

ddev

Maintainer pings

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

1 package built:

ddev


Auth on KV discovery & design doc

closes #3348

@itaiad200 how do we address the user id and group id which are currently a serial type in our database - auto-generated key?

To my understanding, there's no real need for those, right? It's a good opportunity to deprecate them, if there's any reason for keeping them, then yes, auto generated keys should work.

@itaiad200 how do we address the user id and group id which are currently a serial type in our database - auto-generated key?

To my understanding, there's no real need for those, right? It's a good opportunity to deprecate them, if there's any reason for keeping them, then yes, auto generated keys should work.

True. So the lakefs implementation will have to change and we will need to migrate the existing IDs to generated one.


[SR-1834] strange compilation error

Previous ID
SR-1834

Radar
None

Original Reporter
VladimirS (JIRA User)

Type
Bug

Environment
IBM Swift Sandbox
Swift Ver. 3.0 (Jun 6, 2016)
Target: x86_64-ubuntu-linux-gnu

Additional Detail from JIRA

Votes
0

Component/s
Compiler

Labels
Bug, DiagnosticsQoI, TypeChecker

Assignee
None

Priority
Medium

md5: 20812fa1fc8126fe7e5b31e9d5e21b04

Issue Description:
If I have this code, it compiles without errors:
    func group<I: Equatable>(i: I) {
        var groups = [(key:I, value:Int)]()
        groups.append((key: i, value: 1))

        if groups[0].key == i {
            print("Yes")
        }
    }

But if I remove ": Equatable" from declaration, I got strange error :
ERROR at line 16, col 18: cannot subscript a value of type 'inout [(key: I, value: Int)]' (aka 'inout Array<(key: I, value: Int)>')
        if groups[0].key == i {
           ~~~~~~^

I.e. error appears for this code:
    func group2<I>(i: I) {
        var groups = [(key:I, value:Int)]()

        groups.append((key: i, value: 1))

        if groups[0].key == i {
            print("Yes")
        }

    }

I believe compiler produces wrong description of the error and it is a bug.

Auditing some of those old typechecker issues to see if they are still relevant, this seems to be fixed compiler now produces referencing operator function '==' on 'Equatable' requires that 'I' conform to 'Equatable' so I think we can close. Feel free to reopen if you still think there is still something to be discussed.


feat(changelog)!: Merge similar commits within group

When changelog contain commits like Text (#123) and Text (#456) in some group (Features, Fixes, Refactoring, Other) badabump now will merge those commits into Text (#123, #456) line.
Fixes: #189

Coverage increased (+0.06%) to 97.238% when pulling df368c796052bd2e048c4568bf2d49c1bae64158 on feat/189-merge-similar-commits into 41cb2b8229cd934905d716aa3b3ff355b4e9c20d on main.


Add debug flag -RegexBytecodeDebug and initial RegExp bytecode refactoring

(Note: I'll rebase this to a single commit shortly.)
-RegexDebug displays various views of RegExp compilation in ChakraCore. In an effort to improve the memory efficiency of the RegExp bytecode that makes up our compiled RegExp programs, this change adds a new view under the additional flag -RegexBytecodeDebug to view the bytecode binary layout.
For example, for this test case:
let re = /[ab]c+(d|e)?/i;
console.log(re.test('abcce')); // true

Running ch.exe -RegexDebug will display some output including the AST which is used to generate the bytecode. The bytecode is displayed last (in a human-readable disassembly), and looks like this:
REGEX PROGRAM /[ab]c+(d|e)?/
Program {
    source:       [ab]c+(d|e)?
    flags:        ignorecase
    numGroups:    2
    numLoops:     0
    instructions: {
        L0000: SyncToSetAndBackup(set: [Cc], backup: [1-inf])
        L0040: SyncToSetAndConsume(set: [ABab])
        L0078: ChompSet<Plus>(set: [Cc])
        L00b0: TryMatchSet(set: [DEde], failLabel: L0110)
        L00f0: DefineGroupFixed(groupId: 1, length: 1, noNeedToSave: true)
        L0110: Succ()
    }
}

Now you can run ch.exe -RegexDebug -RegexBytecodeDebug and see the following more detailed view of the disassembly which includes the actual bytecode layout.
REGEX PROGRAM /[ab]c+(d|e)?/
Program {
    source:       [ab]c+(d|e)?
    flags:        ignorecase
    numGroups:    2
    numLoops:     0
    instructions: {
        L0000: (0x040 bytes) SyncToNegatedSetAndBackup(set: [Cc], backup: [1-inf])
            0x000001C19403C128[+0x008](0x008) [Inst]: 2f000000 00000000
            0x000001C19403C130[+0x010](0x028) [SetMixin<false>]: 00000000 00000000 00000000 00000000 08000000 08000000 00000000 00000000 00000000 00000000
            0x000001C19403C158[+0x038](0x008) [BackupMixin]: 01000000 ffffffff
        L0040: (0x038 bytes) SyncToNegatedSetAndConsume(set: [ABab])
            0x000001C19403C168[+0x008](0x008) [Inst]: 27000000 00000000
            0x000001C19403C170[+0x010](0x028) [SetMixin<false>]: 00000000 00000000 00000000 00000000 06000000 06000000 00000000 00000000 00000000 00000000
        L0078: (0x038 bytes) ChompSet<Plus>(set: [Cc])
            0x000001C19403C1A0[+0x008](0x008) [Inst]: 4c000000 00000000
            0x000001C19403C1A8[+0x010](0x028) [SetMixin]: 00000000 00000000 00000000 00000000 08000000 08000000 00000000 00000000 00000000 00000000
        L00b0: (0x040 bytes) TryMatchSet(set: [DEde], failLabel: L0110)
            0x000001C19403C1D8[+0x008](0x008) [Inst]: 58000000 00000000
            0x000001C19403C1E0[+0x010](0x028) [SetMixin<false>]: 00000000 00000000 00000000 00000000 30000000 30000000 00000000 00000000 00000000 00000000
            0x000001C19403C208[+0x038](0x004) [TryMixin]: 10010000
        L00f0: (0x020 bytes) DefineGroupFixed(groupId: 1, length: 1, noNeedToSave: true)
            0x000001C19403C218[+0x008](0x008) [Inst]: 3a000000 00000000
            0x000001C19403C220[+0x010](0x004) [GroupMixin]: 01000000
            0x000001C19403C224[+0x014](0x004) [FixedLengthMixin]: 01000000
            0x000001C19403C228[+0x018](0x001) [NoNeedToSaveMixin]:01
        L0110: (0x010 bytes) Succ()
            0x000001C19403C238[+0x008](0x008) [Inst]: 02000000 00000000
            0x000001C19403C230[+0x000](0x010) [NopInst]: (no unique data -- skipping)
    }
}

Background and Future Opportunities
The following is some background on what you're seeing above and possible optimizations this exercise has illuminated.
The bytecode is constructed by using placement-new to allocate structs representing bytecode operations into a memory buffer. Since every opcode (aka instruction "tag") value fits into a single byte (at present), the above view makes it clear how much internal fragmentation results from adding padding for small quantities like the opcode (aka instruction tag), and small quantities.
For many opcodes for small regexes, many quantities also fit into a single byte or a short, instead of an int32, so there is a lot of support for the idea of adding variable-length encoding to save space.
Some portions of the opcode data like SetMixin are wasteful by always including space for a pointer (when we might know from the AST that this is guaranteed to be nullptr before compiling the bytecode), and we could split that opcode into two separate operations to save the 8 bytes used for the pointer when it is not needed. This would also remove a branch from execution and save on runtime cost.
Additionally, the bit vector representing the first 256 characters may be wasteful because 128-255 are not ASCII and characters in that range may be uncommon enough that it would be more efficient overall to encode that information in the same data structure used for other unicode code points (the data structure stored by the aforementioned sometimes-null pointer).
Many opcodes encode one or more booleans which get padded out to 4 or 8 bytes (depending) for a single bit of information. Encoding that information in a template  param would follow the practice of other places in the bytecode and significantly save on the size of those bytecode layouts.
All of the bytecodes start at offset [+0x008] because I'm skipping over the Debug-only vtable pointer when displaying the layout. I'm planning on refactoring out the virtual Print method (in favor of a static alternative) because this code already uses static dispatch based on the bytecode's opcode in many places, and this vtable pointer distorts how the bytecode would be laid-out in production.
Throughout this exercise, I've been bouncing ideas off of @Penguinwizzard. Thanks to him for listening and reviewing my work in progress!

What does perf look like?

@MikeHolman This is a debug-only visualization tool. No changes to perf. This refactoring introduced a slight memory regression which will disappear when I move the boolean fields into template params.

Ah, reading is hard. Thought the optimizations you mention were part of this.

The comments in RegexOpcodes.h - are those left over or intended to serve as pointers for future work?

@rajatd Thanks, I meant to remove most of those comments. I'll do so. Any other TODOs left over in this patch will be indicators for future work.

I will rebase this PR soon. Original commits can be found at dilijev/re-bc-0 in case that aids in reviewing.

@dotnet-bot test Windows 7 ci_dev12_x64_debug please

^ ah, we removed the legacy checks so the dev12 check doesn't exist anymore.
/cc @Penguinwizzard I updated the set of required checks for master. (FYI @MSLaguana)

    PRINT_BYTES(NopInst);

SuccInst

Refers to: lib/Parser/RegexRuntime.cpp:1028 in 27a2cb8. [](commit_id = 27a2cb884df76c7ff968f36296bba2a3f1a67ce3, deletion_comment = False)

    PRINT_BYTES(NopInst);

FailInst

Refers to: lib/Parser/RegexRuntime.cpp:1006 in 27a2cb8. [](commit_id = 27a2cb884df76c7ff968f36296bba2a3f1a67ce3, deletion_comment = False)

// actual opcode name, even though the logic is basically the same in each definition. See notes below.

Could avoid the duplication with a macro

Refers to: lib/Parser/RegexRuntime.cpp:2113 in 27a2cb8. [](commit_id = 27a2cb884df76c7ff968f36296bba2a3f1a67ce3, deletion_comment = False)


[Translations] Partial Update - Spanish

Updated spanish translations

cool

thanks @surfdude29 @oledfish for ur support
We just need to check out 4000 more code lines to check

thanks @surfdude29 @oledfish for ur support
We just need to check out 4000 more code lines to check

It's OK to just update and improve some of the translations, it doesn't all have to be done in one PR :)

Should be ok to leave the word "Post" instead "Publicación" ?
@gentlecode1 @surfdude29 @Kloen @oledfish

Seems all issues has been resolved. Please review one more time to approve them

Wating for review


Add semester model

What this does
This PR adds a Semester model to ResiReg. This is part of the effort to have a concept of Semesters, so that we can work with allocations as a semesterly basis, and archive them when the semester is over.
I still need to do the following:

[ ] Display semester data from the semester in MainWindow
[ ] Implement the archival feature
[ ] Test JsonAdaptedSemester more effectively

Part of #11.
How to test

Notes

I have created a roomFees map in my Semester model, which may be used in the future for billings. I've also written tests for those. However, this architecture may not be the most thought out, so whoever is working on billing in the future can feel free to modify it, and perhaps extract it out to its own class.

Codecov Report

Merging #102 into master will increase coverage by 0.34%.
The diff coverage is 62.60%.

@@             Coverage Diff              @@
##             master     #102      +/-   ##
============================================
+ Coverage     55.74%   56.08%   +0.34%
- Complexity      539      572      +33
============================================
  Files           109      114       +5
  Lines          2185     2309     +124
  Branches        259      276      +17
============================================
+ Hits           1218     1295      +77
- Misses          904      930      +26
- Partials         63       84      +21

Impacted Files
Coverage Δ
Complexity Δ

.../java/seedu/resireg/model/util/SampleDataUtil.java
10.00% <0.00%> (-1.54%)
1.00 <0.00> (ø)

src/main/java/seedu/resireg/ui/MainWindow.java
0.00% <ø> (ø)
0.00 <0.00> (ø)

...ava/seedu/resireg/storage/JsonAdaptedSemester.java
57.57% <57.57%> (ø)
2.00 <2.00> (?)

...ava/seedu/resireg/model/semester/AcademicYear.java
60.00% <60.00%> (ø)
6.00 <6.00> (?)

...a/seedu/resireg/model/semester/SemesterNumber.java
60.00% <60.00%> (ø)
8.00 <8.00> (?)

...in/java/seedu/resireg/model/semester/Semester.java
63.82% <63.82%> (ø)
10.00 <10.00> (?)

...in/java/seedu/resireg/model/semester/RoomRate.java
75.00% <75.00%> (ø)
4.00 <4.00> (?)

...u/resireg/storage/JsonSerializableAddressBook.java
64.10% <75.00%> (-0.61%)
5.00 <0.00> (+1.00)
:arrow_down:

src/main/java/seedu/resireg/model/AddressBook.java
67.69% <100.00%> (+3.28%)
21.00 <3.00> (+2.00)

...rc/main/java/seedu/resireg/model/ModelManager.java
69.31% <100.00%> (+0.35%)
28.00 <0.00> (ø)

... and 6 more

Continue to review full report at Codecov.

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

While working on this PR, I noticed traces of leftover AddressBook in our app. I've purged most of it in 651a1c2 , although the documentation requires updating.


[SMALLFIX] Run key-value examples via bin/tachyon

+1 LGTM

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Tachyon-Pull-Request-Builder/8266/
Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Tachyon-Pull-Request-Builder/8267/
Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Tachyon-Pull-Request-Builder/8270/
Test PASSed.

LGTM


_in_stoplist should return True for entities trimmed out of existence

In an entity like "-aromatic" which is in IGNORE_SUFFIX the resultant entity after running _in_stoplist is of length 0, hence the entity should be ignored (i.e. the function should return True) rather than reporting a 0 length entity.
On an entity which is both in IGNORE_PREFIX and IGNORE_SUFFIX you can get into a situation where the end index is actually before the start end index!

d = Document("non-aromatic")
d.cems
[Span(u'', 4, 3)]

I assume adding this check that the resultant entity's length is > 0 will fix that case as well.

Oops! Nice catch, thanks.


Vmail is great, make it free

Free as in not bound to one specific email provider.
Wouldn't it be nice to have it working with just any IMAP server? Just point it to the right folders (Sent, Archive, Junk, Trash) and voila. It would make a great alternative for mutt.

Please give it a whirl and let me know how it goes:
https://github.com/danchoi/vmail/wiki

Hi,
I'm trying to set Vmail up with Outlook. Having followed those instructions, reading emails works really well, but I can't send emails. Vmail doesn't offer any errors - the only indication that it doesn't work is that I don't receive the emails.
I suspect the problem is in the configuration:
smtp_domain: ```

The SMTP server is obvious - but I've never heard of an "SMTP domain" before. I've tried leaving it blank, filling it in with the domain of my email address (e.g. outlook.com), and filling it in with the SMTP server.

Is there any way to make Vmail be more verbose about how it's trying to send the email?

Thanks!


[WIP] gRPC rebase fixes

This PR is a set of fixes that I discovered while rebasing giantswarm/grpc_engine_communication on top of coreos/master.
This PR cannot be compiled, as necessary vendor packages are missing.
Functional tests also cannot run, because it's based on a slightly old branch that conflicts with current master.
So do not merge please!
FYI, a complete branch based on the current coreos/master is already available : https://github.com/endocode/fleet/tree/dongsu/grpc_engine_communication
/cc @hectorj2f

Thanks @dongsupark, I'll have a look at it.

@dongsupark LGTM, are you done with this PR ?

@dongsupark as you realized, I did some cherry picking here. Thank you 👯 ! Closing this PR at this moment.


Set tray icon as window icon

Currently, the hackaru desktop application does not set a window icon. At least on Ubuntu this is annoying, because it is hard to figure out which window is the hackaru desktop application.

With this PR the hackaru desktop application sets the tray icon as the window icon, which looks like this

This could be improved by using an image with a higher resolution. However, this solution is currently sufficient for me.

I have tested this on Ubuntu and from what I read online (https://stackoverflow.com/questions/31529772/how-to-set-app-icon-for-electron-atom-shell-app) it should work for Linux in general and for Windows as well. However, OS X seems to expect an other icon format. As I do not have access to a system runny OS X I cannot test this.

I agree that this window icon can be further improved using a high resolution image.
so I will consider that create a high resolution image when I have time.

Sounds great. :heart:


Update the UI for gender selection from TextView to Spinner

Give the options in a spinner for the user to pick one among the given options of gender [Create Profile section]

Male
Female
Other
Prefer not to say

I have sent a PR for this issue please check it.


Sensitive outputs logged in runner

I have defined the following terraform output:
output "client_secret" {
  value = keycloak_openid_client.client.client_secret
  sensitive = true
}

and Terraform resource:
apiVersion: infra.contrib.fluxcd.io/v1alpha1
kind: Terraform
metadata:
  name: my-terraform
spec:
  ...
  writeOutputsToSecret:
    name: my-secret
    outputs:
      - client_secret

The tf-runner logs:
2022-05-24T16:25:39.798905345Z Apply complete! Resources: 2 added, 0 changed, 0 destroyed.
2022-05-24T16:25:39.798907631Z
2022-05-24T16:25:39.798909751Z Outputs:
2022-05-24T16:25:39.798911665Z
2022-05-24T16:25:39.798913849Z client_secret = <sensitive>
2022-05-24T16:25:39.806019983Z {"level":"info","ts":"2022-05-24T16:25:39.805Z","logger":"runner.terraform","msg":"error from SIGINT: no such process"}
2022-05-24T16:25:39.812253752Z {"level":"info","ts":"2022-05-24T16:25:39.812Z","logger":"runner.terraform","msg":"creating outputs"}
2022-05-24T16:25:39.812283207Z {"level":"info","ts":"2022-05-24T16:25:39.812Z","logger":"runner.terraform","msg":"[INFO] running Terraform command: /usr/local/bin/terraform output -no-color -json"}
2022-05-24T16:25:39.925853638Z {
2022-05-24T16:25:39.925873087Z   "client_secret": {
2022-05-24T16:25:39.925874988Z     "sensitive": true,
2022-05-24T16:25:39.925876380Z     "type": "string",
2022-05-24T16:25:39.925877826Z     "value": "hunter2"
2022-05-24T16:25:39.925879211Z   }
2022-05-24T16:25:39.925880510Z }
2022-05-24T16:25:39.927366018Z {"level":"info","ts":"2022-05-24T16:25:39.927Z","logger":"runner.terraform","msg":"error from SIGINT: no such process"}
2022-05-24T16:25:39.927705665Z {"level":"info","ts":"2022-05-24T16:25:39.927Z","logger":"runner.terraform","msg":"write outputs to secret"}

Thank you @caspervk
Very good catch.

WOW! Thanks for the very quick fix! Sorry for not getting back to you and testing the PR. Thanks a lot!

You're welcome @caspervk. Please let me know if the problem still persists.

We still observing this issue in v0.14.3. Possibly the same as in https://github.com/weaveworks/tf-controller/issues/637 but with outputs in this case.
It goes there:
https://github.com/weaveworks/tf-controller/blob/0a8ca9f03e3fca095c5a16c0c38265c64e77112a/runner/server_outputs.go#L25

Thank you. I'll prioritize the fix!


sort()

is missing

not in here: https://developer.mozilla.org/en-US/docs/Web/API/URLSearchParams
and not even a link provided to the API method ... I don't think that's the best way to file a bug Steven.
I won't move a finger 'till I have something more to think about.
Regards

MDN is way outdated and should not be your point of reference. Please use the spec: https://url.spec.whatwg.org

please file reasonable bugs or you'll be ignored.

Idiot.

mdn is a wiki; if you believe it to be outdated, please update it yourself.


add: 15 event and 16 assert

The tutorials are too short, you need to add more contents.
For event tutorial, check https://book.starknet.io/chapter_2/event.html
For assert tutorial, check: https://book.starknet.io/chapter_2/asserts.html and https://cairo-book.github.io/ch09-00-error-handling.html

I already finished the event chapter, you can work with the assert one asap.

Sorry that your submissions are too simple to be included in the WTF Cairo tutorials. We have already finished these 2 chapters.


[bug 1191968] Restyle product picker page

This restyles the product picker page so it's more like SUMO's. It's
not exactly like SUMO's, but it's enough like SUMO's that we can use the
product icons they use. That'll make some things easier.
Further, this adds product descriptions, but those aren't shown at the
moment because we need to l10n-ize them and we're lacking that
infrastructure.
r?

@rehandalal Can you look through this for me?

r+ with that one change

^^^ Fixes width. Will wait for Travis and then land.

Thank you!


Add integration test for workspace overview switching

Signed-off-by: Sebastian Malton<EMAIL_ADDRESS>

I am going to close this since we are going to remove workspaces and these two can be verified if necessary before 4.2.1


Fix executive test

the Applyable implementation of TestXT make no use of ValidateUnsigned, but frame-executive is testing for ValidateUnsigned with test validate_unsigned.
So the test is wrong as it is successful for any implementation of ValidateUnsigned.
So TestXT is now checking ValidateUnsigned implementation, also test is refactored to make use of construct_runtime, this is not needed, but is more future proof.

bot merge

@thiolliere you need to merge master.

bot merge

bot merge


Web - Chat - The size of avatar that is displayed when chat starts, or top of the chat is 108x108

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Version Number: 9.0.46-1
Reproducible in staging?: Y
Reproducible in production?: N/A - new feature, doesn't exist in prod
Issue was found when executing this PR: https://github.com/Expensify/App/pull/49555
Email or phone of affected tester (no customers<EMAIL_ADDRESS>Issue reported by: Applause Internal Team
Action Performed:

Go to personal chat and check if icon is of 100x100 size
Go to group chat and check if icon is of 100x100 size
Go to workspace chat and check if icon is of 100x100 size

Expected Result:
The icon size for every chat is 100x100
Actual Result:
The icon size for every chat is 108x108
Workaround:
Unknown
Platforms:

[ ] Android: Native
[ ] Android: mWeb Chrome
[ ] iOS: Native
[ ] iOS: mWeb Safari
[x] MacOS: Chrome / Safari
[ ] MacOS: Desktop

Screenshots/Videos

View all open jobs on GitHub

I think this is not a bug because if we inspect go through the svg avatar we can see the size is 100x100, and why when we hover the size of the avatar is showing 108x108, because there's a wrapper div on top of the svg with border-width 4px

Thanks, @NJ-2020! Agreed.
The SVG itself is 100 x 100:

But it is nested within a div with a border width of 4px:


listen to notification

Hello i am using postgress on cloud and i created triggerfunction and trigger in pgAdmin.
Here is my trigger function
CREATE FUNCTION public.row_notifiy_n()
RETURNS trigger
LANGUAGE 'plpgsql'
COST 100
VOLATILE NOT LEAKPROOF
AS $BODY$
BEGIN
PERFORM pg_notify(
'rowChanged',
json_build_object(
'operation', TG_OP,
'record', row_to_json(new)
)::text
);
Return new;
END;
$BODY$;
ALTER FUNCTION public.row_notifiy_n()
OWNER TO "abc";
and trigger is
CREATE TRIGGER notify
AFTER INSERT OR DELETE OR UPDATE
ON public."Logs"
FOR EACH ROW
EXECUTE PROCEDURE public.row_notifiy_n();
ALTER TABLE public."Logs"
ENABLE ALWAYS TRIGGER notify;
How i can listen this in flutter

Apparently, in old versions, you could listen to notifications with connection.notifications.listen(), but with the replacement of Connection with PostgreSQLConnection, this feature no longer exists.
I also filed this issue and am waiting for help

Yes, this is a show-stopper for us to upgrade to the current versions.

Not sure: are you looking for notification listeners?
https://pub.dev/documentation/postgres/latest/postgres/Connection/channels.html
https://pub.dev/documentation/postgres/latest/postgres/Channels-class.html

I read the source code almost 1 hour ago and realized how to do this, but it is better to put an example in Example so that there is no need to read the API reference or source code.


FE: Added Homepage with connection to store BE: Added store for produ…

…cts singleWine and allWine
Assignee Tasks

[ ] check /store/product.js
[ ] check Homepage "/" console for state logger

Added fetchWines, fetchSingleWines methods to grab all wines and a single wine from our database
Create a simple homepage testing the two new methods.

Checked both commits. Looks good!

Checked both commits. Looks good!


cleanlight world not working

Hey, first of all: great plugin!
/cleanlight  works fine, but the /cleanlight world does not at all.
I've got a pre-generated world (filled with WorldBorder) and tried to fix all light bugs. But there are still bugs:
https://imgur.com/a/ul3xy

FYI: When I use /cleanlight with a radius that is great enogh for the whole world, it is working, I just have to trim the new generated chunks beyond the world border

oh, I didn't know this was broken since it worked when I used it. You can check if it shows cleaning status correctly with /cleanlight status. Strange...what server is this on? Maybe I need a particular server software to reproduce it.

That was on paper spigot runnig version git-Paper-1201 (MC: 1.12.1) (Implementing API version 1.12.1-R0.1-SNAPSHOT)

Same problem.

I suppose it is worth looking into again, I will put it back on the list of priority issues

For now this issue continues under https://github.com/bergerhealer/Light-Cleaner/issues/3


PR against issue #291 || Changes in UI of Home Page

I have made many changes in the UI of the HOME page:

NAVBAR
Earlier:

NOW:

Card Button
Earlier:

NOW:

The idea behind this is that our website has green colour so why we make it unprofessional by using multiple colours.
Card on hovering
Earlier:

NOW:

The idea is that earlier on hovering the spread uses the white colour but in the light background, this effect is diminished that why I use green.
IN LIGHT COLOUR THEME
Earlier:

NOW:

Please @nilisha-jais @harshita2216 have a look at this and if you find this cool I will make changes to other pages as well. Thanks
PS: The idea and website both are the best. I really liked this project.

Looks good. Thank you !

@nilisha-jais so can I make changes to other pages as well?

Go ahead @RahulBansal0502 .Create an issue and make the changes.


string_of_dimensions is missing a space on native

Native:
https://github.com/davesnx/styled-ppx/blob/0f3e805cf05734ad9223952c42c22eae4101911c/packages/css/native/Css_Js_Core.ml#L1679-L1680
JS:
https://github.com/davesnx/styled-ppx/blob/0f3e805cf05734ad9223952c42c22eae4101911c/packages/css/js/Css_Js_Core.ml#L1728-L1731

Fixed in main. Thanks for reporting @purefunctor


Meiyerditch route from Mine to ToB goes through a locked door with no way to unlock

After being teleported to the Meiyerditch mine and following the no-teleportation Shortest Path to Theatre of Blood, you are led to a door you can't pass through.
The door is between the following two tile locations (source: the "World Location" plugin):
x: 3614, y: 3205, z: 0
x: 3614, y: 3205, z: 0
Screenshot attached:

Of interesting note: this door, and similarly-locked doors in the area, seem to have the following examine text: "Blocking the path."
Meanwhile unlocked doors nearby have the examine text: "A crude-looking door fashioned from random bits of wood."
Hopefully that examine text can be useful for not having to manually figure out which doors are passable. :/
Edit: found a door with the "Blocking the path." examine text that can be opened (between (x: 3624, y: 3222, z:0) and (x: 3624, y: 3221, z:0). Not a good heuristic. >.<

This should be fixed in version 1.16.1, which is now live in the client (assuming you have restarted your client to fetch the update). Could you please check if the issue is fixed for you?

Oh, excellent, that route works now! I can probably actually finish the quests here now, thank you!


Native types

umbrella issue for native types

[x] BigInt #437
[x] Decimal #420
[x] Bytes #439

We're currently missing support for the @<nativetype> variants: https://www.prisma.io/docs/reference/api-reference/prisma-schema-reference#default-type-mappings
I'm not sure if you want to create separate issues or just turn this one issue into an epic.

This is actually already supported. See this test: https://github.com/prisma/prisma-client-go/blob/master/test/types/native/schema.prisma
I probably should have clarified before closing, but yeah, they're supported.


DeprecationWarning: pkg_resources is deprecated as an API

Which version are you running? The lastest version is on Github. Pip is for major releases.
Python 3.12.8
pandas_ta , dev(0.4.19b0), stable(0.3.14b)
Do you have TA Lib also installed in your environment?
ta-lib 0.5.1
Have you tried the development version? Did it resolve the issue?
Yes, same issue
Describe the bug
...python3.12/site-packages/pandas_ta/maps.py:4: DeprecationWarning: pkg_resources is deprecated as an API. See https://setuptools.pypa.io/en/latest/pkg_resources.html
from pkg_resources import get_distribution, DistributionNotFound
To Reproduce
import pandas_ta as ta

Hello @vinaychuri
See if #821 solves your issue.
Kind Regards,
KJ


Fix symbolic negative floordiv

(7//-2) = -4, not -3
also a[0-7]//-2 = ((((a*-1)+8)//2)+-4), if you plug in various values of a, the new formula is correct.
a=0, result is 0. a=1, result is -1.

will probably remove the support of // negative numbers, but fine for now


[Proposal] CSS Variables

I know CSS variables has already been discussed (#26596) and I saw this reply
https://github.com/twbs/bootstrap/issues/26596#issuecomment-403342215
So I come with a proposal which not requires heavy refactoring
Proposal
Instead of using directly native functions like darken or lighten we should use wrappers
Ex : try-darken
@function try-darken($color, $percent) {

  @if (not $enable-variables){
    @return darken($color, $percent);
  }

  @if str_slice("#{$color}", 0, 3) == "var" {
    $percent-str: "#{$percent * 100}";
    $percent-value: str_slice($percent-str, 0, str_length($percent-str) - 1);
    $original: str_slice($color, 7, str_length($color) - 1);
    $variable: --#{$original}-darken-#{$percent-value};
    @return var($variable);
  }

  @return darken($color, $percent);
}

This wrapper handles the variable construction and create shades of colors.
Then we need to generate all shades of variables
:root {
  @if ($enable-variables){
    @each $color, $value in $theme-colors-values {
      --#{$color}: #{$value};
      --#{$color}-yiq: #{color-yiq($value)};

      @each $shade in $theme-colors-shades {
        --#{$color}-darken-#{$shade * 100}: #{darken($value, $shade)};
        --#{$color}-darken-#{$shade * 100}-yiq: #{color-yiq(darken($value, $shade))};
        --#{$color}-lighten-#{$shade * 100}: #{lighten($value, $shade)};
        --#{$color}-lighten-#{$shade * 100}-yiq: #{color-yiq(lighten($value, $shade))};
      }
    }
  }
  @else{
    @each $color, $value in $theme-colors {
      --#{$color}: #{$value};
    }
  }
}

This creates all necessary variables.
As you see, this requires to list all necessary shades, and concret theme color values
$primary:    var(--primary) !default;
$secondary:  var(--secondary) !default;
$success:    var(--success) !default;
$info:       var(--info) !default;
$warning:    var(--warning) !default;
$danger:     var(--danger) !default;
$light:      var(--light) !default;
$dark:       var(--dark) !default;

$theme-colors-values: map-merge(
  (
    "primary":     $blue,
    "secondary":   $gray-600,
    "success":     $green,
    "info":        $cyan,
    "warning":     $yellow,
    "danger":      $red,
    "light":       $gray-100,
    "dark":        $gray-800,
  ),
  $theme-colors-values
);

$theme-colors-shades: (7.5, 10, 12.5, 25, 35, 40) !default;

With theses few tricks I managed to create all variables I needed for theme colors, and I can change it on the fly with few javascript
Demo
Here is a demo site with an implementation
https://felx-b.github.io/docs/4.3/getting-started/introduction/
You can change the primary color on fly in the nav bar
Source
Here are the sources
https://github.com/Felx-B/bootstrap-css-variable
Alternative
If the proposal is rejected, it would worth nothing to wrap native function calls (like darken/lighten/mix ...) into upper functions (try-darken, try-lighten ...) so we could override these behaviors to implement CSS variable ourselves.

While this seems useful, I can see issues supporting IE which doesn't support CSS Variables at all.
Bootstrap 4.X lists support for IE10+ (not sure where BS5 will go)
https://caniuse.com/#feat=css-variables

BS5 will (last time I checked into our internal discussions) still support IE11

This feature should not be default, this should remain optional until all supported browsers handle it.
$enable-variables options is here to opt-in for this

This feature should not be default, this should remain optional until all supported browsers handle it.
$enable-variables options is here to opt-in for this

This won't work in not-IE browsers:
$primary:    var(--primary) !default;
$secondary:  var(--secondary) !default;
$success:    var(--success) !default;
$info:       var(--info) !default;
$warning:    var(--warning) !default;
$danger:     var(--danger) !default;
$light:      var(--light) !default;
$dark:       var(--dark) !default;

$theme-colors-values: map-merge(
  (
    "primary":     $blue,
    "secondary":   $gray-600,
    "success":     $green,
    "info":        $cyan,
    "warning":     $yellow,
    "danger":      $red,
    "light":       $gray-100,
    "dark":        $gray-800,
  ),
  $theme-colors-values
);

$theme-colors-shades: (7.5, 10, 12.5, 25, 35, 40) !default;

$enable-variables options is here to opt-in for this

Maybe we can find a solution for this, but I'm a bit worried we'll overcomplicate everything just to make it possible to support css variables.

Yeah, agreed on overcomplicating things here. Unless we move entirely to PostCSS with the build system to support it, we should continue to prioritize great Sass code.

@Felx-B I love Your idea. I think css variables is must-have in bootstrap. With that every user could change interface colors - It's great!

@piernik and it’s easy too, not that the current scss is difficult but this is “easier”

I'm going to lock this conversation because a lot of people will ask for this while we cannot provide support due to the IE11 support. Providing a solution with functions like try-darken could help us out, but we'll overcomplicate our codebase too much to make this possible. Maintaining this & tackling all edge cases will slow us down.
We 'll definitely have a look at support for css variables in the future (v6), but for now we 'll pass on this and focus on improving & extending other Bootstrap functionality.


html: change title of signup page

we havent set a title of the signup page. a block for title is defined in the base template main.html

edit signup.html and set appropriate title

Example

title: Sign Up — Cinlist
✨ Feel free to add your own changes !
💬 drop a comment if you like to take up this issue, so i can assign it to you.

Hi, I'm Interested to take up this issue!

Hey, it done Check #41 .


Messages stalled

I have sent yesterday a newsletter to 15K people. Some of them had messages from Yahoo that says they temporarily refuse my emails, and that I have to try again in 4hs. That is a normal Anti-SPAM technique. Is not bad at all.
The problem is that the emails became stalled, with no option to force to send them or to stop trying (I can not send a email with a Saturday offer, on Sunday).
Today, I'm sending a new email, and all my emails are stalled together with the ones from yesterday.
There is no option to force them to go, or it is?
I restarted Postal, and nothing happens. I got some emails being sent when I wrote postal auto-upgrade, then It  sent some emails, but not even a 10%.
I had some errors like this one, in worker.log
ActiveRecord::ConnectionTimeoutError: could not obtain a connection from the pool within 5.000 seconds (waited 5.000 seconds); all pooled connections were in use
It begun to working with a 2 CPU server, and now I changed it for a 4 CPU one. After that it begun to send, I don't know if when I rebooted it fixed, or what, but now is working. The server reboots when has to change from 2 CPU to 4 CPU.
The problem is that we need a window of time to send the emails, we can not send them 3 days after the promo.
Is there any way to adjust the retry time? I mean, retry only after 4 hours. In that way we can retry maximum 3 times and it is 12hs.
And, is there any way to force sending? They are there, with a pending status, and we can't do nothing. Is a bit maddening.
Thank you in advance for your great software and your support (this is not my first issue).

when you say the messages are stalled, are they being held by the sending limit? Or do you want to re send messages that have soft failed with a particular response?

No, no held messages (I have Unlimited on the settings).
They just say "Pending" in Queued messages.
As I said, I restarted the server with more CPU and it begun to work. It sent around 2000k emails, and stalled again. Not even a log.
I restarted Postal in command line, and it begun to run, but only some logs, because there is no email being sent.

Procodile Version   1.0.16
Application Root    /opt/postal/app
Supervisor PID      1544
Started             2017-06-18 10:45:10 +0000

|| web
|| Quantity            1
|| Command             bundle exec puma -C config/puma.rb
|| Respawning          5 every 3600 seconds
|| Restart mode        usr1
|| Log path            none specified
|| Address/Port        none
|| => web.1            Running      10:45        pid:1550     respawns:0      port:-        tag:-

|| worker
|| Quantity            2
|| Command             bundle exec ruby script/worker.rb
|| Respawning          5 every 3600 seconds
|| Restart mode        start-term
|| Log path            none specified
|| Address/Port        none
|| => worker.1         Running      10:45        pid:1619     respawns:0      port:-        tag:-
|| => worker.2         Running      10:45        pid:1621     respawns:0      port:-        tag:-

|| cron
|| Quantity            1
|| Command             bundle exec rake postal:cron
|| Respawning          5 every 3600 seconds
|| Restart mode        term-start
|| Log path            none specified
|| Address/Port        none
|| => cron.1           Running      10:45        pid:1628     respawns:0      port:-        tag:-

|| smtp
|| Quantity            1
|| Command             bundle exec rake postal:smtp_server
|| Respawning          5 every 3600 seconds
|| Restart mode        usr1
|| Log path            none specified
|| Address/Port        none
|| => smtp.1           Running      10:45        pid:1630     respawns:0      port:-        tag:-

|| requeuer
|| Quantity            1
|| Command             bundle exec rake postal:requeuer
|| Respawning          5 every 3600 seconds
|| Restart mode        term-start
|| Log path            none specified
|| Address/Port        none
|| => requeuer.1       Running      10:45        pid:1633     respawns:0      port:-        tag:-

short of any words of wisdom from atech, have you experimented with extra workers now you have some extra CPU power? we have ten on two cores

10 workers and 40 threads?
I tried before with more workers, I thought it was the problem so now it has 2 workers and 8 threads. I will try with 20 workers and 80 threads (I have 4 cpu)

With 20 workers and 80 threads is the same, nothing.
The emails doesn't even has "last_delivery_attempt", it is like Postal is not trying to send them

After 1hr server stop because "Mysql2 too many connections"
I'm fine tuning mysql, and now it works.
But there is no email being sent

40 threads?
Check main_db.pool_size and make sure it is higher than the maximum number of threads on your worker.

I have this now
workers:
  quantity: 4
  threads: 16

main_db:
  # Specify the connection details for your MySQL database
  host: <IP_ADDRESS>
  username: postal
  password: XXX
  database: postal
  pool_size: 32

But Postal is not sending any email, I have 13000 queued but it's not even trying to send them

Any logs anywhere?

This is the only log with problems that I see, the thing is that all seems to be good, but the pending emails are not being sent.
They say

Message is currently being processed.

 I'd check the locked columns on the queued_messages table in the main DB.

I will try again with a new newsletter. I have deleted the messages and the queued_messages table.
I will check with a new email.

@adamcooke Yes, I think that was the problem. It first stop working, for whatever reason (probably too many threads for only 2 cpu), and then it marked all as locked, so it didn't send anything.
Today It was good.
One question, how can I improve sending speed?
I'm sending 150 mails per minute, but the 4 cores are almost at 10%, so I think I need to increase workers and threads, but something else? I mean, like pool_size
Hoe many workers is safe per cpu? And how many threads? 4 per worker?

To improve speed, you need to scale horizontally by adding more worker processes and/or increasing the number of threads per worker. You can keep increasing threads per worker although I wouldn't go much about 16, myself. At this point, you'll need to start spawning more worker processes. Eventually you'll hit limits like disk IO and database throughput which will be harder to scale.

Thank you

Could some one tell me what is the relation between worker and thread or just give me the link if its documented.


Query hangs on collecting stream from recursive CTE

Describe the bug
Query with recursive CTE hangs while trying to collect stream originated from WorkTableExec -- this might occur when recursive part contains HashJoin/CrossJoin/NestedLoopJoin, and WorkTableExec should be collected in memory as a join build side.
To Reproduce
Reproducer based on this test query:
WITH RECURSIVE "recursive_cte" AS (
    SELECT 1 as "val"
  UNION ALL (
    WITH "sub_cte" AS (
      SELECT 2 as "val"
    )
    SELECT
      2 as "val"
    FROM "recursive_cte"
      CROSS JOIN "sub_cte"
    WHERE "recursive_cte"."val" < 2
  )
)
SELECT * FROM "recursive_cte";

Produces following plan, and is unable to complete, constantly executing CrossJoinExec
physical_plan
RecursiveQueryExec: name=recursive_cte, is_distinct=false
--ProjectionExec: expr=[1 as val]
----PlaceholderRowExec
--ProjectionExec: expr=[2 as val]
----CrossJoinExec
------CoalescePartitionsExec
--------CoalesceBatchesExec: target_batch_size=8182
----------FilterExec: val@0 < 2
------------RepartitionExec: partitioning=RoundRobinBatch(4), input_partitions=1
--------------WorkTableExec: name=recursive_cte
------ProjectionExec: expr=[2 as val]
--------PlaceholderRowExec

In the same time, identical query with swapped inputs works as expected
WITH RECURSIVE "recursive_cte" AS (
    SELECT 1 as "val"
  UNION ALL (
    WITH "sub_cte" AS (
      SELECT 2 as "val"
    )
    SELECT
      2 as "val"
    FROM "sub_cte"
      CROSS JOIN "recursive_cte"
    WHERE "recursive_cte"."val" < 2
  )
)
SELECT * FROM "recursive_cte";

Expected behavior
Ideally, recursive CTE should be able to handle collect() stream, created by WorkTableExec.
Another option might be tracking calls to CTE temp table, and throw runtime error in case their number increases some preconfigured threshold value.
Third option is to forbid recursive CTE temp tables to be used as build-side join inputs (not preferrable as it looks like too much unnecessary optimizer(s) modifications, and not reliable -- same issue might occur in case of aggregations/sorts/maybe some other "blocking" operators).
Additional context
No response

FYI @jonahgao  and @matthewgapp

take

I plan to investigate and try to fix this.

It seems that this is caused by OnceAsync, which makes the left data build only once during the entire query. It should be built every iteration.
https://github.com/apache/arrow-datafusion/blob/b0b329ba39403b9e87156d6f9b8c5464dc6d2480/datafusion/physical-plan/src/joins/cross_join.rs#L61-L62


make encoding of template and output configurable

UTF8 should be the default if no configuration is present.

closed with #8


webview: page is not adaptive

Hello!
Im working with webview of page. Page doesnt show correct (css problems) on webview, but in browser and with plugin 'https://github.com/bradmartin/nativescript-advanced-webview' is working fine.
thx for advance

@Nurgunkalol what is the actual issue you are facing?
It seems that using the plugin nativescript-advanced-webview is resolving the issue so we can thread this one as a feature request for the core WebView. Closing as the provided information is not enough.


Cooldowns overlapping with modui

https://imgur.com/a/GOUZL this happens with Omnicc aswell.

Hi nono234,
while I dislike that addons such as Omnicc and modui just hook(CooldownFrame_SetTimer) into everything they can find and are causing this there is an easy fix for that.
Use /gm opt to open the options menu
Switch to the general menu
Deactivate "Show Cooldowns"
From now on gearmenu no longer shows cooldowns but other addons should still be able to show the cooldown.
Regards,
RagedUnicorn


Bridge Request for bdsmlr.com

Bridge request
General information
Hello. I want use bdsmlr.com for use in RSS reader. Bdsmlr.com does not provide a RSS feed. It is a website similar to tumblr.  People move to bdsmlr.com.
For sure this bridge would make lot people happy.

Host URI for the bridge (i.e. https://github.com):
bdsmlr.com

Which information would you like to see?
New articles. As in tumblr.

How should the information be displayed/formatted?
RSS feed.

Which of the following parameters do you expect?

[X] Title
[X] URI (link to the original article)
[ ] Author
[ ] Timestamp
[X] Content (the content of the article)
[X] Enclosures (pictures, videos, etc...)
[ ] Categories (categories, tags, etc...)

Options

[ ] Limit number of returned items

Default limit: 5

[ ] Load full articles

Cache articles (articles are stored in a local cache on first request): yes
Cache timeout (max = 24 hours): 24 hours

[ ] Balance requests (RSS-Bridge uses cached versions to reduce bandwith usage)

Timeout (default = 5 minutes, max = 24 hours): 5 minutes

Thanks for opening this request.
https://bdsmlr.com/ requires authentication to access contents (due to the nature of its contents I presume).
Authentication, however, is not supported by RSS-Bridge for security reasons, so a bridge for this site won't be accepted into the Repository.

In fact there's no need of authentication when you got the  url of a bdsmlr blog ;) (example.bdsmlr.com).
You can find lot of blog url in a search engine, most are NSFW.

Interesting. Can you provide an example for one that doesn't burn out my eyes?
I suppose the contents are build similarly, such that pasting a URL could bring out the feed. If that works there is no issue adding the bridge.

I suppose these ones can be considered soft :
https://annabella121212.bdsmlr.com/
https://thesecretsubject.bdsmlr.com/

Thanks, this seems feasible now.

I know this bridge request is pretty old and the nature of the content may not be everyone's taste. But I also would appreciate this bridge. And it could be really useful for this community, even more because a lot of related blogs moved from Tumblr to Bdsmlr.

Most bdsmlr blogs have an RSS feed available under blog.bdsmlr.com/rss (e.g. https://safeblogfordemopurposes.bdsmlr.com/rss). This can be disabled by the blog creator, but it is enabled by default. The RSS entries, however, tend to be incomplete, if they are mixed media (e.g. text and pictures).

Thank you for pointing this out. Seems like this is a newer feature, which was not present at the time this thread was initially started.
And you are right, these RSS feeds can be seen now in the browser. But TinyTinyRSS (for example) refuses these feeds, as there is an error in the TLS certificate implementation (just when adding the "/rss" to the URL).
So hopefully bdsmlr will fix that in the future. But this has nothing to do with RSS-Bridge, so I guess this thread can be closed.

I've alerted the bdsmlr team, and they confirmed this issue, so this will hopefully just be a matter of time.
For the TinyTinyRSS I want to highlight this thread for a potential fix: https://community.tt-rss.org/t/60-ssl-certificate-problem-unable-to-get-local-issuer-certificate/4838
And while we are at it, if you are using the FeedIron plugin, this would be a simple config for bdsmlr (if you are using another full feed service, this could easily be adapted):
"bdsmlr.com": {
        "type": "xpath",
        "xpath": [
            "div[contains(@class,'postholder')]"
        ],
        "modify": [
            {
                "type": "replace",
                "search": "display:none;",
                "replace": ""
            }
        ],
        "cleanup": [
            "div[contains(@class,'thumbed')]",
            "div[contains(@class,'post_info')]",
            "div[contains(@class,'viewAll')]"
        ]
    }

First of all I want to thank you very much for all your effort and reaching out to the bdsmlr team and the TinyTinyRSS developer!
I am using a classic install of TinyTinyRSS (no docker) on Debian, latest version. And I understand that I have to download the certificate chain and add it to the Debian system certificate store.
My only question is: How and where to download the certificate chain?
Thank you for your help!

Just tried this on a Debian GNU/Linux 9.13 (stretch) system.

First check, that curl is indeed at fault: curl https://safeblogfordemopurposes.bdsmlr.com
This should report:  curl: (60) SSL certificate problem: unable to get local issuer certificate

There are multiple ways to get the certificate, personally I just use Firefox to save the whole chain. (see Step 4: https://daniel.haxx.se/blog/2018/11/07/get-the-ca-cert-for-curl/).  If done right, there should be 3 certificates in the file. (OpenSSL only saves the last one in the chain as far as I know, not sure if it will work.)

And then it is similar to the docker workflow. Put the file on the server and copy it to the right location: mv bdsmlr-com-chain.pem /usr/local/share/ca-certificates/bdsmlr.crt. The .crt file suffix is important, else it won't be found.

Run `update-ca-certificates``. Sample output:

Updating certificates in /etc/ssl/certs...
1 added, 0 removed; done.
Running hooks in /etc/ca-certificates/update.d...
Adding debian:bdsmlr.pem
done.
done.

Try the curl again, it should now show a bunch of HTML: curl https://safeblogfordemopurposes.bdsmlr.com

Since TTRSS uses curl internally, you should now be able to add these sources. Maybe you need to restart your web server, though.

Works like a charm! Thank you again for these informations to get this finally work!


Semantic HTML

Description:
Semantic HTML gives context to screen readers, which read the contents of a page out loud. This increases the accessibility of a web page.
Semantic HTML can provide the user a good way to navigate and interact with your site.
This story is done when:

[x] Identify + change HTML elements that are good candidates for semantic html
[x] check for proper organization
[x] add meaningful comments to code

Semantic HTML changes:

title
header
footer
aside
section
nav

PR: #7

adds semantic html
adds meaningful comments to code


Battery level indicator for Logitech mice?

All,
I have a G900 and the only thing that Piper/libratbag seems to have missing is the battery level indicator. Is there a way to expose that information somehow? I'd really like to use (or write myself) a notifier that would let me know what the battery level is.
Thanks.

I also need this feature a lot. There is currently no way to check the battery status from Linux.


Issue with inlay_hints

It seems like inlay hints break with this plugin:

Enable inlay hints
Delete a line of code
Observe following error:
Error in decoration provider vim_lsp_inlayhint.win:
Error executing lua: /usr/local/share/nvim/runtime/lua/vim/lsp/inlay_hint.lua:366: Invalid 'col': out of range
stack traceback:
[C]: in function 'nvim_buf_set_extmark'
/usr/local/share/nvim/runtime/lua/vim/lsp/inlay_hint.lua:366: in function </usr/local/share/nvim/runtime/lua/vim/lsp/inlay_hint.lua:320>

It seems like they compete in some sense, and disabling the Spring LSP solves the problem.
Is it possible to disable inlay hints for the spring lsp somehow, so it can run alongside jdtls?

Fixed with this solution:
https://github.com/JavaHello/spring-boot.nvim/pull/9#issuecomment-2281694428


Findings from Sara installing prototype kit

Sara tested installing and having an initial go at using the prototype kit.
Here are the main issues experienced:

20 minutes seems like a long time to download - surprised it's this long
Not clear initially where to find terminal on a mac - need to look in the utilities file but this was not clear from the screen shot initially
Unsure what is meant by 'you will use a new copy of the kit for each new prototype'
Why is it called find-a-pharmacy? Is this just an example? A different example is used later on - should examples be aligned?
Step 5 - looks different to the screenshots for this - see Sara's screenshots
Was not expecting to download developer tools python as it does not say anything about this
What to do if there are errors? Mentioned in step 6
Should vulnerabilities in the code be addressed? What is the impact of this?
npm run watch - could be clearer what is going to happen at this stage - opens in the browser
'You must install the prototype kit' - confusing as already been through steps to install it
Config js to change service name - does not work as expected. Service name is NHS UK prototype in Config js and this does not change the name on the prototype.
Can change prototype H1 by editing the code but not sure how to change title across all.

Sara's reflections:

I had some familiarity with Terminal and VS Code previously so recognised some of the screenshots/steps. I was OK with typing things in but don't always understand what I'm doing and why.
I can imagine some content designers struggling with this, without handholding - or perhaps video to refer to?
If I wasn't familiar with Terminal, I think I'd have been scared by all the code that shows when you type npm install - a long wall of code I don't understand.
I got a bit confused by the changes in language: "code editor", "HTML text editor", "VS Code". I think if we're encouraging people to use VS Code, perhaps just refer to that?

With the 3rd to last bullet point, we could be clearer that "You must install the prototype kit" is on the Before you start page in the Build a basic prototype tutorial. If you've just come from the Download and install guide, it's asking you to do something you've just done.
Once reviewed these issues may turn into a number of tickets to action.

on reviewing the govuk versions, I think that the NHS version is very very behind. The GOV.UK version that is in parallel with ours had a couple of pages that explained instructions (and has been retained in the latest version)
How to install the kit
How to run the kit
Generally the approach in the govuk versions are entirely different fewer setup setps but more about explaining terminal.
I think that the entire install sections need a review to justify why they are so far away from the govuk version that is supposedly the same version


refactor(LineDiagram): extract useRealtime

Also changes liveData prop slightly to represent absent data as undefined rather than null.
no ticket, just a thing I tried to clean up while tinkering with the line diagram. This PR also adds conditional logic, so we can easily turn the line diagram's live data "off" by specifying false as the third argument in useRealtime(). This is hardcoded to true right now and thus doesn't change anything about the current line diagram state.

Whoops, realized I ended up covering this in #1384


A 1.2 to 2.0 converter

When processing 1.2 document, pyswagger would convert them into pyswagger.spec.v2_0.objects. If we could provide a way for pyswagger.spec.base.BaseObj to dump into JSON, pyswagger would become a useful converter.
Since there are many community tools only provides 1.2 spec, it's really useful to have a 1.2 to 2.0 converter in python.
BTW, it's not straight for converting from 2.0 to 1.2, because we need to split one document into multiple. Thus what I plan here is a forward converter, I believe versions after 1.2 would all be single document.

feature release @ v0.8.10. issue would be closed after updating document.


Please include C# version numbers for all the operators in the operators section

Issue moved from MicrosoftDocs/feedback#3513

Please respond to @FaridAhamat.

From @FaridAhamat on Wednesday, February 24, 2021 1:52:18 PM
Is your feature request related to a problem? Please describe.
Sometimes old applications break because I used newer C# syntax out of habit. Would be useful to know which version an operator was introduced. As C# language evolves faster, I think it's useful to know in a glance which version was an operator introduced, rather than read through the lengthy C# changelog.
Describe the solution you'd like
Put the C# version it was introduced right after the header text.
Example, in https://docs.microsoft.com/en-us/dotnet/csharp/language-reference/operators/type-testing-and-cast , the explanation is that the "is" operator is introduced from C# 7.0, but it's better to put it on the header next to the text "is operator" to allow for quick glance.
Describe alternatives you've considered
No other idea.
Additional context
Add any other context or screenshots about the feature request here.
edit by @billwagner to add Document Details block.

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

ID: 7eeac581-6235-2192-d909-e7fe3d21197f
Version Independent ID: 9d8e2a57-1c18-8cba-9bd4-eef546fe2dcb
Content: C# operators and expressions - C# reference
Content Source: docs/csharp/language-reference/operators/index.md
Product: dotnet-csharp
Technology: csharp-language-reference
GitHub Login: @BillWagner
Microsoft Alias: wiwagn

This hasn't had any discussion since 2021.
The compiler error messages have been enhanced to provide more guidance on versions. If you are targeting an older version of C#, the error message indicates what version is required for any feature you try to access. That seems a better solution that adding cross reference or version information to all the docs pages.
Closing as not planned.


Remove top and left gutter on homepage call-to-action on widest screens

Done when:

bottom of call-to-action is flush with bottom of main video
there is no gutter to the top or left of the call to action

@afred: Just to be clear, this is a change from the original spec, right? Just concerned that if we start tweaks like this now, I'm not sure where it will end.

@mccalluc yes, it is a change from the orig spec. See comments on #263, which is related.

Fixed by #302.


[ZEPPELIN-4968] Provide commented out config for helium registry

What is this PR for?
It adds commented-out configuration parameter for Helium registry, so people can simply uncomment it, instead of going to docs, and copy/paste from it.
What type of PR is it?
Improvement
What is the Jira issue?

https://issues.apache.org/jira/browse/ZEPPELIN-4968

How should this be tested?

tested manually

LGTM

Question:
Why is that commented by default?
My problem is that if I click on Helium, there is no list as it was in 0.8.2. If I put it back into the zeppelin-site.xml it appears, so I am wondering what's the purpose of this?

@jolo-dev many people use Zeppelin without direct connection to internet, and having remote helium registry enabled by default is causing delays during startup, or something like.  That's because it was disabled by default (in don't remember exact JIRA for that).


SEP-19: Allocator changes for standby-aware container allocation

This PR makes the following changes:

Adds a map called standbyContainerConstraints which stores standby constraints for each container in the job model. The logic for populating the map using the job model is added as Util class.

Adds a check before runStreamProcessor, where the standbyContainerConstraints is checked before launching, if the check succeeds container is launched. If standbyTasks are not enabled in config, existing behaviour is retained.

Adds logic to handle standbyContainerConstraints check failures, which entails releasing the resource, and making an any-host request for the container.

Adds logic in SamzaResourceRequest to order container request such that active-containers take precedence over standby, otherwise based on request-timestamp.

Adds metrics to capture FailedStandbyAllocations (due to standby constraints) and successful standby allocations.

Tested on dev setup and a test cluster.

Closing it, in lieu of https://github.com/apache/samza/pull/903.