Traits link problem Presenter::SILENT_LINK

Hello,
$presenter->handleInvalidLink(...) can return also symbol '#' (Presenter::INVALID_LINK_SILENT)
https://github.com/ublaboo/datagrid/blob/1205cfe64469a0ae476a226bd671281952273baa/src/Traits/TLink.php#L56
so probably there should be aswell a condition strpos($link, '#') === 0

of course

It has been fixed already :) ... in march :)


Kubectl command reference middle column tables overflow into third column on narrow displays

This is a Bug Report
Problem:
When viewing kubectl reference docs page on a narrow display, middle column flag tables overflow into the third column.
Proposed Solution:
Fix CSS for narrow displays so that tables do not exceed the middle column width
Page to Update:
https://kubernetes.io/docs/reference/generated/kubectl/kubectl-commands

That's a valid issue. Thanks @brianpursley
/kind bug
/priority backlog

/assign

This was fixed in the 1.18 release


[BUG - MISSION CRITICAL] Website Downtime

DEATAILS
The website unexpectedly stopped working after doing some tests and is now completely unfunctional, displaying this error:
Application error: a client-side exception has occurred (see the browser console for more information).
Looking at the browser console, this error is displayed:
290-56fb51b50a0dceac.js:1  TypeError: Cannot read properties of null (reading 'default')
    at isStaticRequire (413-f8f38b98cb9340e9.js:1:4312)
    at getImgProps (413-f8f38b98cb9340e9.js:1:5557)
    at 413-f8f38b98cb9340e9.js:1:2998
    at Hf (bf6a786c-78f4de11ea8ef20a.js:9:39239)
    at uh (bf6a786c-78f4de11ea8ef20a.js:9:57055)
    at rx (bf6a786c-78f4de11ea8ef20a.js:9:116945)
    at Lk (bf6a786c-78f4de11ea8ef20a.js:9:94304)
    at bf6a786c-78f4de11ea8ef20a.js:9:94126
    at bf (bf6a786c-78f4de11ea8ef20a.js:9:94133)
    at cf (bf6a786c-78f4de11ea8ef20a.js:9:91778)

Running this locally in a dev environment displayed the same issue.
SCREENSHOTS
(the deployed version)

(the one running in the dev environment)

UPDATE
I have found the issue - When I was testing, I created a community and then updated its details (name and URL slug) and apparently when it saved those changes to the database, it made the "image" field for the community (the profile picture) "null".

The issue has been resolved by fixing the field in the database.


updated MUC roster style

I've updated MUC roster a little bit for more consistence of the whole site view.

Looks nice, thanks!


fix: language will be escaped when use concatjs

Thanks for your PR. Will be released after my Christmas break.


Comparison with Norm a bit out of date

I think this comparison between TypeCheck and Norm is a bit out of date... https://github.com/Qqwy/elixir-type_check/blob/master/Comparing TypeCheck and Norm.md
For instance, it looks like it's referring to spec rather than @spec!.
I had someone ask me the difference between them today and I was going to link to this, so noticed it. It's a good comparison to have.

Thank you very much for pointing that out!
I have updated the guide's code snippets and also in some places content, as some new TypeCheck features have since been introduced (more support for Elixir's builtin types, typespecs).
The updated guide can be read here on GitHub, but also as part of TypeCheck's documentation pages.


[release-1.8] Upgrade to latest dependencies

bumping 1.8 -dprotaso
/cc knative-sandbox/eventing-writers
/assign knative-sandbox/eventing-writers
Produced by: knative-sandbox/knobots/actions/update-deps

/retest-required

/retest-required

/test channel-integration-tests-sasl-plain


NBS format documentation

