Add angular Frontend

This PR adds Angular front-end functionality
Refactor code for tests
Cleans up code using ESlint-AirBnb
Improves on error handling and rendering to the user
Adds more tests to make sure that all functionality is tested fully

Coverage increased (+0.6%) to 98.068% when pulling a6d4d12ca9727201ea41f90dc720afea3880900d on feture-angular-frontend into eabb2922efe7f21563563c5a2713e1f51378764b on develop.

Coverage increased (+0.6%) to 98.068% when pulling a6d4d12ca9727201ea41f90dc720afea3880900d on feture-angular-frontend into eabb2922efe7f21563563c5a2713e1f51378764b on develop.

Coverage increased (+0.6%) to 98.068% when pulling 05b8153a0c4ec2ccb9b9d7e39615a91025b60cec on feture-angular-frontend into eabb2922efe7f21563563c5a2713e1f51378764b on develop.

Coverage increased (+0.6%) to 98.068% when pulling 05b8153a0c4ec2ccb9b9d7e39615a91025b60cec on feture-angular-frontend into eabb2922efe7f21563563c5a2713e1f51378764b on develop.

Coverage increased (+0.6%) to 98.068% when pulling fa4898494894498678a83d83f8d575eb2f4c8384 on feture-angular-frontend into eabb2922efe7f21563563c5a2713e1f51378764b on develop.

Coverage increased (+0.6%) to 98.068% when pulling fa4898494894498678a83d83f8d575eb2f4c8384 on feture-angular-frontend into eabb2922efe7f21563563c5a2713e1f51378764b on develop.

Coverage increased (+0.6%) to 98.068% when pulling aaee8fc1fdfd0f89e9789b449edcb4944e05d31b on feture-angular-frontend into eabb2922efe7f21563563c5a2713e1f51378764b on develop.

Coverage increased (+0.6%) to 98.068% when pulling aaee8fc1fdfd0f89e9789b449edcb4944e05d31b on feture-angular-frontend into eabb2922efe7f21563563c5a2713e1f51378764b on develop.

Coverage increased (+0.6%) to 98.068% when pulling a50b5f91f7e20be804d6d4876e8b52a00c662f0c on feture-angular-frontend into eabb2922efe7f21563563c5a2713e1f51378764b on develop.

Coverage increased (+0.6%) to 98.068% when pulling 97a183194d4941028e31d6a35e06ae481d5f8495 on feture-angular-frontend into eabb2922efe7f21563563c5a2713e1f51378764b on develop.

Coverage increased (+0.6%) to 98.068% when pulling 871500e8c9702724e55ad205ef97ff593b2e4ebf on feture-angular-frontend into eabb2922efe7f21563563c5a2713e1f51378764b on develop.


Create Demo Notebook

Create a demo Jupyter notebook to show functionality and visualizations of the turfpy.

closing this as this is fixed in #26


[Event Request] Codeunit 6500 SynchronizeWhseActivItemTrkg

We need a new Event OnSyncActivItemTrkgOnBeforeModifyAbsQty in codeunit 6500 in at the end of procedure SynchronizeWhseActivItemTrkg:
[External] SynchronizeWhseActivItemTrkg(WhseActivLine : Record "Warehouse Activity Line")
…
IF TempTrackingSpec.FINDSET THEN
REPEAT
TempTrackingSpec."Quantity (Base)" := ABS(TempTrackingSpec."Qty. to Handle (Base)");
TempTrackingSpec."Qty. to Handle (Base)" := ABS(TempTrackingSpec."Qty. to Handle (Base)");
TempTrackingSpec."Qty. to Invoice (Base)" := ABS(TempTrackingSpec."Qty. to Invoice (Base)");
//+++++++++++++++++++++++++++++++++++++++++++
OnSyncActivItemTrkgOnBeforeModifyAbsQty(VAR TempTrackingSpec);
//-------------------------------------------
TempTrackingSpec.MODIFY;
UNTIL TempTrackingSpec.NEXT = 0;
RegisterNewItemTrackingLines(TempTrackingSpec);

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 access to collectAllSource

What does this PR do?
Fix a null pointer exception (reported in #2802)
Motivation
The launcher holds a reference to the containerCollectAll source (to display containers in the agent status)
It is initialized only after seeing the first container catched by the containerCollectAll source but we try to remove inputs from it in all cases. This case would cause a NPE if containerCollectAll has not been initialized
Additional Notes
Anything else we should know when reviewing?

Codecov Report

Merging #2804 into master will increase coverage by 0.7%.
The diff coverage is 0%.

@@            Coverage Diff            @@
##           master    #2804     +/-   ##
=========================================
+ Coverage   57.12%   57.82%   +0.7%
=========================================
  Files         385      356     -29
  Lines       24746    20030   -4716
=========================================
- Hits        14135    11582   -2553
+ Misses       9673     7614   -2059
+ Partials      938      834    -104

Impacted Files
Coverage Δ

pkg/logs/input/docker/launcher.go
0% <0%> (ø)
:arrow_up:

pkg/collector/runner/runner.go
53.15% <0%> (-13.87%)
:arrow_down:

pkg/metadata/resources/resources.go
75% <0%> (-8.34%)
:arrow_down:

pkg/quantile/key.go
46.15% <0%> (-7.18%)
:arrow_down:

pkg/util/kubernetes/clustername/clustername.go
57.89% <0%> (-5.75%)
:arrow_down:

pkg/procmatch/signature.go
66.66% <0%> (-4.77%)
:arrow_down:

pkg/metrics/event.go
67.1% <0%> (-4.64%)
:arrow_down:

pkg/metrics/service_check.go
56.71% <0%> (-4.33%)
:arrow_down:

pkg/collector/corechecks/embed/jmx/runner.go
54.54% <0%> (-4.08%)
:arrow_down:

pkg/collector/py/py.go
76.59% <0%> (-3.67%)
:arrow_down:

... and 238 more

6.8.1 has already been released, moving to milestone to 6.8.2


Consider logging a warning when the single backtick content is not an identifier and is not followed by a special key

Description:

Currently we don't log a warning when the single backtick content is not an identifier when it is not followed by a special key.
There are several places in the repo where documentation is used in a such a way. Need to check this against the spec and clarify.
if it is invalid we can change combineAndCreateBacktickContentToken method name in DocumentationParser to combineAndCreateInvalidBacktickContentToken as suggested in https://github.com/ballerina-platform/ballerina-lang/pull/27584#discussion_r559955161

Slack reference at the time we introduced new documentation parsing imitating the existing logic with ANTLR based parsing: https://ballerina-platform.slack.com/archives/CAXS256RX/p1594886827449800

When there is no special key before the backtick block, it should be a code reference. Hence closing the issue.

When there is no special key before the backtick block, it should be a code reference. Hence closing the issue.


DynaTech only supports 1.19.x with build 166

Description

Reason

hey not sure if this helps but dev 1015 can also be directly replaced with RC 32 if you want it to be more consistent :D

Good to know thank you :)

how to use the Item Bands i dont know how to applied to armor or weapons XD


Luigi atomic write fails with pandas.DataFrame.to_pickle() in pandas 0.25.0

It seems in pandas 0.25.0 that when to_pickle() is called, it closes the file before Luigi does. This throws the following error:
Traceback (most recent call last):
  File "c:\python37\lib\site-packages\luigi\worker.py", line 199, in run
    new_deps = self._run_get_new_deps()
  File "c:\python37\lib\site-packages\luigi\worker.py", line 141, in _run_get_new_deps
    task_gen = self.task.run()
  File "C:\git\chess-pipeline\chess_pipeline.py", line 81, in run
    df.to_pickle(f, compression=None)
  File "c:\python37\lib\site-packages\luigi\target.py", line 349, in __exit__
    return super(AtomicLocalFile, self).__exit__(exc_type, exc, traceback)
  File "c:\python37\lib\site-packages\luigi\target.py", line 329, in close
    self.move_to_final_destination()
  File "c:\python37\lib\site-packages\luigi\local_target.py", line 40, in move_to_final_destination
    os.rename(self.tmp_path, self.path)
FileNotFoundError: [WinError 2] The system cannot find the file specified: 'C:\\Users\\abcd/Temp/luigi/raw-games.pckl-luigi-tmp-910384860' -> 'C:\\Users\\abcd/Temp/luigi/raw-games.pckl'

Sample Tasks:
class SampleTask(Task):

    def output(self):
        return LocalTarget('testfile.pckl', format=Nop)

    def run(self):
        from pandas import DataFrame

        self.output().makedirs()

        df = DataFrame()

        with self.output().open('w') as db:
            df.to_pickle(db, compression=None)

@requires(SampleTask)
class SampleTask2(Task):

    def run(self):
        from pandas import read_pickle

        with self.input().open('r') as f:
            df = read_pickle(f, compression=None)

The above works in pandas 0.24.2 but not 0.25.0, all other dependencies being equal. I suspect the line that affects it is this one. Any ideas on how to fix this?

to_pickle takes a filepath, not an open file handle: https://pandas.pydata.org/pandas-docs/stable/reference/api/pandas.DataFrame.to_pickle.html. The fact that this worked in 0.24.2 is probably no fault of luigi.
try:
df.to_pickle(self.output().path, compression=None)

Oh, I see. It's a little inconsistent behavior on pandas' side, since other to_... functions accept file buffers too. I'll take a look into temporary_path() and close once I have it, thank you!

Yep, looks like this works:
class SampleTask(Task):

    def output(self):
        return LocalTarget('testfile.pckl', format=Nop)

    def run(self):
        from pandas import DataFrame

        self.output().makedirs()

        df = DataFrame()

        with self.output().temporary_path() as temp_output_path:
            print(temp_output_path)
            df.to_pickle(temp_output_path, compression=None)

@requires(SampleTask)
class SampleTask2(Task):

    def run(self):
        from pandas import read_pickle

        with self.input().open('r') as f:
            df = read_pickle(f, compression=None)

Thanks for your help!


example failing

Steps to Reproduce (for bugs)

Do a fresh install and try to start the app

Context
Trying to start the example project using redux-bootstrap
Your Environment

Version used: latest

Stack trace
ERROR in [at-loader] ./node_modules/redux-bootstrap/dts/interfaces/interfaces.d.ts:21:35
TS2694: Namespace '"/Users/x/Documents/eclipse/workspace/phoneapps/development/examples/redux-bootstrap-example/node_modules/@types/react-router-redux/index"' has no exported member 'ReactRouterReduxHistory'.

Hi, Sorry I think this is an issue caused by the latest version of the react-router and related packages...
I documented the change https://github.com/redux-bootstrap/redux-bootstrap/commit/521c1e4c8789d852c8fb4b0f4882ad3a94bdb561
I need to revisit the example to make sure it works as well. Please feel free to send a PR if you fix it. If now I will fix it when I have some time but please note that it may take a long time...

Based on your PR this issue can be closed right?

Please review the PR #86 which fixes this issue.

I will try to do a release this evening :)


Request for inclusion: yomigana-ebook - a faster tool for adding kanji readings to ebooks

Hello,
I would like to bring your attention to a tool called yomigana-ebook that I have developed. This tool efficiently adds readings for kanji in eBooks and boasts a performance that is 4x faster when compared to furigana4epub, as observed on my computer. The implementation of both tools leverages the multiprocessing library.
I kindly request considering the inclusion of this tool in the list.
Thank you for your time and consideration.
p.s. the title and content is generated from chatgpt because of my poor english.

Hi @rabbit19981023! Thank you for your proposal! I would like to add it to this list. I will include it in the next update and make it available in multiple languages. Please wait a little while!

