Python SDK - Certificate error

Describe the issue:
The following error is shown when trying to run the python SDK :
certificate verify failed: unable to get local issuer certificate (_ssl.c:1131)')))
I have tried adding the certificate of localhost manually to the cert directory, but this issue is still reproducible.
How to reproduce:
Prerequisite: Create a service provider on the management console

Download the Sample SDK
Create a virtual Python environment
Install the dependencies
Update the conf.py file
Run the python app

Expected behavior:
The SDK should run without any errors
Environment information

IS 5.12.0 - alpha 14

@Sathiyakugan Can you have a look on this

This issue is being closed due to extended inactivity. Please feel free to reopen it if further attention is needed. Thank you for helping us keep the issue list relevant and focused!


Improve CLI command parsing

[ ] CHANGELOG.md updated
[x] Rebased/mergeable
[x] Tests pass
[x] Sign CLA (if not already signed)

Now, the output:
$ influx -host <IP_ADDRESS>
Visit https://enterprise.influxdata.com to register for updates, InfluxDB server management, and monitoring.
Connected to http://<IP_ADDRESS>:8086 version 0.9.3
InfluxDB shell 0.9
> user
ERR: error parsing query: found USER, expected SELECT, DELETE, SHOW, CREATE, DROP, GRANT, REVOKE, ALTER, SET at line 1, char 1
Warning: It is possible this error is due to not setting a database.
Please set a database with the command "use <database>".
> use
Could not parse database name from "use".

Refs #4588
Fixes #4544

Forced push with little syntax clean-up.

While this fixes the issue it can be further improved.
>     use metrics
Using database metrics
> use  metrics
Could not parse database name from "use  metrics".

While I understand one may argue this tool is not meant to be dumb-proof, I like to make it harder to commit mistakes while using it. I'll probably be looking at fixing the above.

This makes sense.  Can you add some tests that show what was fixed?

Sure.

@corylanou care to review?

One nit, otherwise +1.  Thanks again!

@corylanou nit fixed. Thank you.

This needs rebasing now.

Makes sense, +1.

Let us know when you rebase @pires and I will then merge.

@otoolep ready.

Thanks again @pires


not printable/special character may corrupt excel file

Hi,
I found when some not printable/special characters in string, the generated excel file cannot be opened.
This is a case I met:
`
test.xlsx
select '' f1, ascii('') f2, length('') f3, lengthb('') f4 from dual
F1                              	F2	F3	F4

                               	26	1 	1
`
The generated excel of above sql attached for your reference.
Regards
Wicky

Hi Wicky,
Thanks for reporting this.
Due to the XML (1.0) nature of the xlsx format under the hood, control characters [0x00-0x1F] except 0x09, 0x0A and 0x0D are forbidden unless they are escaped in the form _xHHHH_ (ref. ECMA-376).
I have to admit I overlooked that part of the specs.
That being said, would you like to :

keep those characters in the resulting file, with some performance overhead due to the escape process (*) ?
remove those characters while generating the file?

(*) could be mitigated if disabled by default, or enabled on a per-column basis

Hi,
Thank you for the prompt response.
I don't want the invisible character in file,
I think option 2 should be better for most of cases.
Regards
Wicky

Fixed in v2.2.1
By default, control characters are now removed from string values.
I'll add later a few options to control this more finely.


Where are vmap() and vsum() defined?

./docs/papers/wpmvp14/experiments/euler06b.py:4:7: F821 undefined name 'vsum'
  a = vsum(r)
      ^
./docs/papers/wpmvp14/experiments/euler06b.py:5:7: F821 undefined name 'vmap'
  m = vmap(lambda i:i*i, r)
      ^
./docs/papers/wpmvp14/experiments/euler06b.py:6:18: F821 undefined name 'vsum'
  return a * a - vsum(m)
                 ^
./docs/papers/wpmvp14/experiments/map0.py:3:9: F821 undefined name 'vmap'
    l = vmap(lambda x: 5*x + 3, r)
        ^
./docs/papers/wpmvp14/experiments/map1.py:3:9: F821 undefined name 'vmap'
    l = vmap(abs, r)
        ^
./docs/papers/wpmvp14/experiments/sum0.py:3:12: F821 undefined name 'vsum'
    return vsum(r)
           ^
./docs/papers/wpmvp14/experiments/sum1.py:3:12: F821 undefined name 'vsum'
    return vsum(r*r)
           ^

It's because it should be piped through wpmvp14/experiments/run_xp_euler06b.sh and such. But don't bother too uch with that :-)


Upgrade Play! version from 2.6.2 to 2.6.7

Upgrade:

play version from 2.6.2 to 2.6.7
twirl version from 1.3.3 to 1.3.12
play-enhancer version from 1.1.0 to 1.2.2
ebean-agent version from 10.3.1 to 10.4.1 (used in play-ebean version 4.0.6)

Upgrade in documentation and test projects:

akka-* dependencies versions to 2.5.6
play-ebean version to 4.0.6
scalatestplus-play version from 3.1.0 to 3.1.2

1.0.0-rc2-SNAPSHOT snapshot deployed

As described in #153, run goal doesn't work now and must be fixed.


[Markets] Something strange happening with /market/info

Details:
Look at total supply

Steps to reproduce:

Run blockatlas with api + sync markets
Go to POSTMAN
QUERY: http://localhost:8420/v1/market/info?coin=60&time_start=1574483028&currency=USD
Look at response body

@EnoRage, probably the problem is here
https://github.com/trustwallet/blockatlas/tree/master/syncmarkets/charts.go#L53
If do you try to request a lot of times from the same provider, starts to return an error and the app gets the value from the next provider


Dev additional plugins

just run cmake with
$ cmake .. -DEOSIO_ADDITIONAL_PLUGINS="<path_to_source_plugin_0>[;<path_to_source_plugin_1] ..."
First additional plugin ever: https://github.com/asiniscalchi/eosio_sql_plugin
following the line you have to add in the root  CMakeList.txt of an additional plugin
eosio_additional_plugin(<plugin_target0>[;<plugin_target1>] ...)
Cheers

@wanderingbort is this stable now and should it be used by the community?

@gleehokie better after https://github.com/EOSIO/eos/issues/5229  is resolved


fix issue-8741: consumer and provider have same tag, but no provider …

…found
What is the purpose of the change
fix #8741
tagDynamicRouter run first and add "noTag" into invocation,
then tagStaticRouter check invocation tag not emtpy, so return wrong invoker.
I just change tagStaticRouter, check tag is not empty and not equals "noTag".
But tagDynamicRouter add "noTag" may be the root reason.
Brief changelog
Verifying this change

Checklist

[x] Make sure there is a GitHub_issue field for the change (usually before you start working on it). Trivial changes like typos do not require a GitHub issue. Your pull request should address just this issue, without pulling in other changes - one PR resolves one issue.
[ ] Each commit in the pull request should have a meaningful subject line and body.
[ ] Write a pull request description that is detailed enough to understand what the pull request does, how, and why.
[ ] Check if is necessary to patch to Dubbo 3 if you are work on Dubbo 2.7
[ ] Write necessary unit-test to verify your logic correction, more mock a little better when cross module dependency exist. If the new feature or significant change is committed, please remember to add sample in dubbo samples project.
[ ] Add some description to dubbo-website project if you are requesting to add a feature.
[ ] GitHub Actions works fine on your own branch.
[ ] If this contribution is large, please follow the Software Donation Guide.

为什么这个校验为什么会不通过，看起来代码是好的。

Why did it fail? It looks good.

Codecov Report

Merging #8762 (8abf4ad) into 3.0 (13c15fd) will decrease coverage by 0.18%.
The diff coverage is 0.00%.

@@             Coverage Diff              @@
##                3.0    #8762      +/-   ##
============================================
- Coverage     63.71%   63.52%   -0.19%
+ Complexity      313      311       -2
============================================
  Files          1145     1145
  Lines         48183    48189       +6
  Branches       7255     7233      -22
============================================
- Hits          30698    30614      -84
- Misses        14115    14190      +75
- Partials       3370     3385      +15

Impacted Files
Coverage Δ

...o/rpc/cluster/router/tag/TagStaticStateRouter.java
59.52% <0.00%> (-1.46%)
:arrow_down:

...ng/transport/dispatcher/all/AllChannelHandler.java
62.06% <0.00%> (-27.59%)
:arrow_down:

...dubbo/common/status/support/LoadStatusChecker.java
46.15% <0.00%> (-15.39%)
:arrow_down:

...ng/transport/dispatcher/WrappedChannelHandler.java
43.47% <0.00%> (-15.22%)
:arrow_down:

...ache/dubbo/remoting/transport/AbstractChannel.java
75.00% <0.00%> (-12.50%)
:arrow_down:

.../remoting/transport/netty4/NettyServerHandler.java
62.79% <0.00%> (-9.31%)
:arrow_down:

.../remoting/transport/netty4/NettyClientHandler.java
77.58% <0.00%> (-6.90%)
:arrow_down:

.../org/apache/dubbo/remoting/ExecutionException.java
15.78% <0.00%> (-5.27%)
:arrow_down:

...nt/metadata/StandardMetadataServiceURLBuilder.java
86.04% <0.00%> (-4.66%)
:arrow_down:

...pache/dubbo/remoting/transport/AbstractServer.java
57.14% <0.00%> (-4.29%)
:arrow_down:

... and 30 more

Continue to review full report at Codecov.

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

I don't understand the checking mechanism.

sometime ut may fail, just retry it.
I have no write auth, so I commit it again.

I am a new comer，auth is to point to?

I don't understand the checking mechanism.

sometime ut may fail, just retry it.
I have no write auth, so I commit it again.

Since my pr also failed to validate,But I don't know what's causing it yet.

I am a new comer，auth is to point to?

I don't understand the checking mechanism.

sometime ut may fail, just retry it.
I have no write auth, so I commit it again.

just submit it again!
ps: manager can trigger ut.


[QUESTION] have error while extend pino-logger

logger.service.ts

import { Inject, Injectable } from '@nestjs/common'
import { Logger, Params, PARAMS_PROVIDER_TOKEN, PinoLogger } from 'nestjs-pino'
@Injectable()
export class LoggerService extends Logger {
    private aProperty: string
    constructor(
        pinoLogger: PinoLogger,
        @Inject(PARAMS_PROVIDER_TOKEN) params: Params,
        aProperty: string
    ) {
        super(pinoLogger, params)
        this.aProperty = aProperty
    }

    aMethod(): void {
        return
    }
}

logger.module.ts

import { Module } from "@nestjs/common"
import { LoggerModule } from "nestjs-pino"
import { LoggerService } from "./logger.service"

