Eureka Registe Invalid CSRF token found

Describe the bug
Please provide details of the problem, including the version of Spring Cloud that you
are using.
<spring-cloud.version>2023.0.3</spring-cloud.version>
Sample
If possible, please provide a test case or sample application that reproduces
the problem. This makes it much easier for us to diagnose the problem and to verify that
we have fixed it.
Eureke Server Config:
  application:
    name: spring-eureka-demo
  security:
    user:
      name: admin
      password: admin

server:
  port: 3000

eureka:
  client:
    register-with-eureka: false
    fetch-registry: false
  server:
    enable-self-preservation: false

logging:
  level:
    org:
      springframework:
        security: DEBUG
        cloud:
          netflix: DEBUG

@Configuration
@EnableWebSecurity
public class SecurityConfig {

  @Bean
  public SecurityFilterChain securityFilterChain(HttpSecurity http) throws Exception {
    http.authorizeHttpRequests((authz) -> authz
            .requestMatchers("/eureka/**").permitAll()
            .requestMatchers("/actuator/health").permitAll()
            .requestMatchers("/actuator/**").hasRole("ADMIN")
            .anyRequest().authenticated()
        )
        .httpBasic(withDefaults())
        .csrf(csrf -> csrf.csrfTokenRepository(CookieCsrfTokenRepository.withHttpOnlyFalse())
            .csrfTokenRequestHandler(new CsrfTokenRequestAttributeHandler())
            .disable()
        );
    return http.build();
  }

}

Eureka client:
spring:
  application:
    name: spring-config-demo
  security:
    user:
      name: admin
      password: admin

  profiles:
    default: native
    active: native

  cloud:
    config:
      server:
        native:
          search-locations: classpath:/configDev/

eureka:
  client:
    serviceUrl:
      defaultZone: http://admin:admin@spring-eureka-demo:3000/eureka/

server:
  port: 8104

when eureka client send request to eureka server, found the return 401 exception.
from eureka server log,
Securing POST /eureka/apps/SPRING-CONFIG-DEMO Securing PUT /eureka/apps/SPRING-CONFIG-DEMO/spring-eureka-demo:spring-conf Invalid CSRF token found for http://spring-eureka-demo:3000/eureka/apps/SPR Responding with 403 status code Invalid CSRF token found for http://spring-eureka-demo:3000/eureka/apps/SPR Responding with 403 status code Securing POST /error

Hello @weiro-9-w7, thanks for reporting the issue. Please see this branch for correct CSRF configuration sample: https://github.com/spring-cloud-samples/eureka/blob/Eureka-With-Security-4.x/src/main/java/eurekademo/EurekaApplication.java . Please let us know if applying this configuration solves your issue.

If you would like us to look at this issue, please provide the requested information. If the information is not provided within the next 7 days this issue will be closed.

Closing due to lack of requested feedback. If you would like us to look at this issue, please provide the requested information and we will re-open the issue.


add embl and refseq cross references to the pept2prot api

[x] Add models
[x] Add to api controller
[x] Add to view
[x] Add tests
[x] Add documentation

Closes #449
Original pull request by @bmesuere on Fri Jan 16 2015 at 14:42.
Merged by @bmesuere on Tue Jan 27 2015 at 10:40.

@silox can you check if this is the info you need? The branch is deployed at scruffy.
Original comment by @bmesuere on Fri Jan 16 2015 at 15:18.

Before this is merged, maybe we should discuss the future use of these identifiers.
Original comment by @bmesuere on Wed Jan 21 2015 at 11:42.


Radio group and code dont appear to have hide or unhide properties

this is such a useful feature to enable dynamic form rendering based on users prior options without this i cant use these advanced  elements. Can this element be added or is there a way to hide them? I am working around it by setting width to zero but it just looks terrible. Tried also using disabled which also doesnt seem to work

I just realised there is code to display if true and false. So used that but is there supposed to be hidden and disabled usable?

@craftzneko ShowIf using code and hidden are two different functionality:

ShowIf was introduced for dynamic forms, which depends on dashboard variables, and skip elements which are not required.
Hidden for String elements was introduced for sending data not visible to the users.


XamlControlsResources does not work when main program class is renamed

Describe the bug
When using WinUI 2, the main program's class name must be App in order for the styles in XamlControlsResources to be applied. If you rename it something else, all the controls lack the WinUI styling.
Steps to reproduce the bug

Create new UWP app (CppWinRT)
Change default namespace in CppWinRT settings
Rename App class to something else (do changes in all files)
Update appxmanifest entrypoint to be NewNamespace.NewAppClass
Add WinUI 2 nuget package and follow directions
Add a AutoSuggestBox to a control
Build + deploy
Notice how controls use W10 styling

Expected behavior
XamlControlsResources works normally
Screenshots

NuGet package version
Microsoft.UI.Xaml 2.7.1
Windows app type

[X] UWP
[ ] Win32

Device form factor
Desktop
Windows version
Windows 11 (21H2): Build 22000
Additional context
This also happens for UWP apps written in C# or VB.

If you have renamed app.xaml (say, to NewAppClass.xaml), then you need to add the following to NewAppClass::OnLaunched() as you have effectively opted out of the framework's automatic load of app.xaml:
Uri resourceLocator{ L"ms-appx:///NewAppClass.xaml" };
Application::LoadComponent(*this, resourceLocator, ComponentResourceLocation::Application);

Sorry about the late reply, your solution works. It is possible to document this behavior somewhere and provide others with information that tells them what to do in this situation?


  Configure Azure Pipelines

This pipeline configuration builds the application and produces a build for the Release configuration.

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.:x: PaulObalonye sign nowYou have signed the CLA already but the status is still pending? Let us recheck it.

Azure pipelines


Error when run npm serve

help with this error

Looks like you're missing the environment.ts files.
Take a peak at the readme.md


fix(diagnostic_graph_aggregator): fix uselessOverride warning

Description
This is a fix based on cppcheck uselessOverride warning
system/diagnostic_graph_aggregator/src/common/graph/units.hpp:142:8: style: The function 'update_status' overrides a function in a base class but is identical to the overridden function [uselessOverride]
  void update_status() override;
       ^
system/diagnostic_graph_aggregator/src/common/graph/units.hpp:132:8: note: Virtual function in base class
  void update_status() override;
       ^
system/diagnostic_graph_aggregator/src/common/graph/units.hpp:142:8: note: Function in derived class
  void update_status() override;
       ^

Though the program is totally the same, I found a TODO comment // TODO(Takagi, Isamu): update link flags..
If you do not want to delete it, please come up with some workarounds!
Related links
Parent Issue:

Link

How was this PR tested?
Notes for reviewers
None.
Interface changes
None.

Effects on system behavior
None.

@veqcc As noted in the TODO comments, this will be processed differently in the future. So this is intentional, can I disable the check on a code block?

@isamu-takagi
Thank you for your quick response.

As noted in the TODO comments, this will be processed differently in the future. So this is intentional, can I disable the check on a code block?

