fix warning from developer console

[ ] Dictionary metadata not coming from api

[ ] Infographics rendering again and again issue

[ ] in Reading plans
ERROR Warning: IGNORE.shouldComponentUpdate(): Returned undefined instead of a boolean value. Make sure to return true or false.

[x] Drawer Commentary (key missing)

[x] Search-missing key

[ ] language List-Non-serializable values were found in the navigation state. Check:-stack > LanguageList > params.updateLangVer (Function)

[ ] setting-WARN Slider has been extracted from react-native core and will be removed in a future release. It can now be installed and imported from '@react-native-community/slider' instead of 'react-native'. See https://github.com/callstack/react-native-slider

[x] hint-key missing

[ ] hint-WARN Animated: useNativeDriver was not specified. This is a required option and must be explicitly set to true or false

[ ] opening parallel pane-ERROR Warning: Each child in a list should have a unique "key" prop.

[ ] ERROR Warning: Can't perform a React state update on an unmounted component. This is a no-op, but it indicates a memory leak in your application. To fix, cancel all subscriptions and asynchronous tasks in the componentWillUnmount method.

[x] Parallel commentary-key missing

[ ] adding notes-Non-serializable values were found in the navigation state. Check:
stack > EditNote > params.onbackNote (Function)

[ ] parallel bible-if book not available it show proper dropdown message

Testing done on Aug 30
Errors in console for the following pages

[ ] Book dropdown
[ ] verse dropdown
[ ] bookmark select from drawer
[ ] highlight select from drawer
[ ] parallel bible key unique
[ ] Download bible on click
[ ] Parallel Commentary
Downloaded bible parallel view
[ ] ERRoR in console
[ ] Audio bible keeps opening
[ ] Next prev arrow shows but doesnt work
Other things to do
[ ] Remove all consoles.log


excuse me but what is the mean of this, why not just let it be one:               [1 / len(results)] * len(results)]

https://github.com/PaddlePaddle/PaddleNLP/blob/46082e300197c3d770bc1eb68f6a2b3fd0e8be0d/pipelines/pipelines/nodes/other/join_docs.py#L113

@FrostML @tianxin


preprocessFile isn't called

The preprocessFile method set in the config object is never called before uploading a file. My configuration is below.
I temporarily switched to use the preprocess method, and confirmed that it is called, however I want to run code for each file, not every chunk. I'll try to use the fileAdded event instead.
var r = new Resumable({
    target: '/test/'
    query: {},
    chunkSize: 1 * 1024 * 1024, // 1MB
    preprocessFile: function (file) {
        console.log(file);
        file.preprocessFinished();
    }
});

These methods were introduced in newer versions (which are present on github), but not released in npm.
I understand this question was asked a long time back, but one way around is to use preprocess method which is called on every chunk. From a chunk, you can access the linked resumableFile instance using resumableChunk.file. Then if the function is called twice on the same resumableFile (match using resumableFIle.uniqueIdentifier), skip those calls.


Vickel: Watch pfrcbd.com

Vickel requests the watch of the watch_keyword pfrcbd\.com. See the MS search here and the Stack Exchange search in text, in URLs, and in code.

pfrcbd\.com has been seen in 1 true positive, 0 false positives, and 0 NAAs.


Add ability to send attachments in pushover notifications

Description:
Added new feature, ability to send attachments in pushover notifications. This requires an upgrade of the python-pushover package from v0.3 to v0.4. Existing functionality preserved.
Example entry for configuration.yaml (if applicable):

Checklist:

[x] The code change is tested and works locally.
[x] Local tests pass with tox. Your PR cannot be merged unless tests pass
[x] There is no commented out code in this PR.
[x] I have followed the development checklist

If user exposed functionality or configuration variables are added/changed:

[x] Documentation added/updated in home-assistant.io

If the code communicates with devices, web services, or third-party tools:

[x] The manifest file has all fields filled out correctly. Update and include derived files by running python3 -m script.hassfest.
[x] New or updated dependencies have been added to requirements_all.txt by running python3 -m script.gen_requirements_all.
[x] Untested files have been added to .coveragerc.

If the code does not interact with devices:

[ ] Tests have been added to verify that the new code works.

Hi @kreegahbundolo,
It seems you haven't yet signed a CLA. Please do so here.
Once you do that we will be able to review and accept this pull request.
Thanks!

CLA signed and docs pushed for "next" branch.

Awesome! Thanks for doing this!


[ENG-6470] Withdrawn version picker

Ticket: ENG-6470
Feature flag: n/a

Purpose

Add a version select dropdown to the tombstone page

Summary of Changes

Add dropdown to tombstone page header to view prior versions
Update tests

Screenshot(s)

Side Effects

QA Notes

Re: indicator for the current version, I will go  ahead and add that now
Re: Preprint DOI dropdown, I will address those issues in a different PR. On that note, I agree that the Preprint DOI dropdown should have the currently viewed version selected by default, but I am wondering if selecting a different version in the Preprint DOI dropdown should redirect or not, if its purpose is to just show the DOI for different versions? It might be annoying if users just want to see the DOI for the different version and the page reloads when they select a version. May be a question for product team though


php artisan command does not bring the command list

Hi I have installed laravel 5.0 on my server, now, after installation I get the following error:

post-update-cmd: php artisan clear-compiled
Script php artisan clear-compiled handling the post-update-cmd event returned with an error

[RuntimeException]
Error Output:
Exception trace:
() at phar:///var/www/vhosts/entrenamiento.com/httpdocs/es/composer.phar/src/Composer/EventDispatcher/EventDispatcher.php:204
Composer\EventDispatcher\EventDispatcher->doDispatch() at phar:///var/www/vhosts/entrenamiento.com/httpdocs/es/composer.phar/src/Composer/EventDispatcher/EventDispatcher.php:94
Composer\EventDispatcher\EventDispatcher->dispatchScript() at phar:///var/www/vhosts/entrenamiento.com/httpdocs/es/composer.phar/src/Composer/Installer.php:335
Composer\Installer->run() at phar:///var/www/vhosts/entrenamiento.com/httpdocs/es/composer.phar/src/Composer/Command/UpdateCommand.php:158
Composer\Command\UpdateCommand->execute() at phar:///var/www/vhosts/entrenamiento.com/httpdocs/es/composer.phar/vendor/symfony/console/Command/Command.php:259
Symfony\Component\Console\Command\Command->run() at phar:///var/www/vhosts/entrenamiento.com/httpdocs/es/composer.phar/vendor/symfony/console/Application.php:844
Symfony\Component\Console\Application->doRunCommand() at phar:///var/www/vhosts/entrenamiento.com/httpdocs/es/composer.phar/vendor/symfony/console/Application.php:192
Symfony\Component\Console\Application->doRun() at phar:///var/www/vhosts/entrenamiento.com/httpdocs/es/composer.phar/src/Composer/Console/Application.php:166
Composer\Console\Application->doRun() at phar:///var/www/vhosts/entrenamiento.com/httpdocs/es/composer.phar/vendor/symfony/console/Application.php:123
Symfony\Component\Console\Application->run() at phar:///var/www/vhosts/entrenamiento.com/httpdocs/es/composer.phar/src/Composer/Console/Application.php:99
Composer\Console\Application->run() at phar:///var/www/vhosts/entrenamiento.com/httpdocs/es/composer.phar/bin/composer:43
require() at /var/www/vhosts/entrenamiento.com/httpdocs/es/composer.phar:25
Then when I try to open the artisan there is no output.

5.0 is not supported. Feel free to ask on the forums though.


Export latest deprecated field

We are deprecating the "scope" field for "az self-help discovery-solution list" command. We were previously advised to add a deprecation message for that parameter and remove the parameter altogether during the breaking change window. The PRs involved here take care of removing the parameters, while the previous PRs that are already merged have included the warning message regarding upcoming deprecation in version 0.4.0.
CLI-Extensions repo PR:
https://github.com/Azure/azure-cli-extensions/pull/7619

@necusjz thanks for approving, can we please merge this PR?


[Explorer]: add reference gas price calculation

Description

Add reference gas price for next epoch
Update next epoch card
https://mysten.atlassian.net/browse/APPS-808

Test Plan
How did you test the new or updated feature?

If your changes are not user-facing and not a breaking change, you can skip the following section. Otherwise, please indicate what changed, and then add to the Release Notes section as highlighted during the release process.
Type of Change (Check all that apply)

[ ] user-visible impact
[ ] breaking change for a client SDKs
[ ] breaking change for FNs (FN binary must upgrade)
[ ] breaking change for validators or node operators (must upgrade binaries)
[ ] breaking change for on-chain data layout
[ ] necessitate either a data wipe or data migration

Release notes

I also noticed that the blue circle of active validators is not using a gradient fill. could we please?

Also, one last point...if there are no new validators this round, let's hide that item instead of showing "0 New"

@Nikhil-Mysten nope, we can release this. This has been an ask for awhile


Core developer guide

Adds a guide for core developers of xarray. Mostly adapted from napari's core dev guide, but with some extra sections and ideas from the pandas maintainance guide.
@pydata/xarray please give your feedback on this! If you prefer to give feedback in a non-public channel for whatever reason then please us the private core team email.

[ ] Closes #xxxx
[ ] Tests added
[ ] User visible changes (including notable bug fixes) are documented in whats-new.rst
[ ] New functions/methods are listed in api.rst

Generally, this looks great, but can we adjust the language/title to emphasize "Core team member" rather than "Core developer" to be more inclusive?

Okay I've addressed all the comments so far, reorganised, and including sections on issue triage from the pandas maintainers guide.

This is great, Tom. I love it!

:grin:

It would be nice to render this on the website. We might have to add myst-nb etc. to the sphinx config (https://github.com/pydata/xarray/issues/7924). Could easily be done in a follow up PR though.

I agree, and I started by rewriting it as RST, but I got sick of reformatting the markdown copied from napari's docs so I decided to just leave  that to a later PR. I think it's useful to concentrate on discussing the actual content of the guide in this PR too.

Thanks @jhamman.
Yes there is definitely some overlap, and this should go in the main docs.
However I thought it was easier just to leave that reorganization for a
follow-up PR, and concentrate on the content in this PR, as Deepak and I
discussed above.
On Wed, Jul 19, 2023, 8:04 PM Joe Hamman @.***> wrote:

@.**** commented on this pull request.
Thanks for writing this @TomNicholas https://github.com/TomNicholas! I
left one comment but have an additional question. Should we put this in the
documentation. There is some overlap here with the contribution guide
https://docs.xarray.dev/en/stable/contributing.html. I wonder if this
should be a page next to the broader guide.
In CORE_TEAM_GUIDE.md
https://github.com/pydata/xarray/pull/7999#discussion_r1268794371:

We prefer that usage questions are asked on StackOverflow with the pandas tag. https://stackoverflow.com/questions/tagged/pandas

If it’s easy to answer, feel free to link to the relevant documentation section, let them know that in the future this kind of question should be on StackOverflow, and close the issue.

⬇ Suggested change

We prefer that usage questions are asked on StackOverflow with the pandas tag. https://stackoverflow.com/questions/tagged/pandas

If it’s easy to answer, feel free to link to the relevant documentation section, let them know that in the future this kind of question should be on StackOverflow, and close the issue.

We prefer that usage questions are asked on StackOverflow with the [python-xarray tag](https://stackoverflow.com/questions/tagged/python-xarray
+) or as a GitHub discussion topic.

If it’s easy to answer, feel free to link to the relevant documentation section, let them know that in the future this kind of question should be on StackOverflow, and close the issue.

—
Reply to this email directly, view it on GitHub
https://github.com/pydata/xarray/pull/7999#pullrequestreview-1538215026,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AISNPI2QKNKSBMOO5SDMZATXRBYXRANCNFSM6AAAAAA2OSK24A
.
You are receiving this because you were mentioned.Message ID:
@.***>


Remove the banshee backend

The Banshee backend is lagging behind added features and testing, and depends on a toolchain that is not standardised (nor there's any assurance it'll ever be) and that has several troubles (lacking of complete support for the C++ stdlib). Hence, the Banshee backend is nowadays unmaintained, nor there's any plan in sight to revive it. Therefore, we may want to remove it from the codebase and add a dedicated git tag for potential future resurrection (a.k.a., gravestone).
Reasons to remove it:

if you search within the code (e.g., with grep or an editor like VSCode), you may see results also from banshee, which makes the search more complicated
the bootstrap.sh script has dedicated logic, essentially untested (and there's no reason to maintain it, if banshee is never used)
there is a dedicated Makefile infrastructure, also unmaintained and untested with the latest versions

Please hold off any work on this one, need to confirm its status


Feature Request: Icon as text matching the time

I'm using ItsyCal to replace the "Day, Month Date" portion of the time in the menu bar. It would be awesome if the style could match the time text. I may take a crack at this at some point, so I'm also creating this issue to remind myself!

Version 0.11.1 lets you set an arbitrary datetime pattern and hide the rectangular part of the icon.


Add RakeTarget pre-commit and pre-push hook

Allow to run rake targets as pre-commit and pre-push hooks.
The first unsuccessfull exit code of a specified rake task
will fail the hook.
Example configuration:
PreCommit:
  RakeTask:
    enabled: true
    targets:
      - 'validate'
      - 'lint'

PrePush:
  RakeTask:
    enabled: true
    targets:
      - 'test'

If considired for merging I would require  a hint howto test for a raised error on missing targets parameter. :-)

@sds thanks for your review!

squashed all FIXUPs.

Thanks @vinzent!


RegexSet take up a lot of memory

v1.4.3
fn main() -> Result<(), Box<dyn std::error::Error>> {
    let mut string_vec = Vec::new();

    for index in 0usize..100_000 {
        let elem = format!(r"(^|\.){}\.com$", index);
        string_vec.push(elem);
    }

    let _regset = regex::RegexSetBuilder::new(string_vec)
        .unicode(false)
        .size_limit(std::usize::MAX)
        .build()?;

    std::thread::sleep(std::time::Duration::from_secs(10000));

    Ok(())
}

In fact, Regex also take a lot of memory. Is it has a way to limit under 20_000kB memory for 100_000 items?
VmPeak:	  428816 kB
VmSize:	  321960 kB
VmLck:	       0 kB
VmPin:	       0 kB
VmHWM:	  399676 kB
VmRSS:	  316124 kB
RssAnon:	  315316 kB
RssFile:	     808 kB
RssShmem:	       0 kB
VmData:	  320640 kB
VmStk:	     132 kB
VmExe:	    1168 kB
VmLib:	       4 kB
VmPTE:	     672 kB
VmSwap:	       0 kB
HugetlbPages:	       0 kB

Uhhh... Looks right to me. Regex sets aren't magic. You can't just create a set with 100,000 regexes and expect it to work well. That's a pretty specialized use case.
I would recommend finding a way to make the aho-corasick crate work for you. If you absolutely cannot do without regexes, then maybe try Hyperscan. But 100,000 regexes is a lot. I wouldn't expect much.


undefined: possiblePkcs11Libs (Windows installation)

When trying to install notary client with the command go install -tags pkcs11 github.com/theupdateframework/notary/cmd/notary I get the following error:
# github.com/theupdateframework/notary/trustmanager/yubikey go\src\github.com\theupdateframework\notary\trustmanager\yubikey\yubikeystore.go:95:22: undefined: possiblePkcs11Libs
Googling the error returned nothing. Is it even possible to install it on windows?

Using the latest release for windows worked fine

I also get this issue. Using the latest release did not work for me.


Uncaught (in promise) InternalError: too much recursion

Hi,
I'm using method
await socketlib.system.executeAsGM('showOptionBox', inputs, activeMods);
to call the Option window for other player/GM.
I got the result:
Uncaught (in promise) InternalError: too much recursion
    be is-binary.js:3
    Ee is-binary.js:20
    Ae is-binary.js:36
    Ae is-binary.js:45
    Ae is-binary.js:45
    Ae is-binary.js:45
    Ae is-binary.js:45
    Ae is-binary.js:45
    Ae is-binary.js:45
    Ae is-binary.js:45
    Ae is-binary.js:45
    Ae is-binary.js:45
    Ae is-binary.js:45
    Ae is-binary.js:45
    Ae is-binary.js:45
    Ae is-binary.js:45
    Ae is-binary.js:45

This happens only if I set target of an action in Foundry.

One limitation of socketlib is, that you can only send data to other clients that can be serialized to JSON. If you try to send a datastructure that references itself at some point, the json serializer gets stuck in an infinite recursion and eventually bails. I suppose that's what is happening here. What are the contents of the variables inputs and activeMods?

While activeMods is empty, inputs differs depending who call it.
In problematic case, it's big nested (tree-like) object.
I made yesterday workaround - the recipient only required a few fields from input, so I have filtered it and it helped.
(The recursion error was displayed on Firefox; on Chrome it was sth about too big thread stack or sth similar)


DOC: Add a link to Modin on the ecosystem page.

[x] Is part of #7266
[x] Tests added / passed
[x] Passes pre-commit run --all-files

Hello, I'm Kirill from the Modin group.
We've implemented conversion to and from Modin from the Dask Dataframe, so we'd like to share this point in your documentation to improve the data science  experience for Dask and Modin users.

Can one of the admins verify this patch?
Admins can comment ok to test to allow this one PR to run or add to allowlist to allow all future PRs from the same author to run.

@phofl, @fjetter, could you take a look at this PR?
Is it ok if the test is down?  I don't quite understand why this happened?


Can you Please add Traralgon? (3844)

sure: https://lukeprior.github.io/nbn-upgrade-map/?suburb=traralgon&state=VIC


Fix breakout example bug - ball flighlying out when collide paddle and wall at the same time

It seems there is a bug of the breakout example, that when ball is collided with both paddle and the wall, the velocity is evaluated and reflected with paddle first, then that break will stop it evaluate if there is other collisions, then the ball will just fly out of the wall.
It is easy to see the bug if you try hold the ball between the paddle and the bottom wall then follow it until the ball hits left or right wall. Then there is a good chance when ball hit paddle and left/right wall at the same time then just flying off the walls.
This should fix this bug in some degree but might not prevent other potential bugs related to this. For example the ball might hit the corner of two walls and only one axis of velocity will change and it might still able to fly off the walls. I was also thinking about collect all the possible collisions then do proper velocity change based on all the current possible collisions, but that might name the code a bit too complicated for an intro example?

Yeah I think the current approach to collisions is pretty naive / breaks easily. But in this case I wasn't able to repro, despite doing this and letting it bounce for a long time (10 minutes):

I think the "ball flying out" problem is more a result of framerate dips (resulting in large deltas + missed collisions) than duplicate collisions, and the only way to fix that is to build a more complicated collision system.
Regardless of whether or not this pr fixes a bug, I think I'd rather not add too many "special collision cases" to example games to hack around the limitations of the collision system, as they're intended to be extremely simple illustrations of game logic.
I'd rather focus our energy on making the collision system itself better (or adopting a third party solution like ncollide for collisions)

I think you need to try it as setting the bar length slightly shorter to have some gap facing the wall, perhaps use a bar length of 860 or 850, then as long as the ball touches the narrow gap, it flies away (takes like 10 or 30 seconds for me to see it.)

I think this could be a bad initial experiences or impression towards the engine when someone new here to try it out the code example for the first time. I just started looking at bevy yesterday, tried to run the first breakout example then while I played it for a few seconds I encountered the bug. Or maybe I was lucky. But initially I had a feeling that this game engine is not "reliable".
If there is a simple way to fix this, while not making the code look too much difficult to read, perhaps we should do it.

Cool I was able to repro with that advice. You're right about "perceived quality" being important and I guess this isn't particularly complicated. Can we change the match statement to this instead?
// break if this collide is on a solid, otherwise continue check whether a solid is also in collision
if let Collider::Solid = *collider {
    break;
}

But yeah if there is a way to make the collision system better or even easier to use, then of course that would be a better solution over all. What I am afraid is others might having the same first impression that "it is buggy" if they just try out the first example then encounter a bug like this.

Sure I can update the comments. Thank you for the correction 👍

Even though this make it less buggy but it might still not being perfect yet as when I tried out this solution, very few occasions it still can go out. But at least it is much more robust for now. Looking forward to a better collision system later then 👍

@liufuyang please respond in #2373 for the relicense to MIT/Apache 2.0. Thanks!


Azurepostgres

Signed-off-by: Shweta Salokhe<EMAIL_ADDRESS>

Please resolve conflicts

Closing this due to conflict. Will create another one.


GIMP: fix the manifest to enable  the console for python-fu

Relates to #8674

[x] I have read the Contributing Guide.

/verify


Load saved should not duplicate existing servers

Currently load saved just instantiates a server per record. We should only add missing servers.

Fixed in f3f5d65


Influencer sign up page backend

Accept POST requests to /signup endpoint. Create a new object in db.influencers collection that stores:

Full name
Image
Bio
login name
password

Upon successful sign-up, forward to account page/ list of campaigns.
If duplicate login name exists, return to sign up page with error message.

Just leaving one task open on the frontend.

We do need to handle the error case here for existing signup names.


Problem with Artist rows in Properties/Details

Description

The "Album Artist" can only show the last person in that section, and the first "Artist"(which doesn't exist in other apps) seems to be copying "Album Artist".
Steps To Reproduce
Open the Properties of any audio file, switch to Details, and that should happen.
Requirements

Remove the first "Artist", or make it more useful.
Let the "Album Artist" display not just the last person.

Files Version
<IP_ADDRESS>
Windows Version
10.0.22621.2283
Log File
debug.log

Thanks for the report, I've added this to the project board.

It looks like File Explorer calls it "Contributing Artists".

Hello,
I have looked into issue. Look like there are one more artist. So I have added detail properties in the file detail via explorer detail tab and comparing with Files app properties. I get duplicate Artist   Please see below image. I am new to open source world. Please guide me what is expected behavior.  I would like to work on it. Please let me know.


fix(autoware_pose_covariance_modifier): fix funcArgNamesDifferent

Description
This is a fix based on cppcheck funcArgNamesDifferent warnings
localization/autoware_pose_covariance_modifier/src/pose_covariance_modifier.cpp:74:73: style: inconclusive: Function 'callback_gnss_pose_with_cov' argument 1 names different: declaration 'msg' definition 'msg_pose_with_cov_in'. [funcArgNamesDifferent]
  const geometry_msgs::msg::PoseWithCovarianceStamped::ConstSharedPtr & msg_pose_with_cov_in)
                                                                        ^

Related links
Parent Issue:

Link

How was this PR tested?
Notes for reviewers
None.
Interface changes
None.

Effects on system behavior
None.

@meliketanrikulu
Please confirm the PR. Thank you very much.


Bug Report: Gap & Unwanted element below the Footer of the website

Reproduction steps
I was checking the website throughly and found a bug that is if you scroll down till the footer of the website. There is a small space below the footer as shown in picture below. I found the same bug for other devices too.

Expected behavior
There should be no spacing below the footer.
Actual Behavior
There is a wide spacing and a small  element as well in below the footer. As you can see in the picture attached below.

👀 Have you spent some time to check if this issue has been raised before?

[X] I checked and didn't find similar issue

🏢 Have you read the Code of Conduct?

[X] I have read the Code of Conduct

@SauravMukherjee44 ,If any one no working ,I can solve this issue.


Swagger API

Closes #2 - Create a swagger specification for the API.
Summary
This update includes both the Swagger API specification, as well as a markdown file for easier to read documentation.
Student
The general design principal is that students can be added to the back-end data store, either as an empty record or as with feedback data. The student feedback data can be accessed, updated, and deleted after a student record has been created.
Comment
The comment end-point is used for getting feedback from the API about a particular comment made to the student. Some possible examples could include that a similar comments has been provided before, that the comment is too long, that it lack sufficient detail, etc.
The comment API only allows the POST request, all other requests will return an error.
Verification

[ ] Verify summaries, description, and method names. These should accurately describe their use.
[ ] Validate the logic of the API - that it provides adequate and logical end-points for desired functionality
[ ] Validate the responses provided by the API, that they are easily understood
[ ] Validate data formats and responses provided by provided by the API. Some requests require objects, return objects, or simply provide a response to the request
[ ] Validate that documentation matches the API

Effort

Task
Time (hrs)
Description

Research
1.5
Research the swagger API and review notes from last quarter

Design
1.5
Make design decisions on which end-points API users would desire, and verify that initial design decisions won't impact future expansion of the API

Create API/API Docs
5
Create the API user the Swagger standard and create a markdown file for documentation

Revision
?
Revise the API based on feedback from front-end and back-end teams.

Total Time: 8 - 10+ hours

Resolved #26

@ZenSorcere. Great feedback, wondering if you could take another peak to verify that changes address your comments.


Execute backup full error exit status 1

Deployed by docker-compose. The following is the meta service information.

good catch, br is not recommended to used in docker environment for now. I'll test it later. @foesa-yang


[UNDERTOW-1559] At ApplicationListeners.requestInitialized, notify al…

…l "requestInitialized" notified listeners of requestDestroyed so they don't end in an inconsistent state

Jira: https://issues.jboss.org/browse/UNDERTOW-1559

This is ok to test


Error in _group_psms with bigger_scores_better param

When setting the param 'bigger_scores_better' and not setting 'validation_score_field' we get this error
bigger_scores_better = self.UNODE_UPARAMS['bigger_scores_better']['uvalue_style_translation'][search_engine]
UnboundLocalError: local variable 'search_engine' referenced before assignment

This is due to the fact, that search_engine is only set, when validation_score_field is None
I guess we should require, that both of these parameters need to be set

I think validation_score_field does not always require bigger_scores_better, but the other way round, yes.
So I agree, we should check this in the corresponding wrappers to get a clear error message.

should be fixed with #88


New version: LinearOperators v0.6.0

Registering package: LinearOperators
Repository: https://github.com/JuliaSmoothOptimizers/LinearOperators.jl
Created by: @abelsiqueira
Version: v0.6.0
Commit: 96aac23abf0e68417162eeaa044d99be753a9ea3
Reviewed by: @abelsiqueira
Reference: https://github.com/JuliaSmoothOptimizers/LinearOperators.jl/commit/96aac23abf0e68417162eeaa044d99be753a9ea3#commitcomment-35656855

Your new version pull request does not meet all of the guidelines for automatic merging.
Specifically, your pull request does not meet the following guidelines:

The following dependencies do not have a compat entry that has an upper bound: FastClosures

If you want to prevent this pull request from being auto-merged, simply leave a comment.
(If you want to post a comment without blocking auto-merging, you must include the text [noblock] in your comment.)

It is important to note that if your pull request does not meet the guidelines for automatic merging, this does not mean that your pull request will never be merged. It just means that your pull request will require manual review by a human.

These guidelines are intended not as requirements for packages but as very conservative guidelines, which, if your new package or new version of a package meets them, it may be automatically merged.

On a separate note, I see that you are registering a release with a version number of the form v0.X.Y.
Does your package have a stable public API? If so, then it's time for you to register version v1.0.0 of your package. (This is not a requirement. It's just a recommendation.)
If your package does not yet have a stable public API, then of course you are not yet ready to release version v1.0.0.

