unable to add accessories

Running Ubuntu 16.04, node 6.1.0, avahi 0.6.32~rc+dfsg-1ubuntu2 and sources from GitHub.  iOS 10 on iPhone 6.  Using "Home" and Apple's Home App.
Running DEBUG=* node Core.js and trying to pair yields the following log messages on the
  EventedHTTPServer Server listening on port 51826 +0ms
  EventedHTTPServer Server listening on port 51827 +7ms
  EventedHTTPServer Server listening on port 51828 +16ms
  EventedHTTPServer Server listening on port 51829 +3ms
  EventedHTTPServer Server listening on port 51830 +2ms
  EventedHTTPServer Server listening on port 51831 +3ms
  EventedHTTPServer Server listening on port 51832 +2ms
  EventedHTTPServer [::ffff:<IP_ADDRESS>] New connection from client +12s
  EventedHTTPServer [::ffff:<IP_ADDRESS>] HTTP server listening on port 42904 +1ms
  EventedHTTPServer [::ffff:<IP_ADDRESS>] HTTP request: /pair-setup +9ms
  HAPServer [1A:2B:3C:4D:5D:FF] HAP Request: POST /pair-setup +1ms
  HAPServer [1A:2B:3C:4D:5D:FF] Pair step 1/5 +3ms
  EventedHTTPServer [::ffff:<IP_ADDRESS>] HTTP Response is finished +146ms
  EventedHTTPServer [::ffff:<IP_ADDRESS>] Client connection closed +204ms
  EventedHTTPServer [::ffff:<IP_ADDRESS>] HTTP connection was closed +4ms
  EventedHTTPServer [::ffff:<IP_ADDRESS>] HTTP server was closed +0ms

On the iPhone I see:
"Add Homekit Accessory"
"This accessory is not certified..."
and then after I continue:
"Error"
"Accessory is not reachable"
I've tried removing the home and re-adding to no avail.
Same result on any of the accessories.
This all used to work prior to iOS updates.

Anyone can help?


Move through the search result with ctrl+j/k

Hi! Thank you very much for your work. Your plugin makes it a lot easier to work without using the mouse. I would like to suggest adding a toggle by search results via the ctrl+j/k keys for vim mod users. That would be incredibly convenient.

@tonygrr
Thank you for your request!
In release 0.6.0, you can customize hotkeys.
Could you manually install and try it?


TD-3676 Updates branding and links in the OpenAPI project to NHS England

