Update pages workflow to new versions

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

Comparison is base (82b660e) 93.17% compared to head (001b7b5) 93.21%.
Report is 3 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #460      +/-   ##
==========================================
+ Coverage   93.17%   93.21%   +0.04%
==========================================
  Files          19       19
  Lines        1803     1814      +11
==========================================
+ Hits         1680     1691      +11
  Misses        123      123

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


Add dotenv to support loading variables from .env without node-foreman

https://github.com/looker/looker-slackbot#running-locally-for-development
discusses using node-foreman for local development since it honors .env
files. It would be great if .env files were always checked, allowing for
alternate deployment mechanisms outside of using actual environmental
variables.

I ended up working around this by using systemd's EnvironmentFile option, but this might still be useful for somebody else. Up to you. :)

This makes total sense, but I should load it up and check that it doesn't bungle node-foreman's env variable thing somehow.


feat: replace deps.compile with deps.loadpaths

Current dependencies on/for this PR:

main

PR #48

PR #49

PR #50   👈

This comment was auto-generated by Graphite.

@mhanberg started a stack merge that includes this pull request via Graphite.

Graphite rebased this pull request as part of a merge.

@mhanberg merged this pull request with Graphite.


Fix default data for shops and disable Edit option except for those who must

-Fix the default data (update pictures, edit texts, add more stores)
-Disable Edit option for store profiles except for admin and vendors who creates it
-Make Edit Store page and link appropriately

Based on comparable code in Digits assignment, I have estimated 3 hours.

Git issues somehow erased all code I've written, had to redo


Crash on iOS 9

gzippedDataWithCompressionLevel(0.1) is crashing 100% of the time.
OS Version:          iOS 9.0 (13A4305g)
Report Version:      105
I'm assuming it has to do with the line:
void *libz = dlopen("libz.dylib", RTLD_NOW);
Unfortunately I can't get that specific. No interesting crash log to report. The symbolicated crash log for the line looks like this:
0   ???<PHONE_NUMBER>00000000 0 + 0

I couldn't reproduce this on beta 4, but can you see if version 1.1.1 fixes it for you?


