Update to dropwizard 0.8

Rework the guice/Jersey linkage to use Squarespace's jersey2-guice.
I have not tested it extensively, but it worked fine when upgrading a number my own projects.
The only issue I saw was:
Sometimes the Guice based annotations got confused in the connection between Guice and HK2.  Switching com.google.inject based annotation to their java.inject counterparts solved the issues I ran into.
In the example app, the com.google.inject.name.Named imports needed to be converted to javax.inject.Named.

Great thx!
Another issue that occured is that JerseyUtil#registerGuiceBound registers my @Path annotated interfaces, not the resource implementing the interface. This works but it gives the following warnings in jersey: Component of class interface com.example.SomeInterfaceWithPath cannot be instantiated and will be ignored.". And also the DropwizardResourceConfig doesn't log the endpoints since the registered resources are interfaces.

Hi @oillio, I've created a PR to merge my unit tests into your code base: https://github.com/oillio/dropwizard-guice/pull/1
They are all passing, please check it out and see if you can make some use out of them. I plan to have a go using your approach at work tomorrow and test out the JIT support. Will write some more tests once I figure out how your linkers work.

There's a slight issue with running Integration Tests where the JerseyTest teardown() fails with Null Pointer Exception when trying to stop the jersey test container. See test:
https://github.com/yunspace/dropwizard-guice/blob/jersey2-guice/src/test/java/com/hubspot/dropwizard/guice/InjectedResourcesTest.java
This is exactly the same symptom as https://github.com/dropwizard/dropwizard/issues/828 where the user is also trying to do resource test with injections.

@yunspace I fixed the teardown issue and merged in your other work on tests.  Thanks for writing up the tests.
The issue was with Jersey dependencies.  jersey2-guice is targeting 2.13 and drop wizard upgraded to 2.15 for rc3.

@jontejj I looked into your @Path issue a bit.  This only occurs when you do something like the below, right?
bind(SomeInterfaceWithPath.class).to(ImplResource.class);

Due to how Jersey registers components, it would be a serious pain to make this work.  Is there a scenario where simply binding ImplResource.class itself is overly problematic?

Nope that could work but then we would have to have a way to disable the
automatic bindings?
On Jan 27, 2015 12:43 AM, "Dan Jasek"<EMAIL_ADDRESS>wrote:

@jontejj https://github.com/jontejj I looked into your @Path
https://github.com/Path issue a bit. This only occurs when you do
something like the below, right?
bind(SomeInterfaceWithPath.class).to(ImplResource.class);
Due to how Jersey registers components, it would be a serious pain to make
this work. Is there a scenario where simply binding ImplResource.class
itself is overly problematic?
—
Reply to this email directly or view it on GitHub
https://github.com/HubSpot/dropwizard-guice/pull/45#issuecomment-71562619
.

I added a check to AutoConfig and JerseyUtil to avoid registering resources that won't be accepted by Jersey.

Great job!
I added a check to AutoConfig and JerseyUtil to avoid registering resources
that won't be accepted by Jersey.
—
Reply to this email directly or view it on GitHub
https://github.com/HubSpot/dropwizard-guice/pull/45#issuecomment-71596628.

Thanks @oillio for fixing the ResourceTest teardown
However after I added an IntegrationTest I was able to reproduce this error again. The weird thing is, when running the test individually it works fine. But when running in a test suite with other tests it throws NullPointerException again. See: https://github.com/yunspace/dropwizard-guice/blob/jersey2-guice/src/test/java/com/hubspot/dropwizard/guice/InjectedIntegrationTest.java

I found the solution, but it isn't pretty.
It is related to the ugly reflection jersey2-guice uses.  Anytime JerseyModule is used in a test, jersey2-guice needs to be reset using BootstrapUtils.reset().  I added the necessary tear down code to the existing tests that need it.
In the future, we will probably want to subclass DropwizardAppRule and ResourceTestRule to make this easier for users.

@oillio Wow you are fast. This work around solves the issue.

Thanks for the contribution @oillio @yunspace and everyone else who commented, this was released as version 0.8.0


[log] suggest logging system

Subject
logging system
Proposal Detail
How about use logging library? I think it is important to log when use network (Especially, in p2p system!)
Maybe it is future work... i suggest some logging library!
Reference

https://github.com/sirupsen/logrus

https://github.com/uber-go/zap

https://github.com/gogap/aop
This is one of concept of spring : AOP (Aspect-Oriented Programming). I used this usefully when debug. just suggestion :)

Log format
suggest
log format
{
  # common
  level: string <"error" | "warn" | "info" | "debug">
  comp: string <"hb" | "acs" | "te" | "rbc" | "bba">
  msg: string
  ts: timestamp (YYYY-MM-dd HH:mm:ss)

  # metadata
  ...
}


Improve link to article

Please improve the link to the Horizon paper in the repo's readme. For reference, the specific link is https://research.fb.com/publications/horizon-facebooks-open-source-applied-reinforcement-learning-platform/

Will do, thanks!


[HOPS-1557]

Make sure there is no duplicate PR for this issue

Please check if the PR fulfills these requirements

[ ] Tests for the changes have been added and passed (for bug fixes / features)
[x] HOPS JIRA issue has been opened for this PR
[x] All commits have been squashed down to a single commit
[ ] The commit message has the following format: [HOPS-XXX] message

Post a link to the associated JIRA issue
https://hopshadoop.atlassian.net/projects/HOPS/issues/HOPS-1557

What kind of change does this PR introduce? (Bug fix, feature, docs update, ...)
bug fix

What is the new behavior (if this is a feature change)?
 (What changes might users need to make in their application due to this PR?)

Other information:

Jenkins test this please.

Jenkins test this please.


Fully implement trainers.

The base files are there, but trainers need to be properly implemented.

Hi there,
I have been using your atan framework for a few months now and I now need to use the Trainer. Have you been able to implement it yet? If not, can you please give me some tips as to how to go about implementing myself and compiling it with Maven.
Thank you
Daniel


[Quantization]: Update simulated_quantize to infer correct layout

In our BYOC flow, we invoke the Convert Layout pass after simulated_quantize annotations are inserted in the IR.
However, after Convert Layout, we observe layout transforms to the original layout are inserted before the simulated_quantize annotations, as the compiler is currently unable to infer the layouts from the input.
This PR allows propagation of the input layouts to avoid the insertion of additional layout transforms.

cc: @junrushao @masahi @vinx13


[SDPV-866,849] ES6 and sonar updates

Code to upgrade elasticsearch image to 6+ (tested on 6.6)

Your current tests are not testing with the elasticsearch 6.6 image.  I have a branch running tests right now (new-esver) that uses the new image.  Once that passes, I'll issue a merge request.


Update examples/glusterfs to v1beta3; add desciption of Glusterfs endpoints in README.md

Update examples/glusterfs to v1beta3; add desciption of Glusterfs endpoints in README.md.
@rootfs @nikhiljindal

fixes #8214

thanks for the v1beta3 endpoints example.

LGTM % a minor typo.

Fixed. Thank you @nikhiljindal.

Updated. Thanks. @nikhiljindal

LGTM, thanks!

Shippable is green. Merging


Distributed channel - config-kafka reloading

Fixes #455

Proposed Changes

Mount config-kafka to controller, dispatcher and receiver
Don't watch config-kafka in dispatcher and receiver
Make changes in the controller so that it does a redeployment of dispatcher and receiver when config-kafka changes

Release Note

- 🧽  Distributed Kafka Channel dispatcher and receiver are now reconfigured when `config-kafka` configmap is updated

Docs

/ok-to-test

/retest

