Use oci-seccomp-gen library for adding seccomp configurations

I added functionality to specify syscalls and their arguments for all actions. Also changed default action and archs flags to ociseccompgen library.
Now we can do things like:
ocitools generate --seccomp-errno clone,write --seccomp-trace getcwd --seccomp-trap umount:0:1:2:NE
The behavior mimics that of: https://github.com/GrantSeltzer/Manhattan
Also updated man page to reflect this change.

@mrunalp Not sure how to fix this error.

@GrantSeltzer Your commit is missing a Signed-off-by.

ae08020d7 has recursive Godeps :p.  I'm not sure if that's an expected
thing, but 103 vendored files and 10k lines is a lot to pull in for
this change.  Maybe we can get a smaller library with the bits we need
without Manhattan → runc → docker → libtrust and similar dependency
chains?

@wking Not  sure what's causing that godeps recursive chain. Do you know why that's happening?

On Fri, Jul 01, 2016 at 04:52:03PM -0700, Grant Seltzer Richman wrote:

@wking Not sure what's causing that godeps recursive chain. Do you
know why that's happening?

I'm not familiar with Godeps, but if it is just based on Git clones
maybe it always pulls in the whole repository (and not just a
particular referenced package)?  If so, the fix is probably “get
whatever portion of $DEPENDENCY you need pulled out into an
independent Git repository”, ideally by getting upstream to split
things up for you.  But maybe the nesting is happening for a different
reason.

I'm not familiar with Godeps, but if it is just based on Git clones maybe it always pulls in the whole repository (and not just a particular referenced package)? If so, the fix is probably “get whatever portion of $DEPENDENCY you need pulled out into an independent Git repository”, ideally by getting upstream to split things up for you. But maybe the nesting is happening for a different reason.

Yea, the issue is that I have the library ociseccompgen and the tool manhattan in the same git repo. I didn't expect  go get to pull the whole repo instead of just the package.
For the sake of this this PR at least Godeps.json should be updated to include the ociseccompgen library, have the actual library stored and remove the embedded Godeps directory which is holding dependencies that ocitools already has (cli, logrus, runtime-sepc)
I can do that along with the fixing spacing and re-adding cgroup flag that I accidentally removed. Thoughts?

On Fri, Jul 01, 2016 at 05:12:30PM -0700, Grant Seltzer Richman wrote:

I can do that along with the fixing spacing and re-adding cgroup
flag that I accidentally removed. Thoughts?

I don't know enough about Godeps to know what the right fix is ;).
But fixing these other two nits would be good.

@GrantSeltzer I follow instruction to update your dependency, it works, no extra 'Godeps' added. I'm not sure if it also helps in your environment.

Hi @GrantSeltzer ,  ocitools generate is a small and basic tool, oci-seccomp-gen looks more like a part of it. So personally I don't like ocitools generete to depend on oci-seccomp-gen. But I think it is good to have oci-seccomp-gen as an independent pkg since it could be used in different scenarios (now we have ocitools and docker). Would you mind/like to 'donate' your project to ocitools?
@mrunalp @wking  @Mashimiao what you do think about this? The code struct could look like this:
ocitools/pkg/seccomp
ocitools/contrib/seccomp-generete (originial from https://github.com/GrantSeltzer/Manhattan/blob/master/main.go)

@rhatdan @runcom @mheon what do you think of that idea? Manhattan can import its functionality from ocitools instead of inside it's own repo. I'm personally fine with it.

I'm +1k to include that library directly in ocitools and have Manhattan or ocitools itself to manage and generate seccomp profiles or OCI configs

On Mon, Jul 04, 2016 at 02:22:52AM -0700, 梁辰晔 (Liang Chenye) wrote:

ocitools/pkg/seccomp

Is ‘pkg’ more idiomatic than ‘lib’?  I don't read enough Go to know,
but for a C project I'd expect ‘lib’.

ocitools/contrib/seccomp-generete (originial from
https://github.com/GrantSeltzer/Manhattan/blob/master/main.go)

Why keep this executable separate?  Isn't this PR just rolling it into
the bigger ‘ocitool generate …’ picture?

No pkg nor lib, just the name of the library is go idiomatic

+1 also on having the main tool work with the library directly instead of creating a contrib dir with binaries in there. So just:
./seccomp
And code from that used in the main ocitools

I like the idea of having the ociseccompgen be part of the ocitools library, that makes a lot of sense.
Just to clarify, are we in disagreement that seccomp should be a flag for the ./ocitools generate command? That way it's consistent with other configuration options that generate supports (i.e. --cgroups-path or --hostname)
Manhattan is fully functioning for generated just the seccomp configurations: https://github.com/GrantSeltzer/Manhattan

Could add a flag --seccomp-only to ocitools generate
Sent from my iPhone

On Jul 4, 2016, at 4:07 PM, Grant Seltzer Richman<EMAIL_ADDRESS>wrote:
I like the idea of having the ociseccompgen be part of the ocitools library, that makes a lot of sense.
Just to clarify, are we in disagreement that seccomp should be a flag for the ./ocitools generate command? That way it's consistent with other configuration options that generate supports (i.e. --cgroups-path or --hostname)
Manhattan is fully functioning for generated just the seccomp configurations: https://github.com/GrantSeltzer/Manhattan
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.

ocitools/contrib/seccomp-generete (originial from https://github.com/GrantSeltzer/Manhattan/blob/master/main.go)
I'm +1 for this.

OK, +1 for  a flag --seccomp-only to ocitools generate.

Alright, I'm working on the following points:

Rehost the ociseccompgen library into the ocitools repo

Add all of the flags like seccomp-allow that I added in this PR

Add --seccomp-only flag to just export a seccomp file

Should this be it's own PR (this would be closed) or squashed with the commit for this one?

On Tue, Jul 05, 2016 at 12:23:18PM -0700, Grant Seltzer Richman wrote:

Add --seccomp-only flag to just export a seccomp file

What is the use-case for this?  Can we accomplish the same thing by
having ‘ocitools generate’ write (optionally?) to stdout and use:
$ ocitools generate … | jq .linux.seccomp

The use case is for use with other tools like docker that you can pass a seccomp configuration file to a container or (soon) the daemon. And yes piping into jq would do the same thing, But I think this would be more user friendly and feel less like a hack.

On Tue, Jul 05, 2016 at 12:39:19PM -0700, Grant Seltzer Richman wrote:

The use case is for use with other tools like docker that you can
pass a seccomp configuration file to a container or (soon) the
daemon. And yes piping into jq would do the same thing, But I think
this would be more user friendly and feel less like a hack.

I'm just not sure how generically we should scope the solution.  I'd
rather not end up with an API that has lots of --*-only flags.  If we
don't want to use jq, we could also ‘--subsection linux.seccomp’ or
some such to handle the drilling internally.
And I think using jq for JSON dicing is using the proper (already
written) tool for the task.  It doesn't feel like a hack ;).
And regardless of how we address this, always writing to config.json
doesn't seem appropriate for the seccomp-only use case.  What
command-line API do we want for “please write the generated config to
stdout”?  I'd rather do that by default, with an ‘--output PATH’
option to write to a file (for situations where shell redirection is
not available).

I think having a flag for generate to rename the output file should be added, although that's a more general issue than this PR. Certainly the default for seccomp output could be something like "config.seccomp".
I'm not sure what other configurations can be independently used besides seccomp but I think the seccomp-only flag would definitely be used and makes more sense than having an independent tool (Manhattan). I definitely concede that using jq wouldn't be a hack but having this feature would give more visibility to the idea of using ocitools for it. I also think the feature adds minimal change to the code base.

On Tue, Jul 05, 2016 at 01:51:53PM -0700, Grant Seltzer Richman wrote:

I think having a flag for generate to rename the output file
should be added, although that's a more general issue than this PR.

I've filed --output as #129.

I'm not sure what other configurations can be independently used
besides seccomp but I think the seccomp-only flag would definitely
be used and makes more sense than having an independent tool
(Manhattan).

I agree that having an independent, OCI-specific tool like Manhattan
is less ideal than either jq or an ocitools flag.  If the current
consensus is that --seccomp-only is unlikely to be the first step down
a slippery slope, I'm ok with going down that path for now.  We can
always revisit the decision if/when we find ourselves sliding down the
--*-only slope ;).

Alright, if people could re-review this PR that would be great. I made the changes as discussed above:

Rehost the ociseccompgen library into the ocitools repo

Add flags for each seccomp action for specifying syscalls and possibly actions. As well as for setting architectures and default action

Add --seccomp-only flag to just export a seccomp file

Updated README and man page to document these changes

On Tue, Jul 05, 2016 at 07:50:27PM -0700, Grant Seltzer Richman wrote:

Rehost the ociseccompgen library into the ocitools repo

Manhattan was under the GPL 1, but with only you and @runcom
contributing 2, I suspect you're both fine relicensing it under the
Apache v2 3.
With ocitools in the URL, I'd also rather see the package called
‘seccomp’ 4 instead of ‘contrib/oci-seccomp-gen’, but that's not a
big deal either way.

The seccomp-gen should not be in the 'contrib' directory.
If there will be more libs, I prefer to put it in ocitools/pkg/seccom-gen.
@wking I only read few golang projects and feel pkg is used more frequently. (docker/rkt/kubernetes)

Why don't just have seccomp-gen directly without any contrib or pkg. The pkg you're referring to is for huge projects where it makes sense to do that. I still vote to have the library along with the other files without any fancy directly....

On Tue, Jul 05, 2016 at 11:38:43PM -0700, Antonio Murdaca wrote:

Why don't just have seccomp-gen directly without any contrib or
pkg. The pkg you're referring to is for huge projects where it makes
sense to do that. I still vote to have the library along with the
other files without any fancy directly....

And (as I said before) I'm not familiar with the Go idioms here, but
using just ‘seccomp’ (or, I guess, ‘seccomp-gen’) sounds good to me.

@runcom  I'm fine with ocitools/seccomp-gen.  I add a pkg just in case If there will be more libs, so now it is not necessary.

Great! :)

Wrt the license, I'm not entirely sure, @rhatdan?

Is there a license issue? Can't the library be issued under a different license?

Nope, just wanted Dan to confirm we don't have issue here

I have no problem re-licensing to Apache.

Fixing conflicts now, I have no problem with it either

Alright so now all of the seccomp actions have flags, and there's a flag for exporting only the seccomp portion. The seccomp-gen library has been moved to within the oci-tools repo. How's it look?

9fd82e5 → 9d1203e restored the old parseArgs in generate.go.  This is probably an accidental side-effect of the rebase.
I sent that^ comment in via email ~40 minutes ago (so maybe GitHub ate it), in case the email eventually gets through and folks wonder why I double-posted it ;).

@wking Woops! Fixing now. I'm not very good with git yet.

@mrunalp Could you take a look at this today?

This PR can now be reworked on top of #131

working on it now @mrunalp

@mrunalp @hmeng-19
I want to double check that it's ok to remove the seccomp-syscalls flag. We would only need the seccomp-ACTION flags and ones for architectures, and default actions. Right now there's only ones for allow and errno anyway.
Sytanx should like this:
ocitools --seccomp-kill clone:0:1:2:NE,getcwd --seccomp-default allow --seccomp-arch mips,arm,arm64

@GrantSeltzer SGTM

@mrunalp Please take a look

Fixed suggestions, @mrunalp You can take another look :)

