Can't load .so/.DLL, Symbol not found

Don't know how to debug that one. I'm using resolver lts-7.2 but forcing ghc-8.0.2 for the osx sierra fix. Stack build succeeds.

stack ghci ghci --with-ghc intero "--docker-run-args=--interactive=true --tty=false" --no-build --no-load --ghci-options -odir=/Users/luntain/p/ta/.stack-work/intero/intero33982m8f --ghci-options -hidir=/Users/luntain/p/ta/.stack-work/intero/intero33982m8f ta
The following GHC options are incompatible with GHCi and have not been passed to it: -Werror
Configuring GHCi with the following packages: ta
Intero 0.1.20 (GHC 8.0.2)
Type :intro and press enter for an introduction of the standard commands.
: can't load .so/.DLL for: /usr/local/Cellar/ghc/8.0.2/lib/ghc-8.0.2/ghc-8.0.2/libHSghc-8.0.2-ghc8.0.2.dylib (dlopen(/usr/local/Cellar/ghc/8.0.2/lib/ghc-8.0.2/ghc-8.0.2/libHSghc-8.0.2-ghc8.0.2.dylib, 5): Symbol not found: _timezm1zi6zi0zi1_DataziTimeziClockziCTimespec_getCTimespec1_closure
Referenced from: /usr/local/Cellar/ghc/8.0.2/lib/ghc-8.0.2/ghc-8.0.2/libHSghc-8.0.2-ghc8.0.2.dylib
Expected in: /Users/luntain/.stack/programs/x86_64-osx/ghc-8.0.2/lib/ghc-8.0.2/time-<IP_ADDRESS>/libHStime-<IP_ADDRESS>-ghc8.0.2.dylib
in /usr/local/Cellar/ghc/8.0.2/lib/ghc-8.0.2/ghc-8.0.2/libHSghc-8.0.2-ghc8.0.2.dylib)

it was some bad state, deleting ~/.stack got rid of it


Implement struct with typedef struct to make syntax cleaner

Describe the problem
Hi,
So i've used the generator for the ISO15118-2 and generate some messages. I immediately saw that within iso2_msgDefDatatypes.h there exists the following declarations for structs:
struct iso2_CostType {
    // costKind, costKindType (base: string)
    iso2_costKindType costKind;
    // amount, unsignedInt (base: unsignedLong)
    uint32_t amount;
    // amountMultiplier, unitMultiplierType (base: byte)
    int8_t amountMultiplier;
    unsigned int amountMultiplier_isUsed:1;
};

This syntax is clear for the declaration but a typedef declaration will make the use of this struct type more readable when implemented within other structs.
The idea is not to always usestruct name_type when used within other structs or function signatures.
Describe your solution
I can modify the generator if this is allowed to make a pull request if needed. The only problem with this change is: if this fix will trigger some global namespace problems (if enums with same name as structs exists, but I don't think so)
Additional context
No response

Seems like a very used pattern ... my fault ... don't have a lot of experience in C.


[BUG] Matched People Sensors Not Showing Up in HA

Describe the bug
Everything is working as designed. MQTT, Deepstack and frigate are all connected and working... that said, for some reason, the People I have matched in the model are not showing up as sensors in HA. I see double_take_unknown and I see 3 double take camera sensors, but none of the 4 people I have matched show up. I'm assuming I'm not doing something write or there's a bug of some kind.
Version of Double Take
v1.13.1-8e2728d
Expected behavior
I expect a person sensor to show up in HA so I can use for triggers in automations
Screenshots
Hardware

OS: Home Assistant 2023.5.3
Browser (if applicable) [e.g. Chrome, Safari]

Additional context
Add any other context about the problem here.

@kksligh have you created a sensor yourself in yaml? I found creating my own one, using the names of people I have defined in DT got round this issue:

mqtt:
  sensor:
    - name: Cheryl
      icon: mdi:account
      state_topic: 'double-take/matches/cheryl'
      json_attributes_topic: 'double-take/matches/cheryl'
      value_template: '{{ value_json.camera }}'
      availability_topic: 'double-take/available'

I didn’t think of this - I’ll give it a try
Thanks,
Kale Sligh

From: James T @.>
Sent: Friday, May 19, 2023 2:54:07 PM
To: jakowenko/double-take @.>
Cc: kksligh @.>; Mention @.>
Subject: Re: [jakowenko/double-take] [BUG] Matched People Sensors Not Showing Up in HA (Issue #314)
@kkslighhttps://github.com/kksligh have you created a sensor yourself in yaml? I found creating my own one, using the names of people I have defined in DT got round this issue:
mqtt:
sensor:
- name: Cheryl
icon: mdi:account
state_topic: 'double-take/matches/cheryl'
json_attributes_topic: 'double-take/matches/cheryl'
value_template: '{{ value_json.camera }}'
availability_topic: 'double-take/available'
—
Reply to this email directly, view it on GitHubhttps://github.com/jakowenko/double-take/issues/314#issuecomment-1555162412, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AT3AQU4ECRIGFYHHROPHFR3XG7FV7ANCNFSM6AAAAAAYIEYTWU.
You are receiving this because you were mentioned.Message ID: @.***>

FWIW, I'm having a similar issue right now, but only for new users. 3 of my first users are working, but my 4th is not. I have tried variations of the name (only simple ASCII in lowercase and not long or anything) but to no avail. Nothing obvious in the logs about failures or other pointers to the issue thus far.

Running into the same problem. First 3 users created were working. I then deleted my third user from DT, but it still shows in home assistant which was odd. Any user created after that does not show on home assistant. They appear in DT, but can't find out what would be causing it not to show in home assistant. All cameras showed up on home assistant just fine.

seeing the same thing with my install. the face is recognized but i do not get a sensor in HA for the person. there is a HA sensor for the camera but not for the person.


refactor: sidebar and header

issues75
final effect

@nibilin33 the repo is public. So anyone can see all screenshots, which may contain your personal information.


VSCode terminal hangs after resizing the terminal window during git diff

Does this issue occur when all extensions are disabled?: Yes/No

VS Code Version:
Version: 1.84.2 (user setup)
Commit: 1a5daa3a0231a0fbba4f14db7ec463cf99d7768e
Date: 2023-11-09T10:51:52.184Z
Electron: 25.9.2
ElectronBuildId: 24603566
Chromium: 114.0.5735.289
Node.js: 18.15.0
V8: <IP_ADDRESS>-electron.0
OS: Windows_NT x64 10.0.19045

OS Version:
Windows 10 Home, 19045.3693

Steps to Reproduce:

Open any project that has git revisioning.
Make enough changes to any file(s) so that git diff won't be able show everything all at once in the terminal.
Use git diff
Use the space key to automatically scroll down until the terminal shows "END".
Instead of immediately pressing "q" to leave, change the terminal's size with your mouse, making it bigger. A bunch of lines with the ~ character should appear.
Now press q
The "END" string disappears, but the terminal input doesn't appear.

Oddly enough this only happens on my private PC as specified above, but not at work, so it might not be easy to reproduce for you, but if there is a way to generate a log to add here I'll create one if it helps.
It happened both for a C++ project and a Python project, so I don't think it's a language-specific bug.

Please see https://github.com/Microsoft/vscode/wiki/Performance-Issues for creating actionable performance problem issues.
/label info-needed terminal

Alright, I created it, see https://github.com/microsoft/vscode/issues/199231

/duplicate https://github.com/microsoft/vscode/issues/199231


Display warning when new version of Apify SDK is available

We could pass current version from platform as environment variable

Done.


Scrolling on ipad at the theming example web page

It is almost impossible to scroll at this page on ipad and the latest ios:
http://semantic-ui.com/examples/theming.html
It is shaking as hell while scrolling...

Hi @czb, what version of Safari is it?

It is iOS 10.2.1

I got hold of a device with 10.2.0, confirming the issue.

Linking a related issue: Semantic-Org/Semantic-UI-Docs#283

This one is probably related as well: #2110.

From #5195:

There is some scroll hell on theming page.
Browser: Google Chrome 56.0.2924.87
OS: Android 5.1 / Flyme
Video

I've fixed this with https://github.com/Semantic-Org/Semantic-UI/commit/52dbc6288e327d1b671b0d86019a23ef6d601bcc and will be deploying this fix early to the docs.

i have just tested
https://semantic-ui.com/examples/theming.html
on iphone 6, ios 10.3 after clearing data in safari and it is probably worse then before. I also tested chrome. The worst thing is that now in chrome on ios it is almost inpossible to click on the top bar to show the address bar as it disappears immediatelly again.

Sorry this issue appears to be related to the use of iframes in the theming example, unrelated to the fixed issue.


[FEATURE REQUEST] Apply private/protected convention for class members

Is your feature request related to a problem? Please describe.
Currently we don't use private/protected (_|__) prefixes inside our Python code.
Describe the solution you'd like
We need to check which members can be changed to private/protected and add the prefix _ or __ if needed.
Describe alternatives you've considered
None.
Additional context
https://www.geeksforgeeks.org/access-modifiers-in-python-public-private-and-protected/

DoD checklist

[ ] Changelog updated
[ ] COMPONENTS.md updated / doesn't need to be updated
[ ] Schema updated / doesn't need to be updated
[ ] Feature has automated tests
[ ] Automated tests passed (QA pipelines)

[ ] apply
[ ] upgrade

[ ] Idempotency tested
[ ] Documentation added / updated / doesn't need to be updated
[ ] All conversations in PR resolved
[ ] Solution meets requirements and is done according to design doc
[ ] Usage compliant with license
[ ] Backport tasks created / doesn't need to be backported

Pls put this form of the feature request templates we have.

No blocked, just on hold to finish stuff for 1.2.0 first.


DataConvertible fix for 9.3

As of Swift 3.1, the conversion from Data to Int64 broke on iOS 9.3 (http://stackoverflow.com/questions/43099577/withunsafebytes-with-swift-3-1-and-ios-9-3)

I've addressed this by adding on the empty bytes manually during the conversion process. And funneling all Data to Number conversions through DataConvertible.
Also fixed one last .pi

Thanks for fixing the last .pi.


verified github action

Would it be possible to get the verified  status for this action on github?  This could help us adopt this because some enterprise organizations require this certification.

I need this, as our organization require verified creator for Github action

Seconded - our organization also requires all actions to be verified by GitHub. It would enable us to use this awesome work :)

We've put in a request to GitHub to become a verified publisher. If approved, the action will show up as verified. So now we wait.

@jennifer-shehane That is great to hear. Thank you for taking care of this :)

@jennifer-shehane Any updates on this !!

Closing this issue as Cypress is a verified publisher.


Association stats box is cut off

Text is cut off on the association stats box

Using margin: 0 auto may help with centering and keeping the text within the container

Yeah I think a big part of the issue is the text size doesn't dynamically resize so i don't want to just make the font smaller because then then it will look weird on big monitors. I'll look into how to fix this at a later date (leaving it here for now so I don't forgot).


Saving Response Blob Data Issue

I'm submitting a bug report

Library Version:
http-client 1.0.0

Please tell us about your environment:

Operating System:
OSX 10.11.6
Node Version:
6.2.2
NPM Version:
3.10.3
JSPM OR Webpack AND Version
JSPM 0.16.39
Browser:
Electron v1.2.5
Language:
ESNext

Current behavior:
I am using the aurelia-http-client library to get binary data from a REST service, and store it to file
(using a Blob).
According to what I is in #93, the data should be in message.content
This is the code I have been trying.
`
.withResponseType('blob')
        .withInterceptor({
            response(message) {
                var defaultFileName = "default.txt";
                var disposition = message.headers.headers['content-disposition']?message.headers.headers['content-disposition']:message.headers.headers['Content-Disposition'];
                if (disposition) {
                    var match = disposition.match(/.*filename=\"?([^;\"]+)\"?.*/);
                    if (match[1])
                        defaultFileName = match[1];
                }
                defaultFileName = defaultFileName.replace(/[<>:"\/\\|?*]+/g, '_');
                fs.writeFileSync(defaultFileName, message.content);

...
`
Expected/desired behavior:
Either there is an issue in message.content or I am way off track.

@bryanrsmith Can you respond to this?

@MMJM Can you provide more information? Do you see an error, or some other unexpected behavior?

If I had interpreted right the OP issue, I'm experiencing a similar problem: I need to get the response content as Blob, but the client gives me a string (failing for binary data). How can I tell the client to return the content as Blob?
In fetch we have fetch(url).then(res => res.blob()) but I can't switch to fetch right now…

Okay, turning client.get(uri) to client.createRequest(uri).withResponseType("blob").send() made it work! Now response.content is a shiny Blob 👍


Hotfix babyproof arrivals terminal and arrivals shuttle

About the PR
Merge #33284 and #33281 into staging, as a hotfix. Make arrivals terminal and arrivals shuttle invulnerable to most damage
Why / Balance
Admins wanted this out on live faster than the next release cycle
Requirements

[X] I have read and am following the Pull Request and Changelog Guidelines.
[X] I have added media to this PR or it does not require an ingame showcase.

Changelog
:cl: IProduceWidgets

fix: The terminal is more tamper proof.
fix: Arrivals shuttle is more tamper proof.

Does this actually need a CL entry, or will the one on master be already in the changelog?
Squash or not?
Needs 2 more approvals due to being hotfix

Added CL entries from the original PRs, I guess we will undupe that bridge when we get there

Note: Check failure is due to the labeler
Something to fix as I've noted in #maintainers on Discord

Not required for now, I suppose

And I'm a dumbass and did a wrong pr


Performance improvement & Memory consumption

Hey guys,
Thanks for the great library.
Currently contentViewControllers requires instantiated view controllers which could cause high memory usage specially with complex views. I tried to change it to array of StoryBoard identifiers and instantiate  VC inside selectContentViewController() which resolves the allocation issue and I can see that previous views gets de-initialized successfully but now I am having a UI Performance issue and changing the SideMenuItemContent seems a bit slow.
Have you tried to fix the memory allocation issue in another way or do you have any suggestions that can improve the overall performance impact ?

Hi, @mosn.
Thanks for this question! Actually, we didn't run into performance issue and large memory consumption. It would awesome to know details of your specific case.
So, could you describe a few things

How many menu items you have?
How complex layout of content view controllers you have?
Did you measure memory consumption by profiling tool? If it's so, what's a result you have? Why do you think it's out of the norm, some comparisons.

So, it requires some investigating to suggest appropriate solution.
Currently, I'd suggest trying to simplify complex layouts, investigate if it's possible to tune some views, use lazy initialization as many as possible.

Hi @AndreyArzhannikov ,
Thank you for quick response,  I haven't done any profiling yet but looking at the current implementation, it seems to me that placing all instantiated view controllers inside the contentViewControllers array  does not seem like best approach, Sadly use of Lazy init or tuning object  will not be beneficial here since with this setup, when the object is initialize it will never de-intialize and stays in memory forever.
I think the best approach to initialize view controllers only when the sideMenu is presented and deinit when it is not activated, Which brings us to my second point of performance impact in the menus with +4 menu items. Do you know why this impact is there. I can see at various stages the subviews are removed from the view, To me it looks like that you are also trying to release the allocated memory for some of these views.
Thanks

@mosn Yes, potentially we might have memory consumption issue if we have many menu items. The current implementation is not perfect in this case. A potential way to fix is to instantiate item VC by request or even caches it. Instantiating on the fly has own cost and here we need to have a lightweight view, tuning stuff and lazy initialization I mentioned before. I'm sorry for misleading you with lazy things above.
So, we added this point to our to-do list.
Thank you for addressing it and feel free to send us pull request with your solution!


Not able to get lag or status info from endpoint on existing consumers

Hi, I'm trying to monitor Kafka consumers on our staging machine, below is the config file:
`
[general]
pidfile="./burrow.pid"
stdout-logfile="burrow.out"
[logging]
filename="log/burrow.log"
level="debug"
maxsize=100
maxbackups=30
maxage=10
use-localtime=false
use-compression=true
[zookeeper]
servers=[ "example.com:2181" ]
timeout=6
[client-profile.stage]
kafka-version="0.10.2"
[client-profile.production]
kafka-version="0.10.2"
[cluster.stage]
class-name="kafka"
servers=[ "example.com:9092" ]
client-profile="stage"
topic-refresh=120
offset-refresh=30
zookeeper-offset=true
[consumer.stage]
class-name="kafka"
cluster="stage"
servers=[ "example.com:9092" ]
client-profile="stage"
offsets-topic="__consumer_offsets"
#group-blacklist="^(console-consumer-|python-kafka-consumer-|quick-).$"
group-blacklist="."
group-whitelist=""
[httpserver.default]
address=":18002"
`
the thing is I am able to see all the consumers using the corresponding endpoint but can't get the lag on the existing consumers, below is a snippet of the log:
 {"level":"debug","ts":1524773696.0555763,"msg":"dropped","type":"module","coordinator":"storage","class":"inmemory","name":"default","worker":7,"cluster":"stage","consumer":"ca_raw","topic":"ca_raw","partition":1,"topic_partition_count":0,"offset":24,"timestamp":1484519101275,"owner":"","request":"StorageSetConsumerOffset","reason":"old offset"}
also this is the result I got from hitting the /lag endpoint:
{
"error":
false,
"message":
"consumer status returned",
"request":
{
"host":
"MacBook-Pro.local",
"url":
"/v3/kafka/stage/consumer/ca_raw/lag"
},
"status":
{
"cluster":
"stage",
"complete":
1,
"group":
"ca_raw",
"maxlag":
null,
"partition_count":
0,
"partitions":
[],
"status":
"NOTFOUND",
"totallag":
0
}
}

Let me know if you need more information, thx in advance!

The debug log shows that the offsets are being dropped because they are very old, and the response shows that burrow is returning a 404 for the request. Can you show the results of calling /v3/kafka/stage/consumer

Attached is the result of calling /v3/kafka/stage/consumer
{ "consumers": [ "s3_uploader_raw", "s3_uploader_push_history_wh", "global_user_update_processor", "refresh_request_v2", "simage_server_inbound_requests_wh", "data_pusher", "win_raw", "image_server_outbound_requests_wh", "attributed_events_attributed_events", "identifier_bulk_push_request", "read_pixel_events_paid", "exhaust_processor_v2", "creative_push_request", "query_performance_log", "pixel_events_organic", "dispatcher", "rtb_bid_feedback", "php_logger_event_wh", "campaign-update", "clk_raw", "pixel_events_organic", "uploade_win_raw", "pixel_events" ], "error": false, "message": "consumer list returned" }

OK, the group that you're querying is definitely not there. And the timestamp from the offset that was dropped was from January 2017. You need to look at why you're not seeing any offsets at all from that consumer group (at least none that are recent)

My bad I didn't include the "ca_raw" in the output, I just double checked, consumer "ca_raw" is showing up on the endpoint call, also we have a v2 version of burrow running on staging machine and it is doing fine, thx for looking into this

I too am able to reproduce this issue.  I'm running a docker build based off master.  What I see is the following:
`

curl -s https://burrow/v3/kafka/palestra_elk/consumer                                                                                {"error":false,"message":"consumer list returned","consumers":["logstash","console-consumer-72901"],"request":{"url":"/v3/kafka/palestra_elk/consumer","host":"burrow-69f7c4f984-fhdll"}}
curl -s https://burrow/v3/kafka/palestra_elk/consumer/logstash                                                                                {"error":true,"message":"cluster or consumer not found","request":{"url":"/v3/kafka/palestra_elk/consumer/logstash","host":"burrow-69f7c4f984-fhdll"}}
curl -s https://burrow/v3/kafka/palestra_elk/consumer
{"error":false,"message":"consumer list returned","consumers":[],"request":{"url":"/v3/kafka/palestra_elk/consumer","host":"burrow-69f7c4f984-fhdll"}}
`

This was run about a minute after the container was brought online.  What I see is that the consumer shows up, until any queries are made against that consumer, and then it appears to be permanently deleted.   Requires restart of container to get the consumer to show back up.


Ticdc(Sink) use ticdc logger to print pulsar log

2 set partitionKey value to message

What problem does this PR solve?

1 use ticdc logger to print pulsar log
2 set partitionKey value to message
Issue Number: ref https://github.com/pingcap/tiflow/issues/9413
What is changed and how it works?
Check List
Tests

Unit test

Questions
Will it cause performance regression or break compatibility?
Do you need to update user documentation, design documentation or monitoring documentation?
Release note
None

/ok-to-test

/retest

/retest

/retest

/retest

/retest

This pull request was split into #9674 #9677.


Update README.md

This PR updates the README.md of Node.js for Oracle NoSQL Database with minor modifications.
Using the same default values for all examples in all documentation - 50,50,25 as for free tables

Signed-off-by: Dario Vega<EMAIL_ADDRESS>

https://github.com/oracle/nosql-node-sdk/pull/14


Separate styles for placeholder and selected items

