Drop aiohttp support

Closes #1490

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

1 of 3 (33.33%)  changed or added relevant lines in 1 file are covered.
97 unchanged lines in 10 files lost coverage.
Overall coverage decreased (-3.5%) to 93.264%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

connexion/apis/abstract.py
1
3
33.33%

Files with Coverage Reduction
New Missed Lines
%

connexion/operations/openapi.py
1
95.29%

connexion/cli.py
2
95.4%

connexion/options.py
2
77.42%

connexion/decorators/response.py
5
89.09%

connexion/apis/abstract.py
6
88.83%

connexion/operations/secure.py
6
73.12%

connexion/jsonifier.py
9
70.0%

connexion/security/security_handler_factory.py
10
81.73%

connexion/decorators/decorator.py
11
55.56%

connexion/security/async_security_handler_factory.py
45
19.67%

Totals

Change from base Build<PHONE_NUMBER>:
-3.5%

Covered Lines:
2520

Relevant Lines:
2702

💛  - Coveralls


Revert "shell: optimize history storage a bit"

This reverts commit f7c4fe677823ae47ce8aa87af8514ef4100a6172.
This commit introduces a hang during z_shell_history_put().
Reverting it fixes #44904.

FYI compliance checks are failing due to a typo in the original code that was here. If anyone wants me to fix that, I can, it will just be a dirty revert after that.

@carlescufi - what's the Zephyr policy on how quickly to revert changes that cause new bugs?  We haven't done any investigation to determine root cause, other than to identify the PR that caused the problem.

@carlescufi - what's the Zephyr policy on how quickly to revert changes
that cause new bugs?  We haven't done any investigation to determine root
cause, other than to identify the PR that caused the problem.

Please don't be so hasty.
Taking into account the Easter holiday, I didn't have a chance to look at
the issue since you filed this bug.

@carlescufi - what's the Zephyr policy on how quickly to revert changes
that cause new bugs?  We haven't done any investigation to determine root
cause, other than to identify the PR that caused the problem.

Please don't be so hasty.
Taking into account the Easter holiday, I didn't have a chance to look at the issue since you filed this bug.

We're holding off on merging this @npitre, let us know when you have a fix.

The actual fix is in PR #44960

A forward fix just landed in PR #44960.


Retinal Isomerase GO:00004744

RHEA:24124 GO:0004744
RHEA:24124, now obsolete. It was created for EC <IP_ADDRESS>, which has been deleted by the Enzyme Commission
Deleted entry: retinal isomerase. Now known to be catalysed by a pathway involving EC <IP_ADDRESS>0, NADP-retinol dehydrogenase; EC <IP_ADDRESS>, phosphatidylcholine--retinol O-acyltransferase; EC <IP_ADDRESS>, retinoid isomerohydrolase; and EC <IP_ADDRESS>5, 11-cis-retinol dehydrogenase.
[EC <IP_ADDRESS> created 1961, modified 1976, deleted 2011]
Best to obsolete this term
Check annotations to see which EC they can be mapped to.
Anne: I noticed that GO:0004744 is used in UniProt
https://www.uniprot.org/uniprot/?query=goa:("retinal isomerase activity [4744]")
I will send a mail to UniProt curators to updated the corresponding entries

Annotation review ticket: https://github.com/geneontology/go-annotation/issues/4136

+[Term]
+id: GO:0106429
+name: 11-cis-retinol dehydrogenase
+namespace: molecular_function
+def: "Catalysis of the reaction:11-cis-retinol-[retinal-binding-protein] + NAD(+) <=> 11-cis-retinal-[retinol-binding-protein] + NADH." [RHEA:55668]
+xref: EC:<IP_ADDRESS>5
+xref: RHEA:55668
+is_a: GO:0016616 ! oxidoreductase activity, acting on the CH-OH group of donors, NAD or NADP as acceptor
+property_value: creation_date 2022-07-15T17:58:20Z xsd:dateTime
+created_by: hjd

+[Term]
+id: GO:0106434
+name: retinal isomerization
+namespace: biological_process
+def: "The reactions involved in isomerization of all trans to all cis retnal." [PMID:21447403]
+comment: This process term replaces 'retinal isomerase' because  the Enzyme Commission has determined that the reaction is known to be catalyzed by a pathway involving multiple steps, catalized by  EC:<IP_ADDRESS>0, EC:<IP_ADDRESS>, EC:<IP_ADDRESS>, and EC:<IP_ADDRESS>5.
+is_a: GO:0042574 ! retinal metabolic process
+property_value: creation_date 2022-09-28T16:44:26Z xsd:dateTime
+property_value: term_tracker_item "" xsd:anyURI
+created_by: hjd

All done


Compiler error E0283

I have a struct that is trying to derive Deserialize and I'm getting this error.
error[E0283]: type annotations required: cannot resolve `V: serde::Deserialize<'de>`
  --> src/session.rs:20:28
   |
20 | #[derive(Clone, Serialize, Deserialize, Eq, PartialEq)]
   |                            ^^^^^^^^^^^
   |
   = note: required by `serde::Deserialize`

I don't see how to give the derive macro more information. Am I a missing something?

Can you show your struct definition? Is your struct generic?

This sounds like https://github.com/Manishearth/rust-clippy/issues/1689. Get rid of the trait bounds on your struct.
- struct Heartsucker<T: Deserialize> { /* ... */ }
+ struct Heartsucker<T> { /* ... */ }

As a general rule, structs should not have trait bounds.

Ah, that fixed it. Was that always the behavior?

This was always the behavior but it is a Rust bug so it may be solved in the future. https://github.com/rust-lang/rust/issues/41617


Skip c_char_def on OpenBSD

Fixes https://github.com/rust-lang/libc/issues/4209

it fixes the build on OpenBSD.


Add many to one support

Basic many to one support.

Reduce table size by keeping key on the "many" table.


Log window font is not monospaced correctly

Version and Platform (required):

Binary Ninja Version: 3.4.4093-dev
OS: macOS
OS Version: 12.4
CPU Architecture: M1

Bug Description:
The text in the log window does not have the same horizontal advance for all characters. This means that lines that were designed to line up in a monospaced grid do not actually line up.
Steps To Reproduce:

Open the python console and enter print("M"*80+"A\n"+" "*80+"A")
Observe the 'A's line up in the console
Now enter log_info("M"*80+"A\n"+" "*80+"A") and look at the log window
Observe the 'A's do not line up

Expected Behavior:
I expected the clearly monospace font to be monospaced.
Screenshots:

Additional Information:
This is with default font settings

Fixed in 3.5.4281-dev

Duplicate of #1674


Fixes to recording

Description
Fixed some issues with recording:

Licode would not record streams that didn't have both video and audio. This was fixed by @zafergurel in #1265. Made minor changes for the linter so all credits to him.
When stopping the recording, a wrong ID was passed resulting in the stream continued being recorded. This was fixed by @nvazquezg in #1565
When checking if the stream had audio the condition if (getAudioSinkSSRC() == 0) was always returning false even if the stream had audio. In this if block the codec was set to PCMU so if other codec was used the resulting audio is corrupted.

Also removed the toggleRecording button in the basic example and changed it for individual buttons so the user can select the stream to record.
[] It needs and includes Unit Tests
Changes in Client or Server public APIs
No changes
[] It includes documentation for these changes in /doc.
No changes

Thanks @Arri98 for giving credit and for your contribution.


net/http: deadlock in TestMissingStatusNoPanic on release-branch.go1.15 branch

2505e26/windows-amd64-longtest
1035bb9/windows-amd64-longtest
b35c38a/windows-amd64-longtest
636a6d14/windows-amd64-longtest
May be related to ##42942
CC CC @alexbrainman @fraenkel @neild @bradfitz @golang/release

@cagedmantis I think this is the wrong test. While the error occurs on TestCancelRquestWhenSharing after a few seconds, the moire obvious one is TestMissingStatusNoPanice which has been stuck for 14m.
goroutine 17971 [chan receive, 14 minutes]:
net/http_test.TestMissingStatusNoPanic(0xc00060e000)
	C:/workdir/go/src/net/http/transport_test.go:5320 +0x450

https://storage.googleapis.com/go-build-log/2ba9aff9/windows-amd64-longtest_d50badd6.log (a TryBot on CL 319710)

Also https://storage.googleapis.com/go-build-log/5200af43/windows-amd64-longtest_e8416262.log (a TryBot on CL 319709).

All of the failures are due to TestMissingStatusNoPanic so far. My fix above will just expose the real issue which we currently aren't seeing.

On the plus side, we now see the actual issue(s) in Go 1.15 failures.

I see at least one of the issues and may be the actual problem....
Notice its all using the same port.
TestTransportPersistConnReadLoopEOF is failing but not sending a bogus conn to the connc channel.

The above is incorrect, the testcase is fine.
The error being reported happens when the system is underload and usually out of resources. One can tweak the TCP settings to reduce the failure.
There is also this https://go.googlesource.com/benchmarks/+/refs/heads/master/http/http.go#42
I guess something got fixed/improved in 1.16+.

