Some H100/A100 systems are not correctly identified by auto detection

The H100 profiles do not match the expected code to file name translation logic since this card reports as:
_CudaDeviceProperties(name=‘NVIDIA H100 80GB HBM3’, major=9, minor=0, total_memory=80994MB, multi_processor_count=132)
There are several H100 variants:
0x2321, 0x1839, 0x10de, "NVIDIA H100 NVL"
0x2330, 0x16c0, 0x10de, "NVIDIA H100 80GB HBM3"
0x2330, 0x16c1, 0x10de, "NVIDIA H100 80GB HBM3"
0x2331, 0x1626, 0x10de, "NVIDIA H100 PCIe"
0x2339, 0x17fc, 0x10de, "NVIDIA H100"
A full list of all Nvidia card identifiers can be obtained in the open drivers (scroll past the nulls):
https://github.com/NVIDIA/open-gpu-kernel-modules/blob/main/src/nvidia/generated/g_nv_name_released.h

We are also seeing this issue affect A100 cards

Hey, I think a long term solution to this is to introduce metadata to the config: https://github.com/instructlab/instructlab/pull/2713
Rather than reading file names, as we scale and want a config to apply to a specific GPU or a config to apply other a set of GPUs, we will need more granular control over the profile system. I think #2173 (I will add this to the profile system in that PR), is the right path forward

Hey, I think a long term solution to this is to introduce metadata to the config: #2713
Rather than reading file names, as we scale and want a config to apply to a specific GPU or a config to apply other a set of GPUs, we will need more granular control over the profile system. I think #2173 (I will add this to the profile system in that PR), is the right path forward

I agree that generally moving away from filename based operations is a good thing
though im not sure I see how that alone will address the issue of not matching the card name perfectly. We will still need some heuristic to properly categorize the card name, we will now just be checking against a field inside the file rather than the name of the file

@jaideepr97 , this heuristic will allow us to store specific types of GPU information in the config.
The PR isn't ready yet, but the SKU field will end up probably being a list like [PCIe, ...] Where if the gpu matches any of the sku information, and the count and type match as well then we can apply the config to the system.
This allows us to apply configs as specifically or as broadly as we want.

The application and matching system will also be much much easier and sustainable

Related to https://github.com/instructlab/instructlab/issues/2489


Issue Tracking 'GitHub' too magical

Some specs list GitHub as an issue tracking source. E.g. bikeshed documentation, but they do not use the Issue Tracking metadata. At times when bikeshed is NOT generating GitHub as an issue tracking option it is confusing as to why not - when other specs do without using the Issue Tracking metadata.
Perhaps update documentation for Issue Tracking to describe when GitHub will be implicitly added? Or, add explicit use examples at least to BS doc?

Somewhat related: The generated spec for this file using the w3c bikeshed website does not include the Github issues link but when generating the file locally the link is included:

The relevant field in the metadata is Repository. If set to a Github repository (implicit if value is org/repo), then Github will be listed in the Issue Tracking section, c.f. https://github.com/tabatkins/bikeshed/blob/master/bikeshed/MetadataManager.py#L129
@g-ortuno If you add Repository: WebBluetoothCG/web-bluetooth to your metadata, the Web service will generate the entry.

@kba Thanks! It would be nice if this were documented.

Done now, sorry for the lack!


When assigned to multiple reviewers, the PR is not listed

Issue Type: Bug
When a PR is assigned to multiple reviewers, it is not shown in the list of pull requests in the extension.
VS Code version: Code 1.27.1 (5944e81f3c46a3938a82c701f96d7a59b074cfdc, 2018-09-06T09:14:32.385Z)
OS version: Darwin x64 17.7.0
Steps to Reproduce:

Assign a PR to two reviewers.
Check the list in the GITHUB PULL REQUESTS section on any of the reviewers' vscode extension.

That's interesting, the query we're using for "Assigned to Me" is assignee:${user}, which I would expect to work in this case. Apparently not. I'll take a look

I haven't been able to reproduce this - when I assign a PR to multiple people including myself, I see it appear under "Assigned To Me". Are PRs correctly appearing under "All"?

Yes, they are... take a look:

You are right. Even when I add a comment it stops showing. I removed the comments and it shows again. I assume it's the expected behavior so I'm proceeding to close the issue.
Thanks a lot for your insight!


Adds thread-safety and improves compatibility with Ruby 2 and Rails 5

@BaylorRae

This looks great! I need to update the travis build to use the current versions of Ruby and and will get this merged in. Thank you!!


Docs: Explain how middleware can execute only for routes

The Fresh documentation seems slightly misleading with regards to when Middleware code is executed. The docs state the following:

Route middleware
A middleware is defined [...] to perform custom logic before or after the route handler.

But when generating a new Fresh project and creating a middleware that logs each Request.url, we see URLs logged from files that are not "routes" (i.e. under the routes/* folder):
GET requests for /

http://localhost:8001/
http://localhost:8001/_frsh/js/edcc08a7371927e996ddee1a948a0cbd0df68a05/deserializer.js
http://localhost:8001/_frsh/js/edcc08a7371927e996ddee1a948a0cbd0df68a05/signals.js
http://localhost:8001/_frsh/js/edcc08a7371927e996ddee1a948a0cbd0df68a05/chunk-PDMKJVJ5.js
http://localhost:8001/_frsh/js/edcc08a7371927e996ddee1a948a0cbd0df68a05/chunk-UGFDDSOV.js
http://localhost:8001/_frsh/js/edcc08a7371927e996ddee1a948a0cbd0df68a05/plugin-twind-main.js
http://localhost:8001/_frsh/js/edcc08a7371927e996ddee1a948a0cbd0df68a05/main.js
http://localhost:8001/_frsh/js/edcc08a7371927e996ddee1a948a0cbd0df68a05/chunk-FJBXTIC3.js
http://localhost:8001/_frsh/js/edcc08a7371927e996ddee1a948a0cbd0df68a05/island-counter.js
http://localhost:8001/_frsh/refresh.js
http://localhost:8001/logo.svg?__frsh_c=edcc08a7371927e996ddee1a948a0cbd0df68a05
http://localhost:8001/_frsh/alive
http://localhost:8001/favicon.ico

Non-route requests

/_frsh/* for client Javascript and Dev socket (x10)
Files from the /static folder (favicon.ico and logo.svg)

Motivation
I think there are great cases for both cases; running middleware only for /routes and running for all requests (including static files and files served by the Fresh runtime).
Running for all requests

Logging
Custom headers
Access control (hiding access to images for anonymous users etc)

Running only for /routes/*

Setting (session) cookie(s)
Notifications/Other UI status (from e.g. Deno KV)
Retrieving navigation-tree (often loaded from other systems in E-commerce)

For the latter scenarios, it's common to add data to ctx.state so that it's available to routes when rendering the app. But it would be very bad if these run for e.g. Javascript files.
export async function handler(req: Request, ctx: MiddlewareHandlerContext<State>) {
  const [notifications, navTree] = await Promise.all([
    getUserNotifications(req.headers),
    getNavigationTree()
  ];
  ctx.state.notifications = notifications;
  ctx.state.navTree = navTree;
}

Feature request
It would be great if it was possible to somehow configure that /routes/_middleware.ts will run only for all routes in the /routes folder, and ignore static/fresh files. I think the most intuitive thing could be that "global" middleware is configured in main.ts as StartOptions and /routes/_middleware.ts only handles routes, but that would be a breaking change.
If it's out of scope to separate global/route middleware, then it would be nice to have some utility/helper method that can help filter out requests for static/fresh files (see workaround below).
Current workarounds
The best I've come up with so far to identify "route" requests for middleware is:

Ignore URLs that start with /_frsh/
If HTTP method is GET, check headers to determine if top-level navigation, using one of the below options

Sec-Fetch-Mode header is navigation
Sec-Fetch-Dest header is document
Accept header has text/html

The above is not perfect, but at least it seems to mostly work okay, and avoids extra network requests for static and fresh-generated files.

Does https://github.com/denoland/fresh/pull/1123 solve your problem? Although I see this isn't mentioned in the docs, so I suppose if 1123 solves your problem, then this request should be changed to "document middleware destination feature".

Oh, I didn't know about ctx.destination! So I guess ctx.destination === 'route' would perfectly match my use case. Yeah, #1123 should match perfectly. I read through all documentation about middleware and checked available properties on the context object, but I guess I didn't immediately connect "destination" to filtering out request based on type.
I think it would be great if documentation would be a bit clearer that by default middleware runs for all requests, but that there is a option to easily filter them to run only for e.g. routes via ctx.destination.
Should I update the issue to reflect this?

I think it makes sense to change the title of this issue. And are you saying that my PR isn't quite clear enough? Should I add something else or change the wording?

I think it makes sense to change the title of this issue. And are you saying that my PR isn't quite clear enough? Should I add something else or change the wording?

No, I meant since basically the entire issue body was written based on the assumption there was no easy way to determine routes, maybe it was better to create a new issue than edit everything. I wasn't commenting on your PR. I've updated the title at least to be more clear...

Ok, thanks for clarifying. Glad you're pleased with the wording. Hopefully the addition will save people time in the future 🤞


Unable to connect to Storage Account

Preflight Checklist

[X] I have installed the latest version of Storage Explorer.
[X] I have checked existing resources, including the troubleshooting guide and the release notes.
[X] I have searched for similar issues.

Storage Explorer Version
1.27.1
Regression From
1.27.0
Architecture
i86
Storage Explorer Build Number
No response
Platform
Windows
OS Version
No response
Bug Description
Unable to retrieve child resources.
Details:
{
"name": "TypeError",
"message": "Cannot read properties of undefined (reading 'version')",
"stack": "TypeError: Cannot read properties of undefined (reading 'version')\n    at connectionIsV1 (C:\Users\Daniel Glynn\AppData\Local\Programs\Microsoft Azure Storage Explorer\resources\app\out\app\renderer\common\CustomConnections\IConnectionInfo.js:8:24)\n    at C:\Users\Daniel Glynn\AppData\Local\Programs\Microsoft Azure Storage Explorer\resources\app\out\app\renderer\common\CustomConnections\CustomConnectionStore.js:235:74\n    at Array.filter ()\n    at #sendTelemetry (C:\Users\Daniel Glynn\AppData\Local\Programs\Microsoft Azure Storage Explorer\resources\app\out\app\renderer\common\CustomConnections\CustomConnectionStore.js:235:14)\n    at async CustomConnectionStore.getCustomConnectionStore (C:\Users\Daniel Glynn\AppData\Local\Programs\Microsoft Azure Storage Explorer\resources\app\out\app\renderer\common\CustomConnections\CustomConnectionStore.js:44:9)\n    at async PersistentStorage.CustomConnections.getCustomConnections (C:\Users\Daniel Glynn\AppData\Local\Programs\Microsoft Azure Storage Explorer\resources\app\out\app\renderer\providers\CustomConnectionsProvider.js:10:39)\n    at async Provider.executeOperation (C:\Users\Daniel Glynn\AppData\Local\Programs\Microsoft Azure Storage Explorer\resources\app\out\app\providers\Provider.js:28:16)\n    at async ElectronRendererHostProxy._handleExecuteRequest (C:\Users\Daniel Glynn\AppData\Local\Programs\Microsoft Azure Storage Explorer\resources\app\out\app\renderer\ElectronRendererHostProxy.js:213:35)\n    at async ElectronRendererHostProxy._handleMessage (C:\Users\Daniel Glynn\AppData\Local\Programs\Microsoft Azure Storage Explorer\resources\app\out\app\renderer\ElectronRendererHostProxy.js:156:24)"
}
Steps to Reproduce

Launch Storage Explorer
Select "Storage Accounts"

Actual Experience
There used to be several dropdowns within "Storage Accounts" that when attempting to access produce the detailed error message
Expected Experience
No response
Additional Context
No response

Duplicate of #6399


[Switch] property onClick doesn't exist

Hello.
I'm trying to use the latest beta of material-ui and have some issues with Switch component.
I've imported Switch component and added it to my own Component. I defined it behaviour via onClick event:
<Switch                   onClick={this.handleToggle(value)}                   checked={this.state.checked.indexOf(value) !== -1}                 />
webpack keeps throwing a following error:
error TS2339: Property 'onClick' does not exist on type 'IntrinsicAttributes & SwitchProps & StyledComponentProps & { children?: ReactNode...'.
I managed to fix this problem defined onClick? property in Switch definition file. Still, i guess it's some kind of a hack.

I'm using react 15.4, typescript 2.5.3.

Sorry guys, it seems i read the docs wrong, cuz now i see there's a onChange event instead of onClick


Validation with 'validator' library

I saw that your library does not support validations using the 'validator' library, there are people like me who have no idea how to work with the yup library that your library supports at the moment, I suggest you do a integration with at least one more validation library.

Hey, why haven't you followed the enhancement template when opening a new issue? 🤔
Please, add more information about the library and at least the GitHub link,

Firstly I wanna ask you for forgiving me, because I didn't read rules about issues.
Here you have the link >: https://github.com/validatorjs/validator.js?files=1

No problem, just make sure you always follow a template while creating an issue.

To integrate with validator, you can use their custom methods on the form submit.
I created an example on CodeSandbox, check it out.
If you have further questions, just reach out.

Thank you so much, I wish success for  and your company.


International Domains

Can you have a domain from a different country registered to a US (or other) tenant?
Example: A customer has a US tenant but own a .com.ve domain from Venezuela.  Can they register this domain with the US tenant?

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: 6dc99803-4432-cd7c-4747-0953276298d9
Version Independent ID: 1d7554f1-8b1c-dd3a-18e6-1106ad2b3bcb
Content: Domains FAQ
Content Source: Office365-Admin/setup/domains-faq.md
Service: o365-administration
GitHub Login: @pebaum
Microsoft Alias: pebaum

@AndySanchezVM - Thank you for submitting feedback.
I think the best way forward is if you can open a service ticket in your tenant so this can get resolved ASAP. Based on outcome, let me know if it is something that can be called out in the docs.
If you don't mind please keep us posted here on the resolution and I really want to get this information you are discovering back into the docs.

@mypil - I submitted an advisory ticket with Microsoft with our Partner Benefits and received confirmation that you can add any domain to your Office 365 account as long as you validate ownership through TXT and MX records.  You can keep  this closed. Thank you!


fix: [email report] add multiple schedules per hour

SUMMARY
when there are multiple schedules per hour (for example */20 * * * *), currently schedule_window function can only dispatch 1st schedule, other schedules seems can not being consumed.
ping @mistercrunch I just found break line was added by:
https://github.com/apache/incubator-superset/pull/11414/files?diff=split&w=1#diff-45c6af05cf2a7764a43dfe33d2f4686d5d2e5487b4684b58756c8f4071f4c2ddL806
is there any reason to break here?
TEST PLAN
CI and manual test
cc @ktmud @bkyryliuk

