Getting celery beat error 'SchedulingError: Couldn't apply scheduled task'

Hi,
Thank you for writing a useful project for celery.
I am attempting to use this with scheduled tasks that run every minute that are configured in settings.py using celery beat config like below:
CELERYBEAT_SCHEDULE = {
    'check_products': {
      'task': 'com.ixlayer.tasks.check_products', 'schedule': timedelta(minutes=1),
    },
}

However sometimes I get an exception in the process that runs celery beat.
Traceback (most recent call last):
  File "/Users/kowsari/workspace/ixlayer-be/venv/lib/python2.7/site-packages/celery/beat.py", line 222, in apply_entry
    result = self.apply_async(entry, producer=producer, advance=False)
  File "/Users/kowsari/workspace/ixlayer-be/venv/lib/python2.7/site-packages/celery/beat.py", line 328, in apply_async
    entry, exc=exc)), sys.exc_info()[2])
  File "/Users/kowsari/workspace/ixlayer-be/venv/lib/python2.7/site-packages/celery/beat.py", line 320, in apply_async
    **entry.options)
  File "/Users/kowsari/workspace/ixlayer-be/venv/lib/python2.7/site-packages/celery_once/tasks.py", line 99, in apply_async
    raise e
SchedulingError: Couldn't apply scheduled task check_products:

I am using this configuration.
app.conf.ONCE = {
  'backend': 'celery_once.backends.Redis',
  'settings': {
    'url': settings.BROKER_URL,
    'default_timeout': 60 * 60
  }
}

Is this celerybeat getting the AlreadyQueued exception? Or is this something else?
Help is much appreciated.
Thanks

Hey @kowsari
This is probably something else! We currently don't have any tests for scheduled tasks, so most likely this is a bug with celery_once.
To help debug this further, what version of celery are you using?

Thanks @cameronmaske,
I am using Celery 4.1.0.
So I added a base=QueueOnce to my tasks which makes the problem go away, which makes sense to me. This could be the way to avoid the SechedulingErors.
But the problem that I have now sometimes (which I am not sure it is related to this) is that sometimes when my worker and beat starts...beat send the tasks, but they never get picked up by the workers. If I change the code and remove the QueueOnce then the workers pickup the tasks.
Thanks for your help.!

I had a very similar problem with periodic task, and solution was to add graceful=True to its options:
@shared_task(base=QueueOnce, once={'graceful': True})
def my_task():
    ...

In my case AlreadyQueued exception was invisible due to logging config, so everything looked like a worker not picking up tasks.

recently, we met a very similar problem with periodic task. we use redis backend and redis-once.
In my case, when celery beat scheduled a task, it published a message to redis. But in some reasons, redis closed the connection, so it raised an Exception: SchedulingError: Couldn't apply scheduled task.
Actually, the message had not been published to redis, but celery-once had set a lock for the task, and it isn't released until the default timeout has expired. So celery beat would't send another task, and everything just looked like celery worker not picking up tasks.
So any solutions for this problem?  Help is much appreciated.

I have very similar issue. It looks like celery beat can not connect to redis. I'm able to connect to redis from the same container and save data in it.
celerybeat_1      | Traceback (most recent call last): celerybeat_1      |   File "/usr/local/lib/python3.7/site-packages/redis/connection.py", line 665, in send_packed_command celerybeat_1      |     self._sock.sendall(item) celerybeat_1      | ConnectionResetError: [Errno 104] Connection reset by peer celerybeat_1      |  celerybeat_1      | During handling of the above exception, another exception occurred: celerybeat_1      |  celerybeat_1      | Traceback (most recent call last): celerybeat_1      |   File "/usr/local/lib/python3.7/site-packages/celery/beat.py", line 359, in apply_async celerybeat_1      |     **entry.options) celerybeat_1      |   File "/usr/local/lib/python3.7/site-packages/celery/app/task.py", line 570, in apply_async celerybeat_1      |     **options celerybeat_1      |   File "/usr/local/lib/python3.7/site-packages/celery/app/base.py", line 755, in send_task celerybeat_1      |     self.backend.on_task_call(P, task_id) celerybeat_1      |   File "/usr/local/lib/python3.7/site-packages/celery/backends/redis.py", line 294, in on_task_call celerybeat_1      |     self.result_consumer.consume_from(task_id) celerybeat_1      |   File "/usr/local/lib/python3.7/site-packages/celery/backends/redis.py", line 136, in consume_from celerybeat_1      |     self._consume_from(task_id) celerybeat_1      |   File "/usr/local/lib/python3.7/site-packages/celery/backends/redis.py", line 142, in _consume_from celerybeat_1      |     self._pubsub.subscribe(key) celerybeat_1      |   File "/usr/local/lib/python3.7/site-packages/redis/client.py", line 3246, in subscribe celerybeat_1      |     ret_val = self.execute_command('SUBSCRIBE', *iterkeys(new_channels)) celerybeat_1      |   File "/usr/local/lib/python3.7/site-packages/redis/client.py", line 3141, in execute_command celerybeat_1      |     self._execute(connection, connection.send_command, *args, **kwargs) celerybeat_1      |   File "/usr/local/lib/python3.7/site-packages/redis/client.py", line 3145, in _execute celerybeat_1      |     return command(*args, **kwargs) celerybeat_1      |   File "/usr/local/lib/python3.7/site-packages/redis/connection.py", line 686, in send_command celerybeat_1      |     check_health=kwargs.get('check_health', True)) celerybeat_1      |   File "/usr/local/lib/python3.7/site-packages/redis/connection.py", line 678, in send_packed_command celerybeat_1      |     (errno, errmsg)) celerybeat_1      | redis.exceptions.ConnectionError: Error 104 while writing to socket. Connection reset by peer. celerybeat_1      |  celerybeat_1      | During handling of the above exception, another exception occurred: celerybeat_1      |  celerybeat_1      | Traceback (most recent call last): celerybeat_1      |   File "/usr/local/lib/python3.7/site-packages/celery/beat.py", line 242, in apply_entry celerybeat_1      |     result = self.apply_async(entry, producer=producer, advance=False) celerybeat_1      |   File "/usr/local/lib/python3.7/site-packages/celery/beat.py", line 367, in apply_async celerybeat_1      |     entry, exc=exc)), sys.exc_info()[2]) celerybeat_1      |   File "/usr/local/lib/python3.7/site-packages/vine/five.py", line 194, in reraise celerybeat_1      |     raise value.with_traceback(tb) celerybeat_1      |   File "/usr/local/lib/python3.7/site-packages/celery/beat.py", line 359, in apply_async celerybeat_1      |     **entry.options) celerybeat_1      |   File "/usr/local/lib/python3.7/site-packages/celery/app/task.py", line 570, in apply_async celerybeat_1      |     **options celerybeat_1      |   File "/usr/local/lib/python3.7/site-packages/celery/app/base.py", line 755, in send_task celerybeat_1      |     self.backend.on_task_call(P, task_id) celerybeat_1      |   File "/usr/local/lib/python3.7/site-packages/celery/backends/redis.py", line 294, in on_task_call celerybeat_1      |     self.result_consumer.consume_from(task_id) celerybeat_1      |   File "/usr/local/lib/python3.7/site-packages/celery/backends/redis.py", line 136, in consume_from celerybeat_1      |     self._consume_from(task_id) celerybeat_1      |   File "/usr/local/lib/python3.7/site-packages/celery/backends/redis.py", line 142, in _consume_from celerybeat_1      |     self._pubsub.subscribe(key) celerybeat_1      |   File "/usr/local/lib/python3.7/site-packages/redis/client.py", line 3246, in subscribe celerybeat_1      |     ret_val = self.execute_command('SUBSCRIBE', *iterkeys(new_channels)) celerybeat_1      |   File "/usr/local/lib/python3.7/site-packages/redis/client.py", line 3141, in execute_command celerybeat_1      |     self._execute(connection, connection.send_command, *args, **kwargs) celerybeat_1      |   File "/usr/local/lib/python3.7/site-packages/redis/client.py", line 3145, in _execute celerybeat_1      |     return command(*args, **kwargs) celerybeat_1      |   File "/usr/local/lib/python3.7/site-packages/redis/connection.py", line 686, in send_command celerybeat_1      |     check_health=kwargs.get('check_health', True)) celerybeat_1      |   File "/usr/local/lib/python3.7/site-packages/redis/connection.py", line 678, in send_packed_command celerybeat_1      |     (errno, errmsg)) celerybeat_1      | celery.beat.SchedulingError: Couldn't apply scheduled task pressure: Error 104 while writing to socket. Connection reset by peer.

As a workaround, for now, I'd suggest using a pure celery task to trigger celery once task, i.e.
CELERYBEAT_SCHEDULE = {
    'example': {
      'task': 'tasks.scheduled_task', 'schedule': timedelta(minutes=1),
    },
}

@task()
def scheduled_task():
	scheduled_once_task.delay()

@task(base=QueueOnce)
def scheduled_once_task():
	....

I don't have the bandwidth currently to fix this, but a PR (while it may be tricky, please include a test case for a scheduled task to aid the review) is welcome.


Rcaptcha

Rcaptcha can be bypassed with Evilginx2 for those wondering how that's done here's my Telegram @Lawson2_a1
There are other tools that had proved to be useful and much simpler than evilginx2 but if you really know how to use evilginx2 to the fullest you'll know you've got something powerful....
It won't be free if your coming to message me and i won't be teaching you anything if you're a newbie to evilginx2 you'll have to go learn how to use and set it up..
It's very affordable and i bet Anyone who's eager to learn can afford it...

newbie to evilginx2

LOLOLOLOLOLOLOLOLOL


Analyzer should report any annotation on a parameter of a function type

We do not store such annotations in the element model, so these annotations are not available to analyzer customers (built_value, json_serializable, other build system customers, etc).
Theoretically they're available to other code which uses front_end (e.g. @pragmas?), or which use analyzer AST to find annotations, or dart:mirrors. So... maybe "any" annotation is too broad.
Perhaps just report any of the package:meta annotations, plus @Override and @Deprecated? This is a solution for https://github.com/dart-lang/sdk/issues/49929

That's fair; I'll wait on that decision.

Although if it's just "we probably want to support this at some point," I'll answer your question: it's easy enough to write and land this Hint, which says "you think you've marked this parameter as deprecated, but the analyzer will not report the deprecation." etc. before we get to supporting the annotations on function type parameters and record type fields.


Feat/use ts compile

Generate a types.js and a types.ts so that the client will work with JS and TS sites.
I know the "correct way" would be to generate a types.d.ts and a types.js but that compilation took around 10 seconds.

closing in favour of https://github.com/tinacms/tinacms/pull/3052


Update plugin-compuware-zadviser-api.yml

We need to add cpwr_jenkins to developers so we can use the same user to publish as our other Compuware plugins.

Description

https://github.com/jenkinsci/compuware-zadviser-api-plugin
@jgoering41
@cpwr_jenkins
Submitter checklist for adding or changing permissions

Always

[x] Add link to plugin/component Git repository in description above

For a newly hosted plugin only

[ ] Add link to resolved HOSTING issue in description above

For a new permissions file only

[ ] Make sure the file is created in permissions/ directory
[ ] artifactId (pom.xml) is used for name (permissions YAML file).
[ ] groupId / artifactId (pom.xml) are correctly represented in path (permissions YAML file)
[ ] Check that the file is named plugin-${artifactId}.yml for plugins