I may be wrong but it appears the "documentation" on the website(https://hielkeminecraft.github.io/OpenNoteBlockStudio/nbs) isn't too hot.
For example, theses are new to the new NBS format, at least according to this, https://www.stuffbydavid.com/mcnbs/format

Also this

But they are not marked in green, luckily I was able to figure what I was doing wrong by comparing from the stuffbydavid.com/mcnbs/format link.
It would just be cool if it could be marked in green and or show what NBS version the data was added in, as "Loop on/off" which shows green was added in version 4 if I'm not mistaken.

Ok well that's a bit weird, I was just looking around in the HTML code, I do see you marked Vanilla Instrument count as new version
 but it doesn't show green, disabling background color fixes this
 but messed up the rest of the tables' colors.
But
 are still not marked new version.

Yeah, I had noticed in the past that the alternating colors in the table override the green color indicating a new feature. I've been looking for a way to fix that in the CSS, but haven't managed to do it yet. I'll have another look at it soon, thanks!

Just saying "marking" all td "attributes" with newversion fixes the issue. I think its kind of a sloppy fix but it does work.


Unable to destroy activity SurveyActivity: NullPointerException

compile "com.mixpanel.android:mixpanel-android:4.9.2"
Stacktrace from Fabric. We can't reproduce it, it's user crash (have a lot of users with this crash)
Fatal Exception: java.lang.RuntimeException: Unable to destroy activity {com.myapp/com.mixpanel.android.surveys.SurveyActivity}: java.lang.NullPointerException: Attempt to invoke interface method 'void com.mixpanel.android.mpmetrics.i$c.b(java.lang.String, java.lang.Object)' on a null object reference
       at android.app.ActivityThread.performDestroyActivity(ActivityThread.java:3877)
       at android.app.ActivityThread.handleDestroyActivity(ActivityThread.java:3895)
       at android.app.ActivityThread.-wrap5(ActivityThread.java)
       at android.app.ActivityThread$H.handleMessage(ActivityThread.java:1417)
       at android.os.Handler.dispatchMessage(Handler.java:102)
       at android.os.Looper.loop(Looper.java:148)
       at android.app.ActivityThread.main(ActivityThread.java:5471)
       at java.lang.reflect.Method.invoke(Method.java)
       at com.android.internal.os.ZygoteInit$MethodAndArgsCaller.run(ZygoteInit.java:726)
       at com.android.internal.os.ZygoteInit.main(ZygoteInit.java:616)
Caused by java.lang.NullPointerException: Attempt to invoke interface method 'void com.mixpanel.android.mpmetrics.i$c.b(java.lang.String, java.lang.Object)' on a null object reference
       at com.mixpanel.android.surveys.SurveyActivity.onDestroySurvey(SurveyActivity.java:4303)
       at android.app.Activity.performDestroy(Activity.java:7296)
       at android.app.Instrumentation.callActivityOnDestroy(Instrumentation.java:1143)
       at android.app.ActivityThread.performDestroyActivity(ActivityThread.java:3864)
       at android.app.ActivityThread.handleDestroyActivity(ActivityThread.java:3895)
       at android.app.ActivityThread.-wrap5(ActivityThread.java)
       at android.app.ActivityThread$H.handleMessage(ActivityThread.java:1417)
       at android.os.Handler.dispatchMessage(Handler.java:102)
       at android.os.Looper.loop(Looper.java:148)
       at android.app.ActivityThread.main(ActivityThread.java:5471)
       at java.lang.reflect.Method.invoke(Method.java)
       at com.android.internal.os.ZygoteInit$MethodAndArgsCaller.run(ZygoteInit.java:726)
       at com.android.internal.os.ZygoteInit.main(ZygoteInit.java:616)

Hi @mtrakal! Thanks for reporting this. Are you unidentifying the user at some point (i.e. log out will  reset data, etc)? It may be possible that the survey is shown after you unidentified the user. This will help us to implement a proper fix. Thanks!

Hi @patedit,
user can logout yourself or delete user data (same as logout). It's possible, that's after logout. I have no idea what user do, but hour later after crash we track that user sign in.
Sorry, that I have no more info, but it's just from Fabric...

Hi @patedit , i have the exact same issue with our android app . This happens while the user is logged in and other mixpanel API calls work fine.

@mtrakal I opened a PR that will fix your issue. Will be available soon. Thanks for your feedback!

Guys, this should be fixed now on the latest release. Thanks for your patience!


DHT Reproviding

https://github.com/lazyledger/lazyledger-core/pull/375 disables providing in Bitswap here. Unfortunately, this options also disables reproviding.
Reproviding is basically re-execution of providing. Another important conceptual part of DHT is that its entries are not persistent and cleaned up through time, thus there should be some logic that automatically renews entries on DHT.
As https://github.com/lazyledger/lazyledger-core/pull/375 disables it automatically we need to enable it manually. Luckily though, we can implement our own strategy for reproviding which only reprovides roots and not all the cids in blockstore, as IPFS excessively do.
Importantly, this is required for the network to work properly, otherwise recently proposed blocks won't be available after ~12 hours. Thus, this needs to be part of #381

This is related to #394 and the story around providing on received data and regular re-providing should be considered together.

Moving this issue over to the celestia-node as it might still be interesting to have it there for context.


Add config['local_domain'] for smtp transport (default is [<IP_ADDRESS>])

Without this, connecting to google smtp relay server fails with SMTP code 421 after the HELO command.

@mjauvin could you add a comment giving more background information?

@LukeTowers You mean a comment in the source code itself?

Yes. Also what tests are failing? Can @jaxwilko or @bennothommo take a look at them?

@mjauvin and what's the benefit of this change over just telling users to set the local_domain property correctly?

@LukeTowers Do you mean users setting this in their config/mail.php file ?

Yes

@LukeTowers I find it's a sensible default for the Storm library in general, many smtp server will fail with the current symfony default.

Could you explain what exactly it's doing? I've never used any of those functions

In the SMTP protocol, the HELO command is the first greeting from the client and it announces where it's connecting from. The SMTP server will usually (but not always) use this information to verify that this hostname matches the reverse lookup from the IP it connected from.
My code is setting the local_domain mail config option. This is querried later by the EsmtpTransportFactory like this:
(see symfony/mailer/Transport/Smtp/EsmtpTransportFactory.php)
        if (null !== ($localDomain = $dsn->getOption('local_domain'))) {
            $transport->setLocalDomain($localDomain);
        }

And is used in the EHLO command here:
(see symfony/mailer/Transport/Smtp/EsmtpTransport.php)
    protected function doHeloCommand(): void
    {
        try {
            $response = $this->executeCommand(sprintf("EHLO %s\r\n", $this->getLocalDomain()), [250]);
        } catch (TransportExceptionInterface $e) {
            parent::doHeloCommand();

            return;
        }
        ...

And for the HELO command here:
(see symfony/mailer/Transport/Smtp/SmtpTransport.php
    protected function doHeloCommand(): void
    {
        $this->executeCommand(sprintf("HELO %s\r\n", $this->domain), [250]);
    }

Those tests are failing @LukeTowers:
1) QueryBuilderTest::testUpsert
Error: Call to undefined method Mock_ConnectionInterface_f1a1bb84::getConfig()

/home/runner/work/storm/storm/vendor/laravel/framework/src/Illuminate/Database/Query/Grammars/MySqlGrammar.php:203
/home/runner/work/storm/storm/src/Database/QueryBuilder.php:347
/home/runner/work/storm/storm/tests/Database/QueryBuilderTest.php:106

2) QueryBuilderTest::testUpsertWithUpdateColumns
Error: Call to undefined method Mock_ConnectionInterface_f1a1bb84::getConfig()

/home/runner/work/storm/storm/vendor/laravel/framework/src/Illuminate/Database/Query/Grammars/MySqlGrammar.php:203
/home/runner/work/storm/storm/src/Database/QueryBuilder.php:[34](https://github.com/wintercms/storm/runs/6263996814?check_suite_focus=true#step:10:34)7
/home/runner/work/storm/storm/tests/Database/QueryBuilderTest.php:149

This is not viable, better to set directly in smtp driver config if needed.


[Server] Fix manifests separated using yaml-separator being not parsed correctly

Notes for Reviewers
This PR fixes #
Signed commits

[ ] Yes, I signed my commits.

@Althaf66 are you fixing these mesheryctl unit tests?


Allow Condition for VM extension

Hi, I cannot use extension on VM ARM template. I need to make condition to add or not add VM extension. Getting this error: Could not find member 'condition' on object of type 'VMScaleSetVMExtension'. Path 'properties.virtualMachineProfile.extensionProfile.extensions[1].condition'

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

ID: 239d3fb9-5c01-fb3d-f587-86ef2622c4e3
Version Independent ID: b23932c8-38aa-b992-657e-3b6e66e691c1
Content: Use condition in Azure Resource Manager templates
Content Source: articles/azure-resource-manager/resource-manager-tutorial-use-conditions.md
Service: azure-resource-manager
GitHub Login: @mumian
Microsoft Alias: jgao

@jjindrich - condition can only be used at the level of the resource. I see that VMSS has an object called extensionProfile with an array called extensions. Are you trying to apply condition at that level? Or, are you using the virtualMachineScaleSets/extensions as a separate resource type?

I'm using extensionProfile and I need to configure extension is deployed based on some parameter. Like OmsAgent=Yes to deploy OmsAgent extension.

Condition can only be applied at the root level for the resource, not on properties within the resource. It determines whether the resource is deployed. You could try using the extensions resource type under virtualMachineScaleSets, and apply the condition to that resource.

@jjindrich - we'll close this issue, but feel free to reopen if you have more questions.

#please-close


Subqueries using let: $current and $parent not found

OrientDB Version: 3.0.2
Java Version: 8
OS: Mac OS
Expected behavior
When executing the command:
select $foo from Bar
let
$foo = (select expand(out('foo-bar')) from $parent.current)
Returns Foo vertices.
Actual behavior
Exception:
com.orientechnologies.orient.core.exception.OCommandExecutionException: Class $parent not found DB name=
Sidenote:
In Orientdb 2.2.x the below used to work
select $foo from Bar
let
$foo = (select expand(out('foo-bar')) from $current)
But in v3.0.2 this also throws exception:
com.orientechnologies.orient.core.exception.OCommandExecutionException: Class $current not found DB name=

Hi @userg
Please try the following:
select $foo from Bar
let
$foo = (select expand($parent.$current.out('foo-bar')))

In v 3.0 $xxx indentifiers are just standard identifiers, so they can also be class names. OrientDB just assumes this is the case, this is why it doesn't work.
Thanks
Luigi

Hi @luigidellaquila
Thank you very much for your response.
But the above query returns empty results,
but when I do
select expand(out('foo-bar')) from Bar
I get results.

Hi @userg
It seems to be a bug, I'm checking it now
Thanks
Luigi

Thank you !

Hi @userg
I think I have a fix for this, I'm pushing it now
Thanks
Luigi

Hi @userg
I just pushed a fix, it will be released with v 3.0.4
Thanks
Luigi

Hi @luigidellaquila
Thanks for the quick response. When can we expect 3.0.4 release?

Hi @userg
I'd say in a very short term, 24-48h hopefully
Thanks
Luigi

Hi @luigidellaquila ,
In the above scenario, how do we order the results of the sub query?
For example,
select $foo from Bar let $foo = (select $parent.$current.out('foo-bar') as d order by d.age)
throws an error saying
Error parsing query: select select $foo from Bar let $foo = (select $parent.$current.out('foo-bar') as d order by d.age)  ^ Encountered " <ORDER> "order "" at line 1, column 226. Was expecting one of: <LIMIT> ... <SKIP2> ... <OFFSET> ... <TIMEOUT> ... <FETCHPLAN> ... <LOCK> ... <LET> ... <NOCACHE> ... <PARALLEL> ... <UNWIND> ... ")...
Is there something we are missing?
Also is there an updated documentation which shows us how to use these variables in child context? Because we are not able to use $parent variable in from clause. Is this by design?

Hi @userg
A SELECT <expression> statement returns only one element, you just cannot sort it.
You could sort the collection contained in your d, but you have to do it with a custom function (eg. a js function)
Thanks
Luigi

Thanks for the response. One issue is that we are having to use select <expression> in the sub-query because this query:
SELECT FROM Document LET $temp = ( SELECT @rid, $depth FROM (TRAVERSE            V.OUT, E.IN FROM $parent.current )
is not working. The $parent is somehow not available in the FROM expression (We get an error saying com.orientechnologies.orient.core.exception.OCommandExecutionException: Class $parent not found DB name=...). This example query is copied from this documentation:
https://orientdb.com/docs/last/sql/SQL-Query.html#sub-query
Has this behaviour changed?
To be more accurate, our query initially looked something like this:
SELECT $foo FROM Bar LET $foo = (SELECT expand(out('foo-bar')) FROM $parent.current)

Hi @RamAnvesh
Regarding this query
SELECT FROM Document LET $temp = ( SELECT @rid, $depth FROM (TRAVERSE V.OUT, E.IN FROM $parent.current )

Does the Parent refer to the Document in the main SELECT? In this case, it is two levels above, so you have to use $parent.$parent.$current
Thanks
Luigi

Hi @luigidellaquila,
The query was whether $parent / $current is supported in the from clause in 3.0.
When we try the following query we are getting the exception:
"com.orientechnologies.orient.core.exception.OCommandExecutionException: Class $parent not found DB name="
select $foo from Bar let $foo = (select expand(out('foo-bar')) from $parent.current)
Though the workaround
select $foo from Bar let $foo = (select expand($parent.$current.out('foo-bar'))) works, wanted to check why the previous statement won't work or if we are missing something

Hi @userg
Sorry, you are right, I lost a bit the train of thought...
The problem here is that in v 3.0 the SQL executor is more strict and expects a legal target in the FROM clause (eg. a class name, a cluster:number, or an index:name).
There is a chance that in the near future we will make it a bit more flexible, but for now you have to stick to the expand() work-around
Thanks
Luigi

Hi @luigidellaquila
I am experiencing a similar problem when I use following query:
SELECT $columns FROM CI LET $columns = (SELECT * FROM (TRAVERSE out() FROM $parent.$parent.$current) WHERE othercolumn = 5)
I receive this error:
com.orientechnologies.orient.core.exception.OCommandExecutionException: Class $parent not found DB name="databases"
I am using OrientDB Version: 3.0.4

@niks368 looks like you cannot use parent and current variables in the FROM clause in OrientDB 3.0

SELECT $b FROM a
LET $b = (SELECT expand($parent.$current.out('foo-bar')))
WHERE id = 'some value'
always returns an empty value even though
SELECT FROM a where id = 'some value' has an OUT "foo-bar" edge
Using OrientDB 3.0.4

@srini6teen is your 'id' indexed? If so you have to use $parent.$current.rid.out. I've raised an issue for the same here https://github.com/orientechnologies/orientdb/issues/8412

@RamAnvesh
Since the parent and current variables don't work in OrientDB 3.0 and the select expand(...) work around only expands one level deep, how can this be done?

Hi @niks368
Actually you can concatenate multiple $parent in the same path, as $parent.$parent.$current, could it fit your use case?
Thanks
Luigi

Hi @luigidellaquila
Thank you for your answer!
In my case I don't know the depth of the traversal, which is why i cannot use select.
When I change the traverse to the select expand(...)  I get the following query:
SELECT $columns FROM CI LET $columns = (SELECT expand($parent.$current.out()))
Unfortunately this query only expand one level deep.
Is there a way to do this with traversal?

Hi @niks368
Did you try the following:
SELECT $columns FROM CI LET $columns = (
  SELECT * FROM (
    TRAVERSE out() FROM (
       SELECT expand($parent.$parent.$parent.$current)
    )
  ) WHERE othercolumn = 5
)

Thanks
Luigi

Hi @luigidellaquila
Unfortunetely this doesn't work. I get an empty array returned.
When I try this query i get the columns, so it should be found by the traverse.
SELECT $columns FROM CI LET $columns = (        SELECT expand($parent.$current.out()) )
Thanks
Niko


Roadmap v0.3

Hi @ilyapt. I finished the roadmap from #5 and created version 0.2.0.
Please for your review and critical comments.
There is next roadmap for version 0.3.0

[x] add changelog
[ ] add eslint https://eslint.org/docs/user-guide/getting-started
[ ] remove ts-ignore from source code
[ ] create documentation of interfaces for organizations and private profiles
[ ] decide if we want to use puppeteer and proxies inside of this package #8, #2
[ ] create video tutorial introducing for new users #3
[ ] autosave of HTML pages controlled by flags
[ ] debug flag for verbose mode
[ ] documentation of interfaces

I think the most important discussion is on issue #8 because I propose approach:

slim, solid, well tested and documented package

but we have also pull request from @mihailShumilov that proposes:

ready for use with a proxy from the console, automatically scrolling

UPDATE:
In the case of Pull request: Add ability to parse public group pages I think we can use both approaches thanks to the concept of drivers selected by end-user. It is described in issue #8

EsLint rules:
✔ How would you like to use ESLint? · style
✔ What type of modules does your project use? · esm
✔ Which framework does your project use? · none
✔ Does your project use TypeScript? · No / Yes
✔ Where does your code run? · node
✔ How would you like to define a style for your project? · prompt
✔ What format do you want your config file to be in?  · No / Yes


Add PowerShell 7 support

Is your feature request related to a problem? Please describe
As Get-ItemProperty output is different → parsing fails

Number of lines
Number of digits

Windows PowerShell 5.1
> $PSVersionTable

Name                           Value
----                           -----
PSVersion                      5.1.22621.2506
PSEdition                      Desktop

> Get-ItemProperty -Path C:\Windows
0|
1|
2|    Répertoire : C:\
3|
4|
5|Mode                 LastWriteTime         Length Name
6|----                 -------------         ------ ----
7|d-----        29/12/2023   6:01 PM                Windows
  < 6  >

PowerShell 7
> $PSVersionTable

Name                           Value
----                           -----
PSVersion                      7.3.10
PSEdition                      Core

> Get-ItemProperty -Path C:\Windows
0|
1|    Directory: C:\
2|
3|Mode                 LastWriteTime         Length Name
4|----                 -------------         ------ ----
5|d----          29/12/2023  6:01 PM                Windows
  < 5 >

In addition the output is colored:
b'd----          29/12/2023  6:01 PM                \x1b[44;1mWindows\x1b[0m'

Describe the solution you'd like
facts/windows_files.py → class File(FactBase): → process method

from  def process(self, output):
      if len(output) < 7:
          return None

      # Note: The first 7 lines are header lines
      return parse_win_ls_output(output[7], self.type)

to  def process(self, output):
      if len(output) < 5:
          return None

      # Note: The first lines are header lines
      return parse_win_ls_output(output[-2], self.type)

facts/util/win_files.py →  WIN_LS_REGEX definition, relax the filetype and mode pattern

from        # filetype and mode
        r"^([darhsl\-]{6})\s+"

to        # filetype and mode
        r"^([darhsl\-]{5,6})\s+"

facts/util/win_files.py →  parse_win_ls_output function, remove ansi sequences

from  if output:
      matches = re.match(WIN_LS_REGEX, output)

to  if output:

      # Patch to remove ansi sequences from PowerShell 7
      # @see https://stackoverflow.com/questions/14693701/how-can-i-remove-the-ansi-escape-sequences-from-a-string-in-python
      ansi_escape = re.compile(r"(\x9B|\x1B\[)[0-?]*[ -/]*[@-~]", flags=re.IGNORECASE)
      output = ansi_escape.sub('', output)

      matches = re.match(WIN_LS_REGEX, output)

I think a much more future-proof, clean and safe (with respect to various kinds of injections when running commands on untrusted hosts) way is to use a proper serialization (JSON/XML) for transport. It is even much easier to handle the results on pyinfra's side while simply switching from Get-ItemProperty -Path C:\Windows to Get-ItemProperty -Path C:\Windows | ConvertTo-Json and possibly requesting only a subset of fields for efficiency.


question about your training code: where is 'network' variable being used for training?

Hello, I'm currently delving into the training code to tailor it for my specific needs. However, I'm puzzled about the integration of the LoRA slider into the main UNet. I noticed that the LoRA adapter is established within the 'network' variable, yet it seems to be utilized solely as a context manager in this section of the code while the training process still calls original unet
https://github.com/rohitgandikota/sliders/blob/main/trainscripts/textsliders/train_lora_xl.py#L205C1-L227C18
Is it some complex Python syntax that I'm not yet familiar with?

It is in lora.py. We wrote it such that during training, we easily load and unload the Loras. If you see LoRANetwork function in lora.py, you will see that the context manager simply turns the scale to 0 when exited.

Thank you for your response. I've looked through the lora.py file and noticed the enter and exit functions. My confusion lies in the usage of self.unet_loras. After its declaration, I don't see it being actively used. The enter function modifies parameters within unet_loras, but it's not clear to me how unet_loras is involved in the train_util.diffusion_xl() process. Initially, I thought there was an in-place modification of the unet passed into the class constructor. However, I noticed that at the end of the init function, the unet object is deleted. Could you clarify how unet_loras is being called during training?

The LoRANetwork class method stores the lora modules. In the method we delete unet, but that does not impact the actual UNet we load in the training. We are just making the lora network class light. the unet_loras object is being used to collect the layers where we wish to add Loras to. We do not use it in training. It is just a helper variable in the class to create Loras.

I want to clarify my confusion. In the training script, it seems that the unet object is consistently the original one from UNet2DConditionModel. In the LoRAModule and LoRANetwork classes, the intention appears to be modifying specific layers and the forward functions of unet, but this modification doesn't seem to propagate back to the unet except the with  statement. So my question is if the with statement is doing something else than setting up the multiplier value to share information with unet.
Additionally, during inference, the network also loads its weights and interacts with unet solely via the with statement, so I guess this with statement has something pivotal here? I have to understand how it works before making edit on the codes.
                with network:                     noise_pred = unet(                         latent_model_input,                         t,                         encoder_hidden_states=prompt_embeds,                         cross_attention_kwargs=cross_attention_kwargs,                         added_cond_kwargs=added_cond_kwargs,                         return_dict=False,                     )[0]

This is from lora.py - as you can see the context entry is based on setting multiplier to non-zero scale and exiting will just set the scale to 0. Since we are loading Loras, those are just additional layers we load on top of pretrained unet. So in practice - W_new = W_old + alpha * W_lora. When LoRANetwork is called, new lora modules are created and kept on top of UNET. When you do with network, the Loras are activated and added to the pretrained model.
def __enter__(self):
        for lora in self.unet_loras:
            lora.multiplier = 1.0 * self.lora_scale

    def __exit__(self, exc_type, exc_value, tb):
        for lora in self.unet_loras:
            lora.multiplier = 0

Thanks. Actually this single line is what I am looking for, not the enter and exit functions. Without this line, I wouldn't know how the lora adapter is attached to the old unet during training.
self.org_module.forward = self.forward


Documentation - requests for clarification

Hi!
Thanks for django-simple-history! I decided to plug it in to my project a couple of months ago and recently had another requirement to do raw SQL queries or handle inter-project dependencies some other way, which prompted me to read the docs more carefully. There are a couple items that I'm still not clear about:

A) What's the relationship between *historical$model and *$model tables? In other words: B) is the latest version stored in *$model, *historical$model, or both?
The documentation is missing example use cases for many of the features. One that I most care about is "save without a historical record". A) What happens if the tables go out of sync? B) Does anything prevent that if data was modified externally, are there any automatic health recovery procedures? C) Are there any failure modes I should be aware of? Also: D) is there something new I need to be mindful of when doing raw SQL inserts/updates/selects?
Why do some people prefer UUID as history_id?

