default Content-Type param defined by the user

closing this PR because i merged it into the branch feature_default_content_type which will be merged as soon as the tests pass.


Reorganize entity packages

This does the following class movements:
Creates two subpackages on n/m/entity/mob, hostile and passive respectively.
Moves contents of n/m/entity/passive -> n/m/entity/mob/passive since all passive entities are mobs.
Moves all hostile entities from n/m/entity/mob to n/m/entity/mob/hostile
Move RaiderEntity to n/m/entity/mob/hostile
Move CrossbowUser to n/m/entity/ai like RangedAttackMob
Move DamageUtil to n/m/entity/damage
Move other misc interfaces to most relevant packages. Since Flutterer is used by Bees and Parrots which are passive, the Flutterer interface is now in n/m/entity/mob/passive
Collapse n/m/entity/decoration/painting package into parent n/m/entity/decoration package.
Considerations:
Should n/m/entity/boss be moved into n/m/entity/mob/boss or n/m/entity/mob/hostile/boss
Should EnderDragonFight go to n/m/server/world since it's primary use is on a server world.
TODO:
Write into readme how entity packages are supposed to be organized.

Closing for later date near end of snapshots


Airsonos fails on start

Problem/Motivation
Can't start airsonos
Expected behavior
Airsonos starts...?
Actual behavior
Log shows (log_level info but debug isn't showing much more):
[s6-init] making user provided files available at /var/run/s6/etc...exited 0.
[s6-init] ensuring user provided files have correct perms...exited 0.
[fix-attrs.d] applying ownership & permissions fixes...
[fix-attrs.d] done.
[cont-init.d] executing container initialization scripts...
[cont-init.d] 00-banner.sh: executing...

-----------------------------------------------------------
 Hass.io Add-on: AirSonos
 AirPlay capabilities for your Sonos (and UPnP) devices.
-----------------------------------------------------------
 Add-on version: 2.1.1
 You are running the latest version of this add-on.
 System: Debian GNU/Linux 10 (buster)  (amd64 / qemux86-64)
 Home Assistant version: 0.99.3
 Supervisor version: 189
-----------------------------------------------------------
 Please, share the above information when looking for help
 or support in, e.g., GitHub, forums or the Discord chat.
-----------------------------------------------------------
[cont-init.d] 00-banner.sh: exited 0.
[cont-init.d] 01-log-level.sh: executing...
Log level is set to INFO
[cont-init.d] 01-log-level.sh: exited 0.
[cont-init.d] airsonos.sh: executing...
[cont-init.d] airsonos.sh: exited 0.
[cont-init.d] done.
[services.d] starting services
[services.d] done.
[16:44:31] INFO: Starting the AirSonos server...
[16:44:31.890673] main:1293 Starting airupnp version: v<IP_ADDRESS> (Aug 20 2019 @ 09:34:28)
[16:44:31.893120] Start:1011 UPnP init failed: -208
[16:44:31.893175] main:1334 Cannot start
[16:44:32] INFO: Starting the AirSonos server...
[16:44:32.892119] main:1293 Starting airupnp version: v<IP_ADDRESS> (Aug 20 2019 @ 09:34:28)
[16:44:32.893189] Start:1011 UPnP init failed: -208
[16:44:32.893241] main:1334 Cannot start

last 4 lines will repeat every second
Steps to reproduce

system: hass.io on debian buster with Docker version 19.03.2, build 6a30dfc
The system has a vlan on eth0 where also sonos is located
Installed and deinstalled plex-plugin so that port 1900 should be available.
the debian system was running minissdpd - I removed it. So nothing is running on port 1900: netstat -tulpn | grep 1900 is empty.
tried de-/reinstall without success.

Proposed changes

Can you post the supervisor log please?

What I found with docker logs hassio_supervisor:
19-10-05 14:41:52 INFO (MainThread) [hassio.addons] Create Home Assistant add-on data folder /data/addons/data/03b5fe11_airsonos
19-10-05 14:41:52 INFO (SyncWorker_7) [hassio.docker.interface] Pull image hassioaddons/airsonos-amd64 tag 2.1.1.
19-10-05 14:42:04 INFO (MainThread) [hassio.addons] Add-on '03b5fe11_airsonos' successfully installed
19-10-05 14:43:33 INFO (SyncWorker_6) [hassio.docker.addon] Start Docker add-on hassioaddons/airsonos-amd64 with version 2.1.1
19-10-05 14:43:33 INFO (MainThread) [hassio.api.security] /host/info access from 03b5fe11_airsonos
19-10-05 14:44:18 INFO (SyncWorker_8) [hassio.docker.interface] Stop addon_03b5fe11_airsonos application
19-10-05 14:44:19 INFO (SyncWorker_8) [hassio.docker.interface] Clean addon_03b5fe11_airsonos application
19-10-05 14:44:31 INFO (SyncWorker_6) [hassio.docker.addon] Start Docker add-on hassioaddons/airsonos-amd64 with version 2.1.1
19-10-05 14:44:31 INFO (MainThread) [hassio.api.security] /host/info access from 03b5fe11_airsonos
19-10-05 14:44:38 INFO (SyncWorker_4) [hassio.docker.interface] Stop addon_03b5fe11_airsonos application
19-10-05 14:44:38 INFO (SyncWorker_4) [hassio.docker.interface] Clean addon_03b5fe11_airsonos application

Not so much I guess...

From what I can see the -208 is a generic network error (not specifically a bind issue).  The addon runs on the docker host network, so its likely a Docker or OS based issue.  Do you have anything running which would prevent access to the ports?  Can you also try another addon that runs on the host network such as Node-Red or SSH.

I've installed Node-Red... which is working. Which part of SSH should I try? ssh-serveror SSH & Web Terminal? Both?

Hassio uses network-manager for a few things, one of the latest changes is to get your local configured DNS servers, so yes I would suggest using it.  I'm actually going to close this out as I doubt its an issue with the addon, but actually local config.  Please feel free to join Discord/Forums to help troubleshoot further.


upgrade

I own version 1.4 of the Bootstrap Starter Kit. It is supposed to lifetime upgrades, but I don't see how to get the latest version without paying.

We don't have an official Bootstrap starter kit—you'll have to ask somewhere else for that.


test(dtslint): add skipWhile

Description:
This PR adds dtslint tests for skipWhile.
While doing this I noticed some inconsistency with the predicate parameter in comparison to other operators. For example some of them also have a source as 3rd parameter, and have thisArg parameter. Or the filter operator also has a type guard. I don't know if this is intended, just wanted to point this out 😄
Related issue (if exists): #4093

Coverage decreased (-0.2%) to 96.809% when pulling d0463cd8e281028fdc5b0fd23115bee77603918f on timdeschryver:pr/dtslint-skip-while into 03ee88cce186ae67ab871383f63e4d746e11ea41 on ReactiveX:master.


SearchableMixin wont work like this with more than one class

I tried to use the events for my own purposes and I came across the following issue with the after_commit method:
@classmethod
def after_commit(cls, session):
    [...]
    session._changes = None

The event is triggered and hits the first after_commit method. This first after_commit will reset the session._changes to None. So if you have have a second class which is listening on the after_commit event (e.g. is using the SeachableMixin) and queries the _changes, it will run into an error because _changes is None at that point. At least that happened in my version.
Other than that thanks for the great tutorial. I'd love to see a bit more about testing of the routes.

I did not consider this case, you are correct. the session._changes attribute needs to independently store changes for multiple classes. I will fix it and update the tutorial.

One more thing I came across which is worth mentioning during a rewrite is: You rely on  session.deleted to be populated when a delete happens. However, you can delete entities without the  session.deleted being filled (both before and after commit). (No clue if it is a bug with sqlAlchemy or not)
This one will leave the  session.deleted empty, yet deletes the entity; meaning  -- in case of the tutorial -- the search index is not deleted.
Model.query.filter_by(...).delete()
This one will fill the  session.deleted
Entity = Model.query.filter_by(...).one()
db.session.delete(Entity)

So one should at least be aware of this so that one can write really good test making sure one does not anything stupid...

@coolkau Yes, the solution that I provide integrates the full-text search with the SQLAlchemy ORM, as shown throughout the tutorial. Any database operations that do not go through the scoped session will not work for this solution.

@miguelgrinberg IMHO, it would be worth writing a comment about exactly that in the tutorial, as I dont really know the difference (until that issue). I dont think there is actually a single delete in the tutorial. So hobby coders like me will go and find the next best solution on stackoverflow, and wonder why it is not working much later... so it would be nice to have a warning... but up to you, its your tutorial after all.


Fast reboot not blocked on change of garded resources

If we clear gard records, we still do a fast-reboot - which may not get the garded hardware back.

I'm not really sure how we're supposed to fix this since we don't really parse the guard partition at all in skiboot. Suppose we could force a full re-IPL if the host does any flash writes? Seems a bit heavyweight.

oohal<EMAIL_ADDRESS>writes:

I'm not really sure how we're supposed to fix this since we don't
really parse the guard partition at all in skiboot. Suppose we could
force a full re-IPL if the host does any flash writes? Seems a bit
heavyweight.

Maybe any flash writes to the GUARD partition?
I don't have a good answer unfortunately, probably why it's sat as a
github issue unaddressed for so long :)
--
Stewart Smith
OPAL Architect, IBM.


Refactor: readability fixes

No need to set a new string to empty, that's the default.

Fix cases where the interface and the implementation had different parameter names (adjust implementation, replace is superior to str, etc.)

Thanks, Miklós!


Handle when test runner exits with no JSON

In particular, if the failing test is segfaulting, then pytest will not get an opportunity to write out JSON

if the process crashes then all bets are off, we can't know we found a correct pair of failing tests without knowing if the test failed or not

Hang on! I think it's still useful to assume that the test failed when there's no JSON. It's a very simple patch (three lines). Are you interested in seeing the PR? lol

no thanks

in particular the lack of json does not mean the test failed, it could be all sorts of other failure modes (and user errors)
without the json it is flying blind and can't just make assumptions like you're suggesting

It's alright, I'll just put the change in a fork :)


add uninassau

Add uninassau

@JonasBernardino First stage of verification passed, official domain for Faculdade Mauricio de Nassau is prof.uninassau.edu.br as far as I verified. This is just an informational note. Pull request is still in the review. I appreciate your patience while waiting for the final decision.

@JonasBernardino Thank you for your patience while waiting for a review and processing your request. Pull request merged.
Please start requesting the licenses in about an hour, to let the changes to propagate through our system.


Force rollout of deployed IOC

https://helm.sh/docs/howto/charts_tips_and_tricks/#automatically-roll-deployments
Fixes epics-containers/edge-containers-cli#122

Im happy that this solves our problem. The following test was done to reproduce the issue:

deploy-local vanilla bl01c-ea-test-01
Edit ioc.yaml to replace  pv: bl01c:IBEK:B with  pv: bl01c:IBEK:BEHAVE
deploy-local edited bl01c-ea-test-01
kubectl describe cm bl01c-ea-test-01-config shows  pv: bl01c:IBEK:BEHAVE
ec logs bl01c-ea-test-01 shows dbpf "bl01c:IBEK:B" "2.61"  -> Conflict here

