Unix timestamps .fromNow()

Hello,
I have a problem with parsing Unix timestamps from the Hacker News API.
When I do moment(unix).fromNow() where unix is<PHONE_NUMBER> is will return 47 years ago with every value.
I'm probably doing something wrong, but can;t find the problem.
Thanks in advance.
Theo

Moment parses integers as the number of milliseconds since epoch for consistency with the native Date object in Javascript.  Unix timestamps are the number of seconds since epoch.  Thus, your sample code should be moment(unix * 1000).fromNow()

Alternatively, you can use the unix function:
moment.unix(unix).fromNow()

Both moment(timestamp, 'X') and moment.unix(timestamp) return the warning:
Deprecation warning: value provided is not in a recognized RFC2822 or ISO format. moment construction falls back to js Date(), which is not reliable across all browsers and versions. Non RFC2822/ISO date formats are discouraged and will be removed in an upcoming major release. Please refer to http://momentjs.com/guides/#/warnings/js-date/ for more info.

Is this intended? How to get rid of it?

moment(1551881459259 * 1000).fromNow();

moment(1551881459259).fromNow(); should work fine

moment(moment(1551881459259).format('YYYYMMDDkkmmss'), 'YYYYMMDDkkmmss').fromNow()

this works, other options doesn't


refactor(workspace): entry points changed to index

What is this change for?

What is this change for?

I changed the folders structure
instead of
src/components/button/button.ts

to
src/components/button/index.ts


How to click and input text?

Firefox its RDP implementation doesn't have any actors to directly interact with the html content like peripherals would do. Though it's possible to use javascript with evaluate_js_async():
let textbox = document.querySelector("#suggestion-search");
textbox.focus();
textbox.select();
textbox.value = "test";

However it might not work for every website.


Buttons Group

Added buttons group directive sm-buttons

Ugh, all the other commits from my previous pull request merged into this.

Closing this, will make a fresh pull request, without the other changes.


Documentation on mapping backspace to various chars

Description
Describe your changes in detail
Add documentation to show how to make BS send ^H. This is the default for a vt220 IIRC.

Motivation and Context
Why is this change required? What problem does it solve?
This is a "common" configuration item for people who connect to different systems.

How Has This Been Tested?

[ ] Please describe how you tested your changes.

I read it twice.

[ ] see how your change affects other areas of the code, etc.

Checklist:
Go over all the following points, and put an x in all the boxes that apply.
If you're unsure about any of these, don't hesitate to ask. We're here to help!

[x] I have read the CONTRIBUTING document in my spoken language, and understand the terms
[x] I have updated (or added) the documentation accordingly.
[ ] I have added tests to cover my changes.
[ ] I have gone through all the steps, and have thoroughly read the instructions

@cqexbesd NB: website and therefore proper documentation is one of the bigger topics for this year, so I hope we'll finally end up having a nice one. Even if it means I'll have to add a bounty to it (because I suck at website dev) :-)


Unable to update haproxy version

Please find the error on below
error: Unhandled error: Traceback (most recent call last): File "/usr/lib/python2.7/site-packages/ansible/config/manager.py", line 554, in update_config_data value, origin = self.get_config_value_and_origin(config, configfile) File "/usr/lib/python2.7/site-packages/ansible/config/manager.py", line 498, in get_config_value_and_origin value = ensure_type(value, defs[config].get('type'), origin=origin) File "/usr/lib/python2.7/site-packages/ansible/config/manager.py", line 121, in ensure_type makedirs_safe(value, 0o700) File "/usr/lib/python2.7/site-packages/ansible/utils/path.py", line 90, in makedirs_safe raise AnsibleError("Unable to create local directories(%s): %s" % (to_native(rpath), to_native(e))) AnsibleError: Unable to create local directories(/usr/share/httpd/.ansible/tmp): [Errno 13] Permission denied: '/usr/share/httpd/.ansible' Traceback (most recent call last): File "/usr/bin/ansible-playbook", line 62, in import ansible.constants as C File "/usr/lib/python2.7/site-packages/ansible/constants.py", line 174, in config = ConfigManager() File "/usr/lib/python2.7/site-packages/ansible/config/manager.py", line 290, in init self.update_config_data() File "/usr/lib/python2.7/site-packages/ansible/config/manager.py", line 566, in update_config_data raise AnsibleError("Invalid settings supplied for %s: %s\n" % (config, to_native(e)), orig_exc=e) ansible.errors.AnsibleError: Invalid settings supplied for DEFAULT_LOCAL_TMP: Unable to create local directories(/usr/share/httpd/.ansible/tmp): [Errno 13] Permission denied: '/usr/share/httpd/.ansible'

Create folder .ansible and do apache owner it

Where i need to create .ansible floder

There is path in your log: /usr/share/httpd/.ansible/