[noblock]

Your new version pull request met all of the guidelines for automatic merging.
I will automatically merge this pull request during the next cron job.
If you want to prevent this pull request from being auto-merged, simply leave a comment.
(If you want to post a comment without blocking auto-merging, you must include the text [noblock] in your comment.)

On a separate note, I see that you are registering a release with a version number of the form v0.X.Y.
Does your package have a stable public API? If so, then it's time for you to register version v1.0.0 of your package. (This is not a requirement. It's just a recommendation.)
If your package does not yet have a stable public API, then of course you are not yet ready to release version v1.0.0.

[noblock]


has a problem

I want to do speech recognition through the program API,
Is there a way or method to do it now? （anxious）

@gd-three No Buzz does not have this feature. If you need an api see this discussion on some APIs available in the cloud and some that you can run on your computer

@gd-three No Buzz does not have this feature. If you need an api see this discussion on some APIs available in the cloud and some that you can run on your computer

Now, I use a string in my program (buzz add --task transcribe --language zh --model-type whisper --model-size small --txt file path)CMD to perform audio recognition, which starts a new buzz process each time, which is inefficient

@gd-three Please explain what is inefficient and what you would like to be different.
If you explain more about your process, I may be able to find a more efficient solution. Where do the audio files come from and how you use the transcription results. Is it something automated?
If the transcription speed is slow, you will need a more powerful hardware or you will need to send your transcription to some API.
This should transcribe using OpenAI API, you will need a API token. Prices according to their price list.
buzz add --task transcribe --language zh --model-type openaiapi --openai-token ABC123DEF456 --txt file path

@gd-three Please explain what is inefficient and what you would like to be different. If you explain more about your process, I may be able to find a more efficient solution. Where do the audio files come from and how you use the transcription results. Is it something automated?
If the transcription speed is slow, you will need a more powerful hardware or you will need to send your transcription to some API.
This should transcribe using OpenAI API, you will need a API token. Prices according to their price list.
buzz add --task transcribe --language zh --model-type openaiapi --openai-token ABC123DEF456 --txt file path

I want to get the results of real-time speech recognition in the program I wrote myself, and then do some things of my own, but the recognition results now only have UI display, there is no storage path, if buzz can provide the path, it is great, it will solve my problem.

@gd-three Buzz stores all transcription results in a SQLite database. Location of the database is printed out in the logs. To see the logs please see this section https://github.com/chidiwilliams/buzz/blob/main/CONTRIBUTING.md#troubleshooting
On Windows you can paste this %USERPROFILE%\AppData\Local\Buzz\Buzz in the address par of your file manager and get to the location Buzz stores it's internal data.

@gd-three Buzz stores all transcription results in a SQLite database. Location of the database is printed out in the logs. To see the logs please see this section https://github.com/chidiwilliams/buzz/blob/main/CONTRIBUTING.md#troubleshooting
On Windows you can paste this %USERPROFILE%\AppData\Local\Buzz\Buzz in the address par of your file manager and get to the location Buzz stores it's internal data.

ok, I know this directory before, I use version 0.8 so I didn't see what you said

