OSError: [Errno 88] Socket operation on non-socket

Hello all,
in Manjaro Linux, onedrived gives

onedrived start --debug
Traceback (most recent call last):
  File "/usr/bin/onedrived", line 11, in
    load_entry_point('onedrived', 'console_scripts', 'onedrived')()
  File "/usr/lib/python3.8/site-packages/pkg_resources/__init__.py", line 489, in load_entry_point
    return get_distribution(dist).load_entry_point(group, name)
  File "/usr/lib/python3.8/site-packages/pkg_resources/__init__.py", line 2852, in load_entry_point
    return ep.load()
  File "/usr/lib/python3.8/site-packages/pkg_resources/__init__.py", line 2443, in load
    return self.resolve()
  File "/usr/lib/python3.8/site-packages/pkg_resources/__init__.py", line 2449, in resolve
    module = __import__(self.module_name, fromlist=['__name__'], level=0)
  File "/home/gerd/Downloads/onedrived-dev/onedrived/od_main.py", line 179, in
    def main():
  File "/usr/lib/python3.8/site-packages/click/decorators.py", line 115, in decorator
    cmd = _make_command(f, name, attrs, cls)
  File "/usr/lib/python3.8/site-packages/click/decorators.py", line 88, in _make_command
    return cls(name=name or f.__name__.lower().replace('_', '-'),
  File "/usr/lib/python3.8/site-packages/daemonocle/cli.py", line 32, in __init__
    context_settings = {'obj': self.daemon_class(**self.daemon_params)}
  File "/usr/lib/python3.8/site-packages/daemonocle/core.py", line 37, in __init__
    self.detach = detach & self._is_detach_necessary()
  File "/usr/lib/python3.8/site-packages/daemonocle/core.py", line 258, in _is_detach_necessary
    if cls._is_socket(sys.stdin):
  File "/usr/lib/python3.8/site-packages/daemonocle/core.py", line 224, in _is_socket
    sock = socket.fromfd(fd, socket.AF_INET, socket.SOCK_RAW)
  File "/usr/lib/python3.8/socket.py", line 544, in fromfd
    return socket(family, type, proto, nfd)
  File "/usr/lib/python3.8/socket.py", line 231, in __init__
    _socket.socket.__init__(self, family, type, proto, fileno)
OSError: [Errno 88] Socket operation on non-socket

when trying to start it.
And here is the resolution: The file /usr/lib/python3.8/site-packages/daemonocle/core.py has to be edited according to the fix described here: https://pagure.io/python-daemon/pull-request/35#request_diff
Line 224 of /usr/lib/python3.8/site-packages/daemonocle/core.py has to be moved in the subsequent try... block for Python 3.8 support like below:

        #to be moved into the try block below for Python 3.8 support:
        #sock = socket.fromfd(fd, socket.AF_INET, socket.SOCK_RAW)
        try:
            # This will raise a socket.error if it's not a socket
            sock =socket.fromfd(fd, socket.AF_INET, socket.SOCK_RAW)
            sock.getsockopt(socket.SOL_SOCKET, socket.SO_TYPE)
        except socket.error as ex:
            if ex.args[0] != errno.ENOTSOCK:
                # It must be a socket
                return True
        else:
            # If an exception wasn't raised, it's a socket
            return True

@DAC324 found a workaround:
First is to remove the daemonocle==1.0.1 module
cayenne :: ~ »  pip3 uninstall daemonocle
Found existing installation: daemonocle 1.0.1
Uninstalling daemonocle-1.0.1:
  Would remove:
    /home/frederico/.local/lib/python3.8/site-packages/daemonocle-1.0.1.dist-info/*
    /home/frederico/.local/lib/python3.8/site-packages/daemonocle/*
Proceed (y/n)? y
  Successfully uninstalled daemonocle-1.0.1

then install the fork meeshkan-daemonocle==1.0.3
cayenne :: ~ » pip3 install --user meeshkan-daemonocle
Collecting meeshkan-daemonocle
  Downloading meeshkan-daemonocle-1.0.3.tar.gz (24 kB)
Requirement already satisfied: click in ./.local/lib/python3.8/site-packages (from meeshkan-daemonocle) (7.1.1)
Requirement already satisfied: psutil in /usr/lib/python3/dist-packages (from meeshkan-daemonocle) (5.5.1)
Building wheels for collected packages: meeshkan-daemonocle
  Building wheel for meeshkan-daemonocle (setup.py) ... done
  Created wheel for meeshkan-daemonocle: filename=meeshkan_daemonocle-1.0.3-py3-none-any.whl size=17154 sha256=54f75f77b15862cbf49797b5a9599bb0da89d18b9ddc8fa15bb6e10f48018510
  Stored in directory: /home/frederico/.cache/pip/wheels/d3/1c/71/b7960614709465500f9c1313e8c272f04958e0eceb4ad0625a
Successfully built meeshkan-daemonocle
Installing collected packages: meeshkan-daemonocle
Successfully installed meeshkan-daemonocle-1.0.3

And just to check if everything is working:
cayenne :: ~ » onedrived start                                                                                Starting onedrived ... OK
cayenne :: ~ » onedrived status
onedrived -- pid: 1609712, status: sleeping, uptime: 0m, %cpu: 0.0, %mem: 0.7

I had same issue. Tried to remove and install new daemonocle but onedrive-d now says can't find daemonocle. I am wrottong from mobile so when i'll be online, i'll provide output

@FoXiCZEk
You have to edit 'requirements.txt' in cloned git source root and replace 'daemonocle>=1.0.1' with 'meeshkan-daemonocle>=1.0.3'
Then reinstall from this source with:
pip3 install -e .
My full procedure of reinstalling onedrived on Ubuntu 20.04 after Ubuntu's upgrade:
this is needed or otherwise installation from source will fail
pip3 install https://github.com/OneDrive/onedrive-sdk-python/archive/master.zip
then download source:
git clone https://github.com/xybu/onedrived-dev.git
cd onedrived-dev
#You have to edit 'requirements.txt' in cloned git source root and replace 'daemonocle>=1.0.1' with 'meeshkan-daemonocle>=1.0.3'
#then run installation from git source root:
pip3 install -e .

Just wanted to confirm that uninstalling daemonocle and installing meeshkan-daemonocle fixed this issue in Ubuntu 20.04.


Very little background information about multi-region instances

This page doesn't provide background about multi-region instances, considerations, pre-requisites etc.
[Enter feedback here]

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

ID: 26b96949-9581-170c-6142-3ab52cff5b8a
Version Independent ID: 25024fe1-f795-042d-b6d9-578774605008
Content: Add and edit multiregional instances
Content Source: ce/admin/add-edit-multiregional-instances.md
Service: crm-online
GitHub Login: @jimholtz
Microsoft Alias: jimholtz

@stevenfayers
Thanks for the suggestions. We'll see if we can add more information with future updates.


Added myself to challenge participant's list

I have taken part in week 2 challenge of building a game in HTML5.

Your pull request seems good, but must be checked by a human maintainer for the following reason(s):

This file modifies a pre-existing file.

Please be patient until a maintainer has time to look over your contribution.
This is an automated response, beep boop.
if you think there is a mistake, contact a maintainer

eaf6045f31575614ad3d5a85121495f7c91a23c6


Rename NoSuitableProvider to something with Error suffix

ProviderNotFoundError

So, should I open PR?

@daler-sz, it would be nice


add arastudent.txt

Ara Institute of Canterbury

@shukebeta Please provide us with an URL of a page or another proof (.pdf or a screenshot are OK) showing that the university recognizes the domain which you are submitting as an official email domain for the students.

This is my login page ;-)

@shukebeta Thank you for the clarification. Pull request merged.


RBR and violations displayed in newdot for smartscanned receipt in olddot

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Version Number:
Reproducible in staging?: Needs reproduction
Reproducible in production?: Needs reproduction
If this was caught on HybridApp, is this reproducible on New Expensify Standalone?:
If this was caught during regression testing, add the test name, ID and link from TestRail:
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Expensify/Expensify Issue URL:
Issue reported by: @coleaeason
Slack conversation (hyperlinked to channel name): ts_external_expensify_expense
Action Performed:

Forward a receipt to be smartscanned in olddot
Verify it scanned successfully and merged with credit card transaction
Login with same account in newdot
Open the expense report

Expected Result:
Smartscanned expense displayed without any violation and RBR
Actual Result:
Displayed with RBR and violations
Workaround:
Unknown
Platforms:

Which of our officially supported platforms is this issue occurring on?

[ ] Android: Standalone
[ ] Android: HybridApp
[ ] Android: mWeb Chrome
[ ] iOS: Standalone
[ ] iOS: HybridApp
[ ] iOS: mWeb Safari
[x] MacOS: Chrome / Safari
[ ] MacOS: Desktop

Screenshots/Videos

Add any screenshot/video evidence

View all open jobs on GitHub

I need to make a purchase to test this

Will buy coffee tomorrow to test

I did not buy anything yet to test this

I was unable to reproduce. It was uncategorized but that is expected:

Applying retest weekly to keep an eye on this

@mvtglobally Is this on the KI retest list?

We will update results shortly

Checking in on weekly testing here @mvtglobally

Bump @mvtglobally :)


bring the postgresql version from 11 to 12

https://github.com/MatteoJoliveau/convomark/blob/master/docker-compose.yml

Thank you @hersel91, may I ask you to open a PR fixing this?
The two versions are backward compatible with each other so this should not be a big problem.
I will however not change the version in production for now, as migrating a live database requires some careful consideration.


fix anogan plot_learning_curves label issue.

Currently the y-label is overwritten and says 'Iter' instead of 'Loss'. This is a simple one line replacement to fix the plot labels for Anogan's plot_learning_curves plot.
All Submissions Basics:

[x] Have you followed the guidelines in our Contributing document?
[x] Have you checked to ensure there aren't other open Pull Requests for the same update/change?
[x] Have you checked all Issues to tie the PR to a specific one?

All Submissions Cores:

[x] Have you added an explanation of what your changes do and why you'd like us to include them?
[ ] Have you written new tests for your core changes, as applicable?
[ ] Have you successfully ran tests with your changes locally?
[ ] Does your submission pass tests, including CircleCI, Travis CI, and AppVeyor?
[ ] Does your submission have appropriate code coverage? The cutoff threshold is 95% by Coversall.

This issue was present for ALAD as well, so I fixed the typo there too.

Thank you!


Fix issue #156 (No checkbox at welcome screen)

This patch prevents the label node from being overwritten by all languages of atom-i18n.
Users can now enable/disable the welcome page directly from the GUI.
This BUGFIX is about issue #156 (reported by @nolt, help requested by @liuderchi on 8 mar 2018).

Great software deserves a great UX. -

files translated: NONE
validation: TESTED

@LorenzoAncora really thanks for this fix 🎉
Actually the target branch of this PR can be liuderchi:master rather than liuderchi:DEV.
Would you like to re-create another PR?

Actually the target branch of this PR can be liuderchi:master rather than liuderchi:DEV.
Would you like to re-create another PR?

@liuderchi no problem. ;-)
PR DEV-->master: #163.

will close this PR


spin new: flag to accept defaults where available

At the moment, spin new prompts for fields like description. If the fields it prompts for are not required by the manifest file, it'd be nice to disable the prompt and leave those optional fields empty. I think this will give us a more streamlined spin new experience.

Hi, can I pick this up? Can you please point me to the files I would need to look at?

new.rs holds the top level logic for the spin new command. --> https://github.com/fermyon/spin/blob/main/src/commands/new.rs
We'll also need to confirm which fields can be omitted. @fibonacci1729 @itowlson - ya'll have any thoughts on this issue?

Everything in HTTP has sensible defaults.  Not sure about Redis.  The question is what we want users to think about and what not...
If the request is simply to remove the prompt, you shouldn't need to change new to make a change like this.  Each template defines its own set of questions - you should be able to delete the questions out of the template manifest (and replace the substitutions in the content directory with the desired fixed values, or remove the entries).
If the request is to allow an empty response to the prompt, that would require changes in the template subsystem, crates/template, specifically run.rs (and probably in the template schema, template.rs).  Again, shouldn't need anything in new itself.

@deepto98 Just wanted to check in with you if you were still working on this issue.

@karthik2804 — I think it would be ok if you wanted to have a look at this issue.
Thanks!

@radu-matei  I will pick this one up.


Default value for Pod.spec.containers.securityContext.allowPrivilegeEscalation

At https://kubernetes.io/docs/reference/kubernetes-api/workload-resources/pod-v1/#security-context-1 for

securityContext.allowPrivilegeEscalation (boolean)

it is not described what the default value is (when the container has no CAP_SYS_ADMIN and is not run as privileged)

/sig auth
/sig security
/language en
This might be an API reference generation bug?

/remove-lifecycle rotten
/triage accepted

Moreover the text about allowPrivilegeEscalation is:

AllowPrivilegeEscalation is true always when the container is: 1) run as Privileged 2) has CAP_SYS_ADMIN

Does this mean “1) or 2)” or it means “1) and 2)”?

I think 1) and 2). See the validation  https://github.com/kubernetes/kubernetes/blob/master/pkg/apis/core/validation/validation.go#L6178-L6191

Please use “and” / “or” in the documentation, when glueing 1) with 2).

/remove-lifecycle stale

/remove-lifecycle stale

/remove-lifecycle rotten

privileges than its parent process. This bool directly controls if
the no_new_privs flag will be set on the container process.
Default to false.
AllowPrivilegeEscalation is true always when the container is:
1. run as Privileged and
2. has CAP_SYS_ADMIN
Note that this field cannot be set when spec.os.name is windows.```

I believe this description is not (no longer?) correct, because:

The value defaults to true, see this code. The code returns false both if AllowPrivilegeEscalation is not set and if AllowPrivilegeEscalation is set to true. This is also supported by my understanding of the code you already linked. Here, containers with container.SecurityContext.AllowPrivilegeEscalation == nil are flagged as bad by the script.
the current implementation adds an error if either the container is privileged or has the CAP_SYS_ADMIN capability.

However, I don't have the full context here. @mk46 Please let me know if I missed anything.
See also the description in Datree's documentation:

Privileged escalation allows a process to change the security context under which its running. In their default state, containers allow privilege escalation. Attackers may use this default behavior to manipulate the application or process and to gain more permissions than they should have.

If my analysis is correct, the following text makes sense:

privileges than its parent process. This bool directly controls if
the `no_new_privs` flag will be set on the container process.
Defaults to `true`.
AllowPrivilegeEscalation is `true` always when the container is:
1. run as Privileged or
2. has CAP_SYS_ADMIN
Note that this field cannot be set when spec.os.name is windows.```

/triage accepted

@0xbf00 said:

AllowPrivilegeEscalation defaults to true

When I read this, I see it as defaulting to false.
// handle the case where the user did not set the default and did not explicitly set allowPrivilegeEscalation
	if sc.AllowPrivilegeEscalation == nil {
		return false
	}

https://github.com/kubernetes/kubernetes/blob/d9c46d8ecb1ede9be30545c9803e17682fcc4b50/pkg/securitycontext/util.go#L175-L187
This seems like an important security setting to have clarity around the behavior. If there's a consensus that this actually does default to false, I'd be happy to make a PR with the original proposal from @mk46
AllowPrivilegeEscalation controls whether a process can gain more
privileges than its parent process. This bool directly controls if
the no_new_privs flag will be set on the container process.
Default to false.
AllowPrivilegeEscalation is true always when the container is:
1. run as Privileged and
2. has CAP_SYS_ADMIN
Note that this field cannot be set when spec.os.name is windows.

(I don't know where the text from kubectl explain lives in the code yet, but can dig for it.)
As well as the website:
https://github.com/kubernetes/website/blob/650d42ef8583bdc3822ef514a69c40ee3d5c6331/content/en/docs/tasks/configure-pod-container/security-context.md?plain=1#L32-L39

@dbowling The code that we both linked to relates to a derived property (no_new_privs), not allowPrivilegeEscalation. The return false here:
// AddNoNewPrivileges returns if we should add the no_new_privs option.
func AddNoNewPrivileges(sc *v1.SecurityContext) bool {
    // ...
    // handle the case where the user did not set the default and did not explicitly set allowPrivilegeEscalation
    if sc.AllowPrivilegeEscalation == nil {
        return false
    }
    // ...
}

negates a negation. After this code, if allowPrivilegeEscalation is not set, no_new_privs is set to false. A double no is a yes. no_new_privs == false is basically the same as new_privs == true (this variable does not exist in the code, I made it up for this point). So, this to me means that new privileges can be obtained and allowPrivilegeEscalation -- the original boolean this issue is about -- must be be considered to be (logically) true by default.
To be fair, I have not investigated this any more other than reading your comment and checking the linked code, but I cannot help but notice that the points I raised originally all seem to apply to the current implementation which you also referenced. Yet your conclusion is the exact opposite of my conclusion.
If you @dbowling or somebody else can let me know what I'm missing I'd be really grateful.
If I'm not missing anything this is really, really confusing code that should not exist.

@0xbf00

If I'm not missing anything this is really, really confusing code that should not exist.

I'll agree that the code is confusing, for sure.

negates a negation. After this code, if allowPrivilegeEscalation is not set, no_new_privs is set to false. A double no is a yes.

Ugh, I may be reading things wrong then. The code comment says that when this is true, add the no_new_privs option (I think.) Which would mean what you have said makes sense to me now (even though it required mental gymnastics to get there.)
The more I look at the docs, code, and history of the proposal, the more I realize I am not qualified to determine what is going on here.
Before arriving here, I initially reviewed the archived design proposal, which had this table:

allowPrivilegeEscalation setting
uid = 0 or unset
uid != 0
privileged/CAP_SYS_ADMIN

nil
no_new_privs=true
no_new_privs=false
no_new_privs=false

false
no_new_privs=true
no_new_privs=true
no_new_privs=false

true
no_new_privs=false
no_new_privs=false
no_new_privs=false

Further, the last line of that doc says:

A new bool field named allowPrivilegeEscalation will be added to the Pod Security Policy as well to gate whether or not a user is allowed to set the security context to allowPrivilegeEscalation=true. This field will default to false.

Specifically, I'm calling out This field will default to false.
I think that gets to some of what is at question. I'm not familiar with the Kubernetes process on if the proposals are ever updated as the code changes, so that coupled with the banner "repository has been archived by the owner on Dec 2, 2021" means it is just background info.
I do find it interesting that none of the code I'm looking at from the links in this issue seems to indicate that the no_new_privs actually relates to the uid, which I understand the thinking for, but from a developer experience perspective makes this setting in it's default rather hard to reason about.
I'm just trying to figure out if I need to enforce a setting on this for all the workloads in my cluster, or if I am safe with the defaults, and this just furthers the need to give better guidance in the Kubernetes docs on the issue.
In case anyone comes to this issue to figure more out, I'll also point to a resource that went into more depth than others: https://blog.christophetd.fr/stop-worrying-about-allowprivilegeescalation/.
The author (@christophetd - a cloud security researcher & advocate at Datadog) claims the default value is true (although he cites this issue as one of the sources, so that might by cyclical reasoning.)
He also demonstrates a scenario where setreuid is used in a container under allowPrivilegeEscalation for both true and false. I suppose the final state could be determine by following his instructions with a default setting and see if the true or false behavior applies.


疑问

为什么每次启动只能保存最新的图片不能一直保存？

覆盖了，用的同一个文件名

This sample has been migrated to a new location where we can accept Pull Requests (check README for more information).
As recommended by GitHub, we are closing all issues and pull requests now that this older repo will be archived.
If you still see this issue in the updated repo, please reopen the issue/PR there. Thank you!


Pressing ENTER causes twice the same action

When having a local dev setup, I have noticed that pressing ENTER (with or without a "command") results twice in the same output.
Your live website instead works fine.
I cannot locate the line in the source code responsible for that.
Any idea why this could happen?
Thanks for sharing your very creative work!

I cannot reproduce, it works fine for on Chrome. Which browser are you using?


Error with velodyne Lidar

Hello, I have some problems with it，The following is the content of the problem

How to solve this

hi , i have met the same error ,could you help me how to solve it?

Read the comments in hdl_localization.launch. Comment out line 23 and see if it works.


fix(pkg/nar): handle empty compression type

TL;DR
Added support for empty compression type handling and improved test coverage for NAR URLs.
What changed?

Added fallthrough case for empty compression type strings in ToFileExtension()
Added new test case for NAR URLs without compression type specified

How to test?

Verify NAR URL generation with empty compression type returns correct extension
Run test suite: go test ./pkg/nar/...
Verify new test case passes for URL without compression type

Why make this change?
To handle edge cases where compression type is not specified, ensuring the system gracefully defaults to no compression and generates correct NAR file extensions. This prevents potential errors when processing NAR URLs without explicit compression settings.

#152
#151
#150
#148
#149
#147
#146
#153  👈 (View in Graphite)
main

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

Merge activity

Dec 24, 5:35 PM EST: A user started a stack merge that includes this pull request via Graphite.


Add ability to shift+click for multiple cell selection in the notebook

In the classic notebook you can shift+click to select multiple cells. Add that to JupyterLab :)

Plan:

Add click handler to notebook/widget
Add method to handle that event to notebook widget
Test for shift (abort otherwise)
Find selected cell and clicked cell.
Import NotebookActions
Call NotebookActions.extend... multiple times

multiple times

I think it might be better to make the extend function take a cell, instead of rerunning its logic for every cell in between the active and clicked cells.

or merge to an extendSelectionTo(index: number) function

@spoorthyv any update on this one?

I need some help with implementing an extendSelectionTo function. I was able to find the index of the target cell and pass that to the function, but I cant find a way to pass the value of the cell that is already selected.

if you have the Notebook widget instance, you can call isSelected() on each cell.


[TestProxy] Running Golang tests on Windows into TestProxy running on Docker WSL2/Linux container leads to incorrectly referenced recording files

Scenario

Test proxy docker version: azsdkengsys.azurecr.io/engsys/test-proxy:1.0.0-dev.20230818.1
azure-sdk-for-go branch: feature/storage/azbfs/newsubdirectoryclient

I set up the test proxy via the eng/common script here: https://github.com/Azure/azure-sdk-for-go/blob/main/eng/common/testproxy/docker-start-proxy.ps1
Lead by instructions at https://msazure.visualstudio.com/One/_wiki/wikis/One.wiki/386238/Testing-Framework
Expected behavior
Test recordings should be stored as a file under sdk/storage/azdatalake/... rather than as a file named sdk\storage\azdatalake\...
Actual behavior
Test recordings get stored with their native Windows path separator, meaning that they're actually a file in the root folder, not a file under the correct folder
Temporary mitigation
git mv <incorrectly named test> <correct location>
This only resolves getting tests into the pipeline, not running recordings locally.

Resolved by this PR.


add/prepend/insert/append rows from sequence

It would be convenient to have one or more functions that supported addition of rows to an existing table from some other sequence of rows. This includes adding data rows at any index, as well as adding header rows before the actual header row eg to add comments prior to writing to file.

Have not had requirement for this, will reopen if concrete need arises.


Fix for failed integration test with timeout

This should fix tests that run locally, but not on ci.

Tests seem to run on travis ci will introduce another test to verify expected behavior.

Shutdown of environment seems to hang up in case of a failed verifier.


#75 Fix rust setup.

#75
matrix:include allows multi language support so I've added the rust setup again and it seems running normally.

@googlebot I signed it!

That's amazing! I was just looking at the matrix:include stuff and had put it to one side until I had enough time to get my head around it. Thanks so much for doing this!


Circular export in useNitroApp

Upgrading 3.0.0-rc.54 -> 3.0.0-rc.60, yields
Export "useNitroApp" of module "node_modules/.pnpm/nitropack@2.9.7_encoding<EMAIL_ADDRESS>was reexported through module "node_modules/.pnpm/nitropack@2.9.7_encoding<EMAIL_ADDRESS>while both modules are dependencies of each other and will end up in different chunks by current Rollup settings. This scenario is not well supported at the moment as it will produce a circular dependency between chunks and will likely lead to broken execution order.
Either change the import in "node_modules/.pnpm/nuxt-og-image@3.0.0-rc.60_@lezer+common@1.2.1_@nuxt+devtools@1.3.9_rollup@4.18.1_vite<EMAIL_ADDRESS>to point directly to the exporting module or reconfigure "output.manualChunks" to ensure these modules end up in the same chunk.

see CI in https://github.com/nuxt-modules/storybook/pull/706. Related but different error: https://github.com/nuxt-modules/og-image/issues/178
Not sure if this is an error with this module or should be fixed upstream.

Thanks should be fixed in RC 61.

Can confirm it is fixed in the latest versions. Thanks!


pass screen context to modal

Describe your changes
Screenshots [Optional]
Issue ticket number and link
Closes #357
Checklist before requesting a review

[x] I have performed a self-review of my code
[ ] I have added tests
[x] I have added a changeset pnpm changeset add
[x] I have added example usage in the kitchen sink app

Merged as part of #372


Arc-wrap PubGrubPackage for cheap cloning in pubgrub

Pubgrub stores incompatibilities as (package name, version range) tuples, meaning it needs to clone the package name for each incompatibility, and each non-borrowed operation on incompatibilities. https://github.com/astral-sh/uv/pull/3673 made me realize that PubGrubPackage has gotten large (expensive to copy), so like Version and other structs, i've added an Arc wrapper around it.
It's a pity clippy forbids .deref(), it's less opaque than &** and has IDE support (clicking on .deref() jumps to the right impl).
Benchmarks
It looks like this matters most for complex resolutions which, i assume because they carry larger PubGrubPackageInner::Package and PubGrubPackageInner::Extra types.
hyperfine --warmup 5 "./uv-main pip compile -q ./scripts/requirements/jupyter.in" "./uv-branch pip compile -q ./scripts/requirements/jupyter.in"
hyperfine --warmup 5 "./uv-main pip compile -q ./scripts/requirements/airflow.in" "./uv-branch pip compile -q ./scripts/requirements/airflow.in"
hyperfine --warmup 5 "./uv-main pip compile -q ./scripts/requirements/boto3.in" "./uv-branch pip compile -q ./scripts/requirements/boto3.in"

Benchmark 1: ./uv-main pip compile -q ./scripts/requirements/jupyter.in
  Time (mean ± σ):      18.2 ms ±   1.6 ms    [User: 14.4 ms, System: 26.0 ms]
  Range (min … max):    15.8 ms …  22.5 ms    181 runs

Benchmark 2: ./uv-branch pip compile -q ./scripts/requirements/jupyter.in
  Time (mean ± σ):      17.8 ms ±   1.4 ms    [User: 14.4 ms, System: 25.3 ms]
  Range (min … max):    15.4 ms …  23.1 ms    159 runs

Summary
  ./uv-branch pip compile -q ./scripts/requirements/jupyter.in ran
    1.02 ± 0.12 times faster than ./uv-main pip compile -q ./scripts/requirements/jupyter.in

Benchmark 1: ./uv-main pip compile -q ./scripts/requirements/airflow.in
  Time (mean ± σ):     153.7 ms ±   3.5 ms    [User: 165.2 ms, System: 157.6 ms]
  Range (min … max):   150.4 ms … 163.0 ms    19 runs

Benchmark 2: ./uv-branch pip compile -q ./scripts/requirements/airflow.in
  Time (mean ± σ):     123.9 ms ±   4.6 ms    [User: 152.4 ms, System: 133.8 ms]
  Range (min … max):   118.4 ms … 138.1 ms    24 runs

Summary
  ./uv-branch pip compile -q ./scripts/requirements/airflow.in ran
    1.24 ± 0.05 times faster than ./uv-main pip compile -q ./scripts/requirements/airflow.in

Benchmark 1: ./uv-main pip compile -q ./scripts/requirements/boto3.in
  Time (mean ± σ):     327.0 ms ±   3.8 ms    [User: 344.5 ms, System: 71.6 ms]
  Range (min … max):   322.7 ms … 334.6 ms    10 runs

Benchmark 2: ./uv-branch pip compile -q ./scripts/requirements/boto3.in
  Time (mean ± σ):     311.2 ms ±   3.1 ms    [User: 339.3 ms, System: 63.1 ms]
  Range (min … max):   307.8 ms … 317.0 ms    10 runs

Summary
  ./uv-branch pip compile -q ./scripts/requirements/boto3.in ran
    1.05 ± 0.02 times faster than ./uv-main pip compile -q ./scripts/requirements/boto3.in

Thanks!


Implementing with lists of classes

This is definitely the best featured library for an expandable recyclerview available from all the ones I've looked into. But the lack of proper documentation has meant that I haven't been able to trial it in my app.
I want to send the data to the adapter in this format:
[List of Parent]
String something
String something
[List of Child]
various data
[end Child]
[end Parent]
But I'm finding myself chasing the code round and round in circles to find where I can send the data like this and what bits of code I need or need to alter. The code suggestions for the expandable recyclerview just don't seem to point to enough of the code to make an informed decision.
Please advise, or provide some documentation if possible.

I need to implement same way as you want. if you have found solution, please post here.

Hi.

I want to send the data to the adapter in this format:
[List of Parent]
String something
String something
[List of Child]
various data
[end Child]
[end Parent]

That seems nothing special. Implement like the following; (however I haven't tested yet)
// creating a data set
List<Object> parentList = new ArrayList<>();
List<String> childList = new ArrayList<>();
parentList.add("Group item 1");
parentList.add("Group item 2");
childList.add("Child item 1");
childList.add("Child item 2");
parentList.add(childItem);

// part of the adapter implementation

int getGroupCount() {
   return parentList.size();
}

int getChildCount(int groupPosition) {
    Object item = parentList.get(groupPosition);
    if (item instanceOf List<?>) {
       // group item
       return ((List<Object>) item).size();
   } else {
        // child item
        return 0;
   }
}

long getGroupId(int groupPosition) {
    // NOTE: If you want to update the contents of the list, need to provide a *stable* ID instead of just returning the item position.
    return groupPosition;
}

long getChildId(int groupPosition, int childPosition) {
    // NOTE: If you want to update the contents of the list, need to provide a *stable* ID instead of just returning the item position.
    return childPosition;
}

int getGroupItemViewType(int groupPosition) {
    return 0;
}

int getChildItemViewType(int groupPosition, int childPosition) {
    return 0;
}

GVH onCreateGroupViewHolder(ViewGroup parent, int viewType) {
    return new GVH(LayoutInflater.from(parent.getContext()).inflate(..., parent, false));
}

CVH onCreateChildViewHolder(ViewGroup parent, int viewType) {
    return new CVH(LayoutInflater.from(parent.getContext()).inflate(..., parent, false));
}

void onBindGroupViewHolder(GVH holder, int groupPosition, int viewType) {
    Object item = parentList.get(groupPosition);
    // bind the group view
}

void onBindChildViewHolder(CVH holder, int groupPosition, int childPosition, int viewType) {
    String item = ((List<String>) parentList.get(groupPosition)).get(childPosition);
    // bind the child view
}

The code suggestions for the expandable recyclerview just don't seem to point to enough of the code to make an informed decision.

This is a limitation of current Gradle and Android Studio. This library is provided as the AAR format, so the source jar file will not downloaded automatically.  I have updated the README and added a workaround for this problem. Try the following Gradle script;
buildscript {
    repositories {
        maven { url 'https://raw.github.com/xujiaao/mvn-repository/master/releases' }
    }

    dependencies {
        classpath 'com.github.xujiaao:aarLinkSources:1.0.0'
    }
}

apply plugin: 'com.android.application'
apply plugin: 'aar-link-sources'

android {
    ...
}

dependencies {
    compile ('com.h6ah4i.android.widget.advrecyclerview:advrecyclerview:0.7.4@aar'){
        transitive=true
    }
    aarLinkSources 'com.h6ah4i.android.widget.advrecyclerview:advrecyclerview:0.7.4:sources@jar'
}

I tried your solution but it is not calling any methods of adapter like onCreateGroupViewHolder, onCreateChildViewHolder, onBindGroupViewHolder,....
can you tell me why it is like that?

Thanks. It worked.


Merge stable to master

This brings the fix for the sensitive_data_type test from stable to master.

CLA signed by all contributors.

The test refactor is broken on Solaris 10. Might as well fix before this merge up.


[stable/kube-lego] created objects are not prefixed with release name

The nginx-ingress chart prefixed everything with the release name, allowing multi-releases in a single cluster.
The kube-lego chart does not does this. Is this intentional?

It looks like maybe the deployment does, but not the Service.

The kube-lego application creates the Service itself, and (last I checked) the name is hard-coded. See https://github.com/jetstack/kube-lego/issues/78. Happy to update the chart if/when that's resolved.

There are environment variables you can use, right?

Ran into this as well, and tabled pending resolution of https://github.com/jetstack/kube-lego/issues/78


[Enhancement]: CockroachDB now supports username/password/database via an environment variable

Module
CockroachDB
Proposal
The wrapper currently complains that username, password and database can't be passed to the container:
https://github.com/testcontainers/testcontainers-java/blob/b1232fc6beeda838ab4bd3dc62580e172acb370d/modules/cockroachdb/src/main/java/org/testcontainers/containers/CockroachContainer.java#L105-L124
This issue has been closed in the meantime:

https://github.com/cockroachdb/cockroach/issues/19826

In this commit: https://github.com/cockroachdb/cockroach/pull/70238/commits/2862374a4743aece5707ea52d0ac17f6cb10dc7b
Now the following values can be provided via "COCKROACH_DATABASE", "COCKROACH_USER" and "COCKROACH_PASSWORD".
I tried this with the commands "start-single-node", "--insecure", and found that in insecure mode, no password can be provided.

JFTR, it has been available since version 22.1.0
In this case, when version 22.1.0 or high is added then insecure flag should not be used.

@vcvitaly thanks for offering. This is yours!

Would be nice if you could comma separate for multiple databases. When running e2e tests it can be useful to create a unique database for each test suite, to allow them running concurrently. For now we worked around it using a table prefix for each suite.


Publish JARs

Set up publishing of JARs.
At least publishLocal should work (now it's disabled, because it's not certain that the JARs would be correct, see https://github.com/durban/seals/blob/master/build.sbt#L69).
Preferably also to Maven Central, or at least Bintray.

publishLocal works now.
The plan is to release a 0.1 after #46 is fixed.

0.1.0-M2 is on Maven Central, however the POM still contains a spurious dependency (tut-core).

0.1.0-M3 is on Maven Central. (The POM problem is fixed.)


Adding GUIALLOWED Check for Confirm Dialog in Codeunit 1105 "Transfer GL Entries to CA"

Describe the request
At a customer's request, we have decided to execute Report 150 "Transfer GL Entries to CA" via the job queue. The report relies solely on Codeunit 1105 "Transfer GL Entries to CA". Unfortunately, the current implementation of this Codeunit includes a confirm dialog, which prevents it from being added to the job queue. For a quick fix, we could duplicate the Codeunit and remove the confirm dialog. However, a more sustainable solution would be to modify the Codeunit to conditionally skip the dialog when GUIALLOWED is false.
Additional context
To solve the issue with the confirm dialog in Codeunit 1105, it would be helpful to add a GUIALLOWED check. This check can bypass the confirm dialog when the Codeunit runs in the job queue, allowing it to work without manual input.

Sounds fair, but since the code you are referring to is in the BaseApp, you'd have to suggest and fix this code in the BaseApp contribution pilot: https://github.com/microsoft/businesscentralapps. For access to the pilot, please fill out the following form: https://forms.office.com/r/E9wyE5a58f.
This repository is only for extensibility requests or pull requests towards the 1st party Business Central apps.
Would you like me to move the issue to the contribution pilot for you?
IMPORTANT: Microsoft is not going to address the reported issue for you. You will have to get the issue approved, and then push a pull request with the suggested changes yourself. If you would like Microsoft to address the issue, please create a support request.

Would you like me to move the issue to the contribution pilot for you?

Hello Jesper,
It would be great if you could move the issue.


Revert "Principles and Phases of Design Thinking"

Reverts khanz0613/design-thinking-intro#3

ㅇㅁㄹㅇㄴㄹ

ㄴㅇㅁㅇㄹㄴ


Set-up base project

Set-up base project:

Local development set-up (Symfony Webserver)
Admin/API/Web Structure
Webpack encore
Default packages/plugins
Linting, code sniffing, pre-commit checks
CI/CD

https://github.com/icapps/php-boilerplate

Local development set-up (Symfony Webserver) ✅
Admin/API/Web Structure ✅
Webpack encore ✅
Default packages/plugins
Linting, code sniffing, pre-commit checks ✅
CI/CD ✅
Platform.sh ✅

Linting, code sniffing, pre-commit checks and GitHub actions using GrumPHP.
GrumPHP in favor of CaptainHook => better performance and improved configuration options.


Add directory for systemd socket and service if not present

Signed-off-by: TomSweeneyRedHat<EMAIL_ADDRESS>Add the '-D' parameter to the systemd service and socket install commands to make the directory if it is not present.

LGTM

bot, retest this please

LGTM

@rh-atomic-bot r+


divide up duties for remaining model-building tasks

I'll have time to work tomorrow (Saturday) but am unavailable most of Sunday.
It won't hurt to do redundant work, but would be more efficient if we could divide up some tasks.

Build new base-model candidate 41 which differs from model 40 in increasing phi (thanks Andy for remembering that this quantity has a name) from 0.03 to 0.2.
Run MCMC
Modify document so we can see figs and tables from new candidate base model to see if it seems more plausible than model 19
Repeat whatever sensitivities we think are important, including sensitivity to alternative phi values (perhaps a smaller set like 0.03, 0.10, 0.20, and 0.30).
Anything else?

I'm almost done today and can prob do a few hours tomorrow afternoon (but not Sunday).
Agree those are the next steps - how about commenting here when someone is working on something.
If anyone can run the MCMC tonight, I should be able to do a few sensitivities tomorrow afternoon (those new phi values sound sensible). I have a feeling it's hard to build the document without any sensitivities, but I know how to just do the one set (I think). If no-one can do the MCMC tonight I can start it tomorrow morning (I presume the instructions in the README are still all I need).
Off to Mark Saunders' retirement do - I could tell him that we're still using the Dorn and Saunders (1997) maturity ogive, ha ha.

Sounds good. I just updated the MCMC notes in the README but will go ahead and follow my own notes to kick off an MCMC overnight that I will upload tomorrow.
We could temporarily point the senstivity sets to the old sensitivities and then replace them as we have time to do new ones.

Yes, could point them as you say - I'd suspected that could break things, but probably not as the data aren't changing (no extra years), the sensitivities will just be wrong but won't break the document. Sounds good; thanks for doing the MCMC.

OK, model 41 is uploaded and MCMC started using commands listed in modified README on my fork. I will upload MCMC results once they're available, hopefully around mid-day tomorrow. Others could run MCMC separately if they wish and presumably get identical results.

"mcmc" sub-directory for model 41 now uploaded to drive

Note: starter file didn't have 201 as MCMC burn-in, so posteriors.sso had too many rows. I just reran -mceval with modified starter and uploaded (10:11am) a new set of mcmc outputs.

Thanks Ian, I'm just watching my kids but will try to take a look in a while.

Yep, I'll be signing on a bit later in the day and through the evening so
will keep checking to see where others leave off
On Sat, Jan 21, 2017 at 10:49 AM, cgrandin<EMAIL_ADDRESS>wrote:

Thanks Ian, I'm just watching my kids but will try to take a look in a
while.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/cgrandin/hake-assessment/issues/195#issuecomment-274280330,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AHk_-iqnqNyHvRMXgKq8vSYWsx6yPXdzks5rUlM1gaJpZM4Lp5T7
.

--
Aaron M. Berger, Ph.D. Research Mathematical Statistician
Fishery Resource Analysis and Monitoring Division
Northwest Fisheries Science Center
2032 SE OSU Dr. Bldg. 955
Newport, OR 97365-5275
<EMAIL_ADDRESS><EMAIL_ADDRESS>

I'm at work for about 2.5 hours - just rebuilding with 41. Thanks Ian for doing the MCMC's.
I will then do the sensitivites that Ian suggested for phi:
Model 42: phi = 0.03
Model 43: phi = 0.10
Model 44: phi = 0.30
with phi=0.20 being the new default. I think these will be useful to see if they give what we expect, and to help justify Model 41 as being a new base case.

Thanks guys, I was able to successfully run everything and build the doc at commit d1db072f11ab20debfd504abf5862712dc225399. There was an error after the forecasts, retros, and partest and then on the load step but I just changed reload.all = FALSE and sourced all.r again and it's happy.

Same for me; I had an error and was just re-running after doing reload.all=TRUE. Still going but sounds like it should work.

Sounds good.
The document is building for me and I think model 41 is looking reasonable. I'm taking a break to go grocery shopping but unless anyone finds an issue with model 41, I think that re-running the sensitivities that Andy is doing makes sense as the next step. Either simultaneously, or after looking at those results, we could probably also redo other sensitivities related to things like steepness and M.

Ian, what do you think about Figure 27? The fishery selectivity samples look wrong to me:

Doesn't look good. Either completely uncertain or MCMC's have not converged?
Bottom figure was well behaved for previous base model.

Presumably the uncertainty carries through into other quantities - I would expect current biomass to also be very uncertain (since model can't decide on the selectivity of the age-2s). Still running all.r, but should have a build soon.

You can try reverting the change I made to the plot in this commit:
https://github.com/cgrandin/hake-assessment/commit/05aaff111bb58a2b5123e1ef072d087c08b6780f
(which was just to change line 174 of figures-selex.r from
selex <- selex.list$selex[[1]]
to
selex <- selex.list$selex[[length(selex.list$selex)]]
but before that change, the blue points for the median were far away from the cloud of lines.
I believe that with more flexible selectivity, the model truly has no idea what the selectivity was for 2016, where some MCMC samples have a peak selectivity at age 1, others with a peak at age 2, etc. However, it seems to have more stable estimates of the selectivity of age-2 relative to age-6 resulting in tighter intervals for the expected values for the fishery age composition in Figure 22.
It's possible that this model has an unrealistic degree of flexibility in the fishery selectivity (Figure 26 shows broad bands for all years), but given how much variability there is in the distribution of fish among the two countries and relative to species that the fleets wants to avoid catching as bycatch, I think there's a lot we don't know about selectivity.
OK, truly taking a break now.

Enjoy shopping. Can you get me some milk?

My all.r crashed (with reload.all=TRUE) at:
getting files from ../../models/41_BasePreSRG_v3/mcmc/forecasts/forecast-year-2017/01-0
Error in quantile.default(newX[, i], ...) (from load-models.r#405) :
promise already under evaluation: recursive default argument reference or earlier problems?
Will investigate (any ideas Chris if you're online?).

I can work on the running the other (no selectivity 'phi') sensitivities this evening.

Andy - that's what happened to me. Just set reload.all = FALSE and source all.r again. It worked fine for me

Good, because that's what I just randomly guessed at.
The phi sensitivities are uploaded to Google.
Will try get them plotted in the document once it builds (and take a look at the MCMC results of the new base case).

Okay, phi sensitivity code done (took a while as I was trying to fix some of those other issues).
I'm still not sure what to make about the broad selectivities shown in Figure 26 that Ian mentioned above, though his paragraph seems to justify an argument for allowing such broadness.
However, new fig 45 still shows that the historical estimates of biomass seem fairly insensitive to phi, which it's comforting. Fig 47 shows sigma_r to have more effect (at least for the chosen values of sigma_r).
Okay, thanks guys- I'm off now until Monday.

Thanks guys for all the work today.
I agree that model 41 is a little funny in the breadth of the selectivity uncertainty, but it seems reasonable in every other way, and seems more plausible than the models where we should be walking across the ocean on the backs of the 2014 year class.
I just kicked off MCMCs for models 42 and 43 so that we have the option to create the same plots of selectivity variability for those cases with less flexibility in the time-varying selectivity for comparison. Having those MCMCs in hand would also make it easier to change the base model to one of those other less-flexible cases if the reviewers don't like our base model or we're otherwise are inspired to make a last-minute change.
I'm quitting for today and it seems like we should all take Sunday off and reconvene on Monday. Perhaps we could do another conference call, either at 10am or slightly later if folks want time to debug anything remaining or rebuild the document.
I'll probably check email at some point tomorrow but feel free to call my cell phone if something comes up that needs my attention between now and Monday morning.


Allow setting viewType selector at runtime

In the Custom Editor API you define a package.json entry like this for example:
"contributes": {
  "customEditors": [{
    "viewType": "extensionid.example",
    "displayName": "example",
    "selector": [{
    "filenamePattern": "*.example"
    }]
  }]
},

But then the selectors / filenamePatterns are hardcoded in the package.json. It would be awesome if we could change these during runtime (for example user-defined via the config).
I'm trying to make an extension that lets the user define which file patterns should be handled by the extension's CustomTextEditorProvider

I have a similar need.
I am dealing with a bunch of JSON files, which in my case, I cannot rely on the location of the files. Instead, I have to read the file first to check if it has a specific key in it, if so get the correct view or fallback to default text editor.
I have hoped to find anything to hook onto before resolveCustomTextEditor is called, however it seems there's no way at the moment.


[DRAFT] Add path parameter to customized plugins

Draft of implementation of new key for plugins in the annotation. ref to

#21

summary:

add new key in in AnnotationKeyclass for path
edit yaml config
edit plugin search glob given the path.

#TODOs:

[ ] add tests
[ ] add documentation

@dmartmillan @CarlosLopezElorduy
This implementation includes the possibility of adding a parameter "path" to where to find the customized plugins.
Although I think this solves only part of the problem, since it would accept the path, although we need to understand a way to handle plugins in packages that uses openvariant (i.e. cgi-pipeline)

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

Project coverage is 84.97%. Comparing base (20acac5) to head (0b55352).

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@             Coverage Diff             @@
##           develop      #22      +/-   ##
===========================================
- Coverage    85.01%   84.97%   -0.05%
===========================================
  Files           44       44
  Lines         1902     1903       +1
===========================================
  Hits          1617     1617
- Misses         285      286       +1

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

Closed as we decided to go with another solution. Ref

#21


Add Anthropic module to client

Hey,
The anthropic module isn't accessible in the new release's type system. Refer to the screenshot below.

Has this been release with 3.1.4?
I think #197 might be related to this.


Fix the json-to-table conversion for nested arrays and values !

to check the issue you can try to convert this json objects:
[
{
"1":"a",
"2":"b",
"3":[{"4":"c","5":"d"}]
},
{
"1":"e",
"2":[{"6":"f","7":"g"}],
"3":"h"
}
]
in this one the "h" value is not present in the able.
and
[
{
"1":"a",
"2":[{"4":"b","5":"c"}],
"3":[{"6":"d","7":"e"}]
}
]

Thanks Mounir for the comment; I am working on the issue and shall update the code in few days

I have updated the code, you can check and let me know if it is working


Restructure schema type hierarchy

Currently, the schema type hierarchy has NamedType at the top (i.e., const SCHEMA: NamedType), which has some strange consequences:

Schemas like NamedType { name: "foobar", ty: U8 } can be defined that have no meaningful interpretation: if it were the schema for u8, it should have name: "u8", and if it were the schema for some wrapper around a u8, it should be a NewtypeStruct/NewtypeVariant.
Types that are represented purely as seqs/maps/tuples in serde's model need to have a name associated with them. This requires workarounds like ignoring struct/enum names altogether in order to treat Vec<T>, [T], and friends as equivalent types.

Reshaping the hierarchy to have DataModelType at the top, and shifting struct/enum names into the relevant variants, would ensure primitives have a single canonical schema and seq/map equivalence can be handled without sacrificing nominal typing of structs/enums. I imagine this would have roughly the same shape as serde_content::Value (which may also be useful as a Value type for postcard-dyn).

2. Types that are represented purely as seqs/maps/tuples in serde's model need to have a name associated with them. This requires workarounds like ignoring struct/enum names altogether in order to treat Vec<T>, [T], and friends as equivalent types.

A consequence of this is that less standard types like nalgebra matrices would also be represented as just seqs with no name attached (that is how they implement Serialize) and would therefore be indistinguishable from other seq types; I'm not sure if this is a feature or a limitation.


Getting lessc running on Windows

Since windows node is 0.5 onwards, 0.6 support will be needed once it's released.  But I think that's a gimme.

Change some path handling logic

e.g.
if (input && input[0] != '/' && input != '-') {
input = path.join(process.cwd(), input);
}

?

Profit

P.S. Yes, I'm successfully and happily using dotLess at the moment,

I'm using it on Windows.  I didn't even notice 1 because I only use relative paths with it.
Anyway, for me all the tests pass and everything works well.
Simple fix for item 1 above:
https://gist.github.com/1360110
-[Unknown]


Refresh Token not being generated

I have been using an oauth request in my Angular JS web app, the authentication results in the access token, but not a refresh token. Is there a workaround?

I'm sry but I give too little information to give any assistance.

I'm closing this. Reporter did not respond.


allow adding icons to menu categories

This pull request adds the ability to add icons to menu categories. Fixes #898
Bootstrap 2:

Bootstrap 3:

@jochenklein would you be up for testing this out?
pip install -e git+https://github.com/pawl/flask-admin.git@add_category_icon#egg=flask-admin

Thanks @pawl, it works just fine! How I did it (example):
def register_admin(app, admin):
    category = 'MyCategory'
    admin.category_icon_classes[category] = "glyphicon glyphicon-ice-lolly-tasted"
    admin.add_view(...)


ui: refactor model related functions

read model data with c++-capnp patterns
delete structs in modeldata.h designed for c-capnp
include modeldata.h in modeld to remove duplicated defines

Can you fix the conflicts?

fixed

Thanks!
This PR introduces a change where the path is only drawn up to path.validLen. Waiting for confirmation from Harald that this value is good to use in the ui.


Fix style issue in docs/source/conf.py

Currently the version update script is introducing a style error (using single quotes instead of double quotes) in the versions in docs/source/conf.py. This fixes that issue

That Greg! 🙏
Also tested the sed command lines to confirm it was doing the right thing
Will check with OPS on reviewing next week

/merge

Thanks all! 🙏


Create stale.yml

stale-bot powered by Github Workflow

What is the advantage of using this, instead of probot?

@ryjones
Good suggestion! I just get to know probot-stale.
I am not member of Hyperledger-labs thus I have no access right to install any probot Github App.

Stale is already installed on this org - if you create .github/stale.yml with the settings you used, it should work


Recyble bin for removable storage

Description of the new feature / enhancement
I want to have recycle bin functionality for removable storage. deleted files are moved to a hidden folder and when the folder size gets larger than specified the older deleted files get deleted for good. It would be great if this merge with windows recycle bin so that the settings and the folder are the same. But even if this works completely local that also is useful.
An alternative implementation could be adding the deleted files to the recycle bin in drive C. this way the file is accessible only on the computer it is deleted. Restoring the file however could be tricky.
Scenario when this would be used?
I use SD memory as a side storage to increase my available storage on my laptop. But deleting a file from the sd memory is different with deleting files from the ssd drive. When I accidentally delete something from my sd memory I have to use recovery software and that's annoying.
Supporting information
No response

Hey @mshastchi, thanks for the request. Unfortunately, this issue is out of scope for PowerToys.


inference

hi,
if i wanna obtain the results for different videos, do i need to run training for each video, or use the ckpt you you provided for inference. if so, how to inference, thank you
thanks,

Hi, you do not need to retrain the model to run inference on different videos. You can use the provided checkpoints directly.
To run inference, you first need to instantiate DOT, e.g., for the default config:
from dot.models.dense_optical_tracking import DenseOpticalTracker as DOT
model = DOT().cuda()

The supported inference modes are listed here:
https://github.com/16lemoing/dot/blob/0c26a74cab68ef361c6cabbbaad712685f79afad/dot/models/dense_optical_tracking.py#L27-L33
If you tell me more about which output you would like to obtain I can help you find the inference mode that is best suited to your use case, maybe we need to create a new one.
You can also look at the demo to have an example of inference.

thank you for your reply. your answer really helps a lot.

Hi,
i would like to know if i wanna process a video, how should I do? I wanna obtain the tracking points between each two consecutive frames.
thx

Hi, can you be more specific? If I understand well, you would like to use DOT to extract the optical flow between consecutive frames, is that correct? (like the function "get_flow_from_last_to_first_frame" https://github.com/16lemoing/dot/blob/0c26a74cab68ef361c6cabbbaad712685f79afad/dot/models/dense_optical_tracking.py#L37 but for arbitrary source and target frames, e.g., consecutive ones) I can implement a generic function that does that if you want.

Hi @16lemoing, can you elaborate more on these modes?
https://github.com/16lemoing/dot/blob/550e00336198dc143493e415d52720eb9a53ab55/dot/models/dense_optical_tracking.py#L27-L33
Let's say if I want to produce tracks that cover all pixels in a given video, how do I leverage your API? Thanks!

Hi @ernestchu, I have added a new mode that does just that. It produces tracks that cover every cell in a video.
https://github.com/16lemoing/dot/blob/e32c6f7de12342460371f2efc4789bd79c4a39a3/dot/models/dense_optical_tracking.py#L34-L35
You can set the size (in pixels) of the cells with the flag --cell_size (by default 1: every pixel), and the number of time steps between cells --cell_time_steps (by default 20 to speed up inference).
I have also updated the demo, you can now visualize an overlay with tracks reinitialized every few frames.
https://github.com/16lemoing/dot/assets/32103788/c0581f9a-0508-423b-b4db-9ec1aca2320a
python demo.py --visualization_modes overlay --video_path cartwheel.mp4 --inference_mode tracks_from_every_cell_in_every_frame --cell_time_steps 20

Hi @16lemoing , I would expect .round() is better than .floor() here. What do you think? :)
https://github.com/16lemoing/dot/blob/e32c6f7de12342460371f2efc4789bd79c4a39a3/dot/models/dense_optical_tracking.py#L228-L229

BTW, I do not understand why you used visited_x / (w - 1) * cw. Consider the case of w = cw, why do we have to modify the predicted coordinate here?

Hi @16lemoing, could you please explain how to track pixels from an input mask through the whole video using your code? I would like to use one of my own videos.
Thank you.

Hi @maxboels, if you have a mask on the first frame, you can use this inference mode:
https://github.com/16lemoing/dot/blob/e32c6f7de12342460371f2efc4789bd79c4a39a3/dot/models/dense_optical_tracking.py#L32-L33
This will return a dictionary from which you can extract "tracks" with the shape [B T H W 3] (with B: batch size, T: time steps, H: height, W: width) corresponding to all the pixels in the first frame.
You can then filter the tracks to only keep those which belong to the mask.

Hi @16lemoing , just want to check it with you.
I want every pixel penetrated by at least one track (after round it or floor it).
Somehow, if I replace these two lines
https://github.com/16lemoing/dot/blob/e32c6f7de12342460371f2efc4789bd79c4a39a3/dot/models/dense_optical_tracking.py#L228-L229
with
visited_x = (visited_x * ((cw - 1) / (w - 1))).round().long()
visited_y = (visited_y * ((ch - 1) / (h - 1))).round().long()

then it gets me what I want, with rounded tracks.

Note: I set cw = w , ch = h and ct=1 in order to deal with all pixels
In addition, I think the variable names are misleading because cw is actually the number of cells in w, instead of "cell width". ct has the opposite meaning though


Localization support

The bot currently does not have any easy way to be localized for different languages. Fixing this would allow the bot to reach more people.
A service like CrowdIn (https://crowdin.com/) could be used to translate the bot, but the implementation process may be difficult.
If you are interested in helping with translation, please join this discord and subscribe to this issue. Updates will be posted soon.

Closing out feature requests to keep this tracker to bugs-only. This does not mean that this feature is a "won't-add". Further discussion is being moved to the Discord.


AbstractActivityParty::registeredAt

Date when the contact registered as participant for the activity

fixed with opencrx-4.2.0+


Elevated messages removed

Pull request checklist:

[x] CHANGELOG.md was updated, if applicable

Description

Do you have any confirmation the experiment is over? While it might not be running currently it could easily just appear again

We can look at removing Elevated Messages once we get some confirmation they aren't coming back, currently the help page is still live.

@Felanbird https://twitter.com/TwitchSupport/status/1722675472218141072

Let's wait a little bit more to see what they do with bits and what-not


Get n Number of emails

Can anyone tell me how I could get the 'n' most recent emails.
I have tried this:
imap.find(mailbox: 'INBOX', read_only: true, count: 50, order: :desc)

But this returns the 50 oldest emails in the inbox.  I've tried order: :asc also with the same result.
Does anybody know the correct parameters?

Need what: :last as well.


wip: update compute status/stop to support agreementId, not documentId, sig fix

Fixes #1820 .
Changes proposed in this PR:

support for agreementId on get compute job status (documentId is not used, only owner, jobId or agreementId)

Looks like there is an issue with the integration test

Looks like there is an issue with the integration test

probably ipfs related, just timeouts... nothing really changed here anyway


Add "Bind" capability to Components to simplify memoized callbacks

Current practices avoid generating anonymous functions for callbacks in render by defining those functions on self in a component's init.
It would be extremely useful for components to provide a bind method on self that can memoize and avoid recreating callbacks.
This might look like:
function MyComponent:onClick()
   self:setState({ --[[whatever]] })
end

function MyComponent:render()
   return Roact.createElement("TextButton", {
      [Roact.Event.Activated] = self:bind(self.onClick)
   })
end```

Is there still any interest in this? I had a go at implementing it: https://github.com/dr01d3k4/roact/commit/5e388ac40606633c541210f70f3af00d2e02ef19


Indexed LValue bug

Summary:
Currently, Stan treats LValues (left-hand-side of assignments) as having a flat indexing structure consisting of a single list of indices. This seems in contradiction with the treatment of RValues (right-hand-side of assignments).
Description:
For example: a[:][2] and a[:, 2] are equivalent as an LValue but not as an RValue.
Reproducible Steps:
Run
model {
    int a[2, 2] = {{1, 2}, {3, 4}};
    a[:][2] = {17, 18};
    print(a);
}

and
model {
    int a[2, 2] = {{1, 2}, {3, 4}};
    a[:, 2] = {17, 18};
    print(a);
}

They give the same output.
Then run
model {
    int a[2, 2] = {{1, 2}, {3, 4}};
    print(a[:][2]);
}

and
model {
    int a[2, 2] = {{1, 2}, {3, 4}};
    print(a[:, 2]);
}

They give different output.
Current Output:
Inconsistent behaviour between indexing in LValues and RValues.
Expected Output:
Consistency. I think the behaviour of RValues is the correct one.
Current Version:
v2.20.0

@mitzimorris , do you know if this would be easy to fix in the code generation? Or would it be a lot of work?
The indexing system is just pretty complicated and it is also not entirely sufficiently documented in the manual.

@VMatthijs  - I will look into it and let you know - can't get to this one right away - tomorrow?

Thanks! No rush at all. It's not something that is blocking me. Just felt like it might be important to let someone know who has more knowledge of the codegen than I do. :-)

I ran this:
parameters {
  real<lower = 0, upper = 1> theta;
}
model {
    int a[2, 2] = {{1, 2}, {3, 4}};
    print("a[2] = ", a[2]);
    print("a[ : ][2] = ", a[ : ][2]);
    print("a[ : , 2] = ", a[ : , 2]);
}

which when run prints
a[2] = [3,4]
a[ : ][2] = [3,4]
a[ : , 2] = [2,4]

As @VMatthijs said in the issue, this is the intended behavior for rvalues.
To write out the semantics more clearly, : used as an index is shorthand for
a[:] = a[1:size(a)]

where size here might be size for arrays, rows for matrices or vectors, and cols for row vectors.  Then the value of a range is just the sequence
lb:ub = {lb, lb + 1, ..., ub - 1, ub }

and in general sequence indexing is defined by
a[idxs][i] = a[idxs[i]]

That generalizes to multiple indexes,
a[idxs1, ..., idxsN][i1, ..., iN] = a[idxs1[i1], ..., idxsN[iN]]

We really need to write out a proper semantics for the language!

We can also leave this unfixed in the old parser, right?

Right---I labeled it won't fix till new compiler.

Makes sense - thanks.
Also, I thought I posted the following on this thread but can't find it - sorry if I'm repeating myself.
I would propose that we move to disallow nested indexing in the lhs of assignment rather than fixing it to do the proper thing. My reasoning is that the proper semantics here would be meaningless - if we move towards rvalue semantics, a multi-index always generates a copy, so the effect of a[multi_index][2] = rhs; is always a no-op if properly interpreted. (where multi-index is any of 2:3, :, 2:, :2, some_int_array).
Another alternative would be to retain the current behavior and just document it, but I'd rather deal with folks who are mad that their model now generates an error message (assuming we also include the "how to easily fix" in the error message) rather than the folks who learned the wrong semantics for lhs and are confused in a weird way about indexing semantics. Perhaps this is because I was myself confused about index semantics in exactly this way and wasted a few full-time days because of that confusion. 😅

I would propose that we move to disallow nested indexing in the lhs of assignment rather than fixing it to do the proper thing.

I'd be OK with that change.  At that point, it might make sense to disallow it on the RHS, too.
The problem that multi-indexing caused is that we no longer have a[i][j] = a[i, j].

My reasoning is that the proper semantics here would be meaningless - if we move towards rvalue semantics, a multi-index always generates a copy, so the effect of a[multi_index][2] = rhs; is always a no-op if properly interpreted. (where multi-index is any of 2:3, :, 2:, :2, some_int_array).

I also think it's clearer to write a[multi_index[2]] than a[multi_index][2], at least in most situations.
We will still allow partial indexes, though, right, so that a[multi_index] is still well formed, even if a[multi_index][2] isn't.  Or do you want to force that to be explicitly a[multi_index, :] or a[multi_index, ].
We could also deprecate the lone : in favor of empty.  I can't even remember why I wanted to support both syntaxes---probably because lone : is the natural edge case of : and empty is common in R/BUGS, etc.

Another alternative would be to retain the current behavior and just document it, but I'd rather deal with folks who are mad that their model now generates an error message (assuming we also include the "how to easily fix" in the error message) rather than the folks who learned the wrong semantics for lhs and are confused in a weird way about indexing semantics. Perhaps this is because I was myself confused about index semantics in exactly this way and wasted a few full-time days because of that confusion. 😅

I agree.
Next time, give me a shout before wasting a couple days.  I'm not always keeping up on the Discourse or issues directly because there's so many of them.

My reasoning is that the proper semantics here would be meaningless - if we move towards rvalue semantics, a multi-index always generates a copy, so the effect of a[multi_index][2] = rhs; is always a no-op if properly interpreted. (where multi-index is any of 2:3, :, 2:, :2, some_int_array).

Could you explain that a bit more? Presumably,
a[multi_index] = rhs; would not generate a copy right and be a no-op, right? How is that implemented? Can we not implement something similar for multiple indices on the LHS? Or is it just not worth the hassle?

On Aug 9, 2019, at 9:26 AM, Matthijs Vákár<EMAIL_ADDRESS>wrote:
My reasoning is that the proper semantics here would be meaningless - if we move towards rvalue semantics, a multi-index always generates a copy, so the effect of a[multi_index][2] = rhs; is always a no-op if properly interpreted. (where multi-index is any of 2:3, :, 2:, :2, some_int_array).
Could you explain that a bit more?

I didn't write that original paragraph, but my guess is that it's not so much a no-op as something you'd want to compose to replace a[multi_index][i] with x[multi_index[i]] rather than trying to evaluate a[multi_index], which requires a copy as things stand.  If we moved to something like expression templates, we could just do everything compositionally without having to copy.  This would go for lvalue or rvalue uses.
Can the code just be written so that lvalues and rvalues both get evaluated as references?

I guess there are a few ways you could interpret it (one of them being the current interpretation, which seems like a bug) and there is a reasonable non-noop operation you could perform after simplification, but it seems better to just limit users to expressing indexing in a single coherent index notation. I just meant that if you were thinking about nested indexing as you think about nested function calls (which is how they work on the rhs) then the inner a[multi_index] would return a copy that [2] then indexes into, so an assignment into that copy would immediately be thrown away. Anyway, doesn't really matter.

Can the code just be written so that lvalues and rvalues both get evaluated as references?

Not 100% sure if this is what you mean, but I think if we allowed something like Eigen Maps everywhere in our Math library this might be possible. It'd be a backwards-incompatible change, though, since currently we provide copies.


Configure loggers for function

Summary
"user" and "function" loggers are created in the runner class
The "user" logger writes the log into /data/{job_id}/user.log file which is /{user_name}/{job_id}/user.log in the persistent storage
The "function" logger writes the log into /function_date/{job_id}/function.log file which is /{provider_name}/{job_id}/function.log in the persistent storage
The function code can get these logger by logger.getLogger("user") and `logger.getLogger("function")
An optional argument type is added to job.logs() method. The 'type value should be "user" or "function".
For the provider, it returns both "user" and "function" logs. For the user, it returns only "user" log
Details and comments

This PR has dependency on #1408

@akihikokuroda let me put this PR on-hold until we release 0.15. I think this PR will require time to discuss it and we have more urgent priorities for that release 🙏

Removing the label now that we released the new version and we can discuss this calmly 😄

log_type: I don't think is needed but correct me if I'm wrong, Aki. Backend logic can be modified to identify if the function is from provider or user without the need to use this variable.

The provider can get 3 different logs (user, function and console output).
The user can get only the user log when the provider function is executed
The user can get  3 different logs (user, function and console output) when the user function is executed.

I see that this is mostly for provider functions. How could we do this for user functions?

The user can put the log entries into the different logs and retrieve them separately.

Right now we have some validations in the logs like this one, how can we check these things with this new approach?

It is using the standard logger function, so if log entries are not there, it returns nothing.

The provider can get 3 different logs (user, function and console output).
The user can get only the user log when the provider function is executed
The user can get 3 different logs (user, function and console output) when the user function is executed.

Well, this workflow is something I would like to validate with Sanket too so let's discuss it internally. Personally I don't see the value to support both kind of logs: user log and console output. Same as that provider would have access to user logs having them their own logs.

The user can put the log entries into the different logs and retrieve them separately.

For me as user sounds confusing to have two kind of logs that basically do the same.

It is using the standard logger function, so if log entries are not there, it returns nothing.

Before merging anything we need to ensure that we are not loosing any feature like that mentioned.
Let me Monday write in our internal issue about this to initiate a conversation with Paco and Sanket.

Well, this workflow is something I would like to validate with Sanket too so let's discuss it internally. Personally I don't see the value to support both kind of logs: user log and console output. Same as that provider would have access to user logs having them their own logs.

The use log entries are generated by the function that are shared with the user.  It's still valuable for the provider to check what are shared with the user.
The console output may include the stack back trace when some exception happens.  The function developer must want to see them :-)

The use log entries are generated by the function that are shared with the user. It's still valuable for the provider to check what are shared with the user.

I agree but I feel a bit uncomfortable doing that so I prefer to confirm that that's exactly what we are looking for. For that is for what we are separating logs but I agree with you in this assumption. Logs management is something that it's not closed yet and I prefer to think well about it taking into account that it's not so urgent.

The console output may include the stack back trace when some exception happens. The function developer must want to see them :-)

Can we not redirect that to the logs? Thinking in that at some point as we talked we were going to remove logs from database (something that I think it has a lot of sense).

Can we not redirect that to the logs? Thinking in that at some point as we talked we were going to remove logs from database (something that I think it has a lot of sense).

That's the original log.  We can take it out but it probably contains critical information to debug the function issues.

Oh yes, I was thinking in that "function" logs not "user" logs. "user" logs should only print that information specifically sent it to it I suppose.

@Tansito Is there any progress for this?  What is the next step?

Sorry @akihikokuroda I couldn't address this yet. Feel free to open the conversation in our internal issue. I will try to add there my comments.


Fix typos on challenges

This PR fixes some typos in some challenges :)

Approved.
Thank you for your contribution, @adrianoapj  💜


Add warning when changing the symbolizer kind.

As @chrismayer suggested here #218. We should think about a warning when switching the symbolizer kind.

@KaiVolland @chrismayer is this still relevant?

I'll close this for know.


roi pointcloud fusion for small objects detection

Checklist

[X] I've read the contribution guidelines.
[X] I've searched other issues and no duplicate issues were found.
[X] I've agreed with the maintainers that I can plan this task.

Description

Vehicle sensor configuration with low resolution LiDAR is limited to detect small objects such as traffic cones since too few points are hit the object and Euclidean cluster failure to cluster those objects.
The analysis shows that current roi_cluster_fusion on sample vehicle is only able detected traffic cones in range between 19m~49m in ideal flat road. It's narrower in the real test and clustering is applied after ground segmentation filter and compare map segmentation filter.

cone size
IoU > 0 range

min_cluster_size: 10 could be change to small to improve small object clustering. However, Euclidean cluster FP unknown objects output also increase rapidly because of noise pointcloud.

Purpose
The 2D image detection model like YOLOX may be strong to handle such unexpected small objects even in quite long distance.
A new approach which more relies on  2D image detector result and use LiDAR cloud as validation as well as depth information could solve this issue.
Possible approaches

Reduce min_cluster_size to cluster cones for roi_cluster_fusion and process the output of fusion to reduce noise unknown object
Add roi pointcloud fusion and focus on small objects ROIs fusion result to cluster/detect the small objects

Definition of done

Successful detected small objects such as traffic cones with less points cloud if recoginzed by 2d image detector.

@badai-nguyen Is this issue already resolved? Or this includes current development item?

@miursh I think the neccessary PRs for this issue are allready merged. The accuracy of this approach also is depend on the performace of Yolox-s model which may need more time to collect and annotate the data, and retrain the model. So I will closed this issue.


Select2 4.0.3

Js Select2 4.0.3 is release from May 2016. On django-autocomplete-light still 4.0.2.
To be default without override class Media. Is there another option to update or get own file reference like define on django settings the path for latest select2 plugin to override internal static select2 on django-autocomplete-light?

Thanks, work has been started in #770


Change from mock server to live OData service

1.) Tutorial URL: https://developers.sap.com/tutorials/webide-live-odata.html
Please specify the step you are referring to
I should add the service (which looked good in preview)
After step 3 I got this:
13:45:13 (UsageMonitoring) The reported Event Type: GENERIC_finishWithNewModel should be configured in the reporter's plugin Plugin.json. sEventType is set to GENERIC_finishWithNewModel
13:45:13 (UsageMonitoring) Event Type GENERIC_finishWithNewMode is too long. Only 25 chars allowed.
I could continue. In the next tutorial:
https://developers.sap.com/tutorials/webide-fiori-deploy.html
I got this (after the deploying):
15:34:24 (Application Index) URI block was not added to the manifest.json file: Request failed: error URI: /sap/flp/flpnwc/sap/bc/ui2/app_index?sap.app/id=com.sap.build.standard.inventoryDashboard&fields=sap.app/id,html5AppName
In final my does not work. I tried the tutorial a second time and got the same errors.
Thx in advance.
Jan

These tutorials are no longer more available.
Because of this I Close this issue. By the way I wonder why this one got no comment.


Bugfixes to kastner file generation

See #13 for description.

Merging the bugfixes to file-generation, without updating the actual files.


Fix issue SMARTSPRITES-98

Guava Needs Upgrade

I've just pushed 0.2.11 to Maven Central, it should be up in a minute.

Hi Stanislaw
Thank you very much for your reactivity.
I really appreciate it.
Cheers,
icefox


Feature/Rename getSharedVersionsOfAttribute

Since this poses a breaking change, I'll merge feature/add-acceptresponseitems into release/v5.

I think we should keep the UseCase that's only for repositoryAttributes and add a new one which acts for all attributes. Then it would not be a breaking change which is cool.


Evaluated model improvements

Several improvements of the evaluated model. Please see the commit messages for details.

Merged, ignoring unrelated Travis error. See #2334 .


3 Dimensional weight matrix -> TypeError: Expected int32, got None of type '_Message' instead

Hi,
i'm trying to replicate the idea from Generating Text with Recurrent Neural Networks where the recurrent matrix "change" wrt to the input 1-hot vector.
Here is the gist of my attempt
Which gives two different errors.
In TensorFlow:
TypeError: Expected int32, got None of type '_Message' instead
In Theano:
ValueError: When compiling the inner function of scan (the function called by scan in each of its iterations) the following error has been encountered: The initial state (outputs_infoin scan nomenclature) of variable IncSubtensor{Set;:int64:}.0 (argument number 1) has 3 dimension(s), while the corresponding variable in the result of the inner function of scan (fn) has 3 dimension(s) (it should be one less than the initial state). For example, if the inner function of scan returns a vector of size d and scan uses the values of the previous time-step, then the initial state in scan should be a matrix of shape (1, d). The first dimension of this matrix corresponds to the number of previous time-steps that scan uses in each of its iterations. In order to solve this issue if the two varialbe currently have the same dimensionality, you can increase the dimensionality of the variable in the initial state of scan by using dimshuffle or shape_padleft.
I get that in the step function input is processed by batches and that naïvely writing:
rec = K.dot(self.REC,i)
 h = K.dot(i,self.INP) + K.dot(hm1, rec )

shouldn't work. So i've tried the following:
i = K.expand_dims(i)          #K.reshape(i,(128,10,1))
dot = K.dot(self.REC, i)
dot = K.squeeze(dot,axis=3)
rec = K.transpose(dot)
h = K.dot(i,self.INP) + K.dot(hm1, rec )

but I always end up on the same error messages.
Perhaps there is some framework magic that i'm not grasping ?

I'm getting the same error message with a BatchNormalization layer.


[Bug] Can't take snapshot of dashboard

Describe the bug
Take a snapshot of dashboard, and panels with infinity datasource are always empty.
Maybe because inifinity not properly sending the query request?
Let's:

Open https://grafana-infinity-datasource.herokuapp.com/d/try/try?orgId=1
Click panel title of JSON.
Click menu Inspect -> Query.
In Query inspector, click Refresh.
Inifinity entered infinite Loading query inspector...  state.

Can you send the exact steps to reproduce the issue? Snapshots of dashboards with infinity datasource are working for me.
BTW, Query inspector won't show the data for infinity datasource and it is a known behaviour. Infinity doesn't use query calls of grafana backend instead it uses resource calls to fetch the data.

If a infinity panel with inline source is out of screen when taking snapshot, then the data will be empty in the snapshot of dashboard.


Add upgrading doc

Coverage remained the same at 97.04% when pulling eb88a76aaae50ce5439fc17618eaaba1dd3809b2 on jodh-intel:add-upgrading-doc into c94e0ab8362e81242a45f604e984a3f2a2e0879d on clearcontainers:master.

Coverage remained the same at 97.04% when pulling 6a9f45d1e269a218528df21cf0eb9e32fad1167c on jodh-intel:add-upgrading-doc into c94e0ab8362e81242a45f604e984a3f2a2e0879d on clearcontainers:master.

Coverage remained the same at 97.04% when pulling 6a9f45d1e269a218528df21cf0eb9e32fad1167c on jodh-intel:add-upgrading-doc into c94e0ab8362e81242a45f604e984a3f2a2e0879d on clearcontainers:master.

Hi @iphutch, @grahamwhaley, @gorozco1 - please could you review?

Coverage remained the same at 97.04% when pulling 1654d7b19041c54029a4ca54ef62f0ad081a6023 on jodh-intel:add-upgrading-doc into c94e0ab8362e81242a45f604e984a3f2a2e0879d on clearcontainers:master.

@grahamwhaley - branch updated.

Coverage remained the same at 97.04% when pulling 82ebe0bdbc2fd1299e97f358a605007c3db57d4b on jodh-intel:add-upgrading-doc into c94e0ab8362e81242a45f604e984a3f2a2e0879d on clearcontainers:master.

@mcastelino -- FYI.

Coverage remained the same at 97.04% when pulling 27b919f7a2a95a6a5d54cee3e10a8605fde682b9 on jodh-intel:add-upgrading-doc into fd786c0be8b213806a5ebbd834e058bc476e95da on clearcontainers:master.

Hi @iphutch - thanks for reviewing. Branch updated based on your feedback.

Coverage remained the same at 97.04% when pulling 4fb1411fc3486dac7da562aae95cbc1af256e4ea on jodh-intel:add-upgrading-doc into fd786c0be8b213806a5ebbd834e058bc476e95da on clearcontainers:master.

Coverage remained the same at 97.04% when pulling 4fb1411fc3486dac7da562aae95cbc1af256e4ea on jodh-intel:add-upgrading-doc into fd786c0be8b213806a5ebbd834e058bc476e95da on clearcontainers:master.

Coverage remained the same at 97.04% when pulling ba94247cf6adb3b9144048241b2592525cf15d6b on jodh-intel:add-upgrading-doc into fa07228124025f0cbad3f1b2eba6d021ddce6b11 on clearcontainers:master.

Coverage remained the same at 97.052% when pulling f4fca16aef319f811d7cbb7fb573c113daa802e0 on jodh-intel:add-upgrading-doc into 97aa6e6ce3136dc183855ec800a82a70bac1d0ee on clearcontainers:master.

Coverage remained the same at 97.052% when pulling 1878a93805ace4ca77239c5e7e2d487bab5b4801 on jodh-intel:add-upgrading-doc into d34cdad4d5087e41cdc4b2fc80d4c66519b07c4d on clearcontainers:master.

Coverage remained the same at 96.825% when pulling 76d5e70b4cb210c4c2db3e4fb500f96c05f022ce on jodh-intel:add-upgrading-doc into 91f2b3e5a2533ec658eceef1e287c078378bb7d2 on clearcontainers:master.

Coverage remained the same at 96.825% when pulling 523f85bf5849ee5a08764f028b59c5ff35e984c1 on jodh-intel:add-upgrading-doc into 0f8ec51920d7123c1858435b3225404a8b487e79 on clearcontainers:master.

but you have the travis revision range issue rearing its head :-(

Coverage remained the same at 96.831% when pulling 3d117567da4bf17aea45b443ec677dbce8305ebb on jodh-intel:add-upgrading-doc into bc51f37b72674b1228ddb61ee5c74a694998edfc on clearcontainers:master.


deepspeed report keyword errors

When I using deepspeed through accelerate, error happens:
Traceback (most recent call last):
File "/home/rubickjiang/miniconda3/envs/deepspeed/bin/accelerate", line 8, in
sys.exit(main())
File "/home/rubickjiang/miniconda3/envs/deepspeed/lib/python3.10/site-packages/accelerate/commands/accelerate_cli.py", line 46, in main
args.func(args)
File "/home/rubickjiang/miniconda3/envs/deepspeed/lib/python3.10/site-packages/accelerate/commands/launch.py", line 1067, in launch_command
deepspeed_launcher(args)
File "/home/rubickjiang/miniconda3/envs/deepspeed/lib/python3.10/site-packages/accelerate/commands/launch.py", line 734, in deepspeed_launcher
from deepspeed.launcher.runner import DEEPSPEED_ENVIRONMENT_NAME
File "/home/rubickjiang/miniconda3/envs/deepspeed/lib/python3.10/site-packages/deepspeed/init.py", line 25, in
from . import ops
File "/home/rubickjiang/miniconda3/envs/deepspeed/lib/python3.10/site-packages/deepspeed/ops/init.py", line 15, in
from ..git_version_info import compatible_ops as compatible_ops
File "/home/rubickjiang/miniconda3/envs/deepspeed/lib/python3.10/site-packages/deepspeed/git_version_info.py", line 29, in
op_compatible = builder.is_compatible()
File "/home/rubickjiang/miniconda3/envs/deepspeed/lib/python3.10/site-packages/deepspeed/ops/op_builder/gds.py", line 42, in is_compatible
gds_compatible = self.has_function(funcname="cuFileDriverOpen",
TypeError: OpBuilder.has_function() got an unexpected keyword argument 'library_dirs'
My deepspeed version is: 0.14.3+69adeab6

Same issue appears after running ds_report with the following output:
[2024-10-21 11:22:16,671] [INFO] [real_accelerator.py:191:get_accelerator] Setting ds_accelerator to cuda (auto detect)

DeepSpeed C++/CUDA extension op report

NOTE: Ops not installed will be just-in-time (JIT) compiled at
      runtime if needed. Op compatibility means that your system
      meet the required dependencies to JIT install the op.

JIT compiled ops requires ninja
ninja .................. [OKAY]

op name ................ installed .. compatible

 [WARNING]  async_io requires the dev libaio .so object and headers but these were not found.
 [WARNING]  async_io: please install the libaio-dev package with apt
 [WARNING]  If libaio is already installed (perhaps from source), try setting the CFLAGS and LDFLAGS environment variables to where it can be found.
async_io ............... [NO] ....... [NO]
fused_adam ............. [YES] ...... [OKAY]
cpu_adam ............... [NO] ....... [OKAY]
cpu_adagrad ............ [NO] ....... [OKAY]
cpu_lion ............... [NO] ....... [OKAY]
 [WARNING]  Please specify the CUTLASS repo directory as environment variable $CUTLASS_PATH
evoformer_attn ......... [NO] ....... [NO]
 [WARNING]  FP Quantizer is using an untested triton version (2.0.0), only 2.3.(0, 1) and 3.0.0 are known to be compatible with these kernels
fp_quantizer ........... [NO] ....... [NO]
fused_lamb ............. [NO] ....... [OKAY]
fused_lion ............. [NO] ....... [OKAY]
Traceback (most recent call last):
  File "/home/user/conda/envs/internvl_env1/bin/ds_report", line 6, in <module>
    cli_main()
  File "/home/user/conda/envs/internvl_env1/lib/python3.10/site-packages/deepspeed/env_report.py", line 190, in cli_main
    main(hide_operator_status=args.hide_operator_status, hide_errors_and_warnings=args.hide_errors_and_warnings)
  File "/home/user/conda/envs/internvl_env1/lib/python3.10/site-packages/deepspeed/env_report.py", line 184, in main
    op_report(verbose=not hide_errors_and_warnings)
  File "/home/user/conda/envs/internvl_env1/lib/python3.10/site-packages/deepspeed/env_report.py", line 53, in op_report
    is_compatible = OKAY if builder.is_compatible(verbose) else no
  File "/home/user/conda/envs/internvl_env1/lib/python3.10/site-packages/deepspeed/ops/op_builder/gds.py", line 58, in is_compatible
    gds_compatible = self.has_function(funcname="cuFileDriverOpen",
TypeError: OpBuilder.has_function() got an unexpected keyword argument 'library_dirs'

Deepspeed version (installed from source): 0.13.5+bc0d2465
Accelerate version: 1.0.0

Can you try updating to the latest deepspeed? Those version numbers appear to old to have this functionality and it certainly exists in deepspeed main branch: https://github.com/microsoft/DeepSpeed/blob/40bde528bcac79f765dc8f15d6d1512b3415199f/op_builder/builder.py#L308

I found that it may have something to do with slurm. This error happens when I install deepspeed from source in the login node. But after I try again on a GPU node, the function works.


some textures i made a while ago

nice 👍 welcome back btw :)
btw: i took the liberty of preparing this: #46 😉


Added support for CSVs, as well as callback functions as the atlas_format

Also added an error if the format is invalid and a new property, atlas_ext in case the user wants to set a custom extension (defaults to None which, in turn, defaults to the default extension for the atlas_format). This is specially useful when using a function as the "format".

Maybe the default file format for a function could be .txt instead of .bin, but I don't think it's sensible to pass a function and not a custom extension.

This should also fix Issue #7