Support non-OpenShift use

Hello. I'm wondering if there's any chance this project might come to support more generic deployments than onto OpenShift? OpenShift is, at the moment, a requirement. I'd love to be able to manage all the pieces of Kogito effectively, but I am running on a different Kubernetes distribution, and would love to still be able to leverage kogito-cloud-operator.
Some questions I have that I hope would help raise understanding of what that might entail:

What challenges would there be trying to build kogito-cloud-operator atop, for example, something like the Operator Lifecycle Manager?
What specifically does kogito-cloud-operator need openshift for?
How feasible would it be to try to run any non-OpenShift specific parts of kogito-cloud-operator without OpenShift?

Hello @rektide ,
Yes, there is an ongoing effort to allow Kogito operator fully operate on plain Kubernetes environment. It is tracked in KOGITO-819 JIRA.
Currently it is possible to use Kogito operator to start and manage Kogito application images. These images however have to be built manually from Kogito project. You can take a look on quarkus-jvm.Dockerfile to see an example of Dockerfile used for image build.
For more informations see https://github.com/kiegroup/kogito-cloud-operator#kubernetes-support
A capability to build Kogito image on Kubernetes environment is currently in development, tracked in KOGITO-1999.
I don't have much experience with OLM on Kobernetes, though I am not aware of anything what would block Kogito operator from being deployed on Kubernetes using OLM.

Hi @rektide many thanks for reaching out.
@sutaakar already replied regarding our efforts on support "vanilla" Kubernetes.You should be able to use the operator with it's full capacity, but building custom services in the cluster, in the next version (estimated in about 2 weeks from now).
I'll try to add on top of the good answer gave by Karel:

What challenges would there be trying to build kogito-cloud-operator atop, for example, something like the Operator Lifecycle Manager?

I'm not sure if I understood your question. We already support OLM by default, you can see it on OpenShift Catalog. Unfortunately, up until the current version (0.10.0) you can only install it via OpenShift. On the next version you will see it on OperatorHub.io, meaning that you will be able to install it using OLM on Kubernetes as well.

What specifically does kogito-cloud-operator need openshift for?

In the beginning we used the BuildConfig resources to build custom Kogito services. That's the main reason. We are now working on add this support to Kubernetes as well. For the next version, 0.11.0, you will be able to deploy your custom Kogito Services images using the operator. On 0.12.0, hopefully you will be able to build the services there as well. To build your custom service, please see this link https://github.com/kiegroup/kogito-cloud-operator#kubernetes-support (pointed out by @sutaakar)

How feasible would it be to try to run any non-OpenShift specific parts of kogito-cloud-operator without OpenShift?

I'd say to wait for 0.11.0 to be released, since we introduce a new CR named KogitoRuntime to be able to deploy your custom services there, also we fixed some bugs that were happening when installing the operator on Kube. Data Index, Jobs Services and the other capabilities already run on Kube.
Reach me out if you have any further questions

Hi @rektide, please see:
https://operatorhub.io/operator/kogito-operator
Now you can use Kogito on any Kubernetes distribution out there. Please let me know if you have questions. :)


⚠ Bofrost.lu has degraded performance

In c5aa1ab, Bofrost.lu (https://www.bofrost.lu) experienced degraded performance:

HTTP code: 200
Response time: 1232 ms

Resolved: Bofrost.lu performance has improved in 047609d after 6 minutes.


Update to use mwdebug1001/2

Poke @Paladox

+1, thanks.

Doh! I made these changes myself before I got around to checking my email for the day. See 55cf22654ccca32adf0347d87c6946613e8f6b28. A new v0.8.3 package has been submitted to Mozilla for review.

@bd808 it looks like I missed one occourance of mw1017 in my PR anyway!
Thanks for your change!


remove an encompassing form-group that caused inconsistent spacing

I rule and have completed some work on Case Manager that's ready for review!
On the abortion information form there are two sets of checkboxes. The set under the input field is wrapped in a checkbox class div whereas the second set to the right were wrapped in a checkbox class div within a form-group div. The margins on the nesting of those two divs was causing the inconsistency. My quick fix is to remove the encompassing form-group as the checkbox class provides better styling for the checkboxes.
This pull request makes the following changes:

Removes two encompassing form-group divs from the app/views/patients/_abortion_information.html.erb

It relates to the following issue #s:

Fixes #1248

Codecov Report

Merging #1255 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master    #1255   +/-   ##
=======================================
  Coverage   93.52%   93.52%
=======================================
  Files          60       60
  Lines        1329     1329
=======================================
  Hits         1243     1243
  Misses         86       86

Continue to review full report at Codecov.

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

@colinxfleming I found you through the hacktoberfest tag


Why when I set low temperature (0.1-0.3), I get a lot of wild cards?

— я не */WILDCARD/* в */WILDCARD/* */WILDCARD/* */WILDCARD/* и */WILDCARD/* */WILDCARD/*, — негромко
произнес */WILDCARD/*. — я не */WILDCARD/* в этом */WILDCARD/* */WILDCARD/*. я не */WILDCARD/* в том, что
я не могу */WILDCARD/* в */WILDCARD/* */WILDCARD/*. я не знаю. мы не могу в */WILDCARD/*
*/WILDCARD/* */WILDCARD/* и */WILDCARD/* */WILDCARD/* в */WILDCARD/* */WILDCARD/*. а я не могу на него
*/WILDCARD/*. я не могу */WILDCARD/* в */WILDCARD/* */WILDCARD/*. скандалисткарты — я не могу в том, что я не могу
*/WILDCARD/* */WILDCARD/*? я не могу */WILDCARD/* */WILDCARD/* */WILDCARD/* и */WILDCARD/* */WILDCARD/*.

while Wildcard per cent: 2.524799 %...
Shouldn't wild cards in output have similar occurrence like wild cards in input?

Sorry, hit the wrong button and accidentally closed the issue. What I was going to say is that I have encountered a similar thing. I think that what is going on is that wildcard is replacing a lot of words that are a lot of different parts of speech and uses, so the network is overfitting to this. Low temperature falls back to safe choices and wildcard can mean anything, which is really safe.
What I'm working on is a way of generating multiple wildcards, say 1% of the number of rejected tokens, and distributing them randomly as we insert wildcards into the text.
I think this didn't come up earlier because of the same bug that was affecting CPU and Opencl, which was somehow suppressing wildcards as well.

Can't torch-rnn understand that the wild card is not a safe choice, but instead something unique like a name or foreign word and insert only where such unique things are expected by context?

Unfortunately no, there is no way of indicating that, forcing overtraining.

Wildcard randomization seems to reduce this significantly in initial experiments, why don't you give the latest commit a try and see how it goes?

Proposed change:
diff --git a/scripts/preprocessWords.py b/scripts/preprocessWords.py
index f684c76..7033b9e 100644
--- a/scripts/preprocessWords.py
+++ b/scripts/preprocessWords.py
@@ -17,6 +17,8 @@ parser.add_argument('--min_occurrences',type=int,default=20)
 parser.add_argument('--min_documents', type=int,default=1)
 parser.add_argument('--use_ascii', action='store_true')
 parser.add_argument('--encoding', default='utf-8')
+parser.add_argument('--wildcard_rate', type=float, default=0.01)
+parser.add_argument('--wildcard_max', type=int, default=10)
 args = parser.parse_args()

 if __name__ == '__main__':
@@ -95,7 +97,7 @@ if __name__ == '__main__':

     # Add a wildcard character onto the end of everything...

-    num_distinct_wild = min(10,int(0.01*total_eliminated))
+    num_distinct_wild = min(args.wildcard_max,int(args.wildcard_rate * total_eliminated))
     wildcard_ids = []

     for wcnum in xrange(num_distinct_wild):
@@ -130,7 +132,7 @@ if __name__ == '__main__':

     if not args.quiet:
         print 'Total unique words: {0}'.format(len(wordlist))
-        print 'Total vocabulary size: {0}'.format(len(token_to_idx))
+        print 'Total vocabulary size: {0} ({1} wild cards)'.format(len(token_to_idx), num_distinct_wild)
         print 'Total tokens in file: {0}'.format(total_size)
         print 'Total wildcards in file: {0} ({1}%)'.format(wildcard_replace_count,100.0*wildcard_replace_count/total_size)
         print '  Training size: {0}'.format(train_size)

Also I sometimes get words glued together without spaces:
голокучитпистолетнойигрушкаминаступлениемгруппировкамоборотнямзабытыхправославнойслучайнойпеченкойкарловичотличиянекрополяпосвящаллекойдвойниктитаникискрухамелеоновзавораживалиnewпрозвучать600кустарникомконтрабандыужитьсялайнскромномзадумаетсягардинызабываетесодратьварианте2a80куропаткакуропаткашокомсводилисьвстречаешьсясовременностизахареинъекциюизловчилсяуединенныйскачекобычномувыполнятпобитьпротиворечияобщественностисогнувшисьверевочкойнаступлениемin

Both on low and high temperature.

Can't wild cards be just artificially disvafoured by adding to score for each wild card in output when training?

I would rather set a wildcard min than max, plus the rate. As for words without spaces, the space is a learned token just like a word or punctuation. It must not be learning that words are divided by spaces somehow.

I also expected minimum and wondered why I get just 10 despite of increasing rate. Then I've looked closer, saw "max" and renamed the argument accordingly.

 If you are using tinyShakespeare, word culling results in poor performance, esp. with small layer sizes.

26M of Russian text (2 bytes per letter).   371409<PHONE_NUMBER>8743 tgcorp.txt

Does Russian rely a lot more on tense than English does? I would be really interested to test it on a language with a very complex but rigid tense system such as Kiswahili, which while it is very tense-specific is also very predictable to determine tense from preceding context.

Just a heads-up to maybe help you out. I added the option to seed text in word mode. Since Lua's regex support is lacking and a bit different from Python's I added a new script tokenizeWords.py which takes a string as a command line argument and transforms it into a JSON file with the appropriate tokens. sample.lua now takes this file as input option start_tokens.

Does Russian rely a lot more on tense than English does?

Note sure. In russian there are less auxilary verbs, but verbs may have ending depending on genus and multitude.
I've tried syllable mode (15k tokens, no wildcards) - it works better. Generated sentences are more coherent. Cases and prepositions mostly match, but genuses are often confused (like as in speech of a foreigner with limited Russian knowledge).
Interestingly, in low temperature (0.05-0.15) it generates variations of basically the same repetitive dialogue, regardless of the token set...
Now training the regular letter mode.

I'm going to close the issue, as it seems to be more architectural than project-specific.


Make API CORS header specific to console

Make API CORS Access-Control-Allow-Origin header specific to console instead of "*", with ability to override (needed when UI is run locally for testing purposes, for example)
Also refactored instance_test.go to cover more cases

approved, but i worry about the env var


monitor(engine): add basic Grafana panel for tiflow

What problem does this PR solve?

Issue Number: close #6995
What is changed and how it works?
Add basic Grafana panel for tiflow
Check List
Tests

Manual test (add detailed scripts or steps below)

Questions
Will it cause performance regression or break compatibility?
Do you need to update user documentation, design documentation or monitoring documentation?
Release note
 `None`.

/area engine

/status ptal

Can we put tiflow.json inside https://github.com/pingcap/tiflow/tree/master/metrics/grafana? It seems no conflicts.

/merge

/hold

Unit test failed @maxshuang
ref: https://ci.pingcap.net/blue/organizations/jenkins/ut-check/detail/ut-check/33127/pipeline

[2022-09-07T11:55:25.326Z] === Failed

[2022-09-07T11:55:25.326Z] === FAIL: engine/servermaster TestCollectMetric (1.00s)

[2022-09-07T11:55:25.326Z]     server_test.go:297:

[2022-09-07T11:55:25.326Z]         	Error Trace:	/home/jenkins/agent/workspace/ut-check/tiflow/engine/servermaster/server_test.go:297

[2022-09-07T11:55:25.326Z]         	            				/home/jenkins/agent/workspace/ut-check/tiflow/engine/servermaster/server_test.go:289

[2022-09-07T11:55:25.326Z]         	Error:      	Condition never satisfied

[2022-09-07T11:55:25.326Z]         	Test:       	TestCollectMetric

[2022-09-07T11:55:25.326Z]

[2022-09-07T11:55:25.326Z] DONE 399 tests, 1 failure in 332.797s

Unit test failed @maxshuang
ref: https://ci.pingcap.net/blue/organizations/jenkins/ut-check/detail/ut-check/33127/pipeline
[2022-09-07T11:55:25.326Z] === Failed

[2022-09-07T11:55:25.326Z] === FAIL: engine/servermaster TestCollectMetric (1.00s)

[2022-09-07T11:55:25.326Z]     server_test.go:297:

[2022-09-07T11:55:25.326Z]         	Error Trace:	/home/jenkins/agent/workspace/ut-check/tiflow/engine/servermaster/server_test.go:297

[2022-09-07T11:55:25.326Z]         	            				/home/jenkins/agent/workspace/ut-check/tiflow/engine/servermaster/server_test.go:289

[2022-09-07T11:55:25.326Z]         	Error:      	Condition never satisfied

[2022-09-07T11:55:25.326Z]         	Test:       	TestCollectMetric

[2022-09-07T11:55:25.326Z]

[2022-09-07T11:55:25.326Z] DONE 399 tests, 1 failure in 332.797s

I will fix it.

Codecov Report

Merging #6991 (0a05ba9) into master (9398176) will decrease coverage by 5.4458%.
The diff coverage is 59.1224%.

Additional details and impacted files

Flag
Coverage Δ

cdc
?

dm
51.8542% <25.0000%> (-0.0937%)
:arrow_down:

engine
62.3330% <60.4316%> (+0.0708%)
:arrow_up:

Flags with carried forward coverage won't be shown. Click here to find out more.
@@               Coverage Diff                @@
##             master      #6991        +/-   ##
================================================
- Coverage   59.8253%   54.3794%   -5.4459%
================================================
  Files           791        420       -371
  Lines         90221      52187     -38034
================================================
- Hits          53975      28379     -25596
+ Misses        31498      21077     -10421
+ Partials       4748       2731      -2017

/run-dm-compatibility-test
/run-dm-integration-test
/run-kafka-integration-test
/run-integration-test
/run-engine-integration-test

/unhold

/merge


Set an arbitrary Girder item as a Minerva GeoJSON dataset

Rather than require the item to be uploaded, it can be marked via the POST minerva_dataset_geojson endpoint.
Added a globe icon to the Girder file list to call the endpoint for a specific file within an item.  When a file is used as a Minerva dataset, a globe with an x is shown, clicking which removes the minerva metadata.
If a dataset that is in use in a session is unmarked (by having the minerva metadata tag removed), don't throw javascript exceptions.
The POST endpoint didn't set all of the metadata that was required for sources, etc.  This has been changed, but has not been tested to make sure it doesn't have ill repercussions.
Fixed some CSS that caused Girder to look poorly and require horizontal scrolling.  Also fixed an issue when the main Minerva body would show a vertical scrollbar.
This PR also merged the changes on master.

@dorukozturk or @aashish24 Did we want to get this reviewed so it can be merged into the sources-refactor branch?

LGTM


Add resistor-color-duo

I’m contemplating upgrading the track to LFE 2.1.4 / Erlang 26. That might help the breakage. There’s nothing wrong with phone number since earlier, it was binary breaking intermittently. However, I suspect a lot of the exercises haven’t been synced for years including phone number so  a parallel task would be to verify that all the tests in the toml file are in fact implemented.


Convert codelab to use emulators

See staged copy here:
https://fir-codelabs-89252--173-zw7hnyvm.web.app/
Major changes:

Change codelab to use emulators
Require Firebase CLI
Move to FirebaseUI for Auth, enable email/password

TODO:

[x] Change build-android-start
[x] Add a step explaining useEmulator
[x] Clean up the final step
[x] Fix google-services.json

cc @kroikie @racheldosh

This PR made my day! Few comments:

Step 5 could use a note about running the app on a real device vs emulator (given localhost might not work).
Step 10 fix rule formatting

Amazing stuff!

I had a comment in my last review (I think it went through?) about Security Rules for Storage. Are the defaults ok or do we need to beef them up?
Other than this question above about rules (and the few nits throughout that I suggested changes for), this PR is gtg.

I filed a separate issue to beef those up.


Added Selenium test coverage for actions.js.

Rebased

buildbot, test on selenium.

buildbot, test on selenium.


chore(deps): update ci dependencies

This PR contains the following updates:

Package
Type
Update
Change

common-dev-assets

digest
5f5fd35 -> 5cf7d9c

github.com/terraform-ibm-modules/ibmcloud-terratest-wrapper
require
patch
v1.42.5 -> v1.42.6

Release Notes

terraform-ibm-modules/ibmcloud-terratest-wrapper (github.com/terraform-ibm-modules/ibmcloud-terratest-wrapper)
v1.42.6
Compare Source
Bug Fixes

deps: update module github.com/gruntwork-io/terratest to v0.48.1 (#905) (511dc17)

Configuration
📅 Schedule: Branch creation - At any time (no schedule defined), Automerge - At any time (no schedule defined).
🚦 Automerge: Disabled by config. Please merge this manually once you are satisfied.
♻ Rebasing: Whenever PR is behind base branch, or you tick the rebase/retry checkbox.
👻 Immortal: This PR will be recreated if closed unmerged. Get config help if that's undesired.

[ ] If you want to rebase/retry this PR, check this box

This PR has been generated by Renovate Bot.

ℹ Artifact update notice
File name: tests/go.mod
In order to perform the update(s) described in the table above, Renovate ran the go get command, which resulted in the following additional change(s):

1 additional dependency was updated

Details:

Package
Change

github.com/gruntwork-io/terratest
v0.48.0 -> v0.48.1

/run pipeline

ℹ Artifact update notice
File name: tests/go.mod
In order to perform the update(s) described in the table above, Renovate ran the go get command, which resulted in the following additional change(s):

1 additional dependency was updated

Details:

Package
Change

github.com/gruntwork-io/terratest
v0.48.0 -> v0.48.1

/run pipeline

/run pipeline


Implement all animations.  #24

add all animations to src/constants.ts
Check if animations is set to all and load from constants
allow type all in animations config

Thanks!

Thanks! That was very quick!


Force chart.js version - otherwise chart.js 2 is getting installed

chart.js 2 beta is released on npm, so when i install react-chartjs,  chart.js v2 is getting installed as peer dependency. force it to v1.

Can you lax it up to where the requirement is just 1.x?

@austinpray The ^ will make any 1.x above 1.1.1 work, btw


chore(cert): regenerates certs for ssl test

Generates a new rootCA and host key/cert for ssl testing.
I've not done this before, but read a few openssl articles and tried a few different things. This has the tests passing. If there's more to it that I missed, let me know!

Fixes #127


Update installation instructions to save to devDependencies

Since this is a package used during development, and is not a package containing files that get published, it should be a dev dependency.
Related PR: https://github.com/11ty/eleventy-img/pull/109

Same as https://github.com/11ty/eleventy-img/pull/109#issuecomment-894322522 and I also applied the change here, thank you!


port mainnet profile to dev

Port https://github.com/MinaProtocol/mina/pull/16292 to develop

!ci-build-me

!ci-build-me


Outter class name

This PR gives you the ability to add a className to the outter div ("placeholder" div I believe?). This ultimately allows you to have more control over this div, which is currently needed in our use case.
I have seen another pull request that has a different way of solving this issue, which I actually agree with, however, I believe this makes it easier to cover more edge cases in that whoever is using this library can just add a className to that div and do what they need to do.

Seems like a good idea to me. Are you able to update this by spelling outter as outer though? 😄

Good catch! Fixed

Out of date with upcoming release of 6.0.0


A single un-reportable point should not fault reporting cycle in WavefrontYammerHttpMetricsReporter

Line 287 in WavefrontYammerHttpMetricsReporter

Wrong repo, moved to https://github.com/wavefrontHQ/java-lib/issues/27


Geometric distortion

Is it possible to use this to measure geometric distortion? As in the quality of a retargeting compared to a full reference?

Our dataset had geometric distortions, and deep features outperformed traditional metrics.

Thank you. Do you have an example I can look at? I would like to compare against metrics for measuring retargeting success such as amount of geometric distortion introduced and information loss. Typical SSIM, L2, L1 losses have not been used for these purposes.

Great. Unfortunately, I don't have specific examples with just geometric distortions.


Wrong endpoint hostname in m4dapplication status

The hostname reported in the endpoint spec in m4dapplication status seems to be different from the python snippet and the service name in m4d-blueprints
the following is from the M4DApplication status:
Read Endpoints Map:
m4d-notebook-sample/paysim-csv:
Hostname:  my-notebook-m4d-notebook-sample-arrow-flight-mo-53080.m4d-blueprints.svc.cluster.local
Port:      80
Scheme:    grpc
client = fl.connect("grpc://my-notebook-m4d-notebook-sample-arrow-flight-mo-53080-arrow-fli.m4d-blueprints.svc.cluster.local:80")
kubectl get services -n m4d-blueprints
NAME                                                              TYPE        CLUSTER-IP     EXTERNAL-IP   PORT(S)   AGE
my-notebook-m4d-notebook-sample-arrow-flight-mo-53080-arrow-fli   ClusterIP   <IP_ADDRESS>           80/TCP    51m

Fixed by https://github.com/mesh-for-data/arrow-flight-module/pull/87


Admin Orders not showing

Bug report
Issue Description
When creating an order for the store, the orders are not reflected in the administrative panel.
The error is due to the fact that the condition in line 178 of this file is not fulfilled and therefore the component is not initialized correctly. Because the statusText is empty """

Possible related to #5118

Bagisto version

[x] 1.3.2

Expected result
View orders in the administrative panel.

Actual result
Nothing is visible

This problem has been solved in #5124


Diff Editor - pattern glitch

Steps to Reproduce:

Open diff editor that has code moved around
Notice the glitch in the pattern

See also #200742

/duplicate #200742


Tick strings on the bottom axis overlaps with its label

Bottom axis tick strings do not seem to account for the drawing of axis labels. This could be recreated by changing line
win = LivePlotWidget(title="Vertical Bar Plot @ 100Hz")

to:
win = LivePlotWidget(title="Vertical Bar Plot @ 100Hz", labels={'bottom': ("hello"), 'left': ("hello")})

in vertical_bar_plot.py of examples. Here is what it looks like:

This does not seem to be a problem with other axes, just the bottom one.

Thanks for debugging!
Yes, this was newly intorduced issue in 0.7.0. It should be fixed in 0.7.1.

Well, it sure doesn't overlap now, but there's more null space:

Compared to in v0.6.8:

It's a minor nitpick though, as it fixes the problem. Ideally it would look exactly the same and the two axes would look more congruent, but it shouldn't matter that much.


Make songs in a playlist download in order

Requested Feature
When downloading a playlist, it would be nice if the songs were downloaded in order they appear in the playlist
Possible implementation
No response

Songs are actually downloaded in order, but it may not appear like that because one song might download faster because it's smaller while other might get downloaded instantly.

The generated m3u is in alphabetic order not in the original playlist order. Is there an option to control this?
The current option to generate a playlist is:
--m3u


Top-level paths are lost during import

Hello. This may be user error, but I'm trying to import from a json file that I created using vkv snapshot save. I actually have 4 of these files, one for each KV engine that I have.
When importing, the top-level paths are lost. Here's an example of the data:
{
  "toplevel/": {
    "sublevel": {
      "AWS_ACCESS_KEY_ID": "laskdfjalskdjfl",
      "AWS_DEFAULT_REGION": "us-east-1",
      "AWS_SECRET_ACCESS_KEY": "kasdhflksdjfls",

My import command looks like this:
./vkv import -f vault-export-2024-02-15/cicd.yaml -p cicd

Then, when I see the secrets in Vault, toplevel is gone, and so the path will be like cicd/sublevel/.
I think I'm probably doing something wrong. Any help would be appreciated. Thanks.

Thanks for the report. I will try to reproduce it and get back to you very soon!

Okay, got it:
You have to omit -p cicd since that tells vkv to import the secrets under the specified mount. I chose that behavior, because this way you can move KV secrets to other mounts or even subpaths :)
LMK if that solved your problem

@FalcoSuessgott it won't even allow that:
[root@ip-10-2-162-214 ~]# ./vkv import -f vault-export-2024-02-15/cicd.yaml
[ERROR] no KV-path given, -path / -p needs to be specified.

I do want cicd/ in the path, as that is the name of that KV engine. So the full path should end up being cicd/toplevel/sublevel.
Thanks for checking on this.

Oh I see, you are using vkv import. I was speaking of  vkv snapshot restore, which lets you restore a snapshot export at the specified path. In general a vkv snapshot works better with vkv snapshot restore. Maybe that also works in your case.
However, using vkv import, you can simply move secrets around by piping the yaml/json output of vkv export into vkv import:
>  vkv export -p secret
secret/
├── v1: admin [key=value]
│   └── sub=********
├── v1: demo
│   └── foo=***
└── sub/
    ├── v1: demo
    │   ├── demo=***********
    │   ├── password=******
    │   └── user=*****
    └── sub2
        └── v2: demo [admin=false key=value]
            ├── admin=***
            ├── foo=***
            ├── password=********
            └── user=****

>  vkv export -p secret -f yaml | vkv import - -p cicd/secret --force
reading secrets from STDIN
parsing secrets from YAML
writing secret "cicd/secret/admin"
writing secret "cicd/secret/demo"
writing secret "cicd/secret/sub/sub2/demo"
writing secret "cicd/secret/sub/demo"
successfully imported all secrets

result:

cicd/secret/
└── secret/
    ├── admin
    │   └── sub=********
    ├── demo
    │   └── foo=***
    └── sub
        ├── demo
        │   ├── demo=***********
        │   ├── password=******
        │   └── user=*****
        └── sub2
            └── demo
                ├── admin=***
                ├── foo=***
                ├── password=********
                └── user=****

>  vkv export -p cicd
cicd/
└── secret/
    ├── v2: admin
    │   └── sub=********
    ├── v2: demo
    │   └── foo=***
    └── sub
        ├── v2: demo
        │   ├── demo=***********
        │   ├── password=******
        │   └── user=*****
        └── sub2
            └── v2: demo
                ├── admin=***
                ├── foo=***
                ├── password=********
                └── user=****

Aha, yes that did. See, I knew I was doing something wrong 😆
Thanks for your help.


TypeGuard for all nodes with a .expression?

Hey David,
Was just working on some AST transforms, and I noticed a pattern that I want to try to take advantage of.
I'm basically trying to replace identifiers such as myVar with this.myVar. I noticed that I basically need to find compiler nodes which have an expression property that contain an Identifier node.
For example, CallExpression, PropertyAccessExpression, IfStatement, and probably many more have an expression property. Was thinking I could transform them with something like this:
// ...traversing nodes
if( TypeGuards.isExpressionableNode( node ) ) {  // <-- does this name make sense?
    if( TypeGuards.isIdentifier( node.getExpression() ) {
        node.getExpression().replaceWithText( 'this.' + node.getExpression().getText() );
    }
}

Btw, any general thoughts on this? Is there a better way to go about this?

By the way, is there any way to manipulate the underlying compiler nodes directly, and tell ts-simple-ast about it?
I realized I could replace the Identifier node with a PropertyAccessExpression by calling:
compilerNode.expression = ts.createPropertyAccess( ts.createThis(), 'myProp' );

But when I go to output the source using sourceFile.getFullText() it seems that the update didn't take place.

Hey Greg, I'll take a look and see. This looks like it would be useful to have.
Regarding using the ts.createX methods, those methods only change the tree and not the underlying source file text. For example, try this out:
// untested, but from my memory I believe it works this way
compilerNode.expression = ts.createPropertyAccess( ts.createThis(), 'myProp' );
compilerNode.getText(); // won't have this.myProp
compilerNode.getSourceFile().getFullText(); // won't have this.myProp

I'm not entirely sure how to propagate that change back up to the parent source file's text or the other node's properties (ex. pos and end) based on the changes. Do you know a way? I know they do this with code fixes. If we figure that out, then it would make sense to add a method to SourceFile that will run that code. There's also some extra work that would need to be done in the replaceTree code to ensure the old nodes are removed from the underlying cache.

Hey David,
Interesting.. Yes, I did notice that the text wasn't changed, and also after asking this question I did some more digging. Is it that ts-simple-ast pretty much does all manipulations on the original source text, and then creates a new AST out of that new source text afterwards?
I thought of a possible workaround yesterday as well to try to achieve my source transformations of expressions / variable assignments / function calls, which is to:

Take the node's compilerNode, for instance a ClassDeclaration's compilerNode
Perform raw AST transformations on it by using a transformer and visitor (http://blog.scottlogic.com/2017/05/02/typescript-compiler-api-revisited.html)
Pretty-print the new AST using ts.createPrinter() (https://github.com/Microsoft/TypeScript/wiki/Using-the-Compiler-API#creating-and-printing-a-typescript-ast)
Take that pretty-printed text and use it with classNode.replaceWithText(prettyPrinted)

Feels like a total hack though.. Maybe it would be good to have an API to more easily allow for these kinds of visitor transformations?

Also a downside of my workaround is that I'm losing single line comments and such that are in the source :/

I'll respond in a bit, but there is a removeComments option on the printer. Try setting that to false. It will nuke any blank lines though and change a lot of formatting. I've tried to use it before... definitely not ideal to use.
By the way, this discussion has been overtaken by supporting node factories. Would you mind creating a new issue that's a copy and paste of the first comment and then renaming this one to something about node factories? I think this discussion would be useful for other people to read and it would be nice to have a searchable title for it :)

So I figured out my issue with missing comments.
It seems the printer won't remove comments by default, but I was creating a synthesized copy of each AST node using ts.getSynthesizedClone(node) (as to not mutate the original), and that's what was actually making me lose my comments.
Removing the cloning and mutating the original AST + printing it at least got me my comments back :) (although I lose my original source formatting, like extra line breaks where it makes sense in the code)
Still wish there was a cleaner way to do this where I could mutate the AST and keep the original formatting around the mutated nodes

This is implemented now. Some examples:
import * as ts from "typescript";

const project = new Project();
const sourceFile = project.createSourceFile("Example.ts", "1; 2; 3;");

// this can be done starting on any node and not just the root node
sourceFile.transform(traversal => {
    const node = traversal.visitChildren(); // return type is `ts.Node`

    if (ts.isNumericLiteral(node)) {
        const incrementedValue = parseInt(node.text, 10) + 1;
        return ts.createNumericLiteral(incrementedValue.toString());
    }

    return node;
});

// outputs: 2; 3; 4;
console.log(sourceFile.getFullText());

import * as ts from "typescript";

const project = new Project();
const sourceFile = project.createSourceFile("Example.ts", `
class C1 {
    myMethod() {
        function nestedFunction() {
        }
    }
}

class C2 {
    prop1: string;
}

function f1() {
    console.log("1");

    function nestedFunction() {
    }
}`);

sourceFile.transform(traversal => {
    // this will skip visiting the children of the classes
    if (ts.isClassDeclaration(traversal.currentNode))
        return traversal.currentNode;

    const node = traversal.visitChildren();
    if (ts.isFunctionDeclaration(node))
        return ts.updateFunctionDeclaration(node, [], [], undefined, ts.createIdentifier("newName"),
            [], [], undefined, ts.createBlock([]))
    return node;
});


Catalog app YAML editor doesn't support keys with dots in answers

What kind of request is this (question/bug/enhancement/feature request): bug
Steps to reproduce (least amount of steps as possible):
Create a catalog app (Prometheus in my case) and supply answers that aim for this structure when using the yaml editor, then apply
alertmanagerFiles:
  alertmanager.yml: value

Result:
The yaml is saved in this format
alertmanagerFiles:
  alertmanager:
    yml: value

I've tried escaping the dot in the yaml, but it is not preserved
Environment information

Rancher v2.1.1
User Interface v.2.1.4
Helm v2.10.0-rancher5
Machine v0.15.0-rancher1-1
HA Installation

@jamesmcminn96
Can you try this pattern, please?
alertmanagerFiles.alertmanager\.yml: value

I think https://github.com/rancher/rancher/issues/13158 will address this issue

Version: Master 2.2 (2/20/19)
Confirmed issue. See #17605 which covered related testing as well.
When you edit as YAML this is passed as a yaml file in valuesYaml. This is working for me due to this change ( https://github.com/rancher/rancher/pull/17576 PR )
I again validated the http/api like I did in my #17605 testing.
When I inspect the valuesYaml string our whitespaces/carraige returns appear correct just like how they were created in the UI.

This still occurs with Rancher 2.3.3.
Repro:
have a "values.yaml" with
ingress:
  annotations:
    kubernetes.io/ingress.class: something

And deploy it with rancher apps upgrade --values values.yaml ...
Will cause cause it to be rendered internally as below, you can also see that when clicking "edit as yaml" in the "upgrade" screen.
ingress:
  annotations:
    kubernetes:
        io/ingress:
            class: something

The workaround with escaping produces the following error:
Failed to install app blog. Error: UPGRADE FAILED: v1beta1.Ingress.ObjectMeta: v1.ObjectMeta.Annotations: ReadString: expects " or n, but found t, error found in #10 byte of ...|ls-acme":true,"nginx|..., bigger context ...|ess.class":"ingressClass","kubernetes.io/tls-acme":true,"nginx.ingress.kubernetes.io/force-ssl-redirec|...

Other users are also affected by this: https://github.com/rancher/rancher/issues/22027

Hi, is there any news on this? i'm having this issue as well

This is still true for rancher 2.3.5
passing key with dots rbac.serviceAccountAnnotations.”eks\.amazonaws\.com/role-arn” renders to
serviceAccountAnnotations:
      "eks:
        amazonaws:

Still the same issue with rancher 2.5.8
spring:
  mail:
     host:


Don't store call-site text offsets in hygiene info

This threads a lot more database references around in order to avoid storing a bare TextOffset in the hygiene info. This TextOffset made hygiene info and ItemTrees more volatile than they should be, leading to excessive recomputation of ItemTrees.
The incremental test added in https://github.com/rust-analyzer/rust-analyzer/pull/8721 is now passing with these changes.
closes https://github.com/rust-analyzer/rust-analyzer/pull/8721

After this PR, we could remove Hygienic::new_unhygienic too !

Threading db everywhere always good. LGTM, but maybe make db a public filed of LowerCtx and don’t pass the two around together?

r=me

bors r=matklad


Git status doesn't populate

The following is the error I get when I run Git: Status.
KeyError: '#'
Traceback (most recent call last):
  File "/Applications/Sublime Text.app/Contents/MacOS/sublime_plugin.py", line 549, in run_
    return self.run(edit)
  File "SublimeGit/sgit/status.py", line 520, in run
  File "SublimeGit/sgit/status.py", line 120, in build_status
  File "SublimeGit/sgit/status.py", line 165, in build_files_status

Not sure what is causing it.

Sorry for highjacking this issue but maybe I've got a related bug. I guess it is caused by files determined as 'copied' by git (shouldn't happen to often). This was my git status --porcelain from command line:
M  www/index.html
M  www/js/app.js
M  www/js/controllers.js
C  www/js/controllers.js -> www/js/library.js
M  www/js/services.js

The error in the Sublime Python console:
Traceback (most recent call last):
  File "/Applications/Sublime Text.app/Contents/MacOS/sublime_plugin.py", line 550, in run_
    return self.run(edit, **args)
  File "SublimeGit/sgit/status.py", line 520, in run
  File "SublimeGit/sgit/status.py", line 118, in build_status
  File "SublimeGit/sgit/status.py", line 156, in build_files_status
KeyError: 'w'

git version 2.3.2 (Apple Git-55)
You have SublimeGit 1.0.34


homeassistant-core-install.sh needs to tackle issues with python3.12 on debian

Please verify that you have read and understood the guidelines.
yes
A clear and concise description of the issue.
When installing homeassistant into a debian LXC Container, the homeassistant-core-install.sh script will fail in this way:
`Reading package lists... Doneatience)
Building dependency tree... Done
Reading state information... Done
Note, selecting 'python3-pip' instead of 'pip'
|E: Unable to locate package python3.12-dev
E: Couldn't find any package by glob 'python3.12-dev'
E: Couldn't find any package by regex 'python3.12-dev'
[ERROR] in line 17: exit code 0: while executing command $STD apt-get install -y git curl sudo mc bluez libffi-dev libssl-dev libjpeg-dev zlib1g-dev autoconf build-essential libopenjp2-7 libturbojpeg0-dev ffmpeg liblapack3 liblapack-dev dbus-broker libpcap-dev libavdevice-dev libavformat-dev libavcodec-dev libavutil-dev libavfilter-dev libmariadb-dev-compat libatlas-base-dev pip python3.12-dev`
This is due to the fact that homeassistant ditched the support of python versions <3.12. However debian bookworm only provides python 3.11.
@pascallj provides a backport of python3.12 specifically to tackle this particular issue within homeassistant. ( #see
I would suggest to implement that within the homeassistant-core-install.sh script. Alternatively the process could provide a hook in the advanced settings to open a shell within the running container prior to the call of homeassistant-core-install.sh.
Cheers
Amie
What settings are you currently utilizing?
Advanced Settings
Which Linux distribution are you employing?
Debian 12
If relevant, including screenshots or a code block can be helpful in clarifying the issue.
No response
Please provide detailed steps to reproduce the issue.
No response

Related issue: https://github.com/tteck/Proxmox/issues/3892
Adding custom python backports from unknown authors not something we want in our scripts (feel free to use it yourself manually though). You should use the default distribution which is clearly stated in documentation.


WIP - add more styles

This gets it closer to Shorlander's mockup.  Any suggestions on how to improve it are welcome!
for reference https://people-mozilla.org/~shorlander/projects/photon/Mockups/windows-10.html

I asked Stephen about this, and he said that I should use the version without any of the options checked, so no extra drag space in this version.  (Perhaps I'll add an about:visuals options page later to let us toggle stuff.  ;) )

@bwinton That would be a great exercise for us to 1. Build some about pages and 2. Have some kind of prefs story.

Is this ready for re-review?

Almost. Adding a few more changes first. I'll comment here once it's ready.

@bwinton please test the changes on windows. @victorporof ready for re-review :)

(I made some minor tweaks on Windows, but other than those, it looks good.)

Note that when the blue flash tab loading animations are displayed, the selected tab's top blue bar disappears. Your choice if you want to fix that here or in a later PR.

New PR.  (It looks like we'll need a new gif from Eric…)

Window controls are still not vertically aligned on macOS, as per #96 (review) Again, your choice if you want to address this in this PR or not, but have you tried looking into it?

We did, and this is hella-hard to fix.  Possibly impossible, unless we want to draw the macOS windows controls ourselves…


Doc typo: "config.resovle"

In the index.js doc comments, config.resovle should be config.resolve. (This is visible in README.md, too.)
FWIW, I tried making a pull request, but I couldn't get verb-cli working because of verbose/verb#62.

thanks for the heads up! I'm working on that verb issue now, trying to refactor it to the basics so we don't have these issues anymore. If you'd like to do a pr to just edit the .verb.md file that would be great, I can build it. Or I'd be happy to fix the typo, whatever you prefer

No problem! Just pushed to #4.


it will override all the other settings to false each time of iterator

https://github.com/OmniLayer/omnicore/blob/a2c114c9645bbff7e3ae5b10ac3713c6589867c8/src/omnicore/log.cpp#L239C70-L239C70
need to change to:
#define ENABLE_LOG(x) msc_debug_##x = none ? false : (all || d == #x || msc_debug_##x)

Detail is stated here #1310


Mobile format has an blank space added to the right outside of viewport after adding AOS

This is:

Bug / - Question

Specifications

AOS version: aos V 2.0
OS: Mobile format galaxy S8+, iPhone 8+, samsung galaxy tab
Browser: Chrome / Samsung internet browser

Expected Behavior
Pages were mobile responsive before using AOS
Actual Behavior
AOS is creating a blank space to the right of page of mobile devices. Laptop screens work as expected
Steps to Reproduce the Problem
http://dninstalls.com/

Possible Solution
Tried aos@next, did not fix.
Tried adding this to CSS / HTML did not fix
CSS
.aos-init:not(.aos-animate):after {
position: fixed;
}
HTML
< div class="aos-init" data-aos="zoom-in-up" data-aos-duration="3000" >

Hi there, I came here to search the exactly same problem haha
I used the CSS code above and indeed don't work, but when I tried to set overflow-x: hidden on the body tag, worked, but the scrollbar appears above the page. But I guess that is a beggining haha
body{overflow-x: hidden}
.aos-init:not(.aos-animate):after {position: fixed;}

@ledones, when I set the body overflow-x to hidden all the content that has AOS features, is hidden.

Hi ,
I had the same problem when I used this long back. It was a simple solution and it had nothing to do with editing CSS. If I remember correctly it was not ending the div or missing an ">". If I look at your website, when that sticky header appears, the website will not have white space on the right.
I believe, the problem is that you made a simple mistake in syntax.  What you can do is remove all the occurrence of AOS and start by adding one section at a time. Find out where the problem starts. I will let you know if I remember exactly how i fixed the problem.

I guess the css transform: translateX while using fadeInRight, slideInRight animation is also creating this issue. Any solution? Tried changing the position to fixed but doesn't work.

@lazehang There were three aos that were causing the problem, fade-up-right, fade-up-left, and zoom-in-up. The zoom-in-up was applied to an entire row with four columns, I am using the zoom-in-up on a single column row that is the title line with no problem.
I did try hiding the overflow on the X axis based on someone's recommendation but it caused all the aos elements to disappear.

The same here.
I have a two section tags, the one below with a negative padding to give a nice effect and the aos is applied on this one.
Everything works fine if i set overflow-x: hidden but in this way i lose the section effect

@djneill Yes, That was the problem. I remember now. I used the effect Fade from Right. Which was creating the issue. Later I used Fade from Down, which rectified the problem.

This is a known issue, and you got to solve it yourself by adding overflow-x: hidden on body or other parent container. I'll put this information in the FAQ section :)
If overflow hides your elements in a weird way it means you used floats and didn't use clearfix on the container.

body{overflow-x: hidden}

Worked perfectly.

Yessir, I can also attest to this. I had the exact same problem with extra white spacing to the right on mobile.
In the mobile @media queries just use the
body {
overflow-x: hidden
}

Hi,
I've the same problem. :-(  I try use:
body {
overflow-x: hidden
}
But not solve the problem. Any idea? Thank you so much.

Could you provide more info?
On Mon, 30 Mar 2020 at 5:17 PM, fabiosilvanet<EMAIL_ADDRESS>wrote:

Hi,
I've the same problem. :-( I try use:
body {
overflow-x: hidden
}
But not solve the problem. Any idea? Thank you so much.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/michalsnik/aos/issues/390#issuecomment-605882170, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AEKS63DS73EHH2ZH4GT4GALRKBPR7ANCNFSM4FRUSRTQ
.

This is a known issue, and you got to solve it yourself by adding overflow-x: hidden on body or other parent container. I'll put this information in the FAQ section :)

NOT or o not put it into the body section! Use the .container or .wrapper oder the #app or #root DIV's your page is using! That will work...
Das einzige was wirkt:  overflow-x: hidden; in einen Übergeordneten Container einfügen!

For some reason using "overflow-x: hidden" on both body and html elements messed up my animations, causing some to not appear, so I've instead used the following jQuery to add "overflow: hidden" to all parents of animated elements, which seems to have done the trick:
$('[data-aos]').parent().addClass('hideOverflowOnMobile');
And then in CSS:
@media (max-width: 768px) {
.hideOverflowOnMobile {
overflow: hidden;
}
}

you my guy are a smart man

For some reason using "overflow-x: hidden" on both body and html elements messed up my animations, causing some to not appear, so I've instead used the following jQuery to add "overflow: hidden" to all parents of animated elements, which seems to have done the trick:
$('[data-aos]').parent().addClass('hideOverflowOnMobile');
And then in CSS:
@media (max-width: 768px) {
.hideOverflowOnMobile {
overflow: hidden;
}
}

Your solution worked perfectly. Thanks

overflow-x: hidden

tnx

Want to flag that using
body {
  overflow-x: hidden;
}

then prevented me from being able to scroll vertically on mobile (overflow: hidden obviously had the same effect).

For some reason using "overflow-x: hidden" on both body and html elements messed up my animations, causing some to not appear, so I've instead used the following jQuery to add "overflow: hidden" to all parents of animated elements, which seems to have done the trick:
$('[data-aos]').parent().addClass('hideOverflowOnMobile');
And then in CSS:
@media (max-width: 768px) {
.hideOverflowOnMobile {
overflow: hidden;
}
}

Works perfectly!

using sveltekit and had everything wrapped in a  tagged and this ended up working perfectly
@media only screen and (max-width: 600px) {
  main {
    overflow-x: hidden;
  }
}

make data-aos-once=true it will make it appear once but it will make the right space disapear its always there because its always active

thanks for the solution  🥹

There was empty space on right in mobile view while using the aos library .I fixed it using
body {
overflow-x:hidden ; }
in the media queries for the mobile version
and it solved the problem

After using AOS I faced some issues with responsive for mobile and tab view. but I  fixed it using.
body{
overflow-x: hidden;
}

"body{
overflow-x: hidden;
}"
It may not always work. Best to add "overflow" with Root div. And "overflow-x" creates a new scroll bar sometimes in right side.
So, If possible, should use " RootDiv{ overflow: hidden;}.

'body > *{
width: 100%;
overflow-x: hidden;
}'
can be executed, but
'body{
overflow-x: hidden;
}'
cannot.

This worked for me:
html {
overflow-x: hidden;
}


Bad CPU type in executable

My system info:
Software:
    System Software Overview:
      System Version: macOS 12.5 (21G72)
      Kernel Version: Darwin 21.6.0
      Boot Volume: Macintosh HD

Hardware:
    Hardware Overview:
      Model Name: MacBook Pro
      Chip: Apple M1 Pro
      Total Number of Cores: 8 (6 performance and 2 efficiency)
      Memory: 16 GB
      System Firmware Version: 7459.141.1
      OS Loader Version: 7459.141.1

To recreate the issue:
installed through curl
curl -fsSL https://install.julialang.org | sh

reload my session
. ~/.zshrc

attempt to start julia and receive error output
julia
> Error: The Julia launcher failed to start Julia.
>
> Caused by:
>     Bad CPU type in executable (os error 86)

I havent seen any issues regarding this, was hoping to get some help.

I think you are running into https://github.com/JuliaLang/juliaup/issues/239. See the discussion there. I'm closing for now, but please reopen if this was caused by something else.

You are completely right. That solved the issue. Thank you


File is not updated even though mtime is updated

Hello,
I'm suffering an issue that file changes are not reflected from S3. The changes made the file size same.
I created a file, upload it to S3 and mount the S3 location to my laptop.
The file is quite simple.
#!/usr/bin/env python
import time

while True:
    print(time.time())
    print(',,.')
    time.sleep(1)

After that, I changed ,,. part to ,,, but goofys didn't recognize the changes. But mtime of the file on S3 and the file on mounted by goofys are the same.
I used goofys 0.19.0 in OSX with options; goofys -f --debug_fuse --debug_s3 --stat-cache-ttl 1s --type-cache-ttl 1s test-s3-bucket:yjpark/ /tmp/s3-mount-test

It works now. There may be latency issues.

Weired. It worked on and off.

It was because of buffer cache of OS.
It worked well when I cleared buffer cache.

I would reopen this as an issue.  One should not have to clear the buffer as that causes an essential down-time on the system to rebuild the inode buffers.  Is there another resolution?

Would anyone explain why OS cache clean could save the failure and what was the exact change for that? //Thanks


openapi-2-kong package generates invalid scopes_required when both global security scope and specific is defined in the same openapi spec

Expected Behavior
the expected generated source should be:
service:
 plugins:
   - name: openid-connect
     config:
       required_scopes:
       - api
routes:
 - path: /api/protected/mock
   plugins:
   - name: openic-connect
      config:
      required_scopes:
       - api
       - protected

Actual Behavior
When using Insomnia to generate deck configuration with x-kong-security-openid-connect extension the generated file does not have the required scopes needed and specified by original openapi specifications.
The output from the generation is:
service: openid-connect  plugin protected with required_scopes: - api OK
route:
/api/protected/mock (protected required_scopes: - protected ) Fail, should be protected by both api and protected scopes.

The generated source is protected by the specific scope but not by the global scope
Reproduction Steps

click the design view
copy the openapi document to insomnia
click on Insomnia dropdown menu/Declarative Config
obtain the generated kong configuration

Is there an existing issue for this?

[X] I have searched the issue tracker for this problem.

Additional Information
Use the following yaml to generate deck file (kong.yaml):
openapi: 3.0.3
x-kong-route-defaults:
  # the defaults for the Kong routes generated from 'paths' below
  # see https://docs.konghq.com/1.2.x/admin-api/#route-object
  preserve_host: true
  request_buffering: false
  response_buffering: false
  strip_path: false
x-kong-name: app-a
x-kong-service-defaults:
  # the defaults for the Kong services generated from 'servers' above
  # see https://docs.konghq.com/1.2.x/admin-api/#service-object
  retries: 5
  connect_timeout: 60000
  write_timeout: 60000
  read_timeout: 60000
  protocol: http

info:
  title: Protected API-REST
  version: 1.0.0
servers:
  - url: http://sample-service-a:8081/
    description: sample-service-a
tags:
  - name: protected
    description: Protected endpoints
  - name: public
    description: Public endpoints
paths:
  /api/protected/mock:
    get:
      tags:
        - protected
      summary: Protected demo endpoint
      description: Protected demo endpoint
      operationId: protectedMockResponse
      responses:
        "200":
          description: successful operation
        "401":
          description: unauthorized
        "403":
          description: access-denied
      security:
        - openId: # <--- Use the same name as specified in securitySchemes
            - protected
  /api/public/mock:
    get:
      tags:
        - public
      summary: Unprotected endpoint
      description: Unprotected endpoint
      operationId: insecureMockResponse
      responses:
        "200":
          description: successful operation
        "401":
          description: unauthorized
        "403":
          description: access-denied
security:
  - openId:
      - api
components:
  securitySchemes:
    openId:
      type: openIdConnect
      openIdConnectUrl: /.well-known/openid-configuration
      x-kong-security-openid-connect:
        config:
          client_id:
          - client_id

Insomnia Version
2023.1.0
What operating system are you using?
Windows
Operating System Version
Windows 10 Enterprise 21H2
Installation method
download from insomnia website and install
Last Known Working Insomnia version
No response

Actual behavior:
expected generated source should be:

service:
 plugins:
   - name: openid-connect
     config:
       required_scopes:
       - api
routes:
 - path: /api/protected/mock
   plugins:
   - name: openic-connect
      config:
      required_scopes:
       - protected

on the route it is required the api required_scopes


Potentially long count from space "space"

Hello! When I used version 0.12.1. And when I selected some space. I got error message. That isn't in previous versions.
The log
pp-replica1  | pp-admin.s1 | 2022-07-26 13:14:12.137 [12] main/117/main I> [[["<=","order",1000000],["<=","published_date",1658841252],["==","status",true],["==","company_id","00000000-0000-0000-0000-000000000000"]],{"offset":0,"limit":12}]
pp-replica1  | pp-admin.s1 | 2022-07-26 13:14:12.181 [12] main/117/main model.lua:370 E> nil
pp-replica1  | pp-admin.s1 | 2022-07-26 13:14:12.181 [12] main/117/main C> Potentially long count from space 'news'
pp-replica1  | pp-admin.s1 |  stack traceback:
pp-replica1  | pp-admin.s1 |    /opt/tarantool/.rocks/share/tarantool/crud/count.lua:106: in function 'check_count_safety'
pp-replica1  | pp-admin.s1 |    /opt/tarantool/.rocks/share/tarantool/crud/count.lua:168: in function 'method'
pp-replica1  | pp-admin.s1 |    ...ool/.rocks/share/tarantool/crud/common/sharding/init.lua:149: in function 'func'
pp-replica1  | pp-admin.s1 |    .../tarantool/.rocks/share/tarantool/crud/common/schema.lua:88: in function 'total_news'
pp-replica1  | pp-admin.s1 |    /opt/tarantool/app/roles/news/model.lua:396: in function </opt/tarantool/app/roles/news/model.lua:374>
pp-replica1  | pp-admin.s1 |    [C]: in function 'pcall'
pp-replica1  | pp-admin.s1 |    builtin/box/net_box.lua:1657: in function <builtin/box/net_box.lua:1643>
pp-replica1  | pp-admin.s1 |    [C]: in function 'local_call'
pp-replica1  | pp-admin.s1 |    ...tarantool/.rocks/share/tarantool/vshard/storage/init.lua:2526: in function <...tarantool/.rocks/share/tarantool/vshard/storage/init.lua:2520>
pp-replica1  | pp-admin.s1 |    [C]: at 0x55cc5660fad0

And a fragment of source code that used in the project
 params.limit = 12
set_data = { { '<=', 'order', 1000 },
                       { '<=', 'published_date', os.time() },
                       { '==', 'status', true },
                       { '==', 'company_id', company_id } }
raw_data_first = crud.select(model.space_name, set_data, { first = params.limit })

Thanks if you see it and will give some tips to resolve the problem!

Logging of potentially long requests was introduced in crud 0.11.3. See "Avoiding full scan" section of select README to determine how you can possibly change your request or data schema to improve your performance. We can help you if figuring it out for yourself would be too complicated, but we'll need your data schema to understand it.

Feels free to reopen if something still bothers you.


Design up to five leaves so that they can be used on the time tree

@rlskoeser what do you think about these leaf shapes? Worth trying them and see how they would look like on your layout?

@gissoo these shapes seem fine but I would like to talk a bit to understand how you are imagining they would be implemented.
You don't need to provide them in more than one color or more than one orientation — I need to color them with css, and I can rotate programmatically (although I'll need to think about the logic a bit!).
 Or do you think it's still feasible to draw these with code based on your designs?
If the latter, I would love to see these leaves lined up in the same orientation so that I can get a better sense of the variation? I can kind of see it, but would like to be more sysematic about it.

@gissoo I did some quick svg sketching in d3 to show what I mean, take a look: https://observablehq.com/d/32bcd3234e4021e1

@rlskoeser thank you! here they are oriented in the same direction, please let me know if there is anything else I can do with them in Figma https://www.figma.com/file/sGOn9MZy6ML0yNxuiQJ8u5/Lenape-Time-Tree?node-id=484%3A396&t=CJH26KrwLUoQzVxD-1

@rlskoeser here is the link to the updated leaves with some specs
you can inspect each line between the dots on each leaf to see the length of the widest point of each leaf. Let me know if you have any questions

Rebecca has revised the leaves based on my latest revision and new documentation in hand off, and we'll probably refine the leaves at a later point. So I'll go ahead and consider this issue as complete.


AQCU-1037/AQCU-1041: Fixed several small issues related to recent...

...changes to CORR reports.

Coverage increased (+0.0008%) to 99.046% when pulling 5c16fd917755902fac84bfcec37b02b4366db255 on zmoore-usgs:bug/corr-issues into a59b279e4ef28b385ee459a7420dc1a7a0bac219 on USGS-R:master.

Coverage increased (+0.0008%) to 99.046% when pulling bf87e95fd9958cd40060e966ad7b6f45bb698a13 on zmoore-usgs:bug/corr-issues into a59b279e4ef28b385ee459a7420dc1a7a0bac219 on USGS-R:master.


creating svg file

most of the work is:

creating a template for svg
recode styles.go, and changing drawio template accordingly
calculation for svg (see styles.go)

todo:
reorganize the functions relevant to geometry, and put them together.
will be done in a separate PR

did some more changes:
replace 'Connectivity' with 'Line'
minor change in the connectivity format


Question mark in method name in ERB template block causes mis-tokened in VSCode extension

Description
Reproduction steps
In VSCode, it appears that including a question mark in a method name in the ERB highlighting prevents the tag from being properly tokenized.
This does not occur for !, only ?.

Open an ERB file
Use a method with a question mark in it
Observe that tokenization is incorrect

Code snippet or error message
<% if account_signed_in? %>
  Text that should not be italic but it is if there is a question mark in the method name
<% end %>

Examples
When an ? is in the ERB block  (I included && true to demonstrate that it occurs if the method name anywhere in the line includes a question mark) it doesn't "close" the tag and includes everything until the end of the document:

When there's no ? it correctly closes the tag at the end %>:

Thank you for the bug report! This is an issue with our ERB grammar.


Mobile App

Is your feature request related to a problem? Please describe.
No
Describe the solution you'd like
I know that I can just add the webpage to my home screen on iOS and will act as "Native" but I would like to have an actual application that I can download from the app store. After download, allow users to enter the address to their Open WebUI service. I think that will give users more of a "native" feel.
Additional context
Thanks for all the hard work!! What an amazing project!

App Store approval is a right pain in the rear. We're not very interested in experiencing it right now.


Possible 2.9 release?

Hi guys!
Sorry to bother - but there are a set of commits just sitting in master (https://github.com/symfony/monolog-bundle/compare/v2.8.2...master) and I'd like to get those into a proper release (probably 2.9 to follow semver). Could one of you tag a release? Or if you want another maintainer to help with releases, let me know.
Thanks!

Friendly ping! We've got a few features dating back about 3 months that aren't on a stable release yet.

Please note that #155 should be merged before the tag so that it's possible to use #153 with XML configurations.

And now #155 is merged. I don't know of any other blockers for a tag.

Friendly (I know, probably annoying) ping!
Also, if I can help out this bundle as another maintainer, I'm open to that :)

https://github.com/symfony/monolog-bundle/releases/tag/v2.9.0 :tada:


Add UploadMetrics to Diffusion workflow?

What would you like to see added in this software?
Great to see the diffusion IQMs going mainstream.
I was just chatting with @rwblair wondering if there's any interest in adding a diffusion collection to the database at mriqc.nimh.nih.gov so mriqc can upload diffusion IQMs as well as T1w and bold. Should be fairly straightforward to do.
Do you have any interest in helping implement the feature?
Yes
Additional information / screenshots
No response

Thanks! We would need to update the schema - Unfortunately, I didn't see this issue before the hackathon, but let's make up for that oversight. @rwblair - do you have some bandwidth for that?


Idea Collection: collaboration between geopandas and PySAL/splot

Space to collect thoughts and ideas for a collaboration between geopandas and PySAl/ splot
The purpose of this issue is to start a discussion between the geopandas and PySAL/ splot community on how to best collaborate in the field of visualising geographic data and spatial analyses.
Open questions:

What is the scope and division of both projects? (Especially regarding map visualisation functionality?)
With both packages supporting each other, will releases be coordinated?
Choropleth visualisations located in geopandas or splot?
How sophisticated are geopandas .plot() methods and splot visualisations? Where to draw the line?
Which other visualisations are splot and geopandas planning on supporting in future? (points, polygons, lines, interactivity...)

Current connections:

Geopandas dataframes as required input parameter for certain visualisations
splot using geopandas .plot functionality for maps
geopandas choropleth visualisation building on esda.mapclassify

Ideas for new mapping features:

North Arrows
Scale bar
grid lines
extended legend functionality

Answered & Decisions:
Potential upcoming meeting:

Sprints/ or Weekday at SciPy

@jorisvandenbossche

@TaylorOshan

Hi,
I know this is rather old but I don't think it was ever discussed on geopandas side (definitely not on GitHub). Was there some progress in the meantime? I think all these questions are still relevant and it would be nice to have an agreement/cooperation.

What is the scope and division of both projects? (Especially regarding map visualisation functionality?)

From my perspective, GeoPandas should support basic plotting covering standard use-cases. It sort of does that at the moment. Anything special (like vba, cartograms...) should be left to other packages. GeoPandas main focus is and will be on spatial data handling, not visualisation.

Which other visualisations are splot and geopandas planning on supporting in future?

We have been doing some minor enhancements and bugfixes recently and we are adding a support for missing values in the next release (https://github.com/geopandas/geopandas/pull/1156). We should give a bit more options to legend (https://github.com/geopandas/geopandas/issues/735), but I would not go much further than that. We do not plan to include any interactivity or similar stuff as it is covered by other packages already.
So things like north arrows, scale bars, grid lines etc., I would personally be happy to leave to splot.

Choropleth visualisations located in geopandas or splot?

As it is one the most basic visualisations, I think it should remain part of goepandas. Not necessarily in advanced versions like vba, but in the same version as it is now certainly.

How sophisticated are geopandas .plot() methods

Not sure how to answer this 😄.
cc @ResidentMario as this discussion should be related to geoplot as well.

Hi @martinfleis, this post was part of my GSOC project and discussed in person at Scipy18 with @jorisvandenbossche. All of your thoughts basically reflect our discussions. Ultimately, splot with it's connection to PySAL is more thought of to be a Seaborne for geospatial statistical analysis.

From my perspective, GeoPandas should support basic plotting covering standard use-cases.

One thought here, as north arrows and scale bars are actually connected to the underlying geographic information and quite standardised for all sorts of geographic visualisation. Maybe that could be something to include in geopandas in future?

One thought here, as north arrows and scale bars are actually connected to the underlying geographic information and quite standardised for all sorts of geographic visualisation. Maybe that could be something to include in geopandas in future?

It is just a matter of preference, but I would be happy with the similar solution we use for background tiles (contextily).
import contextily as ctx
ax = df.plot(figsize=(10, 10), alpha=0.5, edgecolor='k')
ctx.add_basemap(ax)

We could have something similar for arrows and scale bars (maybe within splot?):
import splot
ax = df.plot(figsize=(10, 10), alpha=0.5, edgecolor='k')
splot.add_arrow(ax)
splot.add_scalebar(ax)

But yeah, this can be part of GeoPandas if we agree that it is a better fit.

to be a Seaborne for geospatial statistical analysis

I feel that @ResidentMario is trying to do the same with geoplot.

personally, i think of geoplot more along the lines of proplot, in that it provides a nicer-than-default API, sensible aesthetics out of the box, tooling for better layouts, etc.
whereas splot is more like seaborn in that it implements additional types of statistical plots (e.g. seaborn provides clustermaps and violin plots and splot provides maps of weights matrices and the Moran scatterplot, etc).
that's not a perfectly clean distinction, since geoplot also does things like quadtrees and sankey diagrams, but my own concept is that splot and seaborn are oriented more towards stats and analysis

Thanks for the comments @martinfleis and @knaaptime. It is extremely interesting to me to hear your impressions of how geoplot fits within the broader Python geospatial data science stack.
Let me state my own viewpoint.
geoplot as originally envisioned was absolutely meant to be a seaborn equivalent for geospatial plotting; this is even the tagline the library uses, "like seaborn for geospatial". Some of this emphasis was lost in the ensuing development history, as most of the work I have done past the initial conception stage has been in refactoring APIs, improving error messages, writing and rewriting documentation, and in general doing the work necessary to take the library from my lab project to a powerful, punchy toolkit that's as well-organized and approachable as possible.
In other words, most of my effort has gone into the developer experience of the library, moreso then its power. As a result, geoplot doesn't actually do that much—you can get away with just using geopandas plotting most of the time.
PySAL is actually a great example of a library with the opposite focus by the way: many powerful tools, but everything developed piecemeal overtime, with patchwork coverage of what's possible and what's not, what's easy and what's hard, what's well documented and (well, most things in PySAL aren't well-documented, something I know the team is aware of and working on).
I am now starting to get close to the point where I am happy with the code maturity and ergonomic quality of geoplot (though there's some work outstanding that still needs to be done). The next big steps are going to be investing in more and more powerful plot types. And as the gamut of what geoplot makes possible expands, the incremental usefulness of having this tool in your toolkit will go up greatly, and the library overall will start to seem more useful: a true power-belt, and not just a bunch of auxiliary visual options (as @knaaptime posits). More seaborn-like!
Some of the tools in splot (e.g. Moran plots) feel in-scope to me, e.g. Moran plots. But I also I suspect that some of the tools in splot will be too specialized for geoplot in scope. I would make the following comparison between the Python geospatial plotting stack and the standard Python data science plotting ecosystem, in order of increasing visual complexity:

cartopy is like matplotlib --- low-level plotting tools with full customizability
geopandas is like pandas --- your one-stop shop for quick one-plots, e.g. basic polygonal maps
geoplot is like seaborn --- an accessible expanded visual palette, e.g. kernel density maps
splot is like yellowbrick --- specialized visual power tools, e.g. local autocorrelation plots

I haven't yet seen evidence of how this stack of tools maps into workflows "in the wild".  The best geospatial learning materials I am aware of is the excellent Automating-GIS-process course notes, and these never bother straying above simple geopandas plots. It feels like there's still a lot of work to do here.
I hope that makes sense. Let me know whether you agree or disagree with this assessment.

I didn't mean to be reductive about geoplot, so apologies if it came off that way!
To put it a bit differently, the distinction i see is about the substantive focus of the plotting. seaborn provides a wonderful set of aesthetics and plotting utilities, that are useful in a variety of contexts, but it's intended explicitly for statistical data viz. Every one of the plotting examples in its tutorial is designed to visualize variance, or relationships between two variables, etc.
geoplot is similar in that it provides a wonderful set of utilities and nice aesthetics, but unlike seaborn it is not necessarily concerned with stats. geoplots gallery is full of great examples for plotting single variables--in the specific context of geographic data. In particular, it provides really useful transformations that are specifically useful in the geographic case (e.g. cartograms, quadtrees, KDEs).
splot, meanwhile, shares seaborns statistical emphasis, tailored to the spatial contect. Its focus is visualizing statistical relationships in space.
Yellowbrick seems less appropriate since they specialize in vizualizing ML models, whereas splot isn't really concerned specifically with modeling or ML. pysal's DNA is really more in inferential stats and econometrics than ML, so the analogy I draw is
seaborn : statsmodels :: splot : pysal
like waksom says in the seaborn tutorial,

In the spirit of Tukey, the regression plots in seaborn are primarily intended to add a visual guide that helps to emphasize patterns in a dataset during exploratory data analyses. That is to say that seaborn is not itself a package for statistical analysis. To obtain quantitative measures related to the fit of regression models, you should use statsmodels. The goal of seaborn, however, is to make exploring a dataset through visualization quick and easy, as doing so is just as (if not more) important than exploring a dataset through tables of statistics.

things like the Moran scatterplots in splot don't generate a map or geospatial visualization, but they do give you your bearings before diving into a more rigorous (spatial) statistical analysis. That's the kind of workflow we'd use in something like one of the pysal workshops or my colleagues' forthcoming book

I have only minor quibbles with your assessment, and I like your analogy as well.
Net-net, I'm comfortable with the level of overlap between these three tools (geopandas plotting, geoplot, and splot). I don't think there's much reason for consolidation.


C++ package manager request tracking issue

For the moment we decided to not support additional build systems and package management for C++, since there's simply too many to choose from and this becomes a bit of a maintenance burden that we're not ready for yet.
This ticket collects the various requests that came up so far and what state they're in:

Bazel

There's a minimal example integration here https://github.com/kyle-figure/bazel-minimal-rerun/

CocoaPods #4568
Conan #4420
Conda #4130
Gradle #4336

@Wumpf I've started work on conan recipe (#4420 ) in our local conan system and faced a few things. Maybe, we could discuss it here or in discord or have a call. Here are the topics:
C++ SDK provided in assets has good and clear process of integration with just couple lines of code in CMake, but it seems to be very important that own code is compiled along with some Rerun sources (*.c - files). However, the classical package should contain only pre-built libraries (lib or dll) and header files. I tried to create conan package and it contains only hpp files with these  libs:

arrow_bundled_dependencies.lib
arrow_static.lib
rerun_c__win_x64.lib  (from official SDK asset)
rerun_sdk.lib + rerun_sdk.pdb

Next, when I try to link rerun conan package to existing project XXXX I'm getting these linker errors - apparently libs don't contain all code:
[build] rerun_c__win_x64.lib(re_sdk_comms-0765e134c12468ee.re_sdk_comms.f78a4b73346f03c4-cgu.05.rcgu.o) : error LNK2001: unresolved external symbol __imp_closesocket [XXXXXXXXXX.vcxproj]
[build] rerun_c__win_x64.lib(re_sdk-9a5f3c7fa3d85f9c.re_sdk.a3283a595e0d2873-cgu.00.rcgu.o) : error LNK2001: unresolved external symbol __imp_closesocket [XXXXXXXXXX.vcxproj]
[build] rerun_c__win_x64.lib(std-4ee9ee8805e6ac55.std.ddad90bab7781587-cgu.0.rcgu.o) : error LNK2001: unresolved external symbol __imp_closesocket [XXXXXXXXXX.vcxproj]
[build] rerun_c__win_x64.lib(re_sdk-9a5f3c7fa3d85f9c.re_sdk.a3283a595e0d2873-cgu.12.rcgu.o) : error LNK2001: unresolved external symbol __imp_closesocket [XXXXXXXXXX.vcxproj]
[build] rerun_c__win_x64.lib(re_sdk_comms-0765e134c12468ee.re_sdk_comms.f78a4b73346f03c4-cgu.01.rcgu.o) : error LNK2001: unresolved external symbol __imp_closesocket [XXXXXXXXXX.vcxproj]
[build] rerun_c__win_x64.lib(std-4ee9ee8805e6ac55.std.ddad90bab7781587-cgu.0.rcgu.o) : error LNK2001: unresolved external symbol __imp_WSAStartup [XXXXXXXXXX.vcxproj]
[build] rerun_c__win_x64.lib(std-4ee9ee8805e6ac55.std.ddad90bab7781587-cgu.0.rcgu.o) : error LNK2001: unresolved external symbol __imp_WSACleanup [XXXXXXXXXX.vcxproj]
[build] rerun_c__win_x64.lib(std-4ee9ee8805e6ac55.std.ddad90bab7781587-cgu.0.rcgu.o) : error LNK2001: unresolved external symbol __imp_freeaddrinfo [XXXXXXXXXX.vcxproj]
[build] rerun_c__win_x64.lib(std-4ee9ee8805e6ac55.std.ddad90bab7781587-cgu.0.rcgu.o) : error LNK2001: unresolved external symbol __imp_shutdown [XXXXXXXXXX.vcxproj]
[build] rerun_c__win_x64.lib(std-4ee9ee8805e6ac55.std.ddad90bab7781587-cgu.0.rcgu.o) : error LNK2001: unresolved external symbol __imp_WSAGetLastError [XXXXXXXXXX.vcxproj]
[build] rerun_c__win_x64.lib(std-4ee9ee8805e6ac55.std.ddad90bab7781587-cgu.0.rcgu.o) : error LNK2001: unresolved external symbol __imp_setsockopt [XXXXXXXXXX.vcxproj]
[build] rerun_c__win_x64.lib(std-4ee9ee8805e6ac55.std.ddad90bab7781587-cgu.0.rcgu.o) : error LNK2001: unresolved external symbol __imp_getsockopt [XXXXXXXXXX.vcxproj]
[build] rerun_c__win_x64.lib(std-4ee9ee8805e6ac55.std.ddad90bab7781587-cgu.0.rcgu.o) : error LNK2001: unresolved external symbol __imp_recv [XXXXXXXXXX.vcxproj]
[build] rerun_c__win_x64.lib(std-4ee9ee8805e6ac55.std.ddad90bab7781587-cgu.0.rcgu.o) : error LNK2001: unresolved external symbol __imp_WSARecv [XXXXXXXXXX.vcxproj]
[build] rerun_c__win_x64.lib(std-4ee9ee8805e6ac55.std.ddad90bab7781587-cgu.0.rcgu.o) : error LNK2001: unresolved external symbol __imp_WSASend [XXXXXXXXXX.vcxproj]
[build] rerun_c__win_x64.lib(std-4ee9ee8805e6ac55.std.ddad90bab7781587-cgu.0.rcgu.o) : error LNK2001: unresolved external symbol __imp_send [XXXXXXXXXX.vcxproj]
[build] rerun_c__win_x64.lib(std-4ee9ee8805e6ac55.std.ddad90bab7781587-cgu.0.rcgu.o) : error LNK2001: unresolved external symbol __imp_accept [XXXXXXXXXX.vcxproj]
[build] rerun_c__win_x64.lib(std-4ee9ee8805e6ac55.std.ddad90bab7781587-cgu.0.rcgu.o) : error LNK2001: unresolved external symbol __imp_ioctlsocket [XXXXXXXXXX.vcxproj]
[build] rerun_c__win_x64.lib(std-4ee9ee8805e6ac55.std.ddad90bab7781587-cgu.0.rcgu.o) : error LNK2001: unresolved external symbol __imp_getpeername [XXXXXXXXXX.vcxproj]
[build] rerun_c__win_x64.lib(std-4ee9ee8805e6ac55.std.ddad90bab7781587-cgu.0.rcgu.o) : error LNK2001: unresolved external symbol __imp_WSADuplicateSocketW [XXXXXXXXXX.vcxproj]
[build] rerun_c__win_x64.lib(std-4ee9ee8805e6ac55.std.ddad90bab7781587-cgu.0.rcgu.o) : error LNK2001: unresolved external symbol __imp_WSASocketW [XXXXXXXXXX.vcxproj]
[build] rerun_c__win_x64.lib(std-4ee9ee8805e6ac55.std.ddad90bab7781587-cgu.0.rcgu.o) : error LNK2001: unresolved external symbol __imp_getaddrinfo [XXXXXXXXXX.vcxproj]
[build] rerun_c__win_x64.lib(std-4ee9ee8805e6ac55.std.ddad90bab7781587-cgu.0.rcgu.o) : error LNK2001: unresolved external symbol __imp_NtCreateFile [XXXXXXXXXX.vcxproj]
[build] rerun_c__win_x64.lib(std-4ee9ee8805e6ac55.std.ddad90bab7781587-cgu.0.rcgu.o) : error LNK2001: unresolved external symbol __imp_RtlNtStatusToDosError [XXXXXXXXXX.vcxproj]
[build] rerun_c__win_x64.lib(std-4ee9ee8805e6ac55.std.ddad90bab7781587-cgu.0.rcgu.o) : error LNK2001: unresolved external symbol __imp_NtReadFile [XXXXXXXXXX.vcxproj]
[build] rerun_c__win_x64.lib(std-4ee9ee8805e6ac55.std.ddad90bab7781587-cgu.0.rcgu.o) : error LNK2001: unresolved external symbol __imp_NtWriteFile [XXXXXXXXXX.vcxproj]
[build] rerun_c__win_x64.lib(std-4ee9ee8805e6ac55.std.ddad90bab7781587-cgu.0.rcgu.o) : error LNK2001: unresolved external symbol __imp_connect [XXXXXXXXXX.vcxproj]
[build] rerun_c__win_x64.lib(std-4ee9ee8805e6ac55.std.ddad90bab7781587-cgu.0.rcgu.o) : error LNK2001: unresolved external symbol __imp_select [XXXXXXXXXX.vcxproj]
[build] rerun_c__win_x64.lib(std-4ee9ee8805e6ac55.std.ddad90bab7781587-cgu.0.rcgu.o) : error LNK2001: unresolved external symbol __imp_recvfrom [XXXXXXXXXX.vcxproj]
[build] rerun_c__win_x64.lib(std-4ee9ee8805e6ac55.std.ddad90bab7781587-cgu.0.rcgu.o) : error LNK2001: unresolved external symbol __imp_BCryptGenRandom [XXXXXXXXXX.vcxproj]
[build] rerun_c__win_x64.lib(getrandom-b857062dfe0c4b33.getrandom.2ae02711c9719d53-cgu.0.rcgu.o) : error LNK2001: unresolved external symbol __imp_BCryptGenRandom [XXXXXXXXXX.vcxproj]
[build] rerun_c__win_x64.lib(std-4ee9ee8805e6ac55.std.ddad90bab7781587-cgu.0.rcgu.o) : error LNK2001: unresolved external symbol __imp_bind [XXXXXXXXXX.vcxproj]
[build] rerun_c__win_x64.lib(std-4ee9ee8805e6ac55.std.ddad90bab7781587-cgu.0.rcgu.o) : error LNK2001: unresolved external symbol __imp_listen [XXXXXXXXXX.vcxproj]
[build] rerun_c__win_x64.lib(std-4ee9ee8805e6ac55.std.ddad90bab7781587-cgu.0.rcgu.o) : error LNK2001: unresolved external symbol __imp_getsockname [XXXXXXXXXX.vcxproj]
[build] rerun_c__win_x64.lib(std-4ee9ee8805e6ac55.std.ddad90bab7781587-cgu.0.rcgu.o) : error LNK2001: unresolved external symbol __imp_sendto [XXXXXXXXXX.vcxproj]
[build] XXXXX.exe : fatal error LNK1120: 30 unresolved externals [XXXXXXXXXX.vcxproj]

I would like to understand better the structure of C++ code, how to make it compilable into self-sufficient library and how did you get rerun_c__win_x64.lib - what does it contain in practice.
I'm thinking of maybe forcing linker to produce all symbols by adding following options (not sure it will resolve all linker problems though):
MSVC: /WHOLEARCHIVE option in the linker.
GCC use the -Wl,--whole-archive linker option.
What do you think?

@rgolovanov the missing symbols you hit there are from a bunch of very common system dependencies. These to be accurate ws2_32.dll Bcrypt.dll Userenv.dll ntdll.dll
Afaik whole-archive linking is generally discouraged since it can lead to a lot of bloat (everyone else now using anything from the windows system comes with their own winsock & ntdll etc.). I'm not familiar with Conan, but I'd assume that these system libraries would be explicitly specified as a dependency whenever linking with the static version of Rerun.

@Wumpf thanks for comment. Ok, I'll check why there are problems with system libraries.
Separately, I would like to double check the SDK content. The SDK consists of rerun_c__win_x64.lib (and similar libs for other OS) but when I integrate it following "official instruction" it is compiled from the scratch and produce rerun_sdk.lib + arrow's libs arrow_static.lib and arrow_bundled_dependencies.lib.
What is the purpose of rerun_c__win_x64.lib? Is it a bundle for all other libs: rerun_sdk.lib, arrow_static.lib, arrow_bundled_dependencies.lib. Current SDK content is a bit confusing 🙈  From one hand it has prebuilt static library with headers but from the other hand it should be integrated and compiled from the scratch along with user's code.

@Wumpf thanks for comment. Yes, now I see it should come from here  but apparently was cutoff during conan package creation. I'll check what can be done 👍

For what regards C++ SDK support for conda-forge, this is now solved (at least on Linux/macOS) with the librerun-sdk package, see https://github.com/conda-forge/staged-recipes/pull/25648 and https://github.com/conda-forge/librerun-sdk-feedstock .

@traversaro awesome! Would you mind creating a PR that adds this to the docs somewhere in here https://github.com/rerun-io/rerun/tree/main/rerun_cpp#build--distribution ?

@traversaro awesome! Would you mind creating a PR that adds this to the docs somewhere in here https://github.com/rerun-io/rerun/tree/main/rerun_cpp#build--distribution ?

Sure, there are a bit of aspects that I would to tackle before (support for it in the opencv/eigen example, Windows support, general dogfooding it to understand if it works fine), and then I think we can can add it there, does it make sense?

Of course, whenever you feel comfortable with it :)

Of course, whenever you feel comfortable with it :)

Now that we have the Windows package as well, I think it is fine to add it: https://github.com/rerun-io/rerun/pull/6381 .


Add hint how to run examples to README.md

It took me while to find out the following steps:

npm install
npm run-script start
Open http://localhost:8080

These instructions are actually already in the CONTRIBUTING.md file. But I think this is not the best place for new users.

Good idea! How does this look?
https://github.com/vigetlabs/microcosm#examples

Looks great.
One small thing: the first sentence misses a "be", I guess.
Examples can be found in...

@flof Oh dear... Looks like I got ahead of myself!


Implementing NFC Tag Detection and Removal Notification

Need To Be Solved
Our project necessitates the capability to scan NFC tags. It is essential to not only detect the tag's ID upon its recognition but also to identify when it has been removed.
Solution Suggestion
After exploring available options, we propose integrating the NFC PN532, RFID V3 PN532. Below is the suggested YAML configuration for integration:
pn532_i2c:
update_interval: 1s
id: noconflict_nfc
i2c_id: bus_a
on_tag:
then:
- mqtt.publish:
topic: devicename/pn532_i2c/nfc/state
payload: !lambda 'return x;'
on_tag_removed:
then:
- mqtt.publish:
topic: devicename/pn532_i2c/nfc/state
payload: !lambda 'return "not:"+x;'
Additional resources
For further details regarding the PN532 NFC module, including its RFID V3 variant, please visit:
https://eu.robotshop.com/es/products/elecrow-pn532-nfc-module-pn532-rfid-v3-module

Closed on #efa965e


Fix Android input source checks

Input source types are not pure bit flags, they are combinations of
flags, so != 0 check was incorrect and resulted in crashes later, when
trying to obtain the device.

Cherry picked into 3.0.3


Update underscore.cpp

#35

Hi, before starting any work, I would suggest asking either @Gotham13121997 or me to assign the task to you. This is to ensure that two people are not working on the same task. Unfortunately, this issue was already taken.


Feature request: expand macro

It would be neat if the server supported an "expand macro" message, which took as input the location of a macro invocation (call), and returned the macro expansion for that call.
Clients could then offer a feature to show users the expansion in a hover or something.

@MaskRay why did you label this with wontfix? This would be a great feature (which btw. eclipse also supports), and if there's an existing tool that does the actual job, why not incorporate this?
OK, perhaps this is not the job for cquery, but for the editor integration (e.g. vscode-cquery). But as far as I see most of the discussions/issues about these integrations are done here...

I think the combination of "wontfix" and "libclang" is being used to mean "libclang doesn't support the necessary operations, so we can't fix it in cquery until libclang adds support for them". Several other bugs are annotated this way.

I think macro expansion/indexing will be useful, this issue and https://github.com/cquery-project/cquery/issues/331
I found clang_findReferencesInFile(cursor.cx_cursor, file, visitor); and tried
enum CXVisitorResult VisitMacroExpansion(void* param_, CXCursor cx_cursor, CXSourceRange cx_range) {
  IndexParam* param = reinterpret_cast<IndexParam*>(param_);
  IndexFile* db = param->primary_file;

  ClangCursor cursor(cx_cursor);

  CXSourceLocation start = clang_getRangeStart(cx_range);
  unsigned int line, column;
  clang_getExpansionLocation(start, 0, &line, &column, nullptr);
  LOG_S(INFO) << "+++expand " << line << ' ' << column;
  clang_getFileLocation(start, 0, &line, &column, nullptr);
  LOG_S(INFO) << "+++file " << line << ' ' << column;

  //Range range = ResolveCXSourceRange(cx_range);
  (void)db;
  return CXVisit_Continue;
}

VisitMacroExpansion will be called twice, with  CXCursor_MacroDefinition and
CXCursor_MacroExpansion as the cursor kind, respectively. But there is no information related to location in the replacement-list or macro expansion tokens.


Bind Early/Fail Early for unsupported node

I tend to agree with @bnoordhuis that this would be a better approach. fsevents will likely always be loaded if needed only, since it's a Mac specific thing. In that case it would be much better to fail early and make this catchable.
The cost has to be born anyhow, at least this way the exact point of the exception is predetermined.


[Suggestion] Auto pre-heat step

I'd be handy to have one baking step bake for ~{30%minutes@350F} and have that automatically result in the first step of the recipe being 1. Preheat oven to 350F Probably the sort of thing that is much more tricky than it sounds :)

How would we know that a preheat is required? Is it only for "bake for" keywords? Would there be other instances where pre-heat would be needed?


DevCenter 1.0.0b2

python released: https://pypi.org/project/azure-developer-devcenter/1.0.0b2/

https://www.nuget.org/packages/Azure.Developer.DevCenter/1.0.0-beta.2
https://www.npmjs.com/package/@azure-rest/developer-devcenter
https://mvnrepository.com/artifact/com.azure/azure-developer-devcenter


Can I use immer to maintain immutability in a web worker?

🙋♂ Question
We're using immer for our redux store. We send the store data for some evaluations to a web worker. I was wondering if immer proxies maintain reference when serialized during postMessage? I can't find much in the google searches regarding this.
So if there hasn't been any changes in the object in the webworker,  is the following true?
Obj1 (UI thread) -> webworker -> (Obj2) UI thread === Obj1

There is no shared memory afaik between webworkers and main, so no.
On Mon, 3 May 2021, 12:03 Abhinav Jha, @.***> wrote:

🙋♂ Question
We're using immer for our redux store. We send the store data for some
evaluations to a web worker. I was wondering if immer proxies maintain
reference when serialized during postMessage? I can't find much in the
google searches regarding this.
So if there hasn't been any changes in the object in the webworker, is the
following true?
Obj1 (UI thread) -> webworker -> (Obj2) UI thread === Obj1
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/immerjs/immer/issues/790, or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAN4NBDNNZSL7QAHTPXTCLLTLZ7H7ANCNFSM44ATUNUQ
.


Update Linking-source-code-to-risks.md

If you want to send me the coffee-script sample I am happy to look at it & see how to include it here.


Update compose version

Proposed changes
Updated the version of compose because it was released.

@rumpl Can you please update Compose release notes? Thanks.
https://docs.docker.com/release-notes/docker-compose/
https://github.com/docker/docker.github.io/blob/master/release-notes/docker-compose.md

Deploy preview for docsdocker ready!
Built with commit 868e484860ebef188d2acb53c25c8c584403c199
https://deploy-preview-8978--docsdocker.netlify.com


HAL_I2C_Master_Transmit timeout parameter behaves unexpectedly

Describe the set-up

The board (either ST RPN reference or your custom board)

STM32L4P5AG-DK

IDE or at least the compiler and its version

STM32CubeIDE Version: 1.4.0
Build: 7511_20200720_0928 (UTC)

Describe the bug
The timeout parameter begins counting from the instant the transaction begins, and it doesn't appear that the watchdog is being reset with each incoming byte. Therefore, when you have a long I2C transaction, it will timeout, even if it is working perfectly.
Here is an example with a 250ms timeout:
Invocation:
HAL_I2C_Master_Transmit(&hi2c4, (dev_addr << 1), query_request, strlen(query_request), 250);

How To Reproduce

Select a slow I2C speed (i.e. 100kHz).
Select a easy timeout value that is easy to test (e.g. 100ms).
Send a long I2C transaction that is guaranteed to surpass the given timeout.
The transaction will stop sending midstream, within a few milliseconds of the timeout expiration.
I assume this is isolated to the I2C Timeout watchdog

Expected Behavior
I would expect the timeout to be related to the amount of time the slave device holds the line low, or time between incoming bytes, but certainly not the length of an in progress I2C transaction.

Hi @zfields,
Thank you for this report. It has been forwarded to our development teams. We will get back to you as soon as they provide us with their feedback.
With regards,

Hi @zfields,
Sorry for the delayed answer.
Referring to our development team, the timeout value should be tuned by the user and referring the use case of the application so that it can run without error.  Actually, we de not have a timer counter enabled during the low period of the line, this mean that we cannot associated a notion of timeout with a latency of the hardware or distant device.
The development team, recommends increasing the timeout value so that the application could runs without issues.
Hope this would help you.
With regards,

No, this does not help. In fact, if this is true, then this implementation does not match the implementation of any other I2C bus that I'm aware of.
A timeout (or watchdog) does NOT typically fire, when a device is operating under "normal" or successful conditions. In this case the I2C driver is actively sending information, and it should NOT "timeout". There must be some misunderstanding of my request.

"timeout value should be tuned by the user and referring the use case of the application so that it can run without error."

Given the way your timeout functions, a static system would be required, so you knew the length of every single transaction. This would leave HAL_MAX_DELAY as the only option to handle data arbitrary length.
Please request for the "development team" to take another look a the following section specifically:
In the implementation of HAL_I2C_Master_Transmit(), you make repeated calls to I2C_WaitOnFlagUntilTimeout() and I2C_WaitOnTXISFlagUntilTimeout() without adjusting tickstart with each new call to this function.
The bug likely stems from the fact tickstart is set outside the transmission while loop, and is not updated with each successful byte transfer.
stm32l4xx_hal_i2c.c
1078 /* Init tickstart for timeout management*/
1079 tickstart = HAL_GetTick();
     ...
1108 while (hi2c->XferCount > 0U)
1109 {
         ... (never updated here)
1143 }

Hi @zfields,
The HAL_I2C_Master_Transmit() function is designed so that the timeout value corresponds to the duration set by the user to ensure that the system don't hang during the transfer. The timeout value does not correspond to one character but for the full communication and it should be computed by the user depending on the application and on the amount of data to transfer. This value corresponds the duration estimated by the user after which transfer should be carried out.
Hence, it is up to the application to compute the maximum timeout possible for the number of characters transferred during the communication.
Since this is the normal behavior and that HAL_I2C_Master_Transmit() works as designed for, please allow me thus to close this thread.
With regards,

Does this mean STMicroelectronic's chips do NOT support variable length I2C transmissions, by design?
This would effectively eliminate "full support" of several ICs, by disallowing any reasonable usage of the timeout parameter.
This is disappointing news.

Does this mean STMicroelectronic's chips do NOT support variable length I2C transmissions, by design?
This would effectively eliminate "full I2C support" of several ICs, by disallowing any reasonable usage of the timeout parameter for variable length transmissions.
This is very disappointing news.