I am currently downloading version 1.1 in China very slowly, and only found version 1.0, which has a flash back in real-time recognition.
log------------------------------------------------------------------------
[2024-11-09 17:58:35,558] locale.:12 DEBUG -> UI locales ['zh-Hans-CN', 'zh-CN', 'zh', 'en-US', 'en-Latn-US', 'en']
[2024-11-09 17:58:35,561] model_loader.:40 DEBUG -> Model root directory: C:\Users\gd\AppData\Local\Buzz\Buzz\Cache\models
[2024-11-09 17:58:36,265] utils._find_ffmpeg_extension:114 DEBUG -> Loading FFmpeg6
[2024-11-09 17:58:36,266] utils._find_ffmpeg_extension:120 DEBUG -> Failed to load FFmpeg6 extension.
Traceback (most recent call last):
File "torio_extension\utils.py", line 116, in _find_ffmpeg_extension
File "torio_extension\utils.py", line 105, in _find_versionsed_ffmpeg_extension
File "", line 95, in find_spec
ModuleNotFoundError: No module named 'torio.lib'
[2024-11-09 17:58:36,266] utils._find_ffmpeg_extension:114 DEBUG -> Loading FFmpeg5
[2024-11-09 17:58:36,266] utils._find_ffmpeg_extension:120 DEBUG -> Failed to load FFmpeg5 extension.
Traceback (most recent call last):
File "torio_extension\utils.py", line 116, in _find_ffmpeg_extension
File "torio_extension\utils.py", line 105, in _find_versionsed_ffmpeg_extension
File "", line 95, in find_spec
ModuleNotFoundError: No module named 'torio.lib'
[2024-11-09 17:58:36,266] utils._find_ffmpeg_extension:114 DEBUG -> Loading FFmpeg4
[2024-11-09 17:58:36,266] utils._find_ffmpeg_extension:120 DEBUG -> Failed to load FFmpeg4 extension.
Traceback (most recent call last):
File "torio_extension\utils.py", line 116, in _find_ffmpeg_extension
File "torio_extension\utils.py", line 105, in _find_versionsed_ffmpeg_extension
File "", line 95, in find_spec
ModuleNotFoundError: No module named 'torio.lib'
[2024-11-09 17:58:36,267] utils._find_ffmpeg_extension:114 DEBUG -> Loading FFmpeg
[2024-11-09 17:58:36,267] utils._find_ffmpeg_extension:120 DEBUG -> Failed to load FFmpeg extension.
Traceback (most recent call last):
File "torio_extension\utils.py", line 116, in _find_ffmpeg_extension
File "torio_extension\utils.py", line 105, in _find_versionsed_ffmpeg_extension
File "", line 95, in find_spec
ModuleNotFoundError: No module named 'torio.lib'
[2024-11-09 17:58:36,335] settings.init:14 DEBUG -> Settings filename: \HKEY_CURRENT_USER\Software\Buzz\OrganizationDefaults
[2024-11-09 17:58:36,342] settings.init:14 DEBUG -> Settings filename: \HKEY_CURRENT_USER\Software\Buzz\OrganizationDefaults
[2024-11-09 17:58:36,342] buzz.main:60 DEBUG -> app_dir: D:\software\Buzz_internal
[2024-11-09 17:58:36,342] buzz.main:61 DEBUG -> log_dir: C:\Users\gd\AppData\Local\Buzz\Buzz\Logs
[2024-11-09 17:58:36,342] buzz.main:62 DEBUG -> cache_dir: C:\Users\gd\AppData\Local\Buzz\Buzz\Cache
[2024-11-09 17:58:36,342] buzz.main:63 DEBUG -> data_dir: C:\Users\gd\AppData\Local\Buzz\Buzz
[2024-11-09 17:58:36,353] settings.init:14 DEBUG -> Settings filename: \HKEY_CURRENT_USER\Software\Buzz\OrganizationDefaults
[2024-11-09 17:58:36,358] db._setup_db:38 DEBUG -> Database connection opened: C:\Users\gd\AppData\Local\Buzz\Buzz\Buzz.sqlite
[2024-11-09 17:58:36,364] settings.init:14 DEBUG -> Settings filename: \HKEY_CURRENT_USER\Software\Buzz\OrganizationDefaults
[2024-11-09 17:58:36,381] settings.init:14 DEBUG -> Settings filename: \HKEY_CURRENT_USER\Software\Buzz\OrganizationDefaults
[2024-11-09 17:58:36,425] file_transcriber_queue_worker.run:40 DEBUG -> Waiting for next transcription task
[2024-11-09 17:58:38,342] settings.init:14 DEBUG -> Settings filename: \HKEY_CURRENT_USER\Software\Buzz\OrganizationDefaults
[2024-11-09 17:58:38,369] backend._load_plugins:205 DEBUG -> Loading KWallet
[2024-11-09 17:58:38,369] backend._load_plugins:205 DEBUG -> Loading SecretService
[2024-11-09 17:58:38,369] backend._load_plugins:205 DEBUG -> Loading Windows
[2024-11-09 17:58:38,371] init.:11 DEBUG -> Loaded cffi backend
[2024-11-09 17:58:38,390] backend._load_plugins:205 DEBUG -> Loading chainer
[2024-11-09 17:58:38,391] backend._load_plugins:205 DEBUG -> Loading libsecret
[2024-11-09 17:58:38,391] backend._load_plugins:205 DEBUG -> Loading macOS

Is there an address in China where i can download it quickly? Or does version 1.1 still have this bug?

Is there an address in China where i can download it quickly? Or does version 1.1 still have this bug?

@gd-three If you were using the old version less than 1.0 before and then upgraded to some latest version you may get a crash on startup. Fix for this bug is to delete the old recording history (everything in the directory mentioned above) OR use the latest development version 1.2.0 from latest builds from here https://github.com/chidiwilliams/buzz/actions/workflows/ci.yml?query=branch%3Amain
Select the latest build, scroll down to the artifacts section and download the installation file. You need to be logged in the Github to download link.  This download will be slow unfortunately.

I delete the old recording history and then solve the problem by reinstalling it，thank you for your patience, for you answer.

still, Perferences---->Folder Watch，What is this feature？Does it periodically monitor folders for transcribe？

still, Perferences---->Folder Watch，What is this feature？Does it periodically monitor folders for transcribe？

I found a bug. When there were multiple audio files in the input directory, buzz would create multiple duplicate transcription tasks. After buzz transcribed an audio file, it would move it to the output directory, resulting in subsequent duplicate tasks failing to find the file during transcription.

@gd-three If you were using the old version less than 1.0 before and then upgraded to some latest version you may get a crash on startup. Fix for this bug is to delete the old recording history (everything in the directory mentioned above) OR use the latest development version 1.2.0 from latest builds from here https://github.com/chidiwilliams/buzz/actions/workflows/ci.yml?query=branch%3Amain
Select the latest build, scroll down to the artifacts section and download the installation file. You need to be logged in the Github to download link. This download will be slow unfortunately.

please see below 👇

@gd-three I am unable to replicate this on latest development version 1.2.0. There was such bug but it was fixed.
Can you please ensure you are using 1.2.0 and provide detailed steps on how to replicate.
Test 1:

Have a separate input and output folders
Have 3 files in input folder
Enable folder watch in preferences

This worked, files from input folder were processed.
Test 2:

Have a separate input and output folders
Turn off Buzz
Add files to input folder
Start buzz

This also worked, all files from input folder were processed, no duplicates and no errors.

In that case, it's okay, I'm using version 1.0.1, not developer version 1.2


Spoting memory leak on ubuntu 14.04

Compiled an envoy binary statically on a dev machine(ubuntu 14.04), and deployed it on a cloud virtual machine(ubuntu 14.04).
Spot steady cpu and memory increase until the host is unresponsive. Blue line in the below image represent CPU usage.
We have very low traffic so restart it daily is not a big deal. Just report it, and we are currently migrated to a aws host of ubuntu 16.04, will monitor it further.

Can you potentially provide more info showing that Envoy is the issue? For example 'ps aux | grep envoy', 'top', 'perf top', 'perf record', etc.
Assuming Envoy is the cause of the runaway CPU we won't be able to help debug without more information.

Thanks for your reply, sure I'll gather more info if we get this problem on the new machine.
I'm sure envoy hog up all cpu and cause problem, though I didn't note numbers at that time.

Going to close this out. Please reopen if you have more info and we are happy to help debug.


Create MSI

People are having a problem with Avast thinking the client installer is a virus.
MSIs are nicer anyway so let's try what we can do.
Hint: JavaFX comes with a packager, which is supported by IntelliJ (but I prefer having something that runs from Gradle)

Want to back this issue? Post a bounty on it! We accept bounties via Bountysource.

Do we still want this or will we just stick with intellij?

We can drop this; IIRC MSI doesn't support several features that Install4J odes, some of which are vital to us.


Monitoring Docs: "some_query" syntax

@mnencia
this it the PR following the slack discussion https://cloudnativepg.slack.com/archives/C03D68KGG65/p1684927192220059?thread_ts=1684924144.695639&cid=C03D68KGG65
so that the "some_query" query can be copy/pasted into the example-monitoring configMap
br
Daniel
Signed-off-by: Daniel Chambre<EMAIL_ADDRESS>

@smiyc can you please fix the DCO error reported by the CI?

@mnencia
i rebased my repo but now there are other's commits included.
Im not familiar with git and synced the changes done to the cloudnative-pg:main into my :main.
I guess the easiest  woulb be, if I close this PR and start from scrach?

...starting with a clean branch


Remove unused v_tangent

It looks like v_normal is also unused…

Thanks, I will clean up v_normal.


Invalid fix for PLE1141

I think the autofix is not safe
Ruff version: 0.6.4
Ruff command: ruff check --preview --fix

d: dict[tuple[int, str], bool] = {}
d[1, "a"] = True

for n, c in d:
    print(n, c)

autofixed to
d: dict[tuple[int, str], bool] = {}
d[1, "a"] = True

for n, c in d.items():
    print(n, c)

To summarize the issue:

for n, c in d iterates over the dictionary keys
for n, c in d.items() iterates over the key-value pairs.

It's questionable if PLE1141 should be raised for your example because the code only iterates over the keys. Pylint had a similar issue a long time ago https://github.com/pylint-dev/pylint/issues/3283

Yes exactly. But as I understand it, ruff doesn't use type checking, so it would be difficult to say that the key is a tuple.

It seems like the fix for this should at the very least be marked as unsafe, since in general it will be impossible to determine if the key is a tuple, even if support was added for inferring it in obvious cases.


ENH: Improve Resampler typing

[ ] Tests added: Please use assert_type() to assert the type of any return value

When this goes green, ping me to review

When a function runs any index type, and I just use -> Index?

When a function runs any index type, and I just use -> Index?

Not sure what you mean here???  If you're asking how to type a function that returns any Index type, then use ->Index, but if the function has the index type as an argument, and returns an Index of the same type, you can use a TypeVar on the parameter so it returns the same type.

Not sure what you mean here??? If you're asking how to type a function that returns any Index type, then use ->Index, but if the function has the index type as an argument, and returns an Index of the same type, you can use a TypeVar on the parameter so it returns the same type.

I meant the latter.  Basically it will return a subclass of Index, e.g., DatetimeIndex.


Fix pilot grpc failure in Consul

Fix issue described in #12024.

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

Needs approval from an approver in each of these files:

install/OWNERS

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

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

Needs approval from an approver in each of these files:

install/OWNERS

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

@wenchenglu @duderino

merging as this change is not tested by any of our tests..

@pitlv2109: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-pilot-multicluster-e2e.sh
b484f505d58cd816ebba3080c4bc825f83c5d291
link
/test istio-pilot-multicluster-e2e

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@pitlv2109: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-pilot-multicluster-e2e.sh
b484f505d58cd816ebba3080c4bc825f83c5d291
link
/test istio-pilot-multicluster-e2e

prow/istio-integ-k8s-tests.sh
b484f505d58cd816ebba3080c4bc825f83c5d291
link
/test istio-integ-k8s-tests

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.


Code cleanup

Initial cleanup for code after leaving untouched for a while. A few changes include:

Refactoring all anti-aliasing logic into separate classes, with separate anti-aliasing methods.
Using Immutables for several Pojos.
Migrating raw thread manipulation to an ExecutorService.
Refactoring the project/package structure.
Refactoring out several classes.
Fixing a rendering bug that caused a jagged line of pixels for a diffuse lighting edge-case.

Finally! Hope this is backwards compatible with earlier versions