@eric-sap
Pushed new commits that address the 3rd part in the PR description.
Sorry the PR got big now :(
I also did 1-2 things on the consolidated channel code in this PR, since I found some trivial problems when I compared the code with distributed channel.

/hold
I haven't done the manual tests yet

/unhold
I tested my changes with https://github.com/aliok/kafka-ch-reload-tests. There are 2 pods:

sender sends messages for 2 mins
receiver receives messages for 3 mins with a message frequency of 10 ms

There were no messages lost and no messages undelivered. Also no duplicate messages. I don't get how that's possible actually.
@eric-sap would that be expected? The channel would be unavailable for a few milliseconds in the consolidated channel when config-kafka is changed and thus sender can't send a few messages.
This doesn't happen here in distributed channel.
Perhaps, we're reporting the consolidated dispatcher ready before it is actually ready.
@eric-sap I would use some help in manual testing if receiver deployment is updated when config-kafka is changed. Any instructions would be nice to make the reconcilers create a receiver deployment.

@eric-sap would that be expected? The channel would be unavailable for a few milliseconds in the consolidated channel when config-kafka is changed and thus sender can't send a few messages.
This doesn't happen here in distributed channel.

I haven't looked at all of the code here yet, but ideally when a deployment is changed, Kubernetes starts the new pod and does not switch the target or terminate the old pod until the readiness check says that the new pod is okay.  In the distributed channel's case, that means calling health.SetDispatcherReady() (in the dispatcher's main.go) and the SetChannelReady/SetProducerReady calls made after the receiver's components are created.  Theoretically this means that the instant the pod is "ready" it really can begin processing messages (and local strimzi testing has borne this out).
That said, I have seen duplicate messages happen during rollout restarts when using an external kafka broker and not local strimzi.  Our guarantee is only that no messages will be lost (that is, if a client receives a 202 from the receiver, that means the message will be delivered "at least one" (or retried until final failure and possible redelivery to a dead-letter-queue if configured).

/hold
Receiver is not restarted. Having a look now

/unhold
Thanks @eric-sap.
I pushed the code change that fixes global resync in kafkaSecret controller

I've tested this locally and it seems to do what it's supposed to.  There are still some linter errors but other than that it looks good to me.

Linter error is about a deprecated dependency in test images. We need to fix it, but it is irrelevant to this PR.
@travis-minke-sap can you have a look and lgtm it?


Authenticating on a headless system

I thought it was addressed on 4.4 and on but couldn't find any notes on it.  I have a headless system (rpi) that I run this on for my scrolling Marquee and had not managed to need to authenticate since I upgraded from 4.3 to 4.5, until this evening.  If a note about this could be added to the documentation that would be awesome.  The --noauth_local_(web)server flag was not working... may have had it in the wrong spot... so I just followed the prompts as one would from normal authentication up until the opening a web browser part.  Once it paused I opened the link it gave me, followed the prompts then it tried to open the http://localhost:port/auth/code link.  Naturally that didn't work and attempting to redirect to the remote IP, my case was internal, didn't either.  I copied the link that was given in the local browser, ssh'd a second session into the remote machine, and curl "http://localhost:port/auth/code". After making sure I used the quotes it authenticated as expected.

Hmm, did you try the --noauth_local_server option on 4.5? That should walk you through headless setup. I guess I could add a placeholder --noauth_local_webserver back as well that does the same... the options went away when upgrading off of oauth2client that Google deprecated and it wasn't transparent exactly which options disappeared that people would miss.
Besides port forwarding, you're not going to be able to monkey with the URLs to get something to work. Your browser needs to be able to hit the target URL directly to transfer the token.
Dupe of fixed #734, I believe, apart from any additional documentation you might be asking for.

I guess as far as docs go, one piece that probably needs an overhaul is the stuff in the README about env vars like http_proxy: https://github.com/insanum/gcalcli?tab=readme-ov-file#http-proxy-support.
Pretty sure those haven't worked in ages but I still never figured out exactly what they were referring to so I wasn't sure exactly what updates to make there. They just appeared in 770393f. @insanum do you have any pointer on those?

Yeah not sure why the --noauth_local_server didn't work, I used to have a script run it …

To be clear, it's not going to just work exactly like it used to, the behavior has changed. If you're running it deep inside a script then you might just not have noticed the new output, but the gist is that Google deprecated the old behavior and the command now prints some detailed instructions for SSH port forwarding, which is the best available approach right now.

Ah ok that makes sense, totally glossed over that output until now.  Since I do not have a browser installed on the pi I would venture to add a new Option 1 (preferred?): Open https://the-url-below in your local browser and follow prompts until it opens http://localhost:$port/?... (this will fail).  Copy the URL from your browser into a second ssh session and 'curl "http://localhost:$port/?..."'  This just takes any port forwarding out of the equation.

I'm open to ideas for better workarounds but as far as I know if you're not port forwarding you're doing something even more brittle and complicated. The web browser needs to be able to open a connection to the target device and port where gcalcli is running. If they're on the same network and can access each others' IPs and ports are open then you could replace localhost with another address without needing forwarding, but that's almost never going to "just work".
IIUC the reason it was able to work more seamlessly before is that you had remote Google servers helping bridge communication between the two independent devices (the "Out-Of-Band flow" and similar), but Google decided they don't want to support that anymore for some security risk reasons.

Yeah, that was one of the first things I tried, using the IP and the Domain Name, but I am guessing my pi probably had the port blocked off in the firewall rules because it still failed.  That is where I figured the curl command might work since it's just an API call, I use curl to make adjustments to my sound systems linked between my office and garage.  curl stands in for the web browser since the data being passed back to the gcalcli is just contained in the URL. Everything after that "?" is just JSON structured API data right?
URL provided by gcalcli init:
https:// accounts.google.com/o/oauth2/auth?response_type=code&client_id=########################################&redirect_uri=http%3A%2F%2Flocalhost%3A_port#_%2F&scope=https%3A%2F%2Fwww.googleapis.com%2Fauth%2Fcalendar&state=###########################&access_type=offline
redirect_uri from google with code:
http:// localhost:port#/?state=#######################&code=#################################################################################&scope=https://www.googleapis.com/auth/calendar
... this doesn't return anything graphically does it, since its just the python script waiting for some JSON data to be passed? I mean curl didn't show anything I just got the "Successfully loaded credentials" on the first ssh session.
Sorry if this is getting long winded but just had a whole other brain shift...  Since the init provided URL doesn't have to run on the headless machine and redirect_uri is where Google redirects back the code to the gcalcli webserver, versus the old out of bounds configuration, can the --noauth_local_server option wait for a paste of that URL that is returned from google and handle the curl, parse the pasted URL for the code, or have the end user just copy the code form the URL?
By the way still, thank you again for all the work you have done on this @dbarnett


Safety Check

We want to ensure that refugees arrive safe at their temporary home, so we need a safety check for the Guest and Host.
Current Idea:
When the guest arrives at the hosts place, both parties confirm the safety check on the website. If the safety check is not confirmed within 24 hours, we will get in touch with both parties.
Future Ideas:

Add additional safety checks after x days / weeks?
Additional questions to check the correctness of the place offering

I would currently suggest two safety checks:

within 48h to make sure the people are well and arrived at their place of destination
within one to two weeks to ensure everything stayed in good condition and the hosts / guests behave good.

For now I only implemented the first safety check


Informational Meeting - Tracking Microsoft Purview

This meeting was created by Phani Kumar Vadali.

It will be used to Track the conversation in the informational Session for the Microsoft Purview Service.

Detailed meeting information and documents provided can be accessed here

Duplicate #19807


'pkg-config: command not found' during brew install, but not manual configuration

First off, forgive me if I'm missing something obvious, or this doesn't technically qualify as an issue – I wasn't sure where else to ask.
I'm trying to install elinks with ECMAScript support. I'm not sure this is relevant, but I did a little bit of tweaking in brew edit elinks to get it to download the latest experimental build, 0.13 (rather than 0.12pre6, which is what's on tap).
Configuring manually
To make sure everything would compile correctly, I first manually downloaded the source myself, running ./configure and then make. It didn't recognize my SpiderMonkey install, as evidenced by the following ./config output:
checking for pkg-config... /usr/local/bin/pkg-config
checking pkg-config is at least version 0.9.0... yes
...
checking for SpiderMonkey (1.8.5 or later) in pkg-config mozjs185... no

To fix that, I set the following environment variables,
export PKG_CONFIG=pkg-config
export PKG_CONFIG_PATH=/usr/local/Cellar/nspr/4.12/lib/pkgconfig:/usr/local/Cellar/spidermonkey/1.8.5_1/lib/pkgconfig

which got it to work:
checking for pkg-config... /usr/local/bin/pkg-config
checking pkg-config is at least version 0.9.0... yes
...
checking for SpiderMonkey (1.8.5 or later) in pkg-config mozjs185... yes

Homebrew install fails (to find pkg-config)
But then when I go to brew install --devel -v elinks, I get the following output:
checking pkg-config is at least version 0.9.0... ./configure: line 4211: pkg-config: command not found
no
...
checking for SpiderMonkey (1.8.5 or later) in pkg-config mozjs185... no

Oddly, the output from Homebrew's configuration skips the 'checking for pkg-config' line:
checking for sparse... no
checking pkg-config is at least version 0.9.0... ./configure: line 4211: pkg-config: command not found
no

To my knowledge, I'm working with the exact same source tarball. Any ideas?

Homebrew is designed not to allow packages to find other software that wasn't specified as a dependency; you'll need to add these lines to the dependencies for it to work:
depends_on "pkg-config" => "build"
depends_on "spidermonkey"

holy balls, that was fast. you're amazing!

depends_on "pkg-config" => :build

Just to clarify 😺.


Stats, crash detection, misc.

Hey Kyle!
This is awesome -- I'll go through it as soon as I get a chance and we can take it from there. I love all of your input and work going into this.


PRs reviews and approvals speed

Good evening owners and maintainers,
Would it be possible to speed up the review to the new PRs. That would significantly help with the closing/merging open contributions.
For example, the oldest opened PR was given review comment almost 3 months after opening.
Would be adding CODEOWNERS file to this repository help? https://help.github.com/en/github/creating-cloning-and-archiving-repositories/about-code-owners

@florina-muntenescu thank you for picking up it. I will run over old issue and PRs to check if they are still relevant.

Thanks for bringing this up. Yes we're picking up reviews and working our way through the backlog.


feat(keybindings): add binding for opening config

Listed in #1423 as a good place to get started as adding a keybinding for the "Open Settings" shortcut. Hope it's okay!

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


🍒 Cherry pick PR #5063 to staging 🍒

🍒 Cherry pick https://github.com/Expensify/App/pull/5063 to staging 🍒

🚀 Cherry-picked to staging by @OSBotify in version: 1.0.93-1 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅

@Expensify/applauseleads please QA this PR and check it off on the deploy checklist if it passes.

🚀 Deployed to production by @roryabraham in version: 1.0.93-1 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅


No support for D4J

You could use BotBlockRequests.postGuilds("id", 100, auth); since it doesn't use JDA for this method.


Revamp Splash Screen

Description of Problem or Question
I assume it would be possible to import the App splash screen images and place them as a background to the main screen? It would require moving some of the text and buttons but I think it would be worth it for the good initial impression.
Something like this maybe?
MoM2

Descent

I could find the images as .dds files for MoM in:
C:\Users\username\AppData\Roaming\Valkyrie\MoM\import\img\
But couldn't find the Descent splash, in its similar directory. But I assume it has to be in the content files somewhere.

Some options if we don't want to use the official artwork:
(Using Royalty Free Art)
MoM2:

Descent:

I really like the Royalty free for Descent.
For MoM, I like the design and color, also the evil crow :)
Just to mansion looks more like a castle IMO.

I agree, I liked the whole thing except the castle aspect. But looks pretty cool and it's hard to get clearly defined royalty free stuff without paying for stock.

For MOM splashscreen there are some nice ones on https://wallpapercave.com/haunted-house-background. Examples (high-ress downloads available on the site):

And more...
The site suggests that all images are free of copyrights: They use the strategy that anyone can upload as long as it complies to police and rules.... and it will stay on the site as long a s nobody complaints.

The second mansion is the best IMO.
Nice find @antontimmermans :)

If the images from  https://wallpapercave.com are ok I would prefer this one for descent:

https://wallpapercave.com/w/uwp10758

That's very nice

I'm really not sure about the licensing situation here.
It seems nearly impossible to find the original source of the images even when using search engines like this one:
https://tineye.com/search/3112d12ff7139e601fe5bfe24d02a057ec61f6ef?sort=crawl_date&order=asc&page=2

I don't want to merge anything that doesn't have clear permission. Legally no licensing information means it cannot be used.

That's the original on the page of the author (checked the magic card there to see the name)
https://www.deviantart.com/sandara/art/Temeraire-66213180
Doesn't seem to be free of right though so we can't use it.


test: fixed fscloud ex to scope KMS rule to instance id

Description

Fixed fscloud example to scope KMS rule to instance id.
Added prefix to the IP zone name created by the FSCloud example.

Release required?

[ ] No release
[x] Patch release (x.x.X)
[ ] Minor release (x.X.x)
[ ] Major release (X.x.x)

Release notes content

Run the pipeline
If the CI pipeline doesn't run when you create the PR, the PR requires a user with GitHub collaborators access to run the pipeline.
Run the CI pipeline when the PR is ready for review and you expect tests to pass. Add a comment to the PR with the following text:
/run pipeline

Checklist for reviewers

[ ] If relevant, a test for the change is included or updated with this PR.
[ ] If relevant, documentation for the change is included or updated with this PR.

Merge actions for mergers

When merging, use a relevant conventional commit message that is based on the PR contents and any release notes provided by the PR author. The commit message determines whether a new version of the module is needed, and if so, which semver increment to use (major, minor, or patch).
Merge by using "Squash and merge".

/run pipeline

/run pipeline

/run pipeline

/run pipeline

/run pipeline

/run pipeline


Using pip-compile to generate full requirements

Right now I'm not entirely sure what the status of the requirements.txt file actually is. It looks like we're trying to keep dependencies locked so that an update of a dependency won't inadvertently break the project.
Manually ensuring that each dependency is up to date can be a bit cumbersome especially if you commit to keeping track of transitive dependencies. I think a step in the right direction would be to use pip-compile, that way you'll only need to keep track of and update the top-most dependencies.
https://github.com/jazzband/pip-tools/

Currently taking a look at pip-tools, being able to list app dependencies only, and having comments where transitive dependencies come from should make things easier. I guess dependabot should still be able to work with this setup, and will continue to suggest changes to the requirements files directly.

Something like this: https://github.com/sissbruecker/linkding/pull/618

Yep, dependabot works just fine with a setup revolving around pip-tools. That's what I use on any Django project I work on. Example dependabot PR: https://github.com/joshuadavidthomas/joshthomas.dev/pull/250


proxy parameter in typescript not allows domains population

Hi,
I'm using the version 1.12.1 with typescript and the function "proxy"  doesn't work as expected or maybe I'm using it in the wrong way but I think that problem is due to the definition of type in the interface Sysend.d.ts
proxy(args?: string[]): void;
that have an array of string as param but in the library the function proxy need a string as url
`proxy: function() {
        [].slice.call(arguments).forEach(function(url) {
            if (typeof url === 'string' && host(url) !== window.location.host) {
                domains = domains || [];
                domains.push(origin(url));

....
}`
so if I use proxy in this way
sysend.proxy(["url1","url2"])
the
 [].slice.call(arguments)
give me an array as foreach element and the if condition is never true.
It's no possible to pass other params to the proxy function to have the urls in arguments  because of typescript definition so the only workarond that I have found is to use
sysend.proxy([]).bind(null,"url1","url2")()
to have a true condition but you know is not the best one :)
thanks for help ;)

