GEODE-7344: DSFID implements BasicSerializable

GEODE-7344
As part of eliminating TcpServer dependency on geode-core, this ticket removes TcpServer dependencies on DataSerializable.
The way we did that was to change the 6 message types used to talk to TcpServer so instead of deriving from DataSerializable—they now derive from a new BasicSerializable interface.
That interface is made a base for DataSerializableFixedID. Then we modified geode-serialization so it handles any BasicSerializable i.e. both the base type and the derived (DataSerializableFixedID) type. This entailed a few changes up in geode-core: primarily in InternalDataSerializer.
The six message types that are now BasicSerializable are:

VersionRequest/Response
InfoRequest/Response
ShutdownRequest/Response

With these changes, the on-the-wire format of the new BasicSerializable is the same as the old DataSerializable. This lets a new Geode and an old Geode communicate.
With this PR, the TcpServer package no longer depends on anything in geode-core and TcpServer and friends (in org.apache.geode.distributed.internal.tcpserver)  are ready to move to their very own new module (in a future PR).
For all changes:

[x] Is there a JIRA ticket associated with this PR? Is it referenced in the commit message?

[x] Has your PR been rebased against the latest commit within the target branch (typically develop)?

[ ] Is your initial contribution a single, squashed commit?

[x] Does gradlew build run cleanly?

[x] Have you written or updated unit tests to verify your changes?

[ ] If adding new dependencies to the code, are these dependencies licensed in a way that is compatible for inclusion under ASF 2.0?

Note:
Please ensure that once the PR is submitted, check Concourse for build issues and
submit an update to your PR as soon as possible. If you need help, please send an
email to<EMAIL_ADDRESS>

Hydra tests passed:

PDX tests: http://hydradb-sandbox.gemfire.pivotal.io:9090/hdb/run/13291 (1 auto-diagnosed failure)
Cluster config tests: http://hydradb-sandbox.gemfire.pivotal.io:9090/hdb/run/13294 (all passed)

DistributedTestOpenJDK11 latest CI run failed but I believe it is due to a CI infrastructure problem. Previous CI run of that test was successful and I made no big changes after that. Merging.


IPAM docs

[ ] Tests
[ ] Documentation
[ ] Release note

None required

IPPool blockSize
named pools in CNI config (and annotations)
namespace annotations

Deploy preview for calico ready!
Built with commit a39b8307c17d316f7bf526ca255450340d41d69b
https://deploy-preview-2225--calico.netlify.com

The two changed pages are
https://deploy-preview-2225--calico.netlify.com/master/reference/calicoctl/resources/ippool
https://deploy-preview-2225--calico.netlify.com/master/reference/cni-plugin/configuration


"Already defined" error when tiny_gltf.h included multiple times.

When we include tiny_gltf.h in different .cpp files, since all the implementations of the functions are in the header, compiler will complain for "already defined" error.
In my example, I included tiny_gltf.h in both GLTFInfo.cpp and GLTFImport.cpp.
Here is the message from VS 2017:
1>GLTFInfo.obj : error LNK2005: "bool __cdecl tinygltf::IsDataURI(class std::basic_string<char,struct std::char_traits<char>,class std::allocator<char> > const &)" (?IsDataURI@tinygltf@@YA_NAEBV?$basic_string@DU?$char_traits@D@std@@V?$allocator@D@2@@std@@@Z) already defined in GLTFImport.obj

As described in README
https://github.com/syoyo/tinygltf#loading-gltf-20-model
You should define TINYGLTF_IMPLEMENTATION in single compilation unit(.cpp)

Thanks. Problem solved.

Is it better if we use static or inline to avoid the issue without define TINYGLTF_IMPLEMENTATION?

avoid the issue without define TINYGLTF_IMPLEMENTATION?

Defining ***_IMPLEMENTATION is a commonly known idiom to use header-only library
https://github.com/nothings/stb/blob/master/stb_image.h#L4


Is it safe to detach/attach multiple times?

Upon consecutive calls to 'DetourDetach' and 'DetourAttach' there is any chance to cause any kind of crash or error, into the hooked process? or is it considerable safe?
I'm asking in a scenario where you detach with the intention to pause the hooks, and then reenable them when needed.
or should i create any kind of if else in each API hooked?
if (!hooksEnabled)
    return 'the original function'

Consecutive attach and detach works for me

https://github.com/microsoft/Detours/pull/144 is a workload for this issue.


not working in unity 2018.2

Installed through unity package but road architect menu didnot appear

A couple of API changes in Unity 2018 cause compile errors in a couple of scripts.  There appear to be a couple of other issues as well that I am trying to track down.

@hemanth2410 Please try https://github.com/embeddedt/RoadArchitect.


[iOS] ListView + Lots of Images + overflow: hidden = App Lockup

I've created a github repo that reproduces this problem. The has happened afaik since 0.20 (maybe even 0.19, I forget!)
https://github.com/sgonyea/react-native-listview-images-freezing
Setting overflow: hidden on the ListView's rows is a guaranteed way to trigger this problem, though I'm trying to recall if I've been able to trigger it without that.  More commonly, the app will simply exhaust available memory and crash without it :)
It's a partial lockup, not a full freeze / crash.  You can interact with some elements, but images won't load, you can't make network requests, and a navigator transition will trigger an actual lockup.
Here are some simple steps to reproduce the issue:

Run this app on your iPhone (not the simulator, you need a lot of images)

Scroll down several hundred photos.  If you do this really fast and get really far ahead of the image loading, you'll trigger the bug.

Hit your phone's Status Bar (scroll to the top) and then scroll down really fast.

Images will no longer load.  If you check XCode's memory profiler, you'll see memory use flatline.

Try shaking the phone + reloading.  The app will crash.  If this were embedded inside a real app, images would no longer load anywhere else.  And triggering a Navigator transition would also cause the app to lock up.

I've kicked this just about every possible way.  It seems to me that the only solution (for now) is to simply go without overflow: 'hidden' and risk an app crash from exhausting available memory.
There's a few hackey solutions that I could go with, but they still have some risk of a lockup.  IE, setting the render ahead distance to 0 will largely prevent the crash... and then prevent hitting the status bar to scroll to the top.

Hi there! This issue is being closed because it has been inactive for a while.
But don't worry, it will live on with ProductPains! Check out its new home: https://productpains.com/post/react-native/ios-listview-lots-of-images-overflow-hidden-app-freeze
ProductPains helps the community prioritize the most important issues thanks to its voting feature.
It is easy to use - just login with GitHub. GitHub issues have voting too, nevertheless
Product Pains has been very useful in highlighting the top bugs and feature requests:
https://productpains.com/product/react-native?tab=top
Also, if this issue is a bug, please consider sending a pull request with a fix.
We're a small team and rely on the community for bug fixes of issues that don't affect fb apps.


Recommendation for citations

Currently in text files, there is only a stub:
# Height Distribution
# ===================
# TopoBank version: 0.1
# PyCo version: 0.31.0
# IF YOU USE THIS DATA IN A PUBLICATION, PLEASE CITE XXX.
[...]

We could use decorators here.

We now have a mechanism for this in SurfaceTopography and need to use it in ContactMechanics

ContactMechanics now also reports citations

This is not just downloaded data but could also be shown on the web page.

I've implemented the output of the DOIs into the analysis results (txt, xlsx).
I've added a menu item here:

which opens a modal window:

Clicking on a DOI opens a new tab showing the corresponding article.

Okay fantastic. It would be good if this could be resolved to an actual citation. I think a static dictionary doi->text or html would suffice for that.

Is the citation needed in the download files also or only in the UI?
In the downloads it would take quite some space and it might look a bit cluttered.

I think it would be good in both places. I know which paper this is from the full citation, but not from the DOI. The DOI should still be there of course...

Okay, fine. What do you think of using a service like https://citation.crosscite.org/ in the background and saving the results in the database? These entries could also be edited manually in the admin interface.
I'm a bit worried that maintaining the static dictionary will become tedious, e.g. consider this case:
A plugin uses a new analysis function which results in a new DOI, but the static dictionary hasn't got this DOI yet.
So a new version of a plugin would also enforce a new version of the main application.

If this works automatically through crosscite, fantastic! I just thought a static dictionary would be easier.


How to add Vast Tag in Fluid player in WordPress?

I don't know more about coding so I don't know why vast tag is not working.
This is the code which I have used every this is fine but vast is not playing?
[fluid-player
video="https://file-examples.com/wp-content/uploads/2017/04/file_example_MP4_480_1_5MG.mp4"
vast_file="https://www.movcpm.com/watch.xml?key=af5222c12af4829eb69dfc25d313b9b8"
poster-image="https://upload.wikimedia.org/wikipedia/en/6/6e/Dream_Girl.png"
layout="default"
responsive="true"
]

Current wordpress plugin is not maintained. We plan to take ownership of it at some point, and have it properly configurable, but this is not in the scope of Fluid Player itself.


BUG: panic: runtime error: invalid memory address or nil pointer dereference

What versions are you running?

MacOS
github.com/chromedp/chromedp latest from master
chrome Version 75.0.3770.100 (Official Build) (64-bit)
go version go1.12.1 darwin/amd64

What did you do? Include clear steps.
chromedp.Run(ctx,
	chromedp.ActionFunc(func(ctx context.Context) error {
		_, _, _, err := page.Navigate("https://www.google.com/").Do(ctx)
		if err != nil {
			return err
		}
		return nil
	}),
	chromedp.WaitVisible("input[aria-label=Search]", chromedp.ByQuery),
)

What did you expect to see?
I want program to wait until page load all necessary elements and wait input element appears.
What did you see instead?
panic: runtime error: invalid memory address or nil pointer dereference
[signal SIGSEGV: segmentation violation code=0x1 addr=0xb8 pc=0x1522177]

goroutine 66 [running]:
sync.(*RWMutex).RLock(...)
	/usr/local/Cellar/go/1.12.1/libexec/src/sync/rwmutex.go:48
git.c8data.com/mtd/bot/vendor/github.com/chromedp/chromedp.(*Selector).run(0xc00008a280, 0x1736540, 0xc0001a07e0, 0xc00030a000, 0xc0003441e0)
	/Users/zeka/go/src/git.c8data.com/mtd/bot/vendor/github.com/chromedp/chromedp/sel.go:83 +0x137
created by git.c8data.com/mtd/bot/vendor/github.com/chromedp/chromedp.(*Selector).Do
	/Users/zeka/go/src/git.c8data.com/mtd/bot/vendor/github.com/chromedp/chromedp/sel.go:55 +0x11f

Solution
Add check for cur==nil in Selector.run
if cur == nil{
	continue
}

latest from master

sorry, but this doesn't help me. master changes every other day.
In any case, I can reproduce. The larger issue here is that you're not using chromedp.Navigate, which waits for the page to be fully loaded. You wouldn't encounter that crash then.
Still, using the lower-level APIs should still be possible without crashes. I'll look into a fix.

version: 015708072eadb724b6ef5d648f6eab2bfae88f1c
@mvdan It's not an issue that I'm using chromedp.Navigate. I just want to do some stuff even if page is not fully loaded.


Study apps has no statement category

There is no statement category defined for the mobile_app_content_viewed recipe.
In all other recipes this is defined as: statement.context.extensions.http://xapi.jisc.ac.uk/statementCat for example the VLE login recipe