Hi @rabbit19981023! Based on the information provided, I have published the repository. I have also translated it into multiple languages.
https://github.com/taishi-i/awesome-japanese-nlp-resources/blob/6bca8ee8550a82b147abddd7263d283ea426c239/README.md?plain=1#L26
https://github.com/taishi-i/awesome-japanese-nlp-resources/blob/6bca8ee8550a82b147abddd7263d283ea426c239/README.md?plain=1#L429
If you have a website you would like to promote and would like to be added as a contributor, please edit the following line and send a pull request. Thank you in advance.
https://github.com/taishi-i/awesome-japanese-nlp-resources/blob/6bca8ee8550a82b147abddd7263d283ea426c239/README.md?plain=1#L30

Wow, thank you for the fast update!
No, I have no personal website, it's ok.
And I found another repo which does the same thing: furiganalyse, it seems faster (about 20%) than my tool if he properly use multiprocess (tested on my computer).

Thank you for sharing the new information. I will update this information in the next update.
The information here has been updated and this issue is closed. I'm looking forward to receiving more information from you in the future. Thank you!


fastclick disables all editing when used in iframe on iOS7

This animated gif should describe the symtoms better than I can in words (give it a minute to load):

animation below:

The problem occurs when fastclick (v1.0.2) is used within an iframe on iOS.
I've created a test case here . There is nothing special about the test case, other than it is fastclick in an iframe. Use Chrome on an iPad or iPhone to reproduce the full problem.
Particularly buggy behaviour can be seen in a non-safari iOS webview e.g. Chrome on iOS 7.1.1, or custom apps e.g. cordova or phonegap.
To reproduce:

Type some text into a text field.
Then try long-pressing to move the cursor to the middle of a text field. Make sure the magnifying glass tool comes up before you move your finger.
You won't be able to enter text. You may need to attempt this a few times to repdroduce the error.

Open this page in a non-default web browser on iOS7 (e.g. Chrome or Atomic)
First, perform the actions in step 1. This seems to be a precondition.
Tap the arrows on the virtual keyboard very quickly to move quickly between fields. Move quickly to an empty text field.
Now you won't be able to enter any text at all. If you tap elsewhere to make the virtual keyboard go away, things start working again.
Again, you may have to try quite a few times to reproduce this.

These bugs seem to manifest when fastclick is used in an iframe. The second bug appears to manifest in webviews within other apps (i.e. the second issue works OK in mobile safari, but the first bug is still present).

I have exactly the same problem with ios 9.3 + cordova + iframe, its exist any solution for that?


CI builds fail on master

I created a pr which was just a text change to the readme (#281) and its build fails.

fixed by #289


fix(popover): prevent viewport from cutting off modal popover

fix(popover): prevent viewport from cutting off modal popover

:hammer_and_wrench: Type Of Change

[x] Fix
[ ] Feature
[ ] Refactoring
[ ] Documentation

:book: Description
It is possible for a modal popover to be "cut off" by the viewport if there is a large amount of content in it or the browser window is small. This change uses and intersectionObserver to detect when this happens and set maxHeight on the popover. This should be compatible with the existing maxHeight prop as this will only trigger and set the height when the dialog has crossed the viewport threshold.
This is only necessary for modal popovers as with non modal popovers you can still scroll the window.
:bulb: Context
Fixes an issue noticed with the popover during product leftbar changes.
:pencil: Checklist

[x] I have reviewed my changes

:crystal_ball: Next Steps
Try out in product
:camera: Screenshots / GIFs

:link: Sources
https://developer.mozilla.org/en-US/docs/Web/API/Intersection_Observer_API

How are the steps to reproduce it?
Im trying with contentWidth on anchor and some content on content slot.
But when I try to reduce the viewport the popover just close.

If the anchor scrolls out of view the popover will close (and this is intended)
Try popping out the window by clicking this, and then just copy pasting a bunch of content in the DOM manually

I'm not able to reproduce this either, whenever I change the viewport size the popover closes, I've tried setting the hideOnClick prop to false to prevent this.

Hello Julio!!
You could check this
http://localhost:9010/iframe.html?args=contentWidth:anchor;placement:right;maxHeight:&id=components-popover--default&viewMode=story

Im seeing this correct, I would suggest in the future remove this tippy (it also uses popper wich I dont preffer) and instead build our own popper based on floating ui maybe.

I would suggest in the future remove this tippy (it also uses popper wich I dont preffer) and instead build our own popper based on floating ui maybe.

Yep this might be a good idea for the future

Investigating broken unit tests


ALVR for SLR3.0

This is less of an issue and more of a plea for help. Steam rolled out an update for SteamVR like a day ago, now it's at 2.6, "previous version beta" is now 2.5.5, and the 2.4.4 version, the last one to work with ALVR (only with Steam-play-none compatibility, too) , got put behind an inaccessible, password-protected branch as seen on steamdb. I'd love to get ALVR working on Steam Linux Runtime 3.0 (sniper), and at least on the currently oldest available 2.5.5 version of SteamVR - I can't tell if I'm doing something wrong on my side.

https://github.com/alvr-org/ALVR/wiki/Linux-Troubleshooting
Also it's physically impossible to build alvr with any SLR version rn because of the layer.

The issue causing the current trouble isn't SLR 3, it's that the vrcompositor component of SteamVR is ran in SLR 1, which means we would have to build the vulkan layer for SLR 1, which we can't do. This is something that will be fixed by not having the layer, i.e. direct mode support. Otherwise SLR isn't really needed, it works without it, it would simply provide a minor amount of additional stability guarantees. This is something I'd do as a CI only thing, since it'd only be wrapping existing build steps.
Closing in favor of #2165


Change --passWithNoTests message to match exit code

Summary

Fixes #8187
Test plan

$ y jest blahblah --passWithNoTests
yarn run v1.15.0
$ node ./packages/jest-cli/bin/jest.js blahblah --passWithNoTests
No tests found, exiting with code 0
In /Users/jeanabayo/Side/oss/jest
  2156 files checked across 14 projects. Run with `--verbose` for more details.
Pattern: blahblah - 0 matches
✨  Done in 1.36s.

Codecov Report

Merging #8193 into master will decrease coverage by 0.02%.
The diff coverage is 20%.

@@            Coverage Diff             @@
##           master    #8193      +/-   ##
==========================================
- Coverage    62.3%   62.28%   -0.03%
==========================================
  Files         265      265
  Lines       10473    10478       +5
  Branches     2542     2543       +1
==========================================
+ Hits         6525     6526       +1
- Misses       3366     3369       +3
- Partials      582      583       +1

Impacted Files
Coverage Δ

packages/jest-core/src/getNoTestFoundVerbose.ts
0% <0%> (ø)
:arrow_up:

packages/jest-core/src/getNoTestFound.ts
40% <33.33%> (-2.86%)
:arrow_down:

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 7e4e170...3858345. Read the comment docs.

#8495


allow user list on left side in non-narrow situations

There is an option to display the user list on the left side in narrow windows. However, in my setup, even though it is not automatically triggered, I would like to be able to force this as I like it on the left with the streams (regardless of windows size). Additionally, in my setup, I do not know why the check is not evaluating to true as I have a very narrow window.
Screenshot: zulip in the background is zulip as displayed in the full width of my broswer; zulip in the front is zulip in a separate window, resized to give as much real estate to messages as possible.

hmm, yeah, I see the issue with lower resolution monitors.  We should either offer this as an option or maybe change the existing option to work that way instead

@timabbott I just opened #558 which is very close but a little different.
I'd be glad to change the user profile attribute left_side_userlist so that is has four logical values: always on the right, always on the left, on the left for narrow window, and on the right but hidden
I can look at the code myself, but a small intro would be of great help.
Thanks.

Well there are a few issues involved with doing that.  The boring one is that we'd need to change the type of the field to be an enum or something; I imagine we can work that out.
If you run git grep left_side_userlist static/, you'll see that we only access that variable in ~10 places in the frontend code, and that's including the code for updating the setting (which is most of it).  The right sidebar itself has a small shell templates/zerver/right-sidebar.html but the user list content is primarily generated by frontend templates, primarily static/templates/user_presence_rows.handlebars, and so is pretty easy to control and place dynamically.
What gets complicated is the logic around how big to make everything in the different cases; you should read resize_page_components in static/js/resize.js.
Let me know if more info is helpful!

I'm starting with the auto_hide option.
My quick and simple solution would be to take all the styling found in static/styles/zulip.css under the  @media (max-width: 975px) section, and apply it to the page, if the user has chosen always_hidden as the value of the left_side_userlist attribute.
The questions are:

How can I easily apply this bunch of style (over 70 lines and 20 elements)?
Where should this code be places? I guess it should be in static/js/server_events.js (modifying the existing code) and in resize_page_components in static/js/resize.js. Anywhere else?
Later on I would need to be able to quickly remove this syling (e.g. when the user changes the attribute). This also needs to be taken into consideration when answering the two previous questions.

I think the logic for where things are put should live entirely in resize.js (plus the CSS) and then server_events.js can call into that logic when the user changes the settings.
I don't have a great suggestion at this point for how to do this sort of conditional styling on a large block of the CSS...

We no longer support this feature. It has been replaced by an expandable "Direct messages" section.


API - Help

Hello!
Is there an API to spawn a mob and to setTarget?
I saw you had something with registerEntity how to use that?
Thanks!

Just create entity by Nukkit API and you can set target by using MovingEntity#setTarget(Vector3)

Ugh, just noticed Nukkit hasn't added EntityZombie just EntityCreeper ._.


How to call python methods with 2 arguments

String code "def method():\n"
code +="   A = 10\nB=5\n"
code +="   return A+B"
code +="  method() "
final _result = await Chaquopy.executeCode(code);
_result['textOutputOrError']  is giving me null
if i call method in print then it give me output in string
String code "def method():\n"
code +="   A = 10\nB=5\n"
code +="   return A+B"
code +="print(method())"
output is: 15 in string, but it should be int
My original program I'm returning dictionary with key as String and value as List from program
As per your docs Chaquopy.executeCode(code) should return output in a Map but it is returning String

@vishalideastoimpact You can just assign your script in a single string and put it as a static code, so instead of writing the code as
String code "def method():\n"
code +=" A = 10\nB=5\n"
code +=" return A+B"
code +=" method() "

final _result = await Chaquopy.executeCode(code);

you can just do following :
String code  = "def method():
    A = 10
    B = 10
    return A+B
print(method())"

final _result = await Chaquopy.executeCode(code);
print(_result)

It doesn't work this way
String code  = "def method():
A = 10
B = 10
return A+B
print(method())"
this is not way create python code into String
It gives errors
I think you dont understand the issue
Let me explain again:

Programm is working fine in my way
_result['textOutputOrError']  should contains map not String
As Chaquopy.executeCode(code); return a Map


[New chain]: support iExec Sidechain

Chain Name
iExec Sidechain
RPC URL
https://bellecour.iex.ec/
The chain must be added to https://chainlist.org/

[X] I confirm that the chain is added to chainlist

After creating the issue, a bot will estimate the required pre-fund and post it in the comments. Please check this checkbox after you send the pre-fund.

[X] I sent the pre-fund in accordance with Github Action's comment

Relevant information
We are integrating Safe into our protocol, and we kindly request your support to add our blockchains. Deployment is free; simply provide us with your wallet address for whitelisting. No fees will be charged.

{"jsonrpc":"2.0","error":{"code":-32010,"message":"PermissionDenied, permission denied for tx type: Create, actual permissions: Basic, Call"},"id":44}

The error is occurring because your wallet has not been whitelisted in our blockchain system. Could you please provide us with the deployment wallet address so that we can proceed with the whitelisting process?

Deployer address is 0xE1CB04A0fA36DdD16a06ea828007E35e1a3cBC37

Done, thank you :pray:

0x1ffbfa9f9fe93a2b053f183aa166c641a3b120e357c8685a697e5d0036707d23


Securing te Web UI

Is there a way to prevent unwanted access?

Your question has been answered on the mailing list:
https://groups.google.com/a/openflowhub.org/d/msgid/floodlight-dev/63e09bd2-965b-4628-8726-54ceb46db1e6%40openflowhub.org?utm_medium=email&utm_source=footer


Create too_many_open files errors

When Meilisearch opens too many FDs, you get the following error
{
    "message": "An internal error has occurred. `Too many open files`.",
    "code": "internal",
    "type": "internal",
    "link": "https://docs.meilisearch.com/errors#internal"
}

We want to replace

the code (not type) internal by too_many_open_files
the link #internal by #too_many_open_files

TODO

[ ] Implement changes in Milli
[ ] Release a Milli version containing these changes
[ ] Bump this new Milli version in Meilisearch and merge it into main
[ ] Update the associated specification

Impacted teams
Pinging every teams that might use this, @meilisearch/docs-team (a related issue) and @meilisearch/integration-team
Also @meilisearch/cloud-team I don't know if you would be impacted by this change

please assign me this issue

Hello @semi-infiknight
Thanks for your interest in this project 🔥
FYI, we prefer not assigning people to our issues because sometimes people ask to be assigned and never come back, which discourages the volunteer contributors from opening a PR to fix this issue.
We will accept and merge the first PR that fixes correctly and well implements the issue following our contributing guidelines.
We are looking forward to reviewing your PR 😊

@curquiza Can you help me with finding the place where this change needs to be done? Thanks.

Hey @Rutam21,
I think you'll need to add a variant in the Code to represent the error.
And then in the MilliError enum you should look for every variant that can output a TooManyOpenFiles error and map it to the code you created before :+1:

Moving this to v1 milestones, we will not have the time to fix it for v0.30.0

@curquiza This is not a breaking change anymore based on the decision here https://github.com/meilisearch/engine-team/pull/38 right?

It is still breaking because before people used the internal code to get this error, now they have to use too_many_open_files
The https://github.com/meilisearch/engine-team/pull/38 PR is about warning people using strong languages that would expect a specific and exhaustive list of types (here all the code). Indeed, if we add a new one in a new context (so not breaking) they will have their code broken because this new type is not expected. They have to include an unknown type in the exhaustive list they are expected to retrieve.

Ah, ok, so it is because it is an "update/change", not a "creation", correct?
Because the issue name gave me the sensation that something new would be introduced, but yeah, carefully reading the content + your message made me get it :)

