Apache configuration for suburl with Isso ?

Hi,
I have discovered Isso yesterday and I'm fairly impressed with it. Nice piece of software.
I wanted to install it for my blog and I'm running into an issue I don't understand.

Isso and Blog are not on the same server
Isso is self-hosted on my Pi alongside Nextcloud, Gitea and Wallabag
I use DuckDNS to access my Pi outside
myadresse.duckdns.org leads to my Nextcloud instance
I want Isso to be available as myadresse.duckdns.org/isso

So, In Apache2 config I have :
<Location "/isso">
    ProxyPass "http://localhost:8001"
    ProxyPassReverse "http://localhost:8001"
</Location>

My blog is able to use Isso but when a comment is submitted I receive a link to validate with myadresse.duckdns.org/id/13/activate/MTM.DZaffA.5dORG_6UZzZRU3RcHMXuzV3NzcM
What could I do to fix that ? (not a huge problem but still)
Thanks :smile:

If you are up to some code + testing + PR and take the credit for this, here is the problem:
Where the mail is constructed:
https://github.com/posativ/isso/blob/7f55c81e0875e1564eeab8ec6dd605c5699c9137/isso/ext/notifications.py#L112
Where the variable is set (probably not the good one since it's not taken from the conf, or maybe the good one but we need one more)
https://github.com/posativ/isso/blob/7f55c81e0875e1564eeab8ec6dd605c5699c9137/isso/__init__.py#L126
It uses werkzeug local context manager:
http://werkzeug.pocoo.org/docs/0.14/local/
Feel free to tackle it!

I just submitted a patch, in which I used the host in the general section in the conf file for the uri generation. It seems to be working in my setting in version 0.10.6 from the pypi, so can someone test this patch on the version in the master branch?

This won't work on some cases, for example I use isso on multiple domains, in this case isso.conf.get("general", "host") is a list of domains and it will result here in a TypeError.
You sould do some try/except to catch the TypeErrorand maybe take the first elements of the list or something like that…

Merged

:+1:
Thanks


Add test for Kubed

Run test as documentation, Config syncer

Copied configs to test instead of reading from file


Documentation for Pagermon

Have started on improving documentation for Pagermon.
Below is a link to my pagermon wiki that is currently in DEV state
https://github.com/s3m1s0n1c/pagermon/wiki
Feel free to comment or make suggestions.
I think I'll leave pics out of this one.. takes up too much space and takes too much time for now..

Its looking good so far 😄
One suggestion i will make, is to make use of pages.
Currently everything is on one page, splitting things up into pages will make nicer on the eyes

I disagree, it's no different to readme.md
You can use the menu tree on the right to go to any one section and find info..
@davidmckenzie What format would you like this in?

I'm mostly indifferent - single page makes sense for a basic readme, but isn't really scalable once you start going into detailed topics. I'm more used to documenting on confluence where you can go middle ground and have fancy expando thingies.
If I was writing it I'd probably split into separate pages to make editing easier, but that's just my personal preference :)

Looking good so far.
My personal preference is seperate pages for each block but again that's just my personal preference :-)
I've been meaning to write down my process for spinning up/down instances and setting up NGINX for reverse proxying with certbot for ssl.
I'll write it down next time i break my test env completely and throw in a pull :smile:

mmmm guess the tribe has spoken...
I have made updates and added pages..  Let me know if that's the right direction everyone wants.
@DanrwAU  yeah cool if you could write it down and i'll add it.
Going to document Apache proxy as well for those more hardcore and have server and client separate.
Thanks

Lookin really good @s3m1s0n1c !

+1, looking good!

+1 from me

Note to self.. Add Signal Flow Diagram to Documentation :)

I haven't forgotten about this..
Just been busy with life/works.
Thanks


Windows containers: couchdb fails when data directory mounted as a volume

Description

Steps to reproduce the issue:
Grab this attachment and unzip it.
couchdb_issue.zip
Run the build.ps1 script to build the couchdb image.
Run the good.ps1 script -- couchdb starts up successfully when the C:\couchdb\var\lib\couchdb directory is not mounted as a volume.
Run the bad.ps1 script -- couchdb quits on start-up.
Describe the results you received:
couchdb does not start up successfully when its data directory is mounted as a volume.
Describe the results you expected:
couchdb should start up whether its data directory is mounted as a volume or not.
Additional information you deem important (e.g. issue happens only occasionally):
Interestingly it does sometimes work. If I run the bad.ps1 script many times eventually CouchDB was able to create one file (.delete) in the host folder. After many more times it eventually created the other two files and started up.
By observing the ".delete" path in Process Monitor I can see that the couchdb Erlang code makes a call to CreateFileW. Eventually the FLTMGR.SYS driver makes a CreateFile call and gets a REPARSE response. At this point CouchDB quits.
If I try long enough eventually CouchDB is able to create the ".delete" file. In this case I see a subsequent call by the FLTMGR.SYS driver where it gets an "IS_DIRECTORY" response from an additional CreateFile call. So it seems like sometimes FLTMGR.SYS does not issue a call following a REPARSE result.
 Didn't help.

I have not had a chance, but will try to reproduce this today. Thanks.

There are multiple issues:

Erlang/OTP R16B02 (efile_readlink in beam.dll, called from erl.exe) attempts to resolve the reparse point (symbolic link to the volume) to its final name via GetFinalPathNameByHandleW , which fails with ERROR_FILE_NOT_FOUND. efile_readlink does not check for error, and corrupts memory.
GetFinalPathNameByHandleW generally fails to return the final name for volume directories mapped into containers, returning ERROR_FILE_NOT_FOUND. Based on debugging, it looks like it resolves to the target path on the host, which is invalid in the container.
If GetFinalPathNameByHandleW succeeded, it would likely return a path of the form \\?\ContainerMappedDirectories\<GUID>, a valid, but unconventional Win32 file name. Many cmd.exe commands fail with those file names. Erlang immediately removes the \\?\ prefix, and assumes the rest is valid Win32 file name. For container mapped directories (volumes), that is not the case, so removing the prefix will result in an invalid file name, e.g. ContainerMappedDirectories\<GUID>.

Given these issues, even if Microsoft does update GetFinalPathNameByHandleW to succeed for container mapped directories (for example, by returning the \\?\ContainerMappedDirectories\<GUID> path), callers may fail parsing it. The best option is for a caller to not modify the name at all.
@drnybble Based on code inspection, Erlang/OTP 19.2 checks for failure from GetFinalPathNameByHandleW, allowing for a graceful failure. CouchDB 2.0.0 for Windows is based on Erlang/OTP 17.5. I have not reviewed its source, or tried it, but is 2.0.0 and option for your scenario?

John, thanks for this superb analysis. I will work on Dockerizing couchdb 2.0.0 and see if it works. I guess it was working on 1.6 sometimes by fluke perhaps due to memory corruption as you said.

It looks like there's nothing actionable at this moment, so I'll close this issue for housekeeping, but let us know how you progressed 👍

@donkirkby Sounds good. Yes, I noticed if I prevented the corruption, by changing ecx to 0 so it did not copy any memory, then CouchDB start successfully. Also, I believe that once the directory structure exists, this code path is not encountered.
I took a quick look at CouchDB <IP_ADDRESS>. It seems quite different, and did not work on my first attempt, though it failed in a completely different way. Looking at the Erlang/OTP 17.5 source code, it does check the GetFinalPathNameByHandleW return value, so it should, at least, avoid that problem.


Add brute force node importance

This adds brute force node importance with a class that overrides the node_features method in StellarGraph
See #850

I think this ends up looking a bit clunky still, and I'm a bit unconvinced that it will be reliable since it assumes that given a trained model, we can always define a function that looks like:
def predict(graph, node) -> score

Currently an entire stellargraph ML pipeline has the freedom to be defined in a way that's quite tightly coupled with the original graph that it was created with, so I'm not sure if this will always be possible.
For GCN, the dimensions in a model/layer comes from the dimensions in a generator, and a generator is defined using a graph, so we can be careful that the dimensions are kept consistent even when we feed in new graphs.. but you could theoretically have a generator and model that does something a lot more tightly coupled with the graph. Not sure how likely all this is though.


:sparkles: expense / image file component

@delphinemillet  et @fabienjuif it's so bad ?? 😶

I let this PR atm, just to information.


remove stale code: CurationEditor and FilterSelect which uses react-select

I had to remake a branch to fix the conflicts. This includes the requested changes from the PR #23

Looks good to me


Add VB sample

This topic is C# only right now

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

ID: 1b15cd4c-9cba-c0e2-7618-7d621a3c5936
Version Independent ID: f4681bc6-57be-6e69-cad9-776ce64d93bd
Content: Overview - EF Core
Content Source: entity-framework/core/index.md
Product: entity-framework
GitHub Login: @rowanmiller
Microsoft Alias: divega

Also @divega, should I update the author metadata from Rowan to you?

@mairaw I don't know. I guess it depends on whether it is used for something different from what the word suggests. He was the author of many of the pages so I have left it as is.

@mairaw What's the policy here? Is there going to be a separate set of documentation with samples for VB? Or are the VB samples supposed to be integrated into these docs somehow?

@mairaw  why the issue to provide a VB sample? Who's going to maintain it? We don't provide VB samples in the ASP.NET Core repo. cc @tdykstra

Closing; we have no plans to create VB examples.


Fixed NPE on volume creation from snapshot

Description
This PR fixes a NPE that occurs when trying to create a volume from a snapshot. This happens because when trying to log the virtualMachineProfile in the AbstractStoragePoolAllocator#logStartOfSearch method, its toString() method will throw a NPE since the _vm attribute is null.
Some toString()s were also created in classes that had objects being logged during the volume creation process, as they were being logged as memory addresses since they had no toString() implementation.
Types of changes

[ ] Breaking change (fix or feature that would cause existing functionality to change)
[ ] New feature (non-breaking change which adds functionality)
[X] Bug fix (non-breaking change which fixes an issue)
[ ] Enhancement (improves an existing feature and functionality)
[ ] Cleanup (Code refactoring and cleanup, that may add test cases)

Feature/Enhancement Scale or Bug Severity
Feature/Enhancement Scale

[ ] Major
[ ] Minor

Bug Severity

[ ] BLOCKER
[ ] Critical
[ ] Major
[X] Minor
[ ] Trivial

Screenshots (if appropriate):
How Has This Been Tested?
This was tested in a local lab, before this patch every time I tried to create a volume from a snapshot, a NPE was thrown. After the patch the volume creation from snapshot was fixed.

@acs-robot a Jenkins job has been kicked to build UI QA env. I'll keep you posted as I make progress.

@blueorangutan package

@DaanHoogland a Jenkins job has been kicked to build packages. It will be bundled with  KVM, XenServer and VMware SystemVM templates. I'll keep you posted as I make progress.

Packaging result: :heavy_check_mark: el7 :heavy_check_mark: el8 :heavy_check_mark: debian :heavy_check_mark: suse15. SL-JID 4526

@blueorangutan test

@DaanHoogland a Trillian-Jenkins test job (centos7 mgmt + kvm-centos7) has been kicked to run smoke tests

