sc.datasets.pbmc68k_reduced needs updating

[x] I have checked that this issue has not already been reported.
[ ] I have confirmed this bug exists on the latest version of scanpy.
[ ] (optional) I have confirmed this bug exists on the master branch of scanpy.

Minimal code sample
import scanpy as sc
import numpy as np
import pandas as pd
adata = sc.datasets.pbmc68k_reduced()
sc.pl.pca_loadings(adata, components = '1,2,3')

ValueError: Axis limits cannot be NaN or Inf

Note:  recalculating pca solves the problem
Versions

anndata     0.7.5
scanpy      1.8.0.dev78+gc488909a
sinfo       0.3.1
PIL                 7.2.0
anndata             0.7.5
appnope             0.1.0
attr                19.3.0
backcall            0.2.0
bottleneck          1.3.2
cffi                1.14.0
cloudpickle         1.5.0
colorama            0.4.3
cycler              0.10.0
cython_runtime      NA
cytoolz             0.10.1
dask                2.20.0
dateutil            2.8.1
decorator           4.4.2
get_version         2.1
h5py                2.10.0
idna                2.10
igraph              0.8.3
ipykernel           5.3.2
ipython_genutils    0.2.0
ipywidgets          7.5.1
jedi                0.17.1
jinja2              2.11.2
joblib              0.16.0
jsonschema          3.2.0
kiwisolver          1.2.0
legacy_api_wrap     1.2
llvmlite            0.33.0+1.g022ab0f
louvain             0.7.0
markupsafe          1.1.1
matplotlib          3.2.2
mkl                 2.3.0
mpl_toolkits        NA
natsort             7.1.0
nbformat            5.0.7
numba               0.50.1
numexpr             2.7.1
numpy               1.18.5
packaging           20.4
pandas              1.0.5
parso               0.7.0
pexpect             4.8.0
pickleshare         0.7.5
pkg_resources       NA
prometheus_client   NA
prompt_toolkit      3.0.5
psutil              5.7.0
ptyprocess          0.6.0
pvectorc            NA
pygments            2.6.1
pyparsing           2.4.7
pyrsistent          NA
pytz                2020.1
scanpy              1.8.0.dev78+gc488909a
scipy               1.5.0
seaborn             0.10.1
send2trash          NA
setuptools_scm      NA
sinfo               0.3.1
six                 1.15.0
sklearn             0.23.1
sphinxcontrib       NA
statsmodels         0.11.1
storemagic          NA
tables              3.6.1
tblib               1.6.0
texttable           1.6.3
tlz                 0.10.1
toolz               0.10.0
tornado             6.0.4
traitlets           4.3.3
typing_extensions   NA
wcwidth             0.2.5
yaml                5.3.1
zmq                 19.0.1
zope                NA
IPython             7.16.1
jupyter_client      6.1.6
jupyter_core        4.6.3
jupyterlab          2.1.5
notebook            6.0.3
Python 3.8.3 (default, Jul  2 2020, 11:26:31) [Clang 10.0.0 ]
macOS-10.15.7-x86_64-i386-64bit
4 logical CPU cores, i386
Session information updated at 2021-04-27 11:22

sc.datasets.pbmc68k_reduced is very useful as an example of a very old anndata file, so I'd like to keep it around.
Could you use pbmc3k_processed for this? It hasn't had highly_variable_genes run on it anyways (though maybe it should).

sure, makes sense, thanks!


AVRO-2104: Schema normalization and fingerprinting for Python 3.

An implementation of schema normalization and fingerprinting for Python
3. The code is a fairly literal translation of the Java code and is
entirely additional to the existing code, except for one small change
in the way schema.Name is implemented to improve compatibility with the
Java version.
The new implementation is covered by 138 unit tests which have been
automatically generated by a script (not provided) which executed the
Java Avro normalization and fingerprinting code to provide expected
values for the Python implementation to reproduce.

A link back to the corresponding Jira issue: https://issues.apache.org/jira/browse/AVRO-2104

Looks great @rob-smallshire, can you rebase against master?


On the Login page, a structure containing Email and passWord in Card

Added first steps of simple login page

Do you want

Don't forget to add a description to every commit, issue, PR, comment, etc.
Please edit the description area. @akadir8

Closess basic login page draft


Rocket.Chat Desktop Application quits when trying to join a call

My Setup

Operating System: macOS 10.15.5 & macOS 11.0
App Version: 3.0.x
Installation type: homebrew

[x] I have tested with the latest version
[x] I can simulate the issue easily

Description

Rocket.chat app crashed when clicking a button "Click to Join!" to join a Jitsi call. Problem does not occur when call has already ended (Call already ended notification appears).
Problem does not occur under macOS 10.13.6
Current Behavior

Program quits
Expected Behavior

Open Jitsi call window

Maybe that is your problem https://github.com/RocketChat/Rocket.Chat.Electron/issues/1841

Not sure, might be a different problem as the Meeting Window does not open for me at all but the Rocket.Chat Electron Application quits unexpectedly.

I have the same problem:
OSX 10.15.7
RC client: 3.07
RC server: 3.9.1 , "Jitsi" -> "Always Open in New Window" is on.

Update.
If I start the client from a terminal (iTerm) then after clicking join button I have got a popups asking to allow access of iTerm to microphone and camera. Video call worked fines. Next I started the directly and again had the same issue of client crashing when joining a video call.
So it seems to be permissions issue.

If I start the client from a terminal (iTerm) then after clicking join button I have got a popups asking to allow access of iTerm to microphone and camera. Video call worked fine. Next I started the client directly and again had the same issue of client crashing when joining a video call.

I'm able to reproduce this behavior

The issue is still present in 3.1.1

We have this problem too.
@ggazzo any idea?

rocket.chat have not any feedback 👎

Same issue here.  Launching the app by clicking on it in Finder leads to video crashing.
However, starting the app from a terminal via /Applications/Rocket.Chat.app/Contents/MacOS/Rocket.Chat it asks for permissions and works ok.  Unfortunately it still crashes if I subsequently re-launch the app from spotlight/finder.

We also have this issue we can reproduce with macOS 10.11. We've looked at the tcc log file with /usr/bin/log show -style syslog --predicate 'subsystem == "com.apple.TCC"' --info --last 10m and noticed this part:

This app has crashed because it has a hardened runtime and attempted to access privacy-sensitive data without an entitlement indicating its intent to access this data.  The app must have the 'com.apple.security.device.audio-input' entitlement.

The pkg and dmg downloads have only 3 entitlement keys, nothing for audio or video. The MAS version has for more entitlements (15 keys, including com.apple.security.device.audio-input, com.apple.security.device.camera and com.apple.security.device.microphone) but the latest App-Store version I can get is 3.0.3 (3.2.3 is actual).
May this issue be solved, if the right entitlements would also be defined in the non MAS version?

@cazetto @ankar84
this problem still exists ... whatever version you try to install on a fresh macos ... the app is not asking for rights to use cam, mic or screensharing so the app is crashing as soon as you start a jitsi call
i tried a lot different versions of rocketchat.app releases including the last 2.17.11 and even older ones. no one of them had any entitlements set for cam or mic or screensharing. even the last one available on app store has this problem.
Can you please fix that with high prio cause it seems like you are just not able to use rocketchat + jitsi on any new set up macos!
Thx in advance

https://github.com/RocketChat/Rocket.Chat.Electron/blob/master/build/entitlements.mas.plist this is what is needed in the .dmg .pkg app for mac

@dudanogueira Hey, Duda!
Please help us to fix that and all related annoying issues with ready to go fix #2191 by @chrisi51
Thank you!

----- Original Message -----

