[Issue #289] TextQL - Implement test cases for Predicates

Implemented test cases for SelectAllFieldsPrecicate, SelectSomeFieldsPredicate and KeywordExtractPredicate. The constructor, getters, setters and getOperatorBean methods are tested.
More complex tests will be required in the future with the addition of new methods to these classes.

For our record, we did a Skype review.  After taking care of those comments, please do the merge.  Also create another PR to rename those functions "getXXXBean()" -> "generateXXXBean()".

PR #373 has been created to rename the function getOperatorBean to generateOperatorBean, as suggested above.


Add prettier

Thanks for your contribution. I really appreciate it.
If you do not mind, can you elaborate more about what you were doing?


Rolling upgrades are blocked by nodes that are not properly drained

What happened:
The issue has been described in https://gitlab.com/sylva-projects/sylva-core/-/issues/1595
During rolling upgrades, control-plane node machine is removed from etcd cluster as soon as the machine is being rolled out here.
The issue is that in rke2 deployments, kubelet is configured to use local api server (<IP_ADDRESS>:443), which in turn relies on local etcd pod. But as this node is removed from etcd cluster, kubelet won't be able to reach the API any more, and it will fail to properly drain the node as all pods will remain stuck in Terminating state from kubernetes perspective.
We should probably try to avoid removing etcd member so early during rolling upgrades, we could instead rely on periodic reconcileEtcdMembers that ensures the number of etcd members is in sync with the number of machines/nodes, this way etcd members will be removed only after the node has been properly drained and removed from cluster by capi controller.

@hardys @Danil-Grigorev @alexander-demicev , as I said today on Slack:
it seems to us that the longer it takes to drain, the more likely the occurrences are
failures due to this behavior (which seem to us as being a bug) are a frequent cause of CI failures in Sylva project pipelines, so this problem is quite "hot" for us


Recipe Issues .7.2

Weak Blood Shard Has no Recipe
Draconic Dust requires the Arc Furnace (immersive engineering) which is uncraftable

Weak Blood Shard is not an item with an recipe........
Draconic Dust is also not required considering ur create the ingot directly


Bumps Solnet deps version and Solnet.Pyth to v5.0.0

Status
Type
⚠ Core Change
Issue

Ready
Versioning/Tooling
No
Closes #20

Problem
Releasing v5.0.0
Turning release pipeline .NET version agnostic by using the packaging from the cake build

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 96.186%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
195

Relevant Lines:
196

💛  - Coveralls


using darker colors for scatter plot viz

This would address https://github.com/VEuPathDB/web-components/issues/183.
Darker color palette suggested at https://github.com/VEuPathDB/web-components/issues/183#issuecomment-903996686 is utilized. One issue is that darker colors are not that distinguishable as compared to default color palette. I have checked that the new darker color palette was correctly applied though. Please see the screenshots below. (I am asking @asizemore as well)
a) using default color palette

b) using darker color palette

I'm going to be annoying and ask that darkerColorPalette is defined in https://github.com/VEuPathDB/web-components/blob/main/src/types/plots/addOns.ts.
Before doing this, I think we should hear from @asizemore and make sure the subtle change is worth the effort.

@dmfalke Thank you for your review and comment 👍 I was gauging if I need to add it as a genetic but I thought it might be used at Scatter plot only. But I can definitely move it to addOns like colorPaletteDefault in there.
Actually I have asked her opinion after making this PR at the corresponding ticket, and she basically said okay. See this.

Actually I have asked her opinion after making this PR at the corresponding ticket, and she basically said okay. See this.

Okay, great! (Apologies for the ping, @asizemore!).
Let's go ahead and move it to web-components. Even if it is only ever used here, I would prefer all color palette definitions to be in one place. It's probably obvious, but I think it should be called colorPaletteDark. (ColorPaletteDefault should also be lowercase: colorPaletteDefault, if you don't mind making that change, too.)
Thanks!

Good observation. They should all start with lowercase. Could you make that
change as well?
On Thu, Sep 2, 2021 at 8:40 AM DK @.***> wrote:

Actually I have asked her opinion after making this PR at the
corresponding ticket, and she basically said okay. See this
https://github.com/VEuPathDB/web-components/issues/183#issuecomment-910295947
.
Okay, great! (Apologies for the ping, @asizemore
https://github.com/asizemore!).
Let's go ahead and move it to web-components. Even if it is only ever used
here, I would prefer all color palette definitions to be in one place. It's
probably obvious, but I think it should be called colorPaletteDark. (
ColorPaletteDefault should also be lowercase: colorPaletteDefault, if you
don't mind making that change, too.)
Thanks!
@dmfalke https://github.com/dmfalke No problem, I can move it. Just one
question is that all variable names in the addOnes.ts actually starting
with capital.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/VEuPathDB/web-eda/pull/355#issuecomment-911635036,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AACZEU3UAN3CEYHXBVYX7ETT75WFTANCNFSM5DEKKYWQ
.

Good observation. They should all start with lowercase. Could you make that change as well?
…
On Thu, Sep 2, 2021 at 8:40 AM DK @.***> wrote: Actually I have asked her opinion after making this PR at the corresponding ticket, and she basically said okay. See this <VEuPathDB/web-components#183 (comment)> . Okay, great! (Apologies for the ping, @asizemore https://github.com/asizemore!). Let's go ahead and move it to web-components. Even if it is only ever used here, I would prefer all color palette definitions to be in one place. It's probably obvious, but I think it should be called colorPaletteDark. ( ColorPaletteDefault should also be lowercase: colorPaletteDefault, if you don't mind making that change, too.) Thanks! @dmfalke https://github.com/dmfalke No problem, I can move it. Just one question is that all variable names in the addOnes.ts actually starting with capital. — You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub <#355 (comment)>, or unsubscribe https://github.com/notifications/unsubscribe-auth/AACZEU3UAN3CEYHXBVYX7ETT75WFTANCNFSM5DEKKYWQ .

@dmfalke Yeah I can change all of them but it would take some time to do that because they are used all of the places at web-components (and perhaps some of them at web-eda as well) 😄

Okay, in that case maybe we should follow the current convention of
capitalization and file a separate issue to tidy up, and address later.
On Thu, Sep 2, 2021 at 8:55 AM DK @.***> wrote:

Good observation. They should all start with lowercase. Could you make
that change as well?
… <#m_3439837677151993545_>
On Thu, Sep 2, 2021 at 8:40 AM DK @.***> wrote: Actually I have asked
her opinion after making this PR at the corresponding ticket, and she
basically said okay. See this <VEuPathDB/web-components#183 (comment)
https://github.com/VEuPathDB/web-components/issues/183#issuecomment-910295947>
. Okay, great! (Apologies for the ping, @asizemore
https://github.com/asizemore https://github.com/asizemore!). Let's go
ahead and move it to web-components. Even if it is only ever used here, I
would prefer all color palette definitions to be in one place. It's
probably obvious, but I think it should be called colorPaletteDark. (
ColorPaletteDefault should also be lowercase: colorPaletteDefault, if you
don't mind making that change, too.) Thanks! @dmfalke
https://github.com/dmfalke https://github.com/dmfalke No problem, I can
move it. Just one question is that all variable names in the addOnes.ts
actually starting with capital. — You are receiving this because you were
mentioned. Reply to this email directly, view it on GitHub <#355 (comment)
https://github.com/VEuPathDB/web-eda/pull/355#issuecomment-911635036>,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AACZEU3UAN3CEYHXBVYX7ETT75WFTANCNFSM5DEKKYWQ
.
@dmfalke https://github.com/dmfalke Yeah I can change all of them but
it would take some time to do that because they are used all of the places
at web-components (and perhaps some of them at web-eda as well) 😄
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/VEuPathDB/web-eda/pull/355#issuecomment-911648964,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AACZEU4UPUAMTQV32EXGXODT75X3BANCNFSM5DEKKYWQ
.

Okay, in that case maybe we should follow the current convention of capitalization and file a separate issue to tidy up, and address later.
…
On Thu, Sep 2, 2021 at 8:55 AM DK @.> wrote: Good observation. They should all start with lowercase. Could you make that change as well? … <#m_3439837677151993545_> On Thu, Sep 2, 2021 at 8:40 AM DK @.> wrote: Actually I have asked her opinion after making this PR at the corresponding ticket, and she basically said okay. See this <VEuPathDB/web-components#183 (comment) <VEuPathDB/web-components#183 (comment)>> . Okay, great! (Apologies for the ping, @asizemore https://github.com/asizemore https://github.com/asizemore!). Let's go ahead and move it to web-components. Even if it is only ever used here, I would prefer all color palette definitions to be in one place. It's probably obvious, but I think it should be called colorPaletteDark. ( ColorPaletteDefault should also be lowercase: colorPaletteDefault, if you don't mind making that change, too.) Thanks! @dmfalke https://github.com/dmfalke https://github.com/dmfalke No problem, I can move it. Just one question is that all variable names in the addOnes.ts actually starting with capital. — You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub <#355 (comment) <#355 (comment)>>, or unsubscribe https://github.com/notifications/unsubscribe-auth/AACZEU3UAN3CEYHXBVYX7ETT75WFTANCNFSM5DEKKYWQ . @dmfalke https://github.com/dmfalke Yeah I can change all of them but it would take some time to do that because they are used all of the places at web-components (and perhaps some of them at web-eda as well) 😄 — You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub <#355 (comment)>, or unsubscribe https://github.com/notifications/unsubscribe-auth/AACZEU4UPUAMTQV32EXGXODT75X3BANCNFSM5DEKKYWQ .

@dmfalke yeah it is a good idea to deal with overall changes as another tidy-up 👍

@dmfalke I have updated scatter plot viz to utilize addOns.ts at web-components. Also updated version and merge conflicts.


How to change the pricing tier for the services being deployed?

Hi,
I'm not sure where to post my question, so i pick the bug section to post it. Please advice me if i have posted in the wrong section.
Tool
Name: [Msbot]
Version: 4.0.7-88
OS: Windows 10
Describe the question
I'm using the following code to provisioning the enterprise bot.
msbot clone services --name "EnterpriseBotDemo" --luisAuthoringKey "[myLuisKey]" --folder "DeploymentScripts\msbotClone" --location "westeurope" --verbose --appId "[myAppId]" --appSecret "[myAppSecret]"
How can i change the pricing tier for the services being deployed? I can change most of the services after provisioned but i could not update the search service pricing tier. It seems the azure team saying that this service is not flexible. So i would like to find a way to change the pricing tier before provisioning it or a way to move the indexes from one search service to another one.
Any guidance or help on this will be appreciated.
[question]

This is a good question, but it would be best to post it in stackoverflow, we will address it there and I'll start looking into it for that.
The Microsoft Bot Framework team prefers that how to questions be submitted on Stack Overflow. The official Bot Framework Github repo  is the preferred platform for submitting bug fixes and feature requests.


add revaultd::get_vaults

Retrieve the vaults with a list of outpoints.
This fix the need to retrieve all vaults and filter
them by selected outpoints in the spend_transaction panel.

I absolutely don't see the point of having two different functions, it's redundant and confusing for someone whose comfortable with the revaultd api (why is a single function mapped to two different functions in the GUI?).
If you really want to have two different functions (and I'll repeat myself here: I don't think this is a good idea at all, it just makes things confusing) at least let's give them better names! It's not clear why one of the should be called get and the other one list, and there's no way anyone will remember that. Functions/variables names should be easy to remember or AT LEAST to guess, and get vs list is neither of them. You could instead use list_vaults_status_filter vs list_vaults_outpoints_filter, where at least one knows which function does what!
(But again, why should you do that? What's the problem in having a 'list_vaults' with two parameters?)
I'm sorry, I know I'm making a fuss around function names, but it's pretty clear that this change would make the code way more difficult to review and to write!

Your comment made it very clear: better to have one method :+1: I will change that


[examples] models_box_collision demo broken w/ OpenGL 1.x

The sphere collision doesn't look right...

Are you using latest develop branch? In you screenshots ball seems out of place, that's what I see in my computer:

Being doing lots of changes in internal maths system, please recompile latest develop branch.

Yeah, that was from the develop branch.  When I get home I'll take another look at it.

Just tested it now... it should work...

The sphere is still out of place for me.  I can test this on my OS X machine later any on my laptop tomorrow.  I'm not sure what's up.  I just rebased from develop a few minutes ago.

Reviewed again, it's ok for me. Please, could you verify this line: https://github.com/raysan5/raylib/blob/develop/src/models.c#L333
It must be rlScalef() and rlTranslatef() and not the other way round, it changed some days ago. Just clean installation and try to recompile again, maybe some old version of the library is installed around...

I check on another one of my Linux boxes and it isn't working right.  Doing a diff between your develop branch and my cmake_test doesn't have any differences between any source (other than Makefiles).
I also tested this on OS X, same problem is happening.  The collisions are happening where they are supposed to be, but well, it's the drawing of the sphere that is out of place.
Can you make a second example that has around 5 shapes?  It would be a better test than two.  When I was writing a ray tracer a couple of months ago the first object seemed fine if it was transformed, but when I started adding more and more objects it started to get crazy misplaced.  Turns out it was an issue with un-poping a matrix.

ok, I found the problem. It seems you're compiling raylib with flag GRAPHICS_API_OPENGL_11 to use OpenGL 1.1 backend instead of OpenGL 3.3.
Last week I reviewed raymath library and I messed something up, I mean I corrected lots of maths for OpenGL 3.3 but I reversed OpenGL 1.1 expected behaviour.
Just need to review rlTranslate(), rlScale() and rlRotate() implementations, probably just transposing the matrices before multiplication.

Any reason why raylib supports such and out of date version of OpenGL?  I know it's nice to say that we're backward compatible, but that API terribly old.  I think it would be less likely to have hangups like this if OpenGL 1.1 support was dropped.
While it only lists DirectX at the moment, I think taking a look at the Valve Hardware Survey would be a good metric to see which version(s) of OpenGL are relevant: http://store.steampowered.com/hwsurvey/videocard/

Yeah, this is an OpenGL 1.x issue.  I tried building using non 1.x and the sphere was in the correct location.
As for my previous comment of dropping OpenGL 1.x support, that might be something for a different discussion, or within another release or two of raylib.  I'll leave that up to you.  (my $0.02 though).

About OpenGL 1.1 support: https://github.com/raysan5/raylib/issues/334
I think OpenGL 1.1 immediate mode is quite useful to introduce students to graphics programming, that's the reason of rlgl; actually that layer just emulates a fixed function pipeline over a programmable one, letting the user code in an immediate mode style but translating calls to internal buffers and shaders use.

Yeah, I can understand that.  I've had my CG professors say that OpenGL 1.x is much easier to understand for new comers.  It's good that you're already trying to emulate it, but I think it would be best to actually drop it in the backend.
That way your users can have the OpenGL 1.x feel, but you won't have to actually maintain a legacy API.

Please, close the bug if it works on your side.

Just closing it, please feel free to reopen if issue persist.

Yeah, it's all good.


[TC-367] Download page must not link to dist.apache.org

The dist.apache.org SVN area is only intended as the staging area for the ASF mirror service.
Please do not link to files on it from download pages.
Sigs and hashes should link to:
https://www.apache.org/dist/incubator/trafficcontrol/
instead. And later to
https://www.apache.org/dist/trafficcontrol/
Also the download page needs an https link to the KEYS file
https://www.apache.org/dist/incubator/trafficcontrol/KEYS
and some instructions on how to check sigs or hashes.
Please see:
http://www.apache.org/dev/release-publishing.html#distribution_dist
Author: Sebb
JIRA Link: https://issues.apache.org/jira/browse/TC-367
Found Version: 2.1.0

Please note that hashes and sigs must link to https://www.apache.org/dist/incubator/trafficcontrol/...
They must not link to the 3rd party mirrors.
Also hashes must use the following extensions:
*.md5
*.sha512
See
http://www.apache.org/dev/release-distribution#sigs-and-sums

@hbeatty this is already done,  isn't it?

sebb's issue was that we are fetching the sigs from the third party mirrors. WE should get the sigs directly from apache.org/dist

The hashes are not even supposed to be present on the mirrors.
The sig (PGP) files are not present, so the links are broken.

corrected with commit #920054ba6621c31eae4213b45fe9385f023deb4a to https://git-wip-us.apache.org/repos/asf/incubator-trafficcontrol-website.git


tos

📝 Changed routes:

/tos

(view full size)

Commit e5ddde76c1402e6d6d0ac699a4f7b264e0ba2fd2 (https://thirdweb-848sgi7nw.thirdweb-preview.com).


Cannot disable AutoEnableVR

Unity 2020.3.5
OpenVR XR Plugin 1.0.1
SteamVR launches regardless of SteamVR_Setting.autoEnableVR being true or false
UnityEngine.XR.XRSetting.enabled is also false at build time for all build devices.
Every single camera (except the disabled VR Player's camera) has TargetEye set to none.
XR.Enabled and autoEnableVR do get reset at runtime based on user settings,
but they are both false by default, and SteamVR launches before the scene with that script even loads.

Same problem here. I've disabled auto-enabling toggles in XR plug-in manager settings and SteamVR_Resources/Resources/SteamVR_Settings but SteamVR still runs on app start.

 Screenshots

Unity ver.  2020.3.8.f1
XR plug-in management package ver. 4.0.6
SteamVR package ver. 2.7.3

I may have found part of the problem in the Player log
Player.log
You can see it initializes the subsystems, and then calls XRGeneralSettings.Awake first before anything else.
AFTER that, then you can see where I tried to stop it from happening with "SE_InitializeUnityScene:InitializeVR(SequencedEvent)"
code here:
private void InitializeVR(SequencedEvent a_event)
{
    bool _vrEnabled = MasterConfig.Instance.UnityDisplaySettings.useVR;
    XRManagerSettings _xrSettingsMgr = XRGeneralSettings.Instance.Manager;

    if(_vrEnabled)
    {
        _xrSettingsMgr.InitializeLoader();
        _xrSettingsMgr.StartSubsystems();
        XRSettings.gameViewRenderMode = GameViewRenderMode.RightEye;

        a_event.ProgressTextAddNewLine(($"VR Moduel Detected").AddRichTextTag("color", "yellow"));
        a_event.ProgressTextAddNewLine(($"\t VR Status: [ENABLED]").AddRichTextTag("color", "green"));
    }

    else
    {
        XRSettings.LoadDeviceByName("");
        _xrSettingsMgr.DeinitializeLoader();
        _xrSettingsMgr.StopSubsystems();
    }
}

Still,  SteamVR should not launch automatically if autoEnableVR is false, but here we can see Unity is basically forcing VR to enable if you have any of its subsystems in your application at all.
Hopefully someone smarter than me knows what to do next, because that's all I got

The auto enable vr checkbox is for enabling the steamvr subsystem in unity settings, not for launching it at runtime. Launching it at runtime is controlled by unity.

Ok, but both are disabled, and it's still running.
What's the proper way to prevent SteamVR from launching when my app does?

Hello @Mouledoux -- I just happened upon this issue while looking for something completely else. I agree with your assertion that SteamVR shouldn't launch if auto enable vr is false, and ran into the same problem myself. I discussed it on the Unity XR Plugin page, here: https://github.com/ValveSoftware/unity-xr-plugin/issues/80
@Verex found a solution that makes the OpenVR plugin obey the 'Initialize on Startup' option, and submitted it as a pull request here: https://github.com/ValveSoftware/unity-xr-plugin/pull/102
It hasn't been merged, or even acknowledged, but if you download their fork it works just like it says on the tin.

This could also be connected to the problem I fixed with PR #974 (isn't merged yet)
For me disabling autoEnableVR  just didnt work as this setting always gets overwritten with the default value on each asset import.


fix: proxyllm do not need controller_addr

reason:
for openai user in China, most of us use http_proxy or https_proxy for openai things,
but for the controller_addr  load reason when have this proxy in env, the pilot can not start, so we can check if use llmproxy to ignore it

@fangyinc I changed another
FYI, like https://github.com/eosphoros-ai/DB-GPT/issues/617 if any user behind proxy may have this issue, that's a common issue behind GFW

@fangyinc I changed another FYI, like #617 if any user behind proxy may have this issue, that's a common issue behind GFW

@yihong0618 Thank you for your effort. I believe this PR can now address the issue of DB-GPT not starting when behind a network proxy. As for a proper solution to connecting to OpenAI, we can design a new feature for that in the future.


Reverse DNS lookups FROM App Service is not currently supported

Please explicitly state that Reverse DNS lookups FROM App Service is not currently supported.
I have a customer that has opened a support ticket because he passes a IP address into the System.net.dns.gethostentry" method to perform a reverse lookup of clients that access the site and it is causing significant delays due to DNS failing.
i.e,
Can I configure reverse DNS for Azure App Service?
No. Reverse DNS is not supported for the Azure App Service.
Also, Reverse DNS lookups FROM App Service is not currently supported

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

ID: 61fbc38e-8756-1ce6-e73d-38bbe2a47a5b
Version Independent ID: f067d6bb-93b3-19ef-0b58-c58938fb1d6e
Content: Reverse DNS for Azure services - Azure DNS
Content Source: articles/dns/dns-reverse-dns-for-azure-services.md
Service: dns
GitHub Login: @rohinkoul
Microsoft Alias: rohink

@nelroosevelt , thank you for the feedback. I checked the referenced doc and it is explicitly mentioned in the FAQ section below that Reverse DNS is not supported for the Azure App Service.
https://docs.microsoft.com/en-us/azure/dns/dns-reverse-dns-for-azure-services#can-i-configure-reverse-dns-for-azure-app-service

We are closing this issue for now. If there are further questions regarding this matter, please reply and we will gladly continue the discussion.


Release v4.0.10

Title: Release v4.0.10
Description

Category: release

Changes and notes

Update CI for python 3.7-3.10
Update codeowners
update changelog, setup, and about for release

Testing

Not working - closing for now


Use GitHub workflows for CI

Same as #728, but trying a local branch to see if bors likes that better than a fork.
bors try

bors try

bors r+


Remove 'static + Sync bounds, API/documentation cleanup

Closes #2.
Additionally, I took it upon myself to cleanup the crate a little, adding documentation where I thought it was needed and removing some unsafe code.
I also renamed into_inner to unwrap because it suggests some work has to be done to get the inner value, whereas into_inner implies a simple destructuring.
Squash available upon request.
cc @Gankro, @huonw

r? @huonw
(I've picked a reviewer for you, use r? to override)

r? @reem (send/sync guru)

I think I should add a test that makes sure no returned slices overlap, even though I'm pretty sure they shouldn't. Thoughts?

Sync change looks good to me, but it makes the "returned slices must not overlap" guarantee very important, since it's not ok to remove the bound without it. That said, you can already cause UB via aliasing &mut if slices overlap.

ping @reem what's this status?

Did we just want a test to make sure that it doesn't produce overlapping slices?

A Mutex/Condvar refactoring + tests for overlapping slices is ideal.

I'm trying to think of how we could go about this. We could consolidate a lot using Mutex and Condvar. It could be a Mutex<usize> that replaces refcounting in Arc. AFAIK, the alloc crate will be unstable for quite a while yet but boxed::into_raw() and Box::from_raw() should be stable by 1.1. This crate could potentially work on Rust 1.1 Stable, but it'd require a bit more unsafe code.

@Gankro @reem
I have completed the refactoring, including using Mutex and Condvar to govern the unwrapping of slices. I also discovered, to my immense joy, that I could use *mut [T] instead of raw::Slice, which eliminates all unstable code from the main implementation and changes the scary transmute calls into simple pointer reifying expressions.
The test for nonoverlapping slices could be made more efficient by not testing each pair of slices twice, but implementing that would be much more in-depth than the test calls for, IMO.
TravisCI currently fails because the benchmarks need APIs from nightly Rust. I figure I can move the benchmarks into a separate module with a feature flag so the crate itself can build on stable.

Something also just came to mind. Since it doesn't really matter which thread the data ends up going to, wouldn't it be an ergonomics win if you could convert any ParSlice into a ParVec as well? Since the unwrapping is governed by Mutex there's no risk of a race condition, so any thread could actually try to unwrap it. If it's already been unwrapped then they just get an empty vector.

CC @aturon may be interested

I dunno if you guys want me to squash this or not since I'm the only contributor.

@Gankro @reem Is this good to merge or what?

What ever happened with this?

@apasel422 I really don't know! I thought it would be rude to continue bugging people about it so I just kinda forgot about it. The build still passes, though, so if the changes look good to you I'm fine with the merge.

Agh! Sorry for dropping this. Everything looks good to me, though maybe the note in the README should have the date updated. I'm just going to merge, and that can be updated in a follow-up PR.

End of an era, right here. :sob:


Idea: put-value

Would it be possible to create a version of shyaml/shjson that manipulated key-value pairs?  E.g.:
$ echo '{"a": 1, "b": 2}' | shjson put-value c 3
{
  "a": 1,
  "b": 2,
  "c": 3
}

or
$ echo '{"a": 1, "b": 2}' | shjson put-value b 7
{
  "a": 1,
  "b": 7
}

?

Here's a quick thought on this topic. Nothing really valuable, I'm throwing ideas here in case.
To answer my own question, the way to enter complex types thanks to the command line could be... in shjson itself.
$ cat <<EOF | shjson put-value b.x '{"u": 1, "v": 2}'
{
    "a": [1, "foo"],
    "b": {"x": 1, "y": 2},
    "c": 3
}
EOF
{
    "a": [1, "foo"],
    "b": {"x": {"u": 1, "v": 2},
          "y": 2},
    "c": 3
}

For literal values you would need to double-quote though:
$ ...  | shjson put-value b.x '"my string"'

I'm tempted to take queues from jq here (assuming everything to be a string, but providing functions to cast/interpret to other types).


create guide on how to trace app functionality to code

https://team.restarone.solutions/api_namespaces/2/resources/188
@alis-khadka and @Pralish are often consulted by the team for finding which parts of the code to change, this is an opportunity to create a guide so that their methods are documented for posterity

I usually use following ways to get into the code that is used for the current page.
1.) Using url on the browser's address bar.
Let's see which controller, its action, the template file is being used for the following page:

From the above picture, the url path is /mailbox/message_threads/4. Now lets see the rails routes for that path (filtering with some keyword from the url: **message_threads)**.

From the url http://localhost:5250/mailbox/message_threads/4, we know that 4 is some sort of id (parameter) we provide in the url and it is a GET request. From these clues, we can pin-point that the url is hitting mailbox/message_threads#show action.
Note: If something starts with : (example: :id, :message_thread_id), then it is like a parameter we need to provide in the url.

In the controller level, unless we see some explicit render or redirect_to, it will render the template file under the same path as the controller is  with the action's name. So, in this case, it will use a template file under: mailbox/message_threads/show.html.haml.


Update List of projects

PR Overview
Type of contribution

[x] Code
[ ] Documentation

Reference: Issue or Pull Request (PR)

[x] Fixes #41

Description: What does this implement/fix? Explain your changes.
Updates the projects list under the projects section to include Event board and the Data Umbrella PyMC sprint
Reminders

[x] Did you review the CONTRIBUTING GUIDE?
[x] Did you reference an issue or another pull request?

Any other comments?
None

LGTM.  Thanks @SangamSwadiK


Changing to a region without data doesn't clean the graphics

A clean of the graphics should be done when changing the Region instead of while receiving the data, otherwise old data are displayed.
If No data are received after a timeout, an error should be displayed
E.g. SophiaAntipolis -> Budapest

Regions are now obtained directly through the monitoring API, so all regions have data. Were a region not to have any data, interface would clean all previous data.


DateTime Picker appears and then disappears

There seems to be some spurious error in iPhone Safari where the picker appears briefly and then disappears. This is not consistently occurring.
Any thoughts?
Thanks, Jerry

Similar issue #52 was fixed previously. We checked it on iOS Safari, but it is not happening. Have you checked DateTimePicker Page on Safari? Please let me know if it happens on that page also.

Hello Neha... It's been a while. I actually noticed this problem on my iPad safari too. It happens on both of my iPhone 6 and iPad Pro.
Ok:

Other browsers are all ok (on the same device or otherwise)
Safari on iPad 2 is ok
Safari on Desktop is ok

Not Ok:

Safari on iPhone 6
Safari on iPad Pro

I can only speak for the devices that i have tested so far.
Thanks. Mike


add content to machine api controllers doc

also creates a new procedure doc for configuring repo integrations and
updates the ccm doc with the new common procedure.
also fixes up the process for red hat reps in the creating an openshift
repository doc.

@joelspeed @lobziik @alexander-demichev ptal

update

fix spelling error
fix formatting error with bullet point

@elmiko thanks a lot! Few minor suggestions:

Explain webhooks a bit
Add some references to further materials about things which was not covered there in details, such as MHCs.

Aside this, looks awesome to me

thanks for the great feedback folks, i'll work up some changes

update

many small changes to address feedback, i've tried to comment on each place that i changed

looks great to me :)

updated to address feedback

@elmiko This looks great.  LGTM


Content jumps in steps in after-content scroll animation

Describe the bug
Jittery content scroll, when going past content
Step to reproduce behavior
Method 1

Keep scrolling up slowly past end of content

Method 2

Scroll to end of content.
Wait for scroll bounce to stop bouncing -- or -- for content to stop moving completely
Scroll up a little, then quickly and aggressively scroll down past the content

Screenshots
https://user-images.githubusercontent.com/14130567/150850452-a7bd1d6d-e85a-4ad8-a6ab-a59b6c2d1aff.mp4

Still exists on 0.1.0 (28). Another note is that scrolling up with two fingers (scrolling all the way up, then using another finger to keep pulling up, then alternating back to original finger to keep pulling up) causes some jitters as well. This does not happen at the top of the content when pulling down with alternating fingers. I suspect it has to do with finding new lines / virtualized lists.

Needs more investigation 🤔

Could not reproduce in #321


Refactor QueryProxy delegation in AssociationProxy

Fixes #1242
This pull introduces/changes:

Delegates AssociationProxy querying methods using define_method, instead of using method_missing.

Pings:
@cheerfulstoic
@subvertallchris

[Current coverage][cc-pull] is 98.09%

Merging [#1243][cc-pull] into [master][cc-base-branch] will increase coverage by 0.74%

@@             master      #1243   diff @@
==========================================
  Files           189        189
  Lines         11333      11137   -196
  Methods           0          0
  Messages          0          0
  Branches          0          0
==========================================
- Hits          11033      10925   -108
+ Misses          300        212    -88
  Partials          0          0

Powered by Codecov. Last updated by [c74a98e...e4ca613][cc-compare]
[cc-base-branch]: https://codecov.io/gh/neo4jrb/neo4j/branch/master?src=pr
[cc-compare]: https://codecov.io/gh/neo4jrb/neo4j/compare/c74a98eb585ee364fed50986a4da6baa6891ed61...e4ca61393f1bc607bb17f604411729af3cb1ee7a?src=pr
[cc-pull]: https://codecov.io/gh/neo4jrb/neo4j/pull/1243?src=pr

I definitely like the refactor.  I always have trouble with this code, especially with the caching stuff so I always worry about breaking it, but as long as the specs pass I think we should move forward with it.
If you merge it I can cherry-pick to 7.1.x (and maybe 7.0.x as well?)

@cheerfulstoic Merged!

Released as 7.1.1, thanks!

@cheerfulstoic  ACH I forgot the Changelog as usual!

Done, thanks! ;)  It's better to do it on the release branch anyway when we're going to put something there because then we can just merge that to master


Extending user entity

What is the easiest way of extending the user entity to add more columns.
If you take a look skoruba admin for entity Framework it generates nuget packages and then anyone can extend the user entity
https://github.com/skoruba/IdentityServer4.Admin

Quickly answer your question:
The Identity Framework was isolated at Jp.Infra.CrossCutting.Identity. So if you wanna new properties for users, even changing the Id to int, you've to change Entities files there.
For Users, the file is Jp.Infra.CrossCutting.Identity.Entities.Identity.UserIdentity.cs
If you add new data or change the actual, then you have to track Profile models at

Jp.Application.Services.UserAppService

UserViewModel

Jp.Application.Services.UserManagerAppService

UpdateProfile(UserViewModel model)

If you pretend to use User Management UI, then you have to change Profile component:

JP-Project\src\Frontend\Jp.UserManagement\src\app\management\user\profile

Forgive my ignorance, but I can't see benefits by generating my own packages and providing for developers the ability to extend User Identity, because Identity itself have done that.
I'll appreciate if you can give me an explanation! And I'll check how Skoruba have done that.
What I've thought is to deliver an easy way to change Identity config by isolating it.

Thanks for the explanation. I suppose I want to add couple more fields to the user entity for a multi tenant application

But I guess I can fork the repo and the fields and build the new docker image


[5.10] Enable -user-module-version for 5.10

Following up on https://github.com/apple/swift-package-manager/pull/6795#issuecomment-1895830600, cherry-pick of https://github.com/apple/swift-package-manager/pull/7263.

Explanation:  #if canImport(LibFoo, _version: "1.2.3") is not currently supported by SwiftPM, as this feature was gated on .vNext, even though it landed before SwiftPM 5.10 branched off.
Scope: Isolated to llbuild integration and packages that use swift-tools-version: 5.10, have a valid semantic version with no prerelease or build metadata identifiers.
Risk: Low. It's a source-compatible change that has been previously exposed to tests in SwiftPM - just passes user-module-version through.
Testing: Has a corresponding test in both release/5.10 and main already.
Issue: rdar://121124842
Reviewer:  @neonichu on https://github.com/apple/swift-package-manager/pull/7263

@swift-ci test

@swift-ci test windows

This didn't make it in time for 5.10


DOC: in MWE.ipynb, add event example for magfig()

Also, note for broader impacts: This code was developed to look at HSJs, but could also be used to look at other types of transients.


Check source and destination echos to prevent implictly skipping tests

Please provide a description of this PR:
An example of skipped tests: https://github.com/istio/istio/pull/44711.

@jewertow: The following test failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

integ-pilot-istiodremote-mc_istio
8b8646e355dcf4c2a794a69d35c40284668e25f4
link
true
/test integ-pilot-istiodremote-mc

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

@jewertow: The following tests failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

integ-pilot-istiodremote-mc_istio
8b8646e355dcf4c2a794a69d35c40284668e25f4
link
true
/test integ-pilot-istiodremote-mc

integ-pilot-multicluster_istio
8b8646e355dcf4c2a794a69d35c40284668e25f4
link
true
/test integ-pilot-multicluster

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

/retest


Add a menu for the APP

I think we need to add a menu like Home, About Us, Contact Us at least

Menu created


[corechecks/snmp] Add additional metadata fields

What does this PR do?
[corechecks/snmp] Add additional metadata

Motivation
Adding new metadata fields that can be defined in profiles.
Example of usage in profile: https://github.com/DataDog/integrations-core/pull/10667

Additional Notes

Possible Drawbacks / Trade-offs

Describe how to test/QA your changes

Reviewer's Checklist

[ ] If known, an appropriate milestone has been selected; otherwise the Triage milestone is set.
[ ] The appropriate team/.. label has been applied, if known.
[ ] A release note has been added or the changelog/no-changelog label has been applied.
[ ] Changed code has automated tests for its functionality.
[ ] Adequate QA/testing plan information is provided if the qa/skip-qa label is not applied.
[ ] If applicable, docs team has been notified or an issue has been opened on the documentation repo.
[ ] If applicable, the need-change/operator and need-change/helm labels have been applied.
[ ] If applicable, the config template has been updated.

rebased onto
git rebase --onto main alex/snmp_profile_metadata


Library - Validate CBOR structs according to the cddl schema for nodes/models misc

The ockam_api crate exchanges many structs with the Controller. Those structs are serialized using CBOR and should conform to the cddl schema.
Currently there are only a handful of structs tested for conformance to the schema:

project (and related structs)
space (and related structs)
subscription (and related structs)

The purpose of this issue is to complete the testing by:

adding more quickcheck properties similar to the ones above
adding missing schema.cddl definitions where necessary
additionally remove the lifetime annotations on structs, like this one (and replace CowStr<'a> with String)

To the following list of structs:

nodes/models/base/NodeStatus
nodes/models/credentials/GetCredentialRequest
nodes/models/credentials/PresentCredentialRequest
nodes/models/flow_controls/AddConsumer
nodes/models/forwarder/CreateForwarder
nodes/models/forwarder/ForwarderInfo
nodes/models/identity/LongIdentityResponse
nodes/models/identity/ShortIdentityResponse
nodes/models/policy/Policy
nodes/models/policy/PolicyList
nodes/models/policy/Expression

[!NOTE]
Related epic: https://github.com/build-trust/ockam/issues/6235

We love helping new contributors! ❤
If you have questions or need help as you explore, please join us on Discord. If you're looking for other issues to contribute to, please checkout our good first issues.

did the first 2, looks easy enough. please assign

@shautvast That's awesome, all yours.
Please let us know if your have any questions as you explore. You can also ask questions on the contributors discord https://discord.ockam.io

submitted PR https://github.com/build-trust/ockam/pull/7169


[Question] How i change size of bar?

I use Neovim and increased my font and with that the size of the bar increased.
How i keep size bar with different size of font?

That is not possible

Am 25.02.2019 um 18:29 schrieb Filipe Medeiros<EMAIL_ADDRESS>I use Neovim and increased my font and with that the size of the bar increased.
How i keep size bar with different size of font?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or mute the thread.


Images not using Webp within content blocks

When this plugin gets content from a WordPress WYSIWYG and converts all images into Gatsby Images, it seems gatsby-image-sharp is not passed the webp value (I may be incorrect here - not fully sure on the package). This means articles are not using all that Gatsby Image has to offer. A significant SEO/page speed issue.
Is there any way I can set this? Nothing was included in the settings page for this.
Thanks,
Rob

Yesss this is very needed. Thanks for the feature request. I'm closing it for now as we're tracking the request over in #100
Thanks!


Update library to version 3.2.0

This change is

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 67.23%. Comparing base (8efc011) to head (25f3cfe).

Additional details and impacted files
@@           Coverage Diff           @@
##           master     #210   +/-   ##
=======================================
  Coverage   67.23%   67.23%
=======================================
  Files         441      441
  Lines       35277    35277
  Branches     4721     4721
=======================================
  Hits        23717    23717
  Misses      10466    10466
  Partials     1094     1094

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


Cache bust upon release.

Probably using ? querystring on css and js files.

I can just turn off caching on the CDN, if we're going to do that.
But I'd rather keep it on, if at all possible.

One option is to put the git commit hash as a querystring.
That way it'll still have the advantages of caching when the files haven't changed, but you won't need to wait the two hours for cache to expire. (If we want to be impatient about that)

Yeah, that's what I meant. Time stamp of release not literally the time now
:)
Commit hash also works. Commit hash of last mod to file even better not
sure how easy that is.
On Monday, 7 November 2016, Katelyn Gigante<EMAIL_ADDRESS>wrote:

One option is to put the git commit hash as a querystring.
That way it'll still have the advantages of caching when the files haven't
changed, but you won't need to wait the two hours for cache to expire. (If
we want to be impatient about that)
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/PennyDreadfulMTG/Penny-Dreadful-Tools/issues/583#issuecomment-258874932,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAAGBQhB-ICsk7qJ9dQwsO1vC0xi205Nks5q70o7gaJpZM4KrTBf
.

I added ?v= to the static resources.


Add tests catching table data mutations in pipelines and alpha-rarefaction

Improvement Description
Add unit/reversion tests to diversity pipelines and "informal pipelines" like alpha-rarefaction capable of detecting unintended mutations of table data.
Current Behavior
biom.Table.pa() and biom.Table.norm() mutate data by default, and require setting inplace=False to prevent mutation. Overlooking this parameter once in q2_diversity_lib nearly caused a data integrity bug in core-metrics and core-metrics-phylogenetic in 2020.6, in which q2_diversity_lib.observed_features would have mutated table data used in calculating other diversity measures downstream.
This bug was prevented by QIIME 2's general resistance to incidental data mutation :sweat_smile:. However, "informal" pipelines like alpha-rarefaction, which import and run vanilla python functions without using ctx are still susceptible and should test for misbehavior. Augmenting the core-metrics testing to catch reversions might also be useful.
Proposed Behavior
Tests on alpha-rarefaction and possibly core-metrics that catch table mutations with at least the default measures.
If there's a way of checking all measures for this without significantly increasing test times, that would be great.
Questions

Is there a way of testing this that provides adequate safety without brute-force permutation testing of all available measures?

This issue is being closed.
sdk.Context() appears to make it safe to inject context within Visualizers (e.g. q2_diversity.alpha_rarefaction), alleviating the concern. It's still possible to break things if a dev creates an "informal pipeline" by importing vanilla python functions, but this approach is neither recommended nor supported.
If this kind of test becomes necessary at any point in the future, a brute-force test isn't as bad as expected:

Begin with a collection of only metrics implemented in diversity-lib (local implementation is the most likely source of risk)
Use itertools to create duples of all permutations
For each duple, calculate first and second metrics separately (expected), and then calculate them at the same time by rarefying with the duple as the set metrics passed. Compare expected and observed - differences likely indicate a data mutation.
If the list of implemented metrics grows long enough to impact test run time, remove stable metrics from the tested collection.


feat(docs): Restyle!

Material UI for the documentation? It'd look good with icons. :thinking:

Oh yeah it's the fork, so had to rebase 44 commits. We'll just squash and merge.

Oh yeah it's the fork, so had to rebase 44 commits. We'll just squash and merge.

Also - anyone interested in Netlify CMS? That is just an admin panel, so if it is on our docs, we could make editing the content easier.

Also - anyone interested in Netlify CMS? That is just an admin panel, so if it is on our docs, we could make editing the content easier.

I reallay dont like this at all. But can we keep the deafault docosauraus theme but add the sick gradient background?

Expected that. Have you checked mui.com for the components which you like? I mean there is a lot of it, so if you want, choose some components you want and put it here. I too hate the MUI buttons, but the contact us at component seems good as a pop-up. I'll try adding the gradient, time to get inspired by xplorer.space since they use Docusaurus. Also, what do you think of switching the docs base to Typescript? I kinda like it

Btw since March is exam month for 13-year olds, did you pass this academic year? I did, glad about it, but did you guys get the LGTM of passing? Hope you guys got that!

Do whatever you think is best, let me look at Mui.com

What is the new workflow for?
I like the contact us popup and the icons
I think we should keep evrything as the default theme expect for the gradnt background

can we use #eb400a as the primary color snd the banner base color?

What is the new workflow for?

Which workflow? If it's the formatting thing, I rebased after 44 commits, hence it could have overlapped. I'll delete it now.

I like the contact us popup and the icons

Will add discord and GitHub there 🤔 ok?

can we use #eb400a as the primary color snd the banner base color?

I'll see about it, maybe if I choose different colors form the same color family, then #satisfied?

Yeah the formatting thing
Ok
Yeah, ,but keep thrr samr family since that is the timerx color  in the llogi

I'll try tomorrow. Free to use my dev center at that time.

This was effing open? Dang, I forgot with all the flipping school work. I think there is more to add, but my exams are coming. We both will have our holidays on October if I am not wrong, so I will try checking to integrate gradients and change the icon package. Or maybe we shall move to Astro.build(spoiler, it's freaking fast). Your ideas bud?

Well, ur the website boss

And you are the boss here. So, your ideas? I promise even if it doesn't seem like a good addition, I will make it as my base and refine that idea.

Yeah, the graident might look good. Is there anyway to use Microsoft's Fluent icons on the web?

Not any that I know, but we could try MUI. Microsoft Fluent icons are available via Figma and as SVGs, so I don't know.

Yeah, you can go ahead with MUI

Launching Project Restyle from October 2! Hope to add some pretty ground-breaking features by the end of my holidays!PS: tell more ideas, I will note it. Let's make sure it is less right now, so that we can execute via parts. Capische?

I have asked the Docusaurus team how to add gradients - I searched everywhere for a guide but couldn't find it. Meanwhile, I have a big feature upgrade coming for all of us.
Trust me, it will make our workflow easier.

Cool!


Extra inputs to CALLB PRINT in DISASM

Not probably an issue, but a request for clarification.
I'm playing with the DISASM output.
What's the meaning of the extra pushed arguments before a CALLB PRINT, n ?
For instance:
PRINT 10; 20; 30
leads to:
PUSH% 30
PUSH% 3 # ???
PUSH% 1 # ???
PUSH% 20
PUSH% 3 # ???
PUSH% 1 # ???
PUSH% 10
CALLB PRINT, 8
I can suppose they are internal flags to control the printing...
More generally, are you planning to publish a spec of the bytecode?

Those numbers pushed onto the stack are tags. PRINT has a very complicated syntax where commas and semicolons have meaning and where arguments can be of any type and may be present or not. So these tags encode the meaning of the separators and the type of the following arguments.
The syntax definition for the command is at https://github.com/endbasic/endbasic/blob/6484f66917efad3a71a135ab67773aa65dc28a2d/std/src/console/cmds.rs#L502 and then there is this monster parser at https://github.com/endbasic/endbasic/blob/6484f66917efad3a71a135ab67773aa65dc28a2d/core/src/compiler/args.rs#L534 to convert the arguments into stack values.
I hadn't thought about documenting the bytecode just yet because it's still in flux and it's not that useful on its own. Ideally, I'd like to polish the VM internals first to remove the typed stack (which requires adding a heap for strings and arrays), and then offer a way to assemble bytecode directly from the environment. The latter would be neat, and it's when you'd need documentation :)

Generating the bytecode directly as command line option for the endbasic interpreter would be helpful, if this is what you meant. Please keep also the textual version since easier to parse.
I'm trying to generate proprietary native code from the bytecode for an embedded processor. So the bytecode spec is useful for my scenario. I understand is still fluid, but at this stage I may survive with changes. About removing the typed stack, I'm not sure it will help me. It seems fine as is now. If not typed it will be difficult to transpile to native code.


Update deal

Hi, very nice coding, this is the most complete IG trading API out there in Node.JS.
Unfortunally I can see one important thing missing, and that is the ability to change a trade, like moving a stoploss or target. Is it possible to fix this missing piece??
Regards Fredrik

+1


Console.frm

Hey there,
here is a Version 0.1 of Console.frm
Im still actively working on it, as i need it for my own work, but i figured it would be a nice tool for stdVBA in future, even if its just for example use.
Anyway, your requests are WIP, i reduced it to only Console without the Errorhandling part and added a way to color it and added Intellisense.
Both isnt perfect yet (still proof of concept) but it works.
stdLambda implementation is on my todo list.
Extra feature:  Runtime Variable declaration using ConsoleVariables().
stdVBA implementation will take some time, cause i first have to understand how your library works :)

Hello, you can add an example folder to this seperate repo. Won't be uploading to this repo as discussed.

I've added your console app to the repo.
https://github.com/sancarn/stdVBA-examples/tree/main/Examples/ConsoleUserform
I would strongly recommend looking at the other examples in that repo to get to grips with stdVBA at large.


Lycanitemobs compatibility issue

Forge server stops at initialising DDD mod stage in case Lycanitemobs are present.
Confirmed on Lycanites versions: <IP_ADDRESS> and <IP_ADDRESS>
DDD version: 1.4.2
Steps to reproduce:

Set up a forge server with DDD and Lycanitemobs mods
Launch server

Expected result:
Server starts up
Actual results:
Server crashes on mod init stage
Crash logs attached:
https://pastebin.com/7Z1JLp2C

Converted paste to file as the paste was uploaded to Pastebin with an expiration date: crash.txt

Simple side error. Client stuff being done server side. Should be a simple fix, which I will include in the next update when I get the time to work on it. Thank you!


Replace the parent image only with the latest one

Due to regressing content, it might happen, in rare conditions, that the parent image of the image we are rebuilding after a CVE fix doesn't include all the latest fixes.
Example: we have 3 images:
NVR | CVE fixes
foo-1-125: fix-a, fix-b
foo-1-124: fix-a, fix-c
bar-2-220: fix-a, fix-b
foo is parent of bar. Before this change, Freshmaker would ask Pyxis if there's already a parent image fixing fix-c, and it would return foo-1-124, so that Freshmaker can use that as parent of bar. But that image is missing fix-b!
To prevent this issue, from now on, Freshmaker ask to Pyxis only the latest image.
Tests are not updated, since the current tests should cover the case.

@qixiang @FernandesMF PTAL
Please notice that I didn't add any new tests, since the existing one is testing the function already.
I thought about one that would test this corner case, but since we are mocking the Pyxis data nothing came to my mind.
In case you have any suggestions I'll be happy to add some more tests. Thank you.

@FernandesMF ok, thanks for the heads-up. I think I'll rebase on main to include your changes, and if it keeps failing I'll add a commit to fix the black check.

@qixiang @FernandesMF could you please check again? I've fixed black, but I'm not really sure why it still fails. I've applied all the suggestions it prompted.

Hi @gnaponie! I pulled the code from your branch and took a look at black. It turns out black is acting weird indeed.
When I ran black, it complained about line 61 in test_pyxis_gql_async.py,
        fake_query = ds.Query.find_repositories(page=0, page_size=2, filter={},).select(

and wanted to rewrite it the way you had it before with respect to the arguments of find_repositories.
Seems like we found a bug, and black is not idempotent in this case.
I found a quick solution of disabling it for this line. You can put a   # fmt: skip by the end of the select block:
        fake_query = ds.Query.find_repositories(page=0, page_size=2, filter={},).select(
           ...
        )  # fmt: skip

at least, that did the trick in my test. Notice the two spaces before the #!

Oh, and I think we need to manually squash all our commits together in Freshmaker. So don't forget to squash all your commits when you are ready to merge!

Hi @gnaponie! I pulled the code from your branch and took a look at black. It turns out black is acting weird indeed.

If you're running the same version black (as it is in the github tests) locally, it should show you consistent result. This is caused by a change in black version 23 (https://github.com/psf/black/blob/ddfecf06c13dd86205c851e340124e325ed82c5c/CHANGES.md?plain=1#L323).
It's unnecessary to have the magic trailing comma in:
fake_query = ds.Query.find_repositories(page=0, page_size=2, filter={},).select(

so we can just remove it:
fake_query = ds.Query.find_repositories(page=0, page_size=2, filter={}).select(

The rest of the code looks good to me.


AssociatedAlgorithmIdentifier implementation

This is an implementation of a trait defined at https://github.com/RustCrypto/formats/pull/954

Thanks! Could you possibly 1) make the ID string consts at the top of the file, and 2) put some links in the comments for where the strings come from?

@baloo this PR has support for PKCS1v15. Could you please check whether it works for you?

Yeah, I was missing the blanket impl for SignatureAlgorithmIdentifier. That works for me.

Which of these (https://github.com/RustCrypto/RSA/pull/278 vs https://github.com/RustCrypto/RSA/pull/284) should I be reviewing? They have many of the same changes.

This ons (#278). #284 was a spin-off w/o pkcs1v15 changes which were controvercial.

@tarcieri rsa crate doesn't have direct dependency on spki/der. Would it be possible to cut a dummy point release for pkcs8 to bump its internaly dependecy? Or we'd better declare rsa -> spki dependency?

@lumag already in progress: https://github.com/RustCrypto/formats/pull/983

Thanks! didn't notice it.

Ugh. @tarcieri Another request, but now for pkcs1, to get RsaPssParams::SALT_LEN_DEFAULT

@tarcieri thank you!


Ability to add labels to namespaces

This enables adding labels to namespaces.
Only makes a change to current charts outputs if the namespaces.*.labels value is set.

this LGTM.


Patch/iwa self signed

Following the issue pointed out by @gayl0000 and the advice given by @XuemingRocks, I've updated this sample to return the correct response when encountering AuthenticationChallenge.Type.SELF_SIGNED_CHALLENGE.
I've also noticed that it would be beneficial to check if the entered URL has the 'https://' prefix, and add it if it isn't in there already.

Good point @Rachael-E .
@tschie can you please advise on the merging order with the new release?


Add the possibility to upload images to a company.

 Add the possibility to upload images to a company.

@lgnstn updated.


Try cpu usage on windows

This was a try to test a slightly different way to gather cpu usage on windows, but it provide to have similar results. Closing the issue.


twitter filter pls!

There are more and more twitter content on the main page what makes it hard to read. I think it would be enough to filter out the pictures from twitter, but I am happy with any other solution which gives back the old HUP style when we had more text then images.
Thanks!

Hey!
Published to development channel.
If you wan't to try on Chrome, please contact me, I will need a google account enabled email address, so you can access to it.

Released in v2.5.0


Introduction - Nicole Schwartz (GitLab)

Hello,
My name is Nicole Schwartz and I am part of the GitLab Secure stage (specifically - Software Composition Analysis group). Outside of work I am part of an IoT research project (IoD), board member and temporary deputy-COO of the Diana Initiative information security conference, a staff member of SkyTalks (Def Con village), and a long time hacker / information security enthusiast.
I'm always very interested in improving process for systems so that all participants know what is expected and can contribute more effectively, especially around vulnerability disclosures.
I'd like to participate in the vulnerability disclosure working group, which matches up with my groups areas of open source and security.
Thank you,
Nicole
PR https://github.com/ossf/wg-vulnerability-disclosures/pull/43

@NicoleSchwartz Welcome to the WG!

The corresponding PR (#43) has now landed, I think we can close this issue.


Create A Project Wide Testing Standard

Our front-end testing should be standardized and present for everything we do.

I vote for Jest


Default dimensions feature

Add the ability to choose some default dimensions such as:

720p
1080p
4K

Hi @deamme I am happy to look at this but it requires more information. Where are the 'default dimensions' in reference to?

@seal62 Right now we have a default dimension of 1920x1080 (1080p) which is fine for now but could be cool to have a setting called size or something where you can give it a string of 720p, 1080p or 4K. Although I would say issue #4 is on a higher priority.

@deamme : I would like to pick this

@anusha5695 Go for it!


feat: Galax to Gala io

First use case for as_interop  (#298)
Major cleanup of I/O.
consolidate a lot of private API.
Better docs: 1. plum.dispatch combines the docs for IDEs, 2. we can show round-tripping of the potential.
Access to galpy via galax -> gala -> galpy!

Gonna add more docstring tests.

WTF? the CI isn't triggering.

Potentials requiring GSL remain untested.

Ping @adrn, does this look good to you?

I'm going to branch this PR to get started on the galax <-> galpy I/O.


Adopt Normalize

adopt Normalize https://github.com/smkhalsa/normalize
use gql_pedantic
streamline GitHub actions

@smkhalsa, please review


Question: destroy function overide.

Why you are not creating empty destroy fnction when function is nor provided? Instead of throwing an error

AOT doesn't support it.


Collection: SolrCloud integration

We should provide data collection in cloud mode too. The idea is that we can potentially collect different information from the case of a slaves/pool.

Hi, are you still working on making the solution work in cloud mode?


[Suggestion] Digital Miner.

I am usually of the habit of travelling with my digital miner powering it with an energy cube. On returning to my base I empty it using an AE import bus strategically placed.
I would like to request that an energy cube inside the digital miner would be recharged if the digital miner is being fed power from an external source.
This effectively makes the energy cube an upgrade for the digital miner.

This may come in a later version as an upgrade shipped with all machines - something where the machine's extra energy buffer can fill up a battery placed inside.  I'd rather not make this DM-specific.


Question about Figure 8 in paper

Thank you for your nice work!
I am wondering how to get the attention map of the picked point, could you give a simple introduction?

You cna just take the certain row of the self-attention matrix softmax(QK) and get an HWx1 matrix. Then reshaping it into HxW will be ok. For example, if you want to take the central pixel's attention map, you can choose the (HW/2)th row of the self-attention matrix softmax(QK).

Got it, thank you very much!


Missing dns dependency

After installing and running in NextJS i get the following error on startup
../../node_modules/@immobiliarelabs/dats/dist/dns-cache.js:3:0
Module not found: Can't resolve 'dns'

Import trace for requested module:
<EMAIL_ADDRESS><EMAIL_ADDRESS>

Hi @jajvisser 👋
dns is part of the Node.js standard library, if you want to use this package in Next.js you'll have to use it only on the server-side, eg: getServerSideProps

Hi, thanks for pointing me in the right direction.
The problem lies a bit deeper, the package is always included in the build to the frontend. Bable always includes it in the build. Only using it in getServerSideProps won't work.
I'm currently looking at excluding the package from the frontend build.
  webpack: (config, { isServer, webpack }) => {
    if (!isServer) {
      config.plugins.push(new webpack.IgnorePlugin({ resourceRegExp: /\@immobiliarelabs/ }))
    }
    return config
  },

Unfortunatly this is not a working solution for the project setup we are running (a mono repo). But i'm getting there.

Yes, my solution above works


Login Tool

I love this tool.  I have returned to it time and time again.  However, I am unsure of how to implement the login function tho.  What is the easiest and most straightforward way of utilizing the login before loading the map or layers?  Should this happen via js or php?

Authentication is outside the scope of this template. Probably the simplest method would be Basic Authentication via PHP or Apache.


Link to GitHub Apps listings for Python dependency management

Updates the notifications FAQ to link to a list curated by GitHub (and relatively straightforward for new apps to be listed on). Removes PyPI's implicit curation of apps.
Further discussion at https://github.com/pypa/warehouse/pull/3915.

Temporarily canceling the Travis build here so we can get some higher priority things merged, I'll restart it after I review.

Thanks @greysteil!


Return invalid exec path error when system command returns 127

I ran into an issue while running in circleci. My exec_path wasn't setup correctly but I was still getting an :ok because the ErlangError wasn't getting raised and it was really hard to figure out what's wrong.
So this pr returns {:error, :invalid_exec_path} whenever System.cmd returns the error code 127 that means that the command was not found in our PATH.
I also deleted some clauses that can never match according to dialyzer and fixed one command in the README that was wrong 😄

Thanks for the PR @amarantedaniel, I will add dialyxir to the dependencies as well.


copier: 7.0.1 -> 8.1.0

Description of changes
Closes #248297
Changelog: https://github.com/copier-org/copier/compare/v7.1.0...v8.1.0

Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[x] Tested basic functionality of all binary files (usually in ./result/bin/)
23.11 Release Notes (or backporting 23.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

This pull request has been mentioned on NixOS Discourse. There might be relevant details there:
https://discourse.nixos.org/t/prs-ready-for-review/3032/2555

Result of nixpkgs-review pr 249483 run on x86_64-linux 1

2 packages built:

copier
copier.dist


Wrong date in README.md

I presume you mean to say
Currently (2 Jan 2016) the project is in release candidate state
as opposed to
Currently (2 Jan 2015) the project is in release candidate state
?

Just helping - PR #133 already open for the same.


[RFC] Named macro capture groups

This RFC proposes optional names for repetition groups in macros:
macro_rules! foo {
    ( $group1:( $a:ident ),+ ) => {
        $group1( println!("{}", $a); )+
    }
}

Rendered
Small Pre-RFC: https://internals.rust-lang.org/t/pre-rfc-named-capture-groups-for-macros/20883

@rustbot label +T-lang +A-macros

Aren't there still some subtle questions around nesting one has to worry about? Specifically, the RFC doesn't seem to say anything about nested cases like ( $group1:( $a:ident [ $group2:( $b:ident ),+ ] ),+ ). Presumably I will only be allowed to use $group2 inside a $group1(...) repetition?

Although I cannot remember the exact use-case, I remember wanting something like this to name an optional trailing comma so that I could forward it on to another macro as part of the expansion. 👍

Aren't there still some subtle questions around nesting one has to worry about? Specifically, the RFC doesn't seem to say anything about nested cases like ( $group1:( $a:ident [ $group2:( $b:ident ),+ ] ),+ ). Presumably I will only be allowed to use $group2 inside a $group1(...) repetition? But there can be arbitrarily many other repetitions "in between" -- group2 will always refer to the repetition instance in the "current" group1?

Great point, the scope part is tricky - I think what you said is correct. It is probably best if I just add an appendix of examples that do and don't work with reasoning, especially given your comments on Zulip about the existing macro mental model not being very clear (I completely agree).

cc @markbt


.mjs node module not working after eject

Is this a bug report?
Yes
Can you also reproduce the problem with npm 4.x?
Unsure, using yarn 0.27.5
Which terms did you search for in User Guide?
webkit, constructor
Uncaught TypeError: __WEBPACK_IMPORTED_MODULE_1_universal_router___default.a is not a constructor
Environment

node -v: v8.2.1
npm -v: 5.3.0
yarn --version (if you use Yarn): 0.27.5
npm ls react-scripts (if you haven’t ejected): [ejected]

Then, specify:

Operating system: Mac OS X High Sierra
Browser and version (if relevant):
Chrome Version 62.0.3202.94 (Official Build) (64-bit)

Steps to Reproduce

(Write your steps here:)

Use create-react-app to create a new react app
Eject the react app
Install "Universal Router" using yarn (yarn add universal-router)
Add the following lines to your src/index.js

import UniversalRouter from 'universal-router';
console.log(UniversalRouter);
const router = new UniversalRouter([]);

Expected Behavior
The console output should show that UniversalRouter is a proper constructor function and no errors should occur.
Actual Behavior
The console outputs /static/media/browser.99d6d764.mjs and the following error is shown:
TypeError: __WEBPACK_IMPORTED_MODULE_1_universal_router___default.a is not a constructor

Possible fix
I found that the following fixed this for me:
On line 201 of the generated webpack.config.dev.js replace:
exclude: [/\.js$/, /\.html$/, /\.json$/],
with
exclude: [/\.m?js$/, /\.html$/, /\.json$/],
This might also need to be done for the .prod version.

Should've mentioned that this was originally implemented in #3237

So does jsx suffer from this as well?

Is anyone working on this? If not then I'd like to look into it.

Go ahead @iansu

Didn't have time to go after this earlier.
If I rename a .js file that I include to a .jsx then I end up with the same error as in my original report. If I then change the m?js to m?jsx? it'll correctly match the extension and parse again and that seems to fix the issue.


JpmsProvidesValidator: Read the whole class into memory

Resolves https://issues.redhat.com/browse/RHEL-67337

Strange and interesting. I do wonder if this was the cause of the problems.


Use ws_ping_interval to avoid disconnects

I'm going to merge this to try out if this fixes the problem, we can revert it later if it doesn't

Unfortunately, there are several problems with buildbot 3.10.1+ so I had to downgrade: https://github.com/python/buildmaster-config/pull/240 :(


Syntax highlighting does't work

react-simple-code-editor: "0.9.15"
prismjs: "1.17.1"
Example: https://codesandbox.io/embed/infallible-surf-vuhdn

So the part that the documentation about the demo that is missing is the CSS that controls the syntax highlighting. In the demo starting from L86 https://github.com/satya164/react-simple-code-editor/blob/master/example/styles.css#L86 there are CSS selector rules that control the markup returned by prismjs. You can either choose to do this yourself or choose a pre-defined theme yourself by importing from the prismjs/themes/ folder.
For example import 'prismjs/themes/prism.css'

Thanks! It works now.

@satya164  can you please add about example import statement of css in readme. I had to debug for an hour to find out what's missing and finally found a solution here


Problems installing with Carthage

I will admit this is my first time using Carthage so I may be missing something simple but Carthage keeps failing to update its Cartfile after the addition of this project. It looks like a possible issue with the swift 5 changes?

Hi, sorry for the super late reply. Did you ever get this working?


Add support of anchor deserialzation

@guibescos Hi Guillermo, it took me some time to fix this issue. I tried your suggestion of writing my own PythPriceFeedAccount instead of using the Anchor official Account, but it ended up with pretty long code and complex compiler errors.
My latest solution is to use Price instead of PriceFeed or PriceAccount for the deserialization. The code now seems neat and it is consistent with Anchor's recommended way of deserializing account from non-anchor programs. (Anchor's recommendation is here)
Please take a look. Thanks!

Did you try rewriting Account as I advised

Did you try rewriting Account as I advised. I think that approach is easier to move to the sdk.

Hey Guillermo, yes I did and please see this branch
I copied many things from account.rs to state.rs and renamed Account to PythPriceAccount. And then try to use PythPriceAccount directly in the Accounts struct.
I am getting compiler errors related to the Accounts macro and I don't really know how to solve it. Any help or suggestion would be appreciated!

After discussing with Guillermo, we decided to drop the approach of rewriting Account.
After discussing with Ali, we decided to use PriceFeed and implement Deref for it.


Summary not showing up anymore

I don't know if it's because of some Microsoft Edge update, but I can't see the skill list summary at the top of the tree anymore.
Debug Info:
{
    "Debug Info from Tab 55": {
        "username": "JFPili",
        "userId": "680346058",
        "UICode": "en",
        "languageCode": "zs",
        "language": "Chinese",
        "mastered": [],
        "requestId": 1,
        "requestsPending": 0,
        "onMainPage": false,
        "onLoginPage": false,
        "inMobileLayout": false,
        "progress": [
            [
<PHONE_NUMBER>000,
                0,
                58
            ]
        ],
        "Total Storage Usage": "There are 2243B/102400B (2.2%) being used in total",
        "progress Storage Usage": "There are 51B/8192B (0.6%) being used in a single entry",
        "mastered Storage Usage": "There are 0B/8192B (0.0%) being used in a single entry",
        "options Storage Usage": "There are 2160B/8192B (26.4%) being used in a single entry",
        "treeLevels Storage Usage": "There are 32B/8192B (0.4%) being used in a single entry",
        "chrome.storage.sync": {
            "options": {
                "XPBreakdown": true,
                "XPInfo": false,
                "XPInfoInPopup": false,
                "XPInfoInSidebar": true,
                "XPPrediction": true,
                "addTipsPageBottomButtons": true,
                "addTipsPagePractiseButton": true,
                "bonusSkillsBreakdown": true,
                "checkpointButtons": true,
                "checkpointPrediction": true,
                "crackedPopoutButton": true,
                "crackedSkillsList": true,
                "crackedSkillsListLength": "10",
                "crackedSkillsListSortOrder": "0",
                "crownZeroPractiseButton": false,
                "crownsBreakdown": true,
                "crownsBreakdownShowZerosRows": true,
                "crownsGraph": true,
                "crownsInfo": true,
                "crownsInfoInPopup": true,
                "crownsInfoInSidebar": false,
                "crownsMaximum": true,
                "crownsPercentage": true,
                "darkOptions": true,
                "fixedSidebar": false,
                "focusFirstSkill": true,
                "focusMode": false,
                "focusModeButton": true,
                "focusPriorities": "ncs",
                "goldenTreePercentage": false,
                "grammarMasteredThreshold": "2",
                "grammarSkillsBreakdown": true,
                "grammarSkillsTestButton": true,
                "hideCartoons": false,
                "hideSuggestionNonStrengthened": true,
                "hideSuggestionWithCrackedSkills": true,
                "ignoreMasteredSkills": true,
                "keepQuestionBorders": false,
                "languagesInfo": false,
                "languagesInfoSortOrder": "0",
                "legendaryTipsPageButton": true,
                "lessonThreshold": "4",
                "masteredButton": true,
                "masteredCriterion": "markedOnly",
                "masteredThreshold": "5",
                "needsStrengtheningList": true,
                "needsStrengtheningListLength": "10",
                "needsStrengtheningListSortOrder": "0",
                "needsStrengtheningPopoutButton": true,
                "noStrengthValues": false,
                "practiceType": "0",
                "practiseButton": true,
                "practiseL5CrackedSkills": false,
                "practiseL5NeedsStrengtheningSkills": false,
                "revealHotkey": true,
                "revealHotkeyCode": "Ctrl+Alt+H",
                "revealNewWordTranslation": true,
                "separateGrammarSkillMasteredThreshold": false,
                "separateGrammarSkillsInBreakdown": true,
                "showBonusSkillsInCrackedSkillsList": true,
                "showBonusSkillsInNeedsStrengtheningList": true,
                "showCompletedCheckpointsWithSkillsThatNeedAttention": false,
                "showLeagues": false,
                "showNewWords": true,
                "showOnlyNeededSkills": false,
                "showToggleHidingTextButton": true,
                "showTranslationText": true,
                "sidebarBoxOrder": " ",
                "skillSuggestion": true,
                "skillSuggestionMethod": "0",
                "strengthBarBackgrounds": true,
                "strengthBars": false,
                "suggestionPopoutButton": true,
                "totalStrengthBox": false,
                "treeLevelBorder": true,
                "treeLevelPrediction": true,
                "wordsButton": true
            },
            "progress": {
                "680346058:en->zs": [
                    [
<PHONE_NUMBER>000,
                        0,
                        58
                    ]
                ]
            },
            "treeLevels": {
                "680346058:en->zs": 0
            }
        }
    }
}

Is this version 2.0.26 from Chrome store? That is probably fixed in 2.0.27, but for some reason it's still not up on the store.

Hi,
Just hit submit for review again on the chrome store and it looks like it instantly published. I'm not sure what happened there, maybe I never hit the button before.
Let me know if the new version doesn't sort this and I'll see what I can do.
Toran

@JPili I am going to close this now and 2.0.27 has been published for some time now.
Let me know if there is still an issue.

It’s been fixed for some time now. Thank you for the support!

On Aug 26, 2022, at 8:29 PM, Toran Sharma @.***> wrote:

@JPili I am going to close this now and 2.0.27 has been published for some time now.
Let me know if there is still an issue.
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you were mentioned.


Develop About Course Page

Describe the solution you'd like
Develop an about course page.
Additional context
The page will be linked to this option present currently in each course's more options.

@Abhishek-kumar09 @vasudev19126 Can I work on this?

@sakshi-choudhary yes

@sakshi-choudhary I think this is already desined, https://edu-client.netlify.app/course/2
Are you thinking something else?

@sakshi-choudhary I think this is already created, https://edu-client.netlify.app/course/2
Are you thinking something else?

Okay, yeah but this for new users right? as there's an option to buy as well. So, do we need an about course page for enrolled students? as I saw this about course option the more options tab.

Hey, @sakshi-choudhary  I don't think it will be a good idea for making a difference about course page route for this section rather what we can do is we can make that list to expanded and collapse list and show a summary of about course. This will also help users to see course details while attending the lectures.
Please let me know your thoughts on this :)

Hey, @sakshi-choudhary I don't think it will be a good idea for making a difference about course page route for this section rather what we can do is we can make that list to expanded and collapse list and show a summary of about course. This will also help users to see course details while attending the lectures.
Please let me know your thoughts on this :)

Yes, a collapse will be fine.

@sakshi-choudhary this issue is resolved by your PR (#309), I think you can close it now : )


msgraph-meta #49: mandatory queryParams on /calendarView

… /calendarView

@ivy-rew Thanks for the contribution! It's really appreciated.
@MIchaelMainer @zengin isn't this file supposed to be auto-generated? Most likely pulled from https://graphexplorerapi.azurewebsites.net/openapi?graphversion=v1.0&operationids=*&openapiversion=3 ?? It looks like it hasn't been updated in a while though.

@ivy-rew Thanks for the contribution! It's really appreciated.
@MIchaelMainer @zengin isn't this file supposed to be auto-generated? Most likely pulled from https://graphexplorerapi.azurewebsites.net/openapi?graphversion=v1.0&operationids=*&openapiversion=3 ?? It looks like it hasn't been updated in a while though.

@baywet Yes, I think it is generated with https://github.com/microsoftgraph/msgraph-metadata/tree/master/tools/odata2openapi. @peombwa, how do you handle scenarios like this for PowerShell? Would it be possible to add this as an OpenAPI extension?

@baywet Yes, I think it is generated with https://github.com/microsoftgraph/msgraph-metadata/tree/master/tools/odata2openapi. @peombwa, how do you handle scenarios like this for PowerShell? Would it be possible to add this as an OpenAPI extension?

I see that it doesn't make much sense to contribute change in a generated file.
However, if there is a way to contribute these undocumented query params I'd gladly do so.
Meanwhile I recon, that the /user/{calendar-id}/calendar/calendarView APIs are better documented. (only date date-format schema sub-type is not set there).
Even so, I use the /user resources instead of /me as workaround. Nevertheless, still it would be best to document these start-/endDateTime in a consistent way only once: and re-use them on all APIs that offer these capabilities.

I see that it doesn't make much sense to contribute change in a generated file.
However, if there is a way to contribute these undocumented query params I'd gladly do so.
Meanwhile I recon, that the /user/{calendar-id}/calendar/calendarView APIs are better documented. (only date date-format schema sub-type is not set there).
Even so, I use the /user resources instead of /me as workaround. Nevertheless, still it would be best to document these start-/endDateTime in a consistent way only once: and re-use them on all APIs that offer these capabilities.

@ivy-rew This OpenAPI description is updated by the tool that Michael linked.
However I'm not sure why we have a fully built tool here and not a pointed to the sources instead. (having a built tool requires manual updates, makes the repo heavier...)
@MIchaelMainer do you happen to know where are the sources for that tool?

@ivy-rew This OpenAPI description is updated by the tool that Michael linked.
However I'm not sure why we have a fully built tool here and not a pointed to the sources instead. (having a built tool requires manual updates, makes the repo heavier...)
@MIchaelMainer do you happen to know where are the sources for that tool?

@baywet I'm sorry,  I don't where the source for that tool is maintained.

@baywet I'm sorry,  I don't where the source for that tool is maintained.

@MIchaelMainer @baywet We do use CustomQueryOptions annotation to add query params to paths. This is all done at the XSLT transform layer like this https://github.com/microsoftgraph/msgraph-metadata/pull/18.
StartDateTime/endDateTime query param is currently applied to /calendarView paths of users, groups, and bookingBusinesses. This can be accessed via https://graphexplorerapi.azurewebsites.net/openapi?graphversion=v1.0&operationids=*&openapiversion=3. We may need to add StartDateTime/endDateTime query params to /me as well. This wasn't implemented since we don't support /me in PowerShell.

@MIchaelMainer @baywet We do use CustomQueryOptions annotation to add query params to paths. This is all done at the XSLT transform layer like this https://github.com/microsoftgraph/msgraph-metadata/pull/18.
StartDateTime/endDateTime query param is currently applied to /calendarView paths of users, groups, and bookingBusinesses. This can be accessed via https://graphexplorerapi.azurewebsites.net/openapi?graphversion=v1.0&operationids=*&openapiversion=3. We may need to add StartDateTime/endDateTime query params to /me as well. This wasn't implemented since we don't support /me in PowerShell.

Thanks for jumping in @peombwa . So, is this open API description used at all then?

Thanks for jumping in @peombwa . So, is this open API description used at all then?

@baywet We do use it generate *calendarView cmdlets in PowerShell that have StartDateTime/EndDateTime as parameters https://github.com/microsoftgraph/msgraph-sdk-powershell/blob/dev/src/Calendar/Calendar/docs/v1.0/Get-MgUserCalendarView.md#-enddatetime.

@baywet We do use it generate *calendarView cmdlets in PowerShell that have StartDateTime/EndDateTime as parameters https://github.com/microsoftgraph/msgraph-sdk-powershell/blob/dev/src/Calendar/Calendar/docs/v1.0/Get-MgUserCalendarView.md#-enddatetime.

Hi @ivy-rew
Thanks for your patience. I just had a call with @peombwa to understand the context better and list out the next steps.
This file is outdated and not used anymore, for your generation, you should use the API we pointed at in the last comments.
Now, that doesn't mean we don't want your PR, to the contrary! Can you make the following changes please:

[ ] delete the openapi/v1.0/openapi.yaml
[ ] delete the openapi/beta/openapi.yaml
[ ] delete the directory tools/odata2openapi
[ ] update the XLST so it also applies the transformation for the me Singleton, add a new matching rule

Here is some documentation on how to run the transform process. And don't hesitate to reach out if you have any question!
Thanks again a lot for your contributions!

Hi @ivy-rew
Thanks for your patience. I just had a call with @peombwa to understand the context better and list out the next steps.
This file is outdated and not used anymore, for your generation, you should use the API we pointed at in the last comments.
Now, that doesn't mean we don't want your PR, to the contrary! Can you make the following changes please:

[ ] delete the openapi/v1.0/openapi.yaml
[ ] delete the openapi/beta/openapi.yaml
[ ] delete the directory tools/odata2openapi
[ ] update the XLST so it also applies the transformation for the me Singleton, add a new matching rule

Here is some documentation on how to run the transform process. And don't hesitate to reach out if you have any question!
Thanks again a lot for your contributions!

@baywet Thanks a lot for gathering all this information to supply a new fix.
I'm am amazed how you treat requests from the community. Actually, it inspires me to re-think my own attitude when reviewing code and guiding new members.
Even so, I'm running quit of time to investigate in the feature behind this change request. In addition, though I do have some experience with XSLT, it's not an area where I feel comfortable :-) . So, I can't promise to actually file a new PR. But let it open till Saturday and let's see whether I can contribute something till then.

@baywet Thanks a lot for gathering all this information to supply a new fix.
I'm am amazed how you treat requests from the community. Actually, it inspires me to re-think my own attitude when reviewing code and guiding new members.
Even so, I'm running quit of time to investigate in the feature behind this change request. In addition, though I do have some experience with XSLT, it's not an area where I feel comfortable :-) . So, I can't promise to actually file a new PR. But let it open till Saturday and let's see whether I can contribute something till then.

thanks for the kind words! Give it a try and if it doesn't work (XSLT can be far from user friendly when not "used" to it), just apply the deletions, I'll push the XSLT change to your branch.

thanks for the kind words! Give it a try and if it doesn't work (XSLT can be far from user friendly when not "used" to it), just apply the deletions, I'll push the XSLT change to your branch.

That sounds more than fair @baywet
I've applied the suggested deletions. Feel free to take over :+1:

That sounds more than fair @baywet
I've applied the suggested deletions. Feel free to take over :+1:

If we regenerate the OpenAPI, those parameters should be there.  Give me a few mins.

If we regenerate the OpenAPI, those parameters should be there.  Give me a few mins.

Alright everyone, after chatting with @darrelmiller it appears the open API descriptions are still needed. Sorry about the confusion.
I authored #52 reusing some of your commits, the reasons why I had to put together another PR are:

I don't have the permissions to push downstream
I needed to test the new github actions workflow I added, and github token are not available for PRs from forks

This new PR adds a workflow that will keep the open API descriptions up to date, so you can keep using the files here instead of the graph explorer API. It also adds the missing XSLT pattern for the me calendar view.
Once the PR is merged, the CSDL will be updated (probably next week) and deployed to the GE API, which in turns should trigger an update of the open api descriptions here.
Closing this PR

Alright everyone, after chatting with @darrelmiller it appears the open API descriptions are still needed. Sorry about the confusion.
I authored #52 reusing some of your commits, the reasons why I had to put together another PR are:

I don't have the permissions to push downstream
I needed to test the new github actions workflow I added, and github token are not available for PRs from forks

This new PR adds a workflow that will keep the open API descriptions up to date, so you can keep using the files here instead of the graph explorer API. It also adds the missing XSLT pattern for the me calendar view.
Once the PR is merged, the CSDL will be updated (probably next week) and deployed to the GE API, which in turns should trigger an update of the open api descriptions here.
Closing this PR


codeunit 31356 "Payment Order Management CZB" OnBeforeSelectBankAccount

Please Add new Event to local procedure SelectBankAccount(var BankAccount: Record "Bank Account"): Boolean
local procedure SelectBankAccount(var BankAccount: Record "Bank Account"): Boolean
begin
    // New Event
    OnBeforeSelectBankAccount(BankAccount);
    // New Event
    case BankAccount.Count() of
        0:
            exit(false);
        1:
            begin
                BankAccount.FindFirst();
                exit(true);
            end;
        else
            exit(Page.RunModal(Page::"Bank Account List", BankAccount) = Action::LookupOK);
    end;
end;

[IntegrationEvent(false, false)]
local procedure OnBeforeSelectBankAccount(var BankAccount: Record "Bank Account")
begin
end;

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. Please do not reply to this, as we do not monitor closed issues. If you have follow-up questions or requests, please create a new issue where you reference this one.
Build ID: 10527.


AVIF support!

There is a request for WebP support here: https://github.com/MathewSachin/Captura/issues/519
Both WebP and AVIF are more advanced image formats than the others. AVIF in particular is ahead of everything.
Here is an example of the work ongoing to integrate AVIF https://www.reddit.com/r/AV1/search/?q=avif&include_over_18=on&sort=new
Here is recent AVIF research by netflix https://netflixtechblog.com/avif-for-next-generation-image-coding-b1d75675fe4
JPEG XL is also excellent! https://www.reddit.com/r/AV1/comments/fjddcj/lossless_image_formats_comparison_webp_jpeg_xl/

Quick caveat:
While I am very in favor of AVIF for a long list of reasons including quality vs. size, color improvements over JPEG, patent issues, and support by major companies, etc., Captura does not need to be on the cutting edge of this format until it sees a little wider adoption.
I could have just as easily written up this feature request, and I'm happy to see @ibex-are-goats highlight this, I think Apple made a mistake with pushing all it's new image format (HEIC) until more tools became available to read/edit them.


[QATM-960]update BDT to 0.6.0

Description
Testing

[ ] Unit, integration tests

Documentation

[ ] Changelog update
[ ] Documentation link

LGTM

we close the pull request and try with crossdata-spark2


How to program multiple vehicles in C++?

I saw the examples for single vehicle in the guide and I wonder how to modify them to work with multiple vehicles. I understand how "takeoff an land" example works, but I'm just a beginner and I'm not sure how to make it works with multiple vehicles

Have you seen this example?
https://github.com/mavlink/MAVSDK/blob/develop/example/multiple_drones/multiple_drones.cpp

Duh, that wasn't in the guide -_-
Anyway, thanks for this example and sorry for making problems :)


Pass stringified record to cache and storage connector

If we also pass the stringified record to the connectors then we can avoid the overhead of doing it twice. Since the record-handler already has a stringified version.

This would e.g. allow for sending records to redis without running them trough a JSON.stringify.


Set all frameworks to expired

Applying only the database migrations to an empty database creates several old frameworks in various states. This causes problems when the frontend apps try to load content for frameworks we don't have content for (eg. G-Cloud 5).
Set all existing frameworks  to expired before we start adding new data.
https://trello.com/c/ZAbcTEKR/106-3-create-script-to-generate-test-data-without-dependency-on-any-private-asset

Tested and works.