Include trace server with packaged app

Modifies the way the path is obtained to work with the AppImage application. Also adds meaningful error messages when the path is not correct.
Related to #352
Motivation: Currently, users need to compile the Trace Viewer from source to view their own traces. To avoid the time-consuming dev setup and compilation we now offer a downloadable package (electron desktop application + trace server). Since this package currently needs to be generated and uploaded manually, this PR includes the packaging changes required so that any dev can easily create an updated package in the future.
Signed-off-by: Arnaud Fiorini<EMAIL_ADDRESS>

A workaround to not break any other AppImages that would have relative path to the AppImage location (and not the temporary path of the AppImage), would be to set an environment variable that toggles this behaviour.
The only way I found to include that environment variable at package time with the electron-builder is to make a script that hooks onto the AfterPackContext hook. This script will then change the packaged app to set an env variable when launching the AppImage.

This pull request have been superseeded by #594. Closing this for now


WIP: AIP-5016 batch partitioned data

Coverage decreased (-3.3%) to 94.463% when pulling 2e97123a7e378c39a755fa355cea561c38167946 on tz/AIP-5016-batch-partitioned into 1cbc12d5d8aed24f5415375aaae5d4206ce4e28c on main.


Ref sugar compile result incorrect with destructuring + arrow function expression

Version
3.0.11
Reproduction link
https://sfc.vuejs.org/#eyJBcHAudnVlIjoiPHNjcmlwdCBzZXR1cD5cbnJlZjogKHt9ID0gKCkgPT4gMSk7XG5yZWY6ICh7fSA9IHVzZVNvbXRoaW5nKCgpID0+IDEpKTtcbnJlZjogKHt9ID0gdXNlU29tdGhpbmcoY29tcHV0ZWQoKCkgPT4gMSkpKTtcbjwvc2NyaXB0PiJ9
Steps to reproduce
Check the JS compile result.
What is expected?
Result is:
const {} = (() => 1);
const {} = useSomthing((() => 1));
const {} = useSomthing(computed((() => 1)));

What is actually happening?
Result is:
const {} = ( => 1);
const {} = useSomthing(( => 1));
const {} = useSomthing(computed(( => 1)));

@johnsoncodehk Assigning a value to {} is not a wise choice, because {}==={} returns false and you will never be able to use the {} you defined.
But I think in this case, it would be better to report an exception when compiling.

@ygj6 this is just to show the problem, I use something like ref: ({ foo } = useSomthing(...)) in real code, thanks.

Oh~~ I did see @HcySunYang 's PR.


Class instance methods showing up as "anonymous_..."

I am trying to use nyc on my ES2015 code, having followed the guide. However, when the coverage report is generated, all instance methods of a class are reported as named "anonymous_x". I'm getting a similar result on methods defined on a prototype, but I'll omit that issue for now.
When running:

cross-env NODE_ENV=test nyc mocha "js/**/*.spec.js"

against:
.babelrc
{
    "presets": ["es2015"],
    "plugins": ["istanbul"]
}

Demo.js
export class Demo {
    constructor() {
        this.value = 1;
    }

    getValue() {
        return this.value;
    }

    print() {
        console.log(this.toString());
    }

    toString() {
        return this.value.toString();
    }
}

Demo.spec.js
import {Demo} from "./demo";

describe("demo", function() {
    it("Gets values", function() {
        let demo = new Demo();
        expect(demo.getValue()).to.eql(1);
    });
});

package.json
{
...
  "scripts": {
    "coverage": "cross-env NODE_ENV=test nyc mocha \"js/**/*.spec.js\""
  },
  "nyc": {
    "reportDir": "coverage",
    "require": [
      "babel-register"
    ],
    "include": [
      "js/**/*.js"
    ],
    "exclude": [
      "js/**/*.spec.js"
    ],
    "sourceMap": false,
    "instrument": false,
    "reporter": ["lcov"],
    "cache": true
  },
  "devDependencies": {
    "babel-cli": "^6.24.1",
    "babel-core": "^6.24.1",
    "babel-plugin-istanbul": "^4.1.4",
    "babel-preset-es2015": "^6.24.1",
    "babel-register": "^6.24.1",
    "chai": "^3.5.0",
    "cross-env": "^5.0.0",
    "mocha": "^3.4.2",
    "nyc": "^11.0.2-candidate.0",
    "sinon": "^2.0.0",
    "sinon-chai": "^2.10.0"
  }
...
}

the output is:
TN:
SF:/my/path/to/nyctest/js/demo.js
FN:2,(anonymous_0)
FN:6,(anonymous_1)
FN:10,(anonymous_2)
FN:14,(anonymous_3)
FNF:4
FNH:2
FNDA:1,(anonymous_0)
FNDA:1,(anonymous_1)
FNDA:0,(anonymous_2)
FNDA:0,(anonymous_3)
DA:3,1
DA:7,1
DA:11,0
DA:15,0
LF:4
LH:2
BRF:0
BRH:0
end_of_record

@awexfwex unfortunately this might simply be a symptom of how the babel transpilation is being applied, would need to see what the final instrumented code looks like; but there's a good chance that the one liner export class Demo ends up instrumenting the class in such a way that we lose the name.
do you also lose the name if you do this:
class Demo {
    constructor() {
        this.value = 1;
    }

    getValue() {
        return this.value;
    }

    print() {
        console.log(this.toString());
    }

    toString() {
        return this.value.toString();
    }
}

module.exports = Demo

The actual underlying issue will be in the instrumenter here.


plugins need detailed endpoint definitions

exactly like in Docs, all endpoints and methods need to be listed in an API Reference format.

+1

Closing as stale.


Offload log parsing from UI thread

This change partially addresses #160 by parsing SARIF on background tasks as much as possible.
The file reading and conversion (if necessary) are now being done on a background task. This alone makes the UI much more responsive while the log files are loading.
While the log file are being parsed, status is presented in Visual Studio's task center.
There is remaining work that can be done. For example, the data models can indeed be constructed on any thread and modified on any thread IF they are made thread safe and properly send change notifications on the UI thread for WPF/XAML. Also the "ruin data cache" needs to be made thread safe as well.
The error list implementation is already thread safe.
How verified:
Was able to use the editor to edit source code while a code analysis build and log parsing was in progress.
All unit-tests also pass.

Need to update the version information since the previous version has been distributed for testing.


[GitHub] Better PR search

Description
Current PR search is only finding my PRs, while it would be really nice to find PRs open in the repositories I am being involved as administrator.

Who will benefit from this feature?
GitHub users, who manage repositories. I would love to see a list of PRs I probably want to visit and review.
Maybe a reviewer list would be great here. Not sure about the ideal flow.
Anything else?
No response

Thanks for bringing this up. It appears that this is a duplicate of issue #4058, so I'll mark it as such.


[Feature #4793] Support MQTT protocol

Fixes #4793
Motivation
Explain the content here.
Explain why you want to make the changes and what problem you're trying to solve.
Modifications
Describe the modifications you've done.
Documentation

Does this pull request introduce a new feature? (yes / no)
If yes, how is the feature documented? (not applicable / docs / JavaDocs / not documented)
If a feature is not applicable for documentation, explain why?
If a feature is not documented yet in this PR, please create a followup issue for adding the documentation

@karsonto
I don’t understand the implementation of the publish processor, and I don’t see how messages of the mqtt protocol are converted and sent to eventmesh storage.

我没有理解对于publish processor的实现，没有看到将mqtt协议的消息进行转换后发送到eventmesh storage。

@xwm1992
This is just a preliminary design plan, and  will optimization  persist messages in the future.

This PR is the framework design of the MQTT protocol , and will continue to optimize persist messages in another PR.

Is eventmesh-protocol-plugin implementation the only work left for this protocol in Runtime?

I think there is no need to implement this module for the time being. If it needs to be implemented in the future, I will submit a PR to supplement it.


Make Connection private.

Only pass the peer's keys to the user and handle the Connection objects
in Service.

r? @canndrew
(maidsafe_highfive has picked a reviewer for you, use r? to override)

As this commit is being introduced in another PR, I'm closing it.


how to get sassbook.com/api/ext/summarize/v1a api  key

hello , how to get sassbook.com/api/ext/summarize/v1a api  key . ths.

Please email hello_at_sassbook.com with your API volume requirements and any other relevant usage pattern information. The API is only available as a commercial offering.


Update 041-logfile-plugin.md

The documentation for the logfile plugin reflected a previous version. This is the correct documentation.

@cgivre you have created cgivre-logfile-doc-update-1 branch in Drill project, not in your fork. Please remove the branch.

Hi Arina,
These files don't seem to come across in forks.  Should I just close the PR?  This is the second or third time this has happened to someone, so I'd like to make sure that the docs are correct.

As far as I remember when you delete remote branch PR will close automatically. if not please close it manually and create new from proper branch.
Updating the doc is ok, just don't create branches in Drill project (as a committer you do have write permission to the repo and should be careful with it), create branches in your own fork and then do the PR.


Cumsum rvs

Thank you for opening a PR!
Here are a few important guidelines and requirements to check before your PR can be merged:

[ ] There is an informative high-level description of the changes.
[ ] The description and/or commit message(s) references the relevant GitHub issue(s).
[ ] pre-commit is installed and set up.
[ ] The commit messages follow these guidelines.
[ ] The commits correspond to relevant logical changes, and there are no commits that fix changes introduced by other commits in the same branch/BR.  If your commit description starts with "Fix...", then you're probably making this mistake.
[ ] There are tests covering the changes introduced in the PR.

Don't worry, your PR doesn't need to be in perfect order to submit it.  As development progresses and/or reviewers request changes, you can always rewrite the history of your feature/PR branches.
If your PR is an ongoing effort and you would like to involve us in the process, simply make it a draft PR.

Looks good, but I don't think the CumSum code should be in scan.py—that module is pretty specific to Scan.

Yeah, I was not very happy with it tucked in there either. Aesara puts this is extra_ops though, should we do the same or something more direct?

Yeah, I was not very happy with it tucked in there either. Aesara puts this is extra_ops though, should we do the same or something more direct?

An extra_ops module is fine for now.

Edit: Otherwise, we can organize by conceptual types of RVs... like the mixtures.

Exactly


Implement sending in precedence order

Work towards https://github.com/la5nta/pat/issues/261

Sounds good to me. :) Thanks for the review!

Very happy that we finaly can support this 👍 Thank you!
I'll handle the wl2k-go update of the Pat repo after merging this to master. I usually include a small description of the changes, for convenience when compiling the release changelog 🙂


Exit race condition on library handle usage

There seems to be some sort of race condition on the _LIB handle (in base.py) when calling exit() from python.
Code:
import mxnet as mx
data = mx.sym.Variable('data')
# del data
exit(0)

The code above occasionally trigger the following exception:
Exception ignored in: <bound method SymbolBase.__del__ of <Symbol data>>
Traceback (most recent call last):
  File "/OSM/CBR/D61_RCV/work/and502/build/mxnet/python/mxnet/_ctypes/symbol.py", line 46, in __del__
AttributeError: 'NoneType' object has no attribute 'NNSymbolFree'