Exactly. There is a nuance. In error handler, creation can also be a modification 😊


Retry on failure

Despite the API being very stable I have seen the occasional 500 error and I would like to make the client more stable by retrying failed requests before giving up. This will be particularly helpful for large index requests. We could make use of the faraday-retry middleware for this scenario:
https://github.com/lostisland/faraday-retry

Yes, pretty please 🙏

I haven't seen any timeouts anymore. So I'd be fine with not doing this.


Is there a way to verify whether PDF help link is loaded properly from WinAppDriver

Hi ,
I have a scenario where on clicking a button from my WPF application, a PDF gets opened with a help link. Is there a way I can automate this?
I need to verify whether appropriate page number of the PDF is loaded on clicking button in my desktop application.
Note : PDF opens in Adobe and not inside the desktop app

WAD is only for automating windows applications.

@anunay1 Thanks for the reply . This scenario cannot be automated by any means am I right?

It can be what you can do is, save the pdf file, convert it to excel or test document, & then parse the file.


Pin Node version to 11

See https://github.com/tree-sitter/node-tree-sitter/issues/46#issuecomment-497458521
and https://github.com/mads-hartmann/bash-language-server/issues/131#issuecomment-504308706

🎩 @KevOrr for this: much appreciated!

Fixed the build errors… now we’re getting some test failures of uncertain provenance.

Okay, the issue here is that on my macOS install, the .phpt tests are not recognized if there’s a file extension .001.phpt. We have to change that leading dot to an underscore. I have no idea why this is happening.


UPDATE in comiple argv both new and old method are working

[Compile Code Instructions]
Expected 3 args :
[Example : ./csq cpp hello 'current-directory']
OR
Expected 1 args :
[Example : csq hello ]

[Suggestion] Hey @WithSJ can you allow the files to be compiled with their extensions e.g csq hello.csq instead of csq hello.

oky..

[Suggestion] Hey @WithSJ can you allow the files to be compiled with their extensions e.g csq hello.csq instead of csq hello.

csq hello
csq hello.csq
both are can use to compile code


Update DefaultTokenServices.java

@cjjava Please sign the Contributor License Agreement!
Click here to manually synchronize the status of this Pull Request.
See the FAQ for frequently asked questions.

@cjjava Thank you for signing the Contributor License Agreement!

@cjjava I'm going to close this PR as it doesn't address a specific issue and there are not details in the description. If this is an issue, please provide details and a test that demonstrates the issue.


!listchannels should highlight unjoined channels

It'd be useful if the !listchannels list showed some indication of whether or not you were in a channel. Some options off the top of my head are:

display unjoined channels separately, or
add an icon or label to unjoined channels.

I think it might be useful if this was turned on with a flag, in case people wanted to list channels without listing all of the channels they are in

Closing as we're getting rid of our managed channel system for Discord's version


Performance problems with more than 150 paths

I have a number of paths to display that can grow large at lower zoom levels, hence I noticed a sensible performance degradation when displaying more than 100 paths, up to few hundreds. Around 200 it takes a bit more than 3 seconds to get them on the map at 700 it gets up to 7 seconds, which is a bit too much for an interactive map.
With a bit of profiling in FF I pointed out the issue to lie at textRedraw and setText methods.
Attached a section of the profile which indicates in green the time taken to populate a layer with labelled paths, while the darker red section shows the time taken to add event handlers to the same number of paths.

To be noted also that most of the paths, since they are at a low zoom level (hence seen from up above) end up with having no label at all as it won't fit the path.

Thanks for your feedback!
Don't hesitate to open a pull-request if you have ideas of optimizations!

Confirming with 100+ paths.

Any updates here?


Add codestyle helpers.

This PR adds .editorconfig and eslint.
This enables the definition of coding standards that are explicit and can be verified by using tools.
A new npm script "lint" is introduced, which checks all js files in lib and test against the eslint rules.
$ npm run lint

> hoodie-client-store@ lint /home/m/www/hoodie-client-store
> ./node_modules/eslint/bin/eslint.js -c .eslintrc lib tests

/home/m/www/hoodie-client-store/lib/scoped/update.js
8:19  error  "storedObjects" is defined but never used  no-unused-vars

/home/m/www/hoodie-client-store/tests/specs/api.js
149:34  error  "object" is defined but never used  no-unused-vars

/home/m/www/hoodie-client-store/tests/specs/scoped.js
291:19  error  "addedItems" is defined but never used   no-unused-vars
401:19  error  "addedItems" is defined but never used   no-unused-vars
430:19  error  "addedItems" is defined but never used   no-unused-vars
463:19  error  "updatedItem" is defined but never used  no-unused-vars
493:19  error  "updatedItem" is defined but never used  no-unused-vars
515:47  error  "object" is defined but never used       no-unused-vars
530:47  error  "object" is defined but never used       no-unused-vars
560:34  error  "object" is defined but never used       no-unused-vars

✖ 10 problems (10 errors, 0 warnings)

https://www.npmjs.com/package/standard is used for this.


Argument labels '(jsonDictionary:)' do not match any available overloads

See attached screeenshot.
List the software versions you're using:

SwiftyJSON: ?.?.?
Xcode Version: Version 9.3 (9E145)
Swift Version: Default

Please also mention which package manager you used and its version. Delete the
other package managers in this list:

Cocoapods: 1.4.0.beta.2 (Use pod --version in Terminal)
Carthage: ?.? (Use carthage version in Terminal)
Apple Swift Package Manager - Swift 4.1.0 (swiftpm-14050)

@antsangrigoli
let jsonObject = JSON(jsonRequest)would be fine.


[PowerToys]Add supported languages:Arabic, Farsi, Hebrew, Ukrainian

@alvinashcraft only to be merged after we release 0.78 😄
Adds reference that we support the following languages:

Arabic (Saudi Arabia) "ar-SA"
Farsi/Persian (Iran) "fa-IR"
Hebrew (Israel) "he-IL"
Ukrainian (Ukraine) "uk-UA"

I'm not sure about the guidelines for each language, so can you please confirm how they should be listed?
@alvinashcraft @crutkas
I ended up going with:

Arabic (Saudi Arabia)
Farsi (Persian)
Hebrew
Ukrainian

Taking the example of https://learn.microsoft.com/en-us/windows/apps/publish/publish-your-app/supported-languages?pivots=store-installer-msix#list-of-supported-languages , I've just changed "Farsi (Persian)" to "Persian"


httputil: remove allocation in newCompressedResponseWriter

Replaces a string.Split with an iterator to save one []string allocation in what seems to be a hot path.
It imports a new dependency, which will need to be reviewed. If the reviewers like this pattern, there are several other places where this could be applied.
Some preliminary benchmarks:
Old:
319.6 ns/op	     165 B/op	       7 allocs/op

New:
308.1 ns/op	     147 B/op	       6 allocs/op

Tagging @juliusv as it seems to be their area. Thanks for the consideration.

Thanks! Hmm, it's not really my code area, but I'm not convinced it's worth optimizing this particular instance. As far as I can see, it's just parsing an HTTP header, which is usually going to be a very tiny part of the overall request handling (and we're not doing that many HTTP requests). Now if this was something that was running a million times in a busy loop, that'd be a different story.
/cc @bboreham (more Go performance-minded person) in case he has ideas for using this anywhere else :)

I concur - not worth adding a new dependency and +24 lines to save 11ns.

If you like optimising, take a profile of a real Prometheus server and find something worthwhile.

Thanks.


fix require relative paths

This closes #3. Please review it, because I'm not very familiar with ES6.

Hmm maybe the process.env.PWD isn't the best choice to fix this, I don't know if an exec(pwd) or using a module like execSync would be a overkill or the perfect solution.

And pwd doesn't work cross windows users, but I guess that isn't a major issue, do you know If there's a practical way to see the working directory?

The correct way to get the "working directory" is process.cwd().
However, path.resolve() uses process.cwd(), e.g.
process.resolve('.') // -> /current/path
process.resolve('./some-file') // -> /current/path/some-file
process.chdir('/opt')
process.resolve('./awesome') // -> /opt/awesome

Ah I forgot totally of process.cwd, nice :+1:

so basically process.env.PWD was useless, removed it, and everything is working, even in Atom :smile:

awesome :)


[25.0]Copilot FWS: address customer feedback/FUD with Copilot & AI Capabilities page

Summary
Work Item(s)
Fixes AB#545996

Need to wait for the uptake from Platform

Close this because translation cannot be backported


[UT] delte unstable ut

Why I'm doing:
because FixedLengthColumnBase's append nulls just return false:
[[nodiscard]] bool append_nulls(size_t count attribute((unused))) override { return false; }
so  column “day_of_year_value” only has 10 rows. visit 11th row in vector is undefined behavior
What I'm doing:
Fixes #issue
What type of PR is this:

[ ] BugFix
[ ] Feature
[ ] Enhancement
[ ] Refactor
[x] UT
[ ] Doc
[ ] Tool