Hi @d33tah! I'll go through and answer each question:

The latest version is stored in both tables. The base table knows nothing about the historical table, and the historical table inserts a new row with complete model instance data every time an instance is created, updated, or deleted.
A) Nothing happens if the tables go out of sync. The historical table is super naive. B) Nothing to prevent data form being modified externally, although we provide a hook that would allow you to take a historical record and write it elsewhere, maybe to an immutable db. C) No failure modes. D) DSH works on a post_save/post_delete hook, meaning that every time a model instance is saved/deleted a signal is sent that DSH hooks onto, and creates a new record in the historical table. Since raw SQL edits the db directly, no signals are sent (DSH is at the application level rather than the db level). Thus, to maintain the history table while doing raw sql inserts/updates, you'll need to also do raw sql inserts on the historical table.
https://tomharrisonjr.com/uuid-or-guid-as-primary-keys-be-careful-7b2aa3dcb439

Thanks @rossmechanic! :) This seems to answer my questions exhaustively.


Simplifying code part 2

I have simplified the code by adding comments for greater understandability and replacing pieces of code as well as adding exception statements for proper execution of the test.
Description
I have simplified the code by adding comments, replacing pieces of code with more appropriate code content and adding more exception statements for proper execution of the test.
GUI-related pull requests should be opened against
https://github.com/BGL-core/gui
first. See CONTRIBUTING.md
-->
Notes
Please run the test to verify that there are no errors in the code.
BTC/BGL PR reward address
1MqWcL1BtzJS4phxrxLFdooN911mBqQW1q

Please check this PR. This is my second PR. I am still waiting for rewards for my first PR. Please check that PR too.

Please check this PR. This is my second PR. I am still waiting for rewards for my first PR. Please check that PR too.

What have your really achieved with this PR?

I have simplified the code by adding comments for greater understandability
and replacing long pieces of code with short and efficient code for greater
efficiency and faster executability.
On Thu, 25 Nov, 2021, 2:49 am Emeka, @.***> wrote:

Please check this PR. This is my second PR. I am still waiting for rewards
for my first PR. Please check that PR too.
What have your really achieved with this PR?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/BitgesellOfficial/bitgesell/pull/68#issuecomment-978230175,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AUULJTJPKXQC3HPDTXIWOC3UNVJFFANCNFSM5IVONQVQ
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

I have simplified the code by adding comments for greater understandability and replacing long pieces of code with short and efficient code for greater efficiency and faster executability.
…
On Thu, 25 Nov, 2021, 2:49 am Emeka, @.***> wrote: Please check this PR. This is my second PR. I am still waiting for rewards for my first PR. Please check that PR too. What have your really achieved with this PR? — You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub <#68 (comment)>, or unsubscribe https://github.com/notifications/unsubscribe-auth/AUULJTJPKXQC3HPDTXIWOC3UNVJFFANCNFSM5IVONQVQ . Triage notifications on the go with GitHub Mobile for iOS https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Android https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

Which command did you use to test your code?

I didn't test it yet I am testing it in just 20 minutes and telling you
On Thu, 25 Nov, 2021, 11:31 am Emeka, @.***> wrote:

I have simplified the code by adding comments for greater
understandability and replacing long pieces of code with short and
efficient code for greater efficiency and faster executability.
… <#m_-6851550101760160897_>
On Thu, 25 Nov, 2021, 2:49 am Emeka, @.***> wrote: Please check this
PR. This is my second PR. I am still waiting for rewards for my first PR.
Please check that PR too. What have your really achieved with this PR? —
You are receiving this because you authored the thread. Reply to this email
directly, view it on GitHub <#68 (comment)
https://github.com/BitgesellOfficial/bitgesell/pull/68#issuecomment-978230175>,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AUULJTJPKXQC3HPDTXIWOC3UNVJFFANCNFSM5IVONQVQ
. Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub
.
Which command did you use to test your code?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/BitgesellOfficial/bitgesell/pull/68#issuecomment-978864936,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AUULJTJISH6HHPYNCUUVR3TUNXGKJANCNFSM5IVONQVQ
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

I didn't test it yet I am testing it in just 20 minutes and telling you
…
On Thu, 25 Nov, 2021, 11:31 am Emeka, @.> wrote: I have simplified the code by adding comments for greater understandability and replacing long pieces of code with short and efficient code for greater efficiency and faster executability. … <#m_-6851550101760160897_> On Thu, 25 Nov, 2021, 2:49 am Emeka, @.> wrote: Please check this PR. This is my second PR. I am still waiting for rewards for my first PR. Please check that PR too. What have your really achieved with this PR? — You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub <#68 (comment) <#68 (comment)>>, or unsubscribe https://github.com/notifications/unsubscribe-auth/AUULJTJPKXQC3HPDTXIWOC3UNVJFFANCNFSM5IVONQVQ . Triage notifications on the go with GitHub Mobile for iOS https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Android https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub . Which command did you use to test your code? — You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub <#68 (comment)>, or unsubscribe https://github.com/notifications/unsubscribe-auth/AUULJTJISH6HHPYNCUUVR3TUNXGKJANCNFSM5IVONQVQ . Triage notifications on the go with GitHub Mobile for iOS https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Android https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

