[Bug]: Merge Pets non-functional in Patch 10.2

Are you using the latest version of the addon?

[X] Yes
[ ] No

Addon Version
Version 1.8.85 - Date: 2023-11-09 @ 07:30 GMT
Did you try having only this addon enabled?

[ ] Yes
[X] No

Current Behavior
When pets are summoned, the owner/player is no longer the primary damage source instead of being merged together. They are instead bumped to number 2, and the pet that triggers the error also does not appear on the main window either.
Expected Behavior
Pets to be merged as one combined result.
Lua Error
N/A
Reproduction Steps
Have merge pets enabled
Enter Combat
Summon any temporary guardian
Last Good Version
10/30/2023
Screenshots

Confirm. Merge pets option works incorrectly.

Hello @wordup834 . Thanks for reporting this. It should be Version: 1.8.85 - Date: 2023-08-10 @ 12:39
Please try this version (Version: 1.8.86 - Date: 2023-11-07 @ 07:57) and see if it's fixed because there was a small problem with flags that I fixed and it might have fixed the issue (get 3.4.x, it's for both classic and retail).

Hello @wordup834 . Thanks for reporting this. It should be Version: 1.8.85 - Date: 2023-08-10 @ 12:39 Please try this version (Version: 1.8.86 - Date: 2023-11-07 @ 07:57) and see if it's fixed because there was a small problem with flags that I fixed and it might have fixed the issue (get 3.4.x, it's for both classic and retail).

Unfortunately not, issue still present using 3.4.x - issue possibly related to requiring server name in owner name now following cross faction guilds?

I took a look at the changes and noticed that the game is always adding the realm's name even if players are from the same realm. If you test out this file it would be great! Simply drop it inside Skada\Core folder making sure it asks you to replace the file Init.lua.

This fixes the issue yes, looks like it's merging pets again, Thanks for the fix!

Seems to be broken once again for me - i have tried both the v1.8.85 + init file and the v1.8 .86 + init file.

Thanks for reporting this @ArenfeIdt .. I could test the addon after ages of updates and I could confirm it. Try to update the addon from the mega folder, I believe I fixed it.

Yes, it works again now - however, the Version in the Mega Folder on its own does not work for me. Only after adding the Init.lua File you posted earlier in this Issue Report does it work. Just tested it with the Priest Shadowfiend, BM Hunter Pets and the Voidwalker Warlock Pet. All of them merge properly now. Probably better if you add the Init.lua File directly to the new Version ZIP.

There isn't a big difference between both files. However, the file I posted had an issue, it was using the player's realm name (yours) even for players from other realms, so I just edited that part to use the realm name of each player.

Maybe something is different because i use the Setting to not display Realm Names.

It shouldn't affect anything, that option simply strips off the realm name, for display purposes, not data.

Hmm, i just know that if i use the Init file that comes with the entire Skada Folder it doenst work for me - only after using the extra init file you posted does it work for me,


Can´t use Version 1.2.0

Hi, after update to 1.2.0 I can't use your Plugin. It missed the http _port option that I can set by my self.
1.1.4 works fine.

It now reads the HTTP port from the homebridge-camera-ffmpeg plugin that is required to be set anyways.

I use your plugin in combination with the homebridge-camara-ui Plugin from Seydx. It need to set the http_port to get a Message.

Ah, I didn't consider that people used it with that. I will see about either reading their config or just allowing an override.
Will give it some thought today and determine the best way forward.

Added override_http in v1.2.1


Trying to use --env= parameter without success event after following documentation

Hi,
in need to use fixture as basic preloaded data for now on prod env before using real data. So i'm in PROD and i tryed to put fixutes into ./fixtures/ directory (in flex symfony 4 project)
When you read doc, ./fixtures/ is envoronment-less files ... but i get this :
php bin/console hautelook:fixtures:load -vvv
Careful, database will be purged. Do you want to continue y/N ?y
18:02:53 INFO      [app] fixtures found
[
"files" => []
]
18:02:53 INFO      [app] Purging database with purge mode "DELETE_MODE".
18:02:53 INFO      [app] Loading fixtures.
18:02:53 INFO      [app] Pre-processing objects.
18:02:53 INFO      [app] Flushing objects.
18:02:53 INFO      [app] Post-processing objects.
18:02:53 INFO      [app] Done.
18:02:53 INFO      [app] fixtures loaded
i tryed --env=inte like in the documentation, and move my files to ./fixtures/inte/ but i have this result :
php bin/console hautelook:fixtures:load -vvv --env=inte
Careful, database will be purged. Do you want to continue y/N ?y
'NOTHING IN OUTPUT CONSOLE'
and if i try --env=prod it's the as without --env (still no fixture files found). Like only DEV is taking into account.
Any one can help me understand ? documentation is wrong? there is a bug ?

Hi, thanks for the report. Is there any way to reproduce this or even better could you provide a reproducer?

Sorry it was my mistake i didn't copy the config on prod folder


Gpl 674 prevent merger pcr plates into pool with different barcode sets

Closes #516
Changes proposed in this pull request:

Add toggle to purposes configuration, enforce_same_template_within_pool?, which can be used for Heron tailed pipeline
Add various comments and whitespace for understanding and readability

Ooh, realised last night that this is adding a new purposes.yml option. Any chance to could add it to the config added in: https://github.com/sanger/limber/pull/528 (which I've just merged in). Bit cheeky to ask I know, as the code was still waiting for a merge when you submitted the pull request.

Yeah no probs, will add it to the new docs


SquealGen Constraint Query

Subsuming #3 and #6 and helping to address cross schema foreign keys, I suggest we update the constraint query to:
SELECT
  con.conname AS con,
  con.contype AS typ,
  nsp.nspname AS nsp,
  tab.relname AS tab,
  COALESCE (
    array_agg (
      ALL col.attname ORDER BY array_position(con.conkey, col.attnum) ASC
    ) FILTER (WHERE col.attname IS NOT NULL),
    ARRAY[] :: text[]
  ) as cols,
  fnsp.nspname AS fnsp,
  ftab.relname AS ftab,
  array_agg (
    ALL fcol.attname ORDER BY array_position(con.confkey, fcol.attnum) ASC
  ) FILTER (WHERE fcol.attname IS NOT NULL) AS fcols
FROM pg_catalog.pg_constraint AS con
INNER JOIN pg_catalog.pg_class AS tab
ON con.conrelid = tab.oid
INNER JOIN pg_catalog.pg_namespace AS nsp
ON con.connamespace = nsp.oid
LEFT OUTER JOIN pg_catalog.pg_attribute AS col
ON con.conkey @> ARRAY[col.attnum] AND con.conrelid = col.attrelid
LEFT OUTER JOIN pg_catalog.pg_class AS ftab
ON con.confrelid = ftab.oid
LEFT OUTER JOIN pg_catalog.pg_namespace AS fnsp
ON ftab.relnamespace = fnsp.oid
LEFT OUTER JOIN pg_catalog.pg_attribute AS fcol
ON con.confkey @> ARRAY[fcol.attnum] AND con.confrelid = fcol.attrelid
WHERE con.contype IN ('f', 'c', 'p', 'u')
GROUP BY
  con.conname,
  con.contype,
  nsp.nspname,
  tab.relname,
  fnsp.nspname,
  ftab.relname
;

To see how this query works, start with a prototype table in an otherwise fresh example database:
CREATE TABLE foo (
  x int NOT NULL,
  y int NOT NULL,
  CONSTRAINT pk PRIMARY KEY (x),
  CONSTRAINT fk FOREIGN KEY (y) REFERENCES foo (x),
  CONSTRAINT uq UNIQUE (y),
  CONSTRAINT ch1 CHECK (x = y),
  CONSTRAINT ch2 CHECK (true)
);

Running the constraint query at the top then gives
 con | typ |  nsp   | tab | cols  |  fnsp  | ftab | fcols
-----+-----+--------+-----+-------+--------+------+-------
 ch1 | c   | public | foo | {x,y} |        |      |
 ch2 | c   | public | foo | {}    |        |      |
 fk  | f   | public | foo | {y}   | public | foo  | {x}
 pk  | p   | public | foo | {x}   |        |      |
 uq  | u   | public | foo | {y}   |        |      |

thanks, merged in https://github.com/mwotton/squealgen/commit/eec33d50aaec483524ddfb75ed137a1cbf53086f


client.sendCodeToPhone fails with TypeError: undefined is not a function

This looks like the same issue as #8, but I am using NodeJS version 0.12.7, which should be supported (?)
Provide your phone number in order to receive the verification code from Telegram
Phone number, must have the international code: +xxxxxxxxxxx
/usr/local/lib/node_modules/termgram/lib/client-proxy.js:73
                            client.sendCodeToPhone(phoneNumber, fulfill);
                                   ^
TypeError: undefined is not a function
    at /usr/local/lib/node_modules/termgram/lib/client-proxy.js:73:36
    at Client.<anonymous> (/usr/local/lib/node_modules/termgram/lib/client-proxy.js:124:13)
    at Client.g (events.js:199:16)
    at Client.emit (events.js:104:17)
    at null._onTimeout (/usr/local/lib/node_modules/termgram/node_modules/telegram.link/lib/utility.js:58:26)
    at Timer.listOnTimeout (timers.js:119:15)

Please, try to update the project to the last version and let me know. Consider anyway this example application not yet stable and complete.

Hello, I really love that you're building a node library and a command line client for telegram!
I'm trying to run termgram, but I have the same problem as @0x15:
Provide your phone number in order to receive the verification code from Telegram
Phone number, must have the international code: +39xxxxxxxxxx
/home/fazo/.npm_prefix/lib/node_modules/termgram/lib/client-proxy.js:75
                            client.sendCodeToPhone(phoneNumber, fulfill);
                                   ^
TypeError: undefined is not a function
    at /home/fazo/.npm_prefix/lib/node_modules/termgram/lib/client-proxy.js:75:36
    at Client.<anonymous> (/home/fazo/.npm_prefix/lib/node_modules/termgram/lib/client-proxy.js:126:13)
    at Client.g (events.js:199:16)
    at Client.emit (events.js:104:17)
    at null._onTimeout (/home/fazo/.npm_prefix/lib/node_modules/termgram/node_modules/telegram.link/lib/utility.js:58:26)
    at Timer.listOnTimeout (timers.js:119:15)

This was obtained by cloning the repo, npm install -g the code directly from master and running termgram. I got the same error by downloading the latest release from npm.

I can't get past the phone number request as well. I get either the message saying it is not an international number, or I get stuck with an " undefined '+55xxxxxxxxxx' ".

@tomcpc I'm sorry, I tried to fix termgram but it didn't work out. If you want to see a working client, try my own


xmlschema.rb: use File.exist?

🦟 Bug fix
Fixes compilation with ruby 3
Summary
The File.exists? method is deprecated and has been removed from ruby 3.
Checklist

[X] Signed all commits for DCO
[ ] Added tests
[ ] Updated documentation (as needed)
[ ] Updated migration guide (as needed)
[ ] Consider updating Python bindings (if the library has them)
[X] codecheck passed (See contributing)
[X] All tests passed (See test coverage)
[ ] While waiting for a review on your PR, please help review another open pull request to support the maintainers

Note to maintainers: Remember to use Squash-Merge and edit the commit message to match the pull request summary while retaining Signed-off-by messages.