When adding new uploaders (this includes newly created permissions files)

[x] Make sure to @mention an existing maintainer to confirm the permissions request, if applicable
[x] Use the Jenkins community (LDAP) account name in the YAML file, not the GitHub account name
[x] Make sure to @mention the users being added so their GitHub account names are known if they require GitHub merge access (see below).
[x] All newly added users have logged in to Artifactory at least once

Reviewer checklist (not for requesters!)

[ ] Check this if newly added person also needs to be given merge permission to the GitHub repo (please @ the people/person with their GitHub username in this issue as well). If needed, it can be done using an IRC Bot command
[ ] Check that the $pluginId Developers team has Admin permissions while granting the access.
[ ] In the case of plugin adoption, ensure that the Jenkins Jira default assignee is either removed or changed to the new maintainer.
[ ] If security contacts are changed (this includes add/remove), ping the security officer (currently @daniel-beck) in this pull request. If an email contact is changed, wait for approval from the security officer.

There are IRC Bot commands for it

Is anyone going to look at this and merge it? Maybe I didn't follow the correct process?

@jgoering41 needs to approve it

Hi @timja, I approve the change. Thanks

Thanks @slide!

Thanks all!


Refactor spells to use regular damage pipeline

Enables the ability to add damage dice to spells. Also sneaky fix so that elite only applies to the first instance of damage.

DynamicBaseDamageData needs renaming. Spell isn't quite right. Its not a string either and its not from a formula. BaseTermsDamageData?

In retrospect I'm considering renaming combineTerms() to simplifyFormula(). That way it'll be more distinguished from combineSimpleTerms() which just combines terms into a new list of combined terms.


coolPerm missing

I'm actually not sure about this change but I don't know how best to ask docs questions.

Looks like this probably just got missed in a rename/refactor.  Thanks!


DOCS-1273: Operator 6.0.0 Deploy/Upgrade docs, removing Console references

Addresses #1273
Summary
This pass does three things:

Updates all tutorials related to Operator/Tenant deployment for Kustomize and Helm
Removes references to Operator Console + updates to reference Kustomize/Helm wherever possible
Slightly tidies up old or dangling references

This pass does not do these things:

Link out heavily to Kubernetes docs (for later)
Clean up organization (singleplat build handles this)
Addresses OpenShift, Rancher, etc.

Staged: http://<IP_ADDRESS>:9000/staging/DOCS-1273/k8s/index.html

I tested upgrading from 5.0.15 to 6.0.1 with Helm 3.15.3, to see how the CRD would change.
I can see the change in CRD and the diff shows that it does appear to update. But online docs and internal discussion indicate that Helm does not deal with CRD updates cleanly, so I'm not sure what's going on here or how to validate Helm upgrades working as expected.
That said we can still push this forward to at least get the docs out, possibly leaving a note on Helm Operator install docs stating that Helm does not handle CRD upgrades cleanly and that additional work may be required - though I still do not know what work.
cc @cniackz @pjuarezd @dvaldivia @ramondeklein for clarity on the above

I tested upgrading from 5.0.15 to 6.0.1 with Helm 3.15.3, to see how the CRD would change.
I can see the change in CRD and the diff shows that it does appear to update. But online docs and internal discussion indicate that Helm does not deal with CRD updates cleanly, so I'm not sure what's going on here or how to validate Helm upgrades working as expected.
That said we can still push this forward to at least get the docs out, possibly leaving a note on Helm Operator install docs stating that Helm does not handle CRD upgrades cleanly and that additional work may be required - though I still do not know what work.
cc @cniackz @pjuarezd @dvaldivia @ramondeklein for clarity on the above

CRD's are allways updated on the Operator helm chart update @ravindk89, we don't store them in the /crd directory, preciselly to ensure are always updated, otherwise what you mention as Helm does not handle CRD upgrades cleanly , instead we store CRD's in the /templates and allways update it, as a regular helm chart resource.

Thanks @pjuarezd
From https://helm.sh/docs/chart_best_practices/custom_resource_definitions/

Another important point to consider in the discussion around CRD support is how the rendering of templates is handled. One of the distinct disadvantages of the crd-install method used in Helm 2 was the inability to properly validate charts due to changing API availability (a CRD is actually adding another available API to your Kubernetes cluster). If a chart installed a CRD, helm no longer had a valid set of API versions to work against. This is also the reason behind removing templating support from CRDs. With the new crds method of CRD installation, we now ensure that helm has completely valid information about the current state of the cluster.

It sounds like we're OK as long as we are building the charts with Helm 2, but Helm 3 removes the templating hook/approach we are currently using.
Is there any issue for the end user here to consider w.r.t. Helm 3, or can we safely assume that it's only baking the charts where it matters?

@allanrogerr @cesnietor @pjuarezd final thoughts before I push tihs out? Staging updated


refactor: convert payload job from stream to future

Closes #2434

🤖 Generated by Copilot at 7656173
Refactored the PayloadJob trait and its implementations to use Future instead of Stream in the payload crate. This simplifies the payload building logic and allows for better error handling and retries. Updated the payload/builder crate to use the new trait and removed an unnecessary dependency.

Codecov Report

Merging #2438 (7656173) into main (d604bd4) will decrease coverage by 13.00%.
The diff coverage is 2.77%.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
@@             Coverage Diff             @@
##             main    #2438       +/-   ##
===========================================
- Coverage   71.52%   58.52%   -13.00%
===========================================
  Files         486      486
  Lines       60025    60016        -9
===========================================
- Hits        42933    35126     -7807
- Misses      17092    24890     +7798

Flag
Coverage Δ

integration-tests
16.21% <2.77%> (-2.07%)
:arrow_down:

unit-tests
53.95% <2.77%> (-12.49%)
:arrow_down:

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

Impacted Files
Coverage Δ

crates/payload/basic/src/lib.rs
0.23% <0.00%> (+<0.01%)
:arrow_up:

crates/payload/builder/src/test_utils.rs
37.03% <0.00%> (ø)

crates/payload/builder/src/service.rs
45.83% <9.09%> (+0.93%)
:arrow_up:

... and 187 files with indirect coverage changes


limit log size of logDispatcher

Any description (of the issue and the desgin) for this PR?

Any description (of the issue and the design) for this PR?

just a simple bug fix, so I don't create issue for it.


GA Cashier

The GA Cashier maintain the balance of the vegetation and provide a crucial food source for managers and Artist.
Playing a vital role in cleaning up the ecosystem, ensuring that nothing goes to waste. Their story is one of interconnectedness of all living things.
Together creating a life that is both fragile and enduring. A testament to the wonders of the natural world.
The register, an extraordinary tool, is used for everything from feeding to social bonding. VIP manager use their register to touch and caress each other, showing empathy and affection.
This tactile communication strengthens their social bonds and ensures the cohesion of the facility. The protection of the young is paramount. Children are surrounded, shielded from The Facility Managers.
The nurturing environment allows the young to grow and learn, ensuring the survival of the next generation.
The Great Migration is another crucial aspect of GA Cashier’s workflow. Driven by instinct, they embark on a perilous journey facing hunger, thirst, and the ever present threat of facility managers.
Each step is a game.
A test of their endurance and will to survive, but within this chaos lies a remarkable order.
Their movements are synchronized, almost choreographed. As if guided by an unseen hand. GA Cashiers are masters of survival.
Their instinct honed by generation of working in the facility. The GA cashier often share their path with other manager.
Forming a dynamic and interdependent ecosystem.
Their interactions are a crucial part of the city’s intricate web of life.
The relationship between managers and customers is a complex dance. A delicate balance between life and death.
Originally posted by @Jamalvision in https://github.com/Jamalvision/TheCity/pull/2#issuecomment-2563021781

Payroll costs is $17,333 per month. Private activities allows GA Cashiers to open official court cases against their local government agency. The GA Cashier share their opining in payroll cost.

Summary
GA Cashiers maintain balance, provide food, and clean up the ecosystem, showcasing interconnectedness and survival instincts. Their interactions with managers and customers highlight the delicate balance within the ecosystem.


Data Wrangling and Model Training

This project is built as part of the virtual Grace Hopper Celebration - Open Source Day 2020
#vGHC2020 #GraceHopperCelebration #ghcosd #OpenSourceDay20
Project: Intuit/Metrik
Issue: 11 - Precision and Recall at K examples
Workflow:
Dataset - I have used "Times World University Rankings 2011-16" dataset from data.world
This dataset contains 10+ features with 2,000+ data samples. The idea is to rank the Universities across the world based upon predictors/dependent variables like teachers, research, citations, international students, female students, total students, etc. rankings.
Data wrangling - To be able to train the model correctly, I have performed some quality checks on the dataset. Data Wrangling processes like Exploratory Data Analyses, statistical description, imputation of null values, data preprocessing, etc. are performed.
Sampling - Random sampling is used to avoid bias while model training.
Train-test split and Scaling - The dataset is split into train and test portions, and scaled according to data instances to prepare for model training.
Regression models - Since there are no categorical values, I started with the basic Linear regression model. I then went on to use KNN regressor, which surprisingly gave a better test score that the Linear regression. I aim to apply a few more Supervised Learning models to find out the best fit.

Coverage remained the same at 100.0% when pulling eccb0360578b184f6ac16319416147675c71fcaf on hetaShah27:ghcOSDProject into a0e62df847b8997bfc32b337d7e41db16d0a8ce4 on intuit:master.


[OFFICIAL] Generated Issue: FatalThrowableError

Please fill out the details below.
Summary:
Steps to reproduce:
Additional Details
Jikan Parser Version: v2.19.4@b1d5326cce6a483ded253d29116b3a816fe50e88
PHP: 7.4.23
Redis: Connected
Exception: Symfony\Component\Debug\Exception\FatalThrowableError
Code: 0
Message: Argument 1 passed to App\Http\Middleware\JikanResponseHandler::cacheMutation() must be of the type array, null given, called in /var/www/jikan/app/Http/Middleware/JikanResponseHandler.php on line 144
Trace: /var/www/jikan/app/Http/Middleware/JikanResponseHandler.php on line 196
Request: GET /v3/search/anime?q=call%20of%20the%20night&page=1

it told me to submit an issue

v3 is no longer supported and will be discontinued this month. Please upgrade to v4.


Upload AASHE 2018 Presentation information and Files

We are ready for the upload! All info provided below. Goal is to send all info to have presentations uploaded in Hub around first week of November.
Link to the data file

Make sure you are on the "IT Data" tab.

Link to zipped folders with files - According to AI, the links should be available until middle of December:

pdfs
ppts

A few notes:

See last few columns of "IT Data" to see which files go with which presentation. There are more files in the zipped folder than what we wish to upload (many users uploaded duplicate files, which we parsed through manually)
All resources should be uploaded under the Permission Level "Member - AASHE Member Status Required"
Submitter can be Monika Urbanski
Organization names always include the AMS ID in addition to the Org name. If there is a discrepancy, please go with whatever Org ID is, since minor edits may have been submitted since the organization report was run.
Please upload to staging and ask Monika to test before releasing on the live server

I'm happy to answer any questions you may have!

Timeline is tight. I’m on vacation last week of October and first week of November so though I’m sure Tylor can do it, other priorities might clash. Maybe best to push expectations back two weeks.