Trillian test result (tid-5191)
Environment: kvm-centos7 (x2), Advanced Networking with Mgmt server 7
Total time taken: 38562 seconds
Marvin logs: https://github.com/blueorangutan/acs-prs/releases/download/trillian/pr6839-t5191-kvm-centos7.zip
Smoke tests completed. 104 look OK, 0 have errors, 0 did not run
Only failed and skipped tests results shown below:

Test
Result
Time (s)
Test File


[9.x] Add common base class for paginators

This is a sort of retry of #41204 and #41199.

Currently the paginator classes does not have a common base class. Because of this, there are a big portion of code duplication that could be prevented by using a base class.
My former PRs were closed, because the refacoring was too big and might had some potentional bugs.
In this PR I just added the empty base class, and if it's merged, then we can slowly start a step-by-step refactoring without the risk that a big code chage holds.

I don't see the use case of this? Why do you need this?

As I wrote in my prevous PRs, the paginator classes implement mostly the same functionality with exactly or at least nearly the same code. Also, they don't have any common root/base class, and that feels a bit odd to me.
Reorgaizing a bit would clean up the current classes' code base and would prevent future duplications. Also, if in the future, by any reason a new paginator type would be added, it also could use the base class.

I understand reusage but an empty base class doesn't serves any purpose

Agree with Dries here.

Thanks for your pull request to Laravel!
Unfortunately, I'm going to delay merging this code for now. To preserve our ability to adequately maintain the framework, we need to be very careful regarding the amount of code we include.
If possible, please consider releasing your code as a package so that the community can still take advantage of your contributions!
If you feel absolutely certain that this code corrects a bug in the framework, please "@" mention me in a follow-up comment with further explanation so that GitHub will send me a notification of your response.


Remove/Manage exception from getSingleIdentifierFieldName()

Feature Request

Q
A

New Feature
yes

RFC
yes/no

BC Break
yes/no

Summary

After searching around the source code and trying to implement a feature for EasyAdmin to work with composite keys, I realized that no bundle that calls getSingleIdentifierFieldName() will work with composite keys due to the function throwing an exception.
For example, the reason why EasyAdmin doesn't work with composite keys is because it calls this function and because Pagerfanta calls this function as well. I'd like to suggest removing the exception because I haven't found a case where it could cause a fatal error.
Another suggestion would be to provide an argument (boolean) to pass which determines if the exception will be thrown. For example the edited function could look like...
   public function getSingleIdentifierFieldName(bool $throwsCompositeException = true)
     {
         if ($this->isIdentifierComposite && $throwsCompositeException) {
             throw MappingException::singleIdNotAllowedOnCompositePrimaryKey($this->name);
         }

         if ( ! isset($this->identifier[0])) {
             throw MappingException::noIdDefined($this->name);
         }

         return $this->identifier[0];
     }

This way every bundle that currently calls this function will be safe and unchanged due to the default value provided in the parameter.
I think that removing the exception or using the suggested edit would allow for many bundles that use Doctrine ORM to be able to implement support for composite keys.
Lastly
I do think the best option would be to remove the exception and let the function return the first field instead of throwing the composite key exception.

I fear that this is an X/Y question.
The exception is the correct outcome here. You cannot get a single identifier for something that has more than one identifier, and the Doctrine paginator can't operate on multiple identifier fields anyway.
Instead, the correct resolution is to NOT rely on this method at all when the mappings aren't known upfront.
Closing as invalid here.


Turn on whole module optimizations

After turning this optimization level on, I observed an average decoding
time of 1.611s from 2.523s and an average parsing time of 1.724s from
2.384s.

After reading through this optimization guide I found that whole module optimization was the only thing missing from Argo.

👍


Add-PnPDataRowsToSiteTemplate export dates as UTC string

Type

[x] Bug Fix
[ ] New Feature
[ ] Sample

Related Issues?
N/A
What is in this Pull Request ?
The Add-PnPDataRowsToSiteTemplate cmdlet exports DateTime values as-is. That means that there is no indication that the date is in UTC Time zone. Once you apply the template, the dates are read as local times, which can be a few hours off. This change formats the exported dates as UTC string, so they are not converted to a local datetime when imported.

Thank @bjdekker for this improvement , makes total sense !
Have merged it and will be available in tomorrow's nightly


fix save geometry bug

there's a bug in save geometry, so in ios 13 if the swiftui component has already assigned 2 time or more to coordinator.frames[id] with scrollId, it will produce bug that makes ios 13 crash

Please provide a crash log and an example to reproduce it. I don't think this "fix" does what you think it does.


Service Bus Queue Trigger Retry Policy Not Working as expected

Hi Team,
We are trying to retry 3 times on SB Queue failure .
Here I followed as per this document : microsoft document
"extensions": {
"serviceBus": {
"clientRetryOptions":{
"mode": "exponential",
"tryTimeout": "00:01:00",
"delay": "00:00:00.80",
"maxDelay": "00:01:00",
"maxRetries": 3
},
        "autoCompleteMessages": true

    }
}

I Configured in host.json file But still it is not working. --Please provide suggestion here
Where as if we configured  like this it is working : [ExponentialBackoffRetry(5, "00:00:04", "00:01:00")] here problem is these values need to be configured at compile time means it can not be taken from config and want to know at what retry number it succeed for both configuration -- Please provide suggestion here
For example I configured in azure portal queue deliver count : 2
Max Retry count =3
Now at first deliver count  : 1 and it succeeded at 3 retry now total count is 4 (default try + configured retry count)
Now at second deliver count  : 2 and it succeeded at 2 retry now total count is 3
Now total  run to success processing is : first deliver count + first total retry is (3) +second delivery count+second retry current count (2) =7 (for above iteration)
Microsoft.Azure.WebJobs.Extensions.ServiceBus 5.8.1 version we are using.
What to know at what iteration it succeeded.
For example I configured in azure portal queue deliver count : 2
Max Retry count =3
Now at first deliver count  : 1 and it succeeded at 3 retry now total count is 4
Now at second deliver count  : 2 and it succeeded at 2 retry now total count is 3
Now total  run to success processing is : first deliver count + first total retry is (3) +second delivery count+second retry current count (2) =7 (for above iteration)
PFB Sample application
sampleapp.zip

Hi @raiajithkumarr.  Thank you for reaching out and we regret that you're experiencing difficulties.  What you're describing is expected behavior.  The host configuration and attribute control retries for different contexts and are not equivalent.
The retry values that you're setting in host.json govern the behavior of the Service Bus client library used by the trigger.   These retries take place implicitly and transparently when a service operation fails.  Unless all retries were unsuccessful, your Function code is unaware that retries took place.   Failures in your Function code are not governed by this policy.  These retries very rarely increment the delivery count as the operation to read the message itself failed - the message was never delivered and never abandoned.
The ExponentialBackoffRetry attribute applies a retry policy to your Function code, instructing the Functions runtime to retry invocations of your Function if an exception is surfaced.  In this scenario, you'll generally see the delivery count increase as the message has been delivered to your code and the exception has caused it to be abandoned.
I'm going to mark this as addressed, but please feel free to unresolve if you'd like to continue the discussion.

Hi @jsquire
Thank you for your response
As I mentioned above is there any possibility to pass dynamic arguments or parameters for this attribute ExponentialBackoffRetry

As I mentioned above is there any possibility to pass dynamic arguments or parameters for this attribute ExponentialBackoffRetry

To my knowledge, there is not; attributes in .NET must be bound to constant values.   The docs do not mention that binding expressions are supported.   You may want to test that to be sure, or open an issue for the Functions host team, who would be able to answer that authoritatively.

We will added business logic to retry based on configuration


Fixes #723 : don't accept empty string for bookmark name

Hello Leo, time to sync your PR branch with develop! :)
(code refactoring ... probably a few conflicts to solve)


Move collection filtering to database

Blocklist & permission-based filtering in collections could be done in the database to prevent the chance empty collection pages

Blocklist filter is now in-db, but permissions are still in-memory (getCollection) and could be moved down to store.getStream


Feature request: example commands

A simple example for each command would be super nice 🙏

Hi @OskarAhl,  What programs would you like to see examples of? We are rolling out a new version very soon and covering some examples for rm, mv, tar, cp, npm, git, find and ps. More suggestions are welcome :)

@OskarAhl: you can now upgrade kmdr-cli to version 0.3. It includes example commands for the 10 most popular programs in our platform. Get the latest version with npm install kmdr@latest --global or yarn global add kmdr@latest.
Some screenshots of v0.3🔥

I'm closing this issue. Please create a new one if you want examples for some specific programs :).

@ediardo wow that is awesome, thank you!


[BUG] Should read the multiple blocking pushed element

The following test can pass if set KVROCKS_BIN_PATH to redis-server.
https://github.com/apache/incubator-kvrocks/blob/dbd968b47a08184b3a0b6492f28f9ba8edf6d11b/tests/gocase/unit/protocol/regression_test.go#L58-L63
We should figure out the cause and to fix it.

This's a bug in blocking pop, let's me handle it.

I found the root cause is that the secondary blocking command has been read into the connection but didn't run since the NO read event was triggered again. I need some time to fix it since we can't force running read callback directly.


Content-Type header is set twice

Google chrome 55.0.2883.87. Same in MS Edge.
In version 1.3.3 post to /login is made with correct Content-Type: application/json; charset=UTF-8.
In version 1.3.4 content type is duplicated: Content-Type: application/json;charset=UTF-8 application/json; charset=UTF-8.
In this situation bodyParser fails to parse request body and response is {"status":400,"message":"Invalid username or password."}

The PR fixes this behaviour. 1.3.4 did not check whether the Content-Type was already set. Interestingly, upcoming version 2.0.0 does make this check, so the behaviour was transferred to 1.3.4 as well.
@robertjd or @typerandom, could you please review? I made a version-1.X.X branch based based off 1.3.4 for this (and possible future) tagging and releasing of pre-v2 patches.

This should now be fixed in 1.3.5


Request: Progress bar

When I render a huge map, I really would love to know the progress
A simple percentage could even be enough

you could use the argument  -v which will then print out the processed tiles and the total amount of tiles


GuiTextBoxGetState() and GuiTextBoxSetState() types fixed

Type of GuiTextBoxGetState() and the argument of GuiTextBoxSetState() is GuiTextBoxState, not int.

@yashrk What's the difference of using int instead of GuiTextBoxState? Actually, GuiTextBoxState is an int... In raylib I always use int despite typed enums...

What's the difference of using int instead of GuiTextBoxState? Actually, GuiTextBoxState is an int...

Not always:
#if defined(RAYGUI_TEXTBOX_EXTENDED)
// Gui text box state data
typedef struct {
    int cursor;      // Cursor position in text
    int start;       // Text start position (from where we begin drawing the text)
    int index;       // Text start index (index inside the text of `start` always in sync)
    int select;      // Marks position of cursor when selection has started
} GuiTextBoxState;
#endif

And in the gui_text_box.c example we have exactly
#define RAYGUI_TEXTBOX_EXTENDED

