using "backward" in Windows OS

Hello. I have a small issue with connecting the header library "backward".
I have successfully connected "backward under Linux. I also connected the elfutils library. I built it using the make utility. I'm getting a trace.
I would really like to use "backward" to output stack traces under Windows OS as well.
At the moment, I have added a file.hpp as src. Before the main function, I wrote the following code:
#include "backward.hpp"
#define BACKWARD_HAS_DW 1
namespace backward {
backward::SignalHandling sh;
} // namespace backward

I am building and compiling in the "Debug" mode. But when the program starts, the trace is not displayed.
Can you help me with connecting this header library to the Visual Studio environment in Windows?
Which connection algorithm should I reproduce?

Note that #define BACKWARD_HAS_DW 1 must be defined before #include "backward.hpp". And you would then also need to link with the dw library. But this is specific to UNIX, and doesn't apply to MS Windows.
For MS Windows see for reference https://github.com/bombela/backward-cpp/blob/master/BackwardConfig.cmake#L204 and https://github.com/bombela/backward-cpp/blob/master/backward.hpp#L361. By default it should just work.
backward::SignalHandling sh; will only print a trace when a signal is raise (including on un-handled exception). You can read the documentation (https://github.com/bombela/backward-cpp#stacktrace) for printing traces on demand.

Did I understand correctly that the link #L204 to generate my project via Cmake?
If I don't use cmake and just attach the file .hpp, then should everything work?

Thank you very much for the quick response! You've helped me a lot.


Original files deleted after import without warning

I think we should show a dialog for this. Someone might import a file, then delete the vault, and not realize they just erased their file.
Perhaps after import the user should be asked whether they want to delete the original?

:+1:

I'll dig through the import code and find out where the files get deleted... To be continued.

agree !

The AddFileJob starts the purging progress which first overwrites the file and then deletes it. https://github.com/SecrecySupportTeam/secrecy/blob/master/app/src/main/java/com/doplgangr/secrecy/jobs/AddFileJob.java#L51
We had a discussion about the overwriting of files in https://github.com/SecrecySupportTeam/secrecy/pull/133 if you care.

I'll just try to put the dialog in there first. Then, maybe options to "just delete" and "attempt to purge". I think it's still useful to have the purging -- it just shouldn't happen by default.

I thing stating that files are purged just gives user a false security promise and thus is simply wrong. Telling them that it is tried to purge sound very weird to a user.. "Why is it just tried" etc. but thats not the point here.
In general we shouldn't add too much new features at the moment, instead we should focus on improving the code and fix open bugs. Also our support of the current android storage framework is pretty bad, because e.g. Samsung users can't use the sdcard and things might even get worse with M.
Adding simple dialogs is probably possible atm, but the real issues are elsewhere. If you are willing to fix some of these issues, just say so and I can give you more details.

Deleting files without alerting the user is a pretty serious problem (worse than weird). I ended up nearly losing some files this way, but luckily I had a backup (and I didn't need encryption in the first place).
For the most part, I want to fix the things that annoy me, but I can take a look at other parts. I don't see any issues open for what you mentioned (this?).
In any case, it's hard to trust that the storage is done safely and correctly when stuff just doesn't work how you expect. It's not added features and bloat, just attention to detail. Secrecy needs some polish.

@vixsomnis That's related to the external SDcard support we have been working on. The problem with Samsung is its inconsistent handling of external SD mountpoint, plus the lack of centralized API for storage which does not appear until KITKAT (Storage Access Framework)
What @L-Henke is stating is that we have just introduced storage access framework into our project which is quite immature, we still don't have much evidence to be confident with it. Thus we are feeling quite concerned whether things will work out in Android M.

@L-Henke, send me some details (email in commit log) or post some more issues, and I'll see if I can do anything about them.


Read more descriptions

Home Page

Blockchain consensus group
transactions  per second and per block
general overview and network stat

Entities / Block Producers
Blockchain accounts that have registered to run nodes on the network , their information is gather from on chain tables and their bp.json files
Nodes
Actual nodeos instances that are run by a particular entity for different node types along with  their specific information such as endpoints and location.
API Endpoints
Public API endpoints provided by node operators on the netowrk
Network Topology
Geo Distribution
Rewards Distribution
Block Distribution
Missed Blocks
CPU Benchmarks
BP / Validator Agreement
Accounts and Contracts
Management
Node Config
Remove Help Text from :

About
Help

@xavier506 / @AngeloCG97
Texts for the "read more" sections:
Dashboard
Find data of blockchain networks using EOSIO technology. See more networks here: https://eosio.online/.
Entities
A list of entities part of the network. They can be partner or non-partner entities.
Block Producers
A list of the block producers in the network – blockchain accounts registered to run nodes on the network. It includes information from chain tables and their bp.json files.
Nodes
A list of all the nodes run by entities comprising the network with specific information such as endpoints and location.
API Endpoints
An updated list of public API endpoints provided by node operators in the network.
Network Topology
A visual representation of the network’s current nodes.
Geo Distribution
A visualization of the geographic distribution of the nodes in this network.
Block Distribution
A visualization of the distribution of blocks produced by the nodes in the network.
Missed Blocks
A list of scheduled, produced, and missed blocks by each account in the network.
CPU Benchmarks
A visualization of CPU usage in milliseconds by block producer nodes accounts, with lowest, highest, and average usage data.
Validator Agreement
Read more about the Validator Node Agreement of this network. Contact us if you have any inquiries.
Accounts and Contracts
This tool helps you seek information about network accounts and contracts. Type the name of a contract or account in the space provided to start searching. You’ll obtain the data of the contract actions, contract tables, scope, lower and upper bounds, and limits.
Management
Log in to manage your account configurations.
Node config
Use this tool to configure a new node in the network.


Publications

I am interested in using publications with awsomecv, curious why is this PR closed without requesting a merge?

These were just changes for my specific use-case (modifying build directories, make directives, etc...) with the PR opened by mistake. You're welcome to use any of those changes currently on my fork but they are not supported for any broader usage.


How to enable future dates for start date?

After initial user testing of my app using your (great) Vue.js plugin, every one of the users mentioned that the datepicker, using a range of dates, was broken/not working properly.
They said this because they every start date in the future is disabled, and cannot be selected.
While I understand needing to ensure that the end date > start date, this restriction is confusing to many people on the UI side who would never "think" to select an end date first when entering in arrival/departure times for flights, vacation start/end dates (my use), etc.
How can this be resolved, allowing for full range selection ability?

good idea~, i have resolved this problem @v1.0.16

Works great. Thanks for the prompt response!

@seanodotcom sorry, v1.0.16 has a bug，it was fixed @v1.0.17


Enhancement | Have static content type class

It would be nice to have the string values (IE...application/json) of the content type setup in a static class. This would be handy to have for validating without having magic strings throughout the code.

Good point. I'll make the change next time I'm working on the project.

@aroemen would my project fit the bill for what you're requesting? It is based on this project and is a static mapping.


Use Promise.all for simultaneously load project data

This commit could be considered as a possible optimization for loading project data components.

The change will make it so it'll be nearly identical in performance to chaining a .then() off the Promise.all(). Did you do any benchmark?


Adds warning for unused result for registerCall methods

Description
We have have multiple ways to call  MockCallhandler.registerCall method with and without return values.
This adds warnings messages if you do not use the return value for those MockCallhandler.registerCall  methods that do return a value.
Checklist

[x] Documentation has been added or updated.
[x] All applicable classes have unit tests.
[x] This PR contains no commented out code.

Looks good. For 2.0 I think we could probably build this into the API somehow. I'm merging and releasing this as 1.1.2.


Fix random dev mode warnings

Fixing some more warnings that I found when working on #5835. There are a few more warnings coming from the new sidebar code, but I don't want to change them yet since they'll likely conflict with the feature branch.

Skipping QA review since these warnings only appear during development.


Install problem with node-osrm

I tried to install the server components with npm install and I get:
npm WARN package.json @0.0.1 No repository field.
npm WARN package.json Dependency 'browserify' exists in both dependencies and devDependencies, using<EMAIL_ADDRESS>from dependencies
npm ERR! Failed resolving git HEAD (git://github.com/Project-OSRM/node-osrm.git) fatal: bad object dc53afcbd1e0b58700c4aeab62e149b6250127e4
npm ERR! Failed resolving git HEAD (git://github.com/Project-OSRM/node-osrm.git)
npm ERR! Darwin 14.3.0
npm ERR! argv "node" "/usr/local/bin/npm" "install"
npm ERR! node v0.10.35
npm ERR! npm  v2.1.18
npm ERR! code 128

npm ERR! Command failed: fatal: bad object dc53afcbd1e0b58700c4aeab62e149b6250127e4
npm ERR!
npm ERR!
npm ERR! If you need help, you may report this error at:
npm ERR!     <http://github.com/npm/npm/issues>

npm ERR! Please include the following file with any support request:
npm ERR!     /Users/ustroetz/projects/osrm-matching-inspection/npm-debug.log

Is this version stable and should work or is it still in development?

Should be fixed now. Sorry about that.

Thanks for fixing it! Works fine now.


Addition of active tower damper functionality

Problem description:
For-aft tower motion is well-damped due to aerodynamics. However, side-side motion has far less intrinsic damping.
When wave forces and wind direction are not aligned, waves will excite the sideways motion of the tower (other sources of sideways excitation exist too of course). This side-side oscillation may occur regularly enough and be severe enough to have a significant impact on the lifetime of the wind turbine's tower and/or foundation. Several ways exist and have been proposed to actively dampen this oscillation. One of the more common approaches is to add a variable component to the generator torque (controller output), based on e.g. the tower sideways motion (controller input).
Desired solution:
Addition of an active tower damper to OpenFAST. Several more or less advanced control schemes could be imagined, but a PID using lateral acceleration of the nacelle as input would make sense as a first start.
In line with OpenFAST's Development Philosophy and Guidelines, this post is meant as an initial post to inform you that I intend to add this functionality and to engage with the NREL OpenFAST team to define the scope of the work and coordinate development efforts.

areas of the software that will be impacted:
To be honest, I am discovering the code as I go.
My current view is that the most logical place to add this is in the files ServoDyn.f90, ServoDynIO.f90 and the ServoDyn-input file (and I am currently unaware whether any changes would be required elsewhere).
If you (the NREL OpenFAST team) would reach out to schedule a meeting to talk through the details, that would be really appreciated.

any model validation materials:
None yet. I intend to extend the IEA-10.0-198-RWT model to include this additional functionality. So this will logically be created in parallel with the functionality in the source code.

Dear @GertHeirman,
Thanks for sharing your plans.
I question, though, if what you are proposing really requires a change to OpenFAST.  It sounds like you want to augment the generator torque controller, but other than a few simple built-in control options, OpenFAST (through ServoDyn) mostly supports interfaces to externally defined controllers (e.g., user-defined routines, Simulink, or Bladed-style dynamic libraries).  The ROSCO dynamic library is the most complete wind turbine controller that I'm aware of in the open domain: https://github.com/NREL/ROSCO.   What you are proposing sounds like a feature that could be added to ROSCO.  Perhaps that is a better place to start?
Best regards,

Dear @jjonkman,
After reading some documentation and looking into source code I couldn't agree more with you.
Thanks for pointing me in the right direction.
I will add the functionality to ROSCO.
Regards,


Gradual increases to web_size__compile_test flutter_gallery_dart2js_size

I've been assuming this is normal growth to the addition of code, and I've rebaselined a few times, but I want to make sure before rebaselining again that this assumption holds.
See:

I don't think this is urgent, hence the reasonable priority.
cc @yjbanov

Yeah, this is expected. Rebaselined.


configuration_files.md - Update nicknaming sample usages

Update the nicknaming sample usages to show that iv_pct now is formatted on 3 digits.

@vdubus perfectly noticed! Thank you :)

:+1:


Added tests for Stepper, Mapper, Logger, Plugins and EventManager

Short Description:
More tests for core functions
Changes:

Added tests for Stepper, Mapper, Logger, Plugins and EventManager

@OpenPoGo/maintainers

👍

👍


can't load the image(except the screenShot) in iphone6

i'm using this plugins in iphon6 , and found that the plugin can't open the image properly(the image can't show). at the begining , i thought the image is too big ,but when i using android phone it was ok.

Scaling down the image before using imgcrop solved this issue on iPhone with Cordova for me.

Use the better and maintained library https://github.com/CrackerakiUA/ngImgCropFullExtended

ngImgCropFullExtend solved problem, thanks @rickerd!

Maybe its an idea to close the repo?
And show users to use the extended library.

@rickerd is ngImgCropFullExtended a complete replacement for ngImgCrop?
I have noticed ngImgCrop hasn't been maintained for 2 years.

We're aiming for that yes


Reuse jandex index for faster startup

Vaadin offers the prebuilt jandex index for the purpose of integrating with quarkus. Can we reuse this jandex index to speed up the class discovery of Jetty, and thus speed up the starting time?

Googling for jetty+jandex was unfruitful, but perhaps we can manually mimick jetty quick start, do what it does, and fill in classes accordingly.

The problem is that the application-specific @Routes need to be discovered as well. I could build a jandex index for the app jar though, but it wouldn't work for submodules; and would require the Jandex plugin anyway. But it could be still a viable option for some cases; probably much more reliable in native mode than the Jetty QuickStart... Leaving the ticket open.

The AnnotationConfiguration, ServletContainerInitializerHolder and the whole jetty-annotations package is important her.e

Maybe there's something in jetty's documentation
https://eclipse.dev/jetty/documentation/jetty-12/programming-guide/index.html

Asked on https://stackoverflow.com/questions/77249263/enable-jandex-index-for-jetty


Fix documentation link in README

@swormke @jlongstaf
What issues does this address?
Fixes #121 #114
What's this change do?

I changed the documentation link in the README to point to http://f5-openstack-lbaasv2-driver.readthedocs.io.
I updated the release notes for the supported and unsupported features in the upcoming release (8.0.3).

Where should the reviewer start?
view changes below
Any background context?
I updated the docs URL to point to the new lbaasv2-driver doc set.
@jlongstaf -- this PR should be merged in before liberty is merged up to master / the mitaka branch is created.

Closing this PR out, as I still have a few updates to make. I'll open a new one when everything is ready for review.


split volume manager into separate package

Upcoming changes were running aground on import cycles: the manager needs to refer into the provider implementation packages as it gains responsibility for initializing them during restores from prior state; and the providers of course need to refer to the volume basic types.
The new import graph is host|api -> manager -> {providers} -> volume.

Couldn't you do the same thing we do in the router/controller/host, and move the type definitions into a subpackage? vt github.com/flynn/flynn/host/volume/types

Yeah, it's about six of one, half dozen of the other.  I think in this case it makes sense for the types to be the central package and the manager to be off to the side; the manager is more a bunch of glue than the focal point, and all the providers can technically be used without any reference to one.

LGTM


broken images for legislator photos

Missing photo for Blanche Lincoln at http://capitolwords.org/legislator?chamber=&party=D&state=AR&congress=109
More missing photos at http://capitolwords.org/legislator?chamber=&party=R&state=MN&congress=105
Missing photo for Christensen at http://capitolwords.org/legislator?chamber=House&party=&state=&congress=112

have asked Eric for his take on why there might be two Blanch Lincolns at that first link. Given the duplicates, I'm guessing that this is either an API error or that we just need to filter the records that are coming back in some way.

All congressperson pictures are identified by bioguide ID. On the first link, the second "Blanche Lincoln" has a bioguide ID of L000555, which as it turns out, is a valid bioguide ID for her, but as a redirect to an alternate spelling of her name:
http://bioguide.congress.gov/scripts/biodisplay.pl?index=L000555
On the second page, Rod Grams has been out of office since 2001, and our Legislator API doesn't go back that far (it goes back to, I believe, the 110th Congress that started in Jan of 2007):
http://bioguide.congress.gov/scripts/biodisplay.pl?index=G000367
Same for Gilbert Gutknecht, he's just too old:
http://bioguide.congress.gov/scripts/biodisplay.pl?index=G000536
On the third page, Christensen is the victim of another separate-ID-for-alternate-spelling:
http://bioguide.congress.gov/scripts/biodisplay.pl?index=C001039
How do we end up with these alternate bioguide IDs? They don't appear in our Sunlight API, so I guess Capitol Words does name->bioguide resolution using an alternate source?
For the issue with older Congresspersons, this is going to be widespread if we're going back far enough. Here are two issues missing Grams, and also Tom Daschle:
http://capitolwords.org/date/2000/12/14/S11774-2-serving-in-the-senate
http://capitolwords.org/date/1998/09/17/S10501-2-sense-of-the-senate-regarding-puerto-rico
I think our choices there are either to do a very comprehensive retroactive update of our photo database, or find a cute "this user hasn't uploaded a profile picture!" image to use where we don't have one.

Thanks for looking at this so quickly! Yeah, Aaron filled me in shortly after I emailed you and told me that the project's wide date range forced him to use a custom solution.  Sounds like either Javascript suppression or getting Tim to generate a list of 404s (though I doubt logging was turned on for that bucket) for generic image placement is the way to go.  Unfortunately the S3 media hosting eliminates the possibility of a more elegant nginx fix.
Perhaps I'm missing something and there's a way to detect a missing photo in advance during page rendering, but I don't see how.  I guess we could look the bioguide up against a list of all the photos we have -- maybe that's not too inefficient. Hacky, though.

I think your assessment is basically right, though generating a JavaScript array of known valid bioguide IDs from the legislator API is not hard to do and keep up to date if CW downloads the CSV of legislator info every night. If the bioguide ID from the CR isn't in the array, display the no-photo pic.
As to the alternate solution of filling out our photo databaser - James has some code to automatically fetch and correctly size legislator photos, I don't know how easily it could be adapted to serve older Congresses. It'd be neat if we could just take the list of distinct bioguide IDs that appear in the CapitolWords database and use it to generate a much more complete set of photos. It'd be fine to have photos of legislators that don't appear in the API.

Fixed in d0359d, current behavior is to hide images. A slight tweak could replace them with our 'no photo available' placeholder.


Complete steps 1 of the Carbon React tutorial

Complete steps 1 of the Carbon React tutorial
Step 1. Installing Carbon

not master.close it.


Add support for environment variables, closes #3

general notes
Here is the PR that should close #3. A few changes I made:

Bumped to Python 3.10, so the new match and case could be used
moved the opening and reading of the config file to the __main__ portion of the code
my editor automatically removes extraneous whitespace, so there are a few places where whitespace is removed
this is commented in the code, but casting a string to a bool via distutils.util.strtobool is deprecated and will be removed in Python 3.12, so instead of kicking the can down the road I just addressed it here by making a new function called convert_to_bool
I'm not a python programmer, or even really a programmer at all, so if there are pythonisms or other efficiencies I am not doing, let me know and I can try to fix them
since I am not running this on a Synology, I could use some guidance in the README on how environment variables might be used there
hopefully everything is relatively self-explanatory

test cases
I tried to break it in a few ways, I think it's decently resilient at this point. Here are some of the test cases I ran to ensure it was properly functioning:

no config.json, no environment variables: quits properly with an good error and an exit(1)

$ cat config.json
cat: can't open 'config.json': No such file or directory
$ python init.py
Configuration item 'DSMAddress' wasn't found in config.json, attempting to read from environnment variable 'DSMADDRESS'
Configuration item 'DSMPort' wasn't found in config.json, attempting to read from environnment variable 'DSMPORT'
Configuration item 'Username' wasn't found in config.json, attempting to read from environnment variable 'USERNAME'
Configuration item 'Password' wasn't found in config.json, attempting to read from environnment variable 'PASSWORD'
Configuration item 'Secure' wasn't found in config.json, attempting to read from environnment variable 'SECURE'
Configuration item 'Cert_Verify' wasn't found in config.json, attempting to read from environnment variable 'CERT_VERIFY'
Configuration item 'ActiveBackup' wasn't found in config.json, attempting to read from environnment variable 'ACTIVEBACKUP'
Configuration item 'HyperBackup' wasn't found in config.json, attempting to read from environnment variable 'HYPERBACKUP'
Configuration item 'HyperBackupVault' wasn't found in config.json, attempting to read from environnment variable 'HYPERBACKUPVAULT'
Configuration item 'ExporterPort' wasn't found in config.json, attempting to read from environnment variable 'EXPORTERPORT'
Configuration item 'DSM_Version' wasn't found in config.json, attempting to read from environnment variable 'DSM_VERSION'
ERROR - Missing or bad configuration for ['DSMAddress', 'DSMPort', 'Username', 'Password', 'Secure', 'Cert_Verify', 'ActiveBackup', 'HyperBackup', 'HyperBackupVault', 'ExporterPort', 'DSM_Version'], it couldn't be found in config.json or in any environment variables, exiting
$ echo $?
1

no config.json, but all env vars are specified: works as expected

$ cat config.json
cat: can't open 'config.json': No such file or directory
$ DSMADDRESS=syno.ip.add.ress DSMPORT=5001 USERNAME=user PASSWORD='password' SECURE=true CERT_VERIFY=false ACTIVEBACKUP=false HYPERBACKUP=true HYPERBACKUPVAULT=false EXPORTERPORT
=9771 DSM_VERSION=6 python init.py
Synology Backup Exporter
2022 - raphii / Raphael Pertl
You are now logged in!
INFO - Web Server running on Port 9771

all config in config.json, no env vars: works as expected

$ cat config.json
{
    "DSMAddress": "syno.ip.add.ress",
    "DSMPort": "5001",
    "Username": "user",
    "Password": "password",
    "Secure": true,
    "Cert_Verify": false,
    "ActiveBackup": false,
    "HyperBackupVault": false,
    "HyperBackup": true,
    "ExporterPort": "9771",
    "DSM_Version": 6
}
$ python init.py
Synology Backup Exporter
2022 - raphii / Raphael Pertl
You are now logged in!
INFO - Web Server running on Port 9771

some in config.json, some from env vars. This is also a good example of the type casting, since environment variables are always brought in as strings, I needed to cast them to ints or bools, this showcases that both of those functions work (I added a print() statement to show off the config dict for this example, notice that 'yes' and 'no' are also translated to their proper bool): works as expected

$ cat config.json
{
    "DSMAddress": "syno.ip.add.ress",
    "DSMPort": "5001",
    "Username": "user",
    "Cert_Verify": false,
    "ActiveBackup": false,
    "HyperBackupVault": false,
    "ExporterPort": "9771"
}
$ PASSWORD='password' SECURE=true HYPERBACKUP=yes ACTIVEBACKUP=false HYPERBACKUPVAULT=no DSM_VERSION=6 python init.py
VERBOSE DICT - {'DSMAddress': 'syno.ip.add.ress', 'DSMPort': '5001', 'Username': 'user', 'Cert_Verify': False, 'ActiveBackup': False, 'HyperBackupVault': False, 'ExporterPort': '9771', 'Password': 'password', 'Secure': True, 'HyperBackup': True, 'DSM_Version': '6'} # this is not in the normal output, just showing off the dict
Synology Backup Exporter
2022 - raphii / Raphael Pertl
You are now logged in!
INFO - Web Server running on Port 9771

some in config.json, some from env vars, except the env vars have bad values (bad values are shown as None in the dict): quits properly with an good error and an exit(1)

cat config.json
{
    "DSMAddress": "syno.ip.add.ress",
    "DSMPort": "5001",
    "Username": "user",
    "Cert_Verify": false,
    "ActiveBackup": false,
    "HyperBackupVault": false,
    "ExporterPort": "9771"
}
PASSWORD='password' SECURE=true HYPERBACKUP=2 ACTIVEBACKUP=false HYPERBACKUPVAULT=no DSM_VERSION=no python init.py
Configuration item 'Password' wasn't found in config.json, attempting to read from environnment variable 'PASSWORD'
Configuration item 'Secure' wasn't found in config.json, attempting to read from environnment variable 'SECURE'
Configuration item 'HyperBackup' wasn't found in config.json, attempting to read from environnment variable 'HYPERBACKUP'
Configuration item 'DSM_Version' wasn't found in config.json, attempting to read from environnment variable 'DSM_VERSION'
VERBOSE DICT - {'DSMAddress': 'syno.ip.add.ress', 'DSMPort': '5001', 'Username': 'user', 'Cert_Verify': False, 'ActiveBackup': False, 'HyperBackupVault': False, 'ExporterPort': '9771', 'Password': 'password', 'Secure': True, 'HyperBackup': False, 'DSM_Version': None} # this is not in the normal output, just showing off the dict
ERROR - Missing or bad configuration for ['DSM_Version'], it couldn't be found in config.json or in any environment variables, exiting
$ echo $?
1

One thing to note: If the config.json has bad values, it will continue as normal but could blow up later. I saw this as out of scope since that was true of the code before I made any changes, I didn't try to address that problem.
This was fun! Thanks for responding to my issue and letting me work on this!

This was awesome!
i've built an updated docker-image and already pushed it to docker hub


Docs: Update CONTRIBUTING.md

Why does this always happen 🥲
Right after merging, I noticed there is a typo in the first sentence of the doc... always!

@isabelle-dr No need for a typo fix here! Your original version was correct, since "look", the singular verb, is describing "processes", which is plural. I hope that makes sense. Let me know if you need any clarification.

Oh no worries @isabelle-dr ! I agree with @emmambd, I think the original sentence was correct. I'm closing this PR with this comment. Feel free to re-open it if we are missing something :)

Thank you to you both 🙏😄


[AMBARI-23057] Upgrade fails because of Stale alert definitions (ap…

What changes were proposed in this pull request?
Have added DB consistency check to remove those stale alert definitions before even Upgrade started.
(Please fill in changes proposed in this fix)
(Please fill in changes proposed in this fix)
How was this patch tested?

Have tested in live ambari server with stale alert and without stale and it works fine. pls check the log below.
2019-03-22 22:17:50,835 INFO - Checking to ensure there is no stale alert definitions
2019-03-22 22:17:50,837 WARN - You have Alerts that are not mapped with any services : yarn_nodemanager_webui_percent(YARN1)
(Please explain how this patch was tested. Ex: unit tests, manual tests)
(If this patch involves UI changes, please attach a screen-shot; otherwise, remove this)

Running autofix has removed those stale alerts and references. here is the sample log.

2019-03-27 18:50:27,842  INFO - fixAlertsForDeletedServices stale alert definitions for deleted services
2019-03-27 18:50:27,967  WARN - fixAlertsForDeletedServices - 2 alerts were deleted
Please review Ambari Contributing Guide before opening a pull request.

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/Ambari-Github-PullRequest-Builder/5036/
Test PASSed.

@swagle Could you please review this code change?

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/Ambari-Github-PullRequest-Builder/5046/
Test PASSed.

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/Ambari-Github-PullRequest-Builder/5048/
Test FAILed.
Test FAILured.

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/Ambari-Github-PullRequest-Builder/5051/
Test FAILed.
Test FAILured.

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/Ambari-Github-PullRequest-Builder/5085/
Test FAILed.
Test FAILured.

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/Ambari-Github-PullRequest-Builder/5086/
Test PASSed.

@swagle I have added a test case - can you please review and merge it?

@Akhilsnaik Isn't this same as 2885? I just merged 2885.

@swagle any idea why these changes were not showing up in trunk though changes are merged?


feat: Add ContinuousAt.eventually_mem

This is essentially the same as Filter.Tendsto.eventually_mem, but infers in more cases when applied.
Zulip discussion: https://leanprover.zulipchat.com/#narrow/stream/287929-mathlib4/topic/ContinuousAt.2Eeventually_mem'/near/422664457

bors r+
Thanks!


Poor BOLD-T1w coregistration in ds000229

Could be related to SyN-SDC (which was applied).

This is still happening. Just checked sub-07_task-flavor_run-03. I agree it could be related to SyN-SDC but the correction doesn't look very bad.


TF: remove old DBs and switch to new DBs

DEVOPS PULL REQUEST
Related Issue
#3358
Changes Proposed

Remove TF code for old DBs
Switch any resource referencing the old DBs to the new DBs

Additional Information

The two log blocks removed from azurerm_monitor_diagnostic_setting.postgres is intentional - these blocks are not supported by Postgres Flexible DB.
The terraform plan has a failing change regarding the azurerm_postgres_flexible_server_database.metabase - this is because persistent hasn't updated its outputs yet from this change. If actually run, this value will update, and the real error is that then Terraform will think it has to create that database in the new flexible DB for some reason (it doesn't, it already exists). So, a terraform import for that resource in each env will be needed to finish the deploy.

Testing

This has been tested against and deployed to pentest.

Checklist for Primary Reviewer
Infrastructure

[ ] Consult the results of the terraform-plan job inside the "Terraform Checks" workflow run for this PR. Confirm that there are no unexpected changes!

Cloud

[x] If there are changes that cannot be tested locally, this has been deployed to our Azure test, dev, or pentest environment for verification

LGTM! any reason why the TF checks are failing?

Check out the 2nd bullet point under Additional Information


added support for PostBuildScript Plugin

Thank you for a pull request! Please check this document for how the Jenkins project handles pull requests


FEATURE REQUEST: please create a docker image

I think that your tool can be used as a parsing microservice. In that point, I think you need to have a docker image with your tool. In my opinion, that can help to grow your users' community who wanted just to docker run ...  and parse.
So you need to provide a Dockerfile and upload each stable version into a Dockerhub to allow users to run your service with just one-line docker pull ... command.
And sorry to delete all issue template, I didn't find any 'feature request' template.
Thanks. And sorry.

It’s just one npm install. npm is itself a one-line installation with most package managers.

I understand, but a docker image will allow using your system in a cluster with just docker- compose or kubernetes.
Now to run your code the npm should be installed on a machine. Also, I don't know if any npm version can run your code: I think it is possible that on some npm version your code (and just any code) just can't be run.
On the other hand, if you specify the docker image you can select the proper npm version to be installed on a container. As a result, your code can be run on any machine who has docker installed - that is the key idea of containerization.
In other words, if you prepare the docker image, to run the code on any machine only docker will be required to be installed. At all scale-ready systems (AWS, Google Cloud etc.) docker is everywhere.

Well, why don’t you create a docker image and do a pull request to include
a link to it in the readme? :D
I am not a maintainer here, btw ;)
The code should be useable with the latest version of npm (and node), as it
is under active maintenance. And if it falls out of maintenance, then it’ll
become useless quickly anyhow.
On Sun, Apr 21, 2019 at 1:24 AM Daniil Okhlopkov<EMAIL_ADDRESS>wrote:

I understand, but a docker image will allow using your system in a cluster
with just docker- compose or kubernetes.
Now to run your code the npm should be installed on a machine. Also, I
don't know if any npm version can run your code: I think it is possible
that on some npm version your code (and just any code) just can't be run.
On the other hand, if you specify the docker image you can select the
proper npm version to be installed on a container. As a result, your code
can be run on any machine who has docker installed - that is the key idea
of containerization.
In other words, if you prepare the docker image, to run the code on any
machine only docker will be required to be installed. At all scale-ready
systems (AWS, Google Cloud etc.) docker is everywhere.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/postlight/mercury-parser/issues/390#issuecomment-485170015,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AIUL56XR4EWGN5ZFURTN4LTPRN7JLANCNFSM4HHDQP6A
.

I think a Docker image is more appropriate for the API version.
I use this Dockerfile for the AP versionI:
FROM node:latest

RUN git clone https://github.com/postlight/mercury-parser-api.git
WORKDIR mercury-parser-api
RUN yarn install
EXPOSE 3000
CMD [ "yarn", "serve", "--port", "3000", "--host", "<IP_ADDRESS>" ]

Maybe you can use this.

@fappelman thank you very much! I’ll definitely use your snippet later in my project. I want just a docker container that can parse anything on a web. And this may be a solution. Thanks.

@fappelman This API version seems a bit outdated. Is there a way to run a docker with this mercury-parser project and call its methods with HTTP requests (like other microservices work)?

The only thing I can advise you is to reinstall the parser after you run "yarn install".
Something like:
RUN git clone https://github.com/postlight/mercury-parser-api.git
RUN cd mercury-parser-api ; yarn install

This will make sure you use the latest version of the parser. The API is just a front-end.

The API is just a front-end.
Oww, sorry, I just got that this API imports always a fresh copy of mercury-parser itself. Thanks!

I've created a docker image for it - https://github.com/morejust/mercury-parser-docker
Also, I've uploaded it to dockerhub.

Thanks.


closer for Executor

add a closer for Executor instead of outsourcing that duty to a context.CancelFunc

done in @v0.2.0


Bug: Missing construction code for wxAuiToolbar disabled tool

Description:

Support for a disabled tool was added for wxToolbar, but I forgot to also add code generation for it to wxAuiToolbar. The use of the property does change the constructor to give us a temporary tool name to use, but doesn't generate the Enable(false) code.

wxAuiToolbar doesn't have a Enable(bool) function the way wxToolbar does. Instead it has states for pressed, disabled, hidden and checked which are set via SetState(). Using these requires #include <wx/aui/framemanager.h> in order to get the state constants. That means that instead of using a single checkbox for disabled, a state property should be set with check boxes for the multiple states that can be combined.
I'm moving this into the 1.x release -- I don't want to hold up the 1.1 release to implement this.


Upgrade to Electron 1.6.1

Looks like there is a lot happening in Electron 1.6.0:
https://github.com/electron/electron/releases
They are picking up new versions of Chromium and V8, with some additional performance fixes, so once that is ready it will be great for us to benefit too.

Looks like it was bumped to 1.6.2

Addressed in PR #270


Available Hex-Codes from Cable Provider Unitymedia in different format

There are Hex-Codes that the Germany Cable Provider Unitymedia is making available for there Horizon HD Recorder Box.
But they look very different to what normally is used I know from broadlink-rm.
Unfortunately the original remote can't be used to learn directly on the boradlink hub. So the listed hex codes could be helpful but I have no Idea how.
Anyone who can help? Thanks in advance.
http://bit.ly/2sAKayZ

Hopefully work from #144 can help with this


bug: inline comments are destroyed

My code (trimmed):
    fn get_valid_selection(
        get_actual_edit_transaction: impl Fn(
            /* current */ &Selection,
            /* next */ &Selection,
        ) -> anyhow::Result<EditTransaction>,
    ) -> anyhow::Result<Either<Selection, EditTransaction>> {
        todo!()
    }

Error (trimmed):
 Error formatting file
    - file = xxx
    Caused by:
      Error doing formatting
        Caused by:
          Encountered a bug; some comments were lost during formatting
            - comments = [Whitespace { loc: LineColumn { line: 1530, column: 26 }, mode: Comment(Comment { mode: Normal, lines: "current" }) }, Whitespace { loc: LineColumn
                         { line: 1531, column: 23 }, mode: Comment(Comment { mode: Normal, lines: "next" }) }]

By the way, I just want to express my appreciation of this project, because I'm sick of cargo fmt giving up on lines that are too long.

I think I've released this (0.5.7).  To summarize, I need to check+add comments before every symbol in the AST and missed it before & here.  There's a test now so at least hopefully there won't be regressions.


bio/b_print.c: recognize even 'j' format modifier.

This takes further steps in #3060 context, up to point of dropping PRIu64...

(I'm waiting for the builds to finish before setting the ready label)

I think the arguments are of type uint64_t while the BIO_printf
implementation uses LLONG which is a bn-type, and that
is not necessarily the same, for instance if I configure for
linux-generic32, which you might do when you know nothing
about the target.
Maybe j could mean the bn-LLONG whatever it is?
It would be good to use the ll format and make it mean uint64_t
that would be understood by gcc format checker, OTOH it might
be considered an ABI change.

while the BIO_printf implementation uses LLONG which is a bn-type

Well, LLONG is not really bn-type, as it's defined in b_print.c. BUT! You are correct in sense that there is dependency on BN macros. Which in turn means that b_print.c indeed needs a "face lift". I'd suggest it to be handled as separate request though...

Maybe j could mean the bn-LLONG whatever it is?

Whole point behind introduction of j is to have platform-neutral way to print specifically 64-bit values. Formally speaking ll could do, but gcc warning system doesn't like it. As for concern regarding bn-LLONG. Its presence is conditional and so is limb format modifier. This effectively means that ll is more or less independent of bn-LLONG. BTW, note that linux-generic32 is configured with BN_LLONG, so that all linuces look uniform and ll does prints 64-bit value practically universally. The only exclusion I can think of from the top of my head is 32-bit HP-UX for PA-RISC build with vendor compiler. But let's think if it needs to be taken into consideration...

@levitte, it occurred to me that we might need to take VMS into consideration when defining OSSLzu. As you pointed out yourself is VMS sizeof(size_t)==4 in all cases, i.e. even in LP64, which means that OSSLzu is better be defined as "u" and never as "lu". Of course VMS can be covered implicitly by __STDC_VERSION__, but the question is if it's possible to tell VMS compiler to adhere to earlier standard dialect that doesn't have to have z modifier.
I also will add couple of additional words to [already existing] CHANGES entry...

which means that OSSLzu is better be defined as "u" and never as "lu". ... implicitly by __STDC_VERSION__

To clarify. I mean that fallback definition for VMS is better be "u" in case compiler expects pre-C99 dialect.

VMS should be fine, 'cause arguments are 64-bit entities anyway, so from a va_arg perspective, a passed size_t has been expanded to a 64-bit unsigned integer.

I've a concern, though...  j usually means that a intmax_t or uintmax_t argument is expected.  Can we know for sure that LLONG is large enough?  And also, should we make sure to cast arguments that are being formatted with %jd or %ju, in case intmax_t / uintmax_t is larger than LLONG?
Also, is intmax_t / uintmax_t defined everywhere?  Actually, that's a rethorical question, as I know one platform where it isn't...

Can we know for sure that LLONG is large enough?

That's what @bernd-edlinger is talking about too. At least I think so. And suggestion to address it in separate "face lift" request. As for intmax_t availability. We don't have to rely on it, because BIO_print is our subroutine.

We don't have to rely on it, because BIO_print is our subroutine.

Sure, but then we'd better define very precisely what j expects, and if need be, have a type available to cast to.

I would not mind if j is not implemented at all, I for one have never used that before...

Pushed minor CHANGES update. Since it's not a code change lack of feedback will be treated as implicit approval.


added responsiveness in the Index page

Related Issue

added responsiveness in the Index page and also fixed the title(the image and title were not vertically in center) I made them vertically centered too.

Closes: #130
Describe the changes you've made
Give a clear description what modifications you have made

Type of change
What sort of change have you made:

[x] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[] Code style update (formatting, local variables)
[] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] This change requires a documentation update

Mention any unusual behaviour of your code (Write NA if not)
Any unusual behaviour of your code
NA
Additional Info (optional)
Any additional information you want to give
Checklist:

[x] My code follows the guidelines of this project.
[x] I have performed a self-review of my own code.
[x] I have commented my code, particularly whereever it was hard to understand.
[x] My changes generate no new warnings.
[x] Any dependent changes have been merged and published in downstream modules.

Sure, I will make the changes.

Hey, I will fix the scrollbar issue but can you please elaborate the canvas issue. Currently the canvas is fixed and not the other buttons or section therefore the other sections are moving. So you want to to move canvas also?
https://user-images.githubusercontent.com/61223163/115491957-84a64400-a27e-11eb-9e62-d90433403bb2.mp4

Ohh Okay I will fix this. Thank you

I have made the changes. please review

@ankitapuri Thank you for merging the pull request. Can you please add gssoc and level tag.

@ankitapuri Thank you for merging the pull request. Can you please add gssoc and level tag.

sorry for missing it


How does the 'generate' work with middleware ?

I found a problem what is, I want to use a middleware which called 'userAgent'
in nuxt.config.js,like this:
router: {
middleware: ['userAgent'],
},
as the document said.
but, when I run 'npm run generate' code, it warning
{
Error: ENOENT: no such file or directory, lstat '/Users/summerd/Documents/
workspace/cy/ssr-template/dist/index.html'
errno: -2,
code: 'ENOENT',
syscall: 'lstat',
path: '/Users/summerd/Documents/workspace/cy/ssr-template/dist/index.html'
}
and the dist fold I can't find index.html and 200.html. Does any body know something about this, and know how can I build successfully with router: { middleware: 'xxxx' } ?

It may because of there is unreachable ajax calling in your codes.
Can you try sue "nuxt": "next" in your dependencies of package.json, it will print the error cause, if not please offer a repo can reproduce your issue. 😸

thanks for your answer，I found the resolution of this question，
because when generate run, It has not a server，so req | res not exist, it's error to build a index.html, so...you know.
boom shakalaka.
anyway, thanks for your answer


Adds excludeScoped feature

Hi!
I added a feature to exclude scopes from license check. We do not want our test dependencies checked, because they will not be used in production or shipped. You could also use this to avoid check of libraries with scope "provided".
Best regards!!
Jens

Awesome!! Thanks so much, but of course now I have to dig out my old release instructions ;-) I'll try to cut a new release tomorrow. thanks again

Take your time! Thanks for merging & releasing! ;-)

Hi!
Still waiting for a release - will there be one any time soon?
Thank you!
Jens

Oh crap… Sorry about this. Not sure what I can commit to… any ideas? Can I let you do it somehow?
From: Jens Rummler<EMAIL_ADDRESS>Sent: Thursday, August 13, 2015 12:51 AM
To: mrice/license-check<EMAIL_ADDRESS>Cc: Michael Rice<EMAIL_ADDRESS>Subject: Re: [license-check] Adds excludeScoped feature (#11)
Hi!
Still waiting for a release - will there be one any time soon?
Thank you!
Jens
—
Reply to this email directly or view it on GitHub https://github.com/mrice/license-check/pull/11#issuecomment-130567669 .

Sure I could do it - but I don't know if you want me to:

be able to commit into your master (my GitHub account)
be able to release new Maven artefacts in org.complykit (my Sonatype accout)

I think it would be a lot easier if you would run a release build.
All that is necessary should be:
mvn release:prepare release:perform -Psonatype-oss-release
Then login to Sonatype central repo and close & release the staging repo.
Thank you!
Jens

@jrummler just for the record... as absurdley late as this is, I am working on this over the labor day weekend

@jrummler hey if you have a few free moments this weekend, can you check the readme to see if I stated your changes correctly?

Ok released 0.5.3, but still want to know what you think of the readme updates

Hey! Thanks a lot.
There is a typo in the headline (score instead of scope) and I would prefer not to be mentioned in the readme. Maybe instead it would be great to explain why it makes sense to exclude these scopes - you don't ship them with your code. Maybe this clarifies for some users why you want to set these excludes!
Thank you for this great plugin! And, just for the record, we don't have labor day here in Germany! ;-)


elementary-xfce-icon-theme: 0.12 -> 0.13

Motivation for this change
Update to version 0.13
Things done

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

[x] 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)
[x] Fits CONTRIBUTING.md.

Unexpected error: command failed with exit code 1 on x86_64-darwin (full log)
Attempted: elementary-xfce-icon-theme
Partial log (click to expand)
                                 Dload  Upload   Total   Spent    Left  Speed
copying path '/nix/store/xlj033id9dw5g8ifyskvhh52bwz2s449-librsvg-2.42.4-dev' from 'https://cache.nixos.org'...
copying path '/nix/store/nfb968h86nsrb8sc956wrbbb2zfqk587-pango-1.42.1-dev' from 'https://cache.nixos.org'...
100   137    0   137    0     0    257      0 --:--:-- --:--:-- --:--:--   256
copying path '/nix/store/iqhfysn7yldnpii012ni8fs788pz8fcb-gtk+3-3.22.30-dev' from 'https://cache.nixos.org'...
100 6120k    0 6120k    0     0  2956k      0 --:--:--  0:00:02 --:--:-- 5989k
unpacking source archive /private/tmp/nix-build-source.drv-0/v0.13.tar.gz
fixed-output derivation produced path '/nix/store/p0ccwzrxhyk90skqi3q23srivxsrgsv9-source' with sha256 hash '1rzdyv8c77wjmsymxkn7z2z9yj9xk3hxmfkc86xd4izxh032qjym' instead of the expected hash '01hlpw4vh4kgyghki01jp0snbn0g79mys28fb1m993mivnlzmn75'
cannot build derivation '/nix/store/n59fkw1dv316zh3df6wq2lsp2bjadv1a-elementary-xfce-icon-theme-0.13.drv': 1 dependencies couldn't be built
error: build of '/nix/store/n59fkw1dv316zh3df6wq2lsp2bjadv1a-elementary-xfce-icon-theme-0.13.drv' failed

Success on x86_64-linux (full log)
Attempted: elementary-xfce-icon-theme
Partial log (click to expand)
gtk-update-icon-cache -f //nix/store/7rbhwgyk30lldnljxzylaf3bwfv1bcpr-elementary-xfce-icon-theme-0.13/share/icons/elementary-xfce-darkest
gtk-update-icon-cache: Cache file created successfully.
glibPreFixupPhase
hicolorPreFixupPhase
post-installation fixup
shrinking RPATHs of ELF executables and libraries in /nix/store/7rbhwgyk30lldnljxzylaf3bwfv1bcpr-elementary-xfce-icon-theme-0.13
strip is /nix/store/h0lbngpv6ln56hjj59i6l77vxq25flbz-binutils-2.30/bin/strip
patching script interpreter paths in /nix/store/7rbhwgyk30lldnljxzylaf3bwfv1bcpr-elementary-xfce-icon-theme-0.13
checking for references to /build in /nix/store/7rbhwgyk30lldnljxzylaf3bwfv1bcpr-elementary-xfce-icon-theme-0.13...
/nix/store/7rbhwgyk30lldnljxzylaf3bwfv1bcpr-elementary-xfce-icon-theme-0.13

Success on aarch64-linux (full log)
Attempted: elementary-xfce-icon-theme
Partial log (click to expand)
gtk-update-icon-cache -f //nix/store/mf7i20sivgz7y6krxfa0rx6444j2bgpq-elementary-xfce-icon-theme-0.13/share/icons/elementary-xfce-darkest
gtk-update-icon-cache: Cache file created successfully.
glibPreFixupPhase
hicolorPreFixupPhase
post-installation fixup
shrinking RPATHs of ELF executables and libraries in /nix/store/mf7i20sivgz7y6krxfa0rx6444j2bgpq-elementary-xfce-icon-theme-0.13
strip is /nix/store/y4ymnvgxygpq05h03kyzbj572zmh6zla-binutils-2.30/bin/strip
patching script interpreter paths in /nix/store/mf7i20sivgz7y6krxfa0rx6444j2bgpq-elementary-xfce-icon-theme-0.13
checking for references to /build in /nix/store/mf7i20sivgz7y6krxfa0rx6444j2bgpq-elementary-xfce-icon-theme-0.13...
/nix/store/mf7i20sivgz7y6krxfa0rx6444j2bgpq-elementary-xfce-icon-theme-0.13

The failure on darwin looks like another instance of #39308, probably due to case conflicts in file names:
$ ls -l $(find | sort --ignore-case | uniq --ignore-case --repeated --all-repeated)
lrwxrwxrwx 1 nixbld nixbld 22 Jan  1  1970 ./elementary-xfce/apps/128/Terminal.svg -> utilities-terminal.svg
lrwxrwxrwx 1 nixbld nixbld 22 Jan  1  1970 ./elementary-xfce/apps/128/terminal.svg -> utilities-terminal.svg
lrwxrwxrwx 1 nixbld nixbld 22 Jan  1  1970 ./elementary-xfce/apps/16/Terminal.svg -> utilities-terminal.svg
lrwxrwxrwx 1 nixbld nixbld 22 Jan  1  1970 ./elementary-xfce/apps/16/terminal.svg -> utilities-terminal.svg
lrwxrwxrwx 1 nixbld nixbld 22 Jan  1  1970 ./elementary-xfce/apps/22/Terminal.svg -> utilities-terminal.svg
lrwxrwxrwx 1 nixbld nixbld 22 Jan  1  1970 ./elementary-xfce/apps/22/terminal.svg -> utilities-terminal.svg
lrwxrwxrwx 1 nixbld nixbld 22 Jan  1  1970 ./elementary-xfce/apps/24/Terminal.svg -> utilities-terminal.svg
lrwxrwxrwx 1 nixbld nixbld 22 Jan  1  1970 ./elementary-xfce/apps/24/terminal.svg -> utilities-terminal.svg
lrwxrwxrwx 1 nixbld nixbld 22 Jan  1  1970 ./elementary-xfce/apps/32/Terminal.svg -> utilities-terminal.svg
lrwxrwxrwx 1 nixbld nixbld 22 Jan  1  1970 ./elementary-xfce/apps/32/terminal.svg -> utilities-terminal.svg
lrwxrwxrwx 1 nixbld nixbld 22 Jan  1  1970 ./elementary-xfce/apps/48/Terminal.svg -> utilities-terminal.svg
lrwxrwxrwx 1 nixbld nixbld 22 Jan  1  1970 ./elementary-xfce/apps/48/terminal.svg -> utilities-terminal.svg
lrwxrwxrwx 1 nixbld nixbld 22 Jan  1  1970 ./elementary-xfce/apps/64/Terminal.svg -> utilities-terminal.svg
lrwxrwxrwx 1 nixbld nixbld 22 Jan  1  1970 ./elementary-xfce/apps/64/terminal.svg -> utilities-terminal

This is currently an issue for Darwin.
Fixed by restricting to linux platforms.

Success on x86_64-linux (full log)
Attempted: elementary-xfce-icon-theme
Partial log (click to expand)
/nix/store/7rbhwgyk30lldnljxzylaf3bwfv1bcpr-elementary-xfce-icon-theme-0.13

No attempt on x86_64-darwin (full log)
The following builds were skipped because they don't evaluate on x86_64-darwin: elementary-xfce-icon-theme
Partial log (click to expand)

a) For `nixos-rebuild` you can set
  { nixpkgs.config.allowUnsupportedSystem = true; }
