[fix][broker] Fix MultiRoles token provider NPE when using anonymous clients

Motivation
2023-10-24T01:39:33,477+0000 [pulsar-io-4-2] WARN  org.apache.pulsar.broker.service.ServerCnx - [/<IP_ADDRESS>:39414] Got exception java.lang.NullPointerException: Cannot invoke "org.apache.pulsar.broker.authentication.AuthenticationDataSource.hasDataFromCommand()" because "this.authData" is null
        at org.apache.pulsar.broker.authentication.AuthenticationDataSubscription.hasDataFromCommand(AuthenticationDataSubscription.java:37)
        at org.apache.pulsar.broker.authorization.MultiRolesTokenAuthorizationProvider.getRoles(MultiRolesTokenAuthorizationProvider.java:153)
        at org.apache.pulsar.broker.authorization.MultiRolesTokenAuthorizationProvider.authorize(MultiRolesTokenAuthorizationProvider.java:201)

Documentation

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

Codecov Report

Merging #21429 (38c2db1) into master (618aede) will increase coverage by 43.07%.
Report is 1 commits behind head on master.
The diff coverage is 100.00%.

@@              Coverage Diff              @@
##             master   #21429       +/-   ##
=============================================
+ Coverage     30.15%   73.23%   +43.07%
- Complexity      324    32571    +32247
=============================================
  Files          1709     1888      +179
  Lines        130685   140309     +9624
  Branches      14245    15421     +1176
=============================================
+ Hits          39414   102752    +63338
+ Misses        85327    29472    -55855
- Partials       5944     8085     +2141

Flag
Coverage Δ

inttests
24.19% <0.00%> (+0.05%)
:arrow_up:

systests
24.67% <0.00%> (-0.06%)
:arrow_down:

unittests
72.53% <100.00%> (?)

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

Files
Coverage Δ

...authentication/AuthenticationDataSubscription.java
64.28% <100.00%> (+18.13%)
:arrow_up:

...rization/MultiRolesTokenAuthorizationProvider.java
59.59% <100.00%> (+59.59%)
:arrow_up:

... and 1509 files with indirect coverage changes

relates to #21338


Repeated Chat Notification trigger on world hop

Describe the bug
With Chat Notifications and Chat History: Retain Chat History enabled, if a notification word is sent to chat, the chat notification is triggered every time you hop worlds. This only occurs when the trigger for the chat notification is one word.
To Reproduce
Steps to reproduce the behavior:

Enable Retain Chat History within Chat History
Trigger a single word Chat Notification.
Hop worlds or log out.
On login the chat notification triggers again.

Expected behavior
I would expect to not be spammed by the same chat notification every time I hop worlds, while still being able to see chat history.
Screenshots
Original trigger:

After world hop:

Environment:

OS: MacOS
RuneLite version: 1.6.16
Launcher version: 2.1.2

fixed in e71b18b02b124bff69cc347238551d2e0d3bf29b


[3.x] Set directory dialog folder icon color

Fixes #53132 for 3.x.

Please make sure to make a PR for master too, as this applies to both versions.

Okay, I'm building master now.


to test the the functionality of log analytics workspace.

Script has been added to test the log analytics workspace

not required as of now.


Detect visual studio installation using VSSetup module

Checklist

[X] npm install && npm run lint && npm test passes
[X] tests are included
[X] documentation is changed or added
[X] commit message follows commit guidelines

Description of change
Detect visual studio installation using VSSetup module and Get-VSSetupInstance method as a first attempt, this approach works even in systems with Constrained language mode of the powershell.
Original type-base detection requires FullLanguage mode, which quite often not available on the enrolled devices.

Verified tests on both windows and linux now

Sry last linting issue also fixed. Btw mind to add .eslintrc into the project? VSCode not scanning automatically it looks

@cclauss any other activities or blockers to be done for the PR merge?

@jarig my PR is ready for landing. Do you have time to address the feedback I gave? If you are busy or unavailable, I can take over and make the required changes.

@jarig my PR is ready for landing. Do you have time to address the feedback I gave? If you are busy or unavailable, I can take over and make the required changes.

I'll try to make and finalize changes this week

I plan to land this tomorrow. If there are any concerns, please state them today. cc @lukekarrys

Good!


Typo in ScrollView documentation

There seems to be a typo here in scroll view documentation for the keyboardShouldPersistTaps prop. In particular, the sentence describing the "handled" option.
Current:
"the keyboard will not dismiss automatically when the tap was handled by a children"
Proposed change:
"the keyboard will not dismiss automatically when the tap was handled by children of the scroll view"
Link to the documentation

Fixed, thanks!


[5pt] FIM <IP_ADDRESS> - Branch errors

There were some errors in the <IP_ADDRESS> BED run;
The logs for all can be found in EFS /outputs/fim_4_4_15_0

NextDownID of -1  (affected 4 branches)_
Non-branch zero, loading data for test ...
A catchment geom file and only one NextDownID of -1 were found, testing for backpool criteria...

TIFFReadDirectory:Failed to read directory at offset 51997448: This might be an anomoly of just data run-time collisions. It is worth a small test but if you can't replicate it, don't worry about it.

The non_zero_exit_codes.log - branch error count jumped back up to appx 150 again.

At least some of these are probably being caused by #1096.

1 on your list is a known bug being tracked in #1088


feat: add plausible community edition template

Added Plausible CE template (see repo).
Note: Used the generators:

$SERVICE_REALBASE64_64_xxx and
$SERVICE_REALBASE64_32_xxx

in /bootstrap/helpers/shared.php to generate the necessary BASE64 environment variables. Not sure if these generators are part of the public API yet as they are not discussed in Coolify docs.

Closed due to trademark issues.

Closed due to trademark issues.

Unfortunately, yes.


Header Translation Links

Hi there. I think there is a need to unify the approach to language selection dropdown in site header. Currently Japanese version offers Japanese names of languages (which makes it literally impossible for a random non-Japanese speaker to find his way to English or other version), Chinese doesn't seem to offer any link at all, while Russian spells the very "Translations" menu element in English. That being said, I guess we should find some universal solution and apply it to all translation sites in order to achieve cross-language uniformity.
I propose to:

use both current document's language and English spellings of the word "Translations" for the caption
use both endonym and current document's language name versions

So it will be like that for Japanese:
Translations (言語)

English (英語)
普通话 (中文)
Русский (ロシア語)

Before investing too much energy into this, are we sure it would be solving a real problem? Are there currently users finding themselves in Japanese/Chinese/Russian translations without understanding the language?

People can survive without it, it's for sure. But there's not just bread but also beauty in this world, right?
Also, I don't see if it really requires too much time or energy. Just approve the approach, so I'll spend another 15 minutes on actual PRs, and it's pretty much done.

I think there is not especially a need to unify the approach to language selection dropdown in site header. In translation for japanese users, the reason why not "translations (翻訳)" but "languages (言語)", it easily understand as japanese users.

"Translations" vs. "Languages" isn't important. "I can't read Japanese" vs. "I can't read Japanese, but there's some English" is.

I can understand the desire and I appreciate you bringing it up, but I'm OK with translation maintainers making different decisions here. There may even be reasons to maintain different styles for different cultures. Since it sounds like users are not negatively affected by the differences, I think I just have to veto this.


Test suite compilation failure (probably QuickCheck 2.10)

[7 of 7] Compiling Main             ( test/Spec.hs, dist/build/flat-test/flat-test-tmp/Main.o )

test/Spec.hs:324:58: error:
    Ambiguous occurrence ‘getSize’
    It could refer to either ‘Data.Flat.getSize’,
                             imported from ‘Data.Flat’ at test/Spec.hs:20:1-26
                             (and originally defined in ‘Data.Flat.Class’)
                          or ‘QC.getSize’,
                             imported from ‘Test.Tasty.QuickCheck’ at test/Spec.hs:41:1-44
                             (and originally defined in ‘QuickCheck-<IP_ADDRESS>:Test.QuickCheck.Gen’)
    |
324 |       sz v e = [testCase (unwords ["size of",sshow v]) $ getSize v @?= e]
    |

This was fixed a long time ago, sorry for not closing this issue sooner.


Update discord stable to 2.1.2

W171 and W170 will be fixed in next release as in my opinion it's not worth creating a new release for this.
W505 -> Adapter.setTimeout(...) is used, so no clearTimeout(...) should be needed.

2.1.2 created 04.09.2023
OK


Add keywords to improve discoverability

It's difficult to find electron-builder on npmjs.com. For example if you search for electron installer, it's not even listed.
I added a bunch of keywords that users might use when trying to find a solution for building/generating Electron installers. It should improve the discoverability of this project.

Cool thanks. :+1:
Quick question: should msi be nsi?

I meant msi. It's a popular installer format for Windows. If someone's looking for it with some other keywords, it probably means he/she is looking to create an installer for an Electron app on Windows.

Okay fair enough. :)
Thanks. :bowtie:


[Feature]: Using zotero.lua with Quarto/qmd in RStudio

Hi, love your work which has worked well with pandoc on md documents through terminal.
I've been learning to use qmd/Quarto documents as part of my PhD research as I hope to learn both R & Python.
I was following a similar topic about using zotero.lua on RMarkdown documents. https://github.com/retorquere/zotero-better-bibtex/issues/1882
Is something similar possible for Quarto documents? Ideally just being able to hit "render" on RStudio with qmd documents to use zotero.lua somehow and produce a docx that has live zotero citations.
I've tried adapting @alexflaris & @retorquere yaml headers in the qmd document as below, though no luck in producing live zotero citations
---
title: "testing"
format: docx
output:
  bookdown::word_document2:
    pandoc_args: ["--lua-/Users/matthewkang/Downloads/zotero.lua"]
zotero:
  library: "/Users/matthewkang/Zotero/citekeys_My_Library.bib"
  client: zotero
editor: visual
---

Thank you in advance.

I'm not familiar with how quarto works but it looks like pandoc isn't being ran on compile, and the Lua filter only works in pandoc.

Thank you for the timely response @retorquere
My basic understanding of quarto is that it uses knitr and pandoc, with capacity to incorporate custom Lua filters.
Had a little more progress with the yaml fields below rendered in RStudio
---
title: "rmd_test"
output:
  word_document:
    pandoc_args: ["--lua-filter=/Users/matthewkang/Downloads/zotero.lua"]
date: "2023-04-25"
zotero:
  client: zotero
  csl-style: apa
  author-in-text: false # when true, enabled fake author-name-only cites by replacing it with the text of the last names of the authors
  scannable-cite: false # only relevant when you're compiling to scannable-cite .odt
bibliography: "/Users/matthewkang/Zotero/MyLib.bib"
---

But getting this error, which I presume is the custom zotero.lua & the main quarto lua filters conflicting.
zotero-live-citations f47be20
Error running filter /Applications/quarto/share/filters/main.lua:
/Users/matthewkang/Downloads/zotero.lua:81: attempt to index a nil value (global '_G')
stack traceback:
[C]: in upvalue 'orig_require'
[string "..."]:1353: in function 'require'
/Users/matthewkang/Downloads/zotero.lua:1731: in local 'chunk'
/Applications/quarto/share/filters/main.lua:4073: in local 'callback'
[string "..."]:1789: in function <[string "..."]:1787>
(...tail calls...)
/Applications/quarto/share/filters/main.lua:368: in function 'make_wrapped_user_filters'
/Applications/quarto/share/filters/main.lua:15357: in main chunk
Putting this out there to see if there's anyone else using qmd who've managed to incorporate your lua filter in their document rendering. For now, I'll stick with RMarkdown!

