Add Symbol to Map automatically creates Feature Template

User Story:
After I've selected a symbol and clicked on the map, the Feature Template for that symbol is created and available in the Create Features window. Alternatively, I can export my favorites to Feature Templates.
Another option would be a direct SIDC to feature template capability. This model would take the spreadsheet and produce feature templates/favorites.
@tlauver do you know whether either of these options is possible or not? No worries if you don't have any info, I can look into it at that point.

No, I don't believe I ever looked into whether feature templates were possible or not through that method.  I'd have to take a look at the Pro SDK.

https://pro.arcgis.com/en/pro-app/sdk/api-reference/#topic9917.html @csmoore  so I'm looking at the ArcGIS Pro 1.3 API reference guide, and I found several classes that are related to feature templates. Could we draw from this code to create feature templates from our existing favorites? or from adding a feature to the map in the pro symbol editor?

@Dbarnes1 - nice job finding - that shows how to get a currently selected feature template and how to get a template from a layer, but not really how to add/create a new CreateFeature template to a layer (but that may indeed be there somewhere but just not leaping out at me - although someone with more experience with Pro SDK like @jmccausland or @deepinderdeol might be able to find)

Created related/more specific issue that can be used for CR entry here: https://github.com/Esri/military-symbol-editor-addin-wpf/issues/83


Collect garden tools images from Conq perspective

Create a dataset of garden tools images using src/regrasping_demo/get_detections.py << get_color_img

script added under scripts/collect_images_from_gaze_pose.py in branch image-collection-conq

done


Identifiers validation failed for dataset Código de barras de ocho complejos de especies de aves mexicanas

Identifier validation failed for the dataset Código de barras de ocho complejos de especies de aves mexicanas:

Crawler attempt: 284
Publishing organization: Comisión nacional para el conocimiento y uso de la biodiversidad
Publishing country: MEXICO
Installation: IPTCONABIO
Number of occurrences indexed for this dataset at the time of the issue: 272
Cause: GBIF IDs hit maximum allowed - 50%, duplicates - 96%, total records count 267, absent records count 255
Pipelines execution steps: http://api.gbif.org/v1/pipelines/history/execution/3072657/step