OK, then I will restore the code block and use inline cppcheck suppression (I'll do it tomorrow).
Btw, do you know when the TODO comment will be resolved?

@isamu-takagi
Could you wait till the cppcheck-differential workflow is updated sp that PRs which changes only .hpp or .cpp can avoid false positive or false negative results?
Related Link (TIER IV internal): https://star4.slack.com/archives/C070FQMREVB/p1719877783942809

@isamu-takagi
Now it's ready for being merged!

Btw, do you know when the TODO comment will be resolved?

It's not decided yet. I'll update this if I need more detailed diagnostics after using graph.


Feature Request - Add vercel config

Add vercel configuration file to cache all api.

Hi @mazipan , can you assign this issue to me? thanks!


ScriptSessionLoadError:  cannot satisfy -trust project-m36

On my GHC 8.10.3 installation with Cabal-install 3.2, I get this warning when I do cabal run project-m36-server -- --database mydbname --hostname <IP_ADDRESS>:
Building executable 'project-m36-server' for project-m36-0.9.0..
[1 of 1] Compiling Main             ( src/bin/ProjectM36/Server/project-m36-server.hs, /home/janus/project-m36/dist-newstyle/build/x86_64-linux/ghc-8.10.3/project-m36-0.9.0/x/project-m36-server/build/project-m36-server/project-m36-server-tmp/Main.o )
Linking /home/janus/project-m36/dist-newstyle/build/x86_64-linux/ghc-8.10.3/project-m36-0.9.0/x/project-m36-server/build/project-m36-server/project-m36-server ...
Failed to load scripting engine- scripting disabled: ScriptSessionLoadError <command line>: cannot satisfy -trust project-m36
    (use -v for more information)

Is this intended?
When I do cabal run Example-SimpleClient concurrently, I get another error:
Up to date
Example-SimpleClient: ExceptionError "failed to find socket in client map\nCallStack (from HasCallStack):\n  error, called at ./src/lib/ProjectM36/Server.hs:114:16 in project-m36-0.9.0-inplace:ProjectM36.Server"
CallStack (from HasCallStack):
  error, called at ./src/lib/ProjectM36/Client.hs:305:21 in project-m36-0.9.0-inplace:ProjectM36.Client

Yes, the failure to load the scripting engine happens when project-m36 is not installed in your GHC packages with cabal install. The consequence is that the optional ability to add Haskell-scripted database functions at runtime is disabled.
I'll have to take a closer look at the second error- I haven't yet seen that but it should be quick to fix.
Thanks for the detailed reports!

Yes, the failure to load the scripting engine happens when project-m36 is not installed in your GHC packages with cabal install. The consequence is that the optional ability to add Haskell-scripted database functions at runtime is disabled.
I'll have to take a closer look at the second error- I haven't yet seen that but it should be quick to fix.
Thanks for the detailed reports!

It looks like the example SimpleClient connects to "mytestdb" on localhost which, of course, if one just runs the client standalone, does not exist. I'll make that clearer from the error.

It looks like the example SimpleClient connects to "mytestdb" on localhost which, of course, if one just runs the client standalone, does not exist. I'll make that clearer from the error.


Add option to disable autosizer

Default the WindowView will try to use a proper window size according to the biggest page element. This can be an annoyance when the same page constantly has to be readjusted.

The options panel should use a fixed dimension if autosizer is disabled

The autosizer has been removed in 0.1.2


[fix] update pool reference for 1.2.8 release

pool-reference is broken after 1.2.8 upgrade on main repo.
error message like:
 ImportError: cannot import name 'solution_to_extra_data' from 'chia.pools.pool_puzzles'
This PR fixed the pool-reference by using the renamed function name in the main repo.
Signed-off-by: GrassR00t<EMAIL_ADDRESS>

@mariano54

If pool-reference had to use a part of chia-blockchain API that was not protected against changes between 1.2.7 and 1.2.8, then that's a problem. This is not the first time our pool was hit by an API change and here all the pools based on pool-reference were affected. This is because chia-blockchain does not adhere to SemVer. I think it's not affecting just reference-pool but actually everyone who has chia-blockchain as a dependency. Not sure what I can do about that, other than fixing the pool when API is unexpectedely broken after minor version update.


Initializer for parent of jitclass with inheritance

A simple test of jitclass with inheritance seems to work just fine.  But if I want to call the parent class's __init__ function, I have to jump through a strange hoop.  Here's a working example:
specA = [('valueA', nb.int64)]
# Note that this does not get jitclassed, because we can't (yet) inherit from jitted classes
class A(object):
    def __init__(self, value):
        self.valueA = value

specB = [('valueB', nb.int64)]
# Note that this does not get jitclassed, because we can't (yet) inherit from jitted classes
class B(object):
    def __init__(self, value):
        self.valueB = value

specC = [('valueC', nb.int64)]
@nb.jitclass(specA+specB+specC)
class C(A, B):
    __init__A = A.__init__
    __init__B = B.__init__
    def __init__(self, a, b, c):
        #super(C).__init__(self, a)  # Fails with `Untyped global name 'super'`
        #A.__init__(self, a)  # Fails with `Untyped global name 'A'`
        self.__init__A(a)  # Works fine
        self.__init__B(b)  # Works fine
        self.valueC = c

c = C(17, 23, 31)
print(c.valueA, c.valueB, c.valueC)

This seems to work just fine.  In particular, c.valueA is correctly defined.
I can imagine that making super work might be hard.  Intriguingly, if I change the definition to
    __init__A = super(C).__init__

it fails with class members are not yet supported: _C__init__A.  I don't know what that means in this context, because I can define __init__A to be something else that I had thought would be identical...
But I really don't understand why A.__init__ can be used to define a C method, but cannot be called from within C's own __init__ function.  Is there a way to make this work more naturally, or even get super working?  Is something insidious going on that I'm not aware of, but is about to destroy my code?

Faced the same issue, great to have a work around but I would also like to know if it is realiable/stable.

:( workaround no longer works (python3.8) -- I get:
TypeError: cannot subclass from a jitclass
I suspect from the (object) that this was working in python 2.

@shaunc I suspect the version of numba is more relevant than the version of python.  The example above still works for me on python 3.8, but I'm still on numba 0.51.2.  Which version are you using?

@moble -- oops! Your right -- I accidentally added jitclass decorator to my base class -- silly me! Thanks for nudging me to check.

... What I wonder is whether there is a way to avoid unboxing if I pass a derived jitclass class to a function that accepts the base class (or a jitclassed other derivative of it, otherwise just the same)?

Just to add my 2 cents, subclassing is the OO way to do this which doesn't lower in a straighforward way to LLVM because it requires a vtable (like C++) or some such.  Notably in Julia, there's a type hierarchy but methods are not on the instance but freestanding generic functions.  It seems like Numba could do something similar with generated_jit which could dispatch based on types to the correct method.
import numba as nb
import numba.experimental as nbe

@nbe.jitclass
class Sand:
    def __init__(self): pass
    def density(self):
        return 1850

@nbe.jitclass
class Clay:
    def __init__(self): pass
    def density(self):
        return 1300

@nb.generated_jit(nopython=True)
def density(x):
    return lambda x: x.density()

@nbe.jitclass
class Scale:
    bias: int
    def __init__(self, bias): self.bias = bias
    def density(self, soil): return density(soil)+self.bias

sand = Sand()
clay = Clay()
scale = Scale(10)
for soil in [sand, clay]:
    print(soil, 'density', scale.density(soil), 'kg/m3')

prints the expect result.  What's promising is that the generic weight function doesn't require any if-else stuff, so a bunch of method names could be automatically wrapped, by iterating through a list of jitclasses and inspecting the methods with same name and arguments etc.  That still doesn't allow a jitclass to have a generic attribute which then specializes to a specific pet or soil, but it seems like a step forward.

Very nice approach @maedoc.  I'm a little concerned that the result is still doing a lot of work to get there, though.  We can look at the native assembly code for Scale.density acting on a Sand with
print(list(Scale.class_type.instance_type.jit_methods["density"].inspect_asm().values())[0])

The result is... extensive
        .section	__TEXT,__text,regular,pure_instructions
        .build_version macos, 11, 0
        .globl	__ZN8__main__5Scale11density_249B42c8tJTIeFCjyCbUFRqqOAK_2f6h0kCng1maAA_3d_3dEN8instance8jitclass35Scale_2318ae2d780_3cbias_3aint64_3eEN8instance8jitclass22Sand_2318ae00bb0_3c_3eE
        .p2align	4, 0x90
__ZN8__main__5Scale11density_249B42c8tJTIeFCjyCbUFRqqOAK_2f6h0kCng1maAA_3d_3dEN8instance8jitclass35Scale_2318ae2d780_3cbias_3aint64_3eEN8instance8jitclass22Sand_2318ae00bb0_3c_3eE:
        movl	$1850, %eax
        addq	(%rcx), %rax
        movq	%rax, (%rdi)
        xorl	%eax, %eax
        retq

        .globl	__ZN7cpython8__main__5Scale11density_249B42c8tJTIeFCjyCbUFRqqOAK_2f6h0kCng1maAA_3d_3dEN8instance8jitclass35Scale_2318ae2d780_3cbias_3aint64_3eEN8instance8jitclass22Sand_2318ae00bb0_3c_3eE
        .p2align	4, 0x90
__ZN7cpython8__main__5Scale11density_249B42c8tJTIeFCjyCbUFRqqOAK_2f6h0kCng1maAA_3d_3dEN8instance8jitclass35Scale_2318ae2d780_3cbias_3aint64_3eEN8instance8jitclass22Sand_2318ae00bb0_3c_3eE:
        .cfi_startproc
        pushq	%rbp
        .cfi_def_cfa_offset 16
        pushq	%r15
        .cfi_def_cfa_offset 24
        pushq	%r14
        .cfi_def_cfa_offset 32
        pushq	%r13
        .cfi_def_cfa_offset 40
        pushq	%r12
        .cfi_def_cfa_offset 48
        pushq	%rbx
        .cfi_def_cfa_offset 56
        subq	$24, %rsp
        .cfi_def_cfa_offset 80
        .cfi_offset %rbx, -56
        .cfi_offset %r12, -48
        .cfi_offset %r13, -40
        .cfi_offset %r14, -32
        .cfi_offset %r15, -24
        .cfi_offset %rbp, -16
        movq	%rsi, %rdi
        movabsq	$_.const.Scale.density, %rsi
        movabsq	$_PyArg_UnpackTuple, %rbp
        leaq	16(%rsp), %r8
        leaq	8(%rsp), %r9
        movl	$2, %edx
        movl	$2, %ecx
        xorl	%eax, %eax
        callq	*%rbp
        movq	$0, (%rsp)
        testl	%eax, %eax
        je	LBB1_1
        movabsq	$__ZN08NumbaEnv8__main__5Scale11density_249B42c8tJTIeFCjyCbUFRqqOAK_2f6h0kCng1maAA_3d_3dEN8instance8jitclass35Scale_2318ae2d780_3cbias_3aint64_3eEN8instance8jitclass22Sand_2318ae00bb0_3c_3eE, %rax
        cmpq	$0, (%rax)
        je	LBB1_4
        movq	16(%rsp), %rax
        movq	16(%rax), %r15
        movq	24(%rax), %r14
        movabsq	$_NRT_incref, %r13
        movq	%r15, %rdi
        callq	*%r13
        movabsq	$_PyErr_Occurred, %rbp
        callq	*%rbp
        testq	%rax, %rax
        jne	LBB1_1
        movq	8(%rsp), %rax
        movq	16(%rax), %rbx
        movq	24(%rax), %r12
        movq	%rbx, %rdi
        callq	*%r13
        callq	*%rbp
        testq	%rax, %rax
        jne	LBB1_7
        movq	$0, (%rsp)
        movabsq	$__ZN8__main__5Scale11density_249B42c8tJTIeFCjyCbUFRqqOAK_2f6h0kCng1maAA_3d_3dEN8instance8jitclass35Scale_2318ae2d780_3cbias_3aint64_3eEN8instance8jitclass22Sand_2318ae00bb0_3c_3eE, %rax
        movq	%rsp, %rdi
        movq	%r15, %rdx
        movq	%r14, %rcx
        movq	%rbx, %r8
        movq	%r12, %r9
        callq	*%rax
        movq	(%rsp), %r14
        movabsq	$_NRT_decref, %rbp
        movq	%r15, %rdi
        callq	*%rbp
        movq	%rbx, %rdi
        callq	*%rbp
        movabsq	$_PyLong_FromLongLong, %rax
        movq	%r14, %rdi
        callq	*%rax
LBB1_2:
        addq	$24, %rsp
        popq	%rbx
        popq	%r12
        popq	%r13
        popq	%r14
        popq	%r15
        popq	%rbp
        retq
LBB1_4:
        movabsq	$_PyExc_RuntimeError, %rdi
        movabsq	$"_.const.missing Environment: _ZN08NumbaEnv8__main__5Scale11density_249B42c8tJTIeFCjyCbUFRqqOAK_2f6h0kCng1maAA_3d_3dEN8instance8jitclass35Scale_2318ae2d780_3cbias_3aint64_3eEN8instance8jitclass22Sand_2318ae00bb0_3c_3eE", %rsi
        movabsq	$_PyErr_SetString, %rax
        callq	*%rax
        jmp	LBB1_1
LBB1_7:
        movabsq	$_NRT_decref, %rax
        movq	%r15, %rdi
        callq	*%rax
LBB1_1:
        xorl	%eax, %eax
        jmp	LBB1_2
        .cfi_endproc

        .globl	_cfunc._ZN8__main__5Scale11density_249B42c8tJTIeFCjyCbUFRqqOAK_2f6h0kCng1maAA_3d_3dEN8instance8jitclass35Scale_2318ae2d780_3cbias_3aint64_3eEN8instance8jitclass22Sand_2318ae00bb0_3c_3eE
        .p2align	4, 0x90
_cfunc._ZN8__main__5Scale11density_249B42c8tJTIeFCjyCbUFRqqOAK_2f6h0kCng1maAA_3d_3dEN8instance8jitclass35Scale_2318ae2d780_3cbias_3aint64_3eEN8instance8jitclass22Sand_2318ae00bb0_3c_3eE:
        pushq	%rax
        movq	%rcx, %r9
        movq	%rdx, %r8
        movq	%rsi, %rcx
        movq	%rdi, %rdx
        movq	$0, (%rsp)
        movabsq	$__ZN8__main__5Scale11density_249B42c8tJTIeFCjyCbUFRqqOAK_2f6h0kCng1maAA_3d_3dEN8instance8jitclass35Scale_2318ae2d780_3cbias_3aint64_3eEN8instance8jitclass22Sand_2318ae00bb0_3c_3eE, %rax
        movq	%rsp, %rdi
        callq	*%rax
        movq	(%rsp), %rax
        popq	%rcx
        retq

        .globl	_NRT_incref
        .weak_def_can_be_hidden	_NRT_incref
        .p2align	4, 0x90
_NRT_incref:
        testq	%rdi, %rdi
        je	LBB3_1
        lock		incq	(%rdi)
        retq
LBB3_1:
        retq

        .globl	_NRT_decref
        .weak_def_can_be_hidden	_NRT_decref
        .p2align	4, 0x90
_NRT_decref:
        .cfi_startproc
        testq	%rdi, %rdi
        je	LBB4_2
        ##MEMBARRIER
        lock		decq	(%rdi)
        je	LBB4_3
LBB4_2:
        retq
LBB4_3:
        ##MEMBARRIER
        movabsq	$_NRT_MemInfo_call_dtor, %rax
        jmpq	*%rax
        .cfi_endproc

        .comm	__ZN08NumbaEnv8__main__5Scale11density_249B42c8tJTIeFCjyCbUFRqqOAK_2f6h0kCng1maAA_3d_3dEN8instance8jitclass35Scale_2318ae2d780_3cbias_3aint64_3eEN8instance8jitclass22Sand_2318ae00bb0_3c_3eE,8,3
        .section	__TEXT,__const
_.const.Scale.density:
        .asciz	"Scale.density"

        .p2align	4
"_.const.missing Environment: _ZN08NumbaEnv8__main__5Scale11density_249B42c8tJTIeFCjyCbUFRqqOAK_2f6h0kCng1maAA_3d_3dEN8instance8jitclass35Scale_2318ae2d780_3cbias_3aint64_3eEN8instance8jitclass22Sand_2318ae00bb0_3c_3eE":
        .asciz	"missing Environment: _ZN08NumbaEnv8__main__5Scale11density_249B42c8tJTIeFCjyCbUFRqqOAK_2f6h0kCng1maAA_3d_3dEN8instance8jitclass35Scale_2318ae2d780_3cbias_3aint64_3eEN8instance8jitclass22Sand_2318ae00bb0_3c_3eE"

        .comm	__ZN08NumbaEnv8__main__7density12_3clocals_3e17_3clambda_3e_2410B42c8tJTIeFCjyCbUFRqqOAK_2f6h0kCng1maAA_3d_3dEN8instance8jitclass22Sand_2318ae00bb0_3c_3eE,8,3
        .comm	__ZN08NumbaEnv8__main__4Sand12density_2411B42c8tJTIeFCjyCbUFRqqOAK_2f6h0kCng1maAA_3d_3dEN8instance8jitclass22Sand_2318ae00bb0_3c_3eE,8,3
.subsections_via_symbols

I don't pretend to understand how numba works under the covers, so maybe most of that will be optimized away in more relevant circumstances, but it sure looks scary.

It's interesting to compare this with the julia approach:
struct Sand end
struct Clay end
struct Scale
    bias::Int
end
density(::Sand) = 1850
density(::Clay) = 1300
density(scale::Scale, soil) = density(soil) + scale.bias

sand = Sand()
clay = Clay()
scale = Scale(10)
for soil in [sand, clay]
    println("$soil density $(density(scale, soil)) kg/m³")
end

And we can view the native code with @code_native density(scale, sand), the entire output of which is just
	.section	__TEXT,__text,regular,pure_instructions
        movl	$1850, %eax                     ## imm = 0x73A
        addq	(%rdi), %rax
        retq
        nopl	(%rax)

The type-based dispatch system was able to figure out that Sand's density is always 1850, and this got hard coded into the assembly.  That looks a lot like this little section of numba's output:
__ZN8__main__5Scale11density_249B42c8tJTIeFCjyCbUFRqqOAK_2f6h0kCng1maAA_3d_3dEN8instance8jitclass35Scale_2318ae2d780_3cbias_3aint64_3eEN8instance8jitclass22Sand_2318ae00bb0_3c_3eE:
        movl	$1850, %eax
        addq	(%rcx), %rax
        movq	%rax, (%rdi)
        xorl	%eax, %eax
        retq

If we use scale.density(soil) in a hot loop inside another jitted function, does numba figure out that this is all that is needed?

Thanks for diving deeper on that. It seems like lowering process for jitclass isn’t too smart there.  If there was a dispatch system like Julia’s with Numba functions, one probably wouldn’t even need methods, just the struct itself.

If we use scale.density(soil) in a hot loop inside another jitted function, does numba figure out that this is all that is needed?

Rereading the assembly, I think so: we know that jitted functions are callable from the interpreter and from jitted code (and I guess this distinction doesn't exist in Julia), so there are two entry points, the one with the shorter name you quoted is probably the one used for jit code, while the other needs to be a bit fancier.  To wit, when using the method from another jit function
@nb.njit
def call_scale(scale, soil):
    return scale.density(soil)

call_scale(scale, sand)

the asm is virtually identical as yours: two entry points, one super simple with the literal value 1850, and a second long one.  Just to confirm the idea, we can take a really simple function,
@nb.njit
def call_foo(a, b):
    return a + b

call_foo(42.03, 43.0420)

and inspect asm,
__ZN8__main__13call_foo$2417Edd:
        addsd   %xmm1, %xmm0
        movsd   %xmm0, (%rdi)
        xorl    %eax, %eax
        retq

        .globl  __ZN7cpython8__main__13call_foo$2417Edd
        .p2align        4, 0x90
__ZN7cpython8__main__13call_foo$2417Edd:
        .cfi_startproc
        pushq   %r15
        .cfi_def_cfa_offset 16
        pushq   %r14
        .cfi_def_cfa_offset 24
        pushq   %r13
        .cfi_def_cfa_offset 32
...

From experience and I guess the Numba design by default, we know that calling a simple function like call_foo from a hot loop inside a jitted function should pose no problem, it must be using the short entry point and not the long one.  Also from experience, calling Numba jit functions from the interpreter is a bit slow, so again I'd guess this is what's going on.
That aside, I think a compile-time multiple-dispatch approach would be doable with generated_jit or overload with a straightforward design like
@generic
def density(soil): pass

@method(density, Sand)
def density(sand): return 1850

where generic would be generate something with generated_jit according to the methods registered with method.  To achieve something like Julia's type hierarchy for dispatch, one could map abstract types to regular classes and structs to jitclasses (since jitclasses can inherit from a regular class), which would be less classes and just structs.  There'd be no parametrized types or arrays of user defined stuff but still useful for writing flexible generic code.

Numba dev docs mention polymorphic/multiple dispatch here so perhaps if there were a way to just add new implementations to existing functions for a new set of types, the dispatch machinery wouldn't have to be reimplemented.


BADEK alternate meaning

BADEK alternate meaning created by<EMAIL_ADDRESS>on 2017-01-13 13:57:50

<EMAIL_ADDRESS>replied, Here's what we have for badek now:
mat (or similar object) used as shroud (and brought or sent to house of deceased).
badek a bar, chedecholl, mamed el omail er a ulekoad el chad, mlamengodech, me a udoud a badek.
Looks like Kerresel has something about money but not Josephs.  Is it correct that badek is to kemeldiil as belduchel is to ocheraol?

<EMAIL_ADDRESS>replied, To reiterate Masaharu's definition: badek a uldelid era rasch ma klauchad. Now, it is inclusive to friends, close acquaintances, and co workers as well.


vertical rtl not working

when using vertical:true with rtl:true ,the slides dont show

Can confirm, is there a way to "revert" the order of the slides in vertical mode?

Hey, there's another issue we're using to track this request. #818
Thanks


Test cases for a new model (not saved to the DB)

I have some test cases that depend on the model being new (not yet saved to the database).
Is there a way to do this in ember-cli-mirage and pass it to the visit test helper somehow.
let booking = server.createNew('booking'); //It would be good if this would return an empty booking object.
visit('my/route', booking); //not exactly sure how to pass a model using the visit helper

Maybe it makes no sense using ember-cli-mirage in this context and I should import the model directly into the test.

closing, this is not really an issue with ember-cli-mirage.

I think this provides the feature you are describing:
https://github.com/samselikoff/ember-cli-mirage/pull/459


fix wrong participant.json manifest file path on copy-webpack-plugin

Solve this issue #35

updated mas @praditautama


Loading SA is suddenly much slower than before

As of the most recent commit, the app loads very slowly. At first I thought i t was brokem, because it was only displaying labels.

For some reason it's taking a long time to receive the services data compiled_AR.json specifically from the server. Maybe changing the server config could fix it, that's all I know for now.

Today, it took 21 seconds for the page to load from a pretty decent internet connection. @tuckerbuchy @Jareechang - can one of you look at this?

It also took 24 seconds to switch to English.

Closing this, it should be fixed on the real installation and is therefore not a blocker for launch. Cannot reproduce locally and not worth debugging the issue on the dev server.


[fixed] Ensure after-open css transitions work in Safari 14 & Mobile Safari

Fixes #846.
Changes proposed:

Use requestAnimationFrame to delay the after-open class from being added until it mounts with inital CSS

Upgrade Path (for changed or removed APIs): n/a
Acceptance Checklist:

[x] The commit message follows the guidelines in CONTRIBUTING.md.
[x] Documentation (README.md) and examples have been updated as needed.
[x] If this is a code change, a spec testing the functionality has been added.
[x] If the commit message has [changed] or [removed], there is an upgrade path above.

Additional notes:
It struck me that perhaps the issue was that when the modal is mounted, it is immediately mounted with the open state and initially renders with the after-open class. It's actually kind of surprising, then that Chrome shows a transition, when technically nothing is really changing. Perhaps the old browser interpreter is inferring developer intent or doing something wrong. ¯\_(ツ)_/¯
So I thought it was worth a try to ensure that the modal mounts in the closed state first, and then apply the open class. (See modified sandbox) And that seems to do the trick! But since it's likely a problem affecting everyone, I decided to try fixing it directly in react-modal so everyone could benefit from working transitions in Safari 14 & mobile Safari.
Moral: When using CSS transitions, ensure that the component is rendered first before applying the class that triggers the transition. Test in Safari 14 to ensure you got it right, as other/older browsers seem to be more lenient.

Coverage decreased (-0.5%) to 82.152% when pulling 848513e3057711ba53be5e60c555965426beba75 on benjaminpadula:fix/css-after-open-transition-in-safari-14 into c9d8e2d33a15ca10215ce33ed01c5479bb0ceb62 on reactjs:master.

Coverage decreased (-0.5%) to 82.152% when pulling 848513e3057711ba53be5e60c555965426beba75 on benjaminpadula:fix/css-after-open-transition-in-safari-14 into c9d8e2d33a15ca10215ce33ed01c5479bb0ceb62 on reactjs:master.

Hey @diasbruno, it's been a while since I created this PR. Yes, using requestAnimationFrame() improves the behavior here, but I have still found that it appears to skip the transition maybe 10% of the time? I have had more success locally by using setTimeout() with a 10-15ms delay.
What are your thoughts on this? Would you like me to:
A) Open a new PR using the setTimeout approach?
B) Merge this PR as-is?
C) Solve the problem in some other way?