@mrunalp @rhatdan @runcom
I'd like to open a fresh branch/PR if it's alright, the commit's for this one have gotten unmanageable because of multiple merge commits in the time it's been sitting. Besides the fail for the non-dco, everything seems good to me. Let me know whenever you're ready to look at this, Mrunal.

On Tue, Jul 26, 2016 at 12:33:19PM -0700, Grant Seltzer Richman wrote:

I'd like to open a fresh branch/PR if it's alright, the commit's for
this one have gotten unmanageable because of multiple merge commits
in the time it's been sitting.

No need for a new PR.  Once you have a squashed commit, just
force-push it to your seccomp-cli-features branch.

Ah, and the base for the new PR should be somewhere before #140 landed
in f0ecc98 so we can merge this into the v1.0.0.rc1 branch too.  If
you do open a new PR, my personal preference would be for you to open
it on top of and targetting v1.0.0.rc1 [1,2].

Ah, and the base for the new commit/PR should be somewhere before #140 landed
in f0ecc98 so we can merge this into the v1.0.0.rc1 branch too.  If
you do open a new PR, my personal preference would be for you to open
it on top of and targetting v1.0.0.rc1 [1,2].

I believe the last commit is rebased on top of the current ocitools master branch

@runcom just helped me out, ignore my last comment.
@mrunalp can you please review :)

On Tue, Jul 26, 2016 at 01:01:39PM -0700, Antonio Murdaca wrote:

Ah, and the base for the new commit/PR should be somewhere before
#140 landed in f0ecc98 so we can merge this into the v1.0.0.rc1
branch too.  If you do open a new PR, my personal preference would
be for you to open it on top of and targetting v1.0.0.rc1 [1,2].

I believe the last commit is rebased on top of the current ocitools
master branch

I've rebased it backwards with:
$ git rebase --onto 564b30e HEAD^
After resolving some conflicts in code you'd deleted (but which had
been updated in master), I ended up with 69526b9, which I've pushed to
1.  You can compare with your current commit using:
$ diff -u <(git show --color=always cbd3a2e4) <(git show --color=always 69526b9)
If that looks right to you, I'd like to see 69526b9 pushed to this PR
so it can be merged into both master and v1.0.0.rc1.  The alternative
is for me to PR 69526b9 into v1.0.0.rc1, but then we get to unify
those slightly-different versions of the commit at some later point
when we merge v1.0.0.rc1 into master to pick up other
v1.0.0.rc1-compatible changes.

I've rebased it backwards with:
$ git rebase --onto 564b30e HEAD^
After resolving some conflicts in code you'd deleted (but which had
been updated in master), I ended up with 69526b9, which I've pushed to
[1].  You can compare with your current commit using:
$ diff -u <(git show --color=always cbd3a2e) <(git show --color=always 69526b9)
If that looks right to you, I'd like to see 69526b9 pushed to this PR
so it can be merged into both master and v1.0.0.rc1.  The alternative
is for me to PR 69526b9 into v1.0.0.rc1, but then we get to unify
those slightly-different versions of the commit at some later point
when we merge v1.0.0.rc1 into master to pick up other
v1.0.0.rc1-compatible changes.

sorry, can't be just wait to land this in master and cherry-pick for the release? or create another pr for the release branch? it's so confusing for everyone like this.

@GrantSeltzer will of course take care of opening a PR against the release branch once this is finished and afaict this isn't yet fully reviewed and lgtm'ed

On Tue, Jul 26, 2016 at 01:13:53PM -0700, Antonio Murdaca wrote:

I've rebased it backwards with:
$ git rebase --onto 564b30e HEAD^
After resolving some conflicts in code you'd deleted (but which had
been updated in master), I ended up with 69526b9, which I've pushed to
1.  You can compare with your current commit using:
$ diff -u <(git show --color=always cbd3a2e) <(git show --color=always 69526b9)
If that looks right to you, I'd like to see 69526b9 pushed to this PR
so it can be merged into both master and v1.0.0.rc1.  The alternative
is for me to PR 69526b9 into v1.0.0.rc1, but then we get to unify
those slightly-different versions of the commit at some later point
when we merge v1.0.0.rc1 into master to pick up other
v1.0.0.rc1-compatible changes.

sorry, can't be just wait to land this in master and cherry-pick for
the release? or create another pr for the release branch?

That's the “The alternative is …” route above.  I'd rather not
cherry-pick, because it makes the later merge of v1.0.0.rc1 into
master more confusing.

it's so confusing for everyone like this.

This is how lots of projects work.  For example see Git's section on
picking which branch to base your work on 1.
At some point ocitools is (I think) going to have to support multiple
runtime-spec releases at the same time, so maybe we want to work out
how we see that working now, implement it, and drop the v1.0.0.rc1
branch?  I have some thoughts on that in #98 for config validation,
but the thoughts there aren't all that useful for generate or runtime
validation.

On Tue, Jul 26, 2016 at 01:14:36PM -0700, Antonio Murdaca wrote:

@GrantSeltzer will of course take care of opening a PR against the
release branch once this is finished and afaict this isn't yet fully
reviewed and lgtm'ed

Sounds good, and it is good to wait until the branch is reviewed and
merged into one branch before filing other PRs.  If (for
v1.0.0.rc1-compatible features) we lead with a v1.0.0.rc1-based PR
1, there's no need for a second PR.  The maintainer just merges
v1.0.0.rc1 into master after each v1.0.0.rc1 bump.

I think the API should be easy to use just like rest of generate.
For e.g.
g := generate.New()
g.SetSeccompDefault(...)
g.AddSeccompErrno(...)
g.AddSeccompErrno(...)
g.SaveToFile(...)

The command line option parsing should be left to integration with the cmd.

@mrunalp How's that? I created a seccomp generator in the same way the generate library has.

@GrantSeltzer No, this doesn't help. User shouldn't have to deal with two different generators when trying to generate a config. You could create a seccomp generator API and then use it from the regular generate API. The generate API should be simple and comprehensive.
For e.g.
gen := generate.New()
gen.SetSeccompDefault(..) // These calls could use your own seccomp API behind the scenes.
gen.AddSeccompErrno(..)
gen.AddSeccompKill(..)
gen.SetRootfs(..)
gen.AddCapability(..)
gen.SaveToFile(..)

Just to clarify do you want all of the command line parsing done in the main package?

@GrantSeltzer Yes, it shouldn't be in the API.

I think the check is failing because of vendoring, looking into it now, it's working locally

Can you open a new PR with your commit cherry picked to see if we can make the travis issue go away?

On Wed, Jul 27, 2016 at 04:43:03PM -0700, Mrunal Patel wrote:

Can you open a new PR with your commit cherry picked to see if we
can make the travis issue go away?

New PR should not matter and this PR has already had its history
compacted.  Current failures are pointer issues 1, so we're probably
missing an update around #150.

@wking It builds fine locally.

@wking If you actually check the code you can see that it has been fixed but travis is not finding the correct commit.

On Wed, Jul 27, 2016 at 04:53:37PM -0700, Grant Seltzer Richman wrote:

@wking If you actually check the code you can see that it has been
fixed but travis is not finding the correct commit.

Ah, in that case pushing an amended commit (to bump the commit
timestamp and get a new hash) may work.  Although sometimes we've had
issues with Travis just being slow ;).  Opening a new PR is fine too
if we don't want to wait for Travis to catch up.

Actually, here's the merge commit Travis is testing not using a
pointer receiver 1.  So I don't think any “kick Travis” fix will
work.

