Make CDI to replace EJB

A frequent request for CDI is to replace EJB as CDI is becoming more and more popular and widely adopted. CDI covers many EJB functionalities with a much easier framework. However, there are some gaps to be filled. I would like to use this issue to address the gaps where CDI has no counterpart functionalities. This includes:

startup

Timer
I think the useful feature of EJB is startup, so in CDI, we should provide this equivalent feature. As for Timer support, I think it can be supplied by other specs.

Startup is certainly a gap I have run into so would vote for that being addressed in some way. Or, said differently, a scope that enables an object to operate as a "task" and a way to have that task start as soon as the application is initialised, i.e. it's lifecycle of not triggered by or bound to an HTTP request.

startup is easy: @Observes @Initialized(ApplicationScoped.class)

Hi @tandraschko, thank you for that, I didn't know about that approach. Is that an official way of achieving at-startup or it just so happens to work? I'd worry about the points of lifecycle of watching the annotation class and it changing if this is not the spec defined way of achieving at-startup.
I may be worrying needlessly :)

Its an official way and available since 1.2

Awesome! And thank you again!

What about making that an extension first? It is quite trivial to impl it and IMO it does not belong to CDI which should stay the core IoC.
EE concurrency sounds like an option but truely speaking, if not distributable it is just a matter of creating a scheduledexecutorservice in a cdi bean so guess it should stay distributable to have more added value than just proposing another api than java se one.
So high level, CDI shouldnt aim at replacing EJB because:

Scope is different ("put it all" for EJB vs "IoC" for CDI with a few historical exceptions/mistakes)
EJB already reversed part of its responsabilities (jta, scopes, singleton+startup, ...)
CDI was built with extension api to avoid to redo EJB like specs

So IMHO this issue must be closed and tackled at jakarta ee level (where to dispatch scheduling now since only other missing feature is bean pooling and this is no more needed these days or trivially replaced by a Semaphore when uszd as a throttler).
Just my 2 cents.

Its an official way and available since 1.2

Yes, but I think this approach can be polished a bit =)

Is that an official way of achieving at-startup or it just so happens to work?

Yes, that's official. If you add such an obsever into a bean, an instance of bean will have to be created eagerly to be able to invoke that observer. That works fine for singleton and application scoped bean, other scoped might have issues as the context doesn't need to be active, or in case of dependent bean, a new instance will be created, observer invoked and the instance will then be destroyed again.

@ederks85 Quarkus has defined its own StartupEvent which can be observed like this:
import io.quarkus.runtime.StartupEvent;

@ApplicationScoped
class CoolService {
  void startup(@Observes StartupEvent event) {
  }
}

Perhaps we could standardize this event? I like this approach more than the current @Observes @Initialized(ApplicationScoped.class) approach

@aguibert the reason Quarkus has this is because ApplicationScoped in JVM mode is started before you get to runtime (in STATIC_INIT). This vauses differences in behaviour when running in JVM versus native which may or may not affect what you sim to do with the observer. StartupEvent was introduced to fix that as it is always performed in RUNTIME_INIT phase which is consistent for both modes. See https://quarkus.io/guides/lifecycle#what-is-the-difference-from-initializedapplicationscoped-class-and-destroyedapplicationscoped-class
In other words, from purely CDI perspective @Observes @Initialized(ApplicationScoped.class) and @Observes StartupEvent would be duplicates.

There is one more difference: @Observes @Initialized(ApplicationScoped.class) is not saying anything about the application itself. It's fired when the application context is initialized. Whereas @Observes StartupEvent and @io.quarkus.runtime.Startup (since Quarkus 1.3)  is fired/triggered when the application is started, i.e. it may depend on some other services that are required to run before an application is considered started.

Before adding anything we should ensure we don't duplicate something already there and fully specified.
There is another built-in way to initialize at startup some code since v1.0: through extensions and AfterValidationDeployment event.
Any reason to not enable AfterValidationDeployment event to be observed by any bean instead of limiting it to just extensions?
Concretely the spec would be something like that:
"[existing definition]. AfterValidationDeployment can also be observe by CDI beans, the container is responsible to ensure it is fired through all extensions first, and if there is no exception it is fired to all other beans."

-1. I think that this would be even more confusing for users. AfterDeploymentValidation is a container lifecycle event that should only be observed by extensions (per the current wording).

The line between what events are container lifecycle ones and which are standard ones is (IMO) pretty well defined, I would try and avoid blurring it, so also -1.
Besides, you are saying that you don't want to duplicate what's already there, and @Observes @Initialized(ApplicationScoped.class) is already there and is not really different from what you are suggesting here, it's just another payload.

Right so we already have 2 ways to solve that so we don't need a 3rd. For the story @Observes @Initialized(ApplicationScoped.class) was mainly added by consistency (initialized exists for all built-in scopes) so is an acceptable exception in that regards.
So summary is we are good and close this issue?

Maybe we can call out the @Observes @Initialized(ApplicationScoped.class) is equivalent to EJB startup bean. Once we have done the support for Timer, CDI is kind of parity with EJB. Are there any other gaps?

@Emily-Jiang timers don't belong to CDI (see https://github.com/eclipse-ee4j/cdi/issues/424#issuecomment-593152876), this is why I think we are good. Only remaining gap will be pooled beans and CMP but don't think we want to do it there too ;).

I agree with @rmannibucau regarding timers being out of scope for CDI core.
I'm working on prototyping a portable CDI extension that will allow people to do what I described here and will see if I can deliver it in some OSS org.
If we agree that startup beans already have a solution and timers can be done with a portable extension, I think we can close this issue.

The EE Concurrency API has the following issue open for implementing an @Schedule annotation:
https://github.com/eclipse-ee4j/concurrency-api/issues/98
I added my proposal/POC to that issue, and I think EE Concurrency would be the proper spec to house such functionality.

Shameless jump in from a CDI user.
As a developer, I would prefer @Startup to have the meaning of framework/server as a whole is fully set up, not when the CDI component happens to be up. The @Observes method happens to work 99% of the time but is not correct in theory. What if CDI component initializes before another MP/Jakarta/Custom extension component and I try to use that component in the @Startup/@Observes bean?
There could be a dedicated annotation for this and a specified CDI event which the implementor would fire when they are finished with initialization.
Not sure this even belongs in the CDI, just something I observed from the comments above.

@cen1 it is exactly the same with EJB @Startup, you still have lifecycle issues as soon as you rely on multiple startup beans and dependencies can be hidden (inter modules) and not always resolved with depends on so I guess it is just status quo there + in CDI you can always wrap another bean to initialize it later (through an extension) so more flexible than EJB.

I have analyzed this issue in detail some time ago (perhaps somewhat sadly). My belief is that there isn't much to be done in the CDI specification itself. Most of the work resides in other Jakarta EE specifications absorbing EJB features using the CDI programming model as a baseline. Without going into great details, the following is the mapping I had come up with. In my view most of the discussions need to start with the EJB specification project and fan out to respective destination Jakarta EE projects.

EJB API
Destination
Comments

@TransactionAttribute, etc
Jakarta Transactions – renamed to @Transactional  (already done)
Should be composable via @Stereotype.

@RolesAllowed, @RunAs, etc
Jakarta Security
Should be usable by any CDI bean.

@Asynchronous, etc
Jakarta Concurrency
Should be composable via @Stereotype.

@Schedule, etc
Jakarta Concurrency
Should be composable via @Stereotype.

TimerService, etc
Jakarta Concurrency
These services can be injected into any managed bean.

@MessageDriven, etc
Jakarta Messaging – renamed to @JmsListener, Jakarta Connectors  – renamed to @ResourceListener
Redefine as declarative POJO JMS/JCA listeners on methods, not classes.

@Lock, @AccessTimeout, etc
Jakarta Concurrency
Should be composable via @Stereotype.

@Pooled, @MaxConcurrency (these are very common but non-standard EJB features)
Jakarta Concurrency
Should be composable via @Stereotype.

Reza Rahman
Jakarta EE Ambassadors, Author, Blogger, Speaker
Please note views expressed here are my own as an individual community member and do not reflect the views of my employer.

+1 reza

What about the extended EntityManagers which must be in a stateful EJB?

Stateful entity managers are already handled through scopes/contexts in CDI IMHO.

This looks very interesting and comprehensable, Reze. Good initiative. In
short: +1
Edwin
On Sun, 19 Apr 2020 at 11:20, Romain Manni-Bucau<EMAIL_ADDRESS>wrote:

Stateful entity managers are already handled through scopes/contexts in
CDI IMHO.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/eclipse-ee4j/cdi/issues/424#issuecomment-616083874,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABAPLV2USPCCUBURSBUNGSDRNK66BANCNFSM4KXBYOOA
.

Stateful entity managers are already handled through scopes/contexts in CDI IMHO.

Yes, but is an extended EntityManager is allowed in a CDI bean?

Yes, but is an extended EntityManager allowed in a CDI bean?

Well,  @PersistenceContext is allowed in EE but more important a stateful entity manager is an entity manager you can reuse accross calls which is exactly the definition of scopes in CDI.  Now, in terms of feature, CDI does not miss anything in that area.

What about the extended EntityManagers which must be in a stateful EJB?

With all due respect, I have personally not seen that feature in use in the field for many years. It could just be left in EJB, just like EJB remoting, or JPA could further standardize targeting the appropriate CDI scopes. Should be fairly easy to implement.
Reza Rahman
Jakarta EE Ambassador, Author, Blogger, Speaker
Please note views expressed here are my own as an individual community member and do not reflect the views of my employer.

+1 @m-reza-rahman

@m-reza-rahman The table looks good, but I think the programmatic APIs(as alternatives of the declaration strategy with annotations) should be considered at the same time.
Besides EJB, there are a lot of things that should be aligned with CDI.

The future of JSR 330, which is not updated in the latest 2 Java EE/Jakarta EE.
DI in EJB, Jaxrs, JSF, etc.
Refines Managed Bean1.0(part of Java EE 6.0), EJB, Jaxrs Bean, JSF Beans to make these compatible with CDI.
EL and CDI, to make them work seamlessly. In Spring, SpringEL is used everywhere and every powerful, etc. Esp, I like the usages in annotations, such as Security, Cache settings, etc. and JPA QL(eg. use select * from XX where username=#{principal.name} to read the Authentication info in JPQL directly).

@hantsy few comments on your last post:
1- JSR 330 shouldnt be intended to be updated, it was really the most common part of all IoC and guess it will stay like that, CDI could be updated if needed but JSR330 update sounds unlikely or do you see any precise need?
2+3- it is likely done enabling everything to use CDI, everything else is legacy now
4- EL+CDI is done, it is more about other specs and EL ;)  probably just some "phrasing" to be sure to put the effort where is it needed. Typically, ELProcessor makes is not hard to impl that now (just bindings to handle).

OK. It's important to have the functionality to keep entities in a
persistence context between transactions, like with an extended
EntityManager. It is not often used but it can be useful, for example in
long conversations between a client and a server.
Le 19/04/2020 à 16:57, Romain Manni-Bucau a écrit :

Yes, but is an extended EntityManager allowed in a CDI bean?

Well, @PersistenceContext is allowed in EE but more important a
stateful entity manager is an entity manager you can reuse accross
calls which is exactly the definition of scopes in CDI.  Now, in
terms of feature, CDI does not miss anything in that area.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/eclipse-ee4j/cdi/issues/424#issuecomment-616154471,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAYFS76NIL7M2EAUVKNSZ6TRNMGODANCNFSM4KXBYOOA.

The idea of adding support for extended persistence context (XPC) in CDI was tracked via CDI-162.
One difference between application managed + XPC, is that the container manages the XPC (e.g. there are lot of behind the scenes tracking/management for XPC inheritance performed for stateful session beans).  There is some cost to the tracking/management for XPC inheritance, that we don't have for application managed, however, for the application managed case, the application has to be careful not to leak the persistence context into other (user) requests, as persistence contexts can only be used by one thread at a time (the thread can vary and also be on different JVM instances, but again, never used by more than one thread at a time).
So, IMO, if users want CDI support for XPC, we should discuss the use cases and the JPA container contract, as well as the persistence provider contract as well.

@scottmarlow AFAIK if you produce an entity manager in a scope (let say session or conversation one) then you meet all these requirements:

You have an "inter request" em,
You close it when no more needed (through @Disposes),
Concurrency constraints is trivially manageable either with a @Lock interceptor (as in microprofile) or through the plain old synchronized (fine for this case),

So sounds like the feature is there for me. Do you see something else missing (I ignored the auto join tx which depends the PU style and tx mgt config)?

The table looks good, but I think the programmatic APIs (as alternatives of the declaration strategy with annotations) should be considered at the same time.

Sure. This is next level detail that should be discussed in the respective destination projects.

Besides EJB, there are a lot of things that should be aligned with CDI.

Certainly. I suggest following this up with the respective projects with some concrete details. Top of mind Jakarta REST and Batch certainly make sense. The others I think are largely done (e.g. JSF), the value proposition is tenuous (e.g. managed beans) or too difficult to do (e.g. Servlet).

The future of JSR 330, which is not updated in the latest 2 Java EE/Jakarta EE.

Personally I think this is an awkward part of Jakarta EE best left alone.

use select * from XX where username=#{principal.name}

If I am understanding this correctly, I believe I have already proposed this for Jakarta Security. I will make sure to remind Arjan once the Jakarta EE 9 work is under control. It's actually very easy to implement. That said, again, I encourage you to follow up with details in the right project (probably start a discussion and then file an issue).
Reza Rahman
Jakarta EE Ambassador, Author, Blogger, Speaker
Please note views expressed here are my own as an individual community member and do not reflect the views of my employer.

@m-reza-rahman  I see. These could be discussed under individual issues.

Briefly discussed this on the CDI call today. The 1st comment in this issue notes:

@Startup beans could be easily translated to CDI, which now has the Startup event. We could add an annotation (e.g. @Eager) that would translate to a synthetic observer of the Startup event on the annotated managed bean.
Timers could be specified by Jakarta Concurrency (e.g. methods on managed beans that are annotated @Scheduled), and if CDI had invokable methods (#460, #639), Jakarta Concurrency could probably delegate to that for the actual method invocation (getting interceptors for free). I'm not sure if we could easily make @AroundTimeout work, but maybe that isn't terribly critical?

There's a lot more in EJB, but these 2 items are what we touched on today.


Is the dev on this dead?

Hi all,
Is the dev on this dead? Is this useable?

Hi there were a few bugs and errors.
I have it fixed.
Also I added threading G33 support.
There are still some minor issues, but everything is works perfect on my lathe.
You may check my fork here https://github.com/how-eee/grbl_lathe_UI_threading
Videos are also available there.
Feel free to ask questions on my page. Bdurbrow does not respond here.

I'm alive - but I haven't had any time to work on personal projects like this recently... and I'm pretty sure it's not gonna ease up for a few more months at least.

Still not dead - but although it's going to take a while to get back in the swing of things, I have unburied some of the machines and started on rebuilds. My water heater exploded - well, it didn't explode in the same way that the one in that episode of MythBusters did, but it did vent thru the overload valve filling the garage full of steam. Again. This had the predictable result of rusting most of the cast iron surfaces in there. Oh well... I always wanted to learn hand-scraping anyways, I guess?


Labelling figures within body

I'm having an issue with labelling figures that are embedded within a body. If the figure is outside of the body, then the label works. However, that means that I cannot place the figure in the middle of the content. If the figure is inside the body, then the label is not found. Is this a known issue / by design? I may have missed something in the documentation, but I could not find any reference to this. If anyone is willing to offer some pointers, I'd also happily try to submit a PR to change this behaviour.
#let solution(body) = locate(location => {
  [#body]
})

// works, but cannot embed figure in the body
#solution[
    Figure label: @linking
]
#figure("bonjour") <linking>

// does not work: "label does not exist in the document"
#solution[
    Figure label: @linking1
    #figure("bonjour") <linking1>
]

Ah, I see. I tried to read through the source code but could not figure out the structure well enough to attempt a fix. I'd be happy to try if I could get some guidance on how these compilation passes work, but am not sure of how much time I can dedicate. The labelling (to me) is not critical. However, it would be a nice feature to have and seems like expected behavior.

Should be fixed now because we ignore errors in earlier passes if they go away in later ones.


Dynamically changing page size in datagrid may throw an error

Describe the bug
Having
<clr-dg-pagination #pagination [clrDgPageSize]="pageSize">

if the bound variable pageSize is changed to a size bigger than before, an exception is observed:
preview-d2335ca5bdb955611c1cb.js:1 ERROR TypeError: Cannot read property 'get' of undefined
    at ClrDatagridRow.get [as _view] (datagrid-row.ts:229)
    at eval (datagrid.ts:216)
    at Array.forEach (<anonymous>)
    at QueryList.forEach (query_list.ts:100)
    at eval (datagrid.ts:215)
    at SafeSubscriber.schedulerFn [as _next] (event_emitter.ts:116)
    at SafeSubscriber.__tryOrUnsub (Subscriber.ts:265)
    at SafeSubscriber.next (Subscriber.ts:207)
    at Subscriber._next (Subscriber.ts:139)
    at Subscriber.next (Subscriber.ts:99)

How to reproduce
Please provide a link to a reproduction scenario using one of the Clarity Stackblitz templates. Reports without a clear reproduction may not be prioritized until one is provided.
Steps to reproduce the behavior:

Go to https://stackblitz.com/edit/datagrid-page-size-issue?file=src/app/app.component.html
Click on '2'
Then click on '8'
See the error in the developer console

Expected behavior
No error on page size change.
Versions
App

Clarity: 2.1.0

Additional notes

What causes the issue is the get method in the wrappedInjector of the view getter here:

  datagrid-row.ts:
  ----------------
  public get _view() {
    return this.wrappedInjector.get(WrappedRow, this.vcr).rowView;
  }

If the value in Page#size is changed directly, the page size is updated without error.
Also a chocolate error is thrown, which is intentionally suppressed in the demo, but I'm not really sure it's expected.

@Jinnie
Could you triage this and then add an estimate and move it to the backlog if you feel it's valid?

Updated stackblitz:
https://stackblitz.com/edit/clarity-dark-theme-clr12-cds5-syppty?file=src%2Fapp%2Fapp.component.ts
Check:

Conclusion: Yes, it's still valid.


Models section improvements

Customizing models section should be broken up into each type thats within it, rather than one overarching h3
Rename property types to supported types
List relationships after supported types

Done.


version:10.0.19044.1320

How to edit the rdpwrap.ini to make RDPConf available?

Duplicate of https://github.com/stascorp/rdpwrap/issues/1574


Making some seo changes

Updating SEO

✅ By submitting this PR, I have verified the following

[x] Checked to see if a similar PR has already been opened 🤔
[x] Reviewed the contributing guidelines 🔍
[x] Added my name to the bottom of the list under the Contributors section in the README.md with a link to my personal website or GitHub profile 👥

Yeah lets do that. I'll take care of this tonite.


Single docker toolchain image for vanilla and spacetime OCaml

We should have a single docker image that contains switches for both vanilla and the spacetime variant of OCaml.
For the most part, both switches can install the same packages. For spacetime, we also want to install the packages for prof_spacetime, and make sure the stack is unlimited. See #3471 for more details.

Basically we have to modify the toolchain docker.

Implemented in #3519.


xtreemfs: switch to python3

Description of changes
AFAICT python is only needed during the build process and this builds fine. I don't have the means to properly test things, though.

Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[x] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[x] Tested basic functionality of all binary files (usually in ./result/bin/)
22.05 Release Notes (or backporting 21.11 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[ ] Fits CONTRIBUTING.md.

Hmm, demo.xtreemfs.org has since decayed… pity


IR-119: Add IgnoreInvalidRefs for image pruner

Invalid image references in pods/builds/etc cause the image pruner to abort its job. This is the safest approach because even if the pruner has bug, it won't delete anything that is in use. But as there is no validation for image references, some clusters may have invalid references because of their users mistakes and the pruner can ignore them.
The default value for ignoreInvalidRefs is false to maintain backward compatibility. But the operator will bootstrap the configuration object with ignoreInvalidRefs: true. So new clusters will have ignoreInvalidRefs: true, upgraded from 4.5 clusters will have ignoreInvalidRefs: false.

/assign @bparees

general concept seems fine, suggested a rename of the field though

/retest

/lgtm


NO-ISSUE: Synchronize From Upstream Repositories

The staging/ and vendor/ directories have been synchronized from the upstream repositories, pulling in the following commits:

Date
Commit
Author
Message

2024-03-27 09:41:07
operator-framework/@60f7a3f
dtfranz
UPSTREAM: : Add OpenShift specific files

2024-03-27 09:41:07
operator-framework/@8ce094d
Alexander Greene
UPSTREAM: : Drop commitchecker

2024-03-27 09:41:07
operator-framework/@d63f783
AOS Automation Release Team
UPSTREAM: : Updating ose-olm-catalogd-container image to be consistent with ART Reconciling with https://github.com/openshift/ocp-build-data/tree/e5df10e41b14c724468c8f78e2690f219279bfc4/images/ose-olm-catalogd.yml

2024-03-27 09:41:07
operator-framework/@b50a4d3
AOS Automation Release Team
UPSTREAM: : Updating ose-olm-catalogd-container image to be consistent with ART Reconciling with https://github.com/openshift/ocp-build-data/tree/4022cd290f00a44d667dda03f2d78d84a488c7ed/images/ose-olm-catalogd.yml

2024-03-27 09:41:07
operator-framework/@5ce4d60
Andy Goldstein
UPSTREAM: : Remove ncdc from OWNERS

2024-03-27 17:23:12
operator-framework/@820da40
Mikalai Radchuk
UPSTREAM: : update owners

2024-04-08 09:45:05
operator-framework/@94ae2be
AOS Automation Release Team
UPSTREAM: : Updating ose-olm-catalogd-container image to be consistent with ART for 4.16 Reconciling with https://github.com/openshift/ocp-build-data/tree/cd35345d4d061d63bddd2911a9c77149106dee55/images/ose-olm-catalogd.yml

2024-05-08 14:32:27
operator-framework/@79975a5
Ilias Rinis
UPSTREAM: : manifests: set required-scc for openshift workloads

2024-06-17 17:26:04
operator-framework/@c524c4f
Todd Short
UPSTREAM: : Update generate-manifests.sh

This pull request is expected to merge without any human intervention. If tests are failing here, changes must land upstream to fix any issues so that future downstreaming efforts succeed.
/cc @openshift/openshift-team-operator-runtime
/cc @openshift/openshift-team-operator-ecosystem

/label tide/merge-method-merge

/retest

/retest

/retest

/retest

/retest

/lgtm
/approve

/retest

/retest

/test openshift-e2e-aws

It was failing on... when A ClusterCatalog is created... which is a catalogd-specific test.

/retest

/override openshift-e2e-aws
Catalogd is temporarily removed from the payload in order to merge a breaking change across two repos.

/override ci/prow/openshift-e2e-aws


extension is not yet activated

I was able to use GitHub Copilot Chat until the 1st of August. on the 2nd of August I suddenly found that copliot chat was not answering questions. I thought it would be back the next day.
I tried

Logging out and logging back in
Remove and reinstall GitHub Copilot Chat.
Uninstall and reinstall VSCODE.
and others
But the problem is not fixed.

2024-08-03 02:53:19.964 [info] [FetcherService] Using the Helix fetcher.
2024-08-03 02:53:19.964 [info] [gitExtensionService] Initializing Git extension service.
2024-08-03 02:53:19.964 [info] [RemoteAgents] Unable to fetch remote agents because user is not signed in.
2024-08-03 02:53:19.964 [info] [gitExtensionService] Successfully activated the vscode.git extension.
2024-08-03 02:53:19.964 [info] [gitExtensionService] Enablement state of the vscode.git extension: true.
2024-08-03 02:53:19.964 [info] [gitExtensionService] Successfully registered Git commit message provider.
2024-08-03 02:53:20.425 [info] [auth] Logged in as Neoisava
2024-08-03 02:53:20.436 [info] [auth] Logged in as Neoisava
2024-08-03 02:53:20.448 [info] [auth] Logged in as Neoisava
2024-08-03 02:53:20.459 [info] [auth] Logged in as Neoisava
2024-08-03 02:53:20.468 [info] [auth] Logged in as Neoisava
2024-08-03 02:53:22.102 [info] [auth] Got Copilot token for Neoisava
2024-08-03 02:53:22.128 [info] [auth] Got Copilot token for Neoisava
2024-08-03 02:53:22.133 [info] [chat] copilot token chat_enabled: true
2024-08-03 02:53:22.133 [info] [githubTitleAndDescriptionProvider] GitHub.vscode-pull-request-github extension is not yet activated.
2024-08-03 02:53:22.188 [info] [auth] Got Copilot token for Neoisava
2024-08-03 02:53:22.189 [info] [auth] Got Copilot token for Neoisava
2024-08-03 02:53:22.193 [info] [auth] Got Copilot token for Neoisava

Hi could you please explain what exactly you mean by 'copilot was not answering questions'? A screen recording would be helpful and help us understand the source of the problem.


Merge the frontend repository with upstream (Sharetribe Flex)

The repository needs to be merged with upstream to integrate the last enhancements.
This will be developed on branch merge-with-upstream.

This is live on merge-with-upstream. Two problem have been noticed:

The DealsPage isn't loading, probably because of this error:

Filters are not correctly shown on SearchPage:

Also, we will have to properly test on Mobile before launching

What have been done: We merged with version v2.16.0 of FTW
Pushed to: merge-with-upstream
Tag somebody: @Shane-Walsh
PR: #208
Asking for validation:

Check on mobile and on desktop if the issue have been correctly fixed.
If everything is correct, move the card to accepted column.
If the issue isn't fixed, re-open the comment clearly what the problem is and what is the desired result.
Move the card to have issues column.
Merge (or express approval in) the corresponding PR if needed. Move the card to the launch column.


perseus export --serve does not work in 0.4.0-beta.12

This issue is reporting a bug in the code of Perseus. Details of the scope will be available in issue labels.
The author described their issue as follows:

perseus export --serve does not work in 0.4.0-beta.12: I get a build error that I suspect is triggered by building Perseus with neither --cfg=engine nor --cfg=client (because that would result in no definition of web_log, which is the first error):

  [1/3] 📦 Exporting your app's pages...✅
  [2/3] 🏗  Building your app to Wasm...✅
...
   Compiling perseus v0.4.0-beta.12
error[E0432]: unresolved import `crate::web_log`
   --> /home/marienz/.cargo/registry/src/github.com-1ecc6299db9ec823/perseus-0.4.0-beta.12/src/lib.rs:157:13
    |
157 |     pub use crate::web_log;
    |             ^^^^^^^^^^^^^^ no `web_log` in the root

error[E0432]: unresolved imports `crate::PerseusNodeType`, `crate::Request`
   --> /home/marienz/.cargo/registry/src/github.com-1ecc6299db9ec823/perseus-0.4.0-beta.12/src/lib.rs:160:21
    |
160 |     pub use crate::{PerseusNodeType, Request};
    |                     ^^^^^^^^^^^^^^^  ^^^^^^^ no `Request` in the root
    |                     |
    |                     no `PerseusNodeType` in the root

error[E0432]: unresolved import `crate::PerseusNodeType`
 --> /home/marienz/.cargo/registry/src/github.com-1ecc6299db9ec823/perseus-0.4.0-beta.12/src/translator/lightweight.rs:3:5
  |
3 | use crate::PerseusNodeType;
  |     ^^^^^^^^^^^^^^^^^^^^^^ no `PerseusNodeType` in the root

error[E0432]: unresolved import `crate::utils::PerseusDuration`
 --> /home/marienz/.cargo/registry/src/github.com-1ecc6299db9ec823/perseus-0.4.0-beta.12/src/template/core/setters.rs:2:5
  |
2 | use crate::utils::PerseusDuration;
  |     ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ no `PerseusDuration` in `utils`

error[E0432]: unresolved imports `decode_time_str::ComputedDuration`, `decode_time_str::PerseusDuration`
  --> /home/marienz/.cargo/registry/src/github.com-1ecc6299db9ec823/perseus-0.4.0-beta.12/src/utils/mod.rs:25:27
   |
25 | pub use decode_time_str::{ComputedDuration, InvalidDuration, PerseusDuration}; /* These have dummy equivalents for the browser */
   |                           ^^^^^^^^^^^^^^^^                   ^^^^^^^^^^^^^^^ no `PerseusDuration` in `utils::decode_time_str`
   |                           |
   |                           no `ComputedDuration` in `utils::decode_time_str`

error[E0392]: parameter `M` is never used
   --> /home/marienz/.cargo/registry/src/github.com-1ecc6299db9ec823/perseus-0.4.0-beta.12/src/init.rs:111:36
    |
111 | pub struct PerseusAppBase<G: Html, M: MutableStore, T: TranslationsManager> {
    |                                    ^ unused parameter
    |
    = help: consider removing `M`, referring to it in a field, or using a marker such as `PhantomData`

error[E0392]: parameter `T` is never used
   --> /home/marienz/.cargo/registry/src/github.com-1ecc6299db9ec823/perseus-0.4.0-beta.12/src/init.rs:111:53
    |
111 | pub struct PerseusAppBase<G: Html, M: MutableStore, T: TranslationsManager> {
    |                                                     ^ unused parameter
    |
    = help: consider removing `T`, referring to it in a field, or using a marker such as `PhantomData`

error[E0392]: parameter `G` is never used
  --> /home/marienz/.cargo/registry/src/github.com-1ecc6299db9ec823/perseus-0.4.0-beta.12/src/reactor/mod.rs:71:20
   |
71 | pub struct Reactor<G: Html> {
   |                    ^ unused parameter
   |
   = help: consider removing `G`, referring to it in a field, or using a marker such as `PhantomData`

Some errors have detailed explanations: E0392, E0432.
For more information about an error, try `rustc --explain E0392`.
error: could not compile `perseus` due to 8 previous errors
warning: build failed, waiting for other jobs to finish...

The steps to reproduce this issue are as follows:

Install Perseus CLI 0.4.0-beta.12
Run perseus export -s in a project using perseus::main_export (the Perseus website at version 0.4.0-beta12 and a small example fail the same way).

A minimum reproducible example is available at <>.

Hydration-related: false
The author is willing to attempt a fix: true

Tribble internal data
dHJpYmJsZS1yZXBvcnRlZCxDLWJ1ZyxBLWNsaSxhdXRob3Itd2lsbGluZy10by1pbXBs

Yes, I'm aware of this happening only sometimes, and I've yanked beta 12 accordingly. To my understanding for now, this would appear to be a duplicate of #246, because I've only observed this error when using cargo-clif. Are you seeing this with the default compiler? (Note that this may well be a bug that only occurs sometimes.)

Note that I was able to get the website working perfectly after cargo clean without using the experimental compiler backend.

Actually, now I think about it, this could just be a cargo clean problem coming from the old beta versions. If you're able to reproduce this after a cargo clean && perseus clean (remember Perseus maintains its own target/ directories in dist/), it's definitely a bug.

Now getting this in CI for the website on the new beta 13 (which fixed some fatal bugs from beta 12)...but not locally...

Working theory: lockfile generation is not passed the RUSTFLAGS, meaning it will only work if there's a .cargo/config.toml in your project setting the necessary flags.

@marienz I think this issue is fixed now in v0.4.0-beta.14, would you mind giving that a try? If you're still having issues, do try a cargo clean && perseus clean. I'll close this for now.

I am not using cargo-clif (I only found out about it just now, from this bug and #246).
I can no longer reproduce the problem after upgrading perseus-cli to 0.4.0-beta.14. Thanks for the fix!


MM-27208 MM-T1682 Add E2E test for joining an archived channel via a permalink

Ticket Link
https://mattermost.atlassian.net/browse/MM-27208
https://mattermost.atlassian.net/projects/MM?selectedItem=com.atlassian.plugins.atlassian-connect-plugin%3Acom.kanoah.test-manager__main-project-page#!/testCase/MM-T1682

Cherry pick is scheduled.


Sandboxed Decryption

From @winhowes on March 6, 2015 3:47
We should have a mode which isolates the decrypted text from the page to prevent malicious JS from reading the decrypted text yet without significantly impacting the page's usability or the decrypted text's appearance.
Copied from original issue: winhowes/grd.me#33

From @winhowes on March 6, 2015 3:51
Implemented on Firefox (toggled off by default).  Only thing left on FF would be to securely (as in no xss) get the @font-face css properties copied into the iframe to keep the decrypted text's look consistent.

From @winhowes on March 6, 2015 3:51
Implemented on Firefox (toggled off by default).  Only thing left on FF would be to securely (as in no xss) get the @font-face css properties copied into the iframe to keep the decrypted text's look consistent.

I can confirm that this works on firefox up to version 39, but does not work on with e10s; e10s has only been tested on v39, as it is only enabled in nightly, not beta, aurora or dev, so yeah. logs forthcoming

From @winhowes on March 11, 2015 12:5
This should now work with e10s, but need confirmation. I'm also adding a performance label because though it is implemented, performance is still pretty poor on sites with multiple ciphertexts. What do you think about adding a mode where you have to click to decrypt a message - that way only one message is being decrypted at a time for better performance? If we do this, it may be a FF only feature

From @winhowes on March 11, 2015 12:5
This should now work with e10s, but need confirmation. I'm also adding a performance label because though it is implemented, performance is still pretty poor on sites with multiple ciphertexts. What do you think about adding a mode where you have to click to decrypt a message - that way only one message is being decrypted at a time for better performance? If we do this, it may be a FF only feature

I can confirm that this works on firefox up to version 39, but does not work on with e10s; e10s has only been tested on v39, as it is only enabled in nightly, not beta, aurora or dev, so yeah. logs forthcoming

I like the concept of the selective decryption, if you do it I would suggest billing it as a privacy/security feature, as it kind of is.
Also, as of today's build firefox-nightly v39.0a1, running with [$ cfx  --e10s -b /bin/firefox-nightly run ], the secure decryption is still not working; there is improvement, as it now correctly identifies undecipherable cyphertext, but still hangs on decryptable text.

I like the concept of the selective decryption, if you do it I would suggest billing it as a privacy/security feature, as it kind of is.
Also, as of today's build firefox-nightly v39.0a1, running with [$ cfx  --e10s -b /bin/firefox-nightly run ], the secure decryption is still not working; there is improvement, as it now correctly identifies undecipherable cyphertext, but still hangs on decryptable text.

also, no issues with v39.0a1 non e10s: sandboxed decryption fully functional, albeit it slower

also, no issues with v39.0a1 non e10s: sandboxed decryption fully functional, albeit it slower

From @winhowes on March 13, 2015 13:19
Weird, yeah when e10s enabled, the scripts don't attach to the generated frame. But there's no errors..
I'm going to keep looking, but here's the question on stackoverflow for reference: http://stackoverflow.com/questions/29033866/content-scripts-dont-attach-to-intercepted-frame-on-firefox-nightly-with-e10s

From @winhowes on March 13, 2015 13:19
Weird, yeah when e10s enabled, the scripts don't attach to the generated frame. But there's no errors..
I'm going to keep looking, but here's the question on stackoverflow for reference: http://stackoverflow.com/questions/29033866/content-scripts-dont-attach-to-intercepted-frame-on-firefox-nightly-with-e10s

awesome; np with the close

From @winhowes on March 15, 2015 23:54
The actual implementation of this works, but each browser has different issues with this, so I'm gonna make two new tickets for those issues but close this one.

awesome; np with the close

From @winhowes on March 15, 2015 23:54
The actual implementation of this works, but each browser has different issues with this, so I'm gonna make two new tickets for those issues but close this one.


macho binaries missing LC_VERSION_MIN_MACOSX

The go linker does not specify a min OS X version.
When this is missing lldb cannot identify the os of the binary.
In external link mode the system linker adds LC_VERSION_MIN_MACOSX, set to the current OS X version.
This means only programs compiled in external link mode can be debugged with lldb.
See http://lists.llvm.org/pipermail/lldb-dev/2015-October/008555.html

I'm using go version go1.5.1 darwin/amd64


ts-node-dev .fork() results in 100% cpu and an infinite loop in waiting for a compiled file

Issue description
I am calling `child_process.fork(pathToATypescriptFile, ['--someParam'], {
  env: {
    ...process.env,
    MY_ENV_VAR: 'foo',
  }
});

What I'm seeing is 100% CPU usage. When debugged the same command with --inspect I see an infinite loop
# ps aux | grep server-socket
rush     11915  100  0.0<PHONE_NUMBER>8 pts/6    Rl+  23:35   0:26 /home/rush/.config/nvm/14.15.0/bin/node -r /tmp/ts-node-dev-hook-708417323754666.js --inspect -r tsconfig-paths/register /code/code2flow-magma-integration/node_modules/ts-node-dev/lib/wrap.js /code/server/server-socket.ts --colors

debug output is not showing anything interesting aside of:
[DEBUG] 23:35:31 /code/server/server-socket.ts added to watcher

When I added a breakpoint to by running the forked process by hand with the same parameter, it seems to be stuck in this while(true) loop:

Any idea how to debug, fix this?
Context
OS version (is it docker or host?), ts-node-dev version host, 1.1.8
Did you try to run with ts-node? N/A
Did you try to run with --files option enabled? yes
Did you try to run with --debug option enabled? yes
Do you have a repro example (git repo) with simple steps to reproduce your problem? not yet

Any tips how to debug it further and possibly fix?

Can you tell me how you connected a debugger to ts-node-dev?

Can you tell me how you connected a debugger to ts-node-dev?

@ATheCoder not OP but I'm doing it like  this NODE_ENV=development ts-node-dev --inspect=<IP_ADDRESS>:9221 --exit-child -- ./src/myScript.ts

I've encountered this issue as well and have subsequently switched back to nodemon which works fine.

What I did to debug was to take what was in ps aux and and then re-run manually using --inspect.
node --inspect --max-old-space-size=2048 /code/project/node_modules/ts-node-dev/lib/wrap.js /code/project/worker.js --colors

I'm also getting this problem.

I also have this problem with a larger monorepo where we start multiple ts-node-dev watchers in parallel via tilt for the different services.
This issue only appears for me on macOS, it works fine on Ubuntu.
With some debug output and the timeThreshold in child-require-hook.ts set to 20_000 and a counter in the while(true) loop there I get the following output:
dbg child-require-hook.js compile_process.send {
  compile: '/path/to/monorepo/packages/library/src/util/some-url.transformer.ts',
  compiledPath: '/var/folders/48/hj0ldnq91kg8qn6hsx4ls22m0000gn/T/.ts-node1OZnQ1/compiled/___library_src_util_some_url_transformer_ts_33a17fac1d00b3b0aeccedefe6d3ed9a0b7f99e28deefc0a4a4ff223b65b67f3.js'
}
dbg child-require-hook.js waitForFile 100.000 loops for file /var/folders/48/hj0ldnq91kg8qn6hsx4ls22m0000gn/T/.ts-node1OZnQ1/compiled/___library_src_util_some_url_transformer_ts_33a17fac1d00b3b0aeccedefe6d3ed9a0b7f99e28deefc0a4a4ff223b65b67f3.js.done start<PHONE_NUMBER>100 passed 726 timeThreshold 20000
Error: Could not require /var/folders/48/hj0ldnq91kg8qn6hsx4ls22m0000gn/T/.ts-node1OZnQ1/compiled/___library_src_util_some_url_transformer_ts_33a17fac1d00b3b0aeccedefe6d3ed9a0b7f99e28deefc0a4a4ff223b65b67f3.js.done
    at waitForFile (/private/var/folders/48/hj0ldnq91kg8qn6hsx4ls22m0000gn/T/ts-node-dev-hook-9647777766045864.js:42:19)
    at compile (/private/var/folders/48/hj0ldnq91kg8qn6hsx4ls22m0000gn/T/ts-node-dev-hook-9647777766045864.js:71:5)
    at Module.m._compile (/private/var/folders/48/hj0ldnq91kg8qn6hsx4ls22m0000gn/T/ts-node-dev-hook-9647777766045864.js:81:44)
    at Module._extensions..js (node:internal/modules/cjs/loader:1159:10)
    at require.extensions..jsx.require.extensions..js (/private/var/folders/48/hj0ldnq91kg8qn6hsx4ls22m0000gn/T/ts-node-dev-hook-9647777766045864.js:126:20)
    at require.extensions.<computed> (/private/var/folders/48/hj0ldnq91kg8qn6hsx4ls22m0000gn/T/ts-node-dev-hook-9647777766045864.js:83:20)
    at Object.nodeDevHook [as .ts] (/path/to/monorepo/node_modules/ts-node-dev/lib/hook.js:63:13)
    at Module.load (node:internal/modules/cjs/loader:981:32)
    at Function.Module._load (node:internal/modules/cjs/loader:822:12)
    at Module.require (node:internal/modules/cjs/loader:1005:19)
[DEBUG] 16:29:08 /path/to/monorepo/packages/library/src/util/some-url.transformer.ts added to watcher
dbg index.js compiler.compile {
  compile: '/path/to/monorepo/packages/library/src/util/some-url.transformer.ts',
  compiledPath: '/var/folders/48/hj0ldnq91kg8qn6hsx4ls22m0000gn/T/.ts-node1OZnQ1/compiled/___library_src_util_some_url_transformer_ts_33a17fac1d00b3b0aeccedefe6d3ed9a0b7f99e28deefc0a4a4ff223b65b67f3.js'
}
dbg compiler.js compile before _compile
dbg compiler.js compile_writeCompiled_outer /var/folders/48/hj0ldnq91kg8qn6hsx4ls22m0000gn/T/.ts-node1OZnQ1/compiled/___library_src_util_some_url_transformer_ts_33a17fac1d00b3b0aeccedefe6d3ed9a0b7f99e28deefc0a4a4ff223b65b67f3.js
[DEBUG] 16:29:08 /path/to/monorepo/packages/library/src/util/some-url.transformer.ts compiled in 3 ms
dbg compiler.js compile after _compile
[DEBUG] 16:29:08 Child error
[ERROR] 16:29:08 Error: Could not require /var/folders/48/hj0ldnq91kg8qn6hsx4ls22m0000gn/T/.ts-node1OZnQ1/compiled/___library_src_util_some_url_transformer_ts_33a17fac1d00b3b0aeccedefe6d3ed9a0b7f99e28deefc0a4a4ff223b65b67f3.js.done
[DEBUG] 16:29:08 Disconnecting from child
[DEBUG] 16:29:08 Sending SIGTERM kill to child pid 64763
dbg compiler.js compile_writeCompiled_inner /var/folders/48/hj0ldnq91kg8qn6hsx4ls22m0000gn/T/.ts-node1OZnQ1/compiled/___library_src_util_some_url_transformer_ts_33a17fac1d00b3b0aeccedefe6d3ed9a0b7f99e28deefc0a4a4ff223b65b67f3.js
dbg compiler.js compile_writeCompiled_inner_done /var/folders/48/hj0ldnq91kg8qn6hsx4ls22m0000gn/T/.ts-node1OZnQ1/compiled/___library_src_util_some_url_transformer_ts_33a17fac1d00b3b0aeccedefe6d3ed9a0b7f99e28deefc0a4a4ff223b65b67f3.js
[DEBUG] 16:29:08 Child exited with code null

It seems like the node event loop gets somehow stuck as indicated by the writeCompiled output being split into the output before writeFile in compiler.ts and the output running in its callback completely at the end.
Using writeFileSync in this place does not fix the issue, as the error still gets logged out before writeCompiled.
It seems that the while (true) in child-require-hook.ts loop plays a role here by keeping the event loop busy in some scenarios (many files? multiple node processes?) .

This is a pretty serious issue imo. We're experiencing this too. I'm not sure how actively maintained ts-node-dev is but it would be great to get a fix for this.

We have worked around this by statically importing the file before running it with fork, which seems to work most of the time 😬


Citations not showing on landing page

Citations stopped showing on the landing page. This is different from: https://github.com/jonfroehlich/makeabilitylabwebsite/issues/758. See this video: https://youtu.be/xs8_mLw0D4s. See also the screenshot below.

I asked @lstearns86 about this and he said, wisely, that it was due to my new addition of the line-clamp-one-line class that I placed so that listed projects would not overrun this line.

It looks like the issue is from the "overflow:hidden" style on the line-clamp-one-line class. If I disable that style then the citation shows again (it's actually showing it, but only what fits within that one line, which is entirely blank). If you set the paragraph style to "overflow:visible" then that should override the setting and show the citation hopefully without breaking your other content (although you'll need to find another solution for when that line is too long). Another option would be to target an element outside of that line to contain the citation popup, but that would probably require some custom javascript (I don't quite remember what approach I used to do this, but I think it was something built into one of the libraries we were using).

And, indeed, this worked!

In terms of tracking this down and recommendations to fix, Lee said:

It was an educated guess, followed by some searching up the DOM tree and trial and error. If you hover over the div in the developer tools it shows the outline of where it should be, which usually suggests that either a parent element is using overflow:hidden or something is messed up with the z-indices.
You can't negate overflow in a parent class from a child, so the only options are to remove the line-clamp-one-line class from the parent  element, override it using !important in the pub-download-links class, or move the citation popover elements to some parallel structure. Maybe place it in a separate element that gets pushed all the way to the left or right, so that you can still apply the line clamp to the rest of the links?

Made a quick fix by removing the line-clamp-one-line
Going to implement a "real fix" and document it here: https://github.com/jonfroehlich/makeabilitylabwebsite/issues/758


[TASK] Update H2 Jar

Description
There is an update to H2 that is not compatible with our current code
Existing
Reference
Tasks
Need to update the code to allow the H2 update

We have to configure and install Rclone with our kanini local desktop.

We are still working on the installation of RClone in Kanini Desktop. Due to the Firewall issue we are not able install.

Raised PR for review.

Based on Maher's review comments, the names of the downloaded older mv.db file and the migrated mv.db file need to be changed.

We have changed the code and zip file for rename of db files and had push to git hub.


wip: Require type arguments for generic class instantiation

Motivation

Test plan

See included automated tests.

We have a policy of testing changes to Sorbet against Stripe's codebase before
merging them. I've kicked off a test run for the current PR.  When the build
finishes, I'll share with you whether or how it failed. Thanks!
Stripe employees can see the build results here:
→ https://go/builds/bui_M8gSk5VmMwnJYn
→ https://go/builds/bui_M8gSKjPEYZTFRY

We have a policy of testing changes to Sorbet against Stripe's codebase before
merging them. I've kicked off a test run for the current PR.  When the build
finishes, I'll share with you whether or how it failed. Thanks!
Stripe employees can see the build results here:
→ https://go/builds/bui_M8gbWPr0XqJu1R
→ https://go/builds/bui_M8gbgJ0uM3Ldwf

We have a policy of testing changes to Sorbet against Stripe's codebase before
merging them. I've kicked off a test run for the current PR.  When the build
finishes, I'll share with you whether or how it failed. Thanks!
Stripe employees can see the build results here:
→ https://go/builds/bui_M8giaXy4nHhSFZ
→ https://go/builds/bui_M8giblcfn22l9z

We have a policy of testing changes to Sorbet against Stripe's codebase before
merging them. I've kicked off a test run for the current PR.  When the build
finishes, I'll share with you whether or how it failed. Thanks!
Stripe employees can see the build results here:
→ https://go/builds/bui_M8gjfk4TRtSKLo
→ https://go/builds/bui_M8gjJais0OelW8

We have a policy of testing changes to Sorbet against Stripe's codebase before
merging them. I've kicked off a test run for the current PR.  When the build
finishes, I'll share with you whether or how it failed. Thanks!
Stripe employees can see the build results here:
→ https://go/builds/bui_M8sHWgv4IH6UpO
→ https://go/builds/bui_M8sH7oJQBDLs6A

We have a policy of testing changes to Sorbet against Stripe's codebase before
merging them. I've kicked off a test run for the current PR.  When the build
finishes, I'll share with you whether or how it failed. Thanks!
Stripe employees can see the build results here:
→ https://go/builds/bui_M8sip5xU2YN5fM
→ https://go/builds/bui_M8silwXZYGS1Id

We have a policy of testing changes to Sorbet against Stripe's codebase before
merging them. I've kicked off a test run for the current PR.  When the build
finishes, I'll share with you whether or how it failed. Thanks!
Stripe employees can see the build results here:
→ https://go/builds/bui_M8tdbBJz9cVSLr
→ https://go/builds/bui_M8tdBNvIaAXJ1k

Putting this on hold for the moment because I think that we might be able to do something smarter here like inferring type arguments.


fix(enterprise): spacing adjustment for badges

What change does this PR introduce?
Fixes #1403
This PR fixes the spacing issue for location badges on the 'Enterprise Jobs' page.

BEFORE
AFTER

Thanks 😊


Typo fix - update to MVVM - putting things together article

Change (... which are simple models representing our posts, that like like this...) to (... posts, that are like this...).

Docs Build status updates of commit 01acfe5:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs/mvvm/PuttingThingsTogether.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel

Hi @mahab339, sorry for the delay in looking at this PR. This doc has been migrated to our new docs repo: https://github.com/MicrosoftDocs/CommunityToolkit/blob/main/docs/mvvm/PuttingThingsTogether.md - would you mind checking if this fix still needs to be applied there? Thanks!


win32/msys transition mishandled when invoking git subcommands

I'm not sure of the extent of the problem, but here's how I could reliably reproduce it, on a fresh install of msys2:
$ pacman -Suy mingw-w64-x86_64-make mingw-w64-x86_64-gettext git
$ echo foo: > foo.mk
$ echo -e \\tgit init foo >> foo.mk
$ echo -e \\tgit clone foo bar >> foo.mk
$ mingw32-make -f foo.mk -B

This fails with the following error:
ssh: Could not resolve hostname c: Name or service not known
fatal: Could not read from remote repository.

Running with trace enabled is somewhat enlightening:
$ GIT_TRACE=1 mingw32-make -f foo.mk -B
(...)
git.c: 371     trace: builtin: git 'clone' 'foo' 'bar'
Cloning into 'bar'...
run-command.c:369    trace: run_command: 'ssh' 'C' 'git-upload-pack '\''/msys64/home/glandium/foo'\'''
ssh: Could not resolve hostname c: Name or service not known
fatal: Could not read from remote repository.

Now, the reason the issue subject mentions win32/msys transition is because:

mingw32-make -d mentions calling CreateProcess(C:\msys64\usr\bin\git.exe, ...)
Changing the last line of foo.mk to set a dummy environment variable on the git clone command line works around the issue:

$ echo foo: > foo.mk
$ echo -e \\tgit init foo >> foo.mk
$ echo -e \\tFOO= git clone foo bar >> foo.mk
$ mingw32-make -f foo.mk -B
(...)
warning: You appear to have cloned an empty repository.
done.

In that case, mingw32-make -d mentions calling CreateProcess(C:\msys64\usr\bin\sh.exe, ...) because the dummy environment variable forces it to wrap the git command with a shell.
Interestingly, I cannot reproduce by just invoking git clone foo bar from a cmd shell with the right PATH variables set. It looks like in that case whatever is resolving the path for foo to a full absolute path resolves it to a msys path (/c/...) rather than a windows path (c:...) which seems to be what happens when mingw32-make executes git.
This may or may not be caused by the git-2.8.2-Cygwin-Allow-DOS-paths.patch patch, or something related.

Under MSYS2 always use MSYS make not MINGW make.
mingw32-make is designed to run under CMD

I am invoking mingw32-make from cmd. I'm then invoking msys git from mingw32-make, which causes the problem. It actually worked with version 2.8.1, which is what appveyor has on its hosts before installing unrelated packages forced an upgrade of git to 2.12.0...

Well appveyor upgraded to 2.11.1 anyways, and that version fails too, so forcing the upgrade not to happen doesn't work around the problem.

You could try with https://github.com/git-for-windows/git/wiki/Install-inside-MSYS2-proper

@mati865 this works very well for me - why isn't this git client from git-for-windows, the default git client inside msys2 environment ? It appears to support mixed paths, both windows form and unix form.

@iongion I'm not the right person to answer this question.


Introduced block structure to build log, if output is not captured

This could be useful if pytest process got segmentation fault on teamcity (because of C module, for example) and need to use --capture=no.
(In such case scrolling build log is tedious)

Could I have any commnets of this? I do need to decide whether to publish pypi https://github.com/cielavenir/pytest-teamcity-logblock .

@cielavenir I'm currently on vacation and could not properly evaluate consequences of this patch
What do you want to do here exactly? Just separate setup/call/teardown output or more?

sorry that I forgot to reply, but I remember that teamcity build log screen is messed if the stdout is huge and --capture=no (and this patch changes the build log screen somewhat kind to browser).
maybe I could make another command-line option so that the plugin default behavior does not change.


Worker memory overuse when converting to nwb

When attempting to run:
from trodes_to_nwb.convert import create_nwbs
path = data_path # from step 1
output_dir = "/stelmo/temp/"

# for dual site recording:
probe_metadata_paths=['/cumulus/rio/BS28_raw_old/tetrode_12.5.yml',
                          '/cumulus/rio/BS28_raw_old/128c-4s8mm6cm-20um-40um-sl.yml'],

create_nwbs(
    path,
    output_dir=output_dir,
    header_reconfig_path='/cumulus/rio/BS28_raw_old/20231107/20231107_BS28.trodesconf',
    #probe_metadata_paths=['/home/mcoulter/tetrode_12.5.yml'],
    #probe_metadata_paths=None,
    probe_metadata_paths=probe_metadata_paths,
    convert_video=False,
    n_workers=8,
    query_expression=None,
)

I get this error:
Cell In[2], line 11
      7 # for dual site recording:
      8 probe_metadata_paths=['[/cumulus/rio/BS28_raw_old/tetrode_12.5.yml](http://localhost:6502/cumulus/rio/BS28_raw_old/tetrode_12.5.yml)',
      9                           '[/cumulus/rio/BS28_raw_old/128c-4s8mm6cm-20um-40um-sl.yml](http://localhost:6502/cumulus/rio/BS28_raw_old/128c-4s8mm6cm-20um-40um-sl.yml)'],
---> 11 create_nwbs(
     12     path,
     13     output_dir=output_dir,
     14     header_reconfig_path='[/cumulus/rio/BS28_raw_old/20231107/20231107_BS28.trodesconf](http://localhost:6502/cumulus/rio/BS28_raw_old/20231107/20231107_BS28.trodesconf)',
     15     #probe_metadata_paths=['[/home/mcoulter/tetrode_12.5.yml](http://localhost:6502/home/mcoulter/tetrode_12.5.yml)'],
     16     #probe_metadata_paths=None,
     17     probe_metadata_paths=probe_metadata_paths,
     18     convert_video=False,
     19     n_workers=8,
     20     query_expression=None,
     21 )

File [~/trodes_to_nwb/src/trodes_to_nwb/convert.py:179](http://localhost:6502/trodes_to_nwb/src/trodes_to_nwb/convert.py#line=178), in create_nwbs(path, header_reconfig_path, probe_metadata_paths, output_dir, video_directory, convert_video, n_workers, query_expression, disable_ptp)
    177 # print out error results
    178 for args, future in zip(argument_list, futures):
--> 179     result = future.result()
    180     if result is not True:
    181         print(args, result)

File [~/miniforge3/envs/trodes_to_nwb/lib/python3.12/site-packages/distributed/client.py:328](http://localhost:6502/miniforge3/envs/trodes_to_nwb/lib/python3.12/site-packages/distributed/client.py#line=327), in Future.result(self, timeout)
    326 self._verify_initialized()
    327 with shorten_traceback():
--> 328     return self.client.sync(self._result, callback_timeout=timeout)

File [~/miniforge3/envs/trodes_to_nwb/lib/python3.12/site-packages/distributed/client.py:336](http://localhost:6502/miniforge3/envs/trodes_to_nwb/lib/python3.12/site-packages/distributed/client.py#line=335), in Future._result(self, raiseit)
    334 if raiseit:
    335     typ, exc, tb = exc
--> 336     raise exc.with_traceback(tb)
    337 else:
    338     return exc

KilledWorker: Attempted to run task 'pass_func-4e6bdfebabe77fefd27ed79e0ab49e55' on 4 different workers, but all those workers died while running it. The last worker that attempt to run the task was tcp://<IP_ADDRESS>:40071. Inspecting worker logs is often a good next step to diagnose what went wrong. For more information see https://distributed.dask.org/en/stable/killed.html.

even when running the code on virga which should have more free memory. What can I do to alleviate this issue?

If you're converting large sessions (e.g. continuous overnight recording, lots of probes) you probably can't convert more than one session at a time on any of the computers.
I'd suggest just setting n_workers=1 for most cases


Establishing a Secure Reverse Proxy with Bridge for Accessing Internal Networks from the Public Internet.

I'm trying to figure out a method to securely access resources on an internal network from the public internet using the bridge tool. The setup involves three main components:

An internal machine within the private network.
A publicly accessible server with a known public IP address.
A client machine on the public internet seeking access to internal resources.

The process works as follows:
The internal machine initiates a connection to the public server using bridge, establishing a reverse SSH tunnel:
bridge -b :0 -p ssh://public_ip:12022
The public server runs bridge to listen for SSH connections and forward them to a SOCKS5 proxy:
bridge -b socks5://:1080 -p ssh://:12022
Clients can then access internal network resources through the SOCKS5 proxy on the public server:
curl -x socks5://public_ip:1080 http://internal_service_address
Am I right?
Regards,
Zhao

I guess the following command run on your internal machine will meet your needs.
I recommend using the ssh key to access the server will be safer.
bridge \
  -b socks5://:1080 \
  -b ssh://user@public_ip:22?identity_file=/root/.ssh/id_rsa \
  -p -

Then, you suggest the following steps?

On the internal machine:

bridge \
  -b socks5://:1080 \
  -b ssh://root@public_ip:22?identity_file=/root/.ssh/id_rsa \
  -p -

On the public server:

bridge -b socks5://:1080 -p ssh://:22

Use it as follows:

curl -x socks5://public_ip:1080 http://internal_service_address

The public server does nothing, just add the pub key of the internal machine to /root/.ssh/authorized_keys of it.

By saying The public server does nothing, do you mean the following step can be omitted?

On the public server:

bridge -b socks5://:1080 -p ssh://:22

Yep

Are the following two methods roughly equivalent?

The method discussed here:

bridge \
  -b socks5://:1080 \
  -b ssh://root@public_ip:22?identity_file=/root/.ssh/id_rsa \
  -p -

Pure ssh based method:
ssh public_ip -R 50000:localhost:22 "ssh -D 1080 localhost -p 50000"

Yep, you can do it in any of the 3 ways.
The bridge was originally designed with the need to pass through multiple jump machines.
e.g.
bridge \
  -b socks5://:1080 \
  -b ssh://root@internal_ip_2:22?identity_file=/root/.ssh/id_rsa \
  -b ssh://root@internal_ip_1:22?identity_file=/root/.ssh/id_rsa \
  -b ssh://root@public_ip:22?identity_file=/root/.ssh/id_rsa \
  -p -

It listened to internal_ip_2 1080 via 3 machines

Let me try to give alternative pure ssh-based counterparts to the above bridge method:

Using ProxyJump (-J) for simplicity:

ssh -J root@public_ip:22,root@internal_ip_1:22,root@internal_ip_2:22 \
     -i /root/.ssh/id_rsa \
     -fNT -R 1080:localhost \
     root@other_public_ip_1:22 \
     root@other_public_ip_2:22

Using ProxyCommand for more granular controls:

ssh -o "ProxyCommand ssh -W %h:%p root@public_ip -i /root/.ssh/id_rsa" \
    -o "ProxyCommand ssh -W %h:%p root@internal_ip_1 -i /root/.ssh/id_rsa" \
    -o "ProxyCommand ssh -W %h:%p root@internal_ip_2 -i /root/.ssh/id_rsa" \
    -i /root/.ssh/id_rsa \
    -fNT -R 1080:localhost \
    root@other_public_ip_1 \
    root@other_public_ip_2

Actually, I can't remember the command of ssh or other proxy tools

It is certain that even if they can be implemented, there will be numerous pitfalls and extreme complications.

Another wonderful tool is gost.

It is not the same as Bridge's goal, which leads to the difference in functionality, it can not be implemented to listen to the remote server's port, and as a parameter to the ssh ProxyCommand.


Copier command should include sub-command copy for copier 8.0.0

Hey @tlambert03
Super cool new Python repository template! I was just starting a new python package and decided to try this over the cookie cutter templates.
One thing I noticed with copier 8.0.0 (the default that gets installed with pip install copier) is that the command:
copier gh:pydev-guide/pyrepo-copier your-package-name

doesn't seem to work anymore and I get a Unknown sub-command 'gh:pydev-guide/pyrepo-copier'.
Instead, this command now works:
copier copy gh:pydev-guide/pyrepo-copier your-package-name

Edit: Just to confirm: In copier 7.2.0, that still used to work, so must have been a change in the recent 8.0.0 update

thanks!


.cruft.json contains hardcoded local path upon creation which causes Sync template action to fail

I created an empty repo yesterday which resultet in .cruft.json containing a local path -> https://github.com/scverse/spatialdata-plot/blob/main/.cruft.json
This causes the Sync template GH action to fail -> https://github.com/scverse/spatialdata-plot/actions/runs/3907895412

Hm, yeah. that might be it. So I should just overwrite it with the GH link?

yeah, that should work.
Here's a cruft.json that works: https://github.com/icbi-lab/infercnvpy/blob/main/.cruft.json

Thanks! I guess the Sync template action triggers at night?

it does, but you can also trigger it manually via the Actions menu

It actually still fails. Would you happen to know other locations where that path might've ended up? The GH search doesn't turn anything up for me

odd? Okay, as long as it works, all good :D Thanks again!


Rewrite views as class based views

Complete rewrite of views as CBV.
Working on django1.9 python3.5
Closes #9

@paltman Don't worry i will not take this as a nit-pick, all project have (or should have!) his conventions, I've already read Pinax standard but it's hard to all remember the first time, il will reread it again and again ^^
I will not having time for correcting the PRs today, but i will in the start of the week.

I replace single quote to double quotes and remove the print statement, should be OK now.


Duplicating comments for SCSS variables

How to replicate bug:
Original:
// Comment
$variable: 1rem;

Linted:
// Comment
$variable: 1rem;Comment$variable

Nevermind I feel like this is more of a stylelint issue after taking a look at your code. Thanks for the library btw!


nixos/mastodon: fix definition of mastodon-media-auto-remove

Description of changes
Fixes settings of systemd.services.mastodon-media-auto-remove: script and startAt were erroneously nested under serviceConfig, resulting in both the timer not being created, and an invalid config for the service itself.
Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[x] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
22.11 Release Notes (or backporting 22.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[x] Fits CONTRIBUTING.md.

Thank you!


possible errors in lstm.ipynb

there is a line in lstm.ipynb which is:
"self.logits = tf.layers.dense(self.outputs[-1], output_size)"
What is the meaning of select self.outputs[-1]?
It seems you are doing prediction for each time step,
however the default behavior of dynamic_rnn is time_major=False,
so self.outputs is of size [batch_size, time_step, state_size]
Can you clarify this?
btw, although in your code it works on. because batch_size = 1.
Thank you.

can you paste link to notebook?


why did you code common atoms this way？

COMMON_ATOMS = [('B', 0), ('B', -1),
('Br', 0), ('Br', -1), ('Br', 2),
('C', 0), ('C', 1), ('C', -1),
('Cl', 0), ('Cl', 1), ('Cl', -1), ('Cl', 2), ('Cl', 3),
('F', 0), ('F', 1), ('F', -1),
('I', -1), ('I', 0), ('I', 1), ('I', 2), ('I', 3),
('N', 0), ('N', 1), ('N', -1),
('O', 0), ('O', 1), ('O', -1),
('P', 0), ('P', 1), ('P', -1),
('S', 0), ('S', 1), ('S', -1),
('Se', 0), ('Se', 1), ('Se', -1),
('Si', 0), ('Si', -1)]
when I look your code, I don't understand why you code common_atoms this way ,could you please tell me what the reason for this is?

The first letter is the atom type and the second number is its formal charge. Some atoms may be neutral while others are charged.

I know, I know. Thank you for responding to my question. What I want to ask is why you sort them like this, for example ('B', 0), ('B', -1) and ('Br', 0), ('Br', -1), ('Br', 2), etc. all start with 0 charge, then why 'I' starts with -1 charge，such as ('I', -1), ('I', 0), ('I', 1), ('I', 2), ('I', 3)?


Getting exception when try to build apk

I am implementing a app in which i have added fingerprint authentication so that's why I have changed "FlutterActivity" with "FlutterFragmentActivity" and changed theme with "Theme.AppCompat.Light.NoActionBar" but after that authentication works properly but when try to build apk i am getting following exception.
FAILURE: Build failed with an exception.

* What went wrong:
Execution failed for task ':app:lintVitalRelease'.
> Could not resolve all artifacts for configuration ':app:profileRuntimeClasspath'.
   > Failed to transform libs.jar to match attributes {artifactType=processed-jar, org.gradle.libraryelements=jar, org.gradle.usage=java-runtime}.
      > Execution failed for JetifyTransform: C:\Users\nobit\AndroidStudioProjects\Production\secret_keeper\build\app\intermediates\flutter\profile\libs.jar.
         > Transform's input file does not exist: C:\Users\nobit\AndroidStudioProjects\Production\secret_keeper\build\app\intermediates\flutter\profile\libs.jar. (See https://issuetracker.google.com/issues/158753935)

* Try:

Run with --stacktrace option to get the stack trace. Run with --info or --debug option to get more log output. Run with --scan to get full insights.

* Get more help at https://help.gradle.org

BUILD FAILED in 8m 58s
Running Gradle task 'assembleRelease'...                          542.8s (!)
Gradle task assembleRelease failed with exit code 1
Process finished with exit code 1

@jaydip-pawar
the issue could have been solved in this PR
please consider the possible solutions proposed here
if the problem persists please provide your flutter doctor -v and you complete flutter build apk -v

@jaydip-pawar
the issue could have been solved in this PR
please consider the possible solutions proposed here
if the problem persists please provide your flutter doctor -v and you complete flutter build apk -v

@jaydip-pawar
the issue could have been solved in this PR
please consider the possible solutions proposed here
if the problem persists please provide your flutter doctor -v and you complete flutter build apk -v

After seeing some solutions, I have builded apk but it's crashing. When I run my app from emulator or by connecting physical device it's work properly but when I release the apk it's crashesh and throwing following exception:
E/flutter (29267): [ERROR:flutter/shell/platform/android/platform_view_android_jni_impl.cc(43)] java.lang.IllegalAccessError: io.flutter.plugins.localauth.AuthenticationHelper
E/flutter (29267): 	at io.flutter.plugins.localauth.a.g()
E/flutter (29267): 	at e.a.c.a.i$a.a()
E/flutter (29267): 	at io.flutter.embedding.engine.e.b.c()
E/flutter (29267): 	at io.flutter.embedding.engine.FlutterJNI.handlePlatformMessage()
E/flutter (29267): 	at android.os.MessageQueue.nativePollOnce(Native Method)
E/flutter (29267): 	at android.os.MessageQueue.next(MessageQueue.java:328)
E/flutter (29267): 	at android.os.Looper.loop(Looper.java:191)
E/flutter (29267): 	at android.app.ActivityThread.main(ActivityThread.java:6016)
E/flutter (29267): 	at java.lang.reflect.Method.invoke(Native Method)
E/flutter (29267): 	at com.android.internal.os.ZygoteInit$MethodAndArgsCaller.run(ZygoteInit.java:937)
E/flutter (29267): 	at com.android.internal.os.ZygoteInit.main(ZygoteInit.java:798)
E/flutter (29267):
F/flutter (29267): [FATAL:flutter/shell/platform/android/platform_view_android_jni_impl.cc(942)] Check failed: CheckException(env).

@jaydip-pawar
the issue could have been solved in this PR
please consider the possible solutions proposed here
if the problem persists please provide your flutter doctor -v and you complete flutter build apk -v

After seeing some solutions, I have builded apk but it's crashing. When I run my app from emulator or by connecting physical device it's work properly but when I release the apk it's crashesh and throwing following exception:
E/flutter (29267): [ERROR:flutter/shell/platform/android/platform_view_android_jni_impl.cc(43)] java.lang.IllegalAccessError: io.flutter.plugins.localauth.AuthenticationHelper
E/flutter (29267): 	at io.flutter.plugins.localauth.a.g()
E/flutter (29267): 	at e.a.c.a.i$a.a()
E/flutter (29267): 	at io.flutter.embedding.engine.e.b.c()
E/flutter (29267): 	at io.flutter.embedding.engine.FlutterJNI.handlePlatformMessage()
E/flutter (29267): 	at android.os.MessageQueue.nativePollOnce(Native Method)
E/flutter (29267): 	at android.os.MessageQueue.next(MessageQueue.java:328)
E/flutter (29267): 	at android.os.Looper.loop(Looper.java:191)
E/flutter (29267): 	at android.app.ActivityThread.main(ActivityThread.java:6016)
E/flutter (29267): 	at java.lang.reflect.Method.invoke(Native Method)
E/flutter (29267): 	at com.android.internal.os.ZygoteInit$MethodAndArgsCaller.run(ZygoteInit.java:937)
E/flutter (29267): 	at com.android.internal.os.ZygoteInit.main(ZygoteInit.java:798)
E/flutter (29267):
F/flutter (29267): [FATAL:flutter/shell/platform/android/platform_view_android_jni_impl.cc(942)] Check failed: CheckException(env).

This seems to be a different issue, please open a new issue and fill the issue template. Closing as solved.
Thank you

This seems to be a different issue, please open a new issue and fill the issue template. Closing as solved.
Thank you


Code simplifications

CanvasItemEditor:

p_result == ADD_MOVE is always true in this switch-clause

both parts of the if-else-clause do the same thing and simplified an affine_inverse call

ControlEditorToolbar:

private function ControlEditorToolbar::_anchor_to_position is used nowhere. Looks like copy and paste from CanvasItemEditor::_anchor_to_position

ScrollContainer:

screen_is_touchscreen is always true, because otherwise the function already returned

TextLine: fix #65606

both parts of the if-else-clause do the same thing and simplified return statement

Thanks!

Is time_since_motion = 0; required before and after set_physics_process_internal(true);?
I am trying to optimize Scroll Container for smartphones and could do Cleanup Code
https://github.com/godotengine/godot/blob/da5f39889f155658cef7f7ec3cc1abb94e17d815/scene/gui/scroll_container.cpp#L178-L180


Wabbajack doesn't have the mod list

What happened?
Wabbajack didn't have the mod list so now I don't know how to download the pack
Modlist version
<IP_ADDRESS>

is there any way I can help bring it back together?
from what I've seen, there was an update that broke 3 or so mods. and now it needs to be configured.
I've been tempted to try recreate it. but if there's still work being done, then I would rather help. or I can just wait patiently too.


Feature Request

Import menu to import txt files, docx files
Option to share the whole note for other users to view/search what we see in web application. basically the shared users should be able to view, search but should not have the privilege to edit/delete the notes of the entire notebook
type ahead with number of matches on typing and highlight text in search results in 1st level and second level navigation
navigation arrows to navigate the occurrences of the searched keyword in the notes when clicked via search results

Thank you very much for recommending useful features. I will try to add these features as much as my technical skills allow. If you want to collaborate on open source we can do it together.


Using a fragment of type query, the Unknown field 'xxx' on type 'Query' and an error response will be returned

Describe the bug
See the following code.
fragment Post on Posts {
  id
}

fragment PostList on Query {
  postsCollection(first: $first, filter: $filter) {
    edges {
      node {
        ...Post
      }
    }
  }
}

query PostsPage($filter: PostsFilter!, $first: Int = 100) {
  ...PostList
}

If you run a query like this
Unknown field 'PostList' on type 'Query'
and an error response will be returned at runtime.
The graphql-code-generator works fine, and I believe the query being executed is also normal.
It worked for inline fragments.
To Reproduce
Steps to reproduce the behavior:
1.Execute a fragment, query as in the example
2.Check response from console
Expected behavior
Operate normally
Screenshots
If applicable, add screenshots to help explain your problem.
Versions:

PostgreSQL: [e.g. 14.1]
pg_graphql commit ref: ea2ab60

Additional context
Add any other context about the problem here.

ea2ab60b1f80c3be92c0d1f8453ef0bbec0b7f55 is from before the v0.1.0 and is very out of date at this point. Are you sure that was correct?
There was an issue with inline fragments as recently as last week. It was resolved in (unreleased on supabase platform at time of writing) v0.5.3.
Here is a full reproducible example that uses your snippet on master:
    create table posts(
        id int primary key,
        title text not null
    );

    insert into posts(id, title) values (1, 'asdf');

    select graphql.resolve($$
fragment Post on Posts {
  id
}

fragment PostList on Query {
  postsCollection(first: $first, filter: $filter) {
    edges {
      node {
        ...Post
      }
    }
  }
}

query PostsPage($filter: PostsFilter!, $first: Int = 100) {
  ...PostList
}
    $$);

Output
                             resolve
-----------------------------------------------------------------
 {"data": {"postsCollection": {"edges": [{"node": {"id": 1}}]}}}
(1 row)

If you're using the supabase platform and send your project_ref I can get your project updated so you can start using ^ now
refs:
dupe of #268

Can confirm this is the same issue. Unknown field 'xxx' on type 'Query' where xxx is the fragment name was the error I saw on v0.4

@olirice
Thank you! I think the commit history is wrong. I forgot to edit it. My apologies.
This problem is happening in local development, how can I fix it?
Deleting and re-pulling the Docker image did not solve the problem.

@mayone-du v0.5.3 hasn't been pushed to the supabase/postgres:latest docker image yet so unfortunately that fix won't be available for local development until then. Looking at the release schedule it should trickle down to the CLI by end of next week, sorry for the delay

Oops, looks like it was supabase/postgres:<IP_ADDRESS> . Sorry.
How do I install supabase/postgres:<IP_ADDRESS> or supabase/postgres:<IP_ADDRESS>?

it depends how you're local environment is set up
If you're using the supabase CLI you'll want to upgrade to the latest version brew upgrade supabase (assuming Mac) but the latest docker image can take a few weeks to trickle down to the CLI. We cautious about updating the version used by the CLI because offline setups tend to have a longer lifespan and are harder to update when issues occur
If you're using docker-compose you need to update the docker-compose.yml image tag
or if you're using docker directly, replace the tag in your command


Consistent io.dcos package structure.

Previously sample folder structure an package names seemed out of sync.

@unterstein Does this make sense? No is a valid answer ;)


Convert all Implicit Measures to Explicit SUM measures

Hi,
The code here for Autogenerate SUM Measures is useful but it requires selected each relevant column in each table.
Is there a way to code it so that it filters the columns for what Power BI has deemed an implicit measure already to avoid having to select columns in the first place.
I'm not familiar with C# code but an idea is that the metadata for the column contains the Summarize by (say SUM) info already so it would use that to create the explicit measure.
Thanks,
Paul

This post shares a methodology for converting implicit measures to explicit measures.
https://www.elegantbi.com/post/reportlevelmeasures


Update apps-localization.md

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

File
Status
Preview URL
Details

msteams-platform/concepts/build-and-test/apps-localization.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.

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

File
Status
Preview URL
Details

msteams-platform/concepts/build-and-test/apps-localization.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.


Python Utility Script - Tag Editor

This python script allows easy mass-auditing and editing of tags on recipes to allow easy addition of things like vegan, vegetarian, gluten-free that apply to a wide range of recipes but recipe authors may not have considered adding as tags.
Usage:
python tag-editor.py
Then for every recipe in the src folder, you will be presented with a prompt showing current recipe, ingredient list and current tags.
you can add tags, remove tags, go to the next item or quit:
Adding Tags
Command: '[add / a / +] {tag tag tag...}'
Example:
add chicken gluten-free   --- > Adds "chicken" and "gluten-free" tags to current recipe
Removing Tags
Command: '[remove / r / -] {tag tag tag...}'
Example:
remove pork beef mexican   --- > Removes "pork", "beef" and "mexican" tags from current recipe (if present)
Next Recipe
Command: 'next/ n / [Return_Key]'
Moves to next recipe in the folder.
Quit
Command: 'quit / q' --> quits the program.
Program will also terminate when it has cycled through all recipes in the list.

Could you document usage of it in the thread? Thanks @DollarStarNova

Could you document usage of it in the thread? Thanks @DollarStarNova

I have now documented this in the top comment :)

Awesome @DollarStarNova

@LukeSmithxyz Any reason this has been rejected? I think it's a very useful tool.

Agreed @DollarStarNova


Fix show of variant

Before:
julia> Thing(0x00)
Option{UInt8}: Thing(0)

After:
julia> Thing(0x00)
Option{UInt8}: Thing(0x00)

Codecov Report

Merging #2 (dc19be0) into master (218d6db) will increase coverage by 2.33%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master       #2      +/-   ##
==========================================
+ Coverage   84.81%   87.14%   +2.33%
==========================================
  Files           1        1
  Lines          79       70       -9
==========================================
- Hits           67       61       -6
+ Misses         12        9       -3

Impacted Files
Coverage Δ

src/SumTypes.jl
87.14% <ø> (+2.33%)
:arrow_up:

Continue to review full report at Codecov.

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

Codecov Report

Merging #2 (dc19be0) into master (218d6db) will increase coverage by 2.33%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master       #2      +/-   ##
==========================================
+ Coverage   84.81%   87.14%   +2.33%
==========================================
  Files           1        1
  Lines          79       70       -9
==========================================
- Hits           67       61       -6
+ Misses         12        9       -3

Impacted Files
Coverage Δ

src/SumTypes.jl
87.14% <ø> (+2.33%)
:arrow_up:

Continue to review full report at Codecov.

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


[fix][admin] Fix validatePersistencePolicies that Namespace/Topic persistent policies cannot set to < 0

Motivation
Currently, broker allows bookkeeperEnsemble,bookkeeperWriteQuorum,bookkeeperAckQuorum set to <= 0 when set NamespacePersistentPolicies/PersistentTopic policies.
Add value checker to not allow them set to <= 0.
Does this pull request potentially affect one of the following parts:

If the box was checked, please highlight the changes

[ ] Dependencies (add or upgrade a dependency)
[ ] The public API
[ ] The schema
[ ] The default values of configurations
[ ] The threading model
[ ] The binary protocol
[x] The REST endpoints
[ ] The admin CLI options
[ ] The metrics
[ ] Anything that affects deployment

Documentation

[ ] doc
[ ] doc-required
[x] doc-not-needed
[ ] doc-complete

Matching PR in forked repository
PR in forked repository:  https://github.com/tjiuming/pulsar/pull/16

And please also fix the broker.conf to avoid set 0 or negative number for persistence quorums.

And please also fix the broker.conf to avoid set 0 or negative number for persistence quorums.

these fields in broker.conf annotated with minValue=1, I think no need to fix

Please also update https://github.com/apache/pulsar/blob/master/site2/docs/admin-api-namespaces.md to document this requirement.

@lin-zhao
This is included at https://github.com/apache/pulsar/pull/19001.

/pulsarbot run-failure-checks

Codecov Report

Merging #18999 (547286c) into master (3180a4a) will decrease coverage by 8.93%.
The diff coverage is 57.14%.

@@             Coverage Diff              @@
##             master   #18999      +/-   ##
============================================
- Coverage     46.17%   37.23%   -8.94%
+ Complexity    10359     1989    -8370
============================================
  Files           703      209     -494
  Lines         68845    14452   -54393
  Branches       7382     1577    -5805
============================================
- Hits          31788     5381   -26407
+ Misses        33448     8486   -24962
+ Partials       3609      585    -3024

Flag
Coverage Δ

unittests
37.23% <57.14%> (-8.94%)
:arrow_down:

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

Impacted Files
Coverage Δ

...he/pulsar/client/impl/MultiTopicsConsumerImpl.java
22.80% <0.00%> (-0.07%)
:arrow_down:

...va/org/apache/pulsar/client/impl/ProducerImpl.java
15.66% <0.00%> (-1.34%)
:arrow_down:

.../apache/pulsar/client/impl/BatchMessageIdImpl.java
67.50% <100.00%> (-4.73%)
:arrow_down:

...va/org/apache/pulsar/client/impl/ConsumerImpl.java
15.06% <100.00%> (-0.06%)
:arrow_down:

...a/org/apache/pulsar/client/impl/MessageIdImpl.java
80.85% <100.00%> (+0.29%)
:arrow_up:

.../apache/pulsar/client/impl/MultiMessageIdImpl.java
80.55% <100.00%> (+7.58%)
:arrow_up:

...he/pulsar/client/impl/PartitionedProducerImpl.java
30.34% <0.00%> (-5.13%)
:arrow_down:

.../pulsar/client/impl/BatchMessageContainerImpl.java
55.95% <0.00%> (-1.02%)
:arrow_down:

.../pulsar/client/impl/ProducerStatsRecorderImpl.java
84.04% <0.00%> (-0.62%)
:arrow_down:

... and 508 more

/pulsarbot run-failure-checks


support resource timing and navigation timing APIs

Summary:
Add type definitions for the latest timing and navigation timing APIs
that are in the recommendation status.
The new types are:

PerformanceResourceTiming
https://www.w3.org/TR/resource-timing-1/#idl-def-performanceresourcetiming
PerformanceNavigationTiming
https://www.w3.org/TR/navigation-timing-2/#sec-PerformanceNavigationTiming

Thanks for your contribution!

Thank you for merging!


Multiple select dropdown with checkboxes !

Is it possible to have multiple check boxes instead of options, a select all and unselect all options along?
On clicking the check box updates the name in select box with name or this many items selected !

I am also looking to implement something like this - http://phossa.github.io/smartselect/docs/demo.html#sec-3a
I want a "tree-type, multiple, search" dropdown

Hi @dhvsandeep,
Thank you for posting, but although it’s a valid usage question, we’ve limited GitHub Issues to bug reports and feature requests, keeping the board more manageable for maintainers; see the contributing guidelines for more information on what kind of posts should find themselves into the GitHub Issues board.
To get answers or feedback that might allow you to repost this issue, please use one of our other support resources:

Gitter chatroom
Official forums
StackOverflow


All white

Short description of the issue

When I launch the settings, all I see is Save and Close text. Everything else is white. I'm expecting there to be some settings to configure such as the file path to game installs
Lauhdutin version

3.0.4
Rainmeter version

4.2 Final Release - r3111
Windows version

Windows 10 Home 1803
Error messages

Steps to reproduce the issue

Launching settings or main, etc triggers all white dialog.

Hello!
There is any warning or error in Rainmeter Log? (Right click on Rainmeter tray icon -> About -> Log).

Yes
ImageName: Unable to open<EMAIL_ADDRESS>(Lauhdutin-master\dist\Settings\Settings.ini)

Are you using the code downloaded version for any reason? If not, I suggest you to use the last release or beta, which should solve your issue, cause I think it's an issue with folder places.


TeX Reformat Code custom auto identing with Pandoc template

Hello,
I am using TeXiFy with PHPStorm, and I frequently use the IDE's Reformat Code tool that takes care of auto indenting nested declaration blocks. That option works well with LaTeX files, but I also want to autoformat Pandoc template files, which are regular .tex files containing variables with special meaning:
\begin{document}
    $if(title)$
    $if(beamer)$
    \frame{\titlepage}
    $else$
    \maketitle
    $endif$
    $if(abstract)$
    \begin{abstract}
        $abstract$
    \end{abstract}
    $endif$
    $endif$

    $for(include-before)$
    $include-before$
    $endfor$
(...)

I'd like to setup my IDE in a way that would make it treat theese variables as code nesting points, and would format previous code to:
\begin{document}
    $if(title)$
        $if(beamer)$
            \frame{\titlepage}
        $else$
            \maketitle
        $endif$
        $if(abstract)$
            \begin{abstract}
                $abstract$
            \end{abstract}
        $endif$
    $endif$

    $for(include-before)$
        $include-before$
    $endfor$

I was, however, unable to find any settings that would allow me to archieve that.  Can you think of any solution?
Thank you in advance. Regards,

Jan

This is not possible with the plugin as we only support LaTeX and BibTeX grammar.
It is unlikely we will implement this as there are way more important features.
Unless somebody files a pull request fixing exactly that, don't expect this feature anytime the next few years.


Firefox support plz

hi if possible could you please the launch this extension on firefox
thank you.

@Amitonagasaki i migrated this extension to firefox. Checkout https://addons.mozilla.org/en-US/firefox/addon/leethub-2-0-for-firefox/


Enhance user interface and make it more responsive for mobiles

This enhancement issue is about the user interface: When in details mode:

Current OSM element should be indicated (node id; weblink/shortlink to embed to show OSM in external page with marker?).
Category field should'nt have "Absinth" as default value but "--Choose--"
Category field should be same length as other fields.
Website should be more responsive to mobiles.

This issue is related to https://github.com/osmlab/onosm.org/issues/11

The category stuff should be fixed in #23 hopefully, though select2 is a pain.


Can't resolve 'react-event-listener'

Environment

Tech
Version

material-ui-pickers
1.0.0-rc.9

material-ui
1.2

React
16.3

Browser
Chrome Latest

Platform

Steps to reproduce

npm run build

Expected behavior

The build to pass
Actual behavior

I've gotten all the rest of this issues fixed, except this one.
ERROR in ./node_modules/material-ui-pickers/DatePicker/Calendar.js
    Module not found: Error: Can't resolve 'react-event-listener' in 'C:\Projects\Work\PROJ\src\node_modules\material-ui-pickers\DatePicker'
     @ ./node_modules/material-ui-pickers/DatePicker/Calendar.js 40:26-57

ERROR in ./node_modules/material-ui-pickers/_shared/ModalDialog.js
    Module not found: Error: Can't resolve 'react-event-listener' in 'C:\Projects\Work\PROJ\src\node_modules\material-ui-pickers\_shared'
     @ ./node_modules/material-ui-pickers/_shared/ModalDialog.js 32:26-57

I see the imports in these two files:
https://github.com/dmtrKovalenko/material-ui-pickers/search?q=react-event-listener&unscoped_q=react-event-listener
Yet I see no dependency on it in the package.json, am I missing something?
Live example

+1

Ha. It was included as dependency in material ui

And it’s still included https://github.com/mui-org/material-ui/blob/2d31ad0eb3b04843fcae01a731601d9fa2b63860/packages/material-ui/package.json#L57

That's weird. I had to include it myself.
Ok, no big deal.

Yeah, I had to too, strange

@dmtrKovalenko Why did you closed the issue, is it fixed?

Still have Module not found: Can't resolve 'react-event-listener' error, and still have to add react-event-listener dependency to my project.

I found the reason!
We will fix it in the next release, please do npm i react-event-listener for now

Closed because this was fixed in the previous release :)


Textarea

Maybe this could be an option to the Text input…

Here’s one place I’d want a textarea input: https://observablehq.com/@observablehq/daily-team-shuffler

More examples: https://observablehq.com/@j-f1/sort-lines-of-text?collection=@j-f1/tools, https://observablehq.com/d/4f235f27b1be8f08. These notebooks also use textareas to display read-only output (which makes it easy to select all)


Develop

Codecov Report

Merging #20 into master will decrease coverage by 5.34%.
The diff coverage is 4.87%.

@@             Coverage Diff              @@
##             master      #20      +/-   ##
============================================
- Coverage     24.06%   18.72%   -5.35%
- Complexity      127      165      +38
============================================
  Files            10       11       +1
  Lines           349      470     +121
============================================
+ Hits             84       88       +4
- Misses          265      382     +117

Impacted Files
Coverage Δ
Complexity Δ

src/Util/Composer.php
0% <0%> (ø)
9 <6> (+6)
:arrow_up:

src/Console/TestCommand.php
0% <0%> (ø)
25 <25> (?)

src/Generator/TwigGenerator.php
72.58% <23.52%> (-18.54%)
23 <5> (+6)

src/Util/Inflector.php
57.14% <40%> (-42.86%)
3 <3> (+1)

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 39c07f2...ef367fc. Read the comment docs.


Immune institute

URL institute: https://immune.institute/en/

@ronnysuero First stage of verification passed, official domain for IMMUNE Technology Institute is immune.institute as far as I verified. This is just an informational note. Pull request is still in the review. I appreciate your patience while waiting for the final decision.

@ronnysuero Thank you for your patience while waiting for a review and processing your request. Pull request merged.