From: Kahlil Oppenheimer<EMAIL_ADDRESS>Sent: Tuesday, December 25, 2018 10:57 AM
To: kahliloppenheimer/Java-Ray-Tracer
Cc: Subscribed
Subject: Re: [kahliloppenheimer/Java-Ray-Tracer] Code cleanup (#1)
Merged #1https://github.com/kahliloppenheimer/Java-Ray-Tracer/pull/1 into master.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHubhttps://github.com/kahliloppenheimer/Java-Ray-Tracer/pull/1#event-2043920861, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AF-DnI6puSlALYmLbgqkUhR2aJdbLJ1Tks5u8lkTgaJpZM4ZhKPu.


Page lacks crucial info

This page: https://docs.microsoft.com/en-us/microsoft-365/admin/support/estonia?view=o365-worldwide lacks information on how to add organisations to my account. Lets say I start a new business. I want to have a new OneDrive for business for that. How to achieve having a new organisation?
br,
Kaido

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

ID: 22287d6c-4733-ed93-05db-9b46f2a6fe28
Version Independent ID: 1c66ac29-6277-4c3b-0ada-6ba0127808cd
Content: Support information for Estonia - Microsoft 365 admin
Content Source: microsoft-365/admin/support/estonia.md
Service: o365-administration
GitHub Login: @cmcatee-MSFT
Microsoft Alias: cmcatee

@kaidoloor The page you referenced is only intended to provide support contact information for Estonia. For adding a new organization, you'd have to look at the overall admin content here.
If you bought your subscription directly from the Microsoft web site, you can't add a new organization to your account. You need to set up a new account and buy a new subscription for it. If you bought your subscription through Microsoft Sales, contact your Microsoft representative to set up a new organization within your Microsoft account.


Add SQL functions CURRENT_DATE, CURRENT_TIME and CURRENT_TIMESTAMP

I guess you can also just use std::chrono instead but with those functions (which are also standardized) it might feel more like SQL.
-Jonas

Hi Jonas,
Thanks for adding these functions. Seems very reasonable to me.
Would be great if you could add serialization tests, e.g. similar to https://github.com/rbock/sqlpp11/blob/main/tests/core/serialize/Upper.cpp
Note: In the optional-no-dynamic branch (major overhaul), I decided to drop the _auto_alias_t. We might want to drop it for newly added functions before adding them. WDYT?
Cheers,
Roland

Hi Roland,
I've added some basic serialization tests, let me know if there's more to test but I guess the functions themselves are very basic as well, so I couldn't think of anything else.
I've mostly looked at the trim()-function, that's where the _auto_alias_t came from but I agree that it doesn't seem necessary.
-Jonas
On another note: I've thought about adding more date and time functions, namely year_of, month_of, ... (or maybe extract instead of those), datediff, date_add, date_sub. Is that something you'd be willing to merge or do you think this should rather be done in application code? One problem with this is not being standardized functions of course.
If you're interested I'd create an issue for it to discuss design details.

Regarding the proposed additional functions: Please open a separate issue.

Hi Jonas,
Thanks for the cleanup. This looks good to me.
Merging...
Cheers,
Roland


Skip DANDI upload to allow tests to pass with broken external services

Skip the DANDI upload part of the E2E tests to avoid failing when external dependencies (e.g. the DANDI server) has an issue.

What I'm requesting is that we split the current testing action into two actions; one that runs only the live service tests like DANDI upload, and one that runs only the non-live service tests that we can control
These would show up as two separate CI runs on a PR, and yes the live one would continue to fail until DANDI fixes their end

Sure. Though since this DANDI upload test appears at the very end of the E2E workflow and requires a complete pipeline to finish, would it be sufficient to simply duplicate the testing.yml + keep the DANDI_STAGING_API that was removed here?
We could cache a complete pipeline instead, though that'll take a bit longer.

Alright added that extra testing file


Needs nativeaot compatibility

Although system.text.json is compatible with json, it seems that very few projects use it because it is unfamiliar and only available for dotnet core
a lot of projects use this but because this is not trimmed it has a lot more size compared to the newer library
i wish someone can make it compatible with native aot, or show me how to reduce the compile size with this library

I believe that (with apologies if my knowledge of newtonsoft.json is not up to date:
There are aspects of this library that are compatible with nativeaot (e.g. using JsonObject constructors etc. to serialize and deserialize).
There are also aspects that are not compatible (e.g. taking a dotnet class and using reflection to serialize and deserialize it).
This library then would need to be annotated for trimming. E.g. DynamicallyAccessedCode annotations on Serialize<'T>. Then consumers of the library will not see warnings unless they use the problematic code.


Line graph overlaps labels on other series on mixed series

Version
5.2.0
Reproduction link
https://jsfiddle.net/rwb58gkv/1/
Steps to reproduce
Add a 2 bar series and a line series.
What is expected?
Labels to be on top of all graphs.
What is actually happening?
Line graph overlaps the labels the bar graph labels.

The default z2 value in the line chart seems to be too large here. https://github.com/apache/echarts/blob/master/src/chart/line/LineView.ts#L945
We can change it to 1 so it won't overlap labels.

The default z2 value in the line chart seems to be too large here. https://github.com/apache/echarts/blob/master/src/chart/line/LineView.ts#L945
We can change it to 1 so it won't overlap labels.

Sorry for the late reply. I'll try this out. thanks.

Hi @pissang
I would like to try to fix it, could you guide me to approach this issue?

@stu01509 Hi, you can try the solution I mentioned ealier

The default z2 value in the line chart seems to be too large here. https://github.com/apache/echarts/blob/master/src/chart/line/LineView.ts#L945
We can change it to 1 so it won't overlap labels.


Arnej/minor codec fixes

I confirm that this contribution is made under the terms of the license found in the root directory of this repository's source tree and that I have the authority necessary to make this contribution on behalf of its copyright owner.
@havardpe please review

@geirst @havardpe please take another look


Allow devise users to switch back to the first authenticated user.

Helpers to know who the original user and his scope was.
Helper to generate the path to switch_back properly.

I last worked on this several years ago but find myself needing this again. The merge will not be clean, so I'll have to rebase and see if I can make it work with the current version of switch_user, but I wanted to at least give you a heads up on this idea.

Any news on this??

we already have a switch back feature, close it


Exceptions in the static analysis module

I successfully built the project and successfully generated GoalExplorer-1.2-SNAPSHOT-jar with dependencies. jar under "target".
However, I encountered a problem executing the following command from the readme file.
Command:
java -jar target/GoalExplorer-1.2-SNAPSHOT-jar-with-dependencies.jar ge -i /home/xxx/AndroidTools/GoalExplorer-main/ExperimentSubject/org.tomdroid_14.apk --target "act:org.tomdroid.ui.PreferencesActivity" -s /home/xxx/Android/Sdk -o /home/xxx/AndroidTools/GoalExplorer-main/GoalExplorer/sootOutput
Error reporting information：

I would be grateful if you can help me.

Hello,
Thank you for bringing this to our attention! The latest commit should fix this issue.

Thanks for your help.


Zoom level

It would be great to be able to adjust the zoom level. Is that possible? I am building an app that needs to scan small QR code that can be at a distance as well. Ability to zoom is a necessity it seems.

Version 2.0.3 include a zoom component.


trycourier => courier

Using Python3.11 on RH
Trying to use __login__ throws an error:
2024-05-04 08:12:24.438 Uncaught app exception
Traceback (most recent call last):
  File "/home/riccardo/Repos/streamlit/venv/lib64/python3.11/site-packages/streamlit/runtime/scriptrunner/script_runner.py", line 584, in _run_script
    exec(code, module.__dict__)
  File "/home/riccardo/Repos/streamlit/app.py", line 6, in <module>
    from streamlit_login_auth_ui.widgets import __login__
  File "/home/riccardo/Repos/streamlit/venv/lib64/python3.11/site-packages/streamlit_login_auth_ui/__init__.py", line 1, in <module>
    from streamlit_login_auth_ui.widgets import __login__
  File "/home/riccardo/Repos/streamlit/venv/lib64/python3.11/site-packages/streamlit_login_auth_ui/widgets.py", line 7, in <module>
    from .utils import check_usr_pass
  File "/home/riccardo/Repos/streamlit/venv/lib64/python3.11/site-packages/streamlit_login_auth_ui/utils.py", line 3, in <module>
    from trycourier import Courier
ModuleNotFoundError: No module named 'trycourier'

Installing trycourier itself does not solve the issue,
Changing:
streamlit_login_auth_ui/utils.py", line 3

from trycourier import Courier => import courier seems to resolve the issue.
Considering that this repo is over two years old and that the README tells us to install specific versions of requirements.txt i'm hoping you can find the time to do a rehash of the sourcecode to 2024. I'd try it too, but im not proficient enough.

Replace the line from trycourier import Courier with from courier.client import Courier.
Then, execute the command pip install argon2-cffi in your terminal to support the use of from argon2 import PasswordHasher.
After these adjustments, your application should run without any issues.

Replace the line from trycourier import Courier with from courier.client import Courier.
Then, execute the command pip install argon2-cffi in your terminal to support the use of from argon2 import PasswordHasher.
After these adjustments, your application should run without any issues.

thanks a lot this fixed everything

I also encountered this problem. When the problem will be fixed?

I also encountered this problem. When the problem will be fixed?

You can try the version I forked
https://github.com/Rycen7822/streamlit_login_auth_ui_zh


Some species (species not involved in rate equations?) aren't recorded

Since there doesn't appear to be a way to record the values of non-constant parameters (e.g., needed for some plots in BioModels), I tried to see if I could use an assignment rule to record the values of non-constant parameters (i.e., assign the value of a parameter to a species).
Example: dNFAT (parameter) and dNFAT_species (species) in BIOMD678.xml.txt.
However, PySCeS didn't record the value of dNFAT_species. One possible hint I see is
Info: "dNFAT_species" has been initialised but does not occur in a rate equation

I have two questions:

Is there a direct way to record the values of non-constant parameters?
Is there a way to record the values of such species set via assignment rules? i.e., is there something else I can do to trigger PySCeS to record their values?

This has now been fixed in Release 1.0.2. Assignment rules are tracked by default in a simulation with CVODE, and the values are available via mod.data_sim.getXData(). Refer to https://pyscesdocs.readthedocs.io/en/latest/userguide_doc.html#simulation-results
The change was implemented in https://github.com/PySCeS/pysces/pull/77/commits/8ebbcb326d205e5818e4a7a350eb645c11d83c0e with additional fixes in https://github.com/PySCeS/pysces/pull/77/commits/e9c4444ae2b525cfb32c1e0192eea4b3e20577fd.


Build errors in theories/Mon/FiniteProbabilities.v

I've been trying to get this working in a Docker container, using the following Dockerfile:
FROM ubuntu:latest

RUN DEBIAN_FRONTEND=noninteractive apt-get -y update
RUN DEBIAN_FRONTEND=noninteractive apt-get -y install build-essential software-properties-common libgmp-dev graphviz
RUN add-apt-repository ppa:avsm/ppa
RUN DEBIAN_FRONTEND=noninteractive apt-get -y update
RUN DEBIAN_FRONTEND=noninteractive apt-get -y install opam

RUN useradd -ms /bin/bash ssprove
USER ssprove
WORKDIR /home/ssprove

RUN opam init --disable-sandboxing --enable-shell-hook --shell-setup --enable-completion

RUN opam -y pin add coq 8.14.1
RUN opam -y repo add coq-released https://coq.inria.fr/opam/released

RUN git clone https://github.com/SSProve/ssprove
RUN opam -y install ssprove/ssprove.opam || echo Fail # <-- Fails here.

CMD ["/bin/bash", "--login"]

I've modified the file so it still manages to build the Dockerfile even if the build fails. (The || echo Fail part.)
It fails when building SSProve in the theories/Mon/FiniteProbabilities.v, specifically the obligations for addI, mulI, negI, barycentric_sum and wopProb. I've noticed adding simpl. intros. to the first four, and just intros. to the last fixes it, but I imagine there might be a better fix, hence why I haven't submitted a pull request.

Can you give us the error message in question?

For addI the goal is
forall x y : I, (fun r : R => is_true (0 <= r <= 1)) ((x ∙1 + y ∙1) / 2%:~R)

and the error is
The LHS of divr_ge0
    (0 <= _ / _)
does not match any subterm of the goal

It is a similar story with the rest of them: The function parameters aren't in the context, and (for all but wopProb) the goal is expressed as (fun r : R => is_true (0 <= r <= 1)) BODY.

Which versions of mathcomp-analysis and mathcomp does it install? I guess that might be it.

opam list gives the following:
base-bigarray          base
base-threads           base
base-unix              base
camlp5                 7.14        Preprocessor-pretty-printer of OCaml
conf-findutils         1           Virtual package relying on findutils
conf-gmp               4           Virtual package relying on a GMP lib system installation
conf-perl              2           Virtual package relying on perl
coq                    8.14.1      pinned to version 8.14.1
coq-deriving           0.1.0       Generic instances of MathComp classes
coq-elpi               1.11.2      Elpi extension language for Coq
coq-equations          1.3+8.14    A function definition package for Coq
coq-extructures        0.3.1       Finite sets, maps, and other data structures with extensional reasoning
coq-hierarchy-builder  1.2.1       High level commands to declare and evolve a hierarchy based on packed classes
coq-mathcomp-algebra   1.13.0      Mathematical Components Library on Algebra
coq-mathcomp-analysis  0.3.13      An analysis library for mathematical components
coq-mathcomp-bigenough 1.0.1       A small library to do epsilon - N reasoning
coq-mathcomp-field     1.13.0      Mathematical Components Library on Fields
coq-mathcomp-fingroup  1.13.0      Mathematical Components Library on finite groups
coq-mathcomp-finmap    1.5.1       Finite sets, finite maps, finitely supported functions
coq-mathcomp-solvable  1.13.0      Mathematical Components Library on finite groups (II)
coq-mathcomp-ssreflect 1.13.0      Small Scale Reflection
cppo                   1.6.8       Code preprocessor like cpp for OCaml
dune                   2.9.3       Fast, portable, and opinionated build system
elpi                   1.13.8      ELPI - Embeddable λProlog Interpreter
ocaml                  4.12.1      The OCaml compiler (virtual package)
ocaml-base-compiler    4.12.1      Official release 4.12.1
ocaml-compiler-libs    v0.12.4     OCaml compiler libraries repackaged
ocaml-config           2           OCaml Switch Configuration
ocaml-options-vanilla  1           Ensure that OCaml is compiled with no special options enabled
ocamlfind              1.9.3       A library manager for OCaml
ppx_derivers           1.2.1       Shared [@@deriving] plugin registry
ppx_deriving           5.2.1       Type-driven code generation for OCaml
ppxlib                 0.24.0      Standard library for ppx rewriters
re                     1.10.3      RE is a regular expression library for OCaml
result                 1.5         Compatibility Result module
seq                    base        Compatibility package for OCaml's standard iterator type starting from 4.07.
sexplib0               v0.14.0     Library containing the definition of S-expressions and some base converters
ssprove                dev         pinned to version dev at git+file:///home/ssprove/ssprove#main
stdlib-shims           0.3.0       Backport some of the new stdlib features to older compiler
zarith                 1.12        Implements arithmetic and logical operations over arbitrary-precision integers

Ok, for me analysis is 0.3.11 and you have 0.3.13. It seems they're usually not backwards compatible so I will change it with an equal sign I guess. And I'll work on an update to the latest. Thanks for the report.

I've pushed a quick fix, can you confirm that it works now?

It does.


Added debug tag

Added debug tag to print info for debugging purposes.

@ourarash Awesome! I haven't even noticed that console.debug existed, shame on me! It's merged and available in 1.1.129.

@xpl Thanks! Ideally, there should be a flag (or a level) that controls the debug/warning print out. For example, when the flag is disabled, there will be no debug info printed out. I might add it later.


TEST: Move from Travis to Github Actions for testing

Travis has just announced a change in their pricing structure, and no longer offer free builds for open source projects. https://blog.travis-ci.com/2020-11-02-travis-ci-new-billing
We should switch to another CI platform (maybe Github Actions).

👍 for this! 😂


Information about automatic remediation of new resources

Hi,
I expected to find some information about when and when not non-resources are automatically remediated after a policy with effect DeployIfNotExists or Modify was assigned.
On the What is Azure Policy? page, first section it says " It also helps to bring your resources to compliance through bulk remediation for existing resources and automatic remediation for new resources."
KR
Andreas

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

ID: 99d433a1-4e81-e226-e835-639daa90dab4
Version Independent ID: ab3aa1ad-f438-e2a1-5556-9f9b6269ba76
Content: Remediate non-compliant resources - Azure Policy
Content Source: articles/governance/policy/how-to/remediate-resources.md
Service: azure-policy
GitHub Login: @georgewallace
Microsoft Alias: gwallace

@rncwnd79  - Thanks for bringing this to our attention.
I'm going to assign this to the document author so they can update the document accordingly.

Hi @rncwnd79, thanks for raising this issue. I'll add a link that explains how Azure Policy handles remediation to the "What is" page you referenced. This is the page you might be interested in: Remediate non-compliant resources with Azure Policy
Thanks, Tim


derive: allow supplying source in comment

Using #[template(source = "…")] is tiresome, because you have to escape the input. That makes it difficult to read, too. Using #[template(path = "…") can be tiresome, too, if the code is small and/or if you just want to prototype something.
This PR adds a third option to supply the source of a template: You can supply the source in the comments of a template. This works similar to doctest. Any rinja block gets extracted, and the combined blocks are the source code of the template.

I like the idea! Don't forget to test with /** and #[doc = ""] too. :wink:

I like the idea! Don't forget to test with /** and #[doc = ""] too. 😉

Oof, yeah, my code did not work at all with these two. :) Fixed.

Does it work with code examples like:
///     let x = 12;
///
/// `````
/// let x = 12;
/// `````

?
If not, I'd recommend using pulldown-cmark to parse the markdown and extract code blocks from it instead of trying to do it yourself. ;)

It's just one of many. Not sure if your code would handle:
/// `````
/// ```rinja
/// {{bla}}
/// ```
/// `````

correctly

It's just one of many. Not sure if your code would handle:

It does. I had to turn one panic into an error so I could add the test.

I use pulldown-cmark, now. It has a different opinion how to interpret ↓, but I guess it knows it better. :)
/// ```rinja
/// `````
/// {{bla}}
/// `````
/// ```

