Add redirects for relocated topics

With the new top-level structure of the documentation for dita-ot/docs#121, the file system locations of many topics will change between 3.0 & 3.0.1 when dita-ot/docs#185 is merged with the hotfix/3.0.1 branch.
Although few deep links to 3.0 topics are likely to have been shared so soon after the initial release, several have already been sent via email and other channels, so redirects should be set up to automatically load (at least these) topics from their new locations.
GitHub Pages does not support redirecting via server configuration files such as .htaccess or .conf, but recent versions of the gem do support redirects via the jekyll-redirect-from plugin, which serves HTML files with an HTTP-REFRESH meta tag that points to the new destination.
As the name suggests, the plugin’s default usage is to add redirect_from metadata to the YAML header of files that should answer to alternative URLs, so the file that includes these entries is loaded instead. This approach is less useful in our case, where files are auto-generated from a common layout template, so individual overrides are less practical.
However, the plugin also supports a lesser-known redirect_to mode that can be used to set up (otherwise empty) pages at the old location with a YAML front-matter block that points to the new location:
---
# Redirect from old permalink
layout: base
title: Moved to new location…
redirect_to:
  - /3.0/topics/migration.html
---

I have verified this approach in local testing, and plan to roll out the changes with the 3.0.1 hotfix.

Still need to verify which 3.0 URLs have been shared thus far and set up the redirects for these files.
I don’t think it’s necessary to set up redirects for every relocated topic, so will check analytics data and forums for clues on which URLs need to be redirected.

The following 3.0 URLs have been publicly shared and will need to be redirected for 3.0.1:

[ ] http://www.dita-ot.org/3.0/dev_ref/migration.html
→ /3.0/topics/migration.html
[ ] http://www.dita-ot.org/3.0/dev_ref/plugin-newtranstype.html
→ /3.0/topics/plugin-newtranstype.html
[ ] http://www.dita-ot.org/3.0/dev_ref/html-customization-plugin-bundle-css.html
→ /3.0/topics/html-customization-plugin-bundle-css.html

The changes will of course also affect paths in the dev docs.
While these are by nature more volatile and never promised to be stable, we might consider redirecting a few dev URLs that have been recently shared or receive above-average traffic, such as:

[ ] http://www.dita-ot.org/dev/dev_ref/migration.html
→ /dev/topics/migration.html
[ ] http://www.dita-ot.org/dev/dev_ref/plugin-addgeneratedtext.html
→ /dev/topics/plugin-addgeneratedtext.html
[ ] http://www.dita-ot.org/dev/user-guide/build-migrating-ant-to-dita.html
→ /dev/topics/migrating-ant-to-dita.html
[ ] http://www.dita-ot.org/dev/dev_ref/pdf-customization-best-practices.html
→ /dev/topics/pdf-customization-best-practices.html


Bug: Not preventing device sleep

Checklist

[X] Verified whether your issue exists in the latest version Mux Element(s) release: npm install @mux/[element_name] or yarn @mux/[element_name].

(NOTE: If the issue is not occurring in the latest version, we may not resolve it.)

[X] Confirmed that this issue hasn't already been filed.

Which Mux Elements/Packages has a bug?

mux-player

Which browser(s) are you using?

Chrome
Edge ("Edgeium")
Safari
Chrome Android

Which operating system(s) are you using?
macOS, Windows, Android
How are you using Mux Elements?
<script> tag
Describe the bug
Whenever a video is playing, it should prevent the device from going to sleep. This should be the default behavior of most browsers with the default HTML video element, so I'm not sure what is changing in Mux Player that disables this. Based on my tests, Media Chrome does not have this problem, it is isolated to Mux Player. It doesn't happen on every device, but on those that it does, it happens consistently.
Steps To Reproduce

Set device sleep or lock delay to something short for easier testing.
Start playing a Mux Player video and wait the amount of time for the sleep or lock to potentially activate.

Link to example: https://tmi.mux.com/sessions/opening-remarks (tested this behavior on Mux hosted videos to make sure it wasn't just on our website, it occurs on the TMI videos as well as in the Mux dashboard)
The current behavior
Device falls asleep when video is playing
The expected behavior
Device stays awake as long as video is playing

Thanks for reporting @oburt -- we'll have a look

Confirmed on an Android mobile device. My iPhone didn't have the issue.
was it on mobile devices only or on laptops as well?

We had it reported on both Mac and Windows desktops, as well as Android phones.

thanks, I can confirm on Chrome / Mac as well if I set "turn display off after" to a short time like 1min.
to me this looks like a browser bug because normally a playing video element should make the browser prevent the display from sleeping. it's possible it's related to that the Mux player <video> element is nested in the shadow dom.

Yeah, my uneducated guess would also have been something about the video element existing in the shadow dom as well since it needs to be something fundamentally different in mux-video or mux-player to prevent that default HTML video behavior. I worked my way down the development stack and compared it to media-chrome on some of those browsers that had issues and media-chrome worked correctly.
An example : https://media-chrome.mux.dev/examples/basic.html

I filed a Chrome bug https://bugs.chromium.org/p/chromium/issues/detail?id=1348320
Confirmed it's not an issue on Safari and Firefox.
On Edge it is an issue as well probably because the underlying Chromium.

Here's the workaround I am using in the meantime for this.  (Sorry if this isn't how sharing of this should be done.  If there's a better place for it, point me to it.)
wakeLock = null;

startWakeLock = function () {
    document.addEventListener('visibilitychange', () => {
        if (wakeLock !== null && document.visibilityState === 'visible') {
            requestWakeLock();
        }
    })
    requestWakeLock();
};

requestWakeLock = async function () {
    // Create a reference for the Wake Lock.
    if (navigator.wakeLock) {// create an async function to request a wake lock
        try {
            wakeLock = await navigator.wakeLock.request('screen');
            console.log('requestWakeLock');
        } catch (err) {
            console.log(err.name);
        }
    }
};

releaseWakeLock = function () {
    if (wakeLock)
        wakeLock.release()
            .then(() => {
                console.log('releaseWakeLock');
                wakeLock = null;
            })
};

//get the mux-player
popupVideo = document.getElementById("vidPopup");
//add event listeners
popupVideo.addEventListener("play", function () {
    startWakeLock();
});
popupVideo.addEventListener("pause", function () {
    releaseWakeLock();
});

the fix is seemingly coming in Chrome 107 🎉
https://bugs.chromium.org/p/chromium/issues/detail?id=1348320#c14

According to the tags on the issue, it might even get ported down to 106!

you're right woohoo Labels: -Merge-Request-106 Hotlist-Merge-Approved Merge-Approved-106

Nice! Awesome that it got prioritized and turned around pretty quickly by the Chromium team

this is fixed in Chrome 106 🎉
thanks for reporting!
https://bugs.chromium.org/p/chromium/issues/detail?id=1348320#c19


Sublime v1 loan status updates

I have added pause() in all status updates. Not really sure if this is needed or if it could indeed cause problems.

I have added pause() in all status updates. Not really sure if this is needed or if it could indeed cause problems.

I think this is fine, when the loan is not active (for whatever reason like closed, terminated, cancelled), then there is no point in transferring lending positions to someone else. So it makes sense to pause all lending position transfers.


[Registry Preview] Fix the issue of saving files without truncation

Summary of the Pull Request
Fix the issue of saving files without truncation, according to https://learn.microsoft.com/en-us/dotnet/api/system.io.filemode?view=net-7.0, it should be FileMode.Create, not FileMode.OpenOrCreate, which will truncate the file if it exists.

PR Checklist

[x] Closes: #27049
[ ] Communication: I've discussed this with core contributors already. If work hasn't been agreed, this work might be rejected
[ ] Tests: Added/updated and all pass
[ ] Localization: All end user facing strings can be localized
[ ] Dev docs: Added/updated
[ ] New binaries: Added on the required places

[ ] JSON for signing for new binaries
[ ] WXS for installer for new binaries and localization folder
[ ] YML for CI pipeline for new test projects
[ ] YML for signed pipeline

[ ] Documentation updated: If checked, please file a pull request on our docs repo and link it here: #xxx

Detailed Description of the Pull Request / Additional comments

Validation Steps Performed

@randyrants fyi

File this under "poorly named API" I guess!  Thanks for the fix!


libCEED - v0.7 release

libCEED has updated our API for CeedElemRestrictions to better support mixed finite elements. This change will be part of the libCEED v0.6.1 release.
I have included the updates to the API in this branch. I imagine that there will be other changes required for the libCEED v0.6.1 release, and I am working on figuring out exactly what changes are required.

This being a significant API change, the next libCEED release will be v0.7.

Then you want to update also the libCEED requirement in INSTALL:
https://github.com/mfem/mfem/blob/master/INSTALL#L643-L648

@tzanio thank you.
I am not sure if @dmed256 has another OCCA release coming soon, but the new OCCA backend that will be a part of libCEED v0.7 will require OCCA changes beyond v1.0.9, such as https://github.com/libocca/occa/commit/f0b26b77acb3e0b9493faab6d1c380abc94534da.

I have a draft for the next OCCA version I plan to release one the new backend is merged. I'll spend time this weekend updating the backend based on the PR comments. I'm still unsure why there's a output difference between the local runs and Travis CI runs :(

With the release of OCCA v1.0.10, should this PR update MEFM + OCCA as well, or would a concurrent separate PR be preferred?

With the release of OCCA v1.0.10, should this PR update MEFM + OCCA as well, or would a concurrent separate PR be preferred?

I'll prefer to do it in this PR

This PR is a prerequsite for me adding libCEED's new GPU diagonal assembly capability to MFEM. Would anyone object if I split this PR into

libCEED interface update - mergable now
libCEED v0.7 - megrable soonish

to get these changes into MFEM sooner so I can start my folloup PR to add libCEED diagonal assembly to MFEM this week?

I think it would be good to merge the libCEED interface update, since currently the two master (MFEM and libCEED) branches don't compile together.

No objections

In the v0.7 release PR, I'll also add our new interface that will largely replace CeedVectorSyncArray():
https://github.com/CEED/libCEED/pull/577

LGTM

Merged in next for testing...


NIFI-2062 AWS SDK Update:

Updated the AWS SDK dependency version to 1.11.8
Removed GetDynamoDBTest references to JsonObject class dropped from AWS SDK
Fixed integration test ITPutSNS.testPublish() - exception from not evaluating expressions
Ignored integration test ITPutS3Object.testS3MultipartAgeoff() - did not work before or after SDK update

Build and contrib all look good.  Reviewed release notes from AWS and seems pretty low risk.  Ran through some of my templates I had saved from reviewing many of the components and everything looked good from that perspective.  Will take care of merging.  Thanks for making these upgrades!


Error: New release v4.1 does not display model

If you need I can send the model to the mail.

Hi @vitavitall ,
Please share here<EMAIL_ADDRESS>

Is it also not working for newly created modeler file ?

Does not work for some old files only.
Sent.

Thank you.
Need to remove from model.jpa:
m="true"


Error None reported

I've used the example smtp-gmail.rs for the 10 alpha release.
Unfortunately although it builds fine on Windws10 and Linux(RH7.4) I'm getting strange errors.
I'm looking to eventually replace some perl scripts one of which sends mail:
        my $owner =<EMAIL_ADDRESS>        my $smtp = Net::SMTP->new("smtp.cpy.com",
                        Timeout => 60);
        $smtp->mail("Simon.Heffer\@cpy.com");
        $smtp->to($owner);
        $smtp->data;
        $smtp->datasend("Subject:$subject");
        $smtp->datasend("\n");
        $smtp->datasend("To: $owner");
                $smtp->datasend("\n\n");
        $smtp->datasend("\n");
        $smtp->datasend("\n\n");
        $smtp->dataend;
        $smtp->quit;