['', 'PLAY [<IP_ADDRESS>] ***********************************************************', '', 'TASK [Gathering Facts] *********************************************************', 'fatal: [<IP_ADDRESS>]: UNREACHABLE! => {"changed": false, "msg": "Failed to connect to the host via ssh: Could not create directory '/usr/share/httpd/.ssh'.\r\nFailed to add the host to the list of known hosts (/usr/share/httpd/.ssh/known_hosts).\r\nno such identity: /var/www/haproxy-wi/keys/EUSAUTHXPRX01.pem: No such file or directory\r\nPermission denied (publickey,gssapi-keyex,gssapi-with-mic,password).", "unreachable": true}', '', 'PLAY RECAP *********************************************************************', '<IP_ADDRESS> : ok=0 changed=0 unreachable=1 failed=0 skipped=0 rescued=0 ignored=0 ', '', "error: Can't install Haproxy service

Your log has full problem description

First of all create folder: /usr/share/httpd/.ssh like previous folder

And second error is: no such identity: /var/www/haproxy-wi/keys/EUSAUTHXPRX01.pem: No such file or directory


Fix markdown close on podcast:indexers note + millisat-bitcoin change

Should have worked with branches, now the bitcoin-millisat commit is also included here, sorry.


Logging improvements

Description
We want to improve the log file output to make it easier to understand for end users. I propose to clean up the INFO log level and make the target group for INFO the end user.
[ ] Remove thread and class name on the info log level. I think this does not provide any value to the end users
[ ] Show an informative configuration section at the beginning (including storage, sync mode). Take Geth as example
[ ] Move Refreshing DNS records with enrtree to debug or trace (in progress)
[ ] No sync target, checking current peers for usefulness is too spammy. It should be only logged in a fixed interval
[ ] Snap sync: I think for we should output just the download progress in percent in a regular interval (eg 30s). The other output can be moved to DEBUG.  Geth prints an estimation how long it will take as well, which I find very useful
[ ] Snap sync: Once we are downloading blocks, it would be useful to print the the age of the latest block (eg 3y10m2d) to give a better sense how far we behind we are
DEBUG and TRACE should not be changed for the time being, as the target group are mainly the Besu contributors. There are some extra logs that we should add:
[ ] Add why transactions have been rejected
Refer to protocol-misc # 486

I'd be cautious of only logging download progress at % intervals because you don't know how long it will take to progress and the user needs regular feedback so they don't think it's stalled.
I'd suggest a regular log line, say every 15 seconds, that gives a snapshot of the sync progress. Something like: Syncing - Blocks: 1,000,000/10,000,000 (10%), World state: 25%, Peer count: 25.  They may not be the exact details and there would need to be variants when progress is paused waiting for peers or waiting for info from the CL etc but that's the general idea.
Then the logs about how the sync works could move down to debug level unless:

they're something particularly notable (starting sync, switching between modes in sync etc)
they're something unusual that the user should be aware of (a peer failing to respond is normal, an invalid block during long sync is probably unexpected and worth logging) or
the user needs to take some action (out of disk space, no calls from the CL for too long etc)

This PR filters the DNS refresh messages (they come from tuweni) https://github.com/hyperledger/besu/pull/4155/files

Decided not to investigate  log4j configuration issue further. Closing this issue.


fix: add "UNDERSCORE_CHARSET hexLit/bitLit" in Literal

Signed-off-by: Wang Ruichao<EMAIL_ADDRESS>
What problem does this PR solve?
Close #451 , cc pingcap/tidb#14800
Bad SQL
select _utf8 0xD0B1, _utf8 X'D0B1';
select _utf8 0b1101000010110001, _utf8 B'1101000010110001';

What is changed and how it works?
Add "UNDERSCORE_CHARSET hexLit/bitLit" in Literal.
Check List
Tests

Unit test

 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.Wang Ruichao seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.

@wangggong Please regenerate the parser.go to resolve the conflicts.

@wangggong Please regenerate the parser.go to resolve the conflicts.

@tangenta done, thx
PTAL


Refactor Ceph libvirt secret management code

Replace shell script with Ansible tasks. The shell script creates data structures, has an XML template, and executes commands. Bulild data structures in Ansible, use the template module, add a validation on cephx key value but also run commands where necessary. Aim is to make the Ansible logs easier to debug in the field.
Update edpm_libvirt molecule to test the new Ansible tasks. This includes generating mock data during prepare and confirming libvirt outputs the correct data during verify. Also, add whitespace to prepare.yml for readability.
Jira: OSP-27777

+2

Adding become to configure tasks because this job showed a permissions problem.
https://review.rdoproject.org/zuul/build/bd37f4126f134ca2babd90f1e211c1d7
TASK [edpm_libvirt : render libvirt container config.json] *********************
fatal: [compute-1]: FAILED! =>
  msg: 'Failed to get information on remote file (/var/lib/openstack/config/libvirt/libvirt_virtlogd/config.json): Permission denied'

can you try reverting the become and update the mode for /var/lib/openstack/config/ceph
if you check locally
i.e. ls -al /var/lib/openstack/config I'm expecting it to be 750 and it needs to be 755

Yes, you're right.
Removing the becomes and switching task Create Ceph client file directory
from 750 to 755 [1] made the job [2] get past the libvirt rendering and then
it failed on verify_ceph_secret.yaml which I can easily fix in the next PS.
Thanks.
[1] https://github.com/openstack-k8s-operators/edpm-ansible/compare/9821afa4cdce7dad3f282313aae2a35e0f75619f..664c69ccda08553488b055cd45a7e36195685d6d
[2] https://review.rdoproject.org/zuul/build/78826af93cdd4e2d950b830d1034f541

can you try reverting the become and update the mode for /var/lib/openstack/config/ceph
if you check locally
i.e. ls -al /var/lib/openstack/config I'm expecting it to be 750 and it needs to be 755

Completed.

This latest PS passed the changed parts of edpm-ansible-molecule-edpm_libvirt.
2023-09-07 13:59:36.245451 | controller | TASK [ensure ceph secret is configured] ****************************************
2023-09-07 13:59:36.345681 | controller | included: /home/zuul/src/github.com/openstack-k8s-operators/edpm-ansible/roles/edpm_libvirt/molecule/default/test-helpers/verify_ceph_secret.yaml for compute-1
2023-09-07 13:59:36.345884 | controller |
2023-09-07 13:59:36.346190 | controller | TASK [Get the mock FSID from ceph conf file] ***********************************
2023-09-07 13:59:36.693002 | controller | changed: [compute-1]
2023-09-07 13:59:36.693245 | controller |
2023-09-07 13:59:36.693620 | controller | TASK [Get the mock cephx key from ceph keyring file] ***************************
2023-09-07 13:59:36.999855 | controller | changed: [compute-1]
2023-09-07 13:59:37.000153 | controller |
2023-09-07 13:59:37.000500 | controller | TASK [Get cephx key from libvirt_virtsecretd container by passing FSID] ********
2023-09-07 13:59:37.499345 | controller | changed: [compute-1]
2023-09-07 13:59:37.499608 | controller |
2023-09-07 13:59:37.499962 | controller | TASK [Assert that virsh secret-get-value returned the correct value] ***********
2023-09-07 13:59:37.562518 | controller | ok: [compute-1] => changed=false
2023-09-07 13:59:37.562809 | controller |   msg: All assertions passed

rdo-check

check-github

recheck

recheck


Update to miow to use windows-sys 0.33

This brings miow in line with the parking_lot and winit crates that have already adopted windows-sys version 0.33.

Note that this PR would be a breaking change, as we're re-exporting types from windows-rs in our public API. This would mean mio would need to upgrade their version of miow each time we update windows-rs. That's not a great situation to be in.
Ideally we could evade that by defining our own wrapper with an #[repr(transparent)] that can be converted to either one of windows-rs or winapi types. But I'd like to have that in place before we update windows-rs (having it as this PR would be fine too) so that we can limit the number of breaking changes we have to issue.

(talked privately; we're going to include the required changes as part of this PR)

parking_lot just merged an upgrade to windows-sys 0.34. https://github.com/Amanieu/parking_lot/pull/331. So if the intention is to try to use the same version this PR also needs to update to that.

Feel free to reopen the PR if you'd like to push this forward.


gdp-hmi: Fix copyright date

[GDP-279] Update copyright on HMI applications

+1 (thanks for the change :) )


Respect --config-file when downloading

We have an option for specifying a config file, but that option is only used when opening a session. Whenever a new build is downloaded we throw the config.json in the same tired old location.

Thanks a lot for the fix @Sinetheta.   Could you maybe provide some testcoverage for the fix you did?

That's a tough one. Since not much has been abstracted here I would have to actually hit (or mock) the file system. Which is generally not such a good thing.
A test for copy_config_json would basically be

Add a config file fixture
Add a .zip fixture
Open the zipfile
Pass them both to copy_config_json
Verify that the contents of the zipfile were placed where the config file specified.

Meaning that the spec would do about 4 times as much work as the method, which is why I imagine that it was never tested in the first place.

I've added a spec that just uses a mock to verify the call. A bit tautological, but hey, it's a test :wink:

thanks a lot.  it might be a only a mock spec, but it will prevent a regression of this particular bug.


fix: copy the missing licenses.css file

Connects to #52

Coverage remained the same at 93.333% when pulling 221e66b52df4ddeace5158cb10913025bb3720d7 on helio-frota:licenser-52 into 0f263057b10f6c63bb2d8be7f57ba2dd482463e5 on bucharest-gold:master.


Testing code that generates #[cfg(test)] items

Hi, I'm working on a macro that generates test code, all of which is inside a #[cfg(test)] gate. Unfortunately, this makes it hard to test with trybuild. For example, this test passes:
// main.rs
fn main() {}

#[test]
fn foo() {
    trybuild::TestCases::new().pass("simple.rs");
}

// simple.rs
fn main() {}

#[cfg(test)]
compile_error!("error");

I'm trying to write a test that verifies that it fails if certain trait bounds are not satisfied, but since trybuild doens't compile with test enabled, it's not possible.
I could add stuff to the codegen of the macro to detect the trybuild cfg flag, but that adds a fairly substantial difference between "test" and "production" behavior, which isn't ideal.
I can set RUSTFLAGS="--cfg test", but this applies to dependencies as well, and causes compilation failures.
One solution I can think of is to have an env var TRYBUILD_CARGO_CMD which defaults to run, but could be set to test for this use case. Is that something that sounds worthwhile? If so, I'd be happy to write a PR.
Thanks :grin:

I would prefer not to support this in this library, but it would be reasonable for someone else to maintain a more fully featured fork that serves this situation.


about support image selection in the validation period

Hi,
Thanks for sharing I am checking out the code and could not understand how you select support images during evaluation. From the code, it seems that during testing, the support images are from the testing split also? Thanks

Yes. Please find Fig. 6 of https://ieeexplore.ieee.org/document/9709261


[validator] specify effectively shared objects as tx inputs with id, not (id, version)

Today, we require effectively shared objects (i.e., objects owned directly or transitively be a shared object) to be specified as tx inputs using an (id, version) pair.

This will lead to the "Cardano problem" where it is very difficult for users to craft non-failing transactions that are touching the same effectively shared object (since they will be unable to accurately predict the version, resulting in a failed tx whenever the version doesn't match).
One solution is to require effectively shared objects to be specified as id inputs to txes (just like shared objects).

Closing this in favor of https://github.com/MystenLabs/sui/issues/4203


XIVY-14832 remove Soap WebServiceTester button

it was removed form the ivy-core

as discussed in the quality-gate https://github.com/axonivy/core/pull/6594


Application monitoring - deployment

Sentry setup: https://github.com/SocialFinanceDigitalLabs/cs-demand-model-regional/pull/130


BundleMaker REST call allowing to add ZIP/JAR to catalog - Part #2

Builds on top of #485. It implements the missing bits and pieces, such as:

fields version and bundle are required under brooklyn.catalog when uploading a ZIP/JAR archive
returns 400 HTTP status code is anything goes wrong during the process of the archive (as this is a bad input, therefore a bad request)
avoid using the CatalogPopulator as it reloads all bundles. Instead, it uses the new CatalogBundleLoader that loads only a given bundle
if the bundle has been installed, but the CAMP parser fails, the bundle is uninstalled
adds more unit tests, especially to check error handling

@geomacy I addressed your comments and separated out the CatalogBomScanner, CatalogBundleTracker and CatalogBundleLoader. I also dug into the swagger docs and found out how to hide specific @ApiOperation. I had to bump the version of the swagger-maven-plugin because it was throwing an NPE with the new hidden parameter.
I did a full build locally, with the latest master of the other submodules, everything went fine so I'm guessing the error reported by Jenkins was due to something else

@tbouron Looks like #485 broke the brooklyn-client build, can you try fixing it here.
See https://builds.apache.org/view/Brooklyn/job/brooklyn-client-master/41/console.

@neykov Yep, I'll look into it

Thanks @tbouron.

@neykov Should be fixed now. I also created https://github.com/apache/brooklyn-client/pull/41 to use the non-deprecated call

hi @tbouron I get an error trying to build this
[ERROR] Failed to execute goal com.github.kongchen:swagger-maven-plugin:3.1.4:generate (default) on project brooklyn-rest-api: Unable to parse configuration of mojo com.github.kongchen:swagger-maven-plugin:3.1.4:generate for parameter locations: Cannot assign configuration entry 'locations' with value 'org.apache.brooklyn.rest.api' of type java.lang.String to property of type java.util.List -> [Help 1]

I just synced and built this as-is, should I be merging pre-requisite PRs or something?

@geomacy Hum odd. I have build this multiple time locally with the latest master of all the sub-modules. Jenkins also seems happy.
I did another PR in brooklyn-client to fix the tests in the client (related to the changes I did here) but I don't see how that could be related.

aha - Maven version issue.   Upgrading 3.3.3 -> 3.3.9 fixed the problem.

Sounds like we should bump the maven version requirement - https://github.com/apache/brooklyn-server/blob/master/parent/pom.xml#L948. Or perhaps downgrade swagger-maven-plugin?

@neykov we cannot downgrade swagger-maven-plugin because of the issue I reported in my comment.
We can either update the plugin configuration to properly use list instead of comma separated string or bump the maven requirement. Up to you.

Not sure what you mean by "update the plugin configuration". Couldn't see any related configuration in the pom.xml. No strong opinion though, either one is fine by me.

@neykov I'm talking about this

Thanks for the pointer, missed that in the changes.

@neykov You didn't miss it, I added it after your comment to solve the issue ;)

I hope that PR of mine could just be used as-is to finish off that refactoring.  Personally I wouldn't worry too much about getting this integrated just now with a singleton CatalogBomScanner, especially as this is disabled by default at the minute anyway (see #233), until it can be made to work well with rebind.

I think this looks good; further work as discussed earlier can be done as a separate PR. Will merge.

@tbouron started getting test failures in VanillaSoftwareProcessTest.testVanillaSoftwareProcessCanUseResourcesInBundles around the catalog changes. Could you have a look.

@neykov Fixed in #608

That was quick, thanks @tbouron.

Thanks for polishing this off @tbouron !


feat: [#347] Switch logging channels on runtime

Closes https://github.com/goravel/goravel/issues/347
📑 Description
✅ Checks

[x] My pull request adheres to the code style of this project
[x] My code requires changes to the documentation
[ ] I have updated the documentation as required
[ ] All the tests have passed

ℹ Additional Information

Codecov Report
Attention: Patch coverage is 31.57895% with 26 lines in your changes are missing coverage. Please review.

Project coverage is 69.53%.

Files
Patch %
Lines

log/application.go
34.28%
22 Missing and 1 partial :warning:

log/logrus_writer.go
0.00%
2 Missing and 1 partial :warning:

Additional details and impacted files
@@            Coverage Diff             @@
##           master     #482      +/-   ##
==========================================
- Coverage   69.70%   69.53%   -0.17%
==========================================
  Files         174      174
  Lines       10721    10744      +23
==========================================
- Hits         7473     7471       -2
- Misses       2681     2704      +23
- Partials      567      569       +2

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


2000 character limit exceeded using convert token endpoint

I get an error when I try to convert the Microsoft Azure token to DRF access token using the convert-token endpoint.
I have been getting this error:
{"token":["Ensure this field has no more than 2000 characters."]}
I notice in the ConvertTokenSerializer serializer the max_length is set to 2000.
class ConvertTokenSerializer(Serializer):
    grant_type = CharField(max_length=50)
    backend = CharField(max_length=200)
    client_id = CharField(max_length=200)
    client_secret = CharField(max_length=500)
    token = CharField(max_length=2000)

This is the curl command I ran
curl -X POST -d "grant_type=convert_token&client_id=xxxxxxxxxxxxxxxxx&client_secret=xxxxxxxxxxxxxxx&backend=azuread-v2-tenant-oauth2&token=MY_TOKEN_THAT_IS_3000_CHARS" http://localhost:8000/auth/convert-token
The token is exceeding the 2000 character but it cannot be controlled as Microsoft generates this. I have tried numerous times to reduce the scope on Microsoft's end to limit the length of token but I am unable to do this.
I am not sure if this is by design or this is a bug, please kindly advise. I appreciate the time and respect the effort required to fix this issue if deemed a bug. I fully support the use and creation of this library. Thank you.

https://github.com/wagnerdelima/drf-social-oauth2/pull/216 fixes this.


Topic extraction

Use natural language processing to extract topics from data, and display them in a separate analysis tab.
This is to be done with fast-topics package.
It can be applied to things like YouTube watch history(#47), twitter posts and messaging conversations like whatsapp and telegram.
The interesting thing here is the topics which are extracted. Not classifying each document. Not yet sure how to choose the number of topics to extract.
Need a way of only displaying topics that documents match closely to, and filtering out ones which contain generic words that match every document.

Call to web assembly has been moved to a web worker, freeing up the main thread and keeping the website interactive while loading. A loading icon is shown in the tab and the tab is disabled while its being calculated. Topics are only calculated when a file is selected.
The WASM file is only loaded when a file using topic extraction is processed (ie lazy loading).
A design pattern has been established for off-thread computation. This serves as an example.
Performance timing is done, but results are not shown to the user, but instead logged to the console.
There is no button for the user to recalculate it and the user cannot control how many topics are extracted. This is optional future work.


titles_combined.txt file should be accessible from library

Using the API,

some titles are in title case and some of the titles like .NET are in upper case so we can not change input case into any particular case.

If the file is accessible then we can add some new titles if we want.

Please you should add this if possible

I think I lower-cased everything, since titles can appears as "Vice President" or "vice president".
I think default indexer may be case-sensitive, but there is an alternate algo included that can do case-insensitive search.
File is accessible here:
https://github.com/fluquid/find_job_titles/blob/master/src/find_job_titles/data/titles_combined.txt.gz
But it would probably be a good idea to be able to override this as a library.

oh, thank you for replying.
I figured it out in the library I changed the load_titles() method asked because I am new to python.
now just one thing that when I change the text inside the title file or pass a text file it gives me the error that got the string instead of bytes so, will have to do something about that.

If you paste your code re: load_titles() maybe I can help.
I assume you are using python 3, and the output from load_titles should be list of str, I think

yes python3, I am using your method only with changed file I am doing,
def new_load_titles():
with resource_stream('find_job_titles',
'data/titles_combined.txt.gz') as fhandle:
with gzip.GzipFile(fileobj=fhandle, mode='r') as gzf:
for line in gzf:
# Note: decode rather than "rt" for py2 compat
# TODO: using pyahocorasick this should now be 'str' again ;(
yield line.decode('utf-8').strip().lower())
from find_job_titles import load_titles
load_titles= new_load_titles()
I am changing the content inside the file and converting them to lower case in this method

Solved It by reading the text file as bytes and reading it with open
def fload_titles():
    """
    load job titles as generator from txt.gz file included in the library
    """
    with open("data/titles_combined.txt", "rb") as input:
        for line in input:
                yield line.decode('utf-8').strip().lower()


PD cleanup: nest form fields under new fields key

Nest all fields in forms under 'fields' key - so the only top-level fields are 'stepType' and 'fields'.
Separate hydratedFields from fields.
Do this for moveLiquid, pause, and mix.

Re-wrote #3161, so this ticket is now a duplicate of that


Remember settings

Hi,
I don't know if this is the right place to suggest a small improvement -- or if this is even possible:
It would be great if the 'new session'-form would remember the 'custom singularity' and 'command to initiate' settings like it does for the requested resources.
Thank you and have a great day,
Felix

It seems the setting is actually cached by ood, but in  https://github.com/slaclab/slac-ood-jupyter/blob/3d72f6326effc55db4e91cde6de75c96bab31a64/form.js#L102 it is overwritten. It would be nice, if selecting 'custom image' would use the cached command, which should be accessible for the script at $('#batch_connect_session_context_commands').text()

thanks for the suggestion. in the meantime, feel free to send a PR on form.yml.erb if its a common image that you and or your may want to share.


Inline descriptor inner types

Patch 1 is trivial cleanup
From commit log of patch 2
The `ShInner` and `WshInner` enum's provide no benefit that I can
discern. Both include only tuple variants with private tuple fields. We
can therefore inline the enums into `Sh` and `Wsh` respectively with no
loss of type safety. Doing so marginally simplifies the code, the
biggest benefit is it stops devs wondering why there is an inner type.

This is a breaking change but hopefully not too disruptive of one.

On that topic, whats your view on the status of miniscript, you said in the past that the 1.0 release was a mistake, do you consider the crate is in the same state as rust-bitcoin i.e., we can shake things up a bunch with the aim of trying to stabilise so we can stop making so many changes to the public API?

@tcharding yeah. I think a "real" 1.0 should even be delayed until we get rid of all the recursion, make it impossible to construct invalid objects, and maybe improve the in-memory representation of things.

Cool cheers, plenty to do then.

Rebase. Also fix a couple of match statements that used match &self to use match *self as is customary.

Ahhh, so my commit log shows the problem

The ShInner and WshInner enum's provide no benefit that I can discern.

The problem is with me, there is a difference, as you state - maintaining the invariant. My bad.
Closing, thanks.

Ahhh I kind of feel vindicated, it is not immediately apparent that this function maintains the [undocumented] invariants.
    /// Create a new p2sh descriptor with the raw miniscript
    pub fn new(ms: Miniscript<Pk, Legacy>) -> Result<Self, Error> {
        // do the top-level checks
        Legacy::top_level_checks(&ms)?;
        Ok(Self { inner: ShInner::Ms(ms) })
    }


Docker swarm and InetAddress.getLocalHost() method

InetAddress.getLocalHost() method does not work correctly with docker and produces noise in logs.
My suggestion to change the order of calls:

we are trying to get host value from system properties
try to invoke InetAddress.getLocalHost() if system properties not set

I think you should pass --hostname to docker in such situation
https://stackoverflow.com/questions/27087786/java-application-cannot-get-ip-address-of-the-host-in-docker-container-with-stat


Add recipe for SQL Advent Calendar #15 - Interval Joins

Thanks for the review, @knaufk ! Merging now.


Revise dir structure a bit

This PR does:

Fix a broken link
Remove pyviz channel from the env file
Renames notebooks to reflect their order under each caption
Combine our previously called CH.2 and CH.3 into a single chapter (newer CH.2) that is named "METHODS & LIBRARIES FOR UNSTRUCTURED GRID VISUALIZATION" where we can cover visualization basics such as methods, rendering techniques, and libraries.
Combine the previous CH.4 and CH.5 into a single chapter (newer CH.3) called "UXARRAY VISUALIZATION" that'd house both topology and data variable visualization. I preserved all those notebooks under Data Variable vis. for now, but I believe those could be further combined into fewer notebooks.

Note: The term "Chapter" as I refer to as parts of the cookbook in this text is actually implemented as "Caption" in the repo.

This will be a terrific contribution when it is complete!!!


Rename to GoRI

All references to 'Parks of Rhode Island' should be replaced with 'Get Outdoors RI'.

Fixed as of b70a2f7ff1b5d17b64a5769614a1c8d0c0e93049.


RngCore trait is implemented

While working on this task it turned out that we do not provide true randomness of generated numbers yet (our generator is not TRNG yet), which does not allow us to implement CryptoRng marker trait for our structure yet. The #1120 was created due to that.
For now I left the TODO instead of implementing CryptoRng. When #1120 will be closed, we'll replace TODO with implementation of a marker trait.
Also not sure if this PR should close the initial #942, to be discussed.

(Sorry, attached the wrong issue by mistake)

@JurajSadel
Sorry, didn't notice this warning, fixed, thanks!


Added link to blog post

Added link to "How to configure Security Events collection with Azure Monitor Agent" blog post.

#sign-off


Insert new modules in config.js

Before I would like to thank you for excellent work! I'm from Brazil and maybe this translation is wrong ... I searched the forums but did not. I am unable to do any work module. Follow the instructions but seems to have something wrong. When you turn the program appears to set the screen file, config. js

For support please use: http://forum.magicmirror.builders
Thanks!


feat: Add support for "Prefer Cross-Fade Transitions" into AccessibilityInfo

Summary
This PR adds prefersCrossFadeTransitions() to AccessibilityInfo in order to add support for "Prefer Cross-Fade Transitions", exposing the iOS settings option as proposed here https://github.com/react-native-community/discussions-and-proposals/issues/452.
I believe this would be especially helpful for solving https://github.com/facebook/react-native/issues/31484
TODO

[ ]  Submit react-native-web PR updating AccessibilityInfo documentation.

Changelog
[iOS] [Added] - Add support for "Prefer Cross-Fade Transitions" into AccessibilityInfo
Test Plan
On iOS 14+

Access Settings > "General" > "Accessibility" > "Reduce Motion", enable "Reduce Motion" then enable "Prefer Cross-Fade Transitions".
Open the RNTester app and navigate to the Accessibility page

https://user-images.githubusercontent.com/11707729/154588402-7d050858-3c2d-4d86-9585-928b8c66941b.mov

@makovkastar has imported this pull request. If you are a Meta employee, you can view this diff on Phabricator.

Hi @makovkastar I see that the Facebook Internal - Linter action raised 1 warning, is there something I need to update/change?

Hi @cortinico I see that the Facebook Internal - Linter action raised 1 warning, do you mind checking if there is something I need to update/change?

Hi @gabrieldonadel, thanks for your patience here. In order to make the internal Linter happy, we need to make the getCurrentPrefersCrossFadeTransitionsState() method optional in NativeAccessibilityManager.js. Then add a null check in AccessibilityInfo.js to make sure that the method exists.

Hi @gabrieldonadel, thanks for your patience here. In order to make the internal Linter happy, we need to make the getCurrentPrefersCrossFadeTransitionsState() method optional in NativeAccessibilityManager.js. Then add a null check in AccessibilityInfo.js to make sure that the method exists before calling it.

Got it, thanks for reviewing this @makovkastar, I've just pushed a commit making getCurrentPrefersCrossFadeTransitionsState optional

@makovkastar has imported this pull request. If you are a Meta employee, you can view this diff on Phabricator.

@makovkastar seems that the linter check failed again, do you mind checking what is causing it?

@makovkastar has imported this pull request. If you are a Meta employee, you can view this diff on Phabricator.

@makovkastar seems that the linter check failed again, do you mind checking what is causing it?

There was an unrelated internal breakage, I'm rebasing this PR past the fix.


Create per-endpoint load testing script using Locust for Discover API for READs

https://locust.io/
Break down into CuratorUser, SuperCuratorUser, and ReadUser load test classes

Moving out of performance tuning epic and into ongoing tech debt work

Closing this for now. Will reopen when/if performance tuning is prioritized.


Update to latest standards and dependencies

Alaska Airlines Pull Request
Please include a summary of the change and which issue is fixed. Please also include relevant motivation and context. List any dependencies that are required for this change.
Fixes: #166, #167
Summary:
update repo to latest standards and deopendencies.
Type of change:
Please delete options that are not relevant.

[ ] New capability
[x] Revision of an existing capability
[ ] Infrastructure change (automation, etc.)
[ ] Other (please elaborate)

Checklist:

[x] My update follows the CONTRIBUTING guidelines of this project
[x] I have performed a self-review of my own update

By submitting this Pull Request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
Pull Requests will be evaluated by their quality of update and whether it is consistent with the goals and values of this project. Any submission is to be considered a conversation between the submitter and the maintainers of this project and may require changes to your submission.
Thank you for your submission!
-- Auro Design System Team

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Improve security practices for carbon-components-svelte

Hey there, Carbon team member here! 👋 We'd like to ensure the security practices for packages published from the carbon-design-system GitHub org are in place and up to date. We'd like to work with you to get the following security practices implemented:
- [x] Add a security policy to the repository
- [x] Enable Security advisories
- [x] Enable Private vulnerability reporting
- [x] Enable Dependabot alerts
- [x] Enable Code scanning alerts
- [x] Enable Secret scanning alerts
- [x] Publish your package(s) to npm with [provenance statements](https://docs.npmjs.com/generating-provenance-statements)

Most of these can be found under the Security tab for this repository. You may already have most of these implemented and turned on - if so, awesome!
For establishing a security policy, the existing security policy for the Carbon monorepo can be used as a template if you'd like. It can be modified to include proper version(s) for your package and any other attributes unique to your project that you may want to highlight.
I'm happy to meet up and chat about this if you'd like, just let me know. Thanks in advance for your help in ensuring security and stability across the Carbon ecosystem! 🙏 💙

Thanks Taylor.
I completed the checklist, including adding a new workflow that publishes this package to NPM with provenance.

@metonym Awesome, thanks!


Docs/sqlite for windows

I've expanded the setup instructions in the readme a bit, and set up the database configuration to use sqlite in development/test - rationale there is that it will hopefully be pull and go for most people, and we won't have to worry about explaining how to set up a postgres server on everyone's machine, create their db user for the app, and all that jazz
Feedback very welcome on everything contained in this pr :)

@glsignal the README is so good!!


Automator: update istio/api@master dependency in istio/istio@master

Generated by Automator - 2020-06-17T18:47:50+00:00

/retest

@howardjohn CLA is broken. Can you please manually merge?


new discord update broke theme

this theme has been really nice but i think another discord rebranding update broke it and the theme no longer works. ill send some images here:

i've restarted my computer, reinstalled betterdiscord and disabled and re-enabled the theme, but there might've been a recent update (not the first rebrand one) which prevented it from activating or something

same problem

It's because of a big update and it's not been updated on BD's site / store yet. Fixing soon.

It's because of a big update and it's not been updated on BD's site / store yet. Fixing soon.

it's fixed for me now 👍


Building v2.22 fails on Debian Stretch

Newsboat version: Source from newsboat-2.22.tar.xz
Output of build process:

Coloring converted to HTML
Coloring stripped

"Progress bars" are missing in the latter.
I assume it's a problem on my system here, since its an older version of Debian.
Versions:
gcc             4:6.3.0-4
rustc           1.41.1+dfsg1-1~deb9u1
libstfl-dev     0.22-1.3+b4
sqlite3         3.16.2-5+deb9u3
libcurl3        7.52.1-5+deb9u13
libssl1.0.2     1.0.2u-1~deb9u3
libcurl3-gnutls 7.52.1-5+deb9u13
gettext         <IP_ADDRESS>-2+deb9u1
libsqlite3-0    3.16.2-5+deb9u3
libsqlite3-dev  3.16.2-5+deb9u3
pkg-config      0.29-4+b1
libxml2-dev     2.9.4+dfsg1-2.2+deb9u3
libjson-c-dev   0.12.1-1.1+deb9u1
asciidoctor     1.5.4-2

Hi!

rustc           1.41.1+dfsg1-1~deb9u1

This is too old, we only guarantee 1.44 or newer to work. I forgot to include that in the changelog — sorry about it! You can install a newer version via rustup, or from Debian testing.
We also bump the requirement on each minor release, so e.g. Newsboat 2.22 will require Rust 1.46 or newer.

Thank you for your answer, and the hint pointing to rustup.
After installing rustc this way building succeeded flawlessly.

Awesome! I'll close the issue then.

I would suggest to add a check of the rustc version in config.sh,
and to add a recommendation  to run config.sh before running make
in README.md, if this appropriate.

I doubt that adding anything to the README will help, since the Rust version is already listed there and you missed it ;) Also, we provide a guarantee of which version works, not a requirement of what version is needed — the difference is that an older version might work, we just don't know because we didn't check. So the best we would be able to do is a warning, which will quickly scroll off as the build starts.
Perhaps I should rename config.sh? It seems like people quite often take it for an autoconf-style configure script, whereas in fact it's just a helper to get some info from pkg-config.

the Rust version is already listed there and you missed it ;)

Indeed, I did =:)

