Teams Index List Method PageToken Failure

I noticed that the List method on the Teams Index never returns a NextPage token.  It does return a PreviousPage token, but using that results in a failure indicating the pageToken parameter is invalid.  I know there are more than 250 teams and this code works in the javascript library.
Example Code:
        var auth = new RingCentralAuth
        {
            AppSecret = RingCentralAppSecret,
            AppKey = RingCentralAppKey,
            PlatformUrl = RingCentralPlatformUrl,
            UserExtension = extension,
            UserPassword = password,
            UserName = userName,
        });

        _client = new RestClient(auth.AppKey, auth.AppSecret, auth.PlatformUrl);
        _client.Authorize(auth.UserName, auth.UserExtension, auth.UserPassword).Wait();

        var navigation = new GlipNavigationInfo();
        do
        {
            // I can use prevPageToken here with same result
            var queryParameters = new ListGlipTeamsParameters { recordCount = pageSize, pageToken = navigation.nextPageToken };

            var teams = _client.Restapi().Glip().Teams().List(queryParameters).Result;
            foreach (var team in teams.records)
            {
                yield return team;
            }

            navigation = teams.navigation;

        } while (navigation.nextPageToken != null);  // I can use prevPageToken here with same result

This same code works using the javascript library - I ported it directly from a javascript function that is working in production today.

never returns a NextPage token
Do you mean the value is always null?

For quicker response, you can send email to<EMAIL_ADDRESS>

I actually got it to work, but it's a bit of a kludgy hack.  I was looking at the code that one of your RC developers wrote for us on a contract and basically replicated that.  The code below is what is working for me:
        public async IAsyncEnumerable<GlipGroupInfo> GetTeams(long? pageSize = null)
        {
            var navigation = new GlipNavigationInfo();

            var uri = _client.Restapi().Glip().Groups().Path(false);
            do
            {
                var queryParameters = new { recordCount = pageSize ?? 30, type = new[] { "Team" }, pageToken = navigation.prevPageToken };

                var result = await _client.Get(uri, queryParameters);
                var contentString = await result.Content.ReadAsStringAsync();

                var groups = Newtonsoft.Json.JsonConvert.DeserializeObject<GlipGroupResponse>(contentString);

                foreach (var team in groups.Records)
                {
                    yield return team;
                }

                navigation = groups.Navigation;

            } while (navigation.prevPageToken != null);
        }

Do you have a RingCentral app installed? You can ping me to have a live chat. My email is<EMAIL_ADDRESS>I appreciate your feedbacks to this SDK.

This seems to be an issue with the API and not related to the SDK.  I'm going to close this issue.  Thank you Tyler for looking into this!

Thanks for the update!
Below is just a small trick:
I think
var result = await _client.Get(uri, queryParameters);
var contentString = await result.Content.ReadAsStringAsync();
var groups = Newtonsoft.Json.JsonConvert.DeserializeObject<GlipGroupResponse>(contentString);

could be simplified to
var groups = await _client.Get<GlipGroupResponse>(uri, queryParameters);


Publishing to uBO Regional Filters

Hi. I have been using these filters for awhile. Most common news sites seem pretty decent without all the annoying ads.
How about submitting these filters to the ublock origin's Regional Ads?
(We can use many technical forums to invite more people to contribute for Nepali sites ads submission. Even reddit too.)

Hello @designjeevan, cool. Feel free to submit this list to uBlock origin's Regional Ads and other places too. If you wish to collaborate on this, I would be happy to invite you as a collaborator.


Add PathFlag

This need additional work that is discussed in https://github.com/urfave/cli/issues/666.

FYI I'll rebase and squash the commits when ready.

This looks great @VMitov ! Thank you for adding the comprehensive tests. I just left one note about commenting the interface, but I think this is good to go after that.

:+1: looks great! Let me know if you still want to squash.

Rebased.


Created and added new preamble header for ic-sl sheets

Created new header (kopficsl) and added it to ic-sl sheets (12 sheets)

 header ic sl


Tab icon for safari does not seem to show

For some reason in the current release the tab icon for the Safari browser does not seem to show.
Requires investigation.

This has been fixed in the develop branch. This was as a result of the safari document.icns file no longer exists, so we now check if it does, and if not we use html.icns.
Will push this out as part of the 1.1 release.

Re-opening and leaving open, scheduled for release in v1.1

Releasing


Custom connection on schema building

Feature discussion / request

Explain what is your use case
I´d like to build a schema with a custom connection, this way I can for example disable the foreign_keys of a sqlite database in a migration. Furthermore it just seems consistent to me, as you can run a query with a custom connection. Of course this feature would need be used with caution, as the custom connection is not bound to outer transactions.

Explain what kind of feature would support this

Give some API proposal, how the feature should work
Maybe add connection(connection: Connection) to the schema builder.

@Taggotty Can't you just call migrations on an alternate knex instance?..

This should be done by running schema through separate knex instance as @kibertoad mentioned.


api to work with geo locations

Hello,
What the best way to work with geo locations in clickhouse ? I don't see support for havershine in order to be able to filter for results within a given radius of a given location. For example, filter events within a ten mile radius of San Francisco [-122.42005, 37.77479].
Any suggestions ?
Silviu

https://clickhouse.yandex/docs/en/functions/math_functions.html#tgamma-x

@sirkon  can you please elaborate. I don't see the link between havershine and tgamma.
Silviu

@silviucpp Perhaps they just were linking to the page on mathematical operators?
the equation posted looks like it's for the great circle distance : https://en.wikipedia.org/wiki/Great-circle_distance and it's composed entirely of operators that are already supported

@IanCal yes I figured out this after looking closer. I left this open maybe as an enhancement for clickhouse where we can have a function exported that takes as pramaters the longitude, latitude and a distance, and match the rows in that inverval.
Something like:
geoWithin(latitude1, longitude1, latitude2, longitude2, distance_miles) -> boolean

Will make selects more simple :)

Seems this functionality is already implemented in clickhouse but I don't see any documentation for:
SELECT floor(greatCircleDistance(33.3, 55.3, 38.7, 55.1)) AS distance;

Yes, the function greatCircleDistance is in production state. It is still missing in official documentation.

@alexey-milovidov should I close this bug or you want to close it when doc is updated ?

Better to close.


acme: nginx is required to serve challenge, but does not start without certs