I test the changes as follows:

Clone a local copy of ec-helm-charts and checkout marcelldls-patch-1
Specify ioc-instance path to ec-helm-charts/Charts/ioc-instance in helm/shared/Chart.yaml
deploy-local edited bl01c-ea-test-01
kubectl describe cm bl01c-ea-test-01-config shows  pv: bl01c:IBEK:BEHAVE
ec logs bl01c-ea-test-01 shows pv: bl01c:IBEK:BEHAVE


CmdStan on IBM Power-9 systems

Summary:
CmdStan fails to compile models on IBM Power-9 systems.
This issue was previously discussed in the following Stan discourse post, https://discourse.mc-stan.org/t/error-running-cmdstan-on-ibm-power9-system/22060. A step-by-step solution is given in the PDF document attached to the first post - possibly worth adding to the Stan documentation?
Description:
Trying to compile a Stan model on an IBM Power-9 system results in the following error:
make ../src/models/deaths_and_111_calls

--- Translating Stan model to C++ code ---
bin/stanc  --o=../src/models/deaths_and_111_calls.hpp ../src/models/deaths_and_111_calls.stan
/bin/bash: bin/stanc: cannot execute binary file
make: *** No rule to make target `../src/models/deaths_and_111_calls.hpp', needed by `../src/models/deaths_and_111_calls'.  Stop.

CmdStan is distributed with pre-compiled stanc binary files that were built for x86 architecture, not Power-9. Because of this, we must build and configure the Stan compiler by hand.
Reproducible Steps:
Compile any Stan model on an IBM Power-9 system.
Current Output:
See description,
Expected Output:
A compiled model.
Additional Information:
NA.
Current Version:
Initially v2.23.0, persists v2.27.0

Would this help https://github.com/stan-dev/stanc3/issues/926

the same issue is true for ARM based processors

Thi is false. For ARM we have a separate tarball in the last 2 releases.

Would this help stan-dev/stanc3#926

the same issue is true for ARM based processors

This is false. For ARM we have a separate tarball in the last 2 releases.

My bad, updated post.

Yep, as mentioned in the linked post I'll be updating our release process to automatically build binaries for powerpc systems, so this will handled automatically soon

@mjcarter95 powerpc support should be available now. You can either clone the develop branch of the cmdstan repo and build, or you can just download the stanc binary from the nightly release in the stanc3 repo

@andrjohns Thank you. Just got round to trying this: I can now compile cmdStan and Stan models using Power 9, but am unable to sample from the model.
I am using the model in the previously mentioned Discourse post and gcc 10.2.0
/tmp/slurmd/job249793/slurm_script: line 11: 78191 Segmentation fault      src/models/deaths_and_111_calls sample num_samples=1000 num_warmup=1000 algorithm=hmc engine=nuts max_depth=10 stepsize=0.01 adapt delta=0.8 data file=data/model_input/nhs_sheffield_ccg.data.json init=data/model_inits/nhs_sheffield_ccg/init1.json output file=output/model_fits/bede/samples1.csv

@andrjohns Thank you. Just got round to trying this: I pulled the develop branch of cmdStan and am now able to compile both cmdStan and Stan models. However, when running the model "Segmentation fault" is output to the terminal with no additional information. Any ideas what the cause might be?
I am using gcc 10.2.0 and the model outlined here
./src/models/deaths_and_111_calls sample num_samples=1000 num_warmup=1000 algorithm=hmc engine=nuts max_depth=10 stepsize=0.01 adapt delta=0.8 data file=data/model_input/nhs_sheffield_ccg.data.json init=data/model_inits/nhs_sheffield_ccg/init1.json output file=output/model_fits/bede/samples1.csv
method = sample (Default)
  sample
    num_samples = 1000 (Default)
    num_warmup = 1000 (Default)
    save_warmup = 0 (Default)
    thin = 1 (Default)
    adapt
      engaged = 1 (Default)
      gamma = 0.050000000000000003 (Default)
      delta = 0.80000000000000004 (Default)
      kappa = 0.75 (Default)
      t0 = 10 (Default)
      init_buffer = 75 (Default)
      term_buffer = 50 (Default)
      window = 25 (Default)
    algorithm = hmc (Default)
      hmc
        engine = nuts (Default)
          nuts
            max_depth = 10 (Default)
        metric = diag_e (Default)
        metric_file =  (Default)
        stepsize = 0.01
        stepsize_jitter = 0 (Default)
    num_chains = 1 (Default)
id = 1 (Default)
data
  file = data/model_input/nhs_sheffield_ccg.data.json
init = data/model_inits/nhs_sheffield_ccg/init1.json
random
  seed =<PHONE_NUMBER> (Default)
output
  file = output/model_fits/bede/samples1.csv
  diagnostic_file =  (Default)
  refresh = 100 (Default)
  sig_figs = -1 (Default)
  profile_file = profile.csv (Default)
num_threads = 1 (Default)

Segmentation fault

Are you able to compile and run the bernoulli example model thats included with cmdstan?

Yes, the bernoulli example compiles and runs.

That's a good sign, at least. Can you share your model code and the .hpp that is generated by stan? That way I can check that the same c++ is being generated across systems.
Im assuming that the model compiled and sampled under the stanc3 that you built locally? Can you try using the stanc binary that you built previously, but with the cmdstan that you just cloned? So we can check whether the segfault is due to stanc3 or due to changes in cmdstan.

I've uploaded the model code and .hpp to Google Drive, hopefully you can access them here, please let me know if you have any trouble accessing them.
I removed the stanc3 that was built locally, so the model should have been compiled using stanc3 that is referenced in the develop  branch.

Alright, the generated .hpp is identical to what I get locally, so this might not be a stanc3 issue. Were you able to successfully run the model when using the locally-built stanc?
Can you share some data that reproduces the issue?

Alrighty, I also get a segfault with that data, so this is a cmdstan issue not stanc3 (phew for the multiarch). I'll start digging into this now

It looks like the segfault is related to your initial values, because the model runs fine when they're not included. Odd.

@mjcarter95 was this issue resolved?


List reorder button styling

The list reorder button can't be styled. When I add a class like .button-positive, the .item-reorder .button.icon class still overwrites the color and background.

Greetings @markdegrootnl!
My sensors indicate that you need to update your issue through our custom issue form. We are now requiring all issues to be submitted this way, to ensure that we have all of the information necessary to fix them as quickly as possible.
Click Here To Update Your Issue
I will have no choice but to close this issue if it is not resubmitted through the form. Please fill out the rest of the form, so that I may use my friendly robot powers to assist you.
Thank you!

so it is... good catch
http://codepen.io/perrygovier/pen/xGVdqQ

Hello all! I am going to close this as as it being by design. Thanks for using Ionic!


Increment version to 1.6.0

This PR increments the various version identifiers in the codebase to version 1.6.0

Accidentally created on this repo. Will be closed now.


Added Bridge Loot Scripts

Connected Previous Loot Scripts Directly To Bridge
Added Air Exposure Slider
Updated README
Drawbacks:
No Auto Author Comment
JSON is not Formatted
Upsides:
Direct Access from Bridge
Two Clicks
Less Messy

Please Ignore By Github Struggling xD


Genome browser bug

Sequence search does not work in the genome browser.

Fixed in https://github.com/PATRIC3/p3_web/pull/296/commits/e2eb8258994639b6f3d27116e2b694ba9eace9b5


Fix renovate extends syntax to include balena-io config

Change-type: patch
Signed-off-by: Kyle Harding<EMAIL_ADDRESS>

@balena-ci I self-certify!


Modify timeout for ssh

Since adding ap-southeast-4 regions, those instances are timing out. Trying to see if this is the reason

Canceling CI in order to save resources.


fix: App Service Plan shortname aligned to CAF

BUG FIXES:

App Service Plan shortname aligned to CAF

This should fix #90


Absolute fee interface

Result of build_tx should contain information (size + satisfaction weight) for users to dynamically calculate absolute fee, given a fee/vb.

Added

psbt::get_weight which uses desc & psbt to get full satisfaction weight
fee::get_absolute_fee which uses weight and fee_rate to get absolute fee value

Updated:

psbt::build to use fee_absolute instead of fee_rate


fix: context is dropped in v3.1

Describe the bug
Since v3.1(.2) the Nuxt context is dropped supposedly due to some async functionality. This leads to errors like [nuxt] [request error] [unhandled] [500] router.currentRoute.value is undefined, see reproduction. v3.0.1 works fine.
Remind me of https://github.com/nuxt/nuxt/issues/23074.
Reproduction
https://stackblitz.com/edit/github-fcnmuy?file=package.json
System / Nuxt Info
- Operating System: Linux
- Node Version:     v18.18.0
- Nuxt Version:     3.8.0
- CLI Version:      3.9.1
- Nitro Version:    2.7.2
- Package Manager<EMAIL_ADDRESS>- Builder:          -
- User Config:      devtools, modules
- Runtime Modules<EMAIL_ADDRESS>- Build Modules:    -

Thanks! Please try 3.1.3


fix beam search script

Description
The following run was broken.
python beam_search_generator.py --bos I think --lm standard_lstm_lm_200
This PR fixes the issue by adding state_info to language model while preserving existing behaviors.
Checklist
Essentials

[x] Changes are complete (i.e. I finished coding on this PR)
[ ] All changes have test coverage
[x] Code is well-documented

Changes

[x] add state_info to models
[x] use state_info to decide state shape in beam search

@sxjscience @hhexiy

Job PR-175/6 is complete.
Docs are uploaded to http://gluon-nlp-staging.s3-accelerate.dualstack.amazonaws.com/PR-175/6/index.html

Also, test case with different layouts is needed.

Job PR-175/13 is complete.
Docs are uploaded to http://gluon-nlp-staging.s3-accelerate.dualstack.amazonaws.com/PR-175/13/index.html

Job PR-175/18 is complete.
Docs are uploaded to http://gluon-nlp-staging.s3-accelerate.dualstack.amazonaws.com/PR-175/18/index.html


adds nautic miles for the Control.Scale

yeah there is a plugin but it doesnt work properly for values < 1 nmi. to fix the plugin i had to change core code aswell so i thought you can aswell just put this into core.

Hi @obama :D
Why don't you submit a PR to a plugin?

well you have metric and imperial scale in the core, why not nautical scale aswell

@obama nautical scale is not widely used feature. Also, we want to keep Leaflet lightweight and simple. So I think fixing that plugin is the best solution for this issue.


Inconsistency between predicted round winner and next round players?

Sorry if I'm misunderstanding but if Dimitrov has a 51% chance of beating Anderson in the QF, why is Anderson predicted to be in the SF?
(Canadian Open 2018)

I can now see this is addressed here


Realm Cocoa 4.1 - Results type is no longer hashable

!!! MANDATORY TO FILL OUT !!!

Create a simple dictionary in swift with the value being any Results - it fails to compile on Xcode 11.2.1 with error
Showing Recent Errors Only
Type 'Results< MyRealmObject>' does not conform to protocol 'Hashable'
Goals