Good catch, the statementCat should be optional in the spec but not required in the TSV. It should be set to app in the jisc template. Will update this

For Dec 3rd call:
This is a small documentation error:
statementCat is used by Jisc to categorise statements, but is not provided by institutions in TSV as jisc set it in the json template. The description is missing from the study apps markdown but in other recipes it is currently:
Recommended For querying lookup. Set to category of statement
Description should be added to study app md

From call:
yes, put this in to the markdown file


three-mesh-bvh: Incorrect intersections reported?

It's possible three-mesh-bvh is reporting incorrect intersections since these triangles are reported as needing to be clipped together. Reproduceable with a rotated low complexity cylindrical brush.:
const ogTris = [
	new THREE.Triangle(
		new THREE.Vector3( -0.29389262199401855, 0.40450850129127514, 0.4999999999999999 ),
		new THREE.Vector3( -0.4755282700061798, -0.1545085012912749, 0.5 ),
		new THREE.Vector3( 0, 1.1102230246251565e-16, 0.5 ),
	)
];

const tris = [
	new THREE.Triangle(
		new THREE.Vector3( -1.2246468525851679e-16, -0.5, 0.5 ),
		new THREE.Vector3( -0.4755282700061798, -0.5, 0.15450850129127502 ),
		new THREE.Vector3( 0, -0.5, 0 ),
	),
	new THREE.Triangle(
		new THREE.Vector3( -0.4755282700061798, -0.5, 0.15450850129127502 ),
		new THREE.Vector3( -0.29389262199401855, -0.5, -0.404508501291275 ),
		new THREE.Vector3( 0, -0.5, 0 ),
	),
	new THREE.Triangle(
		new THREE.Vector3( 0.4755282700061798, -0.5, 0.15450850129127502 ),
		new THREE.Vector3( 0, -0.5, 0.5 ),
		new THREE.Vector3( 0, -0.5, 0 ),
	)
];

It's also just not reporting intersections in some cases:


🐛 Upgrade from 2.0 rc7 on Buster stays at 2.0 rc7

Bug
Upgrade from 2.0 rc7 on Buster stays at 2.0 rc7
What I did
Re-running the installation script on Buster to get from 2.0 rc7 to 2.0 rc8.
What happened
After all finished successfully the version stayed at 2.0 rc7
I expected this to happen
I expected to have 2.0 rc8 installed correctly.
Software
Base image and version
PRETTY_NAME="Raspbian GNU/Linux 10 (buster)"
NAME="Raspbian GNU/Linux"
VERSION_ID="10"
VERSION="10 (buster)"
VERSION_CODENAME=buster
ID=raspbian
ID_LIKE=debian
Branch / Release
Refresh index: 100% (309/309), done.
On branch master
Your branch is up to date with 'origin/master'.
Installscript
cd; rm buster-install-*; wget https://raw.githubusercontent.com/MiczFlor/RPi-Jukebox-RFID/master/scripts/installscripts/buster-install-default.sh; chmod +x buster-install-default.sh; ./buster-install-default.sh
Refresh index: 100% (309/309), done.
On branch master
Your branch is up to date with 'origin/master'.
Hardware
RaspberryPi version
Raspberry Pi 3 Model B Plus Rev 1.3

