LogManager threw an exception in a static constructor

Hello. I'm going to use NLog with SharePoint 2010.
I have my custom TimerJob that I want to log.
In a static constructor I tries to set config throught Configuration API and implenet Logger like that:
static TimerJob()
        {
            // Step 1. Create configuration object
            var config = new LoggingConfiguration();

            // Step 2. Create targets and add them to the configuration
            var eventLogInfoTarget = new EventLogTarget();
            config.AddTarget("eventLogInfoTarget", eventLogInfoTarget);

            // Step 3. Set target properties
            eventLogInfoTarget.Layout = "${longdate}\n${logger}\n${level:uppercase=true}\n${message}";
            eventLogInfoTarget.Source = typeof(TimerJob).Namespace;
            eventLogInfoTarget.EventId = "1";

            // Step 4. Define rules
            var rule1 = new LoggingRule(typeof(TimerJob).FullName, LogLevel.Info, eventLogInfoTarget);
            config.LoggingRules.Add(rule1);

            // Step 5. Activate the configuration
            LogManager.Configuration = config;
            Logger = LogManager.GetCurrentClassLogger();
        }

Application compiles well, but when I deploy it to my SharePoint Application I get:
Error occurred in deployment step 'Add Solution': The type initializer for 'TimerJob' threw an exception.
I also tried to add:
InternalLogger.LogFile = @"C:\NLOG.txt";
InternalLogger.LogLevel = LogLevel.Trace;
LogManager.ThrowExceptions = true;
But nothing happened.
Anyone knows why LogManager act like this in a static context?

This is bug which is probably solved in 4.0. Do you have the full exception message with stack trace? Check https://github.com/NLog/NLog/blob/master/CONTRIBUTING.md
The RC of 4.0 is also published. Could you check if it's solved in that version?

Thank for replay.
I haven't StackTrace. This message is everything what I have.
But I'll try 4.0 RC.

Do you have problems when using the Nuget package of NLog 4.0 RC (with .Net 3.5)? Or do you build manually?

Please capture the stacktrace. That would really help. It should be in the internal log or if you capture it with visual studio, in details

I have build NLog 4.0 RC manually from the source code. And I dunno how to get StackTrace, because   it's not possible to enable debug while deploying and internal log not creates as I told in my 1st post (maybe I should configure NLog in other way to write log?)

The error "The type initializer for 'TimerJob' threw an exception" don't tell us much. This errors means that creating the static "instance" of TimerJob failed. The cause of this exception is in the innerexception / stacktrace of the Exception.
There is a big change that the exception is raised before even creating the Logconfiguration and so nothing is logged to NLog.
This should be really easy to debug in Visual Studio.
For example:
This will raise the same Exception and won't ever reach the static contructor.

    public class TimerJob
    {
        static TimerJob()
        {
            // Step 1. Create configuration object
            var config = new LoggingConfiguration();

            // Step 2. Create targets and add them to the configuration
            var eventLogInfoTarget = new EventLogTarget();
            config.AddTarget("eventLogInfoTarget", eventLogInfoTarget);

            // Step 3. Set target properties
            eventLogInfoTarget.Layout = "${longdate}\n${logger}\n${level:uppercase=true}\n${message}";
            eventLogInfoTarget.Source = typeof(TimerJob).Namespace;
            eventLogInfoTarget.EventId = "1";

            // Step 4. Define rules
            var rule1 = new LoggingRule(typeof(TimerJob).FullName, LogLevel.Info, eventLogInfoTarget);
            config.LoggingRules.Add(rule1);

            // Step 5. Activate the configuration
            LogManager.Configuration = config;
            Logger = LogManager.GetCurrentClassLogger();
        }

        public static Logger Logger { get; set; }

        public static String x = Logger.ToString();
    }

Is your issue solved?

Nope. I've decided to use another framework.
2015-06-08 23:39 GMT+03:00 Julian Verdurmen<EMAIL_ADDRESS>
Is your issue solved?
—
Reply to this email directly or view it on GitHub
https://github.com/NLog/NLog/issues/721#issuecomment-110131090.


Thredded incorrectly insists mods have unread posts

Hoping to resolve with:
  user = User.first
  Thredded::Topic.unread(user).each do |topic|
    Thredded::UserTopicReadState.touch!(user.id, topic.last_post)
  end

overwrite_newer: true was the key here, maybe because of deleted posts that were newer and then deleted?....


"data-ts"

"anime dl "https://www1.9anime.nl/watch/ace-of-diamond-second-season.8xo/njwp0n" -e 38:51 -q 720p"
I enter the above command and this pops up
"anime: anime-downloader 3.6.1
anime: Extracting episode info from page
'data-ts'"
I dont understand what I am doing wrong. I have entered same commands before to download animes. But today i keep getting this "data-ts". I dont understand why?
Please help me out. Also im not a techie so dont mind the layman language i have used here

It's the website you're trying to get the anime from I believe. Gogoanime works great for me,.

Which version r u using?

@em0tionull Yeah you are right. Lately I have been using Kissanime.ru and its working fine for me. The problem is with 9anime site. issue #183 explains it might be because of the captcha they recently introduced.
I dont really have a problem anymore so i think i am gonna close this issue

Which version are u using?
Rewrite or master?

@kougamishinyaa I think master.

I am also using master but only animepahe is working for me,the rest r all dead

Try kissanime.ru
The normal kissanime isn't working. Kissanime.ru is cloudflare supported which should work.
You can also try downloading through torrents from Horrible subs or nayee.is
PS animepahe isn't working for me


Converting Fixnum to Integer

Fixnum is now deprecated in Ruby 2.4.0. This change should be fully backward compatible, although I haven't tested it in JRuby 1.9 (which the original code referenced in a comment).

Closes #123

v0.7.1 has been release. Thanks, @benrugg!

Awesome, thanks for releasing it.

On Apr 21, 2017, at 5:53 PM, Steven Harman<EMAIL_ADDRESS>wrote:
v0.7.1 has been release. Thanks, @benrugg!
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.

Thank you for maintaining it!


Describe B-splines

sames smoothness as natural splines, not interpolating (only approximating) but local control
"B" stands for "basis". Why?
control points are called "de Boor" control points?
curve lies within the convex hull of the control points (like Bézier)
starting point for NURBS
generalization of Bezier splines: "If n = p (i.e., the degree of a B-spline curve is equal to n, the number of control points minus 1), and there are 2(p + 1) = 2(n + 1) knots with p + 1 of them clamped at each end, this B-spline curve reduces to a Bézier curve."

The basis matrix for a uniform cubic B-spline is shown (without derivation) in:
Clark, James H. “Parametric Curves, Surfaces and Volumes in Computer Graphics and Computer-Aided Geometric Design.” Technical Report. Computer Systems Laboratory, Stanford University, November 1981. http://i.stanford.edu/pub/cstr/reports/csl/tr/81/221/CSL-TR-81-221.pdf.

The basis matrix is also shown in:
Barsky, Brian A. “End Conditions and Boundary Conditions for Uniform B-Spline Curve and Surface Representations.” Computers in Industry, Double Issue- In Memory of Steven Anson Coons, 3, no. 1 (March 1, 1982): 17–29. https://doi.org/10.1016/0166-3615(82)90028-8.
For a more detailed derivation we are referred to:
Barsky, B.A. (1983) A Study of the Parametric Uniform B-spline Curve and Surface Representations. Technical Report UCB/CSD-83-118. EECS Department University of California, Berkeley. Available at: https://www2.eecs.berkeley.edu/Pubs/TechRpts/1983/5671.html.


[rviz_common] remove catch all try-catch clause

In favor of exact exceptions to catch... See:

https://github.com/ros2/rviz/blob/ca8f42e43ba9eaa6e15434b47906d7580da1c14a/rviz_common/src/rviz_common/frame_manager.cpp#L96-L99

Judging the code as-is, an error could only be thrown while constructing the Time. We use the explict constructor here
https://github.com/ros2/rclcpp/blob/989084b3de84000a15fb4bd5a7d63929001cd769/rclcpp/src/rclcpp/time.cpp#L98-L103
Looking at the lines above, errors can only happen when initialising time source or time point. Since we use default parameters everywhere, this shouldn't happen and if it does, it will also happen in rclcpp::Time::now():

https://github.com/ros2/rcl/blob/38d3725262e25d5beb4c2b5c1155b74402d371d1/rcl/include/rcl/time.h#L112-L121
https://github.com/ros2/rcl/blob/38d3725262e25d5beb4c2b5c1155b74402d371d1/rcl/include/rcl/time.h#L252-L259
and here is the code for rclcpp::Time::now(), which does a little bit of checking on top of calling a constructor:
https://github.com/ros2/rclcpp/blob/989084b3de84000a15fb4bd5a7d63929001cd769/rclcpp/src/rclcpp/time.cpp#L63-L84

This implies that we cannot really implement a good fallback in case of failure, hence I believe we should not even try to catch anything at the moment.

This implies that we cannot really implement a good fallback in case of failure, hence I believe we should not even try to catch anything at the moment.

:+1: for a pr to remove it.


RKE1 etcd snapshot failure retention

Rancher Server Setup

Rancher version: v2.6-head

Information about the Cluster

Cluster Type (Local/Downstream): Downstream

If downstream, what type of cluster? (Custom/Imported or specify provider for Hosted/Infrastructure Provider): RKE1

Describe the bug

RKE1 etcd snapshot config specifies a retention, which is the maximum number of snapshots retained until rancher starts to delete older snapshots. Currently, there is no check ensuring that rancher is retaining at least retention successful snapshots, creating the possibility that rancher has no successful snapshots. Failed snapshots should be retained separately from successful snapshots, so that the user doesn't end up in a situation where the only snapshots are failed ones.
To Reproduce

Provision a cluster with a retention of 2 and a snapshot interval of 1 hour.
Wait for initial etcd snapshot with an s3 endpoint
Watch as the backup is successful
Delete the s3 bucket/folder
Wait for next recurring etcd snapshot
Snapshot should fail with error status
Wait for next recurring etcd snapshot
Watch as last successful snapshot is deleted to make room for failed snapshot

Result
The last successful snapshots is deleted, leaving 2 failed snapshots.
Expected Result

There are at least retention successful snapshots retained (how many failed snapshots should be retained requires discussion)
Screenshots

Additional context

We should count and keep the successful and failed snapshots separately up to the limit.

Root cause
Previously, failed snapshots were counted towards the total # of recurring backups specified in a cluster's spec.BackupConfig.Retention. This meant that for some cluster configurations, it would be possible for backups to start failing (for example, in the event that the s3 credential was no longer authorized), and no usable backups to be available for restore.
What was fixed, or what changes have occurred
A fix was temporarily put in to retain failed etcd backups indefinitely, separate from what is specified in the retention field. Obviously this fix could not be permanent, as it would create significant bloat to etcd after some time, with no way easy for users to delete failed backups. This was fixed by retaining retention number of failed recurring backups as well. Failed manual backups are still retained indefinitely, for the same reason that a manually requested backups is more important than a recurring one, and also a less frequent operation.
Areas or cases that should be tested

