Add readFile method to Filesystem component

Apologies if this PullRequest should not be submitted like this.
I'm rather new to GitHub.
I would love to see read file integration since this is a crucial part of the filesystem.
Is there a reason for this not begin implemented in the component?
(e.g. should I include my own fork)

@fabpot I'm not clear on exactly what you are saying here - is it that reading a file is an easy operation and therefor doesn't need to be handed by this class?  I can agree with that - but I would still very much like to see a read file option here for because it means that if I depend on the filesystem service I can mock it in tests including reading a file.  Otherwise I have to jump through the hoop of creating a class which extends filesystem and provides only this one additional method.
If there is any chance of including a read method I would be happy to bring this PR up to date and add some tests.

I have to use 2 classes in my filesystem operations, just because this method is missing.

Which doesn't make sense, as RUD part of CRUD is already there.

What is the proper way to read a file? The file_get_contents function seems low-level. Since the dumpFile() method exists, it seems natural that there would be a reverse command.


feature/ansible_swarm

:tada: This PR is included in version 2.35.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Ask users to verify email to set up API keys

Refs #6256

Always display the panel, but grey out or disable it

Updates 2fa and API section to specifically instruct users to verify their primary email (this is more explicit than just greying it out)
Links to email section on account admin page, so users can resend verification email.

@ewdurbin @woodruffw  what would be even nicer here:

Change message to To add two factor authentication to your account, first verify your primary email.  [Resend verification email](link that sends verification email).

WDYT?

This is good as is, though your suggested improvement could be reasonable! I'm not sure if it is crucial that we put the link right in the message though, seems best to leave the behavior in a single place.


Check tenant when reusing public net

Signed-off-by: Jiri Prokes<EMAIL_ADDRESS>

@rbbratta @mbanszex