@dudanogueira ( https://github.com/dudanogueira ) Hey, Duda!
Please help us to fix that and all related annoying issues with
ready to go fix #2191 (
https://github.com/RocketChat/Rocket.Chat.Electron/pull/2191 ) by @
chrisi51 ( https://github.com/chrisi51 )
Thank you!
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub (
https://github.com/RocketChat/Rocket.Chat.Electron/issues/1912#issuecomment-969901716 )
, or unsubscribe (
https://github.com/notifications/unsubscribe-auth/AB4UXCM6SBVYCF54X3M6WFTUMHZRPANCNFSM4UU3YT2A )
.
Triage notifications on the go with GitHub Mobile for iOS (
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 )
or Android (
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub )
.

Quy Le,

?


Implement better pagination for our GitHub Actions

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Context
Our GitHub Actions interact with the GitHub API but lack pagination.
Action Performed:
There's not super clear reproduction steps here, but when generating the QA/deploy checklist (always labeled StagingDeployCash), we need to cross-check the list of PRs and make sure we don't include PRs created by our CI/CD bot user, OSBotify. We do that here.
However, we only fetch the latest 100 PRs, while there may be more than 100 PRs merged between deploy cycles (especially including all the automated bot ones). As a result, we include some automated PRs in the deploy checklist, which we do not want to do.
Expected Result:
We need to fetch all PRs between the oldest merged PR from the list passed in here and the newest PR. We can do that by fetching 100 PRs from the API at a time, until the oldest PR we have from the API is older than the oldest PR we were passed in.
Actual Result:
We don't fetch more than 100 results.
Workaround:
Ignore or manually delete automated PRs from the deploy checklist.
Platform:
Where is this issue occurring?
GitHub only.
view the job on upwork here.
View all open jobs on Upwork

Triggered auto assignment to @jboniface (External), see https://stackoverflow.com/c/expensify/questions/8582 for more details.

Triggered auto assignment to @johnmlee101 (Exported), see https://stackoverflow.com/c/expensify/questions/7972 for more details.

I will try to increase the number of pages retrieved by pagination or work on a recursive solution. I cannot guarantee this solution will work because it might be the case that Github prevents greater then 100 item numbers.
Here is an example with a different github api but similar case:
https://stackoverflow.com/questions/18148490/how-can-i-get-more-than-100-results-from-github-api-v3-using-github-api-gem
But I am happy to investigate and see what is possible.

Yes, GitHub prevents greater than 100 items per fetch – you could use potentially use recursion to fetch all the results you need, but I would probably use a while loop.

Ok, If I understood it correctly, We already have a PRList which want to put on the body on the QA/deploy checklist but without the automated PRs.
So we call GitHub to get a list of all PRS and then Find AutomatedRequests out of those. Then we remove these found automated requests from the PRList. but the issue is We only get 100 requests.
So we want to get all PRs here so that we can filter all the Automated requests. Yeah, we can use https://octokit.github.io/rest.js/v18#pagination to get all the PRs which should solve the issue at hand.
But We will need to only get the PRs list that occurred between the oldest merged and Newest merged. SO we can quit early in https://octokit.github.io/rest.js/v18#pagination which allows us to stop at any time.
For Eg:
octokit.paginate("GET /organizations", (response, done) => {
  if (response.data.find((issues) => issue.body.includes("something"))) {
    done();
  }
  return response.data;
});

Nice find with https://octokit.github.io/rest.js/v18#pagination! Ideally we could make the paginator fetch all the results we need and return a single promise, such that we:

Fetch all the PRs at until we get to PRs older than the oldest one on our PRList
Then remove any automated ones from our PRList
Then continue and generate the StagingDeployCash issue body.

Maybe you could use promiseWhile to create a wrapper around the paginated method or something

octokit.github.io/rest.js/v18#pagination! already return a promise So I don't think we need that.

@jboniface Can you please hire @parasharrajat for this on Upwork? Thanks!


Building on Android requires QT_CREATOR_SKIP_CONAN_SETUP

In order to build successfully on Android with Qt Creator, you need to specify QT_CREATOR_SKIP_CONAN_SETUP=On.
The build fails on the benchmark dependency.
This might be able to be resolved by converting conanfile.txt to conanfile.py and dynamically disabling those dependencies on Android. It looks like Conan is not using a cross compiler when it should be:
-- Conan executing: /usr/local/bin/conan install /Users/dmeybohm/projects/cpp/wisdom-chess/conanfile.txt -s build_type=Release -s compiler=clang -s compiler.version=11 -s compiler.libcxx=libc++ -e=CONAN_CMAKE_TOOLCHAIN_FILE="/Users/dmeybohm/projects/cpp/build-wisdom-chess-Android_Qt_6_2_4_Clang_x86_64-Release/conan-dependencies/toolchain.cmake" -g=cmake_paths -g=json -g=cmake --build=missing -if=/Users/dmeybohm/projects/cpp/build-wisdom-chess-Android_Qt_6_2_4_Clang_x86_64-Release/conan-dependencies
Configuration:
[settings]
arch=x86_64
arch_build=x86_64
build_type=Release
compiler=clang
compiler.libcxx=libc++
compiler.version=11
os=Macos
os_build=Macos
[options]
[build_requires]
[env]
CONAN_CMAKE_TOOLCHAIN_FILE="/Users/dmeybohm/projects/cpp/build-wisdom-chess-Android_Qt_6_2_4_Clang_x86_64-Release/conan-dependencies/toolchain.cmake"
benchmark/1.6.0: WARN: Package binary is corrupted, removing: a1e86c206430932d7cbcad958cb43a8af16afc46
conanfile.txt: Installing package
Requirements
    benchmark/1.6.0 from 'conancenter' - Cache
    doctest/2.4.6 from 'conancenter' - Cache
    ms-gsl/4.0.0 from 'conancenter' - Cache
    sqlite3/3.38.0 from 'conancenter' - Cache
Packages
    benchmark/1.6.0:a1e86c206430932d7cbcad958cb43a8af16afc46 - Build
    doctest/2.4.6:5ab84d6acfe1f23c4fae0ab88f26e3a396351ac9 - Cache
    ms-gsl/4.0.0:5ab84d6acfe1f23c4fae0ab88f26e3a396351ac9 - Cache
    sqlite3/3.38.0:24e8071b29b991d623d760671a655983a3d0c8ec - Build

Installing (downloading, building) binaries...
benchmark/1.6.0: WARN: Build folder is dirty, removing it: /Users/dmeybohm/.conan/data/benchmark/1.6.0/_/_/build/a1e86c206430932d7cbcad958cb43a8af16afc46
benchmark/1.6.0: Copying sources to build folder
benchmark/1.6.0: Building your package in /Users/dmeybohm/.conan/data/benchmark/1.6.0/_/_/build/a1e86c206430932d7cbcad958cb43a8af16afc46
benchmark/1.6.0: Generator cmake created conanbuildinfo.cmake
benchmark/1.6.0: Aggregating env generators
benchmark/1.6.0: Calling build()
/bin/sh: cmake: command not found
benchmark/1.6.0:
benchmark/1.6.0: ERROR: Package 'a1e86c206430932d7cbcad958cb43a8af16afc46' build failed

This issue is fixed from https://github.com/dmeybohm/wisdom-chess/pull/19


Installing upgrade assistant only returns NU1101 error

Problem encountered on https://dotnet.microsoft.com/platform/upgrade-assistant/tutorial/install-upgrade-assistant
Operating System: windows
Just installed VS22 (production v17, not the preview) and running dotnet displays the expected help. But when I run dotnet tool install -g upgrade-assistant, I get NU1101: Unable to find package upgrade-assistant.
I've even tried explicitly downloading and installing the SDK (although I'd think VS would do that for me), same result. I'm running WIn10 21H2 with all patches (including Nov's patch Tuesday). And I've created and run some simple VS projects without an issue. Where/how do I install this package as the referenced source implies it should just work ;-)

@BizD3V Are you able to install any other nuget packages?

Thanks Sunanda! That simple question pointed me in the right direction. Somehow (like right out of the box) my system had no online package sources. Once I added it, everything is working. But that does leave me trying to find the "definitive" location for Nuget.org packages. Do you happen to know if it's still https://api.nuget.org/v3/index.json?

Thanks Sunanda! That simple question pointed me in the right direction. Somehow (like right out of the box) my system had no online package sources. Once I added it, everything is working. But that does leave me trying to find the "definitive" location for Nuget.org packages. Do you happen to know if it's still https://api.nuget.org/v3/index.json?

Yes that would be the right package source for nuget.org
<add key="nuget.org" value="https://api.nuget.org/v3/index.json" />

TY Appreciate the quick resolution.


[Fix] #1173 Nginx SSL redirect loop

Headers not being passed was causing nginx to redirect to SSL again & again.

Tested on my production setup.


ctx undefined

In two examples on this web page, the variable "ctx" is used but is neither defined nor initialized.  It is doubtful the code would compile (it likely needs to be changed to "context").

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: bd81df21-6f7d-9822-21a4-7098043e36f0
Version Independent ID: d8b04662-9561-9cfc-a635-cbba170213c4
Content: Handling errors in Durable Functions - Azure
Content Source: articles/azure-functions/durable/durable-functions-error-handling.md
Service: azure-functions
GitHub Login: @ggailey777
Microsoft Alias: azfuncdf

@tscheffel Thanks for reporting this and helping us drive improvements towards MS Docs! I've created a PR that updates the samples and the changes should be live shortly after the merge occurs.
We will now proceed to close this thread. If there are further questions regarding this matter, please reopen it and we will gladly continue the discussion.


Fix triton kernel performance regression

Motivation
We found triton kernel of ROCm platform has a performance drop on the triton-3.1.0 version.
Force roll-back to use triton-3.0.0 version to work-around this issue.
Modifications
Checklist

[+] Format your code according to the Contributor Guide.
[+] Add unit tests as outlined in the Contributor Guide.
[+] Update documentation as needed, including docstrings or example tutorials.

@HaiShaw :
Please help to review it.
Thanks


Make unit test for python 3

Make unit test for python 3

rolling this into acceptance criteria for #9


Update program-list.json

Summary
< A quick summary of the associated pull request >
Quality Assurance Checklist
< Confirmation of this pull request meeting the following requirements, prior to merge >

Review Items
Y/N

Site has a publicly known bug bounty or vulnerability disclosure program

Disclosure policy is publicly available

Public URL

Submission follows the Diodb schema

Your Twitter handle (Optional):

This is a new update to https://github.com/disclose/diodb/pull/405

Thank you!


⚠ ClamAV has degraded performance

In f7c7595, ClamAV (https://spamassassin.apache.org/) experienced degraded performance:

HTTP code: 200
Response time: 203 ms

Resolved: ClamAV performance has improved in 2a915e5.


update subgraph Protocol entity with new fields

What does this pull request do? Explain your changes.
This PR adds a new set of fields to the Protocol entity and introduces 4 new entities.
Specific updates

Adds the following fields to the Protocol entity:
currentRound, inflation, inflationChange, lastInitializedRound, lastRoundLengthUpdateRound, lastRoundLengthUpdateStartBlock, lockPeriod, maxEarningsClaimsRounds, numActiveTranscoders, paused, roundLength, roundLockAmount, targetBondingRate, unbondingPeriod
Introduces the following new entities:
SetCurrentRewardTokens, Pause, Unpause, ParameterUpdate

How did you test each of these updates
The changes haven been deployed and synced on the Livepeer Canary subgraph
Does this pull request close any open issues?
#437
Checklist:

[x] I have read the CONTRIBUTING document.
[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.
[ ] I have added tests to cover my changes.

Changes look good, can we squash the commits on merge?


Pick specific trigger for a specific container

Is it possible to say which triggers I want to apply to which containers?
eg I don't mind if my dashboard gets all updates automatically but I don't want my authentication tool to get anything but minor updates & patches automatically.

Hi,
Unfortunately, it's not supported.
I find your use case valid; I tag your issue as "enhancement".
(I don't have any ETA though...)

Implemented.
Examples:
services:

  traefik1:
    image: traefik:3.0.0
    labels:
      - wud.trigger.include=ntfy.one:major

  traefik2:
    image: traefik:3.0.0
    labels:
      - wud.trigger.include=ntfy.two

  traefik3:
    image: traefik:3.0.0
    labels:
      - wud.trigger.exclude=ntfy.three

An additional tab has been added to the UI to give a look at the triggers associated to each container

It does not work
I have the following trriger:
    - WUD_TRIGGER_PUSHOVER_CGM1_MODE=batch

and then in my service:
services:
syncthing:
labels:
- wud.trigger.exclude=pushover.cgm1
But the trigger is still associated after a complete remove and recreation of wud


Fixed misaligned icons

On the panel with mute, deafen, and settings the icons were a little too high, so I moved them down a little.
Before:

After:

Fixed in latest update


Fix direct event field reference error

Logstash 5.0 error:
Error: Direct event field references (i.e. event['field']) have been disabled
in favor of using event get and set methods (e.g. event.get('field')). Please
consult the Logstash 5.0 breaking changes documentation for more details

https://www.elastic.co/guide/en/logstash/5.0/breaking-changes.html#_ruby_filter_and_custom_plugin_developers

Travis build is broken: public_suffix-2.0.4 requires ruby version >= 2.0, which is incompatible with the current version, ruby 1.9.3p551 (jruby 1.7.25)

Please get this through...

Can we fix the build?


Add hgvsg support to region vep only

https://github.com/genome-nexus/genome-nexus/pull/320 needs to be merged first
supersedes:
https://github.com/genome-nexus/genome-nexus/pull/314

Codecov Report

Merging #321 into master will decrease coverage by 4.51%.
The diff coverage is 22.58%.

@@             Coverage Diff              @@
##             master     #321      +/-   ##
============================================
- Coverage     21.75%   17.24%   -4.52%
+ Complexity      180      136      -44
============================================
  Files           169      169
  Lines          2211     2233      +22
  Branches        287      290       +3
============================================
- Hits            481      385      -96
- Misses         1687     1811     +124
+ Partials         43       37       -6

Impacted Files
Coverage Δ
Complexity Δ

...me_nexus/web/mixin/TranscriptConsequenceMixin.java
0% <ø> (ø)
0 <0> (ø)
:arrow_down:

...g/cbioportal/genome_nexus/util/GenomicVariant.java
0% <ø> (ø)
0 <0> (?)

...exus/util/exception/TypeNotSupportedException.java
0% <ø> (ø)
0 <0> (?)

...ome_nexus/util/exception/InvalidHgvsException.java
0% <ø> (ø)
0 <0> (?)

...s/util/exception/RefTypeNotSupportedException.java
0% <ø> (ø)
0 <0> (?)

...e/cached/CachedVariantRegionAnnotationFetcher.java
0% <0%> (ø)
0 <0> (ø)
:arrow_down:

...ioportal/genome_nexus/util/GenomicVariantUtil.java
0% <0%> (ø)
0 <0> (?)

..._nexus/component/annotation/NotationConverter.java
0% <0%> (ø)
0 <0> (ø)
:arrow_down:

...oportal/genome_nexus/web/AnnotationController.java
52.63% <37.5%> (-16.6%)
4 <1> (-1)

...xus/service/internal/MyVariantInfoServiceImpl.java
35.13% <66.66%> (-0.16%)
5 <1> (ø)

... and 2 more

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 a1bf2ad...421204a. Read the comment docs.


I am frist use react native ,I have a problem ,Cannot find entry file index.android.js in any of the roots

{"message":"Cannot find entry file index.android.js in any of the roots: ["D:\\OK"]","name":"NotFoundError","type":"NotFoundError","status":404,"errors":[{}]}
I use win7 ,

Hey, thanks for reporting this issue!
It looks like your description is missing some necessary information, or the list of reproduction steps is not complete. Can you please add all the details specified in the template? This is necessary for people to be able to understand and reproduce the issue being reported.
I am going to close this, but feel free to open a new issue with the additional information provided. Thanks!


The name and/or id elements can not be added to login button for test automation

The login page form has the following: "Log In" but developer can not add a name or id so can be used for test automation.  The class element could be used but has potential for changes and not fun to find a specific one in the list each time.

It would be helpful if each grommet component had a data-debug-id field for test automation to use for testing.

The data-debug-id could be used if static and include a unique id to component but I have not seen this used consistently by developers.  Sometimes this id is of a dynamic object id which is not helpful since we can not know beforehand.

Much of this will be handled by the resolution of https://github.com/grommet/grommet/issues/85.  We will use this issue to track the specifics of LoginForm since that is not one of the initial components we'll be addressing.

The recommendation is to use query selector to grab the elements without the need for the id.
https://developer.mozilla.org/en-US/docs/Web/API/Document/querySelector
For example:
  document.querySelector('.my-login-form button');

We avoid adding ids inside these components to not face issues with duplicated id in the DOM. LoginForm is a higher-level component which will most likely end up moving to grommet-addons.

Just a note this isn't recommend for TA scenarios. Class names are much more volatile than named locators per selenium guidelines, especially as dom structure changes during refactoring. Class names are low on the list of recommendations to use.
https://mestachs.wordpress.com/2012/08/13/selenium-best-practices/
ID is one option, another option is data-ta-id to give it a named locator
e.g.
data-ta-id="login"
data-ta-id="username" or data-ta-id="username_input"

I see your point, but I don't think this is a good idea for production build either. In this case you are adding debugging information that will be most likely present in your production site (increasing the bundle size).
In the case of a LoginForm, the data-ta-id would need to be bundled with it which can lead to side effects like two login forms in a page warning about duplicated ids.
We talked a lot about this in a recent meeting with the Grommet team. If you want to add the id for LoginForm, our current recommendation is to copy it from the Grommet source and move it to your project. As I said, these high-level components will be soon moved to grommet-addons.

@alansouzati thanks for the update and consideration


I want to change the storagePolicyName used in the default StorageClass deployed in VMware

What would you like to be added:
I want to change the storagePolicyName used in the default StorageClass deployed in VMware.
Why is this needed:
The default storage class deployed to VMware clusters is hardcoded to use the default storage policy in VMware. Many administrators disable that policy and deploy custom policies in their VMware environments.
Note that there is an issue mentioning this #1353 currently. However, that issue refers to the VSphereMachineConfig, which has nothing to do with Kubernetes Storage Policies.

(I am copying my design suggestion over from the previous issue to this one)
I think it might be a bit unclear if we were defaulting to a storgePolicyName defined on nodes, when nodes really have nothing to do with the Kubernetes StorageClasses.
I would rather see us be explicit and define a StorageClass in the cluster.yaml at creation time, essentially overriding the default specified here
For example, in my cluster config (cut for brevity)...
---
apiVersion: anywhere.eks.amazonaws.com/v1alpha1
kind: Cluster
metadata:
  name: ray
spec:
  ...
  defaultStorageClassRef:
  - kind: StorageClass
    name: standard
---
apiVersion: storage.k8s.io/v1
kind: StorageClass
metadata:
  name: standard
  annotations:
    storageclass.kubernetes.io/is-default-class: "true"
provisioner: csi.vsphere.vmware.com
parameters:
    storagePolicyName: "My Storage Policy"

---


Database <IP_ADDRESS> box should use template for xe.rsp

The install.sh script for the Oracle Database <IP_ADDRESS> box modifies the ORACLE_PASSWORD and ORACLE_CONFIRM_PASSWORD values in the xe.rsp file with the generated Oracle password (or the password set in the ORACLE_PWD envrionment variable, if applicable).  This causes two problems:

After the first vagrant up, git treats the xe.rsp file as modified.
It's more difficult to start over after a vagrant destroy.  Unless xe.rsp is edited to change the ORACLE_PASSWORD and ORACLE_CONFIRM_PASSWORD values back to "###ORACLE_PWD###", subsequent vagrant up operations will use the password generated the first time vagrant up was run.  This happens because the string "###ORACLE_PWD###" no longer exists in xe.rsp, so sed can't replace it with the new password).

One solution is to use a template for xe.rsp.  This is how response files are handled for the Oracle Database <IP_ADDRESS> box.  The following changes would fix the issue:

Rename the existing xe.rsp file to xe.rsp.tmpl.
Add one line of code to install.sh to copy xe.rsp.tmpl to xe.rsp before replacing the "###ORACLE_PWD###" strings with the generated password.
Add one line of code to install.sh to delete xe.rsp after the database is created.

If you agree that this is a legitimate issue, and that the suggested changes are the right way to address it, I'd be happy to submit a pull request.

To (hopefully) save you some time, I've submitted a pull request for this.  Handling the response file the same way as the Oracle Database <IP_ADDRESS> box fixes the issue, and it's a straightforward change.

Merged!


Added gitignore-file for Beckhoff TwinCAT3.

Provide a link to the application or project’s homepage:
https://www.beckhoff.com/english.asp?twincat/twincat-3.htm
TwinCAT3 is a realtime kernel and development environment for PLCs (programmable logic controller). The IDE is based on Visual Studio, which TwinCAT is integrated into. It's one of the more popular development environments for industrial automation.
Provide links to documentation:
The official one:
https://download.beckhoff.com/download/document/automation/twincat3/TC3_SourceControl_EN.pdf
This only covers part of the files (specifying that TMC and TPY-files are not necessary to version control). Unfortunately, even Beckhoff (the manufacturer of TwinCAT) have failed to properly document which files should be version controlled and not.
Regarding the other files included in the gitignore-file of the pull request:

*.tclrs - These are license response files, basically license files that are residing on the target PLC device
*.compiled-library and *.library - which are libraries of which the project depends on. These are put here when they are referenced in the project, but the actual libraries needed for development are installed in "C:\TwinCAT\3.1\Components\Plc\Managed Libraries", which need to be installed there prior of compilation. The ones residing in the project must not be version controlled.
*.compileinfo - Self explaining, file created everytime a compilation is made. Beckhoff documentation says: "The compile information was created during the last download of the PLC project and stored in a file".
_Boot/ - Folder generated when project is compiled with all the binaries. This is what is transferred to the actual target device (PLC) for execution.
_CompileInfo/ - Self explaining. Folder created during compilation. Holding the *.compileinfo-files.
_Libraries/ - Folder holding the *.compiled-library and *.library in the local development folder.

There are various github repos for TwinCAT that use this list for gitignore, one being TcUnit, a unit testing framework for TwinCAT3:
https://github.com/tcunit/TcUnit/blob/master/.gitignore
Note that because TwinCAT is using visual studio as development environment (it integrates itself into VS during installation), all visual studio files that are normally ignored shall be ignored as well (therefore the comment # Recommended: VisualStudio.gitignore).
Update 2019-12-05: I've published a blog post that goes through these files and the reason for having a gitignore-file for TwinCAT3 in a much greater detail now which is accessible here:
https://alltwincat.com/2019/12/02/gitignore-for-twincat/
Explain why you’re making a change:
Unfortunately version control by modern VCS is yet quite uncommon in industrial automation, and therefore proper practices and documentation around it have not yet been fully defined. It's quite common within the world of automation to do version control by simply taking a complete project folder and copying it to a new folder, adding the new date and calling that version control. Now that more and more automation engineers are starting to do version control more properly, and also starting to use GIT, it makes sense to create this file.
If this is a new template:
Link to application or projects homepage:
This would be TcUnit, which is a popular unit testing framework for TwinCAT3:
https://tcunit.org/
https://github.com/tcunit

@shiftkey - it's been two months since the creation of this PR and I just wanted to double-check if there is something expected from me that I need to do for this to progress? I tried to document the reason for this addition as well as I could, but maybe there is still some information missing that you expect?

@shiftkey , just doing a ping on this one as it's now been 5 months since the creation of this ticket... :-P

I think that is a good idea to add this line:
.tsproj.bk
these are backup file of *.tsproj

I think that is a good idea to add this line:
.tsproj.bk
these are backup file of *.tsproj

I've already included *.tsproj.bak... which I assume you meant?

.tsproj.bk1
.tsproj.bk2
.tsproj.bk3
thise files are backup of file .tsproj.bak

TwinCAT HMI
Assuming this .gitignore should be applicable to all TwinCAT modules, you can add the following:
## Ignore TwinCAT HMI temporary files, build results, and
## files generated by popular TwinCAT HMI add-ons.
.engineering_servers
liveview_*
*.cache
*.db-shm
*.db-wal
*.pid
tchmipublish.journal.json

These come from a file called .tfignore, which is generated upon adding a new TwinCAT HMI project to a solution, somewhere in your working tree.
Initially I thought that anything under \Server\Extensions should also be ignored, but that is up for debate. For some reason, the underlying .json files are often seen as changed, while in fact they are not. This leads me to believe they were generated. However they do contain information necessary for publishing your HMI project to a HMI server. For now I myself do not ignore them. Instead I undo any changes made to them, unless I can explain those changes.
Backup files
I noted that visual studio/shell did not always recognize backup files, so i simply added *.bak as well.
It should be noted that PLC projects come with the .tcpproj extension, and thus also the .tcpproj.bak should be added to the gitignore file.
Final note
I do not like that the .tmc file must be generated on build, yet saved to the repo. I hope Beckhoff could split any necessary configuration/generation in future versions.

@rudischilder
In the TwinCAT HMI 1.12 beta there are the following further entries added to its .gitignore:
.hmiframework/
.hmipkgs/*-*-*-*/

@rudischilder & @HolgerJeromin, when creating a TwinCAT-HMI project, a .gitignore-file is created automatically, thus should not be included here.

@sagatowski thank you for this.
Should the _Deployment directory be ignored as well? The _Repository directory?

@sagatowski thank you for this.
Should the _Deployment directory be ignored as well? The _Repository directory?

@isherman I haven't managed to reproduce the _Deployment directory. Under what conditions is this created?

Hi,
Any update? This work seems alright to me.

Hi,
Any update? This work seems alright to me.

GitHub are not in a hurry with approving this PR it seems.
@shiftkey It's been almost two years since this PR was created. How much (approx.) will it take before it is approved? If there is some information missing, please let me know.

[] unc0ver is not for sale [] If you purchased unc0ver, please report the seller
[] Get unc0ver for free at https://unc0ver.dev [] Configured to share anonymous OS crash logs
[] Machine Name: iPhone9,4 [] Model Name: D111AP

[*] Kernel Version: Darwin Kernel Version 20.1.0: Fri Oct 30 00:34:15 PDT 2020; root:xnu-7195.42.3~1/RELEASE

Great, now we've got SPAM-bots attacking this PR too. I hope this PR can be accepted & closed.

Just to add, I'd like this PR merged too - used the .gitignore yesterday for a TC3 project, thanks @sagatowski


Azure deploy server

it compiles :-O

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

Coverage data is based on head (ef5893e) compared to base (181cbdf).
Patch has no changes to coverable lines.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
Additional details and impacted files
@@           Coverage Diff            @@
##           develop     #367   +/-   ##
========================================
  Coverage    55.19%   55.19%
========================================
  Files           24       24
  Lines         1683     1683
========================================
  Hits           929      929
  Misses         688      688
  Partials        66       66

Flag
Coverage Δ

unittests
55.19% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.
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.


Rewrite of readXRefTable [obj.js]

Complying with Poppler's request to take down copied code.
/cc @andreasgal

@pdfjsbot test


GSD Request

--- GSD JSON ---
{
  "vendor_name": "Linux",
  "product_name": "Kernel",
  "product_version": "versions from v5.6 to before v5.16.5",
  "vulnerability_type": "unspecified",
  "affected_component": "unspecified",
  "attack_vector": "unspecified",
  "impact": "unspecified",
  "credit": "",
  "references": [
    "https://git.kernel.org/pub/scm/linux/kernel/git/stable/linux.git/commit/?id=a580c76d534c7360ba68042b19cb255e8420e987",
    "https://git.kernel.org/pub/scm/linux/kernel/git/stable/linux.git/commit/?id=14be8d448fca6fe7b2a413831eedd55aef6c6511"
  ],
  "extended_references": [
    {
      "type": "commit",
      "value": "a580c76d534c7360ba68042b19cb255e8420e987",
      "note": "introduced"
    },
    {
      "type": "commit",
      "value": "14be8d448fca6fe7b2a413831eedd55aef6c6511",
      "note": "fixed"
    }
  ],
  "reporter": "joshbressers",
  "reporter_id": 1692786,
  "notes": "",
  "description": "net: bridge: vlan: fix memory leak in __allowed_ingress\n\nThis is an automated ID intended to aid in discovery of potential security vulnerabilities. The actual impact and attack plausibility have not yet been proven.\nThis ID is fixed in Linux Kernel version v5.16.5 by commit 14be8d448fca6fe7b2a413831eedd55aef6c6511, it was introduced in version v5.6 by commit a580c76d534c7360ba68042b19cb255e8420e987. For more details please see the references link."
}
--- GSD JSON ---

This issue has been assigned GSD-2022-1000634


<EMAIL_ADDRESS>and core-js@3 support?

What problem does this feature solve?
For future
https://babeljs.io/blog/2019/03/19/7.4.0
https://github.com/zloirock/core-js/blob/master/docs/2019-03-19-core-js-3-babel-and-a-look-into-the-future.md
What does the proposed API look like?
plugins: [
  ["@babel/transform-runtime", {
    corejs: 3,
  }]
  ["@babel/preset-env", {
    useBuiltIns: "usage",
    corejs: 3,
  }]
]

Will be upgraded in v4, as addressed in the roadmap https://github.com/vuejs/vue-cli/issues/3649


Translate services-networking/service-topology.md in Korean.

This is a Feature Request
What would you like to be added
Translate services-networking/service-topology.md in Korean.
Why is this needed
The following files are not translated into Korean.

[ ] /en/docs/concepts/services-networking/service-topology.md

Comments
Page to update: https://kubernetes.io/docs/concepts/services-networking/service-topology/
I am translating this document.
The translated document will be submitted to dev-1.17-ko.1 or dev-1.17-ko.2 dbranch.
If it is translated very quickly, it is submitted to dev-1.17-ko.1.
/language ko

/assign


Here be vulnerabilities

please name your pull request "CVE-2015-1277 and CVE-2015-1297"

Creating a new pull request with the appropriate changes.


Stake or unstake doesn't show immediately

Describe the bug
in the case when I approve my TIME or MEMO tokens, the UI doesn't update immediately after that, as I need to refresh my screen for it to update. if I click on approve again, it will only approve it again...
To Reproduce
Steps to reproduce the behavior:

Go to 'Stake' or 'Unstake' tab
Click on 'approve' and confirm the tx
after that the UI is still showing 'approve' button and not 'stake' or 'unstake' button

Expected behavior
Once a user approve the smart contract for spending time on their behalf, and the 'Approval' has been received, the UI needs to show 'stake' or 'unstake' button instead of being stuck at showing approve button without the need to refresh the page.

Hello, maybe you need to check if you have enough AVAX in your wallet to pay gas fees. If thats the case, make sure that you are on the Avalanche Network. For further help, I would need more information... any messages from the website?

The issue is only with the UI doesn't update instantly, not sure if the
problem still exists though
On Fri, Dec 17, 2021, 1:43 PM s1k2614 @.***> wrote:

Hello, maybe you need to check if you have enough AVAX in your wallet to
pay gas fees. If thats the case, make sure that you are on the Avalanche
Network. For further help, I would need more information... any messages
from the website?
—
Reply to this email directly, view it on GitHub
https://github.com/Wonderland-Money/wonderland-frontend/issues/44#issuecomment-996693988,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AL6YNXQFCUXVRPEIXS5CJ3TURMV7BANCNFSM5JGNJU7A
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.
You are receiving this because you authored the thread.Message ID:
@.***>

The issue is only with the UI doesn't update instantly, not sure if the problem still exists though
…
On Fri, Dec 17, 2021, 1:43 PM s1k2614 @.> wrote: Hello, maybe you need to check if you have enough AVAX in your wallet to pay gas fees. If thats the case, make sure that you are on the Avalanche Network. For further help, I would need more information... any messages from the website? — Reply to this email directly, view it on GitHub <#44 (comment)>, or unsubscribe https://github.com/notifications/unsubscribe-auth/AL6YNXQFCUXVRPEIXS5CJ3TURMV7BANCNFSM5JGNJU7A . Triage notifications on the go with GitHub Mobile for iOS https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Android https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub. You are receiving this because you authored the thread.Message ID: @.>

Normally blockchain transactions take a little time to validate. This is most likely why you are not seeing an instant update to the UI. The site needs to get data back after the transaction is validated in order to update the UI.


Make Links Clickable

Fixes #1072. Uses react-native-hyperlink to handle detecting links and making them clickable cross-platform.

The module I used to detect hyperlinks and make them clickable doesn't have type declarations. It works well and considering how simple its implementation is, I think it's reasonable to just declare the module without going through and defining our own types for it for the time being.


Get hotspot pitch, yaw after drag

I have similar question as here: https://github.com/mpetroff/pannellum/issues/913
I do use dragHandlerFunc / dragHandlerArgs and my hotspots are draggable in editor, and after drag I want to save hotspot pitch / yaw. The problem is that mouseEventToCoords only prints the pitch and yaw of the click location, so if I click on the different place in hotspot, these values will be different, which depending on how hotspot looks like, will not reflect its actual position after save. Then when I load viewer again, hotspot are not quite where they are expected to be.
Is there a way in dragHandlerFunc to get actual hotspot pitch / yaw center position, not where clicked?

  function getCoords(inputX: number, inputY: number){
    const container = viewer.getContainer();
    const renderer = viewer.getRenderer()!;
    const container_rect = container.getBoundingClientRect();
    const pos = {x: inputX - container_rect.left, y: inputY - container_rect.top};

    const config = viewer.getConfig()!;

    const canvas = renderer.getCanvas();
    const canvasWidth = canvas.clientWidth;
    const canvasHeight = canvas.clientHeight;
    const x = pos.x / canvasWidth * 2 - 1;
    const y = (1 - pos.y / canvasHeight * 2) * canvasHeight / canvasWidth;
    const focal = 1 / Math.tan(config.hfov * Math.PI / 360);
    const s = Math.sin(config.pitch * Math.PI / 180);
    const c = Math.cos(config.pitch * Math.PI / 180);
    const a = focal * c - y * s;
    const root = Math.sqrt(x*x + a*a);
    const pitch = Math.atan((y * c + focal * s) / root) * 180 / Math.PI;

    let yaw = Math.atan2(x / root, a / root) * 180 / Math.PI + config.yaw;
    if (yaw < -180) yaw += 360;
    if (yaw > 180) yaw -= 360;

    return [yaw, pitch];
  }

use this function. inputX and inputY is the position from getBoundingClientRect of hotspot.

Is there a way in dragHandlerFunc to get actual hotspot pitch / yaw center position, not where clicked?

The dragHandlerFunc is called both for the events that start the drag and for those that end it. If you call mouseEventToCoords with the end event (e.g., mouseup), you'll get the final hot spot position, since the interim move events are likewise passed through mouseEventToCoords to update the hot spot position.

Yes, I realized that afterwards. I guess changing hfov can make hotspot appear to be on different place visually.


First start take more than one minute, and modify one file take 10s to recompile. Is this ok?

There are about 700 files in our project, not including files under node_modules directory. Is there anyway to improve the recompile performance? Because I only changed one file, so I want to see the result as soon as possible.
The following is our package.json file.
{
  "name": "web",
  "version": "0.1.0",
  "private": true,
  "scripts": {
    "start": "react-app-rewired start",
    "build": "react-app-rewired build",
    "test": "react-app-rewired test --env=jsdom",
    "build-css": "node-sass-chokidar src/ -o src/",
    "watch-css": "npm run build-css && node-sass-chokidar src/ -o src/ --watch --recursive",
    "format": "standard",
    "format:fix": "standard --fix"
  },
  "standard": {
    "ignore": [
      "build",
      "coverage",
      "node_modules",
      "public"
    ],
    "globals": [
      "it",
      "expect"
    ]
  },
  "proxy": {
    "/api": {
      "target": "http://gtgj.dev.bicisims.com/api/",
      "pathRewrite": {
        "^/api": ""
      },
      "changeOrigin": true
    }
  },
  "dependencies": {
    "@antv/data-set": "^0.8.3",
    "antd": "^3.1.0",
    "axios": "0.17.1",
    "bizcharts": "^3.1.0-beta.6",
    "compare-versions": "3.1.0",
    "deepcopy": "0.6.3",
    "dotenv": "4.0.0",
    "echarts": "3.8.5",
    "echarts-for-react": "2.0.3",
    "immutable": "3.8.2",
    "lodash": "4.17.4",
    "moment": "2.20.1",
    "promise.prototype.finally": "3.1.0",
    "prop-types": "15.6.0",
    "react": "15.6.2",
    "react-day-picker": "7.0.5",
    "react-dom": "15.6.2",
    "react-modal": "3.1.10",
    "react-motion": "0.5.2",
    "react-redux": "5.0.6",
    "react-router": "2.8.1",
    "react-router-dom": "4.2.2",
    "react-select": "1.1.0",
    "redial": "0.5.0",
    "redux": "3.7.2",
    "redux-form": "7.2.0",
    "redux-logger": "3.0.6",
    "redux-persist": "4.10.2",
    "redux-thunk": "2.2.0"
  },
  "devDependencies": {
    "babel-plugin-import": "1.6.3",
    "node-sass-chokidar": "0.0.3",
    "react-app-rewire-less": "2.1.0",
    "react-app-rewired": "1.4.0",
    "react-scripts": "1.0.17",
    "standard": "10.0.3"
  }
}

tl;dr -- yes this is expected, it'll get better eventually
Closing as answered by @viankakrisna, but performance should get much better with the release of 2.x; there have been many performance improvements to webpack 4.


enforce the grammar checking for ε-matches in invalid positions

Codecov Report
Patch coverage: 100.00% and project coverage change: -0.18% :warning:

Comparison is base (c0e75c4) 100.00% compared to head (37c2f0f) 99.82%.

Additional details and impacted files
@@             Coverage Diff             @@
##            master      #18      +/-   ##
===========================================
- Coverage   100.00%   99.82%   -0.18%
===========================================
  Files            9        8       -1
  Lines          617      573      -44
===========================================
- Hits           617      572      -45
- Misses           0        1       +1

Files Changed
Coverage Δ

src/clauses.jl
99.43% <ø> (-0.57%)
:arrow_down:

src/grammar.jl
100.00% <100.00%> (ø)

... and 7 files with indirect coverage changes

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


README: add semgrep rules

Fixes #16

Done. Assuming I can alphabetize correctly today. Been one of those days... :P


Polyfill

On IE11 we had an "Object.assign is undefined" error. It looks like polyfills are not included in Hops. Is that an intended behavior?
In order to make it work we had to add this to our project:
import 'core-js/es6/promise';
import 'core-js/es6/object';

Thanks for this report, @drixsonic. Ought to be fixed in the next beta.


Add support for iOS 13 SDK (Xcode 11)

Fix placeholder text color crash on iOS 13
Change use from private API (_placeholderLabel) to attributedPlaceholder

Will check This thanks
Sent from my iPhone

On 20-Sep-2019, at 10:36 AM, Giorgio Doganiero<EMAIL_ADDRESS>wrote:
Fix placeholder text color crash on iOS 13
Change use from private API (_placeholderLabel) to attributedPlaceholder
You can view, comment on, or merge this pull request online at:
https://github.com/ErAbhishekChandani/ACFloatingTextfield/pull/67
Commit Summary
Fix placeholder color prohibited in iOS 13 SDK
File Changes
M Source-Swift/ACFloatingTextfield.swift (12)
Patch Links:
https://github.com/ErAbhishekChandani/ACFloatingTextfield/pull/67.patch
https://github.com/ErAbhishekChandani/ACFloatingTextfield/pull/67.diff
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or mute the thread.

@Axtel209 I didn't work for me, it still crashes for me

@Axtel209 I didn't work for me, it still crashes for me

Does it give the same crash log?

Yes

I have checked this its working but you have added wrong placeholder color when its in selected state please review and correct this than Its good to go! Thanks
Sent from my iPhone

On 20-Sep-2019, at 10:48 AM, abhishek chandani<EMAIL_ADDRESS>wrote:
Will check This thanks
Sent from my iPhone

On 20-Sep-2019, at 10:36 AM, Giorgio Doganiero<EMAIL_ADDRESS>wrote:
Fix placeholder text color crash on iOS 13
Change use from private API (_placeholderLabel) to attributedPlaceholder
You can view, comment on, or merge this pull request online at:
https://github.com/ErAbhishekChandani/ACFloatingTextfield/pull/67
Commit Summary
Fix placeholder color prohibited in iOS 13 SDK
File Changes
M Source-Swift/ACFloatingTextfield.swift (12)
Patch Links:
https://github.com/ErAbhishekChandani/ACFloatingTextfield/pull/67.patch
https://github.com/ErAbhishekChandani/ACFloatingTextfield/pull/67.diff
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or mute the thread.

Hi, this error is also presented in objective c.


Added Wpf support (control and sample)

I conformed to the Avalonia example as much as I could, but I gave up trying to learn ReactiveUI so I did not use it at all.

I build this for .NET 6... the pipeline uses .NET 3.1. How do you want to proceed?

Hey paviad,
I was looking for a wpf integration for MarcusWVnc and just found your pull request. It works like a charm to connect and interact with my vnc server but i have one problem with it: When i try to disconnect the client by using the CloseAsync method of the RfbConnection, it won't disconnect and just give me an OperationCanceledException. Did you test this case in your integration? You have any idea what could be the issue?

I honestly do not remember, last I touched this project was end of 2021, and I haven't used it since. I'll see if I can find the time to have a look.

I've incorporated #26 into my fork at https://github.com/paviad/MarcusW.VncClient

I added an in memory client using the Skia library (Windows)

Some comments from my end about the sample:

Autoresize doesn't work. It seems that you are setting it based on the ActualHeight and ActualWidth which is incorrect due to being in a ScrollViewer. The ScrollViewer width and height likely changes AFTER the event is called so you are sending the incorrect ActualWidth and ActualHeight, so no resize actually happens on the server-side.
ErrorMessage is not displayed unless the connection succeeds
No input for the Password field?

Some comments from my end about the sample:

Autoresize doesn't work. It seems that you are setting it based on the ActualHeight and ActualWidth which is incorrect due to being in a ScrollViewer. The ScrollViewer width and height likely changes AFTER the event is called so you are sending the incorrect ActualWidth and ActualHeight, so no resize actually happens on the server-side.
ErrorMessage is not displayed unless the connection succeeds
No input for the Password field?

Fixed last two issues (error message and interactive password)
Regarding auto size, I couldn't reproduce the problem you mention, I am not using ActualWidth or ActualHeight, and when I actually tested it it worked fine with or without scroll bars on the view.

Incorporated PR #29 into this one (for Avalonia and Wpf)


[BD-6] Add python 3.8 to the classifiers

This already had tests in python 3.8 but now I am adding the classifiers
Reviewers

[x] @awais786
[x] @ericfab179
[x] Ready for edX review
[ ] @jmbowman

Thanks for the pull request, @morenol! I've created OSPR-4437 to keep track of it in JIRA. JIRA is a place for product owners to prioritize feature reviews by the engineering development teams.
Feel free to add as much of the following information to the ticket:

supporting documentation
edx-code email threads
timeline information ("this must be merged by XX date", and why that is)
partner information ("this is a course on edx.org")
any other information that can help Product understand the context for the PR

All technical communication about the code itself will still be done via the GitHub pull request interface. As a reminder, our process documentation is here.

One minor comment otherwise seems ok.

@morenol Even though your pull request wasn’t merged, please take a moment to answer a two question survey so we can improve your experience in the future.

@awais786 for a strange reason, github is not updating the checks from the last change, but they are green: https://travis-ci.org/github/edx/edx-lint/builds/683581046

@awais786 squashed, I think that this is ready to merge.


UDT/Monoid/Bugfix Update

Needs tests and some comments
closes #76, closes #77, closes #80, closes #82, closes #83, closes #85
Partially #81, needs new version of SSGraphBLAS to finish.

Codecov Report

Merging #86 (cec0508) into master (7aeaf5b) will decrease coverage by 1.24%.
The diff coverage is 35.32%.

@@            Coverage Diff             @@
##           master      #86      +/-   ##
==========================================
- Coverage   16.65%   15.40%   -1.25%
==========================================
  Files          53       53
  Lines        7854     7821      -33
==========================================
- Hits         1308     1205     -103
- Misses       6546     6616      +70

Impacted Files
Coverage Δ

src/SuiteSparseGraphBLAS.jl
90.90% <ø> (ø)

src/chainrules/mulrules.jl
100.00% <ø> (ø)

src/chainrules/selectrules.jl
100.00% <ø> (ø)

src/gbtypes.jl
35.71% <0.00%> (-4.29%)
:arrow_down:

src/matrix.jl
60.34% <0.00%> (+0.02%)
:arrow_up:

src/operations/select.jl
72.72% <0.00%> (ø)

src/operations/sort.jl
0.00% <0.00%> (ø)

src/operators/selectops.jl
58.69% <0.00%> (ø)

src/unpack.jl
31.20% <0.00%> (ø)

src/scalar.jl
44.18% <11.11%> (-0.26%)
:arrow_down:

... and 39 more

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


Sync APIs from karmada repo based on v1.4.0

This PR syncs APIs from karmada-io/karmada for v1.4.0.

@RainbowMango


Update Workbench Docker images

We currently have multiple Docker images each with different dependencies. Consolidate or add broader support for tutorials in a single container:
Completion Criteria

[ ] Add Python to Rstudio environments
[ ] Combine the Rstudio environments
[ ] PlantCV and pyClowder dependences
[ ] Geospatial support in Jupyter

@craig-willis is it possible to also hook these containers up to bety by keeping an SSH tunnel open like
ssh -Nf -L 5432:localhost:5432 bety6.ncsa.illinois.edu

@dlebauer Probably -- but just curious, why is it necessary/desirable to connect to BETY via localhost instead of the full name?

Good question. That is the method @robkooper suggested ... also how we are
connecting GIS software.
On Fri, May 5, 2017 at 11:18 AM Craig Willis<EMAIL_ADDRESS>wrote:

@dlebauer https://github.com/dlebauer Probably -- but just curious, why
is it necessary/desirable to connect to BETY via localhost instead of the
full name?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/terraref/computing-pipeline/issues/304#issuecomment-299509231,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAcX55KiPaanjvrWVfJ-rJITwWu59W4lks5r20vBgaJpZM4NPjoB
.

I know some instances of betydb allow anyone to connect locally but
restrict remote access for security. Not sure if that is related.
On Fri, May 5, 2017 at 11:21 AM David LeBauer<EMAIL_ADDRESS>wrote:

Good question. That is the method @robkooper suggested ... also how we are
connecting GIS software.
On Fri, May 5, 2017 at 11:18 AM Craig Willis<EMAIL_ADDRESS>wrote:

@dlebauer https://github.com/dlebauer Probably -- but just curious,
why is it necessary/desirable to connect to BETY via localhost instead of
the full name?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/terraref/computing-pipeline/issues/304#issuecomment-299509231,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAcX55KiPaanjvrWVfJ-rJITwWu59W4lks5r20vBgaJpZM4NPjoB
.

this is using a ssh tunnel to create so that we don't not have to open the postgres database to anybody that does not have access to the machine itself.

@robkooper I met with David to discuss this.  It sounds like the basic requirement is to allow Workbench users to easily connect to the BETY read-only Postgres user.  To support SSH tunneling, the user would either need to have an account/key or we would need to have some generic account available to Workbench users.  I could setup a service to proxy the request via SSH tunnel.  Another option would be to simply open the BETY server to allow the Workbench server to connect to 5432. Either way is possible, but it seems that opening access from the Workbench service might be easier?

@craig-willis I'm not sure if this helps (you may already understand this) but bety6.ncsa.illinois.edu is "local" once you are through the ssh tunnel.  So the full name is specified in setting up the tunnel but needn't then be specified in the database connection.  While one could allow remote connections directly to the database, Rob considers it more secure to restrict to local connections and then use a tunnel to get around the restriction.

Thanks, @gsrohde. From my understanding, this method also requires each user to have an account on the BETY server with associated credentials (either u/p or SSH key).  In talking with David, it sounded like we want to allow Workbench users to access BETY readonly without requiring a separate OS account to access the BETY server.

@craig-willis Conceivably you could use a shared machine account with limit permissions.  One problem with this is that if the credentials need to be changed, then everyone needs to be notified.  Possibly there are other better ways of doing this that are beyond my ken.  See also https://github.com/terraref/computing-pipeline/issues/262 for some background on this.

We can look and see what needs to be done to allow for a viewer that can access the database inside NCSA. I do want to spend some time to make sure this is secure. I want to make sure we limit this user to only those tables that are needed. For example this viewer account should not need to see the users table (which contains the encrypted passwords, and api keys).

As discussed in weekly meeting: Some users have access to the machine and a postgres account (separate from the web UI). It sounds like there are security issues we'd need to address to allow direct access to BETY. Otherwise, we can use the sync feature to a separate copy. Rob is talking with NCSA security about allowing access to Postgres.  One option would be to setup a Postgres container on Workbench with anonymized data to allow users to do analysis.

related: https://github.com/terraref/reference-data/issues/111

@craig-willis in the R container, please replace devtools::install_github("ropensci/traits") with devtools::install_github("terraref/traits"). Once it is live I will probably have a few simple r package dependencies to add since I haven't tested the code on a single container yet.

I've created an rstudio-terraref image that contains Rstudio, Python2, Python3, Plantcv dependencies.   You can test this in Workbench by importing the following service spec:
{
  "key": "rstudioterra",
  "label": "RStudio TERRA-REF",
  "description": "IDE for the R programming language.",
  "logo": "/asset/png/logos/rstudio-logo.png",
  "image": {
    "registry": "",
    "name": "craigwillis/rstudio-terraref",
    "tags": [
      "latest"
    ]
  },
  "display": "stack",
  "access": "external",
  "ports": [
    {
      "port": 8787,
      "protocol": "http",
      "contextPath": "/"
    }
  ],
  "repositories": [
    {
      "url": "https://github.com/rocker-org/rocker-versioned",
      "type": "git"
    }
  ],
  "readinessProbe": {
    "type": "http",
    "path": "/images/rstudio.png",
    "port": 8787,
    "initialDelay": 1,
    "timeout": 360
  },
  "volumeMounts": [
    {
      "mountPath": "/home/rstudio"
    }
  ],
  "resourceLimits": {
    "cpuMax": 500,
    "cpuDefault": 100,
    "memMax": 2000,
    "memDefault": 50
  },
  "tags": [
    "7",
    "28",
    "40"
  ],
  "info": "https://nationaldataservice.atlassian.net/wiki/display/NDSC/RStudio",
  "authRequired": true
}

Dockerfile is here:
https://github.com/terraref/toolserver-dockerfiles/blob/kitchensink/kitchensink/rstudio/Dockerfile
Let me know if you need any changes.  I'll start working on the Jupyter environment.

I still need to complete the Jupyter image.

Added gdal to Jupyter image, fixed plantcv branch (dev > master).

@craig-willis what steps are remaining before closing this issue?

Create automated builds in Dockerhub (in process) and confirm builds are working (smoke test). Hopefully close out today.

New Docker Images:

https://hub.docker.com/r/terraref/rstudio-terraref/: RStudio (rocker/geospatial) + TERRA packages + Python + PlantCV
https://hub.docker.com/r/terraref/jupyter-terraref/: Jupyter + NetCDF + GDAL + PlantCV

I've updated the original "RStudio Server" application to use the new image instead of the original rocker image.  One snag, I'd like to delete some of the other applications (i.e., R geospatial, Jupyter + NetCDF) -- but many of them are or were in use by team members. I'll need to email folks to get them to remove these applications or let me know if they are still in use.

@dlebauer  Do you want to re-open this ticket or start a separate discussion?  Is this something we need to think about before the November release?

I think open another issue for future (target the Nov 2018 release). I
think it should be sufficient for now if we retain home directories from
deleted containers and encourage people to use github repositories for the
code that they write. Isn't reproducibility planned as part of a core NDS
workbench feature?
On Wed, Jul 26, 2017 at 6:48 PM Craig Willis<EMAIL_ADDRESS>wrote:

@dlebauer https://github.com/dlebauer Do you want to re-open this
ticket or start a separate discussion? Is this something we need to think
about before the November release?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/terraref/computing-pipeline/issues/304#issuecomment-318215607,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAcX54XTfwxzbLpMgOnCutqIHkrJZCQXks5sR9BSgaJpZM4NPjoB
.

https://github.com/terraref/computing-pipeline/issues/344


All / are replaced with .

If a translatable key is not in the source file, then the wTrans() will replace all slashes for dots in it.
That's the case when English strings are used as an input to the translation functions with the plan for later to become IDs for their localizations.
Current behavior:
import { trans } from 'laravel-vue-i18n';

console.assert(trans('Active / Overdue') === 'Active . Overdue'); // true

Expected behavior
import { trans } from 'laravel-vue-i18n';

console.assert(trans('Active / Overdue') === 'Active / Overdue'); // true

To overcome this issue, you're forced to create the en.json where keys and values are going to be equal however why would you need the file to translate English into English?

Before https://github.com/xiCO2k/laravel-vue-i18n/pull/89 it was not possible at all to have slashes in the key. Since then it is, however the source JSON file is mandatory.

https://github.com/xiCO2k/laravel-vue-i18n/pull/116 reveals the issue.


ERROR HELP ME PLS

Describe your problem here.
BUILDING GAME
source/psychlua/HScript.hx:168: characters 89-94 : Type not found : SCall
done.
Press any key to continue . . .
When I want to test the Psych Engine, I get an error like this. I Installed Libraries, Still It Still Gives Errors, Can You Help?
Are you modding a build from source or with Lua?
Source
What is your build target?
Windows x64
Did you edit anything in this build? If so, mention or summarize your changes.
No response

replace SCall with TeaCall

New Error  :
BUILDING GAME
source/psychlua/HScript.hx:291: characters 9-16 : tea.SScript has no field destroy
source/psychlua/HScript.hx:286: characters 27-34 : Field destroy is declared 'override' but doesn't override any field
source/states/PlayState.hx:3161: characters 17-34 : psychlua.HScript has no field parsingExceptions (Suggestion: parsingException)
source/states/PlayState.hx:3161: characters 56-73 : psychlua.HScript has no field parsingExceptions (Suggestion: parsingException)
source/states/PlayState.hx:3164: characters 25-42 : psychlua.HScript has no field parsingExceptions (Suggestion: parsingException)
source/states/PlayState.hx:3164: characters 15-42 : You can't iterate on a Dynamic value, please specify Iterator or Iterable
source/psychlua/HScript.hx:206: characters 15-20 : Type not found : SCall
done.
Press any key to continue . . .

New Error :
New Error  :
BUILDING GAME
source/psychlua/HScript.hx:291: characters 9-16 : tea.SScript has no field destroy
source/psychlua/HScript.hx:286: characters 27-34 : Field destroy is declared 'override' but doesn't override any field
source/states/PlayState.hx:3161: characters 17-34 : psychlua.HScript has no field parsingExceptions (Suggestion: parsingException)
source/states/PlayState.hx:3161: characters 56-73 : psychlua.HScript has no field parsingExceptions (Suggestion: parsingException)
source/states/PlayState.hx:3164: characters 25-42 : psychlua.HScript has no field parsingExceptions (Suggestion: parsingException)
source/states/PlayState.hx:3164: characters 15-42 : You can't iterate on a Dynamic value, please specify Iterator or Iterable
source/psychlua/HScript.hx:206: characters 15-20 : Type not found : SCall
done.
Press any key to continue . . .

1st one, replace destroy with kill, and remove the override. 2nd, do what was suggested, replace parsingExceptions with parsingException and the last one you already know

What did you mean by the last one?

What did you mean by the last one?

.this

I did not understand anything. What do I need to change?

here, maybe learn how to code before you do shit like this https://haxe.org/documentation/introduction/

thank you

np

wtf is happening

i need help or just give me the line of code i need to type

Download and install  Notepad++go to source\psychlua\HScript.hx, open the file, press ctrl and f at the same time, click replace and Find what : SCall and Replace with :TeaCall

Sorununuzu burada açıklayın.
Bina Oyun Kaynağı/Psychlua/Hscript.HX: 168: Karakterler 89-94: Tip Bulunmadı: Scall
Tamamlandı. Devam etmek için herhangi bir tuşa basın . . .
Psych motorunu test etmek istediğimde, böyle bir hata alıyorum. Kütüphaneler yükledim, hala hatalar veriyor, yardım edebilir misiniz?
Kaynaktan mı yoksa lua ile mi yapıyorsunuz?
Kaynak
Yapı hedefiniz nedir?
Windows x64
Bu yapıdaki herhangi bir şeyi düzenlediniz mi? Eğer öyleyse, değişikliklerinizden bahsedin veya özetleyin.
Cevap yok

how about i find you

Not to disturb but source/psychlua/HScript.hx:195: characters 2-3 : Missing ;
PLS

add a semicolon at line 195 in source/psychlua/HScript.hx


Increase and parametrize the maximum number of heterogeneous type in …

…a sequence
This only partially fixes #2040.
It does increase the bound, and it is now possible to change it using --config typing.max_heterogeneous_sequence_size=32.
There is however a subtle error with invalid lookup that requires more care, and probably another commit!

Obsoleted by #2060


trilinear interpolation for coloring by volume grid data

http://paulbourke.net/miscellaneous/interpolation/
http://www.scratchapixel.com/old/lessons/3d-advanced-lessons/interpolation/trilinear-interpolation/

done, see 7cfdbd6832a860c5d2e070c584ae936b26655aaa


Capybara 3 Support

Relaxes the capybara requirement to allow 3.x
Method signature updates to match capybara 3
Use new submits? method in click to simplify method

Thanks for the PR. Hm, looks like this failed on Travis of the pretty old versions of ruby we're testing against.
Everything less that 2.3 is end of life'd. Would you mind updating the travis file: https://github.com/jeroenvandijk/capybara-mechanize/blob/4239490d2725dd815c73b56067a01de714bbc90c/.travis.yml#L9-L10
to include 2.3, 2.4, and 2.5? We can see if that helps travis.

@phillbaker I've updated the list of rubies.  What's the intent behind the capybara 2.4 gemfile?  Is that supposed to be newer or older than what's in the gemspec?

Yikes.  Looks like I need to update the test suite.

Oy, looks like a mess. It looks like Travis isn't pulling in the 3.0 version: https://travis-ci.org/jeroenvandijk/capybara-mechanize/jobs/371859689#L499 ? submits? is undefined for these tests?

What's the intent behind the capybara 2.4 gemfile? Is that supposed to be newer or older than what's in the gemspec?

Hm, it's coming from https://github.com/jeroenvandijk/capybara-mechanize/pull/62. I think the intent was to test both. For the 3.0 tests, it might be easier to create a 3.0 gemfile and check that submits? is defined?

Ping @phillbaker ^

I'm fine with doing a major version bump and only supporting v3.

Ok so the real problem here is that capybara has switched over to rspec 3 and we're still on 2.  I don't really have time to tackle that right now.

Oy. Thanks for the update. When I get some time, I'll take a look.

I've cherry-picked your original commit in e82a008564250728ed037eb85db5deff81e91f2d and released 1.10.0 with compatibility for capybara 3.x. Thanks for starting this @ryansch

@phillbaker It looks like you missed the changes in https://github.com/jeroenvandijk/capybara-mechanize/pull/63/commits/02dc2a0d434c7bddba884a3a883732f368780e96
One of these days we'll get the test suite working.  I've been running these changes in our test suite since early May without issue.

we'll get the test suite working

Not sure I follow, he test suite for this gem is passing? Or are you saying your test suite with the mainline of this gem?

you missed the changes in 02dc2a0

Ah, so I did. Want to cherry-pick on master and PR?


[Docs] NMFFilter Translation

As the title says

this is good. one error: in tab 1 it seems the ezdac is for all the tabs not just itself. I wonder if we could search the full folder for the attribute name and setting to catch any other such non-local ezdac?

Yes indeed we should ripgrep for it and squash them all in one hit before release. I'll file an issue so we remember.


ns-3: init at 27.0

ns-3 is a discrete event C++ network simulator developed at www.nsnam.org popular in the networking research community.
Python bindings might not be functional yet, I could not test them as the required libraries are still in PR https://github.com/NixOS/nixpkgs/pull/30473 but I have yet to meet someone using the bindings.
The manual building doesn't complete yet but it's mostly useful for core developers upon release.
Things done

[ ] Tested using sandboxing (nix.useSandbox on NixOS, or option build-use-sandbox in nix.conf on non-NixOS)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
[x] Fits CONTRIBUTING.md.

If the manual does not build yet, please do not advertise this as an option.

While building the manual I meet
(/nix/store/4agl46x083dpanpbsdydbzpqy1frgd66-texlive-combined-medium-2016/share
/texmf/tex/generic/babel/babel.def)))
(/nix/store/4agl46x083dpanpbsdydbzpqy1frgd66-texlive-combined-medium-2016/share
/texmf/tex/latex/psnfss/times.sty)

! LaTeX Error: File `fncychap.sty' not found.

which apparently belongs to
nix-locate fncychap.sty                                                                                ~/nixpkgs
python36Packages.sphinx_1_2.out                  18,775 r /nix/store/1sydasckcwpwnm635mcjkm9g1r36958n-python3.6-sphinx-1.2.3/lib/python3.6/site-packages/sphinx/texinputs/fncychap.sty
(arelle.doc)                                     18,775 r /nix/store/p829jnd9jvhismkyf8b0ja0qylpcyyyp-python3.4-sphinx-1.2.3/lib/python3.4/site-packages/sphinx/texinputs/fncychap.sty
python27Packages.sphinx_1_2.out                  18,775 r /nix/store/n7cdj6jmjf5vhvq32r4v05m097hgfc1n-python2.7-sphinx-1.2.3/lib/python2.7/site-packages/sphinx/texinputs/fncychap.sty
python27Packages.gateone.out                     18,775 r /nix/store/f6k1gcn281xgw1nipqq5fc26sqrny524-python2.7-gateone-1.2-0d57c3/gateone/docs/build/latex/fncychap.sty
python27Packages.gateone.out                     18,775 r /nix/store/f6k1gcn281xgw1nipqq5fc26sqrny524-python2.7-gateone-1.2-0d57c3/lib/python2.7/site-packages/gateone/docs/build/latex/fncychap.sty

I wonder where the problem lies: should sphinx or ns3 export some value for latex to find teh files ? maybe the problem has already been met

I removed the gsl part as it' not about speed but about precision and this is not used in core so I had not tested it.
I added the --profile switch so that people can build in release mode (ns3 default is debug)
People should instead override the package to use their own library. I suspect there is not much value in having this in binary cache as ns3 users are likely to override the defaults buildInputs and so on. I think I could add the attribute to enforce local build.
As for the options that don't work, I moved them to a let ... inso that the current progress is not lost.
+let
+  # there are various problems with manual generation see PR #31346
+  # once those are solved, promote it to an option
+  withManual = false;
+
+  # required packages are not in mainline yet
+  generateBindings = false;
+in

As for the options that don't work, I moved them to a let ... inso that the current progress is not lost.

If it doesn't work, then it should not go in. You can keep the relevant changes in your own branch.

I updated the package for ns3.28. Bindings and manual now work (though I disabled them by default as it can be pretty expansive to compile stuff).

Thanks!


Let search dropdown-selection directly change currently shown node details

When a node is searched and selected from the search dropdown menu, the currently shown node details should directly change to that selected node. Right now, you have to push RETURN or click on the search-icon to get the details shown.

fixed by commit #9ac444c


Experiment - coercion

This PR serves as a test for coercing inputs based on specs. There's several unanswered questions and problems to solve still. But I wanted to see what people thought about this approach.

I like that very much! I vote for include this into norm. ;-)
Great job @keathley !


When deploying this, warning shows stating that ccouldn't connect to site cloud code is unavailable. What is this issue?

I have cloud code defined in my cloud/functions.js, then a main with the require statement. Why does it keep giving this error? I have tried everything!!!! Do I need to just build this from the main api?

Parse Server attempts to perform a GET request on ${serverURL}/health and if it fails, that response is logged.
The issue is more than likely related to your configuration.
This might help you
Closing as it doesn't seem to be a Parse Server / Parse Server Example issue.
Alternatively, feel free to post in our community forum if the issue continues.


Support for --recursive flag

Example:
mocha spec --recursive

I'm not sure what you're referring to here since this is just a runner for browser tests, not the mocha CLI. Closing unless you meant something else.


plugin rapidly drains GitHub API quota in poll mode

We recently upgraded to 1.32.8 (from 1.31.4, since many intervening versions have been broken in different ways), and we're quickly burning through all of our GitHub API quota:
$ while true; do curl -slL https://api.github.com/rate_limit?access_token=[elided] | jq .resources.core.remaining ; sleep 1; done
592
591
589
587
586
584
582
575
564
553
545
535
529
519
509
500
491
483
478
476
473

Note that I restarted the plugin about 15m ago; this ratelimit value starts at 5000.
Looking at the Jenkins log, it looks like its querying each PR multiple times per minute. Refreshing the log, I'll see things like
Jul 05, 2016 10:05:04 PM INFO org.jenkinsci.plugins.ghprb.GhprbPullRequest updatePR
Pull request #16,062 was updated on repo kubernetes/kubernetes but there aren't any new comments nor commits; that may mean that commit status was updated.
Jul 05, 2016 10:05:04 PM INFO org.jenkinsci.plugins.ghprb.GhprbPullRequest updatePR
Pull request #13,925 was updated on repo kubernetes/kubernetes but there aren't any new comments nor commits; that may mean that commit status was updated.
Jul 05, 2016 10:05:04 PM INFO org.jenkinsci.plugins.ghprb.GhprbPullRequest updatePR
Pull request #13,216 was updated on repo kubernetes/kubernetes but there aren't any new comments nor commits; that may mean that commit status was updated.

repeated again and again for the same PRs, even though nothing is changing.

What is the poll rate set at?

Every two minutes, though it appeared to be polling more frequently than that.

We haven't had any issues with 1.31.4 using the same settings.

may or may not be related but linking #204 here

it would seem we hit this issue with 1.39. reverting the 1.35 seems to have improved it.

is a status on this?

Still seeing this in v1.39. Downgrading the plugin to v1.35 seems to have brought down the rate of requests.
Is there a better way to track the requests sent from GHPRB to github, so that we can triage this issue?

@AboorvaDevarajan I had, a long time ago, instrumented the github API library so that it logged when it made an actual call to the REST API (rather than some cached data).
What are the characteristics of your server?  Do you have lots of jobs that use the plugin?

we have two different jenkins running with the same jobs. The one with version 1.36.0 uses fewer requests that the version with 1.39.0
github rate limit resets every hour(5000 api calls per hour) and by the end of the hour there is a difference of almost 500 requests.

@mmitche There are around 50 jobs in total, in which 12 are using ghprb polling (2 mins polling interval for each), and another 12 are using ghprb webhook.
But clearly there is a difference between v1.35 and v1.39 (with above configuration)
with v1.37 requests made is a lot, around 100-200/min.
with v1.35, requests made are around 40/min.
I will see if I can capture some data using sniffer for more info.

@AboorvaDevarajan That's not a lot of jobs.  I think it's likely that there were only a few changes.  The plugin treats each job as its own unit and doesn't share data between them so it can have horrible multipliers of API requests.

Is this with the latest github-api plugin installed? I'm curious if someone has instrumented against https://github.com/kohsuke/github-api/pull/283 yet.

I am noticing that version 1.35 makes 1 req/repo.
Version 1.39 makes (1 req/ PR in repo) + (1 req/ PR in repo if there is a comment on the PR)
That is a significant increase in the # of requests.

Are you using hooks or only polling?

@bjoernhaeuser that diagnosis was for polling only.
We are in the process of switching to hooks now(we had to jump thru some hoops because our Jenkins setup is in a private/internal network) since that is a more scalable and reliable method.

We switched to hooks some month ago - also having a little bit too complex networking. Hooks work very great.

@toidiu, May I ask how you switched to hooks for a Jenkins server behind the firewall?

Any update on this? We've run into the same issue with hitting the rate limit despite the fact that we only have a few different repositories being polled. (We're hoping to ultimately use hooks.. but being behind a firewall makes this more difficult)
It seems like the solution is to use GitHub's conditional requests to keep from unnecessarily draining the rate limits.

Filed a separate issue for adding If-Modified-Since to request headers when polling PRs: #653

I recommend using webhooks and not polling.  There's tech such as webhookrelay.com which I've personally used if you don't want to expose your Jenkins instance to the internet.

thanks @sgleske-ias  for mentioning it :) Last year I've wrote a blog post on configuring it: https://webhookrelay.com/blog/2017/11/23/github-jenkins-guide/

Jenkins log is full of the messages about very old PRs (some are more than a year old):
Jul 10, 2018 11:14:42 AM INFO org.jenkinsci.plugins.ghprb.GhprbPullRequest updatePR
Pull request #1,117 was updated on repo <myorg>/<myrepo> but there aren't any new comments nor commits; that may mean that commit status was updated.

This repeats every few minutes, and nothing is changing in that PR.

@eplodn PR polling hits all open PRs, even every old ones. That's part of why peoples' quotas are getting drained so quickly.

Sad to see this six year old issue is still pending without any workaround or resolution. Might need to go ahead and try the webhook relay.


--verbose parameter is missing for az bicep

I am missing --verbose parameter for az bicep to get more information from error or waring like this for arm template like using PowerShell
Group
az bicep : Bicep CLI command group.
Commands:
build         : Build a Bicep file.
decompile     : Attempt to decompile an ARM template file to a Bicep file.
install       : Install Bicep CLI.
list-versions : List out all available versions of Bicep CLI.
publish       : Publish a bicep file to a remote module registry.
uninstall     : Uninstall Bicep CLI.
upgrade       : Upgrade Bicep CLI to the latest version.
version       : Show the installed version of Bicep CLI.
To search AI knowledge base for examples, use: az find "az bicep"
Please let us know how we are doing: https://aka.ms/azureclihats

I don't think we hide any information on az bicep build. What information are you looking for from a --verbose flag that is not already being emitted?

@alex-frankel I am pointing for debug/trace information corresponding would be
New-AzResourceGroupDeployment -DeploymentDebugLogLevel
https://docs.microsoft.com/en-us/powershell/module/az.resources/new-azresourcegroupdeployment?view=azps-7.5.0
Specifies a debug log level. The acceptable values for this parameter are:

RequestContent
ResponseContent
All
None

so you can ease trace/debug see all underlying request to resource provider


Support bare internal mappings

This PR includes an example and description of bare internal mappings as an extension to the proposal.
This has been discussed before, and opens up opportunities for extending the behaviours to internal requires allowing full control over both internal and external resolutions.

Two thoughts, not necessarily deal breakers:

How would this interact with the require algorithm? Supercede it?
How would this work for deep imports? We would check the package.json for every deep imported file? (I guess we're doing that anyway.)

One question: A common pattern in universal packages is to remap an implementation detail using the browser field in package.json. Would the expectation be that we'd encourage them to use a bare specifier for these? Or will we extend the relative path mappings to also apply to $packageBaseURL/$path?

I think in general, every path should be resolved/normalized at every opportunity, so that you can always use any path anywhere, and they map as one might intuitively expect.

can you clarify your question? I'm not sure I follow.

I think a high level concern is that these entries have different "hoisting" rules. We're hoisting import map entries that start with ./ to the parent scope but keep entries that don't at the package scope.
/project-root
  /package.json - { dependencies: "x" }
/deps
  /x
    /package.json - { exports: /* see example this PR modifies */  }
  /localdep
    /package.json

If we'd generate an import map (ignoring protocol etc. and pretending that this is all at the root), it might look like this:
{
  "scopes": {
    "/project-root": {
        "x": "/deps/x/src/moment.mjs",
        "x/": "/deps/x/src/util/",
        "x/timezones/": "/deps/x/data/timezones/",
        "x/timezones/utc": "/deps/x/data/timezones/utc/index.mjs"
    },
    "/deps/x": {
      "localdep": "/deps/x/third-party/localdep.mjs"
    }
  }
}

Which is fine, just pointing out that we need to look up the package.json relative to the file for this to work, rather than the one in the root project

Yes exactly this works since we now have a concept of package scope.

I think a high level concern is that these entries have different "hoisting" rules. We're hoisting import map entries that start with ./ to the parent scope but keep entries that don't at the package scope.

I'm not sure I'd consider it as hoisting, but rather just more as thinking in terms of resolution rules for "bare package entry scope resolution" and "internal package scope resolution".
Agree the generated import map would like your example though.

To actually come back to the question above ("will we extend the relative path mappings to also apply to $packageBaseURL/$path"), expressed as an import map, it would be the following:

Thanks that example clearly indicates the question now. Yes I don't think we should do this - the relative mappings should apply only for the bare specifier entry into the package.

Trying to apply this to a library that I maintain: https://github.com/groupon/gofer/blob/66dd001669091b9c1ba74e9dc08e08000761e80b/package.json#L8-L9. This is switching out a (non-public) implementation detail based on platform support. This proposal would suggest I'd do this for the standard module system:
{
  "exports": {
    "fetch-binding": ["./lib/fetch.browser.js", "./lib/fetch.js"]
  }
}

With the assumption that I can somehow guard the first entry somehow (e.g. #29). It feels a bit weird to use a bare specifier to "abstract away" an internal override but it may be okay..?

Right, so how do we guard the fetch.browser.js? Could we even do ["browser:./lib/fetch.browser.js", "./lib/fetch.js"]?
Note that one of the benefits of the indirection through a bare specifier is exactly ensuring a well-defined non-recursive resolver, as if we had relative paths become LHS entries in the map we would be introducing recursion :)

I think people will (rightfully?) complain about the string-based meta data. But I might also just be biased by NIH and lean towards the earlier [{ "browser": true, "from": "./lib/fetch.browser.js" }, "./lib/fetch.js"] strawman. But I think that no matter what we found multiple possible syntax solutions there so I think it's a problem of agreeing on one, not finding a workable solutions. I could live with "pseudo-protocols" even though they may interfere with real protocols.

My hope with conditionals is that we can not get too caught up in trying to craft the perfect generalization of environment detection (which is also a very difficult optimization IMO), but can rather focus on solving the use cases of having different entry points between modules browsers, legacy browsers, modules Node, legacy Node, and production and development environments can be done which I still see primarily as a combinatorial problem of the "main" / "module" / "browser" mains problem applied to arbitrary entry points.

Here is an example of this feature in real-world use in the aws-sdk package:
https://github.com/aws/aws-sdk-js/blob/master/package.json#L51
{
  "browser": {
    "lib/aws.js": "./lib/browser.js",
    "fs": false,
    "./global.js": "./browser.js",
    "./lib/node_loader.js": "./lib/browser_loader.js"
  },
  "browserify": {
    "transform": "./dist-tools/transform.js"
  },
  "react-native": {
    "fs": "./lib/empty.js",
    "./lib/node_loader.js": "./lib/react-native-loader.js",
    "./lib/browser_loader.js": "./lib/react-native-loader.js",
    "./lib/core.js": "./dist/aws-sdk-core-react-native.js",
    "xml2js": "./dist/xml2js.js"
  }
}

In the above plain mappings are used to:

Map fs requires to an empty module in electron and the browser
Map require('xml2js') to a local provided version in react native.

Note how the environments being checked here are very simply - browser, react-native, electron.
Basically, this gives direct package map support to packages, to control their own mappings, which is just as important as allowing control of the exported entry mappings it seems to me.
Any thoughts further on merging this?

browser is a defacto standard; having react-native follow that pattern seems like a good idea on their end.
Should perhaps, instead of exports, the key be named something environment-specific, if it's going to follow the lead of browser? node, perhaps?

Please can we discuss naming and conditions in a separate issue.
This PR is specifically about the case of supporting plain mappings like "fs" alongside the internal mappings like "./lib/node_loader.js" for export maps, following in the browser field behaviour.

So given that npm supports this already (https://github.com/jkrems/proposal-pkg-exports/pull/30/files#r264895480) why should node itself add support for it?

npm aliases provide global aliases that apply only to top-level dependencies. This feature provides aliases which are scoped only to the current package and remain portable with that package such that they still work when installed in node_modules.

I'm confused; I'm pretty sure npm aliases are per-package. It certainly only applies to top-level deps, but due to hoisting, you can add a transitive dep and it'll be the one used for your package.

Ahh, right, I see they are.
The point is more that if we support array maps like import maps support and want to support the alias use cases then this extends that to those, as well as any other features these maps support as well for object forms.

It seems more prudent to only allow non-bare paths here (ie, starts with . or /), so as not to overlap with the solution that npm already provides.

npm aliases don't offer the following features:

Mapping to a local file
Mapping to a subpath file of a package
Supporting array fallbacks if this proposal supports it (https://github.com/jkrems/proposal-pkg-exports/issues/31)
Supporting environment mapping forms if we support it (https://github.com/jkrems/proposal-pkg-exports/issues/29)

These all align with the use cases demonstrated in the example above.
npm aliasing is a different type of use case, which is different from the example above. I'm well aware of aliasing too - jspm implemented this feature back in 2013.

For the first, "whatever": "file://./a/local/file".
Mapping subpaths is a fair point, but you can map to a local folder that has all the subpaths you want as actual folders. Fallbacks don't make much sense for node because everything's loaded off the file system.
There's already environment mappings via browser; that's not something node needs to solve.

For the first, "whatever": "file://./a/local/file".

This doesn't work unless file is an actual folder to link I believe.

Fallbacks don't make much sense for node because everything's loaded off the file system.

We're thinking about the far-future here were import maps fallbacks as a feature allow environment handling. Consider environments with native support for wasi:... etc. where the fallback can allow a form of polyfill detection.
I can certainly appreciate if you think we should come back around on this one later though, but I'm just trying to ensure this spec captures as much of the features as the browser field as possible, which have been proven to be useful.

I assume this is outdated and was replaced by https://github.com/jkrems/proposal-pkg-exports/pull/40 - feel free to reopen if that isn't correct!


nixos/doc: convert 21.05 release notes to CommonMark

Since people are still editing the 21.05 release notes when doing backports, we should backport this conversion. Warning: this may mess up existing release note changes people have made but not merged yet. Our only other alternative is to wait until 2022 to convert this section of the manual, which I think is too long to wait.

@GrahamcOfBorg eval

@ryantm fixed in force-push. Thanks!

@GrahamcOfBorg eval


Quest v0.2/ G6PD: Get a patient's DOB by entering their age as an integer

Name of feature to enhance
Add a client's DOB by entering their age as an integer
Description of feature
Add on the registration questionnaire for Quest the ability to add age either an integer or actual date, add a selection before the DOB as seen in the screenshot below
Describe the enhancement

Add a skip logic question that asks a user to either enter date as a number or date
If Date show a date picker else if number enter a numerical filed
On submit, if used date picker submit date, if integer, get the value and compute the DOB i.e if today is 2021, and entered 21, compute DOB as 2021-21 = 01-01-2020

NB. This ticket is related to #652 and will be done via a structure map , and adding calculation of attributes in the questionnaire

how does this differ from #652?

how does this differ from #652?

https://github.com/opensrp/fhircore/issues/652#issuecomment-973797214


Custom widget issue.

I have an issue with adding custom widget to survey toolbox.
I'm trying to create a component with some logic. For the first i had an issue with components names, but after I've read this article I've decided to create custom widget.
Are you requesting a feature, reporting a bug or asking a question?
Asking a question.
What is the current behavior?
I've already added a widget (according to this example), but when i'm trying to select it from toolbox i have a type error

What is the expected behavior?
Understand how to add custom widgets into my toolbox, for next usage.
Provide the test code and the tested page URL (if applicable)
Test code
Widget code the same as in example;
Adding to toolbar:
    editor.toolbox.addItem({
        name: "customitem",
        category: "Waivers",
        iconName: "icon-text",
        isCopied: false,
        title: "custom item",
        json: {
            type: "textwithbutton",
            name: "q1",
            placeHolder: "put some text here",
            buttonText: "Custom button text"
        },
    });

Specify your

surveyjs platform: jQuery:
surveyjs version: 1.0.64 - 1.0.74

@andrewtelnov Do we need to transfer this issue to the corresponding repository?

@kyparyt I have added your code into our custom widget example and it works fine. Here is the example
Thank you,
Andrew

@kyparyt You are getting the error, because in your code, for some reason, this line is not executed:
Survey.JsonObject.metaData.addClass("textwithbutton", [], null, "text");
It registered a new class in the library.
Thank you,
Andrew

Thank you. It helped me to find solution for my project ;)