JIRA link
TD-3676
Description
Updates references to Health Education England in the OpenAPI project to reflect move to NHS England. When we shared the Swagger document (https://uks-learninghubnhsuk-openapi-dev.azurewebsites.net/swagger/index.html) with NHS Staff App colleagues, it was noticed that the branding in the descriptions was still HEE.

Developer checks
(Leave tasks unticked if they haven't been appropriate for your ticket.)
I have:

[x] Run the formatter and made sure there are no IDE errors
[ ] Written appropriate unit tests for the changes, including:

accessibility tests for new views
tests for new controller methods
tests for new or modified API endpoints

[ ] Manually tested my work with and without JavaScript
[ ] Tested any Views or partials created or changed with Wave Chrome plugin and addressed any valid accessibility issues
[ ] Updated/added documentation in Confluence and/or GitHub Readme. List of documentation links added/changed:

doc_1_here

[ ] Updated my Jira ticket with information about other parts of the system that were touched as part of the MR and have to be sanity tested to ensure nothing is broken
[x] Scanned over my pull request in GitHub and addressed any warnings from the GitHub Build and Test checks.

can see some HEE  references in   services Interfaces and in Helper Class  like BookmarkService.cs . is that will be ok?

@AnjuJose011, just to note, the change to StyleCop.json (solution wide) probably means that lots of other copyright statements are now breaking the rule. Should I go through and update all of them in this PR if so?

@kevwhitt-hee    we can do in this PR as it is throwing warnings while building the solution . entirely we can replace  HEE to NHS

I've added a commit to address this, @AnjuJose011. @ColinBeebyHEE, starting to have doubts about whether we should have a copyright notice at the top of all of our classes given that we are open-source. Should we be removing them? I'll ask Google too.

I've added a commit to address this, @AnjuJose011. @ColinBeebyHEE, starting to have doubts about whether we should have a copyright notice at the top of all of our classes given that we are open-source. Should we be removing them? I'll ask Google too.

Google seems to think we shouldn't be copyrighting code in an open source platform. Should we remove all of them? What do you both think?


Adding Google Tag Manager Snipper to  and   tage

Is there a way to add Google Tag manager  code to  and  ?

@medicareamaze right now you have to customize the Volto component that defines the React helmet config. See kitconcept.seo for an example:
https://github.com/kitconcept/kitconcept.seo
Though, we are already thinking about a plugin architecture that allows plugins and integrators to inject code into the header.

fyi tagmanager is not only about the snippet in the header but also about being able to add data layers  which are additional snippets within the body.
For example https://developers.google.com/tag-manager/enhanced-ecommerce
We did an implementation for "old style" Plone in https://github.com/bluedynamics/bda.plone.gtm ;-)

... overread the second line of the title ... it's still too early 🙈

@medicareamaze right now you have to customize the Volto component that defines the React helmet config. See kitconcept.seo for an example:
https://github.com/kitconcept/kitconcept.seo#add-to-volto-site
Though, we are already thinking about a plugin architecture that allows plugins and integrators to inject code into the header.

Thanks Tisto! How and where in Volto/Plone, do I add the actual Javascript tag? Like the following?
<!-- Global site tag (gtag.js) - Google Analytics -->
<script async src="https://www.googletagmanager.com/gtag/js?id=UA-11843XXXX-X"></script>
<script>
  window.dataLayer = window.dataLayer || [];
  function gtag(){dataLayer.push(arguments);}
  gtag('js', new Date());

  gtag('config', 'UA-118437943-1');
</script>

... overread the second line of the title ... it's still too early 🙈

Thanks Agitator. Right Now I'm Just trying to get my site Volto Report back site visits to Google Analytics. In Site is driven by a bot and I am managing the event logging withing the app - https://www.medicareamaze.com

@medicareamaze right now you have to customize the Volto component that defines the React helmet config. See kitconcept.seo for an example:
https://github.com/kitconcept/kitconcept.seo#add-to-volto-site
Though, we are already thinking about a plugin architecture that allows plugins and integrators to inject code into the header.

Thanks @tisto How and where in Volto/Plone, do I add the actual Javascript tag? Like the following?
<!-- Global site tag (gtag.js) - Google Analytics -->
<script async src="https://www.googletagmanager.com/gtag/js?id=UA-11843XXXX-X"></script>
<script>
  window.dataLayer = window.dataLayer || [];
  function gtag(){dataLayer.push(arguments);}
  gtag('js', new Date());

  gtag('config', 'UA-11843XXXX');
</script>

I figured this out. I site visits are now hitting Google analytics and  I also see the Meta Description Appear in the . Thanks! For those of you who have a problem with this - This ishow I did ita nd it works
`
{/* other stuff... */}
  {/* Global site tag (gtag.js) - Google Analytics  */}
  <script async src=`https://www.googletagmanager.com/gtag/js?id=${googleAnalyticsId}`></script>
  <script>
    {`
      window.dataLayer = window.dataLayer || [];
      function gtag(){dataLayer.push(arguments);}
      gtag('js', new Date());
      gtag('config', ${googleAnalyticsId});
    `}
  </script>

`


Fixed 0 values defaulting to being user editable

Added link generator
Added doc blocks
Updated quantity field

store now calls correctly to website

Resolves #87

it would, but i found a better way to fix the fc_hash_value method from thinking "--OPEN--" is equal to 0 or "" in https://github.com/FoxyCart/FoxyCart-Cart-Validation--PHP/pull/16


Support concurrent execution API

executeConcurrent should be supported and optimised for efficiency.

Works in #40, most of the integration tests enabled in #95


Test/travis

Fixes #55
Use trusty version to get newer browsers.

Note for the future:
fix() is probably the wrong category for this even though it is technically a fix, it's not related to the code functionality. I would recommend to use build() instead when touching the build process and project structure :)


Exchange Server 2016 (Build 2106.2) not identified as local Exchange Server

The script does not identify an Exchange Server 2016 (new CU18 install) as a local Exchange Server when executed in a non-EMS-PowerShell session.
HKLM:\SOFTWARE\Microsoft\ExchangeServer\v15\Setup is missing

The Exchange Server setup completed successfully and the server is fully operational. So I do not know if this is a setup anomaly.
The current release works fine when started in an EMS session.

Thanks for reporting this @Apoc70 . @dpaulson45 I'm not sure if this is by design or an issue. We could check for HKLM:\SOFTWARE\Microsoft\ExchangeServer\v15\MailboxRole or HKLM:\SOFTWARE\Microsoft\ExchangeServer\v15\ClientAccessRole. One of these should be available. What do you think?

@Apoc70 i would like to review the debug log of this scenario to understand it more. Not sure why in EMS would work and PowerShell doesn't.
Can you send the debug log to<EMAIL_ADDRESS>

@Apoc70 are you able to send that information?

@dpaulson45 I will send it by tomorrow.

@dpaulson45 I think we see this behavior because we call this after the if / else block within the Confirm-ExchangeShell function:
Write-VerboseWriter("Returned: {0}" -f $passed) return $passed
It’s no problem when executed in EMS, but it is when executed in PowerShell because we don’t load the Exchange Remote Management shell which we normally do within the if block.

Okay, forget my comment from yesterday. It's never a good idea to check code using a mobile phone.
I've checked the code again using my desktop and I think I've found the reason for the behavior.
@Apoc70 how do you run the script to repro the above-described behavior? Do you use the -Server parameter? If you do, do you use the Fqdn here?
If that's the case we then have the following condition:
   if ($Script:Server -ne $env:COMPUTERNAME)     {         $byPassLocalExchangeServerTest = $true     }
Condition that is checked / passed successful in the Confirm-ExchangeShell function in this case:
   if(($isLocalExchangeServer = (Test-Path 'HKLM:\SOFTWARE\Microsoft\ExchangeServer\v14\Setup')) -or     ($isLocalExchangeServer = (Test-Path 'HKLM:\SOFTWARE\Microsoft\ExchangeServer\v15\Setup')) -or     $ByPassLocalExchangeServerTest)
We then pass the logic and at the end successfully loading the Exchange PowerShell module:
                       Import-Module $env:ExchangeInstallPath\bin\RemoteExchange.ps1 -ErrorAction Stop                         Connect-ExchangeServer -Auto -ClientApplication:ManagementShell                         $passed = $true #We are just going to assume this passed.

@lusassl-msft, I use the following command to reproduce the behavior in a non-EMS PowerShell session.
 .\HealthChecker.ps1 -Debug
The behavior is the same for an elevated or non-elevated PS-Session.
The referenced registry path 'HKLM:\SOFTWARE\Microsoft\ExchangeServer\v15\Setup' does not exist on that particular server.
I installed Exchange Server 10 days ago using build 15.1.2106.2 and the setup finished successfully.

@Apoc70 hm okay, let's have a look at the debug logs.


"run_at":"document_start" seem to be broken

https://bugzilla.mozilla.org/show_bug.cgi?id=1479059
1 clone https://github.com/KOLANICH/run_at_document_start_doesnt_work.git
2 read the code
3 install the extension
4 open the page
5 report the results
For me:
most of the time
test1 broken
test2 broken

only sometimes test1 succeds.

Sorry, been uber busy. We know that for Greasemonkey and ViolentMonkey there are issues with run at doc start. Its why we abandoned one of our scripts ( https://github.com/ghacksuserjs/ghacks-user.js/wiki/4.2.1-User-Scripts - and yup, big warning at the top of the page ) , the clear window.opener one (scroll down) in favor of an extension that earthlng wrote
Not whole ins and outs of who GM & VM have issues with this, and how TM (tampermonkey) uses some hanky BS or something (don't care about TM, screw them) is discussed and documented all over their github issues. GM were getting close to a solution I think. All over my head, TBH.
But it is a problem. If anything I would get the GM and VM guys involved, or there may even be some other tickets open.

I guess it may be possible to workaround that by delaying page and its scripts load untill the success message came from the script by using webRequests, but I don't think this should work this way. IMHO the only point to have document_start is to have guaranties that our scripts end their execution strictly before any of the page content started execution and this must work for both scripts hardcoded  into manifest and run using executeScript on onCommitted. No matter that this may slow down browsing a bit, if a user installs these kind of extensions, it's his decision.

I agree with the sentiment - we/you shouldn't have to use hacks or workarounds - the advertised product should do what it says on the box

Mozilla's QA have verified existance of the bug.

There was an error in my test for static script.


Providing non-corrupted version of oql console icon

The OQL console icon had zero size. Probably result of a previous mistake of mine.

This icon looks OK.

Geertjan, can you please review my change and merge it when you find it? Thanks.

#417 has been integrated. I am waiting for #410 to be integrated too and then I prepare the new binaries. Thanks for the review.


Make sure BLAS is optimized

Honestly, don't totally know what this means or what is involved, but want to double check my BLAS (or whatever the HPC is using) is using all cores.  This will help GAMs scale better.

Looks like R is using the macOS provided BLAS, which should be optimized, but it's not using the macOS LAPACK, and I might gain some performance by figuring out how to link to that.  Tried some instructions I found and it broke R though.

Ok, was able to follow the FAQ here to get a ~30x increase in matrix multiplication:
https://cran.r-project.org/bin/macosx/RMacOSX-FAQ.html#Which-BLAS-is-used-and-how-can-it-be-changed_003f
Not sure how to do something similar for LAPACK, but the improvements in BLAS might be enough!

Next step is figuring out openMP which will allow me to use the ncv.threads argument in gam.control()
https://mac.r-project.org/openmp/

If I end up going with REML and bam() instead of NCV, then I can use a mirai cluster to speed things up with the cluster argument
https://shikokuchuo.net/posts/21-mirai-parallel-clusters/

Have to install mgcv from source and modify Makevars:
https://stackoverflow.com/questions/78590558/how-can-i-enable-openmp-in-mac-os-for-mgcv-r-package
https://mac.r-project.org/openmp/


Run quantize_graph error

System information

Have I written custom code (as opposed to using a stock example script provided in TensorFlow):
No
OS Platform and Distribution (e.g., Linux Ubuntu 16.04):
OS X EI Caption 10.11.6
TensorFlow installed from (source or binary):
binary (pip install)
TensorFlow version (use command below):
TensorFlow 1.2.0-rc1 CPU Only
Bazel version (if compiling from source):
Build label: 0.4.5-homebrew
Build target: bazel-out/local-
opt/bin/src/main/java/com/google/devtools/build/lib/bazel/BazelServer_deploy.jar
Build time: Thu Mar 16 13:37:54 2017<PHONE_NUMBER>)
Build timestamp:<PHONE_NUMBER>
Build timestamp as int:<PHONE_NUMBER>
CUDA/cuDNN version:
CPU Only

Describe the problem
Because of the buffers holding the model weight values are 77MB in size, the memory needed to load these into the app can put a lot of pressure on RAM in Android even before the model is run.
So, I need to further compress the model and round the weights,Even at the expense of accuracy.
So I build /tensorflow/tools/quantization/quantize_graph.There are something wrongs when I run quantize_graph .
Source code / logs
Source code:
bazel-bin/tensorflow/tools/quantization/quantize_graph \
–input=/Users/liba/Desktop/OptimizeCTNModel.pb \
–output=/Users/liba/Desktop/RoundedCTNModel.pb \
–output_node_names=output/outputs \
–mode=weights_rounded

log:
Traceback (most recent call last):
  File "/Users/liba/Desktop/tensorflow/bazel-bin/tensorflow/tools/quantization/quantize_graph.runfiles/org_tensorflow/tensorflow/tools/quantization/quantize_graph.py", line 1301, in <module>
    app.run()
  File "/Users/liba/Desktop/tensorflow/bazel-bin/tensorflow/tools/quantization/quantize_graph.runfiles/org_tensorflow/tensorflow/python/platform/app.py", line 48, in run
    _sys.exit(main(_sys.argv[:1] + flags_passthrough))
  File "/Users/liba/Desktop/tensorflow/bazel-bin/tensorflow/tools/quantization/quantize_graph.runfiles/org_tensorflow/tensorflow/tools/quantization/quantize_graph.py", line 1267, in main
    data = f.read()
  File "/Users/liba/Desktop/tensorflow/bazel-bin/tensorflow/tools/quantization/quantize_graph.runfiles/org_tensorflow/tensorflow/python/lib/io/file_io.py", line 125, in read
    pywrap_tensorflow.ReadFromStream(self._read_buf, length, status))
  File "/Users/liba/anaconda/lib/python2.7/contextlib.py", line 24, in __exit__
    self.gen.next()
  File "/Users/liba/Desktop/tensorflow/bazel-bin/tensorflow/tools/quantization/quantize_graph.runfiles/org_tensorflow/tensorflow/python/framework/errors_impl.py", line 466, in raise_exception_on_not_ok_status
    pywrap_tensorflow.TF_GetCode(status))
tensorflow.python.framework.errors_impl.FailedPreconditionError: .

This quantize_graph tool is outdated. You may try the new transform_graph tool:
https://github.com/tensorflow/tensorflow/tree/master/tensorflow/tools/graph_transforms#eight-bit-calculations

@petewarden This seems like a bad error message even if the tool is deprecated.  Could you take a look?

@Androbin  I try.But it have a error.
log:
ZHANGSH7-MP:tensorflow liba$ bazel-bin/tensorflow/tools/graph_transforms/transform_graph --in_graph =/Users/liba/Desktop/OptimizeCTNModel.pb --out_graph =/Users/liba/DesktoRouddndedOptimizeCTodel.pb  --inputs = 'inputs/X' --outputs = 'output/outputs' --transforms = 'strip_unused_nodes（type = float，shape =“1,256,256,1”）fold_constants（ignore_errors = true) fold_batch_norms fold_old_batch_norms round_weights（num_steps = 256）'
2017-06-12 10:36:01.153946: E tensorflow/tools/graph_transforms/transform_graph.cc:164] Unknown argument --in_graph.
usage: bazel-bin/tensorflow/tools/graph_transforms/transform_graph
Flags:
	--in_graph=""                    	string	input graph file name
	--out_graph=""                   	string	output graph file name
	--inputs=""                      	string	inputs
	--outputs=""                     	string	outputs
	--transforms=""                  	string	list of transforms
	--output_as_text=false           	bool	whether to write the graph in text protobuf format

Transforms are:
add_default_attributes
backport_concatv2
backport_tensor_array_v3
fold_batch_norms
fold_constants
fold_old_batch_norms
freeze_requantization_ranges
fuse_pad_and_conv
fuse_resize_and_conv
fuse_resize_pad_and_conv
insert_logging
merge_duplicate_nodes
obfuscate_names
quantize_nodes
quantize_weights
remove_attribute
remove_device
remove_nodes
rename_attribute
rename_op
rewrite_quantized_stripped_model_for_hexagon
round_weights
set_device
sort_by_execution_order
sparsify_gather
strip_unused_nodes

It can't work!Thanks.

@JcmeLs Seems to be a whitespace issue.
Just try removing the whitespace around the assign.

Hello,
@JcmeLs Can you please tell me the solution you found of the above problem?


使RedisProtocol 支持 jedis所能使用的所有方法.

What is the purpose of the change
XXXXX
Brief changelog
XX
Verifying this change
XXXX
Follow this checklist to help us incorporate your contribution quickly and easily:

[x] Make sure there is a GITHUB_issue filed for the change (usually before you start working on it). Trivial changes like typos do not require a GITHUB issue. Your pull request should address just this issue, without pulling in other changes - one PR resolves one issue.
[ ] Format the pull request title like [Dubbo-XXX] Fix UnknownException when host config not exist. Each commit in the pull request should have a meaningful subject line and body.
[ ] Write a pull request description that is detailed enough to understand what the pull request does, how, and why.
[ ] Write necessary unit-test to verify your logic correction, more mock a little better when cross module dependency exist. If the new feature or significant change is committed, please remember to add integration-test in test module.
[ ] Run mvn clean install -DskipITs to make sure unit-test pass. Run mvn clean test-compile failsafe:integration-test  to make sure integration-test pass.
[ ] If this contribution is large, please file an Apache Individual Contributor License Agreement.

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.Sidney seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.

为什么会这样?


Implement collection.Stats

Description
Closes #3398.

After https://github.com/FerretDB/FerretDB/pull/3464.
Readiness checklist

[ ] I added/updated unit tests (and they pass).
[ ] I added/updated integration/compatibility tests (and they pass).
[x] I added/updated comments and checked rendering.
[ ] I made spot refactorings.
[ ] I updated user documentation.
[ ] I ran task all, and it passed.
[x] I ensured that PR title is good enough for the changelog.
[x] (for maintainers only) I set Reviewers (@FerretDB/core), Milestone (Next), Labels, Project and project's Sprint fields.
[x] I marked all done items in this checklist.

@mergify update


MFRC522: Code review follow-up

Catch errors
Update formatting
Signed-off-by: Eugene Dzhurinsky<EMAIL_ADDRESS>

Codecov Report

Merging #221 into master will decrease coverage by 0.08%.
The diff coverage is 0%.

@@            Coverage Diff             @@
##           master     #221      +/-   ##
==========================================
- Coverage   75.01%   74.93%   -0.09%
==========================================
  Files          69       69
  Lines        6232     6240       +8
==========================================
+ Hits         4675     4676       +1
- Misses       1432     1438       +6
- Partials      125      126       +1

Impacted Files
Coverage Δ

experimental/devices/mfrc522/mfrc522.go
4.83% <0%> (-0.11%)
:arrow_down:

host/cpu/cpu.go
82.14% <0%> (-3.58%)
:arrow_down:

devices/lepton/lepton.go
89.57% <0%> (+1.22%)
:arrow_up:

Continue to review full report at Codecov.

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

gohci


After I run the conda env create -f environment.yml in pycharm, I meet some problems:

First,i receive a warning.
Warning : you have pip-installed dependencies in your environment file, but you do not list pip itself as one of your conda dependencies...
it  prints indefinitely without another action.
Then i add pip under the dependencies,
like following:
dependencies:

pip
_libgcc_mutex=0.1=main
argon2-cffi=20.1.0=py37h8f50634_2

I try again then I received:
Solving environment: failed
ResolvePackageNotFound:
what should I be doing instead

Hi Shenjwei,
I have tried again in my local and everything works well.
My local system setup is

Ubuntu 18.04
conda 4.8.3

Could you tell me what your local system setup is?

I have tried again in my local and everything works well.

win10
conda 4.10.1
Thank you.I found my problem.  so  to create this environment,what i should do?

Hi Shenjwei,
I have posted a new environment_win.yml. Could you pull the latest code and try the following command?
conda env create -f environment._win.yml

Let me know whether it works.

Hi Shenjwei,
I have posted a new environment_win.yml. Could you pull the latest code and try the following command?
conda env create -f environment_win.yml

Let me know whether it works.
First,i receive a warning again.
then i add pip under dependencies:
but it still does not work
Collecting package metadata (repodata.json): done
Solving environment: failed

ResolvePackageNotFound:

libedit=3.1.20191231
readline=8.0
libgcc-ng=9.1.0
libstdcxx-ng=9.1.0
ncurses=6.2

Don't care about the warning. The system would install pip by itself.
Try the latest environment_win.yml I just pushed. Please let me know whether it works.

Don't care about the warning. The system would install pip by itself.
Try the latest environment_win.yml I just pushed. Please let me know whether it works.

it works well.Thank you very much.


Order of elements when mobile

Map
Moves (without log and command line)
Player faction
Research + rounds
Boosters
Other players
Log

A better log, or at least the current log is definitely needed in mobile. It would be nice that it is in the reverse order.
Another option is to add at least the last played move for each player

Solved by https://github.com/donkeytech/gaia-viewer/commit/cf28f083b902c0aebb0413b929402954a58558e6


Hotfix - MSI RP endpoint env var

Which issue this PR addresses:
No Jira
What this PR does / why we need it:
https://github.com/Azure/ARO-RP/pull/3757 introduced a new RP-Config parameter, but I missed a few places where we need to edit our code to make sure the parameter makes it all the way from the config to the running RP.
We found the issue when someone's full service dev RP was failing to start because it found this parameter was missing during startup.
Test plan for issue:
We deployed this change to canary and made sure the RP started up okay.
Is there any documentation that needs to be updated for this PR?
N/A
How do you know this will function as expected in production?
Tested in canary.

/azp run ci, e2e


[SPARK-17756][PYTHON][STREAMING] Workaround to avoid return type mismatch in PythonTransformFunction

What changes were proposed in this pull request?
This PR proposes to wrap the transformed rdd within TransformFunction. PythonTransformFunction looks requiring to return JavaRDD in _jrdd.
https://github.com/apache/spark/blob/39e2bad6a866d27c3ca594d15e574a1da3ee84cc/python/pyspark/streaming/util.py#L67
https://github.com/apache/spark/blob/6ee28423ad1b2e6089b82af64a31d77d3552bb38/streaming/src/main/scala/org/apache/spark/streaming/api/python/PythonDStream.scala#L43
However, this could be JavaPairRDD by some APIs, for example, zip in PySpark's RDD API.
_jrdd could be checked as below:
>>> rdd.zip(rdd)._jrdd.getClass().toString()
u'class org.apache.spark.api.java.JavaPairRDD'

So, here, I wrapped it with map so that it ensures returning JavaRDD.
>>> rdd.zip(rdd).map(lambda x: x)._jrdd.getClass().toString()
u'class org.apache.spark.api.java.JavaRDD'

I tried to elaborate some failure cases as below:
from pyspark.streaming import StreamingContext
ssc = StreamingContext(spark.sparkContext, 10)
ssc.queueStream([sc.range(10)]) \
    .transform(lambda rdd: rdd.cartesian(rdd)) \
    .pprint()
ssc.start()

from pyspark.streaming import StreamingContext
ssc = StreamingContext(spark.sparkContext, 10)
ssc.queueStream([sc.range(10)]).foreachRDD(lambda rdd: rdd.cartesian(rdd))
ssc.start()

from pyspark.streaming import StreamingContext
ssc = StreamingContext(spark.sparkContext, 10)
ssc.queueStream([sc.range(10)]).foreachRDD(lambda rdd: rdd.zip(rdd))
ssc.start()

from pyspark.streaming import StreamingContext
ssc = StreamingContext(spark.sparkContext, 10)
ssc.queueStream([sc.range(10)]).foreachRDD(lambda rdd: rdd.zip(rdd).union(rdd.zip(rdd)))
ssc.start()

from pyspark.streaming import StreamingContext
ssc = StreamingContext(spark.sparkContext, 10)
ssc.queueStream([sc.range(10)]).foreachRDD(lambda rdd: rdd.zip(rdd).coalesce(1))
ssc.start()

How was this patch tested?
Unit tests were added in python/pyspark/streaming/tests.py and manually tested.

Hi @zsxwing, I happened to look into this one. Could you take a look and see if it makes sense please?
cc @zero323 (reporter) too.

Test build #82761 has started for PR 19498 at commit f5a2a88.

Test build #82761 has finished for PR 19498 at commit f5a2a88.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/82761/
Test PASSed.

Merged build finished. Test PASSed.

@rxin, I just noticed you were watching this JIRA. Could you take a look when you have some time?

cc @tdas

Test build #83995 has started for PR 19498 at commit c45b701.

Test build #83995 has finished for PR 19498 at commit c45b701.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/83995/
Test PASSed.

Merged build finished. Test PASSed.

Test build #83996 has started for PR 19498 at commit dc8446a.

Test build #83997 has started for PR 19498 at commit 174ec21.

Test build #83996 has finished for PR 19498 at commit dc8446a.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/83996/
Test PASSed.

Test build #83997 has finished for PR 19498 at commit 174ec21.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/83997/
Test PASSed.

gentle ping @zsxwing, @rxin, @tdas and @holdenk

retest this please

Hi @brkyvz, could you take a look please?

Test build #85184 has started for PR 19498 at commit 174ec21.

Test build #85184 has finished for PR 19498 at commit 174ec21.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/85184/
Test PASSed.

Would it maybe make sense to fix this in the Scala side and accept AbstractJavaRDDLike instead which both JavaPairRDD and JavaRDD extend?

Will check and see if I could make a min fix soon.

To be honest, I think more correct fix might be just to _reserialize but just want to avoid any potential regression by avoiding class check and using existing workaround used.

Test build #90371 has started for PR 19498 at commit 174ec21.

Test build #90371 has finished for PR 19498 at commit 174ec21.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution/3849/
Test PASSed.

Test build #91555 has finished for PR 19498 at commit 174ec21.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/91555/
Test PASSed.

Mind if I go ahead, @rxin, @tdas and @holdenk?

LGTM

Thanks for reviewing this @rxin. Will merge this only to master given the concerns here.

Merged to master.

Thanks @rxin, @tdas, @holdenk and @felixcheung.


[QUERY]

When using the Azure Storage BlobClient to write a blob to Azure Storage, once the call returns is that new blob immediately available to other clients for reading or can there be a brief delay before it's available that developers may need to factor in?
This is not a bug, just a clarification request.
OS: Linux
JDK: 11
Azure Storage Blob 12.8.0 (August 2020) release

@gapra-msft could you please assist @dgpoulet with this Storage question?

@dgpoulet Write operations such as putBlob or commitBlockList are immediately consistent and should be available for read as soon as the service issues a successful response.


pagr.click interface broken refuses to function

puppeteer cannot CLICK and get data from this submit button
the click button interfaces are broken and unusable
try to get this submit to work I spent weeks on it already and its been years the click button interface has had issues
so this should be a good case if it works then show us please
https://www.hoosierlottery.com/winning-numbers/search-winning-numbers?gamename=Daily%204%20Midday
use the above url if you want to test your button click interface
            const selector = 'ContentPlaceHolderDefault_mainContent_CheckWinningNumbers_5_btnSearch'
            const buttonSelector = 'input#ContentPlaceHolderDefault_mainContent_CheckWinningNumbers_5_btnSearch'
            const tableSelector = 'ContentPlaceHolderDefault_mainContent_CheckWinningNumbers_5_gvResults'

            await thePage.setRequestInterception(true)
            await thePage.on('request', async (interceptedRequest) => {
                const headers = interceptedRequest.headers()
                headers['startDate'] = formatTime(startDate)
                headers['endDate'] = formatTime(endDate)
                const overrides: Overrides = {
                    method: 'POST',
                    headers: headers
                }
                interceptedRequest.continue(overrides)
            })

            const response = await thePage.goto(apiUrl + queryParams)
            let composite = await thePage.evaluate(async () => {
                await Promise.all([
                    await thePage.waitForNavigation({ waitUntil: 'networkidle0', timeout: 10002 }),
                    await thePage.click(buttonSelector)
                ])

                return Array.from(document
                    .querySelectorAll('table tr'))
                    .map((ele: HTMLElement) => ele.innerText)
            })

            composite = await composite.filter((item) => item !== '')

composite is always empty
and I have tried a dozen other alternative coding schemes
any help is appreciated this one really burnt me out
the goal being for university quiz to populate table row results into the composite collection

Hi,
I could not even reproduce your code, what is your nodeJS version?
Looking quickly at your code, I see you are using puppeteer functions inside page.evaluate.
The page.evaluate context is the context of the DevTools console, NOT FROM NODE. So, if you have an function in nodeJS, that function will not work inside page.evaluate, because it is another context.
I think this information could be in docs, this could be a little confusing for new players.
Best

thanks friend
im on v8.12.0
what leg of code would you recommend
can you throw something together?

here is another one I tried without success
URL https://www.nhlottery.com/Winning-Numbers
I cannot make anything out of the query params in devtools network on the request so I guessed from what I saw
&ctl00$plcMain$ddlGames=5
sorry if it got out of control this one seems like the one to krack seems so hard falling behind
            const thePage = await theBrowser.newPage()

            const startDate = DateUtil.subtractDays(new Date(), environment.days)
            const endDate = today
            const queryParams = '?startDate=' + formatTime(startDate) + '&endDate=' + formatTime(endDate) + '&ctl00$plcMain$ddlGames=5'
            const crazyConst1 = '&ctl00$plcMain$WinningNumbersSubmit=Search >' +
                '&path=/Winning-Numbers' +
                '&scheme=https' +
                '&__VIEWSTATEGENERATOR=2B9B124E'

            const apiUrl = lottery.entryPointUrl

            const response = await thePage.goto(apiUrl + queryParams + crazyConst1)

            const search = '#ctl00_plcMain_WinningNumbersSubmit'
            await thePage.click(search)

            const selector = '#ctl00_plcMain_WinningNumbersResults > div > div > div > div:nth-child(3)'
            let composite = await thePage.evaluate((selector) => {
                return Array.from(document
                    .querySelectorAll(selector))
                    .map((ele: HTMLElement) => ele.innerText)
            })

upgraded to node 12.3.1 and npm 6.9.0
angular 8+
still wont work

@nhhockeyplayer can you provide us with a small puppeteer script that you expect to work and it doesn't?
Also:

what's your Puppeteer version?
do you use bundled chromium version or you run a custom one?

hi friend
i got latest puppeteer version on npm
puppeteer-core
i just use the chrome browser to run my app
but the puppeteer code is executed on my KOA2 server
same specs
its just javascript

i provided enough code to scrap something together
just goto the url and click

@nhhockeyplayer your code doesn't have definition for formatFunction, nor for the dates you pass as query parameters.
Your code snippet is not runnable, so I went ahead and tried to guess what you actually do. Everything worked fine for me - so I'll go ahead and close this issue.
Next time, please make sure to properly fill in the issue template and respond to all the questions.

I will assume you got data back
maybe it was my dates


-- should be turned into an endash, --- should be turned into an emdash

Think you could take this @paked?

Sure! I can try.

Closed by #53.


fix(vite): conditional exports can't be active when key is browser (#18012)

Description
https://github.com/vitejs/vite/issues/18012#issuecomment-2326745006
Based on my debugging results, the handling here seems a bit too tightly coupled with SSR. For other users encountering this issue, you can try setting the ssrTarget parameter to 'webworker'. However, I believe this area should take into account that the user has already actively declared the conditional.

@NWYLZW Would you elaborate your usecase? Are you testing a library using Vitest for browser on Node without any shims like JSDOM?
I think this is related to https://github.com/vitejs/vite/pull/9860.
@sheremet-va Do you have any thoughts about this?

@NWYLZW Would you elaborate your usecase? Are you testing a library using Vitest for browser on Node without any shims like JSDOM?

I don't need JSDOM here. On the other hand, I feel that using the ssr.target parameter to indirectly influence this is a bit strange. This configuration feels a bit messy; we should respect the user's configuration regarding export conditions.

I think this is related to #9860.

The scenario is somewhat similar to what was mentioned in #9860, but the way it is handled causes node to be activated even when I set the conditional for browser. My export has two implementations based on Node.js and the browser, which leads to the problem.

Hey, I noticed that the current resolve function has switched to using the options.webCompatible check, and this seems to have a significant impact.
I still firmly believe that if a user explicitly declares the corresponding conditional, we should respect their configuration. They shouldn't have to configure another option for this to work. If needed, I can also add related unit tests.

There are some issues with the unit tests, but it seems that I did not introduce them.

Closing as this is now possible since #18395


Publish to composer

Hi,
I wondered when this library will be published to packagist.org so it can be installed with composer? I am happy to use the beta/dev version.
Also it would useful to know if you have any rough ETA for 1.0?
Thanks

+1

Hey!
We are still working on some details, I don't have a clear estimation yet though.
I will be ping you here, when the beta version is on Packagist!

+1 for addition to packagist!


Tapping a Button without handler crashes

I added a Button to a UI with just a title and upon tapping it crashed.
Expected
No crash, no nothing.
Actual
Boom

Boom shockalocka


[Docker] make docker image more lightweight

We don't need to pull the repo every time we build since the file should be in current build context. This should produce less layers than before.
Note that yarn install is now executed before copying the sources to make use of build cache and build even faster!

Pinging @tokkenno for review :smiley:

Yes, I wrote the Dockerfile for my personal use and then decide do the pull request.. can be optimized.
Now I'm in the work, later when I get home, I will do some optimizations and will add some instructions to the README.


chore(main): release 1.6.1

:robot: I have created a release beep boop
1.6.1 (2024-03-15)
Bug Fixes

time series: expose approx_aligned to main module (27399e1)

This PR was generated with Release Please. See documentation.

:robot: Release is at https://github.com/engeir/volcano-base/releases/tag/v1.6.1 :sunflower:


[HOLD] Enable Theme Switching for Users (Go Live)

Add the Preferences Page to the Settings Menu and Enabling Theme Switching - https://docs.google.com/document/d/1KwVVu5-DLjmb-2-UwgIw5ElmyKn0ZXs8wUzxcNZjwDc/edit#bookmark=id.yi1cvj5d9l47

not overdue, Chris has been OOO

not overdue

on hold

nearly there!


Bumping version to include copyright header changes

Blocked by failing integration test:
1) integration-test
       under_armour
         Handle error when shutting down (NUI-547)

@DrGirlfriend This does not affect this PR. You can still merge.
The CI-CD issue is under investigation.

I published a new version 1.3.0<EMAIL_ADDRESS>


Create jwplayer

IMPORTANT: Please do not create a Pull Request without creating an issue first.
Any change needs to be discussed before proceeding. Failure to do so may result in the rejection of the pull request.
Proposed Changes

...
...
...

Relevant issues
Closes #...
Closes #...
Closes #...
Related #...

Closing as the PR doesn't have any context or code changes. @boylovesdt777777 please create an issue and link the PR if there is any issue that we need to resolve


fix: check for CSRF token in the raw body

Description
CSRF check for PUT, PATCH, and DELETE type of requests is made only for JSON data. This PR fixes that by adding the raw input data to check.
Checklist:

[x] Securely signed commits
[x] Component(s) with PHPDoc blocks, only if necessary or adds value
[x] Unit testing, with >80% coverage
[ ] User guide updated
[x] Conforms to style guide

I have no idea what the PHPStan error means because every occurrence of empty was removed from the file, even though I did not change that particular part of the code.

@michalsn run vendor/bin/phpstan analyse --generate-baseline phpstan-baseline.php

@paulbalandan Thanks!


[tests-only] Bump core commit 20201120

Similar to https://github.com/owncloud/ocis/pull/907

bump the core commit id for API tests - gets us:
https://github.com/owncloud/core/pull/38120 [Tests-Only] Update issue tags for test scenarios
https://github.com/owncloud/core/pull/38127 [tests-only] Fix tests WebDav.php BadResponseException

Adjust behat composer.json for API tests to be like oC10 core - does similar to core PRs:
https://github.com/owncloud/core/pull/38122 [tests-only] Bump test tool dependencies
https://github.com/owncloud/core/pull/38128 [tests-only] Use guzzle7 in acceptance tests

Note: I will also wait fr core PR https://github.com/owncloud/core/pull/38130 to run CI and hopefully pass and be merged. Then I  can include that stuff also.

Waiting for core PR https://github.com/owncloud/core/pull/38131 which fixes the failing tests due to assertContains in phpunit9


Add word "Quintessential

Add the word "Quintessential"
Add all the definitions of Quintessential in a file named Quintessential.json
Refer Contribution.md for more details on how to contribute.
Comment here if you're are working on this issue to avoid conflicts.
Add reference to this issue i.e Issue #200 in your pull request description.

This one's for me, love the sound of this word :smile:


Add logpush jobs client

Since Logpush Jobs are available for Enterprise domains, it will be good to have it in Go client.
https://developers.cloudflare.com/logs/logpush/
This changes are required for https://github.com/terraform-providers/terraform-provider-cloudflare/issues/210
API documentation: https://api.cloudflare.com/#logpush-jobs-properties

Closing this now that logpush support has been merged into master. Thanks @SebastianCzoch !


First login is slow

First time after start or re-start of the server seems to be slow. I'm guessing that some cache is being loaded.
@Abyss777, do you think it's a cache issue? Can it be fixed by moving cache loading to startup instead of first web service call?

I'm not sure that it is cache issue. For example my main server is inaccessible by web for 10-15 seconds after start. (it has 4 devices, 1 group, 5 geofences). It is very slow or even can give error 500. An it was always (before 3.7 and 3.6). May be jetty creates threads on demand or something.
Another guess is database.selectLatestPositions request.
Anyway we can add debug prints at the end of every refresh() function to check.

It's not urgent, but it's good to investigate the issue. I noticed it recently, so it must be more or less recent change. Happens on an essentially empty server as well.

Here is my logs
2016-10-08 20:33:28  INFO: Operating system name: Linux version: 3.13.0-93-generic architecture: i386
2016-10-08 20:33:28  INFO: Java runtime name: OpenJDK Client VM vendor: Oracle Corporation version: 24.111-b01
2016-10-08 20:33:28  INFO: Memory limit heap: 494mb non-heap: 118mb
2016-10-08 20:33:28  INFO: Character encoding: UTF-8 charset: UTF-8
2016-10-08 20:33:28  INFO: Version: 3.7-SNAPSHOT
2016-10-08 20:33:36 DEBUG: Groups cache update started
2016-10-08 20:33:36 DEBUG: Groups cache updated
2016-10-08 20:33:36 DEBUG: Devices cache update started
2016-10-08 20:33:36 DEBUG: Devices cache updated
2016-10-08 20:33:40 DEBUG: Users cache update started
2016-10-08 20:33:40 DEBUG: Users cache updated
2016-10-08 20:33:40 DEBUG: Permissions cache update started
2016-10-08 20:33:40 DEBUG: Permissions cache updated
2016-10-08 20:33:40 DEBUG: Geofences cache update started
2016-10-08 20:33:40 DEBUG: Geofences cache updated
2016-10-08 20:33:40 DEBUG: Notifications cache update started
2016-10-08 20:33:40 DEBUG: Notifications cache updated
2016-10-08 20:33:41 DEBUG: Aliases cache load started
2016-10-08 20:33:41 DEBUG: Aliases cache loaded
2016-10-08 20:33:41  INFO: Starting server...

STATUS | wrapper  | 2016/10/08 20:33:26 | --> Wrapper Started as Daemon
STATUS | wrapper  | 2016/10/08 20:33:26 | Java Service Wrapper Community Edition 32-bit 3.5.28
STATUS | wrapper  | 2016/10/08 20:33:26 |   Copyright (C) 1999-2016 Tanuki Software, Ltd. All Rights Reserved.
STATUS | wrapper  | 2016/10/08 20:33:26 |     http://wrapper.tanukisoftware.com
STATUS | wrapper  | 2016/10/08 20:33:26 |
STATUS | wrapper  | 2016/10/08 20:33:27 | Launching a JVM...
INFO   | jvm 1    | 2016/10/08 20:33:27 | WrapperManager: Initializing...
INFO   | jvm 1    | 2016/10/08 20:33:28 | [WrapperSimpleAppMain] INFO com.zaxxer.hikari.HikariDataSource - HikariPool-1 - Started.
INFO   | jvm 1    | 2016/10/08 20:33:31 | INFO 10/8/16 8:33 PM: liquibase: Clearing database change log checksums
INFO   | jvm 1    | 2016/10/08 20:33:32 | INFO 10/8/16 8:33 PM: liquibase: Successfully acquired change log lock
INFO   | jvm 1    | 2016/10/08 20:33:32 | INFO 10/8/16 8:33 PM: liquibase: Successfully released change log lock
INFO   | jvm 1    | 2016/10/08 20:33:32 | INFO 10/8/16 8:33 PM: liquibase: Successfully acquired change log lock
INFO   | jvm 1    | 2016/10/08 20:33:36 | INFO 10/8/16 8:33 PM: liquibase: Reading from `traccar-new`.DATABASECHANGELOG
INFO   | jvm 1    | 2016/10/08 20:33:36 | INFO 10/8/16 8:33 PM: liquibase: Reading from `traccar-new`.DATABASECHANGELOG
INFO   | jvm 1    | 2016/10/08 20:33:36 | INFO 10/8/16 8:33 PM: liquibase: Successfully released change log lock
INFO   | jvm 1    | 2016/10/08 20:33:39 | [WrapperSimpleAppMain] INFO org.eclipse.jetty.util.log - Logging initialized @12476ms
INFO   | jvm 1    | 2016/10/08 20:33:41 | [WrapperSimpleAppMain] INFO org.eclipse.jetty.server.Server - jetty-9.2.19.v20160908
INFO   | jvm 1    | 2016/10/08 20:33:44 | [WrapperSimpleAppMain] INFO org.eclipse.jetty.server.handler.ContextHandler - Started o.e.j.s.ServletContextHandler@598d7c{/api,null,AVAILABLE}
INFO   | jvm 1    | 2016/10/08 20:33:44 | [WrapperSimpleAppMain] INFO org.eclipse.jetty.server.ServerConnector - Started ServerConnector@1fc31d5{HTTP/1.1}{<IP_ADDRESS>:8082}
INFO   | jvm 1    | 2016/10/08 20:33:44 | [WrapperSimpleAppMain] INFO org.eclipse.jetty.server.Server - Started @17558ms

Do not see anything criminal.
Caches loaded in a second.
This time jetty loaded rather fast, but sometime it is slower. And frankly, after everything is loaded, I do not get any issues on logon.
May be we are talking about different issues?

Empty server on local machine.
Server is fully loaded. Last line in the wrapper log:
[main] INFO org.eclipse.jetty.server.Server - Started @10744ms

Here is the first login:

I have finally fixed this issue. Turns out the problem was with slow crypto functions. Moved initialization to initializer instead of the first service call.

Frankly, I did not affected it much, but great anyway.


GUACAMOLE-1325: Migrate from javax to jakarta namespace.

Okay, well, here we go. This PR takes care of the migration from the javax.* namespaces to the new jakarta.* counterparts, updates dependencies, and updates the licenses. I've got it all in a single commit, now, because I couldn't find very clean places to break up the commit - dependency management at this level reminds me of the cat-in-bathtub gif.
The code does compile, I'm able to install it into Tomcat 10.1, and run it, log in (JDBC), and start Guacamole session. It's worth noting that this will be a big change in terms of compatibility:

Right now I'm compiling and running with Java 11.x. I have not gone back to test 8.x, yet - I will, here, soon - but there's a reasonable chance that 8.x will no longer work with some of the dependency updates.
There were a couple of dependencies that I could not fully update because they required Java 17, and I wasn't willing to push quite to that level of breakage.
This completely removes the Websocket support for Jetty8, Jetty9, and Tomcat 7, relying on the standard WebSocket support that should be built in to anything recent.
I have not tested with anything besides Tomcat 10.x, yet. I intend to try Tomcat 11, JBoss, and Wildfly, at least.
There is likely a decent amount of cleanup to be done, still, both in pom.xml files, and in the doc/licenses directory. I suspect I've left a lot of unused cruft around in getting things updated.
I have not tested all of the extensions, yet - just JDBC. I'll try to get to several of them, soon, but there will likely be some I cannot test.
I'm unsure about the future of the RADIUS module - I haven't even tried to compile that one, yet, and, given the age of the jRADIUS library, I'm not sure it can live on as it is. We'll can either find a different RADIUS library, write our own (ick, but not too difficult), drop RADIUS support altogether, or see if there are other options for maintaining compatibility with class loading and the deep magic of Java.

I'm going to put this in Draft mode for now - I welcome comments, requests for changes, and testing.

Right now I'm compiling and running with Java 11.x. I have not gone back to test 8.x, yet - I will, here, soon - but there's a reasonable chance that 8.x will no longer work with some of the dependency updates.

Well, turns out Tomcat 10 requires Java 11, so that's settled.

Hi! Thanks a lot for starting this! I've been trying to find time to try it out.
I assume that the maven snapshots didn't land in a maven repository (yet). Should I fork this and build it separately/internally?
The build is failing, should I take anything into account while building?

@z0rb The code has not been merged, so there will be no artifacts or builds with this change, yet. Yes, you will need to clone/patch locally and build yourself to test it out.

Looks good on Java 17 and Spring-Boot 3.2.5!
Had no problem with guacamole-common (only module I'm using), most migration effort was from Spring Security.
Thanks again! Hope this makes it into main once the open questions @necouchman mentioned are resolved.

Looks good on Java 17 and Spring-Boot 3.2.5! Had no problem with guacamole-common (only module I'm using), most migration effort was from Spring Security.
Thanks again! Hope this makes it into main once the open questions @necouchman mentioned are resolved.

Thank you for testing! It will not go into main, it will go into next, since it is targeted for a future major release (2.0 or something similar). 1.6.0 will be a feature release.

Thanks for the Tomcat10 patch, I've deployed it and the only issue I have is with the OpenID Connect extension that throw this error :

The "openid" authentication provider has been skipped due to an internal error. If this is unexpected or you are the  developer of this authentication provider, you may wish to enable debug-level logging: jakarta.ws.rs.ext.RuntimeDelegate: org.glassfish.jersey.internal.RuntimeDelegateImpl not a subtype

Mind you, I've only used guacamole-history-recording-storage / guacamole-auth-sso-openid / guacamole-auth-totp / guacamole-auth-jdbc.


[1.15] bpf: Add 'host not ready' drop reason if host policy cannot be enforced

This is a manual backport of https://github.com/cilium/cilium/pull/29482.
Add specific drop reason for missing tail calls if the host datapath is not ready yet

/test-backport-1.15

/test-backport-1.15


Places

Largely done?

Separating guides into #604

I guess the structure of this should be fairly similar to discussions?


Dimensions api in android giving the dimension of Landscape mode by default.

Even if the phone is in Potrait mode it's giving dimensions of landscape mode.

Can you provide a sample of this on rnplay.org demonstrating the problem?

I'm closing this issue for lack of information.


assertion fails crash

Was just testing dockerize environment and pushing a few calls between two UAs both are WebRTC clients on mobile networks. After pushing several calls successfully i noticed a crash.
crash log: https://gist.github.com/andrewvmail/fbbdcd78e3ee8b6d4b7601121faf9e54

It looks like the client sends an INVITE
018-10-11 01:38:41.464946 recv 2438 bytes from ws/[<IP_ADDRESS>]:42632 at 01:38:41.464186:
INVITE<EMAIL_ADDRESS>SIP/2.0
Via: SIP/2.0/WS 14bjlup3m357.invalid;branch=z9hG4bK3069245
Max-Forwards: 70
To<EMAIL_ADDRESS>From: <sip:bob@dev-sfo1-p2p-1.superfatcat.com>;tag=6efuij01qs
Call-ID: 3vur9ciggmj4cugku2hn
CSeq: 642 INVITE

then sends a CANCEL:
2018-10-11 01:38:57.083471 recv 342 bytes from ws/[<IP_ADDRESS>]:42632 at 01:38:57.083302:
CANCEL<EMAIL_ADDRESS>SIP/2.0
Via: SIP/2.0/WS 14bjlup3m357.invalid;branch=z9hG4bK3069245
To<EMAIL_ADDRESS>From: <sip:bob@dev-sfo1-p2p-1.superfatcat.com>;tag=6efuij01qs
Call-ID: 3vur9ciggmj4cugku2hn
Max-Forwards: 70
CSeq: 642 CANCEL
Content-Length: 0```
we send a 200 OK to the CANCEL and a 487 to the INVITE, which is all correct.

We then see a retransmit of the the CANCEL from the client for some reason:

2018-10-11 01:38:57.090348 recv 342 bytes from ws/[<IP_ADDRESS>]:42632 at 01:38:57.090104:
CANCEL<EMAIL_ADDRESS>SIP/2.0
Via: SIP/2.0/WS 14bjlup3m357.invalid;branch=z9hG4bK3069245
To<EMAIL_ADDRESS>From: sip:bob@dev-sfo1-p2p-1.superfatcat.com;tag=6efuij01qs
Call-ID: 3vur9ciggmj4cugku2hn
Max-Forwards: 70
CSeq: 642 CANCEL
Content-Length: 0
Now, since we are using websockets I do not think the client should be retransmitting (not to mention we have already send a 200 OK to the prior CANCEL).

What happens next, apparently, is that the app sends a 404 Response, presumably in response to this CANCEL but the transaction id is that for the INVITE.  This is what leads to the crash, because it triggers an assertion:

2018-10-11 01:38:57.095479 Client::read_handler read: efa95f0e-8d0e-46c6-8349-505be2e2609c|sip|0c90fa15-7e50-4e31-b867-564085910a7c|
SIP/2.0 404 Not Found
Call-ID: 3vur9ciggmj4cugku2hn
cseq: 642 CANCEL
from: sip:bob@dev-sfo1-p2p-1.superfatcat.com;tag=6efuij01qs
to<EMAIL_ADDRESS>
First, any idea why the client is retransmitting the CANCEL request?  That should only happen over a udp connection I think.

Secondly, is it possible to get any logs from the drachtio-srf side with DEBUG=drachtio:* for this crash?

Hi Dave,
Thanks for pointing this out, I did not consider client sending multiple cancel. So I found out why client is sending the 2 cancels. When using sipjs and I do session.cancel() then i stop the ua using ua.stop() there is a race or something, the cancel sends a cancel and the ua.stop also sends a cancel. I can just do a ua stop(). So bug reported there if its a bug..
And I cant get you the log for this crash but I can do a new one. I'll do that now and follow up with the logs.

okay logs here, I masked some ip with xxx and username with bob and mary
drachtio-server: https://gist.github.com/andrewvmail/8d9f5a6907a6b7765e321c712af90ec0
drachtio-controller: https://gist.github.com/andrewvmail/df0dc5ed8a143c4e60444a92d8bd62d6

can you retest with latest<EMAIL_ADDRESS>?  I believe I have checked in a fix there for this.  The framework should not be trying to send a 404 to the second CANCEL, which was leading to the assertion

Okay tested and confirmed its fixed.
Thanks!


Remove DBHandler and "logging" of results to DB

This antipattern was added when I was excited to learn about logging.  But there's no reason a user would customize the actual behaviour using logging, and if they change the loglevel they wouldn't expect the unpredictable side effects.
Still need to update the DB, but no sense going through logging infrastructure.

Probably will be a good time to solve #34


Module Level Markdown w/ Metadata to generate Cab and Help Info

We need to create a markdown file that summarizes all of the cmdlets in the module. It will also contain the metadata at the module level that is required for creating a cab.
Here is the example:
[Windows PowerShell Management Cmdlets - Landing Page] (https://technet.microsoft.com/en-us/library/hh849827.aspx)
We need, at least:
For the Cab

Module Name
Module GUID

For the HelpInfo

FW Link to the cab help package for downloading in updateable help
Help Version
Locale

Idea for how to accomplish this. @vors please validate my thoughts if you have time. @sankethka for awareness....
We need to have the ability to generate a landing page as part of the new-markdown functionality. This would only be triggered when you create an entire module.
This suggestion only covers the initially creation, after we have that, we should discuss the update scenario for Update-Markdown.
Using each of the cmdlets names (which become CmdletName.md), the Module Guid, and the Module Name, we create a MD file named ModuleName.md with the following

Module Name: {Populated during New-Markdown}
Module Guid:{Populated during New-Markdown}
Download Help Link: {{Please enter FwLink manually}}
Help Version: {{Please enter version of help manually (X.X.X.X) format}}
Locale: {{Please enter help locale code manually (xx-XX) format}}

{Module Name} Module
Description
{{Manually Enter Description Here}}
{Module Cmdlets} Cmdlets
{CmdletName.md Link}}
{{Manually Enter Description Here}}
{Cmdlet Name.md Link}}
{{Manually Enter Description Here}}

Checked in to master:
New-Markdown with -Module or -Maml creates a landing pages.
Remaining:
Create Help Cab using metadata info from Landing page

Finalizing tests, but this is complete.


Security Exception in Unit testing

Hi,
I tried different unit test environment like (xunit, nspec,...) and getting no success with them.
when unit test code reaches system.Initialize() the Security Exception will be thrown...
the full message is:
"System.Security.SecurityException was unhandled by user code
HResult=-2146233078
Message=ECall methods must be packaged into a system module.
Source=Assembly-CSharp
StackTrace:
at TestSystem.Initialize()
at test.<>c.<when_making_test>b__0_0() in C:\UnityProject\ConsoleApplication1\ConsoleApplication1\test.cs:line 22
at NSpec.Domain.Example.Run(nspec nspec)
at NSpec.Domain.ContextUtils.RunAndHandleException(Action`1 action, nspec instance, Exception& targetException)
InnerException: "
any solution to fix this?
btw I'm working on windows.
Cheers

thanks for the reply,
unblocking Dll didn't help me.
base on this link  seems specific unity functions only can call inside the editor.
by adding simple try catch temporary ignore exception but I don't know why it's throwing from the system..

I had similar problem about error when reaches system.Initialize().
Some configs set before system.Initialize(), and those configs will be loaded in InitializeSystem.
but the path of configs are changed. The configs cannot be loaded.
So, that cause the unit test's error.
You have to ensure every source read correctly before system.Initialize().


Handle new manual instrumentation events in OrbitGl.

Test: Compile
Bug: http://b/194763537; http://b/194764373; http://b/194764259;

Requires #2634 to compile.

It took a bit to review. It could be nice if we have several commits with messages in each of them, but you can try to list the changes you made here, what do you think? For example information of the new kApiScopeAsync.

Thanks for the review. This is already split into 4 PRs. I think this change somehow should stay together. I will elaborate a bit more in the commit message (will update the PR message for that).


[Ergonomics/performance] Can we avoid storing the SimpleComboBox variants twice

Currently only the index is returned, but this requires the variants to be stored twice: once in the SimpleComboBox and once in the closure. (both launch and forward consume the component, so it's impossible to use it in the forward closure).
However, returning E would require E: Clone and returning &E gets us into lifetime troubles.

You can just access the variants field if you need to access the variants after the component has been launched. All other options will cause lifetime issues. For most usecases get_active_elem should be convenient, but if you describe your usecase, we might figure out a better/new API for you.

Em, I can't just access the variants field: let's try to modify the code from the combo_box example and try to output the string instead of the index:
        let combo = SimpleComboBox::builder().launch(SimpleComboBox {
            variants: LANGS.to_vec(),
            active_index: Some(default_idx),
        });
        let combo = combo.forward(sender.input_sender(), |idx| {
            let name = combo.model().variants[idx];
            AppMsg::ComboChanged(idx)
        });

I can't use the SimpleComboBox struct in the closure because it was consumed by launch. Trying to use model correctly triggers the borrow checker. I can't move the combo both into the closure and into forward.

Why would you even need the combo box in the forward closure? You can just access the combo box in update() from the model of the outer component. I see that it would be convenient in your case, but Relm4 has to make the borrow checker happy internally as well, and I don't see an easy to solution to make the model available in the forward closure.

I see, thanks. I'll try to update the example to make it work.


App theme and color palette

We should make global theme for buttons (primary, secondary, active, disabled, etc), texts, and unify all views.
Color palette should be common, theme will be different for web and mobile.

Common styles for web and mobile is not possible in simple way. Styles for mobiles provides the Theme already. Closing for now.


show time periods on details charts

closes #273

I think the date label logic might be off for the hourly chart

I think the date label logic might be off for the hourly chart

This is because its formatted in German time format, so thats the hour. I added the minutes to make it more clear


[Feature Request] autojoin and autoplay on startup

Is your feature request related to a problem? Please describe.
Currently you have to manually join the bot into a voice channel and start playing some music.
Describe the solution you'd like
There should be an option for the bot to automatically join a voice channel and play the default playlist on startup.
Describe the basic flow/steps of using this feature
How this feature would be used:

(Re)start the bot
Automatically join the default voice channel, if set in serversettings.json
Play the default_playlist, if set in serversettings.json

This is already the case. If you use setvc and autoplaylist in a server, when the bot starts it will automatically connect to the VC channel and begin playing the autoplaylist.
Also, do not edit serversettings.json manually; use the commands within the server.

Wasn't obvious to me from the documentation, thanks. Works as expected.

Im sorry my English is bad.
I need help with !autoplaylist.

Im sorry my English is bad. I need help with !autoplaylist. -edit. Shutdown not work Im also new in linux Im not progamer Sorry if it bothers you Thank You. My bot in desktop/botfolder

Hello im sorry, the Bot was work. I have 1 digit excess on the owner ID. Thank You.


[BEAM-2671] Fixing CreateStreamTest#testFirstElementLate

Follow this checklist to help us incorporate your contribution quickly and easily:

[x] Make sure there is a JIRA issue filed for the change (usually before you start working on it).  Trivial changes like typos do not require a JIRA issue.  Your pull request should address just this issue, without pulling in other changes.
[x] Each commit in the pull request should have a meaningful subject line and body.
[x] Format the pull request title like [BEAM-XXX] Fixes bug in ApproximateQuantiles, where you replace BEAM-XXX with the appropriate JIRA issue.
[x] Write a pull request description that is detailed enough to understand what the pull request does, how, and why.
[ ] Run mvn clean verify to make sure basic checks pass. A more thorough check will be performed on your pull request automatically.
[ ] If this contribution is large, please file an Apache Individual Contributor License Agreement.

This PR aims to fix CreateStreamTest#testFirstElementLate() which has been failing lately.
More info can be found in the corresponding JIRA issue as well as in a short write-up describing the problem and the suggested solution.

Run Spark ValidatesRunner

Coverage increased (+0.01%) to 69.981% when pulling f73b3324e4b18f3da9945ae61964723096d71ee0 on staslev:BEAM-2671-fixing-CreateStreamTest#testFirstElementLate into 95cd37faff1c57dc4184ed001d8d168bc42d174e on apache:master.

R: @jbonofre

Coverage increased (+0.01%) to 69.977% when pulling ebe79cad91683018bdf3dc9ab5d152f4a80f36f7 on staslev:BEAM-2671-fixing-CreateStreamTest#testFirstElementLate into d338b4412b9d7f26509afc293047dbdf85e17e61 on apache:master.

R: @amitsela, @aviemzur.
CC: @jbonofre
@aviemzur, I believe I was able to drop the hack in GlobalWatermarkHolder where there were separate flows for production and testing.

Run Spark ValidatesRunner

Run Spark ValidateRunner

Run Spark ValidatesRunner

Run Spark ValidatesRunner

Coverage increased (+0.003%) to 69.971% when pulling 74b04e71ea677482b0008816b9478fa12fcb0574 on staslev:BEAM-2671-fixing-CreateStreamTest#testFirstElementLate into d338b4412b9d7f26509afc293047dbdf85e17e61 on apache:master.

Run Spark ValidatesRunner

Run Spark ValidatesRunner

Coverage decreased (-0.003%) to 69.964% when pulling 22e7e2778b6964979b45687b0d2fc9ab86f15641 on staslev:BEAM-2671-fixing-CreateStreamTest#testFirstElementLate into d338b4412b9d7f26509afc293047dbdf85e17e61 on apache:master.

Run Spark ValidatesRunner

Coverage increased (+0.003%) to 69.971% when pulling b67b30c3f460614dd9a51388d76b92fafc511ae4 on staslev:BEAM-2671-fixing-CreateStreamTest#testFirstElementLate into d338b4412b9d7f26509afc293047dbdf85e17e61 on apache:master.

Run Spark ValidatesRunner

Coverage increased (+0.01%) to 69.977% when pulling 0fd9c7c34732518eebfe589c522be6e5f8654fbf on staslev:BEAM-2671-fixing-CreateStreamTest#testFirstElementLate into d338b4412b9d7f26509afc293047dbdf85e17e61 on apache:master.

@staslev looks like it passed! Failure is a direct runner failure.

Run Spark ValidatesRunner

Run Spark ValidatesRunner

Run Spark ValidatesRunner

R: @amitsela, @aviemzur.
CC: @jbonofre

This PR should resolve the intermitente failures of CreateStreamTest#testFirstElementLate().

While looking into it I believe I came across an issue with the way the watermark block is updated (in production, tests use a workaround since following operations are neither atomic not synchronous:
blockManager.removeBlock(WATERMARKS_BLOCK_ID, true);
// more stuff
blockManager.putSingle(WATERMARKS_BLOCK_ID, newValues, StorageLevel.MEMORY_ONLY(),
true);

This may create a situation where an executor asks for the watermarks and gets a null result (since
the block is not found), in which case it will default to negative infinity and use it as the watermark.
I'll file a separate JIRA to look into it.

Coverage decreased (-0.01%) to 69.967% when pulling d3bab8b5b3d64506ef061eab72da681d0504af55 on staslev:BEAM-2671-fixing-CreateStreamTest#testFirstElementLate into 62294e64cb3f68e37cd0ed133070fb617334c09e on apache:master.

Addressed review comments and squashed.

Run Spark ValidatesRunner

Run Spark ValidatesRunner

Coverage decreased (-0.007%) to 69.974% when pulling 10be9693a1ae7a3619243943b1b75df9b9125a48 on staslev:BEAM-2671-fixing-CreateStreamTest#testFirstElementLate into 9b175ccc2f88d805ead3131c56129c24bdf1045e on apache:master.

Added a comment about the Watermark syncing, but other than that LGTM.

Coverage decreased (-0.02%) to 69.964% when pulling f0af05af2a490c50264d7851658514077f2d9948 on staslev:BEAM-2671-fixing-CreateStreamTest#testFirstElementLate into 9b175ccc2f88d805ead3131c56129c24bdf1045e on apache:master.

Run Spark ValidatesRunner

Coverage remained the same at 69.981% when pulling c0ae451e76f52316dac29c492f5b42ca763004ac on staslev:BEAM-2671-fixing-CreateStreamTest#testFirstElementLate into 9b175ccc2f88d805ead3131c56129c24bdf1045e on apache:master.


I am trying to change the font family of tha random disco light simul…

…ator app that looks like a good .Feel the user like play a game

@suhanachaudhary
Thank you for submitting this pull request! However, I noticed that there isn't an associated issue for this change.
you can create an issue describing the problem you're addressing with this PR. Once you've done that and the issue has been assigned to you, you can create a PR! This helps us track changes more effectively.


Armor Master Issue

Armor Master seems to only give extra max dex bonus starting from when when you take the feat or just gives 1 dex bonus. Ie, taking the feat at MR3 just gives 1 more max dex rather than 3 to the chainmail of the dragonfly. Has anyone else experienced this?

Are you at medium encumberece? This sets max dex bonus to +3 regardless of
any other factors.
On Tue, Dec 20, 2022, 5:21 PM Qatarik @.***> wrote:

Armor Master seems to only give extra max dex bonus starting from when
when you take the feat or just gives 1 dex bonus. Ie, taking the feat at
MR3 just gives 1 more max dex rather than 3 to the chainmail of the
dragonfly. Has anyone else experienced this?
—
Reply to this email directly, view it on GitHub
https://github.com/Vek17/TabletopTweaks-Base/issues/562, or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAWSQPEGQ4RL45VGSNFIP3LWOI5PZANCNFSM6AAAAAATFAT36Q
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>

Thank you! That is exactly what that was.


Scaling to higher resolutions?

Hello! I've been recently trying to use TResNet-M industrially on datasets where the image is pretty large (720x1280+).  When doing this, the memory consumption is very high (batch of 16 takes up a 2080ti at mixed precision). Thus, I'm here asking for any advice on reducing the memory consumption through network modifications. Other modifications to the network to make it work on larger resolutions are also welcomed.

Hi Akamight
TResNet ability to work with larger batch is a significant advantage over other networks.
have you tested other networks (resnet50, efficieneNet ) for maximal batch size ?
anyway, some (semi) practical advices:

i was able to wok with resolution of 900x900 with batch size of 24. so make sure your code is fully optimized in terms of memory usage. it seems you have some room for optimization (loss function ? data loading ? )

your input resolution is enormous. do you really need that ? are results with input of 600x600 really worse?
i have participated and won top places in Kaggle competitions, but never needed resolutions that big.

one trick i can suggest, although i haven't tried it myself, is gradient checkpointing:
https://qywu.github.io/2019/05/22/explore-gradient-checkpointing.html
https://medium.com/tensorflow/fitting-larger-networks-into-memory-583e3c758ff9

on top it all, why do you care so much about increasing the batch size ? since your pictures are enormous, i will guess that one way or another, the GPU is in full utilization.


Class menu item run tests broken

Logged into 3.0.33 as DataCurator,  RowanSample1 loaded.  Select class RowanSample1Test.
The class menu item run tests produces a walkback.
I can run tests just fine from the Sunit tab, so it's not a userId problem per se.
a LookupError occurred (error 2096), reason:assocErrNoElementsDetected, detect: was sent to a collec
LookupError (AbstractException) >> _signalWith: @5 line 25
LookupError (AbstractException) >> signal @2 line 47
IdentitySet (Object) >> _error:args: @15 line 11
IdentitySet (UnorderedCollection) >> _detect: @13 line 9
IdentitySet (UnorderedCollection) >> detect: @15 line 23
JadeServer64bit32 (JadeServer) >> sbClassFrom: @14 line 14
JadeServer64bit32 (JadeServer) >> sbRunClassTests: @4 line 4
GsNMethod class >> _gsReturnToC @1 line 1

Fixed in 3.0.35. Part of the many perils of trying to maintain two separate browsers is that stuff like this has to be maintained in two ways.  When the base is rowanized and we can get rid of the dictionary browser, stuff like this will go away.


The default model is visible when changing models with the command

Hello! Thank you for the wonderful mod.
I see the default model when I change the model with the /cpm command. Is there any way to prevent this?
It seems that if the model is not cached, the default model is visible for a longer time.
I am using CustomPlayerModelsFabric-1.20.2 v0.6.14b fabric.
I am Japanese, so please excuse the machine translated text...
https://github.com/tom5454/CustomPlayerModels/assets/85728971/fd310aee-3c25-404c-a558-f9eb0c7a0123

Merge all of your models into one, and use toggle animations with the command controlled flag. Then set the toggle animations with /cpm animate command
The models are loaded in the background, and in most cases from the network or from the local cache, but that still needs file parsing, texture loading etc.

Thank you for your answer!
In this example, the model was thicker and thinner, but if I frequently replace a model with a very different shape, do I still have to merge it into one model?
For example, changing from a whale to a pig.

If you really want to avoid the less then 1 second load time, then yes.
Or give the player 1 or 2 seconds of invisibility, and spawn some particles, to hide the model change.

Thank you for your thoughtful reply!
I will try to work on this in a creative way using layer toggles and animations.


client: Remove unnecessary progress bar update

Problem
As mentioned in https://github.com/anza-xyz/agave/pull/358#discussion_r1580532890, during send_and_confirm_transactions_in_parallel, there are two immediate updates to the progress bar, making one of them unnecessary.
Summary of Changes
Remove the second one, which contains less information
Fixes #

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 82.1%. Comparing base (d9d52be) to head (586b55f).

Additional details and impacted files
@@           Coverage Diff           @@
##           master    #1073   +/-   ##
=======================================
  Coverage    82.1%    82.1%
=======================================
  Files         868      868
  Lines      234074   234071    -3
=======================================
+ Hits       192373   192390   +17
+ Misses      41701    41681   -20


commands not working for prediction

Hi
Thanks for posting your code! love your work and wanted to reproduce the result.
For 3. Link Prediction (UMLS)
I ran the exact same command as you posted but get this. Can you help me how I can fix this?
python3 run_bert_link_prediction.py
--task_name kg
--do_train
--do_eval
--do_predict
--data_dir ./data/umls
--bert_model bert-base-uncased
--max_seq_length 15
--train_batch_size 32
--learning_rate 5e-5
--num_train_epochs 5.0
--output_dir ./output_umls/
--gradient_accumulation_steps 1
--eval_batch_size 135

Thank you in advance

@ChloeJKim
Thank you! The command should be in the same command line.


Password being logged into the console for a password protected url.

Describe the bug
When any user tries to open the short url, the password gets logged into the console.
To Reproduce
Steps to reproduce the behavior:

Open the short url in a browser.
Open Console of the browser.
If the link was password protected, the password get logged into the console.

Expected behavior
Should not be logged.

Hey @vikashgaya916 , thanks for raising the issue will look into this.


Call for proposals: Inclusion, Diversity, and Empowerment: FAQ

Hi All
The recent call for proposals doesn't seem to specify whether the proposals can come from outside the US. I was thinking of suggesting to the ASA IDEA chapter that this could be a thing they could get behind, but I wanted to check that this would be possible first.

Proposals can come from anywhere. The money comes from the Moore foundation, which is a private foundation and does not restrict the use of the money to the US. However, the funds are hold in a US bank account, so we cannot transfer them to a country where the US imposes financial sanctions (e.g. Iran, North Korea).

Can you send a follow up email to make this clear though? The geographical homogeneity wasn't acknowledged in the email and missing out on clearly stating the almost global eligibility is not helping.

I've decided to wait till the weekend is over and see if any more questions come up so that I can answer them together (and point to this issue for any FAQ that come up after that).

Thank you @hamogu. Saying Western countries is better than saying  nothing, but let's zoom in on the numbers at least here on how much that actually newspeak for the USA.
75% of the coordinating committee are based in the NE USA and 80% of the finance committee. I don't have the numbers for contributors, but it wouldn't be surprising if ratios of contributors from the EU, Canada, and Australia would not reach their ratio within astronomy.

Note: Personal statement. Proposals will be discussed by the community, so my opinion is just one of many voices.
@bsipocz I think it is important to note that we deal with small number statistics though and with people moving around. E.g. even though I'm currently in the US, I did my PhD in Germany and maintain a lot of collaborations there. I think I can claim to understand how German and EU research funding works and what typical working conditions are. On the contrary, I know next to nothing about research work conditions and funding in India or South America. Similarly, I know that another finance committee member is on the way to move to a different continent in the next few weeks, so saying 100% in the US and 80% in the NE of the US is technically true as of today, but does not capture the true problem accurately.
To put it in other words: Personally, for geographic diversity, I'm not worried about the fact that, e.g. no scientist currently working in Denmark is on either committee, but about the fact that all members both come from "western" countries and work in "western" countries now (though not necessarily the same country).
I'm happy to continue this discussion in more detail, but since "geography" is only one (and one could argue not the most important) of several aspects to inclusion, diversity, and empowerment, I would prefer to have any follow-up discussion in  a different forum.

I picked on the word western while it should be USA as the original question in the thread was coming from Australia. Which is arguably a part of the world with high participation in astronomy, yet not really connected into the project, while traditionally considered a western country. I nowhere argued for the need of a person from Denmark or any certain countries to be on committees, but it's a very weak argument that one from 5 moving back to Europe will balance out anything even between just those two continents.
There are zillion reasons why it's like this and all I wanted is to have an acknowledgement of the fact of the NE US centricity and that there is a will in working on changing it. There has been multiple instances of exclusionary behaviour as a result of this, and I'm certain I only know about a subset of those.
So start this work, the bare minimum first step is to state clearly the rules of eligibility, and than has been done now.

(And it seems that disclaimers are necessary, but to be clear I would be the happiest to see proposals that would extend the project toward Africa, Asia, Australia, and South America and will in fact propose something that can work towards that).

I am glad that the question about eligibility was raised and clarified - it certainly was not clear in the email. And I share the assessment that Australia has a significant Astronomy presence but low participation in the astropy project.
Might be worthwhile to reach out to projects funded through IAU OAD, e.g., The Pan-African School for Emerging Astronomers

Since the proposal have been selected, I'm closing this issue.


[STREAMPIPES-<Jira issue #447>] Processing Element: Detect Value Change

Hi,
This is a draft pull request to push my updates incrementally and to seek help. I have created the folder structure and will be implementing the required methods. Any changes in the folder structure can be made.
Hrushikesh

Sure. I'll have a look at it.


BUG | run build

What happened?

When I start packing, it says require doesn't exist, but the project uses es Module syntax
Version
Latest
What OS are you seeing the problem on?
Windows
What browsers are you seeing the problem on?
Chrome
Relevant log output
No response
(OPTIONAL) Contact Details
No response
Code of Conduct

[X] I agree to follow this project's Code of Conduct
[X] I checked the current issues for duplicate problems.

@deno-source Can you check the lock file of your package manager and see which version of plasmo is pinned at?


Subtile snackbar messages

Problem
The snackbar messages, while useful for debugging, were a bit annoying and probably the feature causing most complaints...
Solution in this PR

Success messages are never automatically displayed -- users likely only care about errors or slow loading pages
The "loading" indicator is only shown when the requests takes longer than one second (this time is up for debate)
Some actions - like (Re)benchmark or "One Up" - always show a success message, as their success has no other visual indication in the UI
The "All repo" request does not show any indicator (except for errors) as it might be triggered quite frequently and should be fast

References / Closes
#19

Sounds great!


Quadratic MIP

Is it possible to use or-tools in python to solve quadratic MIP?
If yes, How? otherwise, I would really appreciate it if you could point me towards a solver that does so.

Hi,
Since Cbc seems to support QMIP
and OR-Tools embed Cbc. I will say it must be doable
Also look at https://github.com/google/or-tools/search?p=1&q=quadratic&type=&utf8=✓

I thought MIP was standard, but I can add a note saying that it's also known as MILP if that would help.
-------- Original Message --------
Subject: Re: [google/or-tools] Quadratic MIP (#598)
From: sam-at-github<EMAIL_ADDRESS>Date: Sun, December 09, 2018 1:23 am
To: google/or-tools<EMAIL_ADDRESS>Cc: Subscribed<EMAIL_ADDRESS>I was reading the docs trying to figure out exactly the same thing. Is it worth making it clearer in the doc the "MIP" actually means MILP? —You are receiving this because you are subscribed to this thread.Reply to this email directly, view it on GitHub, or mute the thread.

Not sure about what is standard. I have heard people just use MIP to mean MILP, but I think it is better to clarify. Thanks!


saving container image optional

Describe what this PR does / why we need it
Does this pull request fix one issue?
Fixes #1849
Describe how you did it
Describe how to verify it
Special notes for reviews

Codecov Report
Base: 21.65% // Head: 21.65% // No change to project coverage :thumbsup:

Coverage data is based on head (87643e8) compared to base (f9cc112).
Patch has no changes to coverable lines.

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #1851   +/-   ##
=======================================
  Coverage   21.65%   21.65%
=======================================
  Files          72       72
  Lines        6586     6586
=======================================
  Hits         1426     1426
  Misses       4974     4974
  Partials      186      186

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

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


Add X509_STORE_CTX_set0_store()

The existing function X509_STORE_CTX_set0_trusted_stack() allows the
application to swap out the set of trusted certificates, setting the
other_ctx field of the X509_STORE_CTX to a stack holding the certificates
in question.  However, that function causes lookup to degrade to a linear
search, whereas the native lookup uses an X509_STORE which has lhashes
for efficient lookup.
Advanced consumers may still have a need to replace the trusted certificates
but desire to retain the efficient lookup, so give them the ability to
set custom trusted certificates without compromising runtime efficiency.

Checklist

[x] documentation is added or updated
[ ] tests are added or updated
[x] CCLA is signed

Description of change
The main goal here is to swap out the trusted certificates without taking a performance hit.  But, X509_STORE_CTX->ctx is used for more than just trusted certificates, which is a little annoying but probably okay if documented.  Alternately, we could use other_ctx to hold the new store and define new wrapper functions for lookup and get_issuer, if we want to make a closer analogue of X509_STORE_CTX_set0_trusted_stack() (presumably we would also change the name of the new function to X509_STORE_CTX_set0_trusted_store() in that case).

Needed to 'make update'.
I went with 1_1_0d since we could claim this is an accessor needed due to the opaquification.

You might want to base this on master, and then backport it.

I'm not sure what the intended use of this change is. The X509_STORE_CTX structure can now only be used once for verification of certificates. If you want a different store why not just specify it in X509_STORE_CTX_init() in the first place?

My guess:
SSL users don't get to specify the X509_STORE_CTX, which is created by the SSL library.  But they do get to specify an app_verify callback, typically to wrap the call to X509_verify_cert().

@vdukhovni is right.

rebased and added a link to X509_STORE_set_verify_cb_func, which seems to have more discussion of what it actually does than X509_store_new does.

rebased past the libcrypto.num conflict.

rebased past 1.1.0d (not that that will be good for very long :) )  @vdukhovni understands the use case, but does he think it's a good idea?

Seeing all the other roles that a store plays, I'm concerned that this is too heavy-weight a mechanism for the purpose at hand.  Perhaps we need a pluggable cert/crl interface with user provided access methods which has fewer collateral consequences.
It looks like what we have here is a stopgap solution to a missing more appropriate interface.  Perhaps we'd be better off not adopting the stop-gap, and doing a better design.

I had considered also updating the function pointers that are initialized in X509_STORE_CTX_init(), but decided not to on the grounds that the propsed new API is just to set the backing store, and not completely reinitialize the structure.  Perhaps this would be more clearly indicated by using the other_ctx field after all, but it's not entirely clear whether that is actually a "better design".
I wonder how @levitte 's generic STORE work would impact this issue; unfortunately, I haven't had a chance to look at it closely yet.

ping @vdukhovni and @kaduk to resolve their conflict

In the absence of a design for a "more appropriate interface", I'm happy to rewrite this as an X509_STORE_CTX_set0_trusted_store() that uses the other_ctx field and avoids conflating the storage role with the other roles of the X509_STORE.  But maybe @vdukhovni will still want to hold out for that more appropriate interface to appear.

Ping @kaduk ?

I think I want @levitte to tell me whether the new STORE stuff is going to be usable in the SSL verify callback anytime soon, which would render the whole thing fairly moot.

If others could actually spend some time reviewing the STORE stuff I do, that might take it a step forward.  One of the PRs I've worked on lately (mostly because it's among the least rad) is #8442, which doesn't exclude the use of the X509_STORE stuff.  Still, it's currently fairly unreviewed, except for a comment from @petrovr...

Are you still interested in having this feature included in OpenSSL? If so please rebase the PR to resolve conflicts.
Marking as inactive to be closed at the end of 3.4 dev barring further input.


The zone.js.d.ts file does not work with webpack

I tried using zone.js in webpack , see my codes here:
import "reflect-metadata";
import "zone.js";
import { Component } from '@angular/core';
import { bootstrap }    from '@angular/platform-browser-dynamic';
@Component({
selector: 'my-app',
template: '\<h1\>for test\</h1\>'
})
class AppComponent { }
bootstrap(AppComponent);
the typescript compiler throw error:
error TS2656: Exported external package typings file '/Users/pjl/ng2-webpack-seed/node_modules/zone.js/dist/zone.js.
d.ts' is not a module. Please contact the package author to update the package definition.

i have the problem too.

The problem is not with Webpack. I have this problem while using Browserify, too.
I think this problem is the same as #297 and #365, and I still don't understand how it should work!

Check @IgorMinar's answer at #365.
You have to load zone.js from the first script tag of you index.html file. Do not import it like the way describe above.

It looks like this issue could be closed, by now


Add secret alias support for DSS editor

Purpose
Adding secret alias support for DSS editor

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.

 All committers have signed the CLA.


Fix reference to packed field is unaligned

error: reference to packed field is unaligned
--> src/codec.rs:187:35
|
187 |                     let payload = &event.u.start;
|                                   ^^^^^^^^^^^^^^
|
= note: #[deny(unaligned_references)] on by default
= warning: this was previously accepted by the compiler but is being phased out; it will become a hard error in a future release!
= note: for more information, see issue #82523 https://github.com/rust-lang/rust/issues/82523
= note: fields of packed structs are not properly aligned, and creating a misaligned reference is undefined behavior (even if that reference is never dereferenced)
= help: copy the field contents to a local variable, or replace the reference with a raw pointer and use read_unaligned/write_unaligned (loads and stores via *p must be properly aligned even when using raw pointers)

@flukejones , heads up, this will become relevant to your prioritized users soon [1].
Please review when you get a chance.
Also, the package version captured in this repo does not match the package version published on crates.io [2].
[1] https://doc.rust-lang.org/error_codes/E0793.html
[2] https://crates.io/crates/uhid-virt


Two Things: Adds Money Display Name support and craft/anti craft support

(also switched to materials instead of ids in config)
I added these because there have been other requests for similar features and I need these for the server I'm building. Having the currency be uncraftable/ungatherable gives me more control of economy inflation.

Wow, this is great! I will test it and release next week if possible. Thanks so much :-)

Sounds great, I Had not found any bugs thus far but if you find any issues I'll be happy to fix them. Good luck!


Adds auth file example

Fixes https://github.com/Azure-Samples/azure-sdk-for-go-samples/issues/84
Thanks for your contribution! Please let us know what problems this PR addresses or what functionality it adds. Please link to other related issues and PRs to speed up review. Thank you!

BTW, I have also modified the jenkins job for tests so they can use the authfile :)


[docs-infra] Improve type naming on the API pages

We're using a shorthand notation to mention to type names that seem to be about React's prop-types, like bool.

https://mui.com/material-ui/api/fade/#fade-prop-appear
But it seems to make more sense to use the full name for it to be more accurate with how TypeScript refers to them — e.g., boolean.
This naming design optimizes for a time when most developers would write "propType". Today, these developers are a lot more likely to write TypeScript types, so it will bring a more intuitive DX to make the change:

bool → boolean
func → function
etc.

Search keywords: type naming, api page

Hello,
I agree, it think it makes more sense to use the full name in order to be more accurate with how TypeScript refers to them.
If this improvement is going to be implemented, I would like to help.

@EyaOuenniche sure, yeah! Feel free to kickstart it and ask for a review :)
Let me know if you need any help, and I can give you pointers!

Great! Thank you.
I changed the type names on the API pages for the fade component and I just created a pull request.

@EyaOuenniche hey there! The Fade page linked in the issue is just one illustrative example — the correct approach would be to change in a place where the entire documentation platform is affected (not just the Fade API page).
I've looked into it myself and couldn't precisely figure out where to go to change that, so I'll summon @mui/docs-infra for further help :)

I can see two places. The first one generate the type of the props.
The second recursively resolves composed types.
https://github.com/mui/material-ui/blob/125020839ea4209fc98cba06115de6dc02e1c559/packages/api-docs-builder/utils/generatePropTypeDescription.ts#L146
https://github.com/mui/material-ui/blob/125020839ea4209fc98cba06115de6dc02e1c559/packages/api-docs-builder/utils/generatePropDescription.ts#L53-L55


read_gbq results in lingering system thread after function call

Thanks for stopping by to let us know something could be better!
PLEASE READ: If you have a support contract with Google, please create an issue in the support console instead of filing on GitHub. This will ensure a timely response.
Please run down the following list and make sure you've tried the usual "quick fixes":

Search the issues already opened: https://github.com/googleapis/python-bigquery-pandas/issues
Search StackOverflow: https://stackoverflow.com/questions/tagged/google-cloud-platform+python

If you are still having issues, please be sure to include as much information as possible:
Environment details

OS type and version: Linux CentOS 7
Python version: 3.10
pandas-gbq version: 0.19.0

Steps to reproduce

Start debugger session
Run read_gbq function
Look at Call Stack after function execution and notice extra running system thread.

Code example
ret_df = pd.read_gbq(
                query_str,
                project_id=bq_wrap.bq_billing_project, #Billing project
                configuration={'query':{'defaultDataset':{"datasetId": profile.bq_dataset, "projectId": bq_wrap.bq_project}}},
                credentials=creds,
                use_bqstorage_api=use_bqstorage_api,
                progress_bar_type='tqdm')

Stack trace

Having the same issue! Its also leading to memory leaks for me. These extra threads seem to be holding references to data that read_gbq returns - preventing the garbage collector from removing it.

Thanks for reporting the issue! I am able to reproduce it, but it seems to only happen when tqdm is used.

It seems tqdm opens a new thread when an tqdm object is created, but it's not closed when tqdm is closed.
import tqdm

# Create a list of numbers
numbers = list(range(3))

# Create a progress bar
pbar = tqdm.tqdm(numbers)

# Iterate over the list of numbers
for number in pbar:
    # Do something with the number
    pass

# Close the progress bar
pbar.close()

# There is a "Thread-7" at this breakpoint
breakpoint()

exit(0)

This is caused by tqdm creating a new thread with class TMonitor while creating a new tqdm.tqdm object. A way to patch this is to set tqdm.tqdm.monitor_interval = 0 before using it - for example just after the library is imported. But overall I think it's a bug with tqdm. I opened an issue at tqdm, so I will close this one. Still, please leave a comment or open a new issue if you have any questions :)