splitting out dynamic model, microsim model, tax law parameters, economic parameters

We are currently following a strategy where the user opens TaxBrain and sees three things on the 1st input page:

tax law parameters relevant to the microsim model and dynamic model.
economic assumptions relevant to the microsim model
detailed documentation mostly relevant to the microsim model.

The user then runs a reform and sees microsim results on the 1st output page.
The user can then go to a dynamic analysis page where he sees the 2nd input page:

forthcoming: tax law parameters relevant to the dynamic model such as firm-level taxes.
economic assumptions relevant to the dynamic model.
documentation mostly relevant to the dynamic model.

Issues (please feel free to add to this list)

The user might enter economic assumptions that conflict between the microsim model and the dynamic model. In the worst case, this might lead to spurious results in the dynamic model (would this occur if the user had a non-zero eti on the microsim side?). Slightly less bad, it could also lead to incomparability between the microsim results and the dynamic results (does this always occur when the economic growth rates in the microsim and dynamic model are different?), or it could lead to general user confusion (why do I have to specify a Frisch elasticity when I already specified an ETI?).
A user might come looking for dynamic modeling, but only see documentation about the microsim model up front.
In the future, we might have dynamic models available that don't use all of the tax law parameters form the first input page.
In the future, we might have multiple dynamic models available with overlapping but not identical sets of inputs available.

Suggestion for mitigating these problems in the near term (same numbering as above).

This is probably the most section for near term discussion.

On the 2nd input page, explicitly tell the user which of the parameters from the 1st input page will be overridden by the dynamic model. Then on the dynamic model page provide a link to both the conventional analysis that relies on the user specified assumptions and the purely static (or otherwise) analysis that was used for the dynamic model run.
Add some general description of the dynamic model to 1st input page to let users know that the dynamic run will be possible after they enter the tax law changes.
We don't need to worry about this yet.
We don't need to worry about this yet.

Suggestions for mitigating these problems in the long term.

This section is important for us to be thinking about, but I don't think we should plan on reaching a decision right away.

One option for discussion:
Adopt a system where all tax levers are available on a tax-policy-parameter input page; next to each input would be a list of the models that uses that tax lever, (perhaps the user would also have the opportunity to filter for inputs that are available for a given model).
On a second page would be a list of models that use some set of the inputs from the input page. The conventional model and the dynamic model would be included separately here. From here you could click through to each model individually, where you will find the set of economic assumptions that are relevant to that particular model. (Note that the "dynamic model" here would still use tax-calculator and og-usa, but it would be a separate path from doing  conventional analysis.)
Documentation for each model would be entirely independent. For instance, both the dynamic analysis pathway and the conventional analysis pathway would be responsible for describing the tax-calculator however they would like to.

Hi all, I've been thinking more about the workflow on TaxBrain. I've described a new flow below that I think would solve many of the problems discussed here https://github.com/OpenSourcePolicyCenter/webapp-public/issues/130#issue-126203191. Looking forward to feedback.

Enter tax law parameters and any adjustments to the baseline data (growth factors).
Receive a purely static score (no behavior).
Click "link to dynamic simulations".
Get to a landing page with the following choices, each with a short description underneath it.

Partial equilibrium with micro-behavioral elasticities
Overlapping Generations (DeBacker, Evans, Phillips)
Macro elasticities [this would be new, would apply a Barro and Redlick 2011 or mertens 2015 type elasticity of GDP w.r.t average marginal rates.]
Any other dynamic models that can take advantage of the individual and payroll microsim.

On the first input page we can note that dynamic estimates are available after IIT and FICA reforms have been specified. Once there are firm-specific reforms that are available only through the dynamic models, we can give an option to link directly to those if the use wants to skip the IIT and FICA reforms. Once we get to the point where we have models that only incorporate firm-specific parameters (e.g., a user cost of capital calculator), then we could add another landing page before the user gets to IIT and FICA param input page.
cc @feenberg, @martinholmer, @talumbau, @rickecon, @kerkphil, @jdebacker, @Amy-Xu, @mistakevin

Matt,
I think this workflow sounds reasonable, but a two comments:

Does anyone care about the truly static score?  I'm not sure this is relevant and, even if it is relevant to some users, it conflicts with what traditional tax scoring numbers typically reflect (e.g. those by JCT or Treasury).
If we have just two macro models, I think that the second set of choices will not be too overwhelming for users.  However if we have several options, it might get confusing and we might focus their choices (allowing for more, but not having the full choice set available at this second landing page).

On Thu, 11 Feb 2016, Jason DeBacker wrote:

Matt,
I think this workflow sounds reasonable, but a two comments:

Does anyone care about the truly static score? I'm not sure this is relevant and, even if it is relevant to some
users, it conflicts with what traditional tax scoring numbers typically reflect (e.g. those by JCT or Treasury).

It is extremely useful to know the static score, it is a good summary of
the data to which the behavior will be applied. Without the static score
one can't understand how the behavioral score happens.

If we have just two macro models, I think that the second set of choices will not be too overwhelming for users.
However if we have several options, it might get confusing and we might focus their choices (allowing for more, but not
having the full choice set available at this second landing page).

A user would have to be pretty sohphisticated to make sense of the macro
models, and what basis do we have to endorse any of them?
dan

—
Reply to this email directly or view it on GitHub.[AHvQVRZS2gHFMbV12TU5YPpbqoXEzAFtks5pjLYygaJpZM4HDOzC.gif]

I am sympathetic to @feenberg 's comment that it is useful to know the static score to the Tax-Calculator output before any behavioral responses are added. This is a valuable decomposition. However, I think the guiding principles for the TaxBrain web app output should be focus, simplicity, and summary. For users who want more than the standard output (i.e., more than the static score with behavior plus some dynamic results), we should point them to the repos and give them the tools to run the models locally. I think the behavior vs. no behavior decomposition of the static results is a great example. This is something that the majority of users won't understand and by which they will likely be confused.
This principle should also apply to the dynamic results. Currently, we just have one model. The current method of having a button that says "Link to dynamic model" is sufficient. We can add output tables, but we should not try and deliver the whole functionality of the model through TaxBrain. @feenberg is right that "a user would have to be pretty sohphisticated to make sense of the [guts of the] macro model". And we don't need to endorse any one of them because we only have one of them. That is, we are implicitly endorsing the one that we have on the site.

On Sun, 14 Feb 2016, Richard Evans wrote:

I am sympathetic to @feenberg 's comment that it is useful to know the static score to
the Tax-Calculator output before any behavioral responses are added. This is a valuable
decomposition. However, I think the guiding principles for the TaxBrain web app output
should be focus, simplicity, and summary. For users who want more than the standard
output (i.e., more than the static score with behavior plus some dynamic results), we
should point them to the repos and give them the tools to run the models locally. I think
the behavior vs. no behavior decomposition of the static results is a great example. This
is something that the majority of users won't understand and by which they will likely be
confused.

I don't think we should be overly responsive to the majority of users,
rather we should concentrate on users who will be influencing tax policy.
This really requires the user be able to understand where the results are
coming from. In my experience the results of a simulation are often
surprising and require explanation. Sometimes the AMT or the excess
credits will totally subvert a proposed change. If the calculation is
opaque, it will likely be dismissed. If it is transparent, the user will
have the information he needs to revise his reform.
I also think that few users will have access to data, even among serious
users. So running a local copy of the won't work for them.
dan

@jdebacker said:

Does anyone care about the truly static score?

Yes, I think any serious tax policy analyst would expect this as part of a model's output.

I don't think we should be overly responsive to the majority of users,
rather we should concentrate on users who will be influencing tax policy.

I think that this is an important point.  I perhaps chose poor wording in my comment, but I also had an implicit assumption that the typical use case would be someone with a strong interest in tax policy, but not a person who is a revenue estimator by trade.  I thought it therefore important to provide as the default output something like what that type of person is familiar with - namely revenue estimates published by groups such as JCT, OTA, and TPC, which (except for those related to tax expenditures) include behavioral responses.  Even within these organizations, it is usually the case that the revenue estimator handling the specific item in question (among the several in the proposal as a whole) is the only one who sees the truly static score.
I do think it is very important to allow users to see a truly static score, but if the click through rate from the default output it not very high, then I think we might consider providing users with output that most closely reflects a traditional revenue estimate first.  For those who want to do further comparative statics (e.g. compare to the no behavior case, see how behavioral assumptions affect the numbers, see a dynamic score, etc), we'd have those options.  But those seem most relevant for the very interested and knowledgable user who is more likely to go past the default output page.
However, related to Dan's comments, what I'm thinking may be slightly less relevant when the revenue effects of each line item in the proposed changes are not broken out separately.  When these are broken out as they typically are in the revenue estimates published by the aforementioned groups, that helps transparency and the understanding of what's going on in the proposal as a whole.  Since we don't break things out in this way, perhaps there is more reason to push users towards the estimates ex behavior.
No strong feelings on this.  I just wanted to further explain my initial comment since it may not have been well expressed.

A few thoughts here:
On why it makes sense to provide static results first.

The purely static calculations are very familiar to many users through the context of tax expenditure estimates. In addition to providing "a good summary of the data to which the behavior will be applied", as @feenberg points out, the static score also allows our users to compare our results against the many tax expenditure estimates that are available from JCT and Treasury.

On why it makes sense to include the micro-behavioral model as an alternative to other dynamic models rather than before them.

The partial equilibrium estimates we generate are dynamic in the sense that we allow GNP to change as a result of the behavioral effects. Our PE model is not quite the same as the conventional estimates produced by JCT where GNP is held fixed. In my view, grouping these with the other dynamic estimates makes more sense than grouping them with the static score, partially for the purpose of pointing out the distinction with JCT's conventional scores.
When the partial equilibrium estimates come before the OLG model, we have a major problem in that the user might specify contradictory ETI's and Frisch elasticities within the same sequential workflow. Placing the PE dynamic model as a parallel option to the GE or other dynamic models entirely circumvents this problem.

On why the dynamic landing page would not be too confusing.

@jdebacker, I am thinking that on the new 'dynamic' landing page there would just be a short description of each model. Then you would click through to get a longer description and ability to adjust economic parameters, adjust tax-law parameters that weren't on the very first input page, and ability to run the model. So I don't think the landing page would be too confusing.

On eventually including multiple dynamic models.

@rickecon, I hope that our open source modeling efforts will encourage more people to work on policy-relevant economic modeling. I also hope that some of those efforts will produce innovative and high quality models that we will want to make available to TaxBrain users. This proposal would move us in the direction where it is easy to incorporate those new models and where it is easy to indicate to people we talk to now that we are open to including new models if they are policy relevant, high quality, and open source. That signalling could encourage some economists to do more policy relevant work and help the open source modeling movement to flourish.

@MattHJensen here is what the page looks like where you choose which dynamic sim you want to run:

let me know your thoughts.

@talumbau, this looks really good. I think we should change "behavioral macro simulation" to "partial equilibrium simulation" and list that one first.


How to change the color of hyperlink text?

How do I change the color of hyperlink text? Now add a hyperlink and use color The RGB setting has no effect. It is always the default blue color