And here's your current tip not using a pointer receiver:
$ git grep addSeccomp d297bc1
d297bc1:cmd/ocitools/generate.go:       g, err := addSeccomp(g, context)
d297bc1:cmd/ocitools/generate.go:func addSeccomp(g generate.Generator, context *cli.Context) (generate.Generator, error) {

#159

On Wed, Jul 27, 2016 at 05:56:21PM -0700, Grant Seltzer Richman wrote:

#159

So we should close this one?

Yup @mrunalp

On Thu, Jul 28, 2016 at 07:48:10PM -0700, Grant Seltzer Richman wrote:

Yup @mrunalp

@GrantSeltzer, as the PR author, you should be able to close it too.


fix: ndash for non-unicode support

Alaska Airlines Pull Request
On AScom, the unicode ndashes baked into the component were appearing like so:

This uses the – escaped equivalent, which does work.
Fixes: # (issue, if applicable)
Type of change:
Please delete options that are not relevant.

[x] Revision of an existing capability

Checklist:

[x] My update follows the CONTRIBUTING guidelines of this project
[x] I have performed a self-review of my own update

By submitting this Pull Request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
Pull Requests will be evaluated by their quality of update and whether it is consistent with the goals and values of this project. Any submission is to be considered a conversation between the submitter and the maintainers of this project and may require changes to your submission.
Thank you for your submission!
-- Auro Design System Team

Shouts to @geoffrich for the quick solution

:tada: This PR is included in version 1.1.1 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


icon and caption.

Sorry, I want to have been at the bottom of the display icon and caption.

Now it's possible to show all the titles even if there's more than three tabs in version 1.3.2.
// Show all titles all the time.
mBottomBar.useFixedMode();


feat: support pnpm.patchedDependencies

Automatically applies pnpm.patchedDependencies. Any patches in patches are applied after.

I'm planning to add an e2e so we can exercise it under bzlmod.

This looks great. Just one question of what happens with a patch is listed in pnpm. patchedDependencies but the user forgets to add it to data? Is it a hard fail or a warning that it is missing from data?

This looks great. Just one question of what happens with a patch is listed in pnpm. patchedDependencies but the user forgets to add it to data? Is it a hard fail or a warning that it is missing from data?

It not operates the same as the other data files: you'll get a warning but it will implicitly copy the patches over.

Updated and ready for another pass.


remove how it works guide [PEOPLE-116]

JIRA: https://netguru.atlassian.net/browse/PEOPLE-116
Changes:

remove HowItWorks component

👍


soc: nrf5340: Fix busy waiting on nRF5340 application core

This patch adds a temporary workaround for the incorrect initialization
of the SystemCoreClock global variable that is done for the application
core of nRF5340 (see system_nrf5340_application.c) and that results in
k_busy_wait() producing delays of twice the requested time.
The problem is that the call to SystemCoreClockUpdate() that is done
at the end of SystemInit() correctly sets the value of SystemCoreClock
to reflect the hardware state after reset (HFCLK128M divided by 2),
but then the SystemCoreClock variable is initialized (by z_data_copy()
called from z_arm_prep_c()) to the __SYSTEM_CLOCK value that is defined
as 128000000. This in turn results in nrfx_coredep_delay_us() (used by
k_busy_wait() by default for nRF SoCs) delaying for twice the requested
number of microseconds.
The temporary workaround is to call SystemCoreClockUpdate() at a later
stage of the system initialization, in its nRF53 specific part.
Fixes #21082.

Shouldn't the workaround limit itself to the APPLICATION core of nrf5340?

Right. For the network core, it would be a waste of resources, although not harmful.
Fixed.


[Bug] [Branch 1.x] IDMAP Range Search IP Failed Occationaly

Testcase: IDMAPParameters/IDMAPTest.idmap_range_search_ip/0
UT action run link: https://github.com/milvus-io/knowhere/actions/runs/4487260181/jobs/7890509189

/assign @cydrain


HHH-15895 IllegalArgumentException :Cannot create binding for parameter referencen with criteria builder

https://hibernate.atlassian.net/browse/HHH-15895
different instances of SqmParameter instances can have the same hashcode value, so when the number of SqmJpaCriteriaParameterWrapper is huge the probability of considering 2 instances equals is high if
return Integer.compare( o1.hashCode(), o2.hashCode() ); is used to compare them.

merged upstream


Page Keeps Reloading in Mobile Safari / Mobile Edge and never fully loads

Page Keeps Reloading in Mobile Safari / Mobile Edge and never fully loads

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

ID: 0767aac4-fcd9-eaac-2c30-8f17d8eadd92
Version Independent ID: a380a828-03f9-e037-3a62-e4165fda7cb6
Content: Azure AD built-in roles - Azure Active Directory
Content Source: articles/active-directory/roles/permissions-reference.md
Service: active-directory
Sub-service: roles
GitHub Login: @rolyon
Microsoft Alias: rolyon

@scottstout
Thanks for your feedback! Can you share some more info so i can gain a better understanding of your issue.

What page keeps reloading?
Is the Azure AD built-in roles documentation reloading when you're using Safari on your phone and never fully loading on Edge?

Any additionally information would be greatly appreciated.
If you have any other questions, please let me know.
Thank you for your time and patience throughout this issue.

Steps to reproduce:

Get on an iPhone (latest OS)
Go to google and search for Azure Built In Roles
Click the first link that comes up.
Page keeps refreshing before the load completes and repeats over and over.
Paste the same link into edge mobile to see if it helps.
Notice that it didn't help.

Eventually the happens.

@scottstout
Thank you for the details!
I just tried to reproduce your issue on my phone and wasn't able to.

Have you tried disconnecting from Wifi and using LTE?
Or making sure that your internet connection isn't the issue?
Are you having any problems from a web browser on your computer?

Repeated on LTE (and in a private browsing session)

This works fine from my computer.

@scottstout
Thank you for the quick follow ups and details!
I reached out to my team regarding this and unfortunately none of them were able to reproduce this. However, since it's been about a day, I just wanted to check in and see if you're still running into this issue?

I reproduced it just now. If you don't think this is widespread we can just
drop it. I don't need to get to this page on my phone.
On Wed, Jun 30, 2021 at 4:32 PM James Tran-MSFT @.***>
wrote:

@scottstout https://github.com/scottstout
Thank you for the quick follow ups and details!
I reached out to my team regarding this and unfortunately none of them
were able to reproduce this. However, since it's been about a day, I just
wanted to check in and see if you're still running into this issue?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/MicrosoftDocs/azure-docs/issues/77622#issuecomment-871741676,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AABAE3XXS4BQPFESFZLNOPTTVOENHANCNFSM47QW56UQ
.

@scottstout
Thank you for the quick follow up on this!
I reached out to the author of this article, who tried to repro this as well and wasn't able to. However, since you're internal I'll go ahead and close this out, and the doc author will follow up with you offline.
If you have any other questions or if this issue gets worse, please let us know.
Thank you for your time and patience throughout this issue.


added syntax check

based on https://www.npmjs.com/package/gulp-jsvalidate

Added in 2.0.4. Thanks!

How do I update to this newest version? tried npm update generator-mendix but it has still the old package.json and gulp.js file

First you need to update your generator:
npm update generator-mendix -g

after that, run yo mendix in the folder with the Gulpfile.js and package.json. You can always check the version of the generator you used by checking package.json in your widgetfolder.  This version number is also mentioned at the to of your Gulpfile.js. Look for
"generatorVersion": "2.0.4",

thanks, was not aware of running yo mendix again.


Webagent

First time I ran the code
Enter a profileId (or press Enter to skip):
No profileId provided
Creating sessions
Created airtop session 622ea502-ba6e-427a-8bf5-845571c996bf
Determining whether the user is logged in...
Response if user is logged in
Log into your Glassdoor account. Press Enter once done. Live view URL: https://live.airtop.ai/622ea502-ba6e-427a-8bf5-845571c996bf?disableResize=false&includeNavigationBar=false&token=eyJhbGciOiJSUzI1NiIsInR5cCI6IkpXVCIsImtpZCI6IllJOTFWRFlhOW00UzB6NXZ5SXYxZnRXd2VJcyJ9.eyJhdWQiOiJhaXJ0b3AvbGl2ZS12aWV3IiwiZXhwIjoxNzMzODQ3NjU1LCJpYXQiOjE3MzM4MzMyNTUsImlzcyI6Imh0dHBzOi8vYWlydG9wLmFpIiwianRpIjoiYTM1ZDJlMjItOWI4MS00NjE0LTliZTMtZDk2ZTk3MTRlMzUwIiwiYXV0aF92ZXJzaW9uIjoibGl2ZS12aWV3LjEiLCJjb25maWd1cmVkX3R0bF9zZWNzIjoxNDQwMCwib3JnX2lkIjoiNDllNmEyNTMtYjM0Ny00OTc3LWI1OTAtODMyYWE2ZWUyZDc0IiwicHJvdmlkZXIiOiJjYjIvYXBpLXNlcnZlciIsInNlc3Npb25faWQiOiI2MjJlYTUwMi1iYTZlLTQyN2EtOGJmNS04NDU1NzFjOTk2YmYifQ.ONyhN9pf2LiBwA37dvVb72RF4LOWNGDekCTU7OEYC5OEc5iE97axh_Ucx2E4y18iIhwLO-H3cOnluwmtowWNap2mRlrFJwAc_efA2AEvTFZo01EBDlSyjtoOICje4Yg4nO7dYUY0LYzUgGEx68j99zm2LWaP2YbYd6P9cGvXXQuUP8NYkkpinS5xQJaX5gaK4gdl6xnM8aOP4FAzTFAuZgTU7qg4eaXQqZifBVcJ7pOENdLXnXikvJU_eyxEVkROT4Ki9nGIQRtoAe2en-NOG8ypeyFajgAdOBZh5fNBoLOlTvWtg7f3Z-r6kQ4-jEfFRvub5WJOAwXdkFElyloX9w&windowId=88690842-26e8-49d7-ad83-9ae21e982efe
To avoid logging in again, use the profileId next time: 4bce403b-1c69-4aa0-ae13-be06af96a6fd
Navigating to target URL...
Prompting the AI agent, waiting for a response...
Existing workers
Response:
{
"companies": [
{
"companyName": "OpenAI",
"jobTitle": "Software Engineer, Agent Infrastructure",
"location": "San Francisco, CA",
"salary": {
"min": 360000,
"max": 440000
}
},
{
"companyName": "OpenAI",
"jobTitle": "Software Engineer, Safety",
"location": "San Francisco, CA",
"salary": {
"min": 200000,
"max": 370000
}
},
{
"companyName": "OpenAI",
"jobTitle": "Software Engineer, Infrastructure",
"location": "San Francisco, CA",
"salary": {
"min": 160000,
"max": 385000
}
},
{
"companyName": "OpenAI",
"jobTitle": "Android Engineer, ChatGPT Search",
"location": "San Francisco, CA",
"salary": {
"min": 245000,
"max": 385000
}
},
{
"companyName": "OpenAI",
"jobTitle": "Software Engineer, Anti Fraud & Abuse",
"location": "San Francisco, CA",
"salary": {
"min": 200000,
"max": 370000
}
},
{
"companyName": "OpenAI",
"jobTitle": "Software Engineer, Security Product and Platform",
"location": "San Francisco, CA",
"salary": {
"min": 245000,
"max": 310000
}
},
{
"companyName": "OpenAI",
"jobTitle": "Software Engineer, Full Stack",
"location": "San Francisco, CA",
"salary": {
"min": 160000,
"max": 385000
}
},
{
"companyName": "OpenAI",
"jobTitle": "TLM, Developer Productivity",
"location": "San Francisco, CA",
"salary": {
"min": 245000,
"max": 385000
}
},
{
"companyName": "Grammarly, Inc.",
"jobTitle": "Software Engineer, Front-End",
"location": "San Francisco, CA",
"salary": {
"min": 160000,
"max": 321000
}
},
{
"companyName": "Grammarly, Inc.",
"jobTitle": "Software Engineer, macOS",
"location": "San Francisco, CA",
"salary": {
"min": 195000,
"max": 231000
}
}
]
}
Session terminated
Second time I ran the code
Enter a profileId (or press Enter to skip): 4bce403b-1c69-4aa0-ae13-be06af96a6fd
Using profileId: 4bce403b-1c69-4aa0-ae13-be06af96a6fd
Creating sessions
Created airtop session b0a123e8-45c6-464b-9da7-86ce21aff34c
Determining whether the user is logged in...
Response if user is logged in
User is already logged in. Live view URL: https://live.airtop.ai/b0a123e8-45c6-464b-9da7-86ce21aff34c?disableResize=false&includeNavigationBar=false&token=eyJhbGciOiJSUzI1NiIsInR5cCI6IkpXVCIsImtpZCI6IllJOTFWRFlhOW00UzB6NXZ5SXYxZnRXd2VJcyJ9.eyJhdWQiOiJhaXJ0b3AvbGl2ZS12aWV3IiwiZXhwIjoxNzMzODQ3ODkyLCJpYXQiOjE3MzM4MzM0OTIsImlzcyI6Imh0dHBzOi8vYWlydG9wLmFpIiwianRpIjoiMzJiNDFkMGUtMjQzNi00YWEzLWJjNGUtOGUwMzQ1ZWQ0MWRjIiwiYXV0aF92ZXJzaW9uIjoibGl2ZS12aWV3LjEiLCJjb25maWd1cmVkX3R0bF9zZWNzIjoxNDQwMCwib3JnX2lkIjoiNDllNmEyNTMtYjM0Ny00OTc3LWI1OTAtODMyYWE2ZWUyZDc0IiwicHJvdmlkZXIiOiJjYjIvYXBpLXNlcnZlciIsInNlc3Npb25faWQiOiJiMGExMjNlOC00NWM2LTQ2NGItOWRhNy04NmNlMjFhZmYzNGMifQ.pYCY98kArunxuApqVngTbkarcxCx1JP44u34Zp1kybBHcpo0cHqdgyoGgP-aJga9qpTF5s37eRteol9SbAWRSSKi5D0b0EX2z8hEbkiIM5N6BpTtv_sm3gtzrlcfVzzu9ZNvq8iBaKRc8-BGMXe3qsB1stsc1rJqQTKNIWpDHbvl5u39myIT7gX1JHr8JTheDrMTmJkekYCHnkI-Q5MVGqjNEEY-zkrL7GxgTmz8F_0Ukr4MzorzDmmDiTg7RK_4jLKD5zhdl5GaJy9aoIDm4ujch8qH_BGk7efF_9ea9QPQCT3QbE6S6FwFabW0oQZlA8ROwnu0CCpf11vOrGs3IQ&windowId=83bb0c26-04c6-4ff3-821f-ce907b7930a2
Navigating to target URL...
Prompting the AI agent, waiting for a response...
Existing workers
id                                         target_url                                             prompt
0   1  https://www.glassdoor.com/Job/san-francisco-ca...  This browser is open to a page that lists avai...
Response:
{
"companies": [
{
"companyName": "OpenAI",
"jobTitle": "Software Engineer, Agent Infrastructure",
"location": "San Francisco, CA",
"salary": {
"min": 360000,
"max": 440000
}
},
{
"companyName": "OpenAI",
"jobTitle": "Software Engineer, Infrastructure",
"location": "San Francisco, CA",
"salary": {
"min": 160000,
"max": 385000
}
},
{
"companyName": "OpenAI",
"jobTitle": "Software Engineer, Safety",
"location": "San Francisco, CA",
"salary": {
"min": 200000,
"max": 370000
}
},
{
"companyName": "OpenAI",
"jobTitle": "Software Engineer, Anti Fraud & Abuse",
"location": "San Francisco, CA",
"salary": {
"min": 200000,
"max": 370000
}
},
{
"companyName": "OpenAI",
"jobTitle": "Android Engineer, ChatGPT Search",
"location": "San Francisco, CA",
"salary": {
"min": 245000,
"max": 385000
}
},
{
"companyName": "OpenAI",
"jobTitle": "TLM, Developer Productivity",
"location": "San Francisco, CA",
"salary": {
"min": 245000,
"max": 385000
}
},
{
"companyName": "OpenAI",
"jobTitle": "Software Engineer, Full Stack",
"location": "San Francisco, CA",
"salary": {
"min": 160000,
"max": 385000
}
},
{
"companyName": "OpenAI",
"jobTitle": "Software Engineer, Security Product and Platform",
"location": "San Francisco, CA",
"salary": {
"min": 245000,
"max": 310000
}
},
{
"companyName": "Grammarly, Inc.",
"jobTitle": "Software Engineer, Front-End",
"location": "San Francisco, CA",
"salary": {
"min": 160000,
"max": 321000
}
},
{
"companyName": "Grammarly, Inc.",
"jobTitle": "Software Engineer, macOS",
"location": "San Francisco, CA",
"salary": {
"min": 195000,
"max": 231000
}
}
]
}
Session terminated
When I ran the utils.py after running two times the worker.py
These are the available workers
TARGET_URL: https://www.glassdoor.com/Job/san-francisco-ca-software-engineer-jobs-SRCH_IL.0,16_IC1147401_KO17,34.htm
PROMPT: This browser is open to a page that lists available job roles for software engineers in San Francisco. Please provide 10 job roles that appear to be posted by the AI-related companies.

This is the history of results in the db
[{'id': 1, 'target_url': 'https://www.glassdoor.com/Job/san-francisco-ca-software-engineer-jobs-SRCH_IL.0,16_IC1147401_KO17,34.htm', 'prompt': 'This browser is open to a page that lists available job roles for software engineers in San Francisco. Please provide 10 job roles that appear to be posted by the AI-related companies.', 'result': '{\n  "companies": [\n    {\n      "companyName": "OpenAI",\n      "jobTitle": "Software Engineer, Agent Infrastructure",\n      "location": "San Francisco, CA",\n      "salary": {\n        "min": 360000,\n        "max": 440000\n      }\n    },\n    {\n      "companyName": "OpenAI",\n      "jobTitle": "Software Engineer, Safety",\n      "location": "San Francisco, CA",\n      "salary": {\n        "min": 200000,\n        "max": 370000\n      }\n    },\n    {\n      "companyName": "OpenAI",\n      "jobTitle": "Software Engineer, Infrastructure",\n      "location": "San Francisco, CA",\n      "salary": {\n        "min": 160000,\n        "max": 385000\n      }\n    },\n    {\n      "companyName": "OpenAI",\n      "jobTitle": "Android Engineer, ChatGPT Search",\n      "location": "San Francisco, CA",\n      "salary": {\n        "min": 245000,\n        "max": 385000\n      }\n    },\n    {\n      "companyName": "OpenAI",\n      "jobTitle": "Software Engineer, Anti Fraud & Abuse",\n      "location": "San Francisco, CA",\n      "salary": {\n        "min": 200000,\n        "max": 370000\n      }\n    },\n    {\n      "companyName": "OpenAI",\n      "jobTitle": "Software Engineer, Security Product and Platform",\n      "location": "San Francisco, CA",\n      "salary": {\n        "min": 245000,\n        "max": 310000\n      }\n    },\n    {\n      "companyName": "OpenAI",\n      "jobTitle": "Software Engineer, Full Stack",\n      "location": "San Francisco, CA",\n      "salary": {\n        "min": 160000,\n        "max": 385000\n      }\n    },\n    {\n      "companyName": "OpenAI",\n      "jobTitle": "TLM, Developer Productivity",\n      "location": "San Francisco, CA",\n      "salary": {\n        "min": 245000,\n        "max": 385000\n      }\n    },\n    {\n      "companyName": "Grammarly, Inc.",\n      "jobTitle": "Software Engineer, Front-End",\n      "location": "San Francisco, CA",\n      "salary": {\n        "min": 160000,\n        "max": 321000\n      }\n    },\n    {\n      "companyName": "Grammarly, Inc.",\n      "jobTitle": "Software Engineer, macOS",\n      "location": "San Francisco, CA",\n      "salary": {\n        "min": 195000,\n        "max": 231000\n      }\n    }\n  ]\n}'}, {'id': 2, 'target_url': 'https://www.glassdoor.com/Job/san-francisco-ca-software-engineer-jobs-SRCH_IL.0,16_IC1147401_KO17,34.htm', 'prompt': 'This browser is open to a page that lists available job roles for software engineers in San Francisco. Please provide 10 job roles that appear to be posted by the AI-related companies.', 'result': '{\n  "companies": [\n    {\n      "companyName": "OpenAI",\n      "jobTitle": "Software Engineer, Agent Infrastructure",\n      "location": "San Francisco, CA",\n      "salary": {\n        "min": 360000,\n        "max": 440000\n      }\n    },\n    {\n      "companyName": "OpenAI",\n      "jobTitle": "Software Engineer, Infrastructure",\n      "location": "San Francisco, CA",\n      "salary": {\n        "min": 160000,\n        "max": 385000\n      }\n    },\n    {\n      "companyName": "OpenAI",\n      "jobTitle": "Software Engineer, Safety",\n      "location": "San Francisco, CA",\n      "salary": {\n        "min": 200000,\n        "max": 370000\n      }\n    },\n    {\n      "companyName": "OpenAI",\n      "jobTitle": "Software Engineer, Anti Fraud & Abuse",\n      "location": "San Francisco, CA",\n      "salary": {\n        "min": 200000,\n        "max": 370000\n      }\n    },\n    {\n      "companyName": "OpenAI",\n      "jobTitle": "Android Engineer, ChatGPT Search",\n      "location": "San Francisco, CA",\n      "salary": {\n        "min": 245000,\n        "max": 385000\n      }\n    },\n    {\n      "companyName": "OpenAI",\n      "jobTitle": "TLM, Developer Productivity",\n      "location": "San Francisco, CA",\n      "salary": {\n        "min": 245000,\n        "max": 385000\n      }\n    },\n    {\n      "companyName": "OpenAI",\n      "jobTitle": "Software Engineer, Full Stack",\n      "location": "San Francisco, CA",\n      "salary": {\n        "min": 160000,\n        "max": 385000\n      }\n    },\n    {\n      "companyName": "OpenAI",\n      "jobTitle": "Software Engineer, Security Product and Platform",\n      "location": "San Francisco, CA",\n      "salary": {\n        "min": 245000,\n        "max": 310000\n      }\n    },\n    {\n      "companyName": "Grammarly, Inc.",\n      "jobTitle": "Software Engineer, Front-End",\n      "location": "San Francisco, CA",\n      "salary": {\n        "min": 160000,\n        "max": 321000\n      }\n    },\n    {\n      "companyName": "Grammarly, Inc.",\n      "jobTitle": "Software Engineer, macOS",\n      "location": "San Francisco, CA",\n      "salary": {\n        "min": 195000,\n        "max": 231000\n      }\n    }\n  ]\n}'}]

In order to not make the README file too big, I created another markdown document called "How to schedule worker.md".
I can put it in the README section if it's better


SPEED-828: Deprecate Auto Minify feature from Settings, Configuration Rules and Page Rules

Summary
In preparation for the deprecation of Auto Minify from the Speed > Optimization settings, but also from the Configuration Rules and Page Rules, which is scheduled on August 5th, 2024, this commit removes any mention of Auto Minify.
Documentation checklist

[x] Files which have changed name or location have been allocated redirects.

@pedrosousa As per the discussion we had internally, I'm closing this PR in favor of an internal change already being made.


Improve STEIM2 decode performance

First, apply "forward branches are not likely to be taken" branch pattern on decoded data store branch.
Second, improve nibble selection by altering switch statement to multiple if-else statements.
Third, apply expansion macro for expanding diff array value store for loop.

Hi @Cuda-Chen, thank you for submitting this PR.
In my testing ,the improvement is relatively small, which seems to agree with your own results.  Also, the PR includes lots of "macro magic", which, while potentially effective, makes the code less clear and harder to debug/maintain in my opinion.
I tried my own optimization in PR #102, within which is a test routine and results using your unpackdata.c source file for comparison.  I would be grateful if you could review #102.  Even better, if you have improvements on that code please add a PR for the optimize_steim_decoders branch.  Thank you!

Close this PR as we prefer to use #102 .


No labels in relation concepts

The Book.csv does not return any labels for the relation concepts. Is that a correct behaviour?

The labels are missing in the TableMinder+ result. Nothing is lost in our conversions.

LodLive seems to provide label for "surname" for example, so I suspect that there indeed is some error in the core.

There should be some label in dbpedia. Maybe the predicates in which the label is searched for are incorrectly set up?

Please, @JanVana , fix it as soon as possible.

This will be more complicated than a simple bug fix. As of now, the core algorithm does not search for relation labels and does not event expect to receive labels from the KB search package.

@brodecva, Jan may look on that when search/propose issues are done.

Possible solutions:

Adjust KBSearch algo
Simple service na backendu, ktera ziska name tak, ze dereferencuje URL a vrati label/title?

@brodecva @JanVana What was the decided approach? Or which one do you recommend?

@JanVana @tomas-knap To be honest, I see no reason to not make the algorithm work the propert way and fetch the labels for properties as well.

This issue is implemented in the pull request #348.


Run bosh cleanup after deploy

What
This way we can remove previous releases/packages/stemcells immediately after they stop being used. This is also important for long running environments where we would not run cleanup otherwise and only accumulate previous versions forever, which can also potentially impact BOSH performance and reliability.
How to review
Apply this PR. Run deploy job. Observe it running bosh cleanup task.
Who can review
not @mtekel

Works and cleaned up a lot of things from my dev environment


StringIndexOutOfBoundsException in CfrDecompiler.clean

Upon decompiling a specific class in Recaf 2.17.3 with CFR, the Recaf CFR decompiler integration (not the decompiler itself) runs into "java.lang.StringIndexOutOfBoundsException"
To Reproduce

Download the JAR for https://mvnrepository.com/artifact/com.google.code.gson/gson/2.2.4 (https://repo1.maven.org/maven2/com/google/code/gson/gson/2.2.4/gson-2.2.4.jar)
File -> Load -> gson-2.2.4.jar
Decompile class "com/google/gson/internal/$Gson$Types$GenericArrayTypeImpl"

//exp: decompiles successfully
//act: java.lang.StringIndexOutOfBoundsException
Exception
From console output:
08:38:40.837 [pool-6-thread-3] ERROR: Class failed to decompiled: com/google/gson/internal/$Gson$Types$GenericArrayTypeImpl
java.lang.StringIndexOutOfBoundsException: String index out of range: -1
        at java.lang.String.substring(String.java:1967)
        at me.coley.recaf.decompile.cfr.CfrDecompiler.clean(CfrDecompiler.java:93)
        at me.coley.recaf.decompile.cfr.CfrDecompiler.decompile(CfrDecompiler.java:64)
        at me.coley.plugin.decompile.DecompilePanel.lambda$startDecompile$2(DecompilePanel.java:78)
        at java.util.concurrent.Executors$RunnableAdapter.call(Executors.java:511)
        at java.util.concurrent.FutureTask.run(FutureTask.java:266)
        at java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1149)
        at java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:624)
        at java.lang.Thread.run(Thread.java:748)