Expected Results

Till yesterday - this was working as we were using version 3.4.x of realm-cocoa and just upgrading to realm 4.1 where results has become a hashable.
Actual Results

Compilation failed - Type 'Results< MyRealmObject>' does not conform to protocol 'Hashable'
Steps to Reproduce

Code Sample

class MyRealmObject: Object {    @objc dynamic var id = ""    @objc dynamic var name = ""    override static func primaryKey() -> String? {         return "id"     } } class SomeVc: UIViewController {    var myLookup: [String: Results<MyRealmObject>]? }
Version of Realm and Tooling

Realm framework version: 4.1.0
Realm Object Server version: Not applicable
Xcode version: 11.2.1
iOS/OSX version: iOS 13
Dependency manager + version: Cocoapods 1.8.4

That code sample works fine. The value type of a dictionary doesn't have to be hashable.

Yeah I was also surprised - does not seem to compile. Let me clean build to see if this is a Xcode bug

My Bad - This is NOT failing in the standard dictionary. As a part of dictionary creation - I create a Custom type that required the value to be hashable.
extension Results: Hashable {     public func hash(into hasher: inout Hasher) {          hasher.combine(self)     } }
Will something like above be ok to do?

hasher.combine(self) would just be an infinite loop.
There isn't really anything on Results that you could hash that would give a stable useful result. One option could be to wrap it in a class type that hashes its ObjectIdentifier.

Closing because of tgoyne's input.


⚠ Bofrost.es has degraded performance

In 36dd0a3, Bofrost.es (https://www.bofrost.es) experienced degraded performance:

HTTP code: 200
Response time: 1605 ms

Resolved: Bofrost.es performance has improved in e99988c after 9 minutes.


write iavl interpreter for RawStore

There is docker image you can get here to run it as a service https://hub.docker.com/repository/docker/foamspaceorg/iavl
(@blinky3713 fill in details on how to start the container)
> docker run -p 8090:8090 -p 8091:8091 foamspace/iavl:latest /iavlserver -db-name DATABASE_NAME -datadir "." -grpc-endpoint "<IP_ADDRESS>:8090" -gateway-endpoint "<IP_ADDRESS>:8091"

We will need to also add this to the compose file(s) for the example apps
There's gonna be some work to take care of the Scoped RawStore stuff. If it's unclear, ask @blinky3713
What about the cache? In the case of checkTx throw it away, in the case of deliverTx throw it away at the end of processing.

https://github.com/f-o-a-m/hs-abci/pull/188


distributed algebra

I'm not sure if I should use the function directly, or subclass the torch function.I'll update the gpu version in the next few days

Code needs to be cleaned up, I submit functional modules

tslu can use block-dtensor to reduce code


API differences?

I was testing the new UI and changed the source api in script_min.js from https://supportxmr.com/api/ to https://api.xmrpool.net/ and now doesn't load block info or network and pool Hashrates, are there differences between Snipa "reference" code and sxmr API's that could explain the missing data?
By the way I was testing it over tor and the original works fine but not with Snipas pool
http://pool.xmkwypann4ly64gh.onion/
Thanks in advance

are you sure that the requests are not being blocked by your browser or a
plugin like ublock?
On Wed., Sep. 25, 2019, 4:09 p.m. kic0<EMAIL_ADDRESS>wrote:

I was testing the new UI and changed the source api in script_min.js from
https://supportxmr.com/api/ to https://api.xmrpool.net/ and now doesn't
load block info or network and pool Hashrates, are there differences
between Snipa "reference" code and sxmr API's that could explain the
missing data?
Thanks in advance
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/M5M400/supportxmr-gui/issues/6?email_source=notifications&email_token=AFOTSFZWN4OYLGNIJHKFH5LQLNWKPA5CNFSM4I2NLUC2YY3PNVWWK3TUL52HS4DFUVEXG43VMWVGG33NNVSW45C7NFSM4HNTUVJQ,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AFOTSF6DYSXKXUN3CXI64OTQLNWKPANCNFSM4I2NLUCQ
.

With SupportXmr api works fine just changed the url in the code to Snipas API and it only blocks part of the site strangely Payments for instance are shown ok
http://node.xmr.pt/

We just added two api endpoints (/network/chart/difficulty and /motd). It seems /motd is not yet avail on xmrpool.net - I wonder if it missing is breaking the js, but it should be unrelated.
@Thunderosa ?

Well, so there are some very specific data things happening with this UI that would be pretty sensitive to any API differences.
The motd error is self explainatory,...but eff and tme missing is something else. Without being able to see exactly where it's choking, I'm guessing something with the blocks endpoint specificially. I think you were on the Pool Blocks page when you got this error?
If so, for the first page of results the blocks in the list and the blocks in the chart are the same data source and then the blocks list starts using it's own lookup on page 2.
So that would mean your issue is probably in the 'block' mode of the data function (as opposed to 'blockhistory').
That section requires the following endpoints from the API:
'hash', 'height', 'value' (in atomic units), 'shares', 'diff' & 'valid'
$D['block'] being populated would be the object to check first.

Nope, nevermind,...you found a good bug. It's because you haven't found a block within the window of time. I'll correct this for the next version.
For the motd error a quick fix would be to comment out the line: url = 'pool/motd'; If that url variable isn't set then it won't try to fix. If you decide you want the feature, you can setup the endpoint and restore the value.

Awesome thanks for the help! I'll update and let you know :)


macOS/iOS/etc. C Type Libraries

While Obj-C type libraries are an entirely different beast that may be blocked pending https://github.com/Vector35/binaryninja-api/issues/2973, the public SDKs for *OS systems contain a large amount of C function calls and structs/enums which we currently do not have types for.
Shipping type libraries for these C functions would offer vast improvements to decompilation output for *OS images as a whole, similar to the improvements made by Windows/Linux ones.

Anything here that isn't covered under #2735 or #2736 ?

Anything here that isn't covered under #2735 or #2736 ?

I've updated the issue to target specifically ObjC Libraries. This issue is now 100% blocking on #2973


Products are imported with wrong productType

When importing products of different productTypes provided in one single file the importer creates products with wrong productTypes. Also, updates are not done, cause it uses the wrong type. I guess the caching confuses the productTypes somehow.

Could you give some sample data and/or try out this branch?

that is a current import file of project: merkur-staging-30
MS_product201611111421.json.zip

@Siilwyn this is not an issue anymore. thanks for your time.

Oh great! Could you shortly describe what was wrong to help people in the future? :)

Actually i cannot reproduce this bug. I debugged your module to get more details but the problem didn't occur again. so, it magically restored itself :)


⚠ Bofrost.lu has degraded performance

In 71fe995, Bofrost.lu (https://www.bofrost.lu) experienced degraded performance:

HTTP code: 200
Response time: 1379 ms

Resolved: Bofrost.lu performance has improved in c967f13 after 18 minutes.


Silence recommendation to upgrade my workspace when a new template has been uploaded

Can the extension silence the pop-up recommendations to upgprade my workspace that appear whenever a new template is uploaded?  As a template author, I don't want my users to get the notification when most of my updates are not essential.

@stirby would be able to provide an opinion here.

In Q4 this year, we'll change how we manage VSCode notifications and give users more control.
In the meantime, we advise only pushing updates outside of working hours and enforcing nightly updates with Autostop Requirement. Or use a different staging template for testing and development.
Otherwise this isn't something we plan to implement as a standalone item.


feat: segregate SSO FF into SAML and OIDC

Description
We were using license_sso_enabled feature flag to track both SAML and OIDC features.
This PR will segregate it into two FFs, license_sso_saml_enabled and license_sso_oidc_enabled.
PR fixes following issue(s)
Fixes #https://github.com/appsmithorg/appsmith-ee/issues/2110
Type of change

Chore (housekeeping or task changes that don't impact user perception)

Testing

How Has This Been Tested?

Please describe the tests that you ran to verify your changes. Also list any relevant details for your test configuration.
Delete anything that is not relevant

[x] Manual
[ ] Jest
[ ] Cypress

Test Plan

Add Testsmith test cases links that relate to this PR

Issues raised during DP testing

Link issues raised during DP testing for better visiblity and tracking (copy link from comments dropped on this PR)

Checklist:
Dev activity

[ ] My code follows the style guidelines of this project
[ ] I have performed a self-review of my own code
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[ ] My changes generate no new warnings
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] New and existing unit tests pass locally with my changes
[ ] PR is being merged under a feature flag

QA activity:

[ ] Speedbreak features have been covered
[ ] Test plan covers all impacted features and areas of interest
[ ] Test plan has been peer reviewed by project stakeholders and other QA members
[ ] Manually tested functionality on DP
[ ] We had an implementation alignment call with stakeholders post QA Round 2
[ ] Cypress test cases have been added and approved by SDET/manual QA
[ ] Added Test Plan Approved label after Cypress tests were reviewed
[ ] Added Test Plan Approved label after JUnit tests were reviewed

/ok-to-test

/ok-to-test

/ok-to-test

/ok-to-test


Introducing OL8 box

This fixes #186
Signed-off-by: Gerald Venzl<EMAIL_ADDRESS>

Nice!
Suggestion: It might make sense to change line 17 of install.sh to dnf upgrade -y, since DNF is preferred for OL8. (yum is really symlinked to dnf anyway.)
Also, there are a couple of issues with the current (07-Oct-2019) version of the ol8-latest base box that people will probably run into:

The base box is missing 3 packages (elfutils-libelf-devel, gcc and kernel-devel) that are required to rebuild the VirtualBox Guest Additions after a kernel upgrade. Without them, the GAs (including shared folders) will stop working after a kernel upgrade when the VM is rebooted. (And building this box will upgrade the kernel and reboot.)

Even after installing the missing packages, there's a dracut error anytime the kernel is upgraded or the GA kernel modules are rebuilt:
  dracut-install: ERROR: installing 'virtio'
  dracut: FAILED:  /usr/lib/dracut/dracut-install -D /var/tmp/dracut.TxRUbT/initramfs -N floppy --kerneldir /lib/modules/4.18.0-80.11.2.el8_0.x86_64/ -m xen_netfront xen_blkfront virtio_blk virtio_net virtio virtio_pci virtio_balloon hyperv_keyboard hv_netvsc hid_hyperv hv_utils hv_storvsc hyperv_fb ahci libahci mptspi

I don't know what causes this, but it doesn't seem to affect the functionality of the VM.

If dnf is the default on OL8, then we definitely should change it. Part of these scripts is also to provide a reference implementation/examples for users. I.e. Oracle provides Oracle Linux, then we should also be the first ones to adopt our new features. :)

While I don't disagree, on OL8 dnfand yum are pointing to same executable, so it does not matter too much...
But good to stat to have dnf in the fingers
[opc@ol8 ~]$ ls -l /usr/bin/{dnf,yum}
lrwxrwxrwx. 1 root root 5 May 23  2019 /usr/bin/dnf -> dnf-3
lrwxrwxrwx. 1 root root 5 May 23  2019 /usr/bin/yum -> dnf-3

Thanks!