I can't find the answer to this question either

Color written in the xml. Previewed with the correct color, wrong after open the pptx. Need help, thanks.

Post the minimum reproducible example of the code you used that's not behaving as you expect. https://stackoverflow.com/help/minimal-reproducible-example

Python version: 3.11.4
python-pptx: 0.6.23
prs = Presentation()
blank_slide_layout = prs.slide_layouts[6]
slide = prs.slides.add_slide(blank_slide_layout)
txBox = slide.shapes.add_textbox(Inches(1), Inches(1), Inches(8), Inches(1))
tf = txBox.text_frame
p = tf.paragraphs[0]
r = p.add_run()
r.text = "Test test."
hlink = r.hyperlink
hlink.address='http://www.github.com'
font = r.font
font.size = Pt(20)
font.color.rgb = RGBColor(0, 255, 0)
font.underline = False
hlink = r.hyperlink
hlink.address='http://www.github.com'
Hyperlink color and underline not work, still default blue. hyperlink has no font property.
prs.save('test.pptx')

Hmm, post what you get from print(r.xml) placed just before prs.save() and lets have a look at that ...


Increase linux capacity in devicelab

Add additional linux bots to the devicelab.

Added linux-{9,10,11,12} to staging pool.


Add support for eval_data as ndarray pair.

It supports eval_data as ndarray pair, i.e. a tuple like eval_data = (data, label), where data and label are both ndarray. The shape of label can be (n, ) or (n, 1), and the modification is tested under both cases. Hope my coding style is OK.

Thanks for the PR! Please fix the travis in https://travis-ci.org/dmlc/mxnet/jobs/85069662

@antinucleon Hi, is it OK now? I saw all checks passed. Sorry I didn't know there should be docstring.

Great thanks again, in general it looks good to me!
Could you fix the parameter and return docstring? Also I think we should
check whether the tuple (maybe also support list) contains Numpy NDArray,
if they are Python list, convert to numpy ndarray so that this function is
more general. The last thing is you may consider to rebase into one commit.
On Tue, Oct 13, 2015 at 02:39 II-Matto<EMAIL_ADDRESS>wrote:

@antinucleon https://github.com/antinucleon Hi, is it OK now? I saw all
checks passed. Sorry I didn't know there should be docstring.
—
Reply to this email directly or view it on GitHub
https://github.com/dmlc/mxnet/pull/278#issuecomment-147648531.

@antinucleon I'm so sorry that it seems I have messed up something. How should I rebase my commit now? There exist five commits instead of one. =_=||

@II-Matto Seems that the init_data_iter and init_eval_iter kinda of overlaps with each other, maybe you can re-use the init_data_iter(by is_train=False) logic to do the job?

@tqchen Do you mean _init_iter(self, X, y, is_train)? When is_train is set to false, this function will fill y with zeros. So I should change _init_iter() as well?
BTW, is there a proper way to merge the 5 commits here? Could I reset them all and then do my modifications all over again?

Yes for merge commits. You can do git rebase -i .
On Tue, Oct 13, 2015 at 5:34 PM II-Matto<EMAIL_ADDRESS>wrote:

@tqchen https://github.com/tqchen Do you mean _init_iter(self, X, y,
is_train)? When is_train is set to false, this function will fill y with
zeros. So I should change _init_iter() as well?
BTW, is there a proper way to merge the 5 commits here? Could I reset them
all and then do my modifications all over again?
—
Reply to this email directly or view it on GitHub
https://github.com/dmlc/mxnet/pull/278#issuecomment-147891971.

@II-Matto Please also add your name to https://github.com/dmlc/mxnet/blob/master/CONTRIBUTORS.md in your next commit. I will merge it in once you fixed the comments on init_iter.
Thanks

@tqchen I will try to finish it soon. Direct rebase operation is prevented by some conflicts and I am trying to find a way to deal with it.

@II-Matto  You can fix the confliction, do a git add the fixed the file, and do git rebase --continue

@tqchen I have dealt with the commit merge issue, and using _init_iter() in _init_eval_iter(). I also modified _init_iter(), adding extra checks. Hope my modification to the code is ok. I have tested in various cases using fit() and predict(), and the results are normal.
To make sure I didn't do anything unexpected, I will briefly list what I did to my fork and this PR:

Make a fork and clone a local copy.
Add _init_eval_iter() and then docstring. (2 commits)
Try to rebase the last 2 commits. Fail to push to Github.
Use pull to update the local copy and push again. Succeed.
Push the "Update branch" bottom on this PR page. (5 commits appear!)
Fail to rebase in various ways. Use git reset --hard HEAD~3.
Edit model.py as @tqchen and @antinucleon suggested.
Test modified code in various cases.
git add -A and git commit -m "<Message...>".
Fail to push so use git push --force.
Wait for the travis to finish.

It seems that neither of git reset --hard and git push --force is suggested, and I am not sure whether or not my operation will have some unexpected effect.
If there are any problems with my code, I will try to fix it soon.
BTW, I think the right thing to do is that I wait for this PR to be done before I use pull or "Update branch" to keep my fork synced and don't push other commits (or I will need to start a new branch). And after the PR is done, I can merge the corresponding branch with, say, a "master" branch. Am I right?

@II-Matto It is fine to git push --force to your own fork, As long as the changes are only yours. Thanks Please update the PR to add your name to  https://github.com/dmlc/mxnet/blob/master/CONTRIBUTORS.md and I will merge it in

opps, seems there is some recent changes that makes the branch not able to auto merge. Can you rebase again? Sorry about that

This page says that "This branch has conflicts that must be resolved". What should I do? =_=

Follow the rebase instruction here https://github.com/dmlc/mxnet/blob/master/doc/contribute.md#submit-a-pull-request

@tqchen Sorry for my unfamiliarity with git. Is it OK now?

Thanks, this is merged!

I have also updated https://github.com/dmlc/mxnet/blob/master/doc/contribute.md to include more instructions, you are welcome to contribute your experience as well :)

@tqchen Thanks! This is very considerate.
Since this is the first time I participate in an open source projects, what I really wanted to know, at the very beginning, is the right operations and steps. I had no idea what I needed to know at first, and even didn't know what to search for. After this pull request was finished, I did some googling, and found this blog post very useful: https://gun.io/blog/how-to-github-fork-branch-and-pull-request/ (Unfortunately, this seems to be outiside GFW. A Bing cached page may help in some cases.).
This blog post lists the necessary steps (with some of my personal addition):

Check out the "Network" graph , e.g. https://github.com/dmlc/mxnet/network. It is also necessary to check out existing issues and pull requests.
Open an issue for discussion.
Make a fork. See https://help.github.com/articles/fork-a-repo/.
Clone the fork to get a local working copy.
Add upstream to get synced with the original repository.
Setup a development branch. I guess it is a good idea to use a new branch for each new feature or fix, since "you can only have one Pull Request per branch".
Do the development. Maybe you will commit crazily to the development branch. Also, don't forget to discuss with others.
8 Squash commits, i.e. merge small and unnecessary commit pieces into larger and meaningful commits.
Submit a Pull Request (PR), discuss, fix problems, etc. Work with other contributors to finally make a good patch.
Wait for the owner to merge or reject the PR.
Done!
Maybe merge the development branch to the forked master branch, and start new branch for new features.

I am not sure whether new commits will keep appearing in a pull request after it is merged, or merging the development branch with the forked master branch will ever effect the corresponding pull request.

If you like to share your experience and contribute to the  https://github.com/dmlc/mxnet/blob/master/doc/contribute.md  to make others to get started more easily
It is more than welcomed :) Thanks for the contribution

@tqchen I will do that after I perform some experiments to confirm the right thing to do. Thanks the team for making such a good open source project!


Add uri_path to factories

I just had the idea that, if you create a question, the uri_path of the page is already in the form. This PR adds this. I am not sure, if we should add the / to the path already. If we don't just creating the element without editing will give a validation error (which is good I think), but then we always need to add the slash.

I have no strong opinion. Just for comparison: uri_prefix is defined without slash.

Clarification: The inserted uri_path is the path of the parent, e.g. page catalog/section/page. The idea is that I only need to add "question" to get catalog/section/page/question as uri_path for the question. If we insert catalog/section/page/ people just need "question", but if they just hit save a question with catalog/section/page/ will be saved (which the user does not want). If we insert catalog/section/page people need to type an addtional /, but when they just hit save, there will be a validation error since catalog/section/page is already the uri_path of the page.

This argument supports the uri_path without /.


build: use is-ci before husky install & syntax error in Dockerfile

entrypoint.sh was a simple fix.  It was a syntax error with the EOF token.
DISCLAIMER:  I AM NOT A NODEJS PERSON.
"husky" is specified in packages.json as a devDependency. yet the Dockerfile is running npm ci with "--production".
With this modification, "husky install" will only be run if the NODE_ENV variable is NOT "production".
I have no idea if this is truly the proper way to resolve this issue for all build use-cases, but now it builds and the resulting container functions properly from my minimal testing (account creation via Ferdium, basic operation).

@all-contributors please add @ericreeves for code

@all-contributors please add @nathanelhoun for review

https://github.com/all-contributors please add @nathanaelhoun for review

@all-contributors please add @nathanaelhoun for review

@all-contributors please add @ericreeves for infrastructure


Failed interpolation in placeholder

The variable interpolation does not work in placeholder, if interpolation directly follows the percent character
This fails with an error

"a" failed to @extend "%a". The selector "%a" was not found.: line: 6

$varname: a;
%#{$varname} {
  color: red;
}
a {
  @extend %a;
}

If I add an underscore directly after the percent sign, it works:
This works
$varname: a;
%_#{$varname} {
  color: red;
}
a {
  @extend %_a;
}

The only difference is the underscore
Both of the above work in SassMeister and yield the same CSS code
a {
  color: red;
}

Please note. The interpolation correctly works for class names and ids
No problem with these: .#{$varname} and ##{$varname}

Confirmed, still an issue


[Storage] Publish browser-tests results

https://dev.azure.com/azure-sdk/playground/_build/results?buildId=11192


add 'accept terms', status structure

there should be 3 routes

/ status (home page),
/phoenix (already exists at default which needs to change),
/dashboard (already exists),
/update

The restrictions on the pages are:

/dashboard should only be accessible if authenticated (authentication is separate issue), and user is on dashboard list of authorized users
/update should only be accessible if authenticated (authentication is separate issue), and user is on list of authorized admin users, and then user can only update status on their organization, not everyone else's. Superusers can update any organization
/phoenix and /status pages content is "warning ... (Duncan to supply text) until they click acknowledge button. Ie anyone can see the real content of these pages, but only after clicking acknowledge
/status page is a series of tabs:

overall
project
organization
interval
sprint
opensource

each of status tabs will consist of some explanation text and some data graphically
each status tab will be a summary and can be clicked on to drill down. Eg org page is status of all tasks across all orgs (maybe color coded by org) and clicking on an org brings you to a view of just that org's status.
status is a result of data in a db that gets administered via update page.
there are 10 orgs (https://github.com/oasis-tcs/openc2-usecases/tree/master/SBOM-PoC#2---organizations-participating) which will grow to appx 50
there are 6 projects (OpenC2, SBOM, IACD, CACAO, OCA, SCAP2) which may grow or shrink
there are 3 intervals (single, multi, mega)
there are N sprints which are two week intervals June 4-18, June18-Jul2, Jul 2-16, Jul 16-30, .... late October
there are N open source projects involved - will need to list and admins (of any org) will be able to add more
org admins enter entries (which can be tasks or goals) for that organization. They may have zero-to-all projects associated with a given entry. They should have one of single/multi/mega. They may have 0, 1, ..N open source repos associated with a task
an entry starts out unassociated with any sprint and is considered 'not started' from sprint viewpoint. As the task/goal is started to be worked on, it becomes 'being worked' for the sprint that it starts. An admin can mark it 'complete' for a given sprint. being worked and complete can only be marked on sprints prior to and including the current sprint (eg saying complete next sprint is not allowed)
status consists of a view of the not started, being worked, complete sliced/diced depending on the view

start simpler. No database. No users/authorization. All pages static and 'tabs' above are pages linked off a home page.
Ideally pages would be markdown, not html, so users could to PR's to update


wagtail-schema.org requires a licence

We'd like to make this available as an addon on the Divio Cloud, but we can't unless it has a licence (and indeed, many companies won't be able to use it either).
Will you consider releasing it under a suitable licence?
Thanks.

Oh sorry, I must have forgotten that step. I usually use the BSD 2-clause license. I'll add that and get this updated right away.

Added in https://github.com/takeflight/wagtail-schema.org/commit/82689ae096dd402039bbbfa07626e2c2fa90344e


Remove cpmsctl

Not needed anymore since we introduced the controlplanemachineset generator controller with https://github.com/openshift/cluster-control-plane-machine-set-operator/pull/104.

/retest-required

/retest-required

/retest-required

/retest-required

/retest-required

/retest-required


Civic Camp Saturday Lunch

For Manhattan Cafe, all ordering should be done online. Anything we need, we need to order 24 hours in advance so if we want to go with them for Saturday, we need the order in early Friday.
I talked to them just now and really, the easiest way is to just place the order here: https://manhattancafenc.com/catering/
We don't need delivery cause the food will be ready for us at the cafe on Wilmington Street, 1 block away from the Annex. Two or three can walk down and pick it up, save a few bucks.


Use Array.isArray in the LoopbackPort.

For better serialization of Arrays that belong to different sandboxes.
See also http://logs.glob.uno/?c=mozilla%23pdfjs#c66373
/cc @tobytailor

Indeed, wouldn't just changing https://github.com/mozilla/pdf.js/blob/master/src/shared/util.js#L1097 suffice here? For the polyfill we can use core-js since we already use it for clamped arrays.

Can we use Array.isArray everywhere?

Maybe. There are too many places to check if they will work as expected when changed. For now I'm just addressing regression (and hope will not add another one :).

For IE9, a polyfill can be added to compatibility.js.

Per MDN, Array.isArray is implemented in IE9

Even better. Then we don't need a polyfill at all.

/botio test


Make all tests assert something

To be sure our tests are as effective as possible at catching bugs, we should make sure they all assert something. In my opinion, tests that do not assert anything give a false sense of security, and make running the testing suite go slower with little benefit.

Note that #543 would have been avoided if we asserted that the output equals what we expect in the http staging test.


Upgrade golangci-lint to 1.19.1

Disable godox.
ref: https://github.com/pion/datachannel/pull/44

Codecov Report

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

@@           Coverage Diff           @@
##           master      #45   +/-   ##
=======================================
  Coverage   64.41%   64.41%
=======================================
  Files           4        4
  Lines         222      222
=======================================
  Hits          143      143
  Misses         58       58
  Partials       21       21

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 5c367e5...aa4287a. Read the comment docs.


Error 500 when editor uses a course page template on a page that has no course attached

Bug Report
Actual Behavior
On page which is not a course page, editor is allow to change the page template by a course model page. As this template requires to be linked to a course object, an 500 error occured.
Expected behavior/code
Richie should not crash after an user action.
Steps to Reproduce

Login as editor
Go to the homepage
In the toolbar, navigate to Page > Templates
Select Course Page
And then the bug happens!

Environment

Richie version: 2.1.0
Platform: Mac OS 11.2.1 - Firefox 85.0.2

Possible Solution

Prevent editor to change page's template to a course template.
When a course template is used, check if a course is attached to the page. If not warn editor with an explicit message explaining that he is doing something wrong.

Additional context/Screenshots
https://sentry.io/organizations/gip-fun-mooc/issues/2136906474/?project=1198559&query=is%3Aunresolved

I vote for solution 2 as solution 1 would require modifying the cms... Should be done for all page extension types...


Docs need guidelines re: when to used linked vs when to use single container

https://docs.docker.com/userguide/dockerlinks/  and related should provide guidance on how or why a user would choose one method of creating container versus another (security, management, data).  Also, do we have general guidelines as to which we recommend as the "default" technique.
Corollary, it would be good if containers in the Hub follow the default technique to avoid confusing messages.

How does this change now that links are going away?

@MHBauer It definitely has a lower priority.

Going to close this as it's not a priority at all.


Remove techstack.yml file from the repository

Description

This PR removes the techstack.yml file from the repository.
The file contained metadata about the project and its dependencies.
This change may be part of a larger refactoring or project restructuring effort.

Changes walkthrough 📝
Relevant filesMiscellaneous

techstack.ymlComplete removal of techstack.yml file

techstack.yml
Deleted the entire techstack.yml file.  Removed all metadata and tool information related to the project.

+0/-191

:white_check_mark: Build GuiStracini.Mandae 8.0.609 completed (commit https://github.com/guibranco/GuiStracini.Mandae/commit/0615005f1b by @guibranco)


Feature request: cjxl: Optimize the output image when the input image is unoptimized

Is your feature request related to a problem? Please describe.
When inputting the unoptimized image, the file size of the output image will be larger than when inputting the optimized image.
Describe the solution you'd like
Optimize the output image by removing the alpha (if unnecessary), reducing color depth (if possible), etc.
I think either of the following is better:

Add an option to do this.
Make this the default behavior and add an option to not optimize (preserve the alpha, color depth, etc. of the input image).

Describe alternatives you've considered
Optimize the input image in advance.
Additional context
See #3413 for sample images and details.

Do you think this should be done in libjxl or cjxl?


Add Contribution.md DOC

##Fixes
Close issue #329
##Description
Add Contribution.md DOC with meaningful content and Step by Step guide for contributing to this project.
##Screenshot

Please merge my PR under GSSOC 2023

@silky-modi22 contributing.md pull request already made by someone with issue #285 so make changes in contributing.md then make pull request here .


[proposal] CUB 200-2011 dataset

This is a very widely used dataset in metric learning research and fine-grained recognition ("Birds dataset"): http://www.vision.caltech.edu/visipedia/CUB-200-2011.html
At few years ago I abused ImageFolder and CIFAR10 multiple inheritance classes to add this dataset:
class CUB2011(ImageFolder, CIFAR10):
	base_folder = 'CUB_200_2011/images'
	url = 'http://www.vision.caltech.edu/visipedia-data/CUB-200-2011/CUB_200_2011.tgz'
	filename = 'CUB_200_2011.tgz'
	tgz_md5 = '97eceeb196236b17998738112f37df78'

	train_list = [
		['001.Black_footed_Albatross/Black_Footed_Albatross_0001_796111.jpg', '4c84da568f89519f84640c54b7fba7c2'],
		['002.Laysan_Albatross/Laysan_Albatross_0001_545.jpg', 'e7db63424d0e384dba02aacaf298cdc0'],
	]
	test_list = [
		['198.Rock_Wren/Rock_Wren_0001_189289.jpg', '487d082f1fbd58faa7b08aa5ede3cc00'],
		['200.Common_Yellowthroat/Common_Yellowthroat_0003_190521.jpg', '96fd60ce4b4805e64368efc32bf5c6fe']
	]

	def __init__(self, root, transform=None, target_transform=None, download=False, **kwargs):
		self.root = root
		if download:
			self.download()

		if not self._check_integrity():
			raise RuntimeError('Dataset not found or corrupted.' +
							   ' You can use download=True to download it')
		ImageFolder.__init__(self, os.path.join(root, self.base_folder),
			transform=transform, target_transform=target_transform, **kwargs)

I extended this class for a metric learning task like this:
class CUB2011MetricLearning(CUB2011):
	num_training_classes = 100

	def __init__(self, root, train=False, transform=None, target_transform=None, download=False, **kwargs):
		CUB2011.__init__(self, root, transform=transform, target_transform=target_transform, download=download, **kwargs)
		self.classes = self.classes[:self.num_training_classes] if train else self.classes[self.num_training_classes:]
		self.class_to_idx = {class_label : class_label_ind for class_label, class_label_ind in self.class_to_idx.items() if class_label in self.classes}
		self.imgs = [(image_file_path, class_label_ind) for image_file_path, class_label_ind in self.imgs if class_label_ind in self.class_to_idx.values()]

@fmassa I would appreciate advice on how to make this proper. Multiple inheritance from CIFAR10 looks fragile to me (I remember mixing in CIFAR10 functionality was important, but don't remember why...). And I wonder if abstractions evolved since then: e.g. if manual download / check integrity calls are needed

I think if most of the functionality for downloading the file and extracting it are shared, there might be no need of inheriting from CIFAR10. All you use from it is the download function, which today just calls into download_and_extract and check_integrity basically.
I think just inheriting from ImageFolder might be enough.
And yes, we would gladly accept a PR adding support for CUB200

If I remember well, in this dataset there is also some extra information (like the instance bounding box), but I hope a PR can be accepted first even without them parsed.

Hi @vadimkantorov . I would like to work on this issue, it will give me the opportunity to study datasets in torchvision. Please let me know if you are fine with it or if you are already working on it and would avoid the repetition of work.
cc @fmassa

@fmassa I updated the code in OP (didn't test yet though:) ). I still had to copy download() and _check_integrity() from CIFAR dataset class.


add new configs for entity_paragraph

Hi @SaulLu, I have added the configs for entity_paragraph in this PR and made a few other minor updates.

Hi @manandey ,
Thanks for the PR.
I think we can leave this open until we decide what config we want.


[Repo]: Update README.md files to {Component}.md

This is to ensure consistency in the documentation going forward.

I noticed you used extension mdx in https://github.com/carbon-design-system/carbon-addons-iot-react/pull/2414/ Should we make that the standard?

I noticed you used extension mdx in #2414 Should we make that the standard?

Yes, but it's only valid after #2414 is merged because addon-docs adds the necessary libraries to handle it.


⚠ Bofrost.be has degraded performance

In 39f07ba, Bofrost.be (https://www.bofrost.be) experienced degraded performance:

HTTP code: 200
Response time: 925 ms

Resolved: Bofrost.be performance has improved in 0dfc71d after 7 minutes.


emacs-irony: fix build

Fixes #66556

Motivation for this change
Things done

[ ] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS)
Built on platform(s)

[ ] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nix-review --run "nix-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[ ] Fits CONTRIBUTING.md.

Notify maintainers
cc @

@GrahamcOfBorg build emacs-irony

@GrahamcOfBorg build emacsPackagesNg.melpaPackages.irony

@matthewbauer is irony working for you after this pr, I'm getting
(void-function irony-cdb-autosetup-compile-options)

and am trying to work out if I am doing something wrong or if the package has somehow gone weird after the update

ignore this, I have fixed the issue by adding (require 'irony-cbd) to my config


update standalone CLI doc

Update docs for standalone CLI.

This change is

/assign

Preview: https://deploy-preview-1416--competent-brattain-de2d6d.netlify.com/docs/pipelines/standalone-deployment-gcp/

/lgtm

@rmgogogo Please would you take a look at the remaining comments on this PR?

/close
I'm closing this PR due to lack of activity. Please do re-open it if there's still work ongoing.


Sort/Order by nested

How would I run a query similar to: sort({'embedded.field': -1})?
I have tried order_by(desc: "embedded.field") and order_by(fragment("embedded.field")) doesn't work.

order_by([schema], desc: schema._id) working fine

I was trying to get a similar query to work, and after some trial and error I think I found something that works:
from(c in SomeEctoSchema,
  # ...
  order_by: [desc: c."embedded.field"],
  # ...
)
|> MyApp.Repo.all()

The logs print:
{"$orderby", ["embedded.field": -1]}

And I get apparently correct results.


Optional Chaining not supported

Bug report

What is the current behavior?
Using Optional Chaining in TypeScript results in an error expecting a loader.
If the current behavior is a bug, please provide the steps to reproduce.
Set tsconfig.json => "target": "es2020"
let foo = 0
console.log(foo?.toString())

Compilation results in the error:
Module parse failed: Unexpected token (5:16)
File was processed with these loaders:
 * ./node_modules/ts-loader/index.js
You may need an additional loader to handle the result of these loaders.
| let foo = 0;
> console.log(foo?.toString());

What is the expected behavior?
Since ES2020 and TypeScript support Optional Chaining, the operator should be ignored by the loader and compilation should pass.

Other relevant information:
Stack Overflow indicates the issue lies with the webpack dependency Acorn, which implemented support for Optional Chaining in their 7.3.0 release from 3 days ago.
webpack version:  4.43
Node.js version: 14.3.0
ts-loader version: 7.0.5
Operating System: OS X Catalina 10.15.4 (19E287)
Additional tools:

dupe of https://github.com/webpack/webpack/issues/10227


Implementing MVVM architecture

The project does not follow any particular Android architecture. It would be great if anyone can implement MVVM (Model - View - ViewModel) architecture components in the project.

I would like to work on this issue but can u explain what exactly I need to do.

I would like to work on this issue but can u explain what exactly I need to do.

If you go through the code, you will be able to see that the code is not structured. Classes/Activities/Fragments are scattered randomly. I want the MVVM (Model-View-ViewModel) structural design pattern to be implemented in the project so that the entire code base gets organised.
Models hold application data. They’re usually structs or simple classes.
Views display visual elements and controls on the screen. They’re typically subclasses of UIView.
View models transform model information into values that can be displayed on a view. They’re usually classes, so they can be passed around as references.
For more info on MVVM, refer the link

If you go through the code, you will be able to see that the code is not structured. Classes/Activities/Fragments are scattered randomly. I want the MVVM (Model-View-ViewModel) structural design pattern to be implemented in the project so that the entire code base gets organised.
Models hold application data. They’re usually structs or simple classes.
Views display visual elements and controls on the screen. They’re typically subclasses of UIView.
View models transform model information into values that can be displayed on a view. They’re usually classes, so they can be passed around as references.
For more info on MVVM, refer the link

Can i work on this project

Can i work on this project

Do you mean this issue? @kenkirito

Do you mean this issue? @kenkirito

Yes

Yes

Ok! Assigning the issue. @kenkirito

Ok! Assigning the issue. @kenkirito

Hey @kenkirito, any updates?

Hey @kenkirito, any updates?

Sorry I am busy. I will get back to you tomorrow for updates

Sorry I am busy. I will get back to you tomorrow for updates

Any updates?

Any updates?

Working on it

Working on it

I will like to be assign to this task, i will like to implement with navigation component with this architecture if that's fine.

I will like to be assign to this task, i will like to implement with navigation component with this architecture if that's fine.

Glad to see your interest @Bik-Krlvn. You can start working on it.

Glad to see your interest @Bik-Krlvn. You can start working on it.

Ok

I'm having this trouble with this repo and gradle for awhile now, kinda thought i could make it work buh still no head way, I manage to make it run by turning of firebase deps but can't build my code neither is the layout xml files opening.
please you can unassign me till I manage my issues properly.

I believe you did not clone the repo properly in your system. Try it once again.

Hey @Bik-Krlvn, what is the status?

Please assign me this issue under GSSOC'23.


Not generating any identifier.

Hello there i tried to generate esdoc using thi project but i'm unable to see ANY classes. i've just one in my lib/ folder which contains this code:
class TestClass {
  oneMethod(){}
  testMethod(){}
}

I can see in ~./jsdoc/ast/sourcea file named test.js contaiing all the MethodDefinitions and the ClassDeclaration butidentifiers.html` is basically empty - here is an extract:

References

esdoc version is 4.0.1 and i'm using the following esdoc.json
{
  "source": "./lib",
  "destination": "./jsdoc"
}

Try using export class TestClass ...

@vekexasia Normally, ESDoc targets at export or export default identifier.
export class TestClass {
  oneMethod(){}
  testMethod(){}
}

Or, you can use config.unexportIdentifier: true. See https://esdoc.org/config.html#full-config for more information.

Thanks :)


Restore HA/Control deployment waits, optimize install

Description
Change how Charts are installed and resources waited for.
Restore lost Control and HA waits
After the Control and HA Charts were merged into the unified Chart, the ability to wait on these resources was lost, making CORTX_DEPLOY_CONTROL_TIMEOUT and CORTX_DEPLOY_HA_TIMEOUT in-effective. This code was restored and these deployments are waited on again.
CORTX Chart install
The --wait argument has been removed from the helm install command. Previously, if the Chart fails to install with the default wait timeout, then the deployment script fails completely. This leaves the cluster in a half-installed state. Second, the default timeout is 5 minutes, but we don't know for sure how long it should take. Rather, just let install without waiting and either let the user determine if it's complete, or let our other checks handle it. This gives the user the ability to make an corrective actions later.
Data / Server / Client Chart installs
Instead of installing these serially, which takes a good amount of time, they are now installed in parallel. Based on testing, there doesn't appear to be any dependency on these deployments.
Parallel waits
The deployment flow has been modified to wait for all CORTX deployments after all of the charts have been installed, and does each check in parallel. Previously these were done serially, after each Chart install. This helps reduce the total time of the deployment and provides feedback as deployments become ready.
Type of change

[X] Bug fix (non-breaking change that fixes an issue)
[X] New feature (non-breaking change that adds new functionality)
[ ] Breaking change (bug fix or new feature that breaks existing functionality)
[ ] Third-party dependency update
[ ] Documentation additions or improvements
[ ] Code quality improvements to existing code or test additions/updates

How was this tested?
Deployed a cluster and performed S3 I/O.
Set a timeout low for a deployment and observed it was reported as an error. Checked cluster after and it was working, as expected.
Additional information
Example output
Here's the new output with a timeout.
Waiting for all CORTX Deployments to become Ready..........
Deployment CORTX Control available after 9 seconds
..................................
Deployment CORTX Client available after 43 seconds
.......................
Deployment CORTX Data available after 66 seconds
......................
Deployment CORTX Server available after 89 seconds
........................................................................................................................................................
ERROR: Deployment CORTX HA timed out after 241 seconds

-----------------------------------------------------------

!!! ERROR: The CORTX cluster installation failed. There was a timeout waiting for one or more deployments.

The S3 data service is accessible through the cortx-io-svc-0 service.
   Default IAM access key: sgiamadmin
   Default IAM secret key is accessible via:
       kubectl get secrets/cortx-secret --namespace cortx \
                  --template={{.data.s3_auth_admin_secret}} | base64 -d

The CORTX control service is accessible through the cortx-control-loadbal-svc service.
   Default control username: cortxadmin
   Default control password is accessible via:
       kubectl get secrets/cortx-secret --namespace cortx \
                  --template={{.data.csm_mgmt_admin_secret}} | base64 -d

-----------------------------------------------------------

Related issues
Note that there are existing issues logged for initContainers never completing:

HA deployment: CORTX-31665
Control deployment: CORTX-31536

As a result, the deployment script will install everything, but report these deployments as having timed out, and the services won't be available. This can be verified with the status script. The workaround is to delete the Pods and let them be re-created. The benefit of this change is the ability to correct the cluster state w/o a complete tear-down, as everything has been installed.
Also note that it seems possible to perform S3 I/O with the default user w/o the control Pod ever coming up. I'm not sure if there's some setup that was partially done beforehand, or if that's expected...
Checklist

[X] The change is tested and works locally.
[ ] New or changed settings in the solution YAML are documented clearly in the README.md file.
[X] All commits are signed off and are in agreement with the CORTX Community DCO and CLA policy.

If this change addresses a CORTX Jira issue:

[ ] The title of the PR starts with the issue ID (e.g. CORTX-XXXXX:)

View rendered README.md

Going to resubmit this one.


Move testing from Jest+Enzyme to Jest+react-testing-library

Note to move dependencies to devDependencies


feat: Cross repo integration test triggers

Add triggers to run integration_tests after each repo merges using<EMAIL_ADDRESS>Add analysis tools to the workflow:

[ ] dart analyze
[ ] pana
[ ] papiana

Triggers:

[ ] at_app
[ ] at_client_sdk
[ ] at_libraries
[ ] at_server
[ ] at_services
[ ] at_tools
[ ] at_widgets

Not really needed right now or in the foreseeable future


Yotogi skill names translation

Describe the bug
while all the yotogi skill names are translated in the game, some of them are not being translated in MaidFiddler Yotogi Skills menu
Screenshots
https://imgur.com/a/wJGkqIZ
Version info (please complete the following information):

Game version [1.20.1]
Maid Fiddler version [<IP_ADDRESS>]

Additional context
I think the Japanese text is translated using google translation add-on in the game but it doesn't appear in english in MaidFiddler.

Hey!
Pardon the very late reply -- I have been pretty busy, so I didn't have the time to answer!
This is normal behaviour of Maid Fiddler. The tool gets the names of yotogi skills directly from game without going through AutoTranslate or other translation loaders. Maid Fiddler has its own translation file that you can use to translate yotogi skills. This is done to mainly simplify the development of Maid Fiddler and still allow skills to be translated in some way.


Incorrect fix for ObjectMeta issue in CloudStackMachineTemplateResource in v1beta3 API

/kind bug
What steps did you take and what happened:
[A clear and concise description of what the bug is.]
The v1beta3 API version bump was created to solve an issue related to creationTimestamp being null in a serialization of a CloudStackMachineTemplate resource (See related issues #166 #176 #237)
The solution applied here is to completely remove the ObjectMeta field from the CloudStackMachineTemplateSpec struct, but this is not the right fix for this issue, and against API conventions.
The main problem is that in v1beta1 and v1beta2, CloudStackMachineTemplateSpec is defined as this:
type CloudStackMachineTemplateResource struct {
	// Standard object's metadata.
	// More info: https://git.k8s.io/community/contributors/devel/sig-architecture/api-conventions.md#metadata
	// +optional
	// +nullable
	ObjectMeta metav1.ObjectMeta     `json:"metadata,omitempty"`
	Spec       CloudStackMachineSpec `json:"spec"`
}

while it should be defined as this:
type CloudStackMachineTemplateResource struct {
	// Standard object's metadata.
	// More info: https://git.k8s.io/community/contributors/devel/sig-architecture/api-conventions.md#metadata
	// +optional
	// +nullable
	ObjectMeta clusterv1.ObjectMeta  `json:"metadata,omitempty"`
	Spec       CloudStackMachineSpec `json:"spec"`
}

Note the metav1 vs clusterv1. I've verified this in various other CAPI infra providers (CAPA, CAPZ).
clusterv1.ObjectMeta has been specifically introduced in CAPI to address this issue with creationTimestamp. See the elaborate comment at https://github.com/kubernetes-sigs/cluster-api/blob/36e9abaa070c0bb0dfbfc81d2e2795b245797db5/api/v1beta1/common_types.go#L271
What did you expect to happen:
No complete removal of ObjectMeta in CloudStackMachineTemplateResource.

I suggest we fix this before the release of 0.4.9, and also consider including #328 (to make the API version bump worthwhile) and #329 (cc @g-gaston )


[stable/dask] Documentation error

Hi,
Look at the file: https://github.com/helm/charts/blob/master/stable/dask/README.md.
At the bottom there is a section called "Customizing Python Environment" and there is some code. This code is wrong (I tried it and it did not work). Apparently, you can't declare packages like this. If you look at https://github.com/helm/charts/blob/master/stable/dask/values.yaml , you see the correct version.

Thank you bot!


Add back lost docs on cert-manager components

These were somehow removed from the readme.

/assign @wallrj

/lgtm


Feature request: option to turn off "read more" on excerpt

e.g. [ap_excerpt read_more="off"]

I think you can do this by choosing the custom post option in settings and putting in only [ap_excerpt]


Line profile

@floryst @martinken @aylward @jourdain check it out! Widgets at work. h/t to @martinken for the rendering order explanation.


The available options and their effects on a Windows 10 1909 device are not reflecting the actual situation.

I am configuring the Endpoint Protection device configuration profile for our Windows 10 devices.
In the drop down list for "Local device security options>>Interactive Logon>>Display>>User information on lock screen"  we have only 3 options available (this is also mentioned in the documentation):
1 - User display name, domain and user names
2 - User display name only
3 - Do not display user information
But when setting this policy from the local security policy editor (secpol.msc) on a Windows 10 1909 device we see not 3 but 4 options:
1 - User display name, domain and user names
2 - User display name only
3 - Do not display user information
4 - Only domain and user names
The effects of setting this policy to option 1 results in displaying the "User display name, domain and user names" on the lock screen and only the password field being blanked out. This is the expected behavior for option 1.
The effects of setting this policy to option 2 results in displaying nothing and both username and password field being blanked out, so effectively this is option 3 being applied.  Obviously this is unexpected and unwanted behavior!
The effects of setting this policy to option 3 results in displaying nothing and both username and password field being blanked out. This is the expected behavior for option 3.
The effects of setting this policy to option 4 results in displaying the "User display name" on the lock screen and only the password field being blanked out, so effectively this is option 2 being applied. Obviously this is unexpected and unwanted behavior!
Please look into this and provide us with an Endpoint Protection device configuration profile that lists all 4 options and reflects the correct behavior of these options on the device.
Kind regards, Raimond

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

ID: 773b74b2-7dc2-d0e5-1e3b-bd6445bceda4
Version Independent ID: 34b55057-9078-6bd0-a9ac-2f6403b84be7
Content: Policy CSP - LocalPoliciesSecurityOptions - Windows Client Management
Content Source: windows/client-management/mdm/policy-csp-localpoliciessecurityoptions.md
Product: w10
Technology: windows
GitHub Login: @ManikaDhiman
Microsoft Alias: dansimp

@RaimondMoerman . Just now i checked on windows 10 v2004 build no 19041.331   by using Group policy admx templates and by using  secpol.msc
all the 4 options are displayed in both the ways.
so you must update the latest admx templates.

Hello, thank you for your andere but the issue I reported is not in the GPO templates but in the Intune MDM Device Configuration Profile for Endpoint Protection.
That only lists 3 options and does not give expected results.
Please look in to that.
Kind regards, Raimond
Get Outlook for Androidhttps://aka.ms/ghei36

From: VARADHARAJAN K<EMAIL_ADDRESS>Sent: Monday, July 13, 2020 8:36:18 PM
To: MicrosoftDocs/windows-itpro-docs<EMAIL_ADDRESS>Cc: RaimondMoerman<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Subject: Re: [MicrosoftDocs/windows-itpro-docs] The available options and their effects on a Windows 10 1909 device are not reflecting the actual situation. (#7155)
@RaimondMoermanhttps://github.com/RaimondMoerman . Just now i checked on windows 10 v2004 build no 19041.331 by using Group policy admx templates and by using secpol.msc
all the 4 options are displayed in both the ways.
so you must update the latest admx templates.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/MicrosoftDocs/windows-itpro-docs/issues/7155#issuecomment-657723282, or unsubscribehttps://github.com/notifications/unsubscribe-auth/ANSRQSUSNSCY5UVH3UGI7VTR3NH2FANCNFSM4OYOGG6Q.

@RaimondMoerman , Sorry i dont know about intune MDM configuration. Keep waiting for the expert users solutions

Hello @RaimondMoerman ,
In the drop down list for "Local device security options>>Interactive Logon>>Display>>User information on lock screen" we have only 3 options available (this is also mentioned in the documentation): - I am not sure why the option 4 is not added here, may be it is because this option is for on-premises domains only. Anyway, it will be added to the documentation only after it is added to the product.
The effects of setting this policy to option 2 results in displaying nothing and both username and password field being blanked out, - This is expected behaviour for any computer running Windows 10 later than 1607. To make long story short: after 1607, if Block user from showing account details on sign-in policy is not enabled (I believe its your case), only options 1 and 4 will show the domain\username. In case this policy is enabled, only display name is shown regardless of option you have chosen.
For more details, read https://docs.microsoft.com/en-us/windows/security/threat-protection/security-policy-settings/interactive-logon-display-user-information-when-the-session-is-locked#interaction-with-related-group-policy-settings. The article need to be updated (it doesn't show the option 4, I will create a PR to fix that), but it still provides the general information.
The effects of setting this policy to option 4 results in displaying the "User display name" on the lock screen and only the password field being blanked out - for me it displayed username and domain name as expected:

Are you sure you logged in as a domain user?
Thank you

@MaratMussabekov Tja k you for your reply.
We log in using our email address as the user name as the devices are Intune MDM managed Autopilot deployed Azure AD joined devices in strand of On-Premise Domain joined devices.
So what I want to achieve is to only show the e-mail address and below that the blanked out password field that we van user to unlock the device
Thank you again for your support!
Kind regards, Raimond


Unable to bring in as a dependency

I've tried adding this to my Modulefile, also tried calling it in my Puppetfile but librarian-puppet always returns "Cannot resolve the dependencies". Is there a problem with this module on the forge?

Could have been a temporary glitch as it seems to work just fine on my box...
$ puppet module install arioch-redis
Notice: Preparing to install into /Users/tom/.puppetlabs/etc/code/modules ...
Notice: Created target directory /Users/tom/.puppetlabs/etc/code/modules
Notice: Downloading from https://forgeapi.puppetlabs.com ...
Notice: Installing -- do not interrupt ...
/Users/tom/.puppetlabs/etc/code/modules
└─┬ arioch-redis (v1.1.1)
  └─┬ puppetlabs-apt (v2.1.1)
    └── puppetlabs-stdlib (v4.7.0)

I've cut a new release after posting my first reply. Works fine too...

I've run into this issue, as well. It occurs when puppetlabs-apt is used alongside arioch-redis, because the two specify allowed version sets for puppetlabs-stdlib that do not intersect.
puppetlabs-apt: >= 4.5.0 < 5.0.0 (https://github.com/puppetlabs/puppetlabs-apt/blob/f05e43c28588a280a42333f9ca3a506a7121849d/metadata.json#L11)
arioch-redis: >= 1.0.2 <3.0.0 (https://github.com/arioch/puppet-redis/blob/master/metadata.json#L12)


fix for feedback comment without rating

Issue #:
https://github.com/bcgov/met-public/issues/1389
Description of changes:

Changes made to add a new type for rating i.e. None. In case the user does not provide a rating the rating type we be considered as None.
Enable user to submit a comment with comment type but without selecting a rating

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of the met-public license (Apache 2.0).

Codecov Report

Merging #1425 (dbd4d56) into main (7c2b932) will decrease coverage by 0.01%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##             main    #1425      +/-   ##
==========================================
- Coverage   71.12%   71.11%   -0.01%
==========================================
  Files         324      324
  Lines        9391     9390       -1
  Branches      674      673       -1
==========================================
- Hits         6679     6678       -1
  Misses       2593     2593
  Partials      119      119

Flag
Coverage Δ

metapi
78.75% <100.00%> (+<0.01%)
:arrow_up:

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

Impacted Files
Coverage Δ

...eb/src/components/feedback/FeedbackModal/index.tsx
84.74% <ø> (-0.51%)
:arrow_down:

met-web/src/models/feedback.ts
100.00% <ø> (ø)

met-api/src/met_api/constants/feedback.py
100.00% <100.00%> (ø)

met-api/src/met_api/models/feedback.py
90.90% <100.00%> (ø)


[localization] status effects translated to german

Translated the status effects to german. I used the official GURPS translation from the 3rd Ed. where possible.
Status effects german translation.xlsx

@LordOHelmet If I provided you with the list of strings in English, could you provide the German translation for the parts of the game aid that we have applied internationalization to?

@LordOHelmet Could you review the attached data and let me know that I copied it all correctly? I'll paste the English version followed by the German:
English:
{
  "GURPS.shock": "Shock",
  "GURPS.STATUSCrouch": "Crouch",
  "GURPS.STATUSSit": "Sitting",
  "GURPS.STATUSKneel": "Kneeling",
  "GURPS.STATUSFly": "Flying",
  "GURPS.STATUSFall": "Falling",
  "GURPS.STATUSEuphoria": "Euphoria",
  "GURPS.STATUSCoughing": "Coughing",
  "GURPS.STATUSRetching": "Retching",
  "GURPS.STATUSDrowsy": "Drowsy",
  "GURPS.STATUSSleep": "Sleeping",
  "GURPS.STATUSDrunk": "Drunk",
  "GURPS.STATUSTipsy": "Tipsy",
  "GURPS.STATUSGrapple": "Grappled",
  "GURPS.STATUSPin": "Pinned",
  "GURPS.STATUSNauseated": "Nauseated",
  "GURPS.STATUSMildPain": "Mild Pain",
  "GURPS.STATUSModeratePain": "Moderate Pain",
  "GURPS.STATUSSeverePain": "Severe Pain",
  "GURPS.STATUSTerriblePain": "Terrible Pain",
  "GURPS.STATUSAgony": "Agony",
  "GURPS.STATUSBleed": "Bleeding",
  "GURPS.STATUSPoison": "Poisoned",
  "GURPS.STATUSBurn": "Burning",
  "GURPS.STATUSSuffocate": "Suffocating",
  "GURPS.STATUSReeling": "Reeling",
  "GURPS.STATUSExhausted": "Fatigued",
  "GURPS.STATUSAim": "Aiming",
  "GURPS.STATUSDisable": "Disabled",
  "GURPS.STATUSBlind": "Blinded",
  "GURPS.STATUSDeaf": "Deafened",
  "GURPS.STATUSSilence": "Sileneced",
  "GURPS.STATUSSprint": "Sprinting",
  "GURPS.STATUSStealth": "Sneaking/Stealth",
  "GURPS.STATUSWait": "Waiting",
}

German:
{
  "GURPS.shock": "Schock",
  "GURPS.STATUSCrouch": "Hockend",
  "GURPS.STATUSSit": "Sitzend",
  "GURPS.STATUSKneel": "Kniend",
  "GURPS.STATUSFly": "Fliegend",
  "GURPS.STATUSFall": "Fallend",
  "GURPS.STATUSEuphoria": "Berauscht",
  "GURPS.STATUSCoughing": "Hustend",
  "GURPS.STATUSRetching": "Würgend",
  "GURPS.STATUSDrowsy": "Schläfrig",
  "GURPS.STATUSSleep": "Schlafend",
  "GURPS.STATUSDrunk": "Betrunken",
  "GURPS.STATUSTipsy": "Angetrunken",
  "GURPS.STATUSGrapple": "Ergriffen",
  "GURPS.STATUSPin": "Festgehalten",
  "GURPS.STATUSNauseated": "Übelkeit",
  "GURPS.STATUSMildPain": "Leichter Schmerz",
  "GURPS.STATUSModeratePain": "Mäßige Schmerzen",
  "GURPS.STATUSSeverePain": "Starke Schmerzen",
  "GURPS.STATUSTerriblePain": "Furchtbare Schmerzen",
  "GURPS.STATUSAgony": "Qualen",
  "GURPS.STATUSBleed": "Blutend",
  "GURPS.STATUSPoison": "Vergiftet",
  "GURPS.STATUSBurn": "Brennend",
  "GURPS.STATUSSuffocate": "Erstickend",
  "GURPS.STATUSReeling": "Schwankend",
  "GURPS.STATUSExhausted": "Erschöpft",
  "GURPS.STATUSAim": "Zielend",
  "GURPS.STATUSDisable": "Funktionsunfähig",
  "GURPS.STATUSBlind": "Geblendet",
  "GURPS.STATUSDeaf": "Taub",
  "GURPS.STATUSSilence": "Stumm",
  "GURPS.STATUSSprint": "Sprintend",
  "GURPS.STATUSStealth": "Schleichend",
  "GURPS.STATUSWait": "Wartend"
}

@mjeffw
Looks good to me.
But for the sake of consistency please change this line
"GURPS.STATUSMildPain": "Leichter Schmerz",
to
"GURPS.STATUSMildPain": "Leichte Schmerzen",
Thanks for your effort!


Firebase crashlytics sperochon patch ios keys 1

Description
This PR correctly retrieve and send iOS keys data (known issue).
Related Issues
This PR fixes this issue:
FirebaseExtended/flutterfire#1145
Checklist
Before you create this PR confirm that it meets all requirements listed below by checking the relevant checkboxes ([x]). This will ensure a smooth and quick review process.

[X] I read the Contributor Guide and followed the process outlined there for submitting PRs.
[X] If the pull request affects only one plugin, the PR title starts with the name of the plugin in brackets (e.g. [cloud_firestore])
[] My PR includes unit or integration tests for all changed/updated/fixed behaviors (See Contributor Guide).
[X] All existing and new tests are passing.
[] I updated/added relevant documentation (doc comments with ///).
[X] The analyzer (flutter analyze) does not report any problems on my PR.
[X] I read and followed the Flutter Style Guide.
[X] I updated pubspec.yaml with an appropriate new version according to the pub versioning philosophy.
[X] I updated CHANGELOG.md to add a description of the change.
[X] I signed the CLA.
[X] I am willing to follow-up on review comments in a timely manner.

Breaking Change
Does your PR require plugin users to manually update their apps to accommodate your change?

[ ] Yes, this is a breaking change (please indicate a breaking change in CHANGELOG.md and increment major revision).
[X] No, this is not a breaking change.

When is this merge/update scheduled to be released?
Is there a way to use it now, since I don't get any logs for iOS.
StackOverflow issue: #60111182

@viktor-morin yes you can use it right now by replacing in your pub spec.yaml:
firebase_crashlytics: ^0.x.x
by
  firebase_crashlytics:
    git:
      url: git://github.com/sperochon/flutterfire.git
      ref: firebase_crashlytics-sperochon-patch-ios-keys-1
      path: packages/firebase_crashlytics

@viktor-morin yes you can use it right now by replacing in your pub spec.yaml:
firebase_crashlytics: ^0.x.x
by
  firebase_crashlytics:
    git:
      url: git://github.com/sperochon/flutterfire.git
      ref: firebase_crashlytics-sperochon-patch-ios-keys-1
      path: packages/firebase_crashlytics

@sperochon
I just got time to test this version, however the issue remains that no logs from iOS is reported. Console showing following:

flutter: Flutter error caught by Crashlytics plugin:
flutter: ══╡ EXCEPTION CAUGHT BY GESTURE════════════════════════════════
flutter: The following StateError was thrown while handling a gesture:
flutter: Bad state: Error thrown by Crashlytics plugin
flutter:
flutter: When the exception was thrown, this was the stack:
flutter: #0      Crashlytics.crash (package:firebase_crashlytics/src/firebase_crashlytics.dart:54:5)
flutter: #1      _InformationPageState.build. (package:skinfo/pages/informationpage.dart:69:42)
flutter: #2      _InkResponseState._handleTap (package:flutter/src/material/ink_well.dart:706:14)
flutter: #3      _InkResponseState.build. (package:flutter/src/material/ink_well.dart:789:36)
flutter: #4      GestureRecognizer.invokeCallback (package:flutter/src/gestures/recognizer.dart:182:24)
flutter: #5      TapGestureRecognizer.handleTapUp (package:flutter/src/gestures/tap.dart:486:11)
flutter: #6      BaseTapGestureRecognizer._checkUp (package:flutter/src/gestures/tap.dart:264:5)
flutter: #7      BaseTapGestureRecognizer.acceptGesture (package:flutter/src/gestures/tap.dart:236:7)
flutter: #8      GestureArenaManager.sweep (package:flutter/src/gestures/arena.dart:156:27)
flutter: #9      GestureBinding.handleEvent (package:flutter/src/gestures/binding.dart:222:20)
flutter: #10     GestureBinding.dispatchEvent (package:flutter/src/gestures/binding.dart:198:22)
flutter: #11     GestureBinding._handlePointerEvent (package:flutter/src/gestures/binding.dart:156:7)
flutter: #12     GestureBinding._flushPointerEventQueue (package:flutter/src/gestures/binding.dart:102:7)
flutter: #13     GestureBinding._handlePointerDataPacket (package:flutter/src/gestures/binding.dart:86:7)
flutter: #17     _invoke1 (dart:ui/hooks.dart:273:10)
flutter: #18     _dispatchPointerDataPacket (dart:ui/hooks.dart:182:5)
flutter: (elided 3 frames from package dart:async)
flutter:
flutter: Handler: "onTap"
flutter: Recognizer:
flutter:   TapGestureRecognizer#ddf74
flutter: ════════════════════════════════════════════════════════════════════════════════════════════════════
flutter: firebase_crashlytics: Error reported to Crashlytics.

@viktor-morin Sorry, I didn't look at your issue previously on stack overflow. This PR has nothing to do with your issue. This is another problem.

@viktor-morin Sorry, I didn't look at your issue previously on stack overflow. This PR has nothing to do with your issue. This is another problem.

No worries, was someone else that pointed me to this PR so thought they may had some connection. See if someone can solve that issue as well (feel's kind of mayor when it's not working for iOS)

@dsperochon apologies that this PR became stale. Given that Crashlytics has a major rework I think this PR needs an update or may not even be relevant at all now?
If it's still relevant would you mind raising a new up-to-date PR so we can track it easier, thanks.


Stats rest implementation

#11 implementation. Added stats dto's, urls and tests

@semashkinvg take a look at QueryStringParams.cs. I included datetime fromatting, which those query is valid for server. Is it a good idea?

@ridicoulous will do

@semashkinvg What went wrong with orders testing? I'm adding comments from bitmex docs to classes and fields now

Chat, Settelment, Schema will be later


Flutter crash report

command
flutter build apk --release --target-platform=android-arm64
exception
ProcessException: ProcessException: Permission denied
  Command: /Users/apple/Library/Android/flutter/.pub-cache/hosted/pub.dartlang.org/background_fetch-0.3.2/android/gradlew -I=/Users/apple/Library/Android/flutter/packages/flutter_tools/gradle/aar_init_script.gradle -Pflutter-root=/Users/apple/Library/Android/flutter -Poutput-dir=/Users/apple/Projects/Flutter/PixZ/PIXZ_Gitlab/build/app -Pis-plugin=true -Ptarget-platform=android-arm,android-arm64,android-x64 assembleAarRelease

#0      _DefaultProcessUtils.run (package:flutter_tools/src/base/process.dart:266:37)
<asynchronous suspension>
#1      buildGradleAar (package:flutter_tools/src/android/gradle.dart:548:33)
#2      buildPluginsAsAar (package:flutter_tools/src/android/gradle.dart:711:13)
#3      buildGradleApp (package:flutter_tools/src/android/gradle.dart:257:11)
#4      _AsyncAwaitCompleter.start (dart:async-patch/async_patch.dart:45:6)
#5      buildGradleApp (package:flutter_tools/src/android/gradle.dart:218:28)
#6      buildGradleApp (package:flutter_tools/src/android/gradle.dart:415:19)
#7      _asyncThenWrapperHelper.<anonymous closure> (dart:async-patch/async_patch.dart:73:64)
#8      _rootRunUnary (dart:async/zone.dart:1134:38)
#9      _CustomZone.runUnary (dart:async/zone.dart:1031:19)
#10     _FutureListener.handleValue (dart:async/future_impl.dart:139:18)
#11     Future._propagateToListeners.handleValueCallback (dart:async/future_impl.dart:680:45)
#12     Future._propagateToListeners (dart:async/future_impl.dart:709:32)
#13     Future._completeWithValue (dart:async/future_impl.dart:524:5)
#14     Future._asyncComplete.<anonymous closure> (dart:async/future_impl.dart:554:7)
#15     _rootRun (dart:async/zone.dart:1126:13)
#16     _CustomZone.run (dart:async/zone.dart:1023:19)
#17     _CustomZone.runGuarded (dart:async/zone.dart:925:7)
#18     _CustomZone.bindCallbackGuarded.<anonymous closure> (dart:async/zone.dart:965:23)
#19     _microtaskLoop (dart:async/schedule_microtask.dart:43:21)
#20     _startMicrotaskLoop (dart:async/schedule_microtask.dart:52:5)
#21     _runPendingImmediateCallback (dart:isolate-patch/isolate_patch.dart:118:13)
#22     _RawReceivePortImpl._handleMessage (dart:isolate-patch/isolate_patch.dart:175:5)

flutter doctor
[✓] Flutter (Channel stable, v1.12.13+hotfix.5, on Mac OS X 10.14.5 18F132, locale en-IN)
    • Flutter version 1.12.13+hotfix.5 at /Users/apple/Library/Android/flutter
    • Framework revision 27321ebbad (9 days ago), 2019-12-10 18:15:01 -0800
    • Engine revision 2994f7e1e6
    • Dart version 2.7.0

[✓] Android toolchain - develop for Android devices (Android SDK version 29.0.2)
    • Android SDK at /Users/apple/Library/Android/sdk
    • Android NDK location not configured (optional; useful for native profiling support)
    • Platform android-29, build-tools 29.0.2
    • ANDROID_HOME = /Users/apple/Library/Android/sdk
    • Java binary at: /Applications/Android Studio.app/Contents/jre/jdk/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 1.8.0_202-release-1483-b49-5587405)
    • All Android licenses accepted.

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

[✓] Android Studio (version 3.5)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin version 42.1.1
    • Dart plugin version 191.8593
    • Java version OpenJDK Runtime Environment (build 1.8.0_202-release-1483-b49-5587405)

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

[✓] Connected device (2 available)
    • Redmi Note 7 Pro  • 2a3ccabe                             • android-arm64 • Android 9 (API 28)
    • iPhone 11 Pro Max • 81366090-040D-4D9B-888D-C6D4E460B4FF • ios           • com.apple.CoreSimulator.SimRuntime.iOS-13-3 (simulator)

• No issues found!

Hi @Gursewak-Uppal
from your logs I see that the exception is caused by the 3rd party plugin background_fetch
you may want to open an issue in the dedicate github
Closing, as this isn't an issue with Flutter itself,
if you disagree please write in the comments and I will reopen it
Thank you


Replace 'new ImageData' with 'createImageData' in tf.toPixels

ImageData is not defined in NodeJS.
Instead, call createImageData on CanvasRenderingContext2D to create the
object to set the pixel bytes.
Description

For repository owners only:
Please remember to apply all applicable tags to your pull request.
Tags: FEATURE, BREAKING, BUG, PERF, DEV, DOC, SECURITY
For more info see: https://github.com/tensorflow/tfjs/blob/master/DEVELOPMENT.md

This change is

Hi @teruhisa :-)
Did you see the tfjs-node bindings: https://github.com/tensorflow/tfjs-node
We're working on splitting up these browser-specific ops soon. Core TensorFlow already has an API we need to surface for that in the tfjs-node package.
This is an interesting fix - let's discuss @dsmilkov @nsthorat

toPixels is now under tf.browser.toPixels so this won't be the method to use in Node.js. I think we should close this PR. We can add another nodejs specific API separately (and likely bind to C++).
Nick, should we file an issue to track that?

Either way, I verified this works by checking out the branch (so seems fine to merge). @teruhisa, can you update this to master and then sign the CLA and we can merge?

Sure, I'll make the update when I'm back.
On Tue, Mar 19, 2019, 10:11 AM Nikhil Thorat<EMAIL_ADDRESS>wrote:

Either way, I verified this works by checking out the branch (so seems
fine to merge). @teruhisa https://github.com/teruhisa, can you update
this to master and then sign the CLA and we can merge?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/tensorflow/tfjs-core/pull/1440#issuecomment-474387695,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAWq_5j-NgFY5RQRR0j1ga0pYfOUyHwNks5vYPApgaJpZM4ZJ9jA
.

Looks like the method was removed by #1590.
I might check out the node variant of tfjs library as nkreeger@ mentioned when I get the chance.
Given that this change is obsolete, closing the PR.


Clicking on "Open in Builder" from settings page does not respect the HTTP_BASE_URL

Describe the bug
Opening builder does not use HTTP_BASE_URL
To Reproduce
Steps to reproduce the behavior:
Set up an instance with  HTTP_BASE_URL set to a non default value
Go to settings, list workflows, edit workflow, click on open builder
Expected behavior
It opens the builder using the correct url including the HTTP_BASE_URL
Screenshots

Additional Context
This seems to be the guilty function
https://github.com/cortezaproject/corteza-webapp-admin/blob/8f162269f195fe582225bb9b908000b2b68b8947/src/components/Workflow/CWorkflowEditorInfo.vue#L210

Thank you for raising this issue, will be fixed in the upcoming 2022.9 release


Command not behaving as expected

I have an issue when running Set-NetConnectionProfile on a Virtual Machine with 2 seperate NICs. I have a requirement for one NIC to be a Private profile, the other is Public. When running the command against either of the 2 NIC Profiles (using -InterfaceAlias or -InterfaceIndex) the command always changes the profile for both connections.
I've tried using the command as per the example and it doesn't function at all. The "$Profile.NetworkCategory = Public" line fails with the following error:
"Public : The term 'Public' is not recognized as the name of a cmdlet, function, script file, or operable program. Check the spelling of the name, or if a path was included, verify that the path is correct and try again."
I only have this behaviour in a Server 2016 Virtual Machine, in Server 2012R2 it works as expected and I can set the profiles independently.
Any help would be appreciated! James.

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

ID: 18e96475-a601-25ea-66c8-0c646042cbfb
Version Independent ID: f42c8e2b-b5bd-cf1e-188b-7c4f7836a5b2
Content: Set-NetConnectionProfile (netconnection)
Content Source: docset/windows/netconnection/set-netconnectionprofile.md
Product: w10
GitHub Login: @coreyp-at-msft
Microsoft Alias: coreyp

Hello, @JFarthing  thank you for providing feedback.
Note that the form that you have submitted this feedback from is intended for product documentation issues only.
Please consider checking any relevant resources and asking the community about your problem via the following link.
PowerShell Community
Thanks.

@officedocsbot close


Cargo-apk allows run apk only from the workspace root

Currently, cargo-apk allows run apk only from workspace root. It would be awesome if cargo-apk can run apk from example project.
Test example:
https://github.com/Gordon-F/gfx/tree/bf7bdc12f93485616f6dd06dd574c2e7afe99e16
cd gfx/
cargo apk run --package quad_android --target x86_64-linux-android --features gl
error: --features is not allowed in the root of a virtual workspace

cd gfx/examples/quad_android
cargo apk run --package quad_android --target x86_64-linux-android --features gl
   Compiling quad_android v0.1.0 (/Users/indish/Dev/RustProjects/gfx/examples/quad_android)
    Finished dev [unoptimized + debuginfo] target(s) in 1m 18s
Error: No such file or directory (os error 2)

Hi @Gordon-F, thanks for reporting this!
I think however that we might have fixed this issue last week following https://github.com/rust-windowing/android-ndk-rs/issues/105. Can you update cargo-apk with the latest version of cargo-subcommand (by running cargo install --force cargo-apk) and let us know whether this remains to be a problem?

Hi @Gordon-F, thanks for reporting this!
I think however that we might have fixed this issue last week following https://github.com/rust-windowing/android-ndk-rs/issues/105. Can you update cargo-apk with the latest version of cargo-subcommand (by running cargo install --force cargo-apk) and let us know whether this remains to be a problem?

@MarijnS95 Thank you for your extremely quick answer! The issue is fixed with new update!

@MarijnS95 Thank you for your extremely quick answer! The issue is fixed with new update!


Fix bug of NETWORK_AUTO_DETECT

Hi,

If we got some IP address such as <IP_ADDRESS>, end with zero.
It match the rule "grep -v "0$" " and MON_IP won't set anything.
Using grep -w can help us.

CEPH_PUBLIC_NETWORK alwayls grep the network <IP_ADDRESS>/1.
Maybe using "grep -vE "^127|^0" " is a better choice.

Good catch, thanks!


Support assignment of a tuple

The following should not throw:
cfg = OmegaConf.create()
cfg.foo = (1,2)

closed by #75


Fixed ruby 2.0.0 incompatibility issues

Fixes #68

Coverage increased (+0.3%) to 77.134% when pulling 0ee0e584f05d7249f104aeaf46faa730c6ec7a0a on feature-ruby-downgrade into 962b77822aa7bd0fe0748175d0b58f2176c91dd3 on master.

@jcampbell05 when you get a chance can you do a quick check to make sure this looks good?


Display love icon when contributor has no avatar

Before:

After:

That's a good idea. But it will make the client call the Github twice. The user might barely notice the difference, though. Are you okay with that?

Yes, I am.


About where to find data process code.

Hello, I have recently been studying your method. Since I haven't touched Pytorch_lighting very much before, can you tell me where is the data preprocessing department in your program?

Our BaseDataset class inherits from torch-geometric and rules how raw and preprocessed data are organized.
The behavior of the datasets (and the answer to your question) is explained in the datasets documention. Please have a look.
Also, when instantiating a dataset, the logs explicitly tell you where the preprocessed data is stored.
PS: One important novelty of our BaseDataset class compared to torch-geometric is that we compute a hash to characterize the pre_transform (ie all the preprocessing steps) of a dataset. We use this hash to uniquely identify the preprocessing recipe you have used when first instantiating a dataset, which prevents recomputing / overwriting yoir preprocessed files when instantiating your dataset with a slightly different preprocessing reciper later on.


[GitHub Actions] - Updates in Issue Tagger to runs-on Self-hosted runners

DO NOT MERGE until the infrastructure is in place. Please talk to @dannymartinm @elink21  before merging
As part of the issue 21106, @elink21 and I have implemented the self-hosted runners for the Ubuntu and Windows operating systems.
Please see the Approved Pull Request (PR#22703) for reference.
In this PR you will find the following changes:

Updates in workflow Issue Tagger to runs-on self-hosted runners

Ubuntu

PR#16511 : Previous PR for reference.
BEAM-12812: Original Jira may contain additional context.

Thank you for your contribution! Follow this checklist to help us incorporate your contribution quickly and easily:

[ ] Choose reviewer(s) and mention them in a comment (R: @username).
[ ] Mention the appropriate issue in your description (for example: addresses #123), if applicable. This will automatically add a link to the pull request in the issue. If you would like the issue to automatically close on merging the pull request, comment fixes #<ISSUE NUMBER> instead.
[ ] Update CHANGES.md with noteworthy changes.
[ ] If this contribution is large, please file an Apache Individual Contributor License Agreement.

See the Contributor Guide for more tips on how to make review process smoother.
To check the build health, please visit https://github.com/apache/beam/blob/master/.test-infra/BUILD_STATUS.md
GitHub Actions Tests Status (on master branch)

See CI.md for more information about GitHub Actions CI.

Hi @Abacn @damccorm, could you help us with a PR review please?
Thank you!

Status update: reviewed by @damccorm, waiting for the migration window to deploy the infrastructure and request the merge.


Installing modules trigger script path warning

Prerequisites

[X] Write a descriptive title.
[X] Make sure you are able to repro it on the latest released version
[X] Search the existing issues.

Steps to reproduce
Install pester and it will trigger the script path warning.
Expected behavior
# No warning

Actual behavior
Install-PSResource pester
WARNING: The installation path for the script does not currently appear in the CurrentUser path environment variable. To make the script discoverable, add the script installation path, C:\Users\Thomas\Documents\PowerShell\Modules, to the environment PATH variable.

Error details
N/A

Environment data
beta20

Visuals
No response

I have this too when running Install-PSResource -Name Pester -Scope AllUsers, under PowerShell 5.1 with PSResourceGet 0.5.22.


The pintool is broken.

The pintool is broken. I shouldn't have merged it.
This isn't easy to get right because pintools isn't the right tool for the job. ltrace or LD_PRELOAD should be used to debug dynamic calls. If someone wants to fix this OK. Here are the issues:

There is no guarantee a called function will return using pintools because said function can jump into another one that isn't instrumented. For the pintool to be considered "working" it should handle this case and not be implementation dependent. This is a major issue. For example, on linux, calloc jumps into malloc at some point.
The pintool doesn't detect abnormal termination of the program. Abort or Segfault, etc. In general it should tell villoc that a function didn't return. Villoc will know what to do and mark corresponding blocks as failures. This is important.
The output isn't the same as ltrace's, (there is a missing space after the ',' in argument lists). This could also be fixed on villoc's side by changing the regex. Minor issue.

As an example, the following program produces wrong output:
#include <stdlib.h>

int	main(int argc, char **argv)
{
  (void) argc, (void) argv;

  void *a;
  int i;

  for (i = 0; i < 10; ++i)
    {
      a = malloc(i * 1000);
      if (i % 2)
	free(a);
    }

  return 0;
}

Produces this trace on my setup:
realloc(0x0, 1425) = 0x7fcdd4237758
realloc(0x0, 1169) = 0x7fcdd4237bf0
realloc(0x0, 16) = 0x7fcdd4237bf8
realloc(0x0, 32) = 0x7fcdc0f8f4a0
realloc(0x0, 1178) = 0x7fcdc0f8f950
realloc(0x0, 40) = 0x7fcdc0f8f988
realloc(0x0, 40) = 0x7fcdc0f8f998
realloc(0x0, 56) = 0x7fcdc0f8f9a8
malloc(0) = 0x2527010
malloc(1000) = 0x2527030
free(0x2527030) = <void>
malloc(2000) = 0x2527030
malloc(3000) = 0x2527810
free(0x2527810) = <void>
malloc(4000) = 0x2527810
malloc(5000) = 0x25287c0
free(0x25287c0) = <void>
malloc(6000) = 0x25287c0
malloc(7000) = 0x2529f40
free(0x2529f40) = <void>
malloc(8000) = 0x2529f40
malloc(9000) = 0x252be90
free(0x252be90) = <void>

Villoc shows the final state as being:

Which is obviously wrong as the program is perfectly valid and shouldn't produce overlapping blocks.
@sam-b maybe you could look at this?

Just made a pull request to address some of this - the overlapping blocks seem to be from the loader which does some funky memory allocation that wouldn't normally be valid, the version in the pull request avoids this by only instrumenting after main is called. I also fixed the minor formatting issue.
It should definitely be possible to detect abnormal program termination as people are pin for fuzzing instrumentation, would need to have a look into this though.
I'm not entirely sure if over coming the first issue is easily do-able.
Obviously if you don't think pin is appropriate for this use case you can feel free to remove it from the repository.

Thanks for the work and the investigation! I'll look into this soon.
I don't have anything against pin :) I think it's something lots of people would want to try and it's a good thing an example is available in the repository. What I'm saying is that for simply monitoring calls to malloc & friends hooking the shared library using ltrace or LD_PRELOAD is probably easier and more exact when this is possible. Obviously having the pin tool is very useful when analyzing statically linked stuff or other anti-debugging tricks!

I confirm seeing this issue on my Gentoo into my own pin tool. Maybe one solution is to filter the IMGs we instrument and skeep instrumentation of "ld-linux-x86-64.so.2" :
if (strncmp(IMG_Name(img).c_str(),"ld-linux",8) != 0)
{
	instrImageMalloc(img,v);
	instrImageNew(img,v);
	instrImageRealloc(img,v);
	instrImageCalloc(img,v);
}


gltf extras component

What problem does this solve or what need does it fill?
In my game i would like to store a few variables in the gltf "extras" field in blender and use them to set physics properties on the created entities.
What solution would you like?
Create a struct to store extras data, something like:
pub struct GltfExtras(pub serde_json::Map<String, serde_json::Value>)

and add it when creating entities in the gltf loader. I'm not sure as to what the best type to store the data would be,
according to the gltf spec they should be represented as an object whenever possible, so either a hashmap or something similar.
What alternative(s) have you considered?
I don't think there is another way to do this, but i would like suggestions if there are.
Additional context

An alternative would be to register extras with the gltf loadet such that every extra results in a typed component specific for such extra. This would be similar to app.register_type::<C>() that allows deserializing .ron scenes to concrete components.

Yeah that would work.
Now that you mention it all of these types would have to have register_type<T>() called on them already in order
to be used with in a '$scene.ron' file correct?
In that case would it be possible for the gtlf loader and just load each extra against the types
registered with the scene type registry?
I'm very new to bevy so forgive me if that isn't phrased correctly

Could you share a GLTF file with those extra set?

The One i'm working on is very large so i made a simple example file of a cube with 1 'String' property and 1 array property. (As well as one from blenderkit). I have no problem sharing the larger one if that helps it but only has a single property set on one of the mesh objects.
Github wouldn't let me upload a gltf file so here it is as txt.
custom_prop.txt
I'm not sure exactly what schema i would use for now i only want to create simple Colliders with rapier3d


compile failed using JUJU docker file

Hi, I was trying to compile JUJU image using the docker file that JUJU project provide but got this error message, could anyone help me solve this?

COPY failed: stat /var/lib/docker/tmp/docker-builder158877161/_/bin/jujud: no such file or directory

And other than using docker file to deploy, is there any other automatic deploy methods to deploy juju and other open source projects?

Hi @Houzonglaile
Could you provide us with some more context here? At a high level, what are you trying to do? What commands have you run?
The juju controller image can be found in dockerhub. You shouldn't need to compile your own.
You may want to consider having a look through some Juju tutorials in our docs?
https://juju.is/docs/juju/tutorial

COPY ${TARGETOS}${TARGETARCH}/bin/jujud /opt/
COPY ${TARGETOS}${TARGETARCH}/bin/jujuc /opt/
COPY ${TARGETOS}${TARGETARCH}/bin/containeragent /opt/
COPY ${TARGETOS}${TARGETARCH}/bin/pebble /opt/
hi,@Jack Shaw
We used the dockerfile（https://github.com/juju/juju/blob/3.6/caas/Dockerfile） provided in your Juju project to package and create an image with the following error message. It is likely that there are some project directories missing, which caused the copy to fail. Under what circumstances were these directories generated? Can you provide more detailed context for compiling and building?
[2024/11/06 14:34:19.511 GMT+08:00] Step 25/29 : COPY ${TARGETOS}${TARGETARCH}/bin/jujud /opt/
[2024/11/06 14:34:19.686 GMT+08:00] [ERROR] [制作镜像并推送到SWR仓库:external_docker_builder] : 执行docker命令失败,错误信息为:COPY failed: stat /var/lib/docker/tmp/docker-builder158877161//bin/jujud: no such file or directory


Name and Namespace are duplicate in some log lines

Bug description:
Some log lines have duplicate entries for the name and namespace keys.
They appear to be for any of the loggers prefixed with controller. e.g.:
controller.pipeline-service
controller.supply-chain
controller.workload"

Steps to reproduce:
Deploy and workload and you should see log lines similar to this in the controller's logs:
{"level":"info","ts":1634771997.9039302,"logger":"controller.workload","msg":"finished","name":"my-web-app","namespace":"default","name":"my-web-app","namespace":"default"}

Expected behavior:
Name and namespace should only appear once.
Actual behavior:
Name and namespace are duplicated.

Okay, I've reviewed and this is no longer an issue in the codebase.
I think the contributing guide is good in that it refers to the controller runtime advice, which is to log whole objects. Hopefully that's not too subtle, and certainly something we can all be aware of when reviewing.


Tapo tc85 firmware 1.2.21 401 Unauthorized

TP-Link is slowly rolling out 1.2.21 for certain Tapo cameras, the tc85 included. While the tapo:// protocol works on our cameras running the prior version, cameras that have received this automatic update don't seem to function. These cameras are receiving the typical streams: 401 Unauthorized" stream=streamxyz.
Note that both the md5 and sha256 methods were attempted.
Curious if anyone else has experienced this, and assuming it's reproducible, if there's anything that can be done to help get a fixed merged.

You are not the only one https://github.com/AlexxIT/go2rtc/issues/849

@dgibbs64 Interesting. I must've missed that issue since it's with another model number.
I am noticing that the discussed models and the discussed firmware versions are all different. Also, tapo:// worked wonderfully on this model up until this specific firmware version. Should probably still close thing one as a duplicate and keep them together.


Update TextField value props to string

These are generating prop type warnings.

@RyanMcDonald Nice, Thanks!


Idea: create a separate curl2httpie tool

My idea is simple:
Create a tool (e.g. curl2httpie) which accepts all the commands curl also does.
It tries to parse a curl command with all options

if it fails, by default it executes the command via curl
if it succeeds,by default it prints out the corresponding httpie command
by default, it runs the corresponding httpie command

It could be symlinked to curl, so any time you copy paste a curl command into your terminal, you learn how you would have achieved the same command with httpie plus you get all the benefits in output and so on of actually running httpie.
What do you think?
The reason I am asking here is to make sure I did not miss something like this if it already exists.
I would enjoy implementing this, but only if it does not exist yet.

 It's only convert curl command to httpie command.
$ curl2httpie curl -u user: github.com
http -a user: GET github.com

and there is a curl2httpie online app to do this.
welcome you to contributing!

As it already does exists,
I am closing this one.