Version 4.2.0
Here's what went wrong:
The default text looks like a selected option. It would be better if the placeholder text had it's own style.

@pokidovea Do you have an example of what it should look like?

See this, please. Default text should have a color, which shows, that it is not selected option, but just a placeholder. But when you choose an option, the text should have current color.

Released in version 4.7.1
Closed by #186

I'm just in the process of trying to improve the accessibility of Picky, and one thing that was flagged was the contrast of the placeholder. It has a contrast ratio of 2.07:1 which fails accessibility checks.
I'm changing the placeholder colour to #595959 which has a passing ratio of 7:1 which is AAA.
If you want a lighter colour, by all means, override the styles in the CSS in your project. But Picky should be accessible by default.


Don't automatically add building=yes to Motel and Hotel

Describe the bug
The Motel and Hotel presets should not add building=yes since one of the valid ways to map a motel and hotel is to draw an area around the entire grounds, and then add the buildings as building=motel/hotel inside the area
Expected behavior
Since there's already a "Hotel Building" preset, imo the easiest thing to do is remove the building tag from the Hotel preset, then add a Motel Building to match the hotel presets

This one is a bit tricky. Yes, hotels can also be mapped independently of their building, but the vast majority (about 85%[^1]) are currently mapped together with the building tag. Changing the preset would make mapping the majority of cases less simple. Note that the existing preset does only require one additional click to remove the building tag from the object when one wants to map a hotel campus.

Since there's already a "Hotel Building" preset,

That preset is actually a little misleading, as it refers to buildings whose design/architecture/construction is that of a hotel (e.g. a building with many individually accessible rooms), independently of whether it is used as a hotel or not.
What we could potentially do to improve the tagging schema is to add a dedicated preset for hotels not mapped together with a building. This preset would not include the building tag and field and could be labeled as "Hotel (Campus)", or perhaps better "Hotel Complex"?!
[^1]: source: taginfo https://taginfo.openstreetmap.org/tags/tourism=hotel?filter=ways#combinations

What we could potentially do to improve the tagging schema is to add a dedicated preset for hotels not mapped together with a building.

Can we use the same pattern as for school building vs school ground? I think that works well and is the same kind of issue, right?

We could of course. The differences here are that for amenity=school the combined use with building=* is much much less common (~7%) than for hotels (~85%) and that the wiki does only recommend the combined use for hotels, but not for schools. Surely that difference in usage should somehow be reflected in the presets?!


feat(theme-generator): Add possibility to force custom color @angular/material:theme-color

Feature Description
My problem is that I still can't understand how @angular/material:theme-color works. In the terminal, I need to provide colors for primary, secondary, and tertiary. For each one, I specify a single color, like #ff9500, because that's what I want.
What HEX color should be used to generate the M3 theme? It will represent your primary color palette. (ex. #ffffff) **#005dba** What HEX color should be used represent the secondary color palette? (Leave blank to use generated colors from Material) **#005dba** What HEX color should be used represent the tertiary color palette? (Leave blank to use generated colors from Material) **#005dba**
However, when generating the theme, I see that it creates color palettes, but when I try to get just my color, I use --mat-sys-primary. Unfortunately, it appears significantly darker (#8c5000), and I don't know which variable the color is actually hidden under. I have the impression that due to the low contrast, it is being artificially darkened. However, I selected the option not to generate mixins with high contrast. I would like to be able to use exactly that color without worrying about contrast. Currently, I am forced to manually change all the system token colors because the generated palette still interferes too much with what I want to achieve, taking away the flexibility I need in this case.
Use Case
It would be great to be able to specify in the terminal that I want exactly the colors I mention, without any interference.

Very big +1. The fact that you cannot provide exact colors to an M3 theme is a deal breaker for me. Not being able to use my company's brand colors just because they aren't "up to Material's standards" just feels wrong.
I know I can manually override every color variable, and I will do so when support for M2 gets dropped eventually, but this is just very annoying and I really hope there will be an easier way to use exact colors for a theme.

@zygarios For each color specified, those colors are used directly to create color palettes of tones that match that color's hue and chroma. If you only specify a single color or a subset, the other colors get created in relation to each other. So there shouldn't be much adjusting happening if you specify everything and aren't requesting high contrast.
Although, Material defines --mat-sys-primary as tone 40 in your primary palette. Typically the color used to create the color palette usually closest to tone 50 in the color palette, which is why it is slightly off to what you are expecting.
Although I do see what you are saying, it would be ideal if you can specify maybe the primary color and then the color palette gets extracted from that to get more color accuracy and be more intuitive. I'll leave this issue open so this can be looked into.
In the meantime, overriding colors is your best option for complete customization: https://material.angular.io/guide/theming#system-tokens


Migrate from @babel/plugin-syntax-import-assertions to @babel/plugin-syntax-import-attributes

Description
https://next.babeljs.io/docs/v8-migration/#babelplugin-syntax-import-assertions
Suggested Solution
.
Alternatives
No response
Additional Context
No response

I've just tried to switch to @babel/plugin-syntax-import-attributes and while it does work with Babel, the latest version of ts-node doesn't work with with { type: xxx } syntax and we cannot run our mocha tests.


feature/testprbranch

          1 Warning

:warning:
Please target PRs to develop branch

  Generated by :no_entry_sign: Danger


[AutoPR machinelearningservices/resource-manager] [DO NOT MERGE ] Migrate version to 2019-05-01 in new folder and Adding usages by VM family support

Created to sync https://github.com/Azure/azure-rest-api-specs/pull/6220

This PR has been merged into https://github.com/Azure/azure-sdk-for-python/pull/5598


Fix HTML title parsing bugs.

Summary

This slightly modifies the HTML_TITLE_REGEX and fixes two parsing errors. The first occurred when title tags were empty (e.g. <title></title>) which was parsed as </title. The second occurred when titles were a single character (e.g. <title>A</title>) which was not matched by the regex, and so the title would fall back to link.base_url.
With this change, now when tags are empty, the title falls back to link.base_url, and single character titles are parsed correctly.
I tested the new regex with the edge cases I could think of, and found some malformed HTML will still lead to undesired behavior. A more robust regex could probably be used in the future.
Related issues

#1222
Changes these areas

[x] Bugfixes
[ ] Feature behavior
[ ] Command line interface
[ ] Configuration options
[ ] Internal architecture
[ ] Snapshot data layout on disk

Thanks!


Missing segments in ORDRSP D.96A (openedifact-xsd-libs-96A-1.1-SNAPSHOT.jar file)

Hi,
I downloaded the file openedifact-xsd-libs-96A-1.1-SNAPSHOT.jar from sourceforce and the xsd for ordrsp is missing some segments like SG29 and UNT. Do you know what can be causing it?
Do you have any instructions on how to generate the xsd files using this project? It also seems like almost all fields in the D.96A xsds are defined as numeric even when they are defined as alphanumeric in the spec. Do you know any reason for that?

Hi Anton,
found the first bug (nasty little bug).
The ORDRSP xsd should now be fine for the structure.
I will look into the "numeric" bug now.
Thanks for getting in touch.
Best
Matthias
On Mon, Sep 21, 2020 at 4:58 PM Anton<EMAIL_ADDRESS>wrote:

Hi,
I downloaded the file openedifact-xsd-libs-96A-1.1-SNAPSHOT.jar from
sourceforce and the xsd for ordrsp is missing some segments like SG29 and
UNT. Do you know what can be causing it?
Do you have any instructions on how to generate the xsd files using this
project? It also seems like almost all fields in the D.96A xsds are defined
as numeric even when they are defined as alphanumeric in the spec. Do you
know any reason for that?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/sapstern/edifactconverter/issues/3, or unsubscribe
https://github.com/notifications/unsubscribe-auth/ACPJJIHT3XVUWMDIJAWXW4TSG5SX3ANCNFSM4RUULN4A
.

Hi Anton,
found the second bug as well, this was my poor programming.
I have attached the xsd, it should now be ok.
I will publish the changes later on github.
You can find instructions on how to build the converter as well as the xsd
files in the wiki link:
https://github.com/sapstern/edifactconverter/wiki
Thanks again for your valuable feedback.
Best Matthias
On Mon, Sep 21, 2020 at 4:58 PM Anton<EMAIL_ADDRESS>wrote:

Hi,
I downloaded the file openedifact-xsd-libs-96A-1.1-SNAPSHOT.jar from
sourceforce and the xsd for ordrsp is missing some segments like SG29 and
UNT. Do you know what can be causing it?
Do you have any instructions on how to generate the xsd files using this
project? It also seems like almost all fields in the D.96A xsds are defined
as numeric even when they are defined as alphanumeric in the spec. Do you
know any reason for that?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/sapstern/edifactconverter/issues/3, or unsubscribe
https://github.com/notifications/unsubscribe-auth/ACPJJIHT3XVUWMDIJAWXW4TSG5SX3ANCNFSM4RUULN4A
.

Hi Matthias,
I generated the xsd's for the D96 formats I need an this looks really good.
Many thanks for this tool!
Kind Regards,
Anton


add long running test in ci

just waiting for tests to finish, to see it exits before 6h to not count as a failure

How long does it run?

I don't know how fast the machine is, I think maybe around 3hours


Add --ec2-user-data option to launch command

This PR adds --ec2-user-data option to launch command.
This flag can help users create UserData defined instances.
--ec2-user-data requires the path of UserData script file, and it could support long and complex script under 16KB.[1]
In case of increasing the number of slave instances, new slave instances are going to have the same userdata from the master instance.
I tested this PR by deploying cluster to AWS.
This PR is related to #142.
[1]: Configuring Instances with User Data

I found the error when requesting the spot instances and fixed it.
I'm testing now, and let you know when I'm done.

Ah, I just saw your most recent comment, @SoyeonBaek-dev. OK cool.

I fixed the bug about requesting spot instances with user-data.
Also, I changed the launch --ec2-user-data option type and help message.
Please review, and feel free to give me any comments.

Updated PR looks good to me. Going to test this out now.

I'm seeing this when I try to add slaves to a regular, on-demand cluster:
$ flintrock add-slaves nick --num-slaves 1
Traceback (most recent call last):
  File ".../venv/bin/flintrock", line 9, in <module>
    load_entry_point('Flintrock==0.7.0.dev0', 'console_scripts', 'flintrock')()
  File ".../venv/lib/python3.5/site-packages/flintrock/flintrock.py", line 1039, in main
    cli(obj={})
  File ".../venv/lib/python3.5/site-packages/click/core.py", line 716, in __call__
    return self.main(*args, **kwargs)
  File ".../venv/lib/python3.5/site-packages/click/core.py", line 696, in main
    rv = self.invoke(ctx)
  File ".../venv/lib/python3.5/site-packages/click/core.py", line 1060, in invoke
    return _process_result(sub_ctx.command.invoke(sub_ctx))
  File ".../venv/lib/python3.5/site-packages/click/core.py", line 889, in invoke
    return ctx.invoke(self.callback, **ctx.params)
  File ".../venv/lib/python3.5/site-packages/click/core.py", line 534, in invoke
    return callback(*args, **kwargs)
  File ".../venv/lib/python3.5/site-packages/click/decorators.py", line 17, in new_func
    return f(get_current_context(), *args, **kwargs)
  File ".../venv/lib/python3.5/site-packages/flintrock/flintrock.py", line 661, in add_slaves
    **provider_options)
  File ".../venv/lib/python3.5/site-packages/flintrock/ec2.py", line 48, in wrapper
    res = func(*args, **kwargs)
  File ".../venv/lib/python3.5/site-packages/flintrock/ec2.py", line 283, in add_slaves
    user_data=user_data)
  File ".../venv/lib/python3.5/site-packages/flintrock/ec2.py", line 668, in _create_instances
    user_data = user_data.read()
