Limit number of releases?

We just added a large repository from our self-hosted GitLab, it has about 1500 tags (yes, it's a rather old project...).
repman wants to create a zip for every tag, which will end in 1500 zips with each 50 mb, resulting in 75 gb of release-zips, of which the first 1499 will never be used ;-)
Is there a way to restrict the number of historical releases that are handled?

Downloading all distributions ahead of time was purposeful (@karniv00l maybe you remember why?). In fact, this use case complicates things a bit. We could make a separate option in the instance settings to help with this:
Download dist on sync - if yes then all distibution will be downloaded during synchronization, if no, only metadata (version history) will be downloaded. Default option will be no.
Let me know what you think.

Would work for me!
What I did on our instance was delete all old zips by hand... I know, not an ideal solution, but at least the storage didn't explode :-D

But this should be configurable during package-add...

Hey @temp, we are almost at the end:

@karniv00l one more thing:

[ ] allow to update this limit

@karniv00l Should #294 limit what's shown on the UI or the latest release?
I think the serialised metadata itself is fine, just the DB stored data.
Syncing 3 releases of https://github.com/symfony/finder.git provides:

(Top screenshot app.repman.io, bottom local). Both added with same URL + package restrictions today.

@giggsey we are limiting number of versions and distributions files, this includes dev and stable releases.I will look into this, thanks

Just upgrade to 1.1.0, and will try the feature now. What happens when I limit the releases afterwards? Will it throw away the obsolete releases?

@temp It will remove old releases

Awesome, thanks! Works fine for me!
... Job just finished, all old releases are gone, and storage was freed. Great work!


Cannot find module 'Matter' in version 0.1.6

I just updated this package to version 0.1.6 in my project and I get this error:
[Error] Error: Cannot find module 'Matter'
    at Function.Module._resolveFilename (module.js:470:15)
    at Function.Module._load (module.js:418:25)
    at Module.require (module.js:498:17)
    at require (internal/module.js:20:19)
    at n (/Users/and_re/Projects/hax/node_modules/matter-collision-events/build/matter-collision-events.min.js:1:83)
    at Object.<anonymous> (/Users/and_re/Projects/hax/node_modules/matter-collision-events/build/matter-collision-events.min.js:1:273)
    at Module._compile (module.js:571:32)
    at Module._extensions..js (module.js:580:10)
    at Object.require.extensions.(anonymous function) [as .js] (/Users/and_re/Projects/hax/node_modules/babel-register/lib/node.js:152:7)
    at Module.load (module.js:488:32)

Everything works fine with 0.1.5 so I think it's because of this commit: https://github.com/dxu/matter-collision-events/commit/53c2d2471341d26616c3c3918f6740eed558e42a

hey @And-re, really sorry about that. I've updated the externals config and rebuilt it, if you update to 0.1.7, your issue should be resolved.
Closing, but please reopen or let me know if you continue to experience the issue!

@dxu Thank you. It seems to work. Maybe you should also update this line with a new version https://github.com/dxu/matter-collision-events/blob/master/src/entry.js#L4

Sigh, yeah you're right. I forgot that matter plugins have their own versions that they're reading off of. Thanks for the reminder, i'll bump it with the next version.

@dxu Ok, no problem. At least everything seems to work.
Do you think that adding a way to disable one of the callbacks temporary (e.g. onCollide) is useful?


Version Dropdown Behavior

The version dropdown which appears on the Libraries, Library, and Releases page should work thusly:

All dropdowns show identical menu contents

All dropdowns include master and develop

When master or develop is chosen on the Releases page, instead of showing a release (which does not exist) we show an informational callout box explaining that there are no master or develop releases, along with a link to navigate them to the current release.

Consider moving master and develop to the bottom.

We want master/develop at the bottom of the page too.

The code is already written and merged to exclude master/develop in all three drop-downs (which all have separate code). This will get us to a good spot before launch.
Including custom ordering for the dropdrowns is extremely untriiial, unfortunately, due to the way Django is designed, so "simply including master/develop at the bottom of the list" is a surprisingly tall order. I tried about 45 different ways to do this, without any avail.
We can likely do it, but it's going to have to wait until after launch. We're in a good spot at the moment! re: #1080.

Excluding master and develop from all the drop-downs is the next best thing, yes.
Just an aside, not actionable, but what do you think of putting an ascii 127 character at the beginning of the words "master" and "develop" so they sort to the bottom? :)

That's a good solution! We can do that through the admin.

I'm not seeing Develop and Master in the dropdown, is this still an issue?


[v2.7] LOG2 + Systemview leads to kernel panic.

When LOG2 + Systemview is set, it will lead to kernel panic.
Board: nucleo_h723zg
Impact
Panic  (showstopper)
Kconfig
CONFIG_LOG=y
CONFIG_PRINTK=y
CONFIG_HEAP_MEM_POOL_SIZE=8192
CONFIG_ASSERT=y
CONFIG_GPIO=y
CONFIG_UART_ASYNC_API=y
CONFIG_LOG_BACKEND_SHOW_COLOR=y
CONFIG_NEWLIB_LIBC=y
CONFIG_LOG2_MODE_DEFERRED=y
CONFIG_SHELL=y
CONFIG_SHELL_CMDS=y
CONFIG_GPIO_SHELL=y
CONFIG_LOG_BACKEND_UART=y
CONFIG_SHELL_LOG_BACKEND=n
CONFIG_NEWLIB_LIBC_FLOAT_PRINTF=y
CONFIG_PWM=y
CONFIG_LOG_TIMESTAMP_64BIT=y
CONFIG_LOG_BACKEND_FORMAT_TIMESTAMP=n
CONFIG_SHELL_LOG_BACKEND=n
CONFIG_STM32_BACKUP_SRAM=y
CONFIG_REBOOT=y
CONFIG_LOG_MODE_OVERFLOW=y
CONFIG_LOG_BUFFER_SIZE=4096
CONFIG_LOG_SPEED=y
CONFIG_CBPRINTF_FP_SUPPORT=y
CONFIG_COUNTER=y
CONFIG_COUNTER_RTC_STM32_CLOCK_LSE=y
CONFIG_COUNTER_RTC_STM32_BACKUP_DOMAIN_RESET=n
CONFIG_COUNTER_RTC_STM32_SAVE_VALUE_BETWEEN_RESETS=y
CONFIG_LOG_FUNC_NAME_PREFIX_DBG=n
CONFIG_SHELL_ARGC_MAX=16
CONFIG_NOCACHE_MEMORY=y
CONFIG_TRACING=y
CONFIG_SEGGER_SYSTEMVIEW=y

Logs and console output
Crash log:
[946685754] <err> os: r0/a1:  0x00000004  r1/a2:  0x000000a8  r2/a3:  0x240038a8
[946685754] <err> os: r3/a4:  0x0000000f r12/ip:  0x00000000 r14/lr:  0x0801c113
[946685754] <err> os:  xpsr:  0x61000000
[946685754] <err> os: Faulting instruction address (r15/pc): 0x0803111c
[946685754] <err> os: >>> ZEPHYR FATAL ERROR 4: Kernel panic on CPU 0
[946685754] <err> os: Current thread: 0x240038a8 (logging)

Info and the culprit
the symbol is set CONFIG_LOG_BACKEND_RTT=y
On Zephyr e4da3e528088a34a9989f5a50e7ed3149d57de92

Something similar got fixed by: https://github.com/zephyrproject-rtos/zephyr/pull/41971
That was the issue: https://github.com/zephyrproject-rtos/zephyr/issues/42164
Are you able to check if your issue is also gone? I will see if i can backport it.

I created backport PR with fix mentioned above. Can you check it? #42346

@manoj153 fix got merged. Can you try it?

Yes, I can, give me 24hrs.

@manoj153 fix got merged. Can you try it?

I have tested. No, it did not solve. Also, please be noted my problem is also with the log redirection.

It is an issue because I still plan to use the UART logging backend only. Now without per-say / intent, the logs are going to RTT. Is that not a bug?

Which the PR https://github.com/zephyrproject-rtos/zephyr/pull/42209 trying to solve, suggest what will be the best way to go about this issue?

I would still like to understand what is this error because presence of additional backend should not result in crash. Also sysview should not conflict with RTT use.
Do you have longer crash log? Is this an assert? Can you collect stack trace?
Have you tried to disable logging for rtt CONFIG_LOG_BACKEND_RTT=n.

I would still like to understand what is this error because presence of additional backend should not result in crash. Also sysview should not conflict with RTT use.
Do you have longer crash log? Is this an assert? Can you collect stack trace?
Have you tried to disable logging for rtt CONFIG_LOG_BACKEND_RTT=n.

[946728134] <err> os: r0/a1:  0x00000004  r1/a2:  0x000000a8  r2/a3:  0x24003838
[946728134] <err> os: r3/a4:  0x00000011 r12/ip:  0x00000000 r14/lr:  0x0801bda1
[946728134] <err> os:  xpsr:  0x61000000
[946728134] <err> os: Faulting instruction address (r15/pc): 0x0802e098
[946728134] <err> os: >>> ZEPHYR FATAL ERROR 4: Kernel panic on CPU 0
[946728134] <err> os: Current thread: 0x24003838 (logging)

When  CONFIG_LOG_BACKEND_RTT is disabled, no crash is observed.

Unfortunately, CONFIG_DEBUG_COREDUMP=y is not doing its work otherwise I would be able to debug further.
Feel free to share if you have any ideas.

@nordic-krch ping


process term for  GO:0008479 queuine tRNA-ribosyltransferase activity

I think we have discussed this before, but I can't find a process term for this MF...anyone know what it should be?

tRNA modification?

@hdrabkin

This is a bit suboptimal because we have in pombe 3 genes  so far in the queuine modification pathway, but you can’t distinguish these easily from the other tRNA modifiers at the process level

I now remember that I asked about this before:
https://github.com/geneontology/go-ontology/issues/11905
tRNA-guanine transglycosylation
for organims which only utilize the dietary queuosine to modify tRNA's
The reaction is
tRNA guanine + queuine = tRNA queuine + guanine.
so it seems that its guanine that is salvaged?
from
PMID: 24911101 abstract
Queuosine (Q) is a modification found at the wobble position of tRNAs with GUN anticodons. Although Q is present in most eukaryotes and bacteria, only bacteria can synthesize Q de novo. Eukaryotes acquire queuine (q), the free base of Q, from diet and/or microflora, making q an important but under-recognized micronutrient for plants, animals, and fungi.
I think we should add a term for this, I just had a problem (again) tracking down the genes in the pathway.
Also, the metabolism term still doesn't have taxon restrictions
so we are still picking up mappings to GO:0008616	queuosine biosynthetic process
http://www.pombase.org/spombe/result/SPAC1687.19c#go-biological_process

note to self
http://www.pombase.org/spombe/result/SPAC589.05c
and Anne's session
https://curation.pombase.org/pombe/curs/466ed4e946c15c3f/ro/

queuosine biosynthestic process
add taxon check
never in  eukaryotes (correct?)
I won't bother to  raise a ticket to correct the erroneous mapping for UniProtKB-KW:KW-0671
because it will be removed via the taxon restriction.

Hi @ValWood - sorry I'm confused between this ticket and the other one, #11905, referenced above. Could you summarise what are the needed changes? Just want to make sure I get this right and don't undo previous work.

