HR Brainstorming

We need to brainstorm on the features below for the HR service

Employee Wellness DashboardImplement a wellness dashboard that tracks employee well-being, incorporating data on physical health, mental health, and work-life balance.

Employee Engagement Gamification

Integrate gamification elements to boost employee engagement, encouraging healthy competition, and recognizing achievements.

AI-driven Personalized Learning Paths:

Utilize artificial intelligence to analyze individual employee skills and preferences, creating personalized learning paths for professional development.

Predictive Analytics for Employee Retention:

Use predictive analytics to identify potential turnover risks, allowing HR to take proactive measures to retain valuable employees.

Interactive Org Chart:

Create an interactive organizational chart that includes employee profiles, skills, and project involvement for better team collaboration and transparency.

Voice-enabled HR Assistant:

Integrate a voice-activated HR assistant that allows employees to perform HR-related tasks through natural language commands.

Real-time Employee Feedback:

Enable continuous feedback mechanisms for employees to share real-time insights on projects, teamwork, and workplace satisfaction.

360-Degree Performance Reviews:

Implement a comprehensive 360-degree performance review system that includes feedback from peers, subordinates, and managers for a holistic view.

Automated Onboarding Bot:

Introduce a chatbot for automated onboarding processes, guiding new hires through paperwork, company policies, and introductions to team members.

Emotional Intelligence Tracking:

Develop a tool that assesses emotional intelligence within teams, helping managers understand and improve team dynamics.

Skills Marketplace:

Create a platform within the HR module where employees can showcase their skills, and managers can discover talent for project assignments or mentorship opportunities.

Employee Financial Wellness:

Integrate tools to help employees manage their financial wellness, providing budgeting tips, investment insights, and personalized financial planning.

Virtual Team-building Challenges:

Facilitate virtual team-building challenges and activities, fostering a sense of camaraderie among remote or distributed teams.

Employee Recognition Blockchain:

Utilize blockchain technology for secure and transparent employee recognition programs, ensuring the authenticity of achievements and awards.

Diversity and Inclusion Analytics:

Implement analytics to track and measure diversity and inclusion efforts within the organization, promoting a more inclusive workplace.

Personalized Health and Fitness Plans:

Offer personalized health and fitness plans for employees, integrating with wearable devices to encourage a healthy lifestyle.

Dynamic Goal Alignment:

Implement a dynamic goal-setting system that aligns individual and team goals with organizational objectives, fostering a sense of purpose and alignment.

Employee Resource Groups (ERGs) Platform:

Create a platform for Employee Resource Groups to connect, share insights, and collaborate on initiatives related to diversity, inclusion, and employee well-being.

Smart Scheduling Assistant:

Integrate an AI-powered scheduling assistant that considers individual preferences and peak productivity times when scheduling meetings and tasks.

Employee-driven Innovation Hub:

Establish an innovation hub where employees can submit and collaborate on ideas for process improvements, product innovations, or other organizational enhancements.

These features aim to go beyond traditional HR functionalities, providing a holistic approach to employee well-being, engagement, and professional development within the organization. Customizing these features based on client needs will further enhance the uniqueness and effectiveness of the HR module in the NEXTGEN ERP system.

This is great!!!!!!!

Employee Wellness Dashboard

Employee Engagement Gamification

AI-driven Personalized Learning Paths

[ ] Interactive Org Chart

[ ] Voice-enabled HR Assistant

[ ] Real-time Employee Feedback

[ ] erer5rte

[ ] regr

[ ] ergere


Add Indonesian Translation

Hi There,
I've already translated it into Indonesian.

[*] full_description.txt
[*] short_description.txt

Awesome, thanks for your help @andriannobella! 👍
Can you please put the translation on Crowdin too? You can login with your GitHub account there and follow the instructions. The process is quite straightforward. If you have any questions, please do not hesitate to ask.

Awesome, thanks for your help @andriannobella!
Can you please put the translation on Crowdin too? You can login with your GitHub account there and follow the instructions. The process is quite straightforward. If you have any questions, please do not hesitate to ask.

Done!
Please check it out


[Bug]:   Going into sleepmode stops the break in Windows

Version

[X] I'm using version 1.14.1

Known issues

[X] I've checked Known issues

Existing issues

[X] I've checked Existing issues

Advanced Preferences

[X] I've checked Advanced Preferences

What operating system are you using?
Windows
Operating System Version
Windows 10 version 22H2
Reproduction steps

Start a break
Go into sleep mode
Awake the system

Reproduces how often: Every time you wake up the system in the middle of a break.
Expected Behavior
The break should keep going.
Actual Behavior
The break immediately stops.
Relevant log output
breakPlanner: {reference: startMicrobreak,
            timeleft: נותרו 9 דקות 5 שניות,
            breakNumber: 1,
            postponesNumber: 0,
            Do Not Disturb: false}

            settingsFile: C:\Users\Omer\AppData\Roaming\Stretchly\config.json

            logsFile:  C:\Users\Omer\AppData\Roaming\Stretchly\logs\main.log
            Node.js: 18.15.0
            Chromium: 114.0.5735.45,
            Electron: 25.0.1

Preferences
No response
Additional information
This bug only occurs on Windows, from what I know.
Regardless, thank you for this app, it helps me a lot :)
Code of Conduct

[X] I agree to follow this project's Code of Conduct

Hi that is expected behavior. You can change via advanced preferences:
https://github.com/hovancik/stretchly#advanced-preferences
https://github.com/hovancik/stretchly#pause-breaks-on-suspendlock-


A question

Very interesting tool and preprint.
In your abstract you state "translating the electrical signal it outputs into a sequence of bases still requires high-end computer hardware" - what is the definition of high-end computer hardware that you are using here?

Thanks!
I'm on a laptop I would consider high-end (~€2000, 8-core 3GHz CPU, Nvidia T500 GPU, 16GB RAM) and on a set of 1000 16S reads I barely hit the maximum throughput for a MinION at 225 kb/s with guppy (Version 6.2.1+6588110) in fast mode. On second thought this could have been worded more carefully, maybe high-end laptop would have been a better choice. Would you agree?
Thanks a lot for all the collected resources on running on the Jetson, btw! I'm sad that I couldn't get the Jetson GPU to work in the end, as stated in the paper, but your notes helped a bunch.

Thanks for the clarification. It's more semantics I guess and I think it's one of those things like the accuracy "issues" that always get thrown about that just really aren't true anymore. If FAST basecalling is the goal then a $399 USD Jetson Xavier NX is more than capable of keeping up with a MinION in fast mode - is that high-end computer hardware, I'm not sure (not by my definition) but I think a bit more clarification around what one is defining as high-end is useful for readers as it can mean many different things to different people. In terms of running the HAC and SUP models, something like the new Jetson Orin AGX will easily achieve those sorts of speeds. Additionally, these days you can get fairly entry level gaming laptops that are more than capable of keeping up with FAST basecalling, and even HAC, so I don't think the high-end "stigma" is as big of an issue as it used to be. But this is just my 2 cents. ;)
In terms of the T500 GPU, it is Turing based but it's held back by 2GB of GPU RAM. The Xavier NX and AGX are running Volta GPUs with less CUDA cores than your T500 but they have the advantage of sharing decently fast system RAM between the GPU and rest of the system. That's why the 384 CUDA core NX with 8GB of shared RAM is able to perform FAST calling in real-time.
I'm glad all those notes etc were useful. Unfortunately you were always going to struggle with the Jetson Nano for basecalling, due to the fact that it has a Maxwell based GPU which doesn't have the CUDA compute version able to support Guppy out of the box. It is possible to get Guppy working on the Nano, it requires developer access to the Guppy source and some additional compilation flags. I have a friend that has several Nano's running Guppy and it seems to hold up decently for Flongle levels of data. Before I joined Nanopore I was playing with sequencing on the Nano (it runs MinKNOW just fine) and then using free GPU enabled Google Colab notebooks for near real-time cloud basecalling. It worked really nicely and I always thought it would be a good cheap options for teaching labs, low cost entry etc.
Thanks again for the really interesting manuscript, some really nice work and it's exciting seeing the accessibility and cost of entry lowered for people, I believe this is the heart of the MinION technology. I'd love to chat some time about an idea that I had recently that fit's quite nicely within the framework of what you've done here.

Thanks, had not seen the benchmark before, very useful!
That's a very thorough description of resource requirements, I suppose you're right that on appropriate hardware base calling has become more efficient than is generally believed. I'll try to find some descriptor that better captures the current state of art for the paper we're submitting. :)
I'd very much like to chat about your idea! Could you send me an email at c.v.delannoy[at]tudelft.nl or find me on twitter (@cvdelannoy) so we discuss further or set up a call?


[PHPStanRules] Fixes #2330 Add ForbiddenCallOnTypeRule

All checks have passed 🎉 @TomasVotruba it is ready for review.

All checks have passed 🎉 @TomasVotruba I think it is ready.

Thank you :+1:


Error: Self-referential block

Hi,
When I try to run TF plan command I get the following error:
terraform plan -out .tfplan
var.private_key_password
  Password for private key to use for signing

  Enter a value:

module.network.data.oci_identity_availability_domain.ad: Reading...
module.network.data.oci_identity_availability_domain.ad: Read complete after 0s [id=ocid1.availabilitydomain.oc1..xxxxxx]
╷
│ Error: Self-referential block
│
│   on compute/main.tf line 24, in resource "oci_core_instance" "server_0":
│   24:           server_0_ip = oci_core_instance.server_0.private_ip,
│
│ Configuration for oci_core_instance.server_0 may not refer to itself.
╵
╷
│ Error: Self-referential block
│
│   on compute/main.tf line 24, in resource "oci_core_instance" "server_0":
│   24:           server_0_ip = oci_core_instance.server_0.private_ip,
│
│ Configuration for oci_core_instance.server_0 may not refer to itself.

I had to change the network/data.tf ad_number = 1 just because the region af-johannesburg-1 doesn't allow/have ad_number=2
Any idea on how I can fix this?

Sorry to read this I develop it for the use of two ad. I recommend to use another region.

I have the same issue on the UK region, that does have multiple AD's.
I'll try and create an account on the Frankfurt region, I noticed that there are multiple references to that region in the TF files.

Hi,
I get the same error in UK region, that does have 3-AD's.
I'll try and create an account in the Frankfort region later today.
A separate question, but what is the nsg_id's?

I found the issue of the Self-referential block in the private_ip of the metadata section.

The nsg_id's referencing to network security groups which are allowing ssh traffic:
https://github.com/r0b2g1t/k3s-cluster-on-oracle-cloud-infrastructure/blob/787016ea76229793229fd573f45664f9f60fe4df/terraform/network/main.tf#L52


Add feature to be able to retain the window size and specify coordinates when making borderless

For instance, the window I'm running is 1920x810 (21:9). I would like to make the window borderless in 1920x810, as well as being able to specify them to align to a specific coordinate on my screen, for instance x=0, y=0.
This was a feature from Borderless Gaming that I really liked having and is essential when I use it.
Perhaps when unspecified, the default behavior should be either just specify to x=0, y=0 or place on center of screen?

All-right! I will work on this feature as soon as possible.

Feature added.


Plugin doesn't work with webpack 5 / Next.js 11

