Tracking issue for RFC 62: The MemoryData class.

RFC PR: amaranth-lang/rfcs#62
Implementation:

MemoryData class: #1256
MemoryData._Row class: #1271
Documentation: TODO

We need some documentation; at least, enough to know that to manipulate memory rows in simulation you need to do e.g. memory.data[0].eq(1). Currently memory.data is completely undocumented.


isEmail with display name doesn't support emoji

<EMAIL_ADDRESS>{ allow_display_name: true });

returns false, but it is a valid email format.

Fixed by #1004.


[Error report] java.lang.AssertionError: Wrong line separators: '...Properties\r\n#Mon Mar...' at offset 46

Plugin version: 1.0.2
IDE version: IntelliJ IDEA 2023.3.6
Stacktrace:
java.lang.AssertionError: Wrong line separators: '...Properties\r\n

Thank you for reporting issue! Already fixed, new version will be published today.
Duplicate of https://github.com/vaadin/intellij-plugin/issues/19


Add function to check if OCP version is accepted

/verified

Report bugs in Issues
The following are automatically added:

Add reviewers from OWNER file (in the root of the repository) under reviewers section.
Set PR size label.
New issue is created for the PR. (Closed when PR is merged/closed)
Run pre-commit if .pre-commit-config.yaml exists in the repo.

Available user actions:

To mark PR as WIP comment /wip to the PR, To remove it from the PR comment /wip cancel to the PR.
To block merging of PR comment /hold, To un-block merging of PR comment /hold cancel.
To mark PR as verified comment /verified to the PR, to un-verify comment /verified cancel to the PR.
verified label removed on each new commit push.
To cherry pick a merged PR comment /cherry-pick <target branch to cherry-pick to> in the PR.

Multiple target branches can be cherry-picked, separated by spaces. (/cherry-pick branch1 branch2)
Cherry-pick will be started when PR is merged

To build and push container image command /build-and-push-container in the PR (tag will be the PR number).
To add a label by comment use /<label name>, to remove, use /<label name> cancel

Supported /retest check runs

/retest tox: Retest tox
/retest build-container: Retest build-container
/retest python-module-install: Retest python-module-install

Supported labels

hold
verified
wip
lgtm

/verified

New container for quay.io/redhat_msi/openshift-cli-installer:latest published


Setting isBrowser to false manually

Hi,
I'm having a problem when using another package that redefines the window variable, even when running in a NodeJS envirnoment.
I'd like to ask if it is possible to add a feature where the user can set the variable isBrowser to false manually.
Thank you.
PS: I'm using this package as dependency to the firebase-admin package.

@dudizimber Thanks for filing this.
Can you point at the code you are referring to? We use isBrowser in the Firestore Web SDK, but not (at least not directly) in this SDK, which is bundled in firebase-admin. I suspect some layer underneath us might use similar functionality, but it would help if you could provide some pointers.

Hey @schmidt-sebastian, thanks for your reply.
The file I'm talking about is @google-cloud\firestore\build\src\v1\firestore_client.js.

Now I dove a little deeper, and it looks like the version of this package I'm using is 3.8.6, and the isBrowser check was removed in the 4.0.0 update I guess.
For what I can conclude, the problem is that the package firebase-admin has "@google-cloud/firestore": "^3.0.0" as dependency. So I'm guessing that to solve this issue, they'd need to update firebase-admin to work with a newer version of @google-cloud/firestore.
I'm attaching the error I get when I try to create/update a document in Firestore:
TypeError [ERR_INVALID_ARG_TYPE]: The "path" argument must be of type string. Received type object     at validateString (internal/validators.js:125:11)     at Object.basename (path.js:1289:5)     at GrpcClient.loadProto (/workspace/node_modules/firebase-admin/node_modules/google-gax/build/src/grpc.js:117:29)     at new FirestoreClient (/workspace/node_modules/firebase-admin/node_modules/@google-cloud/firestore/build/src/v1/firestore_client.js:118:38)     at ClientPool.Firestore._clientPool.pool_1.ClientPool [as clientFactory] (/workspace/node_modules/firebase-admin/node_modules/@google-cloud/firestore/build/src/index.js:329:26)     at ClientPool.acquire (/workspace/node_modules/firebase-admin/node_modules/@google-cloud/firestore/build/src/pool.js:87:35)     at ClientPool.run (/workspace/node_modules/firebase-admin/node_modules/@google-cloud/firestore/build/src/pool.js:164:29)     at Firestore.request (/workspace/node_modules/firebase-admin/node_modules/@google-cloud/firestore/build/src/index.js:983:33)     at WriteBatch.commit_ (/workspace/node_modules/firebase-admin/node_modules/@google-cloud/firestore/build/src/write-batch.js:496:48)     at process._tickCallback (internal/process/next_tick.js:68:7) Caused by: Error     at WriteBatch.commit (/workspace/node_modules/firebase-admin/node_modules/@google-cloud/firestore/build/src/write-batch.js:415:23)     at DocumentReference.set (/workspace/node_modules/firebase-admin/node_modules/@google-cloud/firestore/build/src/reference.js:341:14)     at Function.saveOrganization (/workspace/lib/classes/Database.js:28:57)     at createOrganization (/workspace/lib/controllers/organization.controller.js:56:35)     at process._tickCallback (internal/process/next_tick.js:68:7)"
Thank you

@dudizimber Thanks for the pointer. The code you refer is to actually auto-generated, and as such I am not super familiar with its internals. That's why isBrowser didn't immediately ring a bell.
We have updated firebase-admin last week. If you update to version 9, you will be able to use the 4.x version of @google-cloud/firestore. Assuming that this solves your issue, I will close this. Please do comment if there is anything else we can help with.


updated video content

Change Logs
updated video content
Impact
Describe any public API or user-facing feature change or any performance impact.
Risk level (write none, low medium or high below)
none
Documentation Update
none

The config description must be updated if new configs are added or the default value of the configs are changed
Any new feature or user-facing change requires updating the Hudi website. Please create a Jira ticket, attach the
ticket number here and follow the instruction to make
changes to the website.

Contributor's checklist

[ ] Read through contributor's guide
[ ] Change Logs and Impact were stated clearly
[ ] Adequate tests were added if applicable
[ ] CI passed

@bhasudha video blogs are ready


Should redraw trigger here?

After fetching the list of channels and saving them in channels var, the DOM is not refreshed (only after pressing the close button, the list redraws) so for it to work I'm calling redraw.
Do I need to call redraw in situations like this one? or should it works without redraw()?
import karax / [kbase, vdom, kdom, vstyles, karax, karaxdsl, jjson], jsffi, asyncjs, jsutils, strutils

var channels: seq[tuple[name: kstring, url: kstring]] = @[]
var hide = false

proc parseM3u(text: string): seq[tuple[name: kstring, url: kstring]] =
 ...

proc fetchChannels(): void {.async.} =
  log("fetching channels".kstring)
  var res = await fetchText("https://admin.galax.be/TVkanalen/external.m3u")
  channels = parseM3u($res)
  log("got ".kstring & $channels.len() & " channels".kstring)
  redraw()

proc createDom(): VNode =
  result = buildHtml(tdiv):
    tdiv(class="channel-container"):
      tdiv(class="stop"):
        proc onclick() =
          stop()
        text "Stop"
      for channel in channels:
        tdiv(class="channel", id=channel.url):
          proc onclick() =
            changeSource(channel.url)
          text channel.name

fetchChannels()
setRenderer createDom

Works as indented, Karax redraws on event callbacks and inside kajax.nim if requested. That keeps things easy to understand.


unicode: update to version 15.0