I've plugged in the servername/addresses from here into your example but get (on LINUX):
Could not send email: Err(Pool(Error(None)))
I've tried with and without creds (there are none above)
Any further debugging I can do?

It's strange that Pool is in the error, since that example doesn't use a connection pool.
I'm not at the computer at the moment, but just as a test could you try running the example with these cargo flags (I'm assuming you cloned the repo and ran the example directly from the examples folder):
--no-default-features --features "smtp-transport builder native-tls" (you can replace native-tls with rustls-tls if you prefer using rustls as a TLS backend)

Thanks Paolo, it's all new to me (I'm cramming it into a hackweek at work).
Anyway no luck.
cargo build --no-default-features --features "smtp-transport builder native-tls"

error: Package `CheckSDKDIR v0.1.0 (<myhomedir>/rust/CheckSDKDIR)` does not have these features: `native-tls, smtp-transport, builder`

I tried eg Message::builder and SmtpTransport but got same error.

I think I was able to reproduce a similar error, although not the exact one you got. I think the issue might be that your perl script is opening a plain connection and upgrading to STARTTLS later if the server supports it.
SmtpTransport::relay instead opens the connection as TLS, which your smtp server probably doesn't support.
lettre 0.10.0-alpha.1 doesn't provide an easy method for constructing an SmtpTransport which opens the connection as plain and upgrades it with STARTTLS, so at the moment you will have to build the SmtpTransport from scratch yourself. You can do so by taking inspiration from the  code inside of SmtpTransport::builder and change the line .tls(Tls::Wrapper(tls_parameters))) to .tls(Tls::Required(tls_parameters))) or .tls(Tls::Opportunistic(tls_parameters))). Tls doc
The resulting function, assuming you are using rustls will be:
use lettre::{
    transport::smtp::{authentication::Credentials, client::net::TlsParameters, Tls},
    Message, SmtpTransport, Transport,
};
// add rustls = "0.17" to Cargo.toml
use rustls::ClientConfig;
// add webpki-roots = "0.19" to Cargo.toml
use webpki_roots::TLS_SERVER_ROOTS;

fn main() {
    let email = Message::builder()
        .from("NoBody <nobody@domain.tld>".parse().unwrap())
        .reply_to("Yuin <yuin@domain.tld>".parse().unwrap())
        .to("Hei <hei@domain.tld>".parse().unwrap())
        .subject("Happy new year")
        .body("Be happy!")
        .unwrap();

    let creds = Credentials::new(
        "example_username".to_string(),
        "example_password".to_string(),
    );

    // lettre 0.10.0-alpha.1 didn't allow easily configuring an SmtpTransport using STARTTLS.
    // If this SMTP server allowed TLS connections we wouldn't need this and would simply use
    // https://docs.rs/lettre/0.10.0-alpha.1/lettre/transport/smtp/struct.SmtpTransport.html#method.relay
    // like so: https://github.com/lettre/lettre/blob/v0.10.0-alpha.1/examples/smtp_gmail.rs
    //
    // .tls(Tls::Required(tls_parameters)) might be removed in the final release and a simpler
    // way of building this might be provided
    let mut tls = ClientConfig::new();
    tls.root_store.add_server_trust_anchors(&TLS_SERVER_ROOTS);
    let tls_parameters = TlsParameters::new("smtp.server.hostname".to_string(), tls);

    // NOTE: SmtpTransport::builder will be renamed to SmtpTransport::builder_dangerous in
    // the next lettre release. Reasoning explained at https://github.com/lettre/lettre/pull/438
    let mailer = SmtpTransport::builder("same.smtp.server.hostname")
         // might also be 25?
        .port(587)
        .tls(Tls::Required(tls_parameters))
        .credentials(creds)
        .build();

    // Send the email
    let result = mailer.send(&email);

    if result.is_ok() {
        println!("Email sent");
    } else {
        println!("Could not send email: {:?}", result);
    }

    assert!(result.is_ok());
}

Hi Paolo,
Different error this time :)
Could not send email: Err(Pool(Error(Some("unexpected end of file"))))
Si

Different error this time :)

And I imagine no email was sent 🤔.

If you are familiar with debuggers you can try to find out where in the code the error was generated at

Alternatively you can manually create an SMTP connection by using SmtpConnection::connect.
Then you will have to upgrade the connection with STARTTLS by doing this.
After this you should be able to check that the connection is ok and try sending an email.
I haven't checked this since I'm writing from my phone, but at some point in the process it should fail, since we are just reproducing the functionality from the SmtpTransport. You can look at the SmtpTransport implementation if you want to see how lettre internally does it.
If you can't figure it out post the code you were able to write and we'll continue from there.

Hi Paolo,
It's run all the way through to the println! at the end.
I tried rust-gdb but didn't get much info out of it.
Happy to add any useful println!s.
use lettre::{
    transport::smtp::{authentication::Credentials, client::net::TlsParameters, Tls},
    Message, SmtpTransport, Transport,
};
// add rustls = "0.17" to Cargo.toml
use rustls::ClientConfig;
// add webpki-roots = "0.19" to Cargo.toml
use webpki_roots::TLS_SERVER_ROOTS;
fn main() {

    let email = Message::builder()
    .from("NoBody <nobody@domain.tld>".parse().unwrap())
    .reply_to("Yuin <yuin@domain.tld>".parse().unwrap())
    .to("SimonH <simon.heffer@microfocus.com>".parse().unwrap())
    .subject("Happy new year")
    .body("Be happy!")
    .unwrap();

let creds = Credentials::new(
    "<myuser>".to_string(),
    "<mypassword>".to_string(),
);
    let mut tls = ClientConfig::new();
    tls.root_store.add_server_trust_anchors(&TLS_SERVER_ROOTS);
    let tls_parameters = TlsParameters::new("ourserver".to_string(), tls);

// Open a remote connection to gmail
let mailer = SmtpTransport::builder("ourserver")
    .port(25)
    .tls(Tls::Required(tls_parameters))
//    .credentials(creds)
    .build();

// Send the email
let result = mailer.send(&email);

if result.is_ok() {
    println!("Email sent");
} else {
    println!("Could not send email: {:?}", result);
}

Hi Paolo, Tried again with rust-gdb.
All I could see was that at some point the Pool(Err) code was reached but that may be some default value.
33     #[cfg(feature = "r2d2")]
134     impl From<r2d2::Error> for Error {
135         fn from(err: r2d2::Error) -> Error {
136             Pool(err)
137         }
138     }
139
140     impl From<Response> for Error {
(gdb) print err
$11 = Error = {Option<alloc::string::String>}
(gdb) bt
#0  _$LT$lettre..transport..smtp..error..Error$u20$as$u20$core..convert..From$LT$r2d2..Error$GT$$GT$::from::hef82cce56dcaed19 (err=Error = {...})
    at <myhomedir>/.cargo/registry/src/github.com-1ecc6299db9ec823/lettre-0.10.0-alpha.1/src/transport/smtp/error.rs:136
#1  0x000055555577ed43 in _$LT$lettre..transport..smtp..SmtpTransport$u20$as$u20$lettre..Transport$GT$::send_raw::h79e2100a8b5ac98d (self=0x7fffffffdaf8,
Python Exception <type 'exceptions.TypeError'> Could not convert Python object: <gdb.Field object at 0x7ff451552240>.:
    envelope=0x7fffffffd300, email=) at <myhomedir>/.cargo/registry/src/github.com-1ecc6299db9ec823/lettre-0.10.0-alpha.1/src/transport/smtp/mod.rs:266
#2  0x000055555576c5a2 in lettre::Transport::send::h87a8c7a2dbdb7303 (self=0x7fffffffdaf8, message=0x7fffffffd2a8)
    at <myhomedir>/.cargo/registry/src/github.com-1ecc6299db9ec823/lettre-0.10.0-alpha.1/src/lib.rs:148
#3  0x000055555576b585 in CheckSDKDIR::main::h62e04cef3d873af9 () at src/main.rs:43
#4  0x000055555576bf6b in std::rt::lang_start::_$u7b$$u7b$closure$u7d$$u7d$::h43ccdd4d91191773 () at /rustc/5c1f21c3b82297671ad3ae1e8c942d2ca92e84f2/src/libstd/rt.rs:67
#5  0x0000555555bca848 in {{closure}} () at src/libstd/rt.rs:52
#6  do_call<closure-0,i32> () at src/libstd/panicking.rs:297
#7  try<i32,closure-0> () at src/libstd/panicking.rs:274
#8  catch_unwind<closure-0,i32> () at src/libstd/panic.rs:394
#9  std::rt::lang_start_internal::h464df2bbf46c7e7c () at src/libstd/rt.rs:51
#10 0x000055555576bf47 in std::rt::lang_start::h65023bfb4c31d368 (main=0x55555576b130 <CheckSDKDIR::main::h62e04cef3d873af9>, argc=1, argv=0x7fffffffe128)
    at /rustc/5c1f21c3b82297671ad3ae1e8c942d2ca92e84f2/src/libstd/rt.rs:67
#11 0x000055555576b92a in main ()

Could you try compiling without r2d2 (connection pool) suuport? You can do it by importing lettre like so:
lettre = { version = "0.10.0-alpha.1", default-features = false, features = ["smtp-transport", "builder", "rustls-tls"] }

If that doesn't fix it, or doesn't make the actual error clearer, another thing you could try is to "manually" send emails instead of letting SmtpTransport call the low-level APIs for you.
use std::time::Duration;

use lettre::transport::smtp::authentication::{Credentials, Mechanism};
use lettre::transport::smtp::client::net::TlsParameters;
use lettre::transport::smtp::client::SmtpConnection;
use lettre::transport::smtp::extension::ClientId;
use lettre::Message;
use rustls::ClientConfig;
use webpki_roots::TLS_SERVER_ROOTS;

fn main() {
    let hello_name = ClientId::hostname();

    let mut tls = ClientConfig::new();
    tls.root_store.add_server_trust_anchors(&TLS_SERVER_ROOTS);
    let tls_parameters = TlsParameters::new("smtp.domain.tld".to_string(), tls);

    println!("-- connecting...");
    let timeout = Duration::from_secs(30);
    // you can change the last value here from None to Some(&tls_parameters) to make it use a TLS connection instead
    let mut conn = SmtpConnection::connect("smtp.domain.tld:25", Some(timeout), &hello_name, None)
        .expect("connect");
    println!("-- connected");

    println!("can_starttls: {}", conn.can_starttls());
    println!("is_encrypted: {}", conn.is_encrypted());
    println!("test_connected: {}", conn.test_connected());

    println!("-- upgrading with STARTTLS...");
    // comment this out when using a normal TLS connection or if you don't want encryption to the smtp server
    conn.starttls(&tls_parameters, &hello_name)
        .expect("starttls");
    println!("-- successfully upgraded the connection with STARTTLS");

    println!("is_encrypted: {}", conn.is_encrypted());
    println!("test_connected: {}", conn.test_connected());

    println!("-- authenticating...");
    let credentials = Credentials::new("smtp username".into(), "smtp password".into());
    conn.auth(&[Mechanism::Plain], &credentials).expect("auth");
    println!("-- authenticated");

    let email = Message::builder()
        .from("Name <name@domain.tld>".parse().unwrap())
        .to("Paolo Barbolini <paolo@paolo565.org>".parse().unwrap())
        .subject("Happy new year")
        .body("Be happy!")
        .expect("build message");

    println!("-- sending email...");
    conn.send(email.envelope(), &email.formatted())
        .expect("send");
    println!("-- email sent");

    println!("-- sending quit command");
    conn.quit().expect("quit");
    println!("-- quit command sent");
}

first attempt gives
Could not send email: Err(Io(Kind(UnexpectedEof)))
running your code gave:
-- connecting...
-- connected
can_starttls: true
is_encrypted: false
test_connected: true
-- upgrading with STARTTLS...
thread 'main' panicked at 'starttls: Io(Kind(UnexpectedEof))', src/main.rs:31:5

Hope that helps
Si

This narrows it down to writing a command or reading a response.


Accessibility Quiz step 48: a better solution?

Describe the Issue
The template answer for the second part of this challenge is position: fixed. According to MDN, with fixed:

The element is removed from the normal document flow, and no space is created for the element in the page layout.

As a result, the header covers the first h2, Student Info.
Whereas position: sticky does not have this effect.

The offset does not affect the position of any other elements.

Before: Student Info is visible

After adding position: fixed: Student Info is covered

After adding position: sticky: Student Info is visible

Affected Page
https://www.freecodecamp.org/learn/2022/responsive-web-design/learn-accessibility-by-building-a-quiz/step-48
Your code
header {
  width: 100%;
  height: 50px;
  background-color: #1b1b32;
  display: flex;
  justify-content: space-between;
  align-items: center;
  position: sticky;
  top: 0;
}

Expected behavior
My code passes the tests.
Screenshots
No response
System

Device: Laptop
OS: Windows 11
Browser: Edge
Version: 119

Additional context
No response

Never mind. We fix the padding in Step 49. The use of position: fixed seems intentional.


OCM-1410 | fix: correct error message when 'rosa link user-role' with account-id under diff org

Related Issue: https://issues.redhat.com/browse/OCM-1410

lgtm

/lgtm


Create guildMemberUpdate.js

i updated the color and removed the unused line

Looks good! Thankyou for creating this pull request 🔥


[FIXED]: Internet disconnection after 5-10 mins

RESOLVED:
The issue was the addition of Pi-hole query rate-limiting by default with a recent update. The rate limit is set to allow no more than 1000 queries per 60 seconds. The limit us quickly reached using this mod. If you use a Pi-hole, to resolve it you need to edit /etc/pihole/pihole-FTL.conf and add the line RATE_LIMIT=0/0 or if already present change the values to 0/0. This fixes the issue in this case.
Thank you for your work!
BEFORE SUBMIT A BUG, PLEASE READ THE FAQ PAGE HERE: https://github.com/derekhe/msfs2020-google-map/wiki/FAQ
Describe the bug
Everything works great for 5 - 10 minutes the breaks my connection to the internet. All other PC's connected to the network still function normally but not mine. To restore internet I have to "ipconfig /flushdns" and unplug my network cable then plug it back in.
To Reproduce
Steps to reproduce the behavior:
Screenshots
If applicable, add screenshots to help explain your problem.
Desktop (please complete the following information):

OS: [e.g. iOS] Windows 11
MOD Version [e.g. 22]  2021-11-20
MSFS Version, MS Store or Stream: Store

Additional context
Add any other context about the problem here.

Try disabling IPv6, worked for me.

I was about to say "I never have that enabled!" But uhh... somehow it was enabled :)
Thanks for the tip!

