Split bench, test, lib, bin into different workspaces

What have you changed? (mandatory)
There is a pain point when writing benchmarks for TiKV now: we need to wait several minutes (usually around 10 minutes) even if we only added a blank line to the benchmark suite (benches directory).
This is caused by several issues:

Currently the content of tikv library src/lib.rs changes at every build, because we embed a UTC build time in it. This dynamic part is now removed from lib.rs, so that it won't trigger a rebuild if its content is not changed.
When changing files in the benchmark directory, cargo treats it as a change in the package and triggers a rebuild on lib.rs. By splitting benchmarks, tests, libs and binaries into different workspaces, they are isolated and won't affect each other.

In addition, this PR made some other improvements together:

Removed unused imported crates.
Make cargo clippy cover fuzz tests.

[ ] Check profiling feature under Linux.

[x] Check compile performance when only bench is changed.
$ make bench
cargo bench --package tikv-lib --features "default sse no-fail"  -- --nocapture && \
	cargo bench --package tikv-bench --benches --features "default sse no-fail"  -- --nocapture && \
	cargo run --package tikv-bench --bin tikv-bench --release --features "default sse no-fail"
    Finished release [optimized + debuginfo] target(s) in 1.34s
     Running target/release/deps/tikv-9d709eab255f1d37

running 499 tests
……

