HTTPS vs HTTP

I found that the instructions did not work when url was http (vs.net 2019 said that it was unable to start the application pool, even if i manually started the application pool, vs.net 2019 when trying to debug F5 it would stop it..then vs.net would just hang.
After I made it https it all worked.  Is this by design?

I take it that you're referring to dev-time support for IIS in VS? https://docs.microsoft.com/aspnet/core/host-and-deploy/iis/development-time-iis-support
btw --- Use the feedback feature at the bottom of the topic when opening an issue. It will tag the issue with metadata that helps our tracking.

oh ... and a clarification on the screenshot situation. We use them for a number of tutorials. However even in tutorials, they're difficult to maintain, so we try to back off using them when possible.
If you are referring to the dev-time support topic, your timing with this issue is great. Like you, I just installed VS2019 here, and I need to walk the issue. I'll try to use text to explain the screen selections. I'll update for some of the differences that you may have seen between the VS2017-era text and the VS2019 UI.

@dyardyGIT ok ... I have the scoop on this now. Yes, it will work with HTTP, but there's a catch: It can't work with both HTTP and HTTPS.
In the following screenshots, you can see that my app is set up to be reached at /testFDD on localhost. It's one of my framework-dependent deployment test apps.
App Pool:

HTTP
IIS website binding:

VS2019 profile:

HTTPS
IIS website binding:

VS2019 profile:

Hosting Model
You can leave Hosting Model set to Default. The behavior is that if you have ...
<AspNetCoreHostingModel>InProcess</AspNetCoreHostingModel>

... in the project file, it will use in-process as the default. If you don't have the property in 2.2, the default is out-of-process. If you don't have the property and you want in-process in 2.2, then you should change the profile's Hosting Model to In Process.
As for 3.0 coming out later, the default will switch over to in-process. At that point, it wouldn't matter if one had the property for in-process in the project file or not.
Caveat for HTTP/HTTPS
Like I mentioned, either set the whole thing up HTTP or set it up HTTPS. You just want one binding for the website in IIS.
I see several things that I'd like to update in the topic, so I'm going to work on it this afternoon (Saturday). I'll ping u on the PR.


Number Type Finder III

Enter calculator name -
Number Type Finder
Describe calculator functionality -
Given a number entered by the user, find if the number is -

Abundant Number
Disarium Number
Emirp Number
Fascinating Number
Magic Number

Under which category you wish to place it -

[x] Computer Science
[ ] Mathematics
[ ] Physics
[ ] Chemistry
[ ] Biology
[ ] Art
[ ] Finance
[ ] Scientific
[ ] Health
[ ] Sports
[x] Let project owner decide
[ ] Suggest a new category - (Answer here)

Hey, @SarthakKeshari please assign me this issue

Hey @mannbajpai,
Assigned.
You may contribute your code under the folder path -
calc_for_everything\calculators\src\computer_science\number_type_finder
Kindly follow the code format and be careful, do not break the old code already exists.
Issue completion deadline - 26th Oct 6:00:00 pm IST
NOTE - In case the issue completion deadline is not met, the issue will be assigned to a new participant


Adding meshButton3D in Gui3D

meshButton3D will enable the interactableObject from MRTK

Can you remove the dist folder from your PR?

Thanks for the feedback, updating my PR.


BREAKING CHANGE: multiple instruments per proposal

Description
This PR introduces the initial proof of concept for allowing multiple instruments per proposal.
Motivation and Context
This change is required to accommodate scenarios where a proposal requires the use of multiple instruments. This is a common requirement in many scientific research fields and has been requested by users.
Changes

SQL script apps/backend/db_patches/0146_MultiInstrumentPerProposal.sql is added to create database changes needed for supporting multiple instruments per proposal.
Changes have been made in ReviewDataSource.ts and InstrumentDataSource.ts to fetch multiple instruments related to a proposal.
Modifications in ProposalAuthorization.ts and TechnicalReviewAuthorization.ts to handle multiple instruments per proposal scenario.

https://github.com/UserOfficeProject/user-office-core/assets/6333063/1bbd1e2d-c974-4fe6-90dc-e6ca152c3602
How Has This Been Tested?

existing e2e tests
manual tests with empty and develop database

Fixes Jira Issue
https://jira.esss.lu.se/browse/SWAP-3531
Depends On

Tests included/Docs Updated?

[ ] I have added tests to cover my changes.
[ ] All relevant doc has been updated

Why do we have in this file ProposalSelectionInput also callId and workflowId. Isn't that known already from proposalPK? (https://github.com/UserOfficeProject/user-office-core/blob/c6b0e4dfe7d43faecc022a8ab910ac704bcc71c2/apps/backend/src/resolvers/mutations/ChangeProposalsStatusMutation.ts#L15C21-L15C21)
Is it true that a proposal is assigned to one or more instruments but does not necessarily need to have a technical review for all or even any of them?
Is technical review the same as internal review?

@jekabs-karklins first of all thank you for the good feedback. Here are some answers:

I changed this because it is much better to collect that data on the backend instead of sending it as an argument from the outside. Nice improvement done here!
The relation should be one to one. So one instrument on proposal can (or needs to) have one technical review. One technical review per instrument is required.
Technical review is not the same as internal review. Per technical review we can have as much as we want internal reviews which are used only internally to collect little bit more feedback for that specific proposal/instrument/technical review set.

I had a play around with this and it looks great. I ran into a couple of issues and was wondering if you could check if you see them as well?

The FAP spreadsheet export doesn't shows proposals across all instruments
(as a User Officer after reviews have been submitted) If you go to the FAP page, then Proposals and Assignments, View Proposals -> Technical Review (Could this be made plural, please?) and then select an instrument, it takes you back to the general tab.

If you try to drag a proposal from one instrument to another it ends up looking a little weird.

I had a play around with this and it looks great. I ran into a couple of issues and was wondering if you could check if you see them as well?

The FAP spreadsheet export doesn't shows proposals across all instruments
(as a User Officer after reviews have been submitted) If you go to the FAP page, then Proposals and Assignments, View Proposals -> Technical Review (Could this be made plural, please?) and then select an instrument, it takes you back to the general tab.

 If you try to drag a proposal from one instrument to another it ends up looking a little weird.

This should be fixed now @Scott-James-Hurley. You are welcome to give it another try and see if you can find some more improvements.

This should be fixed now @Scott-James-Hurley. You are welcome to give it another try and see if you can find some more improvements.

That's grand. Thanks for taking on my feedback, really appreciate it.
With regards to the question of hard/soft deletes that you mentioned in sprint planning: is that something that should be addressed in this PR or separated into another issue? I was thinking we should hold onto dangling reviews. Maybe if a proposal's removed from an instrument but later it's decided it should be on it after all it would save the instrument scientist from having to re-submit their review? I imagine user officers could weigh in on what to do with them.

This should be fixed now @Scott-James-Hurley. You are welcome to give it another try and see if you can find some more improvements.

That's grand. Thanks for taking on my feedback, really appreciate it.
With regards to the question of hard/soft deletes that you mentioned in sprint planning: is that something that should be addressed in this PR or separated into another issue? I was thinking we should hold onto dangling reviews. Maybe if a proposal's removed from an instrument but later it's decided it should be on it after all it would save the instrument scientist from having to re-submit their review? I imagine user officers could weigh in on what to do with them.

Hi @Scott-James-Hurley thanks for the feedback once again. We plan to address this soft deletion in another issue I hope in the near future. And yes I agree maybe we will need better strategy how to handle removal of the proposal instruments and what are the consequences of it.


Updated docker to tf-night 2.7.0

Changes in the tensorflow lite API requires us to update our docker images.
Update appropriately.

Failure is related to flakes in the already upstreamed code.


Key Indicator Header tooltips

Some key indicator headers in the PDF profile have footnotes. We can do these as tooltips, however, the headers are done through the name parameter of {{#data.indicator}}, not through raw markup. I'm not sure if name needs to be componentized. If we move this to raw markup, it'll be easier to do the header tooltips.

Solved


Mass Saving

When trying to save mass in BoneLab and reloading the avatar, it doesn't save. also it doesn't seem load the masses either. I have 2 screenshots of Before Patch 5 & After Patch 5
https://cdn.discordapp.com/attachments/573327048865480745/1275486229852459049/BONELAB_2024-08-20_11-05-43.mp4?ex=66c6109e&is=66c4bf1e&hm=fbeaaf96eb371bca77fc2cd127bf2fba72fcdd4e75fce559caedda63c3518f42&

i do also have both save and load masses enabled incase you were gonna ask


Any script available for mac os ?

Any prefered way to install sensor on mac os ?

@saranVishva thanks for opening up an issue. Currently we don't have any scripts available in this project. We do have plans to support Mac with install/uninstall scripts in the near future. Check out the following discussion #241 to see an existing project that has some scripts you can use.


Update Composer dependencies (2021-10-02-21-12)

Visual regression test passed!
View the visual regression test report


New component not loading, while old component is loading fine.

The 'old' component is loading fine (https://github.com/zxdavb/evohome/blob/master/evohome.py)
While your 'new' component isn't loading (https://raw.githubusercontent.com/zxdavb/home-assistant/dev/homeassistant/components/evohome.py)
Used your new instructions and changed the REQUIREMENTS in the honeywell.py file.
Still giving these startup errors:
2018-08-18 11:31:18 WARNING (SyncWorker_2) [custom_components.evohome] setup(): Calling API [4 request(s)]: client.__init__()...
2018-08-18 11:31:21 ERROR (MainThread) [homeassistant.loader] Error loading custom_components.climate.evohome. Make sure all dependencies are installed
Traceback (most recent call last):
  File "/usr/local/lib/python3.6/dist-packages/homeassistant/loader.py", line 94, in get_component
    module = importlib.import_module(path)
  File "/usr/lib/python3.6/importlib/__init__.py", line 126, in import_module
    return _bootstrap._gcd_import(name[level:], package, level)
  File "<frozen importlib._bootstrap>", line 994, in _gcd_import
  File "<frozen importlib._bootstrap>", line 971, in _find_and_load
  File "<frozen importlib._bootstrap>", line 955, in _find_and_load_unlocked
  File "<frozen importlib._bootstrap>", line 665, in _load_unlocked
  File "<frozen importlib._bootstrap_external>", line 678, in exec_module
  File "<frozen importlib._bootstrap>", line 219, in _call_with_frames_removed
  File "/etc/homeassistant/custom_components/climate/evohome.py", line 12, in <module>
    from homeassistant.components.evohome import (
ModuleNotFoundError: No module named 'homeassistant.components.evohome'
2018-08-18 11:31:21 ERROR (MainThread) [homeassistant.loader] Unable to find component climate.evohome
2018-08-18 11:31:21 ERROR (MainThread) [homeassistant.setup] Unable to prepare setup for platform climate.evohome: Platform not found.

It is no longer a custom component.
File "/etc/homeassistant/custom_components/climate/evohome.py", line 12, in
from homeassistant.components.evohome import

Having said that it may still work if you copy evohome.py to .homassistant/custom_components and climate/evohome.py to .homassistant/custom_components/climate.

Problem solved by copying files to:
/usr/local/lib/python3.6/dist-packages/homeassistant/components/
and:
/usr/local/lib/python3.6/dist-packages/homeassistant/components/climate/
Topic Closed.


S3 Private Link Customization

Community Note

Please vote on this issue by adding a 👍 reaction to the original issue to help the community and maintainers prioritize this request
Please do not leave "+1" or "me too" comments, they generate extra noise for issue followers and do not help prioritize the request
If you are interested in working on this issue, please leave a comment

Describe the Feature

SEP: /seps/accepted/shared/s3-privatelink.md

endpoints 2.0 is out in 0.4.0


[gpos profile] Filter monospaced fonts out of GPOS table kerning check

Closes https://github.com/googlefonts/fontbakery/issues/2674
Related https://github.com/googlefonts/fontbakery/issues/3072
This PR proposes the addition of a post table, isFixedPitch value filter on the FB check ID com.google.fonts/check/gpos_kerning_info in order to skip this check when a font is monospaced and should not include kerning data.

Yowser.  38 min queue for Travis CI run so far...

Looks good, but please use the same code-style that we have on the other checks and code-tests
Also, please add a CHANGELOG.md entry

needs fixes to code formatting

Np. Are you using an auto formatter tool? If not, what linter or style spec can I use to determine changes?

needs fixes to code formatting

Np. Are you using an auto formatter tool? If not, what linter or style spec can I use to determine changes?

No, I'm not. It is just a loosely defined project code-style, but no tool enforces it. This is another reason why I should document it better, as suggested by Dave at issue #3039
I will fix this one myself and merge it, no worries.

Thanks Felipe


Version <IP_ADDRESS> makes IssueCustomField.Info readonly breaking existing usage

Hi,
I'm not sure how to migrate to the new version as it makes some fields readonly that were previously used to create or update data. e.g. how should we now update the value of an IssueCustomField with .Info?
I didn't find an issue or wiki related to this question so bit uncertain what is the expected upgrade path

+1
@zapadi, I have the same question!
Most base classes like "Identifiable", now have "protected internal set" for "Id".
How it use?

I am having the same issue and overcome it by
public class RedmineIdentifiableName : IdentifiableName
	{
		public RedmineIdentifiableName(int id)
		{
			base.Id = id;
		}

	}

same here ... why????

+1
@zapadi, I have the same question!
Most base classes like "Identifiable", now have "protected internal set" for "Id".
How it use?

The idea behind removing most of the setters is  to follow the serialization process.  If the properties of a redmine entity are not expected by the server, those properties should not have public setters.
The case of identifiable is the same: the ids are assigned by the server.
One of the solutions, for the moment, is to do it like @KEMAL-YILDIRIM did.
If you have a better idea or approach, please, let me know.

I had the same problem when using this .
I did a seach first and then assign the value to the new one .
I'm tied.

public class RedmineIdentifiableName : IdentifiableName
{
public RedmineIdentifiableName(int id)
{
base.Id = id;
}
}

Good job!


gRPC GetMetrics "start is None" Error

I'm trying to get a device metrics using the gRPC, but I'm getting an error:
Error:  <_InactiveRpcError of RPC that terminated with:
status = StatusCode.INTERNAL
details = "start is None"
debug_error_string = "UNKNOWN:Error received from peer  {created_time:"2024-03-08T12:51:17.1366875+01:00", grpc_status:13, grpc_message:"start is None"}"

Here is the python code that I'm testing with:
from   chirpstack_api import api
import grpc
from google.protobuf.json_format import MessageToJson

server  = ""
api_token = ""

channel = grpc.secure_channel(server, grpc.ssl_channel_credentials())
client  = api.DeviceServiceStub(channel)
auth_token = [("authorization", "Bearer %s" % api_token)]

def get_device():

    try:
        req = api.GetDeviceMetricsRequest()
        req.dev_eui = '0018b20000023455'
        resp = client.GetMetrics(req, metadata=auth_token)
        print(MessageToJson(resp))
    except grpc.RpcError as e:
        print('error:',type(e))
        if e.code() == grpc.StatusCode.INTERNAL:
            print('\n  Error: ', e)
    return None

get_device()

Not sure if I'm missing something or not, but other endpoints are working fine.
Thank you for this great project and support!

As the error indicates, you didn't set a start time. Unfortunately this is required for the API call.
Here's my getMetrics call as an example:
def getMetrics(client,token,dev_eui,start,end):
  print("Retrieving metrics from device with EUI: " + dev_eui)

  auth_token = [("authorization", "Bearer %s" % token)]
  try:
    req = api.GetDeviceMetricsRequest()
    req.dev_eui = dev_eui
    req.start.FromDatetime(start)
    req.end.FromDatetime(end)

    resp = client.GetMetrics(req, metadata=auth_token)

    print(resp.metrics)

    return resp

Note that the start and end times I am passing into the function are Datetime objects. Also I've found retrieving metric through the API pretty useless unless you increase the time they are stored in the database before deleting.
For future questions you'll probably have better luck posting them to the Chirpstack Community Forum then here to github


Need to remove usbdeview from recommendations list. Old, unmaintained, not updated to recognize USB 3.1, 3.2, 4.0, Thunderbolt 3/4

Need to remove usbdeview from recommendations list. Old, unmaintained, not updated to recognize USB 3.1, 3.2, 4.0, Thunderbolt 3/4

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

ID: 94f202cf-716f-b82b-1eba-31e4226cbb4f
Version Independent ID: c8ee74d1-b7f8-4807-b36f-1071ee68a30a
Content: USBView in Windows - Windows drivers
Content Source: windows-driver-docs-pr/debugger/usbview.md
Product: windows-hardware
Technology: debugger
GitHub Login: @DOMARS
Microsoft Alias: domars

@cmccane - Thanks for this input on USBView not understanding the latest feature of the newer USB technologies. I added a note to the topic about this and that highlights the use of alternatives, such as DeviceManager to display additional information from later versions of USB. The note points to the list of IOCTLs that USBView uses. As sometimes you just need basic information about which hub your device is connected to, even without being updated, USBView is still usable in those situations. For more advanced work, USB driver developers tend to use specific hardware, for example the MUTT tool described in Overview of Microsoft USB Test Tool (MUTT) devices. Or domain specific tools, such as the HID Descriptor Tool and the other tools provided by usb.org. There are also various third party programs and protocol analyzers available. Thanks again for providing this input on USBView, and good luck with your USB debugging work.


Support for response descriptions

Hi @subosito,
It looks like Apiary's Drafter supports Response Descriptions (https://github.com/apiaryio/drafter/blob/master/features/fixtures/blueprint.apib#L77).  Perhaps snowboard should as well.
Example:
##  Objects [/objects]
### Create an object [POST]
  Creates an object with the given data

  + Response 200 (application/json)

    Successful Response with the created object

  + Response 422 (application/json)

    Error Response when the object failed validation

I was hoping that I would just be able to add this line to a custom template,

{{$transition.Response.Description | markdownize}}

But then I quickly realized that Description isn't implemented on the Response struct (https://github.com/subosito/snowboard/blob/master/api/blueprint.go#L59-L64). I would create a PR but my GO skills are n00b at best.
I can make an attempt to create a PR if you have no time or interest in implementing this feature.  Tell me what you think.
Cheers! 😸

Hi @codebender,
Just pushed changes related to this issue, you can see https://github.com/subosito/snowboard/commit/c8fb717bf9c8d803b0be5acbce713991432d6731.
Please let me know your thoughts :)

@subosito Awesome, that was way too quick! Much Appreciated!
Closing this issue.


Transfer

transfer route not working

Working on it

There were errors in type definitions in Mongoose schema and now have been fixed


Publish RSMP core 3.3.0 to github pages

I guess we just need to add a link to the draft at https://rsmp-nordic.org/specification/?

Yes. However this PR didn't quite work. There was some protection rule for github pages which prevented the new version from being published. I'll changed it now and I will try to merge another PR to the 3.3.0 branch and see if it works.

Update: It works now


IfIsNull does not handle Nullable type

Raise ArgumentNullException if an instance of that type has no value.

Closed in release v2.2.3.


fix: Update aws-crt-swift dependency to 0.17.0.

Issue #

https://github.com/awslabs/aws-sdk-swift/issues/1204
Description of changes

Update aws-crt-swift dependency to 0.17.0.
New/existing dependencies impact assessment, if applicable

Conventional Commits

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

/test


Please add a license to this repo

First, thank you for writing this book and sharing these notebooks with us!
Could you please add an explicit LICENSE file to the repo so that it's clear
under what terms the content is provided, and under what terms user
contributions are licensed?
Per GitHub docs on licensing:

[...] without a license, the default copyright laws apply, meaning that you
retain all rights to your source code and no one may reproduce, distribute,
or create derivative works from your work. If you're creating an open source
project, we strongly encourage you to include an open source license.

Thanks!

@mbrukman thanks for the suggestion (and explaining)! I've just added an MIT license.


Add reflection workflow

Description
Add reflection workflow. This workflow will run when commits are pushed/PRs are merged into the main branch. It will then copy the changes and push to the private repo https://github.com/valora-inc/wallet-release-reflection that will trigger its own workflows to start release processes.
Other changes
None.
Tested
Tested in https://github.com/valora-inc/wallet/pull/1012 (closed in favour of this PR to reduce commits)
Methodology tested in #1018.
How others should test
Not necessary yet.
Related issues
N/A.
Backwards compatibility
N/A.

That's fair! I can talk to someone to get us a service user setup. I've been meaning to do it for a while.


feat: 3835 - country selector similar to language selector

What

The country selector is now similar to the language selector. In particular, the selected country is on top.
2 minor visual bugs are also fixed: the "preferences" app bar title is now on 2 lines if needed, and there's no more overflow on the "welcome" page. In both cases, that impacted small screens- (cf. screenshots).
As a side effect, I refactored OnboardingPage. I was in a "Am I crazy or what?" mode for several days and tried everything to fix my issue. Was there a flaw in the provider? Spoiler alert: no there wasn't, I just coded something inappropriately that cleared the data I'd just entered...
We don't need the refactoring I mention, but the code is a bit clearer anyway.

Screenshots

country selector
language selector

overflow bug (before)
overflow bug (fixed)

title overflow bug (before)
title overflow bug (fixed)

Fixes bug(s)

Closes: 3835

Files

consent_analytics_page.dart: minor refactoring
country_selector.dart: now using the preferences for initial country code; same UI as language selector
main.dart: minor refactoring
next_button.dart: minor refactoring
onboarding_flow_navigator.dart: refactoring around OnboardingPage
permissions_page.dart: minor refactoring
product_query.dart: minor refactoring
user_preferences.dart: minor refactoring
user_preferences_dev_mode.dart: minor refactoring
user_preferences_page.dart: minor UI fix (app bar title on 2 lines)
user_preferences_settings.dart: added the "selected language" parameter; minor refactoring
welcome_page.dart: fixed a visual overflow bug

Codecov Report

Merging #3936 (c7f470b) into develop (69858ba) will increase coverage by 0.01%.
The diff coverage is 1.41%.

:exclamation: Current head c7f470b differs from pull request most recent head bb3e89c. Consider uploading reports for the commit bb3e89c to get more accurate results

@@             Coverage Diff             @@
##           develop    #3936      +/-   ##
===========================================
+ Coverage    10.99%   11.01%   +0.01%
===========================================
  Files          264      264
  Lines        13051    13032      -19
===========================================
  Hits          1435     1435
+ Misses       11616    11597      -19

Impacted Files
Coverage Δ

...s/smooth_app/lib/data_models/user_preferences.dart
15.20% <0.00%> (-0.51%)
:arrow_down:

packages/smooth_app/lib/main.dart
12.87% <0.00%> (ø)

...p/lib/pages/onboarding/consent_analytics_page.dart
0.00% <0.00%> (ø)

...oth_app/lib/pages/onboarding/country_selector.dart
0.87% <0.00%> (+0.87%)
:arrow_up:

...s/smooth_app/lib/pages/onboarding/next_button.dart
4.54% <0.00%> (ø)

...ib/pages/onboarding/onboarding_flow_navigator.dart
0.00% <0.00%> (ø)

...oth_app/lib/pages/onboarding/permissions_page.dart
0.00% <0.00%> (ø)

.../smooth_app/lib/pages/onboarding/welcome_page.dart
0.00% <0.00%> (ø)

...b/pages/preferences/user_preferences_dev_mode.dart
0.00% <0.00%> (ø)

...b/pages/preferences/user_preferences_settings.dart
6.33% <0.00%> (-0.25%)
:arrow_down:

... and 2 more

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more

Thank you @M123-dev for your review!
Regarding the colors being defined outside of the pages, I wouldn't mind if it was coded differently but after all, that forces developers to think about a different color for each onboarding page. Let's keep it that way.


Change System_ToolCache to Agent_ToolCache

Only variable set by server start with System_

I'll make the change in another PR.  I already renamed this file.
Is this change going into the next agent update?

yes, it's part of overwrite tmp
https://github.com/Microsoft/vsts-agent/pull/828


Add advertise-client-urls to etcd2 launch options.

As of etcd 2.0.11 (see here: https://github.com/coreos/etcd/releases/tag/v2.0.11)
the -advertise-client-urls option is mandatory.

Thank you @yoanisgil


Update build-template

Hi @natemcmaster, I'm your friendly neighborhood .NET Foundation Pull Request Bot (You can call me DNFBOT). Thanks for your contribution!
You've already signed the contribution license agreement. Thanks!
The agreement was validated by .NET Foundation and real humans are currently evaluating your PR.
TTYL, DNFBOT;

Could you also update NuGet.master.config while you're at it?
:shipit: once that's done.

Updated.


chore(main): release 1.4.2

:robot: I have created a release beep boop
1.4.2 (2024-01-02)
Miscellaneous Chores

allow users to specify http/https port (db4af2e)

This PR was generated with Release Please. See documentation.

:robot: Release is at https://github.com/mdshack/shotshare/releases/tag/v1.4.2 :sunflower:


Fix v1.3.x integration tests with a bad CIDR

A few tests were failing when using KET v1.3.x, need to use the old <IP_ADDRESS>/16 CIDR for these tests.

LGTM


Go client fails to compile on FreeBSD 14: missing function getSendQueueLen

Describe the bug
When trying to compile on FreeBSD 14, the compile fails with following key message: missing function getSendQueueLen.
This isuue regards tchannel-go repository.
An issue has been opened there but it hangs for a while:
https://github.com/uber/tchannel-go/issues/914
To Reproduce
Is the issue reproducible?

Yes

Steps to reproduce the behavior:
Try to compile a project with dependency to current repository on FreeBSD 14, if fails because of https://github.com/uber/tchannel-go issue.
Expected behavior
Compilation is successfull

@ibarrajo hey, I noticed that the PR on the issue just hangs there, how can we get a review on that?

I pinged a maintainer at Uber to review your PR =]

@moonetic it's been merged: https://github.com/uber/tchannel-go/pull/925


Disable certain dangerous features for un-authenticated users

If we could make a user login before seeing the content, we could still allow anonymous access, but with more restrictions.
For example we could:

Disable forms
Disable links
Other actions which make phishing difficult

We should also consider if any of these are acceptable as defaults, or if they break too much genuine content.

I think there are two ideas here:
Disallowing unauthenticated use.
Disabling certain kinds of HTML features.
I think the first one is a non starter for the main via instance. I need to be able to tweet out a via link and have people easily see my annotations without logging in.
For the second, possibly forms but not links. People need to be able to click thru to another page, though that page doesn’t need to be proxied too.


PoppyRC file: replace descriptor key by locator

This property is not the robot descriptor but its locator.

Done.
As this feature was probably not used, no migration tooling/warning are provided.


diffusers  controlnet support  control_mode  and resize_mode  two parameters

When will Diffuser ControlNet support the two parameters control_mode and resize_mode like sd-webui-controlnet?
Looking forward to your reply, thank you
@lllyasviel

https://github.com/huggingface/diffusers/issues/5070

you should ask at diffusers https://github.com/huggingface/diffusers/issues


Ticking Block Entity crash

I got a crash when trying to place an RFTools Matter Receiver on a Draconic Evolution Creative RF Source.
Crash Log
It seems that a method could not be found.

It seems that some mod is having a wrong CoFH API included. Going to be a bit of a trouble to find out which one...

I have Metroid Cubed 3 installed, referenced as the issue in a later issue here. I followed FirEmerald's instructions on removing the cofh folder to fix this (I have a number of mods that provide the RF API so that is no issue).


Upgrade PostgreSQL extension to 0.25.0

Related: https://github.com/CartoDB/cartodb-postgresql/pull/346

:heavy_check_mark: Tested in staging and already installed for all users.


Documentation in pipelines is empty

Before submitting the issue

[X] I have checked for Compatibility issues
[X] I have searched among the existing issues
[X] I am using a Python virtual environment

Description of the bug
The Documentation check passes but the documentation artifact is actually empty (easily verified with its small size of 48 kB).
The last known Documentation generation job to work was here on August 22.
Was rerun to check for unseen change in a dependency, it still passed.
Fails starting here.
Edit: Might actually go back as far as the addition of CopyButton, where the last check in the PR is ok, however when pushing to master it stops before creating the full doc.
Two problems here:

the documentation check did not see something was wrong with the doc (artifact too small / step too short)
no error was raised although something went obviously wrong

Steps To Reproduce
Run a GitHhub Actions workflow
Which Operating System are you using?
Windows
Which Python version are you using?
3.8
Installed packages
See PR

Fixed. Was due to the pyvista bug for Mesa OpenGL.


[WIP] doc improvements

This is for a round of improvements to the docs before jupytercon...there will be a few other things to tackle but I'll try to get 'em all in this PR, see #86

Oops, I didn't see WIP until I had pressed merge. Feel free to revert but looks like an improvement as is.


Add CONTRIBUTING.md file

For https://github.com/kubernetes/org/issues/938#issuecomment-504695574
/assign @chuckha

thank you !!
/approve


Feature request: add options to change base colors of the components

Issue:
The colors don't always esthetically match rest of the dashboard and it can create rather unpleasant user experience (for the eyes).
Solution:
Add options to customize styles / colors of the shift picker plugin.
Keep up the amazing work with this amazing plugin! :)

Thanks @Sulomus will fix it in the v0.1.1 release.


Declarative Lookup Rollup Summary (DLRS) lookups wiped - fields showing "N/A"

I just noticed that the lookups that I had set up with DLRS have changed without any input from me. Trying to understand whether or not this is expected behavior.
Specifically: - all of the fields I input information in now only show "N/A" - Description reads: "System managed do not touch".
Furthermore, there are some entries that are showing up as being created by another user. When I asked that user, he stated that he has never used the tool - as expected since I didn't grant access to anyone but myself anyway!
It seems that one of the rollups is working while the others are not.

I think this is because of the new Custom Metadata Type feature in new release.
If you go in to Search and Type Custom Metadata Type, you will find one for rollup summaries. Click on the Manage Records and you will have the list of all your rollups. Samething happened with me when I was searching for my roll ups. I found all the roll ups in Custom Metadata Type.
Please let me know if you see the same.

@nishantjadav - They're all there! (Sorry for the delayed reply - never got a reply notification)
What did you have to do to restore the dlrs triggers? I'm assuming that I'm going to have to go in manually edit the fields so that they contain the information that they should contain.

@fdacruz1 what makes you think you need to restore the triggers? As @nishantjadav has said, just ignore these 'shadow' rollup entries, if your using custom metadata based rollups, just focus on using the Manage Rollup Summaries tab only and don't even look at the Lookup Rollup Summaries tab. 👍


Remove Persistent Topics v3 API - use custom media type instead

Fixes #14104
Motivation

PR #12818 changes for PIP-110 broke the API. A newer admin client cannot be used with an older broker.
It’s possible to have the current v2 API for creating persistent topics to handle multiple different payload types
The challenge seems to be that in the old method, the number of partitions was passed in the payload as a single integer without having a JSON object at all. This makes it harder to extend the existing API, but doesn’t make it impossible.
This solution is based on request’s “Content Type”
Content-Type: application/vnd.partitioned-topic-metadata+json on request, @Consumes("application/vnd.partitioned-topic-metadata+json") on method that handles the new payload type.

Modifications

use custom media type application/vnd.partitioned-topic-metadata+json to distinguish the new content payload type

but we should find a way to clearly document this in the documentation
maybe swagger is already well documenting this new media type?

@eolivelli Swagger distinguishes endpoints by verb + path only. It can be a docs issue, see also #18947.


Migrate fake backends

Summary
This PR replaces https://github.com/Qiskit/qiskit-ibm-provider/pull/740 (see original PR for context on the move) in anticipation of the deprecation of the provider repository. This move was previously consulted with @kt474, and it would allow us to directly point users to the final location of the module in the Qiskit deprecation warning (planned for 0.45). An alternative to this PR would be merging https://github.com/Qiskit/qiskit-ibm-provider/pull/740 and having the fake backends migrated in a second step. Tagging @1ucian0 for reference.
Details and comments
Fixes https://github.com/Qiskit/qiskit-ibm-provider/issues/511

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.14 out of 17 committers have signed the CLA.:white_check_mark: levbishop:white_check_mark: mtreinish:white_check_mark: taalexander:white_check_mark: ajavadia:white_check_mark: iuliazidaru:white_check_mark: HuangJunye:white_check_mark: nkanazawa1989:white_check_mark: jakelishman:white_check_mark: 1ucian0:white_check_mark: Cryoris:white_check_mark: to24toro:white_check_mark: ElePT:white_check_mark: itoko:white_check_mark: nonhermitian:x: kevinhartman:x: shivalee12:x: eltociearYou have signed the CLA already but the status is still pending? Let us recheck it.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 1512 (0.0%)  changed or added relevant lines in 110 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-14.2%) to 41.679%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

qiskit_ibm_runtime/fake_provider/utils/init.py
0
1
0.0%

qiskit_ibm_runtime/fake_provider/backends/auckland/init.py
0
2
0.0%

qiskit_ibm_runtime/fake_provider/backends/geneva/init.py
0
2
0.0%

qiskit_ibm_runtime/fake_provider/backends/oslo/init.py
0
2
0.0%

qiskit_ibm_runtime/fake_provider/backends/perth/init.py
0
2
0.0%

qiskit_ibm_runtime/fake_provider/backends/prague/init.py
0
2
0.0%

qiskit_ibm_runtime/fake_provider/backends/rueschlikon/init.py
0
2
0.0%

qiskit_ibm_runtime/fake_provider/backends/sherbrooke/init.py
0
2
0.0%

qiskit_ibm_runtime/fake_provider/backends/tenerife/init.py
0
2
0.0%

qiskit_ibm_runtime/fake_provider/backends/tokyo/init.py
0
2
0.0%

Totals

Change from base Build<PHONE_NUMBER>:
-14.2%

Covered Lines:
2477

Relevant Lines:
5943

💛  - Coveralls

@ElePT If everything still looks good on your end, I can get this in the next release


scroll to move by month

Is your feature request related to a problem? Please describe.
no
Describe the solution you'd like
it will be good to have scrolling if the date is open, what i mean by scrolling is when the calendar open, we can move the datepicker using scroll and the date will move by month, i.e : scroll up will bring the datepicker move to previous month and vice versa.
Additional context
will be usefull rather than have to click next or previous button

Interesting idea, will add it in the next version.

Added in v2.3.6

i just notice that the scroll is on wrong behaviour? isn't scroll down supposed to go to next month and otherwise?

@Jasenkoo

Yeah, but logically speaking, you go next month, meaning up + 1 and vice versa, I'll make an adjustment or config in the next version

Yeah, but logically speaking, you go next month, meaning up + 1 and vice versa, I'll make an adjustment or config in the next version

make sense. i think give a config is a good solution


no notification sound

it doesn't give sound when showing  notification
is there a way to enable it?
os: archlinux
de: kde plasma

It works as expected here on archlinux snap install for me.
Make sure the notification sound is enabled in whatapp settings.

yeah it works on snap only
not work as flatpack oe from AUR
the same for my ubuntu device notification sound doesn't work with flatpak
snap is sucks, slow hangs on sending messages


Update Charm++ FAQ

Original issue: https://charm.cs.illinois.edu/redmine/issues/1482

The “Charm++ FAQ” really needs to be updated. It is outdated in many ways: mentions of old/removed machine layers, no mention of relevant newer features, discourages use of some C++ features that are now standard/portable, and in its answer to the question “Does anybody actually use Charm++?” it only really mentions NAMD.
Go through the whole thing and update it, removing things that are no longer relevant:
http://charm.cs.illinois.edu/manuals/html/faq/manual-1p.html

Original date: 2017-04-13 15:14:12

Original date: 2017-04-18 19:46:45

Charm++ tutorial webpage had this question that also can be added to Charm++ FAQ page:
"Is there syntax highlighting for Charm++ ci files?
Yes, there is for vim editor.
$ cp charm/contrib/ci.vim ~/.vim/syntax/.
$ vi ~/.vim/filetype.vim
paste this in the file:
au! BufRead,BufNewFile *.ci set filetype=ci
"


Assistant Threads

This adds limited support for the Assistants API: we'll send an event to Libretto for each message in a thread.
The event recording happens asynchronously whenever a run is created, currently only via createAndPoll but we'll need to inject the RunObserver into other run creation methods in the future.

LGTM


[Gateway] : 2.5 UI feedback reported

gateway_boba_feedback.pdf

Closing as duplicated with https://github.com/bobanetwork/gateway/issues/32


After 5 minutes socket.io loses token

After ~ 5 minutes (in Chrome browser) socket.io loses token.
Affected version: v0.4.1_rc2

Observations:

last data viewed

session renewal

unknown token

The problem was that the web socket connection was being terminated by timeout, so a new one-time token connection was not generated, so real-time data was no longer retrieved.

When the token expired, the real-time data was not updated.
With token renewal, the update is resumed.

Tested in version: v0.4.2_rc.1


willScrollToItem delegate method is not getting called.

open class PagingViewController<T: PagingItem>:
is not declaring the optional function
    @objc  optional func em_pageViewController(_ pageViewController: EMPageViewController, willStartScrollingFrom startingViewController: UIViewController, destinationViewController:UIViewController)

Which require overriding the function in FixedPagingViewController:
open override func em_pageViewController(_ pageViewController: EMPageViewController, didFinishScrollingFrom startingViewController: UIViewController?, destinationViewController: UIViewController, transitionSuccessful: Bool) `

only then will the delegate method willScrollToItem be called.

Which version are you using? This should be fixed in #63 which was released in v0.7.0.


Issue Report: /en-us/stores/open-api

** Describe the bug **
When using the Store Insights Api, we were unable to pass more than one metric as a parameter in the request (error attached).
Content URL
url = "https://advertising-api.amazon.com/stores/######/insights"
DOC: https://advertising.amazon.com/API/docs/en-us/stores/open-api#tag/Stores-Analytics/operation/getInsightsForStoreAPI
Description of what the documentation currently says
Array of strings (InsightMetric)
Items Enum: "VIEWS" "ORDERS" "UNITS" "SALES" "VISITS" "VISITORS" "SCORE_LEVEL" "PERCENTILE_IN_PEERSET" "RECOMMENDATIONS" "CONTRIBUTORS" "DWELL" "PEER_DWELL"
List of the insight metrics to be fetched. Only one metric should be specified.
** Description of what the documentation should say **
We would like the return of more than one metric to extract the report.
** Screenshots **

Additional context
Simple python script of the request where the error occurs:
import requests
import json
url = "https://advertising-api.amazon.com/stores/######/insights"
payload = json.dumps({
"startDate": "2023-09-01",
"endDate": "2023-09-30",
"metrics": [
"VIEWS",
"ORDERS",
"UNITS",
"SALES",
"VISITS",
"VISITORS"
]
})
headers = {
'Content-Type': 'application/vnd.GetInsightsForStoreRequest.v1+json',
'Accept': 'application/vnd.GetInsightsForStoreResponse.v1+json',
'Amazon-Advertising-API-Scope': '1348112487580090',
'Amazon-Advertising-API-ClientId': 'amzn1.application-oa2-client.73b74b0f4211432aa311926ee842852a',
'Authorization': 'Bearer ####'
}
response = requests.request("POST", url, headers=headers, data=payload)
print(response.text)

Hi @diegovsilva, thanks for providing this feedback. At this time, the API does only support a single metric in the request, so this is working as designed. I will pass this feedback on to the product team, and if any changes are made, we will post in the release notes.


exploring gas of using oz

after

before

Ran forgeup as I know they just shipped some gas improvements
before

after

@xenoliss it would be good if we have an upgradeToAndCall test so we could also see before and after gas there

before

after

Alchemy recently used Solady UUPS with OZ Clone, we could do the same https://github.com/alchemyplatform/modular-account

Interestingly if I had Solady UUPS back in the gas doesn't get much better, investigating

Not pursuing for now


Move csi-data-dir configuration into values.yaml.

This allows a different hostPath to be used, working around an issue with system-tmpfiles-clean task.
Resolves https://github.com/cert-manager/csi-lib/issues/18

/ok-to-test
/approve
Leaving to @JoshVanL to lgtm 😊
We should also consider changing the default for this in future to eg /var/run/cert-manager-csi which I think is a more appropriate location (given we are pre 1.0!)

We should also consider changing the default for this in future to eg /var/run/cert-manager-csi which I think is a more appropriate location (given we are pre 1.0!)

Is this something you would like to see changed in this PR? Is changing the directory during the lifetime of the host instance a bad thing?

We should also consider changing the default for this in future to eg /var/run/cert-manager-csi which I think is a more appropriate location (given we are pre 1.0!)

Is this something you would like to see changed in this PR? Is changing the directory during the lifetime of the host instance a bad thing?

Don’t think I’m against moving the default out of tmp, but yes, this would be a breaking change and disrupt or likely entirely bork an existing installation on upgrade. Given that change would need some further testing, some decent docs + warning to users of this change, (perhaps even some code change to handle a blank dir with missing volumes more gracefully), let’s leave it out for now and just make the field configurable. Changing the default would warrant a minor version code release, but this doesn’t.
@basert, could you increase the version in Chart.yaml to ‘v0.2.1’, not appVersion, and I’ll release a new helm chart with this change once it is merged?

Ok, makes sense.
I updated chart version and used https://github.com/norwoodj/helm-docs/releases/tag/v1.5.0 to generate the new readme/adjusted the comment in Values.yaml.

Thanks for the work @basert :smile:
Expect the chart to be published shortly.
/lgtm


Sync compiler-0.995 with master

Purpose
Sync compiler-0.995 with master

Codecov Report

Merging #15184 into compiler-0.995 will decrease coverage by 41.88%.
The diff coverage is 0%.

@@                  Coverage Diff                  @@
##             compiler-0.995   #15184       +/-   ##
=====================================================
- Coverage             65.15%   23.27%   -41.89%
- Complexity              544     4996     +4452
=====================================================
  Files                  2256     1304      -952
  Lines                100064    65795    -34269
  Branches              13239     8903     -4336
=====================================================
- Hits                  65198    15311    -49887
- Misses                29788    48796    +19008
+ Partials               5078     1688     -3390

Impacted Files
Coverage Δ
Complexity Δ

...wso2/ballerinalang/compiler/tree/BLangService.java
0% <ø> (-77.15%)
0 <0> (ø)

...n/java/org/ballerinalang/model/types/TypeKind.java
0% <ø> (-100%)
0 <0> (ø)

...alang/compiler/semantics/analyzer/SymbolEnter.java
0% <ø> (-83.51%)
0 <0> (ø)

...c/main/java/org/wso2/ballerinalang/util/Flags.java
0% <ø> (-87.5%)
0 <0> (ø)

...ang/compiler/semantics/analyzer/TaintAnalyzer.java
0% <ø> (-81.44%)
0 <0> (ø)

...allerinalang/compiler/tree/BLangErrorVariable.java
0% <ø> (-72.73%)
0 <0> (ø)

.../compiler/semantics/analyzer/DataflowAnalyzer.java
0% <ø> (-80%)
0 <0> (ø)

...alang/compiler/tree/expressions/BLangConstant.java
0% <ø> (-54.29%)
0 <0> (ø)

...allerinalang/compiler/tree/BLangInvokableNode.java
0% <ø> (-82.36%)
0 <0> (ø)

.../ballerinalang/compiler/tree/BLangNodeVisitor.java
0% <ø> (-0.6%)
0 <0> (ø)

... and 1659 more

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 a2d353c...552da8d. Read the comment docs.


socket hang up

I'm trying to send an attachment via mailgun-js. When there were no attachments, it worked fine, but now I'm getting ECONNRESET:
Error: socket hang up
        at createHangUpError (_http_client.js:209:15)
        at TLSSocket.socketOnEnd (_http_client.js:294:23)
        at emitNone (events.js:72:20)
        at TLSSocket.emit (events.js:166:7)
        at endReadableNT (_stream_readable.js:905:12)
        at nextTickCallbackWith2Args (node.js:455:9)
        at process._tickCallback (node.js:369:17)

Nothing appears in the mailgun dashboard logs.

All tests pass and attachments work fine in my test code. Please provide sample code that reproduces the problem.

It stopped happening after I started using streams to create the attachments (although I now have a different problem).

I'm constantly having the "Socket hang up" problem and it seems like the problem occurs randomly - a test script run several times can bear different results.

i get the same issue, need help

Same issue here, any help?

I had the same issue, I was getting a remote file as attachement
const attachement = request('https://www.google.com/images/branding/googlelogo/2x/googlelogo_color_272x92dp.png')

And I was returning the datas I had to send with mailgun using a function.
To fix it, you should use mailgun.messages().send(message) in the same function that you instanciate request


Add Windows ddev bash completion capability to installer

Is your feature request related to a problem? Please describe.
Bash completion is a standard thing in any shell environment. Everybody depends on it. homebrew/Linuxbrew installs set this up automatically (except people have to configure their profile to make sure it's included).
However, we've never provided this explicitly for Windows... mostly because I didn't know if could work. But it works fine. There's just not a global technique to bring in the completions (that I know of).
Describe the solution you'd like

ddev generates a ddev_bash_completion.sh for each release. It's bundled with each tarball.
That can actually be placed anywhere, but probably the best place is /etc/bash_completion.d/ddev, (/etc/bash_completion.d may have to be created). (This assumes the use of git-bash)
The user has to load it in their .profile or .bash_profile or wherever. source /etc/bash_completion.d/ddev - so we'd have to prompt them to do that.

I think the installer could do this pretty easily. It's conceivable that the installer could go looking for the .profile or .bash_profile, but IMO it's best to prompt people to do this.
I think this would be a big step forward for ddev on Windows.

Note that the ddev release process now also adds completion scripts for PowerShell and Zsh, so there may be even more possiblities.

I think git-for-windows has a default location for bash completion.
When I do a grep I see:
$ grep bash_completion.d /etc/ -R
/etc/profile.d/git-prompt.sh:   for c in "$HOME"/bash_completion.d/*.bash

So I placed a ddev.bash file there and it works...
I failed to find documentation for this...

So you put it in a directory named ~/bash_completion.d ? Interesting, thanks!

I did find that git-bash as a login shell would load ddev.bash from ~/bash_completion.d
In cmder I had to re-run bash -l - I haven't explored why that happens.

I think what we have is good enough, and we're likely to discontinue the installer anyway.


Run entities-service validate as a pre-commit hook

Closes #122
Add a pre-commit hook with id validate-entities.
It runs on all known file formats as well as with --verbose.
One can add extra options through the pre-commit args.
Finally, in its current version, one must add ``.[cli]'to the pre-commitadditional_dependencies`.
A separate test job has been set up to test the hook.

Important
NB: This will deprecate the --file/-f and --dir/-d options from the validate and update commands in favor of a SOURCE argument, which may be given multiple times.

Note: The commits have been squashed down to two commits, each representing the introduction of SOURCE (and deprecation of --file/-f and --dir/-d) and the new pre-commit hooks validate-entities, respectively.

The two commits 7bd888a and 1ced968 were what made up this PR.
Due to an extra merge commit in this PR branch' HEAD, GitHub could not recognize that this PR has been merged into main. Instead, when rebasing these commits onto main, it was found locally that this would result in a noop (indeed ...), and after pushing that, the PR has been closed.
TL;DR: This PR was successfully merged (not closed), adding the commits 7bd888a and 1ced968 to main.


Hotfix: Broken link

First link to FeatureTools site in https://github.com/Featuretools/predict-next-purchase/blob/master/Tutorial.ipynb  leads to https://render.githubusercontent.com/view/www.featuretools.com and is broken.

Fixed. Thank you.


Icon works but macro doesn't

The icon works perfectly but I can't get the macro to work.
Right now it is posting the string in chat.
I was wondering if I could get an example macro to cross-reference.
I'm also not sure if I should replace the "Ardittristan" portion with my user name or if that's calling a part of the module.
Foundry Tabletop - Version 10
Build - 291

The name argument is mandatory
window.Ardittristan.InlineViewer.sendUrl("https://example.com", false, 512, 512, "test")


priceUsd of imported poignArt donations is invalid

I figured it out that donaiton has invalid price
query:
select id,"priceUsd",amount,currency, "priceEth", "valueEth" from donation
where "donationType"='poignArt'

Produciton Donation

Staging donation

Can we move this issue to Done? @mohammadranjbarz

@maryjaf Yeah it's fixed now


Appveyor with node 4.1.0 and npm@3

[x] running IE test suite on Appveyor
[x] using same TSD_GITHUB_TOKEN for Tsd fetches as travis build

This fixes #202
NOTE: Administrators should log into http://appveyor.com and add angular2-seed project in order to connect appveyor to the github repos

Looks great! Tomorrow I'll review and merge it.

@mgechev, @ludohenin  you need to log into www.appveyor.com and attach our github repository to appveyor. Just like Travis, a Github administrator has to do this. Please make sure you do it, or Appveyor will never build.

@jesperronn done!


bug: getName not resolving correct for some addresses

Package Version
3.2.0
Current Behavior
The getName function correctly resolves addresses for some ENS names (e.g., renanmav.eth and graynewfield.eth) and fails for others (e.g. alvaroraminelli.eth)
Expected Behavior
getName for the following names:
renanmav.eth
alvaroraminelli.eth
graynewfield.eth
Link to Minimal Reproducible Example (StackBlitz, CodeSandbox, GitHub repo etc.)
https://stackblitz.com/edit/viem-getting-started-uwm8cj?file=package.json,index.ts
Anything else?
No response

already discussed in https://github.com/wevm/viem/issues/1753


gyp: improvements for android generator

When generating Android.mk files for HOST binaries, we need to specify
the LOCAL_CXX_STL to use when linking.
Also, set the variable GYP_HOST_MULTILIB to 'first'. This is used as
LOCAL_MULTILIB, and since we don't provide flags for both types of
builds (32-bit and 64-bit), the build system will try to build both and
fail.
More details about LOCAL_MULTILIB can be found here:
https://source.android.com/source/64-bit-builds.html
Signed-off-by: Robert Chiras<EMAIL_ADDRESS>

cc @bnoordhuis?

Rubber-stamp LGTM.  Landed in 7f1c1b9, thanks Robert.


Fix for ruby 1.9.3

Hi!
I fixed post/get errors. It looks like those methods were backwards compatible till 1.9.2, but since 1.9.3 they are not: http://ruby-doc.org/stdlib-1.9.2/libdoc/net/http/rdoc/Net/HTTP.html#method-i-post
"In version 1.1 (ruby 1.6), this method returns a pair of objects, a Net::HTTPResponse object and an entity body string. In version 1.2 (ruby 1.8), this method returns a Net::HTTPResponse object."
I replaced all occurrences of:
resp, data = http.post(args*)
by:
resp = http.post(args*)
data = resp.body
etc. It fixes nil data errors.
Regards,
Mateusz

thanks! give me until the weekend to pull this. super busy here

No problem. Thanks for the answer. I added 2 more commits. Will you have any questions when checking my changes, do not hesitate to contact me. Regards.

sure, i'll take a look at the commits. once again thanks!

merged


CoreML could not detect object in iPhone camera's portrait mode

I successfully trained an Object Detection model and exported in CoreML format.
My model has 300 iterations and mean_average_precision is about 0.7.
I then validate it with some images using TC and bounding box drawing util and it could recognize the object pretty well.
I then downloaded the sample project for recognizing objects in live capture here:
https://developer.apple.com/documentation/vision/recognizing_objects_in_live_capture
What it's weird is that the object couldn't be detected in iPhone camera's portrait mode, no bounding box is drawn, VNDetectedObjectObservation hasn't returned any results. However, I rotated the iPhone 90 degrees counter-clockwise((home button on the right) and the bounding box appeared. If I move the phone horizontally, the offset between the bounding box and the object become larger.

I'm not sure that it could be a problem of Turi Create or Core ML, or the source code itself. Could anyone explain to me, please?

Hi @kid9591, this is almost certainly because of the way the iPhone captures photos and then associates EXIF data describing the phone's rotation at capture. It's likely that the photos you trained with had a different orientation than the image being used for inference.
You can view the EXIF data of a photo in Finder on macOS - this should help reveal whether the images you used were "rotated" when training the OD model.

Possibly related to #1147.

RectLabel is an offline image annotation tool for object detection and segmentation.
Although this is not an open source program, with RectLabel you can remove the EXIF orientation flags from images and convert images to the front orientation. If necessary, you can rotate the image right/left by 90 degrees.
You can export/import the Turi Create CSV format.
https://rectlabel.com/help#remove_exif_orientation_flags


Add uv as package manger for the generated project

The project itself now uses uv and this is an attempt to bring uv to the generated project too.
All checks are green but heroku and the various CI providers haven't been tested.
Generating the docs with make html requires activating the virtualenv manually.
I used the watch directive to prevent the .venv being overridden in docker but I didn' test the production image.
Feel free to contribute to this draft.

Related to #3083 #1988

Ready for testing :tada:

Hi, there. I played around with using your fork to update some existing projects and eventually got them working but ended up reverting them to their production (pip) versions so that I could revisit when I had more time.
When working on an adapting an existing project, I got an error during the build, which was reproduced when I built a project from scratch using cookiecutter https://github.com/foarsitter/cookiecutter-django --checkout uv-generated-project:

[django python-build-stage 4/6] RUN --mount=type=cache,target=/root/.cache/uv     --mount=type=bind,source=uv.lock,target=uv.lock     --mount=type=bind,source=pyproject.toml,target=pyproject.toml     uv sync --frozen --no-install-project --no-dev:
0.112 Using CPython 3.12.7 interpreter at: /usr/local/bin/python
0.112 Creating virtual environment at: .venv
0.113 error: Could not find root package my-awesome-project
'

I believe that it's caused by having the blank uv.lock. I thought since the local project had the host file rw that it would generate a usable uv.lock during the local build. Is there a way to get a usable version of uv.lock on the host filesystem without running uv sync on the host filesystem? Thanks for this--very cool!

@steveputman thanks looking into this!
During a build you cannot write to the host system.
Your interpreter is inside the container together with uv so you can run uv:
docker compose -f docker-compose.local.yml run django uv lock

Struggled a lot with this and it is not ideal. So hopefully someone has a better solution.

@foarsitter
The Docker documentation was sort of vague on that, but now I know.
Your solution is better than mine (I had uv installed on my host system so just ran uv lock in the project directory.
Thanks for the help! Will let you know if anything else comes up.


Wrecks current Brackets install, have to delete and reinstall and set-up. DON'T INSTALL!

This is the third time I've installed Brackets, got all the extensions I wanted and set them up, and then everything goes to hell when I install your Tabs extension. Even after removing it, the issues persist.
Does your code modify native Brackets code? Seriously, you need to put a disclaimer that people's stuff is going to get wrecked if they use this.
Running: Brackets - Release 1.7 build 1.7.0-16898 (release b0a363b71)
System: Mac OS X El Capitan
Below is a screenshot of the console output. I have spent hours trying to fix this already, and do not have the time to go any further.

Like I said, the issues only start once your extension is installed, but it does NOT go away after disabling or removing it. This extension needs to be taken down immediately. People's time is valuable, they don't need to waste it having to reinstall and set-up their coding environment.

errors are caused by another extension, I'm using on multiple computers and there is no such problem, anyway I try to find a solution...

I installed Brackets 1.6 (same Mac OS X El Capitan system), and there were no issues with the tabs. It's something to do with the new release. I'd have kept the earlier version, but it is lacking some features that I switched editors specifically for.
To be fair to you, every tabs extension causes the same issues. But yours is prettier than the others, so I spent more time with yours, trying to make it work.
It's incredible how many extensions have not been updated for the current build. Almost all of them were giving warnings - at the minimum - but many were throwing fatal exceptions. All of the other ones were easy enough to modify, but yours will either have to be a) a major overhaul of your code, or b) a patch made for Bracket. I imagine the former is the more likely.
I might make a new issue for Brackets.io, since it's every tabs extension that is causing this issue. What really needs to be done is a build of Brackets needs to be put together that gets rid of the "Working Files" bit, and replaces it with proper stacked tabs like every other editor.
Wish I could help you put together a new tabs extension, but no time at the present.

:+1:  great review, I am delighted.
try to remove this extension:

brackets.app.showOSFolder(brackets.getModule("extensibility/ExtensionManager").extensions.NavigationAndHistory.installInfo.path);

then install brackets 1.7 and then see if the problem continues
I also modified other extensions to solve compatibility.

I'm going to have to skip trying this out for a while. By the time I install it, get all the extensions I want, organize their settings, modify the majority of them to not throw warnings or errors, modify multiple files to get the hotkeys set properly, etc, etc... It takes the better part of a work day to fight with everything. I'll let you know if I ever get the time for it, but for the next couple months I'm slammed.
Good luck though.

:+1:


Add support for targeting PHP 8.3

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Warning: This coverage report may be inaccurate.
We've detected an issue with your CI configuration that might affect the accuracy of this pull request's coverage report.
To ensure accuracy in future PRs, please see these guidelines.
A quick fix for this PR: rebase it; your next report should be accurate.

4 of 6 (66.67%)  changed or added relevant lines in 4 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.02%) to 84.245%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

src/Lib/PhpInternals/Opcodes/OpcodeV83.php
0
2
0.0%

Totals

Change from base Build<PHONE_NUMBER>:
-0.02%

Covered Lines:
4866

Relevant Lines:
5776

💛  - Coveralls


Script parsing fails when two scenes refrence eachother

Godot version
v4.1.2.stable.mono.official [399c9dc39]
System information
Kubuntu 22.04
Issue description
If two packed scenes reference each other gdscript parser fails with..
E 0:00:00:0350   _parse_ext_resource: res://scenes/menus/menu-options.tscn:31 - Parse Error: [ext_resource] referenced non-existent resource at: res://scenes/menus/menu-main.tscn
  <C++ Source>   scene/resources/resource_format_text.cpp:162 @ _parse_ext_resource()

I suggest that PackedScenes be lazily evaluated in the same way if they were just a string path.
Steps to reproduce

Make two scenes
write a script that exports packedscene
assign it to each scene
assign scene a to scene b's export
assign scene b to scene a's export
notice you get a parse error

Minimal reproduction project
DumbSceneBug.zip

Duplicate of a multitude of cyclic reference issues

I am unsure if I should close this.

See:

https://github.com/godotengine/godot/issues/80877

Think this matches this:

https://github.com/godotengine/godot/issues/80043

@AThousandShips looks like the godot team are not planning on fixing this and just are just changing the error that is triggered.
That doesn't really fix the issue.
Allowing scenes that reference each other is a very important thing to have

Then I'd suggest you state your opinion on those issues and tracker instead of here, as this is just one of many duplicates or tangential issues

Just putting it here for future reference, I had a cyclical preload statement, (https://github.com/godotengine/godot/issues/70985), but I received the error listed here (... referenced non-existent resource at ...). If anyone is also stumbling into this issue, you might have a cyclical preload, which can be fixed by changing the preload call into a load call.

This is more of a core problem than a GDScript problem. The resource loading system does not support cyclic dependencies.


Build on base landing

📝 Changed routes:

/build/base

(view full size)

Commit deea09bb481365a5eed1d63644853934ac6eeb05 (https://thirdweb-fra5xh5ko.thirdweb-preview.com).

shouldn't the URL be /network/base so we have the same structure as /network/solana?

shouldn't the URL be /network/base so we have the same structure as /network/solana? or is it SEO reasons?

no, it's not a network page


Listener type definition

Current listener type is defined as follows:
listener: Box<dyn Fn(UMessage) + Send + 'static>

From the current definition, I foresee two main issues:

The callback only accepts a UMessage. It is therefore impossible for the uTransport concrete implementation to notify the upper logic about errors.
The callback has the following trait bounds: Send + 'static. However, being Sync not required, it may be an issue in multi-threaded scenarios. E.g. on zenoh the subscriber callback is expected to be Sync because the async executor is multi-threaded and the callback could be called from different threads every time.

What I'd like then to discuss in this issue is the possibility to modify the listener type to the following:
listener: Box<dyn Fn(Result<UMessage, UStatus>) + Send + Sync + 'static>

I'm open to discussion.

FMPOV using Fn(Result<UMessage, UStatus>) shouldn't be a problem at all.
Regarding the adding of the Sync Trait bound:
In general this seems to make a lot of sense. However, I will need to make some practical experiments before issuing an educated opinion :-)


Add BTU (BTU)

Currency name: BTU
Symbol: BTU
Website: https://btu-protocol.com
Theme color: #a47fb7
Logo (SVG): https://btu-protocol.com/images/tokenBTU.svg

Hello @flerunigo, I'm trying to help you out here. You are supposed to upload the SVG icon in a .zip file. I think these should be the correct values:
Add BTU Protocol (BTU)

Currency name: BTU Protocol
Symbol: BTU
Website: https://www.btu-protocol.com/
Theme color: #007bff
Logo (SVG): BTU-Protocol-logo.zip

Hello @Paulsky. Indeed these are the correct values. Sorry for not zipping the icon.

@flerunigo No worries, just wanted to help you out!

@flerunigo There is no circle on the website around the icon. Should we keep it or not? If not do you have an updated SVG file? Also which is the correct color? Thank you.

@flerunigo Any news on this?

Closed for lack of activity.


MINOR: Add System Properties to config documentation section

Kafka repo update  for https://github.com/apache/kafka-site/pull/512/

Merged to 3.5 and 3.4 branches


Can Valibot Detect the Absence of a Key in an Object Without Specifying it in the Schema?

I encounter the "Invalid Type" error while working with this code:
import { email, minLength, object, type Output, parse, string } from 'valibot'; // 0.87 kB

const schema = object({
  url: string(),
});

parse(schema, { });

In reality, the 'url' key and its corresponding value are absent. Is there a way to generate a "missing url key" error? If feasible, I'd prefer not to include it in the schema (for cleanliness).
Can valibot detect the non-existence of the key within the object?

Yes try out the following code:
import { object, parse, string } from 'valibot';

const schema = object({
  url: string('"url" is missing or invalid'),
});

parse(schema, {});

I assume that this issue has been resolved. If not, let me know.


fixLateralViewRef's toSql method is not correctly implemented

Proposed changes
Issue Number: close #xxx

Further comments
If this is a relatively large or complex change, kick off the discussion at<EMAIL_ADDRESS>by explaining why you chose the solution you did and what alternatives you considered, etc...

run buildall