kubectl get clusters does not work consistently

The problem is this:
ubuntu@nephio-r1-e2e:~$ kubectl api-resources | grep clusters
clusters                          cl           cluster.x-k8s.io/v1beta1                  true         Cluster
clusters                                       clusterregistry.k8s.io/v1alpha1           true         Cluster
clusterselectors                               configmanagement.gke.io/v1                false        ClusterSelector
workloadclusters                               infra.nephio.org/v1alpha1                 true         WorkloadCluster
dockerclusters                                 infrastructure.cluster.x-k8s.io/v1beta1   true         DockerCluster
ubuntu@nephio-r1-e2e:~$

At some point, the clusterregistry.k8s.io/v1alpha1 started interfering with the cluster.x-k8s.io/v1beta1. Where did the cluster registry one come from? I never saw this until we update the mgmt cluster to a newer K8s version. I don't know if that's because the algorithm in the older k8s version always picked the right "clusters", or because something else changed and we now have the clusterregistry one and we did not before. It needs to be investigated - do we even need the clusterregistry one? How is it getting added to our system?
Most of the e2e scripts have been fixed to account for this, but it will still cause confusion for end users if we do not fix it.
/kind bug

I could be wrong, but it doesn't come from ConfigSync?

I don't think so. That's more an informational thing in CS for when it is running on clusters that have that installed. Let me grep around.

Oof, I think you're right:
https://github.com/GoogleContainerTools/kpt-config-sync/blob/v1.13/manifests/cluster-registry-crd.yaml

Let me look into why that's there. It's probably for ClusterSelectors which we do not use.

Ok, I filed an issue with CS and also reached out to some folks internally.


Parsing multiline strings

Why does the following code:
var input =
    $"""
    .....
    .F-7.
    .|.|.
    .L-J.
    .....
    """;

var parser = Separated(
    Terms.Text(Environment.NewLine),
    OneOrMany(
        OneOf(Terms.Char('|'),
            Terms.Char('-'),
            Terms.Char('L'),
            Terms.Char('J'),
            Terms.Char('7'),
            Terms.Char('F'),
            Terms.Char('.'),
            Terms.Char('S'))));

var result = parser.Parse(input);

produce a singleton List containing 25 entries, instead of 5x lists each containing 5 entries?
Or should I be parsing multiline strings in a different way?

Terms will implicitly accept spaces so the new lines will be part of the "Char" result. Try to use Literals.Char instead.

Changing the separator parser from Terms.Text(Environment.NewLine) to Literals.Text(Environment.NewLine) still returns a singleton list with 25 elements. Should this not work?
Changing the separator parser to Literals.Char('\r').And(Literals.Char('\n')) (I'm using Windows as my primary OS) also produces the same result. Furthermore, it's a bit of a hassle to have to create a Char parser from Environment.NewLine. Why can't I use Literals.Text instead?

Please ignore my last comment. I figured it out. The following code produces the desired result:
var input =
    $"""
    .....
    .F-7.
    .|.|.
    .L-J.
    .....
    """;

var parser = Separated(
    Literals.Text(Environment.NewLine),
    OneOrMany(
        OneOf(Literals.Char('|'),
            Literals.Char('-'),
            Literals.Char('L'),
            Literals.Char('J'),
            Literals.Char('7'),
            Literals.Char('F'),
            Literals.Char('.'),
            Literals.Char('S'))));

var result = parser.Parse(input);

I tried locally and I believe there is a bug in OneOf, currently investigating.
:
Note that if you want the parsers to not skip whitespaces there is another option by using
var parseContext = new ParseContext(new Scanner(input), true);
var result = parser.Parse(parseContext);

Though this should probably be made easier to access on Parse() directly.


'ReadyRole' in '360' seconds.

Hello again,
If the provider takes more than 5 minutes to create the instance in Azure, it returns this "error":
==> default: Machine failed to reached state 'ReadyRole' in '360' seconds
and the bootstrap script fails.
In US zone deploy VM is taking arround 8 to 10 minutes.
In the doc is not any var to incrase this timeout, can you include it if it doesn´t exists?
Thanks again.

This can be fixed by specifying the boot timeout via config.vm.boot_timeout = 1000 or some other arbitrarily large number of seconds. There is now an example in the readme. #44 changed eliminated the config value from being set, thus allowing the user to specify the boot_timeout in the vagrant file.


ARROW-7364: [Rust] Add cast options to cast kernel [WIP]

@andygrove @alamb @nevi-me
This is my WIP implementation of adding CastOptions to the Rust Kernel and changing the default CastOptions for DataFusion to be safe = false.
From here we have two options:

use some sort of feature flag (like Spark) to set the default (see spark.sql.ansi.enabled here).
add a safe_cast expr to do the same operation but override the default.

Thoughts?

Thanks @seddonm1 this is a really important feature to add.
To answer your question about the two options, I would favor making this explicit and let the user choose between  cast and safe_cast (or whatever name we choose) when building their logical query plan via DataFrame or SQL.
Our optimizer has automatic type coercion rules and I would expect those to default to safe_cast. If the user doesn't like that behavior then they can add explicit unsafe casts instead.

Thanks both. I am away this weekend but will incorporate these changes and will notify early next week for final PR with more tests.

👍

@alamb @andygrove
I have made the changes to the core from your review but I have not built the try_cast vs safe_cast functionality yet.
Here are the two names and their provenance:

SQL Server try_cast
BigQuery safe_cast

My (changed) vote is try_cast for two reasons:

I like referring to another more mainstream SQL dialect (MS SQL has been around a lot longer than BigQuery)
try_cast does feel like a more 'rusty' implementation

Based on the outcome of this vote I will raise a PR against the sqlparser-rs so we can support the same TRY_CAST(x AS y) syntax and then can finish this PR.

Thanks @seddonm1

FYI https://github.com/ballista-compute/sqlparser-rs/pull/299 has been raised to add TRY_CAST to the parser.

@alamb I think this is ready to have a look at as I would appreciate some feedback.
Thanks to @Dandandan for releasing the new sqlparser-rs version so fast 👍

I will check it out later today
On Thu, Mar 25, 2021 at 1:39 AM Mike Seddon @.***>
wrote:

@alamb https://github.com/alamb I think this is ready to have a look at
as I would appreciate some feedback.
Thanks to @Dandandan https://github.com/Dandandan for releasing the new
sqlparser-rs version so fast 👍
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/apache/arrow/pull/9682#issuecomment-806378985, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AADXZMKOLPKIMGQPPWDJPYLTFLEBTANCNFSM4ZBJA54Q
.

@alamb Thanks for the review. This had a bit of a gap while waiting for the sqlparser changes so I lost context. I will finish it Monday (your comments were absolutely correct) and hopefully then we can merge early in the week.

Thanks again @seddonm1

@seddonm1  just let me know when this one is ready and we can 🚀

@alamb I reworked this based on your comments and decided that explicitly creating a cast vs try_cast expression is valuable when reading things like the type_coercion code which now is more obvious which function is being used (try_cast). Can you have another look? (sorry I had to force push due to my doing bad git)

I am running the tests one more time locally, and if they pass I'll merge this in

all looks good 👍 thanks again @seddonm1 !


[1.12.2 Forge] Crashing on startup due to "Found FMLCorePluginContainsFMLMod marker"

Minecraft 1.12.2
Forge <IP_ADDRESS>59
Lootr 0.6.2
I am creating a modpack with the following mods:
modlist.txt
Recently, when I enable Lootr, the modpack fails to start. It works perfectly fine without Lootr.
Here's the latest.log:
Client:
latest.log
Server:
latest.log
It seems like the error is caused by the following:
[02:44:15] [main/WARN] [FML]: Found FMLCorePluginContainsFMLMod marker in lootr-1.12.2-0.6.2.jar. This is not recommended, @Mods should be in a separate jar from the coremod.
[02:44:15] [main/ERROR] [LaunchWrapper]: Unable to launch
java.lang.RuntimeException: An error occurred trying to configure the Minecraft home at /home/tw1zzler/.local/share/PrismLauncher/instances/The Brilliant Expanse/minecraft for Forge Mod Loader

Let me know how else I can help solve this issue.

Please install MixinBooter as outlined on the CurseForge page multiple times.

 Thank you.


rancher-metadata - rancher-metadata versions 2015-07-25 and 2015-12-19 are interchanged for  /stacks//services and /services//container

Server version - Build from master
rancher-metadata - rancher-metadata versions 2015-07-25 and 2015-12-19 are interchanged for  /stacks//services and /services//containers
http://rancher-metadata/2015-07-25/stacks/1/services results in service objects being returned.
Expected Behavior:
Return list of service names.
http://rancher-metadata/2015-12-19/stacks/1/services results in service names being returned.
Expected Behavior:
Return list of service objects.
http://rancher-metadata/2015-07-25/services/1/containers results in container objects being returned.
Expected Behavior:
Return list of container names.
http://rancher-metadata/2015-12-19/services/1/containers results in container names being returned.
Expected Behavior:
Return list of container objects.

Tested with latest build from master:
http://rancher-metadata/2015-07-25/stacks/1/services returns list of service names.
http://rancher-metadata/2015-12-19/stacks/1/services returns list of service objects.
http://rancher-metadata/2015-07-25/services/1/containers return list of container names.
http://rancher-metadata/2015-12-19/services/1/containers return list of container objects.


Online_Network_Player line 247

Uncaught TypeError: Cannot read property 'clearNetworkPlayer' of undefined
It happens when there are many other players in a map when I open the menu for too long (ie 5 seconds or more and there are more than 4 people running in a map)

I removed the line 247 and 1 line at "Remove Player", it seems to work great.

This may be the last bug in the MV_Online project 💯

haha, there are a lot of people in my country that use this project, but they do not know the english language so i can report it to Nelderson, so i usually inform him the smallest error ^^

Yes, but I live in America


Make DrawBuilder non-consuming

By taking &mut self instead of mut self, the user will be able to chain calls
more ergonomically.
This pattern is recommended by the Rust API guidelines.

I think the reason why I didn't do this the first time was the impact to the user-facing API. It has a bit more overhead to consume the data, but that's initialization work not real-time work. Is there a way to do this without making the draw building API more verbose? I think I'd be OK with a small perf hit for some better developer ergonomics.


improve yurtadm join command

What type of PR is this?

Uncomment only one /kind <> line, hit enter to put that in a new line, and remove leading whitespace from that line:
/kind bug
/kind documentation
/kind enhancement
/kind good-first-issue
/kind feature
/kind question
/kind design
/sig ai
/sig iot
/sig network
/sig storage

/kind enhancement
What this PR does / why we need it:

improve yurthub staticpod yaml
remove yurthub image parameter in yurtadm join command, because yurthub image should be defined in yurthub static pod.

Which issue(s) this PR fixes:

Fixes #
Special notes for your reviewer:

Does this PR introduce a user-facing change?

other Note

/lgtm


Error when pretrained on Cascade mask RCNN SwinTransformer

I use the pretrained weight from "https://github.com/SwinTransformer/storage/releases/download/v1.0.2/cascade_mask_rcnn_swin_base_patch4_window7.pth".  However, the error still appear. And the method in https://github.com/open-mmlab/mmdetection/pull/6019https://github.com/open-mmlab/mmdetection/pull/6019  doesn't work either. My config file is as following:
# model settings
pretrained = 'checkpoints/swin_base_patch4_window7_224.pth'
model = dict(
    type='CascadeRCNN',
    backbone=dict(
        type='SwinTransformer',
        embed_dims=128,
        depths=[2, 2, 18, 2],
        num_heads=[4, 8, 16, 32],
        window_size=7,
        mlp_ratio=4,
        qkv_bias=True,
        qk_scale=None,
        drop_rate=0.,
        attn_drop_rate=0.,
        drop_path_rate=0.3,
        patch_norm=True,
        out_indices=(0, 1, 2, 3),
        with_cp=False,
        convert_weights=True,
        init_cfg=dict(type='Pretrained', checkpoint=pretrained)),
load_from = 'checkpoints/cascade_mask_rcnn_swin_base_patch4_window7.pth'

And my mmdet version is 2.23.0. How to fix it ?

hello, i meet the same issue. Did you fix it?


docs: Add deploy to Zeabur button

Summary
This Pull Request add a "deploy to Zeabur" button to let users can deploy LibreChat to Zeabur with one click.

Change Type
Please delete any irrelevant options.

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] This change requires a documentation update
[x] Documentation update
[ ] Translation update

Checklist

[x] My code adheres to this project's style guidelines
[x] I have performed a self-review of my own code
[x] I have commented in any complex areas of my code
[x] I have made pertinent documentation changes
[x] My changes do not introduce new warnings
[x] I have written tests demonstrating that my changes are effective or that my feature works
[x] Local unit tests pass with my changes
[x] Any changes dependent on mine have been merged and published in downstream modules.

At the time, we have to set a precedent for not allowing deployments with specific referral codes unless owned by the maintainers (which helps cover the cost of test deployments on the platforms we host).
Also, for 1-click deployments, I will also be requiring guides. Even though 1-click is simple, it is still handy to have a walkthrough.


Globalfee

Description and Motivation
x/globalfee from the cosmos hub, upgraded to use cosmos-sdk v0.47.x
This PR is incomplete, and at minimum will need fairly extensive changes to tests.
Related Issues
#138

Checklist:

[ ] I have read Migaloo's contribution guidelines.
[ ] My pull request has a sound title and description (not something vague like Update index.md)
[ ] All existing and new tests are passing.
[ ] I updated/added relevant documentation.
[ ] The code is formatted properly golangci-lint run ./... --fix.

@faddat we still missing upgrade handler for v3. I'll make a PR for this.

@hoangdv2429 @sontrinh16 @pysel we should also upgrade x/globalfee to make sure it exactly matches gaia's for better maintainability.


Code issues

Please ask installation, training and test issues in this panel.

Hello. There always something wrong when i download "Lee_VPGNet_Vanishing_Point_ICCV_2017_supplemental.pdf" from the internet, could you send a new pdf to me ? Thank you very much!

@CSUMIT Oh, our supplemental file is not pdf format. It's just a video clip and here is the link :)
https://www.youtube.com/watch?v=jnewRlt6UbI

Oh, it is a supplemental that i find in : http://openaccess.thecvf.com/ICCV2017.py. The supplemental of your paper.

@CSUMIT I uploaded a video file but the organizer seems to have changed the file to pdf format. You may ignore that link.

OK, thank you very much.

 Or use sudo train.sh?

@daixiaogang Yes, please try it again after making ./snapshots.

@SeokjuLee ，I have trained your  network with caltech-lane dataset  correctly.After about 14 hours ,it iterates 30000,but when I plot the test and train loss log,it looks strange. The training loos converge by iteration but the test loss first increase and then unchanged.I did not change any of your configuration,should I make some change? As the dataset is small. (For training, cordova1 ,cordova2,washington1 as the trainlist ,washington2 as the testlist)

@daixiaogang It is normal for the validation accuracy to rise sharply in the beginning and not change significantly afterwards. This is because the size of the object to be inferred is much smaller than the background area. If you expand the validation curve, you can see that it is increasing slightly.

Is there any trained model that can be used for test ?

@cfzd Here is an unofficial weight. Please refer this link :>
https://drive.google.com/open?id=1MBi3G5cL1FJGkESHpRiAbMVediUTULXL

@SeokjuLee ,I want to transfer my label which likes(x1,y1,x1',y1'),(x2,y2,x2',y2').My picture is 1280x1024 which different from yours 640x480,can you give me some advice on how transfer labels ? Or just change the width and height parameter in your vpg_annot_v1.m?

@SeokjuLee ,because my label only have two points(x1,y1,x1',y1'),can not call the function ccvEvalBezSpline() in vpg_annot_v1.m.,so it got wrong. Do we must make spline first to make bouding box? can you give me some advice?

@daixiaogang First, you should decide which input size to use, 640x480 or 1280x1024. If you want to use the latter one, please check the intermediate activation sizes after the branches. Basically the network is full convolutional so various sizes are applicable, but there might need some parameter tunings. Second, Is your label contains only two end points that represent one straight line for each lane? The label doesn't need to be always spline curve. First draw each straight line with two points on the image, then annotate grids through which the line passes.

@SeokjuLee ,Thanks for your explaination，I have make the label like yours. I want to know more about your parameters to make anonation,such as gridsize(8) and thickness(2),should these parameters make some change to fit my picture?

@SeokjuLee ,I have trained your net witch caltech-lane dataset.I want to konw how to test my results or output the cordinate(x1,y1,x2,y2...) of lanes, can you give me some advice?

@daixiaogang Well, you should better not to change grid size (8) because that parameter depends on the rescaling factor between input (640x480) and output (multi-label:80x60) size. The thickness depends on the lane width. If the grid annotation covers the lane markings enough, I don't think you need to change it.

@SeokjuLee ,Thanks for your guide.But I still have some question for the demos and tests. I use the command "./build/tools/caffe test -model models/vpgnet-novp/deploy.prototxt -weights models/vpgnet-novp/snapshots/split_iter_82500.caffemodel -iterations 1 >>output.log 2>&1",I want to know how to input a picture and get the cordinate of the lane?Because deploy.prototxt did not indicate the input. Can you give me some advice?

@daixiaogang Try net.forward_all(data=np.array(transformed_image)); binary_mask = net.blobs['binary-mask'.data[0];

@SeokjuLee ，it get wrong as the following message.
---------------------------------------------------------message-----------------------------------------------------------
WARNING: Logging before InitGoogleLogging() is written to STDERR
W1226 15:32:11.351035  8293 _caffe.cpp:122] DEPRECATION WARNING - deprecated use of Python interface
W1226 15:32:11.351058  8293 _caffe.cpp:123] Use this instead (with the named "weights" parameter):
W1226 15:32:11.351063  8293 _caffe.cpp:125] Net('deploy.prototxt', 1, weights='./snapshots/split_iter_82500.caffemodel')
[libprotobuf ERROR google/protobuf/text_format.cc:274] Error parsing text-format caffe.NetParameter: 452:16: Message type "caffe.LayerParameter" has no field named "tiling_param".
F1226 15:32:11.352567  8293 upgrade_proto.cpp:88] Check failed: ReadProtoFromTextFile(param_file, param) Failed to parse NetParameter file: deploy.prototxt
*** Check failure stack trace: ***
已放弃 (核心已转储)
------------------------------------------------end---------------------------------------------------------------------
it turns out "caffe.LayerParameter" has no field named "tiling_param",but I have found this param in caffe.proto. It is very strange. Can you write a demo to use "deploy.prototxt"