I need your test command.
Paste the output of your test here.

While testing I found a great bug
./contrib/testgen/base58.py:110:12: F632 use ==/!= to compare constant literals (str, bytes, int, float, tuple)
assert get_bcaddress_version('15VjRaDX9zpbA8LVnbrCAFzrVzN7ixHNsC') is 0
^
./contrib/seeds/generate-seeds.py:49:51: F821 undefined name 's'
raise ValueError('Invalid onion %s' % s)
^
./contrib/seeds/makeseeds.py:159:14: E999 SyntaxError: Missing parentheses in call to 'print'. Did you mean print(...)?
print '[%s]:%i' % (ip['ip'], ip['port'])
^
1     E999 SyntaxError: Missing parentheses in call to 'print'. Did you mean print(...)?
1     F632 use ==/!= to compare constant literals (str, bytes, int, float, tuple)
1     F821 undefined name 's'
3
Error: Process completed with exit code 1.
Please solve this and tell whether I would be rewarded

Did I do a good job finding this bug (I suppose this is a bug, rest you shall tell and confirm)?

I have found bugs and wanted to know what's the economic value of this job
I have done. Since I have contributed to the organization, so I am little
bit curious about its value so please don't misunderstand me as a spammer
looking for passtime.
On Thu, 25 Nov, 2021, 11:57 am Emeka, @.***> wrote:

I didn't test it yet I am testing it in just 20 minutes and telling you
… <#m_-4779249822169263219_>
On Thu, 25 Nov, 2021, 11:31 am Emeka, @.> wrote: I have simplified the
code by adding comments for greater understandability and replacing long
pieces of code with short and efficient code for greater efficiency and
faster executability. … <#m_-6851550101760160897_> On Thu, 25 Nov, 2021,
2:49 am Emeka, @.> wrote: Please check this PR. This is my second PR. I
am still waiting for rewards for my first PR. Please check that PR too.
What have your really achieved with this PR? — You are receiving this
because you authored the thread. Reply to this email directly, view it on
GitHub <#68 https://github.com/BitgesellOfficial/bitgesell/pull/68
(comment) <#68 (comment)
https://github.com/BitgesellOfficial/bitgesell/pull/68#issuecomment-978230175>>,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AUULJTJPKXQC3HPDTXIWOC3UNVJFFANCNFSM5IVONQVQ
. Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub
. Which command did you use to test your code? — You are receiving this
because you authored the thread. Reply to this email directly, view it on
GitHub <#68 (comment)
https://github.com/BitgesellOfficial/bitgesell/pull/68#issuecomment-978864936>,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AUULJTJISH6HHPYNCUUVR3TUNXGKJANCNFSM5IVONQVQ
. Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub
.
I need your test command.
Paste the output of your test here.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/BitgesellOfficial/bitgesell/pull/68#issuecomment-978875725,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AUULJTMDWCXIIDG7AUM2EOTUNXJL3ANCNFSM5IVONQVQ
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.


Update example name into MultimodalQnA and update image names

Description
Update image names for embedding, dataprep, retriever, lvm microservices.
Update MultimodalQnAGateway
Issues
MultimodalRAG for v1.0 release
Type of change
List the type of change like below. Please delete options that are not relevant.

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds new functionality)
[ ] Breaking change (fix or feature that would break existing design and interface)
[x ] Others (enhancement, documentation, validation, etc.)

Dependencies
N/A
Tests
All tests predefined for embedding, dataprep, retriever, lvm microservices for multimodal.
Test for MultimodalQnAGateway

@chensuyue @lvliang-intel All tests are passed. Can you please review and merge this one? We need this one to update GenAIExamples. Thanks


QueryBuilder recreated in React

It's a little harder to comment and requires a little more work when you have an entire app at once. Better if it's more interactions and small changes. :)
I see that some things here are not necessary as this is already provided by the base application.
Since this is not yet in use by anyone and is basically a copy of the current query builder app to learn a little React, let's move ahead and fix everything when merging with the base app.


Can you provide a single JS file to convert a text into history books? TKS

Can you provide a single JS file to convert a text into history books?
Src / render.js in your project really didn't undstand how to do it.
Now I realize how to transform a text into an history book.
TKS

你能单独 提供一个把一段文字转成古代书籍的例子吗？我找这功能好久了，您项目中的src/render.js我没有看明白他是如何转换的。
我现在需要实现 把一段文字转化成古代书籍的样子。最好只用JS就可以，不用NODE.JS和PYTHON

Yes, that would be possible. However you'll need to make a few changes. Right now I'm busy working on the language, so I can only give you a couple of steps:

Since you do not want node.js, you need to copy paste all the files require()'ed by render.js into render.js
render.js give keywords different colors, which you probably don't want for generic usage. Therefore, change all the fields of BOOK_COLORS to BLACK

Use this line to compute the SVG as strings
var svgs = render("name of your book here", "copy paste all your text here")

You can then do your thing with the array of SVG's returned. e.g. insert them into HTML, console.log them and copy the result, etc.
If you need help please let me know, I'll be glad to do so when I have time!
Thanks.

your do work is beautifrul!! tks ,i try it now ,

Yes, that would be possible. However you'll need to make a few changes. Right now I'm busy working on the language, so I can only give you a couple of steps:

Since you do not want node.js, you need to copy paste all the files require()'ed by render.js into render.js
render.js give keywords different colors, which you probably don't want for generic usage. Therefore, change all the fields of BOOK_COLORS to BLACK

Use this line to compute the SVG as strings
var svgs = render("name of your book here", "copy paste all your text here")

You can then do your thing with the array of SVG's returned. e.g. insert them into HTML, console.log them and copy the result, etc.
If you need help please let me know, I'll be glad to do so when I have time!
Thanks.

I do it as you suggest,but happend error ,I haved write email to your lingdong0618@hotmail.com,Please guide in your idle time,very tks!!

@heangfat 你这个效果也不错，我已经看了，后面的JS竟然用繁体中文命名，着实服了一把，头一次见这样的

@heangfat I really liked the way you renders ChuCi from that image. Is that web site an open-source project? Can you share your rendering enging for that page? Would really like to start from your project in order to accomplish this: https://github.com/LingDong-/wenyan-lang/issues/321
Thank you in advance and looking forward to hearing from you soon.


jsx footguns and styling

Patch bump with the following ⚠ 's:

react/display-name: DisplayName allows you to name your component. This name is used by React in debugging messages.

react/no-danger: Dangerous properties in React are those whose behavior is known to be a common source of application vulnerabilities. The properties names clearly indicate they are dangerous and should be avoided unless great care is taken.

react/no-deprecated: Several methods are deprecated between React versions. This rule will warn you if you try to use a deprecated method.

react/no-direct-mutation-state: NEVER mutate this.state directly, as calling setState() afterwards may replace the mutation you made. Treat this.state as if it were immutable.

react/no-unknown-property: In JSX all DOM properties and attributes should be camelCased to be consistent with standard JavaScript style. This can be a possible source of error if you are used to writing plain HTML.

react/jsx-indent: This option validates a specific indentation style for JSX.

react/jsx-no-literals: In JSX when using a literal string you can wrap it in a JSX container {'TEXT'}. This rules requires that you wrap all literal strings. Prevents any odd artifacts of highlighters if your unwrapped string contains an enclosing character like ' in contractions and enforces consistency.

react/jsx-no-target-blank: When creating a JSX element that has an a tag, it is often desired to have the link open in a new tab using the target='_blank' attribute. Using this attribute unaccompanied by rel='noreferrer noopener', however, is a severe security vulnerability (see here for more details) This rules requires that you accompany all target='_blank' attributes with rel='noreferrer noopener'.

react/jsx-no-undef:
This rule helps locate potential ReferenceErrors resulting from misspellings or missing components.

react/forbid-prop-types: By default this rule prevents vague prop types with more specific alternatives available (any, array, object), but any prop type can be disabled if desired. The defaults are chosen because they have obvious replacements. any should be replaced with, well, anything. array and object can be replaced with arrayOf and shape, respectively.

Coverage remained the same at 100.0% when pulling<PHONE_NUMBER>e577c29dc9a97018fef15418af45e2 on jsx-styling into d9551f01f0c64650ac889e2d17201916b8b9cca3 on master.

Coverage remained the same at 100.0% when pulling<PHONE_NUMBER>e577c29dc9a97018fef15418af45e2 on jsx-styling into d9551f01f0c64650ac889e2d17201916b8b9cca3 on master.

Coverage remained the same at 100.0% when pulling<PHONE_NUMBER>e577c29dc9a97018fef15418af45e2 on jsx-styling into d9551f01f0c64650ac889e2d17201916b8b9cca3 on master.

Coverage remained the same at 100.0% when pulling<PHONE_NUMBER>e577c29dc9a97018fef15418af45e2 on jsx-styling into d9551f01f0c64650ac889e2d17201916b8b9cca3 on master.

Coverage remained the same at 100.0% when pulling da66b2d47548061660e3fcf3d1e5c2b1168440f6 on jsx-styling into d9551f01f0c64650ac889e2d17201916b8b9cca3 on master.

Coverage remained the same at 100.0% when pulling b5a71386b10cfaf31e55d8be54dc5c4e3c89bcfb on jsx-styling into d9551f01f0c64650ac889e2d17201916b8b9cca3 on master.

Coverage remained the same at 100.0% when pulling c53c7e8adb99bbacb35d977d92a45d02e32b1c41 on jsx-styling into d9551f01f0c64650ac889e2d17201916b8b9cca3 on master.

Coverage remained the same at 100.0% when pulling c5d4ed7e2fc36c8fe1dd68f6d90b20a561213cfb on jsx-styling into d9551f01f0c64650ac889e2d17201916b8b9cca3 on master.


Table style broken?

Can you provide a screenshot? I cannot reproduce the table-style you showed me and I'm not sure what it looked like...


ui-grid: enableGridMenu option is missing under gridoptions

Options enableGridMenu and useExternalFiltering is missing under grid options.
     /**
     * False by default. When enabled, this adds a settings icon in the top right of the grid,
     * which floats above the column header. The menu by default gives access to show/hide columns,
     * but can be customized to show additional actions.
     * @default false
     */
     enableGridMenu?: boolean;

      /**
         * Disables client side filtering. When true, handle the filterChanged event and set data,
         * defaults to false
         * @default false
         */
        useExternalFiltering?: boolean;

Please let me know I can submit a pull request for both these options.

please send a pull request. I'll review it.

PR #6187

PR #6202

enableGridMenu does not work.
http://plnkr.co/edit/?p=preview
When I set enableGridMenu to false, it still shows the menu
Tan

@tandvu Please report it as an issue with the ui-grid repo.
PS: It is working as expected, atleast in the plunker, grid menu is not shown.
@vvakame I hope this issue can be closed as the PR has been merged.