That does seem to be some kind of interference between the quarto filters and my filter. But since this invocation is being orchestrated by Quarto, I think it's better to ask the question there.

Hi @kang0410 , I think you might check how quarto deals with pandoc filters, hope it will help https://quarto.org/docs/extensions/filters.html

If a command line invocation of zotero.lua doesn't work I can look into that, but if quarto runs it's filters differently than that, I'll need directed instructions on how to accommodate that in Zotero.lua.

@retorquere I was pointing @kang0410 to some of the  quarto resources, but it seems that the issue was a bug that was solved in quarto shortly after the issue was posted here, I dont think that @retorquere  you need to do anything here


[Bug] Cannot change DB creds for primary shard

Bug report
I'm trying to change the credentials for my primary shard. I have the following config
[general]
query = "readonly.*"

[store]
[store.primary]
connection =<EMAIL_ADDRESS>pool_size = [
    { node = "indexer.*", size = 30 },
    { node = "readonly.*", size = 80 }
]
weight = 0
[store.primary.replicas.repl1]
connection =<EMAIL_ADDRESS>weight = 1
[store.secondary]
connection =<EMAIL_ADDRESS>pool_size = [
    { node = "indexer.*", size = 100 },
    { node = "readonly.*", size = 80 }
]
weight = 1

and when I change it to the following (note that root was replaced with admin for the primary shard but nothing changed for the secondary shard)
[general]
query = "readonly.*"

[store]
[store.primary]
connection =<EMAIL_ADDRESS>pool_size = [
    { node = "indexer.*", size = 30 },
    { node = "readonly.*", size = 80 }
]
weight = 0
[store.primary.replicas.repl1]
connection =<EMAIL_ADDRESS>weight = 1
[store.secondary]
connection =<EMAIL_ADDRESS>pool_size = [
    { node = "indexer.*", size = 100 },
    { node = "readonly.*", size = 80 }
]
weight = 1

I get an error about the admin user not existing when running migrations.
Relevant log output
❯❯❯ klf proofofplay-query-graph-node-656f87688b-bkmqz
May 24 09:14:13.317 INFO Graph Node version: 0.29.0 (2022-12-05)
May 24 09:14:13.317 WARN GRAPH_POI_ACCESS_TOKEN not set; might leak POIs to the public via GraphQL
May 24 09:14:13.317 INFO Reading configuration file `/etc/graphnode.toml`
May 24 09:14:13.318 WARN No fork base URL specified, subgraph forking is disabled
May 24 09:14:13.318 INFO Starting up
May 24 09:14:13.318 INFO Trying IPFS node at: http://mercury-ipfs-ipfs-service:5001/
May 24 09:14:13.325 WARN Expensive queries file not set to a valid file: /etc/graph-node/expensive-queries.txt
May 24 09:14:13.326 DEBG Cleaning up large notifications after about 300s, channel: store_events, component: NotificationListener
May 24 09:14:13.329 INFO Successfully connected to IPFS node at: http://mercury-ipfs-ipfs-service:5001/
May 24 09:14:13.343 INFO Connecting to Postgres, weight: 0, conn_pool_size: 80, url<EMAIL_ADDRESS>pool: main, shard: primary
May 24 09:14:13.344 INFO Pool successfully connected to Postgres, pool: main, shard: primary, component: Store
May 24 09:14:13.344 INFO Connecting to Postgres (read replica 1), weight: 1, url<EMAIL_ADDRESS>pool: replica1, shard: primary
May 24 09:14:13.344 INFO Pool successfully connected to Postgres, pool: replica1, shard: primary, component: Store
May 24 09:14:13.344 INFO Connecting to Postgres, weight: 1, conn_pool_size: 80, url<EMAIL_ADDRESS>pool: main, shard: secondary
May 24 09:14:13.345 INFO Pool successfully connected to Postgres, pool: main, shard: secondary, component: Store
May 24 09:14:13.346 DEBG Using postgres host order [ReadOnly(0)], shard: primary, component: Store
May 24 09:14:13.346 DEBG Using postgres host order [Main], shard: secondary, component: Store
May 24 09:14:13.356 INFO Setting up fdw, pool: main, shard: secondary, component: ConnectionPool
May 24 09:14:13.356 INFO Setting up fdw, pool: main, shard: primary, component: ConnectionPool
May 24 09:14:13.362 DEBG Release migration lock, pool: main, shard: primary, component: ConnectionPool
May 24 09:14:13.363thread 'tokio-runtime-worker ' panicked at 'migrations failed: store error: user mapping for "admin" does not exist for server "shard_secondary"CRIT',  store/postgres/src/connection_pool.rsmigrations failed:976:13,
errornote: run with `RUST_BACKTRACE=1` environment variable to display a backtrace
: store error: user mapping for "admin" does not exist for server "shard_secondary", pool: main, shard: primary, component: ConnectionPool
thread 'main' panicked at 'called `Result::unwrap()` on an `Err` value: JoinError::Panic(...)', /graph-node/store/postgres/src/connection_pool.rs:484:10
May 24 09:14:13.368 INFO Running migrations, pool: main, shard: secondary, component: ConnectionPool
May 24 09:14:13.369 INFO Migrations finished, pool: main, shard: secondary, component: ConnectionPool
May 24 09:14:13.371 DEBG Release migration lock, pool: main, shard: secondary, component: ConnectionPool
May 24 09:14:13.372 DEBG Setup finished, setup_time_s: 0, pool: main, shard: secondary, component: ConnectionPool

IPFS hash
No response
Subgraph name or link to explorer
No response
Some information to help us out

[ ] Tick this box if this bug is caused by a regression found in the latest release.
[ ] Tick this box if this bug is specific to the hosted service.
[X] I have searched the issue tracker to make sure this issue is not a duplicate.

OS information
Linux

I tried creating an admin user in the secondary shard with the same password as in the primary shard but still got that issue. Here's the command I used to make the user
create role admin in group rds_superuser password '<password>' valid until 'infinity';

Seems like this statement is erroring out because I'm changing the current_user from root to admin which means that the prior entry having an entry for root can no longer be altered.

I also tried running drop server shard_secondary cascade and then restarting my graph node to see if the user mapping would be successfully recreated with the correct options. Unfortunately I still see a null value for the umptions column for the admin user.

looks like graphman database remap solves this.

I had to manually run
CREATE USER MAPPING FOR root SERVER shard_second OPTIONS (user 'root', password '<password>');


#31 Registration has to be reworked so that students can choose from a school when registering

Teacher and Students have to select a school now in order to register. Therefore I had to rework the auth controller. Teachers now have to register with the right combination of license code and school name. Also school name is unique and therefore I use the name as key and value.


As an author, I want a method to view the public posts from other nodes

Add an "Explore" page to pull in the public posts and users from the Nodes we are registered with

"Explore" tab in addition to the "Stream" on the homepage. Which displays public posts from other nodes.

Let's chat about this one when you are online.

Let's chat about this one when you are online.

Sure! I've just pushed a branch (FRNT/135-Explore-Tab) that splits the homepage into two tabs, 'Stream' and 'Explore'. The Explore tab pulls foreign posts via an ajax call to a new api endpoint, /posts/foreign


Is it possible to add onnxruntime-gpu?

Comment:
I'm trying to add a package, but it specfically asks for onnxruntime-gpu.

Nevermind, I found I simply had to add  *=*_cuda


Upwind and other type of averaging schemes

Dear all,
I've noticed that there is an UpwindAD implementation for TPFA. However, as pointed out in the comments, the upwinding process is actually independent of the tpfa object (only the information of which faces are Dirichlet faces is used, for example). I was wondering if there is a plan to have a unified declaration of these classes, at least for all FV methods.
One option would be to migrate the averaging methods to ad/discretizations.py. Or maybe create a separate file ad/averagingSchemes.py (?) to store these classes and make them usable for both TPFA and MPFA.
Any thoughts or concrete plans?

@jhabriel Not sure what you mean by Upwind tpfa - while the upwind scheme certainly is reminiscent of Tpfa in terms of the stencil, it is a separate discretization method, with its own implementation. So, if you want an Ad discretization of a flux term that involves rel perms, you would need both UpwindAd to deal with the rel perm, and a flux discretization (tpfa or mpfa) for the K*grad(p) term.
Do I understand you correctly that you want different averaging methods (one-sided for Upwind, harmonic for tpfa, arithmetic because that is useful - note that the latter can easily be made from g.cell_faces) available in the Ad framework? I'm not sure if I see the need for that - are there other cases than the discretizations where you need averaging over faces? Note that Mpfa would not fit in such a scheme, because the averaging is non-trivial for that method.

Thanks for the answer @keileg. Yes, perhaps I was not clear enough (or I'm a bit confused). To my understanding, the unwinding process only "takes care" of the mobilities (or rel perms), while the fluxes are discretized as in the single-phase case. Then, the advective flux is the product between the upwind mobilities and the (single-phase) fluxes. Assuming what I wrote is correct, I was wondering if it would be a good idea to build the UpdwindAd class independently on the discretization, and ask for the single-flux as an input to obtain the directionality.
I can see now that MPFA won't fit the scheme if the upwind process is based on the potential. But it would be essentially the same for TPFA and MPFA in the case of the flux-based scheme. Is this correct?

I'll close this now.


Missing Getting Started Step

Hey guys,
Just FYI, the README and User Guide are missing a critical step: How to get Create React App
If this is no longer a necessary step due to npx magic, then please be explicit about that. Step 0 for anyone who installs open source tools on a daily basis is parsing the README & Docs for installation instructions.
Thanks,
Michael

I think You can Learn On Udemy, Best Online Course For React and Get Started with creating your first React App.

Hi @vinay72,
I've been using Create React App since it was created. Its not a matter of learning the tool. Its a matter of having complete documentation for people who first visit the repo.

English is my second language, yet I managed to find that line. Do you want it to be even more emphasized?

@miraage

English is my second language, yet I managed to find that line

The line you're pointing to is about npx, not about create-react-app 😃. That line makes sense to you because you already know what NPX is. Pretend you don't know what NPX is. Then view the README and Getting Started Guide from that perspective. Make sense now?
I send a lot of junior people to this repo as good starting point for new react projects and I've heard the question "How do I get create react app" multiple times in the last few days alone. It's not immediately obvious that you don't need to "get create-react-app", that npx handles it for you. Thats what this issue is about.
I'll submit a pull request to address the issue.

@iMerica yep, I totally understand your point. What do you think on making this link instructions for older npm versions more visible to users? Or you plan to just add another link in the top of README?


Setup CI/CD

Need CI/CD to build, test, and deploy.
Most importantly, when the backporting to Json.NET 11.x.x and 10.x.x has started, this CI/CD will have to compile the project towards all the different LTS versions, while only maintaining a single code base.
Perhaps best done with .sln Configurations ?
Thinking of going with CircleCI for this one as well.

Fixed with #6


Update sbt, sbt-dependency-tree to 1.9.3

About this PR
📦 Updates

org.scala-sbt:sbt
org.scala-sbt:sbt-dependency-tree

from 1.9.2 to 1.9.3
📜 GitHub Release Notes - Version Diff
Usage
✅ Please merge!
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

⚙ Adjust future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "org.scala-sbt" } ]