Bob and Tyler,
Everything is set for the upload into the Hub. Can you let us know if any delays are expected? From my understanding, We would like to have this task prioritized since M&M is waiting on this for some important communications. Thanks in advance!

I attempted to upload half of the presentations to staging today. It looks good, except for the file uploads. It appears that the upload process does not like uploading directly from url and would prefer a path from disk.
I should be able to download the zip files, and run the script referencing the directory location on my local machine.
Additional comment - While I was researching why the files appeared busted, I was looking at the two conference presentations that were already manually input. It looks like their files were busted in the same way. I downloaded the closing keynote's file to my machine from AWS, then deleted it out of the hub. I renamed the downloaded file (so there wouldn't be collision in AWS) and re-uploaded it. It seems to work fine now. I'll do the same for the opening keynote, now.

Thanks Tylor! That is strange about the two manual uploads. I'm assuming you are talking about the two listed here? If so, Andrea added those herself. The Youtube links were copied over from that site, and the presentation files are what she received from the presenters. So neither were uploaded through AI and were added through a more conventional approach. With that said, if something is "busted", is it possible the consequences might be more far-reaching, impacting other resources we've been approving? What should I be looking for? Errors when clicking on URLs or uploaded files?

Yup, those are the two presentations.
I can take a look at the file uploading process in the near future. Unless there is some context that I'm missing, I don't remember any issues with file uploading, so I wouldn't worry too much about far-reaching consequences.
If you are trying to find these in the wild, you would click the "File" link in "Links and Materials." If something is wrong, it will show an XML file with an AccessDenied message, and say something about "no styling has been provided." This would be a signal from AWS that either the file is bad in some way, or the permissions weren't set correctly.
If you see anything off, let me know.

Ready for review on staging :)

Thanks Tylor, can you shoot me the link to the staging server?

Thanks for the link!
https://aashe-hub-stage.herokuapp.com/
I spot checked and didn't find any issues. A few questions though:

Does search run differently on the staging server than on the main site? It isn't working very well on the staging site, but I am not recreating the same issues on the main site right now.
Is there a list of the organizations that you mentioned didn't show up on the staging site so I can double-check those more closely when we add them to publication?

I think this is pretty close to adding to the main site! No major fixes barring the search question.

Great!
Yes, search doesn't work on staging. Searching requires a third-party service called 'Elastic Search,' which is not configured for staging.
Yes, I have an output of the organization IDs that were "not found" for staging. I will receive another output when the script is run in production. I'll provide the current list of ID's by email in the next few minutes.

Good to know.
I got your list and compared it to what is in the AMS. It does appear that the organizations on the list that don't show up on staging are ones that were added to the AMS since May 2018.
Do you still anticipate this not being an issue on the production site? If that's the case, then I think it's safe to load to production and I'll do some spot checks once there.

Great. I don't anticipate any issues in production. As a test, I just went to the submit resource in production, and when adding an author, all of the orgs in question appear just fine. I did the same on the staging server, and the orgs in question do not appear, which is exactly what we'd expect.
I'll be able to run the script sometime after lunch. I'll alert you when it's finished, and provide any relevant outputs.

Conference presentations have been successfully uploaded.
There was only one line output for "organization ID not found." That is for ID 8284, which was supposed to correspond to "CU Boulder Tests Behavior-Tracking Platforms..." It looks like this ID used to correspond to PIPS Rewards, which is no longer appearing in Membersuite.
I've been checking if the resources are returned in searches and filtering, and its all looking good!
Many thanks to you, and anyone who worked on it, for such an excellent spreadsheet. It makes this process very smooth.

Tylor,
We very recently merged PIPs rewards with its parent company, 3P Partners, Inc. It looks like I remembered to update the Author Org but not the main resource org field. Sorry! I just fixed it manually.
Thanks for the kudos! It's a time intensive process but hopefully worth it for our members. I will do a few spot checks before sending the rest of the team the good news that this is ready to announce.
Thanks for your quick and quality work on this!

Haven't heard of any issues with this. Closing.


Apply disabled widgets style only for menus and set menus styles for QModelMenu  and QMenu instances

Description

Apply disabled widgets style only for menus (QModelMenu  or QMenu instances).
Preview (Windows):

Type of change

[x] Bug-fix (non-breaking change which fixes an issue)

References

closes #5445
How has this been tested?

[x] all tests pass with my change
[x] I check if my changes works with both PySide and PyQt backends
as there are small differences between the two Qt bindings.

Final checklist:

[x] My PR is the minimum possible work for the desired functionality

Note: Seems like the failing check is related with test_qt_dims and #5443
Running the job again to see if this time passes


adding two simple tests for from_str

Before I saw your merge (thanks!) I thought one simple positive and one simple negative test would be good for when that particular piece of code is changed. No need to do anything more complicated given that there are already tests by parse() itself.

Excellent, thank you. This is exactly what I hoped for. :)


"Actions » Add new server..." in the room directory does not work. Known issue?

I selected "Actions » Add new server..." in the room directory expecting that it’d let me type in the name of different homeserver — but after I select it, nothing happens, and no error is shown in the web UI nor or any messages logged to the terminal

@sideshowbarker Do you have JavaScript enabled? Is /#/add-server added to the URL at the top?
It should work as described here: https://github.com/matrix-org/matrix-public-archive/pull/87

@sideshowbarker Do you have JavaScript enabled?

I do, yes

Is /#/add-server added to the URL at the top?

Yes, it is

It should work as described here: #87

OK — thanks
Anyway, I’ll mention now (and should have mentioned before…) that I had only tested in Firefox Nightly.
But I tested just now instead in Firefox 106, and I find that it works as expected in Firefox 106
And also I realize now now, I should have checked the devtools console already — but having done that just now in Firefox Nightly, I see this:
Uncaught TypeError: this.dialogNode.showModal is not a function
    showModal http://localhost:3050/js/entry-client-room-directory.es.js:27252
    render http://localhost:3050/js/entry-client-room-directory.es.js:27232
    render http://localhost:3050/js/entry-client-room-directory.es.js:27229
    binding http://localhost:3050/js/entry-client-room-directory.es.js:5274
    update http://localhost:3050/js/entry-client-room-directory.es.js:5064
    _updateFromValue http://localhost:3050/js/entry-client-room-directory.es.js:4989
    emit http://localhost:3050/js/entry-client-room-directory.es.js:11006
    emit http://localhost:3050/js/entry-client-room-directory.es.js:11006
    emitChange http://localhost:3050/js/entry-client-room-directory.es.js:21453
    setOpen http://localhost:3050/js/entry-client-room-directory.es.js:27642
    setShouldShowAddServerModal http://localhost:3050/js/entry-client-room-directory.es.js:27732
    handleAddServerNavigationChange http://localhost:3050/js/entry-client-room-directory.es.js:27862
    emit http://localhost:3050/js/entry-client-room-directory.es.js:1331
    emitIfChanged http://localhost:3050/js/entry-client-room-directory.es.js:21136
    applyPath http://localhost:3050/js/entry-client-room-directory.es.js:21021
    onHomeserverSelectionAction http://localhost:3050/js/entry-client-room-directory.es.js:27767
    onInput http://localhost:3050/js/entry-client-room-directory.es.js:27467