While I don't disagree, on OL8 dnfand yum are pointing to same executable, so it does not matter too much...

It matters from a best practice perspective and the fact that there is no guarantee that yum will continue to be symlinked as it is now. Also, the Docker images may or may not have yum at all and the 8-slim image doesn't even have dnf (it uses microdnf). So getting folks used to the new normal is a good thing.


[WIP] BIP mapping pass

Summary
Add BIP-based mapping pass, which does layout and routing at once as described in https://arxiv.org/abs/2106.06446.
Closes #6437
Details and comments
This is a WIP.

[x] Debug
[x] Test more (including no cplex environment)
[x] How to run unit tests (requiring CPLEX) on CI
[ ] Docstring
[x] Lint
[ ] Release note

Optional:

[ ] Replace cplex.Cplex with docplex.mp.model.Model to improve readability
[ ] Add "error_rate" and "balanced" objective

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.1 out of 2 committers have signed the CLA.:white_check_mark: itoko:x: gnanniciniYou have signed the CLA already but the status is still pending? Let us recheck it.

@mtreinish I've confirmed the unit tests added in this PR can be run on free cplex, which can be installed by pip install cplex, in my local env. That means we could run them on CI for some python versions. Could you help me for that?
I've just added MissingOptionalLibraryError here.
What else do you think we should do for users who don't have cplex or have only free cplex?

@itoko https://github.com/Qiskit/qiskit-terra/pull/6580/commits/8c2d609ea9c3ec1dba7949ca51047a826cea762c fixes the logic around the optional dependency, adds a test skip condition on the presence of cplex, and configures python 3.7 to install cplex so we can run the tests in that job.

The tests are running on the linux 3.7 job now, for example: https://dev.azure.com/qiskit-ci/qiskit-terra/_build/results?buildId=28930&view=logs&j=8eacdd59-c2e8-5617-75a5-8ed7c854a78f&t=5df005f4-d5f4-5d2d-0307-5b24b15488fc&l=12227 and being skipped in the other jobs

@gnannicini when you get a chance can you sign the cla with the link from https://github.com/Qiskit/qiskit-terra/pull/6580#issuecomment-861395266 otherwise this will block this PR from merging when it's ready.

@mtreinish Thanks a lot! I'll fix the errors in the lint and test.

I recommend to execute unit test both w/ and w/o (do)cplex in CI to ensure that terra works even without installing (do)cplex.

@levbishop @ajavadia I found that the BIPMapping pass usually does not work (CPLEX cannot find any integer solution within 30 seconds) for backends with more than 7 qubits. Hence, to transpile the QV64/128 circuits, users cannot use the coupling_map provided in the backend.configuration() for backends with more than 7 qubits. That means users have to make a new CouplingMap restricted on the subgraph defined by the 6/7 qubits they want to use and specify the truncated coupling_map to the BIPMapping pass. I stated that in the class docstring, but I'm not sure it is sufficient to tell users such an important guide on the usage of the BIPMapping pass. Any suggestion on this?

@itoko I think that's fine. If the docstring says that this has bad scaling beyond 10-qubit CouplingMaps, then I would say it is up to a user to know this when invoking this on a large architecture.


Mobile app: add AmoLED theme

An AmoLED theme would certainly be a great way to save batter life. Specially for those who use the app for an extended period.
I would suggest keeping the current dark theme but also offer the complete AmoLED black theme as for the Reddit app, for example.

Dark theme already exists and measurements have shown that there's no significant benefit of black over dark gray.

Would be great to read about those measurements. Reddit sync power consumption between black and dark themes had a 41% decrease as far as I read about, but overall expected would vary from 15% to 20%. Well that's at least the information I got out a simple Google search from dark to AmoLED black themes power consumption comparison.

https://www.xda-developers.com/amoled-black-vs-gray-dark-mode/


Plugin: add aggregated completion events

Completion events are standard, openedx events, so supporting them should be part of the Aspects core, cf https://github.com/openedx/openedx-aspects/issues/88.
OpenCraft maintains the completion aggregator plugin app which currently lacks support for tracking events.
Contribution https://github.com/open-craft/openedx-completion-aggregator/pull/173 adds tracking events for the aggregated completion states, and https://github.com/openedx/event-routing-backends/pull/377 proposed pulling them into the ERB. But since these events aren't openedx events, we'd rather keep the event routing logic with the aggregator app.

[x] Get https://github.com/open-craft/openedx-completion-aggregator/pull/173 tested and merged: https://github.com/open-craft/openedx-completion-aggregator/pull/206
[x] Refactor https://github.com/openedx/event-routing-backends/pull/377 into a https://github.com/open-craft/openedx-completion-aggregator PR: https://github.com/open-craft/openedx-completion-aggregator/pull/205
May require some refactoring of https://github.com/openedx/event-routing-backends/pull/377 to support sharing event routing and testing logic.
[ ] Create a dbt extension that adds aggregated  completion views to aspects-dbt. https://github.com/open-craft/dbt-completion-aggregator/pull/1#
[ ] Create a tutor plugin that adds custom tab / charts to display aggregated completion information in instructor dashboard
[ ] Document this example of how to create an Aspects plugin in the How Tos.
Ensure caveats of the aggregator are noted (cf sync vs async) if providing installation instructions.
[ ] Optional: create a management command to read the completions aggregates from the database and send them in a way that we can backfill this data. cf https://github.com/openedx/openedx-aspects/issues/88 for ideas on how to approach this.

Create a tutor plugin that adds custom tab / charts to display aggregated completion information in instructor dashboard

@pomegranited For this task there should be some work on Aspects that allow you to re-use the import_assets_helper module on your tutor-plugin so that it can facilitate it the serialization and organization of plugins. In theory it should work out-of-the-box but let me know how that goes.
Another concern is how will you append a tab for the completions to the Instructor Dashboard on a patch.


Exclude docs directory during npm publish

While releasing v1.0.0-rc.1 I noticed that all the .rst files in the docs directory were being packaged.

I don't think we should be package these when publishing.
We can ignore these by adding docs/ to our .npmignore.

Should we get this in the v1?

Yes I will get this in for v1.0.0.


There may be some problems with the function description of server-id.

In the description of server-id, it is written that server-id will be used to maintain the binlog pos of mysql.
Regarding this point, in the master-slave replication, mysql maintains its own position by the slave, and the server-id is only used as a unique identifier for the slave. And in debezium, I did not find information about server-id used to maintain binlog pos in the document.
Is this an error in the description? Please let me know if I am wrong.

@carryxyh I think the document is ok because MySqlSouce  keep the binlog position in state for each checkpoint, the manage means managing the state and failover using the state.

Closing this issue because it was created before version 2.3.0 (2022-11-10). Please try the latest version of Flink CDC to see if the issue has been resolved. If the issue is still valid, kindly report it on Apache Jira under project Flink with component tag Flink CDC. Thank you!


manual prefixes result in no response from the bot, despite it telling me to use the prefix under its name.

Bug Description
i used to use the bot with a prefix of '-'
but when i updated to 0.3.9 today, it would not work.
i reset the token, the owner id, remade the bot, but nothing worked.
only when i deleted the config, and let the bot regenerate a config did it work. but only with the default prefix of "@mention"
once i changed the prefix to '-', saved config, and restarted the bot, it would no longer respond, despite it saying its "playing: type -help" under its name on the right of discord.
i tried '-', '--', '!', and '!!' (like the config shows in the comments) and NONE of them work.
the bot simply will not respond, or emote on what i type. nor will is DM me the "help" message.

Steps to Reproduce
open config
set prefix to anything other than default? (i tried -, --, !, and !!)
save config
start or restart the bot
note the change in its "playing: type -help" message under its name on the right side
type -help, but get no DM
type -play "song name", but have nothing happen
Expected Result
bot should respond to the prefix i set
Debug Output
System Properties:
  java.version = 1.8.0_333
  java.vm.name = Java HotSpot(TM) 64-Bit Server VM
  java.vm.specification.version = 1.8
  java.runtime.name = Java(TM) SE Runtime Environment
  java.runtime.version = 1.8.0_333-b02
  java.specification.version = 1.8
  os.arch = amd64
  os.name = Windows 10

JMusicBot Information:
  Version = 0.3.9
  Owner =<PHONE_NUMBER>20982273
  Prefix = @mention
  AltPrefix = null
  MaxSeconds = 0
  NPImages = false
  SongInStatus = false
  StayInChannel = false
  UseEval = false
  UpdateAlerts = true

Dependency Information:
  JDA Version = 4.4.1_353
  JDA-Utilities Version = 3.0.5
  Lavaplayer Version = 1.3.78

Runtime Information:
  Total Memory = 380
  Used Memory = 53

Discord Information:
  ID =<PHONE_NUMBER>776040067
  Guilds = 1
  Users = 2

Additional Info
No response
Checklist

[X] I have looked for information about this within the documentation
[X] I have searched for similar issues on the issues page
[X] I am running the latest version of the bot:

#1215

#1215

that fixed it thanks. i searched for similar bugs, but missed this. hopefully it will be added to the guide to help future noobs like me.

Enabling the intents is already in the Wiki though.

Enabling the intents is already in the Wiki though.

oh im sorry i guess i just missed that part


Add type hints

Adds type hints to the following files per https://github.com/adafruit/Adafruit_CircuitPython_IS31FL3741/issues/7 :

adafruit_is31fl3741/__init__.py
adafruit_is31fl3741/issi_evb.py
adafruit_is31fl3741/adafruit_rgbmatrixqt.py
adafruit_is31fl3741/adafruit_ledglasses.py

The particular part of the CI failing is because I've added type hints for a PIL Image where it's expecting one, but Pillow is not part of the requirements.txt.
Would you rather I update the requirements.txt to require Pillow or find some typing workaround?

The particular part of the CI failing is because I've added type hints for a PIL Image where it's expecting one, but Pillow is not part of the requirements.txt.
Would you rather I update the requirements.txt to require Pillow or find some typing workaround?

I think we can add Pillow to the requirements.txt it would be needed in order to use that function if I understand correctly.

Yeah, I'll get to it later tonight.


Improve upgrade CLI workflow

Problem
Today whenever you update the @prisma/cli package, you also need to update the @prisma/client package.
{
  "dependencies": {
    "@prisma/client": "2.7.0"
  },
  "devDependencies": {
    "@prisma/cli": "2.7.0"
  }
}

These two dependencies need to match. This is easy to forget for 2 reasons:

It's unusual to have an implicit "peer dependency" on the Prisma Client.
You didn't install the @prisma/client in the first place, it was added to the package.json for you by @prisma/cli

The upgrade flow is important for us because we're releasing every 2 weeks right now. The more difficult it is to upgrade, the less people will upgrade. The less people that upgrade, the more we need to support older CLIs and Clients.
There are 2 ways to upgrade today:
# Install the new CLI
npm install @prisma/cli@latest @prisma/client@latest

# Check in the package.json that the version match up
cat package.json

# Generate new Client:
npx prisma generate

# Run code
node index.js

Or
# Install the new CLI
npm install @prisma/cli@latest

# Remove old client from node_modules
rm -rf node_modules/@prisma/client