Yes, good catch it should be proxy(...args: string[]): void;

fixed in version 1.12.2


Refactor TaskSocket

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

Project coverage is 80.24%. Comparing base (5937b88) to head (9de4d51).
Report is 98 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #363      +/-   ##
==========================================
+ Coverage   75.75%   80.24%   +4.49%
==========================================
  Files          70       67       -3
  Lines        4615     5194     +579
==========================================
+ Hits         3496     4168     +672
+ Misses       1119     1026      -93

Flag
Coverage Δ

python-3.11
80.24% <100.00%> (+4.58%)
:arrow_up:

python-3.9
?

Flags with carried forward coverage won't be shown. Click here to find out more.

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


Answer Added

@Ntombi20
Check Me!

@radlee
What is the answer lee?


Comments Added

Commets added to rewiring script, command terminal, reactor terminal and turret terminal scripts.

Alex says no!


Improvements to portability, remove "using namespace std," and a little templating. (Plus an integer conversion you might not want.)

Hi Leopold! I'm Riley; currently staff at Sandia. I forked your repo earlier this year so I could run experiments with spaND and poke around in its internals. Nothing has some of these experiments and poking around, but I did put in some effort to improve spaND's portability. I'm opening this PR in case you'd like to discuss merging these changes in. You totally don't have to! If you just leave the PR open indefinitely that'll help people who come by and have similar issues that I did.
Anyway, here's a brief overview of the changes