Should we wait until https://github.com/pulldown-cmark/pulldown-cmark/pull/916 is released to keep our MSRV?

Oh that'd be convenient. Another thing I thought about: should we make it an optional feature?

Another thing I thought about: should we make it an optional feature?

Yes, good call! It will probably a very niche feature. What would you call the feature flag? Is "code-in-doc" too long?

code-in-doc sounds good to me. :+1:

Oh btw I asked on the pulldown-cmark PR and they said hopefully there should be a release this week. :)

New pulldown-cmark version with reduced MSRV has been published.

error: package pulldown-cmark v0.11.1 cannot be built because it requires rustc 1.71.1 or newer, while the currently active rustc version is 1.71.0

I guess it's acceptable if I raise our MSRV by 0.1. :D

You also need to update the book. :)

You also need to update the book. :)

Added. :)

Hm, should #[template(in_doc = false)] succeed if the feature is not implemented? I guess yes?

I'd say yes.

All suggestions applied.

Thanks a lot, it's a very cool feature! :)

Thanks! And thank you for your reviews and suggestions! :)


Can't install v1.5.0

When I try to install v1.5.0, this happens:
kyles-mbp:sandc kyle$ npm install --save-dev tyson
npm ERR! code ENOENT
npm ERR! syscall chmod
npm ERR! path /Users/kyle/sand-lang/sandc/node_modules/tyson/dist/cli.js
npm ERR! errno -2
npm ERR! enoent ENOENT: no such file or directory, chmod '/Users/kyle/sand-lang/sandc/node_modules/tyson/dist/cli.js'
npm ERR! enoent This is related to npm not being able to find a file.
npm ERR! enoent

The problem seems to come from deriving tsconfig.production.json from tsconfig.json.
https://github.com/kylejlin/tyson/commit/93ee9b00c3f45477b223b6b45081b2b90baec8a9#diff-114f008a616ab7e9a7cbbd773820deafL16
I forgot to override the "include" field. Consequently, both src and lib are being emitted into dist, so the cli.js is located at dist/src rather than dist.


label sizes on colorbars and labels running off the page

This has been an issue going back to older versions of Ferret and Pyferret. When there's limited space at the plot edge for a vertical colorbar the labels may run off the page despite code that tries to adjust the label sizes.  If the levels are all non-negative they're ok.

yes? use etopo60
yes? set view ur; shade rose
!  If the levels are all non-negative they're ok.
yes? set view lr; shade  if rose gt 0 then rose

Another issue, in PyFerret only, if the default Hershey font is used, the labels on horizontal colorbar are invisibly tiny.
> pyferret

yes? use etopo60; shade/key=horiz rose

! Use any system font, then they're ok
yes? set text/font=sans_serif
yes? use etopo60; shade/key=horiz rose

This is checked in and merged to the PyFerret branch where the horizontal colorbar issue is also fixed.


Automating stop/start of VMs

Describe the bug
I am trying to stop multiple VMs (over 700+) using a bash script and Jenkins to orchestrate the schedule. I am using az vm start/deallocate -n vm -g rg --no-wait. When it gets to about VM # 200. Azure informs me my environment is being throttled and this causes a lot of VMs to not stop. A lot of the VMs also start in failed state. Cli should be able to handle api calls using the --no-wait option.
Code snippet
echo -n "VM count: " $(cat vm.txt | wc -l)
echo

if [[ -s vm.txt ]]; then
    echo "Using resources from vm.txt..."
    echo
else
    echo "Didn't generate any VMs in the file."
    echo "Sorry, no resources to work with..."
    echo
    exit 1
fi

echo "VM_Name		RG"
echo "----------------------------------"
cat vm.txt
echo
echo

while read VM RG; do
    if [[ "${OPTIONS}" == "stop" ]]; then
        echo "Stopping VM: $VM..."
        az vm deallocate -g $RG -n $VM --no-wait
        #PS=$(az vm show -g $RG -n $VM --show-details --query powerState -o tsv | tr -d '"')
        STATUS="deallocated"
    elif [[ "${OPTIONS}" == "start" ]]; then
      	echo "Starting VM: $VM..."
        az vm start -g $RG -n $VM --no-wait
        #PS=$(az vm show -g $RG -n $VM --show-details --query powerState -o tsv | tr -d '"')
        STATUS="running"
    else
        echo "Try again..."
        exit 1
    fi

done <vm.txt