# Generate new Client
npx prisma generate

# Run code
node index.js

Note: This 2nd approach is more obscure, but slightly easier because it will update the @prisma/client automatically in your package.json.
Suggested solution

A good first step is to detect that there's a mismatch between @prisma/client and @prisma/cli and fail (or warn?).
We also should consider calling this dependency what it is, a "peer dependency". As an "ancient node-ster", I instinctively recoil a little when I think of having "peer dependencies" in my app, so we'll need to make sure expectations are set.
Alternatives
Run npx prisma update or npx prisma upgrade

Suggested by @williamluke4 :-)

I could see this working too in the same way that npm upgrade npm works.
Auto-upgrade @prisma/client

I'd also like us to really think about auto-updating this dependency. Auto-upgrading would eliminate the need to detect a mismatch and always tie the @prisma/client to the @prisma/cli version that last generated the client.
We draw a clear line that we manage this dependency's lifecycle. From creating, to updating, to perhaps deleting. We tell the user that they do not need to think or care about this package. As noted above, we do update this dependency automatically in certain circumstances. This might be a bug, but it does show that it works okay.
Where this can break down is when you have a global CLI and multiple projects. When you run generate with a newer CLI on an older project, it can cause a break. This problem seems bad, but feels like a non-issue to me in practice for a couple reasons:

The package.json's @prisma/client version will tell you what CLI version you need. If you make this mistake it's easy to fix.
Anecdotally, this is the status quo in the Go community today. While it seems like it should be a constant source of pain, in practice this issue almost never comes up. This makes me think, we shouldn't optimize for this workflow of global CLI with many local installations.

I'd love to hear other folks thoughts on this too!

There are 2 ways to upgrade today:
... Or

# Install the new CLI
npm install @prisma/cli@latest

# Remove old client from node_modules
rm -rf node_modules/@prisma/client

# Generate new Client
npx prisma generate

# Run code
node index.js

Maybe I'm misunderstanding, but how can that be a way to upgrade today? You always need to update the @prisma/client dependency by hand today.

We should add a prisma update command

Maybe I'm misunderstanding, but how can that be a way to upgrade today? You always need to update the @prisma/client dependency by hand today.

https://www.loom.com/share/1292e40a381d489dbb8a57814f3dc66f
🤷♂

We should add a prisma update command

I also like this idea!

prisma update - For minor versions 2.4 ->2.5
prisma upgrade For major versions 1.34 -> 2.x

Ok that's a bug, here the "add prisma client" logic is going further than it should do.

Does not seem to be a big problem for our users, so closing this for now.


Repeatedly changing exposure of d435 brings down a camera

Required Info

Camera Model
D435

Firmware Version
<IP_ADDRESS>

Operating System & Version
Linux (Ubuntu 16)

Kernel Version (Linux Only)
tegra ubuntu 4.4.38

Platform
NVIDIA Jetson

SDK Version
2.9.1 }

Issue Description
We are running a system with two d435 cameras connected to an NVIDIA TX2. We discovered that repeatedly changing exposure on one camera will bring the second camera down.
We used the following bash script to invoke ros param server to change the exposure settings:
#!/bin/bash
realsense2_camera="/sensors/camera_front_forward/realsense2_camera"
while :
do
#loop infinitely
rosrun dynamic_reconfigure dynparam set $realsense2_camera rs435_depth_exposure 200
sleep 5
rosrun dynamic_reconfigure dynparam set $realsense2_camera rs435_depth_exposure 30
sleep 5
done
With the latest firmware released by Intel we can get this script to bring down a camera after about 5 hours run time ( interestingly it brings down the second camera down, and not the one on which we change the exposure rates)
With a previous version of firmware we can reproduce this in about two hours with the script above.
We have applied the UVC Linux drivers to the NVIDIA Tegra kernel patch as per installation instructions.

I used a powered USB 3.0 Amazon Basics hub to connect two cameras to TX2 board. We also have a custom board (for our product) which exposes 5 USB 3.0 ports. I was able to reproduce this behavior on both setups.
We are running one camera with RGB set to 640x480 @ 15 Hz. The second camera doesn't have color enabled at all. Depth is captured at 424 x 240 @ 30Hz. We change exposure settings for the camera which has RGB enabled.

[Realsense Customer Engineering Team Comment]
@DrCodeWizzard,
basically, we recommend to use dedicated USB3 for D435 to ensure there sufficient bandwidth  especially for high fps and high resolution.
when you said the second device down, is it not enumerated any more?
is there lsusb -t or dmesg for reference?
by the way, please we have  newer FW,  v5.9.11, which has fix for device recognize issue and some control related issue
https://downloadcenter.intel.com/download/27792/Latest-Firmware-for-Intel-RealSense-D400-Product-Family

Yes, the second device is no longer enumerated. I will check the new firmware.
Also yes, I am aware of the USB3 recommendation and that is why our production board exposes 5 dedicated ports.

[Realsense Customer Engineering Team Comment]
@DrCodeWizzard,
could the new FW help on your issue?

The latest firmware appears to make the camera more stable to crashes due to changing exposure levels. I have been able to run the script for several days without a break.


Chinese Locale Support

Add zh locale support
Add simple url generator in NetworkProducer
fix DateProducerSpec "should generate date between years #fromYear - #toYear", unpass test because the expect result is not correct

Changes Unknown when pulling 5a630c213497029ebdaf260d81014abe2aaab4e7 on Lhfcws:master into ** on Codearte:master**.

Related issue #78

@OlgaMaciaszek I've made some modification according to your reviews, including switching the providers of Person/Company/IBAN into Interfaces.

@Lhfcws thank you very much. I will review your new changes tomorrow. The reason we are hesitant when it comes to switching to Java 8 (although we do miss the features a lot, me at least) is that we have some Android users and it seems the support for Java 8 might not yet be stable. We will definitely make a switch once we see no problems for Android.

Hi @OlgaMaciaszek  . What's the progress of this PR now?

Changes Unknown when pulling 291c79355bb0f70086ae7d7216fc50f34d651315 on Lhfcws:master into ** on Codearte:master**.

Got it. Done.


Is it possible to add a server demo to CAPI?

Is it possible to add a server demo to CAPI?

The C API does not include server types, at the moment. There's some work in #3084, but hasn't been merged.


I'm decoding a map with many fields, but the error I got is:
json: cannot unmarshal hex string without 0x prefix into Go value of type common.Address

There's no information about which field is causing the error.
What did you see happen?
The error information when decoding map is vague, it doesn't indicate which field is causing the error.
What did you expect to see?
I expect to see which field(the specific key or value) is causing the error.

Could you share a specific example that reproduce the issue? Thanks.

Could you share a specific example that reproduce the issue? Thanks.

Here is the minimal program to reproduce.
(In my actual case, the map has a lot of fields.)

@cherrymui maybe the WaitingForInfo can be removed?


Swinject Podfile for ios/tvos and osx

When using Swinject with cocoapods targeting iOS/tvOS as well OSX, the compiler seems not to be able to find the correct header file in "Swinject.h"
changing the line
#import <Swinject/_SwinjectStoryboardBase.h>
to
#import "_SwinjectStoryboardBase.h"
resolves the issue. Sorry for me not having seen that issue before.

Would you tell me more about the problem? Are you using Swinject in Objective-C?

Hi Yoichi. I am actually using it with swift. I started in Xcode with a new iOS project and adding an OSX target. Afterwards closed the project and did the usual pod init, Podfile editing, pod install, pod update ceremony. After that opened the workspace and tried to compile iOS and OSX. When compiling the iOS target, the compiler complaint that <Cocoa/Cocoa.h> could not be found in _SwinjectStoryboardBase.h, so obviously the wrong variant is found by using #import <Swinject/_SwinjectStoryboardBase.h>.

Thank you for the answer.  Did you mean a file contained only #import and it should be fixed to #import _SwinjectStoryboardBase.h. Which file contains only #import?

Seems I failed pasting the path, so it should read changing (line 23 in Swinject.h) from
#import <Swinject/_SwinjectStoryboardBase.h>
to
#import "_SwinjectStoryboardBase.h"
fixed the issue for me.

I think I experienced a similar issue and fixed already. Would you tell me which version you installed?

I am using 1.1.0

You might need to write your Podfile like this:
target :ios do
    platform :ios, '8.0'
end

target :osx do
    platform :osx, '10.10'
end

More info here: https://github.com/CocoaPods/CocoaPods/issues/2043
Does your Podfile has that kind of switch between iOS and osx?

Actually I am using a slightly different syntax:
target 'iosApp' do
  platform :ios, '8.4'
  pod 'Swinject', '~> 1.1.0'
end

target 'macApp' do
  platform :osx, '10.10'
  pod 'Swinject', '~> 1.1.0'
end

I tried your syntax, but that gave me an error:

Unable to find a target named ios

I couldn't reproduce the issue but got pod install command warning "[!] [Xcodeproj] Generated duplicate UUIDs:"
Although I couldn't reproduce the problem, but I think what you see is a problem appearing only when a project contains both iOS and OSX targets and Swinject is installed by CocoaPods. If only a target is contained or Carthage is used, the problem is not reproducible.
If we modify #import <Swinject/_SwinjectStoryboardBase.h> to #import "_SwinjectStoryboardBase.h", we will have a problem with Carthage.
So I would like to provide the workaround you shared for the case targeting both iOS and OSX, and installing with CocoaPods.
The workaround is changing
#import <Swinject/_SwinjectStoryboardBase.h>
in Swinject.h to
#import "_SwinjectStoryboardBase.h"
by each user.
Do you think the workaround is acceptable? Thank you for sharing the problem you experienced with the combination.

I think it is acceptable, sure.
Do you prefer Carthage over Cocoapods, would you recommend moving to Carthage for future projects?

Both Carthage and CocoaPods have their advantages/disadvantages depending on projects. If I have a problem with CocoaPods, I use Carthage in the project, and vice versa.


SAA-178 setting the attendance status to scheduled when attendace records are first created.

Codecov Report
Base: 61.05% // Head: 61.14% // Increases project coverage by +0.09% :tada:

Coverage data is based on head (07f6af9) compared to base (073911e).
Patch coverage: 100.00% of modified lines in pull request are covered.

Additional details and impacted files
@@             Coverage Diff              @@
##               main      #76      +/-   ##
============================================
+ Coverage     61.05%   61.14%   +0.09%
  Complexity      384      384
============================================
  Files           103      103
  Lines          2067     2072       +5
  Branches         37       37
============================================
+ Hits           1262     1267       +5
  Misses          783      783
  Partials         22       22

Impacted Files
Coverage Δ

.../hmppsactivitiesmanagementapi/entity/Attendance.kt
100.00% <100.00%> (ø)

...s/hmppsactivitiesmanagementapi/model/Attendance.kt
100.00% <100.00%> (ø)

...activitiesmanagementapi/util/TransformFunctions.kt
92.51% <100.00%> (ø)

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

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


Can't Select COMPANDER on the menu

Can't Select COMPANDER on the menu

Do you mean it's not there or that you can't change it's value ?