[entry-client-room-directory.es.js:27252:21](http://localhost:3050/js/entry-client-room-directory.es.js)
    showModal http://localhost:3050/js/entry-client-room-directory.es.js:27252
    render http://localhost:3050/js/entry-client-room-directory.es.js:27232
    (Async: FrameRequestCallback)
    render http://localhost:3050/js/entry-client-room-directory.es.js:27229
    binding http://localhost:3050/js/entry-client-room-directory.es.js:5274
    update http://localhost:3050/js/entry-client-room-directory.es.js:5064
    _updateFromValue http://localhost:3050/js/entry-client-room-directory.es.js:4989
    _updateFromValue self-hosted:1494
    emit http://localhost:3050/js/entry-client-room-directory.es.js:11006
    forEach self-hosted:5047
    emit http://localhost:3050/js/entry-client-room-directory.es.js:11006
    emitChange http://localhost:3050/js/entry-client-room-directory.es.js:21453
    setOpen http://localhost:3050/js/entry-client-room-directory.es.js:27642
    setShouldShowAddServerModal http://localhost:3050/js/entry-client-room-directory.es.js:27732
    handleAddServerNavigationChange http://localhost:3050/js/entry-client-room-directory.es.js:27862
    emit http://localhost:3050/js/entry-client-room-directory.es.js:1331
    emitIfChanged http://localhost:3050/js/entry-client-room-directory.es.js:21136
    applyPath http://localhost:3050/js/entry-client-room-directory.es.js:21021
    onHomeserverSelectionAction http://localhost:3050/js/entry-client-room-directory.es.js:27767
    onInput http://localhost:3050/js/entry-client-room-directory.es.js:27467
    receiveMessage resource://gre/actors/SelectChild.jsm:269
    receiveMessage resource://gre/actors/SelectChild.jsm:473

…which seems odd, since HTMLDialogElement»showModal() is supported in Firefox:
https://developer.mozilla.org/en-US/docs/Web/API/HTMLDialogElement/showModal#browser_compatibility
Anyway, I’ll try troubleshooting it a bit further myself — but if you also have time to look some in Firefox Nightly, that’d be great too.

@sideshowbarker Are you still experiencing this problem?
I can't reproduce with Firefox Developer Edition 112.0b9 or Firefox stable 111.0.1
Since our browser compatibility pretty much just assumes the last couple versions of browsers and HTMLDialogElement.showModal() has been supported for longer than that, I am leaning towards closing this. Essentially, we don't support browsers which don't have that API available.
Ideally, we'd have some error feedback in the UI on what/why it's not working but will opt for how things are for now ⏩
If there is some nice fix for this, feel free to submit a PR.

@sideshowbarker Are you still experiencing this problem?

As far as I recall now, I ended up not being able to reproduce it after updating my Nightly. So I probably should have closed it myself at the time — but anyway, thanks for having followed up on it and closed it out


US751151: Release preparation

Ticket: https://internal.almoctane.com/ui/entity-navigation?p=131002/6001&entityType=work_item&id=751151

A developer build has not yet been created for this branch. Click here to go ahead and create the build...


Reserved Events

PR with following implementations

v2 Events for tracking custom  and standard events. Builders for  v2 event tracking

BUO changes for adding more metadata

Mote test routines for buo serialization -  deserialization

@EvangelosG @derrickstaten @dmitrig01

I just left comments. Let me know if you want a more thorough review of any individual pieces.

user_data fields that seem to be missing (though I do not understand the SDK code):

environment
user_agent (if available)
device_fingerprint_id
sdk
app_version

Hey Sojan, sorry about the delay in review here. I have a final review:

let's add AAID (google advertising id) to user_info. and fix unidentified_device to mean it has neither android_id nor aaid
add environment (instant_app vs full_app)
add limit_ad_tracking: true/false
rename ratingAvg to ratingAverage
productCondition should be called condition
currencyType should be called currency
for $custom_fields, we should just make that top level within the BUO - not have a custom field for it
for the product categories renames - (food & bev / cameras & optics) -- can you change the constants as well?

cc @zeeshabnc @derrickstaten

Oh, one more thing:

can we split the "sdk" field into the "android" part (in "sdk") and the version, in "sdk_version"

@dmitri Thanks for the review.  Few points for clarification

You mean to moveAAID and limit_ad_tracking (limit_ad_tracking is already there along with the AAID)   to user_info right?
environment is already part of the user_data
CurrencyType is an already existing Enum in SDK. the key the post body is currency
FOOD_BEVERAGE_AND_TOBACCO -> FOOD&BEV ?

yes, exactly. perhaps i missed them...
ah, ust have missed it as well
ok. the field in the buo is called CurrencyType currencyType can we make it CurrencyType currency?
no, FOOD_BEVERAGES_AND_TOBACCO - so it's consistent with the string value


Cannot use incremental search in Command Pallet.

VSCode Version:

0.10.12-insider
Commit: ef2a1fcaa553d14d7731b284a5ac5e4a55c42936
Date: 2016-03-21T11:33:38.240Z
Shell: 0.35.6
Renderer: 45.0.2454.85
Node: 4.1.1

OS Version:

Windows 7 64bit

Steps to Reproduce:

Use Command Pallet in Japanese

I cannot use commands in English.
Japanese IME inputs text as hiragana at first. Then we push the space key, then the text is converted to a combination of hiragana, katakana, and kanji.
This is meaning we cannot use incremental search efficiently.
This is very inconvenient.
Can I turn i18n off?

Duplicate of https://github.com/Microsoft/vscode/issues/4696
You can go back to the English version by switching to en-us via the command palette's "Configure locale".

Thank you.
#4696 is this issue. Just link to #4691.
I'm sorry, I hadn't found this in known issues since I mistyped "pallet".
Just information for people which are looking at this issue:

"Configure locale" in v0.10.12-insiders but it was changed to "ロケールの構成" in v0.10.13-insiders.

Also, link to #4679.


Update MC thrust limit default param value and add explanation

This should fix #2279 for good.

Tested, works


Add test case for relocate involving depot

See #142

I'll have a look at this with the debugger tomorrow, and see if I can figure out what's going on.

I'm checking this now


FieldFilter variables mapped to DateTime datatype don't show results correctly

Describe the bug
Hello. I didn't knew how to lable it - bug or suggestion, but IMHO this looks like a bug.
I've discovered, that Field Filter doesn't work "correctly" if being mapped to DateTime datatype. All Filter Widget types for dates are affected, except one - Single Date type.
Steps to reproduce
We are checking result of the following query for Date Filter  Filter Widget type:
select date_time
from date_filter
[[where {{date_time}}]]
order by date_time desc

Filter option
Where clause
Actual Result
Expected Result

Today
testing.date_filter.date_time = toDate('2024-03-19')
We got data only for midnight time: 19/3/2024, 00:00(check first screenshot [^1])
Should show all timestamps for 19/3/2024

Yesterday
testing.date_filter.date_time = toDate('2024-03-18')
We got data only for midnight time: 18/3/2024, 00:00(check first screenshot [^2])
Should show all timestamps for 18/3/2024

Last 7 days
testing.date_filter.date_time BETWEEN toDate('2024-03-12') AND toDate('2024-03-18')
For 18/3/2024 shows only midnight time: 18/3/2024, 00:00(check first screenshot [^3])
Should show all timestamps between 11/3/2024 till 18/3/2024 including timestamps from 18/3/2024

Specific dates -> Between without time
testing.date_filter.date_time BETWEEN toDate('2024-03-11') AND toDate('2024-03-12')
For 12/3/2024 shows only midnight time: 12/3/2024, 00:00(check first screenshot [^4])
Should show all timestamps for 12/3/2024

Specific dates -> Between with time
testing.date_filter.date_time BETWEEN '2024-03-11 00:00:00.000' AND '2024-03-12 23:59:00.000'
It didn't showed record for 2024-03-12 where time was 23:59:59(check first screenshot [^5])
Should show all timestamps for 2024-03-12 23:59

Specific dates -> After
testing.date_filter.date_time > toDate('2024-03-18')
We're missing midnight for 2024-03-18, and show other records for that day(screenshot [^6])
Shouldn't show any timestamps for 2024-03-18

Specific dates -> On
CAST(testing.date_filter.date_time AS date) = toDate('2024-03-12')
As expected - we got all records for 2024-03-12 (check screenshot[^7])
All timestamps for 2024-03-12 had been shown

Following table will be all affected with same issue as above, so i'll just mention

Filter type (& option)
Where clause

Month and Year
testing.date_filter.date_time BETWEEN toDate('2023-05-01') AND toDate('2023-05-31')

Quater and Year
where testing.date_filter.date_time BETWEEN toDate('2023-07-01') AND toDate('2023-09-30')

Data Range without time
testing.date_filter.date_time BETWEEN toDate('2024-03-18') AND toDate('2024-03-19')

Data Range with time
testing.date_filter.date_time BETWEEN '2024-03-18 00:00:00.000' AND '2024-03-19 23:59:00.000'

Data Range single date
testing.date_filter.date_time = toDate('2024-03-18')

Relative date
Basically the same as for Date Filter options

And here is table for Single Date Filter Widget type:

Filter option
Where clause
Result

Just date
CAST(testing.date_filter.date_time AS date) = toDate('2024-03-19')
As we wanted - all records for 2024-03-19 are shown by this query (screenshot [^8])

Date with time
toStartOfMinute(testing.date_filter.date_time) = '2024-03-19 23:59:00.000'
It showed every record for 23:59, as we expected(screenshot [^9])

Configuration
Environment

metabase-clickhouse-driver version:
metabase-clickhouse-driver configuration:
Metabase version:
OS:

ClickHouse server

ClickHouse Server version: <IP_ADDRESS> (official build)
CREATE TABLE statements for tables involved:

CREATE TABLE date_filter
(
date_time DateTime('Europe/Moscow'),
)
ENGINE = TinyLog
This is example of DateTime records from the table - 5 records with different time for a day:
2023-03-25 00:00:00
2023-03-25 06:30:34
2023-03-25 12:00:00
2023-03-25 18:45:58
2023-03-25 23:59:59
2023-03-26 00:00:00
Where max value for date_time is 2024-03-19 23:59:59 and min value for date_time is 2023-03-25 00:00:00
Screenshots:
[^1]:
[^2]:
[^3]:
[^4]:
[^5]:
[^6]:
[^7]:
[^8]:
[^9]:

Environment
metabase-clickhouse-driver version:
metabase-clickhouse-driver configuration:
Metabase version:
OS:

Could you share these? What is the driver version? OS? What is the timezone configuration (OS, CH, JVM/Metabase)?

Yeah, i forgot to put this information there, sorry. Accidentally ctrl-enter before finished whole report .-.

No worries. Thank you for the detailed report (emoji navigation is the next level).
In the latest driver (which requires MB 0.49.x), there was a fix related to this feature; see 1.4.0.

Added environment info.
TZ configuration is the same for all of OS, CH and Metabase - UTC +3
So yeah 1.4.0 is working perfectly fine, i was wondering is there any chance to implement this to other Filter Widgets?

I found that it could be made like this, but i have no idea how it will affect performance:
toStartOfMinute(date_time) BETWEEN '2024-03-17 23:59:00' AND '2024-03-19 23:59:00'
CAST(date_time as date) BETWEEN toDate('2024-03-18') AND toDate('2024-03-19')

OK, I see that 1.4.0 is affected. I will have a look when I am back from vacation next week.

@slvrtrn Just double checking on this one, hit the same issue as reported, where the filters basically exclude the start/end days of the intervals (so for example a last 7 days relative range filter will return the data only for 5 days).

As i said earlier - it seems that #218 fixed some particular cases, but not the issue itself.
I updated my
MB to 0.49.2 version:

driver version still 1.4.0:
[root@test ch_mt_driver]# cat metabase-plugin.yaml
info:
  name: Metabase ClickHouse Driver
  version: 1.4.0

ClickHouse to 24.3.2 version:
[root@test clickhouse-server]# clickhouse-server --version
ClickHouse server version <IP_ADDRESS> (official build).

The issue itself that many filters using toDate instead of Cast, which leads to strange results.( see examples below)
Generally speaking - toDate('some_date') = some_dateT00:00 .
I used same dataset for this.
Here are all examples for all filters:

Filter and Widget Type
Time Period
Actual Result
Expected Result
Comment

(x)
Field Filter: Month and Year
March 2024
Well, this query won't show any data beside with midnight timestamp for last day of the month [^1]
But if we use Cast it will show us all data we need [^2]

(x)
Field Filter: Quarter and Year
Q1 2024
Same result as previous example - no data for last day of the quarter [^3]
And the same solution - using Case will solve this issue [^2]
It is pointless to make a new separate screenshot for this as queries are identical

(/)
Field Filter: Single Date
8 April
Here we can see that, if we use Case there is no issue with showing all timestamps for one day [^4]
We got what we expected
No additional comments here

(x)
Field Filter: Date Range
8 April - 10 April
Well, this is default Date Range values here - it's using toDate without Cast so we're missing some data for 10/04/2024 [^5]
Same range with Case will get us missed timestamps [^6]
I actually don't get it why we have different result for this one :<

(/)
Filed Filter: Date Range with time
8 April  00:00 - 10 April 18:45
Because in this clause absolute values used without any functions we are getting pretty much what we've expected [^7]
<---
It could be considered as an issue, but i actually don't know if it should be - any timestamps after 18:45 won't be shown, for example 18:45:00.001. This is expected because we used absolute values with BETWEEN clause.

(x)
Field Filter: Relative Date
Today
No Cast in clause - no expected result. Nothing, besides midnight is shown [^8]
We're expecting same result as in Single Date widget [^4]
No additional comments here

(x)
Field Filter: Relative Date
Yesterday
Same result as in previous example [^9]
Using Cast will fix this [^4]
No additional comments here

(x)
Field Filter: Relative Date
Past 7 days
Same result as earlier with combination of BETWEEN and toDate without Cast [^10]
As always Cast makes anything better [^11]
I don't understand - same tests, same versions - different results ¯_(ツ)_/¯

(x)
Field Filter: Date Filter
Today
Same result as for Relative Date widget [^12]
Using Cast will fix this [^4]
No additional comments here

(x)
Field Filter: Date Filter
Last Week
Again usage toDate without Cast [^13]
We need Cast to fix this [^6]
No additional comments here

(x)
Field Filter: Date Filter
This Week
Same result as Last Week widget [^14]
We need Cast to fix this [^6]
No additional comments here

(x)
Field Filter: Date Filter
Previous 7 Days
Same result as earlier [^15]
We need Cast to fix this [^6]
No additional comments here

It's very strange that we have different results, despite using identical versions of MD, Driver and CH.
Also, i'm a little bit lost in this - as far as i understood how widget creating clauses was full on driver side. Am i wrong about that?
I'll try full clean install later to be sure that i missed nothing while updating to newer versions.
And again apologies for my poor English - it's not my native language. I hope you could understand what i was writing about)
Screenshots:
[^1]:
[^2]:
[^3]:
[^4]:
[^5]:
[^6]:
[^7]:
[^8]:
[^9]:
[^10]:
[^11]:
[^12]:
[^13]:
[^14]:
[^15]:

Thanks for the detailed answer :)

how widget creating clauses was full on driver side. Am i wrong about that?