Publisher email
Hello,
I am contacting you from the GBIF Secretariat about a dataset published by the Código de barras de ocho complejos de especies de aves mexicanas : https://doi.org/10.15468/v86pap.
We noticed that the occurrenceIDs were changed. We have temporarily paused the ingestions of this dataset.
As you might already know, when an occurrence record has a new occurrenceID for a given dataset, our system considers it to be a new occurrence.
This means that it will be given a new gbifid and a new occurrence URL (like this one: https://www.gbif.org/occurrence/1252968762) and the old gbifid and URL will be deprecated.
In this case, this means that the occurrence URLs would be deprecated when ingesting the newest versions of these datasets.
We would like to check with you if those changes were intentional. Do you know if this is the case?
Please let us know, thanks! We are happy to resume the dataset ingestion.
Note that some users rely on those occurrence URLs and gbifids (like https://bionomia.net for example). In an attempt to improve
the stability of the occurrence URLs and gbifids, we have implemented a warning system to detect these type of changes in datasets (see this news item).
If the data publisher can provide us with a list of old and new occurrenceIDs per record, we can avoid the identifier and URL changes. Could that be an option?
Please let us know if you have any question. Thanks!
All the best,

You can skip/fix identifier validation using the registry UI.

Identifier validation still  failing for the dataset Código de barras de ocho complejos de especies de aves mexicanas:

Crawler attempt: 285
Publishing organization: Comisión nacional para el conocimiento y uso de la biodiversidad
Publishing country: MEXICO
Installation: IPTCONABIO
Number of occurrences indexed for this dataset at the time of the issue: 272
Cause: GBIF IDs hit maximum allowed - 50%, duplicates - 96%, total records count 272, absent records count 260
Pipelines execution steps: http://api.gbif.org/v1/pipelines/history/execution/3072785/step

Publisher email
Hello,
I am contacting you from the GBIF Secretariat about a dataset published by the Código de barras de ocho complejos de especies de aves mexicanas : https://doi.org/10.15468/v86pap.
We noticed that the occurrenceIDs were changed. We have temporarily paused the ingestions of this dataset.
As you might already know, when an occurrence record has a new occurrenceID for a given dataset, our system considers it to be a new occurrence.
This means that it will be given a new gbifid and a new occurrence URL (like this one: https://www.gbif.org/occurrence/1252968762) and the old gbifid and URL will be deprecated.
In this case, this means that the occurrence URLs would be deprecated when ingesting the newest versions of these datasets.
We would like to check with you if those changes were intentional. Do you know if this is the case?
Please let us know, thanks! We are happy to resume the dataset ingestion.
Note that some users rely on those occurrence URLs and gbifids (like https://bionomia.net for example). In an attempt to improve
the stability of the occurrence URLs and gbifids, we have implemented a warning system to detect these type of changes in datasets (see this news item).
If the data publisher can provide us with a list of old and new occurrenceIDs per record, we can avoid the identifier and URL changes. Could that be an option?
Please let us know if you have any question. Thanks!
All the best,

You can skip/fix identifier validation using the registry UI.

Identifier validation still  failing for the dataset Código de barras de ocho complejos de especies de aves mexicanas:

Crawler attempt: 288
Publishing organization: Comisión nacional para el conocimiento y uso de la biodiversidad
Publishing country: MEXICO
Installation: IPTCONABIO
Number of occurrences indexed for this dataset at the time of the issue: 272
Cause: GBIF IDs hit maximum allowed - 50%, duplicates - 100%, total records count 272, absent records count 272
Pipelines execution steps: http://api.gbif.org/v1/pipelines/history/execution/3093594/step

Publisher email
Hello,
I am contacting you from the GBIF Secretariat about a dataset published by the Código de barras de ocho complejos de especies de aves mexicanas : https://doi.org/10.15468/v86pap.
We noticed that the occurrenceIDs were changed. We have temporarily paused the ingestions of this dataset.
As you might already know, when an occurrence record has a new occurrenceID for a given dataset, our system considers it to be a new occurrence.
This means that it will be given a new gbifid and a new occurrence URL (like this one: https://www.gbif.org/occurrence/1252968762) and the old gbifid and URL will be deprecated.
In this case, this means that the occurrence URLs would be deprecated when ingesting the newest versions of these datasets.
We would like to check with you if those changes were intentional. Do you know if this is the case?
Please let us know, thanks! We are happy to resume the dataset ingestion.
Note that some users rely on those occurrence URLs and gbifids (like https://bionomia.net for example). In an attempt to improve
the stability of the occurrence URLs and gbifids, we have implemented a warning system to detect these type of changes in datasets (see this news item).
If the data publisher can provide us with a list of old and new occurrenceIDs per record, we can avoid the identifier and URL changes. Could that be an option?
Please let us know if you have any question. Thanks!
All the best,

You can skip/fix identifier validation using the registry UI.


Update skopeo version to v0.2.0

Fixes #484
Signed-off-by: Swathi Gangisetty<EMAIL_ADDRESS>
What this PR does / why we need it:
Which issue this PR fixes (optional, in fixes #<issue number>)(, fixes #<issue_number, ...) format, will close the issue when PR is merged: fixes #:
Special notes:

Since then they've updated to 1.0 and 1.1.0. I think we should try 1.1.0 and use that if it works. I'll update #484 as well.

Since then they've updated to 1.0 and 1.1.0. I think we should try 1.1.0 and use that if it works. I'll update #484 as well.

Was just trying to see if the build process would go through with this PR. 1.0.0 seems like a big version jump, sure it won't break our usage?

I'm going to close this so we can try 1.1.1 as well. I'll give that a try and open a PR if tests look good.


enable use_flash_attention for transformer engine

@ethanhe42 could you fix the code style, please?
black . --check --verbose --diff
and fix the differencies


Update AutoBone steps on the GUI to match the docs

Removes the chair step (yippeee!!)
Adds a "Requirements" page that explains all that is required to use AutoBone
Updates the steps for AutoBone movements to match the docs
Adds steps for AutoBone movements on the "Get ready to move" page (as a temporary workaround to show them before starting the recording)
Fixes the tip on the "Get ready to move" page

I realized after I made this PR that I didn't put in the "Reset all proportions" as a step yet, I don't think I'll add it to this PR, but it would be good to get that done at some point in a later PR. Uriel has been working on some AutoBone stuff, so maybe their PR can include something of the sort.


Ao fazer alteração no arquivo aparece um número que se multiplica de acordo com o que eu altero.

Note I managed to solve the problem by uninstalling two extensions, which are Dracula and Power Mode. But I would like to know what really happened.
My problem is when you make any changes to the file, a number appears that multiplies according to what is changed in the file. This will leave the text of the file out on the line as the biggest lag is left.
Here below is a link to a video with a real example of what my PC looks like.. 👇🏾
Link to video

Hello, @VitorMeloDS! Just so you know, that's a functionality of the Power Mode extension.
You should refer to the extension authors on how to disable the "combo meter".
You can see it demonstrated here: https://github.com/hoovercj/vscode-power-mode

Obrigado, @brunovieira97 eu percebi que era por causa da extensão mais irei ficar sem ela mesmo, de toda forma obrigado!


bump tools and set require go1.21 as minimum

What type of PR is this?
/kind cleanup
/kind feature
What this PR does / why we need it:

bump tools and set require go1.21 as minimum

/hold for test-infra job update to go1.22
/assign @saschagrunert @puerco @ameukam
cc @kubernetes-sigs/release-engineering
Which issue(s) this PR fixes:
None
Special notes for your reviewer:
Does this PR introduce a user-facing change?

NONE

drop eneable checks
WARN [linters_context] gocritic: no need to enable check "appendAssign": it's already enabled
WARN [linters_context] gocritic: no need to enable check "argOrder": it's already enabled
WARN [linters_context] gocritic: no need to enable check "assignOp": it's already enabled
WARN [linters_context] gocritic: no need to enable check "badCall": it's already enabled
WARN [linters_context] gocritic: no need to enable check "badCond": it's already enabled
WARN [linters_context] gocritic: no need to enable check "captLocal": it's already enabled
WARN [linters_context] gocritic: no need to enable check "caseOrder": it's already enabled
WARN [linters_context] gocritic: no need to enable check "codegenComment": it's already enabled
WARN [linters_context] gocritic: no need to enable check "commentFormatting": it's already enabled
WARN [linters_context] gocritic: no need to enable check "defaultCaseOrder": it's already enabled
WARN [linters_context] gocritic: no need to enable check "deprecatedComment": it's already enabled
WARN [linters_context] gocritic: no need to enable check "dupArg": it's already enabled
WARN [linters_context] gocritic: no need to enable check "dupBranchBody": it's already enabled
WARN [linters_context] gocritic: no need to enable check "dupCase": it's already enabled
WARN [linters_context] gocritic: no need to enable check "dupSubExpr": it's already enabled
WARN [linters_context] gocritic: no need to enable check "elseif": it's already enabled
WARN [linters_context] gocritic: no need to enable check "exitAfterDefer": it's already enabled
WARN [linters_context] gocritic: no need to enable check "flagDeref": it's already enabled
WARN [linters_context] gocritic: no need to enable check "flagName": it's already enabled
WARN [linters_context] gocritic: no need to enable check "ifElseChain": it's already enabled
WARN [linters_context] gocritic: no need to enable check "mapKey": it's already enabled
WARN [linters_context] gocritic: no need to enable check "newDeref": it's already enabled
WARN [linters_context] gocritic: no need to enable check "offBy1": it's already enabled
WARN [linters_context] gocritic: no need to enable check "regexpMust": it's already enabled
WARN [linters_context] gocritic: no need to enable check "singleCaseSwitch": it's already enabled
WARN [linters_context] gocritic: no need to enable check "sloppyLen": it's already enabled
WARN [linters_context] gocritic: no need to enable check "sloppyTypeAssert": it's already enabled
WARN [linters_context] gocritic: no need to enable check "switchTrue": it's already enabled
WARN [linters_context] gocritic: no need to enable check "typeSwitchVar": it's already enabled
WARN [linters_context] gocritic: no need to enable check "underef": it's already enabled
WARN [linters_context] gocritic: no need to enable check "unlambda": it's already enabled
WARN [linters_context] gocritic: no need to enable check "unslice": it's already enabled
WARN [linters_context] gocritic: no need to enable check "valSwap": it's already enabled
WARN [linters_context] gocritic: no need to enable check "wrapperFunc": it's already enabled

/unhold


armstrong in c++

hii there an update has been released regarding hactoberfest
Statement _PR should be labeled hacktoberfest-accepted by maintainer
So can u label it pls
or should i make new pr again


Heads-up about the new PyPI

Hi. I'm the project manager for the new Python Package Index (Warehouse), which is currently in pre-production at http://pypi.org/ . Your code integrates with the old PyPI API, so I wanted to make sure you knew that, on the Warehouse roadmap, it looks like the full switch will happen sometime in April. Here's a heads-up about why we're switching, what's changed, and what to expect.
The legacy PyPI site at https://pypi.python.org started in the early 2000s. In recent years, users faced outages, malicious packages, and spam attacks, and the legacy codebase made it hard to maintain and even harder to develop new features.
The new PyPI has a far more modern look, and is up-to-date under the hood as well; a proper web framework (Pyramid), 100% backend test coverage, and a Docker-based development environment, make it easier for current and new developers to maintain it and add features.
Thanks to Mozilla's Open Source Support funding, developers have added many new features, overhauled infrastructure, and made steady progress towards redirecting traffic to the new site and shutting down the old one. As of the middle of last year, package releases must go through the new PyPI, and as of late February, new user account registration is only available on the new site. The full switch will include redirecting browser and pip install traffic from the old site; then, sometime in late April or early May, the legacy site will be entirely shut down.
Shields will probably be able to seamlessly switch to the new site, and thanks to redirects, may not have to change anything immediately. Here's a migration guide. And here's the reference for the updated APIs.
Some new PyPI features:

mobile-responsive UI
easy-to-read project activity journal for project maintainers
support for multiple project URLs (e.g., for a homepage and a repo)
no need to "register" a project before initial upload

Things that are going away, or already have (sometimes for policy or spam-fighting reasons), include:

pythonhosted.com documentation hosting (pypa/warehouse#582)
download counts visible in the API, as you saw in #716 (instead, use the Google BigQuery service)
package maintainers being able to upload a new release via the web UI (instead, the recommended command-line tool is Twine)
package maintainers being able to log in and update release descriptions via the web UI (to update release metadata, they need to upload a new release; see distutils-sig discussion)
users being able to upload a package without verifying their email address with PyPI first
HTTP access to APIs; now it's HTTPS-only

And in the works:

PEP 541 will enable more timely package takeovers, as people get package names transferred to them after conflict resolution
Now that PEP 566 has been approved, developers are working to get Markdown supported for README files on PyPI

You can subscribe to this announcements list to get a heads-up when the big changes are about to happen.
Thank you for integrating with PyPI, and please let us know if you have any questions or problems with the new site!

Hi @brainwane - thanks for posting about this. As a python user I'm eagerly awaiting the removal of the red pre-prod banner.

Shields will probably be able to seamlessly switch to the new site

I've tried switching the base API URL to https://pypi.org/, run the test suite for the pypi integrations and it all passes, so I think you are right :)
Would you advise us to switch our production server over to querying warehouse instead of legacy PyPI now while warehouse is still technically in pre-prod, or would you advise that we should stick with legacy PyPI until April?

@chris48s Thank you! My take is that, now that you've checked and your test suite passes, you should wait till you get the email that says "switch now" via https://mail.python.org/mm3/mailman3/lists/pypi-announce.python.org/ which will probably be in April.
Also is there any chance you'll be at either of the packaging sprints in the next few months?

you should wait till you get the email that says "switch now" via https://mail.python.org/mm3/mailman3/lists/pypi-announce.python.org/

Sorted - cheers. I've signed up to that mailing list. I'll leave this issue open for now and close it once we switch over.

Also is there any chance you'll be at either of the packaging sprints in the next few months?

I don't currently have plans to, but thanks for the info :)


Paths: End on /

First seen in https://github.com/openPMD/openPMD-validator/issues/43
Should we define that all paths end on / in the standard?

I think imposing the final / makes implementations easier (good!) while cluttering the standard with details a little. A path join operation is in principle the logical way to implement it (available in Python and modern C++) which then would be agnostic. Nevertheless some very rudimentary languages might not have such high-level functionality.
The example currently shows a trailing / and I think it might make things indeed easier for implementations which I favor for the community.

OK with me.


Expand/wrap text to window width instead of fixed width

Can you implement functionality where the Ursine Polar/Umbra themes wrap to the width of the open window when the window is large rather than only expanding up to a certain width?

(A bit late to this, sorry!)
I could do that by changing a couple of lines, but this is more of a design choice I wanted to keep from Bear. I personally kinda like that negative space when it comes to note-taking, & other people may like that part of it as well.
I could make another variant available in the releases page that has this, but I fear I'd just clutter each released version like this overtime, since it'd be just a duplicate with a couple of small changes. Not sure if that's the way to go or not, so I'll await a response.
For now though, you can get this by doing a small change to the file. In lines 300-305 in both Ursine Polar and Umbra's CSS files (ursine-polar.css/ursine-umbra.css), you'll find this block of code:
#write {
  position: static;
  width: 90%;
  max-width: 700px;
  line-height: 1.6;
  transform: none;
}

Simply delete the line that has max-width, & that'll do the trick. You can also play around with the percent value of width if you want to increase or decrease the writing area width.

That manual workaround works just fine. I just commented it out instead of deleting it. Thank you!


Resolving records through more than 8 CNAME fails due to hardcoded MAX_RESTART_COUNT

I've run into an edge case where a name passed through 9 CNAMEs (and 10 total resolutions to get the final answer). Because MAX_RESTART_COUNT is hardcoded  there's no way to tune the configuration to work around this situation.
The original bug report is in the pfSense bug tracker here where detailed traces from drill are available.

I've hit this same issue. It prevents me (and anybody using Unbound) from setting up a Windows device with their Microsoft Account (MSA). logincdn.msauth.net, used during MSA authentication has 9 CNAME records (see dig output below).            IN      A

;; ANSWER SECTION:
logincdn.msauth.net.    10      IN      CNAME   lgincdn.trafficmanager.net.
lgincdn.trafficmanager.net. 29  IN      CNAME   lgincdnmsftuswe2.azureedge.net.
lgincdnmsftuswe2.azureedge.net. 1493 IN CNAME   lgincdnmsftuswe2.afd.azureedge.net.
lgincdnmsftuswe2.afd.azureedge.net. 16 IN CNAME star-azureedge-prod.trafficmanager.net.
star-azureedge-prod.trafficmanager.net. 23 IN CNAME dual.t-0009.t-msedge.net.
dual.t-0009.t-msedge.net. 227   IN      CNAME   t-0009.t-msedge.net.
t-0009.t-msedge.net.    59      IN      CNAME   Edge-Prod-WSTr3.ctrl.t-0009.t-msedge.net.
Edge-Prod-WSTr3.ctrl.t-0009.t-msedge.net. 239 IN CNAME edge-prod-wstr3.ctrl.t-0001.trafficmanager.net.
edge-prod-wstr3.ctrl.t-0001.trafficmanager.net. 0 IN CNAME standard.t-0009.t-msedge.net.
standard.t-0009.t-msedge.net. 21 IN     A       <IP_ADDRESS>
standard.t-0009.t-msedge.net. 21 IN     A       <IP_ADDRESS>

;; Query time: 70 msec
;; SERVER: <IP_ADDRESS>#53(<IP_ADDRESS>)
;; WHEN: Fri Apr 02 17:04:47 PDT 2021
;; MSG SIZE  rcvd: 376

I have hit this too, which makes LinkedIn inaccessible at times. Sometimes their CDN, static-exp1.licdn.com sometimes resolves through this chain (but not always):
static-exp1.licdn.com.  300     IN      CNAME   2-01-2c3e-003d.cdx.cedexis.net.
2-01-2c3e-003d.cdx.cedexis.net. 300 IN  CNAME   li-prod-static.azureedge.net.
li-prod-static.azureedge.net. 1800 IN   CNAME   li-prod-static.afd.azureedge.net.
li-prod-static.afd.azureedge.net. 30 IN CNAME   star-azureedge-prod.trafficmanager.net.
star-azureedge-prod.trafficmanager.net. 30 IN CNAME dual.t-0009.t-msedge.net.
dual.t-0009.t-msedge.net. 240   IN      CNAME   t-0009.t-msedge.net.
t-0009.t-msedge.net.    60      IN      CNAME   Edge-Prod-WSTr3.ctrl.t-0009.t-msedge.net.
Edge-Prod-WSTr3.ctrl.t-0009.t-msedge.net. 240 IN CNAME edge-prod-wstr3.ctrl.t-0001.trafficmanager.net.
edge-prod-wstr3.ctrl.t-0001.trafficmanager.net. 0 IN CNAME standard.t-0009.t-msedge.net.
standard.t-0009.t-msedge.net. 240 IN    A       <IP_ADDRESS>
standard.t-0009.t-msedge.net. 240 IN    A       <IP_ADDRESS>

Failures I ran into with this for the sake of anyone else confused:

During initial Windows 10 setup you cannot signin with microsoft account. It just spins and then drops back into the local user setup.
Cannot signin with microsoft account once in Windows 10. Window opens and then closes.
Cannot add a local user in an existing Windows 10 system. You just get a screen with a broken ms logo image, 2 back buttons, and a cookie and privacy policy link.

Granted those are Microsoft Windows problems but 8 is arbitrarily low for a cheap way to prevent CNAME loops. (per code). #461 is nice but perhaps the default needs to be bumped up significantly. There's a lot of MS Windows systems and CDNs out there.

I hit this issue too, with Unbound 1.13.1 on OPNsense. The strange thing is, Unbound does sometimes resolve the domain successfully, but querying immediately afterwards fails again. Based on level 5 logs, I believe the difference is that it succeeds when its cache is empty, and fails when it find the CNAME chain in its cache.

Amusingly, this isn't a worldwide problem and might be specific to people around US-West. A person in Italy told me they have no problems resolving the domain, because the CNAME chain they get is much shorter:
;; ANSWER SECTION:
logincdn.msauth.net.    261     IN      CNAME   lgincdn.trafficmanager.net.
lgincdn.trafficmanager.net. 29  IN      CNAME   lgincdnvzeuno.azureedge.net.
lgincdnvzeuno.azureedge.net. 1762 IN    CNAME   lgincdnvzeuno.ec.azureedge.net.
lgincdnvzeuno.ec.azureedge.net. 3562 IN CNAME   cs1227.wpc.alphacdn.net.
cs1227.wpc.alphacdn.net. 3562   IN      A       <IP_ADDRESS>

@Arnavion It certainly would change from place to place, as that is what a CDN is designed for. In the case of LinkedIn, it will even change throughout the day for me, from a 3-name chain to a 8+ name chain. So...there are certain times of the day LinkedIn will fail to work for me. :)

Yes, it's just that on a few other forums I saw people wondering why something as fundamental as logincdn.msauth.net could be broken and only they were noticing it. I know I certainly was confused, until I saw the debug logs + got the Italian user's CNAME chain confirming that that was what the issue was.
It doesn't help that there are also a few red herrings on the internet, because some people claim they got it working by disabling qname-minimisation, but at least for me doing that doesn't make any difference.

Naturally there are other names besides logincdn.msauth.net affected by this issue, and it can cause problems beyond the ken of mortal man. :-) I was trying to do my taxes in TurboTax for Mac and it hit an error trying to connect to my ItsDeductible account to download charitable contribution info. On a hunch I set my MacBook's DNS resolver to <IP_ADDRESS>. Problem went away. I see folks on the Intuit forums complaining about the issue, and of course nobody there has any idea what the cause could be.

Hi,
Thanks for bringing this up!
Since the problem is that the domain in question has a longer CNAME chain than Unbound is willing to accept, we believe the solution is to raise the MAX_RESTART_COUNT value to something higher like 11 which is still small enough. Small values ensure that Unbound will not waste resources trying to resolve potentially harmful queries.
However we would not like to introduce a configuration option for that since this will only solve the problem for users that mess with the value. The proper solution is for domains to check their CNAME chains because different resolvers have different strategies and limits on how to deal with CNAME chains/query restarts.

@gthess Could we raise it to something like 20? I know that sounds high, but it should still resolve pretty fast, most places won't do this, caching should mitigate when it does happen, and it will keep this bug from recurring too soon. :)

I am reluctant to go that "high". For legitimate purposes that should work fine. But since Unbound does not trust CNAME chains and goes and asks for each record itself, the illegitimate/DoS purposes are what we don't want to enable.

@gthess Yeah, I fully understand that. We are currently dealing with this bug because we never thought someone would make a CNAME chain this long. To prevent this from coming up in the future, I would think we'd either need to make it a high number, or make it configurable so it's an easy work around if something "stupid" happens in the future. Right now, there are a LOT Of downstream users that can't (or don't know how) to upgrade because they use unbound in a project such as PFSense. The bug won't be fixed for them until a fix is released...and then PFSense upgrades the package. A configuration option would let users mitigate this in the future.

Can someone provide reasoning why anyone would need such long CNAME chains?  To me this sounds like the mistake was on that side.  Perhaps it's a bit unfortunate that the protocol does not standardize a particular global limit; I agree you don't want a (web) service telling all their users to reconfigure their DNS.

With regard to why this change might be helpful - if you're in a situation (like I was) where you're maintaining a system that uses unbound (like PFSense) and swapping out the package for a custom build isn't particularly easy or well supported then you're forced to use another DNS resolver altogether to get unblocked. One of the reasons why I thought this would be valuable to have as a configuration option is to avoid defaulting everyone into a higher count just to cover the few cases where it could help but give people that have identified the problem a way to work around it.
Part of this discussion has become what it should be increased to and while that might help for the next instance I think that would just be kicking the can down the road. There's still a good reason to have a limit; even without cycles in the resolution it would be possible to create a domain to DOS a resolver by cycling through an absurdly long CNAME chain so having a limit makes sense. But making that limit have the ability to override it in config would have allowed me to be unblocked while waiting for the service to be fixed.
I'd very much like #461 to be adopted; all of the defaults could remain in place to not cause new behavior for existing deployments but the next time I get into this state because someone did an unusual DNS configuration (very easy to accidentally do) and only tested it in a few configurations then I can still unblock myself.

@vcunat Some CDN systems will configure a very long CNAME chain as part of their normal operation. See the LinkedIn example above.

(For anyone who was tracking this issue because of logincdn.msauth.net specifically, that one's chain has become shorter enough to work now.)
logincdn.msauth.net.    263     IN      CNAME   lgincdn.trafficmanager.net.
lgincdn.trafficmanager.net. 29  IN      CNAME   lgincdnmsftuswe2.azureedge.net.
lgincdnmsftuswe2.azureedge.net. 1763 IN CNAME   lgincdnmsftuswe2.afd.azureedge.net.
lgincdnmsftuswe2.afd.azureedge.net. 263 IN CNAME firstparty-azurefd-prod.trafficmanager.net.
firstparty-azurefd-prod.trafficmanager.net. 0 IN CNAME dual.part-0042.t-0009.t-msedge.net.
dual.part-0042.t-0009.t-msedge.net. 203 IN CNAME part-0042.t-0009.t-msedge.net.
part-0042.t-0009.t-msedge.net. 203 IN   A       <IP_ADDRESS>
part-0042.t-0009.t-msedge.net. 203 IN   A       <IP_ADDRESS>

This issue is very annoying, at least make it possible to be changed in configuration rather than hardcoded.
Yes, long chain of CNAME probably is not a good design, but we can't just simply ask Akamai, Microsoft, Apple, other companies or CDN and Cloud companies say hey my software is not compatible with your design, can you change your design?
Chances are the answer will be: No, why others are fine, only you are complaining, and no, we will not change our design for minor users who is having issue with their own software. Just switch to another software that works.
So, please make this a configuration parameter rather than hardcoded.

Hi all,
Thanks for your input.
For now we are bumping the value to 11 from the previous 8 for the next version. This will allow both examples given here (one of them is already fixed in DNS) to work.
Since this is the first time we get something similar regarding CNAME chain length, we are reluctant to jump to a configuration option at this moment.

I'm not sure which one you are referencing as fixed, but for me, the LinkedIn one (static-exp1.licdn.com) will fluctuate between a 3-long chain and an 8-long chain, meaning sometimes I can access LinkedIn and sometimes i can't. :)

I had the same issue today in pfsense 2.5.2 against the microsoft fqdn content.powerapps.com.
My ISP's (Zen Internet) DNS server returned a response with 9 aliases listed (3 of them were internal looking fallback edge server aliases). Google DNS only listed 6 aliases. So I had to repoint Unbound at google DNS with forwarding enabled to fix the issue.
The more I think about it why doesn't unbound just give up on recursively following all aliases when large lists are returned. They extraneous entries don't really need to be resolved as many of the later aliases wouldn't be very useful for public access.


Line Ending changes ignored in diff editor

VSCode Version: 1.28.1 x64
OS Version: Windows 7 Enterprise

Steps to Reproduce:

Make sure you open a folder in VScode that is a git repository with git-scm installed
Open a file with line ending LF
Modify a line in that file (edit, delete, add, does not matter)
Change the line ending to CRLF
Save the file
Go to 'Source Control' and click on the modified file
An editor opens showing the changes in that file between the original and the modified version
Add a commit message and commit the changes

Problem:
In step 7 the editor does not show that the line endings were modified. After step 8, doing a diff on the original file and the new file the line ending changes become visible, but then it is already too late.
This is especially annoying when working on Windows and Unix. Moreover the real diff becomes unreadable, because basically it shows the whole file has been modified.
Additionally, when clicking in step 7 in one of the files, the cursor position at the bottom status line is updated, but the line ending indicator shows only the line ending format of the new file (for both). Not the original one. So it is impossible to see this change.
Either the line endings should be highlighted in the editor or at least the background of the line ending indicator in the status bar should change to red or something that grabs attention.
Does this issue occur when all extensions are disabled?: Yes

Duplicate of #19254


⚠ Bofrost.lu has degraded performance

In ca51a88, Bofrost.lu (https://www.bofrost.lu) experienced degraded performance:

HTTP code: 200
Response time: 1298 ms

Resolved: Bofrost.lu performance has improved in d1a936f after 6 minutes.


Configure SMTP for Mauro

NHS England have provided the details for SMTP to be used in Mauro. They are:

From address<EMAIL_ADDRESS>SMTP host: send.nhs.net
SMTP port: 587
SMTP username<EMAIL_ADDRESS>SMTP password: talk to @pjmonks

NHS England also provided this documentation page which may be helpful: https://support.nhs.net/knowledge-base/high-sending-smtp-solution/
These are configuration variables defined here: https://maurodatamapper.github.io/installing/configuring/mdm-core-config/
The "from" address is a Mauro API property that is configured at runtime:

Login as any administrator to MDM UI
Go to the "Configuration" page from your user menu
In the table of properties, look for email.from.address and edit that to the one above.

This will require the docker containers to be restarted to take affect. The docker logs starting up Tomcat should show either that SMTP was configured correctly, or show an error for why it failed.
To test that emails work:

Go to the main Mauro UI
Click "Log in"
Click on "Forgot password"
Enter your email address and click "Recover password"
This should trigger an email to be sent to you.

On hold: it seems the credentials provided by NHS England are somehow wrong

This is now on hold indefinitely for the foreseeable future. NHS internally need to audit the Mauro solution before they can setup an email service correctly, which will take at least 6 weeks to approach. This is no longer in scope for System C to manage.


[WFLY-4424] Move WSServiceDependenciesProcessor earlier in the chain

This requires https://github.com/wildfly/wildfly-core/pull/572 to be merged and a new release of wfly-core...

Windows Build 1191 is now running using a merge of 4386ceeceb7af70c7252f0635170915ac9dbdd98

Linux with security manager Build 1055 is now running using a merge of 4386ceeceb7af70c7252f0635170915ac9dbdd98

Linux Build 6078 is now running using a merge of 4386ceeceb7af70c7252f0635170915ac9dbdd98

Linux with security manager Build 1055 outcome was FAILURE using a merge of 4386ceeceb7af70c7252f0635170915ac9dbdd98
Summary: Tests passed: 588, ignored: 240; exit code 1 (new) Build time: 0:03:02
Build problems:
Process exited with code 1

Linux Build 6078 outcome was FAILURE using a merge of 4386ceeceb7af70c7252f0635170915ac9dbdd98
Summary: Tests passed: 636, ignored: 192; exit code 1 (new) Build time: 0:04:19
Build problems:
Process exited with code 1

Windows Build 1191 outcome was FAILURE using a merge of 4386ceeceb7af70c7252f0635170915ac9dbdd98
Summary: Tests passed: 636, ignored: 192; exit code 1 (new) Build time: 0:04:19
Build problems:
Process exited with code 1

This will be superseded by https://github.com/wildfly/wildfly/pull/7285


You're forked. Now what?

Just a question about the "go forward" plan.
The parent maintainers appear to have gone totally dark on their repo, are any of the maintainers here in contact with them?
Assuming that any back channel communications are as dead as the repo appears to be, is there any plan to become "official" and start publishing a perhaps differently named gem on RubyGems?
It makes it a harder sell to include this library in an Enterprise project pulling from GitHub rather than RubyGems.
I'm using this gem #in a side project (and am so-far very happy with it) with the hope of bringing this to my employer's project, but really need to know the "go forward" plan.  We can use "unpublished" gems, but company policy is that we have to fill the repo into our corporate account which brings some level of additional maintenance overhead.  I'd much rather be able to pull from RubyGems, where we automatically mirror the gems we use in our own repo.
Thanks, and keep up the good work!

@JeffLuckett thanks for reaching out!

are any of the maintainers here in contact with them?

We are trying, not very successfully though. Honestly there's indeed quite a bit of uncertainty about how to proceed next. We're trying to play nice with everyone, but lately I feel like we should rename the fork and push forward independently.
An alternative way is to use the Github's gem repository and keep the releases outside of RubyGems for a bit... Let's continue the conversation as part of the #1


When refresh the page github star value go to zero

Hey, I tried multiple time but haven't seen this? can you revalidate? could for a moment or related to github service down

Should be fixed now!


fix(json-rpc): potential dead loop from restarting on halt

Okay, so this will definitely fix #620. The problem was due to restarting a connection that threw an unrecoverable error, for example CrashError, this will guarantee to throw again, which is then caught by the halt logic, which then call the start function again -> infinite loop.
This PR attempt to fix that by preventing connection restart when those unrecoverable errors are caught.
I'm sure there are more nuances to this, please feels free to modify or provide more guidances as needed 💪.

Out of scope for this PR, but might be good to implement some sort of exponential back off with maximum retry cap here.

Hi! Closing in favor of #855. Thanks for your contribution Tien!


Input attempt from subprocesses ignored

My program calls out to certain external programs, and they themselves have certain confirmation prompts. I used to use raw shell calls from base, but am trying to port to Shelly. Shelly seems to ignore attempts by the subprogarms to get input:
colin@yumi ~/c/h/aura> stack exec -- sudo aura -Rsu pkgbuilder
checking dependencies...

Packages (5) asp-2-1  pyalpm-0.8.2-2  python-parse-1.8.2-1  python-srcinfo-0.0.8-2  pkgbuilder-4.2.17-1

Total Removed Size:  0.81 MiB

:: Do you want to remove these packages? [Y/n]
aura:
Ran commands:
pacman -R -s -u pkgbuilder
which pacman

Exception: error running: pacman -R -s -u pkgbuilder
exit status: 1
stderr: :: Do you want to remove these packages? [Y/n]

colin@yumi ~/c/h/aura>

and as you can see, the subprogram pacman immediatel exists with 1 instead of being allowed to accept user input.
Thoughts? Thank you.

The answer was to use runHandles and set the various handles to Inherit, as in:
runHandles c args [ InHandle Inherit, OutHandle Inherit, ErrorHandle Inherit ] f


Revisiting motion libraries for component animations

What will this achieve?
After working with motion on components in Carbon for IBM Products, we noticed that trying to animate elements that leave the DOM can be challenging. @matthewgallo has added utilities to help with this but they can quickly become buggy (relying on state and adding handlers in places to undo "onAnimationEnd" handler.)
We should consider Framer motion because it is easier to use and has a ton of features for reduce motion and hooks that the team can benefit from.

Updated the title since we’re also going to be looking into Motion One.

Framer Motion on Bundlephobia
Motion One on Bundlephobia

CSSTransitionGroup — some Carbon components don’t forwardRef so can be tricky to work with.
Motion One allows us to animate anything in the DOM — can use class or ID, but downside is does not allow mount/unmount animations.
Framer Motion is easier for mount/unmount and easy to learn.

We can abstract layers to our own library code to be reused.
Can only target elements if Carbon forward Refs — doesn’t really work well for animating a Carbon component itself.

Title animation for SidePanel has so much logic — a library would make them way simpler to work with.
Open and close is an example where AnimatePresence would help.
Lightweight version of Framer Motion might be viable.


Create LICENSE

I think it makes more sense to archive this repo.

Agreed, but can we merge this PR before putting it in RO mode?


Feat: TEAM SETTINGS

Describe the feature
Implementation of team settings page
Add ScreenShots
Not requied!!
Record

[X] I agree to follow this project's Code of Conduct
[X] I'm a GSSOC'24 contributor
[X] I want to work on this issue

@subhadeeproy3902 assign me


Switch to qunit.

@Arkni: just remember to make this a major version bump :)

I don't think we need to as only a dev-dependency was updated.

Yeah, I was mostly talking about the previous changes.

Ah you're right. I will make sure to not forgot this as I also have some changes locally but didn't had the chance to continue working on them.
Thanks for the reminder :)


Gradle task kspKotlinIosSimulatorArm64 fails in 1.0.0-ALPHA-38

> Task :shared:kspKotlinIosSimulatorArm64 FAILED
Custom actions are attached to task ':shared:kspKotlinIosSimulatorArm64'.
Caching disabled for task ':shared:kspKotlinIosSimulatorArm64' because:
  Build cache is disabled
Task ':shared:kspKotlinIosSimulatorArm64' is not up-to-date because:
  Input property 'excludeOriginalPlatformLibraries$kotlin_gradle_plugin_common' file /Users/saad/.konan/kotlin-native-prebuilt-macos-aarch64-2.1.0/klib/platform/ios_simulator_arm64/org.jetbrains.kotlin.native.platform.CoreFoundation/default has been removed.
  Input property 'excludeOriginalPlatformLibraries$kotlin_gradle_plugin_common' file /Users/saad/.konan/kotlin-native-prebuilt-macos-aarch64-2.1.0/klib/platform/ios_simulator_arm64/org.jetbrains.kotlin.native.platform.CoreFoundation/default/linkdata has been removed.
  Input property 'excludeOriginalPlatformLibraries$kotlin_gradle_plugin_common' file /Users/saad/.konan/kotlin-native-prebuilt-macos-aarch64-2.1.0/klib/platform/ios_simulator_arm64/org.jetbrains.kotlin.native.platform.CoreFoundation/default/linkdata/module has been removed.
file or directory '/Users/saad/app/shared/src/appleMain/kotlin', not found
file or directory '/Users/saad/app/shared/src/nativeMain/kotlin', not found
file or directory '/Users/saad/app/shared/src/iosSimulatorArm64Main/kotlin', not found
file or directory '/Users/saad/app/shared/src/appleMain/kotlin', not found
file or directory '/Users/saad/app/shared/src/nativeMain/kotlin', not found
Can't find env_blacklist file at /Users/saad/.konan/kotlin-native-prebuilt-macos-x86_64-2.1.0/tools/env_blacklist.
error: org.jetbrains.kotlin.cli.utilities.MainKt

FAILURE: Build failed with an exception.

* What went wrong:
Execution failed for task ':shared:kspKotlinIosSimulatorArm64'.
> java.lang.ClassNotFoundException: org.jetbrains.kotlin.cli.utilities.MainKt

I'm new to KMP-NativeCoroutines and am not sure if I'm doing something wrong here - above is the error excerpt from running ./gradlew :shared:embedAndSignAppleFrameworkForXcode --info as a build phase in my iOS app. Any help would be appreciated.
KMP-NativeCoroutines version: 1.0.0-ALPHA-38
KSP version: 2.1.0-1.0.29
MacOS version 15.2 (Sequoia), Apple M2 chip
XCode Version 16.2 (16C5032a)

Sounds like there is an issue with the ~/.konan directory similar to this issue.
Could you try to sync in IntelliJ/Android Studio and/or clearing the konan directory?

I tried that, to no effect.
I did find that I can build iOS applications from Android Studio though, through the KMP plugin from Jetbrains. For some reason, the build from that run configuration succeeds (and the app launches in an iOS simulator) while the XCode build does not.

Hello. I am facing exactly the same issue. Do you have any update on it?


Added a way for a Platform application to customize some text in the …

…about window.
Now avoiding an exception in BreakpointAnnotationProvider
Note: this is my first pull request on something that is active so let me know if I need to change anything.

One functional comment: the new API essentially allows to replace information in the about box by any module in the system. It should not be the case, the about box have at least a mandatory part that describes the platform or distribution (application). Optionally additional information collected from modules.
Your AboutMessageOverride seems as if it should not be registered in default Lookup, but passed through e.g. system property or a file on classpath (among the startup JARs), so it is strictly under the control of the distribution packager.

As I said before, I don't know how to do that or I would have done so.

Just checking, I would do some Class.forName(System.getProperty("SomePropHere"))? If that is the case, where would the best place to put the property then? Maybe in the branding strings?

In some ways I do what @sdedic suggests - ie, system properties in the app.conf file. However, I don't think this can meet all the needs you discussed, which really needs to dynamically change the parameters passed into MessageFormat.  My thought was something along the lines of -
interface NbBundle.ParameterFilter {
    Object[] filter(Class<?> clazz, String resName, Object[] arr);
}

registered in META-INF and called as part of NbBundle.getMessage().  This should allow the sort of override you're talking about for any message - about panel is not the only problem.  A complication may be that I'm not sure NbBundle could use Lookup for this?  And need to be careful of performance concerns.
Anyway, I'd suggest taking this discussion into JIRA?

Submitted to JIRA: https://issues.apache.org/jira/browse/NETBEANS-352

I meant to have that message be 'tab to space', not 'space to tab'

This change was done on the master branch. I moved them to their own branch, so this one will be closed and a new pull request opened with the correct branch.


Add CF CLI into Tanzu asdf plug

At this moment CF CLI asdf plugin available as stand alone https://github.com/mattysweeps/asdf-cf. Last commit is about 4 years ago. Since it is open source I propose to consider combining it into this plugin that brings Tanzu command line experience to the workstation.

Sorry @a-b for the super slow response, I didn't have alerts set on this repo.
The CF CLi is already in ASDF via https://github.com/mattysweeps/asdf-cf. I've just checked and am able to download version 8.7.7.


Add package-lock.json for the client and the server

Currently yarn install in a freshly cloned repo leaves the working tree dirty.
Those lockfiles should either be added to gitignore or, preferably, be committed to the repo (as this PR does).

@CvX you talk about yarn install but add the package-lock.json files in here? Instead I would add a .npmrc file in with the contents package-lock=false.

You probably meant npm install as the yarn lock files are published. So add the .npmrc file instead so we only work with one lock file system, being yarn.

Oh, I do mean yarn! 🙂
The postinstall script defined in package.json runs npm install in client and server directories, which creates package-lock.json files.
In what situations are client/yarn.lock and server/yarn.lock created/updated? Other than manually?

Seems like the real move is for the postinstall script to not use npm.

No longer relevant with the new structure.


[Bug] Project detail buttons don't reorder

Describe the bug
On the accounts page, under projects, reordering the projects by one of the columns doesn't rearrange the blue 'Detail' buttons. Therefore, each Detail button no longer corresponds to the associated project in that row.
To Reproduce
Steps to reproduce the behavior:

Go to Accounts, then Projects tab.
Select the 'Detail' button for the first entry. The displayed entry will be correct.
Close this pop up.
Click on 'Project Name' column to sort them.
Select the 'Detail' button for the first entry. The displayed entry will be incorrect, and show the one from before.

Expected behavior
The correct project should pop up corresponding to the project in that row position.
Screenshots

(Selecting the button at the top arrow brings up the project below it for example)
Versions (please complete the following information):
Version Beta 5.2.2

Thank you, @DA-Osborne for identifying this issue. We have added this to our backlog and plan to begin working on a fix.


Improve naming of various "envelope" and "message" types.

Fixes #131
This PR changes the names of various "message container" types to give them distinct names.
The result is a structure as follows:

The only type named Message is dogma.Message.
envelope.Envelope is still the in-memory representation envelope, it contains a Message
envelopespec.Envelope is still the on-disk/network representation of an envelope
Parcel is used in various packages to refer to an envelope (of either kind) with additional meta-data that is specific to that package
Pair is used in various packagers as a container for a Parcel and an envelope.Envelope - keeping the marshaled/unmarshaled forms together to avoid excess [un]marshaling.

In a separate branch/PR I'm going to experiment with removing the envelope.Envelope type altogether.

See #152 instead.


rtlmp fails to find solution

Describe the bug
https://github.com/The-OpenROAD-Project/OpenROAD/pull/6391 seems to triggering on designs that used to pass without any issues.
With this PR I'm getting:
[ERROR MPL-0037] Couldn't break flat cluster (root)_glue_logic_0_0 with PAR. The solution is fully unbalanced.
Error: sc_macro_placement.tcl, 115 MPL-0037

Expected Behavior
Macros placed without errors
Environment
OpenROAD v2.0-17721-g5f20e5980
Features included (+) or not (-): +GPU +GUI +Python
This program is licensed under the BSD-3 license. See the LICENSE file for details.
Components of this program may be licensed under more restrictive licenses which must be honored.

To Reproduce
sc_issue_ethmac_job0_skywater130_sky130hd_floorplan.macro_placement0_20241220-034710.tar.gz
tar xvf sc_issue_ethmac_job0_skywater130_sky130hd_floorplan.macro_placement0_20241220-034710.tar.gz
cd sc_issue_ethmac_job0_skywater130_sky130hd_floorplan.macro_placement0_20241220-034710
./run.sh

Relevant log output
Floorplan Outline: (0, 0) (1142.89, 1142.89),  Core Outline: (1.38, 2.72) (1141.72, 1139.68)
	Number of std cell instances: 8017
	Area of std cell instances: 99596.03
	Number of macros: 1
	Area of macros: 419462.72
	Halo width: 40.00
	Halo height: 40.00
	Area of macros with halos: 541366.31
	Area of std cell instances + Area of macros: 519058.75
	Core area: 1296521.00
	Design Utilization: 0.40
	Core Utilization: 0.11
	Manufacturing Grid: 5

[ERROR MPL-0037] Couldn't break flat cluster (root)_glue_logic_0_0 with PAR. The solution is fully unbalanced.
Error: sc_macro_placement.tcl, 115 MPL-0037

Screenshots
No response
Additional Context
No response

@gadfort This is surprising.
The changes in #6391 add an error to prevent MPL2 to move on when PAR does a terrible job partitioning a cluster.
We only throw this error when we give a certain hypergraph input to the partitioner (that represents a certain cluster A made of X macros and Y std cells) and the partitioner outputs an empty partition along with another partition identical to A.
This may trigger an infinite recursion like #6083 or we may get lucky and in the subsequent split for one of the output partitions (we'll give the same problem to the partitioner again) some random seed variation may generate a slightly different output and it will go on. The latter is what seems to be happening here. Here are some debugPrint messages without the new changes to show where the problem is.
[DEBUG MPL-multilevel_autoclustering] Breaking Cluster: root
[DEBUG MPL-multilevel_autoclustering] Setting Cluster Metrics for (root)_glue_logic: Num Macros: 1 Num Std Cells: 8017
[DEBUG MPL-multilevel_autoclustering] Breaking flat cluster (root)_glue_logic with TritonPart
[DEBUG MPL-multilevel_autoclustering] Setting Cluster Metrics for (root)_glue_logic_0: Num Macros: 1 Num Std Cells: 6414
[DEBUG MPL-multilevel_autoclustering] Setting Cluster Metrics for (root)_glue_logic_1: Num Macros: 0 Num Std Cells: 1603
[DEBUG MPL-multilevel_autoclustering] Breaking flat cluster (root)_glue_logic_0 with TritonPart
[DEBUG MPL-multilevel_autoclustering] Setting Cluster Metrics for (root)_glue_logic_0_0: Num Macros: 1 Num Std Cells: 6240
[DEBUG MPL-multilevel_autoclustering] Setting Cluster Metrics for (root)_glue_logic_0_1: Num Macros: 0 Num Std Cells: 174
[DEBUG MPL-multilevel_autoclustering] Breaking flat cluster (root)_glue_logic_0_0 with TritonPart
/* Here's the problem */
[DEBUG MPL-multilevel_autoclustering] Setting Cluster Metrics for (root)_glue_logic_0_0_0: Num Macros: 1 Num Std Cells: 6240
[DEBUG MPL-multilevel_autoclustering] Setting Cluster Metrics for (root)_glue_logic_0_0_1: Num Macros: 0 Num Std Cells: 0
[DEBUG MPL-multilevel_autoclustering] Breaking flat cluster (root)_glue_logic_0_0_0 with TritonPart
[DEBUG MPL-multilevel_autoclustering] Setting Cluster Metrics for (root)_glue_logic_0_0_0_0: Num Macros: 1 Num Std Cells: 6010
[DEBUG MPL-multilevel_autoclustering] Setting Cluster Metrics for (root)_glue_logic_0_0_0_1: Num Macros: 0 Num Std Cells: 230
[DEBUG MPL-multilevel_autoclustering] Breaking flat cluster (root)_glue_logic_0_0_0_0 with TritonPart
[DEBUG MPL-multilevel_autoclustering] Setting Cluster Metrics for (root)_glue_logic_0_0_0_0_0: Num Macros: 1 Num Std Cells: 5022
[DEBUG MPL-multilevel_autoclustering] Setting Cluster Metrics for (root)_glue_logic_0_0_0_0_1: Num Macros: 0 Num Std Cells: 988

I'm working on a change to apply a relaxation to the constraints we give to PAR if outputs such a bad result.


Create Terraform configuration for AKS

Create Terraform configuration for Jenkins X cluster on Azure.
See also https://github.com/jenkins-x/terraform-google-jx and https://github.com/jenkins-x/terraform-aws-eks-jx.
The equivalent resources should be created for Azure.

Trickest part might be the preparation needed for the Vault operator. There is some information here https://github.com/banzaicloud/bank-vaults/blob/master/operator/deploy/cr-azure.yaml. Maybe it might make sense to address Vault in a separate issue?
Creating cluster, buckets and workload identity equivalent should be straight forward by mirroring the repositories mentioned above. Let's try to align structurally as well in supported variables and output.

Let's try to clarify what we are trying to achieve.
We want to get into a state where we can create an AKS based cluster and then be able to run jx boot against it. Initially, we will aim for local secrets (no Vault; Vault setup we'll address in issue #3 and will require changes to jx itself) and no external DNS (we will use the default nip.io for DNS resolution).
To achieve this we will need a minimum set of resources which are described below. The structure of the terraform module should be the same as for terraform-google-jx and terraform-aws-eks-jx. There should be a main.tf, together with variables.tf and outputs.tf. main.tf defines the providers w/ versions and then calls the submodules backup,  cluster,  dns, vault each of which deals with a specific aspect of the required infrastructure. As part of this issue, we are focusing on cluster and the resources which should be created there.
Primarily we need an AKS cluster. We should be able to provision it with as little input as possible defaulting to sensible values were possible. In the case of GKE the minimal required configuration is just the Google Cloud project id. Everything else gets defaulted. There are many configuration options for the cluster, but initially we expose for the user cluster_name, node_machine_type, min_node_count, max_node_count (assuming these things are configurable with AKS). If no cluster name is specified we default to tf--. See https://github.com/jenkins-x/terraform-google-jx/blob/master/main.tf#L59.
It probably is also a good id to create a random id at the top-level Terraform file and pass this id into the submodules. Wherever resource names need a unique identifier to make names unique we can use this generated id. See https://github.com/jenkins-x/terraform-google-jx/blob/master/main.tf#L55.
Next, we need several storage buckets. Creation of these buckets should be conditional based on the variables enable_log_storage, enable_report_storage and enable_repository_storage. The names of the buckets should be logs-${var.cluster_name}-${var.cluster_id}, reports-${var.cluster_name}-${var.cluster_id} and repository-${var.cluster_name}-${var.cluster_id}. See https://github.com/jenkins-x/terraform-google-jx/blob/master/modules/cluster/storage.tf
We also need a set of Kubernetes service accounts - kaniko_sa, tekton_sa, build_controller_sa. The idea is that these service accounts will use Managed Identity configuration to restrict which Cloud API resources they can create/modify. Which exact permissions each service account we need to investigate. Once the infrastructure is in place we will need to experiment with various permissions. At the very least all of these three service accounts need to be able to access the bucket API. They need to be able to read/write data into the created buckets (see above).

For testing the created infrastructure we are using shellspec in terraform-google-jx and terraform-aws-eks-jx. Preferably we do the same in terraform-azurerm-jx. The aim is to verify that the buckets exist, that the Kubernetes service accounts exist and they are limited in their permissions. How to do that is provider specific. In GKE we create a test Pod using a given service account. We then run a command inside the Pod (gcloud auth list) to ensure that Workload Identity is properly setup - https://github.com/jenkins-x/terraform-google-jx/blob/master/spec/k8s_spec.sh#L56. Not sure how to something equivalent for Azure.

It should always be possible to do a clean terraform destroy. Often created buckets cannot be deleted once they contain content. We use a force_destroy variable to allow for full destruction of resources. The default is false to avoid accidental data loss, but it can be set to true for test or development cluster allowing a clean destruction of the cluster via Terraform.

/assign @helayoty

@deanesmith I can see the implementation for this issue has been merged. can we consider it as closed? any work still needed here?

Sounds good

We have a basic terraform module now so closing this issue


docs: Add spanish translation

Hi there well, here I translated the web page into Spanish.
Maybe we can at least start translating the manual, what do you think?
Thanks!

Wow, amazing, that was fast! Thank you so much, this will help a lot of people!
My only concern about starting to translate the manual, is that I've been wanting to do some restructuring of the chapters, but I've been saying that for a long time now, and haven't gotten around to it yet anyways. So maybe it's just better to start out on the translation, if you are up for it. I just don't want to waste anybodies work, if I then start rewriting stuff in a couple of months, and the translation needs to be updated.
But in either case, it's better that there is a Spanish version than none at all. So if you're okay with all of that, I'd very much appreciate the help on the translation!

I'll push this to the live website as soon as I can.

Great!
So, I think that this are the files of the manual, right?
We also need some kind of menu in the index.html to be able to go through the (hopefully) different languages of the manual, what do u think?

Yes, the rst files are the sources. I'm actually not sure how to best go about the language switching. The build system uses Sphinx (which internally uses LaTeX) to compile the manual to various output formats, such as PDF, EPUB and HTML. Maybe Sphinx itself has some support functions for multilingual setups build into it. If it does, I think it is worth using that.


Suggestion: Support for command names from an array

example,
vorpal.command(CommandString)//etc
where CommandString is a string, or the result of json.stringifying the contents of a json file. Would be really useful.

vorpal.command already takes a string as argument. What would be the content of your json file ?

something like,
{  "command":"edit" }
the .json file would contain all the possible commands, but only require a single command code block, if you know what I mean..

I meant an array, apologies.

@ozzie1998 you could do similar tothis using .alias()
vorpal
  .command('test', 'make it work')
  .alias(['nano', 'mkdir', 'wget'])
  .action(function (args) {
    console.log('it works')
  })

On Sat, 25 Feb 2017 at 8:12 am, Artem Kostiuk<EMAIL_ADDRESS>wrote:

@ozzie1998 https://github.com/ozzie1998 you could do similar tothis
using .alias()
https://github.com/dthree/vorpal/wiki/api-|-vorpal.command#commandaliasname-names
vorpal
.command('test', 'make it work')
.alias(['nano', 'mkdir', 'wget'])
.action(function (args) {
console.log('it works')
})
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/dthree/vorpal/issues/214#issuecomment-282468852, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AGptNTir3zg2t5nwUz69c0Lyz8UtjwtYks5rf-JSgaJpZM4MKPMy
.

This is for an app I'm working on where the user can provide their own
commands, that would not work.

I guess I'm out of your app context, but aliases seem to fit your need.
When you need to use single string for command - just pass it to
vorpal.command(). On the other hand, when you want to use an array, you
can put its first element in vorpal.command() and rest of elements into
alias().
25 февр. 2017 г. 11:21 AM пользователь "ozzie1998"<EMAIL_ADDRESS>написал:

On Sat, 25 Feb 2017 at 8:12 am, Artem Kostiuk<EMAIL_ADDRESS>wrote:

@ozzie1998 https://github.com/ozzie1998 you could do similar tothis
using .alias()
<https://github.com/dthree/vorpal/wiki/api-|-vorpal.
command#commandaliasname-names>
vorpal
.command('test', 'make it work')
.alias(['nano', 'mkdir', 'wget'])
.action(function (args) {
console.log('it works')
})
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/dthree/vorpal/issues/214#issuecomment-282468852,
or mute
the thread
<https://github.com/notifications/unsubscribe-auth/
AGptNTir3zg2t5nwUz69c0Lyz8UtjwtYks5rf-JSgaJpZM4MKPMy>
.

This is for an app I'm working on where the user can provide their own
commands, that would not work.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/dthree/vorpal/issues/214#issuecomment-282472095, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AB6D9_Q2o1t0KXylys2GPC6yiapCuqnkks5rf_KBgaJpZM4MKPMy
.

@ozzie1998 This won't work as commands support patterns for arguments, which in turn would make no sense for other items in the array. Use alias instead.

The commands need to be custom set,  alias would not allow that. added or
deleted.
On Thu, May 25, 2017 at 7:55 AM, Miles Johnson<EMAIL_ADDRESS>wrote:

Closed #214 https://github.com/dthree/vorpal/issues/214.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/dthree/vorpal/issues/214#event-1096832892, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AGptNQ6wZH4QEfvZFvc_jSRYEeQLrbMGks5r9SXVgaJpZM4MKPMy
.

@ozzie1998 That doesn't make much sense, as calling command or alias would happen in the same scope. Do you have an example?


[FEAT]: Configure arneis.readthedocs.io

Description
I have been trying ReadTheDocs.
I followed the tutorial and created a sample site https://b-arol-o-test-readthedocs.readthedocs.io/en/latest/ based on the contents of https://github.com/B-AROL-O/test-readthedocs
I think it would be useful to structure the contents in https://github.com/B-AROL-O/ARNEIS so that the documentation can automagically be rendered under https://arneis.readthedocs.io/en/latest/
Additional Information
No response

Can we use markdown as builder ? So that every document already published is added automatically without needed us to rewrite it.
Otherwise we will need to change all the documents to *.rst and use those as standard for the future.

Can we use markdown as builder ?

Maybe - https://www.sphinx-doc.org/en/master/contents.html has all the answers to this question and others.
More specifically: https://www.sphinx-doc.org/en/master/usage/markdown.html
As you can see I have planned resolution of this task by dev-cw03 but so far there are no assignees...
PR accepted 😉

For starting we could take inspiration from https://github.com/luxonis/depthai-docs-website

See also https://www.ericholscher.com/blog/2016/jul/1/sphinx-and-rtd-for-writers.html

and https://www.ericholscher.com/blog/2016/jul/1/sphinx-and-rtd-for-writers.html

This link doesn't seems to work.

and https://www.ericholscher.com/blog/2016/jul/1/sphinx-and-rtd-for-writers.html

This link doesn't seems to work.

Please try again, I added the link doing copy-and-paste on my mobile and I probably added some garbage in the URL

By the way: @gteti I just assigned myself to this task, so now you must take another one otherwise you will have nothing to do for this week ;-)

By the way: @gteti I just assigned myself to this task, so now you must take another one otherwise you will have nothing to do for this week ;-)

Don't worry. I've assigned myself a long term task. See https://github.com/B-AROL-O/opencv-spatial-ai-contest/issues/132

Initial configuration merged to main,
Let's track improvements in related PR TODO


All batch actions should be confirmed

As there is no support for "undo" I believe all actions (not just deletion) that operate on all tasks should be a subject for confirmation if ask_confirmations is set to Yes (overwriting status on all tasks at once by accident can be really frustrating).

Makes sense. Just added this functionality in the recent commit.


Remove 'qualify' page from form

This PR removes the 'how do you qualify' page from the form.
ref #94

Form page removed and approved. - CLOSED


Install instructions say to install "sails-auth"

$ npm install sails-permissions sails-auth --save
sails-permissions install sails-auth version ^2.0.  So why does it need to be installed again?

ok,  well I don't know why, but you do need to install it, and you need to make sure it is version 2 or better.  So do this
$ npm install sails-permissions sails-auth@^2 --save
This also fixes #180

So why does it need to be installed again?

sails-auth makes sure the user is logged in with a valid account ("authenticated"). Then, sails-permissions checks whether the request is authorized, given the particular Roles the user is granted. Both of these steps are necessary.

Ok got it. So the issue here was that different versions where installed?


Table list command. Solves #3653

Table list command. Solves #3653

How to test
Create a new Azure AD App registration that you can use with the CLI for Microsoft. Provide this app the at least:

Azure Service Management - user_impersonation (to access Power Apps API)
Dynamics CRM - user_impersonation (to access Dynamics)
SharePoint - AllSites.FullControl to read tenant admin site

Use this new App Registration to sign in to the CLI m365 login --appId yourAppId --tenant yourTenantId
Then you should be able to run this command :)
⚠ Alert
This can only go live with support from @waldekmastykarz  as he can change our App Registration to include the Dynamics CRM - user_impersonation permission that is currently not yet available in the CLI App Registration

@pnp/cli-for-microsoft-365-maintainers Can I get a few insights before we merge this PR, the current approach is a method in a new Dynamics Base class. Check one do we feel this should be a method like this, or should we go more for AppCommand.ts base where we took a different approach? Or do I need to take a different approach at all with testing? Let me know what you think!

@appieschot we should consider moving the Dynamics instance URL retrieval to a util (we don't have one for PP so we'd create it). That way the method will be available to all the code in the CLI instead of only Dataverse commands.

Hi @appieschot, the build is failing since your latest changes. Could you please look into that?

@martinlingstuyl and here I was promised that as maintainer we would fix the barrel fix that was implemented by Waldek ;-). But I will give it a shot