Or, add this to slow down future updates of this dependency:
dependencyOverrides = [{
  pullRequests = { frequency = "30 days" },
  dependency = { groupId = "org.scala-sbt" }
}]

labels: library-update, early-semver-patch, semver-spec-patch, version-scheme:early-semver, commit-count:n:2

Superseded by #317.


(client) - Migration to TypeScript

Checklist:

[ ] I have read freeCodeCamp's contribution guidelines.
[ ] My pull request has a descriptive title (not a vague title like Update index.md)
[ ] My pull request targets the main branch of freeCodeCamp.
[ ] I have tested these changes either locally on my machine, or GitPod.

Closes #XXXXX

Related to #43778

Hey there,
Two of these files were removed in #43861 as they were no longer used. Could you update this PR to reflect those changes?


ElasticsearchFieldMapper ignores customFieldMapping

CustomFields of type "text" are indexed as numbers in ElasticSearch, if the value of the custom field is a numeric value, despite the fact that might have been defined as "text" in the setting "custom_fields_mapping" in config/packages/elasticsearch.yaml.
Source/File:
https://github.com/shopware/shopware/blob/835bf531480535a2ba971a4eb381b0a6ff336eb4/src/Elasticsearch/Framework/ElasticsearchFieldMapper.php#L195
Here the value is checked for a "numeric" value. The custom_fields_mapping setting is complete ignored at this point.
This whole issue leads string/text data indexed as float data in ElasticSearch.

[public] Automated response: This issue is linked to the internal issue https://shopware.atlassian.net/browse/NEXT-33271.
[created from NEXT-33271, comment 487001]


feat: Add Acknowledgements section in Device Settings

Fixes #1220
Checklist:

[ ] I have checked for PMD and check-style issues
[x] I have read the Contribution & Best practices Guide and my PR follows them.
[x] My branch is up-to-date with the Upstream development branch.

Changes:  Added an acknowledgements section in Device Settings to view all the Open Source libraries used in the project. This section has been made similar to EventBrite.
Screenshots for the change:

@iamareebjamal @Masquerade0097 @mishuvs Please review

@sridharjajoo EventBrite shows the url for the licences as well in a new page, when clicked. How about adding that or dmaybe irectly opening them in browser ?

@Masquerade0097 Actually it is taking place here as well. I haven't shown it in the GIF.

@Masquerade0097 Please merge this approved P.R


[KeyError] Pandas raises KeyError when reading landmak from all.csv for hand dataset

Traceback (most recent call last):
File "/usr/local/lib/python3.7/dist-packages/pandas/core/indexes/base.py", line 3361, in get_loc
return self._engine.get_loc(casted_key)
File "pandas/_libs/index.pyx", line 76, in pandas._libs.index.IndexEngine.get_loc
File "pandas/_libs/index.pyx", line 108, in pandas._libs.index.IndexEngine.get_loc
File "pandas/_libs/hashtable_class_helper.pxi", line 2131, in pandas._libs.hashtable.Int64HashTable.get_item
File "pandas/_libs/hashtable_class_helper.pxi", line 2140, in pandas._libs.hashtable.Int64HashTable.get_item
KeyError: 4064
The above exception was the direct cause of the following exception:
Traceback (most recent call last):
File "main.py", line 32, in
Runner(args).run()
File "/content/drive/MyDrive/YOLO_Universal_Anatomical_Landmark_Detection/universal_landmark_detection/model/runner.py", line 152, in run
self.train()
File "/content/drive/MyDrive/YOLO_Universal_Anatomical_Landmark_Detection/universal_landmark_detection/model/runner.py", line 257, in train
self.update_params(epoch, pbar)
File "/content/drive/MyDrive/YOLO_Universal_Anatomical_Landmark_Detection/universal_landmark_detection/model/runner.py", line 294, in update_params
for i, data_dic in enumerate(loader):
File "/content/drive/MyDrive/YOLO_Universal_Anatomical_Landmark_Detection/universal_landmark_detection/model/utils/mixIter.py", line 52, in next
return next(self.cur_iter_list[idx]), idx  # todo
File "/usr/local/lib/python3.7/dist-packages/torch/utils/data/dataloader.py", line 530, in next
data = self._next_data()
File "/usr/local/lib/python3.7/dist-packages/torch/utils/data/dataloader.py", line 570, in _next_data
data = self._dataset_fetcher.fetch(index)  # may raise StopIteration
File "/usr/local/lib/python3.7/dist-packages/torch/utils/data/_utils/fetch.py", line 49, in fetch
data = [self.dataset[idx] for idx in possibly_batched_index]
File "/usr/local/lib/python3.7/dist-packages/torch/utils/data/_utils/fetch.py", line 49, in
data = [self.dataset[idx] for idx in possibly_batched_index]
File "/content/drive/MyDrive/YOLO_Universal_Anatomical_Landmark_Detection/universal_landmark_detection/model/datasets/hand.py", line 49, in getitem
points = self.readLandmark(name, origin_size)
File "/content/drive/MyDrive/YOLO_Universal_Anatomical_Landmark_Detection/universal_landmark_detection/model/datasets/hand.py", line 65, in readLandmark
li = list(self.labels.loc[int(name), :])
File "/usr/local/lib/python3.7/dist-packages/pandas/core/indexing.py", line 925, in getitem
return self._getitem_tuple(key)
File "/usr/local/lib/python3.7/dist-packages/pandas/core/indexing.py", line 1100, in _getitem_tuple
return self._getitem_lowerdim(tup)
File "/usr/local/lib/python3.7/dist-packages/pandas/core/indexing.py", line 838, in _getitem_lowerdim
section = self._getitem_axis(key, axis=i)
File "/usr/local/lib/python3.7/dist-packages/pandas/core/indexing.py", line 1164, in _getitem_axis
return self._get_label(key, axis=axis)
File "/usr/local/lib/python3.7/dist-packages/pandas/core/indexing.py", line 1113, in _get_label
return self.obj.xs(label, axis=axis)
File "/usr/local/lib/python3.7/dist-packages/pandas/core/generic.py", line 3776, in xs
loc = index.get_loc(key)
File "/usr/local/lib/python3.7/dist-packages/pandas/core/indexes/base.py", line 3363, in get_loc
raise KeyError(key) from err
KeyError: 4064

It's a bug that the labels in all.csv don't contain some images in the hand dataset. You can clone the newly updated codes or change Line 26 in file universal_landmark_detection/model/datasets/hand.py as
    index_set = set(self.labels.index)
    files = [i[:-4] for i in sorted(os.listdir(self.pth_Image))]
    files = [i for i in files if int(i) in index_set]

Thank you very much! The problem troubles me for a long time!


Does OctoKit have anything like Parse's PFUser.currentUser()?

Does OctoKit have anything like Parse's PFUser.currentUser() API?
I am looking for an equivalent to this.
Similar Example:
if nil == PFUser.currentUser() {
   // No user.
   presentLoginViewController()
} else {
   // There is a user.
   println("success")
}

Any ideas?

PS: If this doesn't exist, I'd love to help bring it to life, as it would be a very useful feature for this library.

OCTClient has an isAuthenticated method as well as a method to get the current user. There's no global method because OctoKit supports multiple clients.

Thank you for your reply, @mdiep!

https://github.com/octokit/octokit.objc/issues/268


Could not execute entrypoint stage 'preLaunch' due to errors, provided by 'damagevignette'!

Installing this mod on Fabric 1.19.4 makes it not boot returning the following error: https://mclo.gs/UONz4d7
Downgrading to midnightlib 1.2.1 somehow worked for some time but now it is not working on any version

Caused by: org.spongepowered.asm.mixin.throwables.MixinApplyError: Mixin [modid.mixins.json:ExampleMixin from mod cropped] from phase [DEFAULT] in config [modid.mixins.json] FAILED during PREPARE

Ohhh I did not see that! It was that mod the one causing it. Thanks!


How to stop jetracer

How can I stop jetracer when it is outside the road or with untrained images
where should I put (car.throttle = -0.00 car.steering = -0.00)  in the road following code?

You can always just switch CH3 on the transmitter so that it's controlled manually instead of by the GPU. It's quite a bit faster than setting throttle to 0 in the notebook.

Thank you for your reply


Add some missing picker accessibility IDs

For @jh-bate

@jh-bate I'm going to go ahead and merge this now so it's available to you whenever you need it

@mpangburn and @jh-bate, can you provide a link to the Jira ticket that this work was done for?


Feature: Unzip multiple ZIPs at once

:green_book: Use case:

Suppose I have multiple zips (multiple files with .zip extension) on some directory
I want to extract all of these zips

:dart: Example:

(Maybe) Create a method that accepts the 3 arguments- Source Path. Zip Names, Destination Path
The zips could be stored in a String Array, List, etc

Source Path => /path/to/zips
List of Zips => String[] allZips = { "file1.zip", "file2.zip" } OR List
Destination Path => /path/to/extract
object.extractTheseZIPs(SourcePath, ZIPList, DestinationPath);

Thoughts, @srikanth-lingala ?

This can be achieved with a simple for loop. Pseudo-code would be something like:
for each item in the list of files
ZipFile zipFile = new ZipFile();
zipFile.extractAll();
I am a bit hesitant to adding such feature to zip4j. This logic should be outside of zip4j. Even if I added it to zip4j, the code would be the same as above. I really appreciate your thoughts on this, but I think such logic should exist outside of zip4j.


Improve how data is sent to Data Hub

[x] keep track of which results were already sent and when they were sent
[x] Don't send duplicate results to the data hub
[ ] Automate results to data hub - this should be real-time or near real-time after submit is clicked.
[ ] Have monitoring and retry logic

current work on this issue capture in https://app.zenhub.com/workspaces/simplereport-5ff8be8854ae8b000e378210/issues/cdcgov/prime-simplereport/667


Can you make this as vs code extension?

Can you make this as vs code extension?

Duplicate of #70

ohh! thanks


add rzwiz host profile and update networks.json

Description
Adds a host profile for LLNL rzwiz cluster
Type of change

[ ] Bug fix~~
[ ] New feature~~
[ ] Documentation update~~
[x] Other~~
New system config

How Has This Been Tested?
Tested with 3.3.3 install on rzwiz.
Reminders:

Please follow the style guidelines of this project.
Please perform a self-review of your code before submitting a PR and asking others to review it.
Please assign reviewers (see VisIt's PR procedures for more information).

Checklist:

- [ ] I have commented my code where applicable.
- [ ] I have updated the release notes.
- [ ] I have made corresponding changes to the documentation.
- [ ] I have added debugging support to my changes.
- [ ] I have added tests that prove my fix is effective or that my feature works.
- [ ] I have confirmed new and existing unit tests pass locally with my changes.
- [ ] I have added new baselines for any new tests to the repo.

[x] I have NOT made any changes to protocol or public interfaces in an RC branch.~~

Did you overlook running 'dump_dir_to_networks_json.py' to regenerate networks.json ?


Project name starting with Whiteapce

Qwik Version
latest
Operating System (or Browser)
MacOs
Node Version (if applicable)
No response
Which component is affected?
Starters / CLI
Expected Behaviour
npm create qwik

Should not allow whitespace on start of a directory name
Actual Behaviour
The npm command to start a project allows whitespace at the start of the directory name.
This is confusing when you would do cd project-name. the project becomes inaccessible

Additional Information
No response

Another image here with 2 same directories created with qwik, one begins with a space


Editor in canvas performance is poor

While switch between lessons while editing, we should not see loading occur. The switching effect should be similar when the editor is in readonly mode.

Relates to #59 #64


Pull request #24 (Introduction of AttemptClass and PaymentClass) broke the payment algorithm or simulation

It seems like this merged pull request broke the payment algorithm or simulation:
https://github.com/renepickhardt/pickhardtpayments/pull/24
PickhardtPaymentsExample.ipynb examples don't work anymore:
Round number: 10
Try to deliver 18879883 satoshi:
Statistics about 46 candidate onions:
successful attempts:
p = 100.00% amt: 216543 sats hops: 4 ppm: 1413
p = 94.37% amt: 84263 sats hops: 5 ppm: 1349
p = 90.69% amt: 59164 sats hops: 5 ppm: 1708
p = 75.00% amt: 337453 sats hops: 5 ppm: 1417
p = 73.91% amt: 112351 sats hops: 5 ppm: 1374
p = 99.52% amt: 42718 sats hops: 6 ppm: 1348
p = 100.00% amt: 35946 sats hops: 6 ppm: 1367
p = 94.97% amt: 242846 sats hops: 7 ppm: 1696
p = 77.42% amt: 2351 sats hops: 9 ppm: 1427
failed attempts:
p = 100.00% amt: 88000 sats hops: 2 ppm: 1339
p = 74.25% amt: 288187 sats hops: 2 ppm: 1341
p = 14.57% amt: 1574652 sats hops: 2 ppm: 1359
p = 100.00% amt: 102400 sats hops: 2 ppm: 1381
p = 100.00% amt: 102400 sats hops: 2 ppm: 1356
....
SUMMARY:
Rounds of mcf-computations: 10
Number of attempts made: 448
Number of failed attempts: 356
Failure rate: 79.46%
total Payment lifetime (including inefficient memory management): 8.661 sec
Learnt entropy: 229.60 bits
fee for settlement of delivery: 0.000 sat --> 0 ppm
used mu: 100

yes we introduced a bug but that was supposed to be fixed with the merge of #18 which is currently on my desk for review. Sorry for the slight delay that has been introduced to this repository over the summer months.

No worries :)


remove duplicate config opt

--checkpoint-activations was being set twice.
this the other place:
https://github.com/microsoft/DeepSpeedExamples/blob/ce00e423b9ed4cc3ce20dab98861f6173094a557/Megatron-LM-v1.1.5-3D_parallelism/examples/ds_pretrain_gpt2_pipe.sh#L105

File no longer exists, closing old PR


feat: allow a ci dedicated repo to run workflow

What
Add a new option to run workflow on CI dedicated workflow.
ci_repo = "foo/baz"

Fix https://github.com/rust-lang/bors/issues/168
Test plan
Looking to test it this weekend

Test successful
https://github.com/vohoanglong0107/test-bors/pull/2#issuecomment-2511143789

This PR has grown quite large, I will split it to smaller repos for easier reviewing


PSFR-1814 Remove version defaults from resettlement assessment code

Due to merge conflict closing this PR! will raise another PR for the same


Submitting URL encoded array of structured arguments.

This might be slightly related to #62.
Let's say that I have these data types:
data Something = Something
    { field01 :: Text
    , field02 :: Text
    }

data StepInstruction = StepInstruction
    { id :: Int
    , order :: Int
    , description :: Text
    }

data Recipe = Recipe
    { id :: Int
    , name :: Text
    , something :: Something
    , instructions :: [StepInstruction]
    }

In PHP I could easily submit a form with such data type and get it back in the form of a nested array. Convention for submitting Recipe is as follows:
?id=0
&name=Recipe01

&instructions[0][id]=0
&instructions[0][order]=1
&instructions[0][description]=somedescription

&instructions[0][id]=1
&instructions[0][order]=2
&instructions[0][description]=someotherdescription

Currently Query flat and I'd have to do additional parsing on the argument names to get an array of structured arguments.
I'm not sure if this is a standard, but I've also seen the same convention in C#/MVC so I think it would be a good idea to support such feature, because I might want to add repeating "sub forms" to my main form that should be contained in array, and currently there is no good way to do that from what I can see.
Here is also an example of a form that might submit such query:
<form>
    <input type="hidden" name="id">
    <input type="text" name="name">

    <input type="text" name="something[field01]">
    <input type="text" name="something[field02]">

        <input type="hidden" name="instructions[0][id]">
        <input type="number" name="instructions[0][order]">
        <input type="text" name="instructions[0][description]">

        <input type="hidden" name="instructions[1][id]">
        <input type="number" name="instructions[1][order]">
        <input type="text" name="instructions[1][description]">

        <input type="hidden" name="instructions[2][id]">
        <input type="number" name="instructions[2][order]">
        <input type="text" name="instructions[2][description]">

</form>

Maybe QueryItem should look something like this:
data QueryItem
    = PlainValue ByteString
    | QueryItemList [QueryItem]
    | QueryItem (ByteString, Maybe QueryItem)

Similar to what I said in #62, I am not in favor of changing the existing functions because that would break compatibility and also because this is not the most common way of using form inputs, I think.
However, if you want to make a new function that allows rendering/parsing nested data like that, you could send a pull request with new functions and types and if it looks good, it could be merged.


Duplicate rust-toolchain and rust-toolchain.toml (for compatibility?)

Duplicate rust-toolchain and rust-toolchain.toml in the crate.
This crate use both of them for compatibility?
Here is the doc from rustup book

In these cases the toolchain can be named in the project's directory in a file called rust-toolchain.toml or rust-toolchain. If both files are present in a directory, the latter is used for backwards compatibility. The files use the TOML format and have the following layout:

There is an issue with github-action actions-rs/toolchain: it does not recoganize the rust-toolchain.toml but just rust-toolchain.
It seems it has been fixed. I'm gonna remove the plain text one as I confirmed it is fixed.

Does not work yet.
https://github.com/drmingdrmer/openraft/runs/5246849191?check_suite_focus=true
https://github.com/actions-rs/toolchain/issues/126

But both exist in crates, only rust-toolchain works according to rustup book.

If both files are present in a directory, the latter is used for backwards compatibility


Cvat looking for SAM nuclio function at incorrect port

My actions before raising this issue

[x ] Read/searched the docs
[ x] Searched past issues