Codecov Report

Merging #11414 into master will decrease coverage by 5.58%.
The diff coverage is 0.00%.

@@            Coverage Diff             @@
##           master   #11414      +/-   ##
==========================================
- Coverage   66.49%   60.90%   -5.59%
==========================================
  Files         860      415     -445
  Lines       40869    25858   -15011
  Branches     3686        0    -3686
==========================================
- Hits        27176    15750   -11426
+ Misses      13592    10108    -3484
+ Partials      101        0     -101

Flag
Coverage Δ

#cypress
?

#javascript
?

#python
60.90% <0.00%> (-1.04%)
:arrow_down:

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

Impacted Files
Coverage Δ

superset/tasks/schedules.py
75.00% <0.00%> (-1.37%)
:arrow_down:

superset/db_engines/hive.py
0.00% <0.00%> (-85.72%)
:arrow_down:

superset/databases/commands/create.py
31.91% <0.00%> (-59.58%)
:arrow_down:

superset/db_engine_specs/hive.py
53.90% <0.00%> (-30.08%)
:arrow_down:

superset/views/database/mixins.py
59.64% <0.00%> (-22.81%)
:arrow_down:

superset/db_engine_specs/presto.py
69.76% <0.00%> (-12.32%)
:arrow_down:

superset/db_engine_specs/mysql.py
79.59% <0.00%> (-12.25%)
:arrow_down:

superset/databases/commands/update.py
85.71% <0.00%> (-8.17%)
:arrow_down:

superset/databases/api.py
83.49% <0.00%> (-7.08%)
:arrow_down:

superset/databases/dao.py
94.11% <0.00%> (-5.89%)
:arrow_down:

... and 463 more

Continue to review full report at Codecov.

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

@mistercrunch can you take a look this PR?

I have been running reports that are delivered to slack every 5 min for quite some time, curious why that's the case.
I would encourage to add unit test that demonstrates this issue if possible.

@graceguo-supercat
I have been running reports that are delivered to slack every 5 min for quite some time, curious why that's the case.
I would encourage to add unit test that demonstrates this issue if possible.

Are you using email dashboard report or email alert?
email_reports.schedule_hourly by default set beat once per hour,
alerts.schedule_check has no default setting. but if you set beat once per minute you won't hit this issue.

I'm guessing my intention with the break was to avoid sending more than one alert per tick. Say if you get Celery beat to eval things every minute and someone by mistake adds a "multiple times per second" cron setting. I think we could accomplish something similar with a floor on the grace_period.

EMAIL_REPORTS_CRON_RESOLUTION was designed for this purpose:
https://github.com/apache/incubator-superset/blob/cb3f649a7f1ce19c6e1195b269f70114b6a4c788/superset/tasks/schedules.py#L762
Dashboard/ chart email report used this configuration:
https://github.com/apache/incubator-superset/blob/cb3f649a7f1ce19c6e1195b269f70114b6a4c788/superset/tasks/schedules.py#L828
But email alert has set resolution = 0:
https://github.com/apache/incubator-superset/blob/cb3f649a7f1ce19c6e1195b269f70114b6a4c788/superset/tasks/schedules.py#L841
(But I feel every 1 min beat is way too often)

I will merge this PR in, and will add similar config like EMAIL_REPORTS_CRON_RESOLUTION for email alert.

I have been running reports that are delivered to slack every 5 min for quite some time, curious why that's the case.
I would encourage to add unit test that demonstrates this issue if possible.

@bkyryliuk Are you using email dashboard report or email alert?
email_reports.schedule_hourly by default set beat once per hour,
alerts.schedule_check has no default setting. but if you set beat once per minute you won't hit this issue.

both alerts and emails.

I have been running reports that are delivered to slack every 5 min for quite some time, curious why that's the case.
I would encourage to add unit test that demonstrates this issue if possible.

@bkyryliuk Are you using email dashboard report or email alert?
email_reports.schedule_hourly by default set beat once per hour,
alerts.schedule_check has no default setting. but if you set beat once per minute you won't hit this issue.

both alerts and emails.

How frequent is the celery config to delivery delivered to slack every 5 min?


Debug data import process for West Somerville

Data import process succeeds for students.txt, assessments.txt, and behavior.txt.
Fails for educators.txt:

`take!': ActiveRecord::RecordNotFound (ActiveRecord::RecordNotFound)
from /app/app/importers/file_importers/educators_importer.rb:23:in `import_row'

Looks like the issue involves a missing homeroom model

The missing homeroom in question is WSNS 218.

WSNS 218 is the homeroom for the gym teacher at West Somerville. There are no students assigned to this homeroom, so no homeroom was created during the students.txt import. According to feedback from Uri, the gym teacher does not need Student Insights access.
Looks like the staff data at WSNS is different from Healey, in that all Healey staff with homerooms in X2 were classroom teachers. Will need to tweak our domain model of staff members and homerooms accordingly.


{"code":400,"error_code":"bad_oauth_callback","msg":"OAuth state parameter missing"}

Hello! I've been trying to revive lyntr and i always get this after the discord OAuth2:
{"code":400,"error_code":"bad_oauth_callback","msg":"OAuth state parameter missing"}
I followed FaceDev's Steps Correctly, and still no progress, someone help?

Hey @wetneybor, can you please confirm that this was meant for this repo (postgres.new) vs. Supabase? I'm having trouble seeing how this is related!

i didnt find anywhere that someone will answer so this is my last hope

@wetneybor feel free to create an issue on the Supabase repo:
https://github.com/supabase/supabase/issues/new/choose and the team will triage it!
I'll close this now since this is unrelated to this project.


ForceInputInt Error messages mess up the previous line.

EasyUtilities.cs [Line: 53]
This has to do with cursor position manipulation.
The goal is to send the error message and then send the cursor back up to the line where we previously entered input.
After that we want to make sure to move the cursor back down.
One alternative for fixing this is to just not manipulate the cursor position and send the errors on a new line every time.
That works and has been proven, but I don't like it, so that's not the end goal.
End result should be:
Enter Input: ________ ( Clear this and go back here if error, try again)
Show error here (Go here after valid input if no error)
(Go here after valid input if error)
Also: The solution for this should 100% be some kind of method that can be reused and maybe even passed on to any users of the library in the Utilities.

This was fixed yesterday (or the day before yesterday).
Forceinput now removes the previous input and redraws the prompt above the error message.
Should you proceed with the program and there is a error message bellow the input, that message will be cleared and the program continues as normal.
Looks very neat in use.


Allow excluding just the action name

Frizbee allowed to exclude the whole action name including the tag e.g. actions/checkout@v4. But this is not user-friendly for cases where the exclude would be upgraded over time as the exclude would have to be kept in sync.
Let's also support the case where the exclude is just in the form of owner/action_name.

@JAORMX sorry but can you re-approve? There was a copy-paste bug in the tests, nothing else had changed.


Enable gh dynamic plugin

