Fire mage shows blank screen

Describe the bug
A clear and concise description of what the bug is.
Whenever I go to firemage sims its just a blank screen, no gear showing unable to run sim, add gear, or do anything
To Reproduce
Steps to reproduce the behavior:

Go to '...'
Click on '....'
Scroll down to '....'
See error

Expected behavior
A clear and concise description of what you expected to happen.
Sim Links and Screenshots
If applicable, add links to your sim settings or screenshots to help explain your problem.

Desktop (please complete the following information):

OS: [e.g. iOS]
Browser [e.g. chrome, safari] all browsers
Version [e.g. 22]

Smartphone (please complete the following information):

Device: [e.g. iPhone6]
OS: [e.g. iOS8.1]
Browser [e.g. stock browser, safari]
Version [e.g. 22]

Additional context
Add any other context about the problem here.

@stravosbtw Try to;

Restore default settings
Clear your browser data/settings for the wowsims site

It should always work in a new incognito (private) browsing window.
Make sure to not use the Weakaura to import your gear settings. Use the official WowSimsExporter addon.


BrowserStack IE error SCRIPT5009: 'Proxy' is undefined

We noticed an error in our console coming from bugsnag-client.js
const getErrorBoundary = (client) => {
client.use(bugsnagReact, React);
return client.getPlugin('react');
};
export const client = getClient();  <-----on this line error is thrown
export const ErrorBoundary = getErrorBoundary(client);
error was "'Proxy' is undefined"
This was only seen while running in BrowserStacks testing environment on Windows 10 IE 11.  Let us know if any ideas, thx.

Hi @bjm88, getClient() is not one of the functions that Bugsnag defines and also we don't mention Proxy anywhere in the browser codebase. Additionally our tests run successfully on IE11 on BrowserStack.
I'm inclined to think this isn't a problem with Bugsnag, but happy to stand corrected. Could you provide some more information, a reduced test case, or an error location that is actually coming from Bugsnag's source?
Thanks


lndconnect support for easy Zap integration

Description of the Feature or Idea
I think it would be cool if Joule supported lndconnect (or some format for passing auth info)
With this we (Zap) can host a "Connect to Joule" type of button offering much cleaner onboarding UX for users.
Existing Example(s) of Feature
User downloads Zap and Joule. Syncs to the chain and opens channels in Zap, clicks "Connect to Joule" and is authed into Joule.

I've taken a look at this, and there are two key problems:

Joule exists only in the browser context, so there's not a way to have a link clicked in another context communicate with Joule, unlike native apps that can register a custom URI handler. This would require Joule to have a native component, as outlined in #106.
The current lndconnect format doesn't distinguish between the REST and gRPC APIs (and based on the examples, assumes gRPC) but Joule needs to use the REST API due to browser limitations.

I'm all in favor of being compatible with this URI format, but I think it would require a native application on my end.


Missing parent: GO:0061605 molybdopterin-synthase adenylyltransferase activity

Is this not true?

GO term ID and label for which you request a new superclass

GO:0061605 molybdopterin-synthase adenylyltransferase activity

New superclass (parent) suggested
GO:0070733 protein adenylyltransferase activity

@sjm41

It seems that this case is about the term name implies something broader than its definition. We could rename it to something like "protein L-serine/L-threonine/L-tyrosine residue adenylyltransferase activity". Is that desirable, though?

@raymond91125 Yes, it would be useful if the name didn't suggest it was a broad grouping term. I see EC says that "AMPylase" is an alternative name, so maybe the GO term name could be "AMPylase activity" (and keep the current definition)?
@Antonialock Given this, will the UniProt annotation to P38820 (yeast UBA4) have to change to 'adenylyltransferase activity'?

Renamed to AMPylase activity. Should this be closed? Thanks.

Please reopen if needed. Thanks.


Address bar text is cleared when extension loads on Firefox

Hi,
I'm using the latest version of stellar photos with Firefox 80.0b8 (64-bit) on macOS 10.15.4, and I'm seeing this behavior now, which wasn't there before:

I open a new tab
I quickly start typing (Address bar is automatically focused when opening a new tab)
When stellar photos loads, the text I had already written into the address bar is deleted

I can trigger this behavior almost every time, but sometimes the extension loads really fast and it's not visible. I have a 2017 MBP 15, so performance should not be an issue.

Thank you for opening this issue. I've never noticed this issue in Firefox before, but I will investigate and get back to you soon.

I tried to reproduce this with Firefox 79.0 (i.e., the normal one, not the developer edition) and it shows the same behaviour. I noticed it loads quite a bit faster on 79 than on 80.0b8, so it was more difficult to trigger the effect, but I still managed to run into it a few times.

I'm unable to reproduce this issue on Firefox 85. Feel free to reopen if the bug resurfaces

I'm unable to reproduce this issue on Firefox 85. Feel free to reopen if the bug resurfaces


Add pkgconfig file

I agree a .pc file should be added.
However this one is not produced by CMake (e.g. the version and paths are directly in the file), and it's one file for both libraries. This should be split in 2.
I will look into this as soon as I find some time.

Agreed with you.
Someone ask to merge them into one library,  so we should consider this situation.
FYI: I have submitted a PR to MSYS MINGW package for xlsxio:
https://github.com/msys2/MINGW-packages/pull/8109

I added some code to generate .pc files.
Libraries will remain separate, as applications may only need to use one or the other.
They also have different dependancies, and their code is completely separate.


RNA branch prepare compliance data

Add ingestion of RNA seq data into compliance data script (to be refactored anyway)

this is done


Authentication presumes the callback URL is set to https://<IP_ADDRESS>:8080

It appears that the local server is hard-coded to spawn at <IP_ADDRESS>:8080 for accepting the callback. This implicitly assumes that the user has set up their application with Schwab to this exact callback URL for the auth flow to work smoothly as in the example.
https://github.com/z-Wind/schwab_api/blob/11f81fdcbc453a18ac3c87a8fc9f068b0ea9c45d/src/token/local_server.rs#L22
Would you be fine with me submitting a PR to make this handling of the callback server to match the redirect_url that is provided to the Authorizer?

That sounds great! Thank you for taking the time to address this.


Implement function to animate marker to specified position for Android

could you provide the same functionality for iOS? I would like to have a heterogenous api

Sure, will do it when I have time

@xming13 found some time for that? would love to merge this

As mentioned in other pr, anyone is welcomed to implement the functionality for the iOS part.

Don't worry. I will check this and implement it in Objective-C if no problem when I have a time.
Thank you for sending this pull request.

Since this is already implemented in v2, I close this pull request.
https://github.com/mapsplugin/cordova-plugin-googlemaps-doc/blob/master/v2.0.0/utilities/geometry/spherical/interpolate/README.md


Unclear doc regarding out parameter path

I am not sure what this is supposed to be, so  far I tried to equal it to the directory name of my task execution, hence dir: x would imply path: x:

path: Required. The path of the repository to reference the pull request.

resources:
- name: pr-firmware
  type: pull-request
  source:
    access_token: ((github-access-token))
    private_key: ((sshkey))
    repo: myorg/myrepo
    base: master
    uri: git@github.com:myorg/myrepo.git

jobs:
  - name: check-pr-firmware
    plan:
    - get: firmware
      resource: pr-firmware
      trigger: true

    - get: container-geared-up

    - task: firmware-builds-check
      image: container-geared-up
      config:
        platform: linux
        inputs:
        - name: firmware
        outputs:
        - name: binary
        run:
          path: zsh
          args:
          - -exec
          - |
            build.sh
          dir: firmware

      on_failure:
        put: pr-firmware // fails
        params:
          path: firmware
          status: failure
          context: compile-test

      on_success:
        put: pr-firmware
        params:
          path: firmware  // fails
          status: success
          context: compile-test

      on_failure:
        put: pr-firmware // fails
        params:
          path: pr-firmware
          status: failure
          context: compile-test

yet I am getting:
Identity added: /tmp/git-resource-private-key redactedcomment