I am having a problem with the Segment Anything interactor model. For some reason, CVAT is looking for the function at port 32769 when the SAM model is at 32768, as seen by docker ps. If I issue ``docker restart nuclio-nuclio-pth-facebookresearch-sam-vit-h`, then when it restarts, the port number increments by 1, and then I can use the model in CVAT without the error. But this doesn't make sense to me, as I haven't hard coded any port number anywhere, as far as I am aware. So I don't know why it isn't looking for the function in the right place.
The exact error message is the same one as in issue #6222, but the poster there confirmed his issue is not related to the ports:
Interaction error occurred
Error: Request failed with status code 503. "HTTPConnectionPool(host='host.docker.internal', port=32769): Max retries exceeded with url: / (Caused by NewConnectionError('<urllib3.connection.HTTPConnection object at 0x7f64a03a7a60>: Failed to establish a new connection: [Errno 111] Connection refused'))".

If I grep for the port number in the git repo, the only file that has 32768 is
cvat$ find . -name "*.*" -type f -exec grep -l 32768 {} \+
./cvat/apps/opencv/static/opencv/js/opencv_4.5.4_dev.js

Not sure if this is any help. But I do notice that when it finally does work, it briefly has a message pop up saying something about opencv initialization being successful. There is another recent issue ( #6237 ) regarding opencv initialization, so maybe there's something fishy going on there? Don't really know.

Steps to Reproduce (for bugs)

I reboot my computer, and I open CVAT in chrome by going to the localhost:8080 url.
I open up any task, and try to use the SAM interactor model. Once I click, it returns the error message above

Expected Behaviour

CVAT should be looking for SAM at the port that it is on. Not looking at some hardcoded value
Current Behaviour

I get a 503 error saying it can't reach the nuclio function.
Possible Solution

If I restart the docker container for SAM, that increments the port that it is on up to 32769, the port that CVAT is looking for it on. But this is hacky and I don't want to have to do this.
Instead, perhaps there is somewhere that I can declare which port nuclio should use for SAM, and which port CVAT should be looking for SAM on?
Context

I want to use CVAT for labeling many images, and this is just an annoyance to have to deal with every  time I want to get started.

No
Operating System and version (e.g. Linux, Windows, MacOS):
Ubuntu 20.04
Code example or link to GitHub repo or gist to reproduce problem:
Other diagnostic information / logs:
Logs from `cvat` container

Try to remove / redeploy the function. Usually it helps.
Root cause why it happens is not clear :(

Hmm, this seems to have worked. I rebooted and refreshed the page a few times and hopefully things continue to work. I was able to close some ports on my firewall now, thanks a lot!

I am experiencing the same error. It seems that if intermittent manual intervention like discussed above is necessary, this issue should remain open.

I made the following local change to cvat/serverless/pytorch/facebookresearch/sam/nuclio/function-gpu.yaml:
  triggers:
    myHttpTrigger:
      maxWorkers: 1
      kind: 'http'
      workerAvailabilityTimeoutMilliseconds: 10000
      attributes:
        maxRequestBodySize: 33554432 # 32MB
        port: 32768 # added this line

After redeploying, I am able to use the SAM interactor without errors.


Sprint Challenge Sachin-Benny

@mister-corn

Home Page

[ ] Build the HTML and CSS to create the missing navigation and header.
[ ] Link the About navigation item to the about.html page

Use this list below to correctly style each box:

[ ] box1: teal
[ ] box2: gold
[ ] box3: cadetblue
[ ] box4: coral
[ ] box5: crimson
[ ] box6: forestgreen
[ ] box7: darkorchid
[ ] box8: hotpink
[ ] box9: indigo
[ ] box10: dodgerblue

Comments

About Page
Create the rest of the missing HTML and CSS to match the design file.

[ ] Copy and paste your home page navigation and header into the about page
[ ] Update the header image with the about page image
[ ] Link the Home navigation item back to the index.html page.
[ ] Build the rest of the about page layout to match the design

Comments

DRY Example
Beware of the corset effect.


Embed link

Is there a hosted link to embed this script on a website like nocomment has?

Okay I think I see the problem, you're trying to use it with data-* attributes like nocomment and that doesn't work here. Just import the script, and then use like here: https://github.com/21ideas-org/21ideas.org/blob/e06d11719b73d728020a409d0b82caa2dcdec34f/themes/hugo-book/layouts/partials/docs/comments.html#L5

Is there any reason why the whole plugin doesn’t show up on my mobile device?Sent from my iPhoneOn Nov 27, 2023, at 12:54 PM, franzap @.**> wrote:
Okay I think I see the problem, you're trying to use it with data- attributes like nocomment and that doesn't work here. Just import the script, and then use like here: https://github.com/21ideas-org/21ideas.org/blob/e06d11719b73d728020a409d0b82caa2dcdec34f/themes/hugo-book/layouts/partials/docs/comments.html#L5
—Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this because you authored the thread.Message ID: @.***>


Support Swift 4.2

This pull request adds swift 4.2 support.
I haven't updated the Travis configs since it doesn't have support yet.

Closing because I probably won't be able to support this PR


Dashboard Problems

Hello!
When I go to my dashboard and click any links on the side, it just says "There was an error with the request, please check the console for more information."
I didn't change anything, I installed it and logged in.
Also, when I go back to the home page, it logs me out no matter what.

What error showed up in console?

Nothing shows up in my console. There is nothing under the log folder either.

Wonder if there is a problem with my auth at all.

Alright, you now stay logged in when going back to the home page. The dashboard still has the same problem, clicking any buttons says there was an error. Nothing still shows up in console for some reason.

@hyperdefined can you open devtools, go to the Network tab and show me the Preview tab when you click on the Uploads menu item on the side? The request should look something like this

Also, now it says this in the console when I click Change your password"

Would you mind joining the Discord server for lolisafe, so I can take a closer look at your issue? Because right now I'm super lost as what could be causing your issue.

Problem solved. Stupid me removed the div on the dashboard that loads the content.


Where did the

I am curious in your utils.py where the calculate_gaussian_log_prob(log_std, noise) function came from? It doesn't look like the stable baselines or pytorch Log PDF of the normal distribution. So what is it if you don't mind answering.

Hi, @ChadMcintire
It can be derived by simple mathematics. Note that the standard diviation is 1.

Thank you for responding.
I am sorry I have been trying to derive it and I'm not asking you to derive anything.
$\frac{1}{\sigma \sqrt{2 \pi}} e^{\frac{1}{2}(\frac{x-\mu}{\sigma})^2} $
Which will simplify as follows:
$$\frac{1}{\sigma\sqrt{2\pi}}e^{-\frac12 \big(\frac{x-\mu}{\sigma}\big)}$$
$$\log\Big(\frac{1}{\sigma\sqrt{2\pi}}e^{-\frac12 \big(\frac{x-\mu}{\sigma}\big)}\Big) $$
$$= \log\Big(\frac{1}{\sigma\sqrt{2\pi}}\Big) + \log\Big(e^{-\frac12 \big(\frac{x-\mu}{\sigma}\big)}\Big) $$
$$= -\log\sigma -\log\sqrt{2\pi} - \frac12\big(\frac{x-\mu}{\sigma}\big)^2$$
The equation in the code looks more like this:
$\log \mu(u|s) = -0.5 \times \log(2 \times \pi) \times n + \sum (-0.5 \times \epsilon^2 - \log \sigma^2) $
I'm unsure where the "sum" comes from, as well as the "n".
In your reply, you are saying that you derived the formula from a log normal distribution. The std * I, is that just the standard deviation * the identity is what you are saying.

Hi, please note that the distribution is multivariate gaussian, and variables are independent. To be precise, sigma is  N-dimensional diagonal matrix, whose diagonal elements are equal to stds.
So you need to calculate the probabilities of each dimension and multiply them together. Alternatively, calulate the log probabilities and sum them.
Because there is no need to calculate the probability itself, I calculated it this way to reduce the computation.

So if I start with the multivariate Gaussian PDF.
$(2 \pi)^{-\frac{k}{2}} det(\Sigma)^{-1/2} exp(-\frac{1}{2}(x - \mu)^T \Sigma^{-1}(x -mu))$
Taking the natural log of both sides
$-\frac{k}{2} ln(2 \pi) -\frac{1}{2} \ln det(\Sigma)  - \frac{1}{2}(x - \mu)^T \Sigma^{-1}(x -mu)$
To make it look more like the code:
$-\frac{1}{2} ln(2 \pi) * k  -\frac{1}{2}\ln det(\Sigma)  - \frac{1}{2}(x - \mu)^T \Sigma^{-1}(x -mu)$
$-\frac{1}{2}\ln det(\Sigma)  - \frac{1}{2}(x - \mu)^T \Sigma^{-1}(x -\mu) -\frac{1}{2} ln(2 \pi) * k$
The code looks like this
$\sum (-0.5 \times \mu^2 - \ln std) -0.5  \ln(2 \pi) \times k  $
I'm so sorry I don't understand from you comment how you go from experssion 1 to the expression in your code.
$-\frac{1}{2}\ln det(\Sigma)  - \frac{1}{2}(x - \mu)^T \Sigma^{-1}(x -\mu)$
$\sum (-0.5 \times \mu^2 - \ln std)$

Calculate the probability (or log probability) of each dimension independently and multiply (or add) them together, because these variables are independent.


CBL-3943: Update tests to use collection API instead of deprecated API

Each commit now refers to a specific set of tests (C/C++ grouped), trying to make it easier to review. I will plan accordingly next time...


profile add projection time in scaner

Proposed changes
 -  PerScannerRunningTime:  [1s194ms,  887.315ms,  1s049ms,  1s220ms,  1s209ms,  836.272ms,  1s105ms,  1s037ms,  1s063ms,  1s180ms,  1s191ms,  1s096ms,  883.761ms,  883.103ms,  1s023ms,  1s136ms,  904.661ms,  1s237ms,  1s116ms,  1s029ms,  932.601ms,  1s109ms,  1s184ms,  1s094ms,  1s256ms,  937.526ms,  1s143ms,  1s136ms,  1s047ms,  1s083ms,  940.786ms,  1s132ms,  1s055ms,  968.188ms,  1s024ms,  1s052ms,  1s045ms,  1s163ms,  1s134ms,  841.429ms,  1s064ms,  1s194ms,  1s126ms,  1s036ms,  1s012ms,  1s112ms,  1s219ms,  1s068ms,  ]
-  PerScannerProjectionTime:  [1s182ms,  850.468ms,  1s038ms,  1s164ms,  1s189ms,  825.352ms,  1s084ms,  1s017ms,  1s053ms,  1s168ms,  1s177ms,  1s058ms,  859.193ms,  833.417ms,  1s011ms,  1s122ms,  892.875ms,  1s228ms,  1s098ms,  1s006ms,  923.096ms,  1s096ms,  1s173ms,  1s079ms,  1s228ms,  927.629ms,  1s120ms,  1s099ms,  1s027ms,  1s052ms,  931.977ms,  1s099ms,  1s037ms,  949.553ms,  1s015ms,  1s042ms,  1s032ms,  1s143ms,  1s118ms,  828.954ms,  1s053ms,  1s182ms,  1s090ms,  1s022ms,  1s001ms,  1s096ms,  1s209ms,  1s057ms,  ]

Further comments
If this is a relatively large or complex change, kick off the discussion at<EMAIL_ADDRESS>by explaining why you chose the solution you did and what alternatives you considered, etc...

run buildall


Strange Rendering

hi,
i have encountered two cases of weird rendering. the demo using glfw runs perfectly under linux - i tried to port it to SDL2 when i got this result on the pc:

i know it is most probably not directly related to nanovg but any hint would be appreciated.
when i compile it under android it works okey without antialiasing. with antialiasing i get this:

this is the code. the rest is from demo.
int main()
{
	//GLFWwindow* window;

	DemoData data;
	NVGcontext* vg = NULL;
	PerfGraph fps;
	bool quit=false;
	SDL_Event event;
	//double prevt = 0;

	//if (!glfwInit()) {
	//	printf("Failed to init GLFW.");
	//	return -1;
	//}

	if (SDL_Init(SDL_INIT_VIDEO|SDL_INIT_TIMER) != 0) {
	  fprintf(stderr, "\nUnable to initialize SDL:  %s\n", SDL_GetError());
	  return 1;
	}

	initGraph(&fps, GRAPH_RENDER_FPS, "Frame Time");

	//glfwSetErrorCallback(errorcb);

						//context_=SDL_GL_CreateContext(window);

	//glfwWindowHint(GLFW_CLIENT_API, GLFW_OPENGL_ES_API);
	//glfwWindowHint(GLFW_CONTEXT_VERSION_MAJOR, 2);
	//glfwWindowHint(GLFW_CONTEXT_VERSION_MINOR, 0);

	SDL_GL_SetAttribute(SDL_GL_CONTEXT_PROFILE_MASK, SDL_GL_CONTEXT_PROFILE_ES);
	SDL_GL_SetAttribute(SDL_GL_CONTEXT_MAJOR_VERSION, 2);
	SDL_GL_SetAttribute(SDL_GL_CONTEXT_MINOR_VERSION, 0);

	auto window = SDL_CreateWindow("NanoVG",SDL_WINDOWPOS_CENTERED,SDL_WINDOWPOS_CENTERED,1000,600,SDL_WINDOW_OPENGL);

	//glfwCreateWindow(1000, 600, "NanoVG", NULL, NULL);
//	window = glfwCreateWindow(1000, 600, "NanoVG", glfwGetPrimaryMonitor(), NULL);
	if (!window) {
		//glfwTerminate();
		printf("NO Window - Terminate\n");
		return -1;
	}

	//glfwSetKeyCallback(window, key);
	auto context=SDL_GL_CreateContext(window);

	SDL_GL_MakeCurrent(window,context);

	//glfwMakeContextCurrent(window);

	vg = nvgCreateGLES2(NVG_ANTIALIAS | NVG_STENCIL_STROKES | NVG_DEBUG);
	if (vg == NULL) {
		printf("Could not init nanovg.\n");
		return -1;
	}

	if (loadDemoData(vg, &data) == -1)
		return -1;

	//glfwSwapInterval(0);

	SDL_GL_SetSwapInterval(0);

	//glfwSetTime(0);
	auto prevt=high_resolution_clock::now();
	auto app_launch_time=high_resolution_clock::now();
	//prevt = glfwGetTime();

	//SDL_TimerID my_timer_id = SDL_AddTimer(50, timer_callback, 0);

	while (!quit)
	{

		SDL_PollEvent(&event);

		switch(event.type)
		{
		case SDL_QUIT:
		  quit=true;
		  break;

		}

		int mx, my;
		int winWidth, winHeight;
		int fbWidth, fbHeight;
		float pxRatio;

		auto t = high_resolution_clock::now();
		auto dt = duration_cast<duration<float>>(t - prevt).count();
		auto _t=duration_cast<duration<float>>(t - app_launch_time).count();

		prevt = t;
		updateGraph(&fps, dt);

		SDL_GetMouseState(&mx, &my);
		SDL_GetWindowSize(window, &winWidth, &winHeight);
		fbWidth=winWidth; fbHeight=winHeight;
		//glfwGetFramebufferSize(window, &fbWidth, &fbHeight);
		// Calculate pixel ration for hi-dpi devices.
		pxRatio = (float)fbWidth / (float)winWidth;

		// Update and render
		glViewport(0, 0, fbWidth, fbHeight);
		if (premult)
			glClearColor(0,0,0,0);
		else
			glClearColor(0.3f, 0.3f, 0.32f, 1.0f);
		glClear(GL_COLOR_BUFFER_BIT|GL_DEPTH_BUFFER_BIT|GL_STENCIL_BUFFER_BIT);

		glEnable(GL_BLEND);
		glBlendFunc(GL_SRC_ALPHA, GL_ONE_MINUS_SRC_ALPHA);
		glEnable(GL_CULL_FACE);
		glDisable(GL_DEPTH_TEST);

		nvgBeginFrame(vg, winWidth, winHeight, pxRatio);

		renderDemo(vg, mx,my, winWidth,winHeight, _t, blowup, &data);

		//printf("TIMING: DT: %f, T: %f\n",dt,_t);
		renderGraph(vg, 5,5, &fps);

		nvgEndFrame(vg);

		if (screenshot) {
			screenshot = 0;
			saveScreenShot(fbWidth, fbHeight, premult, "dump.png");
		}

		glEnable(GL_DEPTH_TEST);

		SDL_GL_SwapWindow(window);

		//glfwSwapBuffers(window);
		//glfwPollEvents();
	}

	freeDemoData(vg, &data);

	nvgDeleteGLES2(vg);

	SDL_GL_DeleteContext(context);
	SDL_DestroyWindow(window);
	return 0;
}

thank you for your great work.

You will need to make sure that you create frame buffer with stencil.

thank you very much. first problem has gone. sdl renders correctly on desktop. antialiasing on android still the same.

Looks like the shader is broken on Android. Some of the problems don't make any sense at all, like the gradient on the wave. Are you using the GLES2 version on android?

yes, i'm using GLES2. i tried another phone, antialias works fine on it, but then i got a radient problem a la https://github.com/memononen/nanovg/issues/178 .

i tried to change fragment precision didn't help. well i guess i should test some more devices. i got the antialias bug on a galaxy nexus which is quite old......maybe it's just a driver bug.....the radient problem is on a S5 mini

The second one looks more common. The first android one looked like some data did not get rendered. Also, textures not working, maybe there's just a problem loading the fonts and textures?

no, sorry forgot to say i did not load fonts and textures on purpose didn't have time to deal with the assetmanager so far. the only difference i see between android and the pc is that android likes to default to RGB_565.....

I wonder which shader modifications @olliwang made for iOS. IIRC he initially also had precision problems.

@memononen I didn't do shader modification for iOS. The recent versions works fine for iOS out-of-box, except the performance issue I mentioned in #188.

In case this helps, it happears my issue at #178 is now solved. More info (and a test project) there.

thank you very much :) i will test it as soon as i find the time.....

@kollapsderwellenfunktion  How  enable stencilbuffer with SDL2?


Error: "The authorization mechanism you have provided is not supported."

Hi,
I'm trying to deploy to a S3 bucker in the Frankfurt region but I receive the following error:
botocore.exceptions.ClientError: An error occurred (InvalidRequest) when calling the ListObjects operation: The authorization mechanism you have provided is not supported. Please use AWS4-HMAC-SHA256.
I've found a plausible explanation for this in Stackoverflow:

AWS4-HMAC-SHA256, also known as Signature Version 4, ("V4") is one of two authentication schemes supported by S3.
All regions support V4, but US-Standard¹, and many -- but not all -- other regions, also support the other, older scheme, Signature Version 2 ("V2").
According to http://docs.aws.amazon.com/AmazonS3/latest/API/sig-v4-authenticating-requests.html ... new S3 regions deployed after January, 2014 will only support V4.
Since Frankfurt was introduced late in 2014, it does not support V2, which is what this error suggests you are using.
http://docs.aws.amazon.com/AmazonS3/latest/dev/UsingAWSSDK.html explains how to enable V4 in the various SDKs, assuming you are using an SDK that has that capability.

Rif: http://stackoverflow.com/questions/26533245/the-authorization-mechanism-you-have-provided-is-not-supported-please-use-aws4?answertab=votes#tab-top
Could you please help with this? I must deploy on this AWS region.
Thank you.

Ouch. The underlying issue looks like it's a problem in boto, to me, based on https://github.com/boto/boto/issues/2741.
There are numerous workarounds suggested in there, through stuff like creating a file at ~/.boto which contains this:
host=s3.eu-central-1.amazonaws.com```