This issue is intended as an umbrella tracking issue for Unicode 15 support.
Unicode 15 is now published.
Go currently supports Unicode 13 (#48621 was the issue for Unicode 14)
Additionally (#52485) CLDR support is also lagging.

cc @robpike

Perhaps also cc @mpvl as he's still listed as a secondary owner in https://dev.golang.org/owners.

Yes, I have long forgotten the update sequence.

There's a generator script in the x/text repo that has a doCore flag to update the core unicode package.
There's also a script inside x/text/unicode that only updates the rangetables.
It seems the top-level script calls all of the subpackage scripts.
Maybe after the rangetables are updated, one just copies them into the standard library?

Is there a schedule for #52485  to use the newer CLDR for collation and formats?

It looks like the tables in x/net/idna and various parts of x/text have not been updated for Unicode 15.0, but were updated at the last Go unicode update (to 13.0). Does that need to happen before the 1.21 release?
(Reopening as a release-blocker until that question is resolved.)

For reference, these have pending CLs: https://go.dev/cl/501515 (x/net/idna) https://go.dev/cl/501936 (x/text).


Config frontmatter in schema template isn't copied when creating a new note

Please select the area your bug applies to. (Multiple selections are Possible. You can leave blank if you're not sure.)

[ ] Workspace.  VSCode workspace, vaults, Intellisense/autocomplete, Dendron settings
[ ] Lookup.  Dendron's Lookup Command
[ ] Views.  Dendron Preview, Tree View, Side Panels in the UI
[ ] Schema.  Dendron Schemas
[ ] Pod.  Data import from / export to Dendron
[ ] Publish.  External Site Publish
[ ] Markdown.  Markdown features, such as syntax support and features

Describe the bug
When creating a new note that has a schema template associated, frontmatter stored in the config key is not copied.
To Reproduce

Create a schema template with the following frontmatter: config: { global: { enablePrettyRefs: false}}.
Create a note that will have the schema template automatically applied.

Expected behavior
I expect that the config: { global: { enablePrettyRefs: false}} frontmatter appears in the new note.
Screenshots
If applicable, add screenshots to help explain your problem.
Desktop (please complete the following information)

OS: Windows
Browser (if with published websites; include version): n/a
Versions (all that apply to bug):

Dendron Extension: 0.86.1
VS Code version: 1.65.2
Dendron CLI: 0.74.0

Additional context
Add any other context about the problem here.
Dendron Log file
Please attach the output of > Dev:Dendron: Open Logs here

thanks for the report. will take this as an action item


Compiling message-io with just websocket feature breaks.

Error message:
error[E0433]: failed to resolve: could not find `tcp` in `super`
   --> /home/redact/.cargo/registry/src/github.com-1ecc6299db9ec823/message-io-0.14.1/src/adapters/ws.rs:183:45
    |
183 |                     let tcp_status = super::tcp::check_stream_ready(&stream.0);
    |                                             ^^^ could not find `tcp` in `super`

This might be intended, but in that case there should be websocket = ["tcp"]. It also doesn't make sense for the web either. Maybe have two different websocket features? websocket and websocket-web.

Ouch! This is due to a recent refactorization. I will fix it ASAP!
Thanks!!

As you suggested, this https://github.com/lemunozm/message-io/pull/99 fixes the compilation issue.
Regarding your comment, why you say that this change doesn't make sense for the web? Maybe i'm not understanding something

Websocket seemed to be an implementation of websocket not bindings to web_sys's websocket?

Sorry, I didn't understand you well. Yes, the message-io implementation is based in tungstenite-rs. which seems to not support wasm.
The idea will be to detect the current platform and swap the WebSocket adapter implementation from tungstenite to some other crate as web-sys or ws_stream_wasm. At this moment you can not use WebSocket with wasm. Sorry for the inconvenience. I will create a new issue to tackle this feature.

I close this issue (since the compilation problem is solved) and open https://github.com/lemunozm/message-io/issues/100 instead to tracking the wasm support


operator k8s-triliovault (2.5.3)

New operator bundle
Name: k8s-triliovault
Version: 2.5.3
Certification project: 3cbc2d2703e44a4189a90299906995c1
Test result URL: https://catalog.redhat.com/api/containers/v1/projects/certification/test-results/id/617ed81c2bfbc00e94ed9570
Test logs URL: https://catalog.redhat.com/api/containers/v1/projects/certification/artifacts/id/617ed81939ebd58949bc173b

Hi
This GitHub job is failing because of the wrong ospid passed into the job and therefore it's failing,
This is an issue for the old bundle image that project ID from URL is different from OSPID ID.


How to visualize detection result?

Hello everyone,
Thanks very much for your effort!.
Now I am using the v0.1 branch, since there's no demo.py, I cannot directly visualize the detection result.
So is there anyone kindly provide such tools to visualize the result? with the input cfg and ckpt,  pcd file,.
Best regards.

Maybe you can have a look at my repo and pointcloud_viewer.py in particular.
There are code snippets on how to visualize the stored pickle files from disk after training.


OSM Funds

OSM is distributing funds. See https://wiki.osmfoundation.org/wiki/Engineering_Working_Group/Tendering_Framework
Can Planetiler apply there?

Clearly defines the scope of work to be completed.

The scope could be to implement custom schema definitions as discussed in https://github.com/onthegomap/planetiler/discussions/81

But I don't know if the tendering framework is the best approach. I will close this for now, we can always reopen...


How the menu implements

Using the original Blazor template, authorization authentication can be implemented as follows
<AuthorizeView Roles="admin">

         <NavLink class="nav-link" href="mobilesurvey">
              test
         </NavLink>

 </AuthorizeView>

But the menu of AntBlaozr Pro uses the MenuData of AntDesign.ProLayout.BasicLayout to set the menu, and I don't know how to achieve authorization authentication

Temporarily inject AuthenticationState to solve this problem, I hope Ant Blazor Pro can have an official solution


UI improvements

Hey, can I improve UI of your website
There is a list of my improvements:

add dark theme and CSS variables for it;
Put all css files into css directory and the same thing with html without index.html;
add .gitignore file;
add some improvements with buttons, typography and so on;
add design system and documentation, if you would like;
improve accesability at your site

May I do it?

Hey, now I have some list of improvements, I will plan do it, if you like:

put all jsfrom script tag to the separate file;
add linter for example editorcofig for better code style;
add separate files for adaptive styles, and connect it to the html in better way. I know how to load styles more efectively.

Hi, @OneLiL05 Thanks a lot for the previous Pull request you can go ahead with all these improvements if you are still interested, and once again sorry for the late reply.

@CoderSaty, I have some questions about your project, you use icon fonts, but it isn't good for site rendering, and accesability, can I add my icons, that are already setted up for prefomance, I will add docs how to use them in the next PR (pull request), if you like my proposal. Also I have good idea about code style at this project, if you like, I add test that would check all PR's and say follow they code style or not, I also can add docs about it.


Kontena has ceased operations. Any plans to open source Lens?

Kontena has ceased operations, as stated here: https://blog.kontena.io/farewell/.
Any plans to open source Lens?

I'm really missing Lens, please open source it. It was (is) the best tool for K8s.

https://github.com/kubenav/kubenav is the closest thing that I've found so far.

If we're trading tips on alternatives, kube-commander gives a Midnight Commander / text terminal experience with "kubectl" as the only dependency.  Kontena Lens is still my first choice, but when I have to squeeze into a bastion host's text terminal to reach the cluster, kube-commander saves me much typing.

Kontena went out of business.   There are efforts to open source the Lens app... but it sounds like legally they can't yet.  (I'm not affiliated with Kontena, just a bystander)
You can download the app here:
https://github.com/kontena/lens/releases
The login doesn't work, you can hack around the broken login via:

Library/Application\ Support/Lens/config.json (MacOS X)
./.config/Lens/config.json (Linux)

Set the following config.json:
{
	"lastSeenAppVersion": "2.7.0",
	"__internal__": {
		"migrations": {
			"version": "2.7.0"
		}
	},
	"preferences": {
		"colorTheme": "dark",
		"allowTelemetry": false
	},
	"token": "AAAAA",
	"user": {
		"username": "cooluser",
		"first-name": "",
		"last-name": "",
		"email"<EMAIL_ADDRESS>		"company": "google",
		"url": null,
		"location": "",
		"bio": "",
		"notifications": [
			"newsletter"
		],
		"is-partner": true,
		"user-avatar-src": "https://www.gravatar.com/avatar/9ed1e9439badc6b6034815e8219e3712",
		"verified": true,
		"is-customer": true,
		"lens-invitation-used": true
	},
	"eulaAccepted": true,

In case anyone needs it, the config.json for Windows is here -
%appdata%/Lens/config.json

Lens is by FAR the best UI based kube management app I've come across.
Open sourcing it, or hell, even make just Lens a product would be really cool/smart/awesome (I'd pay $5-10/mo just for Lens, basically forever)

@jicowan haven't seen kubenav before. Looks very similar to Octant, which I think is my #2 after Lens.
The killer features in Lens are the easy multi-cluster management, quick access to pod-level logs, and the integrated shell experience for executing sh against a running pod really quickly (wonderful for troubleshooting).
Octant has none of that, but does have one really nice thing, which is instantly being able to create port-forwarding tunnels into pods in the cluster (again very nice for troubleshooting).
RIP Lens.

When I was looking for such app to manage kubernetes, I wasn't able to find Lens.
So I found a developer to start making such app. We finished our first alpha-beta-gamma version, so others also could try it, but don't expect much from it. It's only able to show resources, and delete them. But we going to develop as much as we can. And I actually need it in my own daily work.
So just check it guys - https://kuberneto.com and let me know if you have comments or suggestions. Sorry app file is not signed and notarized yet, I'm going do that in next few days at least for MacOS.
P.S. I'm still using Lens, it's a nice app and we going to reuse some UI tricks from it.

@ryedin what about https://www.infra.app ? what are your thoughts on it? (I think Lens is better)

Is there a better way to join the Slack to see that post than the link in the Readme? Since kontena.io is down, there isnt an easy way to join.

@dacahill7 try this link

@matti haven't seen that one yet. I'll have to check it out. looks Lens-inspired. Does it have the shell integration? I find myself using that feature in Lens way more than I thought I was going to at first.

@jakolehm thank you!
wonderful news

@matti I've been following the 2 founders behind infra.app for many years. I've been their user since their first tool, Kitematic. I also helped them beta test when they started docker for mac. They've always managed to keep their app easy and simple.

infra.app appears to be mac only though.

infra.app appears to be mac only though.

https://twitter.com/InfraApp/status/1235248477560545281

Hey! We do have plans for both Linux and Windows versions.

c&p from Kontena Community slack channel:

Lens 3.0 will be released today / this weekend + it'll be 100% open source under MIT license.

Source code is now available: https://github.com/lensapp/lens


Wrong response type: string should be int

Hi google team.
@zaibon just opened an issue in my repo (https://github.com/edoardottt/depsdev/issues/51) pointing out that the fields lineCount  and lineCoverCount are strings now, but they should be integers following the docs (https://docs.deps.dev/api/v3alpha/#getproject).
Try this:
curl "https://api.deps.dev/v3alpha/projects/github.com%2Fklauspost%2Fcompress" | grep -i linecount

...
"ossFuzz":{"lineCount":"20711","lineCoverCount":"12156",
...

What are your thoughts on this? Will you change the docs or the APIs?
thanks

Thanks for reporting this. It is indeed a bug in the generated documentation for our the API.
A bit of technical detail: Our API can be accessed via gRPC or as JSON-over-HTTP, with the latter being implemented in terms of the former. The canonical service definition can be found in api.proto, which uses int64 for these fields, which is encoded as string in JSON to avoid precision loss when used from JavaScript, whose number type only supports integers up to 2^53-1.
As a short-term fix, we have updated our documentation generator and regenerated the documentation to correctly describe these fields as string. In a future version of the API, we will change their type to int32/number, but for now, we want to avoid breaking existing clients.

Thank you so much for the answer @sarnesjo.
So I have to assume ALL number types in that docs mean string? (e.g. edges[].fromNode: number in GetDependencies)


How to launch it ?

Hi Noé,
I have assumed to launch your project with docker-compose up, but I get this error:
[+] Running 0/0
 ⠿ Network nouuu_projeqtor_network  Error                                                                                0.0s
failed to create network nouuu_projeqtor_network: Error response from daemon: This node is not a swarm manager. Use "docker swarm init" or "docker swarm join" to connect this node to swarm and try again.

Then if I do docker swarm init, it looks better:
Swarm initialized: current node (5a4ndw0at8mtjf8xbpr43uxdl) is now a manager.

To add a worker to this swarm, run the following command:

    docker swarm join --token SWMTKN-1-26fueduirshi6f69jll1dhh3u87wtobk4lw715pntvmrbn9f53-arxok7aju1epk91603ktcwhpe <IP_ADDRESS>:2377

To add a manager to this swarm, run 'docker swarm join-token manager' and follow the instructions.

$ docker-compose up
[+] Running 1/0
 ⠿ Network nouuu_projeqtor_network  Created                                                                                0.0s
external volume "" not found
$ docker ps
CONTAINER ID   IMAGE     COMMAND   CREATED   STATUS    PORTS     NAMES
$ docker-compose ps
NAME                COMMAND             SERVICE             STATUS              PORTS

Nothing seems to run.
I am not a Docker expert. What is the way forward please?

Hello
I want to learn hacking
Termux

Hi @lalebarde !
I should have specified that the given stack is for Docker Swarm, and not Docker compose.
Your problem is because the network is of type "overlay", which is a swarm network (in case of docker compose only, it should be of type "bridge")
Your second issue on  external volume "" not found is more curious, look like volume name is empty i don't know why.
I am currently updating projeqtor image to last stable version 9.4.2, I will update my readme in the same time, watch for futher updates ! 😉


Bump App Check version to 8.5.0-beta

Should fix https://github.com/firebase/firebase-ios-sdk/runs/3083903668?check_suite_focus=true
Test run: https://github.com/firebase/firebase-ios-sdk/actions/runs/1037955080

Isn't the prerleease script supposed to manage the versions? @granluo

Isn't the prerleease script supposed to manage the versions? @granluo

It doesn't look like we bump the version for prerelease testing (see here) which, I think, we should do.

I'm going to merge this as a workaround to get the prerelease nightly working. We should still get the prerelease scripting fixed to manage the version correctly.

Isn't the prerleease script supposed to manage the versions? @granluo

Yes, the prerelease will manage the versions but for beta version will be excluded. Should we include beta version?

Yes, the prerelease will manage the versions but for beta version will be excluded. Should we include beta version?

@granluo  Yes, we need update update all tags including beta for prerelease testing.


Vulnerability roundup 92: ark-19.12.3: 1 advisory [3.3]

search, files

[ ] CVE-2020-16116 CVSSv3=3.3 (nixos-20.03)

Scanned versions: nixos-20.03: 925ae0dee63.
Cc @ttuegel

Both need a bump to 20.08.1 to fix these. Should I be bumping a major rev in a deprecated stable branch?

No, not for a 3.3 score.


How about small genome size using Hi-C data?

Hello Developers,
Thanks for your excellent pipeline.
I used this pipeline for my plant genomes, which worked well. However, when I used it for my fungal genome, the genome size is about 45 Mb. I got the error as following:
`[2024-11-22  17:03:45] inv_adjust_v2.py -> adjust_inversion line:67 [INFO] : Needs to be moved ctg: {'>scaffold00013:::fragment_4': {'length': '86611', 'start': 14469554, 'end': 14556164}}
[2024-11-22  17:03:45] get_cfg.py -> get_ratio line:49 [INFO] : Ratio(assembly length / hic length) is 1.0
[2024-11-22  17:03:45] adjust_all_error.py -> adjust_all_error line:110 [INFO] : Start moving translocation ctg
load checkpoint from local path: /isilon/projects/J-003316_Canola-activity-10A/BLgenome/3-dna/AutoHiC/error_model.pth
img size:  (1116, 1116)
Traceback (most recent call last):
File "/isilon/projects/J-003316_Canola-activity-10A/BLgenome/3-dna/AutoHiC/autohic.py", line 383, in
typer.run(whole)
File "/isilon/projects/J-003316_Canola-activity-10A/BLgenome/3-dna/AutoHiC/autohic.py", line 225, in whole
black_num_list = adjust_all_error(hic_file_path, asy_file_path, divided_error, mdy_asy_file,
File "/isilon/projects/J-003316_Canola-activity-10A/BLgenome/3-dna/AutoHiC/src/assembly/adjust_all_error.py", line 111, in adjust_all_error
asy_operate.moves_ctg(modified_asy_file, error_tran_info, modified_asy_file)
File "/isilon/projects/J-003316_Canola-activity-10A/BLgenome/3-dna/AutoHiC/src/assembly/asy_operate.py", line 383, in moves_ctg
get_ctg_info = self.get_ctg_info(ctg_name=ctg, new_asy_file=assembly_file_path)
File "/isilon/projects/J-003316_Canola-activity-10A/BLgenome/3-dna/AutoHiC/src/assembly/asy_operate.py", line 131, in get_ctg_info
return ctg_by_name[ctg_name]
KeyError: '>scaffold00009:::fragment_8'
`
Could you help me check this error and give some suggestions to modify the parameters?
Thanks,
Fuyou

can you upload your whole log, .hic and .asy file for check ?

Hello Jwindler,
Thanks for your replying.
You can download all files from the linkage:
https://drive.google.com/drive/folders/0B92oSx9RZqS_fkZDMVRCc19CSkxuRjFyVThoOExMREh4eVBTTTQ1c0h6c2Nhc1RtcGVsSzQ?resourcekey=0-i2eYpLxpKp5_dEnHlB2dVQ&usp=sharing
My genome size is about 45Mb.
Best,
Fuyou

i see your hic file. i think your fungal scaffold is so bad, so AutoHiC
adjust error maybe wrong. i prefer you to use HapHiC or other scaffolder to scaffolding, then run onehic.py to check.

Thanks,
I will try more.


Add support for multi-step pipelines

Improved logging interface
Improved metrics specification
Add support for pipeline definition
Add classification metric

Looks great - let's merge!


Unable to configure Visibility widget

Widget
Visibility
Version of widget
1.0
Bug or Enhancement
Cannot confgure the Visibility widget with a working gp service.
This shows my results in ArcMap using the steps outlined in the doc.

The picture shows two results.
 - Results from running the tool locally
 - Results from the gp service

I copied the following URL in the widget configureation step:
https://fqdn/ags/rest/services/Test/ViewshedTestWidget/GPServer
Repo Steps or Enhancement details

@topowright
I can't hit your service. Also, the URL for the GP service should be:
https://fqdn/ags/rest/services/Test/ViewshedTestWidget/GPServer/Viewshed
Also, please configure your GP service to be synchronous instead of asynchronous.

@kgonzago @topowright
Kevin I think there was an issue with line 155 of the widget.js:
"Elevation_URL": this.config.elevationService.url,
I have removed this line as its not required and tries to reference a object that does not exist.
Just to check also make sure your service is shared with everybody as the widget does not do any token authorization.
Please verify its now working

@adgiles Good catch and verified.

I have verified that this works start from finish.


fix(webpack): allow transpilation of .cjs files

Types of changes

[x] Bug fix (a non-breaking change which fixes an issue)
[ ] New feature (a non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

Description

When trying to transpile a file from node_modules through the build.transpile property, files ending with .cjs were previously ignored.
This PR fixes it.
Checklist:

[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly. (PR: #)
[ ] I have added tests to cover my changes (if not applicable, please state why)
[ ] All new and existing tests are passing.

@danielroe thanks for the approval !
For anyone interested in this feature before it gets merged, you can extend your webpack config like this :
// nuxt.config.ts
export default {
// ...
  build: {
    extend (config: any, { isDev, isLegacy }: any) {
      // https://github.com/nuxt/nuxt.js/pull/10240
      const babelConfig = {
        configFile: false,
        babelrc: false,
        cacheDirectory: true,
        plugins: [
          '@nuxtjs/composition-api/dist/babel-plugin'
        ],
        presets: [
<EMAIL_ADDRESS>{
            corejs: { version: 3 },
            jsx: { compositionAPI: true }
          }]
        ],
        envName: 'client'
      }

      if (!isDev && isLegacy) {
        config.module.rules.push({
          test: /\.c?jsx?$/i,
          exclude: (file: string) => {
            file = file.split(/node_modules(.*)/)[1]

            // not exclude files outside node_modules
            if (!file) {
              return false
            }

            const cjsTranspile = [/defu/]

            // item in transpile can be string or regex object
            return !cjsTranspile.some(module => module.test(file))
          },
          loader: 'babel-loader',
          options: babelConfig
        })
      }
   }
// ...
}

You can edit cjsTranspile to update the list of modules to transpile.

Thank you so much for making this PR @gnuletik ❤  I'm closing it in favor of #10340 which has updated tests.

@gnuletik thanks for your config  code
in my case, i had to change the test regex to  test: /\.cjsx?$/i,


Allow clippy::type_complexity in more places.

Objective

See fewer warnings when running cargo clippy locally.

Solution

allow clippy::type_complexity in more places, which also signals to users they should do the same.

When running clippy in CI we use pre configured lints because we don't want to pollute the repo with a bunch of annotations like that.
Personally, I added this config to my rust analyzer to use the same lints as bevy
   "rust-analyzer.check.command": "clippy",
   "rust-analyzer.check.extraArgs": [
      "--all-features",
      "--",
      "-Aclippy::type_complexity",
      "-Wclippy::doc_markdown",
      "-Wclippy::redundant_else",
      "-Wclippy::match_same_arms",
      "-Wclippy::semicolon_if_nothing_returned",
      "-Wclippy::explicit_iter_loop",
      "-Wclippy::map_flatten",
      "-Dwarnings"
  ],

I thought about that, but there were already about 37 of these in the code (every crate basically except the new asset crate).

I'll update this. To avoid a conflict, we'll need #9794 to land before this.

I removed -A clippy::type_complexity from CI as discussed above and elsewhere. If examples change in such a way that they start to trigger, then they would need to be updated since this PR doesn't edit every single example, only those that currently trigger the lint.

Sure thing. I am almost done with dinner and then will do it

@alice-i-cecile Done! Hopefully everything is correct with the latest push.


Add docs for network filters - DevTools

The network filters should be documented for people. They are undiscoverable in the UX and can be super-useful.
I think we should have some already... just having trouble locating and I don't recall it being a full list.

@Garbee are you adding this to your updates to the network docs? I can do if not, just don't want duplicate efforts

Yup, I'm going to cover this as I go through that page. Getting back to work on this stuff in about an hour, picking up on the Network Throttle page PR before moving onto the network page.

Right now with this, I have created a new "Understanding Resource Timing" doc page that pulls some of the existing Network Panel content out and expands upon it. Once the PR for that is merged in and the panel's content is cleaned up a bit, I'm going to begin looking for anything else that can be pulled out into new pages for debugging in specific contexts. Whatever can't be, will then get updated inline along with adding any new functionality.

done :heavy_check_mark:
https://developers.google.com/web/tools/chrome-devtools/profile/network-performance/resource-loading?hl=en#filter-requests
ALL THE FILTERS


chore: mark nightly build as prerelease by default

right now nightly build are marked as normal release. if someone were to install the app manually from github, they should now it's not a vetted version and could expect some newly introduced bugs

Looks like gh release upload does not accept a --prerelease flag. However, gh release create does.

mm I missed that. but there are 2 places where we upload the files. one on iOS and one on android pipeline. so we need a bigger refactor to make this possible because I would imagine we can't call create multiple times

Looks like gh release upload does not accept a --prerelease flag. However, gh release create does.

adding the one I missed here: 9873e403f7530ced5dc8665ab1c56198e3484f1b

Closing this but @krtk6160 and I can make sure this is taken into account in the new process.


Node Sass does not yet support your current environment (macOS 10.12.1, Node 7.0.0) [CONTEXT: YARN] {FOLLOW UP TO #1764}

NODE: v11.8.0 installed with yarn (not npm)
node-sass: 4.11.0
working in a Express + CRA app, with a server/ and client/ directory (served with concurrently), my problem is only isolated to development on Mac oS 10.13 High Sierra.
It happened after a DOWNGRADE of X Code from 10.0-> 9.4.1
first, I was unable yarn start, consistently getting this
Error: Node Sass does not yet support your current environment: OS X 64-bit with Unsupported runtime (72)

I found closed issue https://github.com/sass/node-sass/issues/1764 which seems to have consensus that the fix is npm rebuild node-sass is the fix. However, this 1) did not succeed at all for me (it crashed), and 2) I am using yarn not npm

NPM version (npm -v):
6.5.0

Node version (node -v):
v11.8.0

Node Process (node -p process.versions):

{ node: '11.8.0',
  v8: '7.0.276.38-node.16',
  uv: '1.25.0',
  zlib: '1.2.11',
  brotli: '1.0.7',
  ares: '1.15.0',
  modules: '67',
  nghttp2: '1.34.0',
  napi: '4',
  llhttp: '1.0.1',
  http_parser: '2.8.0',
  openssl: '1.1.0j',
  cldr: '34.0',
  icu: '63.1',
  tz: '2018e',
  unicode: '11.0' }

Node Platform (node -p process.platform):
darwin

Node architecture (node -p process.arch):
x64

I ran into this problem after what I Think was caused by an X Code 10.1 -> 9.4 DOWNGRADE (I think that's what caused it). First I had some nasty problems with node-sass, then I blew away my node_modules and was then unable to yarn install again without serious problems.
then I got through that but got to
Error: Node Sass does not yet support your current environment: OS X 64-bit with Unsupported runtime (72)

I was able to fix this by switching my system over from (what I think was) brew-installed node, which annoyingly is difficult to downgrade, to nvm (https://github.com/nvm-sh/nvm)
then, once I made sure I had the right node version, I was able to
yarn cache clean
and also yarn install
then I ran into a nasty rabbit hole with the dev environment only, which was simply fixed by restarting my system after which my app started booting in dev again.

Node 12 is only supported by node-sass 4.12+ https://github.com/sass/node-sass#supported-nodejs-versions-vary-by-release-please-consult-the-releases-page-below-is-a-quick-guide-for-minimium-support


[ISSUE #2293] fix knative pmd test error

Fixes #2293 .
Motivation
fix knative pmd test error
Modifications
refactor knative module code
Documentation

Does this pull request introduce a new feature? ( no)
If yes, how is the feature documented? (not documented)

LGTM and this is a very clean PR!
The contents are the same as the previously closed one, so I have no more questions.
@xwm1992 Please help merge this PR if there are no more questions from the community when you have time. Thanks in advance!


[request-pull] list all project issues for GithubIssueBridge when issue is not provided

The following changes since commit 7961f8081ec61e6c49df3fabd230e18038d686de:
[HtmlFormat] Revert use of filter_input to $_SERVER (2016-08-23 19:46:41 +0200)
are available in the git repository at:
https://framagit.org/peetah/rss-bridge GithubIssuesBridge
for you to fetch changes up to c9822bffa75bcedd4cc7eeeff4419a5c836d62d4:
[GithubIssueBriddge] list all issues when 'i' parameter is not provided (2016-08-23 20:29:29 +0200)

Pierre Mazière (1):
[GithubIssueBriddge] list all issues when 'i' parameter is not provided
bridges/GithubIssueBridge.php | 53 +++++++++++++++++++++++++++++++++--------------------
1 file changed, 33 insertions(+), 20 deletions(-)

Merged.
Related commits:

c051730f7ce13f7890172224eb2ab32d7c54f184 - [GithubIssueBridge] fix comments number - by @pmaziere
c9822bffa75bcedd4cc7eeeff4419a5c836d62d4 - [GithubIssueBriddge] list all issues when 'i' parameter is not provided - by @pmaziere


Drop old django support

This removes declared support for versions below 1.11 and removes related testing and functionality. This simplifies the definition of the EnumField class as well as a number of tests.

Thanks!

I'm seeing some test failures on master now unfortunately (do pull; I've pushed some other changes..)
$  PYTHONPATH=../../forks/django ./runtests.py
Creating test database for alias 'default'...
....E..EEE.......E......E..Traceback (most recent call last):
  File "/tmp/tmps96atovz/tests/manage.py", line 10, in <module>
    from django.core.management import execute_from_command_line
  File "/home/lamby/git/forks/django/django/__init__.py", line 1, in <module>
    from django.utils.version import get_version
  File "/home/lamby/git/forks/django/django/utils/version.py", line 71, in <module>
    @functools.lru_cache()
AttributeError: 'module' object has no attribute 'lru_cache'
FTraceback (most recent call last):
  File "/tmp/tmpko999ok3/tests/manage.py", line 10, in <module>
    from django.core.management import execute_from_command_line
  File "/home/lamby/git/forks/django/django/__init__.py", line 1, in <module>
    from django.utils.version import get_version
  File "/home/lamby/git/forks/django/django/utils/version.py", line 71, in <module>
    @functools.lru_cache()
AttributeError: 'module' object has no attribute 'lru_cache'
FTraceback (most recent call last):
  File "/tmp/tmplc7c664w/tests/manage.py", line 10, in <module>
    from django.core.management import execute_from_command_line
  File "/home/lamby/git/forks/django/django/__init__.py", line 1, in <module>
    from django.utils.version import get_version
  File "/home/lamby/git/forks/django/django/utils/version.py", line 71, in <module>
    @functools.lru_cache()
AttributeError: 'module' object has no attribute 'lru_cache'
F.......................
======================================================================
ERROR: test_field_lookup (tests.tests.FieldTests)
----------------------------------------------------------------------
Traceback (most recent call last):
  File "/home/lamby/git/projects/django-enumfield/tests/tests.py", line 414, in test_field_lookup
    self.assertEqual(list(query), [m2])
  File "/home/lamby/git/forks/django/django/db/models/query.py", line 276, in __iter__
    self._fetch_all()
  File "/home/lamby/git/forks/django/django/db/models/query.py", line 1261, in _fetch_all
    self._result_cache = list(self._iterable_class(self))
  File "/home/lamby/git/forks/django/django/db/models/query.py", line 74, in __iter__
    for row in compiler.results_iter(results):
  File "/home/lamby/git/forks/django/django/db/models/sql/compiler.py", line 1034, in apply_converters
    value = converter(value, expression, connection)
TypeError: from_db_value() missing 1 required positional argument: 'context'

======================================================================
ERROR: test_field_lookup_in_slugs (tests.tests.FieldTests)
----------------------------------------------------------------------
Traceback (most recent call last):
  File "/home/lamby/git/projects/django-enumfield/tests/tests.py", line 422, in test_field_lookup_in_slugs
    self.assertEqual(list(query), [m2])
  File "/home/lamby/git/forks/django/django/db/models/query.py", line 276, in __iter__
    self._fetch_all()
  File "/home/lamby/git/forks/django/django/db/models/query.py", line 1261, in _fetch_all
    self._result_cache = list(self._iterable_class(self))
  File "/home/lamby/git/forks/django/django/db/models/query.py", line 74, in __iter__
    for row in compiler.results_iter(results):
  File "/home/lamby/git/forks/django/django/db/models/sql/compiler.py", line 1034, in apply_converters
    value = converter(value, expression, connection)
TypeError: from_db_value() missing 1 required positional argument: 'context'

======================================================================
ERROR: test_field_lookup_in_values (tests.tests.FieldTests)
----------------------------------------------------------------------
Traceback (most recent call last):
  File "/home/lamby/git/projects/django-enumfield/tests/tests.py", line 430, in test_field_lookup_in_values
    self.assertEqual(list(query), [m2])
  File "/home/lamby/git/forks/django/django/db/models/query.py", line 276, in __iter__
    self._fetch_all()
  File "/home/lamby/git/forks/django/django/db/models/query.py", line 1261, in _fetch_all
    self._result_cache = list(self._iterable_class(self))
  File "/home/lamby/git/forks/django/django/db/models/query.py", line 74, in __iter__
    for row in compiler.results_iter(results):
  File "/home/lamby/git/forks/django/django/db/models/sql/compiler.py", line 1034, in apply_converters
    value = converter(value, expression, connection)
TypeError: from_db_value() missing 1 required positional argument: 'context'

======================================================================
ERROR: test_isnull (tests.tests.FieldTests)
----------------------------------------------------------------------
Traceback (most recent call last):
  File "/home/lamby/git/projects/django-enumfield/tests/tests.py", line 446, in test_isnull
    self.assertEqual(list(query), [m1])
  File "/home/lamby/git/forks/django/django/db/models/query.py", line 276, in __iter__
    self._fetch_all()
  File "/home/lamby/git/forks/django/django/db/models/query.py", line 1261, in _fetch_all
    self._result_cache = list(self._iterable_class(self))
  File "/home/lamby/git/forks/django/django/db/models/query.py", line 74, in __iter__
    for row in compiler.results_iter(results):
  File "/home/lamby/git/forks/django/django/db/models/sql/compiler.py", line 1034, in apply_converters
    value = converter(value, expression, connection)
TypeError: from_db_value() missing 1 required positional argument: 'context'

======================================================================
ERROR: test_query (tests.tests.FieldTests)
----------------------------------------------------------------------
Traceback (most recent call last):
  File "/home/lamby/git/projects/django-enumfield/tests/tests.py", line 386, in test_query
    test_field_no_default=TestModelEnum.A,
  File "/home/lamby/git/forks/django/django/db/models/query.py", line 276, in __iter__
    self._fetch_all()
  File "/home/lamby/git/forks/django/django/db/models/query.py", line 1261, in _fetch_all
    self._result_cache = list(self._iterable_class(self))
  File "/home/lamby/git/forks/django/django/db/models/query.py", line 74, in __iter__
    for row in compiler.results_iter(results):
  File "/home/lamby/git/forks/django/django/db/models/sql/compiler.py", line 1034, in apply_converters
    value = converter(value, expression, connection)
TypeError: from_db_value() missing 1 required positional argument: 'context'

======================================================================
ERROR: test_serialisation (tests.tests.SerialisationTests)
----------------------------------------------------------------------
Traceback (most recent call last):
  File "/home/lamby/git/projects/django-enumfield/tests/tests.py", line 525, in test_serialisation
    data = serializers.serialize('xml', TestModel.objects.all())
  File "/home/lamby/git/forks/django/django/core/serializers/__init__.py", line 128, in serialize
    s.serialize(queryset, **options)
  File "/home/lamby/git/forks/django/django/core/serializers/base.py", line 90, in serialize
    for count, obj in enumerate(queryset, start=1):
  File "/home/lamby/git/forks/django/django/db/models/query.py", line 276, in __iter__
    self._fetch_all()
  File "/home/lamby/git/forks/django/django/db/models/query.py", line 1261, in _fetch_all
    self._result_cache = list(self._iterable_class(self))
  File "/home/lamby/git/forks/django/django/db/models/query.py", line 74, in __iter__
    for row in compiler.results_iter(results):
  File "/home/lamby/git/forks/django/django/db/models/sql/compiler.py", line 1034, in apply_converters
    value = converter(value, expression, connection)
TypeError: from_db_value() missing 1 required positional argument: 'context'

======================================================================
FAIL: test_make_migrations (tests.tests_migrations.MigrationIntegrationTests)
----------------------------------------------------------------------
Traceback (most recent call last):
  File "/home/lamby/git/projects/django-enumfield/tests/tests_migrations.py", line 33, in run_command
    shell=True,
  File "/usr/lib/python3.7/subprocess.py", line 395, in check_output
    **kwargs).stdout
subprocess.CalledProcessError: Command 'python /tmp/tmps96atovz/tests/manage.py makemigrations tests' returned non-zero exit status 1.

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "/home/lamby/git/projects/django-enumfield/tests/tests_migrations.py", line 54, in test_make_migrations
    self.run_command('makemigrations tests')
  File "/home/lamby/git/projects/django-enumfield/tests/tests_migrations.py", line 40, in run_command
    self.assertEqual(e.returncode, 0)
AssertionError: 1 != 0

======================================================================
FAIL: test_migrate (tests.tests_migrations.MigrationIntegrationTests)
----------------------------------------------------------------------
Traceback (most recent call last):
  File "/home/lamby/git/projects/django-enumfield/tests/tests_migrations.py", line 33, in run_command
    shell=True,
  File "/usr/lib/python3.7/subprocess.py", line 395, in check_output
    **kwargs).stdout
subprocess.CalledProcessError: Command 'python /tmp/tmpko999ok3/tests/manage.py makemigrations tests' returned non-zero exit status 1.

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "/home/lamby/git/projects/django-enumfield/tests/tests_migrations.py", line 58, in test_migrate
    self.run_command('makemigrations tests')
  File "/home/lamby/git/projects/django-enumfield/tests/tests_migrations.py", line 40, in run_command
    self.assertEqual(e.returncode, 0)
AssertionError: 1 != 0

======================================================================
FAIL: test_stable_migrations (tests.tests_migrations.MigrationIntegrationTests)
----------------------------------------------------------------------
Traceback (most recent call last):
  File "/home/lamby/git/projects/django-enumfield/tests/tests_migrations.py", line 33, in run_command
    shell=True,
  File "/usr/lib/python3.7/subprocess.py", line 395, in check_output
    **kwargs).stdout
subprocess.CalledProcessError: Command 'python /tmp/tmplc7c664w/tests/manage.py makemigrations tests' returned non-zero exit status 1.

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "/home/lamby/git/projects/django-enumfield/tests/tests_migrations.py", line 63, in test_stable_migrations
    self.run_command('makemigrations tests')
  File "/home/lamby/git/projects/django-enumfield/tests/tests_migrations.py", line 40, in run_command
    self.assertEqual(e.returncode, 0)
AssertionError: 1 != 0

----------------------------------------------------------------------
Ran 53 tests in 0.077s

FAILED (failures=3, errors=6)
Destroying test database for alias 'default'...
System check identified no issues (0 silenced).

Hrm, I can't reproduce that using either tox or ./runtests.py locally. Is there anything unusual about your local setup? (Why do you have the custom PYTHONPATH?)

Why do you have the custom PYTHONPATH

I don't have Django installed systemwide so I'm pointing to a git checkout of the upstream code. Perhaps I'm using a too "new" version. :)

Hrm. I'm using a virtualenv with django pip installed, which is the same as what tox does. Odd that you seem to also be getting errors from within that django clone though.


fix bug

Currently the main branch action has some issue
hope fix the issues by this PR.
issues:
https://github.com/sustainable-computing-io/local-dev-cluster/actions/runs/5567468017
https://github.com/sustainable-computing-io/local-dev-cluster/actions/runs/5558945931

It looks like just github action dependence setting issue.

sorry for introducing this issue and thanks for quick fix.


Updated EKS-D for all versions

Issue #, if available:
Description of changes:

Latest EKS-D releases

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

/lgtm
/approve

/cherrypick release-0.15


Design QA - Incorrect Style for Copy Tooltip

The style applied to "Copied!" tooltip is incorrect. Text is not visible:

Please, we should unify style for tooltips:
https://www.figma.com/file/42QoTLQcbObu7esoTPjWRv/Augur-Bet-Design-System?node-id=17%3A12

It's not the same style. Font seems smaller:

Ok!


recording via rdpsnd should be completely disabled by switch

As reported on xrdp-jp group, one of the connectivity issues with Wyse client become clear.
https://groups.google.com/forum/#!topic/xrdp-jp/eh81BoBgz68
Wyse clients seem a little bit stricter. Wyse clients will disconnect when received unknown PDU.  xrdp sends SNDC_REC_NEGOTIATE(39) in sound_send_server_input_formats and this causes disconnection.
Recording via rdpsnd should be completely disabled by switch to be RDP compliant.

created #1427 for this

Thanks, I'll ask the original reporter to test it.

It works fine with Wyse client.


Unpin python-dateutil

Description
Unpins a restrictive upper dependency bound
Related Issue
Fixes  #313
Motivation and Context
Since there‘s no explanation why your python-dateutil requirement is pinned, I’m going to assume it’s no longer an issue.
Upper bounds are bad if not absolutely necessary: https://iscinumpy.dev/post/bound-version-constraints/
How Has This Been Tested?
I’m going to rely on your CI
Types of changes

[x] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

[x] My code follows the code style of this project.
[ ] My change requires a change to the documentation.
N/A I have updated the documentation accordingly.
[x] I have read the CONTRIBUTING document.
N/A I have added tests to cover my changes.
[x] All new and existing tests passed.

I would think a dateutil requirement should remain in requirements in some form if gimme-aws-creds uses dateutil somewhere.  Just not pinned as restrictively.

Interesting, a quick search for dateutil doesn’t seem to show it being used in the code at all.  If that’s true, then, yeah, it should be removed.

Thanks @flying-sheep for the PR, any chance you could push out a release with this change? We currently have some other dependencies (notably jupyter-client) backpinned because of this change so hoping to clear this up if possible!


Problem with Layout Colour black

if you select --bs-black-rgb as a background colour it still does not change the background to black.
This is because it appends the shade value i.e. -bs-black-rgb-500 but there are no styles defined for these black shades

That is correct - Currently Bootstrap do not provide shades for Black and White as they already have some greys instead.
Of course if -500 is black it can't get any darker and if -500 is white it can't get any lighter
They are also in the process of changing quite a lot around the colour system.
However, I could:

Make black and white -5oo
Make black and white remain as such across all shades
make a grey -500 that goes to black and white at the extremes, but I think this may be confusing with the greys

Of course the Black for the Area Colour does work as they are special colours that also have a text colour of the correct contrast in the sass

I will take a look at one of the options I mentioned.

I just created myself some black-rgb-5oo to work around it for now :)


Issue/error with Garage doors implementation?

@rodtoll
I'm getting an error on the new implementation for Garage doors.
Update:  I didn't catch the new config.js requirements.  I'm now retrying.

I just published 0.1.3 which fixes this. (i.e. it handles if you don't specify the garage door element). Thanks for pointing it out.


After trying out Cursor edit

Issue
I find 2 useful features

codebase indexer， can embedding whole code。but it use remote server to storage vectorDB
search web when chat
i think aider can add these features， and use local vector DB

Version and model info
master version

Thanks for trying aider and filing this issue.
This looks like a duplicate of #1005 and #530, so I'm going to close it so discussion can happen there. Please let me know if you think it's actually a distinct issue.


Error when selecting gene expression using interactive viewer

Description

Hi !
I have an error when trying to view gene expression with the graphical interface, while following the tutorial. This cause python to shut down.
...
Minimal reproducible example

Python 3.8.12 | packaged by conda-forge | (default, Jan 30 2022, 23:53:36)
[GCC 9.4.0] on linux
Type "help", "copyright", "credits" or "license" for more information.
import scanpy as sc
import squidpy as sq
import numpy as np
adata = sq.datasets.visium_hne_adata()
img = sq.datasets.visium_hne_image()
viewer = img.interactive(adata)
...

Traceback

After clicking on a gene on the right panel :

>>> /home/user/miniconda3/envs/napari-env/lib/python3.8/site-packages/numpy/core/numeric.py:2446: FutureWarning: elementwise comparison failed; returning scalar instead, but in the future will perform elementwise comparison
  return bool(asarray(a1 == a2).all())
/home/user/miniconda3/envs/napari-env/lib/python3.8/site-packages/squidpy/pl/_interactive/_controller.py:297: FutureWarning: Public access to Window.qt_viewer is deprecated and will be removed in
v0.5.0. It is considered an "implementation detail" of the napari
application, not part of the napari viewer model. If your use case
requires access to qt_viewer, please open an issue to discuss.
  points_controls = self.view.viewer.window.qt_viewer.controls.widgets[layer]
WARNING: Traceback (most recent call last):
  File "/home/user/miniconda3/envs/napari-env/lib/python3.8/site-packages/squidpy/pl/_interactive/_widgets.py", line 41, in <lambda>
    self.itemDoubleClicked.connect(lambda item: self._onAction((item.text(),)))
  File "/home/user/miniconda3/envs/napari-env/lib/python3.8/site-packages/squidpy/pl/_interactive/_widgets.py", line 146, in _onAction
    self._controller.add_points(vec, key=item, layer_name=name)
  File "/home/user/miniconda3/envs/napari-env/lib/python3.8/site-packages/squidpy/pl/_interactive/_controller.py", line 181, in add_points
    self._hide_points_controls(layer, is_categorical=is_categorical_dtype(vec))
  File "/home/user/miniconda3/envs/napari-env/lib/python3.8/site-packages/squidpy/pl/_interactive/_controller.py", line 330, in _hide_points_controls
    gl.replaceWidget(labels[label_key], QLabel("percentile:"))
KeyError: 'n-dim:'
Abandon (core dumped)
...

Any idea on what could cause that ?
Thank you a lot for your time.

Version

scanpy==1.8.2 anndata==0.8.0 umap==0.5.2 numpy==1.21.5 scipy==1.8.0 pandas==1.4.1 scikit-learn==1.0.2 statsmodels==0.13.2 python-igraph==0.9.9 pynndescent==0.5.6
squidpy==1.1.2
...

hi @MarionPerrier
thanks for your interest in Squidpy. Did you try out in a ipython session? Could you try out in a jupyter notebook/lab session?

Thank you for your response !
No, I used python directly.
I have given a try with ipython, and python didn't shut down this time, but I have this result :

And still the message with "KeyError: 'n-dim'"
In [3]: viewer = img.interactive(adata)

/home/user/miniconda3/envs/napari-env/lib/python3.8/site-packages/numpy/core/numeric.py:2446: FutureWarning: elementwise comparison failed; returning scalar instead, but in the future will perform elementwise comparison
  return bool(asarray(a1 == a2).all())
/home/user/miniconda3/envs/napari-env/lib/python3.8/site-packages/squidpy/pl/_interactive/_controller.py:297: FutureWarning: Public access to Window.qt_viewer is deprecated and will be removed in
v0.5.0. It is considered an "implementation detail" of the napari
application, not part of the napari viewer model. If your use case
requires access to qt_viewer, please open an issue to discuss.
  points_controls = self.view.viewer.window.qt_viewer.controls.widgets[layer]
---------------------------------------------------------------------------
KeyError                                  Traceback (most recent call last)
File ~/miniconda3/envs/napari-env/lib/python3.8/site-packages/squidpy/pl/_interactive/_widgets.py:41, in ListWidget.__init__.<locals>.<lambda>(item)
     38 self._unique = unique
     39 self._controller = controller
---> 41 self.itemDoubleClicked.connect(lambda item: self._onAction((item.text(),)))
     42 self.enterPressed.connect(self._onAction)
     43 self.indexChanged.connect(self._onAction)

File ~/miniconda3/envs/napari-env/lib/python3.8/site-packages/squidpy/pl/_interactive/_widgets.py:146, in AListWidget._onAction(self, items)
    144     logg.error(e)
    145     continue
--> 146 self._controller.add_points(vec, key=item, layer_name=name)

File ~/miniconda3/envs/napari-env/lib/python3.8/site-packages/squidpy/pl/_interactive/_controller.py:181, in ImageController.add_points(self, vec, layer_name, key)
    168 layer: Points = self.view.viewer.add_points(
    169     self.model.coordinates,
    170     name=layer_name,
   (...)
    178     **properties,
    179 )
    180 # TODO(michalk8): add contrasting fg/bg color once https://github.com/napari/napari/issues/2019 is done
--> 181 self._hide_points_controls(layer, is_categorical=is_categorical_dtype(vec))
    182 layer.editable = False
    184 return True

File ~/miniconda3/envs/napari-env/lib/python3.8/site-packages/squidpy/pl/_interactive/_controller.py:330, in ImageController._hide_points_controls(self, layer, is_categorical)
    324 slider = RangeSlider(
    325     layer=layer,
    326     colorbar=self.view._colorbar,
    327 )
    328 slider.valueChanged.emit((0, 100))
--> 330 gl.replaceWidget(labels[label_key], QLabel("percentile:"))
    331 gl.replaceWidget(widget, slider)

KeyError: 'n-dim:'

Those black points are here for everything I try to visualize (clusters, genes expression, etc). Any idea on what might cause that ?
This is a screenshot of the window :

Thank you a lot for your time.
Marion

Hi @MarionPerrier ,
You'd need to install the newest version that's on master as pip install git+https://github.com/theislab/squidpy@master and napari==0.4.14.
The KeyError: n-dim: is was fixed in our codebase in #453 , but the black dots are new to me (can confirm the occur in napari==0.4.15, which came 8 days ago).

I just tested, and the new SquidPy version combined with napari 0.4.14 works very fine, even for the dots (that aren't black anymore).
Thank you a lot for your help !
Have a great day.


[CHORE] adding explore page

Description

Screenshots

Checklist

[ ] Pull request has a descriptive title and context useful to a reviewer.
[ ] Pull request title follows the [<catalog_entry>] <commit message> naming convention using one of the following catalog_entry values: FEATURE, ENHANCEMENT, BUGFIX, BREAKINGCHANGE, IGNORE.
[ ] All commits have DCO signoffs.

UI Changes

[ ] Changes that impact the UI include screenshots and/or screencasts of the relevant changes.
[ ] Code follows the UI guidelines.
[ ] Visual tests are stable and unlikely to be flaky. See Storybook and e2e docs for more details. Common issues include:

Is the data inconsistent? You need to mock API requests.
Does the time change? You need to use consistent time values or mock time utilities.
Does it have loading states? You need to wait for loading to complete.

I'm wondering if it would make sense to have a dedicated npm package for this feature. What do you think @nicolastakashi, @jgbernalp ?

That would be desirable. As I was commenting to @nicolastakashi, this might be a separate component that can be included inside a dashboard panel in the future, hopefully the ViewExplore component. So having it as its own package will make it easy to embed in other UIs.

Yeah!
It makes sense to me.
I started under the dashboard package, just to be simpler for now.
But I'll take this in consideration to move to another package.

Awesome!
New feature: add a toggle to show/hide metric list below panel, I think it can be useful 😄

Spotted a minor issue when playing with it locally, when you select a project-specific datasource and then switch to another project:
https://github.com/perses/perses/assets/7058693/c383df41-d49e-48dc-ac15-1cbaad6f8828

LGTM.
Just some nits that should not block this PR and can be addressed in a follow up:
The dropdowns seem all to have different heights, not sure if its caused by its contents.

Is necessary to have the legend name and min step?, it seems this is needed when editing an existing panel, but in this case the explore view purpose might be just to see what's there. A nice feature could be to create a panel from the current explore query.

LGTM.
Just some nits that should not block this PR and can be addressed in a follow up:
The dropdowns seem all to have different heights, not sure if its caused by its contents.
Is necessary to have the legend name and min step?, it seems this is needed when editing an existing panel, but in this case the explore view purpose might be just to see what's there. A nice feature could be to create a panel from the current explore query.

Good catch mate!
Just fix it.


V18 Major

Change-type: major

Resolves: #
HQ:
See:
Depends-on:
Change-type: major|minor|patch

Contributor checklist

[ ] Includes tests
[ ] Includes typings
[ ] Includes updated documentation
[ ] Includes updated build output

How about a commit message like:
BREAKING: Support non-user API keys in auth.isLoggedIn() & whoami()
Ideally the change in the return type of the whoami could be in a separate commit, but I'm fine even w/o that.


data: AryanParashar24

Fixes Issue

Changes proposed

Check List (Check all the applicable boxes)

[ ] My code follows the code style of this project.
[ ] My change requires changes to the documentation.
[ ] I have updated the documentation accordingly.
[ ] All new and existing tests passed.
[ ] This PR does not contain plagiarized content.
[ ] The title of my pull request is a short description of the requested changes.

Screenshots

Note to reviewers

Hey @AryanParashar24, this seems to be a duplicate of #5479. So I'm closing this one.
Once you address the changes requested in #5479, that will be merged by one of our maintainers.


thread 'DNS worker' panicked

Bug Description
We are evaluating Mirrord Enterprise and we are facing this issue:
agent ready - version 3.102.0
2024-05-28T10:52:51.105553Z  WARN ThreadId(07) mirrord_agent::outgoing::udp: interceptor_task -> no messages left
2024-05-28T10:53:00.032631Z  WARN ThreadId(04) mirrord_agent::steal::ip_tables::flush_connections: `conntrack` output is Output {
    status: ExitStatus(
        unix_wait_status(
            256,
        ),
    ),
    stdout: "",
    stderr: "conntrack v1.4.7 (conntrack-tools): 0 flow entries have been updated.\n",
}
2024-05-28T10:54:09.997371Z ERROR ThreadId(14) mirrord_agent::util: Failed setting pid 2853783 namespace "net" with FailedNamespaceOpen(
    Os {
        code: 2,
thread 'DNS worker' panicked at mirrord/agent/src/util.rs:187:42:
Failed setting namespace!: FailedNamespaceEnter(FailedNamespaceOpen(Os { code: 2, kind: NotFound, message: "No such file or directory" }))
note: run with `RUST_BACKTRACE=1` environment variable to display a backtrace
        kind: NotFound,
        message: "No such file or directory",
    },
)

Steps to Reproduce
The command we are running is the following one:
mirrord exec --config-file "./.mirrord/mirrord.json"  --target "deployment/general-service/container/app" --target-namespace "dev-xavi"  --agent-namespace "dev-xavi"  yarn start:meshed

The mirord config file is:
{
  "feature": {
    "env": true,
    "fs": "read",
    "network": {
      "dns": true,
      "incoming": {
        "mode":"steal",
        "port_mapping": [[8080, 8006]],
        "listen_ports": [[8080, 8006]]
      }
    }
  }
}

Backtrace
No response
Relevant Logs
No response
Your operating system and version
MacOS 14.5
Local process
yarn start:meshed
Local process version
3.102.0
Additional Info
This happens only when a request comes in from the public URL

Thanks for reporting this - Are you using the operator?
I suspect that the remote pod you're targeting had a restart, which leads to mirrord failing to access it's Linux namespace.
Can you verify if the container you target had a restart? Sometimes this happens due to health checks failing (timeout) so we recommend using a HTTP filter to avoid stealing the health checks.

hi,
that is correct, we are not using the operator for now and the pod restarts. it does not usually restart when mirrored it is not being executed. what should I do?
thank you in advance

thanks for the clarification - this means you're using mirrord OSS and not enterprise :)
Can you check what's the reason for the restart?

I mean the reason for the restart of the target pod - your service.
P.S if you'd like we can do a screenshare session to debug it.

after the mirrord connection, I am seeing this:
│ app {"level":40,"time":1716896956775,"msg":"[app] [http] SIGTERM received stopping server..."}                                                                                │

I would like you to describe the app's pod, and see if there's any kubernetes event saying restart / health check failed etc

yeah I see this:

What I did not understand is why the other service works perfectly and not this one. Both have the same checks

You can probably run the application with a debugger - put a breakpoint into health and see where it hangs

Would you like to debug it over a call?
meet.google.com/nrq-sstk-qtb

Issue was health checks being stolen and not answered in a timely manner.


Duplicate OBUF insertion (sometimes)

Steps to reproduce the issue
iobuf_bug.zip

Unzip iobuf_bug.zip
Run ./run.sh
Run grep "OBUF " output.v  | wc, outputs

     12      29     140

Example the stat output from circuit:

   Number of wires:                 23
   Number of wire bits:            146
   Number of public wires:          17
   Number of public wire bits:      50
   Number of memories:               0
   Number of memory bits:            0
   Number of processes:              0
   Number of cells:                 52
     BUFG                            1
     CARRY4                          8
     FDRE                           29
     IBUF                            1
     INV                             1
     OBUF                           12

   Estimated number of LCs:          0

Given that the input design only has 8 top level signals to begin with, having the final output have 12 OBUF's is pretty weird.  In this design, all 7 output signals already have OBUF's, but the iopadmap pass appears to insert some "strange" OBUF's which are not required.  It is not clear if the specification of iopadmap is intended to allow for double OBUF's, but it lowers the utility of the pass if it behaves this way.
Expected behavior
Top level signals that lack IBUF or OBUF should gain them, but top signals that already have them should not get a second one.  What is interesting in the provided example is that there is not a straight doubling of OBUF's.  Some path's have 2 OBUF's and some still have only 1.
Actual behavior
iopadmap appears sometimes add a second OBUF when there is a signal alias, for example the input:
module top (
  input  clk,
  output [3:0] led,
  inout out_a,
  output [1:0] out_b
);

...

wire inter_wire, inter_wire_2;
OBUF #(
  .IOSTANDARD("LVCMOS33"),
  .SLEW("SLOW")
) OBUF_7(.I(LD7), .O(inter_wire_2));
assign inter_wire = inter_wire_2;
assign led[1] = inter_wire;

became:
   (* module_not_derived = 32'd1 *)
    (* src = "counter.v:25" *)
    OBUF #(
      .IOSTANDARD("LVCMOS33"),
      .SLEW("SLOW")
    ) OBUF_7 (
      .I(LD7),
      .O(inter_wire_2)
    );
    (* keep = 32'd1 *)
    OBUF _16_ (
      .I(inter_wire_2),
      .O(led[1])
   );

Tested at Yosys 0.9+1706 (git sha1 6edca057, clang 8.0.1-3+build1 -fPIC -Os) which is master as of creation of this issue.

From what I see in the testcase, iopadmap correctly inserts a buffer on the led[0] wire, which does not yet have a buffer. The OBUF_7 instance, on the other hand, is an output buffer connected to a wire that is not a top-level port, and thus invalid.

From what I see in the testcase, iopadmap correctly inserts a buffer on the led[0] wire, which does not yet have a buffer. The OBUF_7 instance, on the other hand, is an output buffer connected to a wire that is not a top-level port, and thus invalid.

If you believe that iopadmap is behaving correctly, then the current specification of iopadmap is unsuitable for general use.  In the provided example, the wires led[0], inter_wire_2 and inter_wire are all the same net, and clearly drive a top level port.
Do you believe the overall script synth_xilinx script behaved correctly in this case by delivering an output netlist that has more OBUF's than top level output ports?  Do you believe the input netlist was malformed?
Let me simplify the example for discussion purposes:
Input verilog:
module top (
	input  clk,
    input in,
	output out_b
);
	bottom_intermediate bottom_intermediate_inst(.I(in), .O(out_b));
endmodule

module bottom_intermediate (
	input I,
	output O
);
	OBUF OBUF_8(.I(I), .O(O));
endmodule

Output verilog:
* Generated by Yosys 0.9+932 (git sha1 8fe9c84e, clang 8.0.1-3+build1 -fPIC -Os) */

(* top =  1  *)
(* src = "counter.v:1" *)
module top(clk, in, out_b);
  (* unused_bits = "0" *)
  wire _0_;
  (* src = "counter.v:6|counter.v:10" *)
  wire \bottom_intermediate_inst.I ;
  (* src = "counter.v:6|counter.v:11" *)
  wire \bottom_intermediate_inst.O ;
  (* src = "counter.v:2" *)
  input clk;
  (* src = "counter.v:3" *)
  input in;
  (* src = "counter.v:4" *)
  output out_b;
  (* keep = 32'd1 *)
  IBUF _1_ (
    .I(clk),
    .O(_0_)
  );
  (* keep = 32'd1 *)
  IBUF _2_ (
    .I(in),
    .O(\bottom_intermediate_inst.I )
  );
  (* keep = 32'd1 *)
  OBUF _3_ (
    .I(\bottom_intermediate_inst.O ),
    .O(out_b)
  );
  (* module_not_derived = 32'd1 *)
  (* src = "counter.v:6|counter.v:13" *)
  OBUF \bottom_intermediate_inst.OBUF_8  (
    .I(\bottom_intermediate_inst.I ),
    .O(\bottom_intermediate_inst.O )
  );
endmodule

Do you also claim this is valid behavior for iopadmap?

The crux of the issue here is what counts as a "direct connection to a top-level port".  The Verilog standard of course doesn't say anything OBUFs, but assigned nets are in general not considered to be a single net (rather, an assign is basically a buffer).  In particular, IEEE 1364.1 explicitely says that tristate (Z) values do not survive assigns, which is perhaps the closest thing to the issue at hand.  Likewise, output port connections of submodules are treated as implicit assigns. So, I would argue that the input netlist is invalid, as the OBUF output is not directly connected to a top-level port.
It may be useful to go beyond what IEEE 1364.1 guarantees and have somewhat more flexible handling of output buffers (ie. allowing detection of OBUFs and/or tristate assignments beyond an assignment or submodule boundary), but I consider it to be an enhancement, and not a bug.  In particular, handling the submodule case correctly would require doing inter-module analysis, which yosys doesn't currently have good infrastructure for.  We will do it at some point, but it's not a high priority item right now.


UI starters

FLASK
#1 Personalized index.html

#2 Personalized stub.html

FASTPAGES
#3 Use of Submenu
#4 Personal usage of JavaScript + HTML fragment table
#5 Scrum Process
#6 Team Project

2.7+
I love how personalized your flask site is. The pictures and descriptions really tell a lot about your passions and hobbies.
You also completed all the requirements such as the submenu and the data table using javascript and HTML fragments.

2.7
Had personalized server and fastpages and did a great job with collaboration and JavaScript

2.7, good work with the SCRUM diagram with roles. Flask server is very nicely customized. Very clear final project purpose, well-put and described. Javascript work is fully complete with customized table. Well done.

2.7+, on the first portion I have her a 0.9+ because it looked very well proportioned and nice, on the second portion I gave here a 0.9+ because it also looked very nice and captured the spirit of the assignment, and on the third portion I gave her a 0.9+ because it was very detailed and javascript.

I would give you a 2.7, all the required elements are included, and the pictures show some nice information about yourself.


dbeaver/dbeaver#19821 exclude all suggestions with dots

Closes dbeaver/dbeaver#19821
Exclude all suggestions with dots from Hippie due to them being error prone.

Verified


feat: Bitbucket branch sync improvements

[x] Support renovate updates for:

[x] bitbucket-pipelines.yml

[ ] Support bitbucket repo sync workflow for any commits on bitbucket branch

Renovate updating, see changes: https://github.com/psychobolt/vite-storybook-boilerplate/commit/24dc999531249ba2e49943b0f9d8f82023d4a76e


Gzip handler uses I_SIZE as definitive size

Gzip's original size is stored as a uint32 – it can't encode sizes larger than 2^32 - 1, as elaborated on in https://github.com/resin-io/etcher/issues/638

fixed by #36


order of optimizer.step() and lr_scheduler.step() in tests

Unit tests raise the following warning
tests/ulyanov_et_al_2016/test_utils.py::test_DelayedExponentialLR
  /home/runner/work/pystiche_papers/pystiche_papers/.tox/tests/lib/python3.6/site-packages/torch/optim/lr_scheduler.py:123: UserWarning: Detected call of `lr_scheduler.step()` before `optimizer.step()`. In PyTorch 1.1.0 and later, you should call them in the opposite order: `optimizer.step()` before `lr_scheduler.step()`.  Failure to do this will result in PyTorch skipping the first value of the learning rate schedule. See more details at https://pytorch.org/docs/stable/optim.html#how-to-adjust-learning-rate
    "https://pytorch.org/docs/stable/optim.html#how-to-adjust-learning-rate", UserWarning)

@jbueltemeier Please look into it.

The warning only occurred in the test, because there it was only tested if the lr_scheduler was working correctly, for which only the lr_scheduler.step() is needed. So it is not important here whether the optimizer.step() is called or not.

Fixed in #120.


Custom compiler question: Ninja wants an intermediate file.

I have a custom compiler which imitates GCC command line.
To compile "test.cpp" it makes a processed file "test.intermediate" and then compiles it with GCC to "test.o".
It is used in cmake as set(CMAKE_CXX_COMPILER) and with "unix make" generator everything works ok.
When I switch to ninja generator, it rebuilds all such files every time because the temporary file "test.intermediate" is missing.
ninja explain: output test.intermediate of phony edge with no inputs doesn't exist
May you help me with some advice?
I need to put the intermediate file next to the original file (to keep relative #includes), but I don't want to keep them after build.
P.S. It happens only on Linux with GCC. On Windows with CL, everything works ok.

Ok, I realized that I don't really need relative includes, so the problem is solved.
Nevertheless it's interesting how to deal with that problem (compiler works via a temporary file) and how did ninja know about that file?


Code for MTBench and Just-Eval

Hi, thanks for the great work.
I would love to ask if you can provide the codes for MTBench and Just Eval codes. Thank you.

Hi,
Thanks! I used the original evaluation from MTBench and Just Eval, and we do have some "bridging code" to deal with the generation part. I am currently on a travel and will update the code soon after coming back~~


Feature request: Consider use YARP as reverse proxy

Microsoft is developing a new reverse proxy "YARP" (https://github.com/microsoft/reverse-proxy)
The performance in YARP is looking really good. Today Azds uses traefk and it adds alot of ms to the requests and is missing key features (like using SSL in DevSpaces).
Please consider working with the YARP team to add the features needed for AZDS so it can replace traefk :)

(Adding @greenie-msft, for FYI)
Good feedback as well. As mentioned in the other issue, would you be willing to reach out to our PM team about this (https://github.com/Azure/dev-spaces/issues/325)?
Thanks!

Hi,
Yes, of course.
I have spoken to @greenie-msft before so I'll contact him to setup a Teams meeting and i'll share out latest ideas/findings :)


It's not possible to output ressource URLs as absolute paths in Development

Package version
3.0.0
Describe the bug
In my project (AdonisJS 6 + Vite) I have the situation that the resource URLs have to be output as absolute URLs. The resources are integrated into the HTML using @vite(['resources/css/app.css', 'resources/js/app.js']). But all my attempts have failed, the CSS and JS files are always, at least in development mode, only integrated as relative and not absolute URLs.

My use case is, that I have an edge Template which I use to generate a PDF file.
Reproduction repo
No response

I think you can use the assetsUrl option. https://docs.adonisjs.com/guides/basics/vite#deploying-assets-to-a-cdn


Support setting routes for serving static files

This is a work in progress, aiming to address #60
There is still much work to do ...

It can work as expected now but with unhandled errors.
ERROR - http.server: Unhandled exception on HTTP::Handler
Closed stream (IO::Error)
  from A:\Scoop\apps\crystal\current\src\http\server\response.cr:204 in 'check_headers'
  from A:\Scoop\apps\crystal\current\src\http\server\response.cr:40 in 'status='
  from A:\Scoop\apps\crystal\current\src\http\server\response.cr:89 in 'status_code='
  from C:\Users\Sunrise\Documents\CodeRepo\Crystal\test\static-server\lib\grip\src\grip\handlers\exception.cr:16 in 'call'
  from A:\Scoop\apps\crystal\current\src\http\server\request_processor.cr:51 in 'process'
  from A:\Scoop\apps\crystal\current\src\http\server.cr:521 in 'handle_client'
  from A:\Scoop\apps\crystal\current\src\http\server.cr:451 in '->'
  from A:\Scoop\apps\crystal\current\src\fiber.cr:141 in 'run'
  from A:\Scoop\apps\crystal\current\src\fiber.cr:93 in '->'

Closed stream (IO::Error)

Can you investigate why does this happen?

Relevant issue: https://github.com/crystal-lang/crystal/issues/14566

Relevant issue: crystal-lang/crystal#14566

Should be fixed by #66

The code works well now and is reviewable.

@yanecc Can you provide an example?

The code works well now and is reviewable.

I am going to modify this code a little bit to be a bit more Grip-esque, I will update the documentation with the news.

@yanecc Can you provide an example?

Sure, here is a simple illustration.
class Application < Grip::Application
  def initialize()
    super(environment: "development", serve_static: true)
  end

  getter host : String = "<IP_ADDRESS>"
  # getter pubilc_dir : String = "./public"
  getter static_routes : Hash(String, String) = {
    "/foo/bar" => "./dir1",
    "/test"    => "./path/to/dir2",
  }
end

# By default, when serve_static is set to true, public_dir will be set to "./public".
# You could use pubilc_dir to set the static directory.

# If static_routes is set, pubilc_dir will be overwritten.
# GET http://<IP_ADDRESS>          #=> 404 Not Found
# GET http://<IP_ADDRESS>/foo/bar  #=> 200 OK -> ./dir1/index.html
# GET http://<IP_ADDRESS>/test     #=> 200 OK -> ./path/to/dir2/index.html

@yanecc Can you provide an example?

Sure, here is a simple illustration.
class Application < Grip::Application
  def initialize()
    super(environment: "development", serve_static: true)
  end

  getter host : String = "<IP_ADDRESS>"
  # getter pubilc_dir : String = "./public"
  getter static_routes : Hash(String, String) = {
    # "/"        => "./public",
    "/foo/bar" => "./dir1",
    "/baz"     => "./path/to/dir2",
  }
end

# By default, when serve_static is set to true, public_dir will be set to "./public".
# You could use pubilc_dir to set the static directory.

# If static_routes is set, pubilc_dir will be overwritten.
# GET http://<IP_ADDRESS>          #=> 404 Not Found
# GET http://<IP_ADDRESS>/foo/bar  #=> 200 OK -> ./dir1/index.html
# GET http://<IP_ADDRESS>/baz      #=> 200 OK -> ./path/to/dir2/index.html

I updated the version of Grip to 3.0.0 since I implemented breaking changes and will update the documentation to reflect your changes as well, added a new DSL macro.

@yanecc Added the documentation, check it out.

@yanecc Added the documentation, check it out.

Looks good. 👍


Broken URL's

Great script, thanks for sharing!
Using your example:
$request = curl_init('https://xxxxxxxx.net/prox/Proxy.php');

curl_setopt($request, CURLOPT_HTTPHEADER, array(
    'User-Agent: Mozilla/5.0 (X11; Ubuntu; Linux x86_64; rv:70.0) Gecko/20100101 Firefox/70.0',
    'Proxy-Auth: Bj5pnZEX6DkcG6Nz6AjDUT1bvcGRVhRaXDuKDX9CjsAc7',
    'Proxy-Target-URL: https://www.github.com'
));

curl_exec($request);

Once the GitHub page loads into browser, all the URL's in their menu become something like https://xxxxxxxx.net/enterprise and that of course is a 404.
Is there any way to use the loaded site with all remote links working?

Hi,
thanks for raising the question. While I understand your use-case, this script is not meant to be used this way. It's primary use-case is to proxy HTTP requests that are made programmatically, not by browsing the response. I'd love to have a feature you describe above, but such task is non-trivial and might require to completely redesign the script.
I think a project like https://github.com/Athlon1600/php-proxy-app might have fit your needs better :wink:.

Totally understand you point. Thanks for taking time to explain.


coredump when do redisAsyncFree

After Server closed the connection, the client will coredump when do redisAsyncFree.
#0  0x00007f08d0cdfc37 in __GI_raise (sig=sig@entry=6) at ../nptl/sysdeps/unix/sysv/linux/raise.c:56
#1  0x00007f08d0ce3028 in __GI_abort () at abort.c:89
#2  0x00007f08d0d1c2a4 in __libc_message (do_abort=do_abort@entry=1, fmt=fmt@entry=0x7f08d0e2a6b0 "*** Error in `%s': %s: 0x%s *\n") at ../sysdeps/posix/libc_fatal.c:175
#3  0x00007f08d0d2855e in malloc_printerr (ptr=, str=0x7f08d0e26801 "free(): invalid pointer", action=1) at malloc.c:4996
#4  _int_free (av=, p=, have_lock=0) at malloc.c:3840
#5  0x00007f08ce3a1ae3 in _dictClear (ht=0x7f0888001490) at dict.c:231
#6  0x00007f08ce3a2137 in dictRelease (ht=0x7f0888001490) at dict.c:239
#7  __redisAsyncFree (ac=0x7f0888001810) at async.c:269
#8  0x00007f08ce82ae8f in operator() (__closure=, c=0x7f0888001810) at redis_script.cpp:416
#9  boost::detail::sp_counted_impl_pd<redisAsyncContext, redis_helper::init_redis(boost::shared_ptr, ev::dynamic_loop&, const string&, short unsigned int)::__lambda1>::dispose(void) (
this=) at ../../../lib/include/boost/smart_ptr/detail/sp_counted_impl.hpp:153
#10 0x00007f08ce80870e in boost::detail::sp_counted_base::release (this=0x7f0888001510) at ../../../lib/include/boost/smart_ptr/detail/sp_counted_base_gcc_x86.hpp:146
#11 0x00007f08ce8111d1 in ~shared_count (this=, __in_chrg=) at ../../../lib/include/boost/smart_ptr/detail/shared_count.hpp:371
#12 ~shared_ptr (this=, __in_chrg=) at ../../../lib/include/boost/smart_ptr/shared_ptr.hpp:328
#13 reset (this=0x295b6e0) at ../../../lib/include/boost/smart_ptr/shared_ptr.hpp:619
#14 send_queue::thread_func (this=0x295b610) at send_queue.cpp:84
#15 0x00007f08cdd3a75a in thread_proxy () from /home/zhe.chen/zk_namespace/gaia3/elete/cpplib/libboost_thread.so.1.55.0
#16 0x00007f08d1076184 in start_thread (arg=0x7f0893fff700) at pthread_create.c:312
#17 0x00007f08d0da337d in clone () at ../sysdeps/unix/sysv/linux/x86_64/clone.S:111
We create the connection as follow:
void init_redis(boost::shared_ptr *context, ev::dynamic_loop &loop, const std::string &host, unsigned short port)
{
context->reset(redisAsyncConnect(host.c_str(), port),
[](redisAsyncContext *c) -> void
{
std::cout << "close redis connection" << std::endl;
redisAsyncFree(c);
});
if ((*context)->err)
{
    std::cout << "can't connect to redis" << std::endl;
    throw new cannot_connect_redis((*context)->errstr);
}

(*context)->data = this;
redisLibevAttach(loop, context->get());
redisAsyncSetConnectCallback(context->get(), redis_helper::scb_connect);
redisAsyncSetDisconnectCallback(context->get(), redis_helper::scb_disconnect);

}

I also encountered a similar situation when I use kqueue as the I/O multiplexing。When I use GDB to debug, I found that redisAsyncDisconnect(it calls redisAsyncFree) was called multiple times which leads to a crash. When the Redis server closes the connection, check whether the socket is readable, if so, you should call redisAsyncHandleRead to handle the read event, it will shift the reply callback, and then When there is no more callback, it will call redisAsyncFree to free the context. Maybe you can try to use redisAsyncDisconnect instead of redisAsyncFree. Because redisAsyncDisconnect can help you deallocate memory and shift pending callbacks.

Closing old issues.  Feel free to reopen if this is still an issue for you.


[Pigeon] Upgrade message retries to 15 from 3 to guarantee performance

As a user, I want paloma to maximall retry my messages if a validator has failed to take up a message and deliver for 15 attempts.  We don't want message delivery failures when volume on the system is low.
Rationale:

I would argue that given a validator set of 30, we want at least 50% which is 15 retries maximum.  If the validator set rises AND we are massively successful with lots of volume, we can turn this number down to 3 retries .
Let's upgrade to 15 attempts from different validators, maximum.
So 15 validators out of the 30 will get a shot at sending the message while we have low volumes and lots of space. Let's not worry about a future that isn't here as yet.

Ideally, this should be a governance proposal update @byte-bandit

according to @verabehr

In our current case, Paloma doesn't think it's a faulty pigeon. Pigeon tries to relay and gets an error back from teh evm. This is the error
both gasPrice and (maxFeePerGas or maxPriorityFeePerGas) specified
. The assigned pigeon relays this error data back to paloma and paloma records it as a failure, but the pigeon did it's job.
I think if a pigeon fails to do it's job, it should be jailed in all cases. We have definitely seen this for example here ⏰┋alerts where we got jailed because we didn't provide evidence (most likely rpc issue

@taariq conclusion:

So this situation was an evm problem. Pigeons were doing their job so the pigeon was NOT at fault. Okay. Then there's no need for 15x more attempts.


--selector flag not working for helmfile apply

Hi,
I'm trying to use the selector cli flag with the apply command but somehow it fails for me.
I run the following using helmfile v0.98.2:
helmfile --selector name=rhsso-broker apply

It fails with the following message:

Incorrect Usage. flag provided but not defined: -selector name

Do you have any idea? Do I miss anything?
Best regards

Something on my end was odd.
I did the following from within a script:
SELECTOR_STRING="--selector name=rhsso-broker"
helmfile "$SELECTOR_STRING" apply

I changed it to the following and now it's working:
SELECTOR_STRING="-l name=rhsso-broker"
helmfile $SELECTOR_STRING apply

And I also updated helmfile to the lastest version.
But I think it's because of the quotes I used before. Could be that I don't understand Shellscript after >10 years  oft working with it lol
Issue can be closed!

Thanks for sharing your findings! Yeah, shell scripting is hard and I'm not good at it either 😄


Iceberg connector support for Materialized Views

This PR is for Iceberg connector changes to support materialized views. Note that only the top commit (
25678de) needs to be reviewed as a part of this PR. Other commits are being reviewed here: https://github.com/prestosql/presto/pull/3283

@anjalinorwood can you rebase now that the materialized view code is merged?

@David Phillips<EMAIL_ADDRESS>I just now rebased. Thanks.
-Anjali.
On Tue, Sep 1, 2020 at 12:58 PM David Phillips<EMAIL_ADDRESS>wrote:

@anjalinorwood https://github.com/anjalinorwood can you rebase now that
the materialized view code is merged?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/prestosql/presto/pull/4832#issuecomment-685099004,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AMODJAZBORACYOXWWZ5PAFLSDVG7PANCNFSM4P7FKBKA
.

@martint and @electrum Did you get a chance to review this PR? Thanks.

@martint Did you want to give this a look? I think @electrum is winding up his review. Thank you both.

Thanks!

https://github.com/prestosql/presto/issues/5892


Add VDT option

https://coffeeequipmentreviews.wordpress.com/2020/02/18/vdt-vibration-distribution-tool/

v37


citation?

Please add information on how to cite the tool to the readme.

The readme has been updated to include both current citations for the MOB-suite

The readme has been updated to include both current citations for the MOB-suite


feat: Https k3d rancher desktop

Updates docs on k3d local and https flow.

@Claywd: can you sign the commits please, and I'll review it after.

@Claywd : in the WSL section, you wrote "There are limitations within Docker Desktop and how it manages volumes which prevent us from supporting it at this moment.", but AFAIK, kubefirst with WSL is working? This part is not clear to me.

That was my bad. I meant rancher-desktop


fix(resize-image): switched imagemagick for sharp library

fixes issue #199.
ImageMagick binary provided by Cloud Functions do not support webp image format.

@laurenzlong  Happy to close this PR if sharp library isn't a desirable option. Here's a couple of links that helped my decision to use it:
imagmagick doesn't support webp
firebase function code sample

Thanks for the context! We'll review it with the team, but it makes sense to me, especially since webp is what Google recommends as an image format for the web: https://developers.google.com/speed/webp

Thanks for putting this together! Let's get this out :)

@russellwheatley Can you please look at PREINSTALL and POSTINSTALL and extension.yaml, and see if we are making explicit reference to ImageMagick, and if so, indicate that we are using Sharp not ImageMagick?

Codecov Report

Merging #207 into next will decrease coverage by 0.38%.
The diff coverage is 0.00%.

@@            Coverage Diff             @@
##             next     #207      +/-   ##
==========================================
- Coverage   40.48%   40.09%   -0.39%
==========================================
  Files          20       20
  Lines         625      631       +6
  Branches       81       83       +2
==========================================
  Hits          253      253
- Misses        370      376       +6
  Partials        2        2

Impacted Files
Coverage Δ

storage-resize-images/functions/src/index.ts
0.00% <0.00%> (ø)


Make reports downloadable on private repositories

For the OTTR_Quizzes repos, that are supposed to be private repos, we still need to be able to download the spell check errors and quiz errors TSV files. But permissions errors make it so the link doesn't work.
I need to figure out a way around this.

So far, I think I'm finding that as long as GH_PAT is set and jhudsl-robot is added as a collaborator, then you can totally have your repository set to private.

I think a better test is if someone who doesn't have my credentials tries to do this.

This is a thing that has implemented but inconsistently so. I know how to do it but we just need to make sure that implementation gets done.


usb: allow adding isochronous endpoints

This adds (basic) support for isochronous endpoints. In theory, isochronous endpoints have guaranteed bandwidths and are support to adhere to strict timings. But it seems that nothing bad happens if you don't follow the specs closely in this regard. Better handling could also still be added in the future.

Note that for this to work on the RP2040, this PR is required, otherwise it might panic because of too big packets.
I have not tested this on any other boards.


Illuminate\Contracts\Routing\ResponseFactory missing file() method

Laravel Version: 5.4.*
PHP Version: 7.1

Description:
Illuminate\Contracts\Routing\ResponseFactory missing file() method
Steps To Reproduce:
Check and compare Illuminate\Routing\ResponseFactory class and Illuminate\Contracts\Routing\ResponseFactory contract.
Illuminate\Routing\ResponseFactory itself has file() method, but Illuminate\Contracts\Routing\ResponseFactory hasn't.
And there is info in official docs about the using both download() and file() methods https://laravel.com/docs/5.4/responses#file-downloads and https://laravel.com/docs/5.4/responses#file-responses.

I think that when file() methods was added by this commit https://github.com/laravel/framework/commit/a96ea3c8c95657832f3d534d42db4b5efc7b8515, the author just forget to add same method to corresponding contract.

Adding a new method to the contract would be considered a breaking change, that's why it wasn't added.

But then file method needed to be removed isn't it?

why removed? You can use it or not use it.

Even if it not defined in contract?  Then, it seems,
} don't understand contracts right)


[fix][fn] Configure pulsar admin for TLS

Motivation
This PR is a combination of https://github.com/apache/pulsar/pull/20482 and https://github.com/apache/pulsar/pull/20513 because cherry picking those PRs produced too many conflicts.
Modifications

Made the same addition as the source PRs, though the code has changed a bit, so I had to make a few extra changes.

Documentation

[x] doc-not-needed

/pulsarbot rerun-failure-checks

The test failures are unrelated, merging anyway.

As discussed on the mailing list https://lists.apache.org/thread/w4jzk27qhtosgsz7l9bmhf1t7o9mxjhp, there is no plan to release 2.9.6, so I am going to remove the release/2.9.6 label


Request: Example of event_handler

Hi!
Looking at the Discord, almost every week someone asks about an explanation on how the event_handler works.
Would it be a good idea to just make an example using the event_handler where it is used to collect messages, message updates and so on?

I just came to the issue tracker to ask about this.
I think this is a good idea

There already is a very basic example in basic_structure.
Nonetheless, I think having an explicit example is good. I have proposed one in #199


Add RocketMQ APIs and Clients to README

Codecov Report

Merging #4728 (e13dd2c) into develop (67b49b5) will decrease coverage by 0.19%.
The diff coverage is n/a.

@@              Coverage Diff              @@
##             develop    #4728      +/-   ##
=============================================
- Coverage      44.81%   44.62%   -0.20%
+ Complexity      7560     7530      -30
=============================================
  Files            977      977
  Lines          67640    67640
  Branches        8934     8934
=============================================
- Hits           30315    30186     -129
- Misses         33580    33706     +126
- Partials        3745     3748       +3

Impacted Files
Coverage Δ

.../broker/subscription/SubscriptionGroupManager.java
68.79% <0.00%> (-14.19%)
:arrow_down:

...n/java/org/apache/rocketmq/test/util/StatUtil.java
16.19% <0.00%> (-14.09%)
:arrow_down:

...va/org/apache/rocketmq/logging/inner/Appender.java
29.21% <0.00%> (-7.87%)
:arrow_down:

...ketmq/common/protocol/body/ConsumerConnection.java
95.83% <0.00%> (-4.17%)
:arrow_down:

...ache/rocketmq/broker/topic/TopicConfigManager.java
59.03% <0.00%> (-3.06%)
:arrow_down:

...ava/org/apache/rocketmq/filter/util/BitsArray.java
59.82% <0.00%> (-2.57%)
:arrow_down:

...a/org/apache/rocketmq/filter/util/BloomFilter.java
60.43% <0.00%> (-2.20%)
:arrow_down:

...tmq/namesrv/processor/DefaultRequestProcessor.java
60.33% <0.00%> (-1.99%)
:arrow_down:

...java/org/apache/rocketmq/logging/inner/Logger.java
54.06% <0.00%> (-1.92%)
:arrow_down:

...he/rocketmq/client/trace/AsyncTraceDispatcher.java
81.42% <0.00%> (-1.91%)
:arrow_down:

... and 16 more

Help us with your feedback. Take ten seconds to tell us how you rate us.

Coverage decreased (-0.2%) to 48.888% when pulling e13dd2c7a3bedacbaf43517c00f047274c76a55f on aaron-ai:pr into 67b49b55068e85551676a9af1f6cfa3776e41e08 on apache:develop.

@aaron-ai It would be better if we could create an issue before submitting the pull request.