ImageMagick fails on Lambda but not locally

When I test locally it successfully pulls from bucket A and puts my resized images in bucket B. I have looked all through the stack and my local seems to match up to Lambda but when I run it on lambda (same example.json file) it fails out with this error:
{
  "errorMessage": "Woops, image process failed: ImageMagick errError: Command failed: convert: Expected 8192 bytes; found 5848 bytes `-' @ warning/png.c/MagickPNGWarningHandler/1777.\nconvert: Read Exception `-' @ error/png.c/MagickPNGErrorHandler/1751.\nconvert: corrupt image `-' @ error/png.c/ReadPNGImage/3789.\nconvert: no images defined `png:-' @ error/convert.c/ConvertImageCommand/3046.\n"
}

I checked the byte size locally, on s3, and on Lambda and they are all the same even though ImageMagick sends a corrupt image error. Any ideas?
config.json:
{
  "bucket": "example-bucket",
  "resizes": [
    {
      "width":  110,
      "height": 145,
      "directory": "small"
    },
    {
      "width":  151,
      "height": 200,
      "directory": "medium"
    },
    {
      "width":  450,
      "height": 350,
      "directory": "large"
    }
  ]
}

Its due to filesize. For some reason a 4 mb file is too big.


Visabilty issues on preferences/advanced window under GTK

The following screenshot shows the issues on the advanced window:

The height of wxChoice is too low (e.g. "Unix (recommended)" is cut at the bottom)
wxSpinCtrl is not wide enough under GTK to show the number after "Wrap at:"
The font is very small (see: These settings affect ...)

This is an example how it looks like after applying commit
https://github.com/c72578/poedit/commit/8dca5cb990ea5fd9983e07ecf8176d6e642d4067

Poedit: 1.8.6
OS: Fedora 21
GTK: 3.14.15
wxGTK3: 3.0.2
gnome-shell: 3.14.4

So far, the following has been implemented, in order to improve readability:

Adjust wrap width wxSpinCtrl width under GTK+ 3
https://github.com/vslavik/poedit/commit/21ab52731265549aff8ab0d198c25c0579a8b88b
Use correct explanation text size even with wxGTK 3.0
https://github.com/vslavik/poedit/commit/747e3a7920e5c882c0ba24c06256639c874900f7

Open, to be tested:

Height of wxChoice in latest wxGTK from git

Here is an update, on how it looks in the meantime using wxGTK 3.0.3:

Poedit version: 2.0.2
OS: Fedora 26 x86_64
wxGTK3 version: 3.0.3-2
When Poedit 2.0.2 is built against  current wxGTK 3.1 (master, commit 3105134), it looks the same:


Support for externalbrowser authenticator via env var

Hello!
Our company uses SSO for all of our employee snowflake identities. We login by specifying our login_name and set the authenticator parameter in the snowflake connector parameters to externalbrowser. This PR was all that was needed to get Titan to be able to use our accounts on the CLI.

LGTM! Thanks for the PR.


Homework3: Ekaterina Zavadskaya

Do homework3
Edit README.md, create branch feature/homework3
--
KateZavadskaya

@Stml89  it seems I am sucessful :)

Approved, now you can merge PR


unmapped reads

debarcer is not currently filtering out unmapped reads. This introduces a bias in QC plots and artificially inflate read depth per umi group.
It seems logical to filter out unmapped reads, and keep the aligned mate (when appropriate) since debarcer is not using information about paired reads during grouping or collapsing.

Unmapped reads are skipped during Grouping. Collapsing already loops over pileup objects which only include mapped reads. The number of mapped and unmapped reads per genomic interval is recorded in the Stats directory, which are then used for QC plots.


DF does not respect escape quotes

Google sheets and many other spreadsheet editing tools escape commas contained in cell values by wrapping the record in double quotes.
DF currently throws a Range Error if a row contains a record with a double quote escaped comma. DF should respect double quote escaping.

Currently working on a solution to this in my own fork, will submit a pull request and link to it from here once it's ready.

Created PR #6 to address this issue. Will close once the PR is merged.

PR was merged!


Keep a changelog

By keeping a changelog, it's must easier to figure out what changed and if it's worth upgrading / if it's risky upgrading.

+1

The release notes is on the website, linked from the download page.

Here is the upcoming version

This does not feel right, as website only reflects 2.0 changes. What about next branch (3.0)?
It is way better to maintain change-log per version.


removed unwrap and (most) expects from the project

Refactored code
I am using the following clippy lints to remove some code smells:
cargo clippy -- -D clippy::complexity -D clippy::correctness -D clippy::perf -W clippy::suspicious -A clippy::enum_variant_names -W clippy::expect_used -A clippy::unwrap_used

removed bodies from mock functions
removed unwraps/expects (expects relating to unset environment variables are still present, as well as errors that mange to propagate to main)
functions that used unwrap/expect have been changed to return a Result so that the caller will handle any possible errors

The other PR had a problem with the changes from main, so i made a new branch 🤷♂


Remove unused import from Faker.Internet

With this unused import being present in Faker 0.8.0, I see this warning when I run mix deps.compile:
warning: unused import Faker
  lib/faker/internet.ex:2

I've added:

[ ] ~USAGE.md docs if applicable~
[x] CHANGELOG.md

Hey @radar thanks for contribution, this is already done in #88, I think it'll be fair to leave it in it and close this one.


shape.movedByCamera and text.movedByCamera

shape.movedByCamera and text.movedByCamera are now reset by the state manager each frame.

Made the requested changes.


Sales channel URLs are required to be unique - however with the store-api (and in other scenarios) they do not need to be

PHP Version
8.2
Shopware Version
6.5
Expected behaviour
For Storefront it makes sense that the domains are unique. However, even in that case, they might be targeted differently, e.g. using the browser IP or user login etc.
With the store-api the sales channel is mostly defined by the access key, so e.g. a domain could be reused for multiple sales channels, e.g. in combination with registration double opt-in. However, it is not possible to specify such domains as the url has a unique index in the database.
Maybe limit the uniqueness to storefront sales channels or as mentioned even there you could add a possibility to find those sales channels differently as well.
e.g. quite often B2B shops use different sales channels after a user logs in and nothing is available beforehand.
Actual behaviour
Currently there is a unique index on the database, so it makes it impossible to reuse the same URL for multiple sales channels.
How to reproduce
see above.

[public] Automated response: This issue is linked to the internal issue https://shopware.atlassian.net/browse/NEXT-37824.
[created from NEXT-37824, comment 488338]


Added wai-routes benchmarking, updated package versions in the README

I added benchmarking code for wai-routes framework. I also updated the README to refer to the wai-routes code. I haven't checked in any benchmark results because those probably vary from machine to machine. Please let me know if some parts of the results need to be checked in.

I ran the benchmarks with updated versions of the other web frameworks, so I changed those versions in the README. It's quite probable that the latest version of wai-routes (v0.7.0 as of now) will not work with older versions. I did not however change the results table in the README to incorporate results for wai-routes.

Thank you very much:smile:
I updated short benchmark result(table only). Please check it.
I'm going to update full benchmark results soon.

Thanks! Looks good!


Solax: add battery control

Fix https://github.com/evcc-io/evcc/issues/11227

@andig getting an decoding error on starting, think it has something to do with your change.
I did not change anything in the config, does not work since the last update:
FATAL 2024/01/01 13:59:27 cannot create meter 'battery': cannot create meter 'template': cannot create meter 'custom': battery mode: 1 error(s) decoding:

error decoding 'Timeout': time: invalid duration ""

meters:
...
- type: template
  template: solax
  id: 1
  device: /dev/ttyUSB0
  baudrate: 19200
  comset: 8N1
  usage: battery
  modbus: rs485serial
  name: battery

Im nightly behoben.


Change remaining uses of typeof to typeof

@erikdoe
This pull request changes the remaining uses of typeof to __typeof__ to allow compatibility with all C, Objective-C++, etc files in addition to Objective-C files.
__typeof__ is simply an alternate keyword for typeof. This change should be very safe. See this link for an explanation on why this is needed:
https://gcc.gnu.org/onlinedocs/gcc/Alternate-Keywords.html#Alternate-Keywords
I'm also requesting that the pod spec's patch version be bumped; the repo tagged; and pushed to trunk.
This change will allow several teams across our company to adopt the OCMock 3 syntax, and we very much would appreciate its inclusion.
Thanks again for the great work on OCMock, Erik!

Joshua

Closing due to inactivity. The changes proposed should be in master.


os-ddclient: unable to use webservice to check IP of various interfaces

Important notices
Before you add a new report, we ask you kindly to acknowledge the following:

[X] I have read the contributing guide lines at https://github.com/opnsense/plugins/blob/master/CONTRIBUTING.md
[X] I have searched the existing issues, open and closed, and I'm convinced that mine is new.
[X] The title contains the plugin to which this issue belongs

Describe the bug
my dns record is being updated with the local ip of my interfaces instead of the public IP as I have 2 "public" interfaces that have 2 different public IPs but are both behind a routher that does NAT
To Reproduce

setup a VPN interface to some cheapo VPN providing you some shared IP
use the old os-dyndns plugin to monitor that VPN interface for yourhost.example.com
you'll see a public IP beeing assigned to yourhost.example.com
now setup os-ddclient to monitor that same interface
you'll see the crappy privateIP assigned to you by the VPN provider beeing assigned to yourhost.example.com
If you're so lucky to have a VPN provider that enable you to do port-forwarding you'll see that yourhost.example.com:1234 will work at step 3 using the dyndns plugin but not at step 5 using ddclient...

Expected behavior
monitoring an interface when the "Check ip method" setting in the "General settings" page is set to anything that is a public webservice should result in a config file using the workaround use=cmd, cmd="<something intelligent>" or something better but not fallback to using use=if
Screenshots
Not applicable
Relevant log files
Not applicable
Additional context
To understand the context, I have 1 public fibre connection and 2 VPNs, all of which are on dedicated interfaces on the opnsense box, but NAT is happening between my opnSense box and the real internet public network somewhere at my ISP or at my VPN providers.
All of these do provide me with a single IP address and all ports forwarded properly, but the real public IP of each of these interface is not visible by opnSense and suceptible to change.
The previous os-dyndns plugin allowed me to select the interface I wanted and use any online service like checkmyip or similar to find out what is the real public IP of the interface.
Now, it is a limitation of ddclient, if the configfile says use=web, web=http://... you cannot specify which interface is going to be used to send out the http request... If you want to specify the interface as is now possible thanks to nice people solving the issue #2784 if will always have use=if which will return the IP of the interface which in my case is not a public IP address and not useful at all :(
The only way around this I see is to have use=cmd, cmd="<something intelligent>" and craft something intelligent to be able to perform an http request over a specific interface and parse the result.
I am not good enough to actually create that cmd to send the http request over a specific interface and parse it, nor to actually update the plugin ui to allow setting it up, so thanks in advance for any help you would kindly provide !
Environment
OPNsense 22.1.1_3-amd64
os-ddclient (installed) | 1.1