@Module({
  exports: [LoggerService],
  providers: [LoggerService],
  imports: [
    LoggerModule.forRoot({})
  ],
})
export class MyLoggerModule {}

import the MyLoggerModule in app.module.ts

@Module({
  imports: [
    MyLoggerModule
  ],

while start app, here is the error message

[Nest] 8420  - 11/30/2021, 12:29:20 PM     LOG [NestFactory] Starting Nest application...
[Nest] 8420  - 11/30/2021, 12:29:20 PM   ERROR [ExceptionHandler] Nest can't resolve dependencies of the LoggerService (PinoLogger, pino-params, ?). Please make sure that the argument String at index [2] is available in the MyLoggerModule context.

Potential solutions:
- If String is a provider, is it part of the current MyLoggerModule?
- If String is exported from a separate @Module, is that module imported within MyLoggerModule?
  @Module({
    imports: [ /* the Module containing String */ ]
  })

Error: Nest can't resolve dependencies of the LoggerService (PinoLogger, pino-params, ?). Please make sure that the argument String at index [2] is available in the MyLoggerModule context.

Potential solutions:
- If String is a provider, is it part of the current MyLoggerModule?
- If String is exported from a separate @Module, is that module imported within MyLoggerModule?
  @Module({
    imports: [ /* the Module containing String */ ]
  })

    at Injector.lookupComponentInParentModules (/Users/delai/nest/nest-node/node_modules/@nestjs/core/injector/injector.js:201:19)
    at Injector.resolveComponentInstance (/Users/delai/nest/nest-node/node_modules/@nestjs/core/injector/injector.js:156:33)
    at resolveParam (/Users/delai/nest/nest-node/node_modules/@nestjs/core/injector/injector.js:108:38)
    at async Promise.all (index 2)
    at Injector.resolveConstructorParams (/Users/delai/nest/nest-node/node_modules/@nestjs/core/injector/injector.js:123:27)
    at Injector.loadInstance (/Users/delai/nest/nest-node/node_modules/@nestjs/core/injector/injector.js:52:9)
    at Injector.loadProvider (/Users/delai/nest/nest-node/node_modules/@nestjs/core/injector/injector.js:74:9)
    at async Promise.all (index 3)
    at InstanceLoader.createInstancesOfProviders (/Users/delai/nest/nest-node/node_modules/@nestjs/core/injector/instance-loader.js:44:9)
    at /Users/delai/nest/nest-node/node_modules/@nestjs/core/injector/instance-loader.js:29:13

waiting for any suggestion from you

Not enough data. how are you injecting that extended logger? You can see working examples here
But my suggestion is don't extend Logger, because this class is made for substitution, the extension support of Logger will be dropped in the next major version. So at least you can extend PinoLogger.


BakkesMod.BakkesMod version 31

[x] Have you signed the Contributor License Agreement?
[x] Have you checked that there aren't other open pull requests for the same manifest update/change?

[x] Does your manifest conform to the 1.4 schema?

Note: <path> is the name of the directory containing the manifest you're submitting.

Microsoft Reviewers: Open in CodeFlow

@microsoft-github-policy-service agree

I have to note that I've heard but I'm not sure that only applications supporting silent installations would be supported. In that case, BakkesMod would be a more complicated case.
What I've submitted here is the Setup.exe which installs the BakkesMod Injector and the current underlying BakkesMod version for you.
However, if only programs supporting silent installs are supported, we could instead make it so that the .exe from here is downloaded. That just downloads the injector and then lets you install the underlying version from inside the injector via an update.

However, I think the second option isn't that good since the .exe that you later need to open the program is what you download and it doesn't get put into ProgramData or anything similar.

Uhm that's not normal, is it?

Opened a case here: https://www.microsoft.com/en-us/wdsi/submission/7d4fdf5c-a7d6-4115-99e3-b5faee7b108c

Unfortunately, MS wasn't able to reproduce any detection: https://www.microsoft.com/en-us/wdsi/submission/7d4fdf5c-a7d6-4115-99e3-b5faee7b108c What am I supposed to do now?

Hi @CanePlayz,
The installer asks if it's okay to close open applications. Is there an installer switch to bypass this check?

@wingetbot run

Hi @CanePlayz,
Please ignore the Internal-Error-Dynamic-Scan labels, as these are a pipeline error we're working to identify.
The installation isn't unattended:

Is there an installer switch to bypass this and have it install automatically?

Unfortunately, I haven't received any information from the devs yet. I will let you know once I've found out something.

The team behind BakkesMod has decided they do not want their software to be featured on winget, and I will respect that of course. I will leave this PR open so you can keep on testing the pipelines if you want to, feel free to close the PR anytime though.

/azp run

This is an Inno installer and the switch /? gives its installer switches:

But even with /VERYSILENT /SUPPRESSMSGBOXES /FORCECLOSEAPPLICATIONS it still pops up with the box. So it doesn't appear to have an unattended mode. Blocked by:

https://github.com/microsoft/winget-cli/issues/910

This is an Inno installer and the switch /? gives its installer switches:
But even with /VERYSILENT /SUPPRESSMSGBOXES /FORCECLOSEAPPLICATIONS it still pops up with the box. So it doesn't appear to have an unattended mode. Blocked by:

Settings for installer modes winget-cli#910

Thanks for your investigations, but as I mentioned above, unfortunately, the devs don't want their software on winget...

The team behind BakkesMod has decided they do not want their software to be featured on winget, and I will respect that of course. I will leave this PR open so you can keep on testing the pipelines if you want to, feel free to close the PR any time though.

Thanks for letting us know.


Properly handle skipped tests for programming exercises

Bamboo can identify and report skipped tests. Artemis does not handle those correctly at the moment and assumes that those tests have passed.
Artemis should handle skipped tests to make sure the calculation of the score is correct!
Error report from one of the instructors:
I have a bug regarding the calculation of the score in Artemis found. If the Compile Test failed, all further tests will be skipped. This is also displayed in Bamboo/Bitbucket. On Artemis, however, skipped tests are classified as successful. Here the students passed 0 of 12 tests but apparently 91% of all points.

@thilo-behnke @ungaralex The bamboo-plugin supports skipped test cases (at least in theory, we might have to check it), so I think this is an error in Artemis then, can you confirm this?

Yes, this is definitely a problem on the Artemis client side. I am currently investigating it.


As a user I want to see Footer

States and styles of all controls look like Figma
Localization on the Ukrainian language is due to mockups from Figma
The Footer is at the bottom of the page
On the Footer we have:
Catalogs: Bagpacks, Bags, Accessories.
Information: About us, Materials, Payment & Shipping, Privacy policy.
Contacts: +38 012 345 678<EMAIL_ADDRESS>Lviv, Ugorska str. 2; Show on map link.
Social networks: Instagram, Facebook.
Everything on the Footer is located according to the mockup (see attachment Footer 1).

Acceptance criteria:
When user clicks any item of the Catalogs list (Bagpacks, Bags, Accessories) user redirected to the page of the selected catalogs.
When user clicks any category from the Information ( About us, Materials, Payment & Shipping, Privacy policy) user is redirected to the selected page.
When user clicks Show on Map link, the user is redirected to the Show on Map page.
When user clicks any item of Social networks, user redirected to the HORONDI page of the selected social network.
When user hovers over the Social network icon (Instagram, Facebook) icon link displayed on a white background, the color of the icons changes to black ( see attachment Footer 2).

Default states of all controls look like mockup (see attachment Footer 1)
#29

New update 08/10/20: #219


[BUILD] Setup CircleCI

@xcambar or @GreatWizard can one of you enable circleci for this project ?


GHC 8.0 release candidate compatibility

Trying to compile with latest GHC 8 release candidate leads to the following error:
HFlags.hs:188:17: error:
    • Couldn't match expected type ‘Q Dec’
                  with actual type ‘CxtQ -> DecQ’
    • Probable cause: ‘dataD’ is applied to too few arguments
      In a stmt of a 'do' block:
        dataDec <- dataD
                     (cxt []) dataName [] [normalC dataConstrName []] []
      In the expression:
        do { (name, short) <- if
                              | length name' == 0
                              -> fail "Flag's without names are not supported."
                              | length name' == 1 -> return (name', Just $ head name')
                              | length name' == 2 -> return (name', Nothing)
                              | name' !! 1 == ':' -> return (drop 2 name', Just $ head name')
                              | otherwise -> return (name', Nothing);
             defE <- defQ;
             flagType <- case defE of {
                           SigE _ flagType -> return $ return flagType
                           _ -> fail
                                  "Default value for defineCustomFlag has to be an explicitly typed expression, like (12 :: Int)" };
             moduleName <- fmap loc_module location;
             .... }
      In an equation for ‘defineCustomFlag’:
          defineCustomFlag name' defQ argHelp readQ showQ description
            = do { (name, short) <- if
                                    | length name' == 0
                                    -> fail "Flag's without names are not supported."
                                    | length name' == 1 -> return ...
                                    | length name' == 2 -> return ...
                                    | name' !! 1 == ':' -> return ...
                                    | otherwise -> return ...;
                   defE <- defQ;
                   flagType <- case defE of {
                                 SigE _ flagType -> return $ return flagType
                                 _ -> fail
                                        "Default value for defineCustomFlag has to be an explicitly typed expression, like (12 :: Int)" };
                   .... }

HFlags.hs:188:44: error:
    • Couldn't match expected type ‘Maybe Kind’
                  with actual type ‘[ConQ]’
    • In the fourth argument of ‘dataD’, namely
        ‘[normalC dataConstrName []]’
      In a stmt of a 'do' block:
        dataDec <- dataD
                     (cxt []) dataName [] [normalC dataConstrName []] []
      In the expression:
        do { (name, short) <- if
                              | length name' == 0
                              -> fail "Flag's without names are not supported."
                              | length name' == 1 -> return (name', Just $ head name')
                              | length name' == 2 -> return (name', Nothing)
                              | name' !! 1 == ':' -> return (drop 2 name', Just $ head name')
                              | otherwise -> return (name', Nothing);
             defE <- defQ;
             flagType <- case defE of {
                           SigE _ flagType -> return $ return flagType
                           _ -> fail
                                  "Default value for defineCustomFlag has to be an explicitly typed expression, like (12 :: Int)" };
             moduleName <- fmap loc_module location;
             .... }

Looks like Template Haskell type of dataD got a new argument CxtQ.
Would it be possible to upgrade this one?

This small patch seems to resolve the issue:
diff --git a/HFlags.hs b/HFlags.hs
index 5644a93..cdb9860 100644
--- a/HFlags.hs
+++ b/HFlags.hs
@@ -185,7 +185,7 @@ defineCustomFlag name' defQ argHelp readQ showQ description =
      let dataName = mkName $ "HFlag_" ++ name
      let dataConstrName = mkName $ "HFlagC_" ++ name
      -- Note: support for splicing inside [d| |] would make all this a lot nicer
-     dataDec <- dataD (cxt []) dataName [] [normalC dataConstrName []] []
+     dataDec <- dataD (cxt []) dataName [] Nothing [normalC dataConstrName []] (cxt [])
      instanceDec <- instanceD
                     (cxt [])
                     (appT (conT ''Flag) (conT dataName))

@mgajda, do you feel like forking the repo and publishing this change?

@mgajda, just did it -- https://github.com/deepfire/hflags/commit/f4090a1131b7cb3ca0078fee2dc797ed8453f41a
It does build with GHC 8 for me.

You might want to #ifdef change on GHC version
On Saturday, 5 March 2016, Kosyrev Serge<EMAIL_ADDRESS>wrote:

@mgajda https://github.com/mgajda, just did it -- deepfire@f4090a1
https://github.com/deepfire/hflags/commit/f4090a1131b7cb3ca0078fee2dc797ed8453f41a
It does build with GHC 8 for me.
—
Reply to this email directly or view it on GitHub
https://github.com/nilcons/hflags/issues/15#issuecomment-192540594.

--
Pozdrawiam
Michał

@mgajda, absolutely!  ..just, maybe not right now. : -)
I was already in exploratory mode, though, this package being rather tangential to my interests, and, on the other hand, I've needed a git repo/commit for my Nix override.. it was done in the github editor, which says a lot..

Sorry, we (both maintainers) are travelling at the moment! But, I'll try to merge Mihal's PR and release a new version asap.

Good news!
Let me know if You travel by Singapore - we have Haskell.SG meetup this week.
Michal

Merged and bumped the version. Will ask Gergely to upload it to Hackage!

Uploaded!


Consistent depth scale?  No normalization.

Thanks for sharing this great work!
I've noticed that the output depth map seems to leverage the full RGB range it has available for every single image it creates, which makes perfect sense for outputting a single image.  However, I would like to test outputting mulitple frames with a constant depth scale.  I've notice that in misc.py there are some "denormilize"  and "normalize" sections.  Can you provide any insight on what I might modify in order to get a consistent rendering of the depth scale across multiple frames?

@noobtoob4lyfe I think you may be looking at the colorized output depth map, which uses the full RGB range to make the depths more visible. The raw depth map (saved as a PNG where each pixel = estimated depth * 256) should be more consistent. The typical way this raw depth file is saved in the code is by getting depth from the model's inference, and then passing that to the zoedepth.utils.misc.save_raw_16bit function to save.
Though in my experience the raw depth scale can sometimes still be inconsistent, particularly when running on scenes that probably don't match the training data that well (I've been running this on forest images).