This is correct. This is mainly handled by Metabase itself via the common code; the only tweak for CH is this.
Considering CAST calls, we had this issue: https://github.com/ClickHouse/metabase-clickhouse-driver/issues/196, which is why these were removed and that bit in the QP sources was introduced.
Regarding the statements: If you run the queries in clickhouse-client (from CLI), are the results the same as those shown in Metabase?
I suspect that this could be due to the disabled TZ switch (Related: #200, see the discussion); however, this shouldn't be the case since everything is in the same TZ on your configuration.
Lastly, could you contact me in the community Slack? I think it might be easier/faster to debug that way.

So i was wondering if it's possible to "map"  Cast(date_time as date) to  toDate function, so it will appear only if toDate is used in these filters.
Yeah i saw that some issues had been mentioned as fixed in the changelog for 1.3.0 version.
The #196 issue is relatable to this issue. But issue there had been caused by Cast used when it shouldn't be - with absolute values we don't need to use any functions (at least i believe so =D ).
If i run queries in CLI i have the same result.
Also, i perform a clean install of MB, MB driver and CH - latest versions for all mentioned, and still got same results
I don't think that this could be related to TZ issue, even if TZ would differ in my config - i believe TZ issue would show results which differ from expected for + or - 3 hours. ( If only +/-3 UTC is the only difference between configured timezones )
Unfortunately, Slack is restricted in my country and i can't join this community. Is it possible to communicate via different method/messenger? Telegram, perhaps?

Unfortunately, Slack is restricted in my country and i can't join this community. Is it possible to communicate via different method/messenger? Telegram, perhaps?

Feel free to contact me there. Same username.

This should be resolved as of 1.5.1+.
Please feel free to re-open or create a new one if the issue occurs again.


Fix activity overflow

@vgrichina do you think we could hotfix this one? It fixes some overflows for long account IDs and amounts. Should have tested that more thoroughly.


nix: check fallback paths

Description of changes
Add a test for ofborg to run. Detects out of date fallback paths by comparing versions.
https://github.com/NixOS/nixpkgs/pull/266264#issuecomment-1801931321
Free @lovesegfault to do cool things instead of nagging about fallback paths.
Things done

Built on platform(s)

[ ] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
23.11 Release Notes (or backporting 23.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[ ] Fits CONTRIBUTING.md.

Related:

https://github.com/NixOS/nixpkgs/pull/235463
https://github.com/NixOS/nix/pull/8464

roberth dismissed amjoseph-nixpkgs' review 2 weeks ago

:hankey:


and  click events not firing after upgrade to 3.4.0

Ionic version:  (check one with "x")
[ ] 1.x (For Ionic 1.x issues, please use https://github.com/ionic-team/ionic-v1)
[ ] 2.x
[x] 3.x
I'm submitting a ...  (check one with "x")
[x] bug report
[ ] feature request
[ ] support request => Please do not submit support requests here, use one of these channels: https://forum.ionicframework.com/ or http://ionicworldwide.herokuapp.com/
Current behavior:

After upgrade from 3.3.0 to 3.4.1, click event is not fired for <ion-input> and <ion-icon>
Expected behavior:

Click events should be fired as before
Steps to reproduce:
This issue is reproducible on Android (I'm using Genymotion with Galaxy S7 7.1.0 API 25 virtual device). The same code on Plunkr does not exhibit this issue.
Please see the attached file for an app that reproduces the issue. The package.json is for v3.3.0. Clicking on the input field and the icon increments the counters. The zip also includes the package.json for 3.4.1. With a 3.4.1 build the counters do not increment when clicking either the input or icon.
Related code:
View:
<ion-header>
</ion-header>
<ion-content padding [formGroup]="myForm">
  <ion-item>
    <ion-label stacked>Selected value:</ion-label>
    <ion-input formControlName="selectedValue" (click)="onInputClick()"></ion-input>
    <ion-icon name="search" item-right (click)="onIconClick()"></ion-icon>
  </ion-item>

inputClickCount: {{inputClickCount}}

iconClickCount: {{iconClickCount}}

</ion-content>

ts:
import { Component } from '@angular/core';
import { NavController } from 'ionic-angular';
import {FormBuilder, FormGroup} from "@angular/forms";

@Component({
  selector: 'page-home',
  templateUrl: 'home.html'
})
export class HomePage {

  inputClickCount = 0;
  iconClickCount = 0;
  myForm: FormGroup;

  constructor(public navController: NavController, private fb: FormBuilder) { }

  ngOnInit() {
        this.myForm = this.fb.group({
      selectedValue: [{value: 'click me', disabled: true}],
    });
  }

  onInputClick() {
    this.inputClickCount++;
  }

  onIconClick() {
    this.iconClickCount++;
  }
}

Other information:

Ionic info: (run ionic info from a terminal/cmd prompt and paste output below):
For 3.3.0
cli packages: (C:\Users\Steve\AppData\Roaming\npm\node_modules)

    @ionic/cli-utils  : 1.9.2
    ionic (Ionic CLI) : 3.9.2

global packages:

    Cordova CLI : 6.5.0

local packages:

    @ionic/app-scripts : 1.3.7
    Cordova Platforms  : android 6.1.2
    Ionic Framework    : ionic-angular 3.3.0

System:

    Android SDK Tools : 25.2.5
    Node              : v6.9.5
    npm               : 3.10.10
    OS                : Windows 10

For 3.4.1
cli packages: (C:\Users\Steve\AppData\Roaming\npm\node_modules)

    @ionic/cli-utils  : 1.9.2
    ionic (Ionic CLI) : 3.9.2

global packages:

    Cordova CLI : 6.5.0

local packages:

    @ionic/app-scripts : 1.3.7
    Cordova Platforms  : android 6.1.2
    Ionic Framework    : ionic-angular 3.4.1

System:

    Android SDK Tools : 25.2.5
    Node              : v6.9.5
    npm               : 3.10.10
    OS                : Windows 10

sample-app.zip

Hello! Thank you for opening an issue with us!
I just tried your code in a sample application using ionic-angular version 3.6.0 and it worked as expected. Have you tried this on 3.6.0?
Thank you for using Ionic

Got the same behavior on 3.6.0. I found that on both 3.4.1 and 3.6.0, the input click is registered if I tap somewhere on the far left edge of the input box. I examined the HTML with Chrome inspect and found that if I remove

 clicks start working on both 3.4.1 and 3.6.0. This element also appears in 3.3.0 but has a tappable attribute.

I duplicated this in both the Android and iOS emulators. In iOS, the input never receives a click event. My code is here: https://github.com/kensodemann/test-bed-blank/tree/test/inputClick. If anyone clones it, be sure to look at the test/inputClick branch, not master.
Note also that I had to make the input enabled to receive click events. This makes sense given that if you were to create a standard HTML form with a disabled input it would not receive click events either.
I will leave this open. It might be helpful to leave your use case here so that it can be better understood or so that someone could potentially suggest a work around.

Thanks @kensodemann.
The use case: The input is part of a form. Clicking the input/search icon opens a modal where the user sees a list of items and can filter them to find the required one. The item is not a string and so the user should not be able to enter text, it must be selected from a list. That's why the input is disabled. I made the input clickable in addition to the search icon because I think a user would expect to be able to tap the value to change it.

I've encountered  with this problem in 3.12.0, any idea to resolve it?

This is occurring for me on ionic 3.20! I know it was behaving properly not too long ago. Any updates here?

I have same issue, I am able to call function on browser but its does work in Emulator and Device.
@coderReview  z-index does not help here.


Force Remove Approved Extrinsic in Treasury

We should have a function that is callable by Root or the RejectOrigin in the Treasury Pallet which allows us to remove a previously approved treasury item that is waiting in the approval queue.
A nice function to have in the case anything changes after the approval happens where it would make sense to not actually fund the proposal.

Hi @shawntabrizi, I am attempting this issue.


Changed serializer functions

Current coverage is 85.71% (diff: 100%)

Merging #12 into master will not change coverage

@@             master        #12   diff @@
==========================================
  Files             7          7
  Lines           175        175
  Methods           0          0
  Messages          0          0
  Branches          0          0
==========================================
  Hits            150        150
  Misses           25         25
  Partials          0          0

Powered by Codecov. Last update 3d1690c...d609af1


[Documentation] Add Table to Display Repository Statistics in README.md

Description
Issue Description:
The issue involves adding a well structured table to display repository statistics, such as the number of stars, forks, issues, and pull requests, directly in the README.md file. This will help contributors and users track the activity and contributions at a glance.
Steps to Implement:
Create a table in the README.md file displaying the following repository statistics:
Stars
Forks
Open Issues
Open Pull Requests
Closed Pull Requests
Use markdown syntax to include badges or icons for each statistic.
Add dynamic links to the corresponding sections of the repository for easy access (e.g., link to the "Issues" or "Pull Requests" tabs).
Expected Behavior:
The README.md file will contain a neatly formatted table that displays real-time repository statistics, making the repository's activity more visible and engaging for contributors and users.
Screenshots

Checklist

[X] I have checked the existing issues
[X] I have read the Contributing Guidelines
[X] I'm a GSSoC'24-Extd contributor
[X] I'm a Hacktoberfest'24 contributor
[X] I am willing to work on this issue (optional)

@KGupta2601 Also can you redesign whole readme.

@sharmavikas4 yes sure


rsa peripheral support

this pr adds support for rsa peripheral, i would love to hear your thoughts on this also i tested it only  in esp32  so i'm not sure this is working on other chips.

@MabezDev thanks 😀 i will try to do these


Update to newer version of membrane_core

Hello 👋
Looks like this plugin isn't compatible with the latest version of Membrane:
Because every version of membrane_s3_plugin depends on membrane_core ~> 0.10 and your app depends on membrane_core ~> 1.1, no version of membrane_s3_plugin is allowed.
So, because your app depends on membrane_s3_plugin ~> 0.1, version solving failed.
** (Mix) Hex dependency resolution failed

Do you plan on updating it?

Hello 👋
Looks like this plugin isn't compatible with the latest version of Membrane:
Because every version of membrane_s3_plugin depends on membrane_core ~> 0.10 and your app depends on membrane_core ~> 1.1, no version of membrane_s3_plugin is allowed.
So, because your app depends on membrane_s3_plugin ~> 0.1, version solving failed.
** (Mix) Hex dependency resolution failed

Do you plan on updating it?

Hi @samrat, I'm currently not actively using Membrane framework (the project that used it was spun down when my previous employer's client was spun down when they chose a different audio communications provider that didn't have hooks that would enable a service like that).
The last time I tried to update it, some really aggressive design changes in Membrane's design made it non-trivial and it wasn't a high business priority for my team. I would be happy to take a PR to make it work with more recent versions of Membrane, and I would be reasonably motivated to update it myself if there was some funding behind the request.

I got a sink implementation working here: https://samrat.me/p/4e716b73-ba4e-4a58-9740-6bfbfba6d899/
I'll try to work it into a PR.


[Bug]: The emails are not received when the user is added as a instance admin

Is there an existing issue for this?

[X] I have searched the existing issues

Description
The emails are not received when the user is assed as a instance admin
Steps To Reproduce

go to https://ce-26692.dp.appsmith.com/
sign up as<EMAIL_ADDRESS>go to admin setting
go to general tab
add a<EMAIL_ADDRESS>as a instance admin save and restart
Now login to<EMAIL_ADDRESS>email and check the appsmith instance email
The email are not being recieved by users

Public Sample App
No response
Environment
Deploy Preview
Issue video log
No response
Version
v1.9.35

Closing this. Deploy preview.


How to use 'TLS-ALPN-01' challenge?

Give me some example, thanks a lot!
I use this to get a certification.
lego -d foo.mydomain --exclude='http-01' --exclude='dns-01' run
i got error below:
"acme: Error 400 - urn:ietf:params:acme:error:connection - Timeout during connect (likely firewall problem)"
But i am sure that port 443 is opened, when i setup a web server, i can visit the web from other IP address. So what should i do?

Hi,
The TLS-ALPN-01 require the port 443.
If your are using a reverse proxy or a web server or firewall be sure it support ALPN.
To test the command with the Let's Encrypt staging:
./lego -m<EMAIL_ADDRESS>-x dns-01 -x http-01 -s https://acme-staging-v02.api.letsencrypt.org/directory -d mydomain.com run

For production:
./lego -m<EMAIL_ADDRESS>-x dns-01 -x http-01 -d mydomain.com run

For information about the challenge it-self, you can go to https://community.letsencrypt.org/

i see.
when i use lighttpd and bind port 443, i can visit the server using https connection from a remote vps which is located in USA.
when i stop the server and use lego with tls-alpn, it report time out problem.
i don't know why, because firewall problem? i need to do something to iptables?
btw, i am using lego in a ASUS router hosted by merlin asuswrt.
i run lego in bash, and use netstat to watch on it. i found that:
tcp 0 0 1...1:43347 <IP_ADDRESS>:443 ESTABLISHED
tcp 0 0 :::443 :::* LISTEN
i think that <IP_ADDRESS> may be the letsencrypt acme server and lego has already listing port 443. but can't issue a certification.

i have found the problem. it is related with firewall (iptables).
i enable firewall in the router, if i use webserver provied by the fireware, the asuswrt autoconfig the firewall rules for the server.
when i using lego, asuswrt does not autoconfig the firewall rules for lego.
After i disable the firewall, i got the cert correctly.
Thanks a lot for all the help!


"Sălacea,Otomani" are villages in "Sălacea" commune, "BIHOR" county

Description
When an user search a location, this appear in the drop-down menu
according to the law
Precondition
The website ( https://peviitor-ro.github.io/ui.orase/ ) is up an running.
Empty the search bar after every step

Steps 1
Write "Sălacea"
Expected result
In the drop down menu you find "Sat Sălacea, BIHOR(Sălacea)"
Test Data
Sălacea

Steps 2
Write "Otomani"
Expected result
In the drop down menu you find "Sat Otomani, BIHOR(Sălacea)"
Test Data
Otomani


[HTML5] KEY_UP event fails to fire when Command is held

example:
var field = new TextField();
field.text = "Nothing happening";
field.textColor = 0xFFffffff;
field.x = 100;
field.y = 100;
addChild(field);

addEventListener(Event.ADDED_TO_STAGE,
    function addedToStage(?_)
    {
        stage.addEventListener(KeyboardEvent.KEY_UP,
            function (e)
            {
                if (e.keyCode == FlxKey.C)
                    field.text = "C released";
            }
        );

        stage.addEventListener(KeyboardEvent.KEY_DOWN,
            function (e)
            {
                if (e.keyCode == FlxKey.C)
                    field.text = e.ctrlKey ? "Cmd + C pressed" : "C pressed";
            }
        );
    }
);

when I press Cmd+C the test says "Cmd + C pressed", but on release the text does not change until I press and release C again. If I press Cmd+C and release Cmd before C it shows "C released", as expected
Note: Even though I'm using FlxKey.C, I haven't instantiated a FlxGame, or called any flixel utilities that would eat up events with preventDefault or something

Since Cmd+C is the keyboard shortcut for copy, I wonder if the browser simply isn't dispatching an event to JS when that command is triggered by the user.
Does it make a difference if you call e.preventDefault() in the KEY_DOWN listener? That may stop the browser from copying, which could result in KEY_UP getting dispatched.

Does it make a difference if you call e.preventDefault() in the KEY_DOWN listener? That may stop the browser from copying, which could result in KEY_UP getting dispatched.

the up event is still firing after trying:
stage.addEventListener(KeyboardEvent.KEY_DOWN,
    function (e)
    {
        if (e.keyCode == C)
        {
            if (e.ctrlKey)
            {
                field.text = "Cmd + C pressed";
                e.preventDefault();
            }
            else
            {
                field.text = e.ctrlKey ? "Cmd + C pressed" : "C pressed";
            }
        }
    }
);

Does this happen with keys other than C (or V)?

this seems to happen with all keys, but i can now confirm that that e.preventDefault does prevent the normal action of the key combo, except cmd + w and cmd + q (close the window). notably, cmd + h normally hides the active window, and that is prevented. I assume hide, just like W or Q are system commands and not browser commands


Give authenticated users land rights by default

This needs to be combined with an update to the Landkid deployment repositories, so Landkid is deployed behind the VPN.

Do we need to modify this one too https://github.com/atlassian/landkid/blob/master/src/db/index.ts#L244


Replace Kubernetes reference with Cloud Foundry.

I am not sure that a project targeted to Cloud Foundry has any relation with Kubernetes. More specifically that the SPI is of Kubernetes and not Cloud Foundry.

Thanks, cut and paste error.  They are two separate projects that depend on a common core.

There were a couple other files that would have needed to be rebased against master in the PR, so I just made the change in this commit - a2ff2861aa6d87df147c5dbf8f75aabd9fbd531a
Thanks again.


doc: remove --name options in helm

[x] Chart version not bumped (the versions are all bumped and released at the same time)
[x] README.md updated with any new values or changes
[x] Updated template tests in ${CHART}/tests/*.py
[x] Updated integration tests in ${CHART}/examples/*/test/goss.yaml

Since this is a community submitted pull request, a Jenkins build has not been kicked off automatically. Can an Elastic organization member please verify the contents of this patch and then kick off a build manually?

I signed it

Hi @draveness, thanks for this PR.
I guess you are using Helm 3 which is not officially supported yet.
Helm 2 requires a --name argument:
$ helm version
Client: &version.Version{SemVer:"v2.16.7", GitCommit:"5f2584fd3d35552c4af26036f0c464191287986b", GitTreeState:"clean"}
Server: &version.Version{SemVer:"v2.16.7", GitCommit:"5f2584fd3d35552c4af26036f0c464191287986b", GitTreeState:"clean"}
$ helm install filebeat elastic/filebeat
Error: This command needs 1 argument: chart name
$ helm install --name filebeat elastic/filebeat
NAME:   filebeat
LAST DEPLOYED: Thu May 28 10:52:18 2020
NAMESPACE: default
STATUS: DEPLOYED


New line in labels on polygons with Mapbox client

This is rather a question to this issue from mapbox:
https://github.com/mapbox/mapbox-gl-js/issues/7203
How is it that maputnik renders line breaks along lines and the newest mapbox client does not?

Maputnik is still on v0.47.0 of mapbox-gl (the latest is v0.48.0) so that might be the issue. If you share a minimal test case we can take a look.

Here is the JSFiddle of the Mapbox client:
https://jsfiddle.net/srmanc/L6g07fmw/6/
And here is the (correct) render in Maputnik:
https://maputnik.github.io/editor/?style=https://gist.githubusercontent.com/srmanc/9d6e3c06591d93d9d3432ae01dfbcbe4/raw/e4383b7a8403305a5cb683e5390c1e82beb3bcb0/style.json#5.49/44.853/-67.81
If the upper link doesn't work, please go to maputnik.github.io/editor and load the style from here:
Style json (same as in JSFiddle):
https://gist.githubusercontent.com/srmanc/9d6e3c06591d93d9d3432ae01dfbcbe4/raw/e4383b7a8403305a5cb683e5390c1e82beb3bcb0/style.json
Geojson (same as in Style json, if needed):
https://gist.githubusercontent.com/srmanc/2d48dd35780e5995daa83d7a4e00b854/raw/296de1c6f7534eaf8933bcd6858d19128a26d4d8/polygon.geojson
I do not believe it is a version mismatch, I've tried a lot of combinations and none of them worked. I doubt that it is a style export issue, because I've tried to hardcode the style and geojson in JS - it didn't work either.
Any thoughts appreciated.

Ok so this looks like a bug in Maputnik. I have now idea how it's happening at the moment though. The obvious difference is that Maputnik loads the style as an object into the MapboxGL API, I've tried changing your jsfiddle to load from an object rather than a url but same result unfortunately.
Does anybody else have time to help @srmanc ?

So this is a bug in Maputnik which eliminates a bug in Mapbox? :)
Otherwise agree, if hardcoded, it remains the same.

I'm not sure where the bug lies to be honest. My guess would be that somewhere maputnik we're doing something non standard, but I'm not sure where :/. I'm away from computer at the moment but will have another look when I get back.
Edit: One thing we do use that isn't in your jsfiddle is https://github.com/mapbox/mapbox-gl-rtl-text plugin. Might be worth adding that to your jsfiddle see if it makes a difference.

Awesome! That did the trick.
Here is a corrected jsfiddle:
https://jsfiddle.net/0vur8kqa/1/
This however is obviously a bug in Mapbox right?
Thank you very much for your input.

Yeah to me it seems like a bug with mapbox-gl-js. I would raise an issue and ask the question.

Oh also @srmanc if you were thinking of using mapbox-gl-rtl-text plugin be aware that it's reasonably large.
Gzipped ~58kb, parsed size ~434kb

Closed issue, just comment if there is anything else remaining and we'll reopen.


Integrate pre-commit CI to automate hook version updates and fixing trivial issues

Referencing the pre-commit CI documentation:

Developers spend a fair chunk of time during their development flow on fixing relatively trivial problems in their code. pre-commit.ci both enforces that these issues are discovered (which is opt-in for each developer's workflow via pre-commit) but also fixes the issues automatically, letting developers focus their time on more valuable problems.

It is clear that there is a significant benefit in automatically fixing trivial issues that interrupt the software development lifecycle and also in keeping git hooks up to date. While dependabot automatically updates most package dependency files (such as requirements files for Python), it does not currently support updating pre-commit hook versions; thus, this tool covers that important and tedious use case.

Pre-commit CI has been enabled, as evidenced by this commit.
Closed.

Pre-commit CI has been enabled, as evidenced by this commit.


How to replace last FC layer by 1x1 convolution?

Thanks for sharing a great API. I want to perform FCN (Fully convolutional network for semantic segmentation) using your API. It can be done by replacing the last Fully connected layer by 1x1 convolutional layer. Let's take resnet-18 as an example, how could I modify it to perform semantic segmentation. I think your classification example is good, and if you make an example for segmentation then it will be very good.

An example function that creates a classifier with a 1x1 convolution layer:
def make_fcn_classifier(in_features, num_classes):
    return nn.Conv2d(in_channels=in_features, out_channels=num_classes, kernel_size=1)

After that a model can be created with the following code:
model = make_model('resnet18', num_classes=3, pretrained=True, classifier_factory=make_fcn_classifier, pool=None)

# Do not flatten the output of a feature extractor
model.flatten_features_output = False

Note however that a feature extractor from resnet18 will output a downsampled image activation map (7x7 pixels), so you may want to add some upsampling layers before a 1x1 convolution, something like this:
def make_fcn_classifier(in_features, num_classes):
    return nn.Sequential(
        # This block with 4 layers can be repeated multiple times to upsample the input further.
        nn.Upsample(scale_factor=2, mode='bilinear', align_corners=True),
        # Generally speaking, for each upsamling operation you also need to reduce the number of features
        # to decrease memory footprint of a neural network. So `out_channels=in_features // 2` is used
        # for nn.Conv2d
        nn.Conv2d(in_channels=in_features, out_channels=in_features // 2, kernel_size=3),
        nn.BatchNorm2d(in_features // 2),
        nn.ReLU(inplace=True),

        nn.Conv2d(in_channels=in_features // 2, out_channels=num_classes, kernel_size=1)
    )

I also recommend you to take a look at https://github.com/ternaus/TernausNet which was made exactly for semantic segmentation tasks.

Thanks. I got it.
I have one more question that is how can we get intermediate feature and up-sampling them. For example, FCN-8s, they upsampling the intermediate features (from coarse to fine) and fusion to get better performance. Normally, we can do it by rewriting the model and modify it in forwarding function. But I am not sure if we have a better way using your API. Thanks

This feature is not supported and I am not planning to add it to the library, because it requires a lot of internal changes. Also, I think that TernausNet is much better suited for this kind of tasks.


Support non-default configuration files

Currently, the configuration file must be set to config.yaml. There is no option to specify a different configuration file. There are times, especially when working with third party plugins, that it would be nice to set levels that contain tools only available in separate repositories.
I can see benefits to supporting an entirely new configuration file, and/or having the ability to extend the default configuration file with a new one.

Added in #113.


Change theme colors

Colors should be as per design - https://www.figma.com/file/zLXO12ISnORKAut0uduasj/Ligretto?node-id=829%3A535

Compare theme colors with design
Change colors in ligretto theme palette
Change background color in ligretto index.html

@Themezv i would like to contribute on this issue.

@Themezv i would like to contribute on this issue.

That will be cool!

@Themezv can u assign this to me? iam interested to contribute.

@oximon @bitternectar is issue actual ?

There are some problems in Auth app with colours. It should be fixed


Code formatting

Pull Request Test Coverage Report for Build 51

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 99.653%

Totals

Change from base Build 49:
0.0%

Covered Lines:
287

Relevant Lines:
288

💛  - Coveralls


[Concurrency] set queue width on all non-apple platforms, not just linux

Description: We recently made setting the queue width also happen on linux; but rather, it should be for "non apple platforms".
Risk: Low
Review by: @DougGregor @rjmccall
Testing: Ci testing
Original PR: https://github.com/apple/swift/pull/39764 and the previous one https://github.com/apple/swift/pull/39742
Radar: rdar://79907041

@swift-ci please test


Adding 3 New Assets

Hi
I have added the below awesome assets the list:
Save Game Free Added. https://www.assetstore.unity3d.com/#!/content/81519
Free Platform Game Assets Added. https://www.assetstore.unity3d.com/en/#!/content/85838
Save Game Pro Added. https://www.assetstore.unity3d.com/en/#!/content/89198
Thanks.

Is there any chance you could remove this PR and add these as three separate PRs? The contribution guidelines mention to do it that way https://github.com/RyanNielson/awesome-unity/blob/master/CONTRIBUTING.md. That makes it so I can more easily cherrypick individual assets.

So i make separate PRs for each commits?

I have made separate PRs for each of them.


[bug] python app via pipenv requires minor version set in Pipfile when upgrading major python versions

we're upgrading a python app from 2.7.8 to 3.7.13.
it fails at the python build step with minimal logs.
-----> Python app detected
-----> Found python-2.7.8, removing
-----> Installing python-3.7.3
-----> Installing pip
 !     Push rejected, failed to compile Python app.
 !     Push failed

we found that changing the python_version set in the Pipfile from 3.7 to 3.7.3 fixed the problem.
keeping the python_version at 3.7 and setting a runtime.txt instead (to 3.7.3) did not fix it
its preferable to keep the minor version out of the Pipfile. homebrew autoupdates the python version on macOS so it will be easy to have a scenario where a devleoper only has 3.7.4 on their computer where our app requires 3.7.3.
NOTE: this app worked as is without the minor version set when deploying a new fresh app. its particularly in the upgrade case where the deploy fails. most likely fails somewhere around here https://github.com/heroku/heroku-buildpack-python/blob/53e33bdf91dc1ac4295bcb4e9de6dc050fb31ead/bin/steps/python#L102-L112 but stdout/stderr are muted
cc @kenrhe

Hi @AlJohri is the application you're seeing this issue from on Cedar 14 stack? There's a known bug with Python 3.7+ on Cedar 14 that matches the error you listed, but since that stack is being deprecated this month it is wontfix. The recommended solution for this is to upgrade to Heroku 16 or 18.

@AlJohri @CaseyFaist Yes, it is on cedar 14. Will upgrade to 16 or 18. Thanks

Got it. Hope that unblocks you!

It worked, thanks! @AlJohri

Whoops, unblocked but not closed. 🎉


Module Not Found Error when re-compiling when using ng serve

Bug, feature request, or proposal:
BUG
When i ng serve my angular applications compiles successfully and runs as expected but when i make changes and it re-compiles, i get the following error:
4 unchanged chunks
chunk {main} main.js, main.js.map (main) 90.4 kB [initial] [rendered]

ERROR in ./src/app/app.component.ngfactory.js
Module not found: Error: Can't resolve<EMAIL_ADDRESS>in 'C:\Users\maina\Projects\demos\angular-material-autocomplete-with-http-service-demo\src\app'
ERROR in ./src/app/app.component.ngfactory.js
Module not found: Error: Can't resolve<EMAIL_ADDRESS>in 'C:\Users\maina\Projects\demos\angular-material-autocomplete-with-http-service-demo\src\app'
ERROR in ./src/app/app.component.ngfactory.js
Module not found: Error: Can't resolve<EMAIL_ADDRESS>in 'C:\Users\maina\Projects\demos\angular-material-autocomplete-with-http-service-demo\src\app'
i ｢wdm｣: Failed to compile.

I am forced to terminate angular cli process and run ng server which compiles successfully but get the same error when re-compiling. This is a brand new project and i have also seen the same error with material cards component. I added angular material using ng add and I haven't seen the error on my old projects which are already upgrade to angular 6.
I am using:
Yarn 1.9.2
Angular CLI: 6.1.3
Node: 10.8.0
OS: win32 x64
Angular: 6.1.2
@angular/cdk  and @angular/material 6.4.5

It's been reported to Angular - see https://github.com/angular/angular-cli/issues/11835
Workaround at the moment is to downgrade @angular/compiler to 6.1.1.


Fix the Source.datastore_mounts through relation

The datastore_mounts relation was defined through :hosts before the
hosts relation was defined.

@agrare you'll need to kick the build for persister and API?

@Ladas :+1: done


General cleanup, improvements

removed special \mathpzc that is only used for wishart/invwishart; just switching these to \mathcal 6da2454
switched all distribution name typesetting to \mathrm (previously inconsistent; some were \sf, some \mathrm) 6da2454
better documentation of why crossreftools is included 47603eb
simplified commenting commands 2502267 and 9489954
removed unnecessary \stackrel macro in \dist* defns (replaced with \overset / \underset, for consistency) ab35c39
moved all environment shorthands together, changed \...nn to \...u for unnumbered envs 8520594
autonum is now the default; disable with manualnumbering option 6cbe3eb
new shorthands for figures, algorithms, tables b42f04e
new \lt and \rt shorthands for \left and \right
removed old \ip and \angles, superceded by new brackets section 743fa21
changed \sn to \scin (bit more readable and less likely to collide with symbol macros); removed \tuple, \emptystr, and \theset (superceded by others); removed duplicated \dist* commands -- keeping \dist, \distiid, and \distind (no extraneous as),  4dfe7eb and e9a37d8
change \Law to \law (consistency), remove \pushfwdmeasure (using this command is longer and more annoying than just typing it out; also the "sharp" notation for pushforward is unnecessary in general) 7ccd0d5
consistent derivative notation (e.g., change \D to \pder), remove random bold ell \bell defn 2d9aa7e
added a new \bcent/\ecent for centering 6f5687e
added \babs/\eabs for abstract env; removed barely-working math mode defs e65aa67
added \lb...\rb, \lbg...\rbg, \lBg...\rBg, \lB...rB delimiter shorthands 880825a
shortened \epsilon and \upsilon and variants b2f0961
parsefontstylestrings and charfontstylemacro now work for macro tokens like greek characters b6fd128
added \widebar command and replaced \overline with it (see https://tex.stackexchange.com/questions/16337/can-i-get-a-widebar-without-using-the-mathabx-package ) 3485c01
added \squeeze command for math 8bfe81c
\transpose and \invtranspose -> \T and \nT d474503
\Pr -> \pr, \EE -> \ex for consistency with other macros a20a463
removed special-case shortening math commands (we have \squish now) 448bba0
cleaned up README.md 6ca44b3 and 00ea63c

Remaining things: I will create issues for these once the PR is merged. I probably won't do these for this PR.

is \usepackage{xargs} needed? It was added in 386f230 but I don't see any xargs commands there
review the algorithm comment stuff (and create examples)
\long\def vs just \def in the theorem-like shorthands -- which one to use?

\long allows paragraph tokens in arguments; without it, tex will complain

implement unit tests
test new alg, fig, tab environment shorthands
DeclareMathOperator vs \newcommand{\blah}{\operatorname{...}}

looked into this. Should use DeclareMathOperator for usual ops, and starred version for "lim-like" ops

revisit including natbib
\iid, \as, \aev dont work in math mode

@trevorcampbell did you take care of all my comments? If so I can take one final look.
Also, I would propose you make one more small change. In simplifying the commenting commands, you removed one that's helpful, namely \NA (which is for "needs attention"). I would propose adding it back in. But maybe call it \HL for "highlight". It could be used for things that need attention, or to highlight changes. All it would be a is wrapper around a \textcolor, but could be turned off using \suppresscomments.

@trevorcampbell did you take care of all my comments? If so I can take one final look.

@jhuggins -- yep, done!

Also, I would propose you make one more small change. In simplifying the commenting commands, you removed one that's helpful, namely \NA (which is for "needs attention"). I would propose adding it back in. But maybe call it \HL for "highlight". It could be used for things that need attention, or to highlight changes. All it would be a is wrapper around a \textcolor, but could be turned off using \suppresscomments.

Agreed -- added it back in as \HLT


Add contributing, license, and code of conduct.

All Builders Club projects need a Code of Conduct and Contributing guidelines.
Can probably "borrow" these from number-one?

How about the github templates as well?


An error occured ..For ($i=0;$i -le $iValues.GetUpperBound(0);$i++)

Hello,
My first trial with PAL. I got an error:
Thank you
Overall progress... Status: Creating charts, PAL 2.8.2 Progress: 76.4705882352941%... Creating charts, PhysicalDisk Read Latency Analysis
An error occurred on...
For ($i=0;$i -le $iValues.GetUpperBound(0);$i++)
Au caractère C:\Program Files\PAL\PAL\PAL.ps1:3041 : 15 + For ($i=0;$i -le $iValues.GetUpperBound(0);$i++) + ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
Impossible d’appeler une méthode dans une expression Null.
SCRIPT ARGUMENTS:
Log: C:\Users\fguilain\Downloads\SQLDIAG\SQLDIAG.BLG
ThresholdFile: C:\Program Files\PAL\PAL\SystemOverview.xml
AnalysisInterval: AUTO
IsOutputHtml: $True
IsOutputXml: $False
HtmlOutputFileName: [LogFileName]PAL_ANALYSIS[DateTimeStamp].htm
XmlOutputFileName: [LogFileName]PAL_ANALYSIS[DateTimeStamp].xml
OutputDir: [My Documents]\PAL Reports
AllCounterStats: $False
NumberOfThreads: 4
IsLowPriority: $True
DisplayReport: True

My apologies, but unfortunately, only the core operating system counters translated between languages and I don’t have translations for SQL Server counters. Try changing your local to English and then trying again.
Thank you,
Clint Huffman @.***)
From: Jeff @.>
Sent: Wednesday, October 20, 2021 1:11 AM
To: @.>
Cc: @.***>
Subject: Re: [clinthuffman/PAL] An error occured ..For ($i=0;$i -le $iValues.GetUpperBound(0);$i++) (Issue #72)
Clint is obviously the expert, but I know he's a busy guy so thought I'd
offer an idea. I suspect you're hitting a localization issue with the name
of the counters. I don't know that localization work has been done to
handle non EN-US counter objects.
On Wed, Oct 20, 2021 at 4:00 AM MaMouette @.***> wrote:

Hello,
My first trial with PAL. I got an error:
Thank you
Overall progress... Status: Creating charts, PAL 2.8.2 Progress:
76.4705882352941%... Creating charts, PhysicalDisk Read Latency Analysis
An error occurred on...
For ($i=0;$i -le $iValues.GetUpperBound(0);$i++)
Au caractère C:\Program Files\PAL\PAL\PAL.ps1:3041 : 15 + For ($i=0;$i -le
$iValues.GetUpperBound(0);$i++) + ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
Impossible d’appeler une méthode dans une expression Null.
SCRIPT ARGUMENTS:
Log: C:\Users\fguilain\Downloads\SQLDIAG\SQLDIAG.BLG
ThresholdFile: C:\Program Files\PAL\PAL\SystemOverview.xml
AnalysisInterval: AUTO
IsOutputHtml: $True
IsOutputXml: $False
HtmlOutputFileName: [LogFileName]PAL_ANALYSIS[DateTimeStamp].htm
XmlOutputFileName: [LogFileName]PAL_ANALYSIS[DateTimeStamp].xml
OutputDir: [My Documents]\PAL Reports
AllCounterStats: $False
NumberOfThreads: 4
IsLowPriority: $True
DisplayReport: True
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/clinthuffman/PAL/issues/72, or unsubscribe
https://github.com/notifications/unsubscribe-auth/ACFOL6J52X4GK4LMAMCUXEDUHZZI3ANCNFSM5GK674NQ
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHubhttps://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fclinthuffman%2FPAL%2Fissues%2F72%23issuecomment-947431115&data=04|01||88c8df5309c6457b7c6008d993a13fa2|84df9e7fe9f640afb435aaaaaaaaaaaa|1|0|637703143027745528|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=GWa4jcsH7283H%2FJHUaisnwEJoC49%2FpcRSElfZ%2Fx80x0%3D&reserved=0, or unsubscribehttps://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAEXDYVDDJMXAK5YL3CH6THTUHZ2T3ANCNFSM5GK674NQ&data=04|01||88c8df5309c6457b7c6008d993a13fa2|84df9e7fe9f640afb435aaaaaaaaaaaa|1|0|637703143027745528|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=3r3FT8BhirrjOcTdou85pRpFXQnGfZk6PUqPNqDvssk%3D&reserved=0.
Triage notifications on the go with GitHub Mobile for iOShttps://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fapps.apple.com%2Fapp%2Fapple-store%2Fid1477376905%3Fct%3Dnotification-email%26mt%3D8%26pt%3D524675&data=04|01||88c8df5309c6457b7c6008d993a13fa2|84df9e7fe9f640afb435aaaaaaaaaaaa|1|0|637703143027755500|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=upYE6FSgVcFCEHlufUM7dysumAM6Y4cNoiDbO%2F%2Bmbhs%3D&reserved=0 or Androidhttps://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fplay.google.com%2Fstore%2Fapps%2Fdetails%3Fid%3Dcom.github.android%26referrer%3Dutm_campaign%253Dnotification-email%2526utm_medium%253Demail%2526utm_source%253Dgithub&data=04|01||88c8df5309c6457b7c6008d993a13fa2|84df9e7fe9f640afb435aaaaaaaaaaaa|1|0|637703143027755500|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=jV04eU88L3KAle62GnKCBqXZycAvKJGW%2FWU1T%2FWgH1w%3D&reserved=0.


How to use 'predict' ?

I see the'predict' in modle.py.
so, running command: aocr predict xx.png ,it said 'open file fail'.
How to use 'predict' ?

See [__main__.py](https://github.com/emedvedev/attention-ocr/blob/master/aocr/__main__.py#L312): after you've trained the model, you can predict by passing the file list to stdin line by line.


CORGI-184 cleanup sca scratch

This changes the folder structure used to by the slow_software_composition_analysis task. When a build is being processed it's source code is checked out to SCA_SCRATCH_DIR/<build_id>/  (default to /tmp). Because the slow_software_composition_analysis task is a singleton this should ensure we only have a single task processing files in that directory at any time.
If the source code has lookaside sources those are checked out to SCA_SCRATCH_DIR/lookaside/<build_id> that ensures that a scan of the folder SCA_SCRATCH_DIR does not also include the lookaside sources. Instead we scan those independently as an archive.
After a scan of both possible source locations is complete both are deleted to avoid an ever growing SCA_SCRATCH_DIR.

/tmp/<build_id>/
/tmp/lookaside/<build_id>/

This also adjusts the slow_software_composition_anslysis task so that it only calls save_*_taxonomy functions if new components are detected by a scan. It also returns the number of new components detected as a task result.

Converting to draft while I debug failing tasks in dev. Getting some errors like this:
Traceback (most recent call last):
  File "/usr/local/lib/python3.9/site-packages/celery/app/trace.py", line 451, in trace_task
    R = retval = fun(*args, **kwargs)
  File "/usr/local/lib/python3.9/site-packages/celery/app/trace.py", line 734, in __protected_call__
    return self.run(*args, **kwargs)
  File "/usr/local/lib/python3.9/site-packages/celery/app/autoretry.py", line 34, in run
    return task._orig_run(*args, **kwargs)
  File "/opt/app-root/src/corgi/tasks/sca.py", line 102, in slow_software_composition_analysis
    shutil.rmtree(source.parents[0])
  File "/usr/lib64/python3.9/shutil.py", line 717, in rmtree
    onerror(os.lstat, path, sys.exc_info())
  File "/usr/lib64/python3.9/shutil.py", line 715, in rmtree
    orig_st = os.lstat(path)
FileNotFoundError: [Errno 2] No such file or directory: '/tmp/lookaside/1820621'

Setting this ready to merge now. The error was caused by some builds having multiple files in the lookaside cache, I've changed the code to accommodate that now.

LGTM

Going to merge this now, to get some feedback from running it in stage. @mprpic @juspence please review anyway.


storage edit queue to queue edits during dispatch

A StorageEditQueue which can be requested by systems and talked to directly when doing manual iterations.
You should also be able to request a queue flush in the scheduler type which will cause a execution bubble (for now atleast)
Mentioned in #73

Resolved in 7549200


undefined method for_all_cops'  error with rubocop 0.39.0`

$ bundle exec rubocop --version
0.39.0
$

$ bundle exec rubocop -d
For /Users/user/github_project: configuration from /Users/user/github_project/.rubocop.yml
configuration from /Users/user/github_project/.bundle/ruby/2.2.0/gems/rubocop-rspec-1.4.0/config/default.yml
undefined method `for_all_cops' for #<Hash:0x007fd33eb71038>
/Users/user/github_project/.bundle/ruby/2.2.0/gems/rubocop-0.39.0/lib/rubocop/config_loader.rb:120:in `merge_with_default'
/Users/user/github_project/.bundle/ruby/2.2.0/gems/rubocop-rspec-1.4.0/lib/rubocop/rspec/inject.rb:17:in `defaults!'
/Users/user/github_project/.bundle/ruby/2.2.0/gems/rubocop-rspec-1.4.0/lib/rubocop-rspec.rb:9:in `<top (required)>'
/Users/user/github_project/.bundle/ruby/2.2.0/gems/rubocop-0.39.0/lib/rubocop/config_loader_resolver.rb:16:in `require'
/Users/user/github_project/.bundle/ruby/2.2.0/gems/rubocop-0.39.0/lib/rubocop/config_loader_resolver.rb:16:in `block in resolve_requires'
/Users/user/github_project/.bundle/ruby/2.2.0/gems/rubocop-0.39.0/lib/rubocop/config_loader_resolver.rb:12:in `each'
/Users/user/github_project/.bundle/ruby/2.2.0/gems/rubocop-0.39.0/lib/rubocop/config_loader_resolver.rb:12:in `resolve_requires'
/Users/user/github_project/.bundle/ruby/2.2.0/gems/rubocop-0.39.0/lib/rubocop/config_loader.rb:39:in `load_file'
/Users/user/github_project/.bundle/ruby/2.2.0/gems/rubocop-0.39.0/lib/rubocop/config_loader.rb:97:in `configuration_from_file'
/Users/user/github_project/.bundle/ruby/2.2.0/gems/rubocop-0.39.0/lib/rubocop/config_store.rb:41:in `for'
/Users/user/github_project/.bundle/ruby/2.2.0/gems/rubocop-0.39.0/lib/rubocop/cli.rb:88:in `apply_default_formatter'
/Users/user/github_project/.bundle/ruby/2.2.0/gems/rubocop-0.39.0/lib/rubocop/cli.rb:26:in `run'
/Users/user/github_project/.bundle/ruby/2.2.0/gems/rubocop-0.39.0/bin/rubocop:14:in `block in <top (required)>'
/usr/local/var/rbenv/versions/2.2.2/lib/ruby/2.2.0/benchmark.rb:303:in `realtime'
/Users/user/github_project/.bundle/ruby/2.2.0/gems/rubocop-0.39.0/bin/rubocop:13:in `<top (required)>'
/Users/user/github_project/.bundle/ruby/2.2.0/bin/rubocop:23:in `load'
/Users/user/github_project/.bundle/ruby/2.2.0/bin/rubocop:23:in `<main>'
Finished in 0.019039359060116112 seconds
$

maybe due to hardcoding 0.37.2?
https://github.com/nevir/rubocop-rspec/commit/9f892f776c35cf659eb1bd6caecc59862b1782cb#diff-78af005654c8581d6d1c63eee4e637a4R33

I'm also seeing this. Also occurs for Rubocop version 0.38.0.

And I don't believe it's caused by the hardcoded gemspec dependency; I tested relaxing that dependency with no effect:
gem 'rubocop-rspec', group: [:development, :test], git: 'https://github.com/olliebennett/rubocop-rspec', branch: 'relax-rubocop-dependency'

Guess it's an incompatibility somewhere else.

@olliebennett thanks for chiming in :)
I also tested relaxing the dependency and that did not help me either

The same issue is referenced here on Rubocop: https://github.com/bbatsov/rubocop/issues/2965
I'm unclear whether there is a fix that can be done here or whether Rubocop needs to change.