Creating a cluster with a retention of 2, and having 2 successful backups and 2 failed backups, should only rotate the failed or successful backups on subsequent recurring backups. (Any # other than 2 can be chosen, however the shortest interval is 1 hour, so the minimal test case would take 6 hours)

What areas could experience regressions?

N/A

Are the repro steps accurate/minimal?
Replace the rke-tools image with something that would cause it to fail (ubuntu:latest should suffice), by provisioning the node, ssh-ing to the node and doing a docker pull ubuntu:latest && docker tag ubuntu:latest rancher/rke-tools:v0.1.80 (or whatever version of rke-tools rke expects to use for this kubernetes version (this is also doable in KDM, but is more of an involved process, since it requires using a custom KDM, RKE, and rancher). Alternatively, it is possible to SSH to the node and stop the etcd container docker stop etcd, but another PR will be created and merged soon to address behavior in rke1 when a snapshot cannot possible be completed on a node (i.e. in the case of etcd not running on one of the nodes in an HA setup), versus a snapshot cannot be completed on any node (a bad rke-tools image), as in the case of the former as long as the snapshot exists on one of the nodes, it can be restored (and will even be restorable from s3), so it should not be failed.

Test Environment:
Rancher version: v2.6-head f4d09c7
Rancher cluster type: HA
Docker version: 20.10
Downstream cluster type: RKE1

Testing:
Tested this issue with the following steps:

Provision a cluster with a retention of 2 and a snapshot interval of 1 hour
Wait for initial etcd snapshot with an s3 endpoint
Watch as the backup is successful
Delete the s3 bucket/folder
Wait for next recurring etcd snapshot
Snapshot should fail with error status
Wait for next recurring etcd snapshot
Watch as last successful snapshot is deleted to make room for failed snapshot

Result - Pass
The successful snapshot is retained and the failed snapshots are retained past the retention value, which is expected.


Fix missing labels Volterra multi-cloud demo

Issue
The Volterra multi-cloud demo folder is using labels to identify sites. When you re-run terraform, then dynamically generated labels as a result of the Volterra site creation will be deleted.
Symptoms
GPS coordinates disappear. The cloud provider metadata is deleted and so is the cloud icon. Many more dynamic key:value items are generated during a Volterra site creation and those are removed at next terraform apply.
Fix
Volterra terraform module recently released some new updates v0.11.4 with a new resource object. Will need to review code and update where needed.
Example...
resource "volterra_cloud_site_labels" "labels" {
  name             = volterra_aws_vpc_site.main.name
  site_type        = "aws_vpc_site"
  labels           = local.volterraCommonLabels
  ignore_on_delete = true
}

fixed with https://github.com/f5devcentral/f5-digital-customer-engagement-center/pull/184


Mangarawplus　Sourse request

Source name
mangarawplus
Source link
https://mangarawplus.org
Source language
japanese
Other details
No response
Acknowledgements

[X] I have checked that the extension does not already exist by searching the extension listing and verified it does not appear there.
[X] I have searched the existing issues and this is a new ticket, NOT a duplicate or related to another open or closed issue.
[X] I have written a meaningful title with the source name.
[ ] I have admitted that I am a clown by having checked this box, as I have not read these acknowledgements.
[X] I will fill out all of the requested information in this form.

Uses madara theme!


egg.json $schema: url @ gets escaped in vscode

🐛 Bug Report

Adding a $schema reference url in your egg.json results in VS Code trying to resolve with the @ symbol url escaped as %40
e.g.
{
  "$schema"<EMAIL_ADDRESS>  // Unable to load schema from 'https://x.nest.land/eggs%400.3.1/src/schema.json': Request vscode/content failed unexpectedly without providing any details.(768)
  "name": "module-name"
}

To Reproduce

create egg.json
open in VS Code
Add $schema reference listed in docs

Expected behavior

Urls with escaped @ symbols should still resolve
Actual Behavior

Returns error message
Environment

Eggs version:
Deno version: 1.10.2
Operating system and version: WSL2, Ubuntu 20.04

Additional context

Hey @josh-hemphill!
Sorry for the late reply. for now, you can use github raw url to the schema. we'll probably make this the default in new releases. we can't fix this on nest because we don't support untagged links so the @ has to be there.
github raw url to schema:
# tagged
https://github.com/nestdotland/eggs/raw/0.3.7/src/schema.json

# latest
https://github.com/nestdotland/eggs/raw/master/src/schema.json

# dev
https://github.com/nestdotland/eggs/raw/dev/src/schema.json


Can't login outside localhost !

Hi there,
I'm considering using Libreboard inside my company since our compliance rules dont allow us to store data outside our own infrastructure.
So far I've set up Libreboard using Meteor on a personal test server, located OUTSIDE my company, and made it run on default port (3000).
I'm encountering a strange issue however. As I said, my instance of Libreboard is running on a personal server outside my company, beyond our firewalls.

If I use SSH to connect to my server and then configure my browser to use SOCKS proxy, then hitting http://localhost:3000, Libreboard works pretty well.
If I don't use SSH to connect to my server and use http://my-ip-address:3000 (which would be a normal behavior normally), I can't create any account or even login with an existing account.

Any ideas about what's wrong ? Why can I use it if I'm considered as localhost but not remotely ?
Thanks :+1:

Hi Shurike,
first of all: It would help if you could provide some logs of your browser and your server.
So I can just do let's say 3 suggestions:

Client firewall/proxy restrictions (maybe at your company)
Server firewall/reverese proxy restrictions (maybe get tunneled via SSH if you are connected)
Unset ROOT_URL environment variable

Well, I guess you don't want to use a backend which forces users to configure anything within their clients.
If you run the board on your remote server and have it configured according to your server/client requirements - the app just runs out of the box.
Further questions would be:

Did you change anything in the source code?
Which environment are you running on (os, meteor, node, etc.)?
Other way round: How do you run the server?

Finally, as you're considering to use the board for your company - that's great!
But have in mind that it is not yet a production ready system (upcoming 0.9)
Hope something helps :)

The installation wiki page now cover the points answered by @mko-x, so I’m now closing this issue. Documentation still need some work though.


[PE-D][Tester B] Add schedule command allows the user to input a end date that is before the start date

Maybe it would be good to parse the schedule command and produce an error if the end date is before the start date?

Labels: severity.Medium type.FunctionalityBug
original: bgopi23/ped#16

Duplicate of #121


Rename plugins to have a simpler less crowded interface

[ ] Find in files -> Find
[ ] Variable Explorer -> The same, that's our trademark ;-)
[ ] IPython Console -> The same too, we really want people to know we have an IPython console.
[ ] History Log -> History
[ ] File Explorer -> Files
[ ] Project Explorer -> Project (Projects when we support multiple projects)
[ ] Static Code Analysis -> Probably Code Analysis, but it's also called "Static Code Analysis" in Matlab (if I'm not mistaken).

Thanks for opening this @goanpeca. Question: should the rename also involve renaming the main classes related to the affected plugins? Or should we only rename the plugins' titles?
I mean, I think for beta2 we should rename just the titles because it's extremely simple and it would let people know how things are going to look in the final release. And we could leave refactoring the code for beta3.
What do you think?

Or should we only rename the plugins' titles?

Just titles :-) for now

Just titles :-) for now

Ok, then let's leave that task to @juanis2112. Juanita, please rename our plugins according to the list in @goanpeca's first comment above.
Note: I edited that list to reflect the actual changes we want.


Nft page uniques spec23

As of introducing Nodle's custom NFT pallet from Nodle's runtime spec 23, the queries for NFTs should go to substrateUniques where the previous storage of NFTs is relocated to.

@Douglasacost apart from some changes inside the .github folder which seems not related to this PR, it looks good. However this can break the proper query of NFTs for all the other chains. Can you find a neat way to limit this change to eden and paradis and only for the specs >= 23?

r this can break the proper query of NFTs for all the other chains. Can you find a neat way to limit this change to eden and paradis and only for the specs >= 23?

That's true. Will adapt it to only change the pallet on our chain

@aliXsed I've applied some changes on a single file and change the api by spec version

@Douglasacost your latest solution is clever in terms of reducing the scope of change and limiting it to a single place. However it seems costly in terms of performance. If I have understood it correctly every time the useApi hook is used, we'd extract the substrateUniques sub records of api.events and api.query and attach them back as uniques.  This doesn't seem quite cheap especially that we do it regardless of the use of useApi. Adding to that useApi is called every 5 sec at least for ApiStatsCtx. I suggest we go for a solution where only the calls for api.events.uniques and api.query.uniques are replaced carefully only if substrateUniques exists or just for our chain spec > 23.

@Douglasacost your latest solution is clever in terms of reducing the scope of change and limiting it to a single place. However it seems costly in terms of performance. If I have understood it correctly every time the useApi hook is used, we'd extract the substrateUniques sub records of api.events and api.query and attach them back as uniques. This doesn't seem quite cheap especially that we do it regardless of the use of useApi. Adding to that useApi is called every 5 sec at least for ApiStatsCtx. I suggest we go for a solution where only the calls for api.events.uniques and api.query.uniques are replaced carefully only if substrateUniques exists or just for our chain spec >= 23.

Yes, that change was executing the override method each time a component dependant of useApi was rendered. It wasn't too costly despite it seems to be as the components and the API itself works like a subscription (looping)
Thanks to your comment I found a better place to make the override on the unique methods! We can override whatever we face in the future 👍

we go in favor of: https://github.com/NodleCode/polkadot-js-apps/pull/34


Add ability to replicate pan behavior as seen in native keyboard view.

I'd like to implement a behavior found on the iPhone's system keyboard:  user touches a key, and other keys are highlighted as the finger drags across the keyboard.  When the finger lifts, the last highlighted key inserts the text.  If user long-taps a key, then its expandedInputView pan gesture is tracked instead of the entire keyboard.

+1 this would be awesome

Yes. How can we do that one? I am modifying this library and still not okay. Can anyone advice?


Some questions about the checkpoint

Hello,
I tried to use your checkpoint to finetune the RetroMAE_MSMARCO model, but the result is lower than the number in your paper(e.g. the MRR@10 is 0.393 in the paper, but I got 0.373). I used the hard_negs in the script, and used the same hyper-parameters. The only difference is that I use one V100 to finetune.

Thanks for your interest in our work!
We use 8*GPUs to fine-tune the model, which can increase the number of in-batch negatives since we share the negatives across GPUs. Using only one GPU may impair performance.
However, this result falls below my expectations. Do you use the same training and testing script in https://github.com/staoxiao/RetroMAE/tree/master/examples/retriever/msmarco?

Thank you for your explanation. I did use the same script. I got MRR@10 0.393 when I directly used RetroMAE_MSMARCO_finetune model for testing, but I can't get the same number when I use the checkpoint.
This is the script I use to finetune
python -m torch.distributed.launch --nproc_per_node 1
-m run
--output_dir /retro-mae/pre_trained_onms/retro_mae_official_fine
--model_name_or_path /retro-mae/pre_trained_onms/RetroMAE_MSMARCO
--do_train
--corpus_file /mydata/marco/data/BertTokenizer_data/corpus
--train_query_file /mydata/marco/data/BertTokenizer_data/train_query
--train_qrels /mydata/marco/data/BertTokenizer_data/train_qrels.txt
--neg_file /mydata/marco/data/BertTokenizer_data/hard_negs.txt
--query_max_len 32
--passage_max_len 140
--fp16
--per_device_eval_batch_size 16
--train_group_size 16
--sample_neg_from_topk 200
--learning_rate 2e-5
--num_train_epochs 4
--negatives_x_device
--dataloader_num_workers 6
--save_steps 120000

