Handlebars can't handle nested  tags

Issue Summary
If I have an  tag that has another  within it, in a theme, handlebars duplicate the first  under every nested tag.
Steps to Reproduce
So for example if I have:

        try

the output will be:

            try

Ghost Version
4.22
Node.js Version
14
How did you install Ghost?
managed
Database type
MySQL 5.7
Browser & OS version
any
Relevant log / error output
No response
Code of Conduct

[X] I agree to be friendly and polite to people in this repository

Nested anchor tags are not valid HTML. There’s no right/wrong/expected output in this case because you’re trying to do something that isn’t allowed within the HTML specification with no defined behaviour for when the spec is broken.
https://html.spec.whatwg.org/multipage/text-level-semantics.html#the-a-element

Content model:
Transparent, but there must be no interactive content descendant, a element descendant, or descendant with the tabindex attribute specified.

The a element can be wrapped around entire paragraphs, lists, tables, and so forth, even entire sections, so long as there is no interactive content within (e.g., buttons or other links).


Check for lack of HDMX and VDMX tables

Per https://forum.glyphsapp.com/t/why-doesnt-glyphsapp-output-hdmx-and-vdmx-tables/9316/3 we should WARN if these tables are in the fonts, and get them removed.
Perhaps even FAIL? But if it was FAIL it would be a low priority one.

I assume HDMX is pointless now, but is VDMX really deprecated? It seems it
is still useful, although I don't know if win apps use it.
On Tue, Aug 7, 2018, 2:02 PM Dave Crossland<EMAIL_ADDRESS>wrote:

Per
https://forum.glyphsapp.com/t/why-doesnt-glyphsapp-output-hdmx-and-vdmx-tables/9316/3
we should WARN if these tables are in the fonts, and get them removed.
Perhaps even FAIL? But if it was FAIL it would be a low priority one.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/googlefonts/fontbakery/issues/2007, or mute the
thread
https://github.com/notifications/unsubscribe-auth/ABQVmK2CDLaZwiIVE9HP747emtvueJJkks5uOdYvgaJpZM4VyoDG
.

Per https://forum.glyphsapp.com/t/why-doesnt-glyphsapp-output-hdmx-and-vdmx-tables/9316/3 we should WARN if these tables are in the fonts, and get them removed.

Georg said "not yet" while Jens said "they're deprecated". Other than that there's no actual discussion on why.

so yeah... any idea why?


[Feat]  Implement Super Admin Functionality on EzyShop Website : Backend schema, server action and necessary routes

Description
Issue: Implement Super Admin Functionality on EzyShop Website
Description
This issue focuses on implementing the super admin functionality for the EzyShop website. The goal is to establish a robust backend infrastructure that enables super admin users to manage the platform effectively, ensuring better control and oversight over vendor and customer activities.
Requirements

Super Admin Role:

Introduce a super admin role within the user management system to distinguish super admins from regular admins and vendors.
This role will have elevated privileges to perform various management tasks across the platform.

Login Functionality:

Implement a secure login mechanism for super admins to access the admin panel.
Ensure proper authentication and authorization checks are in place to protect sensitive operations.

Backend Schema Updates:

Update the existing database schema to accommodate the new super admin role.
This may involve modifications to the user table to include role-based access control.

Server Actions:

Develop server actions for super admin login, handling user authentication, and managing sessions.
Ensure that error handling and validation are implemented to provide a smooth user experience.

Screenshots
No response
Checklist

[X] I have checked the existing issues
[X] I have read the Contributing Guidelines
[X] I'm a GSSoC'24-Extd contributor
[X] I am willing to work on this issue (optional)

@ShivanshPlays You are assigned.


Adding Feeds modules

And integration with Contents and Lists (for blogs for instance)

I enabled the Orchard.Feeds modules.
The url /Contents/Lists/{myBlogId}/rss displays the feed in Rss format.
I am wondering if I will be able to define an alias like this :  /feed.rss to point to this url
Will there be an Alias admin page like in O1?

In a separate enhancement, I would like to be able to add the meta .
It doesn't have to be necessarily automatic or in a Part setting because I would like to be able to specify the url of feed proxy. (FeedBurner for example)
If I could add anything in the head from a setting like I suggested here #422, it would fit my need.

Added Feed Proxy Url and Items count settings.

I had to save the blog settings because FeedItemsCount was null.
Todo: Test that it works after a setup with a blog recipe and enabling the module.
Or will the feature be enabled by default?

With this branch, I have a weird behavior:
When I edit a blog post, the change is not taken in account when I modify any parts. If I go back to master, it works.

Fixed the setup issues, and also refactored content element management quite a bit.

Everything works for me.

The menu management has issues now, it was expected with the refactoring. Fixing it.

Ready to go


How can we pass props to previous screen in Action.Pop() operation

Version
Tell us which versions you are using:

react-native-router-flux v3.30.2
react-native v0.29.0

Expected behaviour
I am passing props to previous screen like Actions.Pop({variable: value}) on click of back button. I am not able to get the props in previous screen because there is no any component life cycle methods triggered.
can anyone please tell me which method is triggered on Action.Pop() operation and what is the way to pass props to previous screen on back button press?
Actual behaviour
In actual only popping is happened and we moved to previous screen.
Steps to reproduce

@vykumar pop(), by definition, will bring back the previous scene with all the context that it was previously set.

You can pass props like this
NavigationActions.pop( {refresh: {variable: value} })
this prop change will be found in
componentWillReceiveProps callback in previous component

@waqas19921 This is nice, but how can the "automatic" pop of the hardware back (on Android) is handled? What should we do in this case, in order for a prop to have a different value there?
Thanks!

@waqas19921 how can we get the props in CDM


its buggy on canary

when using canary some times the channel and server list cant be revealed in server or dm unless you hit ctrl+ , and configure the settings again by hitting save and apply then it only works for a little bit. this may be because of the update that canary had.

I'm having the same issue on PTB

Okay I worked out a fix for now.
#hg_hoverOver {
    z-index: 9001;
}

Not sure if it will work for everyone though I've done some messing around and it seems to be working.

I'll test it one day myself and add this. I've been puzzled with how should I deal with the test builds of discord, I cannot commit myself into supporting them, But fixes like these are always welcome =)

After the update to discord main and my fixes for it, it appears to work just fine on canary too. I'll close this, tell me if this still happens for any of you.


chore: decisions- design

description:

document thought process for any person picking this project up down the road

acceptance criteria:

please document any decisions around design work including us web standards we're adhering to or choices made and why

@marieksmith I think would be great to keep decisions we make in a standard form (https://github.com/joelparkerhenderson/architecture_decision_record/blob/master/adr_template_by_michael_nygard.md) and then either held in a folder or in the wiki.

Yeah good call. I like the idea of adding to the gh wiki.

Yeah same. I think that is a good call. Let assume we put it there.

i did it!!!
https://github.com/deptofdefense/dds.mil/wiki/Documentation:-MVP-Design-Rationale
@ashhath01 added to gh wiki. closing this


Update stable version in repo from 0.4.10 to 0.4.11

Think about update stable version to 0.4.11
Version: stable=0.4.10 (236 days old) => latest=0.4.11 (15 days old)
Installs: stable=124 (79.49%), latest=21 (13.46%), total=156
Click to use developer portal
Click to edit
Note: This is an automatically generated message and not personally authored by bluefox!

@Chris-656
Is there any reason to delay update at stable repository any longer?
If there is no feeback / comment her until 31.10.2023 I will release 0.4.11 to stable anyway.

wait for 0.4.13
0.4.13 is out
By the way I did not get any test results for versions 0.4.12 and 0.4.11

wait for 0.4.13 to become ready for stable :-) (= at least one or two weeks at latest)

This issue seems to be outdated.
This issue suggests to update the stable version of this adapter to 0.4.11 but in the meantime an update to version 0.4.13 is suggested.
So this issue will be closed and replaced by an updated one.


"Enable Audio Recording" - keeps popping up, even after clicking extension action button

When I hit the mining hotkey, the "Enable Audio Recording" popup appears, but even after clicking the asbplayer extension button, and the
What happens:

I hit my mining hotkey
The "Enable Audio Recording" popup appears

I click the asbplayer extension button
The "Audio Recording Enabled" popup appears

I let the video play again, and hit mining hotkey again
The "Enable Audio Recording" popup appears again

This happens on Netflix, Amazon Prime, and YouTube. I used to be able to mine on all of these services, but not anymore. The last time I was able to mine a card was 2024-06-15.
Platform:
Windows 11
Chrome Version 126.0.6478.63 (Official Build) (64-bit)
Asbplayer 1.3.1

@roboticsbrian If you open the extension manager (chrome://extensions) you should see that asbplayer has logged some errors. Can you paste those logs here?

@roboticsbrian If you open the extension manager (chrome://extensions) you should see that asbplayer has logged some errors. Can you paste those logs here?

The service worker console shows this:

background.js:2 Uncaught (in promise) Error: Failed to start recording
    at Object.startWithTimeout (background.js:2:194840)
    at async Object.startWithTimeout (background.js:2:192968)
    at async Object._recordAndForward (background.js:2:200223)
    at async Object.handle (background.js:2:199367)
startWithTimeout @ background.js:2
await in startWithTimeout (async)
(anonymous) @ background.js:2

While I was on the extension page for asbplayer (chrome://extensions/?id=hkledmpjpaehamkiehglnbelcpdflcab), I saw that Collect Errors was turned off. Thinking I might get more informative error messages I turned it on:

Now that it's turned  on, asbplayer no longer prompts to enable audio recording, and I am able to mine cards.
 When Collect Errors is on, I can mine without issues.

That is really interesting info - thanks!

Just started getting this same issue today when using the one click workflow. Toggling Collect errors does not resolve it for me. I see the following errors:

macOS Sonoma 14.3
Arc v1.47.1 (based on Chromium version 126.0.6478.57)
Asbplayer 1.3.2

I'm running into this same error.
MacOS Sonoma 14.6 (23G80)
Brave [Version 1.70.117 Chromium: 129.0.6668.59 (Official Build) (arm64)]
Asbplayer v. 1.4.2

Same thing happening to me.
Asbplayer [v1.6.1]
Here's error:


Mounted cloud-init ISO images cause warnings when updating ESXi hosts

The attached cloud-init ISO images cause warning during a ESXi host update:

we have the same problems at other customers as well


Output Cauldronoid tables with formal charge

Loading molecules in RDKit produces apparently absurd formal charges. It would be interesting if the CSD formal charges are reasonable. They do seem to be available
Also, it would be convenient to output Cauldronoid-style tables, instead of having to read them with RDKit as an intermediary, since RDKit often fails to load.
The CSD API also makes bond orders available, which can also go into the Cauldronoid table.
Combining these ideas, it would be convenient to have Cauldronoid tables, with the CSD annotated formal charges, in the hopes that these are more reasonable than the RDKit ones for organomeetallics. In particular, RDKit often puts absurd (>5) charges on the central metal atoms, whereas the formal charges that appear in CSD's structure diagrams on the web pages appear correct.
Ideally, this would make annotation of zwitterions possible.

Resolved in 5e1c54ef262fceed057c51070d84ac6f063163ef


Neuron support in Axlearn

This PR enables use of neuron devices in Axlearn for model training.

Chooses correct mesh for TRN devices for Fuji 7B with the mesh selector flag --mesh_selector=neuron-trn1.32xlarge-64

@apoorvtintin I see this PR is quite stale for sometime.
If no objection, I'd like to have @Ruixuan who is working on Trn from our end to port your change and continue iterate it?

@apoorvtintin I see this PR is quite stale for sometime. If no objection, I'd like to have @Ruixuan who is working on Trn from our end to port your change and continue iterate it?

Apoorv is on PTO right now. I am OK with you all taking over this PR. Can you add us as a reviewer when you finish? Thanks

Thanks for all the reviews, I fixed most of the comments on the PR.


Styling of devise views

Closing dues to Issac's addition of devise_for_bootstrap gem

Changed mind and tying issacs ticket to this issue

@tekHudson is this done since #141 is merged in?


Ui issues

1.Fixed issue of search Function.
2.Fixed chapter heading and section heading.
3.Removed release key now app is in debug mode

Neetu, I have merged the ui_issues branch. It can now be deleted.

Ok
On Mon, 20 Jul, 2020, 10:08 AM rinsingh<EMAIL_ADDRESS>wrote:

Neetu, I have merged the ui_issues branch. It can now be deleted.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/Bridgeconn/VachanAppReact/pull/3#issuecomment-660796899,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AF5M7KWGEMCTYVJICXV4YT3R4PC23ANCNFSM4O2F4LEA
.


Problem evaluating trajectories with EVO

Hi,
Thanks for open-sourcing this interesting work.
I have built MAD-ICP from source using cmake and make, and tried evaluating and reproducing the results using KITTI. So far I have tried KITTI 00 and 01 sequences but when I tried to evaluate the trajectory and calculate the ATE and RPE using EVO (https://github.com/MichaelGrupp/evo), I kept running into this error: "KITTI pose files must have 12 entries per row and no trailing delimiter at the end of the rows (space)".
The output trajectory file of MAD-ICP indeed has 12 entries per row and no trailing delimiter of any sorts at the end of each row, there is probably spaces so I tried using this ubuntu command (sed 's/[[:blank:]]*$//' estimate.txt) to remove them. But this does not seem to work and I am still having the same error as mentioned before.
I wonder have you seen this error before and if you have, how do you solve it? If you are not using EVO, what software do you use to calculate the ATE and RPE of the trajectory?
Thank you in advance and I am looking forward to hearing from you.

Hi, I have just pushed some changes that should solve your problem, regardless of whether you run MAD in Python or C++. I tested EVO with these changes and it works as expected. However, as we clearly explain in our paper, for evaluating RPE we employ the KITTI odometry development kit.


Accessing root stack from within child stack

Is there a way to access a root stack navigator from an inner stack navigator? For example:
const Root = StackNavigator(
{
  Login: {
      screen: Login,
    },
    TabNav: {
      screen: TabNav
    }
}
);

const TabNav = TabNavigator(
  {
    Content: {
      screen: Content,
    },
    Settings: {
      screen: SettingsStack
    }
  }
);

Within that SettingsStack screen I have a logout button. How would I access the Root Stack to navigate back to Login? Rather than that inner stack in the Tab Navigation just pushing to the login page.

+1

key option helps me.
this.props.navigation.dispatch(NavigationActions.reset({
  index: 0,
  actions: [NavigationActions.navigate({ routeName: 'Login' })],
  key: null
}));

key, string or null - optional - If set, the navigator with the given key will reset. If null, the root navigator will reset.


Enhancements to the csv file for Aria Operations alerts

Code of Conduct

[X] I have read and agree to the project's Code of Conduct.
[ ] Vote on this issue by adding a 👍 reaction to the original issue initial description to help the maintainers prioritize.
[ ] Do not leave "+1" or other comments that do not add relevant information or questions.
[ ] If you are interested in working on this issue or have submitted a pull request, please leave a comment.

Description
As part of end to end automation, in the menu the first step is to create the json.
During this step we should also read the Aria operations Alert List and update the corresponding csv file.
If the Aria Operations Alert List tab of the PNP is not updated by the user, then we should print a message, so the user updates this tab of the PNP and then retries step 1.
Use Case(s)
Aria Operations deployment.
Potential Configuration
update the csv file
References
No response

While trying the automation for this section "Create Notifications in VMware Aria Operations for VMware Cloud Foundation Issues". getting this error
Unable to Add Notification for Alert Definition (AlertDefinition-VMWARE-VMReadLatencySwapping), Due to Incorrect Alert Plugin Name (Email-Alert-Plugin)
Discussed this with a colleague and based on the debugging we figured out that it is because of the value we have under the "alert plugin name" column in the csv, it needs to match the instance name, by default it is mentioned as "Email-Alert-Plugin"
Once we updated it for a line to the same name as the instance name which is in the lab is,  the script exited successfully .


⚠ ClamAV has degraded performance

In faa1472, ClamAV (https://spamassassin.apache.org/) experienced degraded performance:

HTTP code: 200
Response time: 608 ms

Resolved: ClamAV performance has improved in 778d848.


Arbitrary hsl colors

Clear and concise description of the problem
Can't use an arbitrary hsl color like bg-[hsl(0, 100%, 100%)] which would make it easy to use it with variants.
Suggested solution
Add syntax so hsl is possible with arbitrary values.
Alternative
No response
Additional context
No response
Validations

[X] Read the Contributing Guidelines.
[X] Read the README.md of using the package.
[ ] Already used the Interactive Docs.
[X] Check that there isn't already an issue that request the same feature to avoid creating a duplicate.

Incorrect usage. https://unocss.dev/interactive/?s=bg-[hsl(0_100%25_100%25)]


Add svg logos for libs

@raulraja @jorgegalindocruces could you review this PR please?
This PR contains:

FPis.svg (32x32 white)
ScalaCheck.svg  (32x32 white)

Two examples for preview:

I think that for FP we can use a book to illustrate that are exercises from a book. Maybe a book with the letters on the top?

Thanks, much better. LGTM


Cancel PMKAlertController programatically

Is it possible to cancel an PMKAlertController programatically? Let's assume I display an alert like so:

You are currently offline [OK]

Now as soon as I detect that I am no longer offline, I want to dismiss/cancel this alert without the user having to tap OK.

Short answer is no. Our PMKAlertController does not expose its underlying UIAlertController and doesn't currently have a cancelPromisedViewController method in its UIViewController extension. However, this would be a trivial addition: https://github.com/mxcl/PromiseKit/blob/master/Categories/UIKit/PMKAlertController.swift#L70
PR welcome!

Actually I guess it wouldn't be all that trivial now that I'm really looking at it. We will have no reference to the PMKAlertController instance from the UIViewController extension. So the options then would be to either pass the PMKAlertController instance back into the new cancelPromisedViewController method, which does follow UIViewController's dismissViewController API that we would be mimicking, else place the method on PMKAlertController itself, which would be inconsistent with our own API.
Still, neither option would be difficult. It's just deciding which one to go with. I think I'm personally leaning towards the former.


Is it expected that a click on a circle does not prevent a click on map event to be fired ?

Checklist

[X] I've looked at the documentation to make sure the behavior isn't documented and expected.
[X] I'm sure this is an issue with Leaflet, not with my app or other dependencies (Angular, Cordova, React, etc.).
[X] I've searched through the current issues to make sure this hasn't been reported yet.
[X] I agree to follow the Code of Conduct that this project adheres to.

Steps to reproduce
Using the following code snippet:

Expected behavior
When clicking on the circle, the click should not be detected by the map and, thus, the click event of the map should not be fired.
Current behavior
When clicking on the circle, there are two events fired:
click on circle
click on map

The click "goes through" the circle and is detected by the map, firing two events. However, I did not click the map "directly", but the circle.
Minimal example reproducing the issue
https://plnkr.co/edit/MyAAttMrpNgnIfyZ
Environment

Leaflet version:  1.9.4
Browser (with version): Microsoft Edge Version 112.0.1722.58
OS/Platform (with version): Windows 10

What you're describing is called event bubbling.
If you need to prevent the event from bubbling up to the map, it must be done explicitly:
circle.on("click", (ev) => {
	L.DomEvent.stopPropagation(ev);
	console.log("click on circle");
});

You can find it documented on Circle options inherited from Path.


Bug: Cannot save slice where there is not Group by set

Scenario:
I've a database set with mssql+pyodbc table created. When I create a table view slice without group by( remove count() from default) and a list of columns in no group by. After I save and reload the slice the count() in group by reappears, giving me error saying "Exception: Choose either fields to [Group By] and [Metrics] or [Columns], not both". This happens to overwrite as well.
Expected behaviour:
my saved slice should reappear again.
Caravel version: 0.9.0

This is fixed in master and will make 0.9.1!


Support run multiple request at the same time for stress test

Tell us about the task you want to perform and are unable to do so because the feature is not available
Normally, you can only make a single request at the same time, If you want to know how strong is your server so you may want to simulate N of user to use your api at the same time.
Describe the solution/feature you'd like us to add
Support config/script to run multiple request at the same time.
Any other feedback you would like to provide regarding the site
Your response here
comment

Thank you for raising the feature request. It will be included in the roadmap we are building around testing.


dependency scope setting issues

ISSUE TYPE

Other

SUMMARY

Hi,
We are developing a tool to check whether the dependencies' scopes in maven are set rightly and reasonably.
Our tools have found some issues in your project about scope setting in maven' pom.xml file.

in plugins\user-authenticators\ldap\pom.xml, org.zapodot:embedded-ldap-junit scope should be test  instead of compile.
in utils\pom.xml, commons-io:commons-io scope should be compile  instead of provided.
in server\pom.xml, mysql:mysql-connector-java scope should be compile  instead of test.

Please help us confirm whether the above reports are right and explain why it should set like these.
Thanks a lot.

Fixed in https://github.com/apache/cloudstack/pull/6557


Fix inconsistent route name

[ ] CHANGELOG.md updated
[ ] Rebased/mergable
[ ] Tests pass
[ ] Sign CLA (if not already signed)

:+1:


Not able to install on node 18 ( strapi-v4)

Plugin is not installing  on the newer version 18

18.12.1

@amjadniazi48 Can you share error message you're facing?

Fixed in 2.2.3


(Discussion) Keep using kappa library

The kappa library used for the events sources seems to be have been totally  abandoned. The last release is from february 2017...
AWS and Boto3 have a lot new functionalities and it seems that currently we just override the kappa main classes to add those new functionalities when they comes up (e.g the ExtendedSnsEventSource).
Wouldn't it be more maintanable to create a whole new class in the Zappa project for this only purpose ?
There is nothing really hard in this, it just need some time to think about making it really generic, robust and easily maintainable using only Boto3.
All of the main code for event sources is in the utilities.py so there is no huge changes accross the project.
Any other ideas are welcome, and I can take some of my time to start this refacto if you think it can be a good thing to do.
Related issues :
#413
#504 - #545
#512 - #668 - #809
#557
#718
#761 - #762
#817
#854
#861
#907
#958

As an interim approach, perhaps vendoring kappa into the project as a submodule, to get full control and be able to make changes, and then progressively replacing the code would be a good way to minimise the amount of change this might cause?

This issue is affecting my use cases directly around event driven data processing for IoT sensors. I am happy to pitch in with PR's if the core contributors can come up with work breakdown structure and specific issues to address. I am all for vendoring this library and overriding sections to the more maintained boto3 equivalents.

What about using terraform as it is well maintained ?
In the readme there is an example :
https://github.com/dpetzold/terraform-aws-zappa

Reviewing issues, and this looks like something that should be addressed.
I don't think the kappa dependency should be necessary, and agree that the functionality should be pulled in to zappa, ideally without adding a new dependency.
Most likely the code would be re-written with boto3 (an existing dependency of zappa).
Related code.
https://github.com/zappa/Zappa/blob/master/zappa/utilities.py#L219-L244

In my mind, the first step to migrate away from kappa is to encapsulate the kappa components into a new zappa internal "infrastructure" api.
May be make a new "zappa.infrastructure" package with an InfrastructureManager() object to abstract out the kappa code.
Once that's done, we can then work to replace the kappa components from under the InfrastructureManager() api.

here


TTS Improvement

Is your feature request related to a problem? Please describe.
Now we can use the TTS in MTR mod to describe the station. I see the code, it uses com.mojang.text2speech. Yeah, it is easy, but it is bad for MacOS users (Need change native libs by themselves) and Windows users without SAPI (Some copy systems delete many useless things include this)
And the voice now is so machinal and bad. It makes people feel not good.
Describe the solution you'd like
Bundle the high-quality offline TTS engine into mod assets. Don't think about the copyright, they're all free and open-source.
I searched these are very useful:
TTSKit-Java: https://gitee.com/endlesshh/ttskit-java/tree/master Use Pytorch-Java to do. For Simplfied Chinese.
Java-TTS: https://github.com/ekk0/java-tts Normal, no big requires. For Cantonese.
JavaX TTS API+FreeTTS: You can look at this https://www.javatpoint.com/convert-text-to-speech-in-java . For English.
In addition, we can add a option to config if you want to use cantonese to repeat chinese part.
Additional Information
I know add them will make the mod so big and mess, so maybe we can made it to a core, and you can use a extension to do it.
If you feel difficult to use these engines, i can help.
PS: Also there is an engine ekho (Written in C) supports these threes and it is simple, but it is just to merge many pinyin and speak one by one. It is so bad.

I worry this will massively increase the size of the mod, maybe they can be an optional dependency?

This is probably out of scope for the mod.


Electron + better-sqlite3 getting error "A dynamic link library (DLL) initialization routine failed. "

Hello,
I'm using 1.4.0 of better-sqlite3, with Electron 1.4.15 and node 6.5.0, on Windows 64bit.  I've installed it using npm install --save better-sqlite3.  When this line executes: var Database = require('better-sqlite3');  I get an exception that dll init failed, see below for stack trace.  The better_sqlite3.node is in the location that require.resolve is looking for it.
Is there anything else I need to do to make this work? node-gyp rebuild specifying --arch=x64 --target_platform=win32 ?
Thanks
Greg.
D:\devtools\js\tutorials\electrontest>npm install --save better-sqlite3

><EMAIL_ADDRESS>install D:\devtools\js\tutorials\electrontest\node_modules\better-sqlite3
> node install

D:\devtools\js\tutorials\electrontest\node_modules\better-sqlite3>if not defined npm_config_node_gyp (node "C:\Program Files\nodej
s\node_modules\npm\bin\node-gyp-bin\\..\..\node_modules\node-gyp\bin\node-gyp.js" rebuild )  else (node "" rebuild )
Building the projects in this solution one at a time. To enable parallel build, please add the "/m" switch.
  unpack_sqlite_dep
  sqlite3.c
  win_delay_load_hook.cc
  sqlite3.vcxproj -> D:\devtools\js\tutorials\electrontest\node_modules\better-sqlite3\build\Release\\sqlite3.lib
  int64.cc
  data.cc
  database.cc
  statement.cc
  transaction.cc
  open.cc
  close.cc
  binder.cc
d:\devtools\js\tutorials\electrontest\node_modules\better-sqlite3\src\binder\get-error.cc(9): warning C4267: 'initializing': conv
ersion from 'size_t' to 'int', possible loss of data (compiling source file ..\src\binder\binder.cc) [D:\devtools\js\tutorials\el
ectrontest\node_modules\better-sqlite3\build\better_sqlite3.vcxproj]
d:\devtools\js\tutorials\electrontest\node_modules\better-sqlite3\src\binder\bind-buffer.cc(8): warning C4267: 'argument': conver
sion from 'size_t' to 'int', possible loss of data (compiling source file ..\src\binder\binder.cc) [D:\devtools\js\tutorials\elec
trontest\node_modules\better-sqlite3\build\better_sqlite3.vcxproj]
  multi-binder.cc
  better_sqlite3.cc
  win_delay_load_hook.cc
     Creating library D:\devtools\js\tutorials\electrontest\node_modules\better-sqlite3\build\Release\better_sqlite3.lib and obje
  ct D:\devtools\js\tutorials\electrontest\node_modules\better-sqlite3\build\Release\better_sqlite3.exp
  Generating code
  Finished generating code
  better_sqlite3.vcxproj -> D:\devtools\js\tutorials\electrontest\node_modules\better-sqlite3\build\Release\\better_sqlite3.node
<EMAIL_ADDRESS>D:\devtools\js\tutorials\electrontest
`--<EMAIL_ADDRESS>
Exception:
"Error: A dynamic link library (DLL) initialization routine failed.
\\?\d:\devtools\js\tutorials\electrontest\node_modules\better-sqlite3\build\Release\better_sqlite3.node
    at Error (native)
    at process.module.(anonymous function) [as dlopen] (ELECTRON_ASAR.js:173:20)
    at Object.Module._extensions..node (module.js:583:18)
    at Object.module.(anonymous function) [as .node] (ELECTRON_ASAR.js:173:20)
    at Module.load (module.js:473:32)
    at tryModuleLoad (module.js:432:12)
    at Function.Module._load (module.js:424:3)
    at Module.require (module.js:483:17)
    at require (internal/module.js:20:19)
    at bindings (d:\devtools\js\tutorials\electrontest\node_modules\bindings\bindings.js:76:44)"

To be honest I don't have a windows machine so I can't test it myself, but other people have reported it working.
Try going directly into node_modules/better-sqlite3/install/index.js and changing this:
var args = process.env.CI === 'true' ? ['rebuild', '--debug'] : ['rebuild'];

to this:
var args = ['rebuild', '--arch=x64', '--target_platform=win32'];
if (process.env.CI === 'true') {args.push('--debug');}

Then run npm install again in your project directory. Let me know if that works for you.

Hi,
Thanks for replying.  However it didn't work, first reason was running install again seems to replace the files so the changes to index.js got overwritten.  So I ran npm rebuild --save better-sqlite3 and that used the changes in index.js however I still had the same dll load exception.
I googled "node Error: A dynamic link library (DLL) initialization routine failed"  and ended up here:
Using Native Node Modules
Followed the steps for the Easy Way, which are to:
npm install --save-dev electron-rebuild
.\node_modules\.bin\electron-rebuild.cmd

And it's working.  I did revert my changes to index.js before doing the two commands above.  Basically on Windows if using Electron you need to rebuild against the Electron headers.  I'm not well enough versed in node/js/electron to really understand why.  From what I'm reading this seems to only affect Electron on Windows.
Thanks for your help.
Greg.

Great, I'm glad you got it working 👍


Refactor placeholder

Method name change
Package restructure

@all-contributors please

@all-contributors please add @gopinath-langote  for infrastructure

@all-contributors please add @gopinath-langote

@all-contributors please add @gopinath-langote maintainer

@all-contributors please add @gopinath-langote for code

@all-contributors please add @gopinath-langote  for code, design, infra, review
@all-contributors please add @landpro  for code, design, review

@all-contributors please add @gopinath-langote  for code, design, infra, review

@all-contributors please add @landpro for code, design, review


Fix missing wsgi subdirectory

Addresses issue #31.
Addresses the same issue as this existing PR, but in a cleaner way.

I signed it!


Add abstraction for filter state lookup

Some filters would like to store state keyed by packet information like source address, this is useful for #374 and #5
we would like to add e.g a key-value map that is smart enough to expire items from the map after the session duration timeout (similar to our session map impl)

yes, feels like the need for state tracking potentially only applies for filters that "route" (where you only want to make the decision once per flow) and not to filters that modify packet contents.  So that fits with #104.
will be interested to see what you decide - I'm still hoping to implement that load balancer mode that calls out to an external API, and that really does nee to be once per flow.


added condition for existing ECR

Good idea. Let's do that.

Changes have been done, with help from Uchinda
@niro, for your review


[Feature] Allow access to the state store query's filters without having to unmarshalling it first

Describe the feature
Currently, the state store query filters are built when unmarshalling the query struct from JSON string, this blocks scenarios where the query is built using raw data since the unmarshalling process is exclusively used when parsing JSON (e.g pluggable components data are transmitted using protocol buffers instead of JSON).
As the pluggable components SDK for go leverages the components-contrib interfaces this will be necessary to improve the UX experience when developing pluggable components and for reusability of the already created built-in components.
See this.
The solution would be exposing the parseFilter function or make the Filter query attribute be a method instead of an attribute and ensure that it will be built only once per query instance.
Release Note

RELEASE NOTE:

/assign


HIVE-27702: Remove PowerMock from beeline and upgrade mockito to 4.11

https://issues.apache.org/jira/browse/HIVE-27702
What changes were proposed in this pull request?

Remove PowerMock from beeline
Mockito-inline dependency added

Why are the changes needed?

PowerMock seems to be a dead project.
Prerequisite for moving to Java 11

Does this PR introduce any user-facing change?
No
Is the change a dependency upgrade?
Yes
How was this patch tested?
Precommit tests

LGTM


Reading MQ messages in code with MQRFH2 message format in EBCDIC character set

ibmmq-jms-spring version(s): 2.1.2
Java version (including vendor and platform). 1.8 Windows 10 SpringBoot 2.1.6
I have an issue when reading messages of the queue which is in EBCDIC format. I can see they get handled by MQPoison class and then get backed out to the DLQ. The rest in the queue that is string gets processed successfully.
The error I see when debugging is
com.ibm.msg.client.jms.DetailedJMSException: JMSCMQ1000: Failed to create JMS message.
The wrong message type or properties were specified when creating a base message with messageID = '414D5120514D31202020202020202020FD7E425D200805ED' and correlationID = '000000000000000000000000000000000000000000000000'.
Check the linked IBM MQ exception reason and completion code for more information and check the log file for detailed message contents.
Nothing else gest written to the Springboot application log.
<
@Component
public class JMSConsumer {
Logger log = LoggerFactory.getLogger(JMSConsumer.class);
@JmsListener(destination = "INPUT.QUEUE")
public void onAuditMessage(final Message receivedMessage) {
	BytesMessage bytesMessage = (BytesMessage) receivedMessage;
	byte[] bytesreceived;
	try {
		bytesreceived = new byte[(int) bytesMessage.getBodyLength()];
		bytesMessage.readBytes(bytesreceived);
		// convert to MQMessage then to PCFMessage
		MQMessage mqMsg = new MQMessage();
		mqMsg.write(bytesreceived);
		mqMsg.encoding = receivedMessage.getIntProperty(WMQConstants.JMS_IBM_ENCODING);
		mqMsg.format = receivedMessage.getStringProperty(WMQConstants.JMS_IBM_FORMAT);
		String codePage = receivedMessage.getStringProperty(WMQConstants.JMS_IBM_CHARACTER_SET);

		mqMsg.seek(0);
		log.info("message len " + Integer.toString(mqMsg.getMessageLength()));
		log.info("message encoding " + Integer.toString(mqMsg.encoding));
		log.info("message format " + mqMsg.format);
		log.info("message type " + Integer.toString(mqMsg.messageType));
		log.info("message code page " + codePage);

		try {
			log.info("message content:  " + mqMsg.readStringOfByteLength(mqMsg.getMessageLength()));
		} catch (Exception e) {
			log.info("No message content : " + e.getMessage());
		}

	} catch (JMSException e) {
		e.printStackTrace();
	} catch (IOException e) {
		e.printStackTrace();
	}

}

}

I suspect the problem is that you are trying to treat a message with an RFH2 as a bytesmessage. When reading a message that a browse (eg amqsbcg) may show as having an RFH2, the JMS code is probably converting that header to properties (they are interchangable) and then the returned message is actually of whatever format was indicated in the RFH2 header - which could well be MQFMT_STR and hence a JMS textMessage not a bytesmessage. The RFH2 contents are then available via getProperty calls instead.

But it its not even getting to my code to print naything, its failing way before that. I see the detail messages says. I know the format is sound as we have been processing the message with IIB9 for years. Could it be the custom user attributes. I seem to have read somewhere that JMS don't like that.
com.ibm.msg.client.jms.DetailedMessageFormatException: JMSCMQ1050: The MQRFH2 header has an incorrect format.
Received a message with a badly formed MQRFH2 header.
Ensure that any non-JMS applications building messages with MQRFH2 headers create well-formed MQRFH2 headers.

A full trace might give more detailed error information (though that can be extremely verbose and hard to decipher). You said you've got some custom user attributes in the RFH2. Are those in the  folder or are you trying to use an alternative folder? How was the RFH2 constructed? IIB does not use the Java classes for parsing these messages, so it's possible your messages do not fit into the JMS constraints. Only the known folders are permitted - usr, jms, mcd, mqext, mqps - in JMS messages.
The output from amqsbcg might also help to ensure the message looks OK.

The custom fields are in usr but that is not the problem. it's working fine when the message saved comes from a non ZOS environment. It only fails on the ZOS messages coming from IMS and its copybooks.

Since this sounds like it's something being done in the MQ JMS classes, you may need to open a support ticket to get it investigated by the service team. Trace and a browse of the offending message are likely to be needed.


Mac Issue

Cant work on older mac systems even though terratech can work on that older system

the creator said Here, that they wouldn't be updating the mac build of TTSMM, sorry.

Thank you for your time tho
It is still a great mod
On Wed, 4 Oct 2023 at 07:49, dvdiscool @.***> wrote:

the creator said Here
https://forum.terratechgame.com/index.php?threads/ttsmm-the-unofficial-official-workshop-mod-manager.22683/post-158095,
that they wouldn't be updating the mac build of TTSMM, sorry.
—
Reply to this email directly, view it on GitHub
https://github.com/FLSoz/terratech-steam-mod-loader/issues/5#issuecomment-1745781108,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/BCSANXC64CESJQR2VQGG5XLX5SCABAVCNFSM6AAAAAA4XH54N2VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTONBVG44DCMJQHA
.
You are receiving this because you authored the thread.Message ID:
@.***>


raids: layout is not updated when reloading CoX via the steps

Type
Incorrect behavior
Operating System
None
OS Version
No response
Bug description
Jagex added an option to reload CoX using the steps/lobby stairs today. The layout does not get updated when one reloads the raid using this feature.
When reloading a raid via the steps, it currently looks like:

Varbit.IN_RAID goes 1 -> 0 -> 1
Varbit.RAID_PARTY_SIZE 1 -> 0 -> 1
VarPlayer.IN_RAID_PARTY not -1  -> -1 -> not -1

I suspect raid::layout needs to be nulled when reloading. I am not familiar enough with the RaidsPlugin code, the CoX varbits/varps, and CoX itself to properly update the code though. E.g. idk what happens with the varbs during the raids, on login, or when changing floors.
Screenshots or videos
No response
RuneLite version
RL <IP_ADDRESS>
Logs
No response

I noticed this too and was coming to GitHub to report this if a ticket didn't exist. For anyone reading, the workaround is to turn off and turn on the plugin from the sidebar (see below).

should get fixed by https://github.com/runelite/runelite/pull/17243

should get fixed by #17243

Any guess as to when this will get merged in?

fixed in fb98120b41560ba7c0c6195424b65f8e262836ec


Align command's options with our naming convention

We should review all commands and ensure that all options match our naming convention:

if the option doesn't have a prefix (eg. id, title), it refers to the last noun in the command name, eg. spo listitem get > last noun = listitem so id and title refer to listitem
if the option needs to point to a property of a different object, it needs to be prefixed to make this clear, eg. listId, listTitle in spo listitem list. Last noun is listitem but we need the user to specify ID of the list, rather than list item, so just prompting for id or title wouldn't make it clear that we're expecting ID or title of the list

Commands that need adjustments:

[ ] aad approleassignment add: objectId and displayName should be prefixed with app
[ ] aad approleassignment list: objectId and displayName should be prefixed with app
[ ] aad approleassignment remove: objectId and displayName should be prefixed with app
[ ] aad o365group recyclebinitem list: displayName and mailNickname should be prefixed with group
[ ] aad o365group teamify: groupId doesn't need the group prefix and could be just id
[ ] aad policy list: policyType doesn't need the policy prefix and could be just type
[ ] aad sp get: displayName and objectId should be prefixed with app
[ ] flow run cancel: flow should be flowName to disambiguate how we expect users to denote the flow, same applies to environment, which should become environmentName
[ ] flow run get: flow should be flowName to disambiguate how we expect users to denote the flow, same applies to environment, which should become environmentName
[ ] flow run list: flow should be flowName to disambiguate how we expect users to denote the flow, same applies to environment, which should become environmentName
[ ] flow run resubmit: flow should be flowName to disambiguate how we expect users to denote the flow, same applies to environment, which should become environmentName
[ ] flow disable: environment should be environmentName to make it clear how we expect users to denote the environment
[ ] flow enable: environment should be environmentName to make it clear how we expect users to denote the environment
[ ] flow export: environment should be environmentName to make it clear how we expect users to denote the environment
[ ] flow get: environment should be environmentName to make it clear how we expect users to denote the environment
[ ] flow list: environment should be environmentName to make it clear how we expect users to denote the environment
[ ] flow remove: environment should be environmentName to make it clear how we expect users to denote the environment
[ ] outlook message move: messageId should be id
[ ] pa connector export: environment should be environmentName to make it clear how we expect users to denote the environment
[ ] pa connector list: environment should be environmentName to make it clear how we expect users to denote the environment
[ ] pa solution reference add: path should be projectPath
[ ] spfx package generate: packageName should be name
[ ] spo app add: scope should be appCatalogScope
[ ] spo app deploy: scope should be appCatalogScope
[ ] spo app get: scope should be appCatalogScope
[ ] spo app install: scope should be appCatalogScope
[ ] spo app list: scope should be appCatalogScope
[ ] spo app remove: scope should be appCatalogScope
[ ] spo app retract: scope should be appCatalogScope
[ ] spo app uninstall: scope should be appCatalogScope
[ ] spo app upgrade: scope should be appCatalogScope
[ ] spo apppage set: pageName should be name
[ ] spo cdn policy list: type should be cdnType
[ ] spo cdn policy set: type should be cdnType
[ ] spo contenttype field set: fieldId should be id
[ ] spo customaction add: url should be webUrl
[ ] spo customaction clear: url should be webUrl
[ ] spo customaction get: url should be webUrl
[ ] spo customaction list: url should be webUrl
[ ] spo customaction remove: url should be webUrl
[ ] spo customaction set: url should be webUrl
[ ] spo feature disable: url should be webUrl, featureId should be id
[ ] spo feature enable: url should be webUrl, featureId should be id
[ ] spo feature list: url should be webUrl
[ ] spo field get: fieldTitle should be title
[ ] spo field remove: fieldTitle should be title
[ ] spo file checkin: fileUrl should be url
[ ] spo file checkout: fileUrl should be url
[ ] spo file sharinginfo get: url should be fileUrl
[ ] spo folder get: folderUrl should be url
[ ] spo folder remove: folderUrl should be url
[ ] spo folder rename: folderUrl should be url
[ ] spo hubsite connect: url should be siteUrl
[ ] spo hubsite disconnect: url should be siteUrl
[ ] spo hubsite register: url should be siteUrl
[ ] spo hubsite rights grant: url should be hubSiteUrl
[ ] spo hubsite rights revoke: url should be hubSiteUrl
[ ] spo knowledgehub set: url should be siteUrl
[ ] spo list contenttype add: contentTypeId should be id
[ ] spo list contenttype remove: contentTypeId should be id
[ ] spo list view field add: fieldId should be id, fieldTitle should be title, fieldPosition should be position
[ ] spo list view field remove: fieldId should be id, fieldTitle should be title
[ ] spo list view field set: fieldId should be id, fieldTitle should be title, fieldPosition should be position
[ ] spo list view get: viewId should be id, viewTitle should be title
[ ] spo list view remove: viewId should be id, viewTitle should be title
[ ] spo list view set: viewId should be id, viewTitle should be title
[ ] spo listitem list: id should be listId, title should be listTitle
[ ] spo listitem record declare: id should be listItemId
[ ] spo listitem record undeclare: id should be listItemId
[ ] spo page column get: name should be pageName
[ ] spo page column list: name should be pageName
[ ] spo page control get: name should be pageName
[ ] spo page control list: name should be pageName
[ ] spo page control set: name should be pageName
[ ] spo page section add: name should be pageName
[ ] spo page section get: name should be pageName
[ ] spo page section list: name should be pageName
[ ] spo sp grant revoke: grantId should be id
[ ] spo sp permissionrequest approve: requestId should be id
[ ] spo sp permissionrequest deny: requestId should be id
[ ] spo site appcatalog add: url should be siteUrl
[ ] spo site appcatalog remove: url should be siteUrl
[ ] spo site apppermission get: permissionId should be id
[ ] spo site apppermission remove: permissionId should be id
[ ] spo site apppermission set: permissionId should be id
[ ] spo site chrome set: url should be siteUrl
[ ] spo site groupify: siteUrl should be url
[ ] spo site rename: siteUrl should be url
[ ] spo sitedesign rights grant: id should be siteDesignId
[ ] spo sitedesign rights list: id should be siteDesignId
[ ] spo sitedesign rights revoke: id should be siteDesignId
[ ] spo sitedesign task get: taskId should be id
[ ] spo sitedesign task remove: taskId should be id
[ ] spo tenant recyclebinitem remove: url should be siteUrl
[ ] spo tenant recyclebinitem restore: url should be siteUrl
[ ] spo web add: webUrl should be url
[ ] spo web get: webUrl should be url
[ ] spo web list: webUrl should be url
[ ] spo web reindex: webUrl should be url
[ ] spo web remove: webUrl should be url
[ ] spo web set: webUrl should be url
[ ] teams app install: appId should be id
[ ] teams app uninstall: appId should be id
[ ] teams channel get: channelId should be id, channelName should be name
[ ] teams channel remove: channelId should be id, channelName should be name
[ ] teams channel set: channelName should be name
[ ] teams message get: messageId should be id
[ ] teams tab get: tabId should be id, tabName should be name
[ ] teams tab remove: tabId should be id
[ ] teams team archive: teamId should be id
[ ] teams team clone: teamId should be id
[ ] teams team remove: teamId should be id
[ ] teams team set: teamId should be id
[ ] teams team unarchive: teamId should be id
[ ] teams user app add: appId should be id
[ ] teams user app remove: appId should be id
[ ] viva connections app create: appName should be name
[ ] yammer group user add: id should be groupId, userId should be id
[ ] yammer group user remove: id should be groupId, userId should be id
[ ] yammer message like set: id should be messageId
[ ] yammer user get: userId should be id

In some commands we use filePath and userName as options. While we could shorten them to path and name I wonder if in these cases we should treat the options as a single word. What do you think @pnp/cli-for-microsoft-365-maintainers?

name is too generic, so I think we should stick with userName and I'm happy with sticking with filePath I don't think we would gain any value from changing that.

name is too generic, so I think we should stick with userName and I'm happy with sticking with filePath I don't think we would gain any value from changing that.

Agree with @garrytrinder. We should stick with this naming

Across different commands, we also have an inconsistency with the short option name for URL:

sometimes we use -u for --url, --webUrl or --siteUrl
some commands use -w for --webUrl, where -u is used for --url
some commands use -s for --siteUrl, where -u is used for --url

How could we rationalize the use of the -u short so that it makes sense across all commands?`

I would use url only for all commands

Another thing that we should consider and which led to some inconsistencies: in some commands where we have multiple URLs, we chose to explicitly prefix each URL option to disambiguate them, eg:
spo folder get --webUrl --folderUrl. If we were to follow our naming convention, we'd use spo folder get --webUrl --url, because the last URL refers to a folder which is the last noun in the command. The question is though, which one is clearer and in cases where we have multiple sorts of options (names, IDs, titles, URLs) we should choose clarity over consistency?

@plamber:

I would use url only for all commands

That wouldn't be always possible though:

spo folder get requires webUrl and folderUrl
spo group get, using just URL would indicate that it's the group URL rather than web URL (webUrl)

So what would be your suggestion?
Add the noun of the object in front of all url commands?
GroupUrl
SiteUrl
Etc?

So what would be your suggestion? Add the noun of the object in front of all url commands?

@plamber I guess that's the question

do we always prefix, even when it's clear what the option is for, eg. spo web get --webUrl instead of spo web get --url
do we prefix only when an option is ambiguous, eg. spo folder get --webUrl --folderUrl instead of spo folder get --webUrl --url because it's not clear what --url is referring to
do we stick to consistency and naming convention and if the option isn't prefixed, it's implicit that it refers to the last noun

I would go for the last option

@pnp/cli-for-microsoft-365-maintainers what do you think?

Across different commands, we also have an inconsistency with the short option name for URL:

sometimes we use -u for --url, --webUrl or --siteUrl
some commands use -w for --webUrl, where -u is used for --url
some commands use -s for --siteUrl, where -u is used for --url

How could we rationalize the use of the -u short so that it makes sense across all commands?`

@plamber, what about this?

I would always go withe the first or first two letters of the words.
E.g.   siteUrl - su or s

We can only use one letter for a short because -su actually means -s -u.
@pnp/cli-for-microsoft-365-maintainers other opinions?

I would always go withe the first or first two letters of the words.
E.g. siteUrl - su or s

What if we have --listId and --listTitle? Current we solve it by exposing respectively -i and -t. In such cases, -l would be ambiguous because the user wouldn't know if it's list ID or Title.

I reviewed all commands and put all inconsistencies in the list.
To recap, here are the things we need to decide:

Do we want to disambiguate similar options (eg. web URL vs. list URL vs. folder URL). Available options:

we always prefix options, even when it's clear what the option is for, eg. spo web get --webUrl instead of spo web get --url
we prefix only when an option is ambiguous, eg. spo folder get --webUrl --folderUrl instead of spo folder get --webUrl --url because it's not clear what --url is referring to
we stick to consistency and naming convention and if the option isn't prefixed, it's implicit that it refers to the last noun

Do we want to normalize the use of shorts across commands? Eg. in the different commands we use -u, -w and -s for URL. Extra consideration, what if we have multiple URL options in the command (webUrl, listUrl, folderUrl):

We use -u only for web/site
We don't use shorts at all

For 1. @plamber gave his preference to be iii (follow naming convention). What do you think @pnp/cli-for-microsoft-365-maintainers.
What about 2.?

For 1 I agree with consensus so far to use the iii option.
For 2 I would pick i
However I also want to sleep on it a night cause it feels complex, when writing scripts consistency is key, or at least for me.

I reviewed all commands and put all inconsistencies in the list.
To recap, here are the things we need to decide:

Do we want to disambiguate similar options (eg. web URL vs. list URL vs. folder URL). Available options:

we always prefix options, even when it's clear what the option is for, eg. spo web get --webUrl instead of spo web get --url
we prefix only when an option is ambiguous, eg. spo folder get --webUrl --folderUrl instead of spo folder get --webUrl --url because it's not clear what --url is referring to
we stick to consistency and naming convention and if the option isn't prefixed, it's implicit that it refers to the last noun

Do we want to normalize the use of shorts across commands? Eg. in the different commands we use -u, -w and -s for URL. Extra consideration, what if we have multiple URL options in the command (webUrl, listUrl, folderUrl):

We use -u only for web/site
We don't use shorts at all

For 1. @plamber gave his preference to be iii (follow naming convention). What do you think @pnp/cli-for-microsoft-365-maintainers.
What about 2.?

For 2 I would go with 1

To add my thoughts...
I'm not a big fan of using shorts in my scripts, one of the big things I like about CLIs is that each command is descriptive and easy to read, however when using shorts they can introduce ambiguity as to which option is being used, i.e. I am more likely to have to consult the docs to understand what the short relates to. In the majority of cases, we don't use short options in our examples in the docs and prefer the use of the more descriptive long option.
Where there is a potential conflict I think that we should avoid using a short options, if we take the following examples, m365 teams channel get --teamId 00000000-0000-0000-0000-000000000000 --channelId<EMAIL_ADDRESS>could be rewritten as m365 teams channel get --i 00000000-0000-0000-0000-000000000000 --c<EMAIL_ADDRESS>to use the available short options, in contrast to m365 teams channel get --teamName "Team Name" --channelName "Channel Name" which uses the name based long options which do not have available shorts.
As a rule I think we should always be explicit in our naming convention, i.e. webUrl, listUrl, folderUrl, teamName, channelName.
This may introduce some duplication between the command name and the command options, however I would say that clarity always wins.

Thanks for sharing your thoughts @garrytrinder. With prefixing options, would you prefix them always, even though it's clear what the option is for, eg. spo web get --webUrl vs. spo web get --url or would you prefix it only where there are multiple URLs, IDs, and we want to disambiguate?

Thanks for sharing your thoughts @garrytrinder. With prefixing options, would you prefix them always, even though it's clear what the option is for, eg. spo web get --webUrl vs. spo web get --url or would you prefix it only where there are multiple URLs, IDs, and we want to disambiguate?

I would always prefix, it might seem overkill for a simple example like you have given, but it provides clarity for more complex examples where you have multiple options that are similar.

Understood @garrytrinder. @rabwill, @arjunumenon, @VelinGeorgiev care to share your opinion?

Been crazy weeks and that's why had been away from all CLI fun.

Do we want to disambiguate similar options (eg. web URL vs. list URL vs. folder URL). Available options:

I am with what @garrytrinder suggested (Option 1), giving full name for the parameter.

we always prefix options, even when it's clear what the option is for, eg. spo web get --webUrl instead of spo web get --url

Though the con with the approach is that, parameter name would be too big. I would personally be ok with that since our parameters are very clear for usage and the users need not refer the documentation for commands which has multiple id's.
For short params, my vote would be for this,

We use -u only for web/site

Never been a fan of short params myself. So cannot command which would be beneficial.
@waldekmastykarz  - You have done heavy fork-lifting by identifying the commands. Awesome ❤

@patrick-rodgers with having prefixes and renames, old ones will still be working as aliases maybe? in case someone is using it heavily?

@ValerasNarbutas the idea is to introduce these breaking changes in a major version so that we don't need to carry the existing option names forward. Having them available side-by-side would significantly complicate both the codebase and the documentation.

Since the discussion isn't complete and we consider changing our naming convention, we decided to postpone these changes to the next major version.

To better understand the impact of prefixing all options with the command noun, let's have a look at some of the examples from our docs:
m365 spo web add --webTitle Subsite --webDescription Subsite --webUrl subsite --webTemplate STS#0 --parentWebUrl https://contoso.sharepoint.com --webLocale 1033

m365 spo site add --siteType ClassicSite --siteUrl https://contoso.sharepoint.com/sites/team --siteTitle Team --siteOwners<EMAIL_ADDRESS>--siteTimeZone 4 --webTemplate STS#0 --removeDeletedSite --wait

m365 spo serviceprincipal permissionrequest approve --permissionRequestId 4dc4c043-25ee-40f2-81d3-b3bf63da7538

m365 aad groupsettingtemplate get --groupSettingTemplateId 62375ab9-6b52-47ed-826b-58e47e0e304b

m365 aad app get --appObjectId d75be2e1-0204-4f95-857d-51a37cf40be8

m365 search externalconnection add --externalConnectionId MyApp --externalConnectionName "My application" --externalConnectionDescription "Description of your application"

Looking at these examples I'm seeing two things:

examples and help would be harder to read because all (or at least most) options start with the same prefix. This makes it harder to scan the help to find the option you're looking for in help
there is a lot of repetition making the commands very verbose. Especially if you're not using command completion in your shell, that's a lot of extra typing

Does prefixing all options add clarity? Absolutely! But for the few cases where we need to disambiguate some options, I wonder if it makes to extend all options in all commands to be prefixed with the command noun. Looking at the examples above, my preference would be to:

keep the current naming convention
align where we identified misalignment
clarify where we see ambiguity

What do you think @pnp/cli-for-microsoft-365-maintainers?

I'm happy with the proposal 👍🏻
Great work @waldekmastykarz to provide the detail and clarity 👏

Thank you @garrytrinder for bringing up other ideas and constructive discussion on the possible directions 👏

Hi @waldekmastykarz ,
I am fine with your proposal.
I had some more thoughts about the impact of such a change and thought that it might be a good idea to introduce "alias" capabilities for parameters as we do with commands. By doing this we are not breaking any scripts after an upgrade.
In the documentation, I would not list all aliases, but just keep the "latest" version of the command.
Cheers

I think having both old and new option names would significantly complicate our code. Since options are validated, we'd either need to duplicate validation rules or we'd need to extend the CLI so that values passed through aliased options are mapped to the non-aliases equivalents before reaching validation. In both cases, we'd need tests to verify that all code paths work as intended. All of this work is temporary until the next major version and would be removed in the next year, so I'd suggest that we don't spend any additional time on it. We can decide if we want to include these new options in the upcoming v5 scheduled for end of Feb or in v6 in fall '22 to give people more time to update their scripts.

@pnp/cli-for-microsoft-365-maintainers which would have your preference:

include new options in v5 and delay releasing v5 until summer '22
include new options in v6 to be released in the fall of '22

Since this is not a critical issue there is no hurry. I would make it dependent on other activities and priorities

My vote would be to include in v6 rather than including the changes in v5 which may delay the release.
Vote for 2.

include new options in v6 to be released in the fall of '22

Planning for v6 then. If need be, we can always adjust.


Sidekiq 4.0

Hi,
Amazing gem you got here. Is it possible for you to look into moving it over to allow Sidekiq 4.0?
You need to drop the requirement of celluloid though.

Well, sidekiq-throttled does not depend on celluloid directly.
And yes, I'm thinking of making it sidekiq 4.0 friendly. In fact,
we will support both 3.x and 4.x sidekiq branches.
Stay tuned! ;))

+1

Any ETA on this @ixti ?

Thanks for great gem, any news on Sidekiq 4 support?

+1

+1

This looks awesome, appears to combine the utility of two different gems we're using. Supporting Sidekiq 4.0 would definitely be the topper.

:+1:

Any ETA on this @ixti?

Released with 0.3.0.pre. Please report issues if any. Will release 0.3.0 after a week.


[Bug]: Problem with trailing slash in actix routes when deployed into the platform

What happened?
So, I'm defining some actix routes as films/:
pub fn service<R: FilmRepository>(cfg: &mut ServiceConfig) {
    cfg.service(
        web::scope("/films")
            // GET
            .route("/", web::get().to(get_all::<R>))
            .route("/{film_id}", web::get().to(get::<R>))
            // POST
            .route("/", web::post().to(post::<R>))
            // PUT
            .route("/", web::put().to(put::<R>))
            // DELETE
            .route("/{film_id}", web::delete().to(delete::<R>)),
    );
}

When running cargo shuttle run, if I make a request to ``/films/it works. It doesn't if I do/films`. That's the expected.
The problem is that when I deploy this app to the platform, this behavior is different and no matter if I do /films/ or /films I'm getting a 404.
I suspect there must be a component in the platform normalizing the paths and removing this trailing slash.
For the time being I'm removing the final slashes but it would be nice for this behaviour to be consistent both locally and remotely.
Version
v0.18.0
Which operating system(s) are you seeing the problem on?
In deployment
Which CPU architectures are you seeing the problem on?
In deployment
Relevant log output
No response
Duplicate declaration

[X] I have searched the issues and there are none like this.

Ah, so this happens because we use tower-sanitize-path in our gateway proxy (which proxies requests to the users service). Perhaps we should just allow trailing slashes, as there doesn't seem to be a clear consensus on whether or not they should be used, and it getting cleared will be surprising to users. cc @chesedo

Resolved in https://github.com/shuttle-hq/tower-sanitize-path/pull/6, we just need to publish a release with the fix and we should be able to include the fix in Monday's release.


Issue in /src/pages/guides/authentication/JWT/index.md

I would like to get an example curl command, with actual values and not placeholders, to see how to get the token using curl.

Hi, we are no longer enhancing documentation for Service Account (JWT) credentials as they are deprecated.
Here are some links that may be helpful

Migration to OAuth Server-to-Server credentials
[Generating access tokens programmatically ](https://developer.adobe.com/developer-console/docs/guides/authentication/ServerToServerAuthentication/implementation/)


feat(stacks): Introduce Deployment Keys

Short Description
This is to introduce a feature called Deployment Keys, which will generate ECDSA based private and public keys.
The public key then can be leveraged in GitHub for enabling SSH key based Git clone for Stack deployments from a private GitHub repo. This will eliminate the limitation of users not being able to use Basic Authentication while 2FA is enabled.
Issues to Fix
This will fix #1752, #1753 and potentially #2448.
Future Improvements
Allow the Create Stack API to update with the new compose or stack .yml file if Swarm Stack exists.

Just wondering how you determined that 2FA doesn't work?
Check out:
https://help.github.com/en/github/authenticating-to-github/accessing-github-using-two-factor-authentication#using-two-factor-authentication-with-the-command-line
You can use a personal access token to authenticate using the CLI, which includes basic auth. For basic-auth style authentication, you provide A url like<EMAIL_ADDRESS>— or even<EMAIL_ADDRESS>

Hi there!
What is the current situation with this PR?

Is anyone working on this PR? Its quite important.

Upvote on this issue. It would be very nice to have!

Hello everyone! Does anyone know when would be possible to accept this PR? This feature is very important.

We have #1753 earmarked in release 2.2.0 and as such we'll probably be looking again into this by then.

We have #1753 earmarked in release 2.2.0 and as such we'll probably be looking again into this by then.

Closing as this piece of work is now really outdated.
👋 @ssbkang thanks again


fix for #3

fix for #3

better error handling and logging

makes it impossible to ctrl+c. fixed it.


Does this library have an ECCN?

Hello. Our team would like to use/include your library in our product, and since we export outside of the U.S., we're required to provide ECCN (Export Control Classification Number) information for all 3rd-party libraries that may use/include encryption.
We're asking because of code used in:
google-http-client\src\main\java\com\google\api\client\json\webtoken\JsonWebSignature.java
google-http-client\src\main\java\com\google\api\client\util\SecurityUtils.java

My very very unofficial suggestion would be to have your engineers and lawyers get together and classify this open source library with the number you feel is most accurate.
This project is in maintenance mode and we just don't have access to the resources required to come up with those classifications.


Fixed media query issues and added test to prevent it from coming back

Description
Fixed media query issues, removePadding and removeViewInsets removed values that were set to false
Related Issues
Fixes #38465
Tests
I added the following tests:
Tests that checks whether only setting some of the properties does not affect the others.
Checklist
Before you create this PR confirm that it meets all requirements listed below by checking the relevant checkboxes ([x]). This will ensure a smooth and quick review process.

[x] I read the Contributor Guide and followed the process outlined there for submitting PRs.
[x] I signed the CLA.
[x] I read and followed the Flutter Style Guide, including Features we expect every widget to implement.
[x] I updated/added relevant documentation (doc comments with ///).
[x] All existing and new tests are passing.
[x] The analyzer (flutter analyze --flutter-repo) does not report any problems on my PR.
[x] I am willing to follow-up on review comments in a timely manner.

Breaking Change
Not sure if this is considered a breaking change
The api does not change but if someone relies of the incorrect behaviour of the current implementation it may break.
Does your PR require Flutter developers to manually update their apps to accommodate your change?

[ ] Yes, this is a breaking change (Please read Handling breaking changes). Replace this with a link to the e-mail where you asked for input on this proposed change.
[ ] No, this is not a breaking change.

/cc @Piinks

This page from the API docs may be of some help: https://master-api.flutter.dev/flutter/widgets/MediaQueryData-class.html
It explains in detail how viewPadding, viewInsets and Padding are related to each other.

@Piinks I removed the unrelated formatting


Zarf Repair

We need self-healing for various Zarf components as well as troubleshooting guides for when known bad-states happen
Related to https://github.com/defenseunicorns/zarf/issues/752
Zarf agent and registry should be able to be quickly recovered in failure states either via controller / operator based algorithm or by administrative intervention in a simple and idempotent fashion.
This should include any tools and troubleshooting documentation to perform repairs when the system is in known failure states.

Closing this issue now as #1039 and #751 primarily met these objectives and any documentation needs will be picked up under the current documentation reviews.


Add option to force explicit exact and inexact object annotation

Now that exact_by_default  has been added as an option, library authors that expose their type definitions ideally would support both flow project that have exact_by_default set to true or false.
In order to do that, my understanding is that they'll need to explicitly define all object types as either exact or inexact, but Flow doesn't provide a way to enforce that, which makes it harder for lib authors to more safely support both values of exact_by_default.
Proposal
Add explicit_object_declaration that defaults to false, but when set to true will error if any object in the current project is declared without explicitly being defined as exact or inexact.
Example when explicit_object_declaration is set to true:
{ a: number }; // throw error, object needs to be declared as either exact or inexact.
{| a: number |}; // fine
{ a: number, ... }; // fine
$Exact<{a: number}>; // fine

// This is where it gets a little tricky...
type A = { a: number }; // let's say this type is only used on the following `$Exact` declaration
$Exact<A>; // fine

/cc @jbrown215

We have that as a lint already, but have not documented it yet. implicit-inexact-object
Will document it today.
It does not do the "tricky" case you listed above. I don't think that's desirable either, since A can still be used as an inexact object.

Thank you Jordan!
That’s great!
On Thu, Aug 22, 2019 at 8:05 AM Jordan Brown<EMAIL_ADDRESS>wrote:

We have that as a lint already, but have not documented it yet.
implicit-inexact-object
Will document it today.
It does not do the "tricky" case you listed above. I don't think that's
desirable either, since A can still be used as an inexact object.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/facebook/flow/issues/8034?email_source=notifications&email_token=AAAG64RTLHGXEIGVSORSEXLQF2TL3A5CNFSM4IOPQOR2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD45MPBQ#issuecomment-523945862,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAAG64WZ46F26DUFSDQRLFTQF2TL3ANCNFSM4IOPQORQ
.

--
Fabio Miranda Costa
twitter: @fabiomiranda
github: fabiomcosta

This is missing exact part of lint

so you're saying you want the lint to fire even if exact_by_default = false? alternatively, a complimentary "implicit-exact-object" lint?

shouldn't the library author just not use exact_by_default = true in that case to get the desired behavior?

@jbrown215 this is about .js.flow files I think
Library author can't know about what config user would have

Second lint makes sense, error on {} if exact_by_default is enabled, to have {||} (This wouldn't really be exact_by_default then, so idk really)
And implicit-inexact-object is for opposite case.
Also third lint explicit-exact-object which I've asked on Discord (error on {||} for projects)

Huh, I think it's really not possible to force both syntaxes

If implicit-inexact-object follows the behavior I mentioned, that's already what I was asking for.
This brings up a partially related point, maybe the documentation could include a guide/tutorial for setting up a project in a way that consumers of the project will have access to the Flow annotations/types.
That guide would mention things like:

Using flow-copy-source to generate .js.flow files before publish
Go through flow-typed and how the lib developer could leverage instead
Suggest enabling this lint rule
And probably other things that I can't remember now.


[Bug][Presentation Service] SPSP 2014 presentation page vs spsp 2014 presentation page

The SPSP 2014 presentation page has two pages, one doesn't have any posters.
https://osf.io/view/SPSP2014/ takes you to a blank presentation page.
https://osf.io/view/spsp2014/   takes you to the real presentation page.
I would expect both to take you to the same presentation page.
Katy
@jmcarp

Thanks @jmcarp -- This would be good to resolve asap, I think.


Support native DLLs in generate-load-scripts

Description
Related to https://github.com/dotnet/machinelearning/issues/92#issuecomment-390937877. Some .NET Nuget packages (CosmosDB in another culprit) bundle (and use) OS-specific native DLLs in their packages.
Paket's generate-load-scripts functionality correctly does #load and #r for all .NET packages assemblies, but doesn't currently support native DLLs. Unlike .NET assemblies, you need to add the native package folder to the path e.g.
let nativeDirectory = @"C:\Users\Isaac\.nuget\packages\microsoft.ml\0.1.0\runtimes\win-x64\native"
Environment.SetEnvironmentVariable("Path", Environment.GetEnvironmentVariable("Path") + ";" + nativeDirectory)

This is something with should be possible to achieve in Paket.

Would changes like this https://github.com/fsprojects/Paket/pull/3593 now make this possible?

@matthid ?

This should be possible for quite a while. You can already query paket for a list of native libraries for a given Runtime Identifier (RID) and framework moniker.

@isaacabraham What exactly is the expectation? That you load the load scripts and then PATH is modified with several new locations? I'm not sure that is actually a viable model. This is for two reasons:

The obvious potential side effect of editing PATH, which might change some of the downstream code or called software...  We might be able to minimize this by putting the new paths at the end. With the obvious downsides.
PATH itself has size restrictions as you can read here and here so we are not free to just add new stuff as we like.

It's maybe more conservative to provide some functions on every script:
load-script.fsx

let nativeLibraries = [
    "C:\Users\Isaac\.nuget\packages\microsoft.ml\0.1.0\runtimes\win-x64\native\some.dll"
    "C:\Users\Isaac\.nuget\packages\SQL\1.0\runtimes\win-x64\native\sql.dll"
  ]

let enableNativeSupport () =
	AssemblyLoadContext.Current.ResolveNative.add_handler(fun _ ->
        // do magic
    )

Then people could do something like:
myscript.fsx
#load ".paket/load-script.fsx"

LoadScript.enableNativeSupport(); // not sure what the call would look like

I'd be OK with this being an opt-in function that you explicitly call.
The approaches I've used to make this work before:

Add to the local process path with the risks of going too long
Use SetDllDirectory which works well, but only allows a single path so all the native dlls would need assembling. Unfortunately the much more hopeful AddDllDirectory only affects LoadLibraryEx which isn't used by pinvoke.
Copy the native files in the NuGet to where the .NET dlls are (obviously this horrible)

What I do in practice is copy the code to change the Windows environment variable to Jupyter notebooks for each library. Then when I share notebooks with other people I have to remember to add the LD_LIBRARY_PATH variant for Linux and macOS support e.g.
let libraryPath = Paket.RootPath + @"/packages/DiffSharp/runtimes/win/native"
match System.Environment.OSVersion.Platform with
| System.PlatformID.Win32NT ->
    System.Environment.SetEnvironmentVariable("Path", System.Environment.GetEnvironmentVariable("Path") + ";" + libraryPath)
| System.PlatformID.Unix
| System.PlatformID.MacOSX ->
    System.Environment.SetEnvironmentVariable("LD_LIBRARY_PATH", System.Environment.GetEnvironmentVariable("LD_LIBRARY_PATH") + ":" + libraryPath)
| _ -> failwith "Unknown platform"

Which is a lot of noise. We could just add this to the notebook Paket helper script but I'd like to be able to use it more generally.
Perhaps SetDllDirectory with AssemblyLoadContext would be possible? I've never used that, it's possible to intercept the native load with this?

OK, I get it. With the preview API it's very straightforward:
let handler = System.Func<Reflection.Assembly, string, IntPtr>(fun _ name ->
    match lookupNative name with
    | Some path -> LoadLibrary(path)
    | None -> IntPtr.Zero
)
System.Runtime.Loader.AssemblyLoadContext.Default.add_ResolvingUnmanagedDll(handler)

I've not integrated that to Paket yet though. On your FAKE change it looks like you have this working back to .NET Standard 1.6 as well. Is that correct?

@cgravill To get it work on earlier version (without this API) you need to create a new AssemblyLoadContext (basically inherit the type and create a new instance) and overwrite the LoadUnmanagedDll method.
I think the deal is that the "load request" has to start from this new load-context (ie from one of its assemblies). The most straightforward workaround would be to change your  main in a way to re-load in a modified load-context. Fake doesn't need this as we just load all new stuff into an assembly-context anyway and fake itself doesn't need native stuff.

I've tried a few things but not found a way to "re-load in a modified load-context". As far as I can see that fixed and can't be adjusted, am I missing something straightforward?
I can I see how you've got around the lack of API in earlier version, which is nice, and that works for more plugin like environment or separate context. But for cases in scripts like @isaacabraham where I guess you want to run unmodified code I've not been able to right the extension point.
Probably for my own use I'm either using .NET Framework and so will have to resort to modifying the environment variable or I'd be OK with requiring .NET Core 3.x

I ran into the same problem with Microsoft.Azure.Cosmos package.
Generate load script correctly generated #r's for the dlls in the lib folder:
#r "...\\microsoft.azure.cosmos\\3.8.0\\lib\\netstandard2.0\\Microsoft.Azure.Cosmos.Direct.dll"
#r "...\\microsoft.azure.cosmos\\3.8.0\\lib\\netstandard2.0\\Microsoft.Azure.Cosmos.Core.dll"
#r "...\\microsoft.azure.cosmos\\3.8.0\\lib\\netstandard2.0\\Microsoft.Azure.Cosmos.Serialization.HybridRow.dll"
#r "...\\microsoft.azure.cosmos\\3.8.0\\lib\\netstandard2.0\\Microsoft.Azure.Cosmos.Client.dll"

However, additional dlls from the package folder runtimes\win-x64\native are required. Without those dlls the script fails at runtime.
When I add the native folder via #I everything works fine:
#I @"...\packages\microsoft.azure.cosmos\3.8.0\runtimes\win-x64\native\"

As mentioned by Isaac in the description, #I is not supposed to work with native dlls. But it works for me in this case. Has this assumption changed in dotnet core fsi?


Enhancement Request: Update Extension To Not Require F5 User To Be Admin

It seems the only way this extension works is if the F5 user defined in the certificate store config is an F5 admin. Is there way to update this extension in the future to use a less privileged account than Admin?

That is an F5 restriction, unfortunately.

That is an F5 restriction, unfortunately.


⚠ Bofrost.fr has degraded performance

In a0a042e, Bofrost.fr (https://www.bofrost.fr) experienced degraded performance:

HTTP code: 200
Response time: 865 ms

Resolved: Bofrost.fr performance has improved in 46b520a after 7 minutes.


State for last update time

It would be helpful to introduce a new state updatedAt, which is a timestamp representing the last time we get the data from the fetcher or mutator.
Use Case
The typical use case for this feature will be showing a last refreshed time of the data in the UI which is critical for its realtimeness.
Without this new state one can only implement it with an effect hook to monitor the data change. However due to the deep comparison of SWR that doesn't work when data remains the same.

updatedAt then should be equal to Date.now right after the concurrent promise for this key resolves?
      startAt = CONCURRENT_PROMISES_TS[key]
      newData = await CONCURRENT_PROMISES[key]
      newState.updatedAt = Date.now()

Of course keeping the updatedAt as part of the tracked dependencies state.
And is anyone actively working on this?


Can't enter single or double quote using international English keyboard in online.visualstudio.com in Edge on Chrome

On US-International keyboards single and double quote are entered using "double quote followed by space". This simplifies entering umlauts using "double quote followed by a" for ä.

Closing in favor of https://github.com/MicrosoftDocs/vsonline/issues/225


SpatialAnchorManager is improperly configured

As indicated in the tutorial, adding the following components is required.
ARSessionOrigin (Script)
ARAnchorManager (Script)
However, after adding these two, the "SpatialAnchorManager is improperly configured" error message is still there. If I forced to start a session and create an anchor, the app will crash on Hololens. The following is my setup.
Unity 2020.3.11
MRTK 2.6.2
OpenXR 0.9.5
ASA SDK 2.9.0
DefaultOpenSRConfigurationFile is used.

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

ID: 97b7f65b-2300-1e88-3072-5ecc28e5a8e5
Version Independent ID: 2089610b-9581-ea9b-11c5-7be97e53c584
Content: Getting started with Azure Spatial Anchors - Mixed Reality
Content Source: mixed-reality-docs/mr-dev-docs/develop/unity/tutorials/mr-learning-asa-02.md
Product: mixed-reality
Technology: mr-dev
GitHub Login: @jessemcculloch
Microsoft Alias: jemccull

Do you see any other messages in the output like "Need an enabled ARCameraManager in the scene" or "The anchor prefab for ARAnchorManager must be set to null."?
If there isn't anything like this then most likely Spatial Perception Access is not granted. You can enable it on the Windows Store Build Settings select Publishing Settings by adding the “SpatialPerception” capability:

Here is my setup. The output has "Need an enabled ARCameraManager in the scene"  message. and the "SpatialAnchorManager is improperly configured". Are there other things I can do to find the problem?

I got to the tutorial's 4th step by the following change:

using the Azure Spatial Anchor tutorial's "AnchorModuleScript". The multi-user tutorial script has two error messages.
using the Windows Mixed Reality in the XR Plug-in Management. Using OpenXR will have error messages.
following the post, watzizname gave out a sultion.

So far, the tutorial works till step 4. Step 5 does not work.

@ttkrpink Do you have any full logs for us to take a look at?

I used ASA 2.10.0 for other projects, Those projects seem fine. I guess have to reimport everything for this messy project. Closing this issue, thanks for all your help.


MF-681 Fix incorrect message

Fix error message in read-mongoDB test

Codecov Report

Merging #682 into master will increase coverage by 0.05%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master     #682      +/-   ##
==========================================
+ Coverage   86.24%   86.29%   +0.05%
==========================================
  Files          62       62
  Lines        3722     3722
==========================================
+ Hits         3210     3212       +2
+ Misses        355      353       -2
  Partials      157      157

Impacted Files
Coverage Δ

things/service.go
94.15% <0%> (+1.29%)
:arrow_up:

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 83cafeb...e46fac6. Read the comment docs.

@nattaponra you need to sign off your PR in order to pass DCO. Here are the instructions: https://github.com/mainflux/mainflux/pull/682/checks?check_run_id=89125346

@nattaponra Please sign your commits  https://github.com/mainflux/mainflux/pull/682/checks?check_run_id=90470221

@nattaponra Please sign your commits https://github.com/mainflux/mainflux/pull/682/checks?check_run_id=90470221

Thank you for recommend
I'm trying to follow this instruction but still not pass.
Could you give me recommending or detail information about DCO because I am never used it?

Hi @nattaponra,
To sign a commit means that you have to add the '-s" switch to your "git commit" command.
As you haven't signed your older commits you'll have to execute something similar to:
git filter-branch --msg-filter "cat - && echo && echo 'Signed-off-by: Your Name<EMAIL_ADDRESS>HEAD~2..HEAD
HEAD~2 means start from the head (the current revision) and go 2 commits back, as you have made 6 commits you'll have to add HEAD~6 in your command. You can find more info about signing commits here and here about signing old commits.
HTH,
Jovan
P.S. You also need to update your fork. On how to do it check our contribution guide

@nattaponra please fix this PR ASAP or it will be closed due to inactivity.


Use 'with' operator for some download functions

Some functions in streaming_download_manager.py are not closing the file they open which lead to Unclosed file warnings in our code. This fixes a few of them.

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint. The docs are available until 30 days after the last update.

I was mistaken on the intent of those functions, closing the PR.


testing: correct benchmark output when running in parallel

Updates timing calculations to separate process time (summing the time of
goroutines when using b.RunParallel()) to more accurately represent the ns/op
metric and adds an ops/s metric to show benchmark throughput changes
as process count changes.
Fixes #28813

Signed CLA

This one's had no recent activity; closing


Scroll-to-bottom button has awkward highlight

When the user clicks on the scroll to bottom button the hover effect looks a little bit buggy(Basically the square around the button).

This highlight is generated by the WebView; as far as I can tell, we have no direct control over its shape – we'd have to eliminate it (-webkit-top-highlight-color: rgba(0,0,0,0)) and synthesize a new one.


ENH: Do not overwrite global warning formatter

The global warnings formatter is overwritten here:
https://github.com/shap/shap/blob/e51a87db4c9355968524dd6f788f9ec781e81924/shap/explainers/_tree.py#L23
This pattern is actually in the docs of the warnings module. However, this kind of import-time side-effect seems inappropriate for a package, as it will interfere with other packages' use of warnings.
A similar case caused issues in the pypdf2 package.

This decision of updating warnings package was very poor and it really hurt our development procedure. Our production code relies on many packages along and use heavily on warnings and all of the sudden, we got this issue and nobody could figure that out for about a year until we finally decide to delve into the issue. Our codebase is large and we cannot identify which makes this issue until today :(. I logged in to report this issue but it has already been reported. This issue should be more of a bug than an enhancement in my view.


Add user-agent support to the Parameters (Optional)

Some websites like '*.baidupcs.com' set anti hotlinking and verify user-agent, please add this parameter to optional parameters.

I don't know what user-agent is accepted by these sites.
But you can use profiles feature to set different user-agent and http-header-fields for special site:
mpv.conf:
Some options...

[baidupcs]
user-agent=MPV
http-header-fields='Field1: value1','Field2: value2'

userscript:
MPV Profile: default > baidupcs
But userscript doesn't support save per-site profile currently (may be implemented asap)

I don't know what user-agent is accepted by these sites. But you can use profiles feature to set different user-agent and http-header-fields for each site:
mpv.conf:
...

[baidupcs]
user-agent=mpv
http-header-fields='Field1: value1','Field2: value2'

userscript:
MPV Profile: default > baidupcs
But userscript doesn't support saving profiles by site currently (may be implemented asap)

Thanks for your reply, i originally expected to call mpv-handler in some websites via mpv://play/<base64>/?user-agent=pan.baidu.com, but  i find config by profile is a better flexible solution according to your prompt. This issue may probably be closed. Have a good day!


Add openjdk headless jre 8 in a docker container.

Fixes #1. This pull-request will be merged into branch 8 not master. Our automated builds create containers with version numbers that match their branch name. This means that each time we push to branch 8 a new container will automatically be built and tagged as version number 8.

What's the reasoning for openjdk vs. oracle?

I think this answers my question: http://askubuntu.com/questions/437752/openjdk-oracle-is-better

What's the reasoning for openjdk vs. oracle?

Yeah, there's not much of a difference between the two implementations now. Kinda like Chromium and Chrome, except with fewer differences. But openjdk has much more liberal licensing terms and as such is packaged really well for Ubuntu unlike Oracle's proprietary jdk.

:+1:


Financial goal planner endpoint

Issue reference - #444

@ighoshsubho Please review my pr

@ighoshsubho I have resolve the conflicts

@ighoshsubho Please review this pr


Add MongoDB C++ driver.

This PR add MongoDB C++ 11 client, which includes libbson, mongo-c-driver, mongo-cxx-driver, three ports.

Hi @sdcb, I'm your friendly neighborhood Microsoft Pull Request Bot (You can call me MSBOT). Thanks for your contribution!
You've already signed the contribution license agreement. Thanks!
The agreement was validated by Microsoft and real humans are currently evaluating your PR.
TTYL, MSBOT;