i.e. the crash is not in CFR (which I validated to decompile), but in the integration of CFR at https://github.com/Col-E/Recaf/blob/master/src/main/java/me/coley/recaf/decompile/cfr/CfrDecompiler.java#L93

This is not limited to this specific .jar, but apparently a more generic problem.
In a larger context, I get quite a few of these exceptions (with the same stack trace) which would suggest that this is a more general problem. Alas, I do not understand what clean() is trying to accomplish, so I cannot really contribute more constructive feedback at this time.

This is not limited to this specific .jar, but apparently a more generic problem.
In a larger context, I get quite a few of these exceptions (with the same stack trace) which would suggest that this is a more general problem. Alas, I do not understand what clean() is trying to accomplish, so I cannot really contribute more constructive feedback at this time.

Can confirm that, i will try to fix this later.

Can confirm that, i will try to fix this later.

private static final ThreadLocal<DecompilerContext> currentContext = new ThreadLocal();

This sorta stuff is just... 🙄 why you gotta do this FF? Fix was to wrap the crashing bit with synchronized(this)

private static final ThreadLocal<DecompilerContext> currentContext = new ThreadLocal();

This sorta stuff is just... 🙄 why you gotta do this FF? Fix was to wrap the crashing bit with synchronized(this)


Make --template-file parameter optional with .bicepparam file