@benjaminpadula Can you try adding a class (with default properties like left: 0) to be animated. Maybe it's missing the default properties to be animated (?)

Any updates on this issue? Is there another PR to watch for a different approach to fix this? Is there any workaround in the meantime?

@pwfisher You can try @benjaminpadula's branch...It would help validate the fix.

Seems I can't, and I think it's due an error in react-modal's package.json, "main" entry.
"react-modal@github:benjaminpadula/react-modal#fix/css-after-open-transition-in-safari-14":
  version "3.12.1"
  resolved "https://codeload.github.com/benjaminpadula/react-modal/tar.gz/848513e3057711ba53be5e60c555965426beba75"

Error: Cannot find module '/Users/patrickfisher/Code/figs/syconium/node_modules/react-modal/lib/index.js'. Please verify that the package.json has a valid "main" entry

Use the https version:
  react-modal: "https://github.com/benjaminpadula/react-modal#fix/css-after-open-transition-in-safari-14"

@pwfisher @diasbruno ran into a similar problem. lib is in the .gitignore, so the forked version is missing that and can't be consumed as a package. I made my own fork and committed with lib included: https://github.com/atomanyih/react-modal/tree/fix/css-after-open-transition-in-safari-14
There are no other changes from @benjaminpadula's branch

@atomanyih Can you make a patch to remove lib from the .gitignore, please...

@diasbruno Sure, do you want me to make a PR against the master branch of react-modal?

@atomanyih Yes, that would be great.

@diasbruno Sure, do you want me to make a PR against the master branch of react-modal?

Any updates on the status of your branch? Will it be merged soon? Thnx

Sorry for the delay on this...I'm going to push this PR forward as it doesn't seem to cause problems.

Merged manually. See 827796d.
Thanks, @benjaminpadula.

@diasbruno are you planning to release a new version to NPM?


use a list of sites to chose from rather than entering the name manually

- Summary
netlify link makes you manually enter the name of your site, which may result in human error and a tedious process. I wanted to give users a list of sites so there's no human error involved.
- Test plan

- Description for the changelog
Better DX for linking sites in netlify link
- A picture of a cute animal (not mandatory but encouraged)

LGTM - have we tested on an account with lots of sites?

@biilmann should be good with many sites:

I don't think I'd count ~15 sites as "many". Some people have hundreds!

best to use autocomplete/filter to handle many sites


better prune and system df

- What I did

fix some typos;
add an error in log
change args number in system df

- How I did it
- How to verify it
- Description for the changelog

- A picture of a cute animal (not mandatory but encouraged)
Signed-off-by: allencloud<EMAIL_ADDRESS>

LGTM

LGTM


Support for group-tab and code-tab in Markdown #71