/usr/lib/ruby/gems/2.4.0/gems/octokit-4.8.0/lib/octokit/response/raise_error.rb:16:in `on_complete': POST https://api.github.com/repos/myorg/myrepo/statuses/1a602acadff6f269bd964cfc1ea6bee10a6b3e3e: 404 - Not Found // See: https://developer.github.com/v3/repos/statuses/#create-a-status (Octokit::NotFound)

	from /usr/lib/ruby/gems/2.4.0/gems/faraday-0.14.0/lib/faraday/response.rb:9:in `block in call'

	from /usr/lib/ruby/gems/2.4.0/gems/faraday-0.14.0/lib/faraday/response.rb:61:in `on_complete'

	from /usr/lib/ruby/gems/2.4.0/gems/faraday-0.14.0/lib/faraday/response.rb:8:in `call'

	from /usr/lib/ruby/gems/2.4.0/gems/faraday-http-cache-2.0.0/lib/faraday/http_cache.rb:146:in `call!'

	from /usr/lib/ruby/gems/2.4.0/gems/faraday-http-cache-2.0.0/lib/faraday/http_cache.rb:115:in `call'

	from /usr/lib/ruby/gems/2.4.0/gems/faraday-0.14.0/lib/faraday/rack_builder.rb:143:in `build_response'

	from /usr/lib/ruby/gems/2.4.0/gems/faraday-0.14.0/lib/faraday/connection.rb:387:in `run_request'

	from /usr/lib/ruby/gems/2.4.0/gems/faraday-0.14.0/lib/faraday/connection.rb:175:in `post'

	from /usr/lib/ruby/gems/2.4.0/gems/sawyer-0.8.1/lib/sawyer/agent.rb:94:in `call'

	from /usr/lib/ruby/gems/2.4.0/gems/octokit-4.8.0/lib/octokit/connection.rb:156:in `request'

	from /usr/lib/ruby/gems/2.4.0/gems/octokit-4.8.0/lib/octokit/connection.rb:28:in `post'

	from /usr/lib/ruby/gems/2.4.0/gems/octokit-4.8.0/lib/octokit/client/statuses.rb:43:in `create_status'

	from /usr/lib/ruby/gems/2.4.0/gems/octokit-4.8.0/lib/octokit.rb:46:in `method_missing'

	from /opt/resource/lib/status.rb:15:in `create!'

	from /opt/resource/lib/commands/out.rb:59:in `block in output'

	from /opt/resource/lib/commands/out.rb:52:in `each'

	from /opt/resource/lib/commands/out.rb:52:in `output'

	from /opt/resource/lib/commands/out.rb:117:in `<main>'

Is the repo something the access token has access to write to? It's pretty standard for the GitHub API to return 404 when you don't have access to do something. That way it doesn't expose data.

Seems my second comment got lost. It was exactly that. Still I wasn't aware of the issue and a hint by the resource would be appreciated.
If you differ, feel free to close this issue.


Prisma version >= 4.10 performance decreases.

Bug description
Increased CPU consumption when upgrading from Prisma 4.7.1
We wanted to take advantage of the client extensions and are currently on version 4.7.1
We need to use at least Prisma 4.10.0 to be compatible with TypeScript and use nested includes.
Our attempts to upgrade and deploy to production have resulted in decreased throughput and increased CPU usage.  It exceeded our performance limits, and we needed to roll back.
We attempted both 4.12 and 4.10.
In order to make sure the Prisma upgrade is the only difference, we isolated the Upgrade commit and tested it using https://github.com/mcollina/autocannon in our staging environment using an expensive query.
We slowly ramped up the number of concurrent connections and requests until the AWS health-check failed twice in 10 seconds with a 5 second timeout.
I configured the test to have each connection send 10 requests sequentially, and scaled up the number of connections until we got failure messages.
I did these tests over the weekend with no other usage.
The 4.7.1 code could handle 14 concurrent connections
Prisma 4.9.0 could handle 13 concurrent connections (7.2% worse)
Prisma 4.10.0 could handle 12 concurrent connections (14.3% worse)
Prisma 4.12.0 could handle 9 concurrent connections. (55.8% worse)
We do have a lot of optimization to do.  The graphql query we used is to present a list of accounts to a user, so it gets hit a lot in production.  However we would hope the behavior wouldn't degrade so significantly with only upgrading Prisma versions.
The graphql query in particular has 265 properties, with 35 different types, so it looks like there is probably a significant amount of serialization work to do from the database result, and I wonder if something has changed there.
How to reproduce
Currently reporting on our production and staging environments.  I don't have a concise way to test yet.
Expected behavior
A Prisma upgrade shouldn't result in a significant decrease in performance and throughput
Prisma information
Versions 4.9, 4.10, and 4.12.  I'm checking if I can share the query directly.
Environment & setup

OS: Amazon Linux 2 AMI
Database: PostgreSQL
Node.js version: v16.16.0

Prisma Version
prisma                  : 4.10.0
@prisma/client          : 4.10.0
Current platform        : darwin-arm64
Query Engine (Node-API) : libquery-engine ca7fcef713137fa11029d519a9780db130cca91d (at<EMAIL_ADDRESS>Migration Engine        : migration-engine-cli ca7fcef713137fa11029d519a9780db130cca91d (at node_modules/@prisma/engines/migration-engine-darwin-arm64)
Format Wasm             : @prisma/prisma-fmt-wasm 4.10.0-84.ca7fcef713137fa11029d519a9780db130cca91d
Default Engines Hash    : ca7fcef713137fa11029d519a9780db130cca91d
Studio                  : 0.481.0

If performance of recent Prisma versions was generally 50% worse, we would have had a lot of reports of this in the last few months. Our own performance testing also point in the opposite direction - things got generally faster.
Can you provide any additional information that would help us pinpoint what could be going on for you? Are individual queries slower? Can you isolate the CPU impact in any way?

The issue was with logging library incompatibility that killed our AWS sidecar, which then took down our AWS task.  This was not a prisma issue

Thanks for the update <3


add temporary messages as return for commands.

in case we have a logging channel, this should be a feature

Closed for now, but added on feature board.

Closed for now, but added on feature board.


How to collect data from a remote server?

Hi,
I want to collect data from remote server but don't want to install any tool, is it possible can graphite monitor a server and collect data without installing any tool on remote server, if yes which tool we can use?

If the remote server is linux, you can send it with bash scripts and netcat.  Netcat is a standalone binary that you can copy over to the remote server.  Then just pipe your data to netcat.  Here's an example:
echo "1000" | gawk '{print "test.test1 " $1 " " systime()}' | /usr/local/bin/netcat <IP_ADDRESS> 2003


How to run coffee script shebang?

In your document, the coffeescript is supported by doing some hack.
I am not able to follow based on your documents.
Can you show us the example on how to run coffeescript files using bitbar?

I agree that the documentation is a bit misleading. A much simpler fix would be to compile to JS and then test your plugin. For CoffeeScript, your shebang will look similar to this: #!/usr/local/bin/node /usr/local/bin/coffee . Simply the path to Node, then the path to CoffeeScript.


auto format removed space around equal sign in generic default declaration

VSCode Version: 1.26.1

Steps to Reproduce:

function foo<T extends string[]=[string]>() {

}
// should be
function foo<T extends string[] = [string]>() {

}

This issue was moved to Microsoft/TypeScript#26879


Unable to select Year when min and max dates are less than 1 year and overlap 2 unique years

Description
When I set a min and max dates than are less than 1 year but span over 2 unique years, I expect to be able to select both years in the Year View. Instead it only allows 1 year to select (the later one)
Link to Reproduction (or Detailed Explanation)
https://codesandbox.io/p/devbox/black-sun-gc88nf
Steps to Reproduce

Click on 'February 2019' button twice to take you to Year View
Click on year 2018 (no effect) year is not selected
Click on year 2019 - selects year as expected.

Expectation is both years should be selectable.
Ark UI Version
4.4.0
Framework

[X] React
[ ] Solid
[ ] Vue

Browser
Google Chrome
Additional Information
No response

Incase anyone looked at this I had put the incorrect dates in and have now fixed that.


Link from "Making queries" doc to "QuerySet API reference" doc

I wanted to filter a QuerySet by upper and lower bounds (on a DateTime field).
I found myself searching through "Making queries" and saw sections such as "Field lookups" showing __gt examples, but the __range section is not mentioned.
My hope is, linking from "Making queries" to "QuerySet API reference", will make it easier to jump from how queries conceptually work to more details around available filters / manager methods.

@tomleo Thanks :+1: , however "Field lookups" section already contains a reference to the "QuerySet API reference -> Field lookups":
The database API supports about two dozen lookup types; a complete reference
can be found in the :ref:`field lookup reference <field-lookups>`.

I think this is more appropriate place than the header of this topic.


Fix OS::execute() and OS::create_process() command line argument.

Fixes #45153

Thanks!

Thanks!


Improve rule datamodel

Added RuleMetadata equals and hashCode
Changed getParameters() so that it doesn't modify object state

retest this please

@yurloc Please advise when these PRs are good to go...

In the end I changed equals() to consider two objects equal even if one has parameters == null and the other one's parameters is an empty map. I believe this one is the only possible solution without affecting behavior of [get|set]Parameters() which already have a lot of usages.
@manstis ready

(master) http://github.com/droolsjbpm/drools/commit/730386c3c
(6.3.x) http://github.com/droolsjbpm/drools/commit/93677291d


Add slots to Model objects

Model objects have well defined attributes.
It would be nice to implement automatic __slots__ creation in the metaclass in order to improve memory usage

This has been addressed in v2.0.0a0


does the TreeDataProvider API only support typescript ?

I rigistered a provider,and create a TreeDataProvider class with javascript .but it not work

@zsmj1994 Not sure what is not working. Please let us know what is working with some examples or code snippets.


Compile time error -

Steps to Reproduce
I just started getting this error when trying to run my app. flutter run was working working fine and then I did some code cleanup and ran again and I am getting this error. The enviroment did not change and the logic of the code did not change just restructuring. Not quite sure where to even begin debuging this because all the files it's reporting are from the dart kernel.

Execute flutter run on the code sample
...
...

Expected results:
flutter compiles and runs app on device
Actual results:

Code sample

Logs

what's that?

@alexmarkov meant that the crash happens in the CFE (common front end) of Dart, which I am in charge of.
What version of Flutter/Dart are you using?

I'm running this on the beta channel with flutter 3.8.0-10.1 and dart 3.0.0

The crash happens in code that checks exhaustiveness for switch statements/expressions using the experimental "Patterns" feature. Did you enable the experiment or try to use patterns?

yes I am using records and patterns

Ok. Then you are hitting some of the cases we haven't implemented (fully) yet.

Is there a way to figure out which case that is so I can work around it?

Also this seems to be a new problem because I've had records and patterns implements for a couple weeks now and it's been working fine including on this version of dart and flutter

The crash in this case is trigger when an enum value used in a switch case hasn't had its value computed when the switch case is being processed. The order in which this occurs in dependent on the order of the code (within the library or the order of the imports). It is hard to avoid.
It might be that you can avoid triggering the exhaustiveness check that crashes by add a default case to the switch statement.

interestingly enough I haven't changed the switch statements or the code that returns the enum the only thing I did do was add explicit return types for my functions and removing them seems to compile the code again

@bibaswan-bhawal
Can you provide us a minimal code sample that shows this error log ? Also please provide the error log in formatted text for better viewing.
@johnniwinther Should this be directed to dart-lang repo for better tracking ?

I would if I knew what even caused this compile time error. This just seems like the exhaustive pattern matching algorithm for switch statements is going up against a race condition somewhere else in the code. I really have no clue what it could be racing against and for the time being upgrading to the master build channel has solved this.

I think we can close this. We'll track this in the Dart issue tracker: https://github.com/dart-lang/sdk/issues/51716


chore: bootstrap releases for path: libs/dynamic-api/src

Configuring release-please for path: libs/dynamic-api/src

will be replaced by the next


Add native support for WASM targets

Summary
This PR adds support for getting the configured system time zone via the js-sys crate for WASM targets. It also updates the dependencies to include inherent support for WASM targets by including the wasm-bindgen feature from the time crate. It also adds necessary attributes and dependencies for running all the existing tests against WASM targets.
I have a working implementation of this change here.
Reason for the change
I am building a cross-platform application that is a collection of useful widgets for developers. One of the features I currently support is converting a Unix timestamp to different locales. Previously, I was using the chrono-tz crate to get the IANA database, but I am working to adopt this crate instead as it patches https://github.com/advisories/GHSA-wcg3-cvx6-7396.
I was able to migrate my DateTime converter widget to use your crate easily, it only required patching the time crate to add the wasm-bindgen feature for WASM targets. I also wanted to add the system time zone feature, though, since that was not supported at all in chrono-tz, but it was failing for WASM targets since they don't have access to the Unix or Windows TZ syscalls.
Implementation details
The specific approach I took is based on https://stackoverflow.com/questions/61654015/how-can-i-get-the-browsers-local-timezone-using-rust-js-sys. It involves using the js-sys crate to invoke the Intl.DateTimeFormat object, which in its default form returns the time zone locale as configured in the browser/system runtime. Helpfully, the timeZone field is already formatted per the IANA standards, meaning it can easily be parsed by get_by_name, the same as the other targets.
Documentation
I've updated the system module with more verbose documentation and validated it using cargo doc. I've also added a new error Unsupported, which will be returned if get_timezone is called for an unsupported target platform (not WASM, Unix, or Windows).
Testing
I added support for testing against WASM browser targets using wasm-bindgen-test. This does not impact testing or building against other targets as the attribute is restricted to targets in the WASM family. Running these tests requires either the wasm-pack tool or the wasm-bindgen-cli be installed, otherwise the WASM binary cannot be run.
I've also updated the CI pipeline to add a new job that builds the crate against the wasm32-unknown-unknown target and runs all tests against Chrome, Firefox, and Safari.
All tests are passing:
➜  time-tz git:(evan/js-sys) ✗ cargo test --target wasm32-unknown-unknown --all-features
   Compiling time-tz v1.0.3 (/Users/esimk/source/time-tz)
    Finished test [unoptimized + debuginfo] target(s) in 3.19s
     Running unittests src/lib.rs (target/wasm32-unknown-unknown/debug/deps/time_tz-6ba78c3ee7233970.wasm)
Set timeout to 20 seconds...
running 12 tests

test time_tz::posix_tz::parser::tests::basic ... ok
test time_tz::tests::handles_backward_changeover ... ok
test time_tz::tests::handles_broken_time ... ok
test time_tz::tests::handles_after_changeover ... ok
test time_tz::tests::handles_forward_changeover ... ok
test time_tz::tests::dst ... ok
test time_tz::tests::london_to_berlin ... ok
test time_tz::tests::offsets_and_name ... ok
test time_tz::tests::find ... ok
test time_tz::tests::names ... ok
test time_tz::system::tests::get_timezone ... ok
test time_tz::binary_search::tests::test_binary_search ... ok

test result: ok. 12 passed; 0 failed; 0 ignored

Non-Goals
This change has only been validated against the wasm32-unknown-unknown target, as the wasm32-wasi target is still experimental and likely to change. wasm32-unknown-emscripten is generally viewed as legacy so I also have not validated against this target.
I have not added support for running tests against Node as my primary goal here was to support browser compatibility, since I compile natively for Windows/Unix targets. The wasm-bindgen-test crate does not provide good support for targeting tests at both Node and browsers as each test module needs to call the wasm_bindgen_test_configure macro to set up browser compatibility. I thought about exposing the choice to target testing at either browser or Node as a feature flag, but this would interfere with the ability to run test with the --all-features option.

Yeah, the situation for testing WASM targets is a bit messy, the existing test frameworks I found all required their own attributes to be added to the test methods. I can certainly look into swapping the attributes in a preprocessor but if you're fine just checking in the system.rs change for now, I can update the branch and contribute the preprocessor as a separate change.

Thanks for the latest changes, I'll merge this probably tomorrow as the computer I use right now does not have the publish key.

This is now available in v2.0.0. I had to bump major due to the change of error enum (new unsupported variant).


Could you add more details on data preparation?

I could see that the training code is looking for some file which is in json format. Could you provide details with respect to the structure in which the data has to be sent and if possible a example.

Hi, due to intellectual property issues, we can't put actual data examples in the code repo. The structure of the JSON files is as follows: {"review_body": "", "language": "", "label": ""}

Thanks you so much for the response. Will try this out.


Margo: fetching import paths up to 11 seconds

After last update, command ctrl+.+p makes for 11 seconds.
In status bar I see "Margo fetching import paths".

If you have lots of non-Go files in your GOPATH press ctrl+.,ctrl+. and type Edit MarGo Extension
You can use a program like below to filter out whatever directory or file you don't want to search,
like node_modules. After saving, restart Sublime Text.
package gosublime

import (
	"disposa.blue/margo"
	"disposa.blue/margo/meth/importpaths"
	"path/filepath"
	"strings"
)

func init() {
	margo.Configure(func(o *margo.Opts) {
		o.ImportPaths = importpaths.MakeImportPathsFunc(func(path string) bool {
			// note: the default filter includes node_modules

			// use the default filter
			return importpaths.PathFilter(path) &&
				// don't descened into huge node_modules directory
				!strings.Contains(path, filepath.Base("node_modules"))
		})
	})
}


Create flattering_of_list.py

Change the Directory from main to 'Python'. 😊

how to change directory from main to python
On Mon, Oct 24, 2022 at 3:10 PM VIKAS TRIPATHI @.***>
wrote:

Merged #4 https://github.com/ervikasji/Hacktoberfest2K22/pull/4 into
main.
—
Reply to this email directly, view it on GitHub
https://github.com/ervikasji/Hacktoberfest2K22/pull/4#event-7652282011,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AVXC7PDISIV2L4IYMILVK73WEZKSNANCNFSM6AAAAAARMQ2RDE
.
You are receiving this because you were assigned.Message ID:
@.***>


🧑💻 : Dark Mode is not performing fine

:red_circle: Title :  Under Dark Mode Things are not appropriate
:red_circle: Enhancement Aim : The Styling will be done for the Elements under Dark Mode
:red_circle: Changes :

Improve the font style of text
Improve the elements under Dark Mode
Improve the Alignment of Elements

Screenshots 📷

✅ Details to Include When Taking the Issue:
Name : Vansh Garg(Github username: @VanshGarg06)
Participant Role: GSSOC-EXT , Hacktoberfest

Happy Contributing! 🚀
Wishing you all the best on your open source journey. Enjoy! 😎

can you please tell me the directory where is the html document and other file of this is prensent


[+Price] AWS Cloud Driver Implementation

completed via https://github.com/cloud-barista/cb-spider/pull/986


APPS-3730 move into organization directory cms manifests

Whats the status on this?
Have those been transferred to release app and RGs?

ping @sergeyspryker

It can be closed. All the necessary updated already have been applied in RG as far i remember


ENH Speed up fwdret computation

By computing directly from prices rather than using pct_change().

Seeing a 4x increase in speed.

Have you tried multiple version of pandas?

Travis covers those.

I meant if you noticed a speed improvements on multiple pandas version or just one specific version (Quantopian's version I suppose)

I see, I haven't tested that, but I'd be surprised if there were big differences between versions there.


Numput missing parameter

input:
NumPut(1, NumPut(2, test, "int"), "int")
output:
NumPut("int", 1, test)
expected output:
NumPut("int", 2, "int", 1, test)

In v2, NumPut is changed to allow writing multiple entries into memory at one time, as in your 'expected output' example.
In v1, you can only write one item to memory at a time, but your input example tries to work around this by relying on the return value address and nesting two calls inside each other. I don't think the converter can handle this. We'd end up needing to write an entire expression parser to be able to check the return values of all functions.

Thanks for reply,
yea i know it's almost imposible to make parser from other function, but in this case only NumPut, not other functions, as the example I already gave, there is literally a missing value from the 'main' NumPut
Other example:
input:
NumPut(1, NumPut(2,  NumPut(3,  NumPut(4, test, "int"), "int"), "int"), "int")
converter output:
NumPut("int", 3, "int", 2, "int", 1, test)
see what i mean ?, the number 4 missing from output, it should output like this:
NumPut("int", 4, "int", 3, "int", 2, "int", 1, test)

I wasn't aware that NumPut already tried to handle this recursive case. It looks like @dmtr99 tried supporting it here in this commit https://github.com/mmikeww/AHK-v2-script-converter/commit/527b30932be7f2ff07a8fc935c137ea8c2f8a037 however unfortunately there were no tests added alongside. A day later it looks like he added this test (I would prefer that all tests get added in the same commit so its easier to find):
https://github.com/mmikeww/AHK-v2-script-converter/blob/master/tests/Test_Folder/External Libraries/NumputInNumput_WithContinuationSection.ah1
Since the converter already attempts to recursively unravel this, perhaps it is indeed fixable. I will take a look when I get some time, unless someone else does as well

I think the problem occurs because of the way this code is placed
https://github.com/mmikeww/AHK-v2-script-converter/blob/5c05902e1f22cb6f933e55a18d9116369eef01e2/convert/2Functions.ahk#L264
that code placed after this checks whether there is another NumPut
https://github.com/mmikeww/AHK-v2-script-converter/blob/5c05902e1f22cb6f933e55a18d9116369eef01e2/convert/2Functions.ahk#L259-L262
then if there is no other NumPut, then the 'main' value got ignored, because the loop already stopped
basically the solution is just move the ParBuffer assigment before the last check, i can help if you want

feel free to submit a pull request if you solve it.
make sure you add a few test files alongside


Switch (or supplement) the macos github CI runner to aarch64

https://github.com/JuliaLang/Example.jl/blob/63de11d03649f2fc86082a884d53670818af6f4b/.github/workflows/ci.yml#L24-L27
The aarch64 runners are available now:
https://github.blog/changelog/2023-10-30-accelerate-your-ci-cd-with-arm-based-hosted-runners-in-github-actions/
They should run tests faster.
Should we switch the CI.yml to use the new arm macs? or should it do both?

Although we noticed that it seems like GitHub uses an arm64 image for all the mac jobs, even if you specify arch: x64.......
Does anyone understand that? Is the arch field ignored?
CC @d-netto / @drvi

This was fixed, I believe. Anyways, this is now doing the right thing.


AZ CLI failing with Pip error on extension install.

az feedback auto-generates most of the information requested below, as of CLI version 2.0.62

Describe the bug

Az CLI version 2.18 fails with Pip error on extension install. (e.g. az extension add --source <path to .whl>)
cli.azure.cli.core.azclierror: An error occurred. Pip failed with status code 1. Use --debug for more information.
To Reproduce

az extension add --source <path to .whl>
Expected behavior

No failure
Environment summary

Az CLI Version: 2.18
MacOS, Zsh
Additional context

Also met the same error when adding extension. Workaround by upgrading pip version in az python path, but better to be solved by az cli.
See the below debug logs:
....
error: Can not find Rust compiler
Running setup.py install for cryptography: finished with status 'error'
Cleaning up...
Removing source in /tmp/pip-install-87ax_zl5/cryptography
Removing source in /tmp/pip-install-87ax_zl5/psutil
Removed build tracker '/tmp/pip-req-tracker-8c1b1o8u'
Command "/opt/az/bin/python3 -u -c "import setuptools, tokenize;file='/tmp/pip-install-87ax_zl5/cryptography/setup.py';f=getattr(tokenize, 'open', open)(file);code=f.read().replace('\r\n', '\n');f.close();exec(compile(code, __file
__, 'exec'))" install --record /tmp/pip-record-slbdhf8x/install-record.txt --single-version-externally-managed --compile --home=/tmp/pip-target-1wo4hi5b" failed with error code 1 in /tmp/pip-install-87ax_zl5/cryptography/
Exception information:
Traceback (most recent call last):
File "/opt/az/lib/python3.6/site-packages/pip/_internal/cli/base_command.py", line 143, in main
status = self.run(options, args)
File "/opt/az/lib/python3.6/site-packages/pip/_internal/commands/install.py", line 366, in run
use_user_site=options.use_user_site,
File "/opt/az/lib/python3.6/site-packages/pip/_internal/req/init.py", line 49, in install_given_reqs
**kwargs
File "/opt/az/lib/python3.6/site-packages/pip/_internal/req/req_install.py", line 791, in install
spinner=spinner,
File "/opt/az/lib/python3.6/site-packages/pip/_internal/utils/misc.py", line 705, in call_subprocess
% (command_desc, proc.returncode, cwd))
pip._internal.exceptions.InstallationError: Command "/opt/az/bin/python3 -u -c "import setuptools, tokenize;file='/tmp/pip-install-87ax_zl5/cryptography/setup.py';f=getattr(tokenize, 'open', open)(file);code=f.read().replace('\r\n
', '\n');f.close();exec(compile(code, file, 'exec'))" install --record /tmp/pip-record-slbdhf8x/install-record.txt --single-version-externally-managed --compile --home=/tmp/pip-target-1wo4hi5b" failed with error code 1 in /tmp/pip-ins
tall-87ax_zl5/cryptography/
cli.azure.cli.core.extension.operations: Command '['/opt/az/bin/python3', '-m', 'pip', 'install', '--target', '/home/aml-custom-module/.azure/cliextensions/azure-cli-ml', '/tmp/tmpgl54fk79/azure_cli_ml-<IP_ADDRESS>211468-py3-none-any.whl', '-
-extra-index-url', 'https://azuremlsdktestpypi.azureedge.net/modulesdkpreview', '-vv', '--disable-pip-version-check', '--no-cache-dir']' returned non-zero exit status 1.
cli.azure.cli.core.extension.operations: Pip failed so deleting anything we might have installed at /home/aml-custom-module/.azure/cliextensions/azure-cli-ml
cli.azure.cli.core.util: azure.cli.core.util.handle_exception is called with an exception:
cli.azure.cli.core.util: Traceback (most recent call last):
File "/opt/az/lib/python3.6/site-packages/knack/cli.py", line 233, in invoke
cmd_result = self.invocation.execute(args)
File "/opt/az/lib/python3.6/site-packages/azure/cli/core/commands/init.py", line 664, in execute
raise ex
File "/opt/az/lib/python3.6/site-packages/azure/cli/core/commands/init.py", line 727, in _run_jobs_serially
results.append(self._run_job(expanded_arg, cmd_copy))
File "/opt/az/lib/python3.6/site-packages/azure/cli/core/commands/init.py", line 720, in _run_job
six.reraise(*sys.exc_info())
File "/opt/az/lib/python3.6/site-packages/six.py", line 703, in reraise
raise value
File "/opt/az/lib/python3.6/site-packages/azure/cli/core/commands/init.py", line 698, in _run_job
result = cmd_copy(params)
File "/opt/az/lib/python3.6/site-packages/azure/cli/core/commands/init.py", line 331, in call
return self.handler(*args, **kwargs)
File "/opt/az/lib/python3.6/site-packages/azure/cli/core/init.py", line 816, in default_command_handler
return op(**command_args)
File "/opt/az/lib/python3.6/site-packages/azure/cli/command_modules/extension/custom.py", line 18, in add_extension_cmd
version=version, upgrade=upgrade)
File "/opt/az/lib/python3.6/site-packages/azure/cli/core/extension/operations.py", line 318, in add_extension
pip_extra_index_urls=pip_extra_index_urls, pip_proxy=pip_proxy, system=system)
File "/opt/az/lib/python3.6/site-packages/azure/cli/core/extension/operations.py", line 163, in _add_whl_ext
'Use --debug for more information.'.format(pip_status_code))
knack.util.CLIError: An error occurred. Pip failed with status code 1. Use --debug for more information.

HI @fengzhou-msft how's the cryptograpy upgrading status?

What's the workaround for this issue?

What's the workaround for this issue?

one way to workaround is upgrading pip version before az extension add, like this:
cd /opt/az/bin/
./python3 -m pip install --upgrade pip --user

@fengzhou-msft can we get an update regarding the issue?

We've pinned the cryptography in azure-cli in #15687, but it won't fix the installing issue of extensions as the extension will install cryptography again in its own directory and ignoring the cryptography already installed by azure-cli.
The fix needs to be done by the extension to either pin cryptography in their setup.py or remove any dependencies that depends on cryptography if they are already installed by azure-cli.
The latest azure-cli-ml has pinned cryptography to fix  installation.

Is there any workaround for this issue?
Unfortunately, I get the same error message when trying out chjinche's solution:
cd: no such file or directory: /opt/az/bin/
zsh: no such file or directory: ./python3

Is there any workaround for this issue?
Unfortunately, I get the same error message when trying out chjinche's solution:
cd: no such file or directory: /opt/az/bin/
zsh: no such file or directory: ./python3

Found the issue. I screened my debug log again and found the reference that my pip version was too old:
/usr/local/Cellar/azure-cli/2.20.0/libexec/bin/python -m pip install --upgrade pip
helped in my case.


DBName (--db-name) cli flag updates

For some reason on my local it has stopped picking up --build-dir flag. Not sure if this is an issue specific to me or something else.

Needs to be merged with avalanche-internal changes for DBType

Changing the base to dev, since we do not want to merge this to master until the corresponding flag change is released on avalanchego.


Add word matching functionality to search results

DataHub search results indicate which part of the returned item matches the search term. Implement this functionality in the custom front end.
The DataHub highlight colours are a WCAG fail (I think it's FG #8C8C8C and BG #E6F4FF).
But what they have got right is the semantically correct use of the <mark> tag - I think this is equivalent to, but preferred over the ARIA role="mark" role)
GOV.UK implements this by avoiding a background "highlight" and adjusting the font-weight instead. (Don't know if this is in the SCSS out of the box, or if it's a custom addition)
also noticed DH's post-processing to surround search terms in  tags isn't smart enough to handle multiple search terms ("offence" not highlighted in result 3)
check how a screen reader would handle the highlighted search terms
add to alpha assessment deck re usability and accessibility, pref screenshots

GOV.UK publications - search term highlight using font-weight:700

Raise WCAG fail with DataHub
Definition of done

[ ] Code to parse search results and modify markup
[ ] CSS change to "highlight" resulting tags
[ ] Test(s) written

Search searches on name, description, and on column names in a dataset. These are the items which should be highlighted/bolded upon searching.

the dataset name is already bold as part of the govuk class
search summary is being used rather than description on the search screen
will we be copying datahub to display information about column names which match the search term?

https://github.com/ministryofjustice/find-moj-data/pull/78
I have slightly got around the issue by surfacing the matched fields returned by the datahub GraphQL API

Highlighted markdown is rendered into a <strong> tag in html. This is theoretically something which screenreaders should be able to use, but when I used the Mac VoiceOver functionality to test it, there wasn't any difference with the surrounding text.


initial work for barplot

This is an initial draft PR that works for barplot: here is the issue at web-eda
Unlike the initial thought regarding this work, a possible solution I had suggested before turned out to be inappropriate for this task. Thus, switched back to an approach similar to legend/legend title's ellipsis with tooltip. However, the approach did not work for this task directly, so spent quite a long time to figure out the reason. It turned out that the tick label should have pointer-events attribute to activate tooltip.
Initial impression is that this works fine so will try to do the same thing for boxplot and see. Cleanup is not done yet either.

Now boxplot case is also added. It was quite tricky as it displayed strange labels, a mix of label with and without ellipsis: it happened whenever I changed data with ellipsis. Tried several other approaches, like onAfterPlot event instead of onUpdate: in this case, it worked fine however the plot area remained small, i.e., there was big gap between tick label and axis title because Plotly automatically computes plot size considering tick label. onAfterPlot does not offer to rescale plot after applying ellipsis.
After spending lots of time to find the reason, I finally figured it out that the boxplot utilized categoryorder/array, which caused the issue. That is, categoryarray should also be treated as ellipsis so that all labels are correctly displayed.
Here are sample screenshots - currently cutoff number for ellipsis is set to 20 characters like legend item:

However, it is also found that there is an issue in this implementation. I guess current work would work for over 95 % cases, but I could see that there existed labels with the same wording over 50 characters: thus those labels with cuttoff of 20 looked like the same. Unfortunately, Plotly recognizes the same labels as single data thus some of labels are not displayed (overlapped).
Accordingly, it is necessary to add a functionality to check all labels at data and make them different each other - will work for that.
Here is an example of this case. At PRISM2 ICEMR Control, Antimalaria medication is used. In this case they case four labels as follows:
"Artemether/lumefantrine - uncomplicated malaria and >14 days since last episode",
"Not applicable - no malaria diagnosed today",
"Quinine or IV artesunate - Complicated malaria and <=14 days since treatment with artemether/lumefantrine",
"Quinine or IV artesunate - Complicated malaria and >14 days since last episode"

As seen, 3rd and 4th labels are identical up to 51 characters. If it is plotted, then the plot is shown 3 labels, not 4 labels as the two are combined - see the next screenshot

Here is an

The issue of duplicate labels after applying ellipsis is resolved: added incremental space(s) in the next duplicate(s). For the same case above that showed missing label, new screenshots are attached here to show 3rd and 4th labels with tooltips

@bobular This works for barplot and boxplot. I wonder if Mosaic plot(s) should be applied as well ?

Hi DK - I'll test tomorrow - please could you tell me which Study you have
used (so I can find long variable names easily).
Barplot and boxplot have evenly spaced labels and the ellipsis truncation
of horizontal labels makes sense.
Mosaic is a bit different and the diagonal text is probably required.
Maybe don't do anything for Mosaic yet (and there's a ticket about very
narrow "bars" in Mosaic which cause label problems too).
On Thu, Sep 16, 2021 at 7:34 PM DK @.***> wrote:

@bobular https://github.com/bobular This works for barplot and boxplot.
I wonder if Mosaic plot(s) should be applied as well ?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/VEuPathDB/web-components/pull/211#issuecomment-921146668,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AACLLH7AZNHWBVJ7SNESH6LUCI2CHANCNFSM5EAQ4U4A
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

Hi DK - I'll test tomorrow - please could you tell me which Study you have used (so I can find long variable names easily). Barplot and boxplot have evenly spaced labels and the ellipsis truncation of horizontal labels makes sense. Mosaic is a bit different and the diagonal text is probably required. Maybe don't do anything for Mosaic yet (and there's a ticket about very narrow "bars" in Mosaic which cause label problems too).
…
On Thu, Sep 16, 2021 at 7:34 PM DK @.***> wrote: @bobular https://github.com/bobular This works for barplot and boxplot. I wonder if Mosaic plot(s) should be applied as well ? — You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub <#211 (comment)>, or unsubscribe https://github.com/notifications/unsubscribe-auth/AACLLH7AZNHWBVJ7SNESH6LUCI2CHANCNFSM5EAQ4U4A . Triage notifications on the go with GitHub Mobile for iOS https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Android https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

@bobular thank you for your detailed explanation 👍 In my screenshots, floor material ones are from GEMS1A, simply because GEMS1 was not available. The duplicate ones are from PRISM2 ICEMR Cohort. I believe you already know about your example used in the issue at https://github.com/VEuPathDB/web-eda/issues/328#issue-976539599 😄 However, actually I couldn't find it at PRISM2, but it was at PRISM ICEMR Cohort. Also noticed that there might be an update at data as now it shows 5 labels (also checked backend response, just in case), not 4 labels shown in your screenshot at the link earlier.

Hi DK - excellent testing/detective work/solution regarding the identical post-ellipsis labels.
I have a few style suggestions below.
I don't know what you mean about it showing 5 labels now. I see 4. Maybe we are using different back ends (I'm using https://feature.clinepidb.org/eda-data), or something has changed again. Let's not worry about it ;-)
Regarding Mosaic - I see that the angles are not the issue here. I think the ellipsis/mouseovers should be introduced on the independent axis, please. But I've noticed an issue with antimalarial medication x age group (below) that looks like it will complicate work on this component. Oh - I see from the back end data that there are 4 categories on the x-axis but the last two are printed on top of each other (you can see it is slightly bolder, actually). There is basically one main group that takes up 99.7% of the plot, so it's not a bug, per se.
I thought there was a ticket for Mosaic x-axis labels too close together, @SheenaTomko / @danicahelb ? I can't find it in any of the three main repos (web-eda, web-components, EdaNewIssues).

In summary - DK, you could work on ellipsis mouse-overs for Mosaic - but you would need to coordinate with @chowington regarding some open issues for that component. It might be best to close out this ticket and make a new one for Mosaic x-axis label ellipsis/mouseovers.

@bobular thank you for your detailed review, test, and explanation! 👍 👍 Yes I agree that we should separately handle Mosaic in the other ticket/PR. Just for clarification of 4 or 5 labels stuff, I am using d-falke so yes there may be changes in the backend.
I will respond to your suggestions/comments individually

@bobular all of your great suggestions are addressed, I suppose 😄

@bobular I think this is the ticket you wanted https://github.com/VEuPathDB/EdaNewIssues/issues/131. It hasn't been costed or added to a repo yet


Error during video demo

I am encountering the following error, while playing Abuse001_x264.mp4. What could be a fix for the following error?
(It works for other video like Arson005)

Could you share the video here, so that I can reproduce the error?

The video mentioned is a video in the UCF_Crime dataset.  You can find it in UCF_Crime/videos/Abuse or download it from the link:
https://1drv.ms/v/s!AqCREq0NjbXnrbBxED-Mic6czellUA?e=ODITDK

Could you share the video here, so that I can reproduce the error?

The video mentioned is a video in the UCF_Crime dataset. You can find it in UCF_Crime/videos/Abuse or download it from the link:
https://1drv.ms/v/s!AqCREq0NjbXnrbBxED-Mic6czellUA?e=ODITDK

Maybe this can help you
https://github.com/ekosman/AnomalyDetectionCVPR2018-Pytorch/issues/116#issuecomment-1140047699

Hi, sorry for the great delay. For me this works just fine:

@jesung1221 Let me know if you experience any other issues. Otherwise, I'll close this on 30/6. Thanks.


[dev-hilt] Provide an example of Service injection

As per the documentation, injection inside a Service should be as simple as:
@AndroidEntryPoint
class LocationService : Service() {
    @Inject lateinit var someImpl: SomeInterface
}

However, that same SomeInterface is already injected into various components such as activity and fragments.
The corresponding Hilt module is:
@Module
@InstallIn(ApplicationComponent::class)
object AppModule {
    @Singleton
    @Provides
    fun bindSomeInterface(): SomeInterface = SomeImpl()
}

It works great. However injection isn't done for services (I get UninitializedPropertyAccessException).
So I tried to create another module, using this time @InstallIn(ServiceComponent::class) - but no luck (there's an error telling that SomeInterface is already bound).
I'm running out of options. Any idea?

Im facing the same issue when try to inject a dependency inside a BroadcastReceiver and no example is provided.

@manuelvicnt Do you have an idea about this issue?

@peterLaurence Can you share the code you are using becuase i tested you approach and it works fine. I share with you my code.
 IBinder? = null

    @Inject
    lateinit var interfaceImpl: Interface

    override fun onCreate() {
        super.onCreate()

        interfaceImpl.run("Invoke from Service.")
    }
}

@HiltAndroidApp
class App: Application() {

    override fun onCreate() {
        super.onCreate()
        Log.d(App::class.simpleName, "onCreate: Created class")
    }
}

@AndroidEntryPoint
class MainActivity : AppCompatActivity() {
    override fun onCreate(savedInstanceState: Bundle?) {
        super.onCreate(savedInstanceState)
        setContentView(R.layout.activity_main)
        startService(Intent(this, HiltService::class.java))
    }
}

<application
    android:allowBackup="true"
    android:icon="@mipmap/ic_launcher"
    android:label="@string/app_name"
    android:roundIcon="@mipmap/ic_launcher_round"
    android:supportsRtl="true"
    android:theme="@style/AppTheme"
    android:name=".App">
    <activity android:name=".MainActivity">
        <intent-filter>
            <action android:name="android.intent.action.MAIN" />

            <category android:name="android.intent.category.LAUNCHER" />
        </intent-filter>
    </activity>

    <service android:name=".HiltService"/>
</application>

Output:
06-20 15:21:31.498 4919-4919/com.test.myapplication D/InterfaceImpl: run: Invoke from Service.

@cmpeguerog Thank you so much, I finally found my mistake. I had the same code as you, except for one little difference:
In my service, I was trying to access the injected object before super.onCreate()
It's so easy to make this mistake.. so for people reading this: hilt injection happens during super.onCreate() of your service.


SMA-153,SMA-150: Refactor

Summary

Import from OpenZeppelin instead of custom contracts wherever possible.
Move natspec, structs, error and events to dedicated interfaces.
Remove all Paymaster Contracts and Tests.

Related Issue: # (issue number)
Change Type

[ ] Bug Fix
[x] Refactor
[ ] New Feature
[ ] Breaking Change
[ ] Documentation Update
[ ] Performance Improvement
[ ] Other

Checklist

[x] My code follows this project's style guidelines
[x] I've reviewed my own code
[x] I've added comments for any hard-to-understand areas
[ ] I've updated the documentation if necessary
[x] My changes generate no new warnings
[ ] I've added tests that prove my fix is effective or my feature works
[x] All unit tests pass locally with my changes
[ ] Any dependent changes have been merged and published

Very good ! just two things:

as mentioned during the DSM, fixes/ is mostly when we need to fix something in prod, otherwise features/ should be used, even though refactoring is not a feature 😅
I think it would be better to have each interface inside their respective folders, e.g folder/File.sol and folder/IFile.sol or folder/File.sol and folder/interface/IFile.sol

It would be more organized to have each interface located in its respective folder. For example, folder/File.sol and folder/IFile.sol, or folder/File.sol and folder/interface/IFile.sol.
I see what you're saying, but for an external developer looking for just the interfaces, I think having it in one place makes more sense. Here's an example from uniswap's repo: https://github.com/Uniswap/v3-core/tree/main/contracts/interfaces

It would be more organized to have each interface located in its respective folder. For example, folder/File.sol and folder/IFile.sol, or folder/File.sol and folder/interface/IFile.sol.

I see what you're saying, but for an external developer looking for just the interfaces, I think having it in one place makes more sense. Here's an example from uniswap's repo: https://github.com/Uniswap/v3-core/tree/main/contracts/interfaces

Agree that's a good suggestion, in this case let's follow uniswap and move all the interfaces (modules included) into the interfaces folder and create sub-folders


[$250] Expense - On creating split expense via QAB, instead of next&save button split expense shown

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

Version Number: 9.0.60-0
Reproducible in staging?: Y
Reproducible in production?: Y
If this was caught on HybridApp, is this reproducible on New Expensify Standalone?: Y
Issue reported by: Applause Internal Team
Action Performed:
Pre:condition:

Create a room
Invite few members
Create a split expense

Steps:

Launch app
Tap fab - split expense via QAB
Enter an amount
Note instead "next" button split expense button displayed
Tap on split expense
Tap amount
Note instead "save" button split expense button displayed

Expected Result:
On creating split expense via QAB, instead of next&save button, split expense button must not be shown.
Actual Result:
On creating split expense via QAB, instead of next&save button, split expense button is shown.
Workaround:
Unknown
Platforms:

[x] Android: Standalone
[x] Android: HybridApp
[x] Android: mWeb Chrome
[ ] iOS: Standalone
[ ] iOS: HybridApp
[ ] iOS: mWeb Safari
[x] MacOS: Chrome / Safari
[x] MacOS: Desktop

Screenshots/Videos
https://github.com/user-attachments/assets/37da6c00-453d-4661-81c7-6465274a5ebb

View all open jobs on GitHub
Upwork Automation - Do Not Edit

Upwork Job URL: https://www.upwork.com/jobs/~021858387880530531401
Upwork Job ID:<PHONE_NUMBER>530531401
Last Price Increase: 2024-11-18

Issue OwnerCurrent Issue Owner: @mollfpr

Proposal
Please re-state the problem that we are trying to solve in this issue.
When creating split expense from QAB, Split Expense is shown on the amount page Button instead of next/save.
What is the root cause of that problem?
When we select split bill from QAB, we pass skipConfirmation as true.
https://github.com/Expensify/App/blob/058b3afc59bff2f5a5b38b2daca4346858239551/src/pages/home/sidebar/SidebarScreen/FloatingActionButtonAndPopover.tsx#L271-L273
If it's true, then we show Split Expense as the text.
https://github.com/Expensify/App/blob/058b3afc59bff2f5a5b38b2daca4346858239551/src/pages/iou/MoneyRequestAmountForm.tsx#L234-L245
However, pressing the button won't split the expense immediately and instead going to the confirm page. If we look at this comment, we only want to skip split if it's unconfigurable, and only scan is unconfigurable.
https://github.com/Expensify/App/blob/058b3afc59bff2f5a5b38b2daca4346858239551/src/pages/iou/request/step/IOURequestStepAmount.tsx#L181-L183
This means that we shouldn't skip confirmation for the manual amount.
What changes do you think we should make in order to solve the problem?

Since we only want to skip split scan expense, we can remove this code for split scan because it's inside the manual amount page.
https://github.com/Expensify/App/blob/058b3afc59bff2f5a5b38b2daca4346858239551/src/pages/iou/request/step/IOURequestStepAmount.tsx#L180-L201
And then update shouldSkipConfirmation to return false when it's split bill.
https://github.com/Expensify/App/blob/058b3afc59bff2f5a5b38b2daca4346858239551/src/pages/iou/request/step/IOURequestStepAmount.tsx#L78-L84
const shouldSkipConfirmation: boolean = useMemo(() => {
    if (isSplitBill || !skipConfirmation || !report?.reportID) {
        return false;
    }

    return !(ReportUtils.isArchivedRoom(report, reportNameValuePairs) || ReportUtils.isPolicyExpenseChat(report));
}, [report, skipConfirmation, reportNameValuePairs]);

external

Thank you guys for the proposals!

The manual group split can't be skipped but the button text is set to iou.splitExpense when iouType === CONST.IOU.TYPE.SPLIT is true.

@Krishna2323 The RCA should be clear in pointing out what's the start of the problem to the end. Here are some follow questions for your RCA.

Why the manual group can't be skipped?
Why the skipConfirmation value is true but the manual split can't be skipped?

For consistency we will also check IOURequestStepAmount, IOURequestStepScan & IOURequestStepDistance.

What check do we need to do?

We should update this if block to (skipConfirmation && iouType !== CONST.IOU.TYPE.SPLIT) and remove this part.

Also, the solution is not related to the main problem where the skipConfirmation has unintended value. So instead of adding an unnecessary check, we just need to have the skipConfirmation value false in the first place.

@NJ-2020 Your solution works only for the case when the QAB split expense is pressed. Is not working if we navigate to split manual from QAB split receipt/scan.

Since we only want to skip split scan expense, we can remove this code for split scan because it's inside the manual amount page.

@bernhardoj What's the initial reason for having the split scan on the manual amount page? I'm okay to remove it as long we have confirmed that the logic won't be used.

The alternative solution from @Krishna2323 and solution from @bernhardoj pretty much have the same approach to solving the issue, but they are ignoring the fact that skipConfirmation is true for the QAB split manual expense and it's wrong. If we decide that the QAB split manual expense can't skip the confirmation page, then we should set it that way in the first place to give clarity.

@NJ-2020 Let me know the result with your solution of this step case:

Go to group chat
Create a scan split expense
Click on the QAB split receipt
Press on the manual tab

Could you confirm if the text is shown next & save?

@NJ-2020 Here's there result I've got when trying your solution. The text still shows split expense.
https://github.com/user-attachments/assets/c591b22b-ae4a-4a69-a7e5-e7451483d343
The reason why this happens is because when selecting QAB split receipt where it has the skipConfirmation true and it doesn't change when tab on the manual or distance expense.

but they are ignoring the fact that skipConfirmation is true for the QAB split manual expense and it's wrong.

I mentioned it here that we pass skipConfirmation as true and it's not wrong. All QAB should pass it as true,

Including this distance QAB.
https://github.com/Expensify/App/blob/058b3afc59bff2f5a5b38b2daca4346858239551/src/pages/home/sidebar/SidebarScreen/FloatingActionButtonAndPopover.tsx#L275-L277
The skipConfirmation basically tells whether this action is a QAB or not.
shouldSkipConfirmation will be the one that decide whether it's skippable or not.
https://github.com/Expensify/App/blob/058b3afc59bff2f5a5b38b2daca4346858239551/src/pages/iou/request/step/IOURequestStepAmount.tsx#L78-L84

@bernhardoj Yes you did mention it in the RCA, but in your solution, you leave it as is.

I'm saying every QAB should pass the skipConfirmation. Currently, only split expense QAB that passes it as false.
https://github.com/Expensify/App/blob/445fc4258bc4c7e0d43e833c724587b917222cda/src/pages/home/sidebar/SidebarScreen/FloatingActionButtonAndPopover.tsx#L271-L273
Otherwise, this will happen which you mentioned in your comment.

The reason why this happens is because when selecting QAB split receipt where it has the skipConfirmation true and it doesn't change when tab on the manual or distance expense.

but in your solution, you leave it as is.

The skipConfirmation basically tells whether this action is a QAB or not.

shouldSkipConfirmation will be the one that decide whether it's skippable or not.

PR is ready
cc: @mollfpr

BugZero Checklist:

[ ] [Contributor] Classify the bug:

Bug classification

Source of bug:

[ ] 1a. Result of the original design (eg. a case wasn't considered)
[ ] 1b. Mistake during implementation
[ ] 1c. Backend bug
[ ] 1z. Other:

Where bug was reported:

[ ] 2a. Reported on production (eg. bug slipped through the normal regression and PR testing process on staging)
[ ] 2b. Reported on staging (eg. found during regression or PR testing)
[ ] 2d. Reported on a PR
[ ] 2z. Other:

Who reported the bug:

[ ] 3a. Expensify user
[ ] 3b. Expensify employee
[ ] 3c. Contributor
[ ] 3d. QA
[ ] 3z. Other:

[ ] [Contributor] The offending PR has been commented on, pointing out the bug it caused and why, so the author and reviewers can learn from the mistake.
Link to comment:

[ ] [Contributor] If the regression was CRITICAL (e.g. interrupts a core flow) A discussion in #expensify-open-source has been started about whether any other steps should be taken (e.g. updating the PR review checklist) in order to catch this type of bug sooner.
Link to discussion:

[ ] [Contributor] If it was decided to create a regression test for the bug, please propose the regression test steps using the template below to ensure the same bug will not reach production again.

Regression Test Proposal Template

[ ] [BugZero Assignee] Create a GH issue for creating/updating the regression test once above steps have been agreed upon.
Link to issue:

Regression Test Proposal
Precondition:

Test:

Do we agree 👍 or 👎

@mollfpr for the steps above!

Contributor: @bernhardoj paid $250.00 via NewDot
Contributor+: @mollfpr  owed $250.00 via NewDot

Requested in ND.

BugZero Checklist:

[x] [Contributor] Classify the bug:

Bug classification

Source of bug:

[ ] 1a. Result of the original design (eg. a case wasn't considered)
[x] 1b. Mistake during implementation
[ ] 1c. Backend bug
[ ] 1z. Other:

Where bug was reported:

[x] 2a. Reported on production (eg. bug slipped through the normal regression and PR testing process on staging)
[ ] 2b. Reported on staging (eg. found during regression or PR testing)
[ ] 2d. Reported on a PR
[ ] 2z. Other:

Who reported the bug:

[ ] 3a. Expensify user
[ ] 3b. Expensify employee
[ ] 3c. Contributor
[x] 3d. QA
[ ] 3z. Other:

[x] [Contributor] The offending PR has been commented on, pointing out the bug it caused and why, so the author and reviewers can learn from the mistake.
Link to comment: https://github.com/Expensify/App/pull/40386/files#r1886532266

[x] [Contributor] If the regression was CRITICAL (e.g. interrupts a core flow) A discussion in #expensify-open-source has been started about whether any other steps should be taken (e.g. updating the PR review checklist) in order to catch this type of bug sooner.
Link to discussion: Not a critical

[x] [Contributor] If it was decided to create a regression test for the bug, please propose the regression test steps using the template below to ensure the same bug will not reach production again.

[x] [BugZero Assignee] Create a GH issue for creating/updating the regression test once above steps have been agreed upon.
Link to issue:

Regression Test Proposal
Precondition:

Create a split expense in a room or group chat

Test:

Tap fab - split expense via QAB
Enter an amount
Verify the button text is "Next" and press it
Press the amount field
Verify the button text is "Save"

Do we agree 👍 or 👎

https://github.com/Expensify/Expensify/issues/453694

$250 approved for @mollfpr


Fix labels of conditional histograms and improve the charts readability

Conditional histograms had incorrect bar labels. I fixed it and enlarged all the descriptions on the charts.

Merging charts improvements


last slides (1-2) in container with slidesPerView: 'auto' sometimes not visible

Hi,
Thanks for the great work.
I have an application where I use the following Properties for the Swiper component.
{
          direction: 'horizontal',
          loop: false,
          containerClass: 'swiper-container',
          slidesPerView: 'auto'
}

sometimes the last 1 or 2 slides are not visible and when I try to swipe to them I can see them but if I let go it always swipes back to the last slide that is visible. I tried multiple update mechanisms but the bug seems to be there. It's hard to reproduce because it happens randomly.
Do you have any idea why this is happening and how I can reproduce it better?

did you find a solution ?

I did not, but today I opened an issue at the nolimits4web/swiper repo, maybe they can help me with it.
I'll keep this issue updated.
This is the issue: https://github.com/nolimits4web/swiper/issues/3611

solution: https://github.com/nolimits4web/swiper/issues/3611
Hope it helps!


Metro 20 improve web test coverage topic

Codecov Report

Merging #337 (65100fe) into master (d422ca5) will decrease coverage by 1.64%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master     #337      +/-   ##
==========================================
- Coverage   43.69%   42.05%   -1.65%
==========================================
  Files         123      123
  Lines        6340     6340
==========================================
- Hits         2770     2666     -104
- Misses       3357     3477     +120
+ Partials      213      197      -16

Flag
Coverage Δ

integration
?

unittests
42.05% <ø> (+0.47%)
:arrow_up:

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

Impacted Files
Coverage Δ

pkg/messagebroker/factory.go
33.33% <0.00%> (-33.34%)
:arrow_down:

pkg/messagebroker/kafka.go
10.26% <0.00%> (-30.55%)
:arrow_down:

internal/topic/core.go
52.80% <0.00%> (+23.99%)
:arrow_up:

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 d422ca5...65100fe. Read the comment docs.


Minor improvements

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

Project coverage is 100.00%. Comparing base (f711eb9) to head (7cfbd12).

Additional details and impacted files
@@            Coverage Diff            @@
##              main      #396   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files          109       109
  Lines         3322      3322
  Branches       270       270
=========================================
  Hits          3322      3322

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


Monogamous and non-monogamous long-term partner infection

Hi @mmcleod89 there is still quite a bit of work to finish this up but if you could have a look and let me know what you think, that would be great!


fix: missing viewport.js file in publish

should add test cases for this stuff

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Finish moving conditions logic into other property checks

Issue #, if available:
Description of changes:

Add better conditions support for Inclusive, Exclusive, and AtLeastOne rules

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Codecov Report

Merging #680 into master will decrease coverage by 0.09%.
The diff coverage is 73.58%.

@@            Coverage Diff            @@
##           master     #680     +/-   ##
=========================================
- Coverage   85.22%   85.12%   -0.1%
=========================================
  Files         116      116
  Lines        6685     6702     +17
  Branches     1583     1591      +8
=========================================
+ Hits         5697     5705      +8
- Misses        623      631      +8
- Partials      365      366      +1

Impacted Files
Coverage Δ

...rc/cfnlint/rules/resources/properties/Inclusive.py
77.77% <70.58%> (-7.23%)
:arrow_down:

...c/cfnlint/rules/resources/properties/AtLeastOne.py
77.77% <73.68%> (-6.84%)
:arrow_down:

...rc/cfnlint/rules/resources/properties/Exclusive.py
90.69% <76.47%> (-9.31%)
:arrow_down:

src/cfnlint/__init__.py
87.23% <0%> (+0.13%)
:arrow_up:

src/cfnlint/conditions.py
91.27% <0%> (+0.72%)
:arrow_up:

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 ee2ef4e...50d310c. Read the comment docs.


Briefcase CLI - Password/Pem during export of project level encrypted data

Software versions
Briefcase v1.17.4
Problem description
Working with a project level encrypted data set on ODK Central that has been downloaded through ODK Briefcase, the ODK Briefcase CLI appears to have no method to export data as there is no PEM file for the data set and no option for entering a decryption passphrase.
Steps to reproduce the problem
Create a project level encrypted dataset
Download with Briefcase and try to export
Expected behavior
No cypher warning

This is more about rethinking how Central makes project-level encrypted data available so I've moved the issue. It's not immediately actionable so I'll close it for now but it will be available through search. Thanks for detailing what you need and why, @chrissyhroberts.


Visualize the final prediction

Dear Sanghun Jung,
Thank you so much for your amazing works, I was wondering if you could provide the source which helps visualize the b) Unexpected detected and c) final prediction as you stated in the paper?
Thank you and looking forward to hearing from you!

Hello, sorry for the late reply and thank you for your interest in our work!
We currently don't have visualization codes that we used for figures.
For (b) Unexpected detected in the Teaser figure, we visualize them using heatmap, and for the figure (c), we add CYAN color for the OOD class and modified the cityscapes_labels.py code to visualize the segmentation map.
Hope this helps you and please let me know if you have any questions :)
Thank you.


Unable to generate build after project initialization. ADB throwing error

Environment:
OS: Linux 4.4
Node: 6.14.3
Yarn: Not Found
npm: 3.10.10
Watchman: Not Found
Xcode: N/A
Android Studio: Not Found
Packages: (wanted => installed)
react: 16.3.1 => 16.3.1
react-native: 0.55.4 => 0.55.4
OS: Ubuntu 16.04
Android Debug Bridge version: 1.0.40
ADB Version: 4797878
React Native Version: 0.55.4
Emulator: Genymotion
Debug Device: Nexus 5 Android 6
Here is my output:
$ react-native run-android
Scanning folders for symlinks in /home/vivekb/reactnative/tech_stack/node_modules (27ms)
JS server already running.
Building and installing the app on the device (cd android && ./gradlew installDebug)...

What went wrong:
Execution failed for task ':app:installDebug'.

com.android.builder.testing.api.DeviceException: Could not create ADB Bridge. ADB location: /home/vivekb/Android/Sdk/platform-tools/adb

Try:
Run with --stacktrace option to get the stack trace. Run with --info or --debug option to get more log output.

BUILD FAILED
Total time: 12.78 secs
Could not install the app on the device, read the error above for details.
Make sure you have an Android emulator running or a device connected and have
set up your Android development environment:
https://facebook.github.io/react-native/docs/getting-started.html
When I look for adb processes it shows something like this:
$ ps auwx | grep adb
vivekb   12588  0.0  0.0 331664  3736 ?        Ssl  14:28   0:00 adb -L tcp:5037 fork-server server --reply-fd 4
vivekb   12600  0.0  0.0  28268  3224 ?        Sl   14:29   0:00 /home/vivekb/genymotion/tools/adb -s <IP_ADDRESS>:5555 shell
vivekb   12601  0.0  0.0  20072  3288 ?        S    14:29   0:00 /home/vivekb/genymotion/tools/adb -s <IP_ADDRESS>:5555 logcat -v time
vivekb   12687  0.0  0.0  14232   824 pts/2    S+   14:30   0:00 grep --color=auto adb
after killing the process or firing "adb kill-server" adb server automatically starts after some time.
Stuck here as can't install the development app onto emulator or device.
Please help.

Updated with react-native info output

#8401 helped me.

#8401 helped me.


conda fails to create environment from yml

Discussed in https://github.com/TorchSSL/TorchSSL/discussions/10

Originally posted by FadouaKhm November 18, 2021
Most probably, it fails because the YAML includes platform-specific build constraints, which makes it harder to transfer across platforms.
Some suggested solutions:

Omit the build info from the environment definition export:
conda env export -n ssl -f environment.yml --no-builds

Export only what Conda refers to as explicit specifications.:
conda env export -n ssl -f environment.yml --from-history

Discussed in #10
Originally posted by FadouaKhm November 18, 2021 Most probably, it fails because the YAML includes platform-specific build constraints, which makes it harder to transfer across platforms. Some suggested solutions:

Omit the build info from the environment definition export:
conda env export -n ssl -f environment.yml --no-builds
Export only what Conda refers to as explicit specifications.:
conda env export -n ssl -f environment.yml --from-history

UPDATE: I solved the issue by removing the system-dependant packages and installing them manually later on.


feat(plugin-seo): add Chinese translation

Description

[x] I have read and understand the CONTRIBUTING.md document in this repository.

Type of change

[ ] Chore (non-breaking change which does not add functionality)
[ ] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] Change to the templates directory (does not affect core functionality)
[ ] Change to the examples directory (does not affect core functionality)
[ ] This change requires a documentation update

Checklist:

[ ] I have added tests that prove my fix is effective or that my feature works
[ ] Existing test suite passes locally with my changes
[ ] I have made corresponding changes to the documentation

Let me know if anything needs to be done to get this PR process 🥰


Uml 1141 create service resources

Purpose
Briefly describe the purpose of the change, and/or link to the JIRA ticket for context
Fixes Ticket: ####
Approach
Explain how your code addresses the purpose of the change
Learning
Any tips and tricks, blog posts or tools which helped you. Plus anything notable you've discovered about the service
Checklist

[ ] I have performed a self-review of my own code
[ ] I have updated documentation (Confluence/GitHub wiki/tech debt doc) where relevant
[ ] I have added tests to prove my work
[ ] The product team have tested these changes

hmm, I just noticed that the names of the jobs in circleci could be updated to account/environment instead of shared/service

hmm, I just noticed that the names of the jobs in circleci could be updated to account/environment instead of shared/service


Support for Authentik SSO

Hi,
i like firezone a lot and would love to have support for Authentik as an SSO provider.
https://github.com/goauthentik/authentik
Having a gerneral Oauth2 or an example config with oauth_proxy or vouch proxy would also be a good and more general approach.
Thanks!

@fnpanic Thanks for the feature request. Would generic OIDC support like in #501 solve your use-case?

Yes, a generic oidc or oauth2 would be awesome.

@fnpanic #586 has been merged to support this and will be available in the next release.


thellier_gui

has this bug:
Traceback (most recent call last):
File "/Users/ltauxe/Programs/PmagPy/GIT/PmagPy/programs/thellier_gui.py", line 1629, in on_prev_button
self.update_selection()
File "/Users/ltauxe/Programs/PmagPy/GIT/PmagPy/programs/thellier_gui.py", line 1518, in update_selection
self.update_GUI_with_new_interpretation()
File "/Users/ltauxe/Programs/PmagPy/GIT/PmagPy/programs/thellier_gui.py", line 5840, in update_GUI_with_new_interpretation
self.stat_windows[short_stat].SetBackGroundColour(wx.Green)
AttributeError: 'TextCtrl' object has no attribute 'SetBackGroundColour'

and thellier_gui fails to print the B_anc value and sometimes also the sample mean, even if there is a valid one.

here is the symptom and the upgraded data file:

Benaras.zip

I found the bug - line 5840 had some typos (wx.Green) and SetBackGroundColour instead of wx.GREEN and SetBackgroundColour.


deny_rights :write, "Administrators" does not function properly on Windows

for the directory resource, deny_rights :write, "Administrators" adds two permissions to the resource:
a Deny "Write" permission on subfolders and files
a Deny "Write, Special" permission on this folder only - which includes Deny, "Read Permissions"
On Windows, Deny Read Permissions actually prevents the end user from opening the folder/file, so deny_rights :write is actually denying read as well.

ping @chef/client-windows

Repro'd on 14.2.0
PS C:\vagrant> gc repro.rb
directory 'c:\vagrant\test' do
  deny_rights :write, "Administrators"
end

PS C:\vagrant> chef-apply repro.rb
Recipe: (chef-apply cookbook)::(chef-apply recipe)
  * directory[c:\vagrant\test] action create
    - create new directory c:\vagrant\test
    - change dacl
PS C:\vagrant> Get-Acl test | fl

Path   : Microsoft.PowerShell.Core\FileSystem::C:\vagrant\test
Owner  : BUILTIN\Administrators
Group  : WIN-G6GPLUDQKHG\None
Access : BUILTIN\Administrators Deny  Write, ReadPermissions, Synchronize
         BUILTIN\Administrators Deny<PHONE_NUMBER>
         NT AUTHORITY\SYSTEM Allow  FullControl
         BUILTIN\Administrators Allow  FullControl
         BUILTIN\Users Allow  ReadAndExecute, Synchronize
         BUILTIN\Users Allow  AppendData
         BUILTIN\Users Allow  CreateFiles
         CREATOR OWNER Allow  268435456
Audit  :
Sddl   : O:BAG:S-1-5-21-1214013728-3226316827-468114229-513D:AI(D;;FW;;;BA)(D;OICIIO;GW;;;BA)(A;OICIID;FA;;;SY)(A;OICII
         D;FA;;;BA)(A;OICIID;0x1200a9;;;BU)(A;CIID;LC;;;BU)(A;CIID;DC;;;BU)(A;OICIIOID;GA;;;CO)

PS C:\vagrant>


Add support for processing ad tokens

Mopidy-Pandora just skips over ads at the moment. This is not by design but due to the fact that advertisement tracks require special processing.
It would probably be a good idea to provide support for playing ads in order to align with the standard Pandora clients.
Skipping ads will probably also result in the user's Pandora account being disabled if done consistently.
I've started an implementation to support playing ads in a separate branch at https://github.com/jcass77/mopidy-pandora/tree/enhance/ad_support, but it requires some changes to the underlying pydora library before we can roll it out (https://github.com/mcrute/pydora/pull/26)

Adds are now being processed correctly in the 'develop' branch.


Instructions for SSL setup work on mac only, not on linux

The scripts provided at http://bosh.io/docs/director-certs.html only provide working private keys for the certificates, if executed on Mac OS. If you execute it on Linux, it doesn't produce correct keys. You have to run openssl rsa -in uaa-web.key -out uaa-web.rsa.key and openssl rsa -in uaa-sp.key -out uaa-sp.rsa.key to convert them properly.
This is probably due to different versions of openssl on both Operating Systems. Maybe we can find scripts working on both envs?

@voelzmo PR?

@voelzmo I believe this was before vars so the CLI can dynamically generate its own certs. I'm going to close this issue.


Aync Action Demo

Hi there,
Loving the simplicity of ReSub so far, but like with many other projects in this ecosystem I am at a loss until I can wrap my head around an async action example to say hit an API, get some data, and update store/state unidirectionally.
Any guidance or a demo would be greatly appreciated.

We tend to do something of the following pattern:
private _data: Something;
@autoSubscribe
getData(): Something {
if (!this._data) {
RestClient.fetchData().then(data => {
this._data = data;
this.trigger();
});
}
return this._data;
}
You can obviously make it more complicated with booleans to make sure you don't double-load it and such, but the basic principle is that easy.  You return nothing (which means you're loading), the component should render it as such.  Then when it loads, it stores it and re-triggers, causing an automatic update which re-renders the proper data.

This was a huge help. Got it running.
Thanks very much!


Dont Work

Assetio failed
expression pc != nullptr

@carlospombo Thank you for your first ever report.
In a terminal outside of atom run and paste the output of
apm ls
atom --version
apm --version
Also please provide screenshots of issue as supporting information.
Also please read #543

@ivankravets please close duplicate of https://github.com/platformio/platformio-atom-ide-terminal/issues/584


[3.0] upgrade: make a backup of the important database state

in some occasions the files have disappeared for unknown reasons after
the admin server upgrade.
not only because of that it makes sense to copy them away to a safe
place.

needs forward port

I can open another PR for importing those files from crowbar-init, but lets get this discussed/merged before

So... maybe it makes sense to make to backup anyway, but this seems to hide the real error...

So... maybe it makes sense to make to backup anyway, but this seems to hide the real error...

yes, backing up this file is really crucial here. we can add a big warning in crowbar-init when it can't find the file in the db folder so the real error won't be that hidden

@MaximilianMeister could you please reference the bug in the commit message (https://bugzilla.suse.com/show_bug.cgi?id=1012177) and prepare the change for crowbar-init?

@MaximilianMeister could you please reference the bug in the commit message (https://bugzilla.suse.com/show_bug.cgi?id=1012177) and prepare the change for crowbar-init?

@jsuchome done


Suboptimal layout when 'Boilerplate: table-of-contents: off' is used

When you turn Table of Contents generation off, in sufficiently wide windows the document isn't centered; it's instead off to the right. There's all this whitespace on the left where the ToC would have gone.

Huh, yeah, I guess the ToC is pretty intimately tied into the default styling. I'll have to do some minor hackery for this; we do already have good styles that should ignore it (for when the ToC is inline, rather than floating), so I'll see if I can just force it over into that case.


don't see trusted option after setting up admin user

Hi,  after i setting up admin user, i still see Trusted option in  Project settings.
file: drone.sh
docker run -d --name=drone \
  ...........
  --volume=$(pwd)/drone:/data \
  --env=DRONE_USER_CREATE=username:jarrysix@gmail.com,admin:true \
  --env=DRONE_GITLAB_SERVER=https://gitlab.com \
  --env=DRONE_GITLAB_CLIENT_ID=${GITLAB_CLIENT_ID} \
  --env=DRONE_GITLAB_CLIENT_SECRET=${GITLAB_CLIENT_SECRET} \
  --publish=8086:80 \
  --restart=always \
  drone/drone:1

run docker logs drone shows:
{"level":"info","msg":"main: internal scheduler enabled","time":"2020-04-30T00:23:04Z"}
{"admin":true,"level":"info","login":"jarrysix@gmail.com","machine":false,"msg":"bootstrap: account created","time":"2020-04-30T00:23:04Z","token":"Ga3ZlXhWpQG8waXu7iGeGGehSZirlAEM"}

:wave: hey there, per our issue template, for end-user support please use one of our support channels, such as our chatroom or our mailing list.


Fix - error checking asset

🩹Fix the error by always returning a boolean in isClassicStellarAsset to solve issue #302 .
[Aditional]
🩹Fix routes in handleCurrencyBSelect in AddLiquidityComponent to prevent errors.

-🩹Fix always requiring trustline bug on mint custom tokens


Refactors rtm handling

Also does a deep clone of config to prevent leaking internal modifications.
closes #25

whooops, wrong base

Coverage remained the same at 100.0% when pulling c437c0c13efc745b7f69791d6ce4f026cd071f30 on rtm into 35bd2e1ddbfaa8df653c79b7d4a0327a868fd6f9 on master.


JAAS Configuration overwrites Configurations created by other libraries

In what version(s) of Spring for Apache Kafka are you seeing this issue?
We see this issue in 2.8.11, but I believe this has existed since GH-201
Describe the bug
When utilizing Kerberos authentication with both Spring Kafka and the Microsoft SQL Server JDBC Driver, a JAAS Configuration conflict occurs depending on the order of Bean startup.  Microsoft's SQL JDBC driver creates a default Kerberos JAAS Configuration on startup, and if it starts after Spring Kafka, it will delegate requests for the KafkaClient configuration.
However, if Spring Kafka initializes after the SQL Datasource, the JAAS Configuration that Microsoft's SQL driver created is overwritten by KafkaJasLoginModuleInitializer.java:127-144
Configuration.setConfiguration(new InternalConfiguration(configurationEntries));

This creates a complex to troubleshoot issue, as bean startup order determines whether or when exactly the SQL connections will begin to fail with this error:
No LoginModules configured for SQLJDBCDriver at javax.security.auth.login.LoginContext.init(LoginContext.java:264)
To Reproduce
To reproduce this error, you need an environment with a Kerberos Domain Controller, a Microsoft SQL Server (tested with 2017 and 2019), and a Kafka installation, where both Kafka and SQL Server are configured to use Kerberos/SASL authentication.  Configure the Spring Boot datasource JDBC URL to use 'integratedSecurity=true;authenticationScheme=JavaKerberos;encrypt=true` and Configure Spring Kafka to use Kerberos.
Expected behavior
A Spring application configured with Spring-Kafka and a Microsoft SQL data source using Kerberos authentication should be able to coexist.  They should both be able to use their default created Kerberos configurations, and connect without issue.
Sample
I don't have a sample to share as the working examples are behind a corporate firewall.  This problem (or fix) doesn't require any knowledge of the corporation itself or corporate apps, it's the coexistance of Spring Kafka and mssql-jdbc.
I'd like to propose patching InternalConfiguration inside of KafkaJasLoginModuleInitializer to work similar to Microsoft's JaasConfiguration:
    JaasConfiguration(Configuration delegate) {
        this.delegate = delegate;
        this.defaultValue = generateDefaultConfiguration();
    }

    @Override
    public AppConfigurationEntry[] getAppConfigurationEntry(String name) {
        AppConfigurationEntry[] conf = delegate == null ? null : delegate.getAppConfigurationEntry(name);
        // We return our configuration only if user requested default one
        // In case where user did request another JAAS Configuration name, we expect he knows what he is doing.
        if (conf == null && name.equals(SQLServerDriverStringProperty.JAAS_CONFIG_NAME.getDefaultValue())) {
            return defaultValue;
        }
        return conf;
    }

By adding a delegate to InternalConfiguration similar to above, the KafkaClient configuration would be keeping a pointer to the previous Configuration.  It could then delegate requests for configuration names to the previous configuration, whatever it might be, and otherwise return its own configuration entries as it does today.
Let me know if you need any additional detail.  I think the patch will be very small and easy to unit test.

Thanks!  I agree it's an enhancement.  This small improvement will make it even easier for everything to "just work."

Agreed. But I haven’t had a chance to look at it yet, will do so early next week.

@caematos Disable it in what way? It should only be present if you define it as a bean, or if you enable it in Boot via https://docs.spring.io/spring-boot/docs/current/reference/html/application-properties.html#application-properties.integration.spring.kafka.jaas.enabled

i faced same issue in spring cloud stream  with kafka binder.
Spring boot version: 2.7.15
spring-cloud.version: 2021.0.8

This is implemented in 3.0.9, used by Boot 3.0.x and compatible Spring Cloud versions.

Thank you :) , Unfortunately I cannot upgrade from java 11 to make use of Boot 3.0.x. Thank you for quick reply.


ci: update actions to latest

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 100.0%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
64

Relevant Lines:
64

💛  - Coveralls


PACKAGE_NAME_INCORRECT_PATH KMP support

I have a kotlin multiplatform module, and it has the following structure:
module
	build.gradle
	src
		androidMain
			com.org.module
				*.kt files
		iosMain
			com.org.module
				*.kt files
		commonMain
			com.org.module
				*.kt files

In this case PACKAGE_NAME_INCORRECT_PATH work is incorrect:

package com.org.module

package com.org.commonMain.kotlin.com.org.module

hi, thank you for your feedback! We will investigate and fix it this week

actually it will be a little bit hard to fix properly, but we will try to workaround it, will be fixed in 0.1.4.
Until that you can workaround it by disabling this particular rule.


Test what happens if some peers are not endorsing peers

Apparently this could be done by not setting the CORE_PEER_GOSSIP_EXTERNALENDPOINT
means they should not be used to endorse for evaluate or submit, but will just commit

This doesn't work for the gateway as it determines the peers to use by those participating in the channel


[Utilities - Currency converter] Add a graph to show currency rate trends

Steps to reproduce

Go to navigation drawer -> utilities -> currency converter

Expected behaviour
After the user has selected source & destination cities, and press Convert, show a graph at the bottom of the screen showing the change in currency rate over time
Endpoint to use
get-all-currency-rate/<str:start_date>/<str:end_date>/<str:source_currency_code>/<str:target_currency_code>

I want to take up this issue

Sure! @Chinmay-KB

@Swati4star
Any progress here? Can I work on this?

@RanjanDharmraj I think you can take this up. There has been no progress.

@Swati4star
What is simple date format for start date and end date?