Hi, I run the same command with one GPU (set num_train_epochs=1 for efficiency), and get<EMAIL_ADDRESS>Larger num_train_epochs may further improve the result.


No pod connection, no internet

What happened:
I used calico CNI and switched to Azure CNI and got everything smoothly working. The next day, after VM restart, I could not connect to the network inside the pods anymore.
What you expected to happen:
Pods can connect inside and outside the private net.
How to reproduce it:
Set up Ubuntu (20.10 or 21.04) VM in private network (mine is <IP_ADDRESS>/16).
apt update
apt install kubectl kubeadm kubelet containerd
alias k=kubectl

Install CNI plugin as described here for Kubernetes: https://docs.microsoft.com/en-us/azure/virtual-network/deploy-container-networking#download-and-install-the-plug-in:
./scripts/install-cni-plugin.sh v1.4.20 v1.0.1 (v1.4.23 does not work either)

kubeadm init --upload-certs  --cri-socket "/run/containerd/containerd.sock"

Pod cannot connect to Azure DNS:
root@jakob-k8s:/home/adrian# k logs -n kube-system coredns-64897985d-rz8t7
[WARNING] plugin/kubernetes: starting server with unsynced Kubernetes API
.:53
[INFO] plugin/reload: Running configuration MD5 = db32ca3650231d74073ff4cf814959a7
CoreDNS-1.8.6
linux/amd64, go1.17.1, 13a9191
[ERROR] plugin/errors: 2<PHONE_NUMBER>305281932.1728699848247201413. HINFO: read udp <IP_ADDRESS>:48232-><IP_ADDRESS>:53: i/o timeout