Is this backporting #1216 ?

Is this backporting #1216 ?

oops, I hadn't seen that. I'll decline this and cherry-pick that one

Is this backporting #1216 ?

oops, I hadn't seen that. I'll decline this and cherry-pick that one

ok we can use rebase-and-merge for https://github.com/gazebosim/sdformat/pull/1223


Request: Restore from an older value.

Is it possible to make the library persist the last value on disk and restore it after an app restart?
My suggestion:
public class TimeFreeze : NSObject, NSCoding {

    // NOTE: `adjustedTimestamp` is not an optional anymore
    var adjustedTimestamp: TimeInterval { ... }

    public required init?(coder aDecoder: NSCoder) {

        guard
            aDecoder.containsValue(forKey: "uptime"),
            aDecoder.containsValue(forKey: "offset"),
            aDecoder.containsValue(forKey: "timestamp")
        else { return nil }

        self.uptime = aDecoder.decodeDouble(forKey: "uptime")
        self.timestamp = aDecoder.decodeDouble(forKey: "offset")
        self.offset = aDecoder.decodeDouble(forKey: "timestamp")
    }

    public func encode(with aCoder: NSCoder) {

        aCoder.encode(self.uptime, forKey: "uptime")
        aCoder.encode(self.timestamp, forKey: "timestamp")
        aCoder.encode(self.offset, forKey: "offset")
    }
}

public struct Clock {

    // NOTE: `private(set)`
    public private(set) static var stableTime: TimeFreeze?

    public static func restore(from other: TimeFreeze) {

        guard self.stableTime == nil || (self.stableTime!.adjustedTimestamp < other.adjustedTimestamp) else { return }

        self.stableTime = other
    }
}

What you are asking makes sense to me; the caveat though is that Kronos can't guarantee anymore that the clock is accurate since the device could be rebooted in the middle which makes me hesitant to implement this.

@Reflejo It's fine, because there is simply no better way to do so. You can provide an API note that this cannot be guaranteed. In general the clock might be off-sync after a restart of the app, so it's up to the developer to sync it again. The restored value is needed if you close your app and lose internet connection when relaunching it. Currently there is no way to get the clock ticking until the internet connection could be restored.

I'd like to keep that guarantee (non nil means trustfully time). What we can do is expose a Clock.trustfullyOffset = ... so you can implement this logic on the app side.

Fixed here https://github.com/lyft/Kronos/pull/22 thanks to @mallorypaine


hammerjs import error => Cannot find name 'HammerManager'

Bug, feature request, or proposal:
Bug
What is the expected behavior?
You need to 'import hammers' at @angular2-material/core/gestures/MdGestureConfig.d.ts
What is the current behavior?
Error for import.

What version of Material are you using @crossme?

@crossme Can you provide some more info about your Material version and what you're trying to do?

Closing this issue as it does not follow the issue template. Please file a new issue that follows the template, which gives the team the information needed to investigate.


feat(autoware_map_msgs): add msg and srv files releated with dynamic lanelet loading

Description
Add some msg and srv files will use from dynamic lanelet loading.
Related links
Proposal Link
Tests performed

Notes for reviewers

Interface changes

Effects on system behavior

Pre-review checklist for the PR author
The PR author must check the checkboxes below when creating the PR.

[x] I've confirmed the contribution guidelines.
[x] The PR follows the pull request guidelines.

In-review checklist for the PR reviewers
The PR reviewers must check the checkboxes below before approval.

[ ] The PR follows the pull request guidelines.
[ ] The PR has been properly tested.
[ ] The PR has been reviewed by the code owners.

Post-review checklist for the PR author
The PR author must check the checkboxes below before merging.

[ ] There are no open discussions or they are tracked via tickets.
[ ] The PR is ready for merge.

After all checkboxes are checked, anyone who has write access can merge the PR.

Just a confirmation. :pray:
Does this Pull Request mean that approach3 has been adopted?
https://github.com/orgs/autowarefoundation/discussions/4120

Thank you for your PR :+1:
May I make two requests?

Also, the interface is very important to the microautonomy architecture, so if autoware_msgs is to be changed, it should be included in the documentation. :pray:
We are currently trying to deprecate autoware_auto_msgs. Therefore, could you please remove the dependency from this PR? Also, the timing of the merge needs to be adjusted. cc @mitsudome-r
https://github.com/orgs/autowarefoundation/discussions/3862

Okay, I'll update the documentation ASAP. Do I need to change anything else, or is the page you shared enough for now?
Yes. I think it is sufficient. :+1:

I have a question: If I remove the dependency, should I use autoware_msgs/msg/LaneletMapBin? If I switch to this message, it seems like I'll need to make several changes on both the map and planner sides. For example, I'd need to update the planner lanelet subscribers, map_loader publishers, and modify the Lanelet2Map to map binary message function, and so on. Can you confirm if my understanding is correct?

Yes. It is better to use autoware_msgs/msg/LaneletMapBin instead of autoware_auto_msgs.
The situation here is complicated, so it might be better to discuss it in the WG. @mitsudome-r

@StepTurtle @mitsudome-r @yukkysaito Hi
I want to align the message types as much as possible between PointCloudMap and LaneletMap.

In Lanelet, the term "Tile" is used, whereas in PointCloud, the term "Cell" is used. I am fine with either, but I would like to unify the term.

There are slight differences in how the message types are structured between the two.

For Lanelet, the structure is as follows:
LaneletMapTileMetaData.msg
int32 tile_id
float64 min_x
float64 max_x
float64 min_y
float64 max_y

And for PointCloud, the structure is:
PointCloudMapCellMetaData.msg
float32 min_x
float32 min_y
float32 max_x
float32 max_y

PointCloudMapCellMetaDataWithID.msg
string cell_id
PointCloudMapCellMetaData metadata

I am fine with either structure, but I would like to unify them.
Do you have any ideas?

@StepTurtle GetDifferentialLanelet2Map.srv  corresponds to GetSelectedPointCloudMap.srv in PointCloud.
So, could you please change the name to GetSelectedLanelet2Map.srv?

@YamatoAndo
I changed the service name with the name you suggest. 👍🏽

About the aligning the message types:

About 1. matter you mentioned: both of the names are okey eith me too
About 2. matter you mentioned: can you check this review: https://github.com/autowarefoundation/autoware_msgs/pull/81#discussion_r1494368841. We discussed this before.

@StepTurtle Thank you for the reply and the modification!
@mitsudome-r @yukkysaito Do you have any idaes?

I have made the comment here: https://github.com/autowarefoundation/autoware_msgs/pull/81#discussion_r1660488557.

I have made the comment here: https://github.com/autowarefoundation/autoware_msgs/pull/81#discussion_r1660488557.

I also commented on the same part.

@yukkysaito @mitsudome-r
Please also answer this comment.

In Lanelet, the term "Tile" is used, whereas in PointCloud, the term "Cell" is used. I am fine with either, but I would like to unify the term.

I want to align the message types as much as possible between PointCloudMap and LaneletMap.

I think either option is fine, but since the existing one is "cell" and is actually being used, it would be better to use "cell" unless someone else has a strong preference.

@yukkysaito Thank you for the reply!
@StepTurtle Could you please change it to cell_id?

@YamatoAndo
Just changed the tile_id with cell_id

@mitsudome-r @yukkysaito
The review of https://github.com/autowarefoundation/autoware.universe/pull/6241 is almost finished, so could you please merge this PR beforehand?

@xmfcx cc: @mitsudome-r @yukkysaito
Thank you for merging!
I would appreciate it if you could create a new release tag since I would like to use the autoware_msgs with this PR reflected.


Upgrade to Coraza 3.0

Closes #63
Superseedes #58 and  #60

Please fix the lint. I think best option for the offending lines is to add a nolint annotation.


Use ratatui-image to show images in-terminal

It would be nice to see images in-situ for terminals that support images. It seems ratatui-image handles various terminals pretty well, so the usage is pretty simple.
I've not really thought this through very well, and I'm not really sure I've got the motivation or knowledge to see this PR through to a ready state.

[x] Render preview picture in image embed modal
[ ] Render picture(s) in new post view? (after adding picture with image embed modal)

My idea here is that one can add multiple photos. It would be nice to be able to see a preview of them all before posting

[ ] Render picture(s) in post view

Question: We only have the URLs to the pictures at the moment, right? Are there helper functions in the SDK or elsewhere to let us download the contents to display? This applies to the next point as well, I think.

[ ] Render pictures in feed?

Just thumbnails, bigger photos in post view.

[ ] Honor content moderation preferences

Blur/show default "sensitive content" image? How to let the user see the content if they choose? :thinking:

[ ] Config setting to disable pictures

Maybe pictures are too much visual clutter for some, so maybe they want to disable rendered pictures altogether?

I'm not super familiar with this program yet. I started using it today. So I might not know the ins and outs of the way the architecture is meant to work.
The initial commit is only a simple test to see if it's possible at all, and I think I kind of like it!