@daixiaogang Our pycaffe loading lines are almost same as yours. Could you check the pycaffe path? tiling_param is customized caffe parameter.

@SeokjuLee ,thanks for your reminding,I have solved the path problem. But there turns out a new problem: "File "outpt.py", line 30, in  binary_mask = net.blobs['binary-mask'].data[0];
KeyError: 'binary-mask'". there does not exisit 'binary-mask' in deploy.prototxt.

@daixiaogang Please use updated model. The deploy model is updated. "binary-mask" and "multi-label" are softmax output.

@SeokjuLee，can you explain how to visualize the ouput by "binary-mask" and "multi-label" ? I am reading your code (drive_data_layer.cpp and convert_driving_data.cpp),I am wondering are you trying to regression the gridbox(x,y,w,h)like object detetcion?so your output is the bouding box?

@daixiaogang Could you please refer our paper? We elaborate the post-processing in Section 4.4 :)

@daixiaogang @SeokjuLee  hello,how to use models to test my test-data?

@daixiaogang  could you give me a email? i want to discuss this model  in chinese,because my english is so poor!

@Lapin-Lam , you should not use cudnn in compiling caffe. In other words, you should not uncomment
# USE_CUDNN := 1 in Makefile.config.

@cardwing hey! I have been working on this for a while now. How do you display the output? In other words, what am I supposed to do after the last step (run "train.sh"). Thanks!

hello,i have a problem in training vpgnet,the loss of bbox is high while the accurancy is 95%,can anyone have idea that what's cause the phenomenon
like blew:
I0804 03:12:02.197446  6967 solver.cpp:414]     Test net output #0: bb-loss = 1.17429 (* 1 = 1.17429 loss)
I0804 03:12:02.197453  6967 solver.cpp:414]     Test net output #1: pixel-acc = 0.967028
I0804 03:12:02.197458  6967 solver.cpp:414]     Test net output #2: pixel-loss = 0.10597 (* 1 = 0.10597 loss)
I0804 03:12:02.197461  6967 solver.cpp:414]     Test net output #3: type-acc = 0.952496
I0804 03:12:02.197466  6967 solver.cpp:414]     Test net output #4: type-loss = 0.220534 (* 1 = 0.220534 loss)

Thanks for your guide.But I still have some question for the demos and tests. I use the command "./build/tools/caffe test -model models/vpgnet-novp/deploy.prototxt -weights models/vpgnet-novp/snapshots/split_iter_82500.caffemodel -iterations 1 >>output.log 2>&1",I want to know how to input a picture and get the cordinate of the lane?Because deploy.

Hi,I Also met this question ,can you tell me how to solve it?

@SeokjuLee ，it get wrong as the following message.
---------------------------------------------------------message-----------------------------------------------------------
WARNING: Logging before InitGoogleLogging() is written to STDERR
W1226 15:32:11.351035 8293 _caffe.cpp:122] DEPRECATION WARNING - deprecated use of Python interface
W1226 15:32:11.351058 8293 _caffe.cpp:123] Use this instead (with the named "weights" parameter):
W1226 15:32:11.351063 8293 _caffe.cpp:125] Net('deploy.prototxt', 1, weights='./snapshots/split_iter_82500.caffemodel')
[libprotobuf ERROR google/protobuf/text_format.cc:274] Error parsing text-format caffe.NetParameter: 452:16: Message type "caffe.LayerParameter" has no field named "tiling_param".
F1226 15:32:11.352567 8293 upgrade_proto.cpp:88] Check failed: ReadProtoFromTextFile(param_file, param) Failed to parse NetParameter file: deploy.prototxt
*** Check failure stack trace: ***
------------------------------------------------end---------------------------------------------------------------------
it turns out "caffe.LayerParameter" has no field named "tiling_param",but I have found this param in caffe.proto. It is very strange. Can you write a demo to use "deploy.prototxt"