I have actually been able to make it work by manually editing the config file like this :
me@myopnsense:~ $sudo cat /usr/local/etc/ddclient.conf
daemon=300
syslog=yes                  # log update msgs to syslog
pid=/var/run/ddclient.pid   # record PID in file.
ssl=yes
verbose=yes

#
# setup how we expect to retrieve an IP address
#
use=web, web=http://ip1.dynupdate.no-ip.com/

use=cmd, cmd="curl --interface vtnet0 api.ipify.org", \
protocol=dyndns2, \
server=www.ovh.com, \
login=XXX, \
password=XXX\
first.example.com

use=cmd, cmd="curl --interface ovpnc1 api.ipify.org", \
protocol=dyndns2, \
server=www.ovh.com, \
login=YYY \
password=ZZZ \
second.otherexample.com

But that is because the folks at api.ipify.org simply return my IP and there is nothing to parse, other online providers might be less curl-friendly....
Anyway, if someone knows how to add this option (and check-ip-method ?) in the plugin and push it somewhere I'm all open to install it and test it ;) (will need directions if I need to install a plugin manually, I know only of the plus button to install from Firmware/Plugins page, sorry )

If the "checkip" service has rather static ip addresses, you may also add a static route to the desired location using the gateway in question.

There's another issue too. If you have the same hostname name for both IPv4 and Ipv6 you are stuffed. Check IP method needs to be per client and not global.

not sure about the IPv4/v6 stuff as I'm only using v4, but I agree that the ip check method needs to be per account.

I don't think ddclient supports that, but a manual configuration is always an option. Just  make sure to only install ddclient (without the plugin os-ddclient)

It can. It detects the IP version you are updating, and adjusts either the field type for the version to A or AAAA, I have a modified  ddclient where I have added godaddy. Now, if you use web to get the IP, I can use ipv6.nsupdate.info to get the IPv6 address or  ipv4.nsupdate.info to get the v4 address that works and updates correctly. I have also modded the ddclient plugin to allow me to select a different web update client for each entry, same hostname bit one is using v4 the other v6. My issue at present is having to mangle the  ddclient.conf, the one for the plugin not the daemon, to play nicely. Is there any reason we cannot use the real name of the web update site rather than the 'prettified' version that gets saved in the config? i.e.
use=web, web=http://ip1.dynupdate6.no-ip.com/ {%    elif checkip == 'web_nsupdate.info-ipv4' %} use=web, web=https://ipv4.nsupdate.info/myip {%    elif checkip == 'web_nsupdate.info-ipv6' %} use=web, web=https://ipv6.nsupdate.info/myip
if it could just use the real name of the checkip website in the dropdowns it's nice and easy. e,g.
`{%            elif account.service == 'godaddy' %}
use=web, web=https://{{account.checkip}}

It appears you are correct. It only updates the last entry in the config, you cannot spec the same client  twice using v4 and v6 so pants really.

I think your discussion merits another issue, mine here is mainly about the --interface to be used in the check ip mechanism that I would like some help in configuring to allow me to migrate from os-dyndns to os-ddclient.
I'm not willing to continue tweaking the config files manually if I can avoid it.
I have it working in the above example now please focus on questions or help in modifying the GUI to configure this method by account and not globally


common request function

https://www.npmjs.com/package/https-localhost
https://capacitorjs.com/docs/apis/http
https://vite-pwa-org.netlify.app/guide/service-worker-strategies-and-behaviors.html
https://www.npmjs.com/package/axios

fetch() - for WPA service worker
axois - biz logic

https://vueuse.org/core/useFetch/


Extend the ReportGenerator to load the test results as Earl assertions

Done by building triples in an in-memory repository. Output to build/karate-report/report.ttl initially. Other work will build HTML+RDFa outputs.


M3-4374 Email Bounce notification

Description
This adds a notification when an email to a user bounces.

How it works
The API already delivers the following (undocumented) notification types:

billing_email_bounce
user_email_bounce

1. billing_email_bounce
This occurs when an email to your account bounces – i.e. account.email. This can be resolved one of two ways, either:
a) PUT to /account with your existing account.email address (i.e. CONFIRM the address is correct)
b) PUT to /account with a different email address (i.e. CHANGE the address to correct it)
In Cloud, when they click "Yes, it's correct", we do Option A for them. If they click "No, let's update it", we take them to the Account page with the Contact Information Drawer open and the Email field focussed.
Note: there is currently an API bug where Option A doesn't actually clear the notification.
2. user_email_bounce
This occurs when an email to your user bounces – i.e. profile.email. This can be resolved like "billing_email_bounce" notifications are resolved (PUTting to /profile instead). When they click "No, let's update it." we take them to the Profile page with the Email field focussed.
It's possible (likely, maybe) to have both of these occur at the same time. In this case I opted to have two separate notifications.
Housekeeping
I created a new component, <GlobalNotifications /> to house both RegionStatusBanner and the new EmailBounce component.
TODO

[ ] write tests

Note to Reviewers
@WilkinsKa1 @tiffwong I know auto-focus inputs are generally not good re: accessibility, but this seems like a good use case. What are your thoughts?
Please reach out to me for instructions on setting up your dev environment to run this locally. I included a (commented-out) mock, but to really test this you'll need dev env running.
Lots of things to check:

Same email for both account and user
Different emails for account/user, one bouncing, one not, etc.
Restricted user with/without account access
Account and Profile page/forms working regardless of new changes
Failure scenarios

imo the boilerplate and handling to take care of the redirects/re-requesting notifications is bigger than the footprint of putting a single text input inside EmailBounce.tsx and PUTting from there. No sense doing this for the first pass, but I think it would improve the experience as a later enhancement

Good point, I don't disagree. Let's see how the demo goes.

If you're already on /profile, clicking the "update this" button doesn't do anything. Maybe it could scroll to the input? or disable the button? One reason why I think #2 would be helpful.

Just added some auto-scroll behavior to address this.

I'm not sure I entirely grok the API bug, but it seems that when it's fixed, pressing "Confirm" from either banner will dismiss the notification (both banners will disappear). In that case, maybe one banner would be better (if account/profile have the same email address?)

If the email addresses are the same, clicking the "confirm" profile email button should clear both notifications. I did experiment with one banner, but there are UX considerations (what if you need to change both account and profile separately?)

Maybe we can make the buttons smaller on mobile so they can sit inline with each other?

@WilkinsKa1  we'd have to adjust the text so they fit inline, maybe "Confirm" and "Update"?

Thanks for updating, looks good!


Remove symlinks to config files because they are mounted via ConfigMaps.

There's still a symlink remaining on the helm chart. Not sure how to get rid of that one but suggestions welcome.

I don't think it's possible to get rid of that symlink because the rendering of chart happens is actually done by Tiller, and Helm client only sends the chart to Tiller during helm install.

What if we move matchmaker_config.yaml from /config/ to /install/helm/open-match/config/?
It doesn't make much sense to have it stored out of the configmap since 'base' Docker image does not include this file any more. And there's not so much difference which file to edit to customize configuration. (Assuming that make gen-install will create a separate file for configmap in /install/yaml).
Moreover it feels to me that all the config properties should be a separate params to Open-match Helm chart (like openmatch.config.defaultImages.evaluator or openmatch.config.playerIndices). Then these keys could be used in configmap Helm template to produce the output. Thus we'd have both static configmap installation file and a flexibility to override default values during helm install without edits of any files.

I think splitting up the configuration has floated around but nothing has materialized from it. I've created issue, https://github.com/GoogleCloudPlatform/open-match/issues/130.

Also "defaultImages" in matchmaker_config.yaml should have the same version as in chart & install/yaml/. Either we need manage this with tags like "latest" or render these values with Helm template.

I'd like to take #130 which will probably need the merge of configuration from several sources which will likely require some changes in config.go. Maybe it's a time to merge this PR?


[PoE2] - Missing Anoints

Describe the problem
Dont have anoint stuff working probably? haven't gotten an anointed item so I cant test
To Reproduce
Steps to reproduce the behavior:

Go to '...'
Click on '....'
Scroll down to '....'
See error

Expected behavior

Screenshots
If applicable, add screenshots to help explain your problem. This is very helpful for comparing the PoE1 vs 2 problems

I need an item text or two that have anoints so i can test this.

 spoilers
I don't have any :( still act 3 cruel :(
I have poe2 running when doing dev stuff, so you can guess how much time has gone into this xD

Item Class: Amulets
Rarity: Rare
Vengeance Noose
Amber Amulet
Requirements:
Level: 65
Item Level: 82
Allocates Subterfuge Mask (enchant)
+14 to Strength (implicit)
+40 to maximum Energy Shield
24% increased maximum Energy Shield
25% increased Rarity of Items found
+24 to Intelligence
+45% to Fire Resistance
+40% to Cold Resistance

Item Class: Amulets
Rarity: Unique
The Everlasting Gaze
Azure Amulet
Item Level: 78
Allocates Piercing Shot (enchant)
25% increased Mana Regeneration Rate (implicit)
+50 to maximum Mana
50% increased Mana Regeneration Rate
Gain 23% of Maximum Mana as Extra Maximum Energy Shield
What they saw was what they believed, and
they believed Lunaris had not abandoned them.

by any chance how do I test clipboard items while in dev mode?

shift + space to open the overlay, on the main widget there is a paste field

Sorry I've been busy so I still haven't gotten to testing these yet

This is fixed in #170, and will be in next release


Turn on strict mode

I want use strict mode in my project, but in your repo is problem with it ;(

Driverdan/node-XMLHttpRequest has many changes. Please, sync with it or merge this fix.

Sorry for the delayed response


Chore(multiarch): Add multiarch build for argo and mongo images

Signed-off-by: uditgaurav<EMAIL_ADDRESS>
What this PR does / why we need it:
Images included for multiarch

litmus_argocli
litmus_argo_workflow_controller
litmus_argo_workflow_executor
litmus_mongo
litmus-qps-cmd
litmus-app-deployer
litmus-checker

Which issue this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close that issue when PR gets merged): fixes # https://github.com/litmuschaos/litmus/issues/1550
Special notes for your reviewer:

Tested Run

Push event: https://github.com/uditgaurav/test-test-tools/actions/runs/936538733

Build event: https://github.com/uditgaurav/test-test-tools/actions/runs/936636321


Revisit Extension Implementation

https://aeternity-blockchain.atlassian.net/browse/SW-1174
Description
extension: resolve/research "read and change all your data on all websites" (same in metamask)
extension: resolve/research "modify data you copy and paste" (same in metamask)
apply to everything that is needed to push our app on top when you search superhero (extension example Apply for Established Publisher and Featured badges in Chrome Web Store · Issue #1966 · superhero-com/superhero-wallet )

We excluded Manifest 3 from this scope

"read and change all your data on all websites”: We are injecting code into all pages the user visits to check for aepps. If we remove the permission the user will have to first open the extension on the page he wants to interact with some aepp and give it permission to inject code to the specific page.
"modify data you copy and paste”: Copy and paste seems to be working even without the clipboardWrite permission

Closing in favor of #1966 & #2938


Vagrant Setup

How can I configure this package with Xdebug running on Vagrant? I tried setting it up, but without any luck. Working fine with PhpStorm.

Also i have problem with Vagrant, also with other version unix too.
<- breakpoint_set -i 6 -t "line" -f "file:///vagrant/www/app/Lib/MessageQueue/Command/BroadcastCommand.php" -n 57
-> <response xmlns="urn:debugger_protocol_v1" xmlns:xdebug="http://xdebug.org/dbgp/xdebug" command="breakpoint_set" transaction_id="6" id="281660001"></response>

Server Ubuntu got message with prefix file://, it should be not on server unix.
my config's atom:
  "php-debug":
    PathMaps: [
      {
        local: "C:\\dev\\vargant\\www"
        remote: "/vagrant/www"
      }
    ]
    ServerPort: 9000

Ah, I missed one line: xdebug.remote_connect_back = 1 from php.ini and it worked.
https://github.com/gwomacks/php-debug/issues/125 - is helpfully for me.

Closing


Bug: the DeleteAll(T entities) method results in System.InvalidOperationException errors

Bug Description
While trying to pass in a set of models/entities to be deleted using the DeleteAll<T>(T entities, ...) method we recieve a System.InvalidOperationException exception is thrown.
This is reproducible in our integration test whereby we create two new records using mock data and use RepoDb to Insert hte data, and populate the Identity ID values immediately.  This works perfectly as expected.
Then after validating in the test, we attempt to clean up the data by deleting those very same models using the DeleteAll() method passing in the entities themselves.  The expected behavior is that RepoDb would recognize the models, correctly resolve the primary Key, and then use that to explicitly delete those records only from the database without any exceptions being thrown.
Our models have Identity ID values that are part of a base model class of which all Models inherit from.  When using a flat model it works, but when using a Model that inherits from our base class it fails:

Exception Message:
System.InvalidOperationException
  HResult=0x80131509
  Message=The declaring type of the property is not equal to the target entity type.
  Source=RepoDb
  StackTrace:
   at RepoDb.ClassExpression.GetPropertyValuesCache`2.Guard(ClassProperty property)
   at RepoDb.ClassExpression.GetPropertyValuesCache`2.<Do>d__3.MoveNext()
   at System.Collections.Generic.List`1..ctor(IEnumerable`1 collection)
   at System.Linq.Enumerable.ToList[TSource](IEnumerable`1 source)
   at RepoDb.Extensions.EnumerableExtension.AsList[T](IEnumerable`1 value)
   at RepoDb.DbConnectionExtension.DeleteAll[TEntity](IDbConnection connection, IEnumerable`1 entities, String hints, Nullable`1 commandTimeout, IDbTransaction transaction, ITrace trace, IStatementBuilder statementBuilder)
   at RepoDb.Fix.StarterKit.Repositories.TesterRepository.DeleteAll(IEnumerable`1 entities) in D:\Develop\GitHub.OpenSource\RepoDB\Issue991-RefactoredToFailAsExpected\RepoDb.Fix.StarterKit\Repositories\TesterRepository.cs:line 116
   at Program.<Main>$(String[] args) in D:\Develop\GitHub.OpenSource\RepoDB\Issue991-RefactoredToFailAsExpected\RepoDb.Fix.StarterKit\Program.cs:line 49

Schema and Model:
Please share to us the schema of the table (not actual) that could help us replicate the issue if necessary.
CREATE TABLE [dbo].[EnumTable](
	[Id] [int] IDENTITY(1,1) NOT NULL,
	[StringValue] [nvarchar](64) NULL,
	[IntValue] [int] NULL
) ON [PRIMARY]
GO

And also the model that corresponds the schema.
namespace RepoDb.Fix.StarterKit.Models
{
    public class EntityModel : BaseEntityIdModel
    {
        public string StringValue { get; set; }
        public LinkType IntValue { get; set; }
    }

    public class BaseEntityIdModel
    {
        //BBernard: In our Model design we have a base class with common shared properties (actually multiple levels deep) but otherwise these are merely DTO Entity Models!
        [Identity]
        public int Id { get; set; }
    }
}

Library Version:
Example: RepoDb v1.12.10-beta4 and RepoDb.SqlServer v1.1.5-beta4

Here is a functioning RepoDb Startup app that reproduces the issue 😊
DeleteAll Issue#1011- Startup Failing Repro.zip

Excellent news! 👏👏👏


why step_model and train_model instead of one?

Hi, I am reading the code of a2c.py, but confused about the design of step_model and train_model. As I read, both of them share parameters. So, why are two models used instead of one? Thanks.
            # step_model is used for sampling
            step_model = policy(nenvs, 1, sess)

            # train_model is used to train our network
            train_model = policy(nbatch, nsteps, sess)

For a recurrent policy, the training model should take in num_envs * nsteps observations in a batch for each forward pass.  When you're taking individual steps though, you only have nsteps=1  (1 per environment) observations, not a batch of them, so in that case, your model is slightly different.
Ideally you could use the same model object for both (like the model dynamically just uses whatever nsteps you happen to have for some data) so it's possible that tensorflow has some limitation making this hard to do.


Fix calc_heading

This is broken since on the main branch because I changed the sign of the y-axis of the kite reference frame.
To fix it:
[ ] add a unit test for orientation = 0,0,0
[ ] add unit tests for different roll angles
[ ] add unit tests for different pitch angles
[ ] add unit tests for different yaw angles
These tests need to be executed for different wind directions.

Development happens in the fix_yaw branch. One of the test scripts is examples_3d/parking_4p.
To run this script, execute:
julia --project
using Pkg
Pkg.update()
include("examples_3d/parking_4p.jl")

Shall be fixed in #78

Related: https://github.com/ufechner7/KiteModels.jl/pull/81

Not using the fix_yaw branch any longer.


Setting 'sizeInMeters:true' in BillboardGraphics ignores 'rotation' parameter.

I just added sizeInMeters:true and a width and height in meters to the Sandcastle example here:
http://cesiumjs.org/Cesium/Apps/Sandcastle/?src=Hello World.html&label=Showcases&gist=928681213846f9f82cd8db0528efa0db
(Be sure to choose "Set Billboard Properties At Creation", and zoom in to the North East USA.

Thanks @dwhipps!
Here's a trimmed down code example:
var viewer = new Cesium.Viewer('cesiumContainer');

var entity = viewer.entities.add({
    position : Cesium.Cartesian3.fromDegrees(-75.59777, 40.03883),
    billboard : {
        image : '../images/Cesium_Logo_overlay.png',
        rotation : Cesium.Math.PI_OVER_FOUR,

        scale: 30000,
        sizeInMeters : true
    }
});

Any chance this is going to get fixed for the next release?

Sorry @dwhipps, we haven't had a chance to look into this one yet

Looks like @dwhipps tracked down the source of this, it was introduced in 7ce1facd111.

Also this is a dup of #3979.

Thanks @dwhipps and @bagnell.  This fix will be in the October release, Cesium 1.26.


Hook Data Templates

Hi!
We've been thinking about our shiny new RestHooks and thought that it would be neat if there was another column in the Hooks table that let you define an option "template" for the hook data if the hook destination needed it in an alternative format.
Very Simple example. Say I have this normal hook response defined in serialize_hook, but the data destination is expecting "phone_number" instead of "from_number":
data  = {
   'from_number': self.from_number
}

With hook templates I could change that to:
{
    'phone_number': {from_number}
}

Thoughts?
-Lyle

@lylepratt I realize it has been almost two years since you have made this issue so I do apologize for getting to it so late.
Looking at the ticket feature request, it appears that you could currently do this with Django's Template engine. This in turn could create the ideal payload based on your own criteria.
Further following separation's of concerns, I feel more inclined to say that your feature really belongs outside the scope of django-rest-hooks.
I am of course open for more discourse on this to possibly drill down the root of the feature. (i.e. what is django-rest-hooks lacking).
-Andrew

Hi Andrew,
Yes its definitely possible to use Django template stored the data transformation. However, the template would need to be stored with the Hook data and users would need to be able to pass in their own "data templates" when they create hooks using the Rest API. Essentially, this would require an additional model field, and if the hook model contained a valid template in that field, it should run the transformation right before sending off the hook.

Seems like you can do this now with the serialize_hook method.


[bug]: Stacked_Themes ThemeManager

Describe the bug
When I try to use ThemeService I get this error:
"LateInitializationError: Field '_themeManager' has not been initialized."
What operating system do you use?
Linux
Information about the installed tooling
No response
Steps to reproduce the issue
Create a new project with stacked.
Then import the stacked_themes.
Added the line to app.dart:
LazySingleton(classType: ThemeService),
Tried a lot of different setups, including changing the "app.locator" to reflect what is in the readme.
Added the ThemeService to the viewModel, but it also happens as a service.
final _themeService = locator<ThemeService>();

I changed the default methods in home_viewModel to this:
void showDialog() {
    _themeService.toggleDarkLightTheme();
  }

  void showBottomSheet(BuildContext context) {
    getThemeManager(context).toggleDarkLightTheme();
    rebuildUi();
  }

The showDialog function gives me the error:
LateInitializationError: Field '_themeManager' has not been initialized.

The other works, but doesn't rebuild the UI to the new Theme. (Another topic)
Expected behavior
No response
Screenshots
No response
Additional Context
No response

Forgot to mention my main class:
Future<void> main() async {
  WidgetsFlutterBinding.ensureInitialized();
  setPathUrlStrategy();
  await setupLocator(stackedRouter: stackedRouter);
  await ThemeManager.initialise();
  setupDialogUi();
  setupBottomSheetUi();
  runApp(const MainApp());
}

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

  @override
  Widget build(BuildContext context) {
    return ThemeBuilder(
      defaultThemeMode: ThemeMode.light,
      darkTheme: ThemeData(
        brightness: Brightness.dark,
        scaffoldBackgroundColor: Colors.blue[900],
      ),
      lightTheme: ThemeData(
        brightness: Brightness.light,
        scaffoldBackgroundColor: Colors.blue[100],
      ),
      builder: (context, regularTheme, darkTheme, themeMode) => ResponsiveApp(
        builder: (_) => MaterialApp.router(
          theme: regularTheme,
          darkTheme: darkTheme,
          themeMode: ThemeMode.light,
          routerDelegate: stackedRouter.delegate(),
          routeInformationParser: stackedRouter.defaultRouteParser(),
        ),
      ).animate().fadeIn(
            delay: const Duration(milliseconds: 50),
            duration: const Duration(milliseconds: 400),
          ),
    );
  }
}