Also, it seems like the current implementation doesn't remove the image if the path is changed to something that isn't an image (e.g. /home/cn/Pictures/stuff.jp shouldn't display an image). I'll look into that tomorrow.

I don't like it when branches live very long, so maybe it's best if we limit the feature scope for this PR.
The tasks I initially outlined was just brainstorming.
It seems a little bare to leave it the way it is now, though.
I'll focus on the two first tasks for now.

Thanks for the great suggestion and implementation.
ratatui-image is an excellent library and I once considered using it for a feature, but at the time I decided that my priority was to enhance the functionality in the Text-only UI first, so I passed on it.
Also, since the column width can be flexibly changed, I thought it would be necessary to consider many aspects of UI design to make it possible to display the columns without making them difficult to see even when they are narrow.
So I was thinking that it would be nice to have at least the ability to render the full column width of the image selected by the user on the post's detail view page.
If it is not a problem from a UI standpoint, I would be happy to add, as you write, previewing embeds when creating a post.
Still, for users who prefer a text-only UI, I think the ability to turn on/off the image display feature should be controllable via settings.


MIlestone 4

Let me know if there are any changes required or if we want to use another notebook.

Think it's ok!


Teku beaconchain is consuming too many resources

Teku beaconchain service is consuming too many CPU resources, with an average around 30%. This is issue is tracked by Teku team and they are working on it.
They suggest to run teku with a checkpoint sync

Yes! Please use Teku's checkpoint sync. You can learn more about it here:  https://consensys.net/blog/teku/teku-and-infura-team-up-to-make-the-fastest-ethereum-2-0-client-sync/ . There are instructions in the article - you don't have to get the checkpoint state from Infura, any Teku node will do, but Infura is convenient. Be sure to delete your existing beacon directory before doing a checkpoint sync. Also see our own docs: https://docs.teku.consensys.net/en/latest/HowTo/Get-Started/Checkpoint-Start/

This will get Teku up and synced in a couple of minutes. There will be heightened CPU activity for a little while as Teku (a) processes all past deposit events and (b) downloads blocks back to genesis, but that will all die down after a day or so. Meanwhile your node can be fully running.

MUST be used checkpoint sync, according to Teku team, they are not supporting anymore syncing from genesis


feat(virtual-scroll): Allow non fixed-sized containers

Feature Description
When using virtual scroll the  element needs to have a set height, or else, by default it'll be 0. It would be really useful for it to be able to have an automatic height.
Use Case
For example, i've got a page with aprox. 37 accordions, each with something between 5 and 15 mat-buttons inside each. When opening one, the opening animation is choppy because of how many are rendered on screen (this bug has already been reported in #8287).
I think this can be mitigated in this use case by only rendering what's visible with a virtual-scroll, but, because the height is dynamic(as, when an accordion needs to open, the size of the containing element changes), i cant give it a specific height, thus i can' use

Duplicate of #10113

this is not a duplicate of #10113 , as this also happens when the items have a fixed size, but the container doesn't


isValid in FormState Global

In the Form State class we have the validate () method that retrieves the state of the fields' validation by forcing true in iteracted by user.
We need an isValid method in the Form State without forcing iteracted by user to be true.

New function isValid in FormState Example

Hi @megaterrarc
Can you please explain why you need this?
If you reproduce any issue with it, can you please provide your flutter doctor -v and a minimal complete reproducible code sample and flutter run --verbose logs running the minimal code sample?
Thank you

it is necessary to disable a button if the form was not valid before the user iterated with the form. An Angular forms technique:

I will do the minimum complete reproducible code sample to give more details.

Hi @megaterrarc

If you reproduce any issue with it, can you please provide your flutter doctor -v and a minimal complete reproducible code sample and flutter run --verbose logs running the minimal code sample?
Thank you

Thanks, please provide the details to reproduce this

Without additional information, we are unfortunately not sure how to resolve this issue.
We are therefore reluctantly going to close this bug for now.
Please don't hesitate to comment on the bug if you have any more information for us; we will reopen it right away!
Thanks for your contribution.
Could everyone who still has this problem please file a new issue with the exact description of what happens, logs, and the output of flutter doctor -v.
All system setups can be slightly different, so it's always better to open new issues and reference related issues.


Fix core-setup uri

Use public uri

@mmitche do you have merge permissions in this repo? I'll merge once the checks pass if not.

I don't have merge perms here.

Not quite sure why CI isn't starting...hrm...

It's possible CI checks aren't triggered for forks. I'll see if I can make the same change myself…

Closing in favour of #345

@dougbu Triggers were disabled on forks for some reason. I enabled them.


Console Commands

Should their be console commands?

I think there should be! However, i personally don't see this as a priority right now.

If I'm not working on anything else would you like me to take a whack at this?

Sure! That would be awesome!
Could you join Gitter? https://gitter.im/kennyvv/SharpMC


Add basic code of the mixer-adapter

Codecov Report

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

@@            Coverage Diff            @@
##             master      #60   +/-   ##
=========================================
  Coverage     35.02%   35.02%
  Complexity      417      417
=========================================
  Files           128      128
  Lines          5347     5347
  Branches        993      993
=========================================
  Hits           1873     1873
  Misses         3033     3033
  Partials        441      441

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 b87cd10...b9b2abf. Read the comment docs.


Store arrays inside of Variant as shared.

Arrays inside of Variant are unique and use reference counting.
When you assign a variant containing a packed array to another, or
when you call non const functions to arrays, this will work even
if the array is inside a dictionary, so they will from now pass
as reference.
The difference with regular variant arrays is that, once passed
to a function in the C++ API, they are no longer shared. This is
required for security and thread safety, as those arrays are
mainly used to pass data back and forth even between threads.

Thanks!


Remove company name popup

It's confusing to have a company popup and a free-form field, and adds implementation complexity.  I'd recommend removing that and just having people type in the company name.  If not removed, would need a way to add to the popup from the manually entered field.

There's also a confusing behavior if you select from the popup and enter a company description.  It says you can't do that.  Which maybe is fine, but then grey out the name/description when a company is chosen from the popup.

That's a good point.  Filed it upstream here https://github.com/python/pythondotorg/issues/565

Thanks.  I still think it would be better to just remove the popup.  I don't see why the website needs to manage a list of companies.  Seems like unnecessary structure to me.  Also, this field could just be pre-filled with the name of the company for the logged-in user so they only have to type it once.

I wasn't involved in the design of this, but I believe the original idea was to have some of the more prolific Python companies have better structured data in the apps.  This way we can for example list open jobs at Industrial Light & Magic along with their success stories, view all of their open/past jobs, perhaps some day list PSF members who work there, etc.

On 11.02.2015 19:26, Frank Wiles wrote:

I wasn't involved in the design of this, but I believe the original idea was to have some of the more prolific Python companies have better structured data in the apps.  This way we can for example list open jobs at Industrial Light & Magic along with their success stories, view all of their open/past jobs, perhaps some day list PSF members who work there, etc.

In practice this won't work out, though, since companies tend to
want to put job specific descriptions in their company listings.
It's also likely that we'll get postings from different departments
within companies, different company sites, etc. who will want to
use different descriptions.
Let's drop the feature for now until we find some use for it
at some later stage.
Thanks,
Marc-Andre Lemburg
Director
Python Software Foundation
http://www.python.org/psf/

Fixed upstream with https://github.com/python/pythondotorg/commit/98db0ed58e9b52f4ea1f1d94ef3eb4261ce439d6


Type Error When Returning Either From .catchError

At the risk of wearing our my welcome, do you think this is a dartz issue or a Dart issue?
Future<Either<Exception, StreamedResponse>> _getWithHeadersStreamed(String targetUri, JwtAuthService svc) =>
    svc.getRequestHeaders().then((headersE) =>
        headersE.fold(
                (err) => Left(err),
                (headers) {
                  Logger().i("Calling REST on $targetUri");
                  Logger().i("Auth Header is ${headers["Authorization"]}");
                  var request = Request("GET", Uri.parse(targetUri))
                    ..headers.clear()
                    ..headers.addAll(headers)
                    ..followRedirects = false;
                  return http.Client().send(request).then((value) => Right(value));
                }
        )
    ).catchError((err)  => Left<Exception, StreamedResponse>(err));

This works fine if I leave the .carchError call off the end. (Other than the HttpClient spewing exceptions up the call chain.) I think you mentioned something about the type system being changed in Dart 2.x.
E/flutter ( 4358): [ERROR:flutter/lib/ui/ui_dart_state.cc(157)] Unhandled Exception: type 'Future<Object>' is not a subtype of type 'Future<Either<Exception, StreamedResponse>>'
E/flutter ( 4358): #0      _getWithHeadersStreamed (package:asset_management_mobile/client/tmi_client.dart:36:5)
E/flutter ( 4358): #1      returnJsonForUri (package:asset_management_mobile/client/tmi_client.dart:21:3)
E/flutter ( 4358): #2      getSystemUserSummary (package:asset_management_mobile/client/tmi_client.dart:14:5)
E/flutter ( 4358): #3      main (package:asset_management_mobile/main.dart:31:23)
E/flutter ( 4358): #4      _runMainZoned.<anonymous closure>.<anonymous closure> (dart:ui/hooks.dart:241:25)
E/flutter ( 4358): #5      _rootRun (dart:async/zone.dart:1184:13)
E/flutter ( 4358): #6      _CustomZone.run (dart:async/zone.dart:1077:19)
E/flutter ( 4358): #7      _runZoned (dart:async/zone.dart:1619:10)
E/flutter ( 4358): #8      runZonedGuarded (dart:async/zone.dart:1608:12)
E/flutter ( 4358): #9      _runMainZoned.<anonymous closure> (dart:ui/hooks.dart:233:5)
E/flutter ( 4358): #10     _startIsolate.<anonymous closure> (dart:isolate-patch/isolate_patch.dart:301:19)
E/flutter ( 4358): #11     _RawReceivePortImpl._handleMessage (dart:isolate-patch/isolate_patch.dart:168:12)

Hi -- no worries!
I'd say it's a little bit Dart's fault and a little bit your fault :-) Just like the Scala compiler loves to infer AnyRef and friends, Dart has a tendency to fall back to Object when insufficient information for proper type inference is available.
I would highly recommend embracing async/await. It's one of the best features of Dart and is extremely well designed and implemented compared to most languages with similar features. Type inference is greatly improved compared to then chaining and it will be much easier to spot where type information is lost.
Although fold will always be able to take you where you need to go, I would also recommend the use of map, flatMap and other higher level constructs on Either wherever possible. They are less verbose and can also help with guiding the type inference better. Whenever you see the Left side simply being "passed through" like in the fold in your example, it's a sign that you probably should be using map or flatMap instead.

OK if I close this? Don't hesitate to reopen if needed!

Sure, thanks again Björn!
On Wed, May 6, 2020 at 1:47 PM Björn Sperber<EMAIL_ADDRESS>wrote:

OK if I close this? Don't hesitate to reopen if needed!
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/spebbe/dartz/issues/42#issuecomment-624793597, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AAMPUDOZVGGUA36YXLNIHSDRQGPBLANCNFSM4MWEGFCA
.


Facebook success.html displays instead of going back to previous page

Authentication is successful but I an redirected to a white page with the following red text.
"SECURITY WARNING: Please treat the URL above as you would your password and do not share it with anyone. See the Facebook Help Center for more information."
I have only tried this on iOS at this point. It seems as if Xamarin.Auth needs to know to ignore this.
I also documented this at: http://forums.xamarin.com/discussion/25183/xamarin-forms-xamarin-auth-and-facebook?new=1

I'm also facing the same problem,can anyone please provide any solution.Thank you!!

I have the same problem in ios.

@coreyperkins  is this still an issue

no response for two weeks


MEETING TECH PHANTOMS OFFICIAL 101 (7PM on 3 Sep 2020)

We will be introducing our mentor and core members  and interact with the community members clear the doubts and all.
Publicly display our GitHub Organisation too. @vinitshahdeo @JoshuaPoddoku  @prachi-ag @Souvikns  @AnubhavMadhav @garimasingh128 @SavageShift @puranjay123 @mauriceyeng @pixan198

Success Event


Rename Send to Do for like *http.Client

Going to close as it has been a while and I'm not sure if this is relevant anymore.


eventloop in Spanish

in this video it is explained in a simple and understandable way the eventloop would be very good to include it
https://youtu.be/Eqq2Rb7LzYE

Please make a PR to include this video on the Event Loop section, like this:

📜 Desmitificando el Event Loop (Spanish)

done https://github.com/leonardomso/33-js-concepts/pull/70


Multiple line translation

When I try to translate something with multiple lines(\n) like "Hello
World" it will respond {"code":404,"message":"No text to translate}. Although "Hello World" works

Show Screenshot Pls.

Show me Screenshot Pls.

In JSON, strings cannot be written directly across lines. You should use newline characters.
curl -X POST http://localhost:1188/translate \
-H "Content-Type: application/json" \
-H "Authorization: Bearer your_access_token" \
-d '{
    "text": "Hello\nworld!",
    "source_lang": "EN",
    "target_lang": "DE"
}'

If you use this type of API debugging tool, you'll notice that when you paste something that contains line breaks, it's automatically converted to \n.

so what should I do if I want to make it able to do multiple lines. Or is it just not possible?

so what should I do if I want to make it able to do multiple lines. Or is it just not possible?

Is my expression not clear enough? I have already given you quite clear examples above.

I hope you have some basic knowledge of coding, at least to the extent that you understand what \n represents. Otherwise, I think you will not be able to use DeepLX properly.

understand, i need to convert it to \n then reconvert it back to lines

understand, i need to convert it to \n then reconvert it back to lines