Well, still we don’t want to have IP. Just use xxx.xxx.xxx.xxx.
From: Jennifer Li<EMAIL_ADDRESS>Sent: Thursday, January 26, 2017 4:15 PM
To: taf3/taf<EMAIL_ADDRESS>Cc: Park, Seungweon<EMAIL_ADDRESS>Comment<EMAIL_ADDRESS>Subject: Re: [taf3/taf] Check tenant when reusing public net (#17)
@lijenni commented on this pull request.

In taf/testlib/virtual_env.pyhttps://github.com/taf3/taf/pull/17:

@@ -28,7 +28,9 @@

"instance_type": "settings",
"id": "993",
"images_share_path": "/mnt/berta/oses/openstack",
-#  "mgmt_ip_cidr": "<IP_ADDRESS>/8",
+#  "external_net_ip_cidr": "<IP_ADDRESS>/24",
+#  "external_net_pool_start": "100",
+#  "external_net_pool_end": "199",
The ip information are just examples. They don't actually correspond to anything.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHubhttps://github.com/taf3/taf/pull/17, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AV944N07cjW8iv7iHQOujlHAp1Ahh5rkks5rWTb6gaJpZM4LsEMq.

Thank you for the comments, but where is the latest code?

Reviewed 1 of 2 files at r2, 1 of 1 files at r3.
Review status: all files reviewed at latest revision, 12 unresolved discussions, some commit checks failed.

taf/testlib/virtual_env.py, line 302 at r1 (raw file):
Previously, mskrocki (Maciej Skrocki) wrote…
we just created the network, how can it have a subnet?

We may have just reused an existing network (see line 300), notice how the logging about "No reuse" is now inside an else block.

taf/testlib/virtual_env.py, line 461 at r2 (raw file):
Previously, jiriproX wrote…
I've moved check for external_net to the beginning. I've removed router_id which were not used.
Rest remain the same as proposed solution is not working as it is.
show_subnet needs subnet_id,  net_id does not contain 'subnet' as it is just Id. I replaced net_id with corresponding network.
Intersection provides empty set. There seems to be wrong type of object to intersect with.
I stop next further investigation here. Original solution was tested and it is working.

OK, yes, my suggestion was not fully baked and it was further than needed.  I do think that the log_and_return method is something that is generally useful.  In some cases this is done with a decorator, but that may not work/be appropriate in all cases.  Thank you for doing the other suggestions.

taf/testlib/virtual_env.py, line 373 at r3 (raw file):

                continue
            net_id = ext_gw_info.get('network_id')
            if net_id and net_id in nets:

This looks like another continue and it can be combined with the previous lines:
net_id = router.get('external_gateway_info', {}).get('network_id')
if not net_id or net_id not in nets:
    continue
if tenant_id and tenant_id == router.get('tenant_id'):
    self.net_2_router_map[net_id] = router['id']
elif net_id not in self.net_2_router_map:
    self.net_2_router_map[net_id] = None

taf/testlib/virtual_env.py, line 377 at r3 (raw file):

                    self.net_2_router_map[net_id] = router['id']
                else:
                    if not net_id in self.net_2_router_map:

Why not combine this if with the previous else for an elif?

Comments from Reviewable

Review status: all files reviewed at latest revision, 11 unresolved discussions, some commit checks failed.

taf/testlib/virtual_env.py, line 31 at r1 (raw file):
Previously, jiriproX wrote…
external_net_ip_cidr was renamed to external_net_gw_ip_cidr as it have to contain gateway IP in cidr notation.
New example value:  # "external_net_gw_ip_cidr": "<IP_ADDRESS>/24"

Done.

taf/testlib/virtual_env.py, line 33 at r1 (raw file):
Previously, rbbratta (Ross Brattain) wrote…
192.168 is okay or other RFC 1918 addresses are okay as long they don't match actual internal subnets.

Done.

taf/testlib/virtual_env.py, line 253 at r1 (raw file):
Previously, jiriproX wrote…
Done

Done.

taf/testlib/virtual_env.py, line 283 at r1 (raw file):
Previously, esmacgil (Edward MacGillivray) wrote…
Yes, please add any constants instead of using magic numbers.  The value 100 seemed to be the previous minimum, see line 304 of the previous revision and line 305 for the value 254.  If we have a reason to open up this range, then make those values instead of mine.

Done.

taf/testlib/virtual_env.py, line 286 at r1 (raw file):
Previously, mskrocki (Maciej Skrocki) wrote…
Why do we need tanant_id?

We were getting all public networks and all routers. Nevertheless we need to check if router for corresponding tenant exist. Public network is visible across all tenants. Routers are tenant specific.

taf/testlib/virtual_env.py, line 302 at r1 (raw file):
Previously, esmacgil (Edward MacGillivray) wrote…
We may have just reused an existing network (see line 300), notice how the logging about "No reuse" is now inside an else block.

Yes, in case of reused public network subnet is  already present.

taf/testlib/virtual_env.py, line 306 at r1 (raw file):
Previously, jiriproX wrote…
Done.

Done.

taf/testlib/virtual_env.py, line 352 at r1 (raw file):
Previously, esmacgil (Edward MacGillivray) wrote…
I missed the fact that net in my suggestion is only used for the conditional expression, so we don't need the net = nets.get('net_id') and the conditional expression can be as it has been: if net_id and net_id in nets

Done.

taf/testlib/virtual_env.py, line 373 at r3 (raw file):
Previously, esmacgil (Edward MacGillivray) wrote…
This looks like another continue and it can be combined with the previous lines:
net_id = router.get('external_gateway_info', {}).get('network_id')
if not net_id or net_id not in nets:
    continue
if tenant_id and tenant_id == router.get('tenant_id'):
    self.net_2_router_map[net_id] = router['id']
elif net_id not in self.net_2_router_map:
    self.net_2_router_map[net_id] = None

Done.

taf/testlib/virtual_env.py, line 377 at r3 (raw file):
Previously, esmacgil (Edward MacGillivray) wrote…
Why not combine this if with the previous else for an elif?

Done.

Comments from Reviewable

Reviewed 1 of 1 files at r5.
Review status: all files reviewed at latest revision, 7 unresolved discussions.

Comments from Reviewable

Reviewed 1 of 2 files at r2, 1 of 1 files at r5.
Review status: all files reviewed at latest revision, 7 unresolved discussions.

Comments from Reviewable

Reviewed 1 of 2 files at r2, 1 of 1 files at r5.
Review status: all files reviewed at latest revision, 5 unresolved discussions.

Comments from Reviewable

Review status: all files reviewed at latest revision, 5 unresolved discussions.

Comments from Reviewable

-2

Review status: all files reviewed at latest revision, 5 unresolved discussions.

Comments from Reviewable

Review status: all files reviewed at latest revision, 5 unresolved discussions.

taf/testlib/virtual_env.py, line 328 at r5 (raw file):

        return waiters.wait_for_server_status(self.handle.servers_client, vm_id, status)

    #TODO: JP check if create_external_router is needed

For this review to move forward we need to make this investigation: Do we need to create router at all? if not, then we just need to simplify.

Comments from Reviewable

Review scrub: need confirmation from Martin B

Review status: all files reviewed at latest revision, 5 unresolved discussions.

Comments from Reviewable

-1

Reviewed 1 of 1 files at r6.
Review status: all files reviewed at latest revision, 6 unresolved discussions.

taf/testlib/virtual_env.py, line 419 at r6 (raw file):

        _net_cfg.public_router_id = public_router['id']

        if _net_cfg.public_network_id and _net_cfg.public_router_id:

Please return the result of this evaluation instead of the literal values:
return _net_cfg.public_network_id and _net_cfg.public_router_id

Comments from Reviewable

review scrub: what's the status on this?

I requested a change:

virtual_env.py:419

And there are several discussions that are pending Maciej's assent to:

virtual_env.py:31
virtual_env.py:359
virtual_env.py:389
virtual_env.py:492

so one of the following needs to happen:

Maciej assents
we need to provide more evidence that he should assent
the code needs to be changed so that he can assent

The reason I know this is because (on Reviewable) each discussion thread has

a set of circles in the lower-right corner of the discussion box
the top circle shows the active user's level of engagement, one of:

ellipsis: following but neutral on resolving
check mark in dark circle: satisfied ("OK"), what i'm calling assent
check mark in light circle: discussing and not blocking
hyphen in dark circle: blocking
red circle: discussing but not satisfied

this is not a state in the list, but is the result of starting a discussion

click on the top circle and the other circles will

expand leftward showing the state of other users
allow the current user to set their state

The current user can also alter their state by commenting in the discussion.

Review status: all files reviewed at latest revision, 6 unresolved discussions, some commit checks failed.

taf/testlib/virtual_env.py, line 419 at r6 (raw file):
Previously, esmacgil (Edward MacGillivray) wrote…
Please return the result of this evaluation instead of the literal values:
return _net_cfg.public_network_id and _net_cfg.public_router_id

Still waiting for this change

Comments from Reviewable

Closing this, changes done in Maciej's pull request #33


Enables dependabot version upgrade

What this PR does / why we need it
This yaml config should enable dependabot version upgrades for our many go modules. This will open bot PRs for upgradeable dependencies in our go mods.
As documented here: https://docs.github.com/en/code-security/supply-chain-security/keeping-your-dependencies-updated-automatically/enabling-and-disabling-dependabot-version-updates
Details for the Release Notes (PLEASE PROVIDE)

Enables dependabot PRs

Which issue(s) this PR fixes
N/a
Describe testing done for PR
Will have to wait and see it work in GitHub 👀
Special notes for your reviewer
N/a

Should we exclude tanzu-framework, since we know we're not going to be actively updating to every new version?

Should we exclude tanzu-framework

That's a good point. Especially since we update those manually for our releases. Looking at a way to ignore those now

Yeah, let's merge and see what shakes out :)


The first copy, you need to click twice to copy successfully.Tested Google, firefox, 360 security three browsers are the same。

You must have an issue with your implementation.
The demosite works without any issues: https://clipboardjs.com/ (just verified it in Chrome)

When the data is obtained asynchronously, the first copy needs to be clicked twice.How to solve?

I can't take the DOM element with $nextTick in vue's mouted, so I can't pre-initialize it.

The problem was resolved because I didn't get the DOM elements right.

@inputherf then please close Thisted issue

look：

I met the same problem

The problem was resolved because I didn't get the DOM elements right.

Thank you for your answer .The problem is really with the DOM binding


Add more choices to the "srb init" question

Problem
Only one choice, yes, to do for "srb init"
Proposed solution
Suggest that you add more choices to the srb init question so users could better self-diagnosis init problems. They could provide more information in issue reports too.
The current srb init question is:

Would you like to continue? [Y/n]

SUGGESTIONS: Like H, V, S, E, I where

H(elp) would provide "srb init" usage/help information (existing issue: #1003)
V(erbose), T(ell me more), D(ebug) would add more detailed   information such as the phases milestones, etc.
S(ilent) turns off all sorbet-generated warnings and errors so you can see where the messages are coming from
E(rrors) keeps only sorbet-generated errors
I(nfor) turns off all sorbet-generated information-only warnings.

Example: rails app:update has multiple choices: (enter "h" for help) [Ynaqdhm]
One last thing - you can break into request into multiple PRs.
Thanks

Found that I can separate the stdout and stderr output into separate files but that also losses context.

Wishful thinking.


Private network - define CIDR by variable

We should find a solution with which we can define the private network by variable. Similar to the POD and SERVICE CIDR
https://github.com/cluster-api-provider-hcloud/cluster-api-provider-hcloud/blob/ee9b0d534bea1d3fd8f3ca434d41099c5314f449/pkg/cloud/resources/network/network.go#L42-L54

could be set in the new cluster-template.yaml file


do not print list using warn()

otherwise we will have
Exception occurred:
  File "/home/jenkins-build/build/workspace/ceph-pr-docs/build-doc/virtualenv/local/lib/python2.7/site-packages/sphinx/util/logging.py", line 338, in filter
    raise SphinxWarning(record.msg % record.args)
TypeError: unsupported operand type(s) for %: 'list' and 'tuple'

 Sphinx version: 1.6.3
# Python version: 3.5.4 (CPython)
# Docutils version: 0.13.1 release
# Jinja2 version: 2.9.6
# Last messages:
#   writing output... [ 99%] start/quick-rbd
#
#   writing output... [ 99%] start/quick-rgw
#
#   writing output... [ 99%] start/quick-rgw-old
#
#   writing output... [100%] start/quick-start-preflight
#
#
#
# Loaded extensions:
#   sphinx.ext.todo (1.6.3) from /var/ceph/ceph/build-doc/virtualenv/lib/python3.5/site-packages/sphinx/ext/todo.py
#   breathe (4.7.2) from /var/ceph/ceph/build-doc/virtualenv/src/breathe/breathe/__init__.py
#   sphinxcontrib.ditaa (unknown version) from /var/ceph/ceph/build-doc/virtualenv/src/sphinx-ditaa/sphinxcontrib/ditaa.py
#   alabaster (0.7.8) from /usr/lib/python3/dist-packages/alabaster/__init__.py
#   sphinx.ext.autodoc (1.6.3) from /var/ceph/ceph/build-doc/virtualenv/lib/python3.5/site-packages/sphinx/ext/autodoc.py
#   sphinx.ext.graphviz (1.6.3) from /var/ceph/ceph/build-doc/virtualenv/lib/python3.5/site-packages/sphinx/ext/graphviz.py
Traceback (most recent call last):
  File "/var/ceph/ceph/build-doc/virtualenv/lib/python3.5/site-packages/sphinx/cmdline.py", line 306, in main
    app.build(opts.force_all, filenames)
  File "/var/ceph/ceph/build-doc/virtualenv/lib/python3.5/site-packages/sphinx/application.py", line 333, in build
    self.builder.build_all()
  File "/var/ceph/ceph/build-doc/virtualenv/lib/python3.5/site-packages/sphinx/builders/__init__.py", line 288, in build_all
    self.build(None, summary='all source files', method='all')
  File "/var/ceph/ceph/build-doc/virtualenv/lib/python3.5/site-packages/sphinx/builders/__init__.py", line 397, in build
    self.write(docnames, list(updated_docnames), method)
  File "/var/ceph/ceph/build-doc/virtualenv/lib/python3.5/site-packages/sphinx/builders/__init__.py", line 434, in write
    self._write_serial(sorted(docnames))
  File "/var/ceph/ceph/build-doc/virtualenv/lib/python3.5/site-packages/sphinx/builders/__init__.py", line 443, in _write_serial
    self.write_doc(docname, doctree)
  File "/usr/lib/python3.5/contextlib.py", line 66, in __exit__
    next(self.gen)
  File "/var/ceph/ceph/build-doc/virtualenv/lib/python3.5/site-packages/sphinx/util/logging.py", line 225, in pending_warnings
    memhandler.flushTo(logger)
  File "/var/ceph/ceph/build-doc/virtualenv/lib/python3.5/site-packages/sphinx/util/logging.py", line 191, in flushTo
    logger.handle(record)
  File "/usr/lib/python3.5/logging/__init__.py", line 1431, in handle
    self.callHandlers(record)
  File "/usr/lib/python3.5/logging/__init__.py", line 1493, in callHandlers
    hdlr.handle(record)
  File "/usr/lib/python3.5/logging/__init__.py", line 857, in handle
    rv = self.filter(record)
  File "/usr/lib/python3.5/logging/__init__.py", line 716, in filter
    result = f.filter(record)
  File "/var/ceph/ceph/build-doc/virtualenv/lib/python3.5/site-packages/sphinx/util/logging.py", line 338, in filter
    raise SphinxWarning(record.msg % record.args)
TypeError: unsupported operand type(s) for %: 'list' and 'tuple'


{Appservice} Fix appservice tests after storage bump

Description

Testing Guide

History Notes

[Component Name 1] BREAKING CHANGE: az command a: Make some customer-facing breaking change.
[Component Name 2] az command b: Add some customer-facing feature.

This checklist is used to make sure that common guidelines for a pull request are followed.

[ ] The PR title and description has followed the guideline in Submitting Pull Requests.

[ ] I adhere to the Command Guidelines.

[ ] I adhere to the Error Handling Guidelines.

Appservice


Price list depends on API labels for i18n keys

The price list localizes several strings from the cost-model API. The code appears to depend on these API's label_metric string as the i18n key for our localized strings? For example, the node currency rate i18n key below appears to be one of these labels.
"price_list": {
      "modal": {
        "applied_usage_date_range": "Applied usage date range",
        "applied_usage_range": "Applied usage range",
>>> "cluster currency rate": "Cluster price rate tier {{index}} ({{ unit }})",
        "compute request rate": "Compute request rate tier {{ index }} ({{ unit }})",
        "compute usage rate": "Compute usage rate tier {{ index }} ({{ unit }})",
>>> "cpu request rate": "CPU request rate tier {{ index }} ({{ unit }})",
>>> "cpu usage rate": "CPU usage rate tier {{ index }} ({{ unit }})",
        "hours": "hourly",
>>> "memory request rate": "Memory request rate tier {{ index }} ({{ unit }})",
>>> "memory usage rate": "Memory usage rate tier {{ index }} ({{ unit }})",
        "metric": "Metric",
        "month": "monthly",
        "no_match": "No rates were found",
        "no_range_set": "No range set",
>>> "node currency rate": "Node price rate tier {{index}} ({{ unit }})",
        "not_available": "N/A",
>>> "storage request rate": "Storage request rate tier {{ index }} ({{ unit }})",
>>> "storage usage rate": "Storage usage rate tier {{ index }} ({{ unit }})",
        "title": "{{name}} cluster price list",
        "value": "Value",
>>> "volume request rate": "Volume request rate tier {{ index }} ({{ unit }})",
>>> "volume usage rate": "Volume usage rate tier {{ index }} ({{ unit }})"
      }
    },

Unfortunately, when the API label changes, or new ones are added, it breaks the UI. See the screenshot below from issue #1513.

We need to refactor the code so we don't depend on API labels as i18n keys. Or, the API should localize its own strings.
Looking at the cost-models API, it may be possible to use the metric name instead of its label_metric property? That ID seems less likely to change.

Hey @dlabrecq
I probably didn't understand your suggestion but what will happen when a new metric is added like "memory_kb_usage_per_month"? wouldn't it be shown wrongly because it doesn't exist in en.json?
I think the only way to solve it is to keep track on new API metrics or to ask from the backend team to translate the labels in the response.

I was basically asking if there may be a better key that we can rely on? Obviously, when a new metric is added, this will break every time.
I agree that the API should be translating its own labels.

FYI, I created https://github.com/project-koku/koku/issues/2104 to get the discussion going.

IMO, we should not be using the labels as keys. We should display the labels as is. For example, consider creating a string like so:
"rate": "{{metric_label}} tier {{index}} ({{ unit }})",
That way, this won't break when a new metric label is introduced

Per today's backlog discussion, the team agreed that the API will localize its own strings. There will be an epic created to discuss this for the next milestone.
Ultimately, when the API returns a translated string (e.g., German, French, etc.), it will not match our i18n keys and the UI will break. Thus, the UI should display the labels as is.

Created Jira issue:f https://issues.redhat.com/browse/COST-50


input-data option should read data-readings from file not, the whole payload of the data-readings request.

At the moment, when input-data is specified, the agent read the file and blindly sends its content to the server.
Instead of doing that, it should craft an actual DataReadingPost object. The content of the file is the source of DataReadings, not the source of all the payload.

Closing this as the issue is in fact solved. The program is already doing what we expected it todo: just reading and exporting DataReadings.


PROV: Fix DSA and DH private key serializers

If those private key serializer were given a key structure with just
the public key material, they crashed, because they tried to
de-reference NULL.  This adds better checking.

This is extracted from #12512

This pull request is ready to merge

Merged
6cc1dfca88c565ddacd9ea9aa8261ef9c0c37335 PROV: Fix DSA and DH private key serializers


Feature #1: Cleanup Directory

As a contributor, I want to clean up unnecessary files from the repo. This will delete pycache, pyc and pyo files.
Justification

Reduces repository clutter: Removing __pycache__, .pyc, and .pyo files keeps the codebase cleaner and easier to navigate.
Prevents merge conflicts: These generated files can cause unnecessary merge conflicts if tracked in the repository.
Improves repo performance: Excluding these files reduces the overall size of the repository, speeding up clone and pull operations.
Enforces best practices: Only source code and essential files are tracked in version control, while auto-generated files are ignored.
Ensures consistency: These files can differ across environments, so excluding them ensures that everyone works with the same version of the code.
Promotes easier collaboration: Clean, well-organized repositories are easier for new contributors to understand and work with.

Impact
No impact on existing functionality.

alright


Burning through millions of tokens

Describe the bug
[plugin:vite:import-analysis] Failed to resolve import "./components/auth/AuthCard" from "src/App.tsx". Does the file exist?
/home/project/src/App.tsx:7:25
21 |  import { LoginModal } from "./components/auth/LoginModal";
22 |  import { auth, logout } from "./lib/firebase";
23 |  import { AuthCard } from "./components/auth/AuthCard";
|                            ^
24 |  import { Button } from "./components/ui/Button";
25 |  import Navbar from "./components/Navbar";
Link to the Bolt URL that caused the error
https://bolt.new/~/vitejs-vite-ai3nef
Steps to reproduce
testf.txt
test5.txt
test1.txt
I believe these are the scripts?
Expected behavior
For it to resolve the problem and I can keep working on my project
Screen Recording / Screenshot

Platform
Browser name = Chrome
Full version = <IP_ADDRESS>
Major version = 130
navigator.appName = Netscape
navigator.userAgent = Mozilla/5.0 (Windows NT 10.0; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/<IP_ADDRESS> Safari/537.36
performance.memory = {
  "totalJSHeapSize": 112432011,
  "usedJSHeapSize": 92101851,
  "jsHeapSizeLimit":<PHONE_NUMBER>
}
Username = Gigglebox
Chat ID = f906b0ffe338

Additional context
I don't code this tool is the only thing I have to make my dreams come true, I've spent prob 5-6million tokens trying to fix the issue, for it to redo the code but every character is 1 token and its repeating not fixing the code over and over.
The option to download scripts from bolt.new instead of stackblitz would be helpful, as bolt.new keeps crashing. Right now, I have to manually save each script before moving on to the next update.

@Endermittens issue #678 will point you in the right direction!


SPOWeb.SiteUsers not initialized

Reporting an Issue or Missing Feature
Whenever I try to get $web.SiteUsers I get an error.
Expected behavior
Get the list of users in the SPSite / SPWeb
Actual behavior
The collection has not been initialized. It has not been requested or the request has not been executed. It may need to be explicitly requested.
Steps to reproduce behavior
Connect-SPOnline -url [MYWEBURL]
$web = Get-SPOWeb
$su = web.SiteUsers
I also tried using :
Get-SPOProperty -ClientObject $web -Property SiteUsers
I might be doing something wrong but my understanding is that the Get-SPOProperty would load the required property if needed?
Which version of the PnP-PowerShell Cmdlets are you using?

[ ] PnP PowerShell for SharePoint 2013

What is the version of the Cmdlet module you are running?
2.8.1610.0
How did you install the PnP-PowerShell Cmdlets?

[ ] MSI Installed downloaded from GitHub

This is just an annoying behavior of CSOM in PowerShell. PowerShell tries to be helpful and access all properties of the SiteUsers object to display it as output, some of which aren't initialized and throw an exception when accessed. For things like this where you're accessing a property that is a collection/enumerable it's best to assign the result of Get-SPOProperty to a variable, and then use Select-Object or foreach to enumerate the collection members and access only the specific member properties you need.
If you do the following I suspect you'll see that SiteUsers is populated and you can access the data you need. Depending on which user properties you need you may have to call Get-SPOProperty again with the User object as the ClientObject
Connect-SPOnline -Url blah
$web = Get-SPOWeb
$users = Get-SPOProperty -ClientObject $web -Property SiteUsers

$users | Select Id, LoginName, Email

Thanks Chuck!!! It seems obvious now that I know about this super helpful feature of PowerShell.


MXNet results need to be explained

Add explanation of MXNet results at https://github.com/aws-samples/machine-learning-using-k8s/blob/master/mxnet.md

Results are explained at https://github.com/aws-samples/machine-learning-using-k8s/blob/master/docs/mnist/training/mxnet.md


Allow to open/close nested keys

When exploring a record all the content is always expanded, this is not ideal for large records:

curl -H 'Content-Type: application/json' -d '{"hello":"world!", "a": {"b":{"c": [1,2,3]}}}' http://localhost:5489/flb/d0851fc54f867a1286a3b6723627e83a2ff1ca4dd93d8e9c6795e1e3c1308ed9

Please add a way to open/close nested components

partially supported, closing for now.


Go to definition not working

Issue Type: Bug
repo

Try running go to definition in a js/ts file

Bug
Nothing happens. The command may also not show up in the command palette
VS Code version: Code - Insiders 1.30.0-insider (24b0f75b1e33b88f30f2ed9d5b17ca60062c270b, 2018-11-16T06:22:01.663Z)
OS version: Darwin x64 18.2.0

System Info

Item
Value

CPUs
Intel(R) Core(TM) i7-4770HQ CPU @ 2.20GHz (8 x 2200)

GPU Status
2d_canvas: enabledchecker_imaging: disabled_offflash_3d: enabledflash_stage3d: enabledflash_stage3d_baseline: enabledgpu_compositing: enabledmultiple_raster_threads: enabled_onnative_gpu_memory_buffers: enabledrasterization: enabledvideo_decode: enabledvideo_encode: enabledwebgl: enabledwebgl2: enabled

Load (avg)
2, 3, 3

Memory (System)
16.00GB (0.27GB free)

Process Argv
-psn_0_12327873

Screen Reader
no

VM
0%

@jrieken Reopening to track changing the fix in ebe9ea85a0a0bb087dccc98949542605dfc51f6f to use a better id

Note that my comment on the commit is has a copy paste error; both commands were registered with the id editor.action.goToDeclaration. I introduced editor.action.goToRealDeclaration as a temporary workaround

Thanks @mjbvz. It's tricky because the go-to-definition command uses a bogus id to begin with...

This is happening on latest non-insiders version, too.
Version 1.29.1 (1.29.1), MacOs


Fixing ssl deps that are required for libraries

Signed-off-by: Aaron Chong<EMAIL_ADDRESS>
Bug fix
Missing dependencies when exporting rmf_visualization_schedule library, gist here.

Thanks for the fix. Does OpenSSL needs to be added to package.xml as well?

It has already been added, in the past, https://github.com/open-rmf/rmf_visualization/blob/main/rmf_visualization_schedule/package.xml#L27


Error on macOS:    chown: root: illegal group name

macOS:
➜  make config SSHDATA=~/.ssh NAME=rsa
cp -rf ~/.ssh /Users/able/Downloads/agent-3/imageAPI/sshdata.run
sudo chown -R root:root /Users/able/Downloads/agent-3/imageAPI/sshdata.run
Password:
chown: root: illegal group name
make: *** [config] Error 1
add a comment in Makefile #sudo chown -R root:root $(SDRUNDIR), then it works.

Would you like to make PR (pull request), that would be great, and it is going to be really helpful to build the community process for us.

OK.


test

Please add a meaningful description for your change here

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

[ ] 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 or the workflows README to see a list of phrases to trigger workflows.

Run Load Tests Java 11 ParDo Dataflow V2 Batch

Run Load Tests Java 17 ParDo Dataflow V2 Batch

Run Load Tests Java 11 ParDo Dataflow V2 Streaming

Run Load Tests Java 17 ParDo Dataflow V2 Streaming


[BUG] Collective group's rank is incorrect

Background
Alpa initializes collective groups for each cross-mesh communication pair. The call stack to initialize a collective group is:
create_collective_group or init_collective_group from collective.py calls:
create_collective_group of GroupManager class in collective.py calls:
NCCLGroup.__init__ with two different implementations. One is based on cupy, while the other is based on xla.
A NCCLGroup creates and manages nccl communicators for each GPU in this node. When we need to call a nccl function, we finally goes into the NCCLGroup to call it. However, in our current implementation, we use node_rank * num_devices_per_node + local_offset to compute the rank of a local GPU w.r.t. the communication group. An example is here. This is correct in most cases, but when the send mesh has a different number of devices per node with the receive mesh, it is incorrect.
TODO

[ ] Fix the bug above by adding a start_gpu_rank at the initialization of NCCLGroup.
[ ] Add tests for collective communications among meshes. For unit test on cross-mesh communication, please refer to this file.

Goodday,
I am currently working on this bug

@AhmedMAlbreiki Please submit a PR so we can help review, thanks!

Hello, Im helping out with this issue, i have some questions for this
Currently the rank is computed in _get_nccl_collective_communicator where in it sets is like so actual_rank = self.rank * len(device_list) + i. Is this the issue in question where it needs to be replaced by  start_gpu_rank = something magical?
Still trying to fully understand the issue, thanks


Question: ad-hoc properties/fields

How could you ad 'ad-hoc' properties to a 'tcomb' struct?
My use case is that I have some props that I'd like to be fixed type and defined up front but want to allow additional untyped/checked props at some later point.
e.g.:
var Person = t.struct({
  name: t.String,              // required string
  surname: t.maybe(t.String)  // optional string
}, 'Person');

// create  Person
var person = new Person({
  name: 'Giulio',
  surname: 'Canti'
});

// then some time later (age could be anything)
person.age = 23; // tcomb to ignore this extra prop

Is this achievable in any way?
Alternatively, dynamically augment the the original struct.
I realize this is bumps up against the primary objective of tcomb, so please don't flame me ;)

Note. You can do this, instances are immutables:
person.age = 23;

My use case is that I have some props that I'd like to be fixed type and defined up front but want to allow additional untyped/checked props at some later point.

I wouldn't do that! Having said that, let's see...
Statically defined
var Person = t.struct({
  name: t.String,
  surname: t.maybe(t.String),
  age: t.Any // basically skip type checking for this field
}, 'Person');

var person = new Person({
  name: 'Giulio',
  surname: 'Canti'
}, 'Person');

console.log(person); // => Struct {name: "Giulio", surname: "Canti", age: undefined}

console.log(Person.update(person, {age: {$set: 41}})); // => Struct {name: "Giulio", surname: "Canti", age: 41}

Dinamically defined (not recommended)
var Person = t.struct({
  name: t.String,
  surname: t.maybe(t.String)
}, 'Person');

var person = new Person({
  name: 'Giulio',
  surname: 'Canti'
}, 'Person'); // <= it's important to give the type a name, especially if you change the props later

console.log(person); // => Struct {name: "Giulio", surname: "Canti", age: undefined}

Person.meta.props.age = t.Number;

console.log(Person.update(person, {age: {$set: 41}})); // => Struct {name: "Giulio", surname: "Canti", age: 41}

Using a subtype
var Person = t.subtype(t.Object, function (x) {
  return t.String.is(x.name) && t.maybe(t.String).is(x.surname);
}, 'Person');

// beware: here person is NOT an instanceof Person
var person = Person({
  name: 'Giulio',
  surname: 'Canti'
});

console.log(person); // => Struct {name: "Giulio", surname: "Canti", age: undefined}

console.log(Person.update(person, {
  age: {$set: 41},
  afield: {$set: 'a value'},
  anotherfield: {$set: 'another value'}
  // ... you are free to add whatever field you want
})); // => Struct {name: "Giulio", surname: "Canti", age: 41}

I realize this is bumps up against the primary objective of tcomb, so please don't flame me ;)

;)

Wow, thank you, that clear it up for me - awesome lib!


If prop's validateEmail is async function will cause unexpected behavior

Error Code
https://github.com/axisj/react-multi-email/blob/faf323da2f5af47ccb2e8c8b8478978d9180b0dd/react-multi-email/ReactMultiEmail.tsx#L77
unexpected behavior

I will try to check this issue, can you tell me the validateEmail function you used?

It has been fixed in 1.0.23.


Handles countries already correct

Overview
Checks if the given value is already correct and returns it in conversions.

LGTM

👍


Example: symfony app

What kind of example should this be? The full Symfony demo or some more simple (custom) website using Symfony?
For the full demo website see: https://github.com/ties-v/symfony-demo-on-now and  https://symfony-demo.ties.now.sh/
For a little less advanced example I would be able to put something together.

I think we should have 2 examples

tiny minimal example
full standard example

You can start with the minimal. What do you think?

Having issues to build a symfony app. I don't know what to do with cache on a read only file system. And after that i just get weird errors. So i am tagging along on this thread!

Hey @disjfa.
Do you have problem with this?
https://github.com/juicyfx/now-examples/tree/master/php-framework-symfony-microservice
https://now-examples-php-framework-symfony-microservice.now.sh/

I used the now files from there yes. But i started with a base symfony 5.0 checkout. So i will update when i have tested this one. If i found what and where we can update and upgrade.

Cool, the trick is to fix the cache folder in your local kernel. For reference for people who read this:
public function getCacheDir()
{
    if(isset($_ENV['NOW_REGION'])){
        return '/tmp/symfony/cache/'.$this->environment;
    }

    return parent::getCacheDir();
}

public function getLogDir()
{
    if(isset($_ENV['NOW_REGION'])){
        return '/tmp/symfony/log';
    }

    return parent::getLogDir(); // TODO: Change the autogenerated stub
}

Now the next step is to get the static css/js data from the public/build/ folder on the screen. Can't seem to get that working. Also checked a version 2 of the symfony demo on zeit but still no luck. Tried to fix the routes in the now.json to add something like this.
{
    "src": "/build/app.css",
    "dest": "/public/build/app.css"
},

But no luck, if you have more tips, please. Also when i have something practical i surely will share and blg about it.

@disjfa An example for symfony 4.* is available at ties-v/symfony-demo-on-now.
The now.json file might be interesting to you.
One thing to notice is that is not using the new keys like functions  and rewrites etc. Maybe I'll update it some time.
Hope it helps!

Yes, thats the v2 version i was talking about. I want to add some npm stuff (encore) also so that breaks there for now.

Do you have an example repo somwhere? And what is exactly failing: loading of encore assets?

Pushed my repo, just working on an example to use to prototype. https://github.com/disjfa/symfony-prototype
For now i disabled the encore files and encore build step.

There are 2 examples of symfony on vercel, take a look https://github.com/juicyfx/vercel-examples.


Update x/crypto to v0.17.0

What type of PR is this?
/kind bug
What this PR does / why we need it:
Update x/crypto to a newer version to try to resolve ssh issues.
Special notes for your reviewer:
This is essentially a backport of https://github.com/kubernetes/perf-tests/pull/2486 because of https://github.com/kubernetes/kubernetes/pull/124486.

/assign @marseel

/lgtm
/approve

/retest

/retest


In LIKE query does not add the % character

if "LIKE" string shorter than 3 chars, does not add the % character
Problem is in regular expression /^[^%].+[^%]$/ on line 320 in file medoo.php (v0.9.7)
"sfd".match(/^[^%].+[^%]$/)
return ["sfd"]
"fd".match(/^[^%].+[^%]$/)
return  null

Fixed it.

thx :)


Fix service update replicas

cc @vieux  @ehazlett

LGTM

LGTM

lgtm -- thx

I noticed a few more flags were broken so I made constants for a bunch of them and fixes some others.

Re-LGTM :+1:

LGTM

LGTM

All tests are passed, only userns crapped itself.
Merging, thanks!


(#494) Use organizational level templates

Description
This PR removes all of the repository templates which is required to use the organizational level templates.
Related Issue
Fixes #494
Motivation and Context
The repositories in the Chocolatey organization should use the repository level templates where possible.
How Has This Been Tested?
N/A
Screenshots (if appropriate):
N/A
Types of changes

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

Checklist:

[ ] My code follows the code style of this project.
[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.
[x] I have read the CONTRIBUTING document.
[ ] I have added tests to cover my changes.
[ ] All new and existing tests passed.

@pauby thanks for getting this updated!


Enable predict function to generate only one prediction when uncertainty is not needed, for speed up purpose.

The motivation is to speed up prediction when running on a list of benchmark datasets.
The predict() function in Forecaster() class used to always predict 100 samples and return mean, quantiles and samples. To speed up, we only predict one sample when the uncertainty argument in Forecaster() is False, when uncertainty is not needed by the user.

Changed to inline ifs;
Changed to always return the same set of variables. When uncertainty is not required, it returns None for the quantile results.


MAX32690 : Unable to generate SPI Clock > 15 MHz

Using the MAX32690 EVK, an attempt was made to carry out an SPI transaction for a given SPI_SPEED. It was observed that this breaks for a frequency greater than 15 MHz. It is not an issue due to the loopback configuration in the example (can be replicated when not shorting MOSI-MISO). SPI bus is connected with 0 Ohm links to a codec on the EVK. These were removed to verify that this was not causing the issue. Clock is being correctly set by the BSP to the IPO and this was verified by looking at the GCR register. No error codes are returned at any point.
See Example used to test:
`/***** Includes *****/
#include <stdio.h>
#include <stdbool.h>
#include <stdint.h>
#include <string.h>
#include "board.h"
#include "mxc_device.h"
#include "mxc_delay.h"
#include "mxc_pins.h"
#include "nvic_table.h"
#include "uart.h"
#include "spi.h"
#include "dma.h"
#include "led.h"
#include "mxc_sys.h"
/***** Definitions *****/
#define DATA_LEN 1 // Words
// Testing
//#define SPI_SPEED 15000000      //  Works okay
#define SPI_SPEED 20000000 	//  Not seeing clock for SPI_SPEED > 15 MHz
//#define SPI_SPEED 30000000    //
#define DATA_VALUE_8BIT 0xA5         // 8-bit
#define DATA_VALUE_16BIT 0xA5A5      // 16-bits
#define DATA_VALUE_32BIT 0xA5A5A5A5  // 32-bits
#define SPI MXC_SPI0
#define SPI_IRQ SPI0_IRQn
#define MOSI_PIN 28
#define MISO_PIN 27
/***** Globals *****/
volatile int SPI_FLAG;
volatile uint8_t DMA_FLAG = 0;
// Data Buffers
// 8-bit
uint8_t rx_data_8_bit[DATA_LEN];
uint8_t tx_data_8_bit[DATA_LEN];
// 16-bit
uint16_t rx_data_16_bit[DATA_LEN];
uint16_t tx_data_16_bit[DATA_LEN];
// 32-bit
uint32_t rx_data_32_bit[DATA_LEN];
uint32_t tx_data_32_bit[DATA_LEN];
int main(void)
{
volatile int retVal;

printf("\n**************************** SPI MASTER TEST *************************\n");

// Ensure the System Clock uses the 120 MHz internal primary oscillator (IPO)
// Already set in SystemInit()

// Structures for defining pins and parameters
mxc_spi_req_t req;
mxc_spi_pins_t spi_pins;

// Number of clock pulses - use appropriate size for # of bits being sent
int data_size = 8;
//int data_size = 16;
//int data_size = 32;

// Define Chip Select Pins
spi_pins.ss0 = FALSE;
spi_pins.ss1 = TRUE;
spi_pins.ss2 = FALSE;

// Setting the data of the transmit data buffer - only 1 element
// Do this in a loop for larger arrays
tx_data_8_bit[0] = DATA_VALUE_8BIT;
//tx_data_16_bit[0] = DATA_VALUE_16BIT;
//tx_data_32_bit[0] = DATA_VALUE_32BIT;

printf("\nInitializing SPI\n");

// Configure the peripheral
retVal = MXC_SPI_Init(SPI, 1, 0, 1, 0, SPI_SPEED, spi_pins);

if (retVal != E_NO_ERROR) {
    printf("\nSPI INITIALIZATION ERROR\n");
    return retVal;
}

printf("\nSetting SPI Mode\n");

// Set SPI mode
MXC_SPI_SetMode(SPI, SPI_MODE_0);

//SPI Request
req.spi = SPI;

req.txData = (uint8_t *)tx_data_8_bit;
req.rxData = (uint8_t *)rx_data_8_bit;

//req.txData = (uint8_t *)tx_data_16_bit;
//req.rxData = (uint8_t *)rx_data_16_bit;

//req.txData = (uint8_t *)tx_data_32_bit;
//req.rxData = (uint8_t *)rx_data_32_bit;

req.txLen = DATA_LEN;
req.rxLen = DATA_LEN;
req.ssIdx = 1;
req.ssDeassert = 1;
req.txCnt = 0;
req.rxCnt = 0;
//req.completeCB = (spi_complete_cb_t)SPI_Callback;
SPI_FLAG = 1;

printf("\nSetting Data Size\n");

// Setting the clock pulses
MXC_SPI_SetDataSize(SPI, data_size);

printf("\nSetting SPI Width\n");

// Setting full/half-duplex
MXC_SPI_SetWidth(SPI, SPI_WIDTH_STANDARD);
//MXC_SPI_SetWidth(SPI, SPI_WIDTH_3WIRE);

printf("\nCarry out a Transaction\n");

// Carry out a SPI transaction
MXC_SPI_MasterTransaction(&req);

//while(1){};

}
`

Hi @ConWalsh, also try increasing the drive strength for the GPIO pins to maximum.  This will help get sharp edges at higher clock speeds

After looking into it further I made an error on my part. I thought I was using VDDIOH (3V3) for the SPI pins but I was using VDDIO (1V8).
Including the line:
MXC_GPIO_SetVSSEL(MXC_GPIO1, MXC_GPIO_VSSEL_VDDIOH, (MXC_GPIO_PIN_23 | MXC_GPIO_PIN_26 | MXC_GPIO_PIN_28 | MXC_GPIO_PIN_29));
solved my issue.
@sanjayjaroli thank you for that. This will be important for me when I receive my boards as I will use SPI 3 & 4 on the AHB bus.
@Jake-Carter this will be helpful as well. Thanks for the suggestion.

No problem, glad you got it working!


Add StockAlarmClockDisabler

https://forum.kerbalspaceprogram.com/topic/219015-112x-stock-alarm-clock-disabler/
https://github.com/UltraJohn/StockAlarmClockDisabler

Fixes #9773.

@HebaruSan It is intentional that this mod both recommends and supports KAC? It seems a bit strange to have the name pop up twice.

Yes. The supports relationship means that the user will be presented with the option to install this mod when installing KerbalAlarmClock.


Not cross-posting to Medium

I have update the _config.yml with
jekyll-crosspost_to_medium:
  enabled: true
  cache: .jekyll-crosspost_to_medium
  status: public
  license: all-rights-reserved

and also included in post md file.
layout: post
title:  "Automated crosspost to Medium"
date:   2016-01-31 01:31:25
categories: medium blog
tags: featured
image: /assets/article_images/desktop.jpg
crosspost_to_medium: true

I just pushed an update. Please grab the latest version of the Gem and give it another shot. If it fails, we should get back a better message from Medium. When you get that, could you post it here as plain text?

this is the error message from Medium.

Weird.
Would you be up for trying to debug this directly? To do so, comment out the gem in your _config.yml and then grab jekyll-crosspost-to-medium.rb from the lib folder in this repo and drop it into your plugins folder. Then, on line 142, type puts payload.inspect. You can also comment out the next two lines if you don’t want to submit to Medium while we’re debugging.

yes sure, would do it.

here is the output.
Configuration file: /home/daxlab/repos/blog/_config.yml
            Source: /home/daxlab/repos/blog
       Destination: /home/daxlab/repos/blog/_site
      Generating...
{"title"=>"Automated crosspost to Medium", "contentFormat"=>"html", "content"=>"
Automated crosspost to Medium

This is a test and is automated.
\n\n
Liquid error: undefined method `safe' for nil:NilClass
\n\n
Check out the Jekyll docs for more info on how to get the most out of Jekyll. File all bugs/feature requests at Jekyll’s GitHub repo. If you have questions, you can ask them on Jekyll’s dedicated Help repository.
\n\n
Liquid error: undefined method `safe' for nil:NilClass
\n\n
This article was originally posted on my own site.
", "tags"=>"featured", "publishStatus"=>"public", "license"=>"all-rights-reserved", "canonicalUrl"=>"https://blog.mrmandeep.me/medium/blog/2016/01/31/crosspost-to-medium.html"}
                    done.
 Auto-regeneration: enabled for '/home/daxlab/repos/blog'
Configuration file: /home/daxlab/repos/blog/_config.yml
    Server address: http://<IP_ADDRESS>:4000/
  Server running... press ctrl-c to stop.

I wonder if it’s an issue with your tags. Try switching it to an array in your front matter:
layout: post
title:  "Automated crosspost to Medium"
date:   2016-01-31 01:31:25
categories: medium blog
tags: [featured]
image: /assets/article_images/desktop.jpg
crosspost_to_medium: true

ok, let me try.

According to the docs, it shouldn’t matter, but who knows

            Source: /home/daxlab/repos/blog
       Destination: /home/daxlab/repos/blog/_site
      Generating...
{"title"=>"Automated crosspost to Medium", "contentFormat"=>"html", "content"=>"
Automated crosspost to Medium

This is a test and is automated.
\n\n
Liquid error: undefined method `safe' for nil:NilClass
\n\n
Check out the Jekyll docs for more info on how to get the most out of Jekyll. File all bugs/feature requests at Jekyll’s GitHub repo. If you have questions, you can ask them on Jekyll’s dedicated Help repository.
\n\n
Liquid error: undefined method `safe' for nil:NilClass
\n\n
This article was originally posted on my own site.
", "tags"=>["featured"], "publishStatus"=>"public", "license"=>"all-rights-reserved", "canonicalUrl"=>"https://blog.mrmandeep.me/medium/blog/2016/01/31/crosspost-to-medium.html"}
                    done.
 Auto-regeneration: enabled for '/home/daxlab/repos/blog'
Configuration file: /home/daxlab/repos/blog/_config.yml
    Server address: http://<IP_ADDRESS>:4000/
  Server running... press ctrl-c to stop.

Did it publish?

yes my dear, thanks

Hooray! Jekyll should expose the tags as a list, but for some reason that wasn’t happening. Fingers crossed we’ve resolved the issue. Thanks for the report!

although there is a problem. It is giving Liquid error: undefined method `safe’ for nil:NilClass
which is shown in the post. How to remove it from post ?


Composer Error on Laravel 5.3

I still get the following error:

Problem 1
- Conclusion: don't install voku/simple_html_dom 2.0.23
- Conclusion: don't install voku/simple_html_dom 2.0.22
- Conclusion: don't install voku/simple_html_dom 2.0.21
- Conclusion: don't install voku/simple_html_dom 2.0.20
- Conclusion: don't install voku/simple_html_dom 2.0.19
- Conclusion: don't install voku/simple_html_dom 2.0.18
- Conclusion: don't install voku/simple_html_dom 2.0.17
- Conclusion: don't install voku/simple_html_dom 2.0.16
- Conclusion: don't install voku/simple_html_dom 2.0.15
- Conclusion: don't install voku/simple_html_dom 2.0.14
- Conclusion: don't install voku/simple_html_dom 2.0.13
- Conclusion: don't install voku/simple_html_dom 2.0.12
- Conclusion: don't install voku/simple_html_dom 2.0.11
- Conclusion: don't install voku/simple_html_dom 2.0.10
- Conclusion: don't install voku/simple_html_dom 2.0.9
- Conclusion: don't install voku/simple_html_dom 2.0.8
- Conclusion: don't install voku/simple_html_dom 2.0.7
- Conclusion: don't install voku/simple_html_dom 2.0.6
- Conclusion: don't install voku/simple_html_dom 2.0.5
- Conclusion: don't install voku/simple_html_dom 2.0.4
- Conclusion: remove symfony/css-selector v3.0.9
- Installation request for voku/simple_html_dom ^2.0 -> satisfiable by voku/simple_html_dom[2.0.10, 2.0.11, 2.0.12, 2.0.13, 2.0.14, 2.0.15, 2.0.16, 2.0.17, 2.0.18, 2.0.19, 2.0.20, 2.0.21, 2.0.22, 2.0.23, 2.0.3, 2.0.4, 2.0.5, 2.0.6, 2.0.7, 2.0.8, 2.0.9].
- Conclusion: don't install symfony/css-selector v3.0.9
- voku/simple_html_dom 2.0.3 requires symfony/css-selector ~2.8.0 -> satisfiable by symfony/css-selector[v2.8.0, v2.8.1, v2.8.10, v2.8.11, v2.8.2, v2.8.3, v2.8.4, v2.8.5, v2.8.6, v2.8.7, v2.8.8, v2.8.9].
- Can only install one of: symfony/css-selector[v2.8.0, v3.0.9].
- Can only install one of: symfony/css-selector[v2.8.1, v3.0.9].
- Can only install one of: symfony/css-selector[v2.8.10, v3.0.9].
- Can only install one of: symfony/css-selector[v2.8.11, v3.0.9].
- Can only install one of: symfony/css-selector[v2.8.2, v3.0.9].
- Can only install one of: symfony/css-selector[v2.8.3, v3.0.9].
- Can only install one of: symfony/css-selector[v2.8.4, v3.0.9].
- Can only install one of: symfony/css-selector[v2.8.5, v3.0.9].
- Can only install one of: symfony/css-selector[v2.8.6, v3.0.9].
- Can only install one of: symfony/css-selector[v2.8.7, v3.0.9].
- Can only install one of: symfony/css-selector[v2.8.8, v3.0.9].
- Can only install one of: symfony/css-selector[v2.8.9, v3.0.9].
- Installation request for symfony/css-selector (locked at v3.0.9, required as 2.8.|3.0.) -> satisfiable by symfony/css-selector[v3.0.9].

Installation failed, reverting ./composer.json to its original content.

    "require": {
        "php": ">=5.6.4",
        "laravel/framework": "5.3.*",
        "voku/simple_html_dom": "2.0.*"
    },

Can you test it again, please?

cool. no errors now


Is there any way to customise the buttons present

I would like to remove the mass edit button, and also add the "Add to campaign" button. Any way to do this?

I'm sorry to say that currently there is no way to customize the appearance of the button bar. But feel free to fork, extend and pull-request such a change. I believe that this would be of high value.


Does PodGroup support multi jobs?

step 1. I create two jobs which have same tags scheduling.k8s.io/group-name: qj-1 and parallelism: 1;
step 2. I hope all two jobs can not run when resources are not sufficient, but one job is running, other is pending.
Can PodGroup manage multi jobs have same tags scheduling.k8s.io/group-name: qj-1?

BTW, job can not set parallelism: 2, because jobs are not same, such as different envs task_index

I change podgroup.spec.numMember to minMember, it work.  I think podgroup` dont aware of jobs, it aware of pod number, so it can support multi jobs which have same group tag


Require space for class keyword match

Motivation
Closes #2269
In order for class to be a keyword, there needs to be a space between it and the constant name.
Implementation
Switched to \s+ rather than \s*.

(although less likely to be encountered)

@andyw8 good point, we should fix that too https://github.com/Shopify/ruby-lsp/pull/2271.


Test-drive the CLASS/Api()/helper() convention!

Now that MACROS are a thing of the past (mostly), and (therefore) nobody really uses ALL_UPPER constants any more, that frees the way for ALL_UPPERCASE classnames (a'la Eiffel, for example).

Consts/enums can actually still be ALL_UPPER, esp. as enum classes are "classes anyway". :)
Freestanding functions are snake_case().
Data members are _prefixed. Except for interface properties, which are just like normal variable names (typically lower-cased, but no strict rules here, the nature of the property can also have a say now).
Unicode names could help even more.

class TEXTBOX : public WIDGET
{
public:
    TEXTBOX(std::string initial_text, int maxlen);
    operator=(cont TEXTBOX& other) = delete;

    void SetText(const std::string& text);
    std::string GetText();

protected:
// Helpers...
    void move_cursor(int delta);
    void move_selection_head(int delta);
    void move_selection_tail(int delta);

private:
// Callbacks...
    void onSomeEvent();
    void onOtherEvent();

// Data...
    std::string _text;
};

OK, so the weak point is still the variable/data member names, as before...
BUT: camelCase var. names  ARE NOT BANNED!
Their use use simply refined: their lowercased first tag, if followed by a Capped tag, simply means prefixing, as it akways (should) have!

Another weak point is: class filenames. they would really look kinda bad with all-caps.

It's your code, your tools, and you're free to use them to their maximum. In fact, as a good engineer, you're responsible to do so, if it can increase the overall quality of your output.


Use Git state as a store

This is ready for merge

[x] Implements Git as a state store

NOTE: Uses GitHub API for git state store.
Will look at the GitHub API once the git version is measured and tested. I even suggest that comes in a separate PR.

@Ch3ck here's the Implementation idea.
Before getting onto it, please:

Return the Docker script.
Rebase against latest changes, so you have the fresh branch. Tonight, we got massive changes in for cmd package and there was some minor changes as well.

If you have any questions, ping me here on on Slack.

Regarding calling git from go, I have had success with go-git
Example code: https://github.com/kris-nova/klone/blob/master/pkg/klone/kloners/gogit/git.go#L31
Repo: https://github.com/src-d/go-git
Regarding calling github from, I freaking love go-github
Example code: https://github.com/kris-nova/klone/blob/master/pkg/provider/github/github.go#L63
Repo: https://github.com/google/go-github

Thanks @kris-nova those links are very helpful

@xmudrii @kris-nova I went with the simple implementation with go-git however, I need some help performing the proper configurations for git.CommitOptions and git.PushOptions, I don't know how these options will be passed to these commands or stored in memory. More confusing is the PushOptions. However it could be configured in such a way that a default authentication is used for every push or configured each time for every push depending on the repo.
Take a look at this link, it'll be useful: transport.AuthMethod

Are we still blocked on this?

@kris-nova I just need some clarifications on how the git configs will be passed to the git state command

@xmudrii @kris-nova  Issues fixed. I tried adding the dependencies for go-git tool. I get this error with dep ensure: https://github.com/kris-nova/kubicorn/issues/498

@Ch3ck We need to get tests fixed first. You have some incorrect import paths, make sure you fix them.

@xmudrii Issues fixed, the PR can be merged now.

@Ch3ck Tests are still failing, we need to get that fixed.
I'll get back to you with a review as soon as I can.

@xmudrii I'll take another look at the tests today. I should get it fixed soon enough.

@xmudrii Tests fixed. Here are my results.
=== RUN   TestStateFileSystem
2017-12-01T21:14:16-07:00 [!]  Removing path [.test/fstest]
2017-12-01T21:14:16-07:00 [!]  Removing path [.test/fstest]
--- PASS: TestStateFileSystem (0.02s)
PASS
ok  	github.com/kris-nova/kubicorn/state/fs	0.276s
=== RUN   TestJsonGit
2017-12-01T21:14:19-07:00 [!]  Removing path [.test/git-test]
2017-12-01T21:14:19-07:00 [!]  Removing path [.test/git-test]
--- PASS: TestJsonGit (0.01s)
PASS
ok  	github.com/kris-nova/kubicorn/state/git	0.279s
=== RUN   TestJsonFileSystem
2017-12-01T21:14:19-07:00 [!]  Removing path [.test/jsonfs-test]
2017-12-01T21:14:19-07:00 [!]  Removing path [.test/jsonfs-test]
--- PASS: TestJsonFileSystem (0.01s)
PASS
ok  	github.com/kris-nova/kubicorn/state/jsonfs	0.198s
?   	github.com/kris-nova/kubicorn/test	[no test files]
make: *** [Makefile:98: test] Error 1


Use rails convention to underscore collection class names

Bug Fix
Description
This would mean an object such as ActionItem when evaluated in a partial would show up as actionitem by default, instead of the expected action_item. Users of the library could override this with the as: option, such as as: :action_item in this case.
Prior to the change, this code would be need:
<!-- index.html.erb --!>
<%= futurize(collection: ActionItem.all, extends: :div)  { } %>

<!-- _action_item.html.erb --!>
<%= actionitem.description %>

Whereas, we would expect the following to work
<!-- index.html.erb --!>
<%= futurize(collection: ActionItem.all, extends: :div)  { } %>

<!-- _action_item.html.erb --!>
<%= action_item.description %>

Why should this be added
This follows Rails conventions and the principle of least surprise. However, this is a breaking change for users of this library, but is correct in following Rails conventions.
Checklist

[x] My code follows the style guidelines of this project
[x] Checks (StandardRB & Prettier-Standard) are passing

Sorry to bombard you @julianrubisch. Real-world use of this library is highlighting a few items for me.

Good catch, keep it coming!
Don't see an issue since we're targeting 1.0 anyway


High severity vulnerability affecting this repo

Hi there,
We noticed that your repo has a high severity vulnerability:

Regular Expression Denial of Service
Affected package: negotiator <= 0.6.0

Here is the test report for this repo.
If you’d like to fix this vulnerability, Snyk lets you generate a pull request that recommends the best upgrade path - there’s a link to fix this vulnerability on the test report.
Stay secure :-)
Snyk Community

Thanks for letting know, I updated the dependencies 😎👍

Awesome! :)


Update error messages for DNSimple provider

Hi again! Since last PR (#363), we recognised that error messages needed to be tweaked for the sake of usability.
Would you mind to merge this PR with these small adjustments? Thank you.

@xenolf Thank you!


Move to validation-api 1.1

Any major blockers to moving validation extension to validation-api 1.1?

apparently not.  :)


Demo for functional roberta

Update the Roberta demo to be compatible with the ttnn_functional_roberta model.

Please use optimized bert implementation as underlying model models/demos/bert/tt/ttnn_optimized_bert.py

commit for demo roberta token classification task - commit
dataset - conll2003

The demo for functional roberta QA is updated.
Corresponding PR: #6601


Vehicle factory not loading up

so i created my vehicle and rigged the bones in blender, did the physics assessment as in documentation, now in the last step of documentation its unclear its written to open the vehicle factory and add the vehicle but in my case the file is not loading up. Please help team Carla and community.
Thankyou
-->

Hi,
I know, that our documentation about how to implement vehicles in CARLA is a bit confusing. Now we are working to update.
This file is in UnrealEngine. Is a Blueprint that you can open and then put your vehicle there.

no, thats

not the problem i am not able to open the ue4 Editor to open the VehicleFactory asset
and i am not able to open it from the engine itself, it ask me to download carsim which is not free and not make sense because its not mentioned in the documentation, idk the reason for all of these

I see,
From what I see in the picture.
Estas intentado abrir UE4, en una versión nuestra compilada. If you need to add vehicles you need to download whole project and UE4.

Here you have the documentation on how to download the project and UE4 with win and Linux

https://carla.readthedocs.io/en/latest/#getting-started


Potential regression of #108298 after Flutter 3.7.0 upgrade

After upgrading to Flutter 3.7.0, applications using macos_ui are seeing several The Scrollbar's ScrollController has no ScrollPosition attached. errors, without any indication of which Scrollbars and/or Scrollables are the source of the trace in question.
CC @Piinks @moffatman
Steps to Reproduce

Optional: Perform this process on macOS for maximum reproduction reliability (applications written with macos_ui can run on Windows & Linux but it is not recommended)

Ensure you are on the latest stable branch
Create a new Flutter app (ensuring that the macOS runner is present)
Run flutter pub add macos_ui
Replace all code in main.dart with the code found in the "Code sample" section below
Targeting macOS, execute flutter run on the aforementioned code sample
Upon the launch of the app, without attempting to scroll, observe three errors in the console that report The Scrollbar's ScrollController has no ScrollPosition attached.. Note that none of the stack traces report which Scrollbar and/or Scrollable widget from macos_ui is the source of the particular trace.

Expected results:
Prior to #108298 arriving with Flutter 3.7.0, none of the errors being reported here were present. I suppose I expected things to remain that way.
Actual results:
No visual errors, just stack traces in the console.

Code sample

import 'package:flutter/cupertino.dart';
import 'package:flutter/material.dart';
import 'package:macos_ui/macos_ui.dart';

void main() {
  runApp(const App());
}

class App extends StatelessWidget {
  const App({super.key});

  // This widget is the root of your application.
  @override
  Widget build(BuildContext context) {
    return MacosApp(
      title: 'scrollable_bug',
      theme: MacosThemeData.light(),
      darkTheme: MacosThemeData.dark(),
      themeMode: ThemeMode.system,
      home: const MainView(),
      debugShowCheckedModeBanner: true,
    );
  }
}

class MainView extends StatefulWidget {
  const MainView({super.key});

  @override
  State<MainView> createState() => _MainViewState();
}

class _MainViewState extends State<MainView> {
  int _pageIndex = 0;

  @override
  Widget build(BuildContext context) {
    return MacosWindow(
      sidebar: Sidebar(
        minWidth: 200,
        builder: (context, scrollController) => SidebarItems(
          currentIndex: _pageIndex,
          onChanged: (index) {
            setState(() => _pageIndex = index);
          },
          items: const [
            SidebarItem(
              leading: MacosIcon(CupertinoIcons.home),
              label: Text('Home'),
            ),
          ],
        ),
      ),
      child: IndexedStack(
        index: _pageIndex,
        children: const [
          HomePage(),
        ],
      ),
    );
  }
}

class HomePage extends StatelessWidget {
  const HomePage({super.key});

  @override
  Widget build(BuildContext context) {
    return Builder(
      builder: (context) {
        return MacosScaffold(
          toolBar: ToolBar(
            title: const Text('Home'),
            actions: [
              ToolBarIconButton(
                label: 'Toggle Sidebar',
                icon: const MacosIcon(CupertinoIcons.sidebar_left),
                showLabel: false,
                tooltipMessage: 'Toggle Sidebar',
                onPressed: () {
                  MacosWindowScope.of(context).toggleSidebar();
                },
              )
            ],
          ),
          children: [
            ContentArea(
              builder: (context, scrollController) {
                return const Center(
                  child: Text('Home'),
                );
              },
            ),
          ],
        );
      },
    );
  }
}

Logs

Launching lib/main.dart on macOS in debug mode...
Building macOS application...
--- xcodebuild: WARNING: Using the first of multiple matching destinations:
{ platform:macOS, arch:arm64, id:00008103-000970CE0CE2001E }
{ platform:macOS, arch:x86_64, id:00008103-000970CE0CE2001E }
Debug service listening on ws://<IP_ADDRESS>:57150/EIvWoM3zb5s=/ws
Syncing files to device macOS...

======== Exception caught by scheduler library =====================================================
The following assertion was thrown during a scheduler callback:
The Scrollbar's ScrollController has no ScrollPosition attached.

A Scrollbar cannot be painted without a ScrollPosition.

The Scrollbar attempted to use the provided ScrollController. This ScrollController should be associated with the ScrollView that the Scrollbar is being applied to.When providing your own ScrollController, ensure both the Scrollbar and the Scrollable widget use the same one.

When the exception was thrown, this was the stack:
#0      RawScrollbarState._debugCheckHasValidScrollPosition.<anonymous closure> (package:flutter/src/widgets/scrollbar.dart:1597:9)
#1      RawScrollbarState._debugCheckHasValidScrollPosition (package:flutter/src/widgets/scrollbar.dart:1622:6)
#2      RawScrollbarState._debugScheduleCheckHasValidScrollPosition.<anonymous closure> (package:flutter/src/widgets/scrollbar.dart:1551:14)
#3      SchedulerBinding._invokeFrameCallback (package:flutter/src/scheduler/binding.dart:1289:15)
#4      SchedulerBinding.handleDrawFrame (package:flutter/src/scheduler/binding.dart:1227:9)
#5      SchedulerBinding.scheduleWarmUpFrame.<anonymous closure> (package:flutter/src/scheduler/binding.dart:942:7)
#9      _RawReceivePort._handleMessage (dart:isolate-patch/isolate_patch.dart:192:26)
(elided 3 frames from class _Timer and dart:async-patch)
====================================================================================================

======== Exception caught by animation library =====================================================
The following assertion was thrown while notifying status listeners for AnimationController:
The Scrollbar's ScrollController has no ScrollPosition attached.

A Scrollbar cannot be painted without a ScrollPosition.

The Scrollbar attempted to use the provided ScrollController. This ScrollController should be associated with the ScrollView that the Scrollbar is being applied to.When providing your own ScrollController, ensure both the Scrollbar and the Scrollable widget use the same one.

When the exception was thrown, this was the stack:
#0      RawScrollbarState._debugCheckHasValidScrollPosition.<anonymous closure> (package:flutter/src/widgets/scrollbar.dart:1597:9)
#1      RawScrollbarState._debugCheckHasValidScrollPosition (package:flutter/src/widgets/scrollbar.dart:1622:6)
#2      RawScrollbarState._validateInteractions (package:flutter/src/widgets/scrollbar.dart:1564:14)
#3      AnimationLocalStatusListenersMixin.notifyStatusListeners (package:flutter/src/animation/listener_helpers.dart:240:19)
#4      AnimationController._checkStatusChanged (package:flutter/src/animation/animation_controller.dart:822:7)
#5      AnimationController._startSimulation (package:flutter/src/animation/animation_controller.dart:756:5)
#6      AnimationController._animateToInternal (package:flutter/src/animation/animation_controller.dart:619:12)
#7      AnimationController.forward (package:flutter/src/animation/animation_controller.dart:467:12)
#8      RawScrollbarState._handleScrollMetricsNotification (package:flutter/src/widgets/scrollbar.dart:1938:37)
#9      _NotificationElement.onNotification (package:flutter/src/widgets/notification_listener.dart:130:38)
#10     _NotificationNode.dispatchNotification (package:flutter/src/widgets/framework.dart:3170:18)
#11     _NotificationNode.dispatchNotification (package:flutter/src/widgets/framework.dart:3173:13)
#12     _NotificationNode.dispatchNotification (package:flutter/src/widgets/framework.dart:3173:13)
#13     _NotificationNode.dispatchNotification (package:flutter/src/widgets/framework.dart:3173:13)
#14     _NotificationNode.dispatchNotification (package:flutter/src/widgets/framework.dart:3173:13)
#15     Element.dispatchNotification (package:flutter/src/widgets/framework.dart:4534:24)
#16     Notification.dispatch (package:flutter/src/widgets/notification_listener.dart:60:13)
#17     ScrollPosition.didUpdateScrollMetrics (package:flutter/src/widgets/scroll_position.dart:947:93)
(elided 2 frames from dart:async)
The AnimationController notifying status listeners was: AnimationController#9c725(▶ 0.000)
====================================================================================================

======== Exception caught by animation library =====================================================
The following assertion was thrown while notifying status listeners for AnimationController:
The Scrollbar's ScrollController has no ScrollPosition attached.

A Scrollbar cannot be painted without a ScrollPosition.

The Scrollbar attempted to use the provided ScrollController. This ScrollController should be associated with the ScrollView that the Scrollbar is being applied to.When providing your own ScrollController, ensure both the Scrollbar and the Scrollable widget use the same one.

When the exception was thrown, this was the stack:
#0      RawScrollbarState._debugCheckHasValidScrollPosition.<anonymous closure> (package:flutter/src/widgets/scrollbar.dart:1597:9)
#1      RawScrollbarState._debugCheckHasValidScrollPosition (package:flutter/src/widgets/scrollbar.dart:1622:6)
#2      RawScrollbarState._validateInteractions (package:flutter/src/widgets/scrollbar.dart:1564:14)
#3      AnimationLocalStatusListenersMixin.notifyStatusListeners (package:flutter/src/animation/listener_helpers.dart:240:19)
#4      AnimationController._checkStatusChanged (package:flutter/src/animation/animation_controller.dart:822:7)
#5      AnimationController._tick (package:flutter/src/animation/animation_controller.dart:838:5)
#6      Ticker._tick (package:flutter/src/scheduler/ticker.dart:249:12)
#7      SchedulerBinding._invokeFrameCallback (package:flutter/src/scheduler/binding.dart:1289:15)
#8      SchedulerBinding.handleBeginFrame.<anonymous closure> (package:flutter/src/scheduler/binding.dart:1140:11)
#9      _LinkedHashMapMixin.forEach (dart:collection-patch/compact_hash.dart:625:13)
#10     SchedulerBinding.handleBeginFrame (package:flutter/src/scheduler/binding.dart:1138:17)
#11     SchedulerBinding._handleBeginFrame (package:flutter/src/scheduler/binding.dart:1055:5)
#12     _invoke1 (dart:ui/hooks.dart:164:13)
#13     PlatformDispatcher._beginFrame (dart:ui/platform_dispatcher.dart:316:5)
#14     _beginFrame (dart:ui/hooks.dart:101:31)
The AnimationController notifying status listeners was: AnimationController#9c725(⏭ 1.000; paused)
====================================================================================================

reubenturner@Reubens-MacBook-Pro-M1 scrollable_bug % flutter analyze
Analyzing scrollable_bug...
No issues found! (ran in 1.8s)

[✓] Flutter (Channel stable, 3.7.0, on macOS 12.4 21F79 darwin-arm64, locale en-US)
    • Flutter version 3.7.0 on channel stable at /Users/reubenturner/flutter
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision b06b8b2710 (3 days ago), 2023-01-23 16:55:55 -0800
    • Engine revision b24591ed32
    • Dart version 2.19.0
    • DevTools version 2.20.1

[✓] Android toolchain - develop for Android devices (Android SDK version 31.0.0)
    • Android SDK at /Users/reubenturner/Library/Android/sdk
    • Platform android-31, build-tools 31.0.0
    • Java binary at: /Applications/Android Studio.app/Contents/jre/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 11.0.10+0-b96-7281165)
    • All Android licenses accepted.

[✓] Xcode - develop for iOS and macOS (Xcode 13.4.1)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Build 13F100
    • CocoaPods version 1.11.3

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

[✓] Android Studio (version 2020.3)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build 11.0.10+0-b96-7281165)

[✓] IntelliJ IDEA Community Edition (version 2022.3.1)
    • IntelliJ at /Applications/IntelliJ IDEA CE.app
    • Flutter plugin version 71.3.6
    • Dart plugin version 223.8214.16

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

[✓] Connected device (2 available)
    • macOS (desktop) • macos  • darwin-arm64   • macOS 12.4 21F79 darwin-arm64
    • Chrome (web)    • chrome • web-javascript • Google Chrome 109.0.5414.119

[✓] HTTP Host Availability
    • All required HTTP hosts are available

• No issues found!

Hello @GroovinChip. Is it possible for you to isolate the issue, and provide a sample that does not include any 3rd party libraries, i.e. macos_ui?

Hello @GroovinChip. Is it possible for you to isolate the issue, and provide a sample that does not include any 3rd party libraries, i.e. macos_ui?

No, sorry. I'm the author of macos_ui, my goal is to ensure my library remains fully functional and up to date.

@GroovinChip Ahh I see. Didn't know that :) I'll go ahead and label it as I can reproduce the issue with the steps outlined above
flutter doctor -v (Stable)
[!] Flutter (Channel stable, 3.7.0, on macOS 13.1 22C65 darwin-arm64, locale en-US)
    • Flutter version 3.7.0 on channel stable at /Users/nabeelparkar/fvm/versions/stable
    ! Warning: `dart` on your path resolves to /opt/homebrew/Cellar/dart/2.18.6/libexec/bin/dart, which is not inside your current Flutter SDK checkout at /Users/nabeelparkar/fvm/versions/stable. Consider adding /Users/nabeelparkar/fvm/versions/stable/bin to the front of your path.
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision b06b8b2710 (3 days ago), 2023-01-23 16:55:55 -0800
    • Engine revision b24591ed32
    • Dart version 2.19.0
    • DevTools version 2.20.1
    • If those were intentional, you can disregard the above warnings; however it is recommended to use "git" directly to perform update checks and upgrades.

[✓] Android toolchain - develop for Android devices (Android SDK version 33.0.0)
    • Android SDK at /Users/nabeelparkar/Library/Android/sdk/
    • Platform android-33, build-tools 33.0.0
    • ANDROID_SDK_ROOT = /Users/nabeelparkar/Library/Android/sdk/
    • Java binary at: /Users/nabeelparkar/Library/Application Support/JetBrains/Toolbox/apps/AndroidStudio/ch-0/213.7172.25.2113.9123335/Android Studio.app/Contents/jre/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 11.0.13+0-b1751.21-8125866)
    • All Android licenses accepted.

[✓] Xcode - develop for iOS and macOS (Xcode 14.2)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Build 14C18
    • CocoaPods version 1.11.3

[✓] Chrome - develop for the web
    • CHROME_EXECUTABLE = /Applications/Brave Browser.app/Contents/MacOS/Brave Browser

[✓] Android Studio (version 2021.3)
    • Android Studio at /Users/nabeelparkar/Library/Application Support/JetBrains/Toolbox/apps/AndroidStudio/ch-0/213.7172.25.2113.9123335/Android Studio.app/Contents
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build 11.0.13+0-b1751.21-8125866)

[✓] IntelliJ IDEA Ultimate Edition (version 2022.3.1)
    • IntelliJ at /Users/nabeelparkar/Applications/JetBrains Toolbox/IntelliJ IDEA Ultimate.app
    • Flutter plugin version 71.3.6
    • Dart plugin version 223.8214.16

[✓] IntelliJ IDEA Ultimate Edition (version 2022.3.1)
    • IntelliJ at /Users/nabeelparkar/Library/Application Support/JetBrains/Toolbox/apps/IDEA-U/ch-0/223.8214.52/IntelliJ IDEA.app
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart

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

[✓] Connected device (2 available)
    • macOS (desktop) • macos  • darwin-arm64   • macOS 13.1 22C65 darwin-arm64
    • Chrome (web)    • chrome • web-javascript • Brave Browser <IP_ADDRESS>

[✓] HTTP Host Availability
    • All required HTTP hosts are available

! Doctor found issues in 1 category.

I don't think it has anything to do with my new physics, but still looking into it. The scrollbars with the problem are the ones in your ResizablePane. This error comes up when the builder doesn't use the provided scrollController. So it's true that they have no position attached. Still looking into the actual cause and/or why this worked before.

Hey @GroovinChip, did you actually bisect the issue to https://github.com/flutter/flutter/pull/108298 with git? I do not see how that change could have caused this issue.
Unfortunately, we cannot debug packages on pub - there are many of them!
You did not specify which flutter version you were on before upgrading, can you provide that information?
There are many custom widgets in your repro above, can you reproduce the issue without them?
Also, you may be interested in the flutter/tests repo. Anyone can submit tests for their project or package there. We run those tests for every change made in the framework.

The commit that changed the behaviour was here: https://github.com/flutter/flutter/pull/113237
In MacosScrollbar, you construct a ScrollbarThemeData with thumbVisibility=true, meaning you always want the scrollbar to show. Until https://github.com/flutter/flutter/pull/113237, that wasn't respected, and the scrollbar was only shown when scrolled.
Now the error is thrown because sometimes you are constructing a scrollbar that you want to be always shown, but that doesn't actually have a scrollable child (anything attached to the provided controller).

Thank you @moffatman. It sound like this is not a framework issue then? If so we can close this.

Hi @Piinks @moffatman! Thanks for looking into this.
I haven't had much time to delve into what you've written, but I hope to get to it sometime this week. I was able to spend a handful of minutes yesterday looking into the issues related to ScrollbarTheme and not using provided ScrollControllers, and while I did see that the issue where controllers weren't attached to views went away, I was then presented with an issue where controllers were attached to more than one view, so I'm pretty perplexed and haven't been able to look further into it. So I'll find some time to investigate further and report back.
Thanks again!


Bump protractor and cucumber to 6.x

Changes

Bump protractor to 6.0.0 from 5.4.2
Bump cucumber to 6.0.2 from 5.1.0
Add gulp-cli install to travis install sequence.

Testing

gulp test --headless should pass.

This PR is blocked by https://github.com/protractor-cucumber-framework/protractor-cucumber-framework/pull/172

Superseded by https://github.com/cfpb/cfgov-refresh/pull/5345


added an example in EXAMPLES.md

Added a new Product Cards Listing example in EXAMPLES.md
It contributes to #31

Thanks, @vczb!
I have fixed the conflict.

Thanks, @vczb! I have fixed the conflict.

thanks a lot, can you please squash the both commits in a single one ?