I changed the BLAS and LAPACK backend from MKL to portability layers called "BLAS++" and "LAPACK++." This was necessary to run spaND on my macbook, which is based on Apple Silicon. It required changes in many places to handle differences in BLAS and LAPACK enums (which were specified as chars, but are now abstract).
I changed the default integer from "int" to "int64." This may have been an overzealous change. If this is the only change that disqualifies this PR then I can try to reverse them and/or open a new PR that doesn't have this change.
I changed some pivoting-related utilities to be agnostic to the integer width. This involved moving implementations from src/util.cpp into include/util.h.
I removed instances of using namespace std; and added explicit std:: namespace qualifications where need. This took a while.
I moved the cg implementation from a cpp file into a header file so I could template some datatypes.
I added comments to the implementation of Tree::sparsify_adaptive_only.
Switched the C++ standard from 14 to 20, for some reason? I suspect there's a way around this.
I added a command line argument to spaND that can be used to stop the low-rank approximations at a specified level (rather than only the final level).
I added googletest as a git submodule. This makes it possible for someone to handle this dependency with git clone <spandURL> --recursive, rather than tracking down googletest on their own.

Hi Riley. Thanks a lot for the PR. I will take a bit of time to review it but very much open to it. Happy to see this ended up being useful to someone else. Give me a few days and I'll get back to you.
I'd be curious to learn me about your use case if you're open to it (you can find my email on my website, linked in my Github bio).

I need to do a few more passes, but overall LGTM. The int64 conversion is good.

Hi @leopoldcambier. Thanks for the review! Sorry for the delay in responding to it. I've implemented some changes, but more remain. I'll get to them soon(ish ;)).
As for what got me interested in the project -- I've been working on faster algorithms for QR decomposition with column pivoting. I figured it would be cool if I could make connections to applications that DOE really cares about, and solving sparse linear systems is definitely one such application.
In the end I wasn't able to test out my QRCP algorithm. I just didn't have the time. I did verify that under certain parameter regimes, QRCP can be the dominant cost. I also explored empirical properties of the matrices that you're decomposing by QRCP (IIRC, they're at most on the order of a few thousand, and usually non-square).

Hi @leopoldcambier. Thanks for the review! Sorry for the delay in responding to it. I've implemented some changes, but more remain. I'll get to them soon(ish ;)).
As for what got me interested in the project -- I've been working on faster algorithms for QR decomposition with column pivoting. I figured it would be cool if I could make connections to applications that DOE really cares about, and solving sparse linear systems is definitely one such application.
In the end I wasn't able to test out my QRCP algorithm. I just didn't have the time. I did verify that under certain parameter regimes, QRCP can be the dominant cost. I also explored empirical properties of the matrices that you're decomposing by QRCP (IIRC, they're at most on the order of a few thousand, and usually non-square).

Cool thanks. No problem - the code isn't going anywhere, and I'm not touching it anymore :-)
Will look at the other changes soonish, but it might take me some time.
Thanks for sharing the application.


Replace an element defined within the config-file

Hi,
If I want to replace an element defined within the config-file how can I do that. Below one is not working for android. It's get appended to the same element
<config-file target="AndroidManifest.xml" parent="application"> <meta-data android:name="com.facebook.sdk.ApplicationId" android:value="@string/fb_app_id"/> </config-file>

Hi, can you provide me a simple cordova project that produce this issue ? I'm not able to reproduce it.
Thanks!

@mircoc I'm running into the same issue.  I'm trying to add an activity-alias to my AndroidManifest.xml file and the parent it set to "/manifest/application" but when I try to build the project, I get an error in my output stating
Processing settings for platform: android
{ [Error: Cannot use absolute path on element] token: null, message: 'Cannot use absolute path on element' }

It looks like it still might add the entry, but I'm not sure why the error is being thrown.  Here's the logic in my config.xml file.
<platform name="android">
      <resource-file src="res/cs_icon/fc_icon_3322.png" target="app/src/main/res/drawable/fc_icon_3322.png" />

      <config-file target="AndroidManifest.xml" parent="/manifest/application">
            <activity-alias android:configChanges="orientation|screenSize" android:noHistory="true" android:name=".MainActivity_3322" android:enabled="false" android:icon="@drawable/fc_icon_3322" android:label="@string/app_name" android:targetActivity=".MainActivity">
                    <intent-filter>
                        <action android:name="android.intent.action.MAIN" />
                        <category android:name="android.intent.category.LAUNCHER" />
                    </intent-filter>
                </activity-alias>

                <!-- possibly needs to be the last entry??? -->
                <activity-alias android:enabled="true" android:icon="@mipmap/ic_launcher" android:label="@string/app_name" android:name=".MainActivity_default" android:targetActivity=".MainActivity">
                    <intent-filter>
                        <action android:name="android.intent.action.MAIN"/>
                        <category android:name="android.intent.category.LAUNCHER"/>
                    </intent-filter>
                    <intent-filter>
                        <action android:name="android.intent.action.VIEW"/>
                        <category android:name="android.intent.category.DEFAULT"/>
                        <category android:name="android.intent.category.BROWSABLE"/>
                        <data android:host="www.faithconnector.com" android:scheme="http"/>
                    </intent-filter>
                </activity-alias>
            </config-file>
        </platform>

If I change the config-file parent to "manifest/application", it doesn't insert into the proper location.  It looks like it needs to have the slash in front of it.
Thanks
Adam

I might have found a workaround, at least for my case in need to manipulate the application tags inside the AndroidManifest.xml.  In looking at the updatePlatformConfig.js code for the plugin, I noticed when preferenceMappingData is defined (around line 98) that there's an entry to change the application name and the parent is set to "application" instead of "/manifest/application".  When I changed my code to just have the parent of "application" it works.
I think this is a workaround and still should support the other format of "/manifest/application" though.


Memory game added

Please add a background image for the game in the tile :

I dont think it is required as per the contributing guidelines. @nandurijv


Please consider fixing issues detected by repository checker

Notification from ioBroker Check and Service Bot
Dear adapter developer,
I'm the ioBroker Check and Service Bot. I'm an automated tool processing routine tasks for the ioBroker infrastructure. I have recently checked the repository for your adapter vis-2-widgets-ovarious for common errors and appropiate suggestions to keep this adapter up to date.
This check is based the current head revisions (master / main  branch) of the adapter repository
Please see the result of the check below.
ioBroker.vis-2-widgets-ovarious
   -
ERRORS:

[ ] :heavy_exclamation_mark: [E050] '@alcalzone/release-script-plugin-manual-review' must not be listed as dependency at package.json. Please remove and create new release.

WARNINGS:

[X] :eyes: [W156] js-controller 5.0.17 listed as dependency but 5.0.19 is recommended. Please consider updating dependency at io-package.json.

SUGGESTIONS:

[ ] :pushpin: [S526] Consider adding plugin "@alcalzone/release-script-plugin-manual-review".

Please review issues reported and consider fixing them as soon as appropiate.
Errors reported by repository checker should be fixed as soon as possible. Some of them require a new release to be considered as fixed. Please note that errors reported by checker might be considered as blocking point for future updates at stable repository.
Warnings reported by repository checker should be reviewed. While some warnings can be ignored due to good reasons or a dedicated decision of the developer, most warnings should be fixed as soon as appropiate.
Suggestions reported by repository checker should be reviewed. Suggestions can be ignored due to a decision of the developer but they are reported as a hint to use a configuration which might get required in future or at least is used be most adapters. Suggestions are always optional to follow.
You may start a new check or force the creation of a new issue at any time by adding the following comment to this issue:
@iobroker-bot recheck
or
@iobroker-bot recreate
Please note that I (and the server at GitHub) have always plenty of work to do. So it may last up to 30 minutes until you see a reaction. I will drop a comment here as soon as I start processing.
Feel free to contact me (@iobroker-bot) if you have any questions or feel that an issue is incorrectly flagged.
And THANKS A LOT for maintaining this adapter from me and all users.
Let's work together for the best user experience.
your
ioBroker Check and Service Bot
@mcm1957 for evidence
Last update at Wed, 30 Oct 2024 04:54:05 GMT based on commit 417d1a839ae56343e28b46fb657d2e384479ccd1
ioBroker.repochecker 3.1.4

Die Suggestion s526 und der Error e050
sind irgendwie widersprüchlich oder nicht deutlich genug.