(#6) - Include licenses of shaded dependencies in jars

Include non-Apache2 licenses in final jars.

+1 to merge


Shap not called successfully

Hi,
I met a problem when using Explainer.
I can call shap directly, but in hiclass.Explainer it doesn't work.
You can see I have imported shap, and successfully called it in the bottom cell.

try this instruction for knowing the version:
print(shap.__version__)


Simplify executor by merging task_queue and wake_queue

Proposed in https://github.com/phil-opp/blog_os/issues/768#issuecomment-629605586
This pull request also renames the waiting_tasks map to tasks and makes run_ready_tasks operate on &mut Task references instead of removing and re-inserting tasks from the map.
As another simplification, we now use the entry API on the waker_cache map instead of manual if conditions. Unfortunately this requires to destructure self into its fields because RFC 2229 is not yet implemented (see the tracking issue), but I still think that the new approach is simpler.

Post update in https://github.com/phil-opp/blog_os/pull/805.


⚠ Bofrost.at has degraded performance

In 655f3d9, Bofrost.at (https://www.bofrost.at) experienced degraded performance:

HTTP code: 200
Response time: 928 ms

Resolved: Bofrost.at performance has improved in 884f155.


Render tests failing on travis

Via #116

I messed around with the travis-testing branch a bit. Apparently renders are returning corrupted images, except sometimes™ when
...helm head render with not existing uuid defaulting to alex
is the only test that passes.
This is only happening on travis. It works on my machine and also works on dokku.
As these errors only happen with renders, I assume these errors are coming from Cairo. And because node-canvas is just a wrapper for Cairo and (AFAIK) doesn't manipulate images by itself I think it's unlikely the errors are caused by Canvas.
Here is one of these corrupted images as base64.
Usually, PNG images start with these 8 bytes 89 50 4e 47 0d 0a 1a 0a (PNG Signature)
The image linked above, however, starts with ef bf bd 50 4e 47 0d 0a for some reason. After fixing that, I was able to use the file util – apparently the header data was now readable – it reports:

PNG image data, 40 x 35, 8-bit/color RGBA, non-interlaced

that seems correct. I found rvong/png-debugger which reports:
file-size=4747 bytes

0x00000000	png-signature=0x89504E470D0A1A0A

0x00000008	chunk-length=0x0000000D	(13)
0x0000000C	chunk-type='IHDR'
0x00000010	width=0x00000028	(40)
0x00000014	height=0x00000023	(35)
0x00000018	bit-depth=8
0x00000019	color-type=6 	(truecolour with alpha)
0x0000001A	compression-method=0 	(deflate/inflate)
0x0000001B	filter-method=0 	(adaptive)
0x0000001C	interlace-method=0 	(standard)
0x0000001D	crc-code=0xFFFFFFFFEFBFBD39
>> (CRC CHECK)  crc-computed=0xE63948AE 	=>	CRC FAILED

Then I also found wilmerhenao/Portable-Network-Graphics-PNG which has similar output:
Successfully read 4747 bytes from file travis.png
File travis.png has signature:  89 50 4e 47 0d 0a 1a 0a
0000000c : travis.png :  Chunk type: 'IHDR'
00000019 : travis.png :  Result of the CRC test is ae4839e6
Warning: CRC signatures don't match
00000025 : travis.png :  Chunk type: ''
WARNING: '' is not necessarily a supported chunk type
00000029 : travis.png :  ERROR: insufficient data for chunk of declared data length<PHONE_NUMBER>

I'll assume not necessarily means definitely not :laughing:.
I will try to find out more some time, but right now I'm quite sure this is caused by Cairo.

Very similar issue with the same broken PNG signature: jembi/openhim-core-js#171.
When you google that sequence you'll find a few more reports about it.
Looks like an encoding issue, apparently the binary data is encoded to UTF-8 somewhere™ which obviously doesn't work very well.

I was able to reproduce the corrupted binary issue:
b = new Buffer([0x89, 0x50, 0x4e, 0x47, 0x0d, 0x0a, 0x1a, 0x0a]);
// <Buffer 89 50 4e 47 0d 0a 1a 0a>
new Buffer(b.toString());
// <Buffer ef bf bd 50 4e 47 0d 0a 1a 0a>

I was hoping this would be fixed in  6e41423 + b039e97 (view diff). That was another issue with encoding in our tests, which is now fixed, however.
The issue with renders / travis still exists.

So the two commits above mentioned fixed the issue with encoding / corrupted images.
Apparently this issue is related to #32.
I compared an image that failed to pass the crc check vs my local version of that image:

holy shit.

yeah it's quite weird. feel free to mess around with the travis-testing branch


Handle empty response body without throwing JSON parse error

Hi @langjt - Any chance of this getting merged?

Hi @langjt - Any chance of this getting merged?

Sorry, your PR did not solve the problem of empty response body

Hi @langjt - Any chance of this getting merged?

If the JSON parsing exception, return an empty object?


NJWE-1801 - Update params in url

Resolves NJWE-1801
What I did:

when typing new values into the search filter, it updates in the URL address bar
now when reloading the page the filters stay the same
users can more easily share the URL or bookmark where they were
[ ] Dev review
[x] Design review

Thinking on this, and should the url only update once you click "Update Results"? @ChelseaKR

@mflymfly The results update when the user changes their focus away from the filters. I'd want the URL to update when the results do for the sake of consistency - we could also require users to press the button. WDYT?

@mflymfly The results update when the user changes their focus away from the filters. I'd want the URL to update when the results do for the sake of consistency - we could also require users to press the button. WDYT?

@ChelseaKR - I think it is ok for them to update right away but then in that case I don't think we would need the update results button. Agree that the URL should update when the results do though which I think is what is happening now. I think this may have some implications on mobile. I personally would like to move the filters into a popover tray that opens from the bottom.

@ChelseaKR @scwambach - should we huddle on this later to get this resolved? I am not seeing the same behavior as you Chelsea (or maybe I am just interpreting it differently) and just want to make sure we are all on the same page.

@scwambach this seems to no longer be loading results at all now


Triton Support

Triton Support

This branch contains the changes being done for Triton support. With these changes, portkey can now call Triton host.

The implementation is based on Ollama changes in the code.

Description:
Following is the sample configuration that we need to pass in order to call the Triton server running on http://triton-host/v2/models/<model_name>
{
  "strategy": {
      "mode": "fallback"
  },
  "targets": [
      {
        "provider": "triton",
        "custom_host": "http://triton-host/v2/models/<model_name>"
        "api_key": "empty"
      },
      {
        "provider": "together-ai",
        "model": "codellama/CodeLlama-34b-Instruct-hf",
         "api_key": "together_key"
      }
  ]
}

Sample Python code:
port_key = Portkey(
      api_key="empty", config=config, base_url="http://localhost:8787/v1"
)

completion: Union[
    TextCompletion, Iterator[TextCompletionChunk]
] = port_key.completions.create(prompt=prompt, **kwargs)

output = ""
if isinstance(completion, TextCompletion):
    output = completion.choices[0].text if completion.choices[0].text else ""

Motivation:

We wanted to have the fallback mechanism in place where we should connect to our own LLM hosted in our environment, Triton in our case. In case of failover, as a second option it should contact TogetherAI. Since Triton support was not there and we saw Ollama changes being recently pushed into PortKey, we thought of leveraging custom_host feature and made these changes.
We have deployed TensorRT LLM on Triton server. With these changes we are able to achieve our requirements.

These changes do not have support for chatCompletions. We have tested our code with Text Completion use case where model repository of TensorRT-LLM  backend is hosted on Triton.
Thanks!

Thank you so much for contributing, @tapansstardog! We are reviewing the PR now.

@vrushankportkey @narengogi There was a conflict in the branch. I have resolved it and just pushed the changes.
Naren, I am not sure but will you may have to approve it again to merge it into main?

Continuing this in #549 to fastrack the merge. It just resolve conflicts on top of this PR.

The commits from this original PR are intact. Only added one more extra merge commit.


Styling Recipe Search Results

Acceptance Criteria:
When the user searches for recipes
Then they should be presented with clean looking search results.

This has been completed and pushed to production


no chart version found - semver range issue

Version: ghcr.io/fluxcd/source-controller:v0.15.4
I'm getting an error: no chart version found for XXX-09.20.x:
flux get all -A
NAMESPACE  	NAME                             	READY	MESSAGE                                                   	REVISION                                	SUSPENDED
flux-system	helmrepository/XXX-helm-repo-dev	True 	Fetched revision: 9ddf39cf1e565edfdb96edb36465ec1b440f1a9c	9ddf39cf1e565edfdb96edb36465ec1b440f1a9c	False

NAMESPACE  	NAME                           	READY	MESSAGE                                        	REVISION	SUSPENDED
flux-system	helmchart/default-XXX-dev     	False	no chart version found for XXX-09.20.x        	        	False

However it exist in our private repo:
helm search repo XXX --version "09.20.x"
NAME                       	CHART VERSION	APP VERSION	DESCRIPTION
XXX-helm-repo-dev/XXX    	09.20.22     	09.20.22   	XXX Helm Charts

It's also present in the index.yaml file:
cat data/helmrepository/flux-system/XXX-helm-repo-dev/index.yaml
apiVersion: v1
entries:
  XXX:

  - apiVersion: v2
    appVersion: 09.20.22
    created: "2021-09-20T22:13:45.626Z"
    description: XXX Helm Charts
    digest: 0a22e6db6168ff9a297026e7623eb3ab13f2f8ea442a0d9ead25898e9b385e58
    icon: https://upload.wikimedia.org/wikipedia/commons/6/64/XXX.svg
    name: XXX
    urls:
    - https://REPO_URL/artifactory/api/helm/XXX-helm-repo-dev/XXX-09.20.22.tgz
    version: 09.20.22

When I specify version explicitly it works fine, I have problem only when I use some range operator or wildcards.
Edit:
HelmRelease manifest:
apiVersion: helm.toolkit.fluxcd.io/v2beta1
kind: HelmRelease
metadata:
  name: XXX-dev
  namespace: default
spec:
  chart:
    spec:
      chart: XXX
      version: "09.20.x"
      valuesFiles:
       - values.yaml
      sourceRef:
        kind: HelmRepository
        name: XXX-helm-repo-dev
        namespace: flux-system
  interval: 10m

Generated helmchart:
k get helmchart -n flux-system -o yaml
apiVersion: v1
items:
- apiVersion: source.toolkit.fluxcd.io/v1beta1
  kind: HelmChart
  metadata:
    creationTimestamp: "2021-09-21T11:07:02Z"
    finalizers:
    - finalizers.fluxcd.io
    generation: 1
    name: default-XXX-dev
    namespace: flux-system
    resourceVersion: "20314135"
    uid: 05102c57-84d5-42c9-8c74-8e7fa1f91b7e
  spec:
    chart: XXX
    interval: 10m0s
    sourceRef:
      kind: HelmRepository
      name: XXX-helm-repo-dev
    valuesFiles:
    - values.yaml
    version: 09.20.x
  status:
    conditions:
    - lastTransitionTime: "2021-09-21T11:07:12Z"
      message: no chart version found for XXX-09.20.x
      reason: ChartPullFailed
      status: "False"
      type: Ready
    observedGeneration: 1

I found source of a problem - our version was incorrect - 09.20.22 should not have leading 0.
This  was returning an error:
Version segment starts with 0

Not sure if this is expected behaviour that there is no error, for sure it makes troubleshooting more difficult :)

@psolarcz I believe that's because 09.20.22 is not a valid semver: https://regex101.com/r/vkijKf/1/
Found your issue while also playing around with chart's dev version. #205 and #410 were a big help for me, I think something like this will work: 0.0.0-09-20-22
https://play.golang.org/p/Tiy3DQiR8La
https://github.com/Masterminds/semver#checking-version-constraints

Hi
We are encountering somewhat similar issues on HelmRepository, but this time with larger numbers.
It seems something like 3.1008.240831 is accepted but 3.1008.2409008 is not.
I tried looking into the code to see if there is any truncation, but could not find it.
Semver code seems to parse each of the 3 elements as uint64.
Any hint would be appreciated.


⚠ Processors has degraded performance

In f542599, Processors ($STATUS_URL) experienced degraded performance:

HTTP code: 200
Response time: 48 ms

Resolved: Processors performance has improved in 0f4969b.


How to cite the software?

I didn't see any biography for tensorlayer, could anyone suggest how the cite this software if use it in a paper.

I mimic the way how to cite Keras like this:
@misc{hao2016tensorlayer,
	title={TensorLayer},
	author={Dong, Hao and others},
	year={2016},
	publisher={GitHub},
	howpublished={\url{https://github.com/zsdonghao/tensorlayer}},
}

Is this OK?

it's ok ~
we are writing a white paper, so we can also cite your work if you like ~

OK, thanks, if my paper can be published soon, you're welcome to cite it.

Hi all, please cite this paper if you find it is useful in your project.
@article{TensorLayer2017,
author = {Dong, Hao and Supratak, Akara and Mai, Luo and Liu, Fangde and Oehmichen, Axel and Yu, Simiao and Guo, Yike},
journal = {ACM Multimedia},
title = {{TensorLayer: A Versatile Library for Efficient Deep Learning Development}},
url = {http://tensorlayer.org},
year = {2017}
}


App Configuration: recurring time window filter

Add "Recurrence" parameter for TimeWindowFilter using Outlook-style schema:
https://learn.microsoft.com/en-us/graph/outlook-schedule-recurring-events#using-patterns-and-ranges-to-create-recurring-events
Example:
feature-management:
  feature-flags:
    feature-v:
      enabled-for:
        -
         name: TimeWindowFilter
          parameters:
            start: "Fri, 22 Mar 2024 20:00:00 GMT",
            end: "Mon, 01 July 2019 00:00:00 GMT",
            recurrence:
              pattern:
                type: "Daily",
                interval: 1
              range:
                type: "NoEnd"

The main logic of how to check whether current time is within any recurring time window:
Find the previous occurrence of the recurring time window, let's call it prevOccurrenceStart
Check whether the current time: time is within the time window: prevOccurrenceStart ~ prevOccurrenceStart + End - Start

API change check
APIView has identified API level changes in this PR and created following API reviews.
com.azure.spring:spring-cloud-azure-feature-management


chore: upgrade Windows agents

This PR upgrades the Windows agents to build native dependencies with VS 2022 as part of internal compliance requirements.
Verification build: https://dev.azure.com/monacotools/Monaco/_build/results?buildId=302107&view=results
I'm unsure whether using a newer VS could cause issues for users on older versions of Windows.

Can you update the cache https://github.com/microsoft/vscode/blob/main/build/.cachesalt and rerun to verify module dependencies build fine.

I'm unsure whether using a newer VS could cause issues for users on older versions of Windows.

Native modules build with statically linked msvcrt and the runtime is prebuilt with statically linked libc++. If it builds fine and tests pass they will run on our current supported versions.

Verification build https://dev.azure.com/monacotools/Monaco/_build/results?buildId=302142&view=results

New verification build https://dev.azure.com/monacotools/Monaco/_build/results?buildId=303586&view=results


"Calling set on destroyed object"

Note that the InfinityModel is used inside a component.
Very similar to #145, it is however still possible to trigger  in the _loadNextPageof the route mixin.
Triggering lines:

https://github.com/hhff/ember-infinity/blob/master/addon/mixins/route.js#L328
(via https://github.com/hhff/ember-infinity/blob/master/addon/mixins/route.js#L253)
https://github.com/hhff/ember-infinity/blob/master/addon/mixins/route.js#L258
(when I bypass the above trigger)

What seems to happen is that by the time the _requestNextPage finishes (due to slow response), the user has navigated away and the corresponding Infinity component has been destroyed.
I guess I have to reopen and override _loadNextPage to work around this problem.

Hi @rhengeveld !  I'm currently working on a "modularization" of Ember Infinity that should help with this issue.
Out of curiosity - are you using closure actions?  something like action=(route-action 'infinityLoad' model.infinityModel)

Hello @hhff. Thanks for the swift response. This modularisation intrigues me :)
I use closure actions and I [used to] (https://github.com/hhff/ember-infinity/pull/174#commitcomment-18686080) with Ember Infinity, but in this instance the infinity-loader is used in conjunction with a component (sidebar menu).
{{infinity-loader infinityModel=calendarItems scrollable='#scrollable-menu-calendar' triggerOffset=20}}

I currently use this ugly (!) 😞 construction as a workaround. _onNextInfinityPage is an earlier addition to allow adding properties to the loaded items. (Much) more elegant suggestions are welcome.
    InfinityRoute.reopen({
      _loadNextPage() {
        this.set('_loadingMore', true);

        return new Ember.RSVP.Promise((resolve, reject) => {
          this._requestNextPage()
            .then((newObjects) => {
              if (!(this.isDestroying || this.isDestroyed)) {
                this._onNextInfinityPage(newObjects);
                this._nextPageLoaded(newObjects);
                return resolve(newObjects);
              }
              reject();
            })
            .finally(() => {
              if (!(this.isDestroying || this.isDestroyed)) {
                this.set('_loadingMore', false);
              }
            });
        });
      },

      _onNextInfinityPage() {}
    });

Wrapped in a Promise to prevent the then part from triggering in the menu component.
      // modelPath for the load-more, this.set for setting initial state

      this.infinityModel('calendaritem', {
        startingPage: 1,
        perPage: itemsPerPage,
        modelPath: 'calendarItems'
      }).then(infinityModel => this.set('calendarItems', infinityModel));

I think that's all of the relevant code involved.

Oh - are you using the InfinityRoute mixin directly in a component? ie
Component.extend(InfinityRoute, {
   // ...snip
})

If so - that's something we don't support.  Components should rarely own data.  That's what routes are for!  Components just take inputs, and display them, or trigger route / controller / service actions.

Sorry, initially missed the Github update e-mail.
Yes, I had (kind of) mentioned that it's used in a component :)

... and the corresponding Infinity component has been destroyed

Thanks for mentioning it's an Ember anti-pattern. I hope to get around to changing this ... eventually.
In the meantime I've worked around it, so I'll close the issue and let this serve as a historical warning for those using Ember Infinity in a component 😉


CI with elastix

ok this is certainly a slightly annoying problem!
1. installation
the commands to install on linux are below.
wget -P ~/ElastixDownload "https://github.com/SuperElastix/elastix/releases/download/5.0.1/elastix-5.0.1-linux.tar.bz2"
tar -xvf elastix-5.0.1-linux.tar.bz2

to actually use these in a shell, you have to update the path (and the exact installation directory may differ slightly on github:
export Path=$Path:~/ElastixDownload/elastix-5.0.1-linux/bin
export LD_LIBRARY_PATH=$LD_LIBRARY_PATH:~/ElastixDownload/elastix-5.0.1-linux/lib/

Note that this updates the path ONCE in the terminal it is entered, not forever.
if you want to always have this path update, you could edit your bashrc file which contains a list of things that happen every time a terminal is open
nano ~/.bashrc
# then copy this somewhere in the file:
export Path=$Path:~/ElastixDownload/elastix-5.0.1-linux/bin
export LD_LIBRARY_PATH=$LD_LIBRARY_PATH:~/ElastixDownload/elastix-5.0.1-linux/lib/

this you already pretty much figure out, albeit some of the steps were commented out (but I assume this was just debugging)
the issue as it turns out is setting the environment for a command to be executed in linux:
2. calling from inside python
When you start a new process inside python, it launches in a fresh terminal.
so if you have done the .bashrc step, I think the python commands should work (haven't checked this).
If you have not edited the bashrc file, then you can also change the environment a process launches in. so you can replace the test_ElastixInstalled test with something like the following and it should run:
def test_ElastixInstalled():

    home_dir = Path(os.path.expanduser('~'))  # may have to update for github system
    elastix_dir = home_dir / 'ElastixDownload' / 'elastix-5.0.1-linux' / 'bin'
    elastix_lib_dir = home_dir / 'ElastixDownload' / 'elastix-5.0.1-linux' / 'lib'
    my_env = os.environ.copy()
    my_env["PATH"] = my_env["PATH"] + ':' + str(elastix_dir)
    my_env["LD_LIBRARY_PATH"] = my_env["LD_LIBRARY_PATH"] + ':' + str(elastix_lib_dir)
    bashCommand = "/home/brendan/ElastixDownload/elastix-5.0.1-linux/bin/elastix -h"
    subprocess.Popen(bashCommand.split(), env=my_env)

oh if Path isn't included already:
from pathlib import Path

It passed! Thanks for the help.


Prevent dropdown from overflowing out of the viewport

 You should sign our Contributor License Agreement in order to get your pull request merged.

LGTM

Reviewed 1 of 1 files at r2.
Review status: all files reviewed at latest revision, all discussions resolved.

Comments from the review on Reviewable.io


List methods exposed from Fluent API are not fully LAZY and do call the first page during initialization.

Instead we can think about ways to have fully LAZY collection that will do first page call only when caller will start iteration over the collection.
From internal discussion:
One of the design goal of foreach specifically handle this case i.e. iterating a source where data is not immediately available. Think about doing foreach on IEnumerable/IQuerable returned by SQL SELECT query, user knows that data can possibly reach page by page from SQL server and foreach iteration may block. And in this case query executes only when user iterate through the list or apply any LINQ operator that need data (e.g. First())
Between there is async enumerable in C# https://github.com/Reactive-Extensions/Rx.NET/tree/develop/Ix.NET . We should consider this after GA.

not sure there is still something noteworthy to improve here after the work we did a while back, so parking for now..

This issue was moved to Azure/azure-libraries-for-net#60


bug?

Hello, thank you for the plugin. However, there is one thing that is weird to me. After i close the preview window with already existing before hidden buffer, the buffer is unloaded. As i understand noone reported this before, which is wierd, is it possible to fix this? Im on the latest neovim master.

Hey, thanks for the submission.
I'm a little unsure of what you mean though. Could you maybe elaborate?

@rmagatti Okay. Here is a gif of what i mean:

You see after i close goto-preview window with a definition of the TEST macro, previously loaded buffer common.h is unloaded.
Files i use to reproduce:
init.vim
call plug#begin()
     Plug 'neovim/nvim-lspconfig'
     Plug 'rmagatti/goto-preview'
call plug#end()

lua << EOF
      require'lspconfig'.clangd.setup{}
      require('goto-preview').setup{}
EOF

nnoremap gf <cmd>lua vim.lsp.buf.definition()<CR>
nnoremap gpd <cmd>lua require('goto-preview').goto_preview_definition()<CR>
nnoremap gP <cmd>lua require('goto-preview').close_all_win()<CR>

common.h
#define TEST 1

test.c
#include "common.h"

int main (int argc, char *argv[])
{
        int a = 1 + TEST;
        return 0;
}

Assuming you're closing the preview window through the goto-preview mapping. This is what the documentation about the function I use to close it says.
-- Closes the window (like |:close| with a |window-ID|).
--- @param window window #Window handle, or 0 for current window
--- @param force boolean #Behave like `:close!` The last window of a
---               buffer with unwritten changes can be closed. The
---               buffer will become hidden, even if 'hidden' is
---               not set.
function vim.api.nvim_win_close(window, force) end

And this is how I'm calling it.
vim.api.nvim_win_close(win_handle, true)
I'm adding an option to not force close in a PR as I think this is likely the cause of what you're seeing, i.e I think your buffer is becoming hidden.

@0xRZ would you mind testing out the changes by installing with the branch of the PR linked here and setting force_close = false in the setup?

@rmagatti i pulled add-force-close-option, tried to set force_close = false but it doesn't fix anything for me, <cmd>lua require('goto-preview').close_all_win()<CR> still unloads buffer after preview window is closed :man_shrugging:
Im not really sure what this force parameter of nvim_win_close do, but i found out that https://github.com/rmagatti/goto-preview/blob/main/lua/goto-preview/lib.lua#L130 bufhidden option can be changed from wipe to hide, that way existing buffer won't be unloaded, imo it seems to be the correct behaviour. Though i think (?) there can be added logic to check if buffer to open is already loaded before opening floating goto-preview window and use hide parameter only in this case and wipe in others, this way newly opened buffers for the first time won't clutter up buffer list.

Oh interesting. Good find. I'll add an option for that to too.

@0xRZ added the option to change bufhidden too. Lmk if that works for you.

@rmagatti yeah, just pulled add-force-close-option, when specifying bufhidden = "hide", buffers are not lost. thx


Y420 Filters

The style says

Avoid using filters for scanning all properties of a complex object graph. Use filters for select properties.

Can some more information be given maybe such as a sample code block to be "avoided"?  The angular phonecat documentation discusses the checkmark filter but I don't understand about the "complex object graph" and selecting all properties versus just some.  Is this regarding $stateful filters?
If someone could explain that here I'd be happy to make a PR with clarification and examples.
Thanks for this resource!

If you use the filter filter on an object that has an array of customers, which has 10 properties including an array of orders, which has 10 properties including an array of products and sales people, and so on ... basically a large object graph, the filter will evaluate everything by default. This causes an unneeded strain on the digest cycle.
It is best to use filters on smaller sets of data or to reduce which specific properties it should look at.
Or use a javascript function to filter.

Ahh I see. Thanks for the clarification.


Queue: Instance variables not available from failed() function

Laravel Version: 5.3.30
PHP Version: 7.0.11

Description:
I have a job that communicates with servers and also creates a database row in its handle() method. If something goes wrong with the server communication I want to remove the row from the failed() method.
I store the created database row as an instance variable in the job class. But when failed() gets called this variable is set to null. Any instance variables created in the construct() function is available but not any created inside handle().
Steps To Reproduce:
Here is a small job that demonstrates what I mean:
class MyJobClass extends Job implements ShouldQueue
{
    use InteractsWithQueue, SerializesModels;

    private $fromConstruct;
    private $fromHandle;

    public function __construct()
    {
        Log::info('Construct');

        $this->fromConstruct = 'From Construct';
    }

    public function handle()
    {
        Log::info('Handle');

        $this->fromHandle = 'From Handle';

        // Will print From Handle to the log
        Log::debug($this->fromHandle);
    }

    public function failed(\Exception $exception)
    {
        Log::info('Failed');

        // Will print From Construct to the log
        Log::debug($this->fromConstruct);

        // Will print empty value (null) to the log
        Log::debug($this->fromHandle);
    }
}

It sounds like you should be using database transactions to either persist all your changes, or roll them back in case of exceptions.

When the job fails, Laravel unserializes the job class and calls the failed method directly, handle() will not be executed.

It would be good if the job gets updated when it fails thus you don't need to the whole task again.
In my case, I was passing a collection with some texts I would like to analyze through Google Natural Language API. If I get 100 texts and in the 50th one, the job fails, when I retry the job it will reanalyze the first 50 texts again. The only way I found to not do the same thing again was failing the job and dispatching a new one with the updated list (with the text I haven't analyzed yet).

@codeaken did you find any solution for this?...I also got the same issue for me i also wanted to access global variable from the job in the failed function as you are doing....Can you please tell me did you find any solution for this?

If someone got solution for this. Please share :). I am also facing the same issue.


feat: flexible web driver and proxy broker in fetch node

before merging, I think we should make sure that:

the new chromium web driver is what we want to have
fetch node testing functions still work properly

I cannot run ollama until the end of the weekend, so someone else should take care of testing the fetch node functionality.

Thanks! Writing docs


Error:  Message: stale element reference: element is not attached to the page document

Hi,
As of this month im now getting the following errors why trying to confirm 3 hosts with NOIP:
Opening browser
Login successful
Confirming hosts phase
Error:  Message: stale element reference: element is not attached to the page document
  (Session info: headless chrome=102.0.5005.182)
Logging off
Opening browser
Login successful
Confirming hosts phase
Error:  Message: stale element reference: element is not attached to the page document
  (Session info: headless chrome=102.0.5005.182)
Logging off
Opening browser
Login successful
Confirming hosts phase
Error:  Message: stale element reference: element is not attached to the page document
  (Session info: headless chrome=102.0.5005.182)
Logging off

image: simaofsilva/noip-renewer:latest

Hi,
Is this still an issue? Have you tried the image with another tag?


Incorrect variable name

Section: AFD Rule set supported variables > url_path
Incorrect context: uri_path value will be /article.aspx
Corrected context: url_path value will be /article.aspx

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

ID: d8b90978-c47a-bc80-daa7-4598b296c311
Version Independent ID: 1086024a-3cef-e693-0096-4eb7a7dd3ee9
Content: Server variables - Azure Front Door
Content Source: articles/frontdoor/rule-set-server-variables.md
Service: frontdoor
GitHub Login: @duongau
Microsoft Alias: duau

#reassign:duongau

Hello @kwickramasekara, thank you for catching that typo. I made a request to get that changed and it should reflect later today. Thank you! #please-close
PR: https://github.com/MicrosoftDocs/azure-docs-pr/pull/203323


Problem details sample returns Division_by_zero when it should return 404

Problem details code returns
{"type":"https://wikipedia.org/wiki/Division_by_zero",
"title":"Wrong Input",
"status":404,
"detail":"Divison by zero is not defined."}

When there is no endpoint rather than just 404.
@sammychinedu2ky  can you fix that?

ooh ok let me do that

@Rick-Anderson sorry that I missed that in the first PR but can we maybe we could do:

Remove the custom middleware and Set the Feature when needed. https://github.com/dotnet/AspNetCore.Docs.Samples/blob/3ebbb8baf40ec3ee07814cdf41feac1cd7a916f8/fundamentals/middleware/problem-details-service/Program.cs#L47

Have the feature always there but add a new option to the [MathErrorType], eg.: NoError and check this in the CustomizeProblemDetails. (https://github.com/dotnet/AspNetCore.Docs.Samples/blob/main/fundamentals/middleware/problem-details-service/MathErrorType.cs)

In both cases i think we could use the Get instead of GetRequiredFeature and check if not null here https://github.com/dotnet/AspNetCore.Docs.Samples/blob/3ebbb8baf40ec3ee07814cdf41feac1cd7a916f8/fundamentals/middleware/problem-details-service/Program.cs#L16

Fixed in #42


Ex4 file

When i put a metaquotes 4 file with Ex4 extension the program doesn't recognise the suitable assembly automatically and want me to put it manually
But i don't know What is the suitable assembly for this kind of files
So If you could help me

Duplicate of #1173.  This is a file format Ghidra does not support.  I suggest reading that thread for more info.


docs: fix dbt model in part3.mdx

Remove raw tag from example dbt model. Not sure if the raw bit is because of https://docs.meltano.com/contribute/docs#liquid-syntax-error

Thanks @diegoquintanav!
Liquid code is a leftover from our old Jekyll-based docs.
Related to #8202.


Static IP address, Ethernet support

Hi!
Is it possible to have an option (actually function) to set Static IP address?
The main reason why I'm asking it because i have a AP portal using setApFunction(func) which has a static IP field. I can save the Wifi credentials from my custom AP using void setWifiCredentials(const char *ssid, const char *pwd), but the static IP needs to be set before connecting to any AP in STA mode. So when i reboot (even when saving/retrieving this data from NVS) i can't set any static IP because HomeSpan ignores the custom AP function and only checks for the WiFi ssid/pass  found in NVS.
  struct {
   char static_ip[16]   = "";  // "<IP_ADDRESS>"
   char static_gw[16] = "";  // "<IP_ADDRESS>"
   char static_sn[16]  = ""; //  "<IP_ADDRESS>"
   char staticdns[16]  = "<IP_ADDRESS>";  // or -> static_gw
   char staticdns2[16] = "<IP_ADDRESS>";
  } staticIP;

Having a function like  setWifiCredentials(const char *ssid, const char *pwd) for setting these values (when using custom AP) would be a great feature 😄
And this leads to the another possibilities: Ethernet support 🙃 Arduino's Ethernet library uses Wifi command 😄
There are a lot of ESP32 boards with ethernet. I've saved a link about this topic (which pins are used by Ethernet, how to modify the WiFi code etc), If I find it, I'll link it here 🤓

I'll look into adding a hook for a static IP address.
Re Ethernet, see the FAQ section.  A while back I created a version of HomeSpan that uses Ethernet instead of WiFi, but MDNS does not work properly with Ethernet on the ESP32 due to some reported issues with Espressif's implementation of the UDP stack.  I've recently revisited and it still seems there are still the same problems with Espressif's implementation.    Since MDNS is a core component of HomeKit, can't use Ethernet until Espressif resolves this long-standing issue.  Let me know if you find any workarounds.

I'll look into adding a hook for a static IP address.
MDNS does not work properly with Ethernet on the ESP32

This is why important to set up a DNS IP for the static IP address settings too. Without DNS server (the gateways's IP or anything else like Google's <IP_ADDRESS>, <IP_ADDRESS>) mDNS (and NTP) doesn't work properly. But the main API for static IP has 2 arguments for DNS setup, first one could be user selectable, the second one could be any other and constant.

@HomeSpan Is there any way to access the Configmode functions externally using code besides the button/CLI?

homeSpan.processSerialCommand(const char *CLIcommand) allows you to access CLI commands.  See the API Reference for details.

I was looking into creating a static IP Address and it seems that WiFi.config() should already do everything you need.  Just add the following to your script somewhere before your call to homeSpan.begin():
  IPAddress ip(192, 168, 1, 181);
  IPAddress gateway(192, 168, 1, 1);
  IPAddress subnet(255, 255, 0, 0);
  WiFi.config(ip,gateway,subnet);

Note that WiFi.config() takes two additional optional arguments, dns1 and dns2, if you want to set those as well.  WhenWiFi.begin() is called, the parameters set with WiFi.config() are used.

Seems to be working  as expected! I need to solve some issues with the custom AP, but thats the limitation of WifiManager, other than that saving the custom static IP to NVS and reading it, loading the settings upon startup seem to be working and the ESP get the custom IP. I need to do some test, until that I'll leave open this thread 😄

Its working as expected 😄 Thanks 😊

Hi, how did you get WifiManager to work with HomeSpan?


Error "You must feed a value for placeholder tensor 'phase_train' with dtype bool."

I get error when trying to compare two images through compare.py.
python compare.py ../20170131-234652/ ../data/images/Anthony_Hopkins_0001.jpg ../data/images/Anthony_Hopkins_0002.jpg

/home/jack/.local/lib/python2.7/site-packages/sklearn/cross_validation.py:44: DeprecationWarning: This module was deprecated in version 0.18 in favor of the model_selection module into which all the refactored classes and functions are moved. Also note that the interface of the new CV iterators are different from that of this module. This module will be removed in 0.20.
  "This module will be removed in 0.20.", DeprecationWarning)
Creating networks and loading parameters
Model directory: ../20170131-234652/
Metagraph file: model-20170131-234652.meta
Checkpoint file: model-20170131-234652.ckpt-250000
Traceback (most recent call last):
  File "compare.py", line 129, in <module>
    main(parse_arguments(sys.argv[1:]))
  File "compare.py", line 59, in main
    emb = sess.run(embeddings, feed_dict=feed_dict)
  File "/home/jack/.local/lib/python2.7/site-packages/tensorflow/python/client/session.py", line 766, in run
    run_metadata_ptr)
  File "/home/jack/.local/lib/python2.7/site-packages/tensorflow/python/client/session.py", line 964, in _run
    feed_dict_string, options, run_metadata)
  File "/home/jack/.local/lib/python2.7/site-packages/tensorflow/python/client/session.py", line 1014, in _do_run
    target_list, options, run_metadata)
  File "/home/jack/.local/lib/python2.7/site-packages/tensorflow/python/client/session.py", line 1034, in _do_call
    raise type(e)(node_def, op, message)
tensorflow.python.framework.errors_impl.InvalidArgumentError: You must feed a value for placeholder tensor 'phase_train' with dtype bool
	 [[Node: phase_train = Placeholder[dtype=DT_BOOL, shape=[], _device="/job:localhost/replica:0/task:0/cpu:0"]()]]

Caused by op u'phase_train', defined at:
  File "compare.py", line 129, in <module>
    main(parse_arguments(sys.argv[1:]))
  File "compare.py", line 50, in main
    facenet.load_model(args.model_dir, meta_file, ckpt_file)
  File "/home/jack/Desktop/facenet-master/src/facenet.py", line 368, in load_model
    saver = tf.train.import_meta_graph(os.path.join(model_dir_exp, meta_file))
  File "/home/jack/.local/lib/python2.7/site-packages/tensorflow/python/training/saver.py", line 1526, in import_meta_graph
    **kwargs)
  File "/home/jack/.local/lib/python2.7/site-packages/tensorflow/python/framework/meta_graph.py", line 502, in import_scoped_meta_graph
    producer_op_list=producer_op_list)
  File "/home/jack/.local/lib/python2.7/site-packages/tensorflow/python/framework/importer.py", line 285, in import_graph_def
    op_def=op_def)
  File "/home/jack/.local/lib/python2.7/site-packages/tensorflow/python/framework/ops.py", line 2240, in create_op
    original_op=self._default_original_op, op_def=op_def)
  File "/home/jack/.local/lib/python2.7/site-packages/tensorflow/python/framework/ops.py", line 1128, in __init__
    self._traceback = _extract_stack()