Hi @daixiaogang , I also got the same error message. Were you able to solve it? Thanks.

Hello, when I downloaded the code from GitHub and compiled and tested the caffe, I encountered the following problems when running the train.sh file:
I0924 18:00:51.584237 31829 solver.cpp:102] Creating training net from net file:. /train_val.prototxt
[libprotobuf ERROR google/protobuf/text_format.cc:274] Error parsing Text-format caffe.NetParameter: 15:20: Message type "caffe.Layer Parameter" has no field named "drive_data_param".
F0924 18:00:51.584311 31829 upgrade_proto.cpp:90] Check failed: ReadProtoFromTextFile (param_file, param) Failed to parse NetParameter file: / train_val.prototxt.
What's the matter, please? Is the Caffe version used incompatible? I copy the files in the models directory to the models directory of my local compiled caffe.

Hi, Please confirm whether you have set the environment  variable of  caffe you complied.
------------------ 原始邮件 ------------------
发件人<EMAIL_ADDRESS>发送时间: 2019年9月24日(星期二) 晚上6:08
收件人<EMAIL_ADDRESS>抄送: "抑扬人"1433692592<EMAIL_ADDRESS>主题: Re: [SeokjuLee/VPGNet] Code issues (#3)
Hello, when I downloaded the code from GitHub and compiled and tested the caffe, I encountered the following problems when running the train.sh file:
I0924 18:00:51.584237 31829 solver.cpp:102] Creating training net from net file:. /train_val.prototxt
[libprotobuf ERROR google/protobuf/text_format.cc:274] Error parsing Text-format caffe.NetParameter: 15:20: Message type "caffe.Layer Parameter" has no field named "drive_data_param".
F0924 18:00:51.584311 31829 upgrade_proto.cpp:90] Check failed: ReadProtoFromTextFile (param_file, param) Failed to parse NetParameter file: / train_val.prototxt.
What's the matter, please? Is the Caffe version used incompatible? I copy the files in the models directory to the models directory of my local compiled caffe.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub, or mute the thread.

Hi @jiang4869 , try this
sudo vim ~/.bashrc
then add pycaffe to the PYTHONPATH by adding this line -
export PYTHONPATH=/path/to/VPGNet/caffe/python:$PYTHONPATH


How to write to a file?

Reading https://getpino.io/#/docs/api?id=destination-sonicboom-writablestream-string, I am trying to write the log to a file.
When I write
const pino = require('express-pino-logger')('./logs');
I get error
/home/jason/projects/node/h1bhub-app/node_modules/pino-http/logger.js:15
  opts.serializers = opts.serializers || {}
                   ^

TypeError: Cannot create property 'serializers' on string './logs'
    at pinoLogger (/home/jason/projects/node/h1bhub-app/node_modules/pino-http/logger.js:15:20)
    at Object.<anonymous> (/home/jason/projects/node/h1bhub-app/index.js:3:44)
    at Module._compile (internal/modules/cjs/loader.js:689:30)
    at Object.Module._extensions..js (internal/modules/cjs/loader.js:700:10)
    at Module.load (internal/modules/cjs/loader.js:599:32)
    at tryModuleLoad (internal/modules/cjs/loader.js:538:12)
    at Function.Module._load (internal/modules/cjs/loader.js:530:3)
    at Function.Module.runMain (internal/modules/cjs/loader.js:742:12)
    at startup (internal/bootstrap/node.js:282:19)
    at bootstrapNodeJSCore (internal/bootstrap/node.js:743:3)
npm ERR! code ELIFECYCLE
npm ERR! errno 1
npm ERR<EMAIL_ADDRESS>server: `node index.js`
npm ERR! Exit status 1
npm ERR!
npm ERR! Failed at the<EMAIL_ADDRESS>server script.
npm ERR! This is probably not a problem with npm. There is likely additional logging output above.

npm ERR! A complete log of this run can be found in:
npm ERR!     /home/jason/.npm/_logs/2019-03-10T20_57_50_633Z-debug.log

Any idea why?

The full API is defined in https://github.com/pinojs/pino-http#pinohttpopts-stream.

const pino = require("pino")("./logs/info.log");
const expressPino = require("express-pino-logger")({
  logger: pino
});
app.use(expressPino);

This setup will write to ./logs/info.log

maybe pino.destination doesn't satisfy this check?
if (opts && opts._writableState) {
    stream = opts
    opts = null
  }

that is in pino-http?


[DC-1344] Add Sixteenth Street Community Health Center to pipeline

DC-1344 Add hpo site SSHC to deid state lookup

@ksdkalluri Since you reviewed the config update for the site, can you review this as well? They're for the same ticket.


Feature: work on getting disconnect/reconnect working.

When the socket.io connection disconnects, ttyx is ending the session completely.  Since socket.io can do reconnects, I'd like to see that actually work and not drop the connection every time.  Sessions should time-out at a configurable interval - maybe an hour.