Devel - Adding Tower Centric Network Workshop Provisioner

Enhanced Provisioner for more Tower focused network workshops.
Role: populate_tower
Tasks: split into three steps due to dependencies and need connection: local vs. not using local

@IPvSean Round 3


zuul-core: provide a Netty Promise adapter

LGTM, but this is a pretty fundamental change. Should we have a bigger discussion about this? Also, can we add some tests to verify the behavior?


MF-1240 - Return to service transport layer only service errors

Signed-off-by: Manuel Imperiale<EMAIL_ADDRESS>What does this do?
Refactor encodeError method in services transport layer to ensure that service and db errors are properly encoded in the HTTP response.
Which issue(s) does this PR fix/relate to?
Resolves #1240
Have you included tests for your changes?
yes

Codecov Report

Merging #1559 (7b7865f) into master (d51a79c) will increase coverage by 0.08%.
The diff coverage is 48.91%.

@@            Coverage Diff             @@
##           master    #1559      +/-   ##
==========================================
+ Coverage   69.08%   69.17%   +0.08%
==========================================
  Files         134      135       +1
  Lines       11069    10993      -76
==========================================
- Hits         7647     7604      -43
+ Misses       2789     2755      -34
- Partials      633      634       +1

Impacted Files
Coverage Δ

auth/api/http/groups/responses.go
11.32% <ø> (ø)

auth/api/http/keys/responses.go
100.00% <ø> (ø)

auth/api/http/policies/responses.go
87.50% <ø> (ø)

bootstrap/api/responses.go
100.00% <ø> (ø)

certs/postgres/certs.go
0.00% <0.00%> (ø)

consumers/notifiers/api/responses.go
100.00% <ø> (ø)

pkg/errors/errors.go
73.17% <ø> (ø)

provision/api/transport.go
0.00% <0.00%> (ø)

readers/api/responses.go
100.00% <ø> (ø)

readers/cassandra/messages.go
82.07% <0.00%> (ø)

... and 32 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 d51a79c...7b7865f. Read the comment docs.


X-PJAX-URL redirect missing X-PJAX header

If I get a request from pjax, I check eg whether the user is logged on or he has the permission to access. If the users do not have access, I redirect him. For this purpose, I use X-PJAX URL: {new_url} header.
Unfortunately, the new URL is not called with an X-PJAX = true header, which means that the page is displayed as a normal request. But i like to request \login.html with pjax.
Is there a solution?

I added this on pjax - options.success
var redirect = xhr.getResponseHeader('X-PJAX-URL');
            if(redirect && !container.contents){
                pjax({url: redirect, container: options.context})
                return;
            }

You probably don't need to change any client code here. Instead, simply redirect normally on the server. Example:
GET /mypage
X-PJAX: true

HTTP 302 Found
Location: /login.html

# ajax will automatically follow the 302:
GET /login.html
X-PJAX: true

# login page should set X-PJAX-URL to its own url:
HTTP 200 OK
X-PJAX-URL: /login.html

XMLHttpRequest (which pjax is based on) will automatically follow redirects. Hopefully it will preserve the same headers as well.

Let me know if that worked!

It works only partially. It is loaded to the right content, but the URL is not changed.
Example:
Request

Request URL:http://example.com/setting/profile
Request Method:POST
X-PJAX:true
X-PJAX-Container:#content
X-Requested-With:XMLHttpRequest

if the data invalid, i return new content (form with errors)
if the update is success, i just send new header
Response

Cache-Control:no-store, no-cache, must-revalidate, post-check=0, pre-check=0
Connection:Keep-Alive
Content-Length:0
Content-Type:text/html; charset=UTF-8
Location:http://example.com/setting/setting

The content from setting/setting ist displayed, but the url is setting/profile
My solution
I changed the response in my redirect
public function redirect($url, $status = 302) {
		if (isset($_SERVER['HTTP_X_PJAX'])) {
			header('X-PJAX-URL: '.$url,true);
			exit();
		}
		header('Location: ' . $url, true, $status);
		exit();
	}

and added this to the pjax.js
var redirect = xhr.getResponseHeader('X-PJAX-URL');
            if(redirect && !container.contents){
                pjax({url: redirect, container: options.context})
                return;
            }

In this solution, i didn´t send a status 302. Just the new URL.
This will work for me

The content from setting/setting ist displayed, but the url is setting/profile

You could have fixed that by having setting/setting page set the X-PJAX-URL response header:
X-PJAX-URL: /setting/setting

The X-PJAX-URL header does not change the url...

How are you setting it from the server, can I see the line of code? Is it set from a response from setting/setting page? Thanks


Implements SSL check, fixes #6

My SIM800L rev R14.18 was considered < R14.00 and without SSL support

Perfect, thank you for implementing the SSL check in a proper way!


STOXX 600 Equity Bundle help

Hi there,
This is my first ever issue on GitHub so really sorry if my question is unclear or not in the appropriate format.
I am currently trying to build an equity bundle filled with the STOXX 600. Given this is a basket of European stocks, they all trade on different exchanges throughout Europe with different trading holidays and opening hours.
It appears to me that Zipline is very much geared towards equity bundles which contain stocks that all trade on the exact same exchange. Does anyone know how I might be able to ingest a bundle that does not follow only one trading calendar, but multiple?
What I have done so far:
I have followed the guide located here: https://analyzingalpha.com/zipline-equity-bundle
and with this guide, I have been able to utilize BColzDailyBarWriter() to write the stocks into a BColz format. In the below code, I am looping through my asset_metadata table, which contains information on each stock (i.e its associated exchange code like "XLON", its trading start date and trading end date, etc...), and then setting the calendar argument via the get_calendar() function.
for asset_sid, row in assets_metadata.iterrows():

    daily_bar_writer = BcolzDailyBarWriter(
        calendar=get_calendar(row['exchange']),  # Would love to default to LSE for everything
        start_session=pd.Timestamp("2010-01-04", tz='UTC'),
        end_session=pd.Timestamp("2021-07-29", tz='UTC'),
        filename=r'Stefan_Jansen/ch4/Pt_2_zipline_single_factor/bcolz_sxxp_data/bcolz_sxxp.bcolz'
    )

    daily_bar_writer.write(
        data = parse_pricing_and_vol(
            raw_data,
            asset_sid,
            row['symbol'],
            row['sessions']
        ),
        show_progress=True
    )

But when I get to the register() step, the function is forcing me to select a single calendar (via the register() argument, calendar_name) and so I am a bit stumped because all my stocks must then follow the trading days of that one calendar. Should I write a new register() function for every exchange and its associated stocks?
from zipline.data.bundles import register, sxxp_bundle

register('sxxp_bundle', sxxp_bundle.ingest, calendar_name='????')

Any help on this issue would be a huge godsend! Thank you all.

Hi there, did you ever figure out the answer on this one?


Fix word 'anda' to 'kamu' for Indonesian translation

Fixing words suggested in Kubernetes contributor workshop in Openinfra Day Surabaya Indonesia
/language.id
/lgtm
/cc @girikuncoro

Hi @sevenzy, thanks for the PR! Please follow instructions at https://git.k8s.io/community/CLA.md#the-contributor-license-agreement to sign the CLA so that we can review this PR, thanks!


Lock the application after X minutes of inactivity or if the device goes inactive

I think this should be an option.
What do you think?

Yeah definitely make it optional, I'll probably work on this along with the option to force draft-payments.

Using the electron power-monitor module we can easily detect sleep/power-on events


Document steps required to use social providers in the custom UI

Created by: @yetizer Apr 28 2021 at 13:29 CET
Last updated: Jul 07 2021 at 09:28 CET
ClickUp ID: IK-828
Assigned in ClickUp to: Petr Vecera - OLD
You need UI SDK version at least 1.0.7
For login and register you need to use optional parameter "redirectUri" as described in the documentation here.
You need to implement the url path in your application, for example /callback this needs to be the same url as with the redirectUri you have passed in the step #2.
The user is automatically redirected in the flow to this callback. When user access this callback url. You need to call function IKUIOidc.oidcCallback() as described here. This function will return you with the tokens and other information in case you need it.
The example of this implementation can be found here:
https://github.com/indykite/indykite-ui-sdk-sample-react/blob/main/src/components/Oidc.js#L19
https://github.com/indykite/indykite-ui-sdk-sample-react/blob/main/src/App.js#L106
You need to configure the Indykite back-end to allow this URL and also to setup which OIDC providers you want to use.

Commented by: @bradtumy Apr 29 2021 at 14:20 CET
@ Petr Vecera - OLD do you have instructions for this step?

You need to configure the Indykite back-end to allow this URL and also to setup which OIDC providers you want to use.

Closed: May 11 2021 at 14:54 CET


New rule to prohibit usage list of classes

Description
The rule is conteined in the new test-ruleset.xml ruleset.
A new rule has been implemented to check if an prohibited class is used.
Default prohibited classes are:

org.apache.commons.codec.CharEncoding
org.apache.commons.lang3.CharEncoding
org.apache.commons.lang.CharEncoding
org.apache.log4j.Logger

import org.apache.log4j.Logger;
import java.nio.charset.Charset;
import org.apache.commons.codec.CharEncoding;
class Foo {
    private static Logger log = Logger.getLogger(Foo.class); //violation
    Charset cs = Charset.forName(CharEncoding.UTF_8);       //violation
}

Completed work

[x] Added new rule
[x] Added new unit tests
[x] Passed all checks

Related issues:
Implements #21

@rultor merge

@rultor merge

@dgroup OK, I'll try to merge now. You can check the progress of the merge here

@rultor merge

@dgroup Done! FYI, the full log is here (took me 2min)


Scope header is missing

When call listCampaigns,  it returns error "Scope header is missing".

Can Profile Id  be only got from SandBox?
SandBox's Profile Id is working on live( not SandBox)?

Hello,
Integration issues are handled via Jira service desk. If you have onboarded with the API you should have an account.
https://amzn-clicks.atlassian.net/servicedesk/customer/portal/2
Brent


Google Cloud Storage cache adapter

Hi guys,
Is anyone working on a GCP caching adapter?
Would a pull request implementing this be welcome?

@matheusmatos I'm not aware of anyone tacking this issue and a PR would be welcome!

Nice! I have 0 experience with that service so all I can do I help with code review from an implementation standpoint, but not a correct / best practices standpoint specific to the service.

Excellent! In fact, I've experience with GCP but not so much with GoLang. I opened the PR (#891) but I think it's not ready to be merged yet.
I needed to add the cloud.google.com/go/storage package and it created a lot of files inside vendor folder. Is it correct for this to be committed to Git?

Yeah vendoring will copy all the files to the vendor directory. This is our preferred approach. Ideally the vendoring happens in a separate commit from the implementation.

Okay, I'll resend as two commits, implementation and vendoring. Just a moment.