Maybe we should do this?
#if defined(RAYGUI_TEXTBOX_EXTENDED)
// Gui text box state data
typedef struct {
    int cursor;      // Cursor position in text
    int start;       // Text start position (from where we begin drawing the text)
    int index;       // Text start index (index inside the text of `start` always in sync)
    int select;      // Marks position of cursor when selection has started
} GuiTextBoxState} ;
#else
typedef int GuiTextBoxState;
#endif

@yashrk My bad, I didn't check it properly. Thanks for the fix! :)


DocsArchive-<IP_ADDRESS>Auto archive content

Please don't merge this PR until target archive repo PR https://github.com/OPS-E2E-PPE/docs-archive-test-target_1d5aefc8-aefb-474c-9a97-3a3c59617d33/pull/1 is merged into live branch.
Auto archive content to https://github.com/opstest2/docs-archive-test-target_1d5aefc8-aefb-474c-9a97-3a3c59617d33.git

Docs Build status updates of commit 592ce0f:
:x: Validation status: errors
Please follow instructions here which may help to resolve issue.

File
Status
Preview URL
Details

:x:Error

Details

[Error-GitBranchDeletedOrForcePushed] Cannot sync git repo to specified commit because branch Release_Archive_master_2021-06-25-07-34-45 has been deleted or has been force pushed

For more details, please refer to the build report.
If you see build warnings/errors with permission issues, it might be due to single sign-on (SSO) enabled on Microsoft's GitHub organizations. Please follow instructions here to re-authorize your GitHub account to Docs Build.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
Note: Your PR may contain errors or warnings unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel


.ply Export

Hey! There's some tools for working with cleaning up point clouds and I'd love to be able to export a NeRF to .ply for just that purpose! Would that be something implementable? :o

@oculardegen We already have this mesh exportation by default?
https://github.com/ashawkey/torch-ngp/blob/main/main_nerf.py#L160

I'd like to export it to a point cloud so I can do some manual cleanup on it though and remesh it with external tools as opposed to using the built in remesher :o.

I'd like to export it to a point cloud so I can do some manual cleanup on it though and remesh it with external tools as opposed to using the built in remesher :o.

Hi oculardegen.
I am facing the same problem now.
Do you know how to export point clouds now?


Let's be careful with claim deletion

Dumb things that could happen:

If you delete a definition, make sure all claims using that definition aren't automatically false because they're implying something that doesn't exist.
Make sure any zeroth-order claims that reference deleted claims are dealt with in case new claims come along with those ClaimIDs and change the meaning of the formulas.

Also, because the delete button will be right next to the attach definition button, let's create a warning page before proceeding.

Done.


[nemo-plugin-qml-systemsettings] Export time zone transitions to QML.

QML can be notify of the next DST change after referenceDateTime is set:

nextTransition provides the moent of the closest next transition,
and transitionShift the shift in time to be applied at this transition.

@jpetrell this is the missing MR from the migration.

I agree with your renaming. I've followed the naming of Qt in that regard (a transition in the time zone definition is used to save a DST change) and indeed the applied offset is called there a "transitionShift". But your propositions are more meaningful:

nextDaylightSavingTime
and daylightSavingTimeOffset (slightly different from your proposition but better aligned with the other one).
If you validate, I'll update the PR with these names and update the UI part accordingly.

About no integer hour changes, I've no idea. There are for sure half hour time offsets with respect to UTC, but for DST, I cannot say. I don't like much the division by 3600 in UI, it's kind of ugly. I thought first to get this to minutes, but well it would require a division also, so why not leave the seconds ! But I'd follow you advice here. If you prefer, I'll put the division in the middleware. In that case, should we call the property daylightSavingTimeHourOffset to be plain ?

and daylightSavingTimeOffset (slightly different from your proposition but better aligned with the other one)

Sounds good, mistyped it earlier. :)

About no integer hour changes, I've no idea.

If you want to be on the safe side keeping it in the seconds range is ok too. Or we use hours and worry when the (unlikely?) bugs arise.

should we call the property daylightSavingTimeHourOffset

Having "hour" in the name would be superb otherwise, but fear the property name starts to be too much for our cave human brains to comprehend. Could even shorten to daylightSavingOffset.
Anyway LGTM on my part for the PR.

Thank you @jpetrell for your suggestions, I've renamed the two properties and respective private variables.
I've decided (if it's fine), to keep the offset in seconds, to be consistent with Qt. There's a comment in the code to remind that offsets are in seconds. If you know a better place to document it, don't hesitate to mention it. I'll update the PR accordingly.

Now done on the calendar app side so assuming this can be closed.


Java 8: EXCEPTION_ACCESS_VIOLATION if "-XX:+ParallelRefProcEnabled" is set using Client G1GC Args

This issue documents an issue brought up on the Discord Server linked in the README.
Using the current list of arguments for Java 8 + G1GC crashes the game shortly after its start, probably when the garbage collector runs for the first time.
The issue was first to believed to be caused by G1GC itself or the used Runtime GraalVM 21.3.4, but has since then reproduced on Adoptium 8.0.312, and stopped happening after the argument -XX:+ParallelRefProcEnabled were removed.
The argument allows the Garbage Collector to use multiple cores to garbage collect and has a history of being used with G1GC on Java 8 versions of the game, including Aikar's famous arguments.
I am honestly stumped on what exactly could be causing it, with the most reasonable idea being that something in the G1GC Arguments could cause it to happen, whenever it's a rather unusual argument or values. Finding out whats going on could perhaps explain which thought processes others had when designing JVM arguments for Minecraft in the past.

I think I was seeing the a similar crash with adoptium and ended up removing -XX:+UseDynamicNumberOfGCThreads instead.

I think I was seeing the a similar crash with adoptium and ended up removing -XX:+UseDynamicNumberOfGCThreads instead.

I will try to give that a shot. I am not that experienced with the arguments, but both seem to somewhat have in impact on the garbage collector's ability to use multiple cores/threads...?
Previously people removed -XX:+UseG1GC to make it work... which ain't exactly the optimal solution to that. But it helped getting the idea that it may be caused by GC configuration.

both seem to somewhat have in impact on the garbage collector's ability to use multiple cores/threads...?

Yes: ParallelRefProcEnabled enables multithreading for part of the gc process; UseDynamicNumberOfGCThreads lets the gc adjust the number of threads it uses. Presumably they do work together in some cases, otherwise they wouldn't be in the list together. Both are disabled by default in java 8, but UseDynamicNumberOfGCThreads is on by default since java 11, so maybe we're just hitting an edge case that's been fixed since.
Removing -XX:+UseG1GC isn't a great solution in java 8. It means that it defaults to the parallel gc instead, which can struggle when the heap starts getting full.


Discussion for v3.0.0

We are going to address a number of technical issues that will require a major change. This version will not be binary compatible with the previous 2.x versions, but should be source compatible to a degree. We will strive to maintain a migration guide and be aware of what affects changes will have.
This issue is to raise concerns about changes that may be made for discussion. Issues and PRs for this will be flagged using the Milestone
CI builds for this will be pushed to a feed at: https://openxmlsdk.blob.core.windows.net/feed/index.json

So hi there!
Thx for this issue.
I have this https://github.com/dotnet/Open-XML-SDK/issues/1142, which is high on the agenda in my workplace, and I expect it to be similarly high on the agenda for other heritage institutions.
By the end of this year, we will conclude our project on spreadsheets preservation, and this one issue in particular, is important for us to get resolved.

@Asbjoedt One of the things I'm doing is cleaning up the initialization logic so the package is more customizable. I'm hoping to finish that up by the end of February. Once that's done, this kind of thing may be more possible. I'm breaking things up into replaceable parts called features.
To be honest, I'm not seeing anything in the issue that would need to be in v3.0, but the changes I'm doing for v3.0 should enable building it. Sounds like the validator system would need to be customizeable - that's something I was going to investigate after the packages and parts get cleaned up. Let's continue the conversation in #1142


rgw/get all buckets with stats

Hi ceph team,
We had an issue using rgw. Our requirement was to get all buckets with their stats (like it was possible in Quentin's GetBucket), but ListBuckets only return bucket names and calling GetBucketInfo on all listed buckets has too much overhead. Can you add a function which lists all buckets with their stats or with optional params? I can create the merge request if it's a valid function for this lib.
Thank you :)

CCing @thotz who's been doing most of the rgw package maintenance lately.

Hi ceph team,
We had an issue using rgw. Our requirement was to get all buckets with their stats (like it was possible in Quentin's GetBucket), but ListBuckets only return bucket names and calling GetBucketInfo on all listed buckets has too much overhead. Can you add a function which lists all buckets with their stats or with optional params? I can create the merge request if it's a valid function for this lib.
Thank you :)

IMO it is already addressed via https://github.com/ceph/go-ceph/blob/master/rgw/admin/user_bucket.go#L43
Are you looking for more generic, list all the buckets from all different users, if yes then you can totally send a PR. I am very happy to review the changes.

Hi ceph team,
We had an issue using rgw. Our requirement was to get all buckets with their stats (like it was possible in Quentin's GetBucket), but ListBuckets only return bucket names and calling GetBucketInfo on all listed buckets has too much overhead. Can you add a function which lists all buckets with their stats or with optional params? I can create the merge request if it's a valid function for this lib.
Thank you :)

IMO it is already addressed via https://github.com/ceph/go-ceph/blob/master/rgw/admin/user_bucket.go#L43 Are you looking for more generic, list all the buckets from all different users, if yes then you can totally send a PR. I am very happy to review the changes.

Hi again,
Yes, I meant an API to get all users buckets detail [for admin to use]. I'll send a PR but the changes are quite minor and similar to GetUsersBuckets. Please check it out, I hope I didn't get it all wrong.
Thanks


Clarify on how to use image name in image component and how tools can consume it