But that's a kind of crappy solution. In the short term, that might be the best way to be able to deploy; I'll look around to see if there's a more elegant fix.

Thanks @spenczar
Meanwhile I'm just using the aws cli to sync the 'dist' folder with the S3 bucket manually, it's not a big deal.

Did you fix this issue ?
I have the same issue when copy file/folder in s3 bucket stay at eu-central-1, but in console i get this issue
$ aws s3 cp --recursive  s3://bucketname/static-assets/access/ . --region=eu-central-1
usage: aws [options]   [parameters]
aws: error: argument --region: Invalid choice, valid choices are:
ap-southeast-2                           | cn-north-1
ap-northeast-1                           | sa-east-1
eu-west-1                                | fips-us-gov-west-1
us-west-2                                | us-west-1
us-east-1                                | us-gov-west-1
ap-southeast-1
And here
A client error (InvalidRequest) occurred when calling the ListObjects operation: The authorization mechanism you have provided is not supported. Please use AWS4-HMAC-SHA256.
Completed 1 part(s) with ... file(s) remaining

@Van-Tien I ended up using my own bash script for this, they are working fine. In your case I guess your AWS administrator is explicitly excluding eu-central-1 from the available regions.


Find out how to use extern jars to load more rule sets providers

Will be done here - https://github.com/arturbosch/sonar-kotlin/issues/1


[T5A3][F11-B3]Tiong YaoCong

New Font & Color for User Interface , adding of new Icon
Changing of Font

Change font to Helvetica Neue

Changing of Color

Change user input text color from yellow to red
Change border color from black to white

Adding of new App Icon

Nice and apt icon!
@ChaseYaoCong Pls close the PR! Thanks!


fix(autoware_static_centerline_generator): fix unusedFunction