it will be great if you can provide me sample request/response

Did you try out the endpoint given in description?

yes I tried, its throwing 500 server error.

@RanjanDharmraj The date format is like this : %Y-%m-%d
Same request
https://project-travel-mate.herokuapp.com/api/get-all-currency-rate/2018-03-01/2018-03-07/usd/inr

Response
[
    65.325,
    65.261431,
    65.1967000002,
    65.1967000002,
    65.1213999998,
    64.8249999997,
    64.8842999999
]


Mina_base: 2 simple fixes

Fix warnings 16 (see details of first commit)
Remove unneeded monadic style

!ci-build-me


node_modules error when i published on azure app service

Hi,
I developped a bot with recastAI. On local, everthings work... but when i tried to publish my app on azure app service it fails.
Application has thrown an uncaught exception and is terminated: Error: Cannot find module './internal/applyEach'     at Function.Module._resolveFilename (module.js:455:15)     at Function.Module._load (module.js:403:25)     at Module.require (module.js:483:17)     at require (internal/module.js:20:19)     at Object.<anonymous> (D:\home\site\wwwroot\node_modules\recastai\node_modules\form-data\node_modules\async\applyEach.js:7:18)     at Module._compile (module.js:556:32)     at Object.Module._extensions..js (module.js:565:10)     at Module.load (module.js:473:32)     at tryModuleLoad (module.js:432:12)     at Function.Module._load (module.js:424:3)

