basic PubSub Error connect failed -1

config is as follows
AWS_IOT_MQTT_HOST "**************.iot.ap-northeast-1.amazonaws.com"
#define AWS_IOT_MQTT_PORT 8883									// your port
#define AWS_IOT_CLIENT_ID	"seeeduino"						// your client ID
#define AWS_IOT_MY_THING_NAME "seeeduino"						// your thing name
#define AWS_IOT_ROOT_CA_FILENAME "rootCA.pem"           // your root-CA filename
#define AWS_IOT_CERTIFICATE_FILENAME "certificate.pem.crt"                 // your certificate filename
#define AWS_IOT_PRIVATE_KEY_FILENAME "private.pem.key"              // your private key filename

Hi @Daniel-Git-Hub ,
Thank you very much for using AWS IoT Arduino Yun SDK!
As the initial debugging step, can you go ahead and verify the integrity of your connectivity information and credentials using another MQTT client, e.g., MQTT.fx, and see if you can connect?
Please let us know how it goes.
Thanks,
Liusu

thanks yes I can connect to that.
But I fully reset it (using a seeduino) and then re uploaded everything and set it up again
and I could connect using the default MQTT client


enhance jenkins helper functions

refine jenkins helper function, e.g. get_build_status etc
add new func for jenkins related notification in notification_mgr
do local tests as fellows

$ oar -r 4.12.30 image-consistency-check
2023-08-28T16:29:28Z: INFO: job id is not set, will trigger image consistency check job
2023-08-28T16:29:33Z: INFO: triggered image consistency check job: <https://mastern-jenkins-csb-openshift-qe.apps.ocp-c1.prod.psi.redhat.com/job/image-consistency-check/2407/>
2023-08-28T16:29:35Z: INFO: sent slack message to <#release-tests>
2023-08-28T16:29:36Z: INFO: task [Payload-Images Consistency Testing] status is changed to [In Progress]

$ oar -r 4.12.30 image-consistency-check
2023-08-28T16:39:49Z: INFO: job id is not set, will trigger image consistency check job
2023-08-28T16:39:54Z: INFO: triggered image consistency check job: <a new job triggered by you is pending, you can check queue item link: https://mastern-jenkins-csb-openshift-qe.apps.ocp-c1.prod.psi.redhat.com/queue/item/141331/api/json>
2023-08-28T16:39:56Z: INFO: sent slack message to <#release-tests>
2023-08-28T16:39:57Z: INFO: task [Payload-Images Consistency Testing] status is changed to [In Progress]

$ oar -r 4.12.30 image-consistency-check -n 2408
2023-08-28T16:43:33Z: INFO: check image-consistency-check job status with job id:2408
2023-08-28T16:43:37Z: INFO: build status of image-consistency-check/2408 is In Progress
2023-08-28T16:43:38Z: INFO: task [Payload-Images Consistency Testing] status is changed to [In Progress]

$ oar -r 4.12.30 image-consistency-check -n 2407
2023-08-28T16:44:25Z: INFO: check image-consistency-check job status with job id:2407
2023-08-28T16:44:28Z: INFO: build status of image-consistency-check/2407 is ABORTED
2023-08-28T16:44:29Z: INFO: task [Payload-Images Consistency Testing] status is changed to [Fail]
2023-08-28T16:44:30Z: INFO: Overall status is updated to Red

validation for job params

$ oar -r 4.12.30 image-consistency-check -n 2405
2023-08-28T16:47:31Z: INFO: check image-consistency-check job status with job id:2405
Traceback (most recent call last):
  File "/usr/local/bin/oar", line 8, in <module>
    sys.exit(main())
             ^^^^^^
  File "/Users/rio.liu/coderepo/release-tests/oar/cli/__main__.py", line 6, in main
    cli(obj={})
  File "/usr/local/lib/python3.11/site-packages/click/core.py", line 1130, in __call__
    return self.main(*args, **kwargs)
           ^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/usr/local/lib/python3.11/site-packages/click/core.py", line 1055, in main
    rv = self.invoke(ctx)
         ^^^^^^^^^^^^^^^^
  File "/usr/local/lib/python3.11/site-packages/click/core.py", line 1657, in invoke
    return _process_result(sub_ctx.command.invoke(sub_ctx))
                           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/usr/local/lib/python3.11/site-packages/click/core.py", line 1404, in invoke
    return ctx.invoke(self.callback, **ctx.params)
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/usr/local/lib/python3.11/site-packages/click/core.py", line 760, in invoke
    return __callback(*args, **kwargs)
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/usr/local/lib/python3.11/site-packages/click/decorators.py", line 26, in new_func
    return f(get_current_context(), *args, **kwargs)
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/Users/rio.liu/coderepo/release-tests/oar/cli/cmd_image_consistency_check.py", line 64, in image_consistency_check
    job_status = jh.get_build_status(
                 ^^^^^^^^^^^^^^^^^^^^
  File "/Users/rio.liu/coderepo/release-tests/oar/core/jenkins_helper.py", line 87, in get_build_status
    raise JenkinsHelperException(
oar.core.exceptions.JenkinsHelperException: please make sure this build image-consistency-check/2405 is triggered for 4.12.30, cannot find 4.12.30 or 4.12.0-0.nightly-2023-08-17-053025 in payload url registry.ci.openshift.org/ocp/release:4.12.0-0.nightly-2023-08-26-180408

$ oar -r 4.12.30 stage-testing
2023-08-28T16:56:15Z: INFO: job id is not set, will trigger stage testing
2023-08-28T16:56:21Z: INFO: triggered stage pipeline job: <https://mastern-jenkins-csb-openshift-qe.apps.ocp-c1.prod.psi.redhat.com/job/zstreams/job/Stage-Pipeline/824/>
2023-08-28T16:56:23Z: INFO: sent slack message to <#release-tests>
2023-08-28T16:56:25Z: INFO: task [Stage testing] status is changed to [In Progress]

$ oar -r 4.12.30 stage-testing -n 823
2023-08-28T16:58:47Z: INFO: check stage job status according to job id:823
2023-08-28T16:58:51Z: INFO: build status of zstreams/Stage-Pipeline/823 is ABORTED
2023-08-28T16:58:52Z: INFO: task [Stage testing] status is changed to [Fail]
2023-08-28T16:58:53Z: INFO: Overall status is updated to Red

$ oar -r 4.12.30 stage-testing -n 822
2023-08-28T16:57:57Z: INFO: check stage job status according to job id:822
Traceback (most recent call last):
  File "/usr/local/bin/oar", line 8, in <module>
    sys.exit(main())
             ^^^^^^
  File "/Users/rio.liu/coderepo/release-tests/oar/cli/__main__.py", line 6, in main
    cli(obj={})
  File "/usr/local/lib/python3.11/site-packages/click/core.py", line 1130, in __call__
    return self.main(*args, **kwargs)
           ^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/usr/local/lib/python3.11/site-packages/click/core.py", line 1055, in main
    rv = self.invoke(ctx)
         ^^^^^^^^^^^^^^^^
  File "/usr/local/lib/python3.11/site-packages/click/core.py", line 1657, in invoke
    return _process_result(sub_ctx.command.invoke(sub_ctx))
                           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/usr/local/lib/python3.11/site-packages/click/core.py", line 1404, in invoke
    return ctx.invoke(self.callback, **ctx.params)
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/usr/local/lib/python3.11/site-packages/click/core.py", line 760, in invoke
    return __callback(*args, **kwargs)
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/usr/local/lib/python3.11/site-packages/click/decorators.py", line 26, in new_func
    return f(get_current_context(), *args, **kwargs)
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/Users/rio.liu/coderepo/release-tests/oar/cli/cmd_stage_testing.py", line 64, in stage_testing
    job_status = jh.get_build_status(
                 ^^^^^^^^^^^^^^^^^^^^
  File "/Users/rio.liu/coderepo/release-tests/oar/core/jenkins_helper.py", line 87, in get_build_status
    raise JenkinsHelperException(
oar.core.exceptions.JenkinsHelperException: please make sure this build zstreams/Stage-Pipeline/822 is triggered for 4.12.30, cannot find 4.12.30 or 4.12.0-0.nightly-2023-08-17-053025 in payload url quay.io/openshift-release-dev/ocp-release:4.13.10-x86_64

@todo: support trigger job with nightly build

/assign @wewang58
/cc @jianzhangbjz @Xia-Zhao-rh

/retitle OCPQE-17477 enhance jenkins helper functions


how to use comparison?

comparison: Any, optional. If provided, a new request is not made if the comparison value between dispatch calls is the same.
i want the app to only do an initial fetch when the component first load and ignore all subsequent updates, and I am wonder if the comparison flag can be used for this purpose? What do I pass into comparison? It says it can accept any value type? Can you please provide an example?
Thanks!

Hi, indeed, comparison could be used for that. Just give it a unique value (string or number).
Another option would be to check if the state already has the data, and only execute the dispatch method if the state doesn't have the value yet:
class MyComponent extends React.Component {
  componentWillMount() {
    if (!this.props.exampleFetch.value) {
      this.props.dispatchExampleGet();
    }
  }
  // ...
}

reduxFetch([
  {
     resource: 'example',
     request: {
       url: '/api/example'
     }
  }
])(MyComponent);


Unable to pass a function to session.notify (misleading docstring?)

Current Behavior
The docs and docstring for session.notify say that the target "may be specified as the appropriate string [...] or using the function object". I assumed this meant you could pass a @nox.session-decorated function into session.notify. However, the type annotation (target: str | SessionRunner) doesn't permit this, and indeed it doesn't work:
nox > Running session notify
nox > Creating virtual environment (virtualenv) using python.exe in .nox\notify
notify
nox > Session notify raised exception ValueError('Session <nox._decorators.Func object at 0x0000028516004310> not found.')
Traceback (most recent call last):
  File "C:\Users\...\venv\Lib\site-packages\nox\sessions.py", line 761, in execute
    self.func(session)
  File "C:\Users\...\venv\Lib\site-packages\nox\_decorators.py", line 75, in __call__
    return self.func(*args, **kwargs)
           ^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "C:\Users\...\noxfile.py", line 22, in notify
    session.notify(target)
  File "C:\Users\...\venv\Lib\site-packages\nox\sessions.py", line 641, in notify
    self._runner.manifest.notify(target, posargs)
  File "C:\Users\...\venv\Lib\site-packages\nox\manifest.py", line 319, in notify
    raise ValueError(f"Session {session} not found.")
ValueError: Session <nox._decorators.Func object at 0x0000028516004310> not found.

Am I misinterpreting the docstring, or is this not actually supported?
Expected Behavior
Passing a function should run that session as documented.
Steps To Reproduce
With this noxfile:
import nox

@nox.session
def target(session: nox.Session):
    print("target")

@nox.session
def notify(session: nox.Session):
    print("notify")
    session.notify(target)

Run nox -s notify. The above error is raised.
For comparison, this noxfile works as expected:
import nox

@nox.session
def target(session: nox.Session):
    print("target")

@nox.session
def notify(session: nox.Session):
    print("notify")
    session.notify("target")  # this is the only difference

Environment
- OS: Windows 11
- Python: 3.11.0
- Nox: 2023.4.22

Anything else?
It seems like this behaviour/documentation may have been present since all the way back when notify was introduced (d6b3a5ad47d4cc616c821ec1bef82a69ba774fd0)?
Session.notify:
        Args:
            target (Union[str, Callable]): The session to be notified. This
                may be specified as the appropropriate string or using
                the function object.

Manifest.notify:
        Args:
            session (Union[str, ~nox.session.Session]): The session to be
                enqueued.

target appears to be passed verbatim to session, so I'm not sure if this ever actually worked.

The simplest fix would be to compare s.func == session here:
https://github.com/wntrblm/nox/blob/5c82dc553bd04ee017784fc16193da0b35a44ab6/nox/manifest.py#L313
If we go this way, the parameter type would need to change. I don't think there's a reasonable way to get hold of the SessionRunner object in the Noxfile. I'm not sure how we should type the parameter Session.notify and Manifest.notify. Either of these maybe?
https://github.com/wntrblm/nox/blob/5c82dc553bd04ee017784fc16193da0b35a44ab6/nox/registry.py#L26
https://github.com/wntrblm/nox/blob/5c82dc553bd04ee017784fc16193da0b35a44ab6/nox/_decorators.py#L59
They're currently not part of the public interface.

We should also consider changing the docstring and type to simply disallow this. It never worked, and I'm not sure there's a use case for it.

IMO, we should change the docstring for now, and not add this until after working on any reworks planned, like #631 or https://github.com/wntrblm/nox/issues/167#issuecomment-2040967680, as it might complicate those. Long term, I think it makes sense.

If you pass the function instead of a string, typos can be caught at type-check time (in other words, you get a wiggly line in your editor).

This would be great! Here is a workaround I'm currently using to get this:
@nox.session()
def some_session(session: nox.Session):
    session.notify(some_other_session.__name__)

Perhaps this could suggested in the docstring if useful?


[SPARK-42759][BUILD] Avoid duplicated build/apache-maven install when target already exists

What changes were proposed in this pull request?
This pr is just a minor improvement. After SPARK-42673, Spark start to use fixed maven for build and test, but when the target maven version is not same with the system default version, it will cause duplicated install_app process, so this pr add a target directory check condition to skip this process when the target directory exists.
Why are the changes needed?
Avoid duplicated build/apache-maven install when target already exists.
Does this PR introduce any user-facing change?
No
How was this patch tested?
Manual test

Make the system maven version different from Spark requirements, such as 3.9.0
Then run sh -x build/mvn -version multiple times

Before
We always see the following message even if target dir already exists.
+ '[' 003009000 -ne 003008007 ']'
+ local MVN_TARBALL=apache-maven-3.8.7-bin.tar.gz
+ local FILE_PATH=maven/maven-3/3.8.7/binaries/apache-maven-3.8.7-bin.tar.gz
+ local APACHE_MIRROR=https://www.apache.org/dyn/closer.lua
+ local 'MIRROR_URL_QUERY=?action=download'
++ command -v curl
+ '[' /usr/bin/curl ']'
+ curl -L --output /dev/null --silent --head --fail 'https://www.apache.org/dyn/closer.lua/maven/maven-3/3.8.7/binaries/apache-maven-3.8.7-bin.tar.gz?action=download'
+ install_app https://www.apache.org/dyn/closer.lua maven/maven-3/3.8.7/binaries/apache-maven-3.8.7-bin.tar.gz '?action=download' sha512 apache-maven-3.8.7-bin.tar.gz apache-maven-3.8.7/bin/mvn
+ local mirror_host=https://www.apache.org/dyn/closer.lua
+ local url_path=maven/maven-3/3.8.7/binaries/apache-maven-3.8.7-bin.tar.gz
+ local 'url_query=?action=download'
+ local checksum_suffix=sha512
+ local local_tarball=/Users/${userName}/spark/build/apache-maven-3.8.7-bin.tar.gz
+ local binary=/Users/${userName}/spark/build/apache-maven-3.8.7/bin/mvn
+ local 'remote_tarball=https://www.apache.org/dyn/closer.lua/maven/maven-3/3.8.7/binaries/apache-maven-3.8.7-bin.tar.gz?action=download'
+ local local_checksum=/Users/${userName}/spark/build/apache-maven-3.8.7-bin.tar.gz.sha512
+ local remote_checksum=https://archive.apache.org/dist/maven/maven-3/3.8.7/binaries/apache-maven-3.8.7-bin.tar.gz.sha512
+ local 'curl_opts=--silent --show-error -L'
+ local wget_opts=--no-verbose
+ '[' '!' -f /Users/${userName}/spark/build/apache-maven-3.8.7/bin/mvn ']'
+ MVN_BIN=/Users/${userName}/spark/build/apache-maven-3.8.7/bin/mvn
+ cd /Users/${userName}/spark
+ export 'MAVEN_OPTS=-Xss128m -Xmx4g -XX:ReservedCodeCacheSize=128m'
+ MAVEN_OPTS='-Xss128m -Xmx4g -XX:ReservedCodeCacheSize=128m'
+ echo 'Using `mvn` from path: /Users/${userName}/spark/build/apache-maven-3.8.7/bin/mvn'
Using `mvn` from path: /Users/${userName}/spark/build/apache-maven-3.8.7/bin/mvn
+ '[' '!' -z '' ']'
+ '[' '!' -z localhost ']'
+ echo 'Using SPARK_LOCAL_IP=localhost'
Using SPARK_LOCAL_IP=localhost
+ /Users/${userName}/spark/build/apache-maven-3.8.7/bin/mvn -version
Apache Maven 3.8.7 (b89d5959fcde851dcb1c8946a785a163f14e1e29)
Maven home: /Users/${userName}/spark/build/apache-maven-3.8.7
Java version: 1.8.0_362, vendor: Azul Systems, Inc., runtime: /Users/${usrNaem}/Tools/zulu8/zulu-8.jdk/Contents/Home/jre
Default locale: zh_CN, platform encoding: UTF-8
OS name: "mac os x", version: "13.2.1", arch: "aarch64", family: "mac"
+ MVN_RETCODE=0
+ exit 0

After
We no longer see install_app message if target dir already exists
+ '[' 003009000 -ne 003008007 ']'
+ local MVN_DIR=apache-maven-3.8.7
+ '[' '!' -d /Users/${userName}/spark/build/apache-maven-3.8.7 ']'
+ MVN_BIN=/Users/${userName}/spark/build/apache-maven-3.8.7/bin/mvn
+ cd /Users/${userName}/spark
+ export 'MAVEN_OPTS=-Xss128m -Xmx4g -XX:ReservedCodeCacheSize=128m'
+ MAVEN_OPTS='-Xss128m -Xmx4g -XX:ReservedCodeCacheSize=128m'
+ echo 'Using `mvn` from path: /Users/${userName}/spark/build/apache-maven-3.8.7/bin/mvn'
Using `mvn` from path: /Users/${userName}/spark/build/apache-maven-3.8.7/bin/mvn
+ '[' '!' -z '' ']'
+ '[' '!' -z localhost ']'
+ echo 'Using SPARK_LOCAL_IP=localhost'
Using SPARK_LOCAL_IP=localhost
+ /Users/${userName}/spark/build/apache-maven-3.8.7/bin/mvn -version
Apache Maven 3.8.7 (b89d5959fcde851dcb1c8946a785a163f14e1e29)
Maven home: /Users/${userName}/spark/build/apache-maven-3.8.7
Java version: 1.8.0_362, vendor: Azul Systems, Inc., runtime: /Users/${userName}/Tools/zulu8/zulu-8.jdk/Contents/Home/jre
Default locale: zh_CN, platform encoding: UTF-8
OS name: "mac os x", version: "13.2.1", arch: "aarch64", family: "mac"
+ MVN_RETCODE=0
+ exit 0

seems the problem is not serious. close this pr

install_app already check target


UX improvements for sharing local files

Something that deserves own mini-journey:  sharing local files via IPFS
It is a silent one, but honestly this is a "killer feature" when it comes to convenience.
What we have right now:

"Share local files via IPFS" in Companion's browser action menu opens the upload tab
support for drag&drop ✨  (latest beta)
preload at public gateway
ugly URL: produces raw CID

Some ideas on how to make it a better experience:

[x] Display IPFS addresses instead of opening uploaded file (tracked in #423)

[x] Partially delivered by directory wrapping (PR #479)

[x] Save all uploads to MFS (ipfs.files) (tracked in #415)
[x] Progress indicator (tracked in #295)
[x] Keeping original filename in URL by wrapping a file in a directory  (tracked in https://github.com/ipfs-shipyard/ipfs-companion/issues/349, added in PR #479)
[x] Sharing multiple files  (added in PR #479)
[ ] Sharing directories  (tracked in #396)
[ ] Optional symmetric encryption (catering to less savvy users)  (https://github.com/ipfs-shipyard/pm-ipfs-gui/issues/53)
[x] Handle / fix crash when dropping a big file #464
[ ] Feature: option to automatically copy url when adding to ipfs #576
[ ] Support local camera or external services

see https://uppy.io

Related

Design aspect of "Sharing" is currently tracked in https://github.com/ipfs-shipyard/pm-ipfs-gui/issues/34

Closing this one, since its remaining components are tracked in their own issues.


[docs] CONTRIBUTING.md - how to setup SQL Lab in async mode

Rendered markdown here: https://github.com/mistercrunch/superset/blob/async_contrib/CONTRIBUTING.md#sql-lab-async
@xtinec @enricoberti

Codecov Report

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

@@           Coverage Diff           @@
##           master    #7103   +/-   ##
=======================================
  Coverage   64.59%   64.59%
=======================================
  Files         422      422
  Lines       20583    20583
  Branches     2251     2251
=======================================
  Hits        13295    13295
  Misses       7166     7166
  Partials      122      122

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 23e823f...4c50fc3. Read the comment docs.

nice one, thanks! i added a couple of comments


feat(loginmodal): update styles for privacy policy (1.11)

Update styles for privacy policy.
Closes DCOS-22401
Checklist

[x] Did you add a JIRA issue in a commit message or as part of the branch name?
[ ] Did you add new unit tests?
[ ] Did you add new integration tests?
[ ] If this is a regression, did you write a test to catch this in the future?

https://github.com/mesosphere/dcos-ui-plugins-private/pull/790

Please don't review as discussions are made.


Deserialization callback method

Is there a way to mark a method on the object as a method that gets called after object is deserialized?  This can be used to handle initializing helper properties or perform calculations.  Right now, I manually call a function after deserializing an object but it would be nice if that's built in for the times where we have several services deserializing the same object type.

Can you make a minimal example?


Unneeded array init

Tags were set incorrectly

I fixed the ts.

Just noticed I committed the js only... :-)


Enable logging of any Real

change signature of log_value to be less specific
add some tests for log_value

Looks good.
Thanks a lot!


Python is easier

Switch to a single path Python implementation.

Keeping the Go version around in case I regret this


Clientts

Thanks for your contribution @VolodymyrBg. We need a documentation style guide otherwise these changes are difficult to asses. In its current form, I'm not sure if this adds value so I'm going to close this PR.


Connect mailchimp to email signup modal

Under /docs/includes/modal_email.html is the pop-up box used on the homepage when someone clicks the 'Get Updates' button. This has been populated with some template form fields, but these need to be connected so that the submit button actually does something.
We'll use our mailchimp account to collect the information. They allow for custom signup pages.
Instructions on how to add the necessary data to the form fields:
http://kb.mailchimp.com/lists/signup-forms/host-your-own-signup-forms
Signup form to copy info from:
http://eepurl.com/ckGOhr
Ask @nealhumphrey for log in information to the Mailchimp account so that we can perform testing.
'Ready to merge' criteria:

All 4 fields are linked
When someone signs up using the modal, their information appears in mailchimp
The hidden field 'role' field is included in the modal, OR it automatically populates in Mailchimp without being added to the form. Rather than add a 5th question, admins can change the value based on the answer to the 'where do you work' question to know whether to send coding-focused newsletter or housing-focused newsletter.

Note, the form fields were pulled from another bootstrap-based site I made, so there are likely some redundant / unused styles and divs - feel free to remove as needed.
In addition, there is code for hidden error messages, but currently no accompanying Javascript to reveal these messages via validation. Bonus would be to add this javascript, but since there will likely not be a huge volume of signups I consider this mostly beyond current scope.

Completed, thanks @ostermanj


How can you scan with hbase rest interface

I was wondering if there is anyway I can scan, filter etc via rest interface of hbase

Hi debarshri,
To use scanner you essentially need to follow 3 steps:
1.) Create a scanner using PUT request This will return you a scanner location.
You can add your filters here using json inside xml  tag.

write your filter here in json

To check filter syntax you can stringify the filter you want in java and have a look at the json syntax.
Every value you use should be Base64 encoded.
2.) Use GET request to read from the scanner location. It can be used to get data in json,xml and       protbufs.
3.) Use DELETE request to delete the scanner.
Here is the documentation link for using above methods: http://wiki.apache.org/hadoop/Hbase/Stargate


Always warns about running without elevated rights.

I have a Windows 10 laptop and a Windows 7 desktop. Using ember-cli-windows worked great configuring the Windows 10 laptop. It builds quickly and does not warn of not having elevated rights. The Windows 7 machine, however, always warns about not having elevated rights. I have tried:

Running git-bash versus windows powershell.
Running as administrator (which I always do) via right-click
Configuring the shortcut to run as administrator.
Running Set-ExecutionPolicy Unrestricted -scope Process
Of course, running ember-cli-windows
Disabling User Access Control.
I run from "an" administrator account, but apparently there is a hidden superuser administrator account that I am not using (and am not keen to use due to having to reinstall lots of stuff for that user).

In addition and probably unrelated, when I load the page in the browser, the throbber continues to spin and the livereload server never completes the way it does on the Windows 10 laptop.
I have no idea what useful information I can give you. My build times are about 50 seconds from starting ember s and about 8-10 seconds for rebuilding after a file change. Any clue what I might try next?

Hm, that's weird - we just test if PowerShell can verify you as an Administrator. That being said, I heavily recommend Windows 10, or at least Windows 8.1. Windows 7 was released years before Ember was invented, and many years before Ember Cli became a thing. I don't actively develop this for Windows 7 and rely on Pull Requests to make it work - if you find what's wrong, I'll gladly include any fixes.
To help you out, we could add an option that overrides the administrator check and attempts to execute no matter what. I can add that tomorrow - would that help?

@felixriesenberg -- sure. Worth trying. How exactly to you verify that the user is an Administrator? Is there any debugging info that I can give you that would be helpful?
I also wonder if the odd behavior of livereload not completing might be somehow related. Updating to Windows 10 is an option, but I'm not much of a fan. :)

@felixrieseberg - Thank you for the Windows OS version tip. I upgraded 7 -> 10 and both the LiveReload and complaining about not having elevated rights problems went away (without making any other changes). So, please don't make that special version on my account. And thank you for work contribution to the Ember Windows community. :100:

Happy to hear that things are working for you now :clap:!


Support docker cloudera/quickstart

Now that we have a simple way of testing Impala with Docker, It would be great to also have an image that includes the Kudu integration.

Could you provide a little more context? What's the "simple way of testing Impala with Docker" that you're alluding to?

Docker scripts have been merged into the main Apache Kudu repository.


.githubRepoInfo.json not being generated

Looks like this file is required for the website to run on the localhost.
Issue persisted through multiple node/ npm versions.
Git pull/ push were working fine without the fine.
The file was added to .pretrierignore during commit 69c5ce3
Current solution: Obtain the file from someone else and keep it in project root.

You need to add a GitHub auth token to the .env file, e.g. :
FAIRTRACKS_GITHUB_AUTH_TOKEN = 'ghp_hereISmyTOKEN123'
OPTIMIZE_IMAGES_IN_DEV = false

Then the file should autogenerate. If you have an existing token there, it might have expired.

That being said (and if there is no other reason for the issue) this should be better documented and a fallback solution should be implemented, so we should keep this issue for that.

I do have a GitHub auth token, which I set up more than a year ago (it is set for not expiring) and I can push/pull from GitHub without any problem. The credential.helper is set to store, and the credentials are saved in the .git-credentials file. Is there anything in my setup which is not compatible with how things are supposed to be working?

I do have a GitHub auth token, which I set up more than a year ago (it is set for not expiring) and I can push/pull from GitHub without any problem. The credential.helper is set to store, and the credentials are saved in the .git-credentials file. Is there anything in my setup which is not compatible with how things are supposed to be working?

You should be able to generate a new token (leaving the old one untouched) and just add the new one to the .env file.

That being said (and if there is no other reason for the issue) this should be better documented and a fallback solution should be implemented, so we should keep this issue for that.

I agree, the repo shouldn't be coupled with github tokens especially since it's a public codebase. One should be able to simply clone/ fork and get it running even without a github account.


Add link of vim plugin

This changes add the link to a vim plugin which supports the GhostText.

Thank you for your contribution. ☺


Test case for https://hibernate.atlassian.net/browse/HHH-8581

I filed https://hibernate.atlassian.net/browse/HHH-8581; this pull request contains a test case demonstrating the bug.

HHH-8581 was closed


How to rotate the Text?

Version of embedded-graphics in use (if applicable): 0.7.1
Target: real hardware
If using real hardware: OLED 1.3'' I2C

Description of the problem
I would like render two text objects. The first - horizontal, the second - vertical.
How to do it?
    let builder = Builder::new()
    .with_rotation(DisplayRotation::Rotate90)
    .with_size(DisplaySize::Display128x64)
    .connect_i2c(i2c);

    let mut display: GraphicsMode<_> = builder.into();

    display.init().unwrap();
    display.flush().unwrap();

    let text_style = MonoTextStyleBuilder::new()
    .font(&FONT_6X10)
    .text_color(BinaryColor::On)
    .build();

    let text_point = Point::new(0, 0);
    Text::with_baseline("DEVICE", text_point, text_style, Baseline::Top).draw(&mut display).unwrap();

   // Rotate
    let text_point_rotated = //;
    Text::with_baseline("IP: 111.222.333.444", text_point_rotated, text_style, Baseline::Top).draw(&mut display).unwrap();

Which type of vertical text do you mean?
 or
There is no builtin support for either variant at the moment. But you can archive both by using some additional code. To implement the 2nd version you can implement a DrawTarget adapter, which rotates all drawing operations:
impl<'a, T> RotatedDrawTarget<'a, T>
where
    T: DrawTarget,
{
    pub fn new(parent: &'a mut T) -> Self {
        Self { parent }
    }
}

impl<T> DrawTarget for RotatedDrawTarget<'_, T>
where
    T: DrawTarget,
{
    type Color = T::Color;
    type Error = T::Error;

    fn draw_iter<I>(&mut self, pixels: I) -> Result<(), Self::Error>
    where
        I: IntoIterator<Item = Pixel<Self::Color>>,
    {
        let parent_width = self.parent.bounding_box().size.width as i32;

        self.parent.draw_iter(
            pixels
                .into_iter()
                .map(|Pixel(p, c)| Pixel(Point::new(parent_width - p.y, p.x), c)),
        )
    }
}

impl<T> Dimensions for RotatedDrawTarget<'_, T>
where
    T: DrawTarget,
{
    fn bounding_box(&self) -> Rectangle {
        let parent_bb = self.parent.bounding_box();
        Rectangle::new(
            parent_bb.top_left,
            Size::new(parent_bb.size.height, parent_bb.size.width),
        )
    }
}

And use it like this:
  Text::new(
        "Rotated text",
        Point::new(15, 15),
        MonoTextStyle::new(&FONT_5X8, BinaryColor::On),
    )
    .draw(&mut RotatedDrawTarget::new(&mut display))?;

RotatedDrawTarget

Thanks for reply.
Look

Here is an example how this could be implemented in e-g: https://gist.github.com/rfuest/461b47aa16de08aa21be5428a107b9f7

@rfuest You are great! Thx!


rfc: benchmark automation

Once the first benchmarking system #372 lands we can technically start tracking performance. However as hinted at in that PR we need to decide on and resolve a few things in order to fully streamline/benefit.
Principals

Benchmarks need to be run on a consistently spec'd machine that is accessible to any contributor
Benchmarks that are saved should be run on a regular pattern of some sort
Benchmark results (not saved) per commit in a PR should be reviewable (mvp: results for latest commit)
Results should be easy to find, read, link directly to

Achieving Principals

Benchmarks should be run by CCI which guarantees the same machine spec and is accessible to all contributors.

Benchmarks that are saved should be run for master commits only, and only those that are for graphqlgen package, and only those that could affect perf.

filter: git diff must include graphqlgen package
filter: commit message must have one of conventional commit types {feat,perf,fix,refactor}
CCI will run benchmarks, save results, git commit, git push to master
related thread specific to cci, see comments at end

For a branch with an open PR, for each commit, CCI will:

run benchmarks (not save results)
fetch latest saved results
upsert a comment in the PR with new results and diff from last saved results
requires a github api token accessible to the ci
requires a special string "hook" in comment contents to achieve upsert (at Dialogue where I work, we used a markdown comment to achieve this).

An idea; Use codesandbox that pulls in the history.json data from the repo master branch, then uses either https://www.taucharts.com or https://www.chartjs.org. Then we can link to it from the readme or, ideally, if Github ever adds embed support we could put it directly in the readme.
Variations of this idea:

Generate a png chart at build time
Embed the codesandbox into the static doc site
Forgo codesanbox and just use the charting tools directly

I think before investing much more effort into ideas in this issue, we should see how our real world usage of #372 plays out. How we use it, refer to it, etc.


Replacer trait for str is being ignored to determine type info for replace argument on replace_all calls

When attempting to call replace_all on a Regex with a str as the argument for rep(as opposed to a NoExpand), the Replacer trait implementation for str is apparently being ignored, instead falling back to the FnMut implementation. This project demonstrates the following compile error:
src\main.rs:13:23: 13:53 error: the trait `for<'r,'r> core::ops::Fn<(&'r regex::re::Captures<'r>,)>` is not implemented for the type `str` [E0277]
src\main.rs:13         true => regex.replace_all(&source, &replace.as_str()),

Where replace and source are both Strings.
Edit: just fixed the type of replace by adding as_str, as the & was not coercing the String to a str properly. See the latest commit.

If you use replace.as_str() instead of &replace.as_str(), then your code works. The as_str method returns a &str, so &replace.as_str() has type &&str, and no impl for that exists.
I wonder if we can add a impl<'a, T: Replacer> &'a T for Replacer { ... } impl that would fix this. The standard library uses this convention AFAIK.

I feel like it's probably not worth adding this impl unless there's a more solid reason. (It could be added in a backwards compatible manner later if we really wanted to, I think.)


Problem with CommonJS.

Hi, first of all thank you for really great library.
There is one small problem, basically I can't use this library in project with CJS (example NestJS). Whenever I'm trying to import the library I'm getting:
Cannot find module 'swissqrbill/pdf' or its corresponding type declarations.
Cannot find module 'swissqrbill/utils' or its corresponding type declarations.

Can you help me with that?

Can you change moduleResolution in your tsconfig.json to  "bundler", "node16", or "nodenext"?

this is the problem that I can't change it because my whole NextJS application will stop working.
Can you have some workaround for this?

Hi, this was my first thought but I can't change it because my NestJs application stop working.
The "module" must stay as "commonjs" otherwise everything stop working.

I tried to reproduce this, but It seems to work also with "module": "CommonJS" and "moduleResolution": "classic".
Take a look here: https://stackblitz.com/edit/schoero-swissqrbill-bpener?file=src%2Fpdf.ts

According to https://www.typescriptlang.org/tsconfig/#moduleResolution - classic shouldn't be used anymore.
In my case, an Angular 18 project would not build with CommonJS/classic settings.
Saying this, I have a similar problem, just that another library does not support moduleResolution: Bundler (also not in connection with "type": "module" in the package.json).
But it's probably easier to find a workaround in my case.
Anyways, thank you very much for building and maintaining this library 🙏

hi I made very simple example here:
https://stackblitz.com/edit/nestjs-typescript-starter-mtaweg?file=src%2Fapp.controller.ts
As you can see we have problem with import the library:

It seems that there are indeed some things wrong with the package.json. There also seem to be some problems with the type exports.
As a temporary workaround, I have updated the main field in the package.json to reference the bundle in version 4.1.1-beta.0. You should now be able to import it in the beta version as follows:
import SwissQRBill from 'swissqrbill';

Resolving the underlying issues will require a more in-depth investigation. I should have time to look into it in the next 2–3 weeks.


Unreliable ClientRpc is not mapping to UnreliableRpc channel when running as a Host

Describe the bug
If we invoke a [ClientRpc(Delivery = RpcDelivery.Unreliable)], it should be sent over NetworkChannel.UnreliableRpc but it's being sent over NetworkChannel.ReliableRpc.
This bug happens only if we're running as a Host and trying to invoke an Unreliable ClientRpc.
Environment:
occurs on Windows, macOS on 2019.4 and 2020.3 under the latest release/0.1.0 branch (39604a3)

fixed by #638


Network.to_json should handle numpy.ndarray correctly

Pick the fix at https://github.com/keras-team/keras/pull/10754 to tf.keras.

Nagging Assignee @caisq: It has been 14 days with no activity and this issue has an assignee. Please update the label and/or status accordingly.

Can you review this please, @fchollet?

It looks the pylint check failed is not relevant to this PR, I pulled and rebased master code, wish it can pass test.

I fixed the bad indentation, can anyone trigger the build again? Thanks.

@yanboliang Done

@caisq I fixed the bad indentation again, can you re-trigger it? Thanks. BTW, do you know how I can trigger python lint checker locally before I submit the PR?

@yanboliang With the necessary dependencies installed, you should be able to run it with this script (from the root directory of the repo): https://github.com/tensorflow/tensorflow/blob/master/tensorflow/tools/ci_build/ci_sanity.sh

@caisq Great. I just run that script and verified this PR can pass sanity check locally. Can you trigger this test again? I'm sorry to bother you again. Thanks.

Nagging Assignee @caisq: It has been 14 days with no activity and this issue has an assignee. Please update the label and/or status accordingly.

Gently ping @fchollet @caisq . Would mind to have a look at this? Thanks.

@caisq Thanks!


Undesired edge splitting and re-ordering.

Describe the bug
Meshes exported to GLTF with Blender do not preserve index buffer ordering, and split some edges in undesirable ways.
To Reproduce

Open attached .blend file
Note that the plane model is fully triangulated, and all edges are connected.
Export model to .gltf
Import .gltf model
Imported model now has many split edges (try selecting and dragging several vertices until you find a disconnected edge).

Expected behavior
A pre-triangulated model, without UVs or discontinuous normals, should not change mesh connectivity on export/import to GLTF, and preferably should not change index buffer ordering either, or at least gate this behind an option.
Screenshots
Before export to .gltf:

After export + re-import:

.blend file/ .gltf
edge_split_repro.zip
Version

OS: Windows 11
Blender Version: Blender 3.4.1

Additional context
Note that this model is intentionally chopped into strange triangles as I wasn't sure of the topology that would cause this export issue. This is affecting me in a non-trivial model which I am unable to share here.

Update:
It appears this happens only when exporting with Normals included. I suspect blender is generating discontinuous normals, and the exporter is splitting on them. Not sure if this is desired behaviour, but I should be able to work around it for now by not exporting normals (my use case does not include or require them).

The reason the corners get split is the normals; there are tiny differences even when two corners "should" have the same normal.
>>> D.meshes['Plane'].calc_normals_split()
>>> set(tuple(l.normal) for l in D.meshes['Plane'].loops)
{(0.0, 0.0, 0.9999999403953552), (0.0, 0.0, 1.0)}

Note that the low digits vary even between platforms so you may get different results on your computer.
It's been said before that normals should be rounded off when exporting.
As for the order of indices, IIRC I never even thought about it so I'm not surprised it isn't preserved.

Thanks! I believe the index ordering is preserved when normals are disabled, and this suits my purposes fine.
It would be nice if it's possible to ensure that behaviour is preserved going forward, as I have some use cases that depend on it. It might also be useful to have a mode that merges split normals on export, but I don't personally require that.
Thanks for the help,
Nico.

Rounding at 6 digits seems to solve the issue. See #1851

The OBJ exporter uses four digits, see link above.

Ok, let's keep the exporter consistant, let's set it to 4 on glTF too

Perhaps this is something we could mention briefly in the manual as well? Some workflows will depend on being able to reliably export vertices in a particular order and without changing the count. For example, exporting to glTF and storing additional per-vertex animation data in an MDD file or texture. Disabling normals may fix the issue sometimes, but would something like Mesh → Split → Faces & Edges by Vertices be a more reliable workflow?

@donmccurdy good thought, in my case (for various reasons) I specifically need un-split triangles. So my ideal use would probably be the opposite, to merge all discontinuous normals (but don't merge vertices just by position, leave them split if they were in the input).

If you want the original vertex/poly indices from Blender, it will be way less fragile to store them on attributes. Then it doesn't matter what reordering or splitting occurs, you can always reconstruct original data afterward. This geo node setup would work (export with "Apply Modifiers" and "Attributes" on)

That's an interesting idea, but certainly additional overhead over my use case.
My (naive) expectation is that if I go from gltf -> blender -> gltf, I should end up with the same gltf (maybe there's some slight precision changes or something, but the ability to edit geometry without changing connectivity seems pretty important to me).
I have a non-traditional use case that I can't go in to too much detail on, but I imagine lots of game content has similar requirements where a simple modification is required to a model that's already in gltf format, and you don't want to invalidate any surrounding content.
To be clear I've got my use case fulfilled already just by disabling the normals, just want to give some context for my thinking on it.

My (naive) expectation is that if I go from gltf -> blender -> gltf, I should end up with the same gltf

That is certainly impossible in general, for many reasons.

That's fair. I should be more clear also, I don't expect it to be bit-identical, or have the same headers or anything, but I would hope to keep the same mesh structures and instancing as the source gltf.

Neither of those is preserved. For example, whether vertices were shared between prims or not has no result on the import, so it can't be preserved. For instances, Blender has no notion of multiple instances of a mesh with different morph target weights, so instances which might have different weights are split up. Materials are also split up depending on if they need to used on a mesh with vertex colors or not. Etc etc.

I see, thanks for clarifying. I guess in my limited case it works fine, but I understand how it can't be guaranteed in general.
Thanks again for all the help!
Nico.


feat(connector-polkadot): connector plugin should be able to deploy smart contracts in Polkadot

Describe the solution you'd like
The connector plugin should be able to deploy smart contracts on a predefined parachain, in the Polkadot network.
Describe alternatives you've considered
Alternatively, the connector plugin can send the necessary information to a predefined parachain on the Polkadot network, which then deploys smart contracts accordingly.
cc: @RafaelAPB

I will be working on this, Thank you!

I will be working on this, Thank you!

@AnmolBansalDEV Assigned it to you just now, cheers!
FYI: I'll also update the commit message in your PR to declare that it fixes this issue.


PS4-Sounds for Steam Deck AudioLoader

PS4-Sounds
Insert a link to the GitHub repository of the pack.
https://github.com/midnightgospel/DeckUI-Sounds/tree/main/PS4-Sounds
Checklist
Failure to complete this checklist or deleting boxes from the checklist will result in the closing of your pull request unless this is a pack update. Please write any comments regarding this checklist at the bottom of your pull request.
Check every box.

[X] I am the original author of this pack or have permission from the original author to make this pull request.
[X] All copyright of this pack's contents belong to the listed author or is cited in the description and repository linked above.
[X] This theme works properly on the latest versions of SteamOS for Steam Deck, decky-loader and SDH-AudioLoader.
[X] This pack is under 10MB in size and uses the least disk space possible.
[X] This pack is safe for work and does not contain any sexual, drug-related, or profane content.
[X] This pack does not contain loud, distorted, or otherwise bad faith sound files.
[X] This pack is only for either music or sounds and only adds or modifies the intended sound files.

Hopefully everything looks ok. Let me know if I missed something

You have left a .gitkeep file in the image directory, please delete that file from your fork of the repo


[Question] White space in query result area after collapsing sidebar

After installing the latest stable release (1.28 on Ubuntu 16.04) i'm still observing a blank space in the query result section after collapsing the sidebar.
In the pull request regarding the sidebar was a comment about the issue (see https://github.com/sqlectron/sqlectron-gui/pull/380#issuecomment-342307331).
Has a fix been included in the release 1.28?
Best regards
Meykel

I can't see that issue on 1.28. #395 should have fixed that already and it has been included in that version. Could you please provide a gif showing that issue?

Screen for this issue

Look like he is talking about the TRAILING SPACE in the result table after we collapse the sidebar. I can re-pro the issue as well, this should be a fairly straight forward bug to fix.

I also found a few UI defects when you resize the app window.
Look like the headers row is resized, but the result item rows does not get resized at all.


Update the Security Malware page

Update screenshots with the new banner style (the new version of the screenshots are already available in the dataset)
Add a word about the main message
Add instructions for authors

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint.


Test collection detail

This is dependent on this PR:
https://github.com/peaqe/orion-utils/pull/6

I pulled the code down and this is still a work in progress. I will review when ready!


amazonec2 driver using expired credentials on rm command

When using AWS STS, credentials expire 1 hour after the session begins. docker-machine rm tries to make AWS API calls with expired credentials. The command fails and give this error : Error removing machine test-eric-3: unable to terminate instance: unable to terminate instance: Problem with AWS API call: Non-200 API response: code=400 message=Request has expired. EC2 instance termination and private key removal must be done by hand.
I'm using docker-machine version 0.4.0 (9d0dc7a).
IMO, docker-machine rm should take parameters --amazonec2-access-key --amazonec2-secret-key and --amazonec2-session-token and recognize associated environment variables.

This is a great point.  Perhaps we should remove the need for credential storage and pass for each command.
/cc @nathanleclaire

Before rushing into anything like that, we should just design a "provider" model which actually works.  It will burn us in the end to hack provider credential flags into every single Machine subcommand instead of introducing an abstraction which handles that first-class.

That being said, sorry for the trouble @ericcitaire .  We're taking this into consideration for the design of Machine in the future.

I understand the need of an abstraction of credentials providers. Let me know if I can be of any help for this.
In the meantime, the rm subcommand is unusable "as is" in the case of AWS STS temporary sessions, which is a commonly used pattern in the AWS world.
Here is a small bash script which can be used as a workaround:
for cfg in ~/.docker/machine/machines/*/config.json ; do
  jq -c 'if .DriverName == "amazonec2" then . * { Driver : { AccessKey: "'$AWS_ACCESS_KEY_ID'", SecretKey: "'$AWS_SECRET_ACCESS_KEY'", SessionToken: "'$AWS_SESSION_TOKEN'" } } else . end' "$cfg" > "$cfg.new" \
    && mv "$cfg.new" "$cfg"
done

@ericcitaire I had a solution like yours working but since the v1.0 it seems that changing the config.json is not enough.
Are you experiencing the same issue?

@miguelpuyol v1.0 of what? AFAIK, latest version of docker-machine is v0.5.1.

@ericcitaire I got confused with the v 1.0 of swarm! My apologies!

Anyone who found out why the updates of the keys in the config file does NOT work and it still fails with the expired key error? Do we need to restart the docker service?

@AlexOugh Try Machine 0.5.2.  It removes RawDriver which may be the reason updating your credentials in the JSON isn't working.

upgraded from 0.5.0 to 0,5.2 and it works. Thanks!

@AlexOugh :+1:

I just ran into this problem today. Perhaps it would make sense to update the documentation for the amazonec2 driver to reflect the fact that session tokens are not fully supported currently since AWS API calls will fail once the cached token expires?

@imbriaco I'm not extremely familiar with session tokens and how they work normally, but I would be happy to merge such a pull request.

LR;DR - if docker pulls credentials from ~/.aws/credentials and it uses a session token, those credentials should not be stored in the machine config, and credentials should be re-fetched from ~/.aws/credentials every time a docker-machine command is run.
STS credentials are only good for one hour.  Presumably, people who use them (like me) have a method of refreshing the credentials on a regular basis.  AWS has a Python program in their blog that does exactly this when federation is used, such as SAML.  In the end, these credentials get stored in the ~/.aws/credentials file, either in the default profile or another named profile.  It appears that docker only pulls from the default profile, so it would be nice to be able to name a profile when creating a machine/engine.  In any case, these credentials expire often, and are often updated by users so they can run aws CLI commands, etc.  Docker should be re-retrieving these credentials from the ~/.aws/credentials file instead of storing them in a machine config file.  There is only only canonical location for AWS credentials, and that is the ~/.aws/credentials file.  Programs such as docker-machine should not be storing them elsewhere.  It is a security risk if nothing else, and as a side-effect causes people's automated refresh of these credentials to break.

I have the same problem with v. 0.7.0. The keys are definitely being stored somewhere besides .aws since after the STS keys expire, can't get to the instance.

The keys are definitely being stored somewhere besides .aws since after the STS keys expire, can't get to the instance.

Yes, in config.json.
I think the solution will be to ensure that credentials are read from ~/.aws/credentials any time a command is invoked, like @freimer suggested.

Boto3 has support for Assume Role Provider, which automatically handles the STS refresh process for you (similar to the AWS Console).
This is really the ideal solution - I use it with AWS CLI and other Boto3-based code and it works transparently to the user and application.
Not sure though is the AWS SDK for Go supports this capability.

The AWS ECS CLI tool is written in Go and it supports the Assume Role Provider functionality, so it seems it is possible using the Go SDK.


peer connection reset handling

closes #54

Other than the small comment nit it looks great! :)

@homu r+

:pushpin: Commit 1891120 has been approved by Hoverbear

:hourglass: Testing commit 1891120 with merge ccf30ca...

:sunny: Test successful - status


Disabled Front Door, is it charged?

Front Door has, and we can control via API / scripting the possibility to set it in Disabled state.
Would be convenient to have documented somewhere whether this will still occur in hours being charged for the service, or not.

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

ID: f7be424c-f540-a753-6ecc-6ff3913368d5
Version Independent ID: dc80b25f-374d-8775-16e7-ae7d57b4b94f
Content: Azure Front Door Service - Frequently Asked Questions
Content Source: articles/frontdoor/front-door-faq.md
Service: frontdoor
GitHub Login: @sharad4u
Microsoft Alias: sharadag

@biste5 Thanks for the feedback! I have assigned the issue to the content author to evaluate and update as appropriate.

#reassign: amitsriva

Any updates on this?

#reassign: gunjanj

Hello @biste, disabled Azure Front Door resources are not billed. Please see this new added section in the FAQ: https://docs.microsoft.com/en-us/azure/frontdoor/front-door-faq#will-i-be-billed-for-the-azure-front-door-resources-that-are-disabled-. Please let me know if you have any further questions. For now I will be closing this issue. Thank you. #please-close


Delete-many, many, many: more than 20

Offer a tool that uses the moreData to do that on the user's behalf (perhaps with a max count or something).
Probably there's no clean way to parallelize efficiently, but still.

solved by merging 1.0.0


feat: Parse go.mod

resolve: https://github.com/goreapers/gograveyard/issues/12
Added a function Parse to take a byte array from a go.mod file and parse it into a struct that contains slices of all the modules. The module is defined by the path, version and categorized if it is indirect or not.

@powersj Good find regarding replace, retract, and exclude. So I pushed a change to handle replace directives, let me know what you think! I am not sure what to do regarding retract and exclude, they seem to the guards to prevent people from using certain versions and packages? I don't think it changes the dependencies declared in the mod file?


[Feature Request]: Stooge sort

Is there an existing issue for this?

[X] I have searched the existing issues

Feature Description
Stooge Sort is a simple and recursive sorting algorithm. It is known for its high time complexity and is mostly used for educational purposes to understand recursive sorting techniques.
Use Case
Stooge Sort is a simple and recursive sorting algorithm. It is known for its high time complexity and is mostly used for educational purposes to understand recursive sorting techniques.
Benefits
No response
Add ScreenShots
No response
Priority
High
Record

[X] I have read the Contributing Guidelines
[X] I'm a GSSOC'24 contributor
[X] I have starred the repository

@Ajay-Dhangar
Is any points limitation in your repo?


Role Assignment on Storage Account File share

Bicep version
Bicep CLI version 0.4.1008 (223b8d227a)
Describe the bug
Creating a storage account file share and then setting role assignment on the file share (not the storage account) does not target correct "scope". Expected role assignment to scope to the file share
To Reproduce
Bicep file: fileshare-and-permissions.bicep
param storageAccount string
param fileshareName string
@allowed([
  'Cool'
  'Hot'
  'TransactionOptimized'
  'Premium'
])
param storageTier string
param quotaInGB int
param permissionRoleGuid string
param groupsOid array

resource stg 'Microsoft.Storage/storageAccounts@2021-08-01' existing = {
  name: storageAccount
}

resource fileshare 'Microsoft.Storage/storageAccounts/fileServices/shares@2021-08-01' = {
  name: '${stg.name}/default/${fileshareName}'
  properties: {
    accessTier: storageTier
    enabledProtocols:'SMB'
    shareQuota: quotaInGB
  }
}

resource roleDef 'Microsoft.Authorization/roleDefinitions@2018-01-01-preview' existing = {
  scope: subscription()
  name: permissionRoleGuid
}

resource perms 'Microsoft.Authorization/roleAssignments@2020-08-01-preview' = [for group in groupsOid: {
  name: guid(subscription().id,resourceGroup().id, stg.name, fileshareName, group)
  scope: fileshare
  properties:{
    roleDefinitionId: roleDef.id
    principalId: group
  }
}]

$Props = @{
    Name               = 'File-Share'
    ResourceGroupName  = "my-rg"
    TemplateFile       = ".\fileshare-and-permissions.bicep"
    storageAccount     = "mystorageacc"
    fileshareName      = "test"
    storageTier        = "Hot"
    quotaInGB          = 5
    permissionRoleGuid = "0c867c2a-1d8c-454a-a3db-ab2ea1bdc8bb" # SMB Contributor
    groupsOid          = ("48477e2b-7ae7-46da-97c0-6690a03c2ce4")
}
New-AzResourceGroupDeployment @Props

Verifying the role assignment in the portal does not show the just added role/assignment.
Running pwsh:
Get-AzRoleAssignment | select displayname, scope | sort scope

Gives this scope:

Where the green /fileshares/fxprofiles (created by another ARM template) works as expected.
And the red /shares/test are created by the above bicep template and does not work.

If anyone here who has encountered the issue can open a support ticket to have this routed to the Storage Resource Provider team, that would be helpful.

I just filed a support request as this also was raised in CARML and would, if not handled end up in AVM. @alex-frankel I'll implement the workaround proposed by @brwilkinson for now, but sincerely hope this is fixed by the Storage PG. Inventing an artificial resource type 'fileShares' appears to be a major oversight on their part 🔧

I ran into this issue as well and see there was a PR by @AlexanderSehr 🙏  Has it made its way into Bicep or do I have something misunderstood (quite likely 😁)?
FWIW I did try the workaround but am not having any luck with it.  It seems to execute but the Role Assignment is not listed in the share's assignments.  In my case I am wanting to add a Managed Identity, and I noticed the workaround is for a group.  I am curious if this is why this might not be working. 🤔

I ran into this issue as well and see there was a PR by @AlexanderSehr 🙏 Has it made its way into Bicep or do I have something misunderstood (quite likely 😁)?
FWIW I did try the workaround but am not having any luck with it. It seems to execute but the Role Assignment is not listed in the share's assignments. In my case I am wanting to add a Managed Identity, and I noticed the workaround is for a group. I am curious if this is why this might not be working. 🤔

You wish :D No, it was not yet fixed by the Provider and my support ticket is also still open. However - I did implement the workaround both in the CARML, as well as the (to be) Azure-Verified-Modules module if you want to have a reference and hope to get rid of the workaround as soon as I hear back from the Product Group.

Thank you for the clarification @AlexanderSehr.  FWIW I was able to get the workaround working after all. 😌  I had the wrong path/share used (lots of moving pieces over here), so I am set in the meantime.
Also, I did want to mention that in Visual Studio Code, using the above workaround does result in the following warning:

Interestingly enough the warning's URL resolves to here:
https://www.bing.com/?ref=aka&shorturl=bicep/linter/no-deployments-resources
Not sure if this is a bug, but thought I would mention it.  Please note that I am already in a module and wonder if this is contributing to this.

Thank you for the clarification @AlexanderSehr. FWIW I was able to get the workaround working after all. 😌 I had the wrong path/share used (lots of moving pieces over here), so I am set in the meantime.
Also, I did want to mention that in Visual Studio Code, using the above workaround does result in the following warning:

Interestingly enough the warning's URL resolves to here: https://www.bing.com/?ref=aka&shorturl=bicep/linter/no-deployments-resources
Not sure if this is a bug, but thought I would mention it. Please note that I am already in a module and wonder if this is contributing to this.

Nah, it's by design. Microsoft.Resource/deployments is now 'hidden' behind the module myModule 'localPath/main.bicep' = { (...) } syntax. So if you use local references you'll see they in the rendered ARM template they're all just deployment objects that correspond in turn to deployment objects in the Azure Portal. However, in this case you're ferefencing a nested linked ARM template so you don't really get around it. You can supress it though.

There might be a possible relation to another issue I am facing currently:
I want a cluster to have access to a storage account file share. When I assign the role "Storage File Data Privileged Contributor" it gets assigned the permissions
"Microsoft.Storage/storageAccounts/fileServices/fileshares/files/read",
"Microsoft.Storage/storageAccounts/fileServices/fileshares/files/write",
"Microsoft.Storage/storageAccounts/fileServices/fileshares/files/delete",

however, on access it throws an error that the object does not have the permission
'Microsoft.Storage/storageAccounts/fileServices/shares/read'
So it seems the scope is bugged/named wrong. Seems like such an easy fix, or do I misunderstand that?

The link https://aka.ms/bicep/linter/no-deployments-resources has been fixed to correctly redirect to Microsoft Docs.
The linter rule is valid here - instead of:
resource ResourceRoleAssignmentUsers 'Microsoft.Resources/deployments@2021-04-01' = {
  name: 'user-permissions_${now}'
  properties: {
    mode: 'Incremental'
    expressionEvaluationOptions: {
      scope: 'Outer'
    }
    template: json(loadTextContent('./genericRoleAssignment.json'))
    parameters: {
      scope: {
        value: replace(fileshare.id, '/shares/','/fileshares/')
      }
      name: {
        value: guid('${subscription().id},${resourceGroup().id},${stg.name},${fileshare.name},-users')
      }
      roleDefinitionId: {
        value: '0c867c2a-1d8c-454a-a3db-ab2ea1bdc8bb' // Storage File Data SMB Share Contributor
      }
      principalId: {
        value: genSettings.AADGroup
      }
      principalType: {
        value: 'Group'
      }
    }
  }
}

The recommendation is to use:
module ResourceRoleAssignmentUsers 'genericRoleAssignment.json' = {
  name: 'user-permissions_${now}'
  params: {
    scope: replace(fileshare.id, '/shares/','/fileshares/')
    name: guid('${subscription().id},${resourceGroup().id},${stg.name},${fileshare.name},-users')
    roleDefinitionId: '0c867c2a-1d8c-454a-a3db-ab2ea1bdc8bb' // Storage File Data SMB Share Contributor
    principalId: genSettings.AADGroup
    principalType: 'Group'
  }
}

Hello from June 2024, this is still an issue. and I spent an entire day of troubleshooting before I noticed /shares/ <> /fileshares/.  The work around with calling the arm from inside bicep does the trick though.

Hello from June 2024, this is still an issue. and I spent an entire day of troubleshooting before I noticed /shares/ <> /fileshares/. The work around with calling the arm from inside bicep does the trick though.

Same with me in October 2024.

Hitting this December 2024
https://github.com/Azure/azure-rest-api-specs/issues/24568

Two and a half years later and this simple bug has still not been fixed. Incredible, especially considering we pay for this resource type.

If only maximizing shareholder value meant improving the quality of code @mrkesu 🙄


Text alignment missing

No text alignment is found in the document. How do you correspond to the text layer?

There is no official specification on the text formatting structure.
You can inspect the content of engine data if there exists any relevant field

OK, there is no corresponding alignment in the document. I need such a style, such as align: Center. Is there a solution?
In addition, do you have corresponding parameters for the writing mode and direction of the text? See the picture for details
@kyamagu

I do not know the internal specification of engine_dict, and cannot answer. You can always print or pprint low level data structure. Also see TypeLayer doc and its implementation

After repeated inquiry, I finally found the answer in the "justification" parameter. Please leave the answer to others here!!

After repeated inquiry, I finally found the answer in the "justification" parameter. Please leave the answer to others here!!
The corresponding values are:
#layer.engine_dict['ParagraphRun']['RunArray'][0]['ParagraphSheet']['Properties']['Justification']
def getAlign(self,num):
    enum = [(0,'left'),(2,'center'),(1,'right'),(3,'start'),(5,'center'),(4,'end'),(6,'justify')]
    for item in enum:
      n,text = item
      if num == n:
        return text
    return 'left';

Sounds good


Meta Live Caption Tool V2 Checklist

Related to #48 #50.

[ ] Live Caption Tool Additions #48 (make changes to live captions to the app itself so users don't have to fiddle with custom CSS, include one "blank" CSS in the dropdown so advanced users can set up multiple scenes or allow multiple 3rd party computers to connect for various programs)
[ ] Fixed box size #47 (add option to fit gurbani/english or whatever visible fields are there in one line by appropriately resizing text, this is still too advanced for most users, but a few have requested it, so leave it off by default for sure or perhaps add a new "min font size" slider which doesn't allow it to go smaller than that even if it bleeds to two lines)
[ ] add overlay css file selector #197 (drop down that is populated by css files in user directory)
[ ] Add time & date overlays #200 (as an endpoint that can be manipulated by url, e.g. localhost:1699/timedaydate?format="HH:mm ddd DD M YY") must have the "esc" screen true/false in it as well
[ ] add url address to overlay #201 (something that can be easily copy and pasted across the network)
[ ] Live Captions Customization #353


Improve simulated bold fonts advance.

An addition, for #59013, increases glyph advance for emboldened fonts.

Thanks!


Adding and debugging format analyzer

Closes #9
Added an analyzer that detects improperly formatted log lines and reports them. Also added a report messages to this and other analyzers I wrote because I realized I was just dumping the lines in the report with no explanation.

I can't resolve the merge conflicts in this PR through the web UI because Github is confusing the header lines with ======... for merge conflict lines. @kitrady you'll have to merge this with main and resolve the conflicts yourself before I can merge it


syscalls: Remove static muts

Rebased on https://github.com/hermitcore/libhermit-rs/pull/598, @stlankes.


AddRedisQueryStorage Throws & Inconsistent API

Describe the bug
If I remove x.GetApplicationServices() below, Hot Chocolate is unable to resolve IConnectionMultiplexer and throws an exception.
services.AddSingleton<IConnectionMultiplexer>(ConnectionMultiplexer.Connect("localhost:6379"));

builder.AddRedisQueryStorage(x => x.GetApplicationServices().GetRequiredService<IConnectionMultiplexer>().GetDatabase());

Additionally, there is some inconsistency in the way that we resolve a IConnectionMultiplexer in AddRedisQueryStorage and AddRedisSubscriptions. In one call we have to get a database, while in the other we don't.
builder.AddRedisSubscriptions(x => x.GetRequiredService<IConnectionMultiplexer>());

Expected behavior
The code below should not throw and should be able to resolve IConnectionMultiplexer. Additionally, I suggest removing the additional need to call .GetDatabase() in AddRedisQueryStorage.
services.AddSingleton<IConnectionMultiplexer>(ConnectionMultiplexer.Connect("localhost:6379"));

builder.AddRedisQueryStorage(x => x.GetRequiredService<IConnectionMultiplexer>());

This one is now implemented. We now allow the following:
Variant 1:
services.AddSingleton<IConnectionMultiplexer>(ConnectionMultiplexer.Connect("localhost:6379"));

builder.AddRedisQueryStorage(x => x.GetRequiredService<IConnectionMultiplexer>().GetDatabase());

Variant 2:
services.AddSingleton<IConnectionMultiplexer>(ConnectionMultiplexer.Connect("localhost:6379"));

builder.AddRedisQueryStorage(x => x.GetRequiredService<IConnectionMultiplexer>());

Variant 3:
services.AddSingleton<IConnectionMultiplexer>(ConnectionMultiplexer.Connect("localhost:6379"));

// this will get the first registered multiplexer.
builder.AddRedisQueryStorage();


Missing forward slashes in file path breadcrumb affecting behavior of breadcrumbs text and icon rendering

Version: 1.40.2
Commit: f359dd69833dd8800b54d458f6d37ab7c78df520
Date: 2019-11-25T14:54:40.719Z
Electron: 6.1.5
Chrome: 76.0.3809.146
Node.js: 12.4.0
V8: 7.6.303.31-electron.0
OS: Linux x64 4.15.0-72-generic snap (Ubuntu 16.04.6 LTS)
Steps to Reproduce:

Open a Folder
Observe the breadcrumb at the top.

Navigate to a very nested location.
Click on any of the folders in the breadcrumb to go up a few levels in the directory path.
Observe how it is broken.

)

Does this issue occur when all extensions are disabled?: Yes

@tommai78101 The screenshots you've included are the file picker from Ubuntu, which we don't control


with-firebase-hosting example serve not working

Serving with-firebase-hosting example not working
Example name
Serving with-firebase-hosting example not working
Describe the bug
Running npm run dev works, but when running npm run serve I am getting:
⚠  functions: Cannot start emulator. Error: Cannot find module '@google-cloud/functions-emulator/src/config'
(node:3376) UnhandledPromiseRejectionWarning: TypeError: Cannot read property 'exit' of undefined
    at /Users/ariel/git/with-firebase-hosting-app/node_modules/firebase-tools/lib/command.js:82:34
    at <anonymous>
    at process._tickDomainCallback (internal/process/next_tick.js:229:7)
(node:3376) UnhandledPromiseRejectionWarning: Unhandled promise rejection. This error originated either by throwing inside of an async function without a catch block, or by rejecting a promise which was not handled with .catch(). (rejection id: 2)
(node:3376) [DEP0018] DeprecationWarning: Unhandled promise rejections are deprecated. In the future, promise rejections that are not handled will terminate the Node.js process with a non-zero exit code.
i  hosting: Serving hosting files from: dist/public
✔  hosting: Local server: http://localhost:5000

Browsing localhost:5000 shows:
An internal error occurred while connecting to Cloud Function "next"
To Reproduce

npx create-next-app --example with-firebase-hosting with-firebase-hosting-app
cd with-firebase-hosting-app
Add proper Firebase project to .firebaserc file
npm run serve

Expected behavior
Example to serve.
Screenshots

System information

OS: macOS 10.12.6
Browser (if applies) Chrome
Version of Next.js: 7.0.2

Thanks @oBusk. Unfortunately, running npm i -g @google @google-cloud/functions-emulator showed an error:
npm ERR! code EINVALIDTAGNAME
npm ERR! Invalid tag name "@google": Tags may not have any characters that encodeURIComponent encodes.

And removing the @google part installed functions-emulator but didn't solve the issue. :(

Thanks @oBusk. Unfortunately, running npm i -g @google @google-cloud/functions-emulator showed an error:
npm ERR! code EINVALIDTAGNAME
npm ERR! Invalid tag name "@google": Tags may not have any characters that encodeURIComponent encodes.

And removing the @google part installed functions-emulator but didn't solve the issue. :(

Yea oops i had a typo there with @google twice.
I can't really reproduce this issue. What version of Node and NPM do you have? And did you get the exact same issue as the one you pictured, after installing funtions-emulator globally?

same issue here..any solution

Hi, this may be a Next configuration issue. I needed to change the following line in the ./src/index.js function file to get the default with-firebase-hosting example to work with the Firebase npm run serve command:
var app = next({ dev, conf: { distDir: '../../dist/functions/next' } })

Previous versions of the with-firebase-hosting example set this distDir variable in the ./src/app/next.config.js file, but this doesn't work for the current version.

@oBusk node v8.12.0, npm 6.4.1. And yes, emulator is installed globally.

@moflo There is no ./src/index.js. Did you mean ./src/functions/index.js? I updated that one but same error. Were you able to make it run? What node version are you running?

Ok here's how I got this running:

Uninstalled npm, nvm, node and yarn from my mac.
Reinstalled node via nvm.
Installed functions emulator and firebase tools globally
Updated this example's ./src/functions/index.js to look like this (cause it was not finding the build directory):

const path = require('path')
const functions = require('firebase-functions')
const next = require('next')

var dev = process.env.NODE_ENV !== 'production'
var app = next({ dev, conf: { distDir: `${path.relative(process.cwd(), __dirname)}/next` } })
var handle = app.getRequestHandler()

exports.next = functions.https.onRequest((req, res) => {
  console.log('File: ' + req.originalUrl) // log the page.js file that is being requested
  return app.prepare().then(() => handle(req, res))
})


Add the option to set client.id to storm-kafka, STORM-2524

Add the ability to set the Kafka client.id property to storm-kafka and storm-kafka-client (https://issues.apache.org/jira/browse/STORM-2524).
storm-kafka example:
	SpoutConfig spoutConfig = new SpoutConfig(
		brokers,
		topicConfs.topic,
+		"client_id-" + confs.consumerGroupId + "-" + topicConfs.topic,
		"/consumers",
		confs.consumerGroupId + "-" + topicConfs.topic
	);

storm-kafka-client example:
	KafkaSpoutConfig<String,String> kafkaSpoutConfig = KafkaSpoutConfig.builder(KAFKA_LOCAL_BROKER, TOPIC)
		.setGroupId(confs.consumerGroupId + "-" + topicConfs.topic)
+		.setClientId("client_id-" + confs.consumerGroupId + "-" + topicConfs.topic)
		.build();

See https://github.com/apache/storm/pull/2126 for review details, re-creating second time (first attempt was https://github.com/apache/storm/pull/2132) to resolve CHANGELOG.md conflicts.

@revans2, please let me know if I need to make any more adjustments for this to be merged in.

See original PR, https://github.com/apache/storm/pull/2126


Fix speaker information

Pull Request Test Coverage Report for Build 195

0 of 4 (0.0%)  changed or added relevant lines in 2 files are covered.
1 unchanged line in 1 file lost coverage.
Overall coverage decreased (-0.0003%) to 0.093%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

lib/page/session_detail.dart
0
2
0.0%

lib/page/speaker_detail.dart
0
2
0.0%

Files with Coverage Reduction
New Missed Lines
%

lib/page/speaker_detail.dart
1
0.0%

Totals

Change from base Build 193:
-0.0003%

Covered Lines:
1

Relevant Lines:
1078

💛  - Coveralls


Not emitting JS bundle - Webpack

Note: for support questions, please use one of these channels:
https://forum.ionicframework.com/
http://ionicworldwide.herokuapp.com/
Short description of the problem:
ionic-app-scripts build
Latest versions are not emitting JS bundle (no errors).
Debug shows the following warning
Error loading core compiler: xxx\node_modules\ionic-angular\compiler, Error: Cannot find module xxx\node_modules\ionic-angular\compiler'
Ionic angular is version 2.0.0.
Which @ionic/app-scripts version are you using?
Bug introduced in 1.2.0 and is present in 1.3.6.
Bug seems to be in src/webpack.ts:
const bundledFilesToWrite = context.fileCache.getAll().filter(file => {
    return dirname(file.path) === context.buildDir && (file.path.endsWith('.js') || file.path.endsWith('.js.map'));
});

dirname(file.path) = xxx\www\js
context.buildDir =xxx\www
Thus files are not emitted.

I cannot recreate this.
Can you please create a repository on github where I can recreate it?
Thanks,
Dan

Issue has been fixed in https://github.com/driftyco/ionic-app-scripts/pull/938.
Can you please release a package including the merged commits?

@michaelgeorgeattard,
Yes, I will do it today/tonight.
Thanks,
Dan

This is fxied in 1.3.7.
Thanks,
Dan


feat(v2): highlight items in the table of content

Motivation
Implement the highlight functionality for the table of content links on the scroll and on click events.
The implementation is partially based on the solution implemented in v1 but it is drastically changed to make better use of React.
Resolve issue: #1752
Also relates to: #1524
Have you read the Contributing Guidelines on pull requests?
Yes
Test Plan
When clicking on any of the links of a table of content or while you scroll on a section that contains a table of content you are going to notice that the table of content has the link of the section you have on the screen highlighted.

Tested on Safari and Chrome.
Related PRs
(If this PR adds or changes functionality, please take some time to update the docs at https://github.com/facebook/docusaurus, and link to your PR here.)

Strange it works fine for me in Chrome (I'm on Linux), but not in Firefox :man_shrugging:

There are many misbehaviors from preview:

does not pick up the correct color when navigating by url
clicking on level 2 items sometimes does not pick up the correct color
never highlights the first or the last item

The behaviors said above appears in both Firefox and Chrome, the screen record I have here shows Chrome:

I think the implementation in #1524 is correct and the comments around it serves as good pseudocode. Suggest that we follow it directly.

If i go to https://deploy-preview-1896--docusaurus-2.netlify.com/docs/introduction/#gitbook
The Gitbook is not highlighted, i have to scroll around or resize first.
Compare it to https://docusaurus.io/docs/en/installation#running-the-example-website
Update: Clicking the right sidebar directly also doesnt highlight that particular section

Both things work fine for me (the gitbook highlight works fine for me, also clicking the sidebar update directly) I am going to play around until I can reproduce it.

Strange it works fine for me in Chrome (I'm on Linux), but not in Firefox 🤷♂

OK, I am going to test in firefox and see what's going on.

There are many misbehaviors from preview:

does not pick up the correct color when navigating by url
clicking on level 2 items sometimes does not pick up the correct color
never highlights the first or the last item

The behaviors said above appears in both Firefox and Chrome, the screen record I have here shows Chrome:

I think the implementation in #1524 is correct and the comments around it serves as good pseudocode. Suggest that we follow it directly.

If you want me too I can see the issues that are happening (some of them also happen in v1).
About the comments, I am aligned with the Clean Code points of view, Comments are good if you need to explain something that can't be explained by writing descriptive code. In any other situation, you are probably just being redundant. But I can add comments around if that what you need.
Please tell me if you want me to fix the issues describe in the comments or you prefer to delete this PR. I did put time on this but I completely understand if you prefer to go another way.
Thanks

Both things work fine for me (the gitbook highlight works fine, also clicking the sidebar update directly) I am going to play around until I can reproduce it.

Here's the demo :)

I think the issues need to be fixed before merging, the behavior doesn't feel correct to me atm.

I wont merge until both of u approve :) Just putting stamp 😃

Great! I am going to work on the fixes. Thanks

@yangshun I am almost done working on the issues mentioned and the comments you added. I only need an answer to reply to one of your comments related to the new hook file location.
Thanks

Changes (commit 5a2f5390202c02ed15ed8e0ab6e8cdcde1ff8ed9):

Create a custom hook
Address all comments made by @yangshun having into consideration the conversations where also @endiliey give input and feedback
Improve the behavior and address some of the issues shared by @wgao19

Two known issues that I think can be addressed later:

If you scroll up and down fast the wrong item in the table of content might be highlighted
If you are scrolling up to a section header with too much content, you might see the wrong section highlighted until the header of the section which content is in the screen enters the area that activates the table of content items. [let me know if I didn't explain the problem clearly]

Thanks @lex111, personally I agree on most of the changes you propose. I am going to work on them or respond with a comment in the case I think we should discuss something further.

I understand correctly that we will implement all my suggestions, except the last one, about the concatenating of classes?


fix(tags): Exclude parenthesis from tags

Fixes #2148
Adds () characters to the PUNCTUATION_MARKS character list. Added it closer to start of string because I suspect they are more common than the other ones that were already there.
Adds assertion in existing and related unit test case.
Kudos to @SeriousBug for pointers to relevant code so I didn't have to waste time on finding those. 🙇

If this is your first time submitting a pull request to Dendron, copy and paste the full Dendron Review Checklist into this request and check off each item as necessary.
First time pull request

[x] if its your first pull request to Dendron, watch out for the CLA bot that will ask you to agree to Dendron's CLA
[x] if its your first pull request and you're on our Discord, make sure that Kevin gives you the [[horticulturalist|community.roles#horticulturalist]] role by adding your discord as part of the description 👨🌾👩🌾
[x] (optional) ping @Dendron Team in the #dev channel of our discord - we usually respond to PRs within 24h

History

[x] make sure the commit message follows Dendron's [[commit style|dendron://dendron.docs/dev.process.review#commit-style]]
[x] make sure the commit message is in front of the [[pull request|dendron://dendron.docs/dev.process.review#pull-request]]
[x] make sure your branch names adhere to our [[branch style|dendron://dendron.docs/dev.process.review#branch-style]]
[x] if this pull request is addressing an existing issue, make sure to link this PR to the issue that it is resolving.

Docs

[ ] if your change reflects documentation changes, also submit a PR to dendron-site and mention the doc PR link in your current PR

Testing

[x] Write Tests
[x] Confirm existing tests pass
[ ] Confirm manual testing

I tried to test manually following guideline documented at https://docs.dendron.so/notes/dtMsF12SF2SUhLN10sYe2/ but unfortunately I got some errors on some of the steps, e.g. cd $DENDRON_REPO_ROOT/test-workspace && ./scripts/dev.sh yields env: dendron: No such file or directory

I am using volta to manage node/npm/yarn binaries, which I figure is a possible culprit. The binary can be found in other directories outside of test-workspace.
Deleting package.json + yarn.lock fixes this and next script to run, but the test workspace didn't appear to have my fix nevertheless.
Not sure what else might be wrong. I would be happy to jump on a pair call to figure out what might be wrong, if someone on the Dendron team have bandwidth and that is of interest?

[ ] If your tests changes an existing snapshot, make sure that snapshots are updated
[ ] If you are adding a new language feature (graphically visible in VS Code/preview/publishing), make sure that it is included in the test workspace

Analytics

[ ] if you are adding analytics related changes, make sure the Telemetry docs are updated

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


translate refs-must-have-owner warning

@miku86 Is there anything I'm missing here that still needs my review?


Added citation to Mississippi paper

Coverage decreased (-0.09%) to 82.881% when pulling 8b3716a3f9dad0e6c00be92707693d1ee13261f5 on aat669:docs into acd99e78ce97eb07a8667dc55bd974a0a45293e4 on erdc-cm:master.

Coverage remained the same at 82.969% when pulling 8b3716a3f9dad0e6c00be92707693d1ee13261f5 on aat669:docs into acd99e78ce97eb07a8667dc55bd974a0a45293e4 on erdc-cm:master.


v20.2.5 release notes

Fixes https://github.com/cockroachdb/docs/issues/8928.

This change is

Online preview: http://cockroach-docs-review.s3-website-us-east-1.amazonaws.com/cff7c62b8f9142d20def92d5a5e302083fde42ec/
Edited pages:

releases/v20.2.5.md

Online preview: http://cockroach-docs-review.s3-website-us-east-1.amazonaws.com/38378a79095f0f991c277e0aa81a68f4a599f8dc/
Edited pages:

releases/v20.2.5.md

Online preview: http://cockroach-docs-review.s3-website-us-east-1.amazonaws.com/9a4dc05ce7b2fd53389156a7fe688a9a2debdf72/
Edited pages:

releases/v20.2.5.md

Online preview: http://cockroach-docs-review.s3-website-us-east-1.amazonaws.com/2bbbbee2be8b2ab26ea589be8a2d5b41584185eb/
Edited pages:

releases/v20.2.5.md


Pass Node instead of node ID into controller node commands

In each of these commands, the node ID referenced will be from a node already included in the model, so why not let the called pass the node in instead of the node ID?

Are these the only cases where we can pass model instances instead of model ids? I think we should make sure we're consistent across the library before changing this.

good question. I found some others via searching for id in function params. There are two left:

controller.async_unprovision_smart_start_node - This one accepts a DSK or node ID, but because these entries are created before a node is included, I think it makes sense to still accept a node ID because the node may not be an existing node.
https://github.com/home-assistant-libs/zwave-js-server-python/blob/d2566fc5080e9034a57668decc132cc2d375929d/zwave_js_server/model/controller/__init__.py#L267
Node.value_data_idx - this is an internal command used to search for a value dict in the nodes state data. I think it makes sense to leave value_id as an input here too.
https://github.com/home-assistant-libs/zwave-js-server-python/blob/d2566fc5080e9034a57668decc132cc2d375929d/zwave_js_server/model/node/__init__.py#L724

Those are the only two places remaining I could find that accept a potential model ID as an input rather than the model instance, but I think in both of those cases we can't guarantee there is a model instance


Definition missing for gsin terms

Definition missing for the following gsin terms
modified
startValidDate
endValidDate
language
citation
memo
publicationDate
From http://www.gisin.org/cwis438/websites/GISINDirectory/Tech/ProtocolSpecification.php#<IP_ADDRESS> we can get a first definition and improve it

Ass GISIN Protocol repository is not longer maintained, these GISIN terms should be defined within Plinian Core. Souce for those term definitions can be found at https://github.com/tdwg/gisin/blob/master/index.html


switchLocalePath() using differentDomains does not include router.base into its path

Version
@nuxtjs/i18n: 7.2.0
nuxt: 2.15.8
Nuxt configuration

[ ] Applies to a site deployed to a static server (site generated with nuxt generate)
[x] Applies to a site deployed to a server with a Node backend

@nuxtjs/i18n configuration
  i18n: {
    locales: [
      { name: 'English', code: 'en', iso: 'en-CA', domain: 'hubretenglish.ca' },
      { name: 'Français', code: 'fr', iso: 'fr-CA', domain: 'hubretfrench.ca' }
    ],
    differentDomains: (process.env.NODE_ENV === 'production'),
    defaultLocale: 'en',
    strategy: 'prefix_except_default',
    vueI18nLoader: true
  },
  router: {
    base: '/conference/'
  }

Reproduction Link
https://codesandbox.io/s/nuxt-nuxtjs-i18n-base-template-forked-eg6hl7?file=/nuxt.config.js
The button going to hubretfrench.com should go to hubretfrench.com/conference
Steps to reproduce
Use the configuration above.
What is Expected?
The language toggle button (which in this case is now an a tag, since we are using differentDomains) should go to the page the user is on in the corresponding locale (the lang toggle at https://hubretenglish.ca/conference should link to https://hubretfrench.ca/conference)
What is actually happening?
The lang toggle at https://hubretenglish.ca/conference is linking to https://hubretfrench.ca
This might be happening on differentDomains : false as well, but I haven't checked thoroughly.
For now, the workaround is to write a funny .replace() on the switchLocalePath() to insert the base between the domain name and the current route.
Not sure if this behaviour is actually intentional, since there isn't a way to translate router.base (that I know of).

Are you sure that differentDomains option is set to true during nuxt generate?
BTW. If you want more help, I'd need a project that I can just clone and run. I can configure a fake domains locally using hosts file so that's not a problem but I won't have time to recreate the whole project.

the project is not being deployed with nuxt generate. I am certain that the option is being set since my example is running on a production site, with both domains (i.e. differentDomains is working).
I have a suspicion that this is also happening just with router.base set and i18n without differentDomains.
I'll try to get a reproduction for that

Closing this as v7 is not being actively worked on (hotfixes only), if this still applies to v8 and later please open a new issue!


Fix multiple definitions of g_client error

The global variable g_client was defined in both erpc_client_setup.cpp
and erpc_arbitrated_client_setup.cpp. As a result, linker errors were
produced when trying to build eRPC with both files. This change defines
the g_client in a single new place (erpc_setup_globals.cpp).
Resolves: #113

Hi, i think @MichalPrincNXP was fixing something with g_client so i will not answer to this PR if it is correct or not. I let it on Michal. But at least the name of the file should contains _client in name as the only variable is for client and server don't need to include that file.

Also what about usage of pragma weak? We are using that for const numbers in shim code:
#pragma weak erpc_generated_crc
extern const uint32_t erpc_generated_crc = {$crc16};

Hi @Hadatko,
Thanks for taking a look at this. I think I saw an issue where @MichalPrincNXP said he was going to address it, but it's still not fixed, so I thought that maybe he forgot or had other things to do.  I have added two additional commits.  The first one (5a96174) renames the file that contains the global variable to include client in the name.  The second commit undoes the approach implemented in the first two commits and instead uses #pragma weak.  This approach works as well.

Hello @dpfrey , thanks, now I understand the issue that happens when both erpc_client_setup.cpp and erpc_arbitrated_client_setup.cpp are being built in one project/make. Internally, we generate individual projects for different IDEs and because it does not make sense to have erpc_client_setup.cpp and erpc_arbitrated_client_setup.cpp in one project this issue has not be observed. Anyway, the solution by you and Dusan seems to be ok and working. May I ask you to rebase to the develop branch head to be able to trigger Travis build (recent commit fixes Travis builds)? Thank you.

I rebased the change onto develop and squashed away the initial approach so that only the #pragma weak change is now present.

Hello @dpfrey , it seems Clang C++ on Linux and both gcc and clang compilers on Mac are not ok with the way g_client is defined, see Travis build results
May I ask you to try to add extern keyword before the g_client declaration?

I moved the #pragma weak g_client after the variable declaration and that seemed to resolve the Travis build issues.

Thank you David, I will merge this PR after the final v1.8.1 release next week, ok?

Thank you @dpfrey !


Module build failed

Hello
I started getting this error after the most recent update to Chart.js
Failed to compile.

./node_modules/chart.js/src/chart.js
Module build failed: Error: ENOENT: no such file or directory, open '/var/www/html/project_name/node_modules/chart.js/src/chart.js'

I am importing like this (just like before the update when i had no errors):
import Chart from 'chart.js'
Inside the node_modules/chart.js folder there is indeed no /src/ folder
Thanks in advance

Use node_modules/chart.js/dist/Chart.js instead


Build out "Now what?" page

As a tenant who has successfully gathered signatures, or thinking about starting a campaign, I go to the "tools" tab on the top right of the webpage and click on the "Next Steps" link.

Tips for approaching your landlord Tab

Make an appointment with your property manager or landlord to talk about your campaign
Print out the signatures and find neighbors who are interested in going to the meeting with you
Listen to your property manager's feedback or concerns
Let us know what they said!

Our Landlord ignored or denied our request Tab
Boo! Why? FIll out this survey and tell us what the roadblockers towards getting recycling services were.

Survey will be built out in Google Sheets and added to the bottom of the website . See https://docs.google.com/spreadsheets/d/1LjlhDV17ciP0v_vn3Q5K4CJRm6ENgcZMrv5XTIXzLwY/edit#gid=0
Top 5 excuses from property managers

People won't use it right
It's too expensive
There's no room in the enclosure
There's no room on the property for a recycling bin
Contractual issues

Other? Let us know!