NOTE: the error is only triggered "sometimes", and the particular missing attribute may be other than 'NNSymbolFree'.
I guess the issue is that the _LIB handle gets deleted and set to None before all the objects using it have been deleted. If uncommenting the "del data" line, the problem seems to go away, or is at least a lot harder to trigger.
This happens in master as of today using python 3 under Ubuntu 16.04.

What's the version of mxnet? @cjolivier01 had a fix on exit race condition a while back.

This sort of error doesn’t look familiar. What’s the difference in behavior of just letting the script finish rather than calling exit(0)?
I’m not a python expert — does exit(0) cause a more immediate exit than reaching th end of the script? Is python shutdown procedure different?

I guess calling exit(0) before reaching the end changes the order in which objects are deleted. I can't reproduce the error without the exit statement, so leaving it out probably causes the "data" object to be  deterministically deleted before the _LIB handle, while when using exit(), the _LIB handle sometimes gets deleted before the "data" object, causing the errors seen.

yes, I tried reproducing this but could not.  However, I think your assessment is correct -- _LIB gets destroyed before data, since 'data' doesn't take the normal 'going out of scope' destruction path and gets destroyed during general cleanup, which appears to not be deterministic in order.
It does not appear to be related to the engine shutdown, which was the focus of my previous shutdown fix.
I do not know if there is a way to avoid this except to use an exception to exit or something like that (which could be caught near a benign entry point), which will allow scope-destruction to occur in an organized manner.
Do you think we can close this issue?

Please reopen if necessary


s3dis数据提取错误      s3dis  data extraction error

我已经按照官方的指导文件放置好了s3dis数据集，但是在运行   python collect_indoor3d_data.py    指令的时候会报错，提示    FileNotFoundError: [Errno 2] No such file or directory: './s3dis_data\Stanford3dDataset_v1.2_Aligned_Version\Area_1_conferenceRoom_1_point.npy'     。
我的工作目录也切换到了  data\s3dis  目录下

I have placed the s3dis dataset according to the official guide file, but when running    python collect_indoor3d_data.py    will report an error, prompting
FileNotFoundError: [Errno 2] No such file or directory: './s3dis_data\Stanford3dDataset_v1.2_Aligned_Version\Area_1_conferenceRoom_1_point.npy'     。
I also switched my working directory to   data\s3dis

same problem, did you solve it?

Same issue, was a solution found?

This is solved. The problem originates from the fact that the code was implemented for linux and not a windows machine. In collect_indoor_3d_data.py , anno_path.split('/') only splits up the string according to '/', but in windows directories we find '\'.

A solution could be to directly change the out_filename variable to take the last 6 letters in the directory string for elements[-3], such that only Area_1, Area_2 and so on get picked up in the directory.


Add project setup with kubebuilder

Kubebuilder requires no directories and pre-existing files present when creating a new project.
Therefore, created a branch with kuberbuilder project and merged it with main branch.
Getting this in, will make follow up pr's smaller and easier to review.

@Rakshith-R looks  like PR is having conflict can you please check

@Rakshith-R looks like PR is having conflict can you please check

its fixed now, please take a look

I have created https://github.com/csi-addons/kubernetes-csi-addons/issues/9 to track task related to image name and build ci, deploy images.
(changing image name in Makefile needs additional changes in default dev environment.)

rebased on top of https://github.com/csi-addons/kubernetes-csi-addons/pull/15. PTAL

@nixpanic PTAL. if you don't have any comments we can merge this one and have follow-up PR's for enhancements.


[Bug]: Logo size in mobile

Fix the logo for smaller views.

@abhijeet007rocks8 please assign me this issue.


Test importing images

Changes:

add image-rs
import.rs checks the file extension and returns an Import::{Ply,Image} enum
add import_image fn to state
render images on the interface

fixes #8

run cargo clippy to see clippy errors

I think this makes sense, at least initially. We should be able to scroll through a video like this, which makes sense. Go ahead and merge whenever you think it is good.


PostgreSQL versions

First off, I apologize if this is the wrong place for this, but I couldn't find anywhere that looked more correct.  If there is one, please point me at it, and I'll use it.
Is there a way to get a testcontainer with a more current version of PostgreSQL than PostgreSQL 9.6.12 on x86_64-pc-linux-gnu (Debian 9.6.12-1.pgdg90+1), compiled by gcc (Debian 6.3.0-18+deb9u1) 6.3.0 20170516, 64-bit.  Perhaps "current" is the wrong word, "higher initial version number"?  Like 11.X, 12.Y, or 13.Z?
In addition is there any documented mapping from org.testcontainers.postgresql versions to postgresql versions?  I couldn't find a map, then I assumed that 1.13.0 was a container of pre-alpha PostgreSQL 13, but I was wrong...
Thank you very much.

Module version have nothing to do with PostgreSQL version, you can specify version of container while creating container
 new PostgreSQLContainer("postgres:9.6-alpine")

Hi @dabraham02124,
We use PostgreSQL's official Docker images, you will find all requires info about them here:
https://hub.docker.com/_/postgres

then I assumed that 1.13.0 was a container of pre-alpha PostgreSQL 13

FYI Testcontainers' version is independent from PostgreSQL's.

@artjomka  & @bsideup thank you to both of you.
Would it be appropriate if I submitted a PR with javadoc on the PostgreSQLContainer describing this?


rye: Add version 0.1.1

Relates to #11301

[x] I have read the Contributing Guide.

/verify

#11301


fix(notify): remove topic_tbl.advance_days

What this PR does / why we need it:
add option to send notification

[ ] Smoke testing completed
[ ] Unit test written

Does this PR need to be backport to the previous release branch?:
If no, just write "NONE".
If don't know, write "UNKNOWN", and let the reviewer decide.
If yes, write the release branches name in the below format and submit the related cherry-pick PR:

release/3.10
/cc @ioito @zexi @swordqiu

/lgtm


Regression in 0.30.0: changing config after createVitest no longer supported

Describe the bug
Using the node API no longer supports overriding configuration after createVitest occurs.
This worked in<EMAIL_ADDRESS>but no longer since<EMAIL_ADDRESS>const vitest = await createVitest("test", {});
vitest.config.setupFiles = ["vitest.setup.js",  ...vitest.config.setupFiles];
await vitest.start([]);

It is crucial for frameworks building upon vitest to be able to add setupFiles removing the existing setupFiles.
Reproduction
See this git repo: vitest-setup-files.zip
After npm i, you can run node run-vitest.js to see the error—after downgrading vitest: npm i -D<EMAIL_ADDRESS>running node run-vitest.js works.
System Info

System info
  System:
    OS: Linux 5.15 Ubuntu 20.04.5 LTS (Focal Fossa)
    CPU: (16) x64 11th Gen Intel(R) Core(TM) i7-11850H @ 2.50GHz
    Memory: 5.58 GB / 15.49 GB
    Container: Yes
    Shell: 5.0.17 - /bin/bash
  Binaries:
    Node: 18.12.1 - ~/.nvm/versions/node/v18.12.1/bin/node
    Yarn: 1.22.19 - ~/.nvm/versions/node/v18.12.1/bin/yarn
    npm: 8.19.2 - ~/.nvm/versions/node/v18.12.1/bin/npm
  Browsers:
    Chrome: 110.0.5481.77
  npmPackages:
    vitest: ^0.30.0 => 0.30.0

Used Package Manager
npm
Validations

[X] Follow our Code of Conduct
[X] Read the Contributing Guidelines.
[X] Read the docs.
[X] Check that there isn't already an issue that reports the same bug to avoid creating a duplicate.
[X] Check that this is a concrete bug. For Q&A open a GitHub Discussion or join our Discord Chat Server.
[X] The provided reproduction is a minimal reproducible example of the bug.

Vitest now has support for workspaces, so you need to update config on vitest.workspaces[0].config.setupFiles (there can be more than one)
The config on the root vitest is used only for top level options like reporter, runner and so on.

Thanks for the fast response again @sheremet-va ! 🙏
vitest.workspaces doesn't exist, but I can use vitest.getCoreWorkspaceProject().config.setupFiles . What is that exactly and would that add the setup file for each workspace in the project?
There is also vitest.getModuleProjects(), but it needs an ID.

Thanks for the fast response again @sheremet-va ! 🙏
vitest.workspaces doesn't exist, but I can use vitest.getCoreWorkspaceProject().config.setupFiles . What is that exactly and would that add the setup file for each workspace in the project?
There is also vitest.getModuleProjects(), but it needs an ID.

I checked the code and you need to use vitest.projects, not vitest.workspaces

What is that exactly and would that add the setup file for each workspace in the project?

*for each project in a workspace. Please, refer to https://vitest.dev/guide/workspace.html for workspace configuration.

Thanks for all the info! This should be enough to get our @stryker-mutator/vitest-runner working.
F.y.i. we're currently using fs.writeFile to communicate between the test environment and the plugin. We might want to have a more formal approach (I'm pretty sure the file system is inaccessible when browser mode is enabled). Let's discuss that further in #3017 in the future.


Prepatory work for the Az docs rollout

Remove Docker as a supported install method

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs-conceptual/az-0.3.0/authenticate-azureps.md
:white_check_mark:Succeeded

docs-conceptual/az-0.3.0/azureps-vm-tutorial.yml
:white_check_mark:Succeeded

docs-conceptual/az-0.3.0/context-persistence.md
:white_check_mark:Succeeded

docs-conceptual/az-0.3.0/create-azure-service-principal-azureps.md
:white_check_mark:Succeeded

docs-conceptual/az-0.3.0/formatting-output.md
:white_check_mark:Succeeded

docs-conceptual/az-0.3.0/get-started-azureps.md
:white_check_mark:Succeeded

docs-conceptual/az-0.3.0/install-azurerm-ps.md
:white_check_mark:Succeeded

docs-conceptual/az-0.3.0/install-azurermps-maclinux.md
:white_check_mark:Succeeded

docs-conceptual/az-0.3.0/manage-subscriptions-azureps.md
:white_check_mark:Succeeded

docs-conceptual/az-0.3.0/migration-az-0.3.0.md
:white_check_mark:Succeeded

docs-conceptual/az-0.3.0/other-install.md
:white_check_mark:Succeeded

docs-conceptual/az-0.3.0/overview.md
:white_check_mark:Succeeded

docs-conceptual/az-0.3.0/queries-azureps.md
:white_check_mark:Succeeded

docs-conceptual/az-0.3.0/release-notes-az.md
:white_check_mark:Succeeded

docs-conceptual/az-0.3.0/toc.yml
:white_check_mark:Succeeded

docs-conceptual/az-0.3.0/uninstall-azurerm-ps.md
:white_check_mark:Succeeded

docs-conceptual/az-0.3.0/using-experimental-modules.md
:white_check_mark:Succeeded

docs-conceptual/az-0.3.0/using-psjobs.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


Cannot declare a license on scala_import

Hello,
I tried the new scala_import rule, and stumbled upon the following error:
ERROR: [...]/third_party/java/com/twitter/BUILD:237:1: //third_party/java/com/twitter:finagle_http: no such attribute 'licenses' in 'scala_import' rule
ERROR: [...]/third_party/java/com/twitter/BUILD:237:1: third-party rule '//third_party/java/com/twitter:finagle_http' lacks a license declaration with one of the following types: notice, reciprocal, permissive, restricted, unencumbered, by_exception_only

It seems that is not possible either on other Scala rules (unless I am mistaken), and it is indeed required to load third-party Scala rules.

Hey Vincent,
Thanks for the report!
Would you be open to sending a PR?
2 notes:

You can circumvent around this by defining the license on the package
(which is what we do)
There is a bazel issue around making this check non mandatory (AFK so
can't link to it right now)
On Sat, 2 Dec 2017 at 17:27 Vincent<EMAIL_ADDRESS>wrote:

Hello,
I tried the new scala_import rule, and stumbled upon the following error:
ERROR: [...]/third_party/java/com/twitter/BUILD:237:1: //third_party/java/com/twitter:finagle_http: no such attribute 'licenses' in 'scala_import' rule
ERROR: [...]/third_party/java/com/twitter/BUILD:237:1: third-party rule '//third_party/java/com/twitter:finagle_http' lacks a license declaration with one of the following types: notice, reciprocal, permissive, restricted, unencumbered, by_exception_only
It seems that is not possible either on other Scala rules (unless I am
mistaken), and it is indeed required to load third-party Scala rules.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/bazelbuild/rules_scala/issues/353, or mute the thread
https://github.com/notifications/unsubscribe-auth/ABUIF0YBFI8tCLzsnMm163aeaXB-pcOUks5s8WxfgaJpZM4QzV_p
.

What is giving this error?
What are you using to build this third_party directory?

Bazel.
https://github.com/bazelbuild/bazel/issues/3639
On Sat, Dec 2, 2017 at 7:23 PM P. Oscar Boykin<EMAIL_ADDRESS>wrote:

What is giving this error?
What are you using to build this third_party directory?
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/bazelbuild/rules_scala/issues/353#issuecomment-348706301,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABUIFx2aBqqGKsHPlrGIgYK9kKecXnkRks5s8YeGgaJpZM4QzV_p
.

Ahh. We use 3rdparty so we bypass this I guess.
On Sat, Dec 2, 2017 at 07:48 Ittai Zeidman<EMAIL_ADDRESS>wrote:

Bazel.
https://github.com/bazelbuild/bazel/issues/3639
On Sat, Dec 2, 2017 at 7:23 PM P. Oscar Boykin<EMAIL_ADDRESS>wrote:

What is giving this error?
What are you using to build this third_party directory?
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
<
https://github.com/bazelbuild/rules_scala/issues/353#issuecomment-348706301
,
or mute the thread
<
https://github.com/notifications/unsubscribe-auth/ABUIFx2aBqqGKsHPlrGIgYK9kKecXnkRks5s8YeGgaJpZM4QzV_p
.

—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/bazelbuild/rules_scala/issues/353#issuecomment-348708009,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAEJdknsyMtbnCcZTm0Hrju-r7zAqmAaks5s8Y1lgaJpZM4QzV_p
.

--
P. Oscar Boykin, Ph.D. | http://twitter.com/posco | http://pobox.com/~boykin


[feat] - Aux Heating Reset

Hello, thank you so much for this it has been a massive help!
This section is not standard for thermostats
(If the timeout ends and the heater was on for the whole time the thermostat switches to the secondary heater. In this case the primary heater (heater) will be turned off. This will be remembered for the day it turned on and in the next heating cycle the secondary heater will turn on automatically. On the next day the primary heater will turn on again the second stage will again only turn on after a timeout. If the third secondary heater_dual_mode is set to true the secondary heater will be turned on together with the primary heater.)
most thermostats will reset after the setpoint is achieved and on the next heating cycle start with the first primary heater.
to make this code work like a normal thermostat where it always starts with the first heating method and swaps to the second after 15mins.
all i had to do was change this in (/custom_components/dual_smart_thermostat/hvac_device/heater_aux_heater_device.py)
@property
def _has_aux_heating_ran_today(self) -> bool:
    """Determines if the aux heater has been used today."""
    if self._aux_heater_last_run is None:
        return False

    if self._aux_heater_last_run.date() == datetime.datetime.now().date():
        return True

    return False

to
@property
def _has_aux_heating_ran_today(self) -> bool:
    """Determines if the aux heater has been used today."""
    if self._aux_heater_last_run is None:
        return False

    if self._aux_heater_last_run.date() == datetime.datetime.now().date():
        return False

    return False

I have put this in the issues section as I believe this is an issue that should be acknowledged somewhere.

The current logic was implemented based on this request: #4.
I can improve it by introducing a flag that controls wether the thermostat should start a new cycle each time.
What do you think?

Hey

The current logic was implemented based on this request: #4.
I can improve it by introducing a flag that controls wether the thermostat should start a new cycle each time.
What do you think?

Hey swingerman,
This would be great, and maybe even adding a field that we can specify the timeout period. I'm looking at using this to replace the generic thermostats I'm currently using but that was my one hold up. I currently use automations to do this but it's a bit of a pain to change as I move things around.
Basically would love the ability to set time out period, allow individual temperature tolerance between prim and aux, and maybe even allow have it reference a input_boolean toggle for the aux heat to even activate (Or maybe a separate temperature sensor, wouldn't want aux heat to work in lets say 13*c(55F) weather since it just wouldn't be needed. But would be nice to not have to worry about changing this around based on the time of year.

The current logic was implemented based on this request: #4.
I can improve it by introducing a flag that controls wether the thermostat should start a new cycle each time.
What do you think?

Yes please and thank you sir!
But whatever works for you, its your thermostat. :)


Integration Test - Release - 2019/04/04 08:00:13

Auto create for integration test. IssueCommandFlow

@PRMergerTest2 : Thanks for your contribution! The author, @v-caxian, has been notified to review your proposed change.


Delivery Speed is cleared by the Google Side or the api client after update/patch the account shipping rates

I am testing the google content api which update the shipping price only.
The workflow is:
$googleConfig = new Google_Config();
$googleConfig->setClassConfig('Google_Cache_File', array('directory' =>"/tmppath"));
    $client = new Google_Client($googleConfig);
    $client_email = $configs->client_email;
    $private_key = $configs->private_key;

    $scopes = array("https://www.googleapis.com/auth/content");
    $credentials = new Google_Auth_AssertionCredentials(
        $client_email,
        $scopes,
        $private_key
    );
    $client->setAssertionCredentials($credentials);
    $this->merchant_id = $configs->merchantId;
    $this->service = new Google_Service_ShoppingContent($client);
$accountShipping = $this->service->accountshipping->get($this->merchant_id ,$this->merchant_id);
$rateTables=$accountShipping->getRateTables();

//The left flow is loop the ratetables and change the price value only. ..............
foreach($rateTables as $rateTable){
$rules=$rateTable->getContent();
foreach($rules as $rule){
$price=$rule->getRate();
$price->setValue(22);
}
$this->service->accountshipping->update($this->merchant_id,$this->merchant_id,$accountShipping);
The problem is the delivery date is cleared which is already set by the admin GUI.
Is it the API bug?

Did you get this working yet?


forge 1.18.2 crash

the dwarf_sword has a bug,
when i put this into anvil,
and put a enchanted_book,
then game crashed.
i think the reason is
DWARF_SWORD(3, 2000, 9.0F, 1.0F, 12, () -> {
    return null;
})

maybe you should replace the null to Ingredient.EMPTY

My English is terrible, so it's not clear


incompatible character encodings when calling .to_s on tree parsed from single regex

related: https://github.com/rubocop-hq/rubocop/issues/9056
Regexp::Parser.parse((/¡#≥
/x).to_s).to_s

expected: all nodes of tree are utf-8 (they all come from a utf-8 source string)
actual: raises Encoding::CompatibilityError: incompatible character encodings: UTF-8 and ASCII-8BIT
thank you for your great lib!

hi @michaelglass and thank you for the report!
rubocop is really turning out to be a great detector of wrinkles that still need to be ironed out.
i've just released v2.0.0 where this is fixed.
heads up: this version also includes a change to address https://github.com/ammar/regexp_parser/issues/72, which means that any code in rubocop that calculates indices of regexp parts will need to be updated when moving to 2.0.0.


Unable to use mcuboot with zephyr and nrf52840_pca10059 (dongle)

Hi, I am trying the procedure for using mcuboot with nrf52840_pca10059 (dongle) as given in the documentation ( [https://docs.zephyrproject.org/latest/boards/arm/nrf52840_pca10059/doc/index.html?highlight=nrf52840_pca10059]), I have achieved up to step 4 of flashing the zip using nrfutil dfu usb-serial -pkg mcuboot.zip -p /dev/ttyACM0 but next when I do the subsequent steps Boot into MCUboot serial recovery mode by plugging the board in with the SW1 button pressed down I do not get the mentioned /dev/serial/by-id/usb-ZEPHYR_MCUBOOT_0.01-if00 as I am using Ubuntu 19.10 even my /dev/ttyACM0 is missing and hence I am unable to perform the steps which ensue

You do not have any file(s) under /dev/serial/by-id/? Also check that the board was detect with dmesg.

no I do not have any file(s) under /dev/serial/by-id/ will try dmesg again with mcuboot and get back to you, when I ran the zephyr/samples/subsys/usb/cdc_acm with nordic open bootloader I got the following on dmesg
[ 1749.112399] cdc_acm 3-3:1.0: ttyACM0: USB ACM device
[ 1758.442850] usb 3-3: USB disconnect, device number 6
[ 1758.443211] cdc_acm 3-3:1.0: failed to set dtr/rts

fixed by #651


fixup: strip binaries when linking.

This PR adds -s -w to the ldflags option so that produced binaries are stripped of their symbols. This reduces the size of the agent from ~20M to ~14M.
My suggestion is that we refactor this slightly to add a debug build option so that developers can easily build binaries with symbols, if necessary. cc @tomasger @iogakos

Pushed a fixup to enable build-dbg target in Makefile.

Before publishing this change, make sure to take care of the one-line installer command too, it didn't seem trivial to add a flag to a shell script downloaded with curl. Also I'd suggest we only merge it when platform opens as debug mode by default is useful

Before publishing this change, make sure to take care of the one-line installer command too, it didn't seem trivial to add a flag to a shell script downloaded with curl. Also I'd suggest we only merge it when platform opens as debug mode by default is useful

I tried pprof on the stripped binary and looks like tracing is not affected, but I think using any kind of debugger (ie dev, dbg etc) won’t be possible. Regarding the one-liner, I don’t think it’s worth adding dbg support on it. If the extra symbols are needed, one could use make build-dbg and use the produced binary instead.

Production binaries should not have symbols. Symbols are there for developers to be able to debug, not for users.


Full screen off canvas block parameter missing

How to reproduce this bug:

Install ZURB template
run foundation blocks install off-canvas-full-screen
run foundation watch

What should happen:
What happened instead:
Compilation of sass fails with message:
Error in plugin 'sass'
Message:
src/assets/scss/components/building-blocks/_off-canvas-full-screen.scss
Error: Mixin off-canvas-position has no parameter named $size
on line 7 of src/assets/scss/components/building-blocks/_off-canvas-full-screen.scss

n($position: $offcanvas-full-screen-side, $size: 100%);
------------------------------------------^

Foundation Version(s) you are using:
6.4.3
Worked in 6.4.1

You need to update your settings file!

Ok. Tell me how?
settingsfile?
There are hundreds.
Whitch one is it?
Thank you!

Från: Harry Manchanda<EMAIL_ADDRESS>Skickat: den 21 augusti 2017 17:26:57
Till: zurb/foundation-sites
Kopia: kasperhard; Author
Ämne: Re: [zurb/foundation-sites] Full screen off canvas block parameter missing (#10561)
You need to update your settings file!
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/zurb/foundation-sites/issues/10561#issuecomment-323774454, or mute the threadhttps://github.com/notifications/unsubscribe-auth/ASedLf3X0tqxU_n9sppYO8qMyILf9QNxks5saaHAgaJpZM4O9Nlm.

125.000 files.
Whitch one is it?

Tell me how its done: IamManchanda !

@kasperhard This is indeed a small bug (the block needs readjustment as the off-canvass-position mixin was changed.
For now you can edit _off-canvass.scss in components folder of foundation-sites node_module. Just manually add
    width: 100%;
    transform: translateX(-100%);

right after line 198 inside @if $position == left {}
Maybe someone here with a deeper undestanding of the mixins and can offer a cleaner solution, but this one works for now.

@nikitasol Thank you!

Is this solved?

@DanielRuf The workaround works, but the issue is still there. The building block has to be updated to reflect the updated _off-canvas component use of sizes value.

@colin-marshall who can do this?

@DanielRuf The issue is actually because the off-canvas component was updated to support a map (I forget which version but @SassNinja implemented it I think). $size is now $sizes and has to be a map: https://github.com/zurb/foundation-sites/blob/develop/scss/components/_off-canvas.scss#L192
The building block ideally just needs the variable updated with a $sizes map and it should work OK.
I'm not sure how we make changes to building blocks though.

We just need to put in a pull request at https://github.com/zurb/building-blocks/
We have write access @brettsmason. Not sure how they get updated on the building blocks site though.

https://github.com/zurb/building-blocks/blob/3e0f0eca393f02b81b960da03db67af2fa515fa6/gulpfile.babel.js#L75


Split into different types of projects

Folks at the software design group today suggested having three separate templates for each of three different projects:

data analysis for a single project (e.g., not a tool intended for reuse)
tool development (non-API)
tool development with APIs

Split into two project types (ToolDev and DataAnalysis) per discussion with Kate.


Handle negative counts from quasi_probability_dist

What is happening?
Because we do Measurement Error Mitigation, sometimes there are negative counts. We should convert probability distributions to the nearest possible normalized version. See here and here.
What are the current alternatives?
Round negative counts to zero, i.e., for cirq.ResultDict outputs in cirq_superstaq.

+1 for applying maximum-likelihood corrections rather than setting negative entries to zero.

@richrines1 @vtomole - this got brought up during SiW today. Rather than opening a new issue I'll paste my reproducing example here.
Essentially, I wanted to run the Supermarq bit code benchmarks on IBM Guadalupe, but the Bitcode.score() function throws an error because it expects an actual probability distribution to be passed in. Code to reproduce the error:
import cirq_superstaq as css
import supermarq

service = css.Service()
bitcode_job = service.get_job('c27dfa88-8164-4130-af8f-cac83893b27a')
counts1 = bitcode_job.counts()

bitcode_bmark = supermarq.bit_code.BitCode(3, 1, [1,0,1])
bitcode_bmark.score(counts1)

Two things:

Digging into this I realized the benchmark score function was failing because I had incorrectly initialized the object with a str rather than a List[int]. Opened #825 to make sure the benchmarks catch this.
The code to properly convert the quasidist back into a regular counts dictionary is given below. (not quite perfect because the final counts are not integers, but we could instead just return a distribution by dividing by total shots)

quasdist = qiskit.result.QuasiDistribution(counts1, shots=bitcode_job.repetitions())
neardist = quasdist.nearest_probability_distribution()
correct_dist = {f'{key:0{quasdist._num_bits}b}': value for key, value in neardist.items()}

imo it was a mistake for us to just return quasi probabilities as if they were standard probabilities. i think this should be fixed in a server-side update to return both the actual and quasi probabilities in a separate fields


Minor Fixes & Improvements, Formating Rehaul

Added main() function to run the program, allowing more flexibility to allow the user to input master password more than once if they messed up. Also better syntax.
Added ability to return to menu if "add profile" is selected without having to input anything
Cleaned up booleans
main_pwd_manager added to run the manager inside a function that can be called up
fileSetup() loads up salting and verifier
Added all manager functions as methods for better syntax, flexibility, readability, and editability

Great fixes! Thanks.


ninja executable is called ninja_build on Fedora Linux v22

Hi James,
it would be nice to check for ninja_build in case ninja can't be found. My workaround so far:
sudo ln -s ninja-build /usr/bin/ninja

@autopulated

is ninja_build / ninja-build a name that some linux distros use?

Ok should have read the title (=yes, fedora v22). Thanks for reporting this.

@autopulated In case you want to test your changes, you can find a VMware image of a yotta installation on Fedora 22 at http://get.meriac.com/ARMmbed.zip

@meriac which version of python ships with fedora v22? (There's a helpful shutil.which function in python >=3.3, would be nice to use that)

Thanks a lot for the patch!
@autopulated FC22 uses python 2.7

no problem, figured it wasn't too hard to detect things even without shutil.which


Amend the python build to create executable linked to shared library

This commit fixes the issue that the python executable built requires
specifiying the LD_LIBRARY_PATH to load the correct library versions.
This issue is seen by comparing the output of python -V with
LD_LIBRARY_PATH=/usr/local/lib python -V . The former returns an
incorrect version (2.7.5) whereas the latter return the correct version
(2.7.15).
The default prefix when building python from source is /usr/local.
However the executable built loads the shared libraries from /lib64
(symlinked to /usr/lib64) which ends up loading the (incorrect) system
libpython2.7.so instead of loading it from the one that is built in
/usr/local/lib
The libraries being loaded by the executable are visible by executing:
For the correctly built exectuable it is :
[root@90fbc9dd6cfa /]# ldd /usr/local/bin/python
	libpython2.7.so.1.0 => /usr/local/lib/libpython2.7.so.1.0 (0x00007fedc6348000)

whereas for the current version (before this commit) the so is loaded
from /lib64 (-> /usr/lib64)
The fix is to either:

Specify the library path like so before invoking the python
exectuable:
LD_LIBRARY_PATH=/usr/local/lib /usr/local/bin/python -V

OR,

Specify the library path at compile time using LD_RUN_PATH (as done
in this commit) so that it does not need to be specified at runtime and
does not require alterations to existing Jenkinsfile

Do not merge yet

Ready for review and merge

The CI (push) build failed with the error The job exceeded the maximum time limit for jobs, and has been terminated.

The Travis Build are failing due to the --enable-optimizations flag for python builds which triggers Profile Guided Optimizations (PGO) requiring the execution of all tests to generate profile data which  is then used to optimize the compiled code (this results in a faster python executable being generated). However, the side effect of this is that the builds take much longer and hit the timeout limit that Travis CI has for a job.
For the change in build times look at this comment (and the conversation thread itself) : https://github.com/docker-library/python/issues/160#issuecomment-263717667
Travis CI build timeouts are documented here: https://docs.travis-ci.com/user/customizing-the-build/#build-timeouts
Given the above, I am going to merge this, as well as PR #17 in.


ERR_REQUIRE_ESM when trying to start sst with sst dev

After starting SST with sst dev I get the following error:
|  Error
|  Error [ERR_REQUIRE_ESM]: require() of ES Module /Users/alessandrovolpicella/repo/sfn-wait-callback-ion/.sst/platform/node_modules/string-width/index.js from /Users/alessandrovolpicella/repo/sfn-wait-callback-ion/.sst/platform/node_modules/wide-align/align.js not supported.
|  Instead change the require of index.js in /Users/alessandrovolpicella/repo/sfn-wait-callback-ion/.sst/platform/node_modules/wide-align/align.js to a dynamic import() which is available in all CommonJS modules.
|      at Object.<anonymous> (/Users/alessandrovolpicella/repo/sfn-wait-callback-ion/.sst/platform/node_modules/wide-align/align.js:2:19) {
|    code: 'ERR_REQUIRE_ESM'
|  }

×  Failed
   Error [ERR_REQUIRE_ESM]: require() of ES Module /Users/alessandrovolpicella/repo/sfn-wait-callback-ion/.sst/platform/node_modules/string-width/index.js from /Users/alessandrovolpicella/repo/sfn-wait-callback-ion/.sst/platform/node_modules/wide-align/align.js not supported.
   Instead change the require of index.js in /Users/alessandrovolpicella/repo/sfn-wait-callback-ion/.sst/platform/node_modules/wide-align/align.js to a dynamic import() which is available in all CommonJS modules.
       at Object.<anonymous> (/Users/alessandrovolpicella/repo/sfn-wait-callback-ion/.sst/platform/node_modules/wide-align/align.js:2:19) {
     code: 'ERR_REQUIRE_ESM'
   }

sst.config.ts
/// <reference path="./.sst/platform/config.d.ts" />

export default $config({
  app(input) {
    return {
      name: "sfn-wait-callback-ion",
      removal: input?.stage === "prod" ? "retain" : "remove",
      home: "aws",
    };
  },

  async run() {
    new sst.aws.Nextjs("Frontend", {
      path: "packages/frontend",
    });
  },
});

node: 20.12.0
SST/ION: 0.0.224

if you initialize a fresh nextjs project do you get the same issue?

I get this error in my Github action when it tries to sst deploy but I don't see it when I deploy locally

if you initialize a fresh nextjs project do you get the same issue?

No then it works if I use drop-in mode

I got this error locally using Remix component as well, and resolved it by removing the .sst/platform folder and running npm run dev again.

this should be fixed in 0.0.231 but if it happens again feel free to reopen

removing the .sst/platform folder and running

Got the same issue, on ion 0.0.237... removing the .sst/platform folder and running npm run dev worked for me.

Hmm, can you post here if it happens again?


Add new task for running NUnit-Tests

Running tests with Visual Studio Test is a great way to execute your tests. Given the right TestAdapters, you can run all different types of tests.
However, when building a Xamarin.iOS app on a Mac with the VSTS Build Agent, there simply is no vstest.console.exe available.
It would be great if there was a task available, that allows to run nunit-tests via the nunit-console runner.

This is on our backlog, but no timeline as of now. I guess this is not blocking as you can run using ShellScript task and then upload results using Publish Test Results?
Thanks!

Great to hear. For now, running a ShellScript with nunit-console and Publish Test Results is fine. Thanks!

Thanks for confirming!

Just seen this, after I opened my own issue about the same topic, @apacha @atinb any of you can provide me a sample script that would run nunit tests from the command line (never did that before).

I too am trying to get nunit tests run on my mac build agent using terminal, with little success... any sample shell script would be wonderful.

@wes566 look at this capture : https://github.com/Microsoft/vsts-tasks/issues/1904#issuecomment-227664257

@LiohAu
Thanks! :)


More fused silica data

Found this paper that reviews a ton of experimental Fused silica refractive indexes, ranging form hard UV to deep IR.
I might get around to formatting some of this data, but not sure when.
https://www.osapublishing.org/ao/abstract.cfm?uri=ao-46-33-8118

Authors claim that digital data is available upon request. May be worth asking.

I implemented the gaussian function based refractive index / extinction coefficient formula and it works nicely.
Also they say that the Malitson formula (eq. 20 in the paper) has been shown to work for a wider range than originally intended - to around 6.7mkm, up from the current 3.71mkm


How to keep row height in proportion across all breakpoints

Summary
Can you please tell me which setting to use to tell the calendar to keep the "week" row heights in proportion with the cell widths of the "month" view so that the appearance is not so unappealingly stretched out on smaller devices?
Screenshots

Version
1.14.1, installed with npm.

Yeah, I tried lowering the container height, but below a certain value the calendar ignores what I specify when it calculates the row height.


Text for print-succes page

When you've successfully printed your paper-wallet, you will be lead to a print-succes page (see #324 ). This page needs txt that says something similar to: "You've printed your paper wallet. The private key on your paper wallet will always give you access to your SWT balance."
To be doublechecked with Matt

How about: "You've successfully printed your paper wallet. This contains your private key, which will always give you access to your SWT balance."
(Should we warn them to keep it safe, than anyone who has it will get access to their SWT?)

@matthewcarano because it is not protected with a password, maybe not a bad idea.

Then this:
"You've successfully printed your paper wallet. This contains your private key, which will always give you access to your SWT balance."
"Warning! Always keep your private key secure. If anyone were to get ahold of it, they would gain access to your SWT balance."


Standard library entries to match Getting Started guide

This issue has been marked as a "Good First Issue"! If you'd like to tackle this issue as your first contribution to the project, be sure to read the Get Involved section of the README for some help with how to get started.
This is also a very large issue with many parts. It is meant to be tackled piece-by-piece. A contribution that implements even one of these methods will gladly be accepted.
For the past few weeks, I've been working on a Getting Started guide for Myst. In that guide, I make some references to standard library functions that don't actually exist yet. The goal of this issue is to list all of the functions that are referenced that do not yet have an implementation so that they can be added one by one. Since the guide is not finished, entries may be added to this issue in the future.
Additionally, there are some other functions that I think would be good additions for the next release.
Some of these functions will be part of the native library (written in Crystal), while others will be part of the standard library (written in Myst). Each entry here has a small description of what it should do. For the most part, these follow the versions from Ruby, so feel free to look there for inspiration.
Native library

[x] Map#+(other : Map)
Add two Maps together into a new Map. If a key exists in both Maps, the value from the second map should be used.

[x] List#-(other : List)
Return a new List with only the entries from the first List that do not exist in the second.

[x] Integer#<(other)
Return true if the integer is numerically less than other. Return false otherwise.

[x] Integer#<=(other)
Return true if the integer is numerically less than or equal to other. Return false otherwise.

[x] Integer#>=(other)
Return true if the integer is numerically greater than or equal to other. Return false otherwise.

[x] Integer#>(other)
Return true if the integer is numerically greater than other. Return false otherwise.

[x] Float#<(other)
See Integer#<.

[x] Float#<=(other)
See Integer#<=.

[x] Float#>=(other)
See Integer#>=.

[x] Float#>(other)
See Integer#>.

[x] List#<(other : List)
Return true if the List is a proper subset of other. That is, if every element of the List appears in other, but other also contains at least one more value. Return false otherwise.

[x] List#<=(other : List)
Same as List#<(other), but also return true if there are no extra elements in other.

[x] Map#<(other : Map)
Return true if the Map is a proper subset of other. That is, if every key of the Map appears in other, but other also contains at least one more key. Return false otherwise.
The values of the Map are not important. Only the keys determine the subset.

[x] Map#<=(other: Map)
Same as Map#<(other), but also return true if there are no extra elements in other.

[x] IO.print(string : String)
Print the given string to the output of the Interpreter, with no formatting or conversions done. The argument should be expected to already be a String.

[x] List#size
Return the number of elements in the List as an Integer.

[x] Map#size
Return the number of elements in the Map as an Integer.

Standard library

[X] String#size
Return the number of characters in the String as an Integer.

[x] String#empty?
Return true if the String contains 0 characters. Return false otherwise.

[x] List#empty?
Return true if the List contains 0 elements. Return false otherwise.

[x] Map#empty?
Return true if the Map contains 0 elements. Return false otherwise.

[x] Enumerable#all?(&block)
Pass each element of the enumerable to block, returning true if all elements in the enumerable causes block to return a truthy value.

[x] Enumerable#any?(&block)
Pass each element of the enumerable to block, returning true if any element in the enumerable causes block to return a truthy value.

[x] Enumerable#find(default=nil, &block)
Iterate the enumerable, passing each element to block. The return value should be the first element for which the block is truthy. If no elements cause a truthy return value, return the default value instead (which itself should default to nil).

[x] Enumerable#min
Iterate the enumerable, finding the element with the lowest value as determined by <.

[x] Enumerable#max
Iterate the enumerable, finding the element with the highest value as determined by >.

[x] Enumerable#select(&block)
Returns an array containing all the elements of the enumerable that cause block to return a truthy value.

[x] Enumerable#sort
Sort the elements of the enumerable with the ordering determined by the <= method for each element. To start with, the sort can be something simple like insertion sort, eventually replaced by a proper hybrid quick sort.

[x] Enumerable#size
Return the number of elements in the enumerable, determined by incrementing a counter for each element yielded by #each.

[x] Enumerable#to_list
Return a List containing all the elements of the enumerable.

[x] Enumerable#reduce(&block(acc, elem))
For every element in the enumerable, call block with the result of the previous call and the current element as arguments. The first element is used as the initial value of the accumulator; it does not get a separate call to the block.

[x] Enumerable#reduce(initial, &block(acc, elem))
Same as Enumerable#reduce(&block(acc, elem)), but also specifying an initial value to use for the accumulator. In this version, the first element will get its own call to the block.

[x] Int#times(&block)
Call block as many times as the value of this integer. For example, 3.times(...) would call block 3 times.

Implementation
Obviously, there are a lot of things to add. I don't expect that all of these would be added in a single PR. Tackling them one at a time is fine by me.
Adding functions to the native library can be done in the src/myst/interpreter/native_lib folder. All of the types listed above should already exist there. Use NativeLib.method to write the implementation for a function, then call NativeLib.def_instance_method or NativeLib.def_method to add it to the appropriate module. There are plenty of examples in the code already that should help you out.
The standard library exists in the stdlib folder at the top level. Look at the existing entries (specifically, Enumerable) to see how new functions can be added.
Also, please try to add a descriptive comment to each method describing the arguments that it accepts, the values that might be returned and a description of what the method does. The Enumerable module) has some good examples of these comments.
If you'd like to pick up one or more of these functions, please comment below saying which function you would like to implement so that others know they are taken.
As always, if you have any other questions, feel free to ask them here or let me know directly so I can help out :) Good luck!

To expand a bit on how native library methods can be added:
The interpreter provides a few helper macros under the NativeLib module, namely method, def_method, and def_instance_method. These end up getting used by files in the native_lib folder to create and register methods on types in the Kernel.
Adding a new method to the native library is a two-step process:

use NativeLib.method to define the method implementation. For example, the String#size implementation looks like this:
https://github.com/myst-lang/myst/blob/aea25887f8bde444351f5590c69a11f256d61a4d/src/myst/interpreter/native_lib/string.cr#L56-L58
The first argument to this function is the name of the implementation. This is often made up of the name of the type the method is being defined on, followed by the name of the method being implemented. The second argument is the type to use for the self object (available as the variable this) while inside the method.

Any further arguments are parameters for the method, and require a type restriction (normally just Value). The other methods in native_lib/string.cr show this usage.

In the init_* method for the type (e.g., init_string in native_lib/string.cr), use NativeLib.def_instance_method to register the implementation with the type it belongs to. For example, the String#size method shown above is registered with a line like this:

NativeLib.def_instance_method(string_type, :size,   :string_size)

Here, string_type is the Type that represents String in Myst (see it's instantiation at the beginning of init_string). :size is the name of the method as it should appear in Myst, and string_size is the name of the implementation method.
def_instance_method is used to define instance methods for Types. It will register the method in the instance_scope of the given TType. def_method is used to define static methods, or methods for Modules (e.g., IO.puts). The arguments for the two use the same structure.
Hopefully that helps clear up how the Native Library is built up. Feel free to ask any more questions in the #help channel in the discord, or on here.

I've removed the Dir.[pattern : String] method from this issue because it doesn't really fit any of the other things here. Without more methods on Dir and File, it's pretty useless anyway.
For reference, here is the original text:

Dir.[pattern : String]
Return a List containing the file names that match the given pattern, which can be given as a grep pattern like in Bash. Crystal provides this same functionality, so this function can act as a direct passthrough to that method.


Create a Pali section

There is another awesome list for Pali, but it is not quite complete.
To have a complete list of awesome resources for the Pali language, I propose adding a new section specifically for Pali on this list.
It makes sense because Pali is most closely associated to the Theravada tradition.

Completed in d556f2e2d4c7295b290395cab4bf3d1835dfbec0. More content will be added over the next few weeks.


[2308] Redirect /organisations/ to /

Context
Because they both render the same content
Changes proposed in this pull request
[2308] Redirect /organisations/ to /
Guidance to review
:ship:

 Users accessing a provider url without access are seeing wrong message


Expand Development Workflow

Update Development Workflow Overview
Similar to geocat-comp
Operating System
Linux
Summary
Currently, open command does not function in Linux as it does in Mac
If the `_build` folder is generated, but `open` command cannot open the html, you can navigate to the `index.html` document and right-click to `Open With` your desired browser.

Expected behavior
The command open _build/html/index.html does not open the html website
Steps to reproduce
conda env create --file environment.yml
conda activate geocat-applications
make clean html
open _build/html/index.html

Relevant log output
`open` does not open rendered HTML page

Couldn't get a file descriptor referring to the console

Environment
name: geocat-applications
channels:
  - conda-forge
dependencies:
  - cartopy
  - geocat-datafiles
  - matplotlib
  - netcdf4
  - numpy
  - pre-commit
  - python
  - xarray
  - ipykernel
  - ipython
  - jupyter_client
  - matplotlib-base
  - sphinx-book-theme
  - myst-nb
  - sphinx-design
  - nbsphinx
  - pandas

We should expand this issue to cover more than just the open command part. I'm not sure the bug template is necessarily the best fit for the intended scope, if that makes sense. Yes, there is technically a "bug", but it's a one or two line change in the broader scope of this ticket, which should talk about, for example, how we want to expand the section, what subsections we should have, how this section might need to be different than the version we have in geocat-comp, ect.

Agreed, something similar to how geocat-comp is set up:

Set up on local machine (fork, clone, etc..)
Set up branch and pre-commit hooks
Choosing between template options
Opening PR and feedback


Release 2021-03-01 - Comments on Denoising Autoencoder

On Page 650 (pdf 680) - Section 20.3.2
"One useful way to control the capacity of an (overcomplete) autoencoder is to add noise to its input"
Even though the original work of Vincent et al., presents results on overcomplete autoencoders, the denoising criterion could be used also in undercomplete autoencoders. Is this the reason of the parentheses?
Moreover, I suggest to mention the emphasized denoising autoencoder (always in the Vincent et al.). This type of DAE weights differently the errors on input's components that were corrupted and the errors on input's components that were left untouched. This approach (and its variants) is useful in the imputation data tasks.

I removed the confusing (overcomplete) term. I'm not familiar with the emphasized DAE, so will omit this for now.


nit: use already created reference instead of property access

I was reading the source code when this hit me...
Why create a const object reference and not use it afterward... 🤣
Jokes aside, just wanted to point out that you can either remove that declaration or use it instead of accessing this again 🙏

@gurgunday Let's do the same for item.next as well then. gotta save them nanoseconds :D

Haha, sure!
After all a machine only does what we command


Does not run on Ruby 2.7

Helpy does not appear to support Ruby 2.7. On an install that ran fine on Ruby 2.5, I'm getting various errors such as:
App 15729 output: Error: The application encountered the following error: undefined method `new_ostruct_member' for #<Config::Options> (NoMethodError)
App 15729 output:     /usr/local/rvm/gems/ruby-2.7.5/gems/config-1.1.1/lib/config/options.rb:149:in `block in __convert'
App 15729 output:     /usr/local/rvm/gems/ruby-2.7.5/gems/config-1.1.1/lib/config/options.rb:147:in `each'
App 15729 output:     /usr/local/rvm/gems/ruby-2.7.5/gems/config-1.1.1/lib/config/options.rb:147:in `__convert'
App 15729 output:     /usr/local/rvm/gems/ruby-2.7.5/gems/config-1.1.1/lib/config/options.rb:65:in `reload!'


Port changes over from web branch

Added model as field for TagModel
Tweak to train.py to save/update model
Remove unused imports
Code cleanup and language conventions

closed for changes


fix:(noUnknownProperty): prevent warning on CSS custom property

Summary
Closes #3092.
Prevent warnings from being thrown for Custom properties (--*).

Test Plan

There must be so many custom properties in bootstrap.css given the performance numbers. 😆


Multiple tabs - arrow down with showing all tabs with more text

@davidzbiral Not sure if it's needed. Now tabs are limited to 20 and even on this number the label is very small.
Also need to think about the behavior when there is full count of tabs opened (20 now) and new entity is created. Open in tab and close the first one?

@Ptrhnk : I think this is already not needed. Let's:

avoid arrow down;
limit the number of Detail tabs to 10 - nobody will need to have more, it would rather be that they forgot (or were lazy) to close those not needed any more;
when the 11th tab is opened, the 1st one will close automatically. But how to determine the order? It would be great if it was the one with oldest activity (I mean: the oldest click on the tab), rather than the first in the chronological order of how they were initially opened. Is this reasonably simple to implement?

@davidzbiral It seems that the most efficient way to choosing the tab to close - by oldest activity - is sorting by order how tabs were opened (clicked) so It doesn't matter if something was changed inside.
If we try to use audit it would be possible to open tab without changes and then this one would be the first closed because of older audit.

Last conclusion: automatically close from the left with notification about closed tab

I endorse this solution, thanks!


Remove unneeded property

We can just export this from the object itself.

tag @dotnet/project-system


MAINT: Update imports

Companion PR to https://github.com/pyvista/pyvista/pull/4486

I'm going to self-merge this so that CIs are green. But we can adjust further if needed once https://github.com/pyvista/pyvista/pull/4486 is merged


Custom url for "chapters"

I would suggest to use custom urls for each chapter like https://ukrainehelp.hu/en/housing or https://ukrainehelp.hu/housing/en . It would make much easier to share and link information in facebook posts and comments. For example when somebody asks for information related to medical aid we could easly comment https://ukrainehelp.hu/medical .

Short solution could be a simple .htaccess file to redirect any /en/housing to /en#housing (these ids / anchors exist right now). We need some javascript/jquery hack to make the anchored chapter be opened by default.

Other solution to make html files for each chapters as well. e.g.: /en/housing.html . The main index.html can have all the collapsible chapters as well. The same source .md files make the two content equal and that is perfect.

( 3) No more collapsible main content but links to the static chapter files. )
What do you think?

Progress: The opening/closing of chapters already triggers a change in the URL, so the individual chapters are already shareable. They also open on page load and the browser scrolls to the correct section.
What I still want to achieve is to make the urls nicer by removing the # using .htaccess, but I still need to figure out the proper opening/scrolling for that. Bit of time.

Great progress!
Something similar I think:
RewriteEngine On
RewriteRule ^(hu|uk|en)/(emergency|work|money|animals|food)$  /$1/$2/#$3 [L]


Are there any implementations for font atlases and SDF support for Pixi?

At the moment Pixi does not optimize fonts rendering. Are there any plugins or solutions to use characters atlases? And even more challenging what about SDF?
May be any plans or ideas on that?

Hi there! Pixi does indeed support bitmap fonts which is pretty much what you are after. SDF is on the todo list too but not currently implemented. Hope that helps! Thanks :+1:

Does it use font atlas or uploads labels to GPU one by one?
2015-04-18 15:23 GMT+02:00 Mat Groves<EMAIL_ADDRESS>
Closed #1679 https://github.com/GoodBoyDigital/pixi.js/issues/1679.
—
Reply to this email directly or view it on GitHub
https://github.com/GoodBoyDigital/pixi.js/issues/1679#event-284468860.

it uploads a sprite sheet of all characters. Then constructs letters using that sheet, so just the one texture upload at startup.

Great, thank you


docker kill doesn't work with signals that have 'Core' dispositions

I want to send a signal which would terminate the main process inside a container and produce a core dump. However, it seems that docker kill command doesn't work when it's --signal option is set to signal which have 'Core' disposition - the main process is not terminated. It does work correctly with signals that have 'Term' disposition though.

Start a container with the following command: docker run --name container_name image_name app_name
Send a 'Core' signal (e.g. SIGSEGV) to the main process inside the container with the following command: docker kill --signal="SIGSEGV" container_name
Check the status of the container with: docker ps

Describe the results you received:
docker ps gives the STATUS of the container as Up
Describe the results you expected:
docker ps should give the STATUS of the container as Exited and a core dump should be generated (provided /proc/sys/kernel/core_pattern is configured properly)
Output of docker version:
Client:
Version:         1.12.5
API version:     1.24
Package version: docker-common-1.12.5-14.el7.centos.x86_64
Go version:      go1.7.4
Git commit:      047e51b/1.12.5
Built:           Mon Jan 23 15:35:13 2017
OS/Arch:         linux/amd64
Server:
Version:         1.12.5
API version:     1.24
Package version: docker-common-1.12.5-14.el7.centos.x86_64
Go version:      go1.7.4
Git commit:      047e51b/1.12.5
Built:           Mon Jan 23 15:35:13 2017
OS/Arch:         linux/amd64
Output of docker info:
Containers: 0
Running: 0
Paused: 0
Stopped: 0
Images: 1
Server Version: 1.12.5
Storage Driver: devicemapper
Pool Name: docker-8:3-138212-pool
Pool Blocksize: 65.54 kB
Base Device Size: 10.74 GB
Backing Filesystem: xfs
Data file: /dev/loop0
Metadata file: /dev/loop1
Data Space Used: 449.4 MB
Data Space Total: 107.4 GB
Data Space Available: 3.248 GB
Metadata Space Used: 1.081 MB
Metadata Space Total: 2.147 GB
Metadata Space Available: 2.146 GB
Thin Pool Minimum Free Space: 10.74 GB
Udev Sync Supported: true
Deferred Removal Enabled: false
Deferred Deletion Enabled: false
Deferred Deleted Device Count: 0
Data loop file: /var/lib/docker/devicemapper/devicemapper/data
WARNING: Usage of loopback devices is strongly discouraged for production use. Use --storage-opt dm.thinpooldev to specify a custom block storage device.
Metadata loop file: /var/lib/docker/devicemapper/devicemapper/metadata
Library Version: 1.02.135-RHEL7 (2016-09-28)
Logging Driver: journald
Cgroup Driver: systemd
Plugins:
Volume: local
Network: host overlay bridge null
Swarm: inactive
Runtimes: docker-runc runc
Default Runtime: docker-runc
Security Options: seccomp
Kernel Version: 3.10.0-327.22.2.el7.x86_64
Operating System: CentOS Linux 7 (Core)
OSType: linux
Architecture: x86_64
Number of Docker Hooks: 2
CPUs: 1
Total Memory: 3.703 GiB
Name: host
ID: OKKN:WOBF:UK7L:UCI5:SLTC:6ZRB:6QCG:AEVA:WXUB:FIKL:5NAP:QRCT
Docker Root Dir: /var/lib/docker
Debug Mode (client): false
Debug Mode (server): false
Registry: https://index.docker.io/v1/
WARNING: bridge-nf-call-iptables is disabled
WARNING: bridge-nf-call-ip6tables is disabled
Insecure Registries:
<IP_ADDRESS>/8
Registries: docker.io (secure)

Processes usually act different if they're running as PID1, including ignoring certain signals. Could that be the issue here?
/cc @cpuguy83

Yep, try this one-liner:
docker run -d --name test busybox sh -c "n=0; trap 'echo got SIGSEGV but not gonna do anything' SIGSEGV; while true; do echo sleeping; sleep 3; done"

Then you can check docker logs after sending the sigsegv

@cpuguy83 Thanks. After trying this small test I got 'trap: SIGSEGV: bad trap' in the logs.

@bushkov So it looks like this is working as expected?
As @thaJeztah mentioned, a lot of programs will drop default signal handlers when run as pid 1.
One thing you could do if you don't want to add your own signal handlers, use the --init with docker run.

Yes, it seems the issue stems from the fact that the main program is run as PID 1.


fix: Entity props show evaluated value

Description

Header bg color white
Make the entityProperties generation code extendable so the module-related code can be moved to the EE repo.

PR fixes following issue(s)
Fixes #30670
Fixes #30648
Type of change

Bug fix (non-breaking change which fixes an issue)

Testing

How Has This Been Tested?

Please describe the tests that you ran to verify your changes. Also list any relevant details for your test configuration.
Delete anything that is not relevant

[x] Manual
[ ] JUnit
[ ] Jest
[ ] Cypress

Test Plan

Add Testsmith test cases links that relate to this PR

Issues raised during DP testing

Link issues raised during DP testing for better visiblity and tracking (copy link from comments dropped on this PR)

Checklist:
Dev activity

[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my own code
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[x] My changes generate no new warnings
[ ] I have added tests that prove my fix is effective or that my feature works
[x] New and existing unit tests pass locally with my changes
[ ] PR is being merged under a feature flag

QA activity:

[ ] Speedbreak features have been covered
[ ] Test plan covers all impacted features and areas of interest
[ ] Test plan has been peer reviewed by project stakeholders and other QA members
[ ] Manually tested functionality on DP
[ ] We had an implementation alignment call with stakeholders post QA Round 2
[ ] Cypress test cases have been added and approved by SDET/manual QA
[ ] Added Test Plan Approved label after Cypress tests were reviewed
[ ] Added Test Plan Approved label after JUnit tests were reviewed

Summary by CodeRabbit

New Features

Introduced functions to fetch and organize properties for various entity types in the application.

Refactor

Streamlined entity property extraction by removing redundant functions.
Optimized the EntityProperties component for better event handling and rendering based on entity type.

Style

Adjusted layout and styling of the EntityProperties component for improved user interface.

/ok-to-test<EMAIL_ADDRESS>

/build-deploy-preview skip-tests=true

/build-deploy-preview skip-tests=true

/build-deploy-preview skip-tests=true

/ok-to-test<EMAIL_ADDRESS>

@rishabhrathod01 Tested the PR for following cases:

Header bg color white => Looks good to me ✅
Made the Binding text color similar to how it looks in code => Looks good to me ✅
Overall the binding popup looks good => ✅

A couple of queries/feedback:

For insert modal / show modal, the popup looks like this, should we just not show the popup when no bindings?

First binding is shown as opened by default, I hope this is expected behaviour

For refresh button, I see isVisible is the only thing shown in binding but it also has isDisabled state but it's not shown in bindings, Is there any reason for it? How do we determine what goes in show bindings and what does not? For select widget, I saw that isDisabled is also shown

These values are in a input box, makes me think I can edit it [Just a feedback]

Same question, why don't we show border radius or box shadow in bindings when they are clearly written in js mode?

For select widget, filterText accordion is not opening, is this expected?
https://github.com/appsmithorg/appsmith/assets/30018882/a1d1ec51-5ae0-487f-bc6f-31206e917619

First binding is shown as opened by default, I hope this is expected behaviour

@sneha122 This was added to make it easier for users to discover the value section.

These values are in a input box, makes me think I can edit it [Just a feedback]
Same question, why don't we show border radius or box shadow in bindings when they are clearly written in js mode?

@sneha122 Yes, I agree and as we are redesigning the bindings section with IA-redesign I believe we could address these points.
cc: @momcilo-appsmith @rohan-arthur

For select widget, filterText accordion is not opening, is this expected?
For insert modal / show modal, the popup looks like this, should we just not show the popup when no bindings?

@sneha122 Yes, I did notice both of these issues while testing. For now, I avoided extending the scope of this PR. I will create separate tasks for these bugs.

For refresh button, I see isVisible is the only thing shown in binding but it also has isDisabled state but it's not shown in bindings, Is there any reason for it? How do we determine what goes in show bindings and what does not? For select widget, I saw that isDisabled is also shown

@sneha122 we rely on what is exposed to the user in the autocomplete config of the widget. If a property is not shown then probably it is also not supported by autocomplete and we could fix the core issue.


Feature Parity: Implement FSS from 4.x

Community Note

Please vote on this issue by adding a 👍 reaction to the original issue to help the community and maintainers prioritize this request
Please do not leave "+1" or "me too" comments, they generate extra noise for issue followers and do not help prioritize the request
If you are interested in working on this issue or have submitted a pull request, please leave a comment

Description
FSS has not been fully implemented since 4.x but some remnants remain, such as:
https://github.com/oracle-terraform-modules/terraform-oci-oke/blob/v5.1.4/modules/network/nsg-fss.tf
There is another Oracle module for Terraform to work with FSS, but then you run into a lot of networking issues wen trying to make the two work together on the same VCN as the OKE module doesn't play well with others due to underlying cidr math.
While FSS is not directly needed by OKE to function, there is no migration path for customers using the 4.x module who rely on FSS. If the OKE module will not support FSS going forward, please provide a suggestion for integration otherwise.
New or Affected Resource(s)
oci_file_storage_file_system
Potential Terraform Configuration
Implementation would need to vary from https://github.com/oracle-terraform-modules/terraform-oci-oke/blob/v4.5.9/main.tf#L322 as each module is no longer called independently but instead uses the create_* helper variables. Refer to how those are implemented.
# Copy-paste any Terraform configurations for how the requested feature may be used.

References
https://github.com/oracle-terraform-modules/terraform-oci-tdf-file-storage
https://github.com/oracle-terraform-modules/terraform-oci-oke/blob/v4.5.9/main.tf#L322
https://github.com/oracle-terraform-modules/terraform-oci-oke/blob/v5.1.4/modules/network/nsg-fss.tf

This capability is provided but not a default anymore. To get the FSS subnet, the nsgs variable needs to be updated (as shown in the examples) to include the fss subnet (e.g. fss = {}). To create FSS mounts, you will still need to create them manually via resource calls or the module referenced above. This was done due to the CSI support added to 5.x.
Closing ticket as this is a valid go-forward approach.


Milestone -2 - add nameable class, base decorator, capitalize and trim decorators

In this PR, I have implemented the following

create Nameable class and define correct_name method.
extend Person, BaseDecorator classes from Nameable class.
create CapitalizeDecorator , TrimmerDecorator classes and extend from BaseDecorator class.
define correct_name in the above classes with logic defined to meet the project requirement.

Hello @ram1117👋,
This is an additional review as a result of the second opinion request.
I confirm that your code works as expected and does not show any errors. Great job @ram1117 👏

Testing to create a person object:

Running main.rb file:

As this was the only reason that your project was not approved it will be approved as soon as possible.
Please ask for another review in your Dashboard and the next Code Reviewer will mark it as approve right away :)
Keep rocking 🚀!


Update issue templates

the issue is i am new to this github

how to move ahead

erdf

new tech


Setup a local proxy for the ES 2.4 instance on rummager boxes

This is a temporary implementation while we are migrating.  Once the migration
is completed we should update port 9200 to point to the new cluster, we can
then revert the change in rummager so that it connects oin port 9200 again.
Once this is complete we can delete the proxy on 19200
https://trello.com/c/Ew3bOzy5/229-setup-rummager-to-work-with-es-24-on-integration

In our migration work to AWS we're already not using the proxy because we're using ELBs 🎉


Complex JSON passing without a Controller

I am building a project where I get some information based on a webpage filled out by the user, that information is used to create a URL that runs a third party API. The result expected is a JSON with a list of items (it could be over 200 items) with 10-11 fields. I was able to make it work with a Controller, but not with Razor Pages.

Sorry, created this message in the wrong place.


nyc_tlc.to_spark_dataframe() causes exception in .Net libraries

I am running this example on my local machine using WSL and spark 3.0.1.  When I try to execute the command nyc_tlc.to_spark_dataframe(), i get the following error:
Py4JJavaError: An error occurred while calling o24.parquet.
: java.lang.NoSuchMethodError: 'java.util.Properties org.eclipse.jetty.util.log.Log.getProperties()'
at org.apache.hadoop.fs.azure.AzureNativeFileSystemStore.createPermissionJsonSerializer(AzureNativeFileSystemStore.java:410)
i have downloaded the following 3 additional jar files in /opt/spark/jars directory:
azure-storage-8.6.6.jar
jetty-util-11.0.1.jar
hadoop-azure-3.3.0.jar
I do understand that the article is for synapse analytics, but is it possible to  mention what version of the jars are required to do this example on a local setup?
Regards
Ali
Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: 075d1d7b-372e-0926-d74e-80cd173bd7a7
Version Independent ID: 86d2610b-7050-5150-7894-e8b766a81911
Content: Tutorial: Build a machine learning app with Apache Spark MLlib - Azure Synapse Analytics
Content Source: articles/synapse-analytics/spark/apache-spark-machine-learning-mllib-notebook.md
Service: synapse-analytics
Sub-service: machine-learning
GitHub Login: @euangms
Microsoft Alias: euang

@ali-khawaja When you say "I am running this example on my local machine using WSL and spark 3.0.1", have you tested this code on Azure Synapse Analytics - Apache Spark pool?
The code snippets mentioned in this document works with Azure Synapse Analytics - Apache Spark pool.

it probably does.  i wanted to see if i can run this on my local spark.  if this is out of scope, i can close it.


Add optional cog for statbot integration

When we make a repository for other futaba cogs, we can transfer this issue there.

Statbot cog is now in the codebase, this issue is resolved. New feature requests for the cog should have separate issues.


Fix option name so that --background (and -b) work

@marcparadise: is this correct? The --background option didn't work for me until I applied this diff.

Looks good. Appears to have been a basic typo (although -b should have still worked)

On the contrary, prior to this, -b made options[:foreground] true, so that when https://github.com/chef/chef-server/blob/jc/option-name/dev/dvm/lib/dvm/application.rb#L134 looked up  options[:background] it still found nothing.

Ah, makes sense.

I didn't see this earlier , but yes, good find. We realized we were always specifying foreground and so I changed the default behavior but forgot to update the option.


Use standard semantic skills to define chat prompts

Motivation and Context
Currently the prompts are made available inside of appsettings.json which differs from the standard convention of shipping prompts in skprompt.txt files. This PR updates the copilot chat to call Semantic Skills that have been configured with config.json and skprompt.txt files instead of inline prompts in code.
In response to: #1726
Contribution Checklist

[x] The code builds clean without any errors or warnings
[x] The PR follows SK Contribution Guidelines (https://github.com/microsoft/semantic-kernel/blob/main/CONTRIBUTING.md)
[x] The code follows the .NET coding conventions (https://learn.microsoft.com/dotnet/csharp/fundamentals/coding-style/coding-conventions) verified with dotnet format
[x] All unit tests pass, and I have added new tests where possible
[x] I didn't break anyone :smile:

nit: we may not need a new folder for the semantic skills? I think we could utilize the existing Skills folder:
.
└── Skills/
    ├── Native/
    │   ├── ChatSkill.cs
    │   ├── DocumentMemorySkill.cs
    │   └── ...
    └── Semantic/
        ├── Chat/
        ├── ExtractAudience/
        └── ...

I see each config file has its own max_token definition. How are token limits accounted for in the new semantic functions? Does the context value override the config value?

Now the max_token value in each config file is used to set the token limit on all the functions that are loaded in with the Semantic Chat Skills.
The context sets a tokenLimit variable for use by things like the ExtractChatHistory semantic function which don't use a skprompt.txt file because they never calls the LLM.

This change has been reverted due to issues with Copilot Chat deployments. Needs further investigation before resubmitting.

.
└── Skills/
├── Native/
│   ├── ChatSkill.cs
│   ├── DocumentMemorySkill.cs
│   └── ...
└── Semantic/
├── Chat/
├── ExtractAudience/
└── ...

To add to the existing directory structure you've provided, you can create additional directories and files as needed for your project. Here's an example of how you can expand the structure:
.
└── Skills/
├── Native/
│   ├── ChatSkill.cs
│   ├── DocumentMemorySkill.cs
│   └── ...
└── Semantic/
├── Chat/
│   ├── ChatSemanticSkill.cs
│   ├── ChatPromptHandler.cs
│   └── ...
├── ExtractAudience/
│   ├── ExtractAudienceSemanticSkill.cs
│   ├── ExtractAudiencePromptHandler.cs
│   └── ...
└── ...
In this expanded structure:
The "Semantic" directory contains subdirectories for different semantic skills (e.g., "Chat" and "ExtractAudience").
Each semantic skill directory contains the corresponding C# files for that skill, such as SemanticSkill.cs and PromptHandler.cs.
You can continue adding more semantic skills and their associated files as needed within the "Semantic" directory.
Remember to adjust the directory and file names based on the specific skills and functionality you're implementing in your project. This structure is just a starting point, and you can customize it to fit your project's requirements.


Notes from meeting with Bayer

determination: algorithm run datetime vs. source data datetime (determination_datetime)
Seasons (up to 3 in South, vary heavily)

Planted, Harvested (plant once, harvest multiple times)

Ownership changed (management ext?)
Valid from/until
=> Admin vs Activity boundaries
Note from Dan: Timestamps extension could possibly be an Activities extension which has a notion of the field’s seasons’ start and end.  Maybe starting with an activity such as tilling and ending with harvest.

We decided yesterday to have a dedicated meeting for the timestamps extension with at least @ivorbosloper, @cholmes and me.
Other interested parties are welcome to join, please get in touch here if you are interested.


Add a field in struct such that previously created data account also can be deserialised

⚠ Don't use issues for support questions ("How does x feature work in Anchor? How to do x in Anchor?"). Use the discord instead https://discord.gg/GVRw4kHB (but don't ping maintainers). ⚠
We have a streaming contract deployed and we might need to add a field in the struct which is saved in an account. Now, when i make changes in the contract previously created data account doesn't get deserialized. I have added a report as well where i have performed the test. Is there any possible solution.
Impact and Recommendation for Changing Struct Field.pdf

I posted in discord and stack exchange as recommended


[tf.data] Add SkipNext interface to iterator

This is a PR from JIZHI, the AI platform in Tencent.
This pr adds a SkipNext interface to IteratorBase and use this method in SkipDatasetOp and ShardDatasetOp.
If this interface is added, we can gradually implement it for all dataset ops so that some unnecesscary calculation can be avoided.
Thank you for your time on reviewing this pr.

@aaudiber
Could you have a look at  this pr? Thank you!

Thanks for putting this together @zhuzilin! Sorry for the delay in review. Some questions:

Is there a specific use case motivating this change?
Which datasets are you planning to implement skipping for?
What do you think of changing SkipNext(ctx, end_of_input) to Skip(ctx, num_to_skip, end_of_input)? In some cases skipping multiple could be cheaper than skipping one at a time.

@aaudiber
Thank you for your nice questions. Here are some of my thoughts.

Is there a specific use case motivating this change?

The specific use case would be using shard when there are uneven amount of data in the files of the dataset.
For now, we encourage to shard by file names, so that each shard won't process unnecessary data. But if there are not enough files to allocate one for each worker or the size of the file varies a lot, then the user has to shard the dataset after reading them. In such situation, he or she will meet performance loss, especially when the cpu resource is limited.

Which datasets are you planning to implement skipping for?

I think probablity most of them, except TFRecordDataset, for it is using a SequentialRecordReader (I wonder if we can only move the offset without actually reading from the file...). Therefore, it's likely to be lots of additions. I hope we can settle down the design in this pr in order to avoid large scale of refactor.

What do you think of changing SkipNext(ctx, end_of_input) to Skip(ctx, num_to_skip, end_of_input)? In some cases skipping multiple could be cheaper than skipping one at a time.

This is one of the design issues I hope to discuss 😄 . I agree that Skip(num_to_skip) would potentially have better performance than SkipNext, but I'm not sure how to deal with input_impl_.reset() when end_of_sequence is met. Are we going to put the reset in the Skip or shall we return the number of successful skipping back to the iterator?

@zhuzilin Thanks for explaining the motivation, I think it makes sense.

This is one of the design issues I hope to discuss 😄 . I agree that Skip(num_to_skip) would potentially have better performance than SkipNext, but I'm not sure how to deal with input_impl_.reset() when end_of_sequence is met. Are we going to put the reset in the Skip or shall we return the number of successful skipping back to the iterator?

Good point, I think we will need to report the number of skipped elements as well so that the caller can decide what to do when end_of_sequence is reached before the requested number of elements are skipped.

@aaudiber
I've changed the SkipNext to Skip. Could you have another look? Thank you!

@aaudiber
Could you take a look at this pr? Thank you!

@aaudiber
The build failures are due to me forget the & in RecordElement... I've fixed the mistake.Sorry for my carelessness...

Gently ping @aaudiber
I've fixed the build failure in the last commit. Could you have another look? 😄

@aaudiber
The new build failure is related to commit 94ca0bd702, where the signatures of RecordStart and RecordStopare changed. I've updated the Skip method to make it coherent with the GetNext in the current master branch. Could you have another look? Thank you!

@aaudiber
Could you take another look at this pr? Thank you 😄 .

@aaudiber
It seems that the current implementation of Skip may have some conflict with DistributedIteratorMultiWorker and AutoShard. I'm not familiar with the mechanism of auto-sharding and I wonder if you have any idea how we could fix those conflicts? Thank you :).

@zhuzilin AutoShard will update a dataset to insert a shard transformation, either at the start after a list of filenames, or at the very end if a list of filenames cannot be found. It seems like the new skip-based shard implementation must change the behavior somehow. The test failure messages suggest that the test is hitting end_of_sequence too early now.
I recommend using tensorflow/python/distribute:input_ops_test to debug, since the test is simpler than DistributedIteratorMultiWorker, but still fails.

@aaudiber
The reason of early end_of_sequence is that the shard op did not initialize end_of_sequence to false. However, ops like concatenate, which use end_of_sequence as a switch, will pass the *end_of_sequence = true to shard. The new commit add *end_of_sequence = false in the shard op, which should fix the end_of_sequence failure.

@aaudiber
This pr is finally merged 🎉 Thank you for your help and your great patience on it 😄.

@aaudiber
This pr is finally merged 🎉 Thank you for your help and your great patience on it 😄.

You did all the work, I just reported the test failures :-) Thanks for your patience with my slow responses!

Seems auto-merge is not happening but the changes are merged into master now, so we can close this. Thank you for the PR.


Migrate Datadog custom metrics for Kafka

What changed, and why it matters
Source: https://help.aiven.io/en/articles/3359501-configuring-datadog-metrics-with-aiven-for-apache-kafka
Fixes #1024

I think is a good idea to add a link between datadog integration and this file. Also, from this file to datadog integration one. So the reader can find both resources. wdyt?

Yes please, always more links. We will need to do a lot of manual adding of links, so add everything you notice!

I've rebased with the main branch.

sgtm 💯 , but let's clarify your questions about the warnings before merging :)

Closed VS code and reopened and the warning went away?


4.4: Pronouns are not present on the VS Screen and 4/3 layouts

As the title says, unlike with the FGC scoreboards, the pronouns do not appear on the default VS Screen and 4/3 layouts

Update: The issue is also present on the commentator and "simple" overlays in 4.65. Previous instances of the issue are present as well

Fixed in 5.0


Fix3

added utils.py. This code includes:

visualize_seq(): A quick and simple function to visualize the target and probe sequences
export_opool_spreadsheet(): A quick and simple function to export oligo probe sequences in IDT's oPool submission format

To support this function, openpyxl was added in the dependency

The usage of these functions are demonstrated in OligoDesign.ipynb

Also added recipe for Singularity image

Reopen to merge the pull request.


NRE when calculating fall damage

From rogueticket-9273:
FallDamage resolution doesn't happen due to an NRE. This leaves an AttackSequence on the stack, that breaks stratops.
03:17:22.5709708 CombatLog.MechFallSequence [LOG] Create MechFallSequence for: Flea
03:17:22.5711036 CombatLog.MechFallSequence [LOG] MechFallSequence OnAdded for: Flea
03:17:22.5711083 CombatLog.MechFallSequence [LOG] MechFallSequence: Begin
03:17:22.5711197 CombatLog.MechFallSequence [LOG] Flea Begin Knockdown!
03:17:22.7415475 CombatLog.ActorActivation [LOG] Actor Flea (b7eb8d4e-6b3d-4670-a48a-baf2757e024a.0) - ends activation due to Done
03:17:22.8152416 CombatLog.Heat [LOG] Mech Flea final heat = 0 out of 150
03:17:23.1247632 CombatLog.Attacking [DEBUG] [OnActivationEnd] Flea b7eb8d4e-6b3d-4670-a48a-baf2757e024a.0 HasActivatedThisRound False
03:17:23.1247724 CombatLog.ActorActivation [LOG] Actor Flea (b7eb8d4e-6b3d-4670-a48a-baf2757e024a.0) - ends activation
03:17:23.1303050 CombatLog.RoundSequence [DEBUG] [ShouldCompleteActivationSequence] 'this is AITeam' true
03:17:23.1303119 CombatLog.RoundSequence [DEBUG] [CompleteActivation] ActivationSequence.ForceComplete = true
03:17:23.1337716 CombatLog.MechFallSequence [LOG] MechFallSequence: Falling
03:17:27.0036054 CombatLog.MechFallSequence [LOG] MechFallSequence: Finished
03:17:27.0262956 <CBTBE> [WARNING] 07:17:27.026 <CBTBE> DOING IMAGINARY ATTACK FOR: Flea_Pilot_68F0C967 USING: Melee_CBTBE_FAKE_MELEE
03:17:27.0264613 CombatLog.AttackSequence [WARNING] AttackSequence: Weapon Melee is firing at target Flea, but technically cannot fire at that target.
03:17:27.0273781 CombatLog.AttackDirector [LOG] [CreateAttackSequence] ID 41595858
03:17:27.0275246 <CBTBE> [WARNING] 07:17:27.027 <CBTBE> Object reference not set to an instance of an object
03:17:27.0275264 <CBTBE> [WARNING] 07:17:27.027 <CBTBE>Stacktrace available in mod logs
03:17:27.0275280 CombatLog.MechFallSequence [LOG] MechFallSequence OnComplete for: Flea
03:17:27.0654111 CombatLog.MechFallSequence [LOG] Flea ApplyKnockdown!

07:16:45.058 FAILED TO APPLY FALL DAMAGE
07:16:45.058 Object reference not set to an instance of an object
07:16:45.058   at CBTBehaviorsEnhanced.Helper.AttackHelper.CreateImaginaryAttack (BattleTech.Mech attacker, BattleTech.Weapon attackWeapon, BattleTech.ICombatant target, System.Int32 weaponHitInfoStackItemUID, System.Single[] damageClusters, BattleTech.DamageType damageType, BattleTech.MeleeAttackType meleeAttackType) [0x00469] in <537876be2dcf4266a956951e703a3dc6>:0
at CBTBehaviorsEnhanced.Piloting.MechFallSequence_OnComplete.Prefix (System.Boolean& __runOriginal, BattleTech.MechFallSequence __instance) [0x001d3] in <537876be2dcf4266a956951e703a3dc6>:0
07:16:45.058FAILED TO APPLY FALL DAMAGE