🐛 [firebase_messaging] Missing support for communication notifications (dynamic icon avatar)

Hi.
it seems firebase service or client side plugin has no support for push notifications with dynamic icon.
Like you can see in slack whatsapp the users avatar instead of application icon.
Did not find anything in docs.
Will there be a support ?

@fullflash
Take a look at this link and see if it helps.

Yes but you cannot release an app with users avatar all the time)
It should be dynamic like whatsapp does
And this is push notifications should be natively implemented inside plugin.
On Wed, 15 Mar 2023 at 17:23 darshankawar @.***> wrote:

@fullflash https://github.com/fullflash
Take a look at this
https://stackoverflow.com/questions/43303218/dynamic-notification-icon-for-firebase-cloud-messaging
link and see if it helps.
—
Reply to this email directly, view it on GitHub
https://github.com/firebase/flutterfire/issues/10608#issuecomment-1470006321,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AEVLNPE6FFXDDOODASLMNRDW4G7GVANCNFSM6AAAAAAV3WZBRU
.
You are receiving this because you were mentioned.Message ID:
@.***>

It should be dynamic like whatsapp does

I am not sure what this means. I couldn't find any reference for such request in native SDK as well.

And this is push notifications should be natively implemented inside plugin.

Do you mean at Dart side or native (Android or iOS) ?

Hi. this is communication notifications request like in many communication apps.
there is a ios plugin https://github.com/lambiengcode/ios_communication_notification
also you can see the apple docs https://developer.apple.com/documentation/usernotifications/implementing_communication_notifications
Everyone is using whatsapp and you can see the senders avatar in push notifications for android and ios platforms.
P.S
it is a feature like notification service but it is for using custom avatar you can see in the apple docs.
https://firebase.flutter.dev/docs/messaging/apple-integration/#step-1---add-a-notification-service-extension

@fullflash
Since the plugin is a thin wrapper around native SDK, I think the said feature will need to be implemented at native side which the client side can leverage.


Refactor: MaintenanceBanner

MaintenanceBanner refactor
Updated tests configs to support TypeScript types

Preview is available here:http://laboratory-pr597.previews.kube001.services.stellar-ops.com/


any way to bind to all local interfaces

when registering routes
ie:
require('./redbird')({port: 80}).register("http://localhost/", "http://<IP_ADDRESS>/asdas");;
this will mean it only listens on locahost, if the server has multiple interfaces  ie:
locahost, <IP_ADDRESS> (loopbacks), <IP_ADDRESS>,  <IP_ADDRESS> how can we specify redbird to listen on all?
kind regards
oasenov

Sorry if this is a silly idea, but have you tried binding to <IP_ADDRESS>? Typically that will bind to all local interfaces. I have not had a chance to try that with redbird yet, but have done that successfully with other things, including Express.

No. I tried with "<IP_ADDRESS>", but it does not work


Implement aniseed.core/str akin to Clojure str

This implementation borrows entirely from the existing
aniseed.core/println function, differing only in that (1) it does not
put spaces into the final string between its arguments and (2) it
returns the string rather than printing it.
I recognize that a str function is already provided in a different
module. I believe the presence of this aniseed.core/str function will
not cause confusion and will ease the learning curve for Clojure
developers coming to Fennel/Lua, who may not be familiar with Lua's
string concatenation syntax exposed by Fennel and who might be surprised
by certain data types causing errors when passed to .. rather than
providing default string representations.

Further thoughts:

I appreciate your open source contributions and your time in reading this PR.
I'm happy to include the compiled lua/aniseed/core.lua if desired, I wasn't sure if anything environment-specific on my end might cause issues with the output Lua code (I was made further suspicious by lua/aniseed/deps/fennel.lua seeing changes as part of my work).
I'm happy to add/remove/change the tests for this function, I recognize there are many cases one could cover.

Oh nice! I like this, thank you very much 😄 people may assume a str function exists since aniseed.core is a spiritual copy of clojure.core, so I like the sound of including things you'd assume exist. I'll merge this and add a little doc string but the rest of it is fine.
And I agree, not including the compiled Lua is probably best practice since I work on stuff that massively changes the output quite regularly. So we'd just run into silly false positive conflicts all the time anyway, just having a .fnl diff is much easier to merge.
Thanks again!


Remove demand for ConEmu

The current Windows 10 console is able to display powerline fonts. I just grabbed the DejaVu Sans Mono for Powerline font, installed, changed it on the console properties and it worked just fine.

Not every font works, even in the nerd-fonts repo. Some glyphs are not available, and some fonts don't show up on the console options menu. Actually, the only font I got to show the glyph correctly and powerline and show on options was DejaVu, but this is probably just a matter of patching the fonts correctly. The Meslo font you have on your readme does not show on console options after instalation, I guess because it is marked as a PostScript Outlines font:

Whereas dejavu shows as true type outlines:

But I am not sure if this is it.
I had to add the ConEmuANSI env var to fool oh-my-posh to believe it was in Conemu. Everything is working just fine. You could just add the instructions on how to do it, or maybe demand another env var to make sure that this is what the user wants.
Here is the end result with poweshell directly and Conemu:

The only strange thing so far is the yellow color which is showing up white. I will try to understand what is going on and post here later.

It would be handy if the console would have a simple env var that indicates ANSI support. That way I can easily allow the updated clients instead of opening the door to potential issues.

This would be nice, but adding $env:ConEmuANSI = $True to the top of your profile is a good workaround for now.


docs: add a landing page to the documentation

A pydata-sphinx-theme based documentation needs a landing page.
features

A centered 2 column landing hero container
left is a bit of text + title + 3 buttons (leading nowhere at the moment)
right is a demo gif

technical addition

hero container required extra css rules
demo gif is now existing in both dark and light theme
hide the index
the hero container is mainly made of pure html to no break other potential builds (👋  .pdf)

The landing page looks much better than the site-map previously! Awesome the demo gif is now dark-mode enabled!
The buttons/links don't seem to work, but I don't know if this PR is ready yet.

the button are pointing to pages that are not yet existing so I send them to "#", the PR is ready to merge !

Ok, let's wait for those pages to be available, otherwise we'll have broken links on the doc-site when we need to release from master.

require #278 to move forward

What is we removed the dead links for now, then we can merge the landing page. We'll add back the links in PR's that have the actual content?

@mariobuikhuizen I left the code of the buttons but made them disapear. In this configuration, wiring them once the other pages are ready will be super easy and we don't show to users dead links.


Npm install fails when creating new project

Package version

4.0.12
Node.js and npm version

nodejs: v10.15.3
npm: 6.4.1
Sample Code (to reproduce the issue)

adonis new yardisk

Would you mind cd into the created directory and run npm install manually?

@thetutlage works perfectly, Thank You
Is it not in that directory before trying to perform npm install?

It does virtually CD into the directory before running npm install. However, npm install can fail coz of many reasons. Unless, this is a widely faced issue, I will let it go.
In newer versions, we will add a verbose mode, which will show the exact stack trace from any errors faced during the install

I will try to debug from cli package if i can produce the error and view the exact stack trace.

@thetutlage I am also facing the same issue. Would you mind taking a look into this.

It doesn't block you from using Adonis since you can CD into the directory and run the installation yourself.
We will check that after 5.0 release.


Syntax error in SQL statement

Hi,
I got DB schema as on the ERD below:

It's just relevant part of much bigger project which I can't share so please see the important bits below.
I try to translate following working (on H2) SQL query:
SELECT e., n., nd.*  ESCAPE '!' OR LOWER(DEFAULTNOT4_.NOTE2) LIKE ? ESCAPE '!' "; SQL statement:
select count(event0_.id) as col_0_0_ from event event0_ left outer join event_notes notes1_ on event0_.id=notes1_.entity_id left outer join note note2_ on notes1_.note_id=note2_.id left outer join note note3_ on left outer join (note_default defaultnot4_ inner join note defaultnot4_1_ on defaultnot4_.id=defaultnot4_1_.id) on where lower(note3_.note) like ? escape '!' or lower(defaultnot4_.note2) like ? escape '!' [42000-197]
at org.h2.message.DbException.getJdbcSQLException(DbException.java:357) ~[h2-1.4.197.jar:1.4.197]
I've narrowed down the QueryDSL query to following minimal working query:
QueryResults results = from(QEvent.event)
.leftJoin(QEvent.event.notes)
.fetchResults();
which generates SQL:
select count(event0_.id) as col_0_0_
from event event0_
left outer join event_notes notes1_ on event0_.id = notes1_.entity_id
left outer join note note2_ on notes1_.note_id = note2_.id
and following minimal not working query:
QueryResults results = from(QEvent.event)
.leftJoin(QEvent.event.notes)
.leftJoin(QNote.note1)
.fetchResults();
failing with:
org.h2.jdbc.JdbcSQLException: Syntax error in SQL statement
"SELECT COUNT(EVENT0_.ID) AS COL_0_0_ FROM EVENT EVENT0_ LEFT OUTER JOIN EVENT_NOTES NOTES1_ ON EVENT0_.ID=NOTES1_.ENTITY_ID LEFT OUTER JOIN NOTE NOTE2_ ON NOTES1_.NOTE_ID=NOTE2_.ID LEFT OUTER JOIN NOTE NOTE3_ ON [*]"; expected "NOT, EXISTS, INTERSECTS"; SQL statement:
select count(event0_.id) as col_0_0_ from event event0_ left outer join event_notes notes1_ on event0_.id=notes1_.entity_id left outer join note note2_ on notes1_.note_id=note2_.id left outer join note note3_ on [42001-197]
at org.h2.message.DbException.getJdbcSQLException(DbException.java:357) ~[h2-1.4.197.jar:1.4.197]
@Entity
@Inheritance(strategy=InheritanceType.JOINED)
public abstract class Event extends AnnotatedAuditedEntity {...}
@MappedSuperclass
public class AnnotatedAuditedEntity extends AuditedEntity {
@ManyToMany(cascade = CascadeType.ALL)
@MapKey(name = "key")
@JoinTable(
joinColumns=@JoinColumn(name="entity_id", referencedColumnName="id"),
inverseJoinColumns=@JoinColumn(name="note_id", referencedColumnName="id"))
Map<String, Note> notes = newHashMap();
}
@Entity
@Inheritance(strategy = InheritanceType.JOINED)
public class Note {
@Column(length = SHORT_MAX_LENGTH)
private String note;
}
@Entity
public class DefaultNote extends Note {
@Column(length = DEFAULT_MAX_LENGTH)
private String note2;
}
What's wrong?

You're mixing up entity joins and collection joins. As per JPA 2.1 specification, entity joins require an ON clause, even if its just 1=1.
However, the join you're creating is unnecessary with JPA. When InheritanceType.JOINED is used for Note, any association join to Note will automatically join DefaultNote so that DefaultNotes are returned as DefaultNote and not as Note instance.
So just
QueryResults results = from(QEvent.event)
.leftJoin(QEvent.event.notes)
.fetchResults();

will actually generate the join you want...


Alerting devices page not using location slug

Open the list of alerting devices from the dashboard.
Click the device link.
You'll notice the slug is missing. Update link accordingly.

Duplicate of issue #241


[v4.0.0] - Massive Simplification and Full Open Telemetry Integration

