[Feature-#6422] task group queue

Task group queue (TGQ) can limit the concurrency of taskInstance.TGQ also supports priority-based control, which ensures that important tasks can be executed first. Users can also execute a task compulsively, ignoring the TGQ.

Codecov Report

Merging #6722 (0f8b962) into dev (e6fe39e) will decrease coverage by 0.37%.
The diff coverage is 3.96%.

@@             Coverage Diff              @@
##                dev    #6722      +/-   ##
============================================
- Coverage     32.78%   32.40%   -0.38%
- Complexity     1621     1623       +2
============================================
  Files           432      435       +3
  Lines         14385    14584     +199
  Branches       1446     1471      +25
============================================
+ Hits           4716     4726      +10
- Misses         9217     9400     +183
- Partials        452      458       +6

Impacted Files
Coverage Δ

.../dolphinscheduler/common/enums/StateEventType.java
0.00% <0.00%> (ø)

...inscheduler/common/enums/TaskGroupQueueStatus.java
0.00% <0.00%> (ø)

...heduler/remote/command/TaskEventChangeCommand.java
0.00% <0.00%> (ø)

...e/dolphinscheduler/server/master/MasterServer.java
0.00% <0.00%> (ø)

...er/server/master/processor/TaskEventProcessor.java
0.00% <0.00%> (ø)

...ter/processor/queue/StateEventResponseService.java
0.00% <0.00%> (ø)

...er/server/master/runner/WorkflowExecuteThread.java
10.08% <0.00%> (-0.58%)
:arrow_down:

...r/server/master/runner/task/BaseTaskProcessor.java
5.18% <0.00%> (-0.04%)
:arrow_down:

...server/master/runner/task/CommonTaskProcessor.java
5.47% <0.00%> (-0.98%)
:arrow_down:

...lphinscheduler/service/process/ProcessService.java
36.00% <6.09%> (-2.31%)
:arrow_down:

... and 5 more

Continue to review full report at Codecov.

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

i think this feature is OK now, but still there have some works to do.

documents about this feature
upgrade sql-schema in directory sql/upgrade/2.1


GNU ARM Embedded link broken in Getting Started

The link to GNU ARM Embedded on https://docs.zephyrproject.org/latest/getting_started/toolchain_3rd_party_x_compilers.html#gnu-arm-embedded should probably be updated to https://developer.arm.com/tools-and-software/open-source-software/developer-tools/gnu-toolchain/gnu-rm/downloads

Thanks for the report.
Fix: https://github.com/zephyrproject-rtos/zephyr/pull/21583

@danieleggert as I commented in the PR, the existing link seems to work fine? https://developer.arm.com/open-source/gnu-toolchain/gnu-rm

The existing link works for me too, maybe it was a temporary issue with the website. Therefore closing this issue.


All line numbers in tables are padded to the left

Now all line numbers in tables are padded to the left. This improves readability especially for larger tables with many entries and enhances consistence in data presentation across the different outputs.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

2 of 2 (100.0%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.001%) to 96.744%

Totals

Change from base Build<PHONE_NUMBER>:
0.001%

Covered Lines:
2763

Relevant Lines:
2856

💛  - Coveralls


[Gally]: master <- dev

Automatically created by Git-Ally

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Password flow not working

Hi,
I am trying to send Authorization header, but its not working for me.
 headers.append("Authorization", "Basic " + btoa(username + ":" + password)); this.oauthService.fetchTokenUsingPasswordFlow(accountInfo.email, accountInfo.password, headers)
Actually, I am getting this response from my server,
{"timestamp":1514304211712,"status":401,"error":"Unauthorized","message":"Full authentication is required to access this resource","path":"/oauth/token"}
but the same is working fine with curl and postman client.
Let me know if I am doing something wrong
@manfredsteyer Please suggest me something for this.

Can you provide a minimal stackblitz.com sample?


vrgather SEW=8b only works up to 256 elements

The spec already warns about SEW=8 in the text saying:

When SEW=8, vrgather.vv can only reference vector elements 0-255.

However, this effectively makes 256b the largest VLEN where the ISA "fully/completely" works (at LMUL=8, SEW=8, that's VLMAX=256). Past that size, vrgather.vv starts progressively covering fewer cases. At VLEN=512b, vrgather works for LMUL=1,2,4 and all SEWs, but not for LMUL=8. At VLEN=1024, SEW=8 does not work for LMUL=4,8. Etc, etc.
One proposal would be to define EMUL and EEW for the "index" source of vrgather based on the implementation VLEN of the machine for SEW=8,
 if ( SEW == 8 && VLMAX > 256 ) {
   if ( LMUL == 8 ) { illegal instruction }
   else             { EEW = 2*SEW;  EMUL = 2*LMUL }
}

This would now "fix" vrgather for SEW=8 and LMUL=1,2,4. Obviously, this can't work for LMUL=8. And, yes, this only moves the goalpost from VLEN=2048 to VLEN=131072, but that feels more satisfactory...

I believe the current formulation is useful even for eew=8, so eliminating it is problematic.
It is possible to have another variant of vrgather that works on mixed vector for vs1 and vs2 if this use case is warranted.This could be added post v1.0 hopefully with the expedited process in place.
I see this as an after version V1.0 issue.

Software would have to know VLMAX to be able to use this form, so this would break software portability across different VLEN machines.

Proposal from task group is to add a vrgatherei16, which has fixed EEW of 16 for the index vector.  This in particular would support vrgather of >256 8b elements. Another benefit is lower register usage for the index vector for SEW>16b.


Less logs and EC clients clean up

One EC API client instead of two;
Less logs: excluding transactions and other nonrelevant information for consensus client debugging;
local-network:

Second EC is geth instead of besu;
Easier setup for new EC;
New naming conventions for EC clients: ec-1, ec-2, ... ;
Updated npm dependencies.

@goodmai Fixed


⚠ Samsung has degraded performance

In 18ee778, Samsung (https://samsung.com) experienced degraded performance:

HTTP code: 200
Response time: 2227 ms

Resolved: Samsung performance has improved in d298c29.


Resource failed

It seems that the resource fbb170fa64a06b66dd9a16aaa21bf12848042d9a of GCCBootstrap-aarch64-apple-darwin20.v11.0.0-iains.x86_64-linux-musl.unpacked in Artifact.toml has been failed.

indeed, Artifact.toml hasn't been updated for a while, but the old resource should not fail.


Upgrade machine and various build/test fixes

/lgtm
Are you planing to add a CI for this?

/lgtm
Are you planing to add a CI for this?

Oh yes I should add one. You're right

Oh yes I should add one. You're right


Apollo 3.0: Upgraded Architecture

System information

OS Platform and Distribution ( Linux Ubuntu 14.04):
Apollo installed from (source):
Apollo version (3.0):

Hello :+1:
in https://github.com/ApolloAuto/apollo; there is a image that is Apollo 3.0: Upgraded Architecture. and in image,  it have a words that is "Mass production service components", what is means ? Please tell me. thank you

Apollo going through a big makeover, just wait for it.

Hi @breakisx ,
The Mass Production Service Components are the components that are critical for manufacturing autonomous driving vehicles. This module is not open sourced. If you or your company is interested in manufacturing smart vehicles, we are more than happy to connect you with our business development team and they can tell you more.


Default Spectre/Meltdown HW Inits to Kernel mode

Default Spectre/Meltdown HW Inits used to be "0" Kernel+User
mitigations.  This commit changes the default to "1" for only
Kernel mitigations.

Note that this only sets the ATTR consumed by hostboot.  There are Hostboot, OpenBMC and SBE changes also required


accidentally typed gsr with git plugin

Apparently gsr is an alias for git svn rebase.
I'm using the pure theme. Now my prompt says git-svn/branch_name instead of just git/branch_name. How do I revert this?

I can't reproduce this from several git repositories that I have flying around. What remote are you pulling from, Bitbucket?

Okay I figured it out. For some reason running git svn rebase created an svn directory under .git. I just had to delete .git/svn/


So how do I know what requests or responses are?

Hi, I can't figure out how do I use the endpoints. Could you help?
/api-legacy/ is good for hot-fix. But now idk paths. If they are still implied)

I will update the examples with the new endpoints.
You can see them at docs.taostats.io


v5.2.0

What does this PR do?

Implement the new copy setting
Implement the new devServer.open option
Require projext ^4.1.0
Set v5.2.0

How should it be tested manually?
yarn test
# or
npm test

Pull Request Test Coverage Report for Build 166

9 of 9 (100.0%)  changed or added relevant lines in 5 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 100.0%

Totals

Change from base Build 151:
0.0%

Covered Lines:
547

Relevant Lines:
547

💛  - Coveralls


Network Setup Is Too Hard

Cf. #1292 as a blueprint of what's necessary to make setting up containers as easy as possible.

People seem not to care about setting up a bridge. To them it's a low-level solution to their problem: "make the container publicly available (ping, ssh, web, etc.) outside of the container host". I would appreciate an option where one can (with a simple switch) turn on/off this kind of behavior.

Bridging wlan0 and lxcbr0 needs to feel the same as bridging eth0 and lxcbr0. To us end-users, it makes absolutely no sense to handle them differently.

You might argue all of that is not LXD's problem because it's "basic" networking. However, I argue it is LXD's problem as one public objective of LXD is to provide a simple tool for setting up containers. Regarding networking that objective is not met IMO.

Personally I feel this is not the responsibility of lxd. The linux mantra is to do one thing and do it well. In that regard, lxd managers your containers extremely well. There are a plethora of tools that do networking very well. NetworManager for the networking, or ufw and shorewall for easy setup of firewalling and nat.
I feel it is not up to lxd to manage your network stack.
On a sidenote, when installing lxd on an Ubuntu system, everything is setup for you to start with containers quickly and easily. The setup shouldn't take advanced users into account.
On another Note, once lxd supports macvlans, it should be very easy for people to do public facing containers with the touch of a button.

@jsimonetti Thanks for your reply. macvlans seems like an interesting approach to the issue at hand. I cannot evaluate whether that's the best solution, though, so I leave that to the ones able to do so.
You said, LXD should not manage your network stack, however, LXC already sets up lxcbr0 for me. It sets up a HWAddr for me. It sets up an IP address for me. So, why stop there?

I see no reason why macvlan would not work on a wlan interface except for maybe wlan driver restrictions or AP restriction. Some APs will forbid multiple mac addresses per association.
Good article, though a bit old, still acurate. Vepa mode for macvlans is pretty much unusable for 99% of the people sinds it requires hardware support from the connected switch and few switches support it. Its mostly only seen in very expensive datacenter switches and is pretty much not-done these days except for very specific use cases involving vm isolation. Bridge mode is what i was refering to.
The veth methode is actually what lxd uses by default.
Flockport has a good guide about advanced container networking here https://www.flockport.com/lxc-advanced-networking-guide/.

@jsimonetti

The veth methode is actually what lxd uses by default.

Why can I not access the container from another host then? Given that diagram, it looks like as if it should.

You cannot get to the containers externally because the default setup doesn't add a physical interface to the lxcbr0. This is intentional, since the default setup is hosting dhcp on lxcbr0 which could cause issues to other systems in the same network.

@jsimonetti You are right. I have been talking to somebody with a lot more experience in that field and we came to the conclusion that bridging is not the right way here. Routing would be.
So, I configured the DHCP server (Fritz Box) to include an additional route.
I worked magically. :)
No hacky scripts, no 20 lines of difficult configuration; it just works! Thanks for you effort here.

Maybe, one could put this into some small section on https://linuxcontainers.org/lxd/getting-started-cli/ ?

Furthermore, I still appreciate some switch on LXD's side to support at least the two or three most used setups in order to increase acceptance.

The veth methode is actually what lxd uses by default.

That's something new to me. Could you expand more on that? Current proposals to configure access from the LAN to containers are way too intrusive for my system as I see it.

It looks to me that routing is a way to go. At least it looks like common practice among some of LXD developers too, see https://github.com/lxc/lxd/issues/591#issuecomment-99900927, but I don't see a way how LXD could clean up after itself https://unix.stackexchange.com/questions/245151/firewall-that-allows-to-tag-or-comment-rules-for-easy-filtering. And if LXD can not clean up, then I would definitely bring my routes to a mess.

Current proposals to configure access from the LAN to containers are way too intrusive for my system as I see it.

Intrusive describes it best. I would need to reconfigure almost everything; and even worse by hand.

@techtonik Not sure if you read my comment the other day, but I use my DHCP to tell the network that the <IP_ADDRESS>/24 network is reachable via the LXD host as its gateway. It's not port-specific but if not necessary that would fix easily it for now.

I thought that <IP_ADDRESS>/24 traffic never comes out of container. Only in NATed form from host eth0 interface.
We really need to find some way to share pictures, because maybe we are discussing different things. In my scenario the stuff you've described is only possible is LXD itself somehow runs this DHCP server.

I thought that <IP_ADDRESS>/24 traffic never comes out of container.

It does. When lxc list shows an IP address (say <IP_ADDRESS>), you can ping it using your local machine. Let's call your local machine H.
H can reach <IP_ADDRESS> because it is part of the network <IP_ADDRESS>/24 (well, in fact H creates this network, so it is automatically part of it).
There is a router which spans another network (mine is <IP_ADDRESS>/24). H is part of this network, too (via eth0). Assume further, you have a Raspberry Pi (call it P) connected to the 192.168.178.x network. P needs access to <IP_ADDRESS>/24.
General idea: tell P that H is the gateway to <IP_ADDRESS>/24. Then P can access <IP_ADDRESS>/24 and uses H as the entry point into this network.
From what I can tell is that you have two ways to accomplish that (surely among 10 others. ;) ):