It seems you still haven't understood what I meant. If you extract the data from the Response, printing it will automatically line break because it contains \n.

I need to convert it as well right?(like var=var.replace('\n', '\n')
oh, I'm not printing it.  I'm displaying it like a widget so, i convert it back (var=var.replace('\n', '\n'))

Show me your full code.

    text = data["q"]
    target_lang = data["target"]

    if "source" in data:
        source_lang = data["source"]

    deeplx_base_url = os.environ.get("DEEPLX_BASE_URL")

    if not deeplx_base_url:
        return Response(
            status_code=500,
            content=json.dumps(
                {
                    "error": {
                        "message": "No DEEPLX_BASE_URL provided",
                    }
                }
            ),
        )
    print(text)
    text = text.replace('\n', '\\n')
    print(text)
    body = {
        "text": text,
        "target_lang": target_lang,
    }
    if "source" in data:
        body["source_lang"] = source_lang

    try:
        req = ProxyRequest(
            deeplx_base_url, "POST", '', json.dumps(body), query_params={}
#             deeplx_base_url, "POST", headers, json.dumps(body), query_params={}
        )
        resp = await pass_through_request(http_client, req, nohttps=True, noheaders=True)
        resp = json.loads(resp.content.decode("utf-8"))
        try:
            translated_text = resp["alternatives"][0]
            print(translated_text)
            translated_text = translated_text.replace('\\nn', '\n')
            print(translated_text)
            res = {"data": {"translations": [{"translatedText": translated_text}]}}
        except TypeError:
            # res = {"error": {"message": "Failed to translate"}}
            # res = {"data": {"translations": [{"translatedText": "Failed to translate"}]}}
            res = {"data": {"translations": [{"translatedText": text}]}}

        if "source" not in raycast_data:
            res["data"]["translations"][0]["detectedSourceLanguage"] = resp[
                "source_lang"
            ].lower()

        return Response(status_code=200, content=json.dumps(res))
    except Exception as e:
        logger.error(f"DEEPLX error: {e}")
        return Response(
            status_code=500,
            content=json.dumps(
                {
                    "error": {
                        "message": "Unknown error",
                    }
                }
            ),
        )

replace('\n', '\\n')
Why do this?

because otherwise it will return  {"code":404,"message":"No text to translate}


Automated Package Upgrade

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


fix home path on windows

The "HOME" env variable is not present on Windows.
I replaced the variable with Go's method in the os package for retrieving the correct home dir for different operating systems.

Nice one - thank you!


Please approve to add our helm repo

@mlbiam The Artifact Hub is getting ready to go live. You can also list your Helm repo there. To do that, you login (or create an account), and list your repository. You can link it to a user or organization.


Add a license to the repository

I recently came across your project random-question-generator and I noticed that the repository github.com/tula3and/random-question-generator currently does not have a license specified.
Why Adding a License is Important?
As GitHub documents

Public repositories on GitHub are often used to share open source software. For your repository to truly be open source, you'll need to license it so that others are free to use, change, and distribute the software.

Would be possible to add a valid license to this repository? Probably something really OpenSource such as the Apache 2.0 license would fit really well.
Thanks!

Thank you for suggesting adding the license! I added Apache 2.0 license to this project. 👍

thanks


C++ code generator plugin: eliminate use of nullptr

Needed to make examples work under gcc-4.4

LGTM


Retain old images

I don't need to delete, rename and rehash the board image after each action. I can just save the image to the same place and then access it by its full commit URL instead of the latest default branch URL.
This would also allow me to keep a link to that image after each move (#48). I can technically do that now, but it's a PITA.

Image is located at a URL with the form https://raw.githubusercontent.com/OWNER/REPO/COMMIT_SHA/game/current/board.svg.
There's a problem with that: that URL involves the commit hash for a commit that, with #33, does not yet exist. There's no way of knowing what our next commit hash is going to be. I can't just use the current URL for the current one and log the commit hash URL, because then we'd have caching issues and no one would be able to see the current image.

Noting that images currently point to games/current. When a new game is created, that does not need to be changed. The reference by commit hash should be correct.

Confirmed - caching is an issue. I'll have to retain the image hash in the event log.

You know, I wouldn't have to both generate and retain a special hash for the board image if I just used the issue number.


Export CommentToken in token.dart

This exports CommentToken
That class may be reached when doing:
AstNode node;
CommentToken = node.beginToken.precedingComments;

Gerrit review https://dart-review.googlesource.com/c/sdk/+/163561
/cc @devoncarew @bwilkerson

I have already replied to the CL on Gerrit and am waiting for a response to the question I asked.

@bwilkerson thanks!
FWIW the type is part of the public interface https://pub.dev/documentation/analyzer/latest/dart_ast_token/Token/precedingComments.html so it leaks to the user (user can in fact access its public members), but user can't declare any variable with this type explicitly.
final /* CommentToken */ a = tok.precedingComments; // ok, static type inferred CommentToken.
final CommentToken a = tok.precedingComments; // error

I don't know if this is desirable.

As I said on the CL, the original intention was for Token to be the only class in the public interface and for all of the behavior to be available from there. The reference to CommentToken was unintentional and, imo, should be changed to Token. What I'm trying to ascertain is how big of a breaking change it would be to change the return type of precedingComments by understanding what, if any, CommentToken-specific behavior is needed by clients.

@rrousselGit thanks for the contribution, can you please respond to the review comment in https://dart-review.googlesource.com/c/sdk/+/163561 ?

Sorry, I didn't see the comment.
May I suggest a bot that synchronize comments on dart-review with github?
So that people watching the PR get notified

@rrousselGit are you still planning on working on this?

I've already answered the comments:
I don't really care about CommentToken. It just was used by a public API but not available for use
So I don't really care about whether CommentToken would be exported or whether the APIs that use it change to use Token
So we can probably close this


Add Common German Passwords Wordlist

Describe the added commits

feat(wordlist): Added Common German Passwords: Add a wordlist for common German passwords and top 1000, 10,000, 100,000 and 1,000,000 lists. The German_common-password-list.txt contains ~1.8 million passwords.
feat(docs): Added Documentation for Common German Passwords: Add wordlist files to Passwords/Common-Credentials/Language-Specific/README.md

Source
The password list is based on (more or less) public combo lists from multiple leak forums. Over the past year I got multiple German based email:password-lists based on leaked data. All lists together containing round about 2.5 million German passwords. I wrote a small helper tool to parse these lists and remove duplicates. The resulting list is added as German_common-password-list.txt.
Additional context & Purpose of pull request
Language-specific password lists are vital in pentesting, as they improve the chances of identifying weak passwords based on linguistic patterns. These lists reflect cultural and regional preferences, making security assessments more accurate. They enhance the effectiveness of password attacks by aligning with users' likely choices.

@domai-tb This Pull Request has now been merged in Seclists-Express!
Seclists-Express is a frequently-updated fork of Seclists.

Great stuff here @domai-tb ! Big thanks for putting this together!

Sorry for the delay, getting pulled in many directions.


Add initial techno-economic variables

Dear common-definitions-techno-economic group,]
I am excited to be part of techno-economic team in common-definitions project. This PR is an initial implementation about techno-economic variables.
I have solved most of the problems from https://github.com/IAMconsortium/common-definitions/pull/71, like renaming "OM Cost" to "Operation and Maintenance Cost”,  renaming "US$2010" to "USD_2010" .
Next, we will make further improvement about the stucture of variables "OM Cost|Fixed|...", discussing with the team!

@danielhuppmann @phackstock can you take a look and see this inital commit can be merged?

Thank you for coming up with a proposal for the O&M cost variable. I will reimplement this using tag lists to have shorter files, will ping you in a bit.


[SPARK-41346][CONNECT][TESTS][FOLLOWUP] Fix test_connect_function to import PandasOnSparkTestCase properly

What changes were proposed in this pull request?
This PR aims to fix test_connect_function to import PandasOnSparkTestCase properly. If we handle import properly, the test cases are ignored properly because should_test_connect assumes have_pandas
https://github.com/apache/spark/blob/97976a5cc915597fd2606602d18c52c075a03bf6/python/pyspark/testing/connectutils.py#L49
Why are the changes needed?
SPARK-41346 imported PandasOnSparkTestCase outside of  if have_pandas:.
https://github.com/apache/spark/blob/97976a5cc915597fd2606602d18c52c075a03bf6/python/pyspark/sql/tests/connect/test_connect_function.py#L25-L29
Does this PR introduce any user-facing change?
No.
How was this patch tested?
BEFORE
$ python/run-tests --testnames pyspark.sql.tests.connect.test_connect_function
...
ModuleNotFoundError: No module named 'pandas'

AFTER
$ python/run-tests --testnames pyspark.sql.tests.connect.test_connect_function
...
Skipped tests in pyspark.sql.tests.connect.test_connect_function with python3.9:
      test_aggregation_functions (pyspark.sql.tests.connect.test_connect_function.SparkConnectFunctionTests) ... skip (0.004s)
      test_math_functions (pyspark.sql.tests.connect.test_connect_function.SparkConnectFunctionTests) ... skip (0.004s)
      test_normal_functions (pyspark.sql.tests.connect.test_connect_function.SparkConnectFunctionTests) ... skip (0.002s)
      test_sort_with_nulls_order (pyspark.sql.tests.connect.test_connect_function.SparkConnectFunctionTests) ... skip (0.001s)
      test_sorting_functions_with_column (pyspark.sql.tests.connect.test_connect_function.SparkConnectFunctionTests) ... skip (0.001s)

cc @zhengruifeng , @HyukjinKwon

Thank you, @HyukjinKwon !

This is a single test file, test_connect_function.py, change and verified manually. Merged to master.

Argh, this actually breaks the Python linter. let me make a quick followup


translate canary rollout for container service

https://kubevela.io/zh/docs/end-user/traits/rollout

I would try to translate this doc. assign to me:)

@wonderflow need for your preview https://github.com/kubevela/kubevela.io/pull/848

committed and merged, can close now.

thanks for your contribution！ @oudi


Icons not appearing.

You promised me icons, I don't have any!

can you be more specific please?

@Athax you will get icons displayed if you install icons plugin... That's simple :)

Hehe it was qoute, tried to ask in a funny way. It's beacuse in your gif sample pictures there's icons on your tap how do i achive them?

Found the plugin thanks @thco

I me installed this : https://github.com/ivogabe/Brackets-Icons


Examples for redbpf

It would be great if we can have some example programs for using redbpf implementing some elementary ones from https://github.com/iovisor/bcc/tree/master/examples

A full-blown daemon using redbpf is ingraind.
The bpf folder in the repo has the BPF code, while the rust-side is at src/grains.
In reality, there's quite a lot of plumbing that's necessary to make redbpf useful, because in contrast to BCC the design is for pre-compiling the BPF code.
After the plumbing is done, a simple probe's implementation is fairly minimal.
I think, depending on what you're aiming to do, using ingraind as a framework might be a better call than building on the low-level lib that is redbpf.

Resolved by #48


feat(SelectDropdown): onInputChange

Overview

PR Checklist

[ ] Related to designs:
[ ] Related to JIRA ticket: ABC-123
[ ] I have run this code to verify it works
[ ] This PR includes unit tests for the code change

📬Published Alpha Packages:@codecademy/gamut-kit@0.5.39-alpha.7027f6.0@codecademy/gamut-labs@19.23.2-alpha.7027f6.0@codecademy/gamut@36.4.6-alpha.7027f6.0<EMAIL_ADDRESS>
🚀 Styleguide deploy preview ready!https://611e9afebf67343d60c1dddd--gamut-preview.netlify.appDeploy Logs

📬Published Alpha Packages:@codecademy/gamut-kit@0.5.39-alpha.f4cb1b.0@codecademy/gamut-labs@19.23.2-alpha.f4cb1b.0@codecademy/gamut@36.4.6-alpha.f4cb1b.0<EMAIL_ADDRESS>
🚀 Styleguide deploy preview ready!https://611ea23a9d613439433fd3d7--gamut-preview.netlify.appDeploy Logs

📬Published Alpha Packages:@codecademy/gamut-kit@0.5.40-alpha.868816.0@codecademy/gamut-labs@19.23.3-alpha.868816.0@codecademy/gamut@36.4.7-alpha.868816.0<EMAIL_ADDRESS>
🚀 Styleguide deploy preview ready!https://611ea56746ba0c3eb887929a--gamut-preview.netlify.appDeploy Logs

📬Published Alpha Packages:@codecademy/gamut-kit@0.5.41-alpha.05087b.0@codecademy/gamut-labs@19.23.4-alpha.05087b.0@codecademy/gamut@36.4.8-alpha.05087b.0<EMAIL_ADDRESS>
🚀 Styleguide deploy preview ready!https://611ee157df8dc87c6610cdea--gamut-preview.netlify.appDeploy Logs

📬Published Alpha Packages:@codecademy/gamut-kit@0.5.41-alpha.82afc2.0@codecademy/gamut-labs@19.23.4-alpha.82afc2.0@codecademy/gamut@36.5.0-alpha.82afc2.0<EMAIL_ADDRESS>
🚀 Styleguide deploy preview ready!https://611fa92bd0a46e59b4790b2b--gamut-preview.netlify.appDeploy Logs

📬Published Alpha Packages:@codecademy/gamut-kit@0.5.42-alpha.7cf751.0@codecademy/gamut-labs@19.23.5-alpha.7cf751.0@codecademy/gamut@36.5.1-alpha.7cf751.0<EMAIL_ADDRESS>
🚀 Styleguide deploy preview ready!https://611fabc346ba0c6884878c62--gamut-preview.netlify.appDeploy Logs


add TDE hook in pg_checksums

pg_checksums will read and write file directly. run pg_checksums on TDE enabled server will emit a checksum corrupted error. and if with --enable pg_checksums will write data to file, this will cause a checksum corruption.
Add hook in pg_checksums, so the TDE extension can use this hook to encrypt or decrypt data.

@higuoxing I cannot request a review using github...

I am removing --dlopen and --installdir . To keep minimum modification in PostgreSQL


[DDC-2763] Inheritance.  CTI & STI. Improve lazy load associated entity, when target entity in association mapping is not last leaf in class hierarchy.
Issue fixed by adding class "ProxyHydrator" and new fetch mode for associations "FETCH_USE_PROXY".
Users need to mark the associations with "fetch=USE_PROXY" to trigger new lazy loading behaviour.
This PR is not intended to be final. It still lacks of tests.

+1, waiting for that!

It would be a most welcome addiction! +1

@Orcamus can you look at this PR?


docs: fix typo in ConnectButton example

In the responsive example section for ConnectButton, "icon" is not a valid enum value for accountStatus. 🙂

Thank you!


EncryptionHelper TryDecryptAES256 throws exception

When trying to decrypt a string, the method throws the following exception "The input data is not a complete block." and returns the same input given (tested under .NET Core 2.0).
I noticed this when my logs where being flooded with errors on User registration.
Log Entry Example:
{"Level": "Error", "Timestamp": "2018-01-23T12:13:22.6762643+00:00", "Properties": {"ActionId": "28bbddb1-ff40-4880-af22-2a7ef7e6dabc", "ThreadId": 9, "RequestId": "0HLB1K4845IA8:00000001", "ActionName": "Fmp.Web.Controllers.API.AccountController.Register (Fmp.Web)", "Application": "FMP", "MachineName": "RD00155D8A52DB", "ProcessName": "dotnet", "RequestPath": "/api/v1.0/account/register", "SourceContext": "Identity.Dapper.Cryptography.EncryptionHelper"}, "MessageTemplate": "Specified padding mode is not valid for this algorithm."}
I added a Test so you can see the behavior and a possible Fix using the same AES256 Algorithm.
Also, notice that I am not doing Base64 on the Key/IV. You are welcome to revert this.

@terzano sorry for the late.
I had no time to analyse your pull request.
The current method by design return the same input on exception because if you don't encrypt it'll throw exception. What i could do is remove the log of the exception and you'll no longer be flooded by these exceptions.

@grandchamp The issue is not what happens when the exceptions is thrown (I left that part the same). The problem is that the Decryption method is not working, it always throws an exception.

I thought i had some tests for it.
I'll check it out.

@terzano i've added a simple review to your commit.
Anyway, can you update the repo README and the sample to consider your changes?

@grandchamp I modified the vector encoding as suggested.
Keep in mind that there is a breaking change. The Key & IV are not Base64 encoded. I can restore that by adding to maintain full compatibility.
Convert.FromBase64String(_aesKeys.Value.Key)
Convert.FromBase64String(_aesKeys.Value.IV)

Well... I don't see any problems on your implementation with a hex and plain text.
It's your choice, but if we maintain yours, i'll ask you to update the README and the sample, as i mentioned before.

That sounds reasonable. I will try to find some time to make the update today.

Decided to maintain backwards compatibility, less friction for your user base. Made the proper changes and updated the docs and test project as suggested.

Very good job!
Thanks!


Flutter google maps drag marker

Hey flutter team,
Any way to drag a marker around the map and get the location from it ?

Please consider asking support questions in one of the other channels listed at http://flutter.io/support .

Use this package.
https://github.com/johnpryan/flutter_map

I need it too.

No, it doesn't,
but you can use community solutions until it is decided if it will be added and until it lands.

Is there any community solutions for this ?

No, it doesn't,
but you can use community solutions until it is decided if it will be added and until it lands.

Is there any community solutions for this ?

This is the solution I am using.
I am using Markers and setting the draggable to true.
_markers.add(
        Marker(
          markerId: MarkerId('marker_2'),
          position: latLng,
          draggable: true,
          icon: _searchMarkerIcon,
         ),
 );

Then, I am using onCameraMove Event in GoogleMap()
GoogleMap(
         mapType: MapType.normal,
         initialCameraPosition: _kCurrentLocation,
         onMapCreated: _onMapCreated,
         markers: _markers,
         myLocationEnabled: true,
         myLocationButtonEnabled: true,
         onCameraMove: ((_position) => _updatePosition(_position)),
        ),

and my _updatePosition() method handles the new lat lng. and set that new lat lng to my marker object.
void _updatePosition(CameraPosition _position) {
    print(
        'inside updatePosition ${_position.target.latitude} ${_position.target.longitude}');
    Marker marker = _markers.firstWhere(
        (p) => p.markerId == MarkerId('marker_2'),
        orElse: () => null);

    _markers.remove(marker);
    _markers.add(
      Marker(
        markerId: MarkerId('marker_2'),
        position: LatLng(_position.target.latitude, _position.target.longitude),
        draggable: true,
        icon: _searchMarkerIcon,
      ),
    );
    setState(() {});
  }

and this is the output.

Thanks @jawand .

Nicely works :)