Essentially, I need a more specific term then 'tRNA modification' to describe the modification of a tRNA anticodon loop by replacing  guanine with queuonine . The other  linjked ticket is where I asked previously. I settled for tRNA modification, but a more specific term would be useful to group the pathway, so I would still like it.
I'm not sure exactly what it should be called, Harold's @hdrabkin input would be useful here.
GO:0008616 queuosine biosynthetic process needs the taxon restriction never in eukaryotes. Eukaryotes obtain queuosine from dietary sources, but many are annotated to this term.....

Hi,
Can we get this one (looks quick and will enable me to approve a community curated session)
https://curation.pombase.org/pombe/curs/466ed4e946c15c3f/ro
Thanks
Val

note to self curs/466ed4e946c15c3f

Hi @hdrabkin: can you help with this?

" to describe the modification of a tRNA anticodon loop by replacing guanine with queuonine" .
DO you mean something like a class of tRNA modifications based on location of the modification?
like: tRNA anticodon loop modification.
or do you mean to separate the synthesis from scratch  vs the swapping mechanism (euk)

Hi Harold,
Yes I want to describe the modification in Eukaryotes at the process level to be able to differentiate from other types of tRNA modification
so exactly this
" to describe the modification of a tRNA anticodon loop by replacing guanine with queuonine" .
earlier I suggested
tRNA-guanine transglycosylation
for organism which only utilize the dietary queuosine to modify tRNA's
Reaction
tRNA guanine + queuine = tRNA queuine + guanine.
Is this correct?
Thanks
Val

Hi @hdrabkin - if you can confirm which changes need to be made I can implement. Thanks!

Hi @hdrabkin  can you confirm?
I'm trying to close PomBase tracker tickets!
Val

Pinging @hdrabkin for help please!

@hdrabkin I am going to follow you around today until you answer my question ;)

Val
tell me what terms you want; I've commented several times

tRNA-guanine transglycosylation will do fine. Done!
Melanie can you implement ? Thanks!

tRNA-guanine transglycosylation
The modification of a tRNA anticodon loop by replacing guanine with queuonine. Reaction is tRNA guanine + queuine = tRNA queuine + guanine.
PMID: 24911101
is_a GO:0006400 tRNA modification
never in eukaryotes
Please approve or make changes.

Hi Melanie,
That all sounds perfect to me, except it should not have the taxon restriction.
GO:0008616 queuosine biosynthetic process should have "never in eukaryotes".
But this term  is a modification using dietary queuosine (not biosynthesis) and as far as I'm aware is universal.
Val

Ok, so
NTR: tRNA-guanine transglycosylation
The modification of a tRNA anticodon loop by replacing guanine with queuonine. Reaction is tRNA guanine + queuine = tRNA queuine + guanine.
PMID: 24911101
is_a GO:0006400 tRNA modification
And add taxon restriction to GO:0008616 queuosine biosynthetic process: never in eukaryotes
I'll try and get to it tomorrow, thanks for checking!

Done revision 36932.


[BUG] Loading, please wait

Hello,
First of all sorry for my english, i 'm french .
I created a secondary account , grabbed the first pokemon , but I get stuck at "Waiting " . I looked at other similar problems, but there are no other solutions.
Someone would have an idea please ?
console:
server up: null main.html:146 Error getting location, trying second provider PositionError {code: 2, message: "Network location provider at 'https://www.googleapis.com/' : Returned error code 403."} main.html:148 Found location [48.8432,2.5021] by ipinfo.io

OK , after a search on the disk c :, it appears that the requested library is located here:
C: \ Program Files (x86) \ Trend Micro \ OfficeScan Client \ Suspect
So this is the antivirus that has been a problem .
After having copied / pasted the DLL in the appropriate directory , it works !!!


Fix: osde2e flow was missing xml files

junit file name was generated in a way that was problematic. was including command arguments and flags.
This commit solves the issue, and all junit reports will be saved properly.

Regular contributors should join the org to skip this step.

invited you!

/test test-commit

/ok-to-test

/test test-commit

shit, testing of OperatorHub is broken following the release of 1.9.0-beta
...
On Thu, Nov 25, 2021 at 10:17 AM openshift-ci[bot] @.***>
wrote:

@sagidayan https://github.com/sagidayan: The following test failed,
say /retest to rerun all failed tests or /retest-required to rerun all
mandatory failed tests:
Test name Commit Details Required Rerun command
ci/prow/test-commit 02ee01e
https://github.com/openshift-psap/ci-artifacts/commit/02ee01ee35d6b6556473b97ebdec858392e644af
link
https://prow.ci.openshift.org/view/gs/origin-ci-test/pr-logs/pull/openshift-psap_ci-artifacts/288/pull-ci-openshift-psap-ci-artifacts-master-test-commit/1463791784176914432
true /test test-commit
Full PR test history
https://prow.ci.openshift.org/pr-history?org=openshift-psap&repo=ci-artifacts&pr=288.
Your PR dashboard
https://prow.ci.openshift.org/pr?query=is:pr+state:open+author:sagidayan
.
Instructions for interacting with me using PR comments are available here
https://git.k8s.io/community/contributors/guide/pull-requests.md. If
you have questions or suggestions related to my behavior, please file an
issue against the kubernetes/test-infra
https://github.com/kubernetes/test-infra/issues/new?title=Prow issue:
repository. I understand the commands that are listed here
https://go.k8s.io/bot-commands.
—
You are receiving this because your review was requested.
Reply to this email directly, view it on GitHub
https://github.com/openshift-psap/ci-artifacts/pull/288#issuecomment-979002386,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABZVQIXJFWVDO7FV3G275DDUNX5K7ANCNFSM5IX5WP2A
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

Yeah, I noticed :)

trying a fix in https://github.com/openshift-psap/ci-artifacts/pull/289
On Thu, Nov 25, 2021 at 10:28 AM Sagi Dayan @.***>
wrote:

Yeah, I noticed :)
—
You are receiving this because your review was requested.
Reply to this email directly, view it on GitHub
https://github.com/openshift-psap/ci-artifacts/pull/288#issuecomment-979016866,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABZVQISO3VK5DMO2F35DEWTUNX6VDANCNFSM5IX5WP2A
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

fixes have been merged
/test test-commit

/test test-commit

aargl
Nothing to test in 2ac8a160a271a2797a8c6336a7adf399b0a86fea.

/test test-commit

looks good:
https://gcsweb-ci.apps.ci.l2s4.p1.openshiftapps.com/gcs/origin-ci-test/pr-logs/pull/openshift-psap_ci-artifacts/288/pull-ci-openshift-psap-ci-artifacts-master-test-commit/1464558949767319552/artifacts/test-commit/test-commit/artifacts/recipes_run_osde2e_gpu_addon/junit_gpu_operator_run_gpu_burn.xml
https://gcsweb-ci.apps.ci.l2s4.p1.openshiftapps.com/gcs/origin-ci-test/pr-logs/pull/openshift-psap_ci-artifacts/288/pull-ci-openshift-psap-ci-artifacts-master-test-commit/1464558949767319552/artifacts/test-commit/test-commit/artifacts/recipes_run_osde2e_gpu_addon/junit_gpu_operator_wait_deployment.xml

for observability, you may want to generate these two files:
https://gcsweb-ci.apps.ci.l2s4.p1.openshiftapps.com/gcs/origin-ci-test/logs/periodic-ci-openshift-psap-ci-artifacts-release-4.9-gpu-operator-e2e-master/1464368446815866880/artifacts/gpu-operator-e2e-master/nightly/artifacts/ci_artifact.git_version
https://gcsweb-ci.apps.ci.l2s4.p1.openshiftapps.com/gcs/origin-ci-test/logs/periodic-ci-openshift-psap-ci-artifacts-release-4.9-gpu-operator-e2e-master/1464368446815866880/artifacts/gpu-operator-e2e-master/nightly/artifacts/ocp.version
so that we can know quickly which version of ci-artifacts and OpenShift was used in the test
It can be displayed in the ci-dashboard

@sagidayan maybe as part of this PR you can also move the artifacts into a dedicated directory, to avoid this mix:
https://gcsweb-ci.apps.ci.l2s4.p1.openshiftapps.com/gcs/origin-ci-test/logs/osde2e-stage-aws-addon-gpu-operator/1463477578424651776/artifacts/install/
once this is done and merged, I can try integrating these nightlies into a new ci-dashboard

/test test-commit

/test test-commit

/hold
/test test-commit

thanks @sagidayan
/lgtm
/override ci/prow/gpu-operator-e2e
/approve

/unhold


Consider FIPS support on macOS / darwin

As of writing, we've been asked about macOS FIPS support, but we haven't been asked to provide it. I'm opening this issue to keep track of any interest.
My reading of the internal Microsoft crypto policy is that using OpenSSL on macOS would be considered noncompliant. Instead, the platform-provided CommonCrypto, Security Transforms, and Secure Transport APIs must be used. If I'm wrong, or for anyone looking into this from outside Microsoft, maybe this isn't an issue. In that case, https://github.com/golang-fips/openssl/pull/98 is an effort to make the OpenSSL backend work on macOS (and Windows), although we haven't determined for certain that it will work in microsoft/go releases.
I haven't looked into any of those macOS-provided libraries to see what kind of effort it would take to make a backend for it.

Our team would love support for building FIPS compliant darwin binaries; we're building Go binaries that are part of a toolchain that is otherwise using FIPS compliant .NET binaries for Windows, Linux, and MacOS, so ideally we'd be able to produce FIPS compliant Go binaries for the same set of OS targets.

@danegsta supporting macOS is currently not in our backlog. Please send an email to @gdams, our manager, explaining your business case so we can prioritize this effort and see how it can be funded.

My reading of the internal Microsoft crypto policy is that using OpenSSL on macOS would be considered noncompliant. Instead, the platform-provided CommonCrypto, Security Transforms, and Secure Transport APIs must be used.

We got this clarified by someone responsible for the policy--they don't see it this way, which is great news. It's ok to use OpenSSL on macOS, and there isn't any need to rule out the possibility of using macOS native crypto: OpenSSL is always ok, as long as the OpenSSL-specific requirements are met.
Getting this enabled in Microsoft Go will take some work (in addition to tweaking the patch, we would probably want to set up macOS builders to continuously test it), but the path is clear.

@dagood  Thanks for the update! I’m interested in learning more about any progress related to the recent discussions on supporting Microsoft Go on macOS. We have a similar use case where we need to create a FIPS-compliant Go binary for macOS.
Any updates or insights you can provide would be greatly appreciated!

We have a draft PR for a macOS builder at https://github.com/microsoft/go/pull/1279 that we plan to keep working on, but no progress yet on a FIPS backend. I suspect though that the work to be done on that will be:

Modify patches/0004-Add-OpenSSL-crypto-backend.patch to allow darwin along with linux in the relevant build tags.
Regenerate patches/0007-Add-backend-code-gen.patch to update the error messages so go build will let you target darwin without stopping your build.
Modify patches/0001-Add-systemcrypto-GOEXPERIMENT.patch so that systemcrypto uses opensslcrypto on darwin.

I don't think this step is truly necessary to build your own binary if you're comfortable building your own copy of Go and assign GOEXPERIMENT=opensslcrypto directly rather than use the systemcrypto alias. We'll want to have systemcrypto working in official/signed Microsoft Go builds, though.

(We haven't looked at CommonCrypto/etc., and using OpenSSL on macOS is still the plan for this issue.)

@dagood we're attempting to follow the provided instructions to generate a FIPS build on macOS, but we're encountering errors when applying the patches. Could you please provide more detailed instructions? Alternatively, if you already have a version that supports Darwin, we'd be happy to test it.
Thanks

@dagood, in continue to my previous message, I didn't generate patches/0007-Add-backend-code-gen.patch alone, but run the build after updating patches/0004-Add-OpenSSL-crypto-backend.patch with (darwin || linux) where linux appeared in the code.
here is the build output:
sudo pwsh eng/run.ps1 build -refresh
Building MacOS darwin arm64...
In ' /Users/devuser/dev/fips/go/eng/_util', building './cmd/build' -> /Users/devuser/dev/fips/go/eng/artifacts/toolbin/build
Built 'build'. Running from repo root...

Sorry for the quiet here. I've opened https://github.com/microsoft/go/issues/1368 for supporting producing (signed) macOS prebuilt Microsoft Go toolsets, which will make it easier to then work on the FIPS side of things. I'm going to take a stab at getting https://github.com/microsoft/go/pull/1279 through. I don't have direct access to a machine running macOS, but I'm hoping I can get it through CI to the point where others can keep this work going.
Apologies also for the trouble with the patch files... seeing https://github.com/microsoft/go/pull/1362, it looks like you might be editing the .patch files directly rather than using git commands to apply/format/extract them. We have a tool git-go-patch that we use to maintain them. I noticed that we don't link to it anywhere from the microsoft/go repo--I'll fix that.

An important update:
We (the Go team) have been experimenting with several different crypto backends for macOS and intend to go with the CommonCrypto (falling back to CryptoKit where required).
This is still very early days and we don't anticipate having a GA binary for a while but we would be interested in having 1P customers testing out some EA binaries for us (if this is of interest then please drop me an email)
The macOS crypto bindings have been open sourced here: https://github.com/microsoft/go-crypto-darwin and we actively encourage contributions. As with our other backends, it may not be possible to support every single algorithm so we'll fall back to the Go native crypto where necessary.


DateRangePicker initialises itself outside viewport width

As you can seen in the screenshot below, the DateRangePicker component seems to (sometimes) initalise itself without regard to viewport width. In my case, it is actually placing itself in such a way that my entire page breaks (the dark edge is the edge of my viewport, were it not to be expanded by the DateRangePicker).
No, I am not initialising it in a hidden component or anything like that - I am using it completely as intended/expected.
This happens even at screen widths as high as 1000px, if the input is placed close enough to the edge.
In fact, this impacts my entire page even when the datepicker is closed, because of the use of visibility: hidden instead of display: none.
Sorry for not including a live example, but I haven't yet found a decent sandbox to use for npm/es6 stuff.

This is related to https://github.com/airbnb/react-dates/issues/71 (although that solution I think is only a stopgap).
Ideally we should have a totes responsive component that can see if it is overflowing and adjust itself accordingly. That's not there yet (soon! well, relatively). In the mean time you can override the position on the .DateRangePicker__picker (originally set at https://github.com/airbnb/react-dates/blob/master/css/DateRangePicker.scss#L24) in your own stylesheet. You'll probably want to override the :before and :after positioning styles as well (https://github.com/airbnb/react-dates/blob/master/css/DateRangePicker.scss#L56:L86).
https://github.com/airbnb/react-dates/blob/master/css/DateRangePicker.scss#L56:L86).
This is a bummer:

In fact, this impacts my entire page even when the datepicker is closed, because of the use of visibility: hidden instead of display: none.

We use visibility over display right now so that we can do a height calculation while the datepicker is still closed but the DOM has been rendered.  I want to move away from this... but I think that will have to wait until we manage to get a react-with-styles implementation out (which is tricky, but in the works).

I don't think this the sort of thing one could fix with CSS overrides, and to be honest this library is not production ready until this is addressed.
Lots and lots of similar kinds of components exist out in the wild that set themselves appropriately!

Fixed in v3.3.2


Logo exploration

Some exploration on logo 🤗
Title

Inline

Final version. Thanks! @vinciarts

It's awesome! : )) :tada:


IBM Blockchain Platform storage limitation

Hello,
Based on the documentation here to deploy IBM Blockchain Platform.
Persistent storage is required. Host-local storage volumes are not supported.
My question is, are there any type of limitation on any other types of storage ?
Best

Please find the storage requirements and limitations here:
OpenShift: https://cloud.ibm.com/docs/blockchain-sw-25?topic=blockchain-sw-25-deploy-ocp-getting-started#deploy-ocp-storage
Kubernetes: https://cloud.ibm.com/docs/blockchain-sw-25?topic=blockchain-sw-25-deploy-k8#deploy-k8-storage


[Feature Request] Support live statistics in progress bar

Right now the progress bar that shows when one runs inspect eval does not show any running statistics. Sometimes a small fraction of evals can take twice as long or longer than the rest of evals, and it would be convenient in these scenarios to be able to view a running statistic to know how well the run is doing.

Note that we very soon be working on improved live task display (including both statistics as well as drilling down into what's happening w/ individual samples)

Resolved here: https://github.com/UKGovernmentBEIS/inspect_ai/pull/953
We currently show the first metric of the first scorer in the progress bar, but will very soon be adding support for optionally viewing all of the scorers/metrics/reducers for a given task.

Support for showing all scores here: https://github.com/UKGovernmentBEIS/inspect_ai/pull/982


Feature: ensureTorunamentEntered option in submitScoreForPlayer

It would be nice that the submitScoreForPlayerfunction would support the flag ensureTorunamentEntered to automatically enter a tournament if the user has not. This feature already exists for a match as ensureMatchEntered but is not so useful if the user is not in a tournament.
Example usage:
    await client.submitScoreForPlayer({
        matchId: GAMEON.matchId,
        submitScoreRequest: { score },
        player: alexaPlayer,
        ensureMatchEntered: true,
        ensureTournamentEntered: true,
    });

You would need to specify the tournamentId in the payload in the same way the matchId is submitted.
Please feel free to submit a PR for this.
https://github.com/alexa-games/skills-gameon-sdk-js/blob/develop/CONTRIBUTING.md


Reconnect terminals at startup instead of waiting for the terminal view to become visible

The change in terminalInstance.ts is because the first terminal is marked as visible even when the terminal view is not visible, and when we force a resize, that xterm thing has not been initialized yet.
I'm not 100% happy with this change, the exchange between reconnectRemoteTerminals and initializeTerminals is a little weird, but the logic is like this

Start reconnecting from the TerminalService constructor
If the terminal view becomes visible, initializeTerminals is called
In initializeTerminals, the view is now visible, so create the placeholder tempEmptyTab. This will get the Starting... label, and if focus() is called on TerminalView, it will get a flag set so that it will focus any terminal instance it gets
Then wait on the reconnection promise
reconnectRemoteTerminals will use the empty tab if it was set up
Once the reconnection promise is complete, create a new default terminal if there are no terminals, or clean up the empty tab if it was not needed

It would be cleaner to not have to create an empty tab
Test matrix: (Has/doesn't have terminals to reconnect to) x (terminal view becomes visible while connecting/after connection) x (Another terminal is/isn't added from somewhere else)

Pushed a change @Tyriar, is this like what you had in mind? The focus awkwardness remains and moves to the terminal service

Funny, github updated to show this comment without me refreshing the page, but I had to refresh the page to see the new review comment

The change looks good to me, but I removed myself as a reviewer since Daniel should be happy with the change :+1:.

There's a lot going on here, let's defer it to November


fix: avoid numbered top-level domain

Description

The regex has been modified to prevent numbers from being included in the top-level domain.
In the Word component, the URL validation for the URL type has been removed.
In the Word component, the use of UUID in the key has been removed.

ticket: CLNP-2262

Note: In the current change, it does not include domains consisting of English characters and numbers like Unicode. (For example, the domain 청와대.한국 is represented as xn--vk1b187a8ue.xn--3e0b707e)
In the future, it should be modified to detect only top-level domains composed solely of numbers.


Parse error in the helpers.php (PHP version 5.6.3)

Parse error: syntax error, unexpected 'yield' (T_YIELD), expecting '(' in ..\tinyissue-master\app\laravel\helpers.php on line 563

This is an issue with Laravel 3 as it is not compatible with PHP5.5 and above. From php5.5 'yield' is a reserved keyword. I have upgraded Tiny Issue to use Laravel 5. It is still working in progress, but most of the features are working fine.
Repository: https://github.com/satrun77/tinyissue

@satrun77 Great repository! Did you see mine forked repository? It's based on Laravel 4.2.
Are you planning to convert all your layouts to bootstrap? If so, I've make a beginning with it in this branch. (It's still based on Laravel 4.2 but I can try to fork your repository and make a bootstrap branch in that fork)

Thanks @GerritHoekstra. Nice work on the migration to Laravel 4.2.
My fork is based on bootstrap that is included in Laravel 5. Forms and layout is bootstrap. Top navigation is not bootstrap.
My plan is to migrate:

Everything in Tiny Issue v1.3.1 (Completed).
All features that I have developed for v1.3.1. This includes:

Change issue project. (Completed)
Set default assignee.  (Completed)
Filter issue number is content into a url to the issue. (Completed)
Issue quote (Completed)
Project notes (Completed)

Issue tags (Pull request from original repo).  (Completed)
Add tests (codeception.com) (In progress)
Setup build and automated test with Travis CI

To fix the issue in a simple way, just open helpers.php at line 563 and rename the yeld method to _yeld. :)


Update OverrideServiceImpl.java

fix the bug: cant enable override

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.


zOS support

It would be great if Feilong supported zOS lifecycle, where zOS is 2nd level zVM guest:

IPL zOS,
Stop zOS
invoke MVS commands (D IPLINFO, ...) and capture their response

My understanding is that Feilong runs a zlinux under zVM, what it is resource consumption of this image (average CPU utilization, Memory, Disk).  In my case I would ideally avoid extra VM with zlinux to reduce memory and CPU resources because I'm running zVM under zD&T (zPDT)
I already have a POC version implemented off the Feilong project here is an architecture diagram:

In my case I can have many instances of zVM each representing a parallel sysplex, that could be provisioned by a developer, therefore I prefer to avoid zlinux on each zVM and instead have the majority of components off the zVM.
Ideally, I would like to combine LXC and observer into a single VM to minimize overhead of infrastructure
LXC is used to send commands such as:  Q NAMES, CP SEND CP ZOS1 CP IPL ..., CP SEND CP ZOS1 VINPUT VMSG D IPLINFO, CP SET OBSERVER ...
OBSERVER - IUCV2Socket - capture console messages sent from zOS vms and forward them via a network socket to a CLI that runs on linux, POC is based on

Managing z/OS guests on z/VM with IBM Operations Manager for z/VM  interesting article describing how to interact with zOS master console using CP VINPUT VMSG xxx commands and how to capture responses using CP SET OBSERVER ...
Sample REXX program to listen on a TCPIP socket
Sample REXX program to trap IUCV messages using WAKEUP

currently, feilong rely on the IUCV channel to communicate with SMAPI request servers, so it has to stay in the same z/VM system.
We don't have performance data like cpu consumption of feilong services, but according to our daily use, it's relatively a light weight service

from the picture I think you are proposing LXC and other CMS related components to hosting feilong like Linux
but provisioning z/OS Is not hard depend on the LXC related item, is that correct?

correct, that's what I am expecting about z/OS itself ,we can talk about at next meeting
if you can record/show the env that you have now it might be easier to understand
Thanks~

From: Vitek Vlcek @.>
Sent: Thursday, October 12, 2023 8:14 PM
To: openmainframeproject/feilong @.>
Cc: CHEN JI @.>; Comment @.>
Subject: [EXTERNAL] Re: [openmainframeproject/feilong] zOS support (Issue #751)
LXC is used to issue CP commands Q NAMES, CP SEND CP ZOS1 CP IPL .. . , CP SEND CP ZOS1 VINPUT VMSG D IPLINFO, CP SET OBSERVER .. . from a linux VM running off the zVM. In this scenario, Feilong is not running in zLinux under zVM. This picture
ZjQcmQRYFpfptBannerStart
This Message Is From an External Sender
This message came from outside your organization.
https://us-phishalarm-ewt.proofpoint.com/EWT/v1/PjiDSg!2A-qKrTUgROxke9zkqrvJs6yI8Q_iH1OjKqdnawhJKhS7JHeoCcjUMUMy5lLDLq1OvNLpVJdCC5YjG9howfYQK8Qi2FTSyHSfw0TABjigR_bAD65C6hd$
Report Suspicious
ZjQcmQRYFpfptBannerEnd
LXC is used to issue CP commands Q NAMES, CP SEND CP ZOS1 CP IPL ..., CP SEND CP ZOS1 VINPUT VMSG D IPLINFO, CP SET OBSERVER ... from a linux VM running off the zVM. In this scenario, Feilong is not running in zLinux under zVM. This picture covers only the start/stop scenario of zOS. Provisioning of zOS is out of scope.
We could discuss late what it means to provision zOS it is not that easy like in case of linux, provisioning involves multiples volumes.
—
Reply to this email directly, view it on GitHubhttps://github.com/openmainframeproject/feilong/issues/751#issuecomment-1759487911, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AEGMXPKGWOASCMU3KHFDKUDX67NJPANCNFSM6AAAAAA4WYVSIA.
You are receiving this because you commented.Message ID: @.***>

@VitekVlcek-Broadcom We mentioned this issue during today's Feilong TSC meeting.  I see @jichenjc suggested demoing the env.  Is this something you would like to do for the January Feilong meeting?

Hi Vitek, I have seen with interest your demo of your first PoC relying on LXC.
My observations:

I was first confused by "LXC" name, as it is a well known containers system under Linux :smile_cat:
What you do with LXC could also be done via s3270, the "scripting" version of x3270 terminal emulator: send CP commands, analyze output...
I don't think you need a Ubuntu host on your z/VM... You can probably directly use zthin from an outside host. If you do that, you lose the commodities of the REST server and are in direct grip with SMAPI.
I don't know if putting the Ubuntu host outside of the s/390 would work too.
Both suggestions above would fail if zthin needs IUCV to communicate with SMAPI. I think it's not the case and TCP/IP sockets are enough, but we are at the limits of my knowledge.
Your approach is very different in essence from what feilong does: you use CP command to start and stop an existing zVM guest, while feilong deploys a new VM from a preexisting image, by manipulating the z/VM directory via SMAPI.
The fact that the approach is different does not mean you cannot do the same thing - just in a different way :smile_cat: .

I hope that helps.

Hello Eric,

I was first confused by "LXC" name, as it is a well known containers system under Linux 😸

I believe this LXC - https://www.vm.ibm.com/download/packages/descript.cgi?LXC - predates the Linux LXC technology that you are thinking of.

I am quite confused whether zthin component of Feilong needs IUCV to communicate with SMAPI. @bjhuangr says above it does, but 1) I think SMAPI can also be driven with IP sockets, and 2) we accidentally got Feilong to work while we forgot IUCV settings 🤔 . If IUCV is not mandatory, then you could put zthin or even a full Feilong outside of the s390 host.

IUCV is used for Linux VM communication, SMAPI is for VM management through z/VM DIRMAINT/CPU
SMAPI is pure socket thing and feilong covers both SMAPI and IUCV for different purpose
you can create VM USER DIRECT through DIRAMINT but it might not fit feilong whole purpose which covers Linux part as well

However, I do not know whether Feilong assumes it is a Linux system that is being deployed. For example you can access the IP address of the guest, I am not sure how this is achieved and if Feilong needs to access the guest OS to achieve that.

the access to IP address is beyong feilong scope, we used to use ssh to do management but this means this need management entity need ssh to VM which is some case it's security concern and sometimes the network is not reachable at all, so we use IUCV instead

If I were you, I would just try to start and stop an existing zOS system with Feilong, to see if that works.

if it's z/VM guest, I see no reason why those 2 are not supported (as it will be z/VM Start/Stop through Image_Activate and Deactivate API which is pure SMAPI thing which interpret as CP command only)

Thanks for the clarifications @jichenjc. The schema at https://cloudlib4zvm.readthedocs.io/en/latest/architecture.html#internal-architecture had mislead me, I suggest in another issue to fix it.
To summarize this discussion:

handling z/OS VMs from Feilong would work as long as we use only the SMAPI part. That includes starting and stopping VMs.
to do the same thing with z/OS as Feilong does with Linux, one would need to implement the equivalent functionality. The most "Feilongish" way would be IUCV, but you could continue with LXC, I guess
but if you are to continue with LXC, is there still a point to rely on Feilong's SMAPI part? Yes, if provisioning becomes part of your scope.

These are just personal thoughts. I do not represent the Feilong project in any way.

Thanks @Bischoff for the summary, I would love to use SMAPI instead of LXC however. LXC is used for:

start/stop zVM - there should be API calls (based on your reply)
send a command to a VM console eg: Q NAMES, CP SEND CP ZOS1 CP IPL ..., CP SEND CP ZOS1 VINPUT VMSG D IPLINFO, CP SET OBSERVER ... - is there a SMAPI endpoint for it?

start/stop zVM - there should be API calls (based on your reply)

yes, this is easier , Image_Activate/Deactivate I shared above

send a command to a VM console eg: Q NAMES, CP SEND CP ZOS1 CP IPL ..., CP SEND CP ZOS1 VINPUT VMSG D IPLINFO, CP SET OBSERVER ... - is there a SMAPI endpoint for it?

we can create a customerized API doing so
https://www.ibm.com/docs/en/zvm/7.3?topic=programming-creating-custom-apis
we used to have a API that can do so and it trigger some security concern as if something like
CP <input command> then it's able to do everything which we don't want


New Function Request

Please add a container function to be able to do
Get-NavAppRuntimePackage
from the container.
https://docs.microsoft.com/en-us/powershell/module/microsoft.dynamics.nav.apps.management/get-navappruntimepackage?view=businesscentral-ps

Shipped in <IP_ADDRESS>


With nuxt-i18n upgrade from v6.3 to v6.13.12, other plugins are no longer available

Version
nuxt-i18n: 6.13.12
nuxt: 2.14.3
Nuxt configuration

[x] universal

Nuxt-i18n configuration
i18n: {
      locales,
      // defaultLocale: 'fra-fr',
      lazy: true,
      langDir: 'services/lang/',
      seo: false,

      // Prefix all url, including the default one
      strategy: 'prefix',

      // Disable redirection base on browser, we will implement our own strategy
      detectBrowserLanguage: 'false',

      // Localized routes
      parsePages: false,
      pages: {
        'magazine/articles/_article': {
          'deu-de': '/magazine/artikel/:article',
          'esp-es': '/magazine/articulos/:article',
          'ita-it': '/magazine/articoli/:article',
          'por-br': '/magazine/artigulos/:article',
        },
        'products/_id': {
          'esp-es': '/productos/:id',
          'fra-fr': '/produits/:id',
          'fra-ca': '/produits/:id',
          'por-br': '/produtos/:id',
          'deu-de': '/produkte/:id',
          'ita-it': '/prodotti/:id',
        },
        cart: {
          'fra-fr': '/selection',
          'fra-ca': '/selection',
          'deu-de': '/kaufen',
          'esp-es': '/cesta',
          'ita-it': '/acquistare',
          'por-br': '/carrinho',
        },
        checkout: {
          'fra-fr': '/finaliser-ma-commande',
          'fra-ca': '/finaliser-ma-commande',
          'deu-de': '/zur-kasse',
          'esp-es': '/continuar',
          'por-br': '/finalizar-a-compra',
        },
      },
      // TO BE CHANGED LATER FOR LOCAL CHOICE PAGE
      // rootRedirect: 'fra-fr',

      // http://kazupon.github.io/vue-i18n/api/#silenttranslationwarn
      // http://kazupon.github.io/vue-i18n/api/#silentfallbackwarn
      ...(process.env.LVF_I18N_SILENT_WARN || process.env.NODE_ENV === 'production'
        ? {
            vueI18n: { silentTranslationWarn: true, silentFallbackWarn: true },
          }
        : {}),
    }

Steps to reproduce

configure the lazy loading of the lang files
implement axios to get a lang file
write and load a custom plugin with nuxt module (ex: $logger with consola), use it to log in lang requester service
load nuxt-i18n module after the logger "plugin" module
run your nuxt application, open an url, nuxt-i18n will try to request a lang file
$logger plugin is not found in context

// nuxt.config.js
{
    modules: ['~/modules/logger', ['nuxt-i18n', {...}]]
}

// src/modules/logger.js
export default function module() {
  this.addPlugin(path.resolve(__dirname, '../plugins/nuxt/logger.js'));
}

// src/services/lang/ara-ae.js
const cli = axios.create()

export default ctx => {
    ctx.app.$logger.info('Request for ara-ae')
    return cli.get('https://www.mydomain.com/ara-ae.json', {...})
}

What is Expected?

What is actually happening?

 ERROR  Cannot read property 'info' of undefined

Regards

problem is in 6.13.5:
until 6.13.4 there is context.$axios
from 6.13.5 there is NO context.$axios
we need to load translations with $axios, is there any chance to have plugins loaded in this time? ($axios mostly)

I know what the issue is. The plugin system in nuxt is pretty annoying in being a bit unspecified regarding the order...

Fix released in v6.15.1

This "fix" break the documented behavior described here: https://i18n.nuxtjs.org/api#extension-of-nuxt-context.
In particular: you can no longer refer to app.i18n.locale inside plugins.
Simple example:
import moment from 'moment'

export default ({ app }, inject) => {
  moment.locale(app.i18n.locale)
  inject('moment', moment)
}

There is no way to make the order work for all cases.
If there are only modules at play then the plugin registration order is opposite of the module registration order (since adding plugin from a module adds it at the start of the list).
If there are also loose plugins mixed into the picture then I'm not sure what is currently the order but judging from your comment, the "loose" plugins are registered before modules so run first.
In your case, you should probably create a simple module that adds the plugin using this.addPlugin. Then you can control the order by changing the order modules that are registered.
This was was to make the order predictable within modules registration.

In your case, you should probably create a simple module that adds the plugin using this.addPlugin. Then you can control the order by changing the order modules are registered.

In fact this is exactly what is done in my case. I have the code I gave as example in a plugin loaded by module and that module is listed after nuxt-i18n in nuxt.config.js / modules...

Should be listed before nuxt-i18n. As I've said, plugins are registered in opposite order than modules themselves. Blame Nuxt.

Thanks for pointing that out since I did not get it the first time. :)
With reversed module order all is fine!


feat: update context to take lists and rename context field

Issue #, if available:
Description of changes:

Uses Lists of strings instead of string delimiters for multiple contexts since the Ray dataset issue with Lists has been resolved.
Rename the context field to context as we are merging target_context and retrieved_context for RAG evaluations.

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Related CR: https://github.com/aws/fmeval/pull/266


Guidelines for MCs

Introducing and thanking speakers
Timekeeping and signaling to speakers
Helping wrangle questions (e.g. "Please make sure your question ends in a question mark, and isn't being asked primarily to make a point")

Thanks for opening this and the other issue, I'll also open a meta issue for materials later.


automatic refresh to get strikethrough / update colors

When pressing alt-d to complete a task I do not immediately get a refresh for the strike through formatting
When moving tasks with alt-up/down the colors will be off. E.g. green project will become white. the blue color for the tag will color the whole line or similar.

Workaround:

toggle sort of tags will make a refresh of the page, and colors will be ok again. I don't know what else will force a refresh.

Expected behaviour:

I would expect this to work automatically without any manual update

Is this issue about editor or webview? In any case, can't reproduce:

Cool. That inspired me to reinstall the plugin. Another markdown plugin was told to reload, so I uninstalled that as well as a test. Then installed again. Now it works! 👏 But I have another icon than you. (btw I tried 4-5 of these todo plugins before I settled on this, dunno if that is relevant. they are uninstalled now, at least it looks so). Thanks!


Include more fallbacks

Hi,
I think that <Find> and <Filter> could use the same fallback props as <Every>.
It is a common pattern to filter or to try to find an item in a collection and then handle the case where nothing has been found. This would prevent introducing boilerplate code like res.length !== 0
What do you think?

Well nevermind hadn't see this before https://joshwcomeau.github.io/react-collection-helpers/demo/dist/ !

Hey Quentin!
Thanks for the feedback. Yeah, I think I prefer doing it this way, it feels a little more explicit. That said, there's really no reason for Find and Filter to support the same fallback prop that Every does.
In fact, we could even add it to <BaseCollectionHelper>, that way it's available to all helpers. Some of them (eg. <Sort>) would never actually need it, but it would just be ignored in those cases.
If it interests you, I'd accept a PR moving it there :)