Closes https://issues.redhat.com/browse/FLPATH-1791
Starting RHDH 1.3 some plugins are disabled by default, we were using the gh plugin so this change broke our workflow template. See https://docs.redhat.com/en/documentation/red_hat_developer_hub/1.3/html-single/release_notes/index#removed-functionality-rhidp-3187
This PR explicitly enables the gh plugin.

/retest

/retest

Looks good to me.

LGTM
@gabriel-farache can you please double check commits? There is a merge commit in the list and I'd like to be sure that it's ok. Thanks :)

@gciavarrini I rebased on main so the merge commit added by gh when syncing is gone


pass no module estimator

what's more modern than module from babel?

what's more modern than module from babel?

after bugfixes

after bugfixes

better at 6 % now #3

better at 6 % now #3


z-merge.yml

Support backsyncs from v1

@z-merge it

@z-merge it

We don't have fancy bots here.

@z-merge it

@dobladov now we do


[Event Hubs] Allow users to configure retry options in send requests

Allow users to configure retry options in send requests as described in https://github.com/azure/azure-sdk-for-js/issues/2661#issuecomment-491565934 for the below 2 operations on EventHubClient

send()
sendBatch()

Using retries for the creation of sender link is covered in #2835

close with https://github.com/Azure/azure-sdk-for-js/pull/2997


wistia to youtube migrate phase 2

Harness Developer Pull Request
Thanks for helping us make the Developer Hub better. The PR will be looked at
by the CODEOWNERS.
What Type of PR is This?

[ ] Issue
[ ] Feature
[x] Maintenance/Chore

If tied to an Issue, list the Issue(s) here:

Issue(s)

House Keeping
Some items to keep track of. Screen shots of changes are optional but would help the maintainers review quicker.

[x] Tested Locally
[ ] Optional Screen Shoot.

@ravilach There are still some wistia videos left in HDH.

Please check the Execution Link of the Pipeline for the Website Draft URL. This is located in the Preview Step behind the Harness VPN and also is available in #hdh_alerts. E.g Website Draft URL: https://unique-id--harness-developer.netlify.app. Current Draft URL is: https://6566a3744d4d4505f8309b4d--harness-developer.netlify.app


GC crashes of vm/dart/gc tests on arm64-linux

The tests
vm/dart/gc/fragmentation_typed_data_test/0 Crash (expected Pass)
vm/dart/gc/fragmentation_typed_data_test/1 Crash (expected Pass)
vm/dart/gc/fragmentation_typed_data_test/4 Crash (expected Pass)
vm/dart/gc/fragmentation_typed_data_test/5 Crash (expected Pass)
vm/dart/trigger_gc_in_native_test/1 Crash (expected Pass)

are failing on configurations
vm-linux-release-arm64

It's a little unclear why it's crashing. I don't have arm64 HW atm to try to reproduce.
It may also that this has been happening for a while, but was re-surfaced after test renames in ac77af1c077a9e37e489efd54c8e398b9297bd96

These all ended with -9 / SIGKILL. I suspect from the OOM killer: I notice the bot is timing out purple which might be from low memory / paging.


Allow empty list as AccountCreateExtension(s) -- for real this time.

Previous PR #90 erroneously adjusted Buyback_options instead of AccountCreateExtensions. :( Terribly sorry about that. This is the correct fix.

Codecov Report

Merging #106 into develop will not change coverage.
The diff coverage is 100%.

@@           Coverage Diff            @@
##           develop     #106   +/-   ##
========================================
  Coverage    59.41%   59.41%
========================================
  Files           37       37
  Lines         3501     3501
========================================
  Hits          2080     2080
  Misses        1421     1421

Impacted Files
Coverage Δ

bitsharesbase/objects.py
77.41% <100%> (ø)
:arrow_up:

Continue to review full report at Codecov.

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


No space between words in bulleted list

First of all, thanks for creating such a great template!
When copying lines that start with a bullet point, Mac apps such as Skim and Preview remove all spaces between words. For example: "Sampleclients:Apple;Facebook;Google", rather than "Sample clients: Apple; Facebook; Google"
EDIT: The following comment on an Adobe forum makes me think that the space between the bullet point and the start of the text throws off Skim/Preview and causes it to not recognize the comparatively smaller distances between words as spaces.

The only thing a PDF reader can see is what letters are approximately on a line. It can't see spaces as a space character, since there is no such thing in a PDF. All it has is smaller and larger gaps between letters. And thanks to kerning or justified text those aren't even consistent. So what PDF readers usually do is to guess what gaps are spaces and what gaps are not. Depending on the algorithm used the results are fairly good or horrible.

Is there any way to ensure consistent copy-and-paste behavior across all PDF viewers?

Update: I fixed this issue by adding the following line to the main TEX file in order to shorten the space between the bullet point and text:
\setitemize{leftmargin=*, labelsep=0.2cm}


Correct Domain Tooltip

Correct wrong [USERNAME] tooltip to [DOMAIN]
Signed-off-by: Claudio Mezzasalma<EMAIL_ADDRESS>

Please prefix your commit header with luci-app-ddns:
https://github.com/openwrt/luci/pull/5601/commits/d934cfc627432a71f98faebf72373b0c14dcb28c
Please have a look at the other commits

Sorry, I totally forgot. Fixed now

Uhm... How strange. I suddenly remembered that I worked on this issue a long time ago... and in fact #5057 exists that made that very change. Did I do something wrong the first time so that it has been reverted?

We did this only in the master branch.
https://github.com/openwrt/luci/commits/master/applications/luci-app-ddns/htdocs/luci-static/resources/view/ddns
The branch openwrt-21.02 does not have this commit.
https://github.com/openwrt/luci/tree/openwrt-21.02/applications/luci-app-ddns/htdocs/luci-static/resources/view/ddns
So this is ok

I have done a cherry-pick for better traceability.
https://github.com/openwrt/luci/commit/b5a40b3a981893d41cef9b91cac8429bdd31e9aa


Remove config.toml: it's user specfic and shouldn't be in git

Maybe also put it in the .gitignore and ship a config.toml.example instead?


Links missing?

Hello,
I think this section was supposed to be links?
"[uninstall the Mobility service][vmware-azure-install-mobility-service.md#uninstall-mobility-service-on-a-windows-server-computer]"
Thanks,
t

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: 5cbc6494-188a-e72d-eab0-4f29fda88190
Version Independent ID: 2d2eef66-4c19-7fe9-5bd1-9efe69597df0
Content: Migrate on-premises machines to Azure with Azure Site Recovery
Content Source: articles/site-recovery/migrate-tutorial-on-premises-azure.md
Service: site-recovery
GitHub Login: @rayne-wiselman
Microsoft Alias: raynew

@tFable Thanks for bringing this to our attention. Your feedback has been shared with the content owner for further review.

Hi, this has been fixed. Fix should show up later today. Thanks for your input!
#please-close


[ERROR]unknown jsonPayload sysmsg type: NewXmlChatRoomAccessVerifyApplication

2018-09-06: Error: unknown jsonPayload sysmsg type: NewXmlChatRoomAccessVerifyApplication
    at Object.<anonymous> (/data/projects/lightbot/node_modules/wechaty-puppet-padchat/dist/src/pure-function-helpers/room-event-join-message-parser.js:97:27)
    at Generator.next (<anonymous>)
    at fulfilled (/data/projects/lightbot/node_modules/wechaty-puppet-padchat/dist/src/pure-function-helpers/room-event-join-message-parser.js:4:58)
    at process._tickCallback (internal/process/next_tick.js:68:7)
2018-09-06: Error: unknown jsonPayload sysmsg type: NewXmlChatRoomAccessVerifyApplication
    at Object.<anonymous> (/data/projects/lightbot/node_modules/wechaty-puppet-padchat/dist/src/pure-function-helpers/room-event-join-message-parser.js:97:27)
    at Generator.next (<anonymous>)
    at fulfilled (/data/projects/lightbot/node_modules/wechaty-puppet-padchat/dist/src/pure-function-helpers/room-event-join-message-parser.js:4:58)
    at process._tickCallback (internal/process/next_tick.js:68:7)

Need More Information
Sorry, you did not follow the ISSUE Template, and there's not enough information about your problem.
In order to help us to find the problem out, you should provide all the detail information about the issue. Please make sure you are following the ISSUE Template, answer all the question, and the more important: provide the steps to reproduce the problem.
I'll close this issue. Please re-submit a new issue to provide the information that needed.


Tri 2, Week 13 Status

Each of the 3 Teams should provide me report of evidence that they have started work.

Team B (Cason P, Ethan W, Srinaga P, Avantika C, Lara B, and Veera K) has submitted the pull request for the "About Us" page


Bundle install failing in Active Record

Overview
Fresh install of rails-dev-box bundle install after vagrant ssh is failing.
Steps to Reproduce

vagrant up
vagrant ssh
cd activerecord
bundle install
See error message below

Error Message
Gem::Ext::BuildError: ERROR: Failed to build gem native extension.

current directory:
/tmp/bundler20170615-21131-1o4gil7libxml-ruby-2.9.0/gems/libxml-ruby-2.9.0/ext/libxml
/usr/bin/ruby2.4 -r ./siteconf20170615-21131-exgay4.rb extconf.rb
/usr/bin/ruby2.4: warning: shebang line ends with \r may cause a problem
checking for libxml/xmlversion.h in
/opt/include/libxml2,/opt/local/include/libxml2,/usr/local/include/libxml2,/usr/include/libxml2...
no
*** extconf.rb failed ***
Could not create Makefile due to some reason, probably lack of necessary
libraries and/or headers.  Check the mkmf.log file for more details.  You may
need configuration options.

Provided configuration options:
	--with-opt-dir
	--without-opt-dir
	--with-opt-include
	--without-opt-include=${opt-dir}/include
	--with-opt-lib
	--without-opt-lib=${opt-dir}/lib
	--with-make-prog
	--without-make-prog
	--srcdir=.
	--curdir
	--ruby=/usr/bin/$(RUBY_BASE_NAME)2.4
	--with-xml2-config
	--without-xml2-config
	--with-xml2-dir
	--without-xml2-dir
	--with-xml2-include
	--without-xml2-include=${xml2-dir}/include
	--with-xml2-lib
	--without-xml2-lib=${xml2-dir}/lib
 extconf failure: need libxml2.

    Install the library or try one of the following options to extconf.rb:

      --with-xml2-config=/path/to/xml2-config
      --with-xml2-dir=/path/to/libxml2
      --with-xml2-lib=/path/to/libxml2/lib
      --with-xml2-include=/path/to/libxml2/include

To see why this extension failed to compile, please check the mkmf.log which can be found
here:

/tmp/bundler20170615-21131-1o4gil7libxml-ruby-2.9.0/extensions/x86_64-linux/2.4.0/libxml-ruby-2.9.0/mkmf.log

extconf failed, exit code 1