I got it to work.
For anyone who is having this issue, these are the steps:
Add the import in the pubspec.
In the app.dart add this line:
Singleton(classType: ThemeService, resolveUsing: ThemeService.getInstance),

Run "stacked generate"
Which adds this line in "app.locator":
locator.registerSingleton(ThemeService.getInstance());

I tried to add this manually without using Stacked generate and doesn't work.
In the main.dart add the Theme Builder really before the "MaterialApp.router":
Widget build(BuildContext context) {
    return ResponsiveApp(
      builder: (_) => ThemeBuilder(
        defaultThemeMode: ThemeMode.system,
        darkTheme: ThemeData(
          brightness: Brightness.dark,
          scaffoldBackgroundColor: Colors.blue[900],
        ),
        lightTheme: ThemeData(
          brightness: Brightness.light,
          scaffoldBackgroundColor: Colors.blue[100],
        ),
        builder: (context, regularTheme, darkTheme, themeMode) =>
            MaterialApp.router(
          theme: regularTheme,
          darkTheme: darkTheme,
          themeMode: themeMode,
          routerDelegate: stackedRouter.delegate(),
          routeInformationParser: stackedRouter.defaultRouteParser(),
        ),
      ).animate().fadeIn(
            delay: const Duration(milliseconds: 50),
            duration: const Duration(milliseconds: 400),
          ),
    );
  }

Got it working in this Test project. Probably will do a PR to create better documentation in the stacked_themes repo.


Changes for building local and ULINUX on Debian/Ubuntu:

Makefile - Add SHELL=/bin/bash.  Debian variants default to /bin/sh, which breaks brace expansion.
Makefile.kits - Do not set DPKG_LOCATION if /usr/bin/dpkg is present.
@jeffaco
@niroyb

Well, that looks familiar! :smile:
Signed off.


Support Selenium v3.14

Change list
Move to .Net Standard
Support Selenium 3.14
Types of changes
What types of changes are you proposing/introducing to .NET client?
Put an x in the boxes that apply

[ ] Bugfix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[X ] Breaking change (fix or feature that would cause existing functionality to not work as expected)

Details
Make project .Net Standard.

The will require those using the .Net Framework to update to at least 4.6.1
Add AppiumOptions
Mirror what Selenium's move away from desired capabilities
See https://github.com/SeleniumHQ/selenium/issues/6278#issuecomment-420728084 for more info
Add AppiumCapabilities
Create Appium specific capability class

I actually did think about that.
From a maintenance perspective I think we need an AppiumOptions class.
*I am confident that this code will need to be updated when DesiredCaps are no longer available.
That being said, if this change goes through I could see us extending it with iOS, Android, Windows and Tizen specific option classes. That way we get the best of both world.

Sure. This takes a task out of my weekend. Descendants can always be done later if necessary.,
Did you manage to run a test against this branch? Atm, the CI doesn't have verification tests yet but running the last written test should sufficient, I think that would be the IOSLockDeviceTest in order to merge.

@jlipps - This is defiantly a breaking change.

Changes AppiumDriver (and iOS, Android, etc.) constructor signatures
Removes support for .Net 4.6 and below, but adds support for .Net Core, Mono, Xamarin and UWP

Moving over to use the DriverOptions is based off an interaction  with Jim Evens.
*Seems like the best long term strategy for remaining compatible with future versions of Selenium.

@TroyWalshProf @akinsolb in that case, full steam ahead! Selenium and @jimevans compatibility are very important.

TL;DR I’m in favor of this PR, but some thought should be put into the design of this part of the Appium .NET API going forward.
I’m an outsider who is offering an  unsolicited opinion on a project whose community I’m not an active member of. Take the below with a grain of salt.
Dropping support for .NET Framework and moving to .NET Core only at this time would be a mistake, in my mostly irrelevant opinion. As mentioned by @akinsolb, the versions of PageFactory that are currently available (formerly in WebDriver.Support, now deprecated and migrated to the DotNetSeleniumExtras project) are not available in .NET Standard/Core projects.
Moreover, and this is simply a philosophical difference, this change appears to be simply substituting one “allow arbitrary names and values with no validity checking” property bag for another. The whole point of moving from DesiredCapabilities to a type-safe options class in the first place was to provide guidance and prevent user frustration because the capability being requested used the wrong (arbitrary string) name or the wrong (data type or invalid) value. Again, if the Appium project doesn’t care about that user experience, and it’s a perfectly legitimate choice not to, that’s a fine approach. However, I would put some thought into the API before making this the last word on the subject.
To forestall the inevitable question about why such a change is needed in the first place, correct use of DesiredCapabilities has been a source of confusion for users in all but the simplest of cases. It results in cargo cult, copy-paste code with no understanding of why the arbitrary capability is being set or if it’s even appropriate and supported by the target driver. Coupled with the W3C WebDriver Specification’s strictness around capabilities in the protocol, and the ability of implementations to reject connection requests with unknown or invalid capabilities, making the users’ and maintainers’ lives a little easier by preventing those types of errors in the first place seems like an obvious answer.
Finally, and this is simply by way of communication with a large downstream project about future possibilities, I’m thinking about ways to make the browser-specific DriverOptions API clearer, and AddAdditionalCapability is likely to be changing, which probably means deprecation, to be replaced by a new method or methods. I need a better way to solve the problem where a user wants to add a capability which belongs to a browser-specific driver dictionary (e.g., “goog:chromeOptions”, “moz:firefoxOptions”, etc.) and a global or top-level capability (like SauceLabs or BrowserStack user and API key capabilities). Right now, specific options classes have an overload for the method, but this is neither discoverable nor elegant, and the size of the user base using the imprecise code makes it impossible to just change the behavior to something that feels more correct.

@akinsolb - Added .Net 4.5 support back.

@akinsolb - This change also addresses
https://github.com/appium/appium-dotnet-driver/issues/227
https://github.com/appium/appium-dotnet-driver/issues/191
*.Net Core can use .Net Standard libraries

Hi,
Is there an ETA when this change will be available via NuGet?

@theAshenfire This is available in the beta version

What exactly is the resolution to the "Desired Capabilities" deprecated error?  because I'm still getting this error.

@jshinn6788 - Desired capabilities for Selenium has been deprecated.
What they did in the Selenium project was to  create driver specific options.
See: https://github.com/SeleniumHQ/selenium/blob/16559ddb469ec45bbdfef062a5638fb70cb18224/java/client/test/org/openqa/selenium/chrome/ChromeOptionsFunctionalTest.java#L89
This  pattern was followed in Appium.
Now instead of using Desired capabilities you use AppiumOptions.
See: https://github.com/appium/appium-dotnet-driver/blob/1dcb236576a92990c2580b0e600e6c9f3a841c23/test/integration/Windows/StickyNotesTest.cs#L53

Awesome!
This was very helpful
Thank you

On Jul 19, 2019, at 9:15 AM, Troy Walsh<EMAIL_ADDRESS>wrote:
@jshinn6788 - Desired capabilities for Selenium has been deprecated.
What they did in the Selenium project was to create driver specific options.
See: https://github.com/SeleniumHQ/selenium/blob/16559ddb469ec45bbdfef062a5638fb70cb18224/java/client/test/org/openqa/selenium/chrome/ChromeOptionsFunctionalTest.java#L89
This pattern was followed in Appium.
Now instead of using Desired capabilities you use AppiumOptions.
See: https://github.com/appium/appium-dotnet-driver/blob/1dcb236576a92990c2580b0e600e6c9f3a841c23/test/integration/Windows/StickyNotesTest.cs#L53
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.


[Autocomplete] When we enter valid/invalid string in Movie edit field, Screen readers are not narrating displayed search results information.

Duplicates

[X] I have searched the existing issues

Latest version

[X] I have tested the latest version

Steps to reproduce 🕹
Link to live example: gracious-jones-1gb0rh - CodeSandbox
Steps:

Open URL: gracious-jones-1gb0rh - CodeSandbox
Focus lands on the code editor as soon as page gets opened
Press tab to reach out to combo box control.
Now turn on Screen reader, invoke the combo box control, enter some valid / invalid data in edit field and capture the screen reader narration.
Verify whether screen reader is announcing displayed search results information or not.

Current behavior 😯
When we enter valid/invalid string in edit field, Screen readers should narrate displayed search results information.
Expected behavior 🤔
When we enter valid/invalid string in 'Change Type' edit field, Screen readers should narrate displayed search results information.
Ex: 1.When we enter Valid(Quota), JAWS/NVDA/Narrator should narrate as '2 results are returned' and if we press down arrow to navigate the search results, then NVDA/JAWS focus should move to first result, and it should narrate the name along with position of the list item.
2.When we enter invalid(@@), JAWS/NVDA/Narrator should narrate as 'No options are displayed'.
Context 🔦
When we enter valid/invalid string in edit field, Screen readers should narrate displayed search results information.
Users who rely on screen readers are confused to get the search results information, if screen reader is not announcing the results information when we enter valid/invalid string in edit fields.
Attachments:

https://github.com/mui/material-ui/assets/93735775/62493ec7-962d-42f1-a84f-532f4a21f4f7
Your environment 🌎

OS Version: 22H2 OS Build 22621.2134
Browser: Chrome Version 116.0.5845.179 (Official Build) (64-bit)
Screen Reader: JAWS, NVDA and Narrator
#URL: https://codesandbox.io/s/1gb0rh?file=/demo.js

Hi @ZeeshanTamboli , any update on this issue?

Hi @ZeeshanTamboli , any update on this issue?


[R] Implement feature info for DMatrix.

Close https://github.com/dmlc/xgboost/issues/8043 .
Also prepares for categorical data support.

@hetong007 Could you please review when you are available?