Description
This is a fix based on cppcheck unusedFunction warnings.
planning/autoware_static_centerline_generator/src/static_centerline_generator_node.cpp:600:0: style: The function 'update_road_boundary' is never used. [unusedFunction]
RoadBounds StaticCenterlineGeneratorNode::update_road_boundary(
^

planning/autoware_static_centerline_generator/src/utils.cpp:260:0: style: The function 'calcBoundsFromLanelets' is never used. [unusedFunction]
calcBoundsFromLanelets(const lanelet::ConstLanelets lanelets)
^

Related links
Parent Issue:

Link

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

Effects on system behavior
None.

About update_road_boundary
// const auto road_bounds = update_road_boundary(selected_centerline);
The location used above was commented out, so I removed it as it is not currently used.

@takayuki5168
I found a new warning and have removed arePointsClose and areSameDirection.
I would appreciate it if you could check again.


Modal with routes

Adds support for routing for the modals
To Test

load the page
open an icon modal (path should change, page should not scroll to the top)
close the modal (path should change, page scroll position should be maintained)
try searching and/or filtering by icon style
open/close modal and the search settings should be maintained

This supersedes https://github.com/resolvetosavelives/healthicons/pull/102 and that PR should be closed if this approach works.

I'm not qualified to review the React/Next code, but I've tested this and it's working nicely. Well done @nburka


Text.width is not consistent with Container.width

https://jsfiddle.net/starCoding/p49u34hd/6/
I think the width of both of them should be consistent.because the text is extended from the container

Yep, that's a bug


Shift + Enter for new line

Would it possible to add Shift + Enter for the prompt entry?
Love your work!

next version

Fixed at https://github.com/Quilljou/dalle3-playground/commit/815baaefcdc18791c7dab4022795e3cb1c19698f


Admin variables

Admin variables page is complete.
Still to be implemented for admin page:

Registration approval
Post approval
Account suspension
View posts
View users

LGTM!

LGTM!


Box-sizing of input type="search"

In v4-dev when using <input type="search"> something's wrong with its box-sizing, it's pretty clear for example in <td>s.
See http://jsfiddle.net/heruan/vdj9j6rb/ where the first table has type="search" while the second has type="text". I'm on Chrome and using Flexbox.

Indeed. It appears the problem is caused by
-webkit-box-sizing: content-box;
box-sizing: content-box;

in normalize.scss

We should override it back to border-box. It's the browser default (http://w3bug.com/28784) and we prefer border-box anyways.

+1 for overriding it to border-box like in Bootstrap 3:
https://github.com/twbs/bootstrap/blob/master/less/forms.less#L46

Fixed in 468a9d9.


Cannot run MongoDB in systemd-container

Is this a BUG REPORT or FEATURE REQUEST? (leave only one on its own line)
/kind bug
Description

Steps to reproduce the issue:

Create a systemd container based on centos/systemd

Install rh-mongodb34-mongod inside container

Start container with podman ... --systemd=true ...

Describe the results you received:
The service cannot start inside the container, it fails with the following error:

New main PID 817 does not belong to service, and PID file is not owned by root. Refusing.

Describe the results you expected:
Prior to CentOS 7.7 and https://access.redhat.com/solutions/4420581 everything worked great.
Additional information you deem important (e.g. issue happens only occasionally):
Output of podman version:
podman version 1.4.4

Output of podman info --debug:
debug:
  compiler: gc
  git commit: ""
  go version: go1.10.3
  podman version: 1.4.4
host:
  BuildahVersion: 1.9.0
  Conmon:
    package: podman-1.4.4-4.el7.centos.x86_64
    path: /usr/libexec/podman/conmon
    version: 'conmon version 0.3.0, commit: unknown'
  Distribution:
    distribution: '"centos"'
    version: "7"
  MemFree:<PHONE_NUMBER>
  MemTotal:<PHONE_NUMBER>4
  OCIRuntime:
    package: runc-1.0.0-65.rc8.el7.centos.x86_64
    path: /usr/bin/runc
    version: 'runc version spec: 1.0.1-dev'
  SwapFree: 0
  SwapTotal: 0
  arch: amd64
  cpus: 4
  hostname: foreman-chbr-node1.vms.dev.basalt.se
  kernel: 3.10.0-957.12.2.el7.x86_64
  os: linux
  rootless: false
  uptime: 1h 5m 18.04s (Approximately 0.04 days)
registries:
  blocked: null
  insecure: null
  search:
  - registry.access.redhat.com
  - docker.io
  - registry.fedoraproject.org
  - quay.io
  - registry.centos.org
store:
  ConfigFile: /etc/containers/storage.conf
  ContainerStore:
    number: 1
  GraphDriverName: overlay
  GraphOptions: null
  GraphRoot: /var/lib/containers/storage
  GraphStatus:
    Backing Filesystem: xfs
    Native Overlay Diff: "true"
    Supports d_type: "true"
    Using metacopy: "false"
  ImageStore:
    number: 1
  RunRoot: /var/run/containers/storage
  VolumePath: /var/lib/containers/storage/volumes

Package info (e.g. output of rpm -q podman or apt list podman):
podman-1.4.4-4.el7.centos.x86_64

Additional environment details (AWS, VirtualBox, physical, etc.):
Virtual machine running on OpenStack, having run yum -y update on the host prior to starting the container.
Same version of systemd on host and container.
I have also filed an issue with Foreman in case their installation and configuration of MongoDB is the issue: https://projects.theforeman.org/issues/27887
But I am starting to suspect that the issue is with running a systemd container with systemd-219-67 and I suspected podman to fix all the mounts when using the --systemd flag.

I have an easy way to reproduce the issue:
$ cat Dockerfile
FROM centos/systemd
RUN yum -y update && \
    yum -y install centos-release-scl && \
    yum -y install rh-mongodb34-mongodb-server && \
    systemctl enable rh-mongodb34-mongod.service

$ sudo podman build -t test .

... snip ...

$ sudo podman run --systemd=true --name test -d --privileged test
19ae5a643b6b71223068c329b167f9135f3603e32e0e8d5fb920df562c389034

$ sudo podman exec -it test systemctl status rh-mongodb34-mongod
● rh-mongodb34-mongod.service - High-performance, schema-free document-oriented database
   Loaded: loaded (/usr/lib/systemd/system/rh-mongodb34-mongod.service; enabled; vendor preset: disabled)
   Active: activating (start) since Fri 2019-10-04 11:03:03 UTC; 1min 28s ago
  Process: 75 ExecStart=/opt/rh/rh-mongodb34/root/usr/libexec/mongodb-scl-helper enable $RH_MONGODB34_SCLS_ENABLED -- /opt/rh/rh-mongodb34/root/usr/bin/mongod $OPTIONS run (code=exited, status=0/SUCCESS)
   CGroup: /machine.slice/libpod-19ae5a643b6b71223068c329b167f9135f3603e32e0e8d5fb920df562c389034.scope/system.slice/rh-mongodb34-mongod.service
           └─193 /opt/rh/rh-mongodb34/root/usr/bin/mongod -f /etc/opt/rh/rh-mongodb34/mongod.conf run

Oct 04 11:03:03 19ae5a643b6b systemd[1]: Starting High-performance, schema-free document-oriented database...
Oct 04 11:03:03 19ae5a643b6b mongodb-scl-helper[75]: about to fork child process, waiting until server is ready for connections.
Oct 04 11:03:03 19ae5a643b6b mongodb-scl-helper[75]: forked process: 193
Oct 04 11:03:04 19ae5a643b6b mongodb-scl-helper[75]: child process started successfully, parent exiting
Oct 04 11:03:04 19ae5a643b6b systemd[1]: New main PID 193 does not belong to service, and PID file is not owned by root. Refusing.
Oct 04 11:03:04 19ae5a643b6b systemd[1]: New main PID 193 does not belong to service, and PID file is not owned by root. Refusing.

Seems like a systemd issue.  Could you open a bugzilla on this.  I don't believe this is an upstream issue.

Which bugzilla? Should I open a support ticket in the Customer Portal?

I found a Red Hat Bugzilla and created an account to file this bug report: https://bugzilla.redhat.com/show_bug.cgi?id=1758509
I also opened a support ticket in Customer Portal since it happens on our RHEL servers too.

The interesting part is that it works fine when run directly on the host. It's only inside a container that the issue appears. That's why I suspected that there is something wrong with how the container is run, either a fault by me or by podman.

Thanks this very well could be a bug in podman or in mongodb. But bugzilla is the place to put older versions of the tools, rather then upstream.

Ok, I put the component as systemd but maybe that should be changed to something else then. Didn't see podman or "container tools" there, though.
If it would help I could try the upstream libpod, either now (just to check if it's present in master) or when you have branch with a suggested fix.

With podman-1.5.1-3.fc30.x86_64 it runs fine:
● rh-mongodb34-mongod.service - High-performance, schema-free document-oriented database
   Loaded: loaded (/usr/lib/systemd/system/rh-mongodb34-mongod.service; enabled; vendor preset: disabled)
   Active: active (running) since Sun 2019-10-06 14:17:31 UTC; 49s ago
  Process: 55 ExecStart=/opt/rh/rh-mongodb34/root/usr/libexec/mongodb-scl-helper enable $RH_MONGODB34_SCLS_ENABLED -- /opt/rh/rh-mongodb34/root/usr/bin/mongod $OPTIONS run (code=exited, status=0/SUCCESS)
 Main PID: 69 (mongod)
   CGroup: /machine.slice/libpod-049c34384b9dce05a186656fea08b67e827fdefd504b2e29a7b7404e13727c8f.scope/system.slice/rh-mongodb34-mongod.service
           └─69 /opt/rh/rh-mongodb34/root/usr/bin/mongod -f /etc/opt/rh/rh-mo...

Oct 06 14:17:30 049c34384b9d systemd[1]: Starting High-performance, schema-f....
Oct 06 14:17:30 049c34384b9d mongodb-scl-helper[55]: about to fork child proc...
Oct 06 14:17:30 049c34384b9d mongodb-scl-helper[55]: forked process: 69
Oct 06 14:17:31 049c34384b9d systemd[1]: Started High-performance, schema-fr....
Hint: Some lines were ellipsized, use -l to show in full.

Cool! Will the fix be backported to the 1.4 branch anytime soon, since that's what's available on RHEL/CentOS right now?
Also, are there any binaries for CentOS for 1.5 or even 1.6? Or do I need to compile it manually in order to verify that it solves my issue?

@lsm5 Could you point at the Centos 1.5 or 1.6 packages?

Any update on when we can expect to see 1.5 or 1.6 in CentOS (or a 1.4.5 with the fix backported)?

@lsm5 Pointers?

I know the packages for podman 1.6 will show up in RHEL 8.1.1 which is scheduled to come out in January.


no subreads available

Hi,
I am working with sequencing output coming from Sequel IIe meaning that I only have the ccs.bam files available and not the subreads.bam.
Is DeepConsensus can work without the alignment of subreads to ccs using actc in the preliminary step?
Thanks

DeepConsensus requires subreads, which are not available when using Sequel IIe HiFi output.

Thank you for your answer.


Kill dotted view syntax for url reversing

Per django 1.8 docs, the 'app.views.viewname' method of url reversing
in templates with the url tag is deprecated and will be removed in
django 1.10.
This patchset creates a consistent set of names for all urls that need
to be reversed. All names are prefixed with app_name, for example,
lifestream-index.
This patchset also renames the existing lifestream view functions so
they are not prefixed with app_name. This brings them more in line with
other apps in hyperbola.
Fixes GH-31.
https://docs.djangoproject.com/en/1.8/ref/templates/builtins/#url

Things to QA:

[ ] logo link
[ ] header links
[ ] 404 return home link
[ ] lifestream index archive links
[ ] lifestream permalink link
[ ] lifestream paging links
[ ] lifestream archive paging links
[ ] lifestream hashtag paging links
[ ] lifestream hashtag links
[ ] lifestream 404 archive links
[ ] lifestream 404 return home link
[ ] resume link

QA done

deployed


[Feature request] Pass props to render() for aesthetic reasons

Would it be possible to have something similar to Preact where the render methods receives props as the first argument and context as the second one ?
ex:
   ...
   render({ username }) {
        return
Logged in as {username}

   }
}

@nightwolfz is this for render methods inside ES2015 classes? As functional stateless components already have this.

That's right. It's pure sugar but it would be nice to have (even if in a separate inferno package)

This will fall into development for Inferno 1.0 rather than 0.7. 0.8 development is being skipped from now on in favour of 1.0.

@nightwolfz are you on the Inferno Slack?

@nightwolfz did you get my last comment on this thread? Please email me at<EMAIL_ADDRESS>Need to speak to you ASAP :)

on a side note, this is now in 1.0.


Review PRs and Resolve Conflicts

While CI checks, we faced some errors in formatting files, to resolve them we created different branches and updated the files but when we created PRs, faced some conflicts to be resolved.

Since we agreed on ignoring these PRs, do you think we should delete this issue?


fix(merge/concat): merging or concatenating scalar observables will now com…

…plete properly

Removed a superfluous check for _isScalar that was doing the wrong thing and not completing the scalar observable

fixes #1150

:+1: change looks good to me.

@blesh is expand doing the same thing?

@trxcllnt it sure looks like it, yes.


Add running validation to the CLI tool

Currently, the @lionweb/validation package has to be used programmatically. (It's also run against the official test set.)
It'd be good if users could call npx @lionweb/cli validate <serializationChunk.js> and get a print-out (or even a JSON export, using a CLI flag) of all the issues reported by the validators.
Maybe we want to be able to configure what validators are running exactly. Maybe use a configuration file for that?

Done through PR #115 by @indamutsa


remove dependency to lazy_static

requires PR rust-lang/rust#71839
consequently, the latest nightly compiler has to be used

Seems like you have to update the asm! macro to please the CI

The problem was in the boot loader. We used an old version the x86 crate.


Add "Show Taps" feature

Can you add the feature "Show Taps" from Developer Options?
Thanks!

🧐👍

Can you add the feature "Show Taps" from Developer Options? Thanks!

Conser

This will be available in 361.


Enhance SledStorage transaction isolation level to SERIALIZABLE

Current transaction isolation level of SledStorage is SNAPSHOT ISOLATION or REPEATABLE READ.
It is not safe from phantom read concurrent conflict cases.
SledStorage only allows a single writer at the time and it is a bit better than SQLite3 in the matter of concurrency.
SQLite3 provides SERIALIZABLE level, so users don't need to care about concurrency conflicts.
Though it is required to sacrifice performance little bit more than the current, I think that we, SledStorage also needs to support SERIALIZABLE isolation level.
Transaction in SQL database is quite high level feature, so it is also supposed to provide users don't need to care about concurrency issues, I think.

Now we don't have plan to work on this.


To use the ES modules for esri JS Api 4.18 requires ncp copy of node_modules for BUILD and START. what and where do I need to copy assets for working in storybook.js?

To use the ES modules for esri JS Api 4.18 requires ncp copy of node_modules for BUILD and START. what and where do I need to copy assets for working in storybook.js?
Copy assets
You will need to copy the API’s assets, which includes styles, images, fonts, and localization files, from the @arcgis/core/assets folder to your build folder. A simple way to accomplish this is to configure an NPM script that runs during your build process. For example, use npm to install ncp and configure a script in package.json to copy the folder. Here’s a React example:
// package.json
{
    "script": {
        "start": "npm run copy && react-scripts start",
        "build": "npm run copy && react-scripts build",
        "copy": "ncp ./node_modules/@arcgis/core/assets ./public/assets"
    }
}

https://developers.arcgis.com/javascript/latest/es-modules/
I am reporting an issue with

[ ] TypeScript definitions
[ ] another resource in this repository

Haven't really used Storybook, but you could just have a copy script like in that sample and I think in Storybook it could be copied to stories/assets. The copy doesn't have to be done each time, only once in most apps to a directory that holds assets that the build tools will copy for you like. Usually that is in a folder called public, like with create-react-app. It looks like Storybook uses stories/assets for a similar purpose.

Yes you are right  here is what I did.
{
"scripts": {
"start-storybook": "npm run copy && start-storybook -s ./public -p 9001"
"copy": "ncp ./node_modules/@arcgis/core/assets ./public/assets"
}
}


Create a runnable distribution from assemble task

Signed-off-by: Hai Yan<EMAIL_ADDRESS>Description
This PR improves the assemble gradle task. It now calls an additional install task provided by the distribution plugin to create a runnable distribution. With this change, we can build and run data prepper by:

Run ./gradlew assemble
Go to release/archives/linux/build/install/opensearch-data-prepper-{VERSION}-linux-x64 and run bin/data-prepper

Issues Resolved
Resolves #1762
Check List

[ ] New functionality includes testing.
[ ] New functionality has been documented.

[ ] New functionality has javadoc added

[x] Commits are signed with a real name per the DCO

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

Found some unit test failures after rebasing on latest main. Pushed a commit to fix them.

Codecov Report

Merging #1774 (aeaa782) into main (e94d91a) will decrease coverage by 0.16%.
The diff coverage is 89.13%.

@@             Coverage Diff              @@
##               main    #1774      +/-   ##
============================================
- Coverage     94.11%   93.94%   -0.17%
- Complexity     1423     1431       +8
============================================
  Files           187      187
  Lines          4180     4213      +33
  Branches        336      339       +3
============================================
+ Hits           3934     3958      +24
- Misses          167      175       +8
- Partials         79       80       +1

Impacted Files
Coverage Δ

.../opensearch/dataprepper/parser/PipelineParser.java
84.89% <80.76%> (-2.83%)
:arrow_down:

.../dataprepper/peerforwarder/LocalPeerForwarder.java
100.00% <100.00%> (ø)

...taprepper/peerforwarder/PeerForwarderProvider.java
96.55% <100.00%> (ø)

...eerforwarder/PeerForwardingProcessorDecorator.java
100.00% <100.00%> (ø)

...dataprepper/peerforwarder/RemotePeerForwarder.java
86.00% <100.00%> (+2.27%)
:arrow_up:

...peerforwarder/server/PeerForwarderHttpService.java
97.77% <100.00%> (ø)

...rwarder/discovery/AwsCloudMapPeerListProvider.java
92.59% <0.00%> (-2.47%)
:arrow_down:

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.


Trinity hex encoding decoding utils

What was wrong?
Trinity needs it's own hexadecimal encoding/decoding utils
How was it fixed?
Copy/paste from the py-evm ones.  This is a stand alone to help make https://github.com/ethereum/py-evm/pull/310 smaller.
Cute Animal Picture

This should be against the trinity branch.


Can't genreate query with 'LIMIT %%limit int%%'

Driver: MySQL
MySQL version:  5.5.46-0+deb7u1 (Debian)

I've just try this but always getting error:
$ xo mysql://user:password@localhost:3306/db -o models --verbose -N -T BookingListResult << ENDSQL
SELECT * FROM booking LIMIT %%limit int%%
ENDSQL

SQL:
SELECT SCHEMA()
PARAMS:
[]

SQL:
CREATE VIEW _xo_g09uylbm AS (SELECT * FROM booking LIMIT NULL
)
PARAMS:
[]

error: Error 1064: You have an error in your SQL syntax; check the manual that corresponds to your MySQL server version for the right syntax to use near 'NULL
)' at line 1

