[CSL-1247] Put Explorer into CSL source tree for consistency

https://issues.serokell.io/issue/CSL-1247
A new PR, started here - https://github.com/input-output-hk/cardano-sl/pull/1149
Used nix-shell on cardano-sl root to check if the build instructions are good.
Added to the current .travis.yml, currently, the output we get fails:
- pushd explorer
# I'm guessing we are missing something like "nix-build -A cardano-sl-explorer ..."
- pushd frontend
- "./scripts/build.sh"
- nix-shell --run "yarn test"
- popd
- "./update_explorer_web_api_docs.sh"
- popd

Other than this problem with space - https://travis-ci.org/input-output-hk/cardano-sl/jobs/260985531#L7738, time limit for MacOs - https://travis-ci.org/input-output-hk/cardano-sl/jobs/260985532#L6657 and required bump for explorer, seems to be ok.
I need help to bump explorer since there are a lot of changes (I already spent a couple of hours on this, and the number of changes is big) - we should merge this ASAP, since master is changing every 100500 milliseconds.
We can fix any details when we have it in the repo and create branches for it like we had so far - explorer-0.5, explorer-master - or similar.
@domenkozar @gromakovsky @akegalj

Did you use git subtree in this PR too?

Did you use git subtree in this PR too?

Yes. It's one of the reasons why I would prefer to merge this quickly, otherwise, we might have issues with history.