in configuration.nix to override this.

b) For `nix-env`, `nix-build`, `nix-shell` or any other Nix command you can add
  { allowUnsupportedSystem = true; }
to ~/.config/nixpkgs/config.nix.

Success on aarch64-linux (full log)
Attempted: elementary-xfce-icon-theme
Partial log (click to expand)
/nix/store/mf7i20sivgz7y6krxfa0rx6444j2bgpq-elementary-xfce-icon-theme-0.13


manifest,images: add new manifest.NewBuildFromContainerSpec and use in new BootcDiskImage (HMS-3318)

This branch prepares for allowing to have build roots from container sources. This is a bit of a shortcut PR that is mainly tailored to bootc-image-builder.
Alternatively we could change InstantiateManifest() to not take repos []rpmmd.RepoConfig but instead something like:
type RepoContainerStuff struct {
    Repos []rpmmd.RepoConfig
    Containers []container.SourceSpec
}

type ImageKind interface {
	Name() string
	InstantiateManifest(m *manifest.Manifest, repoContainer *RepoContainerStuff, runner runner.Runner, rng *rand.Rand) (*artifact.Artifact, error)
}

of course with a more sensible name. Happy to do this if that is the preferred way. I will move this out of draft (or close it) once I get feedback on this (or maybe there is a third option).
Once this is available, bib will use:
img := image.NewBootcDiskImage(containerSource)
...
_, err = img.InstantiateManifestFromContainers(&mf, containerSources, runner, rng)