Hi !
Can you describe the flow you're using to deploy your bot ?

I used the regular deployment of Azure app service. I think it checks if a package.json exists, and launch npm install and npm start.
I will try to find azure documentation about deployment and paste it here.

This is my package.json :
{   "name": "fnac-bot",   "version": "1.0.0",   "description": "A starter kit for making a bot connected with microsoft bot connector",   "main": "src/bot.js",   "scripts": {     "start": "node src/bot.js"   },   "babel": {     "presets": [       "es2015",       "stage-0"     ]   },   "author": "hcherchi",   "license": "ISC",   "dependencies": {     "@slack/client": "^3.6.0",     "babel-cli": "^6.11.4",     "babel-eslint": "^6.1.2",     "babel-preset-es2015": "^6.13.2",     "babel-preset-stage-0": "^6.5.0",     "babel-register": "^6.11.6",     "lodash.mean": "^4.1.0",     "lodash.meanby": "^4.10.0",     "recastai": "^2.2.1"   },   "devDependencies": {     "azure-storage": "^1.3.1",     "moment": "^2.15.1",     "quiche": "^0.1.0",     "request-promise": "^4.1.1"   },   "engines":{"node": "6.7.0"} }

Azure App Service understands package.json and npm-shrinkwrap.json files and can install modules based on entries in these files.
https://azure.microsoft.com/en-us/documentation/articles/nodejs-use-node-modules-azure-apps/