GopherBot is a little eager (due to #29599 not being resolved yet).

GopherBot is a little fiesty (due to #21312 not being resolved yet).

The root cause of the failures is port exhaustion, which hasn't been addressed, but the deadlock in TestMissingStatusNoPanic should be fixed.
I haven't been able to reproduce the port exhaustion failures manually after running go test net a number of times on a windows-amd64-longtest builder today. Possibly I'm not taking the right steps to reproduce the problem, or possibly whatever was causing that builder to run out of ports has been addressed. I'm going to tentatively assume the latter unless these failures show up again.

I think it might be the combination of things. One test that didn't finish, holding up others than haven't done their cleanup.
It was a bit weird that it was only occurring in one release.


Add an option to disable html escaping

Currently every text node as well as every attribute value is html-escaped. This is done to avoid XSS attacks. Sometimes it's needed to skip the escaping step to actually inject some html. There should be a way to do this.
Here is the proposed syntax:
let header = "
TEXT
";
html! {

{ header | safe }

}

I'm not yet sure about this precise syntax though. One positive thing about it is that it mirrors what Jinja2 (and everything else that borrows its syntax) does, but on the other hand it's ambiguous with the bit-wise or operator.

Maybe a pair of braces can be use instead?  {{ head }}


soul admin注册nacos 异常

can  you modify title to english and update code?

please use english, and discribe more detail.

Hi @zl530380 , Thanks for your report. Could you please change the title to English?


Minor User Experience Improve

Add COLOR to print messages
Remove useless prints(in a dirty way, need to hook up with --verbose)
Support KD233 and Dan Autodownload autodetect
Catch Errors and give messages
Add a retry Limits and regreeting/resetting for every cycle
Other feature I forgot I added.

Nice work!


"Enable acrylic material" is not working

Windows Terminal version
1.17.11461.0
Windows build number
10.0.19045.3271
Other Software
No response
Steps to reproduce
Toggle on "Enable acrylic material" at Appearance>Transparency menu
Expected Behavior
A translucent texture should be applied to the background of the window.
Actual Behavior
The translucent texture is not applied to the background of the window, neither for Prompt command nor PowerShell, Azure Cloud Shell and VS 2022 windows . Tested pretty much everything by enabling it with diferent themes, tools, oppacity values, on both default and specific profiles and even edited settings.json and added a "useAcrylic": true line for different color schemes and still nothing. Does this feature actually work or is it a bug with the installation on my computer?

Hey, for me the "Enable acrylic material" setting only worked while editing the settings on the specific profiles. Initially, the default profile didn't work but after messing with the values in the PowerShell profile and resetting it, the default profile only worked then. I don't know why that happened though because I have never changed the settings in the PowerShell profile before.

Interesting, thanks for the feedback. For me the option is only applied after rebooting, maybe after logging off and in again, but I haven't tested it yet. No big deal, sooner or later it will work.

Interesting, thanks for the feedback. For me the option is only applied after rebooting, maybe after logging off and in again, but I haven't tested it yet. No big deal, sooner or later it will work.

For me the problem was that I had the power saver on, I turn it off and it work again.


Cloning a subsite using export/import --new_url affects the source subsite domain

Describe the bug

When cloning a subsite in a multisite install using export/import, the source subsite domain is updated to the target subsite URL.
Steps to Reproduce

wp mu-migration export all --blog_id=8 /tmp/blog_template.zip --themes --plugins --uploads
wp mu-migration import all /tmp/blog_template.zip --new_url="https://mynew.url"
blog_id=8 gets https://mynew.url as domain in DB

Expected behavior

source subsite is not affected by the creation of a new subsite when using export/import with --new_url

@ppiotr3k Are you importing the exported site in the same subsite? If so this is very likely an unhandled edge case, since when the search-replace takes place to replace the URL, it affects both subsites given they both have the same URL/Domain at the time search-replace is run since search-replace is performed with the --network flag it catches both sites. You'd have the same issue if you didn't provide --new_url and tried to run search-replace afterwards with the --network flag.
We can probably solve this by restricting the search-replace to the imported subsite and handling the search-replace of the necessary global tables manually.

@nicholasio I confirm this is an export/import in the same WordPress multisite deployment. The export/import features are used as a cloning operation, providing a new URL for the target during import.
Also, I haven't noticed previously, but indeed the URL is replaced in the source site in other places as well, with the target site URL, as explained in your possible cause analysis :clap:

Confirmed this is an issue for me as well.

For others following, my workaround was to use wp-cli to change the site address to the "new site address" prior to exporting. Once the export is done, the script changes the site address back. Then, upon importing, you don't need to specify a new address.
I think I've mostly finished with the bash script, if anyone is interested lmk and I'll finish it up and provide.

I think I've mostly finished with the bash script, if anyone is interested lmk and I'll finish it up and provide.

Hey @jmtornetta, still got that script somewhere? :)


Macros not working in remote development

VSCode Version: 1.35.0-insider
OS Version: macOS 10.14.5

Steps to Reproduce

Create a macro in settings.json
Create a keyboard shortcut to trigger that macro
Connect to remote host
Trigger macro

You should see an error that says macros.TheNewMacroYouMade not found

Does this issue occur when all extensions are disabled?: Yes
I found this when trying to trigger the macro I use to return to Normal mode in vscodevim.  However I tested it using native functionality.
Here is the macro I set up from my settings.json
"macros": {
        "macroTest": [
            "workbench.action.terminal.new"
        ]
    },

Then  in  keybindings.json I have this:
{
    "key": "shift+alt+cmd+s",
    "command": "macros.macroTest"
},

Would be really great to have macros back.
Thanks!!!

Create a macro in settings.json

VS Code doesn't support macros, what extension are you using?

Oh no!!  I set this up so long ago I forgot it was an extension.
Sorry for bothering you with a non issue.  I got it to work.
Thanks for the quick response


only EventSet::hup() doesn't generate events on OSX

I wrote this test recently and it passed on Linux but deadlocked on OSX. It looks like the kqueue implementation doesn't like listening with just EventSet::hup()?
extern crate mio;

use mio::{EventLoop, Handler, Token, EventSet, PollOpt};
use mio::tcp::{TcpListener, TcpStream};

fn main() {
    struct H { listener: TcpListener }

    impl Handler for H {
        type Timeout = ();
        type Message = ();

        fn ready(&mut self, event_loop: &mut EventLoop<Self>, token: Token,
                 _events: EventSet) {
            if token == Token(1) {
                let s = self.listener.accept().unwrap().unwrap();
                event_loop.register_opt(&s, Token(3), EventSet::all(),
                                        PollOpt::edge()).unwrap();
                drop(s);
            } else if token == Token(2) {
                event_loop.shutdown();
            }
        }
    }

    let mut e = EventLoop::new().unwrap();
    let l = TcpListener::bind(&"<IP_ADDRESS>:0".parse().unwrap()).unwrap();
    let s = TcpStream::connect(&l.local_addr().unwrap()).unwrap();

    e.register_opt(&l, Token(1), EventSet::readable(), PollOpt::edge()).unwrap();
    e.register_opt(&s, Token(2), EventSet::hup(), PollOpt::edge()).unwrap();

    let mut h = H { listener: l };
    e.run(&mut h).unwrap();
}

The issue has to do with the way mio translates registration calls into kqueue filters. Each (re)register call from mio creates two new filters kqueue: one for read (EVFILT_READ) and one for write (EVFILT_WRITE). Based on the interests from the registration call, the read and write filters are enabled (EV_ENABLE) or disabled (EV_DISABLE). In the case above, there is no read or write interest and both filters are disabled. Thus, no event will ever make it to the handler.
This really boils down to the fact that epoll treats hup as a separate event and kqueue treats hup as part of read/write.

See https://github.com/carllerche/mio/blob/629141abde2b8bafcd5f89e67a9e426f442983c6/src/sys/unix/kqueue.rs#L41 and https://github.com/carllerche/mio/blob/629141abde2b8bafcd5f89e67a9e426f442983c6/src/sys/unix/kqueue.rs#L63 for relevant source code sections.

I think the right way is to require read or write interest in the cross platform API. So, w/o importing OS specific extensions, it should be impossible to register hup interest w/o read or write.

Fixed on dev by adding a check for readable or writable in the interest set.


fix: default parsers conf location wrong on windows

both linux and windows have it at the same location
backport of #1656 to release-4.5

4.5.5 is building here: https://github.com/kube-logging/logging-operator/actions/runs/7817654410


!modules command doesn't appear to work

For some reason, the !modules command doesn't appear to work.
It feels as if this happened just as the automatic camera wall was added.  Maybe it has something to do with that?

this was removed intentionally, it might be worth adding it back in, if there's still demand for it to be used

Why was it removed?  I've seen cases where people do a !uncv to unclaim a module and I don't know the number of the module that was unclaimed.  I want to view the remaining few modules by using the !module command.

I don't think there's demand for this command to return, so I'm going to close this.


text-width does not wrap long URLs and links

Newsboat version (copy from newsboat -v): newsboat 2.10.1-r2.10.1-28-gad51
Steps to reproduce the issue:

Set text-width to 80

Open an article with a URL or link longer than 80 characters

I see that this was actually an implementation to fix https://github.com/akrennmair/newsbeuter/issues/282

Yes, this was conscious decision on my part, and I still stand behind it. But maybe i just didn't consider enough (noone chimed in for a discussion in that issue, as you can see.) Can you describe why you want the URLs wrapped at text-width?

I had just assumed this was an error when I reported it. But since I see that the documentation for text-width at newsboat/doc/configcommands.dsv does mention this behavior and nothing is broken by it, it should be fine to close this issue.


Call to a member function getBody() on null

in localhost it's great smoothly running
but when uploaded to hosting an error like this appears

please advise to fix it
thank you

Mengalami error yang sama. Mohon petunjuk solusinya.
Terima kasih.


Provided handle function name convenction

This is similar to the other PR -- nothing wrong with naming it this way but not particularly better. I don't want to break other people's use of this module by renaming properties without a sound reason. Thank you though!


Implement Add repository modal

[x] The title of my pull request is a short description of the changes

@chad11111


feat(Android) Add onControlsVisiblityChange

Summary
A callback event for when the Android Exoplayer controls show and hide.
Issue request: https://github.com/TheWidlarzGroup/react-native-video/issues/2686
Motivation
Since the Android video player LegacyControls have limited controls (play, pause, rewind, forward, scrubber, etc., and do not provide an interface for changing the Captions or Audio of a stream, or implementing additional buttons), having an event allows a consumer class to overlay buttons / information at the same time as controls are shown.
Changes
Adds onControlsVisibilityChange to VideoEventEmitter
Adds a visibility listener to the playerControlView in ReactExoplayerview to trigger the new event
Not available for iOS, due to no public methods for monitoring native control visibility
Test plan

@freeboub, any idea how I can fix this ESLint error?  I've tried a few approaches

@ashnfb  did you try : "yarn lint" ?

I added 1 minor comment, but code looks good ! Thank you

@KrzysztofMoch @freeboub, corrections made.  I haven't been involved in this project in a while, may I ask what steps are next?

If all check will be green, and @freeboub will approve it (as he requested change) - we can go with it and merge it


Api cameras

Is there a possibility to detect which is the active camera?
I need to start with the rear camera, and for now I am running await AgoraRtcEngine.switchCamera (); when starting and ending the call.
But I think it is not the best solution.

Our SDK uses the front camera by default, you can use switchCamera now.
We will upgrade the flutter SDK in Q2, then you can use setCameraCapturerConfiguration

finished


EA SPORTS FIFA Football Demo [PCSB00084]

App summary

App name: EA SPORTS FIFA Football Demo
App serial: PCSB00084
App version: 1.00

Vita3K summary

Version: v0.1.6
Build number: 2957-4343481c
Commit hash: https://github.com/Vita3K/Vita3K/commit/4343481
CPU backend: Dynamic
GPU backend: both

Test environment summary

Tested by: User01
OS: Windows 10
CPU: AMD Ryzen 5 2600 Six-Core Processor
GPU: NVIDIA GeForce RTX 3060 Ti/PCIe/SSE2
RAM: 8 GB

Issues

Screenshots

Log
vita3k.log
Recommended labels

Nothing

Si me gusta fifa

Vita3K summary

Version: v0.1.6
Build number: 3069
Commit hash: https://github.com/vita3k/vita3k/commit/73fc1be1
CPU backend: Dynarmic

Test environment summary

Tested by: othmanead
OS: Windows 11
CPU: Intel i5-10300H
GPU: RTX 3060 mobile
RAM: 16 GB

Issue:
Game plays with audio and graphical issues
Screenshots:

Log
vita3k.log
Recommended labels

Ingame+
Audio quality
Graphics bugs


sumbission

This is my submission to the SEG ML contest.
Please let me know if I have done the Pull Request wrong.
Cheers,
fvf

Hello! And welcome to the contest. Very happy to see another person/team.
This one scored 0.534, a decent first effort. I hope you get a chance to improve it.


Get comments

How do I get more comments than just the teaser?

Hello, that's a feature that is coming very soon to YouTubeKit. In the meantime you can get them using another repository or by trying to create your own request as explained in the README (I'll be glad to answer your questions if you do it this way).

Added in b5ad965.


Really?

"Enables the command extensions... " For real?  How about explaining what command extensions are or even a link (anchor) to what command extensions are.  This is beyond absurd.  Whoever wrote this must have intended to be aleck.

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

ID: 2c1319ea-20fb-948f-dc98-622d106439ca
Version Independent ID: d8e5fe91-62d8-1565-f625-dbde41df0dec
Content: setlocal
Content Source: WindowsServerDocs/administration/windows-commands/setlocal.md
Product: windows-server
Technology: manage-windows-commands
GitHub Login: @coreyp-at-msft
Microsoft Alias: coreyp

Please note that this makes sense to most people learning in detail how the operating system works in a command line environment.
Regarding your suggestion to explain what command extensions are; it is a topic unto itself and can have different meanings to different people, depending on their scenario. In some scenarios, it can be interpreted to mean any file you can open by entering the file name (without the extension) to open that file in its current default application. In other scenarios, it can mean any script file you can run like a command within the setlocal / endlocal environment.
I recommend reading more about the topic by doing a web search and see if you can find the information you require to understand the concept:

https://www.bing.com/search?q=command+extensions
https://www.google.com/search?newwindow=1&q=command+extensions

If your experience still urges you to say that the documentation needs to be improved, please post again with suggestions on how to add proper, factual information and suggestions on where it fits within the already existing information on the page.
Thank you for wanting to improve the documentation. Appreciated.

my comment stands.  your reply adds no value.

Command Extensions is a mode of the cmd command interpreter that changes the behavior of variable expansion and of many internal commands. These changes are described in the documentation of each command. The cmd documentation topic has a list of affected commands, although it fails to mention the date and time commands. The assoc, color, and ftype commands are not available at all if Command Extensions are disabled.

Thank you for pointing out the details which could be added to improve the article. I notice that the commands date and time have got their own pages, but I also appreciate your point that setlocal affects those commands and that the details should be mentioned.