Thanks a lot for this excellent plugin, it's been really helpful for us!
Problem
This plugin doesn't work correctly when using Next.js 11. Attempts to require images result in an error like the one below during the build:
Failed to compile.

./src/images/my-awesome-image.svg
TypeError: unsupported file type: undefined (file: undefined)

Workaround
It seems like the problem is due to the fact that from version 11 Next.js uses Webpack 5 by default. It looks like it's possible to work around this issue by forcing the use of Webpack 4. This is done by setting webpack5 to false in next.config.js - see the webpack5 option docs for more.
Edit: rather than disabling webpack5, this can be solved more neatly by toggling disableStaticImages in the nextjs config. See @jaimemasson's comment below.

the actual fix for this is to disableStaticImages in nextjs config. see https://nextjs.org/docs/basic-features/image-optimization#disable-static-imports

If next11 does static image optimization itself, what is the benefit of using this package now?

@Veldrovive AFAIK nextjs11 doesn't do static image optimization.

the actual fix for this is to disableStaticImages in nextjs config. see https://nextjs.org/docs/basic-features/image-optimization#disable-static-imports

Thanks for pointing this out @jaimemasson, I wasn't aware of this configuration option! Definitely a better fix. Will edit the original issue to direct people down to your comment.

If next11 does static image optimization itself, what is the benefit of using this package now?

As of now, the next/image component is garbage.

This should be referenced in the README somewhere since next 11 and webpack 5 are now the defaults and running into this problem is almost guaranteed for new projects.

Thank you for the feedback. I updated the original discussion to mention this workaround to help prevent people from running into this issue 🙏


Make rule names more descriptive in generated woke.yml

Today generate_woke_config creates a new set of rules that are named airbnb-0, airbnb-1, etc. We should prefer human-readable names instead, like airbnb-whitelist, airbnb-dummy, as those names are easier to use with woke's inline ignore syntax.
# Clear
let url = URL(string: "www.noninclusive.com/whitelist")  // wokeignore:rule=airbnb-whitelist
# Less clear
let url = URL(string: "www.noninclusive.com/whitelist")  // wokeignore:rule=airbnb-0

