Adding project for Site Recovery CSMv2 based specs.

Adding a new project for specs of Site Recovery Services based on the CSMv2 APIs.
API functionality has been added only for the enterprise to enterprise replication provider.
Basic framework for tests has been provided and test added.

Can one of the admins verify this patch?

Hi @NehaRaje, I'm your friendly neighborhood Azure Pull Request Bot (You can call me AZPRBOT). Thanks for your contribution!

In order for us to evaluate and accept your PR, we ask that you sign a contribution license agreement. It's all electronic and will take just minutes. I promise there's no faxing. https://cla.azure.com.

TTYL, AZPRBOT;

@NehaRaje, Thanks for signing the contribution license agreement so quickly! Actual humans will now validate the agreement and then evaluate the PR.
Thanks, AZPRBOT;

@azuresdkci add to whitelist


Error running command-line in Mac OS X: TypeError: '>' not supported between instances of 'str' and 'int'

Hi,
I have a problem using the CLI of ScribusGenerator.  I seem to be unable to run the scripts successfully.  Are there any suggestions, what I can do to get it working?  I don't get a logger file also, probably because the script does not get that far.
I use a Scribus in MacOS X Catalina 10.15.3.  I'm using Scribus 1.5.5. For python, I use the homebrew version.
Here is my output:
Traceback (most recent call last):
 File "ScribusGeneratorCLI.py", line 124, in <module>
   generator = ScribusGenerator(dataObject)
 File "/Users/hans/PERSONAL/Oedelem/ScribusGenerator/ScribusGeneratorBackend.py", line 73, in __init__
   logging.config.fileConfig(os.path.join(
 File<EMAIL_ADDRESS>line 79, in fileConfig
   handlers = _install_handlers(cp, formatters)
 File<EMAIL_ADDRESS>line 145, in _install_handlers
   h = klass(*args, **kwargs)
 File<EMAIL_ADDRESS>line 146, in __init__
   if maxBytes > 0:
TypeError: '>' not supported between instances of 'str' and 'int'

Thanks for the feedback.
Since the CLI version does not use Scribus at all the error could be somewhere with Python version.
It's working for me with python 2.7.17. What version are you using ?  find out by running python --version from the command line

I see you seem to be running python 3.8.1. Maybe convert the script to Python 3 and give it a go ?
running 2to3 -w *.py could do, more info: https://docs.python.org/2/library/2to3.html

I did use python2.7 in the end.
I tried to do 2to3 but it didn't change ScribusGeneraterCLI.  I did change the other scripts.
Running python ScribusGeneratorCLI still gave the error.  I'll try to have a look at it.

ported to python3 in branch https://github.com/berteh/ScribusGenerator/tree/python3
kindly test and let me know how this works for you.
I just don't want to make this the master, as the python embedded in my stock Scribus 1.5.5 is still python 2.7.

FYI: I was reproducing this error with the current master HEAD, under python 3.10.6 on Ubuntu 22.04.  Checking out the python3 branch seems to fix it.
Thanks for the generator; I use it to generate custom monthly diaries for myself.  It's a bit clunky to get everything together in the end, but I have old janky shell scripts to get the page order right for 2-up duplex printing and staple binding =)

python3 and MacOS are now included by default in master branch. plase update your script, test it out and don't hesitate to feedback !


Adding BrokenPipe to list of errors

Summary of the Pull Request
What is this about?
PR Checklist

[ ] Applies to work item: #xxx
[ ] CLA signed. If not, go over here and sign the CLI.
[ ] Tests added/passed
[ ] Requires documentation to be updated
[ ] I've discussed this with core contributors already. If not checked, I'm ready to accept this work might be rejected in favor of a different grand plan. Issue number where discussion took place: #xxx

Info on Pull Request
What does this include?
Validation Steps Performed
How does someone test & validate?

Sorry for the confusion. I only meant to create this as a draft. It didn't resolve the issue I was having with the Proxy VM. We can toss this PR out.


fix: use git https protocol

Due to lua_modules make target invoking rover lock command before translate_git_protocol (see #1320), commonly used targets like dependencies often fail with a Connection refused error.
Reported in #1375.

I do not think that the .gitconfig is needed because the translate_git_protocol adds the config globally inside the container, but does not do any harm.

Adding globally inside the container is exactly the problem here. Since the git context for global is the project directory itself, git config --global adds the .gitconfig file to the changes. Even after you leave the container, you still see that file uncommitted.
We could try .gitignoring .gitconfig instead if you like.

nce the git conte

Ok I see, $HOME is the APIcast working dir. So, what I would do is to add the .gitconfig to the .gitignore and we do not need to commit it, as the content is duplicated in the Makefile. Not a big deal, as you want. The PR is already approved.


When the manifest.json is not correct it shows "Incorrect file type" instead an actual error.

When sync a catalog with an incorrect manifest.json is not correct it shows "Incorrect file type" instead an actual error. We found out the issue was that we cannot have a default with value and required: true in the parameters, there was a lot of trial/error on catalog sync to figure it out.
Example param:

  - id: location
    name: 'Location'
    description: 'Region location'
    type: string
    default: 'eastus'
    required: true

Hi @mahomedalid, thanks for reaching out!
There is work currently in progress that will make errors like this more specific and easier to debug.

Thank you @ericaguthan ! We also had a few other instances with confusing errors, instead of creating a issue we can maybe just report them here. For example, when an environment fails to be created it shows something like: "Provisioned state failed", without any specifics about why. Of course, sometimes we can go to the deployments and see the error, but sometimes it is something related to the parameters or something else, that does not show up because the deployment has not been created.


AGC Badge App - Add AGC Rental as a Club/Sponsor

Please add "AGC Rental" as a Club/Sponsor to the Calendar App. This will allow me or other Admins to add Corporate Customer events to the calendar. These businesses just rent range time from us periodically.

This must be done on the badgeApp,  Just add a new club with the proper rights.


Home page: Make charts interactive, with the links taking you to a filtered program page

Right now our home page charts are purely informational, like so:

We would like to make them fully interactive in the same way that our program page charts are, taking you to filtered lists of indicators. In this case, the home page chart links would take you to a filtered view on the appropriate program page.
In addition to displaying the links and hooking them up to the right queries, there is highlighting behavior we need to mimic.
Acceptance criteria
Queries
All of the following chart links take you to a filtered program page and display the expected results.

[x] 1. Indicators on track: % are >15% above target
[x] 2. Indicators on track: % are on track
[x] 3. Indicators on track: % are >15% below target
[x] 4. Indicators with targets: Add missing targets
[x] 5. Indicators with results: Add missing results
[x] 6. Results with evidence: Add missing evidence

UI details

[x] 1. Indicators on track:

[x] 1.1. onHover yellow background highlighting of the three clickable metrics work.
[x] 1.2. onClick yellow outline highlighting behavior around the entire block works.

[x] 2. Other three charts:

[x] 2.1. onHover yellow highlighting of the entire block works.
[x] 2.2. onClick yellow outline highlighting of the entire block works.

Deployment Verification

[x] Confirmed “Ready for Deploy” status

The following Filters bring user to the correct filtered program page with related filter clicked:

[x] Indicators on track: % are >15% above target
[x] Indicators on track: % are on track
[x] Indicators on track: % are >15% below target
[x] Indicators with targets: Add missing targets
[x] Indicators with results: Add missing results
[x] Results with evidence: Add missing evidence

The following issue applies to all three gauges to the right of "Indicators on track"
Each of the "Add missing ____" links are unresponsive. You can click on the gauge to go to the filtered program page, but not the call to action link itself.

For comparison's sake, go to the program page and click on those links -- they respond as expected there, activating the filters.

One more detail related to the "Add missing targets" links on all three charts where they appear: Please add a space between the warning icon and the first letter of the string -- can use the Program page as a reference.
Here's how it looks now -- no space

As opposed to this on the program page, with space:

QA Verified

Add missing _ now has more spacing between the icon and words
Tested all 3 LINKS and they filter, and the highlights on the gauges between program/home page also remain functional

Pre-Deploy Check:

[x] Non-regressed
[x] Feature/fix can be found in release branch in staging environment, functioning as expected.

If above is checked, this feature/fix is DEPLOY READY.

Fixed in Hucklepatch


fix(WIP) Adds stack discipline to template block parsing in the markd…

…own-it plugin
Signed-off-by: Jerome Simeon<EMAIL_ADDRESS>Issue #288
Adds proper handling for nested blocks (with stack discipline) to markdown-it-template plugin.

Initial work in PR #300
With it, nested lists get parsed in the properly nested way:

Coverage increased (+0.03%) to 91.752% when pulling 1b6f8be415da5713fc2b716e151436685f5c0bdb on jeromesimeon:js-fix-nested-blocks-288 into 1c6df8ea89fad0b7d7182467bac5916792bc85b2 on accordproject:master.


Pulseaudio and ALSA module input config

I would like to be able to individually enable and disable scroll and click handling for Pulseadio or ALSA. Although there is already a handle-events option, I would like to keep the click to mute without the scrolling. Setting interval = 0 works but feels hacky. I think a new config option would be the best solution. To do this config options for enable-scroll and enable-click should be added to both modules. I can easily add these features, but I wanted some input before I made a pull request. Also, I noticed alsa and pulseaudio are the only modules that use the handle-events option. Should it be replaced or added to more modules?

I have some thoughts on this, will have to wait until after Christmas to respond.


GAdmin has no attribute mappings

GLPI (presumably) has fields that should be mapped to Google Directory. Examples of Google Directory attributes include first name, last name, primary email, phone number, and address. At least one and as many as needed should be mapped to an app attribute (something in GLPI that matches the data in Google Directory to complete user authentication).
Potential resolution: investigate what the GLPI app attribute should be, add to documentation

Currently, there is no field mapping.  The plugin only checks the nameID for an email match.  (Line 64 in phpsaml.class.php).
This would be a good enhancement.  I'll consider it in a future release along with JIT provisioning.

Added field mapping for Name, Firstname, Surname, Emailaddress.
$password = bin2hex(random_bytes(20));

					$input = array(
						"name" => SELF::$userdata['http://schemas.xmlsoap.org/ws/2005/05/identity/claims/name'][0],
						"realname" => SELF::$userdata['http://schemas.xmlsoap.org/ws/2005/05/identity/claims/surname'][0],
						"firstname" => SELF::$userdata['http://schemas.xmlsoap.org/ws/2005/05/identity/claims/firstname'][0],
						"_useremails" => array(SELF::$userdata['http://schemas.xmlsoap.org/ws/2005/05/identity/claims/emailaddress'][0]),
						"password" => $password,
						"password2" => $password,
					);


Valid scope values of the connection auth method object

The scope value of the connection auth method object can be also module when device module sends telemetry message.

@Dietatko : Thanks for your contribution! The author(s) have been notified to review your proposed change. @ash2017

@jlorich, can you review the proposed changes? I'm unable to reach the listed author, and I see that you have contributed to the article.
IMPORTANT: When the changes are ready for publication, add a #sign-off comment to signal that the PR is ready for the review team to merge. Thanks.
#label:"aq-pr-triaged"

@ash2017 @kevinlam1 @jlorich - Can you review this old PR and determine whether it needs to be closed or merged?
IMPORTANT: When the changes are ready for publication, add a #sign-off comment to signal that the PR is ready for the review team to merge.

This is an IoT Hub article that is not under my purview.

@ShannonLeavitt Ashita and Joey have left the IoT Hub team. I've taken over the content since this PR was originally created and approve the change.
#sign-off


Add sleep

Thanks for the PR, but I don't think we should add something like sleep to the library. You can start an issue and see what others think, but my opinion is that anything not related to Vue shouldn't be included in the library

I think that when asynchronous processing is executed in Promise, it needs to sleep asynchronously.

I see.
I will make issue.

I think it is necessary because it is only 3 lines.


KAFKA-13134: Give up group metadata lock before sending heartbeat response

What
Small locking improvement to drop the group metadata lock before invoking the response callback.
Testing
Existing GroupCoordinator tests pass.
Committer Checklist (excluded from commit message)

[ ] Verify design and implementation
[ ] Verify test coverage and CI build status
[ ] Verify documentation (including upgrade notes)

@dajac
do you mind taking a look?

@splett2 do you have the lock profile before and after the patch？


kubefedctl disable should not accept --federation-group or --version parameters

These options are only valid for kubefedctl enable.
/kind bug

disable now accepts --kubefed-group and --version and --delete-crd is provided to allow deletion of a crd in the absence of an FTC.


Stopping one container stops all containers

I am running about 5 containers (4 applicatons and nginx) and I have created a unit file for each. Unfortunately when I stop or restart any one, all the rest go down with it and systemd still shows all of the processes as active. Is this an issue with my unit files? Here are a couple unit files for reference http://hastebin.com/ifahapoqet http://hastebin.com/ramokunube
Here's hoping this works better than when I asked the systemd irc channel!

I think I have a similar issue: when I stop container service it terminates docker.service so container can't be restarted... Any ideas?


Obtain crash report in string format

I would like to obtain the crash report in a string format and put the crash report's contents into my own log file. I dont want to upload the crash report to any server and do not want to send it via email either. Out of all the KSCrash installation types available, KSCrashInstallationConsole seems like the one to use for my scenario. But the console installation has comments that it's used for testing purpose only. So I would like to know what is the right way to retrieve crash report in string format using KSCrash? I am new to obtaining stack trace from iOS apps so any help is appreciated.

First, you can write a custom installation class Inherited from KSCrashInstallation and override - (id<KSCrashReportFilter>)sink method return a custom Sink which conform to KSCrashReportFilter protocol. Then, install the custom installation and call - (void) sendAllReportsWithCompletion:(KSCrashReportFilterCompletion) onCompletion method, you will get filteredReports in the onCompletion callback.

Thanks @ZhiyuWong. This is helpful. Is there any example that does the same/something similar that I can refer to?

@kstenerud - any chance you have an example for this scenario? It would be really helpful. Thanks.

There is an another solution for manual processing of crash reports:
// Filter for Apple format text representation of a crash report
KSCrashReportFilterAppleFmt *filter =
    [[KSCrashReportFilterAppleFmt alloc] initWithReportStyle:KSAppleReportStyleSymbolicatedSideBySide];

// We get all crash report IDs
NSArray *reportIDs = [KSCrash sharedInstance].reportIDs;
for (NSNumber *reportID in reportIDs) {
    // We load dictionary representation of a crash report
    NSDictionary *report = [[KSCrash sharedInstance] reportWithID:reportID];
    // We can do anything with this dictionary(see <KSCrash/KSCrashReportFields.h>)
    // To convert to Apple format we use <KSCrash/KSCrashReportFilterAppleFmt.h>
    [filter filterReports:@[report] onCompletion:^(NSArray *filteredReports, BOOL completed, NSError *error) {
        if (completed) {
            NSLog(@"Report '%@':\n%@", reportID, filteredReports.firstObject);
        }
        else {
            NSLog(@"Failed to generate text report for crash '%@': %@", reportID, error);
        }
    }];
}


Update d.ts and module init

d.ts file was updated. Module can be via /// reference types="jose-jwe-jws" and import * as jose from "jose-jwe-jws";
If module is under nodejs Jose has empty crypto. It doesn't throw exception now
Added new types for algo

LGTM. We'll need a CLA before I can merge.
Let me know if you want me to tag + release on npm or if you don't care about that?

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

Thanks for the contribution! Do you need a release on npm?

Yes, I do. It would be good


[RFC] Include DCEGM main function in HARK

To keep the various figurative balls rolling, it's convenient to start including some the tools in a tool-y manner. The upper envelope step from DCEGM is such a tool, and it is important for the DCEGM REMARKs, the NEGM DEMARK and REMARK, and the TFI deliverable.
@mnwhite I've taken out the function and thrown it in a file. I hope this makes it easier for you to look through.
Compared to the DCEGM PR it is cleaned up a bit, which should make it a) easier to read, b) slightly faster, and c) slightly more correct (an index was off by one).
The reason why I would love to have this merged as a function and not bundled with some discrete/continuous type class, is that it's probably (or certainly) going to be relevant beyond what we can achieve with inheritance. By that I mean, that some users will want to call this directly, not just through some class we might define. One example of this would be the REMARKs and DEMARKs mentioned above.
To be discussed