It didn't work for my use-case .. whenever I drag the marker to a certain place .. the marker goes back to its old place. Any theories about this behaviour?

It didn't work for my use-case .. whenever I drag the marker to a certain place .. the marker goes back to its old place. Any theories about this behaviour?

the same problem

It didn't work for my use-case .. whenever I drag the marker to a certain place .. the marker goes back to its old place. Any theories about this behaviour?

I have used plugin version google_maps_flutter: ^0.5.19+2
please let me know which version you are using.

@jawand I have tried the same approach but it seems to little bit laggy. I mean you can feel the delay while u are dragging the map. Also setting draggable to true or false hasn't changed anything. Do you know what does it change?

@jawand I have tried the same approach but it seems to be little bit laggy. I mean you can feel the delay while u are dragging the map. Also setting draggable to true or false hasn't changed anything. Do you know what does it change?
UPDATE: I have ended up with a slightly different approach. Actually all we need is the center position coordinates which we can get by onCameraMove callback. Marker is only there for to show the center position with precision. Instead of doing it with a marker, I used the Stack widget and positioned an icon on the center. I added some Offset for precision. It does the same thing at the end but the latency is gone and it is probably a bit more efficient than updating the Set constantly with drag.
Stack(
        children: <Widget>[
          GoogleMap(
            mapType: MapType.normal,
            myLocationEnabled: true,
            myLocationButtonEnabled: true,
            initialCameraPosition: initialPosition,
            onMapCreated: _onMapCreated,
            onCameraMove: (cameraPosition) {
              // do something with the current camera position
            },
          ),
          Align(alignment: Alignment.center,child: Transform.translate(offset: Offset(0,-34),child: Icon(Icons.location_on,size: 44,)))
        ],
      ),

I know my approach is a little bit sluggish. I will try your approach.

I am trying to get why they added the draggable parameter, it's really useless.

I am trying to get why they added the draggable parameter, it's really useless.

No, it doesn't,
but you can use community solutions until it is decided if it will be added and until it lands.

Is there any community solutions for this ?

plugin version used google_maps_flutter: ^0.5.19+2
This is the solution I am using.
I am using Markers and setting the draggable to true.
_markers.add(
        Marker(
          markerId: MarkerId('marker_2'),
          position: latLng,
          draggable: true,
          icon: _searchMarkerIcon,
         ),
 );

Then, I am using onCameraMove Event in GoogleMap()
GoogleMap(
         mapType: MapType.normal,
         initialCameraPosition: _kCurrentLocation,
         onMapCreated: _onMapCreated,
         markers: _markers,
         myLocationEnabled: true,
         myLocationButtonEnabled: true,
         onCameraMove: ((_position) => _updatePosition(_position)),
        ),

and my _updatePosition() method handles the new lat lng. and set that new lat lng to my marker object.
void _updatePosition(CameraPosition _position) {
    print(
        'inside updatePosition ${_position.target.latitude} ${_position.target.longitude}');
    Marker marker = _markers.firstWhere(
        (p) => p.markerId == MarkerId('marker_2'),
        orElse: () => null);

    _markers.remove(marker);
    _markers.add(
      Marker(
        markerId: MarkerId('marker_2'),
        position: LatLng(_position.target.latitude, _position.target.longitude),
        draggable: true,
        icon: _searchMarkerIcon,
      ),
    );
    setState(() {});
  }

and this is the output.

Mass. Thanks

Mass and Thanks

This seems to be fixed in v  google_maps_flutter: ^0.5.11 . just add onDragEnd method ( more here )

This seems to be fixed in v google_maps_flutter: ^0.5.11 . just add onDragEnd method ( more here )
I have trued this. I get the lat long but, not able to perform setState() inside.

@jawand I have tried the same approach but it seems to be little bit laggy. I mean you can feel the delay while u are dragging the map. Also setting draggable to true or false hasn't changed anything. Do you know what does it change?
UPDATE: I have ended up with a slightly different approach. Actually all we need is the center position coordinates which we can get by onCameraMove callback. Marker is only there for to show the center position with precision. Instead of doing it with a marker, I used the Stack widget and positioned an icon on the center. I added some Offset for precision. It does the same thing at the end but the latency is gone and it is probably a bit more efficient than updating the Set constantly with drag.
Stack(
        children: <Widget>[
          GoogleMap(
            mapType: MapType.normal,
            myLocationEnabled: true,
            myLocationButtonEnabled: true,
            initialCameraPosition: initialPosition,
            onMapCreated: _onMapCreated,
            onCameraMove: (cameraPosition) {
              // do something with the current camera position
            },
          ),
          Align(alignment: Alignment.center,child: Transform.translate(offset: Offset(0,-34),child: Icon(Icons.location_on,size: 44,)))
        ],
      ),

Hey,
How to get the center coordinates and also usig onCameraMove causes lot latency as it gets each latlong we drag by. Isn't there any solution to get lat long once we set marker

Thanks @jawand for that information but the method is very slow.
What I have done is, since we are receiving the centre of the map in onCamerMove Method, so put a marker image in the centre of the map as Positioned when the onCameraMoveStarted function is triggered and then you can remove the image by using the onCameraIdle Method.
I hope it makes sense :P
Thanks Everyone for your input here :)

Thanks @jawand for that information but the method is very slow.
What I have done is, since we are receiving the centre of the map in onCamerMove Method, so put a marker image in the centre of the map as Positioned when the onCameraMoveStarted function is triggered and then you can remove the image by using the onCameraIdle Method.
I hope it makes sense :P
Thanks Everyone for your input here :)

How can we get center coordinates?

It only works on android on iOS if it has 3 pins, just move one, the others no longer move after moving the first one. Do you know why?

It only works on android on iOS if it has 3 pins, just move one, the others no longer move after moving the first one. Do you know why?

This PR https://github.com/flutter/plugins/pull/2653 will solve the issue. It needs some love from the community.

I had the same problem, the marker has a onDragEnd function that has new coordinates. Simply use it to get the new coordinates

Hi @momoDragon
Does the onDragEnd works for you? You can use it to get the location where the marker was dragged.

Without additional information, we are unfortunately not sure how to resolve this issue.
We are therefore reluctantly going to close this bug for now.
Please don't hesitate to comment on the bug if you have any more information for us; we will reopen it right away!
Thanks for your contribution.
Could everyone who still has this problem please file a new issue with the exact description of what happens, logs, and the output of flutter doctor -v.
All system setups can be slightly different, so it's always better to open new issues and reference related issues.

Reopening this issue as PR is still not yet merged and there are similar requests being filed


Update NuGet Config to pull from DevOps feed

Fixes - https://github.com/Azure/autorest.csharp/pull/823#discussion_r441874276

I think the feature we needed is in a released version of azure.core now. Can you update https://github.com/azure/autorest.csharp/blob/279e810cd54ed877632f99a893ecbbb3c00fb320/test/AutoRest.TestServer.Tests/AutoRest.TestServer.Tests.csproj#L11 to 1.2.2 and drop this feed.


Issue in running miReact

Hi!
First of all thank you for this tool! I am not an expert but I would like to use it on my data (skin human keratynocites) and I am not able to do it...
I downloaded the motif probability files from Synapse and I put them in the motif.models folder
Then I prepared my  own expression data set (13307 genes, 1614 cells) and I put it in the data folder
So I used these commands to generate the output file:
- source ("~/miReact/code/mireact.R")
- efile <- "~/miReact/data/MeP_mireact_exp.rds"
- mireact(exp=efile, species="hs", motifs=7, seq.type="utr3", out.file= "~/miReact/data/mirnaActivity_MeP_K82.rds", out.meonly=T, install.dir = "~/miReact")

After running them, I have this message on RStudio:
creating directory:  /home/grazia/miReact/o2022-06-2212:20:09
[1] "Follow progress in /home/grazia/miReact/o2022-06-2212:20:09/Rscript-[jobid].out"
So no error in R but the whole process fails because I obtain only a new folder with 3 files
- e.RData (I obtain this error when I try to open it:
  Error: bad restore file magic  number (file may be corrupted) -- no data loaded)
- runparameters.RData
- sbatch.script

and no Rscript-[processID].out or my output file in the data folder.
Do you have any suggestion?
Thank you and have a good day!

Hi Grazia,
Can you try to run
e <- readRDS("/path/to/e.Rdata")
To see that it loads ok.
Also, can you post the content of the sbatch.script

Hi!
So I have tried to open the e.Rdata file and it contains the exact same information of the starting efile (so a matrix of 13,307 entries, 1614 total column containing the same values of the efile).
And this is the content of the sbatch.script:
#!/bin/sh
#SBATCH --nodes 1
#SBATCH -c 1
#SBATCH --time 48:00:00
#SBATCH --mem 128g
#SBATCH --job-name Rscript
[ $SLURM_PROCID -ne 0 ] && exit 0
cd  /home/grazia/miReact/o2022-06-2212:20:09
Rscript --vanilla ../code/wrapper3.R

Hi again,
We need to locate the output logs in order to make progress. This may be setup differently on different systems, and thus the message:
'Follow progress in /home/grazia/miReact/o2022-06-2212:20:09/Rscript-[jobid].out'
may not apply to your system. Maybe these outputs ar put somewhere special, e.g. your home?

Hi!
I will try as soon as possible (probably next week) and I will give you updates, thank you!

Hi!
I have not tried your suggestion so far but I was also wondering if it could be possible to obtain a miReact version in PBS instead of SLURM. Thank you again!

Hi Grazia,
I am not familiar with PBS, but the slurm scripts are simply written from R via the sink() command and started with the system() command, so I think it would be trivial to change the R scripts accordingly.

Hi again!
I found out that the problem is in our slurm configuration so we would like to try to run it on a local computer (our dataset has about 1000 cells, so it should not be so big...). I have tried to follow your suggestion here
https://github.com/muhligs/miReact/issues/4 and, as LGGray I obtain a mailmessage.txt with this text
exp ~/Desktop/miReact/data/MeP_mireact_exp.rds
species hs
seq.type utr3
out.file ~/Desktop/miReact/data/mirnaActivity_MeP_K82.rds
out.meonly TRUE
verbose FALSE
install.dir ~/Desktop/miReact
mail NULL
memory 128g
wt 48:00:00
tarbaserun FALSE
wd /Users/admin/Desktop/miReact/o2022-06-3010:07:36
runparameters$motifs: 7-mers

and I have tried to follow your suggestion about wrapper3 (I don't see any prints to the console) putting sink(statments.txt) before the line
cat("\nLoading expression data...\n")
and sink() after the line
sco<- list(exp=e)
and obtaining a statments.txt file with this content
Loading expression data...
2022-06-30 10:15:25

I don't understand the problem...
Thanks!

It shold continue immideately with writing:
running addMotifs...
Did that not happen?

Yes, I am sorry I corrected the previous comment because it was uncomplete

It looks like it has not finished yet. It should either produce error statements or write additional statements from the R scripts in the sink file.
You can look in addMotifs.R and subsequently addMotifExpression.R to see what you can expect to be written in the sink file.

Hi!
In the end I was able to work it out locally, thanks for your suggestion. The problem was in one of the files downloaded from Synapse but after re-downloading it everything was ok.
Now, have the file with miRNA activities but I have another question: what does it mean to obtain a negative activity value? It is not clear for me from the paper...thank you again for your support!
Grazia

Hi Garzia,
You should regard the activity values as a relative measure, so negative values means lower activity than in non-negative samples. If you se a subset of samples with high negative values and the majority of samples having values around zero, it could be because this miRNA in general has high activity in the 'zero' samples and low activity in the sub set. Think of the zero point sort of as the median activity and values high or low as 'higher' or 'lower' than that.

Thanks!
Now I'm in the final step: how to connect the sequence that I obtain with a specific miRNA?
I see that for miR-122-5p in Mus Musculus you obtain
ACACTCC
while the mature miRNA sequence, downloaded from TarBase, is
>hsa-miR-122-5p MIMAT0000421 Homo sapiens miR-122-5p UGGAGUGUGACAAUGGUGUUUG
so for every output sequence I have to do the reverse complement (UGUGAGG) and then I have to read it from right to left (GGAGUGU) and it should be equal to the sequence download from miRBase, starting from position n2? Is that correct?
Thanks again,
Grazia

Sorry for the late reply.
Yeah, sort of. I would say: Take bases from n2 to n8. 'Translate' to DNA (GGAGTGT). Reverse-complement (ACACTCC).
Since the target UTRs are all DNA sequences, that is how the enriched 7-mer would look like in the matrix.
Best,
Morten

I am closing the issue.


Error message during initilization if SPKI Cache file doesn't exist

iOS framework TrustKit v1.4.1.
The following error message is shown if you start an app for the first time or if the SPKI cache file doesn't exist (yet / no more):

*** -[NSKeyedUnarchiver initForReadingWithData:]: data is NULL

The cause for this problem is inside the getSpkiCacheFromFileSystem function:
NSData *serializedSpkiCache = [NSData dataWithContentsOfFile:spkiCachePath];
spkiCache = [NSKeyedUnarchiver unarchiveObjectWithData:serializedSpkiCache];

dataWithContentsOfFile will return a nil value if the file doesn't exist or cannot be read. This case should be handled by the function and the call to the NSKeyedUnarchiver should not be made.

Hello,
This should only display an error in the logs, which is expected behavior, or are you getting a crash?
Thanks!

This should only display an error in the logs, which is expected behavior, or are you getting a crash?

I'm a bit confused, not handling this error is "expected behavior"? Calling unarchiveObjectWithData: with a nil parameter is an invalid call. Apple currently only shows an error message in the console logs but according to the official documentation it actually should raise an exception. Apple could change the current behavior at any point and raise an exception instead.
What's the harm in checking the value for nil as you should always do if you use [NSData dataWithContentsOfFile:]?
NSData *serializedSpkiCache = [NSData dataWithContentsOfFile:spkiCachePath];
if (serializedSpkiCache) {
    spkiCache = [NSKeyedUnarchiver unarchiveObjectWithData:serializedSpkiCache];
}

The documentation is pretty clear about this (which I didn't know) so that's fair - will fix it.

Looks like you already did, thanks again!

Looks like you already did, thanks again!
You're welcome :)


Fixed partial delete issues on compactor; Added Upload/Delete tests.

Fixes https://github.com/thanos-io/thanos/issues/1331
Problem that we are fixing is explained in the linked issue.
Also the problem will be fully fixed once this is also done: https://github.com/thanos-io/thanos/pull/1394 This is because although major parts of Thanos for syncing blocks is resilient to partial uploads, there are some points when we don't handle it properly (crash e.g compactor)
cc @lx223 @metalmatze
Signed-off-by: Bartek Plotka<EMAIL_ADDRESS>

Is there any race between the sidecar uploading (no meta.json yet) and the compactor deleting?

Is there any race between the sidecar uploading (no meta.json yet) and the compactor deleting?

Yea, detecting if the block is partially uploaded (malformed) vs upload is in progress is in current model quite tricky. We do that currently by just waiting for some time called consistencyDelay (configurable on compactor:https://github.com/thanos-io/thanos/blob/c83332f6aac03df3c9eeb5193c3d688da0f980f1/cmd/thanos/compact.go#L86). For now the default is 30m.
This has proven to be ok for all users (as per: no known issue because of that). Once upload takes more than 30 minutes, you might want to increase such consistency delay. This delay is also useful for object storages that are eventually consistent.
Good question is: can we warn/document these details about uploads longer than 30 minutes ):

What about a simple locking method of renaming meta.json to meta.json.deleted, then cleanup the index/chunks, then finally delete the meta.json.deleted file.

I merged already, but happy to discuss alternative ideas @SuperQ

What about a simple locking method of renaming meta.json to meta.json.deleted, then cleanup the index/chunks, then finally delete the meta.json.deleted file.

Do you mean for deleting process? Not sure how is that better (: We want to get rid meta.json ASAP. Once we remove meta.json, block is nicely handled as malformed even if deletion would be disrupted at any point (e.g by pod crash/restart). In fact we would be fine by just deleting meta.json and allow compactor to assume as malformed block in next iteration.

We also discussed some alternatives here: https://thanos.io/proposals/201901-read-write-operations-bucket.md/

There is still some slight chance to restart compactor in a certain moment that will cause overlaps (in the worst case). For that to work we need to implement action items for design doc: https://thanos.io/proposals/201901-read-write-operations-bucket.md/


Attempts to run bat file on linux

The doc says it's multiplatform Linux/Windows/MAc but it seems only was tested on Windows. When I try:
START mobfs
it fails with:
  File "CUT/pentest/mobile/android/QuadraInspect/config/mobfs.py", line 7, in mobfs
    run('{}/run.bat <IP_ADDRESS>:5000'.format(tdir))
  File "/usr/lib/python3.10/subprocess.py", line 501, in run
    with Popen(*popenargs, **kwargs) as process:
  File "/usr/lib/python3.10/subprocess.py", line 969, in __init__
    self._execute_child(args, executable, preexec_fn, close_fds,
  File "/usr/lib/python3.10/subprocess.py", line 1845, in _execute_child
    raise child_exception_type(errno_num, err_msg, err_filename)
FileNotFoundError: [Errno 2] No such file or directory: '/CUT/pentest/mobile/android/QuadraInspect/tools/mobfs//run.bat <IP_ADDRESS>:5000'

Will correct it

Try replacing the code in the config\mobfs.py file with this:
import platform
from rich.pretty import pprint
from subprocess import run

def mobfs(rdir):
    osp = platform.system()
    match osp:
        case 'Darwin':
            pre = ""
            cmd = "run.sh"
        case 'Linux':
            pre = ""
            cmd = "run.sh"
        case 'Windows':
            pre = "bash"
            cmd = "run.bat"
    try:
        tdir = '{}/tools/mobfs/'.format(rdir)
        run('{a} {b}/{c}'.format(a=pre, b=tdir, c=cmd))
        pass
    except KeyboardInterrupt:
        pprint("Ending process ...")

chikkachun@chikdeMBP QuadraInspect-1.1 % ./gradlew build
zsh: no such file or directory: ./gradlew
chikkachun@chikdeMBP QuadraInspect-1.1 % gradle build
FAILURE: Build failed with an exception.

What went wrong:
Project directory '/Users/chikkachun/Documents/QuadraInspect-1.1' is not part of the build defined by settings file '/Users/chikkachun/settings.gradle'. If this is an unrelated build, it must have its own settings file.

Try:

Run with --stacktrace option to get the stack trace.
Run with --info or --debug option to get more log output.
Run with --scan to get full insights.

Get more help at https://help.gradle.org

BUILD FAILED in 809ms
chikkachun@chikdeMBP QuadraInspect-1.1 % build.gradle
zsh: command not found: build.gradle
chikkachun@chikdeMBP QuadraInspect-1.1 % python3 main.py
Traceback (most recent call last):
File "/Users/chikkachun/Documents/QuadraInspect-1.1/main.py", line 3, in
from pyfiglet import Figlet
ModuleNotFoundError: No module named 'pyfiglet'
chikkachun@chikdeMBP QuadraInspect-1.1 % START install_tools
zsh: command not found: START
chikkachun@chikdeMBP QuadraInspect-1.1 % python3.10 main.py
Traceback (most recent call last):
File "/Users/chikkachun/Documents/QuadraInspect-1.1/main.py", line 3, in
from pyfiglet import Figlet
ModuleNotFoundError: No module named 'pyfiglet'
chikkachun@chikdeMBP QuadraInspect-1.1 % pip install -r requirements.txt && python3 main.py
WARNING: Ignoring invalid distribution -inja2 (/Library/Frameworks/Python.framework/Versions/3.6/lib/python3.6/site-packages)
WARNING: Ignoring invalid distribution - (/Library/Frameworks/Python.framework/Versions/3.6/lib/python3.6/site-packages)
WARNING: Ignoring invalid distribution -inja2 (/Library/Frameworks/Python.framework/Versions/3.6/lib/python3.6/site-packages)
WARNING: Ignoring invalid distribution - (/Library/Frameworks/Python.framework/Versions/3.6/lib/python3.6/site-packages)
Ignoring waitress: markers 'platform_system == "Windows"' don't match your environment
Collecting apkleaks==2.6.1
Downloading apkleaks-2.6.1-py3-none-any.whl (14 kB)
ERROR: Could not find a version that satisfies the requirement apkutil==0.1.7 (from versions: none)
ERROR: No matching distribution found for apkutil==0.1.7
WARNING: Ignoring invalid distribution -inja2 (/Library/Frameworks/Python.framework/Versions/3.6/lib/python3.6/site-packages)
WARNING: Ignoring invalid distribution - (/Library/Frameworks/Python.framework/Versions/3.6/lib/python3.6/site-packages)
WARNING: Ignoring invalid distribution -inja2 (/Library/Frameworks/Python.framework/Versions/3.6/lib/python3.6/site-packages)
WARNING: Ignoring invalid distribution - (/Library/Frameworks/Python.framework/Versions/3.6/lib/python3.6/site-packages)
chikkachun@chikdeMBP QuadraInspect-1.1 % pip3.10 install -r requirements.txt && python3.10 main.py
Ignoring waitress: markers 'platform_system == "Windows"' don't match your environment
Collecting apkleaks==2.6.1
Using cached apkleaks-2.6.1-py3-none-any.whl (14 kB)
ERROR: Could not find a version that satisfies the requirement apkutil==0.1.7 (from versions: none)
ERROR: No matching distribution found for apkutil==0.1.7
[notice] A new release of pip is available: 23.0 -> 23.1
[notice] To update, run<EMAIL_ADDRESS>-m pip install --upgrade pip
chikkachun@chikdeMBP QuadraInspect-1.1 % QuadraInspect Main>> : START install_tools
zsh: command not found: QuadraInspect
chikkachun@chikdeMBP QuadraInspect-1.1 %

how to run???

delete the main folder the tool is downloaded in and redownload it. I have done all the necessary changes

how to run???

and also i guess you are running python 3.10


Bug report: error during dev

I'm running into the following error while loading the dev server:

The dev server actually starts without any errors, as shown in this screenshot:

This same error is occurring with both the example folder in this repo as well as a separate project that I tried setting up manually.  Here is a link to the repo: https://github.com/marshallswain/my-vitedge-project
After cloning the repo, just run npm i then npm run dev, then open the dev server in the browser.

This appears to be a related issue: https://github.com/vuejs/vue-next/issues/2244.

@marshallswain Hello! There are 2 problems:

You need to call the plugin because it's a function since v0.6.0: vitedgePlugin() (note the parenthesis). The repo example is updated but I need to update the docs.
The previous fix will make it work with<EMAIL_ADDRESS>However, beta.69 has changed something internally related to plugin aliases. I will update that soon.

After all of that works, you'll need to create functions/props/default.ts or the dev-server will complain.
Btw, any feedback about usage/DX is welcome!

Support for Vite beta.69 is released in 0.6.1 👍

Thank you!  I'll try it out.

I found that after using JSX syntax, it will use createComponentInstance, and then it will report an error. Is there a good solution


symlinks in themes are not allowed

Issue Summary
Using the generic royalfig template. See here: https://github.com/royalfig/ghost-theme-starter

Steps to Reproduce
# Make sure to run 5.55.2
ghost install 5.55.2 local
# link theme
ln -s ./ ../ghost/content/themes/test-theme

Ghost Version
5.55.2
Node.js Version
v18.17.0
How did you install Ghost?
npm install ghost-cli -g
Database type
SQLite3
Browser & OS version
Chrome Edge
Relevant log / error output
See screenshot

Code of Conduct

[X] I agree to be friendly and polite to people in this repository

### Tasks

I think this is due to your ln command. I have a feeling you need to set an absolute path as the source.


[DC-3052]  Account for Skips and whitespace in clean_ppi_numeric_fields_using_parameters.py

moved to bg/dc_3052_a


Mage Sim hit calculation seems to be off? It was perfectly fine until I switched some new pieces in and now something's wrong.

I'm using the Mage Sim https://wowsims.github.io/wotlk/mage/ without any issues until recently. Had a gear upgrade, put in the heroic Reign and heroic tier gloves and now the hit calculation seems to be incorrect. I've tried switching out other gear pieces with it, checked the settings, the enchants, gems, and for some reason the calculation is incorrect. I've never had this issue before, any time I've used the Mage Sim it's always shown the correct % but after this gear change, it's wrong. On my character I'm at 401 hit but on the Sim it has me at 480. Even after calculating it all out, each item individually... the hit is still wrong lol. Any idea why this is?

@bobstaman wowsims includes the 3% spell hit debuff which amounts to 79 hit


SOLR-16281: review boxing unboxing

https://issues.apache.org/jira/browse/SOLR-16281
Description
boxing/unboxing has been improved since Java 5!
Solution
Apply suggestiosn!
Tests
Please describe the tests you've developed or run to confirm this patch implements the feature or solves the problem.
Checklist
Please review the following and check all that apply:

[ ] I have reviewed the guidelines for How to Contribute and my code conforms to the standards described there to the best of my ability.
[ ] I have created a Jira issue and added the issue ID to my pull request title.
[ ] I have given Solr maintainers access to contribute to my PR branch. (optional but recommended)
[ ] I have developed this patch against the main branch.
[ ] I have run ./gradlew check.
[ ] I have added tests for my changes.
[ ] I have added documentation for the Reference Guide

Do you think this is ready now for merging @cpoerschke ??


location sharing in skype bot?

I wonder why is it not possible ? Is it due to some kind of a privacy concern ? I mean I think it is absolutely necessary for a bot to get user's location to serve him in a much much better way.
Thanks,
Palash

This is a popular request, but is not currently in the Skype channel.  I know this is on their feature request list for the future however.

This is definitely something we want to support. Please could you expand on the use case you have in mind as that helps to identify the best solution.

sure. as me move from installing multiple apps to conversational bots, having user's location by their consent is a key thing for bots to function properly. I want my bot to order food, cab, medical help etc. its quick if bot already has my location.
It can be added as a permission in the Get Started page. Just like apps. If user agrees, bots have their location. Otherwise not.

Thanks that's helpful - for these use cases you need location on request rather than continuous location.

I second the need for this feature ("location on request"). Our use case: We have a bot which alerts a user for nearby train delays. The monitoring is actively requested by the user.

I just added Cortana in Skype and it prompted me for my permission to use my location.
How do we implement this feature for our own bots?

If you're talking about accessing user contextual info such as location, addresses, or preferences you can find more information on this under these two links: Cortana User Profile & Contextual Information Reference & User Profile & Contextual Information
In the future (and if the above information hasn't answered your question) please file a new issue with your details and question as this issue has been closed.


Lesson about custom types and @spec

Probably basic, but I'm not sure.
Short summary:

the @spec and how to define "interfaces"
Creating custom types

Why we need it - tuples are too generic
@type and how to use it
short exercise - create custom type of email that will be keyword of [{:name NAME}, {:domain, DOMAIN}] with set of functions like toEmail and fromEmail.

@doomspork @pjhampton any suggestions?

I've wanted to do a typespec lesson for awhile!  I think this is a good idea.

Yep. I also love this idea. Do you want me to write this lesson or do you want to take it @Koziolek?

I'm writing this. Should be ready to review in next few days.

Awesome @Koziolek!  Looking forward to it.

In https://github.com/doomspork/elixir-school/pull/564
@doomspork, @nscyclone  guys, could you take a look and put your comments.

Thank you @Koziolek, I will look today 👍

I like this @Koziolek. Once we firm up the semantics, I will help you fine tune the English. Thanks for contributing as usual ❤

Closing this one because the original lesson is already merged (#564) and the translation task list is in another issue (#850).


Update for API doc generator [1.14]

Revised the config.yaml file for resource changes happened in 1.14.
This also adds the latest swagger.json from release-1.14 branch.
Resource (type) changes:

CSIDriver and CSINode added to storage.v1beta1 group;
VolumeAttachement graduated from storage.v1beta1 to v1;
InitializerConfiguration from admissionregistration.v1alpha1 was
dropped (kubernetes/kubernetes#72972);
PriorityClass graduated from scheduling.v1beta1 to scheduling.v1;
new resource auditregistraion.v1alpha1.AuditSink got added;
new resource coordination.v1.Lease got added;
new resource node.v1beta1.RuntimeClass got added;

/lgtm
/approve

/lgtm


Building a cross-compiled, statically linked binary on Linux?

I'm trying to cross-compile using the following command:
cargo lambda build --compiler cargo --extension --release --target x86_64-unknown-linux-musl

This command fails with the message invalid options: --compiler=cargo is only allowed on Linux if I run it from an ARM machine.
I see that the build process disallows statically linking when cross-compiling on Linux hosts, per the logic from this section. Specifically, the check for is_static_linking returns false:
    #[cfg(target_os = "linux")]
    pub fn is_static_linking(&self) -> bool {
        self.rustc_target == format!("{}-unknown-linux-musl", ARCH)
    }

Since the rustc_target value will not match the ARCH environment variable.
Is it possible to relax this restriction? Or is there something inherent to the compilation process that will lead it to fail, hence the conditional that prevents it from working?

it's probably a bug in the way that that check is supposed to work. It might make more sense to do this:
    #[cfg(target_os = "linux")]
    pub fn is_static_linking(&self) -> bool {
        self.rustc_target == "x86_64-unknown-linux-musl" || self.rustc_target == "aarch64-unknown-linux-musl"
    }

I put up a quick PR to fix it: https://github.com/cargo-lambda/cargo-lambda/pull/736. I'll probably do a new release this week with this and other changes.


add default .gitattributes

I think this change could potentially introduce conflicts (whole file appears to have changed) in helper.go for devs on windows without autocrlf = true. Setting git config --global core.autocrlf true should fix the issue in that case, and is generally recommended on windows anyway (see https://git-scm.com/book/en/v2/Customizing-Git-Git-Configuration, core.autocrlf section)

I think this change could potentially introduce conflicts (whole file appears to have changed) in helper.go for those on windows without autocrlf = true. Setting git config --global core.autocrlf true should fix the issue in that case, and is generally recommended on windows anyway (see https://git-scm.com/book/en/v2/Customizing-Git-Git-Configuration, core.autocrlf section)

Yeah, I did notice that but not sure if it was intentional, sounds good, if intention was to only make change to one file i.e. .gitignore and revert other, I can approve and go ahead with merge if this sounds good? Or you area asking for hand for reverting this?

I think this change could potentially introduce conflicts (whole file appears to have changed) in helper.go for those on windows without autocrlf = true. Setting git config --global core.autocrlf true should fix the issue in that case, and is generally recommended on windows anyway (see https://git-scm.com/book/en/v2/Customizing-Git-Git-Configuration, core.autocrlf section)

Yeah, I did notice that but not sure if it was intentional, sounds good, if intention was to only make change to one file i.e. .gitignore and revert other, I can approve and go ahead with merge if this sounds good? Or you area asking for hand for reverting this?

This should be merged as is, the changes to the other files are only line ending changes, and due to running "git add renormalize" to standardize all text file line endings to LF (what we want the repo side to have, and how it will stay with text=auto gitattribute set).
Otherwise we risk an inconsistent mish mash of CRLF and LF files repo side, which can lead to erroneous conflicts when we have developers working on both windows and *nix in the same repo.


pilot, egress, and ingress (0.1.6) cannot list istioconfigs.istio.io at the cluster scope (k8s 1.7.4)

Just installed istio 0.1.6 onto kubernetes 1.7.4 by following the steps here - implying that I did apply rbac-beta.yaml.  Everything in the default namespace.
The only "strange" thing I did was to clone the repository and checkout the 0.1.6 tag, rather than download the release zip file.
Pilot, Egress, and Ingress pods are in CrashLoopBackOff.  Based on the logs below, I suspect that there are some new bindings that need to be added to istio-rbac-beta.yaml?
From pilot (apiserver and discovery both):
I0829 18:45:55.408322       1 client.go:205] TPR "IstioConfig" is not ready (User "system:serviceaccount:default:istio-pilot-service-account" cannot list istioconfigs.istio.io at the cluster scope. (get IstioConfigs.istio.io)). Waiting...

failed to register Third-Party Resources. User "system:serviceaccount:default:istio-pilot-service-account" cannot get thirdpartyresources.extensions at the cluster scope. (get thirdpartyresources.extensions istio-config.istio.io)

From egress:
I0829 18:56:46.743307       1 client.go:205] TPR "IstioConfig" is not ready (User "system:serviceaccount:default:default" cannot list istioconfigs.istio.io at the cluster scope. (get IstioConfigs.istio.io)). Waiting...

failed to register Third-Party Resources. User "system:serviceaccount:default:default" cannot get thirdpartyresources.extensions at the cluster scope. (get thirdpartyresources.extensions istio-config.istio.io)

From ingress:
I0829 19:02:29.047819       1 client.go:205] TPR "IstioConfig" is not ready (User "system:serviceaccount:default:istio-ingress-service-account" cannot list istioconfigs.istio.io at the cluster scope. (get IstioConfigs.istio.io)). Waiting...

Update: Here's the reason why it's crashing.  Your RoleBindings need to be ClusterRoleBindings.  The helm chart has this correct, but your latest tagged release (0.1.6) does not.

This is fixed now.


Issue on ios SDK 8.2

I am trying to compile it on ios sdk 8.2 following the wiki though getting this error  , kindly help  when building SBServertweak make: ***  [internal-package] Error 133

Which version of theos are you using and which version (commit hash) of DiOS?


fix: error err module not found

:tada: This PR is included in version 1.2.3-beta.1 :tada:
The release is available on:

npm package (@develop dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Improve Test Coverage For Independence Testing

Hi @akelleh,
Wrote some tests to improve the coverage for the independence testing module, if the coverage is better I could assure that the behavior doesn't change when we transition from pymc2 to pymc3. In the process of testing, I found a couple of bugs in the MixedChiSquaredTest function and added the fixes to this pull request. Do these look good? If not, what other tests should I write?
Wanted to get your feedback before I write tests for the MixedMutualInformationTest function.
Saul

BTW, Im sorry about the weird number of commits in this PR. Will fix it before merging with the main branch.

Hi @akelleh,
Can you please take a look?
Do these look good? If not, what other tests should I write?

Hi @akelleh!
I squashed the change list, but have no permission to merge it with master. Please merge it when you get a chance.
PS: Do you know where I can find some reading material about MixedChiSquaredTest and the theory behind it?


NIFI-8671 Create nifi-versioned-components module.

https://issues.apache.org/jira/browse/NIFI-8671
Extract VersionedComponent class family from nifi-registry to the new module nifi-versioned-components.

Thank you for submitting a contribution to Apache NiFi.
Please provide a short description of the PR here:
Description of PR
Enables X functionality; fixes bug NIFI-YYYY.
In order to streamline the review of the contribution we ask you
to ensure the following steps have been taken:
For all changes:

[x] Is there a JIRA ticket associated with this PR? Is it referenced
in the commit message?

[x] Does your PR title start with NIFI-XXXX where XXXX is the JIRA number you are trying to resolve? Pay particular attention to the hyphen "-" character.

[x] Has your PR been rebased against the latest commit within the target branch (typically main)?

[x] Is your initial contribution a single, squashed commit? Additional commits in response to PR reviewer feedback should be made on this branch and pushed to allow change tracking. Do not squash or use --force when pushing to allow for clean monitoring of changes.

For code changes:

[x] Have you ensured that the full suite of tests is executed via mvn -Pcontrib-check clean install at the root nifi folder?
[ ] Have you written or updated unit tests to verify your changes?
[ ] Have you verified that the full build is successful on JDK 8?
[ ] Have you verified that the full build is successful on JDK 11?
[ ] If adding new dependencies to the code, are these dependencies licensed in a way that is compatible for inclusion under ASF 2.0?
[ ] If applicable, have you updated the LICENSE file, including the main LICENSE file under nifi-assembly?
[ ] If applicable, have you updated the NOTICE file, including the main NOTICE file found under nifi-assembly?
[ ] If adding new Properties, have you added .displayName in addition to .name (programmatic access) for each of the new properties?

For documentation related changes:

[ ] Have you ensured that format looks appropriate for the output in which it is rendered?

Note:
Please ensure that once the PR is submitted, you check GitHub Actions CI for build issues and submit an update to your PR as soon as possible.

I reviewed the code and did some testing. Here are my findings:

No no dependencies introduced into nifi root lib/ directory except for nifi-versioned-components-api.
NiFi Registry still starts up and works as expected. Was able to push flows, pull flows, change versions from NiFi.
Was able to push flows, pull flows, change versions when interacting with an older version of registry. Including flows that already existed and newly created/pushed flows.
Can go to localhost:18080/nifi-registry-api/swagger/ui.html and am still able to see all of the appropriate documentation, including the full documentation for VersionedProcessGroup/VersionedProcessor, etc. that I got to from the "flows" section of the API.

I agree with @exceptionfactory that it makes sense to only include the swagger-annotations dependency where the nifi-registry-data-model module is used but otherwise, all looks good to me. I'm a +1 as soon as that gets updated.
Thanks for pulling all this together @tpalfy - I think this is a definitely a good move in the right direction. And thanks to @exceptionfactory  for the thorough review & @kevdoran for helping with the review & providing insights!

Yep, agree with @exceptionfactory and @markap14 regarding that dependency scope, and aside from that this looks good to me.

Thanks guys for the thorough review & progression here! I think this is looking really good. Keeps the API really nice and clean. I'm a +1 as well. Will merge to main