Gem files will remain installed in
/tmp/bundler20170615-21131-1o4gil7libxml-ruby-2.9.0/gems/libxml-ruby-2.9.0 for
inspection.
Results logged to
/tmp/bundler20170615-21131-1o4gil7libxml-ruby-2.9.0/extensions/x86_64-linux/2.4.0/libxml-ruby-2.9.0/gem_make.out

An error occurred while installing libxml-ruby (2.9.0), and Bundler cannot
continue.
Make sure that `gem install libxml-ruby -v '2.9.0'` succeeds before bundling.

 Than ran bundle from /vagrant/rails instead of /vagrant/rails/activerecord.
This issue can be closed. ¯_(ツ)_/¯

Was just about to debug this, thanks for the heads up!


RemoteFunction not testable

Bug Report

Q
A

PHP version
7.4.29, 7.4.14, 8.1.7

SAP NW RFC SDK version
nwrfc750P_9-70002752

Extension version/git ref
2.0.0-beta2

Summary

remotefunction cannot be mocked because of the "final" flag
unable to destroy self-created RemoteFunction objects because of "Segmentation Fault" interrupted by signal 11: SIGSEGV.

Actual behavior
At the beginning: Thank you very much for the terrific work. With a real connection and a RemoteFunction object created by this connection everything works as expected.
But to simulate the behavior without sending and receiving data with a real connection, it would be nice to override or mock up the RemoteFunction.
The final class RemoteFunction cannot be emulated because of final.
I removed the line sapnwrfc_function_ce->ce_flags |= ZEND_ACC_FINAL and the following code works
$function = $this->createMock(\SAPNWRFC\RemoteFunction::class);
$function->method('invoke')->with($data)->willReturn($result);

But the test doesn't work anymore if the variables are " freed". It leads to a "segmentation fault".
Examples to reproduce:
$f = new \SAPNWRFC\RemoteFunction();
unset($f);

or in a test
$function = $this->createMock(\SAPNWRFC\RemoteFunction::class);
unset($function);

use App\SAP\Dto\StockData;
use App\SAP\Service\Client;
use Codeception\Test\Unit;
use SAPNWRFC\Connection;
use SAPNWRFC\RemoteFunction;
use Symfony\Component\Serializer\Normalizer\NormalizerInterface;

Class CientTest extends Unit
{
    public function testTransferStock (): void
    {
        $data = new stockData();
        $normalizedData = [...];
        $result = ['some' => 'result'];

        $normalizer = $this->createMock(NormalizerInterface::class);
        $connection = $this->createMock(Connection::class);
        $function = $this->createMock(RemoteFunction::class);

        $normalizer->method('normalize')->willReturn($normalizedData);
        $connection->method('getFunction')->willReturn($function);
        $function->method('invoke')->with($data)->willReturn($result);

        $classUnderTest = new Client($connection, $normalizer);

        $transferStock = $classUnderTest->transferStock($data);
        self::assertSame($result, $transferStock);
    }
}

Expected behavior
The free method for destructuring self-created RemoteFunction objects does not result in a segmentation fault.

Thanks, I'll have a look in the coming days.

This should be fixed with 7a2a033 "Allow mocking RemoteFunction"


New Detection for Multi Cloud

The detection is centered around Azure and AWS cloud environments, with a primary focus on identifying activities related to IAAS resource abuse within both AWS and Azure cloud environments.
Required items, please complete
Change(s):
New Detection
Reason for Change(s):
Multi-cloud Detection creation
Version Updated:
-yes
Testing Completed:
-yes

@petebryan , Added Cross Cloud detection, kindly review.

@petebryan, I have updated 2 more detections for AWSGurdduty to the Azure portal login, kindly review.


Update QuestNotifier.lua

fix event registration
Issue Fixed #
Proposed Changes

replace self (not existing in that context) with K (globally existing frame) for event registration
when left as self, quest notifier won't work, since self == nil

@Kkthnx-WoW

added fix for typo in quickjoin.lua

Thank you @Baine

np. will add the new enchanting pairings later too.


Add support for yeelight ceiling ambilight

Description:
Add support for yeelight 650 ambilight ( also called background light ). It will probably support also other light, that has ambilight ( new ceiling light ). It creates new light for each yeelight device, that support background light. Its based on https://github.com/home-assistant/home-assistant/pull/22345/ . I will rebase when they gets merged
Checklist:

[X] The code change is tested and works locally.
[X] Local tests pass with tox. Your PR cannot be merged unless tests pass
[X] There is no commented out code in this PR.

@syssi Initial version for ambilight support. Can you take a look if this approach is ok ? Looks like everything works. I want to test it more, and get early feedback, is that's the right way.

Two lint issues wants to be fixed:
./homeassistant/components/yeelight/light.py:85:1: D202 No blank lines allowed after function docstring
./homeassistant/components/yeelight/light.py:334:17: R1714: Consider merging these comparisons with "in" to 'bulb_type in (yeelight.BulbType.WhiteTemp, yeelight.BulbType.WhiteTempMood)' (consider-using-in)


when not logged in, display a message that's responsive to what's available on /request/new

e.g So if the article is available, ask if you want the data.
This probably just needs to happen in the headline.
Article found

The data isn't available, but you can create a request!

Data found

The article isn't available, but you can create a request!

Neither found

The research you need isn't available, but you can create a request!

This works well if you're logged in, but doesn't if you're not

Changed.


value not updated when suggestion selected

The value is not updated in the input field when you select a suggestion from the dropdown menu. I changed this in the component to call the this.props.onChange function inside the selectSuggest function as a callback after the state is updated. I'm just wondering if this should be default behavior? Here's the code I'm referring to:
  selectSuggest(suggest) {
    if (!suggest) {
      suggest = {
        label: this.state.userInput
      };
    }

    this.setState({
      isSuggestsHidden: true,
      userInput: suggest.label
    }, () => { this.props.onChange(suggest.label) });

    if (suggest.location) {
      this.setState({ignoreBlur: false});
      this.props.onSuggestSelect(suggest);
      return;
    }

    this.geocodeSuggest(suggest);

Can you provide a test case on Codepen or Jsbin or similar? When do the suggests not get selected?

when i use my laptop touchpad and the inbuilt mousekeys the click event is never fired.
i was able to replicate this quite a few times using the normal mouse too.

I cannot replicate this, sorry. Please provide a test case and reopen the issue.


How does onDotClicked work?

Rewrote the tutorial game:
https://gist.github.com/kentbrew/505daac747aa17237dfd1bd2105510d9
... tried to work with onDotClicked and hit a wall. It returns a number, which does not seem to correspond to a dot on the screen. Also, when you click the same dot twice in a row, nothing happens. Are there any examples for onDotClicked?

Heya, thank you for taking the time to open an issue!
onDotClicked should be a function which takes two numbers, which correspond to the (x, y) coordinates of the dot that's been clicked. The x and y values are between 0 and 23, and start from the top left corner. Updating your code to:
  onDotClicked = (x, y) => {
    console.log(x, y);
  },

looks like it correctly prints out the coordinate of the dot that's been clicked.

Also, when you click the same dot twice in a row, nothing happens

I think console.log is being called twice - when I tested in Chrome and Firefox both printed a number next to the log line, indicating how many times that line had been logged rather than printing it multiple times. You can see this in the gif above.

Are there any examples for onDotClicked?

At the moment, we've only got the reference documentation, but this is autogenerated from the code, and isn't particularly helpful. I'll add a comment to it saying how it should be called.

Ah, got it. You'd call getDot with those coordinates to get the color of the dot. Thanks!


Flutter doctor should tell me how to unset gradle dir

When I run flutter doctor, I get this line:
Consider removing the gradle-dir setting to use Gradle from Android Studio.

Unfortunately, it doesn't tell me how to unset the option. flutter config -h also doesn't say how to unset an option.

True, we could be more helpful here. flutter config --gradle-dir= should do the trick.

You might want to wait until #8298 is fixed, though.

@jakobr-google are we ready to fix this?

Yep. Should be enough to add some text to either the flutter doctor output, or flutter config -h.


Non Screenplay terminal should show step status (passed tick / failed cross) in the terminal

Hi @jan-molak ,
I am not getting real time status of the executed tests in the console. I am using serenity with the non-screenplay pattern.
My config file says:
 serenity: {
    runner: 'cucumber',
    requirementsDirectory: `${process.cwd()}/test/e2e/features`,
    crew: [
      ArtifactArchiver.storingArtifactsAt('./target/site/serenity'),
      new SerenityBDDReporter(),
      ConsoleReporter.withDefaultColourSupport(),
      Photographer.whoWill(TakePhotosOfInteractions)
    ]
  }

The real-time console output I am getting:

You can clearly see that although all steps are getting passed, I am not able to see the tick marks before the steps. It is blocking me to visualize the real-time status.

Hey @abhinaba1080, so that's an interesting one.
If you look at how the @serenity-js/console-reporter works, you'll see that it prints the name of the task when the task starts, and the name of the interaction together with its status (a tick, a cross, etc.) when the interaction finishes.
Since Cucumber steps emit the same events as tasks (see #362), their names are printed before the task is executed. At that point, we don't know their status, so there's no status to print.
The reasoning behind this design is that the status of a task depends on the status of the interactions that constitute it. While we could print the status of both tasks and interactions, it would add a lot of noise to the output in screenplay-style scenarios.
If there's a huge interest in this feature, we could implement it as a config option.

Closing in favour of #1654, where we can discuss improvements to ConsoleReporter


Issue: Cannot copy default decks anymore

Something has changed and its no longer possible to click to copy the default Tarot or Standard deck.

Good catch! Should be fixed in version 0.3.24. Unfortunately for now it's only possible to copy-paste default cards if you set "Card click behavior" option to "Image" (but it'll create "Pasted image" files). I still haven't fully figured out how to best handle this feature, but I'm working on it

Love your work mate :)


Go to react-redux@8 or declare it as a production dependency?

This is a little complex to explain, so please bear with me. In our application, we use react-redux@8, and we are currently trying to also use @elyra/canvas, which has a peer dependency on react-redux@7:
https://github.com/elyra-ai/canvas/blob/7bf40a9850221f41e404d11a419041781b9b0503/canvas_modules/common-canvas/package.json#L111
In practice, I don't believe these clashing versions of react-redux should cause an issue, because the main CommonCanvas component is encapsulated with its own <Provider>:
https://github.com/elyra-ai/canvas/blob/7bf40a9850221f41e404d11a419041781b9b0503/canvas_modules/common-canvas/src/common-canvas/common-canvas.jsx#L77-L79
However, this version conflict is still causing the following error for us at npm install time:
$ npm install
npm ERR! code ERESOLVE
npm ERR! ERESOLVE unable to resolve dependency tree
npm ERR!
npm ERR! While resolving<EMAIL_ADDRESS>npm ERR! Found<EMAIL_ADDRESS>npm ERR! node_modules/react-redux
npm ERR!   dev<EMAIL_ADDRESS>from the root project
npm ERR!
npm ERR! Could not resolve dependency:
npm ERR! peer<EMAIL_ADDRESS>from<EMAIL_ADDRESS>npm ERR! node_modules/@elyra/canvas
npm ERR!   dev<EMAIL_ADDRESS>from the root project
npm ERR!
npm ERR! Fix the upstream dependency conflict, or retry
npm ERR! this command with --force or --legacy-peer-deps
npm ERR! to accept an incorrect (and potentially broken) dependency resolution.

