Explore image textures for data storage

We need a lot of data for each hex tile, e.g. height values, terrain type, signal strength, etc.
Image textures might be suitable here, as they also work nicely for GPU tasks.
We should explore how to best store the data in textures and how to access and change this data in the code.
Ideally this would be possible with both offset and axial hex coordinates so that we can choose the storage type depending on the data and algorithms that we need for each task.

This is a really neat idea. I think it would be super useful.

Ideally this would be possible with both offset and axial hex coordinates so that we can choose the storage type depending on the data and algorithms that we need for each task.

Can you explain a bit more about what you mean by this?


Correct name of FAQ entry "dcc_no_rules" in "cert_eu_travel"

This commit corrects the name of the referenced FAQ entry in "cert_eu_travel".

@Ein-Tim Thanks.


Use correct captured group range when parsing cross-reference

Fixes #22666 (Replace #22668)

Could we add a test for that space situation?

BUT, there was already one:


gentl.NotAvailableException: 'BUFFER_INFO_WIDTH'

Describe the bug
This issue is related to #175
When I test it it seems to work, but now is not working anymore. (Not on any version before and after the issue was resolved). Maybe I test it on another camera since Ive been working recently on different cameras but Im pretty sure it was this camera.
There doesn't seem to be information on the buffer available, the exception that is being thrown is NotAvailableException but on the commit that fixes this problem initially the exception caught is NotImplementedException.
  File "C:\Users\OmarGarridoMartín\Desktop\OMAR\PROJECTS\DB_HSTM_POC\workspace\db_high_speed_track_monitoring_poc_VENV\lib\site-packages\harvesters\core.py", line 834, in __init__
    self._data = self._to_np_array(proxy)
  File "C:\Users\OmarGarridoMartín\Desktop\OMAR\PROJECTS\DB_HSTM_POC\workspace\db_high_speed_track_monitoring_poc_VENV\lib\site-packages\harvesters\core.py", line 842, in _to_np_array
    w = self._buffer.width
  File "C:\Users\OmarGarridoMartín\Desktop\OMAR\PROJECTS\DB_HSTM_POC\workspace\db_high_speed_track_monitoring_poc_VENV\lib\site-packages\genicam\gentl.py", line 4368, in _get_width
    return _gentl.Buffer__get_width(self)
_gentl.NotAvailableException: GenTL exception: Resource or information is not available. (Message from the source: 'BUFFER_INFO_WIDTH' is not available for this buffer(payload type: PAYLOAD_TYPE_CHUNK_DATA)) (ID: -1014)

To Reproduce
Steps to reproduce the behavior:
Connect to a camera and try to fetch a buffer
from harvesters.core import Harvester

h = Harvester()

h.add_file(r'C:\Program Files\MATRIX VISION\mvIMPACT Acquire\bin\x64\mvGenTLProducer.cti')
h.update()

print(h.files)

ia = h.create_image_acquirer(0)

# Example: Change values from the GenICam Camera, This values are in the node_map
# ia.remote_device.node_map.Height.value = 16
# ia.remote_device.node_map.Width.value = 256

ia.start_acquisition()

with ia.fetch_buffer() as buffer:
    # Let's create an alias of the 2D image component:
    component = buffer.payload.components[0]

    # Note that the number of components can be vary. If your
    # target remote device transmits a multi-part information, then
    # you'd get two or more components in the payload. However, now
    # we're working with a remote device that transmits only a 2D image.
    # So we manipulate only index 0 of the list object, components.

    # Let's see the acquired data in 1D:
    _1d = component.data
    print('1D: {0}'.format(_1d))

# Release and exit
ia.stop_acquisition()

Expected behavior
I guess the buffer should be either filled on not used in this case.
Screenshots
Desktop (please complete the following information):

Windows 10
Python: 3.7
Harvester: 1.2.3, 1.2.4 and 1.2.6 tested
GenTL Producer: matrix vision recommended from harvesters
Camera: TIS DKF 33GP1300

Additional context
The entire buffer seem to not have the information available

@omar-ogm Hi. Thank you for the report. Could you try out the latest master branch when you can, please? /Kazunari

Hi with this I get a buffer with information with it, so we could say that this specific issue is closed. I'm getting the image as BayerRG8 format, one question out of topic, Can I do demosaicing with harvesters?
Thank you for the quick response

@omar-ogm Thank you for the reply. Concerning your additional question, I think OpenCV can help you just taking an image data array from Harvester.


Fix #2873: DataQuality test specifications and APIs

Describe your changes :
see #2873

Type of change :

[x] New feature

Frontend Preview (Screenshots) :
For frontend related change, please link screenshots of your changes preview! Optional for backend related changes.

Checklist:

[x] I have read the CONTRIBUTING document.
[ ] I have performed a self-review of my own.
[ ] I have tagged my reviewers below.
[ ] I have commented on my code, particularly in hard-to-understand areas.
[ ] My changes generate no new warnings.
[ ] I have added tests that prove my fix is effective or that my feature works.
[x] All new and existing tests passed.

Reviewers

merging in as the failed test is not related to code here


Stream not working with two segments subject: .

Observed behavior
Publish events on a stream with subjects["*.*"]triggers a loop where the messages are consumed infinitely.
I test with 1 event, and one publish is executed correctly but the number of consume messages is infinite.
The "NumPending" of message is increasing on every consume interation.
Curiously, this only occurs with the subjects ["*.*"], if you use any other two segments values, it works as expected, like ["tests.*"].
Here it is my NATs configuration on the test:
"Nats" : {    "ConnectionString" : "nats://localhost",    "StreamName" : "test",    "Stream": {         "Name": "test",         "Subjects": ["*.*"]    },    "Durables": {         "durable1": {             "Subjects": "tests.*"         }    } }
Expected behavior
Streams with subjects value ["."] work in the same way as streams with other subjects values.
Server and client version
nats-server
nats:2.10.14-alpine
natsio/nats-server-config-reloader:0.14.2
natsio/prometheus-nats-exporter:0.14.0
nats
v0.1.4
Host environment
No response
Steps to reproduce
No response

Yes it would pick _INBOX.xxxx and that is very bad - aside from whatever happens on the JetStream side, there would be a side effect for all requests that use the inbox - the stream would capture the publish, and then reply its puback to that request, meaning that depending on the timing the response you get may be coming from JetStream ingesting the message.
The fix here is for the stream to be more explicit about the subjects that it wants to capture.

Thanks for you quick reply @mtmk @aricart
Not sure if it's .NET specific.
In my system, we want for the stream to take every message with that segment format, and we did some tests so we guess that there is a problem with the two segment format specifically. This is the result:

* : OK - captures everything
tests.* : OK - captures every message started with "tests" and two segments
*.* : KO - receiving an infinite loop where the message is consumed infinite times
*.*.* : OK - captures every message with 3 segments
.... OK with more segments

Are you using next() call to consume?
if you run the server in verbose with -V mode for example nats-server -js -V you should see the _INBOX.* messages which are sent back from the server confirming pull requests which are used to consume the messages in a stream.

I'm using the Fetch function.
Here you have how I'm consuming the messages:
protected override async Task ExecuteAsync(CancellationToken stoppingToken)
{
    stoppingToken.ThrowIfCancellationRequested();
    while (!stoppingToken.IsCancellationRequested)
    {
        try
        {
            var js = new NatsJSContext((NatsConnection)_connection);
            var consumer = await js.GetConsumerAsync(_streamName, _durableName, stoppingToken);
            SetState(true);
            Logger.Active(Name);
            while (!stoppingToken.IsCancellationRequested)
            {
                try
                {
                    var counter = 0;
                    await consumer.RefreshAsync(stoppingToken);

                    var opts = new NatsJSFetchOpts
                    {
                        MaxMsgs = _subscriptorOptions.BatchSize,
                        MaxBytes = _subscriptorOptions.MaxBytes,
                        Expires = TimeSpan.FromMilliseconds(_subscriptorOptions.MaxWaitMilis)
                    };

                    await foreach (var message in consumer?.FetchNoWaitAsync<NatsMemoryOwner<byte>>(opts: opts, cancellationToken: stoppingToken))
                    {
                        if (stoppingToken.IsCancellationRequested)
                        {
                            break;
                        }

                        await ConsumeMessage(message, stoppingToken);
                        counter++;
                    }

                    if (counter == 0)
                    {
                        await Task.Delay(1000, stoppingToken);
                    }
                }
                catch (NatsJSProtocolException ex)
                {
                    Logger.NatsException(ex.ToString());
                }
                catch (NatsJSException ex)
                {
                    Logger.NatsException(ex.ToString());
                    await Task.Delay(1000, stoppingToken);
                }
            }
        }
        catch (Exception ex)
        {
            Logger.Exception(ex.ToString());
            SetState(false, ex);
            Logger.Inactive(Name);
            await Task.Delay(1000, stoppingToken);
            continue;
        }
        SetState(false);
        Logger.Inactive(Name);
    }
}

private async Task ConsumeMessage(NatsJSMsg<NatsMemoryOwner<byte>> message, CancellationToken cancellationToken)
{
    cancellationToken.ThrowIfCancellationRequested();
    IEvent @event = null;
    try
    {
        @event = await _mapper.MapAsync(_durableName, message, cancellationToken);
        await HandleEventAsync(@event, cancellationToken);
        await message.AckAsync(cancellationToken: cancellationToken);
    }
    catch (Exception ex)
    {
        var errorContext = new ErrorContext(ex, message, @event);
        await HandleErrorAsync(errorContext, cancellationToken);
        if (errorContext.SendAck)
        {
            await message.AckAsync(cancellationToken: cancellationToken);
        }
        else
        {
            await message.NakAsync(delay: errorContext.NACKDelay, cancellationToken: cancellationToken);
        }
    }
}

Thanks @mtmk
I finally did what you explained, adding a more specific subject x.*.* in the stream.
You may close the issue if this is happening in other clients.
For me, it's still a bug in the specific wilcard *.*. Even it's not recommended to use the generic ones, other generic wildcards with different formats (segments), as I explained above, work perfectly. For example: *, *.*.* and *.*.*.*

I tried to add the segment in the prefix and it works now 👍
We will add the InboxPrefix option in our library config with this behaviour explained in case of anyone need to use this two segments format.
Thanks again for your help :)


hi, the drop and drag feature does not work on IE

hi, the drop and drag feature does not work on IE
IE defaults to open a file inside browser

Hi,
do you resolve the problem?
Best regards

Not yet mate.
Still want it to work but, its not working still.

On Fri, Jun 1, 2018 at 3:34 AM, Majusek<EMAIL_ADDRESS>wrote:

Hi,
do you resolve the problem?
Best regards
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/JeremyFagis/dropify/issues/87#issuecomment-393841924,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AMcTDppNHhKjRf6KB4NSVwD5WvRGOVz5ks5t4RirgaJpZM4T5VhD
.

No, but I seen another problem, button "remove" doesn't work too.
Do you have the same problem with this button?

No, for me the remover button works just fine. Check the docs
On Sun, Jun 3, 2018 at 3:15 PM, Majusek<EMAIL_ADDRESS>wrote:

No, but I seen another problem, button "remove" doesn't work too.
Do you have the same problem with this button?
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/JeremyFagis/dropify/issues/87#issuecomment-394195895,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AMcTDkrXikHCNfdLIz2hxVzvaR1WSJ8nks5t5F_3gaJpZM4T5VhD
.

This seems to be because Dropify is "just" an overlay over a regular input element, and IE doesn't support drag-and-dropping on file inputs to set their value. With the current architecture, it will never work on IE.
The fact that the library doesn't work on IE could be a design choice (it's up to the author), but I think it should be documented in the README.


[RULE] Enable automatic instance repairs in Azure Virtual Machine Scale Sets

Existing rule
No response
Suggested rule
As per APRL recommendation, enabling automatic instance repairs in Azure Virtual Machine Scale Sets enhances application availability through a continuous health check and maintenance process.
Resource Type: microsoft.compute/virtualmachinescalesets
Property with expected value: properties.automaticRepairsPolicy.enabled==true
Pillar
Reliability
Additional context
This rule will be used to test against the AVM Bicep VMSS module.
@rainerhalan

Thanks @jeanchg. I think there is some good linkage to: https://learn.microsoft.com/en-us/azure/well-architected/reliability/self-preservation

:shipit:


Method for ordering

Add method in the repository for listing recommendations by date newest -> oldest.

done.


CalcObj/Fun: type of 'origin' is distinct from type of 'value'

+1 in favour of adding the defaults.
I think we need the origin type for functions to extend the context because we can capture contexts. As an example.
Define the expression:
(x) => x + position
of type <T>(context: CalcObj<T>, origin: T) => CalcFun<T>
Suppose I use the context C of type CalcObj<[number]>, therefore my expression should only be called in origins of type [number]. In other words, my context C only knows how to handle origins on the 1D line. The expression desugars to:
(C: CalcObj<[number]>, origin: [number]) => {
  return (trace, origin, args) => args[0] + C.read('position', origin)
}

Note that it has captured the context C, but not origin.
If we give the function the type:
<O = unknown>(trace: Trace, origin: O, args: CalcValue<T>[]) => …
Then we can pass any origin into the function, such as undefined. However, now we pass an origin that is undefined to C that only accepts things of type [number]. I think we need the constraint so that you must provide the same (or more information) to the resulting expressions.

There's definitely something I'm confused about:
If I have a CalcObj<{a : number}>, and I call read('a', origin), I expected .read() to do the following:

Add 'origin' to a set of CalcObj's consumers to notify it if the CalcObj changes.
Return the value for property 'a'.

In the original typing, origin is constrained to be an '{a : number}'.  If that's correct, it makes me wonder if I had this backwards, and your intent was for it to work like this:

CalcObj is created on behalf of a specific consumer.
When I call read('a', origin):

CalcObj records that the consumer depends on 'origin'.
CalcObj extracts the value of 'a' from the origin and returns it.

(In that case, it would make sense that CalcObj requires an origin of .)

Ah ok, so I think this is probably just a confusion from my not very clear code.
The type parameterising CalcObj is the type of the consumer, rather than the type produced or consumed. So in the types IProducer<T> / IConsumer<T> and CalcObj<T>, the two parameters mean different things.
A closer way of describing T in CalcObj<T> would be something like:
CalcObj<T extends IConsumer<Record<string, Value>>

but rather than requiring the type to be an IConsumer I left it unconstrained so that it could be anything, like a tuple.
I never made the type of the produced values explicit like T in IProducer<T> because there wasn't a strong need (tiny-calc expressions don't have intellisense and type-checking like TypeScript code). So the T here was always implicitly something like Record<string, CalcValue<O>> for some origin type.
So to match your expectation I think there would need to be two type parameters in CalcObj. So that:
CalcObj<{a : number}, T> means roughly:
The object of type CalcObj<{a : number}, T> has a property a that can be read, returning a number, and that the object can only be consumed by locations of type T. So a concrete use case would be:
CalcObj<{a : number}, [number, number]> means roughly:
The type of objects that can be read at property a by consumers living in the 2d coordinate space.

CalcObj is created on behalf of a specific consumer.

Yeah I think that is closer to what I mean. Another phrasing is that CalcObj<T> is a producer of type Record<string, CalcValue> that accepts consumers of type T. So you implement the calc object for specific consumer types.

Perhaps what is tripping me up is that CalcObj delivers change notifications in the "key space" of the consumer, while I expected IProducer/IConsumer to deliver change notifications in the "key space" of the producer.
For example, if someone does this:
calcObj.read('a', [x, y]);
And later modifies 'a', the consumer receives a notification that '[x,y]' has been invalidated.
Whereas if someone does this:
producer.open(consumer).read('a');
...and later modifies 'a', the consumer receives a notification that [producer, 'a'] has changed.
In the IProducer/IConsumer model, the consumer is responsible for noting that it read [producer, 'a'] while evaluating [x, y] and recording the dependency so that it can later invalidate the correct cell.
Is that where I went off track?

Perhaps what is tripping me up is that CalcObj delivers change notifications in the "key space" of the consumer, while I expected IProducer/IConsumer to deliver change notifications in the "key space" of the producer.

That sounds like a good summary. Ultimately the types, as they are now, only aim to provide enough information to form a graph from calc objects (which may encapsulate a producer) to consumers (which may be of arbitrary type) along edges labelled with properties.
For instance, a calc object could wrap a producer and enforce the origin to be a consumer, then the read method of the form calcObj.read('a', origin) just calls producer.open(origin).read('a') under the hood. Though equally, calcObj.read('a', origin) could just add an edge into a global dependency graph between the objects id and the origin (completely ignoring the property).
Generally I think that the types allow for the all the same forms of interactions because the methods pass the same information in some form or another. I think the design questions boil down to what is ergonomic for the common cases.

think you could have notifications in the produce...
Heh.  I can verify it works.    I'm pretty sure I'm on the right track now.   Thx.


Bookmark current chapter while in the reader

Describe your suggested feature
Add a button to just bookmark/unbookmark the current chapter as an option in the Reader Bottom Buttons setting

Other details
No response
Acknowledgements

[X] I have searched the existing issues and this is a new ticket, NOT a duplicate or related to another open or closed issue.
[X] I have written a short but informative title.
[X] If this is an issue with an official extension, I should be opening an issue in the extensions repository.
[X] I have updated the app to version 1.11.2.
[X] I will fill out all of the requested information in this form.

This is also a feature from Mihon

you can bookmark chapter in the reader's chapter list (button on bottom left)


Organize/add docstrings in API docs

Replaced single @autodocs block with explicit references to all functions/types/constants with docstrings and organized them under headers (also tried to sort them between the Public API / internals pages as best I could). Added a bunch more things to @docs blocks in internals page which seem like they should be documented but do not currently have docstrings.

Thanks!


docs: Add Jen Duong as a contributor

Summary
Adds @jenbutongit to all-contributors. I forgot to recommend that they add it in https://github.com/trussworks/react-uswds/pull/1687/
How To Test

check that @jenbutongit is added to the all-contributors readme
[ ] verify with @jenbutongit that they're comfortable with being added here (no pressure if you're not!)

Screenshots (optional)

@jenbutongit hey! I just remembered that we have this all-contributors that I didn't encourage you to add yourself to. Are you comfortable being added? We won't merge this PR with your explicit approval 😄

Ah! Thanks @brandonlenz, sounds good!
P.s. my hacktoberfest tee is arriving soon, so double thanks for sorting the contribution stuff.


Find new orgs using civictechindex topic tag on github

Overview
In order to keep our contributors uptodate we need to keep checking to see who is using the tag
Action Items

[x] define what information we need to collect (more to be added after db looked at)

[x] GH org id
[x] Name of Org
[x] logo url
[x] parent
[x] tag

[ ] Write a script to

[ ] Run the Github Search
[ ] Compare results of Github orgs to contributors database
[ ] Generate report for orgs that are NOT in the database and ones that are not using their org name as a topic tag

Resources/Instructions
API endpoint docs https://test-civictechindexadmin.herokuapp.com/swagger/
Github Search https://github.com/search?q=civictechindex&ref=opensearch

I think it could be interesting to show the number of repository for each organization (not sure if this issue is the right place to mention this).

@cnk Any blockers to move ahead.


Abstract aborter into its own library

This draft PR builds on #2047 and abstracts Aborter into its own package and updates storage-blob to use this package.

Now that we've landed the rush change, this is good for a review. If the approach seems good, we can finish this up by:

[ ] use shared aborter across other client libraries
[ ] implement better tests, current tests are tied to the Storage library.

/cc @AlexGhiondea @jeremymeng

browser tests are failing for me. Maybe due to rollup issue.

Just noticed that this PR is still open, should it be closed now that @chradek finished this work?

Yuppp, thanks!


Hdf5 update

Todos:

[x] check for new symbols to name mangle on macOS.
[x] check for new symbols to name mangle on Windows.
[x] check for clobbering of Emscripten fixes
[x] remove _ prefixed mangled symbols

Verified locally on Emscripten and ready to go :-).


Release 3.7.0

Also, in looking at the release notes, everything is in 'future tense'.  Should it be in past tense now that we have actually done it?

We use present tense for release notes, but yes, good catch.


Checking Torque3D with PVS-Studio static analyzer

Hello,
I have found and fixed some bugs using PVS-Studio tool. PVS-Studio is a static code analyzer for C, C++ and C#: https://www.viva64.com/en/pvs-studio/
Analyzer warnings:

V591 Non-void function should return a value. Engine/source/core/util/tSignal.h 165
Comment: If there is needed to prevent the copying of the object, then there is no need to provide the definition of the copy constructor and the assignment operator, because the member functions and friend functions can still call them.
Note: If the code is compiled by a compiler with support for the C++11 standard, you can delete them using the keyword delete like this:
....
public:
  SignalSlot( const SignalSlot&) = delete;
  SignalSlot& operator=( const SignalSlot&) = delete;
....

V547 Expression 'bytesRead < 0' is always false. Unsigned type value is never < 0. Engine/source/sfx/media/sfxVorbisStream.cpp 217
Comment: The variable bytesRead of type unsigned int stores the result of the ov_read function, which returns a signed value (long). Comparison bytesRead < 0 does not make sense.

V547 Expression 'path == ""' is always false. To compare strings you should use strcmp() function. Engine/source/console/consoleFunctions.cpp 2151
Comment: The comparison path == "" will not be as expected. To compare a C-string to an empty string, either use the strcmp function, or compare the first byte of the string with the null-terminal character.

V610 Undefined behavior. Check the shift operator '<<'. The left operand '~mask' is negative. Engine/source/core/strings/unicode.cpp 472
Comment:  	In an arithmetic expression, all the variables whose values can be represented with type int will be promoted to this type. Therefore,  the variable mask will be promoted to the int type and the result of the ~mask expression will be a negative number. By the C++ standard, shifting a negative number to the left leads to an undefined behavior.

V570 The 'box' variable is assigned to itself. Engine/source/forest/forestCollision.h 61
Comment: When copying a cv object, its cv.box field was not copied.

In addition, I suggest having a look at the emails, sent from @pvs-studio.com.
Best regards,
Phillip Khandeliants

Urgh sorry meant that to just be a comment, not a review. Curse you github!

These all look like the types of fixes we've thrown at bugs when they've been caught by other code analyzers with the exception of the  U32 to S64 switch, which would bump that from an unsigned 32 bit to a signed 64 bit . Did you perhaps mean S32 there?
As to "In addition, I suggest having a look at the emails, sent from @pvs-studio.com." Have those been forwarded to someone, like say, @Areloch?

@Azaezel
ov_read function returns result of the long type. The size of the long type depends on the data model used. The table below shows the size in bits of the long type in different data models.

ILP32
LP64 (Linux x64)
LLP64 (Windows x64)
ILP64

long
32
64
32
64

As you can see, long is a very dangerous data type. So I changed the type of the bytesRead variable to int64_t (S64), so as not to lose significant bits on the platforms with the LP64 and ILP64 data model.
Regarding the sent mail, it wasn't delivered to @Areloch, because my colleague could not find his email address.

Very nice! And yeah, PVS is a lovely tool. I'll be taking a look through this, but especially good catch with the different behavior with long, stuff like that is the bane of cross platform for sure.
As for email, no worries! For future reference, it's Areloch at gmail dotcom. :)


docs: add link for emoji list

[][demo]

🧰 Changes
Add a link to the full list of emojis.

This PR was released!
🚀 Changes included in v6.53.1


Name the 2 Windows services that Full mode installs on the VM

Article states multiple times that Full mode installs two Windows services.  Please name the Windows services.

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

ID: 6f2e47a4-1781-8c55-c269-1946adbb46db
Version Independent ID: ed091689-e342-2661-8b3a-5be2ad164b3a
Content: Register a SQL Server virtual machine in Azure with the SQL VM resource provider
Content Source: articles/virtual-machines/windows/sql/virtual-machines-windows-sql-register-with-resource-provider.md
Service: virtual-machines-sql
GitHub Login: @MashaMSFT
Microsoft Alias: mathoma

Thanks for the feedback! We are currently investigating and will update you shortly.

@MashaMSFT @MikeRayMSFT can either of you elaborate on this? I have dug around but am not finding anything concrete.

Hi @kar400 , thank  you for contacting Microsoft! The two service names are:

SqlIaaSExtensionQuery (Display name - Microsoft SQL Server IaaS Query Service)
SQLIaaSExtension (Display name - Microsoft SQL Server IaaS Agent)

You can see both of these services in the services console accessed by "services.msc". Thank you for bringing this gap to our attention! I will add it to the documentation :)
Thanks!
Masha from the SQL Docs team

Hi @kar400, thanks again for contacting Microsoft and letting us know this information is missing! I have plans to add this to the documentation and am tracking that internally ([1645871(https://mseng.visualstudio.com/TechnicalContent/_workitems/edit/1645871)]). We will proceed with closing out this Git Issue in the meantime, but please feel free to comment should you need anything further.
Thanks and hope you have a wonderful rest of your day!
Masha from the SQL Docs team
#please-close


Add a Chinese font?

The current font seems to work fine on Japanese glyphs, but Chinese is a disaster.
And only XMB allows choosing custom fonts.
We surely don't want users seeing this on their first startup...

Should this issue be combined with #336?

I'm going to go ahead and make the executive decision to close this issue in favor of addressing it over on #336.  If you feel this is a mistake, please respond to this issue and we can look at re-opening it.


[#1325] Port graphql integration to graphql-java 16.0

Description
Port integration/graphql to graphql-java 16.0

remove dependency to graphql-java:graphql-java-tools and base on
graphql-java:graphql-java directly
Fix bindings against graphql-java:graphql-java - no features or fixes intended
Implement support for Embedded IDs within EntityViews

Related Issue

#1325

Motivation and Context
Path grounding for integration with netflix-dgs and other graphql-java libraries based on the newer version which is not binary compatible

@beikov I would say this one is good enough for a first review. I should cover a huge range of cases with my application right now. We have a really big spec now, with everything:

Deep EntityView into EntityView mapping
Collections nesting collections
Tons of Enums and Scalars of different types including DateTime, LocalDataTime, Long
All sorts of ViewFilters
MultiFetches, InputTypes
Mixing custom Types with the Types coming from this integration (EV types and other types)
Pagination

The things I would love to discuss improving (which are beyond the port):
a)extract the actual type registry generation from com.blazebit.persistence.integration.graphql.GraphQLEntityViewSupportFactory#create to a custom method and use create to just initialize GraphQLEntityViewSupport. The current is a mix of concerns and this makes it unnecessarily harder to expose GraphQLEntityViewSupport as a bean while letting the graphql runtime wiring know about the types. Splitting those 2 would make this a lot easier and cleaner
b) Current there is no support for nullable and not-nullable values. This means that all types have the mandatory ID! and all other fields are optional. We could fetch the 'nullable' status from the entity field definition and include this. This would help a lot with the from schema to client code generators coming with Appollo or urql and others. Especially when using with TypeScript the missing information of nullable hurts a lot.
Besides that, this is an important information anyway. We could discuss this in an additional topic of course

Just have seen that i accidently commited all the pom.xml files .. trying to get rid of id

Superseded by https://github.com/Blazebit/blaze-persistence/pull/1343
Thanks a lot for the initial implementation. After some time working with this stuff, I realized only a few reflection accesses are necessary to support 9.2 - 16.2 so I added that for now. We can rethink this again in case graphql-java matures or breaks the API again.


Improvements and fixes

A useful little utility. I found some problems with it; the numbers were wrong by a fair bit. I don't know why you were ignoring all sections except .text and .rodata (which doesn't even exist in PE x86 files - it's called .rdata); I switched to including everything but .bss. Maybe something Arduino specific. So maybe that change isn't appropriate for you.

Thank you so much! I used it for ARM development, that's why the linker sections were hard coded.


ROX-20877: Do not process audit logs if Sensor is offline

Description
The compliance service should not send audit log messages to the manager if sensor is offline. This will prevent the channels from blocking the service that enables communication between Sensor and Compliance.
On reconnection Central will send the last processed state and Sensor will request Compliance to start from there. Because of this, there is no need to make sure that Sensor requests Compliance to start from Sensor's last processed state as it will start from Central's last processed state.
Compliance also stops the audit log reader before starting a new one so there is no data race risk.
Checklist

[ ] Investigated and inspected CI test results
[x] Unit test and regression tests added
~[ ] Evaluated and added CHANGELOG entry if required~
~[ ] Determined and documented upgrade steps~
~[ ] Documented user facing changes (create PR based on openshift/openshift-docs and merge into rhacs-docs)~

If any of these don't apply, please comment below.
Testing Performed
Here I tell how I validated my change

[ ] CI
[x] New unit tests added

Reminder for reviewers
In addition to reviewing code here, reviewers must also review testing and request further testing in case the
performed one does not seem sufficient. As a reviewer, you must not approve the change until you understand the
performed testing and you are satisfied with it.

/retest


Replace all occurrences of rmcaixeta by JuliaEarth in the URLs

Badges and possibly other files need to be updated to point to JuliaEarth.

Fixed this one. Also updated the URL at JuliaRegistries/General


[core] Fix ReferenceError in browser builds without process polyfill

Closes #13163

Maybe we shouldn't have MUI_SUPPRESS_DEPRECATION_WARNINGS at all?

@oliviertassinari I didn't know any other solution for our unit tests that didn't involve rewriting all of them.

I didn't know any other solution for our unit tests that didn't involve rewriting all of them.

@eps1lon The current solution is hiding valuable warnings. For instance, yesterday, I have discovered an issue with the legacy button variants: https://github.com/mui-org/material-ui/pull/13158/files#diff-4175d84e63a1a5a340986f250b17c796R108.

I don't think that's useful at all.

@eps1lon We have left variants behind when introducing the warnings. I looking for a solution to limit this possibility as much as possible. Maybe we should collect the warnings in the visual regression tests and throw if we see one.

If they don't want to migrate then they can simply suppress deprecation warnings and pin to MAJOR.

Have you ever seen React providing a way (in the public API) to suppress deprecation warnings? Not I'm aware of. It's done on purpose.

I don't think that's useful at all.

@eps1lon We have left variants behind when introducing the warnings.

The button variant deprecation did not need the env variable, correct. For symmetry I wanted users to have the opportunity for every deprecation. I should've checked for remnants but since they were already documented as deprecated I assumed this was already done.

If they don't want to migrate then they can simply suppress deprecation warnings and pin to MAJOR.

Have you ever seen React providing a way (in the public API) to suppress deprecation warnings? Not I'm aware of. It's done on purpose.

Then we don't do it the next time. There were no objections during review of the PR that introduced them.

There were no objections during review of the PR that introduced them.

I'm only using this argument because we can leverage it to simplify the situation :)


SolastaUnfinishedBusiness

Barbarian level 3 Path (Spirit Path) (Wolf Spirit) Only gives allies advantage and not barbarian. It is supposed to give barbarian advantage
on attacks when raging.

Duplicate


Scope using with() method

Summary of problem or feature request

Scope not allowed to add custom condition based on role or two different session user id.
Add functionality of attributes on scope.
Code snippet of problem
namespace Yajra\Datatables\Contracts;

protected $attributes = []; //  or protected static $attributes = [];
public function with($key, $value = null)
    {
        if (is_array($key)) {
            $this->attributes = array_merge($this->attributes, $key);
        } else {
            $this->attributes[$key] = $value;
        }

        return $this;
    }
public function __get($key)
    {
        if (array_key_exists($key, $this->attributes)) {
            return $this->attributes[$key];
        }

        return null;
    }

System details

Operating System - Ubuntu
PHP Version - 7.1
Laravel Version 5.3
Laravel-DataTables Version 8.3

Not sure if I get this right, but maybe the role condition you're referring should be added before calling the with method?
public function index (UsersDT $dt) {
  if (..) {
    $dt->with(...);
  }

  return $dt->render('view');
}

It's for the datable scope not for the datatable service.
namespace Yajra\Datatables\Contracts;
So we can use with the method in the datatable scope.

If dataTable scope, use addScope(new ScopeClass).


chore(dec-config): add tests to notifier updater

Description
Increase test coverage for the notifier updater in declarative config.
Checklist

[ ] Investigated and inspected CI test results
[ ] Unit test and regression tests added
[ ] Evaluated and added CHANGELOG entry if required
[ ] Determined and documented upgrade steps
[ ] Documented user facing changes (create PR based on openshift/openshift-docs and merge into rhacs-docs)

If any of these don't apply, please comment below.
Testing Performed
Here I tell how I validated my change

see CI.

Reminder for reviewers
In addition to reviewing code here, reviewers must also review testing and request further testing in case the
performed one does not seem sufficient. As a reviewer, you must not approve the change until you understand the
performed testing and you are satisfied with it.

[!WARNING]
This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
Learn more

#10440  👈
#10439
#10421
#10420
#10419
#10418
master

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @dhaus67 and the rest of your teammates on  Graphite


refactor(rtc): simpler metadata updates

Currently, any Timeline::schedule_uploads will generate a fresh TimelineMetadata instead of updating the values, which it means to update. This makes it impossible for #6994 to work while Timeline receives layer flushes by overwriting any configured new ancestor_timeline_id and possible ancestor_lsn.
The solution is to only make full TimelineMetadata "updates" from one place: branching. At runtime, update only the three fields, same as before in Timeline::schedule_updates.

Will merge this after release is cut, even though it shouldn't change anything.


Question: Non Google Device

Hi, is this plugin working for device that without google service? like Huawei Nova.
I done install it and test it, after log "Worklets loaded successfully", my device just display full black screen.
"react": "18.2.0",
"react-native": "0.71.17",
"react-native-vision-camera": "^3.9.0",
"react-native-vision-camera-v3-face-detection": "^1.1.3",
"react-native-worklets-core": "^0.4.0",

Hello, this module should work because I integrated without using play-services.

hi, but i my side didnt display out the camera.
<Camera
          options={{
            performanceMode: "fast",
          }}
          style={StyleSheet.absoluteFill}
          // device={device}
          callback={(data) => {
            console.log("Face Data");
            console.log(data);
          }}
          // {...props}
        />

after i add "isActive"={true}, the camera shows out, but the camera just display in 1 sec then it just freeze on it

working after add photo={true}


How can I add my own headers with any request?

Headers like: "Authorization" header where I can pass api key

the example for the same is mentioned in the readme file
https://github.com/codecat15/HttpUtility#authentication-token


Add CONTRIBUTING & CODE_OF_CONDUCT

Resolves #923 by adding the code of conduct, and takes a stab at contribution guidelines.  CONTRIBUTING felt appropriate to give discoverability to this info from the README.
Testing Done:
View these three files in my branch:

https://github.com/wadells/gravity/tree/code-of-conduct#contributing
https://github.com/wadells/gravity/blob/code-of-conduct/CONTRIBUTING.md
https://github.com/wadells/gravity/blob/code-of-conduct/CODE_OF_CONDUCT.md

Double checked that the links work.

This PR can't be built because Robotest doesn't currently work with forks.  I will close and reopen from a branch within gravitational/gravity.


Unity Console spamming: "ArgumentException: The specified path is not of a legal form (empty)"

I looked this issue up in google and found no help, so I'm posting here.
I'm using the latest version of unity: 2019.2.17f1
And latest version of FMOD unity plugin: 2.00.06
Visual studio version: Microsoft Visual Studio Community 2019, Version 16.4.2
I create a new Empty unity project, install the plugin, Add an FMOD project folder.
I add a single script to an empty scene:
public class TestFMOD : MonoBehaviour
{
    private EventInstance audio;

    void Start()
    {
        audio = FMODUnity.RuntimeManager.CreateInstance("event:/Level1/Music1");
        audio.start();
    }
}

When I ran the scene, the music plays with no issues, but the console keep spamming hundreds of this error message, this makes it impossible for me to locate actual errors in my own code as they get swallowed by this immediately:
ArgumentException: The specified path is not of a legal form (empty).
System.IO.Path.InsecureGetFullPath (System.String path) (at <567df3e0919241ba98db88bec4c6696f>:0)
System.IO.Path.GetFullPathInternal (System.String path) (at <567df3e0919241ba98db88bec4c6696f>:0)
System.IO.FileInfo.Init (System.String fileName, System.Boolean checkHost) (at <567df3e0919241ba98db88bec4c6696f>:0)
System.IO.FileInfo..ctor (System.String fileName) (at <567df3e0919241ba98db88bec4c6696f>:0)
(wrapper remoting-invoke-with-check) System.IO.FileInfo..ctor(string)
UnityEditor.VisualStudioIntegration.SolutionSynchronizer.ShouldSyncOnReimportedAsset (System.String asset) (at C:/buildslave/unity/build/Editor/Mono/VisualStudioIntegration/SolutionSynchronizer.cs:224)
System.Linq.Enumerable.Any[TSource] (System.Collections.Generic.IEnumerable`1[T] source, System.Func`2[T,TResult] predicate) (at <351e49e2a5bf4fd6beabb458ce2255f3>:0)
UnityEditor.VisualStudioIntegration.SolutionSynchronizer.SyncIfNeeded (System.Collections.Generic.IEnumerable`1[T] affectedFiles, System.Collections.Generic.IEnumerable`1[T] reimportedFiles) (at C:/buildslave/unity/build/Editor/Mono/VisualStudioIntegration/SolutionSynchronizer.cs:213)
UnityEditor.SyncVS.PostprocessSyncProject (System.String[] importedAssets, System.String[] addedAssets, System.String[] deletedAssets, System.String[] movedAssets, System.String[] movedFromAssetPaths) (at C:/buildslave/unity/build/Editor/Mono/SyncProject.cs:165)
UnityEditor.AssetPostprocessingInternal.PostprocessAllAssets (System.String[] importedAssets, System.String[] addedAssets, System.String[] deletedAssets, System.String[] movedAssets, System.String[] movedFromPathAssets) (at C:/buildslave/unity/build/Editor/Mono/AssetPostprocessor.cs:159)
UnityEditor.AssetDatabase:SaveAssets()
FMODUnity.EventManager:CopyToStreamingAssets() (at Assets/Plugins/FMOD/src/Editor/EventManager.cs:584)
FMODUnity.EventManager:RefreshBanks() (at Assets/Plugins/FMOD/src/Editor/EventManager.cs:37)
FMODUnity.EventManager:Update() (at Assets/Plugins/FMOD/src/Editor/EventManager.cs:656)
UnityEditor.EditorApplication:Internal_CallUpdateFunctions() (at C:/buildslave/unity/build/Editor/Mono/EditorApplication.cs:303)

Am I doing something wrong or is this a bug?

This issue has already been fixed for the next release: https://github.com/fmod/UnityIntegration/commit/fd6e3065520ec4383229ca3ba20f3ac38ab9bc59


Event reservation information

Based on the event model, there is some information we think would be good to surface to users.

Let's update this document with what we decide?
https://www.notion.so/wellcometrust/Event-messaging-6b0a3a5d68af48c981913a1a9c096e78


Fixed About UI padding & added Install Path & various typos

Summary
Various typos as well as slight, non-impactful design changes are made.
Changes Made
This PR:

completes the "Contributer" -> "Contributor" typo fix in the About UI
creates a new entry in GeneralPreferences that displays the install path of AE
adds padding to About UI subviews for visual appeal

TODO
N/A
Motivation
These changes contribute to the overall design of the product.
Testing
Test build, but these changes are not code-breaking.
Screenshots/GIFs
About UI Padding

Install Path

Typo Fix

Checklist
Please ensure all of the following are completed before submitting the PR:

[ ] Code has been tested and verified.
[ ] Documentation has been updated to reflect the changes.
[ ] All tests pass successfully.
[x] Code follows the project's coding guidelines and style.
[x] The branch is up-to-date with the latest changes from the main branch.
[ ] Reviewed by at least one other contributor.

Related Issues
This PR addresses the following issue(s): []
Additional Notes
N/A

Thanks for submitting a pull request!
We will review it as soon as possible.

Hey @RyloRiz,
Thanks for your contribution.
Your PR has been merged!
Your name will appear on https://auroraeditor.com/#contributors in a few moments, if you have contributed to the editor.
Thanks again for your contribution!
Kind Regards,
Aurora Editor Team.


DataProtection Redis only supports 2.2 of ASP.NET Core

Is your feature request related to a problem? Please describe.
DataProtection Redis only works with 2.2.
Describe the solution you'd like
Make it work with 2.1 if possible
Describe alternatives you've considered
None
Additional context
None

This problem can happen during a nuget restore:
NU1107: Version conflict detected for Microsoft.AspNetCore.Cryptography.Internal. Install/reference Microsoft.AspNetCore.Cryptography.Internal 2.2.0 directly to project <projectName> to resolve this issue.

This combination results in the above failure:
<TargetFramework>netcoreapp2.1</TargetFramework>
<PackageReference Include="Microsoft.AspNetCore.App" />
<PackageReference Include="Steeltoe.Security.DataProtection.RedisCore" Version="2.3.0" />

This combination does not:
<TargetFramework>netcoreapp2.1</TargetFramework>
<PackageReference Include="Microsoft.AspNetCore" Version="2.1.7" />
<PackageReference Include="Microsoft.AspNetCore.Mvc" Version="2.1.3" />
<PackageReference Include="Steeltoe.Security.DataProtection.RedisCore" Version="2.3.0" />

This issue is a direct result of changing the dependency in Steeltoe.Security.DataProtection.RedisCore from Microsoft.AspNetCore.DataProtection.Redis (v0.3) to Microsoft.AspNetCore.DataProtection.StackExchangeRedis (v2.2) -- this change was made as what was seen at the time as the simplest way to take advantage of all the improvements in the StackExchange.Redis 2.0 release.
Two potential changes we could make to allow everything to work would be:

Multi-targeting Steeltoe.Security.DataProtection.RedisCore
Releasing multiple packages to align with what MSFT did with the underlying DataProtection packages

Neither of those changes is clean (likely breaking some apps either way), given that we've already shipped changes in this area. Additionally, the Redis DataProtection package for versions < 2.2.0 has been unlisted from nuget.org, underscoring the reasoning for rolling forward to take advantage of the newer v2 StackExchange.Redis driver. I'm inclined to leave this as a known issue (with the workaround of specifying more packages than AspNetCore.App) until/unless further issues are discovered


Add _delete_single_acl routine in OvnScenario class

Introduce _delete_single_acl routine in order to add the capability to
remove just a single rule in a given ACL table. If _delete_single_acl is
run with default parameters it will erase all the ACL table for a given
switch

but if I remove the flush() in _delete_single_acl() the operation is not
performed

That's not true, because you already added a flush() at L199, which will
make sure all commands appended in the loop finally being executed as a
single transaction.
I mean if you run just _delete_single_acl() not _delete_acl()

Understand, but forcing flush() everywhere is not a good idea. As we know that ovn-nbctl performance is a problem at scale, when we set batch mode we really want it to take effect. In this case, I think we should let the caller of _delete_single_acl() to decide when to do flush(). It should not flush ignoring the caller's attempt of doing it in batch.


On Render Method

I'd like an on render method to be available.  It'd help in that we can hide the container until the flipster is ready, then call a unhide so there isnt odd dom views.  For instance, if we are waiting for scripts to load, you can get "stacked" images until they load and flipster fires.  Or if you want to hide the parent container until it is rendered, you'll get a window between making the parent visible and the actual flipster rendering.

This repository is going into very-low-maintenance mode and declaring backlog bankruptcy


apolloOptions.context type and default value

By default context is undefined and has a type to any.
attachConnectorsToContext from graphql-tools works only with an object typed context.
If context is undefined, it will raise an error for each requests.
So, we have to explicitely set context to an empty object to make it working :
server.use('/', bodyParser.json(), apolloExpress({
  schema: executableSchema,
  context: {},
}));

Maybe apolloOptions.context can have a default value setted to {} and be explicitely typed as Object.

Yeah, it's true that there's a bit of a mismatch. The reason we let it be any is that graphql-js accepts any as the context, and we don't want to change that function signature with apollo server. Maybe there's a place in the documentation where we can make it clear that if connectors are used, the context has to be explicitly set to an empty object first?

@helfer it will be very helpfull, i've moving to new server and spent some time to make it works, before i found the solution!

If someone could submit a PR to the docs, that would be great! 😃

Guys, since context now it an object, how can I modify context based on request? In Meteor the context is filled with userID, so how can this be achieved with the new design ?
I need context to be a function that accepts request.

I think this issue is mostly solved, so I'll close this. Feel free to open again if there's still something to discuss.


[website] Add Diego to About Us page

[x] I have followed (at least) the PR section of the contributing guide.

@danilo-leal Thanks!


Add enterprise binaries

Prioritized features:

Namespaces
Sentinel

@zhenik only namespace is relevant for first iteration.

#195 segregate issue


Add context introspection API to wrapper

Fixes #146.
Adds an API to shallow/mount wrappers called context based on the discussion in the above thread. The below now works.
wrapper.context() // returns context object
wrapper.context('foo') // returns foo off of context object

I suppose I probably need to add documentation for this too.

Fixed!

@blainekasten remember to add the 2 lines in /docs/README.md or else gitbook won't add those pages to the docs site!

Done.


Feedback for: engine/installation/linux/debian.md

URL: https://docs.docker.com/engine/installation/linux/debian.md
When I run sudo apt-get update, I have the error of

Err:10 https://apt.dockerproject.org/repo debian-xenial/main armhf Packages 403  Forbidden

I do have the "/etc/apt/sources.list.d/docker.list" edited with an entry. Any ideas? Thank you in advance

Hi, others have faced similar issues, as seen in https://github.com/docker/docker/issues/20192. One of the troubleshooting steps is to make sure the time is set correctly on your computer. Please have a look through those and also consider searching on forums.docker.org. Let me know if you find your specific problem so that we can determine whether it needs to be documented.


Suggest to add Chinese interface

Dear developers,
I look forward to future software versions including an option for a Chinese interface, which would enhance the experience for Chinese users. Thank you.

@chaserene When will localization be supported? I'm looking forward to implementing it soon, anything I can do to help is appreciated!


Icons rollback

hey Terri! :tada: I "Photoshopped" the icons for the rollback. Merge if you like 'em.
See all icons here

Great job! I see that code of conduct is also is this pull request, which is great. Was there a problem getting the icons in svg format? SVG is just more scalable and lighter, but can work around this if svg is not possible.

Great job! I see that code of conduct is also is this pull request, which is great. Was there a problem getting the icons in svg format? SVG is just more scalable and lighter, but can work around this if svg is not possible.

Oh-no... the Code of Conduct, that's a mistake. I forgot to include it in the .gitignore file. I'm not finished with that.
I don't know how to generate svg format, yet. I made them png, and I know the file size is bigger compared to svg. I will work on that tomorrow 📌

Great job! I see that code of conduct is also is this pull request, which is great. Was there a problem getting the icons in svg format? SVG is just more scalable and lighter, but can work around this if svg is not possible.

Oh-no... the Code of Conduct, that's a mistake. I'm not finished with that and had no intention to share it yet. It's actually listed in my gitIgnore file and somehow it was still committed to the push.
I don't know how to generate svg format, yet. I made them png, and I know the file size is bigger compared to svg. I will work on that tomorrow 📌

No problem. There is no need to rush in doing that. We can work with png files and they are a fallback, so hopefully they are not going to be used very often. Generating 3D models is the hard part and the heaviest part.


Related work for paper review

The one thing missing for me is the related work. I did a very short search for RL and wildfire evacuation strategies and found a few papers:

https://link.springer.com/article/10.1007/s10694-023-01363-1
https://www.climatechange.ai/papers/neurips2021/22
https://www.sciencedirect.com/science/article/abs/pii/S0378437121001175
https://pubs.aip.org/aip/acp/article/3078/1/040005/3284720/A-reinforcement-learning-algorithm-for-the-optimal

and software

https://github.com/elbecerrasoto/gym-forest-fire

I understand that these are  a bit different to what you have done but I guess it should be mentioned somewhere in the paper as it is a specific requirement in the JOSS guidelines.

Hey @SamTov -- sounds good! I believe we should've already looked at the repo, but will take a look and write up a paragraph or two about the related work. Thanks again for the ping!

https://github.com/openjournals/joss-reviews/issues/6739 (just commenting so these get linked :smile: )

Hey @SamTov - thanks for the feedback! We reworked the related work in the Statement of Need to include three of the five references you listed above alongside more references to work done on wildfire evacuation and work that used RL for evacuation.

Looks good on my side, thanks for the additional work!


Running app as root breaks on Wayland

The application launcher starts this app as root, which should be avoided, as it breaks the app on Wayland. Systemd can handle root requests itself when needed. I suggest changing the app launcher/.desktop to just start it with systemd-manager instead of pkexec systemd-manager (aliased as systemd-manager-pkexec).

hi,
what should i do to run the app with gnome wayland?

@Cherkah If you have a recent version of Gnome it should use Wayland by default, otherwise you can select Gnome instead of Gnome on Xorg on the login screen. If you run echo $XDG_SESSION_TYPE in your terminal it should say if you're using Wayland or X.
If that says Wayland then this app will also run natively on Wayland.

ok,
it's my case (debian stretch testing...) so gnome with wayland by default.
as the rustup process do not work with waykand i'll remove rustup and install systemd-manager via the installation script so as to get a proper integration with PolicyKit.
lets a try and confirme next

If that says Wayland then this app will also run natively on Wayland.

sorry but the app do not run on wayland. after the password nothing append.

Yes, that's exactly the problem. Apps can't run as root on Wayland. Only actions that need root access should request root permissions.

Systemd can handle root requests itself when needed.

This is not true. You are referring to the systemctl command. This application is not using systemctl, but communicating directly to systemd via it's dbus IPC API. Attempting to tell systemd to enable/disable a service will give you an interactive authentication required error.


RSDK-1843 Add wrapper for robot

JIRA: https://viam.atlassian.net/browse/RSDK-1843
This PR mainly adds a wrapper for the viam robot, but also introduces a few other changes:

[x] SessionManager uses aforementioned robot wrapper
[x] Documentation
[x] Tests (only for Session Manager at the moment)

TODO:

[x] Revert using @main branch for CI
[ ] Q on frame system: https://github.com/viamrobotics/viam-typescript-sdk/pull/20/files#r1100213286

I'm assuming session manager did not change w.rt. fixes/functionality/flow other than adding the async versions of the functions

Correct, the tests I added passed against the original SessionManager implementation - I did not have have to update them at all after updating.


Tiled is open source

Also, aseprite is open source.

Aseprite has 3 different licenses so i would say it is partially free on our case. #123 will fix all the cases. Thanks


codec(ticdc): simple protocol decoder ignore generated column if the value cannot be found

What problem does this PR solve?

Issue Number: close #10837
What is changed and how it works?

ignore the column if the value cannot be found from the encoded message.

Check List
Tests

Unit test
Integration test

Questions
Will it cause performance regression or break compatibility?
Do you need to update user documentation, design documentation or monitoring documentation?
Release note
`None`

/retest

/retest

/retest

/retest


Font

Hey, I not success to get the default font from your theme, how activate it ?

The default font 'Helvetica Neue' actually is not on Google Fonts. Since slidev pulls fonts from Google Fonts (see docs), I assume this is the reason why it doesn't work.
As a workaround you could replace the font in the front matter with 'Roboto' which is quite similar:
---
fonts:
  sans: 'Roboto'
theme: apple-basic
---

Though, I am wondering why 'Helvetica Neue' is working fine locally, at least in my case.


Revert "Update babili-webpack-plugin to version 0.0.8 🚀"

Reverts chentsulin/electron-react-boilerplate#670

I realized that problem might be from<EMAIL_ADDRESS>(0.1.1 works fine), so it will still not work even if we reverted the webpack plugin version.

What about temporarily disabling babli?

Merging because of https://github.com/chentsulin/electron-react-boilerplate/issues/672#issuecomment-274996314. I also confirmed this with my own local revert

We must ensure CI is passing.. I guess somebody are still use<EMAIL_ADDRESS>because yarn lock, we can get 0.1.2 by npm, or yarn upgrade.
I'll submit a better temporarily fix for that.

This seems to have fixed it. See https://travis-ci.org/chentsulin/electron-react-boilerplate/builds/195083218, which is a test of a branch that has this change. It seemed like the cache was outdated

@amilajack looks like this passed build is still used 0.1.1. (see this line)


Add Rust Toolchain File

Add Rust Toolchain File
Adding a Rust toolchain file allows us to ensure our CI tests against
stable. This file tells Rustup which toolchain to use, and CI will
automatically ensure Rust is running the latest version of stable, since
we include rustup update stable in our CI script.
chore: Add Rust toolchain file so we track stable.
Improve testability of Chi Square calc engine.
This commit is an iteration on the design of the Chi Square calculator.
I realized the calculator should be purely functional so we can write
better test cases, instead of collecting observations within the
calculator itself. This commit rewrites the calculator to be more
testable.
test: Refactor Chi Square calculator to improve testability.

#7  👈
trunk

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @RobbieMcKinstry and the rest of your teammates on  Graphite


Refactor of SocketRelay and polish/comments

Changes:

Refactors SocketRelays
Adds docstrings, especially to user-facing classes and methods
Improves/adds some error messages
Adds some asserts (offensive programming)
Adds a few comments
Fixes a bug or two iirc

TODO: Thorough testing
TODO: Flesh out the list of changes above

good stuff so far!


class_file?

So I tried to run your segments.py file and it asked for a class_file, not sure what it is.  I'm not sure if there are some missing steps in your documentation.
Thanks.

Hey I need to update the README. class_file refers to a file with the list of classes to appear along with the image, line by line.

I would love to use your work, when do you think you will update the readme to reflect the changes?


Uncaught Error: spawn EACCES

[Enter steps to reproduce below:]

...
...

Atom Version: 1.0.2
System: Mac OS X 10.10.4
Thrown From: omnisharp-atom package, v0.22.0
Stack Trace
Uncaught Error: spawn EACCES
At /Users/dopare/.atom/packages/omnisharp-atom/node_modules/rx/dist/rx.js:579

Error: spawn EACCES
  at exports._errnoException (util.js:734:11)
  at ChildProcess.spawn (child_process.js:1145:11)
  at Object.exports.spawn (child_process.js:977:9)
  at StdioDriver.connect (/Users/dopare/.atom/packages/omnisharp-atom/node_modules/omnisharp-client/lib/drivers/stdio.js:108:45)
  at ClientBase.connect (/Users/dopare/.atom/packages/omnisharp-atom/node_modules/omnisharp-client/lib/clients/client-base.js:234:22)
  at Client.connect (/Users/dopare/.atom/packages/omnisharp-atom/lib/omni-sharp-server/client.js:38:24)
  at SolutionManager.getClientForEditor (/Users/dopare/.atom/packages/omnisharp-atom/lib/omni-sharp-server/client-manager.js:213:31)
  at Omni.getProject (/Users/dopare/.atom/packages/omnisharp-atom/lib/omni-sharp-server/omni.js:218:24)
  at /Users/dopare/.atom/packages/omnisharp-atom/lib/omnisharp-atom/features/highlight.js:153:49
  at /Users/dopare/.atom/packages/omnisharp-atom/node_modules/rx/dist/rx.js:4924:20
  at tryCatcher (/Users/dopare/.atom/packages/omnisharp-atom/node_modules/rx/dist/rx.js:567:29)
  at InnerObserver.onNext (/Users/dopare/.atom/packages/omnisharp-atom/node_modules/rx/dist/rx.js:4806:43)
  at InnerObserver.tryCatcher (/Users/dopare/.atom/packages/omnisharp-atom/node_modules/rx/dist/rx.js:567:29)
  at AutoDetachObserverPrototype.next (/Users/dopare/.atom/packages/omnisharp-atom/node_modules/rx/dist/rx.js:5274:51)
  at AutoDetachObserver.Rx.internals.AbstractObserver.AbstractObserver.onNext (/Users/dopare/.atom/packages/omnisharp-atom/node_modules/rx/dist/rx.js:1710:35)
  at /Users/dopare/.atom/packages/omnisharp-atom/node_modules/rx/dist/rx.time.js:276:54
  at invokeAction (/Users/dopare/.atom/packages/omnisharp-atom/node_modules/rx/dist/rx.js:948:7)
  at /Users/dopare/.atom/packages/omnisharp-atom/node_modules/rx/dist/rx.js:1432:60

Commands

Config
{
  "core": {
    "destroyEmptyPanes": false
  },
  "omnisharp-atom": {
    "enhancedHighlighting": true
  }
}

Installed Packages
# User
atom-yeoman, v0.3.11
language-cshtml, v0.1.1
linter, v1.2.4
omnisharp-atom, v0.22.0

# Dev
No dev packages

Do you have mono 4.0.1 (or higher) installed?

Mono JIT compiler version 4.0.2 (Stable <IP_ADDRESS>/c99aa0c Wed Jul  1 14:19:56 PDT 2015)
Copyright (C) 2002-2014 Novell, Inc, Xamarin Inc and Contributors. www.mono-project.com
TLS:           normal
SIGSEGV:       altstack
Notification:  kqueue
Architecture:  amd64
Disabled:      none
Misc:          softdebug
LLVM:          supported, not enabled.
GC:            sgen

I had this problem, too, on two machines both running Linux.  Seems the 0.22.0 update removed the executable bit from a couple of files, namely the omnisharp and dnx scripts. I fixed that with:
chmod +x ~/.atom/packages/omnisharp-atom/node_modules/omnisharp-client/roslyn/omnisharp ~/.atom/packages/omnisharp-atom/node_modules/omnisharp-client/roslyn/packages/dnx-mono.1.0.0-beta4/bin/dnx
Making the omnisharp script executable again revealed a new "ENOENT" error, or file not found, but the atom error popup wasn't telling me which file wasn't found.  So I ran the command the server omnisharp was calling in my terminal:
.atom/packages/omnisharp-atom/node_modules/omnisharp-client/roslyn/omnisharp --stdio -s /dev/myProject/ --hostPID 54321
and got this error from the shell:
zsh: .atom/packages/omnisharp-atom/node_modules/omnisharp-client/roslyn/omnisharp: bad interpreter: /bin/bash^M: no such file or directory
Seems the script was confused by the Windows CRLF line-endings and couldn't find the /bin/bash executable since it considered the ^M part of the file name.  So I ran the omnisharp script through the dos2unix utility and NOW everything works.
Interestingly, my first response was just to uninstall and reinstall 0.20.0 as that version worked fine, but even after doing this the permissions were still incorrect for these files.

Yeah we had an issue with npm not installing correctly.  If it's still
happening, let us know, and I'll try publishing a new version (which should
the client that was built by travis properly)
On Sat, Jul 18, 2015 at 3:58 PM, Kf4btg<EMAIL_ADDRESS>wrote:

I had this problem, too, on two machines both running Linux. Seems the
0.22.0 update removed the executable bit from a couple of files, namely the
omnisharp and dnx scripts. I fixed that with:
chmod +x
~/.atom/packages/omnisharp-atom/node_modules/omnisharp-client/roslyn/omnisharp
~/.atom/packages/omnisharp-atom/node_modules/omnisharp-client/roslyn/packages/dnx-mono.1.0.0-beta4/bin/dnx
Making the omnisharp script executable again revealed a new "ENOENT"
error, or file not found, but the atom error popup wasn't telling me
which file wasn't found. So I ran the command the server omnisharp was
calling in my terminal:
.atom/packages/omnisharp-atom/node_modules/omnisharp-client/roslyn/omnisharp
--stdio -s /dev/myProject/ --hostPID 54321
and got this error from the shell:
zsh:
.atom/packages/omnisharp-atom/node_modules/omnisharp-client/roslyn/omnisharp:
bad interpreter: /bin/bash^M: no such file or directory
Seems the script was confused by the Windows CRLF line-endings and
couldn't find the /bin/bash executable since it considered the ^M part of
the file name. So I ran the omnisharp script through the dos2unix utility
and NOW everything works.
Interestingly, my first response was just to uninstall and reinstall
0.20.0 as that version worked fine, but even after doing this the
permissions were still incorrect for these files.
—
Reply to this email directly or view it on GitHub
https://github.com/OmniSharp/omnisharp-atom/issues/446#issuecomment-122588961
.

Issue reoccurred with 0.22.1 update. Same process as before took care of it.

Should be fixed in 0.22.2

Yep, works for me now. :tada:

Yea, it's fixed. Thank you guys.


Play with a running TrafficOps / Cache

Once we get an AWS environment setup with the cache and the Traffic Ops, I'll need to familiarize myself with how administrating the system works.

This should include modifying a configuration on TrOps and seeing how the ort script handles that and see if it is what I expected.


Change gitter links and references to discord

There's a reference to a non-existing Gitter channel. I assume the Discord server has replaced that.

Just to make sure: @b-mueller Is the invite link going through Daniel fine?

Just to make sure: @b-mueller Is the invite link going through Daniel fine?

Works for me. @cleanunicorn did you set "never times out" when generating the invite link in Discord?

You can also use the invite link used in the readme: https://discord.gg/E3YrVtG

Nvm, I see that the link you used also never expires


[performance]: *OpenChannel.isBorked() is innefficient

Environment
lnd version
lnd version 0.15.99-beta commit=tor/v1.1.0-41-gb8545de5c
(This is a development version I just happen to have compiled, I can test on another one if requested).
go version
go version go1.19.3 linux/amd64
platform
linux/amd64
Description
I have setup the following test env to profile payment performance with 3 nodes (Alice <> Bob <> Charlie) and started a continuous stream of payments from Charlie to Alice, while profiling (with --profile <port>) on Bob.
While the payments were made, I fetched a profile with curl http://<IP_ADDRESS>:<port>/debug/pprof/profile?seconds=60 and reviewed the results.
The top offenders for CPU load during payment that I can eyeball are:

bbolt.Tx.Commit()
lnwallet.DeriveCommitmentKeys()
channeldb.OpenChannel.isBorked()

While the first two are necessary (as they directly involve the protocol for updating channel state), upon reviewing the code for the third one, it seems less efficient than it could be. Specifically, it recreates the entire channel structure just to check the status of one specific field.
While I believe it's not absolutely safe to rely on the c.channStatus to check on the status (because other parts of the app might be using a different *OpenChannel instance to refer to the same underlying channel), this function could at least be improved upon by either reading the offset to the chanStatus field directly or reading and discarding the fields up to chanStatus and then returning immediately, which would improve performance.
I'm opening this issue prior to submitting a PR so that the best approach to solving this can be discussed.

See this issue for some prior discussion re reducing the number of db transactions in the critical path: https://github.com/lightningnetwork/lnd/issues/5186.
Also can you amend the OP to include the lnd/go/platform runtime information? Thanks.

This is indeed a subset of #5186, but note that this specific issue isn't so much about the db access pattern, but more of what exactly that isBorked function does (while inside a tx): even reducing/batching the db txs, it would still be the case that most of the work done by the isBorked function is wasted, due to only really using a single field of the underlying data.

this function could at least be improved upon by either reading the offset to the chanStatus field directly or reading and discarding the fields up to chanStatus and then returning immediately

Alternatively we could consider a design where the in memory OpenChannel behaves as a write through cache and we don't instantiate new OpenChannel objects but only use one instance per channel (reference counted). That way shared ownership would result in fast concurrent reads since all of the members are cached and exists only once. We experimented with something similar in https://github.com/lightningnetwork/lnd/pull/5595, but in the end it's more of a hack around the proper design than a good solution.

How big of a performance issue is this really?  @matheusd could you attach the profiles?
A dead simple improvement we could do immediately is to use fetchChanInfo instead of fetchOpenChannel in isBorked.  In general, it looks like there a many places we are doing full reading/writing with fetchOpenChannel/putOpenChannel when we don't need to.
@matheusd's idea to read just the chanStatus offset is a further optimization.  But it seems like a bit of a bandaid to avoid changing the DB structure.  If we really need to check channel status so frequently, maybe we should put them in a special subbucket.  This would also improve performance for putChanStatus and clearChanStatus.

How big of a performance issue is this really? @matheusd could you attach the profiles?

I can do that later today, but an easy way to see the impact of this is to run
make build-itest itest-only icase="async_payments_benchmark"
both with the current code and then with this following diff:

diff --git a/channeldb/channel.go b/channeldb/channel.go
index cd149c861..fc513f2aa 100644
--- a/channeldb/channel.go
+++ b/channeldb/channel.go
@@ -1454,12 +1454,7 @@ func (c *OpenChannel) ChanSyncMsg() (*lnwire.ChannelReestablish, error) {
 //
 // NOTE: The primary mutex should already be held before this method is called.
 func (c *OpenChannel) isBorked(chanBucket kvdb.RBucket) (bool, error) {
-	channel, err := fetchOpenChannel(chanBucket, &c.FundingOutpoint)
-	if err != nil {
-		return false, err
-	}
-
-	return channel.chanStatus != ChanStatusDefault, nil
+	return c.chanStatus != ChanStatusDefault, nil
 }

 // MarkCommitmentBroadcasted marks the channel as a commitment transaction has

In my personal machine this makes the benchmark's TPS go from ~130 to ~200.

Alternatively we could consider a design where the in memory OpenChannel behaves as a write through cache and we don't instantiate new OpenChannel objects but only use one instance per channel (reference counted).

Yes, IMO a design where only one OpenChannel instance exists throughout the app (and any channel-modifying operations go through that) would be ideal (because then you enforce that every subsystem is getting the most up to date view of the channel and all fields are already in memory).


Fix tsconfig.md regarding ./ prefix

Here is a little fix for the tsconfig.md documentation.
You mention a 'gs' snippet but the example in tsconfig.md differs from the snippet.
Is does not just differ but it seems to have a bug. I tried it and
the ./ prefix does not work for me - so I removed it for you.
Also the generated files section has no ./ prefix.

You are right. Actually can you add tsx as well. For reference this is the actual ideal glob : https://github.com/TypeStrong/atom-typescript/blob/f50fe08e7b3d1b33d01f6a02b9319e488bc97fbb/lib/main/tsconfig/tsconfig.ts#L224-L226 :rose:
Please leave a comment once done :+1:

Please leave a comment once done :+1:
Done 👍🏽

Thank you :rose: :heart:


Are there any plans to support the heic extension?

Is your feature request related to a problem? Please describe.
If you try to import an image with the following heic extension from the S3 bucket using SIH, a 500 error occurs.
heic image link: https://img.tumblbug.com/eyJidWNrZXQiOiJ0dW1ibGJ1Zy1kZXZlbG9wbWVudCIsImtleSI6InJldmlldy80MmIwNGEzZi02NTBmLTRhNTktYjRiMC0yYTg4NGI4MTRmMDMuaGVpYyIsImVkaXRzIjp7InJlc2l6ZSI6eyJ3aWR0aCI6MTI0MCwid2l0aG91dEVubGFyZ2VtZW50Ijp0cnVlfX19
{
  "status": 500,
  "code": "RequestTypeError",
  "message": "The file does not have an extension and the file type could not be inferred. Please ensure that your original image is of a supported file type (jpg/jpeg, png, tiff, webp, gif, avif). Inferring the image type from hex headers is not available for SVG images. Refer to the documentation for additional guidance on forming image requests."
}

The extensions currently supported by SIH appear to be png, webp, tiff, gif, jpeg, and avif, but the extension for photos taken with an iPhone is heic.
Considering the iPhone, which accounts for a large portion of mobile phones worldwide, I think it is a huge disadvantage that SIH does not support the heic extension.
I know that the heic extension has been discussed for a long time, and I wonder if you guys have the will to solve this problem. If you don't have the will to solve it quickly, can you tell me about another alternative?
Describe the feature you'd like
I hope you were able to successfully import the heic extension image using SIH.
Additional context
I am the most active user of SIH at my company, so I am very interested in your service. So I am also curious about why the heic extension is not supported. And this error is occurring in a method called inferImageType. Could you please explain whether this is simply a problem caused by not being able to infer the type, or if there is a further problem?

Hi @Na-hyunwoo,
While Sharp "supports" heic, due to the patent-encumbered nature of the format, it isn't supported out of the box and would require additional customization of our Sharp installation to make work. For this reason, we don't currently have plans to implement it, though that could change in the future. I'll add a task internally to evaluated the difficulty in adding support.
Thanks,
Simon

Hi @simonkrol,
Thank you for your reply.
Does the fact that the format is patented mean that it will be difficult for me to customize and use the solution?
If I can customize the solution and use it, can you tell me in detail what process I need to go through?
Since I do not have much knowledge about image processing and am just learning, I think the more detailed the explanation, the better.
Thank you 🙏


Error Uncaught Snapshot missing on Livewire component with id

Hi, I will raise this error by this means but in reality the problem is in the modal of the wire elements pro package, I did not find any place for support in the pro package, I have the problem with this livewire error Uncaught Snapshot missing on Livewire component with id: YVTGfSlUuUnLbYbvcx7Y that only happens within the pro modal or the slideover, the component simply stops working, its html content is displayed but the error appears in the console and alpine does not push the requests to the methods of the php component.
It is important to mention that this error must necessarily be in the wire elements component because when using the component in another place other than the modal, when accessing it through a route for example, everything works.

Hi @exonos
Thanks for your support! 😄
This error often occurs when you have an unclosed element in one or more components. It could be an unclosed  for example.
If the issue still persist, please create a simple repository that replicates this issue and send it to<EMAIL_ADDRESS>so I can take a look 🙏🏻😄


How to get buildType/flavorDimensions on another gradle file.

Below are the codes which I am trying to get the buildType of AndroidStudio project.
project.hasProperty('buildType')
or
project.getProperties().get('buildType')
Both of above lines return null values. So how can I get the project buildType in another gradle file (like sonar.gradle or jacoco.gradle).
Also from AndroidStudio 3.0 and above supports "flavorDimensions" property so how can I get the buildVarient in another gradle file?

The extension is called android.buildTypes, not buildType.


Completed Next Web Clone

Features

Fully responsive for SM, MD y LG devices, such as smartphone, iPad and desktop screen.
Use of media queries for breakpoints
Fontawesome for the icons!
Flexbox and Grid applied for positioning
CSS linters issues addressed

Changed made as requested

Project fixed and debugged
[x] Address all changes requested by TSE.
Branch does not have any conflict, it is just the temporal github pages branch that can be easily rebased.

Fixed suggestions made by code reviewer.

Changes requested addressed

Conflicts resolved
Readme updated
Linters debugged
Image resizing working properly in Firefox and Chrome


Icon for Dev Workspace Operator

The suggestion is to have a combination of the OLM / Operator and the Devfile:

https://devfile.io/
https://olm.operatorframework.io/

@beaumorley I love that. My preferred is the first one. What about using the blue of the devfile logo instead of the red?

looks, great
@beaumorley would it be possible to provide the base64 version of the first icon so we would verify how would it look when enabled on operator's csv?

@ibuziuk @l0rd I can ask Sunil to do this. I am going to Slack him in a thread with you @ibuziuk in case he has questions.

thank you! the target for the operator with icon is 0.16.0 release

Adding icons provided by the UX team

cb-base64-string.txt
cb-image-tag.txt
cb-bg-css.txt

m


Error

Hey,
The Iconizr task kept freezing so I managed to debug it and I got:
{ [Error: spawn ENOENT] code: 'ENOENT', errno: 'ENOENT', syscall: 'spawn' }
Have you ever come across this?
Thanks

Hi @onearmfrog,
as you did never respond to this issue anymore, I'll close this for now. Please feel free to re-open if you've still got problems.
Thanks, cheers,
Joschi


[build] Add PR validation for flight

Also includes a cosmetic commit that isn't worth a PR of its own

@phillipleblanc Regarging PR checks, should we perhaps include some flavour of cargo test --all-features to ensure that tests aren't broken?
As a side note, all the tests are currently passing, except for test_duckdb_connection_pool_with_attached_databases which is flaky , i.e. fails about half of the time with:
thread 'sql::db_connection_pool::duckdbpool::test::test_duckdb_connection_pool_with_attached_databases' panicked at src/sql/db_connection_pool/duckdbpool.rs:336:14:
Query should be successful: DuckDBError { source: DuckDBFailure(Error { code: Unknown, extended_code: 1 }, Some("TransactionContext Error: Catalog write-write conflict on create with \"attachment_0\"")) }

@phillipleblanc Regarging PR checks, should we perhaps include some flavour of cargo test --all-features to ensure that tests aren't broken?
As a side note, all the tests are currently passing, except for test_duckdb_connection_pool_with_attached_databases which is flaky , i.e. fails about half of the time with:
thread 'sql::db_connection_pool::duckdbpool::test::test_duckdb_connection_pool_with_attached_databases' panicked at src/sql/db_connection_pool/duckdbpool.rs:336:14:
Query should be successful: DuckDBError { source: DuckDBFailure(Error { code: Unknown, extended_code: 1 }, Some("TransactionContext Error: Catalog write-write conflict on create with \"attachment_0\"")) }

Yeah, I thought we were already doing that - so if we're not its a bug. @peasee can you take a look at the flaky test?

Yeah, I thought we were already doing that - so if we're not its a bug.

@phillipleblanc Nope, lib/doc tests aren't even compiled as part of the PR checks.


Make transformer model beam search aware

Make transformer model beam search aware to reduce memory usage and get better speed.
Given beam size N, it reduce memory usage for encoder statue to 1/N, and enable using Nx batch size to get sqrt(N) speed up.
Before submitting

[X] Was this discussed/approved via a Github issue? (no need for typos, doc improvements)
[X] Did you read the contributor guideline?
[ ] Did you make sure to update the docs?
[ ] Did you write any new necessary tests?

What does this PR do?
Fixes #1957.
PR review
Anyone in the community is free to review the PR once the tests have passed.
If we didn't discuss your PR in Github issues there's a high chance it will not be merged.
Did you have fun?
Make sure you had fun coding 🙃

Merge master and remove change in sequence_generator.py, this file is frequently updated. Only keep changes make transformer model aware beam search, which provide all memory saving gain and most speed up.


Query.node is not working when exposed through nexus-prisma

The Query.node field and Node type will soon be removed from prisma-generate-schema anyway. Let's just wait for that to be done

Fixed by #163


Autofillinterfaceemailtruncator refactoring

Refactored AutofillInterfaceEmailTruncator to just EmailTruncator.

Hey @MarlowBrown! Thanks for opening this PR.
Unfortunately, I'll close this for the time being, as this feature is under heavy development internally and this diff is incompatible with changes that are coming down the pipe very soon – we want to avoid as many merge conflict headaches as possible.
On that note, we typically don't accept refactoring PRs without discussing them in an issue first, we'd love to hear from you there if you spot areas of the codebase that you're interested in working on.


use text collision to determine ideal tick density

[ ] Mimic how QuickPlot calculates ideal tick density: it measures every tick and looks for collisions.

[ ] Also give the user a way to increase or decrease the tick density. Ensure the ability to manually define tick density is preserved.

this issue originated as #134

Actually, I don't want to waste energy fixing what's not broken. Tick system isn't too bad as it is now. No one's asking for this...


make -j8 error

Makefile.config:
CPU_ONLY := 1
I am trying to build with CPU only. I don't want to train the model but I wanted just to make an inference and to look in more details in the dimensions of different variables in the model.
Is it possible to build your version of caffe with CPU_ONLY flag? I succeeded it with normal caffe.
Error obtained:
src/caffe/layers/argmax_layer.cpp:99:29: error: macro "STUB_GPU_FORWARD" requires 2 arguments, but only 1 given
STUB_GPU_FORWARD(ArgMaxLayer);
^
src/caffe/layers/argmax_layer.cpp:99:1: error: ‘STUB_GPU_FORWARD’ does not name a type
STUB_GPU_FORWARD(ArgMaxLayer);
^
Makefile:581: recipe for target '.build_release/src/caffe/layers/argmax_layer.o' failed
make: *** [.build_release/src/caffe/layers/argmax_layer.o] Error 1

This is a bug you may encounter if you want to compile with CPU only flag on.
The backward function for argmax layer was not implemented for CPU users, this is why the author use STUB_GPU_FORWARD to register the layer for CPU instead of STUB_GPU like other layers. However, the author may have forgotten to add necessary argument for the function.
Go to argmax_layer.cpp and simply change STUB_GPU_FORWARD(ArgMaxLayer); to STUB_GPU_FORWARD(ArgMaxLayer,Forward); should solve the issue.

STUB_GPU_FORWARD(ArgMaxLayer,Forward);

Thanks a lot and helps my issuehttps://github.com/airsplay/lxmert/issues/89.

STUB_GPU_FORWARD(ArgMaxLayer,Forward);

Thanks a lot and helps my issuehttps://github.com/airsplay/lxmert/issues/89.


bug: input error style is broken

There is an issue with the style of the error, check here when editing a strategy:

Not sure what good workaround would be there for it as since https://github.com/snapshot-labs/sx-monorepo/pull/370 we use border to shift textarea contents down (but border is also used to display error).
@wa0x6e

We could use outline or shadow. Will take a look at a workaround, maybe give the error style to an extra wrapper


Adding terms for LGBT and suicide prevention issues.

Added terms, sources, and notes, and added to the data variable in extract.js.
My first pull request. :)
I was less familiar with node that I thought... I have node installed, but I wasn't certain how to test a local copy of the stuff I had forked. I went straight for the .yml files, but I'm not sure how to verify that this is all good.
Let me know if there are any other steps I should take. Cheers!

@capitocapito Thanks a lot!
First: very nice work, well done, the patterns are great additions.
Second: some slight points.

Could you add a final new line at the end of files? Google them for more
info :)

Maybe the AMAB/DMAB/AMAF/DMAF pattern should be of type simple,
right now, there are is a warning no born a man or born a female,
just for born a man or born a female

The alternative for bathroom bill is pretty hefty:
non-discrimination law/ordinance. Could you replace it with two
seperate suggestions? non-discrimination law and
non-discrimination ordinance is fine.

You can build the project using npm run build, and ensure everything works
using npm run make; Before these two you need to npm install first
though.

And, when you’re done, could you rebase the branch? Google can also help you there re how to do that :smile:

@wooorm Gotcha.

Will add new lines.

Understood. I may have misinterpreted how the or works (I was thinking it was necessary for categories). I'll change that and test.

Will split the bathroom bill suggestions.

Thanks for the hints! I'll play around more with npm, and rebasing shouldn't be a problem.

Thanks for the help! :D

:+1:

@capitocapito How are things going with this? I really appreciate your previous work :ok_hand:

@wooorm Hello! Busy week.
I've been able to complete the tasks that we outlined above, but I still haven't been able to test. I have node installed on my end, and I was able to install, build, and make retext-equality but I'm not sure how to test it locally with my copy of alex. So far, I've tried adding the local copy of retext-equality as a dependency for alex, but after doing the same install, build, and run, when I pass anything, new or not, (e.g. echo 'retard' | node alex) I am simply presented with a new command line with no output.
I'll push my fixes on retext-equality, though. I'd hate to make this a personal troubleshooting thread. :P

@capitocapito Cool! In theory, when you have retext-equality working (with some tests), you could run npm link from inside that package.
Then, you can git clone alex, install its dependencies, and run npm link retext-equality in it.
Both these actions set-up some sort of symlink between the two, enabling you to test your changes in retext-equality by running, e.g., alex <<< "foo bar retard baz"
See this npm doc for more info: https://docs.npmjs.com/cli/link

@capitocapito Are you still working on this? If not, I’d love to try my hands on this and finish the PR (If you’d like, it’s possible to still keep you as an author in Git, which will show your avatar on GitHub).

@capitocapito ping!

Hello! I haven't had a chance to work on this at all. I would say just go with the updates that the other person fixed with the spacing, since that's all I had done at this point and they had committed it first. Thank you, though!


feat(#144): Each Lint timings in benchmark

In this pull I've added BcProgram for measuring each lint timings for XMIR-generated program during the benchmark.
In the Benchmark results we will have the following:
Input: com/sun/jna/Pointer.class
Size of .class: 22Kb (22Kb bytes)
Size of .xmir after disassemble: 1Mb (1Mb bytes, 29630 lines)
Lint time: 42s (42355 ms)

alias-too-long (112 ms)
alias-without-tail (11 ms)
broken-alias-first (522 ms)
broken-alias-second (11 ms)
duplicate-aliases (37 ms)
...

closes #144
History:

feat(#144): timings
feat(#144): typo
feat(#144): Lint#name()
feat(#144): present timings.csv

@volodya-lombrozo could you review this one?

@volodya-lombrozo updated. Take a look, please

@h1alexbel @volodya-lombrozo thanks!


[ERROR:flutter/lib/ui/ui_dart_state.cc(166)] Unhandled Exception: PlatformException(VideoError, Video player had error com.google.android.exoplayer2.ExoPlaybackException: com.google.android.exoplayer2.upstream.AssetDataSource$AssetDataSourceException: java.io.FileNotFoundException: flutter_assets/assets/file.mp4, null

Video Player not loading video

Hey @Zeeshan-progs , this looks like a source error. Can you please check your video file source.
If that doesn't solve your problem, can you please provide the output of flutter doctor -v
Thank you

Hey @Zeeshan-progs , this looks like a source error. Can you please check your video file source.
If that doesn't solve your problem, can you please provide the output of flutter doctor -v
Thank you

flutter doctor and file din't  have any error

Ok, as mentioned in your error logs it was a source error. Can you please look into that

Hi @Zeeshan-progs,
The exception says it can't find the file specified. I see there's space in the video file you are trying to play, mirzapur 2.mp4. Try to remove space from original file name and then in pubspec.yaml and finally use the updated file name in your code and try again.
Also this seems to be code implementation issue and not Flutter issue. Closing.
If you disagree, write in comments and I'll reopen.
Thanks.


Make the text-area more Comprehensive

Users can add any text through the keyboard in the text-area field and applications stop further working. That's not comprehensive, So I put applied the function setEditable(false) on text-area to prevent any input data from the keyboard users can only hit calculator buttons to calculate the data.

Thank u for your contribution

That's really appreciable usama


MAINT: Split FIFF I/O into private submodule

First part is almost done I think -- just need to see what fails to tie up a few remaining loose ends that I'm sure CIs will reveal:

[x] Move non-Raw parts of mne/io to mne/_fiff
[x] Add deprecation wrappers for mne/io/*.py that moved
[x] Add tests for deprecation wrappers
[ ] Get CIs green

The next phase will be a bit more work -- moving sibling MNE-* packages to use only public APIs if possible. This will probably require adding some new functions, but we'll see.

[ ] Check mne-bids
[ ] Check mne-bids-pipeline
[ ] Check mne-connectivity
[ ] Check mne-nirs
[ ] Check mne-realtime
[ ] Check autoreject

Helpful for #11838 but cleans things up in its own right.

FYI, I added autoreject to the list of sibling packages to check.

Okay to merge @drammock ?


AWS S3 File Storage Configuration

When using on a Heroku deployed setup with AWS for file storage, is there a way to configure the slider to use the same settings as my spree store and upload the slide images to AWS S3 bucket.
If there is a way please document.

Hey, I had to do the same thing. This is what I ended up doing.
create a file app/models/spree/slide_decorator.rb
Spree::Slide.class_eval do

  if Rails.env.production?
    has_attached_file :image,
                      storage: :s3,
                      s3_credentials: {
                        bucket: ENV['S3_BUCKET_NAME'],
                        access_key_id: ENV['AWS_ACCESS_KEY_ID'],
                        secret_access_key: ENV['AWS_SECRET_ACCESS_KEY']
                      },
                      url: ":s3_domain_url",
                      path: '/spree/slides/:id/:style/:basename.:extension',
                      convert_options: { all: '-strip -auto-orient -colorspace sRGB' }
  end
end

That should do the trick.

@ddcech  this works very well, thanks!

Looks good to me.


Opening a crates versionless url links to prerelease versions

I just pushed a test crate to https://docs.rs/nemo157, there was a 0.3.0 version published previously and I just added a 0.3.1-pre version. I expected the versionless url to still redirect to the latest non-prerelease version, but instead it links to the prerelease version; and explicitly going to https://docs.rs/nemo157/0.3.0/nemo157/ shows a bright orange "Go to latest version" link.
I think for the purposes of promoting a stable ecosystem it would be best to treat prerelease versions as ineligible to be the latest version unless a crate has no normal versions available. Having them still listed under the versions for the crate and allowing them to be linked to directly would still be useful.

If like to boost the signal in this a bit. It's causing a bit of a yanking party in tokio-rs/tokio#2036

@Nemo157 the test crate makes this much faster to debug, thanks :)

I'm going to try and also fix #221 at the same time.

This is mostly fixed, I'm just trying to solve https://github.com/rust-lang/docs.rs/issues/395 at the same time. If I don't get it in the next half hour or so I'll just make a PR without.