Which area this feature is related to?
/area documentation
Which area is this documentation change is related to?
/area api
Issue details
What mistake did you find / what is missing in the documentation?
The existing imageName field in the image component only has a brief field description. Based on the discussion on the community call on 11/14 (see the recording https://drive.google.com/file/d/1XLTdizST_LeoKpF7xw4Q9k__6LoXKCmo/view?usp=sharing for details), we need to clarify how the imageName field is set and handled by the tools.
Summary of the change:

The user can specify the imageName with either a full URL or a relative location of the image within an image registry to store the built image.
If a relative location is being used, the tools will have their own way to find out the image registry location (e.g. detection or user input). After the image component is built, it will automatically replace the correct image location in memory so that it can be used by an image container or the kubernetes component for deploying the image.
If a full URL is used, the tools will not replace it with the automatically generated URL and will use the full URL to store the built image.

Considering the imageName as a full registry image has collision and permissions problems. We should avoid that.
We should rather consider it an image selector as Tilt does:

my-image is an image selector. Tilt will scan all your workload manifests for images, and match any object that contains the image name my-image (regardless of tag).

Rephrasing that for imageName:
imageName is an image selector. Tools will scan all your conatainer and
kubernetes components for images, and match any object that contains the
image name imageName (regardless of the registry and tag).

example
components:
  - name: devtools-image
    image:
      imageName: devtools
      autoBuild: true
      dockerfile:
        uri: Dockerfile
  - name: devtools
    container:
      image: quay.io/devfile/devtools:ubi8-latest
      memoryLimit: 512Mi
      command: ['sleep', 'infinity']

==> imageName devtools matches the container image quay.io/devfile/devtools:ubi8-latest.

I've created a PR in the Devfile library to handle imageName as a selector: https://github.com/devfile/library/pull/172.
@l0rd @elsony @kadel Please take a look and let me know your thoughts.
There is also a draft PR in odo related to that. It leverages the changes in https://github.com/devfile/library/pull/172 and can be used already to test this via odo: https://github.com/redhat-developer/odo/pull/6768


Reworked branching and merging (not the conflict part yet)

Work in progress.

Ready for review

fixes #49
fixes #73
fixes #77

The git graph is really cool. I think this episode now flows a lot better, is closer to real life and more robust.

Looks good to me, though I was mainly looking at small-level things, not the big picture.  I think big picture is probably good too.  I'd say ready to go, can always be more improvements later.

Thanks! I think it is better then before. I will tomorrow with fresh head apply the remaining changes and merge and then we go from there. I think it is a lot cleaner now. To be confirmed in the classroom.


[WIP] Feature descriptive languages

So, the main idea is instead of detecting languages separately for different parsers and make parsers translate dates on their own, we detect and translate dates outside of parsers and make parsers operate strictly on English dates. Parsers code became very simple because of this and it would also make much more easier to add new parsers if we would ever need to.
Branch in its current state lacks majority of tests, but all functionality is in place. Also as you can see in languages.yaml file, adding new languages is very easy.
This request targets issues #9 and #12.

Hello @Allactaga,
Oops, sorry for interfering then. Since we needed both Thai and Vietnamese in some sites, I decided to get involved. I hope that's more of help than trouble.
Changes merged in #32. I added failing Vietnamese tests too.
I think we don't have a way to check for weekdays, right? So temporarily I added weekday test entries for Vietnamese in freshness date parser tests but they would need to be updated manually depending on time passed from last weekday.

Hi, @asadurski, thanks for getting involved. More help is always better!
I fixed tests for the Vietnamese (btw weekdays are processed by dateutil parser, not by freshness. Think about it as freshness only parse forms like "... ago" and you can not say "Thursday ago").
Although there are couple of cases i did not fix and i think it is best to go without such support:

"19 Jan 2010 lúc 2:19pm", this date is mixing two languages, so it is not a valid date to consider. You can simply fix it with re.sub(ur"\blúc\b", "", date_string") in spider and it would be valid english dates/
"Thg 11 25", while in this particular case it is obvious that it is November, it is not quite clear in the string like "1 năm 1 tháng 1 tuần". It is most likely "1 year 1 month 1 day" but could also be "1 year 1 january day". I think it is better for now to substitute such dates with "Tháng mười một" in spider.

Hey folks -- I've done a bit of reviewing, and put some comments above (mostly some nitpicking).
Anyway, I just wanted to say that this is truly awesome work -- thoroughly tested and well refactored!
Sometimes I'd find some lines of code that were confusing, so I tried commenting them out and running the tests -- and, as expected, some of them failed (proving a good coverage) and then I could debug and figure out the purpose of that code.
Well done, folks, I'm blown away! :wave: :wave:
I look forward to the next release! =)

Hey Elias, thanks for the input. I know it is quite a large PR and really appreciated your comments. :smiley:

Fixes #22 and #38


Big class?

How to set "big" to a specific stream? @aullman

You set the "OT_big" class on the element in the DOM.

You set the "OT_big" class on the element in the DOM.


Start playback with async license fetch

Question: Does shaka player wait for license exchange before starting playback even it there is a clear lead? Looking at the https://shaka-player-demo.appspot.com/demo e.g. Angel-one - it has a clear lead that i can play without license when i directly download the mp4 - but the demo player doesn't start playing till the license is fetched.
Is there a way to start playback while the license is being fetched?

I'm fairly sure playback does start without waiting for a license exchange.  What makes you say that it doesn't start until the license is fetched?  Can you provide steps to reproduce your results?

Mostly eyeballed with eme logger open. The encrypted event and license events happen right in the beginning while the video is preparing to play. I suppose the startup time with or without DRM is comparable to the time it takes to fetch the license?
I just played one of the widevine videos in the shaka-player-demo.

Well, we always start playback without waiting for DRM.  Here's a demo that shows this in action:
https://shaka-player-demo.appspot.com/demo/#asset=https://storage.googleapis.com/shaka-demo-assets/angel-one-widevine/dash.mpd?;license=https://<IP_ADDRESS>/this_will_never_respond;play
It's the same Widevine-encrypted Angel One clip, but with the license server set to an address that will not respond.  You will see playback stop after the 10-second clear lead runs out.
I think what you have been seeing is that our license server responds fast relative to the time it takes to fetch segments.
Does this help?

Cool - ya - i delayed the license response and see the playback starts. Thanks.


fix: Fix injecting settings after the batcher completes waiting

Issue #, if available:
Description of changes:

Fix the time when settings are injected so that we perform settingsstore.InjectSettings() after we get a call to trigger the batcher. This will eventually evolve into an abstract Wait() mechanism for the singletonController

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

9 of 9 (100.0%)  changed or added relevant lines in 2 files are covered.
40 unchanged lines in 2 files lost coverage.
Overall coverage decreased (-0.1%) to 73.889%

Files with Coverage Reduction
New Missed Lines
%

pkg/controllers/provisioning/scheduling/preferences.go
7
86.96%

pkg/controllers/provisioning/provisioner.go
33
85.52%

Totals

Change from base Build<PHONE_NUMBER>:
-0.1%

Covered Lines:
4672

Relevant Lines:
6323

💛  - Coveralls


Fixed: Typo Issue on the Reference Dom

Resolves #538

Thanks @Shahmaz0. As mentioned in the comment in issue #538, this is related to the way how we render particular symbols. To keep the discussion organized and consolidated, please feel free to add to the discussion within issue https://github.com/processing/p5.js-website/issues/447. I will close this PR for now for organization purpose. Thank you!


rubygems: 2.6.10 -> 2.6.13

Motivation for this change
A bunch of nasty CVEs: https://www.ruby-lang.org/en/news/2017/08/29/multiple-vulnerabilities-in-rubygems/
I haven't run nox-review wip as it causes quite a lot of packages to be rebuilt but I did compile all versions of ruby. Do we push something like this to staging first?
Cc: @grahamc
Things done

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

[x] NixOS
[ ] macOS
[ ] Linux

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[x] Fits CONTRIBUTING.md.

No. Let's get this quickly in stable as well.


Kata Containers 1.7.4

runtime: Enable file based backend
backport: Verify and block merge if other projects are not updated

e61e8d7 runtime: Enable file based backend
064cb4b ci: Verify and block merge if other projects are not updated

/test

Is the initrd failure familiar?  /cc @GabyCT


adopt11-hotspot@11-28: hash check failed

ERROR Hash check failed!
App:         java/adopt11-hotspot
URL:         https://github.com/AdoptOpenJDK/openjdk11-binaries/releases/download/jdk-11%2B28/OpenJDK11-jdk_x64_windows_hotspot_11_28.zip
First bytes: 50 4B 03 04 0A 00 00 00
Expected:    311466765ef02a0a3450571417a63f72f515f00c6cd66cee7fcf630c03374e47
Actual:      11a31d269c6527e9ba2e97cd502869fc130ef206ca5f55cccc0fa825401eb44d
The expected hash according to the adopt Website is 11a31d269c6527e9ba2e97cd502869fc130ef206ca5f55cccc0fa825401eb44d  target/windows/x64/hotspot/OpenJDK11-jdk_x64_windows_hotspot_11_28.zip

Should be fixed now.

Fix confirmed.


Support for Rails 5.2.1

I'm trying to upgrade to 5.2.1. Right now it requires 5.2
active_record_distinct_on (>= 0.1.4) was resolved to 0.1.4, which depends on
  activerecord (<= 5.2, >= 4.2)

@rexmadden Thank you for the bug report. This issue should now be fixed in 0.1.5.


demos: scroll is not showing on IE11 or Edge

Edge:

IE11:

Chrome:

Thank you for bringing this up! This appears to be a duplicate of #15.


Remove gpg-pubkey packages from RPM lists.

Signed-off-by: Christopher Sams<EMAIL_ADDRESS>

Related PRs:
https://github.com/RedHatInsights/inventory-schemas/pull/58
https://github.com/RedHatInsights/insights-host-inventory/pull/857

@SteveHNH I'm not sure why this is failing; is this a separate clowder/env issue again?

@CodeHeeler
Sorry for the delay getting around to this one. This actually looks like a bad string comparison in installed_packages may have been introduced.


pkg.go.dev: search algorithm doesn't find similarly named packages

What is the URL of the page with the issue?
https://pkg.go.dev/search?q=blake2
What did you do?
Searched for blake2 at pkg.go.dev
What did you expect to see?
Search results which included both https://pkg.go.dev/golang.org/x/crypto/blake2b and https://pkg.go.dev/golang.org/x/crypto/blake2s
What did you see instead?
Search results that did not include those packages
It looks like the search isn't finding similarly named packages. In this case, I knew there were two blake2-related packages in x/crypto, but when I searched for "blake2", neither of the similarly named blake2s or blake2b packages were in the results

/cc @julieqiu @jba

Thanks for the feedback! This is related to #36806.


ISPN-12441 Upgrade to protostream 4.4.0.Alpha2

https://issues.redhat.com/browse/ISPN-12441

A side effect of this is that org.infinispan.persistence.m.event_logger.UUID protobuf type becomes org.infinispan.persistence.event_logger.UUID and is defined in a different file. As far as I can tell this has no impact of backward compat. I can provide more details on my view if this opinion is challenged.

@anistor I fixed an issue with Docker on the CI agent, but there are now a number of related failures.

The failures are due to the change of default value of the @AutoProtoSchemaBuilder.service flag. Need to turn it off explicitly. This is the perfect example of why I insisted we need to have a way to bypass the automatic discovery of SCIs.

The failures are fixed now.

Rebased and pushed some more minor cleanups. This should be ready now @tristantarrant @gustavonalle

21 failures on Eviction tests, could this be related?

I don't think these failures are related. There's a huge number of new failures (mostly related to ExceptionEvictionTest) in build #14 vs. build #13 and the only difference between them is a squashed commit. Build #13 has just 3 failures. I suspect ExceptionEvictionTest is a bit unstable nowadays. I could not reproduce any of those failures while running locally. And none fails when executef individually from IDE. But let's have a couple more CI runs and see

More CI builds, more unrelated failures.

Thanks @anistor


Playback incorrectly restarts in background without user interaction

We recently received several (nice) complaints from users about radio restarting in the background in the middle of a meeting or when receiving calls. This seems to occur more often as of iOS 14.

This issue is related to automatic restart after the network becomes reachable again. This issue is not new, but according to my tests with an iPad running iOS 10 the new iOS 14 might be more tolerant in background, letting paused playback restart, while this was not the case in the past (to be confirmed for other OS versions, though).
Nevertheless, I made a fix proposal which replaced the stopped boolean we introduced in the past with a more obvious approach: Only recover from unreachable network errors when the network becomes reachable. This will avoid all issues we had in the past with this feature and improve our code.
We will lose automatic recovery from other failures (e.g. player errors due to network being unreachable), but the user can always tap to resume playback, so I think this change is safe and therefore for the best.
Available for review on feature/unexpected-resume-fix.

Issue confirmed by @pyby on iOS 13 as well.

As discussed with @pyby we have further improved error management:

Network errors are extracted recursively from an error and considered first.
Error icon retrieval has been made simpler (recursive retrieval has been dropped as it was required for network errors, which are now recursively processed separately as mentioned above).

As discussed with @pyby, we started with an unrestrictive condition on NSURLErrors to see whether the error message could appear in conditions where it does not make sense.
I found such a case with recently published content with bad streams (either 404 or 416), two episodes from the same TV series The 404 leads to an error containing a NSURLErrorFileDoesNotExist within the underlying error stack (and thus is incorrectly caught by our current greedy condition as a network error), while the 416 isn't.
For this reason, starting restrictive is probably a better idea. The fix is made on a feature/limited-network-errors branch.


dist folder and bower install

Hello,
You have thought that the component can be installed from bower? It would be very useful for some of us.
There is no "react-scroll.js" file which can be included directly in our scripts?
I see that when installed with node generates the structure:
-lib
--components
--- etc...
--mixins
--- etc..
-index.js
index.js is the file which should include projects?

Any plan to make it bower compatible?


Automate Project Account (Unity-Sips-Test) SSM Deployment

The project-level SSM (different from the shared services SSM) needs to be automatically deployed as part of a project deployment (terraform).
Acceptance Criteria:

[ ] Inspection into both the Github Action logs, and the AWS console prove that the project deployment led to the automatic creation of SSM.

@ramesh-maddegoda I would like to update the description of this ticket, to specifically name a few key/values that will be stored at the project-level SSM.  For example, I believe some of the API Gateway-related parameters are intended to be stored here.  Can you please list out a few examples, and then I will update this ticket description?

The terraform SSM creation scripts already exist.  There is an action for it called the the Environmental Provisioner.  It will get rolled into the control plane stuff soon.


⚠ Nextcloud has degraded performance

In 85a58e5, Nextcloud ($URL_NEXTCLOUD) experienced degraded performance:

HTTP code: 200
Response time: 27713 ms

Resolved: Nextcloud performance has improved in 518b2ca after 22 minutes.


Error

Fixes #XX
Description

Other information

Checklist:

[ ] My pull request has a descriptive title (not a vague title like Update index.md).
[ ] My pull request targets the master branch of the repository.
[ ] My commit messages follow best practices.
[ ] I added tests for the changes I made (if applicable).
[ ] I added or updated documentation (if applicable).
[ ] I tried running the project locally and verified that there are no
visible errors.

Remove old commits pls.

Sir i am trying to squash but idk why it is not working,
You can look i tried

This will remove the navbar.


Browserify compatibility, dependency improvements, delimiters and tagOnBlur props

This is a combined pull request for several very small changes.

Adds compatibility with browserify
Adds the default css under a fixed "css" directory (rather than a client having to include the file from examples)
Moves react to be a peerDependency (>=0.12.1 <0.13)
Adds key property to TagComponents
delimiters as prop, so they are customisable
tagOnBlur prop, so if the user enters text, then tags away, a tag is automatically created (when prop is true)

@bruderstein Everything seems cool. Except for one tiny thing. The grunt dev wont work anymore since the example files rely on react. We would at least need to add it as a dev dependency for the demos.

Ok, done.  I didn't realise peerDeps wouldn't be installed when they're in the root. Seems like a reasonable thing for npm to do.
I've rebased on your master again, and added react in as a devDependency in the same commit as it becomes a peerDep.

:+1:


copy to another location directions unclear

overall - great image, and great documentation.  For this one task, i'm unclear how to make a copy of my server to another location.  Do I zip a world file?  Do i set those environment flags, i don't get it, sorry :(

I miss understood this part of the guide.  This is for copying backups to another location, I thought this was starting a new location.


I use ignite on my react native app. How do I test on ignite. Really need your help. Test function mainly, login, snapshot,...

Hey @thpthang10
Thanks for your message but can you please be more explicit because your issue is a huge work.
I suggest you have a look at:
https://github.com/callstack/react-native-testing-library
If it solves your problem can your close your issue?

Hey @thpthang10
Thanks for your message but can you please be more explicit because your issue is a huge work.
I suggest you have a look at:
https://github.com/callstack/react-native-testing-library
or ask your question on https://stackoverflow.com/questions/tagged/react-native
If it solves your problem can your close your issue?

Thank you! I will close it.


Fixing cargo deny for branch x-hp

Time dependency:

[ ] build-data: https://gitlab.com/leonhard-llc/ops/-/issues/11
[x] rustls-acme: https://github.com/FlorianUekermann/rustls-acme/pull/43
[x] wmi should already be fixed, looking at the code

Git dependency:

[ ] derive_more: https://github.com/JelteF/derive_more/issues/259
[ ] ed25519-dalek

x-hp is merged, no deny issues.

This is not done yet.  These are followuos that block us from releasing

Whoops, sorry.


package?

Hi,
I'd like to use this some more in multiple projects, packaging it up would make that way easier. What do you think?
I'd be happy to make it into a package if you then register it on packagist? Or I could do that too even.
;)

Would appreciate that. There are a few updates I can make to this library as well.

Great. PSR-4 ?

Yeah. PSR-4 :)