the menu is there... but changes dont save...

It was saving the setting in VFO mode, but not in channel mode.
Think I've fixed it.

Thanks a lot 🥇


Hybrid autoregressive transducer

I was wondering if there are any existing recipes for the HAT model. It is a straightforward change by modeling the blank distribution as a Bernoulli distribution, and was shown to be useful to integrate external LMs, among other things.
Has anyone tried it in icefall, especially with the pruned loss?

I was wondering if there are any existing recipes for the HAT model. It is a straightforward change by modeling the blank distribution as a Bernoulli distribution, and was shown to be useful to integrate external LMs, among other things.
Has anyone tried it in icefall, especially with the pruned loss?

We have not tried that. Would be great if you can add that.

@csukuangfj Do you have advice on what would be a good evaluation setup for using HAT to integrate external LMs? For example, how did you evaluate the LODR methods?

For a POC, I was just training a model on LibriSpeech, and was planning to use an external RNNLM. But Dan pointed out that LibriSpeech may not be the best test-bed for these experiments.

@marcoyang1998
Could you have a look?

You may try cross-domain evaluation scenarios, e.g. decoding the LibriSpeech model on the Gigaspeech test sets using an RNNLM trained on the Gigaspeech transcripts. I believe I tested LODR in this scenario and it yielded better results than using only shallow fusion.


Problem with MXML_CUSTOM in Version mxml-2.10

Hi,
I am using the MXML_CUSTOM element together with custom callbacks for loading and writing.
This all worked well for years but broke with version 2.10. I tracked it down to line 2928 of file mxml-file.c. In the distributed version the line reads:
if ((data = (*global->custom_save_cb)(node)) == NULL)

but it must read:
if ((data = (*global->custom_save_cb)(current)) == NULL)

With that change my code continued to work again. Please adopt my fix for version 2.11

[master ef7e5d4] Fix writing of custom data nodes (Issue #201)


Close the underlying stream when closing the stream

Currently unlike ReadableResourceStream the WritableResourceStream doesn't automatically closes the underlying stream resource with fclose. This PR aims to add that, but as of creating this PR two tests fail, purposely not updated tests to discuss how to handle those here before fixing them.

Thanks for spotting this bug and filing this PR! :+1: I agree that your suggested fix makes sense and the two tests need to be updated because they rely on some rather pointless assumptions.
One possible option would be using a writer filter function instead of rewinding after close. Does this make sense to you?

One possible option would be using a writer filter function instead of rewinding after close. Does this make sense to you?

The idea makes sense, not sure yet how to implement that right now.

Doh your package make it very easy :heart: !
Ping @clue and @jsor, tests have been fixed and are green across the board.

🎉


Alarm Delay Event from Latest bropat/eufy-security-client Update

I desperately need an alarm delay event exposed in HA for my Eufy HomeBase2 in Home Assistant. I noticed that in bropat/eufy-security-client Changelog 2.1.0 (https://github.com/bropat/eufy-security-client#210-2022-06-12), they actually created an event for this.
What is the usual timeline for this eufy_security HA add-on to get updated using the latest bropat/eufy-security-client ?

I had just updated it, but you also need an event for this, right?

Yes please :)

just updated the integration now, alarm station will be in delayed state for 1 sec when the event is received and return to previous state. As delayed event does not change the state, I preferred this way. Test with new version and comment back if you have issues

Just tested several times by:

Switching my alarm_control_panel.homebase_2 from armed_home to armed_away 3 different ways: via my keypad, via clicking Away mode in the Eufy App, and via alarm_control_panel.alarm_arm_away. The state always just flips immediately from armed_home to armed_away without the interim "delayed" state
So then I wanted to test to see if the "delayed" state would appear upon returning home when the alarm is in the armed_away state.
When I returned home, the 30 seconds of alarm delay started counting down. The state remained armed_away
I let the alarm actually go off and the state changed to "triggered"

So I never see a state "delayed" in the scenarios I tested. But maybe I'm misunderstanding how you implemented this? Maybe I should be looking at an attribute or something instead of state?

Thank you @fuatakgun, I can't wait to test this. It actually sounds like the perfect implementation to me. I have no need to know if it's in a delayed state, I just need to know when it's in an alarm state so I can sound the siren.  What I'm curious to find out in my testing is whether the triggered state remains until it is reset or disarmed.  If not (eg, if it only shows triggered momentarily), I would need to find a way to know that the alarm has been reset (for example when you reset it from the Eufy app).

@hipsi In my testing, is only stays "triggered" momentarily. Also, if you switch from armed_away to armed_home while the alarm is actually going off, it momentarily switches to "triggered" again and then settles to armed_home. These are all unrelated to this thread though. This thread is specifically about having something that indicates when the system is in alarm delay either on your way out the door or when you just get home.


fix: request body edge cases

Readable can emit 'end' after .destroy() as well
as legacy streams might not implement the whole
Readable contract properly. Try to handle these
edge cases better.
Fixes: https://github.com/mcollina/undici/issues/66

valid errors on 10 & 12


Can I pass dynamic props to components by state in v3?

I want to pass dynamic props to components by state in v3. But it didn't work.
It's show a warning "[react-router] You cannot change ; it will be ignored" when I use code below. Then in  component, it always shows this.props.userId is null.
constructor(props) {
        super(props);
        this.state = {
            userId: null,
        }
    }
componentDidMount() {
    this.setState({
            userId: "123",
    })
}
render() {
return (
<Router history={browserHistory}>
            <Route path="/TeachingResourceManagement">
                <IndexRoute component={(props) => (<Login {...props} onChange={this.handleUserIdChange.bind(this)} userId={this.state.userId} />)}/>
            </Route>
</Router>
)}

P.S.
I had tried to use code below (use render to pass props) but it also didn't work and even couldn't render a scene normally.
<Router history={browserHistory}>
            <Route path="/TeachingResourceManagement">
                <IndexRoute render={(props) => (<Login {...props} onChange={this.handleUserIdChange.bind(this)} userId={this.state.userId} />)}/>
            </Route>
</Router>

No, you cannot do this. You shouldn't be wrapping Router in a component. It should live near or at the top of the component tree.

@timdorr Router is at the top of the component tree in my code, I have updated it. And how can I pass dynamic props to components in v3?


Add PSK support

Add PSK support to the library with one basic test.

I'd really like to see this make its way in. Is there a path forward here?

Needs review @alexanderheuts

Close this PR since this is outdated and PSK support is already merged.
Pls check comments in https://github.com/fortanix/rust-mbedtls/pull/205 for more info


[Bug]: requestPermission Future never resolves, when application goes to background, when "Allow to use location" popup is displayed

Please check the following before submitting a new issue.

[X] I have searched the existing issues.
[x] I have carefully read the documentation and verified I have added the required platform specific configuration.

Please select affected platform(s)

[ ] Android
[X] iOS
[ ] Linux
[ ] macOS
[ ] Web
[ ] Windows

Steps to reproduce

Enable location on device and in application.
Execute: final permission = await Geolocator.requestPermission();
"Allow to use location" popup should appear:

Lock the phone.
Unlock the phone.

Expected results

Bring back "Allow to use location" when going back to application.
OR
Complete the Future with proper state.

Actual results

"Allow to use location" is gone.
final permission = await Geolocator.requestPermission(); is waiting forever.
Another call to this method results in exception:

Unhandled Exception: A request for location permissions is already running, please wait for it to complete before doing another request.

Code sample
Code sample
final permission = await Geolocator.requestPermission();

Screenshots or video
final permission = await Geolocator.requestPermission();
Version
12.0.0
Flutter Doctor output
Doctor output
[✓] Flutter (Channel stable, 3.19.6, on macOS 14.4.1 23E224 darwin-arm64, locale en-PL)
    • Flutter version 3.19.6 on channel stable at /Users/xxx/git/flutter
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision 54e66469a9 (5 weeks ago), 2024-04-17 13:08:03 -0700
    • Engine revision c4cd48e186
    • Dart version 3.3.4
    • DevTools version 2.31.1
[✓] Android toolchain - develop for Android devices (Android SDK version 34.0.0)
    • Android SDK at /Users/xxx/Library/Android/sdk
    • Platform android-34, build-tools 34.0.0
    • Java binary at: /Applications/Android Studio.app/Contents/jbr/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 17.0.7+0-17.0.7b1000.6-10550314)
    • All Android licenses accepted.
[✓] Xcode - develop for iOS and macOS (Xcode 15.2)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Build 15C500b
    • CocoaPods version 1.15.2
[✓] Chrome - develop for the web
[✓] Android Studio (version 2023.1)
[✓] VS Code (version 1.89.1)
[✓] Connected device (5 available)
[✓] Network resources
    • All expected network resources are available.

• No issues found!

Dear @tobolski,
This is a known issue in iOS. I'll label this as an enhancement there are multiple issues regarding this situation.
Kind regards


Multiple missing values not writing correct datapackage.json

Current Behaviour
I have a table with missing values represented by "TBA" and null ""
I specify missing values (tried with " " and "")

Validation doesn't work

datapackage.json written as

Expected Behaviour
Validation works with multiple missing values
Datapackage.json converts "" to "" not " \"\"
Your Environment

Data Curator version: 0.10.0
Operating System and version: Windows 7

This should work...

See also https://github.com/frictionlessdata/specs/pull/588

Hi @Stephen-Gates,
Put in the values without quotes

Fixed


[BUG] Use of strnlen() is not portable

strnlen() in fuzz/provider.c is not portable. Causes build failures on NonStop (at least).
      found_len = strnlen((const char *) *buf, *len);
                  ^
"/home/ituglib/randall/openssl-klt/fuzz/provider.c", line 163: error(114):
          identifier "strnlen" is undefined

Seems like the most reasonable thing to do would be to create an OPENSSL_strnlen(const char *s, size_t maxlen) function for this and replace the call in the fuzzer.

I can give it a shot here

actually, scratch that, we have an implementation of OPENSSL_strnlen already, just need to switch to using it

PR for you @rsbeckerca should fix your issue

PR for you @rsbeckerca should fix your issue

Will work on it when I'm back at the office.

PR for you @rsbeckerca should fix your issue

PR is ready to go.

Is there a way we can check automatically for use of strnlen() (and maybe others?) in new code, so that we don't have to go through this dance everything someone only tests their PR on Linux?

Is there a way we can check automatically for use of strnlen() (and maybe others?) in new code, so that we don't have to go through this dance everything someone only tests their PR on Linux?

Yup.  In .github/workflows/ci.yml, we have a -ansi build that enables POSIX.1-2008 (through the _POSIX_C_SOURCE C macro).  A -ansi build with an older POSIX version would probbaly put these discrepancies on display.  See https://www.man7.org/linux/man-pages/man7/feature_test_macros.7.html for a set of possible values.
Note: strnlen() is specified in POSIX.1-2008, that's why our CIs aren't reacting on this one.

Re NonStop, I've lost track of what POSIX version it's supposed to support...  from earlier iterations, it also seems like that support is partial, if I remember correctly.  Is that your recollection too, @rsbeckerca?

In .github/workflows/ci.yml, we have a -ansi build that enables POSIX.1-2008