To be clear, I suggest these changes:

In the setlocal topic, refer to the cmd topic for information about Command Extensions. Place this above the "Testing for command extensions in batch files" remark.
In the cmd topic, mention that setlocal is another way to enable or disable Command Extensions. Place this somewhere near "You can disable command extensions for a particular process by using /e:off."
In the cmd topic, add date and time to the list of commands affected by Command Extensions.
In the color topic, add a note saying that the command is not available from PowerShell, but the BackgroundColor and ForegroundColor properties of $ExecutionContext.Host.UI.RawUI can be set instead.
In the assoc, color, and ftype topics, say that cmd does not recognize these commands if Command Extensions are disabled. Place these near the notes that say these commands are not available from PowerShell.

Command Extensions is a mode of the cmd command interpreter that changes the behavior of variable expansion and of many internal commands.

By "internal commands", I mean commands that run within the cmd process, as opposed to external commands that run as separate processes and have their own .exe files.

There you go. PR #4461 is now available for review, comments, and suggestions. Feel free to join in.

Thank you.  Very much appreciated.  I am of the generation that has been coding since the 80's, and I never saw the concept of command extensions until around the turn of teh new minnenium, and it seems there is confusion among the creators of teh concept about just what it means and exactly what are its impact -- what functionality is added, what functionality is removed, and there is terribly consistency in the industry on which built-in functions behave differently with commadn extensions -- terrible, terrible, genuinely awful consistency.  Thank you for any and all effort to try to better clarify Microsoft's intent for command extensions.  Perhaps some day soon we will even see greater consistency in built-ins conforming to the rules set for command extensions by Microsoft.  I am, however, thankful that MS hasnt alltogther abandoned classic command line and batch and forced migration to exclusively PowerShell.  While PowerSheel certainly brings greater capability,I'd almost rather resort to Tcl than deal with teh exact syntax of the myriad of commands in PS thoug in modern times, I do realize we have IntelliSense/Code-completion and a vast information super highway to misguide us for hours and weeks on end.  Before creating artifical intelligence, I'd like to see us preserve what little human intelligence and rebuild more of the human intelligence that has quickly faded in the past century and at an exponential rate as we see in the madness in all contexts and all cultures over the past decade.

Thank you for the nice words. As a former computer support agent and onsite technician, I am mostly driven by troubleshooting and general software, installation and maintenance knowledge. I see what you are saying, but as a hobby linguist and in the light of cynicism, it is far too easy to imagine that the future will be dictated by money, might, or the masses of mainstream people. We can only hope for the better.

Perhaps some day soon we will even see greater consistency in built-ins conforming to the rules set for command extensions by Microsoft.

I'm not sure what rules you refer to. Anyway, because cmd is in maintenance mode, I don't think Microsoft will make any changes to the syntax of its built-in commands.
Surprisingly, command extensions are currently at version 2:
C:\>IF CMDEXTVERSION 1 (ECHO yes) ELSE (ECHO no)
yes

C:\>IF CMDEXTVERSION 2 (ECHO yes) ELSE (ECHO no)
yes

C:\>IF CMDEXTVERSION 3 (ECHO yes) ELSE (ECHO no)
no

SETLOCAL ENABLEEXTENSIONS and CMD /E:ON don't seem to provide a way to select a different version of command extensions. I wonder when version 2 was added, and how it differs from version 1.

SETLOCAL ENABLEEXTENSIONS and CMD /E:ON don't seem to provide a way to select a different version of command extensions. I wonder when version 2 was added, and how it differs from version 1.

I tried searching the web for clues to an answer to your question. This is one of the hits I found:
https://github.com/imatix/gsl/blob/c67ce0b00947c86a55458587782f04de55f8a092/src/c.bat#L21-L30
:-  Start CMD.EXE version detection
verify other 2>nul
setlocal enableextensions
if errorlevel 0 goto __cmd_ok
echo %0: This command requires command extensions version 2 in CMD.EXE.
echo %0: Please use a supported system (Windows 2000 or newer).
exit
:__cmd_ok
setlocal enabledelayedexpansion
:-  End CMD.EXE version detection

I know this is fairly anecdotal, but maybe it could help a bit with any future research you might want to do for that question.

Very interesting.  Thank you.

Sorry for the lack of changes. The page was updated externally, making me lose the overview of what to keep and what to drop from my own changes. I ended up choosing to close the PR and delete the fork I had created, making it necessary to start over from scratch. I plan to make a new PR when I get my next day off.

Note to myself: https://sive.rs/zipit

(ref. #5409 )

We hugely appreciate your contribution, feedback, and support to our technical documentation.
We’ve phased out GitHub Issues on this repository as the feedback mechanism for content and have replaced it with a new feedback system. For more information see, Content User Feedback. As part of this rollout, we are migrating your issue where our team will use your comments to improve the documentation experience. Unfortunately, this does mean you won't be able to access this issue after the migration is complete.
All issues will be migrated by Thursday 29th August 2024.
Once again, thank you for your valuable feedback, we hope you’ll continue to contribute using our new feedback system.


WSL does not get focus when launching from taskbar

Please fill out the below information:

Your Windows build number:  (Type ver at a Windows Command Prompt)
17134

What you're doing and what's happening:  (Copy&paste specific commands and their output, or include screen shots)
If an instance of WSL is running, to run another copy: right-click on the running instance in Taskbar, and select the instance name (i.e., Ubuntu).

What's wrong / what should be happening instead:
In all other apps (including cmd.exe), the new instance immediately takes focus. With WSL, explorer.exe retains focus and requires either alt-tab to the WSL window or clicking in the WSL window.

Strace of the failing command, if applicable:  (If some_command is failing, then run strace -o some_command.strace -f some_command some_args, and link the contents of some_command.strace in a gist here).
N/A

For WSL launch issues, please collect detailed logs.
Don't imagine logs are needed as I've repro'd this easily on builds from 16299 all the way through 17134.

See our contributing instructions for assistance.

Seeing the same issue on windows 10.0.16299.551

In all other apps (including cmd.exe), the new instance immediately takes focus.

So if CMD is able to focus but WSL is not, I would say check https://github.com/Microsoft/Console/issues/. I think this was addressed in Insiders but it wouldn't hurt to file it over there for verification.

thanks @DarthSpock for the suggestion. I just created issue 233 in the Console project.

Please see the related issues Microsoft/WSL#2597 and Microsoft/WSL#3123, which were replaced by Microsoft/Console#179

The "console" tag is at best loose on this one. It isn't terminal emulation related (console proper) and it certainly isn't Linux kernel ABI related (WSL proper). The label just qualifies the redirect since Microsoft/console#179 is open.


Dataset 3 phase 2 Task 75- 92

@AnshitaSinghBais 1) Can you resolve the conflicts? 2) Can you add the original tasks in this PR. 3) Can you rename the task variants as taskxxx.1_yyyy, taskxxx.2_yyyy, taskxxx.3_yyyy and so on. For e.g. task547.1_spl_translation_entk_en.json, task547.2_spl_translation_entk_en.json, task547.3_spl_translation_entk_en.json and so on. Do this for all the tasks in this PR

@AnshitaSinghBais 1) Can you resolve the conflicts? 2) Can you add the original tasks in this PR. 3) Can you rename the task variants as taskxxx.1_yyyy, taskxxx.2_yyyy, taskxxx.3_yyyy and so on. For e.g. task547.1_spl_translation_entk_en.json, task547.2_spl_translation_entk_en.json, task547.3_spl_translation_entk_en.json and so on. Do this for all the tasks in this PR

Hi, I have made the asked changes except the 1st one. Could you please help me with the conflicts. I am unable to resolve them.
Thank you

@AnshitaSinghBais 1) Can you resolve the conflicts? 2) Can you add the original tasks in this PR. 3) Can you rename the task variants as taskxxx.1_yyyy, taskxxx.2_yyyy, taskxxx.3_yyyy and so on. For e.g. task547.1_spl_translation_entk_en.json, task547.2_spl_translation_entk_en.json, task547.3_spl_translation_entk_en.json and so on. Do this for all the tasks in this PR

Hi, I have made the asked changes except the 1st one. Could you please help me with the conflicts. I am unable to resolve them. Thank you

Can you add the original tasks to the same folder as the variant tasks? All at one place

@AnshitaSinghBais Can you check why the checks are failing and resolve it?

@AnshitaSinghBais Can you raise this PR again? There were issues when I merged. So I have reverted the merge. Can you raise the PR for phase 2 again?

Hello,
My checks are failing because of the original task 74. It is being
downloaded in the rtf format and saving it in the json format is changing
all its formatting which is causing the checks to fail. Should I remove the
original task 74 ?
On Sun, Nov 7, 2021 at 10:13 AM Ravsehaj singh puri <
@.***> wrote:

@AnshitaSinghBais
https://urldefense.com/v3/__https://github.com/AnshitaSinghBais__;!!IKRxdwAv5BmarQ!L-K1pZLQW5iqPRik0MhQ0hGmCDcQwxt6-nzZIma4i9g7_Iod283oW4apZrGsOA$
Can you check why the checks are failing and resolve it?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://urldefense.com/v3/__https://github.com/swarooprm/CSE576HW/pull/157*issuecomment-962647732__;Iw!!IKRxdwAv5BmarQ!L-K1pZLQW5iqPRik0MhQ0hGmCDcQwxt6-nzZIma4i9g7_Iod283oW4ZLNsQ2mw$,
or unsubscribe
https://urldefense.com/v3/__https://github.com/notifications/unsubscribe-auth/AVJIKHB7DSDV27WCCYUXGKTUK2XVBANCNFSM5GP2ANPA__;!!IKRxdwAv5BmarQ!L-K1pZLQW5iqPRik0MhQ0hGmCDcQwxt6-nzZIma4i9g7_Iod283oW4bmTjHxbg$
.
Triage notifications on the go with GitHub Mobile for iOS
https://urldefense.com/v3/__https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675__;!!IKRxdwAv5BmarQ!L-K1pZLQW5iqPRik0MhQ0hGmCDcQwxt6-nzZIma4i9g7_Iod283oW4ar9_Q0bg$
or Android
https://urldefense.com/v3/__https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign*3Dnotification-email*26utm_medium*3Demail*26utm_source*3Dgithub__;JSUlJSU!!IKRxdwAv5BmarQ!L-K1pZLQW5iqPRik0MhQ0hGmCDcQwxt6-nzZIma4i9g7_Iod283oW4Zmd-I-HQ$.

Hello,
I tried to create a new PR, but it says that there are no changes to
compare.
[image: image.png]
On Sun, Nov 7, 2021 at 11:34 AM Ravsehaj singh puri <
@.***> wrote:

@AnshitaSinghBais
https://urldefense.com/v3/__https://github.com/AnshitaSinghBais__;!!IKRxdwAv5BmarQ!M3M9FERACjieAqEBjj-sntcPWBTg97UGEn0ay3c8fi7fYhSFTNXk9q_3ntGv1g$
Can you raise this PR again? There were issues when I merged. So I have
reverted the merge. Can you raise the PR for phase 2 again?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://urldefense.com/v3/__https://github.com/swarooprm/CSE576HW/pull/157*issuecomment-962659816__;Iw!!IKRxdwAv5BmarQ!M3M9FERACjieAqEBjj-sntcPWBTg97UGEn0ay3c8fi7fYhSFTNXk9q_1a49MGw$,
or unsubscribe
https://urldefense.com/v3/__https://github.com/notifications/unsubscribe-auth/AVJIKHAZKGGG3RUIOZNBHB3UK3BD3ANCNFSM5GP2ANPA__;!!IKRxdwAv5BmarQ!M3M9FERACjieAqEBjj-sntcPWBTg97UGEn0ay3c8fi7fYhSFTNXk9q_53bE6LA$
.
Triage notifications on the go with GitHub Mobile for iOS
https://urldefense.com/v3/__https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675__;!!IKRxdwAv5BmarQ!M3M9FERACjieAqEBjj-sntcPWBTg97UGEn0ay3c8fi7fYhSFTNXk9q9twpOXxg$
or Android
https://urldefense.com/v3/__https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign*3Dnotification-email*26utm_medium*3Demail*26utm_source*3Dgithub__;JSUlJSU!!IKRxdwAv5BmarQ!M3M9FERACjieAqEBjj-sntcPWBTg97UGEn0ay3c8fi7fYhSFTNXk9q8f64RqhA$.