To Reproduce
Try deallocating the same amount of VMs as I am in parallel.
Expected behavior
Api calls should be handled by the --no-wait option. It should sleep for some time before moving to the next VM in the loop.
Environment summary
Install Method (yum) / CLI version (azure-cli (2.0.44)) / OS version (RHEL 7.5) / Shell Type (bash)
Additional context
Add any other context about the problem here.

The only solution that comes to mind would be to have the CLI wait and retry on a throttling error, which would be bizarre since you explicitly called --no-wait.
My advice would be for your script to track the failures, requeue them and retry until they all succeed, or introduce some delay between calls to avoid the throttling.

@tjprescott I ended up introducing a 5 seconds delay for the while loop. I've been running this for about 2 days now without any issues. Only thing is my 30 mins Jenkins job now takes 1hr 30.

@Fumes007 you can try opening a support ticket on the portal to try and increase your API bandwitdth--this is a service issue. I think the retry/requeue strategy would probably work best for you, though I'd do that in a Python script instead of bash.

@tjprescott not too familiar with using python with Azure. I have some snippet here:
`try:
vmlist = open("vmlist.txt","r")
for vm in vmlist.readlines():
    values = vm.split()
    if OPTION in ['stop', 'deallocate']:
        # Stop the VM
        print('\nDeallocating the VM')
        async_vm_deallocate = compute_client.virtual_machines.deallocate(GROUP_NAME, VM_NAME)
        async_vm_deallocate.wait()

    elif OPTION in ['start']:
        # Start the VM
        print('\nStart VM')
        async_vm_start = compute_client.virtual_machines.start(GROUP_NAME, VM_NAME)
        async_vm_start.wait()

    else:
        print('\nDoing nothing. Select either start or stop")
        exit()

finally:
vmlist.close()`
Issues: I want to be able to read the VM names/rgs from a file and use that info to start/stop. I would also consider using vm id if thats easier or querying by tags

You'd be better served asking here: https://github.com/Azure/azure-sdk-for-python


Typescript: If an event handler whose event type is MouseEvent|TouchEvent, how do you get it to not throw errors if one of them does not have a certain property?

Using Preact 8.4.2.
Example:
handleDragStart = (e: MouseEvent|TouchEvent) => {
   const val = e.constructor.name === "TouchEvent" ? e.touches[0].pageX : e.pageX;
};

In the above, an error is thrown for e.touches[0].pageX:
Property 'touches' does not exist on type 'MouseEvent|TouchEvent'.
Property 'touches' does not exist on type 'MouseEvent'.

and for e.pageX:
Property 'pageX' does not exist on type 'MouseEvent|TouchEvent'.
Property 'pageX' does not exist on type 'TouchEvent'.

In React, you can use event.nativeEvent to remove this error (see Stackoverflow answer) but no such thing exists in Preact.
What would you do in Preact to get the equivalent behavior?

yes we don't have event.nativeEvent because event is event.nativeEvent 👀

In Preact the event itself is the native event. We don't ship a custom event system like react does.
handleDragStart = (e: MouseEvent|TouchEvent) => {
  if (e instanceof TouchEvent) {
    // I'm a touch event
  } else {
    // I'm not a TouchEvent
  }
};

Oh duh!  I completely forgot Preact doesn't use Synthetic Events.  I should read the docs again more carefully.
Thanks so much!

Also if you're sure it exists, you can tell typescript by using e.touches[0].page!, but you won't need that


Re-using a did:web handle on a new did:web fails

Steps to Reproduce
If I try to re-use a handle that was already associated with a did:web account (but that account is deleted), Bluesky seems to hang onto the old record.
This should be returning did:web:yot.wtf, but its returning the previous did:web:ducky.ws (on both DNS and /.well-known/atproto-did entries): almost as if its caching the value and never attempting to retry it.

Attachments
No response
What platform(s) does this occur on?
Web (Desktop)
Device Info
No response
What version of the app are you using?


"Configure Deployment Source" should be on "Deployments" tree item

Currently "Configure Deployment Source" lives in the web app tree item:

However it feels more natural on the Deployments tree item:

I'm not sure we want to remove it from the web app tree item since some people are used to it there, but I figure we could at least add it to Deployments.

Agreed. This seems especially necessary once #753 is implemented.


reporting with -V option

it will be perfect if it could use the -V option in hledger. That way you can check all your investments.

Thanks for the suggestion!
I haven't added anything related to investments yet, but this might be a good place to start. Could you provide a sample journal file that can be used for development and testing?

Main_.txt
i think this can work.

You can now add custom options :)

how can i use custom options?

You can use custom options by using
puffin -cfg config.json

And in the config.json file, you can setup your own reports.
Based on your comment in the other issue, I assume you have already figured out this part. :)

yes. sorry to bother. i figured out like you said :)


authorizedCallback Validation, iat rejected id_token was issued too far away from the current time

We are facing this issue, in our installation.
angular-auth-oidc-client.js:393 authorizedCallback Validation, iat rejected id_token was issued too far away from the current time
Searching on google we find that is a bug in auth client
https://github.com/damienbod/angular-auth-oidc-client/issues/175
Fix on version v11
https://github.com/damienbod/angular-auth-oidc-client/releases

Id_Token is rejected because of timing issue when server hour is different then client hour
fix validate, fix max time offset #175

Anyone facing the same issue? possible workaround ?
There is a plan to do the upgrade?

Hi Brocy, so far we only had one user who had this issue on a MAC but when he changed to Windows PC the error went away ... not sure what caused it honestly but we are looking into it

Thanks for replaying, we end up making the upgrade if you guys want I can submit a pull request.

Would be great ! Thanks in advance !

Thanks for the PR, we will check it as soon as possible

fixed with 1.0.2

looks like this issue still exists. Anyone experiencing the same problem? I'm using Azure AD.


Edit Custom cluster shows all drivers nodes instead of only custom code

Rancher versions: 2.0 master 2/26
Steps to Reproduce:

Create a custom cluster
Edit custom cluster

Results:  All node driver are showing up in Edit.

@vincent99 how to distinguish custom cluster from the cluster created by node driver?

Version 2.0 master 3/26
Verified fixed


Psuedo targets are added to the "Select Debug Console" dropdown when an attached JS target is restarted

Does this issue occur when all extensions are disabled?: Yes

VS Code Version: 1.75.1
OS Version: MacOS Ventura 13.2 (22D49)