I've enabled the acme service and configured nginx to serve the challenges. I also have a few virtualhosts that use the certificates that will be written by acme. Now, I've ran into a chicken-and-egg-problem: acme will create the certs only if nginx serves the challenges, but nginx does not start because the certs do not exist.
Jan 28 18:50:53 apu systemd[1]: Started Nginx Web Server.
Jan 28 18:50:53 apu polkitd[18970]: Unregistered Authentication Agent for unix-process:32425:32271023 (system bus name :1.32, object path /org/freedesktop/PolicyKit1/AuthenticationAgent, locale en_US.UTF-8) (dis
Jan 28 18:50:53 apu nginx[32436]: nginx: [emerg] BIO_new_file("/var/lib/acme/gillich.me/fullchain.pem") failed (SSL: error:02001002:system library:fopen:No such file or directory:fopen('/var/lib/acme/gillich.me/
Jan 28 18:50:53 apu systemd[1]: nginx.service: Main process exited, code=exited, status=1/FAILURE

I am undecided about how to solve this. Some options:

Create self-signed certificates (requires reload)
Stop running webserver and start a new one to serve challenges (requires stopping and starting)

Both would require knowledge of the webserver that is actually used on port 80.
@fpletz

We currently deploy in 2 stages, once with http only then with https, deploying a snakeoil cert might work, too.
Also we are working nginx improvements:
https://github.com/mayflower/nixpkgs/commit/f9002c9011383594cd4a8cb968aaa8ab53fdf058
https://github.com/mayflower/nixpkgs/commit/e12b6fe880b006b56f4d4faf86fc63da7fbfdab7
https://github.com/mayflower/nixpkgs/commit/b42e7d2d067da74d0316401b6d44e2f0d70d73dc
https://github.com/mayflower/nixpkgs/commit/d8c159b7576d0cbec362fe2970d29219633afaba
nixops:
https://github.com/mayflower/mf-nixops/commit/f0ef0e9b8c18b65bb490aaa7c7da9bbee11f404b
https://github.com/mayflower/mf-nixops/commit/b7fec30f53f6d4a976282653580ab40a33a33f76
https://github.com/mayflower/mf-nixops/commit/50643e0d2ad07580972c5303e446ab03037dab0d
First deployed with forceSSL = false, then with above.

Awesome stuff! Assuming that one reloads the webserver in acme.postRun anyway, creating a self-signed cert would be a ok-ish solution, right?

AFAIR restarting is needed not reloading but no guarantee :wink:

We could also use system.activationScripts to fetch certificates before reloading the dependent services like the webserver, where we the chicken-egg problem is the biggest problem. However, you still need to have a webserver running and configured with a default vhost for serving the acme challenges.
I will try to fix this soonish.

Just for reference: upstream uses snakeoil certs: https://github.com/letsencrypt/letsencrypt/commit/be889d379437be18ce1c0518b2cfec15f6b8dd6a

FWIW I use a custom activation script which generates a self-signed certificate if one is not yet available and symlinks it to a place where nginx picks it from. Then I have a post-process step for acme service where I force this symlink to a new value.

@abbradar yeah I'm working on something similar.

I have a prototype but acme complaints:
Mar 11 22:01:47 eiger acme-hydra.snabb.co-start[27185]: 2016-03-11 22:01:47,682:DEBUG:simp_le:1198: Existing SANs: []
Mar 11 22:01:47 eiger acme-hydra.snabb.co-start[27185]: 2016-03-11 22:01:47,682:ERROR:simp_le:257: Existing () and requested (hydra.snabb.co) SANs mismatch
Mar 11 22:01:47 eiger acme-hydra.snabb.co-start[27185]: Backup and remove existing cert if you want to proceed

We'll have to symlink the certs and unlink when simp_le runs :(

Unfortunately yes, I worked around it by changing the path to some temporary location.

@domenkozar Are you using the version from this PR? https://github.com/NixOS/nixpkgs/pull/13451
This should have support to change SANs.

Nope, older version. I'll check with that PR.

That is good news. So all that needs to be done now is create a snake oil cert at /var/lib/acme/... when the cert doesn't exist?

I'm not using very ugly hack with activationscripts (which should just be a systemd oneshot), see https://github.com/domenkozar/nixpkgs/commit/4e256ca59966fba8e63e2d6c7d6898a6f2abcd3c

I've been asked to setup TLS on http://search.nix.gsc.io/. I'd like to, but have been holding out on doing so until this issue is resolved.
@domenkozar your definitely ugly script looks like it should work. I think that would be an improvement over what we have now. What would you think about putting in a PR for review?

I'm using the changes from @domenkozar on my server. Seems to be working nicely. I agree with @grahamc , it seems more people are looking for such a solution, as there isn't a better one at the moment why not get some more eyes on it with a PR?

I think if someone converts it to a systemd oneshot job and makes nginx/apache depend on it, we're likely to merge it.
nixos hydra uses a variation of this: https://github.com/NixOS/nixos-org-configurations/commit/19b956005ab850dcfc0112288e5a0c3f808c05f0#diff-73514aaa2e0242ed442a1a714999fb45R371

@domenkozar Done: #15562

#15562 has changed quite a bit, but it is now usable for any webserver. It adds the option security.acme.preliminarySelfsigned, which will automatically generate self-signed certificates for any of the ACME certs that isn't created yet. The documentation includes an example for Nginx.
It isn't merged yet, so more eyes are very welcome.

This was merged and it works now.


Internal server error with filesystem or s3

I'm trying to run this script with s3 or filesystem, but getting this error.
Tried on local macOS and remote Digital Ocean droplet.
Any help?
Initializing transfer configuration...
Initializing filesystem adapter
Fetching schema...
{"code":1,"message":"Internal server error."}

npm ERR! Darwin 16.3.0
npm ERR! argv "/usr/local/bin/node" "/usr/local/bin/npm" "start" "config-dev.js"
npm ERR! node v6.9.1
npm ERR! npm  v3.10.8
npm ERR! code ELIFECYCLE
npm ERR<EMAIL_ADDRESS>start: `node index.js "config-dev.js"`
npm ERR! Exit status 1
npm ERR!
npm ERR! Failed at the<EMAIL_ADDRESS>start script 'node index.js "config-dev.js"'.
npm ERR! Make sure you have the latest version of node.js and npm installed.
npm ERR! If you do, this is most likely a problem with the parse-files-utils package,
npm ERR! not with npm itself.
npm ERR! Tell the author that this fails on your system:
npm ERR!     node index.js "config-dev.js"
npm ERR! You can get information on how to open an issue for this project with:
npm ERR!     npm bugs parse-files-utils
npm ERR! Or if that isn't available, you can get their info via:
npm ERR!     npm owner ls parse-files-utils
npm ERR! There is likely additional logging output above.

npm ERR! Please include the following file with any support request:
npm ERR!     /Users/myuser/parse-files-utils/npm-debug.log

fixed same error by using https://api.parse.com/1 instead of http://api.parse.com/1


Implement local signing/verification

@dmitrizagidulin Implement local signing and verifying functions here

Will be useful for OneOrigin onboarding.


validateDefaultPage crash with single ViewController

When using public func setup(#viewControllers: [UIViewController], options: PagingMenuOptions) to  update after a view controller was deleted, and at that time the previous currentPage was the last page in the array, the app crashes on the following function in PagingMenuController.swift:
func moveToMenuPage(page: Int, animated: Bool)

Cause:  page is out of bounds
Proposed fix:
In setup() put currentPage = self.options.defaultPage  directly after self.options.menuItemCount = pagingViewControllers.count the page value will now be the default value, which can be set before updating.

Please use version 0.5.0.


feat!: allow path containing . to fallback to index.html

Description
This PR sets disableDotRule: true to connect-history-api-fallback.
I believe threre's no reason to set this to false for Vite.
fixes #2415
supersedes close #2634
Additional context

What is the purpose of this pull request?

[ ] Bug fix
[x] New Feature
[ ] Documentation update
[ ] Other

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

[x] Read the Contributing Guidelines.
[x] Read the Pull Request Guidelines and follow the PR Title Convention.
[x] Check that there isn't already a PR that solves the problem the same way to avoid creating a duplicate.
[x] Provide a description in this PR that addresses what the PR is solving, or reference the issue that it solves (e.g. fixes #123).
[ ] Ideally, include relevant tests that fail without this PR but pass with it.

@sapphi-red @bluwy This PR has caused some questions so far.
For example, when I use the img tag to load an image on the page if I write the wrong path, the console will print a 404 error before this PR, but the console will not print any information after this PR. This is very unfriendly to development prompts.
In my opinion, this impact seems to be quite large, what do you think? 🤔

@btea I think that should be expected since it's an SPA, and you should be able to opt-out of that with appType: 'mpa', but currently even in MPA it doesn't work (as discussed in https://github.com/vitejs/vite/pull/14142#discussion_r1299209757).
It looks like this happens because of https://github.com/vitejs/vite/pull/9865. The proper fix would be something on the last line of this comment: https://github.com/vitejs/vite/pull/9865#issuecomment-1229153735
We also can't revert this as the linked issue https://github.com/vitejs/vite/issues/2415 had a lot of request, and between two tradeoffs, I think it's better to lean into this PR instead.

Rewriting only when Accept header includes text/html might work. If many static hosting service does that, I think it makes sense to have that.
But I'm not sure how much apps will break if we do that.

Thank you for your replies! Regarding the problems that may be caused by the PR modification I mentioned, perhaps a detailed explanation should be added to the official website.
Or is it possible to provide users with relevant configurations to actively set fallback policies?


Feature request: import TextAdept lexers

https://github.com/orbitalquark/scintillua/tree/default/lexers

This is something I already considered but I'm not sure if it's worth it. We can quite easily manually port any of the languages supported by schintillua to ecode, I'm pretty sure I already did that for a couple of languages, it's just a matter of doing it.

This is something I already considered but I'm not sure if it's worth it. We can quite easily manually port any of the languages supported by schintillua to ecode, I'm pretty sure I already did that for a couple of languages, it's just a matter of doing it.

It's not always easy to port from LPeg to Lua Pattern. If you could support LPeg, it would be much better.

I have been taking a look and it does not seem easily feasible. I would need to integrate lua + lpeg + scintillua lexer. I still think that porting most of the languages from scintillua is pretty easy. I think that what makes more sense is to implement a translator tool from scintillua to ecode or even lite since we can translate from there too. That one wouldn't be hard.

I think that what makes more sense is to implement a translator tool from scintillua to ecode or even lite since we can translate from there too. That one wouldn't be hard.

I don't think such a tool is easy to implement. Finally, we still have to resort to translating manually.


hintrunner/zero: include the ap tracking data into addr calculations

This patch is aimed to include the ap tracking offsets into the produced ApCellRef. The calculation result is saved as an updated ApCellRef offset.
Given the ApCellRef(-3) and ap offset of hint 3 and ref 2, we'll get a new ApCellRef(-4) (-3 - (3 - 2) => -4).
Fixes #197

Looks like this test is flaky.
I'm looking for the source of this non-determinism.


Update sbt-sonatype to 3.9.6

Updates org.xerial.sbt:sbt-sonatype from 3.9.3 to 3.9.6.
GitHub Release Notes - Release Notes - Version Diff
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

Ignore future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "org.xerial.sbt", artifactId = "sbt-sonatype" } ]

labels: sbt-plugin-update, semver-patch

Superseded by #235.


[Bug]: unable to run julia REPL using grun

I'm trying to use julia REPL using glibc grun but it's unable to find sys.so file in ~/julia/lib directory.
however it has loaded libjulia.so.1 library file.
~/julia-1.9.4/bin $ grun --list ./julia
        linux-vdso.so.1 (0x00000078e84db000)
        libdl.so.2 => /data/data/com.termux/files/usr/glibc/lib/libdl.so.2 (0x00000078e84b0000)
        libpthread.so.0 => /data/data/com.termux/files/usr/glibc/lib/libpthread.so.0 (0x00000078e8480000)
        libc.so.6 => /data/data/com.termux/files/usr/glibc/lib/libc.so.6 (0x00000078e8260000)
        libjulia.so.1 => /home/julia-1.9.4/bin/./../lib/libjulia.so.1 (0x00000078e8220000)
        /lib/ld-linux-aarch64.so.1 => ld.so (0x0000003000000000)
~/julia-1.9.4/bin $ grun ./julia
ERROR: could not load library "/usr/glibc/lib/../lib/julia/sys.so"
/usr/glibc/lib/../lib/julia/sys.so: cannot open shared object file: No such file or directory

System information
termux-info:
~/julia-1.9.4/bin $ termux-info
Termux Variables:
TERMUX_APK_RELEASE=GITHUB
TERMUX_APP_PACKAGE_MANAGER=apt
TERMUX_APP_PID=20826
TERMUX_IS_DEBUGGABLE_BUILD=1
TERMUX_MAIN_PACKAGE_FORMAT=debian
TERMUX_VERSION=0.118.0
Packages CPU architecture:
aarch64
Subscribed repositories:
# sources.list
deb https://grimler.se/termux-packages-24/ stable main
Updatable packages:
All packages up to date
termux-tools version:
1.39.6
Android version:
13
Kernel build information:
Linux localhost 4.14.186-perf-g46e858b6644e #1 SMP PREEMPT Mon Aug 28 15:26:14 CST 2023 aarch64 Android
Device manufacturer:
Xiaomi
Device model:
22031116AI
LD Variables:
LD_LIBRARY_PATH=
LD_PRELOAD=/data/data/com.termux/files/usr/lib/libtermux-exec.so

i have just tested it. if not inter proot environment it wouldn't work. it's throwing: bad system call. maybe there is some hardcoded paths in its source code or something i'm not sure though.

it's throwing: bad system call

This happens because system call 1008 is run inside julia - https://github.com/JuliaLang/julia/blob/master/src/scheduler.c#L226. I don’t know why they want to run a non-existent system call, but apparently here need to compile julia for Termux.

@ANIL80531, here are the commands that install and configure julia-1.10.0 on Termux aarch64:
# Make sure you have glibc 2.38-11 installed
pacman -Syu

# Launch custom shell
grun --shell

# Julia installation
wget https://julialang-s3.julialang.org/bin/linux/aarch64/1.10/julia-1.10.0-linux-aarch64.tar.gz
tar xf julia-1.10.0-linux-aarch64.tar.gz

# Setting up Julia
grun --set julia-1.10.0/bin/julia
ln -s ${PWD}/julia-1.10.0/lib/libjulia.so.1.10.0 ~/../usr/glibc/lib/libjulia.so.1.10
ln -s ${PWD}/julia-1.10.0/lib/julia ~/../usr/glibc/lib

# Launch of Julia
./julia-1.10.0/bin/julia


Shouldn't it be dump (without s) ?

https://github.com/gunthercox/ChatterBot/blob/68527223db52c39cbb94393cdf313f39485c5856/chatterbot/trainers.py#L67

Yes, it should.

cool, I didn't feel like doing a pull request for exactly one character.
Thanks for the nice bot ;)


Add deku-cli and deku-node as apps in nix

Problem

Now that we have flakes it would be nice to have a simple way of running our tools with it
Solution

Adding the deku-cli and deku-node as apps.
They can now be run from the repository like nix run .#deku-cli or nix run .#deku-node.
We should also be able to run nix run github:marigold-dev/deku#deku-cli from anywhere.
Testing
Checkout branch and run  nix run .#deku-cli or nix run .#deku-node

Does it worth it to add some documentation?

@gsebil08, personally I don't think so: EDIT: I had the command wrong the first time. Fixed.
nix run .#deku-cli -- --help
nix run .#deku-node -- --help

will print out quite a bit of documentation.
What kind of docs are you looking for?

Basic simple user documentation, like in our readme.md add something like:

Run a node with nix
Simply run the following commands to run a deku-node:
$ nix run .#deku -- data/0


Tolerate $this usage in anonymous functions

Alternative solution for issue #536.
For $this var usage it gives warning

$this might not be bound by invoker of callable or might be bound to object of any class. Consider adding instance class check.

And hints developer to check class of $this before usage.
Class check can be added in form
if ($this instanceof [class])" {
    /*use $this*/
}

Note: if class check added no type checking is done in language server it just hides the warning.
There is a leaf in code marked by comment:

//else we should warn here that invoker can bind $this to object of any class. Should we?

Question in comment is for community.

I've never seen anyone use if $this instanceof so I don't think anyone would actually use that. I don't think it belongs into the language server.

@felixfbecker Here you go: https://github.com/pmmp/PocketMine-MP/blob/1f5f67e08712078588ef2241d870a0712b7a0cf4/src/pocketmine/block/Liquid.php#L182-L190

$this can be bound to ANY object by caller of Closure(anonymous function) so we must warn about it.
It's all about developer must know for sure how his function is called and which object is bound.
Class check for $this var must ensure expected results.
Anyone can suggest their way to do class check. I think there are many ways.
I've choosen $this instanceof because of simplicity and it's favor vs deprecated is_a.
I thought of example where you pass your anonymous function to "blackbox".
You never know for sure how it's called so checking $this var will help from getting into trouble.

I decided for disabling the warnings in these cases instead. This is similar to what e.g. TypeScript does - this is any by default, so no checks. Only in cases where it is known to be a certain type (or type hinted) should we do warnings. In the future, we can give the user the option to type hint $this, but until then we should force the user to write code to circumvent the warning.


pretty-print the ignore_for_file lines

pretty-print the // ignore_for_file: xxx lines (ensure they don't exceed 80 chars)
add a few blank lines in the generated method bodies to make them easier to read

Most of the changes in this PR are in the golden test files.
cc @osa1

@@osa1 - can you merge this? Thanks!


Unable to mount fileshare: Permission denied

I was following this tutorial: https://docs.microsoft.com/en-us/azure/aks/azure-files-volume
After deploying the pod, it keeps the status of "ContainerCreating". When describing it, I get:
Warning  FailedMount  36s (x9 over 2m44s)  kubelet            MountVolume.MountDevice failed for volume "models-volume" : rpc error: code = Internal desc = volume(unique-volumeid) mount "//{RESOURCE_NAME}.file.core.windows.net/test" on "/var/lib/kubelet/plugins/kubernetes.io/csi/pv/models-volume/globalmount" failed with mount failed: exit status 32
Mounting command: mount
Mounting arguments: -t cifs -o dir_mode=0777,file_mode=0777,uid=0,gid=0,mfsymlinks,cache=strict,nosharesock,nobrl,actimeo=30,<masked> //{RESOURCE_NAME}.file.core.windows.net/test /var/lib/kubelet/plugins/kubernetes.io/csi/pv/models-volume/globalmount
Output: mount error(13): Permission denied
Refer to the mount.cifs(8) manual page (e.g. man mount.cifs) and kernel log messages (dmesg)

Kubernetes version is 1.23.5
Client Version: version.Info{Major:"1", Minor:"23", GitVersion:"v1.23.5", GitCommit:"c285e781331a3785a7f436042c65c5641ce8a9e9", GitTreeState:"clean", BuildDate:"2022-03-16T15:58:47Z", GoVersion:"go1.17.8", Compiler:"gc", Platform:"linux/amd64"}
Server Version: version.Info{Major:"1", Minor:"23", GitVersion:"v1.23.5", GitCommit:"8211ae4d6757c3fedc53cd740d163ef65287276a", GitTreeState:"clean", BuildDate:"2022-03-31T20:28:03Z", GoVersion:"go1.17.8", Compiler:"gc", Platform:"linux/amd64"}

I have already double checked the credentials are correct.
Also I have another cluster (but with kubernetes version 1.18) that is mounting the same fileshare with no issue. So to be sure that the issue was not about both using it at the same time, I created a second file share and tried to mount that, unfortunately with the same result.

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

ID: d634c63a-6194-86a6-b522-3310812c81e8
Version Independent ID: fa86da1e-9c86-88fe-170c-5e6aa9230b5f
Content: Manually create Azure Files share - Azure Kubernetes Service
Content Source: articles/aks/azure-files-volume.md
Service: container-service
GitHub Login: @zr-msft
Microsoft Alias: zarhoads

Hello @gionapaolini,
Thank you for your feedback. We will investigate and update the thread further.

@gionapaolini,
If you are using Ubuntu 16.04 version, POD stuck in creating/pending state because can't mount Azure File share with below logs
Warning  FailedMount  36s (x9 over 2m44s)  kubelet            MountVolume.MountDevice failed for volume "models-volume" : rpc error: code = Internal desc = volume(unique-volumeid) mount "//{RESOURCE_NAME}.file.core.windows.net/test" on "/var/lib/kubelet/plugins/kubernetes.io/csi/pv/models-volume/globalmount" failed with mount failed: exit status 32
Mounting command: mount
Mounting arguments: -t cifs -o dir_mode=0777,file_mode=0777,uid=0,gid=0,mfsymlinks,cache=strict,nosharesock,nobrl,actimeo=30, //{RESOURCE_NAME}.file.core.windows.net/test /var/lib/kubelet/plugins/kubernetes.io/csi/pv/models-volume/globalmount
Output: mount error(13): Permission denied
Refer to the mount.cifs(8) manual page (e.g. man mount.cifs) and kernel log messages (dmesg)
Cause
This is not AKS issue, this is a network or kernel bug in 16.04 issue:
The Azure file share need specific port such as 445 and could be blocked by the customer side (Firewall/NSG)
Kernel bug issue in Ubuntu 16.04 version.
Troubleshooting
Make sure that the worker node can mount the azure file share by running this command inside the nodes:
sudo mkdir /mnt/MyAzureFileShare
sudo mount -t cifs //$STORAGEACCT.file.core.windows.net/myshare /mnt/MyAzureFileShare -o vers=3.0,username=$STORAGEACCT,password=$STORAGEKEY,dir_mode=0777,file_mode=0777,serverino
if everything is okay that led to kernel bug and there are patches released to Ubuntu 16.04 but not applied yet and no ETA for that 1  And 2
Workaround
Open the necessary ports to allow the connection to Azure file share
For kernel bug, there are two options:
Use Ubuntu 18.04 in AKS version 1.18.x and above
Or recreate the pv with mount option - nosharesock , if the customer is using statefulset for azure file share make sure to recreate it (pv,pvc and statefulset) in single yaml file

In Ubuntu 18.04 for Statefulsets, when PV and PVC are previously created the error persist.
noshareshock didnt work.
The situation is reproducible just with StatefulSet
K8S controller AZFile Dynamic AzDisk Dynamic AZfile premade AzDisk premade
StatefulSet WORKING WORKING BROKEN BROKEN
ReplicaSet WORKING WORKING WORKING WORKING
Deployment WORKING WORKING WORKING WORKING
Here is more context
After checking the logs and reproducing the situation, effectively the reported error message "mount failed: exit status 32" is part of a known situation creating a race condition between the linux kernel level and provisioning of the Azure File while
associating the POD/Statefulset with the respective PV/PVC.
https://docs.microsoft.com/en-us/azure/storage/files/storage-troubleshooting-files-performance#throughput-on-linux-clients-is-significantly-lower-when-compared-to-windows-clients
Resolution:
Create the statefulset, PVC and PV at the same time using a single script.
Example:
https://github.com/Azure/AKS/issues/1906#issuecomment-720480827
As well another option is to use the Preview feature with Container Storage Interface
https://docs.microsoft.com/en-us/azure/aks/azure-files-csi

@gionapaolini,
I would like to notify you that, this channel is for driving improvements towards MS Docs, for any product related question/issues, I would recommend you to create a thread on the forums- Microsoft Q&A / Stack Overflow. We will help you there.


Mihai's changes on the Fan's latest ontology changes ..

also incorporating some comments like spelling corrections, adding 'storm' etc.. Discussed this with @bsharpataz .. so not adding her as the reviewer ..
Planning to use this version for the creation of the gold dev set for grounding of the MITRE document mentions ..

@ajaynagesh, can you look at master's version and compare to what you started out with that may have been in an email attachment.  I'm concerned that you didn't start out with Fan's latest/last changes.
It looks like the file you just submitted has several blank lines, extra white space on the right, and it also seems to have CR/LF line endings.  Something may not be quite right.  (Maybe I messed something up.)

Hi @kwalcock,
I think the differences are minimal and it captures all the changes which Mihai's made. There are a few stray spaces at the end of some lines (and a couple of empty lines) . Would that be a problem ?

(the one the right is my changes in this PR compared to master on the left)

Hi @kwalcock: (@bsharpataz , @Fan-Luo ), I noticed an extra indentation after - Abiotic_resources:  (This is in the master branch and also in here - since it is based on the master branch)
Could you please check ?

Is a push forthcoming?  If nobody else does it, I will make sure the line endings are LFs.

No push from my end.
I only fixed the extra indentation pointed out by Ajay.
Please see: https://github.com/clulab/eidos/commit/3fd94f1a367be3f3bfca20b8118abd22f35cdd7f https://github.com/clulab/eidos/commit/3fd94f1a367be3f3bfca20b8118abd22f35cdd7f

On Jun 13, 2018, at 1:30 PM, Keith Alcock<EMAIL_ADDRESS>wrote:
Is a push forthcoming? If nobody else does it, I will make sure the line endings are LFs.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub https://github.com/clulab/eidos/pull/333#issuecomment-397075402, or mute the thread https://github.com/notifications/unsubscribe-auth/ALHU_LGYqyhXzW05_lDixgpH2NY67rFFks5t8XZZgaJpZM4UlGvg.

If someone has already committed to master, then this branch had better not go to master.  It should be closed.  In the version that recently went to master I do see "nAtuRAl ReSouRceS" for some reason.

I see that master's indentation was possibly fixed, but I don't know whether or not Ajay's changes were included at all.

For 'nAtuRAl ReSouRceS’ and misspelled ‘malaraia’, they are both from where Becky originally get for me.
I think it was from 'kwalcock-ontology branch' , according to the email Becky sent: 'Working from your kwalcock-ontology branch’.
I only extend several nodes, I did not do spelling check, etc.
By the way, I saw Ajay previously replied corrected  ‘malaraia’, but it is still there in master, do you want me to correct it?
If yes, @Keith, do you want me to create another branch to make this change, instead of committed to master?

On Jun 13, 2018, at 2:39 PM, Keith Alcock<EMAIL_ADDRESS>wrote:
I see that master's indentation was possibly fixed, but I don't know whether or not Ajay's changes were included at all.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub https://github.com/clulab/eidos/pull/333#issuecomment-397096316, or mute the thread https://github.com/notifications/unsubscribe-auth/ALHU_AEMuGMbX68GG-Bzpru41yEIE-L7ks5t8YZ7gaJpZM4UlGvg.

(wrong keith)

Either way as long as you coordinate with others who are making changes.
This PR already has conflicts which I won't know how to resolve.
On Wed, Jun 13, 2018, 11:59 PM Fan-Luo<EMAIL_ADDRESS>wrote:

For 'nAtuRAl ReSouRceS’ and misspelled ‘malaraia’, they are both from
where Becky originally get for me.
I think it was from 'kwalcock-ontology branch' , according to the email
Becky sent: 'Working from your kwalcock-ontology branch’.
I only extend several nodes, I did not do spelling check, etc.
By the way, I saw Ajay previously replied corrected ‘malaraia’, but it is
still there in master, do you want me to correct it?
If yes, @Keith, do you want me to create another branch to make this
change, instead of committed to master?

On Jun 13, 2018, at 2:39 PM, Keith Alcock<EMAIL_ADDRESS>wrote:
I see that master's indentation was possibly fixed, but I don't know
whether or not Ajay's changes were included at all.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub <
https://github.com/clulab/eidos/pull/333#issuecomment-397096316>, or mute
the thread <
https://github.com/notifications/unsubscribe-auth/ALHU_AEMuGMbX68GG-Bzpru41yEIE-L7ks5t8YZ7gaJpZM4UlGvg
.

—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/clulab/eidos/pull/333#issuecomment-397101275, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AIRxOrU4G-0dlCAZhmZCMyuOsTH7qCzKks5t8YstgaJpZM4UlGvg
.

Ajay,
Can you please solve these conflicts?
On Wed, Jun 13, 2018 at 3:09 PM Keith Alcock<EMAIL_ADDRESS>wrote:

Either way as long as you coordinate with others who are making changes.
This PR already has conflicts which I won't know how to resolve.
On Wed, Jun 13, 2018, 11:59 PM Fan-Luo<EMAIL_ADDRESS>wrote:

For 'nAtuRAl ReSouRceS’ and misspelled ‘malaraia’, they are both from
where Becky originally get for me.
I think it was from 'kwalcock-ontology branch' , according to the email
Becky sent: 'Working from your kwalcock-ontology branch’.
I only extend several nodes, I did not do spelling check, etc.
By the way, I saw Ajay previously replied corrected ‘malaraia’, but it is
still there in master, do you want me to correct it?
If yes, @Keith, do you want me to create another branch to make this
change, instead of committed to master?

On Jun 13, 2018, at 2:39 PM, Keith Alcock<EMAIL_ADDRESS>wrote:
I see that master's indentation was possibly fixed, but I don't know
whether or not Ajay's changes were included at all.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub <
https://github.com/clulab/eidos/pull/333#issuecomment-397096316>, or
mute
the thread <

https://github.com/notifications/unsubscribe-auth/ALHU_AEMuGMbX68GG-Bzpru41yEIE-L7ks5t8YZ7gaJpZM4UlGvg

.

—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/clulab/eidos/pull/333#issuecomment-397101275, or
mute
the thread
<
https://github.com/notifications/unsubscribe-auth/AIRxOrU4G-0dlCAZhmZCMyuOsTH7qCzKks5t8YstgaJpZM4UlGvg
.

—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/clulab/eidos/pull/333#issuecomment-397103749, or mute
the thread
https://github.com/notifications/unsubscribe-auth/ABH-zvnO96I3Fhgee-xUCUrMThiAqrUzks5t8Y2ZgaJpZM4UlGvg
.

not for malaria, but the weirdly cased natural resources I think was one of
the "high-freq" guys ajay merged in bc it appeared in UN reports (oddly
enough) -- though perhaps I'm mis-remembering?
On Wed, Jun 13, 2018 at 3:29 PM, Mihai Surdeanu<EMAIL_ADDRESS>wrote:

Ajay,
Can you please solve these conflicts?
On Wed, Jun 13, 2018 at 3:09 PM Keith Alcock<EMAIL_ADDRESS>wrote:

Either way as long as you coordinate with others who are making changes.
This PR already has conflicts which I won't know how to resolve.
On Wed, Jun 13, 2018, 11:59 PM Fan-Luo<EMAIL_ADDRESS>wrote:

For 'nAtuRAl ReSouRceS’ and misspelled ‘malaraia’, they are both from
where Becky originally get for me.
I think it was from 'kwalcock-ontology branch' , according to the email
Becky sent: 'Working from your kwalcock-ontology branch’.
I only extend several nodes, I did not do spelling check, etc.
By the way, I saw Ajay previously replied corrected ‘malaraia’, but it
is
still there in master, do you want me to correct it?
If yes, @Keith, do you want me to create another branch to make this
change, instead of committed to master?

On Jun 13, 2018, at 2:39 PM, Keith Alcock<EMAIL_ADDRESS>wrote:
I see that master's indentation was possibly fixed, but I don't know
whether or not Ajay's changes were included at all.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub <
https://github.com/clulab/eidos/pull/333#issuecomment-397096316>, or
mute
the thread <

https://github.com/notifications/unsubscribe-auth/ALHU_AEMuGMbX68GG-
Bzpru41yEIE-L7ks5t8YZ7gaJpZM4UlGvg

.

—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/clulab/eidos/pull/333#issuecomment-397101275, or
mute
the thread
<
https://github.com/notifications/unsubscribe-auth/AIRxOrU4G-
0dlCAZhmZCMyuOsTH7qCzKks5t8YstgaJpZM4UlGvg
.

—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/clulab/eidos/pull/333#issuecomment-397103749, or
mute
the thread
<https://github.com/notifications/unsubscribe-auth/ABH-zvnO96I3Fhgee-
xUCUrMThiAqrUzks5t8Y2ZgaJpZM4UlGvg>
.

—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/clulab/eidos/pull/333#issuecomment-397108439, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AFIniYW1xxyvHTMBwOTFBIt6y78SscM4ks5t8ZJDgaJpZM4UlGvg
.

I do remember now that 'nAtuRAl ReSouRceS’ is a repeat issue.  Sorry.  I should add a comment about that.

For some reason, I am not seeing any conflict when I pull @kwalcock's changes ? But I see this branch has conflicts on github.

This is merged without any guarantees.  It needed to be done.

@kwalcock @ajaynagesh @MihaiSurdeanu
what was the final status of this?  Do things need to be fixed/changed or are we fine?
thanks!

I don’t know. Ajay?

On Jun 15, 2018, at 3:37 PM, bsharpataz<EMAIL_ADDRESS>wrote:
@kwalcock @ajaynagesh @MihaiSurdeanu
what was the final status of this? Do things need to be fixed/changed or are we fine?
thanks!
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.

It was fixed/changed and merged.  About fine I can't say much.

@MihaiSurdeanu @bsharpataz  These changes look fine to me when I compare it with the version I have. I think there was an issue in the indentation of some nodes from a previous version due to a commit by someone directly on the master, which @kwalcock fixed in this change.

Great, thanks all!
On Sat, Jun 16, 2018 at 8:48 AM Ajay Nagesh<EMAIL_ADDRESS>wrote:

@MihaiSurdeanu https://github.com/MihaiSurdeanu @bsharpataz
https://github.com/bsharpataz These changes look fine to me when I
compare it with the version I have. I think there was an issue in the
indentation of some nodes from a previous version due to a commit by
someone directly on the master, which @kwalcock
https://github.com/kwalcock fixed in this change.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/clulab/eidos/pull/333#issuecomment-397821133, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AFInid_Toe3ezzKnPlAUr6zBSKu0OI4tks5t9Si9gaJpZM4UlGvg
.

Thanks for making this clear.
The only commit I made to master actually is to fix indentation.
I think Keith redid some of the work, and it is because of my miscommunication, which caused more work and unnecessary discussion.
I should also did data sanitization or at least pointed it out.
Sorry and thanks to Keith for fixed everything.
Fan

On Jun 16, 2018, at 9:30 AM, bsharpataz<EMAIL_ADDRESS>wrote:
Great, thanks all!
On Sat, Jun 16, 2018 at 8:48 AM Ajay Nagesh<EMAIL_ADDRESS>wrote:

@MihaiSurdeanu https://github.com/MihaiSurdeanu @bsharpataz
https://github.com/bsharpataz These changes look fine to me when I
compare it with the version I have. I think there was an issue in the
indentation of some nodes from a previous version due to a commit by
someone directly on the master, which @kwalcock
https://github.com/kwalcock fixed in this change.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/clulab/eidos/pull/333#issuecomment-397821133, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AFInid_Toe3ezzKnPlAUr6zBSKu0OI4tks5t9Si9gaJpZM4UlGvg
.

—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub https://github.com/clulab/eidos/pull/333#issuecomment-397823832, or mute the thread https://github.com/notifications/unsubscribe-auth/ALHU_IC9-mpUeZcbRtlCRReVnt2Jdgnaks5t9TKkgaJpZM4UlGvg.


Only show icon and character if relevant attachments exist

Due to the way we handle retrieving reply and post associations, if the ID for the association exists but the relevant association no longer does (eg an icon is deleted), the code would previously expect it to exist and hence display an icon.
This ensures the actual attached data is loaded (eg icon URL, character name) and if not then it assumes the association doesn't exist and so doesn't try to show that data.

This also now clears user avatars that reference the ID.

And clears character IDs when characters are destroyed. (This caused issues including breaking the reply editor if the latest reply in a post used to use the character you just deleted.)


ValueError: data must be 2 dimensional

Hi @kapoorlab ,
I saw your announcement in the image.sc forum, and the combination of napari + TrackMate sounds just perfect for my purposes. Thank you for sharing the code and providing a pip package!
For a quick test, I started with your example notebook and changed the path_xml and SegmentationImage paths according to my file locations and set RawImage as well as MaskImage to None.
When I now continue in the notebook, it correctly creates the "Tracklets", but once it starts to compute them for TrackID 0 the following error message appears:
---------------------------------------------------------------------------
ValueError                                Traceback (most recent call last)
<ipython-input-23-8a4f94a8c1bd> in <module>
----> 1 RegionID, VolumeID, locationID, Tracks, ID, StartID = TM.ImportTrackmateXML(
      2     path_xml,Seg,
      3     image = Raw,
      4     Mask = Mask,
      5     XYcalibration = XYcalibration,

C:\ProgramData\Anaconda3\envs\napatrackmater\lib\site-packages\napatrackmater\Trackmate.py in ImportTrackmateXML(xml_path, Segimage, XYcalibration, Zcalibration, Tcalibration, image, Mask, mintracklength)
    535
    536                                                    TwoDCoordinates = [(prop.centroid[1] , prop.centroid[2]) for prop in properties]
--> 537                                                    TwoDtree = spatial.cKDTree(TwoDCoordinates)
    538                                                    TwoDLocation = (y ,x)
    539                                                    closestpoint = TwoDtree.query(TwoDLocation)

ckdtree.pyx in scipy.spatial.ckdtree.cKDTree.__init__()

ValueError: data must be 2 dimensions

In trackmate I used a series of 3D images. You can find the trackmate xml file as well the segmentation results at:
https://datashare.mpcdf.mpg.de/s/AtEPeYkTtdlnBOi

Ah I see, thanks for trying the package and sending me the data. In this notebook it is expecting to find an image of shape TZYX so you would have to stitch your timepoints to make one large file which then corresponds to the xml file that you get from TrackMate. Then it should work as expected.
Maybe you could make a cell in the notebook where it creates a numpy array after reading the sequence of images from the folder? If you already did that then this problem could be coming from the stacks where there are no cells hence TwoDCoordinates is just None. I will build an exception to catch that and let you know when the new package is available with that change. Thanks again.

Ah I see, thanks for trying the package and sending me the data. In this notebook it is expecting to find an image of shape TZYX so you would have to stitch your timepoints to make one large file which then corresponds to the xml file that you get from TrackMate. Then it should work as expected.
Maybe you could make a cell in the notebook where it creates a numpy array after reading the sequence of images from the folder? If you already did that then this problem could be coming from the stacks where there are no cells hence TwoDCoordinates is just None. I will build an exception to catch that and let you know when the new package is available with that change. Thanks again.

[..] In this notebook it is expecting to find an image of shape TZYX so you would have to stitch your timepoints to make one large file which then corresponds to the xml file that you get from TrackMate. Then it should work as expected.

Thank you for your quick reply, I scrolled a little in your code and relized this as well. I created a numpy array with a 2x (x, y, z) downscaled version of the segmentation results (I took the same approach to reduce the memory footprint in TrackMate).
Now the Tracklets are correctly calcualted!
Unfortunately, another error occured:
---------------------------------------------------------------------------
AttributeError                            Traceback (most recent call last)
<ipython-input-4-8a4f94a8c1bd> in <module>
     11 scale = [1,2,1,1]
     12 totaltracks = len(Tracks)
---> 13 TM.TrackMateLiveTracks(Raw, None, Mask,savedir, scale, locationID, RegionID, VolumeID, ID, StartID, Tcalibration)

~\miniconda3\envs\napatrackmater\lib\site-packages\napatrackmater\Trackmate.py in TrackMateLiveTracks(Raw, Seg, Mask, savedir, scale, locationID, RegionID, VolumeID, ID, StartID, Tcalibration)
    934         UpdateMask = Mask
    935
--> 936     Boundary = GetBorderMask(UpdateMask.copy())
    937
    938     with napari.gui_qt():

AttributeError: 'NoneType' object has no attribute 'copy'

Best wishes,
Eric

[..] In this notebook it is expecting to find an image of shape TZYX so you would have to stitch your timepoints to make one large file which then corresponds to the xml file that you get from TrackMate. Then it should work as expected.

Thank you for your quick reply, I scrolled a little in your code and relized this as well. I created a numpy array with a 2x (x, y, z) downscaled version of the segmentation results (I took the same approach to reduce the memory footprint in TrackMate).
Now the Tracklets are correctly calcualted!
Unfortunately, another error occured:
---------------------------------------------------------------------------
AttributeError                            Traceback (most recent call last)
<ipython-input-4-8a4f94a8c1bd> in <module>
     11 scale = [1,2,1,1]
     12 totaltracks = len(Tracks)
---> 13 TM.TrackMateLiveTracks(Raw, None, Mask,savedir, scale, locationID, RegionID, VolumeID, ID, StartID, Tcalibration)

~\miniconda3\envs\napatrackmater\lib\site-packages\napatrackmater\Trackmate.py in TrackMateLiveTracks(Raw, Seg, Mask, savedir, scale, locationID, RegionID, VolumeID, ID, StartID, Tcalibration)
    934         UpdateMask = Mask
    935
--> 936     Boundary = GetBorderMask(UpdateMask.copy())
    937
    938     with napari.gui_qt():

AttributeError: 'NoneType' object has no attribute 'copy'

Best wishes,
Eric

Hi Eric,
Thanks for pointing this out too, for now to avoid this could you please these lines in the notebook
'''
if MaskImage is not None:
Mask = imread(MaskImage)
else:
Mask = None
'''
to the lines below where you provide a rectangular region as a mask excluding 50 (or some other value) pixels around the border.
'''
if MaskImage is not None:
Mask = imread(MaskImage)
else:
Mask = None
Mask = np.zeros_like(Seg)
Mask[:,:, 50:Mask.shape[2]-50, 50:Mask.shape[3]-50] = 1
'''
I will update the code to incorporate these two issues very soon.

Hi Eric,
Thanks for pointing this out too, for now to avoid this could you please these lines in the notebook
'''
if MaskImage is not None:
Mask = imread(MaskImage)
else:
Mask = None
'''
to the lines below where you provide a rectangular region as a mask excluding 50 (or some other value) pixels around the border.
'''
if MaskImage is not None:
Mask = imread(MaskImage)
else:
Mask = None
Mask = np.zeros_like(Seg)
Mask[:,:, 50:Mask.shape[2]-50, 50:Mask.shape[3]-50] = 1
'''
I will update the code to incorporate these two issues very soon.

It works now! Thank you for your help!
Two comments:
Mask[:,:, 50:Mask.shape[2]-50, 50:Mask.shape[3]-50] = 1

can be written
Mask[:,:, 50:-50, 50:-50] = 1

and I used
RegionID, VolumeID, locationID, Tracks, ID, StartID = TM.ImportTrackmateXML(
    path_xml,Seg,
    image = Raw,
    Mask = None,
    XYcalibration = XYcalibration,
    Zcalibration = Zcalibration
    , Tcalibration = Tcalibration,
    mintracklength = mintracklength)

Because otherwise the import of the xml file takes quite long (I did not wait until it finished).
Best wishes,
Eric

It works now! Thank you for your help!
Two comments:
Mask[:,:, 50:Mask.shape[2]-50, 50:Mask.shape[3]-50] = 1

can be written
Mask[:,:, 50:-50, 50:-50] = 1

and I used
RegionID, VolumeID, locationID, Tracks, ID, StartID = TM.ImportTrackmateXML(
    path_xml,Seg,
    image = Raw,
    Mask = None,
    XYcalibration = XYcalibration,
    Zcalibration = Zcalibration
    , Tcalibration = Tcalibration,
    mintracklength = mintracklength)

Because otherwise the import of the xml file takes quite long (I did not wait until it finished).
Best wishes,
Eric


Mapping TTPs and CVEs

Hi! I am new here!
Is there any way to link TTPs and CVEs through the Caldera?
Through the Pathfinder plugin I have received the network topology including the CVES, and then I have created an operation using as adversary the Thief.
Once the operation is done, I use the Debrief plugin to see how the operation is completed and then I can see the used TTPs.
Is there any way to export which CVEs are used/linked to each used TTP?
Thanks in advance!

You can tag abilities with a CVE or any other piece of information as seen here: https://github.com/center-for-threat-informed-defense/caldera_pathfinder/blob/master/data/abilities/initial-access/315f8fcc-c05a-4db0-9f9a-5daade661540.yml#L11-L12.
Hopefully this resolves your issue. If you’re still having any problems, please feel free to re-open or start a new ticket. Thanks!


wip/6.0 Implement @Filter for loader

https://trello.com/c/RQxAqQaT/104-loaderselectbuilder-implement-use-of-loadqueryinfluencer
The only missing task for this card is the implementation of collection filter feature for filtering should not be enforced for entity filtering in loader.
The gist of this PR is to avoid parsing filter condition. The main difficulty is how to pass filter parameter into JdbcSelect.
After this card is done, I will work on implementing @Filter for HQL or Criteria (card created at https://trello.com/c/KLZjWiX0/118-filter-support-for-hql-criteria). However, this PR will lay solid ground for later endeavour.
Test case included.

In the Hibernate user manual: https://docs.jboss.org/hibernate/orm/6.0/userguide/html_single/Hibernate_User_Guide.html#pc-filter mentioned [Filters apply to entity queries, but not to direct fetching.] , But I very much hope to support it in a future version. I now want to try to develop a multi-tenant model based on data table columns, where I have encountered a bottleneck. https://github.com/vip-efactory/ejpa-example/tree/br_tenant_discriminator, I can only give up temporarily.

In the Hibernate user manual: https://docs.jboss.org/hibernate/orm/6.0/userguide/html_single/Hibernate_User_Guide.html#pc-filter mentioned [Filters apply to entity queries, but not to direct fetching.] , But I very much hope to support it in a future version. I now want to try to develop a multi-tenant model based on data table columns, where I have encountered a bottleneck. https://github.com/vip-efactory/ejpa-example/tree/br_tenant_discriminator, I can only give up temporarily.

Could you work around the limitation by using HQL instead? Guess there is good reason to ignore filter for loader, but HQL works.

In the Hibernate user manual: https://docs.jboss.org/hibernate/orm/6.0/userguide/html_single/Hibernate_User_Guide.html#pc-filter mentioned [Filters apply to entity queries, but not to direct fetching.] , But I very much hope to support it in a future version. I now want to try to develop a multi-tenant model based on data table columns, where I have encountered a bottleneck. https://github.com/vip-efactory/ejpa-example/tree/br_tenant_discriminator, I can only give up temporarily.

Could you work around the limitation by using HQL instead? Guess there is good reason to ignore filter for loader, but HQL works.

Okay, thank you!

Just realized I overlooked ManyToMany filtering scenario. Will open again when it is implemented.

applied upstream, Thanks @NathanQingyangXu


implement core::error::Error

this trait has been stabilised in Rust 1.81.0.
the existing custom Error types cannot be removed / replaced as that'd be a breaking change. for the same reason it's not possible for them to require core::error::Error being implemented.
however, we can already implement the new trait for all cases where the custom trait has been implemented so far.
existing std feature-gated implementations of std::error::Error have also been moved to core::error::Error and the feature gate removed.
this raises the MSRV to 1.81.0 for most crates, but based on the MSRV policy this should not be an issue.

see #629 for an alternative approach which however doesn't seem to work (but would be much more concise if it did!)

How does the error trait fit with defmt? Is it possible to use it properly while still leveraging the greatness of defmt?

Yeah, I get that. The question was more on whether there is a way to utilize the error trait in a defmt application, or if that would always end up being all device side formatting?

2 weeks have passed and no one from the HAL team has raised concerns, let's merge.


v1.7 backports 2020-02-18

#10211 -- test: Fix getNodeInfo in NodePort tests (@borkmann)
#10208 -- update cilium-runtime with golang 1.13.8 (@aanm)
#10215 -- update k8s dependencies to 1.17.3 (@aanm)
#10219 -- [CI] Mark TLS policy test as pending (@nebril)
#10214 -- test: fix ClusterIP IPv6 connectivity checks (@borkmann)
#10207 -- Fix regression to avoid freeing alive IPs (@tgraf)
#10218 -- cilium: encryption + tunnel improvements and fixes (@jrfastab)

Once this PR is merged, you can update the PR labels via:
$ for pr in 10211 10208 10215 10219 10214 10207 10218; do contrib/backporting/set-labels.py $pr done 1.7; done

This change is

never-tell-me-the-odds

test-missed-k8s


Functional update optimization is not performed

According to the KaRaMeL user manual, the following repro should be compiled to a C struct mutation x->fld_a = 1:
module Main
module B = LowStar.Buffer
module U32 = FStar.UInt32
open LowStar.BufferOps
open FStar.HyperStack.ST

type test_struct = { fld_a : U32.t; fld_b : U32.t }

let test_set_field (x : B.pointer test_struct)
  : Stack unit (requires fun h -> B.live h x) (ensures fun _ _ _ -> True)
  =
    x.(0ul) <- { x.(0ul) with fld_a = 1ul }

However krml (70084646) generates the following output:
// Main.c: $KRML_HOME/krml -skip-compilation Main.fst -bundle Main=*
#include "Main.h"

void Main_test_set_field(Main_test_struct *x)
{
  x[0U] = ((Main_test_struct){ .fld_a = (uint32_t)1U, .fld_b = x->fld_b });
}

Replacing the buffer read and update with the pointer operators !* and *= does not trigger the optimization either.

The optimization is performed if one writes:
let xv = x.(0ul) in
x.(0ul) <- { xv with fld_a = 1ul }

Using !* or *= inhibits it.

So it turns out that this optimization disappeared because of another optimization broke its pattern-matching. I re-instated the optimization to begin with so that this now works:
let f (p: B.buffer point): Stack unit
  (requires (fun h -> B.live h p /\ B.length p = 1))
  (ensures (fun _ _ _ -> True))
=
  p.(0ul) <- ({ p.(0ul) with x = 0l })

I need to extend the pattern-matching to recognize the version with the short operators.

https://github.com/FStarLang/karamel/pull/255/commits/9ed0675913766d1b97442c5d5c60eeab8e2f618a works for nearly all the examples I have.
However I encountered this example for which the optimization is not performed:
type test_struct = { fld_a : U32.t; fld_b : U32.t; fld_c : U32.t }
noeq type indir = { ind_ref : B.pointer test_struct; other : U32.t }

let test_set_field2 (r : B.pointer indir)
  : Stack unit (requires fun h -> B.live h r /\ B.live h (B.deref h r).ind_ref) (ensures fun _ _ _ -> True)
  =
    (r.(0ul)).ind_ref.(0ul) <- { (r.(0ul)).ind_ref.(0ul) with fld_a = 1ul }

krml outputs:
Main_test_struct uu____0 = r->ind_ref[0U];
r->ind_ref[0U] =
   ((Main_test_struct){ .fld_a = (uint32_t)1U, .fld_b = uu____0.fld_b, .fld_c = uu____0.fld_c });

whereas it outputs r->ind_ref->fld_a = 1 for a structure with only two fields.
One can let-bind the updated pointer:
let p = (r.(0ul)).ind_ref in
p.(0ul) <- { p.(0ul) with fld_a = 1ul }

to obtain
Main_test_struct *p = r->ind_ref;
p->fld_a = (uint32_t)1U;

Thanks, I generalized the earlier optimization to take this case into account too. Your example should now work with 4adf42ec and I've updated the test file accordingly. Thanks.


fix: info type being too restrictive in handlers

Following https://github.com/accesimpot/graphql-lookahead/pull/102
The info type was too restrictive and didn't allow you to pick it from the handler arguments and pass it to lookahead.

:tada: This PR is included in version 1.3.1-beta.2 :tada:
The release is available on:

npm package (@beta dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Cannot build using conda-forge

I try to build dnpy using conda-forge using WSL Ubuntu. Error while generating dnpy.cpp and getting error 'error while parsing header'
I attach my make log
file.txt
Thanks.

@manvisdr I will take a look at this soon.  I need to update everything to use latest Cppyy version and OpenDNP3 library anyway.

Conda Forge and WSL may be the issue.  I've had pathing problems in that configuration before.  Have you tried building from source?  Also, the code in this repo was built on 1.8 of Cppyy.  I have not verified it will work with later versions.


Add test script and tidy import

Add way to easily test integration python test_chat.py "Your question here"
Remove redundant requirement (I removed util function that I guess was boilerplate)
Validate file existing before loading

Fixed the merge conflict - it's ready :+1:


eu-fr2: Add cf, cr and schematics endpoints

To be able to provision these services in the new eu-fr2 region
we need to define them inside the mapping table.

We have requirement from customer that we should not expose the endpopints for eu-fr2 region in any open source

Alright, makes sense . 👌


Fix chef on macOS

Summary: This fixes chef runs on macOS
Type of change: /kind bug
Test Plan: Ran chef on a macOS machine. Ensured it ran to completion.

test this please

test this please


Minor enhancement for stability

Sometimes the model (Qwen2.5:72b) returns an empty list when extracting entities and relationships. The attached patch allows you to avoid program crashes and skip invalid records.

operate.txt

Thanks for your contribution! In the next commit, I will include your patch to handle this issue


The volume calculation per segmentation is given in mm³ right?

I didn't find any mention of the actual calculation of the volume size for each of the segmented part of the brain. I looked through the code and I believe it to be reported in mm³? Would be cool to have this info in the docstrings and the --help part of the script, as well as the readme.md.

Done.
Good catch, thanks :)


docs(releases): workflow releases happen monthly

Workflow has grown into a mature and stable product, and the team at Deis thinks doing a minor release every two weeks may not be productive. This proposes to change the Deis Workflow release policy to "first Thursday of each month," but the actual change is dependent on your feedback. So please comment here as to whether or not that's a useful release policy for you!
See also deis/seed-repo#48.

Spelling/grammar LGTM.

I think monthly minor releases are fine.
There can always be patch releases if there are important issues that need to be fixed quickly and users that are eager to try out new features can switch to a newer ci image.

I agree with @felixbuenemann if there is some major patch release, yes then we should an extra monthly release.

if there is some major patch release, yes then we should an extra monthly release.

Agreed, and I kept that distinct from the monthly feature-oriented releases:

Patch releases are created at any time, as needed.


Add Oturin's file

much better content

thanks for introduction.
VO.

thanks for introduction.
VO.


Update useChain.ts

name: "Pull request"
about: A new pull request
New Pull Request
Checklist

[x] I am not disclosing a vulnerability.
[x] My code is conform the code style
[x] I have made corresponding changes to the documentation
[x] I have updated Typescript definitions when needed

Issue Description

New chains can't be added to the Metamask. blockExplorerUrl is not exported from chains
Fixes error message:
{code: -32602, message: "Expected null or array with at least one valid string HTTPS URL 'blockExplorerUrl'. Received: ", stack: '{\n  "code": -32602, message: "Expected null or array with at least one valid string HTTPS URL 'blockExplorerUrl'. Received: ")'}

Solution Description

Import infoURL as a blockExplorerUrl

@Y0moo For what chain is this error occurring?
infoURL is different than blockExplorerUrl.
The blockExplorerUrls are defined in https://github.com/MoralisWeb3/react-moralis/blob/main/src/data/blockExplorers.ts and are appended to the chainData

@ErnoW
I added fantom and changed explorer for Mumbai. Also deleted infoURL import

@Y0moo pipeline was failing because of a formatting issue.
Make sure to have the Prettier extension installed for automatic formatting.
I will look into auto-formatting in the CI later on, but for now, it is something to be aware of to check if everything is formatted and no es-lint errors are present.
One missing comma and your PR will be rejected 😅 (I know it is a bit harsh)
For now, I fixed it already. Nice work 👍


feat: migrate to<EMAIL_ADDRESS>after fixed regex

Migrate to<EMAIL_ADDRESS>after fixed regex

Codecov Report

Merging #182 (fda0c43) into develop (9f4fbd2) will decrease coverage by 0.01%.
The diff coverage is 69.88%.

@@             Coverage Diff             @@
##           develop     #182      +/-   ##
===========================================
- Coverage    73.70%   73.69%   -0.02%
===========================================
  Files           24       24
  Lines         5321     5295      -26
  Branches       493      480      -13
===========================================
- Hits          3922     3902      -20
+ Misses        1383     1374       -9
- Partials        16       19       +3

Impacted Files
Coverage Δ

packages/light-godwoken/src/LightGodwokenV0.ts
37.11% <33.33%> (+0.81%)
:arrow_up:

packages/light-godwoken/src/LightGodwokenV1.ts
35.09% <46.66%> (-0.43%)
:arrow_down:

...s/light-godwoken/src/godwoken/godwokenScannerV1.ts
18.18% <50.00%> (ø)

packages/light-godwoken/src/sudtFaucet.ts
66.08% <56.25%> (+1.74%)
:arrow_up:

packages/light-godwoken/src/lightGodwoken.ts
56.37% <56.84%> (-0.57%)
:arrow_down:

...ckages/light-godwoken/src/lightGodwokenProvider.ts
75.73% <81.81%> (-1.86%)
:arrow_down:

...ckages/light-godwoken/src/constants/configUtils.ts
100.00% <100.00%> (ø)

...ight-godwoken/src/constants/lightGodwokenConfig.ts
100.00% <100.00%> (ø)

packages/light-godwoken/src/lightGodwokenType.ts
100.00% <100.00%> (ø)

packages/light-godwoken/src/schemas/codecLayer1.ts
100.00% <100.00%> (ø)

... and 7 more

Continue to review full report at Codecov.

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

Currently I'm working on a refactor task, maybe it will affect your content:
https://github.com/nervosnetwork/light-godwoken/tree/ref-node-env
Changes in the branch:

Remove isMainnet variable from the package
Replace Metamask Ethereum object with EthereumProvider class
Move bridged tokens relevant content to a specific folder
Move LightGodwokenConfig relevant content to a specific folder


initial performance test

fixes #7437
A single perf test that can be run as a launch configuration, or through npm run testPerformance
It specifies the entire filename for Mocha's filename suffix option so that we can only run one test per launch of vscode. That way we can test perf from start to finish and isolate each test.

looks like I broke the actions configuration

Codecov Report

Merging #10025 (0d58cba) into main (580dd68) will increase coverage by 0%.
The diff coverage is n/a.

:exclamation: Current head 0d58cba differs from pull request most recent head 5528bed. Consider uploading reports for the commit 5528bed to get more accurate results

@@          Coverage Diff           @@
##            main   #10025   +/-   ##
======================================
  Coverage     63%      63%
======================================
  Files        211      211
  Lines      10090    10097    +7
  Branches    1610     1611    +1
======================================
+ Hits        6442     6450    +8
+ Misses      3131     3130    -1
  Partials     517      517

Impacted Files
Coverage Δ

src/platform/common/utils/misc.ts
69% <0%> (-3%)
:arrow_down:

src/platform/common/platform/fs-paths.ts
84% <0%> (ø)

src/platform/common/utils/localize.ts
98% <0%> (+<1%)
:arrow_up:

src/platform/api/pythonApi.ts
69% <0%> (+<1%)
:arrow_up:

src/platform/errors/errorHandler.ts
63% <0%> (+<1%)
:arrow_up:


Caution (in docs) against dispatching redux actions or triggering component updates by any other means in componentWillUpdate

(moving from https://github.com/facebook/react/pull/9694)

I believe this corresponds with issue #74 😄

Thanks!


Copy Automatic actions from one project to another

Hello,
Playing for a few weeks now with KB and very pleased with it.
I especially like to possibility to assign colors to task categories using automatic actions.
Unfortunately, I haven't found a way to copy a set of actions to another project. Re-creating all actions for all project is a lot of work and a bit annoying...
Maybe could other user find interesting to have, on the automatic action page, an option to copy actions from another project.
Best regards,
Michel

You can duplicate the project where you've defined the actions that you want. (Settings --> Duplicate)

With the last development branch, you can import automatic actions from another project.


cant connect to metaschool

What version of Wallet are you using?
0.40.1
What is happening?
trying to connect to metaschool.so and says error connecting wallet please ensure wallet is installed and set up correctly .. it is set up and installed because i connected to your guild
steps to reproduce
go to metaschool.so try connect wallet to claim prize ive connected metamask and sui wallet but fuel wallet wont connect
How it should behave?
it should connect to my fuel wallet
What os are you seeing the problem on?
Windows Desktop
What browser?
Chrome
What is your browser version?
130.0.6723.70 (Official Build) (64-bit)

@maggot24977 It sounds like you're having some trouble connecting your wallet to metaschool.so. Since you mentioned that your wallet is set up and installed because you connected to another platform, it might be a compatibility issue or a temporary glitch.I'll have to refer you to the official support live chat with the ticket ID MET07638
Please see the link below to our dedicated support line:
Metaschool Network DApps Support Ticket Requests.
Click on the live chat icon at the bottom corner of the page to initiate chat with a live support agent.

@maggot24977 It sounds like you're having trouble connecting your wallet to metaschool.so even though it's set up correctly. Since you were able to connect to the guild, it might be a temporary issue with metaschool or has it been solved?

still not working
On Wed, Nov 6, 2024 at 12:22 PM Alexahalid @.***> wrote:

@maggot24977 https://github.com/maggot24977 It sounds like you're
having trouble connecting your wallet to metaschool.so even though it's set
up correctly. Since you were able to connect to the guild, it might be a
temporary issue with metaschool or has it been solved?
—
Reply to this email directly, view it on GitHub
https://github.com/FuelLabs/fuels-wallet/issues/1631#issuecomment-2458514781,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/BLW2PLYMWKZ55MHHL6RHU5DZ7FVL5AVCNFSM6AAAAABQVRM2ZOVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDINJYGUYTINZYGE
.
You are receiving this because you were mentioned.Message ID:
@.***>

It sounds like your query would be best dealt with by the support team.
I'll have to refer you to the official support live chat with the ticket ID FUE07638
Please see the link below to our dedicated support line:
fuel wallet Network DApps Support Ticket Requests.
Click on the live chat icon at the bottom corner of the page to initiate chat with a live support agent.
@maggot24977

Reach out to the support team they should be able to figure something out
@maggot24977

Hello @maggot24977
I attempted to reproduce your issue and was unable to do so.
We use GitHub issues as a place to track bugs and other development-related issues. If your issue persists, please write us back using our “Chat Support” icon located on our Fuel wallet Help Center
You can include a link to this issue in the message content.
Alternatively, you can also search for an answer in our help documentation or get help from other users on our community forums.
The issue here will be closed.
Thanks!

Hi there,
I attempted to reproduce your issue and was unable to do so.
We use GitHub issues as a place to track bugs and other development-related issues. If your issue persists, please write us back using our “Chat Support” icon located on our Fuel Wallet Help Center
You can include a link to this issue in the message content.
Alternatively, you can also search for an answer in our help documentation or get help from other users on our community forums.
The issue here will be closed.
Thanks!


[BUG] remove login link from footer section

Has this bug been raised before?

[X] I have checked "open" AND "closed" issues and this is not a duplicate

Where did you find this bug?
production
Version of BioDrop (for example "v1.2.3")
v2.100.2
Description
currently there is a login link on the footer even when the user is logged in
I think we should remove it
Steps to Reproduce
If you know the steps, follow the below format and provide steps to reproduce
For example:

go to https://www.biodrop.io/
scroll to the footer section after while you are logged in.

Screenshots

Do you want to work on this issue?
No
If "yes" to above, please explain how you would technically implement this
No response

I want to work on it if it's not assigned to anyone

I want to work on it if it's not assigned to anyone

You are assigned. Thank you for contributing

I want to work on it if it's not assigned to anyone

You are assigned. Thank you for contributing

Ok , Thank you

Can you assign this issue to @MalekAchich ?

I will gladly accept the assignment on this issue !

@amandamartin-dev Can you assign me this issue ?

@amandamartin-dev Can you assign me this issue ?

Please look at the assignee section before asking for the issue to be assigned we give contributors 2 weeks time to raise a PR and please don't tag specific maintainers.
https://github.com/EddieHubCommunity/BioDrop/blob/main/CONTRIBUTING.md

@amandamartin-dev Can you assign me this issue ?

Assinging the issue to @MalekAchich

Pull Request merged


RTSP Server

Hello @HclX, thanks for your work.
I have a small suggestion, adding optional support to the RTSP server in your hack will be awesome.
If the problem is space, you can store it on NFS.

I have been playing around with this idea but the big issue is the cpu in the camera is not super powerful. You are going to have some artifacts and stuff. I will keep trying different things and let you know the results. Also running at RTSP through the NFS is not advisable because if you lose any connection to the share, the camera could lock up or crash.

What about storing the binary on SD card if space is an issue? If theres a file on the SD, enable it? The dafang hacks seems to use https://github.com/mpromonet/v4l2rtspserver for the rtsp server.

This is a possibility. It would take a little of work but essentially we could mount the sdcard in a different location, set the parameters from the wyzehacks config file, then run the rstp server.

Some corners of the internet: https://www.reddit.com/r/wyzecam/comments/esbfe0/rtsp_firmware_and_the_nfssd_card_hack/ff92czn/?utm_source=reddit&utm_medium=web2x&context=3, claim to have applied this hack to the Wyze RTSP firmware. Is this a viable option?

I had the impression that RTSP will not work with some Wyze's built-in video feature, and that's why they have the special RTSP firmware?

I had the impression that RTSP will not work with some Wyze's built-in video feature, and that's why they have the special RTSP firmware?

That is currently correct.

I made a tool that allows to convert MP4 files from NFS to HLS stream.
HLS stream can be used instead of RTSP in most cases.
Here is the repo: https://github.com/n1ckyrush/wyzecam-hls
@alexandrepossebom

thank you @n1ckyrush I will try.


[WIP] Remove distutils from build process

Not review ready
Proof-of-concept of https://github.com/cupy/cupy/issues/4983#issuecomment-915357337
Closes #4983.

Moving to #6273


Create IG route for /par endpoint

Implemented in: https://github.com/SecureApiGateway/secure-api-gateway-ob-uk/pull/467


kb-data/focal,focal,now 2.29-2 all [installed,automatic]

Does the latest patch  works for the Ubuntu 20.04 lts too? Thank you for you answer.

I have not tested that yet. Please test it and report.

setxkbmap -layout cz -variant coder works fine for me on Ubuntu 20.04

setxkbmap -layout cz -variant coder works fine for me on Ubuntu 20.04

Great. Closing this issue.

Great. Closing this issue.


add checks to reset buttons in construction menu on close

About the PR
Fixed an issue where closing the construction menu while erase or build mode is active leaves those functions toggled and keeps certain top menu functions and interactions disabled.
Why / Balance
The issue is that with erase mode active in particular, there is no indication that erase mode is still active as the window is gone and the top menu button has toggled off. This leads to confusion as you are unable to perform certain interactions and shortcuts with no indication as to what is causing it.
Technical details
I wanted to get some feedback on this as the implementation may be poor or the issue may be intentional and just in need of some tweaking. I moved the code handling EraseButtonToggled to its own function similar to BuildButtonToggled and then added checks for both to reset them upon closing the window.
Media
https://github.com/user-attachments/assets/a53a90b9-aec2-48ff-9145-6d3c9dbb4217
Requirements

[x] I have read and am following the Pull Request and Changelog Guidelines.
[X] I have added media to this PR or it does not require an ingame showcase.

Breaking changes
No breaking changes to note.
Changelog

:cl:

fix: Build and erase modes no longer remain active upon closing the construction menu.


Suggestion for /watched

Now that I see how /watched works, it would be nice to have a subcommand for details.
I would add someone like this
/watched add Maxie "Stole my loot"
And I would run something like /watched details Maxie to see why she was added to the list.
Don't know.. it would be nice to have some sort of description.

Another suggestion is to order people for level or vocation.
I added 2 guilds to test it and I now have a huge list.
Not sure if that is possible though

Yes, this is definitely something I wanted to add, but I probably should show that reason on the watched list itself, cause there's a 2000 character limit on the message, so I would have to show it somewhere else.
The state where the watched list feature is right now is pretty basic, it was just an initial release just to have that feature, but it definitely needs a lot of work. Like maybe instead of keeping a list, sending messages when someone logs in and logs out, or a mix of both.
The task that updates it also needs optimization
So yes, definitely something I need to do, but still needs a lot of details.

I'm so excited about this project I'm thinking about learning python. lol

To expand on it, Maybe should be its own heading, but ill add here and let you decide. The Stalk and watched commands could also potentially check the experience gained and lost via the High scores. It is only limited to characters in the top 300's. Maybe once per day, or i guess on command would work better, /exp could display all discord members who are able to be tracked in high scores, and /expall could display all characters, discord and watched/stalked, to see the exp gains and losses if they are in the high scores list

I've had a few request on the exp gain too. Also a command to show the Tibia server down time. I guess some feel it hints on possible raids.
/downtime amera
NabBot: On the last server save Amera servers were off for 8 minutes.

Also, as for functionality in the discord im hosting, I have multiple guilds set off in sub channels, but a main tibia one for all people in my discord (where all level and deaths go). If there was ever a function to have different watched lists and be able to have multiple spots they show up, So i could have one 'watched' list serve as a "who is online" funtion rather than using the /online function) and Wouldn't find having a separate one or more for tracking specific guild enemies, or players of interest, and of course a msg next to the name with why on there. Right now I just have the few guilds show up in one spot so its easier to find hunting partners with friendly guilds who aren't necessarily in my disc.

@CuJo38 I edited my 'watched list' to hold every guild in Solidera, and have it setup where the list isn't reposted every check, but edited. 10x less notifications. It's close as I've gotten to a direct online list.
And I know from experience its a pain to re add new guilds, or remove non existant guilds, and having to re-add them all, so it stays in alphabetical order.
I will try to add the tracking.py in the form of a cog, of what I use for that, since there isnt a character max limit since the message is split into parts, saves the message ID's, then edits the previous messages on the next check. Renaming a couple things and loading it as a 'cog' might allow you to run multiple watched lists.

@CuJo38 I do want to add something similar to the Watched List but for /online, to show who's online from the current discord.
So exactly what /online does, but on a fixed channel, so you can always look at it instead of having to use the command.
Watched List was something that was done experimentally and I haven't been able to really define the correct implementation for it.

Finally working on this:

Implemented in be887cbcca71f5eeaa32160a8dda7f3a6dbf35fa, will be available on v1.1.0