I believe that this issue could be resolved in one of two ways.

One would be to declare react-redux as a production dependency of @elyra/canvas. It is actually used directly in production so I think this would be logical.
Another would be for @elyra/canvas to upgrade from react-redux@7 to react-redux@8. This upgrade was relatively painless for us when we did it, so I hope you would find the same. For your reference, here are the release notes for<EMAIL_ADDRESS>
In fact, ideally I think you should do both of these things.

OK so cards on the table, the above is a simplification of the real issue. We do not use @elyra/canvas directly, we use a package @icp4i/cp4i-operand-create which uses @icp4i/cp4i-canvas which uses @elyra/canvas. At npm install time, due to the conflict described above, npm does not install @elyra/canvas at the top level in ./node_modules/@elyra/canvas. Instead it installs both @elyra/canvas and its dependency react-redux@7 at a lower level, in ./node_modules/@icp4i/cp4i-canvas/node_modules/@elyra/canvas and ./node_modules/@icp4i/cp4i-canvas/node_modules/react-redux respectively.
Then, during the CSS phase of our webpack build, we attempt to pull in @icp4i/cp4i-canvas's SCSS, which includes the line:
@import '@elyra/canvas/src/index';

This import is trying to find @elyra/canvas at the top level, in ./node_modules/@elyra/canvas, but it fails, which then fails our build:
Error: Can't find stylesheet to import.
  ╷
1 │ @import '@elyra/canvas/src/index';
  │         ^^^^^^^^^^^^^^^^^^^^^^^^^
  ╵
<EMAIL_ADDRESS>1:9                   @use
<EMAIL_ADDRESS>1:1  root stylesheet

If @elyra/canvas were installed at the top level, this issue would go away.
Thank you!

This has been moved to internal tracking Ref: 3703

@tomlyn Can you provide a link to the place where you're now tracking this issue? I'd like to be able to subscribe to updates so that I can know when it gets fixed.

@s100 I sent you a DM

@s100 This will be in the next release of Elyra Canvas v13

@tomlyn When will that be?

@s100 We will release next 13.x alpha version on Monday 4/8.

Thank you. Looking forward to it. An alpha release is fine but I'm really looking for a production-ready version.

@s100 This issue is fixed in version 13.0.0-alpha.1 .

@nmgokhale That's an alpha release. Is it ready for production use?

<EMAIL_ADDRESS>has now been released but the latest tag still points at<EMAIL_ADDRESS>so it's still unclear whether V13 is ready for production use...

Hi s100 - Thanks for mentioning this. We are aware of this problem and have an issue to address it.  Be assured v13.0.0 is ready for production use.


Can't extract skse_loader into Stock Game

Okay so i know i already post for a problem today but here again i don't know how to solve this even after searching for it. During the installation and the extraction of the mods, it keeps failling with the extraction of skse_loader because it don't have the permission to do it.
I'm connected as administrator and i even run wabbajack as administrator. If i move the file with another name into the folder there's no problem (i just can't name it skse_loader) but because the installation failed, Mod Organizer is not ready to launch.
Is there a way to grant permission to extract skse_loader?
I join the log this time t
Wabbajack.current.log
oo

This is an antivirus interference. Please disable it to proceed with install.
See below common information about the case:
Antivirus Interference
If you are experiencing problems running Wabbajack such as the exe being deleted or no progress during the installation phase, try adding the folder where your Wabbajack executable is and the folder where you wish to install the modlist to exclusions in your antivirus of choice.
Find out how to via: https://lmgtfy.app/?q=how+to+add+exclusions+to+antivirus
Note: If you're using Webroot or any other free 3rd party antivirus it's likely that adding the folders to exclusions will not be enough and you'll need to disable or uninstall your 3rd party AV as they can incorrectly mark usvfs_proxy_x86.exe, among other files, as a virus, a file needed for Mod Organizer 2 to work. We recommend doing so anyway in case it's a free one, as Windows Defender is likely much better at stopping threats than that is (according to data from av-test.org https://www.av-test.org/en/antivirus/home-windows/).
If you still think a file is a virus, you can upload it to virustotal and get it scanned by multiple antiviruses. Here is a scan of the file mentioned above, of which 2 antiviruses report a false positive:
https://www.virustotal.com/gui/file/356c029b7bf0bed41460ceacf2c756560101b9b0977c349925d81d76392dd0c4/detection

Thanks again for your help! Everything works now, i even managed to launch the game ^^
I can't wait to test the modpack!

👍🏽
It is modlist, though. Also you may join dedicated Serenity discord - Emporio's Emporium - for discussion and stuff.


bugfix:OSIS-28 query users filter params

Hello saitharunthodupunuru,
My role is to assist you with the merge of this
pull request. Please type @bert-e help to get information
on this process, or consult the user documentation.
Status report is not available.

Incorrect fix version
The Fix Version/s in issue OSIS-28 contains:

None

Considering where you are trying to merge, I ignored possible hotfix versions and I expected to find:

1.0.0

Please check the Fix Version/s of OSIS-28, or the target
branch of this pull request.

/approve

Incorrect fix version
The Fix Version/s in issue OSIS-28 contains:

None

Considering where you are trying to merge, I ignored possible hotfix versions and I expected to find:

1.0.0

Please check the Fix Version/s of OSIS-28, or the target
branch of this pull request.
The following options are set: approve

/reset

Reset complete
I have successfully deleted this pull request's integration branches.
The following options are set: approve

In the queue
The changeset has received all authorizations and has been added to the
relevant queue(s). The queue(s) will be merged in the target development
branch(es) as soon as builds have passed.
The changeset will be merged in:

:heavy_check_mark: development/1.0

There is no action required on your side. You will be notified here once
the changeset has been merged. In the unlikely event that the changeset
fails permanently on the queue, a member of the admin team will
contact you to help resolve the matter.
IMPORTANT
Please do not attempt to modify this pull request.

Any commit you add on the source branch will trigger a new cycle after the
current queue is merged.
Any commit you add on one of the integration branches will be lost.

If you need this pull request to be removed from the queue, please contact a
member of the admin team now.
The following options are set: approve

I have successfully merged the changeset of this pull request
into targetted development branches:

:heavy_check_mark: development/1.0

Please check the status of the associated issue OSIS-28.
Goodbye saitharunthodupunuru.


BlazorGraphApi - added stars

This update adds stars to the  BlazorGraphApi entry.
I now have 11 stars :)

12 stars! :)


MIVisionX - cmake cleanup

@kiritigowda this PR cleans up the CMake files for MIVisionX and fixes an issue for ROCm 5.3+ where MIVisionX couldn't find the HIP package.

@kiritigowda This PR fixes the problem that I have been seeing.

@paveltc -- waiting for CI results to merge -- http://math-ci.rocm.amd.com/blue/organizations/jenkins/compute-rocm-dkms-no-npi-hipclang%2FShort-GPUOpen%2FMIVisionX/detail/PR-943/1/pipeline


Can not verify domain ownership.

After upload 'did-configuration.json' file in /.well-known/ path, Entra Verified ID can not verify domain ownership.

Document Details
⚠ Do not edit this section. It is required for learn.microsoft.com ➟ GitHub issue linking.

ID: f84b158e-eebc-7d96-0992-8864316ddf5c
Version Independent ID: 8909ba9b-bf26-fb5c-29f0-76244984e75a
Content: Tutorial - Configure your tenant for Microsoft Entra Verified ID - Microsoft Entra
Content Source: articles/active-directory/verifiable-credentials/verifiable-credentials-configure-tenant.md
Service: decentralized-identity
Sub-service: verifiable-credentials
GitHub Login: @barclayn
Microsoft Alias: barclayn

@xctpro thank you for reaching out. I can't repro this. Please contact support if the issue persists. #please-close


Update composer.json

Big change here is that we won't blindly accept any Laravel version, it's explicitly 5.1 and 5.2

Done.


Changing Message Description and Button Text

