LPIF question on pipeline behavior

If we leave the active state, do we need to flush the flit data pipeline or is it assumed the flits are either already flushed before we enter the inactive state or those flits are intended to be preserved across the inactive state.
Related, we see the attached screenshot behavior in the VIP. We think it is a bug in the VIP because the VIP is not obeying the trdy and is truncating the last beat of data early. (around 2670ns). A similar state exists around 2664ns (stallreq high, no stallack) and the VIP obeyed the flow control. So it looks like the bug is the last beat of data before the stallack is not obeying flow control. But the spec was a little confusing on the relationship of trdy, stallreq, stall ack. Do you concur it looks like a VIP bug from these waves?

This is a second case that occurs on the "last" transaction from the VIP (as indicated by the irdy going low), but without the stallreq/stallack to muddle the conversation. And similar to before, a few clocks earlier, there is a flit that is split in two by the pl_trdy, and the VIP obeys that flow control (and other identical scenarios earlier). It is only the last one.

I can't tell from these screen caps where the signals are being sampled at. Ideally the waves would more closely match Figure 26 in the LPIF 1.1 Specification with probes of the relevant signals taken at both sides of the pipelining block.
The Link Layers (VIP in this case) have no knowledge of pipeline delay between its interface and the LogPHY interface (CXL LPIF to AIB Adaptor in our case). Our RTL needs to fully account for the total roundtrip latency when changing the values of pl_stallreq/pl_trdy and sampling lp_valid/lp_stallack.
Looking at the waves above and assuming all signals are probed at the LogPHY boundary, this doesn't look like a VIP error to me. It looks like pl_trdy is de-asserted early and lp_valid and lp_stallack are toggled at the correct clock edges when accounting for pipelining. There is no way for the VIP to know exactly when pl_trdy is de-asserted as it does not know the pipeline depth and assumes a direct connection to the LogPHY.

Agreed that we need to make sure all flits are "flushed" through the pipeline before we transition state. This mirrors the general flow that as far as the link layer is concerned, the boundary conditions are definitive and any pipelining / delays should be hidden from the boundary interfaces. In this case, the flits were accepted by the PHY (via TRDY=1) and the state transition will occur only after they really accepted by the internal logic of the PHY after the pipeline.
Michael concurred that the VIP dropping the last beat of data looked incorrect. Still waiting for Synopsys's response.


Meeting Counts

My database says there are 147 meetings.
The count on the meeting list view says there are 105 active meetings., 61 in-person, 46 online, 2 hybrid.
61+46+2 = 109.
Are the hybrid meetings counting in online category and the in-person category? That could eliminate two, but I am still then left with 107, not 105.
Or have I made a mistake somewhere?

The math works out.
You have 61 In-person, which includes the 2 Hybrid, so 59 In-person only
Likewise, the 46 online includes the 2 Hybrid, so 44, Online only
59 In-person only
44 Online only
2 Hybrid
105 Active Meetings
Brian W


SIR

Hard-code basic SIR model into package

[ ] more advanced back-end layout of code

[ ] highcharter to hold vertical line across screen


Improve quality checks and ship type hints

Codecov Report

Merging #21 (6598263) into main (5d7ca94) will increase coverage by 0.04%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##             main      #21      +/-   ##
==========================================
+ Coverage   89.74%   89.79%   +0.04%
==========================================
  Files          22       22
  Lines         956      960       +4
==========================================
+ Hits          858      862       +4
  Misses         98       98

Flag
Coverage Δ

unittests
89.79% <100.00%> (+0.04%)
:arrow_up:

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

kiez/kiez.py
98.61% <ø> (ø)

kiez/analysis/estimation.py
100.00% <100.00%> (ø)

kiez/io/temp_file_handling.py
100.00% <100.00%> (ø)

kiez/neighbors/approximate/faiss.py
90.24% <100.00%> (+0.37%)
:arrow_up:

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


Fixes for util::rmdir().

Refactored generic \Exceptions to \RuntimeExceptions.
Refactored to return true on success (instead of null).
Fixed the ability to remove symlinks to directories on Windows systems.

Coverage increased (+0.57%) to 88.5% when pulling 0d43a9d785d48c02d70e48f36fd3fecaebd94284 on phpexpertsinc:rmdir_symlink_fix into 0a36f569185a2f2fcee0c337615a5950ecb9575b on brandonwamboldt:master.

Again, thank you for the contribution! I'm not a fan of disabling code coverage on the failure cases but I'm also not sure how to reliably test that, so it's fine for now.

You can't test failure of the operating system or underlying base PHP functions without using something extremely esoteric, like using the Runkit extension to rename the builtin functions and then using your own test functions. It is generally considered acceptable to ignore code coverage analysis on such lines.


Pass by value only up to 2 machine words

When we did some testing with List and Str, we realized that even 3 words can be slower than passing by reference.
This matches what most low level languages do (c, c++, rust), so I think it is likely a better default.
In many cases it is required by abi anyway cause values bigger than 2 machine words will just be pushed onto the stack.

I think this is probably breaking our abi with the builtins in some way. Given this is such a low priority change and it isn't a one line, just gonna close it for now. Might circle back to it some time in the far future.


wordoftheday: set show_full_animation

@greg-n More info at https://tidbyt.dev/docs/publish/advanced


[ENHANCEMENT] support recurring job CR management in the helm chart

Hello,
How do you tell longhorn to automatically create tasks when they are not present?
recurringJobSelector:
        enable: true
        jobList: '[{"name":"b6am","task":"backup","cron":"0 6 * * *","retain":3},{"name":"b12pm","task":"backup","cron":"0 12 * * *","retain":2},{"name":"b6pm","task":"backup","cron":"0 18 * * *","retain":2}]'

My list is present but the tasks doesn’t exist
Environment

Longhorn version : 1.2.2

Installation method (e.g. Rancher Catalog App/Helm/Kubectl): helm

Kubernetes distro (e.g. RKE/K3s/EKS/OpenShift) and version : 1.22

Number of management node in the cluster : 1
Number of worker node in the cluster : 1

Node config

OS type and version: ubuntu 20.04 LTS
CPU per node: 4
Memory per node: 16
Disk type(e.g. SSD/NVMe): SSD
Network bandwidth between the nodes: 1Gbps

Underlying Infrastructure (e.g. on AWS/GCE, EKS/GKE, VMWare/KVM, Baremetal) : bare-metal

Number of Longhorn volumes in the cluster : 4

Additional context
See https://rancher-users.slack.com/archives/CC2UQM49Y/p1639750876140300

This is more like a question. @c3y1huang or @jenting is there any way to see if the recurring jobs executed like events or resources status?

We had discussed in the Slack, it's an enhancement to configure RecurringJob CRs thru helm chart.

We had discussed in the Slack, it's an enhancement to configure RecurringJob CRs thru helm chart.

Good. Thanks for the info.

Hello, I would second the calls for being able to define RecurringJobs using the helm chart.


Update docs for Windows

The docs assume an OSX system, but some users may be using Windows.

Hello. We are analyzing your codes for learning, not for commercial use.
However, I'm trying to run them with a docker as written in readme, but supervisor.conf file didn't run. So I wonder if you know about that and if you know already, then how could you solve it?


update python location for cpu init

omitted this in previous bump to 11.3 databricks runtime

no build needed


Brets don't show

https://dicedornot.vengefulpickle.com//?rebbl=100090200d
From inspecting the network, it only downloads images/players/nurgle/home.png

Should be fixed now. I had missed a css in my scrape of rebblvision.


display thumbnail for encrypted files without a remote thumbnail

Now encrypted files without a thumbnail will properly show an image in the adapter.

Signed-off-by: Dillon Kanada<EMAIL_ADDRESS>

Hi @dkanada ,
Thanks for the PR.
According to @giomfo , everything is ok :-)
Have you checked if there was an existing issue about this?
Benoit

I didn't see any. The issue has been around since I started using Riot though, so I figured it was next in line to be squashed.


feat(logging): logstash support

adds support for ELK stack statistical analysis and graphing, and child loggers injecting event ids.
BREAKING CHANGE: executors, handlers and related functions now take a 'log' parameter in order to
support childloggers.  this allows filtering by eventids in cloudwatch/ELK

yeah the this.log = log was needed to satisfy lint. it actually happens in the superclass, so not really reqd  there at runtime.  might be a way to supress.


chore(amplify-codegen-appsync-model-plugin): generate coverage report

Issue #, if available:
Description of changes:
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Codecov Report

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

@@            Coverage Diff             @@
##           master    #3053      +/-   ##
==========================================
+ Coverage   61.37%   62.94%   +1.57%
==========================================
  Files         223      223
  Lines       11494    11207     -287
  Branches     2219     2312      +93
==========================================
  Hits         7054     7054
+ Misses       4132     3840     -292
- Partials      308      313       +5

Impacted Files
Coverage Δ

packages/amplify-cli/src/context-extensions.ts
25.69% <0%> (ø)
:arrow_up:

...lify-graphql-types-generator/src/compiler/index.ts
90.32% <0%> (ø)
:arrow_up:

...y-graphql-types-generator/src/compiler/legacyIR.ts
96.72% <0%> (ø)
:arrow_up:

...ages/amplify-appsync-simulator/src/server/index.ts
0% <0%> (ø)
:arrow_up:

...ackages/amplify-appsync-simulator/src/utils/log.ts

packages/amplify-appsync-simulator/src/index.ts

...amplify-appsync-simulator/src/server/operations.ts

...plify-appsync-simulator/src/server/subscription.ts

...psync-simulator/src/resolvers/pipeline-resolver.ts
0% <0%> (ø)

...-simulator/src/schema/directives/directive-base.ts
0% <0%> (ø)

... and 2 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 9951402...4ce6d08. Read the comment docs.


Bug Report

Bug Report
import DS.Servers:Connect
import DS.Main:Players
import DS.Main:lib
Libraries do not work. Use
from DS import Servers:Connect
from DS import Main:Players
from DS import Main:lib
or
from DS bulkImport [Servers:Connect;Main:Players;Main:lib]

**EDIT :
Malfunction happened in .ds compiler. Do not use libaries until further notice**

Edit 2 :
About Bug

this bug was caught 5 hours ago while creating Server Connection Script. Please do not use libraries (from :: import :: or import ::)

Fixing of

The fix will take cca 30 minutes, stand by. DO NOT use libraries as it can malfunction your code and it may or may not affect your PC. Please stand by until further notice.

What now

Only wait. 🙏 😄 . We will resolve it soon

**About DarkScript 1.25 (BugFixes)

You will be required to download new compiler version of DarkScript. you can find it in manifest.ds file.

We, the team thank you so much for your paitience. Please be patient, we will resolve it in no time. 👍

Wow, this turned up to be really simple. Okay, no need to download, everything fixed. You can use libraries again.
Thank you for your patience


Port Salsa fixes

Port Salsa fixes for #6679, #6699, #6680, #6698, #6707, #6709, and #6741

:+1:

:+1:


run_actions_on_self_hosted

For the self-hosted runner to pick up on actions jobs, they must have the property:
runs-on: self-hosted
I changed only the jobs on CI.yaml for now.

Hi @nachog00 is this PR still active?  Or has it been superceded?

Hi @nachog00 is this PR still active? Or has it been superceded?

no. When self hosted runners handle jobs correctly, merging this will make all jobs be picked up by them.

Since this is blocked on CI... and since it should speed up CI, I am going to merge it.


Option types

From @0xbadb002 on January 18, 2016 15:25
Copied from original issue: ark-lang/stdlib#1

Are these going to go in the standard library, or in the runtime?

I'm on a tilt between the options. The benefits of having them in the runtime would be to always have them available, but it would result in an increase in binary size. Preferably if it was to be included in the runtime we'd need a way for the developer to disable it.

I mean we could do that thing I suggested where we selectively choose runtime features per module? Or we could have the option types natively in the syntax but eh...

I think it's possible for LLVM to remove the option code altogether if it's unused (LTO?)

We should include them in runtime, imagine importing Go's error everywhere.

This is done


Eigen version in cmake mismatch

In tensorflow/contrib/cmake/external/eigen.cmake, both eigen_archive_hash and eigen_HASH don't match those in tensorflow/workspace.bzl and the source (e.g. third_party/eigen3/Eigen/Cholesky). This leads to build failure when using cmake.
Could anyone update tensorflow/contrib/cmake/external/eigen.cmake?

Fixed, I believe.

(with 2c2f422a628d83a591fb5d20d46deecc47655ead)

Updated in #1738

Brilliant! Now even CLion's debugging seems to work.

@phisiart
which version of tensorflow and clion you used, can teach me the configuration process import tensorflow source code . After installing the plugin, import the source directly? Still need to run other steps?


Missing dependencies?

I followed the steps but couldn't get it to run without the following in my dependencies
  "dependencies": {
    "aurelia-animator-css": "^1.0.0-rc.1.0.0",
    "aurelia-binding": "^1.0.0-rc.1.0.3",
    "aurelia-bootstrapper": "^1.0.0-rc.1.0.0",
    "aurelia-dependency-injection": "^1.0.0-rc.1.0.1",
    "aurelia-event-aggregator": "^1.0.0-rc.1.0.0",
    "aurelia-fetch-client": "^1.0.0-rc.1.0.0",
    "aurelia-framework": "^1.0.0-rc.1.0.2",
    "aurelia-history": "^1.0.0-rc.1.0.0",
    "aurelia-history-browser": "^1.0.0-rc.1.0.0",
    "aurelia-loader": "^1.0.0-rc.1.0.0",
    "aurelia-loader-default": "^1.0.0-rc.1.0.0",
    "aurelia-logging": "^1.0.0-rc.1.0.1",
    "aurelia-logging-console": "^1.0.0-rc.1.0.0",
    "aurelia-metadata": "^1.0.0-rc.1.0.1",
    "aurelia-pal": "^1.0.0-rc.1.0.0",
    "aurelia-pal-browser": "^1.0.0-rc.1.0.1",
    "aurelia-path": "^1.0.0-rc.1.0.0",
    "aurelia-polyfills": "^1.0.0-rc.1.0.0",
    "aurelia-route-recognizer": "^1.0.0-rc.1.0.1",
    "aurelia-router": "^1.0.0-rc.1.0.1",
    "aurelia-task-queue": "^1.0.0-rc.1.0.0",
    "aurelia-templating": "^1.0.0-rc.1.0.1",
    "aurelia-templating-binding": "^1.0.0-rc.1.0.1",
    "aurelia-templating-resources": "^1.0.0-rc.1.0.1",
    "aurelia-templating-router": "^1.0.0-rc.1.0.1",
    "bluebird": "^3.4.1"
  },

Do these need to be included or am I missing something?
Not sure if this is a separate issue but I'm also seeing the following in the errors in the build
Starting 'buildTypeScript'...
src/home.ts(324,30): error TS2339: Property 'includes' does not exist on type 'string[]'.
[21:06:32] gulp-notify: [Error running Gulp] Error: src/home.ts(324,30): error TS2339: Property 'includes' does not exist on
type 'string[]'.
[21:06:32] TypeScript: 1 semantic error
[21:06:32] TypeScript: emit succeeded (with errors)

and further down
Writing app-bundle.js...
Writing vendor-bundle.js...
{ uid: 10,
  name: 'writeBundles',
  branch: false,
  error: { [Error: Cannot find module 'source-map'] code: 'MODULE_NOT_FOUND' },
  duration: [ 4, 217453451 ],
  time:<PHONE_NUMBER>151 }
{ uid: 2,
  name: '<series>',
  branch: true,
  error: { [Error: Cannot find module 'source-map'] code: 'MODULE_NOT_FOUND' },
  duration: [ 6, 719903930 ],
  time:<PHONE_NUMBER>163 }
{ uid: 1,
  name: '<series>',
  branch: true,
  error: { [Error: Cannot find module 'source-map'] code: 'MODULE_NOT_FOUND' },
  duration: [ 6, 720458512 ],
  time:<PHONE_NUMBER>163 }
{ uid: 0,
  name: '<series>',
  branch: true,
  error: { [Error: Cannot find module 'source-map'] code: 'MODULE_NOT_FOUND' },
  duration: [ 6, 721402778 ],
  time:<PHONE_NUMBER>163 }
{ [Error: Cannot find module 'source-map'] code: 'MODULE_NOT_FOUND' }

Weird, let me investigate. I am seeing something completely different, but am maintaining two codebases. I'll make sure nothing was missed during the import. The includes error is strange as this is being run on an array (maybe the type is being messed up).

Hey, I am currently building my first aurelia app and stumbled across this project because of its gmaps and firebase support. It wasnt easy to get it running but I want to share my learnings for anyone else who might be in a similar position.
I am pretty sure, the TypeScript error has nothing to do with dependencies and is just a bug in home.ts.
Replacing .includes(...) by .find(...) on line 324 did the trick for me.
However, there were more changes necessary to the html part of the app since the site-header has no access to its parent's properties.
When comparing to the version on pokego.run, I realized some more differences but mine seems to be working now and a great starting point for my app.
Thanks, @Vheissu!


Crash on Checking Boosts Menu

Just a quick crashlog dump basically since im not really familiar with the codebase, if any one of you are able to make sense of this lmk :)
(For reference, no plugins or themes were used, version: 175.16)
If u need the Component tree as well lmk too
Stack trace:
Error: Invalid perk icon - 13
at P (/workdir/discord_android_rn/android/app/src/main/assets/index.android.bundle:6934:5517)
at anonymous (/workdir/discord_android_rn/android/app/src/main/assets/index.android.bundle:6934:6537)
at map (native)
at P (/workdir/discord_android_rn/android/app/src/main/assets/index.android.bundle:6934:6516)
at renderWithHooks (/workdir/discord_android_rn/android/app/src/main/assets/index.android.bundle:43:50477)
at beginWork$1 (/workdir/discord_android_rn/android/app/src/main/assets/index.android.bundle:43:113726)
at performUnitOfWork (/workdir/discord_android_rn/android/app/src/main/assets/index.android.bundle:43:101210)
at workLoopSync (/workdir/discord_android_rn/android/app/src/main/assets/index.android.bundle:43:101092)
at renderRootSync (/workdir/discord_android_rn/android/app/src/main/assets/index.android.bundle:43:100801)
at performSyncWorkOnRoot (/workdir/discord_android_rn/android/app/src/main/assets/index.android.bundle:43:97137)
at flushSyncCallbacks (/workdir/discord_android_rn/android/app/src/main/assets/index.android.bundle:43:31538)
at scheduleUpdateOnFiber (/workdir/discord_android_rn/android/app/src/main/assets/index.android.bundle:43:92892)
at dispatchSetState (/workdir/discord_android_rn/android/app/src/main/assets/index.android.bundle:43:54798)
at anonymous (/workdir/discord_android_rn/android/app/src/main/assets/index.android.bundle:1574:493)
at anonymous (/workdir/discord_android_rn/android/app/src/main/assets/index.android.bundle:1600:1462)
at anonymous (/workdir/discord_android_rn/android/app/src/main/assets/index.android.bundle:1614:775)
at dispatch (/workdir/discord_android_rn/android/app/src/main/assets/index.android.bundle:1613:341)
at anonymous (/workdir/discord_android_rn/android/app/src/main/assets/index.android.bundle:1549:2368)
at anonymous (/workdir/discord_android_rn/android/app/src/main/assets/index.android.bundle:1611:381)
at anonymous (/workdir/discord_android_rn/android/app/src/main/assets/index.android.bundle:1549:2338)
at anonymous (/workdir/discord_android_rn/android/app/src/main/assets/index.android.bundle:1549:3220)
at apply (native)
at anonymous (/workdir/discord_android_rn/android/app/src/main/assets/index.android.bundle:1560:895)
at pushModal (/workdir/discord_android_rn/android/app/src/main/assets/index.android.bundle:1533:6060)
at push (/workdir/discord_android_rn/android/app/src/main/assets/index.android.bundle:1518:415)
at anonymous (/workdir/discord_android_rn/android/app/src/main/assets/index.android.bundle:1518:815)
at anonymous (/workdir/discord_android_rn/android/app/src/main/assets/index.android.bundle:899:697)
at apply (native)
at anonymous (/workdir/discord_android_rn/android/app/src/main/assets/index.android.bundle:84:2028)
at _callTimer (/workdir/discord_android_rn/android/app/src/main/assets/index.android.bundle:84:574)
at _callReactNativeMicrotasksPass (/workdir/discord_android_rn/android/app/src/main/assets/index.android.bundle:84:1016)
at callReactNativeMicrotasks (/workdir/discord_android_rn/android/app/src/main/assets/index.android.bundle:84:3827)
at __callReactNativeMicrotasks (/workdir/discord_android_rn/android/app/src/main/assets/index.android.bundle:67:3302)
at anonymous (/workdir/discord_android_rn/android/app/src/main/assets/index.android.bundle:67:1158)
at __guard (/workdir/discord_android_rn/android/app/src/main/assets/index.android.bundle:67:2901)
at flushedQueue (/workdir/discord_android_rn/android/app/src/main/assets/index.android.bundle:67:1117)
at callFunctionReturnFlushedQueue (/workdir/discord_android_rn/android/app/src/main/assets/index.android.bundle:67:850)

This is a Discord issue as far as I know, probably fixed in later versions.


Update maths page

Trello card
https://trello.com/c/kSyTOLN1/5726-improve-maths-subject-page-following-user-research
Context
We need to make some updates to the maths page to make sure it's consistent with the computing and physics pages.
Changes proposed in this pull request

ensure meta description includes become a maths teacher
update bursary and scholarship link
update non-UK qualifications link to go to non-UK qualifications page
update nav card description
include maths in national curriculum link
delete references to primary in qualifications section
add reference to secondary in intro section
change 'receive this alongside a tuition fee and maintenance loan' to receive a bursary or scholarship...
add loan after tuition fee in the sentence above
make themes you'll cover wording consistent in all sections
update wording in Fund your teacher training section to You may also be able to get extra funding **and **support if you're:
delete surname and location if applicable from quotes
add links to ks3 and ks4 lesson plans where available for both under the relevant sections

Guidance to review

@MaxineCl Something's gone a bit wrong with the promo/footer


Well Done Sir!

Excuse the bug report ;)
I tried a number of these type of components and yours is by far the best. Good job!

Ha! Thanks @timothyerwin. Always happy to hear when hard work is appreciated. Thank you! 🙏


generated patches by repairSStuBs?

Hi,
Thanks for the cool repo.
Are the patches by repairSStuBs available somewhere?
Thanks,
--Martin

Hi Martin,
Thank you, I'm glad to see you interested.
I uploaded the generated patches for the sstubs.json version of the dataset here:
https://www.mediafire.com/file/jd9byxt3qcxy7bu/repair_output.zip/file
However, there are a few things about these results:

Detecting lines are done using a heuristic. Ideally, I should've used a parser to not mess things up.
There might be some overlap between the train and test data since they both are collected from popular GitHub projects.
There are some duplicates in the dataset that I have not removed.
Some bugs have a wrong line number in the dataset that can only be found manually.


Synology no ping as non-root user

Running on synology with PW3, setup fails due to non-root user not having ping permissions
sudo chmod +s /bin/ping will fix
might be better to use curl for the pw3 check for synology users

Thanks @zcpnate !
Brilliant idea. Can you test this script for me on you setup?
#!/bin/bash
#
# Function to test an IP to see if it returns a ping
function test_ip() {
    local IP=$1
    if [ -z "${IP}" ]; then
        return 1
    fi
    if curl --head --connect-timeout 1 --silent ${IP} > /dev/null 2>&1; then
        return 0
    else
        return 1
    fi
}

# check IP and if it returns a ping print "IP is up" else print "IP is down"
echo "Positive test"
if test_ip "<IP_ADDRESS>" ; then
    echo "IP is up"
else
    echo "IP is down"
fi

echo "Negative test"
if test_ip "<IP_ADDRESS>" ; then
    echo "IP is up"
else
    echo "IP is down"
fi


Price getting overlapped in a screen when INR is selected

Bug Report

I just checked in the demo also, so this issue exist since past versions also.
Price getting overlapped in a screen when INR is selected.

Issue Description

Price getting overlapped in a screen when INR is selected.

Bagisto Version

[x] master branch
[x] 1.3.3
[x] 1.3.2
[x] 1.3.1
[x] 1.3.0

Steps To Reproduce

Check the homepage, cart wherever the price is there.

Actual Result

Have a look,

Expected Result

Should not be overlapped.

@devansh-webkul I want to work on it

Sure!

Thanks, @devansh-webkul  I just checked it from demo of bagisto live, But I didn't found any overlapped, can you show me via screenshot in which screen its getting overlapped

I already attached a screenshot, maybe this seems to be an issue with my browser.

I am closing this, if anyone found this. Feel free to comment I will reopen this.


Cap maximal cases per project/cohort to 100

Is your feature request related to a problem? Please describe.
We cannot resolve #309 in time for v1.x. We need a workaround.
Describe the solution you'd like
We should cap the maximal number of cases in a project/cohort that one can create project-wide/cohort-wide queries for to a value of 100.
Describe alternatives you've considered
N/A
Additional context
N/A

Implementation Proposal
In the code for the project and cohort filtration, do not display the filter form if there are more than 100 cases but rather tell user to wait for a future version.
Affected Components
VarFish Server.
Affected Modules/Files

variants

Required Architectural Changes

None

Implementation Sketch

None


Error When Using MAPI on a VPC With Greater Than 50 Subnets

ISSUE:
Currently, the IBM Cloud VPC API sets a default to return only 50 subnets per page on an API call if an option is not set on the API call to return more. However, a single VPC can have up to a maximum of 100 subnets.
If a user configures their machine provider config to look for a subnet that is greater than 50 in the returned list, the MAPI will return an error since it's using the default value of 50 provided by the VPC API.
POSSIBLE SOLUTIONS:

When using the IBM Cloud VPC API to return a subnet list,  set the default limit to 100 so that all possible subnets are returned for the VPC. Such as:
subnetOption.SetLimit=100
https://cloud.ibm.com/apidocs/vpc/latest#list-subnets

Support pagination if we choose to only return 50 results with the default setting

Additional Information:
Here is the code where I believe we could set subnetOption.SetLimit=100 to return the full result. This is also the code where the error is thrown if a subnet name provided in the machine provider config is over 50 in the returned list:
func (c *ibmCloudClient) GetSubnetIDbyName(subnetName string, resourceGroupID string) (string, error) {
	// Initialize List Subnets Options
	subnetOption := c.vpcService.NewListSubnetsOptions()

	// Set Resource Group ID
	subnetOption.SetResourceGroupID(resourceGroupID)

    // Set Limit to 100, ibmcloud limits 100 subnets per vpc <======.  add these 2 lines is my guess
    subnetOption.SetLimit(100)

	// Get a list of all subnets
	subnetList, _, err := c.vpcService.ListSubnets(subnetOption)
	if err != nil {
		return "", err
	}

	if subnetList != nil {
		for _, eachSubnet := range subnetList.Subnets {
			if *eachSubnet.Name == subnetName {
				// Return Subnet ID
				return *eachSubnet.ID, nil
			}
		}
	}
	return "", fmt.Errorf("could not retrieve subnet id of name: %v", subnetName)
}

Link: https://github.com/openshift/machine-api-provider-ibmcloud/blob/8c9ac909b36e619d75f59b073b4b056e5270ec54/pkg/actuators/client/client.go#L531

Thanks for the details, I will see about getting a Openshift Jira bug open to resolve this issue.

Jira has been opened to track and resolve this issue.
https://issues.redhat.com/browse/OCPBUGS-36185
I think this issue can be closed (cannot do so myself) in favor of the Jira.


How to mass scale object interaction?

Do I need to create a "chop" event for 100+ trees I have on the map? If there are any resource gathering skills, how would you go about that? I tried generating 100+ trees on Tiled maps and created a single event similar to the chest event, but it didn't seem to scale.
The demo seems to only allow for 1 of 1 interactions to be manually typed.
Also, could the +gold be duplicated into +wood? and +exp to +woodcuting_xp?

Hello,
Thank you for your questions.
Your questions will interest other people, but on Github, the issues are rather thought for feedback on the source code (bug, etc.)
I have now opened a discussion forum that will be rather appropriate for help:
https://community.rpgjs.dev
Can you repost your request on the forum? I'll be happy to answer afterwards (with an example code)
Thanks

Will do thank you!


Array usage is not PowerShell friendly

This syntax is not powershell friendly and should look more like the second example
# working but not powershell(y)
    resource "vnet1"-ApiVersion 2020-08-01 -Namespace Microsoft.Network -Type virtualNetworks {
        properties {
            addressSpace {
                "<IP_ADDRESS>/24", "<IP_ADDRESS>/24", "<IP_ADDRESS>/24" | % {
                    addressPrefixes $_
                }
            }
        }
    }

# doesn't work and should in my opinion
    resource "vnet2" -ApiVersion 2020-08-01 -Namespace Microsoft.Network -Type virtualNetworks {
        properties {
            addressSpace {
                    addressPrefixes @("<IP_ADDRESS>/24", "<IP_ADDRESS>/24", "<IP_ADDRESS>/24")
                    # even better would be this, it not obvious this is an assignment...i know its a function...just weird
                    addressPrefixes  = @("<IP_ADDRESS>/24", "<IP_ADDRESS>/24", "<IP_ADDRESS>/24")
            }
        }
    }

Same...
I can't seem to figure out how to properly format an array...
param(
    [Parameter(Mandatory)]
    [string] $KeyVaultName,

    [Parameter()]
    [ValidateSet('WestEurope', 'NorthEurope', 'WestUs2')]
    [string] $Location = 'WestEurope'
)

$azADGroup = Get-AzADGroup -DisplayNameStartsWith 'platform-kv-01_secrets*' |
ForEach-Object {
    if ($_.DisplayName.EndsWith('_r')) {
        $secrets = @("Get","List")
    }

    if ($_.DisplayName.EndsWith('_rw')) {
        $secrets = @("Get","List","Create", "Delete","Update","Purge")
    }

    if ($_.DisplayName.EndsWith('_f')) {
        $secrets = @("All")
    }

    [PSCustomObject]@{
        ObjectId = $_.id
        Secrets = $secrets
    }
}

Arm {
    Resource $KeyVaultName -Namespace 'Microsoft.KeyVault' -Type 'vaults' -apiVersion '2019-09-01' {
        properties {
            tenantId (subscription).tenantId
            sku {
                family 'A'
                name 'standard'
            }
            accessPolicies { $azADGroup[0] | Foreach-Object {
                    tenantId (subscription).tenantId
                    objectId $($_.ObjectId)
                    permissions { $_.Secrets | Foreach-Object {
                            secrets $($_)
                        }
                    }
                }
            }
        }
    }
}

I can only generate one $azAdGroup entry at a time...
The JSON file seems fine as far as I can tell for the one entry... I have three azAdGroups...
{
  "$schema": "https://schema.management.azure.com/schemas/2019-04-01/deploymentTemplate.json#",
  "contentVersion": "<IP_ADDRESS>",
  "metadata": {
    "_generator": {
      "name": "psarm",
      "version": "<IP_ADDRESS>",
      "psarm-psversion": "7.1.3",
      "templateHash": "13553475952233288655"
    }
  },
  "resources": [
    {
      "name": "keyvault",
      "type": "Microsoft.Resources/deployments",
      "apiVersion": "2019-10-01",
      "properties": {
        "mode": "Incremental",
        "expressionEvaluationOptions": {
          "scope": "inner"
        },
        "template": {
          "$schema": "https://schema.management.azure.com/schemas/2019-04-01/deploymentTemplate.json#",
          "contentVersion": "<IP_ADDRESS>",
          "resources": [
            {
              "name": "platform-kv-01",
              "apiVersion": "2019-09-01",
              "type": "Microsoft.KeyVault/vaults",
              "location": "westeurope",
              "properties": {
                "tenantId": "[subscription().tenantId]",
                "sku": {
                  "family": "A",
                  "name": "standard"
                },
                "accessPolicies": [
                  {
                    "tenantId": "[subscription().tenantId]",
                    "objectId": "d427a0ec-ac40-48cd-b74d-56edd3f6945b",
                    "permissions": {
                      "keys": [
                        "Get",
                        "List"
                      ]
                    }
                  }
                ]
              }
            }
          ]
        }
      }
    }
  ]
}

I haven't seen any AzureAd resource in bicep... Did notice that Terraform has one 😉 If I can use PowerShell to leverage getting  the objectId... Just think of the possibilities... 💯

I talked about this with @irwins  and found the following syntax does work:
param(
    [Parameter(Mandatory)]
    [string] $KeyVaultName,

    [Parameter()]
    [ValidateSet('WestEurope', 'NorthEurope', 'WestUs2')]
    [string] $Location = 'WestEurope'
)

$azADGroup = Get-AzADGroup -DisplayNameStartsWith 'platform-kv-01_secrets*' |
ForEach-Object {
    if ($_.DisplayName.EndsWith('_r')) {
        $secrets = @("Get", "List")
    }

    if ($_.DisplayName.EndsWith('_rw')) {
        $secrets = @("Get", "List", "Create", "Delete", "Update", "Purge")
    }

    if ($_.DisplayName.EndsWith('_f')) {
        $secrets = @("All")
    }

    [PSCustomObject]@{
        ObjectId = $_.id
        Secrets  = $secrets
    }
}

Arm {
    Resource $KeyVaultName -Namespace 'Microsoft.KeyVault' -Type 'vaults' -apiVersion '2019-09-01' {
        properties {
            tenantId (subscription).tenantId
            sku {
                family 'A'
                name 'standard'
            }
            $azADGroup | Foreach-Object {
                accessPolicies {
                    tenantId (subscription).tenantId
                    objectId $($_.ObjectId)
                    permissions { $_.Secrets | Foreach-Object {
                            secrets $($_)
                        }
                    }
                }
            }
        }
    }
}

I do feel this is counter intuitive syntax and in my opinion the other suggestion by @rdbartram and Irwins original syntax makes a lot more sense..

It may seem counter intuitive but the end Access Policy is definable only to a single objectID not an array of ObjectId's so you can only ever have it as you have above as per the Access Policy docs at https://docs.microsoft.com/en-us/azure/templates/microsoft.keyvault/vaults/accesspolicies?tabs=json#accesspolicyentry-object
Which ultimately is a limitation in ARM which @bmoore-msft / @alex-frankel may be able to help nudge the KV team to update the objectID of the AccessPolicyObject to allow an array in a future api-version update

In the docs you linked I see AccessPolicyEntry, which is an object, and accessPolicies, which is an array.
The PSARM syntax in this case looks like it is is creating multiple instances of the property accessPolicies. In my opinion it makes more sense if it is making multiple instances of AccessPolicyEntry and combine them into the array accessPolicies. A lot like what @irwins tried to do.
That is already happening in bicep, where the syntax to create a for loop would look something like this:
accessPolicies: [for accessPolicy in accessPolicies: {
      tenantId: tenantId
      objectId: accessPolicy.objectId
      permissions: {
        keys: accessPolicy.keys
        secrets: accessPolicy.secrets
        certificates: accessPolicy.certificates
      }
 }]

I would expect something similar to this to be an option for PSArm.
Same goes for the example by @rdbartram, where addressPrefixes is an array, not an object.

@Ba4bes Gave me an idea to try...
{
    "$schema": "https://schema.management.azure.com/schemas/2019-04-01/deploymentParameters.json#",
    "contentVersion": "<IP_ADDRESS>",
    "parameters": {
        "keyVault": {
            "value": {
                "name": "platform-kv-01",
                "sku": {
                    "family": "A",
                    "name": "standard"
                },
                "accessPolicies": [
                    {
                        "objectId": "92f011f6-697c-41a2-8584-defca729382f",
                        "permissions": {
                            "keys": ["All"],
                            "secrets": ["All"],
                            "certificates": ["All"]
                        }
                    },
                    {
                        "objectId": "d427a0ec-ac40-48cd-b74d-56edd3f6945f",
                        "permissions": {
                            "keys": ["Get","List"]
                        }
                    },
                    {
                        "objectId": "b6555562-e39c-407a-b846-9b3f1878df45",
                        "permissions": {
                            "keys": ["Get","List"],
                            "secrets": [ "All"],
                            "certificates": ["All"]
                        }
                    }
                ]
            }
        },
        "location": {
            "value": "westeurope"
        }
    }
}

// Parameter definitions
@description('KeyVault properties')
param keyVault object

@description('Resource location, defaults to resourceGroup location')
param location string = resourceGroup().location

// Resources
resource kv 'Microsoft.KeyVault/vaults@2021-04-01-preview' = {
  name: keyVault.name
  location:location
  properties: {
    tenantId: subscription().tenantId
    sku: keyVault.sku
    accessPolicies: [for accessPolicy in keyVault.accessPolicies: {
      tenantId: subscription().tenantId
      objectId: accessPolicy.objectId
      permissions: accessPolicy.permissions
    }]

  }
}

//Output definitions
output keyVaultUri string = kv.properties.vaultUri

Building this resulted in the following JSON file (Gotta love bicep for authoring...)
{
  "$schema": "https://schema.management.azure.com/schemas/2019-04-01/deploymentTemplate.json#",
  "contentVersion": "<IP_ADDRESS>",
  "metadata": {
    "_generator": {
      "name": "bicep",
      "version": "<IP_ADDRESS>766",
      "templateHash": "10484560537251782542"
    }
  },
  "parameters": {
    "keyVault": {
      "type": "object",
      "metadata": {
        "description": "KeyVault properties"
      }
    },
    "location": {
      "type": "string",
      "defaultValue": "[resourceGroup().location]",
      "metadata": {
        "description": "Resource location, defaults to resourceGroup location"
      }
    }
  },
  "functions": [],
  "resources": [
    {
      "type": "Microsoft.KeyVault/vaults",
      "apiVersion": "2021-04-01-preview",
      "name": "[parameters('keyVault').name]",
      "location": "[parameters('location')]",
      "properties": {
        "copy": [
          {
            "name": "accessPolicies",
            "count": "[length(parameters('keyVault').accessPolicies)]",
            "input": {
              "tenantId": "[subscription().tenantId]",
              "objectId": "[parameters('keyVault').accessPolicies[copyIndex('accessPolicies')].objectId]",
              "permissions": {
                "keys": "[parameters('keyVault').accessPolicies[copyIndex('accessPolicies')].keys]",
                "secrets": "[parameters('keyVault').accessPolicies[copyIndex('accessPolicies')].secrets]",
                "certificates": "[parameters('keyVault').accessPolicies[copyIndex('accessPolicies')].certificates]"
              }
            }
          }
        ],
        "tenantId": "[subscription().tenantId]",
        "sku": "[parameters('keyVault').sku]"
      }
    }
  ],
  "outputs": {
    "keyVaultUri": {
      "type": "string",
      "value": "[reference(resourceId('Microsoft.KeyVault/vaults', parameters('keyVault').name)).vaultUri]"
    }
  }
}

it worked as expected...

I can always update the parameter file JIT with PowerShell 😉


Add option to provide default content-type option

This patch add option to provide default content type to serve with. There is no point for text-file server to return octet-stream content, let it define its default type.

You wanna try throwing down some tests for this? Should be straightforward---the other tests use the default, so just write one that spins up a server with text/plain set as the default and assert you get what's expected.

Yes, I'll add tests & and docs in this patch a little later.

Ok, I spent 4 hours for thing I'll not use (I wrote more hardcore and simple patch for my purpose), found bug in code (mime always return something default, so charset always will be set and I'm not sure it is expected behaviour), lot of duplicated code, similar differently-written code.
I currently have no idea why this crutched test framework, who redirects standard output and throws it away leaving no opportunity to quick debugging except throwing, fails on test 3 with timeout only on node 0.10. I also have no idea what is test 3, I defined only 2 of them.
It is up to you what to do with this pull request, I can add more fixes if you suggest the way to do it, but I want to be user, not developer of this thing, and have no intention of knowing how it works.

I'm not really in a position to console you over your frustration with node-tap, but writing the test myself was easy enough.
Thanks for your contribution, it's available in<EMAIL_ADDRESS>


Fix flaky node labeller test

What this PR does / why we need it:
Make sure we do not override the default map
Which issue(s) this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close the issue(s) when PR gets merged):
Fixes #
Special notes for your reviewer:
Checklist
This checklist is not enforcing, but it's a reminder of items that could be relevant to every PR.
Approvers are expected to review this list.

[ ] Design: A design document was considered and is present (link) or not required
[ ] PR: The PR description is expressive enough and will help future contributors
[ ] Code: Write code that humans can understand and Keep it simple
[ ] Refactor: You have left the code cleaner than you found it (Boy Scout Rule)
[ ] Upgrade: Impact of this change on upgrade flows was considered and addressed if required
[ ] Testing: New code requires new unit tests. New features and bug fixes require at least on e2e test
[ ] Documentation: A user-guide update was considered and is present (link) or not required. You want a user-guide update if it's a user facing feature / API change.
[ ] Community: Announcement to kubevirt-dev was considered

Release note:

NONE

/cherry-pick release-1.0
/cherry-pick release-0.59
/cherry-pick release-0.58
/cc @brianmcarey @dhiller
This was caught by check-tests-for-flakes

/cc @jean-edouard @acardace
Can you have a look?

/kind flake


Add SameSite Cookie Property

Hi,
This descripes the request: https://github.com/aspnet/Security/issues/908
Please add this for .net framework 4.7.x
Thanks

Hello,
Is there any news about this feature request? For us this would also be a highly appreciated feature

Any update on this?
According to this comment: https://stackoverflow.com/questions/38954821/preventing-csrf-with-the-same-site-cookie-attribute#comment88140579_38957177 it's already built in .NET Framework 4.7.2 but it doesn't work for me?!

This still appears to be missing. If you use a DelegatingHandler to manually set a cookie in 4.7.2, you must add a CookieHeaderValue using response.Headers.AddCookies(). But CookieHeaderValue still doesn't have a SameSite property. Chrome 80 is due in less than a month so this is going to start being a problem for people very soon.

@seanwm Comments on closed issues are not tracked, please open a new issue with the details for your scenario.
You seem to be referring to a different set of APIs than the rest of this issue, specifically the System.Net.Http.Formatting APIs. CookieHeaderValue is defined in its own package over at https://github.com/aspnet/AspNetWebStack/blob/master/src/System.Net.Http.Formatting/Headers/CookieHeaderValue.cs, it's not part of the .NET or .NET Core runtimes. Opening an issue in AspNetWebStack would be a good start.
How do you anticipate SameSite impacting your WebApi scenario? WebApi's A) largely don't use cookies, and B) don't do the kind of redirects that would trigger SameSite protections.

@Tratcher

Specifically to https://msdn.microsoft.com/en-us/library/system.net.cookie(v=vs.110).aspx in support of CookieContainer, HttpWebRequest, and HttpClient?

For me, yes. Not trying be lazy here, happy to open a new issue if that's the right course of action. Unless you happen to know a quick answer as to why there's still no SameSite property for System.Net.Cookie on any platform?


Typo in the NCA100v4 public page - need correction.

Could you please correct the typo in the following document?.
Due to increased GPU memory I/O footprint, the NC A100 v4 requires the use of Generation 2 VMs and marketplace images. While the Azure HPC images are strongly recommended, Azure HPC Ubuntu 18.04, 20.04 and Azure HPC CentOS 7.9, CentOS 8.4, RHEL 7.9, RHEL 8.5, Windows Service 2019, and Windows Service 2022 images are supported.
https://learn.microsoft.com/en-us/azure/virtual-machines/nc-a100-v4-series#supported-features
It should be Windows Server 2019 and Windows Server 2022.

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

ID: 643130de-3e16-4acb-7472-7d9144bcd76d
Version Independent ID: 03974480-4b55-728a-5b7a-5b09dded522a
Content: NC A100 v4-series - Azure Virtual Machines
Content Source: articles/virtual-machines/nc-a100-v4-series.md
Service: virtual-machines
Sub-service: sizes
GitHub Login: @sherrywangms
Microsoft Alias: sherrywang

@vinil-v
Thanks for your feedback! We will investigate and update as appropriate.

@vinil-v
Thanks for reporting this! We have created a PR for this issue and the changes should go live soon.


Prevent auto restart on LibreOffice install

Earlier LibreOffice manifests have these flags which prevent automatic system restarts on installation.
See also previous fixups 92ca7106023d0e2b1fa536f6502a0502ab3f06f9 and 1a5c1039ffb73edad007fd9ecaff8b08e335f1b3
and their related PRs and issue tickets.

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

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

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

Microsoft Reviewers: Open in CodeFlow

ops forgot to post comment
You can only change 1 version per PR

Is the multiple manifests why the bot claims the path is not correct?

yes it is

Superseded by #47070, #47071, #47072.


Simplify Decisioning

Remove all decisioning/curatorial reductions like "isAnimal" and primary/secondary face considerations

isAnimal  @markziler is actually just a aggregate counter for the object detection of cat, dog or horse.  the full object detection output is already included in the json output.

all the face details are rendered out too.
a client / user can determine if they do not like the APIs summary of primary and secondary faces.
gender is looked at only for the first and second (primary and secondary face)... the resolution of gender is highly discouraged.  the output currently provides a two salience values that can be used as a gradient.  the resolution of salience to tag is discouraged for almost any real use due to model bias and the cultural sensitivity around gender and the very real issue that facial features do not accurately represent gender.


Replace deprecated FindPythonLibs in CMakeLists.txt

The top level CMakeLists.txt calls find_package(PythonLibs...) which according to https://cmake.org/cmake/help/latest/module/FindPythonLibs.html is deprecated as of CMake 3.12, to be replaced by FindPython2 / FindPython3 / FindPython.

Sounds like a good suggestion. FindPython isn't available prior to CMake 3.12. Linux distros still commonly in use bundle old versions of cmake. Ubuntu 16.04 as an example bundles 3.5.1. Some logic would need to be added that detects cmake version and calls the appropriate find function. PR's welcome.

I wish VFXPlatform or ASWF would pin down a minimum CMake version that we can all count on.

I agree that a minimum CMake baseline would make sense, definitely in the context of ASWF, and I'll raise the topic on the reference platform mailing list. I'll put together a proposed PR with conditional CMake code.

Did anything come of this?
OpenEXR currently supports installation via the gnu autotools as well as CMake, so one option would be to choose a more recent version of CMake, and any distros that can't deal with that could install  via autotools.

That would work for linux, but not windows; autotools is available for mac, but it is not a convenient thing to set up. I'd be interested in some conditional CMake as proposed by @jfpanisset - I don't think I see a PR for it though?

This has been implemented. We are using cmake 3.12 which has built-in support for finding both python2 and python3 at the same time, as commented, and users will have to update their versions of cmake.


User can't edit in sprite editor in JavaScript editor on iPad

Describe the bug
User can't edit in sprite editor in JavaScript editor on iPad
Steps to reproduce the behavior

Navigate to https://arcade.makecode.com/beta
Import below project and click "JavaScript" to switch to JavaScript editor
https://makecode.com/_2DRg6vHVvV5i
Click little painter icon
Try to edit on the editor and gallery

Expect behavior
User should can edit in the sprite editor
Actual behavior
User can't edit in sprite editor in JavaScript editor on iPad
Additional context

OS: ipad(11.4)
arcade version: 0.14.4
Microsoft MakeCode version: 5.21.6

Seems to be fixed with new sprite editor. Tested on ios 12.4 and ios 13.3 simulators


Mars clock - tweak am/pm suffix font

Description
Pull the am/pm suffix into its own widget with separate font
Copilot

copilot:all

:shipit:


Don't start httpd when the embedded ansible role is assigned

We don't expose a webserver for this now that we've moved from AWX
to runner.
This essentially reverts 5a18a2e82affa524a25b3e022c6144df9e95fd75

Ivanchuk backport details:
$ git log -1
commit 6b53aefcb8ecf2d4b4dca1a36c763ca32b32673f
Author: Brandon Dunne<EMAIL_ADDRESS>Date:   Wed Jul 24 11:14:40 2019 -0400

    Merge pull request #19043 from carbonin/dont_require_apache_for_embedded_ansible

    Don't start httpd when the embedded ansible role is assigned

    (cherry picked from commit 82fe63826beb4fa43a472fc93d7ee3fd9e800e6e)


Created Code of conduct.md file

PR Description: Add CODE_OF_CONDUCT.md to CopilotKit Repository
Issue:  #657
This PR addresses the addition of a Code of Conduct to the CopilotKit repository. A Code of Conduct helps to set expectations for behavior, ensuring a welcoming and harassment-free environment for all contributors.
Changes Included:

Added a CODE_OF_CONDUCT.md file to the repository.
The Code of Conduct outlines the following sections:

Statement of Purpose: Explanation of the goal to foster a welcoming and respectful environment.
Expected Behavior: Examples of positive contributions to the community.
Unacceptable Behavior: Clear guidelines on what constitutes unacceptable behavior.
Responsibilities of Maintainers: Details on the roles of project maintainers in enforcing the Code of Conduct.
Scope: Explanation of when and where the Code of Conduct applies.
Enforcement Procedures: Instructions on how to report issues and how those reports will be handled.
Attribution: This Code of Conduct is adapted from the Contributor Covenant, version 1.4.

Expected Impact:
The introduction of this Code of Conduct will:

Help maintain a safe and inclusive environment for all contributors.
Clearly communicate the standards of behavior expected within the community.
Provide a defined process for handling misconduct or inappropriate behavior.

Additional Context:
This is a standard practice for open-source projects, aligning the CopilotKit repository with industry standards and supporting a positive contributor experience.
Review Notes:

The Code of Conduct follows the guidelines from the widely adopted Contributor Covenant, which many open-source projects use.
Maintainers are encouraged to review and make any necessary adjustments to reflect the unique needs of the CopilotKit community.

fixes #657


Flask 3.0.0 breaks the web interface

Web server crashes on launch with flash 3, downgraded to flask 2.3.3 and it works again :)
Crash logs:
 [uWSGI] getting INI configuration from uwsgi_shared.ini
 *** Starting uWSGI 2.0.22 (64bit) on [Wed Oct  4 11:40:04 2023] ***
 compiled with version: 11.4.0 on 04 October 2023 15:35:54
 os: Linux-5.15.0-84-generic #93-Ubuntu SMP Tue Sep 5 17:16:10 UTC 2023
 nodename: localhost
 machine: x86_64
 clock source: unix
 pcre jit disabled
 detected number of CPU cores: 1
 current working directory: /opt/pajbot
 detected binary path: /opt/pajbot/venv/bin/uwsgi
 your processes number limit is 7403
 your memory page size is 4096 bytes
 detected max file descriptor number: 1024
 lock engine: pthread robust mutexes
 thunder lock: disabled (you can enable it with --thunder-lock)
 uwsgi socket 0 bound to UNIX address /var/run/pajbot/pepto/web.sock fd 3
 Python version: 3.10.12 (main, Jun 11 2023, 05:26:28) [GCC 11.4.0]
 PEP 405 virtualenv detected: venv
 Set PythonHome to venv
 Python main interpreter initialized at 0x562da44f8780
 python threads support enabled
 your server socket listen backlog is limited to 100 connections
 your mercy for graceful operations on workers is 60 seconds
 mapped 145840 bytes (142 KB) for 1 cores
 *** Operational MODE: single process ***
 Traceback (most recent call last):
   File "/opt/pajbot/./app.py", line 20, in <module>
     pajbot.web.init(args)
   File "/opt/pajbot/./pajbot/web/__init__.py", line 56, in init
     import pajbot.web.common
   File "/opt/pajbot/./pajbot/web/common/__init__.py", line 1, in <module>
    from . import assets as assets
   File "/opt/pajbot/./pajbot/web/common/assets.py", line 3, in <module>
    from flask_assets import Bundle, Environment
   File "/opt/pajbot/venv/lib/python3.10/site-packages/flask_assets.py", line 8, in <module>
     from flask import _request_ctx_stack, current_app
ImportError: cannot import name '_request_ctx_stack' from 'flask' (/opt/pajbot/venv/lib/python3.10/site-packages/flask/__init__.py)
unable to load app 0 (mountpoint='') (callable not found or import error)
*** no app loaded. GAME OVER ***
 VACUUM: unix socket /var/run/pajbot/pepto/web.sock removed.

@MagicHack Can you try out this PR? #2571

Works perfectly :) thanks for the super quick fix


To get it working behind a proxy:

In the simulated device:
Http1TransportSettings transportSettings = new Http1TransportSettings();
transportSettings.Proxy = new WebProxy("MyProxy", MyProxyPort);
ITransportSettings[] transportSettingsArray = new ITransportSettings[] { transportSettings };
        // Connect to the IoT hub using the MQTT protocol
        s_deviceClient = DeviceClient.CreateFromConnectionString(s_connectionString, transportSettingsArray); // TransportType.Mqtt);

In read-d2c-messages:
var connectionString = new EventHubsConnectionStringBuilder(new Uri(s_eventHubsCompatibleEndpoint), s_eventHubsCompatiblePath, s_iotHubSasKeyName, s_iotHubSasKey);
connectionString.TransportType = TransportType.AmqpWebSockets;
s_eventHubClient = EventHubClient.CreateFromConnectionString(connectionString.ToString());
s_eventHubClient.WebProxy = new WebProxy("MyProxy", MyProxyPort);

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