I'm using VS Code to debug an attached Node app. The problem is that each time that I restart the Node app, new pseudo targets are added to the "Select Debug Console" dropdown of the debug console.
Problem started in 1.75.0. In prior versions, only the "Server" target would appear after restarts (I'm debugging in multiple target environment, so the dropdown is always visible). Since 1.75.0 (and .1), two new targets are added to this list (webpack + server.bundle.js) after each restart. Here's how this dropdown looks like after 3 restarts:

A similar issue happens when attempting to debug a React app - each reattach adds a new pseudo target to the list.
The targets that are added to the debug dropdown are "stale" - switching to one of these pseudo items shows an empty debug console.
Steps to Reproduce:

Start debugging a JS app that is connected via the attach debug mode
Open the debug console
Restart the app using the "restart" icon on the debug panel
Click on the Select Debug Console dropdown, and observe new pseudo targets have been added to the list (and on each subsequent restart)

I tested the same configuration with VSCode 1.74.3, and the problem does not appear.

I noticed this, I haven't looked into it at all but wondering whether it's related somehow to https://github.com/microsoft/vscode/pull/169074 @connor4312

This is a pain, I often restart my server process and I just see this slowly building list of old server processes. I don't need to keep their logs behind, it just clogs the doprdown.

ignore my last comment... I found the setting Debug › Console: Close On End was 'off' by default. Turning it on resolved the problem

Yea this looks expected since we have a process ID as part of the session name, so they get unique terminals. Could argue we shouldn't... but we do, and I've found it handy on more than one occasion

@connor4312 @jpike88 The problem persists after turning on Debug › Console: Close On End (on version 1.75.1). When the said option is on, the terminal/console pane indeed closes when debug stops, but when I restart the process, the pane re-shows with the "select debug console" dropdown populated with the terminated processes.
In any case, the problem surfaced in 1.75.0 so I still think there's a bug (or undocumented fix) that contributes to this issue. May I suggest to re-open the issue?


[CLOSED] Enable email subscription on ESIF finder

 Issue by binaryberry
Tuesday 07 July 2015 at 13:29 GMT
Originally opened as https://github.com/alphagov/specialist-publisher/pull/516

This activates email subscriptions, and links them to the content_id of the three organisations related to  ESIF:

Department for Environment, Food and Rural Affairs
Department for Business, Innovation and Skills
Department for Communities and Local Government

Deploy will require running rake publishing_api:publish_finders to publish finders to publishing-api.
cc @boffbowsh
Story on Trello

binaryberry included the following code: https://github.com/alphagov/specialist-publisher/pull/516/commits

 Comment by boffbowsh
Tuesday 07 July 2015 at 13:34 GMT

Is this line missing? https://github.com/alphagov/specialist-publisher/pull/407/files#diff-71c144586d8931afc1a189b15c7af1a4R9

 Comment by binaryberry
Tuesday 07 July 2015 at 14:07 GMT

@boffbowsh : no, that's the old way to do it! So February 2015.
See current version of cma-cases finder: https://github.com/alphagov/specialist-publisher/blob/master/finders/metadata/cma-cases.json


LuaObject's Call metamethod is not passed in Self

When I use LuaObject
[LuaMetamethod(LuaObjectMetamethod.Call)]
public void Func() {}
to declare the meta method
object() call,
error: bad argument #1 to 'call' (value expected)
It's going to be object(object)

Could you please provide more detailed Lua/C# code? This comment alone doesn't allow us to pinpoint the exact cause.


CI: Restore caches in sync

Requirements for Adding, Changing, or Removing a Feature

Fill out the template below. Any pull request that does not include enough information to be reviewed in a timely manner may be closed at the maintainers' discretion.
The pull request must contribute a change that has been endorsed by the maintainer team. See details in the template below.
The pull request must update the test suite to exercise the updated functionality. For guidance, please see https://flight-manual.atom.io/hacking-atom/sections/writing-specs/.
After you create the pull request, all status checks must be pass before a maintainer reviews your contribution. For more details, please see https://github.com/atom/.github/tree/master/CONTRIBUTING.md#pull-requests.

Issue or RFC Endorsed by Atom's Maintainers

Discussed here: https://github.com/atom/atom/pull/21057#issuecomment-680170267
Description of the Change

To simplify how our CI handles caches, restore all three node_modules folders in sync with one-another.

Track the same files for restoring all three caches. Either they all get restored, or none of them do.
Stop tracking apm/package-lock.json, as it hasn't been stable lately.
Start tracking script/vsts/platforms/templates/preparation.yml, because that's where Node and npm bumps and so on live.

Alternate Designs

Independent cache restores (which we're doing as of before this PR), or maybe more-optimistically restoring apm since it gets native code rebuilt during install anyhow.

Independent cache restores make it hard to track where restored files came from. Also, all the node_modules folders other than apm/node_modules are deleted by npm ci, so restoring them is just wasted time.

Verdict: no.

Optimistically restoring apm/node_modules might be interesting, since it is installed with npm install, not npm ci; It's not deleted during bootstrap. But that's one more thing to test if it works well, so I'm considering it out of scope for this PR.

Verdict: maybe later.

Possible Drawbacks

Reduces the scenarios in which the apm/node_modules cache will be restored. Could add a few minutes to bootstrapping.
Verification Process

Tested in my personal fork's CI. Should work here.
(To verify: watch that the macOS and Windows test jobs restore all three caches, skip bootstrapping, and successfully proceed on to run their tests.)
Release Notes

N/A

For the history of things I tried, here's my personal branch for researching and developing this PR, with all the messy intermediate commits: https://github.com/DeeDeeG/atom/compare/atom-ide-community-master...DeeDeeG:CI-synced-caches

CI failures look unrelated to me, see #117
I have run CI 10 times now, and the Windows renderer2 tests consistently fail, as described in that issue.
This PR's changes themselves appear to be working quite well. Would like to merge this. It's ready to be upstreamed as well.

I posted this to upstream, since it's a pretty simple change, the concept of it was already agreed upon in a previous discussion with upstream, and it works well in testing. (https://github.com/atom/atom/pull/21228)
Let's just wait and let upstream merge this, then we can sync it back to this fork by merging upstream master to ours again.

OK Thanks!


Added support for createOrUpdate call on Contact Object

http://developers.hubspot.com/docs/methods/contacts/create_or_update

Thanks!


[Improvement] Keep last livestream camera facing & increase Glide version

Description

[Improvement] Keep last livestream camera facing
[Update] Increase version of Glide

close because #158


minio-py: Always install last pip version

Building a mint image is currently failing with the following message:
You are using pip version 8.1.1, however version 9.0.3 is available.
You should consider upgrading via the 'pip install --upgrade pip' command.
The command '/bin/sh -c build/minio-py/install.sh' returned a non-zero code: 1

With this PR, we always install the last pip version.
Fixes #270

Travis is failing. Please take a look.

This is passing right now.


Display implementation should show the error's Display, not just the description

Given the following invocation of error_chain!:
error_chain! {
    types {
        Error, ErrorKind, ChainErr, Result;
    }
    links {}
    foreign_links {
        git2::Error, Git2, "git error";
    }
    errors {
    }
}

Producing an error from a git2::Error and then printing that error just produces "git error", without actually printing the underlying git2::Error.  The Display implementation for an error produced from a foreign link should forward to the Display implementation for the linked error.

Thanks for moving this over to the new repo. I agree this would be better. Help wanted.


Add test coverage for marl::Ticket::onCall

This is not currently being tested, and as it is a template method, we don't even check it builds correctly.

Hi @ben-clayton. Wondering what the expected behavior for marl::Ticket::onCall is. The documentation in 0182eb968366364288f5b26a77b87f05a9ce4782 says:
// onCall() registers the function f to be invoked when this ticket is
// called. If the ticket is already called prior to calling onCall(), then
// f() will be executed immediately.
// F must be a function of the OnCall signature.
template <typename F>
MARL_NO_EXPORT inline void onCall(F&& f) const;

It seems that the code schedules the callback for deferred execution regardless of state the ticket (called/waiting) at the time of registration. Please let me know if I'm missing something.

Hi @waelhalbawi,
The documentation isn't quite right, in that will be executed immediately should be will be scheduled for execution immediately.

It seems that the code schedules the callback for deferred execution regardless of state the ticket (called/waiting) at the time of registration

This statement doesn't seem right though. If the ticket hasn't been called yet (we get to here), then the function is copied to the record->onCall field for invoking when the ticket is called.
Cheers,
Ben

Hi @ben-clayton. Thanks for clarifying.

This statement doesn't seem right though. If the ticket hasn't been called yet (we get to here), then the function is copied to the record->onCall field for invoking when the ticket is called.

What I meant to say is that a registered callback is scheduled for execution here, and not invoked in-line, once the ticket is called, which is consistent with your clarification.
Do you have examples of where/how this feature is used?

It's used in SwiftShader, but the code is pretty complex
The basic idea is that you want to avoid starting a fiber that immediately blocks on another sync primitive, as you'll just be allocating a fiber and wasting a bunch of cycles to insta-yield. If you need to wait for a chain of previous events to complete, onCall is handy for deferring some work without an immediate wait.
I've got quite a heavy week on, but I'll see if I can write some tests which can close this bug and act as a basic example.

Thanks!


Create Helm charts registry

Is your feature request related to a problem? Please describe.
Hi, folks! Recently I've started closely work with Pulumi tool. In particular, this utility allows to install Helm charts from registry, but doesn't support usual URL with tar gz. For example, if you append in repo field full path to the archive you'll get error. But I really wanna use your charts like job-executor-service etc. Could you create the Helm chart registry, please?
Describe the solution you'd like
You can utilize GitHub Page technology for this aim. I've already realized this approach, but maybe you wanna make this official...
Describe alternatives you've considered
Yeah, I can download tar gz, untar it, and setup with Pulumi local path, but I'm not sure that it's a modern approach)
Additional context
Example.

Does https://charts.keptn.sh work for you?


Support loading LSP config from a config file

The initial LSP implementation hard codes support for Rust and Dart but longer term ad needs to support configuring language servers via user controlled config. The config should also support opting in to having servers start automatically.

implemented in #58


Fix CI Terra/Aer build under MacOS

Summary
Details and comments

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

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 83.514%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
694

Relevant Lines:
831

💛  - Coveralls


ArgumentsAnalyzer - support PHP8.1 readonly

closes #6221

Coverage decreased (-0.01%) to 93.013% when pulling 0a9c7309dfffe8f672b8d1052b061363486e5dbf on SpacePossum:master_6221_bug into efeab760998cf8ddb44c0e4c87f9bdd385bff64f on FriendsOfPHP:master.


fix: reset error message on retry

https://files.slack.com/files-pri/T1ZUKET9V-FBBDXMV44/screenshot_from_2018-06-21_11-03-04.png

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Error message isn't passed through

 Issue by EmmaZhu
Tuesday Feb 28, 2017 at 07:20 GMT
Originally opened as https://github.com/gsscoder/commandline/issues/417

For example, I have a customized parameter defined like following:
public class MyParameters
     {
         public MyParameters(string init)
         {
             throw new InvalidOperationException("Some error message");
         }
         public string Value { get; private set; }
     }
I'd want to show the accurate error message when the parameter value user input is incorrect. But seems the error message is always like:

Option 'option1' is defined with a bad format.

Is there anyway I can get the error message passed through?

 Comment by nemec
Saturday Mar 04, 2017 at 06:15 GMT

Can you post a minimal reproduction of the issue, with a complete options class and the args array you're passing to the compiler? Also, which version of the library are you using?
From what you've mentioned so far, and assuming you're using the version in master, I think you will need to insert your custom error text when building the help screen:
result.WithNotParsed(errs =>
{
	HelpText.AutoBuild(result, help =>
	{
		help.AddPreOptionsLine("Some error message");
		return help;
	}, e => e)
	.ToString()
	.Dump();
});

You can do whatever you want in the first delegate to AutoBuild, including displaying custom messages.


TcpStats.GetStats() now handles error when host cannot disclose information

Hi! 👋 Long time consumer, first time contributor 😊
I had a quick look at what uses these stats, and it appears the code can handle not having any stats, so I though this was the most reasonable approach.
But happy to update with whatever you guys prefer, equally happy to close this PR and leave to the experts if I'm way off! But, thought i'd try and give back if I can.
Fixes https://github.com/elastic/elasticsearch-net/issues/5176
Thanks!

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?

Sorry, forgot to sign CLA. Done now, but not sure how I can "refresh" this PR with that?

Not sure how to fix this check?

Not sure how to fix this check?

Not sure if you can - don't worry about it

This LGTM, thanks @RPM1984. @russcam Since you've been kindly reviewing, I'll let you have one final glance as well before merging.

I had a look at the test failures, but.. they seem unrelated? Not sure if I missed a step, in the commit/PR process?

@RPM1984 the integration test failures do look unrelated - I think these should be addressed separately, so I've opened https://github.com/elastic/elasticsearch-net/issues/5185 to track

LGTM, thanks for your effort on this, @RPM1984 👍

My pleasure! Glad I could finally give back to this awesome lib. Keep up the awesome work @russcam @stevejgordon ❤

Thanks again @RPM1984. Really appreciate you contributing here.


Enable ManimCE

Hope enable ManimCE

Relevant discussion:
https://github.com/ManimCommunity/manim/discussions/3954
for that I think we need them to implement the checkpoint_paste() functionality

This is probably not something that is going to happen any time soon as the ManimCE version (as far as I know) doesn't currently support OpenGL which 3Blue1Brown's manimgl relies on to show the interactive preview. As behackl stated in this comment:

It mainly is a question of resources; we have been working on a refactor of the render loop for ages, recently with some more progress on our experimental branch. But progress is slow, and people get busy with other things. Help to get things done is, of course, always welcome.

Instead for ManimCE, you might want to explore the concepts of Sections. Also see here in the tutorials.

Until then, this extension will focus on working with manimgl and there are currently no plans to support the community edition (at least I don't have plans to do so in the near future). Instead, we focus on improving the user experience for interactively previewing Manim code from manimgl as this example shows:
https://github.com/user-attachments/assets/13086817-229a-4151-9642-63fc91146f43
Feel free to reopen the issue in the future once ManimCE supports some kind of previewing built-in (maybe as well with an OpenGL renderer). In the best case, they provide a command similar to manimgl that we can just swap out in this extension to support ManimCE as well.


network: don't force send block announcements

There are 2 situations where we send out block announcements:

after importing a block
after voting on a block in GRANDPA

Currently we are always force-sending the block announcement even if the peer already knows this block, which can lead to duplicating block announcements in the following situations:

multiple GRANDPA votes on the same block (on prevote and precommit but also potentially across different rounds)
we import a block from a peer and then re-announce it again to it

I think we can relax the force-sending of announcements. The only worry would be missing some fork that is needed to process some GRANDPA votes, but we already have the reverse mechanism where peers will actively try to sync these forks.

bot merge


Button text color hover not taking effect

Which exact UI Library version are you using? For example: 1.0.1 - don't just write v1.
Default with Umbraco 10.4.1
Bug summary
When specifying --uui-button-contrast-hover on buttons, the color set does not take eefect on hover.
Specifics
No response
Steps to reproduce
Create a uui-button as <uui-button type="button" look="secondary" color="default" style="--uui-button-background-color-hover: var(--uui-palette-violet-blue-dark);--uui-button-contrast-hover:#ffffff"></uui-button>
You'll notice that the background takes effect but not the text.
Expected result / actual result
No response

The fix has been merged and it will be released in 1.4.0. Thanks so much for reporting this!