Note that this is not sufficient for a full container based buildroot, the following is missing:

get EFI binaries from an alternative location (e.g. via https://github.com/osbuild/osbuild/pull/1529 or bootupd)
create a second "bib-build" pipeline that uses the "bib" container as the buildroot and run "qemu-img" from there

I think we discussed this elsewhere but I would prefer if instead we created a Build pipeline interface with two implementations: an RPM-based one, which is the current Build pipeline, and a new container-based one.

Thank you! I forgot, fixed this now. Please let me know if that matches what you had in mind :)

Nice! I like this :)
Any reason it's still draft? Should we add an image build of some sort for testing?

Mostly draft as I was not 100% certain if I got the idea right :) Un-drafted now. I am happy to add more testing, I will look into how to do this best, it's slightly incomplete still (but should be enough to build raw images already with bib).

Let's merge it then and see what happens :D


[BUG] ...

*Describe the bug
Calendar of YNAB will not let me pick date if toolkit extension is added. I turned off the toolkit extension and am now able to access the calendar to pick a date for the transaction.
It would be great if I could use toolkit because  I loved all the graphs and data analysis it provide. HELP please to correct that bug.
To Reproduce
Steps to reproduce the behavior:

Go to any YNAB account associated with your budget.
Click on date field
See error

Expected behavior
Blank and scrolling calendars appear. Dates cannot be picked.
Screenshots
If applicable, add screenshots to help explain your problem.
Desktop (please complete the following information):