AttributeError: 'str' object has no attribute 'read'

I think the issue is that _create_instances() should probably be passed the file contents rather than the file handle during launch, which would make things consistent with add-slaves.

@SoyeonBaek-dev - When you fix this issue, just confirm to me that you've successfully tested the following scenarios and I will merge this in:

Launching a regular cluster.
Adding slaves to a regular cluster.
Launching a spot cluster.
Adding slaves to a spot cluster.

I should have tested all the test in the flintrock, and now I passed all the test except test_launch_with_bad_ami test(this test should be fixed).
Also, I checked all the scenarios you mentioned above.
Please check and test once again.

Retested this and it all works as expected now. Thank you @SoyeonBaek-dev!


Fixes exponential numeric values issue

What is the context of this PR?
This will now restrict the respondent to enter exponential values with in numeric fields (Example: 2e2)
How to review
Launch any business survey where you can enter a numeric value
Enter a numeric value as: 2e2 and submit
Must now throw an error: Enter a number.

Tested and LGTM 👍


feat: add useLinkProfile hook and update profile linking functionality

Problem solved
Short description of the bug fixed or feature added

PR-Codex overview
This PR introduces the useLinkProfile() hook to enhance profile linking capabilities across in-app and ecosystem accounts, along with several UI and documentation updates to support this feature.
Detailed summary

Added useLinkProfile() hook for linking profiles.
Updated sidebar links to include "Ecosystem Wallets" and "Playground".
Modified profile-related comments and documentation for clarity.
Enhanced UI components to display linked profiles.
Adjusted ecosystem connection components for better integration.

✨ Ask PR-Codex anything about this PR by commenting with /codex {your question}

#4906  👈
main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @joaquim-verges and the rest of your teammates on  Graphite


convert/line_splitter_test fails in Chrome/Firefox on Windows

Seems likely to be some kind of line delimiter error.
See, e.g. https://uberchromegw.corp.google.com/i/client.dart/builders/full-win7-ie11-2-4-be/builds/4689

Suppressed in https://codereview.chromium.org/1236793011/

@lrhn , @floitschG , any updates on this one?

The bug is either in the test, dart2js, the VM or the language specification, depending on your point of view.
I can fix the test.

Great, thanks

Fixed (committed by pquitslund)


Implement Support for creating the locking primitives in a const context on stable Rust

This adds a new constructor const_new to lock_api's mutexes, reentrant mutexes and RwLocks that allows creating them in a constant context on stable Rust by manually passing in the underlying raw mutex / rwlock.
Additionally, various helper functions are being added to parking_lot that allow creating mutexes, reentrant mutexes, fair mutexes and RwLocks in constant contexts on stable Rust.

This is a rough initial implementation. I'm not sure if the helpers in parking_lot are something that we would want as well. I can remove them if not.

bors r+

Any chance this could get cut in a release? I could probably deprecate https://crates.io/crates/static_locks if it were.


Output Alacritty config in TOML format

Since version 0.13.0 Alacritty uses TOML as their configuration format. These changes reflect that.
closes #93

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

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

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
0

Relevant Lines:
0

💛  - Coveralls


Other party loop is too long

We get this ~ 4 questions after the name. I suggest gathering all names before moving on to extra details for each defendant, or making it clear on the first screen that there will be room to add more defendants later.

Another alternative is to get more information about each defendant on one screen, but I think separating it makes sense. Just too many screens in between right now.

I think this interview ordering came from feedback from the Trial Court, so I will relabel this as a question and pass it along with our recommendation.


Fixed list index out of bound error

I hope this will fix the error!!

From my limited testing it looks like it did!


python310Packages.aiovodafone: 0.3.1 -> 0.3.2

Automatic update generated by nixpkgs-update tools. This update was made based on information from passthru.updateScript.
meta.description for python310Packages.aiovodafone is: Library to control Vodafon Station
meta.homepage for python310Packages.aiovodafone is: https://github.com/chemelli74/aiovodafone
meta.changelog for python310Packages.aiovodafone is: https://github.com/chemelli74/aiovodafone/blob/0.3.2/CHANGELOG.md
Updates performed

Ran passthru.UpdateScript

To inspect upstream changes
Impact

Checks done (click to expand)

built on NixOS
The tests defined in passthru.tests, if any, passed
found 0.3.2 with grep in /nix/store/kr8nrxrk6rkx8xdgyv2392vq73kbgk34-python3.10-aiovodafone-0.3.2
found 0.3.2 in filename of file in /nix/store/kr8nrxrk6rkx8xdgyv2392vq73kbgk34-python3.10-aiovodafone-0.3.2

Rebuild report (if merged into master) (click to expand)

3 total rebuild path(s)

3 package rebuild(s)

First fifty rebuilds by attrpath
home-assistant-component-tests.vodafone_station
python310Packages.aiovodafone
python311Packages.aiovodafone

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/kr8nrxrk6rkx8xdgyv2392vq73kbgk34-python3.10-aiovodafone-0.3.2 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A python310Packages.aiovodafone https://github.com/r-ryantm/nixpkgs/archive/9c742cd8c4abe42b0f04a0bb4012f8c93e728ca3.tar.gz

Or:
nix build github:r-ryantm/nixpkgs/9c742cd8c4abe42b0f04a0bb4012f8c93e728ca3#python310Packages.aiovodafone

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/kr8nrxrk6rkx8xdgyv2392vq73kbgk34-python3.10-aiovodafone-0.3.2
ls -la /nix/store/kr8nrxrk6rkx8xdgyv2392vq73kbgk34-python3.10-aiovodafone-0.3.2/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review run on x86_64-linux 1

1 package failed to build:

home-assistant-component-tests.vodafone_station

2 packages built:

python310Packages.aiovodafone
python311Packages.aiovodafone

Maintainer pings
cc @fabaff for testing.

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

1 package failed to build:

home-assistant-component-tests.vodafone_station

4 packages built:

python310Packages.aiovodafone
python310Packages.aiovodafone.dist
python311Packages.aiovodafone
python311Packages.aiovodafone.dist


Built in padding doesn't apply in React (or Next.js).

I want to begin by saying that I love this project. It looks great, and it's really easy to use.
Unfortunately, I've found a bug.
In both React and Next.js, all content is placed inside a wrapper
 inside .
For example:

This breaks the responsive vertical padding on , , and , as defined here in the documentation, and here in the source code.
The fix for this is pretty simple, but it involves putting edge cases for React and Next.js into the framework, which might be something you don't want to do.
I'm proposing changing this part of layout/_sectioning.scss (line 21):
body {
  // ...
}

to this:
body, #__next, #root {
  // ...
}

Of course, the CSS version of the framework would also need to be updated.
I'm currently using patch-package to make these patches in my own project.
I'm ready to make a pull request if you'd be happy with it, but if not, that's fine too.

Hi @ja1den,
Thank you.
I will not create a special case for React and Next.js.
I try to keep Pico as neutral as possible, focused on natives HTML elements.
The only thing I could consider is a variable to enable (disabled by default) a container for React and Next.js
// Enable React semantic container
$enable-class-container: false !default;

The clean way to override the default semantic container is to disable all the containers in Pico :
pico-react.scss
// Enable , ,  inside  as a container
$enable-semantic-container: false;

// Enable .container and .container-fluid
$enable-class-container: false;

@import "pico";

And then import your own container layout :
pico-react.scss
@import "layout/react-container";

layout/_react-container.scss
#root {
  > header,
  > main,
  > footer {
    ...
  }
}

But the default Pico style does not have a semantic container.
You don't have to edit the styles to have a container. Just use .container or .container-fluid.

     ...
     ...
     ...

The div between body and main applies to React, Next and Nuxt, too. I agree Pico should be as neutral as possible.
Why not this? Example:
@media (min-width: 768px) {
body > header, body > div[id] header,
body > main, body > div[id] main,
body > footer, body > div[id] footer {
    max-width: 700px;
  }
}

For anyone still looking for a pure CSS-based solution to this problem, paste the following into one of your app's stylesheets.
@media (min-width: 576px) {
  body > div#root > header,
  body > div#root > main,
  body > div#root > footer,
  section {
    --block-spacing-vertical: calc(var(--spacing) * 2.5);
  }
}
@media (min-width: 768px) {
  body > div#root > header,
  body > div#root > main,
  body > div#root > footer,
  section {
    --block-spacing-vertical: calc(var(--spacing) * 3);
  }
}
@media (min-width: 992px) {
  body > div#root > header,
  body > div#root > main,
  body > div#root > footer,
  section {
    --block-spacing-vertical: calc(var(--spacing) * 3.5);
  }
}
@media (min-width: 1200px) {
  body > div#root > header,
  body > div#root > main,
  body > div#root > footer,
  section {
    --block-spacing-vertical: calc(var(--spacing) * 4);
  }
}

body > div#root > header,
body > div#root > main,
body > div#root > footer {
  width: 100%;
  margin-right: auto;
  margin-left: auto;
  padding: var(--block-spacing-vertical) 0;
}

I pulled the relevant declarations out of css/pico.css and modified the selector definitions to include div#root.