I would like to change the message text and button texts for the Update prompt. It seems that this used to be possible in the past (#5), but those properties are no longer exposed. Is there a new way using the current version of Harpy to change these strings, or is forking the repo the only way to do this. I only need it for English.
Thanks!

@ahanmal-carecloud It's no longer possible, as I wanted to streamline the messages, which allowed for easy localization.
In your position, I'd fork the repo and make the changes.


show correct month on event page calendar

javascript dates start counting month numbers at zero. python at 1. It took me 2 hours to track down this 2-character fix :(

epic


Typo

Typo: Evalutates

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: 994c402a-0c34-a387-5592-2416ccf31fa9
Version Independent ID: b79fa1da-b462-eb14-d2c2-00d987bd0ecf
Content: IFeatureFilter Interface (Microsoft.FeatureManagement) - Azure for .NET Developers
Content Source: xml/Microsoft.FeatureManagement/IFeatureFilter.xml
Service: azure-app-configuration
GitHub Login: @CamSoper
Microsoft Alias: casoper

Hi,
I looked into it. The component lives in the FeatureManagement-Dotnet repo.
Looks like the typo was fixed on October 1st, but the latest release is from September 16.
So, the change is not yet available.


Support c++11

The jgemu bsnes fork is now complaint with c++11 where it is likely going to stay. It would also be nice if byuuML would also build as c++11 now.
See: https://gitlab.com/jgemu/bsnes/-/commit/3d01169b0061562e315b75bf77ee96574ff41b4a

Are these really the only changes needed to make it work in C++11? Neat.
Thanks!

bsnes required a bit more effort than byuuML to bring it down from c++14 to c++11, but it nothing like removing nall or even dropping c++17.
For reference:
https://gitlab.com/jgemu/bsnes/-/commit/8fb2d1ae295d8ad34613fc0ce06992ef3497aeff
https://gitlab.com/jgemu/bsnes/-/commit/fa941799b09a8e5ee5d6ec5a4efdd4e5002468b2
https://gitlab.com/jgemu/bsnes/-/commit/1f5d2dbd99907a245f2d7f0b0b7e31d1865a6423
https://gitlab.com/jgemu/bsnes/-/commit/7e027f305a7ee3a5ac31b589fdcd56bab7b9c3ec

I expected as much. It was more that I remembered using more C++14 features while writing byuuML.


Feature dirichlet pr

Hi Dr. Najian,
Here are some issues we have discussed.

@kianoosh1989 Thank you for the feature! This branch has several conflicts with the main branch due to some fundamental changes that are missing in this PR. Could you please merge the main branch into this one and make any necessary adjustments? Once that’s done, we can proceed with the review and merge.

Dear Dr. Najian,
I updated this PR regarding your last comments. In addition, stress function was added.
Best


Flocker does not support mem_limit fig directive

Flocker does not support the Fig mem_limit directive, a single value acting on a container in the same manner as its docker run counterpart.
http://www.fig.sh/yml.html

We are moving our development planning to JIRA. This issue is now being tracked at https://clusterhq.atlassian.net/browse/FLOC-762. You are welcome to file additional issues in GitHub if that's easier for you.


Issue after node update 20

Error: System.ArgumentOutOfRangeException: Specified argument was out of the range of valid values. (Parameter ''using: node20' is not supported, use 'docker', 'node12' or 'node16' instead.')
at GitHub.Runner.Worker.ActionManifestManager.ConvertRuns(IExecutionContext executionContext, TemplateContext templateContext, TemplateToken inputsToken, String fileRelativePath, MappingToken outputs)
at GitHub.Runner.Worker.ActionManifestManager.Load(IExecutionContext executionContext, String manifestFile)

Can you post your workflow definition? I am especially interested in runs-on, does it support node 20?
In mean time can you pin the action to the<EMAIL_ADDRESS>not the latest?

Since this is breaking runtime change, I've released node 20 upgrade under the 2.0.0 version, the 1.x.x will remain pinned to the node 16. Just re-running your action should fix the problem (no need to pin to the older version as suggested in previous post). I'll close the ticket now, but feel free to let us know if you still have problems.


Revert "Verify systemd_unit file during create"

Reverts chef/chef#5210 since this appears to be a breaking change. See https://github.com/chef/chef/issues/5324

well, my reflexive :-1: on reflexively reverting any change.
do we understand why this broke and if we can ship a simple fix that maintains the feature?

do we understand why this broke and if we can ship a simple fix that maintains the feature?

The core issue seems to be that systemd-analyze verify requires the extension to indicate the unit type:
[root@localhost ~]# systemd-analyze verify ./foobar
Failed to load ./foobar: Invalid argument
[root@localhost ~]# systemd-analyze verify ./foobar.service
[root@localhost ~]# echo $?
0

One possible solution would be to create a copy of the tempfile with the right extension, verify that, and then clean it up.

Currently some discussion about a new feature that allows you to specify the extension of the tempfile that we create.

Thanks. I'm closing.

reopenning since the temp file fix, though still of value, has not completely fixed this issue.

👍

👍

verified this is working again in 12.14.90, thanks @mwrock!


rounding issue

感谢作者的工作，请问代码中rounding部分的操作的解释在论文中有体现吗，怎么理解这个rounding操作后的结果就是word的词序号？

Rounding operation maps the word embedding vectors back to discrete tokens. and we still map these tokens into vectors as the input of next-step generation. This operation makes sure that at each step, we use the vectors standing for word tokens instead of vectors without corresponding tokens. We discuss this in Eq.7-Eq.8.

Rounding operation maps the word embedding vectors back to discrete tokens. and we still map these tokens into vectors as the input of next-step generation. This operation makes sure that at each step, we use the vectors standing for word tokens instead of vectors without corresponding tokens. We discuss this in Eq.7-Eq.8.


Separate functionality of SVA into two modules

We should separate the PCA calculation / mean calculation etc and shape learning/classification into two different modules / CLIs
@vicory @juanprietob Mateo Lopez

PCA calculation is already a sperate module. The mean is calculated when the population exploration is done.  The learning/classification is not included in the CLI yet.
Could you give us more details on what the feature you want?

@juanprietob Maybe in our next meeting today, you can do another live demo. I downloaded the extension and was not able to figure out the structure. I will try again this morning so we can re convey later.

Clarified. Closing.


Program is reopening when it closed

Describe the bug
Program is reopening when I close it. (for once)
To Reproduce

Run program
Close
See bug

Expected behavior
Program should not reopen.
Desktop (please complete the following information):

OS: Windows
Version: 10

Additional context
Problem is here:
(main.py/line 455-456)
    # Running mainloop
    ROOT.mainloop()

and
(main.py/line 641-642)
# Running mainloop
ROOT.mainloop()

Program is twice launching by ROOT.mainloop()s

I have fixed this issue.

I really appreciate you for pointing this out. Thanks a lot.


Support SSO (Oauth2, OIDC, or SAML)

Allow user provisioning / access to be controlled via a separate IdP.

Any updates on this? It's the last thing I need backed I can pitch using it for Christmas with my family
I was looking around and came across this: https://flask-oidc.readthedocs.io/en/latest/#
I don't have a ton of experience with python this complex overall but I thought I might as well throw some time at it when I can and see if anything works out

Hello, I've worked on something using Keycloak and OpenID Connect. I've managed to get the information of the user who logs in. I find this very complicated, I know it's authentication so it cannot be simple. It may take a bit of time since I've never worked in that.
I have some questions;
What app are you using for the auth?
What will you use to check the user? Username or email?
Thanks for being still interested.

I've been using Authelia up until now, but I've been trying out Authentik as well. Both support OIDC so they should be compatible regardless
For the identifier, I'm thinking a username might be better? Both Authelia and Authentik require a username, but the email field is optional. Although for Authelia, I always pass 'openid' 'profile' 'email'.
There's a lot of information in Authelia's docs about OIDC scopes, which may help. That link is for OIDC 1, and there's an OAuth 2.0 section as well. And this is the OAuth 2.0 section for Authentik

Here is my new docker-compose.yml:
services:
##################################################################
##################################################################
##################################################################
  giftmanager:
    container_name: giftmanager
    image: icbest/devgift:latest
    volumes:
      - ${dcs}/giftmanager/app/data:/app/data
      - ${dcs}/giftmanager/backup:/backup
    hostname: giftmanager
    ports:
      - 4022:5000
    restart: unless-stopped

And my .env:
dcs=/srv/dev-disk-by-uuid-ca6f47d3-cd3d-4acc-bb5c-53a1d0eb27e5/DockerContainerStorage

KEYCLOAK_CLIENT_ID="[redacted]"
KEYCLOAK_CLIENT_SECRET="[redacted]"
KEYCLOAK_SERVER_METADATA_URL="https://authentik.[redacted].me/application/o/giftmanager/.well-known/openid-configuration"
KEYCLOAK_LOGOUT_URL="https://authentik.[redacted].me/application/o/giftmanager/end-session/"
PRIMARY_OIDC_FIELD=email
SECONDARY_OIDC_FIELD=preferred_username
PRIMARY_DB_FIELD=email
SECONDARY_DB_FIELD=username

When I try to load https://gift.[redacted].me/login_oidc, this is the error I get:
Traceback (most recent call last):
  File "/usr/local/lib/python3.11/site-packages/flask/app.py", line 1498, in __call__
    return self.wsgi_app(environ, start_response)
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/usr/local/lib/python3.11/site-packages/flask/app.py", line 1476, in wsgi_app
    response = self.handle_exception(e)
               ^^^^^^^^^^^^^^^^^^^^^^^^
  File "/usr/local/lib/python3.11/site-packages/flask/app.py", line 1473, in wsgi_app
    response = self.full_dispatch_request()
               ^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/usr/local/lib/python3.11/site-packages/flask/app.py", line 882, in full_dispatch_request
    rv = self.handle_user_exception(e)
         ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/usr/local/lib/python3.11/site-packages/flask/app.py", line 880, in full_dispatch_request
    rv = self.dispatch_request()
         ^^^^^^^^^^^^^^^^^^^^^^^
  File "/usr/local/lib/python3.11/site-packages/flask/app.py", line 865, in dispatch_request
    return self.ensure_sync(self.view_functions[rule.endpoint])(**view_args)  # type: ignore[no-any-return]
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/app/app.py", line 156, in login_oidc
    return oauth.keycloak.authorize_redirect(redirect_uri, nonce=nonce, state=state)
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/usr/local/lib/python3.11/site-packages/authlib/integrations/flask_client/apps.py", line 43, in authorize_redirect
    rv = self.create_authorization_url(redirect_uri, **kwargs)
         ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/usr/local/lib/python3.11/site-packages/authlib/integrations/base_client/sync_app.py", line 318, in create_authorization_url
    raise RuntimeError('Missing "authorize_url" value')
RuntimeError: Missing "authorize_url" value

Opening a shell within the container and running pip instrall authlib comes back with the following:
Requirement already satisfied: authlib in /usr/local/lib/python3.11/site-packages (1.3.2)
Requirement already satisfied: cryptography in /usr/local/lib/python3.11/site-packages (from authlib) (43.0.3)
Requirement already satisfied: cffi>=1.12 in /usr/local/lib/python3.11/site-packages (from cryptography->authlib) (1.17.1)
Requirement already satisfied: pycparser in /usr/local/lib/python3.11/site-packages (from cffi>=1.12->cryptography->authlib) (2.22)
WARNING: Running pip as the 'root' user can result in broken permissions and conflicting behaviour with the system package manager. It is recommended to use a virtual environment instead: https://pip.pypa.io/warnings/venv

[notice] A new release of pip is available: 24.0 -> 24.3.1
[notice] To update, run: pip install --upgrade pip

Here's a picture of the configuration panel within Authentik, if it helps:

Hello, just installed Authentik and tested myself. Everything worked correctly. Could you just make sure your user have the right logins info (emails are similar or usernames) and you also may restart the container after updating the .env in /setup.
If still dont work, you will need to give me more infos because i dont see the problems right now.
Thanks

Unfortunately I've still not been able to get it working. So far I've tried:

Deleting and recreating the container
Deleting the container, deleting the folder its data was in, then recreating the container, then going to /login_oidc
Again with a fresh container and folder, made a new account with the email I use for Authentik, then went to /login_oidc
Changed https://authentik.[redacted].me/application/o/giftmanager/.well-known/openid-configuration to https://authentik.[redacted].me/application/o/giftmanager/

Everything still results in the same error. Can you provide the environment variables you're using for your Authentik setup?

OK, here is the detailed steps I've done:

Installed Authentik on a new Linode vps using this guide

Setup the admin account

Created a new provider called it giftmanager using OAuth2/OpenID Provider and keeping all default parameters.

Ran the container of giftmanager on a machine with public exposed ip using the command:  docker run -p 5055:5000 -d -v /root/data:/app/data icbest/devgift

Created an account with this email<EMAIL_ADDRESS>and this username (icbest). Deleted default profiles.

Restarted the gift app

Created an account, with the same infos, on authentik. Username = icbest   Email =<EMAIL_ADDRESS>

Added a new app in authentik

Gone to /setup in the giftmanager and added the values to .env file here they are: KEYCLOAK_CLIENT_ID='bcG3BBvGI2upvILrdK5hnvUMCIUmx2Y04cMTck7M'
KEYCLOAK_CLIENT_SECRET='QrFVKTrYFf4PHPbfyKeUZVsbSiZWxaDCFw1TGmNI8iMLfEuK5mgiLgpBevvz6OnHa58Fc3aim3c5INMPT0NjDwxzz82mV6bfKpEb3Fteh4EFqRzKNGqf8AK0P9nhcVg6'
KEYCLOAK_SERVER_METADATA_URL='http://ip:9000/application/o/giftmanager/.well-known/openid-configuration'
KEYCLOAK_LOGOUT_URL='http://ip:9000/application/o/giftmanager/'
PRIMARY_OIDC_FIELD='email'
SECONDARY_OIDC_FIELD='preferred_username'
PRIMARY_DB_FIELD='email'
SECONDARY_DB_FIELD='username'

I restarted the gift container after updating the values

Logout of Authentik and gift app

Go to /login_oidc

logged on Authentik using Icbest profile

Returned to /login_oidc get this window and click continue:

Then successfully logged into the gift app.

The Linode is deleted, don't worry for personal infos.
All worked while doing these steps in order. I even tested with a different username but right email, and it still worked.
Hope it works.

Oh... I didn't go into that /setup page at all. Would it make more sense to make these environment variables that can be set within the docker.compose.yml?
Anyway, once I set everything within that page, it srtated to work, sort of. I set the redirect URL: to https://gift.[redacted].me/auth at first, but Authentik gave me an error about a mismatch. I set the redirect to * for now and it works great otherwise

Also it would be really helpful to (maybe as an option) automatically create new GiftManager accounts if a user successfully authenticates with OIDC but does not have a user registered with GiftManager yet

I'm almost done writing everything for release. I just need to know how you want to set the environment variables in the Docker Compose. I'm not sure what the best way to do this is; any recommendations or tips are welcome.
 Sadly I didn't successfully implemented a way to put the values in the docker-compose. You will have to either modify the .env via docker exec or in /setup . I will write the documentation related to OIDC soon. Thanks for the help.

Thank you!

Tried configuring Google as my OIDC provider, but am getting this error when attempting to load https://gift.[redacted].net/login_oidc
`<IP_ADDRESS> - - [22/Nov/2024 01:57:59] "GET /login_oidc HTTP/1.1" 500 -
Traceback (most recent call last):
File "/usr/local/lib/python3.11/site-packages/flask/app.py", line 1536, in call
return self.wsgi_app(environ, start_response)

       ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^

File "/usr/local/lib/python3.11/site-packages/flask/app.py", line 1514, in wsgi_app
response = self.handle_exception(e)

           ^^^^^^^^^^^^^^^^^^^^^^^^

File "/usr/local/lib/python3.11/site-packages/flask/app.py", line 1511, in wsgi_app
response = self.full_dispatch_request()

           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^

File "/usr/local/lib/python3.11/site-packages/flask/app.py", line 919, in full_dispatch_request
rv = self.handle_user_exception(e)

     ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^

File "/usr/local/lib/python3.11/site-packages/flask/app.py", line 917, in full_dispatch_request
rv = self.dispatch_request()

     ^^^^^^^^^^^^^^^^^^^^^^^

File "/usr/local/lib/python3.11/site-packages/flask/app.py", line 902, in dispatch_request
return self.ensure_sync(self.view_functions[rule.endpoint])(**view_args)  # type: ignore[no-any-return]

       ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^

File "/app/app.py", line 152, in login_oidc
nonce = secrets.token_urlsafe(16)

        ^^^^^^^^^^^^^^^^^^^^^^^^^

NameError: name 'secrets' is not defined`
Here is my .env file:
FEED_SEND='' MAILJET_API_KEY='' MAILJET_API_SECRET='' SECRET_KEY='changethis' SYSTEM_EMAIL='' DELETE_DAYS='30' OIDC_CLIENT_ID='[redacted]' OIDC_CLIENT_SECRET='[redacted]' OIDC_SERVER_METADATA_URL='https://accounts.google.com/.well-known/openid-configuration' OIDC_LOGOUT_URL='https://accounts.google.com/Logout' PRIMARY_OIDC_FIELD='email' SECONDARY_OIDC_FIELD='preferred_username' PRIMARY_DB_FIELD='email' SECONDARY_DB_FIELD='username' ENABLE_AUTO_REGISTRATION='true'

Yeah, good. Now corrected; see v.1.0.2
Docker latest and 1.0.2 now available.
Thanks

It seem that there is a small bug with the auto registration feature. I'm on it. I still don't know the cause.

The issue is now fixed.
It was only the docker json url that wasn't referenced correctly in the setup_profile route.
with open(app.config['USERS_FILE'], "w") as file:             json.dump(users, file, indent=4)
See v1.0.3
Image upload to docker hub.


Version comparison does not work for 4 digit versions

My packages.config file:
<?xml version="1.0" encoding="utf-8"?>
<packages>
  <package id="FluentBootstrap" version="<IP_ADDRESS>" targetFramework="net45" />
</packages>

FluentBootstrap has a vulnerability against version <IP_ADDRESS>.
DevAudit output:
> devaudit.exe nuget -f Examples\packages.config.example
Scanning NuGet packages...
Found 1 distinct packages.
Searching OSS Index for 1 NuGet packages...
Found 1 artifacts, 1 with an OSS Index project id.
Searching OSS Index for vulnerabilities for 1 projects...

Audit Results
=============
[1/1] FluentBootstrap (<IP_ADDRESS>) 1 known vulnerabilities, 0 affecting installed version. [FluentBootstrap <IP_ADDRESS>]

This is an issue with the version matching: the spec says that the 4th part of a semantic version is a "pre-release" version which is handled a bit differently than the regular version numbers but .NET uses 4 part versions that don't follow semver rules and Nuget still allows these older version schemes. I'll simply correct the code in NuGetPackageSource to properly handle these versions as described here:
https://github.com/NuGet/NuGet.Client/blob/d316bf56e4f55e94d166b31a0189464015b8a844/src/NuGet.Core/NuGet.Versioning/NuGetVersion.cs

This is fixed in 7685a02 using version 0.1.15 of Versatile.
c:\ASP.NET\DevAudit>devaudit nuget -f Examples\packages.config.example.3
Scanning NuGet packages...
Found 1 distinct packages.
Searching OSS Index for 1 NuGet packages...
Found 1 artifacts, 1 with an OSS Index project id.
Searching OSS Index for vulnerabilities for 1 projects...

Audit Results
=============
[1/1] FluentBootstrap (<IP_ADDRESS>) [VULNERABLE]
1 known vulnerabilities, 1 affecting installed version. [FluentBootstrap <IP_ADDRESS>]
 Possible Cross Site Scripting (XSS)
Strings which are passed to this library via TagExtensions.AddContent and ContentExtensions.Content are not encoded, opening the possibility for an unexpected XSS vulnerability.
Affected versions: <<IP_ADDRESS>

Works on the newest version.


"Touch to search" pane is partly visible when bottom toolbar is up.

"Touch to search" pane is partly visible and accessible when bottom toolbar is visible too.

Probable duplicate of #224

Merged with #224.


Swoole support

Describe the solution you'd like
I see you use loop from react, you could possibly provide client to use one with swoole.
Describe alternatives you've considered
A clear and concise description of any alternative solutions or features you've considered.
Additional context
More about swoole - https://github.com/swoole/swoole-src

Thank you for filing this issue!
Unfortunately it's not as easy as just simply replacing the client. Under the hood more must be done. The HTTP and websocket part need work too. The client just takes the client and distributes it. The only thing the client really uses from the event loop is providing auto destroying timers as soon as the client gets destroyed.
While I do think Swoole is a great native alternative to ReactPHP, I think it's out of scope of this project to provide additional alternative implementations. ReactPHP really just works out of the box while Swoole requires compiling the extension (on Windows not even easy and Cygwin isn't that great either).
But if you really require or want to use Swoole, you can extend the parts yourself and replace the ReactPHP components with Swoole equivalents. The current client already allows to explicitely pass a WSManager and APIManager into the client (they just need to extend the Yasmin classes, to ensure the  methods exist, but aside from that you can replace all the existing methods - if required).
https://github.com/CharlotteDunois/Yasmin/blob/91ab40d63c823cf5a0d38b8c2175bfb2fbec127a/src/Client.php#L252-L261
https://github.com/CharlotteDunois/Yasmin/blob/91ab40d63c823cf5a0d38b8c2175bfb2fbec127a/src/Client.php#L266-L275
For HTTP requests clue/buzz-react is used (only master branch and future v0.6 - current and previous relases use guzzle), but if you don't use that client yourself, you can simply create a class in the same namespace (so the original package doesn't get autoloaded) that uses Swoole, so in the end even though it doesn't explicitely use Swoole, it uses your class which delegates it away to Swoole.
See clue/buzz-reactphp and
https://github.com/CharlotteDunois/Yasmin/blob/91ab40d63c823cf5a0d38b8c2175bfb2fbec127a/src/Utils/URLHelpers.php#L58
If you ever get to a working project and upload it to GitHub, I'd be happy to link to it in the readme!