Major refactor of tabs, group-tabs and code-tabs:

Removes hard-coded reStructuredText for group and code tabs
group and code tab directives now subclass TabDirective to remove duplicated run code
TabDirective and TabsDirective now subclass SphinxDirective to improve access to env

Fixes #71

May need further work to meet tests

well that was quick lol

Just wait for #76 to be merged, then we can see the damage

Sounds good to me, thanks!

My output HTML seems to have lost its indentation, so the XML is no longer matching. @chrisjsewell any idea where/how the indentation is determined?

This test suite is a legacy thing I'm afraid, I haven't touched it.
If/when I do any more work on this repo, I would look to rewrite it with similar pytest fixtures as used in myst-parser, etc

obviously you're welcome to give it a go yourself 😬

Yeah I might have a stab at moving the tests over to pytest... no promises though :sweat_smile:
My question above was actually a general sphinx one - do you know how sphinx determines HTML indentation? I presumed it would just indent with each node level, but the HTML sections from my code here have no indentation at all. I'm not sure what I've broken essentially

Don't know, but in myst-parser etc I use beatifulsoup to prettify the HTML before diffing with pytest-regressions, which a) standardises the HTML and b) provides better error feedback

Thanks, that'll be really useful! AssertionError: XML does not match here isn't the most informative feedback

Ready to go after #88

Sorted 😄

Bonus to close #34


[Bug]: Dropping Columns doesn't work

What happened?
I am unable to drop columns from tables using $table->dropColumn();
I have tested this using a local and remote database and I keep getting the error shown in the screenshot below.

How to reproduce the bug
Create two migrations.
The first migration to create a blogs table with some columns
Schema::create('blogs', function (Blueprint $table) {
    $table->id();
    $table->string('title');
    $table->string('slug');
    $table->text('description');
    $table->text('content');
    $table->string('image');
    $table->timestamps();
});

Then another migration to drop some columns from the blogs table
Schema::table('blogs', function (Blueprint $table) {
    $table->dropColumn('description');
    $table->dropColumn('content');
});

Perform the migration using php artisan migrate
Package Version
1.0.13
PHP Version
8.3.12
Laravel Version
11.27.2
Which operating systems does with happen with?
macOS
Notes
No response

I have this same issue


Generalized REST switch to enable templating and configurable timeout.

Description:
The REST switch was too restrictive for my application. The body_on and body_off fields in the configuration were constants (not templated) and hence could not depend on the state of the system.
Also, the on/off state of the switch was determined by exactly matching with body_on with
the response to the GET request. Again, this was too restrictive, because my device (radio thermostat) can include other attributes in response -- attributes that were not a part of the on/off POST requests. I have a config file below to illustrate this use case.
I think that my change is backward compatible and existing configuration files should work as before.
Related issue (if applicable): fixes #
Pull request in home-assistant.io with documentation (if applicable): home-assistant/home-assistant.io#
Example entry for configuration.yaml (if applicable):
switch:
  platform: rest
  resource: http://thermostat.lan/tstat/remote_temp
  name: "thermostat track a remote temperature sensor"
  value_template: '{{ value_json.rem_mode == 1 }}'
  body_on: '{"rem_temp":{{states.sensor.bedroom_temp.state}}}'
  body_off: '{"rem_mode":0}'
  timeout: 50

Usecase:
Turning on the above switch makes a radio thermostat track a remote temperature sensor (states.sensor.bedroom_temp in this case). According to its API, to turn on this remote temperature sensor mode, one cannot simply POST
 '{"rem_mode":1}'. Instead, one has to post the (current) value of the remote temperature sensor.
Combining the above switch with the following automation makes the thermostat makes decisions based on the temperature sensor in the bedroom instead of the temperature sensor inside the thermostat, which may be in some other room:
automation:
  alias: thermostat update bedroom temp
  trigger:
    platform: state
    entity_id: sensor.bedroom_temp
  condition:
    condition: state
    entity_id: switch.thermostat_track_a_remote_temperature_sensor
    state: 'on'
  action:
    service: switch.turn_on
    entity_id: switch.thermostat_track_a_remote_temperature_sensor

Checklist:
If user exposed functionality or configuration variables are added/changed:

[x] Documentation added/updated in home-assistant.io

https://github.com/home-assistant/home-assistant.github.io/pull/1030
If code communicates with devices, web services, or a:

[ ] Local tests with tox run successfully. Your PR cannot be merged unless tests pass
[ ] New dependencies have been added to the REQUIREMENTS variable (example).
[ ] New dependencies are only imported inside functions that use them (example).
[ ] New dependencies have been added to requirements_all.txt by running script/gen_requirements_all.py.
[ ] New files were added to .coveragerc.

If the code does not interact with devices:

[ ] Local tests with tox run successfully. Your PR cannot be merged unless tests pass

There are many "typing" errors. But no such error is in the parts I changed.

[ ] Tests have been added to verify that the new code works.

🐬

Great job!


Message buttons overlap on iPhone

Perhaps   can change the css a little (margin + display):
https://github.com/Phuks-co/throat/blob/master/app/static/css/main.css#L1468
.pmessage p.container a {
    margin: 4px 4px 0 0;
    cursor: pointer;
    padding: .5em 1em;
    border-radius: 3px;
    background-color: rgba(0, 0, 0, 0.06);
    display: inline-block;
}


Anyone have an example of how to implement subscriptions?

Hi i'm trying to implement subscriptions into next.js with graphcool :) Anyone have any examples of how this can be done.. using the new version of ApolloClient!
Thanks

With graphql? Try subscriptions-transport-ws

I mean't more of a full example :D

+1

Have you seen the npmjs links to GitHunt Api/React? https://github.com/apollostack/GitHunt-API https://github.com/apollostack/GitHunt-React

This worked for me
import { ApolloClient } from 'apollo-client'
import { split } from 'apollo-link'
import { HttpLink } from 'apollo-link-http'
import { WebSocketLink } from 'apollo-link-ws'
import { getMainDefinition } from 'apollo-utilities'
import { InMemoryCache } from 'apollo-cache-inmemory'
import fetch from 'isomorphic-fetch'
import ws from 'websocket'

let apolloClient = null
const ssrMode = !process.browser

const GRAPHQL_URL = 'https://api.graph.cool/simple/v1/cjeuwkokg3lkm016474bx4h46'
const WS_URL = 'wss://subscriptions.us-west-2.graph.cool/v1/cjeuwkokg3lkm016474bx4h46'

// Polyfill fetch() on the server (used by apollo-client)
if (ssrMode) {
  global.fetch = fetch
}

function create(initialState) {
  // Create an http link:
  const httpLink = new HttpLink({
    uri: GRAPHQL_URL,
    credentials: 'same-origin', // Additional fetch() options like `credentials` or `headers`
  })

  let link
  if (!ssrMode) {
    // Create a WebSocket link:
    const wsLink = new WebSocketLink({
      uri: WS_URL,
      options: {
        reconnect: true,
      },
      webSocketImpl: ws.client,
    })

    // using the ability to split links, you can send data to each link
    // depending on what kind of operation is being sent
    link = split(
      // split based on operation type
      ({ query }) => {
        const { kind, operation } = getMainDefinition(query)
        return kind === 'OperationDefinition' && operation === 'subscription'
      },
      wsLink,
      httpLink
    )
  } else {
    link = httpLink
  }

  return new ApolloClient({
    connectToDevTools: process.browser,
    ssrMode: ssrMode, // Disables forceFetch on the server (so queries are only run once)
    link: link,
    cache: new InMemoryCache().restore(initialState || {}),
  })
}

export default function initApollo(initialState) {
  // Make sure to create a new client for every server-side request so that data
  // isn't shared between connections (which would be bad)
  if (!process.browser) {
    return create(initialState)
  }

  // Reuse client on the client-side
  if (!apolloClient) {
    apolloClient = create(initialState)
  }

  return apolloClient
}

This package will help you to create an ApolloClient including subscriptions: next-with-apollo

Here is an example of using subscriptions with graphcool :
subscription newPosts {   Post(     filter: {       mutation_in: [CREATED]     }   ) {     mutation     node {       description       imageUrl     }   } }
you can learn more about subscriptions by viewing this link : https://www.graph.cool/docs/reference/graphql-api/subscription-api-aip7oojeiv/


[macOS] Print full source paths in the trace.

Adds full (or relative) paths to the trace, for consistency with other platforms.
Before:
[17] SceneTree::initialize() (in godot.macos.editor.dev.arm64) (scene_tree.cpp:572)
[18] OS_MacOS::run() (in godot.macos.editor.dev.arm64) (os_macos.mm:800)
[19] main (in godot.macos.editor.dev.arm64) (godot_main_macos.mm:84)

After, with debug_paths_relative=false:
[17] SceneTree::initialize() (in godot.macos.editor.dev.arm64) (/Volumes/Backup/Projects/godot/scene/main/scene_tree.cpp:572)
[18] OS_MacOS::run() (in godot.macos.editor.dev.arm64) (/Volumes/Backup/Projects/godot/platform/macos/os_macos.mm:800)
[19] main (in godot.macos.editor.dev.arm64) (/Volumes/Backup/Projects/godot/platform/macos/godot_main_macos.mm:84)

After, with debug_paths_relative=true:
[17] SceneTree::initialize() (in godot.macos.editor.dev.arm64) (./scene/main/scene_tree.cpp:572)
[18] OS_MacOS::run() (in godot.macos.editor.dev.arm64) (./platform/macos/os_macos.mm:800)
[19] main (in godot.macos.editor.dev.arm64) (./platform/macos/godot_main_macos.mm:84)

Thanks!


Global TOC

As I've been using the wiki, I'm still finding it a bit tricky to get around. There is the alphabetized listing of articles, which is helpful, but I wonder if something more akin to a Table of Contents page would be helpful. This would traverse the created pages (including sub folders) and come up with a hierarchical view.
I have a prototype for this, it includes looking in sub directories and specifying depth. This page would have a slight leg up on the A-Z view, as once sub folder are in and the user starts organizing the wiki, that organization would be clear to see in this view. I will push a PR for you to look at.

PR submitted here: https://github.com/redimp/otterwiki/pull/33

I like the idea of a global TOC. To make it a complete feature it should include all the headlines of the pages (and link to them).

I did consider this (since there is already a mechanism to do get the page toc, which could be utilized), but didn't implement it for two reasons:

It could quickly become a pretty heavy weight operation (opening/parsing every file on disk)
It didn't seem to fit with the 'scope' of what the global TOC would provide. It's a high-level overview of the structure of the wiki (again, this would make more sense with sub directories).

Thoughts?

I’m on the road, so only a quick answer.

My understanding was that a toc should be more complete then just a page index. Both must be able to deal with subdirectories (at least optional).

I agree, this might be a more heavy disk operation. But for getting more than the Page Index out of it, it might be worth the price. (Or a user willing to pay in the waiting time.) In comparison, a search would take the exact same disk lifting.

I would like to give this a shot, will do this later today.

On 15. Oct 2022, at 13:37, Marc Leonard @.***> wrote:

I did consider this (since there is already a mechanism to do get the page toc, which could be utilized), but didn't implement it for two reasons:
It could quickly become a pretty heavy weight operation (opening/parsing every file on disk)
It didn't seem to fit with the 'scope' of what the global TOC would provide. It's a high-level overview of the structure of the wiki (again, this would make more sense with sub directories).
Thoughts?
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you commented.

Yes true! It is the same as search!

I finished the global toc, including the headers of the pages, please check the feature-global-toc if this is useful and works as intended.
Preview:

Will work on this later today (have to check if the subdirectories work as intended.

PageIndex has the feature of displaying the headers:


Update containerd and runc

Signed-off-by: Kenfe-Mickael Laventure<EMAIL_ADDRESS>--
Fixes #25880, #25470, #26995

LGTM

Doesn't update the libcontainer vendoring, do we want to do that too?

Oh, yes, thanks for pointing it out!

LGTM

There's another bug in containerd-shim that I'm in the process of fixing. If we could wait until I update this. The PR is here

Updated with aforementioned fix.

@mlaventure hah, rebase :)
LGTM though

Rebased :)

The commit for github.com/godbus/dbus in https://github.com/opencontainers/runc/blob/master/Godeps/Godeps.json is v3, not v4.0.0 as it is on docker/docker which is probably an error - noticed this when looking at the vendoring before.

Also github.com/syndtr/gocapability needs updating and also maybe github.com/golang/protobuf/proto

@justincormack

libnetwork is using v4 for godbus/dbus (and runc v3), hence the difference
for golang/protobuf, containerd has the most recent vendoring followed by swarmkit then runc (docker/docker has the one from containerd atm)
libnetwork is using a older commit than runc for syndtr/gocapability (which is the one used in docker/docker atm)

How do we reconcile this mess? :sweat_smile:

@mlaventure I'd better ignore deps of deps for now and use docker as source of truth - it has most extensive tests among all other projects.

ping @mlaventure - per https://github.com/docker/docker/issues/26995#issuecomment-251742744 this fixes a regression in 1.12.2; are all these changes good to go into 1.12.2?

@thaJeztah there are new features in here eg ambient capabilities.

@mlaventure ugh. ok. LGTM, as I don't think this should matter now for the uses here.

@thaJeztah going to have to make a different PR for 1.12.2, the containerd used in it is based on a branch to avoid pulling all the GRPC changes.
I've opened a PR here
Will make one on 1.12.x branch once it's merged

@mlaventure clear, thanks!

ping @mlaventure - was a bump / Cherry-pick for 1.12.2 created?

Yes I think so.
On 9 Oct 2016 11:05 a.m., "Sebastiaan van Stijn"<EMAIL_ADDRESS>wrote:

ping @mlaventure https://github.com/mlaventure - was a bump /
Cherry-pick for 1.12.2 created?
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub
https://github.com/docker/docker/pull/27160#issuecomment-252474313, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AAdcPDt6U9ly4BquHLPMnFlQQlfBtlgSks5qyK5sgaJpZM4KO2Qm
.

@thaJeztah
Yes it was: https://github.com/docker/docker/pull/27196


Suggestion: Exclude html comments

I use html comments in Markdown so it would be great if there was an option to exclude words from inside comments.

+1

I'd love to see a PR for this. But please add a single option to exclude comments, and then detect them with a mapping indicated from the file type.

Hello guys! I had the same need to not comment HTML. Me friend made a little addition in the code. Do you think it's interesting that I pull a request?

@vict0rg - do you still plan on submitting a PR or need help doing so? I'd love to have this feature as well. Was getting ready to fork and work on it when I saw this open issue.

@danielbayley feel free to submit a PR if you wont hear anything back from @vict0rg :)

@OleMchls Wrong d! 😉
@davidlday…

My bad! Sorry

@OleMchls - No worries! I'll wait a little bit longer for a response, then proceed. ty!

I had some well working regexes, put them in and saw how easy the code was to modify. Didn't realise there was a discussion on it until I actually made the PR. Woooops.
Hope no one is offended. @davidlday @vict0rg
I also have a separate option for excluding block quotes (which is helpful in case you need to count your own words rather than other people's words).
#94

I've thought about it, albeit only now. It does make sense, to have this, but:
Then the settings already have an array containing the file extensions on which the package would be auto-activated. So when implemented properly, there needed to be an input with grammars on which the stripping of text is activated.
I'm not familiar with the individual grammars, but I can't see why the already existing stripping of code blocks is not wrapped in a grammar check, when this would be warranted for html-comments.
Conclusion: Thinking more about it, and looking at other feature requests, maybe it would be actually even more helpful, to instead have a settings object with grammars and regexes that are stripped before counting? This way the individual user has full control over which regex can be used in which grammar?
Either way, it's implemented now. :)

As my personal time is limited and I do not want this project to get stuck in a dead end. We'll move towards a more open contribution model. As the PR provided by @mbroedl added value to the project I am fine merging it.  But I or anyone else could always file a PR for that. And I do not want to be a gate keeper to added value for this project.

I think this issue can be closed with that. I will release a new version later today. LMK if you think it should not be closed.


Release 1.1

Depends on #89
Anyone else have anything they want in this release?

Needs a version bump for a release

I'm going to wait until the top of the hour, ~40m from now, to wait for more feedback on this, then cut a release.

Together with adding tag for the new release may be add a tag for the resurrection point, just for the history. Name it, would say, phoenix, renaissance, end-of-dark-ages, or any other at your choice :smiley:

Do you want that tag right before @ landed, or right after?

Tag the first new commit, 2a3ca2d6a531d4eadcb53478d558f0bb90364f24

https://github.com/peggyjs/peggy/tree/Renaissance

Let me just say I'm so happy to see the project re-ignited :) So thankful to everyone involved!

https://github.com/peggyjs/peggy/releases/tag/v1.1.0


any chance of re implementing the fake base station support?

hi i notice a few games i play will not register the controllers unless i use an older version of alvr that still has the fake base station support notably spectro the luigi's mansion type game but the older versions have bad stuttering and lag compared to ex 7 thanks

The fake base stations are removed because the RiftS, that ALVR is emulating, has no base stations either.
Does the RiftS work with those games?

ah yes i see what you're saying. well funny you should say, im using virtual desktop now and i get the same issue where when i get to the main menu in spectro the controllers wont recognise (only in game, they are still connected and working outside of the game) even though virtual desktop has fake base station support so it must be an issue with the game itself. but still strange how it worked in alpha 5 version of alvr.

Around the same time, oculus changed their required api for tracking. Does it still work with alpha5?


Remove setup.py from coverage reports.

Don't forget this itty one!  :-)

:) Done.
One concern I had with the coverage stuff is that we may need to exclude
some of the drivers from these checks. We can't really run unit tests of
most because they need real hardware so they shouldn't count against the
coverage percentage.
On Mon, Jan 30, 2017 at 9:15 AM, Tom Swirly<EMAIL_ADDRESS>wrote:

Don't forget this itty one! :-)
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/ManiacalLabs/BiblioPixel/pull/97#issuecomment-276072595,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AA6a6qe6FIS8v_XCbbEVTYn1KDKmZ0OOks5rXfCPgaJpZM4LwhQ3
.

Actually, I'm closing this without submitting because the change I submitted earlier fixes this issue in a better way.  :-D

Well, excluding files is as simple as adding a line in the config file.
To be honest, this little project took longer than I thought.  I'd dropped
coverage into a couple of other projects in literally a few minutes so I
thought BP would be the same.
On the other hand, that "importing BP in its setup.py" would have burned us
eventually.  :-D
On 30 January 2017 at 15:26, Dread<EMAIL_ADDRESS>wrote:

:) Done.
One concern I had with the coverage stuff is that we may need to exclude
some of the drivers from these checks. We can't really run unit tests of
most because they need real hardware so they shouldn't count against the
coverage percentage.
On Mon, Jan 30, 2017 at 9:15 AM, Tom Swirly<EMAIL_ADDRESS>wrote:

Don't forget this itty one! :-)
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
<https://github.com/ManiacalLabs/BiblioPixel/pull/
97#issuecomment-276072595>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/AA6a6qe6FIS8v_
XCbbEVTYn1KDKmZ0OOks5rXfCPgaJpZM4LwhQ3>

.

—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/ManiacalLabs/BiblioPixel/pull/97#issuecomment-276075095,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAPdsuQquI1nRzRLr80ZJew1Gra0MgJlks5rXfL7gaJpZM4LwhQ3
.

--
/t
http://radio.swirly.com - art music radio 24/7 366/1000


Arabic translation

Translation to arabic shows the arabic letter sequence in reverse.
Latin-languages syntax is written from left to right. Arabic syntax is written from right to left.
When trying to translate from a latin-language(english, french, spanish...) The translation in arabic come up with a writing from left to right.

Please provide the output of your trans -V.
In order to output right-to-left languages correctly on the screen it is required to install FriBidi. See Recommended Dependencies.

Hello thank you for the fast feedback. Here is the output of my trans -V:
Package: libfribidi-bin
Version: 1.0.8-2
Priority: optional
Section: universe/utils
Source: fribidi
Origin: Ubuntu
Maintainer: Ubuntu Developers<EMAIL_ADDRESS>Original-Maintainer: Debian Hebrew Packaging Team<EMAIL_ADDRESS>Bugs: https://bugs.launchpad.net/ubuntu/+filebug
Installed-Size: 47.1 kB
Depends: libc6 (>= 2.4), libfribidi0 (>= 0.19.2)
Breaks: libfribidi0 (<< 0.19.2-3)
Replaces: libfribidi0 (<< 0.19.2-3)
Homepage: http://www.fribidi.org/
Download-Size: 9,424 B
APT-Manual-Installed: no
APT-Sources: http://ca.archive.ubuntu.com/ubuntu focal/universe amd64 Packages
Description: Free Implementation of the Unicode BiDi algorithm (utility)
 FriBiDi is a BiDi algorithm implementation for Hebrew and/or Arabic
 languages.
 This package contains a command line interface for the fribidi library.

I have downloaded the latest version (<IP_ADDRESS>) of the software from https://www.soimort.org/translate-shell/trans the same issue is persistent, when running ./trans :ar "hello" and ./trans :ar -bidi "hello".
I have checked, and yes I have the libfribidi-bin library installed.
Thank you :smile_cat:

What terminal emulator are you using?

This issue should be related to #288, which was fixed for at least some types of terminal emulators. You can try the -no-bidi option if the default -bidi doesn't work.
Closing since I can't investigate any further without information about the terminal emulator.


⚠ Turnos 24x7 has degraded performance