fix(chat): fix duplicated disabled folder with a dot at the end after import of chat/prompt(Issue #1236)

Description:
Fix duplicated disabled folder with a dot at the end after import of chat/prompt
Issues:

Issue #1236

Checklist:

[x] the pull request name complies with Conventional Commits
[x] the pull request name starts with fix(<scope>):, feat(<scope>):, feature(<scope>):, chore(<scope>):, hotfix(<scope>): or e2e(<scope>):. If contains breaking changes then the pull request name must start with fix(<scope>)!:, feat(<scope>)!:, feature(<scope>)!:, chore(<scope>)!:, hotfix(<scope>)!: or e2e(<scope>)!: where <scope> is name of affected project: chat, chat-e2e, overlay, shared, sandbox-overlay, etc.
[x] the pull request name ends with (Issue #<TICKET_ID>) (comma-separated list of issues)
[x] I confirm that do not share any confidential information like API keys or any other secrets and private URLs

/deploy-review

/deploy-review


Menu level 2

I'm submitting a ...
[ X ] bug report
[ ] feature request
[ ] question about the decisions made in the repository

Do you want to request a feature or report a bug?

What is the current behavior?

When adding children to a child in the pages.menu.ts, it behaves very weirdly in the UI.
https://i.imgsafe.org/9e5c297eb7.png
In case, the image is not clear enough, this is the state on my menus:

All devices

1809......

Overview
Statistics
Inspect data
- sub menu 1
- sub menu 2

In this example, The "1809...." menu is a child of "All devices" and the parent of "Inspect data".
If I click on "Inspect data", the arrow of "1809..." will move and not the one of "Inspect data".
I look up the current code in baMenuItem and it seems that ng2-admin can only handle 2 levels of embedded menus or am I mistaken ?

What is the expected behavior?
It should behave the same way as with 2 levels menu (Like in the demo)

What is the motivation / use case for changing the behavior?

Please tell us about your environment:

Angular version: 2.0.0-rc.X
Browser: [all | Chrome XX | Firefox XX | IE XX | Safari XX | Mobile Chrome XX | Android X.X Web Browser | iOS XX Safari | iOS XX UIWebView | iOS XX WKWebView ]

Well I just changed the structure of my menu to "solve" this so I guess it can be closed

@terrymainvis  i also met the same question, how did u solved it ? could u show me your code structure? thank u very much


TypeScript Refactoring will cause ng serve produce error

Bug Report or Feature Request (mark with an x)
- [x] bug report -> please search issues before submitting
- [ ] feature request

Command (mark with an x)
- [ ] new
- [ ] build
- [x] serve
- [ ] test
- [ ] e2e
- [ ] generate
- [ ] add
- [ ] update
- [ ] lint
- [ ] xi18n
- [ ] run
- [ ] config
- [ ] help
- [ ] version
- [ ] doc

Versions
Windows 10
G:\Projects\demo2>node --version
v10.11.0

G:\Projects\demo2>npm --version
6.4.1

G:\Projects\demo2>ng --version

     _                      _                 ____ _     ___
    / \   _ __   __ _ _   _| | __ _ _ __     / ___| |   |_ _|
   / △ \ | '_ \ / _` | | | | |/ _` | '__|   | |   | |    | |
  / ___ \| | | | (_| | |_| | | (_| | |      | |___| |___ | |
 /_/   \_\_| |_|\__, |\__,_|_|\__,_|_|       \____|_____|___|
                |___/

Angular CLI: 6.2.5
Node: 10.11.0
OS: win32 x64
Angular: 6.1.10
... animations, common, compiler, compiler-cli, core, forms
... http, language-service, platform-browser
... platform-browser-dynamic, router

Package                           Version
-----------------------------------------------------------
@angular-devkit/architect         0.8.5
@angular-devkit/build-angular     0.8.5
@angular-devkit/build-optimizer   0.8.5
@angular-devkit/build-webpack     0.8.5
@angular-devkit/core              0.8.5
@angular-devkit/schematics        0.8.5
@angular/cli                      6.2.5
@ngtools/webpack                  6.2.5
@schematics/angular               0.8.5
@schematics/update                0.8.5
rxjs                              6.2.2
typescript                        2.9.2
webpack                           4.20.2

Repro steps

Clone project
git clone https://github.com/coolrare/ng-181017.git -b ng-serve-bug`
cd ng-181017
code .

Open src\app\layout\layout.component.ts file

Make a selection on whole LayoutInit function.

Hit Ctrl-. to trigger Quick Action

Hit Move to a new file

Save All

For now, you will see the ng serve will produce the following error:
ERROR in Source file not found: '/G/Projects/demo2/src/app/layout/LayoutInit.ts'.

I recorded a short clip (just 20 secs) to demo this problem:

The log given by the failure
N/A
Desired functionality
The webpack should works perfectly.
Mention any other details that might be useful
N/A

I've just confirmed that this issue has been resolved in Angular CLI v7.0.2.  Thanks!


Option downloadRetries is off by 1

There are two instances in the Downloader.ts file where the comparison downloadTries < options.downloadRetries is. Because a less than operator is being used instead of less than or equal operator, the amount of retries will be options.downloadRetries - 1.

Also the downloadBinary function throws if downloadTries >= options.downloadRetries. This will need to be changed to a greater than operator instead of greater than or equal operator to prevent the off by 1 issue as well.


[DOCS] Improve readme introduction

There's been a few previous attempts to improve the readme though I don't think either hit the mark quite right (#8499 and #8446). This change adds the flow logo in the correct style, centers the logo and introductions.
I've also added a reference to soundness to give new comers the right impression when people want to evaluate what makes flow stand apart from other static typecheckers. Which I think also answers and should satisfy #8424.
Before

After

I think I am satisfied with the current README.md, I think it's easier to read if you're opening it up as a text file. If you're on the web, you can check out the website.


Improvements to horizontal viewport check

Right was calculated incorrectly (but was never used), which has been fixed.
The horizontal viewport check took the absolute value of the element's left,
which didn't accurately check the viewport left/element right bound. This has
been modified to check the element's left against the viewport's right
(whether the element is within the right side of the viewport) and the
element's right against the viewport's left (whether the element is within
the left side of the viewport).

@zeusdeux - It may take me some time, but I'll do my best to get around to it.  It may require a separate test file, and I'll have to look into how isInViewport handles tests.

@npafundi sounds good :+1:

@npafundi could you check if your use case is resolved?
Pushed a few fixes today and bumped the version to 2.2.1 so please use the latest version and hit me back. I have a feeling the fixes in 2.2.1 should resolve your issue.

Hey @zeusdeux, sorry for the very long delay.  The recent updates haven't resolved the problem, but I've updated this pull request with a new test suite.
The new tests will fail when run against the current version of isInViewport, but will succeed with the changes to the calculations in this fork.
The issue occurs when a viewport contains horizontally-scrollable content.  I've made a CodePen in which you can see the desired behavior: [http://codepen.io/npafundi/pen/wBNrop] and another with current behavior: [http://codepen.io/npafundi/pen/XJOzJd].  As you scroll the list, isInViewport should only return a list of items which are currently visible.
Currently, as you scroll the list, isInViewport returns items which have since scrolled beyond the bounds of the viewport.  This fork fixes that issue, so only the content currently in the viewport is returned.
If you'd like me to make any changes, let me know.  I'm not sure why the Travis build is failing -- all the tests complete successfully for me when run locally.  Any idea what may be causing this?  I'm happy to fix it if you can point me in the right direction.
Thanks again for building isInViewport!
-Nick

@npafundi On merge with current master your tests fail. Could you rebase your changes onto current master of zeusdeux/isInViewport/tree/master, fix the failing tests and push the commits please?
Also, could you squash your commits into one and write up a proper commit message like the one you have in 381242dfda728ebefe1f96601afe126b31ef1c7a?
Other than that it LGTM! :+1:

@zeusdeux Sure, I think everything that you mentioned is fixed up.  If there are any other problems let me know!

@npafundi Merged in 79b1d6d053e5bffd969aec94052a6490493840b2
Quality PR. Thanks!

Awesome, thanks for the merge!


Implement access control config client to setup default permissions in petstore

This issue is blocked by #146.
Once the client is implemented, we need to use that client in the petstore example to automatically set the permissions of a newly created resource such that the creator becomes the owner of the resource, i.e., they should be allowed to read (GET /petsandGET /pets/{id}) and delete (DELETE /pets/{id}`) the resource.

#157


C# using declarations is not mentioned

[Enter feedback here]
C# 8 introduced the using declarations syntax, but it's not mentioned in this page.

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

ID: 3d252158-0367-77c8-1816-dbc385c5c40b
Version Independent ID: 575872e5-6273-f8f6-e3e2-2b911cd9d009
Content: The history of C# - C# Guide
Content Source: docs/csharp/whats-new/csharp-version-history.md
Product: dotnet-csharp
Technology: csharp-whats-new
GitHub Login: @erikdietrich
Microsoft Alias: wiwagn

Hi @arnonax-tr
using declarations are mentioned. It's the fourth bullet item in the list. The link goes to the wrong using page (the directive, not the declaration). I've opened a PR to fix that issue.

Thanks!
From: Bill Wagner @.>
Sent: Tuesday, June 20, 2023 4:59 PM
To: dotnet/docs @.>
Cc: arnonax-tr @.>; Mention @.>
Subject: Re: [dotnet/docs] C# using declarations is not mentioned (Issue
#35886)
Hi @arnonax-tr https://github.com/arnonax-tr
using declarations are mentioned. It's the fourth bullet item in the list.
The link goes to the wrong using page (the directive, not the declaration).
I've opened a PR to fix that issue.
—
Reply to this email directly, view it on GitHub
https://github.com/dotnet/docs/issues/35886#issuecomment-1598851015, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/APAQFASEAWFMJ6Y5YMLKMEDXMGUCTANCNFSM6AAAAAAZMXVMUU
.
You are receiving this because you were mentioned.Message ID: <
@.***>


[C++] Attach arrow::ArrayStatistics to arrow::Array

Describe the enhancement requested
GH-41909 introduced arrow::ArrayStatitstics but it's not associated with arrow::Array.
This issue attaches arrow::ArrayStatistics to arrow::Array.
See also:

#42133 includes PoC implementation of this
Discussion at https://github.com/apache/arrow/pull/42133#discussion_r1640175130

Component(s)
C++

We've attached to ArrayData not Array: #43797


Can the vgae model reconstruct graph structures with weights?

Hi @hrnne, Thanks for your interest!
VGAE model basically predicts (the probability of) binary edge. but you can regard the probability of an edge as some kind of weights. Hopefully there would be specific methods dealing with your case in recent publications. so highly recommend searching in Google Scholar.

Hi @hrnne, Thanks for your interest! VGAE model basically predicts (the probability of) binary edge. but you can regard the probability of an edge as some kind of weights. Hopefully there would be specific methods dealing with your case in recent publications. so highly recommend searching in Google Scholar.

I think reconstructing weights requires replacing a decoder. Do you have any suggestions? This is very important for my experiment, but I have been troubled for a long time. As you mentioned, the inner product decoder displays the connection probability between two nodes, so when reconstructing weights, if the predicted values are directly used as weights, vgae tends to assign a probability value to a large number of edges. Because my input is sparse, I actually expect to reconstruct many edges with weights of 0. Therefore, my experimental progress has been hindered. When conceptualizing the experiment, I didn't think it was very difficult to modify this model, but as the experiment progressed, I found that it was not simple and there was very little information about this model. Therefore, I sincerely hope you can provide me with some ideas.PLZ (╥_╥)


feat: add support for ocp-4.15

Description
Issue : https://github.ibm.com/GoldenEye/issues/issues/8745

Release required?

[ ] No release
[ ] Patch release (x.x.X)
[x] Minor release (x.X.x)
[ ] Major release (X.x.x)

Release notes content

Run the pipeline
If the CI pipeline doesn't run when you create the PR, the PR requires a user with GitHub collaborators access to run the pipeline.
Run the CI pipeline when the PR is ready for review and you expect tests to pass. Add a comment to the PR with the following text:
/run pipeline

Checklist for reviewers

[ ] If relevant, a test for the change is included or updated with this PR.
[ ] If relevant, documentation for the change is included or updated with this PR.

For mergers

Use a conventional commit message to set the release level. Follow the guidelines.
Include information that users need to know about the PR in the commit message. The commit message becomes part of the GitHub release notes.
Use the Squash and merge option.

/run pipeline

:tada: This PR is included in version 3.1.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Link of the training data is 404

Thanks for your great work. The link to the training data is 404. Could you please provide a new link of the entire training data? Thanks a lot.

I just check the link and it is ok. Could you show the specific link that is 404 to you?

I just check the link and it is ok. Could you show the specific link that is 404 to you?

Yes. The error is shown as above.

Please check the latest README. We have changed the URL for data.


merge from konsum-frontend-svelte,template-svelte-app

package.json

chore(deps): add ^2.52.8 remove ^2.52.7 (devDependencies.rollup)

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

v1.29.13
npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


[RFC] Deprecate on_pretrain_routine_start and on_pretrain_routine_end LM/callback hooks

Proposed refactor
Deprecate these hooks:
https://github.com/PyTorchLightning/pytorch-lightning/blob/6369e3b77fa3f38613b661517f6361f842f611c9/pytorch_lightning/trainer/trainer.py#L1261-L1269
Motivation
Users can use on_train_start, on_fit_start, or setup for the same purpose.
Additional Context
These hooks were originally added in https://github.com/PyTorchLightning/pytorch-lightning/pull/2850
I see about 30 usages of each hook using grep.app:
https://www.grep.app/search?q=on_pretrain_routine_start
https://www.grep.app/search?q=on_pretrain_routine_end
Part of #7740
cc @borda @tchaton @carmocca @awaelchli @ninginthecloud @daniellepintz

I have suggested some users over slack #questions to use these hooks for their use-case because on_train_start wasn't working for them. I can't remember what their use-case was.

Users can use on_train_start and on_train_end for the same purpose.

not sure if on_train_end is an alternative here for these hooks?

not sure if on_train_end is an alternative here for these hooks?

good catch. it should just be on_train_start
another weird thing with these hooks now is that there is nothing that happens between them, meaning at the very least, these hooks could be collapsed into one, if not fully deprecated. before, the model summary used to be printed here, but that was since refactored out to a callback which could run in on_train_start
similarly, this could be run in setup: https://github.com/PyTorchLightning/pytorch-lightning/blob/aeb0b5595fd73d086f4ae0f99d3f1f112f6a4c29/pytorch_lightning/callbacks/model_checkpoint.py#L257

not sure if on_train_end is an alternative here for these hooks?

yeah good call, updated the description.

meaning at the very least, these hooks could be collapsed into one, if not fully deprecated.

yeah agreed. I think between on_train_start and setup we should be able to give users the same functionality. I searched in the slack #questions channel but was unable to find the users @rohitgr7 was referring to. Rohit, do you think setup would also have worked for the users who were having trouble with on_train_start?

yeah agreed. I think between on_train_start and setup we should be able to give users the same functionality. I searched in the slack #questions channel but was unable to find the users @rohitgr7 was referring to. Rohit, do you think setup would also have worked for the users who were having trouble with on_train_start?

tbh I don't recall the actual use-case, I can't find them either since we have 10K limit for messages on slack. I am ok if these could be collapsed into one. I don't use this hook often. If everyone else agrees, I am ok with deprecation too.

Ok, let's see what other people think about deprecation

Looking at their position here
https://github.com/PyTorchLightning/pytorch-lightning/blob/6369e3b77fa3f38613b661517f6361f842f611c9/tests/models/test_hooks.py#L525-L528
I'd say on_fit_start is a more natural replacement.
I'm not sure at the moment whether they should be removed (after a deprecation ofc), but they sure could be collapsed into one hook.
This proposal is also missing links for
(1): The issue/PR that added them. You can use git blame to find it.
(2): Examples of public usage (or the lack of public usage). You can use www.grep.app for this

This proposal is also missing links for
(1): The issue/PR that added them. You can use git blame to find it.
(2): Examples of public usage (or the lack of public usage). You can use www.grep.app for this

Added both to the issue. From the grep.app search, it does seem to me that these usages could be replaced by on_fit_start or setup.

@tchaton what do you think about deprecating these hooks?

Hey @daniellepintz. Great question. I can see Bolt and other repos use it, but we should incentivize for cleaner code. So I am in favor of the depreciation.

Me too

For anyone who wants to pick up these issue, here is an example PR to follow: https://github.com/PyTorchLightning/pytorch-lightning/pull/10940

I would like to work on this

@krishnakalyan3 Assigned to you, thanks!!

Reopening since we still need to do the LM hooks


added japanese translation site

I found Japanese translation site. (v4)

CC: @juthilo @hnrch02

:+1:

@okmr-d Thanks.


#88 - Store expiry month and year on the Customer object.

This is now stored on the Card model now that Customer can have multiple payment sources and subscriptions.


Cross Compile on target aarch64-unknown-linux-gnu

Question
Cannot Compile on target=aarch64-unknown-linux-gnu
Before asking a question, make sure you have:

Searched existing Stack Overflow questions.
Googled your question.
Searched open and closed GitHub issues
Read the documentation:

TiKV Readme
TiKV Doc
TiKV-Ctl Doc
TiDB & TiKV Stack Doc

Done All Above, but not solution.
Error message:
failed to run custom build command for librocksdb_sys v0.1.0

@jiak94 Can you provide more informations for the compile error?

You're propably not aware of it, but this post is the first hit if using google for learning about -DLLVM_DEFAULT_TARGET_TRIPLE and aarch64


 It's a premature close/race condition issue! If I add the call in closeFiles, which causes it to be delayed slowly, the childs can spawn properly with the FDs.
func (d *Context) closeFiles() (err error) {
	httpClient := &http.Client{
		Timeout: 5 * time.Second,
	}
	req, err := http.NewRequest("GET", "http://<IP_ADDRESS>", nil)
	if err != nil {
		log.Println("error creating request", "err", err)
		return
	}
	resp, err := httpClient.Do(req)
	if err != nil {
		log.Println("error making CLOSE GET request", "err", err)
		return
	}
	defer resp.Body.Close()
	fmt.Println("CLOSE GET request status:", resp.Status)
	cl := func(file **os.File) {
		if *file != nil {
			(*file).Close()
			*file = nil
		}
	}

Adding a sleep for 1 second before closing is a decent solution for now.

The best thing I can find is to wait 2 seconds for the child and worker to fully init before exiting. It's a bit of a change for existing users, but it seems to work and unblock me.


the kakera does not react

I tried the bot but it doesn't react when kakera appears, any solution?

Haven’t gotten to adding button support for kakera only for claims Sent from my iPhoneOn Apr 12, 2023, at 6:56 PM, Fabricio Cuevas @.***> wrote:
I tried the bot but it doesn't react when kakera appears, any solution?
—Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this because you are subscribed to this thread.Message ID: @.***>


Patch CVE-2023-52356

Merge Checklist

All boxes should be checked before merging the PR (just tick any boxes which don't apply to this PR)

[x] The toolchain has been rebuilt successfully (or no changes were made to it)
[x] The toolchain/worker package manifests are up-to-date
[x] Any updated packages successfully build (or no packages were changed)
[x] Packages depending on static components modified in this PR (Golang, *-static subpackages, etc.) have had their Release tag incremented.
[x] Package tests (%check section) have been verified with RUN_CHECK=y for existing SPEC files, or added to new SPEC files
[x] All package sources are available
[x] cgmanifest files are up-to-date and sorted (./cgmanifest.json, ./toolkit/scripts/toolchain/cgmanifest.json, .github/workflows/cgmanifest.json)
[x] LICENSE-MAP files are up-to-date (./SPECS/LICENSES-AND-NOTICES/data/licenses.json, ./SPECS/LICENSES-AND-NOTICES/LICENSES-MAP.md, ./SPECS/LICENSES-AND-NOTICES/LICENSE-EXCEPTIONS.PHOTON)
[x] All source files have up-to-date hashes in the *.signatures.json files
[x] sudo make go-tidy-all and sudo make go-test-coverage pass
[x] Documentation has been updated to match any changes to the build system
[x] Ready to merge

Summary

fix CVE-2023-52356 with patch ported from master
https://gitlab.com/libtiff/libtiff/-/merge_requests/546/diffs
Change Log

CVE-2023-52356

Does this affect the toolchain?

NO
Associated issues

Links to CVEs

https://nvd.nist.gov/vuln/detail/CVE-2023-52356

Test Methodology

Pipeline build id: 522901

Auto cherry-pick results:

main :white_check_mark: -> https://github.com/microsoft/azurelinux/pull/8316

Auto cherry-pick pipeline run -> https://dev.azure.com/mariner-org/mariner/_build/results?buildId=523462&view=results


Put partition values into rows from dir path

Hello, I struggled with problems generating hadoop indexing job.
I'd like to add partition values into input rows even thought those values are not exist in hdfs file. Generally hive partition values are stored not in hdfs file but meta store object. By using 'ParsorHandler', extra meta values could be able to added to InputRow object.
If it doesn't support this feature, I should add extra duplicate partition values into hdfs file for every batch job.
e.g.
file path : /user/hive/warehouse/{dbname}.db/{tablename}/dt=2017051500/part-00000
Please review HadoopConfHandlerTest cases and third case of IndexGeneratorJobTest file.

Thank you for the contribution @soursop can you explain a little more why this cannot be accomplished in an extension?
New features to core druid are reasonable, but changing things in api/ are dangerous in the long run, and if possible it is good to be able to do things in extensions, even if they get adopted into core druid.

The main reason why I put ParseHandler in api is to store meta data that might be necessary information. I inspired this structure from hive meta data class(org.apache.hadoop.hive.ql.metadata.HiveStorageHandler). I though that might be more information that used while running transforming job. but I fully agree with what you're worried about. so you mean, some type of extend class of MapInputRowParser should be moved to indexing-hadoop module? Is it okay to change the module location and re-apply for PR if I understand it correctly?


please delete the mal library

1)please delete the mal library
it is re-installed as new with the name maf
2)update MOC on ver 0.8.3
Thanks!

Hi @fury02 .
There are currently no implemented features for deprecating and removing a package.
You can just keep updating the maf package, and leave mal

Hi. I'm understand.


consider adding multiple language, and next new moon/full moon?   Here is my code

export function calculateMoonPhase(currentDate: Date): number {
  const newMoonDate = new Date(Date.UTC(2019, 0, 6, 1, 28, 0)); // Reference New Moon date
  const lunarCycleLength = 29.53; // Average length of the lunar cycle in days
  const diffInDays = (currentDate.getTime() - newMoonDate.getTime()) / (1000 * 3600 * 24);
  const moonPhase = (diffInDays % lunarCycleLength) / lunarCycleLength;
  return moonPhase;
}

export function estimateNextMoonPhases(currentDate: Date): { nextNewMoon: string, nextFullMoon: string} {
  const currentMoonPhase = calculateMoonPhase(currentDate);
  const lunarCycleLength = 29.53; // Average length of the lunar cycle in days

  let daysUntilNextNewMoon = lunarCycleLength * (1 - currentMoonPhase);
  if (currentMoonPhase >= 0 && currentMoonPhase < 0.5) {
      daysUntilNextNewMoon = lunarCycleLength * (0.5 - currentMoonPhase);
  }

  let daysUntilNextFullMoon = lunarCycleLength * (0.5 - currentMoonPhase);

  if (currentMoonPhase >= 0.5) {
      daysUntilNextFullMoon = lunarCycleLength * (1 - currentMoonPhase) + (lunarCycleLength / 2);
  }

  const nextNewMoon = formatMMDDDate(new Date(currentDate.getTime() + daysUntilNextNewMoon * 24 * 3600 * 1000));
  const nextFullMoon = formatMMDDDate(new Date(currentDate.getTime() + daysUntilNextFullMoon * 24 * 3600 * 1000));
  return { nextNewMoon, nextFullMoon };
}

export function getMoonPhaseName(moonPhase: number): string {
  if(moonPhase < 0.03 || moonPhase > 0.97) {
      return "NewMoon";
  } else if(moonPhase < 0.22) {
      return "WaxingCrescent";
  } else if(moonPhase < 0.28) {
      return "FirstQuarter";
  } else if(moonPhase < 0.47) {
      return "WaxingGibbous";
  } else if(moonPhase < 0.53) {
      return "FullMoon";
  } else if(moonPhase < 0.72) {
      return "WaningGibbous";
  } else if(moonPhase < 0.78) {
      return "LastQuarter";
  } else {
      return "WaningCrescent";
  }
}

export function getMoonPhasesForPeriod(currentDate: Date = new Date()): MoonPhase[] {
    const todayMoonPhaseNumber = calculateMoonPhase(currentDate);
    const todayMoonPhaseName = getMoonPhaseName(todayMoonPhaseNumber);
    const { nextNewMoon, nextFullMoon } = estimateNextMoonPhases(currentDate);

    const today: MoonPhase = {
      date: formatMMDDDate(currentDate),
      moonPhaseIcon: MoonPhaseIcons[todayMoonPhaseName],
      name: todayMoonPhaseName,
    };

    const nextNewMoonPhase: MoonPhase = {
      date: nextNewMoon,
      moonPhaseIcon: MoonPhaseIcons["NewMoon"],
      name: "NewMoon",
    };

    const nextFullMoonPhase: MoonPhase = {
      date: nextFullMoon,
      moonPhaseIcon: MoonPhaseIcons["FullMoon"],
      name: "FullMoon",
    };

    // Determine the sequence based on current phase
    let phases: MoonPhase[];
    if (todayMoonPhaseNumber < 0.5) {
      // Today -> Next Full Moon -> Next New Moon
      phases = [today, nextFullMoonPhase, nextNewMoonPhase];
    } else {
      // Today -> Next New Moon -> Next Full Moon
      phases = [today, nextNewMoonPhase, nextFullMoonPhase];
    }

    return phases;
}

Appreciate sharing the ideas and implementation.
Internationalization support was something I had considered based on the lunar phase enumeration, but adds a hefty amount of complexity.
Any i18n implementation and corresponding translations would be local to that project.
Similar requests for next cycles have been made, in #30, and pull request in #17.
My formula has a small margin of error that I never had time to resolve.  There are some other aspects in regards to how this library handles time windows versus events.


Set up environment

Set up initial environment. Should use:

react
redux
isolated-core
webpack
gulp
eslint
travis
js es6 (babel)
mocha or karma or jest tests

done


Web-OS online dash board not developed

Please use the web-os framework to create it in rails/nodejs (nodejs is best for us)

See #11

We are closing this because it is a duplicate


[enhancement]: Invokation search

Is there an existing issue for this?

[X] I have searched the existing issues

Contact Details
No response
What should this feature add?
Sorry if this is a duplicate. I did search, but the "I have searched the existing issues" line in the template made it return almost every issue ever posted.
The idea is to add a search box to the invokation list, to allow you to search/filter the list for images that used a particular set of terms in the prompt, and other details saved in the metadata. Bonus points for making it possible to save these searches (similar to the concept of smart folders/playlists) to run again when more files have been added. There are some other good ideas to steal copy learn from here: https://github.com/RupertAvery/DiffusionToolkit
Another thing that would be cool would be able to collapse images that have identical metadata except for specified fields (seed, for example) into a folder/stack in the panel, so that you don't have to scroll past dozens or hundreds of variations on one concept to find ones from a previous project. A custom "project" metadata tag might even be used to as a proxy for actual folders.
See also, https://github.com/invoke-ai/InvokeAI/issues/1270
Alternatives
No response
Aditional Content
No response

Closing as a duplicate of #3943


Added pre commit configuration

This PR re-adds the pre-commit configuration from the experimental branch onto the main branch.
Currently, this is not a CI test and can be used as a formatting check locally before committing any changes.
Note: Running this for the first time may have library-wide changes due to black, this can be disabled locally by configuring pre-commit to not use it.

@kc611 thank you for the patch!


Update cypress test. Canvas 3D functionality. Basic actions.

Motivation and context
Added checks:

Count of control buttons
Tooltip

How has this been tested?

Checklist

[x] I submit my changes into the develop branch
[ ] I have added description of my changes into CHANGELOG file
[ ] I have updated the documentation accordingly
[ ] I have added tests to cover my changes
[ ] I have linked related issues (read github docs)
[ ] I have increased versions of npm packages if it is necessary (cvat-canvas,
cvat-core, cvat-data and cvat-ui)

License

[x] I submit my code changes under the same MIT License that covers the project.
Feel free to contact the maintainers if that's a concern.
[x] I have updated the license header for each file (see an example below)

# Copyright (C) 2021 Intel Corporation
#
# SPDX-License-Identifier: MIT

Coverage decreased (-0.02%) to 74.094% when pulling ccc03e4279edf316840e3d7fc508079e1a26cb18 on dvkruchinin:dkru/cayperss-test-update-case56 into adbc74fa29876ca625d178117129c8a834ea8463 on openvinotoolkit:develop.


testing: Set up placeholders for docs-examples-are-tests

I had previously prototyped an instance of this for the ImageOutput chapter.
This patch makes empty placeholders for all of the other API chapters. They don't contain any actual tests or generate any output now, but they do build and run, so people can drop new examples into them without having to worry about how to set up new files for each chapter.

Thanks. I tried to fix all my copy pasta, but it doesn't surprise me that I missed one.


to_string does not properly dump real JSON.

Since it doesn't dump real JSON, it's probably not a good way to dump JSON. So it should probably be removed or renamed for a better API.
The reason it doesn't dump real JSON is because when dumping strings, it ignores the \uxxxx literals and even the escape sequences which makes it wrong.
It might be worth looking into how Python does JSON dumping. This allows it to be relevant to issue #5.

I don't know what you mean by "it ignores the \uxxxx" literaals. The actual "\\u0041" literal is changed?

to_string doesn't know about escape sequences at all. So std::string x("\"\u0041\"") would not turn into that in to_string with json. It'd turn into ""<char at 0x41>"" instead of "\"\u0041\"".

Basically fixed in 10f922c63c796c9d4422004d947cc3a0606f2278.


Make this game engine C++ centered.

We need C++ in Godot 3.1 straigthtforward, well documented, and above all C++ centered. We need C++ supported as a main, built-in scripting language for game logic (see Unreal)? NOT for native libraries and NOT bindings.

I think I understand what you're saying based on your post over at the Polycode project. https://github.com/ivansafrin/Polycode/issues/715
It seems like you want C++ to be the main language and replace scripting languages as being the primary focus and make it C++ instead. If I'm understanding you correctly then it isn't going to happen. While C++ is faster as far as code execution goes, it's also extremely slow to develop with and the costs end up outweighing the benefits. Most of the performance critical code is already in C++.
However, it isn't hard to learn how Godot's nodes work internally. C++ isn't my primary language and I was able to figure it out. So, you can write your entire game by either making your own custom version of Godot, or you can make your own custom modules for your game using GDNative.

I've seen a number of these requests recently and I find them interesting.  It stands to reason that C++ is the most supported language in Godot considering that it's written in it.  A C++ developer working in or on Godot has access to more engine functionality than anything exposed via the supported scripting APIs.

I'm not sure Joe23232 and Raj2032 are the same guy or not. If not, my apology and you can ignore this comment, but if so (as Joe23232 just joined Github on today Dec 29 just to make an issue here, then please read along)
Based on your past questions mostly about C++ centric, I would kindly suggest to familiar yourself with Godot's direction and usage. I think every tool cannot be fully 100% of individual wants, but community at large is there to be heard and gradually adapt to provide if need.
About why not C++ centered, I would say that Godot itself has put C/C++ at first class citizen already with the benefit of high productivity in-mind via scripting language (GDScript). If they are not care about C++, I don't think they went too far to make 3.0 onwards support flexible way i.e. shared library to integrate with your custom C/C++ code. So "C++ centered" should not be in a question context. As community answered to your board questions so many times, including this from myself about C/C++ and its ecosystem in general or specific. I'd see it as information overwhelming and better to move on further from there and not to stress this again and again. If this sounds rude, I don't mean to, but at some points someone has to point this out.
If you want discussion, please see community and select which channel is the best for you to initiate the topic. Might be Questions & Answers, IRC, or Discord. Those places are better suitable for this kind of discussion, not issues on Github which is intended for reporting bug or real issues so community would see to fix it (they have seen ton of issues already, so we better at first filter some for them). But if you are only want to initiate topics/discussion without serious concern, neither community or here is suitable for that I believe.

GDScript is Godot's bread and butter though. The language is very powerful for its intended usage and underrated. Performance will rarely cause for a concern (over 20,000 lines so far) paired throughout hundreds of scenes, with no issues. The engine already makes writing custom C++ plugins extremely easy for rare cases as well.
Godot is in a nice ease-ability:performance ratio spot for development already, any change to this could be catastrophic

But at least stop using native library and bindings and simply allow users to have C++ pre configured straight out of the box without having to deal with native libraries, bindings etc.

As you've already been explained, that is already possible.
Just download the Godot source code off of github and code away. Add your own custom nodes and scripts into the engine, compile it and build your project in the editor using your new nodes.
If you don't want to build your own custom version of Godot and instead use the classic Godot interface (to guarantee compatibility with updates and all), that is where a native library comes into play.
You CAN'T use a compiled programming language like C++ without first compiling it to a library file. That is simply how C++ works. You will face this "problem" in every single C++ project you will ever seek out to ask for "C++ centering" in, because that is simply how C++ works. No way around it, C++ cannot be run as executable code without first compiling it to a binary or library, which then has to be linked to your existing binary, which is what "native libraries" are.

May you please teach me how to use native library?
On Fri, Dec 29, 2017 at 1:39 PM, Tim Schellenberg<EMAIL_ADDRESS>wrote:

As you've already been explained, that is already possible.
Just download the Godot source code off of github and code away. Add your
own custom nodes and scripts into the engine, compile it and build your
project in the editor using your new nodes.
If you don't want to build your own custom version of Godot and instead
use the classic Godot interface (to guarantee compatibility with updates
and all), that is where a native library comes into play.
You CAN'T use a compiled programming language like C++ without first
compiling it to a library file. That is simply how C++ works. You will face
this "problem" in every single C++ project you will ever seek out to ask
for "C++ centering" in, because that is simply how C++ works. No way around
it, C++ cannot be run as executable code without first compiling it to a
binary or library, which then has to be linked to your existing binary,
which is what "native libraries" are.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/godotengine/godot/issues/15134#issuecomment-354389649,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AhTvoV3eg3q2JhB2maz82voPcTEx6bvUks5tFFDWgaJpZM4ROxUK
.

https://www.google.co.kr/search?q=godot+c%2B%2B

Make this game engine C++ centered?...... no thanks!

This question has already been asked several times the few past days, with several and well argued responses. I close this one.


Image caching not rewriting xml

Seems that it is downloading the images (with query params too which is great for saving space, bad for image quality lol) but it doesn't seem to be changing the image URL in the XML file to point to where the cached image is stored on the server.

It is currently only updating the m3u, but will make sure the xml gets updated

Patched in 0.9.21 - To Be Released

Patched in 0.9.21 - Closing for now. Please re-open if this issue persists.


Suggestions to improve the JOSS paper

Based on https://github.com/openjournals/joss-reviews/issues/5639#issuecomment-1628621998

Probably, WSKNN should be in all capitals everywhere in the text
Please implicitly introduce the abbreviation K-NN before using it. Also, choose one style k-NN/K-NN and stick to it.
Please expand the WSKNN abbreviation in the summary.
"The main difference between wsknn and the V-SKNN model from the presented repository is that the latter is a ready-to-use package." --- probably, "former", not "latter". Also, why different naming? Maybe, since W is for "weighted", V-SKNN doesn't work with session event weights, but WSKNN does?
"The package is related to past research projects within a company" --- what company? If it's the same you indicate in your affiliation information for this paper, maybe it's worth mentioning. If not, please add the industry to which the company belonged.
"In the closest future package will be enhanced with tensorflow version of the algorithm" --- please add a canonical citation for Tensorflow (https://www.tensorflow.org/about/bib). Please clarify, whether this work-in-progress is a personal effort of the package author or there is some community around (e.g. an internal project in a private company)
I suggest reworking the sections (without adding or removing the content, but rather moving paragraphs around). For example, now "Statement of need" section talks about the input data format most of the time ("Input data format" would be a great additional section). Quite the contrary, "Related work" explains how WSKNN might be useful and why existing solutions are not enough (compare: "A Statement of need section that clearly illustrates the research purpose of the software and places it in the context of related work" from https://joss.readthedocs.io/en/latest/submitting.html#what-should-my-paper-contain).
"algorithm itself is tuned and enhanced Vector Multiplication Session-Based kNN" --- what are the enhancements contributed by WSKNN?
"The package has built-in evaluation metrics" --- the metrics you mention (MRR, precision/recall) are pretty standard. Why not reuse any existing implementation?
"The other example of a stand-alone repository is ... with Keras implementation of Gru4Rec session-based recommender" --- how exactly this software is different from WSKNN or similar to it? What do you mean by "stand-alone" here?
"The module takes into account the fact that different event actions have their specific weights" --- has the package user to set these weights or does WSKNN learn them somehow?
"additional parameters:" seems to be wrongly indented.
"it allows testing different scenarios in parallel" --- how exactly? What could prevent a user from testing several models in parallel, please elaborate
"The users may control" --- are all the parameters listed arguments of the predict() method? If yes, please state explicitly
"fit() to build a memory representation of a model" --- do you mean a similarity matrix? KNN is a well-known method outside recommender systems world, but "memory representation of a model" sounds arcane.
"As a memory-based model" --- probably "As a memory-based method" (in contract to model-based methods)
"the next n-products" --- probably "the next n product"
"The wsknn package is a lightweight tool for modeling user-item interactions and making recommendations from sequential dataset" --- please cite popular sequential datasets
Please cite any review of existing sequential recommenders in the Summary
"The package’s algorithm can work in a cold-start scenario" --- please elaborate (maybe cite a paper on cold-start problem for sequential recommenders), and explain what do you mean by "cold-start scenario"
"k-NN based approach may be placed in the bigger pipeline" --- please cite a case-study or a industrial-track paper discussing such combined architectures
"Especially powerful is using weights..." --- please cite a paper containing experimental evidence for that claim
"it depends only on the numpy, more_itertools and pyyaml" --- please explain how are you using pyyaml and add a canonical numpy citation
"Items and Sessions that are storing item-sessions and session-items mappings may be updated sequentially" --- Items and Sessions and not mentioned in the API reference in the documentation. How exactly may them be updated sequentially, what does it mean?
"But this is related to the every other model, except simple Markov Model" --- please cite any such model (maybe from sklearn)
"there’s additional overhead related to the preparation of the input... That’s why wsknn has a built in preprocessing module" --- do you mean that preprocessing module helps to reduce the mentioned overhead?

Well done!


Hover-effect building block

Simple underline hover effect

@vallyol Noted some changes ... Please do
All in all ... very cool building block!

@vallyol EOL means End of line (Newline at the end of a document)
https://en.wikipedia.org/wiki/Newline

@IamManchanda
Yes, I know ... I was just surprised :)
My .gitconfig file contains a string
[core] 	autocrlf = input
My OS is Linux. Now I made one more commit for verification. Result is

Everything up-to-date

Just add a blank line at last!

You didn't remove that .directory?

@vallyol Using HTML5 data attribute to apply JS behavior is proper, because of .classes is used for CSS styles and #IDs must be unique.
<div class="menu-centered" data-menu-underline-center>
</div>

$("[data-menu-underline-center] a").addClass("underline-from-center");

@IamManchanda
Thanks!

LGTM Merging this ....
cc @rafibomb for looking into the yml file before deploy!
Great work @vallyol .... Nice building block ... keep it up!


[OpOptimization]add IREE BatchNormalization

the same as iree  matmul

Why there're two copies of each file?


RDS arn missing region

There's an issue tagging RDS instances when region isn't passed in on the command line. In https://github.com/capitalone/cloud-custodian/blob/master/c7n/utils.py#L244 if the region isn't set, it will generate invalid RDS ARNs, for instance. Passing in -r us-east-1 will generate a syntactically correct arn, arn:aws:rds:us-east-1:123123123:db:trenholmtest with out region, and without the  ARN. arn:aws:rds::123123123:db:trenholmtest
In resources/s3.py it uses us-east-1 as the default if it's not specified. Should RDS be the same way or is this by design?
example policy
policies:
  - name: rds-owner-tag
    resource: rds
    filters:
      - "tag:OwnerName": absent
    actions:
      - type: tag
        key: OwnerName
        value: OwnerName

After adding some debug
arn:aws:rds::123123123:db:trenholmtest
An error occurred (AccessDenied) when calling the ListTagsForResource operation: Unknown
Traceback (most recent call last):
  File "/Users/jtrenholm/work/github-cloud-custodian/cloud-custodian/c7n/resources/rds.py", line 151, in process_tags
    tag_list = retry(client.list_tags_for_resource, ResourceName=arn)['TagList']
  File "/Users/jtrenholm/work/github-cloud-custodian/cloud-custodian/c7n/utils.py", line 288, in _retry
    return func(*args, **kw)
  File "/Users/jtrenholm/work/github-cloud-custodian/cloud-custodian/lib/python2.7/site-packages/botocore/client.py", line 251, in _api_call
    return self._make_api_call(operation_name, kwargs)
  File "/Users/jtrenholm/work/github-cloud-custodian/cloud-custodian/lib/python2.7/site-packages/botocore/client.py", line 537, in _make_api_call
    raise ClientError(parsed_response, operation_name)
ClientError: An error occurred (AccessDenied) when calling the ListTagsForResource operation: Unknown

we should probably be falling back to the session's region which will take into account metadata server for an instance.

its a bit unclear how to tackle this cleanly.. and it effects any resource that requires arns for tags.

#835 is an attempt to resolve

This should be resolved in <IP_ADDRESS> as region won't ever be null to custodian, it will always match the aws sdk.