[BUG]: Intersex flag is not being displayed correctly

Was ist passiert / What happened?
The intersex flag introduced in #208 is only displayed correctly for a split second, after that it turns solid yellow when the logo is small, and yellow with random white circles when big (login page)
Wie kann man den Fehler reproduzieren? / How to reproduce?

Set setting 'Pride-Logo' to 'Intersex'
Wait until the page is fully loaded

Erwartetes Verhalten / Expected behavior
Correctly displaying the Intersex SVG as the UZL-Logo mask
System

OS: Windows 10
Browser: Opera GX, LVL5 (core: 109.0.5097.108)
Better-Moodle-Version: 1.37.0

Sonstiges / Additional
SVG is being overwritten by incorrect data:

The resulting logo (login):

On homepage:

A possible fix would be to use a radial-gradient instead of an SVG. Dark reader should not make problems, as no white is used in the flag.
background-image: radial-gradient(circle,
       rgb(229, 195, 0) 0%,
       rgb(229, 195, 0) 15%,
       rgb(92 14 106) 15%,
       rgb(92 14 106) 25%,
       rgb(229, 195, 0) 25%,
       rgb(229, 195, 0) 100%);

Initial testing was successful, but I don't know what causes the issue in the first place. All previous testing was fine.

This seems to be a chromium-only bug in combination with DarkReader.
We may be able to avoid this by adding an !important to the svg rules.

Unless I'm putting it in the wrong place, !important doesn't seem to work, it still gets overwritten:

Further testing shows:

The issue isn't occurring using light mode, hinting more towards DarkReader as the cause of the issue
The issue is occurring on Safari in dark mode


How about adding JBR as known java toolchain vendor?

Currently JvmVendorSpec does not have entry for Jetbrain runtime, so java toolchain configuration can't choose it as an option. I think it would be good to add it.
Expected Behavior

I expected Gradle to know JBR's existence and recognize it as one of vendors.
Current Behavior
It's not aware of JBR and can't use it as an option.

Context

It doesn't affect me much since I just set project's sdk as JBR from Intellij Idea so it is used as default for gradle.
The strength of JBR is that it works quite well with code hot-swapping, it even has DCEVM built-in. that is why I feel like there would be significant number of people using it with gradle.