In 8f36623, Turnos 24x7 (https://reservas-dev.alternativasinteligentes.com/) experienced degraded performance:

HTTP code: 200
Response time: 1123 ms

Resolved: Turnos 24x7 performance has improved in b06666e.


Make date field date and add time field

do work in issue 81

change collection
change create jam session
change jam card


Levels should not always be erased

The interaction of levels and polymorphism pose problems, as demonstrated by the following example:
open import Haskell.Prelude
open import Agda.Primitive using (lzero)

k : a → b → a
k x _ = x
{-# COMPILE AGDA2HS k #-}

testK : Nat
testK = k 42 lzero
{-# COMPILE AGDA2HS testK #-}

In the use site we erase the level argument completely, resulting in the wrong number of arguments when applying the polymorphic k function:
import Numeric.Natural (Natural)

k :: a -> b -> a
k x _ = x

testK :: Natural
testK = k 42

    • Couldn't match expected type ‘Natural’
                  with actual type ‘b0 -> a0’
    • Probable cause: ‘k’ is applied to too few arguments
      In the expression: k 42
      In an equation for ‘testK’: testK = k 42
  |
9 | testK = k 42
  |         ^^^^

Very interesting edge case!
What would be the proper way to prevent this though?
Should agda2hs replace it with () when given to specialized polymorphic functions? Parametricity ought to mean it's fine. But I have no idea how to reliably check this. In particular because the following would still fail:
open import Haskell.Prelude
open import Agda.Primitive using (lzero)

k : a → b → a
k x _ = x
{-# COMPILE AGDA2HS k #-}

l : Level → Nat
l = k 42
{-# COMPILE AGDA2HS l #-}

testK : Nat
testK = l lzero
{-# COMPILE AGDA2HS testK #-}

Which would yield:
import Numeric.Natural (Natural)

k :: a -> b -> a
k x _ = x

l :: Natural
l = k 42

testK :: Natural
testK = l

Seems to me like the only proper fix would be to always compile Level to () but that's a bit sad.

I believe the proper fix here is to decide which arguments to drop based on the uninstantiated type of the function, i.e. when compiling a list of arguments we should decide which arguments to erase based on the type of the function but not on the values of the (previous) arguments. In this case we would then get to actually compiling the lzero term, which should throw an error (since there is no level type in Haskell).

But if you look at the example I posted with a partial application of a polymorphic function, the term lzero is an argument to a function whose argument type has already been unified with Level. And this function is completely fine and does not use levels at all. So I don't think this check would be enough.

Oh you are correct, it would still be a problem. Thinking about it a bit more, the real problem is that the type parameter is instantiated to Level, which shouldn't be allowed because Level is not a Haskell type. So we could fix it perhaps by checking that (non-erased) type arguments are actually valid Haskell types before dropping them.

More careful checking of type parameters reveals an issue in our Prelude implementation: the guard function in Haskell.Control.Monad has the following signature:
guard : {{ MonadFail m }} → (b : Bool) → m (b ≡ True)

However b ≡ True is not a valid Haskell type! With the check, this causes problems when guard is used in a larger expression, e.g. guard ... >> .... To fix this, we should change the type of guard to
guard : {{ MonadFail m }} → (b : Bool) → m (Erase (b ≡ True))

so its type compiles to the proper MonadFail m => Bool -> m ().


v0.5

Codecov Report

Merging #55 (2d972a9) into main (4a24400) will not change coverage.
The diff coverage is 0.00%.

@@          Coverage Diff          @@
##            main     #55   +/-   ##
=====================================
  Coverage   8.89%   8.89%
=====================================
  Files         44      44
  Lines       5297    5297
=====================================
  Hits         471     471
  Misses      4826    4826

Flag
Coverage Δ

oasysgh
1.83% <0.00%> (ø)

unitnumber
7.21% <0.00%> (ø)

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

Impacted Files
Coverage Δ

OasysGH/OasysPluginInfo.cs
57.89% <ø> (ø)

OasysGH/Units/UI/DefaultUnitsForm.cs
0.00% <0.00%> (ø)

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


hdf5_format.py method load_weights_from_hdf5_group() fails when you exclude layers

System information

Have I written custom code (as opposed to using a stock example script provided in TensorFlow): from https://github.com/ahmedfgad/Mask-RCNN-TF2 with some other TF2 fixes

OS Platform and Distribution (e.g., Linux Ubuntu 16.04):  Windows Pro

Mobile device (e.g. iPhone 8, Pixel 2, Samsung Galaxy) if the issue happens on mobile device:  n/a

TensorFlow installed from (source or binary):  via pip install

TensorFlow version (use command below):  v2.7.0-rc1-69-gc256c071bb2 2.7.0

Python version:   v3.7.3:ef4ec6ed12, Mar 25 2019, 22:22:05) [MSC v.1916 64 bit (AMD64)] on win32

Bazel version (if compiling from source):   n/a

GCC/Compiler version (if compiling from source):   n/a

CUDA/cuDNN version:   cuda_11.2.r11.2/compiler.29373293_0

GPU model and memory:   Quadro 2000 32GB

You can collect some of this information using our environment capture script
No windows version
Describe the current behavior
When you call load_weights() using the exclude= parameter you get errors like "You are trying to load a weight file containing 233 layers into a model with 229 layers.". Several issues have been posted about this type error but none of them addressed the fundamental problem which is in the method load_weights_from_hdf5_group().
if len(layer_names) != len(filtered_layers):
    msg = f'You are trying to load a weight file containing {str(len(layer_names))} layers into a '\
          f'model with {str(len(filtered_layers))} layers.'
    raise ValueError(msg)

Describe the expected behavior
Depending on which way layers count goes either being greater than or less than we can either auto-correct the layers or report the actual layer that is missing when it checks for matching layer sizes:
Contributing

Do you want to contribute a PR? (yes/no):  yes
Briefly describe your candidate solution(if contributing):
see https://github.com/tensorflow/tensorflow/pull/53739

Standalone code to reproduce the issue
Provide a reproducible test case that is the bare minimum necessary to generate
the problem. If possible, please share a link to Colab/Jupyter/any notebook.
Other info / logs Include any logs or source code that would be helpful to
diagnose the problem. If including tracebacks, please include the full
traceback. Large logs and files should be attached.

@quaesitor-scientiam
Could please share complete code to replicate the issue or a gist with the error reported for us to help you resolve it.


Change logging in MOSEK

I propose two changes in the MOSEK log output.

Remove automatic printing of infeasibility report/certificate. Since cvxpy does not pass names to MOSEK, this was not very useful in the first place (one has to know the mapping from cvxpy variables to the x variable in MOSEK to understand the report). Now that CVXPY dualizes continuous problems it is simply impossible even for a fairly expert user to immediately relate that report to the original problem. And that report can have lots of rows, obscuring the interesting part of the optimization log.
If we go verbose and log is enabled at all I see no reason why presolve log should be specifically disabled. It is just a few lines and sometimes can actually be informative.

Sounds good to me! Merging.

@rileyjmurray  Hi, I just upgrade my local cvxpy to the latest version v1.1.18, As @aszekMosek said, cvxpy do a mapping internal from cvxpy variables to variables in MOSEK (this do confuse me a lot because I just wanna check which constraint introduce infeasibility to my problem, and it seems hard to understand that report), but now, the LOG just told me the problem is DUAL_INFEASIBLE, I don't get more details
I just want to ask if there is a better way to debug my problem without using MOSEK directly? any suggestion would be helpful

As the documentation mentioned as follows:
https://docs.mosek.com/latest/pythonapi/parameters.html#mosek.iparam.infeas_report_auto
just set this TAG ON, someone can get the infeasible report from MOSEK, this more relates to MOSEK docs but cvxpy.
MOSEK provide some APIs for supporting problem analyzing, someone may check the link following for more detail
https://docs.mosek.com/latest/toolbox/analyzers.html

You can get the infeasibility report in the log if you set that parameter you mentioned. Like you noticed, it will not be very easy to relate the output the cvxpy problem.
https://docs.mosek.com/latest/pythonapi/debugging-infeas.html about infeasibility. Section 8.3.4 has more generic suggestions that can be used from cvxpy.
https://docs.mosek.com/latest/faq/faq.html#cvxpy how to set Mosek parameters from cvxpy.

@aszekMosek Hi, Based on your experiences thus far， I am wondering if solving the problem with MOSEK directly is a better way to debug an infeasible problem, for easy to check the infeasible report?


QAT Assist compression & the algorithm option for database backups

SQL Server 2022 brings in the ability to use QAT Assist compression for database backups, and thus the COMPRESSION option can now specify an ALGORITHM setting (either to use native compression or QAT) - can this be added to the backup proc?

I have been doing some more testing and released it.
https://ola.hallengren.com/versions.html


add function CreateVolumeFromSnapshot of dorado

What this PR does / why we need it:
According to dorado driver of openstack/cinder, implement the same function of opensds.
Special notes for your reviewer:
The volume copy status is right, but I did not check the files in a new volume that create from a snapshot , because the volume must be mounted , and the file system of volume is unknow, so if i want to check file in this volume ,   i should  'mkfs -> mount -> check', 'mkfs' cmd clear the file in this volume.
Is there someway to check these files in volume ?

Coverage decreased (-0.5%) to 43.574% when pulling 64c2e48d9ab7c53ac72f1a29810bb14599090917 on jtt17:snapshot into 64dbd87e483323054a3b758ec7fc2ce810176587 on opensds:development.


Note: duplicate definition of library class [android.net.http.SslError] in 'compileSdkVersion 23'

Note: duplicate definition of library class [android.net.http.SslError]
Note: duplicate definition of library class [android.net.http.SslCertificate]
Note: duplicate definition of library class [android.net.http.SslCertificate$DName]
Note: duplicate definition of library class [org.apache.http.conn.scheme.HostNameResolver]
Note: duplicate definition of library class [org.apache.http.conn.scheme.SocketFactory]
Note: duplicate definition of library class [org.apache.http.conn.ConnectTimeoutException]
Note: duplicate definition of library class [org.apache.http.params.HttpParams]

How can I fix it?

Found a solution to this?

this is not a loopj issue.
+info

I am getting the same issue

@iamatul9074 please use the new version, 1.4.10


Cookies not working, breaks any app using auth/session cookies

Cookies are not working. This means a phonegap app using cookies are not working because the application instance is not enable cookies. This breaks any cookie based session concept used by a remote server.
Can you include the cookie manager by default? This could be simply done by change the onCreate method in the PhoneGap app instance. For android this looks::
# additional import
import android.webkit.CookieManager;

@Override
public void onCreate(Bundle savedInstanceState)
{
    # additional cookie manager config
    CookieManager.setAcceptFileSchemeCookies(true);
    super.onCreate(savedInstanceState);
    super.init();
    // Set by <content src="index.html" /> in config.xml
    super.loadUrl(Config.getStartUrl());
    //super.loadUrl("file:///android_asset/www/index.html")
    com.adobe.mobile.Config.setContext(this.getApplicationContext());
}

Regards Roger

Just following up on this issue. The latest PG Dev App / CLI now uses the file:// rather than http://. I'm going to close this issue, but please let me know if you continue to have a problem!


Get route name when class is defined as callable

While this might not sound like a solid idea (pun intended), I think this is a valid case:
$app->get('/foo', 'App\Controller\StandardController::showPage')->setName('foo');
$app->get('/bar', 'App\Controller\StandardController::showPage')->setName('bar');

Different routes, same controller. I stumbled upon this case by implementing a "mini-CMS" where I want to show a text from DB based on the route name (or URL, but preferably from the route name). So I define all pages in a long list (like above) all pointing to the same controller. Based on route names, it fetches the right text.
Is this possible in Slim v3?

notation should be Controller:method, but that should work yeah.
Although you may find it easier to have single route there with some regex instead of several pointing to the same action.

Yes this is possible.


Ziro Dark Purple : Button to skip adding an existing song to playlist does not show

Describe the bug
The button for not adding a particular song to a playlist (because the song already exists in the playlist) does not show. The button functions properly but the text color is the same as the background color of the popup.
To Reproduce
Steps to reproduce the behavior:

Add a song to a playlist
Click add to playlist on the same song
Select the same playlist as mentioned in step 1
See error

Expected behavior
The text should be in a different color, preferably some color that is easily visible on the light purple background.
Screenshots

Logs
vendor~xpui.js:1 MaxListenersExceededWarning: Possible EventEmitter memory leak detected. 11 change listeners added. Use emitter.setMaxListeners() to increase limit
    at c (https://xpui.app.spotify.com/vendor~xpui.js:29:2656773)
    at t.o.addListener (https://xpui.app.spotify.com/vendor~xpui.js:29:2659055)
    at https://xpui.app.spotify.com/vendor~xpui.js:29:2091056
    at ka (https://xpui.app.spotify.com/vendor~xpui.js:29:2946534)
    at Ps (https://xpui.app.spotify.com/vendor~xpui.js:29:2970386)
    at t.unstable_runWithPriority (https://xpui.app.spotify.com/vendor~xpui.js:29:3190652)
    at hi (https://xpui.app.spotify.com/vendor~xpui.js:29:2916042)
    at Is (https://xpui.app.spotify.com/vendor~xpui.js:29:2970165)
    at ws (https://xpui.app.spotify.com/vendor~xpui.js:29:2954936)
    at vs (https://xpui.app.spotify.com/vendor~xpui.js:29:2953858)
    at https://xpui.app.spotify.com/vendor~xpui.js:29:2916314
    at t.unstable_runWithPriority (https://xpui.app.spotify.com/vendor~xpui.js:29:3190652)
    at hi (https://xpui.app.spotify.com/vendor~xpui.js:29:2916042)
    at yi (https://xpui.app.spotify.com/vendor~xpui.js:29:2916259)
    at vi (https://xpui.app.spotify.com/vendor~xpui.js:29:2916194)
    at As (https://xpui.app.spotify.com/vendor~xpui.js:29:2954357)
    at rl (https://xpui.app.spotify.com/vendor~xpui.js:29:2980770)
    at render (https://xpui.app.spotify.com/vendor~xpui.js:29:2983881)
    at https://xpui.app.spotify.com/xpui.js:1:1035286
(anonymous) @ vendor~xpui.js:1
vendor~xpui.js:1 cosmos-connector:  fetchAdForSlot endpoint=%o sp://ads/v1/ads/hpto
xpui.js:1 Uncaught (in promise) Error: GET request to sp://ads/v1/settings/leaderboard failed with status code 404
    at s.onSuccess (xpui.js:1)
    at onSuccess (xpui.js:1)
pubads_impl_2021091504.js:6 [GPT] Invalid argument: PubAdsService.setRequestNonPersonalizedAds("0"). Valid values: 0,1.
(anonymous) @ pubads_impl_2021091504.js:6
pubads_impl_2021091504.js:6 [GPT] Invalid argument: PubadsService.setTagForUnderAgeOfConsent("0"). Valid values: 2,0,1.
(anonymous) @ pubads_impl_2021091504.js:6
DevTools failed to load source map: Could not load content for https://xpui.app.spotify.com/service-worker.js.map: Connection error: net::ERR_NAME_NOT_RESOLVED
DevTools failed to load source map: Could not load content for https://xpui.app.spotify.com/vendor~xpui.js.map: Connection error: net::ERR_NAME_NOT_RESOLVED
DevTools failed to load source map: Could not load content for https://xpui.app.spotify.com/xpui.js.map: Connection error: net::ERR_NAME_NOT_RESOLVED
DevTools failed to load source map: Could not load content for https://xpui.app.spotify.com/xpui-desktop-routes-homepage-takeover-ad.js.map: Connection error: net::ERR_NAME_NOT_RESOLVED
DevTools failed to load source map: Could not load content for https://xpui.app.spotify.com/xpui-routes-home.js.map: Connection error: net::ERR_NAME_NOT_RESOLVED
DevTools failed to load source map: Could not load content for https://xpui.app.spotify.com/xpui-desktop-modals.js.map: Connection error: net::ERR_NAME_NOT_RESOLVED
DevTools failed to load source map: Could not load content for https://xpui.app.spotify.com/998.js.map: Connection error: net::ERR_NAME_NOT_RESOLVED

Specifics (please complete the following information):

OS: Windows 10 Home
Spotify version: <IP_ADDRESS>6.gbb5ef64e
Spicetify version: 2.6.3
Theme name: Ziro Dark Purple

@schnensch0

Closing this as it was fixed in #614.


Bump version in README.md

Hey, thanks for making this 🏅!
Noticed the version is behind the latest release.

Thanks for this!


Added protocol property for docker push

What changes are being made and why?

Added property which specifies what internet protocol should be used for docker push. By ticket: #32

@loicmathieu could you review?


[Feature Request] [Navigator] Expose current route(s)

Use case
I have chat in my application with notifications.
When the app receives notifications from a specific user while it's open and that conversation is already, the notification should not be displayed.
To know when that happens, I need to get the current route and see which one is it.
Current Solutions Status
I know ModalRoute.of(context) can get you some information on the route, but it will only return the route that is part of the context.
Current Workaround
Currently the only workaround I found is by creating a minimal mimic of the navigator and maintaining (another) history on my own.
That is, when I want to push a new roue I'm adding it to a list, and when it's out I'm removing it.
While it works, that info is already kept on the Navigator - one just can't access it.
Proposal
Add a property to NavigatorState:

List<Route> get history: Return a list of all routes that are "alive"
Route get currentRoute: Return the current route (one that will return true for Route.isCurrent).

Hi @matanshukry
this code sample is not specific for the situation you are describing,
but should represent how to access some route information
with the tools already available;
does this solve the issue for you?
Thank you

@iapicca Not really:
Using the RouteAware will remove the need to use my own push/pop, but I will still need to keep the history information since there's no way to get the current one.
Note:
I did notice there's a _route property, but the context there is "wrong". That is, in my use case the the widget (and hence the context)where the code will need to check for the latest route won't be on a widget that is part of the top route, so ModalRoute.of() won't work there
p.s.
I'm not using initialRoutes, routes and pushNamed, but rather using push while giving the name of the route in the settings. That seems to make ModalRoute.of return null

Hi again
it is fairly easy to get the current route in case of error

code sample
import 'package:flutter/material.dart';

class Routes{
  static const String
  HOME_ROUTE = '/',
  SECOND_ROUTE = '/2';
}

typedef void RouteCallback(String routeDebugString);

class RouteAwareWidget extends StatefulWidget {
  final RouteObserver<PageRoute> observer;
  final Widget child;
  final RouteCallback didPopCallback;
  final RouteCallback didPopNextCallback;
  final RouteCallback didPushCallback;
  final RouteCallback didPushNextCallback;

  const RouteAwareWidget({
    @required
    this.child,
    @required
    this.observer,
    this.didPopCallback,
    this.didPopNextCallback,
    this.didPushCallback,
    this.didPushNextCallback,
  });

  @override
  _RouteAwareWidgetState createState() => _RouteAwareWidgetState();
}

class _RouteAwareWidgetState extends State<RouteAwareWidget> with RouteAware {

  RouteCallback get _didPop => widget.didPopCallback ?? (String route) => print('Pop ' + route);
  RouteCallback get _didPopNext => widget.didPopCallback ?? (String route) => print('PopNext ' + route);
  RouteCallback get _didPush => widget.didPopCallback ?? (String route) => print('Push ' + route);
  RouteCallback get _didPushNext => widget.didPopCallback ?? (String route) => print('PushNext ' + route);

  @override
  void didChangeDependencies() {
    super.didChangeDependencies();
    widget.observer.subscribe(this, ModalRoute.of(context));
  }

  @override
  void dispose() {
    widget.observer.unsubscribe(this);
    super.dispose();
  }

  String get _route => ModalRoute.of(context)
      .debugLabel
      .replaceAll( RegExp('[MaterialPageRoute<dynamic>,(,),]'),'');

  void didPop() => _didPop(_route);
  void didPopNext() => _didPopNext(_route);
  void didPush() => _didPush(_route);
  void didPushNext() => _didPushNext(_route);

  @override
  Widget build(BuildContext context) => widget.child;
}

void main() => runApp(MyApp());

class MyApp extends StatelessWidget {

  final  RouteObserver<PageRoute> _observer = RouteObserver<PageRoute>();

  @override
  Widget build(BuildContext context)=> MaterialApp(
    navigatorObservers: <RouteObserver> [_observer,],
    initialRoute: Routes.HOME_ROUTE,
    routes: {
      Routes.HOME_ROUTE: (BuildContext context) => RouteAwareWidget(
        observer: _observer,
        child: HomePage(),
      ),
      Routes.SECOND_ROUTE: (BuildContext context) =>  RouteAwareWidget(
        observer: _observer,
        child: SecondPage(),
      ),
    },
  );
}

class HomePage extends StatelessWidget {

  @override
  Widget build(BuildContext context) => Scaffold(
    body: Center(
      child: Text(
         ModalRoute.of(context).settings.name,
        textScaleFactor: 5,
      ),
    ),
    floatingActionButton: FloatingActionButton(
      child: Icon(Icons.arrow_forward),
      onPressed: () => Navigator.pushNamed(context, Routes.SECOND_ROUTE),
    ),
  );
}

class SecondPage extends StatefulWidget {

  @override
  _SecondPageState createState() => _SecondPageState();
}

class _SecondPageState extends State<SecondPage> {

  Future<void> _delayedError() async {
    await Future.delayed(const Duration(seconds: 2));
    throw Error();
  }

  void _log({void Function() onError}) async  {
    try {
      await _delayedError();
    } catch (Error) {
      onError();
    }
  }

  @override
  void initState() {
    super.initState();
    _log(
      onError: () => print('error from: ' + ModalRoute.of(context).settings.name)
    );
  }

  @override
  Widget build(BuildContext context) => Scaffold(
    appBar: AppBar(),
    body: Center(
      child: Text(
        ModalRoute.of(context).settings.name,
        textScaleFactor: 5,
      ),
    ),
  );
}

without using named routes I'm not sure what information
a "NavigatorState" should return for currentRoute,
could you please provide some detail
Thank you

@iapicca It looks like the same code on the dartpad you posted; why did you write it? what's different?
Anyway I made a sample in DartPad but can't really understand how to create my own link in there; here's the code though:
import 'package:flutter/material.dart';

class Routes {
  static const String HOME_ROUTE = '/';
  static const String SECOND_ROUTE = '/2';
}

typedef void RouteCallback(String routeDebugString);

class RouteAwareWidget extends StatefulWidget {
  final RouteObserver<PageRoute> observer;
  final Widget child;
  final RouteCallback didPopCallback;
  final RouteCallback didPopNextCallback;
  final RouteCallback didPushCallback;
  final RouteCallback didPushNextCallback;

  const RouteAwareWidget({
    @required this.child,
    @required this.observer,
    this.didPopCallback,
    this.didPopNextCallback,
    this.didPushCallback,
    this.didPushNextCallback,
  });

  @override
  _RouteAwareWidgetState createState() => _RouteAwareWidgetState();
}

class _RouteAwareWidgetState extends State<RouteAwareWidget> with RouteAware {
  RouteCallback get _didPop =>
      widget.didPopCallback ?? (String route) => print('Pop ' + route);
  RouteCallback get _didPopNext =>
      widget.didPopCallback ?? (String route) => print('PopNext ' + route);
  RouteCallback get _didPush =>
      widget.didPopCallback ?? (String route) => print('Push ' + route);
  RouteCallback get _didPushNext =>
      widget.didPopCallback ?? (String route) => print('PushNext ' + route);

  @override
  void didChangeDependencies() {
    super.didChangeDependencies();
    widget.observer.subscribe(this, ModalRoute.of(context));
  }

  @override
  void dispose() {
    widget.observer.unsubscribe(this);
    super.dispose();
  }

  String get _route => ModalRoute.of(context)
      .debugLabel
      .replaceAll(RegExp('[MaterialPageRoute<dynamic>,(,),]'), '');

  void didPop() => _didPop(_route);
  void didPopNext() => _didPopNext(_route);
  void didPush() => _didPush(_route);
  void didPushNext() => _didPushNext(_route);

  @override
  Widget build(BuildContext context) => widget.child;
}

void main() => runApp(MyApp());

class MyApp extends StatelessWidget {
  final RouteObserver<PageRoute> _observer = RouteObserver<PageRoute>();

  @override
  Widget build(BuildContext context) => MaterialApp(
        navigatorObservers: <RouteObserver>[
          _observer,
        ],
        home: HomePage(),
      );
}

class HomePage extends StatelessWidget {
  @override
  Widget build(BuildContext context) => Scaffold(
        body: Center(
          child: Text(
            ModalRoute.of(context).settings.name,
            textScaleFactor: 5,
          ),
        ),
        floatingActionButton: FloatingActionButton(
          child: Icon(Icons.arrow_forward),
          onPressed: () => Navigator.push(
              context,
              MaterialPageRoute(
                  settings: RouteSettings(name: Routes.SECOND_ROUTE),
                  builder: (builderContext) => SecondPage(onPressed: () => _onPress(context)))),
        ),
      );

  void _onPress(BuildContext context) {
    final route = ModalRoute.of(context);
    print('Click!: $route');
  }
}

class SecondPage extends StatelessWidget {
  const SecondPage({Key key, this.onPressed}) : super(key: key);

  final VoidCallback onPressed;

  @override
  Widget build(BuildContext context) => Scaffold(
        appBar: AppBar(),
        body: Center(
          child: RaisedButton(
            onPressed: onPressed,
            child: Text(
              ModalRoute.of(context).settings.name,
              textScaleFactor: 5,
            ),
          ),
        ),
      );
}

It's somewhat close to the usecase I'm saying:

The _onPress method has a context which isn't the context of the top route (SecondPage), so when calling ModalRoute.of(context) the returned route is not the one at the top
Like I said, I'm using push with name in route settings. The RouteAware doesn't seem to work at all (even if it did though, not sure how it would help)

Hi @matanshukry
the second sample I provided shows how can "collect" the route where an error occurs
(see changes in the second page) in your sample you are passing in the callback the "wrong" context which is relative to the previous page, not the one where the callback is called
MaterialPageRoute(
                  settings: RouteSettings(name: Routes.SECOND_ROUTE),
                  builder: (builderContext) => SecondPage(onPressed: () => _onPress(context)))),
        ),
      );

  void _onPress(BuildContext context) {
    final route = ModalRoute.of(context); /// <= [this is still the homepage context! even when is passed to the next page]
    print('Click!: $route');
  }

anyhow could you please provide the details I mentioned above

without using named routes I'm not sure what information
a "NavigatorState" should return for currentRoute,
could you please provide some detail

Thank you

@iapicca

It's not passing the "wrong" context - that's just the thing, it's the context I have. This example is a bit more simple so you have access to the right context, but in my use case you don't, you only have access to the "wrong" context, which is why you can't use ModalRoute.

If it matters, the reason why I only have access to the "wrong" context is because it's coming from an "outside" callback. Specifically from a notification (using FCM). So I always have access to the same context which is high the hierarchy, while somewhere in the bottom there's the "right" context which I don't have access to.

Regarding named routes: You can see i am using named routes, by giving the name in the Settings on push method. If you use my example and switch between the contexts btw, you can see one of them print the right info while the other isn't, so the info is there.


Fix bad-string-format-character when a placeholder occurs within a format spec part

Closes #6767
The presented false positive "{0:.{prec}g}".format(1.23, prec=15) occurs because we treated the entire format specification of .{prec}g as the format type (which is typically at the end of a format specification). The nested replacement was not detected because a leading { was not seen. Instead, . remained the leading character — it's the leading character for a precision clause but was not consumed since the replacement was not a valid number
One way to resolve this is to update parsing of every clause in a format specification to be replacement aware e.g. precision parsing would accept a replacement as well as a number. I determined that this is too complicated and provides little value for our current rules around format specifications.
Instead, after we've parsed most of the format specification parts, we will check for any remaining replacements (i.e. the presence of { anywhere in the remaining string) and consume them and any intermediate characters. This matches our existing intention of basically ignoring any replacements while retaining as much of the normal format specification linting as we can.

@charliermarsh are you saying that the format string gets evaluated at runtime? Like, a can contain a partial format spec and the rest is a static string?

@MichaReiser - Yes that's correct.

(I am the original reporter of this bug in: https://github.com/astral-sh/ruff/issues/6442#issuecomment-1686440347.)

Similarly, for FormatSpec::parse("{a}s"), this could either resolve to !s or :s depending on the value of a, and so treating s as either the format type or the conversion would be wrong.

Maybe I am missing something here, but it seems the colon or exclamation mark must be included statically in the format string, as otherwise it produces a ValueError:
>>> "{{a}s}".format("test", a=":")
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
ValueError: Single '}' encountered in format string
>>> "{{a}s}".format("test", a="!")
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
ValueError: Single '}' encountered in format string

Adding the implicit zero does not make it work either:
>>> "{0{a}s}".format("test", a=":")
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
ValueError: unexpected '{' in field name
>>> "{0{a}s}".format("test", a="!")
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
ValueError: unexpected '{' in field name

@tdulcet - Sorry, you're right, it seems that it doesn't behave that way in that case. The first example does hold though (of width vs. precision), in my retesting.


Add code

as we can se in this PR, that's when bugs appear, he added code

CIAL

rey entra idhi

CODE IS A LIABILITY
CIAL CIAL CIAL CIAL CIAL CIAL CIAL CIAL CIAL CIAL CIAL CIAL CIAL CIAL CIAL CIAL CIAL CIAL CIAL CIAL CIAL CIAL CIAL CIAL CIAL CIAL CIAL CIAL CIAL CIAL CIAL CIAL CIAL CIAL CIAL CIAL CIAL CIAL CIAL CIAL CIAL CIAL CIAL CIAL CIAL CIAL CIAL CIAL CIAL CIAL CIAL CIAL CIAL CIAL CIAL CIAL CIAL CIAL CIAL CIAL CIAL CIAL CIAL CIAL CIAL CIAL CIAL CIAL CIAL CIAL CIAL CIAL CIAL CIAL CIAL CIAL CIAL CIAL CIAL CIAL CIAL CIAL CIAL CIAL CIAL CIAL CIAL CIAL CIAL CIAL CIAL CIAL CIAL CIAL CIAL CIAL CIAL CIAL CIAL CIAL CIAL CIAL CIAL CIAL CIAL CIAL CIAL CIAL CIAL CIAL CIAL CIAL CIAL CIAL CIAL CIAL CIAL CIAL CIAL CIAL CIAL CIAL CIAL CIAL CIAL CIAL CIAL.


Export clips separately

Ability to export a clip type separately. Either each clip of that type would have its own video file, or it would be a Tri-State, with the third option being all clips of that type in the same file.
That was a long sentence.

TODO:

[ ] Add config
[ ] Add field for clip
[ ] Add serialization/deserialization
[ ] Add to algorithm

Closed in f326eca927ba22563e442bf78221c9d5a9c1fc81


confusing warning message

When the operator starts up in dev mode or when performing unit tests, it reports:
15:03:03.651 WARN  [io.quarkiverse.operatorsdk.deployment.OperatorSDKProcessor] (build-13) Use of deprecated check-crd-and-validate-local-model property. Use crd.validate instead.
15:03:03.653 WARN  [io.quarkiverse.operatorsdk.deployment.OperatorSDKProcessor] (build-13) Deprecated property check-crd-and-validate-local-model with value 'true' is overridden by crd.validate property value 'false'

but in my project check-crd-and-validate-local-model is not set, but quarkus.operator-sdk.crd.validate is set to false

This issue has already been fixed. See #70.


Move MSRV to cargo.toml and remove build.rs

This reworks MSRV detection logic, using the rust-version field in Cargo manifest. It allows dropping build.rs and removing the version_check build-dependency.
Ref: https://doc.rust-lang.org/cargo/reference/manifest.html#the-rust-version-field

/cc @ejmahler for review.

The solution with build.rs was created to allow different msrv for different features and architectures. This isn't needed right now, but will be again once this get completed for example: https://github.com/ejmahler/RustFFT/pull/144


robots.txt

We should investigate SEO with robots.txt

@ozdanborne should this still be open? Is there a concrete action here?


check null for names

Summary
Null check to avoid throwing NPE.
Issue Type

Bug fixing

Starter Names

azure cloud foundry service sample

Additional Information

refs #231


KAFKA-1545: KafkaHealthcheck.register failure

KAFKA-1545: java.net.InetAddress.getLocalHost in KafkaHealthcheck.register may fail on some irregular hostnames

some issue with this git branch; closing this pull and recreating.thanks

some issue on this pull/git branch.closing and recreating.thanks


Improve regex to remove svg size parameters

Wasn't accepting some svg links due to variance in how width/height were specified.


reproduce result of office-home

Thanks to the author for contributing the code!
I’m trying to train a model on office-home, but there is only a script for reproducing the visda results.
So I have difficulty to get the office-home results.
Could you provide a script to reproduce the results for office-home?

I add a script for office-home dataset in my main page.
If you had any additional question, please do not hesitate.
Thanks.

Oh, I apologize for the late reply.
I discovered your request yesterday.
Please refer to the attached file and try to reproduce it.
Thanks.
oda_codes.zip


hotrfc not accessible from "Groups" menu

Describe the issue
https://datatracker.ietf.org/group/hotrfc/about/ exists, but I can't figure out how to get to it from within the menu structure.
Code of Conduct

[X] I agree to follow the IETF's Code of Conduct

Because HotRFC (and a few others) were created quickly as "doesn't fit anywhere" groups.
>>> from ietf.group.models import Group
>>> g = Group.objects.get(acronym='hotrfc')
>>> g.type
<GroupTypeName: Ad Hoc>
>>> Group.objects.filter(type='adhoc')
<QuerySet [<Group: IEPG - Internet Engineering and Planning Group>, <Group: Hot RFC Lightning Talks>, <Group: Technology Deep Dive>]>

Do you want an "Active Ad Hoc" item added to Other under Groups, or would you prefer to rethink how these groups are modeled in the first place?

I think some quick fix would be good now - we can rethink this later.

Closing - handled with data model changes as discussed in 4380.


Do away with debbuild/debinstall requirement for VM install?

Maybe the VM can run without a system-level install? These Make tasks don't seem to be a maintenance priority for the Urbit team and can probably be worked around. See urbit/urbit#747
cc @revprez

Worth experimenting while we wait for urbit/urbit#747 to merge.

urbit/urbit#747 merged. Close this as it's now mute?


nix-exec: Bump plugins to new nix list handling.

Fixes #43

I have verified that this PR resolves #43


Text input doesn't accept spaces when used inside menu

Steps to reproduce
Use the md-input with type="text" inside md-menu's md-menu-content area
Also affects md-input type="password" and md-text-area
Typing spaces get ignored on these fields
Which browser?
Chrome 56
Reproduction Link
codepen

Can you create a Codepen with the issue?

updated with link
codepen

The bug still exists

Closing this issue as our focus is on the new 1.0.0 version.


Add command to delete latest prescribing import

:+1: reasonable responses on all fronts — merge at will!


agent: support max_tokens, temperature and memory_size options

Summary
Add max_tokens, temperature and memory_size options to agent. I did not add this as predict params because it required user always feed to run agent. I think it should be a property of agent more properly, so added in agent creation api.
Fixes
Depends on
Test plan

Screenshots

Checklist

[x] My code follows the code style of this project and passes make format.
[x] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.
[ ] My change has adequate unit test coverage.

for #72

Thanks for this submission @bdqfork!
I have some comments in the PR and a general comment:
The memorySize, maxTokens and temperature should be passed in the existing llm dictionary which handles the other llm specific settings such as model. This way of implementing also means that we don't need to add any fields to the DB and you changes there should not be necessary.

Oh, it's indeed a better solution. Thank you for reminding me.

Thanks for this submission @bdqfork!

I have some comments in the PR and a general comment:

The memorySize, maxTokens and temperature should be passed in the existing llm dictionary which handles the other llm specific settings such as model. This way of implementing also means that we don't need to add any fields to the DB and you changes there should not be necessary.

Oh, it's indeed a better solution. Thank you for reminding me.

Ping when you are ready!

Already updated! @homanp

Already updated! @homanp

Thank you!

Already updated! @homanp

Please revert to using procees_payload as well!

Is there anything else to update?

Is there anything else to update?

No thank you! Just testing and then deploying!

@bdqfork did you test this on your end?

I did, is something wrong?发自我的 iPhone在 2023年7月18日，20:37，Ismail Pelaseyed @.***> 写道：
@bdqfork did you test this on your end?
—Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this because you were mentioned.Message ID: @.***>

@bdqfork did you test the memory changes and new prompt template setup as well?! I didn’t so just want to make sure it will work.

I did test in my end. However, we do not have a complete regression testing process. If you are also uncertain, I suggest delaying the merge and waiting for the complete testing action to be filled and passed.

Yes ok, I will run a full test and merge after that!

I will close this pull request. There is some irrelevant code logic included in it, and I will remove it and resubmit a new pull request.

I will close this pull request. There is some irrelevant code logic included in it, and I will remove it and resubmit a new pull request.

Appreciate the effort @bdqfork