Can I do anything extra to get this pull-request through and start using this package? https://github.com/jmathai/php-multi-curl/pull/23

x-posting.
@samthomson Looks like someone has the jmathai/ vendor on packagist for another repository of mine. Any idea how to add this to that or reclaim that vendor prefix?

@samthomson I'm inclined to rev the version of master with the 2.0 (see PR gh-24) version of this library. It's a good time to do it. I've added the tests and configured Travis to run on pushes and pull requests.
https://travis-ci.org/jmathai/php-multi-curl

CI looks good. Not sure about claiming existing vendor name.
I agree, v2, useful to have separation at least until packagist integration is tested.

Merged in gh-23


rendering spine animations that specify draworder keyframes result in misplaced bones

I have an animation that I'm exporting from the spine editor program. It has many animations in it. All of the animations play correctly when previewed in spine. But when I run them through pixi, some of them have very screwed up bone positions. I've noticed that all animations which specify a draworder keyframe have this problem, while the animations that lack that type of keyframe run perfectly.
I'm using pixi v2.2.7, and the newest spine editor (2.1.11 essential)
If there's any other information I can post to make this easier to debug I'm happy to help!

@englercj does the fact that it's been labeled as a bug mean this behavior has been determined to be a bug of pixi? I'm not sure if it is. Just curious if this has been investigated.

No determinations, means "not operating correctly". The tag doesn't assign blame.

ah ok, thanks for the clarification!

fixed in 2.2.8! awesomeness!!

I'm seeing a similar issue, even when using 2.2.8 and with an animation without any draw order keyframes.  @mreinstein Would you mind sharing generic pseudo-code of everything between new PIXI.Spine to stage.addChild()?

@oarmstrong I'm happy to help but given that your animations are giving you problems regardless of the presence of draw order keyframes, it's likely a separate issue. Please create another issue and ping me, we can pick up from there?


using data types of another module breaks puppet 5 compatibility

Error: Evaluation Error: Resource type not found: Icinga2::LogSeverity (file: /etc/puppetlabs/code/environments/production/modules/icinga/manifests/init.pp, line: 51, column: 12)
by applying examples/server1st.pp on puppet 5

If module of the custom data type  is not described in metadata as dependency

Will be fixed temporary by an own datatype Icinga::Loglevel


[boost-python] Fix build issue with feature python2

Fixes https://github.com/microsoft/vcpkg/issues/16081
Fix failures:
error: Name clash for '<pE:\vcpkg\clean\vcpkg\buildtrees\boost-python\x64-windows-rel\stage\lib>boost_python-vc142-mt-x64-1_73.dll'
error:
error: Tried to build the target twice, with property sets having
error: these incompatible properties:
error:
error:     -  <find-shared-library>python27 <include>/E:/vcpkg/clean/vcpkg/installed/x64-windows/include/python2.7 <python>2.7
error:     -  <find-shared-library>python39 <include>/E:/vcpkg/clean/vcpkg/installed/x64-windows/include/python3.9 <python>3.9
error:
error: Please make sure to have consistent requirements for these
error: properties everywhere in your project, especially for install
error: targets.

@PhoebeHui @JackBoosY
You can apply the attached patch to fix build with multiple Python versions.
fix-version-suffix.patch.txt
Usage example: vcpkg install boost-python[python2,python3]
More info: https://github.com/boostorg/python/issues/346

This PR makes python2 and python3 into alternatives, which is against the maintainer policy https://github.com/microsoft/vcpkg/blob/master/docs/maintainers/maintainer-guide.md#do-not-use-features-to-implement-alternatives.
We probably need a fix more like what @yurybura commented.

@yurybura @ras0219-msft, thanks for pointing out this! I will try the patch.

Dear @PhoebeHui,
The problem is in build-modular-build-helper. Please use this patch:
diff --git a/ports/boost-modular-build-helper/Jamroot.jam b/ports/boost-modular-build-helper/Jamroot.jam
index fc6af1355..4117fce60 100644
--- a/ports/boost-modular-build-helper/Jamroot.jam
+++ b/ports/boost-modular-build-helper/Jamroot.jam
@@ -29,7 +29,7 @@ rule tag ( name : type ? : property-set )

 rule python-tag ( name : type ? : property-set )
 {
-    return [ tag $(name) : $(type) : $(property-set) ] ;
+    return [ boostcpp.python-tag $(name) : $(type) : $(property-set) ] ;
 }

 if "@PORT@" != "boost-system"
diff --git a/ports/boost-modular-build-helper/vcpkg.json b/ports/boost-modular-build-helper/vcpkg.json
index d58b458d8..93a19943c 100644
--- a/ports/boost-modular-build-helper/vcpkg.json
+++ b/ports/boost-modular-build-helper/vcpkg.json
@@ -1,7 +1,7 @@
 {
   "name": "boost-modular-build-helper",
   "version-string": "1.75.0",
-  "port-version": 4,
+  "port-version": 5,
   "dependencies": [
     "boost-uninstall"
   ]

Thank you.

@yurybura, thanks for the fix, I have test the feature 'python2' it works well.

Awesome, thanks everyone!


Imported worlds don't carry over scoreboard objectives or teams.

The title basically says it all.
You can test by taking this world,
http://www.minecraftforum.net/forums/mapping-and-modding/maps/2559826-snowbomb-wars-by-epicelite3-v-1-0
Importing it, and then doing /scoreboard objectives list. None of the objectives normally in the map will be there.

Unfortunately I don't think there is anything we can do about this. Multiverse simply asks the server to load the world. If the server doesn't load the scoreboards then the problem lies in the server.

I'm more than willing to look into this (i'm curious) but as @dumptruckman said, I don't think this info is carried in a world folder. Might be a candidate for another plugin.
I do want to peek into the world/server API just to make sure before closing this issue


