determine set level metadata

Set level metadata populates each map-item page's front matter and should include all of the appropriate metadata to describe the set of maps. Most of this information could come from the Library's set-level catalogue record, for example: http://resolve.library.ubc.ca/cgi-bin/catsearch?bid=3194411.
Descriptive

Title [Doesn't have to be verbatim from catalog record]
Publisher
Scale
Series Number
Web URL [a web link to additional information - not catalog record]
Complete [Y/N for completeness]
Type [maps or aerial photos]

Physical Location

Library Location
Call Number
Catalog URL [a permalink to catalog record]

Sort/Filter

Geographic Location [Country or State/Province]
First Year
Last Year

Set level metadata has been determined and added to README.md documentation. This can be revisited if necessary, but for now the issue has been completed.


[dv/chip] fix csr_hw_reset X assertion issue

The assertion issue from csr_hw_reset is mainly due to padring and tb
driving conflicts. If tb does not drive it, then high z will cause the
assertions to fire.
This PR attempt to disable the assertions during csr_hw_reset random
write CSRs. Then turn the assertions back on after hw_reset is issued.
I tried to adjust the pins from the tb side, but it does not work -
1). The main issue is the driving conflicts between tb and padring (pad_wrapper). When the output of the padring connects to some other modules, say GPIO's cio_gpio_i. From there the X propagates to GPIO's data_in_q. When writing to GPIO's interrupt ctrl_en registers, it goes to gpio_intr_o, which is guarded by the assert_known assertions.
2).  From the TB side, the driving pin is: "bootstrap", driving at chip_dv_base.sv:
virtual task pre_start();
   ........
    // Drive strap signals at the start.
    if (do_strap_pins_init) begin
      cfg.srst_n_vif.drive(1'b1);
      cfg.jtag_spi_n_vif.drive(1'b1); // Select JTAG.
      cfg.bootstrap_vif.drive(1'b0);
    end   ........
  endtask

3). From the DESIGN side, the driving force is when: mio_outsel is set to 1: (out is the "mio_outsel" value)
  assign (strong 0, strong 1) inout_io = (oe && drive == STRONG_DRIVE ) ? out : 1'bz;
4). For this conflict, from the TB side, either driving 0 or 1 would not solve this conflict, as mio_outsel could be assigned to 0 or 1 randomly. If not driving from the tb side, high Z will cause the assertions to fail as well.
Signed-off-by: Cindy Chen<EMAIL_ADDRESS>

thanks Cindy. I'll officially close the other PR now.


Error angular2-webpack-starter/node_modules/@types/protractor/index.d.ts:22:27 Module 'webdriver' has no exported member 'IButton'.

####I'm submitting a bug report concerning a fresh install of current the master branch.
Current Node v.6.2.0
Current npm v.3.8.9
####Steps to reproduce:

$ git clone master branch
$ npm install
$ npm install<EMAIL_ADDRESS>--save #https://github.com/AngularClass/angular2-webpack-starter/pull/1018
$ npm start

Error Stackwebpack: bundle is now VALID.
[default] Checking started in a separate process...
[default] /home/osboxes/Downloads/angular2-webpack-starter/node_modules/@types/protractor/index.d.ts:22:27
Module 'webdriver' has no exported member 'IButton'.
[default] /home/osboxes/Downloads/angular2-webpack-starter/node_modules/@types/protractor/index.d.ts:25:24
Module 'webdriver' has no exported member 'IKey'.
[default] /home/osboxes/Downloads/angular2-webpack-starter/node_modules/@types/protractor/index.d.ts:28:45
Property 'Error' does not exist on type 'typeof error'.
[default] /home/osboxes/Downloads/angular2-webpack-starter/node_modules/@types/protractor/index.d.ts:29:40
Module 'webdriver.error' has no exported member 'IErrorCode'.
[default] /home/osboxes/Downloads/angular2-webpack-starter/node_modules/@types/protractor/index.d.ts:36:37
Module 'webdriver.logging' has no exported member 'IType'.
[default] /home/osboxes/Downloads/angular2-webpack-starter/node_modules/@types/protractor/index.d.ts:37:38
Module 'webdriver.logging' has no exported member 'ILevelValues'.
[default] /home/osboxes/Downloads/angular2-webpack-starter/node_modules/@types/protractor/index.d.ts:39:67
Module 'webdriver.logging' has no exported member 'ILevel'.
[default] /home/osboxes/Downloads/angular2-webpack-starter/node_modules/@types/protractor/index.d.ts:40:67
Module 'webdriver.logging' has no exported member 'ILevel'.
[default] /home/osboxes/Downloads/angular2-webpack-starter/node_modules/@types/protractor/index.d.ts:287:39
Property 'stacktrace' does not exist on type 'typeof webdriver'.
[default] /home/osboxes/Downloads/angular2-webpack-starter/node_modules/@types/protractor/index.d.ts:288:42
Property 'stacktrace' does not exist on type 'typeof webdriver'.
[default] /home/osboxes/Downloads/angular2-webpack-starter/node_modules/@types/protractor/index.d.ts:303:35
Module 'webdriver' has no exported member 'stacktrace'.
[default] /home/osboxes/Downloads/angular2-webpack-starter/node_modules/@types/protractor/index.d.ts:340:55
Module 'webdriver' has no exported member 'Locator'.
[default] /home/osboxes/Downloads/angular2-webpack-starter/node_modules/@types/protractor/index.d.ts:414:52
Module 'webdriver' has no exported member 'Locator'.
[default] /home/osboxes/Downloads/angular2-webpack-starter/node_modules/@types/protractor/index.d.ts:462:53
Module 'webdriver' has no exported member 'Locator'.
[default] /home/osboxes/Downloads/angular2-webpack-starter/node_modules/@types/protractor/index.d.ts:664:29
Module 'webdriver' has no exported member 'Locator'.
[default] /home/osboxes/Downloads/angular2-webpack-starter/node_modules/@types/protractor/index.d.ts:673:32
Module 'webdriver' has no exported member 'Locator'.
[default] /home/osboxes/Downloads/angular2-webpack-starter/node_modules/@types/protractor/index.d.ts:704:39
Module 'webdriver' has no exported member 'Locator'.
[default] /home/osboxes/Downloads/angular2-webpack-starter/node_modules/@types/protractor/index.d.ts:725:35
Module 'webdriver' has no exported member 'Locator'.
[default] /home/osboxes/Downloads/angular2-webpack-starter/node_modules/@types/protractor/index.d.ts:801:48
Module 'webdriver' has no exported member 'Locator'.
[default] /home/osboxes/Downloads/angular2-webpack-starter/node_modules/@types/protractor/index.d.ts:808:30
Module 'webdriver' has no exported member 'Locator'.
[default] /home/osboxes/Downloads/angular2-webpack-starter/node_modules/@types/protractor/index.d.ts:1111:32
Module 'webdriver' has no exported member 'Locator'.
[default] /home/osboxes/Downloads/angular2-webpack-starter/node_modules/@types/protractor/index.d.ts:1139:30
Module 'webdriver' has no exported member 'Locator'.
[default] /home/osboxes/Downloads/angular2-webpack-starter/node_modules/@types/protractor/index.d.ts:1362:51
Module 'webdriver' has no exported member 'Locator'.
[default] /home/osboxes/Downloads/angular2-webpack-starter/node_modules/@types/protractor/index.d.ts:1363:42
Module 'webdriver' has no exported member 'Locator'.
[default] /home/osboxes/Downloads/angular2-webpack-starter/node_modules/@types/protractor/index.d.ts:1364:41
Module 'webdriver' has no exported member 'Locator'.
[default] /home/osboxes/Downloads/angular2-webpack-starter/node_modules/@types/protractor/index.d.ts:1440:55
Module 'webdriver' has no exported member 'Locator'.
[default] /home/osboxes/Downloads/angular2-webpack-starter/node_modules/@types/protractor/index.d.ts:1461:60
Module 'webdriver' has no exported member 'Locator'.
[default] /home/osboxes/Downloads/angular2-webpack-starter/node_modules/@types/protractor/index.d.ts:1477:41
Module 'webdriver' has no exported member 'Locator'.
[default] /home/osboxes/Downloads/angular2-webpack-starter/node_modules/@types/protractor/index.d.ts:1491:51
Module 'webdriver' has no exported member 'Locator'.
[default] /home/osboxes/Downloads/angular2-webpack-starter/node_modules/@types/protractor/index.d.ts:1505:58
Module 'webdriver' has no exported member 'Locator'.
[default] /home/osboxes/Downloads/angular2-webpack-starter/node_modules/@types/protractor/index.d.ts:1574:79
Module 'webdriver' has no exported member 'Locator'.
[default] /home/osboxes/Downloads/angular2-webpack-starter/node_modules/@types/protractor/index.d.ts:1594:55
Module 'webdriver' has no exported member 'Locator'.
[default] /home/osboxes/Downloads/angular2-webpack-starter/node_modules/@types/protractor/index.d.ts:1614:36
Module 'webdriver' has no exported member 'Locator'.
[default] Checking finished with 33 errors
Any ideas what's wrong?

same

same

Duplicate of #1031, hotfix there.


Write unit tests for multibyte string usage in crypto functions

We removed the usage of substr() and strlen() in #22 but we didn't provide a full unit tests to cover the multibyte usage. We should add these tests to prevent a potential regression of the code using str functions.

I'm closing this PR due to the  difficulty to use the overload function of Multibyte String (mbstring.func_overload) for testing the multibyte issue in substr() and strlen().


fix: update wrong link

Description (required)
Katie Sylor-Miller's Twitter doesn't seem to exist anymore, so I replace the link with her website.

Lunaria Status Overview
🌕 This pull request will trigger status changes.

Learn more
By default, every PR changing files present in the Lunaria configuration's files property will be considered and trigger status changes accordingly.
You can change this by adding one of the keywords present in the ignoreKeywords property in your Lunaria configuration file in the PR's title (ignoring all files) or by including a tracker directive in the merged commit's description.

Tracked Files

File
Note

en/concepts/islands.mdx
Source changed, localizations will be marked as outdated.

Warnings reference

Icon
Description

🔄
The source for this localization has been updated since the creation of this pull request, make sure all changes in the source have been applied.

I noticed this broken Twitter link too when translating this page some days ago, forgot to create a PR on this...
Thank you @jsparkdev for bringing that up!
LGTM!


Bug Report: The game crashes whenever the tutorial is beaten

I've encountered this bug in both my own compiled version and the downloaded version of 1.6.2. It's pretty simple to reproduce the bug -- just beat the tutorial and boom, the game closes. The video attached is of the release version of KE. No other weeks have this issue, it's only the tutorial that crashes.
I would like to add, the tutorial also crashes in freeplay.
https://user-images.githubusercontent.com/76640329/129488303-93a7294b-d479-418a-ad97-f4cdcb140fb3.mp4

already fixed in the latest master commit.


Matching the nouns to the nodes by LCH score

Hi zhong, Thanks for your solid work which helps me a lot.
I'd like to ask your about is there a LCH score threshold which decides two noun node matching or not? If so, what the exact value the threshold is.
Best,
zhuowei

The LCH threshold was set as 2.6 in this project. Hope this helps.

The LCH threshold was set as 2.6 in this project. Hope this helps.

THX

THX


Add completion handlers for the new commands

/kind feature

Which functionality do you think we should add?
We need completion handlers for the new commands introduced in v3

[ ] odo init
[ ] odo delete
[ ] odo preference

Why is this needed?
To provide a better UX.

/close
Completion is already available for the sub-commands above.


Unable to run rust files begin with #![...]

A rust file usually begin with #![...]. But script can run it.
Here is my case:
#![feature(fnbox)]

use std::boxed::FnBox;

// Return a closure taking no inputs and returning nothing
// which implements `FnBox` (capture by value).
fn create_fn() -> Box<Fn()> {
    let text = "Fn".to_owned();

    Box::new(move || println!("This is a: {}", text))
}

fn create_fnmut() -> Box<FnMut()> {
    let text = "FnMut".to_owned();

    Box::new(move || println!("This is a: {}", text))
}

fn create_fnbox() -> Box<FnBox()> {
    let text = "FnBox".to_owned();

    Box::new(move || println!("This is a: {}", text))
}

fn main() {
    let fn_plain = create_fn();
    let mut fn_mut = create_fnmut();
    let fn_box = create_fnbox();

    fn_plain();
    fn_mut();
    fn_box();
}

Script reports that:
Unable to run

[feature(fnbox)]
Is it in your PATH?
   ...

Hmmm. We interpret #! as the directive for what interpreter to use. Seems like we need to handle special casing for Rust.

yes, for example http://rustbyexample.com/attribute/unused.html instead of write for each function you can write at the top
#![allow(dead_code)]

So it applies globally.


mlp_mnist_simple.ipynb => softmax_cross_entropy_with_logits with named arguments (labels=..., logits=..., ...)

About https://github.com/sjchoi86/Tensorflow-101/blob/master/notebooks/mlp_mnist_simple.ipynb
Recently updated Tensorflow (1.4) will make the following error.
ValueError: Only call softmax_cross_entropy_with_logits with named arguments (labels=..., logits=..., ...)
Just simply adding named arguments will solve this issue.
cost = tf.reduce_mean(tf.nn.softmax_cross_entropy_with_logits(logits =pred, labels=y))
FYI, http://www.edwith.org/deeplearningchoi/lecture/15552/ has the right code.

got it


replay: fix issue when original logfile had topics with zero timstamps

Describe problem solved by the proposed pull request
If the original logfile contained a topic which at some point was logged with a zero timestamp next_file_time would be set to zero and in the next iteration of the for loop next_file_time is updated to the timestamp of the next topic in the _subscriptions list.
The result was that the topics which were before the zero timestamp topic in the _subscriptions list would never be published until the end. This was not visible in most log analysis software since the timestamp would still be correct but the order and timing of the published topics would be wrong.
Additional context
Another thing which makes replay not work as expected is the fact that the logger only tries to subscribe to a topic once a second. So e.g. when a VTOL transitions to cruise flight there will be a time right after transition when the fw_pos_controller publishes the virtual attitude setpoint but it is not logged for up to one second

So you mean a zero timestamp not at the beginning, after already valid timestamp(s)?

Yes exactly

Yes exactly

Is this in upstream, and if so, can we fix that as well?

No I don't think so, for us it was the virtual_attitude_sp after transition to cruise. I am fixing this now but in order to make sure that the functionality stays the same I needed to be able to replay the controllers. Awesome feature btw


Documentation Translation

Would it be okay if I translated the japanese documents into english for people english speakers? Thank you.

I do not speak japanese but I am sure I could use a translating program. It would be helpful for me because I would like to build the tenting version of the crkbd but the only instructions are in japanese. I would just like to know if a pull request for this would be welcome.

Ok, I will just issue a pull request for it and see what happens. Everyone have a great day!


Check dependencies for security vulnerabilities automatically

Audit dependencies automatically using cargo-audit as a part of the CI/CD pipeline.
https://github.com/RustSec/cargo-audit

See also https://github.com/actions-rs/audit-check


[BUG?] Speed modifier always gets applied when coming from ScreenTitleMenu

Is there an existing issue for this?

[X] I have searched the existing issues and none matched.

Operating System
Windows 10 (21H2)
CPU
Intel Core i7 2600 @ 3.40GHz
GPU
8192MB ATI Radeon RX 580 Series
Storage
2 x 1863GB Western Digital WDC WD2003FZEX-00SRLA0 ATA Device (SATA )
Game Version
4.12.0
Describe the problem
The Speed modifier seems to always get applied when coming into the game.
Changing it in the PlayerOptions seems to undo that and it stays that way after play.
But once returning to the ScreenTitleMenu, it gets reapplied.
Video is in the "Additional Content" section.
I really don't know where this is coming from.
All of this seemed to happen after the first "fix" to the "Rate and Pitch" system during 4.9.9 (I think).
It's not even a theme issue since this happens everywhere.
It also can't be because the Save folder, as that had been manually wiped because of 4.11.0 complications.
There either has to be another folder/file OutFox reads stuff from or it is an engine issue.
Describe what should happen
It shouldn't be doing that?
Relevant Log output
I don't think that a log would help. But I would provide one if it's actually needed.
Video proof of said bug (the recording is from 4.11.0 but the bug still persist):
http://y2u.be/PdXeQyaWsHE

Hiya, Thanks for reporting this one, and happy new year!
This is indeed a theme issue where the default option settles on 'speed' when the menu is first entered, and it should be on 'off' as you correctly state. It was due to the optionrow changing when we added speed and pitch.
I'll chat to the design team to fix the menu for the next alpha.

Hiya, Thanks for reporting this one, and happy new year!
This is indeed a theme issue where the default option settles on 'speed' when the menu is first entered, and it should be on 'off' as you correctly state. It was due to the optionrow changing when we added speed and pitch.
I'll chat to the design team to fix the menu for the next alpha.


Add open position and close position

[x] add open position and close position
[x] unit test
[x] e2e test
[x] yarn changeset

Please assist in adding open and close to the type Operation on protocol.type.ts.

LGTM

LGTM


Permissions working for one user but not other

So basically, me and my friend are in the same group and have the same user permissions but my friend can break protected land and I can't break that protected land. We both have the permission to break the land in our group permissions but I can't break it
I'm using PlotSquared and the problem may be with that but I'm just checking if there is anything wrong with my LuckPerms.

I'm going to suggest the protection is caused by PlotSquared and your friend just needs to trust you in their plot.
Otherwise you can always use LuckPerms' verbose mode to see if you are missing any permissions.

Nevermind... This isnt a luckperm issue... its to do with plotsquared or another plugin. I tried removing luckperms and oping my self and the issue is still there.


Expose disk usage to frontend, allow installing on used disks

This changes the installer to allow installing onto disks that are "used", and exposes a flag that lets the frontend show a warning if that's the case.

Companion PR in dpanel:
https://github.com/dogeorg/dpanel/pull/87


Create the Database Model

As a Developer
I need to create the database model
So that we can use to store data
Details and Assumptions

The database model must have below attributes

ID (unique, perhaps a SKU - Stock Keeping Unit)
Name
Description
Price
Available (True, False)
Image_url
Category
Create_time
Update_time

Acceptance Criteria
Given the database model is created
When we perform any operation of data record.
Then they should all follow the model

We can add Create_time and Update_time later, because these should be automatically maintained by the database.


can't install CocosSharp.Forms nuget in my solution of pcl project

Change the profile to 111. It seems to work with that one.


List of TAs

Tomas Larsson can do day 2-5, but not the first day.

Verena is available as TA.

Estelle can come on the first day.


Ensure hpx_main is a proper thread_function

Ensure that hpx_main is launched as a thread_function, so that exit callbacks are respected, and the following code will actually be used:
https://github.com/STEllAR-GROUP/hpx/blob/e7c31a49dd4e550a61134e498f3ecfb585d201cb/libs/core/threading_base/include/hpx/threading_base/register_thread.hpp#L28-L52
I should add that previously the callbacks were never executed, so using add_thread_exit_callback would cause an assertion failure later on thread destruction (L163), due to this sequence of calls:
https://github.com/STEllAR-GROUP/hpx/blob/e7c31a49dd4e550a61134e498f3ecfb585d201cb/libs/core/threading_base/src/thread_data.cpp#L107-L111
https://github.com/STEllAR-GROUP/hpx/blob/e7c31a49dd4e550a61134e498f3ecfb585d201cb/libs/core/threading_base/src/thread_data.cpp#L157-L166
@hkaiser Let me know if this is a good idea, and whether you'd like me to add a regression test.

Performance test reportHPX PerformanceComparisonBENCHMARKFORK_JOIN_EXECUTORPARALLEL_EXECUTORSCHEDULER_EXECUTORFor Each(=)??-InfoPropertyBeforeAfterHPX Commitdcb541576898d370113946ba15fb58c20c8325b23d750bba1e316e816a9f7e1c1cf58446c2db4f4eHPX Datetime2023-05-10T12:07:53+00:002023-08-11T13:11:35+00:00Compiler/opt/apps/llvm/13.0.1/bin/clang++ 13.0.1/opt/apps/llvm/13.0.1/bin/clang++ 13.0.1Datetime2023-05-10T14:50:18.616050-05:002023-08-11T08:20:30.831731-05:00EnvfileClusternamerostamrostamHostnamemedusa08.rostam.cct.lsu.edumedusa08.rostam.cct.lsu.eduComparisonBENCHMARKNO-EXECUTORFuture Overhead - Create Thread Hierarchical - Latch=InfoPropertyBeforeAfterHPX Commitdcb541576898d370113946ba15fb58c20c8325b23d750bba1e316e816a9f7e1c1cf58446c2db4f4eHPX Datetime2023-05-10T12:07:53+00:002023-08-11T13:11:35+00:00Compiler/opt/apps/llvm/13.0.1/bin/clang++ 13.0.1/opt/apps/llvm/13.0.1/bin/clang++ 13.0.1Datetime2023-05-10T14:52:35.047119-05:002023-08-11T08:22:44.551663-05:00EnvfileClusternamerostamrostamHostnamemedusa08.rostam.cct.lsu.edumedusa08.rostam.cct.lsu.eduComparisonBENCHMARKFORK_JOIN_EXECUTOR_DEFAULT_FORK_JOIN_POLICY_ALLOCATORPARALLEL_EXECUTOR_DEFAULT_PARALLEL_POLICY_ALLOCATORSCHEDULER_EXECUTOR_DEFAULT_SCHEDULER_EXECUTOR_ALLOCATORStream Benchmark - Add(=)(=)(=)Stream Benchmark - Scale(=)==Stream Benchmark - Triad(=)=(=)Stream Benchmark - Copy(=)=(=)InfoPropertyBeforeAfterHPX Commitdcb541576898d370113946ba15fb58c20c8325b23d750bba1e316e816a9f7e1c1cf58446c2db4f4eHPX Datetime2023-05-10T12:07:53+00:002023-08-11T13:11:35+00:00Compiler/opt/apps/llvm/13.0.1/bin/clang++ 13.0.1/opt/apps/llvm/13.0.1/bin/clang++ 13.0.1Datetime2023-05-10T14:52:52.237641-05:002023-08-11T08:23:01.362401-05:00EnvfileClusternamerostamrostamHostnamemedusa08.rostam.cct.lsu.edumedusa08.rostam.cct.lsu.eduExplanation of SymbolsSymbolMEANING=No performance change (confidence interval within ±1%)(=)Probably no performance change (confidence interval within ±2%)(+)/(-)Very small performance improvement/degradation (≤1%)+/-Small performance improvement/degradation (≤5%)++/--Large performance improvement/degradation (≤10%)+++/---Very large performance improvement/degradation (>10%)?Probably no change, but quite large uncertainty (confidence interval with ±5%)??Unclear result, very large uncertainty (±10%)???Something unexpected…

Let me know if this is a good idea, and whether you'd like me to add a regression test.

I'd appreciate it if you added a regression test. Thanks!

Performance test reportHPX PerformanceComparisonBENCHMARKFORK_JOIN_EXECUTORPARALLEL_EXECUTORSCHEDULER_EXECUTORFor Each(=)??-InfoPropertyBeforeAfterHPX Commitdcb541576898d370113946ba15fb58c20c8325b25bb36242861a2a24d4756c6eada9c6208f7df91aHPX Datetime2023-05-10T12:07:53+00:002023-08-11T19:23:41+00:00Datetime2023-05-10T14:50:18.616050-05:002023-08-11T14:30:30.574652-05:00EnvfileHostnamemedusa08.rostam.cct.lsu.edumedusa08.rostam.cct.lsu.eduClusternamerostamrostamCompiler/opt/apps/llvm/13.0.1/bin/clang++ 13.0.1/opt/apps/llvm/13.0.1/bin/clang++ 13.0.1ComparisonBENCHMARKNO-EXECUTORFuture Overhead - Create Thread Hierarchical - Latch=InfoPropertyBeforeAfterHPX Commitdcb541576898d370113946ba15fb58c20c8325b25bb36242861a2a24d4756c6eada9c6208f7df91aHPX Datetime2023-05-10T12:07:53+00:002023-08-11T19:23:41+00:00Datetime2023-05-10T14:52:35.047119-05:002023-08-11T14:32:46.088755-05:00EnvfileHostnamemedusa08.rostam.cct.lsu.edumedusa08.rostam.cct.lsu.eduClusternamerostamrostamCompiler/opt/apps/llvm/13.0.1/bin/clang++ 13.0.1/opt/apps/llvm/13.0.1/bin/clang++ 13.0.1ComparisonBENCHMARKFORK_JOIN_EXECUTOR_DEFAULT_FORK_JOIN_POLICY_ALLOCATORPARALLEL_EXECUTOR_DEFAULT_PARALLEL_POLICY_ALLOCATORSCHEDULER_EXECUTOR_DEFAULT_SCHEDULER_EXECUTOR_ALLOCATORStream Benchmark - Add(=)(=)(=)Stream Benchmark - Scale(=)==Stream Benchmark - Triad(=)(=)(=)Stream Benchmark - Copy(=)(=)(=)InfoPropertyBeforeAfterHPX Commitdcb541576898d370113946ba15fb58c20c8325b25bb36242861a2a24d4756c6eada9c6208f7df91aHPX Datetime2023-05-10T12:07:53+00:002023-08-11T19:23:41+00:00Datetime2023-05-10T14:52:52.237641-05:002023-08-11T14:33:02.897976-05:00EnvfileHostnamemedusa08.rostam.cct.lsu.edumedusa08.rostam.cct.lsu.eduClusternamerostamrostamCompiler/opt/apps/llvm/13.0.1/bin/clang++ 13.0.1/opt/apps/llvm/13.0.1/bin/clang++ 13.0.1Explanation of SymbolsSymbolMEANING=No performance change (confidence interval within ±1%)(=)Probably no performance change (confidence interval within ±2%)(+)/(-)Very small performance improvement/degradation (≤1%)+/-Small performance improvement/degradation (≤5%)++/--Large performance improvement/degradation (≤10%)+++/---Very large performance improvement/degradation (>10%)?Probably no change, but quite large uncertainty (confidence interval with ±5%)??Unclear result, very large uncertainty (±10%)???Something unexpected…

retest lsu

bors merge


Is ClientSecret a valid parameter to instantiate CrmServiceClient?

[Enter feedback here]
With document - connectionstring i don't see ClientSecret as a parameter in order to instantiate CrmServiceClient via OAuth auth, but we are able to pass in the app secret and successfully get an instance. Can someone help to elaborate?

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

ID: 4430aad2-5ce8-bc18-e409-009df2ecdf55
Version Independent ID: 6c82245f-9faa-e4f8-d33e-4344c3bea1b4
Content: Use connection strings in XRM tooling to connect to Dynamics 365 Customer Engagement (on-premises) (Developer Guide for Dynamics 365 Customer Engagement)
Content Source: ce/customerengagement/on-premises/developer/xrm-tooling/use-connection-strings-xrm-tooling-connect.md
Service: ****
GitHub Login: @Nkrb
Microsoft Alias: nabuthuk

@vFrankZhang Closing this issue here. Following up with you in our emails.


Internal Server Error when exporting a dataset

My actions before raising this issue

[x] Read/searched the docs
[x] Searched past issues

Expected Behaviour

Generated dataset archive should be sent to the client for download.
Current Behaviour

On chrome a "dataset.html" is attemped to be downloaded, but fails with "Failed - Server problem".
Possible Solution

Python 3 renamed the unicode type to str, the old str type has been replaced by bytes. Seems like the uderlying lib is still using unicode which fails in python3 installations. Perhaps a newer version of the lib should be used? django-sendfile2 might be an option, but have not tested it yet.
Steps to Reproduce (for bugs)

Create a project
Create a Task
Export task dataset (including images)

Context

Just exporting a previously created dataset.
 Kubernetes
Operating System and version (e.g. Linux, Windows, MacOS): Linux
Code example or link to GitHub repo or gist to reproduce problem:
Other diagnostic information / logs:
Logs from cvat_server container

ERROR - 2022-02-23 12:17:13,334 - log - Internal Server Error: /api/v1/tasks/2/dataset
 Traceback (most recent call last):
   File "/opt/venv/lib/python3.8/site-packages/django/core/handlers/exception.py", line 47, in inner
     response = get_response(request)
   File "/opt/venv/lib/python3.8/site-packages/django/core/handlers/base.py", line 181, in _get_response
     response = wrapped_callback(request, *callback_args, **callback_kwargs)
   File "/opt/venv/lib/python3.8/site-packages/django/views/decorators/csrf.py", line 54, in wrapped_view
     return view_func(*args, **kwargs)
   File "/opt/venv/lib/python3.8/site-packages/rest_framework/viewsets.py", line 114, in view
     return self.dispatch(request, *args, **kwargs)
   File "/opt/venv/lib/python3.8/site-packages/rest_framework/views.py", line 505, in dispatch
     response = self.handle_exception(exc)
   File "/opt/venv/lib/python3.8/site-packages/rest_framework/views.py", line 465, in handle_exception
     self.raise_uncaught_exception(exc)
   File "/opt/venv/lib/python3.8/site-packages/rest_framework/views.py", line 476, in raise_uncaught_exception
     raise exc
   File "/opt/venv/lib/python3.8/site-packages/rest_framework/views.py", line 502, in dispatch
     response = handler(request, *args, **kwargs)
   File "/home/django/cvat/apps/engine/views.py", line 902, in dataset_export
     return _export_annotations(db_instance=db_task,
   File "/home/django/cvat/apps/engine/views.py", line 1616, in _export_annotations
     return sendfile(request, file_path, attachment=True,
   File "/opt/venv/lib/python3.8/site-packages/sendfile/__init__.py", line 68, in sendfile
     response = _sendfile(request, filename, mimetype=mimetype)
   File "/opt/venv/lib/python3.8/site-packages/sendfile/backends/xsendfile.py", line 6, in sendfile
     response['X-Sendfile'] = unicode(filename).encode('utf-8')
 NameError: name 'unicode' is not defined

Seems like this is going to be solved in 2.0.0

@japel correct, fixed in the develop


support robots.txt

It would be nice, if pyspider would respect robots.txt of crawled sites.

I will consider how to implement the mechanism


Move Dev Console to use the live devfile registry

Which area this feature is related to?
/area library
Which functionality do you think we should add?
The current devfile sample dev preview of ODC is using a hardcoded list of samples. This item will move the backend of the ODC to start doing live queries on the community devfile registry (https://registry.devfile.io/).
Why is this needed? Is your feature request related to a problem?
This is part of the phased implementation of the devfile dev preview support in ODC. The registry support on samples was not available yet at the time when the dev preview was first implemented. Moving to registry support will allow ODC to reflect the latest samples from the registry.
Detailed description:
See above
Describe the solution you'd like
This item includes:

[x] Changing the backend of ODC to do a live query on a provided list of devfile registry, the API will merge the results to provide a single merged list of devfile entries to the caller
[x] Moving the ODC devfile sample in devfile-samples org to start using devfile 2.2 and make it devfile 2.2 compliant
[x] Moving to the latest devfile library that has both devfile 2.2 support
[x] Propagate the provider information (new in devfile 2.2 spec) to the ODC which currently has UI to provide that information (odc issue to track: https://github.com/openshift/console/issues/10664)
[ ] Change the samples to reference the parent stacks instead of keeping a copy of the parent devfile (https://github.com/devfile/api/issues/715)

Describe alternatives you've considered
Currently using a hardcoded list of sample
Additional context

PR on ODC backend work: https://github.com/openshift/console/pull/10394

Closing since #715 is used to track the remaining piece


Is this repo still maintained?

Simple question: is this repo still maintained? There's been no new version in the last 12+ months, several dependabot PRs open and like no activity. Is it worth considering it?

👋 @LunaticMuch - yes this repo is still maintained. We've been keeping an eye out for any big issues that come across, but definitely haven't been doing a good job of keeping the repo healthy. We have plans to address this very soon but in the meantime please know that Apollo fully supports this project.


[95] Properly adds the shapes of the tensors to the graph.

This PR adds the shapes of the tensors in the protobuf, following the recommendations in #95
TODO: see if the hints can be dropped. This can be tested with a dynamic placeholder.

Looks reasonable. Is there a test that makes sure this works?

@sueann currently, the issue manifests itself with complex graphs. It should be very visible once the tensor shape hints are removed. I am going to merge this PR, but the issue should be investigated further.


Computation Output

Problem: Computation output isn’t useful for researchers
Solution: Add tables to computation results
Related: (Merge these first and bump their package.json versions!)

MRN-Code/decentralized-laplacian-ridge-regression#10
MRN-Code/decentralized-single-shot-ridge-regression#11

#131 merged rebased versions of these refs. Closing.


Washing Machine Hoover H-Wash 550 H5WPB4 27BC8/1-S

wm_13307.zip

Thanks for sharing! :slightly_smiling_face:


[WIP] Automatic tests.

I think it was discussed several times that it would be nice to have some automatic test system that would check for regression. It's common for any software project to have such a system. Unfortunately when it comes to an embedded system it's no that trivial as it requires some hardware manipulations.
Here's my attempt to bring some basic automatic testing to EOR. It's currently in very draft state.
The idea is to have one or two modules connected to the host PC. Each test is built and flashed into the module. Than a python script checks the serial output of the test. The script will report with its return code whether the test succeeded or failed.
There's a support for several modules that are tested simultaneously. For example to test wifi communication two modules connect to each other and send/receive test data (wifi basic test).
The modules should have automatic reset/boot-mode support like nodemcu modules. In order to be flashed without pressing any buttons on the device. I currently don't have such modules :(
Such testing system allows easier rewriting of some binary libraries as a test can be written first and than rewritten implementation can be verified. Or just for general regression testing during optimization or other improvement.
Usage
To build all the tests run make in the test directory:
cd tests
make

To build and run all the tests run make test in the test directory:
cd tests
make test

To build and run a single test run make flash run_test in the specific test directory:
cd tests/spiffs
make flash run_test

For example:
> make flash run_test
   ...
Read speed: 732562 bytes/s
Write speed: 50123 bytes/s
Test finished successfully
Test: SUCCESS

Very nice @sheinz, I will go through this in detail as soon as possible.

@sheinz, this looks pretty good to me!
This probably predates your involvement with the project, but I also started some automated test support:
https://github.com/SuperHouse/esp-open-rtos/tree/feature/tests
I never finished it though (there are working tests that run on a single ESP, but multi-ESP tests I never finished implementing).
You can probably consider that branch "abandoned" now, I'm not going to pick it back up. But there might be some bits of code or tests there that you would find useful to roll into your efforts. Not a problem if you don't want to dig through it, though. :)
Nice work! :)

Closed in favour of https://github.com/SuperHouse/esp-open-rtos/pull/253


Component Popover

Expose a component just like the https://axafrance.github.io/react-toolkit/latest/storybook/index.html?path=/story/components-popover--default

Alright, closing


Unnecessary PuTTY Icons in taskbar

In 1.4.08, I get a Putty taskbar icon for every session I have open.  Related to #475.

<IP_ADDRESS>, Win7 x64, I also get this every time.
Here's what I see in the log viewer
2016-03-27 01:55:10,853 INFO            SuperPuTTY - Opening putty session, id=my.host.com
2016-03-27 01:55:10,862 INFO        PuttyStartInfo - Putty Args: '-ssh -P 22 -load "my.host.com"  -l root my.host.com'
2016-03-27 01:55:10,939 WARN      ApplicationPanel - Unable to get handle for process on first try.  Polling 10 s for handle.
2016-03-27 01:55:11,093 INFO      ApplicationPanel - Successfully found handle via polling 153.0087 ms
2016-03-27 01:55:11,101 INFO      ApplicationPanel - MoveWindow [OnVisChanged   my.host.com] w=1341, h= 466, visible=True, success=True
2016-03-27 01:55:11,103 INFO      ApplicationPanel - MoveWindow [OnVisChanged   my.host.com] w=1341, h= 466, visible=True, success=True
2016-03-27 01:55:11,528 WARN         ctlPuttyPanel - Unable to SetFocusToChildApplication, my.host.com
2016-03-27 01:55:11,530 DEBUG           SuperPuTTY - STATUS: Opened session: my.host.com [SSH]

Another app called mtputty doesn't have this issue, but I'd prefer to use nice and more featureful SuperPutty. This issue is quite critical to my peace of mind, though.


chore(master): release 1.0.1

:robot: I have created a release beep boop
1.0.1 (2024-01-16)
Bug Fixes

update string interpolation syntax (215d9a5)

This PR was generated with Release Please. See documentation.

:robot: Release is at https://github.com/getvoicify/keywind/releases/tag/v1.0.1 :sunflower:


Add publication/dataset detail page sidebar

Enhancement
Add an icon sidebar on the right side in detail pages with 2 icons that open/toggle a drawer when clicked.

Biblio messages

Biblio message
Librarian tags (librarian only)
Librarian note (librarian only)

Add a "dot" to Biblio messages menu item when there is a message added to the Biblio message field.

Details

Author/editor name and date
Source (if applicable)
All identifiers with links to external sources
When published: link to biblio front office

Prototype
Out of scope: Activity history

Added:

librarian tags
librarian note

Suggestion for save: buttons for now until we have a coherent approach to saving everywhere (otherwise we will have different patterns which will be confusing).

Q: Can we remember whether the message tab was opened or closed by default?

Q: Can we remember whether the message tab was opened or closed by default?

@nics I started this a couple of weeks ago but very quickly bumped into the fact that this is needed for both publications and datasets so the question is: is there something we can do to make sure we don't have to build the sidebar twice? We talked about deduping other views but never took any steps AFAIK. I guess it would be good to finally kick this off before we create any new duplicate views.

@verheyenkoen you can dedup them if it isn't too much work

@nics I guess my question wasn't clear. How do we do that? I guess we need some kind of common structure for publications and datasets for that to be possible but I don't know how to achieve a base interface in Go that can be both. As I don't really know how this is done, I can't really estimate the effort for that.

That was my point, making a common interface isn't worth the effort now, the next version already solves that. But if you can easily avoid some duplication now by making the url configurable and passing a common args to the template, then go ahead.

But I guess that requires mapping the common fields on a new struct type before passing it to the view?

Discussion yesterday:
Saving behaviour in the modals in the detail panel VS the save buttons in the side panel: might result into information loss.
@verheyenkoen can you confirm / add any thoughts?

Option 1: We assume people have to save the side panel changes.

Upside:

it's already a few clicks less than the Biblio messages tab
it's consequent behaviour

Downside: if they open a detail panel modal and click save, without saving the edits they made in the side panel, the edits in the side panel will disappear.

Option 1.1: We explicitly ask on blur whether the user wants to undo the changes they made.

Upside: we make consequences explicit.
Downside:

A lot of small tweaking = a lot of maintenance.
Requires customisation we might throw out.

Option 2: We autosave whatever happens in the Biblio messages, librarian tags an librarian notes

Upside: there will be no saving issue
Downside: our saving behaviour will differ in the application, causing mixed expectations which can cause frustration.

There is no autosave anywhere else. This is introducing different behaviour in one application, which can result in different problems.
Introducing autosave is not ideal right now. If we are going to tackle the full page, we might land on different behaviour, and taking that away afterwards is going to be hard.


Option 'disable-interactive-auth' is unknown.

Build
https://dev.azure.com/dnceng/internal/_build/results?buildId=2472121&view=logs&j=5c24930e-0ae1-546c-7353-8c3b261a7651&t=90d1b216-f511-5956-2766-23f92c36b673&l=120
Option 'disable-interactive-auth' is unknown.
...
##[error]PowerShell exited with code '1'.

Build leg reported
No response
Pull Request
No response
Known issue core information
Fill out the known issue JSON section by following the step by step documentation on how to create a known issue
 {
    "ErrorMessage" : "Option 'disable-interactive-auth' is unknown.",
    "BuildRetry": false,
    "ErrorPattern": "",
    "ExcludeConsoleLog": false
 }

@dotnet/dnceng

Release Note Category

[ ] Feature changes/additions
[ ] Bug fixes
[ ] Internal Infrastructure Improvements

Release Note Description
Additional information about the issue reported
No response

Known issue validation
Build: :mag_right: https://dev.azure.com/dnceng/internal/_build/results?buildId=2472121
Error message validated: [Option 'disable-interactive-auth' is unknown.]
Result validation: :white_check_mark: Known issue matched with the provided build.
Validation performed at: 6/12/2024 8:00:28 PM UTC

Report

Build
Definition
Step Name
Console log
Pull Request

2472727
dotnet-cecil
Publish Using Darc
Log

2472716
dotnet-extensions
Publish Using Darc
Log

2472647
dotnet-wcf
Publish Using Darc
Log

2472577
dotnet-diagnostics
Publish Using Darc
Log

2472121
dotnet-runtime
Publish Using Darc
Log

Summary

24-Hour Hit Count
7-Day Hit Count
1-Month Count

5
5
5

@jeffschwMSFT please update Arcade and this will get fixed.

I can see that runtime's arcade has already been updated and this won't flag any more builds now.

All repos here got their arcade updates and will no longer fail. All have a successful publishing since then. Closing.


Getting SkiaSharp to work with open shift container.

I am currently trying to get Skiasharp to work on an openshift container that was provided to me. There are certain restrictions placed on the container and because of that I do not have the rights to change the image at all and hence I cannot add any additional configurations to it. The problem that I have been running into seems to be that I keep getting the dll not found issue and the only workaround

recommended in the forums was including the .so, which I did and I still seem to be getting the same issue.
There also seems to be some additional dependency on libfontconfig but I have no way of installing that on my container because of the restrictions placed on it. Is there any viable solution that you could suggest?
Any help would be greatly appreciated.

To anyone having a similar problem @mattleibow came through with the binary. Here is a link to it:
Source: https://github.com/mattleibow/SkiaSharp-Docker/tree/no-fontconfig
Binary: https://github.com/mattleibow/SkiaSharp-Docker/raw/no-fontconfig/libSkiaSharp.so
Once you have that integrated, the library should work as expected.

This is the PR that I will be merging soon:
https://github.com/mono/SkiaSharp/pull/821


Starship status always shows command success with shell integration enabled

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

VS Code Version: 1.91.0
OS Version: Windows 11

Steps to Reproduce:

Install starship 1.19.0
Enable the status
Run a command that should fail
The return code seen by starship is 0, so the command is shown as successful

With shell integration enabled:

With shell integration disabled:

Thanks.

Note: the same occurs with the oh-my-posh prompt.


Missing libsodium

Sodium is a new, easy-to-use software library for encryption, decryption, signatures, password hashing and more. It is a portable, cross-compilable, installable, packageable fork of NaCl, with a compatible API, and an extended API to improve usability even further.
Upstream: https://github.com/jedisct1/libsodium
License: ISC
(FYI, NaCl was placed in public domain)
Projects that using libsodium or its binding:
shadowsocks
dnscrypt-proxy

@izmntuk
多谢推荐项目（已加入）。
由于 libsodium 派生自 NaCl，俺顺便把 NaCl 也加入。


A typo on the netty home page

Expected behavior
Netty is an NIO client server framework....
Actual behavior
Netty is a NIO client server framework...

I think both are correct since NIO can be pronounced either 'nee-oh' or 'en-eye-oh'.

Pushed a fix


#269: Update contributing and add pr issue templates

Fixes #269
Checklist before requesting a review

[x] Have you checked to ensure there aren't other open Pull Requests for the same update/change?
[x] Have you followed the guidelines in CONTRIBUTING.md?
[ ] Do your commit messages follow guidelines?
[x] Have you performed a self-review?
[x] Have you added yourself into CONTRIBUTORS.md?

Request review from @satwikkansal

Hey @satwikkansal . I've decided to prepare templates and update CONTRIBUTING before creating small issues for fixing markdown format and migrating to embedded snippets. Could you please review and suggest some changes if necessary?

Hey @satwikkansal . I've decided to prepare templates and update CONTRIBUTING before creating small issues for fixing markdown format and migrating to embedded snippets. Could you please review and suggest some changes if necessary?

@nifadyev just did and left some comments, the overarching theme is that we shouldn't make it time consuming for a new contributor to set up an issue or open a PR, as it demotivates them to share something that might be of interest to us. Once they create an issue we can ask for extra details then, that approach has better conversion :)

we shouldn't make it time consuming for a new contributor to set up an issue or open a PR

100%. The problem is to find the balance between ease of creating issue/PR and avoiding duplicates and extra work for reviewers/maintainers. Let's start as you suggested and then adopt if necessary.
Anyway, a good maintained project should not have a lot of open issues and PRs, so points from checklist are not so important

Hey @satwikkansal, I've fixed your remarks and added PR and issue templates for bug reports and translations

@nifadyev, thanks! One minor comment, everything else looks good

Thanks, @satwikkansal , comment about Explanation section has been added

Thanks @nifadyev, I merged!


Support default branch for the workflow. Part 1

Major purpose of the PR is to support master and main default branch instead of hard-coded master.
Changes includes:

Markdown files: description which mention master as default branch. Changed to default master.
Yaml files: Use set-default-branch.yml and use variables $(DefaultBranch) to replace hard code one.
powershell script: Try to use git command to fetch the default ones.
ci.yml: Added main as another trigger branch.

Test pipeline on mirror-repo:
https://dev.azure.com/azure-sdk/internal/_build/results?buildId=804414&view=logs&s=96ac2280-8cb4-5df5-99de-dd2da759617d

The following pipelines have been queued for testing:java - templatejava - template - testsjs - templatenet - templatenet - template - testspython - templatepython - template - tests You can sign off on the approval gate to test the release stage of each pipeline. See eng/common workflow

The following pipelines have been queued for testing:java - templatejava - template - testsjs - templatenet - templatenet - template - testspython - templatepython - template - tests You can sign off on the approval gate to test the release stage of each pipeline. See eng/common workflow

The following pipelines have been queued for testing:java - templatejava - template - testsjs - templatenet - templatenet - template - testspython - templatepython - template - tests You can sign off on the approval gate to test the release stage of each pipeline. See eng/common workflow

No description provided.

Would be good to get a bullet pointed summary in these PRs about the various things you've touched.

The following pipelines have been queued for testing:java - templatejava - template - testsjs - templatenet - templatenet - template - testspython - templatepython - template - tests You can sign off on the approval gate to test the release stage of each pipeline. See eng/common workflow

The following pipelines have been queued for testing:java - templatejava - template - testsjs - templatenet - templatenet - template - testspython - templatepython - template - tests You can sign off on the approval gate to test the release stage of each pipeline. See eng/common workflow

The following pipelines have been queued for testing:java - templatejava - template - testsjs - templatenet - templatenet - template - testspython - templatepython - template - tests You can sign off on the approval gate to test the release stage of each pipeline. See eng/common workflow

This all looks good. I'm OK to see this merged in. We should consider timing (now is good because we are mid-release. But we should make sure that we publish the template packages after it is merged in just to validate everything is working end-to-end and be prepared to back-out if something goes wrong.
Well done @sima-zhu

For the next step, I am going to go through the tier lang repo for hard-coded master. Meanwhile, worked with each feature owner for end-to-end testing.

The following pipelines have been queued for testing:java - templatejava - template - testsjs - templatenet - templatenet - template - testspython - templatepython - template - tests You can sign off on the approval gate to test the release stage of each pipeline. See eng/common workflow

The following pipelines have been queued for testing:java - templatejava - template - testsjs - templatenet - templatenet - template - testspython - templatepython - template - tests You can sign off on the approval gate to test the release stage of each pipeline. See eng/common workflow

/check-enforcer override


Added a double validation for multipath device WWN discovered via multipath -ll

…ests.

This change is

Coverage increased (+0.06%) to 54.995% when pulling fe3ce0187356feec9c2758db3017441c4f2aa4c5 on fix/validate_wwn_using_sg_inq into 0b13f4c9ce4c32e8dff48436cca9fe59c7c5122a on dev.

Review status: 0 of 3 files reviewed at latest revision, 1 unresolved discussion.

remote/mounter/block_device_utils/mpath.go, line 87 at r1 (raw file):

	}
	if _, err = b.exec.Stat(mpath); err != nil {
		return "", b.logger.ErrorRet(err, "Stat failed")

I know it's not in this code change but can we do better with this error message? Stat failed for volume [%s] ?

Comments from Reviewable

Review status: 0 of 3 files reviewed at latest revision, 1 unresolved discussion.

Comments from Reviewable

Review status: 0 of 3 files reviewed at latest revision, 6 unresolved discussions.

remote/mounter/block_device_utils/errors.go, line 47 at r1 (raw file):

}

type wrongVolumeFoundError struct {

rename to wrongDeviceFoundError

remote/mounter/block_device_utils/errors.go, line 49 at r1 (raw file):

type wrongVolumeFoundError struct {
	volName string
	reqVolName string

Add also the dev path as argument

remote/mounter/block_device_utils/errors.go, line 52 at r1 (raw file):

}
func (e *wrongVolumeFoundError) Error() string {
	return fmt.Sprintf("volume [%v] is not found. instead found vol [%s].", e.reqVolName, e.volName)

fix it to
"multipath device [%s] was found as WWN [%v] via multipath -ll command, BUT sg_inq identify this device as different WWN [%s]. Check your multipathd."

remote/mounter/block_device_utils/mpath.go, line 79 at r1 (raw file):

GetWwnByScsiInq
please call the var, SqInqWwn

remote/mounter/block_device_utils/mpath.go, line 84 at r1 (raw file):

	}
	if strings.ToLower(wwn) != strings.ToLower(volumeWwn){
		return "", b.logger.ErrorRet(&wrongVolumeFoundError{wwn, volumeWwn}, "failed")

please add here also dev as argument to see the whole picture not only the WWN

Comments from Reviewable

Review status: 0 of 3 files reviewed at latest revision, 6 unresolved discussions.

remote/mounter/block_device_utils/errors.go, line 47 at r1 (raw file):
Previously, shay-berman wrote…
rename to wrongDeviceFoundError

Done.

remote/mounter/block_device_utils/errors.go, line 49 at r1 (raw file):
Previously, shay-berman wrote…
Add also the dev path as argument

Done.

remote/mounter/block_device_utils/errors.go, line 52 at r1 (raw file):
Previously, shay-berman wrote…
fix it to
"multipath device [%s] was found as WWN [%v] via multipath -ll command, BUT sg_inq identify this device as different WWN [%s]. Check your multipathd."

Done.

remote/mounter/block_device_utils/mpath.go, line 79 at r1 (raw file):
Previously, shay-berman wrote…

GetWwnByScsiInq
please call the var, SqInqWwn

Done.

remote/mounter/block_device_utils/mpath.go, line 84 at r1 (raw file):
Previously, shay-berman wrote…
please add here also dev as argument to see the whole picture not only the WWN

Done.

Comments from Reviewable

Coverage increased (+0.08%) to 55.014% when pulling e0b0342e8de68fefeaa8fb246d3206727eb24247 on fix/validate_wwn_using_sg_inq into 0b13f4c9ce4c32e8dff48436cca9fe59c7c5122a on dev.

Review status: 0 of 3 files reviewed at latest revision, 5 unresolved discussions.

remote/mounter/block_device_utils/block_device_utils_test.go, line 177 at r2 (raw file):

			Expect(err.Error()).To(MatchRegexp(cmdErr.Error()))
		})
		It("Discover fails if wrong wwn is found", func() {

very good UT

remote/mounter/block_device_utils/block_device_utils_test.go, line 199 at r2 (raw file):

			cmd, args = fakeExec.ExecuteArgsForCall(1)
			Expect(cmd).To(Equal("sg_inq"))
			Expect(args).To(Equal([]string{"-p",  "0x83", "/dev/mapper/mpath"}))

not must, but its better to use the "/dev/mapper/" + result

remote/mounter/block_device_utils/errors.go, line 52 at r1 (raw file):
Previously, tzurE wrote…
Done.

Do you think we should tell Dima to add this string to the troubleshooting section in the userguide?

remote/mounter/block_device_utils/mpath.go, line 87 at r1 (raw file):
Previously, ranhrl (Ran Harel) wrote…
I know it's not in this code change but can we do better with this error message? Stat failed for volume [%s] ?

the err it should will be find not found. so i think we are ok here.  But yes we can add here "Stat fileX failed".
BTW this is the same Stat we should do after remove the multipath device.

Comments from Reviewable

Review status: 0 of 3 files reviewed at latest revision, 5 unresolved discussions.

remote/mounter/block_device_utils/block_device_utils_test.go, line 199 at r2 (raw file):
Previously, shay-berman wrote…
not must, but its better to use the "/dev/mapper/" + result

Done.

remote/mounter/block_device_utils/errors.go, line 52 at r1 (raw file):
Previously, shay-berman wrote…
Do you think we should tell Dima to add this string to the troubleshooting section in the userguide?

yes

Comments from Reviewable


Document the bind option for both uvicorn/gunicorn

(so that the fractal-server instance is accessible from all machines in a private network)

For the record, this is obtained as
fractalctl start --host <IP_ADDRESS> --port 8000 ...

or
gunicorn --bind <IP_ADDRESS>:8000 ...

(with the relevant part being the <IP_ADDRESS> ip).
I'll keep this information here, for the moment, and then we should at some point review the deployment docs with all the update knowledge coming from ongoing activity.


optionally including multiple class names

Hi - first, thanks for maud, it's awesome ;-).
I realise I can do div.classA.classB to set multiple classes and checked[1==2] to optionally include an empty attribute.
However, how can I optionally add a class to the existing classes for an element, for example, I want either div.classA.classB.selected or div.classA.classB.
At the moment I am constructing the CSS valid classname outside of the html! code and referencing it, but it is clunky.
I've tried :```
div @if 1==2 {
class="classA classB selected"
} @else {
class="classA classB"
}
but it doesn't work.

I'd really like to be able to do something like `div.classA.classB class[1==2]="selected"` for example.

Programatically building up CSS strings seems like a common use case, so I wonder if I'm missing something :-). Thanks!

Hi @yatesco!
Sorry for the delayed response.
You can use that [] syntax on classes too. See the last example here: https://maud.lambda.xyz/splices-toggles.html#toggles-foo
Does that help?

thanks @lambda-fairy, and no problem at all :-)
It helps some, but unfortunately you can't specify the class property more than once. Does p.classA[booleanCheck].classB[booleanCheck].classC[booleanCheck] work?

Yes, that should work

perfect - thank you :-)


[FEATURE] Add YouTube playlist

I want to add a YouTube playlist for a11y from the channel Google Developers in the README.md file.
Link to the playlist:- https://youtube.com/playlist?list=PLNYkxOF6rcICWx0C9LVWWVqvHlYJyqw7g

That's a brilliant idea. Please go ahead :)


zendesk_messaging: 2.8.0 cannot be run on Android

When I first tried zendesk_messaging: 2.8.0, I got an error something like the following
"Unexpected end of ZLIB input stream",
(I forgot the exact but something along the line)
Since then, whenever I tried to run again, it will automatically close the application and show "Lost connection to device".
Now I'm using 2.7.7, it's working again.

Hi! Please try new version 2.9.1.

Please try the latest version.
If u still meet the problem, please re-open it or create a new issue.


Cant implement

Would somebody show me how to implement the code thanks
my code looks like this, what am im missing?

Assuming you have all the Javascript files in place, you probably don't have an element corresponding to selector #minimal .demo.
Add

 and change #minimal .demo to #bracket.


Binary Reader performance improvement

Description of changes:
This PR is created to improve binary reader and parser performance.
Changes:

IonBinaryReader: removed duplicate maxId check from getSymbolString().
IonParserBinaryRaw: Use bit shift for multiplication for numberOfBytes less than 4 in  _readUnsignedIntAsNumberFrom().

Test:
Verified performance using benchmarkjs and flame graphs from v8 profiling with large binary ion file as input.
Test Results:
Here times are measured using mean time of sample runs while Benchmarking(secs)

Time(before enhancement)
Possible enhancement
Time (after enhancement)
% decrease in Time

10.4261319358
IonBinaryReader (removed duplicate condition checks)
9.9510428
4.556715172274926

10.3857747234
IonParserBinaryRaw (using bitshift for multiplication)
10.3409678528
0.431425404395172

10.5577046098
Both improvements together
9.9062553234
6.170368564728586

Per offline conversation, the ion-test-driver build is currently failing because ion-test-driver does an error message equality check on an Error that this PR changes.


Added dotUltimate version 2021.1.2.

[x] Have you signed the Contributor License Agreement?
[x] Have you checked that there aren't other open pull requests for the same manifest update/change?

[x] Does your manifest conform to the 1.0 schema?

Note: <path> is the name of the directory containing the manifest you're submitting.

Microsoft Reviewers: Open in CodeFlow

How did it fail? It worked fine for me in testing (it takes like 20 minutes though).


Tofi not opening anything

I cant get tofi to open apps for me
If i launch tofi through the terminal i even see it type the program in the terminal for example "librewolf" but nothing happens

drun-launch=true
you need this option in your config to actually launch


Add uninstall method to unregister Vuex ORM from Vuex Store

I want to use vuex-orm in vuex module,can i unregister vuex-orm when router change?
  plugins: [(store) => {
    store.subscribe((mutation, state) => {
      if (mutation.type === 'route/ROUTE_CHANGED') {
        if (state.route && state.route.name === 'chapter') {
          store.registerModule('chapter', chapter);
          VuexORM.install(database)(store);
        } else {
          store.unregisterModule('chapter');
          // how to uninstall VuexORM ?
        }
      }
    });
  }],

Nice request! Currently we don't have unregister option in Vuex ORM, but could you try this? I think it would work
store.unregisterModule('entities')

nice, it is work, thanks!

👍  I think it would be nice to have uninstall method so I'll change this issue title and make it feature request!
Thank you for the reporting : )

any update for this uninstall method.

for now store.unregisterModule(['entities', 'model'] doing the work.

@EslamNemr I think that's probably the best action to take for now. I don't think this is a huge priority at this time. Thinking about it, the installation would need to be abstract and rewritten if uninstalling was a possibility at entity level since the lib needs to register a wrapping module.


CVSL-1433 CVL Amend rules Nomis overrides take precedence

As per Andy's comments(Lets hold off setting the other dates in the database- as we would really need to do some kind of migration to back fill existing records which would be loads of work.) we are not adding PRRD, PED, or NPD to Licence table now and not reading them in both licence creation and data change events.


Clang Formatting

Add in a CI-CD action to clang-format files, as well as check for CRLF line endings and trailing whitespace
Add relevant test cases as well.

Couple of things:

I'd love to see smaller example files used. The example files alone for this PR are MASSIVE. In total this PR is adding roughly 60k lines. It's not exactly easy to review.
It would be nice to have a comment block at the top of the example files explaining how exactly they were generated and what the problem with the file is. While the names are nice, I can't imagine that you change 50k+ lines for formatting failures

Approving because I don't see any stand out problems but I obviously haven't read every single line in depth.

Couple of things:
1. I'd love to see smaller example files used. The example files alone for this PR are MASSIVE. In total this PR is adding roughly 60k lines. It's not exactly easy to review.

Replaced the files with short ones that are 200 lines long at most.
It was an intentional decision to select files from various FreeRTOS Repos:

To show that the formatting actions then also work for files we have.
To make my life easier by not needing to generate C or H files that had issues, as I just grabbed files that failed checks when implementing the clang-format action.
Additionally, certain FreeRTOS Files will fail to parse through uncrustify using uncrustify version .72. Where the idea was to grab long ones so that if you're testing the action itself we see that these long files will handle the new formatting changes, if one was changing the .clang-format file for example.
But not going to hold up this PR about this since both you and @paulbartell  have requested shorter files.

2. It would be nice to have a comment block at the top of the example files explaining how exactly they were generated and what the problem with the file is. While the names are nice, I can't imagine that you change 50k+ lines for formatting failures

Do you still feel this way with the current naming convention?
I have also added a README explaining the purpose of these files.


Disable Span Generations when tracing rate is set to 0

Description
Disable Span Generations when tracing rate is set to 0
Issue reference
Closes #1473
Checklist
Please make sure you've  completed the relevant tasks for this PR, out of the following list:

[x] Code compiles correctly
[ ] Created/updated tests
[x] Unit tests passing
[ ] End-to-end tests passing
[ ] Extended the documentation / Created issue in the https://github.com/dapr/docs/ repo: dapr/docs#[issue number]
[ ] Specification has been updated / Created issue in the https://github.com/dapr/docs/ repo: dapr/docs#[issue number]
[ ] Provided sample for the feature / Created issue in the https://github.com/dapr/docs/ repo: dapr/docs#[issue number]

/ok-to-test

Tested with samples, that spans are not generated when samplingRate is 0.
cc @youngbupark , @yaron2 , @amanbha


Merge pull request #168 from SpineEventEngine/fix-pom-generation

Fix the POM generation.

@armiol Sorry about that, testing another tool and opened this PR in error. Please ignore it.

@jjthurber no worries!


Error: Unable to get local issuer certificate

Hi,
I've configured the vault-sync.yaml file as follows (just leaving the fields with an actual value set) :

id: vault-sync
full_sync_interval: 3600 # 1h

# Source Vault configuration to sync secrets from.
src:
  # Vault URL
  url: http://source.vault:8200/

  # Vault Token auth method
  token: xxx.yyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyy

# Destination Vault configuration to sync secrets to.
dst:
  # Vault URL
  url: https://target.vault:8200/

  # Vault Token auth method
  token:xxx.yyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyy

I then try to run the install with :
docker run -it -v $PWD:/vault-sync pbchekin/vault-sync:0.8.0 vault-sync --config /vault-sync/vault-sync.yaml
Giving the output :
15:42:14 [INFO] Configuration from /vault-sync/vault-sync.yaml:
{
  "id": "vault-sync",
  "full_sync_interval": 3600,
  "bind": null,
  "src": {
    "url": "http://source.vault:8200/",
    "token": "***",
    "token_ttl": null,
    "token_max_ttl": null,
    "prefix": "",
    "backend": "secret"
  },
  "dst": {
    "url": "https://target.vault:8200/",
    "token": "***",
    "token_ttl": null,
    "token_max_ttl": null,
    "prefix": "",
    "backend": "secret"
  }
}
15:42:14 [INFO] Connecting to http://source.vault:8200/
15:42:14 [INFO] Audit device vault-sync exists: false
15:42:14 [INFO] Connecting to https://target.vault:8200/
15:42:14 [INFO] Token: TokenInfo { renewable: false, ttl: None, max_ttl: None }
15:42:14 [INFO] Plan: TokenInfo { renewable: false, ttl: None, max_ttl: None }
15:42:14 [ERROR] Failed to connect to https://target.vault:8200/: reqwest error: error sending request for url (https://target.vault:8200/v1/auth/token/lookup-self): error trying to connect: error:1416F086:SSL routines:tls_process_server_certificate:certificate verify failed:../ssl/statem/statem_clnt.c:1916: (unable to get local issuer certificate)
Error: Reqwest(reqwest::Error { kind: Request, url: Url { scheme: "https", cannot_be_a_base: false, username: "", password: None, host: Some(Domain("target.vault")), port: Some(8200), path: "/v1/auth/token/lookup-self", query: None, fragment: None }, source: hyper::Error(Connect, Ssl(Error { code: ErrorCode(1), cause: Some(Ssl(ErrorStack([Error { code: 337047686, library: "SSL routines", function: "tls_process_server_certificate", reason: "certificate verify failed", file: "../ssl/statem/statem_clnt.c", line: 1916 }]))) }, X509VerifyResult { code: 20, error: "unable to get local issuer certificate" })) })

The connection to the source Vault which is using HTTP doesn't have any issue (it would seem). However, the connection to the target Vault which is restricted to HTTPS only fails because the "local issuer certificate" cannot be accessed.
How could I provide the "local issuer certificate" when installing ?
P.S: I've tested connecting directly on the target Vault UI with the token I provided in vault-sync.yaml and was able to login.
Thank you.

Hey, thank you for reporting this. Do you need an option to provide a certificate, or, as a workaround` a kind of "insecure" mode (see for example https://curl.se/docs/manpage.html#-k) to skip verifying a sever certificate?

Hey, both options are actually interesting. The "insecure" mode for testing purposes comes in very handy, however for more "productive" environments the use of a proper secured connection still is a requirement for many.

Making sure that the .pem certificate file was also in my current directory which is mounted on the docker image and then passing the SSL_CERT_FILE environment variable to point to the .pem file on the docker image proved to be a workaround :
docker run -it -v $PWD:/vault-sync -e SSL_CERT_FILE='/vault-sync/.certs/company-ca-bundle.pem' pbchekin/vault-sync:0.8.0 vault-sync --config /vault-sync/vault-sync.yaml

output
09:51:52 [INFO] Configuration from /vault-sync/vault-sync.yaml:
{
  "id": "vault-sync",
  "full_sync_interval": 3600,
  "bind": null,
  "src": {
    "url": "http://source.vault:8200/",
    "token": "***",
    "token_ttl": null,
    "token_max_ttl": null,
    "prefix": "",
    "backend": "secret"
  },
  "dst": {
    "url": "https://target.vault:8200/",
    "token": "***",
    "token_ttl": null,
    "token_max_ttl": null,
    "prefix": "",
    "backend": "secret"
  }
}
09:51:52 [INFO] Connecting to http://source.vault:8200/
09:51:52 [INFO] Audit device vault-sync exists: false
09:51:52 [INFO] Connecting to https://target.vault:8200/
09:51:52 [INFO] Token: TokenInfo { renewable: false, ttl: None, max_ttl: None }
09:51:52 [INFO] Plan: TokenInfo { renewable: false, ttl: None, max_ttl: None }
09:51:52 [INFO] Dry run: false
09:51:52 [INFO] Token: TokenInfo { renewable: true, ttl: Some(2690330s), max_ttl: None }
09:51:52 [INFO] Plan: TokenInfo { renewable: true, ttl: Some(2690330s), max_ttl: None }
09:51:52 [INFO] FullSync worker started
09:51:52 [INFO] FullSync started
09:51:52 [INFO] Sync worker started
09:51:52 [WARN] Failed to list secrets in : Error in vault response: Vault request failed: Response { url: Url { scheme: "http", cannot_be_a_base: false, username: "", password: None, host: Some(Domain("source.vault")), port: Some(8200), path: "/v1/secret/metadata/", query: None, fragment: None }, status: 404, headers: {"cache-control": "no-store", "content-type": "application/json", "strict-transport-security": "max-age=31536000; includeSubDomains", "date": "Thu, 27 Jun 2024 09:51:52 GMT", "content-length": "14"} }, error message: `{"errors":[]}
Could not read vault response.`
09:51:52 [INFO] FullSync finished in 30ms
09:51:52 [INFO] Secrets created/updated: 0, deleted: 0

I still have another error retrieving the secrets from the source Vault but this is another issue that I have to check.

I have the same issue with an Openshift/ArgoCD deployment using the HELM implementation, I tried to create a secret with CA Cert of my company to implement it as a volume into the deployment. I tried to do the modification directly in the deployment but could not afford it.
I do not want to skip teh verification but want to add my CA, any quick fix related ?

Here is the logs:
07:08:48 [INFO] Connecting to https://vault.src/
07:08:48 [ERROR] Failed to connect to https://vault.src/: reqwest error: error sending request for url (https://vault.src/v1/auth/token/lookup-self): error trying to connect: error:1416F086:SSL routines:tls_process_server_certificate:certificate verify failed:../ssl/statem/statem_clnt.c:1916: (unable to get local issuer certificate)
Error: Reqwest(reqwest::Error { kind: Request, url: Url { scheme: "https", cannot_be_a_base: false, username: "", password: None, host: Some(Domain("vault.src")), port: None, path: "/v1/auth/token/lookup-self", query: None, fragment: None }, source: hyper::Error(Connect, Ssl(Error { code: ErrorCode(1), cause: Some(Ssl(ErrorStack([Error { code: 337047686, library: "SSL routines", function: "tls_process_server_certificate", reason: "certificate verify failed", file: "../ssl/statem/statem_clnt.c", line: 1916 }]))) }, X509VerifyResult { code: 20, error: "unable to get local issuer certificate" })) })

here is my values:
vault-sync:   vaultSync:     src:       url: https://vault.src/       backend: engine       token: xxx.xxxxxxxxxxxxxxxxxxxxxx     dst:       url: https://vault.dst/       backend: engine       token: xxx.xxxxxxxxxxxxxxxxxxxxxx

@lololastico74 it should be possible with the latest version (0.1.3) of the Helm chart:
First, you need to make sure you have a Kubernetes secret with the certificate, for example:
kubectl --namespace=vault-sync create secret generic certs --from-file=ca-bundle.pem=/path/to/certificate.pem

Then you need to provide the following additional chart values:
volumes:
  - name: certs
    secret:
      secretName: certs

volumeMounts:
  - mountPath: /certs
    name: certs
    readOnly: true

environmentVars:
  - name: SSL_CERT_FILE
    value: /certs/ca-bundle.pem

Let me know if that works for you.


[AKS] Update cli tests disable upgrade node image related tests

Current cli tests which involves node image upgrades are not meaningful. These tests is relying on a buggish behavior in upgradeNodeImageVersion API to function. If we are upgrading the node image of an agent pool which is already on the latest node image version, upgradeNodeImageVersion API will exit silently after very long validation on the async side.
Existing nodeimageupgrade related cli tests are all trying to upgrade the node image of clusters which is already on the latest image version. And it only checks if the agent pool goes into UpgradingNodeImageVersion provisioningState. It never verifies whether the upgrade succeeded or not, which make these tests meaningless.
After changing upgradeNodeImageVersion API to return 304 when agent pool is already on the latest version. These tests will stop function. Temporarily disable node image upgrade tests to unblock RP release. We will provide a short term solution in RP to re-enable these tests first. And after we have the API to create clusters with an older image, we will redo these tests to make real node image upgrades.
This checklist is used to make sure that common guidelines for a pull request are followed.

For new extensions:

[ ] My extension description/summary conforms to the Extension Summary Guidelines.

About Extension Publish
There is a pipeline to automatically build, upload and publish extension wheels.
Once your pull request is merged into main branch, a new pull request will be created to update src/index.json automatically.
You only need to update the version information in file setup.py and historical information in file HISTORY.rst in your PR but do not modify src/index.json.

AKS


IO::Socket::SSL missing

Fresh installation on Ubuntu.
GitHub is probably switching communication to https.
# Failed test 'can retrieve http://github.com/'
# at t/issue-7.t line 8
#501 Protocol scheme 'https' is only supported if IO::Socket::SSL is installed <URL:https://github.com/>

# Looks like you failed 1 test of 1
t/issue-7.t .....................
Dubious, test returned 1 (wstat 256, 0x100)
Failed 1/1 subtests
t/parse-url.t ................... ok
t/socket-sanity.t ............... ok
t/stringify-headers.t ........... ok

Test Summary Report
-------------------
t/issue-7.t                   (Wstat: 256 Tests: 1 Failed: 1)
  Failed test:  1
  Non-zero exit status: 1
Files=14, Tests=57, 26 wallclock secs ( 0.09 usr  0.02 sys + 17.00 cusr  1.36 csys = 18.47 CPU)
Result: FAIL
test stage failed for LWP::Simple: Tests failed
  in method throw at /home/hans/.rakudobrew/moar-nom/install/share/perl6/runtime/CORE.setting.moarvm line 1
  in method install at /home/hans/.rakudobrew/moar-nom/install/share/perl6/site/sources/582CB7486602954A4601BDCE5A0EAC54B05DA58A (Panda) line 185
  in block  at /home/hans/.rakudobrew/moar-nom/install/share/perl6/site/sources/582CB7486602954A4601BDCE5A0EAC54B05DA58A (Panda) line 257
  in method resolve at /home/hans/.rakudobrew/moar-nom/install/share/perl6/site/sources/582CB7486602954A4601BDCE5A0EAC54B05DA58A (Panda) line 251
  in sub MAIN at /home/hans/.rakudobrew/moar-nom/install/share/perl6/site/resources/E0D978079BB5081DE986D058BB8AB08252F05CC8 line 20
  in block <unit> at /home/hans/.rakudobrew/moar-nom/install/share/perl6/site/resources/E0D978079BB5081DE986D058BB8AB08252F05CC8 line 165

Failure Summary
----------------
Task::Star(
    *test stage failed for LWP::Simple: Tests failed)

That's a problem with LWP::Simple installation, not rakudobrew itself; feel free to report it there :)