development dependencies wie das releasecscript gehören nicht in dependencies sondern in devDependencies.

@oweitman
Nur falls die ltzte Info untergegangen ist. NICHT al Urgenz da das eh erst für die nächste Release relevant ist:
development dependencies wie das releasecscript gehören nicht in dependencies sondern in devDependencies.

recheck


TestNet RC1

With the Testnet version of QT automatic backups are created, some users may not want this. You are supposed to be able to add -createwalletbackups=0 in the conf file to switch this off. This does not work. Can this be fixed. https://dashpay.atlassian.net/wiki/spaces/DOC/pages/1867878/How+to+safely+back+up+your+wallet

It only keeps the last 10 anyways

Ignore this, it works perfectly.


Cancel CheckInputs in CoinViewRule

Closes #1446.
Uses Parallel.ForEach to run each CheckInput in parallel. If a single CheckInput fails, the loop is stopped and no more inputs are checked.

Talking about allocations I'm not sure it's a good idea to create an array of delegates that are called by another delegate passed to Parallel.ForEach, why not just creating an array of DTO object that have the parameters required by the CheckInputs method and use that array as argument in the Parallel.Foreach ?
you prevent a lot of boxing and delegates being createad, while maintaining the same functionality

here you can see another cons of this implementation

what I propose is having something like this to store the needed parameters
var txToCheck = new List<(Transaction tx, int inputIndexCopy, TxOut txOut, PrecomputedTransactionData txData, TxIn input, DeploymentFlags flags)>();
I'm using named tuples just to show the concept, this is equivalent of creating a class with those property to hold parameters (and that's what internally the compiler does with tuple anyway)
and the instead of
checkInputs.Add(() => this.CheckInput(tx, inputIndexCopy, txout, txData, input, flags));
replace this with
txToCheck.Add((tx, inputIndexCopy, txout, txData, input, flags));
and then in the foreach implementation you call directly the CheckInput
return Parallel.ForEach(txToCheck, (checking, state) =>
{
    if (!this.CheckInput(checking.tx,checking.inputIndexCopy,checking.txOut, checking.txData,checking.input,checking.flags))
    {
        state.Stop();
    }
});

(BTW the Parallel.ForEach should check state.IsStopped before invoking the CheckInput, works like cancellationtoken)

(BTW the Parallel.ForEach should check state.IsStopped before invoking the CheckInput, works like cancellationtoken)


UART-PC communication problems with VMs

There is difficulty communicating with the Neblina when going through UART and communicating from a virtual machine.

The problem is not link to Neblina firmware but rather the pyserial package used to communicate with UART over python. Further investigation is required to precisely pinpoint the problem.


llvm deobfuscator fail

As you can see, there is a problem when we remove the confusion. How can we solve it
Errors is as follows:
Call target 0x0 does not appear to be code in instruction at 0x4005ce Function at 0x4005e0 is too large, skipping analysis Analysis update took 0.050 seconds Error fetching version list: Update authentication failed Analysis update took 0.000 seconds Exception in thread Thread-2: Traceback (most recent call last):   File "/usr/lib/python2.7/threading.py", line 801, in __bootstrap_inner     self.run()   File "/home/XXXXX/binaryninja/plugins/../python/binaryninja/plugin.py", line 420, in run     self.task.run()   File "/home/XXXXX/.binaryninja/plugins/llvm-deobfuscator-e7ec09ea4ae0d52ee9a4d28ff9f391c7efea2a57/__init__.py", line 16, in run     self.func(bv, self.addr)   File "/home/XXXXX/.binaryninja/plugins/llvm-deobfuscator-e7ec09ea4ae0d52ee9a4d28ff9f391c7efea2a57/deflatten.py", line 295, in deflatten_cfg     state_var = func.get_low_level_il_at(addr).medium_level_il.dest AttributeError: 'NoneType' object has no attribute 'medium_level_il'
Thanks

@ZakiG could you provide the binary you're having this issue with?

Of course, this is the binary
Kareem El-Faramawi<EMAIL_ADDRESS>于2018年11月28日周三 上午3:35写道：

@ZakiG https://github.com/ZakiG could you provide the binary you're
having this issue with?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/RPISEC/llvm-deobfuscator/issues/1#issuecomment-442188264,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AQky0K57zIqkSiQGyoKBVW15pqK-C6RLks5uzZQLgaJpZM4YxbWB
.

OK, you can get binary from  here: https://transfer.sh/1OOtg/Interpreter
Kareem El-Faramawi<EMAIL_ADDRESS>于2018年11月28日周三 上午10:22写道：

Email attachments don't work with github, can you upload it somewhere and
post a link?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/RPISEC/llvm-deobfuscator/issues/1#issuecomment-442294083,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AQky0N3TFjPXYtmk9wgBCK_D4x0Xt4ioks5uzfNggaJpZM4YxbWB
.

Emmm, how can I solve this problem  finally ,thanks~
Kareem El-Faramawi<EMAIL_ADDRESS>于2018年11月29日周四 上午4:02写道：

Closed #1 https://github.com/RPISEC/llvm-deobfuscator/issues/1 via
88dcec9
https://github.com/RPISEC/llvm-deobfuscator/commit/88dcec91b80d14f78cd27bbd8c34f2ae873e3c11
.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/RPISEC/llvm-deobfuscator/issues/1#event-1993458110,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AQky0KmOnDErFdnckFkqcNJnKC4gOvp9ks5uzuvjgaJpZM4YxbWB
.

You should be able to just update the plugin and it'll be fixed, commit 88dcec9 fixes the issue. The problem was that binja wasn't fully analyzing the function, and this forces it to complete analysis.


Add the ability to upload/supply materials for learning.

In addition to Google search results, add the ability to upload materials (that may not be public) for learning.

Maybe the streamlit FileUploader component...
import streamlit as st

def main():
    # Create a file uploader component
    uploaded_file = st.file_uploader("Choose a file")

    # Check if a file was uploaded
    if uploaded_file is not None:
        # Read the contents of the file
        file_contents = uploaded_file.read()

        # Display the file contents
        st.write("File contents:")
        st.write(file_contents)

if __name__ == "__main__":
    main()

Was able to add it -- but really messes with the layout....
Streamlit doesn't currently support complex layouts natively


Refactor traits and how they're used

Currently traits are not used properly.
E.g. actual server implementations (PythonServer and SysServer) do not inherit from the ServerTrait.
This is because historically I've done things like return concrete implementations from a function that returns a trait, and this was removed from Mojo after a certain version.
Need to rethink how traits are used and in which cases they are valuable, and refactor accordingly.

I think we can close this now since we removed the python implementation?

@bgreni true, no longer relevant. Also don't think this will come up in the future, we already use traits for things like service definitions, and for things like server they're probably an overkill. Closing this now


build bump

for boost linkage fix

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.


switching to performance.now() for ajax timing

Per http://stackoverflow.com/questions/313893/how-to-measure-time-taken-by-a-function-to-execute
Our telemetry indicates that ajax start time sometimes is bigger than finish time.

Feel free to comment after PR is closed!


Extracted out custom fields

I guess the unit tests are failing due to the previous branch, please confirm


html2canvas isn’t using cached fonts causing rendering issues

Please make sure you are testing with the latest release of html2canvas.
Old versions are not supported and issues reported for them will be closed.
Please follow the general troubleshooting steps first:

[X] You are using the latest version
[X] You are testing using the non-minified version of html2canvas and checked any potential issues reported in the console

Bug reports:
I’m using html2canvas in a PWA with fonts served directly by the application and not over a distribution network like Google Fonts for privacy reasons. The PWA works fully offline, except for the generation of a "screenshot" with html2canvas.
I have noticed three things:

html2canvas fully re-downloads all fonts upon every call, not using the ones cached on the device
As a result, it generates a blank image if it is called when there’s no internet connection
If the connection is anything slower than a local development machine, a fallback font or something else with wrong metrics is used, causing squished text, or text in wrong positions like in this and this issue, likely because the fonts aren’t fully re-downloaded yet when the rendering is done

Specifications:

html2canvas version tested with: 1.0.0-rc3
Browser & version: Chrome on Android, Chrome on Linux, Firefox on Linux, Safari on iOS and macOs, all in the latest stable version

Similar happens to me too. It seems that html2canvas is NOT working on offline at all.. 😢

