Update finance-insights-home-page.md

Privacy/legal wants the privacy notice added ASAP.

:x: Organization membership check failed
Members of the MicrosoftDocs organization should use the private repo. Please move your changes to https://github.com/MicrosoftDocs/Dynamics-365-Operations, and close this pull request.


Differentiate between opening a new app's instance vs switching to existing?

[x] I am on the latest Cerebro.app version
[x] I have searched the issues of this repo and believe that this is not a duplicate

OS version and name: Elementary OS 0.4, Loki (Ubuntu 16.04)
Cerebro.app version: 0.2.7_amd64.deb
Relevant information from devtools (See above how to open it): N/A

Issue

Is it possible to have Cerebro present the option to switch to a currently opened instance of an app rather than opening a new one? I have multiple copies of an app running but only sometimes want to spawn a new, blank app. (eg: switch to an existing instance vs opening a 'new' LibreOffice Writer). I can have upwards of two dozen+ 'windows' open at any given time & find it time consuming to navigate between them/time consuming to the numerous keystokes required via [alt] + [tab].
If such a feature showing this concept could be differentiated between the two actions, this would be very much appreciated.

#138

Closing since it is duplicate. @amsheppard you can add your thoughts about @maximbaz's solution in #138


Prevent Autocommand Pileup

Putting an autocmd DownloadPost in the rc will result in that autocommand being executed extra times for every window that is opened. This doesn't happen if I add the autocommand manually. Is there some way to have something in the rc executed only once? If not, it would be nice if either this was added or something along the lines of vim's augroups/autocmd! for clearing. I have tried au Enter * source /path/to/autocommands (which will still execute for every opened window; if this is the intended behaviour and I'm not using it wrong, I think the documentation could be made more clear).

For the most part, Pentadactyl configuration is done on a per-window basis by sourcing .pentadactylrc each time a new window is launched. For web-page specific autocommands, the current behavior makes sense since you would would a PageLoad autocommand defined within each window. However, for autocommand conditions based on browser-wide events like DownloadPost, you would only want one autocommand per session. I think this requires a patch to somehow define session-wide autocommands (and I think whoever created the DownloadPost and other browser-wide autocommand event triggers must always work within a single window).
I'm not offering a solution here -- just framing the problem because it took me a little while to see what the issue was with the way things currently work.

Yes I agree. It's worth looking at this on the pentadactyl google group for a workaround.
I've also noticed that after opening a new window, my DownloadPost autocmd will show up twice with :js group.autocmd.get(), but Enter autocmds I have still only show up once.


[JOSS Review] Community guidelines

Instructions for opening an issue or getting help are present, but there's nothing for how to contribute. I would like to see a code of conduct and contributing guide. Here's a template from the Turing Way, and some examples: matplotlib, nipype
https://github.com/openjournals/joss-reviews/issues/5200

It's not necessary, but the discussion in #35 reminded me: it might be useful to define a bug report issue template that will prompt users for a  lot of the basic information: what fRAT version, what FSL version, what python version, attach a screenshot or pasted version of the terminal output ,etc.

I think that's a good start, but I think a code of conduct (you can lightly adapt this one) is necessary.
And, while not necessary, the following would be nice to add to the contributing section: what kind of contributions you're interested and uninterested in (more documentation, examples, new features, integration with other tools, etc.), any info about coding style or API, and required documentation and/or tests. Basically, what should they include in the PR? E.g., a description of what it includes, a small working example, the need it fills, etc.

Some guidance on codes of conduct and contributor guidelines from Mozilla's open leadership program.

I've added a code of conduct, contributing guidelines and issue templates, and linked to them in the documentation.

This looks good!


Authentication always fails with unexpected error

Hi,
I installed PGina on two Windows 7 laptops. It is working correctly on one laptop but never succeeds to authenticate an LDAP user on the other laptop.
The logon screen displays that the Pgina Status is "Connected". After submitting a valid login/password, I get the following error :
   Unexpected error while syncing user's info: System.Runtime.InteropServices.COMException (0x80070842): The Server service is not started.

   at System.DirectoryServices.DirectoryEntry.RefreshCache()
   at System.DirectoryServices.AccountManagement.PrincipalContext.DoMachineInit()
   at System.DirectoryServices.AccountManagement.PrincipalContext.Initialize()
   at System.DirectoryServices.AccountManagement.PrincipalContext.get_QueryCtx()
   at System.DirectoryServices.AccountManagement.PrincipalSearcher.SetDefaultPageSizeForContext()
   at pGina.Plugin.LocalMachine.LocalAccount.GetUserPrincipal(String username)
   at pGina.Plugin.LocalMachine.LocalAccount.CreateOrGetUserPrincipal(UserInformation userInfo)
   at pGina.Plugin.LocalMachine.LocalAccount.SyncToLocalUser()
   at pGina.Plugin.LocalMachine.PluginImpl.AuthenticatedUserGateway(SessionProperties properties)

Here is a snapshot of this error using the simulator :

The two laptops have not been installed by the same team. So I suspect an issue in the Windows configuration. The problem occurs on the laptop which should have a more secured configuration, but I did not succeed to identify what could cause this issue.
Thanks for your support.

Hi carguel, I am also experiencing this issue - did you have any luck resolving it?

Unfortunately, no !
I am still waiting for a reply from the maintainer.

@aliask, I finally succeeded to resolve this issue.
Actually, it works with a pgina fork (see MutonUfoAI/pgina#20) but i guess it should work with the upstream pgina.
The solution was simply to start the Windows server service from the Service configuraton panel.
Hope it helps.


what is the meaning of player_direction

Hello！
I read the source code for gfootball and there was one line of code I didn't quite understand below：
info.player_direction =
(movement / GetGameConfig().physics_steps_per_frame).coords;
And I printed it，got like --
x:-0.553865,y:-0.007866,z:0.000000
x:-0.000000,y:-0.000000,z:0.000000
So what I want to ask is the relationship between this data and Euler angles or Quaternion,and
how to switch from one to the other？
Thank you very much！

Direction is represented in a form of movement vector, not angles / quaternions. When game engine does rendering and has to perform rotations etc. it operates on a different representations. Classes have helper methods to convert between them.

helper

I am sorry,I just found:
info.player_direction =
(movement / GetGameConfig().physics_steps_per_frame).coords;
Could you show me the  "helper methods"?Thank you!

What do you mean by helper methods? Why are you looking at internal state
instead of just using public observations documented here:
https://github.com/google-research/football/blob/master/gfootball/doc/observation.md
On Tue, Apr 21, 2020 at 4:54 PM yumianhuli2<EMAIL_ADDRESS>wrote:

helper
I am sorry,I just found:
info.player_direction =
(movement / GetGameConfig().physics_steps_per_frame).coords;
Could you show me the "helper methods"?Thank you!
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub
https://github.com/google-research/football/issues/145#issuecomment-617231053,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AMK2SVXD4B2L4RUG7757AFTRNWXRLANCNFSM4MGILRWQ
.

Because there was something wrong with the python script, I chose to type it directly in the C++ football engine

Then have a look how this is mapped to Python:
https://github.com/google-research/football/blob/0c9dae10322aeb20368b59e5a189961207366898/third_party/gfootball_engine/ai.cpp#L97
and
https://github.com/google-research/football/blob/9de1a8a9d0b7d50175c50b225e5d010ab79435f8/gfootball/env/football_env_core.py#L325
That should answer what is the connection between publicly exposed  observations ad internal value.

Hello! I know this  .def_readonly("direction", &PlayerInfo::player_direction) mapping
so I found this:

But you said this is wrong for player_direction?So I want to know where is the player_direction value in the C++ football engine.
Thank you for your help!


WIP: Population data is out of sync with importers

Couple of notes / questions:

In order to be consistent with continents definition in importers I had to move Turkey and Cyprus to Asia in countries_regions.csv file. This is just a temporary solution (or not?) before we figure out how countries_regions.csv is built
Population density and land area are built separately, but I think they could easily be part of key_indicators table. What do you think? (We can always merge them later)
There are some validation functions to make sure we get similar numbers as in importers, these will be removed before merging together with input and output folders
(Population density is already in world bank dataset, but I assume calculating it ourselves is more robust)

Responding to your questions:
On countries and regions: we want to be the same as our variable 123 in Grapher. If we check the chart preview there, we see that Turkey is in Asia but Cyprus is in Europe (ignoring that half of Cyprus is considered illegally occupied by Turkey). https://owid.cloud/admin/variables/123
@edomt please pipe up if you want to make a different data-editorial decision, though if you do then I think we'd want to change the grapher 123 variable as well, that gets used in a ton of visualisations.
On key_indicators: we could totally just keep them in the key indicators module. At the moment the __init__.py file in that module looks for every table_*.py file and runs it. Perhaps we could just do something simpler and make a manual list, then we can put population density to run after population.
On recalculating this ourselves: since we're calculating population ourselves, and going back far in history, then we also want to be calculating our own population density that matches it

@larsyencken I've addressed your feedback and it's ready for review again. It depends on PR for proper definition of continents, but otherwise is good to go. (inputs, outputs and other validation functions will go away before merging)

Fixed everything, thanks for feedback.

After this, I think we should start adding metadata to the tables.

I'm going to add metadata in a new PR, this one is already a mess.


Show Operation Effects yields 'value is not defined'

example TX:
https://stellar.expert/explorer/testnet/tx/d3e57b4a20ec9d6755745d12c0811e88f705b5a5712baddb34c7d8d0a7c26b4e

Fixed. Thanks for the heads up.


Circularization reference

@stggh, I received an e-mail from a PyAbel user, saying that they found your circularization algorithm really useful for their research. So, good work on that!
Am I correct in thinking that the method is basically what you describe in this paper: http://aip.scitation.org/doi/abs/10.1063/1.4981024
If so, you may want to include a reference to that paper in the abel.tools.circularize_image docstring. Both to get some citations for your paper, but also to let people know where they can get more information on the method/approach.

I appreciate the positive feedback, while also acknowledging the team effort to make the method available.
The algorithm is a simplified version, linear radial scaling vs angle, of what is described in the paper, however, the citation is relevant, and I will include it in the docs.

This will be addressed by #211, so closing this issue.


feat(services): add DynamoDB clients

Current dependencies on/for this PR:

main

PR #57   👈

PR #58

This comment was auto-generated by Graphite.


[docs] better contributing.md

Coverage remained the same at 71.9% when pulling 089a50db9e2e5437d39d075cae03f68b69463d54 on docs/contributing into 888231ca2e9f51f78d320f7ff30d8c7e8e3d00bb on master.


[Question] when are you going to roll out a release with 7.1 syntax support?

Hi @YuraLukashik.
I just created a 0.9.0 release which should now be available via composer (and on packagist.org).
composer require --dev "etsy/phan:0.9"

Let us know if you run into any issues using it.


AttributeError: 'str' object has no attribute 'size'

Hello there!
I'm not sure if this is a noobish problem, but I am out of my wits!
After setting up the package according to the documentation, I did run the proposed minimal code:
import amrlib
stog = amrlib.load_stog_model()
graphs = stog.parse_sents(['This is a test of the system.', 'This is a second sentence.'],True)
for graph in graphs:
    print(graph)

However, I get the following error:
$ python3 stog_test.py
Loading model /home/gandalf/.local/lib/python3.7/site-packages/amrlib/data/model_stog/model.pt
Traceback (most recent call last):
  File "stog_test.py", line 5, in <module>
    graphs = stog.parse_sents(['This is a test of the system.', 'This is a second sentence.'],True)
  File "/home/gandalf/.local/lib/python3.7/site-packages/amrlib/models/parse_gsii/inference.py", line 65, in parse_sents
    return self.parse_file_handle(sio_f, add_metadata)
  File "/home/gandalf/.local/lib/python3.7/site-packages/amrlib/models/parse_gsii/inference.py", line 93, in parse_file_handle
    res = self._parse_batch(batch)
  File "/home/gandalf/.local/lib/python3.7/site-packages/amrlib/models/parse_gsii/inference.py", line 152, in _parse_batch
    beams = self.model.work(batch, self.beam_size, self.max_time_step)
  File "/home/gandalf/.local/lib/python3.7/site-packages/amrlib/models/parse_gsii/modules/parser.py", line 81, in work
    data['pos'], data['ner'], data['word_char'], data['bert_token'], data['token_subword_index'])
  File "/home/gandalf/.local/lib/python3.7/site-packages/amrlib/models/parse_gsii/modules/parser.py", line 62, in encode_step_with_bert
    bert_embed, _ = self.bert_encoder(bert_token, token_subword_index=token_subword_index)
  File "/home/gandalf/.local/lib/python3.7/site-packages/torch/nn/modules/module.py", line 727, in _call_impl
    result = self.forward(*input, **kwargs)
  File "/home/gandalf/.local/lib/python3.7/site-packages/amrlib/models/parse_gsii/bert_utils.py", line 60, in forward
    return self.average_pooling(encoded_layers, token_subword_index), pooled_output
  File "/home/gandalf/.local/lib/python3.7/site-packages/amrlib/models/parse_gsii/bert_utils.py", line 66, in average_pooling
    _, num_total_subwords, hidden_size = encoded_layers.size()
AttributeError: 'str' object has no attribute 'size'

I'm not sure if this is related to amrlib at all, encoded_layers here is a String with the value "last_hidden_layer".
My pip freeze:
amrlib==0.2.1
asn1crypto==0.24.0
beautifulsoup4==4.7.1
blis==0.7.3
Brlapi==0.6.7
catalogue==1.0.0
certifi==2018.8.24
chardet==3.0.4
click==7.1.2
cryptography==2.6.1
cupshelpers==1.0
cymem==2.0.4
dataclasses==0.6
distro==1.3.0
distro-info==0.21
en-core-web-sm @ https://github.com/explosion/spacy-models/releases/download/en_core_web_sm-2.3.1/en_core_web_sm-2.3.1.tar.gz
entrypoints==0.3
evdev==1.1.2
filelock==3.0.12
future==0.18.2
galternatives==1.0.4
graphviz==0.15
html5lib==1.0.1
httplib2==0.11.3
idna==2.6
importlib-metadata==3.1.1
joblib==0.17.0
keyring==17.1.1
keyrings.alt==3.1.1
louis==3.8.0
lxml==4.3.2
meteo-qt==1.0.0
murmurhash==1.0.4
nltk==3.5
numpy==1.19.4
olefile==0.46
packaging==20.7
Penman==1.1.0
pexpect==4.6.0
Pillow==5.4.1
plac==1.1.3
preshed==3.0.4
pycairo==1.16.2
pycrypto==2.6.1
pycups==1.9.73
pycurl==<IP_ADDRESS>
PyGObject==3.30.4
pyparsing==2.4.7
PyQt5==5.15.2
PyQt5-sip==12.8.1
PySimpleSOAP==1.16.2
pysmbc==<IP_ADDRESS>
python-apt==<IP_ADDRESS>
python-debian==0.1.35
python-debianbts==2.8.2
python-magic==0.4.16
pyxattr==0.6.1
pyxdg==0.25
PyYAML==3.13
regex==2020.11.13
reportbug===7.5.3-deb10u1
reportlab==3.5.13
requests==2.21.0
sacremoses==0.0.43
SecretStorage==2.3.1
setproctitle==1.1.10
six==1.12.0
smatch==1.0.4
soupsieve==1.8
spacy==2.3.4
srsly==1.0.4
thinc==7.4.3
tokenizers==0.9.4
torch==1.7.0+cpu
torchaudio==0.7.0
torchvision==0.8.1+cpu
tqdm==4.54.0
transformers==4.0.0
typing-extensions==<IP_ADDRESS>
unattended-upgrades==0.1
Unidecode==1.1.1
urllib3==1.24.1
wasabi==0.8.0
webencodings==0.5.1
word2number==1.1
youtube-dl==2020.12.2
zipp==3.4.0

If you need further information, let me know, I would appreciate any help!

I think the issue is the "transformers" package.  They released version 4.0.0 a few days ago and there are some breaking changes in that release.  I've tested the code with version 3.4.0 and now version 3.5.1 (latest 3.x release).  Can you try uninstalling version 4 and then do...
pip3 install transformers==3.5.1 and let me know if that fixes the issue.  If so, I'll look into code changes to work with the new release.

Thank you so much, it's working now :). Looking forward to work with the package!


[Flake] Sanity tests

Sanity tests are flaky on current master (ba394acd58e4888eb4a1bb3234eb655e37de9d43).
To reproduce:
$  go run ./vendor/github.com/onsi/ginkgo/v2/ginkgo --until-it-fails -race ./test/sanity/...
...
Running Suite: Sanity Suite - /home/rzetelskik/github.com/scylladb/k8s-local-volume-provisioner/test/sanity/set/localdriver
===========================================================================================================================
Random Seed:<PHONE_NUMBER>

Will run 83 of 84 specs
••SS
------------------------------
• [FAILED] [60.080 seconds]
Local CSI Driver CSI sanity Controller Service [Controller Server] [BeforeEach] ListVolumes check the presence of new volumes and absence of deleted ones in the volume list
  [BeforeEach] /home/rzetelskik/github.com/scylladb/k8s-local-volume-provisioner/vendor/github.com/kubernetes-csi/csi-test/v5/pkg/sanity/tests.go:46
  [It] /home/rzetelskik/github.com/scylladb/k8s-local-volume-provisioner/vendor/github.com/kubernetes-csi/csi-test/v5/pkg/sanity/controller.go:209

  Timeline >>
  STEP: connecting to CSI driver @ 05/09/24 10:25:19.402
  [FAILED] in [BeforeEach] - /home/rzetelskik/github.com/scylladb/k8s-local-volume-provisioner/vendor/github.com/kubernetes-csi/csi-test/v5/pkg/sanity/sanity.go:265 @ 05/09/24 10:26:19.458
  << Timeline

  [FAILED] Unexpected error:
      <*errors.errorString | 0xc0004ded00>:
      Connection timed out
      {
          s: "Connection timed out",
      }
  occurred
  In [BeforeEach] at: /home/rzetelskik/github.com/scylladb/k8s-local-volume-provisioner/vendor/github.com/kubernetes-csi/csi-test/v5/pkg/sanity/sanity.go:265 @ 05/09/24 10:26:19.458
------------------------------
P [PENDING]
Local CSI Driver CSI sanity Controller Service [Controller Server] ListVolumes pagination should detect volumes added between pages and accept tokens when the last volume from a page is deleted
/home/rzetelskik/github.com/scylladb/k8s-local-volume-provisioner/vendor/github.com/kubernetes-csi/csi-test/v5/pkg/sanity/controller.go:268
------------------------------
•••••••SSSS•••••••SSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSS•••••••••••SSSSSSSSSSSSS••

Summarizing 1 Failure:
  [FAIL] Local CSI Driver CSI sanity Controller Service [Controller Server] [BeforeEach] ListVolumes check the presence of new volumes and absence of deleted ones in the volume list
  /home/rzetelskik/github.com/scylladb/k8s-local-volume-provisioner/vendor/github.com/kubernetes-csi/csi-test/v5/pkg/sanity/sanity.go:265

Ran 30 of 84 Specs in 61.337 seconds
FAIL! -- 29 Passed | 1 Failed | 1 Pending | 53 Skipped
--- FAIL: TestSanity (61.37s)
FAIL

Tests failed on attempt #3

Ginkgo ran 1 suite in 1m7.147438575s

Test Suite Failed
exit status 1

From what I've seen different tests are failing on connection timeout - it's not a specific test.
/kind flake

https://prow.scylla-operator.scylladb.com/view/gs/scylla-operator-prow/pr-logs/pull/scylladb_k8s-local-volume-provisioner/50/pull-k8s-local-volume-provisioner-master-sanity/1790354916707930112

/remove-lifecycle stale

https://prow.scylla-operator.scylladb.com/view/gs/scylla-operator-prow/pr-logs/pull/scylladb_local-csi-driver/58/pull-local-csi-driver-v0.4-sanity/1813520994044743680#1:build-log.txt%3A23

https://prow.scylla-operator.scylladb.com/view/gs/scylla-operator-prow/pr-logs/pull/scylladb_local-csi-driver/57/pull-local-csi-driver-master-sanity/1813521778492837888#1:build-log.txt%3A23

raising the priority, this seems to be flaking in more than 50% of cases

https://prow.scylla-operator.scylladb.com/view/gs/scylla-operator-prow/pr-logs/pull/scylladb_local-csi-driver/59/pull-local-csi-driver-v0.4-sanity/1813843822224871424#1:build-log.txt%3A23

/remove-lifecycle stale

https://prow.scylla-operator.scylladb.com/view/gs/scylla-operator-prow/pr-logs/pull/scylladb_local-csi-driver/64/pull-local-csi-driver-master-sanity/1851957381261430784#1:build-log.txt%3A30

https://prow.scylla-operator.scylladb.com/view/gs/scylla-operator-prow/pr-logs/pull/scylladb_local-csi-driver/64/pull-local-csi-driver-master-sanity/1851958342826594304#1:build-log.txt%3A26

https://prow.scylla-operator.scylladb.com/view/gs/scylla-operator-prow/pr-logs/pull/scylladb_local-csi-driver/65/pull-local-csi-driver-master-sanity/1852002790944018432#1:build-log.txt%3A29


Avoid memory copies during document writes and reads.

Due to Terracotta Express toolkit migration, memory copies during document
writes and reads have been introduced: those should be avoided because hurting
performance.

Original issue reported on code.google.com by<EMAIL_ADDRESS>on 18 Sep 2010 at 3:02

Fixed by having only one value type: json.
By doing so, no magic number needs to be used for
serialization/deserialization, and so no memory copy is needed (because no
serialization happens altogether).

Original comment by<EMAIL_ADDRESS>on 18 Sep 2010 at 6:08

Changed state: Fixed


remove 1.13-dev builds from master.dockerproject.org

Master was bumped to 1.14-dev, so we should remove old 1.13-dev builds from https://master.dockerproject.org
/cc @mlaventure @andrewhsu

Done, really need to automate this.
The index.html page will be updated at the next merge.


Fix typos in 2016-01-30-iOS-Junior-Interviews

I was reading this fantastic article (thank you for sharing it) and noticed a few typos.
'ice-brakers' -> 'icebreakers'
'More infö' -> 'More info'
Thanks!

ace - thanks


performance issue with blueprint polyhedra meshes

Reported by @BradWhitlock
When plotting very large polyhedral meshes generate_sides() is called multiple times, which is slow.
Brad explored publishing the data as VTK Polyhedra and that improved performance.
Here is his branch:
https://github.com/visit-dav/visit/tree/feature/whitlock/use_vtk_polyhedron_cells
We could also look into caching the generate_sides results.

verdict:
Using full polyhedra support is the path we should pursue.

Polyehdral meshes come up in several plugins (Silo, Exodus, VTK, etc.). It would be good to have a unified way of working with them and not duplicate or use different approaches in each and every plugin that needs it.
This was attempted in Silo plugin...develop a generalized method to load a polyehdral mesh from plugin upwards and in avtTransformManager do whatever magic transformation to convert (if needed) to what VisIt needs internally.


fix deprecated warning

fix deprecated warning

link: webpack/loader-utils#56

published as 4.3.0!


Rename ICartesiDApp functions

📚 Context
The CartesiDApp contract has several functions that can be renamed.

getConsensus :arrow_right: consensus
getTemplateHash :arrow_right: templateHash

Through this renaming, we can use the public keyword on the state variables and remove the getter functions. This change will update the ICartesiDApp interface, as well as the CartesiDAppFactory contract and ICartesiDAppFactory interface.
✔ Solution
Rename the functions in the interface, remove any unnecessary getter function, and make the state variables public. Then, update the tests.

On second thought, let us avoid voluntary breaking changes...


Update CardData.model_name on push_to_hub if its different

If a user set a certain model_name and its different than the model name of the repo they're pushing to, change to use that instead.
Perhaps warn user about this or have a flag to avoid/enable this behavior?

I'm not sure if this is a desired behavior. It would be nice to have time to react to these issues :D


Make clearer that SQL Server arguments are alternatives

by making them an unordered list instead of just putting each term in a new line
(Not a great solution to the problem though 🤔 )

Thanks @janpio I think the bullets do help make it clear that each is a separate option.

Sure. (Generally I am fine with maintainers of a repo to press the button themselves on PRs that are up for review - so you can go ahead next time if you agree. I try to keep not ready PRs on Draft)


DNS search suffix not for namespace on Windows pods

Is this an ISSUE or FEATURE REQUEST? (choose one):
Issue
Which release version?:
v1.0.4-1-gf0f090e
Which component (CNI/IPAM/CNM/CNS):
CNI
Which Operating System (Linux/Windows):
Windows Server version 1803
Which Orchestrator and version (e.g. Kubernetes, Docker)
Kubernetes
What happened:
The DNS suffix is not set based on the pod's namespace.
What you expected to happen:
On Linux, we see the right resolution order.
/etc # cat resolv.conf
nameserver <IP_ADDRESS>
search default.svc.cluster.local svc.cluster.local cluster.local lje1130gi5iehkdbexza4ptcba.xx.internal.cloudapp.net

Windows only has svc.cluster.local
How to reproduce it (as minimally and precisely as possible):
Deploy a Windows cluster with acs-engine v0.17

This is fixed in PR#152

@saiyan86 I see this issue again on acs-engine v0.18.9 Since you did #152 are you on point for fixing this?


Removing default karaf.key

Karaf bundles a default "karaf.key" for SSH. This is not good security practice, and it doesn't really have any practical use unless the user uncomments the hard-coded key in etc/keys.properties. The latter is not something we should support, if the user wants to use keys.properties, then the user should generate a keypair and put the public key into keys.properties as explained in the manual.
I tested that SSH'ing into Karaf using both a password, and a key defined in keys.properties works correctly with this fix, both for openssh and using bin/client.

retest this please

It seems that the GitHub/Jenkins integration doesn't work since yesterday (not only for Karaf, but globally). I'm checking with infra.

retest this please

retest this please

I gonna test this PR manually waiting fix on github/jenkins integration.

retest this please

retest this please

retest this please

The tests failures have been fixed. I'm rebasing to test.


Use ConsumerProperties in KafkaMessageSource

There are still a few more properties that need to be populated using setters on KafkaMessageSource e.g. messageConverter, payloadType, rawMessageHeader.
Does it make sense to create a configuration sub-class of ConsumerProperties?

This PR doesn't honor commit callback, async commits, commit action logging configuration from ConsumerProperties. These can be partially or fully handled in #282
Changes related to topicPattern and topicPartitionsToAssign will be covered as part of #281

These can be partially or fully handled

Agreed.

Does it make sense to create a configuration sub-class of ConsumerProperties?

I think they can remain on the message source (we have a similar situation on the listener containers where a few properties are set there directly).
Let's see what @artembilan says.

@artembilan I incorporated your feedback. Let me know if it looks good now.

LGTM, but we need a note in the what's new about the new default poll timeout - this can be a problem if they have several adapters and are using the default task scheduler which only has 10 threads since the thread will be tied up for 5 seconds if there are no new records.

in the what's new about the new default poll timeout

Right, but that's already a Spring Kafka project mattern. Doesn't look like we document anything here..


Add font-size slider

This PR brings the following changes:

hide sample text input box until one or several fonts has/have been selected
add a range slider input which allow changing font-sizes of samples

Awesome addition! Thank you!


fix(leetcode): now input value will be parsed with proper ways

this PR fixes #1

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Implement service support

I'm able to run the WASI demos (greet, hello) without issue. Both of these demos have READMEs that explain what to expect.
I'm able to run (seemingly) both the WASCC demos (greet, uppercase) but there's no explanation as to how to confirm that these are running successfully and I'm unclear that they are.
It may be that my configuration is not working correct for WASCC but I'm unable to confirm this because I'm unable to explore these demos fully.
Both Pods appear to run HTTP endpoints on :8080.
Absent the ability to kubectl port-forward to these pods, I've tried exposing them as services. But the service(s) don't receive endpoints and appear unable to select the WASCC pods.
I'm evidently misunderstanding and|or doing something incorrectly but the demos don't explain what I should expect or do next.
kubectl get pods
NAME                                    READY   STATUS       RESTARTS   AGE
greet                                   1/1     Running      0          10m
uppercase                               1/1     Running      0          10m

Describing greet appears to show everything running as expected (8080/8080):
kubectl describe pod/greet
Name:         greet
Namespace:    kube-system
Priority:     0
Node:         krustlet/
Labels:       app=greet
Annotations:  Status:  Running
IP:
IPs:          <none>
Containers:
  greet:
    Container ID:
    Image:          webassembly.azurecr.io/greet-wascc:v0.4
    Image ID:
    Port:           8080/TCP
    Host Port:      0/TCP
    State:          Running
      Started:      Tue, 12 May 2020 09:31:32 -0700
    Ready:          True
    Restart Count:  0
    Environment:
      PORT:  8080
    Mounts:
      /var/run/secrets/kubernetes.io/serviceaccount from default-token-7956z (ro)
Conditions:
  Type           Status
  PodScheduled   True
Volumes:
  default-token-7956z:
    Type:        Secret (a volume populated by a Secret)
    SecretName:  default-token-7956z
    Optional:    false
QoS Class:       BestEffort
Node-Selectors:  beta.kubernetes.io/arch=wasm32-wascc
                 beta.kubernetes.io/os=linux
                 kubernetes.io/role=agent
Tolerations:     krustlet/arch=wasm32-wascc:NoExecute
                 node.kubernetes.io/network-unavailable:NoSchedule
                 node.kubernetes.io/not-ready:NoExecute for 300s
                 node.kubernetes.io/unreachable:NoExecute for 300s
Events:
  Type    Reason     Age        From               Message
  ----    ------     ----       ----               -------
  Normal  Scheduled  <unknown>  default-scheduler  Successfully assigned kube-system/greet to krustlet

Exposing both WASCC pods as services (--port=8080 --target-port=8080):
kubectl get services
NAME        TYPE        CLUSTER-IP       EXTERNAL-IP   PORT(S)                  AGE
greet       ClusterIP   <IP_ADDRESS>    <none>        8080/TCP                 10m
uppercase   ClusterIP   <IP_ADDRESS>    <none>        8080/TCP                 10m

But no Endpoints for service/greet (same-same for service/uppercase):
kubectl describe service/greet
Name:              greet
Namespace:         kube-system
Labels:            app=greet
Annotations:       <none>
Selector:          app=greet
Type:              ClusterIP
IP:                <IP_ADDRESS>
Port:              <unset>  8080/TCP
TargetPort:        8080/TCP
Endpoints:         <none>
Session Affinity:  None
Events:            <none>

kubectl describe service/uppercase
Name:              uppercase
Namespace:         kube-system
Labels:            app=uppercase
Annotations:       <none>
Selector:          app=uppercase
Type:              ClusterIP
IP:                <IP_ADDRESS>
Port:              <unset>  8080/TCP
TargetPort:        8080/TCP
Endpoints:         <none>
Session Affinity:  None
Events:            <none>

The Pod(s) aren't getting ClusterIPs which, I assume, is the more fundamental issue; no ClusterIP, means the Pod can't be bound by a Service.

NOTE This is very possibly related to my configuration using MicroK8s.

kubectl get pods --output=wide
NAME        READY   STATUS    IP          NODE
greet       1/1     Running   <none>      krustlet
kuard       1/1     Running   <IP_ADDRESS>   workstation
uppercase   1/1     Running   <none>      krustlet

NOTE Edited for clarity

correct. In the meantime, you can curl localhost:8080 from the same machine running krustlet to reach the endpoint. Right now the waSCC runtime starts the webassembly modules. They're just not hooked up to the cluster.

Aha! This would be useful to add to the architectural overview (diagram).
This explains why I receive address in use errors. Unfortunately, I've an older version of MicroK8s that expose the kube-apiserver on :8080. So, if the Krustlet is attempting to spawn the waSCC runtime on :8080 (?) that explains more of my problems.
IIUC, waSCC on 8080 is not (currently) configurable.
I'm going to try to reconfigure MicroK8s to drop the insecure port for kube-apiserver.
Thanks!

IUC, waSCC on 8080 is not (currently) configurable.

Just change the container port to something other than 8080: https://github.com/deislabs/krustlet/blob/6d2cbfe119cf893761a09c527189dfbec29f5177/demos/wascc/greet/greet-wascc.yaml#L16

Thank you!
IIUC the "wascc:http_server" is being bound to the environment variable value PORT provided by the container. So line 14 sets the runtime port and line 16 exposes it, correct?
  containers:
    - image: webassembly.azurecr.io/greet-wascc:v0.4
      imagePullPolicy: Always
      name: greet
      env:
        - name: PORT
          value: "7777"
      ports:
        - containerPort: 7777

Then:
curl http://localhost:7777
{"method":"GET","path":"/","query_string":"","body":[]}

curl http://localhost:7777/freddie?hello
{"method":"GET","path":"/freddie","query_string":"hello","body":[]}

It would be useful to document this too.
I'm happy to do so if I understand it correctly.

Ah yeah forgot about that. You're right on that. Thanks for finding that out.

Not sure if this is the right spot but it might be a good idea to add to the MicroK8s Prerequisites that you need the '--enable-bootstrap-token-auth' flag on 1.18 for the kube-apiserver. I installed MicroK8s through snap so it was a simple entry to /var/snap/microk8s/current/args/kube-apiserver. I was having trouble with the wasi demos getting a 401 Unauthorized when attempting to apply the CertificateSigningRequest and it wasn't obvious to me that flag was the issue.

We’d love a microk8s guide in the how-to guide section of our documentation if you have the time to write one up. :)
https://github.com/deislabs/krustlet/tree/master/docs/howto