Mmm, guess my rebase game is not that strong as I now have all commits in main in this one as well. @martinlingstuyl would you mind a short call to discuss how to fix this ;)? Build is fixed but I am wondering if a new PR is quicker compared to trying to cherry pick the right commits

Mmm, guess my rebase game is not that strong as I now have all commits in main in this one as well. @martinlingstuyl would you mind a short call to discuss how to fix this ;)? Build is fixed but I am wondering if a new PR is quicker compared to trying to cherry pick the right commits

Are you live on teams?

Dynamics permissions added to our AAD app reg @appieschot

All eyes on @martinlingstuyl to make sure we tackle / agree on the last comment 🚀

You mean the try catch in the command class. That's correct indeed. In that case it's fine as it is! I'll approve it and merge it Monday.

⚡Merged manually, thank you 🤙


Playing the Cash Game REQUIRES Bot Farmining

How else are you going to reach 1e+16? And when you do farm, a certain jacob complains the farming is excessive and rate limits it/ban/quiets it. How about we just eliminate farming altogether and create a new mechanic for gaining cash that everyone can/will do without being banned/quieted?

I'm working on making farming so excessively not required. Probably we need some more items between some of the huge gaps around company / country / world. For now, this issue just seems like you are annoyed about your ban :P.

This was fixed on ebd9ea454601b30ff582b4f20399ba643245715c . ~~@jztech101 please edit your issue to remove your ban compliants. ~~