I'm facing same problem. Does anyone has some workaround ?

I had the same problem. None of the tips or tricks sprinkled throughout the github issues for this library helped. I had to ditch the custom fonts for rendering to proceed normally:
html2canvas(element, {
      // removeContainer: false, //useful for testing changes to the iterim iframe copy
      onclone: (document: Document) => {
        Array.from(
          document
            .querySelector(".analysis-container .ng-star-inserted")
            .children[0].querySelectorAll("*"),
        ).forEach((e) => {
          let existingStyle = e.getAttribute("style") || "";
          e.setAttribute("style", existingStyle + "font-family: Helvetica, sans-serif !important");
        });
      },
    }).then(function (canvas) { //...etc

Many of the elements had very specific css rules for fonts, so just adding a font-family rule to the primary node didn't work. Now I have no weird spacing and the image is nearly as good as the original (minus the custom fonts)

I have a similar issue where fonts are not loaded, but in my case they are defined in @font-face with data URIs. I'd hope that to be the most unlikely scenario to not work.

I had the same problem. None of the tips or tricks sprinkled throughout the github issues for this library helped. I had to ditch the custom fonts for rendering to proceed normally:
html2canvas(element, {
      // removeContainer: false, //useful for testing changes to the iterim iframe copy
      onclone: (document: Document) => {
        Array.from(
          document
            .querySelector(".analysis-container .ng-star-inserted")
            .children[0].querySelectorAll("*"),
        ).forEach((e) => {
          let existingStyle = e.getAttribute("style") || "";
          e.setAttribute("style", existingStyle + "; font-family: Helvetica, sans-serif !important");
        });
      },
    }).then(function (canvas) { //...etc

Many of the elements had very specific css rules for fonts, so just adding a font-family rule to the primary node didn't work. Now I have no weird spacing and the image is nearly as good as the original (minus the custom fonts)

@chriszrc thanks man - this worked brilliantly for me with a little tweak - I apply it to all child elements in the div I'm saving.
In case it's helpful for anyone else, I used:
    const canvas = await html2canvas(document.getElementById(divId), {
        scale: 5,
        allowTaint: true,
        letterRendering: 1,
        onclone: (document) => {
            Array.from(
                document.querySelectorAll("*"),
            ).forEach((e) => {
                let existingStyle = e.getAttribute("style") || "";
                e.setAttribute("style", existingStyle + "; font-family: Helvetica, sans-serif !important");
            });
        }
    });

Hours spent trying to figure this out and this onClone solution was the only thing that worked! Thank you!

Adding the fonts to the cloned document seems to fix the issue for me. In case it's helpful for anyone, I did something like this.
const font1Promise = new FontFace('<Font family that you have used>',url('')).load(); const font2Promise = new FontFace('<Font family that you have used>', url('')).load(); const [font1, font2] = await Promise.all([font1Promise, font2Promise]); const canvas = await html2canvas(document.getElementById(divId), {         scale: 5,         allowTaint: true,         letterRendering: 1,         onclone: (document, element) => {             document.fonts.add(font1);             document.fonts.add(font2);         }     });


Should --fix exit with 0 if it successfully fixed all issues?

Why does --fix exit with error even if it can fix every problem? This does not match what eslint --fix does.
Bug Report

TSLint version: 5.4.2
TypeScript version:2.3.2
Running TSLint via: CLI

Actual behavior
--fix should prints errors and exits with 1
Expected behavior
--fix should print warnings for things it fixed, but exit with 0

I'm running into the same problem. This for example prevents --fix to be used as a precommit hook.
/cc @adidahiya

Actually this seems to be fixed in tslint 5.5.0, @bahmutov

I'm using 5.7.0 and this is fixed. Not sure how far back it's been fixed.

We usually don't port fixes to older releases. So this is fixed in 5.5.0 and above


Explore volumetric simulation data

e.g.

or

where the second is a slice of an intra- or extracellular 3D rxd object.

Note that this is only relevant: (1) when Python is available as rxd specification -- but not simulation -- currently requires Python, and probably (2) when there is a good way for MATLAB and Python to call each other (see #9).


Get followers non followers

Is it possible to get lists of my following users who don't follow back with this API ?

I am sure you can get a list of users you are following and compare it with a list of users who are following you. At least this is how I implemented it. Maybe there is a faster way but I am not aware of it.

You can dedect unfollowers like this:
for (InstagramUserSummary i : following) {
for (InstagramUserSummary j : followers) {
if (i.equals(j)) {
unfollowers.remove(i);
break;
}
}
}

Thanks, will try it,


:lnext and :lprevious don't jump to errors

I have a javascript warning that shows up in my quickfix list when I type :lopen. Syntastic also shows me the yellow // WARNINGS (1)// in my status line. However, When I type :lnext or :lprev I get "No more items" (regardless of if I'm on the errored line or not).

:lnext and :lprevious go to the next / previous issue.  With a single warning in the loclist there
are no next / previous issues, so :lnext and :lprevious won't work, try :lfirst and :llast instead.  This is not particularly intuitive if you ask me, but it's how Vim works, it doesn't have anything to do with syntastic. shrug


sbt-allsources.sh fails with no project 'compilerBridge' in zinc

[x] I've read the CONTRIBUTING guidelines

steps
Run as described in the CONTRIBUTING
$ mkdir sbt-modules
$ cd sbt-modules
$ for i in sbt io util librarymanagement zinc; do \
  git clone https://github.com/sbt/$i.git && (cd $i; git checkout -b 1.x origin/1.x)
done
$ cd sbt
$ ./sbt-allsources.sh

problem
java.lang.RuntimeException: No project 'compilerBridge' in 'file:/home/muki/dev/git/github/sbt/zinc/'.

expectation
No runtime exceptions
notes
It seems that this commit is responsible for this issue.

@eed3si9n The fix for this never made it in to the 1.2.x branch.

@eed3si9n Sorry that's not true, I found the fix, but I'm having another issue.


why delete_by_id function not receive &str args?

call delete_by_id function,compiled error.

+ pub async fn delete_by_id(id: &str) -> Result<()> {
    let db = get_connection()?;
    msg_cache::Entity::delete_by_id(id).exec(db).await?;
    Ok(())
}

The terminal compiled error detail:

   Compiling demo v0.1.1 (/Users/baoyachi/rust-project/demo)
error[E0308]: mismatched types
   --> src/db/msg_cache.rs:71:41
    |
71  |     msg_cache::Entity::delete_by_id(id).exec(db).await?;
    |     ----------------------------------- ^^- help: try using a conversion method: `.to_string()`
    |     |                                   |
    |     |                                   expected struct `std::string::String`, found `&str`
    |     arguments to this function are incorrect
    |
note: associated function defined here
   --> /Users/baoyachi/.cargo/registry/src/sea-orm-0.9.3/src/entity/base_entity.rs:822:8
    |
822 |     fn delete_by_id(values: <Self::PrimaryKey as PrimaryKeyTrait>::ValueType) -> DeleteMany<Self> {
    |        ^^^^^^^^^^^^

The table Generated Entity:

#[derive(Clone, Debug, PartialEq, Eq, DeriveEntityModel, Serialize, Deserialize)]
#[sea_orm(table_name = "msg_cache")]
pub struct Model {
    #[sea_orm(primary_key)]
    pub id: String,
    pub raw_val: String,
}

Hey @baoyachi, thanks for the suggestion! Yes, we can relax it by accepting any Into<<Self::PrimaryKey as PrimaryKeyTrait>::ValueType>. Please check https://github.com/SeaQL/sea-orm/pull/1362

Thx @billy1624 . Expect the new version to use this feature

There will be a new release soon :)


GamesPlayedWhileIdle not working on my main bot

Hello everyone.
This is my current bot config (without crucial information of course) when I start the program it goes on farming like it should but it doesn't idle the games inside the collection, I mean the hours remain the same, could it be a possible bug or am I missing something?
{
  "Enabled": true,
  "StartOnLaunch": true,
  "SteamLogin": "****",
  "SteamPassword": "*****",
  "PasswordFormat": 0,
  "SteamParentalPIN": "****",
  "SteamApiKey": "*****",
  "SteamMasterID":<PHONE_NUMBER>8282737,
  "SteamMasterClanID": 0,
  "CardDropsRestricted": false,
  "DismissInventoryNotifications": false,
  "FarmOffline": true,
  "HandleOfflineMessages": true,
  "AcceptGifts": true,
  "IsBotAccount": false,
  "SteamTradeMatcher": true,
  "ForwardKeysToOtherBots": false,
  "DistributeKeys": false,
  "ShutdownOnFarmingFinished": false,
  "SendOnFarmingFinished": true,
  "SteamTradeToken": "G3UlSeFF",
  "SendTradePeriod": 0,
  "AcceptConfirmationsPeriod": 0,
  "CustomGamePlayedWhileIdle": "Test",
  "GamesPlayedWhileIdle": [
    271590,
    479130,
  ]
}

ASF will play GamesPlayedWhileIdle only if:
a) You own specified games on your account (as in, you can use !play with those appIDs and they work
b) ASF is not currently farming anything (as in you're receiving a message stating that we don't have anything to farm on this account)
This feature works correctly, I even got confirmation yesterday that it does regarding further enhancements. If you're still not sure where is the problem, try !play with your appIDs in order to see yourself if it works or not. Remember that ASF only sends request with your specified appIDs - whether Steam accepts such request, or not, is out of the scope of ASF as a program.
Because this is not a bug neither a suggestion, GitHub is not proper place for this issue.

Ah yes, and if you use FarmOffline feature then you won't actually see those games as being played, but you will notice them on your recently played list in your profile, and hours being increased from time to time (when Steam Network updates). If you want to see the status instead, you must not farm offline.

Also the reason is probably the fact that you set CustomGamePlayedWhileIdle. You can't use CustomGamePlayedWhileIdle and GamesPlayedWhileIdle at the same time - this feature is available since ASF V<IP_ADDRESS> - any version prior to that will use only one of those properties, with CustomGamePlayedWhileIdle being with higher priority than the other.
Therefore, you need to either update to pre-release, or switch CustomGamePlayedWhileIdle back to null.


Remove non exhaustive list binding from docs

From docs:
https://rescript-lang.org/docs/manual/latest/pattern-matching-destructuring
// List
let myList = list{1, 2, 3}
let list{head, ...tail} = myList // 1 assigned to `head`, `list{2, 3}` assigned to tail

Compiler currently crashes in such case.
but aside that, since the the pattern fails for an empty list  I think we should remove this or warn the user about that.

Since the example is documenting pattern matching destruction for lists (and arrays above it) what do you think about updating the example rather than removing it?
For example:
let myList = list{1, 2, 3}

let (head, tail) = switch myList {
| list{head, ...tail} => (Some(head), tail)
| list{} => (None, list{})
}

Since the example is documenting pattern matching destruction for lists (and arrays above it) what do you think about updating the example rather than removing it?
For example:
let myList = list{1, 2, 3}

let (head, tail) = switch myList {
| list{head, ...tail} => (Some(head), tail)
| list{} => (None, list{})
}

This is too complex in the context of destructuring IMO. I prefer adding a note explaining it can be fail and the pattern is not exhaustive.

This is too complex in the context of destructuring IMO. I prefer adding a note explaining it can be fail and the pattern is not exhaustive.

Understood, I see your concern.
What about moving that content to a separate section "Destructuring Arrays and Lists" which includes examples like above.
Including a note explaining that list/array structuring can fail and that the pattern not being exhaustive is good information, but it doesn't help people new to ReScript deal with that problem. And I imagine those people are a primary target audience for the docs.
In other words, if we describe a problem then it would be good to also provide a guide to resolve it (either a link to another page, or describe the solution in the same place).
Also, maybe there is a simpler example we could use?
What do you think?

Understood, I see your concern.
What about moving that content to a separate section "Destructuring Arrays and Lists" which includes examples like above.
Including a note explaining that list/array structuring can fail and that the pattern not being exhaustive is good information, but it doesn't help people new to ReScript deal with that problem. And I imagine those people are a primary target audience for the docs.
In other words, if we describe a problem then it would be good to also provide a guide to resolve it (either a link to another page, or describe the solution in the same place).
Also, maybe there is a simpler example we could use?
What do you think?

This example should now be documented properly in master:

I think we can close this issue?

Yup, Thank you.


build issue

i am using ol with vue .when run script "npm run serve",i got  bugs like this
ERROR  Failed to compile with 2 errors                                                                                上午9:52:24
error  in ./node_modules/_ol<EMAIL_ADDRESS>Module parse failed: Unexpected token (142:35)
You may need an appropriate loader to handle this file type, currently no loaders are configured to process this file. See https://webpack.js.org/conc
epts#loaders
|      * @private
|      */

this.hasFill_ = !!shaders.fill?.vertex;

|     if (this.hasFill_) {
|       this.fillVertexShader_ = shaders.fill.vertex;
@ ./node_modules/_ol<EMAIL_ADDRESS>7:0-76 168:21-40
@ ./node_modules/_cache-loader@4.1.0@cache-loader/dist/cjs.js??ref--13-0!./node_modules/_babel-loader@8.3.0@babel-loader/lib!./node_modules/_cache-lo
ader@4.1.0@cache-loader/dist/cjs.js??ref--1-0!./node_modules/_vue-loader@15.10.1@vue-loader/lib??vue-loader-options!./src/views/LeaderDemand.vue?vue&t
ype=script&lang=js&
@ ./src/views/LeaderDemand.vue?vue&type=script&lang=js&
@ ./src/views/LeaderDemand.vue
@ ./src/router/index.js
@ ./src/main.js
@ multi ./node_modules/_webpack-dev-server@3.11.3@webpack-dev-server/client?http://<IP_ADDRESS>:8081&sockPath=/sockjs-node (webpack)/hot/dev-server.js
./src/main.js
error  in ./node_modules/_ol<EMAIL_ADDRESS>Module parse failed: Unexpected token (165:61)
You may need an appropriate loader to handle this file type, currently no loaders are configured to process this file. See https://webpack.js.org/conc
epts#loaders
|      * @private
|      */

this.hitDetectionEnabled_ = options.hitDetectionEnabled ?? true;

|
|     const customAttributes = options.attributes
@ ./node_modules/_ol<EMAIL_ADDRESS>5:0-72 109:15-39
@ ./node_modules/_cache-loader@4.1.0@cache-loader/dist/cjs.js??ref--13-0!./node_modules/_babel-loader@8.3.0@babel-loader/lib!./node_modules/_cache-lo
ader@4.1.0@cache-loader/dist/cjs.js??ref--1-0!./node_modules/_vue-loader@15.10.1@vue-loader/lib??vue-loader-options!./src/views/LeaderDemand.vue?vue&t
ype=script&lang=js&
@ ./src/views/LeaderDemand.vue?vue&type=script&lang=js&
@ ./src/views/LeaderDemand.vue
@ ./src/router/index.js
@ ./src/main.js
@ multi ./node_modules/_webpack-dev-server@3.11.3@webpack-dev-server/client?http://<IP_ADDRESS>:8081&sockPath=/sockjs-node (webpack)/hot/dev-server.js
./src/main.js

at 7.4.0 there is no problem on windows ,but got this bug on linux; at 7.5.1 got this bug on windows too

it happened when using 7.5.1,webgl/vectorlayers

Optional chaining and the nullish coalescing operator are es6 language features, so I'd say that the dependencies you use for building are not suited and/or outdated.
But maybe I'm wrong and those kind of features should be transpiled when the library is published?

We publish untranspiled modern JavaScript, so if older ES versions are required, users have transpile OpenLayers as part of their build process. Usually there's a transpileDependencies option or similar in the build configuration, depending on the toolset used.

我们发布未转译的现代 JavaScript，因此如果需要较旧的 ES 版本，用户可以将转译 OpenLayers 作为其构建过程的一部分。transpileDependencies通常，构建配置中有一个选项或类似选项，具体取决于所使用的工具集。

Hello, configuring transpileDependencies using the OL installed by CNPM has not been effective. I hope you can try to avoid using these syntax in your next update as much as possible

@m1328224904 Maybe you configured something in the wrong way?

@m1328224904也许您以错误的方式配置了某些内容？

My configuration is not a problem
module.exports = {     publicPath: "./", // 公共路径(必须有的)     outputDir: "dist", // 输出文件目录     lintOnSave: false,     productionSourceMap: false, //去除打包后js的map文件     // runtimeCompiler: true, //关键点在这     devServer: {         port: 6527,     },     css: {         loaderOptions: {             sass: {                 additionalData:
@import<EMAIL_ADDRESS>            }         }     },      transpileDependencies: ['ol/renderer/webgl'] }
The above are all my configuration codes. I am in China and often fail to download using NPM. I just used some special methods to make my NPM available, and there were no errors in the configuration of OL and transpileDependencies downloaded this time

Why not transpile all of ol? transpileDependencies(['ol']). That's how this is usually done.

为什么不转译所有的ol呢？transpileDependencies: ['ol']通常。就是这样做的。我什至不认为transpileDependencies接受模块，它必须用包进行配置。

I just conducted a test, which is also feasible. The topic I want to say is that you need to consider users who use cnpm installation. After installing ol using cmpm, the transpileDependencies configuration will become invalid, or should you not use it?? Because I have reviewed the source code, there is only one place where I have used the syntax?? Grammar of

为什么不转译所有的 ol 呢？transpileDependencies: ['ol']。通常就是这样做的。我什至不认为transpileDependencies接受模块，它必须用包进行配置。

为什么不转译所有的ol呢？transpileDependencies: ['ol']通常。就是这样做的。我什至不认为transpileDependencies接受模块，它必须用包进行配置。

Or rather, I hope you can release the compiled version so that there won't be any issues

为什么不转译所有的 ol 呢？transpileDependencies: ['ol']。通常就是这样做的。我什至不认为transpileDependencies接受模块，它必须用包进行配置。

为什么不转译所有的ol呢？transpileDependencies: ['ol']通常。就是这样做的。我什至不认为transpileDependencies接受模块，它必须用包进行配置。

Or rather, I hope you can release the compiled version so that there won't be any issues

Indeed.

it's not work .

this.hitDetectionEnabled_ = options.hitDetectionEnabled ?? true;

I tried using transpileDependencies: ['ol'] configuration, but it still didn't work. My vue-cli:4.5.0

@haloxzp Did you configure transpileDependencies in the correct place?

this.hitDetectionEnabled_ = options.hitDetectionEnabled ?? true;

我尝试使用 transpileDependency: ['ol'] 配置，但仍然失效。我的 vue-cli:4.5.0

如果你是使用的cnpm安装，确实有可能不起作用，你可以尝试使用npm重新下载所有的包。或者你把this.hitDetectionEnabled_ = options.hitDetectionEnabled ?? true;这一句改一下，使用es5的语法

It's transpileDependencies (plural), not transpileDependency (singular).

Why not transpile all of ol? transpileDependencies: ['ol']. That's how this is usually done. I don't even think transpileDependencies accepts modules, it has to be configured with packages.

@ahocevar Thank you! I solved the issue by including the package "color-parse" as well in "transpileDependencies":
// vue.config.js
module.exports = {
  transpileDependencies: [
    "ol",
    "color-parse"
  ]
}

Hello, I am facing the same issue in angular any idea how to resolve it?


Not Specified template-path uses the label for key instead for id

We have removed the "with: template-path" and in returns the label instead of the ID as the key however the value is correct.
Is there a way to return the ID instead of Label for any template used?

There's no way to guess what the ID is from the information in the issue body, so I think you must specify the template-path.

There is no reliable way to do this; you have to provide a template for IDs to work.
https://github.com/stefanbuck/github-issue-parser/blob/2ac1a18553a52ccc22739a40e99bf97300961e37/README.md?plain=1#L13


[#547] process websocket CloseFrame

If an endpoint receives a Close frame and did not previously send a
Close frame, the endpoint MUST send a Close frame in response.

Thank you for contributing!


Minor grammar fix : duplicated coordinators.

Signed-off-by: Sungho Moon<EMAIL_ADDRESS>

thanks @shyr


Report coverage to codecov

Description:
Resolves #

Codecov Report

:exclamation: No coverage uploaded for pull request base (master@61b5e28). Click here to learn what that means.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master      #43   +/-   ##
=========================================
  Coverage          ?   82.63%
=========================================
  Files             ?       12
  Lines             ?      806
  Branches          ?        0
=========================================
  Hits              ?      666
  Misses            ?       89
  Partials          ?       51

Continue to review full report at Codecov.

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


CI 3.0.6 escape_like_str incorrectly escapes underscores (_)

In CI 3.0.6 underscores are escaped by ! instead of \ resulting in issue where query doesn't return results expected.

I don't see a workaround in #4462; it is just updated documentation on what I think is incorrect functionality for escaping. See http://stackoverflow.com/a/17102616

Well, read the documentation ...

Can't you just change line 338 in DB_driver.php
FROM:
protected $_like_escape_chr = '!';
TO:
protected $_like_escape_chr = '\';

I can, but I shouldn't and I won't.
The escape character isn't the problem, the existence of escape_like_str() is.

Ok I will do the workaround


Interchain Accounts / lupercalia

This PR will bring ICA to Juno, as well as:

wasmd v0.24.0
ibc-go
an upgrade handler folder to better organize code
go 1.18
tighter linting

This PR should be applied after a governance "Software Upgrade" proposal to juno passes.
The governance "Software upgrade" proposal should not be made until this code has been merged to Juno's main branch.

Awesome @nghuyenthevinh2000!
Ok, now we just add the ica stuff which is here:
https://github.com/cosmos/ibc-go/blob/main/testing/simapp/app.go

we should also use wasmd from the same commit (nearly) as Osmosis:

I am making a cosmos-sdk fork with @pomvn's fast iavl patch only, so we don't depend on osmosis-sdk

https://github.com/notional-labs/cosmos-sdk/tree/v0.45.1-junofast

we can keep this pull request only for ibc-go v3 and ICA, and move iavl and other forks to other pull requests.
I also heard that iavl fork throws app_hash errors in osmosis, or does it have another cause?

no one really knows about those apphash errors, we can take fast iavl out

@faddat, i think we should remove iavl from wasmd

yes you're right and thanks very much vuong.
It was kind of all or nothing with fast iavl.

@vuong177 @nghuyenthevinh2000
Hey guys, this needs to be reworked.  I'll click around a little, and see how much is needed.

Hey, I am going to close this in favor of #200
While writing that, it became apparent that it might as well be this.


Azure CosmosDB StateStore Default Schema Value?

Hi,
I do tryouts on Azure CosmosDB for Dapr .Net Sdk. I cannot see the OrderState object I have sent to CosmosDB in the JSON format in Azure Portal Data Explorer. Dapr client brings data from CosmosDB as a deserialized. I can read it 😊 . I need to read Value and write a select query. Id part is not serialized.

Dapr Docs for Azure Cosmos DB In the document, a note has been made that the dapr client is taking care of this regarding serialization.
I just change the Cosmos DB to Redis for state store using components, I can see the Json Result.

Should we have a different setting through this dapr client settings or a different adjustment via Azure Portal?
Additional Info. I can add manually json object to value

Thank you for your help.
Have a nice coding.
İlker

Hi, please refer to these issues, and it would be helpful if you could add your feedback there:
https://github.com/dapr/dapr/issues/2892
https://github.com/dapr/components-contrib/issues/745
The way a component (such as CosmosDB) stores data is not something the SDK can control.


[AutoDiff upstream] fix TF-971

Upstreaming @differentiable function type parsing introduced a bug: The parser can't parse @differentiable functions with throws. The bug wasn't caught by the parsing tests in the upstreaming PR.
This fixes the bug and adds parsing tests.
Resolves TF-971.

@swift-ci please test and merge


Dataset

Coverage decreased (-0.03%) to 92.95% when pulling 3ce880d61d32da341dc30c11154aaf472573716d on dataset into 02f6fd297c9bd63eb54a6e8ff0e54f6ff97df723 on master.


Chromecast wired connection not being discovered

I'll have to triple check as I finalize more things, but the two times I tried to switch from the wifi connection to a wired connection on the same vlan it would not be discovered. Maybe it has something todo with igmp & pim on the switch because all the times I tried doing mDNS myself with an avahi vm, it would work or work much better with those two things disabled. Whereas with this project its the opposite, only works with igmp & pim; atleast in terms of devices connected through wifi I believe.

dunno, all my tests have been wired for sonos and wireless for chromecast
but in general start with all IGMP snooping off
also start with open firewalls   to test as there can be non-broadcast traffic involved.....

no followup - closing