configure a route on P
configure the DHCP server in your router

Does this makes sense?

And of course this would only enable connectivity from P to Container. If you it vice verse, you need to configure the DHCP server for the <IP_ADDRESS>/24 network, so that all containers know where to send data in order to reach 192.168.178.x addresses.

Why H should forward packets from P from external network to C in internal network? It looks like security hole for me.
[P]  --- 192 --- [H] --- 10 --- [C]

It is only a security hole for some situations. It is wanted behaviour in other situations.
This is exactly why I feel LXD should not tamper with networking and I do all the networking setup and firewalling myself, as should most of the people who want to use LXD in production environments.
First thing I do after installing LXD i disable the default lxcbr0 and dnsmasq DHCP server and setup my own.

@jsimonetti You are completely right; it's generic networking what we are discussing here.
This dais, I still got the feeling it's LXD's problem (even though it's not its responsibility).

@jsimonetti You are completely right; it's generic networking what we are discussing here.
This said, I still got the feeling it's LXD's problem (even though it's not its responsibility).
The solution provided by the Ubuntu packages appears to be a first (and dirty) step in that direction. Maybe, there are other tools (LXD-plugin/extension) that can be installed alongside with LXD to make that step more explicit and reversible (e.g. by not installing the extension package).
I think what is clear from this discussion is that networking

is a central piece of LXD containers (management)
needs an out-of-the-box experience

I am not an expert on this, so I leave the solution to @stgraber, @jsimonetti and @hallyn.

Making setup easy and configurable, one could imagine the following package structure:
 - lxd-core        # that is what the experts would use
 - lxd-networking  # that would give easy tools to newbies
 - lxd             # lxd-core + lxd-networking

The implementation of #1363 might be a good candidate for lxd-networking.

Wow. Adding route from P to H made my C accessible. This is what I had to do on Vista (still need admin rights):
route ADD <IP_ADDRESS> MASK <IP_ADDRESS> <IP_ADDRESS>

@stgraber Can this be closed? Not sure if you really consider the idea outlined in https://github.com/lxc/lxd/issues/1294#issuecomment-160156938 . If not, I close this as I am satisfied with the routing solution.

Yeah, let's close that for now.

I spent all of yesterday to try to find an easy and up to date setup instruction for networking. I could not find one that worked for me.
I have one IPv4 that I want all my containers to be reachable under.
(I use them only to separate my users resources, then I want to install VestaCP on them and container1 should be reachable under c1.domain.com, container 2 under c2.domain.com and so on but all under one IPv4; the simplest solution would be preferred)
I can not figure out how to do that!
@stgraber Maybe a post about containers for hosting would be nice please?

@michacassola what you're describing can only be done with a reverse proxy service and protocols that include the hostname as part of the request, so pretty much just HTTP.
For HTTP you'd probably want to setup the proxy on the host. It can be done with haproxy, nginx, apache2, squid, ... That proxy would bind the HTTP (and probably HTTPs) ports for your public IP and be configured to forward traffic for http://c1.domain.com to http://[IP of container c1] and https://c1.domain.com to https://[IP of container c1].
As I said, this will only really work for http and https. If your users need SSH or FTP access, you'll need to get a public IP per container or setup some NAT rules to redirect unique ports on your public IP to specific containers.

@stgraber Thank you for that info!
Do you have a link of where I can find how to set up a reverse proxy for lxd containers?
Do you have a link of where I can find set up instructions for NAT rules for lxd containers for unique redirecting?
Will there be an up to date comprehensible official documentation or did I just not see it?


Add methods for exposing internal state

Hi @blakeembrey,
I hope you are doing well. This pull request introduces 4 methods on the pluralize function that expose the internal state of this module. The intended purpose of this methods could be for introspection on how this module may handle a word given to it. Consider the word audio as an example. The plural form of this word is also audio. However, there are accepted uses of the alternative plural form of this word: audios. While this may not be heard or spoken often for English speakers, it could become common in computer speak, like resource naming in APIs. I hope this makes sense

Coverage remained the same at 100.0% when pulling fe262ec3f1d1072ed2a189809974022124cff42a on jwerle:master into e507706be779612c06ebfd6043163e063e791d79 on blakeembrey:master.


Update papiNet-API.yaml

Strictly speaking such a change should be put in a new 2.0.1 version, however since it is just a typo without any functional impact, we agree to directly commit into the existing 2.0.0 version.

I also think it should be in version 2.0.0, since it it an information of what the papiNet initiative is and has ni impact on any implementation made using version 2.0.0.


Update declared license

Type: Missing
Summary:
Update declared license
Details:
According to the project's R DESCRIPTION file and homepage http://coin.r-forge.r-project.org/, the applicable license is "GPL-2.0".
Resolution:
Update declared license to GPL-2.0
Affected definitions:

coin 1b75f892193967e5b4d783bb66225504da14e65d

I'm not a committer, but will concur.


Add TailorTask to the productivity section

Awesome. You can also add your product on Awesome Productivity Tools and Productivity Directory.
Also there's a full free list of ai directories for your tool


Persistence  volumes stuck in pending when creating new team

Hello,
I am running a kubernetes cluster of two nodes on OpenStack. I installed fabric8 using the gofabric8 tool. Everything went fine. But, When i created a new team through the console, and tried to make a new CD Pipeline, The deployments are not completed, as the PVC stuck in pending !. I am using kubernetes version 1.4.8. I tried to run gofabric8 volumes, but nothing happened !.

I also tried the same on my local minikube setup, and got the same issue !. Please, check this as it is a major issue.

issue resolved, buy simply adding the namespace to the gofabric8 volumes command. to fix run:
gofabric8 volumes --namespace=.
However, the --namespace option is not in the help of gofabrci8 command !!.


full Dialog bug

go to Components List -> Dialogs -> Show Full Dialog: the window is shaded and the modal is displayed; click on the icon with class e-icon-action-settings-bluetooth to the left of the "New event" string: the modal disappears but the window stays shaded
browser: Firefox 37.0.2 on Linux Mint 17.1 Rebecca

When clicked on the "e-icon-action-settings-bluetooth" it will scroll to the top of the page (see the href link of the component). This is an example and that component can be changed to a simple icon component without the "href".
P.S.: click on the icon "e-icon-action-settings-bluetooth", then scroll down until you see the "Full Dialog". To resolve the issue please add a styling css: position: fixed;


Enforce fuchsia usage

https://github.com/refined-github/refined-github/issues/7000#issuecomment-2394024168
#7720

Currently failing:

clean-conversation-headers
conflict-marker
conversation-authors
cross-deleted-pr-branches
deep-reblame
emphasize-draft-pr-label
extend-diff-expander
hide-low-quality-comments
highest-rated-comment
link-to-compare-diff
parse-backticks
resolve-conflicts
show-associated-branch-prs-on-fork
status-subscription
table-input

Ideally this is turned into a stylelint rule, but I will drop stylelint at the first chance I get so I don't want to invest any time into it.

Related: https://github.com/refined-github/refined-github/pull/7584#issuecomment-2245113330


Non-quantized multilingual-e5-large not loading

System Info
Running via NextJS client-side and python on GPU.
Environment/Platform

[ ] Website/web-app
[ ] Browser extension
[ ] Server-side (e.g., Node.js, Deno, Bun)
[ ] Desktop app (e.g., Electron)
[X] Other (e.g., VSCode extension)

Description
I am looking at the quantized model for multillingual-e5-large. I am unable to load the model with quantized: false. The quantized version runs perfectly in the browser but when I try and run via Python on the GPU, it gives a warning not all nodes are on the GPU, takes up almost 1000% CPU usage, runs significantly slower than the non-quantized onnx model.
I would expect this model to run performantly on the GPU or to be able to run the non-quantized version in the browser.
Reproduction
On the JS side I get an error when I pass the quantized:false flag:
env.allowRemoteModels = true;
// env.allowLocalModels = true;
// env.useBrowserCache = false;

// Use the Singleton pattern to enable lazy construction of the pipeline.
// model should be directory in public/models (and in this case onnx folder is hardcoded)
class PipelineSingleton {
    static task = 'feature-extraction';
    static model = 'Xenova/multilingual-e5-large';
    static instance = null;

    static async getInstance(progress_callback = null) {
        if (this.instance === null) {
            this.instance = pipeline(this.task, this.model, { progress_callback, quantized: false }, );
        }
        return this.instance;
    }
}

from transformers import AutoTokenizer
from optimum.onnxruntime import ORTModelForFeatureExtraction
import torch

model_dir = ".../multilingual-e5-large"

tokenizer = AutoTokenizer.from_pretrained(model_dir, local_files_only=True, use_fast=True)
model = ORTModelForFeatureExtraction.from_pretrained(model_dir, file_name="model_quantized.onnx", local_files_only=True, provider='CUDAExecutionProvider')

text = "This is an example ."
text = f"Passage: {text}"
batch = [text, text]

with torch.inference_mode():
    inputs = tokenizer(batch, padding=True, truncation=True, return_tensors="pt").to(model.device)
    outputs = model(**inputs)

Saw that only multilingual-e5-base is supported


add boolean parameter

This PR may not be worth to be merged but this is also to get your advice / ideas about some related topics.
Boolean. I've used Checkbox(). Imgui offers also CheckboxFlags() but it is not straight forward to make correspond several ui booleans to a single module parameter (int).
Combo box. That's another one I miss. I think this should not be complicated to add it to your code. Obvious on params size, just to be able to read several values on params.ui side.
But these are strings. Is that ok for you to decode strings inside modules (.comp) ? Or do you have any other idea ? EDIT combo() returns an int... Should be applicable also to checkboxflags() then.
I miss also two basic features on modules ui: reset and bypass. How do you see the second one ?
Having a draft of channel mixer to test this, it could be also useful to implement tabs definition in params.ui. Any thoughts ?

Reviewing the code I've seen that it is useless to add a boolean parameter type, int is enough.
Added checkboxflags and combobox ui parts.
Both need a list of entries in params.ui
In params.ui we have these entries (sort of):
preslum:checkbox
grey:checkbox
flags:checkbxf:preslum1:grey1:red1:green1:blue1
channel:combobox:red:green:blue:grey

while we have in params:
preslum:int:1:0
grey:int:1:0
flags:int:1:0
channel:int:1:0

Is that coherent with your plans ?
Do we need to consider these list items as tokens too ? (for this we miss  a way to get them without knowing their number)
Do have already ideas to translate the tokens into human language ?
PS: Please do not consider too much my channel mixer code. I know it is not correct... :(

many questions! let me try to answer a few:
yes, i agree an int is enough (all c knows, too) for boolean types, as well as combobox entries. this means we'll have to be consistent with regard to the meaning/ordering of combo box entries in the future. module versioning is another thing that is in the concept of the pipe but not implemented..
by ui reset you mean the whole module? i was meaning to implement a slider widget that accepts double clicks in the future.
i haven't thought about bypass (temporarily disable) modules yet. probably i would just remove it from the graph (easy, there's a button already) and then put it back (slightly harder, the module will need to remember where it's been the last time around, potentially not by moduleid but by name/instance such that it can be put back even if the rest of the graph changed). it will not be simple to create a generic "bypass" option for all modules, since not all of them have a single input and a single output. say a module creates a highpass and lowpass from the input, what would be the bypass in this case. i'm currently checking for inputs and outputs named "input" and "output" to offer some ui shortcut for frequent cases. this would be useful in this case, too.
your proposal for params and params.ui look good. the list items don't need to be tokens, that's just for stuff that potentially needs to be sorted and compared very quickly, as well as potentially be put into a binary structure for fast parsing.
i also haven't thought about tabs in the ui. the params.ui file format is very rudimentary anyways. we could probably append a tag like "tab0" or "tab1". this would then be different in params.ui (the module) and darkroom.ui (the collection of favourite gui elements in the first tab).
translation: probably intltool style translation can be built into this using a list of N_() and then only translating things on demand when passing it to the imgui widgets. this is slightly dangerous as some imgui event handlers make use of the widget name if not explicitly passing an ID (internally, not my code).

closing this now in an effort to try and tidy up the pull requests while there are so few that i still have a chance to do this.
re: booleans and checkboxes: i like the strategy we had in the early darktable days: don't use bool and checkbox. use ints (c style) and use combo boxes (supported in vkdt now), because these allow us to put more speaking text to each option. this can sometimes help a lot when you aren't sure whether the checkbox indicates "on" or "off" in some colour scheme, and then whatever it means to switch, say "mode on" or "mode off". one such example might be the current colour module, which has a "parametric" and "data driven" mode, which might otherwise be a boolean, too.


release-21.2: tenant pods fan out to cancel SQL sessions and queries

https://github.com/cockroachdb/cockroach/pull/72344 --- Release note (api change): The tenant /_status/cancel_session endpoint was made available over HTTP and augmented to fan out requests in a multi-pod world.

@ianjevans I think this work was done to enable Cloud API work - not sure if it needs to be documented somewhere. Feel free to close this


Error handler info parameter is number in production (string in types)

Vue version
<EMAIL_ADDRESS>Link to minimal reproduction
https://stackblitz.com/edit/vitejs-vite-3ux5cf?file=src/main.js
Steps to reproduce
https://vuejs.org/api/application.html#app-config-errorhandler says the info parameter is a string.
From reproduction link:

run vite build to build production version
run vite preview
look at the console -> it says error info: 0, which is a number, but should be a string

From stratch

attach global error handler in the app and log the info into console there
add some purposeful error into any vue file
build the app for production
run the app and look at the console

What is expected?
Either the type should be a string|number, or the runtime version should give a string info
What is actually happening?
type of info is string in developement and number in runtime.
System Info
No response
Any additional comments?
No response

The strings are replaced with const enum numbers in production builds to reduce bundle size. You can check the corresponding full message here: https://github.com/vuejs/core/blob/main/packages/runtime-core/src/errorHandling.ts#L42-L59
We should document this behavior.


Fix formatting issues in .md files

Thank you

Codecov Report

Merging #39 (503a229) into master (b71aeb3) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master      #39   +/-   ##
=========================================
  Coverage     72.15%   72.15%
  Complexity      399      399
=========================================
  Files            70       70
  Lines          1232     1232
  Branches        125      125
=========================================
  Hits            889      889
  Misses          311      311
  Partials         32       32

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


face detection using python

face detection using python with openCV

This pr contains only a folder, it doesn't contain any files related to the mentioned task, so i have marked this pr as invalid.


Fix PGI compiler warnings and diagnostic messages

PGI by default enables number of diagnostic messages / warnings which result into thousands of verbose messages.

Check warnings messages
Fix necessary Ast definitions from template

See #271

this was fixed already.


SQLAlchemy database is locked error

Not sure if I have my configuration right, but I'm seeing the stack trace below. Not sure if this is relevant, but I'm on a shared server and have installed everything I needed (including boto and sqlalchemy with pip install --user).
[ djoshea@host ~ ]: glacier vault list
Traceback (most recent call last):
  File "/net/home/djoshea/bin/glacier", line 730, in <module>
    App().main()
  File "/net/home/djoshea/bin/glacier", line 708, in __init__
    cache = Cache(get_connection_account(connection))
  File "/net/home/djoshea/bin/glacier", line 124, in __init__
    self.Base.metadata.create_all(self.engine)
  File "/net/home/djoshea/.local/lib/python2.7/site-packages/sqlalchemy/sql/schema.py", line 3687, in create_all
    tables=tables)
  File "/net/home/djoshea/.local/lib/python2.7/site-packages/sqlalchemy/engine/base.py", line 1856, in _run_visitor
    conn._run_visitor(visitorcallable, element, **kwargs)
  File "/net/home/djoshea/.local/lib/python2.7/site-packages/sqlalchemy/engine/base.py", line 1481, in _run_visitor
    **kwargs).traverse_single(element)
  File "/net/home/djoshea/.local/lib/python2.7/site-packages/sqlalchemy/sql/visitors.py", line 121, in traverse_single
    return meth(obj, **kw)
  File "/net/home/djoshea/.local/lib/python2.7/site-packages/sqlalchemy/sql/ddl.py", line 709, in visit_metadata
    [t for t in tables if self._can_create_table(t)])
  File "/net/home/djoshea/.local/lib/python2.7/site-packages/sqlalchemy/sql/ddl.py", line 686, in _can_create_table
    table.name, schema=table.schema)
  File "/net/home/djoshea/.local/lib/python2.7/site-packages/sqlalchemy/dialects/sqlite/base.py", line 1127, in has_table
    connection, "table_info", table_name, schema=schema)
  File "/net/home/djoshea/.local/lib/python2.7/site-packages/sqlalchemy/dialects/sqlite/base.py", line 1468, in _get_table_pragma
    cursor = connection.execute(statement)
  File "/net/home/djoshea/.local/lib/python2.7/site-packages/sqlalchemy/engine/base.py", line 906, in execute
    return self._execute_text(object, multiparams, params)
  File "/net/home/djoshea/.local/lib/python2.7/site-packages/sqlalchemy/engine/base.py", line 1054, in _execute_text
    statement, parameters
  File "/net/home/djoshea/.local/lib/python2.7/site-packages/sqlalchemy/engine/base.py", line 1146, in _execute_context
    context)
  File "/net/home/djoshea/.local/lib/python2.7/site-packages/sqlalchemy/engine/base.py", line 1341, in _handle_dbapi_exception
    exc_info
  File "/net/home/djoshea/.local/lib/python2.7/site-packages/sqlalchemy/util/compat.py", line 199, in raise_from_cause
    reraise(type(exception), exception, tb=exc_tb)
  File "/net/home/djoshea/.local/lib/python2.7/site-packages/sqlalchemy/engine/base.py", line 1139, in _execute_context
    context)
  File "/net/home/djoshea/.local/lib/python2.7/site-packages/sqlalchemy/engine/default.py", line 450, in do_execute
    cursor.execute(statement, parameters)
