Request for more detailed step-by-step guide

I have read the Cityflow doc multiple times but it comes very short to providing enough guidance to get me started.
Here are just a few questions from a long list of questions that weren't answered by the guide:

What is "roadLinkIndices"?

What does it mean if "availableRoadLinks":[] for a lightphases?

Why must virtual intersections have "lightphases" when we don't care about the lights in the edges?

How do I change the number of vehicles (i.e. the rate of spawning new vehicles)?

How do you turn on yellow lights? And change the duration of yellow lights?

Where is the origin (the point where x and y are both 0)?

how is roadLinks different from laneLinks?

I see that CityFlow could be leaps better than SUMO, but there doesn't seem to be enough guide to getting started. And I am willing to read any extra material you can provide.
Since there is no guide, I am digging through roadnet.json and flow.json that others have made to deduce what these fields do, which is a painstaking process.
So a more detailed step-by-step guide that doesn't omit any necessary information would be great.

I apologize for our incomplete documents. I agree with you that we should provide some tutorials for users. We plan to do it recently, and I will keep this issue open until we provide additional tutorials.
For now, I can answer some of your questions. But I don't fully understand every detail in roadnet file, so I will figure out if I don't know or confident about the answer.

To be honest, I don't know about that. I guess it means the traffic light can control these roadLink. But I think it may be redundant.
In every lightphases, only the roadLinks in availableRoadLinks are passable. It figures out which lights are green in this phase.
I think it is unnecessary. Maybe it can be deleted (not sure). If not we can improve it later.
Every flow has an attribute called interval, which means every few seconds there is one vehicle appearing.
Since the yellow-light phase has no difference with an all-red phase, you can define 0 phase as an all-red phase. (you can also determine whether cars can turn right)
All positions are relative. So, the origin point has no meaning except it is the initial position in visual UI.
A roadLink contains several laneLinks(i.e. the source road and the destination road have both two lanes, there are 2 * 2 = 4 laneLinks in this roadLink) laneLinks are used for running vehicles while the roadLink will be directly controlled by traffic light.

"roadLinkIndices" is deprecated, I've just removed it from examples/roadnet.json in commit bb22e22

@Hzfengsy
Thanks for your helpful answer :)
A follow up on your second answer:

In every lightphases, only the roadLinks in availableRoadLinks are passable. It figures out which lights are green in this phase.

So in examples/roadnet.json, there are lines like this:
"lightphases":[{"time":30,"availableRoadLinks":[]},

Where a lightphase does not have any link. Does this mean it is essentially a red light on all lanes?

@ThisIsIsaac
You're right. And for virtual intersections, trafficLight is not required and has no effect.

how do you reset the simulation to scratch? Do you simply call eng = cityflow.Engine(config_path, thread_num=1) again?

You can use eng.reset() (Notice this does not reset random state).

 Or is there no need to do that?

You can use del eng to trigger garbage collection.

what are some paramters to eng.reset()?

@Hzfengsy

Since the yellow-light phase has no difference with an all-red phase, you can define 0 phase as an all-red phase. (you can also determine whether cars can turn right)

I am not convinced that yellow light phases are not necessary. If a car was going at 50km/h and the light suddenly turns red without any buffer time -- i.e. the yellow light -- for the car to decelerate, then the car would either have to: 1. hit the breaks to stop immediately, or 2. run the light and perform an illegal action. So without yellow lights, how does a car know when to start decelerating properly?

what are some paramters to eng.reset()? I think it could be helpful if it is in the docs. I will make a push you are busy and can direct me to where I can see the explanations.

There is no parameter in reset(). I've added reset() in docs in c99cee1. Thanks a lot!

So without yellow lights, how does a car know when to start decelerating properly?

In our system design, if a vehicle cannot perform a full stop before the end of the road, then it will just go through the intersection even if the light is red. On the other hand, if it can stop, it will. This is similar to the intention of yellow lights. So we think all red phase is sufficient. But I admit there is some difference, i.e. driver will go through the intersection even if he can stop given a yellow light. We consider adding a yellow light in future version.

@zhc134
Thanks for the detailed explanation.

In our system design, if a vehicle cannot perform a full stop before the end of the road, then it will just go through the intersection even if the light is red. On the other hand, if it can stop, it will.

What is the rate of deceleration? Can we control this? I assume the rate of deceleration is set to be close to the rate of deceleration of real drivers when they see a yellow light that they cannot make through.

@ThisIsIsaac
each vehicle has a parameter maxNegAcc which is specified in flowFile.
please see https://cityflow.readthedocs.io/en/latest/flow.html for more detail

So if the distance from stopping line is too short s.t. the car won't come to stop while accelerating at maxNegAcc, then it just runs the light?

Yes

Is there a way to set where to save replay files in Python, without editing config.json? When debugging an RL agent, I may want to save replays of multiple episodes. However, since the name of the replay file is hard-coded in config.json, older replay files get over-written whenever a new episode starts.

Currently, we do not support changing replayFilePath on the fly. Please open a new issue, clarify your needs and we will add it to our develop plan.
BTW, eng.reset() won't clear old replay information, so you can visualize multiple episodes, they will appear in replay.txt in order.

BTW, eng.reset() won't clear old replay information, so you can visualize multiple episodes, they will appear in replay.txt in order

oh so if there already exists replay.txt, then instead of overwriting the old replay, it appends newer replay to the end?

If you call eng.reset() then yes it appends newer replays to the end. However, if you create a new Engine object with same replayFilePath, it will clear the file first.

Thanks. I feel like this could be very important, but not obvious from the doc. Maybe we can add this to the doc under the replay section?

Sure! Please check 7f2cc2e

Is "interval":  defined in flow.json specific to each vehicle? So interval:5 means the specific vehicle is spawned every 5 seconds. Would that be a correct interpretation?

Exactly

I have checked your doc, but I don't find a function to get the current signal phase?
How to obtain the current signal phase for a given intersection?


[K3s] CIS Benchmark 1.24 requires file permissions to be 600 instead of 644

This issue is to improve the default file permissions of K3s files based on CIS (since 1.24) new requirements.
Here is the list of concerned CIS checks with their concerned files to be set from 644 to 600.

1.1.13 Ensure that the admin.conf file permissions are set to 600 or more restrictive (Automated)
File: /var/lib/rancher/k3s/server/cred/admin.kubeconfig
1.1.15 Ensure that the scheduler.conf file permissions are set to 600 or more restrictive (Automated)
File: /var/lib/rancher/k3s/server/cred/scheduler.kubeconfig
1.1.17 Ensure that the controller-manager.conf file permissions are set to 600 or more restrictive (Automated)
File: /var/lib/rancher/k3s/server/cred/controller.kubeconfig
1.2.19: Ensure that the Kubernetes PKI certificate file permissions are set to 600 or more restrictive (Manual)
File: /var/lib/rancher/k3s/server/tls/*
4.1.3 If proxy kubeconfig file exists ensure permissions are set to 600 or more restrictive (Manual)
File: /var/lib/rancher/k3s/agent/kubeproxy.kubeconfig
4.1.5 Ensure that the --kubeconfig kubelet.conf file permissions are set to 600 or more restrictive (Automated)
File: /var/lib/rancher/k3s/agent/kubelet.kubeconfig
4.1.7 Ensure that the certificate authorities file permissions are set to 644 or more restrictive (Manual)
File: /var/lib/rancher/k3s/server/tls/server-ca.crt
4.1.9 Ensure that the kubelet --config configuration file has permissions set to 600 or more restrictive (Automated)
File: /var/lib/rancher/k3s/agent/kubelet.kubeconfig

Related issue: https://github.com/rancher/rancher/issues/41131
cc @rayandas

We do not harden k3s automatically. These will need to by hand after install to harden k3s for the cis checks. Only RKE2 offers more default hardening with the --cis flag.

@dereknola With regards to the new permission constraints, we have two options for a given check/file mentioned above:

Improve K3s file permissions to 600 (preferable).
or
Deviate from CIS recommendation and keep the hardening guide and profile to 644.

Could you list the files (mentioned above) that could not be changed to 600, and would stick with 644 ?

@andypitcher Most of these will be doable, the only one that likely will need to be remain manual would be:
1.2.19: Ensure that the Kubernetes PKI certificate file permissions are set to 600 or more restrictive (Manual)
File: /var/lib/rancher/k3s/server/tls/*
The PR linked about will ensure that all relevant kubeconfig files have a 600 file perm.

Validated on master branch with commit be442433
Environment Details

Infrastructure

[ ] Cloud
[ ] Hosted

Node(s) CPU architecture, OS, and Version:

PRETTY_NAME="Ubuntu 22.04.2 LTS"

Cluster Configuration:

2 servers, 1 agent
Split role setup: 1 etcd-only, 1 cp-only and 1 agent node

Config.yaml:

# etcd-only S1
disable-apiserver: true
disable-controller-manager: true
disable-scheduler: true
node-taint:
  - node-role.kubernetes.io/etcd:NoExecute
write-kubeconfig-mode: 644
cluster-init: true

# cp-only S2
disable-etcd: true
node-taint:
  - node-role.kubernetes.io/control-plane:NoSchedule
server: "https://<IP of S1>:6443"
token: "<K10 from S1>"
write-kubeconfig-mode: 644

# agent
server: "https://<IP of S1>:6443"
token: "<K10 from S1>"

Testing Steps

Copy config.yaml

$ sudo mkdir -p /etc/rancher/k3s && sudo cp config.yaml /etc/rancher/k3s

Install k3s on server and agent nodes
Ensure the cluster is up and running
Validate the files below are NOT present on the cp-only (as it has the flag disable-etcd: true)  server node

$ sudo ls -la /var/lib/rancher/k3s/server/tls/etcd
...
-rw-r--r-- 1 root root 1258 Jul 17 21:00 server-client.crt
-rw------- 1 root root  227 Jul 17 21:00 server-client.key

Replication Results:

k3s version used for replication:

k3s version v1.27.3+k3s1 (fe9604ca)
go version go1.20.5

Observed the files are present on cp-only node

$ sudo ls -la /var/lib/rancher/k3s/server/tls/etcd
total 48
drwx------ 2 root root 4096 Jul 17 20:42 .
drwx------ 4 root root 4096 Jul 17 20:42 ..
-rw-r--r-- 1 root root 1140 Jul 17 20:42 client.crt
-rw------- 1 root root  227 Jul 17 20:42 client.key
-rw------- 1 root root  566 Jul 17 20:21 peer-ca.crt
-rw------- 1 root root  227 Jul 17 20:21 peer-ca.key
-rw-r--r-- 1 root root 1250 Jul 17 20:42 peer-server-client.crt
-rw------- 1 root root  227 Jul 17 20:42 peer-server-client.key
-rw------- 1 root root  570 Jul 17 20:21 server-ca.crt
-rw------- 1 root root  227 Jul 17 20:21 server-ca.key
-rw-r--r-- 1 root root 1258 Jul 17 20:42 server-client.crt
-rw------- 1 root root  227 Jul 17 20:42 server-client.key

Validation Results:

k3s version used for validation:

k3s version v1.27.3+k3s-be442433 (be442433)
go version go1.20.5

Observed the files are NOT present on cp-only node

$ sudo ls -la /var/lib/rancher/k3s/server/tls/etcd
total 40
drwx------ 2 root root 4096 Jul 17 21:02 .
drwx------ 4 root root 4096 Jul 17 21:02 ..
-rw-r--r-- 1 root root 1140 Jul 17 21:02 client.crt
-rw------- 1 root root  227 Jul 17 21:02 client.key
-rw------- 1 root root  566 Jul 17 21:00 peer-ca.crt
-rw------- 1 root root  227 Jul 17 21:00 peer-ca.key
-rw-r--r-- 1 root root 1250 Jul 17 21:02 peer-server-client.crt
-rw------- 1 root root  227 Jul 17 21:02 peer-server-client.key
-rw------- 1 root root  570 Jul 17 21:00 server-ca.crt
-rw------- 1 root root  227 Jul 17 21:00 server-ca.key

@dereknola could you confirm that all file locations had their permissions set to 600 ?
My understanding is that any kubeconfig in /var/lib/rancher/k3s/server/cred/ were covered by this commit, what about /var/lib/rancher/k3s/agent/ ?

Yes this PR did change the agent file to 600 as well
root@degion:/var/lib/rancher/k3s/agent#  stat -c "%a %n" ./*
600 ./client-ca.crt
600 ./client-k3s-controller.crt
600 ./client-k3s-controller.key
600 ./client-kubelet.crt
600 ./client-kubelet.key
600 ./client-kube-proxy.crt
600 ./client-kube-proxy.key
700 ./containerd
700 ./etc
600 ./k3scontroller.kubeconfig
600 ./kubelet.kubeconfig
600 ./kubeproxy.kubeconfig
755 ./pod-manifests
600 ./server-ca.crt
600 ./serving-kubelet.crt
600 ./serving-kubelet.key


feat(node): fully implement node:querystring and node:url

fixes #290
alternative of #292
Ported implementation of node:url and node:querystring from Node.js v22.7.0.
Remarks:

Process was semi-automated and I tried to not touch implementation as much as possible
Type checks were also added and passed, although some type safety (mostly for nullable values) had been bypassed trusting upstream implementation is safe
Ported tests/parallel/test-{querystring,url}* tests which pass as expected

[ ] Invalid IDNA seems to be uncaught. possibly related to punycode behavior not matching native format binding
[ ] Whatwg URL format with unicode+auth options fails

dear @IgorMinar i noticed you added yourself for review. Do you have any specific concerns about this PR?
I think for next step of adopting hybrid support for workerd, we can try with another PR. (and i have two different methods i like to discuss with your team how we can do it).

(merging to test against nightly channel and iterate)


[Feature Request] - add debug information to the thickness calculators

💻

[X] Would you like to work on this feature?

Feature Description

add lines object to the thickness calculator class (lines that are projected)
store dips as well
keep as a geopandas
object to be names as "lines"

Current Situation
there's no debug information for thickness calculators
Version
m2l  - 3.1.12
lpf - 0.1.13
Proposed Solution
na
Additional Context
No response
Affected Areas

[ ] input data
[ ] project creation
[ ] samplers
[ ] sorters
[ ] stratigraphic column
[ ] data types
[X] Other

[ ] also add warning if too many units haven't been estimated


Include the app core in the editing loop

Current dependencies on/for this PR:

master

PR #65

PR #66

PR #67

PR #68   👈

PR #69

This comment was auto-generated by Graphite.


Support Resolution

The client lacks resolutions higher than 1920x1080 in 16:9 and 16:10 resolution
To facilitate development, I have included the map of resolutions currently in use
https://upload.wikimedia.org/wikipedia/commons/e/e5/Vector_Video_Standards2.svg
(This problem can be solved by a manual edit in the configurations file)

Hi Ulisse, thanks for the report.
The launcher should allow you to input a resolution manually in the SC4 Settings window.
The reason the choices stop at 1920x1080 is because this is the highest resolution supported by SC4 without resolution hacks. But of course, if you want to enter in a higher resolution, you certainly can.


bat version 0.3.6

Codecov Report

:exclamation: No coverage uploaded for pull request base (master@39c11e7). Click here to learn what that means.
The diff coverage is 100%.

@@            Coverage Diff            @@
##             master      #88   +/-   ##
=========================================
  Coverage          ?   92.31%
=========================================
  Files             ?       24
  Lines             ?     1028
  Branches          ?        0
=========================================
  Hits              ?      949
  Misses            ?       79
  Partials          ?        0

Impacted Files
Coverage Δ

bat/__init__.py
100% <100%> (ø)

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 39c11e7...d7a794e. Read the comment docs.


Why is this repository so neglected? Is contribution still appreciated?

I created a pull request on Sep 16 2022 and it did not receive even any feedback. The PHP Storyblok Clients lacks so many features that the NodeJS implementation  provides.
If this project is abandoned, a notice about this would be 'great' to see. It is so frustating trying to improve the repository by contributing only to see that the repository seems to be neglected.
Please note that this criticism is not meant to blame you. You have probably a lot of tasks and I do not want to complain about how long it takes for a PR to be managed. Instead I just want to clarify if further effort by users is appreciated and likely to be considered by the respective persons.

Hi @web-dev-passion , I'm talking internally about how to improve this repo (I'm pretty new in the Storyblok team, so it is the reason why I'm aligning internally, but I'm a PHP addict and lover).
I think this repo is valuable to the PHP community and also to the product. So thank you for your patience, if you have any suggestions please keep sharing them and, I am going to test your PR.

Thats great to hear @roberto-butti. Thank you for your input!
Hopefully you find your way into the new team easily 😉

Yes @web-dev-passion  if you agree, we can close this issue and switch to the PR thread (your getAll PR). i have some specific comments on that.


[suggestion] Suppress console.log in dev-mode

Description:
When atom is running in dev mode(atom -d .), hydrogen write tons of log on chrome dev console.
This verbosity make more important log unnoticed/or difficult to find.
Background: Example two situation why I want to suppress hydrogen's log

While I'm hacking hydrogen, I add ad hoc console.log frequently, but without suppressing default log, finding this console.log is super difficult and easily scrolled over.
I launch atom in dev mode to develop my own package, or Atom-core itself, and use hydrogen to check behavior of some code, but in this situation, I don't have interest of hydrogen's log itself. I want to watch log from other packages I'm working on, but hydrogen's log occupy console log.

Steps to Reproduce:

Launch atom with -d flag(run Atom in dev mode).
Start hydrogen and open chrome dev-console and see how much log is written on console"

Suggestion to change
Make hydrogen write verbose log only when Hydrogen.debug was explicitly enabled regardless of atom's running mode.

Good idea!
Most amount of logs come from the mobx-devtools extension, so we'll need to disable it too.

I agree completely. PR incoming.

Wow thanks @BenTussert working on this.


DnsNameResolverBuilder does not instantiate InflightNameResolver

wrt https://github.com/netty/netty/pull/5267, DnsNameResolverBuilder does not instantiate InflightNameResolver. The only place that InflightNameResolver is used is https://github.com/netty/netty/blob/4.1/resolver-dns/src/main/java/io/netty/resolver/dns/DnsAddressResolverGroup.java#L73. The method is already marked as deprecated and the replaced method (https://github.com/netty/netty/blob/4.1/resolver-dns/src/main/java/io/netty/resolver/dns/DnsAddressResolverGroup.java#L100) uses DnsNameResolverBuilder which instantiates DnsNameResolver instead of InflightNameResolver.
With the above implementation, this causes the DNS servers to be overloaded with requests when there are multiple DNS requests made concurrently. Is it possible to switch to InflightNameResolver everywhere by default?

@trustin - wdty?

@trustin nudge :)

DnsNameResolverBuilder, by its name, is supposed to build a DnsNameResolver, not an InflightNameResolver that wraps a DnsNameResolver.
Perhaps we could:

make DnsNameResolver an abstract class,
move the implementation of DnsNameResolver to some other class such as DnsNameResolverImpl
create a DefaultDnsNameResolver that 1) extends DnsNameResolver and 2) is composed of InflightNameResolver and DnsNameResolverImpl

I'm not sure about the benefit of using DnsNameResolver by default. The default behavior does not look appealing given that it already has a DNS caching implemented but it does not handle the in-flight requests well.
Wouldn't it be simpler to wrap around DnsNameResolver with InflightNameResolver  by default in DnsAddressResolverGroup.java?

@Scottmitch / @normanmaurer what do you think about the proposal?

@trustin's suggestion keeps separation of concerns and we could still implement it such that the inflight behavior is clarified and users can get a good default. I agree that most folks will likely want to use the inflight by default (@trustin WDYT).

Wouldn't it be simpler to wrap around DnsNameResolver with InflightNameResolver by default in DnsAddressResolverGroup.java?

I'm not sure I understood your question. DnsAddressResolverGroup wraps DnsNameResolver with InflightNameResolver by default, unless you override the deprecated newResolver() method. You should override newNameResolver(), not newResolver().

@trustin sorry for not explaining the context. Vert.x seems to use DnsNameResolver instead of DnsAddressResolverGroup. That's why it did not use InflightNameResolver by default. I wonder whether it makes more sense to return InflightNameResolver by default from DnsNameResolverBuilder because of this implementation.
Thanks!


Remove metrics windowing, clean up step names for metrics writing

Removing windowing function before metrics writing because it's redundant.
Cleaned up the pipeline step names to be upper camel case.

/retest

/retest

/lgtm


Object.keys and Object.getOwnPropertyNames on Array returns indices twice

Expected:
Object.keys([1,2,3])
["0","1","2"]
Object.getOwnPropertyNames([1,2,3])
["length","0","1","2"]

Result:
["0", "1", "2", "0", "1", "2"]
["0", "1", "2", "length", "0", "1", "2"]

and also with js_Iteratoror for (i in arr)

I cannot reproduce this behavior. Which version?

sorry for bothering you, finally found it...
I broke it myself (in jsR_setindex)


use of unstable library feature | installation Error

cargo install cargo-binutils :
error[E0658]: use of unstable library feature 'proc_macro_byte_character'
error[E0658]: use of unstable library feature 'proc_macro_c_str_literals'

We don't use these features and they've recently been stabilized and expected for 1.79. Looking at https://github.com/rust-lang/rust/issues/115268 I see that other's are experiencing this.
Are you on nightly? If so, please update your nightly toolchain

Closing this issue.


Minor fixes to the new bake function of the dashboard.

Codecov Report

Merging #1486 (2a35b3a) into master (60bf93a) will not change coverage.
The diff coverage is n/a.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
@@           Coverage Diff           @@
##           master    #1486   +/-   ##
=======================================
  Coverage   81.10%   81.10%
=======================================
  Files         137      137
  Lines        4452     4452
  Branches      149      149
=======================================
  Hits         3611     3611
  Misses        841      841

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


Invite users modal: Convert "Streams they should join" to pills

At present, the "Streams they should join" part of the Invite user modal shows a list of checkboxes. This is quite hard to use in an organization with a large number of streams. Instead, we should match the styling for adding users to groups, as described in #24365.

Input:

Text box with typeahead for streams
When selected, streams are displayed as pills
Each pill should have an x on the right for removal

Hopefully, we can use the same pills as for search (see #26803).
We should keep the current functionality, so:

If "Default streams for this organization" is checked, the stream selection UI is not shown.
The stream input field initially has the default streams for the organization, same as the current checkboxes configuration.

@zulipbot claim

@zulipbot claim

@zulipbot abandon

@zulipbot claim

@zulipbot claim

@zulipbot abandon

@zulipbot claim

I have solved the issue but I am running into some typescript error when I am trying to import "pill_typeahead.js" in "invite.ts".

error TS7016: Could not find a declaration file for module './pill_typeahead'. '/home/vector/zulip/web/src/pill_typeahead.js'
implicitly has an 'any' type.

Can someone help me resolve this?

@Vector73 You're much better off asking this kind of question in the #frontend stream in the Zulip development community, as few people will see it here. Be sure to follow https://zulip.readthedocs.io/en/latest/contributing/asking-great-questions.html. Thanks!

OK, Thanks @alya.

@zulipbot claim

@zulipbot abandon

@zulipbot claim

@alya , I think the implimentation of search pills needs to be completed first, or another way could be that we use stream pills instead of them. should that be labelled as a blocker for this ?

Search pill are not a blocker as far as I know.

Yes I am working on it.


when will flink cdc mysql 2.X  support specific-offset

[ERROR] Could not execute SQL statement. Reason:
org.apache.flink.table.api.ValidationException: Unsupported option value 'specific-offset',
the options [earliest-offset, specific-offset, timestamp] are not supported correctly,
please do not use them until they're correctly supported

Hi @lijunkai ,Pr has opened , pay attention to this pr https://github.com/ververica/flink-cdc-connectors/pull/1157.

PR #1661 fixed this issue.


MiniMongo shell

What do you think of adding a little MiniMongo shell that supports find/findOne operations?
I'd really like this feature and can try to implement it.

It would be cool, I considered this when first developing the extension but will be a lot of work, it is within the realm of possibility though.


GetConnected() on Android using system API call

Hello, by calling Android system API it always reflects the current connection state of the device.
Regards
Markus

Thanks @mfmeisinger for your contribution!


Branch protection lock should appear consistently in quick pick

Status Bar

Quick Pick

Expected

@misolori, should we include the branch icon for ranches that are not protected, and the tag icon for tags?

I wouldn't add icons for every entry as it can get busy/repetitive. @Tyriar and I were just discussing about reducing the repetition of icons in quick picks and only showing them for the ones that really warrant it.

@lszomoru FYI @misolori was suggesting removing the regular branch icons which it doesn't look like the change does. That would align closer with how tasks use icons (defaults are hidden) and the new debug console quick access for example.

This particular quick pick contains items of different types (ex: local branch, remote branch, tag). The only was to distinguish the type if to look at the description of the icon. I think that having an icon for each item based on its type is actually helpful here. I would like to keep it and see what the user feedback will be.

Just by looking at the icons, I'm only seeing the "locked" branch as the different one from the others. There is no difference between the local and remote branches here so I'm not seeing the benefit of showing the icon every single time and still feels very noisy:

Reopening to track discussion

Here is what I am seeing if I am running a search for "release". I can clearly differentiate between tags, locked branches, and branches. @misolori, if I would have a suitable icon for a remote branch, I would be happy to add it. I personally still feel that there is value in showing an icon for item, especially that I think the way people interract with the branch picker is that they open it and start typing in the input to filter down to the item that they would like to open.

We could try to add an overlay on the branch icon, similar to VS:

Love it! If you add this icon, I am happy to place this in the picker to indicate remote branches.


[ARTEMIS-2050] It is possible to get AMQ224000: Failure in initialisa…

…tion: java.lang.NullPointerException during shutdown of backup server with shared store
Issue: https://issues.apache.org/jira/browse/ARTEMIS-2050
PR(2.6.x): https://github.com/apache/activemq-artemis/pull/2265
PR(master): https://github.com/apache/activemq-artemis/pull/2264
Cherry-picked from f45a0c472833cddcbfa5725f299bb3a21956e53f

I cherry-picked this manually. Please close this PR, @JiriOndrusek. Thanks!

Can you close this one as well, @JiriOndrusek?

Closing as requested by previous comment


UndefinedMethodException $resolver->setDefaultTransformers();

Hello
Symfony 3.4 is telling me that the method setDefaultTransformers() is not defined.
I checked Limenius\Liform\Resolver and could not find that method.
This is how my controller looks like:
`use Limenius\Liform\Resolver;
 use Limenius\Liform\Liform;

 class AdController extends Controller
{
/**
 * @Route("/nouvelle-annonce-voiture-occasion", name="new_ad_page")
 */
public function showAction(Request $request)
{
	$resolver = new Resolver();
	$resolver->setDefaultTransformers();
	$liform = new Liform($resolver);

    $ad = new Ad();
    //$form = $this->createForm(AdType::class, $ad);
	$form = $this->createForm(AdType::class, $ad);
	$schema = json_encode($liform->transform($form));
			\Doctrine\Common\Util\Debug::dump($schema);`

Also, you may notice that I removed the csrf_protection => false option from createForm as it would give me the error below. Instead I placed that option on the Type class, as follows:
    public function configureOptions(OptionsResolver $resolver)     {         $resolver->setDefaults(array(             'data_class' => Ad::class,             'csrf_protection' => false,         ));     }
This is the error: (if I leave the csrf_protection where you instruct in docs)
The option "csrf_protection" does not exist. Defined options are: "action", "allow_extra_fields", "attr", "auto_initialize", "block_name", "by_reference", "compound", "constraints", "data", "data_class", "disabled", "empty_data", "error_bubbling", "error_mapping", "extra_fields_message", "inherit_data", "invalid_message", "invalid_message_parameters", "label", "label_attr", "label_format", "liform", "mapped", "method", "post_max_size_message", "property_path", "required", "translation_domain", "trim", "upload_max_size_message", "validation_groups".
Thanks

Hi, @BernardA I wanted to have a look at this during the weekend. How was it solved? is there something we can do so other users don't get the same error?

HI @nacmartin . Thanks but it's mostly my fault again. I started from your Liform component and not from the bundle.

Ah, ok! Thanks for taking the time to answer. I split the bundle in two just in case someone is using Symfony Forms without Symfony, which is possible, but it takes more work to set up (which is what this bundle is doing).


Enabling testsuite/integration/basic to run against bootable jar

Regarding the modele.xml files. Sure, I could let a plugin in pom.xml copy those, however the content of the files needs to be adjusted a bit. To avoid ugly scripting, I decided to rather copy the files to the expected directory structure already adjusted.

Regarding the modele.xml files. Sure, I could let a plugin in pom.xml copy those, however the content of the files needs to be adjusted a bit. To avoid an ugly scripting, I decided to rather copy the files to the expected directory structure already adjusted.

Thanks @OndrejKotek - LGTM, merging.


fix fix time series compaction policy corner case

Proposed changes

When single replica compaction is enabled and BE1 fetchs merged rowsets from BE2, and then BE2 goes offline.
BE1 should perform compaction on its own, the time series compaction may re-compact previously fetched rowsets.
Time series compaction policy needs to skip over the fetched rowset.

If the compaction process has not been successfully executed, the condition for triggering compaction based on the last successful compaction time (condition 3) will never be met.

Issue Number: close #xxx

Further comments
If this is a relatively large or complex change, kick off the discussion at<EMAIL_ADDRESS>by explaining why you chose the solution you did and what alternatives you considered, etc...

run buildall

run compile

(From new machine)TeamCity pipeline, clickbench performance test result:
the sum of best hot time: 45.98 seconds
stream load tsv:          509 seconds loaded<PHONE_NUMBER>9 Bytes, about 140 MB/s
stream load json:         20 seconds loaded<PHONE_NUMBER> Bytes, about 112 MB/s
stream load orc:          65 seconds loaded<PHONE_NUMBER> Bytes, about 16 MB/s
stream load parquet:          30 seconds loaded 861443392 Bytes, about 27 MB/s
insert into select:          28.8 seconds inserted 10000000 Rows, about 347K ops/s
storage size:<PHONE_NUMBER>0 Bytes

run buildall

(From new machine)TeamCity pipeline, clickbench performance test result:
the sum of best hot time: 46.74 seconds
stream load tsv:          509 seconds loaded<PHONE_NUMBER>9 Bytes, about 140 MB/s
stream load json:         20 seconds loaded<PHONE_NUMBER> Bytes, about 112 MB/s
stream load orc:          64 seconds loaded<PHONE_NUMBER> Bytes, about 16 MB/s
stream load parquet:          31 seconds loaded 861443392 Bytes, about 26 MB/s
insert into select:          29.0 seconds inserted 10000000 Rows, about 344K ops/s
storage size:<PHONE_NUMBER>6 Bytes


no setup.py

how do I download this without a setup.py?

does it worked? any way? in 2018.. project seems compactible with old version of xrbl? plz help


Model improvement

How to modify it to support a model with two inputs?

What is needed is a model wrapper that simulates a model with one input.


iCal Export for non-recurring events not working

Describe the bug
When trying to export the iCal for a non-recurring event, the below exception is thrown:
[NullReferenceException: Object reference not set to an instance of an object.]
DotNetNuke.Modules.Events.VEvent.CreatevTimezones(DateTime dtStartDate, DateTime dtEndDate) +379
DotNetNuke.Modules.Events.VEvent.CreateiCal(Int32 iTabID, Int32 iModuleID, Int64 iItemID, Int32 socialGroupId, String iCategoryName, Int32 iCategoryID, String iLocationName, Int32 iLocationID, String calname) +3355
DotNetNuke.Modules.Events.VEvent.ProcessVCalRequest(HttpContext inContext) +1298
DotNetNuke.Modules.Events.EventVCal.Page_Load(Object sender, EventArgs e) +54
System.Web.Util.CalliEventHandlerDelegateProxy.Callback(Object sender, EventArgs e) +51
System.EventHandler.Invoke(Object sender, EventArgs e) +0
System.Web.UI.Control.OnLoad(EventArgs e) +92
System.Web.UI.Control.LoadRecursive() +54
System.Web.UI.Page.ProcessRequestMain(Boolean includeStagesBeforeAsyncPoint, Boolean includeStagesAfterAsyncPoint)
To Reproduce
Steps to reproduce the behavior:

Select an existing non-recurring event to open the Event Details
Click on 'Export' to export the iCal
See exception thrown

Expected behavior
An iCal for the event should be downloaded
Screenshots
If applicable, add screenshots to help explain your problem.
Additional context
I am using Events module version 07.00.03, and DNN version 9.2.0. I upgraded the module from version 06.03.00

I cannot reproduce this on my latest version of Events 07.00.03, so I assume it is resolved in the latest version


Monsterhearts 2

Added sheets for Monsterhearts 2nd edition.

Roll20 has a policy of only supporting one community sheet per game, unless the sheet is for a wildly popular game and is providing a different take on content functionality. Thank you for your work on this sheet, but we've selected another Monsterhearts 2 sheet to represent this game system. If you would like to contribute with this new sheet, you should contact the author of the other sheet and ask if they need any help. You can check out the other pull request here.
If you have any questions, you can message me on Roll20.


speed and comprehension testing for typography

Testing completed. See report at: https://github.com/gctools-outilsgc/design-system/blob/master/ux testing/Report_typography and data viz testing.pdf


Set max length is 3 digits, then type 4 digits. If I would like to delete whole current number in text input, I have to tap on button clear 4 times instead of 3 times as expected.

Set max length is 3 digits: LengthLimitingTextInputFormatter(3)
Then I enter: "1234". Obviously, text input just show "123".
But then I would like to delete "123" then:
Actual: I have to tap on button clear 4 times to delete whole available number ("123") in the text input.
Expected: Tap on button clear 3 times to delete whole available number ("123") in the text input.

Duplicate of #67898
Hey @PhongNgo1776 , I found an open issue addressing your problem. Please follow up there.
Thanks

Hi @PhongNgo1776
This issue is a duplicate of https://github.com/flutter/flutter/issues/67963 and it was already fixed by https://github.com/flutter/flutter/pull/67892.
The fix is on the latest master channel and it will be in the stable version of Flutter in the future.
Thank you


[CAMEL-14374] Camel-irc validation of endpoint's url fails for url

Issue: https://issues.apache.org/jira/browse/CAMEL-14374
Method sanitaze(), which contained a lot of logic about handling the uri, was removed.
Compo model was used instead.
This change led to remove one of the uri syntaxes (irc:nick@host[:port]/#room[?options])
Doc was updated and note to the upgrade guide was added.

@apupier I'll fix coma -> comma typos
and fix irc:nick@host[:port]?channels=#channel1,#channel2,#channel3[?options] to irc:nick@host[:port][?options]

@apupier All findings fixed.
@omarsmak All findings fixed. I've simplify use of "channels" in this way:

channels - comma separated list of channels
channelList - list of IrcChannel objects
(this names are used also for methods)


Startup errors unless worker indexing is disabled for .NET 8

          I wasn't sure if this warranted a separate issue but my function app is in East US 2 (using `FUNCTIONS_WORKER_RUNTIME = dotnet-isolated` + `netFrameworkVersion = v8.0` + `use32BitWorkerProcess = false`) and exhibiting the exact same error exceptions described by the OP in [#9686](https://github.com/Azure/azure-functions-host/issues/9686)

I managed to get around this by using <FunctionsEnableWorkerIndexing>False</FunctionsEnableWorkerIndexing> in my Functions csproj
Mentioning here as 9686 was closed with a request to follow on here.
Originally posted by @vexingpos in https://github.com/Azure/azure-functions-dotnet-worker/issues/2075#issuecomment-1872333050

Hi @vexingpos - could you please add the following information as well? Thanks!

What version of the core worker dependencies (Microsoft.Azure.Functions.Worker and
Microsoft.Azure.Functions.Worker.Sdk) are you using?
For hosted instances where you're seeing this behavior, what is the host version that you are seeing used?
Can you confirm the app is not hosted in an App Service Environment (ASE)?
If this is happening locally, what version of the Core Tools are you using?
Have you tried this in other regions? If so, was there any difference from East US 2?

And I know you mentioned seeing the same error as in another issue, but can you please share the specific traces or exception messages that you are seeing?

I haven't been able to reproduce this issue since, but at the time of the issue, I was using these core packages
"Microsoft.Azure.Functions.Worker" Version="1.20.0"
"Microsoft.Azure.Functions.Worker.Sdk" Version="1.16.3"

Hosted on a regular App Service. Current runtime version is <IP_ADDRESS>00 but this will likely have changed since my first issue report.
Works locally.

Thanks for the update @vexingpos
It is possible that your app was deployed to a region which did not get updated with a hosting config changes we applied (to enable the worker indexing feature e2e) in December. The issue must have been resolved after that hosting config deployment was done.
Going to mark this as closed as OP confirmed everything is working fine now. But if you ever run into issues with dotnet isolated function apps, don't hesitate to open a new issue with relevant details. Thank you!


2019-12-11 tc110 M3SS doesn't execute and generate parameters

This test case was tested with the openNCA application during the Milestone 1 E2E Evaluation and generated a JIRA issue EQ-2150 that indicates that the test case will not execute. The test case has been simplified somewhat here (1 dosing interval) since the original setup represented an invalid configuration. With the simplified config, the test case still will not generate parameters. Steps to run follow:
run with parameterset == "PARAMETERDISPLAYLIST"
# Since the dosing intervals overlap and there's no second dose defined, this is an "invalid" configuration
# remove TOLD2, TAU2 from dataset
d <- d[,-match(c("TAU2", "TOLD2"), names(d))]
parameterset <- "PARAMETERDISPLAYLIST"
parameterset
[1] "PARAMETERDISPLAYLIST"
> mct$OPTIMIZEKEL
[1] "0"
> mct$INCLUDEEXTRAPOLATION
[1] "1"
> mct$INCLUDEINTERPOLATION
[1] "1"
> mct$FLGNOCMAX
[1] "1"
results_list <- run_computation(data=d, map=mct, flag=flags, parameterset=parameterset)
TAU2  as defined in 'map', do not appear in input concentration dataset
TOLD2  as defined in 'map', do not appear in input concentration dataset
Error in value[[3L]](cond) :
  Error in auctau/tau: non-numeric argument to binary operator
For SDEID 49420991
In addition: Warning message:
In run_M3_SS_computation(data = merged_data, map = map_data, method = method,  :

 Show Traceback

 Rerun with Debug
 Error in value[[3L]](cond) :
  Error in auctau/tau: non-numeric argument to binary operator
For SDEID 49420991

run with parameterset=="PARAMETERLIST"
> parameterset <- "PARAMETERLIST"
> parameterset
[1] "PARAMETERLIST"
> results_list <- run_computation(data=d, map=mct, flag=flags, parameterset=parameterset)
TAU2  as defined in 'map', do not appear in input concentration dataset
TOLD2  as defined in 'map', do not appear in input concentration dataset
 Show Traceback

 Rerun with Debug
 Error in value[[3L]](cond) :
  Error in interpolate_log(conc1 = conc_1, time1 = time_1, conc2 = conc_2, : Error in interpolate_log: 'est_time' value is not numeric
For SDEID 49420991

As of 00cdcd3393 this issue can no longer be replicated when parameterset == "PARAMETERDISPLAYLIST".
A different error now occurs when parameterset == "PARAMETERLIST" shown below. Note that TAU2 and TOLD2 are removed from the dataset (e.g. d <- d[,-match(c("TAU2", "TOLD2"), names(d))])
 Error in value[[3L]](cond) :
  Error in `*tmp*`[[j]]: subscript out of bounds
For SDEID 49420991
7. stop(paste0(e, "For SDEID ", unique(data_data[, map_data$SDEID])[i])) at run_M3_SS_computation.R#2502
6. value[[3L]](cond)
5. tryCatchOne(expr, names, parentenv, handlers[[1L]])
4. tryCatchList(expr, classes, parentenv, handlers)
3. tryCatch({
    dof <- list()
    if (comp_required[["DOSECi"]] || comp_required[["DOSEC"]]) {
        dose_c_i <- list() ... at run_M3_SS_computation.R#972
2. run_M3_SS_computation(data = merged_data, map = map_data, method = method,
    model_regex = model_regex, parameter_list = parameter_list,
    return_list = return_list) at run_computation.R#2378
1. run_computation(data = d, map = mct, flag = flags, parameterset = parameterset)

This error no longer occurs as of 5a9f91aa0d.


Production issue

Hi! With the following code in development mode, everything works but when I go to production nothing works anymore:
const connection = new Connection(clusterApiUrl("devnet"));
  const metaplex = new Metaplex(connection);

  const nft = await metaplex.nfts().findAllByOwner({
    owner: publicKey,
  })

  console.log(nft)

I'm using ViteJS with this config
import { defineConfig } from "vite";
import react from "@vitejs/plugin-react";

// https://vitejs.dev/config/
export default defineConfig({
  plugins: [react()], // Or svelte(), etc.
  resolve: {
    alias: {
      stream: "rollup-plugin-node-polyfills/polyfills/stream",
    },
  },
  define: {
    "process.env": process.env,
  },

});

with this repo you can reproduce the problem:
https://github.com/florian-deroo/test-project


French translation contribution

Hi,
I read the part about multilingual support, I'm a French native speaker, let me know how I can help you :)

Hey @bidetaggle, thank you for your availability!
May you please tell us a bit more about you and how you plan to contribute?
You can start by reading CONTRIBUTING.md as well as PURPOSE.md
For contributing, you can look for an open issue (start maybe with good first issue) or:

Help translate all the website in French (this assume a big investment from your side; but also a good learning opportunity)
Add native support for the French language (again a big investment from your side)
Write a tutorial on how to use Texthero on French text/dataset

How much time can you invest in this project?
Regards,

Hi @jbesomi , I plan to contribute time to time when I feel like, for the pleasure of practicing/learning English and discover an open source project on github at the same time. I mainly want to focus on translation.

Help translate all the website in French

I can translate the actual website in French, I see that you already have /pages/en.
I can cp -r pages/en pages/fr then translate what will be prompted and send you a pull request. What do you think?

Add native support for the French language

What is it, more precisely?

Write a tutorial on how to use Texthero on French text/datase

I can translate an actual tutorial but not writing an original french tutorial yet.

Hey @bidetaggle, this seems all good!
For translating, I think we need to use Sphinx Internationalization. The first task is therefore to extract all the docstring text we need to translate ... not sure yet what's the best way to do that. Do you want to make some researches regarding this? :)


No detach option to execution in production envs

README file specifies that celerymon can be executed with --detach to run as a daemon in prod environments, but this option fails to be recognized by the command.
It would be good to reestablish support for this option or at least specify in the README a way to run it in a production environment. Maybe even something rather simple like:
$ nohup python manage.py celerymon &

Closed by #12


WSL: OmniSharp-rolsyn v1.35.3 project loading problem

Loading multi-project solutions with the latest version of the OmniSharp-roslyn server does not result in a working OmniSharp. The server works fine in single-project solutions, and when running without a solution.
This is easy to see when using a statusline displaying the projects loaded, as demonstrated in PR #598 (not yet available in master): only a single project is loaded, so we don't mark the solution as "loaded" until our timer expires.
I'll investigate this in #598. In the meantime, the simple workaround is to continue using v<IP_ADDRESS>:
:OmniSharpInstall v1.35.2

This is actually a WSL issue, reported in OmniSharp-roslyn #1844

OmniSharp-roslyn PR here: https://github.com/OmniSharp/omnisharp-roslyn/pull/1851

Fixed in v1.35.4, by https://github.com/OmniSharp/omnisharp-roslyn/pull/1857


Crash report

Crash reason:  SIGSEGV
Crash address: 0x0
Thread 0 (crashed)
0  phantomjs!JSC::Yarr::YarrPatternConstructor::setupDisjunctionOffsets [YarrPattern.cpp : 734 + 0x0]
eip = 0x08bb10f4   esp = 0xbff49200   ebp = 0xbff49298   ebx = 0x0a4f62bc
esi = 0xb66f0c80   edi = 0xb4f19a20   eax = 0xbff49618   ecx = 0x00000000
edx = 0xbff49480   efl = 0x00010282
Found by: given as instruction pointer in context
1  phantomjs!JSC::Yarr::YarrPatternConstructor::setupDisjunctionOffsets [YarrPattern.cpp : 701 + 0x16]
eip = 0x08bb17ee   esp = 0xbff492a0   ebp = 0xbff49338   ebx = 0x0a4f62bc
esi = 0xb66f0c80   edi = 0xb4f19a20
Found by: call frame info
2  phantomjs!JSC::Yarr::YarrPattern::compile [YarrPattern.cpp : 701 + 0x16]
eip = 0x08bbabec   esp = 0xbff49340   ebp = 0xbff495a8   ebx = 0x0a4f62bc
esi = 0xb554a500   edi = 0xb4f19a20
Found by: call frame info
3  phantomjs!JSC::Yarr::YarrPattern::YarrPattern [YarrPattern.cpp : 1008 + 0xe]
eip = 0x08bbb758   esp = 0xbff495b0   ebp = 0xbff495e8   ebx = 0x0a4f62bc
esi = 0xbff49654   edi = 0xbff49648
Found by: call frame info
4  phantomjs!JSC::RegExp::compile [RegExp.cpp : 105 + 0x30]
eip = 0x08cac276   esp = 0xbff495f0   ebp = 0xbff49688   ebx = 0x0a4f62bc
esi = 0x00000000   edi = 0xb54e0f00
Found by: call frame info
5  phantomjs!JSC::RegExp::create [RegExp.cpp : 87 + 0xe]
eip = 0x08cac887   esp = 0xbff49690   ebp = 0xbff496d8   ebx = 0x0a4f62bc
esi = 0xb54e0f00   edi = 0xb54e0f18
Found by: call frame info
6  phantomjs!JSC::RegExpCache::create [RegExpCache.cpp : 48 + 0x1f]
eip = 0x08cb0dcd   esp = 0xbff496e0   ebp = 0xbff49748   ebx = 0x0a4f62bc
esi = 0xbff49830   edi = 0xb5791394
Found by: call frame info
7  phantomjs!JSC::RegExpCache::lookupOrCreate [RegExpCache.cpp : 41 + 0x33]
eip = 0x08cb16cc   esp = 0xbff49750   ebp = 0xbff497d8   ebx = 0x0a4f62bc
esi = 0xbff49830   edi = 0xb5791394
Found by: call frame info
8  phantomjs!JSC::constructRegExp [RegExpConstructor.cpp : 320 + 0x25]
eip = 0x08cab2ff   esp = 0xbff497e0   ebp = 0xbff49848   ebx = 0x0a4f62bc
esi = 0xb4d2d9c8   edi = 0xb4414550
Found by: call frame info
9  phantomjs!JSC::constructWithRegExpConstructor [RegExpConstructor.cpp : 329 + 0x12]
eip = 0x08cab735   esp = 0xbff49850   ebp = 0xbff49888   ebx = 0x0a4f62bc
esi = 0xb58001b0   edi = 0xb4414550
Found by: call frame info
10  phantomjs!JSC::cti_op_construct_NotJSConstruct [JITStubs.cpp : 2325 + 0x5]
eip = 0x08c37a44   esp = 0xbff49890   ebp = 0xbff49908   ebx = 0x0a4f62bc
esi = 0xb4414550   edi = 0xb4414550
Found by: call frame info
11  0xb4d0d4ea
eip = 0xb4d0d4eb   esp = 0xbff49910   ebp = 0xbff49958   ebx = 0xb4c85960
esi = 0x000001fc   edi = 0xb5800158
Found by: call frame info
12  phantomjs!JSC::Interpreter::execute [JITCode.h : 77 + 0x25]
eip = 0x08c02b41   esp = 0xbff49960   ebp = 0xbff499e8
Found by: previous frame's frame pointer
13  phantomjs!JSC::evaluate [Completion.cpp : 64 + 0x2d]
eip = 0x08147943   esp = 0xbff499f0   ebp = 0xbff49a48   ebx = 0x0a4f62bc
esi = 0xb4c3d2c8   edi = 0xb5db4910
Found by: call frame info
14  phantomjs!WebCore::ScriptController::evaluateInWorld [JSMainThreadExecState.h : 54 + 0x20]
eip = 0x0817ade9   esp = 0xbff49a50   ebp = 0xbff49af8   ebx = 0x0a4f62bc
esi = 0x0a51a354   edi = 0x00000000
Found by: call frame info
15  phantomjs!WebCore::ScriptController::evaluate [ScriptController.cpp : 167 + 0x1e]
eip = 0x0817b0d9   esp = 0xbff49b00   ebp = 0xbff49b18   ebx = 0x0a4f62bc
esi = 0xbff49b44   edi = 0xb6631388
Found by: call frame info
16  phantomjs!WebCore::ScriptElement::executeScript [ScriptElement.cpp : 283 + 0x1a]
eip = 0x08d645ed   esp = 0xbff49b20   ebp = 0xbff49b58   ebx = 0x0a4f62bc
esi = 0xb55cb984   edi = 0xb6631388
Found by: call frame info
phantomjs!WebCore::HTMLScriptRunner::executePendingScriptAndDispatchEvent [HTMLScriptRunner.cpp : 139 + 0xe]
eip = 0x0841b8dc   esp = 0xbff49b60   ebp = 0xbff49be8   ebx = 0x0a4f62bc
esi = 0xb54d8200   edi = 0xb54d81f8
Found by: call frame info
18  phantomjs!WebCore::HTMLScriptRunner::executeParsingBlockingScript [HTMLScriptRunner.cpp : 118 + 0x14]
eip = 0x0841cf91   esp = 0xbff49bf0   ebp = 0xbff49cb8   ebx = 0x0a4f62bc
esi = 0x00000000   edi = 0xb54d81f8
Found by: call frame info
19  phantomjs!WebCore::HTMLScriptRunner::execute [HTMLScriptRunner.cpp : 195 + 0x7]
eip = 0x0841d523   esp = 0xbff49cc0   ebp = 0xbff49cd8   ebx = 0x0a4f62bc
esi = 0xb54d8200   edi = 0xb54d81f8
Found by: call frame info
20  phantomjs!WebCore::HTMLDocumentParser::runScriptsForPausedTreeBuilder [HTMLDocumentParser.cpp : 205 + 0x15]
eip = 0x08411bd3   esp = 0xbff49ce0   ebp = 0xbff49d18   ebx = 0x0a4f62bc
esi = 0xb5448000   edi = 0xbff49cfc
Found by: call frame info
21  phantomjs!WebCore::HTMLDocumentParser::canTakeNextToken [HTMLDocumentParser.cpp : 223 + 0x7]
eip = 0x08411cda   esp = 0xbff49d20   ebp = 0xbff49d38   ebx = 0x0a4f62bc
esi = 0xb5448000   edi = 0xbff49d8c
Found by: call frame info
22  phantomjs!WebCore::HTMLDocumentParser::pumpTokenizer [HTMLDocumentParser.cpp : 261 + 0x15]
eip = 0x08412f4c   esp = 0xbff49d40   ebp = 0xbff49db8   ebx = 0x0a4f62bc
esi = 0xb544805c   edi = 0xb5448000
Found by: call frame info
23  phantomjs!WebCore::HTMLDocumentParser::notifyFinished [HTMLDocumentParser.cpp : 175 + 0xf]
eip = 0x08413b5d   esp = 0xbff49dc0   ebp = 0xbff49de8   ebx = 0x0a4f62bc
esi = 0xb5448000   edi = 0x00000001
Found by: call frame info
24  phantomjs!WebCore::CachedResource::checkNotify [CachedResource.cpp : 144 + 0xb]
eip = 0x0848c81c   esp = 0xbff49df0   ebp = 0xbff49e38   ebx = 0x0a4f62bc
esi = 0xb55cdd80   edi = 0xbff49e18
Found by: call frame info
25  phantomjs!WebCore::CachedScript::data [CachedScript.cpp : 104 + 0x7]
eip = 0x0848f42e   esp = 0xbff49e40   ebp = 0xbff49e58   ebx = 0x0a4f62bc
esi = 0x00000000   edi = 0xb55cdd80
Found by: call frame info
26  phantomjs!WebCore::CachedResourceRequest::didFinishLoading [CachedResourceRequest.cpp : 166 + 0x36]
eip = 0x08e324e3   esp = 0xbff49e60   ebp = 0xbff49e98   ebx = 0x0a4f62bc
esi = 0x0848f3a0   edi = 0xb55c8570
Found by: call frame info

Please upgrade to PhantomJS 2. 1.9.x is no longer supported, and known to have many crasher bugs that are fixed in 2.0.


fix: :bug: resolve css file path relative to file & add demo.css to spec file for pyinstaller

Fixes issues-816
Also fixes issue reported in discord - exe did not have demo.css included
gets demo.css file path relative from index.pv __file__ parent rather than current working dir
add demo.css to spec file datas so its included in root folder of pyinstaller exe

@Shukla-Gaurav  please review and merge

@Shukla-Gaurav tested the changes. Thanks, @FraserHum for the patch.


ChromeKimaiBridge for Nextcloud Deck

Hello @tobybatch
I think it would be a great idea to expand your plugin for Nextcloud Deck as well. Now Nextcloud Deck is finally released and very popular. We started to use it successfully in our work and are very happy with it.
This can be added as follows
https://mydomain.com/index.php/apps/deck/#/board/1/card/140
where
index.php/apps/deck - nextcloud deck indicator (analogy hostname == "trello.com")
board/1 - project ID
card/140 - card ID
It would also be an interesting feature to add a meta field with a link to a card.
What do you think of that?
Thanks!

This is quite a big change.  We'll need to add an setting to the options page that records your domain name so we can detect that page.  We'll need to add a new code block that will be triggered and figure out the correct board/card names to pass to the plugin.
If you want to contribute then I can help you find the correct locations to alter the code but I don't have a Next cloud deck to test/dev against, so I'd not really be able to work on this ATM.
I'll keep this open as adding more clients does seem like a good idea.

Great. I'll try to add my contribution soon. I wanted to make sure you were interested in developing the plugin. You're right to add domain information to the settings. Thank you.
p.s. Sorry if my English isn't so good.

No worries about the English.
I'll create a branch and stub up some pseudo code, give me a little time.

Can you supply me with a sample URL from a Nextcloud deck 'card' please?
I'm thinking about adding support for domain/reg ex to get the card id's

Please see your email

Hello @tobybatch
Did you have time to see the nextcloud deck?
Have a nice day.

Yes, thanks. I'll think about how I patch the plugin.

@krestyanikov Can you take alook at the branch 9-add-support-for-extra-services
You'll need to pull the repo and checkout the new branch and replace the chrome extension but I've added a settings page on /chrome/settings that lest you add host names and regex's to enable any host.

Hello @tobybatch , your regex idea is amazing!
I have experienced some difficulties, but it works! I will not create a commit, but I will describe it right here with your permission.

At first I tried it, but I got a mistake
[2020-08-07 12:25:14] php.CRITICAL: Uncaught Error: Argument 1 passed to KimaiPlugin\ChromePluginBundle\Entity\SettingEntity::setRegex2() must be of the type string, null given, called in /home/krestya5/public_html/kimai2/var/plugins/ChromePluginBundle/Controller/ChromeController.php on line 467 {"exception":"[object] (TypeError(code: 0): Argument 1 passed to KimaiPlugin\ChromePluginBundle\Entity\SettingEntity::setRegex2() must be of the type string, null given, called in /home/krestya5/public_html/kimai2/var/plugins/ChromePluginBundle/Controller/ChromeController.php on line 467 at /home/krestya5/public_html/kimai2/var/plugins/ChromePluginBundle/Entity/SettingEntity.php:60)"} []
I easily fixed it

Next, when I tried to transfer a link from the plugin, I found that it was interrupted on the Score symbol
had to encode the URL in the plugin

The link was received correctly, but only the board code was transmitted
/index.php/apps/deck/#/board/14/card/133
Returning board id=14, card id= [] []
for some reason $matches[1] was not obtained through regex.
I used the following settings and it worked!

Returning board id=14, card id=133 [] []
Thank you very much for your work! There are some other problems that I've discovered. I'll describe them in other Issues

Can you try pulling the latest version, on branch 9-add-support-for-extra-services
There is now a test button next to the config entry that should allow you to run a test URI through the service that parses the URL

Hello @tobybatch
I checked for changes. I noticed a couple of problems

The return of two variables does not work. I changed it.

When testing, there is no way to leave the field blank.

if I insert an empty value into Regex2, I get an error.

Have a nice day.

@krestyanikov Sorry for the delay, I've been away for a bit.  Back now, I'll take a look this week.

@krestyanikov I'm back with another (better) iteration that seems to work for me.  Again you'll need to pull the 9-add-support-for-extra-services branch in the var/plugin/ChromePluginBundle and reload the crx that's in that branch.
I think this should work with nextcloud deck.

Hello @tobybatch
It works fine! You will get more stars and feedback when we start using the plugin in production.
I noticed a slight inaccuracy, but I'm not sure if it's an error.
In the list of time entries, the description is activity, but the description entries are not displayed

I also have other suggestions that I will publish separately.
p.s. do You have a donation channel?
Thanks!

Hello @tobybatch
Can you install a new version of the plugin in the chrome store?
Have a nice day.

Hi @krestyanikov I'll get on this asap.  I need to be careful that I keep backward compatibility though.  I'll put some time in to this soon.


deprecate this in favour of adding reader to napari-btrack plugin?

Looks like this (the transferring) has already been done. Closing


[13pt] Add levee masking to ras2fim DEMs

I think this could be a good pre-processing task for our DEMs, using the NLD levee data that FIM uses to mask out the DEMs before their use in ras2fim.


"Menu" component missing

Before submitting a pull request, make sure it's targeting the right branch:

For fixes to Ink 1.0, use master.
For fixes to Foundation for Emails 2, use v2.0.

Happy coding! :)
Add link to the menu.html file to the navigation on http://foundation.zurb.com/emails/docs/

Thanks!


test: run slow tests without race detector in CI

Should speed up test execution, at the expense of output legibility.

Well, that didn't really help much. Locally I'm seeing 70-90% reductions in test time, but then I also have an 8-core machine. GitHub actions uses 2 cores, I think.
Will try using default parallelism (2) in case the additional jobs just cause more overhead.

Superseded by #338.


install.sh script failed with "usage: sleep seconds"

Describe the bug
The install.sh failed with this message on OSX 12.x but not in OSX 11.x
Pods found with selector '--field-selector metadata.name=etcd' are not ready yet. Waiting 10 secs...
usage: sleep seconds

To Reproduce
Following the Quick Start guide to run a ./scripts/install.sh --namespace modelmesh-serving --quickstart command.
Expected behavior
Installing ModelMesh Serving built-in runtimes
Environment (please complete the following information):

OS: OSX 12.1

It's probably fine to just use sleep 10 since the default unit of time is seconds for most implementations of the sleep util I believe.


[Typo]: Cannot debug the react source code through 'babel-standalone/dev.html' file

Summary
Follow this link to debug the react source code yet failed:
https://legacy.reactjs.org/docs/how-to-contribute.html
Steps:
1).  Make no changes to main branch and just run the yarn build command
2). Open the fixtures/packaging/babel-standalone/dev.html file which includes your changes, but meet the errors from browser console as below:
GET file:///Users/quinvon/Documents/GitHub/react/build/oss-experimental/react/umd/react.development.js net::ERR_FILE_NOT_FOUND
dev.html:4

       GET file:///Users/quinvon/Documents/GitHub/react/build/oss-experimental/react-dom/umd/react-dom.development.js net::ERR_FILE_NOT_FOUND
babel.js:61666 You are using the in-browser Babel transformer. Be sure to precompile your scripts for production - https://babeljs.io/docs/setup/
runScripts @ babel.js:61666
transformScriptTags @ babel.js:336
(anonymous) @ babel.js:327
Inline Babel script:2 Uncaught ReferenceError: ReactDOM is not defined

I think your debug way of source code is out of date!
Page
https://legacy.reactjs.org/docs/how-to-contribute.html
Details
No response

There is a warning at the top of the page.

@Rekl0w, I saw it, and where's the new guidance about how to debug the source code?


HMR not working with Svelte when custom preset files are changed

UnoCSS version
0.50.6
Describe the bug
Making changes to any UnoCSS config files / preset files (like shortcuts or rules) requires the server to be restarted for the changes to appear, so HMR doesn't seem to be working. I am using Svelte and SvelteKit.
My config files look like this. I am creating a custom preset with my own rules and shortcuts. In the vite.config.ts file I am pointing to the UnoCSS config file.
unocss.config.ts
import {
  defineConfig,
  presetAttributify,
  presetUno
} from 'unocss';

import { extractorSvelte } from '@unocss/core';

import { presetCustom } from './src/preset/index';

export default defineConfig({
  extractors: [extractorSvelte],
  presets: [
    presetUno(),
    presetAttributify(),
    presetCustom()
  ]
})

vite.config.ts
import { sveltekit } from '@sveltejs/kit/vite';
import { defineConfig } from 'vite';
import UnoCSS from 'unocss/vite';

export default defineConfig({
	plugins: [
		sveltekit(),
		UnoCSS('./unocss.config.ts'),
	]
});

Reproduction

System Info
No response
Validations

[X] Read the Contributing Guidelines.
[X] Check that there isn't already an issue that reports the same bug to avoid creating a duplicate.
[X] Check that this is a concrete bug. For Q&A open a GitHub Discussion or join our Discord Chat Server.
[X] The provided reproduction is a minimal reproducible example of the bug.

Slight update: HMR works when I put all the configurations in vite.config.ts but not when I just point to the UnoCSS config like so: UnoCSS('./unocss.config.ts')

@bennymi I haven't done anything on basic Svelte + UnoCSS Vite plugin usage and I have a lot of things I'd like to do with svelte-scoped first but if no one else solves your problem, I'll come back to this. Consider adding a Stackblitz repro in the meantime.

@jacob-8 sorry for the late reply, for some reason I didn't see the notification. I created a Stackblitz so you can have a look at it:
https://stackblitz.com/edit/sveltejs-kit-template-default-vpnz8h?file=src/routes/+page.svelte
Essentially this is the setup.

A custom preset is created in unocss-custom-preset.js which is then added to the unocss.config.js file
I added some simple shortcuts to the custom preset
When the server is running and you add a new shortcut and then add it as a class (in the /src/routes/+page.svelte for example, you don't see the styles being added
The styles only get added once you restart the server, so HMR is not working

You can try it out by just adding a new shortcut and then adding it somewhere.

Same solution as in #2613 - as to your repro, just update the vite.config.js file:
import { sveltekit } from '@sveltejs/kit/vite';
import { defineConfig } from 'vite';

import UnoCSS from 'unocss/vite';

export default defineConfig({
	plugins: [
		sveltekit(),
		UnoCSS({
			configFile: './unocss.config.js',
			configDeps: ['./unocss-custom-preset.js']
		})
	]
});


FingerprintAuth is not defined

I think I'm missing something simple. I added the plugin to my meteor project. I'm try to call  isAvailble when I click a button
test() {
		console.log('test');
		FingerprintAuth.isAvailable( function (result) {
			console.log(result);
		});

	}

but I get FingerprintAuth is not defined. Do I need an import statement. What am I missing?

Nevermind. Testing on the desktop where the plugin is not available.


Does VRChat not support splay?

Hi, no it doesn't. But the driver exposes splay from osc outputs which you can then use in vrchat.

Hi, no it doesn't. But the driver exposes splay from osc outputs which you can then use in vrchat.

thx


Question - how to enable sandbox

Hi,
newbie here :) How can I force the usage of the exchange sandbox? I would like to test a bit before trading with real cryptomoney..
Thanks!
Luigi

Hi, Luigi!
Most often you should just change the url of the exchange to sandbox url, like shown in the example bellow:
    // move to testnet/sandbox if possible
    if (exchange.urls['test'])
        exchange.urls['api'] = exchange.urls['test'];

The caveat here is that the test URLs are not fully covered in ccxt yet, so you will have to put correct URLs there. But if you feel like you want to help us to unify that functionality and you want contribute by adding the test URLs where they are missing – we will be happy for your help. Let us know if this answers your question.

Thanks! I will try

I'm closing this for now, let us know if you have difficulties with it, feel free to reopen this if needed, or just ask your questions! Cheers!


Prepare CHANGELOG for 0.10.0 release

[0.10.0] - 2022-08-10
Added

RIGA-301: Add increased memory limits and timeouts to .env file.

Changed

RIGA-282: Update drupal/core 9.3.19 => 9.4.5.
RIGA-282: Update core patch for issue 1356276.
RIGA-297: Save config for Acquia search view after db update.
RIGA-301: Comment out functional tests from develop script.

@swiftwinning Please make the version 0.9.12 and set the release date for tomorrow


[issue-may-be] port or targetPort

in pkg/controller/main.go
func: getServices
since the real_server in /etc/keepalived/keepalived.conf is POD IP, instead of cluster IP,
so we should use Spec.TargetPorts instead of Spec.Ports ?
since TargetPort is the container's port, and Port is the cluster IP port.(https://kubernetes.io/docs/concepts/services-networking/service/)
for _, servicePort := range s.Spec.Ports {

}

Closing. No, the definition in the configmap is about ports, no targetPorts. We lookup targetPorts in the getEndpoints function

oh, sorry for that. My bad

Hmm...
I think I should point to this one:
			svcs = append(svcs, vip{
				Name:     ...
				IP:        ...
				Port:      int(servicePort.Port),  // --> here, should it be servicePort.TargetPort ?
				LVSMethod: ...,
				Backends:  ..,
				Protocol:  ...,
			})

https://github.com/aledbf/kube-keepalived-vip/blob/master/pkg/controller/main.go#L238


Upgrade box to jessie

I created a WIP branch "jessie". I only took care of the python and postgresql roles which seem to be working. You won't be able to test for the moment because I still need to upload the box. Anyway, here's what's left to be done:

[ ] test & update roles
[ ] create virtualbox box
[ ] fix "halt" issue (see https://github.com/mitchellh/vagrant/issues/5497 and https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=751636)

Great job thanks !
Concerning the systemd issue, we could maybe reinstall sysv in the box, it's not that complicated.
Also maybe we could take the opportunity to put most of the "base" packages directly in the box to gain some time ?

Maybe it could be worth going back to sysv since the bug doesn't seem to be about to be fixed soon.
As for the base packages I agree, but we should make sure we have a playbook for that, to make sure the same base packages are installed in every box.

Reviewing the issue with the 'halting' it seems the fix is as simple as running those command once in the box :
systemctl disable ssh.service
systemctl enable ssh.socket
Maybe just add that to the provisioning and consider it done ?

I tried it on a local box but it didn't solve the issue for me.

Fixed in 81eff7c6f0cf4fb838cb51f6e514c3e067739cb7.


Support log level and request logs

Adds support for:

configuring the logger level via the LOG_LEVEL env var
request logging (Fixes #593).  Can be disabled with LOG_DISABLE_REQUEST_LOGGING
a. really simple and lightweight format.  Personally, i found the built-in request logging too awkward and this plugin to be a bit more than what I'm looking for.

example logs from running (dockerized) locally:
server_1          | {"level":30,"time":1667871767331,"pid":1,"hostname":"b18cd7df7dd1","msg":"[200] (<IP_ADDRESS>) POST /graphql"}

@kamilkisiela any thoughts :)?

@kamilkisiela Thoughts on the flag for disabling request logging? I think request logging should be determined via the log level (info) and not an additional option.

I think we could show operation's name as well and the value of the request_id header and I guess making it use info level is fine.
I know I said it should be opt-in but on the other hand it does not bring much noise to the logs, so maybe having it enabled by default (with no option to opt-out) is actually the best option. If you all agree, then I will create a PR to your PR @rperryng to remove the flag LOG_DISABLE_REQUEST_LOGGING flag.

I removed the flag to disable request logging :).
With latest changes, the log message now looks like so:
server_1          | {"level":30,"time":1668522424590,"pid":1,"hostname":"5b5d04c96b55","msg":"[200] (<IP_ADDRESS>) POST /graphql 'organizationActivities' (+6CDHcG1SB+PXYOPpJiD1A/0000000004)"}

Is this close to what you had in mind @kamilkisiela ?

@n1ru4l thanks for your feedback, it should be addressed now - let me know if I misunderstood anything

@rperryng https://github.com/rperryng/graphql-hive/pull/1

@kamilkisiela thanks!  merged your changes 🙂
I figured there must have been a way to avoid re-parsing since all the envelop servers are already doing it, but yeah, wasn't sure how to slide it in the current config 😅.  Maybe I can give it a look when graphql-hive has moved off of fastify


Don't stub out WebSocket by default

Would it be possible to not stub out the native WebSocket by default? I am creating an application where I want to use both mock and real sockets. Is there a current work-around for this?

I'm interested in this as well. Any suggestions?

Fixed in 9.1.0
Example: new Server('ws://localhost:8080', { mock: false });


"Couldn't find key mandatory in dynamic object"

Hi,
When I run this project I get this error:

It's related to the if test here:

Do you know what could be wrong?
Link to the if test
Running on Samsung S7.
React native version 0.29.2
The error occurs when I try to run https://github.com/oney/RCTWebRTCDemo

Its related to this: https://github.com/oney/react-native-webrtc/issues/145


refactor: add region to aws s3 sync instruction

:tada: This PR is included in version 1.2.3 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Reduce duplication in jinja2 templates

As pages have been introduced over time, they've often liberally borrowed from other templates with the same desired layout/behavior. As such, we are now moderately afflicted with a case of templatus duplicatus. It would likely be worthwhile to do a readthrough of the current templates to see where common functionality could be extracted and packaged into a shared template.
See https://github.com/cfpb/cfgov-refresh/pull/5523#issuecomment-594035756 for two examples of template duplication.

Related [ghe]/CFGOV/platform/issues/3226


Feature/cmpv2 api - add api/error handling

Feat:

add api processing (controller/service)
add doGet processing (return badRequest message)
naive/simple error handling (for future using)

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.Tomas Cejka seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.


querydsl.packageSuffix messes with QueryDslJpaRepository

By introducing <querydsl.packageSuffix>.query</querydsl.packageSuffix> I am getting the following exception (for brevity sake, I have eliminated other lines).
Caused by: java.lang.IllegalArgumentException: Did not find a query class a.b.c.d.QEntity for domain class a.b.c.d.Entity!
	at org.springframework.data.querydsl.SimpleEntityPathResolver.createPath(SimpleEntityPathResolver.java:63)
	at org.springframework.data.jpa.repository.support.QueryDslJpaRepository.<init>(QueryDslJpaRepository.java:85)
	at org.springframework.data.jpa.repository.support.QueryDslJpaRepository.<init>(QueryDslJpaRepository.java:69)

Note this wasn't an issue before introducing this suffix.
I still see the query classes getting generated at the desired path using the suffix.
The following is my configuration
  <!-- QueryDSL Query Classes Generation -->
      <plugin>
        <groupId>com.mysema.maven</groupId>
        <artifactId>apt-maven-plugin</artifactId>
        <version>1.1.3</version>
        <executions>
          <execution>
            <goals>
              <goal>process</goal>
            </goals>
            <configuration>
              <outputDirectory>${project.build.directory}/generated-sources/java</outputDirectory>
              <processor>
                com.querydsl.apt.jpa.JPAAnnotationProcessor,lombok.launch.AnnotationProcessorHider$AnnotationProcessor
              </processor>
              <options>
                <querydsl.packageSuffix>.query</querydsl.packageSuffix>
              </options>
            </configuration>
          </execution>
        </executions>
      </plugin>

The dependencies are
 <dependency>
      <groupId>com.querydsl</groupId>
      <artifactId>querydsl-apt</artifactId>
      <version>${querydsl.version}</version>
    </dependency>
    <dependency>
      <groupId>com.querydsl</groupId>
      <artifactId>querydsl-jpa</artifactId>
      <version>${querydsl.version}</version>
    </dependency>

where querydsl.version is 4.1.4.
QueryDslJpaRepository is coming from org.springframework.data:spring-data-jpa:1.11.7.RELEASE
Please let me know what other version of dependencies do you need but I am guessing it might be straightforward to check if QueryDslJpaRepository respects querydsl.packageSuffix?

I have the same problem. It is working only if query class is in the same package as entity. But whats the purpose of "querydsl.packageSuffix" option if its not working with different subpackages?

querydsl.packageSuffix is a way of structuring the generated entities for human consumption.
Spring Data is machine consuming the entities, so I wouldn't be surprised if you shouldn't touch the expected package.
Could you ask the people over at Spring Data whether it's even supported?
In the meantime I would leave it as default.

Yes it is the problem with Spring Data which currently does not support query classes in different packages than entity class. By default JpaRepositoryFactory is creating QueryDslJpaRepository with SimpleEntityPathResolver. And this resolver doesnt support different packages as you can see here

In Spring Data configuration you can provide your custom implementation of JpaRepositoryFactory with factory-class attribute in xml settings:
<jpa:repositories factory-class="x.y.z"/>
This short tutorial shows you howto.

@chosema I didn't quite follow the logic of creating this new factory class for Query class resolution, I expect this parameter querydsl.packageSuffix to be supported out of the box and not when I need to twist things. @Shredder121 I think based on the clear evidence given by @chosema (Thanks!) it should be easy to file an issue with them to support this parameter, otherwise using the querydsl.packageSuffix is really a no-go for folks.

FYI I have filed the issue here : https://jira.spring.io/browse/DATAJPA-1234

Ah, thanks
Although still, the package configuration would have to be kept in sync
But we'll see what they say, whether they want to support it at all or not
:+1:

I'm looking into this from the Spring Data side. Tweaking the method to consider a package suffix shouldn't be much of a problem. But the question is: do we have a way to determine the suffix without forcing the user to configure it again somewhere in Spring Data?

But the question is: do we have a way to determine the suffix without forcing the user to configure it again somewhere in Spring Data?

I don't think so, hence my "Although still, the package configuration would have to be kept in sync".

But the question is: do we have a way to determine the suffix without forcing the user to configure it again somewhere in Spring Data?

I don't think so, hence my "Although still, the package configuration would have to be kept in sync". Apt takes the options at compile time.

Why not under different packages? You just leave the problem to spring data? To pass the buck and turn off the problem directly.

In the same newspaper, the name also conflicts with the entity starting with Q, which is a confusing name and design method

Spring data also does not provide a simple configuration method. So what a pain and hard to use thing!!!!!


Skip default mapping

Could you add flag for skip default mappings?

Ya, it's a good idea. I will quickly implement that feature

https://github.com/dinhhuy258/git.nvim/pull/11 this pull request has resolved the problem. Please refer to the README.md for the new configuration custom_only flag. Thank you

@dinhhuy258 Thanks!
IMHO in README, this configuration flag is not obvious.

Hi @mario-amazing
Thank you. Do you have any suggestions for me?
I am not a native English speaker; it would be good if you could help me choose a meaningful configuration flag.

I'm not native as well))
So I created PR(above) which looks more appropriate in my opinion. If you have any notes, feel free to tell me in PR

Ya, thank you...


VS Code connects to education.github.com despite Telemetry being disabled

Does this issue occur when all extensions are disabled?: Yes/No
Yes

Version: 1.60.1
OS: macOS 11.6
Steps to Reproduce:

Disable telemetry:

"telemetry.enableCrashReporter": false,
"telemetry.enableTelemetry": false,

Disable experiments:

"workbench.enableExperiments": false,

Restart VS Code

Despite the setting being disabled, the application connects to education.github.com (which itself begs more questions, but that's a whole other thing). This is (presumably) the offending code which doesn't appear to check whether the appropriate settings are enabled.

Thanks for the report. I will definitely look into fixing this as we don't need to make this request, but to ease some concerns the telemetry doesn't get sent to Microsoft. We initiate the connection to education.github.com to see if you're a student or faculty using VS Code so we can better gauge how to make VS Code accessible to those in a classroom setting. However, the this._telemetryReporter.sendTelemetryEvent no-ops because it goes to check your telemetry setting and realizes it is disabled. You can find that NPM module here https://github.com/microsoft/vscode-extension-telemetry

Thanks for this @lramos15 !


build: Update xpp module

Fixes developer warnings in cmake 3.17+
Ref: polybar/xpp#24

Hum it doesn't seem to compile on travis

That's weird. This is actually the invocation of find_package_handle_standard_args where I barely changed anything and according to the cmake docs my change shouldn't have changed any functionality.
I will try to debug this in a VM with an older cmake version.

:man_facepalming: Older cmake version interpret the NAME_MISMATCHED flag as a variable it needs to check instead of as a flag.
I guess we're not getting around using FPHSA_NAME_MISMATCHED. I will quickly push that change to xpp and then update this PR

I hope it works for all versions x)

It should the function signature I used exists since cmake 3.0


BulkRead (auto) remove non-existent items.

Is it possible to (auto) remove items which are NOT present in the database?
E.g. myIds contains 1234 (an Id which doesn't exists in the database).
After executing the BulkRead method, the items list still contains the item...
Does EFCore.BulkExtensions provide a bulkConfig to remove the non-existent items?
var items = myIds.Select(tmpId => new Item() { Id = tmpId }).ToList();
var bulkConfig = new BulkConfig { SetOutputIdentity = true, SqlBulkCopyOptions = SqlBulkCopyOptions.KeepIdentity};
_context.BulkRead(items, bulkConfig);

I think the question/ request  will be resolved by pull request Added ReplaceReadEntities property to BulkConfig (#571). #946

This would not be base feature of the lib.
Better to implement it by overriding the BulkRead method, similir example just for Audit: https://github.com/borisdj/EFCore.BulkExtensions/issues/56


Dupe with "Instant Pickup" and double-tall plants when using shears

Versions

Minecraft 1.16.5
Forge 36.0.40
Forgery 1.3.3

Description
When using shears while the "Instant Pickup" feature is enabled, block drops from 2-tall plants are duplicated. This happens with the flowers (sunflower, lilac, rose bush, peony all drop 2x instead of 1x) as well as tall grass and large ferns (4x grass / fern instead of 2x). This also appears to happen with some modded 2-tall plants, as I've noticed with the rice plant from Farmer's Delight.
Debug Log
https://gist.github.com/copygirl/730e46ee356c1566f65fcbcf112eddce

Someone else has reported this happening on Fabric as well.


Testing CDK version to 1.99.0

Codecov Report

Merging #212 (82a3b5b) into master (5659a9b) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #212   +/-   ##
=======================================
  Coverage   97.29%   97.29%
=======================================
  Files           3        3
  Lines          74       74
  Branches       15       15
=======================================
  Hits           72       72
  Misses          2        2

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 150e44f...82a3b5b. Read the comment docs.


Compilation target windows 7 x32

Hi!
Can anybody help me with compiling deta space cli targeting windows 7 (32 bit)?
I've tried to cross-compile on ubuntu 22.04 using both standard and mingw toolchains. Unfortunately, binaries fail in space new with "can not switch console to raw mode" error. Also space version works fine.
p.s.
Am i missing non-interactive space cli mode?

why do you need 32 bit? that's pretty outdated.
the cli has a non-interactive mode. just provide the required args. let me know where you have issues..

Thanks. Seems like I've missed these options.

why do you need 32 bit? that's pretty outdated

Sure, I use 32-bit laptop. Hence need to cross-compile under linux server and stuck at pty problem in windows console.


Add inline doc in the comments for io_uring_prep_slice

Looks good, except s/slice/splice in the commit. Can you fix that?

@axboe Thanks for pointing out the typo in commit message, I've already fixed that.


Generate the "models-summed.csv" and "language-summed.csv" files based on the "evaluation.csv" file

Part of #237

Based on https://github.com/symflower/eval-dev-quality/pull/240

@bauersimon Please, take a final look


Missing original script path in error message

Description of Bug
Color is not defined https://stackblitz.com/edit/node-br9gyh
Steps to Reproduce

Make a new project and install reejs npm i reejs and try to run it via a npm script at package json:

{
"runit": "reejs init"
}

Run the script as npm run runit

Expected Behavior
Reejs Initializing the current project
Screenshots/Screencast

Additional Context/Questions
FYI this framework is mine, and I have tried my best to support stackblitz, but since the error shows from the domain of stackblitz without any context to reejs framework itself, I think this bug is related to stackblitz itself. Still I am not sure though!

I transferred this issue to webcontainer-core. I'm also renaming it to change the scope a bit. We probably can't fix the actual error that happens (unless it's a bug on our side, but it doesn't appear to be at this time). But we should print a more informative error, with the path of the file where the original error happened.

I transferred this issue to webcontainer-core. I'm also renaming it to change the scope a bit. We probably can't fix the actual error that happens (unless it's a bug on our side, but it doesn't appear to be at this time). But we should print a more informative error, with the path of the file where the original error happened.

👍

Any updates?

Hey there team, looks like this issue has been fixed, but I cant say the same because... the way import is being handled here, is not great...

(the import file is being read from /home/projects/[project name]/<file> ) Can we quickly have a fix that searches for files stored locally instead of trying to import from url? Fyi the file is at https://github.com/rovelstars/reejs/blob/master/server/import.js

In brief, any suggestions/ideas to fix on absolute file importing?

import path from "./path.js";
import "../polyfill/process.js";
import { fileURLToPath } from "./url.js";
const __filename = fileURLToPath(import.meta.url);
let __dirname = path.dirname(__filename).split("/").slice(0, -1).join("/");
const originalEmit = process?.emit;
process.emit = function (name, data, ...args) {
    if (
        name === `warning` &&
        typeof data === `object` &&
        data.name === `ExperimentalWarning`
    )
        return false;

    return originalEmit.apply(process, arguments);
};
if (!process.env.REEJS_CUSTOM_DIR) {
    process.env.REEJS_CUSTOM_DIR = __dirname;
}
let dir = fs.existsSync(process.env.REEJS_CUSTOM_DIR) ? process.env.REEJS_CUSTOM_DIR : `${home}/.reejs`;

if (globalThis.fetch) {
    globalThis._fetch = globalThis.fetch;
}
if (!globalThis.fetch) {
    globalThis._fetch = await import(`${dir}/polyfill/fetch.js`);
    globalThis._fetch = globalThis._fetch.default;
}

That import with ${dir} seems to cause the error, found at https://github.com/rovelstars/reejs/blob/master/utils/urlimports.js

@fvsch could you change the name of this issue?

Apart from the error message missing the file information, the issue here is actually that there's an eval(...) which uses an import, e.g.
import color from 'some-pkg';

eval(`
  someFunction(color);
`);

Notice how it's using color inside eval. That's unfortunately a current limitation of our ESM implementation. That's why you're seeing color is not defined.

Ah... Imagine those hacks not working here xD

Apart from the error message missing the file information, the issue here is actually that there's an eval(...) which uses an import, e.g.
import color from 'some-pkg';

eval(`
  someFunction(color);
`);

Notice how it's using color inside eval. That's unfortunately a current limitation of our ESM implementation. That's why you're seeing color is not defined.

shouldn't you consider fixing this though? Its one of the basic tokens/letters in js world ;)

This is not as easy to fix as you'd imagine. ESM is a little more complex and the code you're passing to eval is a string and we can't analyze strings. A fix for now would be to not use an imported expression inside eval. If you really wanted to run the code using a string then I'd suggest to use new Function() and pass the variables you need explicitly, e.g.
import color from 'some-pkg';

const fn = new Function('color', `
  someFunction(color);
`);

fn(color);

That works as expected and it also allows you to use dynamic imports, whereas with eval you can't use that unfortunately. There are some limitations to eval which are tricky to solve.

I think there are some downsides regarding this:

I need to opt-in/specify the variables to use in that function, whereas eval wont.
eval can use sourceMappingURL comment at the end of the file (or actually the string) and whenever the error happens, the error shows the filename for easy debug... I don't know the equivalent in functions...

And except those problems, I think it would be better to fix the eval implementation in webcontainer. I know and I also have been through bugs/problems that are hard to solve, and just asking the clients/users (referring to users/devs in this case) to use alternative methods doesn't seems like a valid fix ;)
Also it becomes one step backward to the nodejs (v16) compability in webcontainer fyi (no hurt sent here 🙃)

Yes I can totally understand where you’re coming from - the points you’re raising are valid. It is something we need to address. That being said, our roadmap does not prioritize this task at the moment and because of that, I can’t speak to an exact ETA when this will be fixed.
There are certain nitty-gritty details I can’t go into here but for the time being, the workaround I have suggested above can work, at least if you want to run this in WebContainers. Additionally, sourceMappingURL as well as sourceURL works perfectly fine inside new Function().
We have fixed the original issue with the stack trace not showing the filename where the error originated - we will most likely deploy the fix next week. I am going to let you know when it's deployed.
On a side note, thank you, @renhiyama, for how invested you are in this issue. I hope the workaround will work for you.

Guess what, Functions did allow using sourceMaps, GG!
Apart from not-so-good-looking codes, a function based implementation seems to work properly.

Btw why is the nodejs v16 in stackblitz not acting same as the real nodejs version v16.15.0 ?

This makes my code working in one place and not in another ah..

Nvm, it works now!

Glad this was solved @renhiyama! I'm closing this issue as Ree.js works now. I created an internal ticket for the import in eval error though. Just so we track it internally.


Updated to angular/cli:^1.6.8 - Error during template compile of 'PlatformRef'

Versions
<!--
Output from: `ng --version`.
If nothing, output from: `node --version` and `npm --version`.
  Windows (7/8/10). Linux (incl. distribution). macOS (El Capitan? Sierra?)
-->

Angular CLI: 1.6.8 (e)
Node: 8.9.1
OS: darwin x64
Angular: 5.2.4
... animations, common, compiler, compiler-cli, core, forms
... http, language-service, platform-browser
... platform-browser-dynamic, router
@angular/cdk: 5.2.0
@angular/cli: 1.6.8
@angular/material: 5.2.0
@angular-devkit/build-optimizer: 0.0.42
@angular-devkit/core: 0.0.29
@angular-devkit/schematics: 0.0.52
@ngtools/json-schema: 1.1.0
@ngtools/webpack: 1.9.8
@schematics/angular: 0.1.17
typescript-extends: 1.0.1
typescript: 2.4.2
webpack-concat-plugin: 1.4.0
webpack-dev-server: 2.11.1
webpack: 3.10.0

Repro steps

Step 1
Step 2
Step 3

Observed behavior
<!-- Normally this includes a stack trace and some more information. -->

ERROR in Error during template compile of 'PlatformRef'
  Function calls are not supported in decorators but 'ɵmakeDecorator' was called in 'Injectable'
    'Injectable' calls 'ɵmakeDecorator'.

Desired behavior

Mention any other details that might be useful (optional)

Same error... was working yesterday.....

Anyone know how to fix this? Or is there a different version I can roll back to?

Please provide the steps you followed to get to this error, or better yet provide a repository that shows this error.

I am getting the same thing, on my own very simple module  which provides a simple config value.
ERROR in Error during template compile of 'MyCoPlatformModule'
Function calls are not supported in decorators but 'MyCoComponentsModule' was called.

FIXED.
For me, I was importing a file out of my project config... changing that fixed the error... Finally..

This is my tsconfig - is that what you mean @accilies ?
{   "version": "2.5.3",   "compilerOptions": {     "module": "commonjs",     "target": "ES5",     "emitDecoratorMetadata": true,     "experimentalDecorators": true,     "moduleResolution": "node",     "baseUrl": "./",     "paths": {       "@angular/*": [         "node_modules/@angular/*"       ],       "@uirouter/*": [         "node_modules/@uirouter/*"       ]     },     "lib": [       "es2017",       "dom"     ]   },   "include": [     "src/**/*.ts"   ],   "exclude": [     "test.ts",     "**/*.spec.ts"   ] }

+1 on what @pcroc shows.  Why is AoT so brittle?

Getting same error here, when I tried to run cmd: ng build --prod.
will not get this if I don't build in production mode.
Log stack as follow:
Date: 2018-04-11T20:00:47.740Z
Hash: e2a911aa7287d5340be6
Time: 10887ms
chunk {scripts} scripts..bundle.js (scripts) 342 kB [initial] [rendered]
chunk {0} styles..bundle.css (styles) 716 kB [initial] [rendered]
chunk {1} polyfills..bundle.js (polyfills) 84 bytes [initial] [rendered]
chunk {2} main..bundle.js (main) 84 bytes [initial] [rendered]
chunk {3} inline.*************.bundle.js (inline) 796 bytes [entry] [rendered]
ERROR in Error during template compile of 'PlatformRef'
Function calls are not supported in decorators but 'ɵmakeDecorator' was called in 'Injectable'
'Injectable' calls 'ɵmakeDecorator'.

@accilies What do you mean by "exporting a file out my project configuration"?

@mderoche But I thought some recent version of TS explicitly solved that problem of "same-but-different" libraries. By the way, did you experiment with --preserve-symlinks?

This error is occurring to me with ^6.0.0 versions.

ERROR in ../@nims/ngutils/lib/collapsible-fieldset/collapsible-fieldset.component.ts(13,2): Error during template compile of 'CollapsibleFieldsetComponent'
Function calls are not supported in decorators but 'ɵmakeDecorator' was called in 'Component'
'Component' calls 'ɵmakeDecorator'.

@mderoche that didnt work for me
i'm trying to create angular-elements element for one component of a bigger app in a different dir.
`import {NgModule,Injector} from '@angular/core';
import {createCustomElement} from '@angular/elements';
import {BrowserModule} from '@angular/platform-browser';
import {SideMenuComponent} from '../../webapp-src/components/app/side-menu.component';
@NgModule({
imports: [BrowserModule],
declarations: [SideMenuComponent],
entryComponents: [SideMenuComponent]
})
export class AppModule {
constructor (private injector: Injector) {
const customElement = createCustomElement(SideMenuComponent, {injector});
customElements.define('side-menu', customElement);
}
ngDoBootstrap () { }
}`
ERROR in ../../webapp-src/components/app/side-menu.component.ts(16,2): Error during template compile of 'SideMenuComponent'   Function calls are not supported in decorators but 'ɵmakeDecorator' was called in 'Component'     'Component' calls 'ɵmakeDecorator'.

ok it did work, just needed to change it to
"baseUrl": "./",
"paths": {
"@angular/": ["node_modules/@angular/"]
},

This usually happenens when using npm link


TF2 Update broke everything

We need to bump SourceMod to the new version, when the issue got fixed.
https://github.com/alliedmodders/hl2sdk/pull/120

its fixed

Thanks Sappho!

We also need to bump tf2-comp-fixes to 1.16.11, there have been changes to gamedata.
https://github.com/ldesgoui/tf2-comp-fixes/releases/tag/v1.16.11