Is there some (relatively easy) way we could implement a banned-apis.list where we put strnlen() along with sprintf()? :)

How's that easier than making a change to check-ansi?

we actually have this check:
https://github.com/openssl/openssl/actions/runs/9960819468/workflow?pr=24912
Which would have caught the new abi addition, it just currently doesn't build the fuzzers, perhaps it should

How's that easier than making a change to check-ansi?

I interpreted the comment "[NonStop POSIX] support is partial" as meaning there isn't one POSIX level that would catch exactly what NonStop provides. If there is, then clearly that would be easier

How's that easier than making a change to check-ansi?

I interpreted the comment "[NonStop POSIX] support is partial" as meaning there isn't one POSIX level that would catch exactly what NonStop provides. If there is, then clearly that would be easier

I was hoping that there was a minimum POSIX level that was supported enough for OpenSSL.

In practice, there are POSIX compliance levels but looking at POSIX-1.* in the NonStop context, it does support POSIX from around 2008 with a lot of additions from POSIX-2* seemingly at random (or more specifically what many of us whined about loudly enough). The next OS release may bring some new changes, but we are stuck with having to support OS releases from 5 years back (HPE's general policy on support). I think the fuzz check would be better as things are discovered missing, like posix_memalign (not at all), aligned_alloc (only c11, which as of today does not generate good thread code - reported).


Auto-add 'new site!' label when creating pull request in _data/sites folder

Seems like this would save you time when adding new sites each week? I would add the labels myself but seem unable to do so...

Is this something GitHub lets you do?

maybe with query parameters?
However, it looks like everyone is just creating issues. The instructions say to create a file in _data, so I've been just doing that and skipping the issue. Am I doing it wrong?

Long term I’d like to either move to Netlify CMS or Netlify Forms for this to allow people to create these files in a UI and get rid of the issues/PRs altogether.
That said I would prefer it if people don’t create issues for them. Eventually I’ll have to go through and close all of those and ask people to submit PRs instead anyway.

CMS or Forms seems like a good idea. 👍


multiple definition of digital_pin_to_bit_mask_PGM

ArduinoShrink is not replacing all of the AVR Arduino core functions defined in wiring_digital.c, resulting in multiple definitions of the pin mapping tables in some cases.

ArduinoShrink replaces the 3 non-static functions defined in wiring_digital.  The issue is due to link order when other libraries are used.  ArduinoShrink must be the last library included, as the IDE uses include order to determine link order.
todo: document include order requirement

fixed in https://github.com/nerdralph/ArduinoShrink/commit/cdae1ac91108ff99261cdd226114a3f8cc21c543


Cleanup experiment creation code in integration tests

A lot of the code to create experiments in our integration tests has been copy/pasted over time and contains unnecessary parameters. This PR tries to minimize them. It also tries to differentiate the artifact locations of the experiments compared in the new flow tests.

Part of the work for #561.


getMinAmountForCloneTransfer should take the same paramaters as duplicate()?

I’m imagining the user flow would be like they first call getMinAmountForCloneTransfer and then call duplicate. So it’s a better UX to have the same parameters for them?

That's a good question. I think that getMinAmountForCloneTransfer is more likely to be called by a contract though, and if somebody already knows the paramaters for dullicate it's pretty easy to hash them together off chain

makes sense.


test: add RunCmd lib function to reduce the work of developing cli tests

Signed-off-by: letty.ll<EMAIL_ADDRESS>
1.Describe what this PR did
Add a lib function: RunCmd. And a demo of usage in : pouch_cli_version_test.go.
With this function, user could fill a struct with their CMD and expected results, this function will execute CMD, check results and assert failure if there is error.
The usage is as following:
+	cmd = PouchCmd{
 +		args:        "version",
 +		result:      true,
 +		outContains: "APIVersion",
 +	}
 +	RunCmd(c, &cmd)

A failure log is as following:
#go test -check.f PouchVersionSuite

▽
package main

----------------------------------------------------------------------
FAIL: pouch_cli_version_test.go:21: PouchVersionSuite.TestPouchVersion

pouch_cli_version_test.go:31:
    RunCmd(c, &cmd)
lib_common.go:110:
    // Add "(?s)" to match "\n", add ".*" to return success as lone as ret.Stdout() contains cmd.stdout
    c.Assert(ret.Stdout(), check.Matches, "(?s).*"+cmd.stdout+".*",
        check.Commentf("[FAIL]: CMD[%s %s] return %d,\n"+
            "Stdout: %s\n Stderr: %s",
            cmd.binary, cmd.args, ret.ExitCode, ret.Stdout(), ret.Stderr()))
... value string = "" +
...     "Arch:            amd64\n" +
...     "BuildTime:       2017-12-01T17:24:10.494219447+08:00\n" +
...     "GitCommit:       \n" +
...     "GoVersion:       go1.8.3\n" +
...     "KernelVersion:   \n" +
...     "Os:              linux\n" +
...     "Version:         0.1.0-dev\n" +
...     "APIVersion:      1.24\n"
... regex string = "(?s).*xxxxAPIVersion.*"
...
[FAIL]: CMD[/usr/local/bin/pouch version] return 0,
Stdout:
Arch:            amd64
BuildTime:       2017-12-01T17:24:10.494219447+08:00
GitCommit:
GoVersion:       go1.8.3
KernelVersion:
Os:              linux
Version:         0.1.0-dev
APIVersion:      1.24

 Stderr:

OOPS: 0 passed, 1 FAILED
--- FAIL: Test (0.10s)
FAIL
exit status 1
FAIL	github.com/alibaba/pouch/test	0.197s

2.Does this pull request fix one issue?

3.Describe how you did it
4.Describe how to verify it
5.Special notes for reviews

GoVersion:       go1.8.3 still exists in repo. It is strange 🤔
I think we have updated that in master?

@allencloud I build pouch on my own test machine, maybe it is related to my ENV?

Ok, Please upgrade your golang version when you have time. 😄

LGTM

@allencloud Thanks a lot~


ISPN-5743 + ISPN-5753 Soft-index file-store bugs

https://issues.jboss.org/browse/ISPN-5743
https://issues.jboss.org/browse/ISPN-5753

Wait a sec, store.size() does not work properly for expired entries.

Fixed store.size() behaviour, ready for review (please label).

@gustavonalle Would you mind taking look on this since you've reviewed the original PR? These fixes should be applied to 7.2 and 8.0 branches too, when merging please apply that there too (or I will file separate PRs once this is reviewed).
Btw., in February you've asked me to speed up clean cache restart - I've implemented that ( https://github.com/rvansa/infinispan/tree/ISPN-5782 ), I'll file PR for that after this is merged.

@rvansa this looks great. I will take a look on Mon.

I've found few places which caused these exception, but stumbled upon a problem with Compactor I can't find an easy/elegant solution for yet.

@rvansa The soft-index test suite is hanging.

@danberindei I am aware of some problems, I'll have to work on this fix more...

I'll close this for now to not delay CI when I now about some shortcomings.


Removed CRD from all index.md files

Issue #, if available:
Description of changes:
Some of the shasums were wrong. We decided it'd just be better to remove the CRD yaml and point out more clearly where people can get this information
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

I like this change, but what ever happened to that ticket to generate this info in a more readable format?

I like this change, but what ever happened to that ticket to generate this info in a more readable format?

@TerryHowe Do you want it be like this?

Name
Version
URI

coredns
v1.8.3
public.ecr.aws/eks-distro/coredns/coredns:v1.8.3-eks-1-20-1

etc.
etc.
etc.

If not, what did you have in mind?

I like this change, but what ever happened to that ticket to generate this info in a more readable format?

@TerryHowe Do you want it be like this?
Name	Version	URI
coredns	v1.8.3	public.ecr.aws/eks-distro/coredns/coredns:v1.8.3-eks-1-20-1
etc.	etc.	etc.
If not, what did you have in mind?

I like that format. How about it @jaxesn

Yeah that looks good to me.  Can we automate that generation?

Working on having this be generated so I don't have to fix all these manually, and we don't have to deal with this in the future either

/retest

This should be it.


avoid clock_gettime syscall by use vDSO

While analyzing just TechEmpower app I discover many clock_gettime syscalls.
We in mormot have the same problem some times ago, and solves it by linking with libc to use vDSO (Linux subsytem what expose some of kernel level functions to user space)  - see https://github.com/synopse/mORMot/pull/281/files
Hope it helps you to improve (already amazing) TechEmpower results

thanks. i think those calls are coming from v8. VDSO should be in use for all those calls. call overhead from JS is only ~12 nanoseconds with a v8 fastcall.


Edited the title following the issue on Mistake in the title of Project Euler Problem 283

Checklist:

[x] I have read freeCodeCamp's contribution guidelines.
[x] My pull request has a descriptive title (not a vague title like Update index.md)
[x] My pull request targets the main branch of freeCodeCamp.
[x] I have tested these changes either locally on my machine, or GitPod.

Closes #XXXXX

Hey @najmieh
title of challenge needs to be changed in the meta.json file as well - https://github.com/freeCodeCamp/freeCodeCamp/blob/main/curriculum/challenges/_meta/project-euler/meta.json - as it can be seen in the details how challenge can be renamed - https://contribute.freecodecamp.org/#/curriculum-file-structure?id=renaming-a-challenge. Note that with this change dashedName stays the same.

Hey @najmieh
title of challenge needs to be changed in the meta.json file as well - https://github.com/freeCodeCamp/freeCodeCamp/blob/main/curriculum/challenges/_meta/project-euler/meta.json - as it can be seen in the details how challenge can be renamed - https://contribute.freecodecamp.org/#/curriculum-file-structure?id=renaming-a-challenge. Note that with this change dashedName stays the same.

Ok, I edit that as well. Thank you for the review.

@gikf can you review the changes and tell me if everything is alright. Thank you in advance.

@najmieh I don't see any new change in this PR, did you add it to the correct branch?

@najmieh I don't see any new change in this PR, did you add it to the correct branch?

I added a new PR :/

Congrats on your first pull request (PR)! 🎉
Thank you for your contribution to the page! 👍 We are happy to accept these changes and look forward to future contributions. 📝

Thank YOU ^^ :)
I appreciate it.


[feature] wagmi, viem v2 support

What problem does this new feature solve?
wagmi, viem v2 support
Describe the solution you'd like
Hi, as the main repository of web3modal has released alpha, which supports wagmi v2, need some support as well.
I'm concerned about potential issues with the operation of wagmi v1.
Thanks!

Hi, @glitch-txs. Sorry to mention you.
https://docs.walletconnect.com/web3modal/v2/about
This document indicates that web3modal v2 is deprecated.
Is there any update regarding support for wagmi v2?
Thanks

Oh that refers to the Web SDK, React Native one is not deprecated :)

We will support Wagmi v2 eventually, I think Nacho is currently working on other prios

Oh, I see. That's great news!
Thank you for confirming.

@glitch-txs Is there any update?

I'm interested in contributing to this feature. Any suggestions on the tasks needed to be done to achieve this?

Hello, any updates refarding support for wagmi v2 ?