test result: ok. 0 passed; 0 failed; 496 ignored; 3 measured; 0 filtered out

   Compiling tikv-bench v0.0.1 (file:///Users/breezewish/Work/PingCAP/tikv/src/tikv_benches)
    Finished release [optimized + debuginfo] target(s) in 6.70s
     Running target/release/deps/tikv_bench-8523a19eecffd635

running 0 tests

test result: ok. 0 passed; 0 failed; 0 ignored; 0 measured; 0 filtered out

     Running target/release/deps/benches-e9c1f9ccbee2aec3

running 12 tests
……

As you can see, if we only change benchmarks in the benches directory (now it is tikv_benches), it only costs about 6 seconds to compile!

What are the type of the changes? (mandatory)

Improvement (non-breaking change which is an improvement to an existing feature)

How has this PR been tested? (mandatory)
CI
Does this PR affect documentation (docs/docs-cn) update? (mandatory)
No
Does this PR affect tidb-ansible update? (mandatory)
No
Refer to a related PR or issue link (optional)
Benchmark result if necessary (optional)
Add a few positive/negative examples (optional)

I feel like moving /benches (which is a standard convention in Rust) to a workspace project is coming the wrong way at the problem.
When I look at https://github.com/habitat-sh/habitat/ , which also uses workspaces, they do nothing like this. Instead they split components apart logically. What do you think?
On a different point:
Would you be willing to split this up and have the other PR resolve these points?

Currently the content of tikv library src/lib.rs changes at every build, because we embed a UTC build time in it. This dynamic part is now removed from lib.rs, so that it won't trigger a rebuild if its content is not changed.

(And maybe?...)

Removed unused imported crates.
Make cargo clippy cover fuzz tests.

Seems like this would be plenty of changes for one PR and it would make the impact of using workspaces in this way more clear.

@Hoverbear
Yes, this is my concern as well 🤣. However this PR is not splitting components (another PR should do this). Also I intentionally moved benches and tests into src/tikv_benches and src/tikv_tests to avoid someone mistakenly treat it as a Rust style. As you can see, they are managed by different workspaces so that it won't work as benches and tests of the root project (dependencies to the "tikv_test" and "tikv_bench" will not be recognized). In addition, for example, in servo, their tests are organized differently and I guess it should be Ok.

Currently the content of tikv library src/lib.rs changes at every build, because we embed a UTC build time in it. This dynamic part is now removed from lib.rs, so that it won't trigger a rebuild if its content is not changed.

This can't be extracted in a single build, because it won't work without a workspace separation 🤣In fact I tried this first, and found out that because the output of build.rs is changed, it will recompile the whole project again.

Removed unused imported crates.

This can be done in separate PRs. However each Cargo.toml will bloat to contain these unused dependencies.

Make cargo clippy cover fuzz tests.

This can be done in separate PRs. In fact this is a side effect of changing the clippy command to cover all workspaces 🤣Previously child workspaces are covered.

This one might be possible 🤣However it may be hard to do. I can have a try though. (and you will find that diffs may not be eliminated 🤣)
The changes above are not involved with logical changes. As long as the compiling pass, it is safe. What do you think?

This is a huge change, I suggest you make an agenda and tell others why and how do you want to do @breeswish

@siddontang Ok

@breeswish

This can be done in separate PRs. However each Cargo.toml will bloat to contain these unused dependencies.

Oh, I better understand why this happened now. I think it's ok to include that in this PR. Sorry. :)

Yes, this is my concern as well rofl. However this PR is not splitting components (another PR should do this). Also I intentionally moved benches and tests into src/tikv_benches and src/tikv_tests to avoid someone mistakenly treat it as a Rust style.

Hm, I'd like to have a strong motivation to so clearly and strongly break conventions. I worry this will impact our ability to receive new contributions. It is already quite some effort to understand the project, and this will only add to it.

When changing files in the benchmark directory, cargo treats it as a change in the package and triggers a rebuild on lib.rs. By splitting benchmarks, tests, libs and binaries into different workspaces, they are isolated and won't affect each other.

I think this might be a bug. Can we make a reproduction case (a very minimal crate) and open an issue about it? I can do this if you're busy. :)

There is a pain point when writing benchmarks for TiKV now: we need to wait several minutes (usually around 10 minutes)

:thinking: My machine spends much more time on kvproto and rocksdb then it does on TiKV. Rebuilding TiKV itself is reasonable. I think it can be faster though.
# build w/ Rocksdb + kvproto uncompiled
Finished dev [unoptimized + debuginfo] target(s) in 5m 59s
# build w/ rocksdb + kvproto compiled after touching `benches/benches.rs`
Finished dev [unoptimized + debuginfo] target(s) in 41.88s
# Check, same as before.
Finished dev [unoptimized + debuginfo] target(s) in 9.20s

One thing I did notice though was that tikv seems to build multiple times (Cargo actually lists tikv like 5 times in it's work queue). I wonder if this is related to the timestamp?
Building [====================================================>  ] 217/222: tikv, tikv, tikv, tikv, tikv

I very much support this effort and would love to see progress towards better compile times and organization, but we should be careful to do it in a way that does not pile us with any technical debt to clean later. This should be a way of relieving some of that debt.

@Hoverbear

I think this might be a bug. Can we make a reproduction case (a very minimal crate) and open an issue about it? I can do this if you're busy. :)

This might be a consequence of no incremental build in release mode.

One thing I did notice though was that tikv seems to build multiple times (Cargo actually lists tikv like 5 times in it's work queue). I wonder if this is related to the timestamp?

Actually it is Cargo building different parts (i.e. tests, libs, bins) of the tikv package. In this PR, since these parts are given different package names, you will see a better progress, like "........ tikv-bin, tikv-lib, tikv-tests".

I made some new progress about accelerating the bench compiling speed. A new PR which is much more simple than this one will be proposed soon.

ref https://github.com/tikv/tikv/issues/3352


Can't run 'php artisan migrate' from workspace

Description:

Can't run 'php artisan migrate' from workspace.
Expected Behavior:

I expect database migrations to happen.
Context information:
Basically I'll run the following to access the container:
docker-compose exec workspace bash

It seems to be able to do everything but it doesn't seem to be able to access the mysql container. If I login to the mysql container I can access mysql but no php access.
Output of git rev-parse HEAD
{paste here}

Output of docker version
<IP_ADDRESS>

Output of docker-compose version
1.25.5

System info: Mac, Windows or Linux. Include which disto/version
Windows 10 Version 1909

Steps to reproduce the issue:

docker-compose exec workspace bash
php artisan migrate
You'll get an error saying the connection is refused

Stacktrace & Additional info:
{paste here}

I believe this issue can be closed. In working with @rotaercz, we were able to determine that there were several factors to this problem.

Docker needed file sharing permissions for the project folder. Steps for this are:

Open Docker Desktop.
Press the gear icon on the toolbar to access settings.
Press the "Resources" button
Press "File Sharing"
Press the add icon and browse to your project folder and select "Add Folder"

Due to some configuration changes made throughout debugging the above issue, it ended up being easier to create a brand new project. Steps for this are below and assume the install of git bash.

Open git bash
Type cd Desktop

This simply changes the current directory to your desktop folder

Type mkdir testing

This creates a new directory called "testing" in the current folder (your desktop).

Type cd testing

This changes the current directory to the testing folder on your desktop.

Type git clone https://github.com/laradock/laradock.git

This downloads Laradock from GitHub and places it in a sub-folder in your project directory.

Type cd laradock

Once more we are changing the directory, this time to the Laradock folder in the project folder on your desktop.

Type copy env-example .env

This copies the example environment file for Laradock to the actual environment file that will be used to configure your Laradock instance.

Now open the .env file in Notepad and change the line that says WORKSPACE_VUE_CLI_SERVE_HOST_PORT=8080 to WORKSPACE_VUE_CLI_SERVE_HOST_PORT=8081

This changes the port that the Workspace container serves Vue CLI on avoiding a conflict with the phpMyAdmin container.

Add the testing folder to folder sharing in Docker desktop as described above.
Go back to your Git Bash window and type docker-compose up -d nginx mysql phpmyadmin redis

This will startup the Nginx, MySQL, phpMyAdmin, and Redis containers.

Your containers should be up and running. Note: we did not install a Laravel project, so nothing will be serving on http://localhost but you should be able to access phpMyAdmin at http://localhost:8080.

Default MySQL server is mysql
Default MySQL root username is root
Default MySQL root password is root

This seems to be the misconfiguration in the laravel .env file or sometime in my local machine case, the MySQL container is ready to be accessed in about 10-15 minutes after it's up and running.

try to connect to your phpmyadmin if it connected correctly then the problem is misconfiguration in your .env file for example check if
DATABASE_HOST=mysql


Probot isn't compatible with latest typescript

Bug Report
Current Behavior
Since I upgraded to the latest version of typescript I'm having an issue with probot.
Run these commands to reproduce:
mkdir probot-ts5
cd probot-ts5
npm init -y
npm i probot typescript
npx tsc --init
echo "declare const bot: import('probot').Probot; bot.on('pull_request.opened', (context) => { context; });" > main.ts
npx tsc --noEmit

will result with this error:
Expression produces a union type that is too complex to represent
Environment

Probot version(s): 12.3.0
Node/npm version: node 18.14.1
OS: OSX 13.2

Yup I'm getting this as well. It's very temperamental, so this library must be right on the internal union count limit in TS: https://github.com/microsoft/TypeScript/issues/53234
The odd complexity in the webhook handler types starts with this: https://github.com/probot/probot/blob/master/src/types.ts#L48. I'm not quite sure why we're running an Omit on all the webhook event names as a starting point.

Thank you @AaronDewes for finding the fix. Do you think #1821 could be merged anytime soon?  If not, I did a tiny one (#1833) to make probot compatible with Typescript +5.0.


[Bug]: Changes Lost and App Crashes when Switched between Dark-Light Modes in Edit Description and Captions

Summary
In Edit Description and Caption Page when you switch between Dark-Light Modes then the captions are lost and app eventually crashes.
Steps to reproduce

Open the App
Go to Any Contribution
Click on the Edit Icon For the Description
Switch Dark Mode/Light Mode
Press the Back Button or Submit Button

Expected behaviour
Edited captions and descriptions should be retained
Actual behaviour
Changes are Lost and the App Crashes
Device name
Xaomi 11 Lite NE
Android version
13
Commons app version
4.2.1-debug-main
Device logs
No response
Screen-shots
https://github.com/commons-app/apps-android-commons/assets/126143257/dd6c8d9c-b8ee-471c-b211-eba587598f4d
Would you like to work on the issue?
Yes

@nicolas-raoul This bug is also related to configuration changes on devices like Dark-Mode and Font Changes, and this seems to be a consistent issue in many activities in the app, I have opened two bugs that I have found because this (Including this one) but overall seems that we aren't handling config changes well in the app as also indicated by @psh , can you please try reproducing this one and assign to me so that I can make the necessary PR to fix this, thanks

Here's a couple of useful links to Google documentation that might help as you're thinking about the problem -

Handle configuration changes
Save UI states

The save UI state documentation includes a helpful table to guide thoughts about whether to use a ViewModel, saved instance state or other persistent storage.

Hi @nicolas-raoul @psh @RitikaPahwa4444  , Please Review the PR so that I can close this issue, thanks

@nicolas-raoul #5503 is solving this issue and is approved , can you please merge it and close this issue, thanks


Exception thrown when running server

I got an exception when running node start
$ node start
      _                     _                              _
   __| | ___  ___ _ __  ___| |_ _ __ ___  __ _ _ __ ___   (_) ___
  / _` |/ _ \/ _ \ '_ \/ __| __| '__/ _ \/ _` | '_ ` _ \  | |/ _ \
 | (_| |  __/  __/ |_) \__ \ |_| | |  __/ (_| | | | | | |_| | (_) |
  \__,_|\___|\___| .__/|___/\__|_|  \___|\__,_|_| |_| |_(_)_|\___/
                 |_|
 =========================   starting   ==========================
INFO | logger ready
INFO | deepstream version: 2.0.1
INFO | messageConnector ready
INFO | storage ready
INFO | cache ready
INFO | authenticationHandler ready: none
PLUGIN_INITIALIZATION_ERROR | permissionHandler needs to implement isReady or be an emitter
~/workspace/ds-realtime-retrospective-board/server/node_modules/deepstream.io/src/utils/dependency-initialiser.js:28
		throw error;
		^

Error: permissionHandler needs to implement isReady or be an emitter
    at new DependencyInitialiser (~/workspace/ds-realtime-retrospective-board/server/node_modules/deepstream.io/src/utils/dependency-initialiser.js:26:17)
    at Deepstream._start (~/workspace/ds-realtime-retrospective-board/server/node_modules/deepstream.io/src/deepstream.io.js:162:17)
    at g (events.js:286:16)
    at emitNone (events.js:86:13)
    at emit (events.js:185:7)
    at DependencyInitialiser._emitReady (~/workspace/ds-realtime-retrospective-board/server/node_modules/deepstream.io/src/utils/dependency-initialiser.js:113:7)
    at _combinedTickCallback (internal/process/next_tick.js:67:7)
    at process._tickCallback (internal/process/next_tick.js:98:9)
    at Function.Module.runMain (module.js:577:11)

My environments:

Mac: 10.11.16
Node: 6.2.0

Looks like the permission handler is custom and old school, should be valve!  @alex-harley
Will get fixed asap

Still not fixed, gets the same error.

Fixed now
worth mentioning that webRTC is only 1.0 compatible, so that part of the demo won't be working

how could it be fixed ...?

worth mentioning that webRTC is only 1.0 compatible, so that part of the demo won't be working

But you can start the server without it crashing

Exception thrown when running server


MAISTRA-2048: Fix annonbin failures

Follow up on MAISTRA-600: This moves the fix from a rpm patch to the
main repo.

Right. Then I think we can do this in 2.1 instead. We can live with it the way it is (patch in rpm) for now.

Right. Then I think we can do this in 2.1 instead. We can live with it the way it is (patch in rpm) for now.


Doc: Tetragon metrics

Closes #928
This PR:

Adds how to read tetragon metrics with different installation setup.
Explain Tetragon metrics

Thank you @prateek041 for this PR. Some high-level comments I have:

The intro is very friendly for people not proficient with Kubernetes, but I'm not sure if it belongs to the metrics page. I would rather add a "Validation" section to https://tetragon.cilium.io/docs/getting-started/deployment/kubernetes/, describing what to expect and how to check if Tetragon is running. Then the metrics page can just refer to the Kubernetes deployment page.
Can we use the tetragon service in this tutorial, instead of port-forwarding the pod directly? That is, kubectl -n kube-system port-forward service/tetragon 2112:2112. The port can also be checked by describing the service. In a tutorial for Kubernetes I think we should use the Kubernetes abstractions instead of interacting with containers directly.

Thank you @lambdanis for the review, making changes now.

Closes #928
This PR:
* Adds how to read tetragon metrics with different installation setup.

I guess this is done right?
Beside container stuff that should be contributed in another PR

* Explain Tetragon metrics

This is the part that is still missing right? can you do it in separate PR, please so we can merge this and have the metrics explained maybe in the reference?  @jrfastab @michi-covalent wdyt reference for metrics is better?

You have one CI link failure I guess at the end you do package.md  should be just package/   without .md

This is the part that is still missing right? can you do it in separate PR, please so we can merge this and have the metrics explained maybe in the reference?

yeah let's do a separate PR

Let me make the suggested changes and push.

by the way, we all reviewed while this was still a draft, please put it in ready for review when you think it's good.

Tetragon will have an autogenerated metrics reference soon™, so there is no need to explain metrics in this PR.

Oh that's great @lambdanis, please ask me to review the PR when you do this as we need to exclude some generated files from the ownership code review thingie https://github.com/cilium/tetragon/blob/22a9fe819db054eb2f478df58a9d5a447fb2070c/.github/CODEOWNERS#L16-L18
And also I'm curious about scripts, we have two at the moment for this purpose:

https://github.com/cilium/tetragon/blob/main/api/export-doc.sh
https://github.com/cilium/tetragon/blob/main/install/kubernetes/export-doc.sh


HW7 incorrect constraint

Hey man, When you look at the Y values you will find a sudden jump from your initial conditions.
When simulating the motion using:
y = q_RK4.';
g = figure(1);
filename = 'testAnimated.gif';
for n=1:length(y)-1
figure(1)
line([0, 0.2cos(y(1,n))],[0.3, 0.3+0.2sin(y(1,n))])
line([0, 0.7cos(y(2,n))],[0, 0.7sin(y(2,n))])
line([0.7cos(y(2,n)), 0.7cos(y(2,n))+ cos(y(3,n))0.6],[0.7sin(y(2,n)), 0.7*sin(y(2,n))+sin(y(3,n))*0.6])
xlim([-1.5 1])
ylim([0 1])
drawnow
frame = getframe(g);
im = frame2im(frame);
[imind,cm] = rgb2ind(im,256);
if n == 1
imwrite(imind,cm,filename,'gif', 'Loopcount',inf);
else
imwrite(imind,cm,filename,'gif','WriteMode','append');
end
clf
end

I found that you incorrectly formulated the constraint relation between phi4 and point A.
you are currently using:
-parms.O4Acos(phi4)+parms.O2Acos(phi2)
First off this does not amount to a total of 0. Secondly it fails to acknowledge the vertical shift of the slider and thus lenght variation of O4A.
I replaced it with:
-y3cos(phi4)+x3sin(phi4)
This correctly incorporates the relation both horizontal and vertical and will provide better results:

Thanks a lot for pointing that out. I was just looking for what I did wrong at this very moment!

Be aware that there might be another small error in the code since two of the plots are wrong. I will look at your comment and see if it also fixed the wrong plots tomorrow.

I think it will fix the problem I had thanks! I will push the new version tomorrow!

If you see another error let me know I checked every script up till 7 still need to check 8 9 and 10.

if you also remove the gravity the plots should also be good 👍

It is indeed working! I just pushed the new version including a modification of your animation script to the GitHub. Thanks for pointing out my error!


Be able to configure PodMonitor via prometheus trait

Currently the only thing is possible to configure at the prometheus trait is the labels, as we can see here: https://camel.apache.org/camel-k/1.7.x/traits/prometheus.html
Would be a nice feature , if we would be able to configure prometheus PodMonitor specs (https://github.com/prometheus-operator/prometheus-operator/blob/release-0.39/Documentation/api.md#podmetricsendpoint).
For example: set a specific interval for the metrics endpoint scrape, in my case the default scrape interval in prometheus is 60s, but I need that my camel-k integrations to be scrape each 15 seconds.
A suggestion for use in command line:
kamel run Integration.java --trait prometheus.pod-monitor.spec.pod-metrics-endpoints.interval=15s
So this would be a nice pattern to customize any parameter in the PodMonitor definition.

I wonder in that case whether creating a single bespoke PodMonitor matching all the integrations, would meet your need, e.g.:
apiVersion: monitoring.coreos.com/v1
kind: PodMonitor
metadata:
  name: camel-k-integrations
  labels:
    # To match the Prometheus PodMonitor selector
spec:
  podTargetLabels:
    - camel.apache.org/integration
  selector:
    matchExpressions:
      - key: camel.apache.org/integration
        operator: Exists
  podMetricsEndpoints:
    - port: metrics
      path: /q/metrics
      interval: 15s

And you could configuration the prometheus trait globally on the IntegrationPlatform, e.g.:
apiVersion: camel.apache.org/v1
kind: IntegrationPlatform
spec:
  traits:
    - prometheus:
      configuration:
        enabled: true
        podMonitor: false

Yes, for that scenario.
But in the case if I need different intervals, this will not fit, or just about the simple case that I just not want to manually manage the PodMonitors.


Fix nox session coverage-3.6 with Python <= 3.6.3

Not sure it's the best way to fix #314 but at least it avoids adding noise when testing locally :)

Thanks, that looks better, though after skipping these tests I ran into a coverage issue, so I had to add another "dummy" test for Python <= 3.6.3 to ensure proper coverage. Should be all good now.

Great, thanks.

Looking at the changes made by this diff out of context I had no idea why it was needed and when I commented out locally of course everything was fine.
I wonder if there is a better way to solve your local problem.
maybe install a supported version of Python 3.6 or remove it from your PATH somehow?


LoadError: cannot load such file -- bitfinex-api-rb

Hi,
I installed bitfinex-api-rb with bundle successfully, after adding it to Gemsfile. Now I'm trying to load the the bitfinex-api-rb into my application in an initializer but get error when trying to start application:

LoadError: cannot load such file -- bitfinex-api-rb

This is the initializer i'm using:
require 'bitfinex-api-rb'

Bitfinex::Client.configure do |config|
  config.key = GLOBAL[:bitfinex_key]
  config.secret = GLOBAL[:bitfinex_secret]
  config.websocket_api_endpoint = "wss://api.bitfinex.com/ws"
end

Any suggestion why?
Thanks.

You need to use
require 'bitfinex'
which is not clear.

Yes, I already figured it out. Thanks anyway.


Sorting of columns incorrect when using tmpl

I have a column which i am rendering through tmpl. However, when i display the table and click on the column title to sort, the sorting is incorrect (ie not alphanumeric).

May I ask to see your code?

Sorting will be correct as long as you also include the data property to sort on. If there is no corresponding data property, then sorting is impossible.


Added nucleo

This adds the memory layout for building applications for the Nucelo F429zi and F446re boards.

Hello @alexandruradovici, thank you for your contribution! Which version of the tock kernel have your changes been tested with? The current supported version (according to the readme) is 1.4.1. If your changes require a higher version we should block this PR until all board layouts have been ported to this version.

I think I tested it with the latest, let me check if it works with 1.4.1 and get back to you.

bors r+


getbootstrap.com won't load Bootstrap CSS in macOS Safari

It looks like Safari on macOS has a different "integrity" string when loading the site's Bootstrap CSS file.
macOS 10.13.6 (17G65)
Safari 11.1.2 (13605.3.8)
This is the error in the console:

Cannot load stylesheet https://getbootstrap.com/docs/4.1/dist/css/bootstrap.min.css. Failed integrity metadata check.

In Safari, this is the full tag for that file:
<link href="/docs/4.1/dist/css/bootstrap.min.css" rel="stylesheet" integrity="sha384-WskhaSGFgHYWDcbwN70/dfYBj47jz9qbsMId/iRN3ewGhXQFZCSftd1LZCfmhktB" crossorigin="anonymous">
Here is the same tag in Chrome, FireFox, and Opera:
<link href="/docs/4.1/dist/css/bootstrap.min.css" rel="stylesheet" integrity="sha384-Smlep5jCw/wG7hdkwQ/Z5nLIefveQRIY9nfy6xoR1uRYBtpZgI6339F5dgvm/e9B" crossorigin="anonymous">

Appears to be working fine for me and haven't seen any other complaints yet. Can you purge your browser cache to see if it resolves things? Safari doesn't include service workers, so I don't imagine it's that causing the issue @XhmikosR?

Safari 11.1 does support service workers, see https://caniuse.com/#feat=serviceworkers.
Anything else shown in the browser console?

Looks like I was doing it wrong. To clear the cache, it's Command+Option+E, not Command+Option+R. That fixed it, so this user error.
How did I end up with a reference to a different "integrity"?

Looks like I was doing it wrong. To clear the cache, it's Command+Option+E, not Command+Option+R. That fixed it, so this is user error.
How did I end up with a reference to a different "integrity"?

Not sure honestly. I didn't think service workers were available in Safari because I didn't see anything in the console for it like I do in Chrome. Perhaps I have something disabled? Haven't dug into this more yet.

Could this be a case where the visitor had visited before v4.1, thus was using the old 4.0 broken SW?

Could be, yeah

Rani into this today in Safari

Something isn't right @mdo in https://github.com/twbs/bootstrap/commit/6c8f6cebd46ed47db6ac31a63dacced34760ffc9
Why were the 4.1 dist files changed? Those should be reverted.


More than 4 events in day-view cause broken view

Hi there, i have a problem when try to represent multiples events in a same date and time as you can see in the next picture:

Note that this image have dummy info(not production database), but this occours too in the production site, all the envents begin at the same hour, suggestions?
Thanks =)

There is a blunt solution: change the max-width in #cal-day-box .day-event ( in calendar.css ). But I expect something more elegant(

Solution #1 make divs of every hour and radically change how the calendar works.
in my own version I have changed this radically, I didn't  want the 'overlay' onto next hours, so I put it on 1 div and made it when more events are there it just only appear on 1 hour.
Solution #2 change the max-width like romuals2014 says, which allows to put it on 1 row.
Solution #3 get creative with CSS change it so it has some partial overlay, when entering a second item, like have a offset on the newly created item, but you have to make sure a maxwidth is set, that is what the initial calendar didn't do, you get scrolling bars on the bottom of the calendar but at least it is on the hour you wanted it on.

Another solution would be if the number of events is more than 4 then display some kind of 'more' button or some kind of extending the data. much like how google does it. If it showed a card like layout to visually show there are many then when clicking on the card stack bring up a list of the events.


Create C4GT: Auto generate content in local language

Project
Auto generate content in local language
Changes
Week 1:

Analyzed the required capabilities of the plugin.
Documented the specific features, input methods, language selection, and integration points.
Started exploring the required Tech stack.

Week 2:

Studied Figma
Started working on the design for the UI of the plugin
Worked on the types of input required (i.e. typing, uploading, referring to existing text)
Created the wireframe for the plugin.
Worked on the design.

Week 3:

Worked on the prototyping and planned the workflow for the plugin's UI.
Explored the documentation of the Bhashini's API
Set the development environment and necessary tools/frameworks.
Gained basic understanding of the existing SunbirdEd Editors: Sunbird Collection Editor, Sunbird Content Editor, Sunbird Generic Editor.
Integrated Bhashini's API

Notes
Project repo: https://github.com/paradise110302/C4gtdiksha
Designs: https://www.figma.com/file/f43Silx7EFHUIR7hK20KFV/Untitled?type=design&node-id=0%3A1&mode=design&t=yyFkfU6cmKTbzV8r-1

Is this PR intended it just contains a single space...

@paradise110302 you can check yourself on the upper right tab file changed.

@paradise110302 I will be closing this PR, if you don't have changes to make...


Initial stab at making deployment plans cheaper. Back port of https://phabricator.mesosphere.com/D476

When marathon gets elected as leader and a lot of deployments were running before, marathon will need to load all of them and calculate all of the dependent applications. The current calculation is not really fast (all dependencies for all apps are calculated) and therefore a lot of memory is consumed and on_elected_prepare_timeout is exceeded.
These patches will increase performance and will allow marathon to load deployments faster.

DeploymentPlanTest fails - please have a look

retest this please


:bug: fix event duration not being affected by display toggle

fix events with durations not affected by display toggle
add test for toggling
add test for event duration

#9  👈
main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @szamanr and the rest of your teammates on  Graphite


Unity 2019.4.31f1 tons of U# compile errors

I'm assuming it's not compatible with the latest version?

Those compile errors are from a different script.


The labelled shortest path between two nodes contains an incorrect first label

Given that the path between nodes A and B currently contains A as well as B, when we label the transitions and simply produce a Vector[(N, B)] the first or the last transition label has to be wrong given that there must be one edge less than nodes in the path.
As I see it there are a few ways to fix this:

Create a custom data type for labelled paths
Remove the start node from the path to keep the incoming edge semantic of the pairs
Add big disclaimer comment that at least states that the first edge label is wrong

@mgttlinger so option 3 is out of the window for me, i'd rather we have a systemic fix. Option 2 seems most appealing but we'd have to audit if that would break any of our existing functionality. @kaiserpelagic can comment more on usage in Nelson.
@rossabaker your input here would also be valued :-)

Option 2 would still result in the odd situation where the unlabelled path contains the start node but the labelled one doesn't.

Option 4: Change the Vector[(N, B)] to (Vector[N], Vector[B]) then they can have different lengths.

I'd prefer option 4 personally. If that would work for your use cases I could implement that. I think that would also fix #22.

Any more comments? Should I implement this?

Hi, @mgttlinger.  Apologies for the slow reply.
What would you think of Option[(N, Vector[(N, B)])]?  That would only permit N=0,B=0 or N=B+1.  The constraints of that type make sense to me, but I'm not sure about the ergonomics.
I'm also planning to start work on #19.  I am unsure whether this makes Quiver more or less compelling for your use case, but wanted to throw that out there.

I'm fine with that type. I will fix this next week.
I don't really care about the cats port as I'm stuck with scalaz 7.1 at work anyway.

Would you prefer one last release built on scalaz with your enhancements, or can you live with cats on the classpath since it won't conflict with your scalaz version?  How important are the scalaz instances to you?

I don't care about the scalaz instances. I just use the graph for pathfinding. If it isn't too much effort a scalaz release wold be nice but not mandatory. Also I'm not sure when I will get around implementing this change. The type change is done, but the search no longer finds the shortest path when the shortest path would be to not make a step at all. I'm a bit held up at work this month...


Fix broken urls in KFServing docs

Easy navigation and easy reading for others

Thanks @terrytangyuan! Pinging @ryandawsonuk for approval.
/assign @ryandawsonuk

Thanks @terrytangyuan! Pinging @ryandawsonuk for approval.
/assign @ryandawsonuk


aws.sesv2.ConfigurationSet.supressionOptions incorrectly requires supressedReasons to have at least 1 item

What happened?
Attempting to create a SES v2 Configuration Set with empty supressed reasons(to override account-wide supression options) fails with this error:
 error: aws:sesv2/configurationSet:ConfigurationSet resource 'xxxxxxxx' has a problem: Not enough list items: Attribute requires 1 item minimum, but config has only 0 declared.. Examine values at 'ConfigurationSet.SuppressionOptions.SuppressedReasons'.
When creating a configuration set on the console, it is possible to override account-level supression options and disable them. I edited the configuration set created by pulumi through the console to override and disable supression options. Then, running pulumi import showed this result:
Do you want to perform this refresh?
No resources will be modified as part of this refresh; just your stack's state will be.
 details
  pulumi:pulumi:Stack: (same)
    [urn=urn:pulumi:sms-email-prod::sms-email::pulumi:pulumi:Stack::sms-email-sms-email-prod]
    ~ aws:sesv2/configurationSet:ConfigurationSet: (update)
        [id=default]
        [urn=urn:pulumi:sms-email-prod::sms-email::aws:sesv2/configurationSet:ConfigurationSet::xxxxxx]
        [provider=urn:pulumi:sms-email-prod::sms-email::pulumi:providers:aws::default_5_39_0::0233d0ba-3de7-4a05-99f4-cc50c04ee517]
        --outputs:--
      + suppressionOptions  : {
          + suppressedReasons: []
        }

This shows that it should be possible to set supressionReasons to an empty array, and that this is the only way to override account-level supression options and disable supression.
Expected Behavior
When defining a ConfigurationSet resource as so:
{
  configurationSetName: "example",
  suppressionOptions: {
    suppressedReasons: []
  },
}

I expect pulumi up to succeed and create a Configuration Set overriding the account-level supression options and disabling supression.
Steps to reproduce

Create a sesv2.ConfigurationSet resource with empty supressedReasons like so:

{
  configurationSetName: "example",
  suppressionOptions: {
    suppressedReasons: []
  },
}

Run pulumi up

Output of pulumi about
CLI
Version      3.66.0
Go Version   go1.20.3
Go Compiler  gc

Plugins
NAME        VERSION
aws         5.39.0
awsx        1.0.2
docker      3.6.1
nodejs      unknown
postgresql  3.8.0
random      4.13.0

Host
OS       arch
Version  22.1.0
Arch     x86_64

This project is written in nodejs: executable='/home/lilyes/.local/share/nvm/v16.13.2/bin/node' version='v16.13.2'

Current Stack: sms-email-prod

TYPE                                         URN
pulumi:pulumi:Stack                          urn:pulumi:sms-email-prod::sms-email::pulumi:pulumi:Stack::sms-email-sms-email-prod
pulumi:providers:pulumi                      urn:pulumi:sms-email-prod::sms-email::pulumi:providers:pulumi::default
pulumi:pulumi:StackReference                 urn:pulumi:sms-email-prod::sms-email::pulumi:pulumi:StackReference::networking-prod
pulumi:pulumi:StackReference                 urn:pulumi:sms-email-prod::sms-email::pulumi:pulumi:StackReference::logging-prod
pulumi:providers:aws                         urn:pulumi:sms-email-prod::sms-email::pulumi:providers:aws::default_5_39_0
aws:ses/domainIdentity:DomainIdentity        urn:pulumi:sms-email-prod::sms-email::aws:ses/domainIdentity:DomainIdentity::xxxxx-ses-domain-identity-sms-email-prod
aws:sns/smsPreferences:SmsPreferences        urn:pulumi:sms-email-prod::sms-email::aws:sns/smsPreferences:SmsPreferences::sms-email-prod-sns-sms-preferences
aws:ses/domainDkim:DomainDkim                urn:pulumi:sms-email-prod::sms-email::aws:ses/domainDkim:DomainDkim::xxxxxx-ses-domain-dkim-sms-email-prod
aws:sesv2/configurationSet:ConfigurationSet  urn:pulumi:sms-email-prod::sms-email::aws:sesv2/configurationSet:ConfigurationSet::xxxxxx-ses-configuration-set-sms-email-prod
aws:route53/record:Record                    urn:pulumi:sms-email-prod::sms-email::aws:route53/record:Record::xxxxxx-domain-dkim-record-1-sms-email-prod
aws:route53/record:Record                    urn:pulumi:sms-email-prod::sms-email::aws:route53/record:Record::xxxxxx-domain-dkim-record-2-sms-email-prod
aws:route53/record:Record                    urn:pulumi:sms-email-prod::sms-email::aws:route53/record:Record::xxxxxx-domain-dkim-record-0-sms-email-prod

Found no pending operations associated with sms-email-prod

Backend
Name           lily-20n3s6gk00
URL            s3://xxxxxxx-pulumi
User           lilyes
Organizations

Dependencies:
NAME                VERSION
@pulumi/aws         5.39.0
@pulumi/awsx        1.0.2
@pulumi/postgresql  3.8.0
@pulumi/pulumi      3.65.1
@pulumi/random      4.13.0
@types/node         14.18.20
mime-types          2.1.35

Pulumi locates its logs in /tmp by default

Additional context
This might be a problem with the upstream terraform provider, but since I have never used terraform and I am experiencing this issue with pulumi I decided to create the issue here.
Contributing
Vote on this issue by adding a 👍 reaction.
To contribute a fix for this issue, leave a comment (and link to your pull request, if you've opened one already).

Hi @lily-es, sorry you're running into this issue. As a workaround while we look into it, does it work if you leave out suppressedReasons: [] altogether?

This is actually a bug in the upstream provider. We'll look into a way to work around this on our side.

The upstream issue https://github.com/hashicorp/terraform-provider-aws/issues/28669 has been resolved as fix, so I'll close this issue too. Please report a new issue if the problem occurs again.


feat: package rename in npm

Addresses #364

@bcoe, would you mind confirming these changes one more time? I had to make all the changes to the repo's names at once, since they changed in google3.


Coyenn/card refactorings

✅ Checklist

[x] I have followed every step in the contributing guide (updated 2022-08-15).
[x] The PR title follows the convention we established conventional-commit
[x] I performed a functional test on my final commit

Changelog

Cards with links will be clickable as a whole rather than just the header

Love it


SHELL-1082 #done Fix exit code after exceptions are thrown by CLI commands

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Investigate Full Story support for iOS 14

Problem
We were forced to support iOS 14 because the Full Story library crashes upon launch. This prevents us from reaching 1.5% of all iOS users.
Solution
Discover whether iOS 14 support is possible and revert the iOS 14 > 15 bump.

Here's the library.
Here's the RN documentation
Slack Channel
They don't let people open issues on their github repo

Hey @danieldoglas, do we have a contact at Full Story? Or a shared slack channel?

I'm asking for an update here

There has been a little bit of discussion and I have bumped the issue again.

Not prioritising yet


fix(planning_launch): fix default parameters for lane change decision making

Signed-off-by: Fumiya Watanabe<EMAIL_ADDRESS>PR Type

Bug Fix

Related Links

Description
To pass the scenario tests, I fix the parameters for lane change decision making.
If we use tier4/autoware_launch, it should abort lane change automatically because we do not use external UI.

Review Procedure
Tested in planning simulator
https://user-images.githubusercontent.com/10190493/184832557-ae772f1f-c24f-45a4-b16b-98793446d91a.mp4

Remarks

Pre-Review Checklist for the PR Author
PR Author should check the checkboxes below when creating the PR.

[x] Code follows coding guidelines
[x] Assign PR to reviewer

Checklist for the PR Reviewer
Reviewers should check the checkboxes below before approval.

[ ] Commits are properly organized and messages are according to the guideline
[ ] Code follows coding guidelines
[ ] (Optional) Unit tests have been written for new behavior
[ ] PR title describes the changes

Post-Review Checklist for the PR Author
PR Author should check the checkboxes below before merging.

[ ] All open points are addressed and tracked via issues or tickets
[ ] Write release notes

CI Checks

Build and test for PR: Required to pass before the merge.
Check spelling: NOT required to pass before the merge. It is up to the reviewer(s). See here if you want to add some words to the spell check dictionary.

@rej55 pls make the same PR for tier4_autoware_launch in universe.

@TakaHoribe I made the PR
https://github.com/autowarefoundation/autoware.universe/pull/1602


Fixed width for the calculator

The calculator needs a fixed width in order to prevent it growing and shrinking along with changes in the size of the browser window. This would give all users the same experience while using it.

Had the exact same thought. Thank you for the pointers. Really appreciate the help

Explicitly setting the width to 500px both restricts the width and prevents the buttons from growing or shrinking depending on the size of their container.


Wrong operation in FT minting

As I see this line about FT minting, the operation should be Add instead of Replace?

Good catch, thanks:)


Background and icon colors fliped

Hi, I'm trying to export the following icons:

And some icons are white with black background:

Am I doing something wrong?
Also, when I try to export dark icons one by one they look good:

Issue:
Icons created with strokes cannot be converted to font icons.
Solution:
Convert icons to use fill instead of stroke.
Steps to Resolve:
Select the stroke elements of the icons and use the key combination (Ctrl+Shift+O) or (Cmd+Shift+O) for Outline Stroke.


Strongswan

Motivation for this change
A few fixes for strongswan.  In order for it to set a DNS server properly with systemd-resolved, it needs to use the system resolvconf.
I also needed to be able to set extra config in strongswan.conf:
    extraConfig = ''
      charon {
        plugins {
          resolve {
            resolvconf {
              iface_prefix = enp4s0.inet.ipsec.
            }
          }
        }
      }
    '';

to deal with https://unix.stackexchange.com/questions/511507/fedora-strongswan-resolvconf-interface-cant-be-the-loopback-interface
Things done

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[ ] Fits CONTRIBUTING.md.

Sorry, I forgot I had an open PR on this branch.  I'm going to close this and try to get something in upstream which will allow the resolvconf path to be specified at runtime.


doc: Add achievements.md

#255
Adds the long-awaited ACHIEVEMENTS.md file to the docs directory. This is generated from the prs.json file in #349, so we should merge that first.

Actually, hold off on merging this one, it might be more fun to do the automatic generation immediately!


fix rest thread memory leak

What is the purpose of the change
dubbo3.2 fix rest thread memory leak
Brief changelog
Verifying this change
@AlbumenJ

Checklist

[x] Make sure there is a GitHub_issue field for the change (usually before you start working on it). Trivial changes like typos do not require a GitHub issue. Your pull request should address just this issue, without pulling in other changes - one PR resolves one issue.
[ ] Each commit in the pull request should have a meaningful subject line and body.
[ ] Write a pull request description that is detailed enough to understand what the pull request does, how, and why.
[ ] Check if is necessary to patch to Dubbo 3 if you are work on Dubbo 2.7
[ ] Write necessary unit-test to verify your logic correction, more mock a little better when cross module dependency exist. If the new feature or significant change is committed, please remember to add sample in dubbo samples project.
[ ] Add some description to dubbo-website project if you are requesting to add a feature.
[ ] GitHub Actions works fine on your own branch.
[ ] If this contribution is large, please follow the Software Donation Guide.

This patch should also be applied to 3.3


Add bookworm to Concourse image build

Adds debian-12 and debian-12-arm64 image build and test pipeline.

Oh, I think we need to add it here as well: https://github.com/GoogleCloudPlatform/guest-test-infra/blob/e3e31926e5f559e9a8ad60bd1d5251673c22b740/concourse/templates/common.libsonnet#L7

/re-test

/lgtm


bug located in modeler.coordinate_systems

Before submitting the issue

[X] I have searched among the existing issues
[X] I am using a Python virtual environment

Description of the bug
I need to get list of coordinate systems in the Icepak model. I am using this following code to get the list.
Manual steps:

Create the Icepak design and
Create few coordinate systems manually
Now open the command window and run the following code, which writes the empty list

import pyaedt
from pyaedt import Icepak, modeler, Desktop
ipk= Icepak()
cs_objects = ipk.modeler.coordinate_systems
cs_names_list = [cs.name for cs in cs_objects]
cs_names_list
Output: Empyt list
Expected output: list with names of CS
In order to get the list we need to save the project before starting the Icepak pyaedt object, which should not be required.
Thanks!
Steps To Reproduce
as described above
Which Operating System are you using?
Windows
Which Python version are you using?
3.8
Installed packages
-e git+https://github.com/pyansys/pyaedt-icepak-rest-api@38a42610ad35d276c9ba310a58481075c3059382#egg=ansys_aedt_wrappers_restapi
attrs==23.1.0
bottle==0.12.25
build==0.10.0
certifi==2022.12.7
cffi==1.15.1
cfgv==3.3.1
charset-normalizer==3.1.0
clr-loader==0.2.5
colorama==0.4.6
contourpy==1.0.7
coverage==7.2.3
cycler==0.11.0
distlib==0.3.6
exceptiongroup==1.1.1
filelock==3.12.0
flake8==6.0.0
fonttools==4.39.3
identify==2.5.24
idna==3.4
imageio==2.28.0
importlib-metadata==6.6.0
importlib-resources==5.12.0
iniconfig==2.0.0
kiwisolver==1.4.4
matplotlib==3.7.1
mccabe==0.7.0
nodeenv==1.7.0
numpy==1.24.3
packaging==23.1
Pillow==9.5.0
platformdirs==3.4.0
pluggy==1.0.0
plumbum==1.8.1
pooch==1.7.0
pre-commit==3.3.1
psutil==5.9.5
py==1.11.0
pyaedt==0.6.61
pycodestyle==2.10.0
pycparser==2.21
pyflakes==3.0.1
pyparsing==3.0.9
pyproject_hooks==1.0.0
pytest==7.2.2
pytest-cov==4.0.0
pytest-html==3.2.0
pytest-metadata==2.0.4
pytest-timeout==2.1.0
pytest-xprocess==0.22.2
python-dateutil==2.8.2
pythonnet==3.0.1
pyvista==0.38.5
pywin32==306
PyYAML==6.0
requests==2.29.0
rpyc==5.3.0
scooby==0.7.1
six==1.16.0
tomli==2.0.1
urllib3==1.26.15
virtualenv==20.23.0
vtk==9.2.6
waitress==2.1.2
zipp==3.15.0

Hi @nnetake!
As we discussed I've tested this issue in newer version 2023.1 and it has been fixed.
The thing to do to overcome this issue in version 2022.2 is, as you are already doing, save the project first and then get the coordinate system.
Thanks!


scripts: support compile_commands.json in gen_app_partitions.py

Fixes: #40590
This commit updates gen_app_partitions.py to include only files present
in the current build by extracting the information from the CMake
generated compile_commands.json file.
This ensures that object files in sub-projects, such as empty_cpu0,
will not be considered by the script.
Using the compile_commands.json instead of walking the whole build tree
for finding object files also improves performance:
Time of executing gen_app_partitions.py (Old):

Executed in  480.06 millis
usr time  425.83 millis
sys time   49.55 millis
Time of executing gen_app_partitions.py (New):

Executed in   76.22 millis
usr time   49.00 millis
sys time   24.59 millis
Signed-off-by: Torsten Rasmussen<EMAIL_ADDRESS>

Some concerns:

This requires all supported toolchains to use -o to indicate output file. Visual Studio uses / instead of - for command line arguments (though we don't support it currently). I think all of our currently supported toolchains uses -o, so this may not be an issue.
This only works for compiled output. If someone decides to include a prebuilt library, this will skip the partitions defined in that library.

This requires all supported toolchains to use -o to indicate output file. Visual Studio uses / instead of - for command line arguments (though we don't support it currently). I think all of our currently supported toolchains uses -o, so this may not be an issue.

Not to mention, that the snippet generated is ld based, meaning it's supported by GNU ld, and thus only toolchains that understand that format. afaik Visual Code built-in linker doesn't support ld files, so really doesn't matter that Visual Studio uses / instead of -.

This only works for compiled output. If someone decides to include a prebuilt library, this will skip the partitions defined in that library.

Exactly as current solution. Only object files are considered, not prebuilt libraries.
Notice this in existing solution:
https://github.com/zephyrproject-rtos/zephyr/blob/f35cdfc1e4d3a59541bffcaefea739def7cd5468/scripts/gen_app_partitions.py#L143-L145
Not to mention that ELFFile() only supports elf files, that is either final elf or object files, not library archives.


Trigger Word Detector in Python

Script Title - Trigger word detection in Python
what will change - A new folder containing source code for trigger word detection will be added.
Instructions

[x] Create a new folder for your script and file/folder name should be appropriate.
[x] Create a README.md in your folder for program Instructions
[x] add requirements.txt if needed

Please add/delete options that are not relevant.

[x] Adding New Code
[x] Improving Code
[x] Improving Documentation
[x] Bug Fix

Programming Language

[x] Python

:star2: Star it :fork_and_knife:Fork it :handshake: Contribute to it!
Discord server  - https://discord.gg/FXyh2S3
Happy Coding,

Can you assign this to me? I have a solution.


x/tools/cmd/gotype: can't find import

$ cat $GOPATH/src/x/y/y.go
package y

var Y bool = true

$ cat $GOPATH/src/x/x.go
package main

import (
	"x/y"
)

func main() {
	_ = y.Y
}

Executing gotype $GOPATH/src/x/x.go results in:
/home/george/.go/src/x/x.go:4:2: could not import x/y (can't find import: x/y)
/home/george/.go/src/x/x.go:8:6: undeclared name: y

go run $GOPATH/src/x/x.go produces no error.

You need to go install x/y first.

I see. I figured as much from the code, however I considered it might be a bug.
Is there any particular reason for why the package has to be compiled in order to check the source code?

please see https://github.com/golang/go/issues/10249#issuecomment-86671707
for a reason.

I see, that makes sense. Thanks!


fix: When typing password, view password is disabled

Details

Fixed Issues

$ https://github.com/Expensify/App/issues/7597
Tests

Open app
Navigate to Settings>Security>Change password
Start typing password and click on Show password

[x] Verify that no errors appear in the JS console

QA Steps

[x] Verify that no errors appear in the JS console

Tested On

[x] Web
[x] Mobile Web
[x] Desktop
[x] iOS
[x] Android

Screenshots

Web

https://user-images.githubusercontent.com/21364901/152928962-b779d427-0ef9-47db-91d9-0febd4b5c0cc.mov
Mobile Web

https://user-images.githubusercontent.com/21364901/152928951-701782e5-d56e-4a31-b380-7a276030e895.mov
Desktop

https://user-images.githubusercontent.com/21364901/152928086-109ae8fa-b750-4a28-97b9-f883a7f72f14.mov
iOS

https://user-images.githubusercontent.com/21364901/152927614-99782521-6f73-4ea0-90c2-b321af09d182.mov
Android

https://user-images.githubusercontent.com/21364901/152747755-a79c62b9-dca0-4f7e-9b3a-45c88c9b3893.mov

:hand: This PR was not deployed to staging yet because QA is ongoing. It will be automatically deployed to staging after the next production release.

🚀 Deployed to production by @Julesssss in version: 1.1.38-3 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅


Can it support .net framework 4.0 to .net framework 4.6?

Can it support .net framework 4.0 to .net framework 4.6 ？
@glennawatson

No. Microsoft has stopped support for net framework 4.6

See https://docs.microsoft.com/en-us/lifecycle/faq/dotnet-framework#what-is-the-microsoft-lifecycle-policy-for--net-framework-


Dapr initialized but Tye can't start on M1 with Colima and Minikube

Describe the bug
Hi, I'm new to Dapr and Tye and I would like to get a simple setup running locally with two .Net applications fontend/backend. I'm using a M1 Mac with Colima and minikube installed.
To Reproduce
colima start
minikube start
docker --version
Docker version 20.10.21, build baeda1f82a
dapr init -k
dapr status -k
NAME                   NAMESPACE    HEALTHY  STATUS   REPLICAS  VERSION  AGE  CREATED
dapr-operator          dapr-system  True     Running  1         1.9.4    12m  2022-11-30 17:05.40
dapr-placement-server  dapr-system  True     Running  1         1.9.4    12m  2022-11-30 17:05.40
dapr-sentry            dapr-system  True     Running  1         1.9.4    12m  2022-11-30 17:05.40
dapr-sidecar-injector  dapr-system  True     Running  1         1.9.4    12m  2022-11-30 17:05.40
dapr-dashboard         dapr-system  True     Running  1         0.11.0   12m  2022-11-30 17:05.40
      I'm starting dapper with -k because just
      dapr init

      gives me:
      ⌛  Making the jump to hyperspace...
     ❌  could not connect to Docker. Docker may not be installed or running

tye run
Loading Application Details...
Drats! 'run' failed:
Dapr has not been initialized (e.g. via dapr init).
Dapr seems to be initialized correctly. Why can't Tye start?
Further technical details

Include the output of tye --version

tye --version
0.11.0-alpha.22111.1+3edef5428949c518c078844d6438e5ba86fce600

If possible rerun the command with -v debug and include the output

tye run -v debug
Loading Application Details...
Restoring and evaluating projects
Resolved metadata for service dapr-api at .../Code/Dapr_Microservices/DAPR/API/DAPR.API/obj/Debug/net6.0/MicrosoftTye.ProjectMetadata.txt
Resolved metadata for service dapr-blazor at .../Code/Dapr_Microservices/DAPR/Blazor/DAPR.Blazor/obj/Debug/net6.0/MicrosoftTye.ProjectMetadata.txt
Restore and project evaluation took: 1873,4099ms
Found application version: 1.0.0
RunCommand=.../Code/Dapr_Microservices/DAPR/API/DAPR.API/bin/Debug/net6.0/DAPR.API
RunArguments=
TargetPath=.../Code/Dapr_Microservices/DAPR/API/DAPR.API/bin/Debug/net6.0/DAPR.API.dll
PublishDir=bin/Debug/net6.0/publish/
AssemblyName=DAPR.API
IntermediateOutputPath=obj/Debug/net6.0/
Found target framework: net6.0
Parsed target framework name: net
Parsed target framework version: 6.0
Found shared frameworks: Microsoft.NETCore.App, Microsoft.AspNetCore.App
IsAspNet=True
Evaluation Took: 1,0104ms
Found application version: 1.0.0
RunCommand=.../Code/Dapr_Microservices/DAPR/Blazor/DAPR.Blazor/bin/Debug/net6.0/DAPR.Blazor
RunArguments=
TargetPath=.../Code/Dapr_Microservices/DAPR/Blazor/DAPR.Blazor/bin/Debug/net6.0/DAPR.Blazor.dll
PublishDir=bin/Debug/net6.0/publish/
AssemblyName=DAPR.Blazor
IntermediateOutputPath=obj/Debug/net6.0/
Found target framework: net6.0
Parsed target framework name: net
Parsed target framework version: 6.0
Found shared frameworks: Microsoft.NETCore.App, Microsoft.AspNetCore.App
IsAspNet=True
Evaluation Took: 0,3184ms
Drats! 'run' failed:
Dapr has not been initialized (e.g. via dapr init).

The platform (Linux/macOS/Windows)
macOS/arm64

Sry, my mistake. I understand now that tye is not meant to be run against kubernetes which is for deploy.
I found out that I was missing a permanent entry for DOCKER_HOME pointing to my Colima docker.sock.
After that I was able to init dapr with docker setup und run tye.
Please keep on with this amazing project


Buses experiment

I am opening this to not forget it.
Before merging it we need to figure out what to do with the experiment output files (.time).

@nvasilakis let's merge this as soon as we get a chance. I removed the results from it so that we can put them in a different location.


[Feature][JsonSplit] refactor process definition update

refer to #4417  refactor process definition update

+1


Will not accept custom package file

npm v3.6.0
node v5.6.0

Running git-labelmaker, enter my password, and choose Add Labels From Package.  I'm prompted for a path.  I have tried all of these (test file located at each)
packages/github_labels.json
github_labels.json
.github_labels.json

contents of file
[
  { "name": "ready", "color": "#159818" },
  { "name": "blocked", "color": "#e11d21" },
  { "name": "open comments", "color": "#eb6420" },
  { "name": "proposal-discussion", "color": "#84b6eb" },
  { "name": "question", "color": "#cc317c" },
  { "name": "On Hold", "color": "#666666" },
  { "name": "Revision Needed", "color": "#bfdadc" }
]

I hope this helps solve an issue - or improve the docs!   Thanks for your help.

Could you try: ./github_labels.json ?

I'm having the same issue - it keeps bouncing right back to the question for path & name of the package.

Thanks for raising this issue @NewAlexandria and @caseybecking.

@Jameskmonger I wonder if they are still using a < 0.7.2 version that doesn't include this change: https://github.com/himynameisdave/git-labelmaker/pull/37
Thanks for raising this issue guys, we're looking into it now!

Not a version issue, it is actually just not accepting anything as a JSON file


Make it easier to use entirely offline

Ideally this would be available offline. I don't know about iOS's ability to save a website for offline (I think it just saves it in Reading mode because Apple is awful), but even in Android there are resources that are loaded only after the page has fully loaded, and I think those don't end up cached.
I have spent only a little time on this, but ideally this shouldn't be something you need internet connection for. If it's easy enough to just make web pages into Android/iOS apps, that might be a solution as well.

Done.


[sival,alert_test] Enable alert_test for sival_rom_ext

Skip otp_ctrl for rom_ext since otp_ctrl CSR accesses are disabled.
This is because all otp accesses are blocked via ePMP. We expect a
more granular approach to locking OTP access, in issue https://github.com/lowRISC/opentitan/issues/20348.
The code has a TODO for changes once that issue is addressed.

The failure in example_sival_fpga_cw310_rom_with_fake_keys  is completely unrelated, so this is merged.


Add animated ghost behind text.

I've added a fun little ghost that nips around the screen, keeping behind the text.
I'm happy to make changes if you think it could work better another way. The container div is positioned using viewport units to make it more responsive.

@dominicduffin1 I love it :)

@scrabill Awesome :)


feat: add issue templates and PR template

Closes #69
Closes #70
This PR adds six new GitHub issue templates:

🐛 Bug
🧹 Cleanup
🚀 Feature
🔨  Major Update
🐾  Minor Update
👀 Tracking

In addition, this PR also adds a Pull Request Template.
These templates will help establish a standardized issue & pull request structure for Learning Lab repos, which will be especially helpful for developers less experienced with GitHub, as well as new PMs.

Did y'all see that there are YAML Issue forms now 👀
https://docs.github.com/en/communities/using-templates-to-encourage-useful-issues-and-pull-requests/syntax-for-issue-forms

oooooo no :0 thats cool willl use sometime!
In this case, not sure if we want to minimize the number of things for PMs to figure out + jiin already worked on this :))
But this is Jiin's pr so totally her choice!

Oh yeah, no pressure to use (esp since internally you're probably not fielding bug reports that frequently). Just seen some more OSS projects like Stylelint take it on, pretty cool feature!

let me try out the YAML template style, I think it's very cool!

Also, I still don't really see a big use case for minor since we can use the label S or XS to denote so on a feature, but it's also not a huge deal, so I don't mind leaving as is


iter is deprecated

LSP prints the warning.
I think it should be replaced.
import { iter } from "./deps/std/io.ts";

<EMAIL_ADDRESS>This is better?


Cache size and prog size

I apologize for possibly asking the question, which has been somewhere answered.
Is it necessary to make cache size and prog size values equal?
I tried to make the cache size == 4096 and prog size == 256 and while executing "lfs_file_close"  after "lfs_file_write" I've got some unrelated memory locations corrupted.
It did not happen while both had been equal to 4096.

Hello.
I set: cache size 4096;
prog_size 256;
read_size 256;
I enter lfs_mount(&lfs, &cfg) and step to:
// zero to avoid information leaks
lfs_cache_zero(lfs, &lfs->rcache);
lfs_cache_zero(lfs, &lfs->pcache);
While stepping I see in my IAR debug watch window valid values for lfs cfg (e.g. valid callback functions for eeprom operations).
As I step into "lfs_cache_zero" I see that all my variables are replaced by 0xff and after that in function "static int lfs_bd_read"
after call "int err = lfs->cfg->read(lfs->cfg, rcache->block,    rcache->off, rcache->buffer, rcache->size);"
I get to hard fault.

Oh interesting. That sounds like a buffer-overflow issue.
It's a bit confusing, but prog_buffer and read_buffer should both == cache_size (the buffer is used for the cache). prog_size and read_size are only used to inform littlefs what the disk geometry is.
Let us know if that works.

Well, if the sizes of cache, read and prog are equal everything is working fine

The size of prog_buffer, read_buffer and cache_size must all be equal. read_size and prog_size are unrelated. I realize this is not the most intuitive.
For example:
uint8_t read_buffer[256];
uint8_t prog_buffer[256];
struct lfs_config {
    .read_size = 1,
    .prog_size = 1,
    .cache_size = 256,
    .block_size = 4096,
    .read_buffer = read_buffer,
    .prog_buffer = prog_buffer,
}


Updated documentation for updated Datadog integration

Updates:

Removed Datadog site restriction (EU is now also supported)
Added information about the Agent environment variables

@markbastiaans
Am going to merge this into another branch as there are some things missing.
I need to put a new picture in, for example, and describe setting the DD_SITE variable. Unfortunately, I can't add a picture to your branch.
I'll ask you to review when I've made the changes.


Decheruk

Decheruk created by<EMAIL_ADDRESS>on 2017-01-07 13:07:45

<EMAIL_ADDRESS>replied, I've come across several examples of the word "decheruk" in song titles (and lyrics): Â Decheruk ra arrkemais, Decheruk ra Beriber, Â Decheruk ra Capitol Hill, Decheruk ra rriil, Decheruk ra Tmong. Â Is this a variation on/shortening of "delecheruk" (my position/standing) or it altogether a different word?

<EMAIL_ADDRESS>replied, it seems to be related with DECHOR. can somebody use DECHOR as noun? does the word DELECHOR exist?

<EMAIL_ADDRESS>replied, Ak dechor er a medal a tuangel.Aika el delechor a dirrek el meringel el tekoi.Delecheruk: while I was standing...Decheruk: as I stood by...

<EMAIL_ADDRESS>replied, Can you translate this one please?
Aika el delechor a dirrek el meringel el tekoi.

<EMAIL_ADDRESS>replied, Oh, man. Â I was going to say "those that stand are also bad news"
On Saturday, January 7, 2017 1:38 PM, Ulang Ngiruchelbad (Debugle<EMAIL_ADDRESS>wrote:

Emails/html
|
|  --- Write ABOVE THIS LINE to reply ---
Ulang Ngiruchelbad posted a new comment on Decheruk :
Continuous standing is a hard work. View this on DebugleStop receiving email notifications about this item.  |
|

<EMAIL_ADDRESS>replied, Those that stand would have a re- prefix, no?

<EMAIL_ADDRESS>replied, On Sat, Jan 7, 2017 at 5:19 PM John Bent (Debugle) <<EMAIL_ADDRESS>wrote:

Yes
Delecheruk
Kededechor
Dilechor
Mededechor
Kldedechor
--- Write ABOVE THIS LINE to reply ---
John Bent posted a new comment on Decheruk
http://debugle.com/projects/28361#issue=324489 :
Those that stand would have a re- prefix, no?
View this on Debugle http://debugle.com/projects/28361#issue=324489
Stop receiving email
http://debugle.com/issues/unsubscribe/17debdce76dfb2d5f007410304e63e3a3db700eb
notifications about this item.

<EMAIL_ADDRESS>replied, Ulang (or anyone), can you please use Decheruk and Delecheruk in sentences please?Also, is there a possessive form of delechor?  Like if you saw someone standing with a strange lean you might say something like, "Ng ko er a ngodech a delecherel"?

<EMAIL_ADDRESS>replied, Decheruk era tmong eng mosiu a keremerm.Delecheruk era bedul a KB bridge.

<EMAIL_ADDRESS>replied, not decherul: only delecherul- his/her standingdelecheruk- my standingdelecherelterir: them standingdecheruk is only reflexive in this word. only for self.

<EMAIL_ADDRESS>replied, > Decheruk era tmong eng mosiu a keremerm.here we have some unknown words... could anyone help to translate it?thank you

<EMAIL_ADDRESS>replied, Aleks, I believe Tmong is a name of a place just as well as a given name. As the darkness fell, I was standing at Tmong.

<EMAIL_ADDRESS>replied, We add places to dictionary. Where is the place?  Is it hamlet or rock island or what?

<EMAIL_ADDRESS>replied, Aleks, can you go through and turn this one into our standard format as well please?  I'll clean up English where necessary.

<EMAIL_ADDRESS>replied, w dechor v.i. xx- decheruk n.poss.1s. my standing.e Decheruk er a Tmong, e ng mosiu a keremerem. --I was standing at Tmong, as the darkness fell.This is what Aleks put in another thread. But Ulang says the word is actually DELECHERUK?

<EMAIL_ADDRESS>replied, And delecheruk is different meaning?

<EMAIL_ADDRESS>replied, Delecheruk is with emotions while decheruk is just the verb. Delecheruk era Uchul a Ngas e kora kmal mlo dobedebekau e mo mekikngit a renguk. While I was standing at Uchul a Ngas, i felt the loneliness as i thought of you. Decheruk era Uchul a Ngas ea bilas a engelakl. I was standing at Uchul a Ngas as the boat went by.


Cell line numbers not rendered when switching tabs

Applies To

[X] Notebooks (.ipynb files)
[X] Interactive Window and/or Cell Scripts (.py files with #%% markers)

What happened?
Type: Bug
On Jupyter notebook editor the line numbers stop being rendered when switching to another tab and then back to the jupyter editor.
Steps to reproduce:

Open any .ipynb file
Enable cell numbers (right click on the right margins -> show cell line numbers
Confirm that they are shown
Switch to another tab so that the notebook editor is not the visible one
Switch back to the notebook
Confirm that line numbers are no longer rendered
Right click on the margin and confirm that "Show cell line numbers" is checked, even though it doesn't render them

Extension version: 2022.8.1002431955
VS Code version: Code 1.71.0 (Universal) (784b0177c56c607789f9638da7b6bf3230d47a8c, 2022-09-01T07:25:38.437Z)
OS version: Darwin arm64 21.6.0
Modes:
Sandboxed: No
https://user-images.githubusercontent.com/6013/188276746-b4eb3398-b0d0-4585-9679-75ef35306572.mp4

System Info

Item
Value

CPUs
Apple M1 Pro (8 x 24)

GPU Status
2d_canvas: enabledcanvas_oop_rasterization: disabled_offdirect_rendering_display_compositor: disabled_off_okgpu_compositing: enabledmetal: disabled_offmultiple_raster_threads: enabled_onopengl: enabled_onrasterization: enabledraw_draw: disabled_off_okskia_renderer: enabled_onvideo_decode: enabledvideo_encode: enabledvulkan: disabled_offwebgl: enabledwebgl2: enabledwebgpu: disabled_off

Load (avg)
2, 1, 1

Memory (System)
16.00GB (1.60GB free)

Process Argv
/Users/rafaelsteil/Desktop/j --crash-reporter-id 2638254e-8f30-4899-98b8-ed2be42dcfb0

Screen Reader
no

VM
0%

A/B Experiments
vsliv368:30146709
vsreu685:30147344
python383:30185418
vspor879:30202332
vspor708:30202333
vspor363:30204092
vslsvsres303:30308271
pythonvspyl392:30443607
vserr242:30382549
pythontb:30283811
vsjup518:30340749
pythonptprofiler:30281270
vshan820:30294714
vstes263:30335439
vscoreces:30445986
pythondataviewer:30285071
vscod805cf:30301675
binariesv615:30325510
bridge0708:30335490
bridge0723:30353136
cmake_vspar411cf:30557515
vsaa593:30376534
pythonvs932:30410667
cppdebug:30492333
pylanb8912:30545647
vsclangdf:30486550
c4g48928:30535728
hb751961:30553087
dsvsc012:30540252
azure-dev_surveyone:30548225
2144e591:30553903

VS Code Version
Version: 1.71.0 (Universal) Commit: 784b0177c56c607789f9638da7b6bf3230d47a8c
Jupyter Extension Version
v2022.8.1002431955
Jupyter logs
No response
Coding Language and Runtime Version
No response
Language Extension Version (if applicable)
No response
Anaconda Version (if applicable)
No response
Running Jupyter locally or remotely?
Local

Thanks for creating this issue! It looks like you may be using an old version of VS Code, the latest stable release is 1.74.0. Please try upgrading to the latest version and checking whether this issue remains.
Happy Coding!

Fixed via https://github.com/microsoft/vscode/pull/169063


EmbedBuilder#setTitle does not allow a URL without a TLD (ex. localhost) if it has no file extension

General Troubleshooting
Hey there! Before you report a bug or suggest a new feature,
please make sure to follow these steps first!

[x] I have checked for similar issues.
[x] I have updated to the latest JDA version.
[x] I have checked the branches or the maintainers' PRs for upcoming features/bug fixes.

This is not the place to learn Java. Please refer to StackOverflow
for your general programming questions.

Issue
Issue Type

[x] Bug Report
[ ] Feature Request

For questions join the official JDA discord server: https://discord.gg/0hMr4ce0tIl3SLv5

Description
Like title says. Attempting to build an embed with EmbedBuilder and setting the embed title URL to something like http://localhost/whatever fails. This is because the URL_PATTERN used expects some sort of TLD or file extension on the end.
https://github.com/DV8FromTheWorld/JDA/blob/3092875bd5fd1eec10f4973a34a13ab6b4170c6e/src/main/java/net/dv8tion/jda/core/EmbedBuilder.java#L43
Using a URL like this would be useful in the case of testing locally. I do know that you can use a hosts file entry like <IP_ADDRESS> localhost.localdomain (and that's what I started doing), but it would be nice to not have to do this.
Regex101 example showing this: https://regex101.com/r/A567ut/1/

Not that i don't see your issue here, but i also don't see why localhost would make any sense here. Seeing as the only one that link would probably work on is you (the one sending the link)

Seeing as the only one that link would probably work on is you (the one sending the link)

Right, it's mainly just a convenience thing for doing local testing and not having to use some sort of alias like localhost.localdomain.

Additionally, im not even sure if discord would accept localhost urls (would be up to testing)

Discord does allow using localhost.localdomain as the URL at least when I tried it, so it seems logical that it'd also allow localhost.

I believe that this was tested when sending rich embeds became possible. However, I just tested this today and using a url such as https://localhost does indeed work now.


pre-release draft

API finalized
Pipelines built
Docstrings and first draft of docs in place
Currently failing a few unit tests on github workflow due to some conda path issue. (passing tests locally).

Merging even though unit tests failing. Problem with GH workflow config; need to bring docs into readthedocs.


Conservative remapping yields ESMC_FieldRegridStoreFile

I'm working on conservative remapping of air-sea CO2 fluxes on ocean grids. The goal is to remap from an arbitrary curvilinear grid to a 180x360 rectilinear grid. I've followed advice from https://github.com/JiaweiZhuang/xESMF/issues/14 and https://github.com/JiaweiZhuang/xESMF/issues/32, but am getting stuck.
As mentioned in those issues, typical ocean output has lat/lon bounds on (N, M, 4) dimensions, where 4 are the grid cell corners. I created a function following @JiaweiZhuang's advice (https://github.com/JiaweiZhuang/xESMF/issues/14#issuecomment-369686779) to convert from that convention to (N+1, M+1):
def _unravel(new_bounds, vertex_bounds, M, N):
    """Unravel vertices to (N+1, M+1) dimensions"""
    new_bounds[0:N, 0:M] = vertex_bounds[:, :, 0]

    # fill in missing row
    new_bounds[N, 0:M] = vertex_bounds[N-1, :, 1]
    # fill in missing column
    new_bounds[0:N, M] = vertex_bounds[:, M-1, 2]
    # fill in remaining element
    new_bounds[N, M] = vertex_bounds[N-1, M-1, 3]
    return new_bounds

def compress_vertices(ds, lat_bnds_name='lat_b', lon_bnds_name='lon_b'):
    """Convert from (N, M, nvertices) to (N+1, M+1) for grid cell corners

    Args:
        ds (xarray Dataset): Dataset with native ocean grid and (N, M, 4)
                                          lat/lon bounds.
        lat_bnds_name (optional str): Name of lat bounds variable.
        lon_bnds_name (optional str): Name of lon bounds variable
    """
    M = ds.x.size
    N = ds.y.size

    # create arrays for 2D bounds info
    lat_b = np.zeros((N+1, M+1))
    lon_b = np.zeros((N+1, M+1))

    # unravel nvertices to 2D style
    lat_b = _unravel(lat_b, ds[lat_bnds_name], M, N)
    lon_b = _unravel(lon_b, ds[lon_bnds_name], M, N)

    # get rid of old coordinates
    del ds[lat_bnds_name], ds[lon_bnds_name]
    ds = ds.squeeze()

    # assign new coordinates
    ds.coords['lat_b'] = (('y_b', 'x_b'), lat_b)
    ds.coords['lon_b'] = (('y_b', 'x_b'), lon_b)
    return ds

When I feed this into the xESMF regridder with 'conservative', I get the following error:
ValueError: ESMC_FieldRegridStoreFile() failed with rc = 506.

The log file isn't very enlightening here:
20190813 094016.108 ERROR            PET0 ESMF_Regrid.F90:360 ESMF_RegridStore Invalid argument  - Internal subroutine call returned Error
20190813 094016.109 ERROR            PET0 ESMF_FieldRegrid.F90:1292 ESMF_FieldRegridStoreNX Invalid argument  - Internal subroutine call returned Error
20190813 094016.109 ERROR            PET0 ESMF_Field_C.F90:1168 f_esmf_regridstorefile Invalid argument  - Internal subroutine call returned Error

However, my modified dataset with lat/lon bounds works with bilinear remapping. I've gotten conservative remapping to work with NCL with the native corners, but am looking to use xESMF/python exclusively.
Any help here is appreciated. Sorry for duplicate issues on conservative remapping. I'd be happy to help implement some utility functions to help out with this for future releases, since it seems to be a persistent issue.
Here is a notebook with my attempt: https://nbviewer.jupyter.org/gist/bradyrx/421627385666eefdb0a20567c2da9976
Here is a dropbox with the notebook and the native CESM2 output: https://www.dropbox.com/sh/lef7i2v88sfke3d/AACNPgjIrXBV35ejLYTRFvqxa?dl=0

Thanks for the complete code!

The log file isn't very enlightening here:

You can make the ESMF log file more informative by setting
import ESMF
ESMF.Manager(debug=True)

The log shows:
20190813 223533.907 INFO             PET0 Running with ESMF Version 7.1.0r
20190813 223544.261 ERROR            PET0 ~~~~~~~~~~~~~~~~~~~~ Degenerate Element Detected ~~~~~~~~~~~~~~~~~~~~
20190813 223544.261 ERROR            PET0   degenerate elem. id=122561
20190813 223544.261 ERROR            PET0
20190813 223544.261 ERROR            PET0   degenerate elem. coords (lon [-180 to 180], lat [-90 to 90]) (x,y,z)
20190813 223544.261 ERROR            PET0   -----------------------------------------------------------------
20190813 223544.261 ERROR            PET0     0  (-39.999603,  71.954628)  (0.237299, -0.199115, 0.950812)
20190813 223544.261 ERROR            PET0     1  (-39.548401,  71.959839)  (0.238793, -0.197185, 0.950840)
20190813 223544.261 ERROR            PET0     2  (-39.097107,  71.965042)  (0.240272, -0.195243, 0.950868)
20190813 223544.261 ERROR            PET0     3  (-39.548401,  71.959839)  (0.238793, -0.197185, 0.950840)
20190813 223544.261 ERROR            PET0 ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
20190813 223544.261 ERROR            PET0 ESMCI_Mesh_Regrid_Glue.C:161 c_esmc_regrid_create() Invalid argument  - - Src contains a cell that has corners close enough that the cell collapses to a line or point
20190813 223544.261 ERROR            PET0 ESMCI_Mesh_Regrid_Glue.C:512 c_esmc_regrid_create() Invalid argument  - Internal subroutine call returned Error
...

It says that the source grid contains a cell that has corners close enough that the cell collapses to a line or point. The location is around (lon=-40, lat=72). You can overlay this point with your input grid with:
plt.scatter(ds['lon_b'], ds['lat_b'], s=0.1, alpha=0.2)  # plot the input grid mesh
plt.plot(-40+360,  72, 'ro')  # plot the problematic point
plt.xlabel('lon')
plt.ylabel('lat')

which shows:

Because the grid cell near the red "pole" have very close corners (you can see two corners are exactly (-39.548401,  71.959839)), ESMF thinks that it is a triangle instead of a quadrilateral (i.e. "degenerated") and throws out this error.
I think this error can be skipped by setting ignore_degenerate=True when calling ESMF.Regrid, as suggested by uturuncoglu/RegESM#15 (not in xesmf yet, will test it). Alternatively, you can manually remove such degenerated cells.

See #61 for a quick fix.


Reinstate automated generation of threats by element

Describe what problem your feature request solves:
Version 1.x has a feature where the threats are suggested by element, for example if STRIDE:
STRIDE per element
          S | T | R | I | D | E
ACTOR   | X |   | X |   |   |
STORE   |   | X | X | X | X |
PROCESS | X | X | X | X | X | X
FLOW    |   | X |   | X | X |

Describe the solution you'd like:
Reinstate automated generation of threats by element
Additional context:

@jgadsden I'm curious, was this feature removed/deprecated in 2.x for any particular reason?

Well, only because we just did not have the engineering resource to implement it
Version 2.x is (almost) a complete rewrite of version 1.6.1, and some features we really wanted but could not get the time to do them - such as automated generation of threats by element
If you have some time to do this @ryeqb it would be very good to have

@jgadsden Im in the process of writing my bachelors thesis on templating threat models for IoT architectures and having had a quick look at Microsofts threat modeling tool, not having automated threat generation definitely stuck out. So did this the existing issue of non-existent templates given that's my thesis' goal https://github.com/OWASP/threat-dragon/issues/220
Hope I can help tacking both issues to some degree

Absolutely good to have any help you can on both these issues @ryeqb
Say if you want me to assign an issue to you
Awesome degree thesis by the way 👍🏾

@jgadsden I am surely work on this. Please assign me this.

@jgadsden , to automate the generation of threats by elements , we can go for below approach i think

We can define the elements of the system for which threat analysis is required. This could include actors, data stores, processes, data flows, etc.
Then we can Implement the STRIDE methodology within the tool or script and Create a mapping between each element/component and the corresponding STRIDE threat categories.
And then we can  Develop algorithms or rules within the tool/script to automatically generate threats for each element based on the mapped STRIDE categories

This is a Google Summer of Code project, and the application dates for GSoC contributors are between March 18th and April 2nd

Hi, @jgadsden! I am interested to contribute this issue under Gsoc. I am looking for some help from you to understand that issue deeply. So, that I can create an outline for the remedy. 🙂
As mentioned the threats by element functionality available in v1.x but not in 2.x!
I tried both the versions 2.2.0 and 1.6.1 -
The difference I found is that -
In version 2.2.0 -

In version 1.6.1 -

Can you help me by providing more exposure to it, please?


Add latitude and longitude to Building model

Added latitude and longitude fields to the Building model. This lays the groundwork for displaying classroom locations on a map (issue #11). A migration will be needed for this change.

Hey, thanks for the PR, will review this once I have some more time.

Hey supercoder-dev, thanks again for the PR.
After some thought, I think its best to avoid making unnecessary changes to the model, as we would also need to update the serializers and views.
If you'd like, I think blucardin's comment is a good step in the right direction, incorporating UBC's existing website. Since it looks like CORS requests aren't allowed, I'd suggest setting up a proxy using nginx, making requests for buildings/room pages through the proxy, and maybe embedding the response in an iframe. You can find the nginx config file in the nginx/conf folder.
If you have any questions, feel free to reply below.


LDAP over SSL using port 636 not working.

Versions -
Kong - 0.14.1-alpine
Kong Ingress Controller - 0.2.2
I am trying to connect to a LDAP server running on port 636 and using ldaps protocol.
I set the following in the LDAP plugin -
                "ldap_port": 636,
                "verify_ldap_host": false,
                "start_tls": true,

I get the following error when I connect with Kong -
2018/12/06 22:56:29 [error] 28#0: *17910 [lua] responses.lua:121: send(): failed to get from node cache: callback threw an error: /usr/local/share/lua/5.1/kong/plugins/ldap-auth/ldap.lua:34: bad argument #1 to 'bunpack' (string expected, got nil)
stack traceback:
	[C]: in function 'bunpack'
	/usr/local/share/lua/5.1/kong/plugins/ldap-auth/ldap.lua:34: in function 'claculate_payload_length'
	/usr/local/share/lua/5.1/kong/plugins/ldap-auth/ldap.lua:122: in function 'start_tls'
	/usr/local/share/lua/5.1/kong/plugins/ldap-auth/access.lua:58: in function 'ldap_authenticate'
	/usr/local/share/lua/5.1/kong/plugins/ldap-auth/access.lua:81: in function </usr/local/share/lua/5.1/kong/plugins/ldap-auth/access.lua:80>
....

I verified that the same server:port and protocol settings using the ldapsearch command, and it works -
ldapsearch -x -W -D "<ID>" -b "<base DN>"  -H ldaps://<ldap server>:636
We cannot change the port number on the LDAP server, it has to be 636.
Is there a way to configure Kong to use ldaps protocol?

This would be handled by https://github.com/Kong/kong/pull/2720 it seems

@p0pr0ck5 : I had checked that, but don't think that was committed. https://github.com/Kong/kong/pull/2720/commits
Is there any other setting or configuration to indicate ldaps to Kong? Thanks.

No there's no. Sorry I should have been more clear - follow that PR for updates. The functionality doesn't currently existing in the LDAP plugin bundled into Kong.


a way to disable the mouse style replacement

Which operating system does the issue occur on?
manjaro
If on linux, are you using X11 or Wayland?
X11
i am not sure if i overlooked this in the autogenerated config or if there is a issue for this already (i did not find one)
alacritty replaces the mouse mounter style  to
i hope this is not a technical limitation or i just overlooked the setting

Mouse cursors in X.Org work in mysterious ways, make sure you're not confusing your GTK with your X.Org mouse cursor. Also make sure that the cursors in use are all correct, by default alacritty should use the I-Beam cursor, not the arrow.

i checked and apparently my theme does not specifically defines one https://i.imgur.com/GYGJE7A.png
once i use a theme where it is define the I beam is displayed properly

This is not an alacritty issue but a problem with your setup.

a restart fixed it apparently


[master] Clean ups

@GrahamCampbell and @keradus sorry for loosing your comments : S
I think this is the result from rebasing and squashing (to remove the merge commits...)

I can see all my comments ;)

:+1:

:+1:

:+1:

Thank you @SpacePossum


unitaryfund/metriq-app#890: Data sets back end

This addresses unitaryfund/metriq-app#890, unitaryfund/metriq-app#891, unitaryfund/metriq-app#892, unitaryfund/metriq-app#893, and unitaryfund/metriq-app#894.
@vprusso This is mostly just a heads-up: we'll discuss at our meeting, later today.

Sorry for taking a while to get to this review, @WrathfulSpatula. As you know, have gotten a bit side-tracked with the local dev environment issues.


Create Sprite for Military Boots

Please see CONTRIBUTING.md first.
Happy Hacking! :smiley:

I'd like to try this.


Fix small typo in documentation

It looks like the original author meant to say "though" here but instead said "thought".

Learn Build status updates of commit a79a046:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

articles/aks/operator-best-practices-cluster-security.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the learn.microsoft.com contributor guidesPost your question in the Learn support channel

@kpalmberg - thank you for the catch!


Enemies not spawning when strings in global config file are left default or blank

It appears that the game is reading the "DEFAULT" and "[blank]" strings in the global config files as "0" instead of whatever the base game default is, preventing anything from spawning. Filling each of these strings out manually appears to solve the problem so the mod seems to be working at least, but the way the file generates currently causes nothing to spawn in the interior area.
Have not tested with the global scrap file or outside enemy files.

Which options are you trying to set? A blank value should always use the original game value, but I may have messed up an option somewhere.

Ive set some power level and spawn cap options for a few enemy types, and made it so some take '9999' seconds to open doors, but everything else is blank.
I tried a fresh mod profile with just LQ, using the default configs that it generates for itself, and everything seems to work fine. but as soon as i specify some of the values, everything stops spawning.
Here are the values i am altering for my SoloPlay rebalance modpack;
Blob.PowerLevel = 2
Centipede.DoorSpeedMultiplier = 9999
Crawler.MaxEnemyCount = 3
DressGirl.MaxEnemyCount = 0
DressGirl.PowerLevel = 4
DressGirl.SpawnChanceCurve = 0:1, 0:1
Masked.MaxEnemyCount = 5
Masked.PowerLevel = 2
Nutcracker.MaxEnemyCOunt = 6
Puffer.MaxEnemyCount = 3
SpringMan.MaxEnemyCount = 3
SpringMan.PowerLevel = 3
Rarity.DressGirl = 0
Everything else is blank.

Ok thanks I'll take a look. I'm assuming that these values are for a moon's Enemies.cfg, and the global Enemies.cfg is also enabled, but blank?

No, these values are for the global enemies config, which is the only one generated and enabled.

Hmm, ok. I'll do some testing my end and get back to you.

Im doing some more testing with various combinations of those values to see which, if any of them, could be the cause

Ok, it seems like I was not using the correct list(an empty one) when searching for the default rarity, which caused the "default" rarity of all enemies to be 0. I'll have this fixed soon, hopefully, but for now if you enable the moon config without modifications, it should spawn enemies like normal.

Pogging. Tanks for looking into it!


[Boosted vs Brand stencils] Toogle buttons with labels

Border color different, focus not visible, no inactive state in Boosted.
Stencil

Boosted

done

KO for me : https://boosted.netlify.com/docs/4.3/boostwatch/#3-way-toggle doesn't have the right look and feel
I think the component pointed by that issue is more btn-group https://boosted.netlify.com/docs/4.3/components/button-group/

http://boosted.orange.com/docs/4.2/components/buttons/#checkbox-and-radio-buttons see if it's possible to custom that component easily to fit the stencil

PR #136

PR #136


[AA-1128] - Wire up Hangfire and Signalr in Startup for .NET core project

Description

Added SignalR client library to the .net core project.
Added temporary files to create a SignalR hub and added SingalR client code to demo a proof-of-concept chathub and client for SignalR.
Added Hangfire.SqlServer, Hangfire.Core and Hangfire.AspNetCore packages to the .net core project.
Configured Hangfire in the .net core project Startup.
Added temporary logic to test that Hangfire works. Used Hangifre.SqlServer. This test needs an empty "HangfireTest" database and the connection string pointing to the server.

[ ] Resolve merge conflicts introduced by recent items merged to main. It is probably a trivial conflict in Startup.cs.
[ ] Study PR #51. The new JS files you're adding have a better home now under wwwroot/Scripts. However, you will find we have a coincidentally named file "signalr.js" there already. I say it's a coincidence because it is not the old signalr library, but instead our related custom progress bar code. I think if you rename our progress bar file appropriately (take care in the Layout and bundleconfig.json for references to it), commit, and then move this real library signalr file down into that folder, that combination should work. Be sure to include signalr.min.js in the Layout near the footer where we now include all js libraries.
[ ] I see that the 3 nuget packages you added in the csproj are consistent with the version we set up in a previous ticket. Note, though, that the already-referenced Hangfire 1.7.14 already includes Hangfire.AspNetCore, Hangfire.Core, and Hangfire.SqlServer of the same version. You can see this when you expand the Hangfire node inside Solution Explorer and in the "Dependencies" section here: https://www.nuget.org/packages/HangFire/1.7.14 so I think you could remove these 3 new PackageReference tags and you would find everything still works and that expanding Hangfire in Solution Explorer still shows the related packages do in fact arrive.

I see how your proof-of-concept commit/revert showed setting up the Dashboard and connecting to SQL Server. I agree with leaving that out for now. I've updated AA-1118 with guidance on introducing it with respect to our SQL Server / Postgres needs and with respect to the older .NET 4.8 code we might need to phase out at the same time. It makes sense to do that while working on the Learning Standards feature since we'll have real feedback at that time.


Get does not return ErrKeyNotFound when reading a key that is never added

I reran FizzBee generated tests. I noticed a regression.
For this sequence of operations
Put("k0", "v0")
FlushWAL()
FlushMemtableToL0()
Get("k1")

The Get should return an err == ErrKeyNotFound ("key not found")
But Get returns an empty error.
A simple reproduction steps.
db.Put([]byte("key1"), []byte("value1"))
db.FlushWAL()
db.FlushMemtableToL0()
data, err := db.Get([]byte("key2"))
if err == nil {
	panic("expected error")
}

Looking at the code, this seems to happen at line db.go:180. When !ok, it doesn't return an error but tries to reuse the old err variable which is obviously nil (because the steps before it ha
iter, err := sstable.NewIteratorAtKey(&sst, key, db.tableStore.Clone(), 1, 1)
if err != nil {
	return nil, err
}
kv, ok := iter.NextEntry()
if !ok {
	return nil, err       // err here is always nil
}

@jayaprabhakar this is fixed with https://github.com/slatedb/slatedb-go/pull/53

I reran the tests, and all passed. Thanks.


commit

i created a branch, created a file and made a commit, and opened a pull request

I created a branch, created a file and made a commit, and opened a pull request

i created a branch, created a file and made a commit, and opened a pull request

I created a branch, created a file and made a commit, and opened a pull request

i created a branch, created a file and made a commit, and opened a pull request. All pull request done successfully


nit: corrected portnum on OPC containers

caught this small mistake just now, needed to correct the portnum on the second OPC PLC container deployment for it to work properly.

I wonder why port 5000 isn't working. Are you running something else that takes that port?


Review skipped tests in Radius repo and try to get rid of them

Engineering Improvement
We have some tests that we skip with t.Skip() in the Radius repo. We should review these and see which ones we can re-enable. If the tests are being skipped because they are no longer needed, we should remove them. If we really really need to keep them skipped, then we should have a good reason to do so.
AB#10677

I actually did this a couple of weeks ago, and updated corresponding issues/logged new issues wherever missing. Had shared it with the triage team. Did this come out of some discussion @willdavsmith? Else I think it will be repetitive and not needed. cc @nicolejms @shalabhms @sylvainsf

Closing this since I identified and shared this gap couple of weeks, and already took care of reviewing skipped tests and ensure that the corresponding issues were triaged and prioritized as important.


docs: Link memoization and work-avoidance

Despite #10769 and #10426 both having examples of memoization not working with the examples having no output, no-one has picked up on this.
To address this improve the documentation for memoization and work-avoidance, linking the two ideas and pointing people who want to skip steps towards work-avoidance unless they are really doing what memoization was designed to do.
Issue #10426 is problematic in that some steps get memoized when perhaps they should't, so this commit shouldn't close it.
Fixes #10769

Thanks for clarifying on intended usage of Argo Workflows features.
Since an issue I've submitted triggered this change, let me describe our use case.
We're using Argo Workflows for CI and we're using memoization for work avoidance and I think it's valid usage. We don't want to run tests again for the same sets of inputs. So our expensive to compute output is... the information that tests pass for the given set of inputs. And we're happy that hooks run for memoized steps, because what we really need is green GitHub status check.
"Work avoidance" is not really Argo Workflows feature. It's just a technique that could be used to implement caching anywhere (which we also use with Argo in some places). But in order to not "abuse" memoization (replace memoization with "work avoidance"), we would have to repeat the same (part of) shell script in every template we want to cache, add input and output artifacts, handle artifact lifecycle etc. That's a lot of repeated YAML and shell code. And also pods would still be scheduled, run, they would fetch all input artifacts (cloning repositories for example) and would waste requested resources, increasing costs by a lot (especially for templates requesting a lot of cpu and memory). Why do that if we could just say "don't run this again for this set of inputs" which is exactly what memoization does?
Why not treat the success of template run part of its outputs and allow to memoize templates without outputs?
Memoization works really good when it works (when we don't run into 1MB limit of configmaps or this). It makes cached steps to be instant and makes Argo Workflows possible with monorepos.
(Let me know if this is not the right place to raise this concern).

I would also argue that this default approach is a bit odd to me. The memoization feature seems extremely powerful and restricting it to only steps with output seems counterintuitive when in a lot of cases it is the outcome with a set of parameters that matters, not necessarily the pure output. So if I memoized on my inputs and knew that the steps will produce the same output every time, I was expecting it not to have to run but give me a stamp of approval on the top level.
I mean I could put all the steps inside into a single template and memoize it but it seems counter intuitive when I can divide it into smaller reusable templates and then mix and match them and memoize the whole thing because the outcome of that whole template will never change for the same set of inputs.

I understand that people are unhappy that memoization doesn't work without outputs, and would like it to work.
This PR doesn't change any behaviour, it just tries to document the current behaviour, which even in the event of a PR to change the behaviour of memoziation needs documenting for users prior to that change.
@terrytangyuan, I'd like you to consider merging this as is unless you feel it's factually incorrect.
I have raised #11280 as a proposal to implement memoization as @moleskin-smile and @mnarozny would like it to work to see what the feelings of the community are.