[ ] Names
[ ] Exact location (what file,...)
[ ] Good tests? (slightly optional, as momentum is first priority here)
[ ] (is lower_extrap really needed, I think the code above should take care of this not happening)
Any any other comments you might have.

P.S. I'm returning a copy of the common grid. This may seem weird, but in the original paper they discuss adding the exact crossing points to the grid for increased precision. I may come back to this, but it didn't seem super important. It can make the discrete choices a bit more precisely located I guess. It's 1D though, so we can afford quite a lot of points (though the NEGM context might be an exception).

For those interested, this again shows how relatively straight forward it is to add tests for the functions we provide. The test I added takes a sequence of points that "could" come from an necessary (but not sufficient) EGM step. It's designed to fold back over itself once, and what I'm basically checking is that the "upper envelope" is actually equal to the value of the segments that is upper... More testing would include verifying that upperM[0], upperC[0], and upperV_T[0] are indeed 0.0, such that stuff like lower_extrap can be avoided.

I didn't mention it explicitly here, though it's visible from the references above, but https://github.com/econ-ark/DemARK/pull/27 shows this in action.

@mnwhite it'd be quite useful if you could voice you opinion on where you think these things should reside. Right now, these functions are used in a DemARK https://github.com/econ-ark/DemARK/pull/27 and REMARK  https://github.com/econ-ark/REMARK/pull/9 that are getting close to ready, but they're both "copying code" from this PR to be able to do so, so it'd be great if they could just be from HARK.xxx import yyy, zzz statements instead.
I'm curious about your thoughts on naming, placement in the folder tree, and if it's fine that they're introduced as separate functions for now, with inclusion in some generalized ConsIndShkConsumer in the future when we discuss the organization/structure-update.

Tomorrow is entirely an Econ-ARK day, expect long email/post about this.
On Mon, Mar 18, 2019 at 7:08 AM Patrick Kofod Mogensen <
<EMAIL_ADDRESS>wrote:

@mnwhite https://github.com/mnwhite it'd be quite useful if you could
voice you opinion on where you think these things should reside. Right now,
these functions are used in a DemARK econ-ark/DemARK#27
https://github.com/econ-ark/DemARK/pull/27 and REMARK econ-ark/REMARK#9
https://github.com/econ-ark/REMARK/pull/9 that are getting close to
ready, but they're both "copying code" from this PR to be able to do so, so
it'd be great if they could just be from HARK.xxx import yyy, zzz
statements instead.
I'm curious about your thoughts on naming, placement in the folder tree,
and if it's fine that they're introduced as separate functions for now,
with inclusion in some generalized ConsIndShkConsumer in the future when
we discuss the organization/structure-update.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/econ-ark/HARK/pull/226#issuecomment-473866985, or mute
the thread
https://github.com/notifications/unsubscribe-auth/ANUQFYGWyaI88xe4iI_UrSUgEp4noBU4ks5vX3PKgaJpZM4bCwOC
.

Tomorrow is entirely an Econ-ARK day, expect long email/post about this.

Sounds great! Whatever you prefer.

This is still at the top of my list, haven't forgotten.
On Mon, Mar 18, 2019 at 5:17 PM Patrick Kofod Mogensen <
<EMAIL_ADDRESS>wrote:

Tomorrow is entirely an Econ-ARK day, expect long email/post about this.
Sounds great! Whatever you prefer.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/econ-ark/HARK/pull/226#issuecomment-474104658, or mute
the thread
https://github.com/notifications/unsubscribe-auth/ANUQFRqEwzhTiTjDJrp5r2GT95u8VHPVks5vYAJXgaJpZM4bCwOC
.

This would also be nice to discuss and merge before a new release cc @shaunagm @mnwhite

Okay, so I pushed some changes last night, and I think we should merge. The changes were: rename to dcegm and rename dcegmSegments to calcSegments. The intended use is then
import HARK.dcegm

dcegm.calcSegments(...)
dcegm.calcMultilineEnvelope(...)

This should be sufficiently clear to users of these functions, which will probably not be many, once the discrete choice solver class lands. The latter calls the former, so the former is really only split out for a) convenience when debugging and b) DemARKs etc. We may want to add more convenience later, but I think this is what we need, and it should be merged soon (now?)

Patrick,
Great, thanks, in my view it's ready to merge.
We'll discuss at 10am.
On Thu, Apr 11, 2019 at 4:16 AM Patrick Kofod Mogensen <
<EMAIL_ADDRESS>wrote:

Okay, so I pushed some changes last night, and I think we should merge.
The changes were: rename to dcegm and rename dcegmSegments to calcSegments.
The intended use is then
import HARK.dcegm
dcegm.calcSegments(...)
dcegm.calcMultilineEnvelope(...)
This should be sufficiently clear to users of these functions, which will
probably not be many, once the discrete choice solver class lands. The
latter calls the former, so the former is really only split out for a)
convenience when debugging and b) DemARKs etc. We may want to add more
convenience later, but I think this is what we need, and it should be
merged soon (now?)
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/econ-ark/HARK/pull/226#issuecomment-482015669, or mute
the thread
https://github.com/notifications/unsubscribe-auth/ABQlfwMNck7jZVykYlpaypyW3NSmnii1ks5vfu9GgaJpZM4bCwOC
.

--

Chris Carroll

@pkofod @llorracc Does "Adds DCEGM main function" make sense for a release note?  If not, please provide alternative, thanks.