ID: 666d1390-a995-fb5f-e1ee-49e28c50e901
Version Independent ID: d699e566-83d8-5af9-56a5-4baa56ae0455
Content: Send telemetry to Azure IoT Hub quickstart (C#)
Content Source: articles/iot-hub/quickstart-send-telemetry-dotnet.md
Service: iot-hub
GitHub Login: @robinsh
Microsoft Alias: robinsh

Hi @shindigo sounds like you are requesting us to add this to the doc so it addresses the behind proxy scenario?

@shindigo Thanks for the feedback! I have assigned the issue to the content author to evaluate and update as appropriate.

Hi Alberto - that's right.  I wanted to make sure that anyone trying to do
this from behind a proxy was not getting stuck like I was.
On Wed, Sep 25, 2019 at 6:59 PM Alberto Vega<EMAIL_ADDRESS>wrote:

@shindigo https://github.com/shindigo Thanks for the feedback! I have
assigned the issue to the content author to evaluate and update as
appropriate.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/MicrosoftDocs/azure-docs/issues/39536?email_source=notifications&email_token=ACXUNIV3FGHENC6ET7JQYOLQLPUNZA5CNFSM4I2QUHK2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD7TV2DY#issuecomment-535256335,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ACXUNIWVDE7PZ43R7EEFVRLQLPUNZANCNFSM4I2QUHKQ
.

@shindigo First thanks very much for your feedback!
For the device side, we recommend switching from MQTT  (port 8883) to MQTT over Websockets (port 443). Of course, if you want to use another transport (HTTPS), you can do so. We've added a prerequisite in all of our quickstarts, tutorials, and how-to's that use MQTT to point out the issue with the blocked port and direct folks to our MQTT topic for ways to work around it. For an example see the last paragraph in the c# send telemetry quickstart prereqs section.
You also raise the issue wrt to the EventHubsClient in the service sample. I'll discuss this with our product team and get back to this thread as soon as I have a response.
Thanks again,
Jimaco

@jlian Can you please have a look wrt the service sample here? Let's discuss it offline.

@rezasherafat knows better than I do here - can you take a look?

The quickstart doc is already way too long. I think the best place to add the proxy configuration is either in the csharp SDK docs itself, or as commented code in the quickstart .cs files, e.g., here for service and here for device. I do see that the overrides already exist.
@ericmitt, can you help with the updates if you agree?

@shindigo -- just an update. I have reached out to our SDK teams and they have a work item to produce samples to demonstrate proxies on our supported platforms. Appreciate your feedback and patience here.

#reassign @robinsh

#reassign JimacoMS4

Hi @shindigo - Thanks again for your feedback. The quickstart you filed this item against has been refactored and no longer calls the service API to read from the built-in endpoint.; however, we did publish samples to show how to read from the endpoint using either AMQP-WS and/or by reading through a proxy using the Event Hubs SDKs. We also published pointers to those samples in this topic:: Read device-to-cloud messages from the built-in endpoint: Use AMQP-WS or a proxy with Event Hubs SDKs.
On the device-side, we used to have a bulletpoint in prerequisites that explicitly mentioned that the sample used MQTT and required port 8883 to be open. If not it pointed folks to a topic that contained mitigation strategies. I'll do a little more investigating here and see if we can add this to the recently refactored quickstarts.
-Jimaco

Hi @shindigo.
Since you filed this issue, we now have proxy samples available for several of our device SDKs. For example:

Node.js
Python

It's been some time since you opened this issue, and, as my last two comments indicated, efforts have been made to add support both on the service-side and the device-side to provide proxy samples. At this time, I'm going to close this issue. Thank you for taking the time to bring this to our attention. If you feel that there is still information that needs to be addressed, I suggest opening an issue against our Device developer documentation as this is where our most of our device-related content is currently located. You could also open an issue against the appropriate SDK if you feel that sample content is lacking. Thanks again for your help!
-Jimaco

#please-close


EnsureJoin is too expensive

It will send a state event each time it is called, which will usually no-op but that's quite unacceptable at scale.

That's kind of the point of EnsureJoin, though, it is the last fallback if anything else fails.

#308 showed how this was supposed to work / EnsureJoin was horribly worded


Tranning Needed memory size

Hi,
My computer have 23GB memory, when I start trainning repr_table6_h36m_model.yaml, I can only load needed 2d data and 3d data.
How can I do? Can I separate 3D data and train stage by stage? are you still remeber your experiment memory?

Can you tell me about your trainning time?

Hi, it took approx 55GB.
My recommendation is to discard 1/3 - 1/2 of the whole dataset in a random manner and train with the remaining data. It would give similar accuracy

Thanks for your quick reply!


Add documentation for interface inheritance

Keeping this as a draft since we need to figure out where to merge.
Same as (approved): https://github.com/onflow/docs/pull/87

@turbolent I guess we can merge this to master now?


Filtered articles should not be built

In my config.rb, I filter the articles with the filter param:
activate :blog do |blog|
  blog.filter = proc { |article|
    article.tags.include? target.to_s
  }
end

The filter works correctly when building the blog index but all articles are built regardless of the filter. It would be more logical to only build the filtered articles.

As an alternative, I was able to this in Middleman V3:
ready do
  blog.articles.each do |article|
    if !article.tags.include?(target.to_s)
      ignore article.path
    end
  end
end

But the blog variable is no longer accessible from the config.rb.

Hiya
You can utilise
sitemap.resources.select{ | d | d.data.tags ....  }.each do | resource |
end

in config.rb
Potentially in conjunction with something like
current_page.is_a? Middleman::Blog::BlogArticle

Not tested but see how that works
I need to investigate the usage for the filter some more
If you can help then please provide some Features / Fixtures tests for this feature


New: add image dimming transform

https://phabricator.wikimedia.org/T169151

for now uses opacity at 65% as outlined in the ticket

Animation of the demo:

:+1: looks good, just crazy trivial stuff inline.

:+1:

Published in v4.5.0.


Newline does not work as expacted

I opened a new windows terminal and made sure oterm is not running anywhere else before.
I am running oterm 0.5.2
Problems:

Clicking newline in multiline mode places newline in the wrong place (from second line onwards)

Pressing shift+enter sends data to ollama

newline position problems
a. Happens only on and after second line. Did not find what are the rules of placement of newline
b. Does not matter if ascii or unicode is used.

https://github.com/user-attachments/assets/6440fefc-01cb-48fa-8719-63f6ca25d0ba

I am away for a short holiday, but I can confirm this is an issue. Will fix asap.

Shift enter is still submitting the message for me here on macOS with v0.6.4.  I don't think it did this until I upgraded to 0.6 recently.

@danyalaytekin Can you please let me know which terminal you use? It works fine for me on iTerm.
Also, are you in multi-line mode when you press shitft+enter?

Hi @ggozad, this was in the stock macOS 14.6.1 terminal. I've just tested in iTerm too. I should probably switch over entirely but it has never quite caught on... anyway, my results:

mode
Terminal
iTerm

normal
submits
nothing happens

multiline
submits
newline

I found a similar issue. It works fine in Kitty with no multiplexer.
But in Kitty with tmux is submits and in Wezterm it also submits.
Thanks

Thank you all for reporting how this works across environments.
I believe fixing this properly is unfeasible as I have to rely on textual's handling of keys and my guess is that it is difficult to do it properly across varied environments.
An alternative would be to allow configurable keymaps. This was introduced in the latest textual release a few days back, I am gonna investigate whether it's a good alternative.
If there are any takers: https://darren.codes/posts/textual-keymaps/

https://github.com/ggozad/oterm/pull/131 allows customizations to key bindings. See the README for how to do that.
If this works for you, I will release asap.

had to use ctrl+enter, on nixos tmux alacritty


fixes issue#729

fixes issue https://github.com/opensds/opensds/issues/729

@wisererik @leonwanghui


[Sink Request] Splunk

Completed in https://github.com/silverton-io/buz/commit/cb2ed3dc384a0d0970502a36bda58d55054128cb


feat: add bubblezone to readme (closes #368)

also fixed a handful of grammar-related items in the readme.
closes: https://github.com/charmbracelet/bubbletea/issues/368

Thanks for the PR! I've move the BubbleZone mention to the main resource section (next to Bubbles, Harmonica, etc). Thanks for the grammar work, too! Once CI finishes up I'll go ahead and merge.

Internal note: coveralls is down but I did the merge anyway since this is just a README update.


UNABLE_TO_GET_ISSUER_CERT_LOCALLY after upgrading from 1.7.6 to 2.1.2

After upgrading my node-apn to 2.1.2, I am no longer able to send notifications.
On 1.7.6 my configuration worked.
Node version 4.6.0
NPM version 2.15.9
Error is UNABLE_TO_GET_ISSUER_CERT_LOCALLY.

Note: I don't think I can use the JWT method as I use PushKit notifications (and as such,  a VoIP certificate) and it doesn't seem like the APN auth key supports this

The following sample code reproduces the issue:
var apn = require('apn');

var config = {
  production: false,
  key: 'conf/apn/key.pem',
  cert: 'conf/apn/cert.pem', // This is a VoIP services certificate
  ca: 'conf/apn/ca.pem'
};

var apnProvider = new apn.Provider(config);
var notification = new apn.Notification();
var apnToken = apn.token('token');

notification.topic = 'com.bundle.id';

apnProvider.send(notification, apnToken).then(onNotificationsSent);

function onNotificationsSent (deliveries) {
  deliveries.failed.forEach(deliveryFail);

  function deliveryFail (failure) {
    // endpoint error: unable to get local issuer certificate when sending to 'token'
    console.log(`${failure.error.message} when sending to '${failure.device}'`);
  }
}

Config options as seen in Endpoint.prototype._connect (error first thrown at this line). The certificates were found (I've removed the buffer data keys for brevity).
{
  "token": null,
  "cert": {
    "type": "Buffer",
    "data": [...]
  },
  "key": {
    "type": "Buffer",
    "data": [...]
  },
  "ca": [
    {
      "type": "Buffer",
      "data": [...]
    }
  ],
  "pfx": null,
  "passphrase": null,
  "production": false,
  "address": "api.sandbox.push.apple.com",
  "port": 443,
  "rejectUnauthorized": true,
  "connectionRetryLimit": 10,
  "host": "api.sandbox.push.apple.com",
  "servername": "api.sandbox.push.apple.com",
  "ALPNProtocols": [
    "h2"
  ]
}

It appears my problem has been resolved by removing the ca.pem from config and not setting notification.topic.
Removing ca.pem halted the error and resolved an an HTTP 400 with reason TopicDisallowed. Not sure whats up with this as I don't have the CA included in my cert.pem and I thought topic was a required value, but maybe VoIP is different.

topic is required for JWT authentication only.
ca is used to verify Apple's server certificate. In general you don't need to set ca, because your system already trusts Apple's certificate.

Thank you :)


GEOMESA-1487 proper generation of accumulo runtime jars for scala 2.10 build

looked over build, issue is isolated to geomesa-accumulo-distributed-runtime

Looks good to me


cargo package does not always work if used with 2018 edition

To reproduce create a project with the following Cargo.toml:
cargo-features = ["edition"]

[package]
name = "example"
version = "0.0.0"
authors = ["Example<EMAIL_ADDRESS>
edition = "2018"

[package.metadata.docs.rs]
features = ["foobar"]

and an empty file as src/lib.rs.
When you try to package this will happen:
orium@laptop-orion cargofixbug % cargo package --allow-dirty --verbose
[...]
   Packaging example v0.0.0 (file:///home/orium/tmp/cargofixbug)
   Archiving .codecov.yml
   Archiving .gitignore
   Archiving .rustfmt.toml
   Archiving .travis.yml
   Archiving Cargo.toml
error: failed to prepare local package for uploading

Caused by:
  values must be emitted before tables

The problem is due to the way the Cargo.toml generated for packaging is serialized.  The edition field is emitted after the metadata table, which is not correct.  PR soon.


Switch Row and Column to flexbox

This PR is stacked on top of #113, so after that PR is merged, this one will become smaller. Meanwhile, the diff between the two PRs can be viewed here.
The Row and Column containers have been changed from display: grid to display: flex. These containers now have the following properties:

align_items
justify_content
flex_wrap

And their children have these properties:

align_self (overrides align_items for an individual child)
flex

The farmyard app has been updated with a couple of examples of things you couldn't do in the previous model:

The number_of_honks label is now on the same row as the slider, with the label taking only as much space as necessary, and the slider taking all the rest.
The geese and pigs now wrap onto multiple lines (previously they overflowed the box)

Following the same pattern, I've also updated the Grid to replace the position property with align_items and justify_items on the container, and align_self and justify_self on the children. This allows the horizontal and vertical position to be set independently using ChoiceProperties (closes #97).
All the Grid, Row and Column child properties are still declared on Component, so that's becoming a bit cluttered. Next I plan to split them out into per-container layout classes, as we discussed the other week.

@ntoll: You asked why we would ever NOT want flex_wrap=True. Here's one example: it causes it to prefer wrapping the children themselves rather than the chlidren's content. Both of these could be useful in different circumstances:

flex_wrap=True:

flex_wrap=False:

In the end, we removed flex_wrap because it potentially complicates changes to drag and drop (#116), and other things  that we haven't had a chance to think through.
We also removed align-items and justify-items, for two reasons:

Having two ways to set alignment in the builder UI is confusing.
We currently need every widget to have an explicit align-self, to override the value of that property that the Paper CSS sets on button, and maybe other widgets. So align-items wouldn't have any effect anyway.


fix: move ffmpeg to correct folder after extract

Fixes: #916
Description
reinstate the replaceFfmpeg code that was removed during cleanup

It needs to be a little bit smarter otherwise it will download ffmpeg every time. we only want to download ffmpeg if it hasn't been downloaded.
your solution will for for now, but ideally it should check if ffmpeg must be downloaded

@ayushmanchhabra hmmm this is still broken
await replaceFfmpeg(platform, nwDir, out);
just ends up copying the zip file over itself. We need some way to get the extracted filename


Feature request: Immediate update visualization in three scenarios

There are two scenarios where it would be nice if the overlay immediately updated:

When toggling the hover-over visualizer.  This would let me easily turn on the pipe visualizer when hovering over a building to understand it.  Currently the visualization only take effect when the hover selection changes, so if I'm hovering over a building when I activate the visualizer I have to move my cursor away then back to see the visualization.
Similarly, it would be nice to immediately remove the current visualization when the hover-over visualization is toggled into the disabled state.
When the full visualizer is active, it would be nice to redraw the visualization immediately when a new building or pipe is placed.

Live entity updates were planned for 2.0 but were postponed because it was taking too long.

also about visualisation, could you make the overlay for machines nicer. i think it would be cool as a frame like when you select a machine just more thick and colored


Show percent increase in last three days

Is your feature request related to a problem? Please describe.
It will be great if it will show what is the percent increase in patient recovered, reported or deceased. Or might also show a graph related to that.
Describe the solution you'd like
It will be great if it shows a graph for the percent increase, which would be very beneficial for maintaining statistics.
Describe alternatives you've considered
It can also filter it according to cities and states.

A graph very similar to the Washington post for corona would be of great help for visualization I guess. That would give us an idea about whether we are approaching a peak or we crossed, etc.


update to 1.3.0

Issue #, if available:
Description of changes:
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

update to 1.3.0


Add Poop

Tested versions
1.0
System information
Windos 5
Issue description
i wish there was poop, maybe make the icon poop XD
Steps to reproduce
open app, notice lack of poop
kill myself
Minimal reproduction project (MRP)
what the heck is MRP?


[Event Request] codeunit 2624 (Stat. Acc. Jnl. Line Post): OnAfterTransferStatisticalAccJournalLineTo

Current code in codeunit 2624 (Stat. Acc. Jnl. Line Post) in Statistical Accounts extension:
local procedure TransferStatisticalAccJournalLineTo(var StatisticalAccJournalLine: Record "Statistical Acc. Journal Line"; var StatisticalLedgerEntry: Record "Statistical Ledger Entry")
begin
    StatisticalLedgerEntry."Statistical Account No." := StatisticalAccJournalLine."Statistical Account No.";
    StatisticalLedgerEntry."Posting Date" := StatisticalAccJournalLine."Posting Date";
    StatisticalLedgerEntry.Description := StatisticalAccJournalLine.Description;
    StatisticalLedgerEntry.Amount := StatisticalAccJournalLine.Amount;
    StatisticalLedgerEntry."Journal Batch Name" := StatisticalAccJournalLine."Journal Batch Name";
    StatisticalLedgerEntry."Dimension Set ID" := StatisticalAccJournalLine."Dimension Set ID";
    StatisticalLedgerEntry."Document No." := StatisticalAccJournalLine."Document No.";
    StatisticalLedgerEntry."Global Dimension 1 Code" := StatisticalAccJournalLine."Shortcut Dimension 1 Code";
    StatisticalLedgerEntry."Global Dimension 2 Code" := StatisticalAccJournalLine."Shortcut Dimension 2 Code";
end;

Please add a OnAfterTransferStatisticalAccJournalLineTo integration event:
local procedure TransferStatisticalAccJournalLineTo(var StatisticalAccJournalLine: Record "Statistical Acc. Journal Line"; var StatisticalLedgerEntry: Record "Statistical Ledger Entry")
begin
    StatisticalLedgerEntry."Statistical Account No." := StatisticalAccJournalLine."Statistical Account No.";
    StatisticalLedgerEntry."Posting Date" := StatisticalAccJournalLine."Posting Date";
    StatisticalLedgerEntry.Description := StatisticalAccJournalLine.Description;
    StatisticalLedgerEntry.Amount := StatisticalAccJournalLine.Amount;
    StatisticalLedgerEntry."Journal Batch Name" := StatisticalAccJournalLine."Journal Batch Name";
    StatisticalLedgerEntry."Dimension Set ID" := StatisticalAccJournalLine."Dimension Set ID";
    StatisticalLedgerEntry."Document No." := StatisticalAccJournalLine."Document No.";
    StatisticalLedgerEntry."Global Dimension 1 Code" := StatisticalAccJournalLine."Shortcut Dimension 1 Code";
    StatisticalLedgerEntry."Global Dimension 2 Code" := StatisticalAccJournalLine."Shortcut Dimension 2 Code";

    OnAfterTransferStatisticalAccJournalLineTo(StatisticalAccJournalLine, StatisticalLedgerEntry);
end;

[IntegrationEvent(false, false)]
local procedure OnAfterTransferStatisticalAccJournalLineTo(var StatisticalAccJournalLine: Record "Statistical Acc. Journal Line"; var StatisticalLedgerEntry: Record "Statistical Ledger Entry")
begin
end;

Please add a OnAfterTransferStatisticalAccJournalLineTo integration event

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


DataflowPipelineJob.waitUntilFinish() crashes when it has created a template.

INFO: Template successfully created.

Exception in thread "main" java.lang.UnsupportedOperationException:
The result of template creation should not be used.
        at org.apache.beam.runners.dataflow.util.DataflowTemplateJob.getJobId(DataflowTemplateJob.java:37)

       at org.apache.beam.runners.dataflow.DataflowPipelineJob.getJobWithRetries(DataflowPipelineJob.java:524)

       at org.apache.beam.runners.dataflow.DataflowPipelineJob.getStateWithRetries(DataflowPipelineJob.java:506)

       at org.apache.beam.runners.dataflow.DataflowPipelineJob.waitUntilFinish(DataflowPipelineJob.java:295)

       at org.apache.beam.runners.dataflow.DataflowPipelineJob.waitUntilFinish(DataflowPipelineJob.java:224)

       at org.apache.beam.runners.dataflow.DataflowPipelineJob.waitUntilFinish(DataflowPipelineJob.java:183)

       at org.apache.beam.runners.dataflow.DataflowPipelineJob.waitUntilFinish(DataflowPipelineJob.java:176)

This is a real error. If a template was created, the job is complete. Instead of crashing by tried to access the job id, as though DataflowPipelineJob doesn't know it made a template, it should instead return successfully. Or perhaps there is another design choice. But just crashes does not make sense. Probably DataflowRunner should not return a DataflowPipelineJob at all in this way.
Imported from Jira BEAM-9337. Original Jira may contain additional context.
Reported by: kenn.

We recently ran into this error when trying to do some after batch cleanup in some of our GCP Dataflow jobs when using a template, and had to revert those changes. Not being able to use waitUntilFinish() to defer cleanup of some of the intermediate resources and do some post pipeline work makes it a bit more difficult to manage these pipelines.

Just faced the same problem with BEAM version 2.51.0, any updates?

Running a pipeline, then running another pipeline or even just some bit of non-pipeline code, when the first is finished, is not possible with this behavior.
Why is this important / valid use case:

Many pipelines end in writing some output somewhere as a terminal state, so you cannot chain another step past it. However, if you want to do something, such as write a marker file, after all output has completed, you must do it after the pipeline ends.
You can only do this if you waitUntilFinish().
If you collect metrics from the run to store them or process them in some way, you probably want to:

    var result = pipeline.run();
    result.waitUntilFinish();
    processMetrics(result.metrics());


Generation of API JSON file from code during build

There is a need to implement an interface that generates swagger JSON file during the build.

We are closing this issue for now, as it seems to be more complicated than we thought. It might be re-opened once the documentation for other management interfaces needs to be created.


Add typemap into shio

Add typemap into shio, moving it in utils::typemap code.
It have :

a no-op hasher (because each TypeId must be unique per build)
restricted to methods used by shio
also the default impl<T> Key for T (with feature "nightly")

Coverage increased (+0.9%) to 80.633% when pulling 3aea8b5eeee9c69bbff50530bda441fed9c3e67a on Meralis40:intern_typemap_base into 033a0b599af8f1564b097e4019883c2aab59c4be on mehcode:master.

@Meralis40 Thanks for being awesome with all the contributions. I pushed this into master with some cleanup.


Relax uniqueness restriction on input to MaxPrioritySearchTree

The code requires all x values to be distinct and all y values to be distinct. This simplifies the algorithm code, but is an unnatural restriction.
Remove this restriction.

Note: x values must still be unique


Use proper steam API library

https://github.com/ValvePython/steam

After further tests, I don't think there is a real benefit from using such library.
Rollback to previous commits.


ProTools Session Info Text: TextEdit Character Encoding

Issue
In Pro Tools during Export → Session Info as Text, if the text encoding is set to TextEdit (which is the default selection), it lossily converts various non-standard/extended characters in text strings (such as Marker name / comments).
Common Examples

Character
Description
Exported
Notes

–
U+2013, En Dash
Ð

—
U+2014, Em Dash
Ñ
Keyboard input character substitution on macOS

’
U+2019, Right Single Quote
Õ
Sometimes when copying from Microsoft Word; should be an apostrophe

…
U+2026, Ellipsis
É
Keyboard input character substitution on macOS

Example 'UTF8' Format
T R A C K  L I S T I N G
M A R K E R S  L I S T I N G
#   	LOCATION     	TIME REFERENCE    	UNITS    	NAME                             	COMMENTS
1   	01:00:00:00  	480000            	Samples  	Test Ellipsis…
2   	01:01:00:00  	3360000           	Samples  	Test Em Dash —
3   	01:02:00:00  	6240000           	Samples  	Test En Dash –
4   	01:03:00:00  	9120000           	Samples  	Right Side Quote’s Not An Apostrophe

Example 'TextEdit' Format (Pro Tool's default when exporting)
T R A C K  L I S T I N G
M A R K E R S  L I S T I N G
#   	LOCATION     	TIME REFERENCE    	UNITS    	NAME                             	COMMENTS
1   	01:00:00:00  	480000            	Samples  	Test EllipsisÉ
2   	01:01:00:00  	3360000           	Samples  	Test Em Dash Ñ
3   	01:02:00:00  	6240000           	Samples  	Test En Dash Ð
4   	01:03:00:00  	9120000           	Samples  	Right Side QuoteÕs Not An Apostrophe

Proposed Solutions

[ ] Add note to documentation suggesting users export using UTF-8 format and not TextEdit format.
[ ] A heuristic could be employed to convert some of the most common lossy conversions back to their original Unicode characters.

Text file encoding is now auto-detected and all extended characters should be decoded correctly regardless whether the text file was exported from Pro Tools using TextEdit (Mac OS Roman) or UTF8 format.
Basic unit tests are implemented but further refining may be necessary in as-yet unforeseen edge cases.


use the adler32 checksum in gfal2 stageout

Our runtime code calculate the adler32 checksum before calling the stageout code. Actually use the checksum in the gfal2 stageout (like we already do for xrdcp stageout).
I have no way to test this myself, syntax rules are based on the xrdcp stageout, which supports the same argument in the same fashion.
@amaltaro , please review and test as convenient

Dirk, what's the difference between L118 and L120? Both use checksum adler32 but only the former does the checksum validation after the transfer? If so, can we simply remove that -k adler32 in L120?
BTW, is it failing at some site(s) or it's "just" to have this additional check on files?

The difference between L118 and L120 is how the reference adler32 checksum is determined. In one case it calculates at source, calculates at destination and compares. The other has a reference specified, which it compares to the destination.
Since we calculate the adler32 checksum in WMAgent runtime code before calling stageout, L118 uses that adler32 checksum. It gives you a tiny bit better protection (since the checksum is calculated earlier) and it should in principle reduce the cpu overheads for the stageout (but that effect is likely somewhat negligible).
L120 is for stageIn, where we do not know the adler32 checksum of the source. It would also trigger if there is no checksum passed to the stageout code (which should never happen).

@amaltaro , good enough feedback ? Is this going to be included in one of the next tests ? Should see some testing at a variety of sites using gfal2 stageout if possible. No real hurry to merge it, would prefer this to be well tested.

@amaltaro, Alan how did we test this before I remember Justas was testing.
If Dirk is confident, we can merge it to include in the new tag and run the test, what do you think?

We better leave it out for now, I didn't test it yet. I know there are more sites out there, but those certainly using gfal are Purdue and CSCS.
It seems production doesn't urgently need it, so I suggest we leave it until me or someone else can have a look at it in real life.

I agree there is no real urgency to merge this and we should test this before. OTOH, I also don't want it to be forgotten and then we make other changes to the gfal2 plugin in the mean time and have to rebase etc.

Adding this patch to vocms0230 right now, such that we can exercise these changes.

Dirk, are you aware of any site that has this option already enabled? Otherwise I'll just send a couple hundred jobs to Purdue and CSCS

The only site I know about that needs the optons is cms@home stageout to the data bridge and we are already using this patch there.
Your tests should just confirm that the patch works for sites that don't use the new option. The stageout command and xrdcp parameters also changed for these sites.

and I had no issues in a couple of workflows, so it's good to get out of my todo list :-D
@ticoann it affects only agent side, feel free to merge it

Thanks Alan


[FEATURE REQUEST] Feature

Feature requested
Add a ToolTip
A ToolTip can be useful for people who often might not be familiar with the skill icon.
For example, we can show a tooltip when we hover on the icon if the icon is for C++ then is should say "C++"

Realized this is a duplicate of #901


Fixes #144: Add settings component (WIP)

Fixes #144
Changes proposed in this pull request:

Add settings component
Added translations for English and Arabic
Improved i18n initializer

@jesusbibieca


Read NetworkConfig from GRPC

Still untested as I couldn't get Vega and Tendermint to communicate locally

This project is no longer maintained.


Builds with cssCodeSplit disabled don't include name of generated CSS file in manifest.json

Describe the bug

When build.cssCodeSplit is set to false, and build.manifest is true, the CSS that's generated is not included in the manifest.json file - which seems necessary for using the Vite backend integration directions, where you're not using the generated dist/index.html (which does link to the generated CSS)

Reproduction

I setup an example repo, which is an app created by running yarn create @vitejs/app, selecting vue and the non-TS variant.  The only change made is to vite.config.js to disable cssCodeSplit and enable manifest generation.
After running npx vite build, a file named style.2bb9ff70.css was generated, but no mention of it appears in dist/manifest.json
The contents of dist/manifest.json are:
{
  "index.html": {
    "file": "assets/index.8d845e67.js",
    "src": "index.html",
    "isEntry": true,
    "imports": [
      "_vendor.642d02b4.js"
    ],
    "assets": [
      "assets/logo.03d6d6da.png"
    ]
  },
  "_vendor.642d02b4.js": {
    "file": "assets/vendor.642d02b4.js"
  }
}

System Info
Output of npx envinfo --system --npmPackages vite,@vitejs/plugin-vue --binaries --browsers:
  System:
    OS: macOS 11.3.1
    CPU: (8) x64 Intel(R) Core(TM) i7-1068NG7 CPU @ 2.30GHz
    Memory: 679.91 MB / 32.00 GB
    Shell: 3.2.57 - /bin/bash
  Binaries:
    Node: 14.15.1 - /usr/local/bin/node
    Yarn: 1.22.10 - /usr/local/bin/yarn
    npm: 6.14.8 - /usr/local/bin/npm
  Browsers:
    Chrome: 91.0.4472.77
    Firefox: 89.0
    Firefox Developer Edition: 62.0
    Safari: 14.1
  npmPackages:
    @vitejs/plugin-vue: ^1.2.3 => 1.2.3
    vite: ^2.3.5 => 2.3.5

Used package manager: yarn
Logs
  vite:config bundled config file loaded in 90ms +0ms
  vite:config using resolved config: {
  vite:config   build: {
  vite:config     target: [ 'es2019', 'edge88', 'firefox78', 'chrome87', 'safari13.1' ],
  vite:config     polyfillDynamicImport: false,
  vite:config     outDir: 'dist',
  vite:config     assetsDir: 'assets',
  vite:config     assetsInlineLimit: 4096,
  vite:config     cssCodeSplit: false,
  vite:config     sourcemap: false,
  vite:config     rollupOptions: {},
  vite:config     commonjsOptions: { include: [Array], extensions: [Array] },
  vite:config     minify: 'terser',
  vite:config     terserOptions: {},
  vite:config     cleanCssOptions: {},
  vite:config     write: true,
  vite:config     emptyOutDir: null,
  vite:config     manifest: true,
  vite:config     lib: false,
  vite:config     ssr: false,
  vite:config     ssrManifest: false,
  vite:config     brotliSize: true,
  vite:config     chunkSizeWarningLimit: 500,
  vite:config     watch: null
  vite:config   },
  vite:config   plugins: [
  vite:config     'alias',
  vite:config     'vite:dynamic-import-polyfill',
  vite:config     'vite:resolve',
  vite:config     'vite:html',
  vite:config     'vite:css',
  vite:config     'vite:esbuild',
  vite:config     'vite:json',
  vite:config     'vite:wasm',
  vite:config     'vite:worker',
  vite:config     'vite:asset',
  vite:config     'vite:vue',
  vite:config     'vite:define',
  vite:config     'vite:css-post',
  vite:config     'vite:build-html',
  vite:config     'commonjs',
  vite:config     'vite:data-uri',
  vite:config     'rollup-plugin-dynamic-import-variables',
  vite:config     'vite:import-analysis',
  vite:config     'vite:esbuild-transpile',
  vite:config     'vite:terser',
  vite:config     'vite:manifest',
  vite:config     'vite:reporter'
  vite:config   ],
  vite:config   define: { __VUE_OPTIONS_API__: true, __VUE_PROD_DEVTOOLS__: false },
  vite:config   ssr: { external: [ 'vue', '@vue/server-renderer' ] },
  vite:config   configFile: '/Users/barryp/github/vite-css-code-split/vite.config.js',
  vite:config   configFileDependencies: [ 'vite.config.js' ],
  vite:config   inlineConfig: {
  vite:config     root: undefined,
  vite:config     base: undefined,
  vite:config     mode: undefined,
  vite:config     configFile: undefined,
  vite:config     logLevel: undefined,
  vite:config     clearScreen: undefined,
  vite:config     build: {}
  vite:config   },
  vite:config   root: '/Users/barryp/github/vite-css-code-split',
  vite:config   base: '/',
  vite:config   resolve: { dedupe: undefined, alias: [ [Object] ] },
  vite:config   publicDir: '/Users/barryp/github/vite-css-code-split/public',
  vite:config   cacheDir: '/Users/barryp/github/vite-css-code-split/node_modules/.vite',
  vite:config   command: 'build',
  vite:config   mode: 'production',
  vite:config   isProduction: true,
  vite:config   server: {
  vite:config     fsServe: { root: '/Users/barryp/github/vite-css-code-split', strict: false }
  vite:config   },
  vite:config   env: { BASE_URL: '/', MODE: 'production', DEV: false, PROD: true },
  vite:config   assetsInclude: [Function: assetsInclude],
  vite:config   logger: {
  vite:config     hasWarned: false,
  vite:config     info: [Function: info],
  vite:config     warn: [Function: warn],
  vite:config     warnOnce: [Function: warnOnce],
  vite:config     error: [Function: error],
  vite:config     clearScreen: [Function: clearScreen]
  vite:config   },
  vite:config   createResolver: [Function: createResolver],
  vite:config   optimizeDeps: { esbuildOptions: { keepNames: undefined } }
  vite:config } +5ms
vite v2.3.5 building for production...
✓ 12 modules transformed.
dist/assets/logo.03d6d6da.png    6.69kb
dist/index.html                  0.47kb
dist/manifest.json               0.28kb
dist/assets/style.2bb9ff70.css   0.20kb / brotli: 0.12kb
dist/assets/index.8d845e67.js    0.84kb / brotli: 0.45kb
dist/assets/vendor.642d02b4.js   42.44kb / brotli: 15.34kb

Before submitting the issue, please make sure you do the following

[x] Read the Contributing Guidelines.
[x] Read the docs.
[x] Check that there isn't already an issue that reports the same bug to avoid creating a duplicate.
[x] Provide a description in this issue that describes the bug.
[x] Make sure this is a Vite issue and not a framework-specific issue. For example, if it's a Vue SFC related bug, it should likely be reported to https://github.com/vuejs/vue-next instead.
[x] Check that this is a concrete bug. For Q&A open a GitHub Discussion or join our Discord Chat Server.

I should also mention, the reason I'm looking for the name of the CSS in the manifest.json is because I'm trying to use this with Laravel and Lumen apps using the Vite backend integration directions, so it's not using the generated dist/index.html, which does link to the generated CSS

https://github.com/vitejs/vite/issues/6477

Looks like #6477 duplicates this, but it has more discussion there. I'll close this in favour of that to keep the issues lean. Please subscribe to that issue to get notifications when that's addressed.


Rethink Variables/Tensor relationship

Currently variables are ints and tensors are pointers.
Is there any reasonable downside to everything being tensors? This way we can support multidimensional variables.

You might want to consider the tensor meta data overhead. For variables that information would be widely useless and bog down device transfers but you could store the metadata in a separate class and only point to that meta data structure when need. You could optionally transfer the meta data if it is None. Same ideas as the actual data, would have to free them together as well.

Besides the potential performance overhead and memory usage, I think it's a good idea to unify the API so that everything is a Tensor. Consistency is king IMO.


build: repair the build with VS2019 16.10.0

The newest VS2019 release updates CMake to 3.20, which picks up
llvm-mt as a preferred manifest tool.  However, we do not build
llvm-mt with libxml2 support currently, which prevents the use of the
just built manifest tool for building libdispatch.  Explicitly opt into
using the MSVC manifest-tool.
Cherry-pick 2990e8bc070aee69b3a1cdecfd526ebf0f7ec5fa

Replace this paragraph with a description of your changes and rationale. Provide links to external references/discussions if appropriate.

Resolves SR-NNNN.

@swift-ci please test

CC: @shahmishal


Red usernames on attackers

How to remove usernames written in game red text? If the number of attackers is high, the game is almost invisible.

the game becomes more interesting when there are many users with names, but the red color should be changed to white and having a black border of the names would be perfect

bagaimana menambah perintah untuk memunculkan pesawat dan perintah pemain terbang.  trimakasih

Color is now white


hotfix Docker release

Fixes a bug in releasing the docker images.

polkadot-companion-build is broken on master, this PR doesn't change any logic.


i3d_32frames_85.63.checkpoint

Hi. Thanks for sharing your work.
May I ask how you got the i3d_32frames_85.63.checkpoint?
When I downloaded i3d checkpoint pre-trained on imagenet/kinetics from https://github.com/rimchang/kinetics-i3d-Pytorch/tree/master/data/pytorch_checkpoints and fine-tuned the network on NTU DB, the accuracy was about  82.56.
And also when I downloaed  i3d_32frames_85.63.checkpoint from your repo and fine-tuned the I3D network on NTU, the accuracy wen up to 88.41.
Thank you.

Hi,
We also used the pre-trained model rgb_imagenet.pkl from the repository you linked. We fine-tuned the model on NTU RGB D with torch.optim.SGD(model.parameters(), lr=0.01, momentum=0.9) and the best model was at 18 epochs.
Out of curiosity, what configuration did you use to get 88.41?

Hi,
We also used the pre-trained model rgb_imagenet.pkl from the repository you linked. We fine-tuned the model on NTU RGB D with torch.optim.SGD(model.parameters(), lr=0.01, momentum=0.9) and the best model was at 18 epochs.
Out of curiosity, what configuration did you use to get 88.41?

Thank you for reply.
I load i3d_32frames_85.63.checkpoint and fine-tuned the model on NTU using following configurations.
optimizer = torch.optim.Adam(model.parameters(), lr=.0001, weight_decay=1e-4),
update_lr(optimizer, multiplier=.1), when training epoch is 10 and 20
The bactch size is set to 48 and I trained the network for 40 epochs.
Thank you.

Thank you for reply.
I load i3d_32frames_85.63.checkpoint and fine-tuned the model on NTU using following configurations.
optimizer = torch.optim.Adam(model.parameters(), lr=.0001, weight_decay=1e-4),
update_lr(optimizer, multiplier=.1), when training epoch is 10 and 20
The bactch size is set to 48 and I trained the network for 40 epochs.
Thank you.


[WIP] Installing example helm charts in CI/CD pipeline

it's WIP because I didn't have possibility to test it, and I'm currently in the process of doing so
Whole makefile changes related to helm is @ivanayov job, so kudos to her

Related to: https://github.com/networkservicemesh/networkservicemesh/issues/922
Signed-off-by: Ivana Yovcheva<EMAIL_ADDRESS>Signed-off-by: Bart Smykla<EMAIL_ADDRESS>
Adding installation of examples helm charts to CI/CD pipelines
Description
This Pull Request is adding installation of helm charts:

nsm
icmp-responder
vpp-icmp-responder
vpn
nsmd-monitoring
on azure, gke and packet clusters during the CI/CD process

Motivation and Context

How Has This Been Tested?

[ ] Covered by existing integration testing
[ ] Added integration testing to cover
[ ] Tested locally
[x] Have not tested

Types of changes

[ ] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)

Checklist:

[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.

Closing in favor of #1088