Here is a log to review
log.txt

Can you open maps.google.com without issue?
I see from the log that there are issue when accessing google.

@lazerwolf22 fixed?

Google Maps always works fine. Mod still broken.

[Errno 11001] getaddrinfo failed over and over again.
That looks like your DNS server is throttling you....

I am having the same problem.
I thought it was a non-mod issue, such as clean install of windows, reinstalling MSFS and XBOX.
Mr/Mis ferryvanaesch's advice was a great tip.
----error log
2021/11/25 08:05:12 [error] 1660#6704: *4687 connect() failed (10060: A connection attempt failed because the connected party did not properly 10060: A connection attempt failed because the connected party did not properly respond after a period of time, or established connection failed because the connected host has failed to respond) while connecting to upstream, client: 127 .0.0.1, server: khstorelive.azureedge.net, request: "GET /results/v1.21.0/vegetation_cgl/vgm120202.cgl?version=5 HTTP/1.1", upstream: " https://<IP_ADDRESS>:443/results/v1.21.0/vegetation_cgl/vgm120202.cgl?version=5", host: "khstorelive.azureedge.net"
My provider is still preparing for IPv6, so I'm connecting via IPv4.
I have tried DNS <IP_ADDRESS> and am getting the same error.
I think this mod is great, but it seems to be difficult in my connection environment.

Yes, your connection issue looks different: time-outs. That's not your DNS indeed. Frankly that could be anywhere, from your own pc to your router, your ISP or even the peering contracts they have. Without some in-depth analysis I can't tell you where the issue lies.
Oh, it's Mr. ;-)

Please try latest version: https://github.com/derekhe/msfs2020-google-map/releases/tag/2.3.2

