Hey there as mentioned in #115 here are some questions that came up during my first encounter with genshinDB.
First of all: THANK YOU for this great package ❤, I am happy to support with PRs once I wrapped my head around a couple of things:

How do I get all items of a similar type - like all slime items or all drops from childe - at best if I am already on a slime item, how do I know there are similar ones "left and right", meaning how can I easily check similar items
I see there is genshinDB.weaponmaterialtypes and genshinDB.talentmaterialtypes

Both of them link to examples in the README but apparently there are no examples.
Why are there no / it would be great if there are genshinDB.levelmaterialtypes and genshinDB.ascensionmaterialtypes

all the links to the categories.json from the README point to 404
why is rarity a string and not an integer? It would be helpful if I could filter for rarity > X without the need to run parseInt first.
How to get the max rarity per item? I would like to get a list of all materials but only the max possible one - for example slime can only go up to 3 while horn can go up to 4
Why does genshindb.materials('ballad') only give me one Ballad item and not all items that match ballad?

Currently there's no easy way to get items of similar type. The closest for slime items is by searching enemies for slime. https://genshin-db-api.vercel.app/api/enemies?query=slime. Sounds like a good idea to add but I'm not sure on the best way to implement this.
weaponmaterialtypes and talentmaterialtypes aren't updated. I've been meaning to remove them and implement it via some sort of pseudo folders.
I was planning to create categories.json in my postprocessing step.
rarity is a string because I accidentally made it a string early on and I decided to be consistent from then on. This can be changed to number probably. Not sure if it'll break anything for other people.
hmm
Unfortunately not possible yet. I'm currently considering a return number of results option. I've been stalling because I couldn't figure out how to create the type for this. Maybe I should just create a separate function instead.

Alright, towards the weekend I'll try to come up with some ideas how we could address some of the points 🤞🏻

We could maybe add an array of tags to every item, so that one can find similar items more easily
Hmm OK - I would need an easy way to get items by categories:

Talent Level-Up Material (Books) --> "Ballad"
Talent Level-Up Material (Weekly Boss) --> "Dvalin's Plume"
Weapon Ascension Material --> "Aerosiderite"
Character Ascension material (Gems) --> "Agnidus Agate Gemstone"
Character Ascension material (Normal Boss Drops) --> "Hurricane Seed"
Common Ascension Material --> General Enemy / Elite Enemy --> "Slime"

With dynamic typing it also works okayish right now, it's just a bit dirty to do '3' + 1
This might be solved if we can distinguish between general enemy drops and elite enemy drops, general are always 1,2,3 - elite are always 2,3,4. We could add another property like elite: true or so?

BTW - since you said you are interested for what it's used:
I am building a farming helper:
https://primajin.github.io/genshin-farming-helper/
https://github.com/Primajin/genshin-farming-helper
This allows you to choose the items you want to track and every time you kill a monster or finish a domain you can quickly tap how many of which item you got. It will automatically count up 3 of each to the next level (unless you tick the lock).

im gonna add data for crafting

Alright great, for now I added a temporary static list https://github.com/Primajin/genshin-farming-helper/blob/main/src/constants/index.js#L21
Also added some fancy design - so the app can be tested in Beta 😍


Implement a Maxheap using array

💥 Proposal
Implementation of the max-heap using array in C++.
->TC= O(logn)
-> SC=O(n) space taken by array
Have you read the Contributing Guidelines on Pull Requests?
yes

/assign


WIP blind

use BSD 3-clause license
update dependencies
remove obsolete files
fix quickstart

Codecov Report

Merging #13 into master will increase coverage by 0.03%.
The diff coverage is 77.41%.

@@            Coverage Diff             @@
##           master      #13      +/-   ##
==========================================
+ Coverage   96.77%   96.81%   +0.03%
==========================================
  Files          46       43       -3
  Lines        3506     3517      +11
  Branches      369      369
==========================================
+ Hits         3393     3405      +12
+ Misses         78       77       -1
  Partials       35       35

Impacted Files
Coverage Δ

skhubness/analysis/estimation.py
79.56% <ø> (ø)
:arrow_up:

skhubness/reduction/local_scaling.py
92% <100%> (ø)

skhubness/reduction/mutual_proximity.py
100% <100%> (ø)
:arrow_up:

skhubness/reduction/__init__.py
100% <100%> (ø)
:arrow_up:

skhubness/reduction/base.py
66.66% <66.66%> (ø)

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 a962720...89dd038. Read the comment docs.


Add pull request template and issue templates

This is an automatic batch pull request.

Codecov Report

Merging #154 into dev will not change coverage.
The diff coverage is n/a.

@@         Coverage Diff         @@
##            dev   #154   +/-   ##
===================================
  Coverage    76%    76%
===================================
  Files        11     11
  Lines      1236   1236
===================================
  Hits        942    942
  Misses      294    294

Pull-request updated, HEAD is now 4630f2cf085916907c2e2f4d81d5c15881774f86


MainPage.idl not in Bookstore project

So I am working through the example in this page "XAML controls; bind to a C++/WinRT property." I get to the step to open MainPage.idl, but that midl file doesn't exist in my project. I tried to go through the steps for this as outlined in the earlier part but I still can't get things to parse out or build properly. Yes I'm a dinosaur who hasn't touched MSVCPP since MFC was the only game in town so I'm trying to learn this complex stuff.
Thank you for your time.
Peter R Vermilye
<EMAIL_ADDRESS>

Well, duh. I created the wrong app type when I started. There are just so many things VS can do. Sorry for the confusion on my part.
Peter


Update download freezes when switching WiFi networks.

Switching WiFi networks halfway through update causes the download to freeze.
Steps to reproduce

Update LBRY w/update release.
Switch WiFi networks.

Expected behaviour
It should just pick up where it left off when back in WiFi zone.
Actual behaviour
It just freezes.
System Configuration
This issue is I think universal. Not a priority but good for regular Joes who might one day want to use LBRY.

This issue was moved to lbryio/lbry#701

Reopening per discussion in : https://github.com/lbryio/lbry/issues/701#issuecomment-406360004
Relevant: https://github.com/electron-userland/electron-builder/issues/3171

@tzarebczan  Can you reproduce this?

This might (or might not) have an impact:
https://developer.mozilla.org/en-US/docs/Web/HTTP/Range_requests