When I first started this project, visibility with consumers was my main contention at work. Prometheus was the primary acceptable solution in 2017/2018 other than APMs.
The landscape has vastly changed. Some of my designs are hard to work around being flexible using the IMetricsProvider. For the last couple of versions I have gotten OpenTelemetry to start working inside the contraints of IMetricsProvider but its messy bulky and I don't like it.
OpenTelemetry is an accepted standard now. I am switching to this through out the HouseofCat.RabbitMQ library because that's what I was working around in the past, there wasn't a standard.
I apologize to anyone who is still using Prometheus. Please stay on 3.2.x. It's very stable as is but I won't be supporting that core library using non-OpenTelemetry as there should not be a need to interchange with flavors of metrics going forward. In fact, Prometheus and OpenTelemetry are compatible with a OpenTelemetry to Prometheus exporter.
Issues Need Solving

Using Trace/Spans for Workflows is a must.
Publishing needs to be incorporated into OpenTelemetry.
a. Need to incorporate proper Activity tagging and Activity Kind Producer.
Consumer needs to be incorporated into OpenTelemetry.
a. Need to incorporate proper Activity tagging and Activity Kind Consumer.
Topologer needs to be incorporated into OpenTelemetry.
a. Need to incorporate proper Activity tagging and Activity Kind Internal.
There needs to be ways for Developers to extend, alter, create their own spans in Workflows. A lot will be auto-wired but I will not strangle engineers to integrate how they want to or even go above and beyond what I propose.

Elephant in the Room
There are some uncomfortable contract changes I will most likely make in order to "align" with not just OpenTelemetry labeling but perhaps how we wrap payloads. The end result should be a much more pleasant and streamlined experience no matter what I choose to do. Continuing the initiative to roll out guides and examples.
Changes So Far

Letter is being renamed Message.

Feedback I got was many people don't like it.
MessageId generation is automatic for convenience.

LetterMetadata is being renamed Metadata.
Metadata.Id renamed to Metadata.PayloadId to help indicate it's purpose with a comment indicating it's the identifier for your inner payload data.
Headers have changed to be of type X-RD-* prefix.
OpenTelemetry header traceparent is now baked into RabbitMQ headers.
RoutingOptions is removed. They are now properties on Message.
Envelope is removed. Properties on Message.
BuildState has no auto-deserialization/statekey required.
there is data and the ContentType now matches.
All GlobalConsumerOptions / PipelineOptions are all gone.

It's now just ConsumerOptions and properties associated for how the consumer is used inside your application.

Ability to disable Queue creation on startup for Consumers.
All generics have reverted back to simplification of property and fields. Will make mutating/extending different but understand the trade off is simplification.
ReceivedData is now ReceivedMessage.
ReceivedMessage now will only auto deserialize ReceivedMessage.Message if it's ObjectType is IMessage.

The ContentType is application/json or application/msgpack (all read from the headers).
The data content length must also be greater than 0.
Should they fail to deserialize the flag FailedToDeserialize will be marked as true but will continue the flow of execution without throwing an exception.

References to data byte[] now align with body byte[] which the RabbitMQ.Client uses.
Cutting back on overlocking with SemaphoreSlim.

Removed from ChannelHost/ConnectionHost on using the Channel/Connection.

IModel Channel is now directly accessible from IChannelHost in stead of a function.

Assignments are controlled by the Host objects themselves, so how they mutate is naturally way more single threaded. Especially with the AutoRecovery changes.

IMetricsProvider is gone. OpenTelemetry is the path forward.
ConsumerWorkflow will auto generate a RootSpan or ChildActiveSpan based on Telemetry traceparent header provided with your messages.
ConsumerWorkflow deserialization/BuildState step no longer deserializes the inner payload. This means that you can put whatever you like in Workflows and use your first custom step to manage what to do with ReceivedMessage.Data as you see fit.
All or nearly 100% of all byte[] are now ReadOnlyMemory<byte>.

#54


rustup: work around rustc optimizations becoming too smart

@bors r+

:pushpin: Commit 210f18d6c7962ae5125947e89d38cac34c65b6a8 has been approved by RalfJung

:hourglass: Testing commit 210f18d6c7962ae5125947e89d38cac34c65b6a8 with merge 2f7af4b230b7a27afebcf51e3bfce31e77816722...

@bors retry r+

:pushpin: Commit 3142ddcd8d808c1d3c2324d54a92fd60182ffbc9 has been approved by RalfJung

:hourglass: Testing commit 3142ddcd8d808c1d3c2324d54a92fd60182ffbc9 with merge 850141b6847f8406e0ba2b4685ab05d4ceb25184...

@bors retry r+

:pushpin: Commit d9bc19a7b078e75052167496fff3c4d91c7bbd66 has been approved by RalfJung

:hourglass: Testing commit d9bc19a7b078e75052167496fff3c4d91c7bbd66 with merge cfc114d942ddda3ce125410e83bd27dc21adeb58...

:broken_heart: Test failed - status-appveyor

@bors r+

:pushpin: Commit 07034cb7e33f9f994639aae4a261c51732e012b7 has been approved by RalfJung

:hourglass: Testing commit 07034cb7e33f9f994639aae4a261c51732e012b7 with merge f3102aed2ef9414660eca50855dfdd7864d61937...

@bors retry r+

:pushpin: Commit 3ba1035d279e0acfc3aaf68e8bd055f2d1ffb205 has been approved by RalfJung

:hourglass: Testing commit 3ba1035d279e0acfc3aaf68e8bd055f2d1ffb205 with merge bb9db2bf8f5562cb90d3643f213972c838c25e6d...

:sunny: Test successful - checks-travis, status-appveyor
Approved by: RalfJung
Pushing bb9db2bf8f5562cb90d3643f213972c838c25e6d to master...


Fix bad package references

Both mamba and quetz from PyPI are unrelated!

mamba: the definitive test runner for Python
quetz: A Python library for compositional programming.

Starting over in #471


Adding a way for make visible Tag with different api-docs URL.

Adding a way for make visible Tag with different api-docs URL.
A suggestion for #100
By example:

Routing.RouteTable.Routes.Add(new ServiceRoute("api-docs", new WebServiceHostFactory(), typeof(SwaggerWcfEndpoint)));
Routing.RouteTable.Routes.Add(new ServiceRoute("api-docs2", new WebServiceHostFactory(), typeof(SwaggerWcfEndpoint)));
SwaggerWcfEndpoint.Add_VisibleTag("api-docs", "Service_1");
SwaggerWcfEndpoint.Add_VisibleTag("api-docs2", "Service_2");

I'm not sure I'm doing it by the best way, but it's working.
Not sure about your standard of naming function and vars, hope it's fine.

why such a complex solution?
Can't you just work out with this?

            if (ConfigurationManager.AppSettings["api-docs_visible"] == "true")
                RouteTable.Routes.Add(new ServiceRoute("api-docs", new WebServiceHostFactory(), typeof(SwaggerWcfEndpoint)));
            if (ConfigurationManager.AppSettings["api-docs2_visible"] == "true")
                RouteTable.Routes.Add(new ServiceRoute("api-docs2", new WebServiceHostFactory(), typeof(SwaggerWcfEndpoint)));

  <appSettings>
    <add key="api-docs_visible" value="false"/>
    <add key="api-docs2_visible" value="true"/>
  </appSettings>

assuming you want to display/hide and endpoint...

no, the goal here is to display different tags depending of the endpoint of the docs.
I put all my function/sub with SwaggerWcfHidden. And I make them visible depending of the url, by example, if I have service1.svc with function1 and function2, with service2.svc with function3 and function4, I can do something like http://URL/service1.svc/api-docs and it will put the swagguer only for function1 and function2.
if I do http://URL/service2.svc/api-docs that will show only function3 and 4

not using the PR you submitted, but I'm applying an update in a few moments that will support this with another approach, look into Global.asax.cs to check it after the updates

thank :) I'll check it

I tried, but maybe I don't understand something, when I do it the first time it seem to work, but if I go on my second api-docs it still show the tags of the first one.
`public void Application_Start(object sender, EventArgs e)
{
SwaggerWcfEndpoint.FilterVisibleTags = FilterVisibleTags;
SwaggerWcfEndpoint.FilterHiddenTags = FilterHiddenTags;
SwaggerWcfEndpoint.DisableSwaggerUI = false;
Routing.RouteTable.Routes.Add(new ServiceRoute("Service_Repartition/api-docs", new WebServiceHostFactory(), typeof(SwaggerWcfEndpoint)));
Routing.RouteTable.Routes.Add(new ServiceRoute("Service_TA/api-docs", new WebServiceHostFactory(), typeof(SwaggerWcfEndpoint)));

Routing.RouteTable.Routes.Add(new ServiceRoute("Service_Repartition/", new WebServiceHostFactory(), typeof(Webservice_Repartition.Service_TA_REST)));
Routing.RouteTable.Routes.Add(new ServiceRoute("Service_TA/", new WebServiceHostFactory(), typeof(Webservice_Repartition.Service_TA_REST)));

Routing.RouteTable.Routes.Add(new ServiceRoute("Webservice_Repartition/Service_Telephonique.svc", new WebServiceHostFactory(), typeof(Webservice_Repartition.Service_Telephonique)));

}
private static List FilterVisibleTags(string[] path, List visibleTags)
{
if (path.Contains("/Service_Repartition/api-docs")) {
	visibleTags.Add("Service_REST");
	visibleTags.Add("Service_REST_Callback");
}

if (path.Contains("Service_TA/api-docs")) {
	visibleTags.Add("Service_TA_REST");
}

return visibleTags;

}`

You do not add tags, you remove them from the list
It’s a filter

Abel Silva
<EMAIL_ADDRESS>

Okay, I just did the remove, but I got the same problem, I jsut did a debug and it seem function don't get called another time when I go on the second api-docs.

Ah... I see...
I think I know what the issue is, I’ll fix it later today
I’m preventing regenerating the swagger json multiple times
I’ll probably change the signature of the filters from string[] paths to
string path
--

Abel Silva
<EMAIL_ADDRESS>

Okay, thank :)

check if latest version fixed this please
also, in the filter function you can add tags as you tried first (you are not limited to removing)

Seem to work, thank :)
But I think the global is not updated, it's still a list of string.

not sure what you meant
The functions are updated:
https://github.com/abelsilva/swaggerwcf/blob/46d6367c174e43e7298c181e735c175c430a6d9e/src/SwaggerWcf.Test.Service/Global.asax.cs#L32
https://github.com/abelsilva/swaggerwcf/blob/46d6367c174e43e7298c181e735c175c430a6d9e/src/SwaggerWcf.Test.Service/Global.asax.cs#L37
The change was for the path parameter, previously it was string[] paths, now it's string path

Forget it, I think I got the cache from my browser :)


Logging levels

Maybe logging levels would work, if there was a rule that a message/action could have a logging level no more important than its parent?

Perhaps with better levels? http://labs.ig.com/logging-level-wrong-abstraction

To what end?  "Would work" for what?

Rewrote description.

or rather than levels, add some kind of marker and configure which markers are logged?
Not an expert here, just trying to think of my situation where I've been using the Python standard logging module + find it frustrating sometimes that I want to look at some specific INFO messages but not others

There's a bunch of ways you can filter already: it's structured data after all. So you can filter by action_type, for example and only look at the specific actions you care about. E.g. if you're using eliot-tree there's filtering of top-level tasks using JMES queries (http://jmespath.org/).
You already have the option of adding a logging_level field of your own choosing (I recommend this methodology, even if not these particular levels): https://labs.ig.com/logging-level-wrong-abstraction
So "I want to mark messages and I want to filter messages" is already there. But since that's not obvious this suggests the need for some documentation on how to do this.
The main use case as I see for this particular issue is the first one: disabling debug/expensive log messages in production.


fix(client): forum help and hints

Closes #36323
Closes #18005

Next things:

[ ] Add tests please.
[ ] Consolidate the helper functions and reduce repetition.
[ ] Get the forum base URL https://www.freecodecamp.org/forum from env and construct further URL bits as needed.

@raisedadead I believe I have taken care of the following:

Consolidate the helper functions and reduce repetition.
Get the forum base URL https://www.freecodecamp.org/forum from env and construct further URL bits as needed.

Now, I will work on some tests.

While you are at this, can you clean up this file:
client/src/templates/Challenges/redux/create-question-epic.js
      const studentCode = window.encodeURIComponent(textMessage);
      const altStudentCode = window.encodeURIComponent(altTextMessage);

      const defaultURI = `https://www.freecodecamp.org/forum/new-topic?category=${category}
                          &title=&body=${studentCode}`;
      const altURI = `https://www.freecodecamp.org/forum/new-topic?category=${category}
      &title=&body=${altStudentCode}`;

      if (defaultURI.length < 8000) {
        window.open(defaultURI, '_blank');
      } else {
        window.open(altURI, '_blank');

This can be simplified. Also the text block can be wrapped in backtick instead of the ugly single quotes.

OK.

Is this ready for review? If yes, can you update the label.

I think Randell was planning to add a few tests for the React components.


Does dependabot support updating transitive dependencies in lockfiles?

I couldn't find this on the features page.

We'll automatically update a dependency's sub-dependencies whenever we update the main dependency, but we don't create PRs if it's just the sub-dependencies that need updating.
In future, I'd like to create updates for sub-dependencies if there are security alerts for them (see this issue). I'm waiting on GitHub for an API for their security alerts in order to do so.

Thanks for the reply. That's too bad, would you take this as a feature request? Take this dependency for example:

editor depends on language service

language server depends on parser

If there was a bug fix in the parser, that directly impacts the editor, but it won't cause a new release in the language server. Because the editor is bundling all its dependencies, we need a PR to update the lockfile of the transitive parser dependency.

Happy to think on it. There are a couple of open issues around this (https://github.com/dependabot/feedback/issues/22 is the most relevant), so it's already on my radar - I'll update you here if/when I get to it.


MapLookupOptions cannot be instantiated in PyArrow

Describe the bug, including details regarding any error messages, version, and platform.
When I try to instantiate MapLookupOptions, I get an error.
The same happens when I call pc.map_lookup with those arguments.
In [1]: import pyarrow as pa
pa.
In [2]: pa.__version__
Out[2]: '15.0.0'

In [3]: import pyarrow.compute as pc

In [4]: pc.MapLookupOptions(query_key='orderEntryTime', occurence='first')
---------------------------------------------------------------------------
TypeError                                 Traceback (most recent call last)
Cell In[5], line 1
----> 1 pc.MapLookupOptions(query_key='orderEntryTime', occurence='first')

File ~/venv/lib/python3.11/site-packages/pyarrow/_compute.pyx:1595, in pyarrow._compute.MapLookupOptions.__init__()

TypeError: __init__() takes exactly 3 positional arguments (2 given)

Component(s)
Python

the same happens with query_key=pa.scalar('orderEntryTime')

Sorry, It is a typo occurence -> occurrence

Just a typo


Add support for mocking properties

Details discussed in pull request #69 which attempted a partial implementation.

I copied this issue to the new official repo: https://github.com/bkabrda/flexmock/issues/2 closing this one


chore: bump version to 0.9.2

Waiting for a new release of https://github.com/containerd/runwasi/pull/370

Waiting for spin 2.0 release next week

I will merge this in now. When Spin release 2.0, we will release a new patch version.


Fix panic

Related to the issue #119 and PR #127
Currently panics are fixed, but the emitted IRs are still incomplete.
TODOs are marked as TODO, FIXME.
Followings are the TODOs

Implement unimplmented functions (e.g., RadecoFunction#locals, RadecoFunction#call_sites)
Fix callfixer.rounded_analysis
Produce correct SSA so as to pass verifier::verfy
In WrappedR2Api, use generic type  instead of R2 of Rc<RefCell>

I would like to deal with above TODOs as other issues.

@kriw This looks good to me. Do you intend to do the above TODOs on a separate PR so that this can be merged?

Just don't forget about those, since there are many TODOs and a bunch of disabled code.

I was plannning to make issues and solve them as soon as possible.
Should I fix them on this PR or on separate PR?

Can you please also rebase the PR?

@kriw If this is ready for merge, please let me know. I'll review it once more and merge it in.

@sushant94 This is ready for merge.
Could you review it?
WrappedR2Api is not fixed now, I will work on it as another issue.

SGTM, Merging. Thanks! @kriw: Open an issue to track WrappedR2API please.


Don't crash if the linked user is not in the admin

Related to issue #99, for the case when the User model is not enabled in admin.

Codecov Report

:exclamation: No coverage uploaded for pull request base (master@ac8638d). Click here to learn what that means.
The diff coverage is 0%.

@@            Coverage Diff            @@
##             master     #134   +/-   ##
=========================================
  Coverage          ?   65.52%
=========================================
  Files             ?       18
  Lines             ?      438
  Branches          ?        0
=========================================
  Hits              ?      287
  Misses            ?      151
  Partials          ?        0

Impacted Files
Coverage Δ

src/auditlog/mixins.py
0% <0%> (ø)

Continue to review full report at Codecov.

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


Feature request: set default alias when installing?

Bottom of my .bash_profile
export NVM_DIR="/Users/kevin/.nvm"
[ -s "$NVM_DIR/nvm.sh" ] && . "$NVM_DIR/nvm.sh"  # This loads nvm

echo ".bash_profile sourced"

Open fresh terminal
Last login: Sat Apr  4 06:08:55 on ttys001
.bash_profile sourced
kevin@walnut ~ $ ember -v
-bash: ember: command not found
kevin@walnut ~ $ nvm install stable
v0.12.2 is already installed.
Now using node v0.12.2
kevin@walnut ~ $ ember -v
version: 0.2.2
node: 0.12.2
npm: 2.7.4

Ok I found it.  "installing a node" makes it current, but current doesn't stick.
Even this doesn't make it stick
 nvm use v0.12.2

If the above command only works for the existing shell then it works perfectly :/

That's absolutely the way it's supposed to work.
If you want every shell to start out with nvm use already ran, you can do nvm alias default node for example, or you can make a .nvmrc file that has "node" or "0.12" or whatever in it.

since nvm is somewhat influenced by rvm and rvm is familiar to many adopters of nvm maybe it would spare a lot of users from frustration if nvm, like rvm, set the first node installed as the default.
installing nvm and a single node and having nothing as default is a baffling experience IMO

That's a very fair point. If I made nvm install always set the default when none was set, then people who did not want that behavior would have to do nvm install X && nvm unalias default, or, I'd have to build something like nvm install X --no-default, neither of which are a huge cost.
I'll keep thinking about it.

I could be wrong but I can't think of a single situation where someone would not have default set.  I would look to how rvm handles this, since most users will be expecting a similar behavior.  However, there's a difference, rvm lets you set the "system ruby", which I suppose is analogous to nvm having no default.
Hold on, I can think of one situation where a user wouldn't want a default set, if they are using the "system node", in which case you should follow rvms  and let the system node get set as the default.
I think I blindly stumbled onto to the fact that installing nvm somehow, without setting up the default, still does something, as in
kevin@walnut ~ $ nvm install stable
v0.12.2 is already installed.
Now using node v0.12.2

Why not make this behavior not only set up "now using node ...", but set the default as well (but only if nothing is set already)  Who the heck is going to have no default set, install a new node, and get upset about it flipping the default switch on that new node?  And if you want to get real rigorous, set a switch where this only happens on the first first node install, it will save you a lot of "custom support" as this software surely gets more popular.

Users will only be expecting a similar behavior if they've used rvm - i used rbenv and have no expectations based on rvm.
The system node is in fact already the default, since that's what's used when nvm is deactivated (which is how it starts when there's no default set).
nvm is typically used to test things across multiple node versions, not as the primary method of managing node (which is also why it doesn't touch the system node in any way). I prefer not to have a default set specifically so that I don't get any false positives - ie, I always explicitly nvm use the node version i want.
Your point is certainly valid, however, that the potential inconvenience from me having to unset the default is much less than the potential confusion of not understanding why there's no active node version.

Ether way is fine, but just a heads up then, I was sent over here from a blog posting which was referenced in a book, looks like some people want to use nvm like rvm, which makes them close cousins when ember.js is used in the front end and rails in the backend.
Here's the book and blog posting if you  aren't already familiar with them
https://leanpub.com/ember-cli-101
http://www.wenincode.com/installing-node-jsnpm-without-sudo/

Done in 3d764e6833bb495288e322cf33a6def7e283f920.


Fix migration issues

I will add here tasks as I go through the website and check it out

[ ] images are not showing
[x] Can't register because the captcha field is not working:

[x] Can't log in with registered user. I tried using an active user in the database and logging in, and I got sent back to the login page:

i couldn't reproduce login bug

[ ] Dataset upload screen is not working

http://<IP_ADDRESS>/datasets/create/
Uploading a tsv file results in a 500 error. Here is a TSV file that works on the original WHG website.
kima-as-in-WHGJuly2021.zip

[ ] registration validation email contains the wrong link and is sent from example.com

[ ]  Dataset upload screen is not working

http://<IP_ADDRESS>/datasets/create/
Uploading a tsv file results in a 500 error. Here is a TSV file that works on the original WHG website.
kima-as-in-WHGJuly2021.zip

looks like the error is coming from celery, celery is not running

What does that mean?

What does that mean?

i clone the project locally and fixing it now

When logging in from tool.mehdie.org the screen does not respond and returns to the default empty screen.

[ ] Fix login bug
[ ] Allow login from https as well

[ ] fix: contact-us page is not working

[ ] Fix: about page is not working:


Fix container alignment, add scrollbar and magic icons to the 2005 ga…

Usage
This will align the container of the different tabs to fit better with the 2005 gameframe. It will change the scrollbar to match. Lastly it will add some subtle icons for the different spellbooks.
Credits to @Kamielvf.
Example

Related
Fix #3007

Reference to how it looks currently:

Also fixed the 2010 logout button overlay.


Use sodium compat as sodium extension throws error on curve check

As discussed on #31

Thanks @gabrielkoerich, will check this out and test it on a few machines.

@neverything any news?

@neverything when can we have this merged @gabrielkoerich great job

Sorry @gabrielkoerich for the delay. Was sick for almost 3 weeks. Will get to this later this week.

Awesome! Hope you're feeling better. Thanks!

Thanks @gabrielkoerich, yes, finally feeling better again 🙌


Error bumping in CI/CD

I use pdm bump in a gitlab pipeline and a new issue has arisen when running pdm bump.
[AttributeError]: '_PdmPep517StorageBackend' object has no attribute '_DynamicStorageBackend__config'

I have updated pdm to the latest version along with the lock file and the issue still persists.

Hi, can you try version 0.9.8?

That resolved it, thanks @carstencodes for the quick turnaround! :smile:

You're Welcome.


refactor(arbitrary): rename toArbitrary to arbitrary

Current dependencies on/for this PR:

main

PR #48

PR #49

PR #50   👈

PR #51

This comment was auto-generated by Graphite.

Graphite Merge Job
Current status: ⏳ Queued to merge
This pull request is currently queued to be merged as part of a stack.
This comment was auto-generated by Graphite.

Job Reference: G9qW3Gh7LcbjHSuwHmyz

Graphite Merge Job
Current status: ⏳ Queued to merge
This pull request is currently queued to be merged as part of a stack.
This comment was auto-generated by Graphite.

Job Reference: cDSyTs4rRKHy1gaGhtoF


Mature Air in WAILA

Log gets lots of "missing translation" errors for Epicurean blocks/items, even though all the translations are present in English. Also an issue:

Interesting to note that onions are the only crop that are a direct HarvestableCropBlock and not a PickableCropBlock. It shouldn't have any effect, but it's happening for some reason.

The missing translation issue is now completely fixed.


Just link to top of repo

https://github.com/hubmapconsortium/portal-docs/blob/master/software.md
These links are all unnecessarily fragile: default branch may change, and there's just more room for typos.
Replace links like: https://github.com/hubmapconsortium/gateway/blob/test-release/README.md
with: https://github.com/hubmapconsortium/gateway#readme

On July 7, @mccalluc Slacked this note:
"The default branch for several repos has been changed from master to main so this should be addressed."
I'll leave these changes to Melissa @schwenk102 .

@mccalluc Please clarify which documents require these changes?

@cebriggs7135 : https://github.com/hubmapconsortium/portal-docs/blob/master/software.md is a place to start

@schwenk102 - would you please respond here or turf to JH within Pitt-DBMI?

@JAH386 - heads up re: this discussion. Would you please work with @schwenk102 to resolve?


OSD-26057: Better logging for assumeRole errors

We were flooding logs with "Unhandled AWS error"
I made it so only the first, 25th, 50th, 75th, and 100th retry is logged to the standard log level. The rest are logged at a higher level. The verbiage is different, but it's essentially the difference between info and debug logs.
This is tracked in OSD-26057

/lgtm

/lgtm


Update readme to include  section on configuring for testing with jest

Update docs to have testing section
Add section on updating config to work with jest, with three methods based to allow for users individual setups

Thanks @AntonyM71 !


Add subject age and sex to convert_session_to_nwb

Should fix #8.
Would this work @CodyCBakerPhD ?

One minor improvement suggested as comments; this was being used as an educational tool for @pauladkisson and I just noticed the missing DANDI requirements while showing it to them

Thanks, the examples are useful I feel. If yo uare fine with this could you aprove it?


Sharding vortex

Enable autosharding for BQ:   #22818
Thank you for your contribution! Follow this checklist to help us incorporate your contribution quickly and easily:

[ ] Choose reviewer(s) and mention them in a comment (R: @username).
[ ] Format the pull request title like [BEAM-XXX] Fixes bug in ApproximateQuantiles, where you replace BEAM-XXX with the appropriate JIRA issue, if applicable. This will automatically link the pull request to the issue.
[ ] Update CHANGES.md with noteworthy changes.
[ ] If this contribution is large, please file an Apache Individual Contributor License Agreement.

See the Contributor Guide for more tips on how to make review process smoother.

GitHub Actions Tests Status (on master branch)

See CI.md for more information about GitHub Actions CI.

Run Java PreCommit

R: @reuvenlax

I triggered the tests. I am not sure what is the next step after that.
@Naireen - it would be good to add a jira in the issue title with some explanation of what you are trying to do.

LGTM, however please attach a JIRA to the title

Run Java PreCommit

run java precommit

LGTM

Run Spotless PreCommit

Run Java PreCommit


After form submission - UX (AFSUX)

from #1118:
instead of blocking the user from proceeding with more submissions / transactions ...

- clear form for new tx's
- show loading icon in sidebar next to transactions link
- include the loading tx in tx history instantly with loading icon and adjust language accordingly
- if it fails, replace loading icons with error icons and replace loading language with error language
- on failure, show button to retry transaction

designs for this more optimistic approach can be seen here: https://zpl.io/bLDW6gQ
Originally posted by @jbibla in https://github.com/cosmos/voyager/issues/1092#issuecomment-418142498

related #2295


I have a question

Is this supposed to claim any usernames on ig? like for example any banned usernames?

Currently, the tool can only claim usernames from active accounts. So if the URL instagram.com/[username]  works, the username can be claimed by the script.
I've also figured out a way to check for the availability of usernames of banned accounts and other accounts. But I've not added it to the script as I've been busy.

My proxies show failed even tho i just purchased them example : <IP_ADDRESS>:3128
<IP_ADDRESS>:3128

Can you tell me the type of proxy you're using?

I use HTTP Proxies

From my testing, I've found that HTTP proxies don't work for Instagram. I've had issues with HTTPS proxies too and only SOCKS5 seem to be working.

Which websites do you recommend for proxies ? i bought from proxyscrape

I personally use residential proxies from Smartproxy. There might be other websites with cheaper options, but I haven't tried other sites yet.

I just bought them you think i should wait how much time for the user to be claimed?

I'm not sure what type of username you're trying to claim. Can you provide me more details?
You can dm me on instagram qxza


🐛 Make IPs match diagram

Fix mismatched IP config

Pull Request Test Coverage Report for Build 07d2dbcb2b251cc1be225310b851b22f1fc52bcf-PR-1072

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 65.977%

Totals

Change from base Build 5ad4c8ccd556cd5782bf4d3fc0b4b6073562245a:
0.0%

Covered Lines:
1148

Relevant Lines:
1740

💛  - Coveralls


Move test fixtures from priv/test to test/fixtures

We usually don't use the priv directory for test fixture files, since it generally ships with the application. test/fixtures is a common location for fixture files.

Pull Request Test Coverage Report for Build dd806564513846793110893c6c95cb50ffc87ba9-PR-72

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 100.0%

Totals

Change from base Build d3b6377edf24884f290efcaae1f5e346c39e3555:
0.0%

Covered Lines:
365

Relevant Lines:
365

💛  - Coveralls

Pull Request Test Coverage Report for Build dd806564513846793110893c6c95cb50ffc87ba9-PR-72
Warning: This coverage report may be inaccurate.
This pull request's base commit is no longer the HEAD commit of its target branch. This means it includes changes from outside the original pull request, including, potentially, unrelated coverage changes.

For more information on this, see Tracking coverage changes with pull request builds.
To avoid this issue with future PRs, see these Recommended CI Configurations.
For a quick fix, rebase this PR at GitHub. Your next report should be accurate.

Details

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 100.0%

Totals

Change from base Build d3b6377edf24884f290efcaae1f5e346c39e3555:
0.0%

Covered Lines:
365

Relevant Lines:
365

💛  - Coveralls

Pull Request Test Coverage Report for Build dd806564513846793110893c6c95cb50ffc87ba9-PR-72
Warning: This coverage report may be inaccurate.
This pull request's base commit is no longer the HEAD commit of its target branch. This means it includes changes from outside the original pull request, including, potentially, unrelated coverage changes.

For more information on this, see Tracking coverage changes with pull request builds.
To avoid this issue with future PRs, see these Recommended CI Configurations.
For a quick fix, rebase this PR at GitHub. Your next report should be accurate.

Details

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 100.0%

Totals

Change from base Build d3b6377edf24884f290efcaae1f5e346c39e3555:
0.0%

Covered Lines:
365

Relevant Lines:
365

💛  - Coveralls


Rename counts to raw counts

And on this line, change property to "Expression Unit"
Remove formatting of the font in the component.


Patch polymorphic association

Changes

Use reflection.try(:foreign_key) instead of reflection.options[:foreign_key]
Patch test for polymorphic associations
Restructure with ActiveSupport::Concern
Updated Ruby 1.9 Hash syntax

@cedric could you take a look at this PR, any feedback would be nice, thanks!

@abookyun Looks good, thanks!


Hardware: wanterproof'ish box

Maybe this box will do, not having to have one box for the equipment and one for connecting stuf

The 12 module, measuring: 225 x 200 x 110 mm.

https://www.biltema.no/bygg/elinstallasjoner/elbokser/koplingsboks-2000034324


[WFLY-13386] Hung process instances and associated server.log WARN "Failed to reinstate timer 'kie-server.kie-server.EJBTimerScheduler' "

If activate the refresh task in the database persistence there are at least
two threads compiting for registering a timer.
The refresh task is executing at the same time a new timer is being registered
if this happens the same timer will be try to registered by ejb and refresh task
this is due the fact the ejb action register in the database in its own tx (outside ejb tx)
the refresh task tries to load from its own tx
so if this order happens
1 ejb thread register the timer in the database
2 refresh task thread  load the timer from the dabase
3 ejb thread registers the timer in the dmr
4 refresh task thread register the timer in the dmr (FAIL as duplicated)

@chengfang do you mind to have a look. This one fix the nasty issue of the refresh task and duplicated resources. Also fixes the problem of null point while sync.


Update the height and width values in examples

The dynamic examples were using heights from the generated data instead of the measurement. That is ${rows[virtualRow.index]}px vs ${virtualRow.size}px.
The variable example was doing the opposite (using ${virtualRow.size}px) even though ref={virtualRow.measureRef} was never set in the variable example.
Also update the condition in infinite scroll (#161)

to try out the changes to the examples on this PR, codesandbox preview builds would be ideal, like the ones we have on react-query. I'm happy to PR the config changes, but we'd need @tannerlinsley to enable the bot as far as I remember, given that he's the owner :)

Updated infinite scroll example:  https://codesandbox.io/s/ecstatic-tharp-u3llm
Updated variable height example: https://codesandbox.io/s/fast-frog-tc4l8?file=/src/index.js
Updated dynamic height example: https://codesandbox.io/s/distracted-tdd-o7wh7?file=/src/index.js

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


[DOCS] Fix broken markdown in PROTOCOL.md

Which Delta project/connector is this regarding?

[ ] Spark
[ ] Standalone
[ ] Flink
[ ] Kernel
[X] Other (Documentation)

Description
Minor markdown fix.
How was this patch tested?
GitHub preview.

Does this PR introduce any user-facing changes?
No.

@watfordkcf can you provide a public (not github no-reply) email to sign the commit? you can write it here or update the commit author

Signed, amended, pushed.


Feature: pLM-BLAST in scikit-bio

Class ProteinEmbedding implements a data structure for the storage of protein embeddings. After it, the alignment of embeddings could be implemented for sequence analysis.

pLM-BLAST
The substitution matrix is constructed from residue-normalized embeddings for both proteins. This substitution matrix can be further used as an input for Needleman-Wunsch alignment or further transformed into a scoring matrix for Smith-Waterman local alignments, providing both local and global alignments in the output.

The implementation should also be compatible with other parts of scikit-bio. For example, the resulting substation matrices should be compatible with skbio.sequence.SubstitutionMatrix. Multiple local alignments from pLM-BLAST might be returned as TabularMSA object.

[ ] Create a class-method SubstitutionMatrix.from_embedding(emb1, emb2)
[ ] Implement global alignment method based on Substitution matrix, which returns skbio.alignment.AlignmentStructure object
[ ] Implement local alignment method based on Substitution matrix, which returns results in skbio.alignment.TabularMSA format

@mortonjt yes, I'm thinking about SubstitutionMatrix.from_embedding(emb1, emb2).
For now, alignment algorithms use dicts based on protein alphabet as substitution matrix. Here, we don't exactly have an opportunity to create a dict from the resulting substitution matrix, because it does not rely on residues.
I don't know if a rework of the aligner is possible, the code looks intricate to me.

Hi,

here is my fork for this issue: https://github.com/Argusmocny/scikit-bio/tree/plmblast
for the substitution-matrix the whole is actually 3 vectorized operations in numpy: normalize each embedding row and then matrix multiplication of emb1 x emb2.T . The code is here: https://github.com/labstructbioinf/pLM-BLAST/blob/438d45a1ac5fbd7a6363ea1c6b26bbb8b5f8a66e/alntools/numeric.py#L366 - the function is much longer because some array operations are not implemented in numba.
Maybe I can write another SubstitutionMatrix class/template which will work when substitution scores will be contextual eg. depending on position in sequence.

Hello @mortonjt @valentynbez @Argusmocny  Sorry for my late involvement in this valuable conversation.

The plan sounds very cool to me! I mean, all of it. Alignments with positional scoring. Substitution matrix from embedding. etc.
I agree that adding Numba support requires careful thinking. I personally had good experience with Numba. I want to note the situations where Numba will actually accelerate: slow Python looping. If your code is already vectorized (like the one @Argusmocny pointed to), Numba will make little difference. This is my guess, but @Argusmocny can benchmark it.
I have long been planning to implement the global and local pairwise alignment algorithms from scratch in scikit-bio. See my issue #1864 . The prototype code is already there. The plan is moving slowly amid many of my other responsibilities as a faculty member.

Here is the Substitution matrix and scoring matrix with SubstitutionMatrix.from_embedding(emb1, emb2) method https://github.com/Argusmocny/scikit-bio/blob/e39550d433bf5e4b5b939ab31f7a1d1d90134c6e/skbio/embedding/_substitution.py#L16
As far as I understand some of basic SubstitutionMatrix fuctionalities will not be compatible with positional based scores, so i moved it to skbio/embedding/_substitution.py for now.  What do you think about this?

Hi @Argusmocny thank you for putting this together. I think it is a good start.
A couple of thoughts here

SubstitutionMatrix here would have name conflict with the skbio.sequence.substitinon_matrix. Maybe we could call it PositionSpecificSubstitutionMatrix or EmbeddingSubstitionMatrix
Do you have unit tests that confirm that the alignment capabilities work with the proposed substation matrix functionality?
It may not be a bad idea to look into having the proposed SubstitionMatrix be a subclass of the original class -- or if that is out of scope, port over the relevant attributes so that the APIs are mirrored.

The only concern that I have is that folks may have trouble finding the proposed SubstitionMatrix functionality if it is tucked away in the skbio.embedding section. It may not be a bad idea to update the toctree / documentation in the alignment section to make this link clear.


Reorganize topographical View

Hi Marcelo @zoccoler,
this is a refactoring of your Topographical views code. The results should be identical with your implementation. I changed the internal code because: The generated image/points/surface were strongly related to the layer properties. Thus, I moved those code sections together. The function that produces the surface now also produces the surface layer properties.
This also allows to simplify the widget code so that there are not if-blocks anymore because I'm using an Enum (related discussion) to allow the user to choose what to visualize. The user-interface now looks like this:

If you think that's a bad idea, I can revert it to the checkboxes, but my intention was: Users may have one preferred way of visualizing the profile and will likely not view multiple views (image, points, surfaces) at a time. Thus, a pull-down may be easier to configure than three checkboxes. I think it was like that before, no? The screen recording looks like...
I simplified the test code as it was hard to read and understand. We should not write tests that take a list of functions as input and then test them. Write one test for one function. In case a test breaks, the bug is then much easier to find. See also: https://youtu.be/gSBR34m1OCE?t=539 at 9:00
I also renamed some variable to become English words and for clarity. I also renamed "sample_factor" to "step_size" so that the term is everywhere the same. Furthermore, I believe the sample_factor was actually a sample_divisor.
The tests pass and I tested the four functions from the user interface (Menu Plugins > napari-plot-profile > Topographical ... and Tools > Visualization > Topographical View) manually.
Would you mind taking another look if everything is ok from your perspective? I also added you as collaborator to this repository, so feel free to modify things on this branch. Also [optionally] you are welcome to add yourself to authors and copyright holders.
Thanks!
Best,
Robert

The tests pass locally and that's sufficient btw.

Hi Robert,
The suggested re-arrangement makes sense to me. The main reason I had left the functions return just the data apart from the layer properties was because, if these functions were to be called by code (even without napari), they would return just the data, which makes sense. But I think any user doing that could just get the first element of the tuple.
I made a small change because the napari-plot-profile was duplicated when accessed from the plugins menu. Also, the negative part (as a 3D images and a surface) was not getting reversed. With these changes, this gets fixed, but only when the plugin is accessed from the Tools menu. Check the screens below:
From Tools Menu:

From Plugins Menu:

I could not find what is causing this. Do you have any guesses?
Best,
Marcelo

Thanks Johannes @jo-mueller , I am converting this to draft to give it a try!

It is working now, thank you Johannes @jo-mueller !
Tests pass locally.
Robert @haesleinhuepf , could you please give it a look again?

Hi Robert @haesleinhuepf ,
do you think this can be merged? 😄

Apologies for the delay @zoccoler . Good that you poked here! :-)


HBASE-24603: Make Zookeeper sync() call synchronous (#1945)

Writing a test for this is tricky. There is enough coverage for
functional tests. Only concern is performance, but there is enough
logging for it to detect timed out/badly performing sync calls.
Additionally, this patch decouples the ZK event processing into it's
own thread rather than doing it in the EventThread's context. That
avoids deadlocks and stalls of the event thread.

_ Prechecks _

+1 :green_heart:
dupname
0m  0s
No case conflicting files found.

+1 :green_heart:
hbaseanti
0m  0s
Patch does not have any anti-patterns.

+1 :green_heart:
@author
0m  0s
The patch does not contain any @author tags.

+1 :green_heart:
test4tests
0m  0s
The patch appears to include 2 new or modified test files.

_ branch-1 Compile Tests _

+0 :ok:
mvndep
2m 20s
Maven dependency ordering for branch

+1 :green_heart:
mvninstall
7m 58s
branch-1 passed

+1 :green_heart:
compile
1m 22s
branch-1 passed with JDK v1.8.0_252

+1 :green_heart:
compile
1m 33s
branch-1 passed with JDK v1.7.0_262

+1 :green_heart:
checkstyle
2m 56s
branch-1 passed

+1 :green_heart:
shadedjars
3m 17s
branch has no errors when building our shaded downstream artifacts.

+1 :green_heart:
javadoc
1m 17s
branch-1 passed with JDK v1.8.0_252

+1 :green_heart:
javadoc
1m 31s
branch-1 passed with JDK v1.7.0_262

+0 :ok:
spotbugs
2m 52s
Used deprecated FindBugs config; considering switching to SpotBugs.

+1 :green_heart:
findbugs
5m 37s
branch-1 passed

_ Patch Compile Tests _

+0 :ok:
mvndep
0m 16s
Maven dependency ordering for patch

+1 :green_heart:
mvninstall
2m  4s
the patch passed

+1 :green_heart:
compile
1m 18s
the patch passed with JDK v1.8.0_252

+1 :green_heart:
javac
1m 18s
the patch passed

+1 :green_heart:
compile
1m 33s
the patch passed with JDK v1.7.0_262

+1 :green_heart:
javac
1m 33s
the patch passed

+1 :green_heart:
checkstyle
0m 30s
The patch passed checkstyle in hbase-common

+1 :green_heart:
checkstyle
0m 37s
hbase-client: The patch generated 0 new + 77 unchanged - 2 fixed = 77 total (was 79)

+1 :green_heart:
checkstyle
1m 43s
The patch passed checkstyle in hbase-server

+1 :green_heart:
whitespace
0m  0s
The patch has no whitespace issues.

+1 :green_heart:
shadedjars
3m  5s
patch has no errors when building our shaded downstream artifacts.

+1 :green_heart:
hadoopcheck
4m 56s
Patch does not cause any errors with Hadoop 2.8.5 2.9.2.

+1 :green_heart:
javadoc
1m  9s
the patch passed with JDK v1.8.0_252

+1 :green_heart:
javadoc
1m 30s
the patch passed with JDK v1.7.0_262

-1 :x:
findbugs
1m 39s
hbase-client generated 1 new + 0 unchanged - 0 fixed = 1 total (was 0)

_ Other Tests _

+1 :green_heart:
unit
2m 42s
hbase-common in the patch passed.

+1 :green_heart:
unit
2m 50s
hbase-client in the patch passed.

-1 :x:
unit
167m 24s
hbase-server in the patch failed.

+1 :green_heart:
asflicense
1m 21s
The patch does not generate ASF License warnings.

232m 29s

Reason
Tests

FindBugs
module:hbase-client

Format-string method String.format(String, Object[]) called with format string "Invalid event of type {} received for path {}. Ignoring" wants 0 arguments but is given 2 in org.apache.hadoop.hbase.zookeeper.ZooKeeperWatcher.processEvent(WatchedEvent)  At ZooKeeperWatcher.java:with format string "Invalid event of type {} received for path {}. Ignoring" wants 0 arguments but is given 2 in org.apache.hadoop.hbase.zookeeper.ZooKeeperWatcher.processEvent(WatchedEvent)  At ZooKeeperWatcher.java:[line 666]

Failed junit tests
hadoop.hbase.mapreduce.TestSecureLoadIncrementalHFiles

Subsystem
Report/Notes

Docker
Client=19.03.12 Server=19.03.12 base: https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1976/1/artifact/out/Dockerfile

GITHUB PR
https://github.com/apache/hbase/pull/1976

JIRA Issue
HBASE-24603

Optional Tests
dupname asflicense javac javadoc unit spotbugs findbugs shadedjars hadoopcheck hbaseanti checkstyle compile

uname
Linux 966ee6e0d634 4.15.0-101-generic #102-Ubuntu SMP Mon May 11 10:07:26 UTC 2020 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
/home/jenkins/jenkins-slave/workspace/Base-PreCommit-GitHub-PR_PR-1976/out/precommit/personality/provided.sh

git revision
branch-1 / 54c38c8

Default Java
1.7.0_262

Multi-JDK versions
/usr/lib/jvm/zulu-8-amd64:1.8.0_252 /usr/lib/jvm/zulu-7-amd64:1.7.0_262

findbugs
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1976/1/artifact/out/new-findbugs-hbase-client.html

unit
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1976/1/artifact/out/patch-unit-hbase-server.txt

Test Results
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1976/1/testReport/

Max. process+thread count
4624 (vs. ulimit of 10000)

modules
C: hbase-common hbase-client hbase-server U: .

Console output
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1976/1/console

versions
git=1.9.1 maven=3.0.5 findbugs=3.0.1

Powered by
Apache Yetus 0.11.1 https://yetus.apache.org

This message was automatically generated.

:confetti_ball: +1 overall

Vote
Subsystem
Runtime
Comment

+0 :ok:
reexec
12m  9s
Docker mode activated.

_ Prechecks _

+1 :green_heart:
dupname
0m  0s
No case conflicting files found.

+1 :green_heart:
hbaseanti
0m  0s
Patch does not have any anti-patterns.

+1 :green_heart:
@author
0m  0s
The patch does not contain any @author tags.

+1 :green_heart:
test4tests
0m  0s
The patch appears to include 2 new or modified test files.

_ branch-1 Compile Tests _

+0 :ok:
mvndep
2m 29s
Maven dependency ordering for branch

+1 :green_heart:
mvninstall
8m 25s
branch-1 passed

+1 :green_heart:
compile
1m 20s
branch-1 passed with JDK v1.8.0_252

+1 :green_heart:
compile
1m 32s
branch-1 passed with JDK v1.7.0_262

+1 :green_heart:
checkstyle
2m 58s
branch-1 passed

+1 :green_heart:
shadedjars
3m 17s
branch has no errors when building our shaded downstream artifacts.

+1 :green_heart:
javadoc
1m 17s
branch-1 passed with JDK v1.8.0_252

+1 :green_heart:
javadoc
1m 30s
branch-1 passed with JDK v1.7.0_262

+0 :ok:
spotbugs
2m 51s
Used deprecated FindBugs config; considering switching to SpotBugs.

+1 :green_heart:
findbugs
5m 35s
branch-1 passed

_ Patch Compile Tests _

+0 :ok:
mvndep
0m 16s
Maven dependency ordering for patch

+1 :green_heart:
mvninstall
2m  4s
the patch passed

+1 :green_heart:
compile
1m 18s
the patch passed with JDK v1.8.0_252

+1 :green_heart:
javac
1m 18s
the patch passed

+1 :green_heart:
compile
1m 32s
the patch passed with JDK v1.7.0_262

+1 :green_heart:
javac
1m 32s
the patch passed

+1 :green_heart:
checkstyle
0m 29s
The patch passed checkstyle in hbase-common

+1 :green_heart:
checkstyle
0m 38s
hbase-client: The patch generated 0 new + 77 unchanged - 2 fixed = 77 total (was 79)

+1 :green_heart:
checkstyle
1m 44s
The patch passed checkstyle in hbase-server

+1 :green_heart:
whitespace
0m  0s
The patch has no whitespace issues.

+1 :green_heart:
shadedjars
3m  5s
patch has no errors when building our shaded downstream artifacts.

+1 :green_heart:
hadoopcheck
4m 53s
Patch does not cause any errors with Hadoop 2.8.5 2.9.2.

+1 :green_heart:
javadoc
1m  7s
the patch passed with JDK v1.8.0_252

+1 :green_heart:
javadoc
1m 30s
the patch passed with JDK v1.7.0_262

+1 :green_heart:
findbugs
5m 59s
the patch passed

_ Other Tests _

+1 :green_heart:
unit
2m 37s
hbase-common in the patch passed.

+1 :green_heart:
unit
2m 37s
hbase-client in the patch passed.

+1 :green_heart:
unit
146m 10s
hbase-server in the patch passed.

+1 :green_heart:
asflicense
1m  6s
The patch does not generate ASF License warnings.

222m  5s

Subsystem
Report/Notes

Docker
Client=19.03.12 Server=19.03.12 base: https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1976/2/artifact/out/Dockerfile

GITHUB PR
https://github.com/apache/hbase/pull/1976

JIRA Issue
HBASE-24603

Optional Tests
dupname asflicense javac javadoc unit spotbugs findbugs shadedjars hadoopcheck hbaseanti checkstyle compile

uname
Linux 6ee779511641 4.15.0-101-generic #102-Ubuntu SMP Mon May 11 10:07:26 UTC 2020 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
/home/jenkins/jenkins-slave/workspace/Base-PreCommit-GitHub-PR_PR-1976/out/precommit/personality/provided.sh

git revision
branch-1 / 54c38c8

Default Java
1.7.0_262

Multi-JDK versions
/usr/lib/jvm/zulu-8-amd64:1.8.0_252 /usr/lib/jvm/zulu-7-amd64:1.7.0_262

Test Results
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1976/2/testReport/

Max. process+thread count
5089 (vs. ulimit of 10000)

modules
C: hbase-common hbase-client hbase-server U: .

Console output
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1976/2/console

versions
git=1.9.1 maven=3.0.5 findbugs=3.0.1

Powered by
Apache Yetus 0.11.1 https://yetus.apache.org

This message was automatically generated.


Learn - Testimony Writing

Mocks here
This covers the main content of the page, not the tabbed navigation.

Mockup: https://zpl.io/Am6AeJq

Fixed by #574 thanks @Dev1nxavier


Profile_card - Mohd Suhail

I have added my code in Code file and changed readme folder. Please merge it!

@DiyaVj Can you merge it?


SB-805: Migrate from EHCache to Hazelcast

Thanks for your help @sbespalov!

Feedback from @sbespalov :

Sergey, 5:56 PM
cahce should be synchronized with transactions
there was EhCacheCacheManager.setTransactionAware(true) for ehcahce but seems that it's not so simple for Hazelcast
they have separate HazelcastTransactionManager which need to be manually synchronized with JpaTransactionManager
probably it can be done with XA transactions
anyway some aditional configuration will be required for this

A potentially useful pull request: https://github.com/hazelcast/hazelcast/pull/7469.

@sbespalov can't we re-implement com.hazelcast.spring.cache.HazelcastCacheManager extending org.springframework.cache.transaction.AbstractTransactionSupportingCacheManager leaving our JpaTransactionManager for our entityManagerFactory as it was ?

@fuss86 : I believe that's what @sbespalov meant and I did incorrectly.

@sbespalov can't we re-implement com.hazelcast.spring.cache.HazelcastCacheManager extending org.springframework.cache.transaction.AbstractTransactionSupportingCacheManager leaving our JpaTransactionManager for our entityManagerFactory as it was ?

yeah, we can try this way. But this is strange why guys from Hazelcast don't use AbstractTransactionSupportingCacheManager or even AbstractCacheManager.

Great, I'll give it a try tomorrow.

Work continued by @fuss86 in #691.


Adding a Global Vehicle Restrictions setting/Default Vehicle Restriction setting

Describe your idea
Currently, for certain challenges (no cars challenge) and certain builds (make a part of the map totally vehicle-less) possible, we weill need to modify the vehicle restrictions on each of the road segments. This can be simply fixed by adding a global setting for a certain kind of specified road, or just adding a adjustable default for new road segments. This would make certain challenges not annoying and actually fun.
Screenshots?

Notes or questions?

A copy/paste, or a multi-segment-select would be wonderful.

A copy/paste, or a multi-segment-select would be wonderful.

You can copy&paste networks with TM:PE setting if you use MoveIt and TM:PE 11.6+ (currently under testing)

A copy/paste, or a multi-segment-select would be wonderful.

You can copy&paste networks with TM:PE setting if you use MoveIt and TM:PE 11.6+ (currently under testing)

Im not talking about copy paste of the road itself, but instead, copy paste of the signage / vehicle restrictions.
Aka, select 10 road segments, or use a Square selection to select many road segments, then set the signage for all of them at once. This could be a grid pattern, or multiple roads with multiple intersections. The current way requires you to do one road at a time, and when it is a grid of say small road / bike paths, I want to restrict them all in a district, I have to do each road by hand due to intersections.

Related: #154


webapp modifs for download page

In GitLab by @fbastian on Aug 17, 2015, 16:49

modify CommandDownload to use the new classes from bgee-core

modify HtmlDownload to receive and use these new objects for the display.

use of AngularJS
when generating the HTML page, put keword-species associations into a javascript variable (between script tag), to be used by AngularJS

See issue#31

In GitLab by @fbastian on Sep 13, 2015, 03:22
@pmoret: currently, URLs to the documentation used in the download pages are generated by download.js, see, e.g. code lines 459-470.
I think we should centralize as much as possible the generation of URLs on the Java webapp-side. So, HtmlDownloadDisplay should generate the documentation URLs, and provide them to the javascript.
Do you agree?
If you do, then search for getRequestURL method in download.js, they should all disappear.
More generally, pay attention to the XXXs and TODOs in HtmlDownloadDisplay and download.js, to directly implement the correct solutions.

In GitLab by @fbastian on Oct 14, 2015, 01:32
See 9168279b055. Need to update the database on production server before doing the release. Keep issue opened until then.


feat: Add replica configuration to create global DynamoDB tables

fixed: removed stuff
Description
Adding replica regions to create global DynamoDB tables
Motivation and Context
Resolves #19
Breaking Changes
No
How Has This Been Tested?
Run terraform apply and terraform destroy on the global-tables example folder.

It's weird that the terraform docs check has filed. Investigating this now.

Looks like this has something to do with terraform-docs 0.10. Upgraded to 0.10.0 from 0.9.1 and this has started failing locally too.

Thanks @mpuri-tyl !
terraform-docs was indeed an unexpected surprise, I have fixed pre-commit-terraform repo for that.
v0.8.0 has been just released.


Fixed creation task problem

Motivation and context

Fixed #3215
Fixed #3226
How has this been tested?

Manually
Checklist

[x] I submit my changes into the develop branch
[x] I have added description of my changes into CHANGELOG file
- [ ] I have updated the documentation accordingly
- [ ] I have added tests to cover my changes
[x] I have linked related issues (read github docs)
[x] I have increased versions of npm packages if it is necessary (cvat-canvas,
cvat-core, cvat-data and cvat-ui)

License

[x] I submit my code changes under the same MIT License that covers the project.
Feel free to contact the maintainers if that's a concern.
[x] I have updated the license header for each file (see an example below)

# Copyright (C) 2021 Intel Corporation
#
# SPDX-License-Identifier: MIT

@ActiveChooN , there are problems with linter and some tests.

@nmanovic, all problems were fixed


added support for crazy unicode in the value of custom fields

Hey @bufke here's what I've come up with tonight... as it turns out I don't have commit access to django-custom-field so a pull-request here will do. Hard to figure out what the user is doing, but these tests seems to say that this works. I'm somewhat losing my mind over this stuff!!!

Is this tested in py3 too? What about None values? Bleh need to fix those travis tests.


resolving ui issues

The resolved issues are as follows:

https://github.com/project-chip/zap/issues/287
https://github.com/project-chip/zap/issues/282
https://github.com/project-chip/zap/issues/271
https://github.com/project-chip/zap/issues/272
https://github.com/project-chip/zap/issues/281
https://github.com/project-chip/zap/issues/283
https://github.com/project-chip/zap/issues/284
https://github.com/project-chip/zap/issues/285
https://github.com/project-chip/zap/issues/288
https://github.com/project-chip/zap/issues/290
https://github.com/project-chip/zap/issues/291
https://github.com/project-chip/zap/issues/292
https://github.com/project-chip/zap/issues/293
https://github.com/project-chip/zap/issues/298

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.2 out of 3 committers have signed the CLA.:white_check_mark: babakzarrinbal:white_check_mark: Mehradml:x: sahar kargarsahar kargar seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.

https://github.com/project-chip/zap/issues/287: See my comment on the bug. There is a weird jump.
https://github.com/project-chip/zap/issues/282: Same problem as above. When table is too small, there is a jump. (I noticed that the jump doesn't happen if the scrollbar is not visible, so jump is only if the table is long enough to cause a scrollbar.)


Add a "Create new materials and images" import option

This is an option to be added to the importer. The reason for its existence is the following:

Dwarf.smf
Hammer.smf
Spider.smf
...

Material Name
"Default"
"Default"
"Default"
...

Image Name
"71"
"70"
"71"
...

In short: there are SMF models that have identical material names, yet have different image/texture names. Additionally, some models have identical image/texture names though they contain different image data (i.e. their texture images are different).

When importing SMF textures in Blender:

Always link the image via a material
Name newly added materials using the SMF image name
Use the image name instead of the material name as the key

The Link Existing Materials/Images option will reuse existing materials in the Blender file and link those to the imported model, based on the image name found in the SMF model, instead of creating new ones with a suffix. This way there is never an issue with the "Default" material name. The image name is most unique (though it not necessarily is).
The option is disabled by default (since that produces the desired behavior on import of the above example models).

This should really be a Create new materials and images option since it is an option to override the most ideal way of doing things. It is enabled by default.

Added in dev d9c2af9


[DOCS] Typo in qwik/packages/docs/src/routes/docs/components/lifecycle/diagram2.svg

How can the documentation be improved?
In use-cases box bottom left:

Read DOM ~necesary~ -> Read DOM necessary
Perform computation ~eagarly~ in the ~broswe~ -> Perform computation eagerly in the browser

Would make a pull request with changed SVG but seemed difficult when you don't have the file it was generated from

Thanks for the report @Oskar-Nilsen-Roos !
I think @manucorporat or @mhevery have the source


Allow URL params for Security plugin intercept calls

Description
This change fixes integ test failures seen in the 2.14.0 integ test for the security-dashboards-plugin: https://github.com/opensearch-project/security-dashboards-plugin/issues/1893
In 2.14.0, the security-dashboards-plugin added support for multiple data sources (MDS) and allows dataSourceId to pass as a url param for MDS enabled Dashboards instances. This change updates the calls to cy.intercept to allow url params. The tests currently hang on these calls.
Check List

[X] Commits are signed per the DCO using --signoff

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

Will help to merge once all the CI complete.

Thanks @SuZhou-Joe!

Checked with @cwperks . Seems: https://github.com/opensearch-project/opensearch-dashboards-functional-test/pull/1266 addresses this issue and will have a follow up. Will remove the 2.14 label for tracking purposes. If that was incorrect @cwperks please re-add. Thank you!


Remove debug panic

Motivation
This was debug that was not removed from #4374
We don't actually want to panic, when trying to service a fetch request and there are no peers.
DevOps Notes

[x] This PR does not require configuration changes (e.g., environment variables, GitHub secrets, VM resources)
[x] This PR does not affect public APIs
[x] This PR does not rely on a new version of external services (PoET, elasticsearch, etc.)
[x] This PR does not make changes to log messages (which monitoring infrastructure may rely on)

@countvonzero  I don't see a reason to check this slice for zero length any more than any other slice passed as a parameter if the outcome either way is to panic. Doing this across the codebase would add a lot of boilerplate, and not provide much value if any.

bors merge

bors merge

bors merge


Conflict resolution at save time is slow

We can probably restructure this code to try an optimistic save first and only enter the conflict resolution loop after the server returns 409 Conflict.

[x] try fast save before entering conflict resolution loop

Part of the slowness is checking for modifications of related things. So if a user modifies big multipolygon, we need to check the multipolygon members for conflicts too.

[x] :warning: That code needs to handle circular relations, probably by adding the loadMore things to the toCheck array so they don't get refetched ad infinitum.
[x] A progress bar or updating the message with some kind of "Conflict Checking 1/50" message would help here.

I recently have a warning for 3090 conflicts, that ID try to keep resolving but not solution ever came out, so I need to discard all my edits otherwise ID dosent work. How I could solve this if happen again?

I recently have a warning for 3090 conflicts, that ID try to keep resolving but not solution ever came out, so I need to discard all my edits otherwise ID dosent work. How I could solve this if happen again? Is anoying to loose so much data

I honestly didn't think it could go up that high.  Can you share more info about what kind of editing were you doing?
I really can't give much guidance other than to just save frequently, especially if you think other users might also be editing in the same area.


Options are discarded

https://github.com/micro/go-micro/blob/e3f818d18e7137f860c5db67d52af6b642c47c38/go-micro.go#L84
This utility func is useful, but discards passed options. Shouldn't we pass options to the s.NewSubscriber(...) ?

Feel free to submit a PR otherwise I will fix later on today

Ok, done


Is there any right way on how to persist to storage?

Is there any right way on how to persist the state to the storage, and set it back on the initial app render?
Thanks
EDIT:
Sorry, my bad. I thought there is a weird behavior in the library that reset the state after I set my persisted state. Turns out, it is caused by useEffect inside component.

@rifaldhiaw Can you share snippets showing how you implemented persistence?


Notifications table continues to throw duplicate entry for id column

Laravel Version: 9.37.0
PHP Version: 8.1.9
Database Driver & Version: MySQL 5.7

Description:
When sending notifications, the notifications table throws the following error:

PDOException: SQLSTATE[23000]: Integrity constraint violation: 1062 Duplicate entry '67f394ac-b1be-4300-8a7e-7f4daf6f65ee' for key 'PRIMARY' in /var/www/domain-monitor-api/vendor/laravel/framework/src/Illuminate/Database/Connection.php:545

Steps To Reproduce:
I have 85,000 notifications in my notifications table for reference.
I've come across this discussion from 2019, except my notifications table's id column length is perfectly fine at 36 characters, what's more, my table migration for notifications for the id column also matches recommendations:
Schema::create('notifications', function (Blueprint $table) {
    $table->uuid('id')->primary();
    $table->string('type');
    $table->morphs('notifiable');
    $table->text('data');
    $table->timestamp('read_at')->nullable();
    $table->timestamps();
});

I'm not sure why this keeps happening randomly, I'll attach a screenshot of my phpmyadmin table structure for notifications as I get this error randomly both locally and in production - I think others will be getting this error still 👍

What is the reason behind using uuid in notifications table?

@ankurk91 That's the default schema inherited from earlier versions of Laravel. When running the php artisan notifications:table command a notifications table is generated and the uuid is what Laravel generates.

Just curious, @sts-ryan-holton did you run your notifications in queue?

@aanfarhan Yeah, they're running in a queue, using Supervisor, and I have about 28 workers running.

Your "Steps To Reproduce:" don't provide any code that reproduces this. Except for the issue you linked to I don't think we've ever had this issue already.
Can you first please try one of the support channels below? If you can actually identify this as a bug, feel free to open up a new issue with a link to the original one and we'll gladly help you out.

Laracasts Forums
Laravel.io Forums
StackOverflow
Discord
Larachat
IRC

Thanks!


HIVE-28662: Enable the dynamic leader election for HMS

What changes were proposed in this pull request?
Enable the dynamic leader election on HMS by default

Why are the changes needed?
In the Hive ACID world, we must enable the metastore.compactor.initiator.on and metastore.compactor.cleaner.on on HMS to trigger the table compaction automatically. In a real warehouse, we might have multiple HMS instances behind, or even deploy for different purposes, running the tasks like these on all instances could be a waste of resources, make the HMS scale out not so easily.

Does this PR introduce any user-facing change?
Default value of the following properties changed:
metastore.housekeeping.leader.election host -> lock
metastore.housekeeping.threads.on false -> true
metastore.compactor.initiator.on false -> true
metastore.compactor.cleaner.on false -> true

Is the change a dependency upgrade?
No

How was this patch tested?
The existing UTs

This seems to be causing a deadlock and HMS is crashing. https://issues.apache.org/jira/browse/HIVE-28659?focusedCommentId=17905565&page=com.atlassian.jira.plugin.system.issuetabpanels%3Acomment-tabpanel#comment-17905565


VecEnv render does not work

Found in #111

Some resources @ajaysub110 found:

https://www.reddit.com/r/reinforcementlearning/comments/a3mwfn/openai_gym_how_to_get_pixels_in_classic_control/
https://github.com/Rowing0914/TF_RL

This might help here: https://github.com/openai/gym/pull/1624

Tracking in #382


Switch the return sign of week.Sub to be consistent with time.Sub

Coverage remained the same at 97.887% when pulling 747a19d6481849bdfaac0c9f0f12e7a5dbedd376 on xkizer:ZSB-465 into 50eaf79a878d3f7a2a4eb4385d6a7a30ab96fc02 on stoewer:master.

Fixes #12


Language chooser

I tried to fix #1087 in a way that should stay with old behavior if CMS_HIDE_UNTRANSLATED is not True. This should make behavior consistent with current documentation, see last paragraph in docs/advanced/i18n.rst

needs testcase


Narrow side buttons + info text to full screen height

Closes #144
Closes #163

I tested it on my tablet and phone and I am able to find these buttons despite the shrinkage (and with TalkBack too)
So hopefully this will improve the look of our app. Of course this is a trade-off between large buttons and wide text. We will ask visually impaired about it.


Common layer interface

Closes #179.

The changes look good to me, though I don't quite understand why it closes #179, since the common interface appears to be limited to getting the width/height, which is then non-existent for infinite layers.

Well, that's really all they have to offer. There's already a get_tile method iirc so what was missing were the w/h properties.

In that sense, actually I think a bounds property returning a rectangle might be more useful. It would be rect(0, 0, width, height) for finite layers, but could return the bounding rectangle of the content for infinite layers, providing a straight-forward way of determining which area to process for tiles while rendering.

Sure, but this would be an even better job for the future tiles iterator. I'll add a bounds function though, it does seem useful.

#179 also mentions the unified storage for finite and infinite tile layers, but I guess that's an idea better split off into its own issue.

Yeah I don't think that's happening at any point, since it'd require another refactor which would completely change the interface again. It's ok as is, I guess.

I'm merging now and introducing the bounds function in another PR, since right now neither of the finite/infinite tile layers have the function so there's nothing to unify (it's just a new convenience function). I think it deserves its own PR


swagger-models 2.0.0 exports wrong packages

Q
A

Bug or feature request?
Bug

Which Swagger-Core version?
2.0.0

Which Java version?
n/a

Which JAX-RS framework & version?
n/a

In the swagger-core/modules/swagger-models/pom.xml the maven-bundle-plugin exports the packages io.swagger.oas.models..., yet the actual packages are named io.swagger.v3.oas.models... (the .v3 is missing fom the pom).
This produces wrong bundle information and makes the bundle unusable in an OSGi runtime.

Thanks for reporting this; it should be fixed in #2664 and 2.0.0-SNAPSHOT available in sonatype and in oncoming release. Please reopen if you're still experiencing issues.


Extract migrations to manageiq-schema repo

This is not ready yet, but I need a PR in case something changes out from under me. It's ready.

@bdunne I am 👍  on this.  Assigning to you so you can coordinate with @Fryguy on when to merge.


Execute Call is called multiple times even if was successful in ICON chain.

The relayer will need info on the dapp. To resolve this the relayer will need to be tightly coupled with the dapp.


Licence information is not visible in NuGet.org

According to blog here: http://blog.nuget.org/20131011/friendly-license-names.html
we should be seeing licence info in packages page and per package pages, but we don't. Investigation required.

This happened before: https://github.com/NuGet/NuGetGallery/issues/1837
https://github.com/NuGet/NuGetGallery/issues/2396

The UpdatePackageLicenseReports job is running correctly and processing new license reports from Sonatype, but it appears there have been no licenses processed since June/July. I was unable to find how any documentation on how our interaction with Sonatype works or how we enqueue packages to be processed so I have reached out to some people who may know and am waiting on their response.

I have reached out to Sonatype and they have affirmed that the issue is on their side. I will update this issue as I get updates from them.

Sonatype has fixed the issue.


Preprocessor cannot have different pattern than files

I have this config file, which works properly:
module.exports = function (config) {
    config.set({
        plugins: [
            'karma-browserify',
            'karma-phantomjs-launcher',
            'karma-jasmine'
        ],
        frameworks: ['browserify', 'jasmine'],
        files: [
            "index.js", "src/**/*.js", 'spec/**/*.spec.js'
        ],
        preprocessors: {
            "index.js": ['browserify'],
            "src/**/*.js": ["browserify"],
            'spec/**/*.spec.js': ['browserify']
        },
        browserify: {
            debug: true
        },
        browsers: ['PhantomJS'],
        reporters: ['progress'],
        port: 9876,
        colors: true,
        logLevel: config.LOG_INFO,
        autoWatch: true,
        captureTimeout: 6000,
        singleRun: false
    });
};

Is it normal that when I change the preprocessors to:
        preprocessors: {
            "**/*.js": ['browserify']
        },

it does not browserify the js files?
Isn't there a way to use different patterns in preprocessors? It is error prone to have non-DRY code.

I think the problem is that */** includes the node_modules as well.

I don't think this is a bug. Browserify adds the libs from node_modules to the preprocessors. The only problem with it, if we have a lib, which has a karma adapter or has require to unknown files. For example mocha is something like that. If I exclude mocha from preprocessing with browserify it appears to work properly.


Merge to live!

Docs Build status updates of commit c7c536c:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs/actionable-messages/message-card-reference.md
:white_check_mark:Succeeded

docs/actionable-messages/security-requirements.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


Is that possible to use google-authenticator as a digital signature

Hey,
I have to design a high-level architecture for legal documents system. Is that possible to use google authenticator for signature? would this be a good idea?

You should get professional help from a cryptographer and a lawyer.

You should get professional help from a cryptographer and a lawyer.


fix: update versions

Version update

:tada: This PR is included in version 0.46.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


OCM-1007 | fix: Improve list account-roles runtime

It seems like there is a redundant API call to listPolicies, the role's policies are fetched from AWS but not used. Removing the policies API call and the Policy field from the role struct.
Note: This change introduces an improvement when listing the roles in our AWS dev account real	0m21.995s  -> real	0m15.018s, we might need another iteration on this ticket.

/lgtm


Add PregWrapper

Solution for https://github.com/FriendsOfPHP/PHP-CS-Fixer/pull/3309 and idea how to handle non-UTF8 strings.
Of course, as @Slamdunk already mendioned, with AutoReview class to check if all preg_* functions are wrapped.

@keradus @juliendufresne any idea how to handle PHP 5.3? I cannot reproduce the problem reported by Travis: http://sandbox.onlinephpfunctions.com/code/255e4b8c9762c6cc8e51a77245777f2481b42c46

@julienfalque @keradus PR updated, unfortunately fabbot.io cannot be satisfied in this case - what do you think about it? I'm not sure if we want to wrap preg_ functions in tests directory - what do you think?

@julienfalque @keradus PR updated, unfortunately fabbot.io cannot be satisfied in this case - what do you think about it? I'm not sure if we want to wrap preg_ functions in tests directory - do we?

You can ignore fabbot.io's report, the applied rule does not exist on branch 2.2 but it exists and is applied on branch 2.10.

@julienfalque I know we have to ignore fabbot.io report in this case - I mean what do you think about this PR? Can you continue review so we have this merged?
@keradus I've removed wrapper from tests as we spoke, feel free to do code review as well.

Thank you @kubawerlos.


how set collate?

I need set collate of database for utf8_unicode_ci, how make this?

You you confused this with mysql?
Btw, Sqlite3 only supports UTF-8 and UTF-16, while UTF-8 seems to be the default value.
Have also a look here: http://www.sqlite.org/pragma.html#pragma_encoding

This is not MySQL - this SQLite.


Set image for containers at plugin level with command specific overrides if necessary

Migrated from GitLab: https://gitlab.com/meltano/meltano/-/issues/3271
Originally created by @tayloramurphy on 2022-02-22 18:00:28

Still relevant, but low priority
Related:

#8713


Panic when configure the IMAP

What happens when i change the global config of an IMAP item?

Use docker  compose of deplyment.

hi，is here


Resnet ckpt file returns None when proccesing tf.train.get_checkpoint_state()

Hello,
This error, No checkpoint Found, Please provide pretrained checkpoint occured when processing tf.train.get_checkpoint_state(). I download pretrained ckpt file from https://github.com/tensorflow/models/tree/master/research/slim#pre-trained-models, and set RESTORE_PATH at scripts/train_flickr_resnet_cmpm.sh.
I found that tf.get_checkpoint_state() returns None. How can I solve this problem?
Thank you.

I found that restore_path didn't use .ckpt file, but checkpoint, and it is text file. close this issue.


Confidence or quality indicator?

Awesome plugin - thanks for writing it.
Are you able to provide a measure of the geocode's confidence level or quality indicator? eg, If you input an address of "6", you get the coordinates (33.8166949, -116.5147457).
Clearly this should have a lower quality rating than a fully-qualified address.

Some APIs (eg, Yahoo) return a "quality" attribute, and some don't. The Geocoder gem has no way of evaluating quality of results outside of what's returned by a geocoding request.


ImportError: No Module named 'Mastodon'

Having this issue...
Traceback (most recent call last):
  File "./hnbot.py", line 3, in <module>
    from mastodon import Mastodon
ImportError: No module named 'mastodon'

Same exact problem. This is when running the initial setup command:
python3 ./hnbot.py<EMAIL_ADDRESS>"yourpassword" "https://instancename.com"

This results in the same error that @owlg mentioned above:
Traceback (most recent call last):
  File "./hnbot.py", line 3, in <module>
    from mastodon import Mastodon
ImportError: No module named 'mastodon'

Any assistance would be appreciated.

pip3 install Mastodon.py

Thanks, @raymestalez—after running that command, the error changes. Now I get this:
Traceback (most recent call last):
  File "./hnbot.py", line 3, in <module>
    from .mastodon import Mastodon
SystemError: Parent module '' not loaded, cannot perform relative import

Sorry for needing more help :(

This is still an ongoing error?!
I installed it and I can see the mastodon folder inside site-packages and the folder contains Mastodon.py and streaming.py however, I am still getting ImportError: cannot import name 'Mastodon' from 'mastodon' any idea how to fix this?

This is still an issue.  Using Python 3.7 on macOS Ventura:
MacBook-Pro:Balancer rmiracle$ python3 -V
Python 3.7.3
MacBook-Pro:Balancer rmiracle$

It is installed:
MacBook-Pro:Balancer rmiracle$ pip3 show Mastodon.py
Name: Mastodon.py
Version: 1.8.0
Summary: Python wrapper for the Mastodon API
Home-page: https://github.com/halcy/Mastodon.py
Author: Lorenz Diener
Author-email<EMAIL_ADDRESS>License: MIT
Location: /usr/local/lib/python3.7/site-packages
Requires: python-dateutil, requests, decorator, blurhash, python-magic, six
Required-by:
MacBook-Pro:Balancer rmiracle$

The Python source code:
from mastodon import Mastodon
The error:
MacBook-Pro:Balancer rmiracle$ python3 balancer.py
Traceback (most recent call last):
  File "balancer.py", line 3, in <module>
    from mastodon import Mastodon
ModuleNotFoundError: No module named 'mastodon'
MacBook-Pro:Balancer rmiracle$

I also tried:
from Mastodon import Mastodon
and
from Mastodon.py import Mastodon
Now if I do a pip3 list, this library is the only one with a .py extension in the name and the only one that is capitalized:
MacBook-Pro:Balancer rmiracle$ pip3 list
Package          Version
---------------- ----------
blurhash         1.1.4
certifi          2019.3.9
chardet          3.0.4
decorator        5.1.1
dice             3.1.0
docopt           0.6.2
idna             2.8
Mastodon.py      1.8.0
mercurial        5.2
pip              19.3.1
pipenv           2018.11.26
protobuf         3.11.2
pyparsing        2.4.7
python-dateutil  2.8.2
python-magic     0.4.27
rbmwgtsite       1.4
requests         2.22.0
setuptools       41.6.0
six              1.13.0
urllib3          1.25.8
virtualenv       16.6.0
virtualenv-clone 0.5.3
wheel            0.33.6
MacBook-Pro:Balancer rmiracle$

So it's clearly installed yet python3 doesn't want to find it.
Any thoughts?


Cannot register the state source 'fooSource' because there is already a class with that Id.

Every time I restart my application (backed by Express), I get this error message. It looks like it's coming from registry.js:81:
if (this.types[type][id]) {
      throw ClassAlreadyRegisteredWithId(clazz, type);
}

As soon as I go into my fooSource and change the id, the application works just fine. This only happens on a fresh development build of my app. This feels super strange but I'm flagging in case anyone has run into something similar or if this rings any bells.

I'm using browserify and was requiring the same file but with different capitalizations. I guess browserify was treating it as two unique implementations of the same file.


Fix: Errors and bugs on the podcast player fixed.

Fix: Errors and bugs on the podcast player fixed.

@godsakani some minor fix on the podcast player here and a review is needed sir!


multiple newline bugs pt. 2

If a file ends in multiple newlines, then you'll get the strange document hidden on startup behavior that raayan first experienced.

only on iOS fwiw


Added Brasilian portuguese translation

Hello! I just added the brasilian translation on pt.json
Thanks for the awesome theme :)

@matheustavarestrindade thank you for the contribution.


Can't resolve '../_mockData'

Module not found: Error: Can't resolve '../_mockData'
Everything is just working fine until I want to add filters, How do I properly import '_mockDate' or maybe this a dependency I just missed ?

https://github.com/OneWayTech/vue2-datatable/blob/master/examples/src/_mockData/index.js


Add GUIDELINES

Add software development guidelines, such as PEP8 and PEP257 compliance, as well as naming conventions.  Should also emphasis use of doctests and unit tests, where possible.

Made a first stab at this.  Committed so that the rest of the crew can modify as they see fit.  Meanwhile, I need to add a blurb about standardizing on context.  And there are probably other things to mention.