For anyone still looking for a pure CSS-based solution to this problem, paste the following into one of your app's stylesheets.
@media (min-width: 576px) {
  body > div#root > header,
  body > div#root > main,
  body > div#root > footer,
  section {
    --block-spacing-vertical: calc(var(--spacing) * 2.5);
  }
}
@media (min-width: 768px) {
  body > div#root > header,
  body > div#root > main,
  body > div#root > footer,
  section {
    --block-spacing-vertical: calc(var(--spacing) * 3);
  }
}
@media (min-width: 992px) {
  body > div#root > header,
  body > div#root > main,
  body > div#root > footer,
  section {
    --block-spacing-vertical: calc(var(--spacing) * 3.5);
  }
}
@media (min-width: 1200px) {
  body > div#root > header,
  body > div#root > main,
  body > div#root > footer,
  section {
    --block-spacing-vertical: calc(var(--spacing) * 4);
  }
}

body > div#root > header,
body > div#root > main,
body > div#root > footer {
  width: 100%;
  margin-right: auto;
  margin-left: auto;
  padding: var(--block-spacing-vertical) 0;
}

I pulled the relevant declarations out of css/pico.css and modified the selector definitions to include div#root.


Incorrect result of arbitrary(distinct x) with mask

Description
I'm implementing the custom result verifier for arbitrary() and saw a failure with the following query.
SELECT
    c2, arbitrary(distinct c0) filter (where c1)
FROM (
    VALUES
        (123, true, 1),
        (456, false, 2),
        (789, null, 3)
) AS t(c0, c1, c2)
group by c2;

Presto returns three rows (1, 123), (2, null), (3, null). However, Velox returns (1, 123), (2, -4982426243126403411), (3, -4982426243126403411).
Error Reproduction
TEST_F(ArbitraryTest, aaa) {
  auto data = makeRowVector(
      {makeFlatVector<int64_t>({123, 456, 789}),
       makeNullableFlatVector<bool>({true, false, std::nullopt}),
       makeFlatVector<int64_t>({1, 2, 3})});

  auto expected = makeRowVector(
      {makeFlatVector<int64_t>({1, 2, 3}),
       makeNullableFlatVector<int64_t>({123, std::nullopt, std::nullopt})});

  auto plan = PlanBuilder()
                  .values({data})
                  .singleAggregation(
                      {"c2"}, {"arbitrary(distinct c0) filter (where c1)"})
                  .planNode();
  AssertQueryBuilder(plan).assertResults(expected);
}

Relevant logs
No response

Interesting that -4982426243126403411 is 0xBADADDBADADDDEAD 🙂
Some specifically invalidated memory.

@kagamiori Is this fixed now?

Yes, it has been fixed. Closing it now.


K8s section

Affected Components

[ ] Content & Marketing
[ ] Pricing
[ ] Test
[ ] Docs
[x] Learn
[ ] Other

I did a drive-by here renaming the otel logo's .svg to be a little more consistent.

Working on a fix, converting to draft since this isn't ready to merge

lefthand toolbar and top breadcrumbs are now fixed

Per slack discussion we may be reorganizing these menus, I think we can merge this branch with main in the interim @Dgiordano33

I agree, let's merge this so I have a clean slate to work from


Round widget coordinates in Settings

This fixes a weird rendering issue, as shown in the screen shot below.
Without this patch, depending on the size and position of the window, sometimes I got a settings window like this (worst case):

Can you also take a screenshot after this patch is applied?


OSX

Hi,
could there be a OSX version?
Many thanks.

Currently there are no plans to do OSX packaging.

hi. I've compiled a MacOS version. try it.
https://github.com/lawrencepn/SonoffToolMac
This works and tested on MacOs Catalina.


Not able to save phone number with 83.. series

I'm trying to store phone number +918369232477 using django phone number field which is a wrapper of libphonenumber, as I have installed this in my django project but this number is not getting stored. Currently this is the new series numbers that reliance jio providers in India have launched with. Giving me error as its not valid phone number. Apart from this all other 10 digit numbers are getting saved such as +919899999999 , +917699889898, ...etc
I think this library doesn't support this series of phone numbers. Please help me in this issue.

In theory this should have been corrected in https://github.com/googlei18n/libphonenumber/pull/1629.

Thank you for using libphonenumber!
Please upgrade to the latest release, this is not a live issue. To check the
latest behavior for yourself, see the demo (instructions in our contribution
guidelines).


Markers with label are now css hoverable

You can't use css hover because you are hovering the "invisible" div and if you try change color or anything else you won't see it because 0.1 opacity.
You must choose between to hover labels by css or to capture events even if the label is in the shadow of a google.maps.InfoWindow.
More info about the problem here

I signed it!


Add unit tests for anonymisation process

Fixes #132

Refactor: move add_private_tag to core.dicom_tags
Only load project config when it's needed
Add note to refactor private tag addition in orthanc-raw
Add DICOM image test fixtures to test anonymisation
Add anonymisation unit tests

The whole private tag class is very odd to me, but that was already in there and everything else looks good. 🚀

What do you think is odd about it?
I guess it's just meant as a consistent interface to add custom private tags to DICOM datasets.


Remove references to missing COPYRIGHT file