@gromakovsky Ok, so we currently can't disable that stupid explorer flag due to linker issues. We have to turn it on by default, which I did while removing it from stack.yaml.
Since that is how it's going to work (for now), your second review point works already - explorer is being built with the "normal" script. If you pass --explorer, nothing changes and I would leave that as is since it's probably we are going to fix that sometime in the future (there is a patch on GHC https://ghc.haskell.org/trac/ghc/ticket/13810).
We need to notify everyone that they are building explorer with cardano-sl now as well.
I know it's not great, but it's not terrible too - it could have been worse.

As a note, we run into disc space issues when building using nix - https://travis-ci.org/input-output-hk/cardano-sl/jobs/262304063#L7720
On the other hand - https://travis-ci.org/input-output-hk/cardano-sl/jobs/262304071 :smiley:

Btw, you should also update some other scripts (e. g. grep.sh).

@gromakovsky Ok, so I fixed things as you suggested. I still don't like that explorer flag and find it annoying. It could be removed and some (unnecessary/extra) cognitive workload can be removed with it.

Nobody likes this flag, but it's not easy to remove it.

@gromakovsky Well, if I move things as you pointed out, then we are back with linker issues.
For example, what's the result of ./scripts/build/cardano-sl.sh --no-explorer?
It enables c part (the actual explorer) but disables the flag. Can I just leave it as is? It's good now.

For example, what's the result of ./scripts/build/cardano-sl.sh --no-explorer?

The result is:
echo "You can't build output with explorer and not use explorer! Invalid flag '--no-explorer'."
exit

@gromakovsky Ok, fixed.

@gromakovsky @domenkozar @akegalj
It's done. Thanks guys for your help!
:rocket:

BTW, it would be useful if we could branch out from master to have a stable version like we had before.
So branches like sl-explorer-0.5, sl-explorer-master, or something similar.


grpcproxy: plans to support client whitelisting?

Are there any plans to add ip whitelisting / ACL to restrict client connections to specific ip addresses?

No plans, but this feature seems like it would belong in the etcd gateway. What's your use case?

We actually are not using the etcd gateway.  This is out of scope, but what would be the advantage of using the gateway instead of grpc-proxy in the event we only need gRPC?

What's your use case?

It's an added security measure.  While user based auth is fine when you're connecting locally, when the grpc proxy is exposed to the open internet, ip based whitelisting is a pretty huge bonus.

what would be the advantage of using the gateway instead of grpc-proxy in the event we only need gRPC?

It's to keep separate layers of the OSI stack from bleeding into each other. Internally, gprcproxy doesn't care about where connections are coming from and is written in such a way it can be used without a network at all. On the other hand, the gateway explicitly reasons about tcp and client connections.


FaceMesh Example Does Not Work on iPad with v0.8.9 (works with 0.8.8)

System information (Please provide as much relevant information as possible)

Have I written custom code (as opposed to using a stock example script provided in Mediapipe): No
OS Platform and Distribution : iOS 11.4.1
MediaPipe version: v0.8.9
Bazel version:
Solution : FaceMesh
Programming Language and version : Objective-C

Describe the expected behavior:
Facemesh displaed on screen
FaceMesh app working on iPhone devices and not working on iPad.
Something changed between v0.8.8 & v0.8.9 as FaceMesh is working on iPad in v0.8.8. With v0.8.9 release FaceMesh does not display the points and the mesh on Face on iPad. Works well on iPhone.

Hi @anappdev2, Could you please elaborate the above error with more details like video recording of the error and error logs if there are any. Thanks!

Its not about video recording its just that the mesh is not displayed properly on iPad (as mentioned works great on iPhone). Its very flaky and just turn into small blob of red dots.
Same thing works on ipad with V0.8.8

@anappdev2  It would be helpful for investigating if you can share the logs or screenshot of the " not displayed on ipad " results.

Here is screen grab of what is displayed on iPad.
https://user-images.githubusercontent.com/59765390/148884935-69b06795-220a-45f5-ba55-3c3731379a01.mp4
Here is the output on iPhone
https://user-images.githubusercontent.com/59765390/148885259-0736cd9b-4051-4d0b-be21-65ae2aca7e50.mp4
Hope this helps resolving the problem.

Hello @anappdev2,
We are upgrading the MediaPipe Legacy Solutions to new MediaPipe solutions However, the libraries, documentation, and source code for all the MediaPipe Legacy Solutions will continue to be available in our GitHub repository and through library distribution services, such as Maven and NPM.
You can continue to use those legacy solutions in your applications if you choose. Though, we would request you to check new MediaPipe solutions which can help you more easily build and customize ML solutions for your applications. These new solutions will provide a superset of capabilities available in the legacy solutions.


Poor performance when scrolling

Description
When scrolling through this python file, it becomes very choppy and the framerate becomes quite bad.
When the tagtree is opened, it becomes a lot worse. Curiously it is only with these specific plugins (see config below).
Some more observations:

With just bufferline disabled, it is smooth, even with the tag tree open.
With airline disabled, is it smooth normally, but with tag tree open it is super choppy.
With tagtree disabled, no chop happens either.
With line numbers disabled it is smooth except when it is about to update the context, then it drops a few frames
The problem completely disappears if I turn context off with :TSContextToggle
I run nvim on my potato laptop with an AMD A12-9720P RADEON R7

Neovim version
NVIM v0.10.0-dev-5ceb2238d
Expected behavior
Smooth scrolling
Actual behavior
Choppy scrolling with low fps
Even much worse when the tagtree is opened.
Minimal config
runtime! debian.vim
set nocompatible

call plug#begin(expand('~/.vim/plugged'))
Plug 'vim-airline/vim-airline'
Plug 'preservim/tagbar'
Plug 'akinsho/bufferline.nvim', { 'tag': '*' }
Plug 'nvim-treesitter/nvim-treesitter', {'do': ':TSUpdate'}
Plug 'nvim-treesitter/nvim-treesitter-context'
call plug#end()

set mousemoveevent
lua << EOF
    require("nvim-treesitter.configs").setup({})
    require("treesitter-context").setup({enable = true, line_numbers=true})
    require("bufferline").setup({})
EOF

set noshowmode
nmap <silent><C-t> :TagbarToggle<CR>

set number
set relativenumber
augroup numbertoggle
    autocmd!
    autocmd BufEnter,FocusGained,InsertLeave * if(getbufinfo('%')[0]['listed'] == 1 || &buftype == 'help') | set relativenumber | endif
    autocmd BufLeave,FocusLost,InsertEnter   * set norelativenumber
augroup END

Steps to reproduce

wget -O /tmp/test.py https://raw.githubusercontent.com/FalcoGer/proxy/877891532bde972f980db0b28aae5d5a24273158/application.py
nvim -u minimal.vim /tmp/test.py (can't do --clean because it doesn't know what Plug is?)
:TSInstall python
gg
Press and hold down

Open tagbar with  and scroll back up to see even worse performance.

Are you sure this is an issue with this plugin?

It started to behave poorly after installing this plugin. I don't know what the issue is. I just want to use nvim, I don't know how to profile it or how to find the bottleneck. All I know is that it becomes better if I change any of those settings.
So where do you propose I post the bug report to? Everybody is just going to blame each other because of the nature of this bug.
What I find most curious in this lineup is bufferline, because as far as I know it doesn't do anything on CursorMoved or WinScrolled.

Issues can't be fixed if they can be properly diagnosed. Just saying this plugin is slow for you isn't a lot to go by, and doesn't provide much motivation for maintainers to fix it.

The problem also completely disappears if I turn the context off (:TSContextToggle).

But also when you disable any one of three other plugins? This could mean anything.

I just want to use nvim, I don't know how to profile it or how to find the bottleneck. I write python, c# and c++ programs, not vim plugins.

Treesitter is still considered quite experimental. If you just want to use nvim, then I suggest you avoid using experimental features and plugins, especially if you're not in a position to debug issues.

So where do you propose I post the bug report to? Everybody is just going to blame each other because of the nature of this bug.

If no useful bug report can be created, then nowhere. Not one vim plugin author gets paid for developing vim plugins, it's something they do for free in their spare time. No one wants to debug an issue for it to turn out to not be an issue with the software they are maintaining, it is a massive waste of their time. Plugin authors just want to use nvim too.

What I find most curious in this lineup is bufferline, because as far as I know it doesn't do anything on CursorMoved or WinScrolled. Maybe the issue is partly because the context is pushed down by one line?

Hmm, I kind of doubt it but maybe. Either way, requiring 4 plugins to reproduce an issue definitely feels like a time sink to debug, especially when 2 of the plugins aren't really used in the nvim community anymore.

especially when 2 of the plugins aren't really used in the nvim community anymore.

Which two, and what are the alternatives? I didn't find any deprecation warnings and from what I can see all of them have had their repos updated relatively recently.

I'll try and repro the issue with the steps you've provided, but I won't spend hours trying to debug this.

Thank you.

Is this a bug or is it not?

Well that has not been confirmed yet. A repro that only requires this plugin would confirm it as a bug. Identifying the root cause of issues is the hardest part, which authors hate doing. Fixing them is often easy.

It seems unreasonable to suggest that it shouldn't be reported because there is no place to do so. What am I to do instead?

Again, no one gets paid to fix these issues and you are not owed anything. You should take this software as is and just be grateful it even exists. You are free to raise issues, but you should never assume or expect they get fixed by someone that isn't you.

Right now my choices are to live with the lag or to not use this combination of settings and plugins. Neither seems very appealing. If you have a solution, I would be happy to try it.

When using open source software in a niche community, fixing the problem yourself should always be an option. Otherwise, yes, accepting the lag or not using this plugin are very valid options.

A workaround that works for context.vim enables you to not use autocommands for CursorMoved and WinScrolled but just on CursorHalted, which would be sufficient for me and probably is implementable with reasonable effort.

The updates are already debounced, so I don't think this will make much difference. We could always increase the debounce duration.

Hey guys, how are you doing?
@lewis6991 Ty by the replies!
It's seems to be a sort of issue/conflict with preservim/tagbar and windwp/nvim-ts-autotag.
You need to open tagbar in order to reproduce the issue.
https://github.com/nvim-treesitter/nvim-treesitter-context/assets/4944009/34490047-7467-4db4-88b1-773ec4de61c9

This is probably mostly off topic aside of performance concerns, but could using the nvim 0.10 virtual lines (like dropbar.nvim) improve the performance in any way? I tried it on a 400 line markdown file (giant lists for easy folding, depth ~4) and it was certainly a noticeable drop in scrolling, though it was quite slow beforehand too.
file in question

No


Drawer menu scrolling - Android

Internal test build:
Bottom drawer scrolling:

@JuliannaR I believe this is a regression.

Yes, focus jumps to the bottom of the drawer here too.

Checked against the beta build and the menu as far as scrolling acts the same.

There is some lag with lower powered devices, two finger gesture, read all work consistently, talkback has some issues if users swipe too quickly left to right to move though content individually.
We can research if there are ways to optimize this experience, but am okay to close for now as it is functioning.


OpenColorIO improvements, especially for 2.2

When OIIO builds against OCIO >= 2.2, in the event that there is no config supplied (e.g., specified by env $OCIO), use he new built-in "cg" config to supply a basic set of color spaces and transformations. Yay for OCIO 2.2 and its built-in configs!

New OIIO-understood aliases (regardless of whether they were in the config or not): "scene_linear", "srgb", "lin_srgb", and "ACEScg". It will do its best to know which, if any, supported color spaces are equivalent to these, even if arbitrarily named. For OCIO >= 2.2, some trickery is employed to use the new features that can convert to built-in config color spaces, transforming several colors in this manner and seeing if it gets what would be expected for various canonical color spaces.

oiiotool --help now a much abbreviated set of color information (just the OCIO version, and the config, and a hint about how to get more info).

A new command oiiotool --colorconfiginfo prints the FULL inventory of color management information, which now has been expanded to reveal the aliases of each color space, print the roles, and generally spruce up the output to be much more readable (though longer) for large configs.

New methods for our ColorConfig class:

resolve(name) turns the name, which could be a color space, an alias, a role, or an OIIO-understood universal name (like "sRGB") into a canonical color space name.
equivalent(name1,name2) returns whether or not the two names refer, ultimately, to equivalent color spaces (to the best of its ability to figure out).

Some minor changes (including additional const qualifiers) on the ColorConfig class.

Make testsuite tests use the built-in OCIO config if OCIO 2.2+ is found.

Build sonar against ocio2.2 so we are primarily testing the new code paths.

For OCIO >= 2.2, because we can count on the built-in configs, conditionally compile a bunch of hard-coded color transforms.  These will eventually disappear completely, someday, when OCIO 2.2 is the minimum supported version.

I think the behavior should all be largely unchanged when running OIIO in a facility that has a proper OCIO config pointed to by the $OCIO env variable and using color space names contained therein. So studio users will (I hope) see no changes. For users that have no OCIO config, or that are writing software that they want to work reasonably well for common cases no matter what config they encounter at runtime, this should give more helpful and consistent behavior when OCIO 2.2+ is used (which I now strongly recommend for anybody not constrained to 2.1 for compatibility reasons.
But because of the risk and the possible behavioral changes, this is all currently envisioned as OIIO 2.5 features, and I am not planning to backport it to the 2.4 release branch. Additionally, a lot has changed and I expect some additional tweaking and identification of edge cases I may have missed.

I should note that this is a big change that can cause behavior changes, especially when people switch to OCIO 2.2. As any unwanted or confusing changes get complaints, we can make amendments to the logic. I suspect this will be modified a few times before summer when it is incorporated into a supported release.
Here's how I think the cases break down:
No OCIO support at build time should not change any behavior. It will still understand how to linear<->sRGB, and not much else.
A fully color managed environment -- i.e. $OCIO points to a config and any color conversion functions reference color space names that are in the config -- should not change any behavior. I expect this is the standard setup for large studios.
Building against OCIO <= 2.1, but no OCIO config found at runtime -- this has traditionally been the usual case for people who get OIIO from package managers that provide the OCIO dependency, but that aren't themselves using a color managed workflow. Behavior shouldn't change, but maybe it did unintentionally and I will fix it if there are complaints.
Building against OCIO 2.2, but no OCIO config found at runtime (this should be increasingly common and the new case going forward of package manager, but no color managed workflow) will have greatly expanded capabilities because it will know about all the color management. There may be behavior changes, and I'm amenable to modifying them, but I'm not sure how to avoid any possible changes, because... Caveat emptor: the built-in OCIO 2.2 configs assume a scene_linear default working space with ACEScg color primaries. This is different from the no-config behavior of OCIO <= 2.1 or no-OCIO, because those previously would have interpreted a scene_linear->sRGB (say) as only a change in transfer function and not in color primaries. People who previously had used "linear" or "scene_linear" to describe color spaces of unknown (but presumed linear) files might want to switch to "lin_srgb" to be more specific that they are presuming the file has sRGB primaries.

Does anybody have concerns about this direction? I expect some refinement rounds before it's in a supported release series.

OK, merging. Let me know if any problems crop up.


Check this primitve message in Chrome

I think it is irrelevant of 3D. It is something with the theme of Wordpress probably.
Uncaught Error: Invocation of form runtime.connect() doesn't match definition runtime.connect(optional string extensionId, optional object connectInfo)
    at Object.normalizeArgumentsAndValidate (VM9568 extensions::schemaUtils:115)
    at Object.<anonymous> (VM9563 extensions::binding:363)
    at f (kwift.CHROME.min.js:1)
normalizeArgumentsAndValidate @ VM9704 extensions::schemaUtils:115
(anonymous) @ VM9699 extensions::binding:363
f @ kwift.CHROME.min.js:1
setTimeout (async)
(anonymous) @ kwift.CHROME.min.js:1
EventImpl.dispatchToListener @ VM9700 extensions::event_bindings:403
publicClassPrototype.(anonymous function) @ VM9706 extensions::utils:140
EventImpl.dispatch_ @ VM9700 extensions::event_bindings:387
EventImpl.dispatch @ VM9700 extensions::event_bindings:409
publicClassPrototype.(anonymous function) @ VM9706 extensions::utils:140
dispatchOnDisconnect @ VM9707 extensions::messaging:376

This is not related to the plugin at all. A browser extension triggers this.

ok you are probably right. I have deactivated all my plugins but dashlane. I will check if still the problem exists.


Continuous ir model

Signal model for continuous acquisition with and without inversion pulses.

Some basic questions / comments:

I do not really understand the parameters of the model.
Why is it required to specify tau and tacq_start?
Shouldn't it be enough to specify when acquisitions happen and when inversions happen? Or maybe I misunderstood the parameters, in which case this is more a comment regarding the docstring ;)

In general, can we maybe come up with better names than tacq or tau  ?
I would say spending a couple more characters to write out the meaning of these could be beneficial. Same  for ind and idx.

Is there a reason for the indexing instead of directly handling the times?
For an extreme example, you could consider looping in forward over the inversion times, for each inversion time selecting the acquisitions after it and before the next one, figuring out the tau, calculating part of the signal. and in the end stacking the signals. Would this be significantly slower, justifying the increased complexity?
If yes, you might consider underscoring the attributes that cannot be changed ofter creating the model.

Is there an advantage of using torch.div over normal "/" (or, if you prefer, t1.reciprocal()) ?

can we write the last reshaping in a way that works even if t1 is just a scalar-tensor?

We might consider making the tensors in operators  either buffers or parameters to allow move to gpu.

I have not managed to get this model to work with vmap.


Sunny branch

This is finally sharing the upgraded web-page.

Getting a hang of this. Already love it!


Updating ssd_fit_dists() to use more default arguments.

removed one more instance where nboot and ci were set unnecessarily. tested everything locally


pull from upstream dec 2023

Rebased commits from the convex fork

looks good, though the CI tests currently fail to compile — probably just syntax?

btw feel free to force-push main to match the upstream-dec-2023 branch — I have a slight pref for our deps being pinned to commits on main rather than commits on another branch, just so it's easier to open the forked repo and see what we've layered on top of the upstream without having to switch branches


[Feature Request] Implement is_running for _WorkflowInstanceImpl

Is your feature request related to a problem? Please describe.
Trying to produce messages to Kafka using aiokafka from a Temporal Workflow fails when trying to check if the current event loop is running. This is caused by the the Temporal event loop not fully implement the AbstractEventLoop interface, which includes is_running. Although more understandable when it comes to thread methods (to_thread, etc...), simpler methods like is_running should be easy to implement.

Describe the solution you'd like
Add an is_running method to _WorkflowInstanceImpl to better conform to the AbstractEventLoop interface, perhaps by checking _deleting:
def is_running(self) -> bool:
    return not self._deleting

Although perhaps it should always be True as long as there is a Runtime defined?

Additional context
The Python doc states that custom implementations of AbstractEventLoop should have all methods defined here. Although they are not being strict about it ("should have defined"), the closer Temporal's event loop gets to the interface, the more it can interoperate with other Python packages, like aiokafka.

We can't implement all methods of the event loop. You should never call Kafka from inside a workflow, only from inside an activity. A workflow is meant to be deterministic code only. There is never a case where you should need is_running and we can't really give an accurate value for that or is_closed like a non-distributed event loop can. I guess technically we could just always return True from it and always return False from is_closed, but it's not really accurate as it's never really running/closed in the background like a normal event loop.

You should never call Kafka from inside a workflow, only from inside an activity. A workflow is meant to be deterministic code only.

A call to Kafka (or any other external service) can be made deterministic: It's a matter of handling possible errors in a deterministic fashion. In my case, I ignore them and move on.

I'm actually kind of glad this raised a not-implemented so it catches illegal things happening to the workflow's event loop, such as code based on whether it's "running" or not as if it was a local event loop.

Not much else to say from my end as this is the direction you are glad to go with. We will deal with the issue from our side. Thank you for taking time to answer this request. I'll go ahead and close it.

Parts of python standard library assume that is_running and is_closed are implemented:
Eg consider this line in aiohttp https://github.com/aio-libs/aiohttp/blob/3f2f4a705f588d79a8649bfb17018cb30431df89/aiohttp/connector.py#L960
        if sys.version_info >= (3, 12):
            # Optimization for Python 3.12, try to send immediately
            resolved_host_task = asyncio.Task(coro, loop=loop, eager_start=True)

And if we look in the stdlib asyncio code we can see this calls:
https://github.com/python/cpython/blob/83827ad81972e8e6438a29274dbdcb923d7b1cc4/Lib/asyncio/tasks.py#L124
        if eager_start and self._loop.is_running():
            self.__eager_start()

But is_running() raises a NotImplementedError because you have left the method abstract.
https://github.com/temporalio/sdk-python/blob/0b327b04a95fc7954ed2c9e4e1a6c5de47c51ebd/temporalio/worker/_workflow_instance.py#L195
If you do not support parts of the standard AbstractEventLoop interface that even the standard core python library assumes exists, you need to at the very least reimplement them and give an error message that makes it crystal clear what is being done wrong. This error wasted hours of time trying to figure out what is going wrong (particularly nasty to debug since the event loop is only this special WorkflowInstance thing in certain circumstances).


Aduc 127 extendedErrorCode when applying a update

Hi, i'm running into a problem when the device update client runs the apply action, which happens right after installation of the update image. I'm getting extendedErrorCode 127 which indicates to me that some tool is missing in my linux system. My question is: Is there a list of dependencies that i need to provide on my linux system or better yet how can i find out what tool is missing on my system.
This is the log output:
2021-10-27T21:42:44.1987Z [I] Install succeeded [Install] 2021-10-27T21:42:44.1988Z [I] Action 'Install' complete. Result: 1, 0 [ADUC_Workflow_WorkCompletionCallback] 2021-10-27T21:42:44.1988Z [I] WorkCompletionCallback: Install succeeded. Going to state InstallSucceeded [ADUC_Workflow_WorkCompletionCallback] 2021-10-27T21:42:44.1989Z [I] Setting UpdateState to InstallSucceeded [ADUC_SetUpdateStateHelper] 2021-10-27T21:42:44.1989Z [I] Reporting state: InstallSucceeded (4) [AzureDeviceUpdateCoreInterface_ReportStateAndResultAsync] 2021-10-27T21:42:45.9129Z [D] ComponentName:azureDeviceUpdateAgent, propertyName:service [ADUC_PnP_ComponentClient_PropertyUpdate_Callback] 2021-10-27T21:42:45.9132Z [D] OrchestratorUpdateCallback received property JSON string ({"action":2,"updateManifest":"{\"manifestVersion\":\"2.0\",\"updateId\":{\"provider\":\"Elicity\",\"name\":\"beacon-firmware-update\",\"version\":\"<IP_ADDRESS>\"},\"updateType\":\"microsoft\/swupdate:1\",\"installedCriteria\":\"<IP_ADDRESS>\",\"files\":{\"fbfaffe17349d14f9\":{\"fileName\":\"beacon-image-update-raspberrypi4.swu\",\"sizeInBytes\":1162 [OrchestratorUpdateCallback] 2021-10-27T21:42:45.9169Z [I] Processing 'Apply' action [ADUC_Workflow_HandleUpdateAction] 2021-10-27T21:42:45.9169Z [I] UpdateAction: Apply [ADUC_MethodCall_Apply] 2021-10-27T21:42:45.9170Z [I] Setting UpdateState to ApplyStarted [ADUC_SetUpdateStateHelper] 2021-10-27T21:42:45.9170Z [I] Calling ApplyCallback [ADUC_MethodCall_Apply] 2021-10-27T21:42:45.9170Z [I] Apply thread started [ApplyCallback] 2021-10-27T21:42:45.9172Z [I] Applying data from /tmp/aduc-dl-211027213111 [Apply] 2021-10-27T21:42:45.9173Z [I] OrchestratorPropertyUpdateCallback ended [OrchestratorUpdateCallback] 2021-10-27T21:42:45.9726Z [E] Apply failed, extendedResultCode = 127 [Apply] 2021-10-27T21:42:45.9727Z [I] Action 'Apply' complete. Result: 0, 127 [ADUC_Workflow_WorkCompletionCallback] 2021-10-27T21:42:45.9728Z [E] Apply failed. error 0, 127 - Expecting service to send Cancel action [ADUC_Workflow_WorkCompletionCallback] 2021-10-27T21:42:45.9728Z [I] Setting UpdateState to Failed [ADUC_SetUpdateStateHelper] 2021-10-27T21:42:45.9728Z [I] Reporting state: 255, Failed (255); HTTP 500; result 0, 127 [AzureDeviceUpdateCoreInterface_ReportStateAndResultAsync] 2021-10-27T21:42:47.8239Z [D] ComponentName:azureDeviceUpdateAgent, propertyName:service [ADUC_PnP_ComponentClient_PropertyUpdate_Callback] 2021-10-27T21:42:47.8240Z [D] OrchestratorUpdateCallback received property JSON string ({"action":255,"updateManifest":"","updateManifestSignature":""}), property version (61) [OrchestratorUpdateCallback] 2021-10-27T21:42:47.8240Z [I] Cancel received with no operation in progress - returning to Idle state [ADUC_Workflow_HandleUpdateAction] 2021-10-27T21:42:47.8241Z [I] Setting UpdateState to Idle [ADUC_SetUpdateStateHelper] 2021-10-27T21:42:47.8241Z [I] Reporting state: 0, Idle (0); HTTP 200; result 1, 0 [AzureDeviceUpdateCoreInterface_ReportStateAndResultAsync] 2021-10-27T21:42:47.8242Z [I] Calling SandboxDestroyCallback [ADUC_MethodCall_Idle] 2021-10-27T21:42:47.8242Z [I] Destroying sandbox /tmp/aduc-dl-211027213111. workflowId:<PHONE_NUMBER>11 [SandboxDestroy] 2021-10-27T21:42:47.8244Z [I] UpdateAction: Idle. WorkflowId:<PHONE_NUMBER>47 [ADUC_MethodCall_Idle] 2021-10-27T21:42:47.8245Z [I] Calling IdleCallback [ADUC_MethodCall_Idle] 2021-10-27T21:42:47.8245Z [I] Now idle. workflowId:<PHONE_NUMBER>47 [Idle] 2021-10-27T21:42:47.8245Z [I] OrchestratorPropertyUpdateCallback ended [OrchestratorUpdateCallback]

Found the issue. I was missing fw_setenv.


[WIP] support protobuf  serialization

Ⅰ. Describe what this PR did
Ⅱ. Does this pull request fix one issue?
fixes https://github.com/seata/seata/issues/823
Ⅲ. Why don't you add test cases (unit test/integration test)?
Ⅳ. Describe how to verify it
Ⅴ. Special notes for reviews

Codecov Report

Merging #871 into feature_protubuf will decrease coverage by 16.05%.
The diff coverage is 8.87%.

@@                  Coverage Diff                   @@
##             feature_protubuf    #871       +/-   ##
======================================================
- Coverage               38.86%   22.8%   -16.06%
- Complexity               1028    1298      +270
======================================================
  Files                     220     330      +110
  Lines                    8501   19808    +11307
  Branches                 1024    2679     +1655
======================================================
+ Hits                     3304    4518     +1214
- Misses                   4812   14791     +9979
- Partials                  385     499      +114

Impacted Files
Coverage Δ
Complexity Δ

...re/protocol/protobuf/TransactionExceptionCode.java
75% <ø> (ø)
2 <0> (?)

...a/core/protocol/protobuf/GlobalStatusResponse.java
81.25% <ø> (ø)
2 <0> (?)

...core/protocol/protobuf/MergedWarpMessageProto.java
20.22% <ø> (ø)
20 <0> (?)

...rotocol/protobuf/GlobalLockQueryResponseProto.java
2.55% <ø> (ø)
3 <0> (?)

...ta/core/protocol/protobuf/GlobalStatusRequest.java
81.25% <ø> (ø)
2 <0> (?)

...otocol/protobuf/TransactionExceptionCodeProto.java
0% <ø> (ø)
0 <0> (?)

...a/core/protocol/protobuf/GlobalCommitResponse.java
81.25% <ø> (ø)
2 <0> (?)

.../core/protocol/protobuf/GlobalRollbackRequest.java
81.25% <ø> (ø)
2 <0> (?)

...ata/core/protocol/serialize/ProtobufSerialzer.java
53.65% <ø> (ø)
3 <0> (?)

.../core/protocol/protobuf/HeartbeatMessageProto.java
3.72% <ø> (ø)
3 <0> (?)

... and 218 more

Continue to review full report at Codecov.

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


Gotta say it: Brilliant project name!

Hey guys,
This isn't a bug, but the site (lessmsi.activescott.com) said I could post "suggestions, problems, comments" here, so I hope it's okay.
As a writer/editor, I had to rave about your brilliant project name. "Less MSIerables" (short version: LessMSI). Man! When I saw that, it stopped me in my tracks. I don't know about you, but it's the first software name I've ever seen that achieved a pun on a computing acronym by means of a Broadway show title—and as if that weren't enough, it even makes a pun on the show title itself ("Les" -> "Less")! If you're as good at writing code as you are at thinking of immensely clever product names, you have nothing to worry about.
With sincere admiration, following initial astonishment that coders could be that funny, A.

😆 👍


error building 'kivy.core.image.img_imageio' extension

I'm trying to build kivy from master with SDL2 support, I've it a bit of a brick wall with the installation though and the error output hasn't shed an awful lot of light on the problem. Wondering if anyone here could suggest what I might be doing wrong?
build output

---
Want to back this issue? **[Post a bounty on it!](https://www.bountysource.com/issues/12369199-error-building-kivy-core-image-img_imageio-extension?utm_campaign=plugin&utm_content=tracker%2F42681&utm_medium=issues&utm_source=github)** We accept bounties via [Bountysource](https://www.bountysource.com/?utm_campaign=plugin&utm_content=tracker%2F42681&utm_medium=issues&utm_source=github).

can't access the log.

Please reopen with the log, thanks


Migrate GP Vendor Classes

This change enhances the GP cloud migration to include migrating GP Vendor Classes to BC Vendor Posting Groups.

For each GP vendor class that has related accounts a new Vendor Posting Group will be created in BC.
Any vendor migrated from GP that is part of a vendor class will have the related Vendor Posting Group assigned.
A configuration option has been added to give the ability to not migrate Vendor Classes, disabled by default.

Availability update: We will publish a fix for this issue in the next update for release 20.
Build ID to track: 43275.

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. Please do not reply to this, as we do not monitor closed issues. If you have follow-up questions or requests, please create a new issue where you reference this one.
Build ID: 43303.


Bugfix: Fixes mutual log verbosity for klog/glog

Fixes: #387

/retest
Sender:Kubernetes Prow Robot<EMAIL_ADDRESS>Sent At:2019 Aug. 2 (Fri.) 17:02
Recipient:kubernetes-incubator/apiserver-builder-alpha<EMAIL_ADDRESS>Cc:Subscribed<EMAIL_ADDRESS>Subject:Re: [kubernetes-incubator/apiserver-builder-alpha] Bugfix: Fixes mutual log verbosity for klog/glog (#388)
[APPROVALNOTIFIER] This PR is APPROVED
Approval requirements bypassed by manually added approval.
This pull-request has been approved by:
The full list of commands accepted by this bot can be found here.
The pull request process is described here Needs approval from an approver in each of these files:
OWNERS
Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or mute the thread.


Inconstant Cluster/Decluster zoom levels

Reported on the forum here: https://groups.google.com/forum/#!topic/cesium-dev/HTAJnvct838

When zooming in and out with billboard clusters, the levels at which billboards cluster and uncluster is inconsistent. It seems that pixelRange is applied to determine when to group billboards into a cluster, but the cluster is "sticky" when zooming in and does not break apart until some zoom level where a significantly larger pixel range is seen.

This can be reproduced with the clustering demo. Here are steps:

Set "Pixel Range" to 100 and "Minimum Cluster Size" to 2.
Zoom in to get to a place where 2 billboards are nearby and may be clustered
Slowly zoom out from the 2 billboards and notice the last zoom level before the billboards combine.
Slowly zoom in and notice that you need to keep zooming past the level from above to get the billboards to uncluster.

If this is intended behavior, we can go ahead an close this issue.

@bagnell?

Likely related: https://community.cesium.com/t/bug-cluster-marker-not-showing-up-in-correct-position-when-zooming-in/26929/1

Reported on the forum: https://community.cesium.com/t/bug-cluster-marker-not-showing-up-in-correct-position-when-zooming-in/26929

I'm fairly certain this is because the createDeclutterCallback's  zoom-in-handler has a typo:

identify all points


Have the diff of the gems somewhere visible in the PR

I use daily depfu and it's awesome.
To be able to really review the changes being introduced, I would like to have the diff between the gems visible somewhere in the PR, for example as comment in the PR or a link to the diff in my repository dashboard

Hi @vpereira, not quite sure I follow? You mean the diff of your Gemfile? Or the code changes in the gems themselves? We do already link to the GitHub diff view in most cases. For some gems we can't find the info for various reason, maybe it's one of those?

Hi @theflow I mean the code changes in the gems themselves. Something like https://github.com/hennevogel/rg-poc/pull/2.

So what more do you want to see if we take https://github.com/hennevogel/rg-poc/pull/2 as an example? It already includes the list of commits, and if you click through it goes to the compare view which allows you to code review the diff.

So what more do you want to see if we take hennevogel/rg-poc#2 as an example? It already includes the list of commits, and if you click through it goes to the compare view which allows you to code review the diff.

Hi @theflow, I don't know if you scrolled his PR down, but if you read https://github.com/hennevogel/rg-poc/pull/2#issuecomment-706135912 (to show the important part) we have the list of files, meta data, diff per file, SHA1 from each file, etc.


New Stacked Borrows, now with better support for interior mutability

Waits on https://github.com/rust-lang/rust/pull/55549 to land

The rustc PR landed, this is ready for review.


refactor(network-exp): Adding the task banner in the network experiment

Signed-off-by: shubhamchaudhary<EMAIL_ADDRESS>
What this PR does / why we need it:

Adding the task banner in the pod network experiment

pod network loss
pod-network latency
pod network corruption

Which issue this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close that issue when PR gets merged): fixes #1423
Checklist

[x] Does this PR have a corresponding GitHub issue?
[ ] Have you included relevant README for the chaoslib/experiment with details?
[x] Have you added debug messages where necessary?
[x] Have you added task comments where necessary?
[x] Have you tested the changes for possible failure conditions?
[x] Have you provided the positive & negative test logs for the litmusbook execution?
[x] Does the litmusbook ensure idempotency of cluster state?, i.e., is cluster restored to original state?
[x] Have you used non-shell/command modules for Kubernetes tasks?
[ ] Have you (jinja) templatized custom scripts that is run by the litmusbook, if any?
[x] Have you (jinja) templatized Kubernetes deployment manifests used by the litmusbook, if any?
[x] Have you reused/created util functions instead of repeating tasks in the litmusbook?
[ ] Do the artifacts follow the appropriate directory structure?
[ ] Have you isolated storage (eg: OpenEBS) specific implementations, checks?
[ ] Have you isolated platform (eg: baremetal kubeadm/openshift/aws/gcloud) specific implementations, checks?
[x] Are the ansible facts well defined? Is the scope explicitly set for playbook & included utils?
[x] Have you ensured minimum/careful usage of shell utilities (awk, grep, sed, cut, xargs etc.,)?
[ ] Can the limtusbook be executed both from within & outside a container (configurable paths, no hardcode)?
[ ] Can you suggest the minimal resource requirements for the litmusbook execution?
[x] Does the litmusbook job artifact carry comments/default options/range for the ENV tunables?
[x] Has the litmusbooks been linted?

Special notes for your reviewer:

pod-network-corruption.txt
pod-network-latency.txt
pod-network-loss.txt


WIP: Add internal wrappers

This PR adds the released INTERNAL wrappers to modeling engines.
| Wrapper | Comments
--- | --- | ---
:heavy_check_mark:   | jyu-lb |  workaround some of the issues in simphony/simphony-jyulb#21. See commit message of 0012609afd52230cf2551c7dd5e49a8bf85765f5
:heavy_check_mark:  | lammps-md |  had to alter LD_LIBRARY_PATH when running tests to that lammps.py could find the lammps library
| kratos | waiting for simphony/simphony-kratos#9
| numerrin |
| openfoam | waiting for simphony/simphony-openfoam#23

@nathanfranklin, I have a quick review. It looks that integration is in the right path. We should push however for haste fixes to avoid problems down the line.

@nathanfranklin, I have a quick review. It looks that integration is in the right path. We should push however for haste fixes to avoid problems down the line.

I totally agree.  There are the three workaround-fixes that are currently added in this PR and then work-around for simphony/simphony-openfoam#25 .  It would be best if these issues were all fixed (and related fixes in the makefile were removed) before we merge this PR.  There are solutions for each of these issues so it should be doable (although i have temporarily given up on the lammps-library one).

(although i have temporarily given up on the lammps-library one).

I have not looked into the details, but the poor mans solution is to manipulate the activate script and add the new library search path.

This is being replaced by #36


SOURCE CONTROL

In Source Control  menu, initializeRepository button is not working

@sidhartha2004, could you please enable trace logging using the steps described in this wiki?  After that please open the folder/workspace, click on the "Initialize Repository" button and share the contents of the Git output window. Thank you!

This issue has been closed automatically because it needs more information and has not had recent activity. See also our issue reporting guidelines.
Happy Coding!


v1.2.0

[x] I have searched the issues of this repo and believe that this is not a duplicate.

Issue
Please tag the 1.2.0 release so that Homebrew cask can pull it.

@chrisfinazzo it's not release yet, it'll be during the week 👌


Debian Buster package missing

Current Debian Testing is codenamed Buster. Distro is missing for .deb package download: https://apache.bintray.com/couchdb-deb/dists/

We will only release a package for this distribution once it hits stable. Sorry.

No, you're mistaken. jessie was stable until a month ago and is now old-stable. The current release is stretch, for which we will be releasing a package shortly.
Our intention is to release binaries only for long-term support releases only once they are released, i.e. Ubuntu 14.04/16.04 LTS and Debian 8/9. If you want to use a testing branch, you are more than welcome to build from scratch using our well-defined development process.

Yes, true. OK, thanks.
Problem is however that it does not build on OTP20. I forced it, it looks that it is working (for now :)).

Hi Uche @uogbuji ,
Buster aka testing is not a fully supported release at this time. Again, when it becomes stable, we'll have an answer - which will likely mean that we provide libmozjs185 for buster as we do for CentOS 6.
Until then you will need to compile and install libmozjs185-dev yourself.

FYI: My current workaround is that I created an LXC container from my current distro*, downgraded it to stretch, installed CouchDB.

Packages for debian stretch are now available.


✨ API call rate-limiting

Description
This PR implements rate limiting to the external APIs within camply.
This PR fixes https://github.com/juftin/camply/issues/191
Has This Been Tested?
CI/CD
Checklist:

[x] I've read the contributing guidelines of this project
[x] I've installed and used .pre_commit on all my code
[x] I have documented my code, particularly in hard-to-understand areas
[x] I have made any necessary corresponding changes to the documentation

This hasn't been an issue closing for now


Add tests for packages download

Fixes #401
Motivation
Current integrate test doesn't cover cases when JarLocation/PyLocation/GoLocation is specified
Modifications
Add integrate tests for cases when user want to download function from pulsar
Verifying this change

[x] Make sure that the change passes the CI checks.

[x] This change added tests and can be verified as follows:

*Added integration tests for cases when JarLocation/PyLocation/GoLocation is specified

Documentation
Check the box below.
Need to update docs?

[x] no-need-doc
add integrate tests

is it possible to use the newly added skywalking-e2e-infra framework for this test?

is it possible to use the newly added skywalking-e2e-infra framework for this test?

sure, I will update this later


Issue using WebView to view videos inside carousel (iOS)

I'm having an issue viewing youtube videos in a webview control inside the carousel view.
I've tested a webview control on another page in my app to view videos from youtube and it works as expected.  I can go from normal screen at load to full screen and back.  However on iOS, when the same webview is placed in a CarouselView and the video is tapped it will open full screen, start to play, and then will go black.
I've used both an array of views and a data template selector to load the webview with the same results.
On inspection of the data template selector, when the video goes to full screen and starts to play the selector fires again and returns the web view template even though it is already loaded. After it returns the screen goes black.  However, if I go to normal screen then back it will play fine and does not hit the data template selector.  The height and Y properties change when trying to play the video as well.

Can you provide a demo?


Keep the letter v at the beginning of the version number for helm chart

/cherry-pick release-1.14

/cherry-pick release-1.14

/cherry-pick release-1.14


Fix def range

Update def range to be a position instead of a range. This fixes a bug
where a hover tooltip would only show one line when pressing cmd/ctrl,
instead of the entire definition.
This is the behavior of the most popular Go language server.

If this is the behavior of the most popular Go language server, LGTM.


[Task]: How to use GPU to call models in C++

What is the motivation for this task?
How to use GPU to call models in C++
Describe the solution you'd like
How to use GPU to call models in C++
Additional context
How to use GPU to call models in C++

The onnx model can run on the CPU, but the GPU cannot run it

As of now we don't support any C++ inference officially so this is a bit out of scope for this repository. Hence, I am closing this issue. You can have a look at https://github.com/openvinotoolkit/model_api in case you are interested in OpenVINO inference using C++.


Update registered vault details on signing devices

Registered vaults on signing devices should have the format: "[VaultName] + [Timestamp]"

Completed


Miscellaneous updates

What does this implement/fix? Explain your changes.

Ignore archives and files under External/ [ci skip]
Unset executable flag
Do not catch exceptions by value
Remove unused variable

@e-kwsm : thanks for this cleanup PR.
Because it touches a large number of files, I am going to wait to merge it until just before the release. This will hopefully be sometime next week.

@e-kwsm : thanks for doing all the work to create these cleanups!


Unix.Unix_error(Unix.EINVAL, "select", "") on Alpine Linux

Looks like lwt doesn't detect libev on Alpine 3.3 and falls back to select mode. Perhaps select mode is buggy (I guess not many people use it)?

Lwt's discover.ml detects libev correctly on Alpine by using pkg-config, but it isn't used when building with opam. Instead, that uses conf-libev, which doesn't support pkg-config.
@avsm is fixing...

Workaround: apk add ncurses-dev

I have a PR doing that #163


Improve the README

Add context and provide more information for non-developers.

Fixed in #37. Closing.


add SigMap over ReactNodes

closes https://github.com/joelburget/react-haskell/issues/22
I've built this locally, but haven't otherwise tested it.

:+1:


What happen to Iko and demo?

What happen to the Iko software. I thought TangoBB is outdated. When will you have the live demo version? Thanks

Iko has been rebranded back to TangoBB.
As for the live demo, no idea.

Like @cxdy says Iko (former TangoBB) has been rebranded back to TangoBB because of popluar demand.
A live demo will be uploaded in the next weeks. Mostly to get rid of all bugs.
We decided to delay the release of 1.5 because we want to add more features in before. For example polls.

Still waiting for a demo...

I think a demo will take a bit. In the last time there were some 'events' which will change TangoBB extremely. I'm not in the team anymore, only @jtpox is working on this. And he will give the bb a new theme engine what might can take a bit.
I split apart from TangoBB to create a new forum IkoBB, and we are creating a complete new core and system. So which way you want to go, you have to wait sadly. But we are doing our best to release a demo as fast as possible

Thanks for noticing me!
Good luck with it!


Image to Image Widget

Would have as input

Image

Would output:

Image

This should be similar in implementation to the image classification widget.

This is now done in repos such as https://huggingface.co/stabilityai/stable-diffusion-xl-refiner-1.0


[WFLY-19684] Upgrade WildFly Core to 26.0.0.Beta3

Jira issue: https://issues.redhat.com/browse/WFLY-19684

Tag: https://github.com/wildfly/wildfly-core/releases/tag/26.0.0.Beta3
Diff: https://github.com/wildfly/wildfly-core/compare/26.0.0.Beta2...26.0.0.Beta3

        Sub-task

[WFCORE-6931] -         Remove usage of deprecated AbstractWriteAttributeHandler constructors in threads subsystem

        Bug

[WFCORE-6956] -         Wrong endpoint-name if the JBoss server name and 'jboss.node.name' are both provided to the server

[WFCORE-6962] -         OverallInterfaceCriteriaUnitTestCase.testMultipleMatches* test cases fail on Win JDK21

[WFCORE-6963] -         AbstractModelResource$DefaultResourceProvider.hasChildren inefficiency degrades with child count

        Task

[WFCORE-6722] -         Standarize the error message shown when the server is launched with an valid but empty option

        Component Upgrade

[WFCORE-6883] -         Upgrade to jboss-logging 3.6.0 / jboss-logging-tools 3.0.0

[WFCORE-6958] -         Upgrade Undertow to 2.3.16.Final

[WFCORE-6965] -         Upgrade Galleon to 6.0.3.Final

        Enhancement

[WFCORE-6881] -         Support Stability Levels for the Management Parsers

Thanks @yersan

and @jamezp  :)


Random error when trying to open a livestream

http://prntscr.com/6icyao This is what I get when I try to open ANY stream, sometimes it's a 404 error "client not found"

Did this happen before or is this only happening now? Because twitch currently has some server issues.

I was watching a stream perfectly fine in source and it crashed so I tried other qualities and it didn't work still. It happened like 20 minutes ago for the first time, you reckon it's twitch not working?

Was happening for me too. Seems fine now

Now it doesn't work again. It is intermittent so I guess it's a twitch issue

Alright cheers. I was worried it was something wrong with the program itself because it was working fine all morning


Type code directly into ipython interpreter

The ability to type and run code directly in the ipython interpreter would be helpful.  This would also need to include code completion, syntax highlighting, etc.

Duplicate of #727

I could very well imagine for the extension a combined line-by-line and cell-wise execution as in my recently published Notepad++ extension PyPadPlusPlus:
https://github.com/bitagoras/PyPadPlusPlus
I really don't understand why people use a python console at all, since you cannot save your work and you cannot edit and repeat several commands easily. You are always in a decision situation: do I use the console or do I write a program in the editor? But there is a solution to this dilemma, as demonstrated with PyPadPlusPlus.
In the philosophy of PyPadPlusPlus the editor is like a workbench with tools and pieces of wood lying around. You can create new things, combine things, use it as a tool, improve it, lay it down somewhere and work at some other parts. Over time you combine all your workpieces until the object gets its finall shape. Carpenters work like this since thousand years, because this is the most efficient, creative and flexible way of working. For data scientist there should be a similar flexibility when working with data. Most environment allows you to either run whole files, only cells or only single commands in a console. This should be unified so that the user has the full choice what he likes to run.

Closing in favor of https://github.com/Microsoft/vscode-python/issues/727
Please comment on that, as that's a duplicate this must be closed in favor of that.


Fails to enable due to context not having user_preferences.

Blender 2.8 built on 2018-12-22.  After installing the plugin fails when attempting to enable it.  Commenting out the two lines that sets user preferences bypasses the issue and allows the plugin to load.

Oh they might have removed or renamed that option. Thanks for reporting, I will fix this

RNA: naming, user-preferences -> preferences

https://developer.blender.org/rB63fcbfc3a7325d79b9e916457d64c302ddfeadfa

Thanks, it's fixed now and I fixed another 2.8 bug that I found along the way.


Requested site: pinchofyum.com

Great package, have had a lot of success with it so far.
Would like to request the site pinchofyum.com be added.
Example recipe: https://pinchofyum.com/couscous-summer-salad
Not sure if/how feasible this site is to scrape but thought i'd ask.
Cheers!

Better late than never? I've added pinchofyum!


distributed gpu error

Thanks for your good works.
I was trying to train mobilenet v3 from scratch. I tried with the h-param you provide.
I have 8 gpus, I want to use two of them, but out of order.
I have assigned the device ID I want in train.py.
but i got a error like below :

Setting OMP_NUM_THREADS environment variable for each process to be 1 in default, to avoid your system being overloaded, please further tune the variable for optimal performance in your application as needed.

Training in distributed mode with multiple processes, 1 GPU per process. Process 1, total 2.
WARNING: 'drop_connect' as an argument is deprecated, please use 'drop_path'. Setting drop_path to 0.200000.
Traceback (most recent call last):
File "train.py", line 754, in
main()
File "train.py", line 292, in main
torch.distributed.init_process_group(backend='nccl', init_method='env://')
File "/home/oxigen/.env_hg/lib/python3.6/site-packages/torch/distributed/distributed_c10d.py", line 393, in init_process_group
store, rank, world_size = next(rendezvous_iterator)
File "/home/oxigen/.env_hg/lib/python3.6/site-packages/torch/distributed/rendezvous.py", line 172, in _env_rendezvous_handler
store = TCPStore(master_addr, master_port, world_size, start_daemon, timeout)
RuntimeError: Address already in use
Traceback (most recent call last):
File "/usr/lib/python3.6/runpy.py", line 193, in _run_module_as_main
"main", mod_spec)
File "/usr/lib/python3.6/runpy.py", line 85, in _run_code
exec(code, run_globals)
File "/home/oxigen/.env_hg/lib/python3.6/site-packages/torch/distributed/launch.py", line 263, in
main()
File "/home/oxigen/.env_hg/lib/python3.6/site-packages/torch/distributed/launch.py", line 259, in main
cmd=cmd)
subprocess.CalledProcessError: Command '['/home/oxigen/.env_hg/bin/python', '-u', 'train.py', '--local_rank=1', '/mnt/DATA/ImageNet/', '--model', 'mobilenetv3_large_100', '-b', '512', '--sched', 'step', '--epochs', '600', '--decay-epochs', '2.4', '--decay-rate', '.973', '--opt', 'rmsproptf', '--opt-eps', '.001', '-j', '7', '--warmup-lr', '1e-6', '--weight-decay', '1e-5', '--drop', '0.2', '--drop-connect', '0.2', '--model-ema', '--model-ema-decay', '0.9999', '--aa', 'rand-m9-mstd0.5', '--remode', 'pixel', '--reprob', '0.2', '--amp', '--lr', '.064', '--lr-noise', '0.42', '0.9']' returned non-zero exit status 1.
(.env_hg) oxigen@ubuntu:~/TEST/RESULT/mobile3_test2_warm/pytorch-image-models$ Traceback (most recent call last):
File "train.py", line 754, in
main()
File "train.py", line 359, in main
model.cuda()
File "/home/oxigen/.env_hg/lib/python3.6/site-packages/torch/nn/modules/module.py", line 307, in cuda
return self._apply(lambda t: t.cuda(device))
File "/home/oxigen/.env_hg/lib/python3.6/site-packages/torch/nn/modules/module.py", line 203, in _apply
module._apply(fn)
File "/home/oxigen/.env_hg/lib/python3.6/site-packages/torch/nn/modules/module.py", line 225, in _apply
param_applied = fn(param)
File "/home/oxigen/.env_hg/lib/python3.6/site-packages/torch/nn/modules/module.py", line 307, in
return self._apply(lambda t: t.cuda(device))
RuntimeError: CUDA error: out of memory
is there any idea on it? Thanks.

@gksruf that just looks like a typical CUDA out of memory error...
The distributed support here is pretty basic, it supports one GPU in each process so I don't know what you mean by out order. If you want to control which devices are available to to the distributed processes it may be easier to set CUDA_VISIBLE_DEVICES to filter your devices visible to the training processes... ie if you have GPU 0,1,2,3 ... set CVD to 3,1,0  and then launch ./distributed_train 3  and it should use GPU 3, 1, and 0.


feat: add typings

I just added this plugin to a TypeScript project, and my IDE just screamed at me. It wanted types, the plugin didn't have types. So I added types.
I figured I'd share.

LGTM! @claudiodekker any chance you can merge this?

@AdrianMrn It looks good to me, but I'll let @reinink do that instead, because if I start merging things he won't be aware of what's in part of the codebase and what isn't 😉

Thanks @innocenzi 👍

Now available in 0.2.2.

Cool, thanks!


Remove parens

Here's my first draft of removing parens.
I should be functionally complete though I still have several things to do.
Most importantly, I'm looking for some guidance on testing and general file to use to get the config opts.  I'm using Mix.Tasks.Format.formatter_opts_for_file("mix.exs") in the implementation and have added a non-exported locals_without_parens: [assert: 1] to this project's .formatter.exs just for testing purposes.  I don't love this, but since I have no clean way to do DI I need to either do it this way, use a mock (which means adding a library), or do some env-checking.  Perhaps there is another way I'm not thinking of.  What do you think?
Other than that I need to:

Refactor.
Add docs.
Add more tests.

And once again I'd love some guidance on what to call this corrector.
Thanks!

Elixir has a bunch of defaults for locals without parens. We should also add these.
  @locals_without_parens [
    # Special forms
    alias: 1,
    alias: 2,
    case: 2,
    cond: 1,
    for: :*,
    import: 1,
    import: 2,
    quote: 1,
    quote: 2,
    receive: 1,
    require: 1,
    require: 2,
    try: 1,
    with: :*,

    # Kernel
    def: 1,
    def: 2,
    defp: 1,
    defp: 2,
    defguard: 1,
    defguardp: 1,
    defmacro: 1,
    defmacro: 2,
    defmacrop: 1,
    defmacrop: 2,
    defmodule: 2,
    defdelegate: 2,
    defexception: 1,
    defoverridable: 1,
    defstruct: 1,
    destructure: 2,
    raise: 1,
    raise: 2,
    reraise: 2,
    reraise: 3,
    if: 2,
    unless: 2,
    use: 1,
    use: 2,

    # Stdlib,
    defrecord: 2,
    defrecord: 3,
    defrecordp: 2,
    defrecordp: 3,

    # Testing
    assert: 1,
    assert: 2,
    assert_in_delta: 3,
    assert_in_delta: 4,
    assert_raise: 2,
    assert_raise: 3,
    assert_receive: 1,
    assert_receive: 2,
    assert_receive: 3,
    assert_received: 1,
    assert_received: 2,
    doctest: 1,
    doctest: 2,
    refute: 1,
    refute: 2,
    refute_in_delta: 3,
    refute_in_delta: 4,
    refute_receive: 1,
    refute_receive: 2,
    refute_receive: 3,
    refute_received: 1,
    refute_received: 2,
    setup: 1,
    setup: 2,
    setup_all: 1,
    setup_all: 2,
    test: 1,
    test: 2,

    # Mix config
    config: 2,
    config: 3,
    import_config: 1
  ]

Hi Andrew,
This looks great. Thanks again for your work. I've left you some comments. Have a nice Sunday.

Elixir has a bunch of defaults for locals without parens. We should also add these.

Oh, and looking at this I now understand :* which I did not know you can do!
When using Mix.Tasks.Format.formatter_opts_for_file("mix.exs") in my applications it returns the above list but doesn't here.  I clearly need to better grok how this function works but will figure it out.
Thanks for all the feedback!  I will finish this up later today.

  Thanks, I learned some stuff there :)
There are only a few tests but I feel they are actually pretty sufficient as they cover the main cases.  Anything more feels like I'm just testing Sourceror.

Thanks for merging but I was planning on adding docs!  I can still do that if you like though fine by me if you'd prefer to.  Very sorry for the delay I got caught up in another personal project.

Oh, sorry. It would be great if you want to could add some docs. I would also like to ask you if it is ok if you to rename the task to LocalsWithoutParens. I have done this alright in main, but we could change the name again.
No worries about the delay, all is well and good. We have no deadlines here.


Feat: Added requantize function and tests

What does this PR do?
This PR addresses issue #162 by adding a requantize function with the following signature, and appropriate tests.
def requantize(model, state_dict):

The function does the following, inplace:

Checks to make sure the model isn't a Transformers model which has been distributed via accelerate (ie device_map=auto)
Moves the model to the meta device
Quantizes it
Moves the empty, quantized model to the cpu
Loads the state dict into the model.

At this point, the user can move the requantized model to their preferred device.
Who can review?
@dacorvo
Also, FYI @ManoBharathi93

Added a new commit creating a new file for the tests. I'm not sure what to do RE:

Here we should also check for weight qtype and parameters. Same for activations.

I am happy to try again with more instruction, or just observe how you would do it with a commit of your own.

Ok, I will keep working on it tomorrow. Thanks for your attention on this.

@calmitchell617  since you have not done any commits last 2 days I made PR by referring your commit with @dacorvo suggestions on your commit.

Hey @ManoBharathi93, appreciate your persistence. I'm going to continue here anyway, as an exercise.

Hey @ManoBharathi93, appreciate your persistence. I'm going to continue here anyway, as an exercise.

Looking forward to learn from you @calmitchell617

Ok, I will make those changes soon.
FYI, I'm writing a review / usage guide of Quanto for my blog. If you would like to see a draft of it to make sure I'm not misrepresenting anything, I would be happy to send you a draft and give you a chance to critique.

Ok, I will make those changes soon.
FYI, I'm writing a review / usage guide of Quanto for my blog. If you would like to see a draft of it to make sure I'm not misrepresenting anything, I would be happy to send you a draft and give you a chance to critique.

That's very nice of you to ask. I will be happy to take a look.

Ok, send me an email to
cmitch29
@
icloud.com
And I'll send you the review, which exists as a Markdown file.

There seems to be an issue with the formatting of your changes. If you applied style locally and don't have that error, please let me know and I will check if we need to update or pin the linter version.

Ah, darn it. Forgot to run make style. I'm too used to working on repos by myself :-D
I amended the commit.


[Xpt] Title

Description to be added

[ ] can_be_archived?() in the Item class

should return true if published_date is older than 10 years
otherwise, it should return false

[ ] move_to_archive() in the Item class

should reuse can_be_archived?() method
should change the archived property to true if the result of the can_be_archived?() method is true
should do nothing if the result of the can_be_archived?() method is false


merge package template from Kronos-Tools/npm-package-template

.npmignore

chore(npm)

Codecov Report

Merging #171 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #171   +/-   ##
=======================================
  Coverage   90.54%   90.54%
=======================================
  Files           1        1
  Lines          74       74
=======================================
  Hits           67       67
  Misses          7        7

Continue to review full report at Codecov.

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

Changes Unknown when pulling 10e66d8742102403aa98fd490d6de071e949bd86 on template-sync-1 into ** on master**.


Add tabled data for HarvardRT

🚀 Feature Request
Add tabled data for the HarvardRT dataset. Of course, we don't need to add PHI (e.g., the clinical data) - but just the features (potentially, not mandatory) and, most importantly, the scores of the linear model (so that whoever wants to replicate the KM analysis or use that model further knows what the median split threshold is).
Once again, I'm happy to have them in the idc-radiomics-reproducibility repo instead of here.
🔈 Motivation
This would help transparency and reproducibility, and would make the predictive model usable directly by others.

@denbonte
The linear model scores should be available already using this download:
https://aim-harvard.github.io/foundation-cancer-image-biomarker/user-guide/analysis/#predictions-pipeline
The foundation_features.csv under HarvardRT folder should contain the the scores on the validation set to get the median split threshold.
As for the features, what do you think? Should we share them? The benefit is we can then also show the linear model training pipeline.

The foundation_features.csv under HarvardRT folder should contain the the scores on the validation set to get the median split threshold.

Perfect!

As for the features, what do you think? Should we share them? The benefit is we can then also show the linear model training pipeline.

To show the training pipeline, we would probably have to share a minimal amount of information from the HarvardRT cohort as well (e.g., survival data) - and I'm pretty sure the cohort is not cleared for that. We can discuss, but for the time being I don't see this as crucial!

Added everything in the last couple of commits (https://github.com/ImagingDataCommons/idc-radiomics-reproducibility/commit/434b547cf2d51df2436ae47fed5f94eea263a49e and https://github.com/ImagingDataCommons/idc-radiomics-reproducibility/commit/0d2a7650b7ff39618c6ff003f41d964e795487b8).
Works perfect. I'm going to push the notebook to the repo in a few minutes.


⚠ Bofrost.lu has degraded performance

In d8f3129, Bofrost.lu (https://www.bofrost.lu) experienced degraded performance:

HTTP code: 200
Response time: 809 ms

Resolved: Bofrost.lu performance has improved in d53b1c7 after 13 minutes.


Add support for retrieving token from uri

Codecov Report

Merging #178 into master will not change coverage.
The diff coverage is n/a.

@@          Coverage Diff          @@
##           master   #178   +/-   ##
=====================================
  Coverage      75%    75%
=====================================
  Files           2      2
  Lines          40     40
=====================================
  Hits           30     30
  Misses          5      5
  Partials        5      5

Continue to review full report at Codecov.

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


Fail to active SpeckleBlender 2.0 - path process error

Expected vs. Actual Behavior
Expected to be able to successfully activate SpeckleBlender 2.0 in Blender Add-ons UI but I get an error.
Reproduction Steps & System Config (win, osx, web, etc.)
OS: Manjaro Linux
python: 3.9.5

Install specklepy from this repo in python site-packages
symlink bpy_speckle from speckle-blender repo in ~/.config/blender/2.92/scripts/addons/bpy_speckle/
Activate SpeckleBlender 2.0 in Blender Add-ons

Error message:
Traceback (most recent call last):
  File "/usr/lib/python3.9/pkgutil.py", line 416, in get_importer
    importer = sys.path_importer_cache[path_item]
KeyError: PosixPath('/usr/lib/python3.9/site-packages/specklepy/api/resources')

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "/usr/share/blender/2.92/scripts/modules/addon_utils.py", line 351, in enable
    mod = __import__(module_name)
  File "/home/cyril/.config/blender/2.92/scripts/addons/bpy_speckle/__init__.py", line 56, in <module>
    from specklepy.api.client import SpeckleClient  # , SpeckleCache
  File "/usr/lib/python3.9/site-packages/specklepy/api/client.py", line 6, in <module>
    from specklepy.api import resources
  File "/usr/lib/python3.9/site-packages/specklepy/api/resources/__init__.py", line 8, in <module>
    for (_, name, _) in pkgutil.iter_modules([Path(__file__).parent]):
  File "/usr/lib/python3.9/pkgutil.py", line 130, in iter_modules
    for i in importers:
  File "/usr/lib/python3.9/pkgutil.py", line 420, in get_importer
    importer = path_hook(path_item)
  File "<frozen importlib._bootstrap_external>", line 1601, in path_hook_for_FileFinder
  File "<frozen importlib._bootstrap_external>", line 1476, in __init__
  File "<frozen importlib._bootstrap_external>", line 177, in _path_isabs
AttributeError: 'PosixPath' object has no attribute 'startswith'

Proposed Solution (if any)
Except there is a better way path apparently needs to be converted to a string.
Optional: Affected Projects

speckle-py
speckle-blender

Hi Cyril,
I don't use Linux so I'm not sure if this differs there, but from what I understand Blender does not use your system's Python, but it's own packaged version located within the Blender 2.92 dir (I believe it's Python 3.7). You'll need to install specklepy in blender/2.92/scripts/modules so it can be picked up by Blender when activating the addon. Let me know if you still get any errors after doing so.
To be honest thought, I'm a bit confused by your error messages as Python 3.9 shouldn't be getting called at all so perhaps there's something else here that I'm missing...

It depends how the package you use have been compiled. But GNU/Linux distribution packages usually compile using system's python (this way all packages are much lighter). If it was not the case I would have got an import error.

As I thought I can reproduce it outside of Blender. That's why I initially opened this issue in speckle-py repo.
Code:
from specklepy.api import resources

Error message:
Exception has occurred: AttributeError
'PosixPath' object has no attribute 'startswith'
  File "/home/cyril/Documents/test.py", line 1, in <module>
    from specklepy.api import resources


Prove success but verifying error with provided cache data 21231350

Hey,
I tried the newest main branch. Everything worked well but the verification errors with
Failed to verify proof.: claim digest does not match the expected digest e41ab753d2f985a952a33460d12932707674cd34c57a9de9ebb9655494a9ae1c; expected 2e9b8f774fb3f130e3b88942e7111b4b4742033e62c3fbc57e6fda2e675609aa
Here are my steps:

just metal
RISC0_DEV_MODE=true just ethereum prove --block-number=21231350 --cache=./bin/ethereum/data

I think it's due to the version of risc0 zkvm used in branch main.

When using Bonsai to make snark proof, Error: Bonsai prover workflow [360ec2c7-b642-4674-99b7-e96329b837a7] exited: FAILED err: Executor failed: Session limit exceeded occurred.


Add "sparse" block attribute.

One existing problem is, TIR schedule primitives would not inherit block attributes: for example, when you call blockize, the newly generated block would not copy the original block attributes.
The behavior should be fixed later.

One existing problem is, TIR schedule primitives would not inherit block attributes: for example, when you call blockize, the newly generated block would not copy the original block attributes.

Nice catch! We should be careful when playing with the primitives 👍


Calibre Mac OS X

Hi,
I've been trying to make your script work in Mac OS X, but the directory /usr/lib64/calibre/calibre/ebooks/conversion/plugins/doesn't exist. The file builtins.py doesn't exist either.
I would really like to give a try to your plugin.
Many thanks.
Nicolas Vaughan

Hi,
I'm afraid, that it is not possible to simply "monkey-patch" Calibre on
Mac OS X. The dmg package contains Mach executables, not raw python
source files.
If you really want to give a try this plugin, you have to run in on
Linux or at least run it from the source code itself (after "applying"
this plugin). I'm not a OS X user, though, so I might be wrong...
Try this guide:
https://manual.calibre-ebook.com/develop.html#os-x-development-environment
Best Regards,
Arek

Hi Arkq, I tried it also on my Linux machine, but it also lacks those directories/files. Is it perhaps because newer versions of Calibre don't contain them? Thanks. Nicolas
On Wed, Jun 22, 2016 at 03:12, Arkadiusz Bokowy<EMAIL_ADDRESS>wrote:
Hi,
I'm afraid, that it is not possible to simply "monkey-patch" Calibre on
Mac OS X. The dmg package contains Mach executables, not raw python
source files.
If you really want to give a try this plugin, you have to run in on
Linux or at least run it from the source code itself (after "applying"
this plugin). I'm not a OS X user, though, so I might be wrong...
Try this guide:
https://manual.calibre-ebook.com/develop.html#os-x-development-environment
Best Regards,
Arek
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub [https://github.com/Arkq/calibre-latex/issues/1#issuecomment-227673491] , or mute the thread [https://github.com/notifications/unsubscribe/ACM7jUKB0yL5bTTLod3gv_aCCUO4kymBks5qOO57gaJpZM4I5FN0] .

I've checked, and it seems, that in the newest version of Calibre, all those directories/files exist:
https://github.com/kovidgoyal/calibre/tree/master/src/calibre/ebooks/conversion/plugins/
https://github.com/kovidgoyal/calibre/blob/master/src/calibre/customize/builtins.py
But I might miss something from the big picture, because I'm currently using Calibre version 1.20, which is kinda old :/
Could you send the exact version of Calibre and the Linux distribution you've been trying to use? Please note, that the installation guide assumes, that one is using 64 bit version of Linux.

OK, thanks.
I'm using Ubuntu 14.04 64 bits.
My Calibre version is 2.59.
Cheers,
Nicolas
On 06/22/2016 01:57 PM, Arkadiusz Bokowy wrote:

I've checked, and it seems, that in the newest version of Calibre, all
those directories/files exist:
https://github.com/kovidgoyal/calibre/tree/master/src/calibre/ebooks/conversion/plugins/
https://github.com/kovidgoyal/calibre/blob/master/src/calibre/customize/builtins.py
But I might miss something from the big picture, because I'm currently
using Calibre version 1.20, which is kinda old :/
Could you send the exact version of Calibre and the Linux distribution
you've been trying to use? Please note, that the installation guide
assumes, that one is using 64 bit version of Linux.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/Arkq/calibre-latex/issues/1#issuecomment-227843491,
or mute the thread
https://github.com/notifications/unsubscribe/ACM7jXD9bgjI4rSBtTK5xqegvIpOpStPks5qOYW0gaJpZM4I5FN0.

OK, so I gather, that you've installed Calibre not from Ubuntu's package repository, but from Calibre homepage, using provided instruction: https://calibre-ebook.com/download_linux Then, you have had to select the installation directory, which is most likely /opt (it could have been anything, though).
From this point, things got a little bit more complicated. The source code, which is provided by this way of installation is "precompiled" (*.pyo files). Buuuuutt, it is possible (theoretically) to patch the source anyway.
Wherever you've installed Calibre, try to find the directory "lib/python2.7/site-packages/calibre/". This is the location of the source code (mangled, though). Then, place the tex_outpu.py plugin file in the plugins location, e.g.: /opt/calibre/lib/python2.7/site-packages/calibre/ebooks/conversion/plugins/. Next step is to register this plugin.... Copy, this file into the lib/python2.7/site-packages/calibre/customize/ directory, and apply the patch (add to the end of the file):
from calibre.ebooks.conversion.plugins.tex_output import LatexOutput
plugins += [LatexOutput]

And that's all. Everything should work flawlessly. I will check this out tomorrow and if it works, I will update the installation instruction.
If my guessing is not correct, please tell me exactly how you've installed Calibre.

I did everything you said. However, I get the following error message:
Traceback (most recent call last):
  File "site.py", line 66, in main
  File "site-packages/calibre/gui_launch.py", line 63, in calibre
  File "site-packages/calibre/gui2/main.py", line 525, in main
  File "site-packages/calibre/gui2/main.py", line 371, in run_gui
  File "site-packages/calibre/gui2/ui.py", line 28, in <module>
  File "site-packages/calibre/db/legacy.py", line 19, in <module>
  File "site-packages/calibre/db/cache.py", line 18, in <module>
  File "site-packages/calibre/customize/ui.py", line 17, in <module>
  File "/opt/calibre/lib/python2.7/site-packages/calibre/customize/builtins.py", line 1858, in <module>
    from calibre.ebooks.conversion.plugins.tex_output import LatexOutput
ImportError: No module named tex_output

I installed Calibre with the official installer from the website.

Sorry. Just checked I made a mistake. It's working now!
A question: would this also work on a Mac installations?

I'm glad, that my guessing was right. As for Mac, I'm not sure, I will have to check, how the source code is delivered in the installation package. If Mac installation contains *.pyo files, this recipe should work for Mac as well. But Mac installation package is like CD image, and it might not be possible to modify it :/ You might give it a try, though :)

It also works on a Mac. Cheers!
On Wed, Jun 22, 2016 at 16:07, Arkadiusz Bokowy<EMAIL_ADDRESS>wrote:
OK, so I gather, that you've installed Calibre not from Ubuntu's package repository, but from Calibre homepage, using provided instruction: https://calibre-ebook.com/download_linux [https://calibre-ebook.com/download_linux] Then, you have had to select the installation directory, which is most likely /opt (it could have been anything, though).
From this point, things got a little bit more complicated. The source code, which is provided by this way of installation is "precompiled" (*.pyo files). Buuuuutt, it is possible (theoretically) to patch the source anyway.
Wherever you've installed Calibre, try to find the directory "lib/python2.7/site-packages/calibre/". This is the location of the source code (mangled, though). Then, place the tex_outpu.py plugin file in the plugins location, e.g.: /opt/calibre/lib/python2.7/site-packages/calibre/ebooks/conversion/plugins/. Next step is to register this plugin.... Copy, this file [https://github.com/kovidgoyal/calibre/blob/v2.59.0/src/calibre/customize/builtins.py] into the lib/python2.7/site-packages/calibre/customize/ directory, and apply the patch (add to the end of the file):
from calibre.ebooks.conversion.plugins.tex_output import LatexOutput
plugins += [LatexOutput]
And that's all. Everything should work flawlessly. I will check this out tomorrow and if it works, I will update the installation instruction.
If my guessing is not correct, please tell me exactly how you've installed Calibre.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub [https://github.com/Arkq/calibre-latex/issues/1#issuecomment-227876974] , or mute the thread [https://github.com/notifications/unsubscribe/ACM7jYa248dB_XifbUR4lVPHNTlQXOrrks5qOaP8gaJpZM4I5FN0] .


Address mcounteren+vstimecmp public-review feedback

mcounteren.TM should affect whether vstimecmp is accessible.
h/t @jhauser-us and @pdonahue-ventana

Thanks, @pdonahue-ventana.  @gfavor Can you check to make sure this is the extent of the public-review feedback that needs to be addressed?


Wrong timestamp

Hi,
I've tried on two different rpie's with raspbian images and they have same timestamp with weird values. Debian VM gives normal epoch time.
Date/time is set up correctly.
(This is opened with wireshark)
Frame 1: 226 bytes on wire (1808 bits), 226 bytes captured (1808 bits) on interface 0
    Interface id: 0 (wlan1)
    Encapsulation type: IEEE 802.11 plus radiotap radio header (23)
    Arrival Time: Jan  1, 1970 01:25:10.864502000 Romance Standard Time
    [Time shift for this packet: 0.000000000 seconds]
    Epoch Time: 1510.864502000 seconds
    [Time delta from previous captured frame: 0.000000000 seconds]
    [Time delta from previous displayed frame: 0.000000000 seconds]
    [Time since reference or first frame: 0.000000000 seconds]
    Frame Number: 1
    Frame Length: 226 bytes (1808 bits)
    Capture Length: 226 bytes (1808 bits)
    [Frame is marked: False]
    [Frame is ignored: False]
    [Protocols in frame: radiotap:wlan_radio:wlan]

In bigger capture file I noticed that starting packets have epoch values 1000/2000 and then on frame 255 it jumps to (-) value, like seen below. Later packets switch back to 1000/2000 and to (-) value again. That was on approx. 1hr long capture file.
Frame 255: 106 bytes on wire (848 bits), 106 bytes captured (848 bits) on interface 0
    Interface id: 0 (wlan0)
    Encapsulation type: IEEE 802.11 plus radiotap radio header (23)
    Arrival Time: Not representable
    [Time shift for this packet: 0.000000000 seconds]
    Epoch Time: -140464757.873059000 seconds
    [Time delta from previous captured frame: -140466901.308239000 seconds]
    [Time delta from previous displayed frame: -140466901.308239000 seconds]
    [Time since reference or first frame: -140466833.115197000 seconds]
    Frame Number: 255
    Frame Length: 106 bytes (848 bits)
    Capture Length: 106 bytes (848 bits)
    [Frame is marked: False]
    [Frame is ignored: False]
    [Protocols in frame: radiotap:wlan_radio:wlan]

I don't quite understand the code and I'm not really sure what to check next.
Any ideas?
 Thank you for reporting it.
It should be fixed by this commit:
https://github.com/ZerBea/hcxdumptool/commit/a911c15698efd52292120c467a6281f2d2ec2800
Frame 1: 293 bytes on wire (2344 bits), 293 bytes captured (2344 bits) on interface 0
Interface id: 0 (wlan0)
Encapsulation type: IEEE 802.11 plus radiotap radio header (23)
Arrival Time: Feb 26, 2019 17:17:57.698497000 CET
[Time shift for this packet: 0.000000000 seconds]
Epoch Time:<PHONE_NUMBER>.698497000 seconds
[Time delta from previous captured frame: 0.000000000 seconds]
[Time delta from previous displayed frame: 0.000000000 seconds]
[Time since reference or first frame: 0.000000000 seconds]
Frame Number: 1
Frame Length: 293 bytes (2344 bits)
Capture Length: 293 bytes (2344 bits)
[Frame is marked: False]
[Frame is ignored: False]
[Protocols in frame: radiotap:wlan_radio:wlan]
Radiotap Header v0, Length 18
802.11 radio information
IEEE 802.11 Beacon frame, Flags: ........
IEEE 802.11 wireless LAN
Please test and let me know if it works for you. In that case I'll move hcxdumptool and hcxtools to 5.1.3

Ok, from a quick test capture it looks like it is fixed!
Thanks for quick response/fix!

Great. Fixed also hcxhash2cap and hcxpcaptool and moved hcxdumptool and hcxtools up to v5.1.3

hi,
I have a question like one asked above.
Q. The time column in Wireshark displays 0.000000 for some packets (Seconds since previous captured packet). My students asked me why it happens and unfortunately I couldn't answer them.
What's the reason? Guide me in very simple way (as I am a novice in Wireshark).
119	0.000393	<IP_ADDRESS>	<IP_ADDRESS>	UDP	77	63194 → 443 Len=35
120	0.000902	<IP_ADDRESS>	<IP_ADDRESS>	UDP	1292	443 → 63194 Len=1250
121	0.000000	<IP_ADDRESS>	<IP_ADDRESS>	UDP	1292	443 → 63194 Len=1250
122	0.000272	<IP_ADDRESS>	<IP_ADDRESS>	UDP	1292	443 → 63194 Len=1250
123	0.000000	<IP_ADDRESS>	<IP_ADDRESS>	UDP	1292	443 → 63194 Len=1250
124	0.000000	<IP_ADDRESS>	<IP_ADDRESS>	UDP	1292	443 → 63194 Len=1250
Thanks in Advance
Anjum Altaf

How the timestamp works is OS dependent:
https://wiki.wireshark.org/Timestamps
https://www.wireshark.org/docs/wsug_html_chunked/ChAdvTimestamps.html
https://www.wireshark.org/docs/wsug_html_chunked/ChWorkTimeFormatsSection.html
The timestamp presentation format and the precision in the packet list can be chosen using the View menu of Wireshark:
https://www.wireshark.org/docs/wsug_html_chunked/ChWorkTimeFormatsSection.html
BTW:
hcxdumptool/hcxlabtool use:
gettimeofday(&tv, NULL);
timestamp = ((uint64_t)tv.tv_sec *1000000) +tv.tv_usec -512;


Enable optimization passes on materials with external samplers

Because external samplers are not properly supported by SPIRV and
associated library (spirv-cross and spirv-tools) we currently disable
all optimizations when we encounter a material with external samplers.
This however causes issues on some misbehaved drivers (not running
the optimizations has a side effect which causes a crash). To enable
the optimization pass we simply rely on the Vulkan codegen target to
substitute samplerExternal with sampler2D. We then analyze the output
GLSL (post-optimization) and revert the relevants sampler2D declarations
to samplerExternal declarations.
This fixup only occurs after optimization and for mobile targets and
if external samplers were declared.

Good point, I'll add an explanation.


how to get the clicked latitude and longitude and pass them to another controller

I have this in my controller which basically will do is to send a pop up telling the  latitude and longitude...
public function getMap()     {         $config['center'] = '9.562389, 44.077011';         $config['zoom'] = '14';         $config['map_height'] = '500px';         $config['scrollwheel'] = false;         $config['onclick'] = 'alert(\'You just clicked at: \' + event.latLng.lat() + \', \' + event.latLng.lng());';         $gmaps = new GMaps($config);         $gmaps->initialize($config);         $map = $gmaps->create_map();         return view('admin.mapTest')->with('map', $map);     }
so my question is... how to send the latitude and longitude to another controller?
note: forexample i wann create save that clicked map

Hey, did you find a solution for this?


feat: Add throttling of log messages

Codecov Report
Attention: 4 lines in your changes are missing coverage. Please review.

Files
Coverage Δ

.../main/java/com/cursorinsight/trap/util/TrapTime.kt
60.00% <0.00%> (-15.00%)
:arrow_down:

...ain/java/com/cursorinsight/trap/util/TrapLogger.kt
84.21% <82.35%> (+50.87%)
:arrow_up:

:loudspeaker: Thoughts on this report? Let us know!.


Oops! Randomization failed. Please try again with clean files.

Hi,
I do not know if this github will get any answers to be honest, but I just decided to use this randomizer, but I keep getting the same error over and over. I am using a brand new SD card if this means anything (I am not the best at this type of stuff). Any help would be appreciated.

Hello
I need help for this same problem (i got the error message "try again with clean files")
Can anyone tell me how it was fixed, please? This is a total begging.
I am new to github, maybe there was a simple way to see how this issue was solved, sorry for the eventual disturbing and my possible blunt behaviour,
Thanks for the understanding
Have a good day, please.

Hello I need help for this same problem (i got the error message "try again with clean files") Can anyone tell me how it was fixed, please? This is a total begging. I am new to github, maybe there was a simple way to see how this issue was solved, sorry for the eventual disturbing and my possible blunt behaviour, Thanks for the understanding Have a good day, please.

Hi. Make sure you're using the latest version HERE and not downloading the actual Github repository. It shouldn't give any problems. If there are still problems, I'm not sure what could be causing them to be honest. You would need to give me a more in depth description of what problem you're having. You can also click the checkmark for debug log and send me that file.

Hi. I have been trying to launch the "OddRhythm.exe"  but it says I need a java runtime environnment. So I did a lot of things for several hours changing settings and following java tutorials to make sure my java runtime environnment file worked, and I only tried the 1.8 version (the latest I think) and I meant to ask, even though I don't want to be annoying, what javaRE version do I need to use in order to make this work? I believe it should be one that came out at the same time that java 18, well my question is long, sorry, I thank you for your time.


[FEATURES]通用的getK8sClusterUsedPort方法支持

What feature you'd like to add:
重构端口分配。未来我们会支持更多的Runtime，因此需要将自动的端口分配能力下沉。
Why is this feature needed:

I think it have already done in #727, how about closing the issue.


fix(Checkbox): fix gap styling in RTL

Warnings

:warning:
There are no updates provided to CHANGELOG. Ensure there are no publicly visible changes introduced by this PR.

  Generated by :no_entry_sign: dangerJS


default should be keyword only in Field otherwise type checkers don't recognize it

Initial Checks

[X] I confirm that I'm using Pydantic V2

Description
when specifying the default value to Field as a positional argument, neither mypy or pyright recognizes it and incorrectly treats the field as mandatory.
i believe this is because of how typing.dataclass_transform works. since dataclasses.field takes default as a keyword only argument, pydantic.fields.Field shouldn't allow it to be specified positionally.
Example Code
from pydantic import Field
from pydantic.dataclasses import dataclass

@dataclass
class Foo:
    a: int | None = Field(None)
    b: int | None = Field(default=None)

foo = Foo() # mypy & pyright error: Argument missing for parameter "a"

Python, Pydantic & OS Version
pydantic version: 2.8.2
        pydantic-core version: 2.20.1
          pydantic-core build: profile=release pgo=true
                 install path: C:\Users\user\project\.venv\Lib\site-packages\pydantic
               python version: 3.12.0 (tags/v3.12.0:0fb18b0, Oct  2 2023, 13:03:39) [MSC v.1935 64 bit (AMD64)]
                     platform: Windows-11-10.0.22621-SP0
             related packages: typing_extensions-4.12.2
                       commit: unknown

This was discussed here: https://github.com/pydantic/pydantic/discussions/7379#discussioncomment-6951557.
Unfortunately making default a kw-only argument is a breaking change so I don't think this can be changed easily.

Thanks @Viicos for the discussion here.
Going to close this as a duplicate of https://github.com/pydantic/pydantic/issues/3617, which, as @Viicos mentioned, explains why this is too breaking of a change to implement. If you're looking for cleaner type checking, you can use the default=... syntax.


Resource limit support in yosys

Design infrastructure for FPGA resource limits in yosys, so that we can make eg. make RAM mapping decisions based on how many block RAMs are available in a given device.
Rough sketch:

[ ] design format of resource limit data, both in-memory and on-disk
[ ] include resource limit data for all supported FPGAs in yosys, design scripts/processes to keep them up to date
[ ] add a way to provide custom limit data by the user (eg. for partial reconfiguration flows targetting part of the FPGA)
[ ] tag all relevant cells in cell libraries with the type and amount of FPGA resource they consume
[ ] add an analysis that determines how many times each module is present in the design hierarchy, so that eg. we know that mapping a given memory to BRAM will actually consume 3 BRAMs because this module is instantiated 3 times
[ ] make the memory mapping process resource-aware
[ ] update memory mapping description files with resource usage information
[ ] make clkbufmap resource limit aware
[ ] make DSP mapping resource limit aware

So, I think there are two kinds of resource limit:

hard limits (if you run out of clock buffers the design almost certainly will not be placeable)
soft limits (DSPs and memories can be mapped back to soft logic).

Though it would probably complicate the implementation, I think any soft resource limit code should be structured as mapping without a limit, waiting for technology mapping to happen via ABC or whatever, then sta is ran to get timing information on the network and unmapping starts with the node with the most timing slack.

FYI - The second one (choosing between hard blocks and soft implementations) is something that ODIN-II tries to do.


Add Chinese translation to translations.ts

I found this translation need in the discord community in i18n-crew-zh, happy to contribute!

Thank you very much for the new translation @Nin3lee !
I'm currently working on a nicer Continuous Integration in this repo and I would like to test it with your PR when it's finished. This just means that it'll probably take some days until the version with integrated Chinese translation is published... I hope this is okay 4u! 🥺


Refs inside v-for does not work within draggable

I put a component in draggable like this:
const refs = ref([])
<draggable
:list="imageUrls"
item-key="url"
ghost-class="ghost"
dragClass="dragitem"
animation=500
handle=".mover"
@start="dragging = true"
@end="endDrag"

<template #item="{element, index}" style="flex-basis:none">

but it seems does not work, refs only represent one component randomly, not an array.
I expect it could be used like this
@David-Desmaisons

Same problem here. I need to use a template ref on the elements inside the draggable template, but I'm only getting one Item and not the whole list. Any updates on this?
Is there another way of getting the complete node list of elements inside the draggable template ?
This does not work:
<draggable         :list="previewArray.images"         group="images"         @start="drag = true"         @end="drag = false"         item-key="id"         class="upload_previews"       >         <template #item="{ element }">           <div class="image">             <img               class="preview imgPreview"               :src="element.image"               ref="imgRefs"             />                       </div>         </template>       </draggable>

Same problem here.

Might be helpful.
https://stackoverflow.com/questions/71414977/refs-inside-v-for-loop-for-vue-v3-2-25-or-above

Might be helpful.
https://stackoverflow.com/questions/71414977/refs-inside-v-for-loop-for-vue-v3-2-25-or-above

Not really.

Might be helpful. https://stackoverflow.com/questions/71414977/refs-inside-v-for-loop-for-vue-v3-2-25-or-above

It doesn't help.

Do you guys have any update for this issue?

I ended up doing something like this,
In <template>,
<Draggable v-model="emails" class="column" @change="update" item-key="id" ghost-class="ghost" group="emails">
      <template #item="{ element, index }">
        <EmailCard
          :key="element.id"
          class="q-my-sm"
          :ref="addEmailCardRef"
          v-model:item="emails[index]"
          :organizations="organizations"
          @remove="removeEmail"
          @changed="updateEmail"
          @invalid="hasInvalidCards = true"
        />
      </template>
</Draggable>

In <script setup lang="ts">,

/**
 * Refs to EmailCard components
 */
const emailCards = ref<Array<InstanceType<typeof EmailCard>>>([]);

function addEmailCardRef(element: InstanceType<typeof EmailCard>) {
  if (emailCards.value.indexOf(element) === -1) {
    emailCards.value.push(element);
  }
}

@yoyosan
Thank you so much but I tried to implement as yours but it seems not to work.

@yoyosan
I already solved my issue, this errors was because missing :key props in

I was still having this issue - and the one mentioned here (and in another github issue I saw) where you use a function to push the element into your own array wasn't working when for me when removing items (we have lists you can order and add/remove items from).
So with some testing I found you can just add a useless v-for on the component itself like:
<Draggable
     ...
    >
      <template #item="{ element: item }">
        <your-component
          v-for="i in [1]"
          ref="itemRefs"
         ..
        />
      </template>
</Draggable>

and in the script
const itemRefs = useTemplateRef('itemRefs')

This handled the case of adding/removing so far without any issues. It feels obviously quite dodgy, but without any information or solutions from the author I'm not sure what else can be done than dodgy hacks unless you're willing/able to move libs for draggable lists.