file_get_contents(internal://provided-schema/code/docs/API/V1/schema.json): failed to open stream: No such file or directory

Hi, I'm getting these errors:
file_get_contents(internal://provided-schema/code/docs/API/V1/schema.json): failed to open stream: No such file or directory

By debugging it I saw that my JSON file is recognized and parsed by src/OpenApiV3Validator.php:79
However the SchemaStorage apparently cannot read it.
About my environment, I'm running this test inside my Docker php:7.3-fpm image.
Might be related to https://github.com/justinrainbow/json-schema/issues/538

A little bit of context for this:
It works if I put my schema.json file online somewhere (like a gist), however when I try to load it locally this error happens.

Ok, the way I solved this was, instead of giving the schema json file path like this:
        $validator = $validatorFactory->v3Validator(_DIR_'/../../../docs/API/V1/schema.json');

I declared it like file:// protocol (http://php.net/manual/en/wrappers.file.php)
        $validator = $validatorFactory->v3Validator('file:///code/docs/API/V1/schema.json');

I don't know if this is really this repository's fault or justinrainbow/json-schema's fault. But anyway...

I believe this is related to justinrainbow/json-schema. I added my suggestion here: https://github.com/justinrainbow/json-schema/issues/538#issuecomment-1126214268


merge_lora.py incompatible with diffusers 0.17

Got an error with missing keys in state dict when checking the VAE when using diffusers 0.17. When downgrading to 0.16 the VAE loads correctly in the merge_lora.py script.
This is the command I was running.
python3 merge_lora.py \
                      --sd_model v1-5-pruned.ckpt \
                      --save_to merged_models.safetensors \
                      --models lora_1.safetensors lora_2.safetensors \
                      --ratios 0.3 1

I've written a script to revert the new VAE state dict format to the old format.
Here's the script and see if it works for you. https://github.com/kohya-ss/sd-scripts/pull/618/files


Award Configurations - PT CSG-DEP (Award)

User Story
As a student, the amount of CSG-DEP I receive should follow the policy and reflect the correct amount if I'm between the income cap and the income threshold.
Current Behaviour
CSGD Award Amount (federalAwardCSGDAmount) is calculated as the max weekly limit if the family income is less than the gross income for the max grant (income cap) based on family size or the larger of weekly max (hardcoded) or the sloped weekly amount (which will always be less than the max weekly). Beyond that it does NOT account for the number of weeks of study.

  if (calculatedDataTotalFamilyIncome <= dmnPartTimeAwardFamilySizeVariables.limitAwardCSGDIncomeCap)
then dmnPartTimeAwardAllowableLimits.limitAwardCSGD3OrMoreChildAmount
else
  max(dmnPartTimeAwardAllowableLimits.limitAwardCSGD3OrMoreChildAmount - ((calculatedDataTotalFamilyIncome - dmnPartTimeAwardFamilySizeVariables.limitAwardCSGDIncomeCap) * dmnPartTimeAwardFamilySizeVariables.limitAwardCSGD3OrMoreChildSlope),
     80
  )

Expected Behaviour
CSGD should calculate the weekly amount as follows:

if income is below the income cap for family size, use the maximum weekly amount
if income is above, use the maximum of (weekly limit - (family income - income cap)*slope) or 0

CSGD should calculate the total award using that weekly amount:

if that weekly amount * offering weeks is less than the weekly limit, use weekly limit otherwise use weekly amount * offering weeks (rounded to nearest full dollar)
use the lesser of the above amount or program year limit

min(
	round(
		max(
			if (calculatedDataTotalFamilyIncome <= dmnPartTimeAwardFamilySizeVariables.limitAwardCSGDIncomeCap)
			then
				dmnPartTimeAwardAllowableLimits.limitAwardCSGD3OrMoreChildAmount
			else
				max(dmnPartTimeAwardAllowableLimits.limitAwardCSGD3OrMoreChildAmount - ((calculatedDataTotalFamilyIncome - dmnPartTimeAwardFamilySizeVariables.limitAwardCSGDIncomeCap) * dmnPartTimeAwardFamilySizeVariables.limitAwardCSGD3OrMoreChildSlope),
			     0
			)
			* offeringWeeks,
			dmnPartTimeAwardAllowableLimits.limitAwardCSGD3OrMoreChildAmount
		)
	),
		dmnPartTimeAwardAllowableLimits.limitAwardCSGDAmount
)

Additional Context
MASSvsSIMS_AssessmentComparison.xlsx
Acceptance Criteria

[ ] Update federalAwardCSGDAmount calculation for the 3 or More Dependant stream based on above calculation
[ ] Update federalAwardCSGDAmount calculation for the 2 or less Dependant stream based on above calculation, substituting limitAwardCSGD2OrLessChildAmount anywhere there is limitAwardCSGD3OrMoreChildAmount
[ ] Ignore rounding in this ticket; it will be analyzed and addressed in a future ticket.

PY limit deductions(dmnPartTimeAwardAllowableLimits) are not part of this ticket as per grooming discussion. As PY limit deductions are going to be handled in separate ticket.


Limitations of Safe Links for Public Folders

The "Safe Links" feature, which is part of Microsoft Defender for Office 365, does not work for email enabled public folders. Unfortunately, this restriction is not documented.
https://docs.microsoft.com/en-us/microsoft-365/security/office-365-security/safe-links?view=o365-worldwide

@FrogThomas Thank you for your feedback. We have made changes to the document via PR#https://github.com/MicrosoftDocs/microsoft-365-docs/pull/5062
Once this is approved by the author you will see the changes reflect on the document.
Thanks
Sri

@msbemba Thank you very much for the contribution and updating the documentation with PR. @FrogThomas Hope this update is helpful for you. Thanks for taking out some time to open the issue. Appreciate and encourage you to do the same in future also.


Parse all XML as proper strings

About the Contributor

This pull request is posted on behalf of the NRK.
Type of Contribution
This is a: Bug fix / Code improvement / BIG refactoring
Current Behavior

As reported in #92, strings can be improperly parsed as numbers in incoming xml data.
This exposed a larger issue, where there where internal issues with type safety when parsing the incoming data as any.
New Behavior

Incoming data are no longer automatically parsed to native types (such as number or boolean).
All incoming data are treated as strings and converted to their specified type per-property.
Changes in types:

<>

Testing Instructions

Other Information

Status

[ ] PR is ready to be reviewed.
[ ] The functionality has been tested by the author.
[ ] Relevant unit tests has been added / updated.
[ ] Relevant documentation (code comments, system documentation) has been added / updated.

Codecov Report
Attention: Patch coverage is 57.06861% with 413 lines in your changes missing coverage. Please review.

Project coverage is 75.03%. Comparing base (ffb8bf6) to head (2964dd4).
Report is 1 commits behind head on master.

Files
Patch %
Lines

...ages/helper/src/mosModel/profile2/xmlConversion.ts
31.00%
69 Missing :warning:

...ages/helper/src/mosModel/profile0/xmlConversion.ts
23.52%
65 Missing :warning:

...ages/helper/src/mosModel/profile3/xmlConversion.ts
19.23%
63 Missing :warning:

packages/helper/src/mosModel/parseMosTypes.ts
59.54%
53 Missing :warning:

...ages/helper/src/mosModel/profile1/xmlConversion.ts
60.30%
52 Missing :warning:

packages/helper/src/mosModel/lib.ts
65.00%
28 Missing :warning:

...ages/helper/src/mosModel/profile4/xmlConversion.ts
16.00%
21 Missing :warning:

packages/connector/src/MosConnection.ts
34.61%
17 Missing :warning:

...ckages/connector/src/connection/mosSocketClient.ts
54.83%
14 Missing :warning:

packages/helper/src/mosModel/ParseError.ts
75.60%
10 Missing :warning:

... and 8 more

Additional details and impacted files
@@            Coverage Diff             @@
##           master     #103      +/-   ##
==========================================
- Coverage   77.55%   75.03%   -2.53%
==========================================
  Files          65       67       +2
  Lines        3596     3997     +401
  Branches      835      949     +114
==========================================
+ Hits         2789     2999     +210
- Misses        748      997     +249
+ Partials       59        1      -58

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


@conda-forge-admin, please rerender

Additional comment:
No response

Is it possible to add osx64, osx-arm64, linux-aarch64, and win support?

https://github.com/icl-utk-edu/papi/issues/190
According to upstream papi does not support macOS.
I am unsure about windows, it failed to build when I submitted this feedstock and I didn't look into it.
From your PR, it appears there is also some issue with cross compiling.


Unable to deploy program using solana cli 1.18.12

Problem
I tried to deploy my program to mainnet but i keep getting the following error.
Error: Error processing Instruction 3: invalid account data for instruction

i have allocated twice the size of program but still get the above error. I can deploy the program on devnet and localnet. I am wondering what might be the issue.

Move my question to stack exchange here.
https://solana.stackexchange.com/questions/13629/unable-to-deploy-program-using-solana-cli-1-18-12


Add About Content

This PR adds content to the about page to reflect the Invision Design. This branch is also currently staged at alice.preview.zooniverse.org/about. A couple items to note:

I initially attempted to use markdown entirely to feed in content from a JS object into a reusable AboutContent component, but I abandoned this for accessibility reasons and to keep the code from getting too rigid. That said, this PR covers a bit more lines and earlier commits here aren't reflective of the final result.
The ContentTable component has some setState code to determine the height of the table of contents. This was necessary as the component's position: sticky div needed a defined height to allow it to work properly.

Closes #148
tagging @beckyrother to provide UI feedback using the link above

Converting this to a draft as we make a few design and content changes.

Quick Review

Content update looks legit to me.
I should give it an Accessibility pass for good measure.
[minor] A curious note, when I'm not logged in:

I can navigate to the About page from the home page ("Link to Documentation" link) just fine.
However, when I type in http://localhost:3000/about directly in the web browser's URL bar, I get shunted back to the home page.

As far as I'm concerned, this PR already has a 👍 for the functionality side of things.

@wgranger Here are the requested updates to About page. Ping me if anything below is unclear.
THROUGHOUT THE DOC:

[ ] Can we add a space break between paragraphs? I’m finding it quite difficult to read in the current format. For example, in the Getting started section, the first paragraph ends with the words "read on." the next paragraph starts with "To build your transcription project, ... " There's a line break, but no space between the paragraphs. For ease of reading, I'd like to add a single space between paragraphs in the same section, but would defer to @beckyrother re: convention.

[ ] Need to make a choice regarding using bold or ‘single quotes’ to reflect buttons/options. @beckyrother do you have a preference for this one? Or just consistency? Note: I'll add specific requests for changes once I've had a chance to communicate with Becky about this.

SPECIFIC COMMENTS (section header in bold for ease of finding):
Reaching out to complete your setup

[ ]  space missing between ‘To do so,please email<EMAIL_ADDRESS>
Adding team members & levels of access

[ ] Missing explanatory text for ‘Viewer access: Tester’.  Should read as follows (following the same format as previous User options):

Viewers are able to do the following:

View subjects

Viewing subjects

[ ] “For each subject[change subject here to group], you will be able to see (and sort by!) the following information:”

Search

[ ] Add this sentence at the end of the paragraph (after sentence ending with “…by clicking on the available column headings.”): “Please note that ALICE only supports search within individual groups, not across an entire Workflow or Project.”

What am I looking at? Images, markings, and transcriptions

[ ] Please update final sentence: “For each individual transcription, you will see the Zooniverse username of the volunteer who submitted it (if they are[change to were] logged into a Zooniverse account), and the time and date the transcription was submitted.”

OPTICS

[ ] “Adjustable Parameters” needs to be a sub-heading of OPTICS

DBSCAN

[ ] “Adjustable parameters” needs to be in bold and should be a sub-heading of DBSCAN

Thanks @snblickhan! The first comment looks like it's been covered based on the checked item in #151 ('add 1em space between paragraphs`)
Bold vs single quotes, I'd suggest single quotes in case someone is viewing in plain text for some reason. But really, consistency is the most important thing.

Standardize formatting for referring to buttons (see above re: bold vs 'single quotes'):
Getting Started:

[ ] "Create your account by visiting https://www.zooniverse.org and clicking REGISTER[change from bold to 'REGISTER']."
[ ] "To create a project, visithttps://www.zooniverse.org/lab, sign into your account, and click Create a new project[change from bold to 'Create a new project']."

Setting up your project for use with ALICE:

[ ] Please set up your workflow using the Line Tool with a Text Entry Sub-task[remove bold].
[ ] "Select Add A Task[change from bold to 'Add A Task'] > Drawing[change from bold to 'Drawing']."
[ ] "In the Tool Selection Area[remove bold], choose Line[change from bold to 'Line'] and add a Text Sub-task[change from bold to 'Text Sub-task']."
[ ] image caption 'SERIES ILLUSTRATING THE PROCESS OF CREATING...': remove all bold from this caption.

Uploading subjects

[ ] "To add metadata to your subjects, include a manifest[remove bold] in your subject upload."
[ ] "The metadata can be accessed by volunteers within Project Builder projects (in the transcription interface), as well as by your collaborators in ALICE (on individual Subject[remove bold] pages)."
[ ] change Internal_ID to internal_id in first bullet point [leave as bold]
[ ] change Group_ID to group_id in second bullet point [leave as bold]
[ ] "The Group_ID[change to group_id] can be the Subject Set Name or ID used in the Project Builder, or you can choose another name. The Group ID[change to group_id] will be the sub-index for a given workflow

Using ALICE with your team:
Search

[ ] "Clicking the Search[change from bold to 'Search'] button will open a pop-up module..."
[ ] "To search for a specific subject, select the ID type (Zooniverse ID or Internal ID), enter the name, and click Search[change from bold to 'Search']."

Editing

[ ] "To change it, select an individual transcription from the list, or write your own in the space provided, and click Replace with selected[change from bold to 'Replace with selected']."
[ ] "To revert to the aggregated transcription, click on the line to reopen the module, select the aggregated transcription from the menu, and click Replace with selected.[change from bold to 'Replace with selected']."

Approving subjects

[ ] "To un-approve a subject, click the Approve button at the top of the page[add single quotes to 'Approve']"

NOTE: Unless noted above, all other currenlty-bold words can remain bold.
Other updates/typo fixes for About page:

[ ] "What am I looking at? Images, markings, and transcriptions" has a line break between "and" & "transcriptions" that needs to be removed.

[ ] Add text to "What am I looking at?" section (following the sentence "Each solid line represents a cluster of individual lines drawn by volunteers, which are represented by dotted lines of the same color."): Circles at the end of each line represent the dots placed by volunteers to generate the underline marking. An open circle represents the first dot placed by a volunteer; a closed circle represents the second dot.


[        ] -> +file:///Users/csells/flutter/packages/flutter/lib/src/widgets/table.dart
[        ] -> +file:///Users/csells/flutter/packages/flutter/lib/src/widgets/text.dart
[        ] -> +file:///Users/csells/flutter/packages/flutter/lib/src/widgets/text_selection.dart
[        ] -> +file:///Users/csells/flutter/packages/flutter/lib/src/widgets/texture.dart
[        ] -> +file:///Users/csells/flutter/packages/flutter/lib/src/widgets/title.dart
[        ] -> +file:///Users/csells/flutter/packages/flutter/lib/src/widgets/transitions.dart
[        ] -> +file:///Users/csells/flutter/packages/flutter/lib/src/widgets/unique_widget.dart
[        ] -> +file:///Users/csells/flutter/packages/flutter/lib/src/widgets/value_listenable_builder.dart
[        ] -> +file:///Users/csells/flutter/packages/flutter/lib/src/widgets/viewport.dart
[        ] -> +file:///Users/csells/flutter/packages/flutter/lib/src/widgets/visibility.dart
[   +2 ms] -> +file:///Users/csells/flutter/packages/flutter/lib/src/widgets/widget_inspector.dart
[        ] -> +file:///Users/csells/flutter/packages/flutter/lib/src/widgets/widget_span.dart
[        ] -> +file:///Users/csells/flutter/packages/flutter/lib/src/widgets/will_pop_scope.dart
[        ] -> +file:///Users/csells/flutter/packages/flutter/lib/widgets.dart
[        ] -> +file:///Users/csells/temp/pointer_fun/lib/main.dart
[        ] -> c81ee8bc-eecb-482f-8fe2-ab5487e98578 build/app.dill 0
[        ] Updating files
[ +106 ms] DevFS: Sync finished
[   +2 ms] Syncing files to device macOS... (completed in 1,603ms)
[   +1 ms] Synced 0.8MB.
[   +2 ms] Sending to VM service: _flutter.listViews({})
[   +3 ms] Result: {type: FlutterViewList, views: [{type: FlutterView, id: _flutterView/0x7fa45d845a20,
isolate: {type: @Isolate, fixedId: true, id: isolates/3540687767463351, name:
main.dart$main-3540687767463351, number:<PHONE_NUMBER>463351}}]}
[   +1 ms] <- accept
[        ] Connected to _flutterView/0x7fa45d845a20.
[   +7 ms] 🔥  To hot reload changes while running, press "r". To hot restart (and rebuild state), press
"R".
[   +2 ms] An Observatory debugger and profiler on macOS is available at:
http://<IP_ADDRESS>:52139/pItnAitVRM0=/
[        ] For a more detailed help message, press "h". To detach, press "d"; to quit, press "q".

~/temp/pointer_fun
➜ flutter analyze
Analyzing pointer_fun...
No issues found! (ran in 5.7s)

~/temp/pointer_fun took 7s
➜ flutter doctor -v
[✓] Flutter (Channel master, v1.8.2-pre.80, on Mac OS X 10.14.5 18F132, locale en-US)
    • Flutter version 1.8.2-pre.80 at /Users/csells/flutter
    • Framework revision ab26e1b4a7 (38 minutes ago), 2019-07-17 19:35:27 -0400
    • Engine revision 1af19ae67d
    • Dart version 2.5.0 (build 2.5.0-dev.1.0 2b3336b51e)

[✓] Android toolchain - develop for Android devices (Android SDK version 28.0.3)
    • Android SDK at /Users/csells/Library/Android/sdk
    • Android NDK location not configured (optional; useful for native profiling support)
    • Platform android-28, build-tools 28.0.3
    • Java binary at: /Applications/Android Studio.app/Contents/jre/jdk/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 1.8.0_152-release-1343-b01)
    • All Android licenses accepted.

[✓] Xcode - develop for iOS and macOS (Xcode 10.2.1)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Xcode 10.2.1, Build version 10E1001
    • CocoaPods version 1.7.3

[✓] Chrome - develop for the web
    • Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome

[✓] Android Studio (version 3.4)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin version 36.0.1
    • Dart plugin version 183.6270
    • Java version OpenJDK Runtime Environment (build 1.8.0_152-release-1343-b01)

[✓] VS Code (version 1.36.1)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension version 3.2.0

[✓] Connected device (2 available)
    • macOS  • macOS  • darwin-x64     • Mac OS X 10.14.5 18F132
    • Chrome • chrome • web-javascript • Google Chrome 75.0.3770.142

• No issues found!

I ran this demo on master and was unable to trigger any crash by moving mouse around. I would assume it is fixed (probably by https://github.com/flutter/flutter/pull/59883).
flutter run -d macos
Flutter 1.20.0-8.0.pre.31 • channel master • git@github.com:flutter/flutter.git
Framework • revision 0268096bcd (59 minutes ago) • 2020-07-09 17:11:01 -0400
Engine • revision a48f05f833
Tools • Dart 2.9.0 (build 2.9.0-20.0.dev 06cb010247)


DEPRECATION WARNING: Passing a string to call()

I found an issue DEPRECATION WARNING: Passing a string to call()
I found how to fix this. How can I submit my modification ?

Duplicate of https://github.com/zurb/motion-ui/issues/99. Resolved in https://github.com/zurb/motion-ui/pull/100


Bobjac/dev

Code for image list and add
Tested CLI and it is consistent with API.
Testing against portal is needed.

Added code for image delete


Fix issue #323

As per issue #323, npm modules installed as dev-dependencies were not respected as such and could cause crashes while the project was being built.
ping @atanasovg @blagoev @Plamen5kov @slavchev

:+1:

:+1:


Change LdapConnection SecureSocketLayer to get value from configuration

I had to change this option to be able to connect to the AD server

@marianahycit : Thank's a lot.
I actually need to rework a bit on this package in order to improve it. However, I think it's quite a good base in order to plug-in easily the LDAP within IdentityServer4.


Allow reading QVD files from other sources than filesystem

Hi,
I'm watching this library with great interest.
I was wondering if it was possible to change the API to allow supplying QVD files via bytes objects or via a supplied file handler.

I'm not completely sure if I understand what you mean... Do you want an option for constructing a QVD file based on a bytes object? Could you please post some minimal code example of the method/API you want?

Thanks for your quick response.
Yeah, that was pretty much my idea.
Currently the usage looks like this:
from pyqvd import QvdDataFrame

df = QvdDataFrame.from_qvd('sample.qvd')
print(df.head(5))

But for use cases where a qvd file would not be stored on disk, for example in object storage, it would be convenient not having to write it on disk first:
from pyqvd import QvdDataFrame
from google.cloud.storage import Client

client = Client()
bucket = client.get_bucket(MYBUCKET)
blob = bucket.get_blob(MYFILE)
downloaded_file = blob.download_as_bytes()
df = QvdDataFrame.from_qvd_bytes(downloaded_file)
print(df.head(5))

But I think sometimes it can be unwieldy to interact with bytes directly (bytes, because the raw data in .qvd files would lead to decode errors for string types). For the specific use case of interacting with object storages there is a library called smart_open that implements Pythons file API on top of object storages.
If PyQvd had an API to read from Python files, it could look like this:
from pyqvd import QvdDataFrame
import smart_open

with smart_open.open("url/to/my/object", "rb") as fin:
    df = QvdDataFrame.from_qvd_file(fin)

print(df.head(5))

I think this would be nice and generic as in this case the _read_data method could go from this:
def _read_data(self):
        """
        Reads the data of the QVD file into memory.
        """
        with open(self._path, 'rb') as file:
            self._buffer = file.read()

to this:
def _read_data(self):
        """
        Reads the data of the QVD file into memory.
        """
        if (isinstance(file, io.TextIOBase)
            or isinstance(file, io.BufferedIOBase)
            or isinstance(file, io.RawIOBase)
            or isinstance(file, io.IOBase)):
            try:
                self._buffer = self._file.read()
            except UnicodeDecodeError as e:
                raise Exception("Supply a raw file access. Use mode \"rb\" instead of mode \"r\"")

Exactly!
Thank you so much for your consideration.

I started working on it and added a feature branch. The commit ccb8604a40b45aa8f0a53b093a499db11f6ee85d add a first version of an extended API that supports reading (from_stream()) and writing (to_stream()) binary streams as an alternative to files. I modified your suggestion and limited the supported streams to binary streams (no text-based streams e.g. TextIOBase are supported). The binary stream must be a subclass of RawIOBase or BufferedIOBase.
from pyqvd import QvdDataFrame

some_stream = ...
df = QvdDataFrame.from_stream(some_stream)

...

other_stream = ...
df.to_stream(other_stream)

Feel free to check it out and comment it. If there are no changes or objections I would include the feature in the next minor release.

The requested feature of reading/writing binary streams as an alternative to local disk files is included in the next minor release v1.1.0.


refactor(ast): move StringLiteral definition higher up

Pure refactor. StringLiteral definition was sandwiched in the middle of RegExp-related code. Move it higher up in literal.rs.
All the rest of the diff is just re-ordering generated code.

#7263
#7270  👈
main

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

Join @overlookmotel and the rest of your teammates on  Graphite

Merge activity

Nov 13, 8:35 AM EST: The merge label '0-merge' was detected. This PR will be added to the Graphite merge queue once it meets the requirements.

This change was originally part of #7263. Have split it out into a separate PR to make the substantive changes in #7263 easier to review.


PE File Sections Missing Permissions

The PE loader is not propagating section permissions from the load file into section semantics.

Fixed in 2.1.2318-dev.


Preserve multiline strings

Like in Python you can do:
node1="""Just a string "with" quotes!"""

as described here: https://github.com/lightbend/config/blob/master/HOCON.md#multi-line-strings
Configurate reads it correctly but always saves it back as followed:
node1="Just a string \"with\" quotes!"

I suggest there should be some type of option to enable triple quotes for strings that contain a least one " quote.

Any chance that Sponge forks the typesafe repo to fix things like that?

Any chance that Sponge forks the typesafe repo to fix things like that?

I would rather not as maintaining yet another library is something I'm not sure we have the man power to provide.

On top of which, Given the under-specification of Hocon, I'd be reluctant to fork the current reference implementation.

Couldn't this be implemented with Mixins somewhere here: https://github.com/lightbend/config/blob/ea45ea3767a201933eeeb9c3f0f13eacc9b51f07/config/src/main/java/com/typesafe/config/impl/ConfigString.java#L82


Refresh()

Please keep the Refresh() method available in the next version. It solves the problem for us with ease.

Same, I needed it to update the elements in the virtual scroller when they were re-ordered from another part of the page.

+1
It's a great tool, keep it please!

+1
Same here.

It appears to be present on VirtualScrollComponent's prototype, as refresh.
Grab a reference in your component with @ViewChild(<your template ref here>) XYZ and you can call refresh on it, easy as XYZ.refresh().
Works on my machine anyway, w/ ngx-virtual-scroller 1.0.16.

Also for us. Refresh() method is import and fix some component problems. Keep it please.

+1

+1, great tool, keep it !

+1


Update video_tutorials.md

added TheNewBoston video tutorials link.

This is one of the best tutorial series since I guess 2011-12!

Yes absolutely, with these tutorials only i learnt android. :)


Bump scipy version

Just use latest scipy - it's causing conflicts with other projects to pin to such a specific version

Wow that introduced some weird bugs. I'm curious what changed in scipy that causes these tests to fail :\ Any ideas?

I am not sure if this PR introduced the failure. The previous PR by Henry got the same error when calling check_grad (see approx_derivative in scipy.optimize._numdiff.py)

Gotcha. Thank you for clarifying @mmahsereci.

By looking at a few last builds, it looks as though at some point builds started pulling scipy 1.5 instead of 1.4, and started failing. Examples:

This PR, failed: Downloading scipy-1.5.2-cp37-cp37m-manylinux1_x86_64.whl (25.9 MB)

Build 3 months ago, succeeded: Downloading scipy-1.4.1-cp36-cp36m-macosx_10_6_intel.whl (28.5 MB)

PR submitted by @henrymoss, failed: Downloading scipy-1.5.1-cp36-cp36m-macosx_10_9_x86_64.whl (28.8 MB)

I have experimented locally and confirm that the error is caused by the scipy version. In the same virtual env, tests pass with scipy==1.4.1 and fail with scipy==1.5.1.

@marpulli perhaps you were modifying a different version of the code... right now what I see in changes is going from scipy>=1.1.0 to scipy=>1. Which to me does not make much difference, and isn't very specific.

Ping @marpulli. Are you happy with how reqs file looks right now, or shall we continue with this PR?

Codecov Report

Merging #315 into master will not change coverage.
The diff coverage is 100.00%.

@@           Coverage Diff           @@
##           master     #315   +/-   ##
=======================================
  Coverage   89.47%   89.47%
=======================================
  Files         123      123
  Lines        4019     4019
  Branches      459      459
=======================================
  Hits         3596     3596
  Misses        328      328
  Partials       95       95

Impacted Files
Coverage Δ

emukit/model_wrappers/gpy_model_wrappers.py
83.33% <100.00%> (ø)

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 3c7c114...451dc6f. Read the comment docs.

Hey, @apaleyes I was relaxing the constraint so that we would bump up to new minor versions automatically, but happy to set it to >=1.5 instead.

Sorry Mark, I really don't follow. What's wrong with just >=1.1.0 as it is now? Esp since we seem to have addressed the issue that was causing the tests to fail, so >=1.5 is not really necessary.  Sorry for being slow on this one.

It's me being slow 😆 . I was thinking that >=1.1.0 would allow any version 1.1.x  but not 1.2.x but that doesn't seem to be true. Closing!

Phew! Thanks Mark! I actually had to do some research to convince myself this is the case. But seems like it is!


Add options for delaycompress and rotation handling

Allow delaycompress to be turned off, and give the option to use something besides copytruncate mode.
I'm working on setting up a Fluentd logging solution and needed something to handle logrotation. This project seems like a great way to go, just needed to be able to turn off copytruncate mode because Fluentd doesn't handle that properly (it tracks logs by inode and offset).
I probably will end up using delaycompress anyway, but I haven't completely decided yet and I figured the option to turn it off would be nice.
The defaults settings are unaffected by my changes. LOGROTATE_DELAYCOMPRESS must be explicitly set to false to turn it off, and copytruncate is the default when LOGROTATE_MODE is not set.

Thank you!


fix: :bug: Fixing name of icon that is built with plugin

Needed to use package name for plugin icon name

I got the same commit on my WIP branch also...  :-)
👍🏼


makedpx problem

I'm having some difficulty with makedpx.py while trying to transcode batches of adverts from tiff to dpx for the iapi event
it worked fine for two batches a few weeks ago, but now it keeps coming up with an error message.
It happens just after the manifest is created on the first advert in the bunch, - just as it begins to transcode.
it seems to be a problem with naming in premis.
i'm attaching the error

i'll be at a meeting for an hour directly after lunch. but the terminal window is open on the macmini if you want to take a look

I'd imagine that the script is expecting the adverts project filenames
scheme. I'll take a look after lunch.
On 31 Aug 2017 13:07, "raecasey"<EMAIL_ADDRESS>wrote:
i'll be at a meeting for an hour directly after lunch. but the terminal
window is open on the macmini if you want to take a look
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/kieranjol/IFIscripts/issues/218#issuecomment-326275737,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABEyvorCU-Y8HFg3QlJrcCQOJkedVZuoks5sdqIEgaJpZM4PIu0u
.

So the issue was that there was no Premis xml in the source directory, so
it couldn't harvest the data required. I think we can close this but feel
free to reopen.
On 31 Aug 2017 13:10, "Kieran O Leary"<EMAIL_ADDRESS>wrote:

I'd imagine that the script is expecting the adverts project filenames
scheme. I'll take a look after lunch.
On 31 Aug 2017 13:07, "raecasey"<EMAIL_ADDRESS>wrote:
i'll be at a meeting for an hour directly after lunch. but the terminal
window is open on the macmini if you want to take a look
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/kieranjol/IFIscripts/issues/218#issuecomment-326275737,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABEyvorCU-Y8HFg3QlJrcCQOJkedVZuoks5sdqIEgaJpZM4PIu0u
.


Compile for CDN (and SRI)

Feature Proposal
This is the last version of Chart.js which supports CDN with SRI:
<EMAIL_ADDRESS>Currently, there is no upgrade path for this version.
Can we please again release minified CDN versions which can be safely delivered with SRI?
Possible Implementation
No response

I don't have any knowledge of SRI, so maby I am seeing this wrong but<EMAIL_ADDRESS>is already minified so if the minification is the issue this is the solution right?

Cool, thank you. I just tried running that file and it is working with SRI. The issue here might be we are not using the expected convention.
For example, on all versions up to 3.9.1 our main UMD file is detected. Example:
https://www.jsdelivr.com/package/npm/chart.js?tab=files&version=3.9.1

However, on more recent versions, our UMD file is not found. jsDelivr is taking it upon itself to "minify" this file for us because it cannot find the minified file.
https://www.jsdelivr.com/package/npm/chart.js?tab=files&version=4.3.3

Maybe the solution is we need to name the file differently or specify in package.json. I'm investigating now.

I see that we have specified it here:
https://github.com/chartjs/Chart.js/blob/v4.3.3/package.json#L13-L15
And in the past we did similarly:
https://github.com/chartjs/Chart.js/blob/v3.9.1/package.json#L7-L9

jsDelivr documentation on this is at: https://www.jsdelivr.com/documentation#id-configuring-a-default-file-in-packagejson
So the problem is that jsDelivr is expecting a minified JavaScript file will always have the extension *.min.js. Whereas we are publishing chart.umd.js.
I think the solution here is to follow established conventions and that we should publish this file as chart.umd.min.js.
This could even be a copy of the existing file if that makes publishing easier.


Can I use this project for semantic segmentation? Cause I also cares about the segmentation and the category of the background.

Hi, great work! Can I use this project for semantic segmentation? Cause I also cares about the segmentation and the category of the background.

I want to achieve semantic or panoptic segmentation, if Grounded-SAM can not do this, what's the recommended model?

@Luciennnnnnn Hello, G-SAM can do panoptic segmentation tasks, you can refer to this project as a good example:

https://github.com/segments-ai/panoptic-segment-anything

@rentainhe Hi, thank you for your recommendation, i'll take a look at it.

So if I want to achieve automatic open-vocabulary panoptic segmentation, is my current best choice RAM++ + G-SAM?

So if I want to achieve automatic open-vocabulary panoptic segmentation, is my current best choice RAM++ + G-SAM?

Yep, maybe this is the simplest way to achieve a good result, you can visualize and double-check the results

@rentainhe That would be great! Thanks!


Feature/Submission New API

Test for submission invtation and paper submission for new API

@melisabok should we add the retry_declined functionality to the venue class as well? We do not use this from the request form.


graphql-XXX-transformer packages depend on graphql-transformer-core and others using exact semvers

Describe the bug
I'm trying to depend on multiple graphql-XXX-transformer packages and the graphql-transformer-core package, so I can configure our own transformers in addition to some amplify transformers.
I need to pin down every version and explicitly run npm dedupe, because the transformers all use exact-version dependencies, and would get their own copy of the dependency -- but then compilation of my code breaks with errors such as this one:
Type 'import(".../node_modules/graphql-transformer-core/lib/TransformerContext").TransformerContextMetadata' is not assignable to type 'import(".../node_modules/graphql-key-transformer/node_modules/graphql-transformer-core/lib/TransformerContext").TransformerContextMetadata'.

Amplify CLI Version
The project is not using the amplify CLI directly, but has these amplify-cli dependencies:
├──<EMAIL_ADDRESS>├─┬<EMAIL_ADDRESS>│ ├──<EMAIL_ADDRESS>deduped
│ └──<EMAIL_ADDRESS>deduped
├─┬<EMAIL_ADDRESS>│ ├──<EMAIL_ADDRESS>deduped
│ ├─┬<EMAIL_ADDRESS>│ │ ├──<EMAIL_ADDRESS>deduped
│ │ └──<EMAIL_ADDRESS>deduped
│ └──<EMAIL_ADDRESS>deduped
├─┬<EMAIL_ADDRESS>│ └──<EMAIL_ADDRESS>deduped
├─┬<EMAIL_ADDRESS>│ ├─┬<EMAIL_ADDRESS>│ │ └──<EMAIL_ADDRESS>deduped
│ └─┬<EMAIL_ADDRESS>│   └──<EMAIL_ADDRESS>└──<EMAIL_ADDRESS>
Expected behavior
npm should be able to deduplicate the dependencies, even with minor/patch-level version differences.
Desktop (please complete the following information):

OS: [e.g. Mac/Windows/Ubuntu]: Fedora
Node Version: v14.15.1

Additional context
Add any other context about the problem here.

This has been fixed as a part of #6442 and would be a part of our next release.

Quick check, looks better already I think: Without running a dedupe explicitly, and with adding a '^' in front of all dependencies I now see this
$ grep graphql- package.json  | cut -f 1 -d : | xargs npm ls amplify-cli-core
├──<EMAIL_ADDRESS>├─┬<EMAIL_ADDRESS>│ ├──<EMAIL_ADDRESS>deduped
│ └──<EMAIL_ADDRESS>deduped
├─┬<EMAIL_ADDRESS>│ ├──<EMAIL_ADDRESS>deduped
│ ├──<EMAIL_ADDRESS>deduped
│ └──<EMAIL_ADDRESS>deduped
├─┬<EMAIL_ADDRESS>│ └──<EMAIL_ADDRESS>deduped
├─┬<EMAIL_ADDRESS>│ └──<EMAIL_ADDRESS>deduped
├─┬<EMAIL_ADDRESS>│ ├──<EMAIL_ADDRESS>deduped
│ └──<EMAIL_ADDRESS>deduped
└─┬<EMAIL_ADDRESS>  └──<EMAIL_ADDRESS>
Still suspicious is the dependency from graphql-transformer-code to amplify-cli-core though.


Feature request: Selection and hovering supported by themes

Hello! While trying to find a temporary solution of the bug of #5293 because I find it so annoying, I tried with all the custom themes of the other repositorie, but I've found that no one changes the colour of selecting items, so It's useless to fix the issue at hand.
So I think, in order to have a bypass until that issue is solved and for future custom themes, could be useful to have those colours supported by custom themes.
Is your feature request related to a problem?
#5293 , which is that on last main all views but column doesn't have a distinctive colour in light mode making difficult to check what file is selected.
Describe the solution
Support for custom themes to change the colour of hover and select items on view modes.
Describe alternatives you've considered
Fixing the main issue, but I tried and I can't find where are those colours.
Scope

Capability
Priority

This proposal will allow developers to add support for adding selection and hovering colours on custom themes.
Must

This proposal will allow end users to be able to replace those colours.
Should

System Information

OS Version: Windows 10 Pro 21H1 compilation: 19043.1055 Windows Feature Experience Pack: 120.2212.2020.0
App version: last commit of main branch.

Additional comment
My proposal is that users be able to change colours like this on last main commit:

To this colour on the last release:

or to another colour that fits better with the other colours of the custom theme.

Closed since this is already proposed on #4643


Update default serve options when using phonegap as a module

For the desktop app, we will now be using PG directly to serve projects. Many options that are passed to connect get defaults when run from the CLI but not when you require PG and call serve.

Resolved with above commit


Derive Macro need improvements

The AnimationEvent derive macro needs improvements.

target is too generic, should be animationevent(target) or something like that
I am pretty sure only named structs with members are supported. No UnitStruct, no TupleStruct.

Fixed with b4e53a1d383204a51fc35f1599e66fbb0eb83d56


Typo in Copyright header

https://github.com/thinkSRS/srsgui/blob/00556493ec1d712110c602aea9ec7c79b15c89bf/srsgui/examples/oscilloscope example/instruments/cg635.py#L2C5-L2C14
This is a recurring typo in the boilerplate Copyright header that I see throughout the source code. I don't know if the boilerplate was auto-generated somewhere, or manually copied-and-pasted, but in many places it reads "Coptright" instead of "Copyright", and "All right reserved" instead of "All rights reserved"
Should read:
##!
##! Copyright(c) 2022, 2023 Stanford Research Systems, All rights reserved
##! Subject to the MIT License
##!

Fixed.


Do not clear swarm directory at the begining of swarm init and swarm …

…join now.
However, do clear the directory if init or join fails, because we don't
want to leave it in a half-finished state.
Signed-off-by: Ying Li<EMAIL_ADDRESS>(cherry picked from commit bf3e9293a66c77a2fddf4e691222898846b4af9f)

Cherry pick of moby/moby#33341

Actually, this is a P2

LGTM
integration tests and unit tests are green


Peer/Team Review 2-27

Calissa:

Self
Person 1
Person 2
Average

N@tm
1/1

CTM
1/1

UI
5/5

JWT
0.7/1

Database
4.7/5

Deployment
1/1

API
1/1

Shraddha:

Self
Person 1
Person 2
Average

N@tm
/1

CTM
/1

UI
/5

JWT
/1

Database
/5

Deployment
/1

API
/1

Bria:

Self
Person 1
Person 2
Average

N@tm
/1

CTM
/1

UI
/5

JWT
/1

Database
/5

Deployment
/1

API
/1

For all...

N@TM: 1/1
CTM 1/1
UI 5/5
JWT 1/1
Database 4.5/5
Deployment 1/1
API 1/1

For everyone...
N@TM: 1/1
CTM 1/1
UI 5/5
JWT 1/1
Database 4.5/5
Deployment 1/1
API 1/1


feat: add publicly accessible

Specifically Im curious if publicly_accessible solves it or I need to create additional rules/routes to access from the internet? also, is there a way to only allow access from defined authorized networks? A bit lost in how best to define/build it here?

A bit more structured explanation by AWS:

https://aws.amazon.com/premiumsupport/knowledge-center/aurora-mysql-connect-outside-vpc/
https://aws.amazon.com/premiumsupport/knowledge-center/aurora-private-public-endpoints/

Hope it helps.

Thanks @antonbabenko ! and another question  - in your example, security_group_egress_rules but I don't see ingress or is it automatically ? because if the ingress is enabled why egress is needed as it is stateful anyway and allowed by default for anything incoming?

There is var.allowed_cidr_blocks which should be used to control ingress IPs.

Still no luck for some reason. Tried to connect. Allowed all for ingress and egress - any idea? any additional config I need to do outside of the rds module ?

I think if I use the database subnet then we also need to add a route table to internet gateway? looks like it's not possible to do it part of the module? or am I missing something?

Yes, I think you are right, you should configure VPC as described in the notes for the VPC module - Public access to RDS instances.

That worked! Thx!!

This PR is included in version 0.9.0 :tada:


SetSourceFilterVisibility requires filterEnabled to be of type boolean

SetSourceFilterVisibility's current filterEnabled property is of type string, so this call does not function as expected.

Seems good, it looks like this change has been made upstream too.


[OTHER] adding multiple links

What would you like to share or ask?
Hey, I want to add some links in these:

projects sub-category under OpenSource category
python,  javascript, C# sub-category under Languages category
blogs, e-book, hosting, project ideas sub-category in resources category
android, computer science subcategory in youtube category
ReactJS subcategory in frontend category

Checklist

[X] I have checked the existing issues
[X] I have read the Contributing Guidelines
[X] I am willing to work on this issue (optional)
[X] I am a GSSoC'23 contributor

Hey @akkshayTando! Just to confirm, you are doing GSSoC’23?

yes

yes

Done!

Heya I have some interesting links that can be added here for backend. Noje.js and JavaScript tutorials. I will be happy if you can assign me with this issue.

Heya I have some interesting links that can be added here for backend. Noje.js and JavaScript tutorials. I will be happy if you can assign me with this issue.

Hi @SamserAliKhan! :) Please it in another issue. It’ll make things more organized.

Can u say in which one. Because I can see that this is the one to add multiple links.

I mean create a separate issue for the links you want to add. Check the Contributing guide to help you. Sorry for not being clear @SamserAliKhan.


Capture unique statement count before doing any transformations

Proposed in this thread. This would let each parser do a sane job of counting how many real statements are used, instead of getting all crazy with trying to reverse-engineer what was once a meaningful statement from the transpiled output.

Originally I hadn't considered branches, but I think the most common case is probably an if/else or if/else if/else where the block that contains the most statements is probably the branch doing the work.
Of course, with loops where n is relatively large and you're expecting the common case to be a no-op and the exception being where you do work, that algorithm falls on its head.  Also, I have to consider that this is geared a lot towards those who are newer to programming and their code very well might not reflect what my gut says is the common case.
But I think it's still possibly more workable than trying to reverse engineer translated code if you accept a wider margin of error.  Anything else that I can think of involves injecting code into the emitted javascript to increment counters.
<thinkingOutLoud>
OTOH, that approach would allow for all languages to be supported if they can be translated to JS, but I suspect there would still be edge cases that are different for each language that would have to be dealt with on a per language basis and then we're back at square 1.
</thinkingOutLoud>

We can just count all unique statements; it's really not important whether they get run or not. The incentives are still there to minimize the amount of code you write.


adding a link to the transparency note for LVA

The documentation for LVA needs to link to the associated transparency note.

//cc @Juliako @naiteeks

@bennage, i think your proposal is good

Hi @Juliako Could you add #sign-off if the PR is ready to merge? Thanks!

Hi @bennage - This pull request was opened in the public repo. PMs should work in the private repo, per the Microsoft Docs contributor guide. We'll keep this PR open for review and merge, but in the future, would you make content updates in the private repo? Thanks!

#sign-off

@Juliako: I'm sorry - only the author of this article, @, can sign off on your changes. But we do have an exception process - if you are on the Microsoft content or product team for this product area, you can ask the PR review team to review and merge it by sending mail to the techdocprs alias.


[Bug]: pull appsmith-ce from docker-hub failed

Is there an existing issue for this?

[X] I have searched the existing issues

Description
i pull appsmith-ce from docker-hub,but meet :manifest for docker.io/appsmith/appsmith-ce:v1.9.17 not found
docker pull docker.io/appsmith/appsmith-ce:v1.9.17
Trying to pull repository docker.io/appsmith/appsmith-ce ...
v1.9.17: Pulling from docker.io/appsmith/appsmith-ce
v1.9.17: Pulling from docker.io/appsmith/appsmith-ce
manifest for docker.io/appsmith/appsmith-ce:v1.9.17 not found
Steps To Reproduce
docker pull docker.io/appsmith/appsmith-ce:v1.9.17
Trying to pull repository docker.io/appsmith/appsmith-ce ...
v1.9.17: Pulling from docker.io/appsmith/appsmith-ce
v1.9.17: Pulling from docker.io/appsmith/appsmith-ce
manifest for docker.io/appsmith/appsmith-ce:v1.9.17 not found
Public Sample App
No response
Environment
Release
Issue video log
No response
Version
1.9.17

@wengad82, I just tried the same pull command, and it seems to be working for me, this is quite strange. Do you see this error only with this image tag? Can you try docker pull docker.io/appsmith/appsmith-ce:latest or docker pull docker.io/appsmith/appsmith-ce:v1.9.16 to see if those fail as well please?
Also, can you share the output of docker info so we have a better understanding of the environment and perhaps reproduce it better please?
Thanks!

[root@bg10 ~]# docker pull docker.io/appsmith/appsmith-ce:v1.9.16
Trying to pull repository docker.io/appsmith/appsmith-ce ...
v1.9.16: Pulling from docker.io/appsmith/appsmith-ce
v1.9.16: Pulling from docker.io/appsmith/appsmith-ce
manifest for docker.io/appsmith/appsmith-ce:v1.9.16 not found
[root@bg10 ~]# docker info
Containers: 5
Running: 4
Paused: 0
Stopped: 1
Images: 129
Server Version: 1.13.1
Storage Driver: overlay2
Backing Filesystem: extfs
Supports d_type: true
Native Overlay Diff: true
Logging Driver: journald
Cgroup Driver: systemd
Plugins:
Volume: local
Network: bridge host macvlan null overlay
Swarm: inactive
Runtimes: docker-runc runc
Default Runtime: docker-runc
Init Binary: /usr/libexec/docker/docker-init-current
containerd version:  (expected: aa8187dbd3b7ad67d8e5e3a15115d3eef43a7ed1)
runc version: 8891bca22c049cd2dcf13ba2438c0bac8d7f3343 (expected: 9df8b306d01f59d3a8029be411de015b7304dd8f)
init version: fec3683b971d9c3ef73f284f176672c44b448662 (expected: 949e6facb77383876aeff8a6944dde66b3089574)
Security Options:
seccomp
WARNING: You're not using the default seccomp profile
Profile: /etc/docker/seccomp.json
selinux
Kernel Version: 3.10.0-957.el7.x86_64
Operating System: CentOS Linux 7 (Core)
OSType: linux
Architecture: x86_64
Number of Docker Hooks: 3
CPUs: 16
Total Memory: 31.26 GiB
Name: bg10
ID: KNIB:DG2X:TMKZ:T4TK:FIZR:XHAN:KAPB:ZZWC:HDAV:YGOP:4QOE:SVF5
Docker Root Dir: /data/docker
Debug Mode (client): false
Debug Mode (server): false
Registry: https://index.docker.io/v1/
WARNING: bridge-nf-call-iptables is disabled
WARNING: bridge-nf-call-ip6tables is disabled
Experimental: false
Insecure Registries:
<IP_ADDRESS>:5000
<IP_ADDRESS>/8
Registry Mirrors:
https://registry.docker-cn.com
http://hub-mirror.c.163.com
Live Restore Enabled: false
Registries: docker.io (secure)

@wengad82, thanks for sharing. I see two points that, I don't know if are causing this, but do get highlighted.

Your Docker server version (1.13.1), is from 2017, which is very old now. Is it possible to upgrade your Docker server version? If yes, can we do that and try please?
You have ~129 images on your system, but only 5 containers. So, I'm guessing a lot of those images aren't used and are taking up space on your system. You can do a docker system prune to clean up and claim your space back.

Let us know if the above suggestions help.

thank you a lot sharat87 ,i upgrade docker-io from 1.13.1 to docker-ce v2.17.3,and then pull the v1.9.17,it can pull down well.


Refactor/change to classes

Thank you for putting this together. While things will be moved towards using more classes, unfortunately, this contains too many changes to be able to be merged.


add-discordjs-link-readme #380

created a documentation section and added the discord.js link under resources

I think the Documentation section should be below requirements, should it not?

What is the issue # for this PR?

What is the issue # for this PR?

#380


import are not working

import * from 'leaflet';
unknown: Unexpected token, expected "as" in localbrowser while in codesandbox another issue
https://codesandbox.io/p/sandbox/modest-beaver-v23ptt?file=%2Fsrc%2Findex.js%3A2%2C1

Works for me.
If you've got problems to bundle Leaflet using rollup/webpack/parcel/whatever, that's your problem, not a Leaflet issue.

Hi, and thanks for taking the time to open a bug report in Leaflet.
However, in this repository we only handle bugs in "vanilla" Leaflet. This means that we do not handle bugs which are specific to frameworks such as:

React
Ractive
AngularJS
Bootstrap
Leaflet for R
Joomla
Wordpress
Ionic
Cordova
Vue
Electron
Polymer
TypeScript
D3
Folium
NativeScript
...etc

Please understand that we only have the time and energy to test Leaflet in plain web browsers.
Please try to either reproduce the bug without using any frameworks, or submit a bug to the appropriate repo.


feat: upgrade Google-Mobile-Ads-SDK from 9.0.0 to 9.14.0

Version 9.14.0 Google-Mobile-Ads-SDK makes it compatible with recent versions of Firebase Analytics that use GoogleAppMeasurement (= 10.3.0), whereas version 9.0.0 depends on GoogleAppMeasurement (< 9.0, >= 7.0).

closing as have merged the Google-Mobile-Ads-SDK update to 9.14.0 from another branch


Changes from new game patch

https://steamcommunity.com/games/553850/announcements/detail/4169846298204053637
Fixes #35

Hi Jason
I will look into this request when i get back to codespaces software
Thanks David


Create zspodskalkou.txt

@Kikismine First stage of verification passed, official domain for ZŠ Pod Skalkou is zspodskalkou.cz as far as I verified. This is just an informational note. Pull request is still in the review. I appreciate your patience while waiting for the final decision.

@Kikismine Thank you for your patience while waiting for a review and processing your request. Pull request merged.


Add truncation capability as to not create invalid messages

Hey, I added a capability to truncate fields according to the specified start and end positions
Lmk what you think

LGTM.
Overall I see why you need it ;-)
But...
On the other hand maybe it would be better to make Encoder returning Try or Either and be able to react on those errors?


feature: Add support for OIDC Configuration without well known openid-configuration

Description
Add Support for Authorization Server that do not have well-known openid-configuration
Configuration
## Override config
disable_well_known_configuration: true
insecure_well_known_endpoint: "https://another.domain.than.issuer.com/.well-known/openid-configuration"
token_endpoint: "https://example.com/oauth2/access_token"
authorize_endpoint: "https://example.com/oauth2/authorize"
userinfo_endpoint: "https://example.com/oauth2/userinfo"
jwks_endpoint: "https://example.com/oauth2/jwk.json"

Implementation
if disable_well_known_configuration: true then don't look up well known endpoint based on issuer url.

1 if  insecure_well_known_endpoint:  is defined, use lookup this alternative url. Using InsecureIssuerURLContext  (https://github.com/coreos/go-oidc/blob/v3/oidc/oidc.go#L87)
2 if  insecure_well_known_endpoint:  is  not defined,  create  a ProviderConfig (https://github.com/coreos/go-oidc/blob/366ac4a7e8b3be7ff45e6e1b0e1ab2e43b47d4ee/oidc/oidc.go#L139) based on the the static endpoint defined in the config, and create the oidc provier with func (p *ProviderConfig) NewProvider(ctx context.Context) *Provider  https://github.com/coreos/go-oidc/blob/366ac4a7e8b3be7ff45e6e1b0e1ab2e43b47d4ee/oidc/oidc.go#L168

Since release v0.11.0 (https://github.com/vdbulcke/oidc-client-demo/blob/main/oidc-client/main.go#L113) you only have to skip the well-known discovery and set static url from the config in the oidc.ProviderConfig
        // Create a oidc Provider Config manually
	providerConfig := &oidc.ProviderConfig{
		IssuerURL:   c.Issuer,
		AuthURL:     wk.AuthorizationEndpoint,
		TokenURL:    wk.TokenEndpoint,
		UserInfoURL: wk.UserinfoEndpoint,
		JWKSURL:     wk.JwksUri,
		Algorithms:  wk.IDTokenSigningAlgValuesSupported,
	}


fix bugs when sdk>=21 and camera fall back toCamera1

when fall back to Camera1,setting in XML or setting before start will lost

I singed it


tf.keras   TypeError: unsupported operand type(s) for +: 'NoneType' and 'float

Please make sure that this is an issue related to keras.
tag:keras_template
Important Notice
Please note that tf.keras code was moved entirely to
keras-team/keras repository
You can open any code/doc bugs, performance issues, and feature requests
in keras-team/keras repository
tf.keras related issues opened in
tensorflow/tensorflow repository may
not get attention as keras-team/keras
repository is dedicated for the development of keras code

@Winndy ,
Please take a look at this issues with similar error.1 and 2. It helps.Thanks!


Change RealtimePlumber to use SegmentAllocate API

@gianm per our conversation on the mailing list, I'm submitting a PR that changes the RealtimePlumber / RealtimeIndexTask to use the "SegmentAllocate" API.  Some notes:
I refactored the "segment tracking" code out of AppenderatorDriver and in to a standalone class, SegmentTracker so that this can be reused.
I struggled with the best way to add this in to the Plumber API given that the RealtimePlumber is used by both the RealtimeIndexTask and the RealtimeManager but it is instantiated in different scenarios.  I ended up passing the SegmentAllocator to PlumberSchool#findPlumber and passing the sequence name as an argument to Plumber#add.  I'm open to suggestions on ways to improve this but this seemed the best way, to me.
In RealtimePlumber, I'm using the task ID as the sequence name.  My understanding from reading the existing  code, particularly the KafkaIndexTask, is that the sequence name should be unique for the duration of the task.  Please correct me if my understanding is incorrect.  I'm not so certain about how to best generate a sequence name for the RealtimeManager though.  The test coverage here was pretty basic, and I don't have experience running realtime nodes, so I'm not sure if this is implemented properly.
In RealtimePlumber, I'm persisting the segment tracking metadata and the segments identifiers as JSON in files, similar to what is done to the identifier in AppenderatorDriver.  Please let me know if this is improper.  The other option would be to store them along with the index metadata but this didn't seem like the right place
Please let me know what feedback you have, thank you!

@gianm Should this behavior be on by default for all tasks or should it be configurable?

@gianm or @jihoonson do you have any thoughts on this one?

Sorry for not taking a look yet @kevinconaway, we're in the middle of getting master ready for an 0.11.0 branch cut and should be able to take a look after that.
Have you been running this patch in your own cluster?

We seem to have developed a need for this as well. I'm gonna build it and test it locally, then possibly push to our staging instance for some real load tests. Just FYI :-).

@KenjiTakahashi this is currently up to date with the 0.11.0 branch.
I'm not going to sync it with master for the time being to keep it stable for our testing.

@kevinconaway Thanks for the heads up, that should be fine with us. I've just tested it locally and it seems to be working as expected so far :-). We're gonna push it to try it out on staging probably tomorrow/early next week.

We've been running it on staging for a couple of days and for some time it worked fine. But now tasks are consistently failing with
2017-10-06T13:39:35,888 WARN [task-runner-0-priority-0] io.druid.segment.realtime.SegmentTracker - Cannot allocate segment for timestamp[2017-09-05T00:00:00.000Z], sequenceName[index_realtime_logdb_data_0_2017-10-06T13:39:26.855Z_ohinnidg].
2017-10-06T13:39:35,910 ERROR [task-runner-0-priority-0] io.druid.indexing.common.task.RealtimeIndexTask - Exception aborted realtime processing[logdb_data]: {class=io.druid.indexing.common.task.RealtimeIndexTask, exceptionType=class java.lang.NullPointerException, exceptionMessage=null}
java.lang.NullPointerException
        at java.util.concurrent.ConcurrentHashMap.get(ConcurrentHashMap.java:936) ~[?:1.8.0_131]
        at io.druid.segment.realtime.plumber.RealtimePlumber.getSink(RealtimePlumber.java:255) ~[druid-server-0.11.0-SNAPSHOT.jar:0.11.0-SNAPSHOT]
        at io.druid.segment.realtime.plumber.RealtimePlumber.add(RealtimePlumber.java:238) ~[druid-server-0.11.0-SNAPSHOT.jar:0.11.0-SNAPSHOT]
        at io.druid.segment.realtime.plumber.Plumbers.addNextRow(Plumbers.java:77) ~[druid-server-0.11.0-SNAPSHOT.jar:0.11.0-SNAPSHOT]
        at io.druid.indexing.common.task.RealtimeIndexTask.run(RealtimeIndexTask.java:350) [druid-indexing-service-0.11.0-SNAPSHOT.jar:0.11.0-SNAPSHOT]
        at io.druid.indexing.overlord.ThreadPoolTaskRunner$ThreadPoolTaskRunnerCallable.call(ThreadPoolTaskRunner.java:436) [druid-indexing-service-0.11.0-SNAPSHOT.jar:0.11.0-SNAPSHOT]
        at io.druid.indexing.overlord.ThreadPoolTaskRunner$ThreadPoolTaskRunnerCallable.call(ThreadPoolTaskRunner.java:408) [druid-indexing-service-0.11.0-SNAPSHOT.jar:0.11.0-SNAPSHOT]
        at java.util.concurrent.FutureTask.run(FutureTask.java:266) [?:1.8.0_131]
        at java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1142) [?:1.8.0_131]
        at java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:617) [?:1.8.0_131]
        at java.lang.Thread.run(Thread.java:748) [?:1.8.0_131]

@kevinconaway Any idea what might be causing this?

We've been running for ~2 weeks now and haven't experienced any issues after I fixed https://github.com/druid-io/druid/pull/4774/commits/5ef8fa5981003c45c2703cbeacc832a9eb08b847 but that was related to segments not being handed off.
There should be some more logging around why the segments can't be allocated.  Do you want to create a gist with the entire task log and share it here?

We do have that fix anyway, so it's not that. Here's the whole log: https://gist.github.com/KenjiTakahashi/602a3d29055a18855e3cb6e9ccc254ac

@kevinconaway I've found the culprit. It turned out that the failures where caused by some older data appearing, for segments which were first created with "stock" 0.10 Druid. And they had a different shardSpec config.
So a mistake on our side in the end, although it would be nice if Druid reported something more meaningful than NullPointerException :-).

although it would be nice if Druid reported something more meaningful than NullPointerException

For sure.  That said, I do believe that this should fail fast and throw an exception rather than silently ignoring or rejecting the data.  If Druid hadn't thrown an error, you might not have noticed the missing data until well later, making it difficult to detect what happened.
Thoughts?

Sorry for not taking a look yet @kevinconaway, we're in the middle of getting master ready for an 0.11.0 branch cut and should be able to take a look after that.

@gianm It looksl ike 0.11.0 is getting more stable.  Do you have time to take a look at this PR now?
We've been running it for a few weeks on our production load without issue.

@kevinconaway Sorry, I somehow missed your comment earlier. Yeah, failing fast is fine in this case, I think. It is ultimately a server misconfiguration and client alone cannot do much about it.
We've been now running this for a couple of days on two instances, no problems thus far.

@gianm is there a path to getting this reviewed and merged?

@kevinconaway, I'm sorry it took so long to look at this.
At any rate I have one main question first. I see you've worked on making the RealtimePlumber use a SegmentAllocator rather than come up with its own identifiers. As an alternate approach did you look at trying to make the RealtimeIndexTask use an AppenderatorDriver rather than a Plumber? The Appenderator system is a bit newer and I was hoping that it would eventually replace Plumbers. They should be just as capable. It should also reduce the need for refactoring anything outside of RealtimeIndexTask.
The main issue I was thinking of with that approach is that the on-disk directory structures aren't exactly the same, so that would affect the ability of tasks to restore. I think it should be ok if it's just one time, though. If we really felt strongly about preserving the ability to restore across the upgrade, then I guess the task could peek at its on-disk structure and migrate it if needed.

I see you've worked on making the RealtimePlumber use a SegmentAllocator rather than come up with its own identifiers.

Yep, the main reason here is to allow the RealtimeIndexTask to append data to existing intervals and allow for late arriving data, similar to how the kafka indexing service works

As an alternate approach did you look at trying to make the RealtimeIndexTask use an AppenderatorDriver rather than a Plumber? The Appenderator system is a bit newer and I was hoping that it would eventually replace Plumbers. They should be just as capable. It should also reduce the need for refactoring anything outside of RealtimeIndexTask.

I did look at that, specifically the AppenderatorPlumber.  I opted not to use it because it wasn't really being used in any of the existing code paths and since I'm not as familiar with the code base, I thought it best to use the code thats been battle-tested, so to speak

The main issue I was thinking of with that approach is that the on-disk directory structures aren't exactly the same, so that would affect the ability of tasks to restore. I think it should be ok if it's just one time, though. If we really felt strongly about preserving the ability to restore across the upgrade, then I guess the task could peek at its on-disk structure and migrate it if needed.

Are you saying that the on-disk directory structures between the RealtimePlumber and the AppenderatorDriver are different?
I don't have a strong opinion on whats best here in terms of preserving that so I'll defer to you on that.

I did look at that, specifically the AppenderatorPlumber. I opted not to use it because it wasn't really being used in any of the existing code paths and since I'm not as familiar with the code base, I thought it best to use the code thats been battle-tested, so to speak

That is right, the AppenderatorPlumber is not used anywhere -- it was meant to be a bridge from the (old) Plumber interface to the (newer) Appenderator layer but it never actually got linked anywhere. Mostly because I thought it would make more sense to port over the callers (like RealtimeIndexTask) to directly use an AppenderatorDriver. Does that look like it would work for you?
If so, I think that'd be the cleanest thing -- it would lead to eventually just removing Plumbers completely.

Are you saying that the on-disk directory structures between the RealtimePlumber and the AppenderatorDriver are different?

That is what I'm saying. They are similar but slightly different, enough that they aren't compatible. You can see it if you run both code paths side by side, or if you look at the differences in bootstrapping code in the respective startJob methods.

I don't have a strong opinion on whats best here in terms of preserving that so I'll defer to you on that.

If the task could do a migration of the older structure to the newer structure when starting up, I think that's best. It would just run one time but it would make people's lives easier when upgrading.

Mostly because I thought it would make more sense to port over the callers (like RealtimeIndexTask) to directly use an AppenderatorDriver. Does that look like it would work for you?

I think so.  It would be less code to refactor as it seems like the AppenderatorDriver implementation keeps changing.
I'll work on this in the coming weeks.
Does that timeline work?

I think so. It would be less code to refactor as it seems like the AppenderatorDriver implementation keeps changing.

Yeah, that is definitely an advantage. The internals are still evolving but the interface should be relatively more stable.

I'll work on this in the coming weeks.
Does that timeline work?

Sure, that sounds good to me!

Closing this out now that #5261 has been merged.
@KenjiTakahashi FYI


Java: add JAXB.unmarshall as XMLParser

JAXB.unmarshall is safe by default in the JDK since version 8, so need to flag it up.

There might still be some value in marking some calls of JAXB a potential sink if someone calls it with a badly configured XmlStreamReader. See: https://hackerone.com/reports/506791 (CVE-2019-5442)
https://github.com/pippo-java/pippo/blob/7da9f4db945d10113cf4ea4ed44ba0f1a7f83a8f/pippo-content-type-parent/pippo-jaxb/src/main/java/ro/pippo/jaxb/JaxbEngine.java#L78

Looks like it can either be flagged in XmlStreamReader or in JAXB.unmarshall. How does XmlStreamReader currently dealing with things like this? (i.e. What are the sinks for a badly configured XmlStreamReader? Or do we just flag user data going into badly configured XmlStreamReader?)

Ah, it looks like we treat calls to createXMLStreamReader as an XMLParser so we should flag the call to createXMLStreamReader as a sink.


KuduDevSetup.cmd doesn't fix up setProfileEnvironment

One manual step that it's missing is:

Open your %WINDIR%\System32\inetsrv\config\applicationHost.config and look for <applicationPoolDefaults>. Under <processModel>, make sure you don't have setProfileEnvironment="false". If you do, set it to true.

@shrimpy fixed this in ef95a2160b9c1c507ab97b94575ec9abace22f64


su does not work innixos-enter: Authentication service cannot retrieve authentication info

Describe the bug
su does not work while chrooting into any NixOS installation, even if it has only the bare minimum installed, it's clean, it's brand new etc.
Steps To Reproduce
Steps to reproduce the behavior:

nixos-enter
su

Expected behavior
I can run the su command normally. Any other distro that I've chrooted into I was able to switch to another user.
Screenshots
[nixos@nixos:~]$ sudo nixos-enter
setting up /etc...
bash: warning: setlocale: LC_NUMERIC: cannot change locale (ro_RO.UTF-8): No such file or directory
bash: warning: setlocale: LC_TIME: cannot change locale (ro_RO.UTF-8): No such file or directory

[root@nixos:/]# su
su: command not found

[root@nixos:/]# export PATH=/nix/var/nix/profiles/system/sw/sbin/:/nix/var/nix/profiles/system/sw/bin/:$PATH

[root@nixos:/]# su
su: Authentication service cannot retrieve authentication info
(Ignored)
Cannot execute /run/current-system/sw/bin/bash: No such file or directory

[root@nixos:/]#

Additional context
By the way, ignore I need to manually set $PATH, that’s another issue I found, but at least I worked around it.
Also, I want to do this because I want to run some commands post-installation to clone some Git repos in my $HOME and I also want to enable Home Manager for my user (I use the standalone installation). Basically my use case is use su to switch users to someone that isn't root.
I've posted this on other sites before: 1 2
Notify maintainers
nixos-enter doesn't appear on search.nixos.org. Taking maintainer from manpage: @edolstra
coreutils for chroot: @dasJ
su doesn't seem to have any maintainers.
Metadata
Basically what you'd get on the latest 23.11 image, because that's where I'm running this. I want to run it post-install, before first boot.
Please run nix-shell -p nix-info --run "nix-info -m" and paste the result.
[nixos@nixos:~]$ nix-shell -p nix-info --run "nix-info -m"
 - system: `"x86_64-linux"`
 - host os: `Linux 6.1.73, NixOS, 23.11 (Tapir), 23.11.3019.8bf65f17d807`
 - multi-user?: `yes`
 - sandbox: `yes`
 - version: `nix-env (Nix) 2.18.1`
 - channels(root): `"nixos-23.11.3019.8bf65f17d807"`
 - nixpkgs: `/nix/var/nix/profiles/per-user/root/channels/nixos`

Add a :+1: reaction to issues you find important.

Is there any known workaround for this? My whole nixos-system is broken since the last unstable-update (all generations are unable to boot) and I can't rebuild my system due to this bug.

and I can't rebuild my system due to this bug.

Does
export SUDO_USER=
nixos-rebuild boot

(yes, empty SUDO_USER)
do anything?

What does sudo have to do with it though? On my config, I have sudo disabled and doas enabled instead and it still happens. It's about su

What does sudo have to do with it though? On my config, I have sudo disabled and doas enabled instead and it still happens. It's about su

It doesn't have anything to do with sudo, I was replying to that specific comment ("and I can't rebuild my system due to this bug.").
I see the same error with sudo in a nixos-enter, and I also couldn't rebuild my system until I did export SUDO_USER=.

I just reinstalled my whole system with nixos-install --flake <bla>. Well, that worked at least...

What does sudo have to do with it though? On my config, I have sudo disabled and doas enabled instead and it still happens. It's about su

It doesn't have anything to do with sudo (except PAM), I was replying to that specific comment ("and I can't rebuild my system due to this bug.").
I see the same error with sudo in a nixos-enter, and I also couldn't rebuild my system until I did export SUDO_USER= since nixos-rebuild uses sudo as long as you have a non-empty SUDO_USER.

I had the same issue and adding export SUDO_USER= before my nixos-rebuild does workaround the PAM error I was seeing, and let me rebuild within a nixos-enter chroot successfully.


Ready to add to Surplus README?

I wanted to thank you for taking the time to make a Parcel plugin for surplus.  Is this at a state where you think it's ready to be linked from the Surplus README page?

Hi @adamhaile ! how are you? It's nothing, it was something that I really want to do to give a easy start developing with Surplus :+1: !
And yes, it's ready to be linked. I can do a PR if you want.


Please invite me to the Recode-Hive GitHub Community Organization

Name
Biradar Nikitha
Discord Username (if applicable)
No response
Additional Context
No response

Invitation sent to join the GitHub Organisation. Welcome to the community 🎉
Don't forget after accepting to make it public so it appears on your GitHub profile for everyone else to see. You can do this by finding your name in the GitHub organisation list and change the dropdown to public https://github.com/orgs/Recode-Hive/people


How to feed bytes to SequentialWaveReader not a string

What i would like to do is to read not from a file on local disk, but from a buffer. But i dont know how to do it the right way.
I've tried to use memory fs and monkey patch builtin open(), but all file opened using kaldi C code. Well, it looks like it can't be patched in a simple way. So i would like to know is there a right way to do it?

hello Vovkd even am interested to do the same did you find anything on it ?


Fast A record (un)packing

The generic struct pack and unpack routines make heavy use of reflection to correctly translate between bytes and Go objects.
Given most questions and answers are A records, writing a specific A record packer/unpacker makes the fast path quicker and generates significantly less garbage.
The pack and unpack routines for specific data types were extracted from the generic routine so that they may be shared with the optimised routines.

Please open a PR against miekg/dns library.


Create parinfer plugin for Clojure and ClojureScript

https://github.com/shaunlebron/parinfer#using-as-a-library

Oh yeah!

I almost jumped on this just a day or two after it was announced. However I managed to hold back. It would be great if someone else from the community stepped up. I'd be happy to help out anyone willing to give it a go :-)

Bad news is a LightTable plugin can't rely on the published library because it uses .cljc and we are running such an old version of Clojurescript - #1973. Good news is that 0.1.0 doesn't use any reader conditionals so we could just use a forked version that renames .cljc to .clj/cljs. We could suggest upstream that they use .clj but I'm assuming they will eventually want to take advantage of .cljc and don't want to be held back by catering to ancient versions of Clojurescript

Let me know if I can help out with this.  I've written more step-by-step docs on how to get this working:
https://github.com/shaunlebron/parinfer/tree/master/lib

@shaunlebron I think any help would be greatly appreciated. But I think we're going to need to fork your code because we're still stuck on a (very) old version of ClojureScript.

I've started a simple plugin in LightTable, based on https://github.com/LightTable/Declassifier.
It's hosted on https://github.com/mauricioszabo/lt_parinfer. The plugin is usable right now, it uses the node.js' version of parinfer. Some simple problems that propably will appear:

The plugin re-evaluates the whole text, so it might be slow with huge files
I didn't find any good documentation on how to work with plugins, so for now we're not able to deactivate the plugin
It just works in "indent-mode", there's no "paren-mode" yet
When installing it, it gives the message "Some plugin dependencies are missing". I wasn't able to discover what's wrong, or even if it's a bug in my plugin or in LightTable

The good news is that it works, so maybe some pull requests will be able to remove these problems above.
A little demo of me using it:

Great work @mauricioszabo!
For some background, parinfer was rewritten in JS, so this makes the cljs-compatibility blocker go away.
To address the points made:

You may notice slowness (50ms delay) at around 5k-6k lines.  Anything lower than 3k will be fine.
(idk)
Cool, this is fine early on, but it's critical to run Paren Mode on a file before it's opened.  This prevents Indent Mode from corrupting your file.
(idk)

I'm guessing caching per-line paren state the way codemirror does for
syntax highlighting is also a non-beta function? :P
On Sunday, 14 February 2016, Shaun LeBron<EMAIL_ADDRESS>wrote:

Great work @mauricioszabo https://github.com/mauricioszabo!
For some background, parinfer was rewritten in JS, so this makes the
cljs-compatibility blocker go away.
To address the points made:

You may notice slowness (50ms delay) at around 5k-6k lines.
Anything lower than 3k will be fine.
(idk)
Cool, this is fine early on, but it's critical to run Paren Mode on
a file before it's opened. This prevents Indent Mode from corrupting your
file.
(idk)

—
Reply to this email directly or view it on GitHub
https://github.com/LightTable/LightTable/issues/2049#issuecomment-183913099
.

I'm closing this as there now is a plugin.

👍


Add driver for hx711 weight scale ADC

Patch adds driver to support hx711 adc designed for weigh scales. Example initialization is added in stm32f411-minimum board.
Zero impact on other code.

Please fix this warning/error:
/home/runner/work/nuttx/nuttx/Documentation/components/drivers/special/hx711.rst:document isn't included in any toctree

Fixed that, and moved that driver from special/ to character/ since it's just a better match.

@mlyszczek thank you! Actually I think you could move it from analog/ to sensors/ because it is more a sensor than an analog device

No, I think you are right, the datasheet said it is a sensor: https://cdn.sparkfun.com/datasheets/Sensors/ForceFlex/hx711_english.pdf

@mlyszczek thank you! Actually I think you could move it from analog/ to sensors/ because it is more a sensor than an analog device

Hmm... I think it may be closer to analog? After all, hx711 is just an ADC with some specialty. On the other hand, because of that specialty it makes it closer to sensor? But in theory it also could be used as an ADC - like connecting tensometer to sim racing pedal, and read pedal position that way. Driver can indeed work with that as well.

@xiaoxiang781216 any idea of this error:
make: *** [tools/Unix.mk:689: olddefconfig] Error 1
  Enabling CONFIG_ARMV7M_TOOLCHAIN_GNU_EABI
make: *** [tools/Unix.mk:689: olddefconfig] Error 1
  Building NuttX...
    Checking NuttX by Codechecker...
/github/workspace/sources/nuttx/tools/testbuild.sh: line 252: CodeChecker: command not found
    Storing analysis result to CodeChecker...
      Generating HTML report...
/github/workspace/sources/nuttx/tools/testbuild.sh: line 256: CodeChecker: command not found
      Compressing logs...
/github/workspace/sources/nuttx/tools/testbuild.sh: line 370: /github/workspace/sources/nuttx/../nuttx/nuttx.manifest: No such file or directory
  Normalize sama5d4-ek/nxwm
make: *** [tools/Unix.mk:689: olddefconfig] Error 1
make: *** [tools/Unix.mk:712: savedefconfig] Error 1
make: *** [tools/Unix.mk:689: olddefconfig] Error 1

@xiaoxiang781216 I think we need to think more about this CI and how to improve it. Espressif implemented internally a CI based on Robot CI (AFAIK). Maybe @tmedicci could give more details.

I saw that the broken of codechecker is more often than other, @W-Mai, could you disable codechecker ci? It's fine since codechecker report many false alarm and ci never enforce it.

@mlyszczek there seems to be a problem with this PR...
boards/arm/stm32/stm32f411-minimum/Kconfig contains:
if STM32F411MINIMUM_GPIO
source boards/arm/stm32/stm32f411-minimum/Kconfig.gpio
endif

But that Kconfig.gpio file doesn't exist, so all builds break with:
boards/arm/stm32/stm32f411-minimum/Kconfig:83: can't open file "boards/arm/stm32/stm32f411-minimum/Kconfig.gpio"
make: *** [tools/Unix.mk:689: olddefconfig] Error 1
ERROR: failed to refresh

At least for me...

@hartmannathan yes, it's a leak from another feature I'm working on for black pill, sorry :( I'll make a quick patch now.


trafficlight.py throwing "AWSIoTPythonSDK.core.protocol.mqtt_core - ERROR - Offline request queue has been disabled"

Hi,
i am going through GG getting started guide, however, i am stuck at module 5, 3b. (https://docs.aws.amazon.com/greengrass/latest/developerguide/comms-disabled.html)
seemed like a common problem as someone posted the same in github without any solution
https://github.com/aws/aws-iot-device-sdk-python/issues/141
as module 6 and 7 have dependencies on module 5, i want to complete it successfully. can someone help?
here's the error that i am seeing when i execute the py script. tried it on both Mac and Windows. seeing the same output.
2019-08-11 13:34:01,298 - AWSIoTPythonSDK.core.protocol.mqtt_core - INFO - Performing sync subscribe...
2019-08-11 13:34:01,298 - AWSIoTPythonSDK.core.protocol.mqtt_core - INFO - Offline request detected!
2019-08-11 13:34:01,298 - AWSIoTPythonSDK.core.protocol.mqtt_core - ERROR - Offline request queue has been disabled
Traceback (most recent call last):
File "trafficLight.py", line 232, in
deviceShadowHandler.shadowRegisterDeltaCallback(customShadowCallback_Delta)
File "/Library/Python/2.7/site-packages/AWSIoTPythonSDK/core/shadow/deviceShadow.py", line 398, in shadowRegisterDeltaCallback
self._shadowManagerHandler.basicShadowSubscribe(self._shadowName, "delta", self.generalCallback)
File "/Library/Python/2.7/site-packages/AWSIoTPythonSDK/core/shadow/shadowManager.py", line 68, in basicShadowSubscribe
self._mqttCoreHandler.subscribe(currentShadowAction.getTopicDelta(), 0, srcCallback)
File "/Library/Python/2.7/site-packages/AWSIoTPythonSDK/core/protocol/mqtt_core.py", line 299, in subscribe
self._handle_offline_request(RequestTypes.SUBSCRIBE, (topic, qos, message_callback))
File "/Library/Python/2.7/site-packages/AWSIoTPythonSDK/core/protocol/mqtt_core.py", line 370, in _handle_offline_request
raise self._offline_request_queue_disabled_exceptions[type]
AWSIoTPythonSDK.exception.AWSIoTExceptions.subscribeQueueDisabledException

Hey @towwei,
Have you figured out the solution as I m also stuck in the same issue?


Collect logs on failure

This GitHub action is really helpful to me. I have a suggestion for a Quality-of-Life feature, that could really be helpful for debugging.
I often have to deal with failing builds of dependencies due to various reasons. When a dependency fails to build, vcpkg creates multiple log files somewhere in the buildtrees folder. I have to manually cat these in a separate step.
It would be nice, if this action adds a step which automatically collects all (error) logs and makes them available to the user somehow. I'm not too familiar with GitHub actions, but if you cannot upload the logs, you may simply cat them?
I'm not sure to always include this additional step, but constraint it to failing builds, or including it only when debug logging is active (i.e. when manually re-running the job).
My current, manual approach is to add the following step immediately after the action:
      - name: cat logs
        if: failure()
        run: cat /path/to/project/vcpkg/buildtrees/dependency/build-x64-linux-err.log

Hi @jdoubleu
I find it is better to capture the logs using upload-artifact rather than trying to dump them to the console and decipher them.
Something like this should work, although I haven't tested this exact example:
- uses: actions/upload-artifact@v4
  with:
    if: always()
    name: vcpkg-logs
    path: vcpkg/**/*.log

Okay, thanks. I think this would work well.
How do you feel about adding an option (e.g. collect_logs: 'on-failure' // 'always', 'on-success', 'on-failure'), which automatically adds such a step?


Add Windows, macOS, and Stack Support (#1)

This commit represents the work done on haskell-CI/setup, specifically the add-windows-macos branch. It closes several issues and pull requests. In doing so, it is effectively a full rewrite of the repo (although certain pains have been taken to keep the git history as sane as possible)

All of the typescript setup is remodeled after the latest template of the typescript action. This has a number of advantages, (like not keeping node_modules in git).
Full windows support.
Full macOS support.
Full support for installing Stack.
Can install all versions of GHC, Cabal, and Stack.
GitHub Actions CI now used to perform e2e tests as well as standard unit tests.
Fully updated documentation. All examples should be fully copy-and-pasteable working.
Code goes through pains to avoid explicitly sheling out and attempts to stay entirely inside node when possible for cross-platform compatibility and reliability.

Linux will check for existing installs of GHC/Cabal and use those when possible. Thus, this offers the benefit of having zero regression in functionality and CPU cycles over the current state while being a strict addition of functionality.
Additional configuration inputs have been added based on my experiences writing CI systems for Galois and common annoyances people have in "real world projects" that make use of Cabal, Nix, Stack, or multiple combinations thereof. All inputs are fully documented and tested.
I will also be following up with GitHub to see if we can get common versions of GHC/Cabal/Stack baked into the environments for all operating systems. I have some suggestions that should save a decent amount of space in their images.
Lastly, I am willing and able to step up to help maintain and improve this project if you so wish. One thing I am very interested in is getting this action on the marketplace.

Fixes #1
Fixes #7
Fixes #10
Fixes  #11

@tclem @ethomson, anything I can do to assist in the process of getting this in?

@jared-w thanks for all the great work here! Let me track down someone working directly on actions every day to help review and answer your other questions about being a maintainer and getting this action in the marketplace (I'm a contributor here just out of needing a haskell build environment 😄).

I will also be following up with GitHub to see if we can get common versions of GHC/Cabal/Stack baked into the environments for all operating systems.

If you haven't found it already, that will happen in https://github.com/actions/virtual-environments

@jared-w, when you get a chance will you merge latest master into your branch here (and fix workflow.yml conflicts)? I enabled running this repo's tests on pull requests.

I am looking to use this to get newer versions of ghc on my project to test a bug I may have found in ghc (https://gitlab.haskell.org/ghc/ghc/issues/18038).  Am I correct in reading the .md file that even after this is merged, there will still be no support for 8.10.1 nor for HEAD?

@JasonGross I think 8.10 should work, since it's installed on the runners and present in the hvr package. GHC HEAD is a trickier proposition, since we'd need to figure out how to build and distribute it and keep it updated.

I've merged in the updates I've been working on for a bit to the haskell-CI/setup master branch. At this point I think I've finished implementing the functionality in a way that'll make it much easier to add support for recognizing pre-installed binaries on macOS and Windows (for when actions/virtual-environments#645 is implemented.
The setup should now be much more ergonomic by default.
GHC 8.10 should indeed work.
Support for HEAD will be trickier as the underlying tools don't always have the best support for it. (In fact, chocolatey, ghcup, and the PPA don't have any support for installing HEAD versions of haskell currently). I don't really know how I could ergonomically add support for that but I'm open to suggestions.

Support for HEAD will be trickier as the underlying tools don't always have the best support for it. (In fact, chocolatey, ghcup, and the PPA don't have any support for installing HEAD versions of haskell currently). I don't really know how I could ergonomically add support for that but I'm open to suggestions.

I don’t think we need to worry about HEAD. Given the difficulty of even installing it, I feel justified in saying that we can pass on that complexity to the people who absolutely need it (and I’m not sure how many people who need GHC HEAD are gonna use GH Actions anyway).

I don’t think we need to worry about HEAD.

This is my thinking as well. The infrastructure around being able to obtain a nightly version of the compiler isn't super fleshed out at the moment as Haskell is catching up to the tooling game.
That being said, I'm not opposed to revisiting that should the windows situation change, but such a change would likely require that ghcup and chocolatey have support for downloading nightlies.
So, as it stands, it looks like this branch is essentially feature complete. Are there any remaining blockers to getting it in? The latest status of actions/virtual-environments#645 looks like it's going to be a while before that gets merged in. Is this completely dependent on that?

This is looking good. I've done a local run through and am now going to test out on one of our haskell projects that was already using the setup-haskell action.
@jared-w, I think we should rev the version here to v1.1 (would be OK to do in a follow up PR) and make sure the README and other documentation reflects that. Also, what do you think about adding a list of contributors to the README?

Build looks good on https://github.com/github/semantic/pull/545/checks?check_run_id=626680941.
@jared-w, I'm going to merge this. Will you open a new PR to do a version rev? I'm happy to cut the release on github if you'll help me craft some release notes.

@tclem absolutely. I'll open the PR and draft some release notes as well. Thanks for all your help!


Woff2 Support

Added in Woff2 support

Seems legit, I'll merge the update, thanks!  Be aware that we are looking for new maintainers & I am not planning on releasing new gem version at this time (see #197).