closing as the krustlet-wasmcloud provider has been moved to its own project: https://github.com/wasmCloud/krustlet-wasmcloud-provider
Please open tickets related to the wasmclou provider there. Thanks.


Bug happens when get NFT from ipfs-gateway.cloud

Description the bug
Bug happens when get NFT from ipfs-gateway.cloud

Coinciding with issue 611


[Marketplace] it looks like the tabs are moving if user switches between them (All applications and My applications)

EPAM AI DIAL chat version
0.18.0
What happened?
[Marketplace] it looks like the tabs are moving if user switches between them (All applications and My applications)
Please see the video
https://github.com/user-attachments/assets/7705db6f-0b47-49c7-aead-0f7823421053
Confidential information

[X] I confirm that do not share any confidential information

use border-transparent for not selected ones

verified on staging successfully


Enable mochawesome rendered context adding

Hi, this PR allows to use your code and enable to add the browser logs on the mochawesome renderer as a context.
Reference plugin renderer: https://www.npmjs.com/package/mochawesomeplusplus

Thanks @EstebanSerfe!


net/osdp: Add OSDP library

This PR adds the OSDP library implementation along with a sample application. This is a mynewt port of OSDP implementation available here: libosdp.
Sample app tested with a control panel script from library above, sending commands over serial to an nrf52840DK. Screenshot:

So, again I have been talking and working with @vikrant-proxy on this. I think as a first pass this looks good. The data structure changes would be with follow on PRs as we proceed. We thought it would be a good addition to the Mynewt ecosystem.

So, again I have been talking and working with @vikrant-proxy on this. I think as a first pass this looks good. The data structure changes would be with follow on PRs as we proceed. We thought it would be a good addition to the Mynewt ecosystem.

What is the licence of circular_buffer.* files? (those don't seem to be part of osdp lib)

What is the licence of circular_buffer.* files? (those don't seem to be part of osdp lib)

What is the licence of circular_buffer.* files? (those don't seem to be part of osdp lib)

@sjanc it is a part of embedded-resources repo under CC0 1.0 license. I have updated the files with the license header.

What is the licence of circular_buffer.* files? (those don't seem to be part of osdp lib)

@sjanc it is a part of embedded-resources repo under CC0 1.0 license. I have updated the files with the license header.

@sjanc Do the copyrights look fine now ?


Video conference mute bug

Microsoft PowerToys version
0.53.3
Running as admin

[X] Yes

Area(s) with issue?
Video Conference Mute
Steps to reproduce
clicking on the "Browse" button not working (look screen shot)

✔ Expected Behavior
This should open a windows so I can select which Photo I want to use over there
❌ Actual Behavior
clicking on the button will do nothing
Other Software
Droidcam (last version for both phone and PC)

sorry this was duplicated here

Duplicate of #13215


url-encoding

All in all, pretty good. Feel free to make any of the changes I recommend above and then submit.
Don’t forget to rename the folders - and consider pulling Layout and Body up to src/ if making components/ reserved for some of the sub elements makes it easier for you.

so I can merge after these changes are done?


[QUESTION] Steps to create a new user with rw rights

I'm having difficulties in creating new users with read/write permissions over a schema/database. I'm also struggling in creating a new "sa" equivalent user.
So far, using TDS, I succeeded in creating a new login via CREATE LOGIN qwe WITH PASSWORD = 'qweqweqwe' and enabling the user to a specific DB through the statement CREATE USER qwe FOR LOGIN qwe.
I was expecting the user to view all the schemas tables, but I was wrong.
The new user cannot view any table, the new user cannot create new tables on any schema.
A workaround could be to GRANT privileges over the various objects, although this does not seem to fix the missing CREATE rights.
Can you point me to some documentation/example? Thanks!

What you describe here is a regular login and ditto user, without any special roles. In SQL Server, that user would not have access to anything except the guest schema.
If you wanted to create an sa-style login, then you have to grant server role sysadmin to the login. That will automatically make the login dbo in all databases. If you just want to create a user with their own schema, then sysadmin role is not needed but you can run (as sysadmin) CREATE SCHEMA qwe_schema AUTHORIZATION qwe

Thanks! Can you confirm me that on version 3.2:

there cannot be multiple db owners
the ALTER AUTHORIZATION ON SCHEMA is not supported

My consideration is that one has to choose between creating a role and assign granular permissions to all the DB objectes, or assign the sysadmin role to the user - the schema authorization seems not very practical (unless planned in advance).
In the newer commits I see advancements in this area, I'm looking forward to check the next release!

Both of those are indeed not supported. The work that you may have seen to be in progress is for GRANT/REVOKE on SCHEMA, not for ALTER AUTHORIZATION ON SCHEMA:: (but GRANT ON SCHEMA will not yet be included in the upcoming release v.3.4).
v.3.4 is expected to include ALTER AUTHORIZATION ON DATABASE:: .


tutorial: replace tree with find

tree might be not installed, find almost certainly is.

Thanks!


Adds sidebar .hover class to dropdown

Additions

Adds .hover class to dropdown and updates docs to show hover/focus state.

Testing

git fetch && git checkout form_states
npm run cf-link
Switch to a CF clone repo.
git fetch && git checkout gh-pages-form_states
npm run cf-link
npm run build && npm start

Screenshots

Notes

Covered under existing changelog entry for cf-forms hover classes.

@nataliafitzgerald

@ans
Can we divide out hover and focus into separate states to look like this?
Hover

Focus

@anselmbradford
This PR looks good. Just want to confirm two things before we merge.
- Default - Body (Avenir Next Regular, 16px), Black (#101820)
- Hover - Border: 2 px, Pacific (#0072ce)
- Disabled - Body (Avenir Next Regular, 16px), Gray (#5a5d61)
As discussed we'll handle the focus state in a separate PR.

Default - Body (Avenir Next Regular, 16px), Black (#101820)

We can't set the font family of the drop-down text (e.g. "Option 1") without some very fancy hacks. Label is Avenir.

Hover - Border: 2 px, Pacific (#0072ce)

This is correct.

Disabled - Body (Avenir Next Regular, 16px), Gray (#5a5d61)

Color fixed in https://github.com/cfpb/capital-framework/pull/606/commits/0edbde00fa3851e711b468306b73ba1c8d1a6a35

Updated screenshot

@anselmbradford
This looks good to go!


Is there any way to dynamically see whether radio values have been changed or not?

I have some content that I wish to show and hide based on the conditions changed in the radiolist. Is there any way I can see the value. (The same way ng-model works)

Couldn't you use e-ng-changed to set some value that could be referenced by a ng-hide or ng-show?

@shanahanbaptista are you still having an issue?

@eugef Please close due to lack of response.


chore: hello-gno app example

Hello world gno example
This PR adds a new sample app under "examples/hello". This app is a minimal app that connects to the blockchain.

I enter:
git clone https://github.com/iuricmp/gnomobile --branch hello
cd gnomobile
make build

It prints the error:
cd /Users/jefft0/work/gno/gnomobile/examples/hello; /Applications/Xcode.app/Contents/Developer/usr/bin/make node_modules
make[1]: *** No rule to make target `node_modules'.  Stop.

Thanks @D4ryl00


npm install failure, tsc not found

We run js-ipfs-api tests as part of go-ipfs CI, and started getting this failure today or very recently: http://ci.ipfs.team:8111/viewLog.html?buildId=7909&buildTypeId=GoIpfs_JsIpfsApiTests&tab=buildLog#_focus=771
[22:25:05][Step 2/3] ><EMAIL_ADDRESS>build:es5 /teamcity/work/3144e2a06f62fd48/js-ipfs-api/node_modules/webcrypto-core
[22:25:05][Step 2/3] > tsc --module commonjs --target es5
[22:25:05][Step 2/3]
[22:25:05][Step 2/3] sh: 1: tsc: not found

Took a quick look at this. Seems webcrypto-core is trying to use a Typescript compiler on install, which is not installed with the module. But, more worrying, I can't even see where webcrypto-core is coming from, it's not listed in our package.json and I only see (empty) when doing npm ls webcrypto-core on current master.
➜  js-ipfs-api git:(master) npm ls webcrypto-core
<EMAIL_ADDRESS>/home/victor/projects/ipfs/js-ipfs-api
└── (empty)

Makes me think that maybe js-ipfs-api-tests are running an old version of js-ipfs-api or something is screwed with the cache. Otherwise, maybe @dignifiedquire have an idea, since this seems related with the crypto stuff.

Hah, apparently npm ls only lists already installed modules instead of what modules would be installed... Installing modules with dry-run tells us which module this comes from:
<EMAIL_ADDRESS>/home/victor/projects/ipfs/js-ipfs-api
├─┬<EMAIL_ADDRESS>│ └─┬<EMAIL_ADDRESS>│   └─┬<EMAIL_ADDRESS>│     └──<EMAIL_ADDRESS>(git+https://github.com/PeculiarVentures/webcrypto-core.git#cb7b6c66030a41e4ff53e9137391efead37b09b2)
└──<EMAIL_ADDRESS>

Ok, now I figured out the problem and a fix. TLDR: We need to merge https://github.com/libp2p/js-libp2p-crypto/pull/34 to update node-webcrypto-ossl whose install was broken for people who don't have Typescript globally installed cc @diasdavid @dignifiedquire
Long story: webcrypto-core had an issue where unless you have the Typescript compiler installed globally, install failed. The version of node-webcrypto-ossl that we use in libp2p-crypto was locked to commit cb7b6c66030a41e4ff53e9137391efead37b09b2 of webcrypto-core where this issue exists. Author have pushed an update to fix this (version 1.0.10) which we got an PR from greenkeeper to merge (as seen here: https://github.com/libp2p/js-libp2p-crypto/pull/34). However, it's breaking the build so some more work is required.

Also, came across this from mediachain where they hit the same issue https://github.com/mediachain/aleph/pull/79

Restarted CI on the libp2p-crypto PR, going to merge and ship as soon as this actually works again on CI

Should be fixed now, please try again with a fresh cache

@lgierth I rerun the test you linked before and install is now successful! http://ci.ipfs.team:8111/viewLog.html?buildId=7994&buildTypeId=GoIpfs_JsIpfsApiTests&tab=buildLog#_focus=748

<3 thank you all <3 it felt really awesome to see a problem being reported, fixed and released involving so many people and happening so fast! :D


Shortcode being ignored

Moving from #130 to here.
@iJos what's the filename and can you paste the snippet of the .md file where you are using it?

Sure, this is a test, for embedding the Twitch player, the template is almost the same as the Youtube shortcode.

twitch.html
`<div {% if class %}class="{{class}}"{% endif %}>

`

page.md

{{ twitch(channel="**channel-name**") }}
I think I've just found the bug pasting the code here, I'm trying to reproduce.

The shortcode is being ignored when the variable I'm passing as "channel" contains an underscore.
For example, when trying to use the shortcode with the official twitch channel (/twitch) the shortcode is being parsed and the html is generated.
When using a name with underscore symbol (the first username I've found in the homepage is "rhobalas_lol") the shortcode is ignored and the output is the plain text "{{ twitch(channel="rhobalas_lol") }}"

Another regex issue :(
I'll fix it and add tons of test for it I guess

At least we know the issue.
I will create a PR with the twitch shortcode, do you want me to wait the fix of this issue?

The issue is the markdown parser splitting the text on the underscore. Looks like more hacks are required!

This should be fixed in master now if you want to try building it from source

Nice! I'll check it!


The button effect is too small

Hi,
So I added these awesome toolkit to my project. When I click a button, it should animate the entire button area right? It only animated the area around the click in a circle. Is there any setting to be done?

Hey, I'm actually working on improving the effects right now, so you there will be some improvements over the next few days.
Note, I've moved the nuget package to beta, so you'll need to include pre-release packages in your nuget search to see the latest changes.

Awesome thanks! Looking forward to it.


Minor fixes, re-wording

This PR includes some minor suggestions to improve wording.

thanks a lot @burigolucas great suggestions! 🔥🦄


README outdated

Hey there!
I'm trying to use this gem and I have some issues
KlaviyoAPI::Events.get_events(opts) => uninitialized constant KlaviyoAPI::Events (NameError) Did you mean?
KlaviyoAPI::EventsApi
probably an outdated README?
Another thing I need to do is to do dynamic api_key and api_secret. This is what I came up, but it still gives me 401
config = KlaviyoAPI::Configuration.new
config.api_key = <API_KEY>
config.access_token = <SECRET_KEY>
api_client = KlaviyoAPI::ApiClient.new(config)

KlaviyoAPI::EventsApi.new(api_client).get_events

Can you help me please?

Hi @Alagaesia93 , you need to configure your client as described in the documentation otherwise you might get unexpected results:
# Setup authorization
KlaviyoAPI.configure do |config|
  config.api_key['Klaviyo-API-Key'] = 'Klaviyo-API-Key your-api-key'
  #config.max_retries = 5 # optional
  #config.max_delay = 60 # optional
end

to use a different private key per call you can do this:
response = KlaviyoAPI::Events.get_events({
  header_params: {
    'Authorization': 'Klaviyo-API-Key your-api-key',
  },
  debug_auth_names: []
})

The client doesn't require an access token.

Ok thanks a lot for the custom key.
The problem is that KlaviyoAPI::Events is not defined, should it be KlaviyoAPI::EventsApi

So we use some wrapper logic to strip out the Api suffix. If you configure the client as noted above, you should be able to use KlaviyoAPI::Events


refactor: 优化OkHttp装配方式

Summary by Sourcery
增强功能：

通过将 OkHttp 客户端配置提取到一个单独的类中进行重构，以提高模块化和可重用性。

Original summary in English
Summary by Sourcery
Enhancements:

Refactor the OkHttp client configuration by extracting it into a separate class to improve modularity and reusability.

@sourcery-ai summary
@sourcery-ai review

@sourcery-ai review

@sourcery-ai summary

@sourcery-ai review


Azure Alerts for Errors - CDC TI services are down/offline

Story
As a developer, I need to know if our services are down/offline, so that we can begin an incident.
Pre-conditions

[ ] Assumptions of prior or future work that's out of scope for this story

Acceptance Criteria

[ ] Alerts trigger when the TI and/or Ingestion Service are down. After discussion, we decided to base this on whether there are an appropriate number of healthy instances - if instances can't stay up/healthy, the service is not working

Tasks
Engineering

[x] Alerts are set up that trigger if TI is down
[x] Alerts are set up that trigger if Ingestion Service is down

Definition of Done

[ ] Documentation tasks completed

[ ] Documentation and diagrams created or updated

[ ] ADRs (/adr folder)
[ ] Main README.md
[ ] Other READMEs in the repo
[ ] If applicable, update the ReportStream Setup section in README.md

[ ] TI Threat model or Ingestion Service threat model updated
[ ] API documentation updated

[ ] Code quality tasks completed

[ ] Code refactored for clarity and no design/technical debt
[ ] Adhere to separation of concerns; code is not tightly coupled, especially to 3rd party dependencies

[ ] Testing tasks completed

[ ] Load tests passed
[ ] Additional e2e tests created
[ ] Additional RS e2e assertions created in the rs-e2e project for any new transformations.  Includes improvements to the assertion code required to make the new assertions

[ ] Build & Deploy tasks completed

[ ] Build process updated
[ ] API(s) are versioned
[ ] Feature toggles created and/or deleted.  Document the feature toggle
[ ] Source code is merged to the main branch

Research Questions

Optional: Any initial questions for research

Decisions

Optional: Any decisions we've made while working on this story

Notes

This story came out of #1379, and #1380.

I believe this is covered by #1397

Already complete?

Alert was successfully activated and deactivated in both repos. We are currently considering how to handle the ADR for Azure Alerts (duplicate them in both repos, exist only in one repo, etc).

ADR was added to both repos.


Refactor : Server (This needs to be divided into several parts)

Ⅰ. Describe what this PR did
Refactor : Server
Ⅱ. Does this pull request fix one issue?

Ⅲ. Why don't you add test cases (unit test/integration test)?
Ⅳ. Describe how to verify it
Ⅴ. Special notes for reviews

@ph3636
[ERROR] Errors:
[ERROR]   DefaultCoordinatorMetricsTest.test:44 » NullPointer
[ERROR]   DefaultCoordinatorTest.beforeClass:79 » NullPointer
[ERROR]   DefaultCoreTest.initSessionManager:76 » NullPointer
[ERROR]   DefaultCoreForEventBusTest.test:59 » NullPointer
[ERROR]   SessionHolderTest.testInit:52 » NullPointer
[ERROR]   SessionStoreTest.testRestoredFromFile:129 » NullPointer
[ERROR]   SessionStoreTest.testRestoredFromFileAsyncCommitting:206 » NullPointer
[ERROR]   SessionStoreTest.testRestoredFromFileCommitRetry:266 » NullPointer
[ERROR]   SessionStoreTest.testRestoredFromFileRollbackFailed:378 » NullPointer
[ERROR]   SessionStoreTest.testRestoredFromFileRollbackRetry:327 » NullPointer
Fix the unit test exception, or you won't pass the ci check

thanks, I know. I'm a little busy now. @lzf971107

thanks, I know. I'm a little busy now. @lzf971107

Oops, sorry to bother you

Thank you for reminding me. @lzf971107

thanks, I know. I'm a little busy now. @lzf971107

Oops, sorry to bother you

Codecov Report

Merging #2100 into develop will increase coverage by 0.02%.
The diff coverage is 44.16%.

@@              Coverage Diff              @@
##             develop    #2100      +/-   ##
=============================================
+ Coverage      55.16%   55.19%   +0.02%
- Complexity      2515     2528      +13
=============================================
  Files            446      448       +2
  Lines          14922    14950      +28
  Branches        1768     1764       -4
=============================================
+ Hits            8232     8251      +19
- Misses          5922     5933      +11
+ Partials         768      766       -2

Impacted Files
Coverage Δ
Complexity Δ

...torage/file/store/FileTransactionStoreManager.java
56.5% <ø> (ø)
29 <0> (?)

...o/seata/server/storage/db/lock/DataBaseLocker.java
16.27% <ø> (ø)
5 <0> (?)

...rver/storage/db/store/DbcpDataSourceGenerator.java
0% <ø> (ø)
0 <0> (?)

...ver/storage/db/session/DataBaseSessionManager.java
33.87% <ø> (ø)
12 <0> (?)

...in/java/io/seata/server/event/EventBusManager.java
66.66% <ø> (ø)
1 <0> (ø)
:arrow_down:

...ver/storage/db/store/DruidDataSourceGenerator.java
0% <ø> (ø)
0 <0> (?)

...ata/server/storage/file/TransactionWriteStore.java
83.33% <ø> (ø)
8 <0> (?)

...c/main/java/io/seata/server/store/StoreConfig.java
83.33% <ø> (ø)
5 <0> (ø)
:arrow_down:

...va/io/seata/server/storage/file/FlushDiskMode.java
77.77% <ø> (ø)
3 <0> (?)

...ta/server/storage/db/lock/DataBaseLockManager.java
0% <0%> (ø)
0 <0> (?)

... and 22 more

@ph3636 Please resolve conflicting files.

@ph3636 Please resolve conflicting files.

This needs to be divided into several parts. At present, one has been disassembled.https://github.com/seata/seata/pull/2134

temporarily closed due to duplication.


Contributors url changed to to show public profiles. Related issue: #14

This PR simply changes URL to contributors list. New URL will show public profiles for every user. Old one showed profiles only for organization members.
Resolves https://github.com/tuist/website/issues/14

[ ] Open Tuist.org contributors url from website header.

Congrats for your first contribution @Austinate 🚀


conflict with SG Optimizer

Hi Guys,
I have a site using plausible analytics that needs to show up accurately. https://firstfleetfellowship.org.au
This site used to have something like 20 000 visitors a month, and now only shows something like 20. I added the plugin again, as it had disappeared, so I am unsure what is happening. It is hosted on siteground and uses SG optimiser, and I can use WPRocket if needed. The attached is from November 22 to date.

Any thoughts?
Cheers
Nigel

hi @thbeach! i checked your site and cannot see our script on it which explains why the stats are not being tracked. there might be a plugin conflict. have you cleared the cache? what version of our plugin are you using? and is the proxy enabled in the plugin? as an alternative do try WP Rocket instead and see if that works better

have added wp rocket and can now see plausible in the script, I have enabled the proxy too, so I am hoping that it is working ok, I can see live visitors now, so I think we are back to normal. Thanks for your help.
cheers
Nigel

great, thanks for the update @thbeach! i'll keep this one open so @Dan0sz can have a look at fixing the conflict with SG Optimizer

Hi @thbeach,
Thanks for reporting this issue. I'm having a hard time replicating this on my local machine. Would you mind sharing your settings in SG Optimizer?
Thanks!

closing this for now as we cannot replicate but we can always reopen if we get more info


Fix issue where RustServerMetrics is not considering the fact that the server can be ready earlier than it thinks

Plugin EarlyQ allows server to start way earlier than RustServerMetrics expects.
This pull request fixes an NRE that will be thrown when a player joins the server before its normal start

I would suggest fixing your CI

Please merge master into your pr for the CI fix

Im not sure the ci will build it :D
there is 2 older commits from this pr that are on almost 3h runtime, they hung or something


don't force-repoll on implicit team chat send hot path

fix the test that this broke, and in general:
fix the notification system internal to the app that adjusts the locally cached version of the teamchain
sometimes bump the chain 0, sometimes 1, sometimes 2, depending on the caller!
previously, always bumping +1 definitely caused some bugs, but we likely got away with it

cc @mmaxim  this should fix the chat perf bug


Correct 32-bit type values

Correct 32-bit type values
HSM Hart Suspend Types, SRST System Reset Types and System Reset Reasons,
SUSP System Sleep Types are all 32-bit wide, so there is no such a value
that can be larger than 0xFFFFFFFF. Correct them.
Signed-off-by: Bin Meng<EMAIL_ADDRESS>

Thanks!


Move HalfCauchy,Binomial upstream

Addresses #1267

ill wait till @neerajprad's PR bumping pytorch's version in travis to rerun these tests before merging

@jpchen Yeah, I'll dump all distributions changes into this PR.

This should be covered in our pytorch-1.0 branch now.


Something wrong with labeling in classifier

when I use this data set
in classification problem. The metric=accuracy was below 0.4 and the prediction in test data is about 0.5(random guess, however I use the same algorithm in python gives me about 0.6 accuracy), auc metric behaves consistent with python sklearn algorithms during run phase. But the prediction in the test data does not match the auc given by stacknet command line information.(Again, it looks like a random guess)
One possible problem is that the data use (-1,1) to encode the target class, which is not normal but since sklearn can handle this pretty well I really hope stacknet can do this as well!

Could you share the whole command you used, parameters file and if possible the dataset (if you did any changes to that one).  In principle, it should work with -1/1, but there could be a bug.

params
command I used
And I made a mistake, the prediction accuracy and auc score is normal in the test data, but command line information about accuracy is wrong. The problem exist when running the original data.  Thanks!
@kaz-Anova


Add Python 3.7 and 3.8 to build matrix

Per https://github.com/google/open-location-code/issues/369#issuecomment-569322868 , Python 3.6 is already tested, however https://github.com/google/open-location-code/issues/369 states:

the Python module does not work in Python 3.7

So, adding python 3.7 and 3.8, which have some potentially breaking changes, to the build matrix for continuous testing.

@googlebot I signed it!

It appears that the java build has problems with newer jdks, specifically with javadoc in jdk11+, and oraclejdk10 quit with a deprecated message, so I've removed all non-LTS Oracle JDKs from the test matrix.

This still leaves the failures for jdk11+, so shall I comment those out, or allow them to fail?

Feel free to comment them out for now. I'll merge in the PR once done.

rubocop still failing as it has from the start, though everything else is green.

__


Fixes/774

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

3 of 3 (100.0%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall first build on fixes/774 at 97.82%

Totals

Change from base Build<PHONE_NUMBER>:
97.8%

Covered Lines:
1366

Relevant Lines:
1391

💛  - Coveralls


dissent

This pull request was generated by the 'mq' tool
[test]
flake rate: 0.1
logical conflict every: 100
sleep for: 300s
close stale after: 4 hours
[pullrequest]
requests per hour: 0
deps=[d]

/trunk merge


Analyzing media from Panansonic DC-GH6 camera takes a very long time

Linked is one of the media files where MediaInfo takes way longer to retrieve Media information than normal.
Looks like MediaInfo 23.11.1 does a full decode and not a quick scan.
Files are on local storage and dont look damaged or corrupt and once the media information has been extracted it looks pretty normal.
Tried also older versions but the result is the same.
Sample File: https://we.tl/t-ZfZDS7deCn

Looks like MediaInfo 23.11.1 does a full decode and not a quick scan.

No, it was a super slow tentative (and fail) to decode a big (it seems to contain a JPEG image) unrecognized metadata block.
Fixed, will be in next development snapshots and next release.
Note that Panasonic metadata (XML or binary) are not yet decoded (on our todo-list but no ETA as we have no sponsor for that so lower priority).


Remove ITask

it is an abstraction that is not used

Unhandled exception. System.TypeLoadException: Could not load type 'Microsoft.Testing.Platform.Helpers.ITask' from assembly 'Microsoft.Testing.Platform, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=b03f5f7f11d50a3a'.
at Microsoft.Testing.Extensions.TrxReportExtensions.<>c__DisplayClass0_0.b__6(IServiceProvider serviceProvider)
at Microsoft.Testing.Platform.Tools.ToolsManager.BuildAsync(IServiceProvider serviceProvider) in //src/Platform/Microsoft.Testing.Platform/Tools/ToolsManager.cs:line 24
at Microsoft.Testing.Platform.Hosts.TestHostBuilder.BuildAsync(String[] args, ApplicationLoggingState loggingState, TestApplicationOptions testApplicationOptions, IUnhandledExceptionsHandler unhandledExceptionsHandler, DateTimeOffset createBuilderStart) in //src/Platform/Microsoft.Testing.Platform/Hosts/TestHostBuilder.cs:line 261
at Microsoft.Testing.Platform.Builder.TestApplicationBuilder.BuildAsync() in //src/Platform/Microsoft.Testing.Platform/Builder/TestApplicationBuilder.cs:line 113
at Program.$(String[] args) in //test/UnitTests/Microsoft.Testing.Platform.UnitTests/Program.cs:line 34
This is used by code that is extending the testing.platform and that is in our non-public repo.  I only see few usages, and I don't see any usages in Tests.
Is this something that seemed like a good idea and did not catch on, or it is a pattern that will propagate to MSTest eventually?

We had to take some shortcuts and so sadly we didn't test code as extensively as we wanted. The main idea of this helper is for us to be able to better test some race conditions as we could control the task.
I would prefer to keep it there for now and revisit dropping it if we didn't manage to invest into some of the tests in the upcoming period.


Add isAdvertised for each listener

Before #85 the only listeners that were advertised were the first in the list. After that PR all listeners are advertised. We need to allow the user to set an isAdvertised boolean for each listener that determines whether a listener gets advertised. This will be an additional prop to the listeners in values.yaml, and a change the advertise loops in statefulset.yaml.

I think the change mentioned in this ticket is not need with how listener advertisement currently works as of this PR: https://github.com/redpanda-data/helm-charts/pull/121
All listeners will be advertised according to whether they are internal or external, and will use the internal or external host name for the advertised value. Will re-open if having non-advertised listeners turns out to be a required feature.


feat(v0.25.3): bump for upstream release flux2 to v0.25.3

Signed-off-by: haarchri<EMAIL_ADDRESS>
What this PR does / why we need it:
update to flux2 upstream v0.25.3 release
https://github.com/fluxcd/flux2/releases/tag/v0.25.3
Special notes for your reviewer:
This will be the last version where flux2 upstream support Kubernetes 1.19 and older. Next release will drop all EOL versions and it will come with a restricted pod security profile.
Checklist

[x] DCO signed
[x] Chart Version bumped
[x] helm-docs are updated
[x] Helm chart is tested
[x] Run make reviewable

LGTM


'RCTImage/RCTImageUtils.h' can not found when archiving release package, but it's ok when debugging.

Problems
I found the 'RCTImage' directory not exist in the Xcode cache directory '~/Library/Developer/Xcode/DerivedData/zaiqiuchang-fwykrwmotbpozgegbmtywbczscln/Build/Products/Debug-iphonesimulator/include', it needs to be created manually and copy 'RCTImageUtils.h' from RN source to there when debugging. But I don't know where to modify when archiving release package.
It can be build when removed 'RCTImage/RCTImageUtils.h' import in RCTWeChat.m, and can share to WeChat. So why import this file?

Env
react-native: 0.40.0
react-native-wechat: newest from GitHub
platform: iOS 10.2, Xcode 8.2.1

I have the same issue. My dev environment:
react-native: 0.42-rc.3
iOS, Xcode 8.2.1

Just remove 'RCTImage/RCTImageUtils.h' to fix.

Did you fixed this problem ?

fixed


multiple word support and attachment support

Since this pr has breaking changes I suggest merging it for v2.
Changes

Change options.json structure (Breaking!!)

Features

Supports attachments.
Supports multiple words including the ability to escape quotes.

Fixes

Being able to change prefix and roleID with edit command or even removing them with remove

lol v2 already xD


typo: Double word "of"

@nschonni : Thanks for your contribution! The author, @mattwojo, has been notified to review your proposed change.


Support tilde expansion for local dependencies on DescriptionV4_2

Problem
According to the documentation, local dependencies are available on SPM DescriptionV4_2.
However, we can't declare paths which start with ~.
// swift-tools-version:4.2
// The swift-tools-version declares the minimum version of Swift required to build this package.

import PackageDescription

let package = Package(
    name: "myexecutable",
    dependencies: [
        // Dependencies declare other packages that this package depends on.
        .package(path: "~/work/MyPackage"),
    ],
    targets: [
        // Targets are the basic building blocks of a package. A target can define a module or a test suite.
        // Targets can depend on other targets in this package, and on products in packages which this package depends on.
        .target(
            name: "myexecutable",
            dependencies: ["MyPackage"]),
    ]
)

$ swift build --disable-package-manifest-caching
error: /Users/giginet/work/myexecutable/~/work/MyPackage has no manifest
'myexecutable' /Users/giginet/work/myexecutable: error: product dependency 'MyPackage' not found

We want SPM to expand ~ as a home directory.
Description
This PR make Package Description v4.2 to support tilde expansion.

@aciidb0mb3r Hi! I fixed your reviewed points.
Can you review again?

@aciidb0mb3r Fix it!

@swift-ci smoke test


Refactor PRIVACY and TERMS

Both documents refer to GoombaLabs, where they should refer to Goober.Cloud
If there's contact information, ensure it says matt[at]goober[dot]cloud
Thanks!

An error occurred: Error code: 400 - {'error': {'message': "Expected tool outputs for call_ids ['call_sXXxv0JqXNWvlKcddyL9DgGm', 'call_PbSe3x68McDLmcE2yFIKI3Sy'], got ['call_sXXxv0JqXNWvlKcddyL9DgGm']", 'type': 'invalid_request_error', 'param': None, 'code': None}}

i would appreciate the help!

please help with this!

An error occurred: Error code: 400 - {'error': {'message': "Expected tool outputs for call_ids ['call_Hea7DSpMVhG0BANoVyA1rvjg', 'call_96CwayHLY0d3QQL4BaY7EJao'], got ['call_Hea7DSpMVhG0BANoVyA1rvjg']", 'type': 'invalid_request_error', 'param': None, 'code': None}}

pls!

An error occurred: Error code: 400 - {'error': {'message': "Expected tool outputs for call_ids ['call_KwRVoJy8s8rgxkbIkuBQydaB', 'call_CHcDrXPzGCpvakMwcVE79G8f'], got ['call_KwRVoJy8s8rgxkbIkuBQydaB']", 'type': 'invalid_request_error', 'param': None, 'code': None}}

:D

An error occurred: Error code: 400 - {'error': {'message': "Expected tool outputs for call_ids ['call_CEsD0wyIe3osaQGhcuoKefnK', 'call_7mIhZuSoSeRFGecYckpK4JE7'], got ['call_CEsD0wyIe3osaQGhcuoKefnK']", 'type': 'invalid_request_error', 'param': None, 'code': None}}

help!

An error occurred: Error code: 400 - {'error': {'message': "Expected tool outputs for call_ids ['call_IB5wtWnobmsSEoovZTlOH9V0', 'call_93TAk4eIuLIgo4xqENBhAahq'], got ['call_IB5wtWnobmsSEoovZTlOH9V0']", 'type': 'invalid_request_error', 'param': None, 'code': None}}

please take a look and update these files! :)

please have a look at these files

please!

pls

can you please have a look at this issue

please!

pls

pls

help

please fix!

pls

No messages found in the run.

pls!

help

It looks like there are no PDF or DOC files in the repository either. Given that we've searched for the terms "GoombaLabs," "PRIVACY," and "TERMS" without success, it's possible that the documentation may not be included in this repository at all or is stored in a different format or location.
Here's what I can do next:

Confirm with you if you have any specific files or directories in mind where these privacy and terms documents may be located.
If you have documentation in another repository or location, let me know so I can assist in making the necessary changes based on that content.

Please provide any additional information that can help locate the desired files!

The files are literally named PRIVACY and TERMS, and should be in the base directory of this repository

Done!


"Cache already exists" despite "Cache not found"

I've observed the "Post Mount" step state
Warning: Cache already exists. Scope: refs/heads/bump-cache, Key: repo-cache-macOS-nixpkgs-1, Version: (null)

while the "Mount" step states
Cache not found for input keys: repo-cache-macOS-nixpkgs-1.

The cache is configured as follows
      - name: Mount Bazel cache
        uses: actions/cache@v1
        with:
          path: ~/repo-cache
          key: repo-cache-${{ runner.os }}-nixpkgs-${{ env.cache-version }}

with
env:
  # Bump this number to invalidate the GH actions cache
  cache-version: 1

This is the third run on this branch and commit. (Unfortunately, I don't know how to access the logs of the first two runs anymore.)

The first run failed with an error in the build step (rate limit on GH API).
The second step succeeded but the cache upload step failed with 503, possibly https://github.com/actions/cache/issues/259.
The third step succeeded but cache download and upload behave inconsistently as described above.

@aherrmann If you are able, I would recommend updating to actions/cache@v2.  All you need to do is replace @v1 with @v2 in your workflow file.  We don't provide any updates to @v1 and there are speed and reliability improvements that should help reduce upload failures.

Thanks, I wasn't aware of v2. I'll try and report if the issue occurs again.

Thank you!  I'm going to close this issue.  Please re-open of create a new one if you see the issue again.


Collection of minor improvements to the code

Working on fixing then numerous bugs we found in production, ended up with a bunch of simple commits.

Should be fixed now.

Hum, shall we drop support for 10.x and 12.x ?


EventLoopScheduler: unexpected Exception after Dispose

When calling Dispose on an EventLoopScheduler, that has at least one item in its working queue, it will throw an ObjectDisposedException. The exception is thrown from its worker thread (which gives us no chance to catch it).
Code to reproduce the problem:
class Program
{
    public static void Main() {
        var event_stream = new Subject<string>();

        var subscription = Observable.Using(
            resourceFactory:    () => new EventLoopScheduler(),
            observableFactory:  scheduler => event_stream
                .ObserveOn(scheduler)
                .Do(LongRunningAction))
            .Subscribe();

        event_stream.OnNext("event 1");

        Thread.Sleep(TimeSpan.FromSeconds(1));
        subscription.Dispose(); // Scheduler is still busy!

        Console.ReadLine();
    }
    private static void LongRunningAction(string event_text) {
        Thread.Sleep(TimeSpan.FromSeconds(2));
        Console.WriteLine(event_text);
    }
}

I've opened a question at stackoverflow (containing a work-around) some time ago. The workaround does not seem to catch all possible cases - I still get this (unhandled) exception from time to time:
System.ObjectDisposedException: Cannot access a disposed object.
   at System.Reactive.Concurrency.EventLoopScheduler.Schedule[TState](TState state, TimeSpan dueTime, Func`3 action)
   at System.Reactive.Concurrency.LocalScheduler.Schedule[TState](TState state, Func`3 action)
   at System.Reactive.Concurrency.Scheduler.<>c__DisplayClass50`1.<InvokeRec1>b__4e(TState state2)
   at System.Reactive.ScheduledObserver`1.Run(Object state, Action`1 recurse)
   at System.Reactive.Concurrency.Scheduler.<>c__DisplayClass50`1.<InvokeRec1>b__4d(TState state1)
   at System.Reactive.Concurrency.Scheduler.<>c__DisplayClass50`1.<>c__DisplayClass52.<InvokeRec1>b__4f(IScheduler scheduler1, TState state3)
   at System.Reactive.Concurrency.ScheduledItem`2.InvokeCore()
   at System.Reactive.Concurrency.EventLoopScheduler.Run()
   at System.Threading.ThreadHelper.ThreadStart_Context(Object state)
   at System.Threading.ExecutionContext.RunInternal(ExecutionContext executionContext, ContextCallback callback, Object state, Boolean preserveSyncCtx)
   at System.Threading.ExecutionContext.Run(ExecutionContext executionContext, ContextCallback callback, Object state, Boolean preserveSyncCtx)
   at System.Threading.ExecutionContext.Run(ExecutionContext executionContext, ContextCallback callback, Object state)
   at System.Threading.ThreadHelper.ThreadStart()

Hi,
Suffering the same issue, for both personnal and pro (Trading application) projects.
Even after reading the Stack link provided above, I also wish it could be fixed.
Exposing a Dispose interface for such things as an EvenLoppScheduler makes sense.
No matter if the underlying thread is "supposed" to be "suspended", Dispose Method is available,
and knowing it can litteraly blow you code when you "simply" want to use it sounds weird ...
If this fix, or another, is not accepted, could you please provide an "official" way to avoid this ?
Even if I use ELS on specific places, shared for specific dispatching purpose, I know the need to properly
dispose everything - including underlying thread - can occur, especially on an application running almost 24 / hours a day.... We really would appreciate some feedbacks on this one.
Thanks for all you great work.

Hi @HebuSan,
Unfortunately it did not work in all situations. The exception was very hard to reproduce and we did no further investigation. Our projects currently use a modified EventLoopScheduler (commit 4c0f9bf).
To avoid wrong usage, we've implemented an service interface (injected):
public interface ISchedulerService {
        ... snip ...

        /// <summary>
        /// Constructs an observable sequence that depends on a <see cref="EventLoopScheduler"/>, whose lifetime is tied to the resulting observable sequence's lifetime.
        /// </summary>
        /// <typeparam name="T">The type of the elements in the produced sequence.</typeparam>
        /// <param name="observable_factory"></param>
        /// <exception cref="T:System.ArgumentNullException"><paramref name="observable_factory"/> is null.</exception>
        /// <returns>An observable sequence whose lifetime controls the lifetime of the dependent resource object.</returns>
        IObservable<T> UsingEventLoop<T>(Func<IScheduler, IObservable<T>> observable_factory);
}

Usage:
var externObservable = CreateAnObservable(..);

subscription = scheduler_service
                .UsingEventLoop(
                    scheduler => externObservable
                        .ObserveOn(scheduler)
			.Do(something))
		.Do(somethingElse)
                .Subscribe();

I can post the code on GitHub if needed for reference.

This issue here is that you end up with a race condition because you are creating the EventLoopScheduler in an Observable.Using. The thing with this is that I don't see the point.  A single observable more or less does this anyway. There's little need to use an EventLoopScheduler.
Can you explain why you're trying to use it like this?

We start & stop a bunch of device drivers (each running in its own thread). All driver output must pass ONE output stream and therefore we have to ensure serial execution.

Hi @Enigmativity,
We start & stop a bunch of device drivers (each in its own thread) that continuously emit events. The output requires us to ensure a serial execution. The drivers MUST NOT be blocked because of further processing  (back-pressure). The EventLoopScheduler helps us to decouple the post-processing from event generation.
Using this example
var outputPipe = driver1.Events
                .Merge(driver2.Events)
                .Merge(driverN.Events)
                .Select(Transform)
                .Do(..);

..it would block each driver during the execution of Transform(..).
The simple solution was to queue the events:
var outputPipe = driver1.Events
                .Merge(driver2.Events)
                .Merge(driverN.Events)
                .ObserveOn(eventLoopScheduler)
                .Select(Transform)
                .Do(..);

Do you think we can achieve the same without EventLoopScheduler?
Maybe our solution is not in an optimal way. Anyway, I think the current behavior of EventLoopScheduler.Dispose() is unexpected and unusable (see the numerous questions at stackoverflow and so on).

You should find that both queries you've just shown me serialize the calls to Transform - that's part of the contract with Rx. Should you have a "rogue" query that breaks the contract then use the .Synchronize() operator to make it serialize.
You should only need to use an EventLoopScheduler if you have multiple queries that run independently and concurrently that you want to serialize - and then you'd share a single instance with multiple observables. However, your query shows that you're creating an EventLoopScheduler for a single query so that shouldn't be the case.
If you do need to stick with an EventLoopScheduler for whatever reason then it's a simple matter of creating the EventLoopScheduler inside a .Create and dispose of it like this:
var query = Observable.Create<int>(o =>
{
	var els = new EventLoopScheduler();
	return
		Observable
			.Return(42)
			.Finally(() => els.Schedule(() => els.Dispose()))
			.Subscribe(o);
});

query.Subscribe();

Hi @Enigmativity,
thanks for your answer.
Maybe I forgot to explain that all driver instances are running independently and MUST NOT block on Observable.OnNext. That was the reason for us to use EventLoopScheduler. In case there are too many events (for a short period of time), it just adds the events into a queue and schedules them for later processing. This approach avoids back-pressure (of the slow Transform call) up to the driver instances.
Sample code to explain our scenario:
https://gist.github.com/danielmue/67f070fe1e7f26fb8f670482e0997125
(please uncomment //.ObserveOn(new EventLoopScheduler()) to see the difference)
Your proposal works and is similar to the already described workaround.
Although it is really interesting to discuss different solutions - Initially I did not open the issue to find out proper usage of EventLoopScheduler :-). The issue is there because users currently get an unhandled exception 💥 after calling Dispose(). Since it implements IDisposable I would expect it to stop the queue/scheduling and cleanup resources.

Hi @Enigmativity,
I also suffered similar issues and discovered the only way to solve it was to schedule the Dispose on the scheduler.
Using .Finally is really nice (and way easier to read). My workaround was a bit "dirtier" (Wrapping it using Dispose.Create), making the fix less "obvious" to someone reading the code.
If "it makes code work", like @danielmue, it sounds strange to me that somehow you "have to figure out how to make the proper call to the IDisposable interface".
And to know that calling it in a different way (without scheduling it) makes everything crash.
To me, IDisposable interfaces should be working the same way all the time.
I could not test @danielmue fix, but if it works, why not accepting it ?

@HebuSan - When you say "To me, IDisposable interfaces should be working the same way all the time." you need to understand that when working with multiple threads and concurrency that things that otherwise should work can fail. Even value++ can fail with using multiple threads.
There are many times that you have to write your code the right way to get it to work in certain circumstances. This is no different.

@Enigmativity,
Thx for answering.
I don't think you value++ is a good exemple at all.  Comparing such a low level method, known to be defensless against multithreading issues (Interlocked.Increment can do the job), with an interface implementation sounds strange to me.
I'll try to explain it more clearly, sorry for my bad english.
Most (all in fact) of the ES logic behin the IScheduler interface is "protected" against threads.
There are gates a bit everywhere, that is perfectly normal. Nowhere it is said that developpers should handle multithreaded matters of the IScheduler implementation of ES "on their side" of the code. No. This is done internally.
When you schedule an Action with ES, you can peacefully call it knowing the implementation of ES class for this IScheduler interface will handle race conditions.
And more, the logic behind IDisposable implementation of the ES also uses the same gate, and the IDisposable logic of the ES already interacts with the IScheduler implementation of the ES. (_disposed is used a bit everywhere. So for me it tends to show there is a (normal) care for the ES implementation to handle all its logic (both froml IScheduler and IDisposable interface) in a consistent way, no matter how many threads are involved.
The fact @danielmue could fix this so "easyly" (No offense, I mean by just by patching internally, without modifying interfaces / adding weird code) shows it can be done, and it could just be a matter of forgotten use case.
I say could because the exception throwing was there intentionaly. So IF this is done willingly, there indeed is something I don't understand. But if so, to me it is not a "simple" multi threading issue, as (IF i understand) you suggest with the value++ case.
I admit my english may not be good enough, and saying ""To me, IDisposable interfaces should be working the same way all the time."  without further explanation may sound strange.
Knowing IDisposable interfaces can be used in many way with RX (within CompositeDisposable, ....) I really think it is saffer to ensure such matters are handled on implementation side of the interface, rather than having to remember that this specific implementation of IDisposable can blow my code.
I hope this (longer) answer can help either validating the PR, or at least getting a clear explanation of why this was done this way.
The stack link given above tends to proove it could be good having either this PR validated (to confirm this is a bug) or a clear answer regarding the IDisposable usage of the ES.
Because even with people like Lee Campbell in the thread, no clear answer came from the discussion (or at least none I could clearly understand).

Maybe there are different ideas about the meaning of different objects / methods.
For me:
What is the purpose of a IScheduler?
Scheduler an action in a specific way. In case of EventloopScheduler this means, shift the action to a specific thread and processes them one by one in the order they arrived.
What the purpose of Dispose-Method?
Stop what every you are doing and clean up your resources. When I call Dispose on a RX-Stream, I expect the stream to stop immediately, When I call Dispose on a ES, I expect the loop to break immediately (or maybe right after ES finished working on the current item). But I never expect a Dispose-method the throw an Exception, because that behavior is unspecified. What's the meaning?

Call me again later?
You have the shutdown the app, because there might be some memoryleak?

Maybe @Enigmativity can tell me, if any of my expectations is wrong?

@BADF00D - You're thinking about this too simplistically. This isn't a single thread running here. It's multi-threading and multi-threading is hard. I think this is a case where the coding of observable and schedulers don't work unless you code it correctly.
This is no different to knowing when to dispose a database connection if you've got an IQueryable<T> that has yet to execute. You just have to know the right way to do things.

I've got bitten by this unexpected behavior as well - lots of things were not getting disposed because EventLoopScheduler was blowing up.
My minimal repro:
using (var scheduler = new EventLoopScheduler())
using (var subscription = Observable.Range(1, 10).ObserveOn(scheduler).Subscribe(x =>
{
	Thread.Sleep(500);
	Console.WriteLine(x);
}))
{
	Thread.Sleep(100);
}

Thx !


Unable to launch ROMs when built from source on OpenSUSE distro

As of commit 5f626ce34f1f3526b658d26d97a5a25d774a13f0, whenever a ROM is launched, all I see is either a blankly rendered X window or an X window with whatever was loaded behind it frozen into the window. For example:

Cen64 when launched on it's own through the terminal runs ROM files just fine:

If I terminate emulation early through Cen64-qt I recieve the following log output:
/home/rob/Emulators/cen64/cen64 -eep4k "/home/rob/Emulators/ROMs/N64/N64 Saves/Super Mario 64 (U) [!].20b854b239203baf6c961b850a4a51a2.eep4k" -eep16k "/home/rob/Emulators/ROMs/N64/N64 Saves/Super Mario 64 (U) [!].20b854b239203baf6c961b850a4a51a2.eep16k" -sram "/home/rob/Emulators/ROMs/N64/N64 Saves/Super Mario 64 (U) [!].20b854b239203baf6c961b850a4a51a2.sram" -flash "/home/rob/Emulators/ROMs/N64/N64 Saves/Super Mario 64 (U) [!].20b854b239203baf6c961b850a4a51a2.flash" -controller num=1 "/home/rob/Emulators/BIOS/N64 BIOS/pifdata.bin" "/home/rob/Emulators/ROMs/N64/Super Mario 64 (U) [!].z64"
I'm running OpenSUSE Tumbleweed and would love to help debug why the ROMs fail to load.

Did it work fine for you before that commit? That commit you referenced only changed the directory name of the temp directory where zipped ROMs are unzipped to, so I don't see how that commit would cause the regression you're describing since you're not using a zipped file.
I'll need to do some testing to see if I can reproduce what you're seeing.

I apologize. I should not have referenced the commit. I merely included it to show the build I made for historical purposes. I had not used cen64-qt prior to this commit, so I am not accusing a commit that changed a directory name of a temporary directory for causing this issue.
I needed to reinstall my distro, so I had to recompile both cen64 and cen64-qt. The issue still remains, sadly. I will try going backwards to each commit to see where the issue could lie, however, the issue may be with OpenSUSE compatibility in general.

OpenSUSE is no longer my daily driver, closing.


v1.1.0 - Patch spark/pyspark kernels to use correct python binary

Issue #, if available:
Description of changes: Adds bug fix to dockerfile of v1.1 release "Patch spark/pyspark kernels to use correct python"binary
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Covered by https://github.com/aws/sagemaker-distribution/pull/127


Enabling horizontal scrolling displays error

I followed the setup instructions, including enabling horizontal scrolling. Playnite displays a message asking the user to disable horizontal scrolling.

Edit: disabling horizontal scrolling works fine, so it's the documentation that is out of date. Installation instructions should not tell the user to copy a screenshot with "Horizontal Scrolling" enabled.


Add GCS Client Logging

PR for updating logging in the GCS Client class.
Refer to issue: https://github.com/elementary-data/elementary/issues/330

Thanks for opening this PR @seanglynn-thrive ! please let me know if you have questions about any of the comments

Thanks for opening this PR @seanglynn-thrive ! please let me know if you have questions about any of the comments

@oravi thank you for the quick review on this. I've made the requested changes in the latest commit.

@seanglynn-thrive approved the PR! we have a small issue in the CI which we will fix soon / tomorrow. Hopefully we will be able to deploy it tomorrow as well!


Alexa cannot set to 1 w/o brightness word

SUMMARY

If I ask Alexa, "module name" to one, Alexa does not know how to do, but
two to hundred works w/o any issues.
If I ask Alexa, set brightness of "node name" to one it's working too.
ECHO DEVICES

Echo Dot 1st generation
Echo Plus 2nd generation
Echo Spot
MODULE VERSION

0.1.9

CONFIGURATION

CONSOLE OUTPUT

n/a

STEPS TO REPRODUCE

Ask Alexa set to one - it's not working.
On German: Alexa, "name of the node" auf 1 -> failed
Alexa, "name of the node" auf 2 (or 3 to 100) -> worked
Alexa, setze Helligkeit von "name of the node" auf 1 -> worked too
EXPECTED RESULTS

Should work with "one" too
ACTUAL RESULTS

see above

I don't have control over this, as Alexa is deciding what to send to the hub, but just tested on my Dot v3, and it works.


Fix for onEndReached not getting called

Description
This PR focuses on fixing onEndReached not getting called. The reason why it sometimes doesn't get called is because of this particular line
layoutMeasurement.height + contentOffset.y >= contentSize.height - paddingToBottom
Where both sides sometimes evaluate to a 6 decimal place number and because of three different measurements (height,offset, total height), comparing them introduces inaccuracy. For example if the 6th decimal point is 0 on left hand side and its 1 on right hand side. Then even if we are at the end, this function will not get called because the fore mentioned line will evaluate to false. To fix this, I have removed the decimal points from the left hand side by using Math.Ceil. This will call the function by removing uncertainties of the decimal points.
Related Issues
N/A
Tests
N/A
Checklist
Before you create this PR confirms that it meets all requirements listed below by checking the relevant checkboxes ([x]). This will ensure a smooth and quick review process.

[✅ ] I read the Contributor Guide and followed the process outlined there for submitting PRs.
[ ✅ ] Run yarn lint && yarn tsc
[ ] Run yarn test or yarn test -u if you need to update snapshot.
[ ✅ ] I am willing to follow-up on review comments in a timely manner.

which version is it??
i use 1.4.1 and still have an issue in android, sometimes onEndReached not called.

@hyochan onEndReached is not called at all in any condition. Please check.


DBCP-592: Support request boundaries

JDBC 4.3 introduced two new methods, beginRequest and endRequest, which connection pools can use to mark request boundaries.
According to JavaDoc, these methods signal to the driver that a connection is active, the driver's response is specific to the vendor and it may not do anything.
Goal is to make DBCP invoke beginRequest when a connection is borrowed and endRequest when the connection is returned to the pool.

Hello @meedbek

How will this not clash for apps that call commit() and rollback()?
Since we do not call commit() and rollback() on behalf of call sites, why should we call these two APIs?

APIs?

Hi @garydgregory,

Based on the JavaDoc if a request was ended then endRequest is a no-op so calling it should not clash for apps that call commit or rollback.
These two methods do not begin or end transactions, they are just a hint to the driver that a request has began or ended. Using them can improve the driver performance and the user experience. Though if there is an active transaction when the connection is released, calling endRequest  may throw an exception depending on the implementation.

I think adding these calls would be a bad idea.  The spec is vague and at least some implementations do things that DBCP already does in activation/passivation and may cause problems for some applications.  See, e.g https://learn.microsoft.com/en-us/sql/connect/jdbc/jdbc-4-3-compliance-for-the-jdbc-driver?view=sql-server-2017.    The strange wording in the spec may also mean that DBCP should not send the calls, as we do work with connection handles.

I agree with @psteitz, this feels like a won't do for now. It's not clear to me there's a site to place these calls for a generic library. This feels more like an app-level call, not something called from the guts of DBCP.

They can be very powerful, not only can they reset internal state with minimum round-trips but also ether are used for things like licenses check and reconnect in some drivers. So it would be good if it optionally can be switched on. But yes I agree it requires in depth Integration tests and also maybe changes. So far that maybe it needs a whole own connection pool logic in regards to caching.

Thanks for weighing in, Bernd.  Do you by chance have some links to what drivers are actually doing here?  I looked at Postgres and Mysql and could not find any implementations.  I would like to understand better what the drivers actually do.  The spec makes it look like, as Gary says, more an app concern, except they expect pool managers to send the messages which does not make sense to me.  I must be missing something.  When you say "caching" I am not sure what you mean.  The whole pool is essentially a cache.
From what I have gleaned so far, it seems to me that the better place to put these calls might be in activate and passivate.  That led me to think that allowing PoolableConnectionFactory to be pluggable or the lambdas for the lifecycle methods would allow users to set this up however they wanted.

I was curious on those begin/end but could not find much of documentation. So I snooped a bit around in the source code of some drivers. Besides the resetting of the client side state (the typical fields inside a SQL Connection object) there is also the possibility of server side session state resets, where the drivers would only send demarcation messages. I think mssql doesnt do that (but it has a reset flag in the protocol). In the MSSQL-JDBC driver I found some code to reconnect broken connection. I thought it was linked to the request state, but cant find it anymore.
I think I have also seen some logic for different timeout handlings of sessions outside the request block and associated diagnostics in oracle documents (but cant find it again at the moment).
So - it might get promising for some drivers in the future to use it. Maybe an app listener hook can also use it, but then the pool would still do the reset/aliveness dance which it could skip for qualified drivers.
MSSQL Resets:
https://github.com/microsoft/mssql-jdbc/blob/bb76a7832ecf1cd24238c75cf7c2c4706d922435/src/main/java/com/microsoft/sqlserver/jdbc/SQLServerConnection.java#L7561
Session recovery
https://github.com/microsoft/mssql-jdbc/blob/bb76a7832ecf1cd24238c75cf7c2c4706d922435/src/main/java/com/microsoft/sqlserver/jdbc/SQLServerConnection.java#L4327
https://github.com/microsoft/mssql-jdbc/blob/bb76a7832ecf1cd24238c75cf7c2c4706d922435/src/main/java/com/microsoft/sqlserver/jdbc/IdleConnectionResiliency.java#L16

All of this hints to me that it is too early for us to integrate these calls into DBCP. Once more FOSS drivers implement these APIs in a meaningful manner will be able to create tests to make sure that (1) we do not break existing clients and (2) properly use the APIs.
WDYT?

Oh, also when we do decide to use the APIs we should NOT use reflection. We can either bump the platform requirement or create a new Maven module with that requirement in a subclass or as some kind of plugin/hook.

Yes its early, maybe wait for concrete requests with a desireable driver behavior.
Having said that, calling endRequest to reset the vendor specific connection attributes seems like a win for us (i.e. we dont have to track them).

Here is btw oracles doc for they replay driver, it uses the request boundaries in this way: https://docs.oracle.com/en/database/oracle/oracle-database/23/jjdbc/application-continuity.html#GUID-AAC6F9B7-9B4C-4098-B0D5-312BF9A13928

With JDBC 4.2, the driver doesn't know when a connection is borrowed from connection cache or when a connection is returned to the cache. It is, however, a very useful piece of information. For example, when a connection is returned, the driver may decide to flush temporary resources or let the database know so that it can compute statistics, or even replace the connection with another one during a planned maintenance event of the Database. When a connection is borrowed from the cache, the driver may start recording calls in its replay queue in case of an unplanned outage so that it can transparently replay a request. This is the reason why these APIs beginRequest and endRequest were added to the JDBC 4.3 specification.
There are endless possibilities.
But this relies on connection pools such as DBCP to call these APIs. The features that rely on request boundaries can only be activated with connection pools that make these calls. This is why we (at Oracle) think that this is a very desirable feature to add to DBCP.