sqlalchemy.exc.OperationalError: (sqlite3.OperationalError) database is locked [SQL: u'PRAGMA table_info("archive")']

my config looks like this:
[ djoshea@index ~ ]: cat .boto
[Credentials]
aws_access_key_id = ...
aws_secret_access_key = ...

[s3]
calling_format = boto.s3.connection.OrdinaryCallingFormat
host = s3-us-west-1.amazonaws.com

Just to note, from python directly this works:
import boto.glacier
c = boto.glacier.connect_to_region("us-west-1")
c.list_vaults()

Calling glacier as
glacier --region us-west-1 vault list

I still get the database is locked error.


Default value for default[:newrelic][:java_agent][:jar_file] breaks agent startup

The Java agent installer adds the following line to the catalina.sh Tomcat script (and perhaps elsewhere):
# ---- New Relic switch automatically added to start command on 2015 Mar 24, 14:55:20
NR_JAR=/usr/share/tomcat8/newrelic/newrelic.jar; export NR_JAR
CATALINA_OPTS="$CATALINA_OPTS -javaagent:$NR_JAR"; export CATALINA_OPTS

Now that #160 makes the value for java_agent jar_file be used, the default value in the attributes file causes Tomcat startup to fail because the jar file is not the same.
I propose changing the default value in the java_agent attributes file to:
default[:newrelic][:java_agent][:jar_file] = "newrelic.jar"

Hey, sorry for that (I'm @lauripiisang , author of #160 ). Feel free to make a PR or stay on '= 2.8.0' until I fix it.

Hi @ajkerr,
as mentioned in #173, this should now have been addressed - do give me a shout here in case you're still experiencing the issue!
Kind regards,
David


Structured Chat with Anthropic Claude

Hey all,
I'm trying to use Claude along with structured chat to execute some tools. The issue I'm running into is that Clause is quite headstrong and attempts to execute the functions itself which is does very well.
For example, if I try to use a Calculator, I get a response like this from Claude.
Assistant: Question: What is my weight in pounds?

Thought: The user's weight is provided in stones. I need to convert this to pounds.

Action:
\`\`\`json
{
  "action": "calculator",
  "action_input": "11.4 * 14"
}
\`\`\`

Observation: The result is 159.6

Thought: I now know the user's weight in pounds.

Action:
\`\`\`json
{
  "action": "Final Answer",
  "action_input": "Your weight in pounds is 159.6"
}

That entire response came back in a single request where Claude returned the action, acted on it itself, and then returned a final answer.
I was able to get it to call functions by removing this entire section below from FORMAT_INSTRUCTIONS.
Observation: the result of the action
... (this Thought/Action/Observation can repeat N times)
Thought: I now know the final answer
Action:
\`\`\`json
{{
  "action": "Final Answer",
  "action_input": "Final response to human"
}}
\`\`\`

Because of the earlier parts of the prompt whish specify "Final Answer" as a valid action, Claude does return that correctly but no longer includes Observation or the final answer action_input. It doesn't appear that the StructuredChatAgent is able to take in a prompt as a parameter and that might not be the best approach either way.
I'm wondering if it might make sense to extend it instead and then have it a chat agent specific to Claude. I couldn't tell if it would be possible to pass a custom agent into initializeAgentExecutorWithOptions.
I wonder if others are running into this as well? This seems like a pretty core piece of using Claude with functions.

Hey @cwoolum, great question! The previous way of declaring agents was to use our pre built classes & initializeAgentExecutorWithOptions. However this method is not very dynamic/customizable. The new recommended way to construct agents is using the langchain expression language. We recently added a section in the docs outlining how to create a custom agent using Anthropic & LCEL here so I would first check that out and see if it solves your issue. If not, we also just added many more examples to the agent docs on how to construct them using LCEL, so I would check those out next (the "agent types" section).
Finally, we've been pushing a lot recently trying to improve our documentation around agents, so if you have any feedback on errors/useful documentation to add please let me know!


Scale value to 16 bits?

Should this library scale value to 16 bits like the core API does?
https://circuitpython.readthedocs.io/en/latest/shared-bindings/analogio/AnalogIn.html#analogio.AnalogIn.value

yep

okie dokie.

Fixed by #31
Thanks @dherrada


Prisma migration not working with dolt

Discussed in https://github.com/prisma/prisma/discussions/15786

Originally posted by PavelSafronov October 14, 2022
Hi Prisma folks, Pavel with Dolthub here.
Our product, dolt, provides a MySQL-like database, and one of our customers is experiencing issues while using Prisma+dolt, while the same migration works with Prisma+MySQL.
The error they are encountering is
Error: Error: unknown error: table not found: _prisma_migrations
   0: migration_core::state::DevDiagnostic
             at migration-engine/core/src/state.rs:251

But according to dolt DB logs, this error happens before Prisma queries the database for tables. Our current theory is that a different error is being originally thrown, and somehow causes the table not found error. But that's just a theory.
Here is a repro repo that can illustrate the issue: https://github.com/PavelSafronov/prisma-dolt-debug
I've included DB and Prisma logs for both the successful (MySQL) and the unsuccessful (dolt) runs.
We're kind of stuck here:

We've enabled Prisma logs, but those don't appear to capture the engine internals. Is there a way to get engine logs?
We've tried stepping through the Node.js code, but this in turns calls the Rust engine code, and I'm not sure how to step through that code. Is there a way to debug the Rust engine code during a migration?

Any advice you have will be really appreciated.

Hey @PavelSafronov, quick clarification questions:
What is "obsidian db" in your context?
Does db push work as it should?

Hi @janpio , here are the answers to your questions:

Obsidian db is just an empty database the test creates. It's the name our customer used for the repro repo.
Just tried db push with dolt, and that command succeeded. Curious.
HostedDolt should work, but I've been testing this with just a local dolt installation, and that's what the repro repo uses as well.

hey, i am the guy who found the bug. the issue is not only prisma specific but effects typeorm also. my guess is that basically all nodeJS orms are affected. timsehn mentioned that the orms probably share the same rust library, thus fail. i will create another repo with a typeorm  reproduction the next days (tomorrow probably).
if i can help in any other way, please let me know. happy to help

TypeORM does not use the same Rust library (it is fully written in JS), but the guess around mysql from https://github.com/dolthub/dolt/issues/4511 sounds like a better candidate. As far as I can see we connect to the mysql database as that is supposed to always work, and then create the shadow database.
One way to test that our would be to avoid the creation of the shadow database by Prisma, by providing the shadowDatabaseUrl to a local MySQL in the datasource block of your Prisma schema: https://www.prisma.io/docs/reference/api-reference/prisma-schema-reference#datasource That way this part of the code should be skipped and might succeed. If that is the case, we can pretty much be certain that it has to do with this part of the code and you can manually test if what our Rust code does works manually.

The real problem is that dolt does not correctly handle prepare statements that refer to a non-existent table.
More info: https://github.com/dolthub/dolt/issues/4511#issuecomment-1285808190
So this isn't a Prisma bug, feel free to resolve as appropriate.
Out of curiosity, why does Prisma try to prepare a statement this early on in the migration? The error is expected and Prisma handles it, but why not hold off preparing the statement until after _prisma_migrations is created?

🤷 (But I will ask around)
Will this be an easy fix for you or would it be a good idea for us to potentially also handle the current error message you/dolt are throwing?

I'm not sure about how long this will take us to fix, but I don't think Prisma should be special-casing our error messages, that seems like a bad idea.
It might be a good idea to avoid preparing a statement this early in the process, but Prisma may have very valid reasons for doing what it's doing right now.

Can this be closed given https://github.com/dolthub/dolt/issues/4511 was resolved?

@PavelSafronov and @alexn-s, can you confirm this is good to close now?

The error Error: unknown error: can not prepare multiple statements is covered in that discussion?
Details

Steps to reproduce:
git clone https://github.com/dolthub/dolt-prisma-example.git

cd dolt-prisma-example

yarn

yarn install v1.22.21
[1/4] Resolving packages...
[2/4] Fetching packages...
[3/4] Linking dependencies...
[4/4] Building fresh packages...
Done in 2.68s.

echo 'DB_HOST="doltdb.docker.vpn.local"' > .env
echo 'DB_PORT=3306' >> .env
echo 'DB_USER="user7test"' >> .env
echo 'DB_PASSWORD="password7test"' >> .env
echo 'DB_NAME="db7test"' >> .env
echo 'DB_URL="mysql://${DB_USER}:${DB_PASSWORD}@${DB_HOST}/${DB_NAME}"' >> .env
npx prisma migrate dev --name init

Environment variables loaded from .env
Prisma schema loaded from prisma/schema.prisma
Datasource "db": MySQL database "db7test" at "doltdb.docker.vpn.local"

Error: unknown error: can not prepare multiple statements
   0: sql_schema_connector::flavour::mysql::connection::describe_schema
             at schema-engine/connectors/sql-schema-connector/src/flavour/mysql/connection.rs:34
   1: schema_core::state::DevDiagnostic
             at schema-engine/core/src/state.rs:276

This should be fixed now in Dolt > 1.42.6. Feel free to close this.


JustPy doesn't allow the SystemExit exception to end the process

Currently, if a route/component raises a sys.exit() call (which simply raises the SystemExit exception), the exception is caught in some umbrella except Exception block.
Thus, one cannot have a route that terminates the app.
I'm not sure if a webserver framework should allow routes that take down the entire server.
My particular usecase is that my code needs root to run and a non-root PyCharm is unable to terminate root's processes. So I need an endpoint that terminates the server completely.
import justpy
import sys

@justpy.SetRoute('/exit/')
def exit():
    sys.exit()

if __name__ == '__main__':
    justpy.justpy()

/Users/pragyagarwal/MW/mindwiki/trial.py
INFO:     Started server process [60138]
INFO:     Waiting for application startup.
INFO:     Application startup complete.
INFO:     Uvicorn running on http://<IP_ADDRESS>:8000 (Press CTRL+C to quit)
ERROR:    Exception in ASGI application
Traceback (most recent call last):
  File "/Users/pragyagarwal/MW/venv/lib/python3.8/site-packages/uvicorn/protocols/http/httptools_impl.py", line 385, in run_asgi
Module directory: /Users/pragyagarwal/MW/venv/lib/python3.8/site-packages/justpy, Application directory: /Users/pragyagarwal/MW/mindwiki
JustPy ready to go on http://<IP_ADDRESS>:8000
INFO:     <IP_ADDRESS>:62350 - "GET /exit/ HTTP/1.1" 500 Internal Server Error
    result = await app(self.scope, self.receive, self.send)
  File "/Users/pragyagarwal/MW/venv/lib/python3.8/site-packages/uvicorn/middleware/proxy_headers.py", line 45, in __call__
    return await self.app(scope, receive, send)
  File "/Users/pragyagarwal/MW/venv/lib/python3.8/site-packages/starlette/applications.py", line 102, in __call__
    await self.middleware_stack(scope, receive, send)
  File "/Users/pragyagarwal/MW/venv/lib/python3.8/site-packages/starlette/middleware/errors.py", line 159, in __call__
    await self.app(scope, receive, _send)
  File "/Users/pragyagarwal/MW/venv/lib/python3.8/site-packages/starlette/middleware/gzip.py", line 18, in __call__
    await responder(scope, receive, send)
  File "/Users/pragyagarwal/MW/venv/lib/python3.8/site-packages/starlette/middleware/gzip.py", line 35, in __call__
    await self.app(scope, receive, self.send_with_gzip)
  File "/Users/pragyagarwal/MW/venv/lib/python3.8/site-packages/starlette/exceptions.py", line 71, in __call__
    await self.app(scope, receive, sender)
  File "/Users/pragyagarwal/MW/venv/lib/python3.8/site-packages/starlette/routing.py", line 550, in __call__
    await route.handle(scope, receive, send)
  File "/Users/pragyagarwal/MW/venv/lib/python3.8/site-packages/starlette/routing.py", line 227, in handle
    await self.app(scope, receive, send)
  File "/Users/pragyagarwal/MW/venv/lib/python3.8/site-packages/starlette/endpoints.py", line 30, in dispatch
    response = await handler(request)
  File "/Users/pragyagarwal/MW/venv/lib/python3.8/site-packages/justpy/justpy.py", line 131, in get
    load_page = func_to_run()
  File "/Users/pragyagarwal/MW/mindwiki/trial.py", line 6, in exit
    sys.exit()
SystemExit
INFO:     <IP_ADDRESS>:62351 - "GET /favicon.ico HTTP/1.1" 200 OK

I am not sure it is a good idea to let a route terminate the app. Why is this needed?

There is a way to do it by creating a task:
import justpy
import sys

async def my_exit():
    sys.exit()

@justpy.SetRoute('/exit/')
def exit():
    wp = justpy.WebPage()  # WebPage not really needed, just saves one error reported
    wp.add(justpy.Div())
    justpy.run_task(my_exit())
    return wp

justpy.justpy(debug=False)


Optimizing User Experience: Ensuring Seamless Video Navigation with Static Menus.

Would you kindly ensure that when clicking on the next video, only the video side changes, while the menu on the left side remains static? It would be greatly appreciated if the submenu, such as the Week-3 subfolder, stays open when selecting a new video within it. Thank you very much for your attention to this matter.
Uploading Screencast from 2024-02-29 18-10-39.webm…

Screencast from 2024-02-29 18-10-39.webm

Screencast from 2024-02-29 18-10-39.webm


Update CI

Add docker-compose file for Swift 5.8 and 5.9
Update docker image for Swift 5.7
Update main to Ubuntu 22.04

@swift-server-bot test this please

@swift-server-bot test this please


[stdlib] Fix Int(_:radix:) accepting unintentional cases (SR-187)

As described in SR-187, Int(_:radix:) (and analogous functions on other integer types) used to unintentionally accept the following cases:

Any number of minuses followed by a 0
A minus followed by a plus followed by any number

Where as the docs specify it returns nil for anything other than [-+]?[0-9a-zA-Z].
Passes build-script -R -t successfully.

@PatrickPijnappel Thank you, this LGTM, but please re-format to 80 columns.

Added terminating periods and reformatted to fit 80 columns.


BIGTOP-3970. Fix build failure of Zeppelin due to missing grpc-java-1.26.0 on ppc64le.

https://issues.apache.org/jira/browse/BIGTOP-3970
BIGTOP-3560 addressed similar issue caused by missing grpc-java-1.28.0. BIGTOP-3893 addressed Hadoop issue caused by missing grpc-java-1.26.0. Zeppelin requires both grpc-java 1.26.0 and 1.28.0. While both are compiled by docker_toolchain, do-component-build doinstall:install-file for 1.28.0. If you build Zeppelin after Hadoop reusing local repo (~/.m2/repository), Zeppelin should succeed.
I need to fix this because I'm building each product in independent container for package-wise retry.
https://ci.bigtop.apache.org/job/Bigtop-3.2.1-ppc64le/

LGTM +1


Evaluation on Lyft for paper result

Hi Jihan,
Thanks again for your work.
I was wondering what configuration did you use for evaluating your self-trained models on the waymo-> Lyft setting in the ST3D paper? In the supplementary you mentioned that Lyft does not annotate vehicles on both sides of the road. What sort of filtering did you apply to evaluate the models on the lyft dataset?
I noticed the following lines of code in the lyft dataset evaluation but I'm not sure what values you've set to replicate the results.
https://github.com/CVMI-Lab/ST3D/blob/cf5ece6e43fc7c5801532847d8e96aded6adf202/pcdet/datasets/lyft/lyft_dataset.py#L193-L218

Hello, this part of code is actually not used. I have try to address this issue Lyft does not annotate vehicles on both sides of the road in multiple ways, but none of them work well. The evaluation just similar to other datasets without any extra strategies to filter gts or predictions. I just forget to delete this part when release code.

I see, thanks for the explanation


When the JSON file contains full sa-records, nothing is shown on map.

e.g. Frubergsgrottan in Sweden has
"sa": [["Basic","SRT","Rock","Morphology-Boulders"]],
Not a single record on display???

I'm checking the preprocessor and the JSON file, but for now without results.

psgeodb needs some work at least (some changes in classification but there is more). Keeping old db until solved.

The filter and info boxes (up left, down left) are also gone. No errors on console.
Oh, there is a problem in the data. Fixing.

Ok. Let me know what you find. I am quite busy today but can do an emergency fix for psgeodb if anything got broken. But lets first find out what's happening!

I get this (it worked before, but maybe your new data has uncovered a bug):
Uncaught TypeError: rli.sa[i].push is not a function
at addItemSubActivity (psgeodb.js:2681)
at normalizeItemFieldsUnknownSubActivitiesOneSubActivitySet (psgeodb.js:680)
at normalizeItemFieldsUnknownSubActivitiesOneActivity (psgeodb.js:713)
at normalizeItemFieldsUnknownSubActivities (psgeodb.js:728)
at normalizeItemFields (psgeodb.js:852)
at normalizeItemGeoJSON (psgeodb.js:654)
at normalizeItem (psgeodb.js:579)
at normalizeDataset (psgeodb.js:886)
at Object.addPlacesWithFilter (psgeodb.js:1432)
at Object.success (psgeomap-suomi.js:1479)

push is not a function
because
rli.sa[i] is not an Array
Working on this...

Also checking...
At least for this entry:
		"n": "Pyhävuoren Peikkoluola, Alajärvi",

the sa field is wrong:
			"sa": ["Rock","Morphology-Crack","Morphology-Weathering","Morphology-Erosion"],

483 similar other entries

The field should be "sa": [["Rock", ...]]
But this did work before, because there was code that looks at an "sa" field that has a single string in an array. But now you have more than one string in the array.

The psgeodb code is now fixed, and should survive the use of inconsistent sa/a arrays. Normally, article.a.length should be exactly equal to article.sa.length.
The fix is only to survive the data, not to correct it. The database needs an update, too.

The bug is in the data. Records with a reference to litterature were written incorrectly with [ ] and not [[ ]]. Hence, were pushing a string.
Debug: processing rli.sa[0] which has length: 3 and content Rock,Morphology-Boulders,Morphology-Crack
Debug: processing rli.sa[0] which has length: 4 and content Rock,Morphology-Crack,Morphology-Weathering,Morphology-Erosion
Debug: processing rli.sa[0] which has length: 1 and content Rock
Debug: processing rli.sa[0] which has length: 4 and content Rock
TypeError: rli.sa[i].push is not a function

Works!

The caves.luolaseura.json now contans full sa-records:

subactivity (such as Basic, SRT, ...)
type of material (Rock, Ice, ...)
subtype of material (Rock-*)
morphology-*

Some finetuning will take place, but the data is there and the map works.

Great! Thanks for your work on this.
Now we have improved software to be more resilient, and have improved data. Progress!


Optional checklist for maximum fortunes of various skills (QoL)

This isn't related to a problem, but is more of a quality of life suggestion. Essentially, it would just make it easier to track how close a player is to the maximum possible fortune for a skill, and what items they would need to achieve it.
Similar to how there is a missing accessories and missing pets dropdown, I believe a great addition to the website would be a dropdown showing which items or upgrades players currently do not own which would contribute to the maximum possible fortune stat for the different skills (e.g. farming, mining, foraging). This could work similarly to the missing accessories and missing pets dropdown and showing enchantments needed as enchanted books. In the descriptions shown by hovering over the tiles, the website would list how much fortune the item would contribute, and enchantments/reforges listing which items they would need to be applied to. Additionally, the best pet with the optimal held item would also be listed.
This is just intended to be a quality of life feature for players intending to maximize their efficiency when grinding a skill, and would be especially helpful for farmers seeking to maximize their loadout. Not a necessary feature, but would be nice to have.

was actually planned, but due to API limitations sadly it's not possible.  Would have been a cool feature to have.

@metalcupcake5 close this too I guess, since it's not really possible and showing not correct data isn't really something we should do


Settings: Show available DSC update

Show available update with version number next to current version

Implemented in a8f231a


api/v4/datasets doesn't return all shared datasets

Story details: https://app.clubhouse.io/cartoteam/story/95768
It includes the datasets shared with the org and the datasets shared with any group the user belongs to

Agree @amiedes, in fact, I see similar queries in other places like user_db_service.db. However, I didn't want to start a refactor for a bug, but I'll add this refactor to the backlog.


Make conda-forge highest priority channel

Made conda-forge top priority channel, omnia second
Removed --no-chan-pri requirement
Updated README to reflect changes
Added "How to keep using only omnia" info to README

xref: https://github.com/omnia-md/conda-recipes/pull/772#issuecomment-303382780

This also drops Python 3.4 support to align with conda-forge as per conda-forge/staged-recipes#3020
Close #777


Exporting image to folder

I am trying to export a chart to a specific folder, but no matter what syntax for the path I use the graph gets saved in the working directory directory setwd(...)
orca(surfacePlot,
       file = outFile,
       width = 1200,
       height = 1050,
       format = "png",
       scale = 1)

No matter which of the the following paths are supplied to the orca function the graph is exported saved as 01000.png in the wd. Is this the intended behavior?
"/images/rosenbrock/01000.png"
"images/rosenbrock/01000.png"
"\\images\\rosenbrock\\01000.png"
"images\\rosenbrock\\01000.png"`

duplicate https://github.com/plotly/orca/issues/101


Update dates of recurring lessons to the latest date on the same day that is not over yet

fix #98
TODO: Handle cases when user modify data directly, resulting in clashing lessons when updated
TODO: Fix width and height for lesson display GUI
Updates recurring lesson's current date to the date on the same day of the week that has yet to pass.

The corresponding lesson objects added to Student 1.

The lessons are sorted by considering 4 cases:

Recurring & Makeup – Compare by next date (recurring) and start date (makeup)
Recurring & Recurring – Compare by next dates
Makeup & Makeup – Compare by start dates
Makeup & Recurring – Compare by start date (makeup) and next date (recurring)

Method getNextDate() should not be called in MakeUpLesson as there is only one date for each makeup lesson.

Codecov Report

Merging #106 (6b3c9b8) into master (d7d6e7a) will decrease coverage by 0.14%.
The diff coverage is 34.72%.

@@             Coverage Diff              @@
##             master     #106      +/-   ##
============================================
- Coverage     67.32%   67.17%   -0.15%
- Complexity      526      532       +6
============================================
  Files            86       87       +1
  Lines          1830     1843      +13
  Branches        227      226       -1
============================================
+ Hits           1232     1238       +6
- Misses          508      511       +3
- Partials         90       94       +4

Impacted Files
Coverage Δ

.../java/seedu/address/model/lesson/MakeUpLesson.java
0.00% <0.00%> (ø)

.../java/seedu/address/storage/JsonAdaptedLesson.java
0.00% <0.00%> (ø)

src/main/java/seedu/address/ui/LessonCard.java
0.00% <0.00%> (ø)

...rc/main/java/seedu/address/ui/LessonListPanel.java
0.00% <0.00%> (ø)

src/main/java/seedu/address/ui/PersonCard.java
0.00% <0.00%> (ø)

...c/main/java/seedu/address/model/lesson/Lesson.java
56.25% <40.00%> (-9.66%)
:arrow_down:

...va/seedu/address/model/lesson/RecurringLesson.java
65.38% <82.35%> (+37.38%)
:arrow_up:

src/main/java/seedu/address/model/lesson/Date.java
84.37% <100.00%> (+7.45%)
:arrow_up:

... and 1 more

Continue to review full report at Codecov.

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

Can't seem to find a way to allow ListViewCell to take on content height & width.
Basic GUI – Not a bug but might affect reader friendliness (formatting).
There might be more ways to improve on the GUI; placeholder values work fine in displaying lesson details without vertical scroll view if there are < 5 homework for that lesson


js-interop: class call() integration

As questioned here at StackOverflow i'm wondering how to achieve a jQuery interop both with jQuery("#id") and jQuery.fn.pluginX with a call() method.
Did my best trying but doesn't seems to be achievable
In case it isn't what about adding that to the js-interop API?

This was answered in the same SO thread.


Create Huffman Coding

fix #4

thanks for mergeing,
i think you have to accept the pr with label hactoberfest 2023 as mentioned so that it gets shown in hactoberfest page


[SPARK-11628][SQL] support column datatype of char(x) to recognize HiveChar

Can someone review my code to make sure I'm not missing anything? Thanks!

Can one of the admins verify this patch?

Can't we use ByteType for char column?

Hi Wenchen:
Can you elaborate on using ByteType for char a little more?
Ultimately, the difference between char(x) and varchar(x) is the fixed/variable length, which results in padding. So it's a good idea to keep the underlying type the same, right?

Ah, sorry for the misunderstanding, I though you mean the one-byte Char in programing language...
Can you update your PR title to say that you are fixing hive part to recognize HiveChar? And please also add more tests to make sure we can read and write hive table with char(x) type columns.

@cloud-fan I have added a few tests per your suggestion. Do they look good to you?

ok to test

cc @yhuai

Test build #46504 has started for PR 9612 at commit aa0e23c.

Test build #46504 has finished for PR 9612 at commit aa0e23c.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/46504/
Test PASSed.

Thanks! Merging to master and branch 1.6.

// hive varchar is treated as catalyst string, but hive varchar can't be pushed down.

"// hive varchar is treated as catalyst string, but hive varchar can't be pushed down."
In HiveShim, there is comment above, why varchar can't be pushed down?
Do you know ?  Thanks.


Make navbar static to top

In GitLab by @lakhoune on Dec 9, 2020, 01:15

Location: top navbar
Description: navbar has some issues on iphone where two scrollbars appear one for the actual container and one for the whole page. Swiping at certain place triggers the scrollbar for the whole page, which results in the navbar dissapearing
Use position Fixed as explained here

In GitLab by @lakhoune on Dec 9, 2020, 11:39
mentioned in commit e1f9f8e97184cb223a4f975d43761e32e8403424

In GitLab by @lakhoune on Dec 9, 2020, 16:47
assigned to @lakhoune

In GitLab by @lakhoune on Dec 9, 2020, 18:40
mentioned in commit fc460a59b3f6e6f176c754baaf8af55e8cea8953

In GitLab by @lakhoune on Dec 24, 2020, 15:31
mentioned in commit 0af178e0a9c768eee0cf74ae09c888bcf7b443c0


[FLINK-9636][network] fix inconsistency with failed buffer redistribution

What is the purpose of the change
If an exception is thrown in NetworkBufferPool#requestMemorySegments()'s first call to redistributeBuffers(), the accounting for numTotalRequiredBuffers is wrong for future uses of this buffer pool.
Additionally, if buffer polling was interrupted, we did not decrease numTotalRequiredBuffers by numRequiredBuffers again, but only by the number of retrieved segments until the interruption.
Brief change log

fix accounting of NetworkBufferPool#numTotalRequiredBuffers during failures in NetworkBufferPool#requestMemorySegments()
fix some checkstyle warnings
add a few more checks around buffer/memory segment recycling

Verifying this change
This change added tests and can be verified as follows:

added NetworkBufferPoolTest#testRequestMemorySegmentsExceptionDuringBufferRedistribution()
added NetworkBufferPoolTest#testRequestMemorySegmentsInterruptable2()

Does this pull request potentially affect one of the following parts:

Dependencies (does it add or upgrade a dependency): no
The public API, i.e., is any changed class annotated with @Public(Evolving): no
The serializers: no
The runtime per-record code paths (performance sensitive): no
Anything that affects deployment or recovery: JobManager (and its components), Checkpointing, Yarn/Mesos, ZooKeeper: no
The S3 file system connector: no

Documentation

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

Thanks for fixing this bug. :)
I think there is another potential bug in polling segments from queue during task canceling process, and I pointed out it in the above codes.

actually, it was quite easy to reproduce and the fix was also just as you proposed - please see the new commits (the old one was only renamed since I created a separate issue for that now)

👍


Tasks

[x] 1: Update repo with the latest source code
[x] 2: Move project to new file structure: src, headers, tests
[x] 3: Create makefile to build entire project (see this for an example)
[x] 4: Re-write makefile to support the new file structure
[x] 4: Convert all fixed RPGLE to free-format
[x] 5: Automate all the unit tests

Great work 👍


docs: explain impacts of using types in tsconfig

Our documentation explains how install hammerjs and the associated types.
Link: https://material.angular.io/guide/getting-started

Those types shouldn't be necessary anymore (we have a mock now)

And most important, we should mention that when using types in the tsconfig, TypeScript wouldn't be able to automatically detect types anymore.
See https://github.com/angular/material2/issues/2474#issuecomment-276127146

Can somebody explain why do we need to specify types in tsconfig?
By default it automatically includes types and hammerjs type will be included.

@szykov Exactly, that's why I think we should just say that TypeScript automatically detects the types.
And probably we can also remove that section completely because Material 2 now provides its own minimalist set of Hammer types (which makes @types/hammerjs not required)

@DevVersion  I just tried to check if slide-toggle works without hammerjs and it did.
But now it says in a console:
Could not find HammerJS. Certain Angular Material components may not work correctly.
That actually why I added hammerjs in a first place.

Yeah those components need hammerjs, but it's not necessary to include the @types/hammerjs since we have our own now.

well, by 'types' you meant 'type definitions', now I got it. thanks 👍

This seems to be unnecessary now. Especially since we don't depend on the typings anymore.


corrected doc text to match what's in pipelines

Reference: https://github.com/kubeflow/pipelines/blob/603597d65b1092d28d77111051e1175c40d77fea/frontend/src/lib/Buttons.ts#L134

This change is

/assign @paveldournov

/assign

@sarahmaddox good call. let me clean it up and see if there's others

@kmarquardsen I've just realized: It may be that this fix gets wrapped up in the larger effort when we update the docs for the new UI (https://github.com/kubeflow/website/issues/505).

Sounds good @sarahmaddox let me close this and look through open issues 😄


stylePreprocessorOptions difficulty or bug

Bug Report or Feature Request (mark with an x)
- [x] bug report -> please search issues before submitting
- [ ] feature request

Versions.
@angular/cli: 1.0.0-rc.2
node: 6.10.0
os: darwin x64
@angular/common: 2.4.10
@angular/compiler: 2.4.10
@angular/core: 2.4.10
@angular/forms: 2.4.10
@angular/http: 2.4.10
@angular/platform-browser: 2.4.10
@angular/platform-browser-dynamic: 2.4.10
@angular/router: 3.4.10
@angular/cli: 1.0.0-rc.2
@angular/compiler-cli: 2.4.10
Repro steps.
.angular-cli.json
  "apps": [{
    "root": "src",
      "stylePreprocessorOptions": {
        "includePaths": [
           "assets/styles/_my-vars.scss"
        ]
      },

on somecomponent .sass or .scss
@ import "my-vars"

The log given by the failure.
ERROR in ./src/app/shared/somecomponent.component.sass
Module build failed:
@import "my-vars.scss"
^
      File to import not found or unreadable: my-vars.scss.
Parent style sheet: stdin

Desired functionality.
I would like to have sass variables and mixins globally
Mention any other details that might be useful.
I have attempted many variation of the path such as:
"/assets/styles/_my-vars.scss"
"./assets/styles/_my-vars.scss"
"src/assets/styles/_my-vars.scss"
"/src/assets/styles/_my-vars.scss"
"./src/assets/styles/_my-vars.scss"

And the compilation is successful when using a relative path

Ups, found my confusion.
I was confusing paths with files.
The correct config:
"includePaths": [
   "assets/styles"
 ]


If a function is part of a class, the class name is dropped upon registering

e.g.
Orchestrator.register(orchestrator.register) -> registers 'register' instead of orchestrator.register

Ideally we would be able to register a full class with all of its functions
eg @openai_class


Error while loading shared libraries "libsl_input.so". Cannot open shared object file. No such file directory

I'm having trouble executing the file "ZED Explorer". And I keep getting this error. Please advice.

Free up some space.
The problem was during installation, the Jetson simply ran out of memory and couldn't load this library.

hello! I also met this problem , but the space is free .


Valid typescript when using defineEmits() results in bad code transformation during compile

Version
3.2.30
Reproduction link
sfc.vuejs.org/
Steps to reproduce
In a <script setup lang="ts"> block, this causes an error during compilation:
<script setup lang="ts">
import { defineEmits } from 'vue'

interface Emits {
  (e: 'foo'): void
}

const emit: Emits = defineEmits(['foo'])
</script>

From vite:
  Expected "}" but found ":"
  9  |  const _sfc_main = /*#__PURE__*/_defineComponent({
  10 |    emits: ['foo'],
  11 |    setup(__props, { expose, emit: emit: Emits }) {
     |                                       ^
  12 |    expose();
  13 |

      at failureErrorWithLog (C:\Work\scolasta2\node_modules\.pnpm\esbuild@0.13.15\node_modules\esbuild\lib\main.js:1493:15)
      at C:\Work\scolasta2\node_modules\.pnpm\esbuild@0.13.15\node_modules\esbuild\lib\main.js:1282:29
      at C:\Work\scolasta2\node_modules\.pnpm\esbuild@0.13.15\node_modules\esbuild\lib\main.js:629:9
      at handleIncomingPacket (C:\Work\scolasta2\node_modules\.pnpm\esbuild@0.13.15\node_modules\esbuild\lib\main.js:726:9)
      at Socket.readFromStdout (C:\Work\scolasta2\node_modules\.pnpm\esbuild@0.13.15\node_modules\esbuild\lib\main.js:596:7)
      at Socket.emit (node:events:390:28)
      at addChunk (node:internal/streams/readable:315:12)
      at readableAddChunk (node:internal/streams/readable:289:9)
      at Socket.Readable.push (node:internal/streams/readable:228:10)
      at Pipe.onStreamRead (node:internal/stream_base_commons:199:23)

What is expected?
It should compile
What is actually happening?
Doesn't compile

This looks like a transformation error in the vue compiler, since the following works fine as a workaround:
const emit = defineEmits(['foo']) as Emits

But the above code is valid typescript and shouldn't result in what is being generated.

you should use type declaration
const emit = defineEmits<Emits>()

I hate using GitHub on mobile... Didn't mean to close

I have this too in version 3.2.31
No interface available. I do not agree that it is a "feature" or a "nice to have"

@nborko how could you work around the problem with
const emit: Emits = rawEmit
as that still needs the Emits type which does not exist...
Thanks in advance.

@nborko how could you work around the problem with const emit: Emits = rawEmit
as that still needs the Emits type which does not exist... Thanks in advance.

You're looking for this?https://github.com/vuejs/core/issues/5393#issuecomment-1034335331

As a quick explanation of my setup, I have s subdirectory for each component with the following structure:
// types.ts

// heavily documented with tsdoc
interface MyComponentProps {
    ...
}

// heavily documented with tsdoc
interface MyComponentEmits {
    ...
}

// synthetic type for api-extractor
interface MyComponent {
    ...
}

<comment><!-- MyComponent.vue --></comment>

<script setup lang="ts">
import { MyComponentEmits, MyComponentProps } from './types'

const props: MyComponentProps = defineProps({
    ...
})

// This is broken
const emit: MyComponentEmits = defineEmits([ ... ])

// some functionality
</script>

// index.ts

export { default as MyComponent } from './MyComponent.vue'

This helps keep the documentation and types separated from the SFC to keep the code clean and easier to maintain, and I point api-extractor to a top level types.ts that imports and re-exports the types from each of the components, for further processing into API documentation.


javascript function to convert days to seconds

Why:
Closes #1909
What's being changed:
new file: program/JavaScript-program-to-convert-days-to- seconds
#1909

bro I am assign to this

@Larryking007, someone was assigned to the issue. Please find other issues to contribute and make sure you type !assign to grab the issue.
Thank you!


Missing filterOff method. Take a look at this delete example


Add URL for Ozzy Mandice

Ozzy Mandice is a legend, and this actually isn't even close to being his best map. He released most of his maps on YouTube, so that is the best place to link to. He is also available to speak to on Discord, he is OzzyMandice#8541.

trying to do this just through the website interface, it didn't branch properly. going to try again


Change paper directory name

check if this is ok (ie check the pdf output of paper report) and merge please, thanks

@brianqiu @changsiyao @pigriver123 @boyinggong


SyncBot: Track incidents of "not allowed to act"

Creating this issue to track incidents of
front reports This teammate is not allowed to act on the specified conversation..
No fixes currently documented.

https://www.flowdock.com/app/rulemotion/s-sales/threads/5Q2A7dHIacu2GWVLji0TwK_AJYc

https://www.flowdock.com/app/rulemotion/s-sales/threads/vpeW0Y2DDCphKhfn90SYx-BL4NI

https://www.flowdock.com/app/rulemotion/s-sales/threads/rtltBgC2XCNOGhgj6di1vzYmSCc


Route not started when deployed in WAR sub module

Hello,
I am trying to a hello world using wildfly camel and following examples on following links
Example 1
Example 2
And I am using wildfly-10.1.0.Final with wildfly-camel-patch-4.5.0. If I deploy my application as war file then it starts the camel context and route is started too. However when i deploy the same war as module in a EAR file it doesn't start the route, however context gets created, couldn't find anything wrong in war.  Can someone let me know, if I missing something or this a known bug and we have workaround or i have encountered a bug.

Possibly a bug. We have unit tests that cover camel EAR deployments with JAR modules but not WAR.
So you could maybe try changing the packaging type and see if that helps.
Let me know.

Thanks James for the pointer, packing a jar inside ear worked.

Thanks for confirming.
I'll look into the issue with the WAR deployment.

What's happening is that the WAR BeanArchive is not visible to camel-cdi, so it never discovers any  Camel routes. Hence when the Camel context is started you see:
Total 0 routes, of which 0 are started.
This will be fixed in next release.

Thanks everyone.


Fixed jumping buttons with Bootstrap tooltip

This pull request fixes an issue where buttons that are part of a Bootstrap button group jump on mouseover when a Bootstrap tooltip is added to them.

Thanks <3


Add codehint for aural features of CSS and Add space-evenly in justify-content

Add Aural features like speak, speech-rate, etc.

Could you sign the CLA located at http://dev.brackets.io/brackets-contributor-license-agreement.html?

I tried but the service is currently not available.

Also, it solves #14477


Interaction matching - body is ignored in certain cases

Hey @neilcampbell, question - would it make sense to run a match on request body even if request body for registered interaction is not specified?
Currently library will consider two requests matching even when body of the actual request is completely different (i.e. not null) from request body for registered interaction (that is null).

@xelibrion Yeah it's a good question and I remember having a conversation about this at some point with someone else, but cant remember the exact outcome. Based on the pact specification for v2 (which we are currently not checking on just yet) the current behaviour is correct. https://github.com/bethesque/pact-specification/blob/version-2/testcases/request/body/no body.json. @bethesque Can you confirm that this is correct? Postel's law is making me question this one.

I think you're right @neilcampbell, @uglyog, this doesn't seem right https://github.com/bethesque/pact-specification/blob/version-2/testcases/request/body/no body.json

When there is no expectation on the body (as opposed to expecting an empty
body), then surely it must match regardless if you receive a body or not?
On 3 September 2015 at 07:35, Beth Skurrie<EMAIL_ADDRESS>wrote:

I think you're right @neilcampbell https://github.com/neilcampbell,
@uglyog https://github.com/uglyog, this doesn't seem right
https://github.com/bethesque/pact-specification/blob/version-2/testcases/request/body/no body.json
—
Reply to this email directly or view it on GitHub
https://github.com/SEEK-Jobs/pact-net/issues/47#issuecomment-137251507.

@neilcampbell @bethesque @uglyog I think there actually might be two parts to this question. First one is matching registered interaction with actual request on consumer side, and the second on is matching consumer expectation and provider response.
As for the latter one, it is perfectly fine to have extra fields or entire body in the response in my opinion, but I don't think that exactly the same rules should apply for the former, although I can see that it might be questionable and we potentially might need two modes of interaction matching on consumer side (strict/non-strict).

For reference to this https://github.com/DiUS/pact-jvm/issues/85

@xelibrion I had a bit of a think through what we discussed on the weekend in relation to matching the correct request with response for the mock. We could definitely do what we talked about, however it would deviate from the what the specification outlines (which I don't want to do). See item 2 in Handling Requests https://github.com/bethesque/pact-specification/tree/master/implementation-guidelines#handling-requests

There's a difference between an expectation of a null body, and no expectation for the body. I think (correct me if I'm wrong Ron), if you expect a null request body, then you should get a null request body. If you make no expectation on the request body, then I think we're not checking it at all. When the request is replayed though, it's going to have a null body, rather than whatever the actual request was, which makes it potentially buggy.

@bethesque That's correct. If the body field is missing from the pact file, it can be anything because we are not checking it. It does not have to be null.

@neilcampbell Fair enough. To solve the problem that I had (spending two hours debugging PactNet because I've forgotten to specify body for a POST request in expectation) is not necessary to change expectation matching rules - would be enough to display a warning/suggestion in test output that body was empty in expectation, but it was present in actual request. Would make dev onboarding slightly easier.

Sounds like a good idea.

@bethesque @uglyog Was just looking at closing this issue and implementing a fix for some of the stuff discussed and wanted to confirm a couple of behaviours.
I can see that when no body has been specified we don't go any further with regards to checking the body (https://github.com/bethesque/pact-specification/blob/version-2/testcases/request/body/no body.json), which is the current behaviour for PactNet.
Now if I explicitly set the expected body to null, should I ensure that the actual body is null as well and serialize that in the pact file ("body": null)? Is there a specification test for both request and response to show this scenario?

I'm not sure if there's a specification, but the behaviour you described in the last paragraph is the behaviour of the Ruby impl.

@bethesque Ok cool. I will create specification test and PR for it, that way it is explicit.


Can't see the ports  on GitHub Codespaces

I guess I am doing something wrong.

The build looks like to be completed, but can't see the ports though
any idea why?

@Webmaster341 This is some issue of GitHub codespaces can you try deleing the current codespace and create new one and try to build again?

Is this issue solved?

I am seeing this issues while in codespaces


Ok/component api improvement

Renamed ElementValue<T> to more simple and expressive Property<T>.
All components now use property setters instead of the fill method as described in #55 .
Added base FormElementParser so parsers can be easily inherited from each other.
Removed address component because it is not very reusable.
Date and DateRange components are now generated from YAML files.
Generator no longer scans other YAML files so there is no need to have all the YAML files inside the target project.

Codecov Report

Merging #57 into master will increase coverage by 0.16%.
The diff coverage is 48%.

@@            Coverage Diff             @@
##           master      #57      +/-   ##
==========================================
+ Coverage   44.99%   45.15%   +0.16%
==========================================
  Files          92       93       +1
  Lines        1958     1962       +4
==========================================
+ Hits          881      886       +5
+ Misses       1077     1076       -1

Impacted Files
Coverage Δ

...b/src/properties/explicit_expression_property.dart
0% <0%> (ø)

...ges/dynamic_forms/lib/src/elements/validation.dart
0% <0%> (ø)
:arrow_up:

...parser/form_element_parsers/validation_parser.dart
0% <0%> (ø)
:arrow_up:

..._forms/lib/src/properties/expression_property.dart
81.81% <0%> (ø)

.../src/properties/cloneable_expression_property.dart
0% <0%> (ø)

...namic_forms/lib/src/form_manager/form_manager.dart
0% <0%> (ø)
:arrow_up:

...s/lib/src/properties/lazy_expression_property.dart
0% <0%> (ø)

...rms/lib/src/form_manager/form_manager_builder.dart
0% <0%> (ø)
:arrow_up:

...rser/form_element_parsers/form_element_parser.dart
100% <100%> (ø)

...forms/lib/src/iterators/form_element_iterator.dart
100% <100%> (ø)
:arrow_up:

... and 11 more

Continue to review full report at Codecov.

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


GroupCogMeta overwrites the superclass's command parents

Summary
JishakuBase's commands are altered by GroupCogMeta when they should not
Reproduction steps

from jishaku import JishakuBase
from jishaku.metacog import GroupCogMeta
from discord.ext import commands

bot = commands.Bot('!')

@commands.group()
async def not_jsk(self, ctx):
    pass

class MyJsk(JishakuBase, metaclass=GroupCogMeta, command_parent=not_jsk):
    pass

MyJsk(bot)

Expected results
Should make a new cog with not_jsk as the parent of all commands
Actual results
Traceback (most recent call last):
  File "/home/starrfox/bots/Discord-chan/venv/lib/python3.7/site-packages/jishaku/cog_base.py", line 494, in jsk_python
    async for send, result in AsyncSender(executor):
  File "/home/starrfox/bots/Discord-chan/venv/lib/python3.7/site-packages/jishaku/functools.py", line 109, in _internal
    value = await base.asend(self.send_value)
  File "/home/starrfox/bots/Discord-chan/venv/lib/python3.7/site-packages/jishaku/repl/compilation.py", line 154, in traverse
    async for send, result in AsyncSender(func(*async_executor.args)):
  File "/home/starrfox/bots/Discord-chan/venv/lib/python3.7/site-packages/jishaku/functools.py", line 109, in _internal
    value = await base.asend(self.send_value)
  File "<repl>", line 13, in _repl_coroutine
  File "/home/starrfox/bots/Discord-chan/venv/lib/python3.7/site-packages/discord/ext/commands/cog.py", line 182, in __new__
    parent = lookup[parent.qualified_name]
KeyError: 'jishaku'

Checklist

[x] I have updated discord.py and jishaku to the latest available versions and have confirmed that this issue is still present
[x] I have searched the open issues for duplicates
[x] I have shown the entire traceback, if possible
[x] I have removed my token from display, if visible

System information

Discord.py:

Python v3.7.3-final
discord.py v1.3.0-alpha

discord.py pkg_resources: v1.3.0a2157+g1b9108f

aiohttp v3.6.2
websockets v8.1
system info: Linux 4.19.0-6-amd64 #1 SMP Debian 4.19.67-2+deb10u2 (2019-11-11)

jishaku:
Name: jishaku
Version: <IP_ADDRESS>
Summary: A discord.py extension including useful tools for bot development and debugging.
Home-page: https://github.com/Gorialis/jishaku
Author: Devon (Gorialis) R
Author-email: None
License: MIT
Location: /home/starrfox/bots/Discord-chan/venv/lib/python3.7/site-packages
Requires: discord.py, braceexpand, import-expression, humanize

The command parent must use @commands.group() (a Command cannot have child commands)

Sorry forgot that when making the reproduction steps, still have the same issue when using commands.group though

Fixing this is kind of a pain. The key problem here is that GroupCogMeta is parenting the Commands it sees on the class, but since the implementation is not overridden, the Commands it finds are from the superclass (JishakuBase) and thus they get associated with the a Group owned by the subclass.
The two ways I can think of going about this while maintaining functionality are:

Creating some kind of 'factory' for JishakuBase-like classes that each consumer should call to receive their own personal baseclass (and thus the Commands of the baseclass being parented is not much of a problem, since it is in effect owned by its subclass). This feels like a pretty bad way of doing it, because dynamically manufacturing new types at runtime makes those types hard to work with (isinstance(x, JishakuBase) would no longer be possible) and it feels like it fundamentally breaks the principle of using baseclasses in the first place

Creating a more complicated generalized metaclass similar to GroupCogMeta, but instead of assigning the existing commands of the class to a provided parent, instead takes in the parent and defines the Jishaku standard commands. The problem with this approach is that subclass command callbacks wouldn't be able to call back to their superclass versions, since they wouldn't be part of the superclass anymore.

Overall, I think that the class redesign, while allowing for more extensibility, has caused a lot more problems than it's probably been worth. I figure I will redesign it when I have a better idea of how to do it properly.

This problem is mostly addressed by the jishaku Feature system in v2.


Search filters as URL query string

Summary
Is there a way where I can provide a URL for Traces(distributed-tracing) page with the traceId(or other atributes) to be search as query string directly in the URL?
Desired Behavior
On the Traces page (distributed-tracing), I can search for the unique "traceId" attribute, and others:

I'm monitoring multiple requests and have the traceId for them, so I would like to create a URL link to new relic directly for that request, and I could use the traceId to mount the URL for it. So it could land on Traces page, already searched for the desired traceId.
For example: https://one.newrelic.com/distributed-tracing?account=3509865&duration=86400000&traceId=07e20d4bf262431db9339cdf4be101ef
I have searched and tried multiple ways, but could not find anything about this, So I'm asking here.
Additional context
Sorry if this isn't the right repository to ask about this.

@victorhugorodrigues Issues raised in this repository are for the browser agent itself. Please use the feedback feature within New Relic, the New Relic support site, or contact your New Relic support rep for issues and suggestions within the New Relic UI.


As a EDS user I would like the EDS storefront Home page to give me an overview of what is new in EDS.

[x]	Remove under construction
[ ]	Add some 'quick links:	cdn, Slack, npm packages, Figma, Github (report issues here text),  React library, icons, release info
[ ] Component status - put in 3 sections - doing, done, to do. This is added to Home page
[ ] Add information about the Core Team

@equinor/design-system is this still relevant? Maybe make a separate task for bullet points

This came from one of our gatherings and we seemed to agree on some improvements. Probably still relevant. Separate task for each bullet point sounds good.

I'm closing this since we have implemented some of these changes in zeroheight now.


Failed to establish a new connection: [WinError 10061] No connection could be made because the target machine actively refused it

Here's the commands I execute:
mongod --dbpath "C:\Program Files\MongoDB\Server\4.0\data" --replSet rs0 --logpath "C:\Program F iles\MongoDB\Server\4.0\log\mongod.log"
mongo-connector -m localhost:27017 -t localhost:9200 -d elastic2_doc_manager -n collection.threads
Services (Windows application) -> Run elasticsearch service
I'm more or less using this as a guide to use mongo-connect with elasticsearch.
What's going wrong here? I notice the error throws before or after running the elasticsearch service so I can't make heads or tails of where I'm screwing up.

The errors disappear now (for no obvious reason) but I still don't get the Mongodb data in elasticsearch. It doesn't look very promising because when I use the wrong port for the -t flag (e.g. localhost:9800), I get the same result.


tests: Fix docker logs test to use common function

Integration docker logs tests should use common function of
runDockerCommand.
Fixes #302
Signed-off-by: Gabriela Cervantes<EMAIL_ADDRESS>

Ooh, I didn't pick up on the dockerrandomname before - nice.
lgtm


Why do I need this as developer? This question and the terminology - 'hot module replacement' should be explained better.

New developers and developers who might not need this feature in present, maybe due to less compilation time required in their normal development procedure, might not understand the idea of hot module replacement. Please explain this idea first and then tell us how to do it.

The main advantage is in development time, Instead of having to reload the page it will automatically appear. Now this is not the most amazing thing. Keeping state is what really speeds up your development process. In a simple app this might seems like overhead, but if you are working on a multipage application and you are working on something 5 components deep it can be a pain to code -> safe -> recreate all the steps -> repeat
I hope this answers your question

Please add something like this to the main page!
You will get a lot more people using your project if you explain it better. Also it would be very important to list the advantages and disadvantages. Keeping state sounds great but I think it is a very complicated thing and it would be good to know what does work or doesn't before I start.

Angular's ng serve does the same, right? Is there any difference?


posthog-destination: support $screen_name for screen events

Currently, the $screen_name property isn't being passed properly to posthog on screen events. You can see that being done in posthog's native integration here

Thank you!


ImportError: cannot import name BaseString

Expected/Desired Behavior:
The import to work without errors
Actual Behavior:
from mininet.util import ( quietRun, errRun, errFail, moveIntf, isShellBuiltin,
ImportError: cannot import name BaseString
Detailed Steps to Reproduce the Behavior:
from mininet.topo import Topo
from mininet.net import Mininet
from mininet.util import dumpNodeConnections
from mininet.log import setLogLevel
Additional Information:

closing as it seems to be fine in master


Changelog for 0.18?

First, thanks for the new version!
In the process of writing this issue, I found a changelog in:
https://github.com/PythonCharmers/python-future/blob/master/docs/whatsnew.rst
However, that's not visible on website, and changelog.rst is very out of date.
Also, could y'all tag a release with the changelog, similar to 0.17?
https://github.com/PythonCharmers/python-future/releases

The formatting of https://github.com/PythonCharmers/python-future/blob/master/docs/whatsnew.rst is a little broken right now; it's fixed in #526

Is this issue about regenerating the website from source (which seems to be a few years out of date) or is this about moving content from docs/whatsnew.rst to docs/changelog.rst?
As per my comments on #302, I definitely want to see the website updated, but I believe the what's new and changelog changes need to also not be lost. I hardly think changes from five years ago need to be in what's new (probably it it best to keep that with just changes from the last major point release; at the time of this writing that was 0.18.0).

only thing missing is to update the website

Hi everyone, good news, as a work-around you can now access the latest docs at https://python-future.readthedocs.io/en/latest/


[KARAF-5934] Upgrade to maven-bundle-plugin 4.1.0

retest this please

retest this please

retest this please


Fix typos and hint text for video page lab

We need to fix this typo here found in the final solution code
https://github.com/freeCodeCamp/freeCodeCamp/blob/main/curriculum/challenges/english/25-front-end-development/lab-video-compilation-page/669e81368e52b3a5c35a2dc5.md
The typo should be fixed to say developers not evelopers

Front End Web Development

Front End Web Development involves designing and building user interfaces of websites using HTML, CSS, and
            JavaScript. It focuses on creating visually appealing, interactive, and user-friendly websites. Front End
            evelopers
            ensure accessibility, performance optimization, and
            adherence to web standards, blending creativity with technical expertise.

Also the wording in this hint text could be reworded

Each `section` element should contain an `h2` element to give a title to that section as its first child.

let's reword it to the following:
Each `section` element should start with an `h2` element that serves as the title for that section.

This has been opened for contribution. The first comprehensive PR created will be reviewed and merged. We typically do not assign issues to anyone other than long-time contributors.
If you would like to contribute and have not read the contributors docs, please do so here: https://contribute.freecodecamp.org/#/
If you have any issues with contributing, be sure to join us on the contributors channel, or on the contributors sub-forum
Files that need to be changed
https://github.com/freeCodeCamp/freeCodeCamp/blob/main/curriculum/challenges/english/25-front-end-development/lab-video-compilation-page/669e81368e52b3a5c35a2dc5.md
Happy coding

working on this.
But first, Imma read the contribution guidelines real quick.


Spacing between atom symbols and bond lines in 2d plot

It seems to me that the distance between atom symbols and bond lines is too small sometimes in 2d plot and it becomes hard to read the symbols. I looked into some options in IPythonConsole.drawOptions but could not figure it out. Is it possible to adjust such details?
IPythonConsole.drawOptions.bondLineWidth = 1
mol = Chem.MolFromSmiles('CC(=O)O[C@H]1CC[C@@]2(C)C@@HC1')
mol

HI @HesamGit.  That's not a valid SMILES string (only one ring locant 2) so it's not possible to see what you mean.  Can you please provide a correct SMILES string, a picture of the drawing you object to and also the version of RDKit you're using and how you installed it.  It is very difficult to do anything without this information.

The issue template asks you a series of questions that make it easier for us to help.
The fact that you've either deleted those or only answered a subset of them makes it more difficult for us to help, so I'm closing this issue.
Please feel free to edit the issue description to include the requested information and reopen the issue and then we'll try to help.


The entire UI rerenders upon updating a node

Describe the Bug
Currently, updating a property, using the node configuration form, forces the entire UI to rerender.
Steps to Reproduce the Bug or Issue

Enable the React profiler
Select a node to open the configuration panel
Write in one TextField and notice how the entire UI gets rendered, slowing down the process.

Screenshots or Videos
Screencast from 2023-10-11 12-08-58.webm
Platform

OS: [Linux]
Browser: [Firefox]
Version: [0.2.0]

Using the metadata editor
Screencast from 2023-10-17 13-58-16.webm
Using the source code editor
Screencast from 2023-10-17 13-47-16.webm

related to #266 ?

Yes, although not exactly the same, it's related