Update s3router.js

throw new Error does not work with AWS Lambda/Express/Serverless and gives the bellow error in production. Do you think you can use new Error("S3_BUCKET is required."); please. Otherwise have to upload all node modules to production/ source code manually.
module initialization error: Error
at S3Router (/var/task/node_modules/react-s3-uploader/s3router.js:19:15)
at Object. (/var/task/app.js:196:16)
at Module._compile (module.js:570:32)
at Object.Module._extensions..js (module.js:579:10)
at Module.load (module.js:487:32)
at tryModuleLoad (module.js:446:12)
at Function.Module._load (module.js:438:3)
at Module.require (module.js:497:17)
at require (internal/module.js:20:19)
at Object. (/var/task/lambda.js:3:13)
at Module._compile (module.js:570:32)
at Object.Module._extensions..js (module.js:579:10)
at Module.load (module.js:487:32)
at tryModuleLoad (module.js:446:12)
at Function.Module._load (module.js:438:3)
at Module.require (module.js:497:17)

The following line of code doesn't actually do anything:
new Error("S3_BUCKET is required.");

The code would continue executing the next line of code, and the requirement that a bucket be specified would not be met.  Why don't you just provide a bucket?  The library doesn't work without one.  Please reopen PR if I'm missing something.  Thanks!

Firstly thank you for bringing this awesome work. It is a life saver for sure.
Sorry I have to clarify clearly. I do understand that it does not do anything.
It is the javascript line parser.  It does not read the word "throw". It is module build error on the server.  If i add the bellow line
new Error("S3_BUCKET is required.");
that works fine. No line parse errors.
throw new Error("S3_BUCKET is required.");  does not work because of the  "throw"  command.
Do you think you can at least add.
return   new Error("S3_BUCKET is required.");
At the moment I have to upload all node modules to the server manually because I have to manually correct your module before it goes on the production server.
Your consideration is much much appreciated.
  It honestly seems like a bug if it chokes on the word throw.  Is there documentation on this?  You might fill out an issue with them.
Regardless, I don't think I'll merge this PR into the main codebase, as this is intentional error handling.  If I instead return new Error(...), then someone would just get a different error, since the return value wouldn't be of the expected type, and they still would never see the error message "S3_BUCKET is required".
Uploading all your code manually for this repo doesn't seem like an ideal solution either.  I think what you should do is this:

Fork this repo
Create a commit with your needed change
Change package.json to rename the module to something else (e.g. iamfree-com-fork-react-s3-uploader)
Run npm publish to publish the newly named module to your own NPM account
Change your project's dependency to iamfree-com-fork-react-s3-uploader
Use the new dependency everywhere.

Best of luck!


Remove @optional protocol methods for breaking-changes release

Reminder issue to clean up all @optional introduced between 3.0 and 4.0 once 4.0 hits.

Removing from the 4.0.0 milestone after some internal discussion. This would be a welcome change to ensure that we can make improvements in the future without silently breaking a bunch of implementations, but given that nobody else has seemed to ask for this since the issue was introduced in 2017, we feel comfortable pushing it to a later release.


how to load language files manually?

<script src="components/vuejs-datepicker/dist/vuejs-datepicker.min.js"></script>
<script src="components/vuejs-datepicker/dist/locale/translations/de.js"></script>

without a build-process involved, this is obviously not working, as it gives a SyntaxError: export declarations may only appear at top level of a module. What's the alternative? Is there a global setter or something where I can register language objects manually?

I need to build the lang files as common js or umd for this to work

That would be great. 👍

I also need some help to use the hungarian language manually. Can you help me, how can i do this?

<script src="components/vuejs-datepicker/dist/vuejs-datepicker.min.js"></script>
<script src="components/vuejs-datepicker/dist/locale/translations/de.js"></script>

without a build-process involved, this is obviously not working, as it gives a SyntaxError: export declarations may only appear at top level of a module. What's the alternative? Is there a global setter or something where I can register language objects manually?

  const datepicker = require('vuejs-datepicker');
  var Language = function Language (language, months, monthsAbbr, days) {
    this.language = language;
    this.months = months;
    this.monthsAbbr = monthsAbbr;
    this.days = days;
    this.rtl = false;
    this.ymd = false;
    this.yearSuffix = '';
  };
  var pt = new Language(
    "Brazilian",["Janeiro","Fevereiro","Março","Abril","Maio","Junho","Julho","Agosto","Setembro","Outubro","Novembro","Dezembro"],["Jan","Fev","Mar","Abr","Mai","Jun","Jul","Ago","Set","Out","Nov","Dez"],["Dom","Seg","Ter","Qua","Qui","Sex","Sab"]
  );
  const app = new Vue({
    el: '#app',
    components: {
      datepicker
    },
    data: {
      ptBR: pt
    }
  })

This should be working now as of v1.6.1 - reference global variable: vdp_translation_xx - see example in the readme.


Publish C++ API and inline documentation with Doxygen

The C++ API and inline documentation should be published to make it more easliy accessible.
Doxygen (still) seems to be king for C++, but if anyone knows any alternatives I am happy to take a look.

That would be cool. There is the Doxygen / Sphinx / Breathe / Exhale combo that might be relevant here.

Do you know if this stack uses reStructuredText? I am personally not a big fan of it and for a lot of people it would be another thing to learn.
Does not look like it is needed:
https://exhale.readthedocs.io/en/latest/quickstart.html?highlight=markdown#do-i-have-to-write-restructuredtext

Exhale takes Doxygen XML documentation and generates reStructuredText, so no reStructuredText has to be written

@birkskyum Do you want the private API to be documented as well? Or just the public C++ API.

I think the public api will suffice for now - would it be helpful from a maintenance standpoint to have the private api Publisher?

@birkskyum Not so much I think. Especially contributors will probably want to look at the source code.


change deactivsation hook

I would suggest maybe changing deactivation hook, as of now when you deactivate the plugin it removes all options.
If you were to disable the plugin to debug other parts of your site, then you would have to add the information all over again, I would maybe suggest using uninstall hook.

Hey, thank you for the suggestion. We'll implement that :)


[Power BI] Add docs for how to use SAS tokens

⚠ Problem
There's no assistance for configuring Power BI reports to use a SAS token.
🛠 Solution
Document the steps needed to use SAS tokens with Power BI reports.
🙋♀ Ask for the community

We could use your help:

Please vote this issue up (👍) to prioritize it.
Leave comments to help us solidify the vision.

Hello @flanakin can you please assign this to me


Filter endpoints by type

Add endpoint type filter to endpoint page list

Need to decide - always show filter or only when above certain number of endpoint types?
Need to decide - show base endpoints or endpoint with subtypes
Need to port upstream (downstream has much better for testing)

Codecov Report

Merging #434 into master will increase coverage by 0.00%.
The diff coverage is 54.68%.

@@           Coverage Diff           @@
##           master     #434   +/-   ##
=======================================
  Coverage   56.28%   56.28%
=======================================
  Files         898      898
  Lines       29318    29367   +49
  Branches     4234     4251   +17
=======================================
+ Hits        16501    16529   +28
- Misses      12817    12838   +21


tispark sql问题请教

你好，用tispark 执行sql查询的时候不能使用索引么？  是不是tispark sql执行时采用并行全表扫描？
因为我对索引字段用tispark sql查询速度慢很多， 要mysql客户端查询很快。
初学，望指教，谢谢

Hi, @carr123.
First of all, please confirm that you have set spark.tispark.plan.allow_index_read to true. This is essential because its default setting is false. You may refer to more configuration from this document
Anyhow, if you have index information on your data, TiSpark MAY choose to use index according to its CBO.
Hope these would help. :)

Hi, @carr123 .
The 10+ seconds that you noticed is an overhead time consumption that would not affect following queries. You may execute multiple queries after you have finished tidbMapDatabase operation, and execution time would be the time of the sql statement itself.
Also, could you state your spark environment as well? How many spark nodes are you using?
If the efficiency still does not meet your expectation, would you kindly send us the SparkPlan by spark.sql("...").explain so we could look into this?

@birdstorm
I test 2  SQL query,
./bin/spark-submit  test.py --num-executors 4  --driver-memory 2G
4 spark workers running on 2 nodes, each 1core, 1GB RAM used.
tikv-server running on the same 2 nodes.