This file is not included in this repository, and not available at the
listed URL (https://www.rust-lang.org/COPYRIGHT).
@Thomasdezeeuw

Hi folks, this (or an alternative) resolution is needed to import this crate into Fuchsia. Is anyone able to take a look? cc @alexcrichton @sfackler @stjepang

I'm assuming this is ok, but I'm going to defer this to @alexcrichton.


migrate relevant terraform infrastructure from hasadna-iac repo to dfc-iac repo

relevant infrastructure to migrate:
https://github.com/hasadna/hasadna-iac/blob/main/modules/anyway

to do on or after 25 September

log-in to livedns, change nameservers for oway.co.il and anyway.org.il to the aws route53 nameservers


[Bug]: Check failing due to IAM Roles created by AWS Control Tower and AFT with AdministratorAccess  policy

Discussed in https://github.com/prowler-cloud/prowler/discussions/3809

Originally posted by @dmkim22-lguplus April 18, 2024
Hello,
IAM Roles created by AWS Control Tower and AFT (Account Factory for Terraform) have AdministratorAccess policy attached, and it seems "Ensure IAM AWS-Managed policies that allow full ":" administrative privileges are not attached" is failing due to this.
Should the following IAM Roles in an account be excluded from this check?

aws-controltower-AdministratorExecutionRole
AWSAFTExecution
AWSAFTService
AWSControlTowerExecution
stacksets-exec-*

Thank you in advance.

Hi @dmkim22-lguplus, in the above PR we have included those resources in the AWS default Mutelist. Please give it a look and let us know if this works as expected.
Thanks for using Prowler 🚀

We need to do further investigation because the check iam_aws_attached_policy_no_administrative_privileges analyzes IAM Managed policies so the resource_id is the policy name. So there is no quick solution for this issue as of today, we need to think about having related resources in the same findings and use the allowlist.

I consider this the correct behavior.  These will generally be accepted risks but they should still be flagged as risks and accepted/suppressed by a downstream process.  Alternately, Prowler could support some sort of mechanism to suppress accepted risks at scan time, eg via resource tags or a configurable ignorelist.  I would still say that should be at the discretion of the end user, and we should not hardcode role namespaces to ignore in this check.

@mtronrd for sure, we were talking about including this into the https://docs.prowler.com/projects/prowler-open-source/en/latest/tutorials/mutelist/. But for that we'd need to store more than one resource into the finding since now the resource_id is just the policy and not the policy and the role attached to it.


Run to Cursor and Add to Watch available in JSON settings file context menu

IMO Run to Cursor and Add to Watch should not appear in the editor context menu for the settings.json file. Additionally Evaluate in Debug Console and Add Inline Breakpoint appear in the editor context menu for the settings.json file when a debug session is active.

I guess they should only appear for languages that have breakpoints enabled


fix(time): Quarters split the year into 4 quarters not 3. close #17057

Brief Information
This pull request is in the type of:

[y] bug fixing
[ ] new feature
[ ] others

What does this PR do?

The quarter format would only go up to Q3 with each quarter having 3 months in them (not the current 4)
Fixed issues

Details
Before: What was the problem?

{q} would return 1 for Jan, Feb, Mar, Apr, 2 for May, Jun, July, Aug, 3 for Sep, Oct, Nov, Dec

After: How is it fixed in this PR?

{q} will return 1 for Jan, Feb, Mar, 2 for Apr, May, Jun, 3 for July, Aug, Sep, 4 for Oct, Nov, Dec

Misc

close #17057

[ ] The API has been changed (apache/echarts-doc#xxx).

[ ] This PR depends on ZRender changes (ecomfe/zrender#xxx).

Related test cases or examples to use the new APIs
N.A.
Others
Merging options

[ ] Please squash the commits into a single one when merging.

Other information

Could you remove unnecessary binary file?


bug: cannot import ES modules

Is there an existing issue for this?

[X] I have searched the existing issues

Description of the bug
Currently, after creating a new project from scratch (with typescript starter) and importing lowdb (which is an ES library, which is the one I want to use as a database), I get an error saying that I cannot import ES modules:
Error when loading '...\test-bot\dist\commands\ping.js': Error [ERR_REQUIRE_ESM]: require() of ES Module ...\test-bot\node_modules\lowdb\lib\index.js from ...\test-bot\dist\lib\database.js not supported.
Instead change the require of index.js in A:\Programación\HTML + CSS + JS\test-bot\dist\lib\database.js to a dynamic import() which is available in all CommonJS modules.

I don't know if this is intentional or a bug in the typescript configuration of the initial project.
Steps To Reproduce

Generate a new project using sapphire new
Select typescript-starter
Import an ES Module (I'm using lowdb)
Compile and see the error

Expected behavior
As @sapphire/ts-config has esModuleInterop enabled, I would expect to compile without problems.
Screenshots
No response
Additional context
Node version: 16.16.0
You can also see additional information on this stack overflow question.

Cannot reproduce without your database.ts file
That said, this is not a bug in the examples I can say that much. The examples are configured for CJS, not ESM. If you want to use ESM then you should:

Add "moduleResolution": "node16" to the tsconfig.json compilerOptions
Change "type": "commonjs", in package.json to "type": "module",
Add extensions to all the relative imports, i.e. in setups change import { srcDir } from './constants'; to import { srcDir } from './constants.js';

And if you don't want to use ESM then you don't need to have the const __dirname = dirname(fileURLToPath(import.meta.url)); line that lowdb has in their example.

I'm opening this issue in the examples because these examples are used as templates for the cli.
But it should work with esModuleInterop set to true as I said in the stack overflow question.
It works making the project a module. But I still don't know why with the esModuleInterop is enabled is not allowing to import ES modules.

esModuleInterop is completely unrelated to importing ES modules. In fact, it's to make CommonJS libraries work better. But even then it's entirely unrelated to your issue.
also see https://www.typescriptlang.org/tsconfig#esModuleInterop


how to package to .exe after obfuscate

Hi,
I am trying to package my code with pkg after obfuscating my code but getting
Dynamic require may fail at run time, because the requested file
  is unknown at compilation time and not included into executable.
  Use a string literal as an argument for 'require', or leave it
  as is and specify the resolved file name in 'scripts' option.

what's the best practice to create an output that be easy to create .exe file?

Hi. Right now you can use the conditional comments or reservedStrings to disable obfuscation of require.
But seems I have to create an option to ignore obfuscation of the strings inside require statements.

an option would be great because it has problem with almost all of my require imports.

I'll create the PR with this feature tomorrow

Released as ignoreRequireImports option in 2.8.0

it is working now. thanks!


Ignoring properties in generated json without using attributes

With swashbuckle, I did the following to remove the specified properties on every generated model. Is this possible using NSwag? It is the only thing I havent figured out how to do in my translation;
I am thinking it could maybe be done with ContractResolver and modifying the serialization, but is there and easier way?
`
services.AddSwaggerGen(c =>
 {
     c.SchemaFilter<SwaggerFieldFilter>();
 }

public class SwaggerFieldFilter : ISchemaFilter
{
    public void Apply(Schema model, SchemaFilterContext context)
    {
        if (model.Properties != null)
        {
            model.Properties.Remove("UniqueId");
            model.Properties.Remove("Hrid");
            model.Properties.Remove("CreatedBy");
            model.Properties.Remove("CreatedOn");
            model.Properties.Remove("ModifiedBy");
            model.Properties.Remove("ModifiedOn");
            model.Properties.Remove("OptimisticLockField");
        }
    }
}

`

You can implement that with a schema processor: https://github.com/RSuter/NJsonSchema/wiki/Schema-Processors
Or with a custom contract resolver (you have to register the resolver in the settings): https://blog.rsuter.com/advanced-newtonsoft-json-dynamically-rename-or-ignore-properties-without-changing-the-serialized-class/

Cool - I went with the contractresolver, it was easier than I expected. One gotcha though, the error message preventing use of a contractresolver and DefaultPropertyNameHandling together. The error message is probably a bit inaccurate - I tried just removing the setting, but that didn't work - neither did setting to null (non nullable type). In the end, I figured out it needs to be explicitly set to PropertyNameHandling.Default
My code based on your example is;
`
        app.UseSwaggerUi3(typeof(Startup).GetTypeInfo().Assembly, settings =>
        {
            settings.SwaggerRoute = "/swagger/v2/swagger.json";
            settings.GeneratorSettings.SchemaNameGenerator = new MySchemaNameGenerator();
            settings.GeneratorSettings.DefaultPropertyNameHandling = PropertyNameHandling.Default;
            settings.GeneratorSettings.IsAspNetCore = true;

            var jsonResolver = new PropertyModContractResolver();
            jsonResolver.IgnoreGlobalProperty("UniqueId", "Hrid", "CreatedBy");

            settings.GeneratorSettings.ContractResolver = jsonResolver;
            settings.GeneratorSettings.OperationProcessors.Add(new OperationSecurityScopeProcessor("JWT token"));
            settings.GeneratorSettings.DocumentProcessors.Add(
                new SecurityDefinitionAppender("JWT token", new SwaggerSecurityScheme
                {
                    Type = SwaggerSecuritySchemeType.ApiKey,
                    Name = "Authorization",
                    Description = "Copy 'Bearer ' + valid JWT token into field",
                    In = SwaggerSecurityApiKeyLocation.Header
                }));

        });

public class PropertyModContractResolver : DefaultContractResolver
{
    private readonly HashSet<string> _globalIgnores = new HashSet<string>();
    private readonly Dictionary<Type, HashSet<string>> _ignores = new Dictionary<Type, HashSet<string>>();
    private readonly Dictionary<Type, HashSet<string>> _includes = new Dictionary<Type, HashSet<string>>();
    private readonly Dictionary<Type, Dictionary<string, string>> _renames = new Dictionary<Type, Dictionary<string, string>>();

    public PropertyModContractResolver()
    {
    }

    public void IgnoreGlobalProperty(params string[] jsonPropertyNames)
    {
        foreach (var prop in jsonPropertyNames)
            _globalIgnores.Add(prop);
    }

    public void IncludeProperty(Type type, params string[] jsonPropertyNames)
    {
        if (!_ignores.ContainsKey(type))
            _ignores[type] = new HashSet<string>();

        foreach (var prop in jsonPropertyNames)
            _ignores[type].Add(prop);
    }

    public void IgnoreProperty(Type type, params string[] jsonPropertyNames)
    {
        if (!_ignores.ContainsKey(type))
            _ignores[type] = new HashSet<string>();

        foreach (var prop in jsonPropertyNames)
            _ignores[type].Add(prop);
    }

    public void RenameProperty(Type type, string propertyName, string newJsonPropertyName)
    {
        if (!_renames.ContainsKey(type))
            _renames[type] = new Dictionary<string, string>();

        _renames[type][propertyName] = newJsonPropertyName;
    }

    protected override JsonProperty CreateProperty(MemberInfo member, MemberSerialization memberSerialization)
    {
        var property = base.CreateProperty(member, memberSerialization);

        if (IsIncluded(property.DeclaringType, property.PropertyName)) property.ShouldSerialize = i => true;
        else if (_globalIgnores.Contains(property.PropertyName) || IsIgnored(property.DeclaringType, property.PropertyName))
            property.ShouldSerialize = i => false;

        if (IsRenamed(property.DeclaringType, property.PropertyName, out var newJsonPropertyName))
            property.PropertyName = newJsonPropertyName;

        return property;
    }

    private bool IsIgnored(Type type, string jsonPropertyName)
    {
        if (!_ignores.ContainsKey(type))
            return false;

        return _ignores[type].Contains(jsonPropertyName);
    }

    private bool IsIncluded(Type type, string jsonPropertyName)
    {
        if (!_includes.ContainsKey(type))
            return false;

        return _includes[type].Contains(jsonPropertyName);
    }

    private bool IsRenamed(Type type, string jsonPropertyName, out string newJsonPropertyName)
    {
        Dictionary<string, string> renames;

        if (!_renames.TryGetValue(type, out renames) || !renames.TryGetValue(jsonPropertyName, out newJsonPropertyName))
        {
            newJsonPropertyName = null;
            return false;
        }

        return true;
    }
}

`

You mean the "The settings DefaultPropertyNameHandling and ContractResolver cannot be used at the same time" exception?

Yes. That one.
It has to be explicitly set to default to avoid the exception.
Just a quick question while I have your attention - does the splitting types to individual files work for any of the typescript client code generating methods yet? I thought I saw there was a way to do it, but havent been able to find it.

Ok, I've updated the exception message to contain more information. Also I've added ContractResolverType setting to the CLI so that you can also define it when running the generator via CLI (see commits above).
At the moment, multiple file output is implemented in NJsonSchema (base of NSwag) but not completely implemented in NSwag (the generators must generate multiple artifacts instead of a single artifact with multiple files in it...) and also we have to expose it in the tooling (CLI, UI, etc.)

Cool. Is there a similar method for excluding an entire type from the model?

I think you can implement a primitive type mapper and map your type to an any schema:
https://github.com/RSuter/NJsonSchema/wiki/Type-Mappers#primitive-type-mapper


sbt 1.3.7 doesn't fully respect the crossPaths setting

steps

clone https://github.com/christian-schlichtherle/fun-io
update to sbt 1.3.7
+test:compile

problem
I can tell from the output that all Java projects now get compiled four times. Needless to say this is a waste of resources. What's wrong?
notes
I'm using sbt 1.2.8 in my open source project Fun I/O (https://github.com/christian-schlichtherle/fun-io). Fun I/O consists of multiple modules - alias "projects" in sbt parlance. All but one of these projects are pure Java code, so I have set crossPaths = false for them. The one Scala project needs to be cross-compiled for Scala 2.10, 2.11, 2.12 and 2.13 using crossScalaVersions := Seq("2.10.7", "2.11.12", "2.12.10", "2.13.1").
Now, if I do +test:compile everything works as expected: All the Java projects get compiled only once - only the one Scala projects gets cross-compiled for the four different Scala versions.

Still applies to SBT 1.3.9.


[Bug]: Index is gone after infinity restart

Is there an existing issue for the same bug?

[X] I have checked the existing issues.

Branch name
main
Commit ID
3efbdfa89a9d58a7f96d2f872c1753a46682bee0
Other environment information
No response
Actual behavior
Index is gone after infinity restart
Expected behavior
No response
Steps to reproduce
- Run fulltext_import_benchmark to create table, create fulltext index and insert some data.
- Run infinity db. It crash with following error:

[17:11:46.160] [5759] [error] Index ft_dbpedia_index doesn't exist
terminate called after throwing an instance of 'infinity::UnrecoverableException@infinity_exception'
what():  Index ft_dbpedia_index doesn't exist@src/storage/meta/entry/table_entry.cpp:208

Passed my test.

After PR #977, running cmake-build-release/benchmark/local_infinity/fulltext_benchmark --mode merge  two times. It print following error message:
[18:55:15.592] [2437] [error] Index ft_dbpedia_index doesn't exist
terminate called after throwing an instance of 'infinity::UnrecoverableException@infinity_exception'
  what():  Index ft_dbpedia_index doesn't exist@src/storage/meta/entry/table_entry.cpp:229
Aborted (core dumped)

Passed my test.


chore: Use PAT for releases

Pull Request Description
Description:
Go back to using PAT for releases since the master branch needs to remain protected
Related Issue:
Checklist

[x] I have added the appropriate prefix  to the pull request title
[x] The CI tests are passing for this pull request.
[x] The pre-commit run -a hooks are passing for this pull request.

This PR is included in version 0.4.1 :tada:


WIP: Stop leaking goroutines in newMultiWatch

This fixes a goroutine leak in cases where Watch() is called again without a new List() inbetween Watch() calls. This happens when one of the underlying watches returns a ConnectionRefused error. See https://github.com/kubernetes/client-go/blob/3fe2abece89efc7b3a2da0ff549525f20f435263/tools/cache/reflector.go#L286
Note that I stumbled upon this leak while fixing another bug; this probably has other implications, so I'll mark it WIP for now.

This happens when one of the underlying watches returns a ConnectionRefused error.
Note that watches are also closed by the API Server every few minutes. That's another case where Watch() is called again without a new List().

This happens when one of the underlying watches returns a ConnectionRefused error.

Note that watches are also closed by the API Server every few minutes. That's another case where Watch() is called again without a new List().

That's correct, but the leak is only triggered when Watch() fails and is not followed by either a call to Stop() or List(). The reflector will in fact call Stop() before calling Watch() again in that case.


Refactor project labels for better organisation

After looking at our labels, I'm thinking there might be a better way to organise our issues and PRs, which might also enable the "projects" boards to work better. So I spent five minutes throwing a short-ish list of what potential labels we could have, to potentially enable better automation in the future.
# Type labels categorise "what" an issue/PR is
type:announcement
type:bug
type:invalid
type:proposal
type:question

# Area labels refer to a high level view of the moving parts of the repo
area:infrastructure
area:interop # *might* be a module?
area:engine
area:tests

# Module labels are related to what part of NovelRT a given issue or PR falls under
module:animation
module:audio
module:ecs
module:dotnet
module:graphics
module:ink
module:input
module:maths
module:timing
module:timing
module:utilities
module:windowing

# Priority labels are how important we think something is (e.g. security bugs vs minor typos)
priority:low
priority:medium
priority:high
priority:critical

# Proposal labels are for tracking the status of proposals
proposal:approved
proposal:needs-review
proposal:ready-for-review
proposal:rejected

# Status labels indicate what state an issue or PR is in
status:available
status:blocked
status:in-progress
status:stale

# OS labels are examples; format is <name>-<arch>-<version>
os:browser # the exception; used to indicate webgl/wasm
os:debian-x64-10
os:debian-x64-9
os:macos-x64-10.12
os:macos-x64-10.11
os:windows-x64-1903
os:windows-x64-1803
os:ubuntu-x64-19.03
os:ubuntu-x64-20.04

# Browser labels are examples; format is <name>-<arch>-<version>
browser:firefox-x64-85.0.1

I'd like some feedback on these labels, so we can potentially look at implementing them.

We probably won't be supporting Debian 9 unless you're explicitly wanting to go out of your wait for that - we probably won't be supporting older versions of MacOS either because of how their tech stack works. Ubuntu 19.03 isn't supported either.
Other than this it looks good off the bat. :)


Add man page to crystal

@werner started this, I'm starting from where he left. Any comments are welcome.
Closes #1291.

Where may I find more information about crystal docs? I've got some problems running it on crystal repository, and couldn't find any description here.

I've made a bunch of changes. Is there something more you would like to see here?

... and so on. That would be the minimum level of verbosity I would expect from a manpage, to have a benefit over --help.

Any progress here, @jhass?

Perhaps we can merge this so we have a man page for others to fix up and contribute to instead of being forever stalled.

Thank you @dread-uo and everyone who reviewed! It took us a bit long, but here we are :).


fix stepper hasErrors to not bottom out at first error

Hey, I just made a Pull Request!
I had realisation after #16196 was merged that there is a bug in the recursive hasErrors function that is used by the Stepper 🤦 .
It was bottoming out at the first check for an __errors array.
This PR fixes that and adds some testing around hasErrors.
:heavy_check_mark: Checklist

[X] A changeset describing the change and affected packages. (more info)
[ ] Added or updated documentation
[X] Tests for new functionality and regression tests for bug fixes
[ ] Screenshots attached (for UI changes)
[X] All your commits have a Signed-off-by line in the message. (more info)

I deleted the changeset here, as it's a bug for unshipped work, so just going to merge ontop of the other changeset :)

thank you!


Detecting step editor unmount

Hi Bart,
when using a custom step editor in the react designer, i have the problem to loose the
current changes in the form when the user accidentally clicks into the editor space
and the stepeditor component gets unmounted.
I remember you wrote somewhere that this is currently not possible in the react version,
but do you have an idea how to work around this ?
Thanks a lot
Tilo

Hello @tskomudek,
basically you can always save the latest value in the step like:
<input type="text" ... onChange={e => setProperty('x', e.value)} />

So in the definition you will have allways the latest changes.
If you have a save step between changing values and applying them, then you can create a hidden property in the step, such as deltaX.
<input type="text" ... onChange={e => setProperty('deltaX', e.value)} />

During saving changes you need to move values to a final property (deltaX => x).

@tskomudek have you resolved the problem?

Not really.
Yes, i can store the modified data somewhere, but i'd rather ask the user if he wants to navigate away from the current step without saving. Do you have any other idea how to achive this ?
Thx
Tilo

It looks like you need a canCloseEditor callback or something like this. Yeah, it's not available now.
Maybe a temporary solution would be to use the isSelectable callback 🤔, something like:
isSelectable: (step) => {
   if (hasUnsavedChanges()) return false;
   // ...
}

When i am using sequential-workflow-designer in my angular app in router-outlet, its not taking full height and toolbox is not getting initialized.
i followed angular demo example

bb45)

When i integrated sequential-workflow-designer-angular in my existing project in place of router outlet but its not getting initialised  properly...svg not taking full height and toolbox not getting initialized

@ShraddhaFutrey it looks like the parent layers have not set any height constraints. The designer doesn't stretch parents, it adjusts to available space. You can check this theory by setting CSS:
.sqd-designer {height: 900px;}

BTW: please don't add comments to not related threads.

Hi Bart,
the isSelectable callback would help partially, but it doesn´t cover when the user just clicks into the canvas and the root editor appears or the user drags a new step from the toolbox.
An isUnselectable or canCloseEditor callback would be more suitable. Also, the onSelectedStepIdChanged event doesn't help here, since when it's called the stepeditor is already unmounted.
Maybe its worth considering at least.
Thanks
Tilo

@tskomudek yes, this feature is planned but with a low priority. If you need this feature faster you can request it under the consulting service.


finished linkedlist lab

Really nice work on the implementation. I would recommend trying to avoid short non-descriptive variable names.  Make sure you are filling out all the info in the Summary Comments.
Very nice work.

Really nice work on the implementation. I would recommend trying to avoid short non-descriptive variable names.  Make sure you are filling out all the info in the Summary Comments.
Very nice work.


Adding the title of the video in the metadata

This request comes from the necessity to orderly save the transcribed file with a relevant and recognizable name.
This addition should contribute to the API involvement in complex apps where you could for instance transcribe and save a batch of videos in an orderly manner.
I would like to have the metadata of the transcription include the title of the video as well.
The alternative is to fetch a YouTube Data API Key through Google Developers Console and use a different dedicated module to request it, which seems a bit and pardon my wording: "overkill."
In my case I'm trying to efficiently save and label video transcriptions file by minimizing the amount of time I have to manually interact with the code.
In my case for every video I have to manually change the file name with the title of the video.
Thank you for reading my request.

I am looking for this feature too, great to see someone came it up

This last days I've this error :
"message": "\nCould not retrieve a transcript for the video ! This is most likely caused by:\n\nSubtitles are disabled for this video\n\nIf you are sure that the described cause is not responsible for this error and that a transcript should be retrievable, please create an issue at https://github.com/jdepoix/youtube-transcript-api/issues. Please add which version of youtube_transcript_api you are using and provide the information needed to replicate the error. Also make sure that there are no open issues which already describe your problem!"
It was worrking before but not now, do you have any feedback ?
I'm using : trans = YouTubeTranscriptApi.get_transcript(yt.video_id,languages=('fr'))

yes this is needed


Terraform plan when PR is created

Shown as a comment. Example in the following repo https://github.com/hashicorp/setup-terraform

Will not do, as it needs access key to perform terraform plan. Will not allow that action to run on a PR as everyone can submit a PR that leaks the credentials.


patch esprima sha hash

What does this PR do?
Because we are using an older version of jshint, and it isn't using a specific esprima version we need to change the hash every now and again.
Screenshot
Related Issue

So if you just rerun npm install locally, it fixes the messed up package-lock.json file, which does this and updates the version, which I believe should be 1.5.134 for both. So this line here: https://github.com/boundlessgeo/MapLoom/blob/master/package-lock.json#L3 should be 1.5.134, right?
Other than that 👍

It should be, but for some reason Jenkins always sets package-lock.json to one behind.
Let me push an update to this branch...

I made the change to jshint which seems to fix the esprima hash issues here: https://github.com/boundlessgeo/MapLoom/pull/190


Rename initData property

Rename initData property to data. Sounds better, when we are able to edit dynamically it.
Breaking change, or we deprecate initData.

Good idea, I think initData is not represent a property. It is confusing name.
but remain the initData for the next version for backwards compatibility.  Just sync data and initData.
How about this?

Sounds good to me.

Using data in config with ^2.0.0


How to create new cuda stream in custom op

I've written a custom op for the GPU that takes X with shape (n,p) and Y with shape (m,p) and returns Z with shape (n,m).
In the backward pass, dX and dY are independent computations so I'd like to have launch two separate CUDA kernels and have them run in separate streams. I know the cuDNN RNN is capable of doing kernels in parallel but I cannot find the source code for how to do this. It seems the only usable stream is provided by the OpKernelContext.
Matrix multiplication must have a similar parallelism when doing the backward pass but I have not found anywhere in the source making use of multiple streams. This is essentially a reopening of this issue as there doesn't appear to a public answer

@chaaland, Please take a look at this doc. Thanks!

Thanks for the link. I've referenced this doc many times while developing the op but I don't see where it specifies how to generate two CUDA streams. The call to d.stream() just returns a stream that has already been created rather than creating a new stream (which is what I think I'd need).
I'm open to any other way to get the dX and dY computations running in parallel as well. Currently I have something like the below which appears to be running in serial
const cudaStream_t& cu_stream = GetCudaStream(ctx);
dx_bprop<<<griddim, blockdim, 0, cu_stream>>>(...);
dy_bprop<<<gridim, blockdim, 0, cu_stream>>>(...);

Any updates on this?

@gowthamkpr yes I'm familiar with the custom op repo. My issue is not with being able to compile a GPU operation as I've managed to get that working already. I'd like to know how (or if it's even possible at all) to launch two CUDA kernels in 2 different streams.
In CUDA this can be done with a simple malloc of a new stream and passing the string as the 4th argument to the kernel itself. TensorFlow seems to have its own management of memory and streams which I didn't want to interfere with by calling a CUDA malloc directly. The example given in the time_two_kernels.cu.cc shows just using a single stream d.stream() which is the case I have working currently. I notice there are other streams created for copying host to device, device to device and so on but I don't see the ability to have multiple streams for compute.
Is this just a fundamental limitation of TensorFlow? I can't find any examples of using multiple compute streams yet I can't think of a reason this shouldn't be possible given CUDA permits it

@gowthamkpr bump on this issue

@chaaland  afaik, currently TensorFlow runtime only creates a single compute stream per GPU device. Thought this might change in the future runtime, but I don't think anytime soon. cc: @iganichev

@yifeif is right. TensorFlow design today is fairly tied to the "there is a single compute stream" assumption. This is usually not a problem because CUDA kernels tend to operate on large enough pieces of data and utilize the whole GPU. Your use case will benefit from parallelism only if your data is too small. In this case, if this op is not a major part of the computation (e.g. done in a tight loop), you might end up saving just a few microseconds.
If you still believe that running in parallel will be beneficial enough, there are a couple of things you can try doing:

You can try putting both computations into the same kernel. A possible approach is to split your CUDA threads into two logical groups and make threads in group 1 do computation for dX and threads in group 2 do computation for dY.
A much more involved and likely slower approach is to create another stream in your kernel (you probably want to cache it somewhere), launch your work on both streams, synchronize the new stream to the one used by TensorFlow to make sure that the TensorFlow's stream will not proceed until all work on your stream is done.

@iganichev  Sounds like fusing the Op's backprop functor is the most straightforward approach. I was under the impression this would provide a perf benefit since as the thread blocks computing dX finish, threadblocks for dY could be scheduled. In my current implementation, all of the thread blocks for dX must finish before any work on dY can be done. Will just have to test and see if it makes any difference. Thanks for your help!


Log any exception thrown by hubspot task helpers

Pre-Flight checklist

[ ] Testing

[ ] Code is tested
[ ] Changes have been manually tested

What are the relevant tickets?
Closed https://github.com/mitodl/mitxonline/issues/1414
What's this PR do?
If any kind of exception occurs when a hubspot task helper function calls a celery task, log the exception instead of raising it.
How should this be manually tested?
(Required)
Where should the reviewer start?
(Optional)
Any background context you want to provide?
(Optional)
Screenshots (if appropriate)
(Optional)

Codecov Report

Merging #1416 (daef0cd) into main (ecc5d43) will increase coverage by 3.72%.
The diff coverage is n/a.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
@@            Coverage Diff             @@
##             main    #1416      +/-   ##
==========================================
+ Coverage   83.19%   86.91%   +3.72%
==========================================
  Files         309      146     -163
  Lines       13201     4325    -8876
  Branches      939        0     -939
==========================================
- Hits        10982     3759    -7223
+ Misses       1949      566    -1383
+ Partials      270        0     -270

Impacted Files
Coverage Δ

hubspot_sync/task_helpers.py

authentication/urls.py

courses/admin.py

courses/api.py

...iblepricing/management/commands/configure_tiers.py

mail/exceptions.py

ecommerce/discounts.py

ecommerce/__init__.py

main/oidc_provider_settings.py

openedx/utils.py

... and 153 more

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


ERROR: SSL_write, code: 336195711

Hi,
I/m encountering this error when I enable TLS in linux 14.04
I'm using swift 3.0.1 and kitura 1.1.2
It's working if TLS is disabled.
VERBOSE: run() Kitura.swift line 71 - Starting Kitura framework...
VERBOSE: run() Kitura.swift line 73 - Starting an HTTP Server on port 8090...
INFO: listen(on:) HTTPServer.swift line 70 - Listening on port 8090 (delegate: SSLService.SSLService)
WARNING: init() ContentType.swift line 44 - Loading embedded MIME types.
ERROR: handleWriteHelper() IncomingSocketHandler.swift line 194 - Write to socket (file descriptor 7) failed. Error = Error code: 336195711, ERROR: SSL_write, code: 336195711, reason: bad write retry.
code:
let myCertPath = "/opt/phoenix/config/certificate.pem"
let myKeyPath = "/opt/phoenix/config/key.pem"
let mySSLConfig = SSLConfig(withCACertificateFilePath: nil, usingCertificateFile: myCertPath, withKeyFile: myKeyPath, usingSelfSignedCerts: true)
Kitura.addHTTPServer(onPort: 8090, with: router, withSSL: mySSLConfig)

Fixed in BlueSSLService v0.11.56.  Refreshing your dependencies should get the fix.

BlueSSLService is now in v0.11.56 but still the same error
ERROR: handleWriteHelper() IncomingSocketHandler.swift line 194 - Write to socket (file descriptor 7) failed. Error = Error code: 336195711, ERROR: SSL_write, code: 336195711, reason: bad write retry.

I ran the exact same test here before posting the fix and I can't reproduce it.  The only difference was I was using my certificates and running on Ubuntu 16.04.  Can you provide a bit more information?  What request triggered the error?  Or did the error appear immediately?  If it appeared immediately, it's likely a problem with your certs.  Any more information you can provide would be extremely helpful.  Thanks in advance.

Been doing more research into this problem.  Turns out the fix I put in is good but only for non-blocking sockets.  I need to make some more changes.

The complete fix for this is now in.  However, it resulted in a version change for BlueSSLService and BlueSocket.  Kitura-net depends on these and will need an update to Package.swift before the changes will be picked up by Kitura.  I've tested the changes here (again on 16.04) and can not reproduce your problem.  I'd still like to get more information from you regarding what triggered the error.  Thanks.

Fixed as part of IBM-Swift/Kitura#858 per @billabt.

I'm still having this issue with latest Kitura 1.6.1. However it seems like it only happens when I try to request content over the static file server that is more than 100KB (like bootstrap.min.css). Hope that helps.

@foldericon This looks to be the same issue as #1036
I've submitted a pull request (IBM-Swift/BlueSSLService#21) to BlueSSLService to fix the issue. Please let us know if this resolves the issue. Thanks!

@foldericon This has been fixed in SSLService-0.12.29. Please verify it works ok now by deleting your Packages directory and fetching/building again. Thanks!


ALM Config number needs to be a string

fixing https://github.ibm.com/ibmprivatecloud/roadmap/issues/38731

/lgtm


Implement Replicator listeners

CBLReplicator_AddChangeListener, CBLReplicator_AddDocumentListener

Is this still open?

I can create a pull request if it's still open

Yes, it's still open. PR welcome :) You can look at the query listener implementation as a guide: use the same internal API found in Listener.hh. That way the threading will be managed correctly.


Execute multiple commands in RPC console of GUI

Hello, I'm using Qtum Core GUI in Windows 10 64bit.
I'm trying to execute multiple command in RPC console in Debug window for addnode command.
For example,
addnode "123.456.789.012:3888 " add
addnode "111.222.333.444:3888 " add
But, Qtum Core GUI is shut down suddenly.
So, I'm thinking to use qtum-cli.exe to attach rpc console to GUI node.
Is there any way to attach it? or execute multiple commands in rpc console in GUI is also fine.
Thank you always.

I think you want to use the qtum.conf file (see attached), in the Qtum directory, which qtum-qt will load at startup. You will have to experiment with this – I haven’t tried it. Rename this file as "qtum.conf" and edit the addnode lines to remove the comment and put your IP address in.
qtum.conf.txt

Thank you! I will try it.

It works really well!
Thanks~


No https support?

It looks like the plugin does not on pages served via https, because browsers block ajax request to http://api.kudosplease.com/"+q
Is there a way to fix it for my users?

+1 for https support. I use KudosPlease on my github-hosted jekyll site. Right now the button works with http, but not with https:
Works: http://fredrikaverpil.github.io/2015/12/28/python-web-server/
Does not work: https://fredrikaverpil.github.io/2015/12/28/python-web-server/

+1

Hi,
I wrote a clone in Golang. There is a host at https://kudos.niels-ole.com/ (also with http support).
You can simply replace the references to the server in the JS with that address.
I wrote two lines at https://www.niels-ole.com/kudos/blog/2016/11/04/now-with-kudos.html
Will release the code shortly.

+1
using niels-ole https endpoint now. Cannot import old kudos :(

@nielsole Thanks for HTTPS support
Also can you implements total website kudos
https://kudos.niels-ole.com/?url=https://miteshshah.github.io/windows/how-to-change-windows-7-product-key/
22

https://kudos.niels-ole.com/?url=https://miteshshah.github.io/
0

Ideally we should return all the kudos for the same domain.

Hi,
the feature is now live. e.g.: https://kudos.niels-ole.com/?domain=miteshshah.github.io
It would be best if further issues and discussions would take place at https://github.com/nielsole/kudos

@nielsole Sure Not aware of that repo https://github.com/nielsole/kudos
Thanks for the quick implements :)

@nielsole HTTPS expired can you renew it

yeah, migrating servers right now, will fix it probably today or tomorrow

Thanks for the quick fix


function Plug.Conn.Query.encode/1 is undefined!!

HI guys,
(UndefinedFunctionError) function Plug.Conn.Query.encode/1 is undefined (module Plug.Conn.Query is not available)
Thanks

Mailgun requires Plug to work, sorry.

https://github.com/swoosh/swoosh/issues/202
https://github.com/swoosh/swoosh/pull/219


Is there any guide how to use this?

Hi,
Does anyone know how to use this extension??

@murugappanrm Hi, please see the readme.

Hi @wenfangdu
Thank you for your reply. I finally figured what this extension does. It is intended to take a part of the code in a file and generate a snippet which them copied to clipboard. I had to paste the generated snippet and make changes if needed then save. I still working on it.
Perhaps the documentation could have been more elaborate with explanation. I am sure you understand that not all of us are as well versed with vscode and snippets.
Thank you so much.


onDataReceived native module listener not removed when calling .remove()

Hey man. You updated the js file (compiled) not the ts file (source).

Thanks @kenjdavidson , I've made the changes in the ts file now.

Ondatarecieved should be updated to use createBluetoothEventSubscription like the rest of the on handlers.
Remove: () =>
Is correct that way.


Implement NumPy trapz

Feature request

From: https://stackoverflow.com/questions/52919039/how-to-use-multi-threading-to-speed-up-nested-for-loop-calculation/52919441#52919441
numpy.trapz is not supported yet.

implemented in #3627, closing.


Incorrect sorting with col with rowspan

First, thanks again for this .js. It works great.
Recently I create a table with a rowspan and notice the sort wasn't work. It was sorting the next left column instead of the column clicked. Tried the last .zip here but still the same. The issue bellow says the same:
#52
Can you take a look if you have a time? Thanks again.

can you make an example in jsFiddle, please?

i will close the issue now, feel free to reopen it, but I will need some example where it doesn't work to be able to fix it (on the demo page it works correctly)