Have the test suite module work with several clients, minions, SSH minions, etc.

spacewalk issue SUSE/spacewalk#10730
problem 1
Identifiers for the different client types lead to absurdities. Instead of
    cli-sles12sp4 = {
      name = "client"
      image = "opensuse152o"
    }

we should have something like:
    suse_client = {
      name = "client"
      image = "opensuse152o"
    }

problem 2
Testsuite module accepts 0 or 1 of each client, it should be 0 to N, especially for the QAM test suite:
  suse_clients = [
       client1 = {
          name = "client-leap-15.2"
          image = "opensuse152o"
    }, client2 = {
          name = "client-sles-15.1"
          image = "sles15sp1"
    }
   ]

problem 3
For big setups like the QAM test suite, and for the uyuni test suite, we have to be able to specify different providers in base module. It could be several libvirt hosts, or even a mix of libvirt and cloud hosts.
This is currently possible when not using the cucumber test suite module, with things like that:
module "cli-sles15sp1" {
  providers = {
    libvirt = libvirt.classic181
  }
  source             = "./modules/client"
  base_configuration = module.base3.configuration
  ...
}

module "base3" {
  providers = {
    libvirt = libvirt.classic181
  }
  source = "./modules/base"
  provider_settings = {
    pool        = "default"
    bridge      = "br0"
    additional_network = "<IP_ADDRESS>/24"
  }
  ...
}

I'm not sure if we can achieve this with current test suite module.

I see problems 1 and 2 as related, and strictly dependent on the testsuite code: it was structured the way it is to follow what the testsuite needed. If the testsuite becomes more flexible in accepting VMs, my opinion is that we should follow that flexibility in the same way in sumaform accordingly.
Question is: does the Cucumber testsuite accept only one "SUSE client" and the version is a variable, or should it accept an array of clients with a bunch of OS's? I tend to think the latter is preferable. So one idea could be to expect an array of clients, each of which will have an OS and some flags or a label that indicates its "role" (traditional client, minion, image build host...).
About problem 3, there is the possibility of defining multiple providers in the top-level main.tf file and then pass a reference to a certain provider down in the modules. It requires some surgery but it can be done, and I do not think there's much of an alternative in this case (unless we want multiple separated main.tfs, which is probably worse).
Thanks for bringing this up

it was structured the way it is to follow what the testsuite needed.

Well, no. It was never "needed" that the test suite handled "SLE 12 SP4 minion". It was never "needed" that the test suite conformed to our (censored) naming of VMs. What was needed was always that the test suite handled a "SUSE family minion" as something generic.
This is mostly an effect of miscommunication. I am probably to blame, for not having detected the issue when I was requested to have a look at the PRs. Sorry, my bad.

Question is: does the Cucumber testsuite accept only one "SUSE client" and the version is a variable, or should it accept an array of clients with a bunch of OS's? I tend to think the latter is preferable.

The current test suite accepts only one client, but the upcoming QAM test suite will need several of them. Currently we work around this by not using the test suite module:
https://github.com/srbarrios/susemanager-ci/blob/migrate-qam-env/terracumber_config/tf_files/SUSEManager-4.0-qam.tf
Apart from this new testsuite, I often feel the need for my own private tests to have several machines of one kind.

So one idea could be to expect an array of clients, each of which will have an OS and some flags or a label that indicates its "role" (traditional client, minion, image build host...).

That is indeed Ricardo's idea and I think that's the way to go.
A possible alternative could be to plan for, say, a maximum 8 clients and hardcode them:
   suse-family-client = {
          name = "client-leap-15.2"
          image = "opensuse152o"
    }
    suse-family-client-2 = {
          name = "client-sles-15.1"
          image = "sles15sp1"
    }

That would avoid the extra burden of declaring an array. @rjmateus, what do you think, would that be doable?

About problem 3, there is the possibility of defining multiple providers in the top-level main.tf file and then pass a reference to a certain provider down in the modules. It requires some surgery but it can be done, and I do not think there's much of an alternative in this case (unless we want multiple separated main.tfs, which is probably worse).

The idea is precisely to avoid several main.tfs. Again, see
https://github.com/srbarrios/susemanager-ci/blob/migrate-qam-env/terracumber_config/tf_files/SUSEManager-4.0-qam.tf

Thanks for bringing this up

Sure. I will take care of it.

However, this proved impossible with HCL: for_each does not work in modules.
So my PR will solve problem 1 only. For problem 2 and 3, we'll have to wait until terraform becomes a real programming language.

However, this proved impossible with HCL: for_each does not work in modules.

https://github.com/hashicorp/terraform/tree/guide-v0.13-beta/module-repetition#terraform-v013-beta-module-for_each-and-count
Will that help?

Yes Moio, that is exactly what is needed to implement this card. Eric didn't close this card so we can wait for the feature to be available and then revisit it on that time

Yes. For now I just address the renaming.


First additions

