Idea for a fix for the portals that are right up the wall..?

I haven't followed up with the recent videos and I don't know if you have mentioned anything about it, but whatever.
I don't remember if you can get the world entity the same way you can get the player entity, but if you can these tricks might work.
I looked at the glua wiki and found these functions:
https://wiki.facepunch.com/gmod/Entity:SetCustomCollisionCheck
https://wiki.facepunch.com/gmod/GM:ShouldCollide
You can maybe disable collisions for the player with everything in the map except for a certain prop that imitates the ground you'd be standing on.
Alternatively https://wiki.facepunch.com/gmod/Entity:SetCollisionGroup with COLLISION_GROUP_PASSABLE_DOOR could do the same effect maybe?

If I recall correctly, it's not entirely possible to disable collisions with the map without rewriting the movement system, which is what the Portal developers opted with. For instance, setting the map to COLLISION_GROUP_PASSABLE_DOOR doesn't change anything, although it does make it easier for you to get stuck in map geometry. When you die, your ragdoll can pass through it no problem. This is because the vanilla movement system is not entirely physics-based, and takes for granted that you will never move through map geometry. I believe that the HL2 devs have spoken on this somewhere, that their intent was to make the movement system 100% physics based, however it did not turn out well. One can only imagine...
So that leaves the option of rewriting the movement system, while theoretically possible, will take a lot of effort and probably produce a lot of conflicts. It can be alleviated by localizing it only to portal environments, but still. The crouching workaround is fine enough for now I think.

Or you could do what Mee just did in the latest commit. It's kind of a little bit of both. Clever or jank?

2 steps ahead of ya

Great then :)


Pause option

Idea:
Use tween.pause() to pause the motion and tween.resume() to continue.

Could also be nice to have a method for restarting the tween process, something like:

tween.restart()
tween.step(0) (or tween.jump(0) if going immediately to those values)

tried using a mousePressed to restart the motion.. but found one can only reset the added steps..

Hi @ffd8
thank you for your input. I added the restart-method with version 1.1.0 now. A step method is much harder, because all previous steps have to calculated or simulated. This could be a new issue 😊


[MNT] tensorflow-probability as a dependency

Describe the issue
tensorflow-probability as a dependency is currently only used in a single place in the forecasting package, to simulate a normal distribution. This seems rather unnecessary at first glance, and alternatives should be investigated.
Suggest a potential alternative/fix
Replace the current usage and tensorflow-probability as a dependency.
Additional context
No response

This is I think the only usage in aeon in forecasting base _predict_proba. Basically used to  create a normal distribution object. I cannot find anywhere this is used, so its purely a convenience: I cannot see there is any contract imposed on the pred_dist, but not sure. Needs investigating
        pred_var = self._predict_var(fh=fh, X=X)
        pred_std = np.sqrt(pred_var)
        pred_mean = self.predict(fh=fh, X=X)
        # ensure that pred_mean is a pd.DataFrame
        df_types = ["pd.DataFrame", "pd-multiindex", "pd_multiindex_hier"]
        pred_mean = convert_to(pred_mean, to_type=df_types)
        import tensorflow_probability as tfp
        d = tfp.distributions.Normal
        pred_dist = d(loc=pred_mean, scale=pred_std)

the docstring says this

scipy has a normal distribution
from scipy.stats import norm
# Create a normal distribution object
# mean (loc) = 0, standard deviation (scale) = 1
normal_distribution = norm(loc=0, scale=1)

so just swapped out this

I agree with this and I'm partially guilty for the tensorflow-probability dependency. The idea was to use it for all distributional forecasting, but in retrospect for the use-cases I've seen, scipy is more practical and of course more lightweight.


Remove Or Update Policies Timestamp

The Policies page (http://webguide.ucsb.edu/policies/) says that "This page was last reviewed by WSG members in September 2014.", but the page was recently updated.
Should we update this timestamp?
Is this a timestamp that should only be updated when the entire page is reviewed? If so, is there a process in place to make sure that occurs on a regular basis?
Or is this something we should just remove since it's a page that continues to evolve? (This is my suggestion but I would to love hear everyone else's opinions)

Hi @loganfranken
I agree with your suggestion to remove the timestamp.  If we do decide to keep it, we may want to consider adding a timestamp to other pages of the Webguide as well for consistency.


Add rule configuration parameters

Issue #, if available:
Fix #547
Description of changes:

Add a process to configure rules
Update rule E3012 to allow for non strict property checking

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Codecov Report

Merging #632 into master will decrease coverage by 0.49%.
The diff coverage is 39.39%.

@@            Coverage Diff            @@
##           master     #632     +/-   ##
=========================================
- Coverage   84.88%   84.38%   -0.5%
=========================================
  Files         113      113
  Lines        6470     6521     +51
  Branches     1513     1528     +15
=========================================
+ Hits         5492     5503     +11
- Misses        627      663     +36
- Partials      351      355      +4

Impacted Files
Coverage Δ

src/cfnlint/core.py
57.84% <100%> (ø)
:arrow_up:

src/cfnlint/config.py
77.77% <20%> (-5.19%)
:arrow_down:

...t/rules/resources/properties/ValuePrimitiveType.py
69.91% <26.66%> (-16.46%)
:arrow_down:

src/cfnlint/__init__.py
86.65% <81.81%> (-0.18%)
:arrow_down:

Continue to review full report at Codecov.

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

Sweet.... Don't forget to update the documentation with this. Pretty awesome "rules themself are now configurable" feature 👍


Updated lib versions

Updated:

Gradle wrapper: 4.6 -> 6.5
Gradle plugin: 3.2.1 -> 4.1.1
Target SDK version : 28 -> 30
Build tools version: 28.0.3 -> 29.0.3
App compat version: 1.0.0 -> 1.2.0
Java version : 1.7 -> 1.8

Other:

Separated AndroidX library versions. Appcompat and Exif tools can be updated separately.
Fixed issue in test/gradle file

@ravindu1024 let me know if you can merge! not sure if all settings are ok

@Canato nope. Can't merge. You can go ahead and merge it. :+1:


Support repeat/logic control for newcommand extension

I want to simulate some commands which supported by amsmath package, but without logic control, it won't be possible, so I wonder is there any way to do this, or will you add this feature?
Syntax might like this:
\newcommand\hdotsfor[1]{%
  \foreach \index in {1, ..., #1} {%
   \ifnum\index!=1{$}  \cdots %
  }}

Invoke \hdotsfor{3} will get:
$$
\cdots & \cdots & \cdots
$$

This is outside the scope of MathJax's mission.  MathJax doesn't implement the low-level looping and flow-control macros.  This is partly to avoid code bloat for control sequences that are going to be rarely used, but also because MathJax is written in javascript, and you can implement your own control sequences with javascript definitions.  How that is done depends on which version of MathJax you are using, but there are certainly examples in the issues here and in the MathJax user's forum.
If you are really keen on having such macros, you could write a TeX input jax extension that implements them.  That would be a nice project, and a useful contribution to MathJax.
Also, I think \foreach is a TikZ command, not a regular LaTeX command, but I'm not certain about that.  And is the dollar sign in your example supposed to be an ampersand?

Ok, I should have some time to do this after I completed docsify-latex repo.
Ignore the example, I just write it without thinking carefully. :)

I was looking at issues with the tag "good first contribution" and found this issue.
If the feature is outside the scope of the MathJax vision, how can this be
seen as a "good first contribution"? I thought "outside scope" meant that
a contribution in that direction is not likely to be accepted.

@soegaard, sorry for the confusion on this.  When I say that something is outside the scope of MathJax's mission, that means that either we don't have the resources to do it (either time or money), or that the design of MathJax precludes it.  For an example of the latter, we don't support TeX's concept of catcodes, and it would be very difficult to add that in without some major redesign.
The request here, however, could be done, but is not something we have the time or funding to do.  But someone interested in contributing to MathJax could write a third-party extension that implements it, and we would consider including that in a collection of contributed extensions.  That is why I listed it as a good first contribution.  Perhaps a better label for third-party contribution is needed.

Thanks for the clarification.


I can't use tasks list like github

I want to use tasks list like below:

[ ] first task
[ ] second task

I thought remark-react correspond to GFM. But, when I use tasks list, a normal list was output instead of a checkbox.

remark-react is safe by default, so you’ll need to specify a sanitation schema in configuration, or pass sanitize: false!

Thanks for telling me it, @wooorm ! I can use tasks list. I will continue to use remark-react from now on!

Perfect!

For anyone who comes across this problem in future, I documented my solution here: https://github.com/syntax-tree/hast-util-sanitize/pull/2#issuecomment-400043867


Issue deploying APIs revisions using ARM templates

Hi @ll,
There is an issue with APIM that we are experiencing for a time and that we tackling with a manual workaround.
We are performing deployments between instances using the azure APIM resource kit, extracting APIs configuration, and deploying them in the following system with the ARM templates generated with the extractor tool.
However, when started having APIs revisions we have realized there is a limitation, or we have not found the way in any of the available documentation, that only allow us deploying revisions in the target system into the active revision.
This means that for being able to deploy a new revision we need, first creating the revision in the target system, them make it current and only then deploying the new API revision configuration. The consequence of this is that while performing new API revisions deployments in the target system there is a period on which the API former active revision is not active and the new one is not yet updated, that can cause service disruptions. In addition, we do not have the possibility of publishing a new revision in inactive mode, which would be the best for avoiding this issue and to decide, after deployment of the new revision is completed, when to activate the new revision.
Many thanks

Seems the Yaml parser is not working properly. Fields regarding revisions are all null when debuging (conversion from json-string to object). As result the current revision in Apim will be updated.

Unfortunately, this is a problem with ARM deployments and not with the DevOps Resource Kit.  As such, we can't correct this behavior.


Fix signing script

Signing script fails sometimes because of bash handling of binary data in variables.

/lgtm


Max Callstack and Unsubscribe Issues with rxjs > beta 12

RxJS version:
greater than beta 12 (all release candidates)
Code to reproduce:
https://github.com/MadMub/rxjs-max-callstack
Expected behavior:
nodejs callstack should not blow up, all client connections close properly
Actual behavior:
the callstack is overloaded, with smaller numbers of connections not all events seem to fire? (see demo). In beta 12 I'm able to connect and drop over roughly 10x the connections.
Additional information:
The README on the repo provides context

If I had to guess, I'd say client.destroy() synchronously triggers another event, either error or close. I haven't run the demo yet, but that's my first inclination.
Also, if I were to implement something like this, I think I'd go more the route of wrapping the client in a single observable.
new Observable(observer => {
  const nextHandler = d => observer.next(d);
  const errorHandler = e => observer.error(e);
  const completeHandler = hasError => hasError ? observer.error(hasError) : observer.complete();

  client.addEventListener('data', nextHandler);
  client.addEventListener('error', errorHandler);
  client.addEventListener('close', completeHandler);

  return () => {
    client.removeEventListener('data', nextHandler);
    client.removeEventListener('error', errorHandler);
    client.removeEventListener('close', completeHandler);
    if (needDestroyed) { // TODO: logic for this decision here?
      client.destroy();
    }
  };
});