You should be able to request the JetBrains Runtime like this: JvmVendorSpec.matching("JetBrains").
Yes, a named entry should be coming at some point.

Fixed in Gradle 8.4 by

https://github.com/gradle/gradle/pull/26014


Not able to call web services

When i tried to debug i am getting generic error on the below X-Code

Foundation.pdb not loaded and objcrt.pdb not loaded.

NSURLConnection * connection = [[NSURLConnection alloc] initWithRequest:request delegate:self          startImmediately:NO];
[connection scheduleInRunLoop:[NSRunLoop mainRunLoop] forMode:NSDefaultRunLoopMode];
[connection start];

Hi All,
Is there any solution or workaround for the above mention issue.
Any suggestions will be helpful.

Hi there,
We're having trouble reproducing your issue. Could you send a representative sample that we can look at?

Is there a chance that you're attempting to access an https resource?

Thanks!

Yes we are trying to access https resource.

Thanks DHowett-MSFT. We will our sample code with you.

Apologies for late response.
Please find the attached code where we aere facing the issues.
VmImporter.zip
These error we are getting during runtime Generic Error  followed by Objcrt not loaded, UiKIt is not loaded.

now available.


Add RobinD42 to maintainer list

Checklist

[x] Used a personal fork of the feedstock to propose changes

This PR is only for adding RobinD42. Another with recipe changes will be forthcoming.

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


Sentry fall if there is a frozenset as key of dict in context

Sentry fall if there is a frozenset as key of dict in context
I found this  when near place of send error was dict with frozenset as key
proof 1
from  raven.utils.json import *
value = {frozenset([1,2,3]):4}
json.dumps(value, cls=BetterJSONEncoder)
Traceback (most recent call last):
  File "<input>", line 1, in <module>
  File "/usr/lib/python2.7/json/__init__.py", line 251, in dumps
    sort_keys=sort_keys, **kw).encode(obj)
  File "/usr/lib/python2.7/json/encoder.py", line 207, in encode
    chunks = self.iterencode(o, _one_shot=True)
  File "/usr/lib/python2.7/json/encoder.py", line 270, in iterencode
    return _iterencode(o, 0)
TypeError: keys must be a string

user case
/pycharm-2016.3.2/helpers/pydev/pydevconsole.py 39277 38925
PyDev console: starting.
import sys; print('Python %s on %s' % (sys.version, sys.platform))
import django; print('Django %s' % django.get_version())
sys.path.extend(['/home/uc/Projects/c2public', '/home/uc/Downloads/pycharm-2016.3.2/helpers/pycharm', '/home/uc/Downloads/pycharm-2016.3.2/helpers/pydev'])
if 'setup' in dir(django): django.setup()
import django_manage_shell; django_manage_shell.run("/home/uc/Projects/c2public")
Python 2.7.12 (default, Nov 19 2016, 06:48:10)
[GCC 5.4.0 20160609] on linux2
Django 1.9.9
log = logging.getLogger('django')
Traceback (most recent call last):
  File "<input>", line 1, in <module>
NameError: name 'logging' is not defined
import logging
log = logging.getLogger('django')
log.error("test error 1")
ERROR test error 1
a = {frozenset([1,2,3]):4}
log.error("test error 2")
ERROR test error 2
log.error("test error 3", extra={'locals': locals()})
ERROR test error 3
Top level Sentry exception caught - failed creating log record
test error 3
Traceback (most recent call last):
  File "/home/uc/envs/c2public/local/lib/python2.7/site-packages/raven/handlers/logging.py", line 89, in emit
    return self._emit(record)
  File "/home/uc/envs/c2public/local/lib/python2.7/site-packages/raven/contrib/django/handlers.py", line 33, in _emit
    return super(SentryHandler, self)._emit(record, request=request)
  File "/home/uc/envs/c2public/local/lib/python2.7/site-packages/raven/handlers/logging.py", line 180, in _emit
    extra=extra, date=date, **kwargs)
  File "/home/uc/envs/c2public/local/lib/python2.7/site-packages/raven/contrib/django/client.py", line 284, in capture
    result = super(DjangoClient, self).capture(event_type, **kwargs)
  File "/home/uc/envs/c2public/local/lib/python2.7/site-packages/raven/base.py", line 610, in capture
    self.send(**data)
  File "/home/uc/envs/c2public/local/lib/python2.7/site-packages/raven/base.py", line 703, in send
    message = self.encode(data)
  File "/home/uc/envs/c2public/local/lib/python2.7/site-packages/raven/base.py", line 745, in encode
    return zlib.compress(json.dumps(data).encode('utf8'))
  File "/home/uc/envs/c2public/local/lib/python2.7/site-packages/raven/utils/json.py", line 51, in dumps
    return json.dumps(value, cls=BetterJSONEncoder, **kwargs)
  File "/usr/lib/python2.7/json/__init__.py", line 251, in dumps
    sort_keys=sort_keys, **kw).encode(obj)
  File "/usr/lib/python2.7/json/encoder.py", line 207, in encode
    chunks = self.iterencode(o, _one_shot=True)
  File "/usr/lib/python2.7/json/encoder.py", line 270, in iterencode
    return _iterencode(o, 0)
TypeError: keys must be a string

Hmm, definitely seems bad. Lemme see if I can fix this today.

Oh. Wait. This is definitely going to be tricky. I'm not sure how we can reasonably translate this to JSON. Since in JSON, you can use a list as a key for an Object.
@mitsuhiko thoughts? The only thing I can think of is just using the repr() and force it to a string.

This seems to be what the internet suggests doing as well. Or storing as key/value pairs instead of a dictionary. Leaning towards just coercing the keys to strings with repr().

I thought we already repr() keys.


docs: Improve wording on always=true validator section for the migration guide

Change Summary
Small change in the docs

Related issue number

Checklist

[x] The pull request title is a good summary of the changes - it will be used in the changelog
[ ] Unit tests for the changes exist
[ ] Tests pass on CI
[x] Documentation reflects the changes where applicable
[x] My PR is ready to review, please add a comment including the phrase "please review" to assign reviewers

please review


Changed package and app names to "Google Drive"

Referenced in Issue #29, changed names in the pkg recipe to "GoogleDrive.pkg" and "Google Drive.app" to reflect Google's name changes for the app.

Well, Google being Google, I guess.  Renaming it back to what it was before, that's cool...
Thank you!


No update registered

Bug report
Describe the bug
My bashlight doesn't ask for a update, when new tag was opened on same day as I updated bashlight
To Reproduce
Steps to reproduce the behavior:

Update bashlight to some tag
Ask André to release a new tag
Start a new terminal
Annnnd. Nothing like a update message comes up.

Expected behavior
You start a new terminal / bash and bashlight asks for a update.
Additional context
Ok, so hear me out about this:
So, today bashlight got the tag v0.2.0 and I upgraded bashlight on my laptop to v0.2.0 and then I began with helping develop v0.2.1.
When André released v0.2.1 bashlight didn't pickup that there was a new update.
Workaround
You need to edit ~/.bin/bashlight/DAY_UPDATE
and subtract 1 from the number that is in that file.
So:
BEFORE: FILE: ~/.bin/bashlight/DAY_UPDATE
015
AFTER: FILE: ~/.bin/bashlight/DAY_UPDATE
014
And then relaunch the terminal.
Checklist

[x] Description
[x] Reproduce
[x] Expected behavior

Oh...
I've just noticed, this is the expected behaviour.


No 'Access-Control-Allow-Origin' header is present on the requested resource (Hugo)

Describe the bug
When deploying a site the CSS gives an error:
Access to CSS stylesheet at 'https://endormi.io/css/coder.min.28d751104f30c16da1aa1bb04015cbe662cacfe0d1b01af4f2240ad58580069c.css' from origin 'https://5da9bf15c0cd28c348beafe0--endormi-portfolio.netlify.com' has been blocked by CORS policy: No 'Access-Control-Allow-Origin' header is present on the requested resource.

Weirdly enough checking the https://deploy-preview-15--endormi-portfolio.netlify.com/ works perfectly and https://endormi-portfolio.netlify.com/ doesn't.
Also the main site which is https://endormi.io, doesn't display anything. It tries to use my old code.
To Reproduce

Deploy site
Checking link

Expected behavior
Displays site and gets the css
Applicable Versions:

Git provider: GitHub
OS: Windows 10
Browser version  chrome 77.0.3865.120

There is not an actual issue here. This is how CORS works.

Weirdly enough checking the https://deploy-preview-15--endormi-portfolio.netlify.com/ works perfectly

Because it is using the https://deploy-preview-15--endormi-portfolio.netlify.com/css/coder.min.28d751104f30c16da1aa1bb04015cbe662cacfe0d1b01af4f2240ad58580069c.css instead of the cross domain link https://endormi.io/css/coder.min.28d751104f30c16da1aa1bb04015cbe662cacfe0d1b01af4f2240ad58580069c.css
You can inspect your page in the browser and track these using the network tab to see.

Okay, thanks 👍. I actually just fixed it. Now the issue that I have is my website doesn't display anything https://endormi.io and the preview deploy works https://master--endormi-portfolio.netlify.com/. So the issue is that it uses my old gatsby code which I now updated to Hugo.
What are some possible fixes?

I see your site  https://endormi.io just fine.

I'm going to close this issue, because this repository is for netlify-cms exclusively. You can go to https://community.netlify.com if you continue to have any issues.


TST/ERR: Add Period ops tests / fix error message

[x] related to #13242

closes #13251

[x] tests added / passed
[x] passes git diff upstream/master | flake8 --diff

This must be after #13079.

merged #13079
see my comments on #13251 (may already be covered here)

thanks @sinhrks all of these tests are really helpful!


binary request for static maps

This fixes staticMaps function by properly setting request configuration

Hi @cmyip thanks for contributing. Could you please point me to the documentation page so I can check the binary parameter and possibly add some info to the README?

hi @fabriziomoscon no reference found, but it was a stack overflow question:
http://stackoverflow.com/questions/14855015/getting-binary-content-in-node-js-using-request

I found a test
https://github.com/request/request/blob/master/tests/test-body.js#L51-L55
and docs

encoding - Encoding to be used on setEncoding of response data. If null, the body is returned as a Buffer. Anything else (including the default value of undefined) will be passed as the encoding parameter to toString() (meaning this is effectively utf8 by default). (Note: if you expect binary data, you should set encoding: null.)

I think this is the relevant source code from request: https://github.com/request/request/blob/master/request.js#L1012-L1045
Although I think it is counter intuitive to set something to binary to achieve a null value. Perhaps we could consider a condition that checks whether encoding is null (since it is a function parameter it is null only if the user has specified it so, opposite to undefined where the user omits it).
if (encoding == null) {
    options.encoding = null;
} else if (encoding) { // we check this so we don't assume that request is checking for undefined
    options.encoding = encoding;
}

What do you think?