Does this PR entail a change in behavior?

[ ] Yes, this PR will result in a change in behavior.
[x] No, this PR will not result in a change in behavior.

If yes, please specify the type of change:

[ ] Interface/UI changes: syntax, type conversion, expression evaluation, display information
[ ] Parameter changes: default values, similar parameters but with different default values
[ ] Policy changes: use new policy to replace old one, functionality automatically enabled
[ ] Feature removed
[ ] Miscellaneous: upgrade & downgrade compatibility, etc.

Checklist:

[ ] I have added test cases for my bug fix or my new feature
[ ] This pr needs user documentation (for new or modified features or behaviors)

[ ] I have added documentation for my new feature or new function

[ ] This is a backport pr

Bugfix cherry-pick branch check:

[x] I have checked the version labels which the pr will be auto-backported to the target branch

[x] 3.2
[x] 3.1
[ ] 3.0
[ ] 2.5

@Mergify backport branch-3.1

@Mergify backport branch-3.2


stack overflow on big blob of bytes in the storage

like %script here: https://better-call.dev/granadanet/KT1P2JiN9zvXy8X8mHChdsB8tYGz7zFWpqPz/storage

Fix in #80 deployed, cf. https://tzcomet.io/#/explorer%3Fexplorer-input%3DKT1WwEZ98PyXu6BqGrDVBZJYbzCRUbPduuT8


add tun device

required for containerd.io 1.7.24. Otherwise Samba can't be used.

It happens with all docker containers that need tun devices. That was the only thing that worked. It happened after updating docker on Ubuntu 24.

It happend in https://github.com/opencontainers/runc/commit/2ce40b6ad72b4bd4391380cafc5ef1bad1fa0b31

Obviously, a container should not have access to tun/tap device, unless
it is explicitly specified in configuration.
Now, removing this might create a compatibility issue, but I see no
other choice.


UI: Remove All Appraisal Elements

Description
creates config driven render for appraisal - currently set to false
How has this been tested?
Types of changes

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

Checklist:

[x] The related ClickUp task has been linked to this PR
[x] The person creating the pull request is listed in "Assignees"
[ ] Change requires updated documentation

Task linked: CU-8678tb80e UI: Remove All Appraisal Elements


Metrics.d.ts should not include function interfaces declared as 'async'

ISSUES will not receive priority that do not complete this template. They will be flagged as "Insufficient Information" and may be closed without any action.
Describe the bug
I am installing OneTable into an AWS Lambda function project generated by SAM. Compilation complains about the async references in the Metrics.d.ts file.  'error TS1040: 'async' modifier cannot be used in an ambient context'. Editing the file directly, and removing all the asyncs, makes everything work. This kills deployment though.
https://stackoverflow.com/questions/48448151/how-can-i-add-an-async-function-to-a-type-def-file
I'm sorry there is no proper repro. I've been trying for 7 days to get OneTable to work. I don't have any more time for this. I'd rather just remove it.
package.json
{
"name": "hello_world",
"version": "1.0.0",
"description": "Gaylord Claims API Lambda",
"main": "app.js",
"repository": "https://github.com/awslabs/aws-sam-cli/tree/develop/samcli/local/init/templates/cookiecutter-aws-sam-hello-nodejs",
"author": "SAM CLI",
"license": "MIT",
"dependencies": {
"aws-sdk": "^2.1458.0",
"axios": ">=0.21.1",
"dynamodb-onetable": "^2.7.0"
},
"scripts": {
"build": "tsc",
"clean": "rm -rf build"
},
"devDependencies": {
"@types/aws-lambda": "^8.10.119",
"@types/node": "^20.6.0",
"chai": "^4.3.6",
"mocha": "^10.1.0",
"typescript": "^5.2.2"
}
}
Sincerely,
Rick Pfeil -<EMAIL_ADDRESS>

Hi, thanks for your comment.
What do you mean "kills deployment though"?
I can understand removing async, as the return type is Promise<>. Does just removing async work?
Michael

Yes, it does work :-). It kills deployment because I can't include my changes. It pulls fresh code from npm.

Okay, understand. We'll push out a fix today.

I super appreciate this! I love what this package does :-). But I made the mistake of trying to run it client side first.

Thank you. Why was client side a mistake?  We do that in one of our apps.

How?!?! The Node.js modules (crypto, process, ect) are no longer provided for clients. I couldn't get any polyfill to work.

SenseDeep uses the AWS SDK in the client and OneTable. We use some 3rd party polyfills and we created the others.


Jacobian computation

Hi,
I'm trying to compute the forward and inverse differential kinematics of Halfcheetah.
I'd like to calculate the velocity of the joints given the velocity of the "torso". I'm currently testing to see how to compute the Jacobian.
What I'd expect is that given a velocity of the Halfcheetah joints, obtained by reading the variable data.q_vel after calling mj.step(), I can obtain the relative velocities of the "torso", following this equation:
$$\dot{x} = J \cdot \dot{q}$$
Furthermore, by doing the inverse of the Jacobian and multiplying it by the torso velocities obtained in the previous step, I return exactly to the joint velocities, following this relationship:
$$\dot{q} = J^{-1} \cdot \dot{x}$$
In particular, I am using the function mj_jac(...) (https://mujoco.readthedocs.io/en/stable/APIreference/APIfunctions.html#mj-jac)
Below is an example of the code I am using:
    import mujoco
    import numpy as np

    m = mujoco.MjModel.from_xml_path("half_cheetah.xml")
    d = mujoco.MjData(m)
    d.ctrl = [0.33747295, 0.3996362, 0.18299277, 0.5981481, 0.15231994, 0.07616442]
    mujoco.mj_step(m, d)
    jacp = np.zeros((3, 9))
    jacr = np.zeros((3, 9))
    mujoco.mj_jac(m, d, jacp, jacr, d.xpos[1], 1)
    pe = np.matmul(jacp, np.array(d.qvel))
    pr = np.matmul(jacr, np.array(d.qvel))
    J = np.concatenate((jacp, jacr), axis=0)
    p = np.concatenate((pe, pr), axis=0)
    J_inv = np.linalg.pinv(J)
    q_dot = np.matmul(J_inv, p)

With the previous code, I can only obtain the velocities of the first three values of the q_vel vector.
By modifying or cycling on the variable "body" of mj_jac, however, I do not get the desired velocity vector. How can I solve this?
Thank you

Your motivation, as stated, does not make sense.

I'd like to calculate the velocity of the joints given the velocity of the "torso".

This is clearly impossible, the velocity of the torso does not uniquely determine the velocity of all the joints. Try clarifying what you want and we might be able to help.

Thank you for your answer.
Yes, you are right I was wrong to indicate the joint name, I meant "root" and not "torso". As far as I know, when we talk about redundant systems, the assignment is not unique, and a given velocity in the Cartesian corresponds to an infinite number of joint configurations. However, if we consider, for instance, a manipulator where we have the Cartesian velocity of the end-effector, it is possible, using the inverse Jacobian, to also trace the velocities relative to the joints. I'm therefore wondering if it is possible in the halfcheetah contest to do something similar: given a velocity at 'root', calculate the relative velocities at the joints. It's currently unclear to me how to calculate the Jacobian correctly. In tests run with the code posted above, I actually run the inverse of the Jacobian multiplied by a velocity in the Cartesian at the "root," and I correctly get back the first three components, which in the documentation should correspond to X, Z sliding velocities and rate of rotation on Y of the "root", while the other velocities at the joints are zero. How can I correctly calculate the complete Jacobian to find the velocities at all joints? In mujoco's documentation, the function mj_jac requires specifying a "const mjtNum point[3]" and an "int body", but I don't understand how to compute the complete Jacobian for all joints.
Thanks again for your help

A very good example of using the jacobian functionality can be found in the LQR tutorial: https://colab.research.google.com/github/deepmind/mujoco/blob/main/python/LQR.ipynb
I recommend browsing through the API documentation, as there are a couple variants you may be interested in (e.g. pure mj_jac vs the CoM variants).
The point[3] specified is the global position of a point attached to body with the given index for which the jacobian is evaluated (as different parts of a body will move at different velocities if it is rotating, which is why you need to provide both).

if we consider, for instance, a manipulator where we have the Cartesian velocity of the end-effector, it is possible, using the inverse Jacobian, to also trace the velocities relative to the joints.

Yes, because the velocity of the end effector is a function of the joint velocities, if there is a smooth function, it has a partial derivatives, the Jacobian exists.

I'm therefore wondering if it is possible in the halfcheetah contest to do something similar: given a velocity at 'root', calculate the relative velocities at the joints.

No, it is not possible. The root velocity is not a function of the joint velocities, but rather the other way around. The equivalent to the arm scenario is the Jacobian of the halfchettah's foot w.r.t all joints, including the root. I understand that this would be nice to have, but it is not possible, as stated.
The only thing that would conceivably work is to use a data-based approach: measure lots of data of the moving cheetah and try to fit a non-parametric function to the data that attempts to predict what you want. This may or may not work well, depending on your dataset and ML proficiency. YMMV, good luck.

@yuvaltassa What if we assume a no-slip condition with the foot/feet in contact with the ground? Would then the end effector Jacobians be closely related to the CoM motion Jacobian (-1 times)?

@Balint-H, yes and no, the clue is in your own question. One could assume a no-slip condition with a foot, in which case you have a simple kinematic chain, but you have to pick a foot. If there are $n>1$ feet touching the ground you have a closed loop which is not analytic. Also, what if you are in a flight phase? Good luck picking a foot in that case 🙂


New version of similar regions (+bug fix to batch_client.py)

Full rewrite of 'similar regions' based on different metric (Frechet, a.k.a. 2-Wasserstein) treating time series data as a distribution. Tested on AWS unix node under python 3.6.9 only.
All code is in samples/similar_regions_Frechet samples/sr.py with metric.py giving API d/l details. See samples/similar_regions_Frechet samples/test.py for usage examples as well as some d/l failure messages ("Connection closed" and "502 Bad Gateway") which would ideally need to be addressed.
Warning: full data download takes many hours. Contact MA if you want to get some already downloaded data which should be sufficient to test region comparison (as opposed to data download) functionality. It is ~57M, so can not be attached here.
Separately, a few lines were added to batch_client.py to fix what I believe is a bug in handling of certain types of downloading errors

code org: we don't want to create another module, i think one is enough at least by default. but if we want to save the old metric, they can coexist as two different options to load. so let;s move similar_regions_Frechet.sr.SimilarRegion to similar_regions.similar_region_frechet.SimilarRegion and we'll have two top-level classes
testing in progress

all changes done  except moving the module.

ok made a few changes to improve performance
only get necessary regions (this is 10x faster when the user only wants province level)
https://github.com/gro-intelligence/api-client/pull/227/commits/b3ea3b6f5490fb6e3c56957c1f1c92bacd00d86a
only retry on api errors and exponentially smaller batch sizes  when retrying
https://github.com/gro-intelligence/api-client/pull/227/commits/fe5cad980f3176892b37fe95749a9bc7321a0f5e
https://github.com/gro-intelligence/api-client/pull/227/commits/e115a19daf5be7770f3eace3adc2e522772adaca
also cleanup caching so it always uses existing cache and the user can still control it explicitly more simply by having different cache dir
https://github.com/gro-intelligence/api-client/pull/227/commits/5853ff3ba8218d26ba7f488a0fbeb5f19e78c0ad
using new GroClient location
https://github.com/gro-intelligence/api-client/pull/227/commits/1e0a5bcf34bd7e6dc1d3341442082a49dd097e42
retested and updated examples, looks good
next step will merge into the  similar_regions module

merged new version into existing  similar_regions module
fccb32d0f24170362ad348ad80961dbe6cc9733e

I still strongly dislike not having explicit user-controlled "update_mode" parameter. It seems that the modified version will behave similar to update_mode='update' in the original. The problem is that often one knows that there has been no updates to the database (for example, we just ran similar regions a second ago!) and local cache is complete even if there are some data points missing. The new version will always go to database if there are missing data which can be quite wasteful. Complete cache recreation option (update_mode='reload' in the original) can of course be simulated by manually deleting cache files/directory but that seems less convenient

re: "reload", yeah.. note that you don't have to delete it, you can just point to a different location. I prefer not to have two ways to do things, (boolean and name of dir)
re: "no update", I considered that but in practice, checking for missing data is much better because

the data can be filled in,  not just due to data quality updates from gro or the source which I agree would not change in minutes, but also more commonly if you have client side or server side network issues  where some data  points were missed despite the retries, then it does help to refill it.
In terms of time, the cost is negligible.    if you are missing a hundred regions it's only a  4 http requests in parallel so 1 round trip time. The absolute worst case I've seen is still  less 10 seconds , and the overall performance is dominated by the time to  to read the cache from disk (a couple of minutes)

Removing region info caching is ok - I was starting this when get_descendant_regions was very slow.
Rebuild is triggered only if new search region is different from self.search_region (line 585), but, as is, it will NOT be triggered on requested level mismatch. So something incorrect will likely happen if one first searches over provinces and then over districts of the same region as in
similar_to(region_id, requested_level=4) # this will build search trees for level 4 only
similar_to(region_id, requested_level=5) # searching over the same region, so no object rebuilding, and no search tree for level 5
(since data structures are now only build for requested levels). Needs to be fixed with some tracking/checking of current level(s) on which data structures exist

Yes! Meant to add level to the tracking but forgot. Done.

Finally, I think returning all parents (and respectively grandparents) is very inconvenient for the user. Many of these parents are very obscure region collections (something like "Ag region 42"), so this will generate potentially a lot of info nobody cares about within which it will be hard to find what is most needed. For example, for a US county parent/grandparent should just be US_state, USA. So at the very least, only return parents/grandparents on correct levels. For 3/4/5 level regions these are expected to form a tree, so it should be sufficient to return just one.

Done.
FYI, I do want to add region level 8 as well so it works for non-admin regions that are still interesting as physical regions and therefore have geospatial data e.g. water basins, agricultural zones etc.   Also in many cases, we expect the user to add more metadata and formatting outside of the sr core logic,  e.g. to draw a map you would get the geojson. Will add an  examples to the notebook separately. So we don't need to add too much to the output here in the core logic, but picking the most important parent and grandparent (the admin parent) is basic enough that it's worth it.

added a few improvements to README and example notebook

final review done offline with @ash-gro


Description for GOCDB roles

Summary
This PR extends the "user attributes" section adding description for GOCDB roles in Service Provider guide of Check-in.

So the Level of Assurance name is dead? :)