feat: allow additionalCommandArgs to be passed to barman-cloud-wal-archive

Closes #4357
Adds the additionalCommandArgs property to WalBackupConfiguration, allowing command line args to be passed to barman-cloud-wal-archive.

Hi @pselliotc, thanks for the contribution. Could you please update docs/src/backup_barmanobjectstore.md to include the new parameter?

/test feature_type=backup-restore test_level=4

/test limit=local

/test limit=local


example of getting the consensus of multiple sequences?

I have many similar sequences of different lengths and need to get the consensus of them.
From the docs, the seq module should work for this, but any example provided?

I've found exmaples in the codes. Thanks.

After reading the docs, I tried the multi type for getting the consensus, but it just did not produce what I want.Dis I missed something? My code is as below:
        m, err := multi.NewMulti("test",                 []seq.Sequence{                         linear.NewSeq("seq1", []alphabet.Letter("AAAATTTT"), alphabet.DNA),                         linear.NewSeq("seq2", []alphabet.Letter("GGGAAAAT"), alphabet.DNA),                         linear.NewSeq("seq3", []alphabet.Letter("ATTTTCCC"), alphabet.DNA),                 },                 seq.DefaultConsensus)         if err != nil {                 fmt.Println(err)                 return         }         fmt.Printf("%- s\n\n%-s\n\n\n", m, m.Consensus(false))
it gives:
AAAATTTT
GGGAAAAT
ATTTTCCC
nnnannnt
but what I want is:
AAAATTTT
GGGAAAAT
ATTTTCCC

If you want an alignment, you need to provide the alignment. The concensus of a multiseq is purely that, just the consensus of the sequences that you give it. For something that does all of this, see this code

It seems that the code relies on external mafft/muscle for multiple alignment, is biogo able to do this by itself?

No. Multiple sequence alignment is non-trivial and it doesn't seem like a worthwhile investment of time given that there are good tools that already do it.

I see. Thanks.


Document multi-process discovery

Document new properties and changes in the properties.
Use a section for limitations. For example, it seems that in docker it's very difficult (or impossible) to instrument a generic executable with other executables, as you need access to the pid/network of that single container (then you can't listen for the others).

Done!


System Audio Mode Request should be 0x7d

case 0x70:
#ifdef DEBUG
					/* System Audio Mode Request */
					printf_P(PSTR("<sys audio>\n"));
#endif /* DEBUG */
					_delay_ms(13);
					set_system_audio_mode(ADDRESS, 0x0f, 1);
					break;

should (most likely) be
case 0x7d:
#ifdef DEBUG
					/* System Audio Mode Request */
					printf_P(PSTR("<sys audio>\n"));
#endif /* DEBUG */
					_delay_ms(13);
					set_system_audio_mode(ADDRESS, 0x0f, 1);
					break;

Thanks for testing this and responding in the commit.  I'll see if my TV is happy with 0x72 in response to 0x7d, but I may #ifdef this anyway because it seems to differ from the spec.  I'll leave the issue open until then.

So according to this (for example): http://www.andrewncarr.com/hdmi/hdmi-cec-codes.xls,
If device does not <Feature Abort> the following message:
   <Give System Audio Status>
 It shall be able to send the message(s):
   <Set System Audio Mode>, <System Audio Mode Status>

So perhaps Apple's implementation is in line with the spec?

I have also tested this with a Philips TV
After this exchange:
-> 05:7d audio mode?
<- 5f:7e audio status

the TV does not send any volume requests over the CEC bus.
But after this (responding with 72 to 7d):
-> 05:7d audio mode?
<- 5f:72 set audio

The TV remote V+/V- button presses are relayed to the CEC bus.
So the Philips implementation behaves identically to the Apple implementation.

You are right, Thomas.
0x70  is responded by 0x72
0x7d  is responded by 0x7e
I'm closing the issue - thanks for clarifying.


changelog: move registry query fixes to the right place

I guess I was sleepy on that day. These three entries should go to other sections.

Thanks!  I pushed a small change to add what looked like a missing period.
@bors r+

:pushpin: Commit 810c04ff50be5963ad070b89a40d8b25066ac09e has been approved by ehuss
It is now in the queue for this repository.

:hourglass: Testing commit 810c04ff50be5963ad070b89a40d8b25066ac09e with merge fa3fa852ab2970af0d0ae05a866d4f3899455c19...

:sunny: Test successful - checks-actions
Approved by: ehuss
Pushing fa3fa852ab2970af0d0ae05a866d4f3899455c19 to master...


[Enhancement] about the add favorite button

Description
it will look good if replaced with a heart icon and filled red when added as favorite
Screenshots

Checklist

[X] I have checked the existing issues.
[X] I follow Contributing Guidelines of this project.

I believe this issue is being addressed in ticket #48.


kubernetes + weave 1.1 guide fails with "No such image"

See https://github.com/weaveworks/weave/issues/1392

@abuehrle you reported this as a failure of weave expose, but the logs indicate that things went wrong before then:

weaveproxy.service: Start-pre operation timed out. Terminating.

Also, there is

Sep 05 22:39:21 kube-01 weave[1265]: Repository weaveworks/weaveexec already being pulled by another client. Waiting.

So I suspect you've hit that well known docker issue where sometimes it will be stuck forever downloading some images.

The issue is to do with missing a explicit docker pull, so and as units get launched in parallel, it hits this .  It was originally in place, but then got remove for a different reason that no longer applies. I have add weave setup in ce739df565cdbfb18cc6e19c4779dac134fceca8.


Valid resource names are rejected when creating Workload resource types

Setup

Rancher version: 2.6.8

Describe the bug

When attempting to create Workload types such as deployment, daemonset and statefulset will show a validation error when their name contains a .
When trying the same via kubectl or yaml the resource is successfully created

To Reproduce

Workloads --> Deployment --> Create

Result
See screenshot
Expected Result

Name validation does not fail for valid names

Screenshots

Additional context

Kube docs state a number of specs applicable to resource name, including two that don't allow .. However it does not state which resources these apply to.
The resource docs all lead to the generic metadata object applicable to many types
https://kubernetes.io/docs/reference/generated/kubernetes-api/v1.22/#deployment-v1-apps --> https://kubernetes.io/docs/reference/generated/kubernetes-api/v1.22/#objectmeta-v1-meta
To resolve this we should manually test which resources are accepted by the kube cli and make sure the UI follows suit

We should first confirm the cli doesn't accept any old junk

Test Environment:
Rancher version: v2.7-head eab28dd
Rancher cluster type: single-node docker install
Docker version: 20.10

Testing:
Tested this issue with the following steps:

Create a deployment with the name "valid.name"

Result - Pass
User is now able to create deployment using "." in the name


Transaction search

https://zpl.io/2GOg03E
https://zpl.io/boMkRYv
https://zpl.io/bJGNzO3

@Miolin make changes according to the design
https://app.zeplin.io/project/5ba9be48f029d520c456f395/screen/5cc2c67bd3744f2de5428dfc

@Miolin Transaction search should search for the first 6 characters in the address

Doesn't search by contact name

Design:

Actual:

Transaction ID and addresses are displayed with a transaction comment icon

To forbid to enter multiple rows(a column)

Must separate the search string and the transaction string by changing the color alternation. For example, the search string is gray, and the first line with the transaction was black.

@Miolin please apply this design to the masternet and testnet.
https://app.zeplin.io/project/5ba9be48f029d520c456f395/screen/5d6d265639bbe3139a1203e2

https://app.zeplin.io/project/5ba9be48f029d520c456f395/screen/5d6d265450db6d73805e9cdf

Make it so that when you open the focus was on the line "Search"
Please increase the area of tapping "x"

checked


arguments to custom setitem are sometimes coerced to string

It appears that if you provide a non-string argument to a custom __setitem__ the results work fine if the code is a literal -- but it appears they are coerced to strings inside of loops.
So in my case I had a class which expected tuple keys:

Class Example:

     def __init___():
          pass

     def __setitem__(self, key, val):
           print ("got>", key, "<>", val, "<")

a = Example()
a[1,0] = 2
#got><object object><>2<
for x in range (3):
     addr = tuple ([1,x])
     a[addr] = x
# no printout, but a now has keys like string `"1,0"`, `"1,1"` etc

It's a limitation because javascript object keys are strings.

but it does catch the tuple key as a python-style non-string key.  I have to correct that example , I realize I typed it wrong -- the setitem code is not triggered at all in the loop

OK, i've isolated it down to the difference between a case where the brackets contain a comma -- which means they call the tuple constructor -- and cases where they arguments are passed as tuples:
#this works
test[2,1] = 1.0
test[2,2] = 1.0
test[2,3] = 1.0

# this works
for r in range(3):
    test[r,1] = 0.5

# this does not work
for xy in ([1,0], [1,1], [2,2]):
    test[xy] = 2.0

The compiled output looks like this:
		test.__setitem__ ([2, 1], 1.0);
		test.__setitem__ ([2, 2], 1.0);
		test.__setitem__ ([2, 3], 1.0);
		for (var r = 0; r < 3; r++) {
			test.__setitem__ ([r, 1], 0.5);
		}
		for (var xy of tuple ([list ([1, 0]), list ([1, 1]), list ([2, 2])])) {
			test [xy] = 2.0;
		}

So it must be an issue with the way the compiler decides when to try __setitem__ and when to use plain JS brackets

Sure -- I think it won't be too bad, the interventions are in only three places I believe.
Incidentally I think the fast overloads is ready to be merged when you get a chance,

oops, did not mean to close this -- you'll probably want it open when you have time to look at it.


[New component] Accordion

Added new Component called Accordion under components/Accordion
Added a demo in the dashbord view

This is a PRO featured component, so the PR will be closed


Update to toggle documentation.

Docs Build status updates of commit 184623c:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

microsoft-edge/devtools-guide-chromium/open/index.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel


YJDH-376 | Refactor and simplify School

Description :sparkles:
Remove deleted schools (i.e. the ones with non-null deleted_at values).
Remove School.deleted_at as unnecessary, all Schools are active now.
Remove SchoolQuerySet as unnecessary.
Replace validate_school use with validate_name, also retrospectively.
Remove validate_school validator as unnecessary.
Remove unique constraint from School.name field.
Update related tests.
Refs YJDH-376
Issues :bug:
YJDH-376
Testing :alembic:
Screenshots :camera_flash:
Additional notes :spiral_notepad:

Codecov Report

Merging #577 (57a3f07) into develop (fcb2b51) will decrease coverage by 1.38%.
The diff coverage is 100.00%.

@@             Coverage Diff             @@
##           develop     #577      +/-   ##
===========================================
- Coverage    95.67%   94.28%   -1.39%
===========================================
  Files          230       97     -133
  Lines         9040     3345    -5695
===========================================
- Hits          8649     3154    -5495
+ Misses         391      191     -200

Impacted Files
Coverage Δ

...ons/0016_add_youth_application_field_validation.py
100.00% <ø> (ø)

backend/kesaseteli/applications/api/v1/views.py
92.74% <100.00%> (ø)

...li/applications/migrations/0017_refactor_school.py
100.00% <100.00%> (ø)

backend/kesaseteli/applications/models.py
97.63% <100.00%> (-0.18%)
:arrow_down:

.../applications/tests/test_application_validation.py
99.15% <100.00%> (+0.78%)
:arrow_up:

.../kesaseteli/applications/tests/test_schools_api.py
100.00% <100.00%> (ø)

...nd/shared/shared/service_bus/service_bus_client.py
33.33% <0.00%> (-57.58%)
:arrow_down:

backend/shared/shared/yrtti/yrtti_client.py
37.50% <0.00%> (-53.13%)
:arrow_down:

backend/benefit/helsinkibenefit/urls.py

backend/benefit/companies/tests/test_models.py

... and 132 more

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 fcb2b51...57a3f07. Read the comment docs.

browser-test menee läpi paikallisesti:
frontend/kesaseteli/youth> yarn browser-test
yarn run v1.22.17
$ testcafe 'chrome --ignore-certificate-errors --ignore-urlfetcher-cert-requests --window-size="1249,720"' testcafe.dotenv.js browser-tests/
 Running tests in:
 - Chrome 96.0.4664.110 / Windows 10

 Frontpage
 √ can fill up youth application

 Frontpage
 √ can change to languages

 2 passed (36s)
Done in 55.95s.

browser-test:ci menee läpi paikallisesti:
frontend/kesaseteli/youth> yarn browser-test:ci
yarn run v1.22.17
$ testcafe 'chrome:headless --disable-gpu --window-size="1249,720" --ignore-certificate-errors-spki-list="8sg/cl7YabrOFqSqH+Bu0e+P27Av33gWgi8Lq28DW1I=,gJt+wt/T3afCRkxtMMSjXcl/99sgzWc2kk1c1PC9tG0=,zrQI2/1q8i2SRPmMZ1sMntIkG+lMW0legPFokDo3nrY="' --screenshots path=report,fullPage=true --video report --reporter spec,custom,html:report/index.html testcafe.dotenv.js browser-tests/
/
 Running tests in:
 - Chrome 96.0.4664.110 / Windows 10

 Frontpage
 √ can fill up youth application

 Frontpage
 √ can change to languages

 2 passed (23s)
Done in 41.98s.

The tests pass now, they were fixed with the merging of PR #583 and rebasing on develop after that.


Matplotlib boxplot bug

BoxPlot raises an error when put in a Layout currently, small fix to ensure it doesn't. Also snuck in a style parameter for HistogramPlot.

Looks good. Merging.


Consider removing abstract toBuilder

Gax uses a pattern of abstract <B extends Builder<SettingsT, B>> B toBuilder(); in a base class, expecting a subclass to override it with a concrete return type.
This causes warnings like:
BigtableTableAdminStubSettings.java:[764,43] build() in com.google.cloud.bigtable.admin.v2.stub.BigtableTableAdminStubSettings.Builder overrides <B>build() in com.google.api.gax.rpc.StubSettings.Builder
  return type requires unchecked conversion from com.google.cloud.bigtable.admin.v2.stub.BigtableTableAdminStubSettings to com.google.api.gax.rpc.StubSettings<B>

It might be cleaner remove the abstract toBuilder() method from the base classes and trust that the subclasses will implement that method

@garrettjonesgoogle can you take a look at this?

It seems like it should be possible to fix the parameterization to get this to work...

I don't think this is possible. The base class's toBuilder return type would have to include a self referential parameter that includes its builder class, which will be an inner class that hasn't been defined yet.

@andreamlin can you add a priority to this?

This is likely an API breaking change and a major version update. Is it worth that?

I don’t think it’s worth doing a major version bump for this, but I would argue that if a major version were to happen, this should be included.
Maybe this can be included in a v2 hotlist?

We missed the opportunity for v2 — is this still worth doing at the next breaking change?

I'm not sure when we'll have the next major version bump, but yes, it would have to be filed under that.


Added the ability to open in an external editor

Just added a checkbox to toggle whether files should be opened in an external editor or in xcode (Lets me get a workflow closer to the old condensed layout from Xcode 3)

Thanks @fjolnir for your contribution! It finally went in ^^' Much appreciated!


linux-7 is dead.

Is there an existing issue for this?

[X] I have searched the existing infra issues

Type of Request
bug
Infrastructure Environment
LUCI, Github, Cocoon scheduler, Autosubmit, etc...
What is happening?
linux-7 is dead.

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

linux-7 is up and running fine, I can login remotely from my laptop.  Will look into more details.
flutter@flutter-devicelab-linux-7:~$ uptime
09:24:07 up 15:21,  1 user,  load average: 0.08, 0.02, 0.01

Rebooting machine to clear the error from Healthcheck monitoring website.


Repo: update readme

This PR gives the repository's readme a much-needed facelift, borrowing largely from OIP 1.
To minimize duplication or confusion with openst-contracts, but to retain the connection, the description of OpenST has been moved to the bottom of the readme, as it is effectively outside of everything else described in the readme, including contributing to the repo.
Resolves #125.

LGTM 👍


Telldus Live: Device without methods is a binary sensor

Description:
Telldus Live reports binary sensors as devices without methods. This PR solves a problem where binary sensors is registered as switches in Home Assistant.
Pull request in home-assistant.github.io with documentation (if applicable): home-assistant/home-assistant.github.io#<home-assistant.github.io PR number goes here>

Hello @rasmusbe,
When attempting to inspect the commits of your pull request for CLA signature status among all authors we encountered commit(s) which were not linked to a GitHub account, thus not allowing us to determine their status(es).
The commits that are missing a linked GitHub account are the following:

ec6ad0826712a59c2d0d969fb4bd5ee6869dcc0f - This commit has something that looks like an email address (git@rasmusbengtsson.se). Maybe try linking that to GitHub?.

Unfortunately, we are unable to accept this pull request until this situation is corrected.
Here are your options:

If you had an email address set for the commit that simply wasn't linked to your GitHub account you can link that email now and it will retroactively apply to your commits. The simplest way to do this is to click the link to one of the above commits and look for a blue question mark in a blue circle in the top left. Hovering over that bubble will show you what email address you used. Clicking on that button will take you to your email address settings on GitHub. Just add the email address on that page and you're all set. GitHub has more information about this option in their help center.

If you didn't use an email address at all, it was an invalid email, or it's one you can't link to your GitHub, you will need to change the authorship information of the commit and your global Git settings so this doesn't happen again going forward. GitHub provides some great instructions on how to change your authorship information in their help center.

If you only made a single commit you should be able to rungit commit --amend --author="Author Name<EMAIL_ADDRESS>
(substituting Author Name and<EMAIL_ADDRESS>for your actual information) to set the authorship information.
If you made more than one commit and the commit with the missing authorship information is not the most recent one you have two options:

You can re-create all commits missing authorship information. This is going to be the easiest solution for developers that aren't extremely confident in their Git and command line skills.
You can use this script that GitHub provides to rewrite history. Please note: this should be used only if you are very confident in your abilities and understand its impacts.

Whichever method you choose, I will come by to re-check the pull request once you push the fixes to this branch.

We apologize for this inconvenience, especially since it usually bites new contributors to Home Assistant. We hope you understand the need for us to protect ourselves and the great community we all have built legally. The best thing to come out of this is that you only need to fix this once and it benefits the entire Home Assistant and GitHub community.
Thanks, I look forward to checking this PR again soon! :heart:

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

@rasmusbe - btw is this how a door sensor presents itself? (I would like to test some stuff and I don't have access to my door sensor at the moment, since it is at another location)

@molobrakos my guess is that it depends on the sensor but most likely yes. I was seeing the problem with a motion sensor.
If you see the door sensor under devices at live.telldus.com my guess is that it will become a binary sensor in HASS

Ok, I just added this https://github.com/molobrakos/tellduslive/blob/udp_listener/tellduslive.py#L257 for instant updates for sensors on the local network and I need to test it since I only have temp/humidity sensors available here. You will need latest version from here as well: https://github.com/molobrakos/tellsticknet
Start with e.g. PYTHONPATH=<path to tellsticknet> ./tellduslive list -vv -r
Let me know if you feel like/have time for testing. Maybe we can continue discussing in some other channel then (gitter.im/discord?)

@molobrakos Sounds great! I'm Rasmus#8791 at Discord


Need "onClick" functionality on "Links"

Is your feature request related to a problem? Please describe.
I want to show a popup on the click on "Link" label.
Describe the solution you'd like
I should be able to pass "onClick" in the links object. And this onClick should be called if I click anywhere on the link
Describe alternatives you've considered
Just make the "Link" label clickable!

I would like a solution to this as well. Simply add a new function return from schema creation to catch link click.


baseUrl and paths setting in jsconfig.json is not working with .vue files

Hi all
I'm using VS Code with Vue.js, I'm trying to setup VS Code to be able to display valid/invalid path of the files
Using following setting I can get correct valid/invalid sign (red underline below invalid path) but with .vue files, I only get invalid signs (both relative path and specified paths in "compilerOptions"."paths"
How to setup jsconfig.json or any other settings to make this working? Please guide
Thanks
Path
|-jsconfig.json
+-src
|    +-components
|    |    |-Foo.js
|    |    |-Foo.vue
|    |    |-entry.js

jsconfig.json
{
    "compilerOptions": {
        "checkJs": true,
        "module": "es2015",
        "target": "es6",
        "baseUrl": ".",
        "paths": {
            "Components": [ "./src/components" ]
        }
    }
}

entry.js
// no errors, Foo.js can be resolved
import F1 from 'Components/Foo'
// no errors, Foo.js can be resolved
import F2 from './Foo'
// errors, no Fo.js anyway
import F3 from 'Components/Fo'
// errors Foo.vue cannot be resolved
import F4 from 'Components/Foo.vue'
// errors Foo.vue cannot be resolved
import F5 from './Foo.vue'

tsc version: 2.3.4

VSCode Version: 1.12.2
OS Version: Windows 10
tsc Version: 2.3.4

I have issues opening up handlebars files with hbs extension. Is  there a way to configure code to respect extensions when specified. If you have two files with same name but different extensions (js and jsx) it always prefers js one.

We don't support vue out of the box. Please try the Vetur extension: https://marketplace.visualstudio.com/items?itemName=octref.vetur
The extension may not support this scenario yet but I know they have been working to have better integration with jsconfig files

@mjbvz
I meant, it's not about IntelliSense, I just want to import .vue files without VS Code displaying errors (red underline)
Is there some way to setup jsconfig.json to allow that ?
Thanks

If you don't want those errors to show in your js files, your best bet is to disable the building javascript validation by setting: "javascript.validate.enable": false
We are looking into better vue/js integration in the future with TypeScript server plugins: https://github.com/angelozerr/tsserver-plugins


decode filename when match an urlencode filename from contentDispositionFilename

pdf.js cannot correctly display urlencode contentDispositionFilename, so I add the decode logic of contentDispositionFilename in extractFilenameFromHeader.

First of all, this needs a better and more readable commit message then just "support urlencode contentDispositionFilename", since it's a bit difficult to decipher when looking at only the commit message (e.g. at the git command line) and not the code itself.
Second of all, please make sure to follow https://github.com/mozilla/pdf.js/wiki/Squashing-Commits.

/botio unittest

Thank you for improving this!


adding transition & animation to social media icons to adventure awaits - frontend project

Add transition and animation effects to the adventure awaits project's social media icons to increase the website's interactive behavior.

@pranjay-poddar kindly assign me this issue under GSSOC'23


update package version in conf.py and version.txt

Summary
Details and comments
Fixes #467

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-2.6%) to 61.465%

Totals

Change from base Build<PHONE_NUMBER>:
-2.6%

Covered Lines:
3005

Relevant Lines:
4889

💛  - Coveralls

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-2.6%) to 61.465%

Totals

Change from base Build<PHONE_NUMBER>:
-2.6%

Covered Lines:
3005

Relevant Lines:
4889

💛  - Coveralls


How do I retain sheet state on orientation change?

I'd like to retain sheet state is it peeked or expanded on orientation change. I've implement this with peek state. But I wonder how to deal with expandSheet() method. If I call it with showSheetWithView before it result will be just peek state. But call to expandSheet() without showSheetWithView() will lead to NullPointerException: Attempt to invoke virtual method 'void android.view.View.setLayerType(int, android.graphics.Paint)' on a null object reference. So how do I use expandSheet() properly?

The next release will support a BottomSheet fragment, which should achieve this effect.


always set locality_weighted_lb_config

Locality weights are only considered when locality weighted load balancing is configured. These weights are ignored otherwise.

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: hzxuzhonghu
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