Alternatively, the change to disable default rules lands (see https://github.com/get-woke/woke/pull/132), we can use simple rules names that don't have an Airbnb prefix.

Wanted to note that the feature to disable the default rules shipped in v0.17 https://docs.getwoke.tech/rules/#disable-all-default-rules


More dynamic tags based on environment

Is your feature request related to a problem? Please describe.
Wanting to add tags for billing purposes we find ourselves that the tags allow mostly static values except two placeholders.
Client wanting tags for example for prod/dev we'd like to map tags based on the environment to some other values.
Describe the solution you'd like
Allow more advanced expressions for tags like mapping based on environment.
If it's feasible it could use CloudFormation intrinsic functions to achieve the most flexibility but after some consideration it seems that for tags a simple map from environment would suffice for most use cases.
Since current implementation of tags only allows strings this could be achieved by using object as a map, and then handle this inside HydrateTags function.
[
  {
    "Key": "user:EnvType",
    "Value": {
      "prod": "production",
      "otherprod": "production",
      "dev": "development",
      "jdoe": "development",
    }
  }
]

With the simpler approach we could consider somehow handling a default value if the mapping doesn't match anything, it could be as "" (empty string) or something like "__default".
If this gets accepted I should be able to submit a pull request for this.

Hi @dsypniewski this is currently a supported feature. Just use the string {project-env} within your "Value" field. Amplify CLI will auto replace that string with the current Amplify env. See more here: https://docs.amplify.aws/cli/usage/tags#using-predefined-variables

I know about this and this feature request is supposed to built upon that to be more flexible.


build: fix on tag workflow

Closes #1028

tested and verified the fix in https://github.com/gitify-app/gitify/pull/1051 this morning


sonarcloud: Use Inclusions Property

Fixes Code Coverage builds by switching to using the sonar.inclusions property.
Some of our build targets set their source directories in the include paths. Those paths are then emitted in the sonar-project.properties file here. The issue is that this can lead to files being indexed twice which the sonar-scanner treats as an error.
For example the generated file can look something like:
sonar.sources=
  foo/src/foo.cpp, \ # Absolute path to cpp file
  foo/src/ # Picked up from include paths. Causes scanner to scan foo.cpp twice

The inclusions property requires that we use a pattern string instead of paths, so include paths are modified with the appropriate pattern to ensure header files are included in the analysis.
See related PR for testing details:

https://github.com/swift-nav/starling/pull/6837

Sonar cloud seems to be reporting "no coverage information" in that PR?

Sonar cloud seems to be reporting "no coverage information" in the test PR?

That's because there weren't any c/c++ code changes. It only reports the analysis for new code.
I posted another PR link where there were cpp files added and it did the analysis correctly. I've added it to this PR as well.


Various fixes for the readme

This PR fixes a couple of issues I noticed while reading the README on PyPI, plus some whitespace trimming.
FYI the TOC is also not up-to-date: my editor has a tool for auto-updating it, I'd be happy to include that in this PR.

An updated TOC would be great. Thanks.

I've rebased on master, resolved the conflicts and added a TOC update, as requested.

Thanks!


Question about using data descriptors for class attributes.

Hi, I'm trying to determine if your library could help me achieve the end result I want. (Sorry for using github issue to ask a question :sweat_smile: ) I have some custom classes that allow attribute access to data stored in redis. I originally wrote the code almost a year ago and it works well. Though I would like to use data descriptors instead of overriding __getattribute__ and __setattr__ like I am doing currently.
The issue I am running into is that the descriptor protocol only works when assigned at the class level and some of the classes have many attributes--so many that I dont want to define them all separately. I am looking for a way to do it by looping through a dict which contains lists of attribute names grouped by value type. I have been searching online for a while but cant find any examples that fit my use-case. Could your library help me in some way? Any advice you could offer will be greatly appreciated. Of course I'd understand if you decide this is outside the scope of what github issues are for (because obviously it is)
Cheers!

Yes, this is not the best use for issues, but it works fine.
From the little bit I can tell, this library won't be much help with that specific issue, but it can help with writing the descriptors in the first place.
Now, I certainly don't mind trying to help you figure this problem out. Could you give me an example of what you'd like the use of that dict to look like? I'm not sure I understand what you're asking for.
Also, please answer this by emailing me<EMAIL_ADDRESS>.

(I know this is closed, but still)
Sounds like a base class with your prefered descriptors would be a good idea. Then you merely have to subclass it to get the descriptors for free! Although I'm not sure if it'd work in your case, it's worth a shot.
(You can also CC your emails to me as well if you want<EMAIL_ADDRESS>)

@sad2project @Vgr255  Thanks for your quick responses and offers to take a look at what I had. I really appreciate it! Though I actually was able to figure it out (finally!). It appears to be working exactly how I want. If you or anyone else who happens to stumble upon this issue would like to check out the solution I came up with, you can find it here:
https://github.com/antergos/antbs/blob/e8f2ce0190e0abbf63751396f8eaa2b2d56b3b1e/antbs/database/base_objects/redis_hash.py
Thanks again! BTW @sad2project I think this library is going to be very useful to many. There really isnt anything like it available (that I could find at least) :+1:

I'm glad to hear it!
Thanks for what you said about the library, too! I was really surprised about how little I could find on descriptors out there, which is why I wrote my book (http://www.lulu.com/spotlight/jacobz_20) which led to me writing this library.


[EasyCodingStandard]  Property "usefulAnnotations" was not found

Updating to the latest version of the library (5.4.16) produces the following error when trying to check the coding style:
PHP Fatal error:  Uncaught Symplify\EasyCodingStandard\Exception\DependencyInjection\Extension\InvalidSniffPropertyException: Property "usefulAnnotations" was not found on "SlevomatCodingStandard\Sniffs\TypeHints\TypeHintDeclarationSniff" sniff class in configuration.  in /var/www/legacy/vendor/symplify/easy-coding-standard/src/Yaml/CheckerConfigurationGuardian.php:44
Stack trace:
#0 /var/www/legacy/vendor/symplify/easy-coding-standard/src/Yaml/CheckerServiceParametersShifter.php(152): Symplify\EasyCodingStandard\Yaml\CheckerConfigurationGuardian->ensurePropertyExists('SlevomatCodingS...', 'usefulAnnotatio...')
#1 /var/www/legacy/vendor/symplify/easy-coding-standard/src/Yaml/CheckerServiceParametersShifter.php(100): Symplify\EasyCodingStandard\Yaml\CheckerServiceParametersShifter->processSniff(Array, 'SlevomatCodingS...', Array)
#2 /var/www/legacy/vendor/symplify/easy-coding-standard/src/Yaml/CheckerServiceParametersShifter.php(79): Symplify\EasyCodingStandard\Yaml\CheckerServiceParametersShifter->processServices(Array)
#3 /va in /var/www/legacy/vendor/symfony/config/Loader/FileLoader.php on line 166

The library claims to be following semver but this is the second time it broke on us. We have to make extra efforts to lock the library and all it's dependencies to a fixed version (otherwise breakage is guaranteed, as we noticed the first time it broke for lack of proper version locking for dependencies).

What version of slevomat\coding-standard you have installed?
How does your ecs.yml look like?

I have not restricted the version of slevomat\coding-standard. symplify/easy-coding-standard requires slevomat/coding-standard: ^5.0.1 which is the latest, so that's the one being installed.
My ecs.yml contains this:
SlevomatCodingStandard\Sniffs\TypeHints\TypeHintDeclarationSniff:
    enableObjectTypeHint: false
    usefulAnnotations:
      - "@throws"

Looking at the breaking changes in slevomat/coding-standard version 5.0.0 it makes sense why it would break. What doesn't make sense is for symplify/easy-coding-standard to introduce (dare I say force) this breaking change without a major version  bump.

This is not related to ECS, but rather to slevomat/coding-standard.
If you'd lock it to v4, this cannot happen.
Update in this version just made that BC break more visible.

Downgrading slevomat/coding-standard to 4.8.7 automatically forces me to downgrade symplify/easy-coding-standard to version 5.4.13. Not doing so (forcing symplify/easy-coding-standard to remain at version 5.4.16) results in Your requirements could not be resolved to an installable set of packages. It's textbook BC.
It's not a big issue for us, we now know what we're dealing with. We expected semver to be respected, but it's not. Its fine, we'll work around it. I'm just unpleasantly surprised to notice the attitude and the misleading advertising.

In case you use slevomat/coding-standard directly, you should have it in composer.json.

Just throwing this out there, maybe it helps drive home the point i'm trying to make. Forcing slevomat/coding-standard to version 4.8.7 and keeping symplify/easy-coding-standard to version 5.4.13 (which does not result in any composer conflicts), results in this error when running a check:
PHP Fatal error:  Uncaught Symfony\Component\DependencyInjection\Exception\RuntimeException: Cannot autowire service "Symplify\TokenRunner\DocBlock\DescriptionAnalyzer": argument "$typeNodeAnalyzer" of method "__construct()" references class "Symplify\BetterPhpDocParser\PhpDocParser\TypeNodeAnalyzer" but no such service exists. in /var/www/legacy/vendor/symfony/dependency-injection/Compiler/DefinitionErrorExceptionPass.php:54
Stack trace:
#0 /var/www/legacy/vendor/symfony/dependency-injection/Compiler/AbstractRecursivePass.php(82): Symfony\Component\DependencyInjection\Compiler\DefinitionErrorExceptionPass->processValue(Object(Symfony\Component\DependencyInjection\Definition), true)
#1 /var/www/legacy/vendor/symfony/dependency-injection/Compiler/DefinitionErrorExceptionPass.php(32): Symfony\Component\DependencyInjection\Compiler\AbstractRecursivePass->processValue(Array, true)
#2 /var/www/legacy/vendor/symfony/dependency-injection/Compiler/AbstractRecursivePass.php(46): Symfony\Component\DependencyInjection\Compiler\Definition in /var/www/legacy/vendor/symfony/dependency-injection/Compiler/DefinitionErrorExceptionPass.php on line 54

Unless I fix ALL symplify/easy-coding-standard dependencies to a certain version, it will keep breaking. You made significant changes from one version to the other (either in symplify/easy-coding-standard or in dependencies which are under the symplify namespace) without any regards to BC breaks. I've never seen this happening to any other package I've been using. One cannot actually rely that using version x.x.x of symplify/easy-coding-standard will be usable. The only version that "might" work is the latest, but only if you're a new user of the library or are lucky enough to not be affected by the BC changes in the dependencies. Anyway, I rest my case.


Refactoring of the runtimes

[ ] Improve the chrome app runtime (e.g. window size and custom icon).
[ ] Implement proper NW.js runtime, so that we have proper apps for both FF and Chromium engine.
[ ] Make app runtime reuse code more.
[ ] Runtime renaming to make things less confusing (e.g. few people know what xul is, or how to pronounce it). At least a simple "app" and "browser" that autoselects.
[ ] Make app icon and app title work in broser and app runtimes.
[ ] Electron? Probably holding this off, since NW.js should be enough and better suits Flexx' ideas.
[ ] Streamline browser apps, e.g. interaction with webbrowser module.

Naming ...
Browsers are easy:

browser - select default browser.
browser-firefox, browser-opera, etc. ... whatever is supported by webbrowser module.
Shorthands for browsers that we support firefox, chrome, chromium, edge (and safari)?

Apps:

app - select available app: xul if firefox is available, otherwise nw.
app-xul  or app-firefox?
app-nw
app-chrome  or maybe drop chrome app completely in favor if nw.js?

Another note: Xulrunner is no longer supported (as in the binaries are not updated) you have to use the firefox exe. This means that you can not package this runtime along with the app. Though we can ask (via dialite #335) the user to install firefox. We can auto-download NW.js though (after prompting the user that we will do so).

Merging this into separate branch. Will do more work and fix tests at the end.


Wrong RPATH in libstdc++.so

/t/nwani_1597448808 ❯❯❯ ldd $CONDA_PREFIX/lib/libstdc++.so
	linux-vdso.so.1 (0x00007ffe5b797000)
	libm.so.6 => /lib64/libm.so.6 (0x00007f0881491000)
	libc.so.6 => /lib64/libc.so.6 (0x00007f08812c7000)
	/lib64/ld-linux-x86-64.so.2 (0x00007f0881791000)
	libgcc_s.so.1 => /lib64/libgcc_s.so.1 (0x00007f08812ac000) <-----------This should have been from $CONDA_PREFIX

/t/nwani_1597448808 ❯❯❯ readelf -a $CONDA_PREFIX/lib/libstdc++.so | grep RPATH
 0x000000000000000f (RPATH)              Library rpath: [/home/conda/feedstock_root/build_artifacts/ctng-compilers_1596393189889/_h_env_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placeh/lib]

This is due to binary_relocation being turned off? Shall we edit the rpath to be $ORIGIN
cc @mingwandroid

Can you check that this issue is fixed?

Yes, I tested it. It works, Thank you!


Update 0.17.0 changelog with minor changes

Issue #, if available:
Description of changes:
Testing (if applicable):
Documentation added/planned (if applicable):
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

/joke

/joke


pace 0.0.4

Type: Missing
Summary:
pace 0.0.4
Details:
Repository linked includes Apache-2.0: https://github.com/cpsubrian/pace
Resolution:
Apache-2.0
Affected definitions:

pace 0.0.4

@LandOfBliss Already Declared correctly with MIT. Closing.
https://www.npmjs.com/package/pace/v/0.0.4#license-mit


Raise error on invalid retained array retrieval

Fixes  #6153
~Depends on #7216~

#7216 is merged.  Could you merge or rebase to master?

@toslunar Thank you for notification.
Rebased to master.

Closing as discussed offline: there's no strong motivation to merge this enhancement to v7.


Update Benchmark modules

Updated Highcharts version;
Updated Mustache version;
Updated lodash to latest;
Updated Handlebars version;
Updated ejs version;

Some templating engines (like ejs) just got faster.

Some templating engines (like ejs) just got faster.

Still not faster than eta, right?

Obviously, but template engine is still faster lol

Thank you! Sorry for the late response.


TypeError: Cannot read property 'err' of undefined

I'm sorry to bother you.

This error occurred when hotReloading, please follow below steps.

using  function in 2nd argument of useAction.

node.useAction("NEW_MESSAGE", () => ({ message: 'a' }));

change something in same file.

The cause of this error is Node has already same keyName ?

Hi, you are not bothering us at all.
Thank you for making time in logging this issue. We will add it to our bug stack.

have solved this issue with the latest v2.2.0 @YutamaKotaro

closing the issue with the same


Script for installing build enviorment on Windows

A lot of people, especially before GSoC starts, write to me that settin up build enviorment on Windows is great pain. Binaries of externals that we store on our site not alway work - I'm not sure why, maybe the reason is that they were compiled for other version of Visual Studio that user have. Second problem is that our externals were compiled only for 32-bit build, so the are useless for 64-bit build on Windows.
So there is a need for a script (python, powershell or ruby) that will download dependencies, compile it with given compiler and install in given directory. Thanks to that we could remove our externals for windows and user will build dependencies with the compiler he/she use.

Here is a script for installing on windows, that worked for me:
run.zip
!!! Read the first lines (how to install) in the script !!!

Closing this old comment.


Incorrect metadata-hash param returned from v2/assets query.

When making a request to the v2/assets endpoint, I get what appears to be an incorrect metadata-hash in the response, apparently copied from a different asset in the payload.
The correct metadata-hash is returned when requesting a specific asset by id /v2/assets/{asset-id}

The incorrect metadata-hash is returned when making a request to /v2/assets?name=RareAF&limit=100 and is identical to (copied from?)  another asset in the same payload.

There may be more params besides metadata-hash returned in the payload that are copied from a different asset but they happen to be identical in my example.
I don't have a test env setup to check but I did browse the relevant section in the indexer and nothing obvious stuck out.
One thing I did notice is the buffered channel here: https://github.com/algorand/indexer/blob/2fbf02d417eafb0720b431c73e81ce1ca69b7c90/idb/postgres/postgres.go#L2521
Adding a buffer to a channel has almost always bit me with a subtle race bug.  Unlikely to be the cause here but maybe something to look at.

Which network is this on? I don't see it on mainnet.

It was on TestNet


Scroll stops working when cursor image disappears using Unclutter

Prerequisites

[X] Put an X between the brackets on this line if you have done all of the following:

Reproduced the problem in Safe Mode: http://flight-manual.atom.io/hacking-atom/sections/debugging/#using-safe-mode
Followed all applicable steps in the debugging guide: http://flight-manual.atom.io/hacking-atom/sections/debugging/
Checked the FAQs on the message board for common solutions: https://discuss.atom.io/c/faq
Checked that your issue isn't already filed: https://github.com/issues?utf8=✓&q=is%3Aissue+user%3Aatom
Checked that there is not already an Atom package that provides the described functionality: https://atom.io/packages

Description
I'm using Unclutter to make my cursor disappear after a few seconds of inactivity. When in Atom, and given that Atom has the focus, the mouse scrolling will cease to work as soon as the cursor image has disappeared. This is reproduced everytime, but that's specific to Atom, other sowftware like Gedit, Firefox or Eclipse still have the scroll working.
Steps to Reproduce

Install and run Unclutter
Open Atom
Give the focus to the editor, scroll a bit to check everything works
Wait for the cursor to disappear
Scroll without moving the cursor -> nothing happens
Move the cursor and scroll again -> it works

Reproduces how often: always
Versions
OS: Ubuntu 17.04
Window Manager : awesome v4.0 (hopefully not relevant)
Unclutter version: standard package version
Atom    : 1.19.2
Electron: 1.6.9
Chrome  : 56.0.2924.87
Node    : 7.4.0

I'm not able to reproduce the fault that's described with the same versions listed (awesome included).
I would suggest that you reinstall the programs, if you haven't tried that already.

Hi,
Ok, that's very weird. Thank you for taking the time to try and reproduce it.
I won't have the time to look at it this week but I'll try to do what you suggest (and other things, too). i'll give you my findings.

New findings:

It does not work with Unity window manager (default wm for my Ubuntu version)
It does not work after purging everything (dpkg -P, removed ~/.atom and ~/config/Atom)
It does work when running unclutter with the -noevents option:

  -noevents
         stops unclutter sending a pseudo EnterNotify event to the X client whose cursor has been stolen.  Sending the event helps programs like emacs think that  they  have  not  lost  the  pointer focus.  This option is provided for backwards compatibility in case some clients get upset.

Used atom 1.20.0 for these new tests.
So there's still something with this unclutter/atom combination, but I don't know what's the configuration difference that prevents you from reproducing it.

Strange, now that  you've purged  both Atom and config we should have a similar setup. I don't have much else to go on as it seem to depend on something I don't know and therefore I will not be able to help you with your focus problem.


Precompiled groovy script plugins fail when containing plugin applications themselves

When porting our script plugins script plugins to precompiled groovy script plugins, those plugins fail with an org.gradle.internal.service.UnknownServiceException.
Context
I'm in the process of porting our script plugins in our elasticsearch build (see https://github.com/elastic/elasticsearch/)  to precompiled script plugins.
Steps to Reproduce
precompiled-groovy.zip
Run ./gradlew someScriptPluginTask on the attached sample project.
Your Environment
Build scan URL: https://scans.gradle.com/s/eynaasrwztzbw

I noticed an interesting workaround for us.
instead of
apply plugin: ...

using explicit
project.apply(plugin:...) seems to fix this problem

see https://github.com/elastic/elasticsearch/commit/1a643c15f7283f30f5c5d60c5e591b5b87470f31


Add a clamd.timeout to handle large files (>10MB)

Files larger than around 10MB will usually cause an error from clamav-rest, unless the bootstrap.sh script is modified to include a clamd.timeout in milliseconds.
Please see this commit for details; https://github.com/ministryofjustice/moj-clamav-rest/commit/c9670b62551e6d12953c879ecbdbadd085b514a7

Done by #30


patient-ssapp v0.6.0 "BROWSE TRIALS" is limited to 100 trials on the server side

Implement a "paginator" on the patient-ssapp client side to browse pages of trials sorted by a particular condition (as it is +/- done in the physician backoffice).

Ready for review

Considering it closed. Thank you @pccosta-pdm


[Feature Request] Download Manager should download chapters in ascending order

Is your feature request related to a problem? Please describe.
When I find a manga to download via one of the plugins, hit "Select All" and select download, it starts downloading the chapters in reverse order.
Describe the solution you'd like
It should download the chapters in ascending order. If it is intentionally queued descending for other reasons, perhaps there should be a toggle button or something.
Describe a small use-case for this feature request
I don't think I'm the only one who hits "Select All" because I am reading a new manga. It would be nice to be able to start reading sooner because of how slow the download process is.

Oh yea that one. When I was still using the built-in download manager it was a pain that it started to download a 300 chapter long manga at 300, so you need to wait a few hours to be able to start to read. So maybe showing the chapters in the current order so you can find the new chapters, but download in the reverse order to get the first chapters first.

Good idea! Will work on this.


adjusted family description in documentation and header file

Signed-off-by: Ansh Sarkar<EMAIL_ADDRESS>

@doug-walker could you pls review this PR

Thank you @Ansh-Sarkar.   Once we get the required second approval, we will merge this.

@doug-walker thanks for reviewing. will be waiting for the 2nd review. Hope to contribute more to this awesome repository.

Thanks for your contribution @Ansh-Sarkar . The change looks good, but only needs to be made now to the header file. It will propagate to the doc RST automatically now. That's the source of the merge conflict in this PR. If you merge with the master branch by allowing those two files to be removed, this should be good to merge.

Also, since we just overhauled the API doc build process, and since you are changing API docs by modifying the header, you will now need to update the frozen Python API RST (in docs/api/python/frozen) by running the following build process:

Make sure you have all the documentation dependencies installed on your machine. This includes Doxygen, Python 3, and the Python packages listed in docs/requirements.txt.
Configure and build OCIO from your build directory with:

cmake <path to OCIO root dir>/. -DOCIO_BUILD_DOCS=ON -DOCIO_BUILD_FROZEN_DOCS=ON
make docs -j<thread count>

git add -u and commit -s the newly generated RST files to your branch, and push to your fork to update this PR.

Apologies that your PR got caught in the middle of this all changing. Let me know if you hit any issues and I can help guide you through it. Thanks again.

@michdolan thank you for guiding. will do the necessary changes at the earliest.

@Ansh-Sarkar, our release is approaching fast, do you think you would be able to complete this PR by Friday?

@Ansh-Sarkar, our release is approaching fast, do you think you would be able to complete this PR by Friday?

Sorry @doug-walker will not be able to deliver. Having problems building the project. I apologize for the inconvenience

Sorry @doug-walker will not be able to deliver. Having problems building the project. I apologize for the inconvenience

Should have informed earlier but was trying since a long time. Sorry once again

Should have informed earlier but was trying since a long time. Sorry once again

Hi @Ansh-Sarkar , just checking in to see if you would like to continue working on this change? Or would you like to close the pull request? If you need help building OCIO to update the needed rst files, let us know. Thanks!

Closing this PR for now due to inactivity. Please feel free to reopen if you want to continue the contribution.

cmake <path to OCIO root dir>/. -DOCIO_BUILD_DOCS=ON -DOCIO_BUILD_FROZEN_DOCS=ON
make docs -j<thread count>

When I ran the above commands I faced the following error
CMake Error at CMakeLists.txt:30 (project):
  Running

   'nmake' '-?'

  failed with:

   The system cannot find the file specified

CMake Error: CMAKE_CXX_COMPILER not set, after EnableLanguage
CMake Error: CMAKE_C_COMPILER not set, after EnableLanguage
-- Configuring incomplete, errors occurred!
See also "C:/Users/akhil/Documents/GSoc/OpenColorIO/CMakeFiles/CMakeOutput.log".

@michdolan can you help me with this?


Implement mTLS for call back  @ EFGS level

Despite the specification stating the mTLS for call back is implemented by the http proxy,
it needs to be implemented on the EFGS service level.
The implementation should

use a JKS to store the client certificate (incl. the private key)
implement certificate pinning for the server certificate based on the information stored in the trusted key list (aka white list)

We have to transform our WebClientConfig into a WebClientFactory and create a WebClient for each receiver.
public WebClient getWebClient(PrivateKey privateKey, X509Certificate clientCertificate) {
    HttpClient httpClient = HttpClient.create()
      .secure(sslContextSpec -> sslContextSpec.sslContext(SslContextBuilder
        .forClient()
        .keyManager(privateKey, clientCertificate)
        .build()
      ))
      .tcpConfiguration(tcpClient -> {
          // configure timeout for connection
          tcpClient = tcpClient.option(ChannelOption.CONNECT_TIMEOUT_MILLIS,
      ...

--> https://docs.spring.io/spring/docs/current/spring-framework-reference/web-reactive.html#webflux-client-builder-reactor
--> https://stackoverflow.com/questions/58193135/how-to-add-client-certificates-to-the-spring-webclient


[FLINK-6852] Fix misuse of GCD

Thanks for contributing to Apache Flink. Before you open your pull request, please take the following check list into consideration.
If your changes take all of the items into account, feel free to open your pull request. For more information and/or questions please refer to the How To Contribute guide.
In addition to going through the list, please provide a meaningful description of your changes.

[x] General

The pull request references the related JIRA issue ("[FLINK-XXX] Jira title text")
The pull request addresses only one issue
Each commit in the PR has a meaningful commit message (including the JIRA id)

[ ] Documentation

Documentation has been added for new functionality
Old documentation affected by the pull request has been updated
JavaDoc for public methods has been added

[x] Tests & Build

Functionality added by the pull request is covered by tests
mvn clean verify has been executed successfully locally or a Travis build has passed

thanks for the PR @Xpray
+1 to merge.

+1

merging.


Fix: Portfolio site leads to 404

This PR fixes...
This PR fixes #147. Appends 'https://' to the urls that do not have 'http://' or 'https://' so that the link navigates to the site instead of 404 error page.
What I did...

add libraries folder and utils file
add util function for validating and mutating the url
pass all urls into util function

How to test...

Navigate to http://localhost:8080/about/
Click 'site' for each card that contains a site button
Verify that it navigates to their website and not a 404 error page

Learning
Using git bash to add one directory into another directory
Say you have a file structure as such:
folder1
folder2
folder3

file1
file2

You want to add folder2 to folder1 so that the file structure looks like this:
folder1

folder2
folder3
file1
file2

In the git bash terminal, you would type:
$ mv folder2/ folder1/

@novellac
I would like you to take a look at the util function to see if it covers all the boundaries. I didn't feel it was necessary to check that the url is a string since the author is adding in a url that we are assuming will be a string with a .com at the end.
The next step is to find where to wrap the url with this function once you approve of the function.


Uninstalling the cert-manager Helm chart removes all my Issuers and Certificate/ CertificateRequest CRs

Currently, it is very easy to shoot yourself in the foot. If you install using --set installCRDs=true and uninstall afterwards, Helm will remove the CRDs and all CRs will be garbage collected.
This is not clearly stated in the values.yaml file: https://github.com/cert-manager/cert-manager/blob/master/deploy/charts/cert-manager/values.yaml#L53
A possible solution for this problem would be to add the Helm keep annotation to the CRD resources: https://github.com/cert-manager/cert-manager/pull/5777
TODO: @inteon further investigate and convince other maintainers of this issue.

CertificateRequest CRs

There shouldn't be a need to keep around CertificateRequests unless re-installing mid-issuance and in that case we anyway cannot guarantee that the issuance flow will pick up from exactly where it was before uninstall.


How to setup default binding for third Vive controller?

As title, any solution or work around for that?
I'm expecting something like providing more input sources
"/user/ViveController/0"
"/user/ViveController/1"
"/user/ViveController/2"

Hm, what's your use case for this scenario?  Other people holding them?

To capture MR videos for instance (third comptroller attaches to actual camera) and why not the two use cases that you mentioned ;-) ?   Also note that Vive Trackers can be used as additional controllers as there are also still a number of trackers out there with patched firmware that identify as controllers (initially to do MR video capture).
And "held in hand" should support button action mappings as if they were actual controllers due to accessibility of the pogo pins for equivalent input (for any number of trackers not just two).

This seems like an issue that's still valid but is more of a OpenVR feature. Might be better served at that repo: https://github.com/ValveSoftware/openvr/issues


chore: Add script to identify broken links

Motivation
Broken links in documentation are making it harder for new comers to learn about KServe and often it is not easy to find what the correct link should be. For project maintainers it is time consuming to revisit documentation to identify broken links manually. It certainly does not make the best impression on new comers if they click on links that return 404 errors.
Proposed Changes
Add new script docs/hack/verify-doc-links.py to identify 93 broken links of the currently 1291 links (581 unique URLs) in 153 Markdown files.
I made a "best-effort" attempt to fix as many the broken links identified by the new script. 11 broken links are remaining. I might need some help from the KServe community to address the remaining ones.
Example
$ ./docs/hack/verify-doc-links.py

Checked 1292 links (572 unique URLs) in 153 Markdown files.

docs/modelserving/v1beta1/lightgbm/README.md:33: https://github.com/kserve/kserve/python/lgbserver -> 404
docs/modelserving/v1beta1/serving_runtime.md:34: https://github.com/kserve/kserve/tree/master/python/lightgbm -> 404
docs/sdk_docs/docs/V1alpha1ClusterServingRuntime.md:11: /docs/sdk_docs/docs/.md -> 404
docs/sdk_docs/docs/V1alpha1ServingRuntime.md:11: /docs/sdk_docs/docs/.md -> 404
docs/sdk_docs/docs/V1beta1CustomExplainer.md:24: /docs/sdk_docs/docs/ResourceQuantity.md -> 404
docs/sdk_docs/docs/V1beta1CustomPredictor.md:24: /docs/sdk_docs/docs/ResourceQuantity.md -> 404
docs/sdk_docs/docs/V1beta1CustomTransformer.md:24: /docs/sdk_docs/docs/ResourceQuantity.md -> 404
docs/sdk_docs/docs/V1beta1ExplainerSpec.md:33: /docs/sdk_docs/docs/ResourceQuantity.md -> 404
docs/sdk_docs/docs/V1beta1PodSpec.md:24: /docs/sdk_docs/docs/ResourceQuantity.md -> 404
docs/sdk_docs/docs/V1beta1PredictorSpec.md:33: /docs/sdk_docs/docs/ResourceQuantity.md -> 404
docs/sdk_docs/docs/V1beta1TransformerSpec.md:30: /docs/sdk_docs/docs/ResourceQuantity.md -> 404

ERROR: Found 11 invalid Markdown links

How does the script work

Find all Markdown files in the project
Extract all Markdown-style links and plain URLs
Create unique list of simplified URLs
Use local files to check relative links, use cached HTTP requests for links outside project
Run HTTP requests in parallel
Retry GitHub rate-limited URLs after (at least) one minute
Report broken links with source file and line number

Script pedigree: I contributed several iteration of this script to MLX and KFP-Tekton projects, and most recently to ModelMesh. This latest one differs from earlier version in that it does not require any non-standard libraries like for Markdown or requests to be installed into a virtual environment prior to running the script.
Possible future enhancements
Add a new PR check to run the docs/hack/verify-doc-links.py script with any PR to identify broken links as soon as possible and hopefully have the PR contributor fix the broken link, even if outside the scope of the PR.

/cc @njhill

/cc @yuzisun


fix: scanning qr code closes activity

Fixes #1116
Checklist:

[x] I have checked for PMD and check-style issues
[x] I have read the Contribution & Best practices Guide and my PR follows them.
[x] My branch is up-to-date with the Upstream development branch.

Changes: The scanning functionality is crucial for the app and needs to be fixed before release. Thus it temporarily removes injection for BarcodeModule which needs to be done through a Sub component as it takes in a view param. It will be re-added in a follow up PR.
Screenshots for the change:

@iamareebjamal Please reivew


lwm2m.c: s/osf/zmp/

Have the devices register as 'zmp:imei:xxx' and 'zmp:sn:xxx' instead of using "osf".
This isn't going to break anybody's scripts, is it?

LGTM. Forgot about this one.

Thanks! Merging with my fingers crossed it won't break anything :)


qiime2 use dead python lib

Python3.13 remove a large amount of libraries, one of them is crypt that is used in qiime2/core/tests/test_cache.py:12
Bug originally reported in https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=1084668
Cheers,

Hey @eamanu,
Thanks for letting us know! When we migrate to python 3.13, our integration system will trigger an error with these dead libraries and we'll be able to address it prior to releasing QIIME 2 with python 3.13.
Cheers 🦎

@lizgehret Can this issue be re-opened so that we can track when this is fixed? The Debian issue tracker is setup to check the status of this issue, so it reflecting if a fix is in would be useful to us.
Thank you!


1.7.9 - does not work

Hi. I can’t get plugin to work in current version. I’m using the docker image. I can set a password and enable it but it doesn’t follow through.
Any tips?

Let me see if I can re-create it on my side.

Let me see if I can re-create it on my side.

Wow thats a quick response. Thank you a lot

Just updated my YOURLS to the latest version, and everything worked as intended. Could you probably be a bit more specific as to what's not working?

Okay thats interesting. I'm using the latest docker image and a reverse proxy.
I install the plugin like any other with git clone, enable it in the tools menu of the admin interface, go to the settings area and pick a link and enter a password and make sure 'enabled' is checked in there.
How can i further assist to figure out why it is not working?

What happens after you set the password? Do you get an error? Does the link simply immediately redirect instead of prompting for a password?

Nothing happens - it all looks good but as you say the link will redirect without prompting for a password.
https://link.dahl.vip/public has a pw of ‘test’ right now - as you can see it does not prompt :)

I see. If you give me some time, I'll spin up a docker image and see if I can re-create it there.
However, have you tried looking at any PHP log, or if docker says anything?

I can this from docker container log:
11; CrOS x86_64 13310.59.0) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/85.0.4183.84 Safari/537.36"
[Tue Sep 29 09:39:09.021383 2020] [autoindex:error] [pid 28] [client <IP_ADDRESS>:59378] AH01276: Cannot serve directory /var/www/html/: No matching DirectoryIndex (index.php,index.html) found, and server-generated directory index forbidden by Options directive
<IP_ADDRESS> - - [29/Sep/2020:09:39:09 +0000] "GET / HTTP/1.1" 403 439 "-" "Mozilla/5.0 (Macintosh; Intel Mac OS X 10_15_7) AppleWebKit/605.1.15 (KHTML, like Gecko) Version/14.0 Safari/605.1.15"
<IP_ADDRESS> - - [29/Sep/2020:09:39:10 +0000] "GET /public HTTP/1.1" 200 418 "-" "Mozilla/5.0 (Macintosh; Intel Mac OS X 10_15_7) AppleWebKit/605.1.15 (KHTML, like Gecko) Version/14.0 Safari/605.1.15"
<IP_ADDRESS> - - [29/Sep/2020:09:39:45 +0000] "GET /admin/ HTTP/1.1" 200 3953 "https://REMOVED/" "Mozilla/5.0 (Macintosh; Intel Mac OS X 10_15_7) AppleWebKit/605.1.15 (KHTML, like Gecko) Version/14.0 Safari/605.1.15"

The IP in context is the internal docker ip of my proxy so now i see that YOURLS doesn't respect the headers sent from my reverse proxy. Yikes. I'm by no means an expert in coding so dunno if you can use it for anything

Everything there seems fine, do you notice any difference before and after you try to access the URL with the password feature enabled?
Additionally, can you confirm if any other plugin which alters redirection in any way is also working?

Okay thanks. This is weird. I just disable my other plugins and it now prompts for a pw. I’ve now enabled the other plugins one-by-one to see which one was the course. I have enabled all and it still works.
Sorry to waste your time in this

No worries! Have a great day.


Let query items borrow from query state to avoid needing to clone

Objective
Improve the performance of FilteredEntity(Ref|Mut) and Entity(Ref|Mut)Except.
FilteredEntityRef needs an Access<ComponentId> to determine what components it can access.  There is one stored in the query state, but query items cannot borrow from the state, so it has to clone() the access for each row.  Cloning the access involves memory allocations and can be expensive.
Solution
Let query items borrow from their query state.
Add an 's lifetime to WorldQuery::Item and WorldQuery::Fetch, similar to the one in SystemParam, and provide &'s Self::State to the fetch so that it can borrow from the state.
Unfortunately, there are a few cases where we currently return query items from temporary query states: the sorted iteration methods create a temporary state to query the sort keys, and the EntityRef::components<Q>() methods create a temporary state for their query.
To allow these to continue to work with most QueryData implementations, introduce a new subtrait ReleaseStateQueryData that converts a QueryItem<'w, 's> to QueryItem<'w, 'static>, and is implemented for everything except FilteredEntity(Ref|Mut) and Entity(Ref|Mut)Except.
#[derive(QueryData)] should generate ReleaseStateQueryData implementations, but that is not currently implemented in this PR.
This PR does not actually change the implementation of FilteredEntity(Ref|Mut) or Entity(Ref|Mut)Except!  That will be done as a follow-up PR so that the changes are easier to review.  I have pushed the changes as chescock/bevy#5.
Testing
I ran performance traces of many_foxes, both against main and against chescock/bevy#5, both including #15282.  These changes do appear to make generalized animation a bit faster:
(Red is main, yellow is chescock/bevy#5)

Migration Guide
The WorldQuery::Item and WorldQuery::Fetch associated types and the QueryItem and ROQueryItem type aliases now have an additional lifetime parameter corresponding to the 's lifetime in Query.  Manual implementations of WorldQuery will need to update the method signatures to include the new lifetimes.  Other uses of the types will need to be updated to include a lifetime parameter, although it can usually be passed as '_.  In particular, ROQueryItem is used when implementing RenderCommand.
Before:
fn render<'w>(
    item: &P,
    view: ROQueryItem<'w, Self::ViewQuery>,
    entity: Option<ROQueryItem<'w, Self::ItemQuery>>,
    param: SystemParamItem<'w, '_, Self::Param>,
    pass: &mut TrackedRenderPass<'w>,
) -> RenderCommandResult;

After:
fn render<'w>(
    item: &P,
    view: ROQueryItem<'w, '_, Self::ViewQuery>,
    entity: Option<ROQueryItem<'w, '_, Self::ItemQuery>>,
    param: SystemParamItem<'w, '_, Self::Param>,
    pass: &mut TrackedRenderPass<'w>,
) -> RenderCommandResult;

Methods on QueryState that take &mut self may now result in conflicting borrows if the query items capture the lifetime of the mutable reference.  This affects get(), get_many(), iter(), and iter_many().  To fix the errors, first call QueryState::update_archetypes(), and then replace the calls with the corresponding _manual methods, which take &self.
Before:
let mut state: QueryState<_, _> = ...;
let d1 = state.get(world, e1);
let d2 = state.get(world, e2); // Error: cannot borrow `state` as mutable more than once at a time
println!("{d1:?}");
println!("{d2:?}");

After:
let mut state: QueryState<_, _> = ...;
state.update_archetypes(world);
let d1 = state.get_manual(world, e1);
let d2 = state.get_manual(world, e2);
println!("{d1:?}");
println!("{d2:?}");

IIUC, the lack of ReleaseStateQueryData impls for FilteredEntityRef|Mut means they can no longer be used in the sort methods? This would mean removing support for sorting with untyped QueryData, which is important for some people. Is this change possible without removing that feature?

IIUC, the lack of ReleaseStateQueryData impls for FilteredEntityRef|Mut means they can no longer be used in the sort methods. This would mean removing support for sorting with untyped QueryData, which is important for some people. Is this change possible without removing that feature?

It might be.  The issue is that the L::Items borrow from the state, and they get included in the returned Iterator<Item=Entity>.  But they aren't actually used after the sort method returns.  The simplest fix would be to collect() entity_iter into a Vec to drop the L::Items, but I saw you had tried to avoid that in #13443.
For what it's worth, I don't think FilteredEntityRef|Mut will actually work with the sort() methods right now, which is why I hadn't been worried about it.  They do transmute_filtered::<(L, Entity), F>(), and calling set_access() on (FilteredEntityRef, Entity) doesn't pass it down to the FilteredEntityRef.  Also, transmute_filtered() calls set_access() before update_component_access(), so the access is empty anyway.

IIUC, the lack of ReleaseStateQueryData impls for FilteredEntityRef|Mut means they can no longer be used in the sort methods. This would mean removing support for sorting with untyped QueryData, which is important for some people. Is this change possible without removing that feature?

It might be. The issue is that the L::Items borrow from the state, and they get included in the returned Iterator<Item=Entity>. But they aren't actually used after the sort method returns. The simplest fix would be to collect() entity_iter into a Vec to drop the L::Items, but I saw you had tried to avoid that in #13443.

Yeah, a collect would regress sort performance. That would make this change a perf trade-off, not a pure gain. By how much, would have to be benchmarked.

For what it's worth, I don't think FilteredEntityRef|Mut will actually work with the sort() methods right now, which is why I hadn't been worried about it. They do transmute_filtered::<(L, Entity), F>(), and calling set_access() on (FilteredEntityRef, Entity) doesn't pass it down to the FilteredEntityRef. Also, transmute_filtered() calls set_access() before update_component_access(), so the access is empty anyway.

The tuple behavior is a bug: #14349. Entity, EntityLocation and &Archetype should always be available QueryData, regardless of access.
That access is not propagated to a transmuted FilteredEntityRef|Mut sounds like another bug, does FilteredEntityRef|Mut even work with transmutes in the first place?

Yeah, an additional collect would regress sort performance. That would make this change a perf trade-off, not a pure gain. By how much, would have to be benchmarked.

Yeah, there are definitely tradeoffs here!  Given that sort doesn't actually work with FilteredEntityRef right now, I'd like to leave the ReleaseStateQueryData constraint in place on those methods.  If we decide to merge this and we fix #14349, then we can revisit the constraint.
(It feels like there must be some way to get the best of both options by dropping the L::Items in place before the method returns without allocating a new Vec, but I don't yet see how to make it work.  ManuallyDrop<T> still captures lifetimes.)

That access is not propagated to a transmuted FilteredEntityRef|Mut sounds like another bug, does FilteredEntityRef|Mut even work with transmutes in the first place?

Sorry, I was wrong; it works fine.  I confused the local component_access, which was empty, with self.component_access, which gets passed to set_access.

To note, I think super let could also solve this kind of problem, once we get some form of it in Rust.

note, I think super let could also solve the kind of problem this PR tackles, once we get some form of it in Rust.

I'm not sure how this would help avoiding to clone the Access in FilteredEntityRef. Do you have some examples?


Add docs for CLI Options with tests

:memo: :sparkles: Add docs for CLI Options with tests.

Codecov Report

Merging #12 into master will increase coverage by 6.37%.
The diff coverage is 98.9%.

@@            Coverage Diff             @@
##           master      #12      +/-   ##
==========================================
+ Coverage   72.94%   79.32%   +6.37%
==========================================
  Files          17       29      +12
  Lines         621      798     +177
==========================================
+ Hits          453      633     +180
+ Misses        168      165       -3

Impacted Files
Coverage Δ

typer/main.py
49.51% <ø> (+0.63%)
:arrow_up:

typer/models.py
99% <ø> (+0.95%)
:arrow_up:

typer/completion.py
66.66% <0%> (-3.34%)
:arrow_down:

docs/src/options/tutorial002.py
100% <100%> (ø)

docs/src/options/tutorial004.py
100% <100%> (ø)

...sts/test_tutorial/test_options/test_tutorial005.py
100% <100%> (ø)

...sts/test_tutorial/test_options/test_tutorial004.py
100% <100%> (ø)

docs/src/options/tutorial003.py
100% <100%> (ø)

docs/src/options/tutorial006.py
100% <100%> (ø)

docs/src/options/tutorial001.py
100% <100%> (ø)

... and 18 more

Continue to review full report at Codecov.

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


[KOGITO-3495] Error handling on RestworkItemhandler

If call fails, event will be null, then exception is thrown

jenkins retest this

jenkins retest this

jenkins retest this

jenkins retest this

jenkins retest this

Including @evacchi since he were working with futures and this might interest him
Since RestWorkItemHandler is implemented asynchronously, throwing an exception when the rest invocation fails will only makes the vertx thread where the hander is invoked "angry", so I think the best we can do is to abort the work item asyncronously, in the same way we are completing it asyncrhonously when the call succeed.
In any case, the process will continue and, in the example case (two consecutive rest invocations, where the second call depends on data retrieved by first one), since the first call failed to provide required data, the second call will fail.
Please note the failure is not reported in the post call that initiate the process. User will have to query the process ide for resolving the status


jsPDF object returns irrelevant data

I have the following snippet of code:
            const data = document.getElementById('content');
            html2canvas(data).then(canvas => {
                // Few necessary setting options
                const imgWidth = 208;
                const pageHeight = 295;
                const imgHeight = canvas.height * imgWidth / canvas.width;
                const heightLeft = imgHeight;
                const contentDataURL = canvas.toDataURL('image/jpeg');
                const pdf = new jsPDF('p', 'mm', 'a4'); // A4 size page of PDF
                const position = 0;
                pdf.addImage(contentDataURL, 'PNG', 0, position, imgWidth, imgHeight)
                pdf.save('MYPdf.pdf'); // Generated PDF
            });

The problem is in new jsPDF('p', 'mm', 'a4'); , when I debug in version 1.4.1 I get this object in debug -> https://ibb.co/r6PPb7N. But when I choose version 1.5.2, in debug this is the object I get -> https://ibb.co/n8cKCrd. Can someone please investigate this issue? Thanks
Note:

In version 1.4.1 everything is ok and the document is downloaded. In 1.5.2 it crashes due to the provided problem.
Maybe this issue is fixed in 1.5.3, but due to https://github.com/MrRio/jsPDF/issues/2437 issue, the only last working version is 1.4.1
Tested with 1.5.1 and the problem is the same as in 1.5.2

And 1.5.3?

Due to https://github.com/MrRio/jsPDF/issues/2437 issue, I can not try with 1.5.3. Can you please investigate that issue so that I can try with 1.5.3 version and update the status of this one?

With the fix of #2437 issue, this bug doesn't reproduce anymore. @arasabbasi thank you


thread '' panicked at 'no entry found for key'

thread '<main>' panicked at 'no entry found for key', /home/rustbuild/src/rust-buildbot/slave/nightly-dist-rustc-linux/build/s
rc/libcore/option.rs:330                                                                                stack backtrace:
   1:     0x7f33a8259230 - sys::backtrace::write::hb7fa0f4f6b33ee3a8Rt
   2:     0x7f33a827acd0 - failure::on_fail::h4388493538a5ad8ck8z
   3:     0x7f33a81e9880 - rt::unwind::begin_unwind_inner::h644ddf1c409df284cNz
   4:     0x7f33a81ea3b0 - rt::unwind::begin_unwind_fmt::h872fa98dc2f66468JLz
   5:     0x7f33a827ab30 - rust_begin_unwind
   6:     0x7f33a82c3680 - panicking::panic_fmt::h4359f3a82e34f47bvym
   7:     0x7f33a89d3440 - option::Option<T>::expect::h13725484968495965889
   8:     0x7f33a89d33c0 - collections::hash::map::HashMap<K, V, S>.IndexMut<Q>::index_mut::h6095433997224322788
   9:     0x7f33a89d2140 - cli::generate_contact::hfcebf25496e6fb0dfwa
  10:     0x7f33a89cd2b0 - cli::add_contact::h4c4cd0e71f37d5c32ra
  11:     0x7f33a89c5820 - cli::cli_main::h513d87d819442dddmha
  12:     0x7f33a89b3670 - main::h3ffbcf7ffbb59354gaa
  13:     0x7f33a82e1e80 - rust_try_inner
  14:     0x7f33a82e1e70 - rust_try
  15:     0x7f33a827c930 - rt::lang_start::h46417f3fa3eb30a5w2z
  16:     0x7f33a89b36a0 - main
  17:     0x7f33a7492f50 - __libc_start_main
  18:     0x7f33a89b3540 - <unknown>
  19:                0x0 - <unknown>

#885 possibly related. I have no idea what went wrong. Code that triggered this looks like this:
my_hashmap: HashMap<String, String> = HashMap::new();
my_hashmap["FOO".to_string()] = ...

Do you have a project that I could repro this issue with?

I'll try to trim down the code a bit... the LOC are several thousands. This might take a while though.

Sure! The manifest is normally enough to repro as well.
(also be sure you're using an up-to-date cargo)

Whoa, that was easy:
use std::collections::HashMap;

fn main() {
    let mut hm: HashMap<String, String> = HashMap::new();
    hm["LOL".to_string()] = "HAHA".to_string();
}

$ rustc --version
rustc 1.0.0-nightly (3d0d9bb6f 2015-01-12 22:56:20 +0000)

FWIW this is ArchLinux running on rust-nightly-bin from the AUR.

Oh! Sorry this is actually the bug tracker for Cargo issues, so I mistakenly thought that the backtrace was from Cargo itself :).
What you've got is actually expected behavior. Currently the IndexMut on HashMap (which the []= syntax is using) looks up the key first and then sets the value. In this case you're seeing the panic because the lookup is failing due to the key not being present. For insertion you'll currently want to use .insert instead of the overloaded syntax.

Ouch... I didn't look myself, sorry.

No worries!


Add tooltips to all icon buttons

At the moment, only the icon buttons in the sidebar have a tooltip. The Send button even uses a title at the moment instead of a tooltip.
I tried to quickly add tooltips to all the icon buttons that did not have one as part of #303, but it turned out to be a bit more tricky than just adding the .tooltipped* classes / wrapping <span> elements so I gave up...

@xPaw, do you think you could help with this?

You would need to wrap the buttons into span to add the tooltip (due to css wonkiness).


Treemap is not calculating color based on the range provided

I am trying to vary the color of treemap from red(incase value is negative) to green(incase value is positive) based on a parameter TotalBaseMTD. In a few scenarios, random color gets picked up.
A scenario that I have tested for:
Input:
when TotalBaseMTD value is +ve for all
Expected Output:
All green squares
Obtained Output:
a blue color square and a red color square
Attached is the sample code:
http://jsfiddle.net/Evw3x/14/
I have seen a similar issue in grouped treemap too.

The color "range" is used for scales that span from negative to positive, while the color "heatmap" is used to scales with all positive values. You just need to override the "heatmap" colors as well as the "range" colors: http://jsfiddle.net/Evw3x/15/

Hi Dave,
I have tried that, it helps to stick within the range but gives the wrong color. In the fiddle that you have posted too , both the squares should be green as color parameter (TotalBaseMTD) has a positive value for both squares.

I'm not sure I understand what the expected behavior is here.
When mapping 2 values to a heatmap, the smaller value will take the first color in the heatmap, and the larger value will take the last color.


Should explain prerequisites early in the article

You should explain the prerequisites early in the article. I have failed to enable this addon on my existing cluster due to a lack of aks-preview extension. Thanks.

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

ID: edf448c9-4082-1124-0a41-f09f1771fe5d
Version Independent ID: 95abeb00-0d5a-6e9a-bc4e-24f5dcc6ddba
Content: Open Service Mesh (Preview) - Azure Kubernetes Service
Content Source: articles/aks/servicemesh-osm-about.md
Service: container-service
GitHub Login: @mlearned
Microsoft Alias: pgibson

@ToruMakabe Thanks for the feedback! We will review and update accordingly.

@mlearned @zr-msft
Can you please check and add your comments on this doc update request as applicable.


Change NSDocumentDirectory to NSCachesDirectory

I moved to the cache directory not to list files in the Files app.

I hope this PR be approved as soon as possible. This PR might solves the problem of files been visible on IOS Files

Any updates @CrossPT?


[RFC00102] Dockerize remote machines #11

You want to make a PR to Jako
Thanks so much :) First, please take a moment to carefully check through
the below items:
Sanity

[x] I'm aware of the implications of the proposed changes
[x] Code is PEP8
[ ] I've changed the version in setup.py based on Semantic Versioning

Docs

[ ] Docs are updated
[ ] Docs version is correct on (index.html and _coverpage.md)

Tests

[ ] Changes have gone through actual use testing
[ ] Tests have been updated to reflect the changes

Hello @abhijithneilabraham! Thanks for updating this PR.

In the file jako/__init__.py:

Line 1:1: F401 '.distribute.DistributedScan.DistributedScan' imported but unused
Line 2:1: F401 '.distribute.RemoteScan.RemoteScan' imported but unused
Line 3:1: F401 '.docker.docker_pull.docker_pull' imported but unused
Line 4:5: F821 undefined name 'distribute'
Line 4:15: W292 no newline at end of file

In the file jako/docker/docker_pull.py:

Line 147:22: E226 missing whitespace around arithmetic operator
Line 147:27: E226 missing whitespace around arithmetic operator
Line 147:33: E226 missing whitespace around arithmetic operator
Line 193:29: E226 missing whitespace around arithmetic operator


Support alternative BCHD installation (as an option)

https://grpc.fabien.cash/

Please check #16


add fully-qualified smiling face emoji

Defined in Unicode 13.0: https://unicode.org/Public/emoji/13.0/emoji-test.txt
Copy 263a.svg to 263a-fe0f.svg. Copy 263a.png to 263a-fe0f.png.

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

 All committers have signed the CLA.


Group Object

I have noticed that the Groups seems to work in a fashion I am not familiar with. When I ask if a user is an admin Sentry responds in the affirmative if they are an admin in any group. Clearly, this is almost useless as I cannot seem to ask if the user is an admin in a particular group.
From the schema it seems that properties that should be associated with the user group object are only associated with the group object. Is this by design?
For instance: two users are in the system, we want both to be in the Group 1, but one should be admin while the other should not be. So, one user is level 1 while the other user is level 10.

What you are asking for is coming soon and will be known as user roles in sentry 1.2 . when this happens you should have a complete ACL'esque authentication system.

Thank you, is there an ETA? We love what you guys are doing!
We began to modify what we currently have, but we would rather use 1.2.

I have it scheduled for some time in late march but its not solid unfortunately. However we do have a developer program starting up soon where those studios or developers interested in contributing to the Cartalyst system can get involved. Anyone who contributes significantly to Cartalyst before RC1 will gain lifetime rights and privileges to the Cartalyst developer subscription package. I'll have info on that out in the next week or two.

My team at Clevyr is very interested in that program. For now we are going to roll our own ACL and perhaps move into your system when it is available.

re-opening and making a note that I need to add in an is_group_admin method.

Is there already a method to determine the level within the group as well?

Probably not.  I will look into adding that in as well.. I need to take a good look at groups as I didn't develop it, but it seems as if some important features are missing.


More network requirement

[Enter feedback here]
I couldn’t get lists of ADF when I create connections to ADF.
If management.azure.com was allowed on NSG, I could get lists of ADF.
So not only AAD but also management.azure.com are required, I think.
But I couldn’t find the above information from Docs.
This requirement should be described in Docs too, I think.
Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: f3f8ffec-fce7-a800-f408-19a9d4b957d1
Version Independent ID: 631029a7-4b2e-d92d-599b-23e261d5c641
Content: Connect privately and securely to your Purview account - Azure Purview
Content Source: articles/purview/catalog-private-link-account-portal.md
Service: purview
Sub-service: purview-data-catalog
GitHub Login: @viseshag
Microsoft Alias: viseshag

@yuharucan Thank you for sharing feedback. We are actively validating it. We will share updates soon.

@viseshag Could you please validate customer feedback and enhance doc as appropriate. Thank you.

@yuharucan - Thanks for your feedback! We are confirming and will update the doc accordingly.

We have listed DataFactory (as service tag) and *.frontend.clouddatahub.net as required endpoints:
https://docs.microsoft.com/en-us/azure/purview/concept-best-practices-security#use-network-security-groups
https://docs.microsoft.com/en-us/azure/purview/manage-integration-runtimes#networking-requirements

#please-close


"Create new note" create not 1 but 8 notes in a row

Using CloudNotes 0.8.0 on Yosemite, clicking the "+" button to create a new note (or "file">"New" for that matter) create not one note, but 8 notes in a row !
Well, I can then delete the 7 other notes, but I assume it's not the usual behavior ?
I'm using OC 7.0.4 and Notes app 0.8.

Same issue with 8.0.3 OwnCloud and 0.9 notes.

Using ownCloud 8.1.1 and 0.8 notes, Yosemite. The number of random note creations varies, er, randomly.

The issue has been resolved in the iOS app. I will include it in the OS X app when I find time, otherwise contributions are welcome.

Any chance this will be addressed in the near term?


port binding error on OpenShift

deployment.yaml specifies port 443. This results in a binding error on OpenShift. It is better to use a port like 9443. The container already accepts argument --secure-port

issue fixed.


Route Not defined

When I put the code in Web.php show this error,

I do the publish and is not working yet
• App\Http\Controllers\Victorybiz\LaravelCryptoPaymentGateway\src\Http\Controllers\CryptoPaymentController was not found: Controller class App\Http\Controllers\Victorybiz\LaravelCryptoPaymentGateway\src\Http\Controllers\CryptoPaymentController for one of your routes was not found. Are you sure this controller exists and is imported correctly?
I neeed publish the files manually?

Sorry, was my bad. Laravel 8 is with route namespace by default, so you need put in null if you make a upgrade from other version

Sorry, was my bad. Laravel 8 is with route namespace by default, so you need put in null if you make a upgrade from other version

I got stuck here, but did not understand what you meant by ' so you need put in null'. With some research I realized that in App/Providers/RouteServiceProvider.php , I had to change protected $namespace = 'App\Http\Controllers'; to protected $namespace = null;
That solved it for me. Thank you


configure: test -a|o is not POSIX

Same as https://github.com/linux-pam/linux-pam/pull/311 and https://github.com/karelzak/util-linux/pull/1216.
I had test: too many arguments when building dhcpcd using sbase.
Please check that I broke nothing.
Thank you.

LGTM, needs testing on OpenBSD as that builds statically by default.


[ThreadPool] Run a thread per CPU in SimpleExecutor

Why
Now we don't bind threads in ThreadPool to physical CPU cores. So we might not be able to get the expected concurrency.
What is changing
This one should be transparent for users.

Do you compare the performance using this improvement? @ChuanqiXu9

Do you compare the performance using this improvement? @ChuanqiXu9

Yeah, I found this one when looking at https://github.com/alibaba/async_simple/pull/99

LGTM


Print default when CUDA and CuDNN versions are not specified

Fix #10456.

@tensorflow-jenkins test this please

I am OOO, but @caisq should be a better reviewer than me for any bash code.

@tensorflow-jenkins test this please

@tensorflow-jenkins test this please

the xla failure is irrelevant, here is a passing run https://ci.tensorflow.org/job/tensorflow-pull-requests-xla/1743/


[WIP] Feature/mui 4

Update to material ui 4, but not yet tested, still a few things to udpate.

[X] Test on Storybook
[X] Basic update following the doc
[x] Check SSR, JSSProvider does not exist anymore, use StylesProvider instead
[ ] Check the Composition Guide for dialogs, modals https://material-ui.com/guides/composition/#caveat-with-refs
[ ] Test on non-trivial app, debug, clean warning
[ ] Update versions in the doc

Migration guide: https://material-ui.com/guides/migration-v3/
@marieqg @juliensl @ErikDakoda

It seems mostly functional, I will test it a bit on Live for good before merge.
The SSR API changed slightly. It uses the ServerStylesheets object which setup the StylesProvider for you, which itselfs replaces the JSSProvider with a similar API.
It needs a bit of testing, I am not 100% sure there is no regression.
Otherwise it's ok, there are no big changes apart those listed in the migration doc. I've tried to update components as much as possible in Vulcan, the same needs to be done in downstream apps.

Sounds great, @eric-burel! Fast work!


getting error while installing npm  some

getting error while installing npm  some

Have you tried deleting the package-lock.json and node_modules then run npm install again?

Yah I tried  but till getting
On Sun, 25 Oct 2020, 05:47 Julius Guevarra<EMAIL_ADDRESS>wrote:

Have you tried deleting the package-lock.json and node_modules then run
npm install again?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/jgudo/ecommerce-react/issues/18#issuecomment-716072511,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ANANKOUQQZO4ZXC27A42JO3SMNVBFANCNFSM4S5WDWHA
.

Even after cleaning the npm cache? npm cache clean --force

This process also done but not worked till getting same to same error
On Mon, 26 Oct 2020, 15:54 Julius Guevarra<EMAIL_ADDRESS>wrote:

Even after cleaning the npm cache? npm cache clean --force
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/jgudo/ecommerce-react/issues/18#issuecomment-716453897,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ANANKOQD45EK4Z5FTSJAHWDSMVE4RANCNFSM4S5WDWHA
.

What version of npm are you using?
Try again deleting node_modules + package-lock.json + running npm cache clean --force. If the problem still exists, try reinstalling/update node/npm. I'm going to close this since I can't reproduce the problem and it seems that problem is in your end.


No config flow ?¿

Describe the bug
added the  integration through hacs, but when trying to add it in HA  through integrations , it says error in config flow...
Expected behavior
opening config UI
Environment details:
TV (QE55Q64RALXXN)
HA latest version
running on ARM CPU
Output of HA logs
Logger: homeassistant.util.package
Source: util/package.py:96
First occurred: 14:44:53 (3 occurrences)
Last logged: 14:50:26
Unable to install package websocket-client==1.0.1: ERROR: Could not install packages due to an EnvironmentError: [Errno 22] Invalid argument: 'pycache' WARNING: You are using pip version 20.2.4; however, version 21.1.2 is available. You should consider upgrading via the '/usr/local/bin/python3 -m pip install --upgrade pip' command.
Logger: aiohttp.server
Source: requirements.py:163
First occurred: 14:44:53 (3 occurrences)
Last logged: 14:50:26
Error handling request
Traceback (most recent call last):
File "/usr/local/lib/python3.8/site-packages/aiohttp/web_protocol.py", line 422, in _handle_request
resp = await self._request_handler(request)
File "/usr/local/lib/python3.8/site-packages/aiohttp/web_app.py", line 499, in _handle
resp = await handler(request)
File "/usr/local/lib/python3.8/site-packages/aiohttp/web_middlewares.py", line 119, in impl
return await handler(request)
File "/usr/src/homeassistant/homeassistant/components/http/security_filter.py", line 60, in security_filter_middleware
return await handler(request)
File "/usr/src/homeassistant/homeassistant/components/http/forwarded.py", line 76, in forwarded_middleware
return await handler(request)
File "/usr/src/homeassistant/homeassistant/components/http/request_context.py", line 24, in request_context_middleware
return await handler(request)
File "/usr/src/homeassistant/homeassistant/components/http/ban.py", line 78, in ban_middleware
return await handler(request)
File "/usr/src/homeassistant/homeassistant/components/http/auth.py", line 144, in auth_middleware
return await handler(request)
File "/usr/src/homeassistant/homeassistant/components/http/view.py", line 135, in handle
result = await result
File "/usr/src/homeassistant/homeassistant/components/config/config_entries.py", line 128, in post
return await super().post(request)
File "/usr/src/homeassistant/homeassistant/components/http/data_validator.py", line 63, in wrapper
result = await method(view, request, *args, **kwargs)
File "/usr/src/homeassistant/homeassistant/helpers/data_entry_flow.py", line 72, in post
result = await self._flow_mgr.async_init(
File "/usr/src/homeassistant/homeassistant/data_entry_flow.py", line 153, in async_init
flow, result = await task
File "/usr/src/homeassistant/homeassistant/data_entry_flow.py", line 171, in _async_init
flow = await self.async_create_flow(handler, context=context, data=data)
File "/usr/src/homeassistant/homeassistant/config_entries.py", line 693, in async_create_flow
await async_process_deps_reqs(self.hass, self._hass_config, integration)
File "/usr/src/homeassistant/homeassistant/setup.py", line 374, in async_process_deps_reqs
await requirements.async_get_integration_with_requirements(
File "/usr/src/homeassistant/homeassistant/requirements.py", line 83, in async_get_integration_with_requirements
await _async_process_integration(hass, integration, done)
File "/usr/src/homeassistant/homeassistant/requirements.py", line 99, in _async_process_integration
await async_process_requirements(
File "/usr/src/homeassistant/homeassistant/requirements.py", line 163, in async_process_requirements
raise RequirementsNotFound(name, [req])
homeassistant.requirements.RequirementsNotFound: Requirements for samsungtv_smart not found: ['websocket-client==1.0.1'].

Issue here is not in integration but a problem in your HA environment to download the correct library version.
What version of HA are you using? What is your OS?

Issue here is not in integration but a problem in your HA environment to download the correct library version.
What version of HA are you using?
You need to investigate and fix this:
Unable to install package websocket-client==1.0.1: ERROR: Could not install packages due to an EnvironmentError: [Errno 22] Invalid argument: 'pycache'
Not an integration issue.

Closing this issue due to inactivity from long time and because not related to integration.


Port conflict for Storm and Spark

I believe the Storm UI is on the same port as Spark uses, and should be changed.

This was intended since I wanted to follow what Apache has for default. I figured it would be the users responsibiility to change this.


Add dapr inject command

Description
Add a command to automatically inject the dapr annotations into a Kubernetes resource configuration as a low friction way to add dapr to existing resources.
This implementation is inspired by the linkerd CLI but has different behavior and less features.
Currently the command is just a little helper to work with the sidecar-injector (only adds annotations) - in theory we could pull out some of the sidecar patching logic and use this as a way to create static manifests for deployment that won't need the sidecar-injector but that is beyond the scope of this PR.
Usage
Inject dapr annotations into a Kubernetes configuration. Supported platforms: Kubernetes

Usage:
  dapr inject [flags] CONFIG-FILE

Examples:

# Inject the first deployment found in the input
kubectl get deploy -l app=node -o yaml | dapr inject - | kubectl apply -f -

# Inject multiple deployments by name in a chain
kubectl get deploy -o yaml | dapr inject -r nodeapp - | dapr inject -r pythonapp | kubectl apply -f -

# Inject deployment in a specific namespace from file or directory by name
dapr inject -r nodeapp -n namespace mydeployment.yml | kubectl apply -f -

# Inject deployment from url by name
dapr inject -r nodeapp --log-level debug https://raw.githubusercontent.com/dapr/quickstarts/master/hello-kubernetes/deploy/node.yaml | kubectl apply -f -

Flags:
      --api-token-secret string         The secret to use for the API token
  -a, --app-id string                   The app id to inject
      --app-max-concurrency int         The maximum number of concurrent requests to allow (default -1)
  -p, --app-port int                    The port to expose the app on (default -1)
      --app-protocol string             The protocol to use for the app
      --app-ssl                         Enable SSL for the app
      --app-token-secret string         The secret to use for the app token
  -c, --config string                   The config file to inject
      --cpu-limit string                The CPU limit to set
      --cpu-request string              The CPU request to set
      --dapr-image string               The image to use for the dapr sidecar container
      --enable-debug                    Enable debug
      --enable-metrics                  Enable metrics
      --enable-profile                  Enable profiling
      --env string                      Environment variables to set (key value pairs, comma separated)
      --graceful-shutdown-seconds int   The number of seconds to wait for the app to shutdown (default -1)
  -h, --help                            help for inject
      --http-stream-request-body        Enable streaming request body for HTTP
      --listen-addresses string         The addresses to listen on
      --liveness-probe-delay int        The delay to use for the liveness probe (default -1)
      --liveness-probe-period int       The period to use for the liveness probe (default -1)
      --liveness-probe-threshold int    The threshold to use for the liveness probe (default -1)
      --liveness-probe-timeout int      The timeout to use for the liveness probe (default -1)
      --log-level string                The log level to use
      --max-request-body-size int       The maximum request body size to use (default -1)
      --memory-limit string             The memory limit to set
      --memory-request string           The memory request to set
      --metrics-port int                The port to expose the metrics on (default -1)
  -n, --namespace string                The namespace the resource target is in (can only be set if --resource is also set)
      --readiness-probe-delay int       The delay to use for the readiness probe (default -1)
      --readiness-probe-period int      The period to use for the readiness probe (default -1)
      --readiness-probe-threshold int   The threshold to use for the readiness probe (default -1)
      --readiness-probe-timeout int     The timeout to use for the readiness probe (default -1)
  -r, --resource string                 The resource to target for injection

Global Flags:
      --log-as-json   Log output in JSON format

Issue reference
We strive to have all PR being opened based on an issue, where the problem or feature have been discussed prior to implementation.
This implementation is based on the linkerd implementation although has different behavior and only adds annotations.
Please reference the issue this PR will close: #870
Checklist
Please make sure you've completed the relevant tasks for this PR, out of the following list:

[x] Code compiles correctly
[x] Created/updated tests
[ ] Extended the documentation

@jjcollinge the Dapr app-id annotation is required for Dapr - if I understand correctly, the current implementation will not set an app-id unless specified explicitly which will result in a failed deployment.
this raises the question of whether we require an app-id flag or default to a value (and if so, what would it be).

@yaron2 yes you are correct. I see 3 plausible options:

if the appId is not provided and not present then error. This is a more clear behaviour for the user but does require additional effort on their behalf.

if the appId is not provided and not present then we generate a new one using a "unique" name generator similar to docker. This is lower friction for the user but will mean the appId is less meaningful i.e. nodeapp vs irratic_moose and unlike docker a Dapr app id is used by clients so should include some identifiable information.

If the appId is not provided and not present then use some data from the manifest such as metadata.name. This is probably the most intuitive, however, we could see conflicts if multiple kinds have the same name. So we'd have to append the kind e.g. nodeapp_deployment, nodeapp_job, etc.

I'd think option 3 will probably give the best user experience. @yaron2  WDYT?

@yaron2 I went ahead and implemented option 3 just to test it out. This testdata shows you what you'd expect from a Pod definition with an autogenerated app id.

@yaron2 I went ahead and implemented option 3 just to test it out. This testdata shows you what you'd expect from a Pod definition with an autogenerated app id. I also added the ability to target a specific resource in a specific namespace (in case you have duplicate names). However, this doesn't allow you to specify a specific kind of resource with a specific name in a specific namespace as I think it's pretty unlikely you have that level of conflict in your K8 manifests.

I think option 3 makes sense, but in documentation we should be explicit about providing an app-id, and if we default to the resource name we should clearly print out the app-id.

@yaron2 I went ahead and implemented option 3 just to test it out. This testdata shows you what you'd expect from a Pod definition with an autogenerated app id. I also added the ability to target a specific resource in a specific namespace (in case you have duplicate names). However, this doesn't allow you to specify a specific kind of resource with a specific name in a specific namespace as I think it's pretty unlikely you have that level of conflict in your K8 manifests.

I think option 3 makes sense, but in documentation we should be explicit about providing an app-id, and if we default to the resource name we should clearly print out the app-id.

Yh that makes sense. Not sure we can print the generated app-id clearly as the command is intended to output to stdout the updated YAML. We could potentially write it to stderr but it wouldn't be clear to the user. Guess we'll just have to document this as clearly as possible.

I was thinking maybe this command should actually be dapr annotate rather than dapr inject. At a later date if we add support to do the actual sidecar injection via the CLI then we could use dapr inject for that as it'd be more accurate. There will be a bunch of shared code between the two commands which will need factoring out but not everything that is configurable on daprd is settable via annotations so we might end up in a difficult position. Alternatively we keep it as a single command and just use flags to drive the behaviour. @yaron2 WDYT?

nudge @yaron2 ^

@jjcollinge Can you resolve the conflicts in the PR?
cc @yaron2 Can you review this PR?

@mukundansundar yep np, please assign me.

@jjcollinge please rebase and fix conflicts ...

@mukundansundar I'm not familiar with this error - any ideas?

@jjcollinge I am also not familiar with this error ... Can you fix the go.mod file and rebase to master and see if the error occurs again?

@yaron2 Can you review this PR?


Missing option to choose between mouse and touch

I'm missing a way to only enable scrollbooster for mouse or touch, currently I use the shouldScroll callback to only make it work with mouse events:
//Only use scrollbooster for mouse interaction
shouldScroll: (data, event) => {
        const isTouch = !!(event.touches && event.touches[0]);
        return !isTouch;
}

Thank for suggestion, added pointerMode option


Switch over to use the common lib api for app engine deployment operations

Adding this for discussion. We may want to hold off on this until its farther along.

The answer is yes.


Installation Failed at the<EMAIL_ADDRESS>install script.

I can't install on linux server. Can you please help me?
balckets@nlss9 [~]# npm install -g better-sqlite3

<EMAIL_ADDRESS>install /home/balckets/lib/node_modules/better-sqlite3/node_modules/integer
node tools/install

make: Entering directory /home/balckets/lib/node_modules/better-sqlite3/node_modules/integer/build'   CXX(target) Release/obj.target/integer/src/integer.o cc1plus: error: unrecognized command line option "-std=c++11" make: *** [Release/obj.target/integer/src/integer.o] Error 1 make: Leaving directory /home/balckets/lib/node_modules/better-sqlite3/node_modules/integer/build'
gyp ERR! build error
gyp ERR! stack Error: make failed with exit code: 2
gyp ERR! stack     at ChildProcess.onExit (/home/balckets/nodejs/lib/node_modules/npm/node_modules/node-gyp/lib/build.js:258:23)
gyp ERR! stack     at emitTwo (events.js:126:13)
gyp ERR! stack     at ChildProcess.emit (events.js:214:7)
gyp ERR! stack     at Process.ChildProcess._handle.onexit (internal/child_process.js:198:12)
gyp ERR! System Linux 2.6.32-773.26.1.lve1.4.46.el6.x86_64
gyp ERR! command "/home/balckets/bin/node" "/home/balckets/nodejs/lib/node_modules/npm/node_modules/node-gyp/bin/node-gyp.js" "rebuild"
gyp ERR! cwd /home/balckets/lib/node_modules/better-sqlite3/node_modules/integer
gyp ERR! node -v v8.9.4
gyp ERR! node-gyp -v v3.6.2
gyp ERR! not ok
npm ERR! code ELIFECYCLE
npm ERR! errno 1
npm ERR<EMAIL_ADDRESS>install: node tools/install
npm ERR! Exit status 1
npm ERR!
npm ERR! Failed at the<EMAIL_ADDRESS>install script.
npm ERR! This is probably not a problem with npm. There is likely additional logging output above.
npm ERR!


Working selector and defaults for InputLiteral using common reducers

On the way to finishing ticket #412, this PR uses Redux to set defaults based on Resource Template ID and URI of property using more generic reducers and a single getProperty selector to map the redux state to the InputLiteral's props.
Fixes #390
Fixes #391
Fixes #428

Coverage decreased (-0.8%) to 76.459% when pulling 96cfebbcd0978618be4ba0f2ae193e2ffb198556 on inputliteral-selector into 949781d0078ed1e27979ef44d791e9fdda12797f on master.

thanks for changing those test descriptions - I much prefer the new ones;  hope it wasn't too time consuming.  My question about that one test still stands -- is it testing what it says it's testing?
I'll leave it up to osh to merge;  I'm not going to the mat over my remaining question.