Parsing command: select userid,count(factor) as t from train group by userid order by t desc limit 5
== Physical Plan ==
TakeOrderedAndProject(limit=5, orderBy=[t#34L DESC NULLS LAST], output=[userid#18,t#34L])
+- *HashAggregate(keys=[userid#18], functions=[sum(count(factor#19)#40L)])
+- Exchange hashpartitioning(userid#18, 200)
+- *HashAggregate(keys=[userid#18], functions=[partial_sum(count(factor#19)#40L)])
+- TiDB CoprocessorRDD{[table: train] , Columns: [factor], [userid], Aggregates: Count([factor]), First([userid]), Group By: [[userid] ASC]}

time taken
run sparksql task :  271s (not including map database)
from mysql client : 242s

SparkSqlParser: Parsing command: select count(*) from train
== Physical Plan ==
*HashAggregate(keys=[], functions=[sum(count(1)#38L)])
+- Exchange SinglePartition
+- *HashAggregate(keys=[], functions=[partial_sum(count(1)#38L)])
+- TiDB CoprocessorRDD{[table: train] [Index: userid] , Aggregates: Count(1)} EstimatedCount:40315849

time taken
run sparksql task :  54s (not including map database)
from mysql client : 25s

Spark的并发模型和TiDB有不同，你可以认为TiSpark的并发完全看spark executor的core数，而上面的测试场景一共2个core，很大程度上会比TiDB的goroutine并发慢很多，因为一次只计算2个region。所以扩大并发或者手动调整一个core多少并发读取TiKV也可以缓解
spark.tispark.task_per_split = 2或者4
另外，TiDB在聚合可以下推的场合，如果没有更进一步需要shuffle数据的计算（比如join），那么速度并不会比TiSpark慢，哪怕给予同样的core数，因为大部分计算在TiKV完成，而TiDB的并发模型比Spark要灵活得多。
因此，如果是上述测试的场合，给予Spark足够的core，会得到和TiDB近似的速度，但是不会更快。如果是更复杂的查询（比如多表join），TiSpark会更快。

thanks a lot


natOutgoing: true seems to not work with private: true

Bug Report
The configuration natOutgoing: true seems to not work with private: true for Subnets.
If I create a subnet with both configurations, the pods in this subnet does not seem to be able to access an external address (making use of the NAT outgoing).
Expected Behavior
I would like to be able to combine natOutgoing: true with private: true, so that internal isolation between the subnets is achieved, while also allowing the pods to access addresses on the internet (e.g. for downloading datasets) via NAT-ing (so that external internet addresses cannot initiate any connection with a pod inside the cluster).
Actual Behavior
natOutgoing: true seems to be ignored when private: true is set.
Steps to Reproduce the Problem

Create 2 isolated subnet that do not overlap with natOutgoing and private:

# nw1.yaml
apiVersion: kubeovn.io/v1
kind: Subnet
metadata:
  name: nw1
spec:
  protocol: IPv4
  default: false
  private: true
  natOutgoing: true
  cidrBlock: <IP_ADDRESS>/28

$ kubectl apply -f nw1.yaml

# nw2.yaml
apiVersion: kubeovn.io/v1
kind: Subnet
metadata:
  name: nw2
spec:
  protocol: IPv4
  default: false
  private: true
  natOutgoing: true
  cidrBlock: <IP_ADDRESS>/28

$ kubectl apply -f nw2.yaml

Create 2 pods, one in each subnet

# nw1-pod.yaml
apiVersion: v1
kind: Pod
metadata:
  name: nw1-pod
  annotations:
    ovn.kubernetes.io/logical_switch: nw1
spec:
  containers:
    - name: nw1-container
      image: "alpine:3.17"
      command: ["sleep"]
      args: ["infinity"]

$ kubectl apply -f nw1-pod.yaml

# nw2-pod.yaml
apiVersion: v1
kind: Pod
metadata:
  name: nw2-pod
  annotations:
    ovn.kubernetes.io/logical_switch: nw2
spec:
  containers:
    - name: nw2-container
      image: "alpine:3.17"
      command: ["sleep"]
      args: ["infinity"]

$ kubectl apply -f nw2-pod.yaml

Now, get inside of the pods and try to ping some address on internet and the other pod:

$ kubectl exec -it nw1-pod -- ip addr
1: lo: <LOOPBACK,UP,LOWER_UP> mtu 65536 qdisc noqueue state UNKNOWN qlen 1000
    link/loopback 00:00:00:00:00:00 brd 00:00:00:00:00:00
    inet <IP_ADDRESS>/8 scope host lo
       valid_lft forever preferred_lft forever
    inet6 ::1/128 scope host
       valid_lft forever preferred_lft forever
939: eth0@if940: <BROADCAST,MULTICAST,UP,LOWER_UP,M-DOWN> mtu 1400 qdisc noqueue state UP
    link/ether 00:00:00:5f:39:c4 brd ff:ff:ff:ff:ff:ff
    inet <IP_ADDRESS>/28 brd <IP_ADDRESS> scope global eth0
       valid_lft forever preferred_lft forever
    inet6 fe80::200:ff:fe5f:39c4/64 scope link
       valid_lft forever preferred_lft forever

$ kubectl exec -it nw1-pod -- ping -w 3 <IP_ADDRESS>
PING <IP_ADDRESS> (<IP_ADDRESS>): 56 data bytes

--- <IP_ADDRESS> ping statistics ---
3 packets transmitted, 0 packets received, 100% packet loss
command terminated with exit code 1

$ kubectl exec -it nw2-pod -- ip addr
1: lo: <LOOPBACK,UP,LOWER_UP> mtu 65536 qdisc noqueue state UNKNOWN qlen 1000
    link/loopback 00:00:00:00:00:00 brd 00:00:00:00:00:00
    inet <IP_ADDRESS>/8 scope host lo
       valid_lft forever preferred_lft forever
    inet6 ::1/128 scope host
       valid_lft forever preferred_lft forever
941: eth0@if942: <BROADCAST,MULTICAST,UP,LOWER_UP,M-DOWN> mtu 1400 qdisc noqueue state UP
    link/ether 00:00:00:f2:d5:6e brd ff:ff:ff:ff:ff:ff
    inet <IP_ADDRESS>/28 brd <IP_ADDRESS> scope global eth0
       valid_lft forever preferred_lft forever
    inet6 fe80::200:ff:fef2:d56e/64 scope link
       valid_lft forever preferred_lft forever

$ kubectl exec -it nw1-pod -- ping -w 3 <IP_ADDRESS>
PING <IP_ADDRESS> (<IP_ADDRESS>): 56 data bytes

--- <IP_ADDRESS> ping statistics ---
3 packets transmitted, 0 packets received, 100% packet loss
command terminated with exit code 1

We can see that the nw1-pod cannot ping either the <IP_ADDRESS> address or the nw2-pod.
Additional Info

Kubernetes version:
Output of kubectl version:
$ kubectl version
Client Version: v1.28.2
Kustomize Version: v5.0.4-0.20230601165947-6ce0bf390ce3
Server Version: v1.28.3

Installed via microk8s (sudo snap install microk8s --classic --channel=1.28)

kube-ovn version:
kubeovn/kube-ovn:v1.10.0   # image of kube-ovn-controller-...

installed via microk8s (microk8s enable kube-ovn)

operation-system/kernel version:
Output of awk -F '=' '/PRETTY_NAME/ { print $2 }' /etc/os-release:
Output of uname -r:
$ awk -F '=' '/PRETTY_NAME/ { print $2 }' /etc/os-release
"Ubuntu 22.04.3 LTS"

/cc @Chrisys93 @JuanMaParraU

@abravalheri, this is an expected behavior. When private is set to true, only traffic for which both the source and destination belong to the subnet CIDR is allowed. You need to set the allowSubnets to allow specific traffic.

Hi @oilbeater thank you very much for the response. So is it correct to assume that it is not possible to allow the pods downloading things from the "general internet", but we need to know beforehand which CIDRs they need to access?
I noticed the the docs mention that the spec.acls parameter can be used for finer control than private: true. Do you know what would be the equivalent ACL that would result in a similar behaviour than private: true? Maybe by playing with the ACLs I can find a way of achieving this...

I have started working on a local update at the end of last week. We need to sort this out. Further, our Kube-OVN continues having stability problems.

Would it be possible to keep this issue open as a "Documentaiom" request?
It would be nice if the docs contain an example of how to achieve this by manipulating the ACLs... I never managed to figure it out myself.

Would it be possible to keep this issue open as a "Documentaiom" request?\n\nIt would be nice if the docs contain an example of how to achieve this by manipulating the ACLs... I never managed to figure it out myself.

@abravalheri could you please help to post a PR about the issue ？ thanks！

I can open an issue, but I can't open a PR unfortunately,  because I don't know how to do it  😅


In the modal window that is shown with RMB, the display of the package name the disappears

If the package is not visible in the links, then it is not displayed in the RMB modal window


Split the library in two modules: logcat and logger

It would be nice to have a pure jvm module with the logcat function so you don't need to pay the compilation time of an Android module only because you want to log something. The AndroidLogcatLogger would be moved to a "logger module" .
This way you just depend on the "logger module" in the :app module. The "logger module" could include the other one as an api dependency so if someone doesn't care about this can only use this dependency everywhere.

Interesting. I thought about this, but the thing is, the logcat dependency doesn't actually declare any dependency on Android, so you can totally use it from a JVM only module, it should work, even if there's a class that references unknown symbols. Of course you shouldn't load the AndroidLogcatLogger class on a JVM. Am I missing something?

@pyricau
As far as you are using com.android.library Plugin the artifact result will be an .aar files that needs to be consumed by an Android Application or Android LIbrary, but can't be consumed by a pure JVM module.
Here the error you get if you try to use logcat into a pure JVM module:
Execution failed for task ':stream-chat-android-core:compileKotlin'.
> Error while evaluating property 'filteredArgumentsMap' of task ':stream-chat-android-core:compileKotlin'
   > Could not resolve all files for configuration ':stream-chat-android-core:compileClasspath'.
      > Could not resolve com.squareup.logcat:logcat:0.1.
        Required by:
            project :stream-chat-android-core
         > No matching variant of com.squareup.logcat:logcat:0.1 was found. The consumer was configured to find an API of a library compatible with Java 8, preferably in the form of class files, preferably optimized for standard JVMs, and its dependencies declared externally, as well as attribute 'org.jetbrains.kotlin.platform.type' with value 'jvm' but:
             - Variant 'releaseApiPublication' capability com.squareup.logcat:logcat:0.1 declares an API of a library, and its dependencies declared externally:
                 - Incompatible because this component declares a component, with the library elements 'aar' and the consumer needed a component, preferably in the form of class files
                 - Other compatible attributes:
                     - Doesn't say anything about its target Java environment (preferred optimized for standard JVMs)
                     - Doesn't say anything about its target Java version (required compatibility with Java 8)
                     - Doesn't say anything about org.jetbrains.kotlin.platform.type (required 'jvm')
             - Variant 'releaseRuntimePublication' capability com.squareup.logcat:logcat:0.1 declares a runtime of a library, and its dependencies declared externally:
                 - Incompatible because this component declares a component, with the library elements 'aar' and the consumer needed a component, preferably in the form of class files
                 - Other compatible attributes:
                     - Doesn't say anything about its target Java environment (preferred optimized for standard JVMs)
                     - Doesn't say anything about its target Java version (required compatibility with Java 8)
                     - Doesn't say anything about org.jetbrains.kotlin.platform.type (required 'jvm')

ah, good point, should look at whether we can release a jar instead then.


john: rolling-2404 -> rolling-2409

Description of changes
Update to the latest git commit.
Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[x] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
24.11 Release Notes (or backporting 23.11 and 24.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

There is no rolling-2409 on https://github.com/openwall/john-packages/releases
If we decided to go for latest git commits instead of upstream rolling releases, we would have to change the version naming to the proper nixpkgs style for unstable versions

I think it would be better to just stick with the rolling releases tagged by the john team. I wasn't aware of those when I initially created the PR.


Feature request: simplify tf.train.write_graph() and tf.import_graph_def()

As can be seen in https://github.com/tensorflow/tensorflow/issues/616 the current mechanism is too painful. (Also #4044 etc )

Could you summarize what the problem is, perhaps with a snippet of code? Thanks.

Nevermind, the old horrible workaround of #616  has been completely streamlined and simplified in one single new function tf.train.import_meta_graph


Fix perl shebang

Checklist

[x] Used a personal fork of the feedstock to propose changes
[x] Bumped the build number (if the version is unchanged)
[x] Reset the build number to 0 (if the version changed)
[x] Re-rendered with the latest conda-smithy (Use the phrase @conda-forge-admin, please rerender in a comment in this PR for automated rerendering)
[x] Ensured the license file is being packaged.

Fixes #38

@conda-forge-admin please rerender

Thanks Kerry! 🙏 This seems reasonable. Let's give it a try 😄


Adding a pdf update cli command.

@pharmaguide So I had some time to look into this. I also took the liberty to make some changes to allow for the create form to start working (it seemed to be broken).
Here is what it ends up creating as far as fields are concerned.

So, it looks like it is not scaling the field positions correctly, but it does appear to be getting most of them. It also looks like it is merging two separate pages into a single page which is also a problem.
It is getting really close, but still needs some work.
Thanks!
Travis.

Our latest PDF server (4.0.0) has this built in now where it automatically pulls out the pdf fields provided a fillable pdf, so closing this PR.


Create permutation&combination

Description of Change

Checklist

[ ] Added description of change
[ ] Added file name matches File name guidelines
[ ] Added tests and example, test must pass
[ ] Added documentation so that the program is self-explanatory and educational - Doxygen guidelines
[ ] Relevant documentation/comments is changed or added
[ ] PR title follows semantic commit guidelines
[ ] Search previous suggestions before making a new one, as yours may be a duplicate.
[ ] I acknowledge that all my contributions will be made under the project's license.

Notes:

Description of Change
Checklist

[ ]  Added description of change
[ ]  Added file name matches File name guidelines
[ ]  Added tests and example, test must pass
[ ]  Added documentation so that the program is self-explanatory and educational - Doxygen guidelines
[ ]  Relevant documentation/comments is changed or added
[ ]  PR title follows semantic commit guidelines
[ ]  Search previous suggestions before making a new one, as yours may be a duplicate.
[ ]  I acknowledge that all my contributions will be made under the project's license.

Notes:

HI @Satyamchaubey07
This can be included in maths folder. But the code you have written is not upto the repository standards. Please read the contribution guidelines. For better understanding of the pr standard we are looking is by checking the recently merged pr. Thanks!


Remove KafkaChannel v1alpha1 API shape

Remove KafkaChannel v1alpha1 API shape as written here: https://github.com/knative/eventing-contrib/pull/1504#pullrequestreview-474244750
Part of umbrella issue, https://github.com/knative-sandbox/eventing-kafka/issues/70

/remove-lifecycle stale

/remove-lifecycle stale


Wire the population sizes for distributions to the YAML generation

Right now only the size carries over.

Solved


feat(logger): use env_logger and RUST_LOG for better control of logging

What this PR does 📖

what the title says
adds command line option to enable writing to file because the LogProfile was removed.
assumes that unless production mode is set the logger will be sending output to the terminal

Which issue(s) this PR fixes 🔨

Resolve #

Special notes for reviewers 🗒
Additional comments 🎤
example: export RUST_LOG=uplink=debug,common=debug,warp_blink_wrtc=trace; target/debug/uplink
careful about doing things like RUST_LOG=error which sets a global log level - this will cause tons of ipfs related logs from other warp modules.
note that each module in the "uplink" workspace needs to be enabled separately. in the above example, uplink (the ui module) and common are enabled but extensions and kit are not.

Great stuff but I think we want tracing and tracing-subscriber instead of log and env_logger. The reasons are documented here but also to keep in mind that most of the Rust projects out there have switched to tracing a while ago.

I was going to say that we can do that in a followup PR but skimming through tracing docs, I got reminded:

Users of the log crate should note that tracing exposes a set of macros for creating Events (trace!, debug!, info!, warn!, and error!) which may be invoked with the same syntax as the similarly-named macros from the log crate. Often, the process of converting a project to use tracing can begin with a simple drop-in replacement.

Great stuff but I think we want tracing and tracing-subscriber instead of log and env_logger. The reasons are documented here but also to keep in mind that most of the Rust projects out there have switched to tracing a while ago.

I was going to say that we can do that in a followup PR but skimming through tracing docs, I got reminded:

Users of the log crate should note that tracing exposes a set of macros for creating Events (trace!, debug!, info!, warn!, and error!) which may be invoked with the same syntax as the similarly-named macros from the log crate. Often, the process of converting a project to use tracing can begin with a simple drop-in replacement.

Yea. tracing can usually be a drop-in replacement by just importing tracing. It can be done in a separate pr though since it may have to be looked into hooking tracing-subscriber into the dev tool logger (though should work fine out of the box for terminal and to file)

Sheldon want's command line options to set RUST_LOG, and a default setting.

Sheldon wants command line options to set RUST_LOG, and a default setting.

Havent tried it myself with log but you should be able to set the env from rust using std::env::set_var. Maybe use dotenv crate to make it configurable? I know in tracing it could be made easier

Sheldon wants command line options to set RUST_LOG, and a default setting.

@tooshel what's the use case here? The defaults from upstream are typically good. You typically set them for debug and you can easily do that with the env.

@zeenix  requesting the tracing stuff be added later so that I can focus elsewhere.
@tooshel  a sensible default has been added. I don't think we were using all the previously available levels and a single default is probably good enough.

maybe we should use a .env file for development but not use it for released software. it does seem like there's a lot of modules to set.

maybe we should use a .env file for development but not use it for released software. it does seem like there's a lot of modules to set.

Usually in released software you use real env vars, not .env files. .env files are a developer convenience (because setting and managing env vars outside a file is painful and error prone). I like the sensible default! Let's get it though QA and merge it and come back to tracing later.

Usually in released software you use real env vars, not .env files. .env files are a developer convenience (because setting and managing env vars outside a file is painful and error prone).

There are so many ways to easily set the env (both locally and globally), .env is just one of them. You can also set it per app (on Linux that's through the .desktop or .service files but I'm sure there are Windows and Mac equivalents of that). You can also provide people a script to run, instead of a command.

I like the sensible default! Let's get it though QA and merge it and come back to tracing later.

For non-developers, the default default is good. Emitting debug log comes at a performance cost and typically non-developers won't need it. when we need more logs, we can always provide an easy way to set it to what we need. We'll likely need tracing level in those cases anyway.

Having said that, I don't have strong opinion on the subject even if it may not seem so. :) So please go ahead and merge.


Update README.md

fix type (files > file, google > Google)

 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.You have signed the CLA already but the status is still pending? Let us recheck it.


⚠ Bofrost.at has degraded performance

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

HTTP code: 200
Response time: 1266 ms

Resolved: Bofrost.at performance has improved in 15db615.


Update create_musicbox.sh

make these commands unattended.

Oops, this should not have been merged into master. Damn.


hope open single file not project

This looks like what I'm hoping to be able to do with Marker but I'll offer a use case, @aohanhongzhi I hope you'll let us know if you have a different use case in mind.
I'd like to be able to just double-click a markdown file and have it open with marker, this would let me use marker as my default markdown editor without needing to create a Project for random one-off files in directories that I don't otherwise have interest in adding to marker.

@jeffWelling This would be great to have. Tauri v2 offers a way to do this, which is currently in beta - I plan to integrate this after it becomes stable.


cURL error 28: Operation timed out

From my logs:
(GuzzleHttp\\Exception\\ConnectException(code: 0): cURL error 28: Operation timed out after 5001 milliseconds
with 0 bytes received (see https://curl.haxx.se/libcurl/c/libcurl-errors.html) for
https://api.torchlight.dev/highlight at
/var/www/html/vendor/guzzlehttp/guzzle/src/Handler/CurlFactory.php:210)

I had the same issue, removing all HTML from my markdown resolved this.


Test against latest marshmallow 2.0 release

Tests against the latest marshmallow 2.0 release in tox and TCI. Fixes test for compat with 2.0.0rc1.

Thanks!


Can not manage users without contact

As there is only a list of contacts in the backend.
Users without a linked contact are not visible in the backend and can not be managed.

as talked with @chirimoya the user contact should not be nullable


Corner Radius Indicator

I simply want to add corner radius to the indicator using indicatorClass might be a little too much? Please provide the fourth customise option for indicator.
pagingViewController.indicatorConerRadius = 2

I've been thinking a bit more about this and I don't think I will add this as an option. One of the goals of Parchment is to be a generic and easily customizable framework. I think adding built-in support for these things will only make it bloated in the end.
You can achieve rounded corners by just adding a few lines of code:
class RoundedIndicatorView: PagingIndicatorView {
  override func apply(_ layoutAttributes: UICollectionViewLayoutAttributes) {
    super.apply(layoutAttributes)
    layer.cornerRadius = layoutAttributes.frame.height / 2
  }
}

Thanks for the suggestion though 🙂

This is not about the simplicity, it's actually a necessary attribute when designing any UI related repo, corner radius improve the beauty of user interface.


Comic Vine API Key invalid

When I use the API key generated by Comic Vine, Comic Tagger says it's invalid. Could I putting an incorrect URL?
I made a new account and same issue.

Did you change the url or the api key?

I copied the API key straight from the site. And the url I copied from the API page at the top of the browser.

I'm very new to using comic tagger and doing anything involving scripts LOL.

So you don't need to set the URL, it's possible that's bugged, unset it and try it

This should be fixed in the latest beta 1.4.4-beta.3


Feedback Email address bounced

Hi,
the<EMAIL_ADDRESS>email address at the bottom of this page bounced when i attempted to send an email to it.
https://docs.neo.org/developerguide/en/articles/wallets.html

Hi @chrismcna
We are no longer maintain that site (https://docs.neo.org/developerguide/). For any feedback you can open doc issues.
Thanks.


[Bug] Router Connected Devices sensor - IP of devices connected to Mesh nodes appear as null

The problem
In the sensor Connected Devices of a router the IP of devices connected to Mesh nodes appear as null.

Your device model
RT-AX88U
Firmware type
Merlin
Firmware version
<IP_ADDRESS>.388.1
Integration version with the issue
0.17.4
Method of the integration installation
HACS
What version and type of Home Assistant installation do you use
2023.1.2
Is there anything useful in the logs?
No response
Diagnostics information
No response
Additional information
No response

Hello. Does it happen to many devices or only to some? Does it happen to the end devices or only to AiMesh nodes, which are also shown as connected devices?

It happens to all end devices not connected to the router, but connected to one of the AiMesh nodes.
The MAP-AC1300 AiMesh nodes don't appear in the list of connected devices of the router.
The RT-AC87U media bridge does appear in the list of connected devices and has an IP listed.

I've rebooted the router and AiMesh nodes today and noticed that only 12 devices appear without IP now.
All 12 connected are connected to AiMesh nodes, not to the router.
The device_tracker entities created for these 12 devices don't have an IP attribute. And the Connection time, Rx speed and Tx speed attributes are missing a value.
Anything to check on the router that would help clarify the cause?

Please, check whether the issue persists with the new version 0.18.0

The issue is still there in 0.18.0.
I've noticed 2 more related issues in 0.18.0:

Now I can see the AiMesh nodes as connected devices and for them node is null.
Devices wired to a media bridge (RT-AC87U) have connection_type: unknown and node: null

You can try checking the 0.23.0b1 beta release. Unfortunately, device tracking with an Asus router is quite a headache and I am still not sure how to make everything work well


Fix windows yaml generation

Fixes https://github.com/istio/istio/issues/19020

In response to a cherrypick label: new pull request created: #654


Config: Implement better defaults

~Better reviewed commit-by-commit.
The large refactoring in main will be quite useful for the admin interface, where it's just a matter of adding an if at the top level.~
Stripped down to 1 commit...

Stripped down to 1 single commit, since this refactor discovered a few bugs.
Going to address everything independently and gradually submit the other commits.
Since the commit has already been approved, will self-merge once green.


AB#15305 Hide application status in the details tab

right now this dynamically updates and the title / breadcrumb does not, also if we are getting rid of the code we should rather remove it instead of just display:none


Fixed insufficient input validation

📊 Metadata *
Please enter the direct URL for this bounty on huntr.dev. This is compulsory and will help us process your bounty submission quicker.
Bounty URL: https://www.huntr.dev/app/bounties/open/1-npm-bson-objectid
⚙ Description *
Added methods for creating new ObjectID from JSON after removing unwanted properties from it.
💻 Technical Description *

Checks that were previously at the beginning of the constructor moved into ObjectID.hasRequiredProps.
ObjectID.sanitizeObject then removes unwanted properties from the JSON.
ObjectID.createFromObject then creates an ObjectID from the scrubbed JSON.

🐛 Proof of Concept (PoC) *
Provide the vulnerability exploit to show the security issue you're fixing.
🔥 Proof of Fix (PoF) *
Replay the vulnerability exploit to show the successful fix and mitigation of the vulnerability.
👍 User Acceptance Testing (UAT)
Run the following unit test
it('should not allow insertion of an arbitrary property', function() {
    var json = {
      "mal_formkey": {
        "payload": "xxxx"
      },
      "_bsontype": "ObjectID",
      "id": "5eecccdc951ca34d04e3ff65",
    };

    var obj = ObjectID(json);
    obj.should.be.instanceof(ObjectID);
    obj.toString().should.eql("5eecccdc951ca34d04e3ff65");
  });

Sorry about that one @toufik-airane @mufeedvh changes have been reset!

Awesome! 👍

Congratulations bbeale - your fix has been selected! 🎉
Thanks for being part of the community & helping secure the world's open source code.
If you have any questions, please respond in the comments section. Your bounty is on its way - keep hunting!

Hey folks - I'm going to manually cherry-pick this, but I'd love your support by having pull requests in future go upstream!

Actually it appears there was a PR, but it had conflicts.


Can support go module mechanism?

add a go.mod file
create a git tag

Why? Unless I misunderstand something you can import a non-module dependency in your module project just fine, it will lock on the current git revision.
In goja each commit is essentially a new release.

It may still make sense to have a go.mod file, even though the library may be effectively locked to v0 (no release tags).
The reason to add it is that it will list compatible library versions used by goja itself.

@dop251 This commit hash approach means that every time someone wants to know how old a version they use might be, they would need to look at a long list of commit hashes and figure out what was implemented when. That's insanity, and it will get worse over time. When the es6 branch is merged in, it will mean someone trying to determine when that merge happened and then figuring out which hash to use. Someone can still use a hash with go.mod. That ability wouldn't go away. As dennwc said, it also gives you the opportunity to specify dependency versions even if you don't make release tags and keep going as-is.


Can't override Address.ss

I can't override the Address template.  I noticed because it interferes with the silvershop admin interface (the field names are different so the address does not show)
I've tried putting overrides in themes, mysite and themename_addressable - nothing overrides it.
when i delete addressable's Address.ss file,  the expected hierarchy of templates works again.

What is your global $project variable set to? Does it work if it's in {project-dir}/templates/Address.ss ?

Thank you for your contribution here. I just wanted to let you know that we’re looking to improve our management of issues and pull requests, with a goal to providing clear direction on how Symbiote contributes to the open source community. As part of this ongoing effort:

issues and pull requests that have been open prior to December 2017 with no recent activity are being closed

this is included

if you can confirm this remains applicable for SS3.6+ please do open it again

remaining issues and pull requests will be

categorised

on a clear path to resolution (whatever the next step may be).

This ultimately brings us one step closer to our ambition; being able to prioritise current issues and contributions in a timely manner. If you have any questions, please see https://www.symbiote.com.au/contributing for further information.


tabsanity.cursorLeft failed

Not sure how to replicate reliably. Is there any way to enable logging for this extension? It has been happening about once a day on my dev system, requiring a restart of VSCode.
cursorRight also fails, but up and down work normally. Can't backspace either. Sorry for such slim info, but that's about all I can identify so far.

Interesting. I haven't experienced this at all and I've been using the plugin religiously. I'm willing to bet there's some sort of strange scenario you've encountered, specifically with your text files, that I haven not experienced myself. For example, perhaps you have mixed tabs with spaces or something else strange?
The plugin should work either way. I'm just trying to assess what the problem is. Have you tried uninstalling and reinstalling the plugin? You sure you have the most up-to-date version?

My extension says it's version is 0.0.8. I'm using the EditorConfig extension to control tab settings, so they should be uniform. Additionally, this can occur with files that only I have ever edited. I have reinstalled the plugin in the past. I understand it's hard to diagnose a problem you can't observe, and I can't even reproduce it reliably myself.

I've only used this plugin in tandem w/ EditorConfig, so that's the intended purpose. You're definitely following the golden path.
Can I get some info about your environment? What's your OS? What are your indentation settings? You can paste-in your .editorconfig file here w /the ini syntax highlighting.

OS: Windows 10
.editorconfig:
`
EditorConfig is awesome: http://EditorConfig.org
top-most EditorConfig file
root = true
[*]
charset = utf-8
indent_style = space
indent_size = 2
end_of_line = crlf
insert_final_newline = true
trim_trailing_whitespace = true
`
settings.json:
{     "diffEditor.renderSideBySide": false,     "editor.rulers": [80], 	"eslint.enable": false,     "explorer.openEditors.visible": 0,     "javascript.validate.enable": false, 	"jshint.enable": false, 	"terminal.external.windowsExec": "%systemroot%\\sysnative\\WindowsPowerShell\\v1.0\\powershell.exe", 	"terminal.integrated.shell.windows": "\\windows\\sysnative\\WindowsPowerShell\\v1.0\\powershell.exe", 	"workbench.editor.showTabs": false,     "search.exclude": {         "**/node_modules": false,         "node_modules/": true,         "*.log": true,         "**/bower_components": true     } }
I work on projects that use either jshint or eslint; I enable them per-project with workspace settings.

Humm, those settings look very similar to mine. Nothing stands out here, so I think we'll have to wait until we can reliably reproduce the issue?

If you could provide a way to at least log a stack trace when errors like this occur, it would probably help in diagnosing errors like this. Otherwise, I'll keep an eye on coming up with reliable steps to reproduce.


Scala section

The problem
Spaceship should support Scala section: https://www.scala-lang.org/
Describe the solution you'd like
Spaceship should detect:

build.sbt, .scalaenv or .sbtenv files
Files with .scala or .sbt extension
Directory .metals

Extract version via scalac -version
Describe alternatives you've considered
No response
Documentation and adoption
No response

I'll take care of this if you don't mind :)

Sure! Go ahead!


Added a submit_buttons block to login template

Hi,
The idea behind this PR is to be able to add buttons (e.g..: Sign up) on the login page without having to copy/paste the entire form.

Thanks @Gagaro! Could you add documentation for the new block to http://docs.wagtail.io/en/v1.8.1/advanced_topics/customisation/admin_templates.html, please?

Sure, doc added.

Merged in 4fadecebd698047ec511e24a3e5640320a4c1d20 - thanks!


Rate limitation

I was trying to cause a rate limitation error, and it seemed impossible. Apparently looking at the documentation, it's 10req/s. I think it's only unauthenticated requests???
We should return an error based on that if it happens or we prevent the call to the api.
There are multiple headers associated with it
ratelimit-limit →600
ratelimit-observed →1
ratelimit-remaining →599
ratelimit-reset →1519126320
ratelimit-resettime →Wed, 20 Feb 2018 11:32:00 GMT

We could, in theory, based the on the last query know in advance if the next one will cause a rate limit error.

Currently we will return the rate limit error that Gitlab returns (that is, I assume it does indeed generate an error for that). But I agree that preventing that you would hit the rate limit would be a nice feature.
Looking at how go-github handles rate limiting, it does seem to require some new moving parts and additional logic if we want to do this right. But that shouldn't be a reason not to add this of course :wink:

I started something, but then while testing it ( trying to bust the API limit ), I think I found a bug on the gitlab side for rate-limitation.
Tell me what you think, I wrote this simple loop :
for i := 0; i < 650; i++ {
	_, resp, err := git.Projects.GetProject(13083) // Get GitlabProject
	fmt.Println(resp)
	fmt.Println(err)
}

I compile, run it...
&{0xc0420fe090 0 0 0 0 0 0 {600 1<PHONE_NUMBER>248 2018-02-21 04:10:48 +0000 GMT}}
<nil>
&{0xc0421d2000 0 0 0 0 0 0 {600 2<PHONE_NUMBER>248 2018-02-21 04:10:48 +0000 GMT}}
<nil>
&{0xc0421dc000 0 0 0 0 0 0 {600 3<PHONE_NUMBER>248 2018-02-21 04:10:48 +0000 GMT}}
<nil>
&{0xc0420fe1b0 0 0 0 0 0 0 {600 4<PHONE_NUMBER>248 2018-02-21 04:10:48 +0000 GMT}}
<nil>
&{0xc0421d2090 0 0 0 0 0 0 {600 5<PHONE_NUMBER>248 2018-02-21 04:10:48 +0000 GMT}}

You can see 600, 5, 595 these are the rate limitation headers. So far so good, it goes up to around 400 observed requests.
Now I start my program in two separate consoles. I expected the rate limitation go up MUCH faster right? Well.... Nope 😕  It gets to a maximum of 10 for the ratelimit-observed and then it resets to 0.
Were you expecting that kind of behavior? Maybe I should file a bug on gitlab side?
Maybe I wrote something weird. https://github.com/Lorac/go-gitlab/commit/b2dc81bba76be96f7721f1334168ba77171b1373
That's the commit for now... Want me to do a PR and review it there instead?

So, in other words, I can bypass the limit by doing more requests in different threads...
yeesh, thats hard to believe, hopefully its just something we're doing. Though it seems like a possible error case if they aren't synchronizing that counter properly. Would suggest checking their issue tracking and if nothings there maybe formalizing the test (like a gist they can just run) and filing a bug. Would be happy to double check your work to avoid possible :egg: :man_facepalming:

I can observe the same (more or less) behaviour with this one-liner:
for i in {1..1000}; do curl -sD - -o /dev/null "https://gitlab.com/api/v4/projects/13083"; done

I have no clue how it is supposed to work, but it looks like there isn't much rate limiting going on. According to the docs (hard to find and now very clear) it seems the limit should be max 10 req/sec per IP address...

I posted an issue on gitlab. https://gitlab.com/gitlab-org/gitlab-ce/issues/43456
Getting no attention so far.

This issue just got some attention on Gitlab side. Hopefully, we can see what is wrong with their HAProxy or maybe we did something weird...

Hi @Lorac! I was just wondering if you think we should still keep this issue open? I'm looking into a little end-of-year cleanup and unless we have an idea what to do with this issue, I think we should close it. What do you think?

Yes you can close, I'll submit a PR at some point, I think we just can't poke the api fast enough to hit the rate limiter

Check... Thanks for the update!


SmartAdServer image sync and noAd

Type of change

[X] Feature: Sync changes/improvement

Description of change
Change 1:
When the default iframe sync is not enabled we will now try to image sync if enabled.
if (iframe sync enabled)
{
sync
}
else if (image sync enabled && sync urls provided)
{
sync with each url
}
Change 2:
We need to handle noAds in our prebidResponse, in case of noad we don't want to build the bidresponse array.
Change 3:
Added tests to cover the first 2 changes.

Everything is ok IMO. You just have to create the PR on the public repo :)

Everything is ok IMO. You just have to create the PR on the public repo :)


chore[sailor-bridge]: change Converter namespace to Convert.

Codecov Report

Merging #111 (f105493) into main (6688770) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##               main     #111   +/-   ##
=========================================
  Coverage     94.45%   94.45%
  Complexity      593      593
=========================================
  Files           128      128
  Lines          2345     2345
=========================================
  Hits           2215     2215
  Misses          130      130

Flag
Coverage Δ

phpunit
94.45% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

...idge/src/Convert/AbstractDateTimeTypeConverter.php
100.00% <ø> (ø)

...rc/sailor-bridge/src/Convert/DateTypeConverter.php
100.00% <ø> (ø)

...rc/sailor-bridge/src/Convert/JsonTypeConverter.php
100.00% <ø> (ø)

...or-bridge/src/Convert/TimestamptzTypeConverter.php
100.00% <ø> (ø)

.../sailor-bridge/src/Convert/TimetzTypeConverter.php
100.00% <ø> (ø)

...rc/sailor-bridge/src/Convert/UuidTypeConverter.php
88.88% <ø> (ø)

src/sailor-bridge/src/Type/DateTimeTypeConfig.php
100.00% <ø> (ø)

src/sailor-bridge/src/Type/JsonTypeConfig.php
100.00% <ø> (ø)

src/sailor-bridge/src/Type/UuidTypeConfig.php
100.00% <ø> (ø)

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 6688770...f105493. Read the comment docs.


(homebrew --HEAD) build error

==> Cloning https://github.com/crystal-lang/crystal.git
Updating ~/Library/Caches/Homebrew/crystal-lang--git
==> Checking out branch master
==> Downloading https://github.com/crystal-lang/crystal/releases/download/0.23.0/crystal-0.23.0-1-da
Already downloaded: ~/Library/Caches/Homebrew/crystal-lang--boot-0.23.0.tar.gz
==> make crystal release=true
==> Downloading https://github.com/crystal-lang/shards/archive/v0.7.1.tar.gz
Already downloaded: ~/Library/Caches/Homebrew/crystal-lang--shards-0.7.1.tar.gz
==> /private/tmp/crystal-lang-20171002-67016-108lnqf/bin/crystal build -o /private/tmp/crystal-lang-
Last 15 lines from ~/Library/Logs/Homebrew/crystal-lang/02.crystal:
Error in src/shards.cr:5: while requiring "./cli"

require "./cli"
^

in src/cli.cr:2: while requiring "./commands/*"

require "./commands/*"
^

in src/commands/prune.cr:2: while requiring "./file_utils": can't find file './file_utils' relative to '/private/tmp/crystal-lang--shards-20171002-67016-1prngkl/shards-0.7.1/src/commands'

require "./file_utils"

to me reads like it is rather crystal related and not homebrew (or I just screwed something)
if any further information/ trial is required please just ask
thanks

Can you give us the content of the file ~/Library/Logs/Homebrew/crystal-lang/02.crystal ?

sure
log

Ok so the error is:
in src/commands/prune.cr:2: while requiring "./file_utils": can't find file './file_utils' relative to '/private/tmp/crystal-lang--shards-20171002-67016-1prngkl/shards-0.7.1/src/commands'

require "./file_utils"
^

Meaning that a file can't be found. The thing is, this error (in shards project) has been fixed the 5 august by https://github.com/crystal-lang/shards/commit/5ebec014b1a86b931342d3d5f7dca34639917f3c, maybe your homebrew isn't uptodate (=> don't get the correct package?).
Maybe try something like brew update and try to re-install crystal?

I am actually pretty sure my hb is up to date
fetching the source should anyway probably be handled by git and thus an old hb should not be an issue?
brew reinstall crystal-lang --build-from-source works btw.

fetching the source should anyway probably be handled by git and thus an old hb should not be an issue?

I agree! I never touched homebrew myself, so I'm just emitting some speculation 😄
If your issue is resolved, can you close this issue?

the --build from source works, ``--HEAD` however does not
to me this seems as if the latest version of crystal has a flaw which the last tagged release does not

maybe it is because they hardcore the shards as
resource "shards" do
    url "https://github.com/crystal-lang/shards/archive/v0.7.1.tar.gz"
    sha256 "31de819c66518479682ec781a39ef42c157a1a8e6e865544194534e2567cb110"
  end

so this needs to be updated?

It's common for crystal (master) to break compatibility which is fixed in shards (master). I won't release a Shards version that may require crystal (master).

The homebrew formula should be fixed to use Shards master instead of the Shards release when --HEAD is passed.
sounds reasonable if the homebrew formula could do that

Closing, this isn't a crystal issue.

one of the maintainers did just resolve the outdated shards issue by modifying the formula however the formula does still not build
it would be good to know if the issue is just one with the current crystal master branch
issue
==> Checking out branch master
==> /private/tmp/crystal-lang-20171004-42284-lx4ezs/bin/crystal build -o /private/tmp/crystal-lang-20
Last 15 lines from /Users/joe/Library/Logs/Homebrew/crystal-lang/02.crystal:

      read_mapping(pull) do
      ^~~~~~~~~~~~

in src/spec.cr:73: instantiating 'read_mapping(YAML::PullParser)'

      read_mapping(pull) do
      ^~~~~~~~~~~~

in src/spec.cr:117: undefined method 'line_number' for YAML::PullParser

            raise YAML::ParseException.new("unknown attribute: #{ key }", pull.line_number, pull.column_number)
                                                                               ^~~~~~~~~~~

thanks

same issue after the latest merges, how can CI pass but building locally not?

@monouser7dig I've created an issue for it, see https://github.com/crystal-lang/shards/issues/175.

ok nice, would still be really interesting to know why CI considers this a passing build? :)

'cause CI does not consider building shards as a part of the testing :)

hm ok interesting decision but maybe worth it if shards have their own CI (but apparently they do not :o)

shards has it's own CI, it just tests based on the last crystal version, not master. Which is fine.
The correct way to package shards is to build the latest shards release with the latest crystal release and then build a crystal compiler from HEAD if you wish. Compiling crystal head requires the latest compiler release anyway, so it shouldn't be too hard.

the issue was that the latest shards release does not work with the crystal HEAD (that was the first issue)
maybe this has to be taken further by someone who knows the deps better than me and still likes homebrew

@monouser7dig my entire answer was that you shouldn't compile the latest shards release with crystal HEAD, you should compile it with the latest release of crystal. And the latest shards release is all you need for using crystal HEAD, not shards HEAD.


Add support for require object and $onInit controller method

Technically they are the features of directives and not components, but it would require too much fragile patching in versions prior to 1.5.
Directives can live fine without them, but they are essential for intercomponent communication, components have no alternatives to link and thus are barely usable without them.
Can we add them, please?

Do you think polyfilling this would be achievable? Unsure if there are things here that rely on Angular internals.

@toddmotto Absolutely. I believe it has to be something like this. It seems to be quite close to what $compile does in 1.5.

Awesome. Do you think hi-jacking the Controller would be better and then running this.oninit first?

Yes, I'm quite sure about that, that's what is done under the hood. All required controllers are available as instance properties when $onInit runs on instance, that's its purpose.
I fixed it, it should be preLink to maintain the proper order of $onInit execution.

Sounds good to me, let's get this added!

Ok. It looks like there's nothing to break down here.

Hi there,
On my project we are really interested on using this polyfill mainly for the $onInit method.
Do you have scheduled the merge of that pull request?
Thank you very much.

Hey. Super busy at moment, will try land it this week, you can pull in the
code from the PR for now though if you'd like as a temporary solution!
On 14 March 2016 at 11:40, Pablo Lázaro<EMAIL_ADDRESS>wrote:

Hi there,
On my project we are really interested on using this polyfill mainly for
the $onInit method.
Do you have scheduled the merge of that pull request?
Thank you very much.
—
Reply to this email directly or view it on GitHub
https://github.com/toddmotto/angular-component/issues/17#issuecomment-196274079
.

--
@toddmotto


chore: release main

:robot: I have created a release beep boop
errors: 1.0.1
1.0.1 (2022-08-01)
Documentation Changes

apply documentation amends (9d83857)
update 'chore' definition (383730a)

log-error: 1.3.0
1.3.0 (2022-08-01)
Features

add new app properties to error logs (c4d9421)

middleware-log-errors: 1.2.0
1.2.0 (2022-08-01)
Features

add new app properties to error logs (c4d9421)

Dependencies

The following workspace dependencies were updated

dependencies

@dotcom-reliability-kit/log-error bumped from ^1.2.0 to ^1.3.0

This PR was generated with Release Please. See documentation.

:robot: Release is at https://github.com/Financial-Times/dotcom-reliability-kit/releases/tag/errors-v1.0.1 :sunflower:

:robot: Release is at https://github.com/Financial-Times/dotcom-reliability-kit/releases/tag/log-error-v1.3.0 :sunflower:

:robot: Release is at https://github.com/Financial-Times/dotcom-reliability-kit/releases/tag/middleware-log-errors-v1.2.0 :sunflower:


Error when using parse mode MarkdownV2

When using sentTo function for telegram with parse mode MarkdownV2 and texts containing . the message isn't sent.
The log file shows following error message:
2022-08-14 12:56:03.782 | error | Cannot send message [chatId - xxxxxxxxx]: Error: ETELEGRAM: 400 Bad Request: can't parse entities: Character '.' is reserved and must be escaped with the preceding '\'

May it be possible to integrate a check box for auto-escape the following chars?
'_', '*', '[', ']', '(', ')', '~', '`', '>', '#', '+', '-', '=', '|', '{', '}', '.', '!'

Same issue here, especially with Blocklys that originally used Markdown instead of the newer Markdown_V2. It took me some time to even find and solve it.
A checkbox, as suggested by @Standarduser, will not work in my opinion, because the desired formatting would then also be escaped.
I used a small js function for now:
return text
   .replace(/\_/g, '\\_')
   .replace(/\*/g, '\\*')
   .replace(/\[/g, '\\[')
   .replace(/\]/g, '\\]')
   .replace(/\(/g, '\\(')
   .replace(/\)/g, '\\)')
   .replace(/\~/g, '\\~')
   .replace(/\`/g, '\\`')
   .replace(/\>/g, '\\>')
   .replace(/\#/g, '\\#')
   .replace(/\+/g, '\\+')
   .replace(/\-/g, '\\-')
   .replace(/\=/g, '\\=')
   .replace(/\|/g, '\\|')
   .replace(/\{/g, '\\{')
   .replace(/\}/g, '\\}')
   .replace(/\./g, '\\.')
   .replace(/\!/g, '\\!');

and I wish there was a Blockly tag for that - what do you all mean?

Added escape option:

If someone has a good translation of "escape chars" into german, I will appreciate it. (PR is better)

"Escapezeichen verwenden" or "Escapezeichen setzen"
Sorry, not able to send a PR at the moment

I would suggest to not escape formatting chars, because that would prevent users to make text italic or bold

From discussion I understand that in MarkdownV2 required it: https://core.telegram.org/bots/api#markdownv2-style
And it is configurable, so the user can prepare text himself as it needed.

You are right. When I opened this issue I did not notice that escaping all chars would prevent formatting of text.
I think * _ ~ | should not be escaped if they appear between char 1 and 126.
Maybe the best solution would be to make it configurable in instance settings?


update to use flowkit/v2 helper functions

Closes #???
Description

For contributor use:

[ ] Targeted PR against master branch
[ ] Linked to Github issue with discussion and accepted design OR link to spec that describes this work
[ ] Code follows the standards mentioned here
[ ] Updated relevant documentation
[ ] Re-reviewed Files changed in the Github PR explorer
[ ] Added appropriate labels

TODO: this doesnt work yet as theres a depedency issue with flixkit-go. Spoke to @bthaile and @jribbink  about it


Update: replace icon.svg

Since we have changed our service's icon recently, we'd like to update Bitrise step icon accordingly.
Is it possible to reflect the new icon without updating the step version?
We don't have any changes within our own step repository.
Please kindly let us know if updating version is necessary.
What to do if the build fails?
At the moment contributors do not have access to the CI workflow triggered by StepLib PRs. In case of a failed build, we ask for your patience. Maintainers of Bitrise Steplib will sort it out for you or inform you if any further action is needed.
New Pull Request Checklist
Please mark the points which you did / accept.

[x] I will not move an already shared step version's tag to another commit
[x] I read the Step Development Guideline
[x] I have a test for my Step, which can be run with bitrise run test (in the step's repository)
[x] I did run bitrise run audit-this-step (in the step's repository - note, if you don't have this workflow in your bitrise.yml, you can copy it from the step template.)
[x] I read and accept the Abandoned Step policy

Thanks, merged.


Added CanBCMSocket.

Implement support for SocketCAN's broadcastmanager.

@mbr I can also squash the commits. Let me know what you prefer.
What likely needs some efficiency improvement is how frames are read in CanBCMSocket::read_frames. Currently MAX_NFRAMES is read, which means 256*CanFrame is allocated for reading. I'm sure there is some way to only allocate nframes during reading but I can not figure it out. I tried reading n times from the buffer as you can see here, but the result is not what I expect (maybe due to alignment?).

Nevermind I missed the datagram part, so this is as efficient I guess as it can get.

Thanks for submitting this. I will need some time to review it, but I'll try to do that as soon as possible.
Thank you!

I've been working on this, but I am not 100% happy with the result yet, I'm sorry. This is not dead yet though!

Certainly, take your time. FYI I have sometimes had trouble with setting the filter using the BcmMessageHead struct. On some machines it worked with the vcan but not with can. The other way around on other machines as well (ARM/Intel). It turned out that  in C the frames array in the BcmMessageHead struct allocates 4bytes (although empty). This lead to adding the _pad: u32, to the end of the struct. Nevertheless this still leads to trouble with some cans where it previously worked without the additional 4 byte. Maybe you have an idea where this comes from.


Turn CSV files into JSON by default

Whenever we use CSV files, we have to run d3.csv on the imported file to do anything useful with them in cookie.
With upgrading sourdough to vite, I have a rollup plugin installed which automatically converts CSV imports into JSON arrays such that a simple import will work.
If we do end up converting cookie to vite (which I'm not sure about), this would still be nice to have even if it just means getting rid of the rollup plugin.
Are there any advantages to keeping a file as a CSV?

We would lose the columns property from running d3.csv:

The returned array also exposes a columns property containing the column names in input order (in contrast to Object.keys, whose iteration order is arbitrary).

An alternative to this is maybe we can do something at the cookie level instead where we set up a CSV transformer.

See https://github.com/MichiganDaily/cookie/pull/51


To add a mapping which associates signal indices to external nullifiers

This adds a mapping signal_index_to_external_nullifier to the Semaphore contract which lets clients look up the external nullifier per signal index.
This enables the "Signal History" table at the bottom of this demo UI, which is just a static website:
https://weijiekoh.github.io/semaphore-ui/

Closing this as it's outdated


jetbrains.idea-ultimate: Patch remote-dev-server

Description of changes
Resolves #153335
Things done

Built on platform(s)

[ ] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
22.05 Release Notes (or backporting 21.11 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[ ] Fits CONTRIBUTING.md.

Still a draft because the changes I made to linux.nix should probably just override jetbrains.idea-ultimate, like it's the case for e.g. CLion. Realized only afterwards. The remote-dev-server component at the moment does not work because JBR fails to start because it cannot load libz.so.
Happy to get comments about switching from "non JBR" to "with JBR". Is that something you wanted to avoid? Should I introduce a new package, e.g. jetbrains.idea-ultimate-jbr in addition to jetbrains.idea-ultimate?

Closed by #243533.


"Grand Unified Theory" Refactoring on query mechanisms to reduce duplication

After @CoreyKaylor 's compiled query work, the Include() feature, all the aggregate functions, and the batched query work I think it's time to reduce some of the duplication we have scattered around and maybe break up DocumentQuery because it's become a bit of a blob class.
If you look at how we handle the process of going from a database query all the way to the desired results, we have these responsibilities:

"Knowing" how to construct the database query to an NpgsqlCommand
Reading one row of a DbDataReader and building/resolving that into a single element in the returned values and/or calling an Include() callback. This is represented by the ISelector<T> interface today
Coercing the elements returned from step 2 into the proper return value (Count() v. ToList() v. FirstOrDefault())

The first is handled today by DocumentQuery and the Configure*****() methods on it. The second is the ISelector<T> interface that I introduced as part of the work for the Include() functionality. The third is pretty adhoc and duplicated across IQuerySession.Query<T>() queries, the batch querying, and the compiled queries.
To reduce the duplication, simplify the internals overall to make Marten easier to extend, and extend the compiled query support to batch querying, I would like to introduce the following abstraction:
    // Would still use an ISelector<TDoc> under the covers
    public interface IQueryHandler<T>
    {
        Type SourceType { get; }

        // It's done this way so that the same query handler can swing back
        // and forth between batched queries and standalone queries
        void ConfigureCommand(NpgsqlCommand command);

        // Sync
        T Handle(DbDataReader reader, IIdentityMap map);

        // Async
        Task<T> HandleAsync(DbDataReader reader, IIdentityMap map, CancellationToken token);
    }

My thought is that we'd move a lot of what is today DocumentQuery into these classes, probably subsume the IScalarCommandBuilder classes into this interface, and eliminate the similary-but-not-general-enough IDataReaderHandler interface in the batch querying.
My hope is to do this work before the v0.9 release, and use this as a precursor to doing #264 and #265.
What think you gentlefolk?

Did a lot of it today, still much more to go.


jquery.min.js:4                  The specified value "AVCACIQUE" does not conform to the required format, "yyyy-MM-dd".

Is your feature request related to a problem? Please describe.

Describe the solution you'd like:

Describe alternatives you've considered:

Additional context:

Closing as incomplete.


Add option to control ANSI color codes

I cant figure out how to disable terminal colors? I get this in Jenkins:
#13 naming to docker.io/library/vsc-devcontainer-example-c17e9ad94bc542f72c1906e018d524aa 0.0s done
#13 DONE 0.1s

#14 exporting cache
#14 preparing build cache for export done
#14 DONE 0.0s

[414703 ms] Start: Run: docker run --sig-proxy=false -a STDOUT -a STDERR --mount

@chrmarti Was this released in version 0.23.2? I'm still having the issue. It would be nice to have a flag or environment variable to control color output.

I only now released this with 0.24.0. Let me know if that fixes it for you. Thanks.


gene.py

[x] Check copyright
[x] Add typing https://realpython.com/python-type-checking/
[ ] Add google style docstrings https://google.github.io/styleguide/pyguide.html
[ ] Format with black

Done with first two subtasks. Currently looking at all docstring related stuff and finally will implement Black check on module before pushing to branch.


[stable/jenkins] allow templating of customInitContainers

What this PR does / why we need it:
This allows to template values for master.customInitContainers
Special notes for your reviewer:
Checklist
[Place an '[x]' (no spaces) in all applicable fields. Please remove unrelated fields.]

[x] DCO signed
[x] Chart Version bumped
[x] Variables are documented in the README.md
[x] title of the PR contains starts with chart name e.g. [stable/chart]

/assign @maorfr

/ok-to-test
/lgtm
Thanks for contributing!


Migrating to 2.x causes IR device (air purifier) not to work

Describe The Bug
Updating to 2x causes IR devices to stop working with error "connection type: undefined"
To Reproduce
Update existing setup from 1.x to 2.x, put in new token and secret, try to turn IR devices on and off
Expected behavior
Devices continue to work
Relevant log output
[10/14/2022, 4:48:23 PM] [SwitchBot] Initializing SwitchBot platform...
[10/14/2022, 4:48:23 PM] Homebridge v1.5.0 (HAP v0.10.3) (Homebridge 48BA) is running on port 51839.
[10/14/2022, 4:48:24 PM] [SwitchBot] Total SwitchBot Devices Found: 1
[10/14/2022, 4:48:24 PM] [SwitchBot] Total IR Devices Found: 1
[10/14/2022, 4:48:24 PM] [SwitchBot] Total IR Devices Found: 1
[10/14/2022, 4:48:24 PM] [SwitchBot] Restoring existing accessory from cache: Air Conditioner DeviceID: 02-202209132036-11252856
[10/14/2022, 4:48:24 PM] [@switchbot/homebridge-switchbot] This plugin generated a warning from the characteristic 'Firmware Revision': characteristic value expected string and received undefined. See https://homebridge.io/w/JtMGR for more info.
[10/14/2022, 4:48:24 PM] [@switchbot/homebridge-switchbot] This plugin generated a warning from the characteristic 'Firmware Revision': characteristic value expected string and received undefined. See https://homebridge.io/w/JtMGR for more info.
[10/14/2022, 4:48:35 PM] [SwitchBot] DIY Air Purifier: Air Conditioner Connection Type: undefined, commands will not be sent to OpenAPI
[10/14/2022, 4:48:41 PM] [SwitchBot] DIY Air Purifier: Air Conditioner Connection Type: undefined, commands will not be sent to OpenAPI

Config for homebridge-switchbot
{
            "name": "SwitchBot",
            "credentials": {
                "token": "xxxxxxxx",
                "secret": "xxxx",
                "notice": "Keep your Token & Secret a secret!"
            },
            "platform": "SwitchBot"
        }

Screenshots
I updated from 1.15 to 2.1.1 and my IR devices stopped working. My entire setup relies on OpenAPI (no bluetooth connections) which made my previous config really easy -  I just put in the auth token and everything works.
However, after updating to 2.1.1 and putting in both token and secret (I double checked and they are correct) the setup breaks. I shouldn't require any additional configuration to get this to continue working, right?
Device & Model
Intel NUC
Node.js Version
v18.10.0
NPM Version
v8.19.2
Homebridge Version
v1.5.0
Homebridge Switchbot Plugin Version
v2.1.1
Homebridge Config UI X Plugin Version
v4.50.1
Operating System
Windows 10 Enterprise

This is a know issue, set your connectionType for your deviceIDs

v2.1.2 will resolve this issue.


Fix video resolution selection

Looks like all the comparison operators for video resolution selection are backwards. Lower resolutions should be selected when maxWidth/maxHeight are greater than or equal to the resolution to include.
This bug was causing resolutions higher than maxWidth/maxHeight to be selected so if you chose for example 1280x720 as max resolution, it would also include 1920x1080 and none of the lower resolutions. For me at least, that mean't the live stream wouldn't work on Apple Watch or iPhone on LTE.

Thanks ;)


[GLUTEN-6317][CORE] Condition store the actual executed filter expressions

What changes were proposed in this pull request?
Fixes: #6317
How was this patch tested?

/Benchmark Velox TPCDS

/Benchmark Velox

There are some problems if condition store the actual executed filter expressions. Filter may have different output nullability,  It will affect DPP anti join. Spark also retains the condition that is pushed down to scan.


Drag and drop example uses old set_cursor_icon() that doesn't exist anymore in v0.20.1.

The drag and drop example uses set_cursor_icon() method which doesn't exist in newer version.
ui.ctx().set_cursor_icon(egui::CursorIcon::Grab);

How do I change the cursor icon in newer version?

The examples on master are for the code on master, and there is set_cursor_icon there.
To see examples for 0.20.1, go to https://github.com/emilk/egui/tree/0.20.1/examples


[VT Render] Investigate not breaking lines for node-pty

From MSFT:19012792

I didn't have any details in here. I presume that this was to track trying to not render a \r\n at the end of a line, so that a terminal would wrap the line itself. IIRC I tried that in RS5, but it broke something else, and there wasn't time to get it working for that release, so we reverted it.

Related: https://github.com/microsoft/terminal/issues/405

I've been up and down the internal PRs around this time, and I can't find where this originally came from. The original bug has no details in it other than "it was a revert of something in the RS5 timeframe". I recall making some change to wrapping and having to revert it, but I can't find it, or the original bug.
Chances are, we've already fixed this along the way. I'm closing this.


Cleanup examples

Maybe we can cleanup examples that doesn't show how to use this crate or test one of this crate functionnality? I think in particular to remove the examples relying on git dependencies.
I don't know if an example using RTFM (once 0.4 is out) is relevant. What do you think?

I think we should move out the examples entirely into a stm32f103-examples (sub-)crate or some potentially into their own crates. Anything that lives in the crate itself should would need to work on any MCU in that family (and since that requires a memory.x that would be kind of meh).

Having example can be really interesting. For example, to check that binaries still works on PR, of on PR to give something to test on real hardware.
I don't know if we can have more than one configuration on examples, to be able to compile for more than one MCU.

Oh, I agree that having code exercising functionality to test for proper compilation would be a nice thing. But the examples here are highly target dependent and need additional hardware to be runnable at all so I'd prefer to have it in board specific repos rather than in the multi-MCU HAL if that makes sense.

adding an stm32f103-examples, and build them on std32f1xx's travis?
We'll have to find a way to use the current code with the examples to do that, but it looks like a good solution.

adding an stm32f103-examples, and build them on std32f1xx's travis?

Yes, that.

Or move it out if it's really more a project in its own.

A crate in the repo sounds like a better idea to me, if nothing else it makes the examples easier to find

An idea.
Add cargo generate option --from-example examplename. This will move example into main.rs.
We can store all templates and examples for different microcontrollers in the one repository in different branches (already working) or in different folders.

@burrbull Sorry, but now I'm confused. :-D Which crate is the one to rule them all and who's going to maintain that?

cargo generate --git https://github.com/rust-embedded/cortex-m-quickstart --branch stm32f103c8 --from-example blinky
cargo run

I think the current example setup works quite well, so 'll close this for now


Outdated URL

Looks like the URL used here is outdated
https://github.com/conda-forge/libtvm-feedstock/blob/a4c7fd2fc8cfdb433c66f20c9cab103679431f54/recipe/meta.yaml#L10
https://github.com/conda-forge/libtvm-feedstock/blob/a4c7fd2fc8cfdb433c66f20c9cab103679431f54/recipe/meta.yaml#L83
The current one appears to be: https://github.com/apache/tvm

@jakirkham it seems this feedstock is unmaintained, can we just archive it? cc: @conda-forge/libtvm

Thanks, @ngam. Maybe you could open a new issue like this one (https://github.com/conda-forge/cub-feedstock/issues/24) and explain a bit why you think it's difficult to maintain, before archiving the feedstock (after which we can't make new posts)?


Moved highlighting documentation to foundations > typography

What is the context of this PR?
Closes #1869
Also...

Changed terminology 'foundations' to 'layout' or 'styles' within each sub-group – to make more sense.

How to review
Check docs and check no ill effects of moving scss file

Close old discussion when merging: https://github.com/ONSdigital/design-system/discussions/24


Port N14 Mapper Assets (#1315)

By request from Old Dance Jacket, who wanted to be able to use N14 Mapping Assets in SSS.
:cl:

add: Added a large number of mapping assets from Nuclear14

Not using SSS namespace because i would rather die, and we are going to be modifying the base game so much I don't care about keeping everything in the namespace.

also this adds explosive barrels :^)


Cannot return null for non-nullable field

Hello, I tried to use the Prisma 2, but I have an error when I execute this request :
query {
  post(id:"1"){
    id
    title
        author {
        id
        }
    }
}

In my mutation, I have this :
post(parent, args, { prisma }, info) {
    const post = prisma.post.findOne({
        where: {
            id: parseInt(args.id)
        }
    }, info)

    if(!post) {
        throw new Error('Post not found.')
    }

    return post
}

In my schema.graphql :
type User {
    id: ID!
    name: String!
    email: String
    password: String!
}

type Post {
    id: ID!
    title: String!
    content: String!
    published: Boolean!
    author: User!
}

type Query {
    post(id: ID!): Post!
}

Bug description
When I execute my request, I have this error :
{
  "errors": [
    {
      "message": "Cannot return null for non-nullable field Post.author.",
      "locations": [
        {
          "line": 5,
          "column": 5
        }
      ],
      "path": [
        "post",
        "author"
      ],
      "extensions": {
        "code": "INTERNAL_SERVER_ERROR",
        "exception": {
          "stacktrace": [
            "Error: Cannot return null for non-nullable field Post.author.",
            "    at completeValue (/Users/user/Sites/prisma2/node_modules/graphql/execution/execute.js:578:13)",
            "    at completeValueCatchingError (/Users/user/Sites/prisma2/node_modules/graphql/execution/execute.js:513:19)",
            "    at resolveField (/Users/user/Sites/prisma2/node_modules/graphql/execution/execute.js:444:10)",
            "    at executeFields (/Users/user/Sites/prisma2/node_modules/graphql/execution/execute.js:280:18)",
            "    at collectAndExecuteSubfields (/Users/user/Sites/prisma2/node_modules/graphql/execution/execute.js:731:10)",
            "    at completeObjectValue (/Users/user/Sites/prisma2/node_modules/graphql/execution/execute.js:721:10)",
            "    at completeValue (/Users/user/Sites/prisma2/node_modules/graphql/execution/execute.js:609:12)",
            "    at completeValue (/Users/user/Sites/prisma2/node_modules/graphql/execution/execute.js:575:21)",
            "    at /Users/user/Sites/prisma2/node_modules/graphql/execution/execute.js:510:16",
            "    at processTicksAndRejections (internal/process/task_queues.js:97:5)"
          ]
        }
      }
    }
  ],
  "data": null
}

But If I execute only :
query {
  post(id:"1"){
    id
    title
    }
}

without author, I have no error (but I have no author)...
Environment & setup

OS: Mac OS
Database: MariaDB
Prisma version: 2.0.0
Node.js version: 13.10.1
apollo-server: 2.14.4
SQL:

CREATE TABLE User (
  id INTEGER PRIMARY KEY AUTO_INCREMENT NOT NULL,
  name VARCHAR(255),
  email VARCHAR(255) UNIQUE NOT NULL,
  password VARCHAR(255) NOT NULL
);
CREATE TABLE Post (
  id INTEGER PRIMARY KEY AUTO_INCREMENT NOT NULL,
  title VARCHAR(255) NOT NULL,
  createdAt TIMESTAMP NOT NULL DEFAULT now(),
  content TEXT,
  published BOOLEAN NOT NULL DEFAULT false,
  fk_user_id INTEGER NOT NULL,
  FOREIGN KEY (fk_user_id) REFERENCES User(id)
);

schema.prisma:

model Post {
  content    String?
  createdAt  DateTime @default(now())
  fk_user_id Int
  id         Int      @default(autoincrement()) @id
  published  Boolean  @default(false)
  title      String
  author     User     @relation(fields: [fk_user_id], references: [id])

  @@index([fk_user_id], name: "fk_user_id")
}

model User {
  email    String   @unique
  id       Int      @default(autoincrement()) @id
  name     String?
  password String   @default("")
  Post     Post[]
  Profile  Profile?
}

Do you have an idea how can I have the author ?

You need to add resolver for Post.author to your graphql server.
Check out Write your GraphQL type resolvers in prisma docs.

I added include: {author: true} to bind :
post(parent, args, { prisma }) {
const post = prisma.post.findOne({
where: {
id: parseInt(args.id, 10),
},
include: {
author: true,
},
})
if (!post) {
    throw new Error("Post not found.")
}

return post

}

That is actually not correct solution. You will run into problems and you will need more and more code to fix it.
Please read something about graphql resolvers, for example here.

Hi @dodas it's better to have a specific resolver ?
const Post = {
    author(parent, args, { prisma }, info) {
        return prisma.user.findOne({
            where: {
                id: parseInt(parent.fk_user_id)
            }
        }, info)
    },
}

export { Post as default }

and import it :
import Query from "./Query"
import Mutation from "./Mutation"
import Post from "./Post"

const resolvers = {
    Query,
    Mutation,
    Post,
}

export { resolvers as default }

Yes, I think you would need exactly this:
Query: {
  Post: {
    author:(parent, args, { prisma }) {
      return prisma.post.findOne({
        where: { id: parent.id }
      }).author()
    }
  }
}

facing same problem @dodas  can you help me ?


Restarts Randomly without Blue screen

Describe the Bug
after debloating, laptop restarts randomly. No blue screen.
I have tried to replicate the issue but failed to do so.
Windows11 Home x64
Ver 10.0.22621 Build 22621
I have disabled fast boot, updated graphics driver. It's a new laptop.
How can this issue be reproduced?
I have tried to replicate the issue but failed to do so.
Expected behavior?
No response
Operating System
Windows 11
Operating System Version
10.0.22621 Build 22621
Screenshots
No response
Logs
No response
Additional Info
No response

Things like BSoD happens randomly, even I got somes, but I still don't think its 100% related to the script. All of the people I know whom tested and debloated didn't got any BSoD, so it's mostly a rare thing.


Fix indexer to filter out invalid msg

What changed?
Fix indexer to filter out invalid msg

Why?
ES has a limit of 512 for docID size. Cadence use wid~rid as docID for visibility in ES, but currently do not have limit on wid size.
For users who use advanced visibility on ES, this PR will filter out those workflow with wid too long.
Team is also considering a breaking change to limit wid in near future.

How did you test it?
local and staging tests

Potential risks
For user who use advanced visibility on ES, if the wid+rid is longer than 512, it will not be searchable because of this change.

Coverage decreased (-0.2%) to 66.821% when pulling 843ea6158f0a8ac9f599695a0a6418572935f5eb on fixindexer into 3a3ba20f200db7c560a83872f2c5f7812e9f69be on master.


Update README.md

Changed "URL into a PDF file" into "Web Page into PDF file" in the README for Issue #37

Thank you, Make a PR, so I can merge it


nova.cz not working

Checklist

[x] I'm reporting a broken site support
[x] I've verified that I'm running youtube-dl version 2020.03.24
[x] I've checked that all provided URLs are alive and playable in a browser
[x] I've checked that all URLs and arguments with special characters are properly quoted or escaped
[x] I've searched the bugtracker for similar issues including closed ones

Verbose log

[debug] System config: []
[debug] User config: []
[debug] Custom config: []
[debug] Command-line args: ['--verbose', 'https://novaplus.nova.cz/porad/superstar-2020/epizoda/43471-superstar-2020-7-dil']
[debug] Encodings: locale cp1250, fs mbcs, out cp852, pref cp1250
[debug] youtube-dl version 2020.03.24
[debug] Python version 3.4.4 (CPython) - Windows-10-10.0.17763
[debug] exe versions: ffmpeg git-2019-12-15-ed9279a
[debug] Proxy map: {}
[Nova] 43471-superstar-2020-7-dil: Downloading webpage
[NovaEmbed] GRlTXX064Hr: Downloading webpage
ERROR: Unable to extract formats; please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; type  youtube-dl -U  to update. Be sure to call youtube-dl with the --verbose flag and include its complete output.
Traceback (most recent call last):
  File "C:\Users\dst\AppData\Roaming\Build archive\youtube-dl\ytdl-org\tmpjwbwqymm\build\youtube_dl\YoutubeDL.py", line 797, in extract_info
  File "C:\Users\dst\AppData\Roaming\Build archive\youtube-dl\ytdl-org\tmpjwbwqymm\build\youtube_dl\extractor\common.py", line 530, in extract
  File "C:\Users\dst\AppData\Roaming\Build archive\youtube-dl\ytdl-org\tmpjwbwqymm\build\youtube_dl\extractor\nova.py", line 38, in _real_extract
  File "C:\Users\dst\AppData\Roaming\Build archive\youtube-dl\ytdl-org\tmpjwbwqymm\build\youtube_dl\extractor\common.py", line 1005, in _search_regex
youtube_dl.utils.RegexNotFoundError: Unable to extract formats; please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; type  youtube-dl -U  to update. Be sure to call youtube-dl with the --verbose flag and include its complete output.

Description

Here is tested and working free Czech VPN: https://ceskevpn.cz/ovpnconfigs/WE_CESKEVPN_CZ_18_2.ovpn

---------------------------
OpenVPN GUI
---------------------------
Connecting to WE_CESKEVPN_CZ_18_2 has failed.
---------------------------
OK
---------------------------

It is working for me, but here is another (also tested and working): https://www.vpnjantit.com/assets/cz1.vpnjantit.com.zip
Username: uifahfliub-vpnjantit.com
Password: uifahfliub


Automated Workflow For Blocked Label

Describe what should be investigated or refactored
When the blocked label is added a workflow is triggered to automatically add it to the blocked column. Similar to the triage label.
Links to any relevant code
Using triage as an example https://github.com/defenseunicorns/lula/blob/main/.github/workflows/triage-label.yaml

not against this but I am curious how the locality of the workflow will impact permissions.
The triage label is issue-centric - an issue is created -> workflow runs -> issue is modified
Adding something to the Blocked column is now overlaying issues onto a project. Just for scope and weight purposes I would imagine this to be more complex then the triage example - but we shall see.


I got a detection speed of 4fps

Hi！
First of all, thank you for your willingness to share your work. When I found a problem in the reproduction process, I connected a 1080p camera video stream with a detection rate of 4 fps.
Here are some of my parameters：
Model : centerface_736x1280.onnx
Hardware : Jetson Nano 4G 、Jetpack 4.5.1

Hi！
I found an interesting phenomenon，When the batchsize is 1, the size of the TRT model obtained by using model centerface_736x1280.onnx is 13M, but model a is only 7.18M, but the size of the TRT model obtained by using model centerface.onnx is 7M, but both models are centerface, but the input sizes are different.


Update application yaml for aws 186631586

When I push PR's for il-gcc it keeps creating the PR here in the starter app 😞, closing.


Automated cherry pick of #6650: fix migrate with multi local storage

Cherry pick of #6650 on release/3.2.
#6650: fix migrate with multi local storage

Codecov Report

Merging #6651 into release/3.2 will decrease coverage by 0.01%.
The diff coverage is 0.00%.

@@               Coverage Diff               @@
##           release/3.2    #6651      +/-   ##
===============================================
- Coverage         7.03%    7.02%   -0.02%
===============================================
  Files              650      650
  Lines           111812   111827      +15
===============================================
- Hits              7869     7856      -13
- Misses          103334   103362      +28
  Partials           609      609

Flag
Coverage Δ

#aFlag
7.02% <0.00%> (-0.02%)
:arrow_down:

Impacted Files
Coverage Δ

pkg/baremetal/tasks/bm_register.go
0.00% <0.00%> (ø)

pkg/baremetal/tasks/ipmiprobe.go
0.00% <0.00%> (ø)

pkg/cloudcommon/db/namevalidator.go
0.00% <0.00%> (ø)

pkg/cloudcommon/db/opslog.go
1.50% <0.00%> (-0.01%)
:arrow_down:

pkg/cloudcommon/db/scoperesource.go
0.00% <0.00%> (ø)

pkg/cloudcommon/db/tenantcache.go
2.93% <0.00%> (-0.35%)
:arrow_down:

pkg/cloudcommon/policy/resources.go
0.00% <ø> (ø)

pkg/cloudprovider/cloudprovider.go
1.53% <0.00%> (ø)

pkg/cloudprovider/instance.go
0.00% <ø> (ø)

pkg/cloudprovider/objectstore.go
2.48% <0.00%> (-0.01%)
:arrow_down:

... and 20 more

Continue to review full report at Codecov.

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


[PLAT-12231] Browser error correlation

Goal
Implement error correlation for browser performance
Design
Call addOnError from bugsnag notifier when provided as a configuration option
Changeset
Added new bugsnag configuration option to core client
Testing
Added unit tests for adding span and trace id to a reported error

Could we add a maze runner test for this?

Covered in PLAT-12233


⚠ Secret Website 3 has degraded performance

In a59f6d9, Secret Website 3 ($SECRET_WEBSITE_3) experienced degraded performance:

HTTP code: 200
Response time: 2136 ms

Resolved: Secret Website 3 performance has improved in 8c4e071.


HAWQ-1458. The maximum value of guc share_input_scan_wait_lockfile_timeout should be greater than the default value.

fix a bug which cause HAWQ debug version failed in initializing.

LGTM. +1

+1


Google UI partially breaks when using extensions

Icons in both the GMail and Google Calendar web interfaces get squeezed or misaligned when PaperMemory is activated.
Desktop (please complete the following information):

OS: MacOS Monterey
Browser chrome
Version 100
PaperMemory version 0.5.2

Interesting I noticed the same thing on my computer but had not realized it was PM's fault. Thank you for reporting I'll investigate

Ok found it I changed the content scripts' css it'll show in the next release. Or you can install from source in the mean time


Display of events and their duration on calendar is incorrect

In the above example, the event "Sleeping" was set from 12:00 AM 12:00 pm (12 hours), however the duration on the calendar shows that it goes from 12:00 AM to 12:00 AM (24 Hours), completely skipping the 12 PM stop time.

Resolved in the pull request
https://gyazo.com/3d071f885b102eeb6be97aad8febd8c3


fix(swagger-ui-web-component): ensure there are 4 grid columns for each operation

Summary
Currently there is some CSS that has 3 columns, but a recently upgraded version of Swagger UI regrouped the elements so that it is now necessary to have 4 columns to prevent the open/collapse toggle button from flowing into the next row.
The current CSS rule that is offending is:
.swagger-ui .opblock .opblock-summary {
  grid-template-columns: 80px auto 30px;
}

before:

after:

Resources

Creating a package docs

Need to verify this has the expected result, but wanted to throw up this PR since I noticed the issue


Release building

Currently a release build fails with the following:
/home/crc32/Documents/fossil/build/app/tmp/kapt3/stubs/release/io/rebble/fossil/di/AppComponent.java:8: error: [Dagger/DependencyCycle] Found a dependency cycle:
public abstract interface AppComponent {
                ^
      error.NonExistentClass is injected at
          io.rebble.fossil.di.LibPebbleModule.provideBlobDbService(protocolHandler)
      error.NonExistentClass is requested at
          io.rebble.fossil.di.AppComponent.createProtocolHandler()

From what I see, issue lies with libpebble. In debug mode, we use libpebblecommon-android-debug artifact which is normal, but on release mode we use libpebblecommon-android which appears to be completely empty:

This is a blocker for uploading to Play Store for internal alpha testing.

This is a blocker for uploading to Play Store for internal alpha testing.

@crc-32 & @matejdro  I can confirm this was fixed by https://github.com/pebble-dev/mobile-app/commit/339a4ae47388ab9e5899a3324439098cbb0eaafc and https://github.com/pebble-dev/mobile-app/commit/74a3a93c157d8e7edd02b2aa2475bc88a8d77010

Awesome


UAT | Filing | Mandatory Fields Check

Describe the bug
Let's allow the users to toggle between complainant details, cheque details, etc. without filling in the mandatory fields. It would be nice to let the user explore the system freely and understand the details to be filled before they start filing. They will also be able to collect all details at one place before filing.
We could let the check of filling mandatory details be there before clicking the Submit button.
To Reproduce
Steps to reproduce the behavior:

Go to '...'
Click on '....'
Scroll down to '....'
See error

Expected behavior
A clear and concise description of what you expected to happen.
Screenshots
If applicable, add screenshots to help explain your problem.
Desktop (please complete the following information):

OS: [e.g. iOS]
Browser [e.g. chrome, safari]
Version [e.g. 22]

Smartphone (please complete the following information):

Device: [e.g. iPhone6]
OS: [e.g. iOS8.1]
Browser [e.g. stock browser, safari]
Version [e.g. 22]

Additional context
Add any other context about the problem here.

Tested in QA env, Working as Expected, User can toggle between different sections without entering the details
Ref,
https://jam.dev/c/a72f71fe-7e6d-4955-86bf-7d970c5f1a60


(1) Add ISD Documentation

[ ] Document functions
[ ] Document that this should be a class
[ ] Document the iPython notebook(s)

Assigning myself for now, still have hours to burn :fire:


 This loosens up the requirements to find "any image wrapped in a link to the artwork page" instead of trying to target a specific UI element class. It also uses one tracking implementation to fix the mobile issue (nice benefit of having these merged now).
One question? Is it okay to use slugs in place of the ids? It was hard to parse those out this time.
It's possible we'll want to consider redoing this in React Tracking in our efforts to consolidate the artwork brick component across platforms cc @alloy

Thanks!


Console reporter improvenments

To review the console output and refactor to become more readable.

PR >>> https://github.com/VadimNastoyashchy/scripterio/pull/5