Hey @r-khr @vrakovs-abc @ayaamdouni @judeProground we released an alpha version for wagmi v2:
yarn add npm install @web3modal/wagmi-react-native@alpha wagmi viem @tanstack/react-query @walletconnect/react-native-compat
Here are the alpha docs:
https://alpha-docs.walletconnect.com/web3modal/react-native/about
And also check the new email wallets section 🖖

Awesome! Works for me, thank you @ignaciosantise


feat almost all models, implemented State Listeners, some refactoring

What did you fix?
closes #12312312

Reproduction steps

Evidence/screenshot/link to line

Considerations

Does this contain a new dependency? [Yes]
Does this introduce opinionated data formatting or manual data entry? [Yes]
Does this pr include updated data files in a separate commit that can be reverted for a clean code-only pr? [Yes]
Have I run the linter? [Yes]
Is is a bug fix, feature request, or enhancement? [Everything]

um, lol, no? if you want me to review something, i'm happy to, but i'm not just gonna "look at the code" for a huge 1500 line change that doesn't have any cohesiveness

there you go xD


add ODH-ADR-0001 first draft

add ODH-ADR-0001 first draft
we still need to finalize an approval flow, but I want to get a skeleton file in here first

we don't have an approval process yet, so I'm going to merge this and we can make follow up PRs as needed


W3Schools

I was really glad to have found your repo where you have mentioned so many free links . Thank You. Yet, I thought of adding a little new link to you repo which has helped me and I am sure many other people as well.

Thanks for the PR. However, we want specific resource to a topic. Not a general website for everything.


Fix Unix ioctl call and exception text in waitForChange

ioctl call TIOCMIWAIT expects the flags as an immediate parameter, not a pointer.

@rhulme can you point to any documentation on this? I can't find anything specific about this. The only thing I can find is this on the ioctl man page:

The third argument is an untyped pointer to memory.  It's traditionally char *argp (from the days before void * was valid C), and will be so named for this discussion.

-- https://man7.org/linux/man-pages/man2/ioctl.2.html

@rhulme can you point to any documentation on this? I can't find anything specific about this. The only thing I can find is this on the ioctl man page:

The third argument is an untyped pointer to memory.  It's traditionally char *argp (from the days before void * was valid C), and will be so named for this discussion.

-- https://man7.org/linux/man-pages/man2/ioctl.2.html

@wjwwood : ioctl is a variadic function.  It's up to the function itself to decide how many parameters it expects after the fixed parameters (in this case two) and what type they are.  So the same way printf decides how to interpret the parameters that come after the format string.
As ioctl is a general-purpose interface for pretty much any kind of driver, the number and meaning of the parameters will vary from driver to driver (or at least between classes of driver).
The commands and parameters for the TTY class are described here:
https://man7.org/linux/man-pages/man4/tty_ioctl.4.html

TIOCMIWAIT
         Argument: int arg

Just pinging this as the PR is still marked as "awaiting feedback" but I provided feedback in June 2021.


Carrie updates

making day 3 stuff

some subsetting (filter, pull, select, mutate, rename)
some cleaning -  (rename, recode, case_when- might delete this- but wanted it just in case there was time)...
updating schedule
updating cheatsheet

updating workflows for day 3

This is basically ready but some rendering issues ... I am having trouble seeing the full log... but did see something about it struggling to find the image files for the manipulating data lecture... if we didn't render the other modules that might help. I'm a bit confused.

This is basically ready but some rendering issues ... The log says something about it struggling to find the image files (pivot.jpg) for the manipulating data lecture... if we didn't render the other modules that might help. I'm a bit confused as to why this would be the rendering issue now with this PR since we aren't modifying anything with that file.

@avahoffman  helped me realize I had this lecture in the folder :P -fixed that


feat(doc): Update documentation for the bonita docker image

Describe how to create a Postgres database to use with a Bonita docker

@danila-m why is this PR done on OUR fork, and not on the official docker repository?

@danila-m why is this PR done on OUR fork, and not on the official docker repository?

Look at the date.
It was I think to update the documentation, and required a review on our side, before pushing to the public repo.

Out of date


Allow token fn to return null

The token passed as a value can be null already, so the code that calls the function will continue to work as intended.
Allowing a null value makes it possible to still open a connection without a token while attempting to fetch the token in an async function.

@janthurau sorry about that, I think prettier kicked in when I commited the file and I didn't double check the diff after that.
I made a new PR 🙏 .


[Recommendation] Will RDKit add GetNeighborBond?

RDKit: 2020.09.1 - Windows 10
When I search in Chem.module, I found out that we have class Atom with GetNeighbors() to retrieve nearby Atoms? But I am wondering why RDKit don't add GetNeighbors in the class Bond to retrieve nearby Bonds
At the current time, I have to use this code to take nearby bonds (the main target of this code is too help searching for radicals that are part of cis-trans bonds to detect rotation).
I hope that
parentMol = Chem.MolFromSmiles('C/C=C/C)
parentMol = Chem.AddHs(parentMol)
for bond in parentMol.GetBonds():
      if str(bond.GetStereo())[6:] != 'NONE':
            CisTransBond.append(str(bond.GetStereo())[6:])
            CisTransIdx.append(int(bond.GetIdx()))
for i in range(0, len(CisTransIdx)):
      beginAtom = temp.GetBeginAtom()
      endAtom = temp.GetEndAtom()
      beginAtomNeighborsIdx = [int(neighborAtom.GetIdx()) for neighborAtom in beginAtom.GetNeighbors()
                                                   if int(neighborAtom.GetIdx()) != endAtom.GetIdx()]
      endAtomNeighborsIdx = [int(neighborAtom.GetIdx()) for neighborAtom in endAtom.GetNeighbors()
                                                if int(neighborAtom.GetIdx()) != beginAtom.GetIdx()]
      ............
      ............

Look at atom.GetBonds()

Look at atom.GetBonds()

Thank you for the help. However, the recommendation may be a new features to enrich molecule method
for i in range(0, len(CisTransIdx)):
     temp = newMol.GetBondWithIdx(CisTransIdx[i])
     a = [newBond.GetIdx() for newBond in temp.GetBeginAtom().GetBonds() if newBond.GetIdx() != CisTransIdx[i]]
     b = [newBond.GetIdx() for newBond in temp.GetEndAtom().GetBonds() if newBond.GetIdx() != CisTransIdx[i]]
     checkList = a + b


Element Attribute is not deleted when you go to another page

Vue version
3
Link to minimal reproduction
https://stackblitz.com/edit/github-bfob1d?file=src%2Fpages%2Fhome.vue
Steps to reproduce
There is something strange. It happens when I move from one page to another page that contains a specific component. Within this component, I give a data-Attribute element, and when I return to the first page and then return, I find that the Attribute has not been deleted.
Unlike usual, when you visit a page and, for example, give certain properties to HTML elements, after exiting the page I think they are deleted.
ex :
You Have 2 Pages  :

home.vue

<template>

Index Page

    <button @click="go">To About</button>

For Test :

    <button @click="setItem">Local Storage Set</button>
    <button @click="removeItem">Local Storage remove Set</button>

</template>

about.vue

<template>

About Page

    <TestComp />
    <RouterLink to="/">To Home</RouterLink>
    <button @click="getAttr">getAttribute (data-setData)</button>

</template>

in about page have testComp.vue :

<template>

</template>

Now when I go to the about page directly, everything is excellent, but when I go to the index page, then go to the about page, then return to the index page and put set in localStorage, then return to about, I find that the Attribute setData has not been deleted!
try it yourself, https://stackblitz.com/edit/github-bfob1d?file=src%2Fpages%2Fhome.vue
1 - open home page
2 - Go to the about page (click Button) (the element .notRemoveAttr will be given Attribute, which is data-setData)
3 - Go back to index (click Button)
4 - Click on (Local Storage Set) - so that .setAttribute('data-setData', 'hi') does not work when we return to the about page
5 - Go back to the about page (click on getAttribute (data-setData)) and you will find that it brings it to you!!
Make sure when you want to try again, click on Local Storage remove Set
What is expected?
I expect that when I leave the page and return to it, I find everything as if I had reloaded the page.
What is actually happening?
I gave an attribute element, but it is not deleted when you leave the page and return to it...
System Info
https://stackblitz.com/edit/github-bfob1d?file=src%2Fpages%2Fhome.vue

Any additional comments?
No response

Vue re-uses elements where it can, for performance reasons.
Vue cannot track any DOM manipulation that you do imperatively.

So as a consequence, it will reuse the div when switching the components, but not remove the attribute you added because it's not aware that it exists.
Solutions:
A: Remove the attribute yourself on unmount, or
B: add it with Vue in the template instead of using DOM APIs yourself.

Vue re-uses elements where it can, for performance reasons
Vue cannot track any DOM manipulation that you do imperatively through DOM APIs, anyway.

So as a consequence, it will reuse the div when switching the components, but not remove the attribute you added because it's not aware that it exists.
Solutions:
A: Remove the attribute yourself on unmount, or
B: add it with Vue in the template instead of using DOM APIs yourself.

Isn't there anything automatic, is there a way to make it re-render

See my edited comment, misunderstood your problem.

When you go from home back to about, the About component is mounted again, so it adds the attribute again. Works as you designed it to work.
1. Vue re-uses elements where it can, for performance reasons
3. Vue cannot track any DOM manipulation that you do imperatively through DOM APIs, anyway.
So as a consequence, it will reuse the div when switching the components, but not remove the attribute you added because it's not aware that it exists.
Solutions:
A: Remove the attribute yourself on unmount, or
B: add it with Vue in the template instead of using DOM APIs yourself.

No, it is not. I mentioned steps for you to try. Read them in the question at the end. The problem is, I set localeStorage so that it will not be executed again... Try that.

I see. Seems to be related to Suspense. Removing that makes it work.

I tried to remove the , but it still doesn't work as expected.
I think the reason for this is as @LinusBorg  listed, for , vue will consider it static and resue it. So the dom carry the attributes when entering About again.

The issue can no longer be reproduced in v3.5.12


addGeometry -> setGeometry

More inline with traditional get and set naming scheme

done in https://github.com/mapbox/gl-draw/commit/d7c7795644e58d391adafa966ff7119aa7ae1553


[RFE]  Validate that the new "Status" field cannot be changed by users with standard permissions.

related to Global Roles v2
SURE-6202

Assuming standard permissions means a Standard User, by default they do not have the ability to edit Global Roles. In fact, the only builtin global roles that give edit permissions to global roles are Administrator and Restricted Administrator. Even the Manage Users only gives get, list and watch for globalroles.
Users with edit permission can modify the status field, but it immediately reconciles and gets corrected. It is only extremely briefly incorrect. I don't think this issue requires new work, so I'll be closing it.


ModuleNotFoundError: No module named 'options'

Google colab returning error 'ModuleNotFoundError: No module named 'options'' while running

The error you get seems related to the directory you use to put the code there and the way you import modules.
The 'options' file is this one: https://github.com/ipc-lab/private-ml-for-health/blob/main/private_training/src/options.py

Thank you