InvalidArgumentError (see above for traceback): You must feed a value for placeholder tensor 'phase_train' with dtype bool
	 [[Node: phase_train = Placeholder[dtype=DT_BOOL, shape=[], _device="/job:localhost/replica:0/task:0/cpu:0"]()]]

I experience a the same issue on validate_on_lfw.py
Any suggestions how to solve this?

Yep. That's works. Thank you!


handle missing column for aggs

handle missing column for aggs
add error when field is not fast

Codecov Report

Merging #1920 (4b97bf5) into main (7fae4d9) will decrease coverage by 0.01%.
The diff coverage is 96.49%.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
@@            Coverage Diff             @@
##             main    #1920      +/-   ##
==========================================
- Coverage   94.46%   94.45%   -0.01%
==========================================
  Files         307      307
  Lines       55927    56306     +379
==========================================
+ Hits        52831    53185     +354
- Misses       3096     3121      +25

Impacted Files
Coverage Δ

src/aggregation/mod.rs
93.60% <ø> (-0.66%)
:arrow_down:

src/aggregation/agg_req_with_accessor.rs
94.40% <80.00%> (-3.66%)
:arrow_down:

src/fastfield/readers.rs
90.08% <88.00%> (-0.27%)
:arrow_down:

src/aggregation/agg_tests.rs
100.00% <100.00%> (ø)

src/aggregation/intermediate_agg_result.rs
97.74% <100.00%> (ø)

src/collector/top_score_collector.rs
97.58% <100.00%> (ø)

src/fastfield/mod.rs
100.00% <100.00%> (ø)

src/schema/schema.rs
99.06% <100.00%> (+0.27%)
:arrow_up:

src/schema/text_options.rs
97.40% <0.00%> (-2.60%)
:arrow_down:

src/schema/flags.rs
45.00% <0.00%> (-2.37%)
:arrow_down:

... and 14 more

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


ETHUSDT

when buying ETH
Traceback (most recent call last):
File "D:\Downloads\Schneider\ScriptingPath\Crypto\Binance-News-Sentiment-Bot-main\news-analysis.py", line 447, in
buy(compiled_sentiment, headlines_analysed)
File "D:\Downloads\Schneider\ScriptingPath\Crypto\Binance-News-Sentiment-Bot-main\news-analysis.py", line 348, in buy
test_order = client.create_test_order(symbol=coin+PAIRING, side='BUY', type='MARKET', quantity=volume[coin+PAIRING])
File "D:\Program_files\Python\Python39\lib\site-packages\binance\client.py", line 1633, in create_test_order
return self._post('order/test', True, data=params)
File "D:\Program_files\Python\Python39\lib\site-packages\binance\client.py", line 295, in _post
return self._request_api('post', path, signed, version, **kwargs)
File "D:\Program_files\Python\Python39\lib\site-packages\binance\client.py", line 242, in _request_api
return self._request(method, uri, signed, **kwargs)
File "D:\Program_files\Python\Python39\lib\site-packages\binance\client.py", line 237, in _request
return self._handle_response()
File "D:\Program_files\Python\Python39\lib\site-packages\binance\client.py", line 285, in _handle_response
raise BinanceAPIException(self.response)
binance.exceptions.BinanceAPIException: APIError(code=-1013): Filter failure: LOT_SIZE

Thanks, this should now be resolved in the last commit.


Add support for tag blocks in Javadoc and Doxygen. [phab:D1026]

Created by zarko at 2016-08-29 21:33:13:
return, param and friends create delimted sections in
a piece of documentation that need to be appended to
separately. This diff adds support for these tag blocks
(and normalizes return to use them). It also adds
support in the HTML renderer for CSS styling and changes
overall markup handling to work as a pipeline.

fromberger commented inline (context lost during migration) at 2016-08-30 16:21:26:
Suggestion:
"Include this stylesheet path in the resulting HTML."
…to make it clear that it's a path, not literal CSS text.

fromberger commented inline (context lost during migration) at 2016-08-30 16:21:26:
For "style" I think you mean "class" on each of these.

fromberger commented inline (context lost during migration) at 2016-08-30 16:21:26:
Nit: I think the style guide generally prefers output parameters last.

fromberger commented inline (context lost during migration) at 2016-08-30 16:21:26:
Why *?

fromberger commented inline (context lost during migration) at 2016-08-30 16:21:26:
This function is getting kind of long. Is there any practical way to pull out some of the pieces into self-contained functions?

shahms commented inline (context lost during migration) at 2016-08-30 18:17:11:
constexpr?

shahms commented inline (context lost during migration) at 2016-08-30 18:17:11:
nit: prefer using the properly scoped types to the underscored names

shahms commented inline (context lost during migration) at 2016-08-30 18:17:11:
This line is unnecessary, as is the find:
out = &tag_blocks[block];

shahms commented inline (context lost during migration) at 2016-08-30 22:06:33:
Mostly.  Everything between auto block = ...; and this is unnecessary.

shahms commented inline (context lost during migration) at 2016-08-30 22:06:33:
I'd argue for making life a little better by writing a namespace scoped:
Printable::RejectPolicy operator | (Printable::RejectPolicy lhs, Printable::RejectPolicy rhs) {
using impl = std::underlying_typePrintable::RejectPolicy::type;
return static_castPrintable::RejectPolicy(
static_cast(lhs) | static_cast(rhs));
}

zarko wrote at 2016-08-30 23:16:49:

Address comments.

zarko commented inline (context lost during migration) at 2016-08-30 23:17:21:
Yes, that's the case. Entering a tag block changes the buffer that's being appended to.

fromberger commented inline (context lost during migration) at 2016-08-30 23:42:20:
I'm not entirely sure I believe a form with three static casts counts as more readable, though… 😈

fromberger accepted this revision at 2016-08-30 23:42:20

zarko closed via commit ada9fdee2d495480a572249c5c75c56147bc1125 at 2016-08-31 03:44:45

shahms commented inline (context lost during migration) at 2016-08-31 15:20:10:
Sure, the implementation is less readable to localize the pain, rather than pushing the static cast everywhere :-)


Share Audio / Alsa with another application (playback).

Would it be possible for another application to use ASLA playback?
Moving question from: https://github.com/roleoroleo/qr_scanner/issues/1
Sample code for testing:
https://github.com/roleoroleo/qr_scanner/issues/1#issuecomment-807998534

Maybe this:
form: https://superuser.com/a/903415
You can use the dsnoop plugin to copy the sample data to multiple virtual capture devices.
Put something like this into your ~/.asoundrc or /etc/asound.conf, and use snooped_digital as device name in both applications:
pcm.snooped_digital {
    type dsnoop
    slave {
        pcm "hw:2,1"   # the actual device
        channels 2
        rate 48000
    }
}

amixer and aplay compiled
amixer-aplay-arm.zip

dsnoop is used normally for capturing device.
Am I wrong?
We should use dmix but the kernel lacks the support for System V IPC. dmix is based on SYSV semaphore and it doesn't work with this kernel.

I'm working on it.
I create a new hack of the libalsa.
Maybe it works.

Its amazing ....

I released the new library in the last version.


The application “HelloWorld” can’t be opened.

My script works, however after making an .app using this package, it cannot be opened.
(MacOS Sonoma arm64)

using a different shebang line solved the issue.


Update types.mjs

resources/app/common/types.mjs

Consider if these are still supposed to be in the global scope or to change them to module exports as types a la client.mjs
Make individual type fixes for the v11=>v12 changes
Possibly the "correct export" for #2674

another TODO left from #2600 - remove DocumentModificationContext. Blocked by updates to Document and its immediate subclasses.


AST to type mappings

There should be a way to get the type from a AST expression. While initially this was done using unique identifiers, I have a new idea.
Instead of 'id's on AST, instead use their source position data! It is unique between AST nodes (of the same kind) and already exists, so no need to make AST structs larger. When synthesising expression AST it can use the position/span as a key for assigning a type value. A span is equivalent to a Range<u32>, so will work with anything that can convert to it. And any AST formats given to Ezno will have position information.
Retrieving these types is useful when doing optimisations after the type synthesis.
And they are also used for retrieving data in the LSP...
Finding a type mapping through looking in a range
In the LSP, a hover request sends a position in the source. A novel way of mapping positions would be HashMap<Span, TypeId>, however when trying to search data it would have to iterate through each key in the map. While it would be per source LSP it still might be a costly iteration. Instead I am thinking of a data structure which keys values not based on if they match of the range but instead in the range. For example in the following there are several ranges.
let x = 3, y = 5;

Type   x + y + 2
3      -
5          -
5              -
8      -----
10     ---------

Given a scalar index is should be able to find the closest mapping.
A quick prototype of the data structure
Here is what I came up with to solve the problem
use std::collections::BTreeMap;
use std::fmt::Debug;
use std::ops::Range;

#[derive(Debug)]
pub struct RangeMap<T> {
    entries: BTreeMap<u32, Vec<(u32, T)>>,
}

impl<T> RangeMap<T> {
    pub fn new() -> Self {
        Self {
            entries: Default::default(),
        }
    }

    pub fn push(&mut self, range: Range<u32>, item: T) {
        if let Some(existing) = self.entries.get_mut(&range.start) {
            existing.push((range.end, item));
        } else {
            self.entries.insert(range.start, vec![(range.end, item)]);
        }
    }

    pub fn get(&self, point: u32) -> Option<&T> {
        self.entries
            .range(0..(point + 1))
            .rev()
            .find_map(|(_, v)| v.iter().find_map(|(e, v)| (*e > point).then_some(v)))
    }

    pub fn get_exact(&self, range: Range<u32>) -> Option<&T> {
        self.entries
            .get(&range.start)
            .and_then(|v| v.iter().find_map(|(e, v)| (*e == range.end).then_some(v)))
    }
}

After writing this I found btree-range-map, but think it is more complicated than above
Note this works because while they can intersect, any intersections are encapsulated (no overhangs).
$$
S_1 \cap S_2 \neq \emptyset \implies S_1 \subset S_2 \vee S_2 \subset S_1
$$

Maybe related, there's also https://docs.rs/rust-lapper/latest/rust_lapper/

We need to think about this a little more because this has a huge impact on the infra. I'll set aside some time to study what's written so far.

@Boshen Hey, have you found anything from research yet.
My above code looks okay right. Should I publish it as a crate so it can be used? Can always swap it out with something else at some point.

Let's give it a try. Just export it from ezno-checker and see if it works?

There is now a passing test for this https://github.com/kaleidawave/ezno/blob/main/checker/tests/type_mappings.rs
Also this works in the JS / WASM version as of the last release 🎉


Choose Data Store

@fortunkam - I propose closing this item, as we may want to decide on a datastore per model. So, perhaps close this item and make the 'Data Store Decision' a checkbox on each model?


Revert "fix(define-remix-app): invalid input with initial underscore"

Reverts wixplosives/codux-core#1254

close as this was approved by @PeterShershov and there is no need to revert


Remove errant comma from viewer_audio.css

Just a minor typo fix.  I don't even know if there is a noticeable symptom as I haven't used audio yet.  Mainly just wanted to clear the IDE red flag. :)

Thank you, merged.


fix #99 - flag to set quantity of allocated memory.

This PR adds a flag that allows set initial memory allocation, now users with low ram machines can set a lower value to be allocated.

Also, we are still not setting the memory size on the compiled version.


GHA fails with dependencies not found: cache issue?

I created a repo, did a few PRs which spawned GHA that worked fine. Then, I renamed the repo (typo!) and now the caching seems wrong. The Setup the environment gets a cache which contains nothing: no isort, black, or anything at all. This, of course, causes the next step to fail horribly:
🚀 Checking code formatting: Running isort
make: isort: Command not found
make: *** [Makefile:13: check] Error 127

Any idea what could be going wrong there?

That is a strange issue. I just tried to replicate it by creating a private repository using the ccp command and filling in the prompts. I then created a Pull Request and the GH Actions ran successfully. I then renamed the repository, and updated the PR. The GH Actions still ran without an issue. Is there anything else other than the name of the repository that changed?
If not, it seems there simply is an issue with the cacheing mechanism., and as of yet there is no way to manually clear the cache. I would suggest raising an issue https://github.com/actions/cache?
The simplest solution approaches I can think of for now would be to:

Change the key of the cache in the Load cached venv step. Or if that fails;
Delete the repository on GitHub, create it again, add the remote with git remote add origin, and push your branch.

Hope this helps!

As an experiment, I did:

Created a new branch
Added something to poetry which caused poetry.lock to update
The cache is re-generated, as expected!
The quality now passes fine.

So, I guess there must have been some issue at my end which caused an empty (or incomplete) cache to be generated. I think we can mark this ca closed. If anyone has the same issue, they can just add a new module and the cache should update – not sure what will happen if they remove it? ¯\_(ツ)_/¯


Fix "stop tracking on browser close" hook

Please remember the Contributing Guidelines :heart:
:star2: What does this PR do?
Tweaks the code to "detect when browser is closed, and stop the running entry". Removed a counting variable and moved everything into the window callback. Tried to await all the promises that happen so we have the best chance of our code being allowed to complete before the browser is gone.
Note: I couldn't reproduce the issue but it looked pretty likely that something could go wrong with it.
Also removed some dead code that was trying to do the same thing

:bug: Recommendations for testing
All changes should be tested across Chrome and Firefox.

Enable the "stop automatically" setting
Start a timer
Close the browser
Open the browser again and see that the entry was stopped ✅

Do you know or can you spot a better way to make this behaviour more reliable?
:memo: Links to relevant issues or information
Fixes #1687. (Hopefully)

I could not get this to work (also doesn't work on master) :(
It looks like on close listener doesn't trigger for me at all, could be a :penguin: thing. @shantanuraj can you try it out?


AQCU-1112: TSS Report Renderer

Still working on adding tests, but functionally I believe this is ready for review.

Coverage decreased (-5.9%) to 93.03% when pulling 4d206dad0216085d9dda7ee63ebcb38a669ea8a3 on zmoore-usgs:feature/tss-render into efde032dea73f76905a5480ab837595513dc3caa on USGS-R:master.

Coverage decreased (-5.9%) to 93.03% when pulling 4d206dad0216085d9dda7ee63ebcb38a669ea8a3 on zmoore-usgs:feature/tss-render into efde032dea73f76905a5480ab837595513dc3caa on USGS-R:master.

Coverage decreased (-5.5%) to 93.407% when pulling feb6b8f633c08ba5b884551abf47adc3d8946893 on zmoore-usgs:feature/tss-render into efde032dea73f76905a5480ab837595513dc3caa on USGS-R:master.

Coverage decreased (-4.0%) to 94.938% when pulling af2f4f5ab82b5fa5baadaee33498e204c46ee87b on zmoore-usgs:feature/tss-render into efde032dea73f76905a5480ab837595513dc3caa on USGS-R:master.

Coverage decreased (-3.5%) to 95.409% when pulling af2f4f5ab82b5fa5baadaee33498e204c46ee87b on zmoore-usgs:feature/tss-render into efde032dea73f76905a5480ab837595513dc3caa on USGS-R:master.

Coverage decreased (-3.3%) to 95.598% when pulling 1276a5ee1978df2f0fd6f580e63ec8e3e2ee35a8 on zmoore-usgs:feature/tss-render into 3d189c775c9576b8d1528db00bbd30c7cc05b058 on USGS-R:master.

Coverage decreased (-3.3%) to 95.598% when pulling c56abfbe184551cbb895bc7c77621f538e32a2c7 on zmoore-usgs:feature/tss-render into 3d189c775c9576b8d1528db00bbd30c7cc05b058 on USGS-R:master.

Coverage decreased (-3.3%) to 95.598% when pulling 17fc4593a6d22c0460a433e0ee7240b8f9863bfa on zmoore-usgs:feature/tss-render into 3d189c775c9576b8d1528db00bbd30c7cc05b058 on USGS-R:master.

Coverage decreased (-3.3%) to 95.582% when pulling 17fc4593a6d22c0460a433e0ee7240b8f9863bfa on zmoore-usgs:feature/tss-render into 3d189c775c9576b8d1528db00bbd30c7cc05b058 on USGS-R:master.

Coverage increased (+0.07%) to 98.966% when pulling e371b0dc70ee58ada035108ead7e4946f272db8b on zmoore-usgs:feature/tss-render into 3d189c775c9576b8d1528db00bbd30c7cc05b058 on USGS-R:master.


Add ‘min/max port-channel id’ attributes to ‘apstra_datacenter_generic_system’ resource

Closed by #540


Add support for unix and webassembly ports

The unix and webassembly ports are different since they don't have boards. They do, however, have variants - but they're listed differently, as a variants folder.
One way to handle this would be to use the port names as boards:
>mpbuild build unix
>mpbuild build webassembly pyscript

Would love this, can you add a Windows build as well as that seems similar to the Unix and Webassembly builds

I'd like to support Windows builds too, but that'll be a little more challenging since it won't work the same way with container builds. Still, may be possible...

If it runs in GHA, it can run in a container,
So it can be a standard dev container with a call to tools/ci.sh
https://github.com/micropython/micropython/blob/master/tools%2Fci.sh#L722-L735

Oh yes, we can definitely cross-compile to Windows (using mingw) but can't easily support VS/msbuild on Windows itself.
Sure, will add support for that.

Mostly resolved with #30. Still need to add support for the containers for windows/webassembly builds.


Allow overriding the pattern's name

This is especially useful when multiple people are working on a pattern
an have been using different names:
$ make help |grep Pattern:
Pattern: multicloud-gitops
$ make NAME=foobar help |grep Pattern:
Pattern: foobar

lgtm


rm: added write-protected check for files

Hi,
This fixes #3813. also this doesn't check the permission of directories right now.

Looks like it broke a test:
---- test_rm::test_rm_interactive_never stdout ----
current_directory_resolved:
touch: /tmp/.tmpgyUHXS/test_rm_interactive
run: /home/runner/work/coreutils/coreutils/target/debug/coreutils chmod 0 test_rm_interactive
run: /home/runner/work/coreutils/coreutils/target/debug/coreutils rm --interactive=never test_rm_interactive
thread 'test_rm::test_rm_interactive_never' panicked at 'assertion failed: !at.file_exists(file_2)', /home/runner/work/coreutils/coreutils/tests/by-util/test_rm.rs:348:5

And please add a new test showing that the initial issue is fixed
thanks

In this line https://github.com/uutils/coreutils/blob/2aede0382a6112208712e0b0dc385d4cedc9848c/src/uu/rm/src/rm.rs#L115 we are making InteractiveMode::Never as default without any arguments but in case of #3813 we need to be interactive in default mode .
Maybe we can add a new InteractiveMode?

Sure, you can do that :)

I'm not sure about this, but it sounds like this issue is separate from the existing InteractiveMode. I.e. there is prompting for recursive removal and for removal of protected files. If they are linked I would like the naming to be a bit more descriptive than Default, maybe PromptProtected or something like that (I'm sure there's a better name).

@tertsdiepraam Ok I'll change the name to PromptProtected

Fails on windows
failures:

---- test_rm::test_prompt_write_protected_no stdout ----
current_directory_resolved:
touch: C:\Users\RUNNER~1\AppData\Local\Temp\.tmp0yZTql\test_rm_prompt_write_protected_2
run: D:\a\coreutils\coreutils\target\debug\coreutils.exe rm test_rm_prompt_write_protected_2
thread 'test_rm::test_prompt_write_protected_no' panicked at 'assertion failed: at.file_exists(file_2)', D:\a\coreutils\coreutils\tests\by-util\test_rm.rs:394:5

failures:
    test_rm::test_prompt_write_protected_no

To test on Windows:
https://developer.microsoft.com/windows/downloads/virtual-machines/

@sylvestre removed the failing test!

@stefins why did you remove it ?

It was failing on all the OS's!

sure but it is an issue with the test or with the code ?
was it failing on linux too ?

Yeah It was failing on ubuntu also

Seems like the cfg(feature = chmod) should apply to the test signature instead of just the single line. Otherwise it will fail on all platforms that don't have chmod enabled. Maybe that fixes the test?

@tertsdiepraam done

Not sure what you mean but it works here:
Warning: Congrats! The gnu test tests/rm/isatty is no longer failing!

This test https://github.com/uutils/coreutils/runs/8071421179?check_suite_focus=true

oh that, don't bother ;)
we have intermittent issues

well done ;)


Feature Request: Mixed faction list

I'm playing in a Trinity Tournament (3 ships one from each faction). It would be great if I could build a list of mixed factions.

Unfortunately the faction restrictions are baked in pretty deeply, and I am focusing on official FFG tournament legality.


Can key have other types?

Question 19

A key is a special string attribute.

I am just confused if the key has only the string type or it can has other types ( number ).

Key can accept string or number but React will convert it to string type inside. I will update the answer to avoid the confusion.


Top Menu Text Hard to Read

The tan-colored text on the top border is very difficult to read, as it is the same color as the image in places:

If you want to keep the color scheme, I'd suggest a very subtle darkening of the area behind the text with a text-shadow, and a slight lightening of the text.

#headerNav nav {
    color: #dac68b;
    text-shadow: 0px 0px 12px #000;
}

Thanks for the issue submission @sfentress. I haven't been involved in Cosecha's digital work as much as before, but I think the website has been moved to Squarespace.
I'm not sure what processes there are for open source collaboration are these days, so I'll ask some folks by tagging them here. That way, if you are interested in participating in whatever capacity, they can more appropriately let you know how. 😄
@Cosecha/cosecha-dev-village @Cosecha/organizers @ghortiz-pena


Refactor IPC target validation

Followup to #1173

ipc config option shouldn't fail if target doesn't exist (don't refuse to create an ipc policy just because the file doesn't exist. Borks startup)
Added resolve_ipc_path() to reduce boilerplate
Policy allocators now store the result of validate_ipc_target()  under policy->validated.
cmd_permit/_reject use policy->validated to make decisions.
sway_log instead of sway_assert on unvalidated target, we don't want to abort, even in debug builds.

ipc config option shouldn't fail if target doesn't exist (don't refuse to create a an ipc just because the file doesn't exist. Borks startup)

Why? In what situation does this occur? Seems to incur extra complexity to handle an unlikely scenario imo.
A lot of changes seem to stem from that. I don't like the idea of pushing validation deeper into runtime, I'd rather do it all on startup - give warnings for bad configurations straight away and simplify checks later on.

Why? In what situation does this occur? Seems to incur extra complexity to handle an unlikely scenario imo.

Right now, a policy targeting a missing file prevents sway from starting up:
nonexistent ipc target, should warn
04/20/17 20:17:55 - [permit.c:62] cmd_permit:Unable to resolve IPC permit target '/usr/bin/swaybar'. will issue empty policy
04/20/17 20:17:55 - [security.c:156] Unable to allocate security policy
./test.sh: line 20: 19485 Segmentation fault      (core dumped) sway -V

The policy allocators can currently only communicate to a caller by returning a policy or not.
On the other hand allocators don't have the context to perform policy decisions, that's the caller's job  (warn, don't give perms, ignore). So instead, the newly created policy is returned to the caller with information about how validation fared. I think we do want to integrate validation down into the allocators, so that it's impossible to "forget" to do it.

A lot of changes seem to stem from that.

not really, I just took the opportunity to also tidy up.

We probably shouldn't abort, but we shouldn't defer validation. We should just ignore the policy and print a warning.

Policy allocators now store the result of validate_ipc_target() under policy->validated.

cmd_permit/_reject use policy->validated to make decisions.

These changes are ones that make me concerned. By the time you obtain a reference to a policy, that policy should have already been validated.

We should just ignore the policy and print a warning.

What does "ignore" actually mean? Earlier you asked that permit return an empty policy. It did and does.

We probably shouldn't abort, but we shouldn't defer validation

We don't.

that policy should have already been validated.

"checked" or "enforced"?
Right now, if validation fails, the allocator tells the caller it's out of memory. That's wrong. This PR fixes it. That's it.

Anyway, right now if validation fails the allocator tells the caller it's out of memory. That's wrong. This PR fixes it. That's it.

Returning NULL doesn't signal OOM. It just means it didn't work. It would set errno = ENOMEM if we wanted to specify the specific failure, but that's not important here.

I still haven't looked over the code, but the only change I think is necessary is to avoid aborting if the target doesn't exist.

Returning NULL doesn't signal OOM. It just means it didn't work.

the only change I think is necessary is to avoid aborting if the target doesn't exist.

How will the caller tell the difference?

Why does the caller need to tell the difference?

Why does the caller need to tell the difference?

Because an OOM is non-recoverable and a bad validation potentially is.
You seem to feel that the change needed to fix this is trivial and what I've done unnecessary, so I'm going to just close this and open an issue instead. You can deal with it as you think best.


System crash when hbloader is being loaded as an exefs (2345-0031)

Horizon Version: 6.0.1
Atmosphere Version: 0.7.1
hbloader version: 2.0.0
Crash Error: 2345-0031
SDCard format: FAT32
Tested titles:
Mega Man 11 Demo Version (titleid: 010076800B06E000)
The Legend of Zelda: Breath of the Wild (titleid: 01007EF00011E000)
HBL through Album runs just fine.

This is being thrown by hbmenu right, as in nothing displayed after launching the title before the fatalerr?

I selected the user, the switch did it's title boot animation just to be abrupted with the crash.

Need to build a test app for this later - wondering if it's a 6.x issue.

Try with this at "/hbmenu.nro" & post the content of "/libnx_log" (this is not nx-hbmenu).
https://www.dropbox.com/s/2f2qvyz1hzhk99t/am_test.nro?dl=0

__nx_applet_type: 0x2
appletInitialize() rc = 0x19680