Sorry for my late response, it looks good to me, thank you!


Decide on the master system for the PLC (Difficult because technical limitations and restrictive licensing)

Summary from meeting (Davy's Message):
The biggest issue/headache at this moment is the master system for the PLC. A system like Simatic AX seems to have everything technically needed, but the licensing is very restrictive. Other options:

Codesys / Codesys Go (not released yet)
B&R
Beckhoff
Siemens TIA
Higher level language
Open source PLC IDE and runtime (for example https://github.com/beremiz/beremiz)
... ?

Adding a reference to the repo comparing the openness of the different systems:
https://github.com/SASE-Space/ot-openness-comparison
Some of the requirements that should be considered for the selection:

Plain text source files / ability to collaborate over git
Open download for IDE and simulator
Linux support (easier for building automated workflows with containers)
Testing framework
CLI tools
Permissive licensing for an open source project like this
Open documentation

The idea is that we select one 'Master' PLC system that will be used to develop and test the library. Then use this to ideally automate the generation of the other systems.
One possible technique to realize this could be the use of jinja templates: https://jinja.palletsprojects.com/en/3.1.x/templates/
For the 'Master' system we ideally can build a modern software development pipeline, which means automated builds, testing and maybe deployment to a demo environment. 'Classical' PLC development environments and tooling has originally not been built with this in mind. Although they are meant to automate, it is difficult to automate their own workflows.
The table in the post above shows that systems like Codesys, B&R, and Beckhoff are furthest in this. (Maybe also Bosch Rexroth?). A special note for Siemens TIA with Openness, although Openness is a very specialized and scripting-intensive solution.
Two interesting systems are around the corner that are building the IDE's and toolchain from the ground up.
First is Simatic AX, based on Visual Studio Code: https://www.siemens.com/global/en/products/automation/industry-software/automation-software/simatic-ax.html
Second is Codesys Go: https://www.codesys.com/fileadmin/data/Images/Messen_und_Veranstaltungen/CODESYS-Feature-Briefing-032024/15-CODESYS_Feature-Briefing-032024-CODESYS-go.pdf
Simatic AX is in limited release for around 1000 USD/month/user with a free trial period of 3 months (source: https://www.dex.siemens.com/industrialsoftware/automation-software/simatic-ax?cclcl=en_GB). There is no licensing option that would work for an open source project like this. Even if the core devs would have access to the licenses, it would be a major roadblock for people who would want to join.
Codesys Go is not released yet. There is no offical information about pricing, release date or wether there will be an open license available.

AX is 1k/y per dev instead of per month, but your point still stands.
Agree with everything above, although I think Siemens AX would be ok if we could get some floating licenses for this.
I think at the start it's mostly important to have a dev environment that would enable the core PLC team here to develop and test as efficiently as possible, with the least amount of friction, and I think AX lends itself to that the best.
Beckhoff and Codesys probably 2 and 3.
I think the master templates will/should soon move to a higher level language anyway, and then to get adoption we'd have to generate libraries for Siemens TIA (EU) and Rockwell (US) no matter what.

I think if we can use something like AX or Go then we shouldn't switch the language. Just stick with standard ST. I believe AX is also much closer to the standard than TIA is.
It's worth to reach out to Siemens and ask if it's a possibilty. Who knows they might already be thinking about how to make the licensing more premissive. Would be a pity if we missed it.

On Codesys: I don't know the system. I just spinned it up and I see the code is not stored in plain files. So the question would be how to sync it with git?
Two possible advantages of Codesys:

They are one of the two runtimes being used in the first OPA (Open Process Automation) projects (with the other one being Universal Automation)
Codesys Go is on the Horizon, it might be pathway to switch in the future

An updated on our latest discussions: we are thinking in the direction of:

VS Code as IDE (possibly with an extension like this: https://github.com/wadoon/vscode-iec61131)
Beremiz (https://github.com/beremiz/beremiz)
RoboPLC (https://www.bohemia-automation.com/software/roboplc/)
Rusty (https://github.com/PLC-lang/rusty)
...

Good news: Siemens has agreed to let us use AX, so we will go with this.
People who want to help with the PLC-part of the library can reach out the SASE slack :https://sase.space/
Channel: #open-process-library

'a bit fast' -> only 6 months since my last post here 😁
Jakob launched the request to open up de AX licenses a couple of weeks ago:
https://www.linkedin.com/posts/jakob-sagatowski_plcprogramming-openupindustrialautomation-activity-7261304202742677504-wqZE
Let's see what happens.
On the short term I think we should prove that the workflow that we envision works.
If we get into licensing issues then we adjust.


No work export

export not working

I found the same problem too

+1

+1

and also very very slow

@wangrenzhu in your example exporting working, but this is very slow, it would be great if you find time to deal with it, export very important part of processing, but of course you know about it. anyway i have to say, that you made good job! thanks!


disable travis CI

@osg-cat We need to disable Travis CI support in this repo, I have sufficient access to do it myself but wanted to check in with your first. If you'd prefer to do it the instructions are available at https://opensciencegrid.org/technology/documentation/publish-osg-pages/#disabling-travis-ci

If you have the time to make the changes, go right ahead! I can make a trivial site update after you’re done, to independently test that all is well.

If you have the time to make the changes, go right ahead! I can make a trivial site update after you’re done, to independently test that all is well.


Allow partially-manually defined Ops classes.

I thought this already existed, so apologies if it’s a duplicate.
It would be nice to be able to still use @typeclass even if the Ops trait (and some of the operations) are manually defined.

It's been on my mental list since TLS Philly last year but I haven't gotten to it. We should definitely support this. I'll try to carve out some time for Simulacrum work soon. Feel free to beat me to it though. :)


Documented Prometheus Metric keda_scaler_activity is actually keda_scaler_active

The documentation for prometheus metrics references a metric called keda_scaler_activity. The metric is actually keda_scaler_active.
The documentation should be updated to reflect this.
I think it was just a typo between the app repo and the docs repo from here https://github.com/kedacore/keda/issues/4114
Expected Behavior
Documentation should reference keda_scaler_active
Actual Behavior
Documentation references keda_scaler_activity which does not exist
Steps to Reproduce the Problem
Compare docs to code.
Specifications

KEDA Version: 2.10.0
Platform & Version: Ubuntu 20.04 LTS
Kubernetes Version: 1.25
Scaler(s): N/A

Good catch! It is indeed a typo. Fixed, thanks.


NetBSD package

Hi,
I've just packaged markdown-reader for NetBSD :)

I wonder, though if, I should wait with merging the package into the main branch until the TODO list has been fixed.
Currently, only simple .md files will render, e.g. your own README.md doesn't.
The application panicked (crashed).
Message:  not yet implemented
Location: tui-markdown/src/lib.rs:115

NOTE: Please check the TODO section in the README for a list of missing features.

Backtrace omitted. Run with RUST_BACKTRACE=1 environment variable to display it.
Run with RUST_BACKTRACE=full to include source snippets.
Exception: mdr exited with 101
  [tty 1]:1:1-14: mdr README.md

What do you think?

I appreciate your eagerness! This is way too new and experimental to be worth considering for packaging yet. I mostly published to crates.io in order to reserve the package names.

That's fine, that's why I asked.
Feel free to ping me when/if you think it's ready 😄
For now, I've added the package to our wip repo, https://mail-index.netbsd.org/pkgsrc-wip-changes/2024/02/27/msg029601.html
This is safer than keeping it locally.

Sounds good - closing this for now (I'll ping you on this issue when it's more ready for consumption)


Builds fail when using distcc

Is it possible to modify this utility to build properly with distcc enabled?  Currently, builds fail due to local ones using pie but remote ones not using it so they cannot be linked together.
...
/usr/bin/ld.gold: error: CMakeFiles/kodi-xrandr.dir/build/kodi/src/xbmc-17.3-Krypton/xbmc-xrandr.c.o: requires dynamic R_X86_64_
/usr/bin/ld.gold: error: CMakeFiles/kodi-xrandr.dir/build/kodi/src/xbmc-17.3-Krypton/xbmc-xrandr.c.o: requires dynamic R_X86_64_
collect2: error: ld returned 1 exit status
distcc[3304] ERROR: compile (null) on localhost failed
make[2]: *** [CMakeFiles/kodi-xrandr.dir/build.make:98: kodi-xrandr] Error 1
make[1]: *** [CMakeFiles/Makefile2:369: CMakeFiles/kodi-xrandr.dir/all] Error 2
/usr/bin/ld.gold: error: CMakeFiles/TexturePacker.dir/src/decoder/PNGDecoder.cpp.o: requires dynamic R_X86_64_PC32 reloc against
/usr/bin/ld.gold: error: CMakeFiles/TexturePacker.dir/src/decoder/PNGDecoder.cpp.o: requires dynamic R_X86_64_32 reloc which may
...
/usr/bin/ld.gold: error: CMakeFiles/TexturePacker.dir/build/kodi/src/xbmc-17.3-Krypton/xbmc/guilib/XBTF.cpp.o: requires dynamic
/usr/bin/ld.gold: error: CMakeFiles/TexturePacker.dir/build/kodi/src/xbmc-17.3-Krypton/xbmc/guilib/XBTF.cpp.o: requires dynamic
collect2: error: ld returned 1 exit status
distcc[3602] ERROR: compile (null) on localhost failed

hardening-wrapper is now deprecated as Arch will be adopting these features in the standard toolchain.


Add GraphQL (very basic)

Potential duplicate of https://github.com/highlightjs/highlight.js/pull/1543

So sorry for the long delay in getting back to you on this PR.
@morleym We're going to close this as a duplicate of  #1543.   We do truly appreciate your time and contribution though.  Hopefully your contribution would be welcome over at: https://github.com/dpeek/highlightjs-graphql
If you see anything major missing perhaps you could make a PR there?
You're always free to split this off into your own repository as well, but hopefully since this seems to be a very simple grammar we can keep all the contributions in a single 3rd party language module.


Chore: Remove non-neutral resource types

Currently neutral tests rely on vendor specific resource names. See: https://github.com/aws/karpenter-core/pull/9/files

Closed with #12


testing actions dont merge

Closes #
✅ Checklist

[ ] I have followed every step in the contributing guide (updated 2022-10-06).
[ ] The PR title follows the convention we established conventional-commit
[ ] I performed a functional test on my final commit

Changelog
[Short description of what has changed]

Screenshots
[Screenshots]
💯

its working :) sorry for the pings


Using SkiaSharp for imageing but not System.Drawing.Common for linux support

System.Drawing is not suit for linux, for libgdiplus has memory leak problem in linux, and have been removed since .net7.

Thank you very much for your report! You are absolutely right. I think that a little later I will transfer the stack to another core for working with images

sys.drawing namespace also can't be used in UWP (not supported), use bytes or stream is better, I think.

@bnuzhouwei , @songshizhao you can use float[][,] methods
https://github.com/FaceONNX/FaceONNX/blob/9f1fd201d9801dcad20a68f4c67f3545b1faaccc/netstandard/FaceONNX/face/classes/FaceDetector.cs#L91

Any chance you could update the examples to use this new method? Then it would be properly cross-platform. I'm going to use ImageSharp to load the bitmap, but it would be great if there was an example showing either ImageSharp or SkiaSharp bitmaps.

Any chance you could update the examples to use this new method? Then it would be properly cross-platform. I'm going to use ImageSharp to load the bitmap, but it would be great if there was an example showing either ImageSharp or SkiaSharp bitmaps.

I asked ChatGPT and it gave me this example:
using SkiaSharp;
using System.Numerics;

public float[][,] ConvertSKBitmapToFloatArray(SKBitmap bitmap)
{
    // Ensure the bitmap is in BGR8888 format
    if (bitmap.ColorType != SKColorType.Rgb && bitmap.AlphaType != SKAlphaType.Opaque)
    {
        bitmap = bitmap.Copy(SKColorType.Rgb, SKAlphaType.Opaque);
    }

    // Lock pixels to get access to pixel data
    using (var info = bitmap.PeekPixels())
    {
        if (info == null)
        {
            throw new InvalidOperationException("Failed to peek pixels from bitmap.");
        }

        var bytes = info.GetPixels();
        var width = bitmap.Width;
        var height = bitmap.Height;

        // Convert byte array to float array
        var floatArray = new float[3, height, width];

        for (int y = 0; y < height; y++)
        {
            for (int x = 0; x < width; x++)
            {
                var index = (y * width + x) * 4; // Each pixel is 4 bytes
                var b = bytes[index];
                var g = bytes[index + 1];
                var r = bytes[index + 2];

                // Normalize values
                floatArray[0, y, x] = b / 255.0f; // Blue
                floatArray[1, y, x] = g / 255.0f; // Green
                floatArray[2, y, x] = r / 255.0f; // Red
            }
        }

        return floatArray;
    }
}

Maybe helpful?

Any chance you could update the examples to use this new method? Then it would be properly cross-platform. I'm going to use ImageSharp to load the bitmap, but it would be great if there was an example showing either ImageSharp or SkiaSharp bitmaps.

@Webreaper , @Misaka12456 or you can try something like this https://github.com/FaceONNX/FaceONNX/blob/main/netstandard/Examples/FaceEmbeddingsClassification/Program.cs

I already submitted a PR (https://github.com/FaceONNX/FaceONNX/pull/22) which fixes this and makes it cross-platform.

I already submitted a PR (#22) which fixes this and makes it cross-platform.

I actually use it in Emgu.CV Mat, but I didn't find a best solution...
@asiryan Do you have any other example?

I already submitted a PR (#22) which fixes this and makes it cross-platform.

I actually use it in Emgu.CV Mat, but I didn't find a best solution... @asiryan Do you have any other example?

Do you need an example with Emgu.CV?

yeah.... planned to use faceonnx for a real-time face detection
------------------ 原始邮件 ------------------
发件人: "Valery @.>;
发送时间: 2024年7月25日(星期四) 晚上10:54
收件人: @.>;
抄送: @.>; @.>;
主题: Re: [FaceONNX/FaceONNX] Using SkiaSharp for imageing but not System.Drawing.Common for linux support (Issue #16)
I already submitted a PR (#22) which fixes this and makes it cross-platform.
I actually use it in Emgu.CV Mat, but I didn't find a best solution... @asiryan Do you have any other example?
Do you need an example with Emgu.CV?
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you were mentioned.Message ID: @.***>

@Misaka12456 I think you can find information about images and their transformations here https://www.emgu.com/wiki/index.php/Working_with_Images

thx


Install Promtool

We should install the promtool so that configuration can be checked before reloading the service.
./promtool -help
usage: promtool <command> [<args>]

Available commands:
  check-config    validate configuration files for correctness
  check-rules     validate rule files for correctness
  help            prints this help text
  version         print the version of this binary

Only a link should be sufficient since the promtool is already in the prometheus binary distribution. On my systems I found it on /opt/staging/prometheus-${version}.linux-amd64/

that is correct. I was thinking more if we should run it before we restart the service?

I believe we can run it before reloading or restarting the service. It may be a good idea. However, it will complicate a bit the startup scripts.


Refer to emotion-theming instead of @emotion/theming

The @emotion/theming package 404s on npm. emotion-theming is the correct package name.
Excited to check out rebass v4! 🙌

Codecov Report

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

@@          Coverage Diff          @@
##           master   #630   +/-   ##
=====================================
  Coverage     100%   100%
=====================================
  Files           4      4
  Lines          21     21
=====================================
  Hits           21     21

Continue to review full report at Codecov.

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

Oh man, thanks! 😬


Marking dictionary objects as required is ignored in generated code

Problem statement
If a property is defined as a pure map (without any other fixed fields), making it "required" does not work in generated code.
Swagger specification
Consider this object:
"properties": {
    "currencies": {
      "type": "object",
      "additionalProperties": {
        "$ref": "#/definitions/amount"
      }
    },
},
"required": ["currency"]

This produces (correctly) a go representation of map[string]Currency. It seems however that the required directive is ignored, requests that do not include currencies are accepted and defaulted to an empty map.
I want them to be rejected as per the json-spec.

You have set the property minProperties to 1 or more. The required just checks if something is null or not

I'd say this is a duplicate of #2300


dxcam not working in an Azure VM

I have an Azure VM running T4 GPU with DirectX extensions, the XBox app runs and I can play games, but dxcam only returns black and sometimes pauses for a very long time until I move the mouse?

Turns out this might be a windows bug and apparently is fixed in Windows 11 24H2.

Fixed in https://github.com/lovettchris/wincam


Apply to be a maintainer

If you still look for a maintainer, let me know.
I would like to become a contributor.

@binhle410 that would be awesome and ok for me if other @knplabs are ok too (@PedroTroller @NiR- , ...)

Hi, I'm current new maintainer (= doing all the merges and issues).
Help woudl be great. Can I ask you to go through the issues and identify those most likely to close and ping me there? Old, unclear, probably solved etc.
Thank you
Next step would be finishing this: https://github.com/KnpLabs/DoctrineBehaviors/issues/241


Husky setup for eslint and commitlint

Closes #33

@avinashupadhya99
I just have a small doubt that while setting up the project I wouldn't need to run any command to setup husky right? It will be activated on it's own when I will be making a commit right?

@avinashupadhya99
I just have a small doubt that while setting up the project I wouldn't need to run any command to setup husky right? It will be activated on it's own when I will be making a commit right?

No additional setup required. Just the dependencies have to be installed.

Good to hear that, so should I merge it or will you be doing the devDependencies thing I mentioned?

I thin you can go ahead and merge this. That can be handled as a separate issue/PR.

@avinashupadhya99
Congrats and looking forward to some more wonderful contributions ahead from you 🎉


adds tests reproducing the error described in issue665

Adds three tests:

creates the error described in #665 using OSM data and the GraphHopper API

shows that the Graph resulting from the OSM data is invalid in this scenario

reproduces the invalid Graph and shows how the error is produced using the bidirectional Dijkstra algorithm between the two nodes of the Graph directly

I simply put all three tests inside OSMReaderTest for now.

Thanks again!
The bug is that we do not set the reverse speed in the motorbike encoder and so the speed==0 leads to access==false due to the fix for #367 when calling reverseFlags. This method is called for 'opposite' base nodes ie. when the flags are stored relatively to the adjacent node and needs swap all speeds and access properties. I'm not yet sure why this happens only that infrequent.
In the Bike2WeightFlagEncoder we handle reverse speed via calling a special method handleSpeed in handleWayTags which makes sure both speeds are always properly filled.
So the real reason is that adding 'reverse speed' is overly complicated and too many things can go wrong when adding this to an encoder.
But probably also the magic happening in the fix for #367 is not good.

Thanks a bunch again to @ammagamma for making it very easy for me to reproducing this. Closing due to #711


[Feature Request]: Implement Auto Labeling for Issues

Is there an existing issue for this?

[X] I have searched the existing issues

Feature Description
Title:
Implement Auto Labeling for Issues
Description:
This feature aims to automate the process of assigning labels to issues when they are created or edited. The following labels will be automatically added:

gssoc-extd: Marks the issue as part of GSSOC-extended contributions.
hacktoberfest: Marks it as eligible for Hacktoberfest.
level?: A placeholder label indicating the level, which will remain until the project admin assigns the appropriate level during the Pull Request (PR) review.

Tasks:

Create a GitHub Action that listens to issue events (opened, edited).

Automatically assign the following labels:

gssoc-extd
hacktoberfest
level?

Ensure the placeholder label level? remains on the issue until the project admin assigns the final level during the PR stage.

Test the automation by creating and editing issues.

@prajapatihet
Pls assign me this issue and give me level3,hacktoberfest-accepted and gssoc-extd
Use Case
Acceptance Criteria:

[ ] Labels are added to issues automatically upon creation or editing.
[ ] Placeholder label level? is correctly assigned to issues.
[ ] Final level is assigned by the project admin during PR review

Benefits
No response
Add ScreenShots
No response
Priority
High
Record

[X] I have read the Contributing Guidelines
[X] I'm a GSSOC Ext'24 contributor
[X] I'm a Hacktoberfest 2024 contributor
[X] I have starred the repository

Not planned for any github action


[Refactor] Clean up after the namespace vectorized removal

What type of PR is this：

[ ] BugFix
[ ] Feature
[ ] Enhancement
[X] Refactor
[ ] UT
[x] Doc
[ ] Tool

Which issues of this PR fixes ：

Fixes #13805
Problem Summary(Required) ：

Follow up fixes after #15429
Checklist:

[ ] I have added test cases for my bug fix or my new feature
[ ] This pr will affect users' behaviors
[ ] This pr needs user documentation (for new or modified features or behaviors)

[ ] I have added documentation for my new feature or new function

Bugfix cherry-pick branch check:

[X] I have checked the version labels which the pr will be auto backported to target branch

[X] 3.0
[ ] 2.5
[ ] 2.4
[ ] 2.3

run starrocks_admit_test

run starrocks_be_unittest

@Mergifyio backport branch-3.0


x is not defined

I came across an issue using a JSON array as a top level element with an IOStream.
https://github.com/samoconnor/LazyJSON.jl/blob/2a66639fca3ea8f3428c93e0f59398cb4f4aa6f6/src/AbstractArray.jl#L37
io = IOBuffer("""[
              {"type":"Bar","event":{"a":"Hello","foo":{"a":1,"b":2}}},
              {"type":"Bar","event":{"a":"Hello","foo":{"a":1,"b":2}}}
              ]""")
o = LazyJSON.value(io) # This breaks on doing the repl print
o[1] # This just breaks

Thanks for reporting that!

https://github.com/samoconnor/LazyJSON.jl/releases/tag/v0.0.6
https://github.com/JuliaLang/METADATA.jl/pull/18459


Missing github links in package.json

so npmjs package doesn't have the link to here :(

Let's fix it! PR, anyone?


Refactor Handlers for None-correctness

Follow up of the discussion with @lbarcziova in https://github.com/packit/packit-service/pull/1579#discussion_r921034064
Currently, handlers are instantiated in 2 times, once in Steve and then in the celery task corresponding to the handler. However, the use-cases for these two initializations are different:

Steve just uses pre_check() and perhaps some other stuff but doesn't run it
celery task runs the handler

This causes problems with regards to None correctness as explained in the comment linked above. Ideally, a task that can be retried should not have its CeleryTask argument set to None (without the task attached, retrying is not possible and does not make sense). However, the initialization inside Steve does not have a celery task yet to attach, hence it must be set to None (a bit of a chicken-egg problem).
A solution that I assume could work (however, feel free to come up with anything else):

split each handler (including the abstract classes) into 2 parts (naming is just a suggestion):

BaseHandler - implements just the pre_check and similar stuff
RunnableHandler - also implements run and inherits from BaseHandler

inside Steve initialize just the BaseHandler which does not need a CeleryTask even for a retriable job handler.
inside tasks, initialize RunnableHandler which in the case of retriable handlers will have non-Optional CeleryTask argument

This is a blocker for #1433 (the implementation from #1579 would not pass mypy's strict checks -- however it does not make sense to block it on this rather large refactoring)

Follow-up from arch:
Another option to handle this situation is to not instantiate a handler in Steve. This would mean that pre_check and check_if_actor_can_run_job_and_report would have to be made static/class methods and everything required would have to be passed to them. From briefly looking at this, the methods access all kinds of stuff from the instance attributes (job/package config, helper instance -- this would have to be instantiated, which requires ogr Project, trigger and a lot of other stuff) so this approach may not really be feasible, the code of these methods may get quite messy, if we wanted to pass everything there.


Electron should be an optional dependency

What problem does this feature solve?
Vue devtools builds electron and includes it in the final bundle even if its not necessary, leading to an increased build time and log messages like 'Hoisted from "@vue#devtools#electron"'.
What does the proposed API look like?
Perhaps it can be included in optionalDependencies or peerDependencies in the package.json

Maybe the connecting part can reside in a separate package?


MM Ported Over Derived Table Instance series

New derived table ported from LDP to MetaDB for series information in instance records

[v] Pull request has a complete description
[v] Query runs without errors
[v] Query output is correct
[v] Query logic is clear and well documented
[v] Query is readable and properly indented
[v] Table and column names are in all-lowercase
[v] Query begins with user documentation in comment lines
[v] File name is listed in runlist.txt after dependencies
[v] All columns have indexes
[v] Table is vacuumed and analyzed


IDT 92hd89e2 not supported

I have that codec and there doesn't seem to exist any workaround to get it to work, any reason why that is.
The device hardware id is 111d 76c7

Sorry. I don't know.

Please create a new issue once you have working resources, and we will add them.


Support Binary Builds

I believe this is a feature request. If not, please do point me to the right documentation. I'd like to be able to perform the equivalent of oc start-build <build-name> --from-dir=<local-directory> using the openshiftBuild or a similar step.

thanks, we're already carrying this feature request on our trello board: https://trello.com/c/kJrtI2kF/1034-jenkins-plugin-binary-build-support


[Feature Fix] Convert string into date with moment()

Purpose
In moment.js, utc() can't process the date format created by Dropbox API in Safari and IE. It is something like -- Sat, 21 Aug 2010 22:31:20 +0000. Therefore, it is shown as invalidate and not displayed.
Resolve [OSF-4294]
Changes
After testing, I find  moment() is more stable to accept all kinds of string date, as it mentioned in docs,

When creating a moment from a string, we first check if the string matches known ISO 8601 formats, then fall back to new Date(string) if a known format is not found.

So I choose to use moment() to create date object firstly and then apply utc() to get UTC time object.
Related Stackoverflow question (Asked by me) -- http://stackoverflow.com/questions/33508049/moment-utc-doesnt-work-in-ie-and-safari
Side Effect
Since it is helper function used in many other places, it is better to check most of it. In theory, it should be find, because moment() can process the date string that utc() can do in the past.

Looks good to me @haoyuchen1992! Done with code review 1 :octocat:


CCM-SMP support - updated the what's supported section

Preview link: https://64f5ef6adbe81d5a10f3c48f--harness-developer.netlify.app/docs/cloud-cost-management/whats-supported

Please check the Execution Link of the Pipeline for the Website Draft URL. This is located in the Preview Step and also is available in #hdh_alerts. E.g Website Draft URL: https://unique-id--harness-developer.netlify.app

@Rohit-Kaliki I made your suggested changes. Please review and approve if it looks good now.

Please check the Execution Link of the Pipeline for the Website Draft URL. This is located in the Preview Step and also is available in #hdh_alerts. E.g Website Draft URL: https://unique-id--harness-developer.netlify.app

@KimberlyFields made one change, supported infra was AWS not Azure

Please check the Execution Link of the Pipeline for the Website Draft URL. This is located in the Preview Step and also is available in #hdh_alerts. E.g Website Draft URL: https://unique-id--harness-developer.netlify.app


Error debuging an ES6 Proxy

VSCode Version: 1.7.1 (1.7.1)
OS Version: OSX El capitan 10.11.2
Node.js version: v7.1.0

Steps to Reproduce:

Defining a object as ES6 Proxy
Running script with the Debug tool (Shift+CMD+D)
Typing the object in the console

Throw this error
#
# Fatal error in ../deps/v8/src/runtime/runtime-debug.cc, line 1423
# Check failed: args[0]->IsJSObject().
#
==== C stack trace ===============================
    0   node                                0x0000000100bca643 v8::base::debug::StackTrace::StackTrace() + 19
    1   node                                0x0000000100bc77c9 V8_Fatal + 233
    2   node                                0x00000001007bd586 v8::internal::Runtime_DebugGetPrototype(int, v8::internal::Object**, v8::internal::Isolate*) + 438
    3   ???                                 0x00001bed7b0063a7 0x0 +<PHONE_NUMBER>2183

Can you share sample code?

@Josema please try to use our new experimental node debugger by setting the type attribute in your launch config to node2.

@ramya-rao-a
let target = {a:123};
let p = new Proxy(target, {
  set: (target, property, value, receiver) => {}
});
console.log( p );

Set a breakpoint on the console.log line
@weinand Working with node2 thanks!!

dup of #12749


Error Code=-1007 "too many HTTP redirects" UserInfo={NSLocalizedDescription=too many HTTP redirects

Got below error after changes server configuration,
Error Domain=NSURLErrorDomain Code=-1007 "too many HTTP redirects" UserInfo={NSLocalizedDescription=too many HTTP redirects
Alamofire version : 4.7.3
Please help me out in this issue.

This is usually a server issue, so there's no much we can help you with here.
Sorry, we use our GitHub project for bug reports and feature requests. In the future, you should open questions like this on Stack Overflow..
Cheers. 🍻

From our Contribution Guidelines
Asking Questions
We don't use GitHub as a support forum. For any usage questions that are not specific to the project itself, please ask on Stack Overflow instead. By doing so, you'll be more likely to quickly solve your problem, and you'll allow anyone else with the same question to find the answer. This also allows maintainers to focus on improving the project for others.


Lxc copy for clone lxc non exsiste in pylxd

there is no function to clone the container with the native function of lxc copy?

You can use the create function to clone containers by setting the source to container.
something like:
config  = {
    'name': ' name ',
    'source': {
        'type':'copy',
        'source': 'original-container'
    }

lxd.create(config) # will clone original-container

see: https://pylxd.readthedocs.io/en/latest/instances.html#examples


Issue 58

Codecov Report

Merging #59 (b6f15fc) into php (46c20e7) will decrease coverage by 0.12%.
The diff coverage is 45.00%.

@@             Coverage Diff              @@
##                php      #59      +/-   ##
============================================
- Coverage     85.10%   84.97%   -0.13%
- Complexity       94       99       +5
============================================
  Files             9        9
  Lines           423      446      +23
============================================
+ Hits            360      379      +19
- Misses           63       67       +4

Impacted Files
Coverage Δ

app/Coding/Disk.php
48.83% <0.00%> (-3.67%)
:arrow_down:

app/Commands/WikiImportCommand.php
86.53% <56.52%> (-4.05%)
:arrow_down:

app/Coding/Wiki.php
88.28% <100.00%> (+6.63%)
: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 46c20e7...b6f15fc. Read the comment docs.


test(a11y): Replace IBM stuff with gulp-axe-webdriver

Overview
Decided to adopt a new automated a11y testing tool (gulp-axe-webdriver, which is based on axe-core library.)
How to use
Start the server and run the tests. Axe needs to test against real local or external URLs.
npm run dev
gulp test:a11y

Test a11y for single components:
# remember: npm run dev
gulp test:a11y --name=accordion

About Axe configuration

Uses chrome browser by default
Violations only
Tests against rules that comply with wcag2aa and wcag2a standards, see rules here.
Exclude nodes that are part of demo environment ('.offleft, #flex-col, #flex-row')
All results are saved as JSON files in tests/axe

Drawbacks
Must test against real URLs. Once sandbox is up, we can test against that external URL so that Travis CI can run a11y test

Did a demo of this today for the FEDs and it sounds like @chrisdhanaraj wants to merge this in after getting a Selenium environment setup first before merging this in?


Release 2.0.0 is not compatible with IE11

Release 2.0.0 is not compatible with IE11 due to the use of '=>' function declaration. It would be nice you either fixed or documented this incompatibility.

This module mainly targets Node.js, not the browser. It's up to you to transpile it with Babel if you want to use it in the browser. You can find a more detailed explanation here: https://github.com/sindresorhus/ama/issues/446
If you use Webpack, check out babel-engine-plugin, which transpiles only the dependencies that needs to be transpiled.
If you use Create React App, upgrade to Create React App v2. It supports automatic transpilation of dependencies, which will make this package just work.

Hello, same issue with IE


Unify model-parallel vs non model-parallel codepaths

Patch Description
Unified model-parallel vs non model-parallel codepaths: merged files from metaseq.model_parallel to metaseq.models. Solving issue #389
Testing steps
MP2
python -m PROJECT.sweep_baseline -g 4 -n 1 --model-size 8m --prefix test_01  --local --data /checkpoint/TEAM_NAME/datasets/consolidated/v4.0
MP1
python -m PROJECT.sweep_baseline -g 4 -n 1 --model-size 8m_mp1 --prefix test_01  --local --data /checkpoint/TEAM_NAME/datasets/consolidated/v4.0
Results
MP2

Metric
Before
After
Comment

num_updates
100
100
---

loss
14.406
14.406
---

wps
911142
909002
<0.25% difference, noise

MP=1

Metric
Before
After
Comment

num_updates
100
100
---

loss
15.857
15.857
---

After merging
Need to re-install metaseq:
rm -rf metaseq
copy metaseq from main branch
cd ~/rsc/metaseq
pip install --no-build-isolation -e .

One more TODO (perhaps for me):

resume training from an existing checkpoint to see if that still works as expected
test if inference on existing checkpoint still works as expected

(don't think either will be affected here, but will need this sanity check to be safe)

Good point, I'll test this as well.

Ok tested resuming from a previous run and things look good. From the diff, it doesn't seem like any configurations is actually impacted, so approving to move onto the next step!

Ok tested resuming from a previous run and things look good. From the diff, it doesn't seem like any configurations is actually impacted, so approving to move onto the next step!

yes, my next diff would touch on some of the configs, when I combine the model arch: transformer_lm_megatron and transformer_lm arch. #583


Not all schemes are building with Carthage

@jfoley and I are trying to use PCK (on master) with Carthage. We've got to the point where PCK appears to build without an error (by using the fix in PR #175), however we only see the following schemes being built:
± ms+jf |master ✗| → carthage update --no-use-binaries --platform ios
...
*** Building scheme "CoreLocation+PivotalSpecHelper-iOS" in PivotalCoreKit.xcworkspace
*** Building scheme "UIKit+PivotalCore-iOS" in PivotalCoreKit.xcworkspace
*** Building scheme "WatchKit" in PivotalCoreKit.xcworkspace
*** Building scheme "Cedar-iOS" in PivotalCoreKit.xcworkspace
...

There's a number of schemes missing (notable for us are the spec helpers and stubs). Having spoken with @younata, it appears that this could be a bug in Carthage, but we're filing an issue here to track this on the PCK side.

I wouldn't expect Carthage to build targets that aren't dynamic frameworks. can you confirm you are building a version of PCK where these targets are in fact dynamic frameworks?  Previously these targets were static frameworks built by a series of scripts inside an Xcode scheme, which Carthage doesn't support.

Yes, we were building against master (which I understand has the dynamic frameworks as canonical targets).

Current carthage release (0.11) doesn't work, because the pull request that enables support for projects structured like PCK was merged after that release was cut. As of this comment, they haven't made another release. However, Mike is definitely aware of that.

Had some time to follow up on this today. I have also just verified with carthage 0.15.2 that master appears to build all 19 frameworks:
± |master ✓| → carthage build --no-skip-current
*** xcodebuild output can be found in /var/folders/j_/wwl3kb7d6mld45lx8cmpdzqh0000gn/T/carthage-xcodebuild.uCk5hz.log
*** Building scheme "CoreLocation+PivotalSpecHelper-watchOS" in PivotalCoreKit.xcworkspace
*** Building scheme "CoreLocation+PivotalSpecHelper" in PivotalCoreKit.xcworkspace
*** Building scheme "CoreLocation+PivotalSpecHelper-tvOS" in PivotalCoreKit.xcworkspace
*** Building scheme "CoreLocation+PivotalSpecHelper-iOS" in PivotalCoreKit.xcworkspace
*** Building scheme "Foundation+PivotalSpecHelper-tvOS" in PivotalCoreKit.xcworkspace
*** Building scheme "Foundation+PivotalCore" in PivotalCoreKit.xcworkspace
*** Building scheme "Foundation+PivotalCore-tvOS" in PivotalCoreKit.xcworkspace
*** Building scheme "Foundation+PivotalCore-watchOS" in PivotalCoreKit.xcworkspace
*** Building scheme "Foundation+PivotalSpecHelper" in PivotalCoreKit.xcworkspace
*** Building scheme "Foundation+PivotalSpecHelper-iOS" in PivotalCoreKit.xcworkspace
*** Building scheme "Foundation+PivotalSpecHelper-watchOS" in PivotalCoreKit.xcworkspace
*** Building scheme "Foundation+PivotalCore-iOS" in PivotalCoreKit.xcworkspace
*** Building scheme "UIKit+PivotalCore-tvOS" in PivotalCoreKit.xcworkspace
*** Building scheme "UIKit+PivotalSpecHelperStubs-tvOS" in PivotalCoreKit.xcworkspace
*** Building scheme "UIKit+PivotalSpecHelper-tvOS" in PivotalCoreKit.xcworkspace
*** Building scheme "UIKit+PivotalCore-iOS" in PivotalCoreKit.xcworkspace
*** Building scheme "UIKit+PivotalSpecHelper-iOS" in PivotalCoreKit.xcworkspace
*** Building scheme "UIKit+PivotalSpecHelperStubs-iOS" in PivotalCoreKit.xcworkspace
*** Building scheme "WatchKit" in PivotalCoreKit.xcworkspace

Unless I missed something, this doesn't appear to be an issue anymore, so I'm going to close it.


split GenerateUpdateClusterConfigInput in 2 methods: Logging and VPC

Signed-off-by: Adrien Zieba<EMAIL_ADDRESS>
Description of your changes
AWS CLI prevents us from updating the VPC config and the Logging in the same call. This PR is splitting the method GenerateUpdateClusterConfigInput in 2 distinct ones: GenerateUpdateClusterConfigInputForLogging, GenerateUpdateClusterConfigInputForVPC

Fixes #1014
I have:

[x] Read and followed Crossplane's [contribution process].
[x] Run make reviewable test to ensure this PR is ready for review. The test steps is failing on my laptop with the following error package github.com/crossplane-contrib/provider-aws/pkg/clients/mock: build constraints exclude all Go

How has this code been tested

Create a cluster with Logging enabled
...
        forProvider:
          logging:
            clusterLogging:
              - enabled: true
                types:
                  - api
                  - audit
                  - authenticator
                  - controllerManager
                  - scheduler
...

Then change the configuration to disable one of the type
...
        forProvider:
          logging:
            clusterLogging:
              - enabled: true
                types:
                  - api
                  - audit
                  - authenticator
                  - controllerManager
                  - scheduler
              - enabled: false
                types:
                  - controllerManager
...

NOTE: I noticed that the types needs to be in alphabetical order or the provider detects a constant drift with the aws resource.
[contribution process]: https://git.io/fj2m9

The ordering would be a nice to have. When you say at the provider level, do you mean adding some logic in CreatePatch to sort types before comparison?
Maybe we would have to sort also 1 level above (ClusterLogging items) -> To be tested

would be cool to see a PR for the sorting stuff ;)


fix: make the color of the logger more highlighted

修复 issues https://github.com/alibaba/lowcode-engine/issues/2498

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.解成博 seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.

分支指向 develop。

分支指向 develop。

已修正


Generated models have optional properties when defined as required in schema

In my schema.prisma I have a model defined with what I understand to be a required field, since it's not using the optional modifier:
model Person {
  id   Int    @id @default(autoincrement())
  name String @db.VarChar(60)
}

I've run the generator as per the docs, with:
npx prisma generate

But the generated Person class has both the id and name properties as being optional:
class Person {
  const Person({
    this.id,
    this.name,
  });

  factory Person.fromJson(Map json) => Game(
        id: json['id'],
        name: json['name'],
      );

  final int? id;

  final String? name;
}

Is this the intended behaviour or have I missed something?
Using version 4.0.0-beta.3

This is the expected behavior. All fields of model are optional fields. The main purpose is for me to adapt the select and include functions (for selecting only some fields or additional fields containing relationships)

@medz Thank you for clarifying.
Since this appears to be different behaviour to Prisma JS, it might be worth adding something in the model docs.
I'll close this issue as you've confirmed it's expected behaviour

@devklick

Since this appears to be different behaviour to Prisma JS

Yes, since Prisma JS/TS relies on TS's type calculation, types can be constructed dynamically. Other languages do not have support for this feature. If you want to implement select/include, you can only use nullable types.

@medz are you planning to do a Dart specific fix for this case?
Having ALL nullable types means no null safety and no null safety means ultimately no type safety, that is the major reason to use an ORM tbh.

@AndryHTC I don't know if I should do this. In TS, using select can use type calculation to get safe types.
But Dart is the only one that is not possible, and most other compiled languages are not. In order to support the select function, I chose to generate all models as nullable.
In actual use, it does require developers to fully know what columns they select and use the ! symbol to make it correct.
But the reality is that most people will not use the select function, and the type of the model should be consistent with the schema.
So I have a plan to optionally remove the select function in future versions, and then extract the include function. To ensure that the model class and schema def are consistent.

Actually, I have another crazy idea, but it will not appear in prisma-dart, which is to design a set of query methods that conform to Dart. I have tried many things in the oh package before, but I didn't find a better way than prisma CRUD APIs.
I will continue to try when I am not busy in the future. The oh package will be built on top of prisma-dart and provide APIs different from Prisma Client.

All this is still in the experimental stage

What about something like this?
class Attribute<T> {
  final T _value;

  Attribute._(this._value);

  factory Attribute.fromJson(dynamic json) {
    if (json == null) {
      throw ArgumentError('Cannot instantiate Attribute with null value');
    }
    return Attribute._(json as T);
  }

  T get value => _value;
}

So in the Dart model you can still make all the attributes nullable, but if they are included in the select, we can use their original type and nullability
Example:
// jsonData has all the attributes
final model = Model.fromJson(jsonData);

// Safe access to the uuid value
print(model.uuid!.value);  // This is guaranteed to be a non-null String
print(model.nullableField!.value);  // This is nullable String

@medz


Cash

Add dependencies payment in python

Hey there! Thank you for suggesting that, if you have a moment please do it Here to be considered in future updates!

Hey there! Thanks for the request! If you have a moment, please suggest it here so it can be considered for future implementations. We appreciate your input!