It seems that the limit clause does not support NULL.

There isn't a way I can fix this in xo, that I'm aware of. This is really a limitation of MySQL. You could use custom query template to work around this.


Gatling 3.5.x support

Is this project maintainable or dead? Are there any plans to move on and support Gatling 3.5.x?

I also stumbled over this, but it seems like this project is no longer maintained, and the go-to solution (that seems to work fine from my point of view) is https://github.com/Tinkoff/gatling-jdbc-plugin/.


e2e grpcproxy tests

Enables -tags cluster_proxy for e2e.
Turns out a lot of stuff was broken! Skips v3 auth tests for now.

Does https://github.com/coreos/etcd/pull/8277/commits/caae11e473583ce16586d264795369e6be976147 need backport to 3.2 branch?

@gyuho no need to backport the transport one-- there's special cases everywhere to set it in the tls.Config derived from the tlsinfo. The snapshot EOF patch seems worth backporting though.

What is EndpointsJSON for?

@gyuho EndpointsJSON wasn't needed; removed


Compile the generated resources cannot find. / libs/three js/build/three. Min. Js

The picture is my version information. It can be reproduced without any changes


dut reference argument in subroutines(async functions other than @cocotb.test)

Describe the bug
A clear and concise description of what the bug is.

To Reproduce
Steps to reproduce the behavior:

Deploy any microbenchmark's benchmark into dv flow of alkaid_efpga
run the test and find the result failing
Cross check the converted testbench in dv/preconfig_cocotb/...

Expected behavior
Bitstreams should not be forced each time we call a subroutine

Screenshots

as you can see bitstream is forced and dut is used in reference, it gives object error, since reset_dut does not have dut in its scope
Enviornment (please complete the following information):

OS:

[x] Ubuntu 18.04

Tool:

[x] Synopsys IC Compiler

Version:

Additional context
Add any other context about the problem here.

@usmanali-rs Can you update your cocotb_utils to the latest? I have upgraded cocotb_utils which should already addressed this issue.

@usmanali-rs Can you update your cocotb_utils to the latest? I have upgraded cocotb_utils which should already addressed this issue.

yes working on updated utils. Thanks


Add comments and escapes to ECR docs

PR #2004 added some new features to ECR, here are the docs to go with them.
Also, I felt like the language was a little awkward in the examples. Originally the class: Greeting would greet a user with the phrase Greeting, KC but I believe it is more natural to call the class a Greeter and I'm sure (as a native English speaker) that the idiom is "Greetings, KC".
Sorry if that kind of input isn't welcome, I don't mean to be pedantic. I'm happy to revert that aspect of this PR if requested :)

@straight-shoota Thanks for the comments, I believe I implemented them all correctly.


HIVE-25250: Fix TestHS2ImpersonationWithRemoteMS.testImpersonation

What changes were proposed in this pull request?

Fix flaky test  TestHS2ImpersonationWithRemoteMS.testImpersonation. It was already fixed as part of HIVE-25093 but that is not the right way to do. Raising the PR again to create separate patch.
Why are the changes needed?

It is failing intermittently and putting unwanted pressure on hive-precommit  setup.
Does this PR introduce any user-facing change?

No
How was this patch tested?

Yes

@kgyrtkirk  Could please review the PR?

please run a flaky check on your changes to prove that it fixes the issue
http://ci.hive.apache.org/job/hive-flaky-check/235/

@kgyrtkirk
Thanks for pointing to flaky check pipeline. I ran and got the green build there
http://ci.hive.apache.org/job/hive-flaky-check/269/
Please review and merge the PR.

I'm a little bit amazed that changing the default maxRetries fixes the issue - could you please give some details about it; I'm really interested :)
+1

or the fix was in HIVE-25093? ...in case that's true I still don't see the connection between timestamps and the hs2 connection reset :)

@kgyrtkirk  Thank you reply.
When I did some changes in date_format UDF (HIVE-25093) and push the PR. In hive-precommit test only test which was failling is TestHS2ImpersonationWithRemoteMS.testImpersonation. After investigation I found that it has no relation with UDF but the test it self was flaky. So instead of created an new jira and raising a new PR for fixing TestHS2ImpersonationWithRemoteMS.testImpersonation. I fixed it as part of date_format UDF(HIVE-25093). Before (HIVE-25093) get merged you created new jira (HIVE-25250) and marked the test ignored and merged to master.
Now coming to why test failed. In test testImpersonation it try to connect to miniHs2.
DriverManager.getConnection(miniHS2.getJdbcURL(), "foo", null);
Since this is a network call there is a retry in place to avoid unwanted network failers. So we have a variable ("maxRetries") which control the number of retries and default value of which was "1" and that variable can be overridden by passing "maxRetries={somevalue}" as part of jdbc URL. Since default value "maxRetries=1" is very less. So I changed it to "maxRetries=5" as part of (HIVE-25093). But since you have marked this ignored so I am rasing this PR to uncomment the test and "maxRetries=5" seems to be at higher side. So I am making it maxRetries=3" and uncommenting the test.

@kgyrtkirk  Could you please merge the PR?

you have modified the default value of retries inside  jdbc/src/java/org/apache/hive/jdbc/HiveConnection.java in an unrelated changeset - and this changeset  follows that pattern;
I think going from 5 to 3 would be okayish but going from 1 to anything else is not the same - because you will enable the retries by default.
I don't think we should change things like that...that change should have done in a separate patch - because it have touched production code; and it changed the default behaviour as well.
It seems like maxRetries is not documented here https://cwiki.apache.org/confluence/display/Hive/HiveServer2+Clients
I think the correct approach is to set it back to 1 and change the retry number for the tests thru the jdbc url.

@kgyrtkirk  completely aligned with you. So I have reverted  the change and fix the test by adding retires with in the test cases.
Got green build in flaky test also
http://ci.hive.apache.org/job/hive-flaky-check/270/

thank you!

@kgyrtkirk Could you please merge the PR?


HIVE-26508: Remove netty transitive dependency from hcatalog and hbas…

…e handler pom files

What changes were proposed in this pull request?

Why are the changes needed?

Does this PR introduce any user-facing change?

How was this patch tested?

Hadoop needs to upgrade. Cannot remove the transitive dependency in hive.


clean up the readme

general syntax cleanup
move images to spec defined folder
crop images to fit nicely horizontally

Merged with master branch


'musicdl' is not recognized as an internal or external command, operable program or batch file.

'musicdl' is not recognized as an internal or external command,
operable program or batch file.

please help :D


[BACKLOG-29696] Add missing dependency to PDI Client

@pentaho/millenniumfalcon please review.

Build Skipped
:white_check_mark: There was nothing to build. The changes are probably excluded by the build rules. If this is an error, please contact your system administrator.


Fix issue with lead time not being added

See this issue for more information: https://github.com/tastyigniter/TastyIgniter/issues/1004

Do not merge this just yet, please.
I just discovered a major flaw: once the end time is being reached, this code throws an error. I will try to fix this issue today, and update my code.

I think a seperate extension suited better here, as multiple methods need rewriting to get this done, whereas a simple extension fixes the issue. On top of that, this is a much safer option for me instead of changing core files. I created the extension here: https://github.com/krozendaal/ti-ext-fixleadtime which is free to use for everyone.


create add function as optional parameter

Pull Request Test Coverage Report for Build 42

24 of 24 (100.0%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 100.0%

Totals

Change from base Build 40:
0.0%

Covered Lines:
59

Relevant Lines:
59

💛  - Coveralls

Closed since its highly specific for your use case.


Disable replace method combo box if bulk load is used

Describe the problem you're observing:
We need to disable replace methed because it can't be changed when using bulk load which uses it's own native tool's method

Verified


var substitution cleanup

Made a few changes that i think will simplify things:

easier to remove all those init functions - they would just confuse things and are far more verbose - harder to read

btw, the double underscore should not be used in code - its reserved for the python built-in methods

handle VAR and FIELD_MAPPING with the same regex
Minor breaking change:  don't allow "-" symbol in the variable names (not used in OMT anyway)

* easier to remove all those init functions - they would just confuse things and are far more verbose - harder to read

  * btw, the double underscore should not be used in code - its reserved for the python built-in methods

I agree that it's verbose, but it's on purpose to make the code clearer instead of a suite of for loops. I didn't know about the reserved usage of the double undescore, it seems to be a feature in the doc: https://docs.python.org/3/reference/lexical_analysis.html#reserved-classes-of-identifiers

* handle VAR and FIELD_MAPPING with the same regex

* Minor breaking change:  don't allow "-" symbol in the variable names (not used in OMT anyway)

Looks good!

@francois2metz fair enough, i added a function (should still not be 3 though imo).  Also i did what I should have done a  long time ago - ran a full local make test  -- apologies for the noise, should work now.


errors clean up

After the initial work on the errors, there is some more work that needs to be done to clean things up:

[ ] We need to add a way to handle the errors at the top of the package (probably gorgonia.go) which uses errors.Print to print the stack trace of errors which are caught to the user.
[ ] For this, we may wish to have a Handle(err) function which allows the users of this library to cleanly have access to our errors, which will errensially do a `errors.Print(err) if err is not nil.
[ ] We may wish to add this error handling to our tests. Currently, we user t.error(err) when an error is encountered which does not give us the stack trace.adding a Handle(err) would be a very cheap and effective way to add a stack trace to the errors in our tests which would help in debugging.

Updated your comment to turn it into a todolist. I love all the suggestions. If possible, we could even create a struct such that #60 could be tied in.