Disabled media in offer causes nullptr exception

How to reproduce:

Call setRemoteDescription with OFFER which contains enabled audio media and disabled video media.
Call setLocalDescription with generated ANSWER.
Transceiver for video was not created and than you get nullptr exception here:

      var isIceLite = SDPUtils.matchPrefix(sessionpart,
          'a=ice-lite').length > 0;
      sections.forEach(function(mediaSection, sdpMLineIndex) {
        var transceiver = pc.transceivers[sdpMLineIndex];
        var iceGatherer = transceiver.iceGatherer;
  ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^

My solution now is:
      var isIceLite = SDPUtils.matchPrefix(sessionpart,
          'a=ice-lite').length > 0;
      sections.forEach(function(mediaSection, sdpMLineIndex) {
        var transceiver = pc.transceivers[sdpMLineIndex];
        if (!transceiver)
              return;
        var iceGatherer = transceiver.iceGatherer;

Though of course I don't know all the details but this can give you some ideas on how to properly fix this  case.

Thank you! This was a bit worse since the answer created simply ignored that m-line.
I have a tentative fix in #90 but i doubt you're sending the rejected m-line just for fun and never use it again thereafter, right?

have you tried edge's native RTCPeerConnection? Its renegotiation support is still a bit better (but it lacks in other areas)

should be fixed in version 1.2.9. Let me know if it still happens!


x/tools/gopls: migrate to jsonrpc2_v2

The new version of the jsonrpc2 package (authored by @ianthehat) has improved APIs and shutdown semantics. @bcmills has also recently worked on improving its concurrency model.
This issue tracks work to do the necessary refactoring to migrate to the new APIs.

jsonrpc2_v2 should be stable at this point, whereas the old jsonrpc2 is at least a little flaky (#55179). We should do this migration at some point in the not too distant future.

I almost got to do this, but Alas, the unrelenting pressure of priorities prevented it, and I bailed to implement Async handling in jsonrpc2 (v1).
Here are reasons to make this switch, summarized from my investigation:

In order to make jsonrpc2 v1 async, we'd need to make the ServerHandler dispatch async, exposing the Replier to gopls calls. (EDIT: we ended up doing this by adding magic jsonrpc2.Async(ctx) calls similar to t.Parallel()
...but much of the existing handler behavior is offloaded into middleware: CancelHandler and AsyncHandler. CancelHandler is responsible for handling $/cancel notifications to cancel ongoing requests, but that means it doesn't see $/cancel until its delegate handler has processed prior messages. To solve this problem, the AsyncHandler handles messages (trivially) as fast as it can, enqueuing them onto distinct goroutines.
Additionally, in jsonrpc2 v1 reads from the network connection are synchronous to handling, which could lead to distributed deadlocks with clients blocked on writing. Here again the AsyncHandler mitigates the problem, though suboptimally.
Since this logic is implemented in middleware, and not core to jsonrpc2 v1, the AyncHandler builds up a queue of goroutines waiting to handle requests, and we still have to wait for this queueing to get to $/cancel notification. Furthermore, we need to wait for gopls to actually handle the request (albeit with a cancelled context) before we reply to the client.
This also means that the CancelHandler has to have its own control plane, separate from the jsonrpc2 session lifecycle, which means that closing the jsonrpc2 connection won't cancel ongoing requests.

By comparison, jsonrpc2_v2 exposes the Preempter interface, so that gopls sees the $/cancel notification before it is enqueued. Furthermore, async reading from the network connection is built-in, and requests are always handled asynchronously to reads, on a single dedicated goroutine. Cancelled calls are never seen by gopls. When the connection is closed, ongoing requests are automatically cancelled.


Local Storage - Form

The evacuee will be completing a form requiring the following information.
Frist/Last Name
Family Count
and Generate Ticket Button.

I'll work on the form - using bulma css.

Created modal, and added form to pop-up. the modal can be closed by pressing the X icon or the background (outside of the modal window). Created new modal.js file for the onclick-functions.
added some styling to the modal container.

pushed to main - Kameron is the man for finding out why the modal was not working after merger!


More sensible climatological time points

🚀 Pull Request
Description

Closes #4098 and #4665 which both relate to points being chosen oddly in a climatological aggregation. Main issue arose from the average of Januarys in different years potentially being July, as identified by @trexfeathers on #4098.
I've introduced a "climatological" keyword to iris.cube.Cube.aggregated_by that makes it choose the point from the first cell in each climatology, and sets the climatological attribute of the resultant coord to True if it can.
I've also pulled the tests for aggregated_by out into their own file, because test_Cube is huge and this seems like the first step in rationalising that.

Consult Iris pull request check list

@rcomer - as someone with more experience of actually wanting to aggregate climatologies than me, what would you expect to happen to a 2D AuxCoord when one of its axes is the collapsing axis? Should that just fail the whole operation or should it get aggregated in some way?

Hi @wjbenfold, I think that for any 2D coordinate, collapsing/aggregating on one of its dimensions should give something consistent to having sliced the coordinate and done the collapse/aggregate on 1D slices.  E.g. see this test for the standard numerical aggregated_by case:
https://github.com/SciTools/iris/blob/e3f3bb2d2027aab569db57b7368e0fd1447b03e5/lib/iris/tests/unit/cube/test_Cube.py#L755-L771

I've also pulled the tests for aggregated_by out into their own file, because test_Cube is huge and this seems like the first step in rationalising that.

Just to further muddy the testing waters: I've just become aware of the extensive testing in tests/test_aggregate_by.py.
As far as I'm aware, tests in the tests/ root is a deprecated practice - we now place all test modules in one of the appropriate subdirectories - so there's an opportunity for further rationalisation if you're feeling brave!

My intuition is that all this should only apply to time coordinates

Or maybe time-like coordinates.  I.e. coord.units.is_time_reference().  The obvious example (other than “time”) is “forecast_reference_time”.  This would make sense for seasonal/decadal hindcasts: I have members initialised on 1st November each year for lots of years.  If I aggregate across these, it would make sense that the output “forecast_reference_time” is still 1st November.

This also closes #1422.  @wjbenfold the OP of #1422 links to a gist containing test cases.  Might be useful 👀.

so there's an opportunity for further rationalisation if you're feeling brave!

I feel like bravery and spotting a quagmire are different, so I'll happily take the follow-up PR to rationalise them but I'd rather not drag them into this PR (in the hope that this one gets merged sometime soon)

This also closes #1422. @wjbenfold the OP of #1422 links to a gist containing test cases. Might be useful 👀.

This was a nice check - I still get test failures but they fail how I expected because my view on what's sensible (give them the first point that was used to make the climatology) is different to the author's. Importantly, the months match (just not the years)

From discussion on 4/5, should use first bound rather than points of first cell because for e.g. season aggregation (month and season) we end up with Dec16 referring to Dec1 - Feb28

Thanks @wjbenfold.  LGTM.


Fix warning about missing logout key

🐛 Bug Report
The logout link in the user icon in the navbar is missing a translation per the warning message in the console:
i18next::translator: missingKey en-US translation logout logout

To Reproduce

Load any page
See the warning

Expected behavior
The warning should not appear.

Hi,
I wish to work on this issue.
There is no translation key available for 'logout' in locales/enUs.
Should I add one?
Thanks

@hsorellana I've assigned this to you.

There is no translation key available for 'logout' in locales/enUs. Should I add one?

Yes, I believe this would be the solution to this issue.


Make run and throwable

I already mentioned this request in #92. I think it is a very common use case to make all run commands throwable.
What I would like to avoid is the extra check for the error and manually print the error like this:
        let out = run("mint", args)

        if let error = out.error {
            print(out.stderror)
            throw error
        }

The following would be much cleaner imho and the failure reason could be part of the error or an underlying error maybe:
        let successfulOut = try run("mint", args)

Originally posted by @stherold in https://github.com/kareman/SwiftShell/issues/92#issuecomment-698853537

You're right, it is a bit cumbersome the way it is now. The problem with your suggestion is that it makes it more cumbersome to get the result of a failing command. And it is a breaking change, so it would have to be in SwiftShell 6.0. But your use case is probably more common.
I've been wanting to improve the API for a long time now, but I just haven't been able to come up with a good solution that handles all aspects well, like error handling, synchronicity, shell interpreter, stdin, joined or separate stdout/stderror, printing output, piping output to another command, and probably even more that I haven't thought of yet.


Add signum function

Should this really allocate a new BigInt? I feel like this should just return an int, so that the user can choose if they want that extra allocation, by manually calling initBigInt. Do you have a concrete use casein mind for this, where it's more useful to have this return a BigInt?

Should this really allocate a new BigInt? I feel like this should just return an int, so that the user can choose if they want that extra allocation, by manually calling initBigInt. Do you have a concrete use casein mind for this, where it's more useful to have this return a BigInt?

The reason it returns a BigInt is for the invariant x == abs(x) * sgn(x). I'd be okay with returning an int if BigInt arithmetic allowed implicit widening (like other numeric types do).

I think we should support arithmetic where one of the arguments is an int sooner or later.

I think we should support arithmetic where one of the arguments is an int sooner or later.

Any reason why initBigInt(SomeSignedInt): BigInt isn't an implicit conversion? That seems like the easy solution

NVM - opened https://github.com/nim-lang/bigints/pull/109
Feel free to tell me there why this is a bad idea :-p


bgzip.cwl error writing to read-only file system

This could be a problem with how permissions are set in the samtools:1.3.1 docker image created by ./mc3/utils/dbsnp_index/Dockerfile, or it could be related to exadocker.
Example command used: cwltool mc3/utils/bgzip.cwl bgzip.json
bgzip.json file:
{   "file" : {     "class" : "File",     "path" : "/home/exacloud/lustre1/SpellmanLab/chiotti/gdan_pipelines/mc3_dev/dbsnp_132_b37.leftAligned.vcf"     } }
Resultant error:
/var/lib/cwl/stg09e12223-8974-43c8-88e6-fb6eb0673c59/b37_cosmic_v54_120711.vcf > /tmp/tmpB6L_8k/b37_cosmic_v54_120711.vcf.gz [bgzip] can't create /var/lib/cwl/stg09e12223-8974-43c8-88e6-fb6eb0673c59/b37_cosmic_v54_120711.vcf.gz: Read-only file system [job bgzip.cwl] completed permanentFail

Added to bgzip.cwl at lines 8-9:
    - class: InitialWorkDirRequirement
      listing: [ $(inputs.file) ]

Note: Will only write to working directory rather than original location of input file.


fix: Allow passing multiple args to terraform-docs

The args variable was mistakenly double-quoted in one of the recent commits, which made it impossible to pass multiple arguments to terraform-docs.
With this change it is possible to configure the hook like this:
    hooks:
      - id: terraform_docs
        args:
          - '--args=document --indent 3'

I can confirm that your fix works for me.
@antonbabenko Any idea when we can expect to see this merged in?

Thanks @sergei-ivanov for the work and @eytanhanig for the verification and confirmation!
v1.28.0 has been just released.


create policies to backup and restore persistent apps

The Policies available here provide backup and restore support for stateful applications running on managed clusters or hub. Velero is used to backup and restore applications data.
Policies:

oadp-hdr-app-install installs oadp and configure connection to the storage on managed clusters
oadp-hdr-app-backup backups applications on managed clusters
oadp-hdr-app-restore applications on managed clusters

The configmap  hdr-app-configmap in the PR sets configuration options for the backup storage location, for the backup schedule backing up applications, and for the restore resource used to restore applications backups.
https://issues.redhat.com/browse/ACM-4708

/hold

@gparvin this is replacing the PR using policy sets
https://github.com/open-cluster-management-io/policy-collection/pull/404
I am not using Policy Sets so I would like to contribute this to the policy section ( not policy generator ) ; if you can please review and thanking you for your support with this !

@gparvin @dhaiducek can you please help review this PR

@dhaiducek @gparvin do you have any updates on this ? thank you ..

@dhaiducek @gparvin do you have any updates on this ? thank you ..

The automation failure is due to the configmap existing in the directory structure.  Since it appears that configmap contains a credential, can you just document the configmap creation.  I don't want that be seen as a best practice associated with the policies.

@gparvin I renamed the config files and used a .txt extension to show that they are not to be applied or used as is
I hope this solves the issue

The validation script failed because we weren't expecting PolicySet CRs so I created https://github.com/open-cluster-management-io/policy-collection/pull/410 to fix that.

hi @gparvin is there any update on this ?

@dhaiducek The validation is still failing with the same error.  Does Valentina have to merge in that PR to get validation to pass?

@dhaiducek @gparvin do you have any updates on this PR ?

/cherry-pick 45159b502f6ad7cc262552822a4c8133442f1309

@birsanv I think @gparvin might have been trying to bring in the new validation test to make the "Validation tests" workflow pass in this PR. Would you please:

fetch and rebase on the main branch to pick up the workflow updates
squash the commits in this PR
Let me know if further guidance is needed on either of those.

@dhaiducek thank you , I rebased my PR and the tests seem to pass now

/remove-hold

Thanks! Would you please also squash the commits? Because of the way we sync repos, we merge PRs as a rebase rather than a squash, so we need the author to squash the commits beforehand.

closing this PR and moving the request to this one https://github.com/open-cluster-management-io/policy-collection/pull/415


Installation issues that need fixing

Things to improve installation experience:

[x] pin rpy2 3.4.2

install specifically via conda (if possible)

[x] scib: try-except around ro.r('library(Seurat)’) to report that library installs were problematic
README:

[x] Add “This is a 3-5 step installation” to give overview of long installation section-> add links to sections
[x] remove details, improve flow
[x] First have small statements of simple instructions, then go deeper to details

@mumichae I will take a look at the README, maybe you could take a look at the first 2 steps

@mumichae
The below also causes issue for me:
(base) [karin.hrovatin@icb-lisa scib-pipeline]$ pwd
/storage/groups/ml01/code/karin.hrovatin/scib-pipeline
(base) [karin.hrovatin@icb-lisa scib-pipeline]$ . envs/set_vars.sh $HOME/miniconda3/envs/scib-pipeline
dirname: invalid option -- 'b'
Try 'dirname --help' for more information.
cp activate_env_vars.sh /home/icb/karin.hrovatin/miniconda3/envs/scib-pipeline/etc/conda/activate.d/env_vars.sh
cp: cannot stat '/env_vars_activate.sh': No such file or directory
cp deactivate_env_vars.sh /home/icb/karin.hrovatin/miniconda3/envs/scib-pipeline/etc/conda/deactivate.d/env_vars.sh
cp: cannot stat '/env_vars_deactivate.sh': No such file or directory

@Hrovatin Are you using shell or bash? Maybe try with
bash envs/set_vars.sh $HOME/miniconda3/envs/scib-pipeline

That worked, thanks.

Which environment is that from?

@LuckyMD I just checked the code and calling Seurat via rpy2 is something that is only done in the scib package. Are points 1 and 2 also something you want reflected in the scib package?

Yes, i think that would be useful.

rpy2 library loading checking is addressed in scib and all other suggestions have been applied in #20


Android Mobile App Apollo SDK Update - Schema Introspection Error

Version
v4.0.0
Summary
Integrated Apollo SDK v4.0.0 in Android mobile app project, but receiving introspection error on code compile when using a large size schema file given to us by backend team. Please let us know what could be the issue
Steps to reproduce the behavior
No response
Logs

Exception is:
org.gradle.api.tasks.TaskExecutionException: Execution failed for task ':app:generateServiceApolloSources'.
Caused by: com.apollographql.apollo3.relocated.com.apollographql.apollo3.ast.SourceAwareException: e: src/main/graphql/<package_name>/schema.json: (-1, 0): names starting with '__' are reserved for introspection
at com.apollographql.apollo3.relocated.com.apollographql.apollo3.ast.IssueKt.checkNoErrors(SourceFile:8)
at com.apollographql.apollo3.relocated.com.apollographql.apollo3.ast.GQLResult.valueAssertNoErrors(SourceFile:1)
at com.apollographql.apollo3.compiler.IncomingOptions$Companion.resolveSchema(SourceFile:146)
at com.apollographql.apollo3.gradle.internal.ApolloGenerateSourcesTask.taskAction(SourceFile:54)

Hi! Are there any types in that schema that start with __? (Don't hesitate to attach the schema to the issue so we can have a look).

Hey @balacognizant , I was wondering if you had any news on this issue?

Closing for now, please don't hesitate to leave a comment on this issue if you have more info


fix: export variable

What is the purpose of this pull request? (put an "X" next to an item)
[ ] Documentation update
[X] Bug fix
[ ] New feature
[ ] Add something to the core
[ ] Other, please explain:

What changes did you make? (Give an overview)
Is there anything you'd like reviewers to focus on?

Codecov Report

Merging #25 (4e772ba) into main (c520a28) will decrease coverage by 0.35%.
The diff coverage is 85.71%.

@@            Coverage Diff             @@
##             main      #25      +/-   ##
==========================================
- Coverage   93.83%   93.48%   -0.36%
==========================================
  Files          12       12
  Lines         211      215       +4
==========================================
+ Hits          198      201       +3
- Misses         13       14       +1

Impacted Files
Coverage Δ

lib/src/core/environment_variables.dart
85.71% <75.00%> (-14.29%)
:arrow_down:

lib/src/core/core.dart
94.20% <100.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 c520a28...4e772ba. Read the comment docs.


Async solutions

@gtklocker @dionyziz It is ok to merge ?

That's one reason we need unit tests to make such refactorings trivial ;)
OK let's do it later.
On Tue, Apr 30, 2019 at 2:01 PM Christos Nasikas<EMAIL_ADDRESS>wrote:

@cnasikas commented on this pull request.
In db/judge/SolidityJudge.js
https://github.com/decrypto-org/blockchain-course/pull/44#discussion_r279705890
:

@@ -140,6 +163,20 @@ class SolidityJudge extends BaseJudge {
hash (...args) {
return this.web3.utils.soliditySha3(...args)
}

promisifyTransaction (tx, args) {

I know man :). Even that is a pain. I have to test it again for each
assignment. So ensureConfirmed or awaitTx ? I don't care. This await
tx.ensureConfirmed() let's leave it (if it is easy and possible) to
another PR
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/decrypto-org/blockchain-course/pull/44#discussion_r279705890,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAEE6PHNEGHDPU6J3COS33DPTARKHANCNFSM4HH6KWTQ
.


Remove Timeline._makeTics

Timeline._makeTics- remove in [1.71.0]

Pushing this along because #8885 (which deprecates this) hasn't even been merged yet.

Since https://github.com/CesiumGS/cesium/pull/8845 wasn't merged I'm going to close this issue.


Minor Update: Fix styling on Lesson 5 page

Fix the styling here so that the highlighted part does not end so abruptly on the right side:

You may need to modify the styling of the actual code comparison component to do so.

This one is good


Update rds-whats-new.md

adding rd ip virtualization on 2019

@jobende : Thanks for your contribution! The author(s) have been notified to review your proposed change.

in case i messed up link - https://docs.microsoft.com/en-us/troubleshoot/windows-server/remote/remote-desktop-ip-virtualization


I am getting an issue that says  cannot load such file -- sass (LoadError) Did you mean?  sassc for rails 6.1 and ruby 3.3

Ignoring ffi-1.9.25 because its extensions are not built. Try: gem pristine ffi --version 1.9.25
Ignoring gem-wrappers-1.4.0 because its extensions are not built. Try: gem pristine gem-wrappers --version 1.4.0
Ignoring msgpack-1.3.3 because its extensions are not built. Try: gem pristine msgpack --version 1.3.3
Ignoring msgpack-1.2.4 because its extensions are not built. Try: gem pristine msgpack --version 1.2.4
Ignoring nio4r-2.5.2 because its extensions are not built. Try: gem pristine nio4r --version 2.5.2
Ignoring nio4r-2.3.1 because its extensions are not built. Try: gem pristine nio4r --version 2.3.1
Ignoring nokogiri-1.10.9 because its extensions are not built. Try: gem pristine nokogiri --version 1.10.9
Ignoring nokogiri-1.8.4 because its extensions are not built. Try: gem pristine nokogiri --version 1.8.4
Ignoring pg-0.21.0 because its extensions are not built. Try: gem pristine pg --version 0.21.0
Ignoring puma-4.3.3 because its extensions are not built. Try: gem pristine puma --version 4.3.3
Ignoring puma-3.12.0 because its extensions are not built. Try: gem pristine puma --version 3.12.0
Your Gemfile lists the gem dotenv-rails (>= 0) more than once.
You should probably keep only one of them.
Remove any duplicate entries and specify the gem only once.
While it's not a problem now, it could cause errors if you change the version of one of them later.
/Users/nealsutaria/.rbenv/versions/3.3.0/lib/ruby/3.3.0/bundled_gems.rb:74:in require': cannot load such file -- sass (LoadError) Did you mean?  sassc 	from /Users/nealsutaria/.rbenv/versions/3.3.0/lib/ruby/3.3.0/bundled_gems.rb:74:in block (2 levels) in replace_require'
from /Users/nealsutaria/.rvm/gems/ruby-2.7.0/gems/bootsnap-1.18.3/lib/bootsnap/load_path_cache/core_ext/kernel_require.rb:17:in `require'

Similar issue on ruby 3.3.4, reverting to 3.2.2 solved it. Will have to dig a little deeper when I find some time for it.


Fix DM row highlight linger in left siderbar.

This PR fixes the behaviour of DM row highlight lingering on view changed.
PR fixes two sub issues

When user navigates away from selected DM row to recents or inbox, the dm remains highlighted because update_private_message builds the dm list again from pm_list_data with is_active set as true for the last DM which should actually be set to false. get_active_user_ids_string() is returning last selected DM row since message_lists.current wasn't updating to return undefined. We fix it by updating current before updating the DM list again.
When the user navigates away to other left sidebar narrows, the DM highlight lingers as the handle_narrow_activated isn't calling update_private_message. We fix it by updating the dm list again on unnarrowed.

Fixes: zulip#27698.

Screenshots and screen captures:
Screencast from 26-03-24 02:01:23 AM IST.webm

Self-review checklist

[x] Self-reviewed the changes for clarity and maintainability
(variable names, code reuse, readability, etc.).

Communicate decisions, questions, and potential concerns.

[x] Explains differences from previous plans (e.g., issue description).
[x] Highlights technical choices and bugs encountered.
[x] Calls out remaining decisions and concerns.
[ ] Automated tests verify logic where appropriate.

Individual commits are ready for review (see commit discipline).

[x] Each commit is a coherent idea.
[x] Commit message(s) explain reasoning and motivation for changes.

Completed manual review and testing of the following:

[x] Visual appearance of the changes.
[x] Responsiveness and internationalization.
[x] Strings and tooltips.
[x] End-to-end functionality of buttons, interactions and flows.
[x] Corner cases, error conditions, and easily imagined bugs.

@timabbott Do you want to review this one first? Please tag someone else otherwise. I haven't tested.

Merged, thanks @pratik-pc!


Restyle project admin email notifications

@scytacki The test failure here doesn't seem related to the small changes I made to the notification emails. Am I misreading it though?


Temporal fix for rworkflow bug

This commit provides a temporal fix to rworkflow by bumping the r version of MacOS and Ubuntu and
fixing a  bug in R/estimateDiversity.R.

Bumping r version for Mac workflow tends to work. I think the same for Ubuntu at least, until the fix in
R/estimateDiversity.R is merged.

Alright, maybe we could go like this as a temporary fix. Can you fix estimateDiversity.R in this PR (change the title also according to that)

@TuomasBorman I think we can finalize this now. Unless, you have more changes in mind

Looks good, but still not working. Update manual pages, let's see if that works

Now there is another error. Can you remove the rworkflows file and then create a new one with rworkflows::use_workflow(). That should update the workflow. Remember to specify R version to 4.4.0 (auto seems not to work)
In the future, create branch directly to mia. It is simpler since then I can also contribute much easier.

https://github.com/neurogenomics/rworkflows/issues/130
Try to put R back to auto. It should work

From the rworkflows issue

One thing that might help is removing the ubuntu-latest rspm line, or replacing it with rspm: ~

Can you try that fix?

Yeah, I tried this. Still failing. I think the issue here is permission to microbiome/mia.git is denied to github-actions[bot]. I'm not quite sure but did we alter the secrets?

From the rworkflows issue

One thing that might help is removing the ubuntu-latest rspm line, or replacing it with rspm: ~

Can you try that fix?

Yeah, I tried this. Still failing. I think the issue here is permission to microbiome/mia.git is denied to github-actions[bot]. I'm not quite sure but did we alter the secrets?

Yes, I noticed that too. However, nothing should have changed. I will check


Bug at JGProgressHUDRingIndicatorLayer code

In JGProgressHUDRingIndicatorLayer.drawInContext(), center is not correct.
CGPoint center = CGPointMake(rect.origin.x + (CGFloat)floor(rect.size.height/2.0f), rect.origin.y + (CGFloat)floor(rect.size.height/2.0f));
This code should be changed to
CGPoint center = CGPointMake(rect.origin.x + (CGFloat)floor(rect.size.width/2.0f), rect.origin.y + (CGFloat)floor(rect.size.height/2.0f));

Please open a pull request where this is fixed in all relevant places.


Update mixer submodule

The fix is verified at: https://autopush.datacommons.org/tools/timeline#place=wikidataId%2FQ1445%2Ccountry%2FIND%2CwikidataId%2FQ22048&statsVar=Count_Person_BelowPovertyLevelInThePast12Months_AsFractionOf_Count_Person%2C__UrbanPovertyLine%2C

The fix is verified at: https://autopush.datacommons.org/tools/timeline#place=wikidataId%2FQ1445%2Ccountry%2FIND%2CwikidataId%2FQ22048&statsVar=Count_Person_BelowPovertyLevelInThePast12Months_AsFractionOf_Count_Person%2C__UrbanPovertyLine%2C

wow, thanks. To confirm, what triggered the auto-push? The base cache build or some code change?

The fix is verified at: https://autopush.datacommons.org/tools/timeline#place=wikidataId%2FQ1445%2Ccountry%2FIND%2CwikidataId%2FQ22048&statsVar=Count_Person_BelowPovertyLevelInThePast12Months_AsFractionOf_Count_Person%2C__UrbanPovertyLine%2C

wow, thanks. To confirm, what triggered the auto-push? The base cache build or some code change?

The fix is verified at: https://autopush.datacommons.org/tools/timeline#place=wikidataId%2FQ1445%2Ccountry%2FIND%2CwikidataId%2FQ22048&statsVar=Count_Person_BelowPovertyLevelInThePast12Months_AsFractionOf_Count_Person%2C__UrbanPovertyLine%2C

wow, thanks. To confirm, what triggered the auto-push? The base cache build or some code change?

The mixer code change https://github.com/datacommonsorg/mixer/pull/453 triggered the autopush.  To confirm, what triggered the auto-push? The base cache build or some code change?

The mixer code change https://github.com/datacommonsorg/mixer/pull/453 triggered the autopush.


Investigate long startup times

Since v1.7.0 the app takes a long time too start, even exceeding the loading screen.
To be useful the app must allow inputs the second it os opened.

Profiling
To investigate reasons the app was run in profile mode on an old Xiaomi Mi A3. To allow for testing startup main.dart was modified to wait for interaction before starting.
main.dart
import 'package:blood_pressure_app/app.dart';
import 'package:flutter/material.dart';

/// Run the [App].
void main() => runApp(const Test());

class Test extends StatefulWidget {
  const Test({super.key});

  @override
  State<Test> createState() => _TestState();
}

class _TestState extends State<Test> {
  bool init = false;

  @override
  Widget build(BuildContext context) {
    if (init) return App();
    return GestureDetector(
      onTap: () => setState(() {
        init = true;
      }), child: Container(width: 100, height: 100, color: Colors.red,));
  }
}

Raw data
CPU usage was profiled with the dart debugger:
CPU-profile

Before first frame
During startup

Frames

During starupt

Evaluation
TODO


Panic at parsing a single file

What happened
thread 'main' panicked at 'called `Option::unwrap()` on a `None` value', src\main.rs:49:54
note: run with `RUST_BACKTRACE=1` environment variable to display a backtrace
error: process didn't exit successfully: `target\debug\rssg.exe -i ".\Silver Blaze.txt/"` (exit code: 101)

Reproduce
cargo  run -- -i "Silver Blaze.txt"

Expected
Program generates a single Silver Blaze.html inside dist

Thanks Andrew this bug is now fixed


Rename percentile + json flag

... so that json is lower camel-case

related to https://github.com/swift-sunshine/swsui/pull/37


Fix invalid URI in path with special chars

Coverage remained the same at 95.288% when pulling cadb42f1c5d61ccb8cf9366f4b2d962464a63acd on jderusse:fix-path-with-special-char into dba71e22e2c14ee1e7f436f73df078ff84505ecd on BigstickCarpet:master.


Add a way to reference another piece of MSML with double brackets

If you had [[Param A]] then in both the markdown file and then MSML it would recognize you are referencing something specific

But do this in a way that if it is in the description and we see a parameter we can flag to the user that there is possibly a parameter not referenced


Add timeout for connection

:tada: This issue has been resolved in version 2.0.2 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Can't access lua documentation

Type: Bug
While hovering on lua function, the pop-up disappeares while trying to click on 'view documentation'
VS Code version: Code 1.71.0 (Universal) (784b0177c56c607789f9638da7b6bf3230d47a8c, 2022-09-01T07:25:38.437Z)
OS version: Darwin x64 21.6.0
Modes:
Sandboxed: No
Remote OS version: Linux x64 5.4.207-126.363.amzn2int.x86_64

Marking this checkbox solved it


Jv rox 16819 e2e test for udp port 0 issue

Description
There have been cases where UDP port 0 is reported in the network graph. This occurs in the connection between collector and kube-dns when the kernel module probe is used. More information can be found here https://issues.redhat.com/browse/ROX-16722
Checklist

[ ] Investigated and inspected CI test results
[x] e2e test fails locally when kernel module probe is used

Testing Performed
Deploy using your favorite method. Set the collection method to KERNEL_MODULE.
cd qa-tests-backend
export ROX_POSTGRES_DATASTORE=true
Follow other steps needed to run e2e tests locally
./gradlew test --tests=NetworkFlowTest
The following is observed
NetworkFlowTest > Verify ports are greater than 0 FAILED
Condition not satisfied:
property.port > 0
|        |    |
|        0    false
protocol: L4_PROTOCOL_UDP
last_active_timestamp {
  seconds:<PHONE_NUMBER>
  nanos: 977963108
}

/test gke-postgres-nongroovy-e2e-tests

The check seem alright, however, I did not completely follow the PR description. Are you adding this test to track the occurrences of UDP 0 to make appropriate changes to collector? OR Collector changes have already been made, and this PR is adding test coverage for the new the path?

The collector issue has not been fixed, but it only occurs for kernel module probes and those are in the process of being removed and are not tested by CI anymore. This test is to ensure that this does not happen with ebpf or core_bpf probes.


[v.2.0] - how to remove points?

Hi,
I've switched to v.2.0 and suprisingly there is no option to remove bullets from line charts.
Is this desired situation? Or this one is just skipped in documentation and works different way?
Honestly I would like expansion of functionality not cuts ;-)
Best regards

@KonradDeskiewicz to remove points, simply remove them from the data you passed into the chart. Then you can call update() and the chart will update.

I wasn't precise enough.. how to turn off dots on chart.
There was
pointDot: false,
in V.1 but it doesn't work anymore.

oh, sorry.
The easiest way is to set the radius of the points to 0. Using the latest from the v2.0-dev branch the points will not draw.
In your config:
{
    elements: {
        point: {
            radius: 0
        }
    }
}

You can set the radius per dataset as well
data = {
    datasets: [{
        radius: 0, // radius is 0 for only this dataset
    }]
}

:) Many thanks :)

helped a lot.


Be less paitent when waiting for a DAG to complete.

Because we're only waiting on ourselves, it's completely reasonable to
poll for completion fairly frequently. It's extremely cheap and it's
not interfering with anybody else.
The ideal solution would be to use an Event that was set, but getting
that to work would be much more invasive.

speling

@thetorpedodog should we merge this to include in the next release?


Add support for requesting only specific fields on details requests

@antoniobg do we want to merge this? or close?


add process result scripts to tasks

Describe your changes
Issue ticket number and link
Closes #xxxx (Replace xxxx with the GitHub issue number)
Checklist before requesting a review

[ ] I have performed a self-review of my code

Check the correct box. Does this PR contain:

[ ] Breaking changes
[ ] New functionality
[ ] Major changes
[ ] Minor changes
[ ] Bug fixes

[ ] Proposed changes are described in the CHANGELOG.md

[ ] CI Tests succeed and look good!

@rcannood WDYT about this ?

also worth mentioning the results workflow still does not contain the auto workflow, which means the state.yaml cannot be used. For some reason it would not work for me. Maybe this can be looked at.

The auto workflow can't be used because it doesn't contain the trace file -- correct?
Since it's just one set of results, that's okay for now.
My only consideration is whether it's worth really running this on NF Tower -- we'd be waiting longer for the instances to spin up than to just run it locally :p I'd be inclined to change the scripts to run it locally, or even in the future add a github action to do the nextflow run and create a PR for the website. WDYT?


preset-env does not convert import of core-js

After removing all dependencies, it's ok to reinstall the dependencies, but the following code runs as if core-js won't parse it
const swc = require("@swc/core");
const output = swc.transformSync(`
import 'core-js'
const a = Promise.resolve('111')
a().then(console.log)
`,
{
  "module": {
    "type": "commonjs"
  },
  "env": {
    "targets": {
      "chrome": "79",
      "ie": "11",
      "safari": "11.1"
    },
    "mode": "entry",
    "coreJs": 3
  },
  "jsc": {
    "parser": {
      "syntax": "typescript",
      "tsx": true,
      "decorators": true,
      "dynamicImport": true
    },
    "target": "es2019",
    "loose": true,
    "transform": {
      "optimizer": {
        "globals": {
          "vars": {
            "__DEBUG__": "true"
          }
        }
      }
    }
  }
}
)
console.log(output.code)
// 'use strict';
// require('core-js');
// const a = Promise.resolve('111');
// a().then(console.log);

Core-js parsing failed
Originally posted by @chenyulun in https://github.com/swc-project/swc/issues/779#issuecomment-630655975

It works with

.swcrc
transformSync + filename

but it does not work with

transformSync without filename

I only tested this scenario. There may be problems with other scenarios. You should test them all, thank you


Get rid of Ended

I'm pretty confident this is a good idea.

Current coverage is 90.12%

Merging #34 into master will increase coverage by +0.71% as of a4dc8f3

@@            master     #34   diff @@
======================================
  Files           16      16
  Stmts          444     415    -29
  Branches        35      35
  Methods          0       0
======================================
- Hit            397     374    -23
  Partial          0       0
+ Missed          47      41     -6

Review entire Coverage Diff as of a4dc8f3

Powered by Codecov. Updated on successful CI builds.


Crashlytics dispatch Rollouts writes async to prevent hangs

https://github.com/firebase/firebase-ios-sdk/issues/12913 reports hangs writing to rollouts.
I was able to reproduce this in a test by putting a long running operation on the FIRCLSGetLoggingQueue(). Root cause is dispatching synchronously for writes to rollouts.
Added a test case to ensure updateRolloutsStateToPersistenceWithRollouts returns to the caller without blocking on the dispatch queue.

We have a ton of flakes in Crashlytics...


Duplicate header UI

Related issues
Resolved #74
Summary

Fix duplicate header UI
Fix lint
Add point & total_time properties to the combo interface

How to test

Verify bug duplicate header UI is fixed in notify & change password screen
Run yarn lint:fix without errors

yarn lint:fix

Concern points
Screenshots

---
---

@KMTus @truongtdph40209 Reivew


Link to CLA gist

Could you give me the link to the original CLA gist used by the CLA bot?

https://cla-assistant.io/ethereumproject/emerald-wallet

I mean the gist.github.com url. I would like to reuse it for one of my other projects if that is okay?

https://gist.github.com/splix/58d4ae5b9edd5727bd9d150780cda1e2

Thank you


[BUG] Publishing dataset fails due to 'exceeds block gas limit' error

Summary
When trying to publish a dataset to the Gen-X network with the given configs a web3 error is thrown.
Current Behavior
The publishing doesn't work and the error 'exceeds block gas limit' is thrown
Expected Behavior
The dataset is published to the DLT and can be seen in the portal.
Steps to Reproduce
Network_Config:
{
network: 'genx',
chainId: 100,
metadataCacheUri: 'https://aquarius510.v4.delta-dao.com',
nodeUri: 'https://rpc.genx.minimal-gaia-x.eu',
providerUri: 'https://provider.v4.genx.delta-dao.com',
subgraphUri: 'https://subgraph.v4.genx.minimal-gaia-x.eu',
oceanTokenAddress: '0x0995527d3473b3a98c471f1ed8787acd77fbf009',
oceanTokenDecimals: 18,
oceanTokenSymbol: 'OCEAN',
fixedRateExchangeAddress: '0xAD8E7d2aFf5F5ae7c2645a52110851914eE6664b',
dispenserAddress: '0x94cb8FC8719Ed09bE3D9c696d2037EA95ef68d3e',
nftFactoryAddress: '0x6cb85858183b82154921f68b434299ec4281da53',
providerAddress: '0x68C24FA5b2319C81b34f248d1f928601D2E5246B',
euroeAddress: '0xe974c4894996e012399dedbda0be7314a73bbff1',
euroeDecimals: 6
}

Pricing_Config:
type: 'fixed',
freCreationParams: {
    fixedRateAddress: config.fixedRateExchangeAddress,
    baseTokenAddress: config.euroeAddress,
    baseTokenDecimals: config.euroeDecimals,
    datatokenDecimals: 18,
    fixedRate: '1',
    marketFee: '0',
    marketFeeCollector: '0x0000000000000000000000000000000000000000'
}

Error message:
\service-pub\node_modules\web3\node_modules\web3-core\lib\commonjs\web3_request_manager.js:258
throw new web3_errors_1.InvalidResponseError(response, payload);
^
InvalidResponseError: Returned error: exceeds block gas limit
at Web3RequestManager._processJsonRpcResponse (\service-pub\node_modules\web3\node_modules\web3-core\lib\commonjs\web3_request_manager.js:258:23)
at Web3RequestManager. (\service-pub\node_modules\web3\node_modules\web3-core\lib\commonjs\web3_request_manager.js:156:29)
at Generator.next ()
at fulfilled (\service-pub\node_modules\web3\node_modules\web3-core\lib\commonjs\web3_request_manager.js:21:58)
at process.processTicksAndRejections (node:internal/process/task_queues:95:5) {
innerError: { code: -32600, message: 'exceeds block gas limit' },
code: 101,
data: undefined,
request: {
jsonrpc: '2.0',
id: '2a351139-c976-447a-8c48-c58e6a1711d0',
method: 'eth_sendRawTransaction',
}
}
Environment

OS: Windows 11
Node: 18.15.0
npm: 9.6.4

Anything else

It's not account dependent. I tried two different accounts for publishing and they both create this error.

Could you provide the web3 and nautilus package versions you are currently using (specified in your package.json)?

Even after reinstalling the packages and using the following versions, the error still occurs
web3 ^4.0.3
nautilus ^0.2.2

Please try "1.9.0" for web3. We are experiencing issues with the other versions. Make sure to remove the remove the old installation. The easiest solution is to delete the node_modules folder and run npm i again.

After experimenting a bit with the right web3 version and the tsconfig, the problem is resolved. Thank you.


Support for Orin NX 16 GB affected by PCN 211361 (kirkstone)

Hi there,
Is it on the roadmap to patch in L4T 35.6 to the kirkstone branch of meta-tegra? I recently became aware of PCN 211361 (a Micron DRAM change on the Orin NX 16 GB), which requires updates to the BCT and DVFS. So far, I've been unsuccessful in getting my distributor to provide a patch with only the necessary changes to support the new DRAM.

Yes, that's the plan. We'll be porting the 35.6 update back to kirkstone after it goes into the scarthgap-l4t-r35.x branch. Don't have a specific timeframe for that, but hopefully within the next several weeks.

Excellent, thank you!

Both the scarthgap-l4t-r35.x and kirkstone branches have been updated to L4T R35.6.

Great, thank you for the update and the hard work on this!

Great, thank you for the update and the hard work on this!


RunTimeError: Caught an unknown exception

I passed the following inputs to BasicLayer:
feats.shape= [2,256,256] ,two batches each one has dimension 256*256 , I have reshaped them to size= [256,512]
xyz.shape=[2,3,256]  reshaped to [3,512]
each batch has two scenes, each scene has 256 points with 256 features for each one  size =[256,256]  for each scene (point cloud)
I putted the  offset =[256,512]  which means that the first point cloud ends at 256, the second at 512.
I got the following error in this line:
pointops_cuda.attention_step1_forward_cuda_v2

what would be the source of this error and how it might be resolved?
thank you so much for your help ..

You should keep feats and xyz to the shape of [N, C] and [N, 3], respectively.

thank you for your reply
feats and xyz have shapes [N, C] and [N, 3] as you mentioned, but no v2 function works with me, all gives exception
still don't know where is the error..


Sweep: I was trying the -i option to improve existing code , instead of the ai modifying it I got a rate limit error

Details
INFO:openai:error_code=rate_limit_exceeded error_message='Rate limit reached for 10KTPM-200RPM in organization org-xxx on tokens per min. Limit: 10000 / min. Please try again in 6ms. Contact us through our help center at help.openai.com if you continue to have issues.' error_param=None error_type=tokens message='OpenAI API error received' stream_error=False
WARNING:langchain.llms.base:Retrying langchain.chat_models.openai.ChatOpenAI.completion_with_retry.._completion_with_retry in 4.0 seconds as it raised RateLimitError: Rate limit reached for 10KTPM-200RPM in organization org-xxx on tokens per min. Limit: 10000 / min. Please try again in 6ms. Contact us through our help center at help.openai.com if you continue to have issues..

Checklist

[ ] gpt_engineer/ai.py

• Identify the part of the code that is responsible for making API calls to the OpenAI API. This is likely where the completion_with_retry method is implemented.
• Understand how the completion_with_retry method works. This will help us understand why the current retry mechanism is not preventing the rate limit error.
• Investigate the retry mechanism. If the delay before retrying is too short, it might not be enough to prevent the rate limit error.
• Consider whether the rate limit could be increased. If the -i option requires more tokens than the current rate limit allows, it might be necessary to request a higher rate limit from OpenAI.
• If increasing the rate limit is not possible or does not solve the problem, consider whether the code could be optimized to use fewer tokens. This could involve making fewer API calls, making smaller API calls, or caching the results of API calls.

Housekeeping: Closing stale issues.


update kubernertes version

I should be able to take this for a spin tomorrow. Thanks for your patience !

I'll need to do a pass and update some other dependencies, so let's get this in for now. Thanks !


Returning False from the EmailAccount.has_object_write_permission disables OPTIONS request object metadata

If we are using DRYRestPermissions and have denied a user 'write' permission so that they can not create an object (but allow them 'update' permission) then the rest api does NOT return the object metadata in response to OPTIONS.
So for now allow True on method == PUT.

Fixed by allowing True when method==PUT


State is not persisted between redirects in Safari

Storing the state and PKCE values in LocalStorage in Safari seems to result in that data not being available when the browser is redirected back to the app after logging in. I suspect this may have something to do with Safari's anti-tracking protection.
May need to replace the LocalStorage parts with storing this state in cookies.

Greetings, can give us a example of storing state in cookies?. Thanks, have a nice day!


Add Chromecast support

It would be good for presenting options to be able to chromecast directly without need of an HDMI or other cables to connect to a display.

if you have an VGA port you can force windows to output to it, and you can stream your desktop with chrome.

+1


Sprin cloud config server- I am trying to clone data in linux machine user defined directory instead of linux default system directory /tmp/config-repo-

As per Spring cloud documentation-
With VCS-based backends (git, svn), files are checked out or cloned to the local filesystem. By default, they are put in the system temporary directory with a prefix of config-repo-. On linux, for example, it could be /tmp/config-repo-. Some operating systems routinely clean out temporary directories. This can lead to unexpected behavior, such as missing properties. To avoid this problem, change the directory that Config Server uses by setting spring.cloud.config.server.git.basedir or spring.cloud.config.server.svn.basedir to a directory that does not reside in the system temp structure.
We are trying to clone the data to user defined location /var/**/configserver/data using below configuration for the same but it doesn't create. Please suggest. To be informed , we are using Java 17, Spring boot 2.7.0 , Spring cloud 2021.0.3 which loads config server and config client dependencies with version 3.1.3
spring:
cloud:
config:
server:
prefix: /config
composite:
-
type : git
uri: git@github.com:xyz/eai-123-{application}${levelname}.git
ignoreLocalSshSettings: true
passphrase: ${PASSPHRASE}
privateKey: ${DEPLOY_KEY}
basedir: /var/abc/configserver/data
clone-on-start: true
-
type: git
uri: git@github.com:xyz/eai-123-common${levelname}.git
bootstrap: true
ignoreLocalSshSettings: true
passphrase: ${PASSPHRASE}
privateKey: ${DEPLOY_KEY}
basedir: /var/abc/configserver/data
clone-on-start: true
Please ignore if pasted configuration here, has any space issues, it may be because of copy paste. My original yml does not have any space issues so is a valid yml.

We no longer support this version of Spring Cloud plus update to Spring Cloud 2023.0.x and try and reproduce the error.  If you can still reproduce the error please provide details on the error you are seeing.

Thanks for your response. I am closing the issue for now as i can't upgrade the spring cloud version for now.


svg2js: add base implementation

Requires #3
The easiest way is to chain svg2xml then xml2js

Closed by #24


Hijri date according to JAKIM

السلام عليكم و رحمة الله و بركاته
I enjoy this app and love the updates. Great job.
I recently realized that the Hijri date it was providing was incorrect. However, I found the Hijri offset option under the app settings.
But I thought: since this app is for Malaysian prayer times, would it not be better to offer a JAKIM sync option so that the Hijri calendar is in accordance with JAKIM?

Yes. I have checked at www.e-solat.gov.my but it seemed they do not provide any Hijri dates there.
Since there are no hijri date information there, this feature is not able to be developed at the moment.


Add Problem Set 2 solution

Adds CUDA solutions (and serial C++ reference solutions) for problem set 2.

Looks good to me! :+1:


Add Javadocs for Monitor Query [WIP]

Revamps Javadocs for Azure Monitor Query.

API change check
API changes are not detected in this pull request.


Gird: Gutters are not present in rows with columns

Is your feature request related to a problem? Please describe.
Nope
Describe the solution you'd like
Currently when implementing columns there are no gutters in the row. There should be a default gutter size at each breakpoint. 10 to 20 pixels.
This is the current implementation, but there should be gutters between each column.

2/3rd space

1/3rd space

Describe alternatives you've considered
Ideally, gutters should be default and adding the is-gutterless class to the row would remove them.
Additional context
None

Interested.


Give us onLoaded and onError hooks and .load() method for Image.network()

Requesting a feature addition to Image.network() for the following

onLoaded hook
onError hook
.load() async method

This will make it very easy to build out widgets that do not rely on third party libraries (a big 👍 of flutter imo) with succinct custom logic for

retry loads
hide loading animations
show error widgets

https://pub.dartlang.org/packages/flutter_image provides at least some of what you request.

You can use ImageProvider, which would get you Image stream to which you can hook the listeners.

And here I am again at 1PM on a Monday reading through Flutter source to figure out how to call something when a Network Image loads or errors... 😒

community handled by flutter_advanced_networkimage
Perhaps Flutter team can consider adding this to master. This is a great package.
https://github.com/mchome/flutter_advanced_networkimage/commit/1345829496bf3273e3a0e4c9bc8e544ab98edcd1

See also https://github.com/flutter/flutter/pull/18424


[WIP] atomic_sequence support: error complaining about non-existant apply f…

I've likely butchered something here, but I think this is a rough idea to get atomic_sequence working.
Currently when trying to compile a regex w/ an atomic sequence I get errors about a missing apply function.

Bit of a note, though not adding any atomic functionality, using this rule:
	static constexpr auto rule(d, ctll::term<'>'>) -> ctll::push<reset_capture, ctll::anything, content_in_capture, ctll::term<'\x29'>>;

Allows (?> syntax to be recognized as a sequence*, may be temporary patch.

Roughly takes care of #108


Thread-safe date formatter

When handling multiple threads, the shared date formatter can provide a result different than expected due to one thread having a different date_format than another one and being performed at the same time.

I've created a per-thread cache for NSDateFormatter.
It was published with 5d5f25abf6fda25358bb55fe25fbad316fa6801d and available via cocoapods as 1.0.5.
Basically I save and retrive the object from inside the NSThread dictionary; this ensure you to get the instance relative to that thread called only once. It should be enough, tell me if you have problems.
private class func cachedObjectInCurrentThread<T: AnyObject>(key: String, create: () -> T) -> T {
	if let threadDictionary = NSThread.currentThread().threadDictionary as NSMutableDictionary? {
		if let cachedObject = threadDictionary[key] as! T? {
			return cachedObject
		} else {
			let newObject = create()
			threadDictionary[key] = newObject
			return newObject
		}
	} else {
		assert(false, "Current NSThread dictionary is nil. This should never happens, we will return a new instance of the object on each call")
		return create()
	}
}

private class func localThreadDateFormatter() -> NSDateFormatter {
	return NSDate.cachedObjectInCurrentThread("com.library.swiftdate.dateformatter") {
		let dateFormatter = NSDateFormatter()
		dateFormatter.dateFormat = "yyyy-MM-dd'T'HH:mm:ss.SSS'Z'"
		return dateFormatter
	}
}


Internet Explorer - jquery error "crud is not defined"

Bug report
What I did:
Open admin panel in Internet Explorer. Go to an entity to see all the results.
What I expected to happen:
See an overview all of the rows of that model in my database.
What happened:
The data didn't show and a jquery error was shown in console. Jquery error: "crud is not defined".

Backpack, Laravel, PHP, DB version:

Laravel version: 5.5
Backpack version: 3.4
PHP version: 7.2

PS. Just merged a patch for IE11 - https://github.com/Laravel-Backpack/CRUD/pull/1306
Please check if after a composer update you're still having the issue.
Cheers!

This bug happen before #1306.
On my side the console got more errors, but in the end, crud wasn't defined.
Tested in IE11 Admin LTE works like Bootstrap 3.3.7 so > 9.
I wish I can forget about IE and old safari.

Hi guys,
Just got to a Windows computer and tested it - worked fine for me:

After a composer update, maybe also try php artisan cache:clear and running in an Incognito Window?
Cheers!


Addressing loc team feedback

based on questions from the localization team, i've added in additional comments

@enricogior can someone approve this?  These are just comments

Great, once CI passes i'll merge.  Updated comment


Typo in build.md

dev-mssql profile:
<database.catalog>watlzdb</database.catalog>
Should be waltzdb instead of watlzdb.
Only a readme file, but this causes issues for new users, since jOOQ doesn't generate any classes because of this.

Thanks will fix it immediately


Add phase banner includes

This PR adds two new includes for the alpha and beta phase banners, using the snippets provided as examples by govuk elements, seen here.

Looks good!


syntax error

i get the following syntax error
rocketman.sh: 35: rocketman.sh: [[: not found
rocketman.sh: 41: rocketman.sh: [[: not found
rocketman.sh: 78: rocketman.sh: Syntax error: "(" unexpected (expecting "}")

Can you try again?  I'm still getting some routing issue.  VPN works fine...just not as a reverse yet.

rocketman.sh: 37: rocketman.sh: [[: not found
rocketman.sh: 43: rocketman.sh: [[: not found
rocketman.sh: 80: rocketman.sh: Syntax error: "(" unexpected (expecting "}")
although i launch it using sh rocketman.sh

I don't believe you are using bash in that case.  You should just launch with ./rocketman.sh since it has the !#/bin/bash

using ./rocketman.sh produces permission denied error

chmod +x rocketman.sh

working great,
if we choose no on get external IP does it pick server ip from the sh file?
i'd like to use my subdomain name instead of external IP

Closing this issue since syntax is OK but I added option to ask for ip/domain.


added a parser for HHsearch outputs.

Addressing issue #3 :
I decided to split the process into four methods:

the complete parsing of the HHsearch output file: parse_pdb_match
the selection of those hits that we want (= e-Value < t & no overlap): select_hits
Since the HHsearch file does not contain the whole query sequence, I split the desired result in two functions:
list of PDB ids and their corresponding query coverage: report_hits
all sequences (n >= 40 residues) not covered by the PDB: report_uncovered_subsequences

Assume f_q is the fasta file holding the query and f_o the HHsearch output for this file, we obtain the information by the following code:
all_hits = parse_pdb_match(f_o)
good_hits = select_hits(all_hits)
res_hits = report_hits(good_hits)
res_uncovered = report_uncovered_subsequences(good_hits, sequence string of f_q)

Coverage increased (+6.08%) to 91.796% when pulling 18fc3a7b3108f8a91623e07d52cd42eff9ee3f11 on sjanssen2:PDBmatch_parser into e93304f74113e2d6880ba712c320b6a01831278b on biocore:master.

Coverage increased (+6.6%) to 92.274% when pulling 27f56304f424b5a03f1c16f4b38bd0338230101a on sjanssen2:PDBmatch_parser into e93304f74113e2d6880ba712c320b6a01831278b on biocore:master.

ready for your reviews @tkosciol @qiyunzhu

Coverage increased (+6.6%) to 92.308% when pulling 07155e1fce147da8ac4502192b55c6a75dc48cc7 on sjanssen2:PDBmatch_parser into e93304f74113e2d6880ba712c320b6a01831278b on biocore:master.


Inputs lacking readonly attribute

There currently does not seem to be any way to add the readonly-attribute to inputs, i.e. dmc.TextInput and dmc.NumberInput.
Dash-Boostrap-Components' dbc.Input supports this as a boolean parameter.
Since there also does not seem to be a method of adding it "manually", it'd be great to have it added similarly to the way boostrap does it.

Added readOnly prop. Will be available in next release.

Are there any workarounds here until that release drops and we have time to migrate?


Plugin doesn't work with latest ILSpy-version

One issue is missing Mono.Cecil.dll;
A bigger problem is a change introduced in this commit: the GetAssemblyDefinitionOrNull was removed.

Thanks for the hint! I will look into this ASAP.


19041.1503 says not listening and not supported

any help with this issue ive been trying for hours to fix this and nothing has helped

Have the same issue (Listener state: Not listening [not supported]) with the latest v.1.6.2 and with latest ini files .\RDPWInst.exe -w.
It seems MS released new version of DLL which we cant wrap.
During the re-installation wrapper found new "Terminal Services version: 10.0.19041.1387"
and reports:
[-] This version of Terminal Services is not supported.
Send your termsrv.dll to project developer for support.

here is the new trmsvr.dll I have.
winver reports: Win10, Version 20H2 (OS Build 19042.1466)

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


fix getter处理中对javaBean is方法的处理逻辑

参考
javabeans约定里（章节8.3.2）
同时参考对比了jackson的处理

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.

Current coverage is 91.62% (diff: 100%)

No coverage report found for master at 053f130.

Powered by Codecov. Last update 053f130...ee0a94f


Cannot catch exception when using Sharding-JDBC route to multiple data nodes

For English only, other languages we will close it directly.
Please answer these questions before submitting your issue. Thanks!
Which version of Sharding-Sphere do you using?
3.0.0.M1
Expected behavior
Can catch SQLException when exception occur
Actual behavior
Cannot catch SQLException when exception occur
Reason analyze
Because Sharding-JDBC use multiple thread to execute SQL, if exception occur, cannot catch it on main thread.

fixed it at #1039


Can't download the all images correctly

I don't know whats the problem

Please fill out the issue template


Interceptors, Mixins, Components, Plugins, ...

Your framework is awesome, but there is a way to setup the app initialization?
Where I can put the Vue.use() sections for interceptors and plugins, or add global mixins and components?
Thank you.
-L

Considered as new feature, please see https://github.com/scriptPilot/app-framework/issues/593


Trying to Install Xpand in Clean Azure VM

After installing Xpsh in a new Azure VM machine, when trying to install Xpand (Install-Xpand cmd), I receive the following error

it works if I use the -version option

there is a dedicated pipeline testing the installer https://dev.azure.com/eXpandDevOps/eXpandFramework/_build?definitionId=43., However it is disconnected at the moment due to the recent CI works. I will see to it, thanks

actually it looks like this issue most probably related to your network try open the firewall

it works if I use the -version option

oops I guess this was not it if you already installed. I get back to u once enab;e the pipeline

the pipeline is now green https://dev.azure.com/eXpandDevOps/eXpandFramework/_build?definitionId=43&_a=summary
added also a badge to monitor its status at the releases page

Closing issue for age. Feel free to reopen it at any time.
.Thank you for your contribution.


chore(master): release 0.18.2

:robot: I have created a release beep boop
0.18.2 (2022-07-06)
Bug Fixes

don't link to latest release (#477) (498294e)
enable desktop update notifications on all platforms (#476) (33fff93)

This PR was generated with Release Please. See documentation.

🐝 PR preview in Swarm
Preview URL: https://bah5acgzaxt6gh422ud7mnw3mayjlwjkjb3wyhne37amporofiqqkf22zpq5a.bzz.link
Swarm Hash: bcfc63f35aa0fec6db6c0612bb25490eed83b49bf818f745c54420a2eb597c3a
Commit Hash: eb5e9fd34567b12deaef1a81117e65a375cfa236
Commit Message: chore(master): release 0.18.2


Explicitly specify type so illustrative example will work out of the box

Fixes #473

Coverage remained the same at 100.0% when pulling c5cb28daddb65f17e3cd7dd7ceaaa779e19c69ac on feature/fix-issue-473 into 9f3385228600c94064a1d834eb259fcef00ea36d on develop.

Coverage remained the same at 100.0% when pulling 5b6f9de47ff8db5c2d78327484dae977c578c40e on feature/fix-issue-473 into 9f3385228600c94064a1d834eb259fcef00ea36d on develop.


Custom string value validation

Use case
When writing cssinjs, currently we can validate only properties and simple values. When a string value is composed out of multiple strings for e.g. {border: '1px solid red'}, we can only tell the value should be a string. What we would like to tell is much more, basically it could use a full CSS value parser.

Given the wide variety of things you could want to check a string for, this is outside the scope of Flow and should be handled in userland.

I was asking for a hook that allows to implement any custom validation in USERLAND. Or is this already possible?

Flow is a static type checker. You can statically validate that a value is a literal, or that a value is a specific type, but you can't statically check the contents of a value.

What if we could have a custom type defined in the user land, which checks whatever is needed, not just primitives typeof check?

You mean you would want to write some sort of OCaml module to do your custom type check?
Can you give an example of how you would want to write and use that?

Something like this:
defineType('CssValue', (value) => {
  if (typeof value === 'string') {
    parse(value)  // throws if value is invalid
  }
})

type style = {
  border: CssValue
}

{
  border: '1px solid red'
}: style

@kof It looks like opaque types in the next flow version. But it will require to use function wich returns such type.
/cc @andreypopp

This would require Flow to actually execute the javascript code it is analyzing. AFAIK that's not on the roadmap.


FreeIPA+Mokey+Hydra docker-compose

If anyone finds it useful we are developing an OpenID based on FreeIPA+Mokey+Hydra easy and quick to set up using docker-compose. You can find it at https://gitlab.com/isard/isard-openid
Many bugs still there to be fixed but it is working now. Feel free to open us issues if you try it.

This looks great! Thanks for sharing.

I have spend some work as well based on this compose file, you can find it here:
https://git.conesphere.cloud/conesphere/podman/hub/mokey
and here https://git.conesphere.cloud/conesphere/podman/hub/freeipa
it is using my own Interpretation of podman container orchestration instead of docker-compose available here https://git.conesphere.cloud/conesphere/podman/tools/podman_tools
I may point out that the registration of the container to ipa does not seem to be necessary at the moment.
As far as I can tell,  the container only needs a valid keytab file, which can be generated on any ipa host, and the ipa ca certificate.


Elastic APM tracer implementation

What does this PR do?
Adding support for Elastic APM tracing system. https://www.elastic.co/products/apm
Motivation
It is an OpenTracing compatible tracer and we are planning to use it.
More

[ ] Added/updated tests
[x] Added/updated documentation

Thanks @amine7536 for you contribution
I tested your PR with an example available here, everything  works well.

Hi, thanks for the approval.
Keep up the good work, Traefik is awesome 💪.


[PROPOSAL] Add custom environment variables to the RunEvent

Hi,
I would like to capture some custom environment variables and send it along with the column lineage to the server. This feature will be of great use as there is a potential use case to capture and persist some environment-specific metadata from spark clusters.
Wondering if this feature can be supported, or is on the roadmap potentially? Thanks a lot!
cc: @pawel-big-lebowski

@pawel-big-lebowski Hi, is there a potential roadmap for this issue? Thanks!

Yeah, this seems to be pretty cool generic feature.
It's great we have an issue for that.
I don't think it's on a roadmap soon, but I would be happy to help other contributors to tackle with this.

Hi @pawel-big-lebowski I have extended the EnvironmentFacet class, to have a CustomEnvironmentFacetBuilder clas to support this use case for my own use. However, I would like to generalize it, to contribute back to the open source. Just bouncing off an idea to see whether this might be appealing. I was thinking if the following flow might work for a more general use case.

Set an env variable "CUSTOM_VARS"='["VAR1","VAR2","VAR3","VAR4"]'
If CUSTOM_VARS is present in env, then CustomEnvironmentFacetBuilder will collect VAR1, VAR2, VAR3, and VAR4 env variables, and pass it onto the start event.
Thanks!


MDNS loopback resolution to the same Matter node (CON-1125)

If you have a light bulb and a light switch on the same Matter bridge (or Matter device), you can't bind them to each other because the loopback Matter binding will fail. I just noticed MDNS_SKIP_SUPPRESSING_OWN_QUERIES which is meant for testing ESP MDNS but it might actually allow Matter loopback binding to work.
I think the reason why the loopback binding fails on ESP32 is because when you put your own node ID into the binding list, the Matter stack can't resolve the node ID back to its own IP address. Loopback MDNS resolution would allow this.
Can this get tested, and if it works, can you promote MDNS_SKIP_SUPPRESSING_OWN_QUERIES to a standard option instead of a test one? Maybe rename it to MDNS_SUPPORT_LOOPBACK_QUERIES.
Why do you want this? Because a light switch should be able to bind to any light bulb, not just ones on a different node.

This needs to get sorted out with the CHIP people. Every bridge and everything using remote sensors has this problem.
Here is the room air conditioner example. Note the two sensors endpoints.

Endpoint 1 has the thermostat cluster which has to have a client cluster for the sensor endpoints. Page 80 of the Device type spec.  You then bind the client cluster to the local end points. This has to be a binding because it is also legal to have remote sensors.
And then there is the bridge case, have a light switch and a light bulb on the same bridge and bind them.
I have been filing issues on this for years but they are ignored. Here is the first of many....
https://github.com/project-chip/connectedhomeip/issues/21626

Any word from the CHIP people about making loopback bindings work?


Issue Template Updates for Figma link added #391

Description
Current Behavior
The project's current issue templates are missing an open invitation link where new contributors can join the community's Figma team and view user interface designs and other UX projects.
Desired Situation
Each template that has a reference to Figma in its resources section should have an invite link added.
This PR fixes #391
Figma invite link added.
Notes for Reviewers
Signed commits

[ ] Yes, I signed my commits.

Let's discuss it on the websites call.
Please add this as an agenda item in the meeting minutes, if you would :)
https://docs.google.com/document/d/1XczAHXVe2FIWPqiF57ospJ43zw5cZQ7ui8mn39v5EvA/edit#heading=h.lohhtewfwima

Let's discuss it on the website call. Please add this as an agenda item in the meeting minutes, if you would :)
https://docs.google.com/document/d/1XczAHXVe2FIWPqiF57ospJ43zw5cZQ7ui8mn39v5EvA/edit#heading=h.lohhtewfwima

Sir @sudhanshutech I am a newcomer and I don't know how to add an agenda what will I talk about it in the meeting.?

@csengineer1990, thanks much, please update rest of the issue templates too

I have made the changes please check and please let me know if anything wrong?

hey @csengineer1990 replace the lines that has a reference to Figma . Right now u added the links again , just replace with the lines which have a reference to Figma because they are missing an open invite link.

ok, sir. I replace it. please check.


if auth is enabled or not, it doesn't seem to be taken into account, but a wrong password is

Expected behavior
If I enable authentication in the server config, I would expect a client to be unable to connect, unless it uses username and password.
Actual behavior
If  I enable authentication in the server config, it correctly detects connection attempts with wrong username or password, refusing to connect, but if I simply disable auth in the client, it can connect anyway to the server, regardless of its auth settings.
Steps to reproduce
I'm configuring a broker server like this (platform: Android):
Server server = BrokerInstance.getBrokerInstance(); //returns a server instance via `new Server()`
final Properties props = new Properties();

props.setProperty(BrokerConstants.HOST_PROPERTY_NAME, "<IP_ADDRESS>");
props.setProperty(BrokerConstants.PORT_PROPERTY_NAME, "1883");
props.setProperty(BrokerConstants.WEB_SOCKET_PORT_PROPERTY_NAME, "8080");

props.setProperty(BrokerConstants.NEED_CLIENT_AUTH, "true");

String username = App.settings.getString(App.getCtx().getString(R.string.mqtt_username), App.default_username);
String password = App.settings.getString(App.getCtx().getString(R.string.mqtt_password), App.default_password);
String sha256hex = sha256Hex(password);
String filename = "password.conf";
File file = new File(App.getCtx().getFilesDir(), filename);

String fileContents = username + ":" + sha256hex;
try (FileOutputStream fos = App.getCtx().openFileOutput(filename, Context.MODE_PRIVATE)) {
    fos.write(fileContents.getBytes());
    props.setProperty(BrokerConstants.PASSWORD_FILE_PROPERTY_NAME, file.getAbsolutePath());
} catch (IOException e) {
    //pass
}

MemoryConfig memoryConfig = new MemoryConfig(props);
server.startServer(memoryConfig);

Let me know if I need to write down also the client configuration (org.eclipse.paho.client.mqttv3).
Minimal yet complete reproducer code (or URL to code) or complete log file
The logs are from my app; I'm not sure what kind of logs are requested.
Here I'm publishing a message on the server's app build (it shares the connection properties with the server, they cannot be different)
Maybe an unusual use case, but I have a "publisher" build of the app that enables the server and a 'publishing only' client
and a "subscriber" build that is receiving messages only.
---------------------------- PROCESS STARTED (6022) for package dentex.mqtt.publisher ----------------------------
17:47:53.018 BrokerServiceConnection dentex.mqtt.publisher    D  Starting MQTT broker service
17:47:53.043 BrokerService           dentex.mqtt.publisher    D  Starting broker notification service
17:47:53.061 Broker                  dentex.mqtt.publisher    D  Using authentication
17:47:53.069 Broker                  dentex.mqtt.publisher    D  -- listing properties --
                                                                 port=1883
                                                                 websocket_port=8080
                                                                 host=<IP_ADDRESS>
                                                                 password_file=/data/user/0/dentex.mqtt.publisher/fi...
                                                                 need_client_auth=true
17:47:54.216 Broker                  dentex.mqtt.publisher    D  MQTT broker service started
17:47:54.216 BrokerService           dentex.mqtt.publisher    D  Thread is running
//publishing a message
17:50:16.910 MessageInterceptor      dentex.mqtt.publisher    I   -> published messages: 1
17:50:17.305 Client                  dentex.mqtt.publisher    I  message 'Epoch is<PHONE_NUMBER>850' published

Now the "subscriber" build of the app, first connecting without authentication
---------------------------- PROCESS STARTED (6777) for package dentex.mqtt.subscriber ----------------------------
17:54:57.109 ClientServiceConnection dentex.mqtt.subscriber   D  Starting MQTT client service
17:54:57.127 ClientService           dentex.mqtt.subscriber   D  Starting client notification service
17:54:57.128 Client                  dentex.mqtt.subscriber   D  Request for a new MQTT client received
17:54:57.130 Client                  dentex.mqtt.subscriber   D  Server URI: tcp://<IP_ADDRESS>:1883
17:54:57.133 Client                  dentex.mqtt.subscriber   D  MQTT client service started
17:54:57.138 ClientService           dentex.mqtt.subscriber   D  Client thread is running
17:54:57.664 Client                  dentex.mqtt.subscriber   V  'test_subscriber' connected to broker
17:54:57.670 Client                  dentex.mqtt.subscriber   D  Subscribing to topic 'DEV'
17:54:58.352 Client                  dentex.mqtt.subscriber   I  message '{"epoch":"Epoch is<PHONE_NUMBER>850","number":1}' received

here, I'm enabling auth preferences, but a wrong password is used
17:55:08.203 SettingsFragment        dentex.mqtt.subscriber   V  onPreferenceChange
17:55:11.154 SettingsFragment        dentex.mqtt.subscriber   V  Reconnect MQTT Client preference clicked
17:55:15.757 Client                  dentex.mqtt.subscriber   D  Disconnecting client
17:55:15.769 Client                  dentex.mqtt.subscriber   D  Reconnecting client
17:55:15.770 Client                  dentex.mqtt.subscriber   D  Request for a new MQTT client received
17:55:15.786 Client                  dentex.mqtt.subscriber   D  Server URI: tcp://<IP_ADDRESS>:1883
17:55:15.787 Client                  dentex.mqtt.subscriber   V  Client connecting using authentication
17:55:16.481 Client                  dentex.mqtt.subscriber   W  Failed to connect

...and a correct password:
18:03:02.267 SettingsFragment        dentex.mqtt.subscriber   V  onPreferenceChange
18:03:07.998 SettingsFragment        dentex.mqtt.subscriber   V  Reconnect MQTT Client preference clicked
18:03:09.838 Client                  dentex.mqtt.subscriber   D  Disconnecting client
18:03:09.839 Client                  dentex.mqtt.subscriber   D  Reconnecting client
18:03:09.840 Client                  dentex.mqtt.subscriber   D  Request for a new MQTT client received
18:03:09.841 Client                  dentex.mqtt.subscriber   D  Server URI: tcp://<IP_ADDRESS>:1883
18:03:09.841 Client                  dentex.mqtt.subscriber   V  Client connecting using authentication
18:03:10.207 Client                  dentex.mqtt.subscriber   V  'test_subscriber' connected to broker
18:03:10.210 Client                  dentex.mqtt.subscriber   D  Subscribing to topic 'DEV'
18:03:10.914 Client                  dentex.mqtt.subscriber   I  message '{"epoch":"Epoch is<PHONE_NUMBER>850","number":1}' received

Moquette MQTT version
0.16
JVM version (e.g. java -version)
11.0.15
OS version (e.g. uname -a)
Android, the version actually doesn't matter, realdevice, or emulator.
This is the emulator: Linux localhost 4.14.175-g6f3fc9538452 #1 SMP PREEMPT Wed Apr 8 17:38:09 UTC 2020 i686

I had a quick look at the code while on the train. The NEED_CLIENT_AUTH property is only evaluated for SSL connections. While at first it may seem sensible to not allow passwords to be sent over non-encrypted connections, this fails to take into account that Moquette may be behind an SSL-terminating proxy. Thus even though Moquette may not know it, the connection may still be secured.
This problem also does not happen when explicitly setting an Authenticator and AuthorizorPolicy using the four-parameter Server.start() method.

Hello. Thanks for reaching out.
Are you saying that this issue can be avoided by setting an Authenticator and AuthorizerPolicy using an alternate Server.start() method?
Would this be a way to explicitly set authentication and authorization policies for the connection, rather than relying on the NEED_CLIENT_AUTH property?

Yes, the alternative start method seems to be a good workaround for this problem for now.
Using the alternative start method you can pass your own implementations of the Authenticator and AuthorizorPolicy. This means you can use any source for your user database, like a relational database, or ldap server. The NEED_CLIENT_AUTH does not need to be set for this to work.

You will agree with me the "NEED_CLIENT_AUTH" doesn't do exactly what it says...
Anyway, thanks for the advise. For the moment I'll skip this one, since it seems a bit too complicated for me.

I think I may have implemented the 3 additional parameters for the Server.start() method.
public static class MySslContextCreator implements ISslContextCreator {
    @Override
    public SslContext initSSLContext() {
        SelfSignedCertificate cert;
        try {
            cert = new SelfSignedCertificate();
        } catch (CertificateException e) {
            throw new RuntimeException(e);
        }

        try {
            Log.d(DEBUG_TAG, "Using certificate: " + cert.cert().getSubjectX500Principal().getName());
            return SslContextBuilder.forServer(cert.key(), cert.cert())
                    .sslProvider(SslProvider.JDK)
                    .build();
        } catch (SSLException e) {
            throw new RuntimeException(e);
        }
    }
}

public static class MyAuthenticator implements IAuthenticator {
    @Override
    public boolean checkValid(String clientId, String username, byte[] password) {
        String passwordString = new String(password, StandardCharsets.UTF_8);
        String un = App.settings.getString(App.getCtx().getString(R.string.mqtt_username), App.default_username);
        String pw = App.settings.getString(App.getCtx().getString(R.string.mqtt_password), App.default_password);
        boolean isValid = un.equals(username) && pw.equals(passwordString);
        Log.v(DEBUG_TAG, "checkValid: "  + clientId + " " + username + " -> " + isValid);
        return isValid;
    }
}

public static class MyAuthorizatorPolicy implements IAuthorizatorPolicy {
    @Override
    public boolean canWrite(Topic topic, String user, String client) {
        return true;
    }

    @Override
    public boolean canRead(Topic topic, String user, String client) {
        return true;
    }
}

doing
server.startServer(memoryConfig, interceptors, new MySslContextCreator(), new MyAuthenticator(), new MyAuthorizatorPolicy());
I noticed that the behavior is the same as before. If I don't pass username and password, configuring the client, I can connect anyway. Basically, the logic in MyAuthenticator is skipped. I can see no logging as
checkValid: CLIENT_ID USERNAME -> false
@hylkevds can you provide any insight?

I think I may have found how to circumvent the issue...
boolean isAuthEnabled = App.settings.getBoolean(...) //get the boolean from the app's settings.
props.setProperty(BrokerConstants.ALLOW_ANONYMOUS_PROPERTY_NAME, String.valueOf(!isAuthEnabled));

...and although settings also props.setProperty(BrokerConstants.NEED_CLIENT_AUTH, String.valueOf(isAuthEnabled)); would make sense, it's irrelevant.

need_client_auth property is used to authenticate the client's TLS certificate.
This means that if the client's certificate could be authenticated (or validated) against the CA authorities or the certificates present in the broker's trust store, it's permitted to open a connect to the server. This doesn't mean that it's authenticated in terms of MQTT protocol user/password credential pair.
That's credentials are instead validated against the authenticator provided, which could reject also if the TLS authentication was successful.

I close it as question.
If you want to contribute describing better this in the doc, feel free to create a PR to https://github.com/moquette-io/moquette/blob/gh-pages


Analysis server terminated (IntelliJ)

Analyzer Feedback from IntelliJ
Version information

IDEA IC-191.6183.87
2.2.0
IC-191.6183.87, JRE 1.8.0_202-release-1483-b39x64 JetBrains s.r.o, OS Linux(amd64) v4.4.0-141-generic, screens 1920x1080

Exception
Dart analysis server, SDK version 2.2.0, server version 1.23.0, error: Exception while using a AvoidShadowingTypeParameters to visit a GenericTypeAliasImpl in CompilationUnitImpl
NoSuchMethodError: The getter 'typeParameters' was called on null.
Receiver: null
Tried calling: typeParameters
#0      Object.noSuchMethod (dart:core/runtime/libobject_patch.dart:50:5)
#1      _Visitor.visitGenericTypeAlias (package:linter/src/rules/avoid_shadowing_type_parameters.dart:65:27)
#2      GenericTypeAliasImpl.accept (package:analyzer/src/dart/ast/ast.dart:6304:20)
#3      LinterVisitor._runSubscriptions (package:analyzer/src/lint/linter_visitor.dart:743:14)
#4      LinterVisitor.visitGenericTypeAlias (package:analyzer/src/lint/linter_visitor.dart:353:5)
#5      GenericTypeAliasImpl.accept (package:analyzer/src/dart/ast/ast.dart:6304:20)
#6      NodeListImpl.accept (package:analyzer/src/dart/ast/ast.dart:9046:20)
#7      CompilationUnitImpl.visitChildren (package:analyzer/src/dart/ast/ast.dart:2446:21)
#8      RecursiveAstVisitor.visitCompilationUnit (package:analyzer/dart/ast/visitor.dart:758:10)
#9      LinterVisitor.visitCompilationUnit (package:analyzer/src/lint/linter_visitor.dart:126:11)
#10     CompilationUnitImpl.accept (package:analyzer/src/dart/ast/ast.dart:2439:49)
#11     LibraryAnalyzer._computeLints (package:analyzer/src/dart/analysis/library_analyzer.dart:290:10)
#12     LibraryAnalyzer.analyzeSync.<anonymous closure> (package:analyzer/src/dart/analysis/library_analyzer.dart:160:11)
#13     _PerformanceTagImpl.makeCurrentWhile (package:analyzer/src/generated/utilities_general.dart:258:15)
#14     LibraryAnalyzer.analyzeSync (package:analyzer/src/dart/analysis/library_analyzer.dart:155:35)
#15     LibraryAnalyzer.analyze.<anonymous closure> (package:analyzer/src/dart/analysis/library_analyzer.dart:91:14)
#16     _PerformanceTagImpl.makeCurrentWhile (package:analyzer/src/generated/utilities_general.dart:258:15)
#17     LibraryAnalyzer.analyze (package:analyzer/src/dart/analysis/library_analyzer.dart:90:43)
#18     AnalysisDriver._computeAnalysisResult.<anonymous closure> (package:analyzer/src/dart/analysis/driver.dart:1438:63)
#19     PerformanceLog.run (package:analyzer/src/dart/analysis/performance_logger.dart:34:15)
#20     AnalysisDriver._computeAnalysisResult (package:analyzer/src/dart/analysis/driver.dart:1416:20)
#21     AnalysisDriver.performWork (package:analyzer/src/dart/analysis/driver.dart:1216:17)
<asynchronous suspension>
#22     AnalysisDriverScheduler._run (package:analyzer/src/dart/analysis/driver.dart:2145:24)
<asynchronous suspension>
#23     AnalysisDriverScheduler.start (package:analyzer/src/dart/analysis/driver.dart:2075:5)
#24     new AnalysisServer (package:analysis_server/src/analysis_server.dart:213:29)
#25     SocketServer.createAnalysisServer (package:analysis_server/src/socket_server.dart:86:26)
#26     StdioAnalysisServer.serveStdio (package:analysis_server/src/server/stdio_server.dart:37:18)
#27     Driver.startAnalysisServer.<anonymous closure> (package:analysis_server/src/server/driver.dart:526:21)
#28     _rootRun (dart:async/zone.dart:1124:13)
#29     _CustomZone.run (dart:async/zone.dart:1021:19)
#30     _runZoned (dart:async/zone.dart:1516:10)
#31     runZoned (dart:async/zone.dart:1463:12)
#32     Driver._captureExceptions (package:analysis_server/src/server/driver.dart:605:12)
#33     Driver.startAnalysisServer (package:analysis_server/src/server/driver.dart:524:7)
#34     Driver.start (package:analysis_server/src/server/driver.dart:427:7)
#35     main (file:///tmp/tmpyaVT9t/dart-2.2.0/dart/pkg/analysis_server/bin/server.dart:12:11)
#36     _AsyncAwaitCompleter.start (dart:async/runtime/libasync_patch.dart:49:6)
#37     main (file:///tmp/tmpyaVT9t/dart-2.2.0/dart/pkg/analysis_server/bin/server.dart:10:10)
#38     _startIsolate.<anonymous closure> (dart:isolate/runtime/libisolate_patch.dart:298:32)
#39     _RawReceivePortImpl._handleMessage (dart:isolate/runtime/libisolate_patch.dart:171:12)

I tried to define a typedef like this (I was experimenting with what's possible, I know this is not valid):
typedef EntanglingTextBox = TextBoxModel & Entanglement<TextBoxModel>;

TextBoxModel is just a normal class... Entanglement is a mixin with an unbounded type parameter.
IntelliJ just went nuts: the analyser kept trying to restart and crash again. I had to turn off all inspections to make it stop doing that.

This has not been reported again since Dart 2.2; please re-open if you see this in a recent version of Dart/Flutter.

This has not been reported again since Dart 2.2; please re-open if you see this in a recent version of Dart/Flutter.


ST layer scales down input and produces wrong output during random iterations

Hi,
(1) I found that the ST layer always scales down input such that it looks like a transformed image on a black canvas.
(2) When the batch size is large (>300) the transformed output contains negative values in alternate iterations. I found this extremely bizarre.
Any help with these issues will help! I'm running on a deadline so a quick response would also be much appreciated.

Hi, @swarnakr,
For (1), you could add additional loss to make the scaling factor in the transformation matrix big. It's possible that ST layer will generate black padding around the images. My experiment shows that this will not affect the final classification on MNIST and CUB dataset.
For (2), sorry that I have no idea. It is irregular to have batch size larger than 64 to processing images I think. Never encounter this problem. I think there should be no negative values if your inputs are all non-negative since ST layer is just doing interpolation.
Bests,
Kaichun

Thanks for your response.
Following up on (2), I find that almost all the affine parameters predicted by the localisation network results in very bizarre transformations (like negative x,y coordinates).
How do you make sure that the affine coordinates are proper? I'm not sure that there is a simple loss function  that can ensure this.
Also, in the original paper by Zisserman et al, they don't mention using any loss on thetas, so do you have any idea what the difference in their implementation might be?
Many thanks,
Swarna

Hi, @swarnakr
Thank you for your interest in my code.
I'm actually not sure about how the original authors produce their results. But I guess the following rules should help:

Make the learning rate for the localization network smaller than the regular. Make it 1e-2 ~ 1e-3 times smaller will help the prediction to change slowly.
You may add some penalty loss for the magnitude of the transformation to make the transformation small and smooth.

Thanks.
Kaichun


new blocks arriving doesnt change confirmed status and enable shuffling of those coins, wallet restart was necessary

@fyookball I hooked into the event emitters which appear to most reliably inform the other parts of the wallet when a transaction gets confirmed.  It doesn't seem to work consistently but the status does update without a restart given enough time.
I'm happy to make further changes to this so that it works more quickly and consistently but not until I get some input from @jbdtky or the other members of the wallet team on how to best do this.  The code base is too big for me to waste more time through trial/error and guessing.  It would take the wallet team seconds to just tell me how to do it.


Specifying a propTable still renders two prop docs

 Issue by FarhadG
Wednesday Aug 24, 2016 at 06:49 GMT
Originally opened as https://github.com/storybooks/react-storybook-addon-info/issues/59

I'm currently exporting the component along with a wrapped version (default export) so that I can document the component in isolation, since the default one that is wrapped with several other higher order functions doesn't provide reference to the propTypes for react-storybook to use.
For example:
export class SampleComponent extends Component {

}

export default compose(...)(SampleComponent);

I'm using the { SampleComponent } import for documentation purposes, however, I get two rendered UIs for the propTables.
"SampleComponent" Component
property    propType    required    default
contextStyles   object  no  {{}}
height  number  no  -
image   string  yes -

"SampleComponent" ComponentNo propTypes defined!

Even though I'm specifying the options for the propTables with { propTables: [SampleComponent], inline: true }, I get both rendered versions. Any way to disable the automatic propTable for the component that is rendered? It seems like that's what's happening: the rendered component's propTypes are also being used, hence the undefined since it's exported after several higher order function wraps.
Perhaps, there's a better way to achieve what I need altogether...
Thank you for your suggestion.

 Comment by carlosrberto
Thursday Jan 26, 2017 at 17:08 GMT

I think propTables should replace the default propTables

 Comment by FarhadG
Monday Jan 30, 2017 at 17:41 GMT

An approach I've found to be quite useful is to simply export the "wrapped" and "unwrapped" component so that I can not only use them to easily test the wrapped component (and its higher order enhancements) but also the bare component but also the ability to document the component with Storybook quite easily.
This resolves the double rendering issue as I use the unwrapped component for that portion of the documentation and can be used to leverage other Storybook functionalities.
I hope this is useful for others that are running into the same issue.

I commented on the outdated original issue, and now that that's been closed, I found this open issue, so I'll copy my comment over in case it's useful for anyone else: in cases where a wrapper is unavoidable (e.g., a story for an SVG component that requires a top-level <svg> wrapper), you can specify a propTablesExclude option to blacklist (all of) the component(s) you don't want prop tables rendered for.


escape error string punct fixes #1033

test added

Looks fine, but it doesn't seem like the test tests the changes, does it?

Never mind, I see it now from issue #1033. LGTM, thank you!


Update Changelog about 1.4

Coverage remained the same at 96.0% when pulling 80d44ab79760eb8ffaa4e4332920b88eb744e554 on JulienItard:update-changelog into 24acc34ced17745ec1972d1b9494f1d8778e300a on intridea:master.

Coverage remained the same at 96.0% when pulling 80d44ab79760eb8ffaa4e4332920b88eb744e554 on JulienItard:update-changelog into 24acc34ced17745ec1972d1b9494f1d8778e300a on intridea:master.

Is there a maintainer here ?


Implement pg_catalog.pg_matviews

Problem Statement
Some tools (see labels) run queries that look for this system view.
https://www.postgresql.org/docs/current/view-pg-matviews.html
Possible Solutions
Implement it empty
Considered Alternatives

For dbt --> use the overrides from https://community.cratedb.com/t/using-dbt-with-cratedb/1566

Metabase explicitly looks for the view under pg_catalog

CREATE TABLE pg_catalog.pg_matviews (
	schemaname TEXT,	matviewname TEXT,	matviewowner TEXT,
	tablespace TEXT,	hasindexes BOOLEAN,	ispopulated BOOLEAN,
	definition text	);

--> SQLParseException[Cannot create relation in read-only schema: pg_catalog]

Relates to https://github.com/crate/crate/issues/10661

Hello @hlcianfagna!
I am very interested in this task. Can I take this on?

@julant7 Sure, I've assigned it to you. Thank you!


Exceptions raised because of base_color

./logs/app.log:7939:[2024-03-14 09:50:51] DEBUG [django.template._resolve_lookup:925] Exception while resolving variable 'base_color' in template 'socialaccount/login.html'.
./logs/app.log:7946:KeyError: 'base_color'
./logs/app.log:7954:AttributeError: type object 'RequestContext' has no attribute 'base_color'
./logs/app.log:7962:ValueError: invalid literal for int() with base 10: 'base_color'
./logs/app.log:7969:django.template.base.VariableDoesNotExist: Failed lookup for key [base_color] in [{'True': True, 'False': False, 'None': None}, {'csrf_token': <SimpleLazyObject: 'abc123'>, 'request': <ASGIRequest: GET '/accounts/google/login/?process=login&next=%2Fsign-up-proxy%2F'>, 'user': <SimpleLazyObject: <function AuthenticationMiddleware.process_request.<locals>.<lambda> at 0x>>, 'perms': PermWrapper(<SimpleLazyObject: <function AuthenticationMiddleware.process_request.<locals>.<lambda> at 0x>>), 'messages': <FallbackStorage: request=<ASGIRequest: GET '/accounts/google/login/?process=login&next=%2Fsign-up-proxy%2F'>>, 'DEFAULT_MESSAGE_LEVELS': {'DEBUG': 10, 'INFO': 20, 'SUCCESS': 25, 'WARNING': 30, 'ERROR': 40}}, {}, {'provider': <allauth.socialaccount.providers.google.provider.GoogleProvider object at 0x>, 'process': 'login'}, {'block': <Block Node: content. Contents: [<TextNode: '\n
, <Block Node: whitebox. Contents: [<TextNode: '\n        '>]>, <TextNode: '\n
\n
\n    '>]>}, {'block': <Block Node: whitebox. Contents: [<TextNode: '\n
, <IfNode>, <TextNode: '\n
\n
\n<form meth'>, <django.template.defaulttags.CsrfTokenNode object at 0x7f8e2dc6f990>, <TextNode: '\n  '>, <IncludeNode: template=<FilterExpression '"account/_button.html"'>>, <TextNode: '\n</form>\n'>]>}, {'text': 'Continue'}]

Seems like there is a "with base_color" in account/_button that might be the culprit, but I don't see why it won't just default to bg-blue if it's not found.
Should I set the base_color, if so how?
Feels like I'm missing something and this should be a quick one, @danihodovic.

What page is this happening on?

I can have a look at this next month as I'm currently busy with work. If you figure it out open a PR

I see, and that is totally understood.
If it's a non-obvious error I will investigate further and open a PR if I find something of course :+1:

It's not a big code base. If you can navigate allauth this is a piece of cake. If I find it in the meantime I will update you.


Add steps for installation from source in README

Added section in README to have installation steps for various methods like brew, from source, from binary release, etc..
And this PR adds steps to install from source.

As we are gonna add multiple ways to install, we cannot have stevedore --help images after each section. I feel, stevedore --help command is enough to verify. Thoughts ?

@dineshba
we can have separate sub section for verify installation and have this image there.
or i am fine with removing the image as well

@jskswamy @dineshba I removed the image now, please review it again and merge the PR

looks good to me @prabhu43 @dineshba we can merge

Merged


Support pulumi-kubernetes >= 4.0.0

Hello!

Vote on this issue by adding a 👍 reaction
If you want to implement this feature, comment to let us know (we'll work with you on design, scheduling, etc.)

Issue details
This repo only supports pulumi-kubernetes < 4.0.0, which is now very old, pulumi-kubernetes is on 4.11.0 already. I cannot use this package without support for the latest pulumi-kubernetes, since I do not want to downgrade my pulumi-kubernetes package.
Affected area/feature
pulumi-kubernetes integration

Is this repository still supported? cc @mikhailshilkov


Auto-import confuses package and class with same name and different case

Environment data

Language Server version: 2020.7.1
OS and version: 10 (18363)
Python version (& distribution if applicable, e.g. Anaconda): 3.6

Expected behaviour
Using PyLance "Quick Fix" feature on the following code should suggest to add from abc import ABC:
class myclass(ABC):
    pass

Actual behaviour
Suggested imports all reference package abc, if it is clicked, the written code is changed! (IMO an "auto-import" quick-fix should never modify the code from which it was activated, it's very confusing).
Code Snippet / Additional information

For clarity, the Pylance is fuzzy in these quick fixes, showing instances where the case may not match in case you have misspelled a module or similar. It's doing the right thing in that it's offering you things and replacing them on selection; if you click on a quick fix it's going to fix the code.
The core issue is that the quick fix import abc is not the right fix for this code, from abc import ABC is, but we don't yet have support for fixes that involve variables inside of unimporteed modules (only module names themselves).

should work in next version.

This issue has been fixed in version 2020.9.0, which we've just released. You can find the changelog here: https://github.com/microsoft/pylance-release/blob/master/CHANGELOG.md#202090-3-september-2020


Changed to print the directory name according to the l10n language.

Fixed: #18472

/cc @gochist

/lgtm

/assign @kbhawkey


chore: optimize argoserver init

Fixes #TODO
Motivation

Modifications
optimize argoserver initialization to simplify getting environment variables

Verification

Closing stale PRs