Perhaps I should rename config.sh?

Perhaps something like check_versions.sh?

That'd be misleading, because that script doesn't check versions. (Well, okay, it asks for at least libjson-c 0.11, but that's it). And I don't want to turn it into an autoconf-style configure because, as you probably know, autoconf got born out of people's frustration with writing those scripts by hand.


Removing widgets enhancement request.

Flutter 1.2.1 • channel stable • https://github.com/flutter/flutter.git
Framework • revision 8661d8aecd (3 months ago) • 2019-02-14 19:19:53 -0800
Engine • revision 3757390fa4
Tools • Dart 2.1.2 (build 2.1.2-dev.0.0 0a7dcf17eb)
I was wondering if there can be a Remove Widget With Children SDK option to use in an IDE such VS Code. Or better yet, have a Highlight Widget option. Then the user can press Delete key to remove it.
A question along this. Why is Remove Widget sometimes not available in an IDE such as VS Code?
Thanks

/move to flutter/flutter


Chat refactor

It would be great if we could fix the chat so avatars appear in the chat message area even when they aren't cached from visiting the user's view. But that didn't work before either, so it's not a priority.

@jjeffryes You'll have to talk to the server folks on that one to consistently provide us with the avatar hash. If the conversations API returns one, the chat app is using it to call get_image. If it doesn't, we show the generic avatar. Most of the time, it doesn't return one.


Snapshot/Volume fix

This PR removes the EBS volume config in the block device mapping of the launch configuration.
This is an accompanying PR for https://github.com/amplifylitco/asiaq_astro_config/pull/653

after remove extra_disk related code. I think I will give it a +1

We'll implement the solution using Autoscaling group lifecycle hooks.


Update stcs.rst

fix scts doc title

this PR is not relevant anymore as documetation for this was rewritten to adoc.


Set all the values of amc controls on start up.

Fix for RS5-11488

As we said -- please check with Aviad because it's not what the request was: the request was to fix all the values when the user clicks on custom and leave the FW behavior (changing the digital gain values automatically) as it was for the other presets.
As you made it, it looks like there's simply no meaning to 'digital gain', no? It has no meaning by itself, right?

The digital gain control also affect TIA, per Aviad.


adjust for goto_block refactoring

The Miri side of https://github.com/rust-lang/rust/pull/66646

@bors r+

:pushpin: Commit fbc1fb20ac606dfcb6f91fb7823272760e583f85 has been approved by RalfJung

:hourglass: Testing commit fbc1fb20ac606dfcb6f91fb7823272760e583f85 with merge 392a75a4c17ae30c86f3426cba3c6b6ef753b072...

:broken_heart: Test failed - status-appveyor

@bors r+

:pushpin: Commit a04620eb697eb064902d5c1b5f7584fa68a1bcfd has been approved by RalfJung

:hourglass: Testing commit a04620eb697eb064902d5c1b5f7584fa68a1bcfd with merge 3bf51f591f7ffdd45f51cd1d42a4002482af2bd5...

:sunny: Test successful - checks-travis, status-appveyor
Approved by: RalfJung
Pushing 3bf51f591f7ffdd45f51cd1d42a4002482af2bd5 to master...


Can provide templates to the user for its portfolio

i would like to work on this issue as a part of contributor hack (Hakincodes'20)

Add a list for Personalised template for resume and fetch data in them depending on user's choice.

Can I take up on this ?
KWoC 2020


How to add digital filter?

Hi, thank you so much for your work. It's awesome, I'm planning to add low pass filter to remove the ambient/jittering noise, but I'm only a beginner and I'm not sure how and where to add the code for this. I don't have any idea how to start. If you could just point me in the right direction, maybe I'd manage.
Thank you again, keep up the good work!

Hi there, I had a look at this when I was playing around with the noise issues.
As always, there's a couple of options :)
There are some nice tools that you can use to build the code for the actual filter - this seems to be a good one - http://t-filter.engineerjs.com/
The code they generate tends to be pretty similar. You push new samples into the filter and you pull out the current value.
In my code there's quite a convenient place to do this - lib/transport/src/Tansport.cpp- has a function:
void add_sample(int16_t sample)

This would probably be a good place to do it as it is the single place where samples are pushed into a buffer before being sent out.


Move tests of storage to the root of the package folder.

Codecov Report

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

@@           Coverage Diff           @@
##           master    #2437   +/-   ##
=======================================
  Coverage   72.16%   72.16%
=======================================
  Files         334      334
  Lines       18741    18741
  Branches     2772     2772
=======================================
  Hits        13524    13524
  Misses       4350     4350
  Partials      867      867

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 22d906e...1fa92e9. Read the comment docs.


Experiment with Signed API without ELB

It should be possible to deploy Signed API without an load balancer and expose its public IP. Note, that we never use the IP directly as we have the CDN put on top of it. Using the ELB seems extraneous because there is just a single instance it forwards the requests to.
This could decrease the AWS costs by a large margin as can be seen on the invoice from January.
The ELB returns 502 when the service is unavailable or under heavy load. I am suspicious whether it works correctly, because we see 502 errors but AWS reports low CPU and memory usage. But it serves as a natural SPAM protection because it will guard the service from potential spam attacks.

Why is this on hold?

I added the label after I opened https://github.com/api3dao/signed-api/pull/218 which was the prototype, but with some heavy downsides. I added "on hold" label, because I wasn't sure whether we want to proceed with this or not.

I've looked at the solution described in the link and I am not sure how it can be applied to our problem.
The problem discussed in the issue is to make the outbound traffic from the service come from a static IP address. For that one can use NAT gateway which translates the private IP address of the service with the static IP of the NAT gateway.
One of the comments suggests whitelisting the whole subnet, but that seems to refer to the target service (called by the AWS service that needs to have the static IP). In fact, our service allows traffic from any IP.
The problem we have is that service IP changes each time a task is redeployed, but we need it to be static to use it with CDN. To assign static IP, it seems we need to use ELB.

Yeah I see, https://stackoverflow.com/a/73038635 only gives you an outbound static IP

I discussed the problem with Stefi and we came up with an alternative, that I like the most. Have a single load balancer that distributes the traffic to different services based on path. This is currently done by CDN (but it's costly).
With this new format, we would have a CDN (for caching) that forwards the traffic to ALB (because it checks target healthiness, and reliably forwards traffic to the service). The costs for load balancer should be much less, because only 1 will be running (and one of the things we're paying is hourly rate). The CDN bill should decrease as well (because we pay for each evaluated path forward rule) and this is not a cost in ALB.

All Signed APIs under API3 are redeployed as proposed here


fix(request-transformer): preserves empty json arrays after decoding and re-encoding

backport FTI-4205
The original PR

Do we have original PR to refer?

@chronolaw Can we please add this to changelog? Could not find this in the changelog for v2.8.2


WikiDot

Domain: Wiki farm, lots of random context
~10m pages
Does not use wikitext - https://www.wikidot.com/doc-wiki-syntax:start Can probably just remove all [[...]] blocks, ||, and @@.

10 m pages will likely turn into less than 1B tokens. I recommend putting working on this off and instead focusing on larger or higher value datasets

I would consider this a highish-value dataset. 1B token doesn't seem that small to me either, given that we have so many sources. I agree that we shouldn't put a crazy amount of effort into this source though.

Based on a simple check "wikidot" in m["metadata"]["identifier"] for m in metadata, none of the wikidot wikis are included in the wikiteam wikis on the Internet archive.
Based on https://github.com/search?q=repo%3Asaveweb%2Fwikiteam3 wikidot&type=code it looks like the wikiteam3 scraper is able to handle wikidot so we should be able to scrape them ourselves pretty easily.
As the wiki markup is different than standard wikitext we should probably try to keep the wikidot implemention separate from the rest of the wiki code

Agreed!


Normalize file paths between POSIX and Windows

I am running builds for https://github.com/aloetesting/aloe on Travis CI and AppVeyor (for Windows). On Windows, directory and file names are separated by a backslash (), everywhere else it is a forward slash (/).
Coveralls should normalize the paths coming from different build systems to properly merge the files coverage.

Thanks for the report! I'll put this on my backlog. PRs are always welcome, as well :)
Sort-of a duplicate of #118.

@koterpillar this issue should now be resolved as of release 1.5.0.


Missing values

Hello,
I'm missing some values which are listed in IP:8080/user/menu. Can I add them anywhere?
Or can they be added generally? If yes, I can send my list.
Thanks in advance.
Best regards
threadstone

I switched to the fork of Tidone. Here all values are shown.


Small detail for section 3, DSN types subdomain

[Enter feedback here]
In the section you have the following text:

To map a subdomain like www.contoso.com with an A record instead of a recommended CNAME record, your A record and TXT record should look like the following table instead:

Azure recommends Godaddy for Domain and Godaddy recommends A type sub domain record as standard. https://uk.godaddy.com/help/add-a-subdomain-4080
Following your guide I tried cname type and it didn't apparently resolve overnight to my app service (I could see from https://dnschecker.org/ that my sub domain had resolved globally based on cname), but the app service refused with

Today I am trying A type, combined with the previous config of txt type: asuid.{subdomain} + id
I tried to add an additional cname for the same sub domain in Godaddy and got an unspecified error (when prepended with www.sub domain it accepted)
I am a total newbie to this area of config, so entirely reliant on your super useful guides, and anything else I can "google down".
Perhaps it would be good to add an explanation of why cname is recommended in preference to A type for a sub domain, and perhaps (might be out of scope for this) - as Azure recommend Godaddy - an explanation of Godaddy configuration?
Once DNS propagation propagation resolves, I'll come back here an update as to if A name for sub domain in Godaddy actually has worked!
A clear message that propagation takes time - so be patient and make sure you have time would be pertinent!
This is a grab of my Godaddy settings (I even added ns for an Azure name server)-

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

ID: 224b73dd-c59d-a833-0d4d-a778af6086a9
Version Independent ID: c6b167c9-9589-45d0-ee6a-7a47a4aec41e
Content: Tutorial: Map existing custom DNS name - Azure App Service
Content Source: articles/app-service/app-service-web-tutorial-custom-domain.md
Service: app-service
GitHub Login: @cephalin
Microsoft Alias: cephalin

Thanks for the feedback! We will review and get back to you shortly.

Okay I have got it working - as promised an update, up to you to use or not!
Hopefully it might be useful - the key missing information on both the app service and in this guide (for a newbie like me) is that in Godaddy (probably all Domain reg sites) you don't add your subdomain as the full url, only as the sub part - in the example below only test
Say you have a domain on Godaddy
mysuperdomain.com (for example perhaps using an application Gateway) and you want to add a sub domain of test.mysuperdomain.com (for example using an Azure app service)
In the image below (no separate save button now on Godaddy)
A record @ ip address  <IP_ADDRESS> would be the public IP address of the Application Gateway - going to mysuperdomain.com would give, as expected 404 - Not found, Microsoft-Azure-Application-Gateway/v2 (this proves you are landing on the Gateway, but haven't allocated a web page to the IP address)
The A record test with ip <IP_ADDRESS> is the Azure application service ip found when you go to add custom domain
Finally the asuid.test is the id shown when you go to add custom domain
Importantly all entries after the @ - ip address entries will only have the sub domain part of the URL (in this example "test")
After a short while you should be able to see the sub domain propagate across the plannet using something like https://dnschecker.org/ see second screenshot below

I didn't realise, when I paint-shopped to "mysuperdomain.com" that the domain exists and is Microsoft Outlook - so it it might need a small update!

Thanks for the feedback! We are currently investigating and will update you shortly.

@JerryMWeeks, the Recommended record type column under DNS record types tells why CNAME records are preferred for sub domains:

CNAME record. You can map a subdomain to the app's IP address directly with an A record, but it's possible for the IP address to change. The CNAME maps to the app's default hostname instead, which is less susceptible to change.

This means if you ever scale your app service up or down, you don't have to go in your registrar and update the A record. If you're binding your mysuperdomain.com as root domain to your-app-service.azurewebsites.net, then you can configure A record to your app service IP address.

But if you're wanting to use to www.mysuperdomain.com or as you currently have test.mysuperdomain.com, then your CNAME will be named test (or www) and the value will be the you-app-service.azurewebsites.net

The name you set is the sub domain you intend to use sub.domain.com, test.domain.com, www.domain.com and the value is URL of your-app-service.azurewebsite.net.
If you have further questions, please feel free to comment down below.


Can Portman generate fuzzy parameter test cases to test dependencies keyword of json

Wanted to know if this can be achieved in Portman. We are using Portman to generate fuzzy parameter testing based on OpenAPI 3.1 specification and we want to test conditional mandatory parameter testing using dependencies keyword of json and will Portman generate those test cases when it generates fuzzy parameter testing

hi @techwizardg
It is not really clear what you what you mean

test conditional mandatory parameter testing using dependencies keyword of json

Can you provide more info?

We are having JSON objects in our request pay load in which some member objects are conditional mandatory based on some values of other member object. This in JSON can be specified using dependencies key word and all our API definitions are based on Open API 3.1. If I use Open API 3.1 definition based APIs with JSON dependencies keyword, will Portman generate fuzzy parameter testing to best conditionally mandatory use cases

Do you have a OpenAPI example that you can share?
Since I find it still difficult to understand the expected result?


Test Reload_Detector through the language server

Write a test that checks that the Managed_Resource cache-clearing method in Reload_Detector works correctly, without using a fake trigger, as the test in Fetch_Spec.enso does. Original discussion
Options are:

Add a builtin that allows scheduleFinalizationOfSystemReferences to be called from Enso
Write a test like RuntimeExecutionEnvironmentTest.scala
Write a test like ManagedResourceTest.java

Add a builtin that allows scheduleFinalizationOfSystemReferences to be called from Enso

No, I am against adding builtins just to artificially increase test coverage.

Just to repeat myself: Logic of Managed_Resource.register ref on_finalize Boolean.True is unit tested at
https://github.com/enso-org/enso/blob/cc0b020e1eb2391f1af21e4a3c3edb4fb720b6c2/engine/runtime-integration-tests/src/test/java/org/enso/interpreter/runtime/ManagedResourceTest.java#L96
We have a unit test of Managed_Resource and another unit test for ReloadDetector, then I believe we have 100% coverage and there is no need to seek more coverage. At least I would personally wait for first bug report about something being broken. Just my 2 Kč.

had the tests pass but the feature fail to work

Yes we already had a bug in there. There's no report because we managed to find it before the PR introducing it was even merged. But that suggests that the test coverage is not enough for this feature - as @GregoryTravis noted - all unit tests were passing but the feature was not working.

If one has too... then it is up to you guys... just no builtins or anything that would be visible (and I don't count ## PRIVATE comments as hiding anything) for Enso users.

What about a private builtin? Only testable in Base_Internal_Tests which has the private access enabled? It would be far easier than instrumenting a language server scenario.

I believe you can make a copy of ManagedResourceTest:
https://github.com/enso-org/enso/blob/9f0c0c025783b5652a4ff31ea6ae518457b17149/engine/runtime-integration-tests/src/test/java/org/enso/interpreter/runtime/ManagedResourceTest.java#L22
I believe it tests everything:

scheduleFinalizationOfSystemReferences
assertGC
Managed_Resource

It should be a great base for your test.

btw. reading the #11673 PR I think it would also be good to add a test that ensures that clicking the refresh button once, ensures that reload happens only once. To avoid a situation where clicking refresh button would cause caches to be effectively disabled.
Thus the test should not be testing Reload_Detector (which is missing the 'reset' logic) but the ReloadDetector.java.


final tweaks, large update to dask portion

hey @fmaussion and @byersiiasa here is a final rough draft. I have started from a clean kernel and run the whole notebook without issue. that result is saved in  xarray-egu-tutorial-answers.ipynb for your perusal.
I will merge it as is, but please look through it one last time if you have a chance.
we should probably discuss tomorrow who will take which sections.

@fmaussion could you also please review the dask portion to make sure we haven't missed anything you'd like there?
thanks!


ORM: the query result rowset becomes invalid after a for loop

I have a very strange issue, when I got a soci::rowset of the query result. I can use a for loop to print its values, but if I run the for loop again on the same rowset, the iterator got invalid.
Here is a simple test code:
            soci::rowset<person> rs = (sql.prepare << "SELECT * FROM testtable");

            // Loop through rowset using a range-based for loop
            std::cout << "first for loop" << std::endl;
            for (const auto& row : rs) {
                std::cout << row.id << std::endl;
            }
            std::cout << "second for loop" << std::endl;
            for (const auto& row : rs) {
                std::cout << row.id << std::endl;
            }

The result is: the first for loop can print the results, suppose I have two result items, and their ids will be printed. But the second for loop just did nothing, none of the items were returned.
The person class is very simple:
class person
{
public:
    int id;
    std::string name;
    int age;
};

namespace soci
{
    template<>
    struct type_conversion<person>
    {

        typedef values base_type;

        static void from_base(const values& v, indicator ind, person& row)
        {
            row.id =   v.get<int>("id", -1);
            row.name = v.get<std::string>("name", "");
            row.age =  v.get<int>("age", 0);

        }

        static void to_base(const person& row, values& v, indicator& ind)
        {
            v.set("id",   row.id);
            v.set("name", row.name);
            v.set("age",  row.age);

            ind = i_ok;
        }
    };
}

This looks very strange, that the rowset got invalid after the first loop.
The original question in-fact comes from the another issue, that I would like to convert the rowset to a std::vector<persion>, but it looks like the iterator got invalid if I run a for loop, see my test code here:
https://github.com/SOCI/soci/issues/1020#issuecomment-1650967359
Any one can explain this?
Thanks.

When reading the document: soci/docs/statements.md at master · SOCI/soci
It looks like the prepare is very special?
I mean the row object(instance) is only constructed when they are accessed in the for loop? After that, they just got destroyed?

OK, I just debugged for a while, I have change the definition of person class like below, and try to check how many time the constructor and the destructor got called.
class person
{
public:
    person()
    {
        std::cout << "person constructor" << std::endl;
    }

    person(const person & old)
    {
        id = old.id;
        name = old.name;
        age = old.age;
        std::cout << "copy constructor" << std::endl;
    }

    ~person()
    {
        std::cout << "destroy id = " << id << std::endl;
    }
public:
    int id;
    std::string name;
    int age;
};

The result is:  The constructor and the destructor only called once. And the person instance get reused in every for loop.
So, I see only one instance of person object.

When using rowset, you only get a convenience wrapper around statement, with which you can access data via an iterator. It is not a container for results that you can iterate over multiple times. You could more or less rewrite it as
Person person;
soci::statement stmt = (sql.prepare << "SELECT * FROM testtable", soci::into(person));
stmt.execute();

while (stmt.fetch()) {
    std::cout << person.id << std::endl;
}

A stmt.fetch() is exactly what's happening when advancing a rowset_iterator. When fetch() returns false, i.e. no more data is available, the iterator becomes invalid. Just like can't re-use stmt after the loop, you cannot re-use rowset's iterator.

Hi, @zann1x thanks for the explanation. I strongly suggest that your comments should be put in the document of soci. Because this will happens for all kinds of beginners like me. What do you think?
Another issue I see is that in your comments in this ticket (ORM and rowset · Issue #1020 · SOCI/soci), you try to use the move semantic, such as the std::vector<person> ret{std::make_move_iterator(rs.begin()), std::make_move_iterator(rs.end())};. But since there is only one instance of "Person" instance, I think the 'move" semantic is not necessary. What do yo think about this?
Thanks.

Hi, @zann1x thanks for the explanation. I strongly suggest that your comments should be put in the document of soci. Because this will happens for all kinds of beginners like me. What do you think?

That would surely be a great addition to the docs. Feel free to create a merge request for it!

Another issue I see is that in your comments in this ticket (ORM and rowset · Issue #1020 · SOCI/soci), you try to use the move semantic, such as
std::vector<person> ret{std::make_move_iterator(rs.begin()), std::make_move_iterator(rs.end())};

But since there is only one instance of "Person" instance, I think the 'move" semantic is not necessary. I mean the normal constructor like below is enough.
std::vector<person> ret{s.begin(), rs.end()};

What do yo think about this?

It is true that only a single instance of Person is created, but the move operation also ensures that all the individual variables of the struct aren't copied unnecessarily. For every fetched row, a value is copied into each every variable of Person in type_conversion::from_base. To get the values into the final std::vector, we would have to copy them again. Moving them instead and basically extracting them from the initial Person instance without additional memory allocations feels like the correct thing to do for me.

Hi, thanks for the explanation. Though I'm still not quite understand the move structure, I will try to learn it more.
About the document improvement, I will do it in the future. (I'm not a native English speaker, so writing some document is a bit hard for me, but I will try it later)


MetaNWAlgo Pipeline

Hello, I missed the conversation on google group about the Dec 9 deadline.
Evan, what are you working on presently? I had updated MetaNeedalmanWunchAlgo.py few weeks ago, it it complete now. It takes local assembly as input, and gives global assembly. But we need to give it correct input using Alex's file. I think you were writing some code for that. Let me know what part you are working on and/or you would want me to work on, so we can divide the work?
Best,
Aarti

Hi @aartijajoo88
I just noticed this! Sorry I missed this issue previously.
Alex beat us to re-writing this script as a perl script.
https://github.com/NCBI-Hackathons/Graph_Genomes_CSHL/tree/master/scripts


iOS Safari "Add to Home Screen" web-app fails to load PDF in helloworld example - status 206

Configuration:

Web browser and its version: Safari
Operating system and its version: iOS 9.3 (latest)
PDF.js version: latest

Steps to reproduce the problem:

Add <meta name="apple-mobile-web-app-capable" content="yes"> to <head> of helloworld example
Replace PDF with a larger PDF
View in iOS Safari, then hit Share and "Add to Home Screen"
Open web-app, PDF will fail to load;
Any code inside PDFJS.getDocument(pdfUrl).then(function(pdf) { will not run.

This works as expected in iOS Safari. When the page is saved to the home-screen, it stops working.
Using Safari's web inspector, in the real iOS Safari app, in the Network tab - the full PDF transfers with a status of 200 and it displays the full file-size.  When saved to the home screen, only 64kb of the PDF transfers and the status is 206.
I have tried using compatibility.js as I've read this is due to range requests, however the same problem persists.  Can anybody help me please?

Further information:
This isn't necessarily a bug with PDF.js.  I've just tested loading a large PDF into an iframe on iOS Safari and the same problem appears:  status 200 and a full-download in Safari,  but status 206 and a 64kb download when saved to the home screen.
Interestingly though, this works when using viewer.html.  If I add the web-app-capable meta tag to viewer.html, point it to a large PDF and save it to the home screen, it has no problem downloading the PDF file.  Does something exist in viewer.js that bypasses the way helloworld downloads the PDF?

Notice that in web/compatibility.js we disable HTTP range request. Only Safari has an issue with caching of the range requests, so see if setting disableRange=true will help your case (or use entire compabitility.js)

@rorysmorris notice that once you used range requests once they stay in cache. try to purge the cache and try using compatibility.js again.

Thanks for your reply @yurydelendik
I found that by forcing disableRange to true or false, by hacking the code in compatibility.js - this made no difference at all. I even reset my iPad back to factory settings to make sure it wasn't a caching issue.
I have no idea why, but the issue seemed to originate from my environment itself, when I upload the web-app I'm working on to an FTP server, the problem goes away.
I was serving up my web-app through MAMP on OSX, and accessing it on the iPad via my network IP address, i.e.  http://<IP_ADDRESS>:8888
Whether the problem is MAMP, my machine, my network, the iPad - I do not know.

From the description here it sounds like this problem is not related to PDF.js itself, but more related to a specific configuration. Therefore I don't think there is anything we can do about this on the PDF.js side. Closing for now, but we can reopen this if there is more information that the problem is really caused by PDF.js.

@rorysmorris I have the same problem. It never gets loaded. I've placed a console.log in the callback of PDFJS.getDocument(srcToPdf) but it never gets called. In safari, it loads fine.
Is iOS webapps really capable of loading web workers?I've tried setting disableWorker = true; and it says 'using fake worker' but it still tries to load the worker file. Is this correct behavior? What is 'disableWorker = true' supposed to do?

@loveaurell I managed to get this working in the end, the problem was due to the environment - when I uploaded to a proper web-server it all worked okay.
In your Network tab, do you see the PDF downloading as only 64kb? If so that's problem the same issue I had.

@rorysmorris I can't find I way to properly debug in "add-to-homescreen" mode. I'm trying to use safari on my macbook and debug the iphone through usb but somehow the network requests are not showing up correctly in "add-to-homescreen" mode. So it's hard to say if it starts to download the pdf file at all. Any advice on how to monitor the network calls is appreciated!

@loveaurell
I'd suggest downloading XCode which has an iOS Simulator you can run on your MacBook. Then, when you save your app to the home screen on the Simulator, and then launch the app from your home screen, a new menu item appears in Safari (when you go to the Develop menu item at the top, you'll see Simulator > App Name).
That is how I was working anyway.

@rorysmorris
I managed to monitor network traffic with simulator/safari and noticed that the downloaded size is indeed 64kB and it does not continue to download the other chunks. Also tried using a small 21kB file which downloaded fine. I really don't know what to do about this, since I've tried using both my own nginx-server as well as putting the pdf on dropbox and downloading from there with the same result. If the chunk size was configurable I could probably around it by setting it higher than 64kB.


DeepBBT : the outputs of modified Roberta don't have hidden states

Hello, there is a bug in RobertaL Model ouput  when I run deepbbt because it do not have hidden state even the config.output_hidden_states = True (Line439).
Then I found the Line 975 of deep_modeling_roberta.py only output a dict with logit rather than the original MskedLMOutput, which leads to the above bug.
When I try to directly replace the logit of MaskedLMOutput to fix it, it suggest the inner model also do not output hidden state. I would appreciate if you could help to fix that !

Thanks for your issue! I've updated the deep_modeling_roberta.py, please try again :)

Thank you very much :)  It works well for me.


wiki: 3.2-supported-list need update

Please answer these questions before submitting your issue.
What version of sky-walking are you using?
3.2.1
What version of your OS?
windows
What version of your JRE?
1.8
What company or project?
广州文尔
What did you do?
sky-walking 3.2.1 support nutz web/http, but not display in 3.2-supported-list yet

Based on release notes, nutz framework was supported in 3.2.1 . And 3.2-supported-list doesn't represent all components supported in 3.2.x, it is simple meaning the first version of 3.2 series.
Check here: https://github.com/OpenSkywalking/skywalking/releases/tag/v3.2.1

@wendal Can I close this?


Allow multiple bridges on a door account

Apologies. I accidentally opened this on the wrong repo. I meant to open it on my local repo first. Closing.


Ayush5042 patch 1

Hey @Ayush5042, for this repo, we only accept changes to the Contributors.js and Contributors.md files. If you want to add or update a feature, please first create an issue. We will review it and assign it to you accordingly.

btw, this repo is for learning open-source contribution. If you need help, you can follow the guide available in the Contributing.md file.


[FLINK-7204] [core] CombineHint.NONE

FLINK-3477 added a hash-combine preceding the reducer configured with CombineHint.HASH or CombineHint.SORT (default). In some cases it may be useful to disable the combiner in ReduceNode by specifying a new CombineHint.NONE value.
We don't currently have unit tests for this part of the code but I'm happy to discuss and add some.

Hi @greghogan, I think this is definitely a valuable improvement.
However, I looked at the plan of a query with a ReduceFunction with CombineHint.None and found that it still has a combine node:
[IN] -FWD-> [COMBINE(SORTED_REDUCE)] -HASH-> [REDUCE(SORTED_REDUCE)] -> [OUT]

I think we should

set combinerStrategy to DriverStrategy.NONE in ReduceNode for CombineHint.NONE.
change ReduceProperties.instantiate() with a check for combinerStrategy == NONE and not add a node for the combiner in that case.
add a test in ReduceCompilationTest similar to testGroupedReduceWithHint() to check that the resulting plan has no combine node.

Best, Fabian

@fhueske updated and manually verified. I had misinterpreted DriverStrategy to imply that SORTED_REDUCE would not use a combiner.

Hi @greghogan, the changes look good and the PR is ready to be merged.
If you like, you can simplify the test before merging by using field position keys (as in testGroupedReduceWithFieldPositionKey()) instead of a KeySelector. That would remove the key extractor and projector map operators and make the plan a bit easier to verify.

Test updated.

+1 to merge


Affected Scenario: 28 - Incorrect monster icons used

Affected Scenario(s): 28 Outer Ritual Chamber
Describe the bug
The Virtual Gameboard uses an incorrect monster icon. In the top room, there should be three Night Demon monsters. The Virtual Gameboard shows three Living Spirits instead. Living Spirits aren't actually in this scenario.
To Reproduce
Steps to reproduce the behavior:

Go to 'Scenario 28'
Click on 'the top door with the open door tool selected to reveal the next room'
See error (compare to official scenario book)

Browser : [Chrome]

Hi @Gritchu , sorry for the late reply on this - I'm hoping to get a fix for this into the next release, so it should be fixed in the next few weeks. This is much later than I'd normally want to fix a bug like this, but lots of things have come together to make releases a bit difficult at the moment... sorry for the delay :-/


SSR with Material-UI, styled-components and Next.JS

Problem description
I'd like to use Material-UI together with Styled-components and Next.JS. I setup my styled-components SSR according to this tutorial (https://www.styled-components.com/docs/advanced#server-side-rendering) and it works. But I can't make it work together with the example from your repository (https://github.com/callemall/material-ui/tree/v1-beta/examples/nextjs) due to it uses JSS. Do you know if it's possible to make it work with styled-components? Thanks!

Sorry, I can't help. I have never used styled-components. Let us know if you find a solution. Also you can try StackOverflow.

Related to this... is there any intention to research styled-components? I feel they would be a better fit for how MaterialUI is performing styling and theming. Of course, this is just a hunch, and I don't have any kind of POC that would validate this as a dependency.
[See the video](https://www.youtube.com/watch?v=jaqDA7Btm3c that explains/defends the use of styled-components

@dmt-13 have you ever figured out how to do this?

@dmbaranov sorry my bad. I meant to say @oliviertassinari. I updated it now.
@oliviertassinari Stackoverflow is not the right place for this as this is a compatibility issue.
See https://github.com/styled-components/styled-components/issues/1282 and https://github.com/styled-components/styled-components/issues/1146

styled-components interoperability with Material-UI is documented under this page: https://material-ui-next.com/guides/interoperability/#styled-components.


MIPS PLT Offset off by one entry

For mipsel binaries there seems to be an issue with loading only the plt (the rest of the binary seems fine) where plt entries resolve by the entry adjacent.
Tested on 3.5.4526 Personal and 3.6.4665-dev

The following are some MIPS binaries from a device I was doing research on
mips_binaries.zip

Can you be more specific as to which exhibit this problem? A quick look through several and they all look fine to me unless I'm misunderstanding the issue:

A quick look doesn't show any sections with "plt" in the name either if that's required?
>>> import binaryninja
>>> import glob
>>> binaryninja.disable_default_log()
>>> for file in glob.glob("*"):
...   with binaryninja.load(file, update_analysis=False) as bv:
...     print(bv)
...     for section in bv.sections:
...       if "plt" in section:
...         print(f"\t{section}")
...
<BinaryView: 'cmd_cron', len 0x2cd>
<BinaryView: 'ip6tables-save', start 0x400000, len 0x1a084>
<BinaryView: 'updatednsip', start 0x400000, len 0x19924>
<BinaryView: 'net-scan', start 0x400000, len 0x13814>
<BinaryView: 'hostapd', start 0x400000, len 0x80984>
<BinaryView: 'brctl', start 0x400000, len 0x16fd4>
<BinaryView: 'ssmtp', start 0x400000, len 0x1a6b4>
<BinaryView: 'aclctl', start 0x400000, len 0x119f4>
<BinaryView: 'dhcp6c', start 0x400000, len 0x3cd04>
<BinaryView: 'ntpst', start 0x400000, len 0x13214>
<BinaryView: 'update_smb', start 0x400000, len 0x135b4>
<BinaryView: 'miniupnpd', start 0x400000, len 0x21454>
<BinaryView: 'ez-ipupdate', start 0x400000, len 0x1bd84>
<BinaryView: 'chat', start 0x400000, len 0x15a04>
<BinaryView: 'ppp-nas', start 0x400000, len 0x13c54>
<BinaryView: 'proftpd', start 0x400000, len 0xa54f4>
<BinaryView: 'ripngd', start 0x400000, len 0x53d24>
<BinaryView: 'iwspy', len 0x0>
<BinaryView: 'apply_config', len 0x56e>
<BinaryView: 'net-dump', start 0x400000, len 0x11854>
<BinaryView: 'crond', len 0x0>
<BinaryView: 'ebtables', start 0x400000, len 0x10ca4>
<BinaryView: 'radvdump', start 0x400000, len 0x13f64>
<BinaryView: 'zebra', start 0x400000, len 0x51034>
<BinaryView: 'dns-hijack', len 0x107>
<BinaryView: 'wget_netgear', len 0x1f2>
<BinaryView: 'update_user', len 0x357>
<BinaryView: 'aclhijackdns', start 0x400000, len 0x11be4>
<BinaryView: 'uhttpd', start 0x400000, len 0x20ca4>
<BinaryView: 'px5g', start 0x400000, len 0x1fa74>
<BinaryView: 'wpa_cli', start 0x400000, len 0x1d014>
<BinaryView: 'iwlist', len 0x0>
<BinaryView: 'inetd', start 0x400000, len 0x122b4>
<BinaryView: 'select_partition', len 0x2c0>
<BinaryView: 'iwgetid', len 0x0>
<BinaryView: 'iwpriv', len 0x0>
<BinaryView: 'potval', start 0x400000, len 0x13214>
<BinaryView: 'acl_update_name', start 0x400000, len 0x10c44>
<BinaryView: 'tc', start 0x400000, len 0x4cd24>
<BinaryView: 'net-wall', start 0x400000, len 0x27c24>
<BinaryView: 'dhcp6ctl', start 0x400000, len 0x13d04>
<BinaryView: 'dev-scan', start 0x400000, len 0x10ff4>
<BinaryView: 'iwconfig', start 0x400000, len 0x20954>
<BinaryView: 'usbled', start 0x400000, len 0x11014>
<BinaryView: 'ntpclient', start 0x400000, len 0x17ec4>
<BinaryView: 'utelnetd', start 0x400000, len 0x129a4>
<BinaryView: 'ip6tables', start 0x400000, len 0x199f4>
<BinaryView: 'stamac', start 0x400000, len 0x13214>
<BinaryView: 'ipp', start 0x400000, len 0x12664>
<BinaryView: 'telnetenable', start 0x400000, len 0x13e14>
<BinaryView: 'wpa_supplicant', start 0x400000, len 0x83c54>
<BinaryView: 'usb_cfg', start 0x400000, len 0x15f34>
<BinaryView: 'minidlna', start 0x400000, len 0x5b2c4>
<BinaryView: 'net-cgi', start 0x400000, len 0xdfe54>
<BinaryView: 'dhcp6s', start 0x400000, len 0x39f24>
<BinaryView: 'noip2', start 0x400000, len 0x1f5c4>
<BinaryView: 'vol_id', start 0x400000, len 0x13c14>
<BinaryView: 'ip6tables-restore', start 0x400000, len 0x1ad94>
<BinaryView: 'chroot', len 0x0>
<BinaryView: 'restart_ap_udhcpc', len 0x220>
<BinaryView: 'iptables', start 0x400000, len 0x19cd4>
<BinaryView: 'potd', start 0x400000, len 0x13214>
<BinaryView: 'detwan', start 0x400000, len 0x19594>
<BinaryView: 'net-disk', start 0x400000, len 0x12bb4>
<BinaryView: 'acld', start 0x400000, len 0x13504>
<BinaryView: 'start_br_udhcpc', len 0x314>
<BinaryView: 'ip', start 0x400000, len 0x43ca4>
<BinaryView: 'pppd', start 0x400000, len 0x91964>
<BinaryView: 'nmbd', len 0x127b44>
<BinaryView: 'hostapd_cli', start 0x400000, len 0x15db4>
<BinaryView: 'ripd', start 0x400000, len 0x16fe4>
<BinaryView: 'radvd', start 0x400000, len 0x257c4>
<BinaryView: 'lld2d', start 0x400000, len 0x2b9b4>
<BinaryView: 'dsyslog', len 0x4e2>
<BinaryView: 'smbd', len 0x34c184>
<BinaryView: 'phddns', start 0x400000, len 0x28e04>
<BinaryView: 'dnsmasq', start 0x400000, len 0x33034>
<BinaryView: 'ntgrddns', start 0x400000, len 0x20074>

We've had a discussion internally about this and come to a few conclusions.
First, yup, there's still a bug with the improved recognition that we need to fix. Ironically, for this particular binary (not sure about others) though you shouldn't ever need these to be fixed since the PLT isn't used at all.
The bigger issue is probably that when you want to find the cross-references for an import you should be navigating to the GOT entry. This means if you search, say, "memcpy" in the sidebar, you'd want to hit the second entry, not the first one. Likewise, if you use "g" to navigate to the right function, you'd want to navigate to the one in the .got section (which is shown in the drop-down). We're discussing some ways we can improve the default behavior so you're less likely to run into this in the first place, but let me know if either of those work for you!

Oh I've not had issues following cross references. (I can just click an imported symbol as it appears in the IL pane and follow the references) It's not too impactful of a bug, I just thought I'd report it anyway. It only makes following cross references via the plt a bit wonkey which is what my and I'm sure some other people's brains default to when they want to just follow some imported symbol off rip via the plt.

That makes sense, just making sure it wasn't blocking anything.

Note dump before I forget stuff:
The enhancements that corrected the the previous issue were activated here, but the type of our RTL_Resolve() inserted at the start of the GOT was not propagating. Peter recognized this was a section semantics issue after seeing the odd placement of RTL_Resolve(), about 8 bytes prior to the GOT:

Since it's not in a section, it doesn't adopt the ReadOnlyDataSectionSemantics:
>>> bv.get_sections_at(0x115648)
[<section .got_recovered_115648: 0x115648-0x116bf0>]
>>> bv.get_sections_at(0x115648)[0].semantics
<SectionSemantics.ReadOnlyDataSectionSemantics: 2>

I learned it is not necessary to have sections at all. You can just have program headers (segments) instruct the loader to map the necessary bits of the file to memory, and then you can communicate the location of important parts via the program header types and their contents. For example, the location of the GOT is communicated with a program header with type PT_DYNAMIC, which then has a dynamic entry type DT_PLTGOT:

When the GOT is specified this way, Binja synthesizes the ".got_recovered_XXX" section. Note the address in .got_recovered_115648 disagrees with the ELF file who specifies 0x115640. So this reduces to fixing got recovery in elfview.

Minor correction: view-elf was calculating recovered GOT section by collecting all relocations and inferring a sort of bounding box around them. This would miss the very first pointer in the GOT, where RTL_Resolve() is placed.
The area shown in the screenshot by fuzyll now looks like:

Fixed in: https://github.com/Vector35/view-elf/commit/582360669c5953d71b94940d944c3fe817a562bb and https://github.com/Vector35/binaryninja/commit/ca11e60e10d878a86a5cb6248a7d043fca580e82

There's still a problem with typing there - those RTL_Resolve calls should just be resolved to extern calls, otherwise every external function takes 0 parameters and doesn't return.
RTL_Resolve is part of the loading process and provided by the loader, it's not actually runtime-dynamic even though it looks that way. If I remember correctly the symbol index there is the index into the dynamic symbols table, which you could use to statically resolve which extern it should call.


To address issue #2502 which I opened...

Added support for disabling the randomization of IP addresses in the response to DNS hostname lookup, as well as support for sorting the IP addresses in ascending or descending order.
Full backwards compatibility is maintained.
Performance penalty is 1 map lookup + 1 if statement with simple boolean equality check + 1 switch statement + sorting algorithm:

NONE: constant
ASCENDING: O(nlogn)
DESCENDING: O(nlogn)
RANDOM: O(n)

3 new functions are added to Resolver interface to support sort order:

SetIPSortOrderDefault(int)
SetIPSortOrderSpecific(string, int)
ClearIPSortOrderSpecific(string)

3 new function implentations are added to support the interface:

func (r *resolver) SetIPSortOrderDefault(ipSortMethod int)
func (r *resolver) SetIPSortOrderSpecific(hostname string, ipSortMethod int)
func (r *resolver) ClearIPSortOrderSpecific(hostname string)

1 new type is added to specify sort order:

NONE
ASCENDING
DESCENDING
RANDOM

Signed-off-by: Ian Morris Nieves<EMAIL_ADDRESS>

Note we have migrated this codebase over to github.com/moby/moby/libnetwork.
We are not accepting PR's on this repo anymore except for backports to be included in moby 20.10

The feature request is reasonable, however the implementation is incomplete. And I would like to investigate alternate approaches which do not depend on stateful configuration. I have transferred the issue to moby/moby so we don't forget about it.


Simple bash script to clean homie device from MQTT (Mosquitto)

Works great for me to get rid of devices in MQTT, which are not active anymore.
Maybe it is helpful for someone else...

STARTING :: Checking for valid topics one after the other....
0 Checking for Topic ....
homie/a020a617ea1d/temperature/$type >> sending empty payload to topic with retained flag
1 Checking for Topic ....
homie/a020a617ea1d/temperature/$properties >> sending empty payload to topic with retained flag
2 Checking for Topic ....
homie/a020a617ea1d/temperature/unit >> sending empty payload to topic with retained flag

#!/bin/bash
#
# Simple Script to remove all reatined messages for a homie device

echo
echo "Clearing for device $1 all messages, last clean will block kill with Ctrl+C when nothing happens anymore."
echo
#mosquitto_sub -v -h <IP_ADDRESS> -t "homie/$1/#" -C $2
echo "STARTING :: Checking for valid topics one after the other...."

MAX_TOPICS_TO_CHECK=100

for c in `seq 0 $MAX_TOPICS_TO_CHECK`
    do
    echo "  $c Checking for Topic ...."
    for i in `mosquitto_sub -v -h <IP_ADDRESS> -t "homie/$1/#" -C 1|cut -d' ' -f1`
        do
            # echo "mosquitto_pub -h <IP_ADDRESS> -r -n -t \"`echo $i | sed s#'\\$'#'\\\\\\\$'#g`\""
            echo "    $i >> sending empty payload to topic with retained flag"
            mosquitto_pub -h <IP_ADDRESS> -r -n -t "`echo $i | sed s#'\\$'#'\\\$'#g`"
        done
done

I like this, ill include it in my homie management project

There is also this project: https://github.com/contactless/mqtt-tools, which offers a similar function

@andreasgruener if you want, make a little mark down page about this script and ill include it in the docs ?
how to use and the code ?

@luebbe , worth also including in the docs ? maybe write a little doc page for it (gist, like ^) ?

@timpur do you mean: "add a reference to the tool to the homie docs? or "Please write some documentation about this tool"?.
It's not my tool, the "docs" there are only the command line help and that's it. It's a bit trial and error when you use the scripts...

@timpur sorry for the delay!
I added some comments to the code and as comment on the gist

On my to do list.

I added reference within trouble shooting section.
Thanks @andreasgruener for the script.


Fixes #2479

Added textual definitions and references.
Updated logical definitions

Please give the PR a more informative name (will be used in release report). Ta.


[Enchancement] Gradle Library

Just making sure that people know the  Gradle library module is on the agenda as an enhancement.

This is done now :)
https://github.com/yahoo/android-range-seek-bar#setup-as-gradle-dependency


Enable scorlling when prompt response is being generated

Requirements

[X] I have searched the issues of this repository and believe that this is not a duplicate
[X] I can confirm this feature request is not already implemented in the app

Description
As prompt responses can be quite fast, I'd suggest letting the user scroll while the prompt is being generated. This is for instance possible with the official ChatGPT app. It's a quality of life improvement since, if a very large rresponse gets generated, one can not read until the LLM has finished with it's response.
Possible solution
Not sure how it works but possibly disable auto scrolling with new content?
Additional context
No response

Hi there 👋
Sorry for not answering for so long. I looked into it and as far as I can tell, it's easily possible - well kinda. In one of the recent commits, I already implemented most of the things that have to be done. One problem I'm currently facing is that the underlying library I'm using has no option to disable the auto scrolling. I'll see if there's anything I can do, and I'll keep you updated about it.
Thanks for reporting

Just thought I'd drop a +1 for this. Every time I ask a question i have to wait for the response to finish, then scroll back up and start reading it from the top again. Disabling auto scroll would be a big improvement!


[WIP] Router

I'll write the explanation up tomorrow, I just wanted to get this up before I went to bed so people can take a look if they wanted to.
I haven't implemented several pages, but the majority are in.
Also enables object rest spread and upgrades buble (meaning that the async/await support will need to be disabled).

Take a look at #1341. that is probably a merge conflict. If not, that code needs to be moved to the correct place.

Also FYI, though I haven't got this far yet, I'm probably going to recommend moving all the stuff in /pages/ alongside layout.


Some API suggestions:)

Frist thanks for sharing, some suggestions below:

Provides an extension definition for Kotlin ?
Provides an extension for Vertx Function?
How about keep the Cffu a clean room, and not extends the Future and CompletionStage directly, but provides methods like fromCompletionStage and toCompletionStage.
then we can have method like failed instead of failedStage.

Provides an extension definition for Kotlin ?

Yes! Kotlin extension methods is amazing, a MUST have. 🎉

Provides an extension for Vertx Function?

VertX is another (full-fledged) concurrency framework.
Maybe keep cffu only concern CompletableFuture,
and be a tiny sidekick lib of CompletableFuture is ok.

How about keep the Cffu a clean room, and not extends the Future and CompletionStage directly, but provides methods like fromCompletionStage and toCompletionStage.

Good design idea!
The  that Cffu extends the Future and CompletionStage instead provides methods like fromCompletionStage and toCompletionStage,

@He-Pin
this commit(66e06dcc5e5507261bf0a02419e13d75abc5956a) has added a Kotlin extension method:
public <T> Cffu<T> asCffu(CompletionStage<T> stage)

Any suggestions about Kotlin extension methods is welcome~ ❤


chore: croc-hunter-jenkinsx to 0.0.76

chore: Promote croc-hunter-jenkinsx to version 0.0.76

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by:
To fully approve this pull request, please assign additional approvers.
We suggest the following additional approvers:
If they are not already assigned, you can assign the PR to them by writing /assign  in a comment when ready.
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment