Code licence

Hello
Thanks for the app.
Is this code available openly for use. Can I use it?

Hello
Thanks for the app.
Is this code available openly for use. Can I use it?

Thank you for your interest in the code. Yes, this code is available openly for use. However, I kindly request that you use it only for personal purposes and not for commercial projects. Your understanding and cooperation are greatly appreciated.

Hi ,
Thanks, but i wanted to use it for commercial use. I don't understand why you don't agree for commercial use. I don't see any difference. Any way, if you don't agree, i will not use it.

You are welcome to use the logic and code from the app for your personal projects, but I kindly request that you do not use the complete design and layout. Please feel free to modify the code and adapt it to your own needs, but avoid using the entire design for any commercial purposes.

Hi ,
Thanks, but i wanted to use it for commercial use. I don't understand why you don't agree for commercial use. I don't see any difference. Any way, if you don't agree, i will not use it.

You are welcome to use the logic and code from the app for your projects, but I kindly request that you do not use the complete design and layout. Please feel free to modify the code and adapt it to your own needs, but avoid using the entire design for any commercial purposes.

Thanks, I will not use the entire design, only the functions with some modifications to suit my needs, I will develop my own UI.


invalid clients can publish will message

Environment

OS: Debian Testing(buster)
Erlang/OTP:  R20/9.0
EMQ:  v2.3-beta.1

Description
We haveing http backend for client authorization.  Client sets the will messages for helping servers detect the clients connection status (e.g. online, offline).
The issue is that the will message (e.g. offline) has been published, even the clients do not pass authorization.

@emqplus
Attention PLEASE!!!!
This will message will not be valided by any auth interface!  It means user will hack on your MQTT Broker, and fire messages on ANY topics they want!!!!!

@srdgame, Thanks for the security report. The issue will be fixed in 2.3-rc.1

Code:
logging.debug('%s\t%s\t%d\t%d', msg.topic, msg.payload.decode('utf-8'), msg.qos, msg.retain)
LOG:
Sat, 30 Sep 2017 14:59:51 mqtt_sub.py[line:55] DEBUG i_am_a_hack	this is hack content, which breaks your auth module	0	0

@srdgame The issue has been fixed in 2.3-rc.1 release. https://github.com/emqtt/emqttd/commit/88c77cf4c2e11e6cb1be27316c353060921b4018#diff-83e84d76b96bcb793b363258fac67e26


⚠ Spryr has degraded performance

In 22e04d2, Spryr (https://spryr.com) experienced degraded performance:

HTTP code: 200
Response time: 6742 ms

Resolved: Spryr performance has improved in 596ce14 after 12 minutes.


isort: how to specify parent folder rather than all children folders in "known-x" rule?

Hi, I'm trying to organise my imports using ruff.
my dj_apps folder has multiple apps like app1, app2, etc.
How can I specify the known-first-party setting to tell  "all folders inside dj_apps"?
I tried:

known-first-party = ["dj_apps"] -> no effect
known-first-party = ["dj_apps/*"] -> no effect

The only thing which currently work is known-first-party = ["app1", "app2"]
This is not very handy as I should remember to edit my Ruff config as soon as I get a new django app.
Thanks!

Unfortunately I don't think there's a way to do this right now, since dj_apps isn't actually part of the module path (I'm guessing?). E.g., when you import these apps, I'm assuming you're doing from app1 import foo rather than from dj_apps.app1 import foo, is that right?


callbacks priorities

Hi,
I am wondering whether it is possible to increase the priority of callback over others callbacks from other modules. E.g. if I want set higher priority for 1 RPC callback over every other callback.
Netopeer2 v1.1.53
Thanks,
Matko

It does not make sense to want to set an RPC callback priority with regard to other RPCs. They are all independent and if possible executed concurrently.


e2e: Enable multiple test servers to run without port conflict

Randomizing the ports configured for a given test server invocation
allows multiple servers to run concurrently without conflict. This is
useful when developing in parallel with multiple branches (e.g. with
git-worktree) and invoking tests with --use-default-server.

Rebased


Feat/noise graph join

 Implement spatial join for traffic noise values to the new graphs

 Interpolate noise values to edges at boundaries between municipalities (fill 10 m gaps in noise surfaces).


Add support for peppermint

Description
It could be interesting to implement widgets for peppermint to have tickets under control directly from the homarr dashboard
Priority
Low (Nice-to-have)

Hello,
Could you tell us more about this app and why it would need to be added as a separate integration that iframes don't already provide?

https://github.com/Peppermint-Lab/peppermint
its an issue tracker, unsure if there are plans to implement something like this into homarr or keep it separate but this is an option.


feat: return self when num === 0

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

Comparison is base (0fd28cc) 95.23% compared to head (27be576) 95.23%.

:exclamation: Current head 27be576 differs from pull request most recent head 440d46b. Consider uploading reports for the commit 440d46b to get more accurate results

Additional details and impacted files
@@           Coverage Diff           @@
##             next     #158   +/-   ##
=======================================
  Coverage   95.23%   95.23%
=======================================
  Files          28       28
  Lines        2561     2561
  Branches      388      389    +1
=======================================
  Hits         2439     2439
  Misses        122      122

Files
Coverage Δ

src/util/index.ts
64.32% <100.00%> (ø)

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


YouTube Downloader

Extension
https://www.raycast.com/vimtor/youtube-downloader
Raycast Version
1.78.1
macOS Version
14.6 Beta
Description
After i copied a yt link to the link field and wanted to start the download process, I received an error and the video will not download.

Steps To Reproduce

Current Behaviour
no video / audio downloading
Expected Behaviour
download

Same issue here.
Thanks for the otherwise awesome extension.


fix: expand tag column on truncated post

Motivation
Just noticed on one of sites added to the showcase (https://cloudywithachanceofbigdata.com/) a small bug related with tags layout.
Current behavior:

But it should be like this:

Have you read the Contributing Guidelines on pull requests?
Yes
Test Plan
Preview (but we have too little tags on our blog to reproduce this issue)
Or see test posts https://deploy-preview-5667--docusaurus-2.netlify.app/tests/blog
Related PRs
(If this PR adds or changes functionality, please take some time to update the docs at https://github.com/facebook/docusaurus, and link to your PR here.)

We can add dogfooding posts for this. Another thing I noticed that can be simultaneously fixed in this PR: https://deploy-preview-5667--docusaurus-2.netlify.app/tests/blog

LGTM thanks!


Chart - Time spent by the users

Implement Time spent by the users chart

https://github.com/openvitae-tech/blackboard-lms/pull/354


Trigger doc update

Trigger build on relevant documentation repo on master build
Closes #201

@silverhook , @talha131 , @AWegnerGitHub  can you review please?

Thanks a lot @iranzo. It's a great addition. I am sure it will save heaps of time.


Initial Pi 2 + Pi Zero support

Adds basic support for Pi 2 and Pi Zero.
Pi Zero also requires: https://github.com/f-secure-foundry/tamago-go/pull/2 (Pi Zero does not support the lsb, isb, dmb instructions, so instead does equivalents using MCR-style instructions.

@abarisani I didn't see any documents on the contribution process to this repo.  Is raising this PR sufficient?

@abarisani I didn't see any documents on the contribution process to this repo. Is raising this PR sufficient?

Yes it is. However we are all on vacation so please have some more patience :). Thanks!

Some initial comments/requests:

we are changing to board/$vendor/$model and soc/$model layout (it's not completed yet because we have to migrate imx6/usbarmory on all linked projects, but we juts added support for mx6ullevk in this fashion). Could you please move paths accordingly (e.g. board/raspberry/pizero, board/raspberry/pi2 and soc/bcm2835) ?

Some style/consistency notes with the rest of our code base, these are not defects or mistakes on your part, but we just like the codebase to be consistent:

bcm2835.HardwareInit() should be named Init() to be consistent with imx6.Init()

please note that

The arm package exports TimerMultiplier and TimerFn intentionally. We either change this to allow rationale for InitSpecificTimer or, my preference, I think this function can be spared and you can just set ARM.TimerMultiplier and ARM.TimerFn when allocating &arm.CPU{} or in bcm2835.Init().

PeripheralBase should not be exported by the package as it is accepted by the initialization function, the initialization function should validate it against common/known ranges if possible (like in this example).

The comment "care must be taken to ensure that no heap allocation is performed (e.g. defer is not possible)" is no longer relevant. Please remove it and defer in a manner consistent with the rest of the package.

Please convert Line: 0x2f in pi/pizero/board.go with use of the proper constants used to derive this value (if any can be applied to describe its meaning).

The purpose of pi/pizero/tls.s is unclear, could you clarify its need?

There other style changes or improvements that I'd like to suggest, but for a first round I think this is probably plenty :). In the meantime thank you so much for this contribution, I look forward to merge it.

Given the properties of Go constants we find that uint32 can be spared from them as it detracts, in my opinion, from "syntactic sugar" and also results in uint32(foo) casting required when performing pointer arithmetics. Given that the type is inferred by function arguments it is our style to avoid this, please see this commit.

bcm2835/rng.go, bcm2835/uart.go should be consistent with imx6/rngb.go, imx6/uart.go, and therefore use an instance. This improves the way documentation is presented to the user.

I wonder if a busyloop can work better than the dummy function in bcm2835/uart.go. Also any reason for simply not using time.Sleep() there ?

I am not sure I understand the usefulness of var GPIO = gpio{} given that the struct holds no state/lock (which btw for consistency with the rest of the codebase should be &gpio{}).

Thanks for the detailed review, I'll work through these.
I didn't really understand the rationale for using instances vs not in the packages, so i somewhat randomly used them vs not.  If i understand the purpose, it's for documentation and/or when there's state to hold?  I added GPIO so that code can do pi.GPIO.Set() - I'll rework to be pi.SetGPIO() style.
I exposed PeripheralBase because code outside of the package that is common to both boards needs access to the value.  It's a fundamental base address for all hardware register access in the Pi's.   I'll expose it as bcm2835.PeripheralAddress(offset) so that code outside of the bcm2835 package can still access it and remain board agnostic.
If I have any questions on the other items I'll get back - but the all seem clear.

Thanks for the detailed review, I'll work through these.
I didn't really understand the rationale for using instances vs not in the packages, so i somewhat randomly used them vs not. If i understand the purpose, it's for documentation and/or when there's state to hold? I added GPIO so that code can do pi.GPIO.Set() - I'll rework to be pi.SetGPIO() style.
I exposed PeripheralBase because code outside of the package that is common to both boards needs access to the value. It's a fundamental base address for all hardware register access in the Pi's. I'll expose it as bcm2835.PeripheralAddress(offset) so that code outside of the bcm2835 package can still access it and remain board agnostic.
If I have any questions on the other items I'll get back - but the all seem clear.

I do like pi.GPIO instances but only when they represent an individual GPIO instance rather than the overall GPIO functionality of the Pi. But honestly it can go both ways, maybe add a mux to reason it ;) (which is probably not a bad idea to prevent concurrent operation) ? Or wouldn't an instance that holds information on the line and its configuration make more sense to match more what we are doing here ? I think it's nice if you can have an instance of a configured gpio that then can be set to in/out high/low and that's it, see how we implement leds here.
It's fine to expose PeripheralAddress, my rationale is that it should be set only at initialization time and if anyone needs to access it then it shall be done through a function so that it's read-only post initialization.
Thanks!

@abarisani - I think I've addressed all comments.  I've rebased due to directory structure change, and done one commit per comment.
Looking again, it might be better to solve pi/pizero/tls.s in tamago-go.  In essence, because PiZero support is compiled with GOARM<7, the Go runtime expects the symbol runtime·read_tls_fallback.  I can think of three approaches:
1 - what we do here, solve this board specific problem in tamago code-base
2 - add the symbol to the tamago-go code-base
3 - change the conditional logic that requires this symbol in tamago-go code-base (https://github.com/golang/go/blob/master/src/cmd/internal/obj/arm/obj5.go#L69)
I think this symbol will be needed on any SOC where GOARM<=6, so a generic solution may be best.

Can you change the PR so that it targets our development branch rather than master, so that we can keep working on development while we refine these changes?
Overall everything looks ok, again I have some further change proposals but we I can do a PR myself for you to review against the development branch.
Concerning runtime·read_tls_fallback I agree this should be handled outside tamago, but rather in tamago-go. To this end you can remove your code related to it as I've just committed its definition in tamago-go1.15.1 (which will be carried on in future releases and branches).

I've moved to dev branch and removed read_tls_fallback symbol.

Merged. Please expect my PR sometime early next week. In the meantime thanks!

@kenbell can you please review https://github.com/f-secure-foundry/tamago/pull/6 ?


Type-level guard taking higher priority than field-level

Hi! I am trying to setup field-level and type-level permissioning, and I'm running to an issue where the type guard seems to have a higher priority than the field. Here is my code:
module Types
  ExampleType = GraphQL::ObjectType.define do
    name "Example"
    description "Example Description"

    implements GraphQL::Relay::Node.interface

    guard ->(obj, _args, context) { false }

    global_id_field :id
    field :weight, types.String
    field :pulse, types.String
    field :height do
      type types.String
      guard ->(obj, args, context) { true }

      resolve lambda {|obj, _args, _context|
        obj.height
      }
    end
  end
end

I'm not able to see any of the fields, including height, even though the guard returns true for height.

Hi @pooja335,
Which version of graphql gem do you use? I can't reproduce it with 1.7.14 or 1.8.5. Just in general here is how the code which detects the priority looks like (fields first, then type):
https://github.com/exAspArk/graphql-guard/blob/c8b379e182256ea4c2435272036ccaab4b57b4a9/lib/graphql/guard.rb#L49-L54

Closing the issue, feel free to reopen if you have more details.


build: fix ci cacheing to use recently built caches

@alexeagle
Github actions caches don't update already existing caches, so the cache that gets loaded could be old. This sets a new cache using a cache key derived from a hash of files. The use of restore-keys: bazel-cache- will find the most recent cache stored scoped to this branch or a parent branch.

Updated. @alexeagle I wrote a more verbose (but hopefully clearer) explanation. @f0rmiga I removed the cache location flags outside of the bazelrc file as it doesn't support environment variables.


TF-TRT - Reorganize testing utilities

The main benefit of this PR is to replace some utility functions with gMock matchers. This allows GTest failures to print more informative information than is possible when using functions with EXPECT_X embedded without littering SCOPED_TRACE(...) in the testing bodies.

Factor out conversion test utilities to testutils library
Rename GetTestDims to CreateDims
Add ostream<< printers for nvinfer1:: Dims and INetworkDef
Add GMock matchers for nvinfer1::Dims and INetworkDef,and
unit tests for matchers
Replace existing util ExpectsX functions with GMock matchers
Avoid using const char* where string types are required
Minor formatting corrections.

Added (8/10):

Moved two std::vector templated utility functions InitTestVector and CastTestVector to trt_testutils.h from convert_nodes_test.cc. I renamed these functions CreateVectorIota and CastVector respectively.

@tfeher

@bixia1 Can you help review? Thanks

Thanks for this PR @christopherbate, it is nice to see some clean up in the test routines! In an offline discussion you have mentioned that "replacing ExpectsX functions with GMock matchers eliminates indirection in the test error result so the test error reports the actual failing line in the test [...] and correctly prints the nvinfer types." Could you update the PR description to include this information?

@bixia1,
Updated based on your comments, and fixed a subtle issue with the TRT_ShapedWeights gMock matcher. Tests look good to go on my end.
I have a follow on PR for additional test refactoring which I'm about to create. I can either create a new PR or append to this one? Thanks.

@bixia1 I applied your asks/recommendations.

Would you please also do these before I will approve the PR:
(1) squash the commits into one commit. This is because we don't want the commit history for this PR to show in the final change, to simplify the commit message for the change.
(2) Would you please revise/simplify your PR description, such as removing "Added (8/10)" etc? This is because the PR description will become part of commit message, and we only want to keep the description about the final code change in the PR.

Will do, thanks

@bixia1 Can you please take a look on above comments from @christopherbate. Thanks!

@gbaned I am still waiting for @christopherbate to address https://github.com/tensorflow/tensorflow/pull/51235#issuecomment-899784046

@bixia1 done!

@christopherbate Can you please address Ubuntu Sanity errors? Thanks!

@gbaned I applied the builifier recommended changes.

Rebased on master

@bixia1 @gbaned Please merge these queued dynamic shape support PRs first, then I will rebase this if required:
https://github.com/tensorflow/tensorflow/pull/51468
https://github.com/tensorflow/tensorflow/pull/51462

@bixia1 I added the suggested change for the empty dims DimsAreArray({})


It is possible to use 4 touch zones on 1/2/3 gang relay versions

As described (in the video I think), the touch sensor seems to have 10 zones, so technically up to 10 programmable touch fields could be used.
So I assume that one can also use 4 touch fields on a switch that only has one relay.
New face plates can be bought on itead, so one can get a black star or the animal 4 touch fields plate to replace.
I just realized this and wanted to share it. I'll test it some time if face plate switching works (I have a 3 relay one, and a few 4 relay plates).
But could be super useful for folks that want more touch (maybe building their own faceplace)

Imo you'd have to tweak the code a bit so it doesn't try to switch on/off the relays but theoretically speaking you can have up to 10 zones.
https://github.com/SmartHome-yourself/sonoff-tx-ultimate-for-esphome/blob/1723392ac7a1cf0e4ad540aa97585974668abbed/tx_ultimate_local.yaml#L799
Extending this part would be a good start because you can just reuse the 3 size and add new part with 10 where you'd put 10 "ifs" to handle each touch field.
If you are happy with 4 then you can just use code of others who have it but I don't know it so in detail to tell if it wouldn't look for the 4th L relay.

I use this one for my 4 relais:
https://github.com/SmartHome-yourself/sonoff-tx-ultimate-for-esphome/pull/55
You can set those variables:
toggle_relay_1_on_touch: "true"
toggle_relay_2_on_touch: "true"
toggle_relay_3_on_touch: "true"
toggle_relay_4_on_touch: "true" <== this needs to be false
And then maybbe some switch should be cleaned up.


ModuleNotFoundError: No module named 'utils'

when I run "import libs" box
i keep getting this error
ModuleNotFoundError: No module named 'utils'

someone else opened same issue just before me


feat: Slider onValueCommited event

Describe the feature in detail (code, mocks, or screenshots encouraged)
Issue comes from shadcn-svelte:
https://github.com/huntabyte/shadcn-svelte/issues/1325
Looking into this it looks like melt-ui is outdated and doesn't export onValueCommited would need to go from 0.76.2 to at least 0.80.0.
Then we could add the props to the component.
I guess the real question is if this should be taken care of now or just moved to @next let me know and I can open a PR!
What type of pull request would this be?
New Feature
Provide relevant links or additional information.
No response

Added to @next.


Add flake8-bugbear rule B028

Part of #2954.
Tests are from bugbear: https://github.com/PyCQA/flake8-bugbear/blob/main/tests/b028.py

Sorry for the delay, will merge this tomorrow!

Thank you, great PR :)


Make platforms optional during build.

we should add args to the C# build project to skip platforms we don't care about, specifically, the Android build will fail if the version of Unity you have installed is missing the Android module.

No news, but we welcome a pull request if you have the capacity to implement it yourself.


Pass Date header in PUT request to override file mtime

Please double-check that Date is the correct header to use here. I see that Date is listed as a forbidden header.
This feature makes it easy for hyperdrive.el to mirror a directory of files, skipping files which have not been updated since the last upload.

How about we use Last-Modified instead? It's more directly related to caching.

Released new version but changed it to use the Last-Modified header instead


Controlling Detox Relaunch behaviours in between running tests

Description
I am facing issues in controlling detox behaviour in between running test cases.
For Ex:-
I have role-based login implemented, to test these from single test case file (login.test.jsx) I want to relaunch detox when jumping from 1 test case to another but I don't want this for all the test cases present in the file so can't use beforeEach.
I am currently using device.launchApp() after finishing the required test.
Next Issue is I want my next file (dashboard.test.jsx) should have contained the app state and does not require login again, but after finishing each test case file, detox autmatically reinstalls the app and does not allow to maintain app state.
I have already done a lot of research on this to implement test case using --reuse, device.reloadReactNative, etc...
If possible please create simpler indi-direct methods to controll detox behaviours in more dynamic way.
Thanks
Your environment
Detox version: v20.25.6
React Native version: v0.70.3
Node version: v20.17.0
Device model: Pixel 5
OS: Android 14
Test-runner (select one): jest

Also if possible please run react-native bundler by default when running detox test's in debug as its required and close as soon as test's are finished.

Could you explain this with an example, please? Some dummy tests with your comments would be helpful to understand your situation better.


Is it possible to Extend the model class ?

Please suggest whether it is possible to extend the model class.
i.e. for humour
@TableModelSpec(className = "Organization", tableName = "organizations")
public class OrganizationSpec {
}

Sorry, I'm not sure if I understand what you're asking. The code you posted will generate:
public class Organization extends TableModel {
    ...
}

Do you then want to create a subclass of Organization? What specifically are you hoping to accomplish? You can of course subclass the generated model classes but needing to do so is an uncommon use case -- there may be a better approach depending on what you're actually trying to do.

Thanks. While reading the data (json formatted) from the server using retrofit, I am converting the json into objects for which I have simple POJOs which are quite similar to Models required for Squidb. As you can imagine then I have to convert them in to Squidb models to persist them in the local db.
Now, any change I bring in I have to modify both the classes.
I was thinking of deriving one from another ... that will reduce the potential errors
For e.g.
public class OrganizationResponse extends Organizations {
}
OR
public class OrganizationSpec  extends OrganizationResponse {
}

I see. There are several considerations here:
The first thing to note is that squidb model specs are not really meant to be used as POJOs -- they're just defined that way as a shorthand for a table schema. They're processed by the code generator and translated into entirely different objects, and these generated classes are the ones you actually work with when reading/writing the database. No matter what, you will need to define some kind of logic for mapping your JSON to the generated squidb models in order to persist them.
That being said, there are options, and you can pick whatever approach seems best to you:

Define some logic to map the JSON directly into your squidb models, bypassing the intermediate POJO representation. This could perhaps even be automated with some kind of code generation plugin, although that would take some extra work to make it work with all possible JSON -> squidb model mappings.
Make your POJO object for the JSON extend from the model spec class. This doesn't help with the mapping problem, but at least it guarantees if you modify the model spec, the same field will be added to the JSON POJO, if that's what you want.
Make your model spec extend from the POJO object. This might be the most difficult of the options, and also will not solve the mapping problem; again it would only guarantee that any field added to one class will be included with the other. This option is more difficult because squidb only process the fields in the model spec class by default (not fields in any of its superclasses), so you would need to write a code generator plugin to handle this as well.
Make your POJO class and your model spec class one and the same. This is kind of a weird option, as I imagine fields in your JSON may not correspond directly to database columns in all cases. You'd have to keep in mind that in squidb model specs, field <--> column, and potentially annotate fields that didn't correspond to columns with @Ignore, and do something similar for any database columns that didn't correspond to a JSON field. Again though, you would still need to define logic for mapping your POJO to the generated squidb model.

Personally I would probably go with option 1 much of the time, but I don't know all the details of your use case and you may very well have different needs (e.g., maybe you're using an API that already defines those POJOs and precludes that option).
For information on code generation plugins, see this wiki page.
If you have more questions or want to provide any more specific details, happy to continue helping or offering advice! I hope that gives you some useful things to think about though.

I'm going to close this issue as there hasn't been any activity on it in a while, but feel free to reopen if you have any more specific questions.


Time API

Hello,
does this library support time API as in Python?
For example, are there similar constructions to get time like rospy.get_time()?
There is no problem with reality where we can use system time.
But simulation time is different from the latter, therefore I wonder how could we get it.

Hello, simulated clocks are currently supported but they may be in the future.

A Time API, which supports simulated clocks, has been added.
You can now use
node.TimeNow()
node.TimeSleep(duration)
node.TimeRate(duration)

Thank you for very quick feature implementation!

At the moment you have to set NodeConf's UseSimTime to true to switch to a simulated clock, but this option will be removed soon and clock seleciton will be automatic.


Updated PostCSS, and added warning when variable cannot be resolved along with option to turn warning on or off ('warnOfUnresolved'). Upped version. Etc.

Updated postcss dependency to 6
Made required updates due to upgrade to version 6
Added the new postcss 'results' parameter so I can add warnings (via results.warn(..))
Added option 'warnOfUnresolved' and defaulted to true.
Documented new option.
Updated the version to 2.0.0. since the update of postcss dependency
Updated eslint, and postcss-scss to newest versions
Added "postcss-reporter" as dev dependency so we can see postcss warnings when runing tests. Helpful when resolving some bugs uncovered when I started reporting warnings that were not obvious before, and also helpful when resolving issues resulting from upgrading postcss
Added and documented support for variables being a function instead of an object

Hi @jonathantneal
Do you believe you will have time to merge this change before end of year? I'm planning some changes to my team's use of PostCSS and knowing timing would be helpful.
Thanks!

Seems a little complex. Transparent result through the functions calls. Make this PR looks not  relieved.

There is a lot going on in this PR. My thoughts are:

we need more maintainers for this project.
we need a maintainer for precss.

Actually, I stopped maintaining precss almost immediately after it was released, going as far as to not follow the project. I am just now seeing how many issues it has and how popular it remains despite that — which makes no sense to me.
This is a lot to take in.

@deemstone Yes, passing result around is not my preferred solution either, but other solutions would require a more significant rewrite, and this is what works with the current coding -- and the current coding is good and I see no reason to do a larger rewrite.
@jonathantneal  I agree. It is a lot. There are a few extras but 95% of it was mandatory to accomplish adding warnings via postcss 6's new API, or were directly the results of upgrading to postcss 6. The other 5% are things myself and my team really wanted to see (e.g. warnings on unresolved variables). With this larger commit merged in future PRs can be much smaller to add some new features and fixes. But this larger PR seems unavoidable to get postcss-advanced-variables upgrade to postcss 6.
How can I help get this merged and help maintain postcss-advanced-variables? I don't have the time to help with precss, but I can help with this postcss-advanced-variables if you want.

Thanks @jonathantneal . Now that it is merged in, can it be published to NPM?


WMS ID 4862 Quarterly QA

WMS ID: 4862
I've corrected the images and markdown files so they are:

descriptive
lowercase

This is consistent within the individual markdown files as well.
Thanks!
Chris

Closing had a few more changes I just caught. I've made those. Will open a new PR now.

See above


Explain the makeup of a redcode instruction in the wiki

We need a detailed explanation / spec of the makeup of a redcode instruction.
The Redcode address modes and modifiers should be explained with enough detail to enable implementors to basically only reference the wiki.
NOTE:
The explanations / spec of all of the instructions themselves are not part of this story.
Completion criteria:

[ ] The Redcode address modes have been explained in the wiki
[ ] The Redcode modifiers have been explained in the wiki
[ ] Examples have been given, if applicable

@flamion Take a look please


autorelease is failing

HTTPError('403 Client Error: Forbidden for url: https://api.github.com/repos/GoogleCloudPlatform/cloud-sql-go-connector/issues/167/labels')

Please triage

All is green now.


Unify the matchAll options

The first result of matchAll is case sensitive, but the rest are case insensitive. This pull request fixed this.
And added two methods to specify compareOptions.

Great work :+1:

才发现原来你是个中国人，哈哈

然而在这里用中文并不好..已经解决的问题可能要用英文再解释一遍..你懂得

兹词楼上

= =, 我竟然反应了一下才看懂..


Green edge incorrectly (?) deemed as yellow

.git folder: git.zip
CLI v2.14.0 output:

Plugin v0.3.0-SNAPSHOT+git.cec61d0 output:

Likely more remote branches are taken into account in the plugin:

plugin: all remote branches
CLI: just remote tracking branches of the local branches

(let's first check if a fork point override was set up for develop, but this time I think not xD)

Meh, ok, fork-point override again, ignoring the issue ;D


Add Technical Secondary School no 5 in Częstochowa

Hello, i'm student in Technical Secondary School nr 5. I'd like to add this domain as majority of students (around 60% or 70%) are in IT classes.
Here is website link: Click!

@DesantBucie Pull request merged. Thank you!


[No QA] Pass required secrets to setupNode

Details
Fixing https://expensify.slack.com/archives/C03V9A4TB/p1692658836630509
Fixed Issues
$ n/a
Tests
Merge + CP to test.

[x] Verify that no errors appear in the JS console

Offline tests
None.
QA Steps
None.

[x] Verify that no errors appear in the JS console

PR Author Checklist

[x] I linked the correct issue in the ### Fixed Issues section above
[x] I wrote clear testing steps that cover the changes made in this PR

[x] I added steps for local testing in the Tests section
[x] I added steps for the expected offline behavior in the Offline steps section
[x] I added steps for Staging and/or Production testing in the QA steps section
[x] I added steps to cover failure scenarios (i.e. verify an input displays the correct error message if the entered data is not correct)
[x] I turned off my network connection and tested it while offline to ensure it matches the expected behavior (i.e. verify the default avatar icon is displayed if app is offline)
[x] I tested this PR with a High Traffic account against the staging or production API to ensure there are no regressions (e.g. long loading states that impact usability).

[x] I included screenshots or videos for tests on all platforms
[x] I ran the tests on all platforms & verified they passed on:

[x] Android / native
[x] Android / Chrome
[x] iOS / native
[x] iOS / Safari
[x] MacOS / Chrome / Safari
[x] MacOS / Desktop

[x] I verified there are no console errors (if there's a console error not related to the PR, report it or open an issue for it to be fixed)
[x] I followed proper code patterns (see Reviewing the code)

[x] I verified that any callback methods that were added or modified are named for what the method does and never what callback they handle (i.e. toggleReport and not onIconClick)
[x] I verified that the left part of a conditional rendering a React component is a boolean and NOT a string, e.g. myBool && <MyComponent />.
[x] I verified that comments were added to code that is not self explanatory
[x] I verified that any new or modified comments were clear, correct English, and explained "why" the code was doing something instead of only explaining "what" the code was doing.
[x] I verified any copy / text shown in the product is localized by adding it to src/languages/* files and using the translation method

[x] If any non-english text was added/modified, I verified the translation was requested/reviewed in #expensify-open-source and it was approved by an internal Expensify engineer. Link to Slack message:

[x] I verified all numbers, amounts, dates and phone numbers shown in the product are using the localization methods
[x] I verified any copy / text that was added to the app is grammatically correct in English. It adheres to proper capitalization guidelines (note: only the first word of header/labels should be capitalized), and is approved by marketing by adding the Waiting for Copy label for a copy review on the original GH to get the correct copy.
[x] I verified proper file naming conventions were followed for any new files or renamed files. All non-platform specific files are named after what they export and are not named "index.js". All platform-specific files are named for the platform the code supports as outlined in the README.
[x] I verified the JSDocs style guidelines (in STYLE.md) were followed

[x] If a new code pattern is added I verified it was agreed to be used by multiple Expensify engineers
[x] I followed the guidelines as stated in the Review Guidelines
[x] I tested other components that can be impacted by my changes (i.e. if the PR modifies a shared library or component like Avatar, I verified the components using Avatar are working as expected)
[x] I verified all code is DRY (the PR doesn't include any logic written more than once, with the exception of tests)
[x] I verified any variables that can be defined as constants (ie. in CONST.js or at the top of the file that uses the constant) are defined as such
[x] I verified that if a function's arguments changed that all usages have also been updated correctly
[x] If a new component is created I verified that:

[x] A similar component doesn't exist in the codebase
[x] All props are defined accurately and each prop has a /** comment above it */
[x] The file is named correctly
[x] The component has a clear name that is non-ambiguous and the purpose of the component can be inferred from the name alone
[x] The only data being stored in the state is data necessary for rendering and nothing else
[x] If we are not using the full Onyx data that we loaded, I've added the proper selector in order to ensure the component only re-renders when the data it is using changes
[x] For Class Components, any internal methods passed to components event handlers are bound to this properly so there are no scoping issues (i.e. for onClick={this.submit} the method this.submit should be bound to this in the constructor)
[x] Any internal methods bound to this are necessary to be bound (i.e. avoid this.submit = this.submit.bind(this); if this.submit is never passed to a component event handler like onClick)
[x] All JSX used for rendering exists in the render method
[x] The component has the minimum amount of code necessary for its purpose, and it is broken down into smaller components in order to separate concerns and functions

[x] If any new file was added I verified that:

[x] The file has a description of what it does and/or why is needed at the top of the file if the code is not self explanatory

[x] If a new CSS style is added I verified that:

[x] A similar style doesn't already exist
[x] The style can't be created with an existing StyleUtils function (i.e. StyleUtils.getBackgroundAndBorderStyle(themeColors.componentBG))

[x] If the PR modifies code that runs when editing or sending messages, I tested and verified there is no unexpected behavior for all supported markdown - URLs, single line code, code blocks, quotes, headings, bold, strikethrough, and italic.
[x] If the PR modifies a generic component, I tested and verified that those changes do not break usages of that component in the rest of the App (i.e. if a shared library or component like Avatar is modified, I verified that Avatar is working as expected in all cases)
[x] If the PR modifies a component related to any of the existing Storybook stories, I tested and verified all stories for that component are still working as expected.
[x] If the PR modifies a component or page that can be accessed by a direct deeplink, I verified that the code functions as expected when the deeplink is used - from a logged in and logged out account.
[x] If a new page is added, I verified it's using the ScrollView component to make it scrollable when more elements are added to the page.
[x] If the main branch was merged into this PR after a review, I tested again and verified the outcome was still expected according to the Test steps.
[x] I have checked off every checkbox in the PR author checklist, including those that don't apply to this PR.

Screenshots/Videos

Web

Mobile Web - Chrome

Mobile Web - Safari

Desktop

iOS

Android

All workflows using setupNode will fail until we merge this.

I don't think the token needs to be configured for all senarios that do call setupNode composite action

Agree, but by making it required you can never forget to pass it. That's my thinking anyways.

AdHoc builds will not pass until after this is merged b/c they use the version of setupNode from main. So we have to live-test this

AdHoc builds will not pass until after this is merged b/c they use the version of setupNode from main. So we have to live-test this

sounds good thank you for taking care of this 🙇

See comments above why this was an emergency – we broke stuff and all tests dependent upon Node.js or any dependency or devDependency from our package.json would continue to fail until after this is merged.

🚀 Deployed to staging by https://github.com/roryabraham in version: 1.3.56-2 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅

🚀 Deployed to production by https://github.com/roryabraham in version: 1.3.56-24 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
failure ❌

🕸 web 🕸
success ✅

🚀 Deployed to staging by https://github.com/roryabraham in version: 1.3.58-0 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅

🚀 Deployed to production by https://github.com/luacmartins in version: 1.3.58-5 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅

Removing the emergency label. The same instance of this


remove skip from db tests

Checklist:

[x] I have updated the necessary documentation
[x] I have signed off all my commits as required by DCO
[x] My build is green

closes #88

No changeset added.  Only tests are affected.

fantastic - thanks for following up on this @wss-alubic!


Changes for v3

Here are a few things I'm thinking about implementing for V3 of this boilerplate.

Replace JSONDoc with API Blueprint + Aglio for documentation
Use Dredd for Documentation Validation / testing
Docker

Unfortunately I'm not a big fan of Typescript or Angular2 so that's not really one of my goals atm.
It shouldn't be hard to convert it over to typescript anyway. Fork the repo and add the transforms


AP Gateway

Message - Expected AP gateway = <IP_ADDRESS> or <IP_ADDRESS> but got <IP_ADDRESS>
no connection and cycle through until failed.
Is there a config issue?

As we already asked in discord, and you ignored and made two github issues instead...
Are you sure it is a Beken or Tuya device?
https://github.com/tuya-cloudcutter/tuya-cloudcutter/wiki/FAQ#im-not-sure-what-module-is-in-my-device-how-do-i-find-out
Tuya devices are usually not <IP_ADDRESS>, and if they are, they are not beken devices.  Does this device already have a 3rd party firmware on it?

Hello, thanks for the reply and please do accept my apology, I did not see and ignore the discord response.
Confirming the BSSID lookup shows “No vendor details found.”
Thanks for your support.
From: Cossid @.>
Sent: Wednesday, October 11, 2023 8:40 AM
To: tuya-cloudcutter/tuya-cloudcutter @.>
Cc: dreleith @.>; Author @.>
Subject: Re: [tuya-cloudcutter/tuya-cloudcutter] AP Gateway (Issue #500)
As we already asked in discord, and you ignored and made two github issues instead...
Are you sure it is a Beken or Tuya device?
https://github.com/tuya-cloudcutter/tuya-cloudcutter/wiki/FAQ#im-not-sure-what-module-is-in-my-device-how-do-i-find-out
Tuya devices are usually not <IP_ADDRESS>, and if they are, they are not beken devices. Does this device already have a 3rd party firmware on it?
—
Reply to this email directly, view it on GitHub https://github.com/tuya-cloudcutter/tuya-cloudcutter/issues/500#issuecomment-1757598845 , or unsubscribe https://github.com/notifications/unsubscribe-auth/ARCL57HSJMLGV6CCSI4EOJDX62HRZANCNFSM6AAAAAA533CLJY .
You are receiving this because you authored the thread.  https://github.com/notifications/beacon/ARCL57GGDAZQMTOZCI6R4XDX62HRZA5CNFSM6AAAAAA533CLJ2WGG33NNVSW45C7OR4XAZNMJFZXG5LFINXW23LFNZ2KUY3PNVWWK3TUL5UWJTTIYLKH2.gif Message ID: @.*** @.***> >

What is the bssid?  That is almost certainly an Espressif ESP chip.  If you subtract hex 2 from the first character set, it will probably match Espressif.  That means the device is not compatible with CloudCutter.

de:4f:22:22:d9:df
ee:fa:bc:9e:ae:43

dc:4f:22:22:d9:df is Espressif
ec:fa:bc:9e:ae:43 is Espressif
Both of those are ESP devices, and not compatible with Tuya-CloudCutter.  You may consider trying Tuya-Convert, and failing that, you would need to serial flash.


Break out Line implementation for reuse

Pull out the wide line implementation for reuse by other layers.  In general we want to be able to do this for symbol, line, fill and circle.  So think about how it could work more generally.
But we definitely need line for the fill layer.

Duplicate.


How to do ensemble learning?

Since the code does not provide an ensemble learning option, it seems that I need to implement it by myself. But it is very hard to do so because I can't access to the detailed decoding process. So how can I efficiently do this?

BTW, it seems that AttentionWrapper does not support variables sharing.
With AttentionWrapper, it raises following error messages when I set reuse=True in all BasicLSTMCell:
ValueError: Variable dynamic_seq2seq/decoder/attention/luong_attention/attention_g already exists, disallowed. Did you mean to set reuse=True in VarScope? Originally defined at:

Without AttentionWrapper but only BasicLSTMCell and MultiRNNCell it just works fine.

Hi @lyy1994
For the error:
The Variable is created in the AttentionMechanism, but I think there is no way to pass in a reuse setting to the AttentionMechanism or AttentionWrapper, a work around is setting reuse=True in higher level reuse variable scope, such as, the "decoder".
For ensemble:
I assume you want to average multiple models' outputs during decoding process.  You might be able to do it by implement a custom decoder class. You can read the BasicDecoder's implementation, especially the initialize() and step() methods, to have an idea how the class works during the decoding process. I would imagine you want to take multiple cell, hidden_state and output_layer in your custom decoder, and averaging the outputs at each timestep. I haven't done this myself, but hope this maybe helpful.

Thanks!
For the error:
I try to set reuse=True in the outer variable scope, but still receive the same error. Not sure where the problem is.
For ensemble:
Yeah! I want to average multiple models' outputs during decoding process! I have already implement a new decoder class, and it works fine on Seq2Seq model (without attention) with some cost in speed.

Hi @lyy1994
can you share the ensemble code, or give some advice to implement it.
Thanks!

@lyy1994
And for the attentionwrapper variable sharing,I have met it before.
you can add a with tf.variable_scope('att') in front of the decoder and attention code, and you can share the variable between train and generate stage.
Best!


Add --no-prefix global flag to remove command string from spawned output

I've created a globalOptions singleton to store global flags to prevent passing args down everywhere.
I considered adding useBasename as an option as well but didn't think it would be useful enough for the end consumer to add as a flag. We only use it when executing yarn commands.
I also made some other yarn commands print basename and silenced the output of the user agent call we make.

LGTM

Released in 0.24.5


Add spacing tokens mdx story

Description of Change

Adds a Spacing tokens storybook page with a table of tokens and visual example, similar to our color tokens page and VADS's docs.
Testing Packages

N/A
Screenshots/Video

Testing

[x] Tested on Web

PR Checklist
Code reviewer validation:

General

[x] PR is linked to ticket(s)
[x] PR has changelog label applied if it's to be included in the changelog
[x] Acceptance criteria:

All satisfied or
Documented reason for not being performed or
Split to separate ticket and ticket is linked by relevant AC(s)

[x] Above PR sections adequately filled out
[x] If any breaking changes, in accordance with the pre-1.0.0 versioning guidelines: a CU ticket has been created for the VA Mobile App detailing necessary adjustments with the package version that will be published by this ticket

Code

[x] Tests are included if appropriate (or split to separate ticket)
[x] New functions have proper TSDoc annotations

Publish

If changes warrant a new version per the versioning guidelines and the PR is approved and ready to merge:

[ ] Merge main into branch
[ ] Merge branch to main
[ ] Verify that Check Component Integrations workflow ran successfully
[ ] Publish new version after merging

Updated text
Left aligned table column headings
Changed table to 100% width

@TimRoe @jessicawoodin Please take a look at the updated screenshot and let me know if there's anything else you'd like to see


Simplify mappings

At the moment there are different mappings for different loading mechanisms:

IGC REST API mappings
istool mappings
IA REST API mappings

Would be better if there were just one consistent way of doing mappings, and we transparently translate that in the background to the form that we might require for a particular scenario (otherwise user needs to configure the same mapping multiple times across each area).

... or simply recommend an approach which means defining key attributes to be mapped in a separate vars file, and leave it to the playbook itself to determine how to apply those mappings?  (That way users that define the target values for mapping don't need to be aware of / worry about the source values?)


fix(QB-18396): update table - iOS

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


feat: Adds Area chart migration logic

SUMMARY
This PR adds the Area chart migration logic (NVD3 ➡ ECharts). Users can execute this migration using the https://github.com/apache/superset/pull/25304 CLI command and disable the legacy version with the VIZ_TYPE_DENYLIST configuration.
@sadpandajoe @jinghua-qa
BEFORE/AFTER SCREENSHOTS OR ANIMATED GIF

TESTING INSTRUCTIONS
1 - Upgrade a Area chart using the CLI command
2 - Check the new chart
3 - Downgrade a Area chart using the CLI command
4 - Check the legacy chart
ADDITIONAL INFORMATION

[ ] Has associated issue:
[ ] Required feature flags:
[ ] Changes UI
[ ] Includes DB Migration (follow approval process in SIP-59)

[ ] Migration is atomic, supports rollback & is backwards-compatible
[ ] Confirm DB migration upgrade and downgrade tested
[ ] Runtime estimates and downtime expectations provided

[ ] Introduces new feature or API
[ ] Removes existing feature or API

I'm a tad complexed, given that the PR title states we're adding an area chart migration yet there was an existing "MigrateAreaChart" class.

Even though we had a previous migration, it was incomplete and wasn't being used.

Given there's also a refactor in combination with the are chart migration it might be helpful to annotate (via comments) which parts represent new logic.

Added comments


Some icons do not respect the "color" prop

Some icons do not respect the color prop and use their own colors instead:

CrosshairsIcon
All Payment Icons
All Service type icons

I did a PR in regards to this at https://github.com/freenowtech/wave/pull/76 but I need input in regards to if I can split the folders on "assets/icon" or this needs some extra communication.


Why protractor still use old Jasmine (2.8.0)

Why protractor 7 still use old version Jasmime (2.8.0 - almost 3 years)?
Actual Jasmine version is: 3.5.0...

Maybe @kyliau can give more information on the current plans for protractor considering he has authored the latest commits.

Any chance we can get a response on this one? I'm actually running into an issue with my custom reporter. I need it to be async but async reporter support wasn't added until jasmine 3.x. Protractor is exiting before my reporter is done executing because it thinks the run is over. This is causing some of my results not to be reported properly. I found a workaround/hack that appears to be working but I would prefer not to take that route as it adds more complexity than is necessary. I tried manually updating all the jasmine things in my local copy of protractor but that presented a whole list of other problems.
Any work being done in this area right now? Anything at all you can share?

In addition to that I would like to add that there is a Regular Expression Denial of Service (ReDoS) vulnerability in jasmine 2.8.0 that can be fixed by upgrading to 3.5.0.

Any chance we can get a response on this one? I'm actually running into an issue with my custom reporter. I need it to be async but async reporter support wasn't added until jasmine 3.x. Protractor is exiting before my reporter is done executing because it thinks the run is over. This is causing some of my results not to be reported properly. I found a workaround/hack that appears to be working but I would prefer not to take that route as it adds more complexity than is necessary. I tried manually updating all the jasmine things in my local copy of protractor but that presented a whole list of other problems.
Any work being done in this area right now? Anything at all you can share?

@amilbeck I'm also on the same boat waiting for the Protractor team to update the Jasmine to the latest. Unfortunately, we have no update from them 😞

My apologies for the late response. We are actually planning to remove Jasmine entirely from Protractor and remove control flow (all call sites should use ES2017 async/await). This will be a breaking change, but it's the only solution we see that could help us move forward to a more modern codebase.

@kyliau thanks for the update. Do you have any timelines for the next release? Maybe a road map will help the community. We are left in dark and we are not sure whether to rely on Protractor for our testing needs.

We are currently working out the scope of the change. Since it's big, we need to make sure all users including internal users in Google are prepared for the breaking change. Once we have a more solid plan in place, we'll share the roadmap with the broader community.
With regards to your concerns about Protractor, we are definitely committed to maintaining and improving it.

Hey @kyliau, I'm the author of Serenity/JS, an acceptance testing framework that uses Protractor to interact with web interfaces.

We are actually planning to remove Jasmine entirely from Protractor and remove control flow (all call sites should use ES2017 async/await). This will be a breaking change, but it's the only solution we see that could help us move forward to a more modern codebase.

This sounds great! If you're open to collaboration and exploring how other frameworks such as Serenity/JS use Protractor's APIs I'd love to help out.


Jesse: Watch e[\W_]+way[\W_]+scooters?

Jesse requests the watch of the watch_keyword e[\W_]*+way[\W_]*+scooters?. See the MS search here and the Stack Exchange search in text, in URLs, and in code.

e[\W_]*+way[\W_]*+scooters? has been seen in 0 true positives, 0 false positives, and 0 NAAs.

Approved by Spevacus in Charcoal HQ


Perfomance penalty for Nginx Ingress Controller vs Nginx

In the same enviornments, one host exposes service with Nginx and another one exposes with Nginx Ingress Controller. I used tool Vegeta. 30 concurrent connections and 60 seconds setting are used in the testing. The results are below:
Nginx: [https://myhost1.com]
Bucket           #     %       Histogram
[0s,     5ms]    2537  84.57%  ###############################################################
[5ms,    10ms]   428   14.27%  ##########
[10ms,   15ms]   20    0.67%
[15ms,   20ms]   5     0.17%
[20ms,   25ms]   3     0.10%
[25ms,   30ms]   1     0.03%
[30ms,   50ms]   2     0.07%
[50ms,   80ms]   1     0.03%
[80ms,   100ms]  1     0.03%
[100ms,  150ms]  2     0.07%
[150ms,  300ms]  0     0.00%
[300ms,  +Inf]   0     0.00%
Ingress: [https://myhost2.com]
Bucket           #     %       Histogram
[0s,     5ms]    2524  84.13%  ###############################################################
[5ms,    10ms]   338   11.27%  ########
[10ms,   15ms]   33    1.10%
[15ms,   20ms]   12    0.40%
[20ms,   25ms]   28    0.93%
[25ms,   30ms]   43    1.43%   #
[30ms,   50ms]   7     0.23%
[50ms,   80ms]   6     0.20%
[80ms,   100ms]  0     0.00%
[100ms,  150ms]  0     0.00%
[150ms,  300ms]  9     0.30%
[300ms,  +Inf]   0     0.00%
I checked code of Nginx Ingress controller and found the controller itsself takes control of incomming connections and then forwards the content to Nginx server. Why should it be designed so?

Why should it be designed so?

To allow ssl passthrough, i.e. TLS termination in the pod behind the service.

@aledbf , But for Niginx Controller implimentation, even for SSL passthrough, Nginx itsself supports this feature. The current situation is Remote client --> Controller --> Nginx --> service pod. It seems no need to have Controller here. Did I make a misunderstanding on SSL passthrough for ingress?
BTW, SSL passthrough seems not working in the master code. I followed the instruction in #1854 # but I failed to get the correct Nginx configuration in the Controller.

Agree with @jerryjxj
At least it looks redundant, if you don't need SSL passthrough. There is no configuration to connect to Nginx directly.
Also Ingress Controller seems to be not optimized for high load. So on our system sometimes Controller stops responding on HTTPS request (direct request to Nginx on 80 port works good) even on 10-20 rps (maybe this is not related to the number of requests).


vulkaninfo fails while vkcube works

vulkaninfo fails with the following error...
$ vulkaninfo
ERROR at /home/spundun/svntogit-packages-mesa/trunk/src/Vulkan-Tools-1.3.216/vulkaninfo/vulkaninfo.h:231:vkGetPhysicalDeviceSurfaceFormats2KHR failed with ERROR_SURFACE_LOST_KHR

vkcube succeeds with the following output
$ vkcube
Selected GPU 0: Virtio-GPU Venus (Intel(R) HD Graphics 615 (KBL GT2)), type: IntegratedGpu

I've installed this package on arch-linux running in a virtual machine on a chrome-os laptop. Vulkan is supposed to work through Virtio vulkan driver.
I've installed latest upstream releases of mesa and vulkan-headers and vulkan info by patching the PKGINFO for all those packages locally

You can find my patch to update those packages to the latest version(22.1.1 with virtio drivers for mesa, and 1.3.216 for vulkan-headers/tools) in my gist here: https://gist.github.com/spundun/0fefb599352ef0082883512ce9a4c5bd
I mostly followed this guide to install Arch Linux and Vulkan on Crostini : https://github.com/judjdigj/Vulkan_in_Crostini

Notice that the author of this guide also faced the same exact problem with vulkaninfo: https://github.com/judjdigj/Vulkan_in_Crostini/blob/main/README.md?plain=1#L139-L147
That guide is nearly the same as https://wiki.archlinux.org/title/Chrome_OS_devices/Crostini
I first used mesa-git but then decided to patch the stable package instead to improve the packaging for everyone.

Is there any additional information you can provide about the system it is running on? I see that its erroring on vkGetPhysicalDeviceSurfaceFormats2KHR, however because it is passing through the virtio driver theres plenty of other things that could be wrong. Since the error is ERROR_SURFACE_LOST_KHR, it originates from the driver. Whether vulkaninfo is correctly using the function is another matter, though so far I have no reason to believe vulkaninfo is incorrect here. Running vulkaninfo with validation layers (you can use vkconfig to enable them globally) would be a good thing to do
vkGetPhysicalDeviceSurfaceFormats2KHR requires that the VK_KHR_get_surface_capabilities2 extension is supported. Since that is an instance extension, this issue could be caused by 1 driver supporting that extension, while the driver you are using does not. Thus, it errors when it is called.
The vulkan-loader 'emulates' the call if the function isn't supported, source here. It is conceivable that the 'emulation' code is wrong, and causing the issue, if indeed emulation is happening.
Can you run vulkaninfo with the env-var VK_LOADER_DEBUG=all set? And attach the output here. That way the loader knows to report all messages.

Thanks for the response! Ack, on there being other moving parts(I'm no expert but it seems things in the way could be virtio venus driver, sommelier wayland server, VirGL rendere, does vulkaninfo use X11? then also XWayland, I guess?).
Here's the output you requested.
https://gist.github.com/spundun/1010984b5e56100de9c4c19ea09e27a7

After I patched vulkaninfo with #663, it only fails on one extension(VK_KHR_wayland_surface) and is able to output the rest of the information.
ERROR while creating surface for extension VK_KHR_wayland_surface : /home/spundun/svntogit-packages/trunk/src/Vulkan-Tools-1.3.217/vulkaninfo/vulkaninfo.h:231:vkGetPhysicalDeviceSurfaceFormats2KHR failed with ERROR_SURFACE_LOST_KHR
Any thoughts on the next steps here? Should I take this up with mesa ?

I think there definitely is some driver issue at the heart of this, but making vulkaninfo not die is exactly the solution vulkaninfo should have to work around the issue, greatly appreciate you going ahead and making a PR.
Virtio is in active development, so I would think that make a mesa issue is a good next step. (and probably close this issue after wards because its very likely an issue on their end.)

Oh, forgot to include the vulkaninfo output, https://gist.github.com/spundun/f2c5a57bb90acb3848d274357e5cc725 in case that has any more hints as to the underlying issue.
My best guess is that there's a gap in the negotiation between mesa-virtio(the vulkan driver)/sommelier(wayland server)/VirGL(the Venus renderer that mesa and sommelier should be talking to) I might be missing a component here, I'm not sure what handles compositing on the host side, I'm assuming it's VirGL but it could be yet another component.

Thanks for making the issue there and creating a reproducible! That will definitely help the Mesa devs figure out what’s going on.


Honor draft status of pages under homepage/

Just a small change so that pages with draft: true are not rendered unless the page is build with BuildDrafts.

Hi @ibutra, nice catch ⚾ 👍. I simplified the code a little bit, see https://github.com/janraasch/hugo-scroll/commit/8171dde83b5b5e5417f5912219f5376681c2424d. Please give it a try 🙏🏻 .


feat: Create read job post

The Create and Read functionalities for a job post were implemented, The Create has an endpoint 'CreateJobPost', The Read has two endpoint 'FetchAllJobPost' and 'FetchJobPostByID'.
Description

Implemented the Create functionality and also the Read functionalities for the job post/listing feature, these endpoint followed the requirement of the code base such that the router -> controller -> service -> model -> repository ... and hence no direct interaction with db from controllers or service file, the controller only handles validation, calling of service and logging appropriate messages.

Related Issue (Link to Github issue)

54
53
52

How Has This Been Tested?

The test case covers create a new job post, and fetching the created job post, and also all job post available in the DB.
E2E Test case can be run by:  go test -v .\tests\jobpost_test.go
Screenshots (if appropriate - Postman, etc):
CREATE ENPOINT:

FETCH JOB POST BY ID:

FETCH ALL JOB POST:

Types of changes

[ ] 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 change)

Checklist:

[x] My code follows the code style of this project.
[x] I have updated the documentation accordingly.
[x] I have added tests to cover my changes.
[x] All new and existing tests passed.

@Cyberguru1 please help review

@NwokoyeChigozie please help review.


.corr() takes orders of magnitude longer than np.corrcoef()

I want to create a correlation matrix for a large dataframe (>1k columns, >2 million rows).
What happened:
The numpy calculation runs in ~6sec for a dataframe of shape (1000000, 1000). The dask calculation did not finish in >1hour.
What you expected to happen:
I expected Dask to finish the calculation in a reasonable time similarly to the numpy runtime.
Minimal Complete Verifiable Example:
# Reproduceable Dataset
import pandas as pd
import numpy as np

num_cols = 10**3
num_rows = 10**6
# Create a random matrix with values in [0, 1, 2, 3, 4] and transform the dtype to int8 to save memory
df = pd.DataFrame( (np.random.rand(num_rows, num_cols)*5) // 1, columns=["feature_" + str(i) for i in range(num_cols)], dtype="int8")

# numpy solution
data = df.to_numpy()
corrmat = np.corrcoef(data.T)

# Dask
from dask.distributed import Client
import dask.dataframe as dd
client = Client()
client

import dask.dataframe as dd
ddf = dd.from_pandas(df, npartitions=16)
ddf = ddf.repartition(partition_size="100MB")
features = [c for c in ddf.columns if c.startswith("feature_")]

corrs = ddf[features].corr(method="pearson")
corrs = corrs .compute()

Anything else we need to know?:
Environment:

Dask version: 2021.09.0
Python version: 3.8.11
Operating System: Windows 10
Install method (conda, pip, source): pip

This is kind of an odd comparison. The dask analog for np.corrcoef would be dask.array.corrcoef. Here's an example using that:
import dask.array as da

arr = da.from_array(data.T)
da.corrcoef(arr).compute()

The non-distributed equivalent for dask.dataframe is pandas.

That is a fair point. I started to use np.corrcoef before I tried dask since the pandas version takes longer (apparently since it checks NAs). I think that Dask might simply not fit for my use case and will focus on other approaches.
Here are more timings with every dask version taking significantly longer in case others are interested and wants to look into it.
num_cols, num_rows = 10**2, 10**5

df.corr() # 1.3s
ddf.corr().compute() # 15s

num_cols, num_rows = 10**3, 10**5

df.corr() # 138s
ddf.corr().compute() # 1438s

num_cols, num_rows = 10**3, 10**6
data = df.to_numpy() # Switching to numpy since it takes ages with pandas

np.corrcoef(data.T) # 7.1s

import dask.array as da
arr = da.from_array(data.T, chunks="25 MIB") # tried 100, 50 and 25 MiB
da.corrcoef(arr).compute()

dask.array.corrcoef did not finish with 100, 50 and 25 MiB chunk sizes on my 32 GiB RAM (8 GiB per worker) machine. Some worker die regularly and the it eventually causes a KilledWorker error. Maybe I am using Dask wrong, the code I used is the same as above. But as I said, I will try other solutions first.

On my machine the following takes about 30 seconds and the numpy version takes 13 seconds.
import dask.array as da

arr = da.from_array(data.T)
da.corrcoef(arr).compute()

On my machine, for the code of the comment above, I'm seeing longer times for the Dask code but it does complete (116 s). The time for the numpy version (14 s).
I'm on a mac with the M1 chip, 16GB of RAM.

@Nickkontschak This issue seems to be resolved, as the computations are completing as reported in the two comments above. I'll will close this issue for now, but if you have more problems or any updates, feel free to re-open it.


[BugFix] fix invalid partition predicate bug

What type of PR is this：

[x] BugFix
[ ] Feature
[ ] Enhancement
[ ] Refactor
[ ] UT
[ ] Doc
[ ] Tool

Which issues of this PR fixes ：

Fixes #19352
Problem Summary(Required) ：

fix invalid partition predicate bug by removing useless predicates.
Checklist:

[ ] I have added test cases for my bug fix or my new feature
[ ] This pr will affect users' behaviors
[ ] This pr needs user documentation (for new or modified features or behaviors)

[ ] I have added documentation for my new feature or new function

Bugfix cherry-pick branch check:

[ ] I have checked the version labels which the pr will be auto backported to target branch

[ ] 3.0
[ ] 2.5
[ ] 2.4
[ ] 2.3

[FE PR Coverage Check]
:heart_eyes: pass : 49 / 56 (87.50%)
file detail

path
covered_line
new_line
coverage
not_covered_line_detail

:large_blue_circle:
com/starrocks/sql/optimizer/rule/transformation/materialization/ColumnRewriter.java
1
2
50.00%
[189]

:large_blue_circle:
com/starrocks/sql/optimizer/rule/transformation/materialization/RangeSimplifier.java
28
34
82.35%
[168, 170, 172, 190, 193, 202]

:large_blue_circle:
com/starrocks/sql/optimizer/rule/transformation/materialization/MaterializedViewRewriter.java
20
20
100.00%
[]

[FE PR Coverage Check]
:heart_eyes: pass : 37 / 44 (84.09%)
file detail

path
covered_line
new_line
coverage
not_covered_line_detail

:large_blue_circle:
com/starrocks/sql/optimizer/rule/transformation/materialization/ColumnRewriter.java
1
2
50.00%
[189]

:large_blue_circle:
com/starrocks/sql/optimizer/rule/transformation/materialization/RangeSimplifier.java
28
34
82.35%
[168, 170, 172, 190, 193, 202]

:large_blue_circle:
com/starrocks/sql/optimizer/rule/transformation/materialization/MaterializedViewRewriter.java
8
8
100.00%
[]

[FE PR Coverage Check]
:heart_eyes: pass : 37 / 44 (84.09%)
file detail

path
covered_line
new_line
coverage
not_covered_line_detail

:large_blue_circle:
com/starrocks/sql/optimizer/rule/transformation/materialization/ColumnRewriter.java
1
2
50.00%
[189]

:large_blue_circle:
com/starrocks/sql/optimizer/rule/transformation/materialization/RangeSimplifier.java
28
34
82.35%
[168, 170, 172, 190, 193, 202]

:large_blue_circle:
com/starrocks/sql/optimizer/rule/transformation/materialization/MaterializedViewRewriter.java
8
8
100.00%
[]

run starrocks_admit_test

run starrocks_be_unittest

run starrocks_clang-format

run starrocks_fe_unittest

run all

run starrocks_fe_unittest

run starrocks_admit_test

[FE PR Coverage Check]
:heart_eyes: pass : 34 / 35 (97.14%)
file detail

path
covered_line
new_line
coverage
not_covered_line_detail

:large_blue_circle:
com/starrocks/sql/optimizer/rule/transformation/materialization/ColumnRewriter.java
1
2
50.00%
[189]

:large_blue_circle:
com/starrocks/sql/optimizer/rule/transformation/materialization/MaterializedViewRewriter.java
8
8
100.00%
[]

:large_blue_circle:
com/starrocks/sql/optimizer/rule/transformation/materialization/RangeSimplifier.java
25
25
100.00%
[]


Per world cooldown

Plugin Version:
3.0.3
Request:
Add per world cooldown.
How/Why:
Servers might have multiple worlds with specific uses, for example we have main worlds and also mining worlds which are reset each month.
I'd be nice if a different cooldown could be set for each world, e.g. higher for the main worlds and lower for the mining worlds. This would encourage players to use the mining worlds to gather resources, not the main ones.
Additional Context:
This is an idea from one of our players – I'd great to have such a feature, but it's not necessary, therefore I wouldn't mind if you declined this.

The way worlds are setup where they are fairly dynamic MIGHT allow for this feature without a huge cooldown recode. But this would be a fairly large change in how cooldowns are stored and controlled. I would say this might be a little far away to implement, but I will keep this in mind.
Thank you for the suggestions!

The way worlds are setup where they are fairly dynamic MIGHT allow for this feature without a huge cooldown recode. But this would be a fairly large change in how cooldowns are stored and controlled. I would say this might be a little far away to implement, but I will keep this in mind.
Thank you for the suggestions!

I was a bit sad to see this wasn't included in the very customizable per-world settings. So I would just like to second this feature request! I would love to see this in a future update.
What I specifically need, is a cooldown difference between "Normal worlds" and "Nether worlds", to be able to allow players to use RTP in the Nether as well. So I was wondering: If you were to program a seperate cooldown for normal worlds and one for nether worlds - Would that still be a large change in how cooldowns are stored? Or would that be a simpler solution?
Either way, I plan to use your plugin, since it's better than all the others with the current features. I will try to find alternative ways to make seperate cooldowns for the time being.
Have a nice day!

I was a bit sad to see this wasn't included in the very customizable per-world settings. So I would just like to second this feature request! I would love to see this in a future update.
What I specifically need, is a cooldown difference between "Normal worlds" and "Nether worlds", to be able to allow players to use RTP in the Nether as well. So I was wondering: If you were to program a seperate cooldown for normal worlds and one for nether worlds - Would that still be a large change in how cooldowns are stored? Or would that be a simpler solution?
Either way, I plan to use your plugin, since it's better than all the others with the current features. I will try to find alternative ways to make seperate cooldowns for the time being.
Have a nice day!

With the new cooldowns feature being rethought, world cooldowns is around the corner ;) I will continue developing and adding this VERY soon.

I'm glad to hear this!

yeah please i also expect this feature to be added soon <3 nice plugin btw

After many months of work, this feature is finally out on the much awaited 3.4.0 update! Check it out here https://www.spigotmc.org/resources/betterrtp.36081/update?update=452508

thanks sir! :D

Thank you for your tremendous effort, I appreciate it a lot – our players will definitely enjoy this feature!


Updates

Hey @Dipanjali1 thank you for your contribution, really means a lot
Happy Hacktoberfest!! <3

Hey @GeekGuy-29  although my pr was accepted , but it is showing excluded by the hacktober fest team . Can you explain why?

@Dipanjali1 I too don't know why.. I asked about it in the server, they said this won't happen
But idk,
But don't you bother there are thousands of repository to contribute, you should check those as well
Don't loose hope buddy
At this point I can't do anything as well
If this situation prevails, I'll private this repo and won't waste someone else's time


new command

any chance you could add a command like 7d!connect > it would show people the ip:port

Hello! I'll make a note of this for a future addition.
I'm closing out feature requests to keep this issue tracker for strictly bugs and support. Feel free to bring further discussion to the Discord.


Update Archivematica users to use .org email addresses

In early September, staff email addresses will be switching from @wellcome.ac.uk to @wellcome.org. We need to update all the users in Archivematica, or nobody will be able to log in any more.
See also, discussion in Slack: https://wellcome.slack.com/archives/CGXDT2GSH/p1597680347038000

Everyone will be able to log in with a .org address when the switch happens. I’ll open a new ticket to track deleting that.


Pyvista unstructured grid example

@banesullivan please take a look.
@jourdain any thoughts or tips here?

Why just vtkUnstructuredGrid?:
https://github.com/InsightSoftwareConsortium/itk-jupyter-widgets/blob/cf33dad9082965aeca9b21767c53f0474dae38e8/itkwidgets/_transform_types.py#L297
Why not everything besides vtkImageData? That way all VTK and PyVista types could easily be passed? E.g.:
elif have_vtk and isinstance(geometry_like, (vtk.vtkUnstructuredGrid,
                    vtk.vtkStructuredGrid,
                    vtk.vtkRectilinearGrid)):

Also there are flashier datasets than the hexbeam:
import pyvista as pv
from pyvista import examples
from itkwidgets import view
import os

try:
    from urllib.request import urlretrieve
except ImportError:
    from urllib import urlretrieve

# Download data
file_name = 'uGridEx.vtk'
if not os.path.exists(file_name):
    url = 'https://data.kitware.com/api/v1/file/5d3752c2877dfcc9022724d9/download'
    urlretrieve(url, file_name)

mesh = pv.read(file_name)
view(geometries=mesh)

PolyData
mesh = examples.download_nefertiti()
view(geometries=mesh)

StructuredGrid
mesh = examples.download_st_helens().warp_by_scalar().extract_geometry()
view(geometries=mesh)

And heres another unstrucured grid but it throws an error when trying to view it... looks like something with the vtkjs mapping
mesh = examples.load_channels().threshold([2,5])
import vtk
print(isinstance(mesh, vtk.vtkUnstructuredGrid))
view(geometries=mesh)

Traceback:
ERROR:root:
UnicodeDecodeError while processing traceback.

ERROR:root:
UnicodeDecodeError while processing traceback.

True
---------------------------------------------------------------------------
KeyError                                  Traceback (most recent call last)
~/Documents/OpenGeoVis/External/itk-jupyter-widgets/itkwidgets/trait_types.py in validate(self, obj, value)
    271                 if not isinstance(geometry, dict) or not 'vtkClass' in geometry:
--> 272                     geometries[index] = to_geometry(geometry)
    273             return geometries

~/Documents/OpenGeoVis/External/itk-jupyter-widgets/itkwidgets/_transform_types.py in to_geometry(geometry_like)
    300         geometry_filter.Update()
--> 301         geometry = to_geometry(geometry_filter.GetOutput())
    302         return geometry

~/Documents/OpenGeoVis/External/itk-jupyter-widgets/itkwidgets/_transform_types.py in to_geometry(geometry_like)
    271                     'size': vtk_array.GetSize(),
--> 272                     'dataType': _vtk_data_type_to_vtkjs_type[vtk_array.GetDataType()],
    273                     'values': vtk_to_numpy(vtk_array) } }

KeyError: 8

During handling of the above exception, another exception occurred:

During handling of the above exception, another exception occurred:

AttributeError                            Traceback (most recent call last)
~/.local/lib/python3.7/site-packages/IPython/core/interactiveshell.py in showtraceback(self, exc_tuple, filename, tb_offset, exception_only, running_compiled_code)
   2038                         # in the engines. This should return a list of strings.
-> 2039                         stb = value._render_traceback_()
   2040                     except Exception:

AttributeError: 'TraitError' object has no attribute '_render_traceback_'

During handling of the above exception, another exception occurred:

TypeError                                 Traceback (most recent call last)
~/.local/lib/python3.7/site-packages/IPython/core/interactiveshell.py in run_code(self, code_obj, result, async_)
   3340             if result is not None:
   3341                 result.error_in_exec = sys.exc_info()[1]
-> 3342             self.showtraceback(running_compiled_code=True)
   3343         else:
   3344             outflag = False

~/.local/lib/python3.7/site-packages/IPython/core/interactiveshell.py in showtraceback(self, exc_tuple, filename, tb_offset, exception_only, running_compiled_code)
   2040                     except Exception:
   2041                         stb = self.InteractiveTB.structured_traceback(etype,
-> 2042                                             value, tb, tb_offset=tb_offset)
   2043
   2044                     self._showtraceback(etype, value, stb)

~/.local/lib/python3.7/site-packages/IPython/core/ultratb.py in structured_traceback(self, etype, value, tb, tb_offset, number_of_lines_of_context)
   1383         self.tb = tb
   1384         return FormattedTB.structured_traceback(
-> 1385             self, etype, value, tb, tb_offset, number_of_lines_of_context)
   1386
   1387

~/.local/lib/python3.7/site-packages/IPython/core/ultratb.py in structured_traceback(self, etype, value, tb, tb_offset, number_of_lines_of_context)
   1286             # Verbose modes need a full traceback
   1287             return VerboseTB.structured_traceback(
-> 1288                 self, etype, value, tb, tb_offset, number_of_lines_of_context
   1289             )
   1290         elif mode == 'Minimal':

~/.local/lib/python3.7/site-packages/IPython/core/ultratb.py in structured_traceback(self, etype, evalue, etb, tb_offset, number_of_lines_of_context)
   1148         exception = self.get_parts_of_chained_exception(evalue)
   1149         if exception:
-> 1150             formatted_exceptions += self.prepare_chained_exception_message(evalue.__cause__)
   1151             etype, evalue, etb = exception
   1152         else:

TypeError: can only concatenate str (not "list") to str

But it would look something like:

@banesullivan I think you need to extract_geometry() on your unstructuredGrid unless it is supposed to happen automatically? (@thewtex ?)
Otherwise a quick glimpse through the code seems fine to me. @thewtex what were you curious about from my end? Feel free to call me today, I'm still in NY, so we have the same time zone...

@banesullivan I think you need to extract_geometry() on your unstructuredGrid unless it is supposed to happen automatically? (@thewtex ?)

We shouldn't need it because of:
https://github.com/InsightSoftwareConsortium/itk-jupyter-widgets/blob/cf33dad9082965aeca9b21767c53f0474dae38e8/itkwidgets/_transform_types.py#L297-L302
which applies the vtkGeometryFilter to make unstrucured grids possible in this PR. If I do apply PyVista's equivalent of this, extract_geometry(), I still get the error which is due to having int64 value in the cell data of this mesh. If I cast those values to int32 then all is good:
...
mesh = examples.load_channels().threshold([2,5]).extract_geometry()
import vtk, numpy as np
assert isinstance(mesh, vtk.vtkPolyData)
assert mesh['facies'].dtype == np.int64
mesh['facies'] = mesh['facies'].astype(np.int32)
view(geometries=mesh)

(same traceback)
@thewtex: can you fix issues with int64 values?

@banesullivan fantastic suggestions -- we'll add these!
@jourdain thanks for the thoughts. I was wondering if you take a similar approach when encountering 64 bit integer data array. Glad to hear you are back in The States. I am in Long Island visiting family, but I will be back on Monday.

Yes I do use the same approach converting the int to something manageable on the JS side.

Why just vtkUnstructuredGrid?:
https://github.com/InsightSoftwareConsortium/itk-jupyter-widgets/blob/cf33dad9082965aeca9b21767c53f0474dae38e8/itkwidgets/_transform_types.py#L297
Why not everything besides vtkImageData? That way all VTK and PyVista types could easily be passed? E.g.:
elif have_vtk and isinstance(geometry_like, (vtk.vtkUnstructuredGrid,
                    vtk.vtkStructuredGrid,
                    vtk.vtkRectilinearGrid)):

Addressed in #178

file_name = 'uGridEx.vtk'

Since this is already in a VTK example, I'll use others for variety.

mesh = examples.download_nefertiti()

Nice! Appended to the pyvista.PolyData example.

mesh = examples.download_st_helens().warp_by_scalar()

Awesome! Added to a new pyvista.StructuredGrid example. :rocket: :guitar:

mesh = examples.load_channels().threshold([2,5])

But it would look something like:

Fix added and appended to the pyvista.StructuredGrid example.


pinned pip version

we pinned pip version in the ci script bc it was breaking, at some point we should remove that - when a working version is out

It's passing. I am closing the issue


metasploit: 6.2.10 -> 6.2.11

Description of changes
Update to latest upstream release 6.2.11

Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[x] Tested basic functionality of all binary files (usually in ./result/bin/)
22.11 Release Notes (or backporting 22.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[x] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 185943 run on x86_64-linux 1

2 packages built:

metasploit
msfpc

Result of nixpkgs-review pr 185943 run on x86_64-darwin 1

2 packages built:

metasploit
msfpc


Add Prefix & Suffix Widget in TextField

Currently TextField doesn't have any icon as prefix icon or suffix icon. It's better to have optional prefix and suffix icon. So, When User need it they can directly used for performing different actions on textfield.
Currently

Expected
Prefix Icon

Suffix Icon

Development Walkthrough

Create optional parameter for prefixIcon & suffixIcon in lib/src/input_fields/primary_input_field.dart.
Pass the parameters to the actual components.
Test the Component is it working as expected.
onClick is working perfectly or not.
Raise the PR of your code

Thank You!

#113

#113 Merged to development 🔥


WIP: Feature/creating deleting user addresses

With this the controller can now:

add new addresses
delete addresses
get all addresses from a user

When you want to merge it the restriction in the API that only two addresses are allowed needs to be removed.

@sandermangel  any news on this pr?


Allow @ to be space instead in format for from_str on DateTimeTuple

Consider making space default formatting as I believe it's closer to ISO-8601

Actually, ISO-8601 uses T - perhaps allow any of the three


Update Hashdiff Constant

Why This Change Is Necessary
The hashdiff gem is changing its constant and thus we need to do the
same.
How These Changes Address the Issue
Update HashDiff references to Hashdiff.
Side Effects Caused By This Change
None known.

Actions:

References #822

@jfelchner thank you. we can sync releases or the code can try to load Hashdiff constant and fallback to HashDiff is not present.

@bblimke no problem! hashdiff 0.4.0 was just released and the PR should be safe to merge now.

Once 0.4.0 is in the wild for a couple months, and your users have a chance to upgrade to latest, we'll release 1.0.

@bblimke because of webmock's popularity and the fact that the prior versions aren't pessimistically locked, we may want to make these same changes on the 1.x and 2.x branches and release new versions for them as well.
There are some popular gems that are locked to the 1.x and 2.x versions.

If you re-run the travis build, it should work, now that 0.4.0 is released.

@teeparham Yep. At this point I'm just waiting for a reply from @bblimke

@jfelchner Thanks for fixing this annoying warning. 👍

@bblimke can we get this merged?

@jodosha strictly speaking, this won't remove the warning just yet.  But it will as soon as we release hashdiff 1.0 and you bundle update.

@bblimke do you want me to create patches for v1 and v2 as well?

@jfelchner not sure it's worth supporting these old releases. people can always set versions of hashdiff in their Gemfiles. Though feel free to prepare these. I can release them as well.
I'll try to cut a new 3.x release tonight.

@bblimke not a problem.  Up to you.  I just wanted to offer. :)

For anyone else who wants to upgrade and remove the warning:
I upgraded to webmock version 3.6.0, which includes this change (renames the constant from HashDiff to Hashdiff). I also followed these instructions to upgrade to the hashdiff 1.0 beta:
gem 'hashdiff', ['>= 1.0.0.beta1', '< 2.0.0']

Then run bundle update hashdiff. This makes sure you are using the latest versions and will get rid of the warning.


Missing files in sdist

It appears that the manifest is missing at least one file necessary to build
from the sdist for version 3.0.8. You're in good company, about 5% of other
projects updated in the last year are also missing files.
+ /tmp/venv/bin/pip3 wheel --no-binary tago -w /tmp/ext tago==3.0.8
Looking in indexes: http://<IP_ADDRESS>:9191/root/pypi/+simple/
Collecting tago==3.0.8
  Downloading http://<IP_ADDRESS>:9191/root/pypi/%2Bf/88b/83c3433f626ee/tago-3.0.8.tar.gz (12 kB)
    ERROR: Command errored out with exit status 1:
     command: /tmp/venv/bin/python3 -c 'import sys, setuptools, tokenize; sys.argv[0] = '"'"'/tmp/pip-wheel-hiy_haiw/tago/setup.py'"'"'; __file__='"'"'/tmp/pip-wheel-hiy_haiw/tago/setup.py'"'"';f=getattr(tokenize, '"'"'open'"'"', open)(__file__);code=f.read().replace('"'"'\r\n'"'"', '"'"'\n'"'"');f.close();exec(compile(code, __file__, '"'"'exec'"'"'))' egg_info --egg-base /tmp/pip-wheel-hiy_haiw/tago/pip-egg-info
         cwd: /tmp/pip-wheel-hiy_haiw/tago/
    Complete output (5 lines):
    Traceback (most recent call last):
      File "<string>", line 1, in <module>
      File "/tmp/pip-wheel-hiy_haiw/tago/setup.py", line 8, in <module>
        with open('requirements.txt', 'r') as f:
    FileNotFoundError: [Errno 2] No such file or directory: 'requirements.txt'
    ----------------------------------------
ERROR: Command errored out with exit status 1: python setup.py egg_info Check the logs for full command output.

@thatch I'm not sure what you doing. But I just fire up a docker on using python3.7, the install was completely with success.
Also, the requirements.txt is not on package because it's load up on pip package build. You can see it, just download the package, unpack it, and go to ./tago.egg-info/requires.txt
If you need make a strange installation of that package, feel free to open a PR with your changes, if it make senses I can merge it without problems.
Thank you

Just created #19