Printing Windows Form in C#

I have followed the instructions but when the app enters the:
private void PrintDocument1_PrintPage(System.Object sender,
System.Drawing.Printing.PrintPageEventArgs e) =>
e.Graphics.DrawImage(memoryImage, 0, 0);
section, the memoryImage is always null, even after it is not null during the CaptureScreen() function.
NOTE: I found my error. Sorry to have bothered you. Thanks.
Glen

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

ID: 897fff16-d70f-5e07-0d49-a718e35df446
Version Independent ID: 08014790-627a-e56f-9eea-309f4d70c016
Content: How to: Print a Windows Form - Windows Forms .NET Framework
Content Source: dotnet-desktop-guide/framework/winforms/advanced/how-to-print-a-windows-form.md
Product: dotnet-framework
Technology: dotnet-winforms
GitHub Login: @adegeo
Microsoft Alias: adegeo

No problem, glad you got it working 😁


Add function to get vulnerabilities for ORT runs

Please see individual commits for details.

@mnonnenmacher could you please have a look?


bundle root certs?

  ➔ irb
> require 'http'
=> true
> HTTP.get 'https://www.google.com/'
OpenSSL::SSL::SSLError: SSL_connect returned=1 errno=0 state=SSLv3 read server certificate B: certificate verify failed
	from /Users/john/.rbenv/versions/1.9.3-p0/lib/ruby/1.9.1/net/http.rb:799:in `connect'
	from /Users/john/.rbenv/versions/1.9.3-p0/lib/ruby/1.9.1/net/http.rb:799:in `block in connect'
	from /Users/john/.rbenv/versions/1.9.3-p0/lib/ruby/1.9.1/timeout.rb:54:in `timeout'
	from /Users/john/.rbenv/versions/1.9.3-p0/lib/ruby/1.9.1/timeout.rb:99:in `timeout'
	from /Users/john/.rbenv/versions/1.9.3-p0/lib/ruby/1.9.1/net/http.rb:799:in `connect'
	from /Users/john/.rbenv/versions/1.9.3-p0/lib/ruby/1.9.1/net/http.rb:755:in `do_start'
	from /Users/john/.rbenv/versions/1.9.3-p0/lib/ruby/1.9.1/net/http.rb:744:in `start'
	from /Users/john/.rbenv/versions/1.9.3-p0/lib/ruby/1.9.1/net/http.rb:1284:in `request'
	from /Users/john/.rbenv/versions/1.9.3-p0/lib/ruby/gems/1.9.1/gems/http-0.0.1/lib/http/client.rb:80:in `request'
	from /Users/john/.rbenv/versions/1.9.3-p0/lib/ruby/gems/1.9.1/gems/http-0.0.1/lib/http/chainable.rb:56:in `request'
	from /Users/john/.rbenv/versions/1.9.3-p0/lib/ruby/gems/1.9.1/gems/http-0.0.1/lib/http/chainable.rb:10:in `get'
	from (irb):2

A root cert document could be distributed with the gem, so that it wold work with https servers. See: https://gist.github.com/996510

This might be a nice way to implement certificate checking:
https://github.com/stevegraham/certified


Avoid blocking (external) requests

Problem description
Currently the client is issuing a number of blocking requests to remote endpoints during initialization and I assume also during operation. This can lead to major degradation in function if for whatever reason these endpoints are slow or even completely unreachable.
Suggested solution
Convert all requests into non-blocking asynchronous requests to avoid these issues. It may be necessary to introduce a "locked" state for any front end widgets that indicate to the user that the widget is working in the background and to prevent the user from issuing additional inputs.
Note: I've labeled this issue as a bug and not as a feature request, because it can lead to a significant enough degradation in function that the widget should be considered as non-functional.

This has been avoided, partially, by introducing response caching through CacheControl (see PR #161).

How does #161 resolve the primary issue of blocking on timeouts?

The request responses are cached, meaning there will be no timeouts. It doesn't solve the issue of the original request, nor what happens when the cache is flushed.
The timeout is specified explicitly for all requests (currently 10 s), after which the connection will be raise a timeout, which will in all cases be handled within a try/except block.
Currently, the startup time is not considered substantial, especially not with the upgraded Voilà v0.2.x.
However, the question still remains for running this client outside Voilà, as well as when/if "many" providers will start offering OPTIMADE APIs through the OPTIMADE providers list.

Note, if the requests should be converted to asynchronous requests, the actual waiting time until the application is useful will not change. At least not in the current design.

Thank you for the explanation. While I'm sure that the caching will reduce some waiting times and server load, I do not consider this issue resolved.

So your solution would still be to acknowledge the waiting/startup time will not change, but prefer the widgets loading in, locking the UI concerning user inputs until all requests have been performed? Instead of doing it the current way, where one simply waits for the application to start, and when the UI is present, it's ready for user input?

So your solution would still be to acknowledge the waiting/startup time will not change, but prefer the widgets loading in, locking the UI concerning user inputs until all requests have been performed? Instead of doing it the current way, where one simply waits for the application to start, and when the UI is present, it's ready for user input?

From a UX perspective it is much preferable to keep the UI responsive. Blocking requests freeze the UI which is not only frustrating to users, but will in many cases lead to the reasonable assumption that the app has crashed.
Furthermore, with the current behavior, the start up of the app can freeze for a significant amount of time (I consider anything more than just a few seconds unacceptable, let alone more than 10 seconds). Unless anything significant has changed, the UI will actually freeze during start up for about half a minute when servers are down, because there are multiple requests with individual timeout. The vast majority of users (including me until I figured the issue) will assume that the app has crashed and not wait that long.

For the latter part, this is hopefully where the cacheing will come in handy - but that will of course depend on a number of things, and will not solve this issue.
I hear you, and I will keep the issue open for tracking this issue until a viable solution or final decision presents itself.
Hopefully this interaction also clarifies the issue further for future reference :)


Battery reads much higher and narrower than expected

With a 4.2-3.7V LiPo battery plugged in, MagTag().peripherals.battery reads 5.3-5.2V.

Originally reported in https://github.com/adafruit/circuitpython/issues/6148, fix in progress in https://github.com/adafruit/circuitpython/pull/6246. This is a core CircuitPython issue.

Thanks for the quick response. Is there any workaround?

Try waiting at least one second before reading the battery voltage after creating the MagTag() object. Are you calling it immediately?

More precisely, you can call it once, then wait at least one second and call it again (this is only the first time):
dafruit CircuitPython 7.2.4 on 2022-03-31; Adafruit MagTag with ESP32S2
>>>
>>> from adafruit_magtag.magtag import MagTag
>>> m = MagTag()
>>> m.peripherals.battery
5.33599
>>> m.peripherals.battery
4.28992

Thanks, that works. Why are the two reads necessary, no matter how far apart the two reads are?


update true for required atttributes

Update the required attributes in the MODEL-AD data model after RFC work.

@ameliakallaher Unfortunately for some of these attributes, we will need to use the new Component Based Rule Setting validation rule feature, rather than just setting the Required column to TRUE. This lets us specify that the same attribute may or may not be required based on which manifest it appears in.

for example, ageDeath should be required in the mouse metadata but not in the human metadata -- we occasionally have data from living individuals
individualID should be required in the individual metadata templates but not in the file annotation template, since multispecimen files won't have an individualID annotation

This is one of the many nuanced schematic headaches that Mialy spent like two months helping us figure out -- it is for sure not intuitive! I'll dig in later today and try to get the validation rules set up to replicate the requirements you have above.

Talking with @ameliakallaher , we determined that the following attributes need validation rule requirements, since whether they are required depends on which manifest they appear in:

individualID (not required in FileAnnotationTemplate)
individualIdSource (not required in IndividualHumanMetadataTemplate)
ageDeath (not required in IndividualHumanMetadataTemplate)
ageDeathUnits (not required in IndividualHumanMetadataTemplate)

@ameliakallaher the changes look good and the test templates look like they have the "required" logic we wanted! This is ready to merge unless you have more changes you want to make. 🎉


Tries to read invalid ftype field in Dir2DataEntry

According to xfs_filesystem_structure.pdf, the ftype field of struct xfs_dir2_sf_entry will only be present if the XFS_SB_VERSION2_FTYPE feature is set in the superblock.  However, xfs-fuse does not check that field, assuming instead that the ftype field will always be present.  Dir2DataEntry::from then reads garbage into that field.
This shows up when doing a readdir.  For some directory types, this results in "Unknown file type" errors, and the readdir stream terminates early.

Update: there is an error in the documentation.  It says " The XFS_SB_VERSION2_FTYPE feature must be set, or this field will not be present."  However, the Linux kernel module, which I take to be the source of truth, considers this field to be present if XFS_SB_VERSION2_FTYPE is set in sb_features2 OR XFS_SB_FEAT_INCOMPAT_FTYPE is set in sb_features_incompat .  So my initial analysis was incorrect.  Instead there's some other reason why xfuse is not correctly reading the file type in a block directory.


Message validation middleware

It would be helpful to have a out of the box message validation middleware included in the command_bus stack. It will use the standard symfony's ValidatorInterface to validate messages. It's priority should be greatest so it executes first before all others.
Will make a PR soon.

Though... not sure if this bundle is the place for the middleware class, or it should go to the MessageBus lib. Please clarify, so I can make a PR.

Maybe related: https://github.com/SimpleBus/MessageBus/issues/19

Hi @tmilos, as described in the issue pointed out by @ruudk: I don't recommend using middleware to validate a command, especially in the case where commands are instantiated because of a user action (they clicked a button, filled in a form, etc.). You should validate the command before handing it over to the command bus in the first place, which allows you to give the user some feedback. Then wherever the data from the command is used inside your domain model, add assertions and throw exceptions when anything is not right.

Sure, users of the bus should behave and do themselves the validation prior to calling the bus and present a nice message to their users. But, each layer should do its own validation, and if the bus does not guarantee that delivered messages are valid, then each command handler must have additional code and dependencies to do the validation himself. So, if there's no a validation middleware present in the stack, I would have to put in each of my handlers

public function validate($command)
{
    $errors = $this->validator->validate($command);
    if (count($errors) > 0) {
        throw new InvalidCommandExceptiion((string)$errors, $errors);
    }

    // now do interesting stuff...
}

Making each of the handlers dependent on the validator service and repeating these 4 lines of code in each of them seems quite tedious. That's not what you're thinking, right? Please explain if I'm wrong.

Right, no that's not what I meant. I mean command validation is one thing, but domain model validation (also known as invariant protection) is another thing. Once the command is actually being handled, rely on exceptions to be thrown inside the domain classes themselves.

Yes, domain throwing exceptions is great, but I'm having trouble with validation that requires additional queries, like for example uniqueness of a username. Can not base on the persistence layer DBALExceptions since then EntityManager closes and can not use it anymore. Then, what do you think about domain validation component listening on doctrine pre_persist event? Just came to my mind, though didn't see something like that before.

Ah, right. You can still have that check both in the command validator (DoctrineBundle comes with a Unique constraint validator) and in the domain layer (where you just query the repository and thrown an exception if a username is already in use).

Ok, so you don't find this as a part of the standard configuration. I'll have to figure and make something on my own. Still don't see how a command validation middleware can hurt. Anyway, thanks. Please close the issue if you don't find it relevant anymore.

Of course, it should be very easy to create and use your own validation middleware (and, why not create a package for it? ;)).
I believe it shouldn't be the recommended way to validate your commands, since inside the middleware there's no way to talk back to the user anyway, except maybe to just throw an exception, but that is not useful at all. In that case, you'd better fall back to the domain-level validations.

I'm closing this because it is open for a long time and i belief it should not be done as a core component for SimpleBus.


add support for 3.11

hello, I'm trying to add support for py3.11 for nbdev, I'm trying some stuff on my fork. The PR will also eventually close #1372.
I had a question for @jph00, there's only one test case which fails for 3.11 which is in showdoc. This has only come into existence because the docstring for the Enum was changed in 3.11. How should I try to handle the test case in such a situation? The test case is listed below,
AssertionError                            Traceback (most recent call last)
Cell In[1], line 2
      1 e = enum.Enum('e', 'a b')
----> 2 test_eq(str(show_doc(e)), '---\n\n### e\n\n>      e (value, names=None, module=None, qualname=None, type=None, start=1)\n\nAn enumeration.')

File ~/mambaforge/envs/fun/lib/python3.11/site-packages/fastcore/test.py:37, in test_eq(a, b)
     35 def test_eq(a,b):
     36     "`test` that `a==b`"
---> 37     test(a,b,equals, cname='==')

File ~/mambaforge/envs/fun/lib/python3.11/site-packages/fastcore/test.py:27, in test(a, b, cmp, cname)
     25 "`assert` that `cmp(a,b)`; display inputs and `cname or cmp.__name__` if it fails"
     26 if cname is None: cname=cmp.__name__
---> 27 assert cmp(a,b),f"{cname}:\n{a}\n{b}"

AssertionError: ==:
---

### e

>      e (value, names=None, module=None, qualname=None, type=None, start=1,
>         boundary=None)

Create a collection of name/value pairs.

Example enumeration:

>>> class Color(Enum):
...     RED = 1
...     BLUE = 2
...     GREEN = 3

Access them by:

- attribute access::

>>> Color.RED
<Color.RED: 1>

- value lookup:

>>> Color(1)
<Color.RED: 1>

- name lookup:

>>> Color['RED']
<Color.RED: 1>

Enumerations can be iterated over, and know how many members they have:

>>> len(Color)
3

>>> list(Color)
[<Color.RED: 1>, <Color.BLUE: 2>, <Color.GREEN: 3>]

Methods can be added to enumerations, and members can have their own
attributes -- see the documentation for details.
---

### e

>      e (value, names=None, module=None, qualname=None, type=None, start=1)

An enumeration.

Message ID: @.***>Thanks for looking into this. I guess the best thing to do is to just find the bit of the repr that it consistent, and have the test confirm that is correct.

I think even that won't be possible. They've changed the entire docstring and they've even added a new arg (boundary=None) to the Enum class.
# new in 3.11
e (value, names=None, module=None, qualname=None, type=None, start=1, boundary=None)
Create a collection of name/value pairs.

# in previous versions
e (value, names=None, module=None, qualname=None, type=None, start=1)
An enumeration.

Message ID: @.***>OK maybe just get rid of that test then...

Will do!

@jph00 It's ready for review.

Thank you!


Features

https://flowbite.com/docs/components/carousel/

Phpunit test

https://github.com/vueuse/motion

https://www.youtube.com/watch?v=jRknsy851IM


link is 404

https://docs.microsoft.com/windows/desktop/api/ntdef/ns-ntdef-_list_entry
check link plz

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

ID: d8fa9c5e-355f-16db-1948-f094ab67751c
Version Independent ID: 1053b3ae-c27e-397c-992d-0168dbc9916d
Content: ExInterlockedRemoveHeadList function (wdm.h) - Windows drivers
Content Source: wdk-ddi-src/content/wdm/nf-wdm-exinterlockedremoveheadlist.md
Product: windows-hardware
Technology: windows-devices
GitHub Login: @tedhudek
Microsoft Alias: tedhudek

Thanks for the heads-up @dgkim20 and apologies for the delay.
I found 42 occurrences of that bad link across the repo, and just pushed the fix to update them to https://docs.microsoft.com/windows/win32/api/ntdef/ns-ntdef-list_entry.
The fix should appear on the live site within half an hour.  Appreciate the feedback!

thanks for your support.
I confirmed the link.
thank you.


Inconsistent order of columns in getting-started-movielens tutorial

Fixes https://github.com/NVIDIA-Merlin/NVTabular/issues/1714
After https://github.com/NVIDIA-Merlin/NVTabular/pull/1715 was opened, the notebooks were moved to this Merlin repo. I'm making the identical changes since we can't merge the changes in NVT.
Thanks to bilzard for the careful read and opening the issue.

rerun tests


Added O_EXEC and O_SEARCH to solaris libc.

See #566.

@bors: r+

:pushpin: Commit 252be4d has been approved by alexcrichton

@bors: r+

:pushpin: Commit be1cbaf has been approved by alexcrichton

:hourglass: Testing commit be1cbaf with merge 0203490...


tests: logging: Add stress test

Add stress test for logging to check that it is coherent when logging from various contexts. Logging coherency means that each message ends up processed by the backend or adds up to dropped messages which are notified to backends.
Test is using ztress framework. Basic test is executed on all emulation platforms but real test (with higher sys clock frequency to generate more preemptions) is executed on selected platforms (some qemus are slow and may timeout).
Additionally, converted log_core to use spin lock.
It is based on #41969. Otherwise test would fail.

DNM until  #41969 is merged.

merged but reverted? https://github.com/zephyrproject-rtos/zephyr/commit/49d08c25fe86b2849f331a28dd73714a7c1f1e88


fix(detailed response): Update _to_dict function to support JSON array results

fixes #13

@emlaver this PR looks good! Thanks for your contribution 👍
I am wondering if we even need to support str.

@ehdsouza Thanks for reviewing!


Enhancement - higher resolution integration image

Dear myTselection,
I have noticed that the image of your integration has a rather low resolution.
If possible, could you provide it with an image with a higher resolution?
Thanks in advance.
Kind regards,
Robin

hi, which image do you mean exaclty? not fully clear to me

Dear myTselection,
I mentioned the image of the Telenet Telemeter on the integration page.
It seems like it was snipped from the website of Telenet or somewhere else, as it has a low resolution and it is a bit odd.
Only a minor enhancement, but looks nicer in the integrations page.
Thanks in advance.
Kind regards,
Robin

all HA integration images have the same mandatory resolution (256x256). The transparancy at the borders was maybe not fully optimal, but only visible in dark mode.


Codechef_CPP_COLE2020_DooffiresBrackets_CLBRKT

No description provided.

Please add your image in image folder nd update readme.md file it is all for contributors
do this then only i will be able to merge this..
Check this
#90 (comment)

okay

This branch has conflicts that must be resolved


How to setup recursion?

Hello,
For the life of me I cannot figure out how to set the -r or -R or --recursion flag to have filebot search for media in subfolders. Can it be done or am I missing something? Thank for all your efforts!

For the UI or the AMC script?
For the UI, you can add the FILEBOT_CUSTOM_OPTIONS variable and set its value to --recursion. For AMC, use the AMC_CUSTOM_OPTIONS variable instead.
https://github.com/jlesage/docker-filebot#environment-variables

Thank you!


Microsoft.Data.SqlClient throwing exception on Powershell under Linux

Describe the bug
Unable to make a SQL connection in Powershell on Linux using Microsoft.Data.SqlClient.SqlConnection.
We are using self hosted linux/ubuntu agents in Azure and need to make SQL connection with Authentication="Active Directory Password". Getting below exception . Please assist.
Exception message: Exception calling "Open" with "0" argument(s): "The type initializer for 'Microsoft.Win32.Registry' threw an exception."
Stack trace:
ErrorRecord                 : Exception calling "Open" with "0" argument(s): "The type initializer for 'Microsoft.Win32.Registry' threw an exception."
WasThrownFromThrowStatement : False
TargetSite                  : Void CheckActionPreference(System.Management.Automation.Language.FunctionContext, System.Exception)
Message                     : Exception calling "Open" with "0" argument(s): "The type initializer for 'Microsoft.Win32.Registry' threw an exception."
Data                        : {System.Management.Automation.Interpreter.InterpretedFrameInfo}
InnerException              : System.TypeInitializationException: The type initializer for 'Microsoft.Win32.Registry' threw an exception.
                               ---> System.PlatformNotSupportedException: Registry is not supported on this platform.
                                 at Microsoft.Win32.RegistryKey.OpenBaseKeyCore(RegistryHive hKey, RegistryView view)
                                 at Microsoft.Win32.Registry..cctor()
                                 --- End of inner exception stack trace ---
                                 at Microsoft.Data.ProviderBase.DbConnectionPool.CheckPoolBlockingPeriod(Exception e) in /_/src/Microsoft.Data.SqlClient/netcore/src/Microsoft/Data/ProviderBase/DbConnectionPool.NetCoreApp.cs:line 18
                                 at Microsoft.Data.ProviderBase.DbConnectionPool.CreateObject(DbConnection owningObject, DbConnectionOptions userOptions, DbConnectionInternal oldConnection) in /_/src/Microsoft.Data.SqlClient/netcore/src
                              /Microsoft/Data/ProviderBase/DbConnectionPool.cs:line 779
                                 at Microsoft.Data.ProviderBase.DbConnectionPool.UserCreateRequest(DbConnection owningObject, DbConnectionOptions userOptions, DbConnectionInternal oldConnection) in /_/src/Microsoft.Data.SqlClient/netcor
                              e/src/Microsoft/Data/ProviderBase/DbConnectionPool.cs:line 1759
                                 at Microsoft.Data.ProviderBase.DbConnectionPool.TryGetConnection(DbConnection owningObject, UInt32 waitForMultipleObjectsTimeout, Boolean allowCreate, Boolean onlyOneCheckConnection, DbConnectionOptions
                              userOptions, DbConnectionInternal& connection) in /_/src/Microsoft.Data.SqlClient/netcore/src/Microsoft/Data/ProviderBase/DbConnectionPool.cs:line 1214
                                 at Microsoft.Data.ProviderBase.DbConnectionPool.TryGetConnection(DbConnection owningObject, TaskCompletionSource`1 retry, DbConnectionOptions userOptions, DbConnectionInternal& connection) in /_/src/Micr
                              osoft.Data.SqlClient/netcore/src/Microsoft/Data/ProviderBase/DbConnectionPool.cs:line 1130
                                 at Microsoft.Data.ProviderBase.DbConnectionFactory.TryGetConnection(DbConnection owningConnection, TaskCompletionSource`1 retry, DbConnectionOptions userOptions, DbConnectionInternal oldConnection, DbCon
                              nectionInternal& connection) in /_/src/Microsoft.Data.SqlClient/netcore/src/Microsoft/Data/ProviderBase/DbConnectionFactory.cs:line 122
                                 at Microsoft.Data.ProviderBase.DbConnectionInternal.TryOpenConnectionInternal(DbConnection outerConnection, DbConnectionFactory connectionFactory, TaskCompletionSource`1 retry, DbConnectionOptions userOp
                              tions) in /_/src/Microsoft.Data.SqlClient/netcore/src/Common/src/Microsoft/Data/ProviderBase/DbConnectionInternal.cs:line 339
                                 at Microsoft.Data.ProviderBase.DbConnectionClosed.TryOpenConnection(DbConnection outerConnection, DbConnectionFactory connectionFactory, TaskCompletionSource`1 retry, DbConnectionOptions userOptions) in
                              /_/src/Microsoft.Data.SqlClient/netcore/src/Common/src/Microsoft/Data/ProviderBase/DbConnectionClosed.cs:line 39
                                 at Microsoft.Data.SqlClient.SqlConnection.TryOpen(TaskCompletionSource`1 retry, SqlConnectionOverrides overrides) in /_/src/Microsoft.Data.SqlClient/netcore/src/Microsoft/Data/SqlClient/SqlConnection.cs:
                              line 1792
                                 at Microsoft.Data.SqlClient.SqlConnection.Open(SqlConnectionOverrides overrides) in /_/src/Microsoft.Data.SqlClient/netcore/src/Microsoft/Data/SqlClient/SqlConnection.cs:line 1277
                                 at CallSite.Target(Closure , CallSite , Object )
HelpLink                    :
Source                      : System.Management.Automation
HResult                     : -2146233087
StackTrace                  :    at System.Management.Automation.ExceptionHandlingOps.CheckActionPreference(FunctionContext funcContext, Exception exception)
                                 at System.Management.Automation.Interpreter.ActionCallInstruction`2.Run(InterpretedFrame frame)
                                 at System.Management.Automation.Interpreter.EnterTryCatchFinallyInstruction.Run(InterpretedFrame frame)
                                 at System.Management.Automation.Interpreter.EnterTryCatchFinallyInstruction.Run(InterpretedFrame frame)

To reproduce
$sqlAdminUser = "xxxxx"
$sqlAdminPass = "xxxxx"
$InstanceName = "xxxx.database.windows.net"
$authType = "Active Directory Password"
$connectionString = "Server=$InstanceName;Database=sqldb;Integrated Security=False;User ID=$sqlAdminUser;Password=$sqlAdminPass;Authentication=$authType"
$query = "some query to DB"
$connection = New-Object Microsoft.Data.SqlClient.SqlConnection
$connection.ConnectionString = $connectionString
$connection.Open()
Expected behavior
Should be able to connect to SQL database and execute query
Further technical details
Microsoft.Data.SqlClient version: .3.0.1
.NET Core SDK (reflecting any global.json):
Version:   3.1.415
Commit:    52a108342e
Runtime Environment:
OS Name:     ubuntu
OS Version:  20.04
OS Platform: Linux
RID:         ubuntu.20.04-x64
Base Path:   /usr/share/dotnet/sdk/3.1.415/
Host (useful for support):
Version: 3.1.21
Commit:  df8abc0f7e
.NET Core SDKs installed:
3.1.415 [/usr/share/dotnet/sdk]
.NET Core runtimes installed:
Microsoft.AspNetCore.App 3.1.21 [/usr/share/dotnet/shared/Microsoft.AspNetCore.App]
Microsoft.NETCore.App 3.1.21 [/usr/share/dotnet/shared/Microsoft.NETCore.App]
SQL Server version: Azure SQL Server 12
Operating system: Agent running ubuntu 20.04 ,
Powershell version : 7.2
Additional context
In order to make Microsoft.Data.SqlClient work in Powershell on Linux,  had to to the following:
Register-PackageSource -provider NuGet -name nugetRepository -location https://www.nuget.org/api/v2 -Trusted
Install-Package Microsoft.Data.SqlClient -Provider nuget -SkipDependencies -Destination /opt/microsoft/powershell/7/ -Force
Install-Package Microsoft.Identity.Client -Provider nuget -SkipDependencies -Destination /opt/microsoft/powershell/7/ -Force
Add-Type -AssemblyName System.Data
Add-Type -Path('/opt/microsoft/powershell/7/Microsoft.Identity.Client.4.37.0/lib/net461/Microsoft.Identity.Client.dll')
Add-Type -Path ('/opt/microsoft/powershell/7/Microsoft.Data.SqlClient.3.0.1/runtimes/unix/lib/netcoreapp3.1/Microsoft.Data.SqlClient.dll')
Install-Package Azure.Core -Provider nuget -SkipDependencies -RequiredVersion <IP_ADDRESS> -Destination /opt/microsoft/powershell/7/ -Force
Add-Type -Path('/opt/microsoft/powershell/7/Azure.Core.1.6.0/lib/net461/Azure.Core.dll')
Install-Package Azure.Identity -Provider nuget -SkipDependencies -RequiredVersion <IP_ADDRESS> -Destination /opt/microsoft/powershell/7/ -Force
Add-Type -Path('/opt/microsoft/powershell/7/Azure.Identity.1.3.0/lib/netstandard2.0/Azure.Identity.dll')

Looks like it's using the wrong binary. That's a windows binary as you can see because it's using the registry. You need to use nuget to restore the correct one.
See if this issue helps, https://github.com/dotnet/SqlClient/issues/1301

Yes, I am am using nuget, You can see that in the above in Install-Package command lines, the Provider is nuget. Also I followed the solution described in #1301 and added few more assemblies on top of it as required.

I wonder if Microsoft.Data.SqlClient provider been tested with Powershell on Linux with Authentication="Active Directory Password" ? If yes, Can someone please provide examples or documentation on that. Thanks

Watch out for what you’re installing, these are .NET Framework DLLs, not for Linux.
Add-Type -Path('/opt/microsoft/powershell/7/Microsoft.Identity.Client.4.37.0/lib/net461/Microsoft.Identity.Client.dll')
Add-Type -Path('/opt/microsoft/powershell/7/Azure.Core.1.6.0/lib/net461/Azure.Core.dll')

@umersyed is the issue solved? if yes, please feel free to close the issue.

@cheenamalhotra thanks for pointing out that, managed to load the right DLLs for linux.
@JRahnama Will confirm soon as still in process of setting up and testing the new provider. thanks

Its working for me now. Had to load below packages :
Register-PackageSource -provider NuGet -name nugetRepository -location https://www.nuget.org/api/v2 -Trusted -ErrorAction SilentlyContinue
Install-Package Microsoft.Data.SqlClient -Provider Nuget -SkipDependencies -Scope CurrentUser -RequiredVersion 3.0.1 -Destination ~/PackageManagement/NuGet/Packages -Force
Install-Package Microsoft.Identity.Client -Provider Nuget -SkipDependencies -Scope CurrentUser -RequiredVersion 4.37.0 -Destination ~/PackageManagement/NuGet/Packages -Force
Add-Type -AssemblyName System.Data
Add-Type -Path('~/PackageManagement/NuGet/Packages/Microsoft.Identity.Client.4.37.0/lib/netcoreapp2.1/Microsoft.Identity.Client.dll')
Add-Type -Path ('~/PackageManagement/NuGet/Packages/Microsoft.Data.SqlClient.3.0.1/runtimes/unix/lib/netcoreapp3.1/Microsoft.Data.SqlClient.dll')
Install-Package Azure.Core -Provider Nuget -Scope CurrentUser -SkipDependencies -RequiredVersion <IP_ADDRESS> -Destination ~/PackageManagement/NuGet/Packages -Force
Add-Type -Path('~/PackageManagement/NuGet/Packages/Azure.Core.1.6.0/lib/netstandard2.0/Azure.Core.dll')
Install-Package Azure.Identity -Provider Nuget -Scope CurrentUser -SkipDependencies -RequiredVersion <IP_ADDRESS> -Destination ~/PackageManagement/NuGet/Packages -Force
Add-Type -Path('~/PackageManagement/NuGet/Packages/Azure.Identity.1.3.0/lib/netstandard2.0/Azure.Identity.dll')
Can close this issue now
Thank you

Its working for me now. Had to do the following :
Register-PackageSource -provider NuGet -name nugetRepository -location https://www.nuget.org/api/v2 -Trusted -ErrorAction SilentlyContinue
Install-Package Microsoft.Data.SqlClient -Provider Nuget -SkipDependencies -Scope CurrentUser -RequiredVersion 3.0.1 -Destination ~/PackageManagement/NuGet/Packages -Force
Install-Package Microsoft.Identity.Client -Provider Nuget -SkipDependencies -Scope CurrentUser -RequiredVersion 4.37.0 -Destination ~/PackageManagement/NuGet/Packages -Force
Add-Type -AssemblyName System.Data
Add-Type -Path('~/PackageManagement/NuGet/Packages/Microsoft.Identity.Client.4.37.0/lib/netcoreapp2.1/Microsoft.Identity.Client.dll')
Add-Type -Path ('~/PackageManagement/NuGet/Packages/Microsoft.Data.SqlClient.3.0.1/runtimes/unix/lib/netcoreapp3.1/Microsoft.Data.SqlClient.dll')
Install-Package Azure.Core -Provider Nuget -Scope CurrentUser -SkipDependencies -RequiredVersion <IP_ADDRESS> -Destination ~/PackageManagement/NuGet/Packages -Force
Add-Type -Path('~/PackageManagement/NuGet/Packages/Azure.Core.1.6.0/lib/netstandard2.0/Azure.Core.dll')
Install-Package Azure.Identity -Provider Nuget -Scope CurrentUser -SkipDependencies -RequiredVersion <IP_ADDRESS> -Destination ~/PackageManagement/NuGet/Packages -Force
Add-Type -Path('~/PackageManagement/NuGet/Packages/Azure.Identity.1.3.0/lib/netstandard2.0/Azure.Identity.dll')

Can close this issue now
Thank you

@umersyed  thanks for the update.
closing as the issue is resolved.

Its working for me now. Had to do the following :
Register-PackageSource -provider NuGet -name nugetRepository -location https://www.nuget.org/api/v2 -Trusted -ErrorAction SilentlyContinue
Install-Package Microsoft.Data.SqlClient -Provider Nuget -SkipDependencies -Scope CurrentUser -RequiredVersion 3.0.1 -Destination ~/PackageManagement/NuGet/Packages -Force
Install-Package Microsoft.Identity.Client -Provider Nuget -SkipDependencies -Scope CurrentUser -RequiredVersion 4.37.0 -Destination ~/PackageManagement/NuGet/Packages -Force
Add-Type -AssemblyName System.Data
Add-Type -Path('~/PackageManagement/NuGet/Packages/Microsoft.Identity.Client.4.37.0/lib/netcoreapp2.1/Microsoft.Identity.Client.dll')
Add-Type -Path ('~/PackageManagement/NuGet/Packages/Microsoft.Data.SqlClient.3.0.1/runtimes/unix/lib/netcoreapp3.1/Microsoft.Data.SqlClient.dll')
Install-Package Azure.Core -Provider Nuget -Scope CurrentUser -SkipDependencies -RequiredVersion <IP_ADDRESS> -Destination ~/PackageManagement/NuGet/Packages -Force
Add-Type -Path('~/PackageManagement/NuGet/Packages/Azure.Core.1.6.0/lib/netstandard2.0/Azure.Core.dll')
Install-Package Azure.Identity -Provider Nuget -Scope CurrentUser -SkipDependencies -RequiredVersion <IP_ADDRESS> -Destination ~/PackageManagement/NuGet/Packages -Force
Add-Type -Path('~/PackageManagement/NuGet/Packages/Azure.Identity.1.3.0/lib/netstandard2.0/Azure.Identity.dll')

Can close this issue now Thank you

Thanks for this, spent several days trying to figure this out!

Agreed, that super helped, @umersyed thank you


CrashReporter: History load failure

Looks like a history load error:
"@timestamp": "2015-06-22T16:23:35.060-05:00",
"level": "ERROR",
"thread_name": "safe-fixed-unlock-wallet-0",
"logger_name": "org.multibit.hd.core.error_reporting.ExceptionHandler",
"message": "Uncaught exception",
"stack_trace": "org.multibit.hd.core.exceptions.HistoryLoadException: Could not parse input stream to protobuf
at org.multibit.hd.core.store.HistoryProtobufSerializer.readHistoryEntries(HistoryProtobufSerializer.java:93) [multibit-hd.jar:na]
at org.multibit.hd.core.services.PersistentHistoryService.loadHistory(PersistentHistoryService.java:177) [multibit-hd.jar:na]
at org.multibit.hd.core.services.PersistentHistoryService.initialise(PersistentHistoryService.java:98) [multibit-hd.jar:na]
at org.multibit.hd.core.services.PersistentHistoryService.(PersistentHistoryService.java:78) [multibit-hd.jar:na]
at org.multibit.hd.core.services.CoreServices.getOrCreateHistoryService(CoreServices.java:577) [multibit-hd.jar:na]
at org.multibit.hd.ui.views.wizards.credentials.CredentialsWizardModel.checkPasswordAndLoadWallet(CredentialsWizardModel.java:781) [multibit-hd.jar:na]
at org.multibit.hd.ui.views.wizards.credentials.CredentialsWizardModel.access$500(CredentialsWizardModel.java:57) [multibit-hd.jar:na]
at org.multibit.hd.ui.views.wizards.credentials.CredentialsWizardModel$10.call(CredentialsWizardModel.java:711)

We'll need better error handling here (and the related Contacts and Payments streams) with a possible fallback to a local backup.

The History error handling was different to Contacts and Payments in that it handed over an error to the crash reporter rather than passing it up the stack. However the problem of correctly handling it remains.

Backups for history/contacts/payments is already raised as #407

Given that the longer term fix is currently scheduled for Release 0.2 there's no need for this issue to remain open.
Ready for review and close.

This is now OK but #407 needs to be implemented.
If there is a problem with the history the user sees a 'the password cannot unlock the wallet' rather than the crash reporter. This is better but needs a bit more work.

Reopened as still seeing:
"@timestamp": "2015-06-30T21:28:19.670+02:00",
"level": "ERROR",
"thread_name": "safe-fixed-unlock-wallet-0",
"logger_name": "org.multibit.hd.core.error_reporting.ExceptionHandler",
"message": "Uncaught exception",
"stack_trace": "org.multibit.hd.core.exceptions.HistoryLoadException: Could not parse input stream to protobuf
at org.multibit.hd.core.store.HistoryProtobufSerializer.readHistoryEntries(HistoryProtobufSerializer.java:93) [multibit-hd.jar:na]
at org.multibit.hd.core.services.PersistentHistoryService.loadHistory(PersistentHistoryService.java:177) [multibit-hd.jar:na]
at org.multibit.hd.core.services.PersistentHistoryService.initialise(PersistentHistoryService.java:98) [multibit-hd.jar:na]
at org.multibit.hd.core.services.PersistentHistoryService.(PersistentHistoryService.java:78) [multibit-hd.jar:na]
at org.multibit.hd.core.services.CoreServices.getOrCreateHistoryService(CoreServices.java:577) [multibit-hd.jar:na]
at org.multibit.hd.ui.views.wizards.credentials.CredentialsWizardModel.checkPasswordAndLoadWallet(CredentialsWizardModel.java:781) [multibit-hd.jar:na]
at org.multibit.hd.ui.views.wizards.credentials.CredentialsWizardModel.access$500(CredentialsWizardModel.java:57) [multibit-hd.jar:na]
at org.multibit.hd.ui.views.wizards.credentials.CredentialsWizardModel$10.call(CredentialsWizardModel.java:711) [multibit-hd.jar:na]
at org.multibit.hd.ui.views.wizards.credentials.CredentialsWizardModel$10.call(CredentialsWizardModel.java:703)

Should be fixed by #647
Awaiting review and closing.

Now that history is gone this should also disappear. Closing.


Align description of Unsigned Tx in Platform Reference

A small layout fix.
But I noticed that the content of : build/references/platform-transaction-format.md and build/references/platform-transaction-serialization.md is the same (I did the change in both files)

It looks like build/references/platform-transaction-format.md is an unused file.

It looks like build/references/platform-transaction-format.md is an unused file.

It looks like build/references/platform-transaction-format.md is an unused file.

Should I delete it in this PR ?

It looks like build/references/platform-transaction-format.md is an unused file.

Should I delete it in this PR ?


Write proper log

I think there has to be some kind of file that logs everything that has happened in runtime.

We have to write to some file. I suggest using txt


Remove package-lock.json

Following https://github.com/pkgjs/wiby/pull/38#pullrequestreview-483336058 and https://github.com/pkgjs/wiby/pull/23#issuecomment-667562987, this PR:

removes package-lock.json file
setups npm to not create it
adds lock files to .gitignore

Gah, this has a conflict now - mind updating with master and then I can merge?

@dominykas sure, rebased

:tada: This PR is included in version 0.1.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

:tada: This PR is included in version 0.1.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


hero-section task completed

Issue: 6
Short description of what this resolves:
added text file for hero-section

Nice work @jaydeepxx👍


Make Music Playlist Shuffler

Spotify? No python!
Develop a script that reads a folder of music files and generates a shuffled playlist based on user preferences, such as genre or artist.

@sumanth-0 Please assign this to me


Angular2

This is a great library and on some projects we're starting to move to Angular2.  Are there plans to make an Angular2 version?

Yeah I plan on, at some point, porting this to Angular 2.

The Angular 2 Material team just indicated they are punting on a data tables component and wont start developing something until at least the end of the year.  md-data-table has a good opportunity to become the data table that developers go to for Angular 2.  It's incredible in Angular 1 and I look forward to your Angular 2 port.  Good work @daniel-nagy !!

@daniel-nagy I'm curious if you've given any thought to what the design would be like in ng2? Would the template structure stay pretty much the same or (optionally?) consolidate using new ng2 tricks like ng-content and @ContentChildren(TemplateRef)?

@ollwenjones honestly I have't played around much with Angular2 yet. I imagine the structure would be mostly the same. Components should clean things up nicely in general.

Thanks

+1 Very very nice component. Please port it into Angular 2

@pantonis https://github.com/swimlane/angular2-data-table

@jraadt Yeah with the announcement that they're basically only in maintenance mode for angular material 1 I think it's pretty definite that they won't be making an official version or try to integrate this. At least in the near future.
@DzmitryShylovich Thinking it might be intresting to perhaps try to integrate the styles in this project as a theme to https://github.com/swimlane/angular2-data-table. As it's a lot more feature-rich as a data-table whereas this project is primarily low-level styling for table's in general. @ThomasBurleson, the head of the angular material team has also previously indicated he was at least interested in the solutions of the ng1 version https://github.com/swimlane/angular-data-table.

Interesting. I started using this library first. Tried a few others, including the SwimLane one and eventually switched back. I really liked the SwimLane table but had a lot of issues trying to use it and it seemed to really stagnate for a while. I just visited the site while posting this and all of the demos appear to be working now. I have always liked the styling of md-data-table the best and think it adheres to the Material Design spec better than any of the others I've tried.  md-data-table seems to leverage Angular Material the best of the data tables I looked at and feels more consistent with how Angular Material is implemented.

Hi all - angular2-data-table author here :) ...
@Koslun I'm absolutely interested in improving the material theme in my table. If you wanna collab with me on it, hit me up on gitter channel. We don't use material internally so I don't want to couple the component to that ( nor would anyway ) but if we can match general styling / etc that would be great.
As for my direction on the table, I aim to deliver a table solution that renders data fast and is flexible. It doesn't make any assumptions about the way you filter/search/edit/fetch your data. I find as I built applications each use case is different and the one-size-fits all filter/etc often just causes extra pain when trying to integrate and then of course it bloats the download.
@f1tech The angular1 version is still active, I'm using it in production for some time without issues. I haven't had time to address every concern that someone has brought up but always pointed them in the right direction and gave feedback if they wanted to PR and fix.
I'll be on ngAir next month talking about the table, would love feedback or questions you might want answered. https://plus.google.com/app/basic/events/crosrdtjjeh64edh931etvrra5g

@amcdnl thanks for responding here. Currently md-data-table is meeting all of my needs and we'll be in Angular 1.5.x for a while. There was a lot I liked about the swimlane data table. At the time I was evaluating md-data-table seemed to be more production ready. I'd love to see the 2 converge for Angular 2.

@amcdnl Yeah I think I would be interested in helping out to some extent. In the short-term I am however more committed to trying to contribute to generator-angular-fullstack, where we're trying to make a transition to Angular 2.
In connection to that I also want to help move the generator to Webpack 2, TypeScript 2 and using Angular 2's offline compiler. Where I'm kind of waiting on Angular 2's offline compiler to become more production-ready with corresponding documentation and similarly also waiting on documentation for Webpack 2.
Think I'll be more free to help out after that but can at the very least give feedback and ball ideas in the mean time.
On that note, avoiding being dependent on Angular Material is one of the things I did not consider. Another is whether you were planning on supporting tree grids, but can see now that you are.
Regardless, I think getting angular2-data-table to comply with the material design specification can be broken down into two bigger features, where I would also advocate for a third.

"Just" making the table look more like the material design specification should be pretty doable and mostly just require some CSS changes, where this project surely can serve as an example. What you lose by not depending on Angular Material 2 in this respect is probably theming but not sure that it matters much for a table. Also figure that you separate the toolbar to a separate component.

The largest change or added feature I imagine is needed for the table is however inline menus, edit dialogs and other potential controls, like a datepicker. If any custom control would be included by default I think there would likely be a definite need for some kind of dependency on Angular Material 2 core and even more Angular material 2 components like the md-select component used for the inline menu in this table. Can think of two approaches to this.
a. Some kind of plugin system. Where you separate this theme to a separate npm package which in turn has all of these dependencies or peer-dependencies.
b.  Be able to include any component or directive, with certain restrictions, into the cells of a certain column. Think this would be the ideal solution depending on the restrictions but imagine it would also be one of the trickiest things to do. It is however something you can easily do with this project given its more low-level nature.

A final feature I think would be neat is the ability to propagate a click event on each cell of a column to an arbitrary function. Where I figure adding an onCellClick EventEmitter would be ideal. Think this is radically easier than number 2 and something you would also need to do for number 2 either way. Or at least stop the row click event propagation like it is done in this table. Though it might be done very differently in Angular2-data-table.

@amcdnl General thoughts and which of the three do you think would fit well into Angular2-data-table?

Yes, totally agree. Wasn't a huge priority for me as I don't use that styling but I know its popular so I implemented it as a base.

There is no plans for date pickers/edit dialogs/menus/etc in this project.  If your looking for that, I'd look at something like KendoUI.

You can absolutely do all that already, we have expressive cell templates where you can define your own components/data/etc for each cell. See this: https://github.com/swimlane/angular2-data-table/blob/master/src/demos/expressive.ts#L26

@amcdnl Regarding 3), interesting and very cool :), totally missed that. Then I take it my 2 (b) is more or less done already. As you mentioned and I see now is described in the example documentation you can just create your own "rich component within your grid" already. So I take it that you could just include any angular material component or some other cool component already. As I tried to mention, I think this is definitely the ideal approach, so think 2) is a good stance. The angular2-data-table demos seem to not be rendering for some reason so haven't quite tried it out yet though.
What I still wonder is if you can bypass the click event on a cell-by-cell basis. I.e. will the row selection be triggered or completely absorb every click on the cell even if I put an ngClick on the cell? Or can I disable it depending on the cell? So that you don't get the row being selected when you're trying to interact with a cell. Like when you're changing an inline menu you've thrown into a cell.
Seems like the ng2 documentation isn't there yet for Template ref but a bit about it in the Basics section and in the Developer Guide. However still not completely sure how you're using the template so might look into that later.

@Koslun
Yes, thats the approach i'm taking. Keeps it nice and lite.
Ugh, yes I know the demos are rendering I've been hacking at it to make it work better with webpack and broke the output file. Will be working on that Friday.
You can hook whatever event you want up, your template runs outside the scope of the table and I don't swallow those events.
Yes, it was very tricky hehe. I worked w/ some of the core angular guys to figure out a clever approach.

@amcdnl

You can hook whatever event you want up, your template runs outside the scope of the table and I don't swallow those events.

Oh ok, that sounds great. So does that mean that putting in a template which listens to these click events  will swallow up any click events directed at the entire row and thus avoid row selection being triggered when clicking on cells with ngClick?

Yes, it was very tricky hehe. I worked w/ some of the core angular guys to figure out a clever approach.

Yes, I can only imagine, great job with both projects so far :+1:. Definitely moving to use Angular2-data-table when I move to Angular 2 and think this project (md-data-table) should definitely just integrate their styles into the material design or similar theme and possibly create multiple other separate components like an edit-dialog component and a md-data-table-toolbar component.
Given that @daniel-nagy or someone else doesn't beat me to it, I'll try to help with that contribution effort with aforementioned material design tweaks.

just wrote myself a JIRA to try the swimlanes table out @amcdnl :smile: My availability to contribute to ng2-material table turned out to be pretty limited. :frowning:

Selection control is optional and not really tied to any state inside the component, you pass it whats selected so you could turn off selection and just do your own thing if you had some special case.
I welcome PRs! Look forward to seeing what you guys come up with!

didn't catch what was said about material spec earlier, but I spent some time getting the ng2-material styling tight with the spec, if that SCSS helps anyone. The only thing that isn't right that I know of is the checkbox style got bumped in the upgrade from ng2-material checkbox to official material 2 checkbox.

@amcdnl
Cool seems manageable then. Will look into it more later on but take it that a PR for selection is not necessarily needed. Might be worth demonstrating as a demo though if nothing else.
@ollwenjones
Probably. Need to add a few styles to get Angular2-data-table to mirror the material design specification as well as this project. While we don't want to have to include a lot of ng2-material. Thinking you could probably use the expressive templates to add ng2-material checkboxes in the first column. Thus likely making it appropriate to showcase a demo of both a selection and checkbox setup that mirrors the material design specification more closely.

I checked all available alternatives. But this one rocks! Would be cool to see this ported to angular2

Up


Update file_writer.c

occured -> occurred

Dear Ikko Eltociear Ashimine
I submitted the pull request, thank you very much for your insight.
I hope to see your requests again.
Thanks
On Tue, Jan 30, 2024 at 6:50 PM Ikko Eltociear Ashimine <
@.***> wrote:

occured -> occurred
You can view, comment on, or merge this pull request online at:
https://github.com/KaisenAmin/c_std/pull/5
Commit Summary

20e1e56
https://github.com/KaisenAmin/c_std/pull/5/commits/20e1e56f808ad1a2615d1d902fbf03ff338a9de8
Update file_writer.c

File Changes
(1 file https://github.com/KaisenAmin/c_std/pull/5/files)

M file_io/file_writer.c
https://github.com/KaisenAmin/c_std/pull/5/files#diff-939ef6b8063cc487005048b4df5fffb43584876ae5a45db7943408a299d4e2be
(4)

Patch Links:

https://github.com/KaisenAmin/c_std/pull/5.patch
https://github.com/KaisenAmin/c_std/pull/5.diff

—
Reply to this email directly, view it on GitHub
https://github.com/KaisenAmin/c_std/pull/5, or unsubscribe
https://github.com/notifications/unsubscribe-auth/AI23CKIXVIWZZUUEYYNMXXDYREFVHAVCNFSM6AAAAABCROECHGVHI2DSMVQWIX3LMV43ASLTON2WKOZSGEYDQMJSGU2TEMY
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>


Ordering/offset

Codecov Report

Merging #277 (55f908a) into feature/ordering (e9b93fc) will increase coverage by 0.00%.
The diff coverage is 9.37%.

@@                Coverage Diff                @@
##           feature/ordering     #277   +/-   ##
=================================================
  Coverage             45.66%   45.66%
=================================================
  Files                   110      110
  Lines                  5120     5122    +2
=================================================
+ Hits                   2338     2339    +1
- Misses                 2607     2609    +2
+ Partials                175      174    -1

Flag
Coverage Δ

integration
29.20% <ø> (ø)

unittests
43.14% <9.37%> (+<0.01%)
:arrow_up:

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

Impacted Files
Coverage Δ

internal/offset/model.go
100.00% <ø> (ø)

internal/subscriber/subscriber.go
0.00% <0.00%> (ø)

internal/subscription/core.go
13.98% <0.00%> (-0.23%)
:arrow_down:

internal/topic/core.go
23.00% <0.00%> (-0.21%)
:arrow_down:

internal/offset/core.go
46.51% <25.00%> (+1.05%)
:arrow_up:

internal/subscription/model.go
100.00% <100.00%> (ø)

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 e9b93fc...55f908a. Read the comment docs.


core dumped in PyCall 1.0.2

I met core dumped in irb using PyCall 1.0.2
irb(main):001:0> require 'pycall'
=> true
irb(main):002:0> PyCall.import_module("math")
Could not find platform independent libraries <prefix>
Could not find platform dependent libraries <exec_prefix>
Consider setting $PYTHONHOME to <prefix>[:<exec_prefix>]
Fatal Python error: Py_Initialize: Unable to get the locale encoding
ModuleNotFoundError: No module named 'encodings'

Current thread 0x00007fcfae91c700 (most recent call first):
zsh: abort (core dumped)  irb

My environment is...
Python installed using pyenv(anaconda3-4.4.0).
% python --version
Python 3.6.1 :: Anaconda 4.4.0 (64-bit)

Ruby install using rbenv
% ruby -v
ruby 2.3.1p112 (2016-04-26 revision 54768) [x86_64-linux]

It is same as #36.


I want to know how many epoches do I have to run..

Hello, I'm ml student and I'm studying about R-net. While I doing that, I ran this code about 10 epoches,
and I get only 10-13% acc. And loss doesn't go down anymore when it reaches 16.xx~. I think it is not right result that you said. I want to know where I miss..
Because of hardware limit, I ran with glove - 50 dim, and batch size - 30. And other things same.
Can you let me know how many epoches do I have to run until I get your result?

@e05087 May you try again with updated instructions?
Also, consider choosing Theano as backend, it may help you speed up training process.
We got EM=57.52% and F1=67.42% with model trained about 30 epochs.


Color change to be dynemic.

Dear Y Blake,
I tried using your app in below business case and it contradicts.
For lower  expenses,  I need color to be reversed (Green if expenses are less). But App is showing red color.
example - Budgeted Value 1,00,000 Actual Value 75,000.
This should show -25% (Which is correct now also) and color should be green )Now its red).
So kindly bring one option of reverse color.
Its great if you allow to chose color for -ve and +ve % value.
It would be fantastic if multicolor based on range is allowed
Example- Green for range -100 to -90, Yellow 90 to 100 and Red for 0 to 100

Hi,
This is not an issue, but I'll keep as new feature request.
May be the new range slider property in Sense 1.1 will help for that.
We have to handle the fact that sometime you may have expenses AND ressources in same data, i.e. when you analyze an accountancy balance.


[BUG] Loading Saved Marlin Quantized Models Fails

Describe the bug
After saving a marlin model to disk with save_pretrained, reloading the model fails since the quantization config still has gptq in in.
Hardware details
A100
Software version
Current main
To Reproduce

Load a model in marlin format and save to disk

from auto_gptq import AutoGPTQForCausalLM

model = AutoGPTQForCausalLM.from_quantized("TheBloke/Llama-2-7B-Chat-GPTQ", use_marlin=True)
model.save_pretrained("/network/rshaw/llama_marlin")

Restart python and try to reload the saved model:

from auto_gptq import AutoGPTQForCausalLM
model = AutoGPTQForCausalLM.from_quantized("/network/rshaw/llama_marlin", use_marlin=True)

Fails with:
{
	"name": "ValueError",
	"message": "QuantLinear() does not have a parameter or a buffer named B.",
	"stack": "---------------------------------------------------------------------------
ValueError                                Traceback (most recent call last)
Cell In[2], line 3
      1 from auto_gptq import AutoGPTQForCausalLM
----> 3 model = AutoGPTQForCausalLM.from_quantized(\"/network/rshaw/llama_marlin\", use_marlin=True)

File /network/rshaw/gptq-benchmarking/AutoGPTQ/auto_gptq/modeling/auto.py:129, in AutoGPTQForCausalLM.from_quantized(cls, model_name_or_path, device_map, max_memory, device, low_cpu_mem_usage, use_triton, inject_fused_attention, inject_fused_mlp, use_cuda_fp16, quantize_config, model_basename, use_safetensors, trust_remote_code, warmup_triton, trainable, disable_exllama, disable_exllamav2, **kwargs)
    123 # TODO: do we need this filtering of kwargs? @PanQiWei is there a reason we can't just pass all kwargs?
    124 keywords = {
    125     key: kwargs[key]
    126     for key in list(signature(quant_func).parameters.keys()) + huggingface_kwargs
    127     if key in kwargs
    128 }
--> 129 return quant_func(
    130     model_name_or_path=model_name_or_path,
    131     device_map=device_map,
    132     max_memory=max_memory,
    133     device=device,
    134     low_cpu_mem_usage=low_cpu_mem_usage,
    135     use_triton=use_triton,
    136     inject_fused_attention=inject_fused_attention,
    137     inject_fused_mlp=inject_fused_mlp,
    138     use_cuda_fp16=use_cuda_fp16,
    139     quantize_config=quantize_config,
    140     model_basename=model_basename,
    141     use_safetensors=use_safetensors,
    142     trust_remote_code=trust_remote_code,
    143     warmup_triton=warmup_triton,
    144     trainable=trainable,
    145     disable_exllama=disable_exllama,
    146     disable_exllamav2=disable_exllamav2,
    147     **keywords
    148 )

File /network/rshaw/gptq-benchmarking/AutoGPTQ/auto_gptq/modeling/_base.py:1109, in BaseGPTQForCausalLM.from_quantized(cls, model_name_or_path, device_map, max_memory, device, low_cpu_mem_usage, use_triton, use_qigen, use_marlin, torch_dtype, inject_fused_attention, inject_fused_mlp, use_cuda_fp16, quantize_config, model_basename, use_safetensors, trust_remote_code, warmup_triton, trainable, disable_exllama, disable_exllamav2, **kwargs)
   1103     model = convert_to_marlin(model, quant_linear_class, quantize_config, repack=False)
   1104 else:
   1105     # Loading the GPTQ checkpoint to do the conversion.
   1106     # TODO: Avoid loading the model with wrong QuantLinear, and directly use
   1107     # Marlin ones. The repacking can be done directly on the safetensors, just
   1108     # as for AWQ checkpoints.
-> 1109     accelerate.utils.modeling.load_checkpoint_in_model(
   1110         model,
   1111         dtype=torch_dtype,  # This is very hacky but works due to https://github.com/huggingface/accelerate/blob/bd72a5f1a80d5146554458823f8aeda0a9db5297/src/accelerate/utils/modeling.py#L292
   1112         checkpoint=model_save_name,
   1113         device_map=device_map,
   1114         offload_state_dict=True,
   1115         offload_buffers=True
   1116     )
   1118     model = convert_to_marlin(model, quant_linear_class, quantize_config, repack=True)
   1120     # Cache the converted model.

File ~/.conda/envs/autogptq-env/lib/python3.10/site-packages/accelerate/utils/modeling.py:1550, in load_checkpoint_in_model(model, checkpoint, device_map, offload_folder, dtype, offload_state_dict, offload_buffers, keep_in_fp32_modules, offload_8bit_bnb)
   1548                 offload_weight(param, param_name, state_dict_folder, index=state_dict_index)
   1549         else:
-> 1550             set_module_tensor_to_device(
   1551                 model,
   1552                 param_name,
   1553                 param_device,
   1554                 value=param,
   1555                 dtype=new_dtype,
   1556                 fp16_statistics=fp16_statistics,
   1557             )
   1559 # Force Python to clean up.
   1560 del checkpoint

File ~/.conda/envs/autogptq-env/lib/python3.10/site-packages/accelerate/utils/modeling.py:301, in set_module_tensor_to_device(module, tensor_name, device, value, dtype, fp16_statistics)
    298     tensor_name = splits[-1]
    300 if tensor_name not in module._parameters and tensor_name not in module._buffers:
--> 301     raise ValueError(f\"{module} does not have a parameter or a buffer named {tensor_name}.\")
    302 is_buffer = tensor_name in module._buffers
    303 old_value = getattr(module, tensor_name)

ValueError: QuantLinear() does not have a parameter or a buffer named B."
}

I believe this occurs because the safetensors file has the marlin formatted model, but the quantization config is still gptq.
Expected behavior
It would be nice if there were a way to serialize marlin models cleanly, such that they can be reloaded.
I am targeting the vLLM case right now and I need a serialized format that I can reload cleanly by iterating the safetensors file to make the integration with vLLM work nicely. I would rather rely on HF/AutoGPTQ's serialization format rather than making my own if possible.
Is there any way we can create a quantization config that will allow for reloading these models?
Screenshots
If applicable, add screenshots to help explain your problem.
Additional context
Add any other context about the problem here.

Perhaps rather than having a full new config, we can have an attribute called is_marlin: bool in the GPTQ config that we consider when loading the model

@fxmarty helpful if I post a PR for this?

see https://github.com/AutoGPTQ/AutoGPTQ/pull/522
@fxmarty @qwopqwop200

@rib-2 Thank you!


Fix hestia / vesta adapters

Closes #470

Okay, so it turns out hestia was sometimes outputting a return with a space IE "   0" instead of "0" so the comparison will be a failure. I've wrapped the result with intval which looks to work correctly as a fix for it


[FEATURE] script_score sub-type in rerank processor

Is your feature request related to a problem?
I have successfully setup and run hybrid search using OpenSearch on my data but I want to tune the results so that the results which are larger in (string) length (for a specific field) are boosted slightly over those results which are relatively smaller in length. The motivation for this is that sometimes, semantic search returns results that are much shorter (think a sentence) than a very long query string (think a paragraph of text) and I would like to reduce the relevance of such results.
What solution would you like?
Ability to write my own simple script to rerank like in the script_score function.
What alternatives have you considered?
An inferior alternative would be to use script_score (within function_score ) and compare the length of the query string with the length of the result. But this comparison is different from the request, since it is a comparison between the individual search result and the query. What I want is to compare all the search results with each other using some simple function (like script_score) which is not currently possible.
Do you have any additional context?
Using the Painless scripting language might be more fitting.

@joelthe1 if I understand the feature request properly, you are requesting an ability in re-ranking processor which can run a custom logic on the final search results to re-ranking the documents? is that a correct understanding?

@HenryL27  will you be able to take look?

I'm not gonna have time for this, no. Sorry

@joelthe1 if I understand the feature request properly, you are requesting an ability in re-ranking processor which can run a custom logic on the final search results to re-ranking the documents? is that a correct understanding?

Yes @navneet1v, that is correct. Unfortunately, I am not in a position to work on this myself but do think it would benefit more than just me.

@joelthe1 Could you provide more details to clarify the request? For example, an API example and the expected behavior would be helpful.


React/Dropdown/Trigger. Click on any dropdown item makes every item styled with font-weight 700 and background color like if all  items selected at the same time.

Bug Report
Steps
Just open Dropdown/Trigger in semantic ui react and click on any dropdown item.
Expected Result
Click on any item make only that item styled.
Actual Result
Click on any dropdown item makes every item styled with font-weight 700 and background color like if all  items selected at the same time.
Version
0.88.1
Screenshots

Please create a reproducible test case using the supplied Codesandbox link at the bottom of the bug report template.
Closing this report until a reproducible case is provided.


Hydra: Add support for using substitutes

Passing this  info to container is ok, but a bit curious what info is going to be used (out of topic a bit...)


[RC] static site generator relies on bilk:ford, can't find

When i try to start the static site generator on release-candidate, I get
/~zod/home/git-d08108fb90/sys/vane/ford:<[4.794 18].[4.794 59]>
/~zod/home/0/app/static/hoon::[21 1].[125 3]>
/~zod/home/0/app/static/hoon::[27 3].[30 5]>
/~zod/home/0/app/static/hoon::[28 7].[28 41]>
/~zod/home/0/app/static/hoon::[28 24].[28 40]>
/~zod/home/0/app/static/hoon::[28 30].[28 39]>
-find.bilk

It appears that new ford doesn't have ++ bilk. Not sure what old bilk did. What do?

@ixv @belisarius222 @eglaysher

yeah this isn't up to date, bilk is the old term for pair of beak and silk

Obviously!
(In one of our 2019 cleanup passes, if not sooner, we need a general review
of naming.)
On Mon, Sep 17, 2018 at 1:52 PM ixv<EMAIL_ADDRESS>wrote:

yeah this isn't up to date, bilk is the old term for pair of beak and silk
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/urbit/arvo/issues/818#issuecomment-422166799, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AALyAe7pnJ1sBUYA9WL42CcIjAaO1Kp4ks5ucAt_gaJpZM4Wssoo
.

But silk no longer exists ...
This is the tool that we'll be using for compiling docs, so this is not not urgent. Can one of @ixv @belisarius222 or @eglaysher take a look?

sure I will take care of it

@ixv, once fixed, can you re-enable this in %/app/test/hoon?


Allow checking for any command exit value.

This is pretty much what Assert::fails_with(self, expect_exit_code: i32) provides, except that an exit code of 0 is accepted. The current implementation fails the assertion if expect_exit_code=0 and the command returns with an exit code of 0.
Rationale for the functionality: I'm testing for specific exit codes with something that looks like:
        let t = vec![
            (&["all","fine"], 0),
            (&["user","error"], 1),
            (&["program","error"], 2),
        ];
        for (args, exit) in t {
            Assert::main_binary().with_args(args).fails_with(exit).unwrap();
        }

But this fails unless I special-case exit==0 to remove the fails_with() call.
I'm not sure if fails_with() should be modified to accept 0 as an argument (ie this issue is a bug) or if a new function (say exits_with()) should be created (ie this issue is a feature request). Tell me which and I'll try to send a PR.
assert_cli 0.5.4 rust 1.25.0

Addressed in assert_cmd
https://github.com/assert-rs/assert_cmd
assert_cli is going to morph into a wider-focused tool, including things like assert_fs.  See https://github.com/assert-rs/assert_cli/issues/41


Process diags

Add server configuration that is a function which can be used to manipulate diagnostics. The handler takes one argument which is the list of raw diagnostics and it expected to return a new list of diagnostics.
I'm not really sure about the performance implications, if it's a big deal, then we could consider refactoring the handler to take each diagnostic individually and return a new one, returning null could work as a filter to remove the diagnostic. This would remove one pass though the diagnostics, but introduce many more function calls.

I'm not really sure about the performance implications

Does this feature have performance implications even if unused?


build: install CoreFoundation libraries for static builds

The static builds have dependencies on CoreFoundation which need to be
resolved when linking (not all platforms provide relocatable linking).
Install the CoreFoundation libraries during static builds.

@swift-ci please test

@swift-ci please test macOS

@swift-ci please test Linux platform

@swift-ci please test macOS


repo: Set python version (for dependabot)

Signed-off-by: Ryan Northey<EMAIL_ADDRESS>Commit Message:
This should fix problems - as seen here:
https://github.com/envoyproxy/envoy/pull/19722/files#diff-bb622deaf176384b0aeb5bf4f767ea4ac5d71a1f371100ce2c46813564cb936fL409
...where dependabot is trying to uninstall libs that are required for 3.8 and 3.9, but not for 3.10
(officially we use 3.8, i tend to test locally with 3.9)
Additional Description:
Risk Level:
Testing:
Docs Changes:
Release Notes:
Platform Specific Features:
[Optional Runtime guard:]
[Optional Fixes #Issue]
[Optional Fixes commit #PR or SHA]
[Optional Deprecated:]
[Optional API Considerations:]

cc @kfaseela


A Crate for Handling Properties, Tags, and Scores

Description
Create a new crate to handle properties, tags, and scores, which are currently implemented separately in ark-cli and arklib-android. This crate will provide a unified API for managing these aspects of resources.
Current State
Currently, ark-cli and arklib-android each implement their own API to handle properties, tags, and scores. These functionalities are spread across different parts of the codebase. See:

https://github.com/ARK-Builders/arklib-android/tree/main/lib/src/main/java/dev/arkbuilders/arklib/user
https://github.com/ARK-Builders/ark-cli/blob/main/src/util.rs

Proposal
We should create a separate crate within ark-rust workspace. This crate will centralize the logic for managing properties, tags, and scores so to provide a consistent API.
Next Steps

Implement the functionality to store, load, and manipulate properties, tags, and scores.
Integrate the new crate into ark-cli and arklib-android, replacing the existing implementations with calls to the new crate's API.

Tags and Scores are the simplest storages (single file) and can be started as soon as #10 is completed.
Properties is more advanced and requires folder storage be implemented first.
Repo arklib-android is going to be replaced by ark-android. The transition is same as from arklib to ark-rust. It is in progress right now, we should separate existing modules better on Android side so Rust crates would map directly to Android libraries.


Improve blocked upgrade service log wording

Alerts are relevant to SREs but not to customers. In most cases they probably don't understand how they work. We should translate the alerts into simple text so that the customer can understand the issue, impact and has a clear ask about the action they need to perform to resolve their stuck upgrade.
This PR, removes alert from the message/summary and adds parameters for Customer Action and Documentation.
(The name of the template should probably change, but I couldn't come up with a better one, happy to change or keep it as is)

/lgtm

/lgtm


Support multi-resource runs

Feature
dbt should be able to run different types of resources in the same invocation. dbt should support the specification of resource types, including:

models
seeds
tests (both data tests and schema tests)
archives (eventually)

This might also mean that dbt run should in fact run all resource types by default. That would be a pretty big change, so it might be worth adding a new subcommand and deprecating dbt run, or keeping the run subcommand and preserving the existing behavior.
Exact syntax TBD, but let's use this issue to explore different approaches.
Using this example, we can select all resources using the --select flag, or specify selectors for specific resource types using --models, --tests, --seeds, and --archive. Ex:
$ dbt run all --select source:snowplow+ --models some_model+ --tests some_other_model

Resolved by #3490!


Nothing works

Is there anyone who was able to run this? Last error:
stager@StagerServer:~/tilestrata/example$ node index.js
/home/stager/node_modules/tilestrata/lib/TileRequestHandler.js:39
	if (this.provider) throw new Error('There is a provider already registered to this layer');
	                   ^

Error: There is a provider already registered to this layer

I broke down...

This error means there's a problem with your configuration (attempting to use two providers on a single layer). If you want more specific help, you'll need to post your layer configuration.
strata.layer('basemap')
    .route('tile.png')
        .use(mapnik({ ... }))
        .use(mapnik({ ... })) // will trigger that error

Just noticed you were referencing the example that I forgot existed in the repository. The problem there was there was a small typo:
https://github.com/naturalatlas/tilestrata/commit/0eacf8c393332a6aaa5c9c25da8eeae7786c4c8e#diff-f8c408fd257ff44ce4b01e5f8422b1e1R15


🖼 test/image: Fix test cases directory path in doc and code

The directory with image-tests test cases has been renamed from cases
to manifests. This has not been previously reflected in the test/README.md
and osbuild-image-tests code. osbuild-image-tests hardcodes the test
cases directory path and uses it in case no test case are passed
to it on the command line. Since the image_tests.sh CI test case looks
for image-tests test cases in the correct directory and passes the
relevant ones to osbuild-image-tests, the CI didn't detect this issue.
Running osbuild-images-tests without any argument and let it run all
test cases from the default test cases directory as part of CI probably
does not make sense. Due to this reason, I'm not adding any new test.

Thanks and sorry for the confusion!

No problem. At least there is no doubt that I look at it and tried to use it :smile:

Thanks and sorry for the confusion!

No problem. At least there is no doubt that I look at it and tried to use it :smile:

PSI OpenStack is misbehaving, otherwise we're green.

PSI OpenStack is misbehaving, otherwise we're green.


Create chugye.txt

Join

@subukkid Thank you for your patience while waiting for a review and processing your request. Pull request merged. Please start applying for JetBrains educational subscription at https://www.jetbrains.com/shop/eform/students in an hour, to allow the changes to propagate through JetBrains systems.


Authentication

We should use something like http://www.passportjs.org/ for the authentication and probably something like mongodb for the db.
Perhaps we could use GraphQL as the api backend? would be good experience
This would be a precursor for https://github.com/gareththegeek/corewar/issues/33

This repo has a simple auth + graphQL impl - https://github.com/jferrettiboke/react-auth-app-example

Shelved till phase 2 - hills


Buy the players only if they have a style

It has been closed by the boss but you didn't answer me
Is it possible to buy only players that have a style. Any style, not a particular one.
I wanna buy ALL richarlison that have ANY style, not limit the AB on one style
Please answer

Hi, your question in my opinion is unclear, because the AB already searches for ANY style, and by ANY style, I mean the default mode when you don't choose a style. What I think you want to say, is a "list of given styles" which is different then "any style". If that is what you want, the answer to you question is NO, he can't, because like we already said, we can only filter using the filters EA provided us, normally you can't select multiple styles. And applying custom filters on the players we receive from the server is pretty much useless, then you will come back saying you don't get any buys because you receive from the server only richarlisons with basic and Hawk chemistry, but you wanted to buy him with Hunter and Shadow chemistry.

You got my question very wrong. Let me explain you why

People who don't understand it generally do not play fifa. Let me summarize you something quickly. People will buy players more often if they have a style. Especially now, one month after the release, where people got a lot of coins and they will buy even if it's at a higher price. So I wanna profit from this : I wanna buy every richarlison that have a style and resell him 500 coins higher. I don't need basic one, the one that have no style 'cause no one will buy them now
So I say it again : I want to buy every rycharlison that has ANY style (like hunter,sniper,finisher, name them all)

It is true that we do not have this function directly in the web app. However, we didn't have the option to filter the rating of a player??
If the AB finds a player who doesn't fit in this rating range, well it will skip it

And that's a similar option I ask. Search all players, however, if it's a basic style, skip it, however, if it isn't basic, then buy it. In theory, it should be simple. He doesn't have to add every style, only to add a ''if style isn't <>, then buy''. However, I know programming isn't always as easy as said. I would appreciate if you or the original author respond to this

Hope it's more clear now,
Thanks for your time

@ramb6six , not going to implement as no one else requested for this.


Translation websites not picking up/understanding the json

Hey, I was wondering if you managed to use a site like transifex to do the translations. It does not seem to pick up strings for the json I generate... Hope you have some thoughts. Cheers!

It's key value :+1: . transifex likes it like that. Sorry to spam you ;)


chore: update variables5.rs book link

chore: Update variables5.rs book link

As a first time user, I was not able to figure out which book the exercise was talking about.


Feature to add skip verify TLS

Adds K6_ELASTICSEARCH_INSECURE_SKIP_VERIFY to the configuration, which allows tunning Elasticsearch clusters with self-signed certificates. Defaults to false.
Modifies the README to reflect the new configuration, and adds a docker-compose file for a quick local test.
Upgrades to k6 latest release v0.43.1.

@danielmitterdorfer, if you have the time, can you have a look at this PR? It would be great to have a docker-compose that allows testing the extension locally. Thanks!

Hi @danielmitterdorfer, many thanks for the review. I'm happy to contribute to the ES output. Let me know if you'd change anything else or if we can merge 😄

Thanks @immavalls! The changes look fine to me and I've just merged your PR.


Add option for watch/ skip as containing spoilers

Fixes https://github.com/lemonmade/watch/issues/116


Is there a way to turn off the cookies for basic auth?

The software is awesome but the inability to change users or passwords limits the use-fullness.

Use Firefox. Closing the browser clears the sessions and it asks for user relogin (HTTP Basic Auth).

@vidcentum it would be nice to clear session without restarting the browser

see solution in #85


clean up readme before archiving

Remove extra details from the readme so that the archival message is
more clear.

/lgtm

/lgtm
/hold to give some time if needed


Support custom python debug adapter path

For #8720

[x] Pull request represents a single change (i.e. not fixing disparate/unrelated things in a single PR)
[x] Title summarizes what is changing
[x] Has a news entry file (remember to thank yourself!)
[x] Appropriate comments and documentation strings in the code

I had made this change earlier to create a VSIX build for the debugger team. This PR should still be blocked by #6816

Codecov Report

Merging #9377 into master will decrease coverage by 0.31%.
The diff coverage is 0%.

@@            Coverage Diff             @@
##           master    #9377      +/-   ##
==========================================
- Coverage   60.73%   60.42%   -0.32%
==========================================
  Files         550      550
  Lines       28749    28751       +2
  Branches     4349     4345       -4
==========================================
- Hits        17460    17372      -88
- Misses      10338    10426      +88
- Partials      951      953       +2

Impacted Files
Coverage Δ

src/client/debugger/types.ts
100% <ø> (ø)
:arrow_up:

src/client/debugger/extension/adapter/factory.ts
86.11% <0%> (-2.47%)
:arrow_down:

src/client/testing/serviceRegistry.ts
49.09% <0%> (-47.28%)
:arrow_down:

src/client/testing/codeLenses/main.ts
40% <0%> (-30%)
:arrow_down:

src/client/datascience/jupyter/jupyterUtils.ts
61.11% <0%> (-27.78%)
:arrow_down:

src/client/formatters/serviceRegistry.ts
75% <0%> (-25%)
:arrow_down:

...t/datascience/jupyter/jupyterDataRateLimitError.ts
50% <0%> (-25%)
:arrow_down:

...ience/jupyter/jupyterDebuggerRemoteNotSupported.ts
60% <0%> (-20%)
:arrow_down:

...cience/jupyter/jupyterDebuggerNotInstalledError.ts
50% <0%> (-16.67%)
:arrow_down:

src/client/common/utils/icons.ts
83.33% <0%> (-16.67%)
:arrow_down:

... and 10 more

Continue to review full report at Codecov.

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


aAfter and before as named arguments

Just a idea: couldn't before and after be named arguments? I mean adverbs...
I think that is accessor-facade(&bla, &ble, :&after) or something like that would be great!
Great module, thanks!

Maybe even accessor-facade(:setter(&bla), :getter(&ble), :&after)

Hi,
yeah I actually decided that this would be a better interface some while ago as subsequently I found there are interfaces that require various permutaions of  the getter, setter, before and after that doesn't play nicely with positional arguments, I only didn't do it at the time as I didn't want to go back and change all the existing things that use it :)
So, yes, it's a good idea and I will do it at some point.
Thanks,

Hi,
I'm going to re-open this as I'll forget otherwise.

Hi,
v0.0.5 contains a named option, I have gone in the first place for all named parameters  or all positionals but I may add mixed later if this works out.
Thanks again for the suggestion.

😊


test: correct flaky TestManager_SkipLoadingV3IntegrationsWithNoWarnings test

…gs test

Pull Request Test Coverage Report for Build 239494405

0 of 0   changed or added relevant lines in 0 files are covered.
3 unchanged lines in 2 files lost coverage.
Overall coverage increased (+0.04%) to 55.874%

Files with Coverage Reduction
New Missed Lines
%

internal/agent/event_sender.go
1
72.56%

pkg/metrics/sampler/sampler_routine.go
2
94.74%

Totals

Change from base Build 228907731:
0.04%

Covered Lines:
8956

Relevant Lines:
16029

💛  - Coveralls


Caching with Relationships

I want to completely disable caching - I know this will be a performance hit but our table sizes are small and we'll re-enable the caching once we configure the TCP Notification Server and have tested it in our environment/ with redundancy etc.
So, to disable the caching I set cacheType = none as follows and sure enough I see all application finds going to the db (sql logs and profiler show that)
 <MithraObjectConfiguration
  className="com.greenhedges.Account" cacheType="none"/>

 <MithraObjectConfiguration
  className="com.greenhedges.AccountAgreement" cacheType="none"/>

My issue is the one-to-many relationship are not  refetched/queried from the db each time - e.g. say an Account can have n number of AccountAgreements:
<MithraObject objectType="transactional">
<PackageName>com.greenhedges</PackageName>
    <ClassName>Account</ClassName>
    <DefaultTable>account</DefaultTable>
...
    <Relationship name="aggreements" relatedObject="AccountAgreement" cardinality="one-to-many" relatedIsDependent="true">
         AccountAgreement.id = this.id
    </Relationship>
</MithraObject>

Is there a way to force the relatedObjects to be reloaded always?

There is no such thing as "completely disable caching". A cache is defined as a temporary copy. As soon as the bytes leave the database server, you're facing a temporary copy, aka a cache. So the only valid outlook is "what is the contract of the cache?" (because, again, for emphasis, there is always a temporary copy, aka cache).
The various cache configurations change the contract. It sounds like you want a rather odd contract: if you have an object with an attribute "foo" and another attribute "bar", with corresponding getFoo()/getBar() methods, for some foos ("relationships"), you want to hit the database but for some bars (not "relationships") you don't. This inconsistency will cause your code to do really bad things.  Calling a get method is not a find. CachType="none" is also meant as a last resort and temporary measure. Every time you use it, you incur technical debt.
Setting up the notification server is by far the easiest way to deal with this, because of the consistency it brings to your code in treating method calls on your objects.
If you really want to shoot yourself in your foot, have a look at the cacheTimeToLive and relationshipCacheTimeToLive configuration options.
P.S. The performance hit has little to do with the table size. It's the wire latency that's the real issue.

Perhaps i didn't explain my issue clearly or I'm not understanding the timing of query issuing in reladomo.
I want the full graph of Account loaded each time Account is found from the db - just like the first time i do a find, it loads both the the Accounts and Account Agreements by issuing two queries.
Whenever an Account instance is retrieved from the db the list of AccountAggreements should also be fetched again.    This is what happens the first time I find the instance from the database.   On subsequent finds the second query to the related AccountAgreement table is not issued.
for your snippit of code i would not expect any queries to the db but for
  Account account = AccountFinder.findByPrimaryKey(id, validDateTimestamp,
            transactionDateTimestamp);

I'd expect a find from the Account Table and the AccountAgreement table to be run in order to retrieve the Account Object.
Am I thinking about this the wrong way?      If I do the find in a transaction the Account Agreements would have to be fetched wouldn't they?
I will setup the TCP Notification Service but i would like to understand the whats wrong with the way i'm thinking about the find and the relationship

Eager graph loading in Reladomo is done via deep fetch on the list object, which means you can't really use the findByPrimaryKey convenience method if you want to load a graph eagerly. It's not hard to use a list even if you want to load one object with some of its graph:
AccountList accountList = AccountFinder.findMany(someOp);
accountList.deepFetch(AccountFinder.agreements());
Account a = accountList.get(0);

When you use the list to eagerly fetch part of the graph, you're effectively forcing a refresh in scenarios where the data may have changed in the db.
Deep fetching is always local to the piece of code: there is no way to globally change the behavior of calling a find (e.g. always forcing a certain relationship to be loaded eagerly). We believe forced graph loading in all contexts is an anti-pattern.
Most of the time, when you want to use a single object, there is no difference between eager and lazy loading with some exceptions:

A traversal through a to-many relationship to another deeper object.
Db changing without notification.

Just to complete the picture, when you have an account object (regardless of whether you got it from a list with a deep fetch or findByPrimaryKey), a relationship traversal (calling account.getAgreements()) will hit the cache if it can, and then the database if it must. In other words, relationships will lazily fetch data from the database if they must, but they are not affected by the cacheType="none" configuration (because then the piece of code I pasted would do very bad things). Another way to think about this is that the eager fetching via deep fetch is just priming the cache, not sewing a graph.
As you've said, transactions don't trust the cache at all, until they load something in that transaction from the database (which is really part of the locking scheme, with data refresh as a side effect). As far as eager/lazy fetching, the transactional context doesn't make a difference (other than knowing when the cache is trustworthy).
A lot of this behavior is also predicated by the "uniquing" feature of Reladomo: each persistent object, identified by its PK, is guaranteed to have reference stability across the JVM.

Thanks for the detailed answer.  You've convinced me that my best option is to setup the notification server.   It looks pretty straightforward - I've reviewed the docs here.
Can different applications backed by totally different databases share the same notification service if they set their DatabaseIdentifier differently or must I run a notification server for each separate application?

Sharing the notification server is fine.

Can notification server run as part of JVM that is using database. Can each instance run with its own notification server?

Notification doesn't help when one or more of your processes are not Reladomo JVM based?
If i have a sprinboot application that is using reladomo do i still need the Notification server?

All instances that read/write to the same database must share the same server. The server can run in the same JVM as one of the instances, so long as that instances is up all the time.

Have you seen reladomo being used in mircroservice architecture? If so, what are the most common deployment topologies that you have seen in regards to notification server and reladomo in general?
Thanks.

Whether using a microservices architecture or not, you should think of your domain as a singular, coherent API. That means defining your domain in one place. Avoid defining the same thing in multiple places and avoid splitting your domain (especially where you'd be breaking natural relationships).
Your services can then use that consistent domain to serve their logic.
From a topology perspective, think of the domain as a single jar that you deploy to all your instances/services and have a single notification server for that cluster of services that use the same objects.


Problem with initializing SQL file

After node schema and mysql -y root -p < init_db.sql, SQL gives error as follows
 ERROR 1064 (42000) at line 1: You have an error in your SQL syntax; check the manual that corresponds to your MySQL server version for the right syntax to use near 'loading config_development... configuration loaded: {   "domain": "<IP_ADDRESS>",  ' at line 1
and terminates. Please help. Thanks.

Update - issue fixed by using mysql -y root -p <schema.sql


Kill confirmation

In my opinion it is important to know exactly when you've killed your opponent. This way you don't have to waste ammo, and you can reorientate quickly on what you want to achieve next.
After having played with Community Edition and the frag message that is implemented there, I came back to the vanilla client, and had difficulties with determining whether or not I had killed my opponent. I think newcomers have this problem too (if they play anything else than insta), for them the game is quite confusing already.
Because of these reasons I'd like to suggest two ways of confirming the kill:
- A visual confirmation. IMHO a frag message is not necessary; if you want to know who you killed,     look at the left upper corner. But for example a red circle around the crosshair that blinks a few times would be great. Not too big and annoying though.
 Basically just play this sound instead of the hit sound when you give your enemy the fatal blow. Again: shouldn't be annoying, just as subtle as the hit sound is.
And of course vars to toggle both things on/off, but have them toggled on by default (right now the hitsound is off by default I think).

+1

@Calinou:

Perhaps we could have hitsounds depending on damage (higher pitch = more damage)?

Is this still open? I would like to tackle this issue as a first-time contributor.

@syn228 yep! It is ready to be taken. Make sure to look into the following repository: https://github.com/inexorgame/ui-client-hud
Also this needs to be sent via the inexor tree through the core and flex. That is all I know.
We are constantly improving how things are organized.
However, putting in the first effort makes it easier to port this to any other migration we have.

I can't recommend to take this over right now as a good first issue.
There is no sounds output and the UI technology stack is broken and/or incomplete in every single point.


MWE for path error when using libpython310 on macOS

This is a MWE for an issue building a package that uses an embedded Python interpreter by linking to libpython3.10.dylib and starting the interpreter from the Python C API. The problem is that in the embedded interpreter, sys.path refers to paths in the base conda environment, rather than the active conda environment corresponding to the libpython3.10.dylib that is linked. This then makes it impossible to import packages that are installed (only) in the active conda environment.
Examining sys.path
Relevant files:

meta.yaml
mwe.cpp
test_mwe.sh

This example builds the following C++ program:
#include "Python.h"

int main(int argc, char** argv)
{
    Py_Initialize();
    PyRun_SimpleString(
        "import sys\n"
        "print('Hello, world')\n"
        "print('embedded sys.path:', sys.path)\n"
        "sys.stdout.flush()\n");
    return 0;
}

with the compiler command:
clang++ -I${CONDA_PREFIX}/include/python3.10 -L${CONDA_PREFIX}/lib mwe.cpp -o mwe -lpython3.10 -Wl,-rpath,${CONDA_PREFIX}/lib

This shows that the embedded interpreter's sys.path is (incorrectly) defined as:
['/Users/runner/miniforge3/lib/python310.zip',
 '/Users/runner/miniforge3/lib/python3.10',
 '/Users/runner/miniforge3/lib/python3.10/lib-dynload',
 '/Users/runner/miniforge3/lib/python3.10/site-packages']

While the python binary in this environment reports sys.path as:
['',
 '$PREFIX/lib/python310.zip',
 '$PREFIX/lib/python3.10',
 '$PREFIX/lib/python3.10/lib-dynload',
 '$PREFIX/lib/python3.10/site-packages']

(where the instances of $PREFIX are output substitutions introduced by Azure Pipelines).
Specific conditions required to trigger behavior
This error occurs when:

Using a conda-forge feedstock
With macOS x64 (Intel)
With Python 3.10

Other versions of Python (3.8, 3.9, and 3.11) all work fine, as do all builds for Linux and Windows. I have also successfully built a minimal embedded Python test using the conda-forge build of Python 3.10 for macOS x64 outside of the conda-forge feedstock infrastructure (see repository and CI results here).

@speth was this resolved upstream?

Not to my knowledge. The issue I created (https://github.com/conda-forge/conda-forge.github.io/issues/1996) is still open and got no response. I'm not sure how to update/re-run this to test it again.

I asked around if there was someone who could help out with that issue. Let's see if it gets a response

I'm not sure how to update/re-run this to test it again.

I wonder if the same approach as the autogenerated PRs ([example]) would work:


fix: Apply labels and annotations at jobserver-deployment.yaml (#87619)

labels and annotations from ICMJob will be copied to the jobs being created by the operator based on the ICMJob spec

PR Type

[x] Bugfix
[ ] Feature
[ ] Code style update (formatting, local variables)
[ ] Refactoring (no functional changes, no API changes)
[ ] Build-related changes
[ ] CI-related changes
[ ] Documentation content changes
[ ] Application / infrastructure changes

Release
Be sure to include PR label major, minor or patch when merging into main. This determines which part of the semantic version number needs to be bumped automatically.
What Is the Current Behavior?
Labels and annotations of job-specs are not present.
Issue Number: Closes #
What Is the New Behavior?
Labels and annotations are present and can be used for monitoring and to select pods.
Does this PR Introduce a Breaking Change?

[ ] Yes
[x] No

Other Information

The annotations and labels are copied to the "Job" spec and not to the Job.spec.template section for definition of "pods". This creates ICM-JobServer pods without the annotations and spects.