I have squashed both commits in a single one.


CNN example with Deeplearning4j in Java

Thank you for reviewing!
I made all the required changes, have a look


incorrect timestamps

Output transcriptions such as .vtt and .srt files, have correct transcriptions, but the timestamps for what is said when is wrong. A 3 second sentence is marked as 3 minutes instead. It doesn't appear to be exact seconds to minutes though.
What we're outputting
00:00.000 --> 03:36.000
- Things you prioritize, like the most useful
03:36.000 --> 07:44.000
and interesting conversations, it goes parties,
07:44.000 --> 10:52.000
then workshops, then conference session.
10:52.000 --> 15:28.000
- I'm sorry, but you asked to be roasted, I will roast.
15:28.000 --> 17:56.000
- I'm talking about the level of intelligence
17:56.000 --> 20:52.000
of the cat or dog, okay?

What is outputted by yt-whisper and should be output instead
00:00.000 --> 00:06.500
Things you prioritize, like the most useful and interesting conversations, it goes parties, then workshops, then conference sessions.

00:06.500 --> 00:09.200
I'm sorry, but you asked to be roasted, I will roast.

00:09.200 --> 00:12.500
I'm talking about the level of intelligence of a cat or a dog, okay?

@sloganking Could you check if https://github.com/m1guelpf/whisper-cli-rs/commit/3f7558a85428df2c2ac580e973e495088115209f fixes this?