Thank you.It is unclear to me what is going on, so I am attaching the LOG.The location is Kitakyushu, Fukuoka, Japan.No mods are installed. I'm glad if it's helpful. Windows の メール から送信 差出人: hesicong送信日時: 2021年11月28日 17:13宛先: derekhe/msfs2020-google-mapCC: tarukuu; Comment件名: Re: [derekhe/msfs2020-google-map] [BUG] Internet disconnection after 5-10 mins (Issue #114) Please try latest version: https://github.com/derekhe/msfs2020-google-map/releases/tag/2.3.2—You are receiving this because you commented.Reply to this email directly, view it on GitHub, or unsubscribe.Triage notifications on the go with GitHub Mobile for iOS or Android.

Sorry, if you sending from the phone, I can't see the attachment. Please post it here.

OK Again  Windows の メール から送信 差出人: hesicong送信日時: 2021年11月29日 19:06宛先: derekhe/msfs2020-google-mapCC: tarukuu; Comment件名: Re: [derekhe/msfs2020-google-map] [BUG] Internet disconnection after 5-10 mins (Issue #114) Sorry, if you sending from the phone, I can't see the attachment. Please post it here.—You are receiving this because you commented.Reply to this email directly, view it on GitHub, or unsubscribe.Triage notifications on the go with GitHub Mobile for iOS or Android.

Sorry, I still can't the posts, you had to put your attachment into https://github.com/derekhe/msfs2020-google-map/issues/114

https://drive.google.com/drive/folders/10GBcP5pk8XEVVgSz01qEdxKvSbtC8vJx?usp=sharing
its Error Log files

new virsion  2.3.3 log here
https://drive.google.com/drive/folders/10TFsIheGRBFdI6I7an4DFkRpBu6riVmt?usp=sharing
I think it is being replaced by Google maps. However, there is a delay in reading. It went offline due to the fact that it has a narrow bandwidth in a hair-trigger way.
AMD Ryzen 9 5900X
RTX 3070Ti
Memory 32GB
Internet connection speed 63.99 Mbps
--exsample--
2021/12/02 14:50:04 [error] 34796#17056: *1831 WSARecv() failed (10054: An existing connection was forcibly closed by the remote host) while reading response header from upstream, client: <IP_ADDRESS>, server: kh.ssl.ak.tiles.virtualearth.net, request: "GET /tiles/akh3112231302310113.jpeg?n=z&g=9591 HTTP/1.1", upstream: "http://<IP_ADDRESS>:39871/tiles/akh3112231302310113.jpeg?n=z&g=9591", host: "kh.ssl.ak.tiles.virtualearth.net"
2021/12/02 14:50:04 [warn] 34796#17056: *1831 upstream server temporarily disabled while reading response header from upstream, client: <IP_ADDRESS>, server: kh.ssl.ak.tiles.virtualearth.net, request: "GET /tiles/akh3112231302310113.jpeg?n=z&g=9591 HTTP/1.1", upstream: "http://<IP_ADDRESS>:39871/tiles/akh3112231302310113.jpeg?n=z&g=9591", host: "kh.ssl.ak.tiles.virtualearth.net"
2021/12/02 14:50:04 [warn] 34796#17056: *1831 upstream server temporarily disabled while reading response header from upstream, client: <IP_ADDRESS>, server: kh.ssl.ak.tiles.virtualearth.net, request: "GET /tiles/akh3112231302310113.jpeg?n=z&g=9591 HTTP/1.1", upstream: "http://<IP_ADDRESS>:39871/tiles/akh3112231302310113.jpeg?n=z&g=9591", host: "kh.ssl.ak.tiles.virtualearth.net"
2021/12/02 14:50:04 [error] 34796#17056: *1831 no live upstreams while connecting to upstream, client: <IP_ADDRESS>, server: kh.ssl.ak.tiles.virtualearth.net, request: "GET /tiles/akh3112231302310113.jpeg?n=z&g=9591 HTTP/1.1", upstream: "http://localhost/tiles/akh3112231302310113.jpeg?n=z&g=9591", host: "kh.ssl.ak.tiles.virtualearth.net"

Thanks, I will have a look.

It's still working fantastic! I flew for several hours last night! Thank you again!

New released Thank you.
I test flight and got data.
https://drive.google.com/drive/folders/11GgC52RmIAZkLZyf_nWF1AT6lMKBnRE9?usp=sharing

@tarukuu what's your problem?


Fix a concurrency bug with the parallel image building

Before, and after parser has generated the variants scripts and Dockerfiles, orchestration used to build the variants concurrently.
Every variant build consisted of copying the variant scripts and Dockerfile to the source directory and launching a docker build of the image.
This obviously is incorrect, as there is only one source directory and multiple variants
The fix makes use of docker's recent ability to use a custom Dockerfile:
We no longer copy the variant files to the source directory.
Instead, the image build is performed using the job root directory as the build context, and individually adding the variant's script files
to the image (using COPY commands in the variant's Dockerfile).
Fixes #155

This fix depends on a PR on go-dockercommand.
The PR is already merged, so you'll need to run make updatedeps before testing it.


[core] Improve performance for small downloads

When downloading parts of BLOBS (ranges) which result in one download the whole download is performed before
the first chunks are published to the result stream (PartStream).
Solution: Remove the PartStream and replace it with an "OrderedChunkStream" and add an optimization which directly passes the chunks through if only 1 part is downloaded. If only one part is downloaded, the chunks will have the correct ordering.
This will result in a braking change.

@Alexander-Dubrawski This might be the cause for the delay on the first page of a segment. This would also explain why all following pages are processed immediately.


Find + Pagination

In the tutorial: https://github.com/CakeDC/tags/blob/master/Docs/Tutorials/Find-Tagged-Objects.md#pagination-usage-example
There's a "Pagination usage example". This example provides an array of 'Tagged' and 'Tag' elements. This is different than the Find example, which retrieves an array of the Model elements, i.e. 'Recipe', ready to set to the view.
It would be useful to have an example ready to copy & paste, as the Find usage examples are, which includes Pagination, so just using $this->set('recipes', $recipes); the view would have the paginated results including the Recipe elements.

One possibility is the following code. It will retrieve all the recipes and bring them to the view, ready to show.
public function index() {
    $recipes = $this->Paginator->paginate('Tagged', array(
        'Tagged.model' => 'Recipe',
        'Tag.keyname' => $this->passedArgs['by']
    ));

    $this->set('recipes', $recipes);
}

Thanks for the feedback @rogerpro!
The example you provided is more like an different way to achieve the same result, but for me more examples are always better. 👍

Closing issue - Please re-open if necessary!


set targetView as Toolbar menu item?

Hai, I have some items in toolbar menu and I need to use it as view. I tried findViewById(int) but it's not working.
Any solution ?
Thanks

Take a look at this post:
http://stackoverflow.com/questions/8614293/android-get-view-reference-to-a-menu-item

I have successfully implemented Spotlight on a Toolbar menu item using the method described in the second answer from this post (using ViewTreeObserver).


elixir fails to package when multilib is in use

When using multilib, libdir = /usr/lib64 not /usr/lib.  The FILES statements should all be using nonarch_libdir instead of libdir.  Using libdir causes most of the files to get skipped and not packages which of course causes the packaging to fail.

I would have to figure out which parts of the config are relevant.  But it is building for x86_64 with support for x86 32 bit as well, which means the libdir is /usr/lib64 for x86_64 packages and /usr/lib for x86 packages.  Most things are built for x86_64, including erlang and elixir.  But since the elixir stuff is not architecture specific (I hope), it seems it is installing in /usr/lib rather than /usr/lib64.  I suppose that could be a mistake and it is actually using the wrong location for the libraries.
We are pretty much doing what yocto shows here:
https://github.com/intel/luv-yocto/blob/master/meta-skeleton/conf/multilib-example.conf

It might be necessary to set the correct erlang version for rabbitmq, according to https://www.rabbitmq.com/which-erlang.html. I'm testing it too.
Are you getting this error ?
| 20:47:36.454 [error] Loading of /build/tmp/work/core2-64-poky-linux/rabbitmq-server/3.9.8-r1/rabbitmq-server-3.9.8/deps/.mix/archives/hex-0.21.3/hex-0.21.3/ebin/Elixir.Hex.Version.beam failed: :badfile
|
|
| 20:47:36.455 [notice] Application hex exited: exited in: Hex.Application.start(:normal, [])
|     ** (EXIT) an exception was raised:
|         ** (UndefinedFunctionError) function Hex.Version.start/0 is undefined (module Hex.Version is not available)
|             (hex 0.21.3) Hex.Version.start()
|             (hex 0.21.3) lib/hex/application.ex:12: Hex.Application.start/2
|             (kernel 8.4) application_master.erl:293: :application_master.start_it_old/4
|
| 20:47:36.463 [notice] Application inets exited: :stopped
|
| 20:47:36.463 [notice] Application ssl exited: :stopped
|
| 20:47:36.463 [notice] Application public_key exited: :stopped
|
| 20:47:36.463 [notice] Application asn1 exited: :stopped
|
| 20:47:36.463 [notice] Application crypto exited: :stopped
| Could not start Hex. Try fetching a new version with "mix local.hex" or uninstalling it with "mix archive.uninstall hex.ez"
| ** (MatchError) no match of right hand side value: {:error, {:hex, {:bad_return, {{Hex.Application, :start, [:normal, []]}, {:EXIT, {:undef, [{Hex.Version, :start, [], []}, {Hex.Application, :start, 2, [file: 'lib/h}
|     (hex 0.21.3) lib/hex.ex:5: Hex.start/0
|     (mix 1.13.4) lib/mix/hex.ex:59: Mix.Hex.start/0
|     (mix 1.13.4) lib/mix/dep/loader.ex:203: Mix.Dep.Loader.with_scm_and_app/4
|     (mix 1.13.4) lib/mix/dep/loader.ex:156: Mix.Dep.Loader.to_dep/3
|     (elixir 1.13.4) lib/enum.ex:1593: Enum."-map/2-lists^map/1-0-"/2
|     (mix 1.13.4) lib/mix/dep/loader.ex:373: Mix.Dep.Loader.mix_children/2
|     (mix 1.13.4) lib/mix/dep/loader.ex:18: Mix.Dep.Loader.children/0
|     (mix 1.13.4) lib/mix/dep/converger.ex:57: Mix.Dep.Converger.all/4
| make[2]: *** [Makefile:106: escript/rabbitmqctl] Error 1
| make[2]: *** Waiting for unfinished jobs....
| Inlining: inline_size=24 inline_effort=150
|  APP    observer_cli.app.src
| make[3]: Leaving directory '/build/tmp/work/core2-64-poky-linux/rabbitmq-server/3.9.8-r1/rabbitmq-server-3.9.8/deps/observer_cli'
| make[2]: Leaving directory '/build/tmp/work/core2-64-poky-linux/rabbitmq-server/3.9.8-r1/rabbitmq-server-3.9.8/deps/rabbitmq_cli'
| make[1]: *** [../../erlang.mk:4512: deps] Error 2
| make[1]: Leaving directory '/build/tmp/work/core2-64-poky-linux/rabbitmq-server/3.9.8-r1/rabbitmq-server-3.9.8/deps/rabbit'
| make: *** [erlang.mk:4512: deps] Error 2
| ERROR: oe_runmake failed
| WARNING: exit code 1 from a shell command.

Yes exactly that.  I will try changing to erlang 24.3.3 then.

Try to use these versions:
# meta-erlang-elixir
PREFERRED_VERSION_elixir = "1.12%"
PREFERRED_VERSION_elixir-native = "1.12%"
PREFERRED_VERSION_nativesdk-elixir = "1.12%"

# meta-erlang-erlang
PREFERRED_VERSION_erlang = "24.2"
PREFERRED_VERSION_erlang-native = "24.2"
PREFERRED_VERSION_nativesdk-erlang = "24.2"

That is the correct combination for rabbitmq 3.9.8 (I'll bump the version later, or at least keep one for each major erlang/elixir combination).
Now, it looks like rabbitmq-server recive also needs change about multilib.

Yeah I did that and now it works nicely if I use the for-master-elixir-nonarch_libdir branch.  Hope that gets merged to kirkstone branch.

Hi.
Done. For master and kirkstone
Thanks.


[Bug] beacon picker doesn't care about the beacon color

I could fix this, should I open a PR?

Sure, that would be great!


Add ability to use own ajax transport

With this pull request you can use your own transport ajax.
From jQuery 3 you can customize the dependencies of jQuery, for example by removing the ajax module.
Also with the new fetch standard you can leverage api already in the browser.

Thank you for the PR.
Unfortunately, I will not merge this - it is fairly easy to use the core.data as a function option and implement whatever strategy you want:
https://github.com/vakata/jstree#populating-the-tree-using-a-callback-function
This PR pretty much duplicates this functionality.
Best regards,
Ivan


Homekit wont start even after manually triggering it.

Home Assistant release with the issue:

Last working Home Assistant release (if known):
.96.5
Operating environment (Hass.io/Docker/Windows/etc.):

Component/platform:

Description of problem:
As indicated, Homekit won't start even after manually triggering the automation.
Here are the relevant lines in the logs. There isn't any other debug information available yet.
Sep 30 10:10:26 hassbian hass[12119]: 2019-09-30 10:10:26 INFO (MainThread) [homeassistant.helpers.script] Script Start HomeKit: Executing step delay 0:00:30
Sep 30 10:10:46 hassbian hass[12119]: 2019-09-30 10:10:46 INFO (SyncWorker_13) [zoneminder.zm] Initializing runstate 1
Sep 30 10:10:57 hassbian hass[12119]: 2019-09-30 10:10:57 INFO (MainThread) [homeassistant.helpers.script] Script Start HomeKit: Executing step call service
Sep 30 10:10:57 hassbian hass[12119]: 2019-09-30 10:10:57 WARNING (SyncWorker_1) [homeassistant.components.homekit] HomeKit is not ready. Either it is already running or has been stopped.

With the additional information in traceback below, Looks like it is related to the fan settings below, but I do see a value for the fan 'on' and the fan 'off'
Problem-relevant configuration.yaml entries and (fill out even if it seems unimportant):
homekit:
  auto_start: False

And also from automations.yaml I have this information:

- id: start_hk
  alias: Start HomeKit
  trigger:
  - platform: event
    event_type: zwave.network_ready
  - platform: event
    event_type: zwave.network_complete
  - platform: event
    event_type: zwave.network_complete_some_dead
  action:
  - delay: 00:00:30
  - service: homekit.start

fan:
  - platform: mqtt
    name: "SW BDRM"
    command_topic: "cmnd/swbdroom/FanSpeed"
    speed_command_topic: "cmnd/swbdroom/FanSpeed"
    state_topic: "stat/swbdroom/RESULT"
    speed_state_topic: "stat/swbdroom/RESULT"
    state_value_template: >
      {% if value_json.FanSpeed is defined %}
        {% if value_json.FanSpeed == 0 -%}0
        {%- elif value_json.FanSpeed > 0 -%}4
        {%- endif %}
      {% else %}
        {% if states.fan.sw_bdrm.state == 'off' -%}0
        {%- elif states.fan.sw_bdrm.state == 'on' -%}4
        {%- endif %}
      {% endif %}
    speed_value_template: "{{ value_json.FanSpeed }}"
    availability_topic: tele/swbdroom/LWT
    payload_off: "0"
    payload_on: "4"
    payload_low_speed: "1"
    payload_medium_speed: "2"
    payload_high_speed: "3"
    payload_available: Online
    payload_not_available: Offline
    speeds:
      - off
      - low
      - medium
      - high

Traceback (if applicable):
Traceback (most recent call last):
  File "/usr/local/lib/python3.6/concurrent/futures/thread.py", line 56, in run
    result = self.fn(*self.args, **self.kwargs)
  File "/srv/homeassistant/lib/python3.6/site-packages/homeassistant/components/homekit/type_fans.py", line 157, in update_state
    hk_speed_value = self.speed_mapping.speed_to_homekit(speed)
  File "/srv/homeassistant/lib/python3.6/site-packages/homeassistant/components/homekit/util.py", line 201, in speed_to_homekit
    speed_range = self.speed_ranges[speed]
KeyError: 'off'

Additional information:

I have since had an answer on the forums that answered this. Turns out the solution for this issue is to put quotes around the names of the speeds as follows:
    speeds:
      - "off"
      - "low"
      - "medium"
      - "high"

Closing as fixed per the following:
https://community.home-assistant.io/t/using-ifan03-with-home-assistant-and-homekit/140034/3


Show compile errors somewhere

Normally, Fighweel shows any compile errors inside the browser. Could be cool to have the same in the simulator. Currently, compile just seems to fail silently (but they can be recovered from just fine, which is great!)

Is your console open in Chrome? Compile errors show in the web browser console in yellow for me. There is no heads up display since we don't have access to the DOM inside of web workers. You'll see a ton of errors if you set :heads-up-display to true in the project.clj file.
Having it in the simulator could be done, but would probably need to modify figwheel core code to know about and use reagent/om instead of direct document.createElement calls. Seems best to leave it in the browser's console to me.


Allow require to find modules in node_modules, etc

As a developer,
I want require to be able to find modules in the same way as nodejs
So that I can install packages such as bluebird, lodash, or custom libraries from NPM in my application.

Unless I am missing something here, at compile time, you know all the libraries that could possibly be referenced. For all static includes, i.e. require("lodash") or require("../mylib"), you could easily map and change the requires to the correct path before packaging the app.
Dynamic requires, i.e. require(myvar + "mylib") would require a bit more effort such as creating a function that could lookup the desired library at runtime and use that with the relative folder to find it in a list of libraries in the app (assuming the list was generated at compile time.)
You could also look in the package.json files at compile time to see if there is a main referenced. Support for index.js could also be added at compile time.
I've tried a few of the methods such as ti-commonjs which look for things at runtime, but the performance hit from those was far too great.
Here is the nodejs load order that we need to support:  (from https://nodejs.org/api/modules.html#modules_all_together)
require(X) from module at path Y
1. If X is a core module,
   a. return the core module
   b. STOP
2. If X begins with './' or '/' or '../'
   a. LOAD_AS_FILE(Y + X)
   b. LOAD_AS_DIRECTORY(Y + X)
3. LOAD_NODE_MODULES(X, dirname(Y))
4. THROW "not found"

LOAD_AS_FILE(X)
1. If X is a file, load X as JavaScript text.  STOP
2. If X.js is a file, load X.js as JavaScript text.  STOP
3. If X.json is a file, parse X.json to a JavaScript Object.  STOP
4. If X.node is a file, load X.node as binary addon.  STOP

LOAD_AS_DIRECTORY(X)
1. If X/package.json is a file,
   a. Parse X/package.json, and look for "main" field.
   b. let M = X + (json main field)
   c. LOAD_AS_FILE(M)
2. If X/index.js is a file, load X/index.js as JavaScript text.  STOP
3. If X/index.json is a file, parse X/index.json to a JavaScript object. STOP
4. If X/index.node is a file, load X/index.node as binary addon.  STOP

LOAD_NODE_MODULES(X, START)
1. let DIRS=NODE_MODULES_PATHS(START)
2. for each DIR in DIRS:
   a. LOAD_AS_FILE(DIR/X)
   b. LOAD_AS_DIRECTORY(DIR/X)

NODE_MODULES_PATHS(START)
1. let PARTS = path split(START)
2. let I = count of PARTS - 1
3. let DIRS = []
4. while I >= 0,
   a. if PARTS[I] = "node_modules" CONTINUE
   c. DIR = path join(PARTS[0 .. I] + "node_modules")
   b. DIRS = DIRS + DIR
   c. let I = I - 1
5. return DIRS

Associated JIRA Ticket(s):
https://jira.appcelerator.org/browse/TIMOB-20472
https://jira.appcelerator.org/browse/TIMOB-16078
https://jira.appcelerator.org/browse/TIMOB-19392

Also see:

https://jira.appcelerator.org/browse/TIMOB-16078
https://github.com/tonylukasavage/ti-commonjs

Thanks @FokkeZB!  ti-commonjs is what I actually started with a couple years ago but as soon as I started using it with Android, all the file reads slowed the app way down.  I saw your JIRA ticket sometime around then as well and thought it might be in the product soon.  I am sure my implementation of this will be somewhat "hacky" seeing as I am doing it in Alloy and not in Titanium but maybe it can be a stepping stone to something better.   I know you have a ton of good ideas floating around in your head so feel free to comment on anything you see that could potentially be done better!

I think there's a two different things to this:

With Alloy we there's things we can resolve from NPM modules during the transpile step, like widgets (PR and maybe themes relatively simple.

At runtime, since Titanium require is not NodeJS compatible, any CommonJS module requiring in other modules via relative paths or expecting it to resolve node_modules will fail. Apart from making Titanium require NodeJS compatible (TIMOB-16078) we could also make Titaniumifer part of the Alloy compile or even Titanium build process to read in a {{package.json}} and for everything used at run-time (flagged, detected or otherwise) run Titaniumifier to make it Titanium compatible before dumping it into Resources.

Feature available for testing in the beta branch of Adamantium now.
https://github.com/mobilehero/adamantium/blob/next/1.7.35/adamantium/features/support-for-nodejs-style-require.md


Network Error 403: The user has no policy access to service location.

Hello! I am facing a similar issue as #44 and #41. A few weeks ago, I ran the code below and I was able to get the output as a CSV file. However, when I tried running the same code now, it kept on giving the following error: "Network Error 403: The user has no policy access to service location." I am running the code by calling the python script (written in sublime) in the command prompt. The virtual environment set-up where I installed the 'fsf-api-access-python' went fine for me.
Code:
import firststreet
api_key = 'My API key'
fs = firststreet.FirstStreet(api_key, connection_limit=100, rate_limit=20000, rate_period=1, log=True)
location_summary = fs.location.get_summary(search_items=[511447411, 511447412, 511447413], location_type="property", csv=True)
I am running python version: 3.9.7 on a windows 10 desktop.
Thanks!
Utkarsh
Screenshots:

Hi Utkarsh,
Could you send me an email at<EMAIL_ADDRESS>There should have been an email sent out earlier about the API.
Thanks,
Kelvin

Issue resolved, closing

I also had a similar issue:
UnknownError: Network Error 403: The user has no policy access to the service location. ('https://api.firststreet.org/v1/location/summary/property?address=300 River Park Drive, Mankato, MN, 56001&', '300 River Park Drive, Mankato, MN, 56001', 'location', 'summary')
I am using my supervisor's API key. Could you please let us know how to solve this issue? Thank you!

Hi! Send an email to FSFL asking for the data you need.

Hi, thanks for your reply. Does it mean that we cannot get the data through the API and have to request data using email? Thanks.

Yes.


[WIP] Allow providing initial points (parameters and their evaluations) to the minimize functions.

Here is a proposition of a way of bootstrapping the surrogate model with user provided list of input points and their evaluations. two new parameters are provided to gp_minimize, forest_minimize, gbrt_minimize and dummy_minimize : x0 and y0
x0 is a list of input parameters and y0 a list function evaluations of the input parameters at
each element of x0.
Solution for #114

@MechCoder posted in #114 that he is also working on this. Can we make a collaborative effort between you two?

@betatim Sure, sorry @MechCoder I have just seen that you were working on this as well.

Thanks for the addition @mehdidc ! Let us know when you have addressed the comments.

Thanks! Before going further, I think we should agree on the semantics of x0, n_start and maxiter, and how they interplay with each others. This is not really clear at the moment. E.g. if n_start > 0 and x0 is not None, what is the expected behaviour?

Either one of these two:

If both n_start and x0 provided, we assume that the user wants to do a random start in addition to x0. If he would like to disable this, then he should explicitly set n_start=0

Allow n_start='auto'. If x0 is provided, this will be set to zero, otherwise it will be set to the default value of 10.

WDYT?

And then what about maxiter, does this correspond to the number of calls to func?  (i.e. does this behaves differently when y0 is None and vice-versa?)

Are you +1 for the first or the second behaviour of n_start? Thinking about it, the second one seems a bit better to me.
I believe the behavior of maxiter should be the number of calls to func as you say and the number of surrogate model fits should be maxiter - n_start

Oh sorry, I misread. That is a tough question :P

I tend to look at maxiter - n_start as the number of surrogate model fits.

If x0, y0 and n_start is provided, we assume that the user wants to random start along with the given initial parameters. In that case the behaviour of maxiter is obvious.

If only x0 is provided, we assume that the user is lazy to do the function call outside the *_minimize functions and hence do not consider these function calls either in n_start or maxiter. We still do n_start random initializations and maxiter - n_start surrogate model fits.

How does that proposal sound?

In case of 1. does this mean we build maxiter - n_start - len(x0) models? If yes, then +1 for this.

Can we change maxiter to model_iter (or some other better name?). That will resolve confusion and make things clearer?
So irrespective of what x0 and y0 are, we do n_start random calls to func and build model_iter number of models.

Let's rename n_start with n_random_start to be self-explanatory and maxiter with n_model_iter with the behaviour you suggest?

https://github.com/scikit-optimize/scikit-optimize/pull/123

Thanks for your comments @MechCoder  and @glouppe, I updated the code. Let me know what you think about it. Also, I guess I should update the code after #123 is merged.

Thanks a lot @mehdidc for the contribution and also for having patience with us trying to figure stuff out on the fly!

In case of 1. does this mean we build maxiter - n_start - len(x0) models? If yes, then +1 for this.

I would have expected that maxiter - n_start models would be fit, as maxiter sets the budget for the minimiser so it should be independent of len(x0). As a user if I provide x0 and maxiter=10 I am thinking: please do another 10 iterations and here are some points I previously evaluated. Instead of having to set maxiter = len(x0) + 10 to achieve that.

I would have expected that maxiter - n_start models would be fit, as maxiter sets the budget for the minimiser so it should be independent of len(x0). As a user if I provide x0 and maxiter=10 I am thinking: please do another 10 iterations and here are some points I previously evaluated. Instead of having to set maxiter = len(x0) + 10 to achieve that.

Yes, but only if y0 is not None. If y0 is None, then it requires calls on func.

Also i think maxiter is a terrible name. We should be more specific, and rename to that something like n_calls.

rename maxiter to n_calls. This sets the upper limit of how often a minimiser will call func (see #123)
if x0 is not None and y0 is None use len(x0) calls to evaluate func,  the number of models fitted: n_calls - len(x0) - n_random_start
if x0 and y0 are provided, perform n_random_start random samples, the number of models fitted: n_calls - n_random_start

Reasoning for 2.: if func is expensive you won't be lazy and not provide y0 if you have it -> if y0 is not provided it means we need to call func which means we are using up the budget set by n_calls. If you are resuming/warm starting then you have x0 and y0 so you should provide them.
Show your agreement with emoji reactions, if you disagree write a comment 📝 .

@mehdidc Would you be able to rebase over master? Sorry for the troubles.

@MechCoder Sure, I will do that.

I think you might have to rebase again. @glouppe changed the default n_calls of dummy_minimize in master. (Sorry :/)

There are several occurrences of if type(x) is int which should be if isinstance(x, list) (same for other types).
To check if a list contains an item (len(x) > 0) in python you simply test the truth value of the list: if x: instead of computing the length.
Take a look at PEP8 which we try and follow when possible. You can also install pep8 the command line program to give you advice.

@mehdidc Almost there! Thanks!

Thanks @mehdidc for contribution and patience. Just a few more minor changes and it should be good to go :)

Thanks for addressing those last comments Mehdi! Merging this.

🍨


duration_format support with Python3 and background exclusion

When feature file is written in another language, the keyword 'Background' will change. Searching and removing by type will work for any language;
Builtin long() does not exist anymore, changing to int() adds python 3 support when using duration_format=True;

@avanzzzi nice <3

63d6097 Fixes #6

Did you verify your changes are backward compatible with python 2.7?
Thanks.

Did you verify your changes are backward compatible with python 2.7?
Yes, works as expected.
The int() function produces the same result as the long() function except that in python 2.7, when dealing with long numbers, the int() function will suffix the number with the letter L. The json parser in python 2.7 knows how to handle this and will write the number without the L in the output json.


math.sw primitive u256 missing trait implementations

No

sqrt
logarithm
binary logarithm

Is this prioritized already? I can pick this up.

@sudhackar Hi, feel free to make a PR to fix this issue, after you add the implementation and the tests you can request a review from me


Speed and parameters about the model

@Lizhuoling Hi, thanks for your great work. Regarding Table 4, I wonder why VoxelFormer looks more lightweight than BEVFormer. I think if all the architectural specifications are aligned between them, VoxelFromer actually adds a depth prediction net (borrowed from BEVDepth). The inference time and parameter volumes might slightly increase. Correct me if I miss something.

Thanks for your concern. First of all, the depth head is much slighter than the one in BEVDepth. Secondly, there are some other differences between BEVFormer and VoxelFormer. For example, BEVFormer samples feature from 4 FPN levels, while we only use 1 level. Besides, BEVFormer needs to predict the sampling offsets, while VoxelFormer does not predict them. Due to these issues, VoxelFormer behaves faster than BEVFormer and contains fewer parameters.


docs: fix typos

Fix typos and nonsences

The changes in this PR have been addressed by #382, so I will close this for now.


Disabling submit with hosted fields

I had contacted support with this question, but I'll re-enter it here to capture it.
I am using custom integration with hosted fields and I would like to disable the submit button while waiting to get the payment nonce. My current approach is to:

Add a submit listener on the form which always disables the submit UI button upon seeing a submit event.
In braintree onError callback, I re-enable submit button in case of any validation errors.
In onPaymentMethodReceived callback, I then send the payment nonce to my server.

This basically works ok and prevents duplicate submits if and only if the user clicks on the “submit” button.
Unfortunately, if the user uses the "return" key to submit the form, then the native submit event isn’t fired. This makes it so I cannot give my user any visual feedback that something is going on, increasing the likelihood that they may try to resubmit if it takes a long time.
Perhaps I’m missing something, but I’ve searched the web a bit and it seems that a lot of people want to do this flow, but it is more difficult than it seems it should be. It seems like adding some sort of “beforeSubmit” callback after everything is validated, but just before submitting to braintree server would make this flow a lot more intuitive.
Thanks.

Unfortunately, the behavior you've described is not something we currently support. That being said, we totally understand this use case and are including this functionality in the next major version of our SDK. We'll update you here when it is available!

Thanks for the response and, yes, definitely report back and close this issue when you have something available.

Hmmm.  I have a click event where I disable the Submit button and that seems to work when  is hit.
JS/Jquery code (which is probably obvious :-)
$("#myForm button[type=submit]").click(function() {
$(this).prop('disabled', true);	// Disable the Submit button so it can't be clicked again
$("#ErrorSummary").html("")
});
Not sure if the declaration order matters, but the click event is defined early on.
HTH

Yes, if you disable the submit button, then the user is not able to click it. However, if the user presses "return" to submit the form, then there is no click on the submit button, but the form is still submitted. Further, if you disable the submit button, the user is still able to press "return" to submit the form again...at least for me.

@rickhall you are correct. Upon hitting enter within one of Hosted Fields' inputs we attempt to tokenize the card data and submit the form, bypassing any JavaScript handlers, including disabled buttons. Not the friendliest way to do things if you need to get some code inbetween the lifecycle stages of user interaction, tokenization, and submission.
One of our goals for the next major version of our SDK is to eliminate any dependency on forms and their submit events. We are actively working on this next release and will update this issue when it is available.

@mrak Yes, that was my understanding from the previous discussion, so I look forward to it. I was just responding to @ds00424 . Thanks again.

@rickhall - Ah, what I was missing was if the return is hit when within one of the hosted fields.  I have other fields in the form and if the return is hit when within one of those fields - the click event fires.  Thx.

@mrak Any way you can provide event callback in setup options that will be triggered when tokenizing card data and submitting form, so we can hook up to that callback (something like onPaymentMethodRequested or something like that)?

If you use onPaymentMethodReceived we will call it instead of submitting the form.

I know, i meant in context of this issue, you can use new event that will be triggered when you start submitting data to braintree servers. That way, we can use that new event to disable the button, hide the form or do whatever until we receive the response with onPaymentMethodReceived. That new event need to be triggered either by user submitting the form, or pressing enter key in hosted fields. That would solve the problem OP (and me) has

Ah, sorry for the misinterpretation.
While we likely will not add more lifecycle hooks to our current SDK version, our current work on the next major version directly addresses this issue.

Great.
Thanks

We don't have a hard ETA yet, but we are planning to release a beta soon.

Glad someone filed this issue. Having similar prob.

We've released a beta of our next major version. A new feature is a separate inputSubmitRequest event in Hosted Fields that you can bind to a form submit if you choose to do so. It does not have the default action of submitting a form as it does in v2.
Here are some links to our official documentation:
Setup for v3
Migrating from v2 to v3
Events in Hosted Fields

If you're using the onPaymentMethodReceived callback with Hosted Fields version 2 (I'm using 2.8) then a simple solution would be to check if the nonce has already been received.  I do this by saving the nonce to a hidden input field and only allowing the custom form submit.call() the first time a nonce has been seen. I'm already sending that hidden input field as part of the POST so it was already part of the form.
....
onPaymentMethodReceived: function (details) {
  if ($("#nonce").val() === '')
  {
    $("#nonce").val(details.nonce);
    var form = document.getElementById('credit-card-info');
    HTMLFormElement.prototype.submit.call(form);
  }
 },
....

I'm using js.braintreegateway.com/web/3.0.0-beta.9/js/hosted-fields and had the problem with users being able to submit form multiple times. I did something similar to allella, adding a counter field and only allowing the form to submit if the value of the counter === 1:
`
submit.addEventListener('click', function(event) {
  event.preventDefault();

  hostedFieldsInstance.tokenize(function(err, payload) {
    if (err) {
      console.log(err);
		alert(err);
      return;
    }

    // This is where you would submit payload.nonce to your server
    //alert('Submit your nonce to your server here!');
	// Put `payload.nonce` into the `payment-method-nonce` input, and then
    // submit the form. Alternatively, you could send the nonce to your server
    // with AJAX.
	 counter = parseInt(document.querySelector('input[id="counter"]').value);
	 console.log("counter = " + counter);
	 if (counter === 1)
	 {
		document.querySelector('input[name="payment_method_nonce"]').value = payload.nonce;
		document.querySelector('input[id="button-pay"]').value = "Please wait...";
		document.querySelector('input[id="button-pay"]').disabled = true;
		document.querySelector('#loading').style.display = 'block';
		form.submit();
		counter++;
		document.querySelector('input[id="counter"]').value = counter;
		form.style.display = 'none';
	 }
  });

`


ARROW-1325: [R] Bootstrapping R bindings subproject

test CI (after rebasing master)

Codecov Report

Merging #2196 into master will decrease coverage by <.01%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master    #2196      +/-   ##
==========================================
- Coverage   86.52%   86.52%   -0.01%
==========================================
  Files         244      244
  Lines       41892    41892
==========================================
- Hits        36246    36245       -1
- Misses       5646     5647       +1

Impacted Files
Coverage Δ

python/pyarrow/tests/test_plasma.py
96.28% <ø> (ø)
:arrow_up:

cpp/src/arrow/util/thread-pool-test.cc
98.91% <0%> (-0.55%)
:arrow_down:

Continue to review full report at Codecov.

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

I wouldn't worry about the Plasma issue -- if you're getting it on master, please open a JIRA so that others can take a look

Yeah, don't worry about the Plasma issue (but feel free to push your fix, we can see if that fixes it -- see also https://github.com/apache/arrow/pull/2207)

Ok. I think I can fix the CI/plasma issue.
I will remove the plasma changes from this PR, and work on a fix from #2207 .

@wesm @Bernolt I'm interesting in keeping an eye on the progress of R bindings for Arrow. Is this PR the official place where that development is happening? Or is it just one possible proposal? Can you recommend how I can best track the progress for R?
Thanks!

Awesome! Thank you so much for the quick and detailed response.

as of soon, a chunk of my work time (~2days/week) will be dedicated to R/arrow work, so things should start moving soon.

Closing this PR since work is proceeding in the original one


[cocoon] Branch logic is including forks

https://pantheon.corp.google.com/datastore/entities;kind=Branch;ns=__$DEFAULT$__/edit;key=0%2F%257C6%2FBranch%257C35%2Fname%253Agodofredoc%2Fcocoon%2Ffilter_forks?project=flutter-dashboard
You can see here it's pulling forks, which isn't expected. It should only pull the supported list.
As a side note, I see this is only from godofredoc's repo, so they might have flutter-dashboard installed against their fork.

@CaseyHillers this should be closed?

@XilaiZhang is this fixed?

Oh forks have been fixed long long time ago :)


Improve console.group in Debug Console

vs

Can we introduce a concept like this into the debug protocol, and improve the display to support collapsible sections of logs? Do other runtimes have a concept like this?
It's now more visible when debugging vscode.

Our console for sure would gain much with simple object formatting / tree structure. Is the debug console rendering virtualized today?

The debug console is based on a tree, so this should be simple to implement.

@roblourens is the <Start group>/<End group> markup your adhoc invention and the DA is adding this to the stack trace or does this come from CDP?
I was able to achieve the following with the existing DAP:

So, the only issue is the array index that appears in front of the stack frames.

The  markup is just something I added so the group/groupEnd calls aren't totally invisible.
My original example isn't the best example, it's logging a single item as a one-off group. In your screenshot it looks like you're logging an array of strings, which is possible, but I don't think it's the right model for logging a group.

The group should be updated live as log messages are produced, where an array could only be logged once the group is closed. At least I don't think I can log an object, then change the object.
Groups should be expanded by default
They need a log location for each line
Nested groups should look good
Multiline group titles and grouped messages should look good

What are "good" examples for a dynamic group?
What do you mean by "They need a log location for each line"?

Code like
console.log('foo 1');
console.group('group 1');
console.log('foo 2');
console.log('foo 3');
console.group('group 2\nmultiline group\ntitle');

let i = 0;
const interval = setInterval(() => {
    console.log('bar ' + i++);
}, 1000);

setTimeout(() => {
    console.groupEnd();
    console.log('ended group 1');
    console.groupEnd();
    console.log('ended other group');
}, 4000)

Looks like this in chrome devtools

By "log location" I mean the location that we added for each log last month.

I just discovered console.group and console.groupCollapsed for using in my react native project and was surprised they didn't actually group with the collapsible arrow like logged objects do. PLEASE implement good visual collapsible grouping sooner rather than later. Although not "critical", it's such a simple thing and will improve my quality of dev-live :)

This is open for over one year... Any updates on this? Are we waiting for vscode to support Grouping?

@weinand I didn't want to just add a +1 (we would really like this to happen) so here's another great example of grouping (and the illegible results in Code): redux actions logged by redux-logger
Safari:

Chrome:

VS Code:
This is so verbose and unclear (especially with the additional whitespace after <Start group>) that it makes the output in Debug Console hard to read and much less useful than the same output in Chrome / Safari.

If you "would really like this to happen", you could submit a PR...

Haha, of course, but it's not an option with our current work pipeline. Plus, I wouldn't know where to start!
Am I mistaken in thinking you had something of a partial implementation, mentioned in https://github.com/Microsoft/vscode/issues/34981#issuecomment-342827987?

Same issue here. Debugger console is unusable with redux-logger.

@weinand how did you get the expand/collapse output in the debug console shown in https://github.com/microsoft/vscode/issues/34981#issuecomment-342827987?
As far as I can tell, OutputEvent only deals with strings? I have some tree-like errors I'd like to show here and that looks ideal.

@DanTup this is just an OutputEvent with a variablesReference for the exception object. VS Code will render the variable as an expandable object.

An additional aspect of groups is that after a group has been created with a specific name, later console output can be appended to the group if the same name is used.

@weinand, @isidorn: I might have confused you wrt names. Console groups are not named, they are synchronous grouping and there is no way to add to the group post-groupEnd. A better example would be a console group that was opened for a very long time (forever). Now all the console output should be getting into that group, so we need to dynamically populate a second level of the object tree.
Unrelated and FYI: console.context was introduced to address these synchronous shortcomings and be closer to the traditional logging systems. console.context('scope') returns an object that has the same API as a regular console. Application can create console.context for any component of theirs and have independent 'consoles' for those. We never told the world about it, so it was never adopted, but I guess we should have. In this case, everything within the context could be logged with the [scope]  prefix or within the group.
[server] Starting server on port 8000
[server] Port listening
[client] Connecting to port 8000

Not going to happen in november. Removing the Milestone and I leave it up to @weinand to assign a new milestone and to drive the changes in the DAP needed for this.

Adding this to the February milestone since it is on the plan.
Once we have this in the debug adapter protocol I can look into the implementation.

My summary of the node.js console.group documentation:

console.group(one or more labels):
Increases indentation of subsequent lines by two spaces.
If one or more labels are provided, those are printed first without the additional indentation.
If group is shown with a collapsable/expandable UI, it should be initially expanded.
console.groupCollapsed():
Increases indentation of subsequent lines by two spaces.
If group is shown with a collapsable/expandable UI, it should be initially collapsed.
console.groupEnd():
Decreases indentation of subsequent lines by two spaces.

My DAP proposal:
We add an optional enum property group  to the Output event with these values:

start: start a new group in expanded mode. Subsequent Output events are members of the group and should be shown indented. An optional output attribute becomes the name of the group and is not indented.
startCollapsed: start a new group in collapsed mode. Subsequent Output events are members of the group and should be shown indented (as soon as the group is expanded). An optional output attribute becomes the name of the group and is not indented.
end: end the current group and decreases the indentation of subsequent Output events.

/cc @isidorn @roblourens @connor4312

For further discussion of the group feature in the DAP, I've create this issue.

@isidorn I've added a group attribute to DAP and updated debugProtocol.d.ts in VS Code.

That proposal looks good to me.

I have pushed support for this.
@connor4312 @roblourens it would be very cool if the node2 or the javascript debugger could support console groups this milestone, then we could add a test plan item which uses node and not mock debug. Anyways let me know how it goes, so I prepare a test plan item in time. Thanks!

It would be a 💪 to do in the remaining days this iteration, but the test plan could literally for Rob or I to implement and verify grouping in js-debug. Andre has written a few items like that for me in the past 🙂

@connor4312 writing a test plan like that would work for me. Let's see the status on this tomorrow evening and I will update the test plan accordingly. Thanks!

Great idea to implement the protocol for js-debug as part of the test plan item ;-)


[TS] GIF support

Automatically generated task. Any updates to the title and body will be overwritten. Do NOT modify the labels.
TS implementation of #2871

GIFs already work in TS


Setting lua.format.lineWidth seems to no longer work in VS code

It just uses the default of 120 no matter what you put in

I haven't been able to reproduce this, is this still an issue in later versions of VSCode? Its working as intended in my version:
Version: 1.31.0-insider (user setup)
Commit: e02e44464c02d617b98d0a3265bf5efef7794436
Date: 2018-12-21T06:20:35.852Z
Electron: 3.0.10
Chrome: 66.0.3359.181
Node.js: 10.2.0
V8: 6.6.346.32
OS: Windows_NT x64 10.0.17763

Can someone please explain, how or where I set the lua.format.lineWidth = x
This extension is great, but documentation is a bit lacking :(
Thanks

@geoffw123 That is a setting that the extension supplies to VSCode.
Nowadays there is a GUI for settings (ctrl+comma), but there is also a tiny button to open the underlying settings.json file, and that's where you would see/change the "lua.format.lineWidth" value if you change it from the default 120.
In the GUI it appears as "Lua > Format: Line Width".

2024, lua.format.lineWidth is no longer a valid option.


Testnet faucet - recaptcha invalid domain for site key

@katomm
The preview/preprod update for the testnet faucet at link (https://testnets.cardano.org/cardano-testnet/tools/faucet)   seems to be unusable due to a reCaptcha error "ERROR for site owner:
Invalid domain for site key". The funds transfer will not proceed until this is fixed.
FIX: Maybe refer the steps here https://help.whmcs.com/m/troubleshooting/l/1189046-troubleshooting-recaptcha-invalid-domain-for-site-key
This needs to be fixed so that we can start using the faucet for preview/preprod networks.
Thanks in advance!

Might need to transfer it to
https://github.com/input-output-hk/cardano-documentation

oh. I'm sorry. I didn't realize that this was maintained by IOHK. Sure, will close it here, and I will check if this issue still exists and report it in the IOHK cardano-documentation issue tracker.


Sun graphic not shown when inbox is empty

The sun graphic does not appear when the inbox is empty. In theory this was fixed a few days ago (https://github.com/kallepersson/inboxtheme/issues/36).

Browser: Chrome 73
OS: Mac OS Mojave

@kallepersson Chrome extension version of this repo is 1.1.4, but chrome web store is 1.1.3.


Upgrade Constructr to 0.17.0

Closing this because this version apparently isn't properly released yet, only prepared https://github.com/hseeberger/constructr/commit/1742a0f164caf043f0e87ec0e12e534364184de5


Slot ID is wired to 00

Is this intentional?  It differs from Xavier is why I ask.

Can we use this as a board ID?  Would require a change on next board spin to slot ID 01.  Need something to use to determine if NIC_PWR_GOOD/LED is wired up.

if we spin the board we could possibly add jumper options

I think I drove some confusion.  The slot ID is hard wired to 00, which is fine - it just differs from Xavier.  Didn't know if anyone caught that.  Don't you want to test other slot IDs?
As far as a board rev ID goes, Jon's suggestion to add 3 pins is good.  The pins will be internally pulled down to ground, and a resistor can be used to pull one+ up to ID the board rev.

Agree.  if you also agree, can you close this.  otherwise let us know if we should change the slot id. thanks :)

The slot ID is not actually "wired" to 0.  There are NP resistors. Closing as resolved.


make HTTP errors more spec compliant

Does two things to the HTTP errors:

Remove ResponseError from Error() function. The value is still set, but when printing the error, this information should not need to be surfaced to the user. An alternative would be to keep it here and require the platforms that use this client to be responsible for not surfacing this information.
Changes how failure responses are unmarshaled. Originally, if the response body contained either "error" or "description" keys but their values were not strings, we would get an unmarshaling error and would lose all conformant keys. Now, by unmarshaling into a map[string]interface{}, we can selectively keep the conformant keys.

This way, we now get the correct spec behavior where uninterpretable error messages are simply ignored.

Coverage increased (+0.08%) to 91.294% when pulling 6f228e77f066b54c9928b8d3bbc347b1edef28a1 on kibbles-n-bytes:change_err into 9e257974620c1db53c4d3549fd9eddaa2d2d653e on pmorie:master.

Coverage increased (+0.08%) to 91.294% when pulling 95b3d9edd82acb2abf98f8a181b2002d829de831 on kibbles-n-bytes:change_err into 9e257974620c1db53c4d3549fd9eddaa2d2d653e on pmorie:master.

Hmm... Looking at it again, by changing the unmarshaling strategy, we should now be guaranteeing that that the ResponseError field will only be set by a truly invalid JSON object, not when it just doesn't match our expected response structure. That means it would contain useful information for the user should it be set, so it'd be fine to surface it still.
I'll change the message to include it again.

Coverage increased (+0.08%) to 91.294% when pulling f001973302e3e046ce05312290d0a9e7a4869738 on kibbles-n-bytes:change_err into 9e257974620c1db53c4d3549fd9eddaa2d2d653e on pmorie:master.

Coverage increased (+0.08%) to 91.294% when pulling f001973302e3e046ce05312290d0a9e7a4869738 on kibbles-n-bytes:change_err into 9e257974620c1db53c4d3549fd9eddaa2d2d653e on pmorie:master.

Done, PTAL.

Thanks for the patch!


Add option to notebook.formatOnSave not to add a new line at the end of each cell when formatting jupyter notebooks

When I format a jupyter notebook with black, it doesn't add a new line to the end of each cell. When I format a notebook with VSCode it does. I would prefer it if the notebook.formatonsave.enabled setting would behave the same as black for notebooks. Would it be possible to make this an option?

I think this has already been fixed (at least for me)


Add afterInit function after init and adaptHeight

This PR address the issue found here, regarding sudden flash prior to inner-slider loading.
We've have added this to Conde's internal fork and have seen great success!

Can't support props that deviate from slick.
Thanks for the PR though.


README badges

I'm adding badges to all the components' repos. Which style do folks prefer?

flat:
flat squared:
plastic:

I dig flat squared.

I vote Flat, but let's make sure the type color contrast is 508-compliant.

Welp, it fails with a contrast ratio of 2.12:1

JK the drop shadow on flat pushes it to 4.75:1. Going with flat.

You can customize the colors, too, right?

Only when creating custom badges with custom text. There may be other services that let you customize the colors but I'm cool with using flat.

Ah, okay. I'm cool with that if the shadow makes it compliant.


Release on ansible-galaxy

This may not happen due to no longer running my infrastructure under ansible. https://github.com/imsofi/infra


SecurityService should encrypt RemoteEvent/RemoteFunction names via GUID

Currently, remote's names are not encrypted, making them easier to target by exploiters. SecurityService should encrypt RemoteEvent/RemoteFunction names via HttpService:GenerateGUID() to make it harder for exploiters to determine which remote does what.
I'm not entirely sure how I'm going to pass remote names on to the client though - this will need some more planning before it can be implemented.

Closing, as SecurityService no longer exists.


Change type of EditorConfig.parent

Allows setting DocumentFragment (ShadowRoot) as a parent.

Maybe we should just use Node? That's where TypeScript puts appendChild, which is the only method that's used on parent.

I considered it, but document is also Node and that can't be used 🤷♂

Neither can TextNode or Attr, I guess. Yeah, the union type is more helpful for people to figure out what is expected here. Merging as-is.


Init.sh: download of evosuite dependencies times out (fixed)

Evosuite.org has been down for a while. For now, I have moved the evosuite dependencies to a different server. The latest revision works fine.
If you are working on one of the releases, the following change to init.sh will allow you to successfully initialize Defects4J:
--- a/init.sh
+++ b/init.sh
@@ -48,7 +48,7 @@ cd $BASE && wget -nv -N $MAJOR_URL/$MAJOR_ZIP \
 echo
 echo "Setting up EvoSuite ... "
 EVOSUITE_VERSION="0.2.0"
-EVOSUITE_URL="http://www.evosuite.org/files"
+EVOSUITE_URL="http://people.cs.umass.edu/~rjust/defects4j/download"
 EVOSUITE_JAR="evosuite-${EVOSUITE_VERSION}.jar"
 EVOSUITE_RT_JAR="evosuite-standalone-runtime-${EVOSUITE_VERSION}.jar"
 cd $DIR_LIB_GEN && [ ! -f $EVOSUITE_JAR ] \

Evosuite.org seems to be back online.

Seems like this is resolved, i'll close it.


How to access the refresh token?

The AuthenticationResult at AuthenticationCallback success only gives the access token. How do we get the refresh token using this implementation?

MSAL doesn't expose the refresh token today. Can you speak to why your apps need it? The calling pattern assumes you can ask for a fresh access token anytime your app needs to call an API. Does this not cover a scenario for you?


Corrected the command line build statements

Hi Kevin,
We prefer the maven profile way. It's less verbose and the result is the same.
Thanks for your interest

When I try to run the commands as-is, I get a "missing profile" error (hidden in the logs) and all projects are built. Can you please let me know if you get the same behavior, or if you are able to successfully build one project at a time?
Thanks,
Kevin

Sorry, you are completely right! That was the previous way we packaged.
Thanks


Bugfix bedtools closest: Do not segfault when b file is empty.

Only try to print a null record when the record type is known.
In case of an empty b file, the record type can not be determined.
This fixes "Segmentation fault with closestBed":
https://github.com/arq5x/bedtools2/issues/486

Fantastic, nice catch. Thanks!


KOGITO-4595 Downgrade Quarkus 1.11

https://github.com/kiegroup/kogito-runtimes/pull/1115
https://github.com/kiegroup/kogito-apps/pull/692
https://github.com/kiegroup/kogito-examples/pull/601
https://issues.redhat.com/browse/KOGITO-4595

The Linux check has failed. Please check the logs.

🤔
stderr: remote: Repository not found.
fatal: repository 'https://github.com/evacchi/optaplanner.git/' not found

jenkins please retest this

The Linux check has failed. Please check the logs.

The Linux check has failed. Please check the logs.

jenkins please retest this

The Linux check has failed. Please check the logs.

jenkins please retest this

The Linux check has failed. Please check the logs.

Jenkins please retest this

The Linux check is successful.


CI

@peter-mlabs We need to set CACHIX_KEY. I don't have it. Once we have that we'll have a simple CI that checks if it runs.


[fix][broker] Fix unsubscribe non-durable subscription error

Fixes #20947
Motivation
Documentation

[ ] doc
[ ] doc-required
[x] doc-not-needed
[ ] doc-complete

Codecov Report

Merging #21099 (e309d97) into master (dab5b2f) will increase coverage by 0.13%.
Report is 10 commits behind head on master.
The diff coverage is 83.33%.

@@             Coverage Diff              @@
##             master   #21099      +/-   ##
============================================
+ Coverage     73.03%   73.16%   +0.13%
- Complexity    32363    32409      +46
============================================
  Files          1887     1887
  Lines        139860   139899      +39
  Branches      15384    15394      +10
============================================
+ Hits         102140   102359     +219
+ Misses        29634    29453     -181
- Partials       8086     8087       +1

Flag
Coverage Δ

inttests
24.10% <22.22%> (?)

systests
25.00% <11.11%> (-0.11%)
:arrow_down:

unittests
72.46% <83.33%> (+0.05%)
:arrow_up:

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

Files Changed
Coverage Δ

...va/org/apache/pulsar/client/impl/ConsumerImpl.java
77.17% <50.00%> (+0.63%)
:arrow_up:

...he/pulsar/broker/resources/NamespaceResources.java
80.28% <75.00%> (-0.44%)
:arrow_down:

...sar/broker/service/persistent/PersistentTopic.java
80.43% <75.00%> (+0.98%)
:arrow_up:

...pache/pulsar/broker/resources/PulsarResources.java
83.72% <100.00%> (ø)

...g/apache/pulsar/client/impl/MessageIdAdvUtils.java
84.61% <100.00%> (+1.28%)
:arrow_up:

...va/org/apache/pulsar/client/impl/ProducerImpl.java
83.88% <100.00%> (+0.60%)
:arrow_up:

...ache/pulsar/client/impl/UnAckedMessageTracker.java
91.59% <100.00%> (+0.91%)
:arrow_up:

... and 83 files with indirect coverage changes


Doc error

Changed Line 94- You can't use Hooks inside of a class component, but you can definitely mix classes and function components with Hooks in a single tree.
to
"You can't use Hooks inside a class component, but you can definitely mix classes and function components with Hooks in a single tree. "

Thank you!


Update Dockerfile

I was executing all the steps on the Quickstart guide, but the docker container immediatly exited. I think it did, because the entrypoint is bash and I was able to run the container by commenting out the bash entrypoint and uncommenting the line above. I don't know what this entrypoint does exactly, but it worked for me.

There was an unintentional change in the Dockerfile that should never have been merged. It has already been fixed, so this PR is not needed anymore.
Thanks for trying out QLever, let us know if you encounter any further issues.


git_keep_meta is undocumented

I'm setting it to true for a project of mine to avoid the "fatal: not a git repository" message with every bundler invocation.
I am not sure about the tradeoffs of this option.

can you run capistrano with flag --trace and then attach logs here?

What do you need to know? Maybe this suffices:
$ bundle exec rails console -e staging
fatal: Not a git repository (or any of the parent directories): .git
fatal: Not a git repository (or any of the parent directories): .git
fatal: Not a git repository (or any of the parent directories): .git
Loading staging environment (Rails 4.1.3)

I have a gem bundled as a submodule (using the :path option), which causes these messages when .git has been cleansed (probably in the submodule, did not check that).

And I confirm that it goes away with git_keep_meta set to true.

I'm not using rails, I'm using pure capistrano3.
I can not answer why bundle exec rails console -e staging fails =(
If you can run this command in debug mode you should see a stack trace.

Nothing is actually failing, sorry for being unclear. I get message on stdout, that's all.
The message comes when bundler is used, even without capistrano. Minimal reproduction:
bundle gem new
cd new/
bundle gem submodule
echo "gem 'submodule', path: 'submodule'" >> Gemfile
bundle
bundle console # fine
rm -rf .git
bundle console # error message

I reproduced your problem on clean new virtual machine WITHOUT capistrano-git-submodule-strategy.
I think this problem is not in git_keep_meta, bundler requires existing .git folder.

The message does NOT come from the submoduled gem's gemspec (the generated gemspec contains spec.files         = git ls-files -z.split("\x0"), replacing that did not help)

@schnittchen problem still reproducing ?

It looks like you added documentation in 8dc9ad28d279d8c8a1233dd6992581b1e6a6b159 :)
I just upgraded to 0.1.19. The mentioned warning messages during deploy disappeared a while ago when I started using the the option.


Adding language learning resources & reformatting learning section

This is starting to be pretty big. I worry the document ends up being a google search result with so much information that you need a search tool to go through it :)
Perhaps we have to scope its content as such:

ESDC And GoC Alignment materials (must be materials that can be used by IT as instruments to demonstrate alignment. I.e. whatever IT does, it must be within the range of those materials)
Materials that the IT Strategy team uses. If we don't use it, we don't list it. These include worthwhile book materials, online information, tutorials, software (if they are not worthwhile, we don't list them perhaps?)
Crowd Sourcing info. This would relate to this milestone that @Moro-Code is working on so the community has the ability to add on to it

Aye, it is getting sizeable. But we were asked for a page with the content we've added (tools folks can use and educational stuff folks can do awhile off network)
After this merge, I agree. I'll create an issue for splitting this out into numerous documents. Maybe even a new section on the site?


GetCurrentClassLogger based on Environment.StackTrace

Tested with UWP, and it also works. Credits @stil

Guess #2140 should be merged to CoreCLR-branch (Together with rest of 4.4.10)

FYI, i'm thinking to re-do the .NET standard implementation and then no netstandard 1 (but only netstandard 2). It's waiting on #2154
See also #2152

Looks like NetStandard 2 will make things very easy. Just have to wait 3 months time for it to be released.

@stil & @304NotModified Added your suggested solution.

Thanks! It should also resolve #2162
I'd hate it if we needed to wait next couple of months for upcoming .NET Standard 2.0.
However it seems it doesn't pass some tests.

Yes you're right. Would be great to have this working!

Very strange the unit test in the CoreClr-branch was different from the main-branch. Now updated the CoreClr-branch.

@Stil Please try the NuGet-package and see if it works on your machine:
https://ci.appveyor.com/project/nlog/nlog/build/5.0.0-beta5394/artifacts

@snakefoot I installed it into my projects and it seems to work. I haven't encountered any problems yet. I'll start using it and I hope @304NotModified finds time to review this PR and merge it. New beta release would also be nice.

great work!


Results of compound assignment expressions

The 'Result of assignment operators' section is vague about whether it is also applicable to compound assignment operators; in fact, it does but with a caveat.
Compound assignment operators return the object specified by the left operand, which has been already modified by the operation despite the operators are apparently placed after variables. In contrast, postfix operators result in the pre-modification values of the variables. Such difference may cause confusions to those who are careless like me.
int i1 = 2;
int postfix_op = i1++; // == 2

int i2 = 2;
int assign_op = i2 += 1; // == 3

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

ID: 1fb7e11f-cd0d-5e4d-fcf2-6b1060b7586c
Version Independent ID: 90046753-8233-1a30-09d8-126f8350b3fc
Content: Assignment operators
Content Source: docs/cpp/assignment-operators.md
Product: visual-cpp
Technology: cpp-language
GitHub Login: @TylerMSFT
Microsoft Alias: twhitney

We are in the process of moving away from GitHub issues for C++ documentation (we still accept pull requests). For details, please see Announcing a new way to give feedback on Microsoft Learn This issue will be closed after it is copied to an internal system.


How to reference the type “SourceryRuntime.Protocol” using Swift templates?

If I just write Protocol, the compiler will complain of ambiguity between SourceryRuntime.Protocol and ObjectiveC.Protocol:
/private/var/folders/qr/m95k0bgx4cldcf37ftvqst900000gn/T/SwiftTemplate/1.6.0/Sources/SourceryRuntime/Protocol.swift:15:33: note: found this candidate
@objcMembers public final class Protocol: Type {
                                ^
ObjectiveC.Protocol:1:14: note: found this candidate
public class Protocol {
             ^

If I use SourceryRuntime.Protocol directly, Swift will treat the SourceryRuntime in SourceryRuntime.Protocol as a type (but it is a module) and .Protocol as referencing the protocol metatype of SourceryRuntime.
One solution would be doing
import class SourceryRuntime.Protocol

But I wonder if this is another way to do this? If not, should we consider adding a handy type alias for this type?

it's just regular swift, so normal swift limitation but there is a typealias already that we use SourceryProtocol


Hero Animation Improvements

Would be good and useful if the Hero animation also move some Object with him. Like, if I'm moving an Text widget, they should move the String inside that Text widget, so that on the other side Hero just show the same data, without re-populate with a new data.

Hey Flutter team, any update on this?


FR: Could Dispatch include some of the actual GDoc selected text in the new Task?

Is your feature request related to a problem? Please describe.
As an incident commander, when assigning and reviewing tasks the default description is not super helpful. Eg, looking at one of my recent incidents, 9 out of 10 of the tasks were just "Description: Description: @". One could of course re-type salient details into the comment, but that's extra time and redundant information.
Describe the solution you'd like
I think a better experience would be for the incident description in Dispatch (shown automatically slack and in /dispatch-list-tasks) would be to include highlighted text.
So if I have a bullet in the doc that's * Gina: contact vendor for further details on foo, highlight it, make a comment, and assign it to her (by her email addr), then how dispatch would summarize it would be:
Description: Gina: contact vendor for further details on foo
Creator: Patrick Thomas
Assignees: Gina Fullname

Describe alternatives you've considered
No a showstopper, so "leave as is" works.
Additional context
FWIW, I checked several other incidents and they seem to have the same issue, but one (hit me up internally, I'll point you to it) seems like it was showing this behavior. Not sure what to make of it.

I believe you're describing a bug that was fixed in the latest release by PR (https://github.com/Netflix/dispatch/pull/1546) please let me know if you continue to see this issue.

Yep, I think you're right. Darn it, even read those release notes and didn't see it.
Thanks folks!