Related command

az deployment
Description

In order to support .bicepparam files which link directly to Bicep Registry, Template Spec or JSON content, we must relax the requirement for the user to provide the --template-file parameter for deployment-related commands. This parameter is currently redundant, as .bicepparam files require the user to link to a .bicep file via the using statement.
To maintain backwards-compatibility, continue to support the "old" command if the user supplies the --template-file parameter.
Testing Guide
az deployment group create \
  -g myRg \
  -p .\bicepparam\storage_account_params.bicepparam

Notes

Integration tests in this PR are LiveScenarioTest because we need to do a Bicep CLI version check (meaning it has to be installed) before performing any .bicepparam related logic

History Notes

[ARM] az deployment group create: Make --template-file parameter optional when used with .bicepparam parameter file.

This checklist is used to make sure that common guidelines for a pull request are followed.

[x] The PR title and description has followed the guideline in Submitting Pull Requests.

[x] I adhere to the Command Guidelines.

[x] I adhere to the Error Handling Guidelines.

Thank you for your contribution! We will review the pull request and get back to you soon.

@zhoxing-ms would it be possible to get this reviewed please? The Azure PowerShell counterpart PR has been merged (https://github.com/Azure/azure-powershell/pull/22728)

@zhoxing-ms please could you take another look?


Handle cycles when showing types

For example,
var arr = [arr];
If we try to show the type of arr, the stack overflows.

Close by #31


Removed deprecated Sway from list of record types

…being deprecated
Also fix for https://github.com/MicrosoftDocs/office-docs-powershell/issues/6328

Docs Build status updates of commit d346662:
:clock10: Full build: the average full build time is 17 min(s) 37 sec(s), this is based on the last 1000 build(s) for this repository.

Docs Build status updates of commit d346662:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

exchange/exchange-ps/exchange/Get-UnifiedAuditLogRetentionPolicy.md
:white_check_mark:Succeeded
View (exchange-ps)

exchange/exchange-ps/exchange/New-ActivityAlert.md
:white_check_mark:Succeeded
View (exchange-ps)

exchange/exchange-ps/exchange/New-ComplianceSearchAction.md
:white_check_mark:Succeeded
View (exchange-ps)

exchange/exchange-ps/exchange/New-UnifiedAuditLogRetentionPolicy.md
:white_check_mark:Succeeded
View (exchange-ps)

exchange/exchange-ps/exchange/Search-UnifiedAuditLog.md
:white_check_mark:Succeeded
View (exchange-ps)

exchange/exchange-ps/exchange/Set-ActivityAlert.md
:white_check_mark:Succeeded
View (exchange-ps)

exchange/exchange-ps/exchange/Set-UnifiedAuditLogRetentionPolicy.md
:white_check_mark:Succeeded
View (exchange-ps)

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

Docs Build status updates of commit 6a4b0d9:
:clock10: Full build: the average full build time is 17 min(s) 37 sec(s), this is based on the last 1000 build(s) for this repository.

Docs Build status updates of commit 6a4b0d9:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

exchange/exchange-ps/exchange/Get-UnifiedAuditLogRetentionPolicy.md
:white_check_mark:Succeeded
View (exchange-ps)

exchange/exchange-ps/exchange/New-ActivityAlert.md
:white_check_mark:Succeeded
View (exchange-ps)

exchange/exchange-ps/exchange/New-ComplianceSearchAction.md
:white_check_mark:Succeeded
View (exchange-ps)

exchange/exchange-ps/exchange/New-UnifiedAuditLogRetentionPolicy.md
:white_check_mark:Succeeded
View (exchange-ps)

exchange/exchange-ps/exchange/Search-UnifiedAuditLog.md
:white_check_mark:Succeeded
View (exchange-ps)

exchange/exchange-ps/exchange/Set-ActivityAlert.md
:white_check_mark:Succeeded
View (exchange-ps)

exchange/exchange-ps/exchange/Set-UnifiedAuditLogRetentionPolicy.md
:white_check_mark:Succeeded
View (exchange-ps)

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

Docs Build status updates of commit f69ab9c:
:clock10: Full build: the average full build time is 17 min(s) 37 sec(s), this is based on the last 1000 build(s) for this repository.

Docs Build status updates of commit f69ab9c:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

exchange/exchange-ps/exchange/Get-UnifiedAuditLogRetentionPolicy.md
:white_check_mark:Succeeded
View (exchange-ps)

exchange/exchange-ps/exchange/New-ActivityAlert.md
:white_check_mark:Succeeded
View (exchange-ps)

exchange/exchange-ps/exchange/New-ComplianceSearchAction.md
:white_check_mark:Succeeded
View (exchange-ps)

exchange/exchange-ps/exchange/New-UnifiedAuditLogRetentionPolicy.md
:white_check_mark:Succeeded
View (exchange-ps)

exchange/exchange-ps/exchange/Search-UnifiedAuditLog.md
:white_check_mark:Succeeded
View (exchange-ps)

exchange/exchange-ps/exchange/Set-ActivityAlert.md
:white_check_mark:Succeeded
View (exchange-ps)

exchange/exchange-ps/exchange/Set-UnifiedAuditLogRetentionPolicy.md
:white_check_mark:Succeeded
View (exchange-ps)

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


Wrong SQL generated when filter is used

I have a data table with a field that contains data with double quoted text at the end:
some text "another text"

The last double quote is dropped in generated SQL when I use this field in a filter.
Actual results
WHERE org_name IN ('some text "another text')

Expected results
WHERE org_name IN ('some text "another text"')

How to reproduce the bug

Create dataset with data as above
Create chart
Create filter
Create dashboard
Use filter in dashboard
See chart is empty.
Explore chart. View chart SQL code in SQL Lab

Environment

DB - PostgreSQL 10
superset version: 0.37.2
python version: 3.6

Hi, any update on this?
version 1.3.1 still has this issue.

I have the same issue, using 1.3.0 version with PostgreSQL connector


Feature/2418 manual ref html

Submission Checklist

[x] Declare copyright holder and open-source license: see below

Summary
Defines a bookdown document for the reference manual.
Remove the reference manual from the user's guide and reference manual, renaming it.  Included reference from user's guide to new reference manual.
Intended Effect
Make it easy to edit and search our manuals.  The rest of the manual will get broken out in the same way.
How to Verify
Generate the new manual using the included R script.
Side Effects
The manual is no longer self-contained as a reference and user's guide.
Code
Nope, just doc.
Copyright and Licensing
Please list the copyright holder for the work you are submitting (this will be you or your assignee, such as a university or company):
Columbia University
By submitting this pull request, the copyright holder is agreeing to license the submitted work under the following licenses:

Code: BSD 3-clause (https://opensource.org/licenses/BSD-3-Clause)
Documentation: CC-BY 4.0 (https://creativecommons.org/licenses/by/4.0/)

Thanks Bob! This is going to be really helpful.

This is just step 1.  It's just pulling the reference manual out.  When we pull the function reference out, we'll coordinate to make sure it's useful for R/RStudio.

script depends on package "bookdown".
changed Rscript command:
Rscript --default-packages=bookdown -e "bookdown::render_book('index.Rmd', 'bookdown::gitbook')"

this works.
similarly, if building via RStudio, you must first load the bookdown library before knitting gitbook.

gitbook version of manual also depends on pandoc-citeproc
on my mac (High Sierra, osx 10.13), I used homebrew and installed latest pandoc as well as pandoc-citeproc.  not sure, but I think homebrew messed up local fonts for latex - I had to reinstall lucida fonts, also did a total reinstall of MacTeX.

I'll re-open this when all the doc's split apart.


Can't get sms when use phone number Auth ?

I use the quickstart's sample c# code into a new unity project， I have the following problems:
1.it can't use the Phone No. Auth to register a new phone number(sign up)?

When I use the phone number which create on firebase console， I can get the log: "phone auth code sent" , but I can't get any real sms from my phone?

@uokivan I recommend opening up a support request on http://firebase.google.com/support .  Folks will be able to work through the issue with you there with access to your Firebase project.


How to NOT log post data in PART C of modsec_audit.log

Right now i have modsec 2.5 installed on apache. Seem to work fine except for the part where it clearly shows PII info like ssn and password in post. SantiseArgs seem to not work for json posts. So my question are (1), how to do santise json post or (2), how do i make sure part C is not in my logs?

The version of modsec you're running is quite old - some of these issues might be fixed in newer version.
To remove a part of an audit log you can use ctl as follows ctl:auditLogParts=-C


some bugs

@ run_atari_dqn.py
before using variable "seed", should add
seed = args.seed
run_random_policy(args.fill_buffer) takes quite a long time, so I commented that line and now it runs well

hey, thank you for mentioning the bug! ill fix it!
For the fill buffer you can just set the argument to 0. So no random samples are created and stored in the buffer before training.
If you find any other problems feel free to mention. Ill soon update the repo, make some changes in creating the evaluation data and adding IQN (+per, +noisy layer +n_step addon).


There maybe a mistake in the code

At line 134 and 135, reconstruct_mesh.py:
a1 = 2*np.pi/np.sqrt(3.0)
a2 = 2*np.pi/np.sqrt(8.0)

I think the two np.sqrts are incorrectly added here. Please have a check.

At line 134 and 135, reconstruct_mesh.py:
a1 = 2*np.pi/np.sqrt(3.0)
a2 = 2*np.pi/np.sqrt(8.0)

I think the two np.sqrts are incorrectly added here. Please have a check.

I think you are right, a1 is actually A_1^{hat} from An Efficient Representation for Irradiance Environment Maps. I have been checking this out for the whole morning.

At line 134 and 135, reconstruct_mesh.py:
a1 = 2*np.pi/np.sqrt(3.0)
a2 = 2*np.pi/np.sqrt(8.0)

I think the two np.sqrts are incorrectly added here. Please have a check.

I think you are right, a1 is actually A_1^{hat} from An Efficient Representation for Irradiance Environment Maps. I have been checking this out for the whole morning.

但是加不加好像结果都差球不多....


fix: 3.0 Client cannot connect to server on subdirectory

Closes #1775

Thank you for pointing out. I have confirmed that the additional commit by tassoevan works on my server.

@tassoevan I'm confused here. Is this supposed to be fixed in 3.0.3? Because we have the issue with 3.0.3 still.

The same issue is still open. #1796


Document rpc limitations during pipeline sync

Describe the bug
I am running reth using:
/usr/local/bin/reth node --datadir /datadrive/reth --http --http.addr <IP_ADDRESS> --http.port 8545 --http.api all --http.corsdomain "*" --authrpc.jwtsecret /secrets/jwt.hex
and lighthouse using:
/usr/local/bin/lighthouse bn --network mainnet --execution-endpoint http://localhost:8551/ --execution-jwt "/secrets/jwt.hex" --checkpoint-sync-url https://mainnet.checkpoint.sigp.io/ --datadir /datadrive/lighthouse --disable-deposit-contract-sync
The node is still syncing.

But when I tried to send json-rpc request to the node, some RPC requests returned the result, while others returned empty/null.
JSON-RPC:
eth_getBlockByNumber - worked

eth_getBlockReceipts - returned empty response

Steps to reproduce

Run reth and lighthouse
Make RPC calls to the server when it is still syncing.

Node logs
No response
Platform(s)
Linux (x86)
What version/commit are you on?
0.1.0-alpha.8
What database version are you on?
Current database version: 1
Local database version: 1
If you've built Reth from source, provide the full command you used
Used this tutorial: https://paradigmxyz.github.io/reth/installation/source.html and then ran reth as systemd service
Code of Conduct

[X] I agree to follow the Code of Conduct

this is due to how stage-sync works.
the required data may only be available after all stages finished, then the node is technically synced. during pipeline sync some endpoints some endpoints can't be served.
for example, receipts is only available after execution stage where the receipts are generated.
We should document this better in the book if we don't have already

this is due to how stage-sync works.
the required data may only be available after all stages finished, then the node is technically synced. during pipeline sync some endpoints some endpoints can't be served.
for example, receipts is only available after execution stage where the receipts are generated.
We should document this better in the book if we don't have already

Ohh, Alright thanks.
is there any discord/TG devs channel for reth?


Error extracting tracksplit.pssg

Open \locations\germany__germany_rally_01.nefs.
Try to extract tracksplit.pssg.
Following error happens:
Error extracting item tracks\locations\germany\germany_rally_01\tracksplit.pssg.
System.IO.InvalidDataException: Block length does not match with its complement.
   at System.IO.Compression.Inflater.DecodeUncompressedBlock(Boolean& end_of_block)
   at System.IO.Compression.Inflater.Decode()
   at System.IO.Compression.Inflater.Inflate(Byte[] bytes, Int32 offset, Int32 length)
   at System.IO.Compression.DeflateStream.Read(Byte[] array, Int32 offset, Int32 count)
   at System.IO.Stream.InternalCopyTo(Stream destination, Int32 bufferSize)
   at System.IO.Stream.CopyTo(Stream destination)
   at VictorBush.Ego.NefsLib.NefsItem.Extract(String outputDir, String outputFileName, NefsProgressInfo p) in E:\Git\victorbush.ego.nefsedit\VictorBush.Ego.NefsLib\NefsItem.cs:line 515
   at VictorBush.Ego.NefsEdit.UI.EditorForm.<>c__DisplayClass9_0.<ExtractItems>b__0()

This is the only file with which the bug happens in all locations nefs. Also the weird part is that it always crashes on the same block even in different files 6540 off of the top of my head. But I might be wrong.
Tried to look into it but am stumped.

I also get this when extracting s_amb_germany_rally_01.bnk.

Also strange note: when extracting the tracksplit.pssg, it seems to choke on blocks of 0x10000 bytes. If i skip those blocks and treat them as unencrypted/uncompressed (just copy them direct), then I can open the resulting pssg file in the pssg editor.
That hack didn't work for s_amb_germany_rally_01.bnk though.

Incredibly weird... Maybe there's some metadata in parts 5/6/7/8 of the
header we're missing for these blocks.
Also have you checked the that the files extracted from pssg and the size
are correct? If I recall correctly PSSG files will open even if semi
corrupt.

File size is correct. There are 797 chunks that it chokes on. The PSSG viewer allows browsing of most textures, but there are some that it can't view, so possibly some corruption in there.

It seems treestextures.pssg is suffering from the same problem.
[09:01:06 ERR] Failed to extract item treestextures.pssg.
System.IO.InvalidDataException: Found invalid data while decoding.
   at System.IO.Compression.DeflateStream.EndRead(IAsyncResult asyncResult)
   at System.IO.Stream.<>c.<BeginEndReadAsync>b__43_1(Stream stream, IAsyncResult asyncResult)
   at System.Threading.Tasks.TaskFactory`1.FromAsyncTrimPromise`1.Complete(TInstance thisRef, Func`3 endMethod, IAsyncResult asyncResult, Boolean requiresSynchronization)
--- End of stack trace from previous location where exception was thrown ---
   at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(Task task)
   at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
   at System.IO.Stream.<CopyToAsyncInternal>d__27.MoveNext()
--- End of stack trace from previous location where exception was thrown ---
   at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(Task task)
   at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
   at VictorBush.Ego.NefsLib.Utility.StreamExtensions.<CopyToAsync>d__2.MoveNext()
--- End of stack trace from previous location where exception was thrown ---
   at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(Task task)
   at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
   at System.Runtime.CompilerServices.TaskAwaiter.ValidateEnd(Task task)
   at VictorBush.Ego.NefsLib.IO.NefsTransformer.<DetransformChunkAsync>d__5.MoveNext()
--- End of stack trace from previous location where exception was thrown ---
   at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(Task task)
   at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
   at VictorBush.Ego.NefsLib.IO.NefsTransformer.<DetransformAsync>d__4.MoveNext()
--- End of stack trace from previous location where exception was thrown ---
   at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(Task task)
   at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
   at VictorBush.Ego.NefsLib.IO.NefsTransformer.<DetransformFileAsync>d__6.MoveNext()
--- End of stack trace from previous location where exception was thrown ---
   at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(Task task)
   at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
   at VictorBush.Ego.NefsEdit.Workspace.NefsEditWorkspace.<ExtractFileAsync>d__75.MoveNext()

https://github.com/ptasev/Ego-Engine-Modding/issues/24

It seems treestextures.pssg is suffering from the same problem.
[09:01:06 ERR] Failed to extract item treestextures.pssg.
System.IO.InvalidDataException: Found invalid data while decoding.
   at System.IO.Compression.DeflateStream.EndRead(IAsyncResult asyncResult)
   at System.IO.Stream.<>c.<BeginEndReadAsync>b__43_1(Stream stream, IAsyncResult asyncResult)
   at System.Threading.Tasks.TaskFactory`1.FromAsyncTrimPromise`1.Complete(TInstance thisRef, Func`3 endMethod, IAsyncResult asyncResult, Boolean requiresSynchronization)
--- End of stack trace from previous location where exception was thrown ---
   at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(Task task)
   at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
   at System.IO.Stream.<CopyToAsyncInternal>d__27.MoveNext()
--- End of stack trace from previous location where exception was thrown ---
   at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(Task task)
   at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
   at VictorBush.Ego.NefsLib.Utility.StreamExtensions.<CopyToAsync>d__2.MoveNext()
--- End of stack trace from previous location where exception was thrown ---
   at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(Task task)
   at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
   at System.Runtime.CompilerServices.TaskAwaiter.ValidateEnd(Task task)
   at VictorBush.Ego.NefsLib.IO.NefsTransformer.<DetransformChunkAsync>d__5.MoveNext()
--- End of stack trace from previous location where exception was thrown ---
   at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(Task task)
   at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
   at VictorBush.Ego.NefsLib.IO.NefsTransformer.<DetransformAsync>d__4.MoveNext()
--- End of stack trace from previous location where exception was thrown ---
   at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(Task task)
   at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
   at VictorBush.Ego.NefsLib.IO.NefsTransformer.<DetransformFileAsync>d__6.MoveNext()
--- End of stack trace from previous location where exception was thrown ---
   at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(Task task)
   at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
   at VictorBush.Ego.NefsEdit.Workspace.NefsEditWorkspace.<ExtractFileAsync>d__75.MoveNext()

https://github.com/ptasev/Ego-Engine-Modding/issues/24


fix: dupe CtrlValue in debug message, WasHit issued with connected Control too

Duplicated controller value with PrintLogMessage enabled
WasHit wasn't issued with a wired Control parameter
useless Control.Value update (happens only in the if(_teachingActive) section)

Thanks. Combined, these incremental improvements really make a difference!


The results obtained with pyhealth are much lower than in the paper.

Thanks for your great job!
But I wonder why the results reported on the PyHealth homepage are much lower than those reported in the paper of SafeDrug. And according to the results reported by PyHealth, GAMENet performs better than Safedrug, which contradicts the paper's results.

Below are the results reported in the SafeDrug paper,

We are actively fixing this problem. Will reply to you soon.

Thanks! Looking forward to your reply.

Hi, these results shown on the webs are old results, which will be updated soon. Could you please run the scripts in example/ and obtain your own new results? Here are the performance of GAMENet I obtained by running python examples/drug_recommendation_mimic3_gamenet.py.
{'pr_auc_samples': 0.7071216948197928, 'loss': 0.2621602493930947}

feel free to re-open the issue if you have further questions.

Hello ^ ^, thank you very much for the PyHealth framework's contribution to drug recommendation.
I have a similar situation, that is, the jaccard_samples of my locally run GAMENet model can reach about 0.43, and the pr_auc_samples of my SafeDrug model can also reach about 0.65.
Here is the problem: the jaccard_samples of my local SafeDrug can only reach about 0.33. Theoretically, the jaccard_samples of SafeDrug should be similar to GAMENet. Why is there such a big gap?
Here are the performance of SafeDrug I obtained by running python examples/drug_recommendation_mimic3_safedrug.
2023-09-17 11:10:52 --- Eval epoch-16, step-6035 --- 2023-09-17 11:10:52 jaccard_samples: 0.3246 2023-09-17 11:10:52 loss: 0.3497 2023-09-17 11:10:52 New best jaccard_samples score (0.3246) at epoch-16, step-6035
Looking forward to and thank you for your reply! ^^

Hello @Tyunsen  , I find the SafeDrug adaptive loss has issues and already solved it. Please install the new pyhealth version by "git clone" and "pip install ." Thanks!


Fix TTL log

This is printing the date its expired, not the TTL. Now its the
duration.
[ ] Configuration Infrastructure
[ ] Docs
[ ] Installation
[ ] Networking
[ ] Performance and Scalability
[ ] Policies and Telemetry
[ ] Security
[ ] Test and Release
[ ] User Experience
[ ] Developer Infrastructure
Pull Request Attributes
Please check any characteristics that apply to this pull request.
[ ] Does not have any user-facing changes. This may include CLI changes, API changes, behavior changes, performance improvements, etc.

@howardjohn: The following test failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

integ-security-multicluster-tests_istio
7a1342a30697a9bcddf1b40b58d7a1976877706f
link
/test integ-security-multicluster-tests_istio

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


[SU-286] Fix a11y issues in Azure preview form

A few form elements in the Azure preview form are missing a label.
Noticed these warnings when running unit tests.
For accessibility, Checkbox needs a label. Resolve this by doing any of the following:
  * add an aria-label property to this component
  * add an aria-labelledby property referencing the id of another component containing the label
  * create a label component and point its htmlFor property to this component's id

For accessibility, TextArea needs a label. Resolve this by doing any of the following:
  * add an aria-label property to this component
  * add an aria-labelledby property referencing the id of another component containing the label
  * create a label component and point its htmlFor property to this component's id

I understand the motivation for using a generated ID for components that may appear multiple times on a page (like LabeledCheckbox). But in this case, there will only ever be one instance of the form, so a static ID seems like it would be fine.


When map return undefined, type will became Option

When map return undefined, type will became Option
import { Option, some, none } from 'fp-ts/lib/Option'
const data = some({ name: 'kate' })
  .map(() => undefined)

With the proplem such code will fall in runtime:
import { Option, some, none } from 'fp-ts/lib/Option'
const data = some({ name: 'kate' })
  .map(() => undefined)
  .map(({ name }) => `hey ${name}`)

Sorry, this is wrong inferred type from function () => undefined, TS think what it's (): any


Remove OpenCV and SceneFromVideo

opencv does not seem to be used anywhere other than in ScenefromVideo, which seems to be broken.

I vote we 100% scrap this file and opencv. Not only because it's broken but because this file is actually trying to use manim in a way that it's not intended. It seems like an awesome project to be honest, but I don't think it belongs inside manim itself. In the future, if anyone wants to work on this, I think it should live inside the ManimCommunity organization, but in a separate repo than manim itself.


The multi-task learning setting is different from the original paper

According to the GLM paper, multi-task learning has two way, one is a mixture of the blank infilling object and the sentence-level objective, another is a mix of the blank infilling object and the document-level objective.
But when I read the pre-training code (/config/ds_block_chinese.sh), I found that multi-task learning takes 40% blank infilling object, 30% sentence-level objective, 30% document-level objective.
Am I understanding it wrong?
...
gpt_options=" \
       --block-lm \
       --task-mask \
       --bert-prob 0.4 \
       --gap-sentence-prob 0.3 \
...

The two ways of multi-task learning are designed for ablation study of different objectives. To enable adaption to different downstream tasks, we mix all the three types of objectives in the Chinese model.


static datatypes and attributes

@mbastian  said

Add an optional edge kind attribute: This is the base of the property graph and defined some kind of label that defines the nature of the relationship. Unfortunately we already use type (for directed/undirected) and label (unique string for each edge) but "kind" seems clear enough.

idtype: The current 1.2 specifications describe an idtype attribute one can set on graph to configure the type of element ids. This is actually something that we never implemented in the import/export and could now be done because GraphStore supports it. So I suggest to start supporting this and also add long in addition of string (default) and integer.
slice mode: To make the distiction between dynamic graphs contained into a single GEXF (i.e. our current 'dynamic' mode) and dynamic graphs contained in a collection of GEXF files I suggest to introduce an additional 'mode' attribute names 'slice'. The possible values are currently 'static' (default) and 'dynamic'.
Add a new “hex” attribute to viz:color: It would allow to use things like hex=“#FF00FF”. I would suggest to support alpha there as well.

slices aren't defined in the grammar yet, since i m not 100% clear on what they should do

Slices are a way to characterize a complete <graph> over time and say something like "this graph is at that state at timestamp t. You can imagine the scenario where a researcher is making a snapshot of a given graph at regular periods. Each snapshot would then be a different GEXF file and I would call this snapshot a slice of the global over-time network. For gephi to allow the merge of these different GEXF files it needs to know when was this snapshot taken and that's the role of the timestamp attribute on graph. Alternatively, we allow to characterize a slice/snapshot with an interval.

Slices are a way to characterize a complete <graph> over time and say something like "this graph is at that state at timestamp t. You can imagine the scenario where a researcher is making a snapshot of a given graph at regular periods. Each snapshot would then be a different GEXF file and I would call this snapshot a slice of the global over-time network. For gephi to allow the merge of these different GEXF files it needs to know when was this snapshot taken and that's the role of the timestamp attribute on graph. Alternatively, we allow to characterize a slice/snapshot with an interval.

ok got it, seem that the remaining work for slices depends on the commits on the other issue. Closing this one


Allow to get one route by route id

I added a new API for getting special one route information.

[x] read the CONTRIBUTING guidelines
[ ] raised a GitHub issue or discussed it on the projects chat beforehand
[ ] added unit tests
[ ] added integration tests
[ ] updated documentation if needed
[ ] updated CHANGELOG.md

Hi! as part of #1473, we have reorganised a lot of the code.
To clear PRs that needs to be rebased or redone, we are closing open PRs that will require significant code change to be merged.
In addition, the issue of the PR might in some cases have been fixed, change or no longer relevant, so it would be great if this is considered as well.
Thank you for your contribution!
If it is still relevant and the PR is reopened, we will aim at getting the changes into the next release after the reorg if accepted.


Setup a larger / heigher display

Hi, is there a way to setup up a larger display ?
Thanks

Yes. Sorry for not documenting it. You can supply a named parameter initial_window_size as a tuple to the function initializeTurtle like the following:
initializeTurtle(initial_window_size=(1080,720)
This will make the turtle display with 1080 width and 720 height.
Hope this helps.

Could we add this to Readme.md ?
Thanks a lot for your work


RFP - Xtreme Download Manager (xdm)

Checklist

[x] The package I am requesting does not already exist on https://chocolatey.org/packages;
[x] There is no open issue for this package;
[x] The issue title starts 'RFP - ';
[x] The download URL is public and not locked behind a paywall / login;

Package Details
Software project URL : https://github.com/subhra74/xdm
Direct download URL for the software / installer : https://github.com/subhra74/xdm/releases
Software summary / short description: A download manager.

Status: Review
https://chocolatey.org/packages/xdm


Feature/start app script

Description
Adds a createapp command to arctic
Fixes #387
Type of change

New feature (non-breaking change which adds functionality)

Coverage increased (+0.7%) to 74.585% when pulling 1cece4051e22a4be81ccb6cf08bad9e1572c8d1c on feature/start-app-script into b01ac484db0012ea96e60bacbeb87555897e94c3 on develop.


Tooltips not hiding in color / size controls in mobile

Context
In the new color / size controls, when in mobile, if you click on the different radio buttons the tooltips start to stack.

At first it seems a problem with mobile and tipsy or Backbone forms, but there are other forms that works flawlessly, like this radio buttons in Create distance / Travel buffers.

I'll reserve this ticket to Victor. If a customer runs into this before we can pass it to the RT.

FYI client is checking in on this issue. When it's fixed please notify this SupportBee client @CartoDB/support-team

Hi @ztephm
We shipped a fix for this issue. It still fails on mobile but on desktop it presents some improvement. Can you contact the customer and ask him if their particular issue is gone? Thanks

Yep! asked client in this SupportBee Ticket.

Closing this. The client did not answer. We can reopen it if needed.


Fixes pT ordering issue in processor, reruns leptonKinematics

This changes the ordering on objects by p4 (which has been replaced) to the pt attribute. It also seems to change the "version" number in leptonKinematics.ipynb, but I'm not able to find where this is actually placed in the file.

Thanks @yfv2ev! For context, this update was introduced in an earlier commit but accidentally reverted in https://github.com/btcardwell/SIDM/pull/22. I'll merge now.


Please add padding to the rule category intro

@SebastienBoissiere
We need to add padding to each rule category intro.
E.g. https://www.ssw.com.au/rules/rules-to-successful-sales-and-account-management
currently shows as

...should be:

A way to achieve is by adding the same "p-12" class that is the rest of the content to the intro div
AB#58628

Hi @tiagov8
Done - See https://www.ssw.com.au/rules/rules-to-successful-sales-and-account-management

Figure: Padding added to the rule category pages


Don't restore _removeFlag

If I add two or more arrays of markers to the cluster, when I remove an array their markers are marked with the _removeFlag = true.
If I go back to add this array their markers continue marked with _removeFlag = true and then if I remove other array of markers this array is also removed (their points).
I tried to solve it temporarily with:
		for (i = 0, l = this._markers.length; i < l; ++i) {
			this._markers[i]._removeFlag = false;
		}

before
        for (var i = 0, l = markers.length; i < l; ++i) {
            markers[i]._removeFlag = true;
        }

        var newMarkersList = [];
        for (i = 0, l = this._markers.length; i < l; ++i) {
            if (!this._markers[i]._removeFlag) {
                newMarkersList.push(this._markers[i]);
            }
        }

        this._markers = newMarkersList;

It solves the problem but sure there's a better way.

I close the issue because the _removeFlag is deleted in the RegisterMarker method and you are using directly the private _markers class member.
However, your solution should work correctly.


Feat/titlescreen

What is the body.css for. It does not seem like it is in the right language.
#4

Closes #4


Replace heroku/jvm-function-invoker with libcnb 0.2.0 version

Replaces the existing implementation with a new one that uses libcnb 0.2.0.
Highlights:

Clear separation of concerns between individual layers, detect and build
Layer lifecycles are managed by libcnb, reducing boilerplate
Uses libherokubuildpack for various helpers

Uses the libcnb error handler from the library as well for consistent error formatting.

TOML metadata deserialization is now automatically handled by libcnb
Buildpack specific error types for all errors + friendly error messages for all of them
Proper parsing of project.toml to detect the function type

Closes GUS-W-9831675

Are there any more tests we could/should write?
We can always use more tests. :) A big chuck of the tests previously in this repo moved to the libraries instead. Most of what the buildpack does is glueing components together, there isn't much logic to test.

That said, we need to improve the testing utilities that come with libcnb to make testing easier. The main offender being GenericPlatform here. It currently can't be created without reading the environment from disk. Otherwise, it would be easy to at least test detect as a pure function.


The engine -> framework roller is >67 commits behind

https://github.com/flutter/flutter/pull/62080 is the current roll PR and it doesn't appear to be going well.
Assigning @flar who is the current engine sheriff, and @jason-simmons who is shepherding a manual Dart VM roll.

Can the engine->framework auto roller be stopped, the framework and engine trees closed until this is resolved ?

This will probably be blocked on https://github.com/flutter/flutter/issues/62146

I've switched the engine autoroller to dry-run. I don't believe we have the ability to close the engine tree. Maybe @tvolkert knows if we can.

We don't have the ability to truly close either tree - only by asking on chat channels / email for people to hold off on committing.

The engine tree is also red on https://github.com/flutter/flutter/issues/62146, so no one will be landing anything until that is resolved.

The Dart SDK associated with the pending engine roll PR (https://github.com/flutter/flutter/pull/62080) is version 2.9.0-21.0.dev.flutter-a3815b6590.  So flutter_tools can resolve its dependencies and build without causing the problem seen in https://github.com/flutter/flutter/issues/62146.
However, the roll is failing because some framework/tooling tests are timing out.  I can reproduce one of the errors by running:

cd dev/devicelab
../../bin/cache/dart-sdk/bin/dart bin/run.dart -t smoke_test_success

This will show a series of VM service errors and will fail to complete.  Currently investigating why this started happening.

Bisected the start of the smoke_test_success failure to a Dart->engine roll (https://github.com/flutter/engine/commit/183dccecd2765e9a66b1d8fa1250ffd096d6c21c)

@jason-simmons Let's prepare a revert of that roll, notify the Dart team, and disable the auto-roller.

It looks like the test failure started with https://github.com/dart-lang/sdk/commit/157ebfe50f399c59a20d603acdf5143ceaa4cccf in the Dart SDK
@bkonyi

A workaround for these failures is to pass --disable-dart-dev to the dart process. I'll take a look into the actual failure.

It looks like our LWE builds are failing due to an analyzer sdk mismatch.

@flar The builder may have pulled and cached the dev tag for 2.10. @christopherfujino might know whether that's the case here.

The engine has rolled forward. The engine tree still has some redness, but that is being tracked separately, so I will close this.

The engine has rolled forward. The engine tree still has some redness, but that is being tracked separately, so I will close this.

Isn't this tracking the engine -> framework roll? Isn't that still blocked? https://github.com/flutter/flutter/blob/master/bin/internal/engine.version

Ah, right. It is still blocked. Let's leave this open until we can restart the auto-roller. Thanks!

Triage: This is complicated by the 2.10 Dart roll and a lot of folks are already on this.

What's the status here?

The engine roller has been re-enabled. Closing the issue.


Incomplete Syntax definition

Grants assembly if id is a previously defined label, variable, or symbol.

IFDEF id ;;
directive...

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

ID: e51bbd50-0552-ad73-26f6-f57e688f0487
Version Independent ID: f91c0df0-cb50-e344-d911-1e190d5867e4
Content: IFDEF
Content Source: docs/assembler/masm/ifdef.md
Product: visual-cpp
Technology: cpp-masm
GitHub Login: @corob-msft
Microsoft Alias: corob

@Player1UK thanks! See comment for #1123.


Further FS doc updates

See https://trello.com/c/zd9PQRlo/232-better-fs-docs-with-diagrams

Haven't got around to doing more here yet so I'm going to merge this for release today alongside the updated what's new pages.


how to treat custom types in events?

We have the issue that our SubstraTEE-client needs to decode custom types in events.
A workaround is this:
https://github.com/scs/substrate-api-client/blob/ea4192357ddc02bf05c0218e24026b2419289228/src/events.rs#L101
this does work but we don't want to insert custom types here.
Could we leave the decoding to the user?

Yes, agreed.
The problem is rather (not 100% sure atm) that decoding vector Vec::<frame_system::EventRecord<Event, Hash>> is screwed up if there are unknown types.
However,  decoder.register_type_size  is public. Primitives can easily be added by the user by just calling the method on the decoder that is returned by EventDecoder::try_from(metadata) to add an extra type.
Which means, we need to rewrite the workflow such that the user has access to the decoder, which is now hidden in the api.wait_for_event method. But this can be easiliy done.
I am unsure what problem the following poses:
pub struct Request {
   pub shard: ShardIdentifier,
   pub cyphertext: Vec<u8>,
}

have you tried it with your workaround, does it work?

haven't tried Request

What is the recommended solution to this now? If I want to decode custom events from custom pallets?

This is how we do it: https://github.com/scs/substraTEE-worker/blob/e2619abfff07a2fd9054fcbd4e23879cc0d0eba3/client/src/main.rs#L554
Structs are currently not supported.
You could also import the runtime's events as we do it in the example. But (I believe this is still true) importing the runtime types cannot be done independently of the runtime itself, which leads to a lot of code-bloat you import.


int.addc issue

This came in via a slightly different error reported in IRC. Also related to int.addc.
On OSX with LLVM 3.8, commit f573f95
actor Main
  new create(env: Env) =>
    let a: I8 = 127
    let b: I8 = 127

    var d: I8
    var o: Bool
    (d, o) = a.addc(b)

results in
PromoteIntegerOperand Op #2: t16: ch,glue = CopyToReg t14, Register:i8 %DL, t7:1, t14:1

Do not know how to promote this operator's operand!
UNREACHABLE executed at /private/tmp/llvm3820160316-8183-11u7j5k/llvm-3.8.0.src/lib/CodeGen/SelectionDAG/LegalizeIntegerTypes.cpp:875!
Stack dump:
0.	Running pass 'Function Pass Manager' on module 'loop'.
1.	Running pass 'X86 DAG->DAG Instruction Selection' on function '@I8_box_addc_c2cb'
[1]    26203 illegal hardware instruction  ponyc

with debug and release versions of the compiler.

I get the same result as @SeanTAllen with LLVM 3.8.0 on 64-bit linux.

From IRC report, the error is apparently slightly different when running in VirtualBox but still traceable to addc.

Reduced example:
actor Main
    new create(env: Env) =>
        I8(127).addc(I8(127))

Compiling the generated IR with llc shows the problem:
Intrinsic has incorrect return type!
{ i8, i8 } (i8, i8)* @llvm.sadd.with.overflow.i8

The intrinsic must return a {i8, i1} tuple. This bug was probably introduced by the switch from i1 to i8 for Pony's Bool.

Also, this is catched by passing the --verify flag to ponyc. I suggest we add this flag to CI builds.

Nice work, @Praetonus. That's the problem, and that's exactly when it was introduced.
I think in gencall.c:548 we need to generate the signature differently for intrinsics. Specifically, any Bool in an intrinsic parameter or return type must be converted to an i1. This is both for types and for actual values, which will be a bit fiddly.

PR #946

Looks like @sylvanc has a PR to fix this issue, but in order to prevent regressions, I think we should also be sure to capture this in our "minimal test cases" folder, or in a builtin test before we mark this issue as resolved.

@jemc we could merge my bad_pony test PR. thoughts?

@SeanTAllen I don't like using badpony.cc to test code that is actually good pony code that should compile - I know there is one test like that there already, but they should both either be builtin tests or be captured in our minimal test cases folder to later become integration tests.  In this case, I think this one should be a builtin test.

I've never thought of builtin as a test to make sure we don't have a "won't compile" regression. That seems odd for builtin test.

In general, you're right, the builtin tests probably shouldn't really be about compiling or not compiling.
But in this case, I8.addc is a method on a type in builtin.  If we had full test coverage for all methods on all types in builtin, we would have had a test for addc (and other related methods that were also broken by the Bool change), and we never would have had this regression in the first place.
So, we don't necessarily need to run out and implement full test coverage for builtin right now (that's a big task, albeit worthwhile), but we can at least start by adding tests for these methods, which will be a great way to ensure that they keep working for users that are depending on them to work.


Lockdown versions of projectcalico repos

Update build to specify most recent upstream releases.
<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>

Moving to candidate branch. Leave master as bleeding edge


Linux download broken

https://hyper-updates.now.sh/download/linux It's showing Version not found: latest.

#799 😅


feat(config): update Inovelli LZW31 parameter 52 for FW 1.54

New enumeration for parameter 52 added in version 1.54 of the inovelli lzw31 firmware.
 The individual commit messages aren't checked.


Duplicated whole page when accessing site by your-domain.com/index.html

Have you read the Contributing Guidelines on issues?

[X] I have read the Contributing Guidelines on issues.

Prerequisites

[X] I'm using the latest version of Docusaurus.
[X] I have tried the npm run clear or yarn clear command.
[X] I have tried rm -rf node_modules yarn.lock package-lock.json and re-installing packages.
[X] I have tried creating a repro with https://new.docusaurus.io.
[X] I have read the console error message carefully (if applicable).

Description
Description
Accessing docusaurus build by direct URL domain.com/index.html causes a duplicated page issue.
It looks like shipped docusaurus serve CLI implements a redirect from index.html > /. And it is a specific logic which are not presented in http servers by default.
Background
I'm trying to migrate docusaurus 2.4.3>3.1.1 and faced an issues with merge-request integration in GitLab pipelines. GitLab pipeline produces a docusaurus build artifact which can be accessible by URL's like
some-domain.com/-/repoName/-/jobs/12345/artifacts/build/index.html.
Attempts to omit index.html part, i.e. some-domain.com/-/repoName/-/jobs/12345/artifacts/build/ caused of HTTP404 error. Probably it's possible to have a workaround fix by adjusting redirect rules for my case. Anyway, duplicated content behavior looks like a bug.
Reproducible demo

https://tlzj5p-9095.csb.app/index.html (not sure how long it will be accessible)
https://docusaurus.io/index.html

Steps to reproduce

install from scratch

npx create-docusaurus@latest my-website classic --typescript
cd my-website/
npm install
npm run build

run any http server to serve build over http (I'm using python http server to avoid side-effect redirects which shipped with docusaurus serve)

cd build
python3 -m http.server 9005

visit http://localhost:9005/index.html

Expected behavior
No duplicated content appears
Actual behavior
Whole site is rendered twice. Duplicated top-level menu and page content.

You can also observe this on https://docusaurus.io/index.html

Your environment

Public source code: https://codesandbox.io/p/devbox/beautiful-archimedes-qdeo7
Public site URL:

https://tlzj5p-9095.csb.app/index.html (not sure how long it will be accessible)
https://docusaurus.io/index.html

Docusaurus version used: 3.1.1
Reproduced locally using OSX 14.2.1 + NodeJS 18.18.2

Self-service

[ ] I'd be willing to fix this bug myself.

This has already been discussed here:

https://github.com/facebook/docusaurus/issues/9552
https://github.com/facebook/docusaurus/issues/9871

See my suggestions here: https://github.com/facebook/docusaurus/issues/9871#issuecomment-1954641623
You should serve pages by their canonical URLs. By default, this means serving them without the .html extension.
If this is a problem, there's often a "pretty URL" option on CDNs and static hosts you can turn on, otherwise you can use reverse proxy settings.

Thanks for your response and valuable feedback! Maybe it would be helpful to mention about new behavior at the migration guides. I followed the instructions step by step, but this behavior was unexpected.

Regarding my specific case with GitLab artifacts deployment for merge requests, I couldn't find the ability to implement a redirect on the GitLab side. GitLab integration requires an index.html file for viewing artifacts of MR.
I added a workaround trick to replace the URL on the client side. It looks messy, and I don't like it, but it resolved the issue. Perhaps it can be helpful to someone else:
// docusaurus.config.ts
import type {Config, HtmlTagObject} from '@docusaurus/types';

// see https://github.com/facebook/docusaurus/issues/9916
const redirect = `
(function() {
  var pathname = window.location.pathname;
  if (pathname.endsWith('index.html')) {
    history.replaceState(
      null,
      pathname,
      pathname.replace('index.html', '')
    );
  }
})();
`.replace(/\n/g, '');

const config: Config = {
  // ...
  headTags: [
    {
      tagName: 'script',
      attributes: {
        src: 'data:text/javascript,' + redirect,
      },
    },
  ] satisfies HtmlTagObject[],
};

We'll try to fix this for V4 in https://github.com/facebook/docusaurus/pull/10059