Just as a general reference. It will be more verbose, but it might be more correct for you, and should definitely perform better.

@blesh thanks for single observable idea, in practice I think that is a way better idea and I will switch to that. I'm still concerned by what's happening though so I'd like to ask a few questions:

you mentioned something about destroy potentially synchronously making close emit, if that were the case, is it bad that i don't see this issue on beta 12, should I be able to hit max callstack somewhat easily on > rc1? Is there special condiseration with events firing too close together?

I will need to confirm it, but I think when I close the dummy clients script the server isn't firing error ever, simply close. How come rxjs beta 12 can support unsubscribing 4k connections but > rc1 only a few hundred. My next step is to try and look into this issue myself , step through the commits made between beta 12 and rc1.

I love rxjs and its hard for me not to use it for event patterns (i'm hooked!) but my current use case does require pretty high performance. Thank you for the feedback hopefully I can contribute and fix this problem (or identify if its something erroneous on my end and usage of rxjs)

@MadMub if performance is a critical concern, I'd definitely look into my recommendation above.
I'll try to get a chance to look at this issue more in-depth soon.

ping @blesh , months later I upgraded to rxjs5.2  and started to bring rxjs back into my backend server. The server is a socket proxy server and I use an observable on an event emitter to perform logic when "timeout" is fired (a socket has no bytes received in a while).
When connecting 50 sockets and letting them sit idle for 30 seconds (the configured timeout), all 50 timeouts fire, however only 3-7 of the callbacks/observable chains rigged up to those events actually fire, the other ~40 leak. When I roll back to beta.12 (which at this point is quite out of date) the issue goes away. At an even bigger scale (more than 400 sockets), I think the leak leads to an unexpected crash (error message is Unsubscribe error, maxcallstack something...)
As I mentioned in my initial description something changed between beta.12 and rc1 around core subscribe/unsubscribe? At this point if I want to rely more on rxjs I need to try and hunt this down myself. I just figured I'd chime in and inform that the issue is still present.

@MadMub this issue has been fixed by merging #2238, but hasn't been published to npm yet. If you change the subscribe() calls in your demo repo to be subscribe({}), it should work (at least, it did for me).

Okay, I will wait till next release. Glad this one has been solved, wish I had chased it down more back in November thanks for the update!

I think this issue can be closed now via #2238 and other suggestions. Feel freely reopen if problem persists.


The lastest plugin on plugins.jetbrains.com is very old

Please package a new release and push it to plugins.jetbrains.com
the last update is from February.

@long1eu would you be able to do that? the new Gradle build options should help with that.

@noordawod can you provide instructions how to build and install this plug-in from the git repo?

git clone ...
cd flutter_i18n/
./gradlew buildPlugin -PideaVersionPrefix={prefix}
{prefix} is one of: 181, 182, 183, 191

Latest plugins are now on JetBrains.


feat: Add device group hierarchy support

This adds knowledge and handling of device group hierarchy to
pan-os-python.
Handling of this is moved to the opstate namespace since device
group hierarchy is not stored in the XPATH of PAN-OS.
Note that the "async" job to update a device group's parent seems
to finish instantaneously.  Most times in my testing the job has
already finished by the time I could query the job status with
pan-os-python.

This is a replacement for #300


[Question] RC React Native

Was wondering, do RC team maintain this app too. Repo for it?
https://play.google.com/store/apps/details?id=chat.rocket.reactnative&hl=en
I thought the focus would be on native this[Rocket.Chat.Android] app.
In that case would not it be better to focus on one app?

@mmarif4u https://github.com/RocketChat/Rocket.Chat.ReactNative is the repo of react native app.

@aniketsingh03 thanks for the link.
Still waiting for a proper answer to the many apps theory here.

@mmarif4u I understand the confusion with our apps. But let me try to make it more clear.

Our official apps, the ones that we recommend everybody fork if they want to customize and download to use, is our native apps. They're all called Rocket.Chat+ in Google Play and App Store.

We do have a Cordova version of the app in App Store and Google Play+. This version is deprecated and should not be used to fork and customize;

Recently we also started a project using ReactNative technology. This isn't suppose to be our official app, but since we're an open source company, we thought that creating an app with such framework could have some impact in the community. The idea behind using this technology is to foment and see how the community respond to the efforts on the app;


feat(url): support url to redirect or call API in background

This requires NotificationsPortlet 4.6.0

Okay, cleaned up those package.json files!


Create a Dashboard Module for Resources (with React)

Description of changes
Intermediary pull-request for the work on making Hobbies and Resources use ResponseiveImages. #2252

Adds 'active' field to resources.
Creates a few re-usable dashboard components for commonly used things in Dashboard, which can be reused for other modules in React.

I don't really intend to make a perfect implementation of this. Validation is almost non-existent at this point, and I don't know if things might end up a bit wonky in certain situations. I just want to get this working, and move on with my life :)
Screenshot(s) if appropriate
Looks like Dashboard :man_shrugging:

https://github.com/dotkom/onlineweb-frontend/wiki/(Opinionated)-Structuring-and-ordering-of-imports :)

dotkom/onlineweb-frontend/wiki/(Opinionated)-Structuring-and-ordering-of-imports :)

That is Onlineweb Frontend, not this repository :)
I have left the linting rules of Onlineweb4 as they were.


support for Scala 2.12

Enables cross-compilation against 2.11.8 and 2.12.0 with the same codebase.
At the moment, crossScalaVersions is still set to Seq(2.11.8), because we can't compile the readme Scalatex project under 2.12 (https://github.com/lihaoyi/Scalatex/issues/38), so our CI doesn't yet publish 2.12 builds automatically.

@dwijnand How do I conditionally disable lines 30 and 34 in https://github.com/xeno-by/scalameta/blob/ca99041624600c57136ec5caeb4ab64a96491fa4/project/build.scala when running under Scala 2.12? I tried to wrap these lines in if (scalaVersion.value.startsWith("2.11")), but that didn't help.

You need to use Def.taskDyn to avoid the side-effect:
http://www.scala-sbt.org/0.13/docs/Howto-Dynamic-Task.html

Here's a better link http://www.scala-sbt.org/0.13/docs/Tasks.html#Dynamic+Computations+with

@dwijnand It works, thank you! https://github.com/scalameta/scalameta/pull/554


[rank1]:[E ProcessGroupNCCL.cpp:523] [Rank 1] Watchdog caught collective operation timeout: WorkNCCL

Use colbert to index about 20GB data on 4xA800 GPUs, following errors raised:
Clustering 111099511 points in 128D to 524288 clusters, redo 1 times, 4 iterations
Preprocessing in 8.98 s
[rank1]:[E ProcessGroupNCCL.cpp:523] [Rank 1] Watchdog caught collective operation timeout: WorkNCCL(SeqNum=5, OpType=ALLREDUCE, NumelIn=1, NumelOut=1, Timeout(ms)=600000) ran for 600659 milliseconds before timing out.
[rank1]:[E ProcessGroupNCCL.cpp:537] Some NCCL operations have failed or timed out. Due to the asynchronous nature of CUDA kernels, subsequent GPU operations might run on corrupted/incomplete data.
[rank1]:[E ProcessGroupNCCL.cpp:543] To avoid data inconsistency, we are taking the entire process down.
[rank1]:[E ProcessGroupNCCL.cpp:1182] [Rank 1] NCCL watchdog thread terminated with exception: [Rank 1] Watchdog caught collective operation timeout: WorkNCCL(SeqNum=5, OpType=ALLREDUCE, NumelIn=1, NumelOut=1, Timeout(ms)=600000) ran for 6006
59 milliseconds before timing out.
Exception raised from checkTimeout at ../torch/csrc/distributed/c10d/ProcessGroupNCCL.cpp:525 (most recent call first):
frame #0: c10::Error::Error(c10::SourceLocation, std::string) + 0x57 (0x7f99d5781d87 in

solved by setting Timeout to 6000_000 in distributed.py

Hi I have the same bug, can you tell me how to fix the problem, I have no idea about your solution: setting Timeout to 6000_000 in distributed.py. Because I haven't found Timeout variables in distributed.py. Hope for your reply


[images] Remove animated GIF thumbnailing; long live WebM

Eh, let's just leave well enough alone.


[Feature][Code Safety] Provide a MakeVocabType generic?

I find it unfortunate that currently, there is no documentation regarding how to enforce different locales to be exhaustive.
https://next-international.vercel.app/docs/writing-locales
I think we should be able to ensure that every (or just some) locales exhaustively implement an another locale schema.
At the moment, I proceed like this:
export type MakeHomogeneousValuesObjType<Obj extends object, ObjValuesType> = {
  [K in keyof Obj]: Obj[K] extends object ? MakeHomogeneousValuesObjType<Obj[K], ObjValuesType> : ObjValuesType;
};

import DEFAULT_LANGUAGE_OBJ from '@/i18n/locales/fr';
export type VocabBase = typeof DEFAULT_LANGUAGE_OBJ;

type AllowedVocabObjValuesTypes = string;
type VocabObjValue = AllowedVocabObjValuesTypes;

export type VocabType = MakeHomogeneousValuesObjType<VocabBase, VocabObjValue>;

:arrow_up: :information_source: Here, the idea behind VocabObjValue actually echoes with LocaleData: https://github.com/QuiiBz/next-international/issues/221

// locales/fr.ts

export default {
  whatever: {
    a: "J'aime les gaufres",
    b: 'Et le chocolat'
  }
} as const;

// locales/en.ts

import { VocabType } from '@/types/i18n';

export default {
  whatever: {
    a: 'I like waffles',
    b: 'And chocolate'
  }
} satisfies VocabType; // * ... LOL, this "satisfies" operator is so cute!

So, MakeVocabType could be:
export type MakeVocabType<Locale extends object> = {
  [K in keyof Locale]: Locale[K] extends object ? MakeVocabType<Locale[K]> : LocaleData;
};

And used like:
// * ... import { MakeVocabType } from '...'

import DEFAULT_LANGUAGE_OBJ from '@/i18n/locales/fr';
type VocabBase = typeof DEFAULT_LANGUAGE_OBJ;

export type VocabType = MakeVocabType<VocabBase>;

Also, idk if MakeVocabType would be a good name for this.
I guess you would prefer to name this CreateVocabType, as the CreateParams type in international-types.

You might be able to use the defineLocale() helper (from createI18nClient), though it probably doesn't work well with nested-objects locales. It hasn't been touched and used since a long time, but the main idea is here.
We'll need to make it work with both dot-notation and object-notation locales, and add it to the documentation.


Using aws cp instead of aws sync

I am currently using this fork https://github.com/tpaschalis/s3-cp-action which essentially boils down to being able to cp a single file up to S3 instead of syncing a whole dir's contents.
It requires less permissions on the S3 side so you can use it in write-only mode which is quite nice as that ensures the build system will not be able to list or read or delete any other objects in the bucket.
Just thought it might be nice if this action allowed cp too optionally, then the fork wouldn't be needed and duplication is reduced.

Actually you can use https://github.com/aws-actions/configure-aws-credentials and do what ever you want with AWS


html textarea not binding to model

Hi
I am doing a form post and finding that the  control is not binding to my model.
I am wondering if I am doing something wrong.
My form looks like this:
`
            <label for="Comment">Comment</label>
            <textarea class="form-control" cols="30" rows="8" id="Comment" form="VisitorsLogForm"<EMAIL_ADDRESS>
Here is my post handler:
`Post["/sign-visitors-log"] = parameters =>
        {
            try
            {
                var model = new Models.FlickaVisitorsLog();
                this.BindTo(model);`

Now this works great for all the other controls in the form. But I cannot get the text area to bind. It comes through as null.  I have a workaround where I put another hidden input control next to the text area and populate that with jquery on the button click and use that value in the model.  But that's not pretty and I would like to solve this.
My model is thus:
`public class FlickaVisitorsLog
    {
        public int Id { get; set; }
        public string Name { get; set; }
        public string Country { get; set; }
        public string Email { get; set; }
        public string Comment { get; set; }
        public DateTime CreatedDate { get; set; }
  }`

Am I doing something silly?  Any ideas?
I am using Nancy 1.43 with VS Community 2015. ASP.NET hosted with Razor

can't seem to get the MD formatting to work at all. Will start again.


Component.write_gds_with_metadata

What happened to this function?  I wrote a script a few months ago using function and now it doesn't work.
Does gdsfactory have a policy about not removing or changing things, namely keeping it backwards compatible?  It will be difficult to build projects if the core is changing.
My suggestion would be to put a label on a function "deprecated", and provide a date for when you plan to remove it from the code.  A window of 2 years would be reasonable, I think.
thank you
Lukas

Hi Lukas,
we deprecated the function, and removed after a few months,
you are right, we should solidify the code
you can use Component.write_gds(with_metadata=True)


About Anaconda.Miniconda3

Miniconda's every release is divided into three sub-releases (can't get a better word to describe this):

Python 3.7
Python 3.8
Python 3.9

Currently, the repository has one version of Python 3.7 and two of Python 3.9 which is dirty.

So, the current package in the repo needs to be purged and we should make a fresh package under the following Package Identifiers:

Anaconda.Miniconda3.Python3.7
Anaconda.Miniconda3.Python3.8
Anaconda.Miniconda3.Python3.9

cc @jedieaston @OfficialEsco @itzlevvie
@denelon please assign this issue to me.

I'm currently looking to deploy miniconda using winget and I'm running into this confusion, especially around upgrades. Is the current "advice" still to split this package into 3? If so, I'm happy to take a stab at it.

@vedantmgoyal9 - Can this be closed?


Issue with k8s.io/docs/tasks/administer-cluster/change-pv-reclaim-policy/

This is a Bug Report

Problem: It would be better to start with create a PersistentVolume first because some clusters don't have pv exist.
Proposed Solution:
# Create a PersistentVolume
$ kubectl apply -f https://k8s.io/examples/pods/storage/pv-volume.yaml
    | persistentvolume/task-pv-volume created
# Changing the reclaim policy of a PersistentVolume
$ kubectl get pv
    | NAME             CAPACITY   ACCESS MODES   RECLAIM POLICY   STATUS      CLAIM   STORAGECLASS   REASON   AGE
    | task-pv-volume   10Gi       RWO            Retain           Available           manual                  6s
$ kubectl patch pv task-pv-volume -p '{"spec":{"persistentVolumeReclaimPolicy":"Delete"}}'
    | persistentvolume/task-pv-volume patched
$ kubectl get pv
    | NAME             CAPACITY   ACCESS MODES   RECLAIM POLICY   STATUS      CLAIM   STORAGECLASS   REASON   AGE
    | task-pv-volume   10Gi       RWO            Delete           Available           manual                  2m29s

Page to Update:
https://kubernetes.io/docs/tasks/administer-cluster/change-pv-reclaim-policy/#changing-the-reclaim-policy-of-a-persistentvolume
Kubernetes Version: 1.15.0
$ kubectl version
Client Version: version.Info{Major:"1", Minor:"15", GitVersion:"v1.15.0", GitCommit:"e8462b5b5dc2584fdcd18e6bcfe9f1e4d970a529", GitTreeState:"clean", BuildDate:"2019-06-19T16:40:16Z", GoVersion:"go1.12.5", Compiler:"gc", Platform:"linux/amd64"}
Server Version: version.Info{Major:"1", Minor:"15", GitVersion:"v1.15.0", GitCommit:"e8462b5b5dc2584fdcd18e6bcfe9f1e4d970a529", GitTreeState:"clean", BuildDate:"2019-06-19T16:32:14Z", GoVersion:"go1.12.5", Compiler:"gc", Platform:"linux/amd64"}

As the title suggests, you should have a persistentvolumeclaim already created. At the bottom of the page, there is a link to persistentvolumeclaims that explains how to create them:
https://kubernetes.io/docs/concepts/storage/persistent-volumes/#persistentvolumeclaims
/close


[BREAKING CHANGES] Clean up adjacency and related methods

As we discussed, I moved _incidence_to_adjacency to the utility package. I also cleaned up duplicate adjacency methods in CellComplex, resulting in breaking changes.

I don't think removing s breaks anything as it didn't work before
However, removing node_ajacency_matrix and cell_adjacency_matrix will inevitably break something (they could be re-added as proxy methods for adjacency_matrix if you prefer)
Also, we could think about changing the behavior of adjacency_matrix: Currently, the default is to use the lower adjacency. For rank 0, this does not really make sense. I'm not sure what I prefer:

changing it to default to upper for rank 0, but lower for other ranks is the way it is used most often, but then the default is not consistent over all ranks
changing it to always default to upper or lower would be consistent, but then it would need to be specified more often.

@ffl096 I think you're right, thanks. I'll update the code accordingly.

@ffl096 I think you're right, thanks. I'll update the code accordingly.

Thanks! A few questions @josefhoppe

Why: "As we discussed, I moved _incidence_to_adjacency to the utility package."?
I think using the lower adjacency as a default is intuitive, because "adjacency" in graphs means lower adjacency.

I'm puzzled with flake8-docstrings is not picking up on the docstring formatting: docstrings should start with verb in imperative mode, which is not respected in one of your test files.

Why: "As we discussed, I moved _incidence_to_adjacency to the utility package."?

Maybe I misunderstood you. I included it in #40 as it existed in mostly the same form in both CombinatorialComplex and CellComplex and you said you generally agreed with de-duplicating code (now that I think of it, that could also be a reference to other de-duplications).
Either way, I still think that it makes sense to move it to the utility package, but if you disagree we can also revert that change.

I think using the lower adjacency as a default is intuitive, because "adjacency" in graphs means lower adjacency.

Does it? Nodes (0-cells) are upper adjacent to each other via edges (1-cells), aren't they?
Regarding the doc strings: Thanks for catching that, I'll fix the ones you pointed out.

the duplication issue between many complexes is indeed a problem. NetworkX does not attach the adjacency matrix as a graph method but rather as a part of the linear algebra module :
https://github.com/networkx/networkx/blob/main/networkx/linalg/graphmatrix.py
same for the incidence matrix. I am not sure we can do that for all complexes though since the parameters might differ (CC for instance takes two arguments).

lower_adjacency_matrix and upper_adjacency_matrix are the same thing as adjacency matrix and coadjacency matrix. Reason why I kept adjacency and coadjacency matrix is to be consist with graphs when the rank is zero. I am not sure if our output convention matches that of networkx's adjacnecy matrix but preferably so.


Add functional tests package

Description
This pull request creates the functional-tests package for managing all the test that target services. We modified the NPM script that runs the functional tests, and modified the build pipeline with these changes.
Specific Changes

Move botframework-connector/auth.test.js to functional-test folder.
Rename the tests folder inside functional-tests to botframework.
Modify the NPM script that runs the functional tests.
Remove hardcoded appId and appPassword from code and configure them as Azure Pipelines variables.

Configuration
These changes require some small new configuration in the Azure Pipeline build.
Azure Pipelines uses the build pipelines variables as environment variables for the project if the name matches. This does not work with encrypted variables, so in order to use them encrypted we need to configure a Powershell Task to assign them.
The following guides show how to configure it using either the encrypted password or the decrypted one.
With AppSecret encrypted
Add two variables called authAppId and authAppPasswordEncripted, and set their corresponding value.

Add a PowerShell script and configure it as follows:

With AppSecret unencrypted
Add two variables named authAppId and authAppPassword, and assign their corresponding value.

Testing

@cleemullins The changes were applied to the Pipelines configuration. This is ready to be merged.

@paradoxARG - Can you resolve the conflicts so we can take this?

Sure! We'll look into this.

@cleemullins Conflicts solved.

Pull Request Test Coverage Report for Build 71982

0 of 0   changed or added relevant lines in 0 files are covered.
131 unchanged lines in 11 files lost coverage.
Overall coverage decreased (-3.8%) to 85.322%

Files with Coverage Reduction
New Missed Lines
%

libraries/botframework-connector/src/auth/credentialProvider.ts
1
50.0%

libraries/botframework-connector/src/auth/claimsIdentity.ts
2
42.86%

libraries/botframework-connector/src/auth/endorsementsValidator.ts
5
8.33%

libraries/botframework-connector/src/auth/microsoftAppCredentials.ts
5
68.18%

libraries/botframework-connector/src/auth/jwtTokenValidation.ts
7
39.34%

libraries/botframework-connector/src/auth/channelValidation.ts
8
23.08%

libraries/botframework-connector/src/auth/governmentChannelValidation.ts
9
23.81%

libraries/botframework-connector/src/auth/enterpriseChannelValidation.ts
10
23.81%

libraries/botframework-connector/src/auth/openIdMetadata.ts
23
6.52%

libraries/botframework-connector/src/auth/emulatorValidation.ts
27
12.22%

Totals

Change from base Build 71693:
-3.8%

Covered Lines:
3348

Relevant Lines:
3764

💛  - Coveralls

Closing out out of date due to current Functional Test Plan


[Typing Indicator] Issue: Doesn't display on guilds

Before you submit an issue please restart your discord. If the issue is still there feel free to write a bug report.
Describe the bug
When someone types in a server chat it shows next to the channel like it should but it doesn't show up on the guild icons like it did before the newest discord update
To Reproduce
Steps to reproduce the behavior:

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

Expected behavior
A clear and concise description of what you expected to happen.
Screenshots
If applicable, add screenshots to help explain your problem.
Please also check the console by pressing Ctrl + Shift + I and send screenshots of errors (red messages)
Information (please complete the following information):

OS: Windows 10
Plugin Version: 0.2.1
Discord version: Stable
Compact Mode enabled? no

Additional context
Add any other context about the problem here.

fixed in 528f88b


Landing page for batlit.org

Move the Intro section up on the page
Add bat pictures / images -- make more user friendly from the start

Landing page has been created https://batlit.org


Modify MiqExpression Contains to work for Rails 5.2

Alternative to #20051 Addresses #20032
We need the sql for:
MiqExpression.new(
  "CONTAINS" => {"field" => "Vm.guest_applications-name", "value" => "foo"}
)

Rails suggests has a suggestion to solve this:
contains = Vm.joins(:guest_applications).where(:guest_applications=>{:name=>'foo'})
Vm.where(:id => contains)

There ends up being an extra join in there to vms, but it is using the rails framework instead of fighting it is a big win. We also have bigger performance battles to fight.
Before
"vms"."id" IN (
  SELECT DISTINCT "guest_applications"."vm_or_template_id"
  FROM "guest_applications"
  WHERE "guest_applications"."name" = 'foo'
)

fails for rails 5.2
After
"vms"."id" IN (
  SELECT "vms"."id"
  FROM "vms"
  INNER JOIN "guest_applications"
  ON "guest_applications"."vm_or_template_id" = "vms"."id"
  WHERE "guest_applications"."name" = 'foo'
)

Fewer lines and works with rails 5.2

"vms"."id" IN (
  SELECT DISTINCT "guest_applications"."vm_or_template_id"
  FROM "guest_applications"
  WHERE "guest_applications"."name" = 'foo'
)

fails for rails 5.2

In what way?  It looks like valid SQL to me, so I assume it is a generation issue that we are fixing, correct?
(I may have never understood what your were solving for Rails 5.2... only that you were solving "something"...)

fixed 2 (minor) cops.

Merging since this passes against 5.1 on master and we've tested it with cross repo on 5.2 in the past.  We will address the points raised here in the 5.2 PR: https://github.com/ManageIQ/manageiq/pull/20076

Jansa backport details:
$ git log -1
commit 240b9380bf8945e7504caa4de0873028cb3b5846
Author: Joe Rafaniello<EMAIL_ADDRESS>Date:   Wed Apr 29 17:03:10 2020 -0400

    Merge pull request #20110 from kbrock/miq_expression_contains_3

    Modify MiqExpression Contains to work for Rails 5.2

    (cherry picked from commit cff8306c9fbcd68b4b6285cc537fdf304093a3a8)


Updated New-CMTaskSequence Link

Link for New-CMTaskSequence referenced Get-CMTaskSequence instead.
Looked like a typo.

@jacorbello : Thanks for your contribution! The author, @mumian, has been notified to review your proposed change.

Docs Build status updates of commit dd716a7:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

sccm-ps/ConfigurationManager/New-CMTaskSequence.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.

Thanks @jacorbello for catching these issues and contributing the changes!
#sign-off
FYI, there is a publishing freeze this week in advance of the Build conference next week. Your changes should be merged and published on (or soon after) Monday, 6 May. No further action required of you.

@aczechowski: I'm sorry - only the author of this article, @mumian, can sign off on your changes. But we do have an exception process - if you are on the Microsoft content or product team for this product area, you can ask the PR review team to review and merge it by sending mail to the techdocprs alias.


Multiple authorization support not being recognized from CLI

Describe the bug
When trying to set up a graphql api that allows a model to be publicly accessible with an API key but also have @auth restrictions with cognito user pools, it only respects the first/default option selected. In the backend-config.json this is the result of running the steps below:
...
              "authConfig": {
					"additionalAuthenticationProviders": [],
					"defaultAuthentication": {
						"authenticationType": "API_KEY",
						"apiKeyConfig": {
							"description": "public api",
							"apiKeyExpirationDays": 7
						}
					}
				}
...

Manually updating App Sync from the console to have the additional authorization providers only partially works, because you still wont be able to update your schema to have the public and owner rules in the @auth directive. The CLI will throw @auth directive with 'userPools' provider found, but the project has no Cognito User Pools authentication provider configured.
Amplify CLI Version
4.13.4
To Reproduce
from the cli:
? Please select from one of the below mentioned services: GraphQL
? Choose the default authorization type for the API API key
? Enter a description for the API key: public api
? After how many days from now the API key should expire (1-365): 7
? Do you want to configure advanced settings for the GraphQL API Yes, I want to make some additional changes.
? Configure additional auth types? Yes
? Choose the additional authorization types you want to configure for the API
(chose cognito user pool)
? Configure conflict detection? No
Expected behavior
I would expect the CLI to be able to add the secondary auth provider to the API and allow for multiple auth rules.
Desktop (please complete the following information):

OS: Mac
Node Version: v12.6

From blog posts and docs, i have found a couple resources explaining this functionality:

https://dev.to/dabit3/12-new-features-of-aws-amplify-4i8h
https://aws-amplify.github.io/docs/cli-toolchain/graphql#allowed-authorization-mode-vs-provider-combinations
https://dev.to/robertbroersma/querying-owned-posts-vs-all-posts-in-aws-amplify-35jl

Hi @RussMax783 in order to add the secondary auth provider you would need to go through the update api flow, running amplify update api. Please feel free to comment if you have further questions

@nikhname thanks for the response. Even when going through the update it does not add the secondary. I've tried re-installing the CLI, installing previous versions of the CLI, and started up a project from scratch. In the backend-config.json, it doesn't add anything to the additionalAuthenticationProviders field. Even if I manually put some values in there, no change is detected when i run amplify status.

Same thing here. After updating api with adding api key, amplify status reports No Change

@kjetilge @RussMax783 If you install the latest version of the CLI - you can use amplify push --force to push changes for just your auth settings to the cloud.

I updated my backend-config file and tried doing a force push. In the terminal it gives me a GraphQL API KEY, but things are still in a bad state. I can't add @auth directive to my models to allow for public access as it still give me an error when i try and update. After trying to run an amplify command the backend-config file went back to only having the one defaultAuthentication and removed the api key setting I added in additionalAuthenticationProviders


.Net: Bug: try to run HomeAutomation

Describe the bug
I'm trying to use the demo code https://github.com/microsoft/semantic-kernel/tree/main/dotnet/samples/Demos/HomeAutomation
When I try to HomeAutomation through dotnet build and dotnet run It throws an issue Microsoft.Extensions.Options.OptionsValidationException: DataAnnotation validation failed for 'AzureOpenAIOptions' members: 'ChatDeploymentName' with the error: 'The ChatDeploymentName field is required.'.; DataAnnotation validation failed for 'AzureOpenAIOptions' members: 'Endpoint' with the error: 'The Endpoint field is required.'.; DataAnnotation validation failed for 'AzureOpenAIOptions' members: 'ApiKey' with the error: 'The ApiKey field is required.'.
Even though we have updated the appsetting.json  file

@shashankms9 - make sure you setup keys, it looks like these are missing which is why you are getting the error.

@evchaki, I have tried both approach methods mentioned in the readme.md file still throwing the same error

@markwallace-microsoft  can you take a look?

@markwallace-microsoft can you please help in resolving this issue?

Hi @evchaki, the issue has been resolved after updating

The appsetting.json file with AzureOpenAIOption it was AzureOpenAI as in AzureOpenAIOptions.cs file provided class name as AzureOpenAIOptions

Updated the Program.cs files to use Azure OpenAI option and IOption to AzureOpenAIOption from AzureOpenAI

After updating this I was able to run the application without any issue

@Niladri24dutta go ahead, I would like to take a look on the PR, and suggest or approve it. THanks.


feat(input): add (limited) support for text masking

Bug, feature request, or proposal:
How can I set a text mask in date picker component so, if user doesn't want to select a date in calendar, he can type and fill correctly the field?
What is the expected behavior?
When click on input date picker, show the mask so user can type date instead of selecting a date from calendar
What is the current behavior?
empty input
What are the steps to reproduce?
Providing a StackBlitz/Plunker (or similar) is the best way to get the team to see your issue.
Plunker starter (using on @master): https://goo.gl/DlHd6U
StackBlitz starter (using latest npm release): https://goo.gl/wwnhMV
Just add a datepicker component
What is the use-case or motivation for changing an existing behavior?
Maybe user wants to type date instead of selecting form calendar.
Which versions of Angular, Material, OS, TypeScript, browsers are affected?
@angular/cli: 1.3.0
node: 6.10.3
os: darwin x64
@angular/animations: 4.3.5
@angular/cdk: 2.0.0-beta.8
@angular/common: 4.3.5
@angular/compiler: 4.3.5
@angular/core: 4.3.5
@angular/flex-layout: 2.0.0-beta.8
@angular/forms: 4.3.5
@angular/http: 4.3.5
@angular/material: 2.0.0-beta.8
@angular/platform-browser: 4.3.5
@angular/platform-browser-dynamic: 4.3.5
@angular/router: 4.3.5
@angular/cli: 1.3.0
@angular/compiler-cli: 4.3.5
@angular/language-service: 4.3.5
Is there anything else we should know?

a) material doesn't provide mask behavior
b) it's currently impossible to open datepicker with focus, while keeping focus on the input (because of how Overlay works)

Hey @fxck ! Thanks for the answer!
Well... there's no need to keep focus on text input wile calendar is open! If user click on input, just show the mask. If user clicks on button, hide mask and show calendar! Is that possible?

Yes, but as I said, material doesn't provide that functionality, you can use something like https://github.com/text-mask/text-mask

https://github.com/text-mask/text-mask dont work with ngmodel

As mentioned above, text-mask won't work with material2. This is because agnular doesn't allow input fields to have more than 1 ValueAcessor.
A work around is to use the vanilla version of text-mask:
1 - Import the library in your component
import * as textMask from "vanilla-text-mask/dist/vanillaTextMask.js";
2 - Add your mask in the component and init your input (don't forget to import AfterViewInit and implement it to your component)
`
mask = [ /[1-9]/, /\d/, /\d/];
maskedInputController;
@ViewChild("input", { read: ViewContainerRef }) public input;
ngAfterViewInit(): void {
setTimeout(() => {
this.maskedInputController = textMask.maskInput({
inputElement: this.input.element.nativeElement,
mask: this.mask
});
});
}
ngOnDestroy() {
this.maskedInputController.destroy();
}
3 - in your template, make sure the input element has an #input attribute
<input matInput [matDatepicker]="picker" [(ngModel)]="selectedValue" #input>
<mat-datepicker-toggle matSuffix [for]="picker">
<mat-datepicker #picker>

`

I @remborg ! I was trying to set a mask like 'dd/mm/yyyy' and validate things like day 32 or month 13. I can't figure out how to do that using vanilla-text-mask, do you know if is possible?
Thanks!

Hi @WandersonAlves
I can see 2 ways to do that, the first one would be to use the date pipe in the text-mask-addons lib, but you'll likely have to create a custom pipe for some mask formats.
The second way would be to do it on the angular side, catching when the input value changes and then doing a check. Not really clean I think because then the input validation is half vanilla, half angular... but as a workaround that would work.
Good luck and please let me know what you go for :)

Upping the priority of this as we'll need to add at least limited masking support to allow for prefix and suffix text now that we're only recommending matPrefix and matSuffix for icons (or other centered content) .
See related discussion on #9488

We were able to get a mask directive working with Material inputs using the text-mask library mentioned above. You can see the code here if it's helpful (at least until an official solution is offered).

Any way of using mask in mat-input without using a third party library? It would be nice to use a currency mask based on the currency pipe and the LocaleID of the app.

is there an official solution without using third party library today ?

Input masking is a really basic functionality to have in a UI library. Is there any news on this issue? Is this planned or what?

Isn't it the default behavior of native date input? https://developer.mozilla.org/en-US/docs/Web/HTML/Element/input/date
In mui (for react) there is a date picker that is material shaped and has native behavior like: https://mui.com/x/react-date-pickers/date-picker/
It even works for localized dates 😻  : https://mui.com/x/react-date-pickers/adapters-locale/
Why is angular material different?


Make Target class resposible for its own state management

State management for the Target class is currently spread out over the Browser, Target and Page classes which makes it hard to verify correctness. Therefore it makes sense to localize all code that determines when certain Target operations can and/or should be performed in the Target class itself.

Hey @ggeurts I don't want our code to differ too much from the original (Puppeteer code). Unless it's something that we do need to different in C#
This helps us keep up with the changes they make in their own code.
If you have new ideas, I think it would be more productive for you to begin with an issue, so we can debate the implementation first.


tech-insights: Not propper dependency on util i production code🐛 :

Plugin Name
tech-insights
📜 Description
I believe that production code shouldn't be dependent on mocks and test utils package.
👍 Expected behavior
No need to depend on @backstage/backend-test-utils in production
👎 Actual Behavior with Screenshots
Need to be dependent on: @backstage/backend-test-utils
https://github.com/backstage/community-plugins/blob/main/workspaces/tech-insights/plugins/tech-insights-backend/src/service/persistence/persistenceContext.ts#L40
👟 Reproduction steps
Just update to newest version
📃 Provide the context for the Bug.
No response
👀 Have you spent some time to check if this bug has been raised before?

[X] I checked and didn't find similar issue

🏢 Have you read the Code of Conduct?

[X] I have read the Code of Conduct

Are you willing to submit PR?
None

@secustor it is your change that introduces it in the newest version... is it something you wanted to achieve or just a mistake?
There is probably a null logger or a similar thing that can be used instead of mock.

We should make sure to include the error as others might be searching using that. For reference:
[1] /Users/dweber019/Documents/ProjectPrivate/backstage-community-plugins/workspaces/tech-insights/node_modules/@backstage/backend-test-utils/src/next/services/mockServices.ts:64
[1]     child: jest.fn().mockImplementation(createLoggerMock),
[1]            ^
[1]
[1]
[1] ReferenceError: jest is not defined
[1]     at createLoggerMock (/Users/dweber019/Documents/ProjectPrivate/backstage-community-plugins/workspaces/tech-insights/node_modules/@backstage/backend-test-utils/src/next/services/mockServices.ts:64:12)
[1]     at <anonymous> (/Users/dweber019/Documents/ProjectPrivate/backstage-community-plugins/workspaces/tech-insights/node_modules/@backstage/backend-test-utils/src/next/services/mockServices.ts:406:7)
[1]     at Object.mock (/Users/dweber019/Documents/ProjectPrivate/backstage-community-plugins/workspaces/tech-insights/node_modules/@backstage/backend-test-utils/src/next/services/mockServices.ts:114:18)
[1]     at Object.<anonymous> (/Users/dweber019/Documents/ProjectPrivate/backstage-community-plugins/workspaces/tech-insights/plugins/tech-insights-backend/src/service/persistence/persistenceContext.ts:40:31)
[1]     at Module._compile (node:internal/modules/cjs/loader:1369:14)
[1]     at Module._compile (/Users/dweber019/Documents/ProjectPrivate/backstage-community-plugins/workspaces/tech-insights/node_modules/pirates/lib/index.js:117:24)
[1]     at Module._extensions..js (node:internal/modules/cjs/loader:1427:10)
[1]     at Object.newLoader [as .ts] (/Users/dweber019/Documents/ProjectPrivate/backstage-community-plugins/workspaces/tech-insights/node_modules/pirates/lib/index.js:121:7)
[1]     at Module.load (node:internal/modules/cjs/loader:1206:32)
[1]     at Module._load (node:internal/modules/cjs/loader:1022:12)
[1]     at Module.require (node:internal/modules/cjs/loader:1231:19)
[1]     at require (node:internal/modules/helpers:179:18)
[1]     at Object.<anonymous> (/Users/dweber019/Documents/ProjectPrivate/backstage-community-plugins/workspaces/tech-insights/plugins/tech-insights-backend/src/service/persistence/index.ts:17:38)
[1]     at Module._compile (node:internal/modules/cjs/loader:1369:14)
[1]     at Module._compile (/Users/dweber019/Documents/ProjectPrivate/backstage-community-plugins/workspaces/tech-insights/node_modules/pirates/lib/index.js:117:24)
[1]     at Module._extensions..js (node:internal/modules/cjs/loader:1427:10)

Did a quick update of the title to make it a bit more clear. Thanks for logging this @ppiwow-apay 👍

same here, I updated the community plugins and having this error:
...
[1] //Repos/backstage/app/node_modules/@backstage/backend-test-utils/src/next/services/mockServices.ts:64
[1]     child: jest.fn().mockImplementation(createLoggerMock),
[1]            ^
[1]
[1]
[1] ReferenceError: jest is not defined
[1]     at createLoggerMock (//Repos/backstage/app/node_modules/@backstage/backend-test-utils/src/next/services/mockServices.ts:64:12)
[1]     at <anonymous> (//Repos/backstage/app/node_modules/@backstage/backend-test-utils/src/next/services/mockServices.ts:406:7)
[1]     at Object.mock (//Repos/backstage/app/node_modules/@backstage/backend-test-utils/src/next/services/mockServices.ts:114:18)
[1]     at Object.<anonymous> (//Repos/backstage/app/node_modules/@backstage-community/plugin-tech-insights-backend/src/service/persistence/persistenceContext.ts:40:31)
[1]     at Module._compile (node:internal/modules/cjs/loader:1256:14)
[1]     at Module._extensions..js (node:internal/modules/cjs/loader:1310:10)
[1]     at Object.newLoader [as .js] (//Repos/backstage/app/node_modules/pirates/lib/index.js:121:7)
[1]     at Module.load (node:internal/modules/cjs/loader:1119:32)
[1]     at Module._load (node:internal/modules/cjs/loader:960:12)
[1]     at Module.require (node:internal/modules/cjs/loader:1143:19)
[1]     at require (node:internal/modules/cjs/helpers:119:18)
[1]     at Object.<anonymous> (//Repos/backstage/app/packages/backend/src/plugins/techinsights.ts:25:31)
[1]     at Module._compile (node:internal/modules/cjs/loader:1256:14)
[1]     at Module._compile (//Repos/backstage/app/node_modules/pirates/lib/index.js:117:24)
[1]     at Module._extensions..js (node:internal/modules/cjs/loader:1310:10)
[1]     at Object.newLoader [as .ts] (//Repos/backstage/app/node_modules/pirates/lib/index.js:121:7)

@secustor it is your change that introduces it in the newest version... is it something you wanted to achieve or just a mistake?
There is probably a null logger or a similar thing that can be used instead of mock.

That has been on purpose to keep the public API unchanged, but it seems like @backstage/test-utils has jest not defined as dependency even though it needs it.
And no there is no longer a VoidLogger.


Proper explanation / phrasing of isocortex segmentation

A user reported: "Parcellation isocortex is not clear. " Indeed it is not really a parcellation, but it is of course a segmentation. We should discuss the presentation of this segmentation mask.

perhaps use unmappedIndicies e.g. https://jugit.fz-juelich.de/t.dickscheid/brainscapes-configurations/-/blob/master/maps/fsaverage-jba30-labelled.json?ref_type=heads#L1786-1815
ping @AhmetNSimsek


'dev' is not recognized as an internal or external command,

After installing Node.js and pnpm on Windows 10, and running these:

pnpm create solid && pnpm i
pnpm add solid-map-gl maplibre-gl mapbox-gl<EMAIL_ADDRESS>
the 'pnpm dev' command gives error:

'dev' is not recognized as an internal or external command, operable program or batch file.
ERR_PNPM_RECURSIVE_EXEC_FIRST_FAIL  Command "dev" not found

Any ideas? Thanks.

Had to cd into the subfolder and run 'pnpm install' first.


Add testing images and update tests

Add images to data folder
Update tests

Looks good to me .

Changes look good now! Merging the PR.


Note removal of Signal.combine in changelog.

Incorporated in newer pull request #268 after discussion in issue #266.


Update references-cpp.md

The reference object is also being initialized in the example (e.g., S& SRef = s;), so I'm proposing an update to the in-line comment to reflect the code more accurately.

@TylerMSFT - can you review the proposed changes?
IMPORTANT: When the changes are ready for publication, add a #sign-off comment to signal that the PR is ready for the review team to merge.
#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team

#sign-off


[Summary] Get Allergy Intolerances (List, by ID)

As a Authorized eHealth User
I want to see patient's allergy intolerances as a part of patient summary
So that i can provide high quality medical care for patient
Implement service according to spec (List, by ID) and logic
child of #3206

[ ]  should be 404 if patient does not exist but not empty array

GET /api/patients/5386d3e9-0d55-41ba-bb61-f06ec599f5d1/summary/immunizations HTTP/1.1
Host: https://api.dev.asclepius.com.ua
Content-Type: application/json
Authorization: Bearer K0JocVFqQklyTGFoaHF3MTlybVJ2UT09
api-key: eDBidGdHakQvWXB6M0RRZWUwM3BhUT08
cache-control: no-cache
Postman-Token: 13ea41c9-ce3f-4076-b2ed-c3c9e7461a40


Small bug and question

Just wanted to let you know the female default you provided has the arm tattoo vertical positions at -45 then they should be at 0, no big deal.
Also, how does this exactly work when it comes to the additional edit options? Could I just simply copy the list under the first and add a 2 after body/left/right?

Thanks! I'll get that fixed ASAP!
Unfortunately I haven't configured the edit options yet where it toggles more settings, I need to go through a figure out a good way. I have a few ideas though so don't.

Sound good! I'll share my presets to Nexus to help get attention to this now that it works perfectly!
Also, not sure if you can do this but I noticed when I had a missing int it gave an error which is fine, but could you configure it to default the missing int to 0 but still show in console "Missing Int X, defaulting to default.ini"

Good idea, can implement!
If you want you can also submit any presents as a Pull request here to get it included by default!

They are uploaded!
https://www.nexusmods.com/dragonsdogma2/mods/158
Thinking about it, I think it would defeat the purpose of pick and choosing presets if they are all included as defaults. Thank you for the offer though! If I think of other options or recommendations I'll let you know!

Hmmm, how hard would it be to implement something like a "Community presets button"? Something that would pull up a separate window with preset pictures, an install button, and of course providing a link to the nexus page if provided.
This would allow users to just unpack provided presets (without being flooded with options on default menu) that are included WITH the auto slider, give a link to the original presets for any updates to them, and provide a way to give examples of how to make presets. Like, once you figure out the toggle edit 2 options I'm using it lol
Also, would it be possible to have the browse button replaced with a drop down menu that allocates its names form the templates folder? You can of course just move the browse option as a back up, but just a thought to I guess.

Heads up; I rewrote the entire config logic, take a look.
Sorry about that, but some of the issues/bugs couldn't be fixed with 'math' based logic.

@ItsJawn Heads up, users are making issues for your mod:
https://github.com/Mx772/dragons-dogma-2-auto-character-creator/issues/27#issue-2214683583
Looks like it needs be updated to the latest formatting. (See the new defaults)
(You can use the new tool OR just convert the headers)


video_player plugin breaks flutter build ios

When I try to build for an iOS device/simulator I get this error message
Building test_project for device (ios-release)...
Automatically signing iOS for device deployment using specified development team in Xcode project: **********
Running pod install...                                              1.4s
Running Xcode build...
 ├─Building Dart code...                                     2.4s
 ├─Generating dSYM file...                                   0.2s
 ├─Stripping debug symbols...                                0.0s
 ├─Assembling Flutter resources...                           1.1s
 └─Compiling, linking and signing...                         1.2s
Xcode build done.                                            6.1s
Failed to build iOS app
Error output from Xcode build:
↳
    ** BUILD FAILED **

Xcode's output:
↳
    === BUILD TARGET Runner OF PROJECT Runner WITH CONFIGURATION Release ===
/test_project/ios/Runner/GeneratedPluginRegistrant.m:6:9: fatal error: 'video_player/VideoPlayerPlugin.h' file not found
    #import <video_player/VideoPlayerPlugin.h>
            ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
    1 error generated.
/test_project/ios/Runner/GeneratedPluginRegistrant.m:6:9: fatal error: 'video_player/VideoPlayerPlugin.h' file not found
    #import <video_player/VideoPlayerPlugin.h>
            ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
    1 error generated.

Encountered error while building for device.

If I remove the video_player plugin from the pubspec.yaml it works as intended.
I've tried many versions of the video_player plugin, all w/ the same result
Steps to Reproduce

Create a new Flutter project
Add video_player: 0.10.1+3 to the dependencies
Attempt to build for iOS device or simulator, through Android Studio or command line

Logs

flutter build ios --verbose yields
Encountered error while building for device.

#0      throwToolExit (package:flutter_tools/src/base/common.dart:24:3)
#1      BuildIOSCommand.runCommand (package:flutter_tools/src/commands/build_ios.dart:94:7)
<asynchronous suspension>
#2      FlutterCommand.verifyThenRunCommand (package:flutter_tools/src/runner/flutter_command.dart:559:18)
#3      _asyncThenWrapperHelper.<anonymous closure> (dart:async-patch/async_patch.dart:77:64)
#4      _rootRunUnary (dart:async/zone.dart:1132:38)
#5      _CustomZone.runUnary (dart:async/zone.dart:1029:19)
#6      _FutureListener.handleValue (dart:async/future_impl.dart:126:18)
#7      Future._propagateToListeners.handleValueCallback (dart:async/future_impl.dart:639:45)
#8      Future._propagateToListeners (dart:async/future_impl.dart:668:32)
#9      Future._complete (dart:async/future_impl.dart:473:7)
#10     _SyncCompleter.complete (dart:async/future_impl.dart:51:12)
#11     _AsyncAwaitCompleter.complete.<anonymous closure> (dart:async-patch/async_patch.dart:33:20)
#12     _rootRun (dart:async/zone.dart:1124:13)
#13     _CustomZone.run (dart:async/zone.dart:1021:19)
#14     _CustomZone.bindCallback.<anonymous closure> (dart:async/zone.dart:947:23)
#15     _microtaskLoop (dart:async/schedule_microtask.dart:41:21)
#16     _startMicrotaskLoop (dart:async/schedule_microtask.dart:50:5)
#17     _runPendingImmediateCallback (dart:isolate-patch/isolate_patch.dart:115:13)
#18     _RawReceivePortImpl._handleMessage (dart:isolate-patch/isolate_patch.dart:172:5)

old but possibly related
https://github.com/flutter/flutter/issues/18265

can you try this version without  putting ^  ?   video_player: 0.5.2

Got the same error
Error output from Xcode build:
↳
    ** BUILD FAILED **

Xcode's output:
↳
    /test_project/ios/Runner/GeneratedPluginRegistrant.m:6:9: fatal error: 'video_player/VideoPlayerPlugin.h' file not found
    #import <video_player/VideoPlayerPlugin.h>
            ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
    1 error generated.

Encountered error while building for device.

That's not the issue with Flutter itself, but with the plugin you are playing with. You should have more luck reporting issue there.

@vintage
https://github.com/flutter/plugins
Please file any issues, bugs, or feature requests in the main flutter repo.
This is plugin maintained by the Flutter team and they've asked to open issues for it on this main repo.

I think I found the issue, looks like my ios/Podfile wasn't setup correctly.
Found what it was supposed to look like by doing a fresh flutter create blank and adding the video_player plugin to the pubspec.yaml. After that, I built the project for iOS and a ios/Podfile was created that looked like this
# Uncomment this line to define a global platform for your project
# platform :ios, '9.0'

# CocoaPods analytics sends network stats synchronously affecting flutter build latency.
ENV['COCOAPODS_DISABLE_STATS'] = 'true'

project 'Runner', {
  'Debug' => :debug,
  'Profile' => :release,
  'Release' => :release,
}

def parse_KV_file(file, separator='=')
  file_abs_path = File.expand_path(file)
  if !File.exists? file_abs_path
    return [];
  end
  pods_ary = []
  skip_line_start_symbols = ["#", "/"]
  File.foreach(file_abs_path) { |line|
      next if skip_line_start_symbols.any? { |symbol| line =~ /^\s*#{symbol}/ }
      plugin = line.split(pattern=separator)
      if plugin.length == 2
        podname = plugin[0].strip()
        path = plugin[1].strip()
        podpath = File.expand_path("#{path}", file_abs_path)
        pods_ary.push({:name => podname, :path => podpath});
      else
        puts "Invalid plugin specification: #{line}"
      end
  }
  return pods_ary
end

target 'Runner' do
  # Prepare symlinks folder. We use symlinks to avoid having Podfile.lock
  # referring to absolute paths on developers' machines.
  system('rm -rf .symlinks')
  system('mkdir -p .symlinks/plugins')

  # Flutter Pods
  generated_xcode_build_settings = parse_KV_file('./Flutter/Generated.xcconfig')
  if generated_xcode_build_settings.empty?
    puts "Generated.xcconfig must exist. If you're running pod install manually, make sure flutter packages get is executed first."
  end
  generated_xcode_build_settings.map { |p|
    if p[:name] == 'FLUTTER_FRAMEWORK_DIR'
      symlink = File.join('.symlinks', 'flutter')
      File.symlink(File.dirname(p[:path]), symlink)
      pod 'Flutter', :path => File.join(symlink, File.basename(p[:path]))
    end
  }

  # Plugin Pods
  plugin_pods = parse_KV_file('../.flutter-plugins')
  plugin_pods.map { |p|
    symlink = File.join('.symlinks', 'plugins', p[:name])
    File.symlink(p[:path], symlink)
    pod p[:name], :path => File.join(symlink, 'ios')
  }
end

post_install do |installer|
  installer.pods_project.targets.each do |target|
    target.build_configurations.each do |config|
      config.build_settings['ENABLE_BITCODE'] = 'NO'
    end
  end
end

Replacing my current ios/Podfile w/ the above solved my issue.


TestDialWithBackoff from network/transports_test.go fails by not recognizing expected error

Expected Behavior
Test should pass without errors.
Actual Behavior
Test fails with:
$ gotestsum -f testname -- -race -count=1 -run 'TestDialWithBackoff' ./network
=== RUN   TestDialWithBackoff
    transports_test.go:91: Error = dial tcp4 <IP_ADDRESS>:8888: connect: network is unreachable, want: timed out dialing(...)
--- FAIL: TestDialWithBackoff (0.00s)
FAIL network.TestDialWithBackoff (0.00s)
FAIL network

=== Failed
=== FAIL: network TestDialWithBackoff (0.00s)
    transports_test.go:91: Error = dial tcp4 <IP_ADDRESS>:8888: connect: network is unreachable, want: timed out dialing(...)

DONE 1 tests, 1 failure in 0.489s

Steps to Reproduce the Problem

gotestsum -f testname -- -race -count=1 -run 'TestDialWithBackoff' ./network

Additional Info
It looks like the expected order of the error message has changed. The Go code should rely on errors.Is to check errors instead of string prefixes.
Invalid code is at:
https://github.com/knative/pkg/blob/b558677ab03404ed118ba3e179a7438fe2d8509a/network/transports_test.go#L90-L91

Strangely, this test is being reported as passing in prow unit tests or Github's unit test check. How come?!

Okay. I think I got the root cause of this. The test is using a special address <IP_ADDRESS>:8888 that looks to work on the CI environment. I think a good idea would be to refactor tests to work on every environment, cutting this external dependency.

/assign


[Diagnostics] Change phrasing of the existential_requires_any warning

The previous warning "must be explicitly marked as 'any'" isn't clear if you don't already know the feature, this new phrasing should make it clearer by including the correct spelling of the type as an example.
Examples before:
protocol 'EqualComparable' as a type must be explicitly marked as 'any'
'Constraint' (aka 'Input') as a type must be explicitly marked as 'any'

Examples after:
use of protocol 'EqualComparable' as a type must be written 'any EqualComparable'
use of 'Constraint' (aka 'Input') as a type must be written 'any Constraint'

Fixes rdar://90384448

@swift-ci please smoke test


[TPU Provisioner] Delete node pool when JobSet is completed, failed, or deleted

This PR includes the following changes:

Update node reconciliation based deletion controller to delete node pool when JobSet is completed, failed, or deleted
Refactor deletion controller integration tests to be more extendable so we can add new test cases more easily
Move controller integration tests to a test/integration/controller directory. When we add the Job webhook, we can add integration tests in test/integration/webhooks.  E2E tests can go in test/e2e. This will help keep things organized.

/gcbrun

/gcbrun


Add BigDecimal

This is a totally naive and somewhat broken implementation. I'm new to rust and BigDecimal implementation, but I thought I'd get this out there early for some guidance on ways I can improve my code before getting too deep.
#8 is the second oldest issue on this repo and I hadn't seen any code. So, I figured the work I've done would be useful to future prospective implementors, even if I don't manage to finish.
Add, Sub, and Mul all seem to work correctly (for my single test case each :smile:)
Div is currently broken, as it only returns the integer portion instead
of a float as one might expect. Other traits are stubbed out to satisfy
the Num trait.

It seems like a good start!  I won't nitpick this too much yet, but I'm glad you're working on it.
A big consideration is what you expose in the public interface.  In particular, set_scale sounds like it should modify the object in place, but you're taking &self and returning an adjusted clone.  I would also expect this to allow decreasing the scale.
Do you really want the semantics of a negative scale?  So I guess something like -2 means the number represents hundreds?
In general, try to avoid clones if you can.  For instance in Mul it could multiply the int_val by reference.

Hey there,
is this still on  anybody's radar? We could really use a good Decimal implementation :grinning:

I'm still open to it.  Perhaps if @iterion is too busy or no longer interested, someone else could run with it.

There's the option for anyone to publish it as a separate crate/repo first and then discuss inclusion. That might help speed of development in the start.

I'm still interested, but don't have time right now. So, if someone has time/motivation they should definitely take this over.

I've extended @iterion's BigDecimal class and added more test cases, and I think things are looking pretty solid: https://github.com/akubera/num/blob/topic/big-decimal/bigdecimal/src/lib.rs.
My first step was to merge the current master with @iterion branch, so the diffs might look like more complex than it is. Here's the simplified diff-to-iterion-fork.
I added an arbitrary precision limit to 100. This is something that should be discussed; should there be a 'precision context' object that may be set during runtime or a compile-time constant which simplifies use? You can see the effect of this magic 100 number at this line of the division operator test.
I tried to make pretty thorough tests (bottom of file). The parser ignores underscores in numbers, and understands exponential form (1e3). Please send me any questions/comments!

I'm not sure what the procedure is to merge a fork of a fork. Let me know if I should create a pull request directly to the @iterion fork. If requested, I could rebase my work before my "master merge" to make the graph a little nicer, or do it as-is.

I don't have time to review today, but I'd say open a new PR with a linear
history.  Either rebased before master or fully rebased onto master is
fine, though I lean toward the latter.  It's nice if you can keep the full
commit history with @iterion's commits before yours, but if that's too hard
than a simple attribution/acknowledgement in your own commit message should
suffice, I think.

Good news, git keeps author information in a rebase, so @iterion's commits are kept. I rebased his branch with num/master, and cherry-picked all commits after my master-merge; everything should be linear.
This can further be discussed in pull request #224.
Thanks.

Thanks @akubera! Closing this since we have a replacement in the works.


How to use properly create_direct_message ?

Hi ! I'm trying to use your function but I meet a problem... Here it is :
/home/a1/.rvm/gems/ruby-2.5.1/gems/twitter-6.2.0/lib/twitter/rest/request.rb:81:in `fail_or_return_response_body': Sorry, that page does not exist. (Twitter::Error::NotFound)
And here is my code :
client.create_direct_message(1447374817, "Salut, dis moi si ça fonctionne stp !")
1447374817 is the user ID of the target. If you know what is going wrong, I'd be glad if you'd tell me !
Thanks,
Alex

This is related to #908. In this post it looks like an update is on the way but I'm not sure when.

Thanks !


sourcing in bash fails with syntax error

bash: path/_zig: Zeile 159: Syntax error on unexpected symbol »(«
bash: path/_zig: Zeile 159: `      ${^=LD_LIBRARY_PATH:-/usr/lib /usr/local/lib}/lib*.(a|so*)(:t:fr:s/lib//)'
Sourcing in .bashrc with . path/_zig

This currently only supports Zsh and is quite out of date.


Ability to run MAPL with two phases

@rmontuoro has requested the ability to run both run phases of GOCART independently in the UFS. This is so that the UFS can better emulate how GOCART is run within GEOS. This will require changes to MAPL so that the run method for the MAPL_CapGridComp supports the optional ability to run multiple phases.
Currently, the UFS does not support running multiple phases; however, there is a push on the NOAA side to get the UFS ready to run GOCART in multiple phases. Thus we need to ready MAPL (and GOCART) to support this feature.

It seems to me, in GEOS_PhysicsGridCompMod, it already runs two phases of GEOS_ChemGridComp, which runs two phases of GOCART.  What is the eventual order to run the phases of GOCART?

GOCART in the UFS is run independently of the GEOS_PhysicsGridComp and the GEOS_ChemGridComp (neither of these is used at all).

NOAA is not using Physics, nor using Chemistry.       So they cannot drive it indirectly the way it currently happens in GEOS.


Error after Alt-Rightclick on byproduct

This error happened after I Alt-rightclicked on one of the byproducts (playing Pyanodons) at the top to open in Recipe Book. The error seems to be specific to the byproducts; I didn't have it happen for the ingredients for example. I also had just downloaded the latest update of Factory Planner.
Log file: factorio-current.log

I can't seem to be able to reproduce this easily unfortunately. Are you able to upload your save for me? Just drag it to a textfield on here and it should upload, if it's not too large.

It seems to be resolved now, thank you!
Weirdly when I first tried it with the new update both byproducts and ingredients at the top of the window either got linked to seemingly random other items in Recipe book or caused a crash again, but when I restarted the game it was all fine again.

That one will be fixed in the next release as well, just for context. #191


Set CONTAINER_ENGINE/container_engine to docker

The clusters we use for dev/testing(Zuul)/stg/prod don't like images built & pushed with podman.

:<
I wonder if podman devs know about this


Mac support

This is such a great project that it deserves to be available to Mac users. Is there any chance that we can have MPS support?

I don't have a Mac device and therefore could not test it. Most of the time I think we are passing this device around https://github.com/hkchengrex/MMAudio/blob/ec2de628a5f226f8b96aa7686339326c4a817e41/demo.py#L65
I think you can try changing that to mps and see if you encounter any errors. Pull requests to fix them are welcomed -- I cannot test them and therefore it would be quite inefficient for me to try to fix them.

#19


Custom validator does not include Enum fields in 'values' argument

Bug
Output of python -c "import pydantic.utils; print(pydantic.utils.version_info())":
             pydantic version: 1.9.1
            pydantic compiled: True
                 install path: /home/shade/.virtualenvs/kafka_sap_pi_adapter/lib/python3.8/site-packages/pydantic
               python version: 3.8.13 (default, Mar 19 2022, 05:55:35)  [GCC 11.2.0]
                     platform: Linux-5.10.126-1-MANJARO-x86_64-with-glibc2.34
     optional deps. installed: ['dotenv', 'typing-extensions']

Hello!
I have a problem with custom validator and a model including Enum fields.
Given the following model which works like expected:
from pydantic import Field, validator
from pydantic.main import BaseModel

class E1ADHDR(BaseModel):
    message_type: str = Field(alias="MESTYP")
    message_type_lng: str = Field(alias="MESTYP_LNG")
    e1state: List[E1STATE] = Field(alias="E1STATE")

    @validator("e1state", each_item=True)
    def check_names_not_empty(cls, val, values, **kwargs):
        if values["message_type"] == "RELEVANT_OOS_DETECTED" and val.stapa4_lng is None:
            raise ValueError("stapa4_lng is required")
        return val

If I change the type of message_type  field from str to Enum like this:
class E1AdhrMessageType(Enum):
    """Possible values of E1STATE's status field."""

    trigger = "OOS_RELEVANT_DETECTED"
    invent = "WVINVE_UTP"
    writeoff = "WRITEOFF_UTP"

the message_type field disappears from values in validator resulting in KeyError exception

You mean change from str to E1AdhrMessageType?

Yes

Is that the only change u did?
is it possible u also changed the fields order?
from the docs: https://pydantic-docs.helpmanual.io/usage/validators/

Validation is done in the order fields are defined. E.g. in the example above, password2 has access to password1 (and name), but password1 does not have access to password2. See Field Ordering for more information on how fields are ordered

Sorry for late answer.
This seems to be the only change I did.
I've re-checked again and noticed that version with enum:
from enum import Enum
from typing import List, Optional

from pydantic import Field, validator
from pydantic.main import BaseModel

class E1AdhrMessageType(Enum):
    """Possible values of E1STATE's status field."""

    trigger = "OOS_RELEVANT_DETECTED"
    invent = "WVINVE_UTP"
    writeoff = "WRITEOFF_UTP"

class E1ADHDREnum(BaseModel):
    message_type: E1AdhrMessageType = Field(alias="MESTYP")
    e1state: List[int] = Field(alias="E1STATE")
    stapa4_lng: Optional[str]

    @validator("e1state", each_item=True)
    def check_names_not_empty(cls, val, values, **kwargs):
        print("validator_enum", val, values)
        if values["message_type"] == "RELEVANT_OOS_DETECTED" and val.stapa4_lng is None:
            raise ValueError("stapa4_lng is required")
        return val

m_enum = E1ADHDREnum.parse_obj({
    "MESTYP": "xxx",
    "E1STATE": [1, 2, 3],
})

crashes with the KeyError
validator_enum 1 {}
Traceback (most recent call last):
  File "./model_test.py", line 31, in <module>
    m_enum = E1ADHDREnum.parse_obj({
  File "pydantic/main.py", line 578, in pydantic.main.BaseModel.parse_obj
  File "pydantic/main.py", line 404, in pydantic.main.BaseModel.__init__
  File "pydantic/main.py", line 1040, in pydantic.main.validate_model
  File "pydantic/fields.py", line 734, in pydantic.fields.ModelField.validate
  File "pydantic/fields.py", line 767, in pydantic.fields.ModelField._validate_sequence_like
  File "pydantic/fields.py", line 899, in pydantic.fields.ModelField._validate_singleton
  File "pydantic/fields.py", line 737, in pydantic.fields.ModelField.validate
  File "pydantic/fields.py", line 913, in pydantic.fields.ModelField._apply_validators
  File "pydantic/class_validators.py", line 275, in pydantic.class_validators._generic_validator_cls.lambda2
  File "./model_test.py", line 26, in check_names_not_empty
    if values["message_type"] == "RELEVANT_OOS_DETECTED" and val.stapa4_lng is None:
KeyError: 'message_type'

only in case if invalid value for message_type is given.
If the field value matches the E1ADHDREnum values validation works as expected.
Is it accepted behavior not to check enum values first in this case?

Yep, this is the expected behavior, and it is also documented here
see the second bullet:

where validators rely on other values, you should be aware that:
Validation is done in the order fields are defined. E.g. in the example above, password2 has access to password1 (and name), but password1 does not have access to password2. See Field Ordering for more information on how fields are ordered
If validation fails on another field (or that field is missing) it will not be included in values, hence if 'password1' in values and ... in this example.


Remove typedoc-plugin keyword

This keyword is used by TypeDoc to automatically discover and load plugins by default. If this package is installed, this will result in errors when running TypeDoc since require("@enio.ai/typedoc").load is not a function.

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.

Hey @Gerrit0,
My apologies for the unlucky coincidence! Duck-typing repos would be a good idea.
I am happy to accept this change. I rather keep some sort of "safe" keyword that still mentions typedoc.
Is there any that could be used?

TypeDoc does check - so it doesn't crash, but it still complains. The first keyword listed is "typedoc", so I think that's still covered.

Fair enough 👍
Do you mind signing the CLA? It is required to merge commits.
Otherwise, I can make this change on your behalf.

@Gerrit0 See comments above. Are you scanning the npm registry or GitHub? If the former, I guess a patch version will need to be published.

Go ahead and just change it - didn't want to read through another cla.
TypeDoc scans your node_modules tree. The site also scans npm using the keyword to build a list of plugins

Closing PR in favor of https://github.com/enio-ireland/enio/pull/35


Create RockPaperScissor.py

Created a Popular Game known as Rock Paper Scissor in python. Hope you like it.

Star this repo


Cannot set inputStyle on mobile device

For me the contenteditable experience on mobile is unusable. I'm unable to focus my cursor with my Pixel 3.
So I tried setting the inputStyle directly, but that throws an exception on init:
"inputStyle can not (yet) be changed in a running editor")
It doesn't seem possible to override the behavior on a mobile device looking at this line of code: https://github.com/codemirror/CodeMirror/blob/6d38fc2148e9f97ffbb55c78629dcecd9902ac29/src/edit/options.js#L77-L79
I've gone in to Chrome Dev Tools and changed my UA string in the Network Conditions UI and then when I load up the editor things are waaaay better.
Is there any way to get this to work without forking the entire project right now and editing that line of code?
fwiw I'm actually using the react-codemirror2 wrapper but it doesn't seem they're doing anything special wrt how options are set.
Thanks!

hrm, gonna close for now as https://github.com/scniro/react-codemirror2/pull/113 makes me think the bug is actually in react-codemirror2 not setting the options on the constructor. Will reopen if I'm wrong.


fix: Require at least go 1.18

This PR aims to fix https://github.com/Evertras/bubble-table/issues/179 by raising the minimum supported version of go from 1.17 to 1.18.

WDYT @Evertras  ?

Apologies, catching up! Let's go with this, yes. I think we can reasonably break off <1.18 compatibility given the underlying dependency did too.


improvement support datev2 and datetimev2

Proposed changes
Issue Number: close #xxx
support datev2 and datetimev2
Further comments
If this is a relatively large or complex change, kick off the discussion at<EMAIL_ADDRESS>by explaining why you chose the solution you did and what alternatives you considered, etc...

run buildall

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


(game management) high level goal autofilling the game creation page based on the BGG api (owen, will, daniel)

When a staff member is adding a game through the game create page, they can use a search bar to search for games in BGG and select a game. The game creation page is then autofilled with the details of the specific game supplied by the BGG API.
See #45 for documentation on using BGG API and some thoughts on how to use it. See #46 for a demo page that uses the BGG API to search and display contents of the result.

First we will need to finalize #92 so that we know exactly what game information needs to be autofilled by the BGG API.

I have a first pass of this working. This task is dependent on completing #192
https://github.com/uchicago-cs/chigame/assets/69015620/d09db827-32b9-4776-ad70-a079dc55d517

Branch has been added. Currently, it does not include a way to autofill categories or mechanics.

Issue Score: Excellent
Comments:
As said on the related PR, this is phenomenally well done issue.
Thank you all for your work!


Is not the max of meteor score 100?  (nltk.translate.meteor_score)

I used nltk.translate.meteor_score to calculate the meteor score of a example, strangely I got a score of 105.6.
Is not the max of meteor score 100?
The example is following:

candidate:   creates or updates a virtual machine scale set
reference:   create or update a vm scale set
METEOR score:   105.6

Is there anyone know that? Thank you very much!

You are correct. The METEOR score should be 0.7806 (or 78.06%), and definitely not more than 100%.
Through this issue, I've found a bug in how NLTK computes the METEOR score, and have created #2763 to fix this issue.
I suspect this will be fixed in the next minor version.
Thank you for bringing this to our attention!


Client freezes when right-clicking on server

Describe the bug
When right clicking on a server in the server menu, client will freeze and remain frozen until Discord is restarted. Have already attempted:

repairing installation
deleting version folder in .config/discord, then uninstalling and reinstalling BetterDiscord

System Information:

Manjaro KDE 5.11
Stable

Which Discord AUR package are you using?

I'm using the Discord client from the Official Repositories (community). However, I'm no longer seeing the issue after a restart (despite that not fixing the issue before), so there's not much more for me to diagnose.


add my first file

I have created a new branch, created a file, and made a commit!

I finished the Readme portion of GitHub
I created a new branch
I created a file, and made a commit too


Support i1 datatype with an experimental flag.

Enable packed i1 datatype storage
This commit introduces support for packed storage of the i1 (bit) datatype. When subbyte type packing is enabled via the --iree-experimental-packed-i1-storage option, vectors of i1 elements will be stored in a compact packed representation.
For example, a vector<6xi1> will occupy a single byte of memory with the 6 bit elements packed together and 2 padding bits. A vector<3x3xi1> will take up 2 bytes, with the 9 bit elements packed across the bytes and 7 padding bits.
Limitations:

To ensure correct behavior, the tiling configuration aligns the innermost dimension data loads with byte boundaries. This is necessitated by the current lack of emulation for unaligned subbyte vector loading/storing.
Unaligned subbyte emulation support can be added in the future, though it may incur some performance overhead.

This change requires corresponding updates in the frontend to utilize the packed i1 storage format.

I'll revisit the PR because now we have the support for cross-byte i1 loads.  @lialan you're still working on the store part, right? Also, could you add an e2e i1 load test to the PR? I think we can iterate on that for the load support.

I'll revisit the PR because now we have the support for cross-byte i1 loads. @lialan you're still working on the store part, right? Also, could you add an e2e i1 load test to the PR? I think we can iterate on that for the load support.

@hanhanW Looks like the stores are not our highest priority at the moment. We want to have i1 support for loads primarily

@lialan I'm not sure if the PR is ready or not because you added few commits. Can you mark it draft and turn it open when it's ready for review?
If you need some inputs and it's not ready for review yet, feel free to tag me or ping me on discord.

@lialan I'm not sure if the PR is ready or not because you added few commits. Can you mark it draft and turn it open when it's ready for review?
If you need some inputs and it's not ready for review yet, feel free to tag me or ping me on discord.

Apologies, not sure what automation or my mis-click marked this as open. I turned it back to draft. There are some issues with e2e tests and I am fixing it.

Turns out, when converting to flow dialect, some operations such as tensor.insert_slice will be turned into vector stores to memref. For that we need emulated vector store support.

@benvanik Ben can you review it for us?


A11y_MySQLextensionforAzureDataStudio_Connect to a MySQL Server_ Screen Reader: When we navigate to the Server Required field, which is blank, the Screen reader announces "Invalid."

"Check out Accessibility Insights! - Identify accessibility bugs before check-in and make bug fixing faster and easier.”
GitHub Tags:
#A11yMAS; #A11yTCS; #A11ySev3; #DesktopApp; #GH-MySQLextensionforAzureDataStudio-Win32-Mar23; #MySQL extension for Azure Data Studio; #Win32; #Info and Relationships; #JAWS; #Screen Reader; #Benchmark; #AILimited;   #Element:Editfield: #NVDA; #JAWS;
"Please do not close this bug. This bug should only be closed by TCS, C+AI Accessibility or the Divisional Driver."
Environment and OS details:
Azure Data Studio Version: 1.42.0 (user setup)
Windows 11 Enterprise Version	22H2
Installed on	28-12-2022
OS build	22621.1413
Experience	Windows Feature Experience Pack 1000.22639.1000.0
NVDA Version 2023.1
JAWS Version 2023
Pre-Requisite:

Install Azure data studio From here https://learn.microsoft.com/en-us/sql/azure-data-studio/download-azure-data-studio?view=sql-server-ver16&tabs=redhat-install%2Credhat-uninstall
Once installation done , Install MySQL Extension in Azure data studio
Tab Till  new connection and press enter
Enter a valid server name username and password and press connect

Repro Steps:

Launch Azure Data Studio.
Turn on NVDA
Tab till New connection and Press enter
Tab till Server
Verify that Narrator is announcing Invalid entry or not

Actual Result:
When we navigate to the Server Required field, which is blank, the NVDA announces "Invalid."
Issue Repro with JAWS Also
Notes
Similar issue repro in Following Screen also
New Data Base :

Launch Azure Data Studio.
Connect to server.
In the Connections pane, right-click on a server connection.
Then select 'Manage' from the context menu to view the dashboard.
From dashboard, click on New Database.
Tab till Name and Verify Screen reader is announcing Invalid or not.

New deployment - Azure Database for MySQL

Launch Azure Data Studio.
Select the Connections icon (if not already selected) from left bar.
Click on the ellipses '...' and select "New Deployment"
In the Deployment options, select "Azure Database For MySQL" card.
5.Verify Screen reader is announcing invalid for accept terms of use checkbox or not

Expected Result:
NVDA should not announce as an invalid entry.
Attachment:
Title Issue :

Notes Issue 1:

Notes Issue 2:

This issue is external to MySQL extension as it is a common UI of Azure Data Studio.

External Existing bug Found :https://github.com/microsoft/azuredatastudio/issues/22220
Could You please Reopen the bug since We have follow External process for this bug As per the External Bug process,
Once external bug is resolved and closed, we will act on this bug accordingly.Please don't close the bug


How do you change the default dashboard in my config.ru? I'd like to run two dashboard projects on the same linux box

Hello,
How do I change the default dashboard in my config.ru?
Thanks

In the config.ru
Locate the section that says
configure do
  set :auth_token, 'YOUR_AUTH_TOKEN'
  # ...
end

Add default_dashboard to it.
configure do
  set :auth_token, 'YOUR_AUTH_TOKEN'
  set :default_dashboard, 'sampletv'
  # ...
end

This issue can be closed. The docs contain some info about it too https://github.com/Shopify/dashing/wiki/How-To:-Change-the-default-dashboard


Add a possibility to create HttpMessage instances with pre-existing Headers

Addressing issue #4315
Allow passing HttpHeaders instance to DefaultHttpMessage in order to avoid eager creation of Headers to allow users reuse their Headers instance.
Modifications:
Added a constructor with HttpHeaders to DefaultHttpMessage, Modified DefaultHttpResponse and DefaultHttpRequest to receive HttpHeaders instances.
Result:
Users can now pass HttpHeaders instance when  constructing Http Requests and Responses.

:+1:

@slandelle I hope I'm not stepping on your toes :) just tring to be helpful

@ifesdjeen maybe also adding the constructors for FullHttp* ?

@ifesdjeen Absolutely not, just I had this one in mind too :)

@ifesdjeen also please update the commit message to have a "one-line description" of the change as first line like her:
http://netty.io/wiki/writing-a-commit-message.html

True. I have somehow missed the FullHttp*. But it looks like they may benefit from this change even more.

Sorry, I've taken the comments from pervious version and overlooked it :(

@ifesdjeen - Thanks!