Confusing Typo

step 2 under Add the Kubernetes Cluster to the marketplace is says Add Services,  it should be All Services.
Also I missed the private preview notice. so when I couldn't find Kubernetes Cluster in the marketplace.  maybe a not there when the user does not see expected results would be helpful.

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

ID: 9e6bf86c-b6b3-19be-e357-00ef6fd0d1fe
Version Independent ID: 0729d43b-ad6d-2300-8c46-c9846697794a
Content: Add a Kubernetes Cluster to the Azure Stack Marketplace
Content Source: articles/azure-stack/azure-stack-solution-template-kubernetes-cluster-add.md
Service: azure-stack
GitHub Login: @mattbriggs
Microsoft Alias: mabrigg

@jeffbearer Thanks for the feedback! I have assigned the issue to the content author to evaluate and update as appropriate.
@mattbriggs it looks like this is already fixed on the pr, otherwise I would push this myself.

Thanks for your feedback, @jeffbearer. This has been updated, and will appear in the next content build. #please-close

@jeffbearer We will now proceed to close this thread. If there are further questions regarding this matter, please tag me in your reply. We will gladly continue the discussion and we will reopen the issue.


Docs

[ ] document general functionality overview, installation, configuration etc. in the README.
[ ] make sure API docs are complete for the public API

for rdoc.info


Update README.md

Example said "Symbol" when really it was "ticker". Works fine with ticker. Also edited surrounding code for 2fa, as wait for 2fa is not a requested argument, it just asks for the code without it, and when you add it it doesn't work. Feel free to ask for clarification if you're confused.

Good catch on the ticker arg, I didn't notice that!
I understand what you're saying, but I think you're confused about the wait_for_code argument. Yes, if you just provide your email, you can enter the code via the command line (which is a Python input() function), and it will work. But the wait_for_code flag is there for when you want to enter the code a different way. For example, I have a Discord bot that uses this API, and it asks the user to enter their Fennel email 2FA code through Discord instead of the command line. In this case, wait_for_code is set to False initially, so that the login process is started, the 2FA email is sent. Then it throws an exception related to 2FA, that is caught by the main program, and then the user is asked to enter the code through Discord. The same login function is then called again with code set to the 6 digit code, and it logs in successfully.
Hopefully this makes more sense and you can see why that part is needed in the README.

Good catch on the ticker arg, I didn't notice that!
I understand what you're saying, but I think you're confused about the wait_for_code argument. Yes, if you just provide your email, you can enter the code via the command line (which is a Python input() function), and it will work. But the wait_for_code flag is there for when you want to enter the code a different way. For example, I have a Discord bot that uses this API, and it asks the user to enter their Fennel email 2FA code through Discord instead of the command line. In this case, wait_for_code is set to False initially, so that the login process is started, the 2FA email is sent. Then it throws an exception related to 2FA, that is caught by the main program, and then the user is asked to enter the code through Discord. The same login function is then called again with code set to the 6 digit code, and it logs in successfully.
Hopefully this makes more sense and you can see why that part is needed in the README.

I did not understand it entirely. I'm just running this in my personal program. I still think the documentation could be expanded more to explain how it works (as this is meant for an API, not specifically your auto-rsa bot.) for different use cases. I will update the pull request accordingly tomorrow and get back to you. Thank you!


How I get the Message From listener it is printed in console

How I get and Store the Chat message from console it is shown but in flutter how i take the message from smack console

You must set a listner as mentioned in example file.
Did you already defined and listner and still not coming message into the listners, Please share your raw XML which is printing in your console.
@VasuBhav

I get the instant messages of the user I ask about the how to take the whole archieve messages from Mam I request the data but I didn't get any response on that function but smack print the sent and revc data

closing due to inactivity. @VasuBhav , feel free to reopen if you have  same problem.


panicked at 'index out of bounds: the len is 0 but the index is 0' from chalk code

Anytime I try to use rust-analyzer on the rustc codebase now, I get:
thread '<unnamed>' panicked at 'index out of bounds: the len is 0 but the index is 0', /home/runner/.cargo/registry/src/github.com-1ecc6299db9ec823/chalk-ir-0.71.0/src/fold/subst.rs:58:19
stack backtrace:
   0: rust_begin_unwind
             at /rustc/c8dfcfe046a7680554bf4eb612bad840e7631c4b/library/std/src/panicking.rs:515:5
   1: core::panicking::panic_fmt
             at /rustc/c8dfcfe046a7680554bf4eb612bad840e7631c4b/library/core/src/panicking.rs:92:14
   2: core::panicking::panic_bounds_check
             at /rustc/c8dfcfe046a7680554bf4eb612bad840e7631c4b/library/core/src/panicking.rs:69:5
   3: <chalk_ir::fold::subst::Subst<I> as chalk_ir::fold::Folder<I>>::fold_free_var_ty
   4: <chalk_ir::Ty<I> as chalk_ir::fold::SuperFold<I>>::super_fold_with
   5: <hir_ty::CallableSig as chalk_ir::fold::Fold<hir_ty::interner::Interner>>::fold_with
   6: <chalk_ir::Ty<hir_ty::interner::Interner> as hir_ty::chalk_ext::TyExt>::callable_sig
   7: hir_ty::infer::expr::<impl hir_ty::infer::InferenceContext>::infer_method_call
   8: hir_ty::infer::expr::<impl hir_ty::infer::InferenceContext>::infer_expr_inner
   9: hir_ty::infer::expr::<impl hir_ty::infer::InferenceContext>::infer_expr
  10: hir_ty::infer::expr::<impl hir_ty::infer::InferenceContext>::infer_expr_inner
  11: hir_ty::infer::expr::<impl hir_ty::infer::InferenceContext>::infer_expr
  12: hir_ty::infer::expr::<impl hir_ty::infer::InferenceContext>::infer_block
  13: hir_ty::infer::expr::<impl hir_ty::infer::InferenceContext>::infer_expr_inner
  14: hir_ty::infer::expr::<impl hir_ty::infer::InferenceContext>::infer_expr_coerce
  15: hir_ty::infer::infer_query
  16: salsa::runtime::Runtime::execute_query_implementation
  17: salsa::derived::slot::Slot<Q,MP>::read_upgrade
  18: salsa::derived::slot::Slot<Q,MP>::read
  19: <salsa::derived::DerivedStorage<Q,MP> as salsa::plumbing::QueryStorageOps<Q>>::try_fetch
  20: <DB as hir_ty::db::HirDatabase>::infer_query
  21: hir_ty::db::infer_wait
  22: hir::source_analyzer::SourceAnalyzer::new_for_body
  23: hir::semantics::SemanticsImpl::analyze_impl
  24: hir::semantics::SemanticsImpl::descend_into_macros
  25: ide::hover::hover
  26: std::panicking::try
  27: rust_analyzer::handlers::handle_hover
  28: std::panicking::try
  29: <F as threadpool::FnBox>::call_box
note: Some details are omitted, run with `RUST_BACKTRACE=full` for a verbose backtrace.
[Error - 9:29:00 PM] Request textDocument/hover failed.
  Message: server panicked: index out of bounds: the len is 0 but the index is 0
  Code: -32603
Panic context:
>
version: 009e6ceb1 2021-09-27 stable
request: rust-analyzer/inlayHints InlayHintsParams {
    text_document: TextDocumentIdentifier {
        uri: Url {
            scheme: "file",
            cannot_be_a_base: false,
            username: "",
            password: None,
            host: None,
            port: None,
            path: "/usr/local/google/home/tmandry/rust/compiler/rustc_codegen_llvm/src/attributes.rs",
            query: None,
            fragment: None,
        },
    },
}

It keeps panicking over and over so hopefully shouldn't be hard to reproduce.
Also mentioned in https://github.com/rust-analyzer/rust-analyzer/issues/10386#issuecomment-930033176

I'm getting something similar pretty often too.
thread '<unnamed>' panicked at 'index out of bounds: the len is 1 but the index is 1', crates\hir_def\src\nameres\path_resolution.rs:395:33
stack backtrace:
note: Some details are omitted, run with `RUST_BACKTRACE=full` for a verbose backtrace.
[Error - 7:15:16 PM] Request textDocument/hover failed.
  Message: server panicked: index out of bounds: the len is 1 but the index is 1
  Code: -32603

@tmandry this looks different than the one in https://github.com/rust-analyzer/rust-analyzer/issues/10386#issuecomment-930033176 since the version of your stacktrace is "009e6ceb1 2021-09-27 stable", which was released before #10305 got merged.
@cherryleafroad your error doesn't happen in chalk, but during name resolution. It may be #10084 instead.

Yeah, I thought this was due to a recent change but actually it occurs in the last 3 stable releases. I'm not sure why I was hitting it yesterday but not last week. I'll post more information if I hit it again.

Got the same issue even with the latest stable / nightly (2022-03-08 / 2022-03-09-nightly)

I see a similar panic in rustdoc.

thread '<unnamed>' panicked at 'index out of bounds: the len is 1 but the index is 1', /home/runner/.cargo/registry/src/github.com-1ecc6299db9ec823/chalk-ir-0.76.0/src/fold/subst.rs:60:19
stack backtrace:
   0: rust_begin_unwind
             at /rustc/9d1b2106e23b1abd32fce1f17267604a5102f57a/library/std/src/panicking.rs:498:5
   1: core::panicking::panic_fmt
             at /rustc/9d1b2106e23b1abd32fce1f17267604a5102f57a/library/core/src/panicking.rs:116:14
   2: core::panicking::panic_bounds_check
             at /rustc/9d1b2106e23b1abd32fce1f17267604a5102f57a/library/core/src/panicking.rs:84:5
   3: <chalk_ir::fold::subst::Subst<I> as chalk_ir::fold::Folder<I>>::fold_free_var_ty
   4: <chalk_ir::Ty<I> as chalk_ir::fold::SuperFold<I>>::super_fold_with
   5: chalk_ir::fold::boring_impls::<impl chalk_ir::fold::Fold<I> for chalk_ir::GenericArg<I>>::fold_with
   6: <smallvec::SmallVec<A> as core::iter::traits::collect::Extend<<A as smallvec::Array>::Item>>::extend
   7: chalk_ir::fold::boring_impls::<impl chalk_ir::fold::Fold<I> for chalk_ir::Substitution<I>>::fold_with
   8: chalk_ir::_::<impl chalk_ir::fold::Fold<I> for chalk_ir::WhereClause<I>>::fold_with
   9: chalk_ir::fold::binder_impls::<impl chalk_ir::fold::Fold<I> for chalk_ir::Binders<T>>::fold_with
  10: chalk_ir::Binders<T>::substitute
  11: hir_ty::chalk_db::convert_where_clauses
  12: hir_ty::chalk_db::impl_datum_query
  13: salsa::runtime::Runtime::execute_query_implementation
  14: salsa::derived::slot::Slot<Q,MP>::read_upgrade
  15: salsa::derived::slot::Slot<Q,MP>::read
  16: salsa::QueryTable<Q>::get
  17: <DB as hir_ty::db::HirDatabase>::impl_datum
  18: chalk_solve::clauses::program_clauses_that_could_match
  19: chalk_recursive::solve::SolveIteration::solve_iteration
  20: chalk_recursive::fixed_point::RecursiveContext<K,V>::solve_goal
  21: chalk_recursive::fixed_point::RecursiveContext<K,V>::solve_root_goal
  22: <hir_ty::db::TraitSolveQueryQuery as salsa::plumbing::QueryFunction>::execute
  23: salsa::runtime::Runtime::execute_query_implementation
  24: salsa::derived::slot::Slot<Q,MP>::read_upgrade
  25: salsa::derived::slot::Slot<Q,MP>::read
  26: salsa::QueryTable<Q>::get
  27: <DB as hir_ty::db::HirDatabase>::trait_solve_query
  28: hir_ty::db::trait_solve_wait
  29: <DB as hir_ty::db::HirDatabase>::trait_solve
  30: hir_ty::method_resolution::iterate_trait_method_candidates
  31: hir_ty::method_resolution::iterate_method_candidates_by_receiver
  32: hir_ty::method_resolution::iterate_method_candidates_dyn
  33: hir::Type::iterate_method_candidates_dyn
  34: hir::Type::iterate_method_candidates
  35: ide_completion::completions::dot::complete_dot
  36: ide_completion::completions
  37: std::panicking::try
  38: rust_analyzer::handlers::handle_completion
  39: std::panicking::try
  40: <F as threadpool::FnBox>::call_box