I sent the two commits. I'm not sure if the vendor commit is correct. Any help is appreciated. Thanks!

Hi @ARolek, thanks for the great work. I am interested in this PR too (thanks @matheusmatos too!). Any news when it can be merged ?
Best

Ok great thanks for the reply @ARolek


Official Google Maps Api Flutter

I beg you to give us an Official Flutter Google Maps SDK for run android and ios with all same apis on on that devices
i so love flutter because its a realy cool and fast native cross-platform mobile framework but google maps official dart sdk bruise is my real big problem with Flutter !!!!

You can try this plugin developed by Flutter team: google_maps_flutter. Or you can use the plugin developed by AppTree Software: flutter_map.

The plugin is limited , when will Flutter provide Google Map View widget?

@zulfahmi93  this plugin can't be work if build in IOS because some plugin need code use_framework! in pod file and google_maps_flutter error if i write use_framework! in pod file

I followed the instructions:
google_maps_flutter:    git:      url: git://github.com/flutter/plugins      path: packages/google_maps_flutter
and receive the following error:
<path to flutter> --no-color packages get Running "flutter packages get" in google_maps_flutter... Error on line 18, column 3 of pubspec.yaml: A package may not list itself as a dependency.   google_maps_flutter:   ^^^^^^^^^^^^^^^^^^^ pub get failed (65) Process finished with exit code 65
I am new to flutter but have several other examples working without issues.

Received an error using the Github documentation (the Github ReadMe file):
`Launching lib/main.dart on moto x4 in debug mode...
Initializing gradle...
Resolving dependencies...
Running 'gradlew assembleDebug'...
/.pub-cache/git/plugins-6891204e59182157a7ea705d64f3ebd820a053c0/packages/google_maps_flutter/android/src/main/java/io/flutter/plugins/googlemaps/GoogleMapFactory.java:11: error: no interface expected here
public class GoogleMapFactory extends PlatformViewFactory {
^
/.pub-cache/git/plugins-6891204e59182157a7ea705d64f3ebd820a053c0/packages/google_maps_flutter/android/src/main/java/io/flutter/plugins/googlemaps/GoogleMapsPlugin.java:34: error: incompatible types: GoogleMapFactory cannot be converted to PlatformViewFactory
"plugins.flutter.io/google_maps", new GoogleMapFactory(plugin.state, registrar));
^
/.pub-cache/git/plugins-6891204e59182157a7ea705d64f3ebd820a053c0/packages/google_maps_flutter/android/src/main/java/io/flutter/plugins/googlemaps/GoogleMapFactory.java:22: error: method does not override or implement a method from a supertype
@Override
^
Note: Some messages have been simplified; recompile with -Xdiags:verbose to get full output
3 errors
FAILURE: Build failed with an exception.

What went wrong:
Execution failed for task ':google_maps_flutter:compileDebugJavaWithJavac'.
Compilation failed; see the compiler error output for details.

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

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

BUILD FAILED in 2s
Finished with error: Gradle build failed: 1
`

It seems 11 days ago there was a commit that changed this from an interface to an abtract class ...
https://github.com/flutter/engine/commit/091492601479a6a234c7c4c91b19b7080fabfb6b#diff-403038bc2f72df8dff8d503497dee676
must not be available in the latest flutter upgrade yet.