@gwarf Yes, 'Level of Assurance' has been renamed to 'Identity Assurance'
If you mention the term 'Level of Assurance' in any of your guides, please replace it with 'Identity Assurance'.

@gwarf Yes, 'Level of Assurance' has been renamed to 'Identity Assurance'
If you mention the term 'Level of Assurance' in any of your guides, please replace it with 'Identity Assurance'.

OK, notted. And entitlements are also renamed to groups? Aren't they more than just a group? But yes it will be more straightforward for some people.
Apart this I doubt that the file excerpt are really vim/vimL excerpt, if you don't have anything more specific you could use something generic like text.


Return error on certificateexpirationstatus monitor when IngressController is invalid rather than panic

Which issue this PR addresses:
Fixes ARO-5227
What this PR does / why we need it:
The ARO Monitor's certificateexpirationstatus monitor expects the IngressController resource on-cluster to be well-formed, namely it requires a default certificate name to be present within .spec.defaultCertificate.name.
The Go type definition for IngressController allows the DefaultCertificate property to be nil, and we do not currently perform a nil check here.
This causes the monitor service to panic and terminate when evaluating a cluster that has such an invalid IngressController resource.
Test plan for issue:

Unit test added for this specific case
All existing unit and e2e tests continue to pass

Is there any documentation that needs to be updated for this PR?


Request for mini-batch training codes

Hi, the codes now are all in full-batch training way, which run OOM on Pubmed. Would you please upload the mini-batch training version?

Yeah thanks for the notice. I'll upload the version with minibatch once I have those codes cleaned.

Yeah thanks for the notice. I'll upload the version with minibatch once I have those codes cleaned.

sorry for the late reply. I just uploaded the code for GAugO with minibatch.

Hi, the codes now are all in full-batch training way, which run OOM on Pubmed. Would you please upload the mini-batch training version?

Hi，I'm also trying to run this code on Pubmed, but I can't find data/graphs/pubmed_tvt_nids.pkl, as well as everything of Pubmed. I have generated edge_probablities of PubMed, but how did you generate pubmed_tvt_nids.pkl and other graph files? Or where could I find them? Thamks!

Hi, the pubmed_tvt_nids.pkl file is just a pickled file storing the training/validation/testing node indexes. It's a list containing three 1-D numpy.ndarray. The data formats for all files used in this repo can be found here. You can load the PubMed dataset with public implementations such as DGL or the public implementations.


Managing two or more queues

My solution has a couple of webjobs, binded to a different queues.
How to configure app setting if it just have ONLY one option "AzureWebJobsStorage"?

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

ID: a4500b14-64b4-4d1b-aae3-f0fd4dde85f2
Version Independent ID: 8fb75741-37b9-b45f-c0ca-9948ddaf1c06
Content: How to use the WebJobs SDK - Azure
Content Source: articles/app-service/webjobs-sdk-how-to.md
Service: app-service-web
GitHub Login: @ggailey777
Microsoft Alias: glenga

@angelieri Thanks for the question! We are investigating and will update you shortly.

@angelieri If I have understood right, you want to manage two or more queues
In this case, you can try to bind different storage accounts so your function can look like that:
public static void JobQueue1(
[QueueTrigger("queueName1"),
StorageAccount("storageAccount1ConnectionString")] string message)
{
}
public static void JobQueue2(
[QueueTrigger("queueName2"),
StorageAccount("storageAccount2ConnectionString")] string message)
{
}
You can also implement a custom INameResolver if you want to get the connectionstrings from the config :
public class ConfigNameResolver : INameResolver
{
public string Resolve(string name)
{
string resolvedName = ConfigurationManager.AppSettings[name];
if (string.IsNullOrWhiteSpace(resolvedName))
{
throw new InvalidOperationException("Cannot resolve " + name);
}
    return resolvedName;
}

}
to use this:
var config = new JobHostConfiguration();
config.NameResolver = new ConfigNameResolver();
...
new JobHost(config).RunAndBlock();
So, your new functions may look like:
public static void JobQueue1(
[QueueTrigger("queueName1"),
StorageAccount("%storageAccount2%")] string filename)
{
}
public static void JobQueue2(
[QueueTrigger("queueName2"),
StorageAccount("%storageAccount1%")] string filename)
{
}
In this storageAccount1 and storageAccount2 are the connection string key in the appSettings.

@angelieri If I have understood right, you want to manage two or more queues
In this case, you can try to bind different storage accounts so your function can look like that:

I think this is for 2.x version.
I'm trying to use the new 3.x

@angelieri We typically try to keep the Azure-Docs channel scoped to feedback directly about the docs. Since your question goes beyond what is discussed in the doc, we kindly invite you to share your question . Thank you for your understanding


fix: vue-demi version overwrite & update deps

Hey @hannoeru 😄
Thank you a lot for that!
It looks good to me, merging this right now and releasing a new version of @vueuse/motion 😄


Target STM32F3

These series fixes some tests on STM32 F3, and update the SMT32F3 hal.
With the pull request #2090 , the tests are passed.

The first commit should not be here, it was pushed ot the release branch :) please split patches - hal update should be separate PR ?

This pull request now contains only the HAL update.
Other separate pull request will come with the other fix.

Do I need to update something to have it integrated?

It would be helpful to share the test results (how was this PR tested).


Collateral changes in 1.3.0?

Hello Sia guys.
Did you change collateral handling in Capricorn?  I noticed some different wallet activity related to collateral in the update.  So to test, I upped my Collateral per TB to 500 for the last 12 hours.
I previously had 1200SC in this wallet, and sent it an additional 400.  So I should have 1600 - collateral taken for contracts formed.
In the last 12 hours, I went from 583 contracts to 759 (+176).  Locked collateral went from 20.98K to 25.21K (+4,230) and risked went from 4.001k to 5.262K (+1261)
In that time, the only wallet activity I have is a 62SC transaction and a whole lot of .04 (which I would imagine are related to a different bug opened).  My wallet balance is still ~1500SC.

https://github.com/NebulousLabs/Sia/issues/2144

Recommendation to devs: Close as this bug is tracked in #2144


[Android] plugin-upload download function cant works

i dont know why permission denied, and I have authorized the permission to read and write files， but still not works

The error makes no sense to me tbh but the plugin doesn't work on mobile yet. Closing as duplicate of https://github.com/tauri-apps/plugins-workspace/issues/1156


Create new ARM flavor with larger disk

Change back all ARM flavors to default disk size and create a new flavor, same as default one with larger disk. Create a new one to not increase default thus increasing running cost for all builds that do not need more space.
KFLUXINFRA-1141

/lgtm

I think we need to add the platform to line 14
dynamic-platforms: "\

Good point, I know this but somehow forgot..

I think we need to add the platform to line 14
dynamic-platforms: "\

Good point, I know this but somehow forgot..

Done

As usual, merging this change as e2e are not covering the files change in this PR


Add Status Update Functionality and "Clear All Completed" Button

This pull request adds new functionality to the existing codebase, enabling users to update the status of tasks and clear all completed tasks.
Changes Made:

Added a function to clear all completed tasks using the filter method.
Updated the index of remaining tasks after clearing completed tasks.
Enhanced the UI by refreshing the task list after clearing completed tasks.

Updated the displayItems method in the UI class in the ui.js module.
checked
property of the checkbox based on the
completed
property of the item when the page is loaded.

Hello @ClarenceAtim 👋,
This is an additional review as a result of the second opinion request.
I can see that your project has been approved.
PS: Next time you request a Second Opinion Review, please wait for it before asking for a regular review again. While it may take longer to receive the second opinion, it will save a review for you in case there are any issues. Additionally, please refrain from pushing any commits after receiving the initial review so that we can examine the project again and provide a comprehensive second opinion.
Keep rocking 🚀!


add i18 beamline definition

Fixes #709
Instructions to reviewer on how to test:

Do `dodal connect i181
Confirm all the devices expected to work, work

Checks for reviewer

[ ] Would the PR title make sense to a scientist on a set of release notes
[ ] If a new device has been added does it follow the standards
[ ] If changing the API for a pre-existing device, ensure that any beamlines using this device have updated their Bluesky plans accordingly
[ ] Have the connection tests for the relevant beamline(s) been run via dodal connect ${BEAMLINE}

@DominicOram the devices are skipped as we don't need all of them at the moment, as we roll out the devices following the plans we make, so it's gradual and agile

so it's gradual and agile

The more agile way to do it would be don't put the devices in at all until you need them otherwise we now have code in that never runs and we don't know if it works.

we need them for the plans in a different repo. also aren't devices skipped all the time depending if they are off or taken out of a beamline physically?

lint error
found 0 vulnerabilities
/home/runner/work/dodal/dodal/src/dodal/devices/i18/sim_motor_set.py
  /home/runner/work/dodal/dodal/src/dodal/devices/i18/sim_motor_set.py:57:12 - error: Cannot assign to attribute "delay" for class "SynAxis"
    Expression of type "float" cannot be assigned to attribute "delay" of class "SynAxis"
      "float" is incompatible with "int" (reportAttributeAccessIssue)
1 error, 0 warnings, 0 informations

test error:

FAILED tests/devices/i22/test_dcm.py::test_configuration - AssertionError: Expected dcm to produce
{'dcm-bragg-motor_egu': {'value': '', 'timestamp': , 'alarm_severity': }, 'dcm-bragg-velocity': {'value': 0.0, 'timestamp': , 'alarm_severity': }, 'dcm-energy-motor_egu': {'value': '', 'timestamp': , 'alarm_severity': }, 'dcm-energy-velocity': {'value': 0.0, 'timestamp': , 'alarm_severity': }, 'dcm-crystal_1_roll-motor_egu': {'value': '', 'timestamp': , 'alarm_severity': }, 'dcm-crystal_1_roll-velocity': {'value': 0.0, 'timestamp': , 'alarm_severity': }, 'dcm-offset-motor_egu': {'value': '', 'timestamp': , 'alarm_severity': }, 'dcm-offset-velocity': {'value': 0.0, 'timestamp': , 'alarm_severity': }, 'dcm-crystal_2_pitch-motor_egu': {'value': '', 'timestamp': , 'alarm_severity': }, 'dcm-crystal_2_pitch-velocity': {'value': 0.0, 'timestamp': , 'alarm_severity': }, 'dcm-crystal_2_roll-motor_egu': {'value': '', 'timestamp': , 'alarm_severity': }, 'dcm-crystal_2_roll-velocity': {'value': 0.0, 'timestamp': , 'alarm_severity': }, 'dcm-perp-motor_egu': {'value': '', 'timestamp': , 'alarm_severity': }, 'dcm-perp-velocity': {'value': 0.0, 'timestamp': , 'alarm_severity': }, 'dcm-crystal_2_d_spacing': {'value': 3.13475, 'timestamp': , 'alarm_severity': }, 'dcm-crystal_2_usage': {'value': 'Bragg', 'timestamp': , 'alarm_severity': }, 'dcm-crystal_2_type': {'value': 'silicon', 'timestamp': , 'alarm_severity': }, 'dcm-crystal_1_reflection': {'value': [1, 1, 1], 'timestamp': , 'alarm_severity': }, 'dcm-crystal_2_reflection': {'value': [1, 1, 1], 'timestamp': , 'alarm_severity': }, 'dcm-crystal_1_d_spacing': {'value': 3.13475, 'timestamp': , 'alarm_severity': }, 'dcm-crystal_1_usage': {'value': 'Bragg', 'timestamp': , 'alarm_severity': }, 'dcm-crystal_1_type': {'value': 'silicon', 'timestamp': , 'alarm_severity': }}
but actually got
{'dcm-bragg-motor_egu': {'value': '', 'timestamp':<PHONE_NUMBER>31, 'alarm_severity': 0}, 'dcm-bragg-velocity': {'value': 0.0, 'timestamp':<PHONE_NUMBER>13, 'alarm_severity': 0}, 'dcm-energy-motor_egu': {'value': '', 'timestamp':<PHONE_NUMBER>64, 'alarm_severity': 0}, 'dcm-energy-velocity': {'value': 0.0, 'timestamp':<PHONE_NUMBER>09, 'alarm_severity': 0}, 'dcm-crystal_2_pitch-motor_egu': {'value': '', 'timestamp':<PHONE_NUMBER>6, 'alarm_severity': 0}, 'dcm-crystal_2_pitch-velocity': {'value': 0.0, 'timestamp':<PHONE_NUMBER>2, 'alarm_severity': 0}, 'dcm-offset-motor_egu': {'value': '', 'timestamp':<PHONE_NUMBER>21, 'alarm_severity': 0}, 'dcm-offset-velocity': {'value': 0.0, 'timestamp':<PHONE_NUMBER>42, 'alarm_severity': 0}, 'dcm-crystal_1_roll-motor_egu': {'value': '', 'timestamp':<PHONE_NUMBER>73, 'alarm_severity': 0}, 'dcm-crystal_1_roll-velocity': {'value': 0.0, 'timestamp':<PHONE_NUMBER>24, 'alarm_severity': 0}, 'dcm-crystal_2_roll-motor_egu': {'value': '', 'timestamp':<PHONE_NUMBER>46, 'alarm_severity': 0}, 'dcm-crystal_2_roll-velocity': {'value': 0.0, 'timestamp':<PHONE_NUMBER>66, 'alarm_severity': 0}, 'dcm-perp-motor_egu': {'value': '', 'timestamp':<PHONE_NUMBER>08, 'alarm_severity': 0}, 'dcm-perp-velocity': {'value': 0.0, 'timestamp':<PHONE_NUMBER>78, 'alarm_severity': 0}, 'dcm-crystal_2_type': {'value': 'silicon', 'timestamp':<PHONE_NUMBER>79, 'alarm_severity': 0}, 'dcm-crystal_1_usage': {'value': 'Bragg', 'timestamp':<PHONE_NUMBER>65, 'alarm_severity': 0}, 'dcm-crystal_1_reflection': {'value': [1, 1, 1], 'timestamp':<PHONE_NUMBER>63, 'alarm_severity': 0}, 'dcm-crystal_2_reflection': {'value': [1, 1, 1], 'timestamp':<PHONE_NUMBER>42, 'alarm_severity': 0}, 'dcm-crystal_1_d_spacing': {'value': 0.31356, 'timestamp':<PHONE_NUMBER>51, 'alarm_severity': 0}, 'dcm-crystal_2_d_spacing': {'value': 0.31356, 'timestamp':<PHONE_NUMBER>96, 'alarm_severity': 0}, 'dcm-crystal_2_usage': {'value': 'Bragg', 'timestamp':<PHONE_NUMBER>45, 'alarm_severity': 0}, 'dcm-crystal_1_type': {'value': 'silicon', 'timestamp':<PHONE_NUMBER>42, 'alarm_severity': 0}}
=========== 1 failed, 578 passed, 2 skipped, 27 deselected in 21.10s ===========
@iain-hall I'll try tackling the second one

lint error
found 0 vulnerabilities
/home/runner/work/dodal/dodal/src/dodal/devices/i18/sim_motor_set.py
  /home/runner/work/dodal/dodal/src/dodal/devices/i18/sim_motor_set.py:57:12 - error: Cannot assign to attribute "delay" for class "SynAxis"

I would not use the Ophyd (i.e. not ophyd-async) SynAxis

the test_dcm error comes from a failure to correctly read in the 'ANY' mock value.

 src/dodal/beamlines/i18.py:17: in <module>
    from dodal.devices.i18.sim_detector import SimDetector
/opt/hostedtoolcache/Python/3.10.14/x64/lib/python3.10/site-packages/dodal/devices/i18/sim_detector.py:2: in <module>
    from ophyd_async.sim import PatternGenerator
/opt/hostedtoolcache/Python/3.10.14/x64/lib/python3.10/site-packages/ophyd_async/sim/__init__.py:1: in <module>
    from .pattern_generator import PatternGenerator
/opt/hostedtoolcache/Python/3.10.14/x64/lib/python3.10/site-packages/ophyd_async/sim/pattern_generator.py:14: in <module>
    import h5py
E   ModuleNotFoundError: No module named 'h5py'
____________ ERROR collecting src/dodal/devices/i18/sim_detector.py ____________
ImportError while importing test module '/home/runner/work/dodal/dodal/src/dodal/devices/i18/sim_detector.py'.

@DiamondJoseph how do I ask for the ophyd-async to be installed with the [sim] dependencies?
the error in question: ModuleNotFoundError: No module named 'h5py'

I'll answer on sSlack to avoid everyone getting emails

waiting until pydantic 2

=========================== short test summary info ============================
ERROR src/dodal/beamlines/i18.py - TypeError: 'str' object is not callable
!!!!!!!!!!!!!!!!!!!! Interrupted: 1 error during collection !!!!!!!!!!!!!!!!!!!!
======================= 30 deselected, 1 error in 4.31s ========================
ERROR: InvocationError for command /opt/hostedtoolcache/Python/3.10.15/x64/bin/pytest -m 'not s03' --cov=dodal --cov-report term --cov-report xml:cov.xml (exited with code 2)

why doesn't tox show me which line is it?
or am I missing something?

=========================== short test summary info ============================
ERROR src/dodal/beamlines/i18.py - TypeError: 'str' object is not callable
!!!!!!!!!!!!!!!!!!!! Interrupted: 1 error during collection !!!!!!!!!!!!!!!!!!!!
======================= 30 deselected, 1 error in 4.31s ========================
ERROR: InvocationError for command /opt/hostedtoolcache/Python/3.10.15/x64/bin/pytest -m 'not s03' --cov=dodal --cov-report term --cov-report xml:cov.xml (exited with code 2)

why doesn't tox show me which line is it?
or am I missing something?

Scroll up above the coverage
 File "/home/runner/work/dodal/dodal/src/dodal/beamlines/i18.py", line 61, in <module>
    @skip_device("not ready yet")
     ^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/opt/hostedtoolcache/Python/3.11.10/x64/lib/python3.11/site-packages/dodal/utils.py", line 106, in decorator
    if precondition():
       ^^^^^^^^^^^^^^
TypeError: 'str' object is not callable

Probably I have failed you there with the @skip_device add string.

it could have now....

fails due to dcm changes, awaiting the other merge https://github.com/DiamondLightSource/dodal/pull/870/files

getting an error when upgrading to a higher ophyd-async version
ERROR: pip's dependency resolver does not currently take into account all the packages that are installed. This behaviour is the source of the following dependency conflicts.
blueapi 0.6.1a3.dev20+gb6a47ba5.d20241113 requires bluesky==1.13.0a3, but you have bluesky 1.13 which is incompatible.
blueapi 0.6.1a3.dev20+gb6a47ba5.d20241113 requires event-model==1.21, but you have event-model 1.22.1 which is incompatible.


fig.3 in paper is not the same with code

thanks for sharing the paper and code.
I found that in Fig.3 the convolution block for skip layer should probably be downsampling-block and in blue color, however currently Fig.3 in paper is green color, which stands for same-block.

Oh, this is my mistake. skip layer is downsampling-block. Thank you!


Add require distributed_lock to relevant tasks

Some of the rake tasks were failing due to:
NameError: uninitialized constant LocalLinksManager::DistributedLock

LGTM RKE 👍


Automated Package Upgrade

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Removed iris.util.as_compatible_shape and now unused imports

🚀 Pull Request
Description

Deprecation as per #4394
Removed unused imports in addition to the deprecated function.

Consult Iris pull request check list

Thanks @wjbenfold !


Update edge-computing-solutions-support.md

updates

@RickCoxDev please merge this for tomorrows deployment


Incompatible with modded accessories

Description
When the mod makes you more transparent, some accessories added by other mods are not affected, in this screenshot I am using a crown from Clutter mod, the backpack from the Inmis and InmisAddon mod, and weapons attached to the body from the YDM's Weapon Master mod

https://modrinth.com/mod/clutter
https://modrinth.com/mod/inmis
https://modrinth.com/mod/inmisaddon
https://modrinth.com/mod/weaponmaster

If it is possible to add compatibility, I ask you to publish an update for 1.20.1
Minecraft version
1.20.1
Incompatible mod name
Clutter 1.20.(0-1)-0.5.7 / YDM's Weapon Master 3.0.5+1.20.1 / Inmis 2.7.2+1.20.1 / InmisAddon 1.0.4+1.20.1
Incompatible mod version
.
Game Logs
https://mclo.gs/AT2STET

Thank you for the report. I looked into this and here are my findings:

Clutter renders hats as items and not as armor (see why this creates a problem below)
Inmis renderes the backpack as an item (see why this creates a problem below)
InmisAddon renders the backpack in a way, such that no transparency can be applied to it (see source code here)
Weaponmaster is a closed source mod, but I suspect that it also renders its visuals as items  (see why this creates a problem below)

The problem with items:
Due to the way minecraft is programmed, it is not possible to implement item/block model transparency on item models, only while rendering the player. This alone would already require a lot of changes to the vanilla minecraft codebase, including a lot of hardcoded transparency rules for individual items. Furthermore, the required changes might introduce visual bugs elsewhere in the game, which can already be observed when looking at water through a glass block and a nether portal . There is also no guarantee that the changes fix the compatibility with every other mod.
To summarize, it is unfortunately not possible to implement transparency for item models - at least currently. As a workaround, I suggest disabling the transparency feature alltogether in the config, if you dislike the visual bug.

I opened a PR on their github with the required changes.


Handle spaces in current directory path

RuboCop doesn't work well when pwd contains spaces. So i have to call it
:RuboCop "pwd"
to get the desired functionality.
Please resolve the issue.

Perhaps it is the reason why files from quick fix doesn't open correctly. Because i can't open them via  in QuickFix list.


chore(release): v1.1.0 🎉

:robot: I have created a release *beep* *boop*
1.1.0 (2022-10-09)
Features

obs: save value to cache file as observably as possible (11204fe)

This PR was generated with Release Please. See documentation.

:robot: Release is at https://github.com/ehmpathy/simple-on-disk-cache/releases/tag/v1.1.0 :sunflower:


nix-copy-closure has a hidden dependency on ssh

The nix-copy-closure script included in nix uses ssh to copy closures around.
Since the calls to ssh are hidden in perl system calls (see e.g. https://github.com/NixOS/nix/blob/master/perl/lib/Nix/SSH.pm#L90), the dependency is not detected when nix is built via nix.
So on a system where ssh is not installed globally, nix-copy-closure fails.
A simple solution would be to add the ssh dependency and wrap the perl scripts as needed, but that would make nix as a whole depend on ssh, which may not be desirable.
Instead, I think nix-copy-closure should be a separate derivation.
If nix-copy-closure is too tightly coupled with nix, maybe at least the nixpkgs nix package could have an option to add the dependency or not.
I would be glad to help solving this, though I am not familiar with how nix injects dependencies.

It might be intentional, in a way like parted has hidden dependencies on too many filesystem-specific packages and it just does not support the filesystems which utilities are not installed.

I see the comparison, but this kind of solution is very contrary to the purpose of nix, and IMHO should only be used in cases like parted where it would be too complicated to add all the optional dependencies to the nixpkgs package.
Here, I don't see why adding an optional dependency on openssh would be too hard.
Also, I forgot to mention but nixos-rebuild fails to work when used with both --build-host and --target-host because of this missing dependency.

FWIW, the issue can be reproduced using nix-shell, e.g. like this:
$ nix-shell --pure -p nix --run 'NIX_REMOTE=daemon nix-copy-closure localhost /nix/store/derpderp'
sh: line 0: exec: ssh: not found
unable to connect to ‘localhost’; falling back to old closure copying method
path â/nix/store/derpderpâ is not valid at /nix/store/5s702qd8a7n6byk3wlh2777g3j67cl06-nix-1.11.13/lib/perl5/site_perl/5.22.3/x86_64-linux-thread-multi/Nix/CopyClosure.pm line 77.

In the meantime, for anyone blocked by this issue (as I was), I have made the following dirty wrapper around nix that adds ssh to the PATH of nix-copy-closure: https://github.com/Nadrieril/dotfiles/blob/67061b004fa5c3c1f7f498f200b9bddfb6bee11c/.config/nixpkgs/overlays/11-nix-wrapper.nix

Is this still an issue?
The output looks different:
[davidak@gaming:~]$ nix-shell --pure -p nix --run 'NIX_REMOTE=daemon nix-copy-closure localhost /nix/store/derpderp'
these paths will be fetched (0.15 MiB download, 0.74 MiB unpacked):
  /nix/store/22lm9jbc6m8za9030k2p4k52l5pa0m3i-nix-2.3.6-dev
  /nix/store/38dq3mnjmxk7c9gpsjayir0ccg7fvpyg-boehm-gc-8.0.4-dev
  /nix/store/wqfv04xz2rpl2326ijghn1f2qgiyplb4-bash-interactive-4.4-p23-dev
copying path '/nix/store/wqfv04xz2rpl2326ijghn1f2qgiyplb4-bash-interactive-4.4-p23-dev' from 'https://cache.nixos.org'...
copying path '/nix/store/38dq3mnjmxk7c9gpsjayir0ccg7fvpyg-boehm-gc-8.0.4-dev' from 'https://cache.nixos.org'...
copying path '/nix/store/22lm9jbc6m8za9030k2p4k52l5pa0m3i-nix-2.3.6-dev' from 'https://cache.nixos.org'...
error: path '/nix/store/derpderp' is not in the Nix store

And nix-copy-closure is no longer a perl script.
[davidak@gaming:~]$ file /nix/store/csrmkvlyfikij2h0kdqsgbkmqlsjspwg-nix-2.3.6/bin/nix-copy-closure
/nix/store/csrmkvlyfikij2h0kdqsgbkmqlsjspwg-nix-2.3.6/bin/nix-copy-closure: symbolic link to nix

[davidak@gaming:~]$ ll /nix/store/csrmkvlyfikij2h0kdqsgbkmqlsjspwg-nix-2.3.6/bin/
total 1776
-r-xr-xr-x 1 root root 1816736 Jan  1  1970 nix
lrwxrwxrwx 1 root root       3 Jan  1  1970 nix-build -> nix
lrwxrwxrwx 1 root root       3 Jan  1  1970 nix-channel -> nix
lrwxrwxrwx 1 root root       3 Jan  1  1970 nix-collect-garbage -> nix
lrwxrwxrwx 1 root root       3 Jan  1  1970 nix-copy-closure -> nix
lrwxrwxrwx 1 root root       3 Jan  1  1970 nix-daemon -> nix
lrwxrwxrwx 1 root root       3 Jan  1  1970 nix-env -> nix
lrwxrwxrwx 1 root root       3 Jan  1  1970 nix-hash -> nix
lrwxrwxrwx 1 root root       3 Jan  1  1970 nix-instantiate -> nix
lrwxrwxrwx 1 root root       3 Jan  1  1970 nix-prefetch-url -> nix
lrwxrwxrwx 1 root root       3 Jan  1  1970 nix-shell -> nix
lrwxrwxrwx 1 root root       3 Jan  1  1970 nix-store -> nix

[davidak@gaming:~]$ file /nix/store/csrmkvlyfikij2h0kdqsgbkmqlsjspwg-nix-2.3.6/bin/nix
/nix/store/csrmkvlyfikij2h0kdqsgbkmqlsjspwg-nix-2.3.6/bin/nix: ELF 64-bit LSB executable, x86-64, version 1 (GNU/Linux), dynamically linked, interpreter /nix/store/xg6ilb9g9zhi2zg1dpi4zcp288rhnvns-glibc-2.30/lib/ld-linux-x86-64.so.2, for GNU/Linux 2.6.32, BuildID[sha1]=c4d6609a3510b047c9ef08fcd9c330d5f29f257b, not stripped

Then it looks fixed! I'll close this and someone can message if they still have this problem


Show segment in 3DUniverse at new timepoint does not work

When selecting a segment in the table that is at a new time point it is not shown in the 3DUniverse.
At least one cannot see it, could also be that the view is not adjusted.
When one adds a second segment at the same time point it is shown and also focused in 3D; why doesn't this work for the first one?

This seems to be specific to certain segments, e.g. label 89 at time point 1.
Creating the mesh throws an ArrayIndexOutOfBounds exception, not sure yet why.


Make Gitlab optional

Add code to handle gitlab not available
Raise GitlabInitializationError instead of RuntimeError in GitlabManager
if unable to initialize gitlab.
Skip tests that require gitlab if GitlabInitializationError is raised.

Fix metadata_path typo in asset_groups/models.py
Replace "metatdata_path" with "metadata_path" in asset_groups/models.py

Run tests without gitlab in testing workflow
Set GITLAB_REMOTE_URI to a non existent url to run tests without gitlab

Just wondering about the logic of disabling the tests on gitlab failure. I know I see that if I've started docker and not waited long enough for gitlab to start. Is this getting around a particular issue?

Just wondering about the logic of disabling the tests on gitlab failure. I know I see that if I've started docker and not waited long enough for gitlab to start. Is this getting around a particular issue?

@hwindsor This PR is to enable us to develop and run the site without gitlab, so tests that absolutely rely on gitlab (like cloning from gitlab) are skipped if gitlab isn't available.


Details dialog

Not sure what happened, but after some time of using picotorrent i got this view of torrent details window. And it looks very funny when I move this window >_<

Very strange - does the problem persist if you restart PicoTorrent?

Nope,  after restart problem gone, and I can't reproduce it yet.

I had something like this happen when I opened the files tab on a torrent with a lot of files on it (varies, but over 5000 will always trigger it), I narrowed it down to the file extension icons that was causing it (I built a copy with it disabled, and the problem stopped)
From what I've seen, the newest build after the WTL refactor doesn't have the icons at the moment, so you could try that build and see if it keeps happening.

Strange - as @zp said the WTL refactor removed the icons in the files tab so you shouldn't have an issue there. I'll remember to test with +5k files when adding back the file icon feature :smile:
Thanks everyone! Considering this closed for now.


Add Magic Tiles Game

I want to add magic tiles game onto the funfusion website. I am attaching a screenshot below for reference.

@Priyanshi662 Please assign me this issue under JWOC.


How to fit axes to the data for a scatter plot?

After rendering the plot I can use a right mouse click to open a Settings dialog. In the settings dialog the horizontal and vertical axis sections have a Fit button.
How can I make horizontal and vertical axis size to fit the min/max of the data?
I tried
DischargeGraphPlot.plt.AxisAuto(xExpandOnly: true, yExpandOnly: true);
This is how the graph appears:

This is what I want:

Hey @pbanta, you're pretty close! Just call AxisAuto() without arguments.
DischargeGraphPlot.plt.AxisAuto();

Using the "expand only" arguments was preventing the axis limits from contracting to fit your data. Let me know if this does the trick! 👍

Also, FWIW, if your data is evenly spaced on the horizontal axis consider using PlotSignal() or PlotSignalConst() for improved performance.

Yes, that fixed it in addition to calling AxisAuto() after PlotScatter().
I will experiment with PlotSignal() and PlotSignalConst() because the data is evenly spaced.
Thank you very much for the help!


Port VmaReplay to Linux

On my TODO list.

I am sorry, but I cannot accept this contribution. Although the library itself is portable across many platforms, it is developed mostly on and for Windows PC, and so we are not able to support and maintain projects like the sample app or VmaReplay for other platforms. Support for other platforms is welcome as external projects / forks.


marketplace

Now that there is a logseq plugin marketplace, this theme should be updated to work with that!

Now that there is a logseq plugin marketplace, this theme should be updated to work with that!

yep, thanks for the encouragement, it's definitely on my to-do list!

Saw it in the marketplace. Thanks!


Mpp fix

Just a heads up, could you guys merge this in please. Took me a while to figure out the mask token dimensions problem. The MPP example given in the README works out of pure luck due to the specific numbers chosen.

@zankner I wanted to know about reference paper for: masked patch prediction. Can you tell me? Think you very much.

@zankner I wanted to know about reference paper for: masked patch prediction. Can you tell me? Think you very much.

I believe this is the one:
https://openaccess.thecvf.com/content/CVPR2022/html/He_Masked_Autoencoders_Are_Scalable_Vision_Learners_CVPR_2022_paper

ok, thank you


PIL analyzer roadmap

In the future, the PIL analyzer should be a multi-stage component that can both work with pil embedded into powdr and pil stand-alone. The different stages would take different types of ASTs (probably just parameterised) and return modified types, enriched with information gathered in the stage.
The stages are:

Imports

the import system loads modules and gathers all declarations/symbols in a hierarchical module tree. All symbols receive a unique absolute path.

Reference Resolution

All References in expressions are resolved to absolute paths. The type of Reference inside Expression probably changes from String (or SymbolPath) to AbsoluteSymbolPath, although it will also need to incorporate local variables, so it will more likely be an enum over AbsoluteSymbolPath and LocalVariable.

Type deduction

Types are deduced for all symbols and all expressions. Expressions receive their type as additional annotation.

Type checking

It is verified that constraints all have the proper type (int -> fe or (int -> fe)[]) and declarations are grouped into witness, fixed and intermediate columns and auxiliary symbols. It must be possible to "inline" all auxiliary symbols so that we only end up with the three categories.

Compression

(Fixed column generation is run before compression)
Compression removes all auxiliary symbols and just returns the set of fixed, witness and intermediate columns and the list of identities. It simplifies the expression tree in that more complicated things like function calls, match expressions, etc are removed. All auxiliary symbols are inlined in order to do this. It also assigns unique IDs to all symbols, so that the witness generator and the export modules can better work with them.

Overall sounds good. One thing I dislike thought, is the approach of a mutable AST vs different ASTs transformed in a cascade.

The type of Reference inside Expression probably changes from String (or SymbolPath) to AbsoluteSymbolPath

Expressions receive their type as additional annotation.

More specifically I'm referring to these. I don't think ASTs should receive new info and mutate with annotations etc, but rather be built with that type/information in the constructor.

Oh the different ASTs will have a different type and the various stages get the previous type as input and produce a new ast with the new type. I'm not talking about mutables fields or fields that are Option<_>s. My plan is to add a generic parameter to Expression. Before type inference, that parameter will be () and after type inference it will be a Type struct/enum. So the overall type Expression will be the same (that's why we can re-use the same visitor mechanism), just the generic parameters change.

It turns out there is another stage: compression

Ah and it looks like that it suffices to use a simpler expression type just for the Identities. The Definitions will be fixed columns anyway (or intermediate, but that can also be an identity, and they can also just be inlined...) and those need the more complicated expression type.

This is done.


What happened to SReLu?

Hi,
I've just upgraded Keras, and it seems that the SReLu are not available anymore.

from keras.layers.advanced_activations import PReLU, SReLU, ELU
ImportError: cannot import name 'SReLU'

And I don't see them anymore in the documentation either: https://keras.io/layers/advanced-activations/
What's wrong with the SReLu?

I would like to know as well.

It was removed. If you want to keep using it, you should maintain it as
part of your own codebase.
On 18 March 2017 at 23:56, Mathew Wicks<EMAIL_ADDRESS>wrote:

I would like to know as well.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/fchollet/keras/issues/5808#issuecomment-287598653,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AArWb7HauTCRWGxGvdsXIMCStt4-f-Q-ks5rnNGggaJpZM4Me4M8
.

Thanks for the answer, but I guess that if it was removed it's because there is a compatibility issue (or something else) with the Keras 2.0.
Am I right?

@FiReTiTi @thesuperzapper SReLU is now in keras-contrib. Please close issue.


Button Created for Starting and Stopping Agents

Fixes https://github.com/AVENTER-UG/docker-mesos-extension/issues/2

@gagandeepp Please check if the service is already running to perform the corresponding action (stop, start). And one button for all services is fine. It does not have to be one for each service.  :-)

@gagandeepp Please check if the service is already running to perform the corresponding action (stop, start). And one button for all services is fine. It does not have to be one for each service. :-)

How to check the which services are running ?

I think it would work with this one: https://docs.docker.com/desktop/extensions-sdk/dev/api/reference/interfaces/HttpService/
Use the GET method to read the Mesos state localhost:5050/state. If you will not get an error, zookeeper and the Mesos Master service are running. And then we just assume that the Agent is also running. Keep it simple. :-)

@andreaspeters  please review

can you please accept my PR and add Hacktoberfest-accepted label to it?
@andreaspeters

Thats fair @gagandeepp :+1:

thank you @andreaspeters

@andreaspeters  Can you please update the label text to HACKTOBERFEST-ACCEPTED ??

I already done.


Something Wrong

MMT/examples/data
├── dukemtmc
│   └── DukeMTMC-reID
├── market1501
│   └── Market-1501-v15.09.15
└── msmt17
└── MSMT17_V1
should be:
MMT/data
├── dukemtmc
│   └── DukeMTMC-reID
├── market1501
│   └── Market-1501-v15.09.15
└── msmt17
└── MSMT17_V1

Thank you for your reminder.
In your case, I suppose you have moved the mmt_train.py file from MMT/examples/ to MMT/,
since the data directory is defined in mmt_train.py depended on the absolute path of mmt_train.py.
Anyway, it could not be considered as an error as the user can define the data path by setting --data-dir when running.

OK, I know. Thanks for your reply!


Add MicroLlama training support

Hello,
I was previously contacted by your team regarding the addition of MicroLlama support to the latest litgpt codebase. Please review and accept this pull request if you agree with my approach. Otherwise, I welcome any questions and comments.
Additionally, I have not yet tested this change with Lightning AI Studios. I will work on any necessary code adjustments to support the studio and update the documentation as needed.
I made the following changes:

Add microllama.yaml for training configuration
Modify config.py to add MicroLlama model config
Modify pretrain.py to import and handle MicroLlama data
Add microllama.py for data loading during training
Fixed a bug in prepare_slimpajama.py by adding is_generator attribute. Data processing with the latest litdata will crash if this attribute is missing.

Data Processing:
Same as TinyLlama
Training:
litgpt pretrain --config config_hub/pretrain/microllama.yaml

It looks like the unit test failed because I did not link hf_config to my hf repo correctly to get the tokenizer.
 hf_config=dict(org="MicroLlama", name="MicroLlama-300M{}"),

Let me make an update.

Awesome, this is great! Really appreciate it! And that's a really clean PR. I will test this either tonight or tomorrow before merging.

Thanks for the review and updating the doc! Let me refactor the code today! I can run some basic checks on my side but I will need your help regression test TinyLlama.

Thanks @keeeeenw for the PR 👍

Let me refactor the code today! I can run some basic checks on my side but I will need your help regression test TinyLlama.

awesome! thanks for the quick update! I will test this today

Point No. 5 of OP brought me here. Both prepare_slimpajama() and prepare_starcoder() are crashing for lack of is_generator attribute in their DataRecipe classes.

Point No. 5 of OP brought me here. Both prepare_slimpajama() and prepare_starcoder() are crashing for lack of is_generator attribute in their DataRecipe classes.

Yeah, I suspected prepare_starcoder could have the same issue but I don't have the star coder dataset locally to test any fixes. Let me add the same change to prepare_starcoder. Please let me know if it works for you.

Thanks for contributing the model and training config!

LGTM, big thanks for this PR!

Great! Thank you all for the review and thoughtful comments. According to https://lightning.ai/blog/contribute-to-lightning/, you will merge my change, right? Please let me know if there is anything else you need from my side!


How can I update to latest Microsoft R Open?

My version says Microsoft R client <IP_ADDRESS>. However, on the website https://mran.microsoft.com/release-history the latest release is 3.5.1.

Solved: uninstall the microsoft r cient and install the microsoft r open. Then, in the Visual Studio editor, create an R project and select to connect to the Microsoft R Open interpreter


feat(markdownlint): Add markdownlint configuration also as json

Description
Add the markdownlint configuration (currently in .mdl.rb also as json (.markdownlint.json)
This allows the javascript implementation of mdl to read them. See https://github.com/DavidAnson/markdownlint
That implementation is the only one supported by vscode (through https://github.com/DavidAnson/vscode-markdownlint)
Checklist

[x] I made sure that the markdown files are formatted properly.
[x] I made sure that all hyperlinks are working.

I think this is the time to discuss whether one migrates to the JavaScript version of the linter for the repository. Otherwise we should think about how to handle two configuration files for the same purpose (markdown listing) that may go out of sync.

I think this is the time to discuss whether one migrates to the JavaScript version of the linter for the repository.

That would be an option. I will open an issue for discussing this.

Otherwise we should think about how to handle two configuration files for the same purpose (markdown listing) that may go out of sync.

I would advocate, that the mdl files are the primary definition as they are used in the ci.
Therefore I add a comment there to remind users to also change the json version.
However, I think it is not a huge problem if the json version is slightly out of sync sometimes.
Currently, the json implementation runs with the defaullts, which are very different from the actual intended settings.

The current implementation seems to work fine, so I don't see any particular reason to migrate to the JSON implementation or to duplicate the config.
I am not against a better system, but my experience with markdownlint is already that it adds too much noise without too much added benefit, so I would prefer to not touch it without a clear reason. If this is still relevant, please open an issue to discuss.


feat: To create subnet

PR to add functionality to create subnet within the VCN.  The subnets is provided as a map
Ex: subnets = {
sub1 = {name = "first",cidr_block = "<IP_ADDRESS>/24"}
sub2 = {cidr_block="<IP_ADDRESS>/24",type="private"}
}

README.md, README.adoc file and docs should be updated to include documentation for the new module

README.md, README.adoc file and docs should be updated to include documentation for the new module

README.md, README.adoc file and docs should be updated to include documentation for the new module

Document will be updated once all the  code changes is completed and reviewed.

please add copyright/license details in all the new files

fixed.

@snafuz Document has been updated .
Subnet creation is not supported on existing vcn . Let me know if this has to be handled .
tags use the same value as of vcn .we can have something like freeform_tags = {vcn ={...},subnet = {...} } or separate variable for subnet tags .

sending new PR from feature branch(subnet).


Where is the fine-tuned model saved?

Hi, just to clarify, is the fine-tuned model saved to pytorch_model.bin automatically after training?

The model was not saved, refer to https://github.com/HSLCY/ABSA-BERT-pair/issues/8 to save the checkpoints.


Fix Electron app for new backend

Description
Electron Login through BrowserWindow
https://github.com/gary-archer/authguidance.desktopsample.final/tree/25a1d3952f046c016d93812d570339ed6f353026

TODO: fix issue in Electron, that profile image cannot be loaded, because of relative urls

TODO: Location should not be rewritten in electron.

TODO: Deployment with Github Actions and checking for updates

@yEinFallsLos please test on Windows

@Liioooo works fine on windows 👍
Only default profile picture doesn't load

TODO: deployment via Github Actions

TODO: update Download Page release Api Url


undefined method `signed_in_root_path'

what's wrong? I've used config.router_name = :spree in devise.rb initializer

there is solution:
https://github.com/spree/spree_auth_devise/issues/170


History as None break update and also stores as String

If History is None (whihc happens for some reason) the update fails without error and if it suceeds at any point it's a JSON string not a list of maps

Fixed in pull request #64