az command not working: import issue

This is autogenerated. Please review and update as needed.
Describe the bug
Command Name
az login
Errors:
cannot import name '_thread' from 'six.moves' (unknown location)
Traceback (most recent call last):
python3.7/site-packages/knack/cli.py, ln 206, in invoke
    cmd_result = self.invocation.execute(args)
cli/core/commands/__init__.py, ln 560, in execute
    raise ex
cli/core/commands/__init__.py, ln 618, in _run_jobs_serially
    results.append(self._run_job(expanded_arg, cmd_copy))
cli/core/commands/__init__.py, ln 611, in _run_job
    six.reraise(*sys.exc_info())
Users/tamir.amram/six.py, ln 491, in reraise
    raise value
...
site-packages/dateutil/parser/__init__.py, ln 2, in <module>
    from ._parser import parse, parser, parserinfo
site-packages/dateutil/parser/_parser.py, ln 50, in <module>
    from .. import tz
site-packages/dateutil/tz/__init__.py, ln 2, in <module>
    from .tz import *
site-packages/dateutil/tz/tz.py, ln 21, in <module>
    from six.moves import _thread
ImportError: cannot import name '_thread' from 'six.moves' (unknown location)

To Reproduce:
Steps to reproduce the behavior. Note that argument values have been redacted, as they may contain sensitive information.

Put any pre-requisite steps here...
az login

Expected Behavior
Environment Summary
Darwin-18.5.0-x86_64-i386-64bit
Python 3.7.3
Shell: bash

azure-cli 2.0.64 *

Additional Context

Hello @amramtamir, what is your CLI installation method?

Hi,
brew install azure-cli

@marstr, can you please take a look?

Investigating now

I need more information, @amramtamir! When I run brew install azure-clion my Mac running 10.14.4, all seems well. Is there anything special about your machine? Have you installed Python on your machine outside of using Homebrew?

Hi @marstr ,
Nothing special on my Mac - I used brew to install python.
I also tried to:
reinstall azure-cli, python, brew, Six package.

Hi @marstr ,
Nothing special on my Mac - I used brew to install python.
I also tried to:
reinstall azure-cli, python, brew, Six package.

This might help? https://stackoverflow.com/questions/27630114/matplotlib-issue-on-os-x-importerror-cannot-import-name-thread

Hi @marstr
It seems to work!
I have done this steps:
opened python from the terminal
import six
print six.file
the output showed that the six.pyc and six.py was on my home path
I ran rm -rf six.*
and the "az" command is now working :)

This is troublesome to me, as my hope was that using a Virtualenv in the Homebrew installer would isolate us from these sorts of problems.
@lmazuel, my take away from this is that in addition to invoking the python inside of the virtualenv, I should be invoking it with the -s flag. Do you think that's the right conclusion?

@marstr venv should not load any system or user system directory by default, so that option should be useless. If you're able to show me an example with venv and user directory where "-s" makes a difference I'm curious :)
That being said, it seems the story is more subtle:
https://github.com/pypa/pip/issues/4141
https://github.com/pypa/pip/issues/5702

I'm going to close this out as a one-off error for now. However, if other people stumble on this and it needs a fix from us, I'm happy to open it back up.


Added the abiliy to save photos

So does it work on competition hardware @lutet88 ?

yes

ok then


Create production branch

The first production branch will be 21.11 which will be cut from the 21.11.0 tag and and then point towards 21.11.1-SNAPSHOT until such time a release is done

Note that I am not going to require back porting of any issue filed against current. If a customer wants something fixed against the 21.11 branch a separate issue should be filed. If it is a back port of an existing issue filed against current then they should be linked. If the issue does not exist and the bug also applies to current then a separate issue should be filed against current. Note I will create a 21.11 label which should be used to identify an issue filed against that branch. Please do NOT use the title of the issue for this.


[FW] Enable Turrets By Default

Current FW mega turrets're diabled until they first seeing an enemy, make them unable to be damaged at start, which is confusing

Need someone to do code review and then it should be ready to merge ^^


Defender

defender detects the created file
bunun üstesinden nasıl gelebilirim

Do you build it as native? And if it detects it try rebuilding it.

I couldn't understand my english is not very good

Try rebuilding it with "Native" selected.

ok i will try

defender is caught again what should i do

Rebuild it, your files MD5 checksum can be registered in a database, meaning that the file or code itself probably isn't detected but just the MD5 checksum/hash of the file.

so what should i do

Rebuild the file.


fix: updated capabilities to warn the user in the case there could be capability overlap

Description
Pepr capabilities can overlap each other. Right now it creates a possible ambiguous problem where 2 bindings/callbacks can match an object.
In the following example, the order matters, and might not be obvious to the developer why a configmap with name test in namespace test will be denied. This fix does NOT change the behavior, but will provide a warning (which is WIP) that this could happen.
When(a.ConfigMap)
  .IsCreated()
  .WithName("test")
  .Validate(request => {
    return request.Deny();
  });

When(a.ConfigMap)
  .IsCreated()
  .InNameSpace("test")
  .Validate(request => {
    return request.Approve();
  });

The suggested fix would be to merge these two into a single one where the application developer clearly understands the relationship and what they're trying to do. Think of this as linting for pepr capabilities.
Related Issue
Fixes: https://github.com/defenseunicorns/pepr/issues/311
Type of change

[x ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Other (security config, docs update, etc)

Checklist before merging

[ ] Test, docs, adr added or updated as needed
[ ] Contributor Guide Steps followed

@bdw617 as you can see from both the project board and the issue #311 which you referenced this issue is already assigned and a work in progress.. If you want to contribute in the future you should checkout an issue that is unassigned.

Closing this for now due to this reason. Feel free to open a new issue if you need to
https://github.com/defenseunicorns/pepr/issues/311#issuecomment-1832846453


Validation of existing notes

Is it possible to perform validation on existing notes using the metadata menu plugin? I'd love to be able to see which required fields are missing and which ones don't have the right value type or value not part of the defined Select

Hi, the fileField api method is doing that. Though it isn't yet satisfying for fields validation. you could test it and see if it suits your needs https://mdelobelle.github.io/metadatamenu/api/#filefields
for example "isValid" always return true for select, multiselect, file, and multiFile fields : I've not finished all the validation methods

What about the built in fileClass table? Is it possible to have a way to indicate for each row whether it is fully valid according to schema? And a way to filter for those that are valid vs invalid?


Inactivity timeout & Ticker Animation

[x] Implemented inactivity timeout on Quiz level by using react-idle-timer. You can control it through the appConfig.js file.
[x] Implemented ticker animation on Attract Screen.

Preview Link
https://ctsc-career-quiz-git-timeranimation-rauf-tech.vercel.app?_vercel_share=WNgKnzTQPrBlP4471BMsRKZ7DQOviFtG

Thanks @abdulrauf11 - I noticed the profile image crops have uneven padding, which is making the carousel spacing look a little off. (https://d.pr/i/Nssuhn)
If my team adjusts those crops so the circles are perfectly centered with equal left-right padding, that should fix the spacing and alignment, right?
If that's the case, I'll approve this after you slow down the animation a little. My team can fix those images later this week.

@tnordberg that is correct! The uneven padding is because of the different image sizes and padding around them.
You may also need to alter the previous image icons as well.

@abdulrauf11 could you please merge this approved PR?


ARTEMIS-4517 - cancel message references in queue sequence order

this needs more eyes please

pushed an update to rename the idComparator the queueSequenceComparator

@gtully beware I added a change to your branch. please update your local fork.. feel free to squash the change if you prefer

I think this is ready to merge...


edgemesh code decoupling

According to the proposal: https://github.com/kubeedge/kubeedge/pull/2627, we have decoupled edgemesh from edgecore:

code transplantation comes from: https://github.com/Poorunga/edgemesh
update the readme.md file

/lgtm

/cc @fisherxu

/approve


Comparison between dates, times and strings, ...

Follow up from; https://github.com/hannes/duckdb-rfuns/pull/33#discussion_r1498806449

comparing dates to dates, or times to times is fine:
duckdbrfuns:::spaceship(Sys.Date(), Sys.Date())
#>                x          y   ==    !=     <   <=     >   >=
#> rfuns 2024-03-25 2024-03-25 TRUE FALSE FALSE TRUE FALSE TRUE
#> r     2024-03-25 2024-03-25 TRUE FALSE FALSE TRUE FALSE TRUE
duckdbrfuns:::spaceship(as.POSIXct(Sys.Date()), as.POSIXct(Sys.Date()))
#>                x          y   ==    !=     <   <=     >   >=
#> rfuns 2024-03-25 2024-03-25 TRUE FALSE FALSE TRUE FALSE TRUE
#> r     2024-03-25 2024-03-25 TRUE FALSE FALSE TRUE FALSE TRUE

but we get different results when comparing dates and times for some reason:
duckdbrfuns:::spaceship(Sys.Date(), as.POSIXct(Sys.Date()))
#>                x          y    ==    !=     <   <=     >    >=
#> rfuns 2024-03-25 2024-03-25  TRUE FALSE FALSE TRUE FALSE  TRUE
#> r     2024-03-25 2024-03-25 FALSE  TRUE  TRUE TRUE FALSE FALSE

Created on 2024-03-25 with reprex v2.1.0

We can always fall back to R in these weird cases, but this will have to be detected at the binding stage. So, if a function doesn't exist for specific arguments, it will work out fine.
Here, we should just omit the implementation with a comment.

Let me know if #66 does not work for you;

Can you expand on "this will have to be detected at the binding stage" ?

Evaluation stage: calling rel_to_altrep_df()
Binding stage: calling other rel_*() functions

Yes, failing at rel_to_altrep() is too late.
Can we supply our own binding logic instead of relying on duckdb for this? We might be able to rebuild a part of the infrastructure that currently does the binding for us, and then upstream if useful

I will have another look, but I'm not sure how to make it happen now.

You can define a custom binding function for a ScalarFunction and then just throw an exception if you don't like the argument types

IIUC, the bind_scalar_function_t bind argument to ScalarFunction::ScalarFunction() can help here.

You can define a custom binding function for a ScalarFunction and then just throw an exception if you don't like the argument types

That's essentially what we are doing.


Epic: prepare content and docs for technical preview

Documentation for the landing:

[ ] #382
[ ] #910
[ ] #911
[ ] #912
[ ] #913
[ ] #914
[ ] #915
[ ] #916
[ ] #917
[ ] #1471

Github:

[ ] update zenith README.md, link to zenith landing page with 'create on zenith' button
[ ] clean up zenith repo directory structure
[ ] create integration examples reporitory, cover with tests  to ensure nothing breaks

Blog posts content plan:

[ ] Welcome: team, mission what we have done since March
[ ] Separation of storage and compute (why bother, pros/cons) and tech overview

Rest:

[ ] terms of service
[ ] privacy policy
[ ] do we need a cookie consent?

see also #982 for the further blogposts plan

https://hackmd.io/


Unnamed pages are not movable

Exemplified in the screencast:
https://user-images.githubusercontent.com/4778481/120906251-4e166400-c647-11eb-9296-2706c5e895d5.mp4
OS: Linux
GT: v0.8.690

Should be fixed in the latest release


Fix KeyboardAvoidingView padding for android

On Android this:
<KeyboardAvoidingView behavior={'padding'} style={{ flex: 1 }}>
    <View style={{flex: 1}} />
    <TextInput style={{width, height}} />
</KeyboardAvoidingView>

produces incorrect result (because frame's height also changes when keyboard shows up), so bottomPadding: 0 for container should work fine.

By analyzing the blame information on this pull request, we identified @nicklockwood and @davidaurelio to be potential reviewers.

I'm not sure if this is right -- might it depend on the windowSoftInputMode?

Thanks for pointing out to https://developer.android.com/guide/topics/manifest/activity-element.html#wsoft. Maybe it really is.
I tried out react-native initdefault installation and still could reproduce that issue, though (ios works as expected). I tried to change android:windowSoftInputMode="stateUnspecified|adjustUnspecified" to other values, didn't notice any changes, but maybe I'm doing something wrong. Link to the repo https://github.com/ptmt/rn-android-keyboardavoidingview-padding
If it's just me I could easily fix it in userspace for now (by changing the behavior prop).

It's been a while since the last commit was reviewed and the labels show this pull request needs review. Based on the blame information for the files in this pull request we identified @nicklockwood as a potential reviewer. Could you take a look please or cc someone with more context?

I was having similar behavior. I changed my windowSoftInputMode to adjustResize and didn't choose a behavior for the KeyboardAvoidingView, and all my android devices play nice now. Also for iOS I am using padding.

This fix seems to solve my issue on Android!

I use this patch to solve my keyboard issue of post page on Android!

Can we detect the current input mode and do the correct thing automatically?

@satya164 Did you mean by detecting windowSoftInputMode? I guess we should be able to fix this by changing it manually at first, but I failed to achieve that.

@ptmt Yes. I think otherwise we might be breaking people's code who don't use adjustResize.
We should be able to detect it with,
getCurrentActivity().getWindow().getAttributes().softInputMode

@ptmt
1st of all, sorry for my poor English and thank you.

If the textInput was embedded in a container view with paddingBottom, the rest part of the content would be covered by the keyboard as not the same as iOS.
I've solved my problem like this:

Make my copy of 'KeyboardAvoidingView' and mod the 'relativeKeyboardHeight'.
`relativeKeyboardHeight(keyboardFrame: ScreenRect): number {
if (Platform.OS === 'ios') {
const frame = this.frame;
if (!frame) {
return 0;
}
const y1 = Math.max(frame.y, keyboardFrame.screenY - this.props.keyboardVerticalOffset);
const y2 = Math.min(frame.y + frame.height, keyboardFrame.screenY + keyboardFrame.height - this.props.keyboardVerticalOffset);
return Math.max(y2 - y1, 0);
} else {
return this.props.keyboardVerticalOffset;
}
},`

When using in Android, I give a 'keyboardVerticalOffset' value which equal to the distance from textInput.bottom to the bottom of the container to keep it look exactly as the same as iOS.

Hope these helps.

setting android:windowSoftInputMode to adjustResize works in my case!

Hi @ptmt - is this pull request still on your radar? It looks to me like there is some unresolved feedback from @satya164 and @cmmobiraoxin so I'm tagging this with "needs revision".

I believe this PR can be closed since it uses the wrong approach, and I have failed to fix it another way so far.

Thanks for letting me know.


Vcell example combine archive fails to run

This is the namespace issue discussed previously

I fixed the namespace of the Vilar example. I confirmed that VCell <IP_ADDRESS> runs the corrected file.
Note, VCell <IP_ADDRESS> appears to have other issues that prevent it from running this and other examples. See /biosimulators/Biosimulators_VCell#60.


ZONE module

This allows the zonefiles to be used as inputs, passing a *dns.Token to the callback as an alternative, and generalizes all other modules to obey the new interface. A new module, ZONE, is added that takes the zonefile, and does ALOOKUPs for each domain to its specified nameservers, making use of glue records. If IPs are found, more requests are not made to the subsequent nameservers.
The only catch is that we can get >1 output per domain, e.g. when the first nameserver fails. The only way to get around this would be to have some way to know that no more names are coming, and do more chan magic.
Looking for feedback.

The only catch is that we can get >1 output per domain, e.g. when the first nameserver fails. The only way to get around this would be to have some way to know that no more names are coming, and do more chan magic.

Why can't you just not output the first one? Couldn't we just lock on that name?
Otherwise, couldn't we just pass a list of tokens into the module at once that are all for the same domain and let the DoLookup function take care of that logic internally?

Why can't you just not output the first one?
Because there could be N nameservers for one domain

Couldn't we just lock on that name?
How could a token know it is the last one in this scheme?

Otherwise, couldn't we just pass a list of tokens into the module at once that are all for the same domain and let the DoLookup function take care of that logic internally?

We had this before, but you did not like how it led the framework to be less general- see the last PR for ZONE. I'm not particularly sold either way right now.

Yeah, I see what you mean. How ugly.

Also, re framework being less general, my concern was over you have static if statements in the framework that were dependent on a module, not how much data was passed to the DoLookup (or similar) call at any given time.
But, I do agree it's a bit ugly for the framework to group together. I think we can definitively say that sometimes outputting bad or repeated data is not a good option.

Maybe I'm missing something, but it seems like this could easily be avoided by just checking if it has been output already before outputting it:
def DoLookup(token)
	if domain in hashcache:
		return
	retv = lookup domain(token)
	output_already = cachehash.chkadd(domain, retv)
	if output_already:
		return
	return retv

What if the first output is a failure?
This may work if we forward the result instead of a bool though...
Sent from my iPhone

On Aug 13, 2016, at 1:40 PM, Zakir Durumeric<EMAIL_ADDRESS>wrote:
Maybe I'm missing something, but it seems like this could easily be avoided by just checking if it has been output already before outputting it:
def DoLookup(token)
if domain in hashcache:
return
retv = lookup domain(token)
output_already = cachehash.chkadd(domain, retv)
if output_already:
return
return retv
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or mute the thread.

If the first attempts fails, then it wouldn't get added to the cachehash and the second one would output it?

What about where two requests happen in parallel, and the one that returns first is a failure?
Sent from my iPhone

On Aug 13, 2016, at 4:45 PM, Zakir Durumeric<EMAIL_ADDRESS>wrote:
If the first attempts fails, then it wouldn't get added to the cachehash and the second one would output it?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or mute the thread.

I'm pretty sure that situation is handled here. You don't add anything to cachehash if you're not successful.

I do not see a path where a failure is outputted, in this more explicit edit. It will always fail out at the success check.

This is true---failures are not currently output. Do we need to output
failures though? This wouldn't be horrible either. We'd just need to have
an extra state in the cache hash and output failure on eviction.
On Sat, Aug 13, 2016 at 3:14 PM Benjamin VanderSloot <
<EMAIL_ADDRESS>wrote:

I do not see a path where a failure is outputted, in this more explicit
edit. It will always fail out at the success check.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/zmap/zdns/pull/36#issuecomment-239644227, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AAMSUNZ-olrM9Z9pFgFfbIHjGNLSejeeks5qfkHagaJpZM4JjnJV
.

That could be it. Not outputting is easy to change on this PR. I am just not sure about how to add a general CB that can output in golang. I guess I can give it a shot.

What do you mean general CB? like in the cachehash library?

Yeah, and one that will actually be useful. I haven't put much thought into it, I just see no register CB in golang cachehash.

On Aug 13, 2016, at 5:24 PM, Zakir Durumeric<EMAIL_ADDRESS>wrote:
What do you mean general CB? like in the cachehash library?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or mute the thread.

we'll have to add it
On Sat, Aug 13, 2016 at 3:27 PM, Benjamin VanderSloot <
<EMAIL_ADDRESS>wrote:

Yeah, and one that will actually be useful. I haven't put much thought
into it, I just see no register CB in golang cachehash.

On Aug 13, 2016, at 5:24 PM, Zakir Durumeric<EMAIL_ADDRESS>wrote:
What do you mean general CB? like in the cachehash library?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or mute the thread.

—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/zmap/zdns/pull/36#issuecomment-239644728, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AAMSULkQ-yYfQjQfJqDP0tyRhcrv51igks5qfkTtgaJpZM4JjnJV
.


Enabling SonarQube

That's it!

Codecov Report

Merging #95 into master will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master      #95   +/-   ##
=========================================
  Coverage     93.44%   93.44%
  Complexity      293      293
=========================================
  Files            33       33
  Lines           687      687
  Branches         84       84
=========================================
  Hits            642      642
  Misses           25       25
  Partials         20       20

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 f45ba64...c1e266e. Read the comment docs.


Support trunc function for python script

What type of enhancement is this?
User experience
What does the enhancement do?
trunc is temporarily removed in #2659 due to the signature change. We should find another way to support it again.
Implementation challenges
No response

due to the signature change

Could you elaborate it a bit? From what to what, and why it breaks things ..

Related change: https://github.com/GreptimeTeam/greptimedb/pull/2659/files#diff-329ba8f324e7dc1c3dbb07f61662e867afdd61d40abce9b44a3fd73ada219b11R556-R561

@waynexia Yeah I found the linked PR. It's a huge one and the related line seems:
diff --git a/src/script/src/python/pyo3/builtins.rs b/src/script/src/python/pyo3/builtins.rs
index 20d1c364415..6051a5307b9 100644
--- a/src/script/src/python/pyo3/builtins.rs
+++ b/src/script/src/python/pyo3/builtins.rs
@@ -90,7 +90,7 @@ pub(crate) fn greptime_builtins(_py: Python<'_>, m: &PyModule) -> PyResult<()> {
             floor,
             ceil,
             round,
-            trunc,
+            // trunc,
             abs,
             signum,
             exp,
@@ -281,7 +281,8 @@ fn sqrt(py: Python<'_>, val: PyObject) -> PyResult<PyObject> {

*/
bind_call_unary_math_function!(

sqrt, sin, cos, tan, asin, acos, atan, floor, ceil, trunc, abs, signum, exp, ln, log2, log10

sqrt, sin, cos, tan, asin, acos, atan, floor, ceil, abs, signum, exp, ln, log2,
log10 // trunc,
);

Not quite a useful info.


Reason on live

Describe the bug
The reason langage is indicated in the documentation.
.
But it is not not present on the live.

Reproducing
Go to https://comby.live and note
Expected behavior
Reason langage should be present on the combi.live.

Thanks for catching! I added Reason. There are a handful of others missing in that list, which I'll add a bit later :-)
Fixed in https://github.com/comby-tools/comby-ui/commit/a31d6ab7317ebb4a7d27b8932bf5e8931038bff6


There seems to be a bug at datasets/bases.py#L157.

At datasets/bases.py#L157, you directly pass caption_tokens to the function _build_random_masked_tokens_and_labels and caption_tokens has be modified in this function. Thus, the masked captions are also used in the sdm task and id task, which is inconsistent with the clarification in the paper.

Please see the reponse at #13

Hi, I would like to ask if you have the CUHK-PEDES dataset? If so can you send me a copy easily? My email<EMAIL_ADDRESS>Thanks!!


Opelline/draw interaction extend polygons

The Draw interaction has only support for extending the drawing of a previous LineString Feature (the extend function). Appending coordinates from a feature to a drawing has not been supported.
This branch contains a new appendCoordinates function, which supports appendign both when drawing Polygons and LineStrings, by appending linear coordinates, such as coordinates from a LineString feature or a LinearRing of a Polygon feature. The coordinates are appended to the set of previously drawn coordinates.
I have also created an example for demonstration.
(Edit: Updated the description according to the separation of the appendCoordinates and extend functions)

Hi,
I personally have a hard time seeing a use case for this. Isn't it prone to confusion to be able to clone the geometry in the same place and modify the new one? Besides, it looks like the new function you implemented is basically the equivalent of cloning the feature & extending it with the draw interaction. Sorry if I'm missing something here!

Hi,
Thanks for the comment: this was built for an actual use case, as the user needs to be able to follow the borders on a vector layer, without snapping each edge point by hand. This could be a border of a region, or some part of a neighbouring feature. This of course requires some app specific handling for doing the snapping and slicing of those line segments before they can be added to the current geometry.
The extend feature doesn't allow adding points in the other end of a segment, and it also doesn't work on polygons. I was first trying to implement this using the extend feature, but I got confused and thought it was buggy. It was only after runnign the automated build tests that I actually understood what the extend feature was for, and why it was working the way it was. So now I've clarified the feature documentation to separate the two.
One might think all this could be achieved by just taking the coordinate arrays to be joined, concatenate them as you like, and creating a new feature, which then could be used in the extend feature. However, this doesn't work with polygons, as it will break the draw interaction.

@jahow Please tell me if you have any other comments or feedback!

@jahow Please tell me if you have any further questions or comments on this!

Hi @opelline, thanks, I understand better now the limitations of the extend method. It indeed does make sense to be able to inject an arbitrary series of coordinates in the currently drawn feature, that would allow for nice user experience when drawing.
I think the API description could be a bit more clearer, ie:
  /**
   * Append coordinates to the end of the geometry that is currently being drawn.
   * This can be used when drawing LineStrings or Polygons. Coordinates will
   * either be appended to the current LineString or the outer ring of the current
   * Polygon.
   */

Also some remarks on the actual code:

this method will need a few dedicated tests
I'd be in favor or reusing addToDrawing_ and rework this private method a little to accept a coordinate instead of an event; this would insure consistency with the other behaviours of the interaction & avoid duplicating code/logic
you should also rebase this PR on master (the draw examples behave weirdly on this branch, the sketch feature is not visible)

But other than that I think this contribution is definitely valuable and interesting. Can someone else maybe confirm this should be pushed forward? @fredj maybe?
Thanks a lot @opelline

Hi @jahow , I've been quite busy for the past few weeks, but now I had the time to do the requested changes, and I'd like to reopen this.


Client-side password validation is inaccurate

At the moment, client-side validation checks whether the password given when creating a user is between 6 and 32 characters, and fails if it is not.
However the password is required to contain at least:

1 upper-case letter
1 lower-case letter
1 digit
1 non-alphanumeric character

So client-side validation can pass without the password being valid.
I'm submitting a PR shortly to fix this.

I don't want to rewrite this if the rules change 😄

I'll make sure to maintain it 😁

Fixed


Version string check does not accept major version 0

The NAMEID_VERSION_STRING (nameID=5) value must follow the pattern Version X.Y. Current value: Version 0.003;

Looking into the code, I think it is because the major version is 0. If this is intentional then the message needs to be clarified.

I agree


Fix update after alter table adding a blob type.

(this needs to be rebased changing also the tests numbers if #92 lands before).
This is similar to #68 but it fixes a panic after adding a blob type (also time
type and every other type saved as a blob).
Note: if using the provided test without the fix, the tests will enter an infinite loop in ql.go:1869 due to pc != db.cc.
Now I haven't looked why this happened. Looks like after panicking a commit or
another operation is done causing bad behavior.
Additionally aren't pc and db.cc pointers to TCtx and the equality check should
be done between their referenced values? (*pc != *db.cc)?

Thank you very much for the PR. It seems to me I should first have to dive into this and understand/fix the infinite loop before merging. I hope to get to it this weekend.


Confirm design were Any knowledge leaks all the way into Message

If you look at the C++, thinks like type.googleapis.com, and the type urls don't bleed all the way into the base message class, instead the Any implementation is the only thing that needs to know about these defaults: https://github.com/google/protobuf/blob/39f9b43219bc5718b659ed72a2130a7b2ce66108/src/google/protobuf/any.h#L53-L62.  When filling in an Any the type.googleapis.com can be the default or another call can be used to provide a custom value.
Likewise, our Message protocol has serializeAnyJSON, which shows the knowledge.  That should likely live in over in Any, or if really needed, Any can add it as an extension.

Took a quick scan of the Java Any support, they seem to have a type registry that comes into play so the things can be looked up and decoded during the JSON parse.  The helpers for pack() operations also take the type url with the default.  So this approach seems to keep everything out of the base message and in the helpers instead.
It's likely also worth pointing out, for a field that is an Any, C++ (and it seems Java) don't actually use the generated proto class.  They instead use a stand in with the extra methods and they hook it into serialization/parsings for the different formats as needed.

All moved out.


Power BI report for RIVM Corona dataset

Hi Jonathan,
As discussed by email, you can share my Power BI report for RIVM Corona dataset on your GitHub page, maybe with a reference to my blog-post (where I just uploaded v2 of the report)
https://worktimesheet2014.blogspot.com/2020/03/coronovirus-in-netherlands-power-bi.html
http://tiny.cc/ncwblz
regards,
Maarten van Reek
<EMAIL_ADDRESS>

Thanks, Done

where can I find it?
Op ma 16 mrt. 2020 om 08:49 schreef Jonathan de Bruin <
<EMAIL_ADDRESS>
Thanks, Done
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/J535D165/CoronaWatchNL/issues/27#issuecomment-599391495,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AKYMDKXRDXCXJOXOYEZWPZ3RHXK2BANCNFSM4LLPZ24A
.

Made a section with "interesting links" https://github.com/J535D165/CoronaWatchNL#interesting-links. It might be a bit hidden, but the data needs to be the main focus of the repo.
Are you interested in making a list of fatalities to add to the repo? I'm too busy to work on it right now (I'm working on a large scientific project regarding corona).

Ok, I saw the link, thanks.
I'm afraid I also don't have enough time to work on adding fatalities. I'm not even sure if RIVM offers this data?


vscode-extensions.matklad.rust-analyzer 0.2.384 out of date, missing security updates

Notes
rust-analyzer itself has been steadily updated in nixpkgs but nixpkgs' packaging of its bundled vscode extension, vscode-extensions.matklad.rust-analyzer has not been updated in >2 months.
Additionally, during this time, the extension's dependencies have been updated to remove at least two security vulnerabilities: https://github.com/rust-analyzer/rust-analyzer/pull/11420.
I would like to help keep this extension up to date, but I am unfortunately insufficiently familiar with nixpkgs' nodePackages infrastructure (and the extension's packaging's use of it) to prepare an appropriate PR.
Thus:
a) can someone who is familiar with the extension's packaging please update it, perhaps to 0.3.953 or to one of the recent git tags such as https://github.com/rust-analyzer/rust-analyzer/releases/tag/2022-02-22, to match rust-analyzer?
b) additionally, if/when you do, could you also add a paragraph of comments (maybe to the nixpkgs manual, if not directly in the code?) on what the steps are to update it so that this is more clear?
Thanks!
courtesy cc: @inicola, who has been improving the extensions' compatibility with NixOS.
Checklist

[x] Checked the nixpkgs master branch

[x] Checked the nixpkgs pull requests

Project name
nix search name: vscode-extensions.matklad.rust-analyzer

current version: 0.2.384
desired version: 0.4-dev
Notify maintainers

maintainers: @jonringer
Note for maintainers
Please tag this issue in your PR.

What I think happened here is that in https://github.com/NixOS/nixpkgs/pull/150775 the parts of the rust-analyzer package's update script that also updated the VS Code extension were deleted. I have a branch where I pull those deleted parts into a new update script for the extension itself. It's working for me locally, so today or tomorrow I will put up a PR (I encountered some issues relating to I think the fact that we're not on Rust 1.59.0 yet, and now I have to re-run the ~1h20m Node packages update).
I think for updating Node packages in general, it involves running the same generate.sh script as adding a new package. I might add that to the docs as well, I don't think it's very clear what the intended process is currently. As for this particular VS Code extension, I think adding the update script in the same folder should make it fairly clear how to update it.

@austinbutler -- thanks very much for both of these updates!


Block: Prepare repo

As someone setting up the project infrastructure, I want to have a detailed guide on how to prepare the repo so that I can get this done fast and without errors.

create repo
push required branches

This task includes preparing an example repo with all steps done, so it can be cloned right away.

https://app.zenhub.com/files/396782963/9e284055-48a8-407a-9fe2-b7c7c56fc8e4/download


Beta player consistently crashes upon launch

The launcher will constantly crash after bloxstrap indicates roblox is opening. This will happen a exact amount of 3 times for me until the beta player refrains from freezing and crashing and will run normally. Not sure if this a problem for anyone else, but this has been happening since the last patch.

The issue I'm pretty sure stems from the recent addition of the mobile UI update, as to why I think this is because before the player crashes each time it will launch first as the mobile interface and then freeze up and crash all while  still stuck on mobile UI.

See https://github.com/pizzaboxer/bloxstrap/wiki/Information-on-Roblox-Crashes.

See https://github.com/pizzaboxer/bloxstrap/wiki/Information-on-Roblox-Crashes.

Thank you will look into this!


Adds support for raspbian stretch

Overview
Basically uses the jessie repo for raspbian stretch due to raspbian
jessie download links being taken off the official raspbian site.
NOTE
This is only a temporary fix and should not be considered permanent
Signed-off-by: Eli Uriegas<EMAIL_ADDRESS>ping @StefanScherer @alexellis

Excellent! That will make many Raspbian users happy to get started with Docker 17.05 very easy.


Update sdks-overview.md

Add CLI to supported languages.

#sign-off

@FaithOmbongi, @msewaweru, kindly merge.


Add a default collapse effect  for outline

Some level is very deep, one-time all is not suitable for viewing, only one layer, easy to have a big picture

I mention the pr may not be very formal, I changed a vsix for their own use, the pr can be closed, hope later versions can consider to join this function ~


Consider removing increment / decrement unary operators

In replace unary operator, it seems to me that it is worth removing increment / decrement operators, so:
++x -> x
--x -> x
x++ -> x
x-- -> x
However, we should think through whether (a) this will work from a typing perspective (l-values, r-values, etc.), and (b) whether this leads to redundancy due to operator insertion.

After some thought, I don't think we need to do this because:

Removing ++ is akin to adding -- (and vice versa)
We have a mutation for adding -- (and ++) to every r-value right before it is converted to an l-value

So this is pretty much covered.


Download broke

It broke when I rolled from 112 to 117.
It's cuz of https://chromium-review.googlesource.com/c/devtools/devtools-frontend/+/4381791
exception:

Error: Failed to execute 'showSaveFilePicker' on 'Window': Cross origin sub frames aren't allowed to show a file picker.

chromium source: https://source.chromium.org/chromium/chromium/src/+/main:third_party/blink/renderer/modules/file_system_access/global_file_system_access.cc;l=189;drc=dbf0295f91fe41035e15d8d6d9e7cdc7f674d2a5

Will either have to check for this case within dt or trace-cafe can provide its own download button.

We can use the real button now that https://chromium-review.googlesource.com/c/devtools/devtools-frontend/+/5146853 is in
dda7843aecd8f134ff7153e65bf944fc8214258a reverted the jank replacement download button. yay


How to test a Hyper service?

What kind of test harness do I need to test against call of my service, e.g. a POST request? I'm relatively new to Rust and having a hard time testing a Hyper service. I tried to call let response = service.call(req); and got an error about, "must run in tokio runtime."
In Clojure I'm able to write request handlers as plain functions that take a request as a hash-map (e.g. {:method :post :uri "/upload" :body ...}) and return a response map (e.g. {:status 201 :body "Uploaded"}). I would like to be able to have a similar convenience using Hyper in Rust. Thanks!

Short answer: I'd suggest using Tokio's block_on method. I haven't tested this code, but it should look something like this:
#[test]
fn post_endpoint() {
  use tokio::runtime::Runtime;
  // Create the runtime
  let mut rt = Runtime::new().unwrap();
  // `resp` assumed to be defined elsewhere.
  let resp = rt.block_on(resp);
}

Long answer—every future created by Hyper must be run in the context of Tokio's event loop. Unlike, say, Node, there isn't a default/global event loop in Rust, so you have to create instances/handles to Tokio yourself.
(If you can send a PR that uses this to test endpoints, that'd be greatly appreciated!)

As you've noticed, you can largely just test your Service without needing to start up a full server. However, depending on what that service is doing, you may need a runtime active. Like if you service spawns sub tasks/futures (like the hyper Client does), or using tokio-fs, which needs the threadpool available.
But in other examples, you can just call it. The Hello World example in hyper can just be called, as you described.


ng new with routing does not create app-routing.module.ts

6.0.0-rc.5
<!--
Output from: `ng --version`.
Angular CLI: 6.0.0-rc.5
Node: 8.11.1
OS: win32 x64
Angular: 6.0.0-rc.5
... animations, cli, common, compiler, compiler-cli, core, forms
... http, language-service, platform-browser
... platform-browser-dynamic, router

Package                           Version
-----------------------------------------------------------
@angular-devkit/architect         0.5.7
@angular-devkit/build-angular     0.5.7
@angular-devkit/build-optimizer   0.5.7
@angular-devkit/core              0.5.7
@angular-devkit/schematics        0.5.7
@ngtools/json-schema              1.1.0
@ngtools/webpack                  6.0.0-rc.5
@schematics/angular               0.5.7
@schematics/update                0.5.7
rxjs                              6.0.0-uncanny-rc.7
typescript                        2.7.2
webpack                           4.5.0
-->

Repro steps

ng new myApp--style scss --prefix app --routing true
ng build

Observed behavior
ERROR in src/app/app.module.ts(4,34): error TS2307: Cannot find module './app-routing.module'.

Desired behavior

Mention any other details that might be useful (optional)

It worked for me with this version:
Angular CLI: 7.0.4
Node: 8.11.3
OS: win32 x64
Angular: 7.0.2
... animations, common, compiler, compiler-cli, core, forms
... http, language-service, platform-browser
... platform-browser-dynamic, router

Package                           Version
-----------------------------------------------------------
@angular-devkit/architect         0.10.4
@angular-devkit/build-angular     0.10.4
@angular-devkit/build-optimizer   0.10.4
@angular-devkit/build-webpack     0.10.4
@angular-devkit/core              7.0.4
@angular-devkit/schematics        7.0.4
@angular/cli                      7.0.4
@ngtools/webpack                  7.0.4
@schematics/angular               7.0.4
@schematics/update                0.10.4
rxjs                              6.3.3
typescript                        3.1.6
webpack                           4.19.1

@code2nguyen please open a new issue if you're still facing the problem with latest CLI. Thanks! :)


Text pasted from other apps is not sanitized

Describe the bug
When text or "content" is pasted into the editor from another app (browser, Word, VS Code, etc) its formatting is imported along with it, even if the editor does not support it.
To Reproduce

Select some content from a website, and paste it into a paragraph.
Observe that the text keeps its original formatting, and elements like links, images and videos are displayed inside OmniaWrite.
Switch scenes back and forth; the content is eventually sanitized.

Expected behavior
Pasted text should be scrubbed of all unsupported formatting, i.e all tags except <b>, <i>, <u>. Line breaks should also be handled somehow, as they are (currently) not allowed inside paragraphs.
Screenshots

Fixed 🥳


German Cloud has different Ubuntu images

So the deployments fail because the specific image version isn't in German Cloud. I think this should be fixed in the platform rather than making every single piece of software and vendor in Azure handle different images for different clouds.

$ az group deployment create     --name="kubetest"    --resource-group="kube_test"   --template-file="/Users/firur/acs/src/github.com/Azure/acs-engine/_output/Kubernetes-12051246/azuredeploy.json" --parameters<EMAIL_ADDRESS>At least one resource deployment operation failed. Please list deployment operations for details. Please see https://aka.ms/arm-debug for usage details. {
  "error": {
    "code": "ImageNotFound",
    "target": "imageReference",
    "message": "The platform image 'Canonical:UbuntuServer:16.04-LTS:16.04.20170307' is not available. Verify that all fields in the storage profile are correct."
  }
}  Correlation ID: a655ce1d-64cf-489b-8775-2c9bb2073013

Presumably distribution of the Ubuntu images is far from our control so we'll need to be able to handle different image URNs for the different locations/clouds.

cc: @raulfiru

@raulfiru Actually, for us to fix this, I think we'll need you to tell us what Ubuntu 16.04 images are available... Can you let us know which one you ended up using? az vm list-images or something similar should be a starting point for getting a list of the Ubuntu images... Thanks.

i don't fully get it.. since i get multiple results...
az vm image list --all give me this:

"urn": "Canonical:UbuntuServer:16.04-LTS:16.04.201701130",
"urn": "Canonical:UbuntuServer:16.04.0-LTS:16.04.201604203",
"urn": "Canonical:UbuntuServer:16.04.0-LTS:16.04.201605161",
"urn": "Canonical:UbuntuServer:16.04.0-LTS:16.04.201609071",
"urn": "Canonical:UbuntuServer:16.04.0-LTS:16.04.201610200",

az vm image list (cached)

"urn": "Canonical:UbuntuServer:16.04-LTS:latest",

and the portal has this comment:

Ubuntu Server 16.04 LTS amd64 20170330 Public Azure, 20170330 Azure China, 20170113 Azure >Germany, 20161221 Azure Gov.

the one i used and worked was:  16.04.201701130 (the one in the portal)

This isn't the most fun. @anhowe do you have any suggestions? Otherwise we're going to have to make these sorts of things part of the internal model as well... and as I'm looking at doing this, I realize we never fixed where the defaults are set... we need time to look into this and fix it correctly...

As a shorter-term solution, @szarkos may be able to publish the image we use today into GermanCloud. I'll update more as I hear about this.


RZAPIRenderer::DrawIndexed doesn't work with ImGuiRenderer uses wrong scissor rect

RZAPIRenderer::DrawIndexed() is not able to use the Scissor rect set throught RZAPIRenderer or RZCommandBuffer (I've personally tried both the design changes) both vkCmdSetScissor and vkCmdDrawIndexed need to be set simultaneously in the same scope for it to work. The reason is that scissor rect is not properly set before the draw call for indexed rendering, it's losing that information.
 for (uint32_t i = 0; i < imDrawData->CmdListsCount; ++i) {
                const ImDrawList* cmd_list = imDrawData->CmdLists[i];
                for (int32_t j = 0; j < cmd_list->CmdBuffer.Size; j++) {
                    const ImDrawCmd* pcmd = &cmd_list->CmdBuffer[j];
                    RZDescriptorSet* set = (RZDescriptorSet*) pcmd->TextureId;
                    RZAPIRenderer::BindDescriptorSets(m_ImGuiPipeline, cmdBuffer, &set, 1);
                    // Fix this for Vulkan

                    RZAPIRenderer::SetScissorRect(cmdBuffer, std::max((int32_t) (pcmd->ClipRect.x), 0), std::max((int32_t) (pcmd->ClipRect.y), 0), (uint32_t) (pcmd->ClipRect.z - pcmd->ClipRect.x), (uint32_t) (pcmd->ClipRect.w - pcmd->ClipRect.y));
                    vkCmdDrawIndexed(*cmdBuf, pcmd->ElemCount, 1, indexOffset, vertexOffset, 0);
                    //RZAPIRenderer::DrawIndexed(cmdBuffer, pcmd->ElemCount, 1, indexOffset, vertexOffset, 0);
                    indexOffset += pcmd->ElemCount;
                }
                vertexOffset += cmd_list->VtxBuffer.Size;
            }

ImGui only work if RZAPIRenderer::DrawIndexed is not used
============================================================
Razix Engine Stats
Version : 0.30.0
Configuration : [Debug | Release | Distribution]
Release Stage : Development
Release Date : 6-3-2022
OS : Windows 10 Home
Processor : Intel(R) Core(TM) i7-9750H CPU @ 2.60GHz 2.60 GHz
Vulkan API Version : 1.3.194
GPU Name : NVIDIA GeForce RTX 2060
Vendor ID : 4318
Device Type : DISCRETE GPU
Driver Version : 511.316.0

 RZAPIRenderer::DrawIndexed Doesn't work with ImGui Renderer

Workaround :   I don't see putting such hacky stuff in here, I don't want to be a bitch about making everything super decoupled, use macros and GetRenderAPI to make this work, this doesn't affect the performance at all. Hence this is OK!
Will be investigated by ME only it's a major problem when dealing with multi-viewport rendering.
Until then, If anyone want to solve this I will give you 69 jelly beans as a gift. Free shipping to anywhere in the world.


How can we be transparent with anti-harrassment policies?

How do we communicate the policies around harassment?
How can someone report harassment to organizers?
In the unlikely event that an issue comes up with an organizer how can someone escalate this for mediation?

How do we communicate the policies around harassment?
How can someone report harassment to organizers?
In the unlikely event that an issue comes up with an organizer how can someone escalate this for mediation?

This year at my event I issued a policy. That was sent as a separate email to all attendees, speakers, and sponsors.  For sponsors, it was printed out and left at their tables.  The policy included how to report an incident. A room was reserved to handle any complaints and a male and female representative were designated to handle any complaints. It was easy.
The current process is up to each and every event to handle on their own.  I have done a few sessions collecting feedback and working with a few other SQLSaturday organizers so we could work together and feel supported.

I would advocate for a standard conduct/anti-harrassment policy.
Any event that decides not to use it, or decides to use a different policy, should make their reasons for not following the standard policy very clear to all attendees.

When I set up satRdays, we started with the policy from RLadies https://knowledgebase.satrdays.org/diversity/

@Alex-Yates  The current set up has a policy, the policy is vague and is likely written to protect the overseeing umbrella.  We did have that option turned on.  I think it is still currently optional.  We wrote one that was based on several policies that were suggested.
During the SQLSat sessions, I did in 2019 it was discovered that a lot of folks would like to see to promotion of positive behaviors not a list of things that are not tolerated. Also, the current setup gives you no feedback on how to set up a policy, plan, and what to do if something happens. By nature, they can't tell you that because then they could be liable.

I prefer events to be mostly independent, but still held to a common standard. To implement that, my proposal would be a baseline policy / guidelines that event organizers are free to use. They can use their own but it has to include or exceed all elements of the baseline. And this baseline policy would not be anti-harrassment only, it would also be very explicit on inclusion.
Enforcement / dealing with complaints should be done locally, by the event organizer. If an event doesn't enforce their policy, then the ultimate penalty would be to lose eligibility to call themselves a DataSaturdays event, and lose access to whatever resources DataSaturdays makes available to events.

My 2 cents is that the Calgary Data User Group uses a policy derived from confcodeofconduct.com.

Gah. I missed this issue when I created #47.
In that issue, I list these requirements for an event Code of Conduct:

The event-specific supplement should include who to report violations to, and how/where to contact them.

Reporting/assistance should be a direct, unambiguous path from the Code of Conduct. (ie, a monitored email address, a front desk person at an in-person event, etc)
Provide guidance to event organizers on eliminating single points of failure for individuals seeking help in reporting problems. Event organizers are often busy & not able to provide a fast initial response

The event Code of which is featured prominently & is easy to find. When an attendee is "in crisis" as the victim of harassment, it should not be difficult to find the Code of Conduct (and hence the path to reporting/assistance)

I think it's important & fair to have a baseline code of conduct that all events are held to. Each event will need to have a supplement to include reporting details for their event. That supplement could also be used for other additions to the baseline Code of Conduct. However, if an event organizer disagrees with the baseline Code of Conduct, they would be free to host their event elsewhere without a Code of Conduct, or with a different one. Allowing any event to have any code of conduct they wish (including one that may be contrary to the community's baseline standards), isn't a scalable approach if we want to ensure folks are able to bring their whole self to the community.


Attributes missing from module systemd.network

Describe the bug
There r attributes provided by systemd-networkd but is missing from module systemd.network.
Steps To Reproduce
For example, the attribute
Group =

is provided by systemd-networkd.
[Link] Section Options
The [Link] section accepts the following keys:

MACAddress=
The hardware address to set for the device.

MTUBytes=
The maximum transmission unit in bytes to set for the device. The usual suffixes K, M, G, are supported and are understood to the base of 1024.

Note that if IPv6 is enabled on the interface, and the MTU is chosen below 1280 (the minimum MTU for IPv6) it will automatically be increased to this value.

ARP=
Takes a boolean. If set to true, the ARP (low-level Address Resolution Protocol) for this interface is enabled. When unset, the kernel's default will be used.

For example, disabling ARP is useful when creating multiple MACVLAN or VLAN virtual interfaces atop a single lower-level physical interface, which will then only serve as a link/"bridge" device aggregating traffic to the same physical link and not participate in the network otherwise. Defaults to unset.

Multicast=
Takes a boolean. If set to true, the multicast flag on the device is enabled. Defaults to unset.

AllMulticast=
Takes a boolean. If set to true, the driver retrieves all multicast packets from the network. This happens when multicast routing is enabled. Defaults to unset.

Promiscuous=
Takes a boolean. If set to true, promiscuous mode of the interface is enabled. Defaults to unset.

If this is set to false for the underlying link of a "passthru" mode MACVLAN/MACVTAP, the virtual interface will be created with the "nopromisc" flag set.

Unmanaged=
Takes a boolean. When "yes", no attempts are made to bring up or configure matching links, equivalent to when there are no matching network files. Defaults to "no".

This is useful for preventing later matching network files from interfering with certain interfaces that are fully controlled by other applications.

Group=
Link groups are similar to port ranges found in managed switches. When network interfaces are added to a numbered group, operations on all the interfaces from that group can be performed at once. Takes an unsigned integer in the range 0…2147483647. Defaults to unset.

RequiredForOnline=[¶](https://systemd.network/systemd.network.html#RequiredForOnline=)

(pasted from freedesktop documents)
However, such a key is absent from https://github.com/NixOS/nixpkgs/blob/nixos-unstable/nixos/modules/system/boot/networkd.nix, in which it should take a place in "sectionLink = "

I have found that neither of the following have the required effect:
systemd.network.networks.wg0.networkConfig.IPForward = true;
systemd.network.networks.wg0.networkConfig.IPForward = "ip4";

giving
# cat /proc/sys/net/ipv4/ip_forward
0

whereas, using this in the same module instead:
boot.kernel.sysctl."net.ipv4.ip_forward" = 1;

gives
cat /proc/sys/net/ipv4/ip_forward
1


[ ASE ] Investigate sourcing docker username/password used for WebApp CD from KeyVault

Description
As a developer, I'd like to not have credentials for my ACR visible in the Azure Portal. Instead I'd like to source them from KeyVault so that I don't leak secrets
Acceptance Criteria
Reference: [Done-Done Checklist] (https://github.com/Microsoft/code-with-engineering-playbook/blob/master/Engineering/BestPractices/DoneDone.md)

[ ] Determine if sourcing the secrets from KeyVault is possible
[ ] If it is possible, write implementation story

Resources
Relevant Documentation
Tasks
Stories are intended to be completed in a single sprint; if task breakdown creates addition work then team should discuss promoting the Story to an Epic.
Reference: [Minimal Valuable Slices] (https://github.com/Microsoft/code-with-engineering-playbook/blob/master/Engineering/BestPractices/MinimalSlices.md)
Reference: [How to Write Better Tasks] (http://agilebutpragmatic.blogspot.com/2012/04/splitting-story-into-tasks-how-to-write.html)
Assignee should break down work into tasks here

Using key vault to store the secrets for ACR Service Principle works. Calling this done...


Expect search for BadKeyException would yield json_annotation towards the top

URL: https://pub.dartlang.org/packages?q=BadKeyException
Do we take into account ClassCasing or memberCasing when sorting search results? Might be good to prioritize API search in these cases?
See https://pub.dartlang.org/documentation/json_annotation/latest/json_annotation/BadKeyException-class.html

@kevmoo: we've released a new version with the fix. Does this match your expectation?

Looks good!


[Feature] Phoenix HTML 4.0 not supported

Versions Used
Kaffy: 0.10.2
Phoenix: 1.7.10
Elixir: 1.16.0
What's actually happening?
==> kaffy
Compiling 26 files (.ex)

== Compilation error in file lib/kaffy/resource_error.ex ==
** (RuntimeError) use Phoenix.HTML is no longer supported in v4.0.

To keep compatibility with previous versions, add {:phoenix_html_helpers, "~> 1.0"} to your mix.exs deps
and then, instead of "use Phoenix.HTML", you might:

    import Phoenix.HTML
    import Phoenix.HTML.Form
    use PhoenixHTMLHelpers

What should happen instead?
Update for Phoenix HTML 4.0 https://hexdocs.pm/phoenix_html/changelog.html
Screenshots
If applicable, add screenshots to help explain your problem.

Well.
Not sure if this is to classify as a bug.
Im running
Kaffy: 0.10.2
Phoenix: 1.7.10
Phoenix_live_view: 0.20.3
Phoenix_html: 3.3
Elixir: 1.16.0
and I have not encountered the runtime error you have - Trying to upgrade to Phoenix_html: 4.0 is stopped - as kaffy depend on Phoenix_html: 3.x
Bumping the version of Phoenix_html: 3.x to 4.x in kaffy - I guess the solution then is to do what the changelog and runtime error suggest. Change the use of Phoenix.HTML to import...... and add the dependency phoenix_html_helpers, "~> 1.0"
import Phoenix.HTML
import Phoenix.HTML.Form
use PhoenixHTMLHelpers

I dont have any plans to bump up to phoenix_html 4.0 in any current project for the moment. Perhaps this would be a perfect moment for you to test making the change in kaffy repo - bumping the versions and make a pull request?
Otherwise, test to downgrade Phoenix_html and follow the instructions in the readme.
Happy hunting.

We can just continue to use Phoenix HTML 3.3 for now.
I updated the title to classify it as a feature request, but I can't modify the label myself.

@nathany-copia I made a pull-request solving the issue you reported!
Would be awesome if you could test it out and report back if you find any bugs/errors.
Pull request #306

Thank you @drobban.
My server and tests are all working after updating a few spots from use Phoenix.HTML to the following:
import Phoenix.HTML
import Phoenix.HTML.Form
use PhoenixHTMLHelpers

Though I must admit, I'm not using Kaffy very heavily, but what I saw continued to work.
I used the line {:kaffy, github: "drobban/kaffy", ref: "fd0ee0b"} in mix.exs because I like to lock down the commit being pulled in.

Smart move 😅 didn't really think so much about until now after you mentioned it. Pinning to commit is definitely safer and less risk of headaches.

Hi! Any plans to merge and release that change? :)

Hi! Any plans to merge and release that change? :)
Would guess that @aesmail will have a look at it when he has time over for it.
At mean time, it would be great if you could test that change out and report back - Super simple to do, just specify the commit in your deps list. =)

@drobban thanks for the answer :) I plan to test that branch in the next week and give you feedback :)

@phortx  as @nathany-copia pointed out

"I used the line {:kaffy, github: "drobban/kaffy", ref: "fd0ee0b"} in mix.exs because I like to lock down the commit being pulled in."

Is a much smarter move then to use branch as I did in my example... In that way you wont be surprissed by any changes in that branch that I might make.

It's probably best to make a fork and use that. Otherwise the branch (and commit) could be deleted (such as when merged) and break the build.

is this repo still maintained?

is this repo still maintained?

it is as far as i know. Why do you ask?

Any updates?

Any updates?

I guess if there where any to report, you would of seen it the relevant issue topic.
Feel free to read this until there is any updates.
https://github.com/sponsors/aesmail

works for me now

Thank you.
I'm using {:kaffy, github: "aesmail/kaffy", ref: "d864ea1"} which is working here.


markdown: Accept raw HTML

Please follow the rules of the original Markdown and permit the inclusion of raw HTML elements in the documentation. This will simplify a lot the translation of the Doxygen’s markup.
See: http://daringfireball.net/projects/markdown/syntax#html

closed by #1497


Is this project only for Live2D-based character ? Not for VRoid ?

Excuse me, I wanna ask for character bridge of this project. I've seen the readme for example, you demonstrate it with VTube Studo where using Java-based Live2D. About that, this project will compatible with VRoid or not ? They using Unity-based 3D character from pixiv. Tell me is compatible or not.

Excuse me, I wanna ask for character bridge of this project. I've seen the readme for example, you demonstrate it with VTube Studo where using Java-based Live2D. About that, this project will compatible with VRoid or not ? They using Unity-based 3D character from pixiv. Tell me is compatible or not.

Hello
actually there is a reason why I use VTS
pairing between audio and character
as far as I know, only VTS software is capable of pairing audio with character mouth rigs.
so that's why I recommend using VTS, but if you have ideas for using other software, please try it as well as a reference for the further development of this project.
Thank you also for the open issue ^w^


Aspects don't intercept the methods of NSFileHandle

Hello,
I tried to intercept the methods of NSFileHandle like that :
[NSFileHandle  aspect_hookSelector:@selector(writeData:) withOptions:AspectPositionAfter usingBlock:^(id info, NSData *data){
NSLog(@"writeData: after | object:%@ | instance:%@ | class:%@",info,[info instance],[[info instance] class]);
} error:NULL];
[NSFileHandle aspect_hookSelector:@selector(writeData:) withOptions:AspectPositionBefore usingBlock:^(id info, NSString *data){
NSLog(@"writeData: before | object:%@ | instance:%@ | class:%@",info,[info instance],[[info instance] class]);
} error:NULL];
NSString *docPath=[NSSearchPathForDirectoriesInDomains(NSDocumentDirectory, NSUserDomainMask, YES)objectAtIndex:0];
NSString *filePath=[docPath<EMAIL_ADDRESS>NSString *resultLine=[NSString stringWithFormat:@"coucou c dhekra"];
NSString *newFilePath = [NSHomeDirectory<EMAIL_ADDRESS>NSFileHandle *handler = [NSFileHandle fileHandleForUpdatingAtPath:filePath];
NSLog(@"newFilePath %@ %@",newFilePath,[resultLine dataUsingEncoding:NSUTF8StringEncoding]);
if(!handler) {
handler= [NSFileHandle fileHandleForWritingAtPath:newFilePath];
}
[handler seekToEndOfFile];
[handler writeData:[resultLine dataUsingEncoding:NSUTF8StringEncoding]];
But i didn't have a result why??

Things are probably different for toll-free bridged classes or class clusters. Feel free to investigate more. Closing this for now as using aspects for file writing isn't a use case I'd recommend.


variable value not consistent within 1 rule execution

my g_dev_units variable's value is not consistent within execution of a rule, on the rule page.
it appears in the subject and in the body, and has different values
{{if gt (.Eval .Alert.Vars.g_max_erratic_warn) -1.0}}

Erraticness of {{.Alert.Vars.tpl_unit}}, Global

Erraticness - increased deviation - could be indicative of a spike or drop.
Low values up to {{.Eval .Alert.Vars.g_max_erratic_warn}} are ok. {{.Eval .Alert.Vars.g_max_erratic_crit}} are critical

        {{if gt (.Eval .Alert.Vars.g_dev_units) (.Eval .Alert.Vars.g_max_erratic_crit)}}

        {{else if lt (.Eval .Alert.Vars.g_dev_units) (.Eval .Alert.Vars.g_max_erratic_warn)}}

        {{else}}

        {{end}}
            {{.Eval .Alert.Vars.g_dev_units}}  <-----------

         {{.Graph .Alert.Vars.g_reqs }}

        {{end}}

in subject:
    subject =`
    {{.Last.Status}}: {{.Alert.Name}} :

    Global {{.Alert.Vars.tpl_unit}} {{.Eval .Alert.Vars.g_strength | printf "%.3f"}}

   {{if gt (.Eval .Alert.Vars.g_max_erratic_warn) -1.0}}
       - erraticness {{.Eval .Alert.Vars.g_dev_units | printf "%.3f"}}<-----------
   {{end}}

 | {{.Eval .Alert.Vars.s_med_issues | printf "%.0f"}} {{.Alert.Vars.tpl_s_tag_plural}} with dropped median:
        {{range $r :=.EvalAll .Alert.Vars.s_med_bad}}
        {{ if gt $r.Value 0.0}}
             {{ index $r.Group $.Alert.Vars.tpl_s_tag }}
         {{end}}
    {{end}}`

note, most of the time it works fine. like right now i'm seeing:
normal: requests_by_country : Global Web requests 0.883 - erraticness 9.186 | 0 countries with dropped median:
....
Erraticness - increased deviation - could be indicative of a spike or drop. Low values up to 2 are ok. 3 are critical
9.18607582825225

A Theory:
When the notifications are sent, we re-evaluate the expressions with the
time set to the same time the alert triggered. Since the data is not passed
directly, there could be some discrepancies if the data in graphite has
changed (for the same time period - think backfilled or aggregated) between
trigger and notification time.
Does that sound possible?
On Fri, Feb 6, 2015 at 6:14 PM, Dieter Plaetinck<EMAIL_ADDRESS>wrote:

note, most of the time it works fine. like right now i'm seeing:
normal: requests_by_country : Global Web requests 0.883 - erraticness 9.186 | 0 countries with dropped median:
....
Erraticness - increased deviation - could be indicative of a spike or drop. Low values up to 2 are ok. 3 are critical
9.18607582825225
—
Reply to this email directly or view it on GitHub
https://github.com/bosun-monitor/bosun/issues/700#issuecomment-73330826.

another way you can tell the discrepancy (i think):
$g_max_erratic_warn = 2
    warn = $s_med_issues > 0 || ($g_max_erratic_warn != -1 && $g_dev_units > $g_max_erratic_warn)

note in my previous post how g_dev_units is 9.1 and yet the alert claims "normal", it should be warning.
@kylebrandt that might in theory be possible, but does it apply here? alert emails are not involved here, i see the inconsistency within 1 single rule test execution on the rule page

#715 should address this, along with Matt's work to compute the templates in the alerting run

this should be fixed now since #715 is merged. will close this

Reopen if still present.

orly? Screenshot?

wrong bug... know which one was for the notification not matching the the trigger? Body and subject match, but shouldn't have triggered:

Or rather I should say based on the notification it shouldn't have triggered - probably was in a state to trigger, but notification didn't match trigger condition.

Did this come after building a new prod bosun? There were 39 changes there. Could have been because of one of them?

Yes. this was after the build, was 3ish EDT, and confirmed that bosun did restart at that time on the server. Can verify this is bosun as well:


max (max) steps - 50,000

What if a person specifies 10,000,000 steps?  Let's have a realistic limit.   50,000 is good enough - 20,000 is the "normal" max steps.

@johnml1135 If the user specifies a number of steps larger than the max amount of steps, should we fail the job and give them a chance to reenter a max steps that is under the limit? Or should we just automatically change it to the upper limit of 50000 while logging a warning without failing? The second option would be a smoother experience, although it wouldn't be as transparent to the user since I'm not sure that users will be able to see warning logs in machine.py.

We should throw an error if they are over the limit.


Set Font Family programmatically

Hi, Smeuli!
Thanks for your awesome component, I like styling and how it works out of the box.
The only issue I have for now is that I don't see any option to set active font family after component is already rendered on the page. Could you help, please?
Would be nice to have some prop like activeFont that could be changed through React app state
<FontPicker
   activeFont={someFont}
   defaultFont={defaultFont}
   apiKey={GOOGLE_API_KEY}
   onChange={font => this.handleFamilyChange(font.family)}
/>

Hi Alexander,
Thanks again for opening the issue.
I have made the changes to the component and published version 2.0. You can read here what has been changed exactly, and I have added an example to the README.
I hope this helps, otherwise let me know! :)


Remove tracing::instrument from apply_debug_overrides

Motivation
When logs are configured with span events, for example:
tracing_subscriber::fmt()
    .with_max_level(Level::INFO)
    .with_span_events(FmtSpan::CLOSE)
    .finish();

A warning is always logged in Config::infer() because apply_debug_overrides is always called, and has #[tracing::instrument(level = "warn")].
Solution
Remove #[tracing::instrument(level = "warn")] because I'm not sure how this span is useful.
Alternate Solution
Only call apply_debug_overrides when overrides are defined.
Or add skip(self) (#[tracing::instrument(level = "warn", skip(self))]) to minimize output.

The deny error has a fix in the resource scope pr, feel free to cherry pick that in.


average_over_time() expression function doesn't take vectors or vector components

Describe the bug
We have a number of expressions that compute something over time. These are called the time iterator functions. An example is
average_over_time(<var> [, "pos_cmfe", <fillvar-for-uncovered-regions>] [, start-time-index, stop-time-index, stride])

However, I tried this function on direction vector variable of wave.silo like so...
da = average_over_time(direction, 0, 50, 5)

and then did a vector plot of da and it fails causing a segv in the engine.
Next, I tried it on each component of direction like so...
da0 = average_over_time(direction[0], 0, 50, 5)
da1 = average_over_time(direction[1], 0, 50, 5)
da2 = average_over_time(direction[2], 0, 50, 5)
da = {da0,da1,da2}

doing a vector plot of da. That fails in expression parsing not recognizing direction[0]. I also tried above with the expression syntax delimiters < and > and that didn't work either.
I susepct same may be true of other time iteration expression functions. All should be tested for this.
I rated impact as medium instead of low because doing time iteration expressions on vectors by breaking into components is actually very much slower than operating on the vector variable.
To Reproduce
see description above.
I tried only on OSX. However, I think the problem is on not OS specific.

Seems related to #3547


[2.0.0] Text box for typing messages in is too small

I don't think it's a Facebook issue because it was bigger in 1.5.0. Now it's on the same line with the emoji buttons and such; they used to be on their own line.

Thanks for reporting
@Aluxian

I think it's back to full width now? Please try the latest version: https://github.com/aluxian/Messenger-for-Desktop/releases
Is this still an issue?


Contribution Guidelines link is broken

I was looking on how to start contributing to this project. I saw this issue here:
"Contributing
Contributions are very welcome! Check out the Contribution Guidelines for instructions."
The hyperlink that is embedded to "Contribution Guidelines" is broken. As i click it, its leading me to "404" error.

Good catch. Fixed now!


One authorization for two applications

I use Azure AD, OpenID Connect and Microsoft.Owin for authorization users in my two ASP.NET MVC 4.X applications. Is it possible to use a common cookie for both sites and open the login popup once but not for every site?

Do you have any update on this?

I also would like to know about this.

Cookies are only for one domain..


UUID Logical Type generating java.lang.String

Apologies if i'm doing something wrong here, but from what I can tell my configuration is correct.
I have a simple avro definition with a UUID logical type as follows.
{
  "name": "id",
  "type": {
    "type": "string",
    "logicalType": "uuid"
  }
}

The generated class is still using a java.lang.String type though. Am I missing something here? I would expect this to be of type java.util.UUID.

The schema looks valid to me, as per the Avro specification.  However, it doesn't appear to "work" (as in result in a UUID being used) when using the avro-tools compile schema command.  I recommend requesting support from the Apache Avro project.

Thanks for the quick reply, I will do that!

Just for history.
I solved this issue by adding custom conversion like:
avro {
    customConversion(Conversions.UUIDConversion.class)
}

Is this merged PR relevant? https://github.com/apache/avro/pull/1362

It might mean that if the plugin is using a version that includes that PR, it wouldn't be needed to resister the UUID conversion.


cmd_search: Add support for user@service queries

This adds support for doing queries like keybase search shazow@twitter by adding a service parameter to the user search function in the avdl.
The server endpoint already supports the "service" param, we just weren't passing it through.

You're adding an extra field to SearchArgs, but don't you need to make sure that extra field gets used somehow? I'm looking at where the first field gets used, here: https://github.com/keybase/client/blob/master/go/service/user.go#L139

Looks like the queries get serialized here rather than via the service runner thing.
This PR works locally fwiw.

cc: @oconnor663 ping

@maxtaco Jack already took a look but would prefer another person to check it out. Maybe @mmaxim?

This doesn't actually change the service.

	eng := engine.NewSearchEngine(engine.SearchEngineArgs{
		Query: arg.Query,
	}, h.G())
	ctx := &engine.Context{
		LogUI:     h.getLogUI(arg.SessionID),
		SessionID: arg.SessionID,
	}
	err = engine.RunEngine(eng, ctx)
	if err == nil {
		results = eng.GetResults()
	}
	return
}

LGTM as far as the code goes, though I'm still woefully confused about how it actually works. (It looks like the SearchArg struct makes it to go/service/user.go::UserHandler::Search, and there the Service param gets ignored. I think Patrick's asking the same thing.)

service/user.go:Search.
You'll need to change that to plumb through the service parameter.

engine/search.go needs to change too.

Chatted with Patrick, looks like it's using the user/autocomplete endpoint but we could switch it to use user/user_search which might be better—would need to confirm that the frontend is not using this anywhere.
Also no clue how my local version worked at all given that it wasn't plumbed through.
I'm going to put this PR on hold for a bit while I resume work on some extension frontend stuff.


Added a docker.sh script which builds the driver using the same version of libstdc++ as steamvr!

docker.sh create a docker container with Ubuntu 16.04 and everything needed to build SteamVR-OpenHMD.
This method makes the driver works with STEAM_RUNTIME_PREFER_HOST_LIBRARIES=0 environment var, which essentially forces Steam to use it's own libraries instead of the system one.
In this case, we must build the driver with the same GCC used to build SteamVR, or else the driver will fail to load! (so the driver can work with steam runtime libstdc++)
More information can be found here: https://github.com/ValveSoftware/steam-for-linux/issues/4475
(search for "STEAM_RUNTIME_PREFER_HOST_LIBRARIES:")

Cool, someone tries to use this. :)
I've never had such issues on Arch, maybe because I have steam-native-runtime installed, but I'll assume the docker stuff works too.
I'm kinda waiting for OpenHMD to finish positional tracking & controller support for the Vive before finishing up this plugin e.g. performance might be bad unless the pose from OpenHMD is updated in a separate thread, but all contributions are very welcome.
Open source steamvr is most likely not going to happen anytime soon, unless someone is making on themselves. https://github.com/ChristophHaag/openvr_api-libre is a start but there is a lot to implement. Most likely OpenXR is out and most applications will support OpenXR before it gets useful and at this point OpenHMD will be an OpenXR runtime anyway.

... maybe because I have steam-native-runtime installed...

I just realized... when you mention steam-native-runtime, you're actually referring to a Arch package, right?
I just looked on yaourt and found it:

I don't have that installed at all... Didn't even knew about it!
Why do you use it?
Steam work just fine for me without it. It's just steamVR that doesn't, due to crashes on vrcompositor.
According to Valve,  STEAM_RUNTIME_PREFER_HOST_LIBRARIES=0 forces steam to ignore all the system installed libraries in favor of the ones that come with Steam, like libstdc++.so, libusb, etc.
But i have no idea about steam-native-runtime. Do you think it could help me with the vrcompositor crahes?

Steam somewhat recently implemented "intelligent" library lookup by default so it only uses libraries from the Steam Runtime if the library isn't found on the system in /usr/lib etc. I think they also have a list of "pinned" libraries where this doesn't happen. It's just a complicated system, but for me it works without any variables like STEAM_RUNTIME_PREFER_HOST_LIBRARIES=0. (RX 480 though, not nvidia)

I decided to do this because I keep having vrcompositor crashing before start, and trying to run steam with STEAM_RUNTIME_PREFER_HOST_LIBRARIES=0 to check if there's some Arch library causing the crash would not work, since the driver would crash because it can't find the symbol GLIBCXX_3.4.2 in Steam's libstdc++.so (which is GCC 5.1.0).

Well, the old solution was just to delete the worst libraries for this kind of mess:
find ~/.steam/root/ \( -name "libgcc_s.so*" -o -name "libstdc++.so*" -o -name "libxcb.so*" -o -name "libgpg-error.so*" \) -print -delete

And then it would use the ones in /usr/lib etc.
Now with the new library lookup this should not be necessary if you install steam-native-runtime and do not use STEAM_RUNTIME_PREFER_HOST_LIBRARIES=0.
And yes, it's an arch meta package that just has dependencies on all libraries that are part of the steam runtime, so you can install all at once.

So, is this not functional yet? Could it be the cause of vrcompositor crashing then?

No, headset tracking and distortion correction as far as OpenHMD has display parameters work.
It's just that I'm not sure the position updates are smooth enough because it only polls OpenHMD for an updated pose in RunFrame(), where there was a comment in the sample driver.
void RunFrame()
	{
		// In a real driver, this should happen from some pose tracking thread.
		// The RunFrame interval is unspecified and can be very irregular if some other
                // driver blocks it for some periodic task.

It looks okay currently though.
Controllers currently have the "steam controller" model and I only tested briefly with Nolo controllers and their position was weird. Oh yea and the button presses on the controllers still need to be implemented.
That's what I mean.

Also a short demo video: https://www.youtube.com/watch?v=1J6nERqybu4. Since it's the Vive, it doesn't have positional tracking and to the left there are two steam controller models hovering that aren't tracked yet.


feat(interactions): deserialize channel from data

Summary
Implements https://github.com/discord/discord-api-docs/commit/88a761816b26d6891f8a276683bf57a4133aa233, deserializing the channel field from the interaction payload and no longer using the channel_id field.
This has no effect for channels which are already cached, but it now provides a proper channel object in cases where it isn't (no bot in guild, private threads, ...).
A few comments on safety
The continued assumption is that all non-PING interactions contain this channel field, which is what we're already assuming for channel_id. However, this is not documented.
Making the channel field optional in the future is definitely worth consideration, imo, but for now it might be too much of a breaking change in terms of typing.
Similarly, we expect some fields to exist in this channel field, particularly id + type (and name + parent_id + thread_metadata in the case of threads)
While all these are documented to exist in interaction.resolved.channels, this might not be the case for interaction.channel, which really only guarantees id and type.
In the end it's probably a fair assumption, but still something we should improve in the future.
related thread
Checklist

[x] If code changes were made, then they have been tested

[x] I have updated the documentation to reflect the changes
[x] I have formatted the code properly by running pdm lint
[x] I have type-checked the code by running pdm pyright

[x] This PR fixes an issue
[x] This PR adds something new (e.g. new method or parameters)
[ ] This PR is a breaking change (e.g. methods or parameters removed/renamed)
[ ] This PR is not a code change (e.g. documentation, README, ...)

The continued assumption is that all non-PING interactions contain this channel field, which is what we're already assuming for channel_id. However, this is not documented.

I really don't like this assumption, and would like some fallback to current behavior in the event that the channel data is not present (it was already rolled back once for breaking d.js).

Looks like the assumption that parent_id is included in threads doesn't always hold for interaction.channel, unlike interaction.data.resolved.channels: https://github.com/discord/discord-api-docs/discussions/6270
I can't reproduce it, but I was afraid this might happen. It makes this field a lot more annoying to work with; not sure how to handle this properly right now, I don't want to just slap a try/except around it.


TypeError raised when command takes capitalized argument

Description:
When you add a command that expects a capitalized argument a TypeError is raised (TypeError: hello() got an unexpected keyword argument 'name').
How to replicate:

paste this in a file:

import click

@click.command()
@click.argument("nAme")
def hello(nAme):
    click.echo(f'Hello {nAme}!')

if __name__ == "__main__":
    hello()

run it

$ python github_example.py GodSaveTheDoge
Traceback (most recent call last):
  File "github_example.py", line 9, in <module>
    hello()
  File "/home/doggo/.local/lib/python3.8/site-packages/click/core.py", line 829, in __call__
    return self.main(*args, **kwargs)
  File "/home/doggo/.local/lib/python3.8/site-packages/click/core.py", line 782, in main
    rv = self.invoke(ctx)
  File "/home/doggo/.local/lib/python3.8/site-packages/click/core.py", line 1066, in invoke
    return ctx.invoke(self.callback, **ctx.params)
  File "/home/doggo/.local/lib/python3.8/site-packages/click/core.py", line 610, in invoke
    return callback(*args, **kwargs)
TypeError: hello() got an unexpected keyword argument 'name'

Expected behavior:
I expected it to output Hello GodSaveTheDoge!
Environment:

Python version: 3.8.5
Click version: 7.1.2

Click does some normalization on the command and argument names. I don't think we need to change this.

you could at least hint that in the documentation...


Sampling for Jordan and Philippines study area errors

Warning: Error in h: error in evaluating the argument 'CRSobj' in selecting a method for function 'spTransform': NA
  73: %>%
  72: observeEventHandler [/home/guevarra/ShinyApps/spatialsampling/server.R#590]
   1: runApp

This error is more likely an issue with spatialworks/spatialsampler package rather than with the spatialsampling application.
Will keep this open and then transfer this issue to spatialworks/spatialsampler


Falco won't load macro containing macro reference

If you try to load the following as falco_rules.local.yaml:
- macro: sh_writing_resolv_conf
  condition: (proc.name=sh and fd.name=/etc/resolv.conf)

- macro: user_known_write_etc_conditions
  condition: sh_writing_resolv_conf

You get this error:
$ sudo falco -r /mnt/sf_mstemm/work/src/falco/rules/falco_rules.yaml -r /mnt/sf_mstemm/local_rules.yaml
Mon May 20 21:14:28 2019: Falco initialized with configuration file /etc/falco/falco.yaml
Mon May 20 21:14:28 2019: Loading rules from file /mnt/sf_mstemm/work/src/falco/rules/falco_rules.yaml:
Mon May 20 21:14:28 2019: Loading rules from file /mnt/sf_mstemm/local_rules.yaml:
Mon May 20 21:14:28 2019: Runtime error: Error loading rules: /usr/share/falco/lua/compiler.lua:65: Undefined macro 'sh_writing_resolv_conf' used in filter.. Exiting.

However, moving the contents of sh_writing_resolv_conf directly into user_known_write_etc_conditions works:
#- macro: sh_writing_resolv_conf
#  condition:

- macro: user_known_write_etc_conditions
  condition: (proc.name=sh and fd.name=/etc/resolv.conf)

I think the problem is that the macro user_known_write_etc_conditions is being loaded before sh_writing_resolv_conf has been loaded--rules/macros are generally loaded in the order in which they are found in the files. In this case, the bug is that user_known_write_etc_conditions is being read from its first reference in falco_rules.yaml and not the second reference in falco_rules.local.yaml.

I think I figured it out. This might not be an issue.
macro user_known_write_etc_conditions has been defined in right before macro write_below_etc_common.
If I put macro  sh_writing_resolv_conf before the original macro user_known_write_etc_conditions, it worked, if I put it after, the it won't work. I guess falco only load macro once and ignore the duplicated one later on?

Right--I think it's (mistakenly) sensitive to the position of the original macro in the original file, when it should really use the position in the second file.

We should still work on a better fix for this.

This seems related to #706 findings (ordering / positioning of macros) imho.

@mstemm: There is not a label identifying the kind of this issue.
Please specify it either using /kind <group> or manually from the side menu.

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

/kind bug
/kind rule-create

keep

Issues go stale after 90d of inactivity.
Mark the issue as fresh with /remove-lifecycle stale.
Stale issues rot after an additional 30d of inactivity and eventually close.
If this issue is safe to close now please do so with /close.
Provide feedback via https://github.com/falcosecurity/community.
/lifecycle stale

Stale issues rot after 30d of inactivity.
Mark the issue as fresh with /remove-lifecycle rotten.
Rotten issues close after an additional 30d of inactivity.
If this issue is safe to close now please do so with /close.
Provide feedback via https://github.com/falcosecurity/community.
/lifecycle rotten

Rotten issues close after 30d of inactivity.
Reopen the issue with /reopen.
Mark the issue as fresh with /remove-lifecycle rotten.
Provide feedback via https://github.com/falcosecurity/community.
/close

Rotten issues close after 30d of inactivity.
Reopen the issue with /reopen.
Mark the issue as fresh with /remove-lifecycle rotten.
Provide feedback via https://github.com/falcosecurity/community.
/close

@poiana: Closing this issue.

In response to this:

Rotten issues close after 30d of inactivity.
Reopen the issue with /reopen.
Mark the issue as fresh with /remove-lifecycle rotten.
Provide feedback via https://github.com/falcosecurity/community.
/close

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

@poiana: Closing this issue.

In response to this:

Rotten issues close after 30d of inactivity.
Reopen the issue with /reopen.
Mark the issue as fresh with /remove-lifecycle rotten.
Provide feedback via https://github.com/falcosecurity/community.
/close

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.


chore(main): release 1.1.0

:robot: I have created a release beep boop
1.1.0 (2024-07-02)
Features

add commands to support the new assertions (3ac6ef0)
docs: add a documentation for new commands (73a425e)

This PR was generated with Release Please. See documentation.

:robot: Created releases:

v1.1.0
:sunflower:


[1.11.2] Wind Generator recipe not working.

With the latest 1.11 branch, crafting a Wind Generator is not possible. This has also been tested and confirmed with Mekanism(+Generators/Tools) being the only mods installed.
Forge: 1.11.2 - <IP_ADDRESS>9
Mekanism: 562c127

Can confirm it's not working with all handcrafted items.

Fixed!

For reference: b355b80cf4edc429ed5fb3782fdb93f175656592


Facebook OAuth doesn't Work

Here is the link I got when trying to login with Facebook:

https://www.facebook.com/v2.2/dialog/oauth?channel=https%3A%2F%2Fstaticxx.facebook.com%2Fconnect%2Fxd_arbiter%2Fr%2FhsBwMj6iLmk.js%3Fversion%3D42%23cb%3Df1d59cf6cb8ddbc%26domain%3Dwww.instagram.com%26origin%3Dhttps%253A%252F%252Fwww.instagram.com%252Ff1e785ccc1b029c%26relation%3Dopener&redirect_uri=https%3A%2F%2Fstaticxx.facebook.com%2Fconnect%2Fxd_arbiter%2Fr%2FhsBwMj6iLmk.js%3Fversion%3D42%23cb%3Df3ad6a06654cdec%26domain%3Dwww.instagram.com%26origin%3Dhttps%253A%252F%252Fwww.instagram.com%252Ff1e785ccc1b029c%26relation%3Dopener%26frame%3Df2fcec483b825fc&scope=public_profile%2Cemail&response_type=token%2Csigned_request&domain=www.instagram.com&origin=2&client_id=124024574287414&sdk=joey&_rdr

But it loads up an empty page and I still can't login to the app.

I did a little bit research online, looks like facebook dropped support of OAuth v2.2.
Reference on StackOverflow
Maybe it's time to update :)

Now you can use Chrome dev tools (in mobile view) to post photos.

@elektro-NIK What do you mean? Does that somehow relate to OAuth?

@chen8160 No no. While the authorization is not fixed, instead of Ramme you can use mobile version of the Instagram (in Chrome dev tools) for publishing the photo from PC.


Virtual Garage Scroll

When the virtual garage has 5 or more vehicles, the stored tab does not appear to have a scroll bar. You can still use your keyboard to go down the list but it will not be visible. Using Scroll wheel will not work as well.

Thx for the feedback @Gr8z, I'll look into it.

Thank You @Bjanski

@Gr8z Since I'm not pushing the update now, you can fix it by changing "RscListBox" to "RscExileItemListBox" in file "XM8_apps\apps\XM8_VG\scripts\XM8_VG_mainVGSlide_onLoad.sqf" on line 70.
From
_ctrl = _display ctrlCreate ["RscListBox",_idc,_parent];
To
_ctrl = _display ctrlCreate ["RscExileItemListBox",_idc,_parent];
This will use the RscExileItemListBox instead which has the scrollbar already built in.
Will try to find a better fix for it but since I can't use mission defined controllers I have to use the mod defined or built in ones and for some reason ArmA's predefined doesn't have the scrollbar built in.

I have used RscListBox before in other XM8 apps. and they seem to perform fine. Also Forgot to mention, when you scroll wheel. Every single dialog in the app shifts up and down.

The Temp fix does not fix the scrolling issue.

@Gr8z I'm aware of that but haven't had any time to look into the code yet.
Do you encounter this for all Improved XM8 Apps or is it only the Virtual Garage?

Do you encounter this for all Improved XM8 Apps or is it only the Virtual Garage?
Only the Virtual Garage has this problem.

I made a change and got it here http://image.prntscr.com/image/008944bec74544c2966880e937b3c881.png

this is fully fixed after f222940


Fix regexp to not find extensions in the middle of a module name

This PR fixes the Regexp used to filter modules with the supported extensions. Previously it was incorrectly finding modules that contained the extension in the middle of the name. This was both because we were not searching for the last characters in the module (using $) and also because it was not escaping the . caracter properly.
Added some more cases to the tests for this fix.
Oh and by the way, thanks for creating this module! Very helpful!

Thank you both @leoasis @jvivs 👍 Released as 0.1.1. I am glad you found this module helpful :)

Great! thanks @michalkvasnicak


Adding doi of Poinelli 2023b

Updating README with doi of Poinelli 2023b
Adding Poinelli2023b.pdf

Hi @ynakayam , I am adding some more info about our 2023 paper on Larsen C.

Thank you @ynakayam !


fix(clustering) Handle connection error on version negotiation

If the CP doesn't respond to the version negotiation request, the DP
will assume the "old" protocol.
A failure to establish the websocket connection repeats the negotiation
attempt, as it would probably indicate a control plane rebooting or real
connectivity issues.  Later errors would (first) attempt only reconnect,
as they would likely be transient issue.  But if that reconnection
fails, it would again retry to renegotiate.

Changing version negotiation flow.  Do not merge

as mentioned above, the version negotiation specs have changed (implemented in #8775 ), but the current master has at least one bug when failure to connect leads to restarting only part of the cycle, locking into the wrong protocol.  this patch fixes that by assuring that failures restart from the start.


TILA-1736 | Remove obsolete id references and change necessary to pk

In application graphql types there was some *_id fields declared which
we don't need in graphql since we can just add the corresponding
relative type.
This removes the id fields and introduces relatative types if needed and
changes one id to _pk which is what we prefer.
TILA-1736

Added @siren and @kieferbonk also as reviewer so you could check UI things before we merge and :boom:


requestFileSystem

If you make this change, it will play nicer with the Chrome browser for desktop testing
$window.requestFileSystem = $window.requestFileSystem || $window.webkitRequestFileSystem
function getFilesystem() {
  var q = $q.defer();
  try {
    $window.requestFileSystem($window.PERSISTENT, 1024 * 1024, q.resolve, q.reject);
  } catch (err) {
    q.reject(err);
  }
  return q.promise;
}

That's a good idea, I'll get on that. But since the File plugin is now using window.resolveLocalFileSystemURL, I'll use this instead:
window.resolveLocalFileSystemURL = window.resolveLocalFileSystemURL ||
                                   window.webkitResolveLocalFileSystemURL


Change minimum take to 18%

Description
Minimum take is 18% now so that setting take is disabled until admin sets a different minimum take value.
Related Issue(s)
n/a
Type of Change

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] Documentation update
[x] Other (please describe): Updating a runtime parameter

Breaking Change
n/a
Checklist

[x] I have performed a self-review of my own code
[x] I have commented my code, particularly in hard-to-understand areas
[x] I have run cargo fmt and cargo clippy to ensure my code is formatted and linted correctly
[ ] I have made corresponding changes to the documentation
[x] My changes generate no new warnings
[x] I have added tests that prove my fix is effective or that my feature works
[x] New and existing unit tests pass locally with my changes
[x] Any dependent changes have been merged and published in downstream modules

Screenshots (if applicable)
n/a
Additional Notes
n/a

@gztensor can you now also open a PR doing the exact same change into development just so we come full circle?


Switch back the toolchain to "stable" in the build and test CI jobs

Description
A workaround has been created by freezing the version of rust in the CI jobs to v0.16.4 to unblock the Windows CI job failure with the latest Rust versions.
When the v0.17.0 version of Rust is published, we should switch back to "stable" in the toolchain setting.
Lines:

https://github.com/delta-io/delta-rs/pull/923/files#diff-5c3fa597431eda03ac3339ae6bf7f05e1a50d6fc7333679ec38e21b337cb6721R42
https://github.com/delta-io/delta-rs/pull/923/files#diff-5c3fa597431eda03ac3339ae6bf7f05e1a50d6fc7333679ec38e21b337cb6721R67

Related Issue(s)

#923

we switched back to stable as part of #1097


Failing test: test_pageserver_catchup_while_compute_down

https://app.circleci.com/pipelines/github/zenithdb/zenith/3877/workflows/3a9bb165-5b51-4d33-84f7-c5b266f5af8e/jobs/34213/tests
It failed on a refactor-only change on my branch so I'm sure it will fail on main eventually. Seems like a serious bug since it loses data.

Failed for my PR too: https://app.circleci.com/pipelines/github/zenithdb/zenith/3886/workflows/0c464e8a-9fc1-48a0-91eb-c66ff48858e0/jobs/34337
I've run the test locally for 10 times and there was no errors

Could be explained by https://github.com/zenithdb/zenith/issues/1074
In this test we stop all safekeepeers while pageserver is down. In the issue above you'll see that safekeepers currently don't wait for pageserver to write an s3 snapshot before they go down, so data gets lost there.

@anna should this be included in the limited preview milestone? I think there's one more flaky test as well.

We haven't seen this for a while. It's probably been "accidentally" fixed. Closing, but let's reopen if it reappears.


[bug]: useToast created in a wrong place in my Next.js project

Describe the bug
when I ran the cmd
bun x --bun shadcn@latest add toast

it was supposed to create a file use-toast.ts inside the components/hooks/use-toast.ts but it created it in separate folder /hooks/use-toast.ts that is not inside the components folder.
Is it something related to how I set up the path?
PFA as well for more reference.

Affected component/components
Toaster
How to reproduce
NA
Codesandbox/StackBlitz link
No response
Logs
No response
System Info
na

Before submitting

[X] I've made research efforts and searched the documentation
[X] I've searched for existing issues

nevermind, my mistake