I'm getting a similar error but with Method Not Found:
Launching lib/main_dev.dart on Android SDK built for x86 in debug mode...
compiler message: file:///Users/gonzalo/flutter/.pub-cache/git/plugins-4ef2f60f021279be81d5d140cdba3483e422fbb9/packages/google_maps_flutter/lib/src/google_map.dart:20:14: Error: Method not found: 'AndroidView'.
compiler message:       return AndroidView(
compiler message:              ^^^^^^^^^^^
compiler message: file:///Users/gonzalo/flutter/.pub-cache/git/plugins-4ef2f60f021279be81d5d140cdba3483e422fbb9/packages/google_maps_flutter/lib/src/google_map.dart:20:14: Error: The method 'AndroidView' isn't defined for the class 'google_maps_flutter::_GoogleMapState'.
compiler message: Try correcting the name to the name of an existing method, or defining a method named 'AndroidView'.
compiler message:       return AndroidView(
compiler message:              ^

---

FAILURE: Build failed with an exception.
* Where:
Script '/Users/gonzalo/flutter/packages/flutter_tools/gradle/flutter.gradle' line: 435

Looks like AndroidView is not found. Are there any workaround on this?
Thanks

Started working for me yesterday after the new Flutter beta was released. Seems there were code changes pending in the master branch that are now in the beta branch. Run flutter upgrade from Studio or the prompt.

Yes, that did the trick!! Thanks a lot!

Hey @zoechi,
I have this error that I can't get rid off it happened when I used: map_view plugin and: google_maps_flutter plugin, is someone been able to fix it.
Thanks,
this is the error below:
`Launching lib\main.dart on Android SDK built for x86 in debug mode...

Error running Gradle:
Exit code 1 from: C:\Users\chadi\Desktop\Flutter projects\activmap\android\gradlew.bat app:properties:

BUILD FAILED
Total time: 4.131 secs
FAILURE: Build failed with an exception.

Where:
Build file 'D:\SDKs\flutter.pub-cache\git\plugins-a6a469a0da9e90b83d957c2c612a994ff1c73adc\packages\google_maps_flutter\android\build.gradle' line: 6

What went wrong:
A problem occurred evaluating project ':google_maps_flutter'.

Could not find method google() for arguments [] on repository container.

Try:
Run with --stacktrace option to get the stack trace. Run with --info or --debug option to get more log output.
Please review your Gradle project setup in the android/ folder.
Exited (sigterm)`

bump.  i also had this error @chadidi  - any luck?

Hey @neiljaywarner,
I think I solved somehow but I forgot how, I will take a look this Sunday and let you know!

I beg you to give us an Official Flutter Google Maps SDK for run android and ios with all same apis on on that devices
i so love flutter because its a realy cool and fast native cross-platform mobile framework but google maps official dart sdk bruise is my real big problem with Flutter !!!!

Here here, I can't a gree with this guy more. He is absolutely right. Currently there is no decent maps solution for flutter and it limits what we can do with the API. This issue has delayed the launch of my product for months now. When will flutter have maps!!!

I was working great with Flutter, and I love it !, until I was ready to use Google Maps as traditionally in my projects and this was the big problem :( Please, if you could give official support for Google Maps in Flutter we would be enormously Thank you, so far, it's the only inconvenience I've found, everything else is perfect, let Flutter go ahead.

how can I listen the marker's event when the position is changed? anybody know?

@DiegoOpenheimer
Please consider asking support questions in one of the other channels listed at http://flutter.io/support or create a new issue if you want to report a bug or feature request.

The sample works on Android, but I got this error on iOS:
Launching lib/main.dart on iPhone 8 in debug mode...
Starting Xcode build...
Xcode build done.                                            6.5s
Syncing files to device iPhone 8...
*** First throw call stack:
(
0   CoreFoundation                      0x00000001112f41bb __exceptionPreprocess + 331
1   libobjc.A.dylib                     0x0000000110892735 objc_exception_throw + 48
2   CoreFoundation                      0x00000001112f4015 +[NSException raise:format:] + 197
3   Runner                              0x000000010bdce173 +[GMSServices checkServicePreconditions] + 182
4   Runner                              0x000000010bdcafb2 +[GMSServices sharedServicesSync] + 110
5   Runner                              0x000000010bdbaa09 -[GMSMapView sharedInitWithServices:camera:forPlaces:] + 156
6   Runner                              0x000000010bdb9e71 -[GMSMapView initWithFrame:camera:] + 123
7   Runner                              0x000000010bdb9c95 +[GMSMapView mapWithFrame:camera:] + 104
8   Runner <…>
Lost connection to device.

google_maps_flutter is officially supported by the flutter team. We do not have full api compatibility with the sdk on ios and android. Please file issues or submit PRs for specific features you'd like to see.

When will google_maps_flutter will provide navigation on map? Like from the current device location to specific marker.

Error running Gradle:
ProcessException: Process "G:\Work File\FlutterApp\ecomm_app\android\gradlew.bat" exited abnormally:

Configure project :app
WARNING: API 'variant.getMergeAssets()' is obsolete and has been replaced with 'variant.getMergeAssetsProvider()'.
It will be removed at the end of 2019.
For more information, see https://d.android.com/r/tools/task-configuration-avoidance.
To determine what is calling variant.getMergeAssets(), use -Pandroid.debug.obsoleteApi=true on the command line to display a stack trace.
WARNING: API 'variantOutput.getProcessResources()' is obsolete and has been replaced with 'variantOutput.getProcessResourcesProvider()'.
It will be removed at the end of 2019.
For more information, see https://d.android.com/r/tools/task-configuration-avoidance.
To determine what is calling variantOutput.getProcessResources(), use -Pandroid.debug.obsoleteApi=true on the command line to display a stack trace.
*********************************************************
WARNING: This version of image_picker will break your Android build if it or its dependencies aren't compatible with AndroidX.
See https://goo.gl/CP92wY for more information on the problem and how to fix it.
This warning prints for all Android build failures. The real root cause of the error may be unrelated.
*********************************************************

FAILURE: Build failed with an exception.

What went wrong:
A problem occurred configuring project ':map_view'.

Failed to notify project evaluation listener.
java.lang.AbstractMethodError (no error message)

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

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

BUILD FAILED in 3s
Command: G:\Work File\FlutterApp\ecomm_app\android\gradlew.bat app:properties
Please review your Gradle project setup in the android/ folder.

Hi, I get the following errors when trying to display a Google map on iOS.
Running Xcode build...
Xcode build done.                                           23.0s
Syncing files to device iPhone Xʀ...
Lost connection to device.
*** First throw call stack:
(
	0   CoreFoundation                      0x0000000109c2a6fb __exceptionPreprocess + 331
	1   libobjc.A.dylib                     0x00000001091ceac5 objc_exception_throw + 48
	2   CoreFoundation                      0x0000000109c2a555 +[NSException raise:format:] + 197
	3   Runner                              0x00000001041030e3 +[GMSServices checkServicePreconditions] + 182
	4   Runner                              0x00000001040fff22 +[GMSServices sharedServicesSync] + 110
	5   Runner                              0x00000001040ef979 -[GMSMapView sharedInitWithServices:camera:forPlaces:] + 156
	6   Runner                              0x00000001040eede1 -[GMSMapView initWithFrame:camera:] + 123
	7   Runner                              0x00000001040eec05 +[GMSMapView mapWithFrame:camera:] + 104
	8   Runner <…>

@iskakaushik I don't see how https://pub.dev/packages/google_maps_flutter solves this issue.
I believe the original request is for a native Dart/Flutter implementation of Google Maps, properly done by the Google Maps API team.
As someone who uses flutter on platforms that are not Android and iOS, but still needs Google Maps support, I feel an issue should stay open that illuminates the need for direct rendering of the Maps in Flutter.


Support suggestion for missing TLD

Hi! We like the port to Ruby. There's just one case that it didn't cover for us. For an email like ron@imhr it suggests ron@imhrcom which is missing the dot "." in between the domain and tld.

Thanks @more-ron, it looks reasonable and tests pass, merging!

👍 👍


Ag package term suggestion soil_horizon MIXS:0001291

New term details
Term name - soil horizon
Term ID - MIXS:0001291
Structured comment name - soil_horizon
Definition - soil horizon is defined as the layer parallel to the soil surface whose physical, chemical and biological characteristics differ from the layers above and beneath. Horizon can be described both in absolute terms (particle size distribution for texture, for instance) and in terms relative to the surrounding material, i.e. ‘coarser’ or ‘sandier’ than the horizons above and below.
Expected value - enumeration
Value syntax - enumeration
Example - [O horizon|A horizon|E horizon|B horizon|C horizon|R layer|Permafrost]
Package(s) - Agriculture

Additional context
Term suggested as part of Ag package.
Points for discussion
there is already a term called "horizon" [MIXS:0001082] that matches this description, we should probably update the existing term name

We agree with @only1chunts .
ENVO will be working on terms for soil horizons.

OK. so there is no need to add the new term MIXS:0001291, instead, we will update MIXS:0001082, see ticket #232 for the update.


Use fuel-toolchain.toml in our templates

It is possible to define a fuel-toolchain.toml file at the root of a project to use a specific forc and fuel-core version for that project and fuelup will pick it up and use it instead of the currently active global toolchain. We can add this file in our pnpm create fuels templates and have it match the version supported by fuels.

We should consider applying this approach only to the template dApp generated by create fuels but not to our internal tests and CI — for these, we should continue using our internal forc and fuel-core packages, which provide working with non-published versions and git branches.

Going to pick this up as I've been working with create fuels templates this past week, would be helpful


Support to VRE tools returning a directory

It needs to compress the results returned by VRE tools if their tools return a directory.
Compressions will be made in zip format.

Fixed by me through commit 979f462be477b73ef661c70b53fb2586d6044169


[fix](jdbc catalog) Disable oracle scan null operator pushdown #41563

cherry pick from #41563

run buildall


How to get list of running Tasks ?

Hello,
I use vcenter 8.0
How do I do get list of running Tasks related  a VM? How can I do that? I've tried searching this on the documents.
Please give any informations and I want some samples.

@harukinakazono :
Try below rest api to get the Tasks -
https://developer.vmware.com/apis/vi-json/latest/sdk/vim25/release/TaskManager/moId/recentTask/get/

@harukinakazono : Try below rest api to get the Tasks - https://developer.vmware.com/apis/vi-json/latest/sdk/vim25/release/TaskManager/moId/recentTask/get/

Thank you.
Sorry for the lack of explanation.
I want to know how to get the task using the vsphere-automation-sdk-python package.
Please tell me how to get it and give me an example of it.
Thanks.


multistream handling

per discussion with @ggarber it is probably easiest to just make one row per stream/track

an additional thing to take into consideration: a stream may not be active all the time.
How does that

affect averages like bitrate (ignore 0s?)
duration -- it can not be calculated as last - first

wire protocol bump required: https://github.com/opentok/rtcstats/pull/83
WIP here

note: this needs to be stream-based, not track-based.

this is now on the master branch.


Deliver git-phlow also using chocolatey packages

We will deliver git-phlow with a chocolatey package on Windows also.

this seems to be possible to automate it without the biggest of hassle https://puppet.com/blog/chocolatey-creating-your-own-chocolatey-packages


CB-11741: Nigthly e2e Azure tests should use encryption.

CB-11741: Nigthly e2e Azure tests should use encryption.
I have added integration test for Azure CMK. Is it sufficient to ensure nightly e2e for azure?

@juhi-09 we need to setup the jenkins side of the story. Do you need help ?

@juhi-09 we need to setup the jenkins side of the story. Do you need help ?

@doktoric We would need jenkins to setup the EnvironmentTestDto, right? Yes, I would need some guidance on it.

@juhi-09 we need to setup the jenkins side of the story. Do you need help ?

@doktoric We would need jenkins to setup the EnvironmentTestDto, right? Yes, I would need some guidance on it.

lets schedule a meeting for this

@juhi-09 Have you discussed this? Have you executed this e2e test locally?

@juhi-09 we need to setup the jenkins side of the story. Do you need help ?

@doktoric We would need jenkins to setup the EnvironmentTestDto, right? Yes, I would need some guidance on it.

lets schedule a meeting for this

@juhi-09 Have you discussed this? Have you executed this e2e test locally?

@horadla23 Yes Adam, this is covered. We can merge the PR. e2e tests are executed locally and jenkins setup is done to create a encryption key -  https://cloud-jenkins-secrets.vault.azure.net/keys/key-azure-cmk-e2e/c1a1df3097f441b4a10cfcb7c8a3bcd9 is created in cloud-daily resource group

FYI: If the test fails for some reason in the next 1 week, we are going to disable it and mark it as unstable.


CLI improvements

Description
Why is this change required? What problem does it solve? Describe your changes in detail:
This incorporates the Rich library for better CLI UI

Rich will now be used through the CLI to create a more consistent theme.
If it fixes an open issue, please link to the issue here:
Screenshots (if appropriate):
Types of changes
What types of changes does your code introduce? Put an x in all the boxes that apply:

[ ] Documentation update
[ ] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

Codecov Report

Merging #32 into master will decrease coverage by 0.03%.
The diff coverage is 0.00%.

@@            Coverage Diff             @@
##           master      #32      +/-   ##
==========================================
- Coverage   40.80%   40.76%   -0.04%
==========================================
  Files          21       21
  Lines         647      628      -19
  Branches       48       48
==========================================
- Hits          264      256       -8
+ Misses        383      372      -11

Impacted Files
Coverage Δ

stockroom/cli.py
0.00% <0.00%> (ø)

...ockroom/external/importer/torchvision_importers.py
0.00% <0.00%> (ø)

tests/conftest.py
100.00% <0.00%> (ø)

tests/test_cli.py
100.00% <0.00%> (ø)

tests/test_model.py
100.00% <0.00%> (ø)

tests/test_repository.py
100.00% <0.00%> (ø)

stockroom/external/importer/base.py
0.00% <0.00%> (ø)

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 afd01e0...6b18a48. Read the comment docs.

there is an issue apparently with isort <5.1.4  and setuptools v=49.2 more here
so I specified a version of isort.

it seems to be due to the fact that pylint doesn't support isort>5
Collecting isort>=5.1.4
  Downloading isort-5.4.2-py3-none-any.whl (94 kB)
     |████████████████████████████████| 94 kB 2.8 MB/s
ERROR: pylint 2.5.3 has requirement isort<5,>=4.2.5, but you'll have isort 5.4.2 which is incompatible.
Installing collected packages: isort
Successfully installed isort-5.4.2

Also, make sure your branch is rebased with the latest master


PR #89 must be reviewed to handle min and max dates

It seems we also need to change _isDisableDays function to do logic with UTC instead with local TZ.

I'm working on it, so when finished I'll make a new PR if necessary...

@userquin Cool.

@motss , I have a problem. I have merge my PR #89 to my master and then create a new branch for PR #90 from my master  but when I request a new one PR for your repo all changes from PR #88 when I  compare across forks.

You have moved PR #89 to 2.11.0, my repo comes from 2.10.1. I think you dont have problems merging to 2.11.0.


fix typo

Fix typo on README - the first page each developer will encounter!
from BUIDL > BUILD

ROFL

ser it is like HOLD -> HODL it's a thing


Cholmod error 'problem too large'

Hi all,
I'm getting the following error when trying to create the Tempora object with ImportSeuratObject:
Error in asMethod(object) :
  Cholmod error 'problem too large' at file ../Core/cholmod_dense.c, line 105

The dimensions of the Seurat object are as follows:
> subset_merge_allCells
An object of class Seurat
60218 features across 102868 samples within 2 assays
Active assay: SCT (26680 features, 3000 variable features)
 1 other assay present: RNA
 2 dimensional reductions calculated: pca, umap

I know this is a large object, I've actually cut it down in half (I have a little over 200k cells sequenced). And so I wonder if anyone has some suggestions for:

stripping down the data further - could I get rid of all genes except the 3000 most variable ones?
remove more cells during the prelim QC of cells? I've already halved the dataset, so I'd rather not do this one..
is there a general dimension/size limit for creating the tempora object?

Thank you!

quick update - I was able to get around this issue by removing genes expressed in less than a certain number of cells. The new object is 12355 features x 102868 cells, and ImportSeuratObject does not complain.
I'll leave this issue open for now, in case someone can reply with more specific limitations for the importseuratobject function.

quick update - I was able to get around this issue by removing genes expressed in less than a certain number of cells. The new object is 12355 features x 102868 cells, and ImportSeuratObject does not complain.
I'll leave this issue open for now, in case someone can reply with more specific limitations for the importseuratobject function.


statefulset failover IP change

Hi there,
First thanks very much for contribute your solution for redis cluster which is super useful.
I just have a concern that statefulset only guarantees no change on domain of each node, however, the ip will still be changed. Will this break the cluster meet in cluster mode?
Thanks,
Jiabin

Hi,
Redis cluster reconfigures the ip address automaticly when they detect
changes. The redis container thats down must just point to a valid redis
container when its back up. It will be best to run each statefulset on a
seperate host. If everything goes down you have a problem and you will have
to edit the nodes.conf file manually, but this should be rare.
Theres just one bug in redis that you need to manually hack around
https://github.com/antirez/redis/issues/4645
Also if you are using the latest redis version, the cluster commands are
now built into the redis-cli.
Hope this helps :)
On Thu, 16 Aug 2018 at 07:17, johnjavabean<EMAIL_ADDRESS>wrote:

Hi there,
First thanks very much for contribute your solution for redis cluster
which is super useful.
I just have a concern that statefulset only guarantees no change on domain
of each node, however, the ip will still be changed. Will this break the
cluster meet in cluster mode?
Thanks,
Jiabin
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/sanderploegsma/redis-cluster/issues/6, or mute the
thread
https://github.com/notifications/unsubscribe-auth/AHW99sOYhZxM286vICuc3YQmb54yhiuJks5uRQBrgaJpZM4V_Np6
.

--
Riaan Jacobs
<EMAIL_ADDRESS>

Thanks for the response @riaan53, since Redis is no longer part of my day job I kind of stopped focusing on it. Great point on the cluster commands being integrated in Redis 5.0, I updated the repo to reflect it.
Running the pods on separate nodes is always a good idea, if a master and its slave live on the same VM and that goes down, you lose one of the shards. Using podAntiAffinity to make sure pods are not co-located is an easy way to solve this issue.


⚠ Bofrost.de has degraded performance

In 459132d, Bofrost.de (https://www.bofrost.de) experienced degraded performance:

HTTP code: 200
Response time: 775 ms

Resolved: Bofrost.de performance has improved in 36d7ae2 after 49 minutes.


tsserver doesn't quit on vim exit

I've had a few cases where, after fully exiting vim, the coc-tsserver plugin will leave the instance of tsserver it started (invoked via tsserverForkStart) running.  I'm not sure how I can capture this a log format for follow-up, the most I can say is that I've seen a bunch of entries in htop trying to recompile after big code changes (eg: branch checkouts) even though there are no running vim processes.

I am having a similar problem.
I use neovim, and whenever I close it, the node instances of tsserver don't close.

tsserverForkStart not used any more, try upgrade coc-tsserver.


Adds the current models model with recent run stats

This resolves #76, by instead bringing in a new model rather than replacing the old one for backward compatibility. Practically it probably should replace most usages of the existing model but that's a job for people using the package.

@NiallRees - do you think that this PR should also remove/replace the existing fct_dbt__latest_full_model_executions? I was hesitant to just totally rip it out because of the the dependency through to fct_dbt__critical_path - but I'm not sure how the concept of the critical path maps to our current use case.


Mouse Integration

Mouse events should be forwarded to the terminal for programs that have mouse integration like htop and vim.

[x] trackpad scrolling
[x] mousewheel scrolling
[x] basic mouse reporting
[x] SGR reporting

This is half implemented in the mouse-reporting branch. Still needs line-based mouse scrolling and SGR reporting.

Line-based mouse scrolling has been added

Added SGR reporting in 82bfc41af7d6eff384bdf95cedaf0d62b5450979


flutter channel foo should have a option to upgrade first?

It's a bit silly that when I flutter channel foo it downloads all the artifacts for that version when I just plan to upgrade again immediately.  Would be nice to be able to skip that download and/or just switch and upgrade at the same time?

I believe @tvolkert fixed this recently.

https://github.com/flutter/flutter/pull/14896 -- we fetch upstream refs before switching channels, so upgrading immediately after switching channels should be a no-op.

@tvolkert I guess it wold be ideal if flutter channel would ask if it should upgrade in case that there is a newer version.
This feature would also prevent some error reports because people forgot to upgrade after a channel switch.

This is what flutter channel master & flutter channel upgrade looks at the moment:
Switching to flutter channel 'master'...
git: From https://github.com/flutter/flutter
...
Successfully switched to flutter channel 'master'.
To ensure that you're on the latest build from this channel, run 'flutter upgrade'

Checking Dart SDK version...
Downloading Dart SDK from Flutter engine 0e9b2508439aba972b1dfa0b38d11a4c6200e0e3...
Unzipping Dart SDK...
Building flutter tool...
Running pub upgrade...
Upgrading Flutter from D:\programs\flutter...
Checking Dart SDK version...
Downloading Dart SDK from Flutter engine 9b3e3410f02681e446ca48038164a553ababb143...
Unzipping Dart SDK...
...

It is truly a bit silly and really wasteful 😕


pfsshell: init at 1.1.1

Motivation for this change
init pfsshell, which is being used in the tutorial for https://nixos.wiki/wiki/Playstation2
#79142
Things done

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[x] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
[x] Determined the impact on package closure size (by running nix path-info -S before and after)
[x] Ensured that relevant documentation is up to date
[x] Fits CONTRIBUTING.md.


Add .equ support

I'm not sure what is and isn't out of scope for your project, so just thought I'd raise a few issues to confirm before attempting to research an implementation to send a pull request for :+1:
The below syntax would be nice to support in the racc parser:
.equ SYS_SOCKET, 0x200003b
ldr x16, =SYS_SOCKET

But I understand this might be considered out of scope, as it potentially opens up the floodgates to users expecting similar but different features such as:
.equ AF_INET, 0x2
.equ IP, 0x0100007f
.equ PORT, 0x5C11
#...
.balign 16
caddr:
  .short AF_INET
  .short PORT
  .word IP

@AlanFoster I'm fine with adding this stuff to the parser. Please send a PR, I'd be happy to merge it


[BUG]: User can interact with holes even when the game is not running

Description
Currently, the users are able to interact with the mole holes even when the game is in a non-active state (i.e., when the game is not running or after the timer has expired). This allows users to click on the holes and possibly trigger interactions, which is unintended behavior.
Demo
https://github.com/user-attachments/assets/5f6600fd-f725-4458-afba-1a1bb6b9db6f
Steps to Reproduce

Open the Whack-a-Mole game.
Attempt to click on any of the mole holes.
Observe that the holes still responds to user clicks, even when the game isn't running.

Expected Behavior

The user should only be able to interact with the mole holes when the game is actively running.
After the game ends, the mole holes should be unresponsive to user clicks, and no points or game interactions should occur.
The holes should only become clickable again when the game restarts.

Thanks for raising this issue!

Hey @naikmubashir! I'd like to work on this one. Assign it to me

@naikmubashir Have a look.


Encryption support shall be needed for know root packages & su binary paths to avoid cyber security threats

For the below constants, It would be better to keep the encrypted file path strings in the string array
1. knownRootAppsPackages
2. knownDangerousAppsPackages
3. knownRootCloakingPackages
4. suPaths
5. pathsThatShouldNotBeWrtiable
So that we can avoid chances for changing the file path names and  breaking the root detection checking by hackers
Use decrypted strings where ever using this file path.

I like this idea. Here's my current thoughts about how it should be implemented:

Keep the code readable
Encryption / encoding doesn't need to be secure as it's always going to be possible to reverse it.

So we could do something simple like base64 encoding, but also putting all the decoded values in a comment above to keep readability the same. As the comments will be dropped during compilation a simple string search will not longer work.
/*
The following array stores the following package names in base64:

            "com.noshufou.android.su",
            "com.noshufou.android.su.elite",
            "eu.chainfire.supersu",
            "com.koushikdutta.superuser",
            "com.thirdparty.superuser",
            "com.yellowes.su",
            "com.topjohnwu.magisk"
*/
    public static final String[] knownRootAppsPackages = {
            "Y29tLm5vc2h1Zm91LmFuZHJvaWQuc3U=",
            "Y29tLm5vc2h1Zm91LmFuZHJvaWQuc3UuZWxpdGU=",
            "ZXUuY2hhaW5maXJlLnN1cGVyc3U=",
            "Y29tLmtvdXNoaWtkdXR0YS5zdXBlcnVzZXI=",
            "Y29tLnRoaXJkcGFydHkuc3VwZXJ1c2Vy",
            "Y29tLnllbGxvd2VzLnN1",
            "Y29tLnRvcGpvaG53dS5tYWdpc2s="
    };


[grpc] invoke vcpkg_fixup_pkgconfig

Adds vcpkg_fixup_pkgconfig() to grpc package.

What does your PR fix?

N/A

Which triplets are supported/not supported? Have you updated the CI baseline?

No change. N/A

Does your PR follow the maintainer guide?

Yes

Seems like the dependency on openssl is also missing.

Hi @coryan
The failures on current CI are like this, can you look into this?
CMake Error at scripts/cmake/vcpkg_fixup_pkgconfig.cmake:83 (message):
  pkg-config error output:Package libssl was not found in the pkg-config
  search path.

  Perhaps you should add the directory containing `libssl.pc'

  to the PKG_CONFIG_PATH environment variable

  Package 'libssl', required by 'openssl', not found
Call Stack (most recent call first):
  scripts/cmake/vcpkg_fixup_pkgconfig.cmake:184 (vcpkg_fixup_pkgconfig_check_files)
  ports/grpc/portfile.cmake:75 (vcpkg_fixup_pkgconfig)
  scripts/ports.cmake:139 (include)


Fix += decimal error

This change fix the +=decimal error when integer is zero or fraction is zero.
In this situation, the += operator will make a mistake.

#1225


Invalid DOM property colspan. Did you mean colSpan

What happened?
When using {% colspan=2 %} or {% rowspan=2 %} this warning or equivalent is logged serverside.
To reproduce
{% table %}

- 1
- 2
- 3

---

- 1
- 2 & 3 {% colspan=2 %}

---

- 1
- 2 & 3 {% colspan=2 %}

{% /table %}

Version
0.2.1
Additional context
@markdoc/next.js

@lemcii Thanks for filing, your bug report makes sense. React expects DOM attributes to be camelCase rather than lowercase so that warning is coming from React. Let me circle back with the team and see how we want to fix this in our react renderer meanwhile should be safe to ignore for now since React will safely pass colspan through to the DOM: https://reactjs.org/blog/2017/09/08/dom-attributes-in-react-16.html

These attributes are defined in Markdoc's built-in schema for the td node type. It's possible to override what the render tree outputs for those attributes by adding a render property to the schema definition:
attributes: {
    colspan: { type: Number, render: 'colSpan' },
    rowspan: { type: Number, render: 'rowSpan' }
}

It's a bit of a tricky situation because we really only want them to have this camel casing for the React renderer, it's not applicable to the HTML renderer. There are a few different ways we could approach it, but maybe what we want to do is apply the casing in the built-in schema definition and then have the HTML renderer just call toLowerCase on its attribute values so that the HTML output is still conforming with standards. I will collect some feedback from the team and then move forward with that change if there's consensus.


Fix: Syntax in readme.md

This one bugged me for quite a while 😓
Let's get this merged!

Ok thank


Workflow Task Completion Metadata

Rendered

So, we talked more within the SDK team and we decided that the flexibility here comes at a bit too high of a cost in terms of size. We don't actually have any concrete use cases at the moment that require writing completely arbitrary data. Everything we want to write in here is well defined for at least one consumer (even if it's only the SDK consuming its own notes).
Hence, everything we know we want to add here is already a proto. The only question is how it will get encoded. (Hence the desire for Any in my original version). But, both Any and Payload are byte-wasteful.
The main motivation for adding this was speculation about needing to add generic stuff (but we don't know what that is) and decoupling SDK-only stuff from the long approval process / lifecycle in this repo. The first we can deal with if/when it comes up. The second we can deal with by simply introducing an sdk package in this repo with different codeowners. We'll do that.


Ingest IntAct SARS-CoV-2 protein - human protein interaction data

Name of the dataset
IntAct protein - protein interaction data, downloadable from here, I think:
https://t.co/OUGKWbpQHG?amp=1
Mapping or relevant fields
To be added. Data are in a zip file, containing many XML files with interaction data scraped from recent literature.
If possible, highlight which fields map to nodes and which fields map to edges.
Refer to Data Preparation for guidelines on how the final transformed data should be represented.
Additional context
See also this ticket, and a discussion of these data with Birgit, Chris, Laurent-Philip.
If we write an adapter for this XML, this will be a good avenue to get ongoing, updated viral - host PPI data

Note that these data include the Gordon et al. PPI data in #31