If you have a npm-shrinkwrap module, can you post it the content here?
Do you still have this issue if you remove the 'engines' fields in your package.json?


Bluetooth: host: Fix CCC cfg not cleared when overwriting oldest bond

Fix CCC cfg not cleared when overwriting oldest bond. Calling
bt_unpair with a pointer to the key will result in the key addr being
memset to zero and bt_gatt_clear is called with an zero-set address.
This happens because unpair (hci_core.c) calls bt_keys_clear before
calling bt_gatt_clear.
Fixes: #23961

CC: @mfiumara

@jhedberg This could have been fixed by re-ordering bt_gatt_clear and bt_keys_clear. I'm wondering if we should do both fixes. Assuming re-ordering has no negative effects.

@joerchan bt_gatt_clear() ends up calling bt_addr_le_is_bonded() when clearing Service Changed information, so I think the order must be kept (would be good to have a code comment though so no-one goes and changes this)

@joerchan bt_gatt_clear() ends up calling bt_addr_le_is_bonded() when clearing Service Changed information, so I think the order must be kept (would be good to have a code comment though so no-one goes and changes this)

That is just removing the entry from the settings, not sure if that is already done implicitly by bt_unpair, it doesn't looks like it does anything related to settings though, it seems it just clear the keys entry so actually doing the bt_gatt_clear before bt_keys_clear should make sure the settings storage is cleanup properly.

@Vudentz It looks like the SC would not be cleared from flash then. The modified check would also affect this.

@Vudentz @joerchan so what's the consensus on this PR? Do we merge it it it needs additional changes?

@Vudentz @joerchan so what's the consensus on this PR? Do we merge it it it needs additional changes?

I would say merge this one. And I'll look if there is a separate problem and submit a PR if that is the case.


Update logo with new SVG

Description

Urgency

[ ] Blocker
[ ] High
[ ] Medium

Requirements

Test all commands and procedures.
Build content locally and test for formatting/links.
Add redirects to dcos-website/redirect-files.
Change all affected versions (e.g. 1.7, 1.8, and 1.9).
See the contribution guidelines.

@judithpatudith when you get a moment could you review and merge?