Pod cannot ping (e.g. VM IP) inside vnet:
kubectl taint nodes --all [node-role.kubernetes.io/master-](http://node-role.kubernetes.io/master-)
k run b --image busybox --rm -it
ping <IP_ADDRESS>

Outbound connection is also not working inside the pod:
ping <IP_ADDRESS>

When I applied source NAT rule:
iptables -t nat -A POSTROUTING -m iprange ! --dst-range <IP_ADDRESS> -m addrtype ! --dst-type local ! -d <IP_ADDRESS>/16 -j MASQUERADE

Not even the VM can connect to the internet nor private network anymore.
I need to remove the rule to make it work again:
iptables -t nat -D POSTROUTING -m iprange ! --dst-range <IP_ADDRESS> -m addrtype ! --dst-type local ! -d <IP_ADDRESS>/16 -j MASQUERADE

I reproduced the issue on 2 different VMs in the same private net.
No firewall rules are applied inside the VM.
The log of the CNI plugin also looks suspicious. The CNI Plugin seems to stop soon after machine launch without any error:
root@kube-master-3:/home/adrian# tail -f /var/log/azure-vnet.log
2022/04/19 08:25:27 [16631] Get number of endpoints for ifname eth0 network azure
2022/04/19 08:25:27 [16631] Execution mode :
2022/04/19 08:25:27 [16631] Trying to retrieve endpoint id 6a2330fc-eth0
2022/04/19 08:25:27 [16631] [cni-net] Failed to query endpoint: Endpoint not found
2022/04/19 08:25:27 [16631] Release ip by ContainerID (endpoint not found):6a2330fcee50fcceef9481c4ee5b91ec02d9d520fd831a637e1a6673f8665874
2022/04/19 08:25:27 [16631] [cni] Calling plugin azure-vnet-ipam DEL nwCfg:&{CNIVersion:0.3.0 Name:azure Type:azure-vnet Mode:transparent Master: AdapterName: Bridge: LogLevel: LogTarget: InfraVnetAddressSpace: IPV6Mode: ServiceCidrs: VnetCidrs: PodNamespaceForDualNetwork:[] IPsToRouteViaHost:[<IP_ADDRESS>] MultiTenancy:false EnableSnatOnHost:false EnableExactMatchForPodName:false DisableHairpinOnHostInterface:false DisableIPTableLock:false CNSUrl: ExecutionMode: Ipam:{Type:azure-vnet-ipam Environment: AddrSpace: Subnet:<IP_ADDRESS>/16 Address: QueryInterval:} DNS:{Nameservers:[] Domain: Search:[] Options:[]} RuntimeConfig:{PortMappings:[] DNS:{Servers:[] Searches:[] Options:[]}} WindowsSettings:{EnableLoopbackDSR:false} AdditionalArgs:[]}.
2022/04/19 08:25:27 [16631] [cni] Plugin azure-vnet-ipam returned err:<nil>.
2022/04/19 08:25:27 [16631] [cni-net] DEL command completed for pod pdnsutilsm with err:<nil>.
2022/04/19 08:25:27 [16631] [cni-net] Plugin stopped.
2022/04/19 08:25:27 [16631] Released process lock

Orchestrator and Version (e.g. Kubernetes, Docker):
Kubernetes v1.23
Operating System (Linux/Windows):
Ubuntu 20.10
Kernel
5.4.0-1068-azure
and
5.11.0-1027-azure
Anything else we need to know?:
Azure support advised me to open an issue here.
When using Calico CNI, coredns service works in Kubernetes and I can ping pods on the same node and other VM IPs inside the private net, but pods on different nodes cannot communicate and internet inside pods also does not work.

/cc @mattstam @phealy is this a supported scenario?

Reference

I reproduced the issue on another machine without Calico and I am not using AKS.

I reproduced the issue on another machine without Calico and I am not using AKS.

@elchead  can you check if ipforwarding enabled on your VM? sysctl net.ipv4.ip_forward
If enabled, can you paste output of  sudo iptables -t filter -L FORWARD
Also can you take packet capture of VM eth0. Run this cmd in VM sudo tcpdump -ni eth0 -e icmp and ping from pod. Destination should be vnet ip. It can be different VM ip or pod ip in different VM.

@tamilmani1989 ipforwarding is enabled. It's also enabled in the Azure network interface.
root@kube-master-3:/home/adrian# root@kube-master-3:/home/adrian# sudo iptables -t filter -L FORWARD
Chain FORWARD (policy ACCEPT)
target     prot opt source               destination
KUBE-FORWARD  all  --  anywhere             anywhere             /* kubernetes forwarding rules */
KUBE-SERVICES  all  --  anywhere             anywhere             ctstate NEW /* kubernetes service portals */
DOCKER-USER  all  --  anywhere             anywhere
DOCKER-ISOLATION-STAGE-1  all  --  anywhere             anywhere
ACCEPT     all  --  anywhere             anywhere             ctstate RELATED,ESTABLISHED
DOCKER     all  --  anywhere             anywhere
ACCEPT     all  --  anywhere             anywhere
ACCEPT     all  --  anywhere             anywhere

When pinging the vnet ip (of the VM running the pod) inside the pod, I don't receive any traffic

I gave up on Azure plugin, but got it to work with calico. Thanks for your help!


File Explorer Namespace Restrictions

Hi,
I try to configure the "NoRestriction" option in "File Explorer Namespace Restrictions"
But PPKG installation fail and I receive this error in event viewer:
"
FileExplorerNameSpaceRestrictions" cannot be empty according to the DTD/Schema.
ErrorCode(0xC00CE011)
"
I try to directly copy the xml sample "[Preview] Folder Access sample xml"
https://docs.microsoft.com/en-us/windows/configuration/kiosk-xml#preview-folder-access-sample-xml
But I received the same error message.
I use Windows build 1903 and I try with 1909 but same error...
Below, XML line

When i try with "Download" it works...
Thanks
Degoa

@degoa - Thank you for submitting feedback.
I think the best way forward is if you can contact support and open a service ticket for Windows 10 products  so this can get resolved ASAP. Based on outcome, let me know if it is something that can be called out in the docs.
If you don't mind please keep us posted here on the resolution and I really want to get this information you are discovering back into the docs.


Missing Whiteboard journal page type

Describe the bug
When I create a new Journal and Journal page, there is no Whiteboard to choose among the page types

To Reproduce
Create New Journal
Open New Journal
Add New Page
Click on New Page Type menu
Expected behavior
Finding Whiteboard in the list, as shown in your screenshot
Screenshots
If applicable, add screenshots to help explain your problem.
Environment (please complete the following information):

Foundry VTT: 10.291
Browser Opera GX
Module Version 11.0.6
DnD 2.2.1

Additional context:
The only two Journal related Modules I have are GMScreen, Thumbnail for Journal Entries and Journal Whiteboard

I discovered that Foundry V10 has not sub-type support like Foundry V11. I am currently searching for a backport to Foundry V10.

Got it, thanks for letting me know. Maybe Monk’s journal module has a workaround for this ? The module offers multiple subtypes of journals and works with v10.

I have noticed that the Monk's Journal Module requires several modifications to function properly on Foundry V10. Unfortunately, I have decided not to backport this module to Foundry V10.


Fix function comments based on best practices from Effective Go

Every exported function in a program should have a doc comment. The first sentence should be a summary that starts with the name being declared.
From effective go.
I generated this with CodeLingo and I'm keen to get some feedback, but this is automated so feel free to close it and just say opt out to opt out of future CodeLingo outreach PRs.

It looks like there's one fix up that updates the tense and read wells.
The rest, like this basically just add colons to conform to standards. Arguably they make readability worse. Let me know if you think they're valuable.
Cheers :smile:


TIC-TAC-TOE

Project title
tic tac toe game
Web stack
HTML , CSS , JAVASCRIPT
Screenshot(mandatory)

Additional Context(mandatory)
2 players can play this game

@GAGANRAGHAV under which open source programme you are contributing?

Codepeak 2023


Update test icons for new 2.3 test files

Overview
Updates the Unity icons for our test files to match the current standard. Nice spread of new coverage tests this release!

/azp run


⚠ Bofrost.de has degraded performance

In 3ea44b0, Bofrost.de (https://www.bofrost.de) experienced degraded performance:

HTTP code: 200
Response time: 1003 ms

Resolved: Bofrost.de performance has improved in 976c96a after 310 days, 22 hours, 30 minutes.


issue with Spellchek.php. HTTP request fail while fetching translate.googlecom/*

Warning: file_get_contents(http://translate.google.com/translate_a/t?client=t&sl=en&tl=en&hl=en&sc=2&ie=UTF-8&oe=UTF-8&uptl=en&oc=1&otf=1&ssel=3&tsel=0&q=c): failed to open stream: HTTP request failed! HTTP/1.0 403 Forbidden in C:\xampp\htdocs\webcrawl\inc\spellcheck.php on line 15

The problem is Google blocked us from doing spell check. Hence the error.

in the last commit you have made some changes that to remove the dependency of google translate spellcheck by introducing inc/class.spellcheck.php in /search.php on line 14. But there is no such file found the directory /inc/.

Fixed in commit https://github.com/subins2000/search/commit/ac90d8af9b541b467af20c17f2c7560f19610ee4


Project structure

We got this suggestion from Szalay Richárd:

Just a general suggestion: it is not a good idea to have multiple top-level packages if you are developing "one" application. So having an "ELME" and a "MODEL" package is a bit redundant, conventionally there is a top-level package, and then subpackages, like "elme.model", "elme.graphics", etc.

Should we restructure our project? If so, how? Should we do it now, or should we wait for merging more of the active branches?

We can just make a small fix in master branch now. And I will make same change in my camera control branch

Yes, probably would be worth listening to the suggestion. As it is not all that big of a change, I share Andrey's idea. If things go south, we should be able to handle the conflict as (as I already said) would not be too big of a change. On the other hand, if you plan to merge with main in the following days, I think we can wait until that.

I have just created a branch for this issue. It could possible be merged today, after that the change could be made.

Since the branch that you created, has been merged, I believe now it is possible to change the location of model directory @vismate

Yes, we should do it now.


2.0.16 broke JSON responses

What steps will reproduce the problem?
Request content by json. For example, I use yii\jui\AutoComplete for a autocomplete box, but when I type, nothing happens, because I get a incorrent JSON response. I get the JSON object plus a null string at the end of it.
What is the expected result?
I should get the autocomplete popup appear.
What do you get instead?
Nothing. The popup does not appear, since the response cannot be parsed correctly.
Additional info

Q
A

Yii version
2.0.16

PHP version
7.2.14

Operating system
Linux

This is related to issue #17094 with response http status 204. This issue is for http response 200 (it has content). Downgrading to <IP_ADDRESS> fixes the issue.

How do you generate this JSON response?

With something like this:
$result = Yii::$app->db->createCommand($query)
    ->bindValue(':afm', $term . '%', \PDO::PARAM_STR)
    ->queryAll();
Yii::$app->response->format = Response::FORMAT_JSON;
echo Json::encode($result);
Yii::$app->end();

in the controller action.

You should not echo response in controller, you should return it like:
$result = Yii::$app->db->createCommand($query)
    ->bindValue(':afm', $term . '%', \PDO::PARAM_STR)
    ->queryAll();
return $this->asJson($result);

Ok, I didn't know that one. Yes, it works fine. Thanks! Sorry for the noise.

If you still want to end the app,you can change the code
from
...
Yii::$app->response->format = Response::FORMAT_JSON;
echo Json::encode($result);
Yii::$app->end();

to
...
$this->asJson($result);
Yii::$app->end();

@j0hnh that will not send the Response.


KeyError: 'DISPLAY' in stop

Hi,
Thanks for the awesome wrapper!
I'm getting the below error running dryscrape with xvfbwrapper.
Ubuntu 16.04.2 LTS (Ubuntu Mate)
Python 2.7.12
xvfbwrapper-0.2.9
Error in sys.exitfunc:
Traceback (most recent call last):
File "/usr/lib/python2.7/atexit.py", line 24, in _run_exitfuncs
func(*targs, **kargs)
File "/usr/local/lib/python2.7/dist-packages/xvfbwrapper.py", line 84, in stop
del os.environ['DISPLAY']
File "/usr/lib/python2.7/os.py", line 498, in delitem
del self.data[key]
KeyError: 'DISPLAY'
I'm not sure this is the best solution, but I was able to fix this by adding "if 'DISPLAY' in os.environ:" to check that 'DISPLAY' actually exists.
            if self.orig_display is None:
                if 'DISPLAY' in os.environ:
                   del os.environ['DISPLAY']

I have the same problem, to temporary fix I added the DISPLAY key on os.environ dict before start:
    def start(self):
        # fixing an exception raised in xvfb
        os.environ['DISPLAY'] = ':0'

        self._xvfb.start()

Python 3.7.3
xvfbwrapper-0.2.9


How to access?

Hi There, thanks so much for this project by the way.
I'm struggling to find how I'm supposed to access my ES instance, from Python, let's say. I'd need to access it from another openshift app. Or do I?

Hi,
I have the same "problem" as yamatt, but in my case I wanna connect with tomcat (my webapp is in j2ee). I cannot figure it out how to connect.
Thanks

Hi @yamatt, sorry for the delay in responding. No, you don't necessarily need to access ES from another openshift app. You should be able to access it anywhere. From python something like this should work:
from datetime import datetime
from elastic search import Elasticsearch

es = Elasticsearch([{'host':"your-es-instance-name.rhcloud.com", 'port':80}])
# index a document
es.index(index="my-index", doc_type="test-type", id=42, body={"any": "data", "timestamp": datetime.now()})
# get the document
es.get(index="my-index", doc_type="test-type", id=42)
# returns:
# {u'_type': u'test-type', u'_source': {u'timestamp': u'2015-05-30T19:04:43.418911', u'any': u'data'}, u'_index': u'my-index', u'_version': 1, u'found': True, u'_id': u'42'}

Hi @arrakisdunes, I've not tried to connect from tomcat but I think you'll need to do something like they show here: https://www.elastic.co/guide/en/elasticsearch/client/java-api/current/client.html
I guess you can specify the hostname/port in some tomcat config file, but, again I haven't tried it so I'm afraid I don't know what it is.

I did check the link (thanks for that) and also I search about this subject. What I found is the port 80 is REST and is hard to use this. Better is to use the transport client (usualy 9300). How can I enable this port?

@arrakisdunes That is exactly what my problem was (I did work it out in the end) that the ES port in OpenShift is 80 and not the default.

Do you mean to say that one should not use the transport client? Or should the port 80 be used as the transport client port?

The REST port here is port 80. I've not used the transport client.


Don't notify subscribers when after action state hasn't changed, add test

name: Optimization of subscriptions
about: Performance
PR Type
Does this PR add a new feature, or fix a bug?
No.
Why should this PR be included?
Problem
Up to 50% of all dispatched actions in the apps that use redux-saga or redux-thunk could be ones that don't change the redux state but start some async logic.
Current implementation notifies subscribers for each such action even if state hasn't changed, causing all active selectors to be reevaluated without any good reason.
In huge apps there can be hundreds of subscribed selectors and dozens of actions per second, so this leads to additional CPU usage for no good reason.
Solution
Don't notify subscribers when action did not change the state.
Can it break existing code?
As far as I can see, the only reason to subscribe to the store is to get the new state using store.getState(). There is no sense in subscribing to get the called action because there is no way to get the latest action called from subscription. So it should not break any existing code.
But I would vote for raising major version here.
Checklist

[x] Have you added an explanation of what your changes do and why you'd like us to include them?
[x] Is there an existing issue for this PR?
No, as far as I know.
[x] Have the files been linted and formatted?
[x] Have the docs been updated to match the changes in the PR?
[x] Have the tests been updated to match the changes in the PR?
[x] Have you run the tests locally to confirm they pass?

Hi! I recognize your username from some recent discussions in the React-Redux repo.
This is actually an intentional design decision, and not something we will be changing:

https://redux.js.org/faq/design-decisions#why-doesnt-redux-pass-the-state-and-action-to-subscribers

However, you can implement this same logic as a Redux store enhancer, and add it to your own store in your app.

As for the layer where to fix this issue, it makes sense to me that it is better to remove the root cause of a problem which is here. I don't understand what lies beyond that design decision to notify subscribers when state hasn't changed considering that we don't pass state and action to the subscriber.

Hmm. It used to bail out. Maybe we lost something in the transition to useSyncExternalStore in v8.
I'm not questioning whether the change here makes a difference. I'm saying it's a piece of core behavior we're not going to alter, and that it can be overridden at the enhancer level.

Filed https://github.com/reduxjs/react-redux/issues/2089 and I'll try to look at it soon.


Remove mapping types

As part of v2.0 release, mapping types are getting removed from OpenSearch engine. Below are the changes in the opensearch-engine.

Removal of type from rest end-points
Removal of include_type_name parameter from API requests.

As part of this issue, please verify if type removal change on OpenSearch engine impacts this repository. If yes, then please remove the type references/usage from this plugin.
Top level changes are captured in gist below:
https://gist.github.com/dreamer-89/d76eaf639171e8ab32fa7f8b9d6c93d3
For more detailed changes, please check meta issue below:
Related: OpenSearch-engine meta issue

https://github.com/opensearch-project/OpenSearch/issues/2480

Tracking as a breaking change - https://github.com/opensearch-project/OpenSearch/issues/2480


Issue with delimiter when dealing with a connection-throttled SFTP server and big files

Hi,
We seem to be encountering an extremely specific issue with delimited files, originating from an SFTP site. The delimiter is the '~' symbol. However, some rows in the source files contain a semicolon. When processing the files, sometimes they process successfully, while other times the following error is thrown:
{
"errorCode": "2200",
"message": "ErrorCode=UserErrorSourceDataContainsMoreColumnsThanDefined,'Type=Microsoft.DataTransfer.Common.Shared.HybridDeliveryException,Message=Error found when processing 'Csv/Tsv Format Text' source '<file>.csv' with row number 195426: found more columns than expected column count: 51.,Source=Microsoft.DataTransfer.Common,'",
"failureType": "UserError",
"target": "ChannelData_Sales"
}
These files can easily be a few hundred MB. I tried deleting all but a handful of rows in order to see if that would trigger the error, but the file will process fine at such times. What does seem to make a difference is the number set in "Max concurrent connections".
If I leave the setting blank, my pipeline will process correctly (showing it uses 10 peak connections). Setting it to 5, consistently will ensure the error is thrown.
Setting the value to 10 works for a single file, but we're interested in processing an entire folder at a time. Emptying the filename and setting the wildcard while retaining the setting shows Data Factory starts processing 2 files, using 9 peak connections. This again causes the error to occur.
Setting the value to 20 seems to ensure we hit the throttle limit, and then see errors related to authentication issues (See my comments at https://social.msdn.microsoft.com/Forums/azure/en-US/968b77d1-c005-4140-b390-badc740ed1ff/sftp-to-blob-storage-pipeline-issues at the bottom).
We have requested our supplier to increase the throttle limit for connections, but it would be nice if someone from Microsoft could help us troubleshoot this issue, and see whether the semi-colon being regarded as a column delimiter, despite using a custom delimiter might be some sort of a bug in the Delimited Source component, or due to some memory limitations of the component.

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

ID: 2309a152-b9bc-7876-e010-b3bde6945d42
Version Independent ID: 751c7679-b01c-301d-8211-30f43ff50c54
Content: Delimited text format in Azure Data Factory
Content Source: articles/data-factory/format-delimited-text.md
Service: data-factory
GitHub Login: @linda33wj
Microsoft Alias: jingwang

@pes-kapacity Thank you for reaching out and bringing this to our notice. At this time we are reviewing the feedback and will provide an update as appropriate

Thank you. I'd be happy to supply one of the files if that helps.

@pes-kapacity Hi Peter, for deeper analysis on the issue, we have enabled your subscription for a one time free technical support. Please follow the instructions sent in the email to open a support ticket and do share the support ticket number for tracking purpose.
Since we are following up this offline through email and as well as in below forums threads,
we will now proceed to close this thread. If there are further questions regarding this matter, please tag me in your reply and reopen the issue.. We will gladly continue the discussion.
Here are MSDN thread links:
Old : https://social.msdn.microsoft.com/Forums/en-US/968b77d1-c005-4140-b390-badc740ed1ff/sftp-to-blob-storage-pipeline-issues?forum=AzureDataFactory
New: https://social.msdn.microsoft.com/Forums/en-US/be063cd5-b02a-4f9f-91a7-41bbeacef341/adf-v2-issue-with-delimiter-when-dealing-with-a-connectionthrottled-sftp-server-and-big-files?forum=AzureDataFactory


add getformat

Because of this issue https://github.com/rust-lang/rust-clippy/issues/8527,
I can't check this on my local.

Because of this issue https://github.com/rust-lang/rust-clippy/issues/8527, I can't check this on my local.


cal.com service

Add cal.com self hosting options
https://docs.cal.com/self-hosting/docker

Thanks for the issue. Duplicated of #242

thanks!


Release checkpoints on Hugging Fac

Hello @LSanghyeok 🤗
I'm Niels and work as part of the open-source team at Hugging Face. I discovered your work through the Hugging Face daily papers as it got featured: https://huggingface.co/papers/2411.15241. The paper page lets people discuss about your paper and lets them find artifacts about it (your model for instance) you can also claim the paper as yours which will show up on your public profile at HF.
Would you like to host the model you've pre-trained on https://huggingface.co/models? I see you're using Google Drive for it. Hosting on Hugging Face will give you more visibility/enable better discoverability. We can add tags in the model cards so that people find the models easier, link it to the paper page, etc.
If you're down, leaving a guide here. If it's a PyTorch model, you can use PyTorchModelHubMixin class which adds from_pretrained and push_to_hub to the model which lets you to upload the model and people to download and use models right away.
If you do not want this and directly want to upload model through UI or however you want, people can also use hf_hub_download.
After uploaded, we can also link the models to the paper page (read here) so people can discover your model.
You can also build a demo to your model on Spaces we can provide you an A100 grant.
Let me know if you need any help.
Kind regards,
Niels

Hello Niels!
Thank you for reaching out to feature our work!
We appreciate your suggestion regarding hosting our pre-trained model on Hugging Face.
We will be uploading it to Hugging Face, soon.
Thank you once again for your kind support.
Sincerely,
Sanghyeok

Hi @LSanghyeok, I noticed all model checkpoints link to Google Drive.
Would you be able to update the links to the Hugging Face hub? Happy to assist with the integration.
Kind regards,
Niels

It could be done similar to the MambaVision integration: https://huggingface.co/collections/nvidia/mambavision-66943871a6b36c9e78b327d3


Use well-formed XML syntax

Fixes https://github.com/readthedocs/sphinx_rtd_theme/issues/985
ref https://github.com/sphinx-doc/sphinx/commit/267eeabc008ce7ab1a0034cc9c7ce2eafc52514b

Thanks for the fix!


Update README.md

a small typo.

Small typo ended on a wrong push. As a tip my friend, never ever create PR from your fork's master branch. Always create a new one.


Fix mono reload_file tests

as discussed in https://github.com/NLog/NLog/pull/968, needs Mono 4.2+

not sure if we should merge this. Mono 4.2 isn't RTM?

Current coverage is 71.60%

Merging #1006 into master will not affect coverage as of f105038

@@            master   #1006   diff @@
======================================
  Files          262     262
  Stmts        14793   14793
  Branches      1611    1611
  Methods          0       0
======================================
  Hit          10592   10592
  Partial        415     415
  Missed        3786    3786

Review entire Coverage Diff as of f105038

Uncovered Suggestions

+0.09% via ...c/NLog/LogFactory.cs#180...191
+0.08% via ...anceCounterTarget.cs#308...318
+0.08% via ...anceCounterTarget.cs#294...304
See 7 more...

Powered by Codecov. Updated on successful CI builds.

Release date: 25 Aug 2015
Mono 4.2.0 is available in the alpha channel.

http://www.mono-project.com/docs/about-mono/releases/4.2.0/

This is already fixed, tests has been moved to LogFactoryTests.cs


Add new module for shared Scala lib code

As we add more Scala components, such as #50 , where we need to do similar actions, such as reporting CloudWatch metrics or interacting with DynamoDB, it makes sense to take this code out of a specific sub-repo and put it in a shared/common module that the others can import from so there's less code duplication.

This hinges on #46 getting merged first, for sure


fix: remove default timeout

Removes the default http timeout of 10 seconds.

Removed the default 10s HTTP timeout, which was causing issues where the backend continued processing requests after the UI had already timed out.
Since we do not enforce timeouts on the backend, having a frontend timeout is not practical.
Now relying on the infra default timeouts (cloudflare) for better synchronization between the UI and backend responses.

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


renderer: Add blacklists for default collapsed project-leveled files

eg:
package-lock.json
.gitignore

OK, I'll work on this soon.

[x] cli: Add collapse=true to generate files in tuture.yml
[ ] renderer: Collapse those generated files by default


Only render the dungeon where the player is located

Map regions can consist of multiple dungeons (also locations like: Blast Furnace, Random Events etc.) that are being rendered when the GPU plugin is active. It would be nice if only the dungeon that the player is located in, is being rendered. Other dungeons could be completely hidden or fog could appear around the borders of the dungeon where the player is standing in.
Before:

After:

Duplicate of #6478, also this is possible for some locations without using the GPU plugin.


Controller namespacing

At the moment controller namespaces are prefixed with:
%{AppName}::Controllers::%{ControllerName}
This is a good choice for container applications but perhaps "too much" for simpler apps.
One idea is to do a check in Application.inherited and see if the application lives inside a namespace, and skip it if it doesn't, and then make the Controllers part configurable and disabled by default.

I removed the "forced" Controllers namespace for controllers. This is going to be a configurable "feature" in the future.


Relax version dependecies

In library type projects, version numbers should only be restricted to a fixed major version number if there is a good reason to do so. This commit relaxes the constraints and turns fixed major version numbers (^1.3.4) into lower bounds (>=1.3.4)

I ran poetry lock --no-update after and committed the result.


Translate the site into German

I would like to have this site translated into German. Who wants to help?

I'm going to start now :)

So I kinda sorta started on this, but I don't know German! Who wants to help?

@kentcdodds how about Chinise version? I have filed an issue here #202 :smile:

Still want to get the site translated into German, but we no longer need to have this issue open.


Update emea-ltu-saf-t.md

:x: Organization membership check failed
Members of the MicrosoftDocs organization should use the private repo. Please move your changes to https://github.com/MicrosoftDocs/Dynamics-365-Operations, and close this pull request.


Add threading support to the FIPS provider.

The FIPS provider does not include threading code directly. The core passes thread related methods to the providers so that the FIPS provider has access to threading.
This allows complex algorithms such as ARGON or HSS to use threads once they are implemented in the FIPS provider.
An existing PR for HSS will use and test this code.

Checklist

[x] documentation is added or updated
[ ] tests are added or updated

Argon isn't available in FIPS yet.
HSS will be in due course.

Rather than have people review roughly 70 files for the HSS work, I think it makes sense to break up the work into digestable sized PR's that have a chance of being more thoroughly reviewed. This is one such PR.

In ref to Argon, See https://csrc.nist.gov/news/2023/proposal-to-revise-nist-sp-800-132-pbkdf

Dont really see why it needs a feature branch. The work is mainly done.

I think a feature branch makes sense - the review process is going to be "fun" and then we can keep it ready to hit the tree in a single chunk once all the various PRs get reviewed.

Is the 'feature' branch  process documented somewhere? Am I supposed to create it?

For things such as https://github.com/openssl/openssl/pull/25499 I dont think they need to be on a feature branch.. (i.e.- a general fix).

concur with feature branch suggestion.  I think we need to start moving more work to feature branches to make inclusion decisions later in the development cycle more manageable

This PR is in a state where it requires action by @openssl/committers but the last update was 30 days ago

This PR is in a state where it requires action by @openssl/committers but the last update was 61 days ago

This PR is in a state where it requires action by @openssl/committers but the last update was 92 days ago


Update seems to be broken

Howdy all!  Been a while.  I haven't been using Live TV for a while, but I was in the process of updating things in my environment, and fHDHR was one of those things.
I did a git pull in the main fHDHR directory (that fast-forwarded, all good, no warnings), then I did the same in each of the plugin directories, (), again, all good no issues.
Kicked off fHDHR, started without an issue, clicked around, and everything seemed to be working.
When I clicked the "Ceton" button, the website says, "Internal Server Error", and the log dumps this out:
Traceback (most recent call last):
  File "/opt/fHDHR/lib/python3.8/site-packages/gevent/pywsgi.py", line 970, in handle_one_response
    self.run_application()
  File "/opt/fHDHR/lib/python3.8/site-packages/gevent/pywsgi.py", line 917, in run_application
    self.result = self.application(self.environ, self.start_response)
  File "/opt/fHDHR/lib/python3.8/site-packages/flask/app.py", line 2450, in wsgi_app
    response = self.handle_exception(e)
  File "/opt/fHDHR/lib/python3.8/site-packages/flask/app.py", line 1867, in handle_exception
    reraise(exc_type, exc_value, tb)
  File "/opt/fHDHR/lib/python3.8/site-packages/flask/_compat.py", line 39, in reraise
    raise value
  File "/opt/fHDHR/lib/python3.8/site-packages/flask/app.py", line 2447, in wsgi_app
    response = self.full_dispatch_request()
  File "/opt/fHDHR/lib/python3.8/site-packages/flask/app.py", line 1952, in full_dispatch_request
    rv = self.handle_user_exception(e)
  File "/opt/fHDHR/lib/python3.8/site-packages/flask/app.py", line 1821, in handle_user_exception
    reraise(exc_type, exc_value, tb)
  File "/opt/fHDHR/lib/python3.8/site-packages/flask/_compat.py", line 39, in reraise
    raise value
  File "/opt/fHDHR/lib/python3.8/site-packages/flask/app.py", line 1950, in full_dispatch_request
    rv = self.dispatch_request()
  File "/opt/fHDHR/lib/python3.8/site-packages/flask/app.py", line 1936, in dispatch_request
    return self.view_functions[rule.endpoint](**req.view_args)
  File "/opt/fHDHR/fHDHR/plugins/fHDHR_plugin_origin_ceton/web/ceton_html.py", line 31, in __call__
    return self.get(*args)
  File "/opt/fHDHR/fHDHR/plugins/fHDHR_plugin_origin_ceton/web/ceton_html.py", line 65, in get
    origin_status_dict["Tuner"+str(i)]['Device'] = int(self.plugin_utils.origin_obj.tunerstatus[str(i)]['ceton_device'])
KeyError: '4'

Any help?

UPDATE: New config parameter required.  There is now a device_tuners and a tuners parameter.  I've set them both to 6, and now the page loads properly.
Plex can no longer stream, however, I now get this error in the log:
[2022-11-22 11:29:10,276] INFO - [ceton] Selected Ceton tuner#: 1
[2022-11-22 11:29:10,278] INFO - [ceton] Ceton tuner 1 to be started
[2022-11-22 11:29:10,467] INFO - [ceton] Preparing Ceton tuner 1 on port: 49991
[2022-11-22 11:29:11,396] INFO - [ceton] Initiate streaming channel 1412 from Ceton tuner#: 1
[2022-11-22 11:29:11,399] INFO - Attempting to Select an available tuner for this stream.
[2022-11-22 11:29:11,400] INFO - Tuner #1 Acquired.
[2022-11-22 11:29:11,402] INFO - ceton Tuner #1 to be used for stream.
[2022-11-22 11:29:11,404] INFO - Preparing Stream...
[2022-11-22 11:29:11,405] ERROR - Unable to Stream: TunerError: Tuner Setup Failed (lazily handled): Tuner Setup Failed (lazily handled):KeyError 'buffsize'- /opt/fHDHR/fHDHR/fHDHR/streammanager/stream/__init__.py line 81
[2022-11-22 11:29:11,415] INFO - <IP_ADDRESS> - - [2022-11-22 11:29:11] "GET /api/tuners?method=stream&channel=855&origin=ceton&stream_method=ffmpeg&accessed=http%3A//<IP_ADDRESS>%3A5004/hdhr/ceton/auto/v855 HTTP/1.1" 503 2648 1.696196


[ios] Support enabling new architecture from use_react_native!

Summary
currently we can only enable new architecture through environment variables: RCT_NEW_ARCH_ENABLED=1 pod install. that's not ideal for expo managed apps setup. this pr introduces two new options, new_arch_enabled and codegen_discovery_enabled in the use_react_native!() method. so we can also enable new architecture without environment variables.
Changelog
[iOS] [Added] - Support enabling new architecture from the use_react_native! options
Test Plan
that is complicated for different option combinations. even for original code, what if someone set fabric_enabled=false and run with RCT_NEW_ARCH_ENABLED=1 pod install. i have added a unit test in scripts/spec/react_native_pods_spec.rb
that is the steps to run the unit test:
$ gem install rspec
$ cd scripts
$ rspec -f d

so far the test cases are:
  with get_default_flags
    should return false values for old architecture
    should return true values when RCT_NEW_ARCH_ENABLED=1
    should return hermes_enabled=true when specified explicitly
  new_arch_enabled tests
    should return true values when new_arch_enabled=true
    should support non-hermes in new architecture mode
    should show a warning when new_arch_enabled=true and fabric_enabled=false
    should show a warning when new_arch_enabled=false and fabric_enabled=true
    should show a warning when new_arch_enabled=true and codegen_discovery_enabled=false
    should show a warning when new_arch_enabled=false and codegen_discovery_enabled=true
    should support classic codegen

Quick note, currently RCT_NEW_ARCH_ENABLED=1 pod install is NOT the only way to enable this. You can also include ENV['RCT_NEW_ARCH_ENABLED'] = '1' in your podfile.

cool! how can i miss this possibility. do you think this pr still valuable? i can close the pr if this pr doesn't make sense.

Judging from
    should show a warning when new_arch_enabled=true and fabric_enabled=false
    should show a warning when new_arch_enabled=false and fabric_enabled=true
    should show a warning when new_arch_enabled=true and codegen_discovery_enabled=false
    should show a warning when new_arch_enabled=false and codegen_discovery_enabled=true

it doesn't really make sense to have all these parameters if they have to be set to the same value anyway. Wouldn't it be enough to just have two parameters (new_arch_enabled and hermes_enabled) for the next release?

i hope so as well, that's really complicated.  however, rn-tester right now still by default has codegen_discovery_enabled=false. @cortinico do you have any insight from the future of codegen maybe?

@Kudo I believe this PR can be closed as the code evolved so far right?


Translation: Chinese

Repository: https://github.com/OCselected/open-source-guide
Preview: https://ocselected.github.io/open-source-guide/
Locale: zh-CN
On the gh-pages branch of your fork:

[x] Update title, description, and locale in _config.yml
[x] Copy _data/local/en-US.yml to a new file with the name of your locale and translate the strings.
[x] Create a new directory in _articles/ for your locale and translate each guide:

[x] best-practices.md
[x] building-community.md
[x] code-of-conduct.md
[x] finding-users.md
[x] getting-paid.md
[x] how-to-contribute.md
[x] leadership-and-governance.md
[x] legal.md
[x] metrics.md
[x] starting-a-project.md

[ ] ??? TODO - other steps here
[x] Update CNAME to include your locale (e.g. de.opensource.guide)

Once a translation is completed:

[x] Open a Pull Request on the upstream repository to add your fork to translations in _config.yml

As stated in #299, please don't start any new translations until we've had the opportunity to make sure our proposed process works. Since this isn't something we're ready to support, I'm going to close this Issue for now. You may want to subscribe to #299 for updates in the meantime.

@lee-dohm, I actually said in https://github.com/github/open-source-guide/issues/288#issuecomment-281137887:

If anyone has free time this week and interest in doing another translation, I'd be up for kicking that off and opening an issue like #299 for your language. I don't want to do more than a couple while we're still ironing out how exactly it will work, but a couple might help to get more feedback.

My mistake for not communicating that more clearly.

@lijiangsheng1 Thanks for kicking this off!  Would you mind pushing your translations to the gh-pages branch so GitHub Pages builds the website for them?

Ah ok, I missed that. Thanks @bkeepers and my apologies :bow:

@bkeepers thanks for your encourage.  @lee-dohm It's ok .
OK, I will push branch which name is gh-pages-translation-chinese.

OK, I will push branch which name is gh-pages-translation-chinese.

@lijiangsheng1, let's get your fork rendering on https://ocselected.github.io/open-source-guide/. To do that, you'll need to merge the gh-pages-translation-chinese branch in to gh-pages and push to your fork. Once you've pushed to the gh-pages branch, the guide should show up on https://ocselected.github.io/open-source-guide/. This way people can see the progress as your translation progresses.

@bkeepers Thanks for your help. I just merged gh-pages-translation-chinese to gh-pages`, But https://ocselected.github.io/open-source-guide/ not work, Could you help me  review that ?

Hey @lijiangsheng1, sorry for the issue. It's been fixed in #348. If you merge the latest changes from the i18n branch in #295 into your gh-pages branch, it should work now.

@bkeepers It's work now,Thanks very much! I will update it.

@lijiangsheng1 I am almost done with translation. Chekout my repo https://github.com/liadbiz/opensource-contribute-guide-chinise
feel free to use it~:blush:

O, Very great job. I'll review and merge to main branch gh-pages.  Thanks a lot. @liadbiz you can access  https://ocselected.github.io/open-source-guide/how-to-contribute/ . I'll update every weekly.

@lijiangsheng1 I have post it to my blog, which is generated by hexo. I will finished  the last two articles this weekend. just mention me if I can help :)

@lijiangsheng1 i made a picture of a brain map  for chapter one ,if u need i post it out in chinese.

@joqk12345 You are great! Would you mind commit PR  to my fork, I need more time for merge main branch. I will sync translation later.

@lijiangsheng1 I have just sent a PR, but may be related to the changing of main repo, please come to see 😄

@liadbiz
Really thank you for providing us the translation of all articles of Open Guide!
@lijiangsheng1
Really thank you for maintain the project of open-source-guide(Chinese Translation)!
I see your discussion here, so I try to add the translation of @liadbiz to the repo of @lijiangsheng1 , and send a Pull Request,
Here is the PR link -  add translation of all remaining articles by @liadbiz OCselected/open-source-guide#7
Before the PR be merged,
You can also preview this Open Source Guide(Chinese Version) here first - http://blog.zoejane.net/open-source-guide/

@zoejane  Thanks! Really good work, I translated articles several months ago, and I remember there exists some mistakes that needto be fixed. It would be great you can proofread all the content :+1:

@liadbiz
When I first saw the Open Source Guide, I really wish there would be a Chinese Version. But I am not so good at translation, and don't have enough time and energy to do such a work.
So when I Googled and found your translation accidently , I am so happy!
I‘ve fixed some typos when I add you translation to the PR, but I haven't read the whole articles deeply, yet. I will read it more and try my best to proofread it, because this is such a great guide :)

@zoejane @liadbiz
Great jobs! Thanks very much.

@zoejane @liadbiz @Jasonnor
I Just merged Zoe 's PR, but I find a lots of translate issue. (Sorry, @liadbiz Please don't mind) .
Now, We are just finished step 1 , still need a lot of review jobs. Because not good, If you read it in Chinese, You can't understand clear.
Thanks all, Enjoy it.

@lijiangsheng1 I am sorry for translate and typo issues, and thanks for point it out! I have some time this weekend, we can make it better. Last but not least, half of that were translated by @cnbo.
@zoejane You are so kind, just enjoy the contribution. :smile:

@lijiangsheng1 I just saw you update the whole translation of building-community 3 hours ago, and thank you for your patience and professional attitude.
Together we can try and make it better.
I will try my best :)

..... I will sync and PR .

my fork repo name is open-source-guide, I used script/sync-translation can't work. So  I need more time to fix this or fork anther repo.


Update copyright year in PR template

Summary

Small fix: update to 2023

How to test

Checklist

[ ] I submit my changes into the develop branch
[ ] I have added description of my changes into CHANGELOG
[ ] I have updated the documentation accordingly
[ ] I have added tests to cover my changes
[ ] I have linked related issues

License

[ ] I submit my code changes under the same MIT License that covers the project.
Feel free to contact the maintainers if that's a concern.
[ ] I have updated the license header for each file (see an example below)

# Copyright (C) 2021 Intel Corporation
#
# SPDX-License-Identifier: MIT

Codecov Report
Base: 78.26% // Head: 78.24% // Decreases project coverage by -0.02% :warning:

Coverage data is based on head (b6ac4ab) compared to base (fd4d695).
Patch has no changes to coverable lines.

Additional details and impacted files
@@             Coverage Diff             @@
##           develop     #823      +/-   ##
===========================================
- Coverage    78.26%   78.24%   -0.02%
===========================================
  Files          184      184
  Lines        23563    23563
  Branches      4880     4880
===========================================
- Hits         18442    18438       -4
- Misses        4028     4032       +4
  Partials      1093     1093

Flag
Coverage Δ

macos-11_Python-3.8
?

ubuntu-20.04_Python-3.8
78.24% <ø> (ø)

windows-2019_Python-3.8
?

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

Impacted Files
Coverage Δ

datumaro/plugins/synthetic_data/utils.py
83.33% <0.00%> (-1.67%)
:arrow_down:

datumaro/util/test_utils.py
83.06% <0.00%> (-1.06%)
:arrow_down:

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

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.

Because this is a change not dependent on CI and just a small change for Github PR template regardless of our code, I'll merge it without review.


Configure linter

We need to configure eslint as a dev dependency and add a file to configure it.

Apparently create-react-app does this under the hood, and it conflicts with other installations of eslint if they aren't the same version.
Closing this for now as the spec doesn't require it.


Agent not recognized properly in Botium

I am using Botium to validate dialog flow  Intents and Training phrases. I have 3 agents in my dialog flow account. In which one agent only recognizing all the intents properly.others are not. Mostly the intents using fulfillment responses not recognized.It always calling default falback intent.
Can you help me to solve this issue ?

pls use stackoverflow for questions. i recommend to check the language code for your agents and botium.


Provide a way in CMAKE to add a static plugin library

Right now add_commonlibsse_plugin() creates a shared library, which makes unit testing cumbersome because we have to export all the to-be-tested classes/functions. We also have to worry about DLL boundary quirks when writing tests.
Being able to statically link tests with plugin code would make things a lot less painful.

This is supported by using the following style instead of add_commonlibsse_plugin:
add_library(MyPlugin STATIC Main.cpp)

target_commonlibsse_properties(MyPlugin)


CadastralParcel mapping

Hi!
I try to convert to geojson a Spanish Cadastre Inspire Service with jsonix but i can´t find a way to do it.
This is a GetFeature example of GML that i want to parse:
http://ovc.catastro.meh.es/INSPIRE/wfsCP.aspx?service=wfs&version=2&request=getfeature&STOREDQUERIE_ID=GetParcel&REFCAT=36055A01000100&srsname=EPSG:25829
I try to generate mappings with the xsd of the service. The generation works fine, but when i unmarshal with jsonix, it give me an error:
Element [{http://www.opengis.net/gml/3.2}gml:FeatureCollection] could not be unmarshalled as is not known in this context and the property does not allow DOM content.
I check the mapping file and the gml:FeatureCollection is not present. Can you help me please?
Thanks!

Hi,
I have solved the problem by another way.
Thanks.


Update Contributing.rst

Change the support email address and explain how to switch the doc for different PyAEDT versions.

@Maxjrey I talked to Roberto about making the changes that I've made in this PR before Zurich--and he agreed. I searched the PyAEDT directory and found that the<EMAIL_ADDRESS>appears in the project's pyproject. toml file (line 13) and in the project's README (line 128). Should these two email addresses also be changed to<EMAIL_ADDRESS>

@Maxjrey I talked to Roberto about making the changes that I've made in this PR before Zurich--and he agreed. I searched the PyAEDT directory and found that the<EMAIL_ADDRESS>appears in the project's pyproject. toml file (line 13) and in the project's README (line 128). Should these two email addresses also be changed to<EMAIL_ADDRESS>
@PipKat Yes that is correct. I was supposed to do it.


New OptimalPacker added

The new OptimalPacker class is basically a dummy that is used to select the "optimal" packing method.
I originally tried to implement the functionality inside the packer but found out that the sheet handling logic was actually outside of the packer and a new instance was generated for each sheet. So in the end I had to modify the PackProcessor and use the OptimalPacker as a dummy for selecting this packing method.
During profiling with our real-world test data I noticed that the optimal packer and method combination varies wildly based on the source images. Also the performance impact of this brute-force approach is negligible since the image processing takes up a large portion of the total cost.
Disclaimer: Our data set has a smaller number of large images, I would imagine for people with massive amounts of small images the impact is greater - which is why I've not made this the default packer method.
This pull request implements #5
ps. There's also some bonus TextureRenderer whitespace cleanup that I did while splitting render into getSize and render so that I could re-use the size calculations in the search for the optimal packing.

Hi @qtiki, thanks for this! I have clone your project and am trying to test your changes but you have some es6 features which are incompatible with our node version (6.9.4), and example of which is the spread operator in PackProcessor.js. Simply updating our node is not an option since this is fixed on our Team City build agents, I assume this will be the case for others, please could you look into making updates so this is compatible with es5?

Seems like its only the spread operators that are an issue, tried Object.assign with success

@kane-mason was just about to look into this but it seems you got it covered?

@qtiki sure as soon as i get a moment. Will need access to your fork? Also planning to report back on results

@qtiki so after reprocessing 28 sprite sheets, we had an average of 4.6% savings in area, the greatest in a singe sheet being 11.4%, which is significant so nicely done! There were a 3 cases where result got slightly bigger, by 2.2% in the biggest case. Is this something you need to look into or is it expected?

@qtiki so after reprocessing 28 sprite sheets, we had an average of 4.6% savings in area, the greatest in a singe sheet being 11.4%, which is significant so nicely done! There were a 3 cases where result got slightly bigger, by 2.2% in the biggest case. Is this something you need to look into or is it expected?

Check the amount of textures generated in the results. My guess is that the cases where the wasted area is bigger actually generate 1 texture less than before.
The primary goal is to generate as few textures as possible within the given width and height constraints. Only in the case of a tie (same number of textures) we use the efficiency as a tie-breaker. If you think about it we could easily reach 100% efficiency by not packing the textures at all, which would kinda go against the whole point of this package.

@kane-mason I added you as a contributor in our fork.
Just a note about the branches: the master branch contains a fix to the MaxRectsPacker rotation: https://github.com/odrick/free-tex-packer-core/pull/6
This pull request is in the optimal-packer branch. They are both based on the master commit from the free-tex-packer-core so that I could do the two simultaneous pull requests.
ps. I recommend merging that rotate fix too since it will greatly improve the efficiency of the MaxRectsPacker if rotate is enabled.

Check the amount of textures generated in the results. My guess is that the cases where the wasted area is bigger actually generate 1 texture less than before.
The primary goal is to generate as few textures as possible within the given width and height constraints. Only in the case of a tie (same number of textures) we use the efficiency as a tie-breaker. If you think about it we could easily reach 100% efficiency by not packing the textures at all, which would kinda go against the whole point of this package.

The amount of textures was the same. I think the difference comes from the update you did to the MaxRectsPacker, I see you said in some cases where the old implementation was better? Maybe this is all thats happening but in most cases (as is evident in my tests) we see improvement

@kane-mason I added you as a contributor in our fork.

Thanks, will get on to this asap

Just a note about the branches: the master branch contains a fix to the MaxRectsPacker rotation: #6
This pull request is in the optimal-packer branch. They are both based on the master commit from the free-tex-packer-core so that I could do the two simultaneous pull requests.
ps. I recommend merging that rotate fix too since it will greatly improve the efficiency of the MaxRectsPacker if rotate is enabled.

Great, yes I did notice and included the rotation fix for my tests

The amount of textures was the same. I think the difference comes from the update you did to the MaxRectsPacker, I see you said in some cases where the old implementation was better? Maybe this is all thats happening but in most cases (as is evident in my tests) we see improvement

@kane-mason I see what you mean now. In some cases the changes to the MaxRectsPacker are indeed worse than before. I actually did a pull request for the packing method to be an option: https://github.com/soimy/maxrects-packer/pull/11
My hope is that once that gets approved I would add the alternative packing option as a new method (or methods) to this package's MaxRectsPacker implementation. After that the OptimalPacker in this pull request would automatically select the better of the MaxRectsPacker's methods if they're better than the results from the other packer.
In the end this would mean that the result is never worse than it was before. Only better or in some cases the same.

@qtiki sounds great, thank you for these meaningful contributions!

Packing logic option has been merged, with changes enum option type

My hope is that once that gets approved I would add the alternative packing option as a new method (or methods) to this package's MaxRectsPacker implementation. After that the OptimalPacker in this pull request would automatically select the better of the MaxRectsPacker's methods if they're better than the results from the other packer.
In the end this would mean that the result is never worse than it was before. Only better or in some cases the same.

I will wait for this update before running some more tests

@qtiki any update here?

@kane-mason haven't had the time to look into this yet. Also the total lack of activity in regards to getting these pull requests approved isn't the greatest motivator for finding time either. But in any case I'll try and get the MaxRectsPacker changes tested/integrated at some point.

@odrick any chance of getting this and #6 reviewed, approved and possibly merged?

@odrick can we have some feedback here please?

Sorry guys. I was really very busy :)
I'm merge this pool request and can add you as contributors.
But it will be necessary to establish communication for the rebuild and publish of all related projects.
Thanks!

Tomorrow I will try to publish all the relative FTP builds.


Add User Menu

In the upper right of most pages, where Sign Out appears currently

Include indicator that notifies when there are Guides to review, like this thing:
Source


cookie()->forever(...) create cookie at most for one year

Laravel Version: 9.25.1
PHP Version: 8.1.9
Browser: Google chrome 104.0.5112.101 (Official Build) (64-bit)

Description:
Current behavior:
cookie()->forever('MyCookie', 'Test', secure: true) will create MyCookie in browser at most for 1 year
Expected behavior:
cookie()->forever('MyCookie', 'Test', secure: true) must create MyCookie in browser for 5 years:
2628000 minutes => 5 years
Steps To Reproduce:
1- Create a resource controller
2- Assign GET /test route to show method
3- In show method:
public function show()
{
    return response()->json(['test' => 'yes'])
        ->withCookie(cookie()->forever('MyCookie', 'Test', secure: true));
}

4- Open /test in google chrome
5- MyCookie with Test value will create in browser for only 1 year instead of 5 years

Seems like 400 days is the maximum amount of time so I guess Chrome limits this automatically: https://httpwg.org/http-extensions/draft-ietf-httpbis-rfc6265bis.html#section-<IP_ADDRESS>

Sent in a PR to update the comment. Thanks for making us aware.


Teacher: Will the CPU and GPU configuration of the computer affect the final output sound quality?

Teacher: Will the CPU and GPU configuration of the computer affect the final output sound quality?

cpu and gpu will not affect the final quality, they only affect the speed of processing.

cpu 和 gpu 不会影响最终的质量，它们只影响处理速度。

thank you


Use separate password_resets_table for each

This package uses same database table (password_resets) for each guard.
But there is problem using this -
Lets take an example :
There are two types of users

Admin
Customer

An admin can be a customer and a customer can be an admin. Same person have two different account but with same email address.
Now when that admin user make any password reset request, application will overwrite any existing (this can be himself as customer) request for same email address. This way the tokens gets lost and email link becomes invalid.
The possible solution is to use a separate table for each user type -

Create a new admin_password_resets table

<?php

use Illuminate\Support\Facades\Schema;
use Illuminate\Database\Schema\Blueprint;
use Illuminate\Database\Migrations\Migration;

class CreateAdminPasswordResetsTable extends Migration
{
    /**
     * Run the migrations.
     *
     * @return void
     */
    public function up()
    {
        Schema::create('admin_password_resets', function (Blueprint $table) {
            $table->string('email')->index();
            $table->string('token')->index();
            $table->timestamp('created_at')->nullable();
        });
    }

    /**
     * Reverse the migrations.
     *
     * @return void
     */
    public function down()
    {
        Schema::drop('admin_password_resets');
    }
}

Specify this table name in app/config/auth.php file

'passwords' => [
        'admins' => [
            'provider' => 'admins',
            'table' => 'admin_password_resets',
            'expire' => 60,
        ],

        'users' => [
            'provider' => 'users',
            'table' => 'password_resets',
            'expire' => 60,
        ],
    ],

That's it, Laravel will use this new table for admins password reset tokens from now

Thats good idea thanks. i will add it to the next version.

P.S.
Laravel 5.4 going to remove password reset table, see
https://github.com/laravel/laravel/compare/master...tillkruss:passwords

Oh.. but why? Maybe there will by any artisan command.

checkout this PR
https://github.com/laravel/framework/pull/16850


Any crate in contrib/crates.sh after fuzz is ignored

In
# Build and test for each crate, done with each toolchain.
build_and_test() {
    for crate in "${CRATES[@]}"; do
        local test_vars_script="$REPO_DIR/$crate/contrib/test_vars.sh"

        # Building the fuzz crate is more-or-less just a sanity check.
        if [ "$crate" = "fuzz" ]; then
            pushd "$REPO_DIR/$crate" > /dev/null
            cargo --locked build
            popd > /dev/null
            break
        fi

The break should be a continue otherwise any directory after fuzz is ignored.

Why are we special-casing fuzz anyway? Why not just run cargo test in there like normal?


[8.x] Add updateOrFail method to Model

The updateOrFail() method makes it more convenient to handle update errors in update actions, as it is less verbose than checking the return value of update().

sql drivers always throw an exception? what driver are you using?

@crissi I'm using the MySQL driver, but what is the difference between save() and saveOrFail() if both throw an Exception?

I see... did not know saveOrFail or existed.

The PHPDoc confirms my assumption that saveOrFail() throws, while update() doesn't:
https://github.com/laravel/framework/blob/6de01746680d7bc7e239b20ec5cdfe70ce586235/src/Illuminate/Database/Eloquent/Model.php#L864-L871
https://github.com/laravel/framework/blob/6de01746680d7bc7e239b20ec5cdfe70ce586235/src/Illuminate/Database/Eloquent/Model.php#L986-L994

Note: saveOrFail() was added in #11202.


Streaming doesn't work

Even Azure models that support streaming won't do it; the entire response is always returned in one chunk.
I have found no way to enable streaming using configuration, and from the code it doesn't seem possible. The problem appears to be with the can_stream property of the llm.Model class. Even if you define it using a config.yaml, it is ignored by the llm-azure plugin. AzureChat extends the OpenAI Chat, which in turn extends llm.Model. In Chat, can_stream is True by default, but this doesn't take effect because AzureChat doesn't call super().__init__(), so it becomes effectively False for all Azure models.
I propose to check config.yaml for a can_stream key, use it if present, and assume True otherwise. I will submit a PR shortly.

thank you for the heads up, it is fixed with the newest release :)


Update automation-dsc-overview.md

Changed "\" to "" in a file path.

Hi @alexandair, I'm your friendly neighborhood Azure Pull Request Bot (You can call me AZPRBOT). Thanks for your contribution!
You've already signed the contribution license agreement. Thanks!
We will now validate the agreement and then real humans will evaluate your PR.
TTYL, AZPRBOT;

@coreyp-at-msft Corey, can review Aleksandar's contribution and let me know if it can be merged? Thanks

#sign-off

@alexandair Thanks for the contribution Aleksandar. I'll get these added to the article.


Caching not only 'factory' content types

Discussed in https://github.com/strapi-community/strapi-plugin-rest-cache/discussions/29

Originally posted by niddu85 September 28, 2022
Hello,
I'm trying to cache results from strapi-plugin-navigation but I'm not able to properly configure the cache plugin since it needs the content type name to cache (eg api::button.button )
Is there anyone that was able to configure it properly?
On top of this discussion is there a way to get content types created by plugins to be cached?
I am also interested in the specific navigation plugin and rest cache interaction, but if someone has a more generic solution also for all kinds of content types (or costum routes created by strapi cli) that would be highly appriciated!
Just for discussions sake the content types generated by this specific plugin are:

plugin::navigation.audiance
plugin::navigation.navigation
plugin::navigation.navigation-item
plugin::navigation.navigation-item-related

And the route I am specifically interested in: /api/navigation/render/:uid
Seting up
the stragegy like
     strategy: {         contentTypes: [              {             contentType: 'plugin::navigation.navigation',             routes: [{ path: '/api/navigation/render/:uid', method: 'GET' }],           },           .....          ]}
or
       strategy: {         contentTypes: [              'plugin::navigation.navigation',           .....          ]}
did nothing sadly.
It is totally fine if this is a feature request, but if this should work even now than I think the documentation should cover this.

Looking at the code and current documentation this should work.

@laszlo-san Your path is almost correct, but the navigation plugin is registering the path slightly differently: /api/navigation/render/:idOrSlug
It seems to work when using the exact path that the navigation plugin registers or a regex that matches that path.
config/plugins.js
strategy: {
  contentTypes: [
    {
      contentType: 'plugin::navigation.navigation',
      routes: [
        {
          path: '/api/navigation/render/:idOrSlug',
          method: 'GET',
        }
      ],
    }
  ]
}

@garionberg You are right!
Thank you, everything works as it should, closing this as it was user error on my side.


Make certs available for containers

Need to provide cert and key to host ssl connections

AWS Generated Certificates are not downloadable.
Might need to add the option to upload owned certificates.

I want to set ssl in the running ec2 instance, is that possible ? to access https

@LeoDiazL any update ?

Hi @sostenesapollo !
Thanks for reaching out and expressing your interest in this GitHub Action!
At this moment, certificate creation and usage is only available with domains managed with Route53.
More info on the required inputs here
More info on the the requirements here.
If your scenario is somehow different, we'd be happy to help you with the functionality you're looking for. However, we'll need more details on what you're specifically looking for so we can get a better understanding of your needs.
Once we have more information, we'll prioritize it based on our current goals and see how we can integrate it into our future plans. Please keep in mind that I can't guarantee when it will be implemented.
Thanks again for your interest and I look forward to hearing back from you.


removing lighten and darken from tests bc they change too much

When Sass updates the implementation of lighten and darken the hardcoded test values in here break. We only want to test certain parsing of functions, so changing them to functions that hypothetically should never change.

You probably want to change your git username and password on your computer for this repo otherwise these changes won't show up on your account page as it stands now.
PR LGTM

I'm aware. Was at lunch 🍴


Adding comment about gas

Description
As specified during the daily, here is a small PR that adds documentation to the sov-rollup-starter in the README.md on gas consumption.

 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.


Error: Error: Cannot find module<EMAIL_ADDRESS>Require stack:

I'm getting this error "Error: Error: Cannot find module<EMAIL_ADDRESS>When my code is deployed on Vercel (Node.js + TypeScript) Does I get confused because when I try to run the same code on my local machine it works fine
But as I deploy it on Vercel it gives that error. Obviously, i do have<EMAIL_ADDRESS>installed as said in the package https://github.com/FFmpeg-wasm/FFmpeg.wasm#installation So how do I fix it?
I'm currently using it like this

import { FFmpeg } from<EMAIL_ADDRESS>
async function myFunction() {
    const ffmpeg = await FFmpeg.create({
        core<EMAIL_ADDRESS>        log: true,
    });
}

However, in the docs, i see doing like this
  import { FFmpeg } from<EMAIL_ADDRESS>
     const ffmpeg = await FFmpeg.create({
            core<EMAIL_ADDRESS>            log: true,
        });

    async function myFunction() {
        // use ffmpeg here
    }

But then i start getting errors like "top-level 'await' expressions are only allowed when the 'module' option is set to 'es2022', 'esnext', 'system', 'node16', or 'nodenext', and the 'target' option is set to 'es2017' or higher."
I tried a couple of StackOverflow solutions but nothing works in my case. So what should I do?
tsconfig.json:
{
    "compilerOptions": {
     "module": "CommonJS",
     "esModuleInterop": true,
      "allowSyntheticDefaultImports": true,
      "target": "es2017",
      "noImplicitAny": true,
      "moduleResolution": "node",
      "sourceMap": true,
      "outDir": "dist",
      "baseUrl": ".",
      "paths": {
        "*": ["node_modules/*", "src/types/*"]
      }
    },
    "include": ["./src/**/*", "src/firebase/serviceAccountKey.ts"]
  }

But then i start getting errors like "top-level 'await' expressions are only allowed when the 'module' option is set to 'es2022', 'esnext', 'system', 'node16', or 'nodenext', and the 'target' option is set to 'es2017' or higher."

This is because CommonJS does not support top-level await.
You can change Module to one of the four values above, but if you need to retain support for old versions of node, then the only way is to avoid using this syntax.

@DreamOfIce That's not the issue, Is there any other way to use @ffmpeg.wasm/core-mt

@DreamOfIce That's not the issue, Is there any other way to use @ffmpeg.wasm/core-mt

I'm still working on that.
Can you set NODE_ENV to development and send the log here?

@DreamOfIce Well i think its becuaes of Ram CPU etc.

 /opt/render/project/src/node_modules/@ffmpeg.wasm/core-mt/dist/core.js:15
Oct 1 04:53:37 PM  Error("bad memory");var Aa=z.buffer;e.HEAP8=A=new Int8Array(Aa);e.HEAP16=C=new Int16Array(Aa);e.HEAP32=E=new Int32Array(Aa);e.HEAPU8=B=new Uint8Array(Aa);e.HEAPU16=wa=new Uint16Array(Aa);e.HEAPU32=F=new Uint32Array(Aa);e.HEAPF32=xa=new Float32Array(Aa);e.HEAPF64=ya=new Float64Array(Aa);za=z.buffer.byteLength;var H,Ba=[],Ca=[],Da=[],Ea=[],Fa=[],Ga=!1,Ha=0;function Ia(){return noExitRuntime||0<Ha}function Ja(){p||(e.noFSInit||I.Nb.Mc||I.Nb(),I.jd=!1,K.root=I.wb(K,{},null),Ka(Ca))}
Oct 1 04:53:37 PM  ^
Oct 1 04:53:37 PM
Oct 1 04:53:37 PM  Error: bad memory
Oct 1 04:53:37 PM      at /opt/render/project/src/node_modules/@ffmpeg.wasm/core-mt/dist/core.js:15:1
Oct 1 04:53:37 PM      at Function.<anonymous> (/opt/render/project/src/node_modules/@ffmpeg.wasm/main/dist/index.js:165:64)
Oct 1 04:53:37 PM      at Generator.next (<anonymous>)
Oct 1 04:53:37 PM      at fulfilled (/opt/render/project/src/node_modules/@ffmpeg.wasm/main/dist/index.js:28:22)

my code

import express from "express";
const router = express.Router();
import { readFile, writeFile } from "fs/promises";
import { FFmpeg } from<EMAIL_ADDRESS>import path from "path";

const currentDirectory = process.cwd();

const inputFilePath = path.join(currentDirectory, "video3.mp4");
const inputFilePath1 = path.join(currentDirectory, "wolf-howl-6310.mp3");
const outputFilePath = path.join(currentDirectory, "video3-2.mp4");

const ffmpeg = await FFmpeg.create({ core<EMAIL_ADDRESS>log: true });

try {
  ffmpeg.fs.writeFile("/video3.mp4", await readFile(inputFilePath));
  ffmpeg.fs.writeFile("/wolf-howl-6310.mp3", await readFile(inputFilePath1));
  console.log(ffmpeg.version);
  await ffmpeg.run(
    "-i",
    "/wolf-howl-6310.mp3",
    "-i",
    "/video3.mp4",
    "-c",
    "copy",
    "-map",
    "0:a:0",
    "-map",
    "1:v:0",
    "/video3-2.mp4"
  );
  await writeFile(outputFilePath, ffmpeg.fs.readFile("/video3-2.mp4"));
  console.log("Conversion successful!");
} catch (error) {
  console.error("Error:", error);
}

router.post("/", (req, res) => {});
export { router as Merge };

I had this issue on Vercel as well, it was because there is no import to tell it to include the lib.
I used import core from<EMAIL_ADDRESS>and then passed the core to the create function.
I'm battling other issues on vercel right now though with this lib, so I'll keep you posted


Add subscription type to GraphQL schema explorer.

Hi @cameronr-nulogy, thanks for the PR! Code looks great. 😄
Since there is a UI change involved, could you please add a screenshot/gif? Thanks! 🎉

Here ya go:

Nice one!