I have made a Repos page and linked it up to the homepage @CodingSpecies. Thanks (:

@CodingSpecies made the changes


ZuulFilterInitializer is not friend of servlet 2.5

In container of servlet 2.5, I have to wrap ZuulFilterInitializer because it can not use directly in web.xml.
public class ZuulFilterInitializer implements ServletContextListener {

  @Autowired
  private Map<String, ZuulFilter> filters;

  private org.springframework.cloud.netflix.zuul.ZuulFilterInitializer delegate;

  @Override
  public void contextInitialized(ServletContextEvent sce) {
    SpringBeanAutowiringSupport.processInjectionBasedOnServletContext(this,
                                                                      sce.getServletContext());
    delegate = new org.springframework.cloud.netflix.zuul.ZuulFilterInitializer(filters);
    delegate.contextInitialized(sce);
  }

  @Override
  public void contextDestroyed(ServletContextEvent sce) {
    delegate.contextDestroyed(sce);
  }
}

Spring Boot in general is not friendly to servlet 2.5 and recommends a third party package.  Have you followed that advice?

Yes, we follow 74.5 Deploying a WAR in an Old (Servlet 2.5) Container.
However, because it's servlet 2.5, we have to register ZuulFilterInitializer as a servlet listener in web.xml. But ZuulFilterInitializer in Spring cloud has no default constructor, which cannot be used in web.xml directly.

What you did seems like a reasonable workaround.  What could we do that might help you?

it's better original ZuulFilterInitializer can register dependencies by itself, so we can use it in web.xml directly

Closing this due to inactivity. Please re-open if there's more to discuss.


Census prod

@epintozzi @bermannoah @NZenitram
All tests are passing, new gem and API routes are set. Merge when ready, make sure to bundle! 🐈

Awesome!


RegisterRuntimeInitializeAndCleanup Fatal Error after numerous starts/stops

Overview
With a simple project (used HeadsetAdjustment scene), hitting start/stop 27 times in rapid succession eventually causes Unity to throw a Fatal Error and quit.
Error message (manually typed from the dialog) is:
Fatal Error!
RegisterRuntimeInitializeAndCleanup::Register has reached the limit 700
MaxLimit 700
UnityEngine.XR.WSA.Input.InteractonManager:.cctor()
HoloTooIkit.Unity.InputModuIe.MotionControIIerVisuaIizer::Awake() (at
Assets/HoIoTooIkit/Input/Scripts/MotionControIIerVisuaIizer cs: 74)

Expected Behavior
That it doesn't do a fatal error
Actual Behavior
That it does
Steps to reproduce
Load HeadsetAdjustement scene.  Hit Start/Stop 27 times.  See error dialog
Unity Editor Version
2017.4.1f1
Mixed Reality Toolkit Release Version
2017.2.1
This could obviously be a Unity bug, I'm sending it to them also.

Hit Start/Stop 27 times.

haha, I can image spamming the play button that many times 😂

I was trying to reproduce an error I had previously, where Holographic Emulation would cause Assertions and require a restart after 5-ish start/stops.   That appears to have gotten fixed in 2017.4, but then I ran into this error..... eventually......

Unity has been able to repro this bug, so I suspect it's on their side.  I'll close for now.


chore(release): v1.0.3 🎉

:robot: I have created a release beep boop
1.0.3 (2023-09-17)
Bug Fixes

pkg: forward the getError method from error-fns for convinience (365b2d2)

This PR was generated with Release Please. See documentation.

:robot: Release is at https://github.com/ehmpathy/test-fns/releases/tag/v1.0.3 :sunflower:


Update official lambda layers with newer pandas version

Is your feature request related to a problem? Please describe.
The current "official" lambda layers from here ship with pandas 1.3. This is quite outdated as there is already a new major version of pandas available. Pandas 2 comes with great performance enhancements which are currently not usable with awswrangler without messing around with homegrown lambda layers.
Describe the solution you'd like
An official awswrangler lambda layer built on Pandas 2.1. Based on the pyproject.toml file, Pandas 2 is in principle supported.
Describe alternatives you've considered
If a lambda layer was already built by someone, I would be happy to use it.

The link you shared is pointing to version 2.15. The latest version of the library is 3.4 which does package pandas 2.x in the layers

Thank you and sorry for not finding it myself, somehow the older version showed up before the current one in a google search.


Bad argument in arithmetic expression in Extreme.slice_content/3

While processing many of events per second, we are randomly running into a bad argument in arithmetic expression error.
** (EXIT) an exception was raised:
        ** (ArithmeticError) bad argument in arithmetic expression
           (extreme) lib/extreme.ex:432: Extreme.slice_content/3
           (extreme) lib/extreme.ex:425: Extreme.process_package/2
           (extreme) lib/extreme.ex:407: Extreme.handle_info/2
           (stdlib) gen_server.erl:616: :gen_server.try_dispatch/4
           (stdlib) gen_server.erl:686: :gen_server.handle_msg/6
           (stdlib) proc_lib.erl:247: :proc_lib.init_p_do_apply/3
            (elixir) lib/gen_server.ex:774: GenServer.call/3

We've used the Logger.debug from inside Extreme.slice_content to see that data that is coming in:
2018-03-09 12:09:14.407 [error] We have unfinished message of length nil(2): <<36, 0>>

While searching logs in our environments, I couldn't find such message. It will be hard to reproduce. Can you please give me any clue how to do it. What is order of "processing many of events per second"? It would be great if you could reproduce it in a test. State.should_receive = nil is set in few places and that is situation you are facing with. We've been processing with listeners something around 20k events today and didn't have any exceptions

We see this when pushing ~5.000 events in a short timespan to 11 persistent subscriptions, so in the order of 60.000 events in a few minutes flowing through the system.
This is also seemingly not bound to a specific event or anything, sometimes it happens - sometimes it doesn't.
We also saw a different behaviour that might be connected (not sure yet though): Sometimes Eventstore logs a heartbeat problem on one of our subscriptions which kind of smells like a full inbox in one of the subscription processes

Hi all,
@burmajam  If I'm not mistaken, there is one code branch which is never executed and could be the reson why this occur.
If you check this line and look into code, you will see that such match {:unfinished_message, expected_message_length, data} will never be generated in code by procees_message function. Also the 3rd function process_content will take everything BUT resets state.should_receive to nil, which is exactly what @janpieper described at the bottom of issue report.
2018-03-09 12:09:14.407 [error] We have unfinished message of length nil(2): <<36, 0>>
So there is still part of message but should_receive is nil.
This is issue where received tcp package was just part of protobuf message. But what is interesting here and odds are a bit low, is probably that extreme received partial message as TCP packages more than once in row, so after reset on line extreme.ex:453 it finally hit this buggy case.

We were able to reproduce the ArithmeticError in our codebase and @timbuchwaldt created a PR which fixes the issue for us.

Hi guys,
I've noticed that one test fails now ( https://github.com/exponentially/extreme/blob/master/test/extreme_test.exs#L806 ). It is tagged as manual so we didn't catch it by travis. I'll have to revert back pull request because of that. It would be great if you could write test for your case and rewrite solution.
Best,


Translation Delivery

Translation Delivery

This pull request is for translation delivery. Please review and approve them,  @D067796

This pull request is for translation delivery. Please review and approve them,  @D067796


ansible-galaxy is not updated with the latest master branch

Hi,
I noticed that the ansible-galaxy (https://galaxy.ansible.com/detail#/role/1453) is not updated with the latest changes.
Can you please update it?
Thanks
Shay.

+1

Done.


Update API-Extractor

Description
Update the version of API Extractor we use.
Reviewer Guidance
The review process is outlined on this wiki page.

Oops, I updated it across the whole repo, not just client. Will fix.


Confusion in ReadMe.md

Hi Customer-0
The comment Readme regarding DNS entry in step 7-8 seems little confusing. As we saw that DNS terraform step is not part of init-iaas any more and moved to deploy-iaas.
So it seems confusing and we are not sure how it actually work if we dont update the NameServer entries (in AWS Route 53) on the fly.
So it would be great if you can please update the Readme with the details.
Gaurab & @lojoe

@degaurab sorry its been a bit for me to look at this.   The readme only applies to the 'c0-gcp-concourse-poc.yml' pipeline variant which basically removes any trigger links from the init-env job to the deploy-iaas job for an intentional pause.  This is done so that an operator running the pipeline for a POC can use any DNS service and be provided with the IPs they will need for A/CNAME records.   If you look at the 'ci/pipeline-variants/c0-gcp-concourse-base-ci.yml' pipeline variant,  we simply added an additional task to the init-env job to use DYN for this purpose in an automated fashion.   That approach doesn't even use the GCP managed zone at all, its just creating it for doc purposes really.    We will be refactoring the pipelines very soon and will prob move creating the managed dns zone for GCP (Azure & AWS too) as part of the init-iaas job(s) as well so a DNS delegation could be used to refer to the GCP nameservers from a Route 53 registered domain ... or anywhere really,  as it stands now, using the GCP managed zone would have to occur at the end of the deploy-iaas job as you have already noted, but could be done similarly to the DYN integration task.  Not sure if the readme is inaccurate on this as its not requiring the use of the GCP managed DNS zone,  but id be happy to edit it if u have a suggestion :)

Closing this as it is no longer applicable. The README has been updated to reflect the many changes that happened to the pipeline.


Shouldn't changes in Cargo.toml trigger the check?

Pretty much the title.
If I change the Cargo.toml file and add a dependency it is not rechecking to add this dependency into rust-analyzer's context and the lsp never sees these crates until I reload the whole project.
Is there a way to accomplish this?

Looks like we need a custom command and an autocommand. For now you can run CargoReload in any rust buffer after updating your Cargo.toml


Update from microsoft/* to mcr.microsoft.com/*.

Switch .NET Core Images from the legacy location at Docker Hub Registry (microsoft/*) to a new location at Microsoft Container Registry (mcr.microsoft.com/*).

According to Richard Lander, .NET Core 2.0 images won't be published to MCR. Therefore, I'm closing this PR.


Installation ERROR : unexpected 'HyvaGridInterfaceFactory' (T_STRING), expecting function (T_FUNCTION) ..

PHP Parse error:  syntax error, unexpected 'HyvaGridInterfaceFactory' (T_STRING), expecting function (T_FUNCTION) or const (T_CONST) in app/code/Hyva/Admin/Block/Adminhtml/HyvaGrid.php on line 11
Magento Version :  2.3.5-p2
PHP Version : 7.2.34
I am not sure, but does it only support 2.4 ?

I think the problem is PHP 7.2.
Check this:
https://github.com/hyva-themes/magento2-hyva-admin/blob/main/doc/1. Overview/1. Installation.md
Minimum requirement is PHP 7.3

I think the problem is PHP 7.2.
Check this:
https://github.com/hyva-themes/magento2-hyva-admin/blob/main/doc/1. Overview/1. Installation.md
Minimum requirement is PHP 7.3

Yeah, probably an issue with PHP 7.2, It shouldn't be too hard to backport though.
Since the class is in app/code/, it seems like you cloned the git repo instead of using a composer package installation.
That's fine, but I wanted to note that there is a php73 branch which might work better/be less work to backport, compared to the main branch which is for PHP 7.4.

Yeah, probably an issue with PHP 7.2, It shouldn't be too hard to backport though.
Since the class is in app/code/, it seems like you cloned the git repo instead of using a composer package installation.
That's fine, but I wanted to note that there is a php73 branch which might work better/be less work to backport, compared to the main branch which is for PHP 7.4.


MGMT-14922: Allow 'patch' in custom manifest file name

Related to https://issues.redhat.com/browse/MGMT-14922

I've closed the PR because we need some BE changes for this. I'll create a new one when BE have a solution.


Renaming __DO_NOT_USE_OR_YOU_WILL_BE_HAUNTED_BY_SPOOKY_GHOSTS

Hey :wave:
After reading through another issue about this (#2475) it seems like there aren't any plans for __DO_NOT_USE_OR_YOU_WILL_BE_HAUNTED_BY_SPOOKY_GHOSTS to be deprecated as jest-styled-components depends on it.
I think in that case it probably makes sense to rename this back to whatever it was called before or something else :+1:
Let me know what you think, I would be more than happy to submit a PR for this! 👊

So some clarification here, it was never called something else and was purely intended to hold the StyleSheet.
The reasoning behind that is rather simple, if we find a new way to manage stylesheets that changes their － otherwise rather stable  －  API then we're left with bumping the major version.
It's a class that is highly connected to how our internals work but doesn't affect our public API otherwise. Hence ServerStyleSheet is also separate.
Even with prior changes we don't change Stylesheet's API that often, but we want to make it clear in the code that it's not a public or stable API that might change to give us room for improvements. It's still exposed as an escape hatch, but the import name makes clear that it's just that.
Hope that makes sense :)

That makes sense, thanks for the quick response @kitten :+1:


CFFL

CFFL

Linter check found the following problems:
The following problems have been found:
ERROR: /tmp/abs_c5vrs5qb7l/clone/recipe/meta.yaml:15: pip_install_args: pip install should be run with --no-deps.
ERROR: /tmp/abs_c5vrs5qb7l/clone/recipe/meta.yaml:21: missing_wheel: For pypi packages, wheel should be present in the host section
ERROR: /tmp/abs_c5vrs5qb7l/clone/recipe/meta.yaml:38: missing_dev_url: The recipe is missing a dev_url
ERROR: /tmp/abs_c5vrs5qb7l/clone/recipe/meta.yaml:38: missing_documentation: The recipe is missing a doc_url or doc_source_url
ERROR: /tmp/abs_c5vrs5qb7l/clone/recipe/meta.yaml:38: missing_license_family: The recipe is missing the about/license_family key.
WARNING: /tmp/abs_c5vrs5qb7l/clone/recipe/meta.yaml:38: missing_description: The recipe is missing a description
Errors were found


fix: npm package not being published automatically on CI pipeline.

Description:
Fixes Apollo CI pipeline to just publish the npm package when a new release is triggered.

Code Coverage

Total Project Coverage
91.72%
:white_check_mark:

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

v1.2.15
GitHub release

Your semantic-release bot :package::rocket:


Legacy: Life Among the Ruins sheets not appearing in sheet dropdown?

After submitting these sheets it looks like they were approved, and added to the approved.yaml file, but they're not appearing in the selection dropdown on the site. It's looking like nothing added below Fire Emblem Tabletop in the approved.yaml file is available on the site - is it just a matter of waiting for things to filter through, or can I do something else to speed things along?

When I submitted new sheets there usually have taken longer time for new sheets to appear than any updates to existing ones, so surely they are up within 24h now. But yeah often this occurs on Tuesdays.

We delayed our push for the character sheets code this week until today, the sheets should be going live within a few minutes.  Though we do our best to push out these out regularly each Tuesday after they are accepted into the repository, on occasion there are times at which we delay the push to the following day or later.
Thank you for your patience on our part; if the PR was accepted, it will eventually (usually within 24h) be live on the site.


Fix jOOQ compilation with GraalVM

Fixes https://github.com/quarkiverse/quarkus-jooq/issues/73
Bumps jOOQ to 3.15.11 (didn't bump to 3.16 because it will require Jakarta to properly work)
Adds GraalVM substitutions to remove references to postgresql lib if the lib is not found in the classpath

/cc @angrymango

@gsmet this lib seems to be abandoned, would you be able to do a review of this PR and maybe do a release to fix GraalVM >= 21.2 compilation?


⚠ Bofrost.at has degraded performance

In dd73920, Bofrost.at (https://www.bofrost.at) experienced degraded performance:

HTTP code: 200
Response time: 883 ms

Resolved: Bofrost.at performance has improved in 6b5e744 after 8 minutes.


Add Crunchy DB for TEST and PROD

currently single bitnami postgis is deployed, need to move to crunchy db for HA in TEST and PROD

@acoard-aot , I will create a PR next week on this, I have already got the object store creds for crunchy backups.

completed as part of #124 and #125


IndexError out of bounds in exoctk.contam_visibility.visibilityPA.using_gtvt

I'm seeing that under certain circumstances (ra/dec) we are getting an out of bounds error in the contam_visibility.using_gtvt function at this line of code: https://github.com/ExoCTK/exoctk/blob/master/exoctk/contam_visibility/visibilityPA.py#L408
It looks like when we go to retrieve values from badPA there's an index out of bounds error.
Does it makes sense that badPAs would ever be empty?
Here's the ra/dec that we're using.
exoctk.contam_visibility.visibilityPA.using_gtvt(ra, dec, instrument, targetName="Target", output="bokeh")

where our inputs are
ra = '88.5897756488149'
dec = '-64.1297956126618'
instrument = 'NIRISS'

and the error is
exoctk/contam_visibility/visibilityPA.py", line 400, in using_gtvt
    grouped_badPAs = [[badPAs[0]]]
IndexError: index 0 is out of bounds for axis 0 with size 0

Hi @falkben --- thanks for finding this!
We'll have a look together with @jaymedina, who is the tool champion for this. I think badPAs will only be an empty array if the target is always observable, which is the case for the Continuous Viewing Zone (CVZ; https://jwst-docs.stsci.edu/jwst-observatory-hardware/jwst-observatory-coordinate-system-and-field-of-regard). The RA of this target is ~6 hours and declination is -64 degrees, which actually falls in the CVZ (see Figure 6 in the article) --- so this seems to be indeed the problem.  I would like, however, for @jaymedina to confirm this is the case. If it is, then this needs some little tweaks to solve this edge case, but should be relatively straightforward (just a couple of if statements in this and other functions that assume badPAs is not empty).
Thanks again for submitting this! Will let you know how this gets solved.
Néstor

Hi @falkben , thanks for opening this issue. @nespinoza is right, the badPAs array would be empty in cases where the target is visible 100% of the time, which seems to be the case with this target. I've opened a PR with a fix that accounts for cases like this ( PR https://github.com/ExoCTK/exoctk/pull/438) , and running the visibility calculator in this branch gives me the following output:

It looks like there is always at least 1 position angle in which this target can be viewed in (thus, no "bad PAs"), which is consistent with what Nestor mentioned about this target landing in the CVZ.
We will update this ticket when the PR gets merged and you can run the visibility tool as normal. Thanks for catching this bug!


Windows 11 config file problem

The essence of the problem is shown in the picture. I run the program through the standard settings of your configuration file and the parser of your program does not read the configuration file, freezing at a certain moment. For clarity, I ran it through bash, but it also freezes during normal launches. Installed your program through the Windows installer

Layered WallPaper hasn't been tested on Windows 11, so this could be a bug.
I suppose this is a user config file (the one in AppData ), because the program would print a message if it failed to load it. If that's the case, try deleting (or renaming) it and let the program use the default one instead. Let me know if it works.

Hello dear jszczerbinsky! The default file didn't work either, so after that I already tried to put my custom one, which also did not lead to anything. Thank you for giving me your time!

This is almost certainly related to some bug of config parser. The latest version uses libconfig to parse configuration files, so I'm going to close this issue.


Update FLAC to v1.4.3

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

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

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
306

Relevant Lines:
309

💛  - Coveralls


Development work for About page

This is a placeholder for the development work that will take place this quarter. Issues will be built out once designs and content are done.

Dropping assets for this page here
about page assets.zip

@segacy1 These assets don't include Shelby. I added a comment to the About page designs regarding the same issue.

Added Shelby in the designs and including her image here:

@segacy1 Can you re-export the image for Shelby so it matches the image sizes of the rest of the illustrations? It's a bit short:

@machikoyasuda try this one


Deleted required custom field on an opening still requires the custom field when creating a new opening

Bug Description
Making a required custom field for openings and then opening a new job with an opening.
Then delete that opening custom field.
When trying to make a POST request to the job to create a new opening the following error returns a 422 with this message:
"message": "Missing required custom field.",
      "field": "{deletdCustomFieldId}"

Then if you try to make a POST request on that job to create a new opening with that custom field also included in the body the following error returns a 422 with this message:
"message": "Couldn't find a custom_field in organization using {deletedCustomFieldId}"

So then it looks like you are unable to create a new opening via the API for this job since that custom field has been deleted
Last upstream Slate commit (run git log --author="\(Robert Lord\)\|\(Matthew Peveler\)\|\(Mike Ralphson\)" | head -n 1):
Put the commit hash here

Hey @jacobcieski, this repo (slate) is just for generating static API docs. You'll need to find the appropriate channel for whoever put up the docs and report this bug there.


feat($parse): provide a mechanism to access the locals object

Extends the built-in identifiers definitions by adding $local. This is a non-assignable reference to the locals object.
Closes: #13247

I would prefer $locals (over $local), since this is how we've been referring to it elsewhere (e.g. in the docs).
Other than that (and the test nitpick), it LGTM.


master branch checked out in stand-alone dir builds -- fails building in electron apps ./node_modules/iohook

environment: os: macOs 11.6.1
$ node_modules/electron/dist/Electron.app/Contents/MacOS/Electron --version
v13.2.3
$ node_modules/electron/dist/Electron.app/Contents/MacOS/Electron --abi
89

I can clone iohook to a stand alone directory and successfully build with this command:
$ git clone git@github.com:wilix-team/iohook.git
$ cd iohook
$ npm install
$ node build.js --runtime electron --version 13.2.3 --abi 89 --upload=false

gyp ends with an ok and this file is generated:
$ ls -l build/Release/iohook.node
-rwxr-xr-x  1 stephen  staff  70616 Oct 31 02:14 build/Release/iohook.node

So I tried an equivalent series of commands to do this in my electron applications node_modules directory:
$ npm install github:wilix-team/iohook#229774494 --save
$ cd node_modules/iohook
$ npm install
$ node build.js --runtime electron --version 13.2.3 --abi 89  --upload=false

Which fails with this error:
make: *** No rule to make target `Release/obj.target/uiohook/libuiohook/src/logger.o', needed by `Release/uiohook.dylib'.  Stop.

  Stop.
gyp ERR! build error
gyp ERR! stack Error: `make` failed with exit code: 2
gyp ERR! stack     at ChildProcess.onExit (/Users/stephen/dev/00-clients/rlmg/cfa-own-electron/node_modules/iohook/node_modules/node-gyp/lib/build.js:194:23)
gyp ERR! stack     at ChildProcess.emit (events.js:400:28)
gyp ERR! stack     at Process.ChildProcess._handle.onexit (internal/child_process.js:282:12)
gyp ERR! System Darwin 20.6.0
gyp ERR! command "/Users/stephen/.nvm/versions/node/v14.18.1/bin/node" "/Users/stephen/dev/00-clients/rlmg/cfa-own-electron/node_modules/iohook/node_modules/.bin/node-gyp" "configure" "rebuild" "--target=13.2.3" "--arch=x64" "--dist-url=https://atom.io/download/electron" "--v8_enable_pointer_compression=1" "--build_v8_with_gn=false" "--enable_lto=false"
gyp ERR! cwd /Users/stephen/dev/00-clients/rlmg/cfa-own-electron/node_modules/iohook
gyp ERR! node -v v14.18.1
gyp ERR! node-gyp -v v7.1.2
gyp ERR! not ok
Error: Failed to build...
    at ChildProcess.<anonymous> (/Users/stephen/dev/00-clients/rlmg/cfa-own-electron/node_modules/iohook/build.js:188:23)
    at ChildProcess.emit (events.js:400:28)
    at Process.ChildProcess._handle.onexit (internal/child_process.js:282:12)

I already have iohook in the package.json dependencies for the electron app so when I manually create the destination path and copy iohook.node successfully built in the git clone in a separate directory iohook and the application work without error.
I'd like to be able to do this in electron-forge more automatically and build distributable artifacts for Windows and macOS.
This manual process worked:
$ mkdir -p './out/CfA OWN Electron-darwin-x64/CfA OWN Electron.app/Contents/Resources/app/.webpack/main/builds/electron-v89-darwin-x64/build/Release'

$ ls -asl './out/CfA OWN Electron-darwin-x64/CfA OWN Electron.app/Contents/Resources/app/.webpack/main/builds/electron-v89-darwin-x64/build/Release'
total 0
0 drwxr-xr-x  2 stephen  staff  64 Oct 31 02:40 .
0 drwxr-xr-x  3 stephen  staff  96 Oct 31 02:40 ..

$ ls -l ../iohook/build/Release/iohook.node
-rwxr-xr-x  1 stephen  staff  70616 Oct 31 02:14 ../iohook/build/Release/iohook.node

$ cp ../iohook/build/Release/iohook.node './out/CfA OWN Electron-darwin-x64/CfA OWN Electron.app/Contents/Resources/app/.webpack/main/builds/electron-v89-darwin-x64/build/Release'

$ ls -l './out/CfA OWN Electron-darwin-x64/CfA OWN Electron.app/Contents/Resources/app/.webpack/main/builds/electron-v89-darwin-x64/build/Release'
144 -rwxr-xr-x  1 stephen  staff  70616 Oct 31 02:41 iohook.node

When you do npm install iohook only the essential "prod" part of the library is installed inside node_modules/iohook. When you clone the iohook repo you get the complete library source with everything needed to manually build. You could for convenience move the entire dev repo into node_modules/iohook and build from there, but then you'd have a lot of unnecessary junk packaged with your electron app unless you add exceptions to your Forge config to exclude them. The easiest thing to do in this situation IMHO is to have your separate iohook clone folder where you do your manual build, then copy your build files *.node and *.dylib into e.g. [your electron app source]/node_modules/iohook/builds/electron-v89-darwin-x64/buildRelease/ and then Forge will automatically include those in your packaged app.


clarified Cloud menu integration edge only

What's changed

clarified that integrated Cloud menus are currently edge only

Reviewers
@FrenchBen @jeanlaurent @friism @mstanleyjones @joaofnfernandes
Signed-off-by: Victoria Bialas <victoria.bialas@doc

LGTM, merging to get this published for now


Implement Black's rules around newlines before and after class docstrings

Summary
Black allows up to one blank line before a class docstring, and enforces one blank line after a class docstring. This PR implements that handling. The cases in crates/ruff_python_formatter/resources/test/fixtures/ruff/statement/class_definition.py match Black identically.

Would you mind rebasing the PR? It seems to contain many unrelated changes, which makes it difficult to spot the relevant lines.

Fixed.


protein import is broader than targeting

https://www.ebi.ac.uk/QuickGO/GTerm?id=GO:0030150
GO:0030150    protein import into mitochondrial matrix
is currently a child of
GO:0006626    protein targeting to mitochondrion
This is a true path violation.
Targetting refers to bringing the cargo to the mitochondria. import improves transport across the membrane.

Other problems with targeting and the relationship between targeting an import are described here:
https://github.com/geneontology/go-annotation/issues/2453
Can we discuss this on a future call?

Yes !


Any way to get Router for Linux?

Hello,
I have written a java Application with your Library. Is there any Way to put the Application on a Linux Server. Primarily the Problem is the Missing Router i think. Is there a Router planed for Java or do i have to write it on my own?
Regards Thomas

Hi @UserwithnoUsername,
unfortunately, this library does not currently have support for Linux systems.
However, here are a few resources that you might be able to build upon:

https://github.com/Beckhoff/ADS
https://github.com/Beckhoff/TF6000_ADS_DOTNET_V5_Samples/tree/main/Sources/RouterSamples

Best Regards, Jannik Voss

Hi @UserwithnoUsername,
Hi @martinste1n ,
I added Linux support using the "adslib.so" from https://github.com/Beckhoff/ADS/tree/master instead of the "TcAdsDll.dll", which as I mentioned, is not available on Linux.
There are some limitations+differences but it should be fine more most use-cases. I would call it a Beta. Details:
https://github.com/Beckhoff/AdsToJava/tree/linux?tab=readme-ov-file#status-of-the-linux-support
I would really appreciate feedback/contributions on this before merging it into the main branch.
Best Regards, Jannik Voss

FYI, I merged it into the main branch and added a CI build action (Ubuntu). The binaries are published as a release artifact called "linux-x64.zip".


Godot mobile editor crash after error

Godot version
Mobile Editor  3.6.beta.2 android
System information
I am on a samsung a04s android 11
Issue description
When you run a piece of code that has a error in it in the mobile version of godot the scene becomes black and it also does not let you leave the scene the only way to go back is by manualy leaveing the app i havd no idea what causes this or how to fix it but it would be nice if you would i have also provided a little demostration down below kind regards
:  random poor  kid
Steps to reproduce
Go into godot type a error and run it should brick the entire thing
Minimal reproduction project

I also got this behaviour when I tested the example code. @m4gr3d

me too

I think the normal logic would be something like this: There is a problem -> Go back to the engine edit page and display the error message and memory information. Just like on the computer side

I am also facing the same problem. Is godot going to fix this bug anytime soon?
There might be many users who want to use Godot but might be leaving it because of this error.

The issue occurs because the gdscript error causes the Godot debugger to freeze the running project until it's resumed.
This is not visible on a phone because the limited screen estate prevents seeing both the editor and the running project at the same time, but the behavior can be observed on a tablet as shown in the recording below.
https://github.com/godotengine/godot/assets/914968/faaff508-11df-4ebb-9eab-484ff5c0096e
A simple fix would be to have an option to disable that behavior from the debugger.
@lawnjelly @Calinou Do you know if such an option already exists?

Thanks for the report! Consolidating in https://github.com/godotengine/godot/issues/77272, as this is due to the same cause.

A simple fix would be to have an option to disable that behavior from the debugger.
@lawnjelly @Calinou Do you know if such an option already exists

Btw, do you know if we already have the option to disable the described behavior from the debugger?

I don't think so, but I'd suggest starting to look at the breaked signal callback which is sent to the script editor by the debugger.
It seems to have some logic to bypass some stuff if using an external editor, maybe that could help inspire a fix:
void ScriptEditor::_breaked(bool p_breaked, bool p_can_debug) {
	if (bool(EditorSettings::get_singleton()->get("text_editor/external/use_external_editor"))) {
		return;
	}

	debug_menu->get_popup()->set_item_disabled(debug_menu->get_popup()->get_item_index(DEBUG_NEXT), !(p_breaked && p_can_debug));
	debug_menu->get_popup()->set_item_disabled(debug_menu->get_popup()->get_item_index(DEBUG_STEP), !(p_breaked && p_can_debug));
	debug_menu->get_popup()->set_item_disabled(debug_menu->get_popup()->get_item_index(DEBUG_BREAK), p_breaked);
	debug_menu->get_popup()->set_item_disabled(debug_menu->get_popup()->get_item_index(DEBUG_CONTINUE), !p_breaked);

	for (int i = 0; i < tab_container->get_child_count(); i++) {
		ScriptEditorBase *se = Object::cast_to<ScriptEditorBase>(tab_container->get_child(i));
		if (!se) {
			continue;
		}

		se->set_debugger_active(p_breaked);
	}
}

But I'm not sure that would be sufficient.

Is this error fixed in the new updates ? If not then is there any way to fix this from inside the Godot editor or we have to suffer this until it gets fixed. It has been 6 months since i wanted to use Godot on android but this error is not letting me use it. Whenever there is an error in the code then the editor should return me to the debugger and tell me what the error is but nothing happens. I am stuck in the main game window. Pls anyone fix this faast

As far as I can see the correct fix is for the focus to change from the running app to the editor when a bug is found, and back again when rerunning.
But maybe there are reasons why this is not simple / possible, maybe @m4gr3d can explain, most of us will know next to nothing about swapping processes on Android. :blush:

@m4gr3d do you have any idea of when this big will be fixed ?

OK when I set editor/run/window_placement to side by side, I no longer have inoperable at script error.
Instead the screen is split into two with the editor short in portrait on the left, but the mouse still works (just can't get to the parts to stop the running project from the debug menu).
It looks like one key problem is that the back button is disabled, this presumably would be better used to to be able to quit the debug session (in this situation at least, when the debugger is active).
Really we want to minimize the running debug app, and maximize the debugging app.

Still the major bug preventing use of the Android Editor in 3.x so re-opening.
When last discussed @m4gr3d suggested discontinuing support for Android Editor in 3.x, but should be left open in case we can find some resolution.
Also probably the same issue as:
#94159
#86005

@lawnjelly Now that GodotCon is over, and if there are no further objections, I can take a look at discontinuing support for the Android editor in 3.x.

I've just unpublished the 3.x version, which means that new users cannot find and install it, although existing users can still find it.
I think that should be sufficient for the time being, and this still gives us the option of re-publishing this version of the editor if we want.

@lawnjelly Note that regarding this issue, I believe both https://github.com/godotengine/godot/pull/95700 and the upcoming https://github.com/godotengine/godot/pull/97257 mitigate it for 4.x, the issue being able to backport either of them for 3.x.


Add extra service support for topological - example sources

Add support for extra additional services to topological_worker. This allows us to switch to different backend for other entities and may help us in future in transition to more unified data collection. This example allows us to switch to sources service for /sources endpoint (since this will be removed from topological inventory) next week.
This PR also provides further simplification on sub/main collection methods.
It's intended to work like this:

In configs/topological_queries.yml you can specify the source service for the entity

sources:
    main_collection: sources
    service: SOURCES

The service specifier is used as a key to SERVICES_URL dict. If the service doesn't exist in this dict or is not defined in the query spec above, we default to topological inventory
The service url is used as a base for API calls for this entity.

It works the same for both main collections and sub collections...
It requires SOURCES_HOST and SOURCES_PATH being present in env, see related PR:
https://github.com/RedHatInsights/e2e-deploy/pull/240

Codecov Report

Merging #35 into master will increase coverage by 0.23%.
The diff coverage is 41.17%.

@@            Coverage Diff             @@
##           master      #35      +/-   ##
==========================================
+ Coverage   36.28%   36.51%   +0.23%
==========================================
  Files          14       14
  Lines         350      356       +6
==========================================
+ Hits          127      130       +3
- Misses        223      226       +3

Impacted Files
Coverage Δ

collector/env.py
100% <100%> (ø)
:arrow_up:

collector/topological_inventory.py
26.58% <33.33%> (-0.09%)
:arrow_down:

Continue to review full report at Codecov.

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

There's a bug in this PR! Let's let @AparnaKarve do some bug hunting. :dancer: The challenge has already started!

@AparnaKarve it's not an edge case, but it would be an regression. :slightly_smiling_face:  See the new commit https://github.com/ManageIQ/aiops-data-collector/pull/35/commits/4468d2505f8039b9f76cc5690f49572e43138a37
The problem was that for pagination (next pages listing) a fixed host was used, so it would query different endpoint for sources on next page. :wink:

Pagination...yes, that would be it! Did not catch the bug with visual inspection, and Topo. Inv Sources output wasn't large enough to catch the issue with testing
Sorry for missing that, and great catch!


Add checkout session metadata to one time payments

Bug report

Extension name: firestore-stripe-subscriptions

Describe the bug
When I pass metadata with a single payment the same way I pass metadata for a subscription, I would expect the metadata to show up for the Stripe payment. I can see the correct metadata passed to the checkout session document, but it doesn't flow through to the payment document.
Additional context
I've only quickly skimmed through the code, but it looks like the metadata for one time payments might be getting lost here?
https://github.com/stripe/stripe-firebase-extensions/blob/114d09c6f4d3310acd3cc67702991ad02808047d/firestore-stripe-subscriptions/functions/src/index.ts#L154-L160

Looking into the Stripe documentation:
https://support.stripe.com/questions/using-metadata-with-checkout-sessions
Maybe the above snippet should be something like this:
 if (mode === 'subscription') {
   sessionCreateParams.subscription_data = {
     trial_from_plan,
     metadata,
     default_tax_rates: tax_rates,
   };
 } else {
   sessionCreateParams.payment_intent_data = {
     metadata,
   };
 }

This same code change was brought up in #181


Add ability to suppress next line diagnostics inline

Code action to suppress next line diagnostics inline

Please add some baseline files to https://github.com/Azure/bicep/tree/main/src/Bicep.Core.Samples/Files for some typical use cases, as well as any edge cases that you can think of (e.g. unusual formatting). This helps visualize the parse tree, and ensures the syntax can be round-tripped. For example, things like:
#disable-next-line                                                       massive-space /* inline comments */ next-diag
#disable-next-line//block comment
#     disable-next-line // is a space allowed?

Please include the grammar changes in https://github.com/Azure/bicep/blob/main/docs/grammar.md

Please also update the common grammar libraries:
https://github.com/Azure/bicep/tree/main/src/textmate/src/bicep.ts
https://github.com/Azure/bicep/tree/main/src/monarch/src/bicep.ts
https://github.com/Azure/bicep/tree/main/src/highlightjs/src/bicep.ts
Feel free to create a separate task for this if you don't want to include it in this PR - we should just try to get it done before the next Bicep release.

TODO: Add integration tests for below scenario
var vmProperties = {
  diagnosticsProfile: {
    bootDiagnostics: {
      enabled: 123
      storageUri: true
      unknownProp: 'asdf'
    }
  }
  evictionPolicy: 'Deallocate'
}
resource vm 'Microsoft.Compute/virtualMachines@2020-12-01' = {
  name: 'vm'
  location: 'West US'
#disable-next-line BCP036 BCP037
  properties: vmProperties
}

Please add some baseline files to https://github.com/Azure/bicep/tree/main/src/Bicep.Core.Samples/Files for some typical use cases, as well as any edge cases that you can think of (e.g. unusual formatting). This helps visualize the parse tree, and ensures the syntax can be round-tripped. For example, things like:
#disable-next-line                                                       massive-space /* inline comments */ next-diag
#disable-next-line//block comment
#     disable-next-line // is a space allowed?

Added some baseline files as part of below two commits:
https://github.com/Azure/bicep/commit/3a23303bcb1c6f16a825cf431d768dd721985cb5
https://github.com/Azure/bicep/commit/c8a394a0db17a9407491c80f7c2f9f7adcf0fd66

I was playing around with this in a codespace and noticed the directive kind of blends with the keyword on the next line:

We may need to revisit the syntax highlighting to make it more visually distinctive (ok if it's a separate PR).

Looks like the formatter is eating the #disable-next-line directives:

Looks like the formatter is eating the #disable-next-line directives:

#majastrz that issue must be fixed with latest changes. Let me know if it isn't.

@majastrz that issue must be fixed with latest changes. Let me know if it isn't.
Yeah good pt. Let me check...

@majastrz that issue must be fixed with latest changes. Let me know if it isn't.
Yeah good pt. Let me check...

Yup, works now.

This is out of scope of this PR, but is it possible for us to validate things like this?

It would be neat if we could tell the user that it's not a real diagnostic. (Especially in the future when custom analyzers can be imported.)

It would be neat if we could tell the user that it's not a real diagnostic. (Especially in the future when custom analyzers can be imported.)

+1 to this. We've also discussed having a linter which checks for unused disable-next-line directives, which should be straightforward to implement, and may also solve this need.

We may need to revisit the syntax highlighting to make it more visually distinctive (ok if it's a separate PR).

The theme I was using in the codespace was broken. It looks different with the default dark theme:

Btw, using the codefix doesn't preserve indentation like the formatter does:

We should probably make that consistent with what the formatter does.

Please also update the common grammar libraries: https://github.com/Azure/bicep/tree/main/src/textmate/src/bicep.ts https://github.com/Azure/bicep/tree/main/src/monarch/src/bicep.ts https://github.com/Azure/bicep/tree/main/src/highlightjs/src/bicep.ts
Feel free to create a separate task for this if you don't want to include it in this PR - we should just try to get it done before the next Bicep release.

I'll do this in a follow up PR. Create https://github.com/Azure/bicep/issues/5156 to track the work.

It would be neat if we could tell the user that it's not a real diagnostic. (Especially in the future when custom analyzers can be imported.)

+1 to this. We've also discussed having a linter which checks for unused disable-next-line directives, which should be straightforward to implement, and may also solve this need.

We could add a new linter rule to check if the codes are invalid. I'll do a separate PR for this. Created https://github.com/Azure/bicep/issues/5157 for tracking purposes.

TODO: Add integration tests for below scenario
var vmProperties = {
  diagnosticsProfile: {
    bootDiagnostics: {
      enabled: 123
      storageUri: true
      unknownProp: 'asdf'
    }
  }
  evictionPolicy: 'Deallocate'
}
resource vm 'Microsoft.Compute/virtualMachines@2020-12-01' = {
  name: 'vm'
  location: 'West US'
#disable-next-line BCP036 BCP037
  properties: vmProperties
}

Added test for above scenario.

Please include the grammar changes in https://github.com/Azure/bicep/blob/main/docs/grammar.md

Updated grammar:
https://github.com/Azure/bicep/commit/4314c185ad4c5a200019edaf8ec7fc2a7c096dc9

Found an example. The one I was looking for was:
#disable-next-line secure-secrets-in-params


Question-How much maximum participants it supports per room & How much rooms can be served simultaneously?

How much participants does it able to handle

https://webrtcglossary.com/mesh/
The video conference demo is mesh architecture. As you can see on the link, mesh architecture cannot be used by a large number of users(just 4~6 participants) because each peer must transmit and receive video stream data. If you only use the audio conference, a little more users can participate because the each peer has less data to send and receive than the video conference.
The broadcast demo is a little different. show this link. https://github.com/muaz-khan/WebRTC-Scalable-Broadcast. According to the link, a large number of users can use it because only one peer transmits video stream data and the rest of the participants receive it.

how can i increase number of user to 50

how can i increase number of user to 50
As far as I know, yout should use MCU or SFU structure open source using media server  such as Janus, kurrento, mediasoup, jitsi.


per-exposure output data model

The current per-exposure output data model is fairly absurd. For each exposure, there are currently 3 very similar output files containing tables with one row per CCD. These 3 files should either be combined into one single table, or combined into one multi-extension FITS file per exposure.
There is a similar situation for the -astrom and -photom outputs. We should either combine these into one table, or write them out as a single multi-extension FITS file.
The above suggestions would take us from 5 output files per exposure to only 2, saving a lot of inodes. An even more extreme alternative would be to write the contents of the current 5 output files per exposure to one single multi-extension FITS file, reducing the number of output inodes by 5x.

Also, there should be a directory structure for these outputs that doesn't involve putting all files into one giant flat directory.

in 0d83d8f, added a level of directory structure, and eliminated astrom and zpt outputs.


Correct way to enable Server GC in RC2?

The migration guide suggests something like the following, which seems from https://github.com/aspnet/Announcements/issues/175
{
  "runtimeOptions": {
    "gcServer": true,
    "gcConcurrent": true
  }
}

However, when I tried on RC2, it doesn't work for me.  I had to use the code below
  "runtimeOptions": {
    "configProperties": {
      "System.GC.Server": true
    }
  }

Covered by the Annoucement: https://github.com/aspnet/Announcements/issues/175

@ajaybhargavb @piotrpMSFT
Here is what I see in our RC2 templates:
  "runtimeOptions": {
    "gcServer": true
  }

When you say that the above doesn't work for you can you clarify what exactly you are seeing? How are you detecting whether server GC is being used?

I have this code:
            if (GCSettings.IsServerGC)
            {
                Console.WriteLine("Server GC");
            }
            else
            {
                Console.WriteLine("Workstation GC");
            }

It prints out "Workstation GC" when I use gcServer": true

A related issue: https://github.com/dotnet/cli/issues/2121
A related commit: https://github.com/aspnet/benchmarks/commit/b7632666c7aa9a90f1893f5c677239d1795001b8

@danroth27, I think the structure of runtimeOptions has changed since the announcement. https://github.com/aspnet/Home/issues/1418#issuecomment-219815694

@sayedihashimi @phenning @ajaybhargavb @piotrpMSFT @rachelappel  So what we have in the templates for RC2 is incorrect? The JSON schema also appears to be incorrect:

The runtime team owns the schema, and I think it can be updated on the fly.

@sayedihashimi @barrytang This is more than just a schema issue. It looks like what we have in our template project.json files is wrong.

I'm a bit lost. Is this valid also for webjobs (console apps) targeting net461?

No, for console apps target the full .NET Framework you would need to enable server GC in app.config.


enable MMALPythonSource to send MJPEG and H264 streams

With this changes I am able to play h264 files with the given example.
There is still an output buffer overrun. (Some channel is missing a 'block=True' probably)
It's nice to see the bird from the userland test.h264 falling the first time :-)
And very educational to see the effect of missing frames...

Sorry - this isn't quite the intended usage of MMALPythonSource; it's really meant to be sub-classed for usage. But I take the point there should definitely be more examples of mmalobj usage in 1.14 (and I'm intrigued that buf.length needs setting - I'll have a dig into this for 1.14). Many thanks for the PR anyway, it'll definitely help in forming some new examples! I'll leave this open for now until I get to it in the course of 1.14.

hey @waveform80! any news on this topic? i'd be interested in an example to send a MJPEG stream to the MMAL chain.


hard fileupload dependency introduced in 2.0.2

This commit 7b9ee40724053abe63c7de90d6746bac89766cb8 introduces a hard dependency on the fileupload plugin. When it's not included, it crashes.
For the user app we are using cloudinary only for displaying of images, so we don't need the fileupload.
As a bonus it would be nice if there was a smaller build which doesn't include the uploading capabilities at all.

Hi @romanbsd,
Sorry for the delay. If you are not using the upload capabilities, you should use cloudinary-core or cloudinary-jquery (which includes the jQuery plugin but not the fileupload functionality). This was the major reason for version 2.
If you are using bower you can get them using bower install cloudinary-core etc.


Block redirect js by flashscore.com

When a try the command splash:go()
to navigate on flashscore.com
The website redirect me to another page.
This webpage says "refresh your browser".
I used a RotatorUseragent class and on SplashRequest argoument I set headers={dont follow=True}
Suspected js code block me!
(On scrapy the webpage show me the correct result html but without dynamic contents!)

P.s. (settings.py)
ROBOTSTXT_OBEY=False


Authorization server stateless

Hi,
I am building an authorization server which deliver a JWT. Once the authorization code flow is completed between the client and the auth server, the client set the JWT received as Cookie. After 5 minutes, the JWT is expired so my client asks for a new token. I would expect the user to be redirect to the auth server to login again. (Refresh token is disabled on the auth server).
What I can see is that the client receives the token directly without going to the form login because when the user logged in the first time, the auth server created a JSessionId.
Is it the normal behaviour ?
I tried to set the auth server stateless, but the redirect after the /login is not working since the information about the request are not known after the POST on /login. (I found a post on StackOverflow about it https://stackoverflow.com/questions/45180747/redirecting-to-original-url-for-stateless-session).
Thank you!
Regards,
Jaip

Hi @vpm911,
I found a workaround for this : https://github.com/spring-projects/spring-security-oauth/issues/140
Hope it helps

Thank you so much! This was exactly what I was looking for!
On Wed, 26 Sep 2018, 17:45 jaipablase<EMAIL_ADDRESS>wrote:

Hi @vpm911 https://github.com/vpm911,
I found a workaround for this : #140
https://github.com/spring-projects/spring-security-oauth/issues/140
Hope it helps
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/spring-projects/spring-security-oauth/issues/1457#issuecomment-424693125,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AMkFDo4reIDLWWvgfJ8LXuozvm5dmoe9ks5ue2_vgaJpZM4V_fGF
.


6.ADD user controllers

added the requirement to userController:
find all user
find one user by id
create user
update user by id
delete user by id
add friend by id
update friends list to remove friend by id

oh also added comments and formatted all existing documents


Feature: Support FTS indexes on nested fields

SDK
Python
Description
import lancedb
import pyarrow.parquet as pq
# Connect to LanceDB
db = lancedb.connect("data/cases2.lance")

print(db.table_names())

data = pq.read_table("data/part-00000-bd7b5d85-e1bd-440b-8817-95ec234884f3-c000.gz.parquet")

# table = db.create_table("cases", data)
table = db.open_table("cases")
print(table)

# # Load the dataset
# Add a full-text search index on the 'opinion_text' field
table.create_fts_index(["opinions.opinion_text"], replace=True)

Data is from this dataset (has data dictionary)
Traceback (most recent call last):
  File "/workspace/finch/rag/.pixi/envs/default/lib/python3.12/site-packages/lancedb/fts.py", line 107, in populate_index
    f = table.schema.field(name)  # raises KeyError if not found
        ^^^^^^^^^^^^^^^^^^^^^^^^
  File "pyarrow/types.pxi", line 2792, in pyarrow.lib.Schema.field
KeyError: 'Column opinions.opinion_text does not exist in schema'

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "/workspace/finch/rag/index.py", line 16, in <module>
    table.create_fts_index(["opinions.opinion_text"], replace=True)
  File "/workspace/finch/rag/.pixi/envs/default/lib/python3.12/site-packages/lancedb/table.py", line 1420, in create_fts_index
    populate_index(
  File "/workspace/finch/rag/.pixi/envs/default/lib/python3.12/site-packages/lancedb/fts.py", line 109, in populate_index
    f = resolve_path(table.schema, name)
        ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/workspace/finch/rag/.pixi/envs/default/lib/python3.12/site-packages/lancedb/fts.py", line 169, in resolve_path
    raise KeyError(f"field {field_name} not found in schema {schema}")
KeyError: 'field opinions.opinion_text not found in schema id: int32\njudges: string\ndate_filed: date32[day]\ndate_filed_is_approximate: bool not null\nslug: string\ncase_name_short: string\ncase_name: string\ncase_name_full: string\nattorneys: string\nnature_of_suit: string\nposture: string\nsyllabus: string\nheadnotes: string\nsummary: string\ndisposition: string\nhistory: string\nother_dates: string\ncross_reference: string\ncorrection: string\ncitation_count: int32\nprecedential_status: string\narguments: string\nheadmatter: string\ncitations: list<element: string not null>\n  child 0, element: string not null\nopinions: list<element: struct<author_str: string, per_curiam: bool not null, type: string, page_count: int32, download_url: string, author_id: int32, opinion_text: string, ocr: bool not null, opinion_id: int32> not null>\n  child 0, element: struct<author_str: string, per_curiam: bool not null, type: string, page_count: int32, download_url: string, author_id: int32, opinion_text: string, ocr: bool not null, opinion_id: int32> not null\n      child 0, author_str: string\n      child 1, per_curiam: bool not null\n      child 2, type: string\n      child 3, page_count: int32\n      child 4, download_url: string\n      child 5, author_id: int32\n      child 6, opinion_text: string\n      child 7, ocr: bool not null\n      child 8, opinion_id: int32\ncourt_short_name: string\ncourt_full_name: string\ncourt_type: string\ncourt_jurisdiction: string\n-- schema metadata --\norg.apache.spark.version: \'3.4.0\'\norg.apache.spark.sql.parquet.row.metadata: \'{"type":"struct","fields":[{"\' + 2737'

That's odd. We were supposed to have fixed this in https://github.com/lancedb/lancedb/pull/723, but it appears we may have had a regression. Also see discussion in https://github.com/lancedb/lance/issues/1739
We have a test here for this:
https://github.com/lancedb/lancedb/blob/9708d829a991cefcee0e8354187cbf5681ce0897/python/python/tests/test_fts.py#L380-L383
I wonder what is different? use_tantivy still defaults to true, so I don't think it's that:
https://github.com/lancedb/lancedb/blob/9708d829a991cefcee0e8354187cbf5681ce0897/python/python/lancedb/table.py#L502

Oh, here is the field for opinions:
opinions: list<element: struct<author_str: string, per_curiam: bool not null, type: string, page_count: int32, download_url: string, author_id: int32, opinion_text: string, ocr: bool not null, opinion_id: int32> not null>

Opinions is a list, not a struct. We don't currently support list columns in FTS.


PostCode table procedure LookupPostCode parameters length

Why are the parameters of procedure LookupPostCode(var CityTxt: Text; var PostCode: Code[20]; var CountyTxt: Text; var CountryCode: Code[10]) on table 225 "Post Code" (unlimited) text? And not the actual length of the fields?
Further in the procedure these parameters are assigned
            PostCode := PostCodeRec.Code;
            CityTxt := PostCodeRec.City;
            CountryCode := PostCodeRec."Country/Region Code";
            CountyTxt := PostCodeRec.County;

where
field(2; City; Text[30])
field(5; County; Text[30])

Which means CityTxt and CountyTxt can never be more than the original field length of 30.
Calling this function with own fields (of the correct size), results in warning AA0139:
PostCode.LookupPostCode("From City", "From Post Code", "From County", "From Country/Region Code");
Possible overflow assigning 'Text' to 'Text[30]'.AL(AA0139)

Checked in Microsoft_Base Application_18.4.28601.29407
Edit:
procedure ValidateCity(var CityTxt: Text[30]; var PostCode: Code[20]; var CountyTxt: Text[30]; var CountryCode: Code[10]; UseDialog: Boolean) uses the correct lengths

The functions with long Text parameters for City and County was added 3 years ago and at that time we got some warnings saying that this function is called with parameters longer than 30 chars. The function got Text params.
Now as it is a public function we cannot simply change Text to Text[30] as it may break some partner's code.
The only solution is to make another function with Text[30] params and replace all or most calls of old functions. But the only benefit would be a reduced number of false warnings, and they are false because don't lead to an actual overflow unless someone overwrites the standard code using the event OnBeforeLookupPostCode().
The risk of breaking someone and the small benefit do not worth such a change like replacing all calls of LookupPostCode() and CheckClearPostCodeCityCounty().

Understandable, thank you for the explanation. :)


Vulnerability: Silent fallback to default JWT Key

https://github.com/CyferShepard/Jellystat/blob/d9d5cad5d42a1165bd8e8318f13d541892a1ea79/backend/auth.js#L6
The API should not silently fall back to a hardcoded JWT token. This may lead to vulnerable instances.

Change has been made to throw an error instead of silently falling back to a default value


[WIP] Implement fast_rms_norm as custom operators.

PR types

Performance optimization
PR changes

Others
Description

基于fast_ln，支持了fast_rms_norm。
性能影响（8x80G-A100）：

模型
训练策略
develop吞吐
PR优化后

GPT-3 13B
gbs32, tp2sharding4-mbs1-acc8
2551.522
2550.284

Llama-2 7B
gbs8, sharding8-mbs1-acc1
4454.693
4490.384

Llama-2 13B
gbs8, pp4sharding2-vpp5-mbs1-acc4
2229.921
2252.541

精度影响：

 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.You have signed the CLA already but the status is still pending? Let us recheck it.


Added _read_mp4box_stem_titles function

Basically, for stems which have been created by the NI creator have the titles of the streams in the udta metadata.
I added the functionality to try to read the stream titles if MP4Box is available, else fall back to original default numbered streams.
This is useful if you want to extract wav files using the stem2files, and instead of getting filenames like "stem_0.wav", you can get "Drums.wav" or "Vox.wav" if this information is available in the stem files.
The code is taken almost verbatim from the StemMetadataViewer class in https://github.com/axeldelafosse/stemgen/blob/master/ni-stem/_internal.py

I added sudo apt-get update to fix the previous issue of 404 errors while installing apt-get packages.
But still, the CI failed with some errors in FFmpeg unable to find x264 libraries.
ffprobe: error while loading shared libraries: libx264.so.138: cannot open shared object file: No such file or directory

@faroit sorry for the ping -- can you have a look when you have some free time please? 🙏