The version information is maintained here: https://github.com/MiczFlor/RPi-Jukebox-RFID/blob/master/settings/version
This numbering is updated rarely, so currently the information is not very useful. The installer uses the latest commit from master branch, so you are up to date, but I agree the version should reflect this or be dropped altogether. Maybe we should use the commit number, see #828.
The rc8 you were expecting is currently set only in develop not master branch (where it's still rc7), which makes it even more confusing.

I would coordinate this enhancement in #828, so we could close this ticket here, if you agree.

If this is the same problem we should merge it. (I follow the rule: Only one issue per bug report.)

Tracked in #828


ceph_key: add fetch_initial_keys capability

This is needed for Nautilus since the ceph-create-keys script goes away.
(https://github.com/ceph/ceph/pull/21305)
Now the module if called with 'state: fetch_initial_keys' will lookup
keys generated by the monitor and write them down on the filesystem to
the right location (/etc/ceph and /var/lib/ceph/boostrap*).
This is not applicable to container since keys are generated by the
container only.
Signed-off-by: Sébastien Han<EMAIL_ADDRESS>

TODO:

[ ] finish up the tests

jenkins test dev-centos7_cluster

jenkins test dev-docker_cluster

jenkins test dev-centos7_cluster

jenkins test dev-centos7_cluster

jenkins test dev-centos7_cluster

jenkins test pipeline

jenkins test dev-centos7_cluster

Needs https://github.com/ceph/ceph-container/pull/1237/


(PDB-1934) remove two-database events query acceptance test

This is duplicated in the http facts test.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://jenkins.puppetlabs.com/job/platform_puppetdb_intn-sys_pr/1828/

@pljenkinsro retest this please

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://jenkins.puppetlabs.com/job/platform_puppetdb_intn-sys_pr/1832/

I agree that the facts tests here duplicate this test, but one thing the acceptance test covers that the facts one does not is configuration of the two databases. The http/facts test side-steps the configuration side of it.

reopened on stable


Review final transfer of job folder from remote to local

I saw some cases where the local folder was empty, at the end of a task execution, even if the local tar.gz file includes all files.
Something is likely wrong in the extract-archive (see https://github.com/fractal-analytics-platform/fractal-server/blob/1537-new-slurm-ssh-executor/fractal_server/app/runner/extract_archive.py#L25-L28), e.g. with the internal hierarchy of the tar.gz archive (AKA do we have an internal subfolder in the archive or not? this changed, in some recent commits)

We should re-visit this issue after https://github.com/fractal-analytics-platform/fractal-server/pull/1641, since the overwriting behavior upon both extracting and compressing is now defined and tested.

As of now, I cannot reproduce this with version 2.3.7.
Closing.


Type duplication

We have a lot of duplicated types, like GL.Texture vs GPUTexture, GL.Program vs GPUProgram. We need to sort that out.

First of all, what is Core? Does it make any sense to use core to represent bare concepts, without render purpose? I guess so. Does a core texture have a sense? I don’t think so.
A texture is a render object only. Just like shaders. We don’t have shaders in Core.


Learning rate scheduler for PaSST

Summary
The learning rate scheduler for PaSST is used in image models, so we started to include this module as one of the learning rate schedulers of audyn.
close #115

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 84.11%. Comparing base (2d931a6) to head (915c56e).
Report is 3 commits behind head on main.

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@            Coverage Diff             @@
##             main     #125      +/-   ##
==========================================
+ Coverage   84.09%   84.11%   +0.02%
==========================================
  Files         123      123
  Lines       12622    12641      +19
==========================================
+ Hits        10614    10633      +19
  Misses       2008     2008

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


istioctl install  output installed version inconrrect

Is this the right place to submit this?

[X] This is not a security vulnerability or a crashing bug
[X] This is not a question about how to use Istio

Bug Description
when i specify install version is 1.21.4 , then we can seen the describe of install version status is istioctl version

Version
$ istioctl version
client version: 1.23.0
control plane version: 1.22.4
data plane version: none

$ kubectl version
Client Version: v1.31.0
Kustomize Version: v5.4.2
Server Version: v1.31.0

Additional Information
No response

This is not incorrect, how you are installing is. The istioctl binary embeds the Kubernetes manifests which are tightly coupled to a version. Overriding the tag to a different version is making an amalgamation of 2 versions of istio which is bound to not work

to be clear - use istioctl 1.22.4 to install 1.22.4

maybe the fix is to warn folks about this state? "tag != istioctl version"

istioctl should pull the charts from the Istio OCI repo like it does images, and then we can ditch manifest embeds, IMO.

istioctl should pull the charts from the Istio OCI repo like it does images, and then we can ditch manifest embeds, IMO.
Should be able to fetch and override charts exactly like images in all cases (and would make it easier to move between versions without switching binaries) - this would generally be more intuitive than explaining that istioctl embeds a copy of the charts (but only the charts) and the various bad states this can allow you to get into given that chart and image overrides are handled very differently.
(This may still result in mismatches between istioctl and what it expects the manifests to look like - but IMO those assumptions and mismatches are bugs/tight coupling problems we should fix or explicitly treat as a public API contract in istioctl/helm/etc)

I also believe that installing with istioctl should allow installation from a standard Helm Repo, rather than forcing a coupling between ctl and the installation version.
Recently, Istio has also been promoting standardized Helm installations, which is likely what everyone is hoping for.


Allow motion sense to be imprecise or vague

Closes https://github.com/foundryvtt/pf2e/issues/15060

Motion sense is a precise sense for creatures lacking vision


add conn hijack

What type of PR is this?

What this PR does / why we need it (English/Chinese):

Which issue(s) this PR fixes:

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.aaron seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.


SyntaxError: Non-ASCII character '\xe2' in file /home//anaconda3/envs/py2/lib/python2.7/site-packages/scipy-1.2.0-py2.7-linux-x86_64.egg/scipy/stats/_continuous_distns.py o n line 3346

I get this error while trying to run the score2perf model in the DataGeneration/Preprocessing step on GCP inside a VM.
I installed magenta-gpu as per the instructions given on the main README.

Figured out the issue.
pip install magenta-gpu installs scipy-1.2.1
which is incompatible and while running setup.py it also installs scipy-1.2.0
after uninstalling both versions and installing scipy-1.2.0 after python setup.py install started working fine.

Can you give some more details on how installing the magenta pip package ended up installing scipy-1.2.1? Our setup.py should restrict it to 1.2.0: https://github.com/tensorflow/magenta/blob/master/setup.py#L53
That said, I'm not sure why 1.2.1 would cause that error anyway.

So in my conda environment, I would typically install all magenta dependencies with pip install magenta-gpu. If I don't do this and directly go with setup.py it fails to install many of the other dependencies that are required. It is in this step that scipy 1.2.1 is installed and after that during the setup.py installation scipy 1.2.0 is also installed making both the versions useless. But after uninstalling both versions and reinstalling scipy 1.2.0 solves the problem.

Yeah, installing via pip rather than setup.py (even if it's pip -e .) definitely does a better job of handling dependencies. Glad you go things working.


upgrade go version of github action to go1.16.x

🎉 I'm going go to fork this project to prologic/eggos to play with now! 🤣 Thanks!


PC001: Thread.SetApartmentState(ApartmentState) isn't supported on Linux and macOS

I want my application to be supported in Linux and macOS any suggestions would be appreciated

Why do you call SetApartmentState? If you need to call it on Windows only, you could achieve that by changing your code to something like:
if (RuntimeInformation.IsOSPlatform(OSPlatform.Windows))
    Thread.SetApartmentState(…);


RFC: Edit distances for strings

Checklist

Please ensure the following tasks are completed before filing an issue.

[x] Read and understood the Code of Conduct.
[x] Searched for existing issues and pull requests.
[x] If this is a general question, searched the FAQ for an existing answer.
[x] If this is a feature request, the issue name begins with RFC:.

Description

Description of the issue (or feature request).

This RFC proposes the inclusion of edit distance functions to compare two strings, for example the Levenstein distance.
Related Issues

Does this issue (or feature request) have any related issues?

No.
Questions

Any questions for reviewers?

What would be a good place inside of stdlib for string distance functions?

For string distance functions, in the @stdlib/string namespace. I believe one of the initial reasons for not including Levenstein, Hamming, and others, was whether we needed to expose generic interfaces. For example, Hamming could be used to compute the "distance" between two arrays. I was not sure how general we need to be.

@Planeshifter We should probably turn this into a tracking issue for implementing various metrics (Hamming, Levenshtein, etc), and cross-link to the existing issues.

I looked at Levenshtein distances for strings this morning.  From the above discussion I am not sure if this is still wanted?
Anyway, some notes:
For fun, my implementation of Wikipedia's naive recursive algorithm, in Standard ML:
fun min_of_3(a, b, c) = let
val min_1 = Int.min(a, b)
in
Int.min(min_1, c)
end
fun lev(a, []) = length a
|   lev([], b) = length b
|   lev(x::xs, y::ys) =
(if x=y then)
if Char.compare(x, y) = EQUAL then
lev(xs, ys)
else
1 + min_of_3 ( lev(xs, y::ys), lev(x::xs, ys), lev(xs, ys) )
fun timeLevStr s1 s2 = let
val tmr = Timer.startRealTimer()
val d   = levstr s1 s2
val t   = Timer.checkRealTimer tmr
in
(d, t)
end
Very very very slow.  This takes about 5 minutes.

timeLevStr "alphabetically" "betamaxrecording";
val it = (14,TIME {usec=306635216}) : int * Time.time

Some links:

https://xlinux.nist.gov/dads/HTML/Levenshtein.html
-- links to this java implementation:
https://people.cs.pitt.edu/~kirk/cs1501/Pruhs/Spring2006/assignments/editdistance/Levenshtein Distance.htm
-- I have this Java running and the same lev("alphabetically", "betamaxrecording") takes no noticeable time.
Perl CPAN: https://metacpan.org/pod/Text::Levenshtein

Do we want to do this?

@rgizz Thanks for following up here. We have an initial implementation of Levenshtein as @stdlib/string/base/distances/levenschtein; however, that implementation is currently sub-optimal and needs to be refactored. I have notes, somewhere, which I need to dig up which outline what changes need to be made; however, the changes are non-trivial.
One additional hurdle is that the refactoring needs to address handling of grapheme clusters, as linked to in the following tracking issue: https://github.com/stdlib-js/stdlib/issues/1062.


[Request for External] codeunit 9150 "My Records Filter Sub."

Hi,
We have the similar functionality as in CU 9150 "My Records Filter Sub." but with different tables.
Can you make functions in this codeunit external so that anyone can reuse this code to extend Filter Text with own/different tables:

GetMyFilterText
AddToFilter

Also, in GetMyFilterText function that exit when checking MyTableNo should be changed to allow different tables also.
It was already asked in #378 to make those functions globaly available (external).
Thanks,
Zoran

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. We will update this issue with information about availability.

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. We will update this issue with information about availability.


Fix antidrag in bank interface

Set the drag delay on either bank or bank inventory widget loaded.

fixed in f8d8aa762c9c9e000a634aae0ae2387a8928c597

fixed in f8d8aa7

Is this in the current runelite version? I still have issues with the inventory anti-drag not working right away when I open the bank. https://gyazo.com/412c219a0a59a6eabcbd7b7a867f926b

no, it is in master


If multiple RowFilters are passed to ReadRow(s) in a variadic slice of bigtable.ReadOption, only one is applied.

If multiple filters are passed as ReadOptions to ReadRows, only one is applied since there is only one Filter on the request (type ReadRowsRequest) For instance:
tbl.ReadRow(ctx, row-key, bigtable.RowFilter(bigtable.LatestNFilter(1)), bigtable.RowFilter(bigtable.FamilyFilter("family")))

Instead, the correct approach is to use ChainFilter or InterleaveFilter This is not obvious behavior and I could not find any documentation in the api docs. This should be documented in the api and possibly be detected and return an error as well.

It is indeed incorrect usage, though I'd be nervous about special-casing particular options as not being accepted in duplicate. A documentation fix is probably the right move here.

I think the document should mention this explicitly until this bug is fixed.


Update the transitive node-fetch dependency

GitHub complained to us about https://github.com/advisories/GHSA-w7rc-rwvf-8q5r, which is not relevant to us right now, and probably never will be. However, we should of course still update once we can. The problem is: We can't (easily) do it right now.
node-fetch is a transitive dependency of ours via some of the Relay dependencies, and the patched version does not fit the specific version ranges used in those chain. So we can just wait for any and/or all of the packages in the chain to update their dependencies in a way so we can get the update.
For the record: The chains in question are
<EMAIL_ADDRESS>><EMAIL_ADDRESS>><EMAIL_ADDRESS>><EMAIL_ADDRESS><EMAIL_ADDRESS>><EMAIL_ADDRESS><EMAIL_ADDRESS>><EMAIL_ADDRESS>><EMAIL_ADDRESS><EMAIL_ADDRESS>><EMAIL_ADDRESS>><EMAIL_ADDRESS>><EMAIL_ADDRESS><EMAIL_ADDRESS>><EMAIL_ADDRESS>><EMAIL_ADDRESS>><EMAIL_ADDRESS>><EMAIL_ADDRESS>

The easiest thing would be for isomorphic-fetch to update, which they did, but unfortunately in a major update (3.0.0), which we can't get with the constraints above.
The next best thing would be for fbjs to do it. Let's see how they handle it.

Apparently at least our use case was fixed "above" the level of fbjs. PR incoming shortly. \o/


[Windows] Mojibake in Japanese adapter name

Fyne includes Noto Sans font without Japanese characters. We need to set FYNE_FONT environment variable for Japanese Windows users (https://github.com/fyne-io/fyne/issues/400).

Released as v0.1.5
https://github.com/mikan/arpg/releases/tag/v0.1.5


feat:  make finding the position of js variable / functions more robust

This PR
related to https://github.com/ashish10alex/vscode-dataform-tools/pull/51

[x] Improves robustness of finding variable / function when using go to definition
[x] Single function to get variable or function from js file / block

Tested against following input .Updated with scenarios which would have previously failed
config {
  type: 'table'
}

/*
create a parameters.js file in the includes directory and add
getDateMonthsBefore function

function getDateMonthsBefore(date, months) {
  date.setMonth(date.getMonth() + months);
  let date_string = date.toISOString().split('T')[0];
  date_string = `'${date_string}'`;
  return date_string

}
*/

js {
  let some_mod = require("includes/parameters.js")
  const team = "barcelona"
  const six_months_ago_from_now = some_mod.getDateMonthsBefore(new Date(), -6)
  const four_months_ago_from_now = some_mod.getDateMonthsBefore(new Date(), -6)
  const some_month = "january"
  const some_number = 11;
  const barcelona = "a team"
  let some_coniditon = true;
  let another_coniditon = true;
  var some_floating_points = 9.4;
}

select
  *
, ${six_months_ago_from_now}  some_time_1
, ${four_months_ago_from_now}  some_time_2
, ${some_number}  some_number
, ${some_coniditon}  some_condition
, ${another_coniditon}  another_coniditon
, ${some_floating_points}  some_floating_points
, "${some_month}"  some_month
, ${some_mod.getDateMonthsBefore(new Date(), -4)} date_wt_func
, ${parameters.OXRC_DATA_STARTING_POINT}  as date_from_file
, "${barcelona}" as some_team
FROM ${ref("0100_CALENDAR")}

Nice! The search seems to work as intended now. I'll merge this PR


Update current-version.md

Updated current version to 2.0.60 which was released yesterday.

OPS Build status updates of commit 8f9efc9:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs-ref-conceptual/includes/current-version.md
:white_check_mark:Succeeded
View (azure-cli-2017-03-09-profile) View (azure-cli-2018-03-01-hybrid) View (azure-cli-latest)

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.

Closing as this duplicates the same changes in this PR: https://github.com/MicrosoftDocs/azure-docs-cli/pull/1338


fix: language switch not working on upcoming and past events

This PR fixes the language switch on UpcomingEvents and PastEvents pages.
Fixes #85

LGTM!
Thanks for the PR and the issue!


Upstream resolver is not routed through the tunnel on macOS / iOS

Use case:
Customer has an internal DNS server they'd like to use for resolving private names. Doesn't want to set up DNS resources because then they'll have to manage DNS records in two places: Firezone, and their DNS server.
It seems that for some reason, queries forwarded by connlib to upstream_resolver don't get routed through the tunnel.

It seems like this issue may have been fixed with the DNS changes yesterday:
debug	dev.firezone.firezone	connlib	FirezoneNetworkExtensionmacOS	existing connection: NameServerConfig { socket_addr: [2a07:a8c0::d3:c3c3]:53, protocol: Udp, tls_dns_name: None, trust_negative_responses: true, bind_addr: None }	09:41:17.907588-0800
2a07:a8c0::d3:c3c3 is currently the upstream resolver set in the portal.

Seems to have been resolved in the latest release.


BigDecimal divideAndRemainder bad result

Given this code:
val x = BigDecimal.fromShort(Short.MIN_VALUE)
val result = x.divideAndRemainder(BigDecimal.ONE)

x is displayed in debugger with correct value -3.2768E+4. Result.first should be the same, and result.second should be zero. Instead, this is displayed in debugger:
result.first = -3.2769E+4
result.second = 1.0
So far this is the only case I've hit in early unit testing, so am unsure what other ways the underlying bug may manifest. It's possible this happens whenever signs of numerator and divisor differ.
Environment:
kotlin 1.4.0-rc
com.ionspin.kotlin:bignum:0.1.6-1.4.0-rc-SNAPSHOT
running junit5 unit test using Kotest 4.2.0 invoking multiplatform common functions using BigDecimal

More info, indicating wider bug. Given this code running in the same environment
    val limit = BigDecimal.fromInt(Int.MAX_VALUE)     val lowLimit = BigDecimal.fromInt(Int.MIN_VALUE)
limit gets correct value of 2.147483647E+9 or<PHONE_NUMBER>
lowLimit however gets incorrect value -1.8446744071562067968E+18
lowLimit should be -2.147483648E+9
have not tried to debug into code to see why yet

Thanks for reporting! I'll give it a look today.

The BigDecimal.fromInt(Int.MIN_VALUE) is weird. The debugger displays the element in magnitude of the BigInteger it is constructing with the correct value of -2147483648, and sign is correct, but the debugger value displayed for the BigInteger itself is -18446744071562067968. I'm not sure why, but sure enough when it gets to function numberOfDecimalDigits() in BigInteger, that returns 19 when it should be 10. I'm still looking around :-)

The debugger is dynamically calling toString, and it comes back wrong due to trouble in divrem again. In the Biginteger toString method when it does copy divrem baseArray the answer comes back with the huge result<PHONE_NUMBER>1562067968. I'll stop looking here unless I can do something useful to help you, let me know if I can. I'm a motivated new user, with time :-)
Easy to recreate:
val badness = BigDecimal.fromInt(Int.MIN_VALUE).toString()

Hey @skolson, I just got off work so I can take a look at this.
But first, your effort is greatly appreciated, I don't have a lot of time and such debugging insight can be a great help when solving issues.
I'll first look into the divrem problem that you saw, as that might be the most serious.
The number you saw<PHONE_NUMBER>1562067968) looked very familiar, at first I thought it was ULong.MAX_VALUE, but I was wrong, it turns out it's ULong.MAX_VALUE - Int.MAX_VALUE

Ok, this is a fun issue, the actual problem stems from BigInteger.fromInt(intValue), because the BigInteger class is actually backing implementation of the BigDecimal. So when I tried to get a BigInteger.fromInt(Int.MIN_VALUE) I also got that weird ULong.MAX_VALUE - Int.MAX_VALUE
So let's look at what BigInteger.fromInt(intValue) does:
override fun fromInt(int: Int) = BigInteger(int)

it calls a constructor for BigInt that takes just an integer, which is this:
constructor(int: Int) : this(arithmetic.fromInt(int), determinSignFromNumber(int))

The next step is going into actual arithmetic implementation arithmetic.fromInt(int) which is this:
override fun fromInt(int: Int): ULongArray = ulongArrayOf(int.absoluteValue.toULong())

This one needs a bit of explaining. The ULong array is there because all BigIntegers are represented as an array of 63 bit numbers and a separate sign variable. To make sure that the number is always positive we use absoluteValue.
So in our case where we try BigInteger.fromInt(Int.MIN_VALUE) we would expect a BigInt that has a long array of [2147483648] and a Sign.NEGATIVE, but we got a [18446744071562067968] a Sign.NEGATIVE. It looks like the absolutValue didn't work in our case, but that is a kotlin call!
And the solution comes from the actual absoluteValue documentation, which states:
/**
 * Returns the absolute value of this value.
 *
 * Special cases:
 *   - `Int.MIN_VALUE.absoluteValue` is `Int.MIN_VALUE` due to an overflow
 *
 * @see abs function

Which is exactly our situation, so instead of returning what we expected to be<PHONE_NUMBER>, we got -2147483648 (note the sign), then converted that to ULong (unsigned long), which became<PHONE_NUMBER>1562067968, and continued to break everything else, because that number has a 1 set at bit 64, which shouldn't be possible in the 63 arithmetic implementation that BigInteger currently uses.
That shouldn't happen to Short though, so I still need to look into that divrem with Short.MIN_VALUE and 1.

The second part of the problem, wrong divrem, was a bit more convoluted, as it was in the rounding function, code which I have trouble remembering the most. The problem there was that the function tried to apply rounding even if the digit that was being discarded is 0, so in this case a negative number and default rounding, which is FLOOR caused the Short.MIN_VALUE (which internally when dividing in this case becomes -32768.0 and then needs to be rounded back) was decreased by one to -32769, instead of just returning -32768

Awesome, thanks for finding this so quickly. Is there already a 0.2.0 snapshot built with 1.4.0-rc that I could try? If not, lemme know and I can look into building the current master tomorrow myself and see how it acts.

Hey @skolson, sorry for the late reply, just got off work again, each merge to master is automatically built and published to sonatype snapshot repository, it's not 0.2.0 yet, but you can find the current build here:
repositories {
    maven {
        url = uri("https://oss.sonatype.org/content/repositories/snapshots")
    }
}
implementation("com.ionspin.kotlin:bignum:0.1.6-1.4.0-rc-SNAPSHOT")

I just tested with this and this fix looks good. Unfortunately it hits a comparison error with two negatives :-) Since this is closed, I'll open a new issue.


Feature/daef 179 show terms of use screen on first application start

This PR introduces "Terms of use" screen which is shown to the user on first application start and user must accept terms in order to use the application.
As a part of this PR "Terms of use" screen was implemented along with the UI logic and acceptance test for this feature.

TODO:

[ ] Add real "Terms of use" text

@DominikGuzei will check it out. Thanks for reporting!

@DominikGuzei fixed :)

@nikolaglumac if you have the rights you can merge this PR although the CI is not yet approving it (or trigger another build)


ZC Post Async Posting: Basic functionality in ck-core

https://github.com/UIUC-PPL/charm/pull/3386 -> https://github.com/UIUC-PPL/charm/pull/3387 -> https://github.com/UIUC-PPL/charm/pull/3388

I realized that charmxi changes in https://github.com/UIUC-PPL/charm/pull/3387 are required to get this PR built, so I'd have to merge this PR into that PR.
I'm closing this PR to use https://github.com/UIUC-PPL/charm/pull/3387 as the primary PR.


WebGL Version 1 and 2 issues

Hi,
I've got the following console errors in chrome:

Thanks your help in advance!

Do you have some code I could pull down to test?

(This may be of interest https://github.com/tensorflow/tfjs/issues/4284 - posting Tensorflow.js tf.env flags may be helpful too as it is likely all upstream)


T05a.03-Solution: Bug with writing onStop in the TextView

"onStop" is written twice instead of once in the TextView (as shown in the screenshot) if you follow these steps:

run the app
press the "Overview" or "Recents" device button
press the image to restart the Lifecycle app in the "Overview"
rotate the device (I started with horizontal orientation, just to be able to take a vertical orientation screenshot)

To solve this bug, you could add to onRestart() the following line:
mLifecycleCallbacks.clear();
The bug happens because, after pressing the "Overview" device button, the "onStop" String is added to mLifecycleCallbacks and also to the TextView (it's just that it's not visible). Then onRestart gets called, and now the activity's mLifecycleCallbacks incorrectly contains an "onStop" String, that will eventually be printed to the TextView again if onCreate gets called.

Unable to duplicate.


Add freedesktop-22.08 and kde-5.15-22.08

I'll trigger the Docker images action manually to generate these images, thanks!


Use legendItem.usePointStyle value

On legend drawing use legentItem.usePointStyle value instead of value in options.
This allows to overide value with custom generateLabels function.

Change looks fine. Is it possible to add a test for this?

Checked it out, currently this will be a breaking change, If you set usePointStyle to true in the legend in master it will change the style to a circle while with this change it will become a square. Also when you specify a pointStyle in the options for a dataset it doesn't get applied anymore: https://codepen.io/leelenaleee/pen/gOXLvgZ

ks were not suc

Thanks @LeeLenaleee.
It's possible to initialise legendItem.usePointStyle with value from options in generateLabels. This will fix the problem of pointstyle in dataset.
Let me commit a new version.

@maxime700 are you still able to update this PR?

Closing as stale. I'm open to coming back to this in the future


Add Getting Started section to README

README Update
Please add a getting started started section to the README so folks know how to start with this project

Added Getting Started Section in README file


ui : ready package.json for npm publish

Move current ui package to react-project-initiation-pipeline
Add sample frontend to notion-page-tree (#10)

18fbf9bb9389eda12443ad0c531aadb364330ad9


When is the 6 run available

Detailed Description
6 o'clock run is not available at 6, but when is it available?
it doesnt seem to be availabe at 6.30, but it is at 10.30. Would be nice to narrow this down a bit
Context
Good to know when its available so we can get the most recent data
Possible Implementation
ask the met office? or run it

So, I think the 6 o'clock run is the run that starts executing at 6am, and then probably takes an hour or two to execute, and then a bit of time to upload the forecast. I'm happy to ask the Met Office if you'd like! But, yeah, it might be better to empirically test when the NWPs become available.

Its not ready at 7.28

It is ready at 8.00

This give a bit more insight to when models were available
https://gridded-data-ui.cda.api.metoffice.gov.uk/model-run-status/atmospheric#mo-uk

no obviously patterns, but in the range of 1:20 to 2 hours to run


🐛 [Bug] Passwort reset security issue

🐛 Bugreport

When resetting password the server answers with "invalid email" -> this allows to query which emails are in our database.
The server must return a success at all times "email successfully sent".

Yes, you are right.
Added in #519


Multiline append only adds last to first

from Standard.Base import all

main =
    test = [1] + [2] + [3] + [4]
    test.to_text

results in [1,2,3,4]
however
from Standard.Base import all

main =
    test = [1] +
        [2] +
        [3] +
        [4]
    test.to_text

results in [1, 4]

This is due to the syntax of the block. Operator-block syntax is recognized when the lines of the child block start with an operator. In this case because the lines don't start with an operator but the block is introduced by an operator, it is a body block. So the [2] and [3] lines are evaluated (each is an operator-section in a void context), and then [4] is the expression at the end of the block; the block evaluates to its final expression.
We could improve this by emitting a warning when an expression with no side effects is evaluated without being used. That's probably a tall order for the backend to implement in general, but this is not the first time a bug has been reported due to this pitfall--to catch this specific case, we could have a warning if an operator-section occurs in statement context. This would be a syntax warning, so I could implement it in the parser.

Found the previous issue: #9944

a related issue is also tracked as #8024

not the first time a bug has been reported due to this pitfall

@kazcw, why don't we change the Body code blocks specification to say that a block where lines end with an operator like +:
from Standard.Base import all

main =
    test = [1] +
        [2] +
        [3] +
        [4]
    test.to_text

is equivalent to concatenation - e.g. [1] + [2] + [3] + [4]?

That's a good question. A rule change could allow recognizing this case:

Only = or -> introduces a body block (currently any operator can take a body block as an operand, but this is not used in practice).
Any other operator may introduce a postfix operator block.

There is a clear argument for this: Two language users have already expected it to work; maybe it should.
However my opinion, from the perspective of the syntax rules, is that supporting this would make writing easier at the expense of greater complexity in reading and learning the language.

The difference from prefix operator blocks looks like a style difference, but has a subtle semantic difference in associativity: Current (prefix) operator blocks are evaluated with top-to-bottom line associativity; the natural associativity of a postfix operator block would be bottom-to-top (or potentially as if lines were concatenated?). There may be cases where having a choice of associativity would be useful, but I think more often it would be a stumbling block.

The postfix syntax doesn't fit into the language as well as the current syntax. This is more subjective, but I think the postfix syntax could impede an intuitive sense of how blocks and operator sections work; in the prefix syntax, all the indented lines have the same shape, but in the postfix syntax the first operator belongs to the parent line, and subsequent operators belong to all the non-final lines--even though the first operator and later operators are logically "equals" in their roles in the AST.

But, more important than a language implementer's opinion is the perspective of the people writing Enso every day, so I have a question for @jdunkerley (CC @radeusgd):
Would being able to write expressions this way be useful enough to justify a bit more language complexity? Or would it be better to have only one way to do it, and a parser warning directing users toward the prefix syntax?

OK, let's stick with prefix syntax for now and implement runtime detection of these discarded values

#5430

However @radeusgd would prefer a static check. @kazcw, maybe the postfix syntax doesn't make much sense from the language perspective to be implemented, but would it make a sense to detect it? Report it as a warning? What does it mean when line ends with an operator anyway? I tried:
> a f = f 2 5
> a (*)
>>> 10

but that's an operator surrounded by parenthesis! However at the end I found valid code that ends the line with *:
> a f = f 5
> a <| 2 *
10

possibly the a function could have some side-effects, so it seems to me (again) there is no way to statically detect whether the line ending with an operator has a meaning or not.

However at the end I found valid code that ends the line with *:
> a f = f 5
> a <| 2 *
10

possibly the a function could have some side-effects, so it seems to me (again) there is no way to statically detect whether the line ending with an operator has a meaning or not.

But in this valid code, the result of 2 * is not discarded.
What if we decide to create a convention that operators like * should not have side-effects, and thus we could warn when their usage is discarded? Of course this would only be a convention, but then it would only be a warning and not an error.

so it seems to me (again) there is no way to statically detect whether the line ending with an operator has a meaning or not.

Yes, it's impossible to detect all cases statically. But a syntax warning for one case would go a long way: An operator section in statement context is always dead code. I suspect detecting anything that parses as such would have caught the issue here and in #9944 as soon as it occurred--it certainly would catch the issue in the simplified code included in the bug reports.

so it seems to me (again) there is no way to statically detect whether the line ending with an operator has a meaning or not.

Yes, it's impossible to detect all cases statically. But a syntax warning for one case would go a long way: An operator section in statement context is always dead code. I suspect detecting anything that parses as such would have caught the issue here and in #9944 as soon as it occurred--it certainly would catch the issue in the simplified code included in the bug reports.

That's exactly what I wanted to suggest.


Update Tamil and Sinhalese Languages.

Hi @nareshwar, I'm your friendly neighborhood Microsoft Pull Request Bot (You can call me MSBOT). Thanks for your contribution!

This seems like a small (but important) contribution, so no Contribution License Agreement is required at this point. Real humans will now evaluate your PR.

TTYL, MSBOT;


:book: Fix docs to support dark-light theme toggling

Preview at https://kproche.github.io/kubestellar/doc-darklight
Summary
This PR enables the rendered website to toggle between dark and light modes:

It adds and activates the toggle icon next to  the search bar
architecture images with transparent backgrounds have had a white background added
the small KubeStellar banner image has had its background set to transparent
the primary (link and menu bar) color is set to light blue in dark mode for better readability
the accent (link mouseover) color is set to cyan in dark mode

Related issue(s)
#2386
Fixes #

Preview at https://kproche.github.io/kubestellar/doc-darklight/direct/architecture/

@KPRoche , as you noted in https://github.com/kubestellar/kubestellar/issues/2386#issuecomment-2310823108 :

BTW, for the record, the main problem was fixed by just adding a style="background-color:white" attribute into the main SVG tag in each file generating a diagram; that keeps them from rendering with a transparent background, which is what causes the visibility issue in dark mode.
Adding the dark/light toggle and tweaking the blue accent colors for dark mode into the mkdocs.yml are just additional tweaks to improve the user experience.

Do we need/want to take on the responsibility for a toggle in the website itself? There are already multiple controls in the browser (as my journey to see the problem shows) and in the desktop. Is another one really a boon to the user?

taps the sign

taps the sign

Would you believe that this time I did do a rebase and squashed almost all my commits into one? Honest!

Do we need/want to take on the responsibility for a toggle in the website itself? There are already multiple controls in the browser (as my journey to see the problem shows) and in the desktop. Is another one really a boon to the user?

The toggle is a very simple accessibility fix, supported and implemented just by adding about 12 lines to the mkdocs.yml profile.
There are certainly days (if I'm working late for instance) when I appreciate a website letting me flip its color mode.
Also, given how recalcitrant the browser controls proved to be, and how little visual real estate the toggle consumes, I think it's a net plus. All the hard work is built into the mkdocs engine and materials theme we're using.

@KPRoche: good job on the squash. My problem is the other commit, the one that merges main into this PR's branch. That is why I tapped the sign.

@KPRoche: good job on the squash. My problem is the other commit, the one that merges main into this PR's branch. That is why I tapped the sign.

That was because in the approximately 20 minutes between when I pushed my branch back up to github, previewed the results and then could do the PR, you did an update to your go-lang fixes. Because they involved no overlapping files, letting the github.com sync merge those fixes from main back into my branch did not seem like it would break anything.

Of course, now there even more changes since I did the PR...

Oooh, the "files changed" preview is particularly handy to show the background changes in the images:
https://github.com/kubestellar/kubestellar/pull/2411/files


iOS9 bitcode support.

Do we know when this library will support iOS 9 bitcode?

There's a discussion on this question over here: https://github.com/card-io/card.io-iOS-SDK/issues/119


Include new hi-fidelity implementation and container images

Note images not yet verified still WIP but pushing this PR out there to unblock folks.

LGTM


patch the JRuby TCPSocket#read(0) (originated by @duelinmarkers)

See #197

Any chance you can add a test for this. With a test case I'd have no problem merging this in.

@halogenandtoast I've added "blind" specs (couldn't run those due to some local installation issues). Please double check those before merging. Sorry for that, hope that's not a big hassle.


Maven Artifact Not Found

Getting this after adding dependency as specified.
"The POM for com.amazonaws:aws-dynamodb-encryption-java:jar:0.0.3-SNAPSHOT is missing, no dependency information available"
Are you sure you uploaded to Maven central repo?
A search by group and artifiact at search.maven.org yields nothing.

I just checked and can download the jar from:
https://oss.sonatype.org/service/local/artifact/maven/redirect?r=snapshots&g=com.amazonaws&a=aws-dynamodb-encryption-java&v=0.0.3-SNAPSHOT&e=jar
It's a pre-release, so not sure if that could be related.

I've updated the README.md on the main page to include this info:
https://github.com/awslabs/aws-dynamodb-encryption-java
Thanks for asking!

Thank you! Yes, the repository you are using is different. Maven central repo: search.maven.org has no entry for this artifact.


Fix packaging releases

Sets correct env var for compiling the binary for NFPM

Summary by OpenAI
Release Notes
This pull request fixes packaging releases by setting the correct environment variable for compiling the binary for NFPM. It also updates the git push command with the --set-upstream flag. This is a bug fix that ensures the binary is compiled correctly and can be packaged for release.

Once merge, will cherrypick onto stable branch.
Manually merged brew formula update.
Will not created nfpm packages manually - will wait for rc.2.


Gossip - direct probe over TCP

In case of failed direct probe over UDP, attempt additional direct probe over TCP.
TCP probe should be started in parallel to issuing indirect probes over UDP.
From lifeguard document:

This helps with situations where TCP traffic is correctly routed, but UDP is not, which is a pathology sometimes encountered in network configuration.

Resolved by #17


Filter notebooks stored in ".ipynb_checkpoints" dirs from html genera…

Hi, @jph00
I encountered this problem because jupyter notebook creates these hidden directories .ipynb_checkpoints inside the
same dirs as where the notebooks are located.
Jupyter then saves versioned copies of the main notebooks inside these directories and these are then named xxxx-checkpoint.ipynb where xxxx.ipynb is original notebook.
This then cascades a file open error since _get_title will look for the generated html file name (generated by _nb2htmlfname) does not exist.
This problem doesn't show up on Github CI since .ipynb_checkpoints are git ignored and are never exported to github,
but it does show up if you call nbdev_build_docs in a local environment where jupyter notebooks have been saved with
checkpoints.
My fix is to just filter out those notebooks inside .ipynb_checkpoints directories.
Best regards,
Butch

You need to clean up the notebooks to pass the CI (nbdev_clean_nbs).
I would also filter 'checkpoints' not in f.parts instead of the .ipynb_checkpoints' not in f.parts

Hi @tcapelle,
I just checked --  runningnbdev_clean_nbs only cleans the notebooks themselves. They don't remove the notebooks in the .ipynb_checkpoints directory themselves.

The problem is not the notebooks themselves -- its the fact that running nbdev_build_docs looks for the notebooks in the directories using patterns '**/*.ipynb' which  will include the checkpointed versions of the notebooks in the directory .ipynb_checkpoints (i.e. they will be named -checkpoint.ipynb).
When you run nbdev_build_docs (even if you already ran nbdev_clean_nbs) this triggers an error.

Also, the filter condition checkpoints in f.parts  will not match the notebooks inside .ipynb_checkpoints directory -- this is because f.parts is the list of path segments (not the whole path string) for the file e.g. /nbs/.ipynb_checkpoints/03_core.ipynb will have the f.parts ['nbs','ipynb_checkpoints'] so checking if checkpoints is in f.parts will not match, but looking for '.ipynb_checkpoints' will match f.parts.
I've also tested the fix on the same library and it fixes the problem.
I also reran the nbdev_clean_nbs on my pr so that all checks now pass.

Thanks!


activity: Use setTimeout instead of setInterval.

Instead of using setInterval to call send_presence_to_server repeatedly, we now use setTimeout and then make next call to send_presence_to_server only after the first request succeeds.
This change helps us to avoid the condition where during resume from suspend, the browser catch ups all the un-called setInterval calls and thus hits the server repeatedly and gets a 429 response.
Fixes #23438.

Self-review checklist

[ ] Self-reviewed the changes for clarity and maintainability
(variable names, code reuse, readability, etc.).

Communicate decisions, questions, and potential concerns.

[ ] Explains differences from previous plans (e.g., issue description).
[ ] Highlights technical choices and bugs encountered.
[ ] Calls out remaining decisions and concerns.
[ ] Automated tests verify logic where appropriate.

Individual commits are ready for review (see commit discipline).

[ ] Each commit is a coherent idea.
[ ] Commit message(s) explain reasoning and motivation for changes.

Completed manual review and testing of the following:

[ ] Visual appearance of the changes.
[ ] Responsiveness and internationalization.
[ ] Strings and tooltips.
[ ] End-to-end functionality of buttons, interactions and flows.
[ ] Corner cases, error conditions, and easily imagined bugs.

@timabbott this PR is ready for review. Will do the change for other setInterval calls in separate PR.

But now we do not wait for the previous request to finish before making another request, not sure if it can lead to some weird behavior like setInterval did when resuming the client from suspend mode.

I think this should be fine; no code executes at all while the device is asleep, so it's not going to be possible to accumulate more requests.
That said, there is an alternative fix, which would be to keep using setInterval, but just have a request_in_flight boolean field that we check, and do nothing if that's already true.
@alexmv thoughts?

We still need resend_request parameter to make sure we do not make repeated requests when calling from mark_client_active.

Ahh, cool; we should probably make mark_client_active do this explicitly.

But now we do not wait for the previous request to finish before making another request, not sure if it can lead to some weird behavior like setInterval did when resuming the client from suspend mode.

I think this should be fine; no code executes at all while the device is asleep, so it's not going to be possible to accumulate more requests.
That said, there is an alternative fix, which would be to keep using setInterval, but just have a request_in_flight boolean field that we check, and do nothing if that's already true.
@alexmv thoughts?

I think we already have at least one "request in flight" bool somewhere, and I think adding more isn't the right direction to go.

OK, pushed an update that just documents the decisions made in this implementation and their motivation in more detail.

@alexmv pointed out in the chat.zulip.org thread that putting the setTimeout inside get_full_presence_list_update would probably be cleaner; after thinking about the big block comment explaining why we're not using setInterval here, I agree. So went with that design.


Roll Flutter Engine from d1a08064e193 to c3482189e568 (4 revisions)

https://github.com/flutter/engine/compare/d1a08064e193...c3482189e568
2024-11-21<EMAIL_ADDRESS>Roll Dart SDK from dde57dc75c15 to b36e4d731d67 (1 revision) (flutter/engine#56723)
2024-11-21<EMAIL_ADDRESS>[native assets] Consume NativeAssetsManifest.json (flutter/engine#56727)
2024-11-21<EMAIL_ADDRESS>[Impeller] support GLES 3.0 MSAA without extension. (flutter/engine#56705)
2024-11-21<EMAIL_ADDRESS>Updated some impeller benchmark urls (flutter/engine#56721)
If this roll has caused a breakage, revert this CL and stop the roller
using the controls here:
https://autoroll.skia.org/r/flutter-engine-flutter-autoroll
Please CC aaclarke<EMAIL_ADDRESS>on the revert to ensure that a human
is aware of the problem.
To file a bug in Flutter: https://github.com/flutter/flutter/issues/new/choose
To report a problem with the AutoRoller itself, please file a bug:
https://issues.skia.org/issues/new?component=1389291&template=1850622
Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+doc/main/autoroll/README.md

Commit queue failed; closing this roll.


Investigate ways to improve processing times for large transaction bundles

Is your feature request related to a problem? Please describe.
Transaction bundles can contain thousands of resources and are processed sequentially (one thread). Large bundles can exceed the configured transaction timeout, causing the transaction to be rolled back, throwing away a lot of work.
Describe the solution you'd like
Investigate ways to streamline the processing of bundles.
Batch bundles could be processed by a thread-pool internally to parallelize the effort, but all the work for transaction bundles must be completed within the scope of one UserTransaction (single thread).
Things to look at:

Delay the start of the transaction to be as late as possible...not until the first execute call.
Consolidate parameter processing from multiple entries to reduce the number of database round-trips.
Make parameter updates asynchronous (complex to maintain transaction semantics for the batch).

Describe alternatives you've considered
Alternatives will be investigated as part of this issue.
Additional context
Increasing Liberty transaction timeout may require similar updates to gateway/proxy configuration to avoid timeouts along the client call path.

While improvements here would be great, I also think there should be some reasonable "max size" for the transaction bundle.
For example, for the search and history operations, we cap the the size of the response bundles at 1000.
Asking clients to not exceed 1000 resources in a single transaction bundle does not seem unfair to me.
Hopefully they could find some grouping of resources, perhaps "per encounter", that could be committed atomically in smaller batches.
Alternatively, if resource count is not a good proxy, we can limit the request by content-size instead.
For example, in at least one of our cloud deploys, we reject requests that are over 1MB.

Number of resources in a bundle is a good proxy due to the sequential nature of processing. Currently bundles which are too large will be rejected, but only after the transaction times out and therefore after you've spent a long time consuming compute and IO resources. But I still think there are some good opportunities, particularly with the way parameters are stored to reduce the number of database round-trips which should see fairly significant reductions in processing times.


Report-backs to GEI, Q3

Provide a brief reportback on the activities of the WG at each GEI Open Office meeting in Q3 2024.
Assigned to Jenn
$20 per instance - total of 6 instances.
To be paid at the end of the quarter.
(NB the GEI meeting on 2nd July was just an introduction - no reportback was given)

[x] 16th July
[ ] 30th July
[ ] 13th August
[ ] 27th August
[ ] 10th September
[ ] 24th September

30th July reportback doc https://docs.google.com/document/d/10aSXKo14xxbeZx5l0eH-Zqh-Eo8MKiIE4FU1Q7E_YjI/

Moving to Done


option to disable redis

Unfortunately, I don't understand the point of this method to disable Redis. The best option to disable Redis completely is just to pass redis: null in the option or set useRedis: false, but this is to disable Redis for queries

The option redis: null not working
The package tries to connect redis no meter what...

ah, sorry. I will check this tomorrow. I don't want to keep Redis disabled by default, because in most cases it's needed

I checked and the null value in Redis works correctly. Maybe the code is not apparent, unfortunately.

Thanks a lot for the suggestion. I fixed it in the latest commits


changed nostr relays from examples

I was seeing this issue #100, I went to test it and it seems that the wss://nostr.zebedee.cloud relay is no longer available.
I switched to wss:///relay.stoner.com and now it seems to be working fine.

Thank you, but can I ask you to remove the extra / from your URLs before I merge?

Thank you, but can I ask you to remove the extra / from your URLs before I merge?

For sure, I ended up copying it wrong with the extra /. Adjusted!

Thank you, but can I ask you to remove the extra / from your URLs before I merge?

For sure, I ended up copying it wrong with the extra /. Adjusted!


SAK-31697 allow icons for tools that don't start with "sakai"

Sure thing.  I think I understand the problem and you're right, using icon- would conflict with the bootstrap icons.
The goal of the wildcard selectors was to allow the Sakai icon library (Font Awesome currently) to be used easily in any context in the application, without the developer having to define an explicit selector.  GradebookNG and Tool Order both use this to show tool icons, and doing a git grep icon-sakai I can see that Forums, Assignments and Site Info use the mechanism to show other Font Awesome icons.
Could an option 3) be we add those alternate prefixes to this list, e.g.
[class^="icon-sakai"],
[class*=" icon-sakai"],
[class^="icon-myprefix"],
[class*=" icon-myprefix"],
.icon-myawesometool,
.icon-myotherawesometool {
  @extend .fa-lg;
  @extend .fa;
}

This would be my preference, as it puts the onus on the tool developer to make one change rather than other UI developers having to add the selector of every spot they want to show a tool icon.

Perhaps it's all much of a muchness... Do we want the onus on the developer adding the tool or the UI developer showing the icons? Maintenance of those specific selectors might become a headache, but maintaining a long list of icon prefixes is annoying too.  I'll leave all this up to the Morpheus folks to decide!
If we go with option 2), could you please add selectors for these cases:
Gradebook:

Tool Order

Site Info tool:

Forums:

Assignments:

Hi Payten,
Thanks for all the thought on this...
The one thing that I liked the most was that someone could easily got to the file and with one line add their icon or update an existing. I would try to make it the easiest for the non-tech and simply want to add their icon or add a new one.
I would also appreciate others that would like to way in (nudge morpheus folks)
Maybe we just need a better namespace than "icon-", i.e. "sakai-icon--sakai-assignment"

OK this PR has been completely revamped. The idea is to stick with the selector we had previously and create a namespace for icons in sakai:
icon-sakai--
Icons for tools use the prefix plus the tool id with any "." converted to "-" in the tool id for example:
tool id -> css icon
sakai.samigo => icon-sakai--sakai-samigo

@payten any comments on the new approach?

Yep, looks good @ern.  It satisfies the "I can add a Sakai icon anywhere without having to change the CSS" case  - which is something I really like in the current implementation.


class 'PaystackPlugin' doesn't have a default constructor

I kept getting a class 'PaystackPlugin' doesn't have a default constructor  whenever i try setting an instance of the plugin as stated in your readme. Is there something i am getting wrong?

yes. I tried another method of initializing directly, but i kept getting a public key not set or invalid.

What version of the package are you using?

i didn't set a version.
flutter_paystack:

Can you use this: flutter_paystack: ^1.0.5?
Delete pubspec.lock and run flutter packages get

Thank you. I have been able to fix it


Repository: Missing reports - Number of panels (samples) sent per county/facility/program

Requirement:

Number of panels (samples) sent per county/facility/program over a given time period (eg Jan-Dec 2016)
County coordinators should be able to list the same for their counties
Accompanying bar/column graph
This data/analysis should not necessarily be limited to a specific provider.

With these we should be able to answer the following questions:

How many panels were sent out?
How many panels went to a specific county/facility/program? Do a side by side comparison with other counties/facilities/programs.

We don't have that data with the current data input. Left for later.


Upload the contents of the app_dir, not the dir itself

The most basic no-op stager should simply be able to download,
extract, upload without modifying the app.  However, upload
currently wraps the files that were downloaded in another folder
before tarring them back up; if you had multiple stagers this
really adds up.
If the user really wants their files wrapped with a particular
folder, it is very simple to use execute_app to move everything
into a new folder.

@zquestz LGTY?
This modifies the default behavior; not sure how concerned you are with that.
I could modify it so we just add another upload_files method but 1) it wouldn't work with the convenience method "complete" 2) It is just extra tests, extra methods for very little payoff since the library still provides the means to wrap in a directory manually via:
execute_app("mkdir new_dir && cp ./* new_dir")
Seems better to me to just put the more basic behavior as the built in method.

fyi: tests are "failing" but I put in an issue for this.  The failing tests are the ruby 1.8.7 tests which are also failing on master.

If we do merge as a change of default behavior; I should modify the gemspec and move from version 0.2.5 to 0.3.0 and document the change in CHANGES.md, right?

Changing the default behavior is a no go, as if we do that it will break existing stagers based on the library which are in sample apps. Unless we audit all the existing stagers and confirm that this is working NAK. I would have a flag, or other way to change the behavior and offer both options.

If you do go the route of auditing the existing stagers, we should probably do a major version bump due to the breaking change, and make sure that all the sample stagers are working with the new version.

So clearly getting a bunch of errors here since I'm modifying the method signature for extract.  If we are uploading just the files in the @app_path then it doesnt really matter which directory the @app_path is since it wont be tar-ed up as well.  So there isn't really a point to forcing the user to pass a string along and extracting into that path, is there?
I'm not sure of some edge cases here, so I'll let you make the call.  In the most simple cases (download, then upload, or mix in just a few execute_app calls) it would seem really contrived/confusing to have to supply a do-nothing string to extract.
[Wanted your input on this before I update all the tests]

@zquestz Did a tour of the samples; most don't need any sort of accommodation now since the legacy behavior is still supported.
There was one sample where the stager specifically did not want the extra directory and they had to remove it; now that wasn't required.

LGTM!


Plugin settings refactor

This PR adds a new endpoint to the plugin detail endpoint.
Includes changes changes from #4182
Needed for #3901

@matmair looks like some conflicts here which need to be addressed :)

@SchrodingersGat conflicts are fixed and I added more tests.


:seedling: Pinned dependencies: create findings from processing errors

What kind of change does this PR introduce?
Refactoring
(Is it a bug fix, feature, docs update, something else?)

[x] PR title follows the guidelines defined in our pull request documentation

What is the current behavior?
Currently processing errors are not converted to findings before evaluation logs them.
What is the new behavior (if this is a feature change)?**
Change the pinned dependencies evaluation so that processing errors are converted to findings with OutcomeNotAvailable before evaluation logs them.

[ ] Tests for the changes have been added (for bug fixes/features)

Which issue(s) this PR fixes
NONE

Special notes for your reviewer
Does this PR introduce a user-facing change?
NONE

/scdiff generate Pinned-Dependencies


Can i restrict drives redirection and clipboard transfer?

Centos + XRDP.
Can I restrict for users local drive redirection and clipboard transfer for security reason ?

You can disable both features by modifying /etc/xrdp/xrdp.ini.
In the [Channels] section, set rdpdr=false to disable drive redirection and cliprdr=false to disable clipboard redirection.
Then restart your xrdp service to use the new settings.

Thanks! That what I need.

Thanks! That what I need.


fix (?) kinematics parts to write all Q vars to VDS

Comment seems to suggest it was meant to, nice to have for off-axis error checking.
Alternative or additional change would be to have all known scannables which are being transformed linked and named in the VDS (currently only the active axes are).
Almost definitely won't get time to merge this before leaving, so will likely need to be picked up.

It looks harmless enough. @BenBradnick do you have anything using kinematics at the moment to test this with?

@thomascobb I don't have anything currently, I think this will be waiting on Huw to be able to test on J13.

I should now be able to test this at some point as part of the new I18 sample stage work.


Added support for default platform in requiring filenames

Summary
When react-native application has some component with platform-specific code, Jest can't resolve paths properly. Namely, if import Foo from ../../utils/foois used, where../../utils/foo.<defaultPlatform.jsdo exist, but../../utils/foo.js` does not, Jest fails.
Test plan
In sample app create foo.ios.js and foo.android.js then from some component try to resolve foo. It should pick up the implementation set by defaultPlatform from config.

This is another approach of #1947

We'll keep iterating on #1947. Thank you for your pull request!


[CHALLENGE SUBMISSION] 12

Please provide the following information for your submission:
What is your name? (First, Last)
Rudy, Scott
Where is your GitHub Repo ?(With your challenge solution)
https://github.com/scottrudy/25-days-of-serverless-dotnet/tree/master/day12
What Challenge is this for? (Challenge number between 1 and 25)
12
(Optional) Anything else we should know? e.g., language used, location, blog post?
Azure Functions 3.0 using .Net Core 3.1 with C#. HTTP triggered function leveraging input binding for Azure Table Storage for caching. Leverages the Github Content API to get README files for a repo and the MarkdownDeep library from Topten Software. The Github ETag is used to check for cache invalidation.

From all of us on the Cloud Advocacy team at Microsoft, thank you for creating this issue!
Have a Happy Holiday season!
Feel free to share your solution on Twitter with the tag #25DaysOfServerless

THANK YOU FOR YOUR #25DaysOfServerless SUBMISSION
We checked out your repository. We're thrilled to say your solution is now listed in our Hall of Fame
Thank you for your contributions!
This issue is now resolved! 🎉


Enabler: Improve CI pipeline for better productivity

Synopsis
As a team
We want a faster and more stable CI pipeline
So that we can be more productive
Acceptance-Criteria

[ ] The CI pipeline is significantly more stable
[ ] If viable: The CI pipeline does not execute .NET or Databricks actions when not necessary

The latter AC is very likely to involve team Mighty Ducks.

@mknic and I have decided that MD will look at this issue, since most of the items from @defectiveAi's research, points towards them. MD will contact us, if we need to contribute.


add: Depth-First Search (DFS)

Describe the algorithm/code
Issue Description : Depth-First Search (DFS) is a graph traversal algorithm that explores as deeply as possible along a branch before backtracking. It's used to visit all the nodes in a graph or tree, typically starting from a root node. DFS is often implemented using recursion or a stack data structure and is useful for tasks like finding connected components, topological sorting, and pathfinding.
Hi there 👋
I'm excited to contribute to Hacktoberfest 2023, and I'd like to work on this issue. It looks like a great opportunity to learn and contribute to the project.
Please assign this issue to me, and I'll start working on it right away. If you have any specific instructions or guidelines, please let me know, and I'll make sure to follow them.
Thanks a lot for considering my request!
Best regards,
Prateek Sethi
Do you want to work on this feature?

[X] I am willing to implement this feature.

I guess we already have this implemented.
Check here: https://github.com/himanshu-03/Python-Data-Structures/tree/main/Graphs

Ok sir
I just see the algorithms present that file
I don't refer this file
Thankyou for telling me


Import or Export?

These two phrases repeated twice; first time in Export section, second in Troubleshooting the System users entity. Either it is incorrect or we missed some detail here.

While importing the system users entity, you may receive an integrity violation error if there is a guest user in the exported package. The guest user must be deleted from the package in order for the entity to work.
If a record already exists in the UserInfo table (the Admin record would most likely always exist), the import will fail for those records but work for other records

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

ID: 1016eba2-a0f9-8890-beb4-7c9f9f6df4fc
Version Independent ID: 57758813-9d36-975a-eb6f-551b7da3e276
Content: Data management overview - Finance & Operations | Dynamics 365
Content Source: articles/fin-ops-core/dev-itpro/data-entities/data-entities-data-packages.md
Service: dynamics-ax-platform
Product: ****
Technology: ****
GitHub Login: @Sunil-Garg
Microsoft Alias: sunilg

@wojzeh, thanks for the feedback.
@Sunil-Garg, please review.

@wojzeh thanks for highlighting that; we have removed that content from this doc as this was not the right doc for entity specific comments, anyway. Thanks again.


Use .text() instead of .append() for default translation

Use text instead of append for default translation
In https://github.com/angular-translate/angular-translate/blob/master/src/directive/translate.js#L291 is used .append() instead of .text() This prevents escaping of <> and other special characters. We use translate directive in mixed cases. The interpolated value could be a translation key or a normal string which is binded to the controller or scope.
Your environment

2.12.1 of angular-translate
1.5.8 of angular
chrome latest

Steps to reproduce
https://plnkr.co/edit/AmfWUn75kEtvVZcrbFN9?p=preview
Expected behaviour
Escaping should be happen.
Actual behaviour
Escaping is not active.

Hi, please have a look at our guide, in chapter Security.
An overview of the modes is available at http://plnkr.co/edit/rJ0R6GzHGbMVxhSa237C?p=preview
We have some open discussion in general about this topic.
I'll close this issue, because we will not replace it. Otherwise we could never support unescaped content.

This issue is not about the content. It is about the key which is used to determine a translation. We use HTMl content as well, but for the default translation case, which is, not to find any content for the key, should be escaping like a normal {{ctrl.binding}}-binding does.

Actually, this is content. And you are pointing to the content's injection part.
{{ctrl.binding}} is an implicit filter binding and will be escaped always (see also my last link). Again: We have different modes to support.
We strongly recommend to use IDs as translation keys which is meant without HTML or other structure code. Otherwise I cannot follow you..

OK, thank you for the clarification. I will use the a factory for useMissingTranslationHandler to escape keys.


Pretrain issue

Hi, I have a error when i was pretraining the model. The error is "RuntimeError: CUDA out of memory."
I have tried to set the batchsize:2( 2*1), but it was occurred the above error.
I only modify the pretrain dataset to my cunstom dataset to pretrain.Other configurations are the same with your code.
Here is the config:
Data
train_file: [
"./RSTPReid/pretrain/train_image.json",
]
train_dataset_size: 18505 # for IterableDataset
images: {image_key: "file_path",
image_root: "./RSTPReid/imgs/",
is_image_rpath: True, # read path or base64 encoding
caption_key: "captions",
tokenized: False,  # whether texts have been tokenized
batch_size: 2,  # 128 x 8 = 1024
num_workers: 1,  # better -> the total number of training files % (world_size * num_workers) == 0
}
train_file_regions: [
'./RSTPReid/pretrain/train_bb.json',
]
regions: {image_key: "image", image_root: "/mnt/csip-101/reid-datasets/RSTPReid/imgs/",
is_image_rpath: True, caption_key: "caption", tokenized: False,
iter_perc: 1, batch_size: 128, max_images: 50, max_regions: 5, min_perc_in_image: 0.5, num_workers: 1}
Vision Encoder
use_beit_v2: True
vision_config: 'configs/config_beit2_base.json'
image_res: 224
patch_size: 16
local_attn_depth: -1
Text Encoder (& Cross Encoder)
text_encoder: './weight/bert-base-uncased'
text_num_hidden_layers: 18  # include cross
text_fusion_start_at: 12
Training
mixed_in_batch: True
calc_image_bbox_loss: False
embed_dim: 256
temp: 0.07
max_words: 40
max_tokens: 40
mask_prob: 0.5
max_masks: 12
mask_whole_word: True
skipgram_prb: 0.2
skipgram_size: 3
Other Settings
ckpt_frequent_step: 50000
ckpt_frequent:<PHONE_NUMBER>  # epoch
optimizer: {opt: adamW, lr: 1e-4, weight_decay: 0.01, lr_mult: 2}
schedular: {sched: linear, lr: 1e-4, epochs: 101, num_warmup_steps: 2500}  # 之前是跑 200k steps, 现在感觉要跑 500k steps
accelerator: {SYNCBN: false, FP16_OPT_LEVEL: O1, FP16_LOSS_SCALE: dynamic, RNG_SEED: 42, GRAD_ACCUMULATE_STEPS: 1, CLIP_GRAD_NORM: 1.0}

figure out, region batch size did not set.


Could not load the default wrapper

I get the error " UserWarning: Could not load the default wrapper for your platform: cpy, performance may be affected!".
Can and should I do something about it?

Hi, the problem for me was the extension did not compile during python setup install. I needed to install python dev libraries and then it works. Hope it helps.

Are you in a virtual environment?
I found that I could install with my system pip, but using the virtualenv pip failed. I believe it's an include directory issue, but I don't have enough knowledge to fix it.
Like @stereoit said, if you want to use it in a virtualenv, check out the code, build it locally, then pip install the local copy.


No platform events produced with standalone Kafka consumers

Describe the bug
We are not seeing any EntityChangeEvent_v1 events being triggered in actions (e.g. the Hello World action).
It seems that the messages are not even produced for the platform event topic. Messages for other topics e.g. the metadata change log topic are produced and consumed successfully. I can see debug log messages in GMS about the KafkaEventProducer.java producing MetadataChangeLogs but there's nothing about platform events (e.g. method https://github.com/datahub-project/datahub/blob/e6d706687ab5af6b888176bb775c338ff9[…]java/com/linkedin/metadata/dao/producer/KafkaEventProducer.java).
After some investigation we think it's related to running standalone Kafka consumers. All the required ENV vars (PE_CONSUMER_ENABLED, MAE_CONSUMER_ENABLED) are set correctly and visible on the pods. Topics and consumer groups are set up correctly and the ENV vars referring to the topics also seem to be correct.
Switching back to integrated consumers shows EntityChangeEvent_v1 events being triggered again. Changing back to standalone consumers by setting datahub_standalone_consumers_enabled to true in the Helm values brings back the issue.
We're using the datahub Helm chart version 0.3.8 and Datahub 0.12.0
More information in this Slack thread: https://datahubspace.slack.com/archives/C029A3M079U/p1707290303707169?thread_ts=1653649465.225869&cid=C029A3M079U
@david-leifker already looked into this and if I understood correctly, he was able to reproduce the issue.
To Reproduce
Steps to reproduce the behavior:

Switch to standalone consumers by setting datahub_standalone_consumers_enabled to true in the Helm values
Run the Hello World action
Add an owner to a dataset an check in the action logs if the EntityChangeEvent_v1 was logged
See that no platform event was triggered

Expected behavior
Action should log an EntityChangeEvent_v1 event.

@haeniya
I suppose this has already been fixed in Datahub 0.13.0 here

We just updated to Datahub 0.13.2 and can see the platform events again in the actions. Thanks @trialiya.


helm package fails without local repository

The package command in helm will fail without a configured local repository.  I'm building a chart repository with Jenkins and don't require this, it would be nice if it did not error out.
Errors:
helm package ./dir
> Error: open /home/jenkins/.helm/repository/local: no such file or directory
mkdir -p /home/jenkins/.helm/repository/local
helm package ./dir
> Error: open /home/jenkins/.helm/repository/local/index.yaml: no such file or directory

Work-around:
mkdir -p /home/jenkins/.helm/repository/local
cat <<EOF > /home/jenkins/.helm/repository/local/index.yaml
apiVersion: v1
entries: {}
generated: 2016-12-19T11:33:32.034887011-06:00
EOF
< go about your business >

thanks, this works too

Cool! That's good to know. Thanks. I'm closing this issue out, since I don't think there's additional work.
Thanks again!

I encountered this issue while migrating from helm 2 to 3. I had to copy the old index.yaml from repository/local in the old helm folder and copy it to ~/.cache/helm/repository/local-index.yaml


chore(dependencies): use hermes-engine 0.11.0

Codecov Report
Patch and project coverage have no change.

Comparison is base (1a8f0ed) 66.66% compared to head (308b13c) 66.66%.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
Additional details and impacted files
@@           Coverage Diff           @@
##           master      #84   +/-   ##
=======================================
  Coverage   66.66%   66.66%
=======================================
  Files           1        1
  Lines           3        3
  Branches        1        1
=======================================
  Hits            2        2
  Misses          1        1

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


New flow charts and README text

This PR resolves #150 (and some of #148).

using Mermaid (supported by GitHub) instead of Graphviz DOT
two charts are better than one
new rendered images and a hint for others to create them
removed "A note about CPE" from README.md
edited README.md to reference flowcharts.md

Nice. Thanks @amanion-cisa !


sip-hass-card SIP clients do not connect to Asterisk Add-on

I have two sip clients configured. Details attached: sip-hass-card-config-yaml.txt.txt
The clients do not connect to the Asterix addon.
I cannot see a connection when checking with pjsip list endpoints.
I successfully connected SIP clients to Asterix addon using various SIP User Agents.
I then checked for java script errors in the browser console and found:  console-errors.txt
It seems like the java-script variable this.config.prefix is not defined in webpack://ha-sip/src/sipjs-card.ts:715
I have no idea where such prefix needs to be defined.

I found the problem:  The entry prefix: "" was missing in  sip-hass-card.
I did a lot of playing and changing in this config file.
Is there a way to start from zero with all the endpoints in asterisk?
The ones in asterisk are not the same as specified in sip-hass-card


Off CPU profiling

This is the code that backs
https://github.com/open-telemetry/opentelemetry-ebpf-profiler/pull/144. It can be reused to add features like requested in https://github.com/open-telemetry/opentelemetry-ebpf-profiler/issues/33 and therefore can be an alternative to
https://github.com/open-telemetry/opentelemetry-ebpf-profiler/pull/192.
The idea that enables off CPU profiling is, that perf event and kprobe eBPF programs are quite similar and can be converted. This allows, with the dynamic rewrite of tail call maps, the reuse of existing eBPF programs and concepts.
This proposal adds the new flag '-off-cpu-threshold' that enables off CPU profiling and attaches the two additional hooks, as discussed in Option B in https://github.com/open-telemetry/opentelemetry-ebpf-profiler/pull/144.

Rebased proposed changes on current main and resolved merge conflicts.

I tried running this on aarch64 linux 6.10.14 and one of the programs failed to verify:
brennan@denver ~/code/opentelemetry-ebpf-profiler [env]$ sudo ./ebpf-profiler -collection-agent=<IP_ADDRESS>:7070 -disable-tls -tracers '' -off-cpu-threshold=30
INFO[0000] Starting OTEL profiling agent v0.0.0 (revision head-ed083e95, build timestamp<PHONE_NUMBER>)
ERRO[0000] 0: R1=ctx() R10=fp0
ERRO[0000] 0: (7b) *(u64 *)(r10 -368) = r1       ; R1=ctx() R10=fp0 fp-368_w=ctx()
ERRO[0000] 1: (85) call bpf_get_current_pid_tgid#14      ; R0_w=scalar()
ERRO[0000] 2: (bf) r8 = r0                       ; R0_w=scalar(id=1) R8_w=scalar(id=1)
ERRO[0000] 3: (b7) r6 = 0                        ; R6_w=0
ERRO[0000] 4: (7b) *(u64 *)(r10 -360) = r8       ; R8_w=scalar(id=1) R10=fp0 fp-360_w=scalar(id=1)
ERRO[0000] 5: (bf) r1 = r8                       ; R1_w=scalar(id=1) R8_w=scalar(id=1)
ERRO[0000] 6: (77) r1 >>= 32                     ; R1_w=scalar(smin=0,smax=umax=0xffffffff,var_off=(0x0; 0xffffffff))
ERRO[0000] 7: (7b) *(u64 *)(r10 -376) = r1       ; R1_w=scalar(id=2,smin=0,smax=umax=0xffffffff,var_off=(0x0; 0xffffffff)) R10=fp0 fp-376_w=scalar(id=2,smin=0,smax=umax=0xffffffff,var_off=(0x0; 0xffffffff))
ERRO[0000] 8: (15) if r1 == 0x0 goto pc+233      ; R1_w=scalar(id=2,smin=umin=umin32=1,smax=umax=0xffffffff,var_off=(0x0; 0xffffffff))
ERRO[0000] 9: (bf) r1 = r8                       ; R1_w=scalar(id=1) R8_w=scalar(id=1)
ERRO[0000] 10: (67) r1 <<= 32                    ; R1_w=scalar(smax=0x7fffffff00000000,umax=0xffffffff00000000,smin32=0,smax32=umax32=0,var_off=(0x0; 0xffffffff00000000))
ERRO[0000] 11: (77) r1 >>= 32                    ; R1=scalar(smin=0,smax=umax=0xffffffff,var_off=(0x0; 0xffffffff))
ERRO[0000] 12: (15) if r1 == 0x0 goto pc+229     ; R1=scalar(smin=umin=umin32=1,smax=umax=0xffffffff,var_off=(0x0; 0xffffffff))
ERRO[0000] 13: (85) call bpf_ktime_get_ns#5      ; R0_w=scalar()
ERRO[0000] 14: (7b) *(u64 *)(r10 -384) = r0      ; R0_w=scalar(id=3) R10=fp0 fp-384_w=scalar(id=3)
ERRO[0000] 15: (bf) r2 = r10                     ; R2_w=fp0 R10=fp0
ERRO[0000] 16: (07) r2 += -360                   ; R2_w=fp-360
ERRO[0000] 17: (18) r1 = 0xffff4a81e8230000      ; R1_w=map_ptr(map=profile_off_cpu,ks=8,vs=8)
ERRO[0000] 19: (85) call bpf_map_lookup_elem#1   ; R0_w=map_value_or_null(id=4,map=profile_off_cpu,ks=8,vs=8)
ERRO[0000] 20: (15) if r0 == 0x0 goto pc+221     ; R0_w=map_value(map=profile_off_cpu,ks=8,vs=8)
ERRO[0000] 21: (79) r1 = *(u64 *)(r0 +0)         ; R0_w=map_value(map=profile_off_cpu,ks=8,vs=8) R1_w=scalar()
ERRO[0000] 22: (7b) *(u64 *)(r10 -392) = r1      ; R1_w=scalar(id=5) R10=fp0 fp-392_w=scalar(id=5)
ERRO[0000] 23: (85) call bpf_ktime_get_ns#5      ; R0=scalar()
ERRO[0000] 24: (7b) *(u64 *)(r10 -400) = r0      ; R0=scalar(id=6) R10=fp0 fp-400_w=scalar(id=6)
ERRO[0000] 25: (b7) r9 = 0                       ; R9_w=0
ERRO[0000] 26: (63) *(u32 *)(r10 -344) = r9      ; R9_w=0 R10=fp0 fp-344=????0
ERRO[0000] 27: (bf) r2 = r10                     ; R2_w=fp0 R10=fp0
ERRO[0000] 28: (07) r2 += -344                   ; R2_w=fp-344
ERRO[0000] 29: (18) r1 = 0xffff4a818a184200      ; R1_w=map_ptr(map=per_cpu_records,ks=4,vs=4328)
ERRO[0000] 31: (85) call bpf_map_lookup_elem#1   ; R0_w=map_value_or_null(id=7,map=per_cpu_records,ks=4,vs=4328)
ERRO[0000] 32: (bf) r7 = r0                      ; R0_w=map_value_or_null(id=7,map=per_cpu_records,ks=4,vs=4328) R7_w=map_value_or_null(id=7,map=per_cpu_records,ks=4,vs=4328)
ERRO[0000] 33: (18) r6 = 0xffffffff              ; R6_w=0xffffffff
ERRO[0000] 35: (15) if r7 == 0x0 goto pc+206     ; R7_w=map_value(map=per_cpu_records,ks=4,vs=4328)
ERRO[0000] 36: (b7) r1 = 256                     ; R1_w=256
ERRO[0000] 37: (6b) *(u16 *)(r7 +4340) = r1
ERRO[0000] invalid access to map value, value_size=4328 off=4340 size=2
ERRO[0000] R7 min value is outside of the allowed memory range
ERRO[0000] processed 35 insns (limit 1000000) max_states_per_insn 0 total_states 2 peak_states 2 mark_read 2
ERRO[0000] Failed to start agent controller: failed to load eBPF tracer: failed to load eBPF code: failed to load kprobe eBPF programs: failed to load finish_task_switch

With https://github.com/open-telemetry/opentelemetry-ebpf-profiler/pull/196/commits/d4a09ade23a3bf100abf0d7833d9fcb9ef681b60 the propagation of off CPU profiles in user space and reporting was added. The PR was rebased to resolve recent API changes in the reporter package.

Needed to force-push to the branch to resolve merge conflicts. Looking forward on getting feedback.

Needed to force-push to the branch to resolve merge conflicts. Looking forward on getting feedback.

friendly ping @open-telemetry/ebpf-profiler-approvers for feedback.

To help review this PR a special version of devfiler was created that can differentiate between on- and off-CPU profiling information.
One can fetch this special devfiler version via:
curl -L -H 'Authorization: 236319c33321f5c7' -o 'devfiler-v0.9.0-samplekind.tar.gz' https://upload.elastic.co/d/0bf3587c1af900d95bed82bdade72359db8e940aaf92aa62de39f309d760b38f

In this special devfiler version one can select between Mixed, On CPU and Off CPU data in the drop down menu next to the time selector at the right top of the application.
I hope this helps to get more feedback for this PR.

friendly ping @open-telemetry/ebpf-profiler-approvers for feedback.

Needed to force-push to the branch to resolve merge conflicts. Looking forward on getting feedback.

Needed to force-push to the branch to resolve merge conflicts. Looking forward on getting feedback.


Add Query builder

#291
I'm not sure how close this is to what was wanted.
Would love some feedback!
:)

Something that is missing though is some documentation. I can add that myself after merging unless you want to take a shot at it.

Something that is missing though is some documentation. I can add that myself after merging unless you want to take a shot at it.

If you feel up to it. I probably won't get to it until Monday otherwise.

This doesn't seem to get passed through the compile time checks, is that correct?

@OskarPersson the latest commit is on a different feature branch for PR #1790, when this PR was merged it was passing CI.


Debugging SlimTrade

Running SlimTrade from the command line
SlimTrade can be run from the command line to cause debugging information, as well as errors, to be printed out. The easiest way to do this is with PowerShell.
Step 1
Open the folder containing SlimTrade, shift + right click inside the folder (not on the program), and select "Run PowerShell window here".

Step 2
When PowerShell opens, run the following command: java -jar "slimtrade.jar"

SlimTrade will then run normally, but if an error occurs it will be printed into the console. This functionality will be expanded later on to include more debugging information.
Powershell comes standard with Windows, but if for whatever reason you don't have access to it, here is an article explaining how to run a program using command prompt instead. Switch out the final command for the one above.

Moved to wiki.