OS: [e.g. OSX, Windows]
Browser [e.g. chrome, firefox]
Toolkit Version [e.g. 2.22.0]

Setting Export
Please replace the text 'REPLACE_ME_WITH_SETTINGS' below with your exported Toolkit settings. You can export these by going to the Toolkit Options page, click the Import/Export Settings button at the top right and copying the text in the modal which appears.
Note! DO NOT replace the leading and trailing ``` characters as they are required formatting characters.
REPLACE_ME_WITH_SETTINGS

@Ingridrahaim I'm unable to reproduce this issue. Can you please re-submit a ticket and fill out the entire form? The Setting Export is the most important part.

Hello Josh,
I corrected the situation by removing the toolkit extention and then adding it back in. If anyone else has that issue in the future that was the fix for now.
Cheers,
Ingrid

On Jun 4, 2022, at 9:47 AM, Josh Madewell @.***> wrote:
Closed #2773 https://github.com/toolkit-for-ynab/toolkit-for-ynab/issues/2773 as completed.
—
Reply to this email directly, view it on GitHub https://github.com/toolkit-for-ynab/toolkit-for-ynab/issues/2773#event-6741741483, or unsubscribe https://github.com/notifications/unsubscribe-auth/AZPFRHJKZLGK3AKJQLLVUUTVNNM7FANCNFSM5X2I7ICA.
You are receiving this because you were mentioned.


Discrepancy in Task Execution number and Test CNX on Crynux Node UI versus the Web UI

Problem Description

Describe the problem in details

I see that there is a discrepancy with what I can see on the Crynux Node UI versus what I see by accessing the dashboard through the IP Address:port on a different computer but under the same network.
Please see the attached images.

Error Log
On the Node UI, there's persistent Network error. Will try again later msg.
In the Node UI, the Task Execution for today stays at 284 with a green running status.
The task Execution on the Dashboard by accessing the IPaddress from a different computer under the same network, shows 445 with an Idle status.
There's also a discrepancy between the number test CNX accumulated displaying on both UIs.

Optionally, upload the log file as attachment

Please see attached files. They were taken at the same time.

Device Position

At home

Operating System

Windows
Windows 11

How do you Start the Crynux Node

Download the Crynux Node zip file
Unzipped the zip file
Opened the Crynux Node on Windows

Error message on the WebUI

If there is an error message on the WebUI, provide it here

Currently, the Node UI dashboard does not update its task execution number. But the test CNX number does change. The web UI task execution number and the test CNX number change after a refresh of the page.

Node UI is lagging the Web UI. Task execution number is reflected higher on the Web UI versus the Node UI stuck at the same number.

It might be related to the Rollapp down. Can you please try it again when the Rollapp is running?

In the latest version, there is still a discrepancy between task execution number on the node versus on the web UI.

In the latest version, there is still a discrepancy between task execution number on the node versus on the web UI.

I think this is due the Node UI stopped refreshing due to some previous errors. Is there error messages on the Node UI? And can you also provide us the logs when it happens?

Yes. This is from the Web UI.

Yes. This is from the Web UI.

Can you please DM me in the Discord to send the logs regarding to this error message?


Germany

Google blocks download of the app on mobile phone with US-american settings that is located, and indicated to be located, in Germany.  When going to the play store to install the corona warn app I get the notification:
"There aren't currently any contact tracing apps in your area that are endorsed by local of national authorities. Check back as new apps may become available. See other COVID-19 resources on Google."

First of all, why would Google decide I'm not in Germany when I'm not sharing my location with them
Why would an informed person in another country be blocked from the German covid app, when he/she explicitly searches for it.
Apparently the app-download can not be very anonymous, when one is blocked in this way.

(And no, neither is this issue post)
So the request is, allow installation of the app anywhere on the globe.

Similar to https://github.com/corona-warn-app/cwa-backlog/issues/9.

Why would an informed person in another country be blocked from the German covid app, when he/she explicitly searches for it.

I think the unfortunate answer to this question is "for legal reasons";

After a detailed examination of the situation it was determined that for legal reasons any publication of the Corona-Warn-App in App Stores outside of Germany is currently only possible after a case-by-case assessment. Legal consultations have shown that in the case of publication in international App Stores, the law of the respective country must be considered and applied to the Corona-Warn-App. This applies in particular to data protection, any necessary claims for information by local authorities and other contractual and consumer protection regulations.

Source: https://www.coronawarn.app/en/faq/#international

Cynically, "legal reasons" prevents a lot of good to happen on this planet. Many others take data "protection" more serious, including the owners of the app-store. I guess they simply have more money to pay more lawyers. Well then, no corona warnings for me it seems... and an appeal to fix the legal reasons will probably not work here.

Well then, no corona warnings for me it seems...

You can change the location of your Google-play store once a year. If you change it to any of the countries listed in the FAQ entry CWA should work 🙂.
Other than that: as you seem generally not super happy with having to use Google services, there seems to be a solution based on microG which avoids Google-play-services: https://github.com/corona-warn-app/cwa-wishlist/issues/57#issuecomment-691930875
And CWA APKs can be side-loaded e.g. here: https://www.apkmirror.com/apk/robert-koch-institut/corona-warn-app/

Internal Tracking ID: EXPOSUREAPP-2041

I suggest to change the title here, since normally the issues requesting the distribution of the app through the app stores of new countries have the name of the country in the title.
Without reading the OP I would assume that one asks to make the app available in the German app stores, although it already is.


Problem with example 2

Why does example 2 have a "!" in it? Is that a mistake?

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

ID: bfc6a4cd-b57c-8df6-221d-85aead960a91
Version Independent ID: d0acb956-b718-d5d4-f422-52110f7a1242
Content: TRIM (Transact-SQL) - SQL Server
Content Source: docs/t-sql/functions/trim-transact-sql.md
Product: sql
Technology: t-sql
GitHub Login: @WilliamDAssafMSFT
Microsoft Alias: wiassaf

Hi @xerxel, thanks for your question.
Here's the query being executed in SSMS 19 preview, against SQL Server 2022 (Preview) CTP 2.0.

In a real-world environment, you might have a common list of characters that you want to remove. In this example, you can see how the '.,! ' portion is being used as a list of characters that should be removed from the string '     #     test    .' if they exist. However, only the space and period characters are being removed.
#please-close

Well what you said makes sense but the description on the page is misleading:

B. Remove specified characters from both sides of string
The following example removes a trailing period and spaces from before # and after the word test.

The description seems to imply that in order to remove a trailing period and spaces you should include a ",!". Would be easier tfor users to understand if you either included your comments from above or removed the ",!" from the command.

An upcoming refresh will include clarifying language. Thanks again for your feedback.


Time benchmarking of examples?

Hello,
How can I test the runtime of each examples? As I can see every example will invoke int run_example(int argc, char **argv, std::unique_ptr<Example> example)  from utils/Utils.cpp, can I just calculate the graph run time in this function. Or am I missing something?
One more thing, do I must need to run graph with actual weights and test image to calculate time?
As run time also can be calculated just like this:
./build/examples/graph_mobilenet

I am not sure it will be actual run time or now.
Any help will be greatly appreciated.
Thanks.

Hello @ghimiredhikura,
In your scons command you can set benchmark_tests=1, this should create binaries for each example with the prefix benchmark_. Then you can use the benchmarking framework infrastructure to benchmark each example.
e.g.
benchmark_graph_alexnet --iterations=10 --example_args="--target=cl,--type=f32" --instruments="wall_clock_timer_ms"

You can check all the common graph parameters here and you can check the instruments that you can use here.

Thanks @GeorgeARM. Great.
Just check - for vgg16 model using single thread, f32 and neon in RPI3B+(armv8a) I got graph run time just 92 ms. Do you think it is correct?

Hello @ghimiredhikura,
Can you post the command you executed?

Hi, GeorgeARM ,
I set benchmark_tests=1 in my scons command :
scons arch=arm64-v8a neon=1 opencl=1 embed_kernels=1 extra_cxx_flags="-fPIC"
benchmark_tests=1 validation_tests=0 os=android -j16
But I didn't find those binaries for each example with the prefix benchmark_,  e.g benchmark_graph_alexnet.
Did I make a mistake somewhere?
Thanks!

Hello @WenguoLi,
seems we haven't exposed it in the public repo, so I suppose the only way is to do it manually.  Just do not forget to sync at the end of the pipeline in case of CL execution. So yes you can benchmark in the run_example function.

Thanks @GeorgeARM

@WenguoLi, same here.
@GeorgeARM, waiting for the new release. Closing the issue.

If you build master from the dev repo: https://review.mlplatform.org/#/admin/projects/ml/ComputeLibrary if you build with benchmark_examples=1 you will get two sets of binaries: ./build/examples/graph_* and build/tests/benchmark_graph_*. The latter one is the instrumented one.


Anything else?


Not all ESPHome Sensors are Showing as Entities

Checklist

[ x] I checked for similar existing requests (both open and closed) before posting.
[ x] My request is generic, other users may benefit from it too.

Proposal
Allow more entities to be tied to the alarm
Additional info
I have a Zwave smoke/co2 alarm that shows up as environmental. Have them show up the same way. Both are tagged by Home Assistant for what they already are
And Yes I've unchecked show hidden sensors. They are still nonlisted.

Keep open

Remove the stale tag as this is still very much an issue.

This issue is stale because it has been open 30 days with no activity. Remove stale label or comment or this will be closed in 7 days

As a reply, remove this as it is still very much an issue

Still an issue


Collision

Hi dude,
This rules.Do you think it would be possible to load a .png with alpha as a collision object?
Or even have a input luminance or alpha from a video as a collision object as well?

https://processing.org/reference/loadImage_.html
https://processing.org/reference/libraries/video/Movie.html


Failing component creation

Current Behavior

Currently due to helm templates in the yaml (metadata.labels)passed for component creation, component creation fails from kubeopenapi-jsonschema as templates are not resolved and yaml is marked invalid.
Expected Behavior

As we do not need labels anyways, we can remove that in meshkit.
Screenshots/Logs

Environment

Host OS: Mac Linux Windows
Platform: Docker or Kubernetes
Meshery Server Version: stable-v
Meshery Client Version: stable-v
Meshery Adapter Version: stable-v

Contributor Guides and Resources

🛠 Meshery Build & Release Strategy
📚 Instructions for contributing to documentation

Meshery documentation site and source

🎨 Wireframes and designs for Meshery UI in Figma
🙋🏾🙋🏼 Questions: Layer5 Discussion Forum and Layer5 Community Slack

Is this issue only affecting the Meshery Adapter for Linkerd?

@Revolyssup will you have a look at this one again?


Creating an array of objects

Hey all,
What's the easiest way of creating an array of objects using grape, so for example,
  namespace :communities do
    desc 'Create an array of communities', is_array: true, entity: Entities::Community
    params do
      requires :communities, type: Array[JSON], desc: 'An Array of communities' do
        requires :community, type: Hash do
          requires :name, type: String, desc: 'Name of community'
          requires :type, type: String, desc: 'Type of community'
          requires :description, type: String, desc: 'Description of community'
        end
      end
    end

    post do
      params[:communities].each do |community|
        Community.create!(
          name: community['name'],
          type: community['type'],
          description: community['description'],
        )
      end
    end
  end

If I wanted to create an array of communities, then what would be the correct way of doing that within grape?

That's a bit of an open question. Is Community an ActiveRecord class? A Mongoid object?
Generally this isn't great API design because if creating the 3rd community fails you're now stuck in a partial state.
I'll close this. Please use the Grape mailing list for questions.


In a bar chart containing three series, trims the columns.

Description:
If a bar chart contains three series and each series has two values, then the outer columns are truncated.
Now I use ApexCharts v3.28.3, but it was the same on earlier versions.
Expected Behavior:
All columns should be fully displayed.
Screenshot:

Reproduction Link:
https://codepen.io/abvas/pen/LYLmPML

@abvas You are missing labels labels: ['1','2'] *make sure data type to be string. X-axis type must be 'categorical'. Also tickPlacement can result in same bug for me.

The provided codepen doesn't illustrate the issue.


Polestar

Please excuse me if this is a completely irrelevant question, but are the Volvo AAOS Solution and servers hosting their API vastly different from Polestar?
If not, is it possible to use this HA integration for Polestars? Alternatively can this integration be Branched to access the Polestar API?
Kind regards,
//S

I don't now. Do you have some documentation about the Polestar API ?

This may be possible, but without any documentation, I can't do anything. Sorry!

Ok, thank you for the quick response, and I'll be sure to be back if I find anything at all to support that!


fix: MVP changes for performance improvement. #419

Summary:
Improvement to the execution model of previous Loader, with the usage of multiple green threads for increasing efficiency.
Issue Reference(s):
Fixes #419
To boost the computation, removed the vector allocation from &[DataLoaderRequest] which performed following sorting as mentioned below.
let mut keys = keys.to_vec();
// expensive as we clone and perform comparison on that.
keys.sort_by(|a, b| a.to_request().url().cmp(b.to_request().url()));

and some single threaded iteration were modified to support operating with multiple green thread which internally spawned on the default runtime of tokio
let urls = keys[1..].iter().map(|key: &DataLoaderRequest| {
        let key_request_result = key.to_request();
        tokio::task::spawn(async move {
          ...
          ...
          //move some IO computation over here
        })
      });

this should theoretically improve the loader performance as we are offloading work to multiple green threads
Build & Testing:

[x] I ran cargo build successfully.
[ ] I have run ./lint.sh --mode=fix to fix all linting issues raised by ./lint.sh --mode=check.

Checklist:

[x] I have performed a self-review of my own code.

@artech-git thanks! Are you able to observe any improvement in performance?

@tusharmath Needed help for that only, actually! on how to benchmarks that test ? which you added yesterday.

@artech-git thanks! Are you able to observe any improvement in performance?

some benchmarks result here (sorting procedure for keys eliminated and no explicit heap allocations)
from these I bought it to about ~ 18.768% only!
without batching...
 wrk -t4 -c100 -d5s -s ./ci-benchmark/wrk.lua http://localhost:8000/graphql
Running 5s test @ http://localhost:8000/graphql
  4 threads and 100 connections
  Thread Stats   Avg      Stdev     Max   +/- Stdev
    Latency     6.89ms    3.53ms  26.62ms   66.54%
    Req/Sec     3.68k   464.98     7.20k    95.07%
  74325 requests in 5.10s, 381.49MB read
Requests/sec:  14583.61
Transfer/sec:     74.85MB

with batching
wrk -t4 -c100 -d5s -s ./ci-benchmark/wrk.lua http://localhost:8000/graphql
Running 5s test @ http://localhost:8000/graphql
  4 threads and 100 connections
  Thread Stats   Avg      Stdev     Max   +/- Stdev
    Latency     5.77ms    1.74ms  16.72ms   70.46%
    Req/Sec     4.35k   171.60     4.69k    81.86%
  88327 requests in 5.10s, 453.35MB read
Requests/sec:  17320.32
Transfer/sec:     88.90MB

Iam little surprised to the improvements here, so I request you to verify the same, meanwhile I will run more test cases and revisions.

Looking in the changes I have doubts that the changes actually improved the performance despite the numbers in benchmarks.
There could be done deeper investigations with tracing etc to figure out the cause of the performance downsides since when running tc server we are proxying the requests to another server that could affect the result numbers, right?

Looking in the changes I have doubts that the changes actually improved the performance despite the numbers in benchmarks.
There could be done deeper investigations with tracing etc to figure out the cause of the performance downsides since when running tc server we are proxying the requests to another server that could affect the result numbers, right?

yup exactly since tailcall is just a relay to a actual graphql server ! yeah I agree, but since I eliminated the sorting after allocation of slice of DataLoaderRequest's, It is natural to pick up more speed since we are only iterating over slice here and not by dereferencing a pointer to heap of DataLoaderRequest(as I understood it is not nessecary as of now), but yeah it is very important though to have a precise benchmarks for multiple sections in the modified commit as it will enables to look at a better picture and plan for changes accordingly😊.

Looking in the changes I have doubts that the changes actually improved the performance despite the numbers in benchmarks.
There could be done deeper investigations with tracing etc to figure out the cause of the performance downsides since when running tc server we are proxying the requests to another server that could affect the result numbers, right?

yup exactly since tailcall is just a relay to a actual graphql server ! yeah I agree, but since I eliminated the sorting after allocation of slice of DataLoaderRequest's, It is natural to pick up more speed since we are only iterating over slice here and not by dereferencing a pointer to heap of DataLoaderRequest(as I understood it is not nessecary as of now), but yeah it is very important though to have a precise benchmarks for multiple sections in the modified commit as it will enables to look at a better picture and plan for changes accordingly😊.

btw do you actually get many entries for keys in single call in your tests? I think the delay: 0 in the issue example may effect this

Looking in the changes I have doubts that the changes actually improved the performance despite the numbers in benchmarks.
There could be done deeper investigations with tracing etc to figure out the cause of the performance downsides since when running tc server we are proxying the requests to another server that could affect the result numbers, right?

yup exactly since tailcall is just a relay to a actual graphql server ! yeah I agree, but since I eliminated the sorting after allocation of slice of DataLoaderRequest's, It is natural to pick up more speed since we are only iterating over slice here and not by dereferencing a pointer to heap of DataLoaderRequest(as I understood it is not nessecary as of now), but yeah it is very important though to have a precise benchmarks for multiple sections in the modified commit as it will enables to look at a better picture and plan for changes accordingly😊.

btw do you actually get many entries for keys in single call in your tests? I think the delay: 0 in the issue example may effect this

yeah I was thinking about the same here, need some help with that. 🤔

Looking in the changes I have doubts that the changes actually improved the performance despite the numbers in benchmarks.
There could be done deeper investigations with tracing etc to figure out the cause of the performance downsides since when running tc server we are proxying the requests to another server that could affect the result numbers, right?

yup exactly since tailcall is just a relay to a actual graphql server ! yeah I agree, but since I eliminated the sorting after allocation of slice of DataLoaderRequest's, It is natural to pick up more speed since we are only iterating over slice here and not by dereferencing a pointer to heap of DataLoaderRequest(as I understood it is not nessecary as of now), but yeah it is very important though to have a precise benchmarks for multiple sections in the modified commit as it will enables to look at a better picture and plan for changes accordingly😊.

btw do you actually get many entries for keys in single call in your tests? I think the delay: 0 in the issue example may effect this

yeah I was thinking about the same here, need some help with that. 🤔

the simplest way to check this is to add log with keys.len() and run the tests to see what is logged 😀

@meskill the facing trouble with logging feature of log here🤔, tried to enable that using feature flag of of env_logger didn't worked, can you help me with that ?

@meskill the facing trouble with logging feature of log here🤔, tried to enable that using feature flag of of env_logger didn't worked, can you help me with that ?

could you provide more details on what have you tried?
if you're trying to enable additional logs you could use flag --log-level when running tc
another option is to use just builtin macroses like println and dbg if you want to test something quick

the simplest way to check this is to add log with keys.len() and run the tests to see what is logged 😀

keys.len() is 1 for { delay : 0 }

@meskill I have been working on several possible ways to optimize this tried several approach but unfortunately none of them shows any remarkable improvements, also most of the points you mentioned are applicable too it's like we can't assume a scenario to be very valid here  additionally what complicates this problem more here is the presence of a function signature here
// Required method
    fn load<'life0, 'life1, 'async_trait>(
        &'life0 self,
        keys: &'life1 [K]
    ) -> Pin<Box<dyn Future<Output = Result<HashMap<K, Self::Value>, Self::Error>> + Send + 'async_trait>>

which only return a std::collections::Hashmap here and that too with unchangeable hasher, so we can't even use any other hashing algorithm to speed up the process, and since converting a concurrent hashmap to a one returned by the load method is again wastes lot's of CPU cycles in the process as first we have to acquire a lock to write then process it and then covert it back again to Hashmap by recalling the internal hasher of the std::collections::HashMap so in my understanding it is better that if we keep the return type unchanged and avoid using any third-party crate for the process,
Now since we can't a option of distributed insertion here through concurrent hashmap, maybe we our best shot is to figure out distributing the certain segments of code concurrently. but again as we had a talk about it chats here. This time I am thinking of something different
So spawning indefinite number of green threads won't do much better here too, what is going in mind to have a optimum number of threads spawned and that too depending upon factors as follows:

Depending upon number of values present &[DataLoaderRequest] in this case so then calling the spawn operation makes sense
Another aspect is how many threads should we spawn to optimize the process here should not be hard coded first to avoid contention in among other instances of thread.
Either it could decided through the user here directly via input from the config file, a param like distribute-loader-key=5 this will divide the task if the key size here is greater than 5 and will spawn one thread for the job and also give user the ability to hard control the number of actual green threads which can be spawned via, like for example loader-thread-limit=3 then if there more than 25 keys available to process but the it will limit the work to 3 only.

Let me know what you about this approach.

@tusharmath any further inputs ?

@amitksingh1490 Can we benchmark this locally and see the improvement in performance.
@artech-git The expectation was that the performance should be equal to that of what we get when enableHttpCache: true.

Hey @artech-git
Thank you for all of your efforts. I've discussed it with Tushar in more detail.
As you mentioned, there is indeed a little room for improvements due to strict constraints on the trait implementation from async-graphql library and unproven assumption that splitting logic inside load function will improve anything. We think, that actually room to improvements lies in the DataLoader implementation in the async-graphql library itself. It uses hashmap inside mutex that could pile up the synchronization time for mutex especially with small value of batch delay. Also, there could be other consideration as a Hasher implementation you've mentioned in the required trait

Hey @artech-git Thank you for all of your efforts. I've discussed it with Tushar in more detail.
As you mentioned, there is indeed a little room for improvements due to strict constraints on the trait implementation from async-graphql library and unproven assumption that splitting logic inside load function will improve anything. We think, that actually room to improvements lies in the DataLoader implementation in the async-graphql library itself. It uses hashmap inside mutex that could pile up the synchronization time for mutex especially with small value of batch delay. Also, there could be other consideration as a Hasher implementation you've mentioned in the required trait

I agree on it @meskill, so is there something else can we do ?

I agree on it @meskill, so is there something else can we do ?

We could investigate the possibility of writing our own dataloader implementation based on the async_graphql's and try to optimize performance of it.

I think that's going to be better. Feel free to discuss on discord with @meskill or @tusharmath if the objective isn't clear. Closing this PR for now. Thank you for the attempt 🙏


Rustup integration

Do we need to add rustup integration into VSCode and command like "Install Rust"?

Personally I think it goes a little too far, but it can be a great addition if though through.


Unable to start sign-in page on Web

Hello - thanks you for sharing the hard work you've done in this library. I wanted to help by reporting that when trying to open the sign in module in Flutter Web, I get the following error:
Error MissingPluginException(No implementation found for method launchFlow on channel firebase_auth_ui)

I'm using the        FirebaseAuthUi.instance().launchAuth([ method as shared in the examples. I see that uses some sort of reflection (I'm not a Dart expert).
Is this a valid defect? If so, how can I help fix it?
Also, is there a workaround? How can I manually trigger the login screen (launchFlow)?
Thanks so much in advance.

Hi, web isn't supported ATM. If you want, you can add this functionality and submit a PR for it :) I am currently too busy with other stuff and there are already some open issues which I'll need to address first when I get some time.

Hi @sam0829 ! Do you plan to support the web soon? And if not, are there any other alternatives?


Make 'Start Reactor' work the same as 'Stop Reactor' when reactor needs to be stopped

Kills reactor when VSCode exits, verified against insiders channel. Fixes #14.


Context.setProperty

Maybe I'm missing something, but why is there an getProperty method, but no setProperty method on sap.ui.model.Context?
Currently I always have to call the setProperty method of the model. If I only have the context, I need to get the model form the context and pass the context (which contains the model) in the method of the model, which seems kind of redundant, or the contexts path, which I would also need to get from the content.

Hello @tstorch !
I assume that you are using OData V2, but still I would like to understand this a bit better from my OData V4 perspective. There we rely on sap.ui.model.PropertyBinding#setValue as an API.
What exactly is your use case where you do not have a property binding, but want to set properties directly on a context?
Best regards,
Thomas

Hi @ThomasChadzelek
Actually I'm using a JSON Model. This model is meant to keep UI or control data, which is not in the OData model. I probably need to take a closer look at PropertyBindings.
Nonetheless I was wondering why there is no setProperty as this seems to a pretty obvious thing to have.
Regards,
Tristan

The context is a pointer to the model which can give you information about the content of the bound object. To change the data, you need to get the model and to work with it (which produces a new copy of the model and changes the context respectively).
I am uncertain about the exact scenario, but setting the binding mode to two-way can save you time from setting manually properties - https://openui5.hana.ondemand.com/#/api/sap.ui.model.PropertyBinding
Let us know if this is the functionality you search for.
Thanks

Surly I could set the value of a property via an property binding, but if I already have a context, why would I want to create a binding based on the context just to change it? What is the difference between changing a property via Model.setProperty and doing it via a property binding (except the ability of getting notified on further changes, which I do not need)?

When you use property bindings, you can be notified for changes and bindings are updated respectively. You can get the binding context (information about the current state of the data), and read, validate, save changes.
If you want to make any changes, this can happen by working with the model or the through the bindings.
It will be better to illustrate how you use sap.ui.model.Context and the point at which it should be used for updating data. Details for the whole scenario will be helpful as well.

Since there is no response from the author for more than 4 weeks I'm closing the ticket.


Variable substitution stopped working in version 1.7

Brief Issue Summary
I am having an issue similar to the one described in https://github.com/microsoft/vscode-cmake-tools/issues/1526. I am trying to set the cmake.sourceDirectory using the variable ${buildKit} as follows:
    "cmake.sourceDirectory": "${workspaceFolder}/project_1/${buildKit}",
I cannot configure the cmake project and I receive the error shown below.
I was able to later solve this by installing the previous version of cmaketools version. 1.6.0, and the substitution worked fine. I believe something broke in the transition to 1.7
CMake Tools Log

[expand] Invalid variable reference ${buildKit} in string: ${workspaceFolder}/project_1/${buildKit}
[rollbar] "sourceDirectory" is not a directory {"sourceDirectory":"/workspaces/copateneida/project_1/${buildKit}"}

Platform and Versions

Operating System: Ubuntu 20.04
CMaketools Version: 1.7.3

Are you using CMakePresets.json to configure your project?  Or are you using Kits and Variants?

I believe I am using kits. I have configured the kits in a separate cmake-kits.json file. And I am setting the cmake.sourceDirectory in the settings.json

I see what happened.  This commit disabled using certain variables in cmake.sourceDirectory.  @andreeis can explain further.  Is there a way we can make this work again?
https://github.com/microsoft/vscode-cmake-tools/blob/c1f11cb574bfb049e07d452e31ea950c6bba0533/src/extension.ts#L437-L438

Yes I can fix this. The original mistake on my part was that I was focused on scenarios where we want to identify if a project is or not a CMake project and that needs to happen before that annoying popup that we had about setting a kit. So I thought any kit information that is missing shouldn't block our analysis of having or not having a CMakeLists.txt (via "cmake.sourceDirectory"). But I didn't realize that if we do have a kit set then we need to expand any variables kit related. Quick fix.

Hwo to fix?

I try to use ${activeConfigurePresetName} in .vscode/settings.json and it is not getting evaluated.
${workspaceFolder} expansion works.
I use it as an argument to the clangd startup and it passes the non evaluated string ${activeConfigurePresetName} to the clangd call.
Is this behavior related to this issue?
I use CMakePresets.

@ToBoMi we support that feature, but your syntax is incorrect which is why it's not substituting it properly.  Try ${command:cmake.activeConfigurePresetName}.  (variable substitution documentation)

Already tried that and many variations after reading the docs. None of them worked.
Here is the current output:
This:
    "clangd.arguments": [
        "--compile-commands-dir=${workspaceFolder}/${command:cmake.activeConfigurePresetName}",
    ]

leads to the following clangd call (I replaced some things with [...] at some places)
I[17:29:29.866] argv[1]: --compile-commands-dir=c:\Users\[...]\swdev\[...]/${command:cmake.activeConfigurePresetName}

We don't own the clangd.arguments setting.  If that extension doesn't support command variable expansion, then you'll need to open an issue in that extension's repository to ask for the feature.


fix: update add.go

This PR fixes: #596
default "gpt-3.5-turbo" and default "open ai" were removed from the description.
Before

After

Made an extra empty commit just for semantic check to pass :smile:


I am unable to run it under Modal.

The modal, once opened, does not load the library.
Js
document.addEventListener('DOMContentLoaded', function () {
    // Initialize JSpreadsheet
    var spreadsheet = jspreadsheet(document.getElementById('spreadsheet-container'), {
        data: [
            ['Example 1', 'Example 2'],
            ['Example 3', 'Example 4']
        ],
        columns: [
            { type: 'text', title: 'Column 1', width: 120 },
            { type: 'text', title: 'Column 2', width: 120 }
        ],
    });

    // Handle row insertion
    document.getElementById('insert-row').addEventListener('click', function () {
        // Insert a new blank row at the end
        spreadsheet.insertRow([], spreadsheet.getData().length);

        // Alternatively, to insert at a specific index, specify the index:
        // spreadsheet.insertRow([], 1); // Inserts a row at index 1 (second position)

        alert('New row inserted!');
    });
});

Html

The outcome is an empty modal.
The libraries are loaded but the jexcel or jspreadsheet are not recognized.
Not sure what is the problem.

I have decided to discard the modal way. However, I am trying to add
dynamic selection of cells during formula building, in case the cell value
starts with =.
I have managed to add a Ctrl+click to individually click the cells and add
the reference but I am unable to add the range in case the click is not up.
The latter does not work. Would you have any advice on where I should look
into the library to achieve this behavior?
On Wed, Nov 20, 2024, 9:50 AM HW @.***> wrote:

You might need to create Jspreadsheet only when the modal IS opened and
the DOM is visible. I believe there are erros on the calculations since JS
will return 0 since the elements inside the modal are hidden.
—
Reply to this email directly, view it on GitHub
https://github.com/jspreadsheet/ce/issues/1713#issuecomment-2487921493,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AIZV5F6G54US2HOI6UMX5SD2BREMTAVCNFSM6AAAAABSB7G2XSVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDIOBXHEZDCNBZGM
.
You are receiving this because you authored the thread.Message ID:
@.***>


Fedora 33 - can't install any Snaps

Windows Build Number
Microsoft Windows 11 Preview [Version 10.0.22000.51]
WSL Version

[X] WSL 2
[ ] WSL 1

Kernel Version
5.4.72
Distro Version
Fedora-33
Other Software
No response
Repro Steps
sudo dnf install snapd
sudo snap install core
Expected Behavior
Snap is supposed to install core.
Actual Behavior
error: cannot communicate with server: Post "http://localhost/v2/snaps/core": dial unix /run/snapd.socket: connect: no such file or directory

Diagnostic Logs
No response

Snaps rely on systemd support and as of right now WSL doesn't support systemd. We're tracking this issue here:
https://github.com/microsoft/WSL/issues/994
I'll close this out as this is part of that feature request. Thanks for filing!
/dupe #994


add disable cache options

add disable cache options

Coverage increased (+0.01%) to 95.291% when pulling 6226f53fcc1774442fb3339feba7c75a0bd95c38 on northwolf0614:master into d1d360ff9352be5b2894a093e22fac42f84a5938 on randing89:master.


Add links to Workbook.ActiveSheet and revise text

Deleted text that describes the Window.ActiveSheet property.

:warning: Validation status: warnings

File
Status
Preview URL
Details

VBA/Excel-VBA/articles/workbook-activesheet-property-excel.md
:warning:Warning

Details

VBA/Excel-VBA/articles/workbook-activesheet-property-excel.md

Line 48: [Warning] Invalid file link:(~/Excel-VBA/articles/window-activesheet-property-excel).
Line 50: [Warning] Invalid file link:(~/Excel-VBA/articles/workbook-activechart-property-excel).

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

:warning: Validation status: warnings

File
Status
Preview URL
Details

VBA/Excel-VBA/articles/workbook-activesheet-property-excel.md
:warning:Warning

Details

VBA/Excel-VBA/articles/workbook-activesheet-property-excel.md

Line 48: [Warning] Invalid file link:(~/Excel-VBA/articles/window-activesheet-property-excel).
Line 50: [Warning] Invalid file link:(~/Excel-VBA/articles/workbook-activechart-property-excel).

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

All content in this repository has been migrated to a new repository (https://github.com/MicrosoftDocs/VBA-Docs). The changes in this PR have been applied in the new repository (as part of pull request #61), so this PR is being closed. The changes will be published to the live docs soon. Thank you for your contribution!


Change text size of subnav elements

Could you include a screenshot of the change?
It's a good habit even when we've discussed the change in-person - helps out people in the future doing code archaeology.