note: Some details are omitted, run with `RUST_BACKTRACE=full` for a verbose backtrace.
[Error - 12:07:30 AM] Request textDocument/completion failed.
  Message: server panicked: index out of bounds: the len is 1 but the index is 1
  Code: -32603

I can't reproduce the original error, rust-analyzer analysis-stats . on the rustc repo runs to completion. It's probably been fixed by the recent const generics changes. @jyn514 I don't get any panics on librustdoc either, though yours is during completion, so it may be running some query that we otherwise don't get. If you still encounter it, can you post a new issue? I'm going to close this one.


Migrate tests to Jest

Migrate tests to Jest – Jest (https://jestjs.io/) supports parallel test-running, which would be nice to have over Mocha. Most of its API surface is similar, so this shouldn't take too long.

Originally posted by @Zarel in https://github.com/Zarel/Pokemon-Showdown/issues/2444
Did some investigation here. Have some failing tests still, but I figured I'd dump what I've figured out so far:

Jest has a pretty similar API, but since it doesn't use this we could switch function () { ... } to () => { ... } which is the preferred style. I didn't bother with this.

A lot of our tests do weird things with timeouts, possibly due to them getting hit by Dex loading? Why do half of these fiddle with timeouts? I can see why team generator might, but basic tests for abilities or moves? Smells like a bug to me.

Jest's test runner is much noisier by default, and we'd probably need to write our own/depend on a new one. The biggest difference is that its logging even on success (which I'm OK with), but this means we get tons of:
PASS  test/simulator/abilities/roughskin.js
● Console

  console.log server/monitor.js:135
    NEW GLOBAL: global
  console.log server/monitor.js:135
    NEW CHATROOM: lobby
  console.log server/monitor.js:135
    NEW CHATROOM staff

for each test. Maybe we could stub out Monitor.notice etc in the beforeAll?

Hot take: Jest will be slower.  Is what my computer looks like running Jest, and I'm pretty sure most PS! developers don't have that kind of hardware sitting under their desk. To make matters worse - even on my box, its slower (13-14s instead of 10-12s)! I'm guess this is possibly due to the verbosity of the logging (writing that much to the console is slow), but I'd bet even on my computer its not going to be much faster than Mocha due to Dex loading. Dex loading is super slow, and if we're naively running tests in parallel processes the way Jest does, we're paying the price multiple times. The optimal way of running PS is to spin up a number of processes roughly equal to the number of cores (maybe NUM_CORES-1, to allow for the GC thread), and then run Dex loading once in each one, and then run multiple battles in a loop (incidently, the PS server does this :P)

Jest doesn't like Dex loading, it seems to cause failures when a test is skipped:

ReferenceError: You are trying to `import` a file after the Jest environment has been torn down.

      at ModdedDex.includeFormats (.sim-dist/dex.js:1643:17)
      at ModdedDex.get formats [as formats] (.sim-dist/dex.js:246:10)
      at ModdedDex.loadData (.sim-dist/dex.js:1584:80)
      at ModdedDex.includeData (.sim-dist/dex.js:1550:10)
      at Object.<anonymous>.process.nextTick (server/chat-plugins/info.js:2549:7)
.sim-dist/dex.js:1651
      throw new TypeError(`Exported property 'Formats' from "./config/formats.js" must be an array`);
      ^

TypeError: Exported property 'Formats' from "./config/formats.js" must be an array
    at ModdedDex.includeFormats (.sim-dist/dex.js:1651:13)
    at ModdedDex.get formats [as formats] (.sim-dist/dex.js:246:10)
    at ModdedDex.loadData (.sim-dist/dex.js:1584:80)
    at ModdedDex.includeData (.sim-dist/dex.js:1550:10)
    at Object.<anonymous>.process.nextTick (server/chat-plugins/info.js:2549:7)
    at processTicksAndRejections (internal/process/task_queues.js:79:9)

All in all, useful exercise, but not super sure about how much we want to invest in Jest?

If I had to guess Jest is slower because it runs multiple processes, which prevents the JIT from optimizing too much, and loading takes a while. 48 cores is kinda a lot.

That is because Chrome’s build system is smart enough to spawn more processes if you have more cores, which means that there are more terminating processes fighting over the global lock.

'smart' is used generously here, because clearly its resulting in worse results. Having more cores can definitely slow things down if they're not used properly though.

If I had to guess Jest is slower because it runs multiple processes, which prevents the JIT from optimizing too much, and loading takes a while

Yup. I'm curious though, when I get home I'll try it on my laptop which I think only has maybe 4 cores. I think its still going to be slower because the processes Jest spawns wont reuse Dex loading or benefit from caching etc.
That being said: https://github.com/facebook/jest/issues/6694

In my machine, with a barebones initialization, and running only the simulator/ tests, mocha uses ~23 seconds in my machine (2 cores [1]).
For a similar setup, jest uses ~140 seconds. This duration doesn't change noticeably with --maxWorkers=2, but it does increase a lot with maxWorkers=4 (despite Jest's estimates that each of those would result in halving the run time).
On the other hand, switching to a very simple runner which condenses all the simulator test suites into 4 suites reduces the total time to ~40 seconds. If it's condensed into 2 suites, the total time is ~30 seconds.... It seems to be just slightly faster than the case of 1 suite (~33 seconds), but I don't think that's a conclusive result....
Overall, it's worrying that this results in a performance degradation of anywhere from 50% to 500%.

[1] Initially I thought I had 4 cores, which is why I also tested for the cases o 4 workers / 4 suites.....

TL;DR: "Jest seems slow". Do we want to even bother to try and make it work? I'm happy to close this (once @Zarel weighs in) and remove the idea from #2444.

Jest has a pretty similar API, but since it doesn't use this we could switch function () { ... } to () => { ... } which is the preferred style. I didn't bother with this.

ESLint's rule prefer-arrow-callback has an autofixer. Just add it to test/.eslintrc and run it with --fix.

A lot of our tests do weird things with timeouts, possibly due to them getting hit by Dex loading? Why do half of these fiddle with timeouts? I can see why team generator might, but basic tests for abilities or moves? Smells like a bug to me.

Those sometimes to often timed out in my machine. The only explanation I could fathom was that they ran heavy events, but taking several seconds was always odd. That said, since the sim is at least 50% more performant nowadays, those custom timeouts might not be needed anymore.

we get tons of:

PASS  test/simulator/abilities/roughskin.js
● Console
console.log server/monitor.js:135
NEW GLOBAL: global
console.log server/monitor.js:135
NEW CHATROOM: lobby
console.log server/monitor.js:135
NEW CHATROOM staff
for each test. Maybe we could stub out Monitor.notice etc in the beforeAll?
We only need to tweak Config.loglevel. Furthermore, we don't actually need to start up the whole server for each test. It's possible to start just the sim itself nowadays. Of course, application tests still need the server, so that's the scope of the fix.

Jest doesn't like Dex loading, it seems to cause failures when a test is skipped:

It doesn't like us loading Dex in process.nextTick() from chat plugins. We can probably wait for a setImmediate() to resolve in the beforeAll() for application/plugins tests.

ESLint's rule prefer-arrow-callback has an autofixer. Just add it to test/.eslintrc and run it with --fix.

Yeah, jest-codemods does this as well, but I didn't want to cause so much churn.

Those sometimes to often timed out in my machine. The only explanation I could fathom was that they ran heavy events, but taking several seconds was always odd. That said, since the sim is at least 50% more performant nowadays, those custom timeouts might not be needed anymore.

I think this is worth followup. I don't know that I'm the best to try it though, because I think the tests run the fastest on my machines. Maybe @KrisXV can help us here?

We only need to tweak Config.loglevel.

Ahh, TIL.

Furthermore, we don't actually need to start up the whole server for each test. It's possible to start just the sim itself nowadays. Of course, application tests still need the server, so that's the scope of the fix.

I was going to say we should followup on that (not starting the server), but I guess it currently doesnt matter if we're sticking with mocha because the application tests run in the same process as the sim tests anyway.

It doesn't like us loading Dex in process.nextTick() from chat plugins. We can probably wait for a setImmediate() to resolve in the beforeAll() for application/plugins tests.

Good call. Though once again, not necessarily worth changing if we're not going to use Jest.
Thanks @Slayer95 for diving into this! You definitely did a better job than my naive attempt!

I'd agree with closing and removing the idea. I only ever wanted Jest because it seemed like parallel tests would be a speed boost, but I did give up on using Jest on client after benchmarking. It looks like it doesn't offer anything to be worth it.

I've been using it for all my personal libraries because I thought it would be better as well, but after now benchmarking on PS im reevaluating. Anyway, will close and remove the suggestion, thanks!


Add sort to libraries.html

I assume the qbk documentation that this links into will be built automatically in a master release.  Algorithms seems like the appropriate category for the new sort library.

Oops, I just noticed this is on master, I'm going to revert it, pull requests should be made against develop.

On Sun Feb 01 2015 at 4:06:02 AM Daniel James<EMAIL_ADDRESS>wrote:

Oops, I just noticed this is on master, I'm going to revert it, pull
requests should be made against develop.
—
Reply to this email directly or view it on GitHub
https://github.com/boostorg/boost/pull/53#issuecomment-72357223.

Got it, thanks.


Microwin cannot download iso

Describe the bug
When using the latest development version of winutil. trying to download the 24H2 iso fails.
To Reproduce
Steps to reproduce the behavior:

launch the latest dev build of winget
go to microwin tab
check get newest iso automatically and select 24H2
click get windows iso
See error

Invoking WPFGetIso
         _                     __    __  _
  /\/\  (_)  ___  _ __   ___  / / /\ \ \(_) _ __
 /    \ | | / __|| '__| / _ \ \ \/  \/ /| || '_ \
/ /\/\ \| || (__ | |   | (_) | \  /\  / | || | | |
\/    \/|_| \___||_|    \___/   \/  \/  |_||_| |_|
oscdimg.exe on system: False
[INFO] oscdimg.exe SHA-256 Hash: AB9E161049D293B544961BFDF2D61244ADE79376D6423DF4F60BF9B147D3C78D
Hashes match. File is verified.
oscdimg.exe was successfully downloaded from github
Error: The requested name is valid, but no data of the requested type was found. (vlscppe.microsoft.com:443)
Could not download the ISO file. Look at the output of the console for more information.

Expected behavior
The ISO should start downloading
Additional context
both download oscdimg.exe from CTT github and use ISO directory for scratch are checked.

This is an error from Fido. Please report these problems on the Fido repository

@FatBastard0, this is a different issue. You should make a separate issue

Ok, I will do that.
I would like to thank you in advance for any assistance that you can
provide in this matter. Please feel free to contact me at the below listed
address, Telephone and/or email.
Richard K. L'Italien
Disable Veteran of the US Army
The Fat Bastard
5637 Monte Corita Circle
Citrus Heights California 95621
(916) 915-3883 http://voice.google.com/calls?a=nc,%2B19169153883
http://voice.google.com/calls?a=nc,%2B19169153883 Voice
(916) 943 4967 http://voice.google.com/calls?a=nc,%2B19169434967
http://voice.google.com/calls?a=nc,%2B19167453437

This communication (including any attachments) may contain privileged or
confidential information intended for a specific individual and purpose,
and is protected by law.  If you are not the intended recipient, you should
delete this communication and/or shred the materials and any attachments and
are hereby notified that any disclosure, copying, or distribution of this
communication, or the taking of any action based on it, is strictly
prohibited.
Thank you.
On Wed, Oct 9, 2024 at 9:20 AM CodingWonders @.***>
wrote:

@FatBastard0 https://github.com/FatBastard0, this is a different issue.
You should make a separate issue
—
Reply to this email directly, view it on GitHub
https://github.com/ChrisTitusTech/winutil/issues/2896#issuecomment-2402767823,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/BJPGEYFWEY7G5VIAXLLPRBDZ2VJUDAVCNFSM6AAAAABPUWSNIOVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDIMBSG43DOOBSGM
.
You are receiving this because you were mentioned.Message ID:
@.***>

Issue closed due to inactivity, and due to the fact that it's not a MicroWin issue. /close


Remove support for python 3.10

Codecov Report

Merging #1121 (d7cee15) into devel (19b0996) will decrease coverage by 0.01%.
The diff coverage is 100.00%.

:exclamation: Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.
@@            Coverage Diff             @@
##            devel    #1121      +/-   ##
==========================================
- Coverage   90.69%   90.69%   -0.01%
==========================================
  Files          48       48
  Lines        5544     5543       -1
==========================================
- Hits         5028     5027       -1
  Misses        516      516

Files Changed
Coverage Δ

hahomematic/const.py
100.00% <100.00%> (ø)

hahomematic/platforms/custom/climate.py
99.38% <100.00%> (ø)

hahomematic/platforms/custom/const.py
100.00% <100.00%> (ø)

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


Cache account requirements

My understanding (per info from Rochak Mittal) is that cache accounts should be v1 storage accounts. It would be great if this could be documented here or possibly in the FAQ. Thanks!

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

ID: 8e22f219-b22a-b9d4-b0b7-e080b134da8d
Version Independent ID: b2e66450-6b64-c541-c4d3-c6caeaecac4e
Content: Configure replication for Azure VMs in Azure Site Recovery
Content Source: articles/site-recovery/azure-to-azure-how-to-enable-replication.md
Service: site-recovery
GitHub Login: @asgang
Microsoft Alias: asgang

@dcbrown16 Thanks for the feedback! I have assigned the issue to the content author to investigate further and update the document as appropriate.
@asgang Can you please take a look? Thanks a lot!

Rochak pointed out the documentation in the support matrix below:
https://docs.microsoft.com/en-us/azure/site-recovery/azure-to-azure-support-matrix#support-for-storage-configuration
As such, I think this can be closed, thanks for the support!


Security issues

Your snprintf implementation does not properly zero-terminate buffers, you might want to take a look here: https://github.com/mpaland/printf/issues/7
Just a nice hint for you, since I saw you using the library and just discovered these issues.

Thanks. zf_log doesn't rely on this string to be null terminated though. It uses returned length.

Might or might not be correct, either.


RTSP over WLAN0

Is your feature request related to a problem? Please describe.
I have an RTSP server installed in a raspberry PI. streaming over internet throught nrok and connected by wifi with my smartphone.
Describe the solution you'd like
I want to play the streaming of the raspberry with my smarphone in VLC.
Describe alternatives you've considered
Journalctl - f shows:
uri="rtsp://<IP_ADDRESS>:8554/unicast/track1"
Additional context
Thats URI is not correct because I want connect by Wifi, not LAN.
Can I change the interface?
Many thanks.

Hi,
By default live555 choose interface sending a multicast packet and looking which interface is receiving it.
You can override this default choice using  -I <wlan ip> argument.
Best Regards,
Michel.

Hi,
By default live555 choose interface sending a multicast packet and looking which interface is receiving it. You can override this default choice using -I <wlan ip> argument.
Best Regards, Michel.

HI again,
I test with "-I x.x.x.x", "-I x.x.x.x:8554" and -I "rtsp://x.x.x.x:8554" and no changes, follow with exit:
mar 22 08:00:18 sentinel v4l2rtspserver[26198]: Authorization: Digest username="admin", realm="LIVE555 Streaming Media", nonce="xxxxxxxxxxxxxx", uri="rtsp://<IP_ADDRESS>:8554/unicast/track1", response="xxxxxxxxxxxxxxxxxx"
thanks in advance for your support!

Hi,
It should be -I x.x.x.x
Best Regards,
Michel.


add message length prefix to stream

Thank you for contributing to Mina! Please see CONTRIBUTING.md if you haven't
yet. In that doc, there are more details around how to start our CI.
Explain your changes here. see #7724
will build off this for message types
Explain how you tested your changes here.
Checklist:

[ ] Document code purpose, how to use it

Mention expected invariants, implicit constraints

[ ] Tests were added for the new behavior

Document test purpose, significance of failures
Test names should reflect their purpose

[ ] All tests pass (CI will check this if you didn't)
[ ] Serialized types are in stable-versioned modules
[ ] Does this close issues? List them:

Closes #0000
Closes #0000

@noot It looks like some commits from https://github.com/MinaProtocol/mina/pull/7724 are missing (from 606a369 onward). Do we want to pull those into this PR?

!ci-build-me

!ci-build-me

!ci-build-me

!ci-build-me

!ci-build-me


Adjust brief to say fontbakery-check-ttf.py combines 3 checks (files, sets of files, and families)

I think the BRIEF set out having 1 tool that takes as input individual TTF files (section 2.1)
And then section 2.2 sets out a tools that checks a set of TTF files.
And 2.5 sets out another tool that takes as input a directory, that contains TTF files + METADATA.pb file + DESCRIPTION file, and ensures they all align.
I think you need to split up https://github.com/googlefonts/fontbakery/blob/master/fontbakery-check-ttf.py to match what the BRIEF describes.

On reflection, I think this is not very important, I think the current tool is fine. I'll update the BRIEF to reflect the tool.

done.


I know that you stole this code from me

If you do not take this down I will file a lawsuit and/or a creative commons license issue to take down this code now! If you don't take this down I will get this code taken down.

Bro his is updated and it’s not forked, I am pretty sure he’s legit also u can’t really sue someone for a code, especially it’s a hack. Both of you are gonna end up in jail.

you guys are both retards

I'm serious smint take this down now. We both know that this is mine and you are trying to get back at me for giving you a hard time with the blooket ui

@Gsnipe1blookethacks Just get your codes back up and if he tells u to take it down. then dont.


Add improper torsions, fix multi-term torsions in OpenMM export

Description
This PR adds support for improper torsions in SMIRNOFF and the export to OpenMM. Some of this is copied from the corresponding code in the toolkit, but energies are not yet equal.

Tests are still failing, although it's been difficult to isolate the cause. The proper torsions seem to be incorrectly implemented in some cases (only simple cases were tested before).

Thanks @j-wags for the invaluable contributions of  debugging and isolating useful test cases!

Thanks @j-wags for the invaluable contributions of  debugging and isolating useful test cases!


Add version: CodeF0x.violin version 3.1.1

Pull request has been created with WinGet Automation :rocket:
Microsoft Reviewers: codeflow:open?pullrequest=https://github.com/microsoft/winget-pkgs/pull/113344&drop=dogfoodAlpha

Possible duplicate package entry.
Similar installer SHA256 hash found in manifest:

InstallerSha256: 910d7dea9ffdf720d03226973f7fd6f6125f3b4e224ab7f983e3a56ae7accab8

manifests\c\CodeF0x\violin\<IP_ADDRESS>

Do we have a duplicate version?


enable SMS codes, update signup to link phone and email

Adminstrative Info
[Github Issue: Closes #]
Make sure your branch name conforms to:
<feature/staging/hotfix/...>/[username]/[3-4 word description seperated by dashes]. Otherwise, please rename your branch and create a new PR.
Changes
What changes did you make?

TODO

Testing
How did you confirm your changes worked?

TODO

Confirmation of Change
Upload a screenshot, if possible. Othwerwise, please provide instructions on how to see the change.

[TODO Screenshot Title]

[TODO Screenshot Link]

TODO:

[x] Implement SMS verification
[x] Update code to link phone and email after signup
[x] Implement confirm phone -> verification code flow
[ ] Implement ResetPassword page
[ ] Update phone number inputs to include country code (new package?)
[ ] Test that signing up now links phone and email to same account
[ ] Test general recaptcha flow


Added Effective Dart and copyWith method

The readme file needs to be corrected. I added just few badges.
How It looks Dart syntax after.

@iamarnas thank you :heart:

@iamarnas thank you :heart:

@hiranthaR It coming more fun stuff soon. Next PR will support Array of arrays 😊

@hiranthaR It coming more fun stuff soon. Next PR will support Array of arrays 😊


zork support?

Sorry this is a question rather than an issue. I haven't yet had a chance to use your script. Does your script support zork builds as well as rammus?

@darren3 Yep, should support. Report any errors if occurred


Kata/builder/start

Module 3 - Builder Patterns Exercises
Please review my code

Great work Maud, well done!


add macOS support

Resolves

Resolves #379

Proposed Changes

Firebase does not support macOS
change CONTENTS_FOLDER_PATH to UNLOCALIZED_RESOURCES_FOLDER_PATH in Run Script

Reason for Changes

Firebase does not support macOS
change CONTENTS_FOLDER_PATH to UNLOCALIZED_RESOURCES_FOLDER_PATH in Run Script

Test Coverage

[ ] macOS Catalina 10.15.6

Things that should be checked and done:

[ ] Camera: taking picture
[x] Recording: record and play sound
[ ] Exporting project
[ ] Importing project
[ ] Window resize?