Fix menu styles for smaller breakpoints

Summary
This PR fixed the menu styling at smaller breakpoints. Specifically, the top nav items wrap to two lines before we get to the mobile menu. I've made a couple reductions in font size and padding to remediate the issue.
cc: @cwarnermm – Not sure who else to tag as a reviewer on this. I've temp tagged Ben, but feel free to change if you have other ideas.
I'm just gonna view the preview site, and then remove the WIP tag.
Thanks!

@emdecr - Is the goal of this PR to eliminate broken menu text lines altogether, or to eliminate the majority of bad line breaks?

@cwarnermm I'm trying to mitigate just the top level items from wrapping to two lines altogether. If the sub-menu items wrap, that's less of an issue imo.

Are you able to give me an example of a viewport width that causes text to wrap like in the photo?
Also how does your local site look? My looks fine, but I think we're getting that weird thing again when the preview sites are wonky?

@emdecr I haven't tried building this one locally yet since the generated preview has been behaving. I can see the top-level wrap occur at 978x855.

@cwarnermm Weird. I sent the preview site to other folks and the generate preview is broken for them as well.
http://mattermost-dev-docs-preview-pulls.s3-website-us-east-1.amazonaws.com/1010
Weird thing is, previous preview site of PRs that were merged are also broken (ex. #1002):
http://mattermost-dev-docs-preview-pulls.s3-website-us-east-1.amazonaws.com/1002
@spirosoik My branch is up to date with master, so I don't think I changed any of changes you made to fix the broken preview sites last time. Do you know if we need to do anything on the s3 bucket? Thanks!

@emdecr @spirosoik - previews are also broken for me now too.

/update-branch

@emdecr - @spirosoik updated this branch with the latest master by typing /update-branch as a GitHub PR comment. This appears to have resolved the generated preview issues we were experiencing. This is a good option to try in the future if we see this issue again.
@BenLloydPearson Can you please confirm that previews for this PR are working for you too?

@emdecr - This PR can be merged once you're ready to remove the WIP label.

@cwarnermm Blerg – yet again, I've forgotten. 'Tis removed. Thanks y'all!


no-rtti factor

There should be a factor that adds -fno-rtti and /GR- for CMake builds and also rtti=off for the b2 workflow
Something like this seems to suffice:
B2_ARGS+=(${{ (inputs['no-rtti'] == 'true' && '"rtti=off"') || '' }})

I've also come to learn, /GR- on msvc toolchains seems to necessitate us explicitly passing /EHsc which I assumed it did by default but I guess it doesn't. Updating the code to appned entry['cxxflags'] += ' /GR- /EHsc' seems to work fine in practice.

I've included the rtti option for the b2-workflow action. true adds rtti=on to the args, false adds rtti=on to the args, and everything else adds nothing. I'll convert this action to JavaScript at some point to make it easier to maintain.
About always recommending '/EHsc'` directly from the matrix, this should remain as the user's responsibility because it's not composable or generic enough and can minimize surprises because the convention in CMake is not to assume the user has to pass required flags manually: if only the user needs /EHsc, they can provide it via the command line. If the project always needs /EHsc with no exception on MSVC, then it should embed it in the (interface) target.


Improve ergonomics between SockaddrIn and underlying libc type

What does this PR do
Checklist:

[x] I have read CONTRIBUTING.md

[x] I have written necessary tests and rustdoc comments

[x] A change log has been added if this PR modifies nix's API

Add Into<libc::sockaddr_in6> for SockaddrIn6

Add Into<libc::sockaddr_in> for SockaddrIn

My typical use case for these Into traits is when I am bridging from safe world into the world of pain (C). For example,  presently I have to go over as_ref() to access the underlying sockaddr_inX . This is a bit verbose for my liking, so I figured it would be great to have into() to do the job for me.  Usually From is preferred but I am not certain if we want that, hence I have implemented Into but can easily flip to From if that makes sense.
let addr_sa = SockaddrIn6::from(SocketAddrV6::new(addr, 0, 0, 0));
let mut req: in6_aliasreq = unsafe { mem::zeroed() };
-req.addr = *addr_sa.as_ref();
+req.addr = addr_sa.into();

I also noticed that repr(transparent) is set on these structs and naturally I thought I could use addr_sa as sockaddr_in6 but that doesn't seem to compile.

This is indeed something we should do to allow Nix to be used in such scenarios, see #2327, and thanks for your interest in contributing to Nix!

CI failure in test_sigsuspend is probably unrelated.


[FATAL] toml: type mismatch for main.StaticConfig: expected table but found []interface {}

When trying to install and start the service after configuring the .toml file, I am receiving the error in the title. I put these lines at the very bottom of the file per the documentation (I have been following the documentation and using the example servers and relays.)
[static]
routes = [
     { server_name='odoh-cloudflare', via=['odohrelay-crypto-sx', 'odohrelay-surf'] },
     { server_name='odohrelay-koki-ams', via=['odohrelay-koki-ams', 'odohrelay-koki-bcn'] }
]

You don't need to add a static entry at the very end of toml file. You just need to uncomment below part under Anonymized DNS section of the file and edit servers in it.
# routes = [
#    { server_name='example-server-1', via=['anon-example-1', 'anon-example-2'] },
#    { server_name='example-server-2', via=['sdns://gRIxMzcuNzQuMjIzLjIzNDo0NDM'] }
# ]


Localization updates

fixes issue #600
moved (partially) Get.locale to rootController (should be in it's own package domain)
fixed updateLocale to invalidate MaterialApp
added Get.systemLocale and Get.systemLocale to suggestion of using dart:ui
updated comment docs in several places.
updated README with internalization details and steps to include  flutter_localizations (which should be mandatory).

sounds nice to me 😄
now we have the basics of how to make your app accept localizations, but do we have docs of how to use getx localization?

I push the update to the README.

Thanks for that PR, it's a really good fit in the docs, and in the code, and Get.defaultLocale is cool.
I just have a few caveats:
1- In web projects, using the flutter_localizations sdk is often not an option. In my last project this was vetoed due to the significant increase in the .js file.
I think maybe it would be better to demonstrate how to use it without any package (like now), and how to use it in conjunction with flutter_localizations.
2- I am afraid that using update + forceAppUpdate will cause double rebuilds. I haven't tested it, but looking at the code, it looks like it does it.

Regarding
1 - Check this article for Deferred loading localizations. So, at least in my experience, if you changed the Locale, and dont update the MaterialApp, and then you change the ThemeMode, app will crash cause lots of Material Widgets have localized Strings (like semantics or toolrips) that can't be found. If there's a workaround for that, please let me know so I can update the README... but according to my tests, flutter_localizations was a forced things to add.
2 - I believe you can't escape the forceAppUpdate() call, and certainly need to update MaterialApp to apply the update to material widgets (RTL change for instance), several users had the issue of not having the app refreshed, forcing a call to Get.changeTheme/Get.changeThemeMode to invalidate the rootController.

Thanks Jonny:
1 - Check this article for Deferred loading localizations. So, at least in my experience, if you changed the Locale, and dont update the MaterialApp, and then you change the ThemeMode, app will crash cause lots of Material Widgets have localized Strings (like semantics or tooltips) that can't be found. If there's a workaround for that, please let me know so I can update the README... but according to my tests, flutter_localizations was a forced things to add.

Deferred loading does not reduce the size of the .js file in Flutter (unfortunately).
Material is not used in web applications due to the size of the .js file (at least in my projects, it is always necessary to comment on the "use-material: true" of pubspec, and all material widgets that use icons, such as datepicker are redone from scratch).
I created internationalization for a real need in a project that I was inserted to have less than 800kb of site load, and I believe this is a recurring problem in Flutter, so the localization sdk, in my opinion, cannot be mandatory . It may even be suggested, but not mandatory, precisely because of the Flutter web file size. Well, at least in my experience with Flutter web in professional projects it required this.

2 - I believe you can't escape the forceAppUpdate() call, and certainly need to update MaterialApp to apply the update to material widgets (RTL change for instance), several users had the issue of not having the app refreshed, forcing a call to Get.changeTheme/Get.changeThemeMode to invalidate the rootController.

You are right, but I will try to think of an alternative to improve this in the future.
Thank you for another contribution.
I would just like to see the change in the sdk and internationalization part in a separate topic, such as "using get with flutter_localizations", and inform that this is necessary for RTL and for internationalization of Flutter's internal widgets.

@jonataslaw , sure i will change that in the README. You sure you have no exceptions when passing a different Locale?

@jonataslaw , sure i will change that in the README. You sure you have no exceptions when passing a different Locale?

Yep, as we talked about yesterday, the exception is only thrown when "supportedLocales" is filled.
In addition, I did a PR to support RTL, so the use of the Flutter_internacionalizations SDK is restricted to who will use Material. As Material is not used on the web due to the size of the js file, I believe that the SDK part should be in a separate topic for those who want to use both

This was a good PR, I will reuse it in the current version


[Refactoring] Check/Update level of log messages

[ ] Define how to decide which log level to use
[ ] Check/update all existing usages to match strategy

We should discuss in the meetings to Define how to decide which log level to use


remove duplicate /bin

"/bin/**" is present twice in sample.toml and README.md.
Signed-off-by: Thomas Sjögren<EMAIL_ADDRESS>

nice thanks!


Trafilatura can't be loaded after installing it to local folder

Hi,
I just tried to install your awsome project to an local folder (pip install --target {path}/package trafilatura). After installing it I cant load it with from package import trafilatura:
---------------------------------------------------------------------------

KeyError                                  Traceback (most recent call last)

~\anaconda3\lib\site-packages\pkg_resources\__init__.py in get_provider(moduleOrReq)
    359     try:
--> 360         module = sys.modules[moduleOrReq]
    361     except KeyError:

KeyError: 'trafilatura'

During handling of the above exception, another exception occurred:

ModuleNotFoundError                       Traceback (most recent call last)

<ipython-input-37-57dd4ca0a626> in <module>
      1 from pprint import pprint
----> 2 from package import trafilatura
      3 import time
      4
      5 trafilatura

~{path}\package\trafilatura\__init__.py in <module>
     14 import logging
     15
---> 16 from .core import extract, process_record
     17 from .utils import fetch_url
     18

~{path}\package\trafilatura\core.py in <module>
     17
     18 # own
---> 19 from .external import justext_rescue, sanitize_tree, SANITIZED_XPATH, try_readability
     20 from .filters import content_fingerprint, duplicate_test, language_filter, text_chars_test
     21 from .htmlprocessing import (convert_tags, discard_unwanted,

~{path}\package\trafilatura\external.py in <module>
     36 from .settings import JUSTEXT_LANGUAGES, MANUALLY_STRIPPED
     37 from .utils import trim, HTML_PARSER
---> 38 from .xml import TEI_VALID_TAGS
     39
     40

~{path}\package\trafilatura\xml.py in <module>
     21 LOGGER = logging.getLogger(__name__)
     22 # validation
---> 23 TEI_SCHEMA = pkg_resources.resource_filename('trafilatura', 'data/tei-schema.pickle')
     24 TEI_VALID_TAGS = {'body', 'cell', 'code', 'del', 'div', 'fw', 'head', 'hi', 'item', \
     25                   'lb', 'list', 'p', 'quote', 'row', 'table'}

~\anaconda3\lib\site-packages\pkg_resources\__init__.py in resource_filename(self, package_or_requirement, resource_name)
   1143     def resource_filename(self, package_or_requirement, resource_name):
   1144         """Return a true filesystem path for specified resource"""
-> 1145         return get_provider(package_or_requirement).get_resource_filename(
   1146             self, resource_name
   1147         )

~\anaconda3\lib\site-packages\pkg_resources\__init__.py in get_provider(moduleOrReq)
    360         module = sys.modules[moduleOrReq]
    361     except KeyError:
--> 362         __import__(moduleOrReq)
    363         module = sys.modules[moduleOrReq]
    364     loader = getattr(module, '__loader__', None)

ModuleNotFoundError: No module named 'trafilatura'

Python code used:
from package import trafilatura
trafilatura

Note:
I'm using Python 3.7.6 with pip 20.0.2.
Edit:
A quick fix (for me) is replacing TEI_SCHEMA = pkg_resources.resource_filename('trafilatura', 'data/tei-schema.pickle') in "xml.py" with TEI_SCHMEA = './data/tei-schema.pickle' and
change line 11 in settings.py from from trafilatura import __version__ to from ..trafilatura import __version__

Thank you, it was indeed a problem, I fixed it in 1a5763585211eb021f8194743c9173e50bc23043, could you please confirm by trying the version straight from the repository? (pip install -U git+https://github.com/adbar/trafilatura.git with your --target)

Hey, thanks for the fast response! I just made a new environment and tried to set up everything. However, (even if the package gets downloaded into the "package"-folder) I get following error:
Traceback (most recent call last):
  File "{path}/function.py", line 1, in <module>
    from package import trafilatura
  File "{path}\package\trafilatura\__init__.py", line 16, in <module>
    from .core import extract, process_record
  File "{path}\package\trafilatura\core.py", line 16, in <module>
    from lxml import etree, html
ModuleNotFoundError: No module named 'lxml'

Hi, the changes I introduced created a bug on some platforms but I don't
think it was the issue here.
I guess you face a package managing issue, as lxml should have been
installed and added to your Python path.
It seems that errors linked to target directories with pip are not fully
documented: https://github.com/pypa/pip/issues/8725
Maybe this kind of approach could be useful for you:
https://stackoverflow.com/questions/24174821/how-to-change-default-install-location-for-pip/24175174#24175174

Afaik, lxml needs to be compiled and installed for each machine. Therefore, „portable“ compatibility (install it to a folder and copy it anywhere and run it) can’t be achieved with lxml. In my case I would like to push it on AWS Lambda (localstack; can't install things easily there). I guess it won’t work as long as this project builds on lxml. However, I already found some alternatives for now (I expose your project via a REST-Service on my machine for development purposes). Later (on AWS) I should be able to use EC2 to install all necessary packages such as lxml.

Hi @HaIDsIEx, please refer to this answer and this code snippet, both show how to solve the issue with LXML.


xsdgen issue - root element

From the xsd specification http://www.fdsn.org/xml/station/fdsn-station-1.0.xsd, the "Root element" defined:
<!--  Root element  -->
<xs:element name="FDSNStationXML" type="fsx:RootType"/>
<!--  Type definitions  -->
<xs:complexType name="RootType">
<xs:annotation>
...

Which means the root element struct name should be "FDSNStationXML", e.g.:
type FDSNStationXML struct {
	SchemaVersion float64       `xml:"schemaVersion,attr"`
...

However, currently xsdgen's output is:
type RootType struct {
	SchemaVersion float64       `xml:"schemaVersion,attr"`
...

This problem only happens in the root element.

In this case the xsdgen package is behaving as intended. This is a type declaration, and in the schema, the type name is fsx:RootType. I'm open to discussing whether this is desired behavior. Currently, the xsd package totally ignores top-level elements and attributes, and just searches them for type declaration. Honestly I don't quite know what to do with top-level element declarations in an xml schema.
For this specific instance, you can get what you want like this:
xsdgen -r 'RootType -> FDSNStationXML'


query failed after the ZooKeeper Service is restarted

Bug Report
When the ZooKeeper service connected to the Byconity is stopped and then restarted, the SQL query fails after the ZooKeeper service on the Byconity is normal.
Briefly describe the bug

Component Name
Number of Instances
host

TSO
3
host1,host2,host3

Server
4
host1,host2,host3,host4

default-worker
5
host5,host6,host7,host8,host9

write-worker
3
host10,host11,host12

DaemonManager
1
host1

ResourceManager
1
host1

zookeeper
3
host13,host14,host15

The result you expected
the SQL query can be execute successfully
How to Reproduce
CREATE TABLE  IF NOT EXISTS  test.test_S3_RMT3(
did UInt32,
num1 Int8,
num2 Int16,
num3 Int32,
num4 Int64,
unum1 UInt8,
unum2 UInt16,
unum3 UInt32,
unum4 UInt64,
fnum1 Float32,
fnum2 Float64,
a1 String,
dataCycle Date default today(),
updatetime DateTime default now(),
aa Array(Int16))
ENGINE = CnchMergeTree()
Partition by toYYYYMMDD(dataCycle)
Order by did SETTINGS index_granularity=8192, storage_policy = 'bytes3_test';

insert into test_S3_RMT3 select (
rand() % ((1000000 - 1) + 1)) + 1,
(rand() % ((1000000 - 1) + 1)) + 1,
(rand() % ((1000000 - 1) + 1)) + 1,
(rand() % ((1000000 - 1) + 1)) + 1,
(rand() % ((1000000 - 1) + 1)) + 1,
(rand() % ((1000000 - 1) + 1)) + 1,
(rand() % ((1000000 - 1) + 1)) + 1,
(rand() % ((1000000 - 1) + 1)) + 1,
(rand() % ((1000000 - 1) + 1)) + 1,
(rand() % ((1000000 - 1) + 1)) + 1,
(rand() % ((1000000 - 1) + 1)) + 1,
concat('jwx', toString((rand() % ((1000000 - 1) + 1)) + 1)),
(toDate('2023-05-05') + (rand(3)%((90)))),
now(),['1']  from  numbers(5000);

stop one of the zookeeper service
restart one of the zookeeper service
execute query on byconity
select * from test.test_S3_RMT3 limit 10;
query failed.

Version
0.1.0-GA

What the purpose of the zookeeper here? Would you please describe your deployment topology? and the configuration of your zookeeper?

Hi @blueChild , currently we do not use zookeeper for byconity deployment, we will use internal keeper instead in near future.

ok, we will try it when the ability is available

What the purpose of the zookeeper here? Would you please describe your deployment topology? and the configuration of your zookeeper?

the zookeeper cluster is just for election. if you do not use zookeeper for byconity deployment, we will try the clickhouse-keeper later when the ability is available

Hi @blueChild , we've created an issue here to support HA deployment.


Code stops running when network fails to connect.

I successfully connected to the network repeatedly, and then on a refresh, received two different errors on two different resets - one unknown network error and one authentication error. Each time, it failed to the error and the code stopped running, and it required a hard reset to begin running the code again.
Through discussion with @makermelissa, we both agree it should retry a network connection on a failure. She is currently working on a refactor, and this could be included.

Transferred issue here because that's where the applicable code now resides.

@Kattni, have you noticed this issue since the refactor? I haven't seen it myself.

@Kattni, have you noticed this issue since the refactor? I haven't seen it myself.

@makermelissa I haven't had the opportunity to use any code where this might have happened. If you're confident it's resolved, feel free to close the issue.

@makermelissa I haven't had the opportunity to use any code where this might have happened. If you're confident it's resolved, feel free to close the issue.

Sounds good, we can always reopen.

Sounds good, we can always reopen.


mupc 1.4 missing example folder

I was getting this error when trying to mupc init.
fs.js:549
return binding.open(pathModule._makeLong(path), stringToFlags(flags), mode);
^
Error: ENOENT: no such file or directory, open 'C:\Users\Ronan\AppData\Roaming\npm\node_modules\mupc\example\mupc.json'
I checked the actions.js and it was looking for the example files to copy.
var exampleMupJson = path.resolve(__dirname, '../example/mupc.json');
var exampleSettingsJson = path.resolve(__dirname, '../example/settings.json');
copyFile(exampleMupJson, destMupJson);
copyFile(exampleSettingsJson, destSettingsJson);
I created the folder and mupc.json and settings.json and it then copied the files to my Meteor App successfully.

Thanks.  Submit a PR and I'll merge it.

Hi,  I don't think you need me to do a pull request. It was just a simple case of adding a directory called 'example'  and then placing the sample mupc.json and settings.json files within it. The 'example' directory should then be visible in your Code Tab above with the other lib, bin directories etc.

same problem on the master branch, the instructions to fix this should be in the readme but really this should be fixed in the repo

Sure, I'll make it a point to add the examples when I get the cycles, or I'll accept a PR to add them.  Sorry everyone.

I think really it should be included in the repo so that the files don't need to be created manually

Sure, I'll make it a point to add the examples [to the repo] when I get the cycles, or I'll accept a PR to add them [to the repo]. Sorry everyone.

I went ahead and create a PR to fix this issue

@x13121651 this should be fixed for you as well

thank you for the PR @codeHatcher


Shortcut to focus in and out of terminal panel without closing

I'm using VSCode on MacOS BigSur (Intel and Apple Silicon chip versions).
Today if I'm writing code in a window, and I want to use the built-in terminal, I can use the ctrl+<back tick> or the cmd+j shortcuts to open and focus the terminal panel.
If I press any of these shortcuts again, I restore the focus to my code, but the terminal window closes when I do this.
Also, if I'm in the code panel, and the terminal is open, pressing ctrl+` would focus the terminal, while cmd+j would close the terminal.
I'd like to have a function shortcut that toggle focus to the terminal without ever closing the terminal window/panel.
Steps:

I'm writing code.
Press ctrl+`.
VSCode opens the terminal window, and set my cursor focus there.
I run some config and npm run watch* command in the terminal, I'd like to keep that window open when I'm coding
If I press ctrl+` again, the terminal window closes, my cursor focus goes back to my code.
Today, if I need to keep the terminal open, I need to use my mouse to click the code panel.
Now that I'm coding again with my terminal open, if I press ctrl+` I would return to the terminal, if I press cmd+j I would close the terminal (in my case by accident, ofc).

*. there is a shortcut for npm run watch, I know, it's not exactly my use case, but an example.

From what I've been told, this is as intended. I've found something that helps me out by changing the key bindings.


Disable profile edits for users that don't have custodial permissions

Adds new state to the Redux tree for memberships which already come back from an existing API call which represent various permissions the logged in user has on associated users. Then uses that state to determine whether or not to render the Edit Info link and disables un-editable inputs.
Trello: https://trello.com/c/P1gjaEmv

Ping @cbwebdevelopment for PR review as almost all the changes are in JSX and Redux code.

 [uploader] VCA appears to be able to change non-custodial patient email address, but silently fails


Marketing API

Any plans for contacts / campaigns API?

Yes, we will be releasing a roadmap on this repo soon with further details. Thanks!


arch: X86: update x86 arch porting with new acpica interface

update x86 arch porting with new acpica interface. As part of this the old x86 acpi implementation has been removed and updated test apps and other board configurations with new  acpica interface support.

Will add board/soc update for other x86 platforms once tested

Few CI job are failed and this will resolve once acpica PR get merged.

@andyross could you please have a look into the changes ?

@tbursztyka update as per your review comment and also responded to your few comments

@najumon1980 looks like there's a bunch of build failures:
/__w/zephyr/modules/lib/acpica/source/os_specific/service_layers/oszephyr.c: In function 'AcpiOsReadMemory':
/__w/zephyr/modules/lib/acpica/source/os_specific/service_layers/oszephyr.c:390:31: error: implicit declaration of function 'sys_read64'; did you mean 'sys_read32'? [-Werror=implicit-function-declaration]
  390 |         *((UINT64 *) Value) = sys_read64 (Address);
      |                               ^~~~~~~~~~
      |                               sys_read32
/__w/zephyr/modules/lib/acpica/source/os_specific/service_layers/oszephyr.c: In function 'AcpiOsWriteMemory':
/__w/zephyr/modules/lib/acpica/source/os_specific/service_layers/oszephyr.c:441:9: error: implicit declaration of function 'sys_write64'; did you mean 'sys_write32'? [-Werror=implicit-function-declaration]
  441 |         sys_write64 ((UINT64) Value, Address);
      |         ^~~~~~~~~~~
      |         sys_write32
/__w/zephyr/modules/lib/acpica/source/os_specific/service_layers/oszephyr.c: In function 'AcpiOsGetRootPointer':
/__w/zephyr/modules/lib/acpica/source/os_specific/service_layers/oszephyr.c:778:13: error: cast from pointer to integer of different size [-Werror=pointer-to-int-cast]
  778 |     return ((ACPI_PHYSICAL_ADDRESS) efi_get_acpi_rsdp ());
      |             ^

@najumon1980 looks like there's a bunch of build failures:
/__w/zephyr/modules/lib/acpica/source/os_specific/service_layers/oszephyr.c: In function 'AcpiOsReadMemory':
/__w/zephyr/modules/lib/acpica/source/os_specific/service_layers/oszephyr.c:390:31: error: implicit declaration of function 'sys_read64'; did you mean 'sys_read32'? [-Werror=implicit-function-declaration]
  390 |         *((UINT64 *) Value) = sys_read64 (Address);
      |                               ^~~~~~~~~~
      |                               sys_read32
/__w/zephyr/modules/lib/acpica/source/os_specific/service_layers/oszephyr.c: In function 'AcpiOsWriteMemory':
/__w/zephyr/modules/lib/acpica/source/os_specific/service_layers/oszephyr.c:441:9: error: implicit declaration of function 'sys_write64'; did you mean 'sys_write32'? [-Werror=implicit-function-declaration]
  441 |         sys_write64 ((UINT64) Value, Address);
      |         ^~~~~~~~~~~
      |         sys_write32
/__w/zephyr/modules/lib/acpica/source/os_specific/service_layers/oszephyr.c: In function 'AcpiOsGetRootPointer':
/__w/zephyr/modules/lib/acpica/source/os_specific/service_layers/oszephyr.c:778:13: error: cast from pointer to integer of different size [-Werror=pointer-to-int-cast]
  778 |     return ((ACPI_PHYSICAL_ADDRESS) efi_get_acpi_rsdp ());
      |             ^

This need following acpica PR need to merge and update the west.yml accordingly.
https://github.com/acpica/acpica/pull/882

This need following acpica PR need to merge and update the west.yml accordingly. acpica/acpica#882

Please submit a PR to our ACPICA mirror (which will not be merged - we’ll sync the tree when the ACPICA upstream PR is merged), and follow up these instructions: https://docs.zephyrproject.org/latest/develop/modules.html#process-for-submitting-changes-to-existing-modules
That way we can verify that CI passes with the combination of the ACPICA PR and this PR.


Typescript definition file

A typescript definition currently doesn't exist for this at the moment. I'm working on creating it, and I will eventually submit it to DefinitelyTyped so it can be used for those who are working with this Hub factory in a Typescript project.
Here's a link to my fork of DefinitelyTyped:
https://github.com/AdamSantaniello/DefinitelyTyped

Cool! I haven't used TypeScript myself, but I'm sure there will be lots of people grateful for this.

Is there any examples on how to instantiate a hub using the Typescript definition? I'm still learning how to use TS effectively, so I don't understand how the HubFactory interface relates to the original javascript.

Maybe you can try the following:
module signalR.Test {
	declare var Hub: ngSignalr.HubFactory;

	export class TestService {
		private signalrHub: ngSignalr.Hub = null;

		constructor() {
			this.signalrHub = new Hub('testHub', { rootPath: '', listeners: {}, methods: [] });
		}
	}
}

I haven't tried this myself yet, but it gives the following output in plain javascript:
var signalR;
(function (signalR) {
    var Test;
    (function (Test) {
        var TestService = (function () {
            function TestService() {
                this.signalrHub = null;
                this.signalrHub = new Hub('testHub', { rootPath: '', listeners: {}, methods: [] });
            }
            return TestService;
        })();
        Test.TestService = TestService;
    })(Test = signalR.Test || (signalR.Test = {}));
})(signalR || (signalR = {}));

@DeBiese Thank you. I was making a silly oversight and didn't realize the Hub object you instantiate was in fact the ngSignalr.HubFactory type. Once I realized that it made sense to me.

Sorry for my delayed response, but I did want to add a little bit of clarification. ngSignalr.HubFactory is intended to be a strongly typed way of representing the constructor function that gets injected into your angular module. The hub object that gets instantiated is still of type Hub.
I think the HubFactory was probably the underlying source of confusion. I understand it may be a bit of a stretch to call it a factory at all. At the time I honestly couldn't really think of a better name. IHub didn't make sense and HubConstructor also didn't quite seem right either.

Hey Adam, do you think it would be helpful to have an example of how to instantiate with typescript? I haven't used typescript myself, so I'm not sure if this is necessary or helpful. We could make a Wiki page for it.

I think that's a great idea. I'd be more than happy to write one up in the near future.

Great, thanks Adam.

@AdamSantaniello I don't know if this is considered orthogonal or not, but I still get confused on how to leverage d.ts files with libraries I'm including with webpack. The issue I run into is I want to write typescript code like this:
import * as ngSignalr from "angular-signalr-hub";

...but what I run into is the modules declared in the definition file usually use some other name that doesn't match the npm package name. For example, the d.ts file declares a module like so:
declare module ngSignalr {
    interface HubFactory {
        /**
         * Creates a new Hub connection
         */
        new(hubName: string, options: HubOptions) : Hub
    }
...

So in a case like this library I end up patching the typescript definition files so the module they export matches the npm package name so both typescript and webpack are happy.
If this is something that could be included in the example, especially if I'm doing something really stupid here, it might be helpful for others as well.

Hi Guys, if someone still need it, here   I created a simple ASP.NET solution which contains a typescript implementation of the SignalR wrapper. The code you would be interested it is inside real time-app/collaborationService.ts
I hope it helps.
Cheers,
Rocco

And just for anyone else coming here wondering about this topic... here is how you instantiate a hub within typescript.


Find public contracts with prices and people responsible for it

Products and services acquired by the government must use a public contract.
This can be searched here.
We can scrape values, agencys and people involved with some contracts that may look suspicious.
Example: contracts that don't need biddings (pt-br: "licitações");

Hi @rroriz — that looks interesting.
Nowadays our main dataset is from CEAP — which does not require public bids or contracts.
Just to clarify: are you proposing another branch of analysis or to draw some hypothesis linking these public bids & contracts to CEAP data?

I was proposing a new branch of analysis.
But after reading more about CEAP, there is something that can be linked with biddings: some expenses could be the subject of a bidding .
I'm closing this issue until I have something more concrete to do with biddings.


Web API Implementation for OPERANDI

Describe the feature you'd like
A first implementation of the Web API according to the spec. The purpose of the implementation is to have some feedbacks about the spec and adapt it if necessary. Therefore, the implementation doesn't have to strictly stick to the current spec.
User story

As a developer of the Web API spec, I need the spec to be implemented once in order to know:

if it's appropriate for other Implementation Projects to implement it as well.
what needs to be changed.

As already discussed, the processing part in the spec must not be implemented in this Epic since it do not belong together.

A while ago I started with the webAPI here: https://github.com/joschrew/ocrd-webapi-test, and then stopped, because we separated the work and I focused (maybe accidentally) on other things.
I want to continue there and already briefly started but have to find time to really start over in the next sprint


Security Issue: Please allow the very simple delay of the installation of updates by n days

Type: Bug because of a Security Issue
VS code and plugin updates are in a security-critical context:

on the one hand, you always want / need to install updates as fast as possible in order to close security issues.

on the other hand, there are situations in every project where you must not install updates of the tools used at any price. Because: It could have fatal consequences if a used function suddenly stops working, works different or engineers suddenly invest hours to get the IDE running again with the pipeline.

VS Code currently only offers to install updates “later”. EVERY TIME you restart the IDE, you are asked again to install the updates, because apparently “later” has already been reached.
If you have answered the question about the installation 10 times with “later”, this has the consequence that developers deactivate the automatic updates.
This makes the update logic of VS Code a security risk.
Please allow us to define in how many days we should be asked again when updating.
This solves both problems mentioned at the beginning and provides a professional solution.
Thanks a lot, kind regards,
Thomas
VS Code version: Code 1.93.1 (38c31bc77e0dd6ae88a4e9cc93428cc27a56ba40, 2024-09-11T17:20:05.685Z)
OS version: Windows_NT x64 10.0.19045
Modes:

System Info

Item
Value

CPUs
AMD Ryzen 9 5900X 12-Core Processor             (24 x 3693)

GPU Status
2d_canvas: enabledcanvas_oop_rasterization: enabled_ondirect_rendering_display_compositor: disabled_off_okgpu_compositing: enabledmultiple_raster_threads: enabled_onopengl: enabled_onrasterization: enabledraw_draw: disabled_off_okskia_graphite: disabled_offvideo_decode: enabledvideo_encode: enabledvulkan: disabled_offwebgl: enabledwebgl2: enabledwebgpu: enabledwebnn: disabled_off

Load (avg)
undefined

Memory (System)
127.91GB (35.72GB free)

Process Argv
--crash-reporter-id 856f7e37-1fcf-4a79-b795-cf947af05bd0

Screen Reader
no

VM
0%

Thank you for your answer.
Yes, the issue still exists - and it's pretty clear because the concept which causes this issue has not been changed.
Please note that fixing this security (and usability) issue should be pretty simple because VS Vode should already have all needed functions built in.
Thanks a lot, kind regards,
Thomas


No longer applying the sourcemap

Leaving the sourcemap application and generation to node-sass
Fixes #639

This is what allows gulp to combine sourcemaps through multiple plugins i.e. minifiers, prefixes. It can't be removed.


[Docs]: Add docs for appsmith.theme object

Is there an existing issue for this?

[X] I have searched the existing issues

Documentation Link
No response
Discord/slack/intercom Link
No response
Describe the problem and improvement.
Adding docs for appsmith.theme object

Appsmith theme object
Appsmith theme object contains the details of the theme properties applied to the application. We can use the properties from this object to

Set certain properties in widgets to be aligned with theme. For example, setting the font family of text widget to refer to the theme’s font family.
Write custom logic like to suggest a font color based on the theme’s primary color.

Appsmith theme has the following properties which are accessible for your reference

colors:

primaryColor - It refers to the background color set in the theme section of app settings
backgroundColor - It refers to the background color set in the theme section of app settings

borderRadius

appBorderRadius - It refers to the border radius set in the theme section of app settings

boxShadow

appBoxShadow - It refers to the border radius set in the theme section of app settings

fontFamily

appFont - It refers to the font family set for the app in the theme section for app settings


Preserve alternative names of authors from TEI data

Currently, a contributor can only have one name, so we have no way of representing any alternative names of an individual.
The Arabic TEI data is a good example of why this is needed - names are recorded in Arabic and in a Latin alternative (e.g. Avicenna, not necessarily a straight transcription).
For discovery, we should preserve those names.
As an interim measure before the Concepts work is complete enough to have an impact, we could record the multiple names as separate Contributors (?).

we could record the multiple names as separate Contributors (?).

I would lean towards multiple names on the same Contributor, if we can find a way to do that – so if we have a work with multiple contributors, each with multiple names, we can easily match names.


Allow null content type for multipart form field

Other clients like curl + guzzle don't send a content type for simple multipart form fields. I don't see a reason why a content type is enforced.

If there's no content type, text/plain is assumed by the other party anyway.

Sure, but why transfer if it is default? It's nits, I just stumbled across it during tests + comparison with other popular clients.

Is there any disadvantage in being explicit? If you already use a bloated multipart form field instead of urlencoded data, it doesn't really harm to just send the default content-type.

As already mentioned, these are nits. You can close the PR if you don't like it.

Any HTTP/1.1 message containing an entity-body SHOULD include a
Content-Type header field defining the media type of that body. If
and only if the media type is not given by a Content-Type field, the
recipient MAY attempt to guess the media type via inspection of its
content and/or the name extension(s) of the URI used to identify the
resource. If the media type remains unknown, the recipient SHOULD
treat it as type "application/octet-stream".

This means:

A Content-Type header is not required.
When missing it does not imply text/plain. Our default to text/plain is therefore an error. When missing the content type should be omitted.


Cherrypick: Use cluster name as SNI when using Istio mTLS (#8778)

From https://github.com/istio/istio/pull/8778
Signed-off-by: Shriram Rajagopalan<EMAIL_ADDRESS>

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

Needs approval from an approver in each of these files:

pilot/pkg/networking/OWNERS

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

/lgtm

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

Needs approval from an approver in each of these files:

pilot/pkg/networking/OWNERS

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


Current status of this project?

Heyo!
I landed by "mistake" on the https://macos-defaults.com/ website, and I loved it.
I was specifically looking for screenshot commands.
Was this ever left? Is there any motivation missing behind it?

Hi @i5o,
The project is still ongoing, I just need to find some time to work on it 🙂

How can we help you @yannbertrand?

How can we help you @yannbertrand?

Actually not much. I hope I can find some time this summer to clean the project. But thanks for asking, it's greatly appreciated!


Change audio source to GPIO input?

I am using the Raspberry Pi setup. Do you know how to change the audio source from USB to a GPIO pin?
Currently, I am capturing audio from an analog microphone attached to an Arduino, then sending that into a GPIO pin. Alternatively, I could use an ADC to capture the microphone and attach that to the Raspberry Pi.
In any case, I don't know how to setup my input pin as a PCM or ctl device so that it can be used with PyAudio. My alternative solution to this would be to write a little streaming interface with the RPI.GPIO library

I'm not 100% sure that is possible, but I'm not super knowledgeable on the pi. Maybe someone might be able to chime in on this?


[Snorkell.ai] Please review the generated documentation for your repository

Title: Update Documentation Across Codebase
Description:
This PR focuses exclusively on updating and refining the documentation throughout the codebase. There are no functional changes to the code itself.
Changes:
🐍  src/crewai/agent.py
🐍  src/crewai/agents/cache/cache_handler.py
🐍  src/crewai/agents/exceptions.py
🐍  src/crewai/agents/executor.py
🐍  src/crewai/agents/output_parser.py
🐍  src/crewai/agents/tools_handler.py
🐍  src/crewai/crew.py
🐍  src/crewai/task.py
🐍  src/crewai/tools/agent_tools.py
🐍  src/crewai/tools/cache_tools.py
🐍  src/crewai/utilities/i18n.py
🐍  src/crewai/utilities/logger.py
🐍  src/crewai/utilities/prompts.py
🐍  src/crewai/utilities/rpm_controller.py
🙏 Request:
Please review the changes to ensure that the documentation is clear, accurate, and adheres to our project's standards.
Any feedback regarding areas that might still need clarification or additional details would be highly appreciated.
✍ Note:
Given that these changes are purely documentation-related, everything should remain unaffected.
This Documentation has been generated using Snorkell.ai
Reference Links:
Github App: https://github.com/apps/snorkell-ai
Website: https://www.snorkell.ai/
Email<EMAIL_ADDRESS>

This documentation is beyond worthless, things like "SomeException: An explanation of when this exception is raised." Snorkelai is a complete scam.

I kind like some pieces but not all of it, also formatted a little weird, closing for now taking into account a lot these calsses changes since then as well


Txn: move some commands functionality around.

What have you changed?
Following up from earlier refactoring, this PR moves some command functionality around to make things a bit tidier. This is mostly just to move things closer to where there are used/belong. The most interesting change is moving the decoding of protobufs into Command objects from server/service/kv to storage/txn/commands, the motivation is that the kv service should be fairly minimal and not have knowledge of storage internals.
In addition I remove one unused command.
What is the type of the changes?
Engineering (engineering change which doesn't change any feature or fix any issue)
PTAL @MyonKeminta @youjiali1995

I've rebased and addressed comments. PTAL @zhangjinpeng1987 @MyonKeminta
Note that I've added another commit which I was going to post separately, since it resolves some of the review comments.

rest LGTM

rest LGTM

sorry, I don't see a comment to address

sorry, I don't see a comment to address

@nrc Please resolve the conflicts, and i will give my LGTM.

rebased and conflicts fixed. PTAL @zhangjinpeng1987 @MyonKeminta

Build failed.

There is an error in 'tests/failpoints/cases/test_storage.rs:42:10'.
reset LGTM.


Use JSON-RPC standard for responses

Closes #495.

I inverted the if logic in the applicationRelay() at v1.controller.ts — which is something I should have done in a separate PR. I can revert that change for a better review process.

another point is that I saw you used the same -32000 for all the error responses, check the JSON-PRC Specification on section 5.1 for a more complete list of error objects to confirm is some of the ones set here can be more semantically correct

all right, LGTM!

There's one test failing called sends successful relay response as string — I think this might as well be removed, or asserted on fail because we don't really want to return a string to the user at any time. The payload from calling a node should always be a JSON.


Comma character "," triggers the jump to radix even though 'radixPoint': '.' is set to period "." symbol

My users enter their currency in by typing the comma.
For example they type in order: 1,345  For 'one thousand three hundred forty-five'.
As soon as they type the comma, the cursor jumps to the radix decimal, so they end up typing 1.34 for their currency price.
If I manually set the 'radixPoint' to a '.' (period), then a comma character should not trigger/jump the cursor to the radix.
How can I 'turn off' the comma trigger altogether or a workaround for this issue?

@wilsmex ,
Have a look at https://github.com/RobinHerbots/Inputmask/blob/5.x/README_numeric.md#substituteradixpoint


ampmake build fails to generate AMP CLI for Windows

Trying to run ampmake build on master branch fails to generate AMP CLI for Windows
$ ampmake build
.......
cross-compiling amp cli for supported targets
/go/src/github.com/appcelerator/amp/bin/alpine/amd64/amp
/go/src/github.com/appcelerator/amp/bin/darwin/amd64/amp
/go/src/github.com/appcelerator/amp/bin/linux/amd64/amp
/go/src/github.com/appcelerator/amp/bin/windows/amd64/amp.exe
# github.com/appcelerator/amp/vendor/github.com/appcelerator/docker/vendor/github.com/docker/docker/pkg/system
vendor/github.com/appcelerator/docker/vendor/github.com/docker/docker/pkg/system/meminfo_windows.go:10: undefined: windows.NewLazySystemDLL
Makefile:151: recipe for target 'buildall-cli' failed
make: *** [buildall-cli] Error 2

Nope. Reproduced this issue on Windows machine as well.

I just built a Windows .exe yesterday. What changed?
$ ls -la bin/windows/amd64/
total 79328
-rwxr-xr-x  1 tony  staff  38425600 Jul 31 15:03 amp.exe*

Please paste the results of docker info for the systems you tested here.

I can reproduce the issue on Docker for Mac Version: 17.06.1-ce-rc1

Yes, it failed for me just now as well. The last successful build was yesterday afternoon.

This seems to indicate conflicts in vendor can cause the issue -- and be resolved:  https://github.com/containous/traefik/issues/1422
The only update I did to vendor yesterday was to add vendor/github.com/appcelerator/docker (with its own vendor directory: vendor/github.com/appcelerator/docker/vendor).

@neha-viswanathan Thanks. Forgot to also ask for docker version to get the go version, but I think we have enough information now to identify the issue and hopefully resolve it.

Good catch indeed @neha-viswanathan :+1:

Docker Version on Mac
$ docker version
Client:
 Version:      17.07.0-ce-rc1
 API version:  1.31
 Go version:   go1.8.3
 Git commit:   8c4be39
 Built:        Wed Jul 26 05:20:09 2017
 OS/Arch:      darwin/amd64

Server:
 Version:      17.07.0-ce-rc1
 API version:  1.31 (minimum version 1.12)
 Go version:   go1.8.3
 Git commit:   8c4be39
 Built:        Wed Jul 26 05:25:01 2017
 OS/Arch:      linux/amd64
 Experimental: true


connect to database with unintegrated features

There were minor conflicts on routing from App.jsx and my test feature on Home.jsx


Fungible inscriptions

Should we support fungible inscriptions? We could make an inscription apply to the first ordinal of all outputs. This would mean, however, that the size of the set would be limited to the number of outputs in a transaction.
I'm leaning towards reusing prime ordinals, and a fungible collection is just all ordinals inscribed by a prime ordinal. We can make this easy by:

An inscription can be empty. No content, no mime type, etc. For a fungible collection, inscribe the prime ordinal with the content, and then make a bunch of empty inscriptions with it.
An inscription applies to the first ordinal of each output, except the prime ordinal, if any.

We could add a metadata field to an inscription that said "the next N sats after this inscription" are fungible tokens. Support in ord would be hard, because special tracking for all those sat ranges would be difficult, but it's such a simple protocol-level extension that I would be open to adding it, if only so that others could experiment with it in other repos.

I'm inclined not to do this. Taproot Assets and RGB are much better ways to do fungible assets.


feat(components): ability to hide directory path

It would be nice to be able to hide the directory path. I have that context in my status line so would be nice to reduce duplicate information.
show_dirname = true (default)

show_dirname = false

Looks like you're autogen-ing the vimdocs, so I didn't update that by hand. Awesome plugin!

Thanks for the PR!
Related: #8

Hi,
Is show_dirname = false supposed to be working already? For me it is not hiding the directory path...
The solution in https://github.com/utilyre/barbecue.nvim/issues/8#issuecomment-1265419804 does work though.
Thanks in advance!

Hello, double check these two:

show_dirname is at the root of options table you pass to setup function.
you have the latest commit installed.

My mistake, the latest commit was not installed. I had used your installation instructions for lazy.nvim and got some older commit installed ( due to version = "*"). Thanks for clarifying. It is working now.


Acquisition tab does not show data, keeps loading

Hello, we have the module installed but in the acquisition tab it does not show data, it remains as if it were loading, can you tell us what is happening?

Hello @Artdinamica1 I am sorry there is a confusion 😅 this repository is not the ps_metrics repository and we are not the ps_metrics team.
This repository is a Symfony 5 application that collects and display data for project maintainer team.
In order to get help with ps_metrics team, you need to reach out to PrestaShop company support https://prestashop.com/contact/
I close this issue 😉 as it's not the right place


linux-27 lost external connection phone device.

Type of Request
bug
Infrastructure Environment
LUCI, Github, Cocoon scheduler, Autosubmit, etc...
What is happening?
linux-27 lost external connection phone device.

Steps to reproduce
Step 1:
Step 2:
..
Step n:
Expected results
I expect to see X when Y is finished.

Rebooting phone device to restore connection.


Add support for linking using gold

gold is an ELF linker which can be faster than the traditional ld.

I added in my local repo an option to use LLD when using clang (if you don't tell it, it defaults to the default linker and that's often the one from GCC). I did basically what @Web-eWorks said here but for LLD instead of gold and added the check only when "using_llvm" is set too. In my case I set "use_lld" to false per default as not everyone who uses Clang also uses LLD (support still varies on the 3 big desktop platforms).
I did it only for Linux as I can't test for other platforms right now, but it would be basically the same for each platform two lines or so. Could be done the same for using gold.
Here my repo:
https://github.com/Valentactive/godot/tree/add-lld-linker-option

lld can't be used together with gcc's LTO. IT makes little sense to make this a default in any way.

lld can't be used together with gcc's LTO. IT makes little sense to make this a default in any way.

I guess it would still be useful to support it for development purposes (where you don't use LTO), so one can iterate faster when working on Godot.


Assert all the expected DNSNames are part of the HTTP01 challenge

This allows us to include additional domains as part of the
challenge (this is needed for long URL domains workaround where we add a shorter commonName)
Unblocks: https://github.com/knative-sandbox/net-certmanager/pull/478

/cherr-pick release-1.9

/lgtm

/cherr-pick release-1.9

/cherry-pick release-1.9

/unhold

/cherry-pick release-1.8


Re-invoke ListView.itemBuilder on reassemble

Related: https://github.com/tvolkert/chicago/issues/6

@Hixie


Add a GitHub Action to auto deploy the site

We can use this GitHub action to auto deploy the site to the master branch whenever someone pushes/merges a commit to the develop branch.

The action needs to run npm run deploy on pushes and pull requests to the develop branch. The current setup using the prebuilt GitHub action won't work for us.

Fixed in 248eb3d83e33aa28f65544f11489658c4ed770d3


kernel warnings about invalid overlay mounts

When running our integ tests, checking the kernel logs shows periodic warnings about invalid overlay mounts:
overlayfs: lowerdir is in-use as upperdir/workdir of another mount, accessing files from both mounts will result in undefined behavior.

The kernel logs can usually be checked by running sudo dmesg.
I see this locally from time to time when running the full integ tests with dagger call --source=.:default --docker-cfg=file:/home/sipsma/.docker/config.json test all --parallel=8 --timeout=40m

Not every time, ~ every couple of runs
local uname -a output: Linux dagger_dev 6.9.7-arm64 #1 SMP Debian 6.9.7-1 (2024-06-27) aarch64 GNU/Linux

If you open a shell in our CI infra to a dagger engine pod and run dmesg you will typically see a lot of these messages (seemingly many more than I get locally, at least by eye).
This is possibly related to various flakes (e.g. https://github.com/dagger/dagger/issues/8031), but that's purely hypothetical and based on the idea that "undefined filesystem behavior" could result in files getting mixed up between different mounts and/or having unexpected contents.
However, the fact that these messages show up means there is without a doubt a bug somewhere in dagger/buildkit/containerd/etc. and that bug may be resulting in files/dirs getting corrupted in unpredictable ways. So opening this up as a separate issue.

This is exceptionally tricky to track down because the kernel does not give you any details on the actual invalid mount, just that some process made some mount that had this improper configuration.
strace is not really tenable here because even with the --seccomp-bpf option the overhead is way too high.
The best option I've found so far is this ebpf based tool that traces every mount syscall made on the machine: https://github.com/iovisor/bcc/blob/master/tools/mountsnoop.py
There's some limitations though:

The mount options get cut off after 412 characters due to some ebpf stack size limitation
You need to have the kernel headers for the machine installed where you run mountsnoop, which is extremely difficult in our CI since it would require getting the amazon linux kernel headers inside the alpine-based pod container

This is certainly doable one way or another with enough effort if it comes down to it though

Given the above, I have so far only managed to catch this one time when running on my local machine:

I start mountsnoop in one terminal as root
In another I run as root dmesg -C; dmesg -w | grep -q overlay && kill <mountsnoop pid>

The last chunk of mountsnoop logs I got are here: https://gist.github.com/sipsma/c0b92759b52f6fec0532ca122e7d40f8

These aren't logs from the entire run since I didn't redirect them to a file 😞, so I just got a chunk from the terminal output... Trying to repro again where all the logs are saved.

Unfortunately, I did not find any obvious instances of a dir being used as a lowerdir while still mounted as an upperdir elsewhere. However:

As mentioned above, these are not the entire logs from the whole test run yet
This is error prone to check manually, may be worth a script
There's various complicated scenarios that are very hard to track down just by manual inspection of the logs like

bind mounting an overlay mount (or subtree of one) elsewhere, which means it needs to be unmounted from multiple places before being "gone"
use of MNT_DETACH resulting in the unmount not synchronously happening and thus allowing the mount to continue to exist for some undefined time period after the unmount syscall occurs

Another option for debugging would be to just change the code to log every mount (which would avoid some of the limitations of mountsnoop), though there are many places this happens in buildkit/containerd/runc and quite a bit of layers of abstractions over the actual mount syscall, so this could easily miss places if not extremely thorough.

I'm sure there's plenty of crazy options in terms of forking go itself to log these and then rebuilding everything with that. But very much a last resort.

cc @jedevc @gerhard

Matias let me know there's a sysadmin pod on each node that has all the privileges needed + host mounted at /host (to get kernel headers), so I can actually run mountsnoop there in CI too.
After opening shell in the pod:

apt update && apt install -y bpfcc-tools
mountsnoop

I'm surprised I didn't have to point to the kernel headers under /host, but it seems to work. Either it found them under /host somehow or there are kernel headers in the pod container that happen to work (not the case in alpine based pod containers) 🤷♂

I had a random vague memory of this happening in the past and finally managed to dig it up in our GH issue history: https://github.com/dagger/dagger/pull/3785#discussion_r1019701477
I saw these messages back in 2022 too, so whatever this is, it potentially has been an issue for a very long time. That makes me suspect slightly more that the impact is closer to something like 3. in my list above since I'd have expected us or users to have experienced something more obviously broken for a very long time. Still unknown overall though, just felt worth noting.


gen-nncp.sh - add IPv6 support

When NETWORK_ISOLATION_IPV6 (default: false) is true, configure the NNCP for IPv6. Likewize NETWORK_ISOLATION_IPV4 (default: true) controls if IPv4 is configured.
Enabling both IPv6 and IPv4 should work for dual stack.

recheck

recheck

recheck


Site failes to load with Firefox when cookies are disabled

Firefox has a long standing bug that it will return a security exception on localStorage.getItem() when the user has disabled cookies. The frontend hits this bug and fails to load completely. The error message is:
SecurityError: The operation is insecure. bundle.js (line 11)

Hm, do you have any idea how we could circumvent this? Looking at the thread there doesn't seem to be a clear work-a-round emerging.

I've used [modernizr][https://modernizr.com] to check if local storage is available. It also detects this kind of problem.

Had the same issue in Mozilla/Seamonkey. Cookie settings were on default which is here "always ask". But OSRM did not even ask, so the page did not render with the OP's error message in the log.
I now allowed session cookies manually, after which it works.


Protocol message to notify streaming status.

With StreamAllocator, SFU could pause/resume streams
based on bandwidth availability. Adding an update
message to notify clients about the state of streaming.

Reviewers: please let me know if you can think of better names for the fields.


fix: add preconditions to check for puppeteer page availability

Added some preconditions to check for puppeteer page is open or not.
resolves #1767

This PR depends on #1760.


Final proofread of docs

Documentation Scope
Final proofread of the following docs, looking for outdated and missing information:

[x] main readme English review. required by @Startrekzky
[x]  main readme Chinese @Startrekzky
[x] wikis. review required by @Startrekzky
[x] Github readme. updated by @mindlesscloud, review required by @e2corporation
[x] Jira readme.  updated by @klesh, review required by @e2corporation
[x] Gitlab readme.  updated by @mindlesscloud, review required by @kevin-kline
[x] Jenkins readme @klesh has updated,  review required by @kevin-kline

solved


Add types to server-middleware/healthcheck.js

Description

Add type checking to healthcheck.js. This module has zero dependencies. Make sure to add it to tsconfig.json's include list.
Note: The exported function should be typed using @nuxt/types's ServerMiddleware type.
Additional context

Part of an ongoing effort to add type checking to parts of the project that can be type checked.
Implementation

[ ] 🙋 I would be interested in implementing this feature.

@obulat  I would be interested in implementing this feature.

Great, @kickeyhickey :slightly_smiling_face: The issue is assigned to you. Let us know if you run into any road blocks :rocket:


Provide prebuilt binaries

The Real-ESRGAN project provides prebuilt binaries which makes it so much easier to get started. It would be amazing if that could be provided here too.
The reality is that it's difficult to get the correct environment set up. Most threads on the internet (for example, Reddit) about Stable Diffusion or any other ML thing end up being a 100+ thread about various Python dependency issues.

I second this, if it is possible under the source licenses. I would guess it will depend on getting the main branch stable for a new release first?

Yes please, was not able to run Real-ESRGAN and lost many hours on it

@nihui makes them, ask her.
Unless someone else here is familiar with NCNN.

Just had a look at the NCNN website. It isn't clear to me that we can use it unless we completely rewrite the ML code to use the NCNN framework. Correct me if I'm wrong.
An alternative would be Docker, which I have more experience (and comfort) with. Would this be helpful? It might be one large image.


Change from JS to Typescript

If anyone wants to try out the autotile.kwinscript and provide any input on whether it works, that would be appreciated

If no one has input, this is being merged tomorrow.


TypeError: Cannot read property 'remove' of undefined

Hi There
I got the following js error when closing Dialog(Modal)
TypeError: Cannot read property 'remove' of undefined
at angular-daterangepicker.min.js?compile=false:1
Just wondering if you know how to get around this
Cheers
Wallace

Seems like this was fixed in #35, but there is no tag for this fix so bower doesn't pull in latest. Could we get a new release?

Should be released already.


HKG: add electric motor ECU query

Hyundai EVs do not appear to have a dedicated engine or transmission ECU (no response whatsoever when querying many data identifiers).
Puts this ECU in data-collection mode. This is also the only ecu that returns the vin, so it likely replaces function of the engine and transmission, will check to make sure

Checked Hyundai Tech Info, Ioniq 5 does not have a transmission module (TCM) listed, while 2020 Sonata does.
Of course, no engine ECU at 0x7e0 either

According to this article from Bosch about VCUs (which is what the parts sites report for the hardware id for the ECU on 0x7e2 for our EV6), the VCU assumes control of the inverter, battery management, engine and transmission control on fully EV cars: https://www.bosch-mobility-solutions.com/en/solutions/vehicle-computer/vehicle-control-unit/

Not sure if we need this just yet


Compressing response breaking error page

Environment
Working directory: /home/projects/nuxt-starter-xkjsve                                                                                1:20:34 PM
Nuxt project info:                                                                                1:20:34 PM

Operating System: Linux
Node Version:     v18.18.0
Nuxt Version:     3.11.2
CLI Version:      3.11.1
Nitro Version:    2.9.6
Package Manager<EMAIL_ADDRESS>Builder:          -
User Config:      devtools
Runtime Modules:  -
Build Modules:    -

👉 Report an issue: https://github.com/nuxt/nuxt/issues/new                                                                                1:20:34 PM
👉 Suggest an improvement: https://github.com/nuxt/nuxt/discussions/new
👉 Read documentation: https://nuxt.com
Reproduction
https://stackblitz.com/edit/nuxt-starter-xkjsve
Describe the bug
I use code below to compress response before output using brotlli/gzip/deflate method, it's working good when response 200, but for 404/500 error pages this code doesn't work in async page:
import { Readable, pipeline } from 'node:stream';
import zlib from 'node:zlib';

export default defineNitroPlugin((nitroApp) => {
  nitroApp.hooks.hook('beforeResponse', async (event, response) => {
    compressResponse(event, response);
  });
});

async function compressResponse(event: any, response: { body?: unknown }) {
  if (typeof response.body !== 'string') {
    return;
  }

  // using deflate as fallback here bcz stackblitz doesn't include it in req headers
  // You can test locally without setting fallback
  const acceptableEncoding =
    getRequestHeader(event, 'accept-encoding') ?? 'deflate';

  if (
    !acceptableEncoding ||
    !/\b(deflate|gzip|br)\b/.test(acceptableEncoding)
  ) {
    return;
  }

  const originalBody = response.body;
  const stream = Readable.from(response.body);

  if (/\bbr\b/.test(acceptableEncoding)) {
    setResponseHeader(event, 'Content-Encoding', 'br');
    response.body = pipeline(
      stream,
      zlib.createBrotliCompress(),
      event.node.res,
      () => (response.body = originalBody)
    );
  } else if (/\bgzip\b/.test(acceptableEncoding)) {
    setResponseHeader(event, 'Content-Encoding', 'gzip');
    response.body = pipeline(
      stream,
      zlib.createGzip(),
      event.node.res,
      () => (response.body = originalBody)
    );
  } else {
    setResponseHeader(event, 'Content-Encoding', 'deflate');
    response.body = pipeline(
      stream,
      zlib.createDeflate(),
      event.node.res,
      () => (response.body = originalBody)
    );
  }
}

Additional context
No response
Logs
No response

Since /__nuxt_error is broken trying to compress, adding the following to compressResponse may work.
    if (getRequestURL(event).pathname.startsWith('/__nuxt')) {
      return
    }

However, it may be recommended to do this with a proxy rather than compressing with the nitro plugin.
https://github.com/unjs/nitro/issues/1007#issuecomment-1453362030


Need a configuration pages

to select import export some databases and "preferences".
In a first time i need it to focus on "homedata" /server/controllers/homedata  see /server/model/homedata

with commit 25e56c866 i did push a quick modal template in CSS, to be able to update homedata value. i need to fix the inputs with ref to the database.
next, i will think if i'm going with some kind of react-modal like:
https://github.com/cupcake/react-modal
https://github.com/marcio/react-skylight
https://github.com/yuanyan/boron
https://github.com/tajo/react-portal
https://github.com/tryolabs/react-examples/tree/master/modal

check the inputs to get along with React
input {
id: "xid",
type: 'x,
defaultValue: "xValue"}


Random crashes on Windows in Github actions

I tend to run a very large amount of builds in Github Actions and I have seen on several occasions what appears to be a random crash.
It is quite rare, I have seen it only in Github Actions, always on Windows, always during the configure phase.
Here is one example:
https://github.com/mmomtchev/magickwand.js/actions/runs/11283317315/job/31382560438
I haven't been able to link this to any similar issues on the official CMake tracker.
Obviously, finding it will be very difficult - especially if it happens only in Github Actions. Maybe it is an interaction between the compiler runtime and the Github Actions Windows DLLs?
I am creating this issue to track the occurrences.

Maybe it is an interaction between the compiler runtime and the Github Actions Windows DLLs?

Maybe, I don't know. My first thought was for some resources that are exhausted, from memory to some internal handles or similar.
Could you find a trick to save the logs as artefacts, for later inspection?


error-logs: Zip files into Downloads folder when user clicks on Show App Logs

This PR adds function that when a user clicks of "Show App Logs" from the menu, open the Downloads folder with a zip file containing all the Logs and User Configs. This will help the user to easily share logs with us.

Thanks, @abhigyank. Let's work on implementing the sentry now.


Suggestion: Add schemas for resource packs

Fairly self-explanatory. Most other resource pack schemas on the internet are pretty old. It would be convenient to have them in this repository, because it is well-maintained and up-to-date.

Whoops. Internet bugged out and duped this.


Export appveyor.yaml

It would be nice to have the appveyor.yaml in the repo so we can contribute to it. You just need to go to your appveyor project settings and there is a "export yaml" setting there.

Closed in https://github.com/RichieSams/FiberTaskingLib/pull/85


Add grep option

This feature is under development. It is not final result.

Coverage remained the same at 96.829% when pulling 4505244f4ec9a94eaca9e8745771ceb22eb62962 on b5y:feature/support_grep into 7e9b1b583b89be447e5ae92654c2fea79c626b3d on Yelp:master.

Coverage remained the same at 96.829% when pulling 4505244f4ec9a94eaca9e8745771ceb22eb62962 on b5y:feature/support_grep into 7e9b1b583b89be447e5ae92654c2fea79c626b3d on Yelp:master.

Coverage remained the same at 96.829% when pulling 4505244f4ec9a94eaca9e8745771ceb22eb62962 on b5y:feature/support_grep into 7e9b1b583b89be447e5ae92654c2fea79c626b3d on Yelp:master.

Refs #20.

@evhub Thanks!

@b5y before you start implementing, please join the discussion at the issue page, thanks!

@b5y after the discussion, we decided to add a tips section instead of adding this feature into undebt cli. In this case, we don't need to reinvent wheels. Thanks!

@paiweilai Okay.


Establishing a db connection can block the editor

Better:

Limit retries
Open in background

Running in own thread still blocks the editor, after using connetion.setAllowUserInteraction(false) still popping up a window, but pressing Cancel will do the job


Topic/linvel - [merged]

In GitLab by @nmansard on Feb 28, 2019, 13:25
Merges topic/linvel -> devel

In GitLab by @nmansard on Feb 28, 2019, 16:32
mentioned in commit cc691a7b2e2271ce79c09c772b238a954f2cf1b0

In GitLab by @nmansard on Feb 28, 2019, 16:32
merged


Missing 0.2.0-react16

Could not find artifact day8.re-frame:re-frame-10x:jar:0.2.0-react16 in jcenter (https://jcenter.bintray.com)
Could not find artifact day8.re-frame:re-frame-10x:jar:0.2.0-react16 in clojars (https://repo.clojars.org)

Sorry, should be there now.


Add Replicate demo and API

Hi @mv-lab,
Here is the Replicate demo https://replicate.com/mv-lab/instructir :)

@chenxwh thank you very much 😄

你好@mv-lab,
这是复制演示https://replicate.com/mv-lab/instructir :)

What kind of prompt should be used to restore image quality for this type of image?


missed overrideMimeType method

Note that this method is not present in old GWT, but it is part of the standard, accepted by closure, etc:

https://developer.mozilla.org/en-US/docs/Web/API/XMLHttpRequest/overrideMimeType
https://xhr.spec.whatwg.org/#the-overridemimetype()-method
https://github.com/google/closure-compiler/blob/master/externs/browser/html5.js#L3643-L3647