Fixed when syncSession and sessionTimeout are set.  (Now need to test when they aren't set.)


What are the most important things to reproduce the result on my own dataset?

UDA(https://github.com/google-research/uda) could achieve good accuracy by only 20 training data on text classification. But I find it is hard to reproduce the result on my own dataset.
So I want to know the reason why UDA or ReMixMatch works. And I want to know what is the most important thing to reproduce the result.
@carlini  Thank you very much.

I have responded to this question in the MixMatch repository
https://github.com/google-research/mixmatch/issues/27
The answer here is unlikely to be different.

Thank you.


Multiplayer issue

Me and my friend have been having an issue with mystic Items mod since the 1.1.13 update. When we try and join one another multiplayer session we get kicked because of the mystic items mod. The Message we get is "incompatible mod. Host has mystic items version 1.1.12 but you have mystic items 1.1.13" We try and downgrade to an older version of the game but we still get the same error message with the same version. Has anyone else experience this error? Is there anyway we can fix it?

Hi!
(If you're using r2modman or Thunderstore Mod Manager, go to Settings > Browse profile folder, from there you can find the BepInEx folder)

This should no longer happen starting from mod version 2.0.13


fix: add overflow to card content (for card/help)

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


xWebApplication module problem when calling get-dscconfiguration

As per #302 - same issue with xWebApplication - when calling get-dscconfiguration you get:

Get-DscConfiguration : A general error occurred that is not covered by a more specific error code.
At line:1 char:1

Get-DscConfiguration

  + CategoryInfo          : NotSpecified: (MSFT_DSCLocalConfigurationManager:root/Microsoft/...gurationManager) [Get
 -DscConfiguration], CimException
  + FullyQualifiedErrorId : MI RESULT 1,Get-DscConfiguration

This is avoided in the same manner. If you comment out the line in MSFT_xWebApplication.psm1 as below:
        #AuthenticationInfo       = $cimAuthentication
Once this has been done, you can run get-dscconfiguration without issues.

Had a similar error with the xSqlserverSetup resource,  and after some digging, @johlju suggested something.  I applied his suggestion to the problem here too and it worked.  So, changing the line to this: AuthenticationInfo  = [System.String[]] $cimAuthentication
solve the problem.  Si I take it the value returned to $cimAuthentication might not be the one expected in some case.(that was the case with the xSqlserverSetup problem.)

I started investigating this issue. I tried it on several machines and with WMF 4.0 and 5.0 it seems to work just fine. The problem occurs only on machines where I have WMF 5.1 installed.

Raised the issue with the PowerShell Product Group. They were able to reproduce the issue in WMF 5.1 and are looking for a solution.

@ykuijs Have you heard of a solution to this yet?

@ykuijs Added a comment to the other issue https://github.com/PowerShell/xWebAdministration/issues/302#issuecomment-383914623. Was it the conclusion, that it was the extra field PSComputerName that generates this error?

@BerheAbrha is the engineer that troubleshooted this issue and submitted PR #335 to fix this issue. As it turns out there is an issue with the specified type of the instance, where WMF 5.1 has an additional check which resulted in this issue. The PR should fix this issue.
Unfortunately tests for this PR are failing. I have asked him to update the PR.

@ykuijs Thanks for the clarification!


Cannot convert a line of decimal numbers to hexadecimal

I tried to launch :Dec2Hex in normal mode on this line of decimal numbers:
0, 0, 0, 0, 1, 1, 1, 1, 2, 2, 2, 2, 3, 3, 3, 3, 4, 4, 5, 5, 6, 6, 7, 7, 8
but the command converted only the first number
0x0, 0, 0, 0, 1, 1, 1, 1, 2, 2, 2, 2, 3, 3, 3, 3, 4, 4, 5, 5, 6, 6, 7, 7, 8
It happens on more lines also

I'd appreciate help with this as well as #5

So would a possible approach be converting a single word or digit under the cursor in normal mode, while an entire line in visual mode?

It's not that simple. http://vimdoc.sourceforge.net/htmldoc/visual.html#blockwise-operators


Support the enforcement of RedHat signatures

While it is possible to create a Kubernetes stack which enforces RedHat style container image signatures at the platform level it requires non-generic platform components ... specifically it requires container runtime to be CRI-O, effectively "be OpenShift".
Additionally the configuration of requirements is not exposed to the Kubernetes API and and cannot be selectively configured at Kubernetes namespace granularity.
Portieris support for this signature style will be generic and have Kubernetes API visibility.
Adding support for RedHat signatures would be indicative of the intent of this project to work with emerging conventions and to track the evolving container image standards with the expectations that in time there will be a single industry standard for signatures, but in the interim different approcahes have benefits, and in RedHat signatures there is simplicity and transportability today.
The primary requirement would be to accept and check signatures stored using the registry extention API while noting that there are other ways of supplying this style of signature which are supported by this image library: https://github.com/containers/image

Raised https://github.com/containers/image/issues/807 (related to the value proposition of simple signing IMHO)


Fixed the num crate

Note that there's a rust-1.0 branch. The version on crates.io comes from that one.

@tomaka master still needs to be fixed. Merging myself since no one cares.


FancyZones not remembering settings on restart.

Microsoft PowerToys version
0.46.0
Running as admin

[X] Yes

Area(s) with issue?
FancyZones
Steps to reproduce
Set up fancyzone and then restart, the zones will be reset.
✔ Expected Behavior
Fancyzones to remember my settings
❌ Actual Behavior
Fancyzones resets to factory.
Other Software
No response

Duplicate of #13069

Thanks for reporting this. This issue is a duplicate. We'll track this in #13069 to centralize the discussion thread.


Window's alwaysUse24HourFormat not updates when settings change on iOS

Steps to reproduce:

Go to device settings, general, date & time and note the "24-Hour Time" setting

Run MediaQueryData.fromWindow(WidgetsBinding.instance.window).alwaysUse24HourFormat, the value should be correct.

Go again to device settings, general, date & time and switch the "24-Hour Time" setting

Switch back to your Flutter app

Run again MediaQueryData.fromWindow(WidgetsBinding.instance.window).alwaysUse24HourFormat, the value is incorrect, unchanged.

flutter doctor
Doctor summary (to see all details, run flutter doctor -v):
[✓] Flutter (Channel beta, 1.18.0-11.1.pre, on Mac OS X 10.14.6 18G4032, locale en-CA)
[✓] Android toolchain - develop for Android devices (Android SDK version 29.0.3)
[✓] Xcode - develop for iOS and macOS (Xcode 11.3)
[✓] Android Studio (version 3.5)
[✓] IntelliJ IDEA Ultimate Edition (version 2019.3.3)
[✓] VS Code (version 1.45.1)
• No issues found!

Hi @johngithubby
You need to restart to the app,  it works fine  beta and dev channel

Code Sample
import 'package:flutter/material.dart';

void main() => runApp(MyApp());

class MyApp extends StatelessWidget {
  @override
  Widget build(BuildContext context) {
    return MaterialApp(
      debugShowCheckedModeBanner: false,
      title: 'Material App',
      theme: ThemeData.dark(),
      home: Home(),
    );
  }
}

class Home extends StatelessWidget {
  @override
  Widget build(BuildContext context) {
    final value = MediaQueryData.fromWindow(WidgetsBinding.instance.window)
        .alwaysUse24HourFormat;

    return Scaffold(
      appBar: AppBar(
        title: Text('Material App Bar'),
      ),
      body: Center(
        child: Container(
          child: Column(
            mainAxisAlignment: MainAxisAlignment.spaceEvenly,
            children: [
              Text(
                'MediaQueryData.fromWindow(WidgetsBinding.instance.window).alwaysUse24HourFormat',
                textAlign: TextAlign.center,
              ),
              Text(value.toString()),
            ],
          ),
        ),
      ),
      floatingActionButton: FloatingActionButton(
        child: Icon(Icons.add),
        onPressed: () {},
      ),
    );
  }
}

flutter doctor -v
[✓] Flutter (Channel dev, 1.19.0-1.0.pre, on Mac OS X 10.15.4 19E287, locale
    en-GB)
    • Flutter version 1.19.0-1.0.pre at /Users/tahatesser/Code/flutter_dev
    • Framework revision 456d80b9dd (2 weeks ago), 2020-05-11 11:45:03 -0400
    • Engine revision d96f962ca2
    • Dart version 2.9.0 (build 2.9.0-7.0.dev 092ed38a87)

[✓] Android toolchain - develop for Android devices (Android SDK version 29.0.3)
    • Android SDK at /Users/tahatesser/Code/SDK
    • Platform android-29, build-tools 29.0.3
    • ANDROID_HOME = /Users/tahatesser/Code/SDK
    • Java binary at: /Applications/Android
      Studio.app/Contents/jre/jdk/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build
      1.8.0_212-release-1586-b4-5784211)
    • All Android licenses accepted.

[✓] Xcode - develop for iOS and macOS (Xcode 11.5)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Xcode 11.5, Build version 11E608c
    • CocoaPods version 1.9.1

[✓] Chrome - develop for the web
    • Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome

[✓] Android Studio (version 3.6)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin version 45.1.1
    • Dart plugin version 192.8052
    • Java version OpenJDK Runtime Environment (build
      1.8.0_212-release-1586-b4-5784211)

[✓] VS Code (version 1.45.1)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension version 3.10.2

[✓] Connected device (4 available)
    • Taha’s iPhone • 00008020-001059882212002E • ios            • iOS 13.5
    • macOS         • macOS                     • darwin-x64     • Mac OS X
      10.15.4 19E287
    • Web Server    • web-server                • web-javascript • Flutter Tools
    • Chrome        • chrome                    • web-javascript • Google Chrome
      81.0.4044.138

• No issues found!

That is unacceptable of course, you mean I should ask our users to restart our app in order to have the correct information?

Duplicate of https://github.com/flutter/flutter/issues/13703.


Fixed servlet 3.1 feaures

Description

Motivation and Context

Fixes #355
Types of changes

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

Checklist:

[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.
[ ] I have added tests to cover my changes.

:+1:


fix(react-native): arrows

Three distinct parts were making arrows not work in React Native
cc: @brycnguyen

Verified that it works! Thanks!


Application needs permission to access resources in your organization that only an admin can grant. Please ask an admin to grant permission to this app before you can use it

Followed all the steps as mentioned in doc, however I see an issue of admin consent although the storage account was created by me. Here is the error notification.
"Application needs permission to access resources in your organization that only an admin can grant. Please ask an admin to grant permission to this app before you can use it"

Is it required that the registered app would need Admin consent although the storage account is created by a user?
Should we always use the resource id as "https://storage.azure.com/"? Will it be changed ever since the id would be different for each storage account

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

ID: d02de0bf-09d7-a7fc-1b4a-213426e96b2d
Version Independent ID: 36f6c8fe-6bc1-6643-4b66-2c38bcd2e5f0
Content: Authenticate with Azure Active Directory to access blob and queue data from your applications
Content Source: articles/storage/common/storage-auth-aad-app.md
Service: storage
Sub-service: common
GitHub Login: @tamram
Microsoft Alias: tamram

@rajmusuku Can you please share the link to the document which you are referring so that we can assign it to the correct team.

https://docs.microsoft.com/en-us/azure/storage/common/storage-auth-aad-app

@rajmusuku Thank you for the details. We will review and provide an update as appropriate.

@rajmusuku , The easiest way to allow your service account to connect is to enable user access to Enterprise apps. From your Admin portal, go to Admin Centers > Azure AD > Users and Groups > User Settings then make sure "Users can consent to apps accessing company data on their behalf" is enabled.
If this option is set to yes, then non-admin users may register custom-developed applications for use within this directory. If this option is set to no, then only users with an administrator role may register these types of applications.

@rajmusuku, We will now proceed to close this thread. If there are further questions regarding this matter, please tag me in your reply. We will gladly continue the discussion and we will reopen the issue.

Hi @YASWANTHM-MSFT, I am having the same issue as @rajmusuku's and Admin Centers > Azure AD > Users and Groups > User Settings > "Users can consent to apps accessing company data on their behalf" is enabled

@YASWANTHM-MSFT I sorted this out with my org's admin, thanks!

Glad to know that your issue is resolved.

Any idea how to solve this?

@YASWANTHM-MSFT I sorted this out with my org's admin, thanks!

What did you do with your organisation admin to resolve this?

Can you please help me with the issue. Can I know what you have done with your Org's Admin to resolve this @YASWANTHM-MSFT

@Karthiktandra your Azure Active Directory Admin, must grant admin consent to your app for some azure api, using the button Grant admin consent for Application at the bottom of the API Permissions blade:

To address this part of the original question:
Should we always use the resource id as "https://storage.azure.com/"? Will it be changed ever since the id would be different for each storage account
You can use the storage account root URI as the resource ID. We've recently updated the documentation to add this information. For example, see Well-known values for authentication with Azure AD.

Hi @patriziobruno,
Thanks for the information, I will make the relevant changes.

I am getting the same error. I wanted to understand how can I find out the admin name for my org to login to azure active directory admin portal. This is bit urgent and help here will be highly appreciated.

Any updates here?

Hi @AbhaPatankar ,
Please check in Subscriptions- Access Control - Classic Admin.
List of admins will be visible there.
You can ask the global admin to grant the permission.

Hello Greg,
I am Abha, part of Microsoft’s Azure DevOps team. We use “Supa Tool” to create the work items by automatically reding the mailbox/ github source.
To do this, we use GraphMailSouce by creating an App. However, while running the tool, I got the following error:
“Application needs permission to access resources in your organization that only an admin can grant. Please ask an admin to grant permission to this app before you can use it”
Please can you grant permissions to the app so that I can start using it to import the work items.
Regards,
Abha
From: Karthiktandra<EMAIL_ADDRESS>Sent: Thursday, November 28, 2019 10:35 AM
To: MicrosoftDocs/azure-docs<EMAIL_ADDRESS>Cc: Abha Patankar<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Subject: Re: [MicrosoftDocs/azure-docs] Application needs permission to access resources in your organization that only an admin can grant. Please ask an admin to grant permission to this app before you can use it (#29197)
Hi @AbhaPatankarhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FAbhaPatankar&data=02|01|Abha.Patankar%40microsoft.com|28fb375b5c8740d0002008d773c07b2d|72f988bf86f141af91ab2d7cd011db47|1|0|637105142853080679&sdata=cle9Mrzjl%2BSK17hhDTJh1ImFASFM7wPjUMLdx3zGKXU%3D&reserved=0 ,
Please check in Subscriptions- Access Control - Classic Admin.
List of admins will be visible there.
You can ask the global admin to grant the permission.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FMicrosoftDocs%2Fazure-docs%2Fissues%2F29197%3Femail_source%3Dnotifications%26email_token%3DAHWLI7QEVZV2OMUE2D2EUKLQV5GOXA5CNFSM4HFCMXYKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEFLOMPI%23issuecomment-559343165&data=02|01|Abha.Patankar%40microsoft.com|28fb375b5c8740d0002008d773c07b2d|72f988bf86f141af91ab2d7cd011db47|1|0|637105142853080679&sdata=%2FxqXr3A9C%2Fs5c34BMEinc3W3A6zNXCJhnEHYn09ibW0%3D&reserved=0, or unsubscribehttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAHWLI7SBAPJT4XEKFRTR74DQV5GOXANCNFSM4HFCMXYA&data=02|01|Abha.Patankar%40microsoft.com|28fb375b5c8740d0002008d773c07b2d|72f988bf86f141af91ab2d7cd011db47|1|0|637105142853090678&sdata=1BzedKawmEaBKSAhaCeYCSLx5mXuGoReLtIZM0tjct0%3D&reserved=0.

Hi Greg,
Any update on this? It is a bit urgent for us as it is impacting ~40 people in our org.
Regards,
Abha
From: Abha Patankar
Sent: Thursday, November 28, 2019 11:01 AM
To: Greg Arnits<EMAIL_ADDRESS>Cc: Mention<EMAIL_ADDRESS>MicrosoftDocs/azure-docs<EMAIL_ADDRESS>MicrosoftDocs/azure-docs<EMAIL_ADDRESS>Sadagopan Rajaram<EMAIL_ADDRESS>Vijayaraghavan Vedantham<EMAIL_ADDRESS>Madhuri Vadali Vadali (TECH MAHINDRA LTD<EMAIL_ADDRESS>Subject: RE: [MicrosoftDocs/azure-docs] Application needs permission to access resources in your organization that only an admin can grant. Please ask an admin to grant permission to this app before you can use it (#29197)
Hello Greg,
I am Abha, part of Microsoft’s Azure DevOps team. We use “Supa Tool” to create the work items by automatically reding the mailbox/ github source.
To do this, we use GraphMailSouce by creating an App. However, while running the tool, I got the following error:
“Application needs permission to access resources in your organization that only an admin can grant. Please ask an admin to grant permission to this app before you can use it”
Please can you grant permissions to the app so that I can start using it to import the work items.
Regards,
Abha
From: Karthiktandra <notifications<EMAIL_ADDRESS>Sent: Thursday, November 28, 2019 10:35 AM
To: MicrosoftDocs/azure-docs <azure-docs<EMAIL_ADDRESS>Cc: Abha Patankar <Abha.Patankar<EMAIL_ADDRESS>Mention <mention<EMAIL_ADDRESS>Subject: Re: [MicrosoftDocs/azure-docs] Application needs permission to access resources in your organization that only an admin can grant. Please ask an admin to grant permission to this app before you can use it (#29197)
Hi @AbhaPatankarhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FAbhaPatankar&data=02|01|Abha.Patankar%40microsoft.com|28fb375b5c8740d0002008d773c07b2d|72f988bf86f141af91ab2d7cd011db47|1|0|637105142853080679&sdata=cle9Mrzjl%2BSK17hhDTJh1ImFASFM7wPjUMLdx3zGKXU%3D&reserved=0 ,
Please check in Subscriptions- Access Control - Classic Admin.
List of admins will be visible there.
You can ask the global admin to grant the permission.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FMicrosoftDocs%2Fazure-docs%2Fissues%2F29197%3Femail_source%3Dnotifications%26email_token%3DAHWLI7QEVZV2OMUE2D2EUKLQV5GOXA5CNFSM4HFCMXYKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEFLOMPI%23issuecomment-559343165&data=02|01|Abha.Patankar%40microsoft.com|28fb375b5c8740d0002008d773c07b2d|72f988bf86f141af91ab2d7cd011db47|1|0|637105142853080679&sdata=%2FxqXr3A9C%2Fs5c34BMEinc3W3A6zNXCJhnEHYn09ibW0%3D&reserved=0, or unsubscribehttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAHWLI7SBAPJT4XEKFRTR74DQV5GOXANCNFSM4HFCMXYA&data=02|01|Abha.Patankar%40microsoft.com|28fb375b5c8740d0002008d773c07b2d|72f988bf86f141af91ab2d7cd011db47|1|0|637105142853090678&sdata=1BzedKawmEaBKSAhaCeYCSLx5mXuGoReLtIZM0tjct0%3D&reserved=0.

was there a conclusion?

If, by any chance, you found this issue while configuring rclone to use your company's onedrive account. The answer you need, lies here.
You're welcome.

as @patriziobruno said, I needed the admin consent in my api permissions and it works perfectly on msdal lib to angular


can you add please ?

i need more language in my server.
so can you add support please ?
https://www.spigotmc.org/resources/advanced-multi-language.21338/

Please see here: https://github.com/lucko/LuckPerms/wiki/Locale-and-Translations


version control

is there any possibility there could be a system in which the user can select what version they wish to update/downgrade/install to?
an example of this would be the following:
curl -sSL<EMAIL_ADDRESS>| bash

similar to install dependencies with yarn or npm

Yes, this is actually planned 😄


Bug report form

Hello @s-omas! Thanks for opening this PR. We checked the lines you've touched for PEP 8 issues, and found:

In the file interactive/dashboard/forms/report_bug_form.py:

Line 3:1: E302 expected 2 blank lines, found 1
Line 4:80: E501 line too long (91 > 79 characters)
Line 5:80: E501 line too long (84 > 79 characters)
Line 6:1: W391 blank line at end of file

In the file interactive/dashboard/views.py:

Line 31:1: W293 blank line contains whitespace
Line 32:44: E225 missing whitespace around operator
Line 32:80: E501 line too long (120 > 79 characters)
Line 34:80: E501 line too long (93 > 79 characters)

In the file interactive/interactive/settings.py:

Line 138:45: W292 no newline at end of file


fix!: Don't provide defaults for public repositories

This PR drops the default values for the public_repository keys under remote_repository_config. This fixes the underlying issue with configuring custom remote repositories, but it will require the data structures for folks using the public repositories to be a bit more verbose, unfortunately.
closes #32

@prabhu34 - Got the CLA sorted out here. Would love to get this into the mainline so we don't have to maintain a fork. :)

Need a rebase now @frmrm.
@prabhu34 doesn't look like to be very active this days, maybe @apeabody could help us here?

(triggering the stuck lint workflow)

From the lint test:
Checking for documentation generation
diff -r '--exclude=.terraform' '--exclude=.kitchen' '--exclude=autogen' '--exclude=*.tfvars' '--exclude=*metadata.yaml' /workspace/README.md /tmp/tmp.aG4F7F4BFl/generate_docs/workspace/README.md
56c56
< | remote\_repository\_config | Configuration specific for a Remote Repository. |
object({
    description                 = optional(string)
    disable_upstream_validation = optional(bool, true)
    upstream_credentials = optional(object({
      username                = string
      password_secret_version = string
    }), null)
    apt_repository = optional(object({
      public_repository = optional(object({
        repository_base = string
        repository_path = string
      }), null)
    }), null)
    docker_repository = optional(object({
      public_repository = optional(string, "DOCKER_HUB")
      custom_repository = optional(object({
        uri = string
      }), null)
    }), null)
    maven_repository = optional(object({
      public_repository = optional(string, "MAVEN_CENTRAL")
      custom_repository = optional(object({
        uri = string
      }), null)
    }), null)
    npm_repository = optional(object({
      public_repository = optional(string, "NPMJS")
      custom_repository = optional(object({
        uri = string
      }), null)
    }), null)
    python_repository = optional(object({
      public_repository = optional(string, "PYPI")
      custom_repository = optional(object({
        uri = string
      }), null)
    }), null)
    yum_repository = optional(object({
      public_repository = optional(object({
        repository_base = string
        repository_path = string
      }), null)
    }), null)
  })
 | `null` | no |
---
> | remote\_repository\_config | Configuration specific for a Remote Repository. |
object({
    description                 = optional(string)
    disable_upstream_validation = optional(bool, true)
    upstream_credentials = optional(object({
      username                = string
      password_secret_version = string
    }), null)
    apt_repository = optional(object({
      public_repository = optional(object({
        repository_base = string
        repository_path = string
      }), null)
    }), null)
    docker_repository = optional(object({
      public_repository = optional(string)
      custom_repository = optional(object({
        uri = string
      }), null)
    }), null)
    maven_repository = optional(object({
      public_repository = optional(string)
      custom_repository = optional(object({
        uri = string
      }), null)
    }), null)
    npm_repository = optional(object({
      public_repository = optional(string)
      custom_repository = optional(object({
        uri = string
      }), null)
    }), null)
    python_repository = optional(object({
      public_repository = optional(string)
      custom_repository = optional(object({
        uri = string
      }), null)
    }), null)
    yum_repository = optional(object({
      public_repository = optional(object({
        repository_base = string
        repository_path = string
      }), null)
    }), null)
  })
 | `null` | no |
Error: Documentation generation has not been run, please run the
'make docker_generate_docs' command and commit the above changes.

@apeabody I have rebased on the latest main and run the doc generator. I'm honestly not sure where they stuck the checkbox to allow maintainers to send commits to my fork. 🧐

Do you want the upgrade guide additions as a part of the same commit or as a subsequent commit?

Do you want the upgrade guide additions as a part of the same commit or as a subsequent commit?

I'd prefer everything in this PR, so we keep the repo aligned.  Thanks!

@apeabody I have rebased on the latest main and run the doc generator. I'm honestly not sure where they stuck the checkbox to allow maintainers to send commits to my fork. 🧐

Here is the doc, but I believe some Orgs may restrict it: https://docs.github.com/en/pull-requests/collaborating-with-pull-requests/working-with-forks/allowing-changes-to-a-pull-request-branch-created-from-a-fork#enabling-repository-maintainer-permissions-on-existing-pull-requests

That's probably the issue - our enterprise may have it set at the enterprise level and our OSS forks org is subject to that.

@apeabody Updated. Let me know if that's what you're thinking or if further changes are needed.

/gcbrun

@apeabody Are you needing anything further from me on this?

@apeabody Are you needing anything further from me on this?

Thanks @frmrm! - Nothing at this time.
@prabhu34 - Can you please review this breaking change as well?


Make it easier to serve an app installed with uv, pipx, etc.

I'm usually not in an environment, where I have a proper terminal available. It would be nice, if I could open textual apps, installed via uv, pipx, rye, etc. on localhost and port forward via SSH to my local browser easily.
Right now I can do something like this:
uv run --with "harlequin[odbc, postgres]" --with "duckdb<1.1" --with "textual-serve" -- \
    python -c "from textual_serve.server import Server; Server('python -m harlequin',port=12345).serve()"

The command gets even longer if I want to provide arguments to the textual app. I could work around this by write a few bash functions, but it would be nice, if there was a way to maybe do something like
uv run --with "harlequin[odbc, postgres]" --with "duckdb<1.1" --with "textual-serve" -- \
   textual-serve --port=12345 --cmd 'harlequin path/to/my/duck.db'

or
uv run --with "harlequin[odbc, postgres]" --with "duckdb<1.1" --with "textual-serve" -- \
   textual-serve --port=12345 -- \
   harlequin path/to/my/duck.db

If you install textual-dev you can use textual serve. Does that help?

Oh, thanks! That's it exactly.
I can now just use an alias like this
alias harlequin_local='uvx --quiet --from textual-dev --with "harlequin[odbc, postgres]" --with "duckdb<1.1" -- \
   textual serve --port=12345 --command \
   harlequin'

and run it like this
harlequin_web path/to/my/duck.db

I cannot install it as a tool, but uv is fast enough, so it doesn't really matter.


[Card] Issues with Centering

I have run into a very frustrating issue when trying to center cards. You can reproduce with this code:

To my understanding, the centered attribute on the grid container should make the columns center their content. As you can see on the first screenshot, this is not the case:

The space on the left is smaller than the space on the right, which led me to believe that the columns were still left aligning their content. On closer inspection, though, it turns out that they actually do center it:

As you can see, the columns center their content. The issue seems to be that the card takes up the whole space, but inside the card itself, the content is placed on the left, leaving the remaining space on the right to stay empty. This is even more visible when you check the card div:

As you can see, the card div takes up the available space as if it was fluid, but it will not fill its entire space with the content, leaving the empty space and causing the columns to appear not centered.
I think it should be possible to center the content inside the orange card element. However, I have not yet found a way to do so. This leads to a very ugly experience with stackable grids on mobiles. Cards will be displayed one per line, on the very left side, with a lot of empty space on the right.

After understanding the issue better, I looked into the CSS and managed to fix my issue.
I changed this:
.ui.card {
  margin: 1em 0em;
}

To this:
.ui.card {
  margin: 1em auto;
}

I'm not a CSS expert, so I'm not sure if this won't break something in some browser or under some circumstances.

You can use a fluid card to expand the cards to the width of the columns.
Centered columns align center themselves amongst over columns but do not change column content positioning.

I'm aware of the fluid property. However, when using fluid cards, they turn out too big in certain configurations, i.e. on mobile. I don't want a card to take up the entire width on mobile when the cells are stacked. I also prefer having more space between cards in the widest configuration.
What if I want to center a card's contents in a grid cell without having it take up the whole cell? It does not seem possible at the moment. Why do card divs take up more space than they need for their contents? Why do they align their contents to the left of this space?
If you check out my screenshots, I don't think this particular case has to do with the column's content, but rather with the card's content. The column cells seem to be centered just fine. It's the inside of the card that seems to be off.

I'm adding a centered card variation. This should handle your case.
You have to keep in mind semantic is based of CSS layout, so we're limited to the kludgy system which these layout principles are inherited in css.

That seems sensible. I actually tried using it, but it didn't work and I obviously didn't find it in the variations of the card documentation either. Great to see it so quickly solved/added. Cheers.

Hi Im having this exact same issue , and I tried all of what you have posted here with no luck, Im curious if you ever found a solution


fix(engine): pause hardware API when engine is paused

Overview
This PR adds calls to hardware_api.pause and hardware_api.resume when engine.pause and engine.resume are called, respectively.
Closes #7923
Changelog

fix(engine): pause hardware API when engine is paused

Review requests
This PR needs to be tested on actual hardware rather than the simluator or emulator.
We need to check these acceptance criteria with both PAPIv2 protocols and JSONv6 protocols, because some functionality that was scoped specifically to PAPIv2 protocols has now moved into the ProtocolEngine proper.

[ ] When a protocol is paused, it can stop in the middle of a command

The command should remain in a running state
The run should shift to a paused state
Writing a protocol with big moves can make this easier to test and observe

[ ] When the protocol is resumed, the command picks up where it left off

Pause when door is opened
This ticket shouldn't affect "pause on door stop" behavior for PAPIv2 protocols. In advance of #9125, we should check that this PR does not negatively affect our expected PAPIv2 behavior. These tests are not expected to work on JSONv6 protocols until 9125 is completed.

Before the run is started

[ ] Door state does not affect the ability to run LPC (intent: setup) commands

Starting the run

[ ] Starting the run (issuing a play action) starts the run in a paused state
[ ] Trying to resume the run (issuing another play action) will reject if the door is still open
[ ] If the door is closed and then another play action is issued, the run starts

During the run

[ ] If the door is opened, the run becomes paused and the robot stops moving
[ ] Trying to resume the run (issuing another play action) will reject if the door is still open
[ ] If the door is closed and then another play action is issued, the run resumes

Risk assessment
I started today pretty nervous about this, but once I got past the surface and looked at the ProtocolEngine state behaviors we've already implemented around door state, I'm feeling pretty confident about this!

I tested 75cea85e on an OT-2 and found the door switch stuff to still be working with PAPIv2. I'm going to mark this as ready for review so pausing the hardware API in the engine natively can happen with #9125

When the run is on a protocol.pause() and is waiting for a resume, the commands list (and run log) will show the command following the pause command as current. I'm not sure why.

Expected behavior with PAPIv2, given how pauses work. The ctx.pause sets the HW pause flag and returns, allowing the protocol to enter the next ctx method (dispatching the message broker event) before the protocol hits a HW call that pauses motion.
This behavior is not the case with JSONv6 protocols, where waitForResume will hold in a running state until the protocol is resumed


improve test coverage to 100%

Codecov Report

Merging #5 (94c8285) into main (24fd991) will increase coverage by 21.80%.
The diff coverage is 100.00%.

@@             Coverage Diff              @@
##             main        #5       +/-   ##
============================================
+ Coverage   78.19%   100.00%   +21.80%
============================================
  Files           3         3
  Lines         188       185        -3
============================================
+ Hits          147       185       +38
+ Misses         41         0       -41

Impacted Files
Coverage Δ

chunksum/parser.py
100.00% <ø> (+4.54%)
:arrow_up:

chunksum/cdc.py
100.00% <100.00%> (+7.14%)
:arrow_up:

chunksum/chunksum.py
100.00% <100.00%> (+29.83%)
:arrow_up:

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


Fix typo in extension filename

Hello 😃
This is nothing urgent by my OCD is going nuts with this little typo.
Regards
FYI, this is my first commit to an open source project so I hope I did this right. If not feel free to show me the right way :)

Hey @gregoirefra. thanks for the PR!


Bytecode VM: Fix VariableGetInstruction deserialize

The old code resulted in the following disassembled instruction:
variable_get Natalie::Compiler::VariableGetInstruction

This fixes it to:
variable_get x

We can now run the following piece of code
puts ['a', '0'].map { |x| x.ord }

This one is also included as a unit test.

And implement variable_declare as well. This can now run:
a = 1
puts a

Which is included as a unit test as well.


[MODFQMMGR-87] Remove non-shared FQM code

Jira MODFQMMGR-87
This PR removes essentially the entire org.folio.fqm.lib package, with the exception of FqlValidationService, which is moved to org.folio.fql.lib.
As a bonus, it also pulls in Spring Boot (must also pull in a spring-boot-starter-*, otherwise spring itself doesn't come with), enabling autowiring/component declarations of FqlService and FqlValidationService (moved into org.folio.fql.service).

This required some additional configuration files to enable consuming modules to pick it up (see this SO post for why)

SHOULD BE MERGED BEFORE https://github.com/folio-org/mod-lists/pull/39 and https://github.com/folio-org/mod-fqm-manager/pull/77

I dunno about you, but I LOVE the feeling I get from deleting a lot of code from a project (but when I'm just moving it to another project, that gets balanced our by the corresponding bad feelings when I add a lot of code to the dest project 😆)


Review & document customisations to generated code for OBIE data model

We are using openapi-generator-maven-plugin to generate code for the OBIE read/write API from their OpenAPI definition files for particular schema versions.
When we wish to support a new version or update an existing version to the latest fix version (i.e. 3.1.10r5 => 3.1.10r6) then care needs to be taken to ensure that customisations and fixes applied to the generated code are retained after running the codegen. This is a manual process and it is easy to make mistakes.
Known customisations

Using joda API rather than native java DateTime (technically the codegen handles this via a config flag but it would be good to review whether there is any need to use joda anymore). Joda is officially deprecated. Note that from Java SE 8 onwards, users are asked to migrate to java.time (JSR-310) - a core part of the JDK which replaces this project. see here
OBDomesticVRPConsentRequestData.ReadRefundAccountEnum, codegen incorrectly changing the Yes/No enum values to true/false (seems to be a bug in the codegen)
Creating java enums for x-namespaced-enum schema types, see https://github.com/OpenBankingUK/read-write-api-specs/issues/13 and documentation here - these types may be extended by ASPSPs so I think treating them as strings is a better approach
Classes under: uk/org/openbanking/datamodel/common

Classes under: uk.org.openbanking.datamodel.error
VRP Links object vs common Links object - VRP seems to have diverged from the reuse pattern
Changing type of OBDomesticVRPInstruction.supplementaryData  from Object to OBSupplementaryData1
Cleaning up generated code: formatting, optimising imports and adding our license. Optimising the imports is especially important as the codegen can add unused imports for classes not on the classpath which will give compile errors

Any customisations that we decide to keep should be documented and scripted where possible. The current README.md contains some useful info but needs to be updated.
Breaking changes in 3.1.10 OBIE read/write spec, when upgraded to this version in the OBRI codebase (see issue:https://github.com/forgecloud/ob-deploy/issues/876), we noticed some breaking changes for backwards compat on VRP payments. This led to us placing breaking definition changes under a v3_1_10 subpackage. Review if we want to adopt this approach in SBAT
We should also look to upgrade to the latest codegen version, as we are using 5.1.0 and v6 is now available.

Point 2. appears to be a genuine bug in the codegen, treating "Yes", "No" values in the spec yaml as true/false booleans. It is rather odd that OBIE spec would define an enum for something which is essentially a boolean.
Either we need to edit the generated code to fix these enums or we can edit the spec yaml and quote the Yes/No values here.

RE: Point 8 - the unused import not on the classpath is: import org.openapitools.jackson.nullable.JsonNullable;
We can prevent the generator doing this by add the following configuration to the mvn plugin:
<configOptions>
    ...
    <openApiNullable>false</openApiNullable>
</configOptions>```


Array Buffer Allocation Failed

Hi, I appreciate your work here, though I am real bummed I have lost $100+ through OpenAi because this spent hours calling the API, but fails to combine at the end and I lose everything. I have used this on a Mac and PC with 64GB of RAM.
All works fine for small requests, but large requests for 100k, 400k, chars etc it fails at the end with nothing to show for it, and I've lost money. It seems to call the API right, but errors out when merging. It is a not a system memory issue.
And it seems none of the data was saved from the API calls so I can't even recover and manually merge the wav files. Can you store the data into a data store or something that can be recovered if this happens or have some safe fallback so it's not all wasted? Any way to make this work with very high char counts?
Thank you.
main.js:34 Uncaught (in promise) RangeError: Array buffer allocation failed     at new ArrayBuffer (<anonymous>)     at new Uint8Array (<anonymous>)     at concatenateWAVBuffers (main.js:34:32)     at mergeAudioBlobsAndDownload (main.js:18:29)

This just happened to me. Same exact error. Thankfully, I only lost $5, but it was still inconvenient after waiting for an hour. It also shows how little text is required to trigger this error. This was for about 300k characters.


Implement AccordionMenu

Thanks, I will reflect these into the code

I see that we already have a component called Accordion, but that one kinda looks like a FAQ entry to me 🤔 I'm tempted to suggest to rename the existing one to Faq or something, and then call this new one just Accordion. Do you have any thoughts? May also be good to discuss the naming of these components with Line as well

I think this should be discussed with Line. Should we have 2 different accordions? Are they used for the same purposes? Are they really just the same component on different backgrounds?

@fredrikloberg For the naming, you are right there is an overlap with Accordion component. Maybe we can call this a Details, DetailsSummary component. What do you think?

I see that we already have a component called Accordion, but that one kinda looks like a FAQ entry to me thinking I'm tempted to suggest to rename the existing one to Faq or something, and then call this new one just Accordion. Do you have any thoughts? May also be good to discuss the naming of these components with Line as well

I think this should be discussed with Line. Should we have 2 different accordions? Are they used for the same purposes? Are they really just the same component on different backgrounds?

I think they are the same. Initially, I tried to make use of Accordion and then it turned out I had to change some details in the code like the background color. Or the title was only expecting a string in the original Accordion and it was a problem for Line's sketch which showed icons next to titles. Then I wanted to re-do it with an influence from Accordion but I really like the <details><summary> usage in this version because it eliminates the need for message in the interface.

Could be good to have a quick chat with Line about this😊

I see that we already have a component called Accordion, but that one kinda looks like a FAQ entry to me thinking I'm tempted to suggest to rename the existing one to Faq or something, and then call this new one just Accordion. Do you have any thoughts? May also be good to discuss the naming of these components with Line as well

I think this should be discussed with Line. Should we have 2 different accordions? Are they used for the same purposes? Are they really just the same component on different backgrounds?

I think they are the same. Initially, I tried to make use of Accordion and then it turned out I had to change some details in the code like the background color. Or the title was only expecting a string in the original Accordion and it was a problem for Line's sketch which showed icons next to titles. Then I wanted to re-do it with an influence from Accordion but I really like the <details><summary> usage in this version because it eliminates the need for message in the interface.

I'm a little bit confused on how this accordion should work in other use cases. Could be good to have a quick chat with Line about this😊


Add Nodebalancers and Improve testing

corrects nodebalancer and volume methods
adds nodebalancer, nb config, and nb node create/update/delete mechanics
adds WaitForInstanceStatus
updates API_SUPPORT.md
Documents testing and fixture maintenance
moves tests/fixtures.yaml to fixtures/*.yaml (now used in Examples too)
adds better sanitizing of fixtures

🎉 🎈 thanks @displague


📊 Adding UN IGME dataset and long-run child mortality data

A combination of two child mortality sources to create a long-run indicator. I'm not sure I am handling the metadata correctly, the resulting variable at the end has two producers, but I can't see this reflected in the datapage preview.
Would it make more sense to manually curate the metadata for this indicator and condense the information into one producer?

There are several things that would be good to change with respect to metadata. If you need help, please let me know.

Thanks @pabloarosado! I think I have addressed your points now; thanks for taking the time to review.
I am still not certain it is working quite as expected e.g. the data page preview features mostly Gapminder, but I think that might be because the UN IGME data was imported with the old style metadata?


feat: secure api with token

Summary
Secure api with token
Key Points

[ ] This is a breaking change
[ ] New or existing documentation is updated

Description
The table auth_tokens stores API tokens which would be checked against for every request
Does this close any open issues?
https://github.com/merico-dev/lake/issues/551
Current Behavior
APIs could be accessed publicly
New Behavior
The API caller should set his/her token to the header X-Lake-Token. If the authentication fails, HTTP status code 401 would be returned from the server.
Screenshots
Include any relevant screenshots here.
Other Information
Any other information that is important to this PR.

@mindlesscloud can this be closed?


Getting error - java.lang.NoSuchFieldException: modifiers while running the scripts

QAF Version
ctsjv-qaf-automation
Steps To Reproduce
While running the scripts we are getting modifiers issue -
java.lang.NoSuchFieldException: modifiers
at java.base/java.lang.Class.getDeclaredField(Class.java:2610)
at com.qmetry.qaf.automation.util.ClassUtil.setField(ClassUtil.java:599)

The information provided here is not enough. Provide QAF version and execution environment details. You can find this details in report under environment-information. You can also provide details about test authoring used for instance, BDD or Java (Testng Test)
Looks like duplicate of #331

Please find more details below -
Regarding QAF -

4.0.0

com.wellsfargo.ctsjv
ctsjv-qaf-automation-parent
<IP_ADDRESS>
</parent>

We were using the same framework in WellsFargo (Oracle Java 8 used) and it was working fine now we shifted to CPU and we are getting issues here - (Now RedHat java version 11 and RedHat Java version 8 we are using)
If we are running with Java 11 -
We are getting below mentioned error
;
java.lang.NoSuchFieldException: modifiers
at java.base/java.lang.Class.getDeclaredField(Class.java:2610)
at com.qmetry.qaf.automation.util.ClassUtil.setField(ClassUtil.java:599)
If we change to Java 8 then we are getting complication issues -
java.lang.UnsupportedClassVersionError: com/cts/base/CSBaseTestCase has been compiled by a more recent version of the Java Runtime (class file version 55.0), this version of the Java Runtime only recognizes class file versions up to 52.0
Can you please help into this.

Please find the full pom.xml below -

4.0.0

com.wellsfargo.ctsjv
ctsjv-qaf-automation-parent
<IP_ADDRESS>
</parent>

<artifactId>SeleniumCTSGatewayAutomation</artifactId>
<packaging>jar</packaging>
<version>2.0.0-SNAPSHOT</version>

<!-- <artifactId>wf-eqe-automation-web</artifactId>
<packaging>jar</packaging>
<name>wf-eqe-automation-web</name>-->

<name>SeleniumCTSGatewayAutomation</name>

<properties>
	<jdk.source.version>1.8</jdk.source.version>
	<jdk.target.version>1.8</jdk.target.version>
	<eqe.version>3.3-SNAPSHOT</eqe.version>
</properties>
<build>
<plugins>
	<plugin>
		<groupId>org.apache.maven.plugins</groupId>
		<artifactId>maven-compiler-plugin</artifactId>
		<version>3.8.1</version>
		<configuration>
			<source>${jdk.source.version}</source>
			<target>${jdk.target.version}</target>
		</configuration>
	</plugin>
	<plugin>
		<groupId>org.sonarsource.scanner.maven</groupId>
		<artifactId>sonar-maven-plugin</artifactId>
		<version><IP_ADDRESS>5</version>
</plugin>
</plugins>
</build>

<dependencies>

	<dependency>
		<groupId>com.wellsfargo.ctsjv</groupId>
		<artifactId>ctsjv-qaf-automation</artifactId>
		<version>10.0.0-SNAPSHOT</version>
		<scope>compile</scope>
	</dependency>

	<!-- dependency for Acceptance BOT -->
	<dependency>
		<groupId>org.imgscalr</groupId>
		<artifactId>imgscalr-lib</artifactId>
		<version>4.2</version>
	</dependency>
	<dependency>
		<groupId>com.madgag</groupId>
		<artifactId>animated-gif-lib</artifactId>
		<version>1.2</version>
	</dependency>
	<dependency>
		<groupId>com.qmetry</groupId>
		<artifactId>qaf-execution-recorder</artifactId>
		<version>0.0.1</version>
		<scope>system</scope>
		<systemPath>${project.basedir}/lib/qaf-execution-recorder-0.0.1.jar</systemPath>
	</dependency>
	<!-- for run history listener -->
	<dependency>
		<groupId>commons-jxpath</groupId>
		<artifactId>commons-jxpath</artifactId>
		<version>1.3</version>
	</dependency>
 <dependency>
        <groupId>org.testng</groupId>
        <artifactId>testng</artifactId>
        <version>7.1.0</version>
        <scope>test</scope>
    </dependency>
</dependencies>

This is TestNG

@parulbangar16 QAF version and execution environment details are still missing. You can find this details in report under environment-information.
Regardless of that you should contact to internal team who is maintaining ctsjv-qaf-automation library.

@cjayswal  QAF report - dashboard.htm is not generating that is why we are not able to share the environment - information.

You should contact to internal team who is maintaining ctsjv-qaf-automation library for the issue. Report issue to internal team.


Make client's  hgd_check_svr_response() translate error mnemonics.

Ie, if server responds E_SHTDWN, also show "The server is going down".

I think Edd fixed this now.

done


release: add a small script to "automate" bugfix releases

Changes
Adding a bugfix-release.sh script to automate bugfix releases. Today
it is meant to be started by a maintainer, like the following
./tekton/bugfix-release upstream/release-v0.47.x

It will detect the old version, and commit related to the version
It will compute the new version number
It will detect the release name
If there is no different between the old version and the new
one (aka, no commit in the release branch since last release), it
will be exit gracefully.

The idea is that, from there, we can automate the bugfix release
completely. We could run this script weekly for each "supported"
release branch, and it would automatically do the release for us.
Signed-off-by: Vincent Demeester<EMAIL_ADDRESS>/kind misc

Submitter Checklist
As the author of this PR, please check off the items in this checklist:

[ ] Has Docs if any changes are user facing, including updates to minimum requirements e.g. Kubernetes version bumps
[ ] Has Tests included if any functionality added or changed
[x] pre-commit Passed
[x] Follows the commit message standard
[x] Meets the Tekton contributor standards (including functionality, content, code)
[x] Has a kind label. You can add one by adding a comment on this PR that contains /kind <type>. Valid types are bug, cleanup, design, documentation, feature, flake, misc, question, tep
[ ] Release notes block below has been updated with any user facing changes (API changes, bug fixes, changes requiring upgrade notices or deprecation warnings). See some examples of good release notes.
[ ] Release notes contains the string "action required" if the change requires additional action from users switching to the new release

Release Notes
NONE

cc @afrittoli @chitrangpatel @JeromeJu
"Perfect is the enemy of good" 😛 I wrote that script this afternoon to "ease my pain" of doing 4 bugfix releases — it's not perfect but it should be a good starting point.

/test check-pr-has-kind-label

/lgtm


Dockefile

Hello everyone
I find Dockware a super interesting project! Extremely cool!
For me it serves as a source of knowledge to learn and deepen the handling of Docker based configurations.
I'm not that familiar with Twig and Orca, unfortunately. How do you create a Dockerfile from the twigs? Is it possible to send me a rendered Dockerfile?
Just to give me food for some thoughts. ;-)
I would be interested in the following configuration:
version: "3"

services:

    shopware:
      image: dockware/play:latest
      container_name: shopware
      ports:
         - "80:80"
      networks:
         - web

    db:
      image: mysql:5.7
      container_name: mysql
      networks:
        - web
      environment:
        - MYSQL_ROOT_PASSWORD=hidden
        - MYSQL_USER=shopuser
        - MYSQL_PASSWORD=secret
        - MYSQL_DATABASE=shopware

networks:
  web:
    external: false

I would be happy if this could be done....
Carsten

Hi Carsten
thank you for your nice words
so yes, thats indeed possible, dockware can be mixed with any other image
...i guess you have that from our docs?
all you need to do in that config is to change the shopware db connection to the separate container,
and maybe import or install the database there
regarding twig and orca...its actually pretty easy.
if you clone the dockware repo, there is a makefile.
just use
make install
that fetches orca
and then...
make generate
and now you have all files in the "dist" folder

Hi Carsten
thank you for your nice words
so yes, thats indeed possible, dockware can be mixed with any other image
...i guess you have that from our docs?
all you need to do in that config is to change the shopware db connection to the separate container,
and maybe import or install the database there
regarding twig and orca...its actually pretty easy.
if you clone the dockware repo, there is a makefile.
just use
make install
that fetches orca
and then...
make generate
and now you have all files in the "dist" folder


MATIC (on Polygon) Price data is missing

Even though the price data is configured for some other tokens in Polygon, it is missing for the native coin MATIC.

Forgot to close previously.


Fix repeating message when user sends new message in chat on chat-page

fixed

still occurs when closing chat and opening again to send a new message


Buttons

Can you please, add buttons, to this?

Interactive buttons on the Discord precense?

I will be closing this ticket due to inactivity.


Change icon for links in left pannel and custom name displayed

Description
Just the possibility to change icon for each link and add a custom name displayed like it was in the old dashboard. It could help to the understanding.
Have you provided an initial effort estimate for this issue?
I have provided an initial effort estimate

Could be nice to add a field on page configuration under we add URL to add icon by name. Like it's used on switch widget.


Mastodon verification

Codecov Report
All modified lines are covered by tests :white_check_mark:

Comparison is base (2ad55fc) 100.00% compared to head (cb69175) 100.00%.

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@            Coverage Diff            @@
##              main        #9   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files            6         6
  Lines           55        55
=========================================
  Hits            55        55

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


this is the closest I've been able to get.  Sopmething is not joining…

… properly in the second join to authorships... so it is missing authorships of other users.  May need to link up another set of publications.

Hey @EricDurante .  I modified the method to use SQL statements instead.  I'm not too sure this is exactly what we would want in terms of refactoring, but will probably be better in terms of efficiency.  Only problem is I cannot get the second join to authorships to work correctly.  I want the joins to look something like this:
users > authorships > publications > authorships > waivers
   V
user_organization_memberships

The far right linking of authorships should give us the ability to query on whether or not another authorship (linked to one of the User's publications) already has a waiver or scholarsphere upload.  However, if you run the specs, you'll see they fail in both of these instances.  I've wracked my brains all day yesterday trying to figure it out, but with no success.  If you have the time, could you take a look?  I can revisit this tomorrow possibly, but need to focus my attention on something else today.


A fix to the use of  tags rather than  in the documentation comments

This PR fixes #1775

by fixing the code generator (commit a7c4f6db5dfbcaf50d4722272afab50c8aa45430), and
by fixing two non-generated source files (commit f0d1c08bc540428d50474df167c00de611bd245d).

The changes to the generated source files (i.e., those in generated/) are not included in this PR, because they will be re-generated by the updated code generator when building. (You will need to commit those changes to the repository by yourself...)
Note that, after applying this PR to your working tree, you may need to restart Windows and run dotnet clean, before running dotnet build.

I have read the Contribution License Agreement.
@dotnet-policy-service agree

Thanks @AlissaSabre. Can you commit the changes to the generated code? We persist those so we can see what the changes are - yeah, it makes for a larger change anytime you make a change there, but it's proven helpful to see the impact.

After applying this PR to your working tree, you may need to restart Windows and run dotnet clean, then remove all files in (subdirectories of) generated/ manually before running dotnet build.

What you're getting at here is probably that VS (or any long lived roslyn process) doesn't reload analyzers/generators/code fixers. I've never had to restart Windows - restarting VS is usually sufficient. Also, removing the generated/ directory shouldn't matter as it isn't used for compilation and will be overwritten (there is a ci step that verifies that the files there match what the compiler is emitting).


alphabetize rule doesn't support nested query objects

Issue workflow progress

Progress of the issue based on the Contributor Workflow

[ ] 1. The issue provides a reproduction available on GitHub, Stackblitz or CodeSandbox

Please make sure the graphql-eslint version under package.json matches yours.

[ ] 2. A failing test has been provided
[ ] 3. A local solution has been provided
[ ] 4. A pull request is pending review

Describe the bug
When applying role to queries that have nested query objects.
Error is Cannot read property '0' of undefined within a graphql document.
we should be able to apply this rule to complex queries.
To Reproduce
Steps to reproduce the behavior:
Try to lint a query with a nested query object eg.:
query {
  root_object {
     a
     b {
        subprop_a
        subprop_b
    }
}

Expected behavior
Linting analyses alphabetical order within every nested object.
Environment:

OS:
@graphql-eslint/eslint-plugin: 3.10.3
Node.js: 14.17.4

eslint setup:
overrides: [
        {
            files: ['src/graphql/**/*.ts'],
            processor: '@graphql-eslint/graphql',
        },
        {
            files: ['*.graphql'],
            parser: '@graphql-eslint/eslint-plugin',
            plugins: ['@graphql-eslint'],
            rules: {
                '@graphql-eslint/alphabetize': ['error', {
                    fields: [ 'InputObjectTypeDefinition','InterfaceTypeDefinition', 'ObjectTypeDefinition' ],
                    selections: ['FragmentDefinition', 'OperationDefinition'],
                    variables: ['OperationDefinition'],
                    arguments: ['FieldDefinition','Field','DirectiveDefinition','Directive'],
                }]
            },
            parserOptions: {
                operations: './src/graphql/**/*.ts',
                schema: './src/graphql/schema.json',
            },
        },
    ],

Your GraphQL document is invalid, you have a missing } bracket at the end, I can't reproduce your error even with a valid document. Consider submitting a PR with the failed test in this file https://github.com/B2o5T/graphql-eslint/blob/master/packages/plugin/tests/alphabetize.spec.ts if you really found a problem with this rule.


Try HTTP after HTTPS fails (with -f/--force-non-ssl)

Hi,
Does reg support connecting to registries over HTTP (not HTTPS)?
I have a docker registry available only over HTTP, and I haven't yet found a way to get reg to connect to my registry.
Example:
$ docker run \
    --rm \
    --net=host \
    -v $HOME/.docker/daemon.json:/root/.docker/daemon.json:ro \
    jess/reg:v0.15.8 \
    tags -f my-registry-here/my-image-here
Get https://my-registry-here/my-image-here/v2/: dial tcp my-ip-here:443: connect: connection refused

My $HOME/.docker/daemon.json contains:
{
  "insecure-registries" : [
    "my-registry-here"
  ]
}

...compared to docker pull my-registry-here/my-image-here:tag-here that works fine. 😊

have you tried the -f flag :)

Yes, this was using the -f flag.
Sorry for being unclear, I mentioned in the title, and in my example (as tags -f my-registry-here/my-image-here) but I didn't mention it in the description clearly enough. 😊
Seemed to me that the -f flag just stopped when it wasn't successful on HTTPS, instead of (as I had assumed) continuing with trying HTTP after failing with HTTPS — similar to what I suppose is happening with the regular docker client, as I'm able to do it there (when having set the insecure-registries config).

can you try with the latest release
On Wed, Nov 14, 2018 at 1:38 PM Victor Sollerhed<EMAIL_ADDRESS>wrote:

Yes, this was using the -f flag.
Sorry for being unclear, I mentioned in the title, and in my example (as tags
-f my-registry-here/my-image-here) but I didn't mention it in the
description clearly enough. 😊
Seemed to me that the -f flag just stopped when it wasn't successful on
HTTPS, instead of (as I had assumed) continuing with trying HTTP after
failing with HTTPS — similar to what I suppose is happening with the
regular docker client, as I'm able to do it there (when having set the
insecure-registries config).
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub
https://github.com/genuinetools/reg/issues/151#issuecomment-438771004,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABYNbJbxp1vO3616Qiuo30kNxPa6jZQoks5uvGMmgaJpZM4XSh24
.

--
Jessie Frazelle
4096R / D4C4 DD60 0D66 F65A 8EFC  511E 18F3 685C 0022 BFF3
pgp.mit.edu http://pgp.mit.edu/pks/lookup?op=get&search=0x18F3685C0022BFF3

Yes, works with latest. 👍

great thanks!
On Wed, Nov 14, 2018 at 1:48 PM Victor Sollerhed<EMAIL_ADDRESS>wrote:

Yes, works with latest. 👍
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub
https://github.com/genuinetools/reg/issues/151#issuecomment-438774570,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABYNbIIC40eJlsYv-ZE8MgyPOLlERU1Aks5uvGWOgaJpZM4XSh24
.

--
Jessie Frazelle
4096R / D4C4 DD60 0D66 F65A 8EFC  511E 18F3 685C 0022 BFF3
pgp.mit.edu http://pgp.mit.edu/pks/lookup?op=get&search=0x18F3685C0022BFF3

I couldn't find a newer version tagged on Docker Hub...
https://hub.docker.com/r/jess/reg/tags/
...but I saw that you had released v0.16.0, so I guess it's on its way. 😊
https://github.com/genuinetools/reg/releases/tag/v0.16.0


feat: upgrade to sdk 13 beta and compiler 7 (wip)

is this backwards compatible when an old compiler version is used? (which can be configured, right?)

@marc0olo haven't tried, but I wouldn't guess so, backwards/forwards compatibility is not really supported in aeproject anyways

waiting for a final sdk v13 release before being ready with this PR

@davidyuk @marc0olo this is now ready for review


Add tests for command splitting

Thanks @Makpoc! Great work - tests are thorough and nice job finding the weirdness in Windows.


Add  to CODE_TAGS

Summary
In order for <textarea> tags to be multiline, I believe they need to be included in CODE_TAGS.
Testing
I have done no thorough testing, but after playing around with it in my own framework, this seemed to be the solution.

Heya, thanks for the PR! Could you perhaps also include a test? I've kinda forgotten what those lines do, and having a regression test might mean this becomes a bit more solid going forward. Thanks a lot!

@yoshuawuyts I don't think there are any meaningful non-redundant tests to write here. The tests for the <pre> tag seem to cover the same logic (as is the case with the <code> tag).
https://github.com/shama/bel/blob/903bf5e4209beb3c33a6acb9f52c6ed533875831/test/elements.js#L143-L152

Thanks! This was merged in #108; sorry about the duplicate work :(

@goto-bus-stop no worries! Just happy the tag was added (although adding these hard coded tags feel really hacky)


Recalculate surface analyses on topography deletion

This is missing so far.

No, it is implemented. All surfaces analyses are deleted after topography deletion. So they are recomputed id requested again.
However, they could also be triggered automatically.

I've disabled automatic triggering of analysis throughout. Since we now have batch upload, if a user uploads 100 data files this would trigger 2000 Celery tasks, essentially blocking the worker machine for a long time. Running calculations on demand seems to be the simplest load-balancing strategy.


VPD Caching Issue with CVPD when SN/PN not programmed

On a recent system bringup, I was encountering several VPD errors related to CVPD. All the reads were resulting in errors stating the entire record/keyword was missing. This was a configuration wiith combined centaur + planar VPD on one chip. The issue was that when CVPD PNOR Caching was enabled, the code determined the cache was up to date and did not correctly populate the PNOR partition. I believe this was due to no PN/SN being programmed in the VPD at this point. I think we need to add an additional check to the logic determining whether to write the VPD cache or not taking into account an empty PN/SN scenario.

Adding @cvswen

Note - we should be getting a board this week so this scenario could be recreated/verified as well.

Code should be there.. just double check..

Currently if SEEPROM VPD part number and serial number are 0 and values in our VPD cache are 0 then we do not write the VPD cache.  Need to always write the VPD cache in this case.


feat: Add outputs for AWS resources created for karpetner and aws-node-termination-handler

What does this PR do?

Add outputs for AWS resources created for karpetner and aws-node-termination-handler

Motivation

These are necessary to pass to relevant resources, such as the Karpenter provisioner template

More

[ ] Yes, I have tested the PR using my local account setup (Provide any test evidence report under Additional Notes)
[x] Yes, I ran pre-commit run -a with this PR

For Moderators

[ ] E2E Test successfully complete before merge?

Additional Notes

can you pls fix the typos in the title and description so the changelog/release notes has those fixed?
thanks


Windows install error

❓ Questions and Help

What is your question?
Windows install fairseq failed,  ERROR: Failed building wheel for mecab-python3.So who can help me ,thanks a lot.
Code
Building wheels for collected packages: mecab-python3
Building wheel for mecab-python3 (setup.py) ... error
ERROR: Command errored out with exit status 1:
command: 'D:\softwares\anconda\envs\FairSeqTrain\python.exe' -u -c 'import sys, setuptools, tokenize; sys.argv[0] = '"'"'C:\Users\lhx18\AppData\Local\Temp\pip-install-8yh4itws\mecab-python3\setup.py'"'"'; file='"'"'C:\Users\lhx18\AppData\Local\Temp\pip-install-8yh4itws\mecab-python3\setup.py'"'"';f=getattr(tokenize, '"'"'open'"'"', open)(file);code=f.read().replace('"'"'\r\n'"'"', '"'"'\n'"'"');f.close();exec(compile(code, file, '"'"'exec'"'"'))' bdist_wheel -d 'C:\Users\lhx18\AppData\Local\Temp\pip-wheel-eohox3gr'
cwd: C:\Users\lhx18\AppData\Local\Temp\pip-install-8yh4itws\mecab-python3
Complete output (9 lines):
running bdist_wheel
running build
running build_py
creating build
creating build\lib.win-amd64-3.6
creating build\lib.win-amd64-3.6\MeCab
copying src\MeCab_init_.py -> build\lib.win-amd64-3.6\MeCab
running build_ext
error: [WinError 2] system can't find specific file。
ERROR: Failed building wheel for mecab-python3
Running setup.py clean for mecab-python3

What's your environment?
windows 10
ancoanda
python3.6.5
pip install -e .

Only sarcrebleu require install mecab-python3, which cause error when build in windows. Remove it from setup.py unblock me build this project.
Remove this line. https://github.com/pytorch/fairseq/blob/v0.9.0/setup.py#L141