`
Here is the output

It's using AppletType_LibraryApplet, so it's not detecting that it's running as an application. :-/


APNS priority sample

What is the namespace for ApnsNotification class in the "APNS priority" code sample?

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

ID: 5fa4b158-79fb-e664-3604-0c8a2ef925d1
Version Independent ID: 33d2079e-1cd9-7933-b9fd-f57aeb0794fc
Content: Azure Notification Hubs iOS 13 updates
Content Source: articles/notification-hubs/push-notification-updates-ios-13.md
Service: notification-hubs
GitHub Login: @sethmanheim
Microsoft Alias: sethm

@bychkov Thanks for the feedback! We are currently investigating and will update you shortly.

@bychkov you're right, that's an error. It should be AppleNotification not ApnsNotification
I'll fix the docs immediately.
@SnehaAgrawal-MSFT please close.


[BUG] "Error communicating with API"

Describe the bug
After installing HA Core 2024.6.2 all entities created by the Enphase Envoy (Installer) are shown as 'Unavailable'. The integration status cycles between 'Initializing' and 'Failed setup, will retry: Error communicating with API:'. Before the update everything worked fine.
Restarted the envoy, restarted HA, restarted router, reloaded the integration nothing helps.
Also -and I think this means that this is something caused by Enphase, not the integration- if I  open in a browser (both Chrome and Edge) http://envoy.local, the page takes a very long time (several minutes of just a wait cursor) to open, then
504 Gateway Time-out
openresty/<IP_ADDRESS>

is shown. If I use the envoy's ip address, http://<IP_ADDRESS>, the exact same thing happens. Yet, my envoy successfully reports to Enlighten, the data in there does get updated plus shows a success status, and even the live status screen -which is supposed to connect to the envoy directly- works correctly.
Account type

[ X ] Installer
[ ] DIY / DHZ
[ ] Home Owner (This integration is not (fully) functional with a home owner account!)

Envoy

[ X ] Metered with CTs installed

[ ] Metered without CTs

[ ] Standard

FW version: D7.6.175

Amount of micro inverters connected: 16

Home Assistant

Version: 2024.6.2

Additional context
home-assistant_enphase_envoy_2024-06-12T12-30-25.741Z.log

Hi @rrozema, the process pushing data to Enlighten is a different one from serving the API on the Envoy. It is possible that the one broke and the other still working. Can you try to power cycle your Envoy?

I already did that, and I power cycled the router and ha too (several
times). I also removed the integration and reinstalled it. I also tried
removing the integration, removing the hacs project and tried installing
the official Enphase envoy integration. Still same result. I verified in my
router the envoy still has the same ip: it has a reserved op address in the
dhcp and it is on that ip.
So I am sure now that it is not something in the installer integration, or
at least if it is, it is in the official Enphase envoy integration too.
To me it looks like my IP is getting blocked (maybe intentionally or not?)
when trying to refresh the token required for getting local api access.
Op do 13 jun. 2024 08:05 schreef Vincent Wolsink @.***>:

Hi @rrozema https://github.com/rrozema, the process pushing data to
Enlighten is a different one from serving the API on the Envoy. It is
possible that the one broke and the other still working. Can you try to
power cycle your Envoy?
—
Reply to this email directly, view it on GitHub
https://github.com/vincentwolsink/home_assistant_enphase_envoy_installer/issues/133#issuecomment-2164518428,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AANKYDS5DFT4KXB5RGP3SETZHEZD5AVCNFSM6AAAAABJGIRHA6VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDCNRUGUYTQNBSHA
.
You are receiving this because you were mentioned.Message ID:
<vincentwolsink/home_assistant_enphase_envoy_installer/issues/133/2164518428
@github.com>

FYI: just to check I opened https://entrez.enphaseenergy.com/ from my work network and got myself a token from there for my envoy. Tonight I will try to do the same from my home network.

I have an identical problem. Also started yesterday. My gut says enphase did a booboo...

entrez.enephaseenergy.com works from home too, but gettting a fresh token doesn't fix the problem.
I just posted in the Enphase forum a question for help too:
how-to-fix-504-gateway-timeout-openresty11781-when-calling-httpenvoylocal-or-httplocal-ip

I have an identical problem. Also started yesterday. My gut says enphase did a booboo...
Edit: i just did a reboot gateway command from enlighten diagnostics dashboard and that fixed the issue.

How do I get to that enlighten diagnostics dashboard?

I have an identical problem. Also started yesterday. My gut says enphase did a booboo...
Edit: i just did a reboot gateway command from enlighten diagnostics dashboard and that fixed the issue.

How do I get to that enlighten diagnostics dashboard?

Found it.
For me "reboot gateway" did not yet fix the problem. But the situation did change...
However, when I tried opening http://envoy.local/production.json, it showed me this screen
 (401 Authorization required)
When I clicked the link in that page, the page again loaded for a long time, but this time I did get a page like this:
 (You are not authorized to view that page)

Not an issue in the integration.


Fix and rewrite demo with regl

Thanks for the module! I only just learned about matcaps yesterday! Except I was looking at the demo and realized it looked plausible but wasn't quite right. By the time I'd debugged and fixed, I found I'd also rewritten it with regl, which also took it from 172 lines to 82 lines, which seems like a plus.
The solution mostly involved some view transformations but no changes to the actual module code.
If published, it needs a switch from gh-pages to master since I removed bundle.js from .gitignore. (Otherwise couldn't convey fixed version for easy inspection.) Probably nice to get rid of a dangling branch and easy to make the switch anyway. Glad to revert though.
See updated demo. It changes this:

into this:

It's way different (better?), but I'm still not convinced it's 100% correct. There's a weird dark reflection I feel like maybe shouldn't be there:

Looks great, thanks @rreusser! Adding you to the repo/package should you want to make any further changes :D

Also, you're not in the stackgl org? :O Adding now :)


Earn Badges (0.5.0)

Use React.

There's are some TODOs and FIXMEs in here that I should do.

There's are some TODOs and FIXMEs in here that I should do.


Feature/login

This is my version for  users management system: register, login, and current logged-in user.
This version doesn't use fast-api_users library.
There are two different versions for login management. This version is one of them. It is not clear yet which one of them will be used in production. This PR is just for CR for now
Included are few functions/routs not for production, just for functionality representation, for your CR. they meant to give an example regarding how to protect routes, and how to get logged-in user details in other routes. Those are marked as "# Not for production".
those functions will be removed, and tests functions will be added after receiving confirmation that this is the system that will be used.

Codecov Report

Merging #195 (329683d) into develop (b8e0992) will decrease coverage by 4.16%.
The diff coverage is 72.98%.

@@             Coverage Diff             @@
##           develop     #195      +/-   ##
===========================================
- Coverage    99.31%   95.14%   -4.17%
===========================================
  Files           34       41       +7
  Lines         1323     1567     +244
===========================================
+ Hits          1314     1491     +177
- Misses           9       76      +67

Impacted Files
Coverage Δ

app/internal/security/ouath2.py
38.09% <38.09%> (ø)

app/internal/security/dependancies.py
42.85% <42.85%> (ø)

app/routers/login.py
51.28% <51.28%> (ø)

app/routers/register.py
97.56% <97.56%> (ø)

app/database/schemas.py
100.00% <100.00%> (ø)

app/internal/security/schema.py
100.00% <100.00%> (ø)

app/internal/user.py
100.00% <100.00%> (ø)

app/main.py
97.43% <100.00%> (+0.88%)
:arrow_up:

... and 5 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 b8e0992...b9ee9d3. Read the comment docs.

If i clicked on 'resoved' on an issue, without addressing it, it was just by mistake. In few of the cases, i just thought that the CR is for another line of code, and i thought i did resolve it.

I tried to answer the best i can, hopefully it clarifies everything that was not understood.
I also tried to fix what i could, and i hope that will be good enough.


Q: fabio memory usage is large, Why?

maxconn=5000
runtime.gogc=500
Soon it went up to 700m+, about one day
Is there any way to reduce it?

The only thing we ever did to tune memory usage was to reduce runtime.gogc to have more collections. Smaller number (e.g. 100) usually equal more collections, higher numbers less collections. The default value is tuned for throughput of short-living connections.
If you terminate TLS or have lots of long-running connections w/o timeout you may see increased memory usage. Also, there was a memory leak in the TCP proxy which was fixed in #321 but I don't know how big the impact was. This has been fixed in 1.5.2.
My suggestion is to run fabio with GODEBUG= gctrace=1 ./fabio ... and look at the output of the GC in the logs.  Then slowly reduce the GOGC until this stabilizes.

 6191 root       20   0 15.4G  9.9G  5352 S  3.2 15.8     0     0 26h34:23 /usr/local/bin/fabio -cfg /etc/fabio.conf
10GB of ram! but i'm using 1.5.0, i will upgrade to 1.5.2 to see how much it will help

@danielmotaleite the typical solution to this problem is to tune the runtime.gogc parameter to your use case. See the previous comment on how to address this. There was a memory leak in the TCP proxy which was addressed in #321 and is in 1.5.2. If you're using the TCP proxy then this might affect you.

After the upgrade and more that 24h later:
fabio    26033  4.8  0.1  96852 68932 ?        Ssl  Aug07  67:45 /usr/local/bin/fabio -cfg /etc/fabio.conf
much better... so yes, the memory leak in 1.5.x <1.5.2 was big, but it's now fixed!  :)

We set gogc to 100 and memory remains relatively under control. That said, maybe the default of 800 is a bit too royal?

The 800 value worked well for go1.4 and go1.5 for non-TLS connections and lots of short-lived requests. Might be time to run another test suite.


std.simd work in progress, for consideration

I'll leave this here for visibility and commentary.

Noice! ETA for getting this to reviewable state?

I really don't know, there are hurdles...
I don't know what SIMD level was supplied to any compiler on the command line (ie, '-msse2' needs to be reflected in a version or something), DMD doesn't even have such a command line option. The 'target' attribute has problems in GCC.  unittests are being a problem; CTFE doesn't work for loads of maths stuff (for unittesting).
There's a bunch of things, I'll attempt to recall the complete set and perhaps bug them individually or something?

Yes, "bug" them individually in Bugzilla sounds like a good idea.

ping
Any updates on this?  It's been a year...

I saw that @lakh has done his own implementation - maybe he might be able to tell us his reasons.
Btw DAutoTest is failing ...

It's still blocked on the compilers. I still have no way to detect what simd version was supplied to the compiler on the command line on GCC/Clang, and DMD has no such concept. The library can't emit opcodes that violate the simd level request made to the compiler; I need to know the level requested and then I can produce the best code for that level using static if.
@greenify His implementation comes at it from a totally different angle. His focuses on just wrapping up the arch-specific intrinsics rather than making a usable portable library, it's lower level than mine. It has its place, but I obviously don't agree that's the right approach or I would have written mine that way. Anyone who wants to hit the intrinsics directly will do just that. I tried very hard to strike a practical balance between low-level-ness, efficiency, high-level-usability, and portability.
I'm more-or-less happy with my API, but it doesn't work unless the compilers can provide the information I need at compile-time.

@TurkeyMan Are there bugs for these problems?

newsgroup thread:
http://www.digitalmars.com/d/archives/digitalmars/D/Any_usable_SIMD_implementation_282806.html
bugzilla:
https://issues.dlang.org/show_bug.cgi?id=15873

DMD has no such concept

See the D_SIMD predefined version:
https://dlang.org/spec/version.html

ping @TurkeyMan - did the SIMD discussion solve your problems?

Not really.
Honestly, I've been trying to identify the key thing that keeps me from pushing this task to my foreground, and possibly the biggest thing that's holding me back here is writing the literally zillions of unit tests required to get this across the line, and spending god-knows-how-many hours scrutinising code generation across the suite of architectures I care about. I just haven't had the time :/ .. The other problems I have discussed are indeed awkward and produce a decent friction to getting back on track, but I could find the motivation to push through them if not for the much larger load of grunt work waiting for me on the other side.
The color module is more critical to my current work, and I think it's significantly smaller in scope too... if I can't even get that in good shape to everyone's satisfaction, this feels even more daunting.
Anyway, I am keen to see this done, but it's a particularly hard module to get right. I think it might be best to approach broken into small sets of related functions?
Or if someone wants to help me with the unittesting the thing, I can then take that time to scrutinise codegen properly for all the target arch's.

Anyway, I am keen to see this done, but it's a particularly hard module to get right. I think it might be best to approach broken into small sets of related functions?
Or if someone wants to help me with the unittesting the thing, I can then take that time to scrutinise codegen properly for all the target arch's.

Yep I would recommend to put it on dub, it's more visible to other people, easier to test and submit PRs ;-)

It's been there for like, a long time: https://code.dlang.org/packages/simd

Why not make it work on one architecture just to start? Because it's experimental your proposal doesn't have to be feature complete.

I'm going to close this as it'll have to go through the Phobos review process regardless, and it's on dub so there's not a huge rush to get it into Phobos.


use KONG- prefix for Kong specific headers where applicable.

X- prefixes have been deprecated since RFC 6648 along with a set of reccomendations for creating new parameters:
3.  Recommendations for Creators of New Parameters

   Creators of new parameters to be used in the context of application
   protocols:

   1.  SHOULD assume that all parameters they create might become
       standardized, public, commonly deployed, or usable across
       multiple implementations.

   2.  SHOULD employ meaningful parameter names that they have reason to
       believe are currently unused.

   3.  SHOULD NOT prefix their parameter names with "X-" or similar
       constructs.

since Kong specific headers prefixed with X- depict new and common patterns of API Management usage, we can start with Kong- prefix, or even API- prefix to give it more meaning.
as we make our way into standardizing these headers as common practice, we can suggest them as an RFC to IETF in the future.
Proxy specific headers should remain intact, or apply existing standards where applicable (such as in #270)

I am fine with replacing X- with Kong-, however that doesn't feel very "white-label" to providers using Kong on premise. Should we go for API- maybe? That doesn't feel very right either...

that's what I was referencing in using a more generic name (so we can later even introduce as standard to the rest of the API community)
options that come to mind:

Proxy
API-Proxy
Admin
Service

Well I think the least worst is Service...

By default Kong- and for white-label allow custom name prefix for header.

Just want to point out that this change would be a breaking change and our users will need to update their implementations - not sure it's worth it.

@thefosk we can:

make it configurable
introduce a fallback for the interim version in which this is released (so, we'd be sending both X- and Kong-)
remove the fallback in the next version after the one that introduces this change.

use option 1, also works best for white-label on premises. Then break it by setting the new default to Kong- and document how to configure it the way it was before if needed.

I have been thinking about making this configurable in the datastore, so all nodes can be aware of it (instead of the configuration file), and this could be configurable on a per-API basis. Maybe as an attribute of the "Services" entity, which would default to "Kong-".

I would actually disagree on making this configurable per API, seems like a core functionality of the system and especially concerning SDKs and integrations.
e.g. an SDK library dealing with requests from Kong now has to be aware of each API/route prefix for headers...
therefore I think it should remain a global setting.

This is a non-sequitur. If the SDK is developed by the same company, then said company can just have the same prefix for all of its APIs. If the SDK is developed by a third party, then the same problem arises since we are making it configurable anyways.
One company might have 2 services and want them to be considered separated from a user perspective, hence not the same prefix.
Finally, an SDK could simply make this configurable if ever needed.

I have been thinking about making this configurable in the datastore, so all nodes can be aware of it (instead of the configuration file)

shouldn't the entire config be stored in the database? (only db access info and credentials locally provided)

e.g. an SDK library dealing with requests from Kong now has to be aware of each API/route prefix for headers... therefore I think it should remain a global setting.

 Even if you don't provide this flexibility, the fool will find another way to mess up his config.

Yes, and yes.

On Mar 17, 2016, at 6:32 PM, Thijs Schreijer<EMAIL_ADDRESS>wrote:
I have been thinking about making this configurable in the datastore, so all nodes can be aware of it (instead of the configuration file)
shouldn't the entire config be stored in the database? (only db access info and credentials locally provided)
e.g. an SDK library dealing with requests from Kong now has to be aware of each API/route prefix for headers... therefore I think it should remain a global setting.
 Even if you don't provide this flexibility, the fool will find another way to mess up his config.
—
You are receiving this because you commented.
Reply to this email directly or view it on GitHub


Configurable tabsize in HTML reports

Is your feature request related to a problem? Please describe.
In #6, it was reported that \t get handled in HTML reports.
When checking html.py this mentioned code seams to be gone.
When using tab characters for indentation in Python files, coverage.py converts all tabs to 8 spaces (checked in Chrome browser).
When checking the HTML source code, then 8 spaces can be found:

HTML Code
HTML View

Describe the solution you'd like
Offer an option in the configuration file, so the conversion of tab characters to spaces can be specified.
[html]
tabstyle = spaces
tabsize = 2

Or this
[html]
tabstyle = tabs
tabsize = 2

This would allow CSS to specify the representation with a default of 2.
Describe alternatives you've considered

The documentation contains no options to configure the tab to space translation process.

I think modifying CSS is not possible, because the HTML code contains already 8 spaces.
So the representation of tab characters can't be modified.
#source { padding: 1em 0 1em 3.5rem; font-family: SFMono-Regular, Menlo, Monaco, Consolas, monospace; }
#source p { position: relative; white-space: pre; }
#source p * { box-sizing: border-box; }
#source p .t { display: inline-block; width: 100%; box-sizing: border-box; margin-left: -.5em; padding-left: 0.3em; border-left: 0.2em solid #fff; }

Additional context

CSS settings for tab sizes: https://developer.mozilla.org/de/docs/Web/CSS/tab-size
tab-size: 2;

Added research about CSS for tabs.

I'd like to handle this by reading .editorconfig files, but that will require some refactoring, and using an optional third-party library.

The .editorconfig sounds also good.
But shouldn't there be a fallback to the pyproject.toml or similar?

I would rather not invent a coverage.py-only setting for this.  editorconfig files are good, and more people should use them.

I also would prefer the .editorconfig.
What do you mean by "coverage.py-only"?
Coverage.py is already reading from pyproject.toml.
[tool.pytest.ini_options]
# Don't set 'python_classes = *' otherwise, pytest doesn't search for classes
# derived from unittest.Testcase
python_files = "*"
python_functions = "test_*"

[tool.coverage.run]
branch = true
omit = [
	"*site-packages*",
	"setup.py"
]

[tool.coverage.report]
skip_covered = true
skip_empty = true
exclude_lines = [
	"raise NotImplementedError"
]

[tool.coverage.html]
directory = "report/coverage/html"

[tool.coverage.xml]
output = "report/coverage/coverage.xml"

What do you mean by "coverage.py-only"?

I meant that a setting like [tool.coverage.html] tab_size=2 would only be read by coverage.py. It would be duplicating information that's already in the editorconfig file.

@nedbat any progress on this topic?

I have not put any work into this.  BTW: one of the reasons people prefer spaces to tabs is because it avoids having to add a tab-size feature to everything that displays code.


itest: mine less blocks to speed up tests

This PR changes the multi-hop-related tests to mine fewer blocks and shuffle test cases so blocks are more even among different tranches.

@yyforyongyu, remember to re-request review from reviewers when ready


WL-4643: Long lists of Favourite sites do not wrap in MSIE

This is a patch from Sakai.

Could we have a Sakai reference in this?

Yup sorry the Sakai Jira is: https://jira.sakaiproject.org/browse/SAK-31731 and the PR is : https://github.com/sakaiproject/sakai/pull/3552

There maybe a better fix for this that Sakai will use to holding off on it.

There is a better(?) fix in Sakai now which we should go with instead of this one.


Reinstalling on installation failure

Installations flagged as failed by verification.sh will be reinstalled

@BhaviniMadhuranath Done

You have 3 hours time
Bounty points are 40

This issue is fixed


About receiver

Hello, your project is very impressive
I want to ask something about the receiver's firmware
How did you write the firmware for the wireless keyboard with a receiver
I would be very grateful if you could answer

Not sure what you're asking.. the code for the firmware is in this repository. The QMK part is available on the official QMK repository.


[cherries] Picking a couple of SQL parsing cherries

CATEGORY
Choose one

[x] Bug Fix
[ ] Enhancement (new features, refinement)
[ ] Refactor
[ ] Add tests
[ ] Build / Development Environment
[ ] Documentation

SUMMARY
Cherry picking a couple of bug fixes:

https://github.com/apache/incubator-superset/pull/7374
https://github.com/apache/incubator-superset/pull/7490

TEST PLAN
CI.
ADDITIONAL INFORMATION

[ ] Has associated issue:
[ ] Changes UI
[ ] Requires DB Migration.
[ ] Confirm DB Migration upgrade and downgrade tested.
[ ] Introduces new feature or API
[ ] Removes existing feature or API

REVIEWERS
to: @etr2460 @graceguo-supercat @michellethomas

Codecov Report

Merging #140 into airbnb-master will increase coverage by 0.01%.
The diff coverage is 93.54%.

@@                Coverage Diff                @@
##           airbnb-master     #140      +/-   ##
=================================================
+ Coverage          64.97%   64.98%   +0.01%
=================================================
  Files                427      427
  Lines              20671    20680       +9
  Branches            2312     2312
=================================================
+ Hits               13430    13439       +9
  Misses              7117     7117
  Partials             124      124

Impacted Files
Coverage Δ

superset/sql_parse.py
99.2% <100%> (+0.06%)
:arrow_up:

superset/security.py
75.86% <60%> (-0.11%)
:arrow_down:

Continue to review full report at Codecov.

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


The setup wizard could not be loaded: Unknown error.

Describe the bug
As soon as I add the integration in HA the error occurs after a certain time.... The setup wizard could not be loaded: Unknown error.
Screenshots
If applicable, add screenshots to help explain your problem.
Additional Information (please complete the following information)

Model Name [e.g. dreame.vacuum.p2028]: Dreame L20 Ultra
Firmware Version [e.g. 1156]: latest
Home Assistant Version:  (Virtual on Proxmox) Home Assistant 2023.10.3 Supervisor 2023.10.0 Operating System 11.0 Frontend 20231005.0 - latest with 8GB RAM
Configuration Type [With or without map support]:
Errors or warnings shown in the HA logs (if applicable): no errors


Shrev setup example doesn't compile

Using this code:
extern crate specs;
extern crate shrev;

use specs::{System, Read, Resources};
use shrev::{EventChannel, ReaderId};

struct Event {
    s: String,
}

struct MySystem {
    reader: Option<ReaderId<Event>>,
}

impl<'a> System<'a> for MySystem {
    type SystemData = (Read<'a, EventChannel<Event>>);
    fn run(&mut self, events: Self::SystemData) {
        for event in events.read(&mut self.reader.as_mut().unwrap()) {
            println!("{:?}", event.s);
        }
    }

    fn setup(&mut self, res: &mut Resources) {
        use specs::SystemData;
        Self::SystemData::setup(res);
        self.reader = Some(res.fetch::<EventChannel<Event>>().register_reader());
    }
}

And this as my dependencies in Cargo.toml:
[dependencies]
specs = "0.12"
shrev = "1.0"

I get the following error from cargo check:
error[E0596]: cannot borrow immutable borrowed content as mutable
  --> src/lib.rs:26:28
   |
26 |         self.reader = Some(res.fetch::<EventChannel<Event>>().register_reader());
   |                            ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ cannot borrow as mutable

error: aborting due to previous error

For more information about this error, try `rustc --explain E0596`.

I attempted to rewrite the two lines of setup as
        let mut x: specs::shred::Fetch<'_, EventChannel<Event>> = res.fetch::<EventChannel<Event>>();
        self.reader = Some(x.register_reader());

But then I get this error:
error[E0596]: cannot borrow immutable borrowed content as mutable
  --> src/lib.rs:27:28
   |
27 |         self.reader = Some(x.register_reader());
   |                            ^ cannot borrow as mutable

warning: variable does not need to be mutable
  --> src/lib.rs:26:13
   |
26 |         let mut x: specs::shred::Fetch<'_, EventChannel<Event>> = res.fetch::<EventChannel<Event>>();
   |             ----^
   |             |
   |             help: remove this `mut`
   |
   = note: #[warn(unused_mut)] on by default

error: aborting due to previous error

For more information about this error, try `rustc --explain E0596`.

So the book has an invalid example at this point. (Section 7: Setup).

The issue seems to be fixed if I replace the last two lines of setup with:
        let mut channel: Write<'_, EventChannel<Event>> = SystemData::fetch(res);
        self.reader = Some(channel.register_reader());

Does this make sense given that res (Self::SystemData)'s type is Read<'a, EventChannel<Event>>? Why is it valid to use Write here?

Yes, the register_reader call needs a mutable event channel.

Why is it valid [..]?

You can fetch whatever you want in setup, it's completely independent from your SystemData type.
You can also just use fetch_mut instead of fetch in your original example.

The fixed code sample in #474 should work, please tell me if not.


HF Padding-Free Support

Switching to upstream padding-free implementation in HF Transformers:
https://github.com/huggingface/transformers/blob/main/src/transformers/data/data_collator.py#L1617

#257 addresses this