pilot/pkg/networking/OWNERS [hzxuzhonghu]

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

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

Test name
Commit
Details
Rerun command

prow/istio-pilot-multicluster-e2e.sh
3f6055b3e31bfe7f692c36fa1fe55af66a3dfd31
link
/test istio-pilot-multicluster-e2e

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

Why do we want this change? We are doing more work than we need to... Before when we weren't using locality we didn't do any of this. Now (with this + your previous) we are always using locality weighted lb, by simulating non-locality weighted lb. It seems like a lot of extra complexity for 0 gain, and probably a performance hit.

Without this, the weight set from ServiceEntry is not handled right

The ServiceEntry weight feature has been around for a while and seems to work fine. If this is not the case can you add a test for this?

Locality weights are only considered when locality weighted load balancing is configured. These weights are ignored otherwise.
The weight is ignored by envoy.

Ignoring them does the same thing as summing the weight (#12714) so this is unneeded. Locallity LB is an optional feature we shouldn't always turn it on

Ignoring them does the same thing as summing the weight (#12714) so this is unneeded.

@howardjohn I am not sure what do you mean?
As you said https://github.com/istio/istio/pull/12714#issuecomment-478837179, if this locality_weighted_lb_config   is not set, the weight is ignored, and which result in ServiceEntry.Endpoints.Weight has no effect. because all the endpoints are evenly routed.
btw, istio split endpoints by their locality, which exists longer before locality lb exists. In other word, even if no locality lb setting, the behavior is not correct. So we should fix it. And I donot see any side effect setting locality_weighted_lb_config

Like you said we have had this for a long time, and no issues? So why is this change needed? If this is needed then please add a test that shows it is needed

Of course, a test can show whether this is needed.

Weird, without this config, the result shows the lb weight is respected.

create the httpbin ap, by kubectl apply -f sample/httpbin/httpbin.yaml

create a ServiceEntry, with 2 endpoints, the ip is the httpbin pod ip.

apiVersion: networking.istio.io/v1alpha3
kind: ServiceEntry
metadata:
  name: httpbin
spec:
  hosts:
  - httpbin.com
  ports:
  - number: 80
    name: http
    protocol: HTTP
  resolution: STATIC
  location: MESH_INTERNAL
  endpoints:
  - address: <IP_ADDRESS>
    locality: us/us-east-1/az-1
    weight: 80
  - address: <IP_ADDRESS>
    locality: us/us-east-2/az-1
	weight: 20

curl from sleep pod many times, curl http://httpbin.com:80/status/404, and i find the two httpbin pods receives request consistent with the weight 80 : 20.

istioctl pc cluster sleep-6df94d7667-cv5xg --fqdn="httpbin.com" -ojson
[
    {
        "name": "outbound|80||httpbin.com",
        "type": "STRICT_DNS",
        "connectTimeout": "1s",
        "loadAssignment": {
            "clusterName": "outbound|80||httpbin.com",
            "endpoints": [
                {},
                {},
                {
                    "locality": {
                        "region": "us",
                        "zone": "us-east-1",
                        "subZone": "az-1"
                    },
                    "lbEndpoints": [
                        {
                            "endpoint": {
                                "address": {
                                    "socketAddress": {
                                        "address": "<IP_ADDRESS>",
                                        "portValue": 80
                                    }
                                }
                            },
                            "loadBalancingWeight": 80
                        }
                    ]
                },
                {
                    "locality": {
                        "region": "us",
                        "zone": "us-east-2",
                        "subZone": "az-1"
                    },
                    "lbEndpoints": [
                        {
                            "endpoint": {
                                "address": {
                                    "socketAddress": {
                                        "address": "<IP_ADDRESS>",
                                        "portValue": 80
                                    }
                                }
                            },
                            "loadBalancingWeight": 20
                        }
                    ]
                }
            ]
        },
        "circuitBreakers": {
            "thresholds": [
                {
                    "maxRetries": 1024
                }
            ]
        },
        "dnsRefreshRate": "5s",
        "dnsLookupFamily": "V4_ONLY"
    }
]

I am not sure why the result is opposite to envoy doc Locality weights are only considered when locality weighted load balancing is configured. These weights are ignored otherwise.

There are two layers of load balancing weight - the outer locality one and the inner endpoints one. I think this config is only for the outer locality one which is not used in your example

oh, ic


How to get mFrameRect

how to get the field: mFrameRect?
i want to get mFrameRect.left,mFrameRect.right,mFrameRect.top.mFrameRect.bottom;
but it is private.

 Rect cropRect = cropImageView.getCropRect();  Log.e(TAG,"截图区域的原始图的4点坐标,left:"+cropRect.left                 +",top:"+cropRect.top                 +",right:"+cropRect.right                 +",bottom:"+cropRect.bottom);
get 4 coordinates for a rectangle,and it is about the source;
and then
`if (bitmapWidth >= screenWidth) {
float scale = compressBitmapWidth * 1.0f / screenWidth;
float cropScreenLeft = cropRect.left / scale;
float cropScreenRight = cropRect.right / scale;
float cropScreenTop = cropRect.top / scale;
float cropScreenBottom = cropRect.bottom / scale;
        Log.e(TAG,"截图区域在屏幕上的4点坐标,left:"+cropScreenLeft
                +",top:"+cropScreenTop
                +",right:"+cropScreenRight
                +",bottom:"+cropScreenBottom);
    }`

i try to calculate the scale between bitmap and phone window,suggest bitmap larger than phone window,then ..seems to be worked;
:8ball: hope it is right....


[PDI-12524]: Fixed UDJC on Java 7 for Mac

@mattyb149 - can you add a test case for this ?

@mbatchelor the code is insulated by the method; it's more of an integration test, which we don't run on Mac. What to do?


Create nextEpochAfter function.

Context
In order to support the ability for users to query the API about the next epoch, we need a function capable of calculating the next epoch number and start time from any given time.
Decision
Create a new function within Cardano.Wallet.Primitive.Types with a definition similar to:
nextEpochAfter :: SlotParameters -> UTCTime -> EpochInfo
nextEpochAfter sp time =

Where EpochInfo is a record type similar to:
data EpochInfo = EpochInfo
    { epochNumber :: !EpochNo
    , epochStartTime :: !UTCTime
    } deriving (Eq, Generic, Show)

Acceptance Criteria

There must be a nextEpochAfter function in Wallet.Primitive.Types.
The result of calling nextEpochAfter must be a compound value with both the next epoch number and start time after the given time.

Development

[ ] Add EpochInfo type.
[ ] Add nextEpochAfter function.

QA

[ ] Add property tests for the nextEpochAfter function.

Task goals/development plan looking good and clear

lgtm!


Do not allow -c with -q or -s

This pull request derives from #302. So needs to be handled after that.

Code coverage works only when testing XSLT. But the command line (xspec.sh and xspec.bat) accepts -c -q (XQuery) or -c -s (Schematron) and generates nonsense reports.
This pull request makes the command line reject -c -q or -c -s.
Ant build is already made so. (impl, test)
This pull request has tests whose failure is demonstrated on Travis and AppVeyor.

Although this pull request contains a lot of commits for historical reasons, all that needs to be reviewed in the end is a single commit 79fcb91cb0079c746611ab0654298282c4c49360.


Solid Kobalte Select example not working

Tried the example listed here. The TextField example works but the Select example doesn't. I put a log on the createEffect portion but it only logs once.
createEffect(() => {
  console.log("Trigger value: ", props.value);
  setSignalValue(props.options.find((option) => props.value === option.value));
});

Change the Select value works but it the form doesn't observe the new values.
What worked is adding an onChange listener directly on the Select.Root component and using setValue on the form.
onChange={(e) => {
  setValue(form, "field", e.value);
}}

Thank you very much for the hint. It is possible that my code samples are not up to date. If you need help with Kobalte, I recommend the Kobalte channel on the SolidJS Discord. If you find out what the problem is, I'd be happy if you let me know here.

I also tried to attach logging event listeners to the Select.HiddenSelect component and they are all triggering when I change the select value. Just to confirm, the props (2nd argument) passed to the children are used to update the reactive store of the forms, right?

Since I'm currently writing my bachelor thesis, my mind is on other things at the moment. Unfortunately I can't give you a detailed answer. Please share your further research with me here, so that I can fix the Kobalte documentation on the Modular Forms website in September. If you have questions about Kobalte, I recommend this channel in the SolidJS Discord.

Maybe related: I am having problems with kobalte's combobox and modularforms where I control the value of the combobox. What I see is that the onChange and onInput listeners of Combobox.HiddenSelect fire, but the onBlur listener not. Getting the value of the input using the browser console gives me its correct value, but when modularforms validates, the value is always an empty string. Is the blur listener somehow important here?

For Modular Forms only onInput is important for capturing the new value.

Oh I think I oversaw something... currentTarget is set to the document on the input event from kobalte. Might this be the problem? But it can't be it actually, because my selects work and when I select something manually in the combobox's dropdown it works too. Only when I set a new value programatically, modularforms doesn't seem to recognize it, but the events look almost identical. The only difference is, that explicitOriginalTarget is once set to the list item that is selected and once to the input of the combobox.

I'm sorry, due to time constraints I can't take a closer look. If the problem still exists in September, I can check and fix the Kobalte Guide in our docs.

Somehow, I fixed my issue. It has something to do with zod's parsing/validation. Initially, I was using zod's nativeEnum with numbers as enum values instead of strings. When I changed the enum values to strings, it worked as intended.
@apollo79 if you happen to also use zod and nativeEnum, try change the values to strings.
enum E {
FOO = "FOO"
}

Instead of:
enum E {
FOO = 1
}

The type property specifies which data type is captured, but if the <input /> or <select /> element used does not support it, no input is captured at all.

The type property specifies which data type is captured, but if the <input /> or <select /> element used does not support it, no input is captured at all.

I see, maybe an exception should be thrown when there's a mismatch between the value and the type?

Good idea. Will have a look at it in September.

Okay, I found a workaround for the combobox issue which is simply delaying setting the new value with a setTimeout(() => setValue(newOption), 0). I still don't know if the issue is on my side and the option just doesn't exist at the time I set the value or if it is something with kobalte / solid, but I think the issue isn't by modular-forms.

I'll chime in, as just a regular user trying to follow the docs. I can confirm at least getting stuck on trying to use Kobalte Select in combination with zod form validation and a field value of type number. I got typescript errors that were really hard to understand, so I just figured that I must be lacking the know-how to make this work. I'm quite relieved to see that this is an issue on the repo and can't believe I didn't check this out sooner. I fully respect you are setting firm limits on your time to complete your education @fabian-hiller, although I at least would have really benefited from a notice on compatibility issues on the site. If you make the time to rework the examples, I'll be even grateful for the ability to combine two awesome libraries :)

@fabien-ml do you have time to revise the code of the Kobalte guide in the Modular Forms docs?


Building R library with G++5.4 brings in the nullptr error (similar to issue 1393)

The internet says the g++ command lines need to add "'-std=c++11' to solve the issue, and I'm trying to find where in the R library build setup I have to put it in.
Environment info
No LSB modules are available.
Distributor ID:	Ubuntu
Description:	Ubuntu 16.04.2 LTS
Release:	16.04
Codename:	xenial
Compiler:
omri@SERVER2:~/xgboost/R-package$ g++ --version
g++ (Ubuntu 5.4.0-6ubuntu1~16.04.4) 5.4.0 20160609
Package used (python/R/jvm/C++):
R

The commit hash (git rev-parse HEAD)
omri@SERVER2:~/xgboost/R-package$ git rev-parse HEAD
be1f76a06acbacb9d925cccb338fae82fb9d2743

Logs will be helpful (If logs are large, please upload as attachment).
omri@SERVER2:~/xgboost/R-package$ R CMD INSTALL .

installing to library ‘/home/omri/R/x86_64-pc-linux-gnu-library/3.3’
installing source package ‘xgboost’ ...
configure: creating ./config.status
config.status: creating src/Makevars
** libs
g++  -I/usr/share/R/include -DNDEBUG -I../..//include -I../..//dmlc-core/include -I../..//rabit/include -I../../ -DXGBOOST_STRICT_R_MODE=1 -DDMLC_LOG_BEFORE_THROW=0 -DDMLC_ENABLE_STD_THREAD=1 -DDMLC_DISABLE_STDIN=1 -DDMLC_LOG_CUSTOMIZE=1 -DXGBOOST_CUSTOMIZE_LOGGER=1 -DRABIT_CUSTOMIZE_MSG_ -DRABIT_STRICT_CXX98_    -fopenmp  -fpic  -g -O2 -fdebug-prefix-map=/build/r-base-6WVosl/r-base-3.3.2=. -fstack-protector-strong -Wformat -Werror=format-security -Wdate-time -D_FORTIFY_SOURCE=2 -g -c xgboost_R.cc -o xgboost_R.o
In file included from xgboost_R.cc:2:0:
../..//dmlc-core/include/dmlc/logging.h: In constructor ‘dmlc::LogCheckError::LogCheckError()’:
../..//dmlc-core/include/dmlc/logging.h:66:25: error: ‘nullptr’ was not declared in this scope
LogCheckError() : str(nullptr) {}
^
../..//dmlc-core/include/dmlc/logging.h: In destructor ‘dmlc::LogCheckError::~LogCheckError()’:
../..//dmlc-core/include/dmlc/logging.h:68:33: error: ‘nullptr’ was not declared in this scope
~LogCheckError() { if (str != nullptr) delete str; }
^
../..//dmlc-core/include/dmlc/logging.h: In member function ‘dmlc::LogCheckError::operator bool()’:
../..//dmlc-core/include/dmlc/logging.h:69:34: error: ‘nullptr’ was not declared in this scope
operator bool() {return str != nullptr; }
^
In file included from xgboost_R.cc:2:0:
../..//dmlc-core/include/dmlc/logging.h: In destructor ‘dmlc::LogMessageFatal::~LogMessageFatal()’:
../..//dmlc-core/include/dmlc/logging.h:289:17: error: ‘nullptr’ was not declared in this scope
if (msgs != nullptr) {
^
xgboost_R.cc: In function ‘SEXPREC* XGBoosterSetAttr_R(SEXP, SEXP, SEXP)’:
xgboost_R.cc:419:33: error: ‘nullptr’ was not declared in this scope
const char *v = isNull(val) ? nullptr : CHAR(asChar(val));
^
/usr/lib/R/etc/Makeconf:139: recipe for target 'xgboost_R.o' failed
make: *** [xgboost_R.o] Error 1

If you are using R package, please provide

The R sessionInfo()

sessionInfo()
R version 3.3.2 (2016-10-31)
Platform: x86_64-pc-linux-gnu (64-bit)
Running under: Ubuntu 16.04.2 LTS

Editing Makevars.in to add that flag resolved the issue for me. But I don;t know how to DTRT and leave it compatible with Windows and Mac, so I'll leave this as a ticket rahter than a pull request.

`omri@SERVER2:~/xgboost/R-package$  git diff
diff --git a/R-package/src/Makevars.in b/R-package/src/Makevars.in
index 26aa538..d35b215 100644
--- a/R-package/src/Makevars.in
+++ b/R-package/src/Makevars.in
@@ -10,7 +10,7 @@ XGB_RFLAGS = -DXGBOOST_STRICT_R_MODE=1 -DDMLC_LOG_BEFORE_THROW=0
-DDMLC_LOG_CUSTOMIZE=1 -DXGBOOST_CUSTOMIZE_LOGGER=1
-DRABIT_CUSTOMIZE_MSG_ -DRABIT_STRICT_CXX98_
-PKG_CPPFLAGS=  -I$(PKGROOT)/include -I$(PKGROOT)/dmlc-core/include -I$(PKGROOT)/rabit/include -I$(PKGROOT) $(XGB_RFLAGS)
+PKG_CPPFLAGS= -std=c++11 -I$(PKGROOT)/include -I$(PKGROOT)/dmlc-core/include -I$(PKGROOT)/rabit/include -I$(PKGROOT) $(XGB_RFLAGS)
PKG_CXXFLAGS= @OPENMP_CXXFLAGS@ $(SHLIB_PTHREAD_FLAGS)
PKG_LIBS = @OPENMP_CXXFLAGS@ $(SHLIB_PTHREAD_FLAGS)
OBJECTS= ./xgboost_R.o ./xgboost_custom.o ./xgboost_assert.o ./init.o
`

`omri@SERVER2:~/xgboost/R-package$  git diff
diff --git a/R-package/src/Makevars.in b/R-package/src/Makevars.in
index 26aa538..d35b215 100644
--- a/R-package/src/Makevars.in
+++ b/R-package/src/Makevars.in
@@ -10,7 +10,7 @@ XGB_RFLAGS = -DXGBOOST_STRICT_R_MODE=1 -DDMLC_LOG_BEFORE_THROW=0
-DDMLC_LOG_CUSTOMIZE=1 -DXGBOOST_CUSTOMIZE_LOGGER=1
-DRABIT_CUSTOMIZE_MSG_ -DRABIT_STRICT_CXX98_
-PKG_CPPFLAGS=  -I$(PKGROOT)/include -I$(PKGROOT)/dmlc-core/include -I$(PKGROOT)/rabit/include -I$(PKGROOT) $(XGB_RFLAGS)
+PKG_CPPFLAGS= -std=c++11 -I$(PKGROOT)/include -I$(PKGROOT)/dmlc-core/include -I$(PKGROOT)/rabit/include -I$(PKGROOT) $(XGB_RFLAGS)
PKG_CXXFLAGS= @OPENMP_CXXFLAGS@ $(SHLIB_PTHREAD_FLAGS)
PKG_LIBS = @OPENMP_CXXFLAGS@ $(SHLIB_PTHREAD_FLAGS)
OBJECTS= ./xgboost_R.o ./xgboost_custom.o ./xgboost_assert.o ./init.o
`

Some things are odd here... Note that src/Makevers.in already has CXX_STD = CXX11 defined, which should be enough. Also, while your R is 64 bit, compiler is not using the -m64 flag. Was your R installation customized in any way? Could you please post the output of the following commands:
gcc -v
grep CXX `R RHOME`/etc/Makeconf | grep -v '^#'
diff src/Makevars.in src/Makevars

omri@SERVER2:~$ cd xgboost/R-package/
omri@SERVER2:~/xgboost/R-package$ gcc -v
Using built-in specs.
COLLECT_GCC=gcc
COLLECT_LTO_WRAPPER=/usr/lib/gcc/x86_64-linux-gnu/5/lto-wrapper
Target: x86_64-linux-gnu
Configured with: ../src/configure -v --with-pkgversion='Ubuntu 5.4.0-6ubuntu1~16.04.4' --with-bugurl=file:///usr/share/doc/gcc-5/README.Bugs --enable-languages=c,ada,c++,java,go,d,fortran,objc,obj-c++ --prefix=/usr --program-suffix=-5 --enable-shared --enable-linker-build-id --libexecdir=/usr/lib --without-included-gettext --enable-threads=posix --libdir=/usr/lib --enable-nls --with-sysroot=/ --enable-clocale=gnu --enable-libstdcxx-debug --enable-libstdcxx-time=yes --with-default-libstdcxx-abi=new --enable-gnu-unique-object --disable-vtable-verify --enable-libmpx --enable-plugin --with-system-zlib --disable-browser-plugin --enable-java-awt=gtk --enable-gtk-cairo --with-java-home=/usr/lib/jvm/java-1.5.0-gcj-5-amd64/jre --enable-java-home --with-jvm-root-dir=/usr/lib/jvm/java-1.5.0-gcj-5-amd64 --with-jvm-jar-dir=/usr/lib/jvm-exports/java-1.5.0-gcj-5-amd64 --with-arch-directory=amd64 --with-ecj-jar=/usr/share/java/eclipse-ecj.jar --enable-objc-gc --enable-multiarch --disable-werror --with-arch-32=i686 --with-abi=m64 --with-multilib-list=m32,m64,mx32 --enable-multilib --with-tune=generic --enable-checking=release --build=x86_64-linux-gnu --host=x86_64-linux-gnu --target=x86_64-linux-gnu
Thread model: posix
gcc version 5.4.0 20160609 (Ubuntu 5.4.0-6ubuntu1~16.04.4)

omri@SERVER2:~/xgboost/R-package$  grep CXX R RHOME/etc/Makeconf | grep -v '^#'
CXX = g++
CXXCPP = $(CXX) -E
CXXFLAGS = -g -O2 -fdebug-prefix-map=/build/r-base-6WVosl/r-base-3.3.2=. -fstack-protector-strong -Wformat -Werror=format-security -Wdate-time -D_FORTIFY_SOURCE=2 -g $(LTO)
CXXPICFLAGS = -fpic
CXX1X = g++
CXX1XFLAGS = -g -O2 -fdebug-prefix-map=/build/r-base-6WVosl/r-base-3.3.2=. -fstack-protector-strong -Wformat -Werror=format-security -Wdate-time -D_FORTIFY_SOURCE=2 -g
CXX1XPICFLAGS = -fpic
CXX1XSTD =
OBJCXX =
SHLIB_CXXFLAGS =
SHLIB_CXXLD = $(CXX)
SHLIB_CXXLDFLAGS = -shared
SHLIB_CXX1XLD = $(CXX1X) $(CXX1XSTD)
SHLIB_CXX1XLDFLAGS = -shared
SHLIB_OPENMP_CXXFLAGS = -fopenmp
R_XTRA_CXXFLAGS =
ALL_CXXFLAGS = $(R_XTRA_CXXFLAGS) $(PKG_CXXFLAGS) $(CXXPICFLAGS) $(SHLIB_CXXFLAGS) $(CXXFLAGS)
ALL_OBJCXXFLAGS = $(PKG_OBJCXXFLAGS) $(CXXPICFLAGS) $(SHLIB_CXXFLAGS) $(OBJCXXFLAGS)
$(CXX) $(ALL_CPPFLAGS) $(ALL_CXXFLAGS) -c $< -o $@
$(CXX) $(ALL_CPPFLAGS) $(ALL_CXXFLAGS) -c $< -o $@
@$(CXX) -M $(ALL_CPPFLAGS) $< > $@
@$(CXX) -M $(ALL_CPPFLAGS) $< > $@
$(OBJCXX) $(ALL_CPPFLAGS) $(ALL_OBJCXXFLAGS) -c $< -o $@
$(OBJCXX) $(ALL_CPPFLAGS) $(ALL_OBJCXXFLAGS) -c $< -o $@

omri@SERVER2:~/xgboost/R-package$  diff src/Makevars.in src/Makevars
14,15c14,15
< PKG_CXXFLAGS= @OPENMP_CXXFLAGS@ $(SHLIB_PTHREAD_FLAGS)
< PKG_LIBS = @OPENMP_CXXFLAGS@ $(SHLIB_PTHREAD_FLAGS)

PKG_CXXFLAGS= $(SHLIB_OPENMP_CFLAGS) $(SHLIB_PTHREAD_FLAGS)
PKG_LIBS = $(SHLIB_OPENMP_CFLAGS) $(SHLIB_PTHREAD_FLAGS)

So, was your R installation modified in any way? I'm asking since your system Makeconf doesn't properly define CXX1XSTD, and that's why the CXX_STD = CXX11 setting in xgboost has no effect. It might also be related to this: https://bugs.r-project.org/bugzilla3/show_bug.cgi?id=17189
E.g., for comparison, here is the relevant piece of /usr/lib64/R/etc/Makeconf from the same R version 3.3.2 on a RHEL 6.8 server:
CXX = g++ -m64
CXXCPP = $(CXX) -E
CXXFLAGS = -O2 -g -pipe -Wall -Wp,-D_FORTIFY_SOURCE=2 -fexceptions -fstack-protector --param=ssp-buffer-size=4 -m64 -mtune=generic $(LTO)
CXXPICFLAGS = -fpic
CXX1X = g++ -m64
CXX1XFLAGS = -O2 -g -pipe -Wall -Wp,-D_FORTIFY_SOURCE=2 -fexceptions -fstack-protector --param=ssp-buffer-size=4 -m64 -mtune=generic
CXX1XPICFLAGS = -fpic
CXX1XSTD =  -std=c++0x
OBJCXX =
SHLIB_CXXFLAGS =
SHLIB_CXXLD = $(CXX)
SHLIB_CXXLDFLAGS = -shared
SHLIB_CXX1XLD = $(CXX1X) $(CXX1XSTD)
SHLIB_CXX1XLDFLAGS = -shared
SHLIB_OPENMP_CXXFLAGS = -fopenmp
R_XTRA_CXXFLAGS =

Oh! The one thing I did was pin the r-base packages to Ubuntu 17.

THis is all I did to make the change, which makes it a mix of 16.04 and 17.04
`more /etc/apt/preferences.d/pin
Package: r-base-core
Pin: release n=xenial
Pin-Priority: -10
Package: r-base-core
Pin: release n=zesty
Pin-Priority: 900
`

It looks like 17.04 uses GCC 6.3 which has the default language standard of -std=c++14, so it doesn't need an extra -std=c++11 flag to compile C++11 code. That's probably why your R package was built with no such special flag defined.

Oh!. Wow, thank you! It would have taken me ages to figure this out on my own.


"running gcc failed" : Cannot build and compile in Alpine Linux 3.3.3

What version of Go are you using (go version)?

# go version
go version go1.6.1 linux/amd64

What operating system and processor architecture are you using (go env)?

# go env
GOARCH="amd64"
GOBIN=""
GOEXE=""
GOHOSTARCH="amd64"
GOHOSTOS="linux"
GOOS="linux"
GOPATH="/root/hyperbuild"
GORACE=""
GOROOT="/usr/lib/go"
GOTOOLDIR="/usr/lib/go/pkg/tool/linux_amd64"
GO15VENDOREXPERIMENT="1"
CC="gcc"
GOGCCFLAGS="-fPIC -m64 -pthread -fmessage-length=0"
CXX="g++"
CGO_ENABLED="1"

What did you do?

Tried to build an compile https://github.com/hyperhq/hyperd.git

mkdir -p ${GOPATH}/src/github.com/hyperhq
cd ${GOPATH}/src/github.com/hyperhq
git clone https://github.com/hyperhq/runv.git runv
git clone https://github.com/hyperhq/hyperd.git hyperd
git clone https://github.com/hyperhq/hyperstart.git hyperstart
cd hyperd
./autogen.sh
./configure
make

What did you expect to see?

Two go binaries hyperd and hyperctlto be built succesfully like it happens in Ubuntu/debian.

What did you see instead?

During make, the error thrown was:
# make
make[1]: Entering directory '/root/hyperbuild/src/github.com/hyperhq/hyperd'
go build -tags "static_build   libdm_no_deferred_remove exclude_graphdriver_btrfs" -ldflags "-X github.com/hyperhq/hyperd/utils.VERSION 0.5.0" hyperd.go
# command-line-arguments
link: warning: option -X github.com/hyperhq/hyperd/utils.VERSION 0.5.0 may not work in future releases; use -X github.com/hyperhq/hyperd/utils.VERSION=0.5.0
**/usr/lib/go/pkg/tool/linux_amd64/link: running gcc failed: exit status 1**
/usr/lib/gcc/x86_64-alpine-linux-musl/5.3.0/../../../../x86_64-alpine-linux-musl/bin/ld: /tmp/go-link-108672170/000004.o: unrecognized relocation (0x2a) in section `.text'
/usr/lib/gcc/x86_64-alpine-linux-musl/5.3.0/../../../../x86_64-alpine-linux-musl/bin/ld: final link failed: Bad value
collect2: error: ld returned 1 exit status

Makefile:785: recipe for target 'build-hyperd' failed
make[1]: *** [build-hyperd] Error 2
make[1]: Leaving directory '/root/hyperbuild/src/github.com/hyperhq/hyperd'
Makefile:359: recipe for target 'all-recursive' failed
make: *** [all-recursive] Error 1

Posted Makefile with the error above at: http://hastebin.com/evikapojav.mel
Thanks in advance!

Have you checked whether this is a duplicate of any previously-reported Alpine problems? https://github.com/golang/go/issues?utf8=✓&q=is%3Aissue+alpine
/cc @ianlancetaylor

@bradfitz: couldn't figure out, before I posted and therefore created a new issue.

Your linker is complaining about an object that was generated by your compiler or assembler.  This is not a Go bug and there is nothing that Go can change that will fix it.
Basically you have an assembler that generates a R_X86_64_REX_GOTPCRELX relocation but your linker doesn't understand it.
Basically a dup of #13678.

go build -a fixed this issue for me

apk add libc-dev fixed this issue for me.


4.0 (incomplete) in RSwitch menu

Hi,
I just upgraded my R version from 4.0.0 to 4.0.1.  It was working perfectly fine on 4.0.0 before.  I was wondering why in my RSwitch menu, it now says "4.0.0 (incomplete)"
Best wishes,
Lucy

Also, you probably need to re-install the earlier version, then install the newer version following the instructions above, if you want to keep the earlier version.

Had the same issue installing R4.2.0 over R4.1.3. Using sudo pkgutil --forget org.R-project.R.fw.pkg solved indeed that for me! I did not need to reinstall the newer version, only the older that became "incomplete".

This is super helpful.
I'll try to reproduce and see if can add some detection bits a button to
autofix
On Wed, Apr 6, 2022 at 8:35 AM jgilis @.***> wrote:

Had the same issue installing R4.2.0 over R4.1.3. Using sudo pkgutil
--forget org.R-project.R.fw.pkg solved indeed that for me! I did not need
to reinstall the newer version, only the older that became "incomplete".
—
Reply to this email directly, view it on GitHub
https://github.com/hrbrmstr/RSwitch/issues/24#issuecomment-1090216961,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAD4PNUXEHO5RU3NJQ5ULV3VDWAKFANCNFSM5GCI2U2Q
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>


CRASH: java.util.ConcurrentModificationException

Please fill in the following fields:
Unity editor version: 2018.4.11f
Firebase Unity SDK version: 6.13
Source you installed the SDK (.unitypackage or Unity Package Manager):.unitypackage
Firebase plugins in use (Auth, Database, etc.):
Additional SDKs you are using (Facebook, AdMob, etc.):analytics,messaging
Platform you are using the Unity editor on (Mac, Windows, or Linux): Windows
Platform you are targeting (iOS, Android, and/or desktop): Android
Scripting Runtime (Mono, and/or IL2CPP):IL2CPP
Please describe the issue here:
(Please list the full steps to reproduce the issue. Include device logs, Unity logs, and stack traces if available.)

java.lang.RuntimeException:
at android.app.ActivityThread.performLaunchActivity (ActivityThread.java:3135)
at android.app.ActivityThread.handleLaunchActivity (ActivityThread.java:3278)
at android.app.servertransaction.LaunchActivityItem.execute (LaunchActivityItem.java:78)
at android.app.servertransaction.TransactionExecutor.executeCallbacks (TransactionExecutor.java:108)
at android.app.servertransaction.TransactionExecutor.execute (TransactionExecutor.java:68)
at android.app.ActivityThread$H.handleMessage (ActivityThread.java:1969)
at android.os.Handler.dispatchMessage (Handler.java:106)
at android.os.Looper.loop (Looper.java:214)
at android.app.ActivityThread.main (ActivityThread.java:7124)
at java.lang.reflect.Method.invoke (Native Method)
at com.android.internal.os.RuntimeInit$MethodAndArgsCaller.run (RuntimeInit.java:494)
at com.android.internal.os.ZygoteInit.main (ZygoteInit.java:975)

Caused by: java.util.ConcurrentModificationException:
  at androidx.collection.SimpleArrayMap.put (SimpleArrayMap.java:465)
  at com.google.android.gms.measurement.internal.zzin.zza
  at com.google.android.gms.measurement.internal.zzid.onActivityCreated
  at android.app.Application.dispatchActivityCreated (Application.java:245)
  at android.app.Activity.onCreate (Activity.java:1103)
  at com.unity3d.player.UnityPlayerActivity.onCreate (Unknown Source:4)
  at android.app.Activity.performCreate (Activity.java:7335)
  at android.app.Activity.performCreate (Activity.java:7326)
  at android.app.Instrumentation.callActivityOnCreate (Instrumentation.java:1275)
  at android.app.ActivityThread.performLaunchActivity (ActivityThread.java:3115)

Please answer the following, if applicable:
Have you been able to reproduce this issue with just the Firebase Unity quickstarts (this GitHub project)?
no
What's the issue repro rate? (eg 100%, 1/5 etc)
0.3%
(It seems to have been fixed https://firebase.google.com/support/release-notes/android#analytics_v17-2-3. when to update Firebase Unity SDK version?)

Hi @vincent0y23,
Thanks for your incredibly thorough bug report! The next release (6.14) should have the needed fix.
In the meantime, if you don't mind an untested association between Java and Unity (it should work, but obviously 6.14 will have a full battery of internal testing), you can open up Assets/Firebase/Editor/AnalyticsDependencies.xml and change:
<androidPackage spec="com.google.firebase:firebase-analytics:17.2.2">

to:
<androidPackage spec="com.google.firebase:firebase-analytics:17.3.0">

Let me know if you have any other questions!
--Patrick

Hi @vincent0y23,
Could you please confirm if 6.14 fixed this issue?  Thanks!


Does FreeRDP store RDS Per-Device CALs?

I have the issue that each freerdp client gets multiple per-device CALs issued, although it's the same device with the same MAC, DNS and IP.
If I read the code correctly, freerdp uses the MAC as hardware id, so the only question that remains is, does freerdp also store temporary / permanent RDS CALs?
My source:
http://www.virtualizationadmin.com/articles-tutorials/terminal-services/licensing/terminal-services-cal-allocation-process-part2.html
Thanks

I looked into this problem some months back.  FreeRDP doesn't attempt to cache the license.  I experimented with saving off a copy and sending it back per RDPELE spec, but I never saw any difference in behavior.  That said, I never observed a case where multiple licenses were being issued to one device - the license server always sent me the same bits and the license manager didn't report multiple licenses for that device.
If you're still observing this problem, please report back on what Server OS is in use, as the license server behavior varies between 2003/2008/2012.

@dbungert, could you please send me a reference to your code where you tried to save off a copy?
I have a similar problem and want to try if your experiment helps me.

@dbungert did you ever get feedback?  I have two configurations that are exhibiting this issue.  Is there a way to log these "bits" that could further diagnose this?

I have this issue with a 2008 R2 Session Host with a 2012 R2 License Server. It is remote sites with the license servers ran centrally, so there is possibility for the WAN link to be down. When this happens any freerdp clients refuse to connect as they don't cache the cal as expected. Thankfully we don't have many of these left as we have been steadily repolacing with rdesktop as this does work correctly

#4979 should address these issues.


tracing feature enable

I have enabled tracing feature on leptos crate version = 0.6. I use leptos_i18n::load_locales!(); to load locales json file. I got this error when compiling :
error[E0433]: failed to resolve: use of undeclared crate or module `tracing`
  --> src/lib.rs:14:1
   |
14 | leptos_i18n::load_locales!();
   | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^ use of undeclared crate or module `tracing`
   |
   = note: this error originates in the attribute macro `::leptos::leptos_dom::tracing::instrument` (in Nightly builds, run with -Z macro-backtrace for more info)

I need to use tracing. Thanks for your help.
Config

Linux 22.04.1-Ubuntu
cargo 1.78
cargo-leptos 0.2.17
leptos nightly

After looking at the Cargo.toml file, I forgot to enable tracing in the hydrate feature. My bad.

I'm sorry I did'nt saw this issue as it was closed fast, this crates use tracing for now, there is no tracing annotations generated or used anywhere, so it must comes from other generated code like leptos. If you want tracing used within this crate feel free to open an issue and PR are always welcome !