@m1guelpf That did not fix is. A 4 second sentence is still registered as 10 minutes long.

printing out fragment.start and fragment.stop in as_vtt(). They don't seem to correlate with milliseconds at all.
fragment: Utternace { start: 0, stop: 624, text: " Hello world, this is my second test." }
fragment: Utternace { start: 624, stop: 696, text: " Can you hear me very well?" }
fragment: Utternace { start: 696, stop: 1696, text: " [BLANK_AUDIO]" }

This is the output from a 8.3 second audio clip. The first sentence is finished in just under 4 seconds. Yet I don't see how 624 relates to 4s at all.


Fix CHANGELOG: rm entry mentioning a test fix

Please see https://github.com/docker/docker.github.io/pull/5499

Please sign your commits following these rules:
https://github.com/moby/moby/blob/master/CONTRIBUTING.md#sign-your-work
The easiest way to do this is to amend the last commit:
$ git clone -b "patch-1" git@github.com:alb-i986/docker-ce.git somewhere
$ cd somewhere
$ git commit --amend -s --no-edit
$ git push -f

Amending updates the existing PR. You DO NOT need to open a new one.


No such file or directory

The same error with the latest ver.
 cat: .tmp/probed_tmp_scrap3.txt: No such file or directory
find: ‘.tmp/gospider/’: No such file or directory
grep: .tmp/gospider.txt: No such file or directory
sort: cannot read: webs/webs.txt: No such file or directory
cat: '.tmp/gospider/*': No such file or directory`

Fixed, remember to run ./install.sh


Skip deprecated go track 1 packages in release doc generation

Resolves https://github.com/Azure/azure-sdk/issues/5998
This removes packages with empty ServiceName values from go-packages.csv and go-all.md because they are deprecated/track 1.

This is what go-all looks like now:

Still a couple of gaps, notably sdk/security/keyvault/internal


WSL2 Increase the maximum amount of inotify user watches

Please consider increasing the maximum amount of inotify user watches (fs.inotify.max_user_watches) in WSL2.
The current limit (8192) is too low for most projects.
For a seamless Visual Studio Code Remote Development experience a new limit of 524288 is suggested. That would allow to watch up to 524288 files, using up to 540 MiB kernel memory.
Alternatively make fs.inotify.max_user_watches configurable, which would probably require the init system of WSL2 to read configuration files placed in /etc/sysctl.d/. See #4232.
The current workaround is:

Create a script:

#!/bin/sh
sysctl fs.inotify.max_user_watches=524288
sysctl -p

Run sudo chown root:root and sudo chmod 755 on the script
Either move the script to a directory owned by root or run sudo chattr +i on it, otherwise it can be deleted
Run sudo EDITOR=nano visudo and add this:
%sudo ALL=(ALL:ALL) NOPASSWD:/path/to/script.sh
Add this to .profile:
sudo /path/to/script.sh
Don't forget to remove the line above from visudo if you delete /path/to/script.sh

#4232

@Comevius thank you, thank you, thank you! This is really a brilliant workaround, and actually the only one that works. I spent several hours trying to make it work and the most of 'google' solutions claimed that:
sudo sysctl fs.inotify.max_user_watches=524288 && sudo sysctl -p should've been enough to make it pernament. It is not the case for WSL. Now everything works smoothely.
This really needs to be documented (at least) or implemented in WSL2.

I use  sudo passwd root to enable root user ,
then cd /root
then I create a init.sh file in /root
then I sudo chown root:root init.sh and sudo chmod 755
then vim /etc/sudoers
add %sudo ALL=(ALL:ALL) NOPASSWD:/root/init.sh
then vim .profile
add sudo /root/init.sh
the result : fail
#4232
please help!!

@jiangbo0216 it works for me.
You don't need to enable password for root, just use
sudo su -
You can't use vim directly to edit /etc/sudoers, you should use visudo (pay attention to the original instructions).
There is already a line in /etc/sudoers for the %sudo group, the order of records is important, so you should put the additional record after the existing one:
%sudo   ALL=(ALL:ALL) ALL
%sudo   ALL=(ALL:ALL) NOPASSWD:/root/init.sh

You need to exit the editor the changes to take effect.
To check, that everything works just open another bash shell and try to execute:
sudo /root/init.sh
it should not ask for a password.
After placing script call into the .profile there should be init.sh output visible during the shell start.

https://auo.nu/posts/wsl2-livereload/
This blog post says that WSL2 only watches file in the WSL-related folders. So I move my original one from Windows-related to WSL-related and it works! Many thanks to this blog.


Custom loggers

For now I use logging.debug, logging.info etc to log on the root debugger.
It works well enough, but consider passing debuggers later to make this more custom ?
This is however not really a priority and may be scrapped altogether.

In addition, hunt for all "print" in the code and replace them with logging calls when appropriate.


Import remade cannon worker

Seems to me that the cannon.js branch of this project is left out. After all, it will be remade using the GoblinPhysics engine with intergrated collision listening. However, I have rewritten the worker entirely from scratch and added support to trimeshes, yet it is far from complete (still no joint support).

Anybody had time to check this out?

Wellllll, the last merge from @chandlerprall was ~2 years ago and you made this PR over a year ago, so I'm gonna say no :(

Yeah, that sucks. Just would've hoped this branch could've been saved before complete inactivity.

I hear ya. Maybe one day it'll be revived, but it looks like Chandler's busy being a lead developer elsewhere so I don't necessarily blame him for putting this project on the backburner.
He seems to be pretty active on Twitter. Perhaps you could try reaching out to him there?


Model evaluation

Hi,
I have a question about cam_example.py. Shouldn't the model be in eval() mode? I noticed without setting the model.eval() the prediction labels are wrong while after setting to eval mode the ScoreCam and SSCAM heatmaps are zero. Could you please check this?

Hey there @gulleh,
I just made a PR #15 to address this. However, I was not able to reproduce your issue of having zero-value heatmaps for ScoreCAM and SSCAM. Did you use the default script arguments?
Here is what I got removing some CAMS:

I made some changes to check the classification score first. Without setting to eval the predictions were wrong and the heat maps were are the same for different images. My problem is that after setting the model.eval(), the heatmaps won't change for different images.

I made some changes to check the classification score first. Without setting to eval the predictions were wrong and the heat maps were all the same for different images. My problem is that after setting the model.eval(), the heatmaps won't change for different images.

Thanks for the feedback! If you used different argument values than the default ones or modified the script, would you mind telling me? I'm unable to reproduce the behaviour you mentioned. As you can see, the output I got was from the script using the latest PR commit (only modifying the CAM selection), and the outputs are different from one method to another as shown on the picture I uploaded.
Perhaps you were offline when you used the script? Then the pretrained parameters of the model would not have been loaded, and random initialization shields roughly similar classification scores.
Also you mentioned using it on different images, did you batch them ? or forward them one after another? (should work both ways, but there might be some issues with one of those methods). If you batched them, you have to change the .squeeze(0) for dispatching correctly the intended heatmaps (cf. https://github.com/frgfm/torch-cam/blob/master/scripts/cam_example.py#L70)
And for your image outputs, could you upload the result (with subplots showing the heatmaps that should be different)
Let me know if that helps!

Thanks for the reply. Here is the snippet of the code I used.
fig, axes = plt.subplots(5, len(cam_extractors), figsize=(15, 15))
model.eval()
model.zero_grad()
for index in range(0, 5):
    img, target = iter(val_loader).next()
    for idx, extractor in enumerate(cam_extractors):
        scores = model(img.to(device))
        _, pred_label_idx = torch.topk(F.softmax(scores, dim=1), 1)
        predicted_label = idx_to_labels[str(pred_label_idx.item())][1]
        # Select the class index
        class_idx = scores.squeeze(0).argmax().item() if class_idx is None else class_idx
        if extractor is None:
            result = to_pil_image(denormalize(img.squeeze(0)))
        else:
            # Use the hooked data to compute activation map
            activation_map = extractor(class_idx, scores).cpu()
            # Clean data
            extractor.clear_hooks()
            # Convert it to PIL image
            # The indexing below means first image in batch
            heatmap = to_pil_image(activation_map, mode='F')
            # Plot the result
            result = overlay_mask(to_pil_image(denormalize(img.squeeze(0))), heatmap)

        axes[index, idx].imshow(result)
        axes[index, idx].set_title(extractor.__class__.__name__, size=8)
        axes[index, idx].set_xlabel(predicted_label)
        target_label = idx_to_labels[str(target.item())][1]
        if predicted_label != target_label:
            print('Predicted:', predicted_label, 'Target is:', idx_to_labels[str(target.item())][1])
        axes[index,idx].axis('off')

plt.tight_layout()
plt.show()

As you can see the heatmap does not change for different images.

Thanks for the details @gulleh!
I think I see why you get this behaviour: the sample script was designed to visualize different cams on a single picture, so I iterate only once per extractor. Which allows me to clear the hooks from the model before the next extractor gets to set his own.
In your case, on top of iterating on extractors, you iterate over pictures. When you keep the same extractor, but change the picture, if you use .clear_hooks it will basically clear all the hooks that were set when they were instantiated.
If I'm correct, you could easily correct this behaviour by:

iterating first on extractors
then iterating on pictures (without clearing hooks between them, only when you're finished with this CAM extractor)

Alternatively, if you want to keep your loop order, you would have to repopulate correctly the hook_handles attribute of the extractor, but it seems less efficient that my previous suggestion.
Let me know if you encounter other issues :)

Thanks for the suggestion. However, I changed the position of clear_hooks and it seems to be working but I'm not sure.


[DPP] [BUG] EDC 0.7.3 incompatibility because of "@id" policy definition

Current Behavior
When the catalog is called in the new EDC 0.7.3 the policies are configured in a different way as mentioned in this ticket:
https://github.com/eclipse-edc/Connector/issues/4179
This affects our digital product pass policy parsing logic.
Expected Behavior
Change the structure of the backend and frontend to support only DID Documents, with the "@id" at the left operand.
Fix in frontend here, to get the value by "@id":
https://github.com/eclipse-tractusx/digital-product-pass/blob/d61dd719d97109983ae2c96673cac048bfbff7b3/dpp-frontend/src/utils/edcUtil.js#L214-L220
Fix in the backend here, to parse as DidDocument like the operator:
https://github.com/eclipse-tractusx/digital-product-pass/blob/d61dd719d97109983ae2c96673cac048bfbff7b3/dpp-backend/digitalproductpass/src/main/java/org/eclipse/tractusx/digitalproductpass/core/models/negotiation/policy/Constraint.java#L47-L50

I believe I have solved the problem ;)

Will be included in v4.0.0-rc2


VBA Highlighting Syntax issues

this is a port of the question
https://codegolf.meta.stackexchange.com/questions/13670/vba-syntax-highlighting from meta.codegolf.stackexchange.com
VBA Code that uses conditional formatting, comments, date literals or open-ended strings is not highlighted correctly. For example,

is not Highlighted like it is in the Microsoft Office Visual Basic Editor

What is happening?
It would seem that instead of properly identifying ' as the comment starting literal, it was incorrectly assigned to being a char literal (note that VBA does not support char literals in any fashion). It would seem that the # character was then assigned to being the comment starting literal, instead of the Conditional Compilation literal.
Of significant not is that # also serves as the wrapping literal for declaring date type objects inline as seen in the example below

Note that the #12:00:01 AM# is highlighted to indicate that it is commented out, rather than being represented like any other string or number literal

Beyond this, VBA does not support multiline string literals of any form, and in certain cases  supports string literals without a terminating ". However, the current Prettify.js setup for VBA supports multiline strings, meaning that you an end up with code blocks that look like the one below on our site.

Note that line 3 is included in the string highlighting, despite not being a string
What needs to be changed?
In the Prettify.js config,

The comment literal needs to be changed from # to ' (which can be escaped by wrapping string)

The ' char(?) literal needs to be removed, as VBA does not support chars

Removal of highlighting for multiline strings should be considered (if there are no characters, such as , or ),  expected after the end of a VBA string, VBA will autoformat to add a trailing " at the end of the line, and they are otherwise illegal strings)

It should be considered highlighting any, non-string, section of code which is wrapped in [...] (where ... is some expression) in the same light red color as the string and number literals are, as this equivalent to a string which is wrapped in an Application.Evaluate("...") call

TL;DR: syntax highlighting for VBA via Prettify.js is incorrect; Lets fix it.

VB language handler is implemented as an extension, i.e you should load lang-vb.js in addition to prettify.js.
Here is a live example to demonstrate: https://jsfiddle.net/amroamroamro/1dpsjjeb/

On the Stack Overflow network, in addition to the core prettify.js, they chose to include only a subset of the most common language extensions (for size reasons). Fortunately for you, lang-vb.js is included (they bundle it all in one JS file).
This means you just have to specify the correct language hint, either by question tag or a special HTML comment (for details refer to What is syntax highlighting and how does it work?).


Account page loading indefinitely after logging

Currently the home page is loaded, but it is still an issue.