LocalAi integration not working  with k8sgpt-operator

Checklist

[X] I've searched for similar issues and couldn't find anything matching
[X] I've included steps to reproduce the behavior

Affected Components

[ ] K8sGPT (CLI)
[X] K8sGPT Operator

K8sGPT Version
v.0.3.8
Kubernetes Version
1.24
Host OS and its Version
linux
Steps to reproduce
1.deploy local-ai chart with the default configuration as detailed here: https://github.com/go-skynet/helm-charts - latest version V2.1.0
2. deploy k8sgpt-operator helm chart - latest version V0.0.21
3. apply k8sgpt-config.yaml with localai base url configured to port 80:
apiVersion: core.k8sgpt.ai/v1alpha1
kind: K8sGPT
metadata:
name: k8sgpt-local-ai
namespace: default
spec:
ai:
enabled: true
model: ggml-gpt4all-j
backend: localai
baseUrl: http://local-ai.local-ai.svc.cluster.local/v1
noCache: false
version: v0.3.8
Expected behaviour
integration should work as it is working with azureopen ai
Actual behaviour
No response
Additional Information
No response

Hey @harshsing2891 can you share some logs whilst also making sure that the local's ai endpoint is reachable from k8sgpt's pod.


Power BI GitHub App - unable to use personal GitHub repo

Hi,
I'm trying to visualise my GitHub commits and issues using Power BI.
Following the steps on
https://docs.microsoft.com/en-us/power-bi/connect-data/service-connect-to-github#connect-to-data-sources
I have installed the GitHub app ok to Power BI (free & now on 30 day trial of Pro)  - the dashboard and reports work fine on the default Power BI repo that's connected to the App.
When I change to my personal account and private repo I get the following error - data refresh failed:

In the set up I am using my GitHub owner/handle and the full URL to the private repo - changing full URL to just repo name also does not work
The Outh authentication works and the app is in GitHub OK

Am I missing something here - are private repos supported?

Strange - got an email saying refresh failed last night
So changed repo back from full URL to just repo name and it now works!
GitHub has failed to refresh.
Failure details: Web.Contents failed to get contents from 'https://api.github.com/repos/futuredesignUK/https://github.com/futuredesignUK/NAME-OF-REPO' (404): Not Found


Release liboxen on crates.io

Would you be so kind to create a crate for this and publish it on crates.io?

Yes, added to backlog and will let you know when it's up there!

Fwiw for our crates we're quite happy with cargo-release to help do the releases, really streamlines some things

Sorry this took so long! But it is now released on crates.io.
https://crates.io/crates/liboxen
Also working on a python wrapper that you can pip install as well if that is better in your workflow.
https://pypi.org/project/oxenai/


AutoPlay in list view

Hello, I want to do AutoPlay in Flatlist but every videos are playing and I need to play only 1 video and the previous one should be auto paused. What will be the best behaviour to achieve this. The same as in Facebook
Here is the configuration of my YouTube component
   <YouTube
                                            videoId="nQ7mhC5_Rys" // The YouTube video ID
                                            play={false} // control playback of video with true/false
                                            fullscreen={false} // control whether the video should play in fullscreen or inline
                                            loop = {false} // control whether the video should loop when ended
                                            showFullscreenButton = {false}
                                            showinfo = {false}
                                            controls = {0}
                                            rel = {false}
                                            onReady={e => this.setState({ isReady: true })}
                                            onChangeState={e => this.setState({ status: e.state })}
                                            onChangeQuality={e => this.setState({ quality: e.quality })}
                                            onError={e => this.setState({ error: e.error })}
                                            style={{ alignSelf: 'stretch',width: '100%',
                                                 }}
                                        />

@tapannathvani You need to somehow control the different players state in you App code


Error: Failed to download Helm from location https://get.helm.sh/helm-v1.2.1-linux-amd64.zip

What happened?
Run<EMAIL_ADDRESS>Downloading v1.2.1
  Error: Error: Failed to download Helm from location https://get.helm.sh/helm-v1.2.1-linux-amd[6](https://github.com/coder/code-server/actions/runs/3316135595/jobs/5478016380#step:4:7)4.zip

Version

[X] I am using the latest version

Runner
Ubuntu 20.04.5
Relevant log output
##[debug]Evaluating condition for step: 'Install helm'
##[debug]Evaluating: (success() && (steps.changed-files.outputs.any_changed == 'true'))
##[debug]Evaluating And:
##[debug]..Evaluating success:
##[debug]..=> true
##[debug]..Evaluating Equal:
##[debug]....Evaluating Index:
##[debug]......Evaluating Index:
##[debug]........Evaluating Index:
##[debug]..........Evaluating steps:
##[debug]..........=> Object
##[debug]..........Evaluating String:
##[debug]..........=> 'changed-files'
##[debug]........=> Object
##[debug]........Evaluating String:
##[debug]........=> 'outputs'
##[debug]......=> Object
##[debug]......Evaluating String:
##[debug]......=> 'any_changed'
##[debug]....=> 'true'
##[debug]....Evaluating String:
##[debug]....=> 'true'
##[debug]..=> true
##[debug]=> true
##[debug]Expanded: (true && ('true' == 'true'))
##[debug]Result: true
##[debug]Starting: Install helm
##[debug]Loading inputs
##[debug]Evaluating: secrets.GITHUB_TOKEN
##[debug]Evaluating Index:
##[debug]..Evaluating secrets:
##[debug]..=> Object
##[debug]..Evaluating String:
##[debug]..=> 'GITHUB_TOKEN'
##[debug]=> '***'
##[debug]Result: '***'
##[debug]Loading env
Run<EMAIL_ADDRESS>
::group::Downloading v1.[2](https://github.com/coder/code-server/actions/runs/3316135595/jobs/5478071781#step:4:2).1
Downloading v1.2.1
  ##[debug]isExplicit: 1.2.1
  ##[debug]explicit? true
  ##[debug]checking cache: /opt/hostedtoolcache/helm/1.2.1/x64
  ##[debug]not found
  ##[debug]Downloading https://get.helm.sh/helm-v1.2.1-linux-amd64.zip
  ##[debug]Downloading /home/runner/work/_temp/7[3](https://github.com/coder/code-server/actions/runs/3316135595/jobs/5478071781#step:4:3)8081a7-2c00-[4](https://github.com/coder/code-server/actions/runs/3316135595/jobs/5478071781#step:4:4)e8b-8118-a9822f6644a[5](https://github.com/coder/code-server/actions/runs/3316135595/jobs/5478071781#step:4:5)
  ##[debug]Failed to download from "https://get.helm.sh/helm-v1.2.1-linux-amd[6](https://github.com/coder/code-server/actions/runs/3316135595/jobs/5478071781#step:4:6)4.zip". Code(404) Message(Not Found)
  Error: Error: Failed to download Helm from location https://get.helm.sh/helm-v1.2.1-linux-amd64.zip
  ##[debug]Node Action run completed with exit code 1
  ##[debug]Finishing: Install helm

https://github.com/coder/code-server/actions/runs/3316135595/jobs/5478071781

facing the same issue

Faced the same issue but replaced azure/setup-helm  with the below code for now
      - name: Install Helm
        run: |
          curl https://raw.githubusercontent.com/helm/helm/main/scripts/get-helm-3 | bash

got same problem, the download link was updated or something ?

Getting same problem

They relied on the default ordering of the GH query and that probably got changed...
Changing the query used here and loosing the reverse (why do that if the API can do it for you?):
https://github.com/Azure/setup-helm/blob/b70d33f56dce95f7970896b826403d7c1bbe8a22/src/run.ts#L58-L73
query {
  repository(name: "helm", owner: "helm") {
    releases(first: 10, orderBy: {field: CREATED_AT, direction: DESC}) {
      nodes {
        tagName
      }
    }
  }
}

Maybe I find time to PR this, but if somebody wants to do it be my guest!

But even when an order is added some computation of the latest need to be done. The latest could be some patch for previous major/minor.

@marcindabrowski / @vdhpieter we might want to use the isLatest attribute from the GitHub API for this. See - graphql/reference/objects#release.
Related: recent announcement that the meaning of the isLatest attribute would change a bit.

#100 will be merged in and released shortly. That fixes this issue. Thanks everyone for the help!

The linked PR was merged but this still isn’t resolved, are we waiting for anything else?

@ImRodry the PR (https://github.com/Azure/setup-helm/pull/100) is not yet merged

Ah ok was looking at https://github.com/clusterpedia-io/clusterpedia-helm/pull/15
Do you have an ETA for when that will be merged?

This has been released in v3.4 (and v3). Please upgrade to one of these versions. Thanks!


[BitwiseCopyable] Don't derive on @unchecked field

Allow for a type to conform to BitwiseCopyable but not in such a way that aggregates containing an instance will derive conformance automatically.  Useful for types which don't involve reference counting but nonetheless can't simply be copied bit-by-bit.

@swift-ci please test

@swift-ci please test

@swift-ci please test

@swift-ci please test


Error 413 when exporting model

Hi,
I'm trying to make model for image classification with 50 classes.
Each class has 10 images.
I tried to export the model into Quantized model for tensorflow lite, but I got this error message Error: MetadataBrowserHTTPRequest.save() failed due to HTTP response status 413.

Is there any info about the limitation for the number of classes in teachable machine?
How can I solve the issue?

I'm having the same issue :(

I am getting the same error

Any update ?

I have the same issue, can anyone guide?

Same problem, any update ?

We made a big update to the converter a few weeks back that solved a lot of the other converter issues reported here on GitHub. Can you confirm what converter is giving you issues, if it's happening consistently, and if it's the same error in the console as reported above?

I am facing the same problem as shown in  question image, even I try today's morning

I use TensorFlow Lite Quantized and it's happening every time I use it and it's the same error in the console as reported above but when I use other converter it's works but I need Quantized

Are you using images you have uploaded manually, or from webcam?
And, any chance you can share a tm file with me here that you see the issue with. I havent been able to determine if its an issue with the way the data is stored, or if its a backend issue.

Hi !
I have put my tm file on my Google Drive you can find it here :
https://drive.google.com/file/d/12PO4kh5qdMZnni2wyBCbUy7e8omim-sT/view?usp=sharing

Any update, I can't add more than 11 classes ?

Hi,
I have the same issue.... while it was working fine in the past.
I'm trying to export a quantized model for tflite, but it gives me the same issue!
Any updates?


Ember 2.1 initializer errors

Looks like more stuff has changed which causes this to break again. Specifically, instance.container() is undefined.
My work around for now is something like this which is ugly. My feeling is that this initializer needs to be an instance one? Not sure though, but this should be relevant: http://emberjs.com/deprecations/v2.x/#toc_initializer-arity
export function initialize(instance) {
  let container = this.container;

  if (!container && instance.container) {
    container = instance.container();
  } else {
    container = instance.__container__;
  }

  let config = container.lookupFactory('config:environment');

  if (!config.mapbox || !config.mapbox.accessToken) {
    console.error('Please specify your mapbox.accessToken in your config.');
    return;
  }

  L.mapbox.accessToken = config.mapbox.accessToken;
}

export default {
  name: 'mapbox-access-code',
  initialize: initialize,
};

Thanks for raising @Soliah, related to #13. I'll be getting this fixed shortly.

Fixed by #13


Switch to eth_estimateGas for all full clients

Codecov Report

Merging #382 into master will decrease coverage by <.01%.
The diff coverage is 0%.

@@            Coverage Diff             @@
##           master     #382      +/-   ##
==========================================
- Coverage   17.59%   17.58%   -0.01%
==========================================
  Files         187      187
  Lines        2905     2906       +1
  Branches      498      498
==========================================
  Hits          511      511
- Misses       2013     2014       +1
  Partials      381      381

Impacted Files
Coverage Δ

src/store/vault/tokens/tokenActions.js
4.87% <ø> (ø)
:arrow_up:

src/components/tx/CreateTx/create.js
0% <0%> (ø)
:arrow_up:

src/store/network/networkActions.js
0% <0%> (ø)
:arrow_up:

Continue to review full report at Codecov.

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


wrap-qt-apps-hooks leads to very slow wrapper scripts

Describe the bug
Wrapper scripts for plasma apps such as krunner or plasma-emojier are slow. For example, executing just the wrapper for plasma-emojier takes 100ms on my fast machine.
The reason for slowness is that the wrapper itself repeatedly adds various paths:
$ rg -F "QT_PLUGIN_PATH='" $(which plasma-emojier) | wc -l
44

This leads to quadratic behavior. Profiling with perf shows that the bulk of the time is spend in string manipulation routines.
The relevant logic is here:
https://github.com/NixOS/nixpkgs/blob/07ab967e1c3e1e8a4f2fb08fcff1fecf585ebe69/pkgs/development/libraries/qt-6/hooks/wrap-qt-apps-hook.sh#L43
Essentially, we are calling
wrapProgram program \
  --prefix QT_PLUGIN_PATH ...
  --prefix QT_PLUGIN_PATH ...
  --prefix QT_PLUGIN_PATH ...

and makeWrapper then faithfully translates each --prefix, instead of pre-computing thee concatenation.
Steps To Reproduce
# Strip the `exec` line
$ <$(which plasma-emojier) head -n -1 > wrapper.sh && chmod a+x ./wrapper.sh

# Time the execution of the wrapper itself
$ time ./wrapper.sh
real	0m0.117s  😱
user	0m0.115s
sys	0m0.002s

Expected behavior
Wrapper itself shouldn't take more than a handful of milliseconds. It seems achievable if I manually replace incremental quadratic addition to environmental variables with setting them once.
Not sure what's the best fix here, some options I see are:

fix wrap-qt-apps-hook to pre-concatenate variables and pass just a single --prefix argument per variable to wrapProgram
make wrapProgram itself smarter and teach it to collate identical prefixes
fix wrap-qt-apps-hook to use --set rather then --prefix

As I don't know bash, I don't plan on working on the fix myself
Notify maintainers
cc @ttuegel
Metadata
Please run nix-shell -p nix-info --run "nix-info -m" and paste the result.
[matklad@Ishmael:~/tmp]$ nix-shell -p nix-info --run "nix-info -m"
 - system: `"x86_64-linux"`
 - host os: `Linux 6.2.10, NixOS, 23.05 (Stoat), 23.05.20230411.3fb8eed`
 - multi-user?: `yes`
 - sandbox: `yes`
 - version: `nix-env (Nix) 2.13.3`
 - channels(matklad): `""`
 - nixpkgs: `/nix/store/k9vs66va04886gl91m6lpcwaymqiihqx-9lkm36ac7hfl7f9h2z9dj00mdh30mjhi-source`

This issue has been mentioned on NixOS Discourse. There might be relevant details there:
https://discourse.nixos.org/t/plasma-emojier-is-very-slow-makeqtwrapper-is-accidentally-quadratic/27160/3


Default configuration not used as fallback if one option has been changed

If you change one option configuration you must rewrite all defaults for other engines.

That's by design: compiling diagrams can be a security risk, so we are very conservative about what to allow. If any diagram engine is configured, then all others must be marked as enabled explicitly. This way, when the filter adds support for a new diagram engine, the engine is off my default, which is important for security-oriented users who compile documents created by untrusted users.
It's enough to set the engine to true to enable it:
diagram:
  engines:
    # Compile graphviz "dot" diagrams, but not mermaid diagrams.
    graphviz: true
    mermaid: false

See also the section on security in the README.

Ok I understand it is intented. Thanks for your answer.


qutebrowser: 1.4.1 -> 1.4.2

Motivation for this change
Things done

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

[x] NixOS
[ ] macOS
[ ] other Linux distributions

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

@GrahamcOfBorg build qutebrowser

No attempt on x86_64-darwin (full log)
The following builds were skipped because they don't evaluate on x86_64-darwin: qutebrowser
Partial log (click to expand)

a) For `nixos-rebuild` you can set
  { nixpkgs.config.allowUnsupportedSystem = true; }
in configuration.nix to override this.

b) For `nix-env`, `nix-build`, `nix-shell` or any other Nix command you can add
  { allowUnsupportedSystem = true; }
to ~/.config/nixpkgs/config.nix.

Success on aarch64-linux (full log)
Attempted: qutebrowser
Partial log (click to expand)
/nix/store/1hxrlgwq8c4c2r8m5h8aslv1fx4nmx7f-qutebrowser-1.4.2/share/qutebrowser/userscripts/openfeeds: interpreter directive changed from "/usr/bin/env python3" to "/nix/store/sb88r9crdhinhkm6vpxwzf5zmij86p8v-python3-3.6.6/bin/python3"
/nix/store/1hxrlgwq8c4c2r8m5h8aslv1fx4nmx7f-qutebrowser-1.4.2/share/qutebrowser/userscripts/open_download: interpreter directive changed from "/usr/bin/env bash" to "/nix/store/fqm2x6kiay1q4vg7pqp4wp17bdijlyc3-bash-4.4-p23/bin/bash"
/nix/store/1hxrlgwq8c4c2r8m5h8aslv1fx4nmx7f-qutebrowser-1.4.2/share/qutebrowser/userscripts/getbib: interpreter directive changed from "/usr/bin/env python3" to "/nix/store/sb88r9crdhinhkm6vpxwzf5zmij86p8v-python3-3.6.6/bin/python3"
/nix/store/1hxrlgwq8c4c2r8m5h8aslv1fx4nmx7f-qutebrowser-1.4.2/share/qutebrowser/userscripts/format_json: interpreter directive changed from "/bin/sh" to "/nix/store/fqm2x6kiay1q4vg7pqp4wp17bdijlyc3-bash-4.4-p23/bin/sh"
/nix/store/1hxrlgwq8c4c2r8m5h8aslv1fx4nmx7f-qutebrowser-1.4.2/share/qutebrowser/userscripts/dmenu_qutebrowser: interpreter directive changed from "/usr/bin/env bash" to "/nix/store/fqm2x6kiay1q4vg7pqp4wp17bdijlyc3-bash-4.4-p23/bin/bash"
/nix/store/1hxrlgwq8c4c2r8m5h8aslv1fx4nmx7f-qutebrowser-1.4.2/share/qutebrowser/userscripts/cast: interpreter directive changed from "/usr/bin/env bash" to "/nix/store/fqm2x6kiay1q4vg7pqp4wp17bdijlyc3-bash-4.4-p23/bin/bash"
checking for references to /build in /nix/store/1hxrlgwq8c4c2r8m5h8aslv1fx4nmx7f-qutebrowser-1.4.2...
wrapping `/nix/store/1hxrlgwq8c4c2r8m5h8aslv1fx4nmx7f-qutebrowser-1.4.2/bin/.qutebrowser-wrapped'...
postPatchMkspecs
/nix/store/1hxrlgwq8c4c2r8m5h8aslv1fx4nmx7f-qutebrowser-1.4.2

Success on x86_64-linux (full log)
Attempted: qutebrowser
Partial log (click to expand)
/nix/store/5v2mm13wfwzh719qxc3sskahvfwnq362-qutebrowser-1.4.2/share/qutebrowser/userscripts/view_in_mpv: interpreter directive changed from "/usr/bin/env bash" to "/nix/store/czx8vkrb9jdgjyz8qfksh10vrnqa723l-bash-4.4-p23/bin/bash"
/nix/store/5v2mm13wfwzh719qxc3sskahvfwnq362-qutebrowser-1.4.2/share/qutebrowser/userscripts/taskadd: interpreter directive changed from "/usr/bin/env bash" to "/nix/store/czx8vkrb9jdgjyz8qfksh10vrnqa723l-bash-4.4-p23/bin/bash"
/nix/store/5v2mm13wfwzh719qxc3sskahvfwnq362-qutebrowser-1.4.2/share/qutebrowser/userscripts/tor_identity: interpreter directive changed from "/usr/bin/env python3" to "/nix/store/hy65mn4wjswqih75gfr6g4q3xgqdm325-python3-3.6.6/bin/python3"
/nix/store/5v2mm13wfwzh719qxc3sskahvfwnq362-qutebrowser-1.4.2/share/qutebrowser/scripts/keytester.py: interpreter directive changed from "/usr/bin/env python3" to "/nix/store/hy65mn4wjswqih75gfr6g4q3xgqdm325-python3-3.6.6/bin/python3"
/nix/store/5v2mm13wfwzh719qxc3sskahvfwnq362-qutebrowser-1.4.2/share/qutebrowser/scripts/dictcli.py: interpreter directive changed from "/usr/bin/env python3" to "/nix/store/hy65mn4wjswqih75gfr6g4q3xgqdm325-python3-3.6.6/bin/python3"
/nix/store/5v2mm13wfwzh719qxc3sskahvfwnq362-qutebrowser-1.4.2/share/qutebrowser/scripts/importer.py: interpreter directive changed from "/usr/bin/env python3" to "/nix/store/hy65mn4wjswqih75gfr6g4q3xgqdm325-python3-3.6.6/bin/python3"
/nix/store/5v2mm13wfwzh719qxc3sskahvfwnq362-qutebrowser-1.4.2/share/qutebrowser/scripts/open_url_in_instance.sh: interpreter directive changed from "/bin/bash" to "/nix/store/czx8vkrb9jdgjyz8qfksh10vrnqa723l-bash-4.4-p23/bin/bash"
checking for references to /build in /nix/store/5v2mm13wfwzh719qxc3sskahvfwnq362-qutebrowser-1.4.2...
wrapping `/nix/store/5v2mm13wfwzh719qxc3sskahvfwnq362-qutebrowser-1.4.2/bin/.qutebrowser-wrapped'...
postPatchMkspecs

seems to work fine.


Consider a call for maintainers

Open source software is a boon for software developers around the world. It also takes a lot of effort to maintain a repository. Please consider recruiting some maintainers to help get and keep the ball rolling.
e.g. https://github.com/selectize/selectize.js/issues/752
cc @vsn4ik @acrobat

I know what you mean.  There's other repositories that haven't had an update in 3 years and nobody is responding to bugs and such.
Case in point: I use the Bootstrap-Checkbox plugin (https://github.com/vsn4ik/bootstrap-checkbox) in my application framework.  I added new functionality and posted it, but no responses what so ever.


IntegrityError at /api/password_reset/ null value in column "ip_address" violates not-null constraint

Hi, I have a problem with this dependency on production server. Locally works very good, but problem appears on server. It Django issue is?
django-rest-passwordreset version: 0.9.7
Django: 1.11.5

If you need any logs, let me know what you need.

Hi,
it might be possible that your request does not have "ip address" set.
Take a look at this PR and the comments, the HTTPXForwardedForMiddleware might help you: https://github.com/anx-ckreuzberger/django-rest-passwordreset/pull/7

This should be fixed by PR #40. It's not yet in a release, but it's in the master branch.

Should be fixed by release 1.1.0rc1, you can install it using
pip install "django-rest-passwordreset==1.1.0rc1"

@anx-ckreuzberger  cant get it to work 😞

Can you provide an error message/exception/...?

pip install "django-rest-passwordreset==1.1.0rc1"

in my case it does not work, because the:
request.META.get('REMOTE_ADDR', getattr(settings, 'DJANGO_REST_PASSWORDRESET_REMOTE_ADDR', ''))

returns empty string, not null add a validation and it worked :(
solve it like this:
ip_address = request.META.get('REMOTE_ADDR', getattr(settings, 'DJANGO_REST_PASSWORDRESET_REMOTE_ADDR', ''))
user_agent = request.META.get('HTTP_USER_AGENT', getattr(settings, 'DJANGO_REST_PASSWORDRESET_HTTP_USER_AGENT', ''))
token = ResetPasswordToken.objects.create(
    user = user,
    user_agent = getattr(settings, 'DJANGO_REST_PASSWORDRESET_HTTP_USER_AGENT', '') if user_agent == '' else user_agent,
    ip_address = getattr(settings, 'DJANGO_REST_PASSWORDRESET_REMOTE_ADDR', '') if ip_address == '' else ip_address
)

@anx-ckreuzberger I've also bumped to the pre-release version 1.1.0rc1, but the IntegrityError still remains, even when I've set DJANGO_REST_PASSWORDRESET_HTTP_USER_AGENT and DJANGO_REST_PASSWORDRESET_REMOTE_ADDR in my settings configuration.
This is pretty weird because I've checked them about a hundred times but it still tries to set a null value for ip_address, which raises the error.
Any ideas? :)

look for the file in your local project venv/lib... modify the file in the line that I put
https://github.com/anx-ckreuzberger/django-rest-passwordreset/blob/85d30b7129605cf7651a79ebabb3dbac5e01e181/django_rest_passwordreset/views.py#L140
modify as I put in my previous comment

I know I'm the one that pressed the merge button on PR #40 , but I guess
                    # no token exists, generate a new token
                    token = ResetPasswordToken.objects.create(
                        user=user,
                        user_agent=request.META.get('HTTP_USER_AGENT',
                                                    getattr(settings, 'DJANGO_REST_PASSWORDRESET_HTTP_USER_AGENT', '')),
                        ip_address=request.META.get('REMOTE_ADDR',
                                                    getattr(settings, 'DJANGO_REST_PASSWORDRESET_REMOTE_ADDR', ''))
                    )

is just wrong?
I believe the implementation suggested by @vonorm here https://github.com/anx-ckreuzberger/django-rest-passwordreset/pull/40#issuecomment-506752772 is correct.
Any thoughts?

The idea is to work and not return error.
The problem that I had was that this request.META.get ('REMOTE_ADDR') returned empty text and therefore never set the default value assigned from the setting.
that say this: request.META.get ('REMOTE_ADDR',
                                                     getattr (settings, 'DJANGO_REST_PASSWORDRESET_REMOTE_ADDR', '')) returned empty text, which does not allow it to be inserted into the database.
There are several alternatives, I suggested what worked for me
PD: thanks

The bugfixes have been released on PyPi in version 1.1.0rc3.
You should be able install and test it using
pip install django-rest-passwordreset==1.1.0rc3

I am still getting the error: null value in column "ip_address" violates not-null constraint
even after installing django-rest-passwordreset==1.1.0rc3 :(

There is another error Data too long for column 'ip_address'.
This happens with the header
X-Forwarded-For | 2605:e000:100b:81b3:a5bb:913e:80c0:6e4, <IP_ADDRESS>

DJANGO_REST_PASSWORDRESET_IP_ADDRESS_HEADER is set to use HTTP_X_FORWARDED_FOR.
Do I have to use HTTP_X_REAL_IP?

Fixed since 1.4.0


fixes #360 - added link to non-core materials

If this isn't what you envisioned, please let me know.

Thank you SO much for you contribution, @scroniser :) We really appreciate it!
Would you mind creating the pull request against our "staging" branch first? We stage our changes for a final look before pushing live to production.
I am going to include a screenshot of your work so that others can comment or give suggestions about the positioning. Thank you again!

@kstack7 - As requested, created a pull request on the staging branch. Sorry about that!


S2U-39 <IP_ADDRESS> Wiki: Add group support to Wiki pages - fix

Thanks for the great work @JuanDavid102 .
Are we expecting more fixes for this?

Yes, just now i will send the lastest modifications