Include the certified Open Liberty Operator

The Open Liberty Operator has been re-launched as a native Go operator, inheriting all of the functionality from the Appsody Operator + adding day-2 operations.  The high level design can be seen here.
This issue tracks the subscription / installation of the Open Liberty Operator by the Kabanero Operator.
A simple approach is to always install the Open Liberty Operator, in addition to the Appsody Operator.
A more advanced approach would be for the Kabanero Operator to only install the Open Liberty Operator if a stack has been configured to use that Operator.
Please note that there are no collisions between the Open Liberty Operator and the Appsody Operator.  They listen for different CRs (OpenLibertyApplication vs AppsodyApplication), so they will each work with different deployments.

We have typically taken the following approach
If the APIs that are provided are mature (non-beta/alpha), and the Operator exists in an RH or Certified catalog, then we create a Subscription to that Operator via the RH or Certified catalog. We trust that automatic patch releases into the channel will not break us.
If the APIs that are provided are beta/alpha, we capture the CSV at a tested level and put them in the Kabanero catalog. This ensure that the version does not get automatically upgraded and break us.
Sometimes a channel is just stable/canary
Sometimes a channel is a Minor release stream
OLO is v1beta1
Kabanero Operator will not manage Subscriptions. Bad Idea.
Kabanero Operator will not set required APIs in its CSV, due to the current unpredictable nature of what OLM will decide to install.
Even RHOS documentation instructs users to install Operators one-by-one.
Is OLO going to be in the Certified catalog? What Subscription channels will OLO maintain?

hey @dacleyra - the Open Liberty Operator is already in the certified catalog.  It's using the beta channel.   We're planning on having a 1.x channel once we release 1.0.0 - we're currently iterating over 0.x at the moment, similar to Kabanero.

@dacleyra lets copy it to our catalog.  with a beta channel we can't stop old versions of Kabanero from picking up 0.4 or 0.5 etc.

This is fixed in #485


Aspect fits split

fix bulk aspect data product writes out only one packet to fits

Ah I don't under stand where all these commit are coming from there should only be one ed389f7 ?


Implement and test clj:conj

This version of it works as discussed in #268, but if there are any stylistic changes to be made, too few tests, or other problems, let me know and I'll get on it.
As for the failing tests on R16, I'm trying to work out how to gracefully degrade on pre-17 releases or really anywhere that doesn't support maps.

If you change the definition of conj to use map? then it will be usable even for R16. Now it just complains that unbound function: #(is_map 1). Then perhaps just removing the map case for conj in clj-test.lfe will make it succeed in travis. This is not a good solution but the easiest.
And rebase, then I can pull it.

Thank you. I was kinda stuck on making it work for r16. But I'll fix that and rebase asap.

Okay, it's now updated and rebased.

I think you need to use maps:from_list/1 or similar to make it work, instead of the literal constructor. There should be a working example or two in clj-tests.lfe.

Cool. It works when using maps:from_list for it in conjunction with IFF-MAPS. I think it's ready now, with tests and everything. I guess I should remove the "remove tests" commit though.

This looks great, by the way, I'm just nitpicking. I think this is ready to merge whenever. Thanks for the patch.

The nitpicking is helpful. I'm learning all kinds of things about LFE from it, so feel free :smile:

Cool. This works for me as-is then.


Make DataView::GetToolTip return DataView::GetValue by default

There are plenty of cases where the content of a cell of a DataView doesn't
fit in the width of the column. In some cases, even expanding the column is not
enough to reveal the full content, as the DataView itself is not wide enough.
We enable tooltips everywhere by making it show the content of the cell by
default.
Test: Take a capture, verify tooltips in all DataViews.

I'm assuming this is not controversial. Let's discuss if you think it is, in which case we need to enable tooltips for the columns in the DataViews that are most affected.

Happy to see this change. Code wise looks all good to me.
But I would like to discuss the special cases. The PresetsDataView shows the same tooltip for all columns. Sometimes the "Date Modified" column does not have enough space but the tooltip is still only related to the preset filepath and load state.
Shall we consider show GetValue as the tooltip for the "Data Modified" column instead?

But I would like to discuss the special cases. The PresetsDataView shows the same tooltip for all columns. Sometimes the "Date Modified" column does not have enough space but the tooltip is still only related to the preset filepath and load state. Shall we consider show GetValue as the tooltip for the "Data Modified" column instead?

Hehe, you caught exactly the case that I was unsure how to handle and hence I didn't touch. The reason why I was unsure is that the "Modules" and Hooked Functions" columns make sense when seen together, because they have multiple rows with a 1-1 relation. However, the other columns can be tackled independently, and these two can be improved. I'll do that and come back to you.

@vickyliu-go4it So, I did a little rework of PresetsDataView::GetToolTip. The "Modules" and "Hooked Functions" columns show the same tooltip that merges the information of the two, I think it looks decent. The first column's tooltip shows "(loaded)" instead of "*" to indicate that the preset has been loaded. I left the "None of the modules in the preset can be loaded." string confined to the "Preset" column as it felt a bit too much to add it to the tooltip of every column now that they show different content.
Let me know what you think.


New Tractus-X project contributor

Before requesting the project contributor role, please make sure, that you:

Make yourself familiar with the Eclipse Foundation project roles: https://eclipse-tractusx.github.io/docs/oss/contributor-committer
Make ourselves familiar with prerequisites, like a signed ECA: https://eclipse-tractusx.github.io/docs/oss/getting-started
Check the committer list of our project and highlight at least one of them on this issue, since every committer
can add project contributors: https://projects.eclipse.org/projects/automotive.tractusx/who

Necessary Information
Email address to add<EMAIL_ADDRESS>Committers, that could take over: @evegufy, @tunacicek
Please make sure, that you accept the invitation you receive from GitHub, after you have been added as project contributor.

Hi @mgarciaLKS I invited you to the GitHub Org, you should get a notification soon


Dependency conflicts while installing via Command Prompt

Python version: 3.10.10
Strawberry Fields version: 0.23.0
Operating System: Windows 11 Home System Language 22H2
Issue: It is throwing an error while installing the package through "pip install strawberryfields --upgrade" command in the command prompt.

Hi @mvsg2, thank you for opening this issue. Could you please copy-paste your full error traceback? If you can paste it as text instead of an image we might be able to get additional details. Thank you!
In the meantime, you can use Strawberry Fields on Xanadu Cloud. We have hosted Jupyter notebooks with Strawberry Fields pre-installed.


Switch from pi-hole to CoreDNS

Since CoreDNS is already used in k3s cluster, it might be a good idea to replace pi-hole with CoreDNS.
Pros:

one server type to learn
native prometheus metrics exposition (instead of faulty eko/pihole-exporter) with proper metrics :)
DNS-over-HTTP support
full GitOps management
stateless and easily scalable

Cons:

No webUI to quickly whitelist sites
ads plugin is not included natively in CoreDNS

Happened in https://github.com/thaum-xyz/ankhmorpork/commit/c3306a329bdd7e247dcfcfd8042b87747813affc and  https://github.com/thaum-xyz/ankhmorpork/commit/87365b6d3195007d08f5e040f6bb18397312d997


Fix error with formatWebpackMessages trimming warnings

Hey, I've been using the updated react-dev-utils and found a bug in formatWebpackMessages that happens with newer webpack versions.
Basically webpack (or the thread-loader) in the stats.toJson().warnings now outputs also this line, containing a Module Warning or Module Error (?):
Module Warning (from ./node_modules/thread-loader/dist/cjs.js):

so the lines array now becomes this (see second line):

instead of the old one

This fucks with the thread-loader message remover and actually removes the body of the warning.
This PR checks if that line is present and removes it since it's not relevant to the user.

This seems like a good fix. I'd like to get the test passing before merging it though. We're currently working on fixing the tests.

Do you guys think we should check also for /thread.loader/i in that line just to be extra sure?

Done it, I think it makes sense if seomhow the Module Error message changes

Another case with the thread loader happened, this one happens when there are issues in the build, not the user code. Here is the lines array of when this happens

The error message was clipped also in this case, fixed it in fdcdd9a

This is fixed in next, thanks!


fix: write response header should not break req

Although it won't take effect without calling Write or WriteHeader
for now.
Fix #64.

Codecov Report

Merging #65 (8037394) into master (195bd64) will not change coverage.
The diff coverage is 100.00%.

@@           Coverage Diff           @@
##           master      #65   +/-   ##
=======================================
  Coverage   78.88%   78.88%
=======================================
  Files          11       11
  Lines         701      701
=======================================
  Hits          553      553
  Misses        111      111
  Partials       37       37

Impacted Files
Coverage Δ

internal/http/response.go
100.00% <100.00%> (ø)

internal/plugin/conf.go
89.04% <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 195bd64...8037394. Read the comment docs.


Event attendees don't get email notification

I am creating test events with --who<EMAIL_ADDRESS>The user does see an invite from within the calendar app, but strangely enough never gets an email notification that they've been invited to an event. Is this the intended/expected behavior? Whenever a calendar item is created in Google Calendar web interface with an attendee, that person is notified by email. Is there any way to trigger the notification from the CLI?

Figured it out. Patch below that causes email invites to be sent; perhaps this should be added as a command-line switch?
`--- gcal.py.bak 2021-07-01 12:50:04.814334100 -0400
+++ gcal.py     2021-07-01 12:59:23.631646400 -0400
@@ -1371,7 +1371,7 @@
     event = self._add_reminders(event, reminders)
     events = self.get_cal_service().events()

   request = events.insert(calendarId=self.cals[0]['id'], body=event)

   request = events.insert(calendarId=self.cals[0]['id'], body=event,sendUpdates="all")
   new_event = self._retry_with_backoff(request)

   if self.details.get('url'):

@@ -1606,7 +1606,8 @@
.events()
.insert(
calendarId=self.cals[0]['id'],

                                       body=event

                                       body=event,

                                       sendUpdates="all"
                                   )
                           )
               hlink = new_event.get('htmlLink')

@@ -1625,7 +1626,8 @@
.events()
.insert(
calendarId=self.cals[0]['id'],

                                       body=event

                                       body=event,

                                       sendUpdates="all"
                                   )
                           )
               hlink = new_event.get('htmlLink')`

Worked for me, cloned the source and made your edit and emails send now. Tyvm!

@dbarnett are you open to incorporating this into the tool? I can play around with argparse to make an option that enables sendUpdates and submit it as a P.R. if you're open to it (and not planning to do something along the lines already).

Sure, if you can send a PR I can review.
Do you know if there's any downside or what the other sendUpdates options do?

I've been using it for a while, when I remember to patch gcalcli at least, and it seems to work fine. I imagine most people actually expect a calendar invite to actually send an invite to the invitees as a default option. So far I haven't discovered any downside, particularly if it's an end-user option whether to send invites or not.
I haven't tested it out but I assume the sendUpdates options are either to send a notification to everyone on the invite (all) and then there must be some other option that only sends notifications to people who are added or dropped from a previously existing invite, although I don't know how that part would actually intersect with how gcalcli is modifying (?) existing invites.


More git diff and git log in on-demand

Overview
TL;DR
We should add more git log and git diff to our courses.
Extra Detail
@bas brought up a great point while we were chatting in Amsterdam, that the courses now tell users to type git log but the users aren't given much explanation on what they should look for. It would be helpful to have more git log and git diff within the courses, along with explanations of what to expect and what it means.
@github/services-training what do you think?

Closing this out since our on demand courses are now deprecated.


azure-mgmt-datamigration: LICENSE missing in PyPi

Package Name: azure-mgmt-datamigration
Package Version: 9.0.0
Operating System: Linux
Python Version: 3.9.6

Describe the bug
The LICENSE.txt file is missing from the azure-mgmt-datamigration package in PyPi.
Additional context
I am making a PR to fix this now.

@major, thanks for the feedback and future contribution (again)! Directing it to the right people, so they know!

same issue : https://github.com/Azure/azure-sdk-for-python/issues/20183


SNOW-1733956: fix sql count of modin tests with cte enabled

Which Jira issue is this PR addressing? Make sure that there is an accompanying issue to your PR.
<!---
In this section, please add a Snowflake Jira issue number.
<p>Note that if a corresponding GitHub issue exists, you should still include
the Snowflake Jira issue number. For example, for GitHub issue
https://github.com/snowflakedb/snowpark-python/issues/1400, you should
add "SNOW-1335071" here.
--->
Fixes SNOW-NNNNNNN

Fill out the following pre-review checklist:

[ ] I am adding a new automated test(s) to verify correctness of my new code

[ ] If this test skips Local Testing mode, I'm requesting review from @snowflakedb/local-testing

[ ] I am adding new logging messages
[ ] I am adding a new telemetry message
[ ] I am adding new credentials
[ ] I am adding a new dependency
[ ] If this is a new feature/behavior, I'm adding the Local Testing parity changes.
[ ] I acknowledge that I have ensured my changes to be thread-safe. Follow the link for more information: Thread-safe Developer Guidelines

Please describe how your code solves the related issue.
Please write a short description of how your code change solves the related issue.

if this change also affects daily precommit, can you post a successful run for those as well

yes, https://github.com/snowflakedb/snowpark-python/actions/runs/11485902072, though some tests seem failing with an unrelated error:
[31mFAILED tests/integ/modin/io/test_read_csv.py::test_read_csv_usecols_nonexistent_negative[usecols1] - snowflake.connector.errors.ProgrammingError: 000612 (02000): Result not found


Use .continueignore to exclude from auto complete

Validations

[X] I believe this is a way to improve. I'll try to join the Continue Discord for questions
[X] I'm not able to find an open issue that requests the same enhancement

Problem
To exclude files from auto complete I have to add disableInFiles
Solution
It would be better I exclude/hide files from auto complete using the .continueignore file so there is one file to exclude from auto complete and indexing

@salah-8 I had the same issue a few days ago but on second though, I believe these should be separate. The files you want to ignore may not always overlap. For example, you may want to disable completions in all Markdown files but still want to have them indexed.

@tomasz-stefaniak Hey, I understand your point. But it would be better to at least make it a file like .continueignore instead of adding a long list in the config.json file

Out of curiosity, what's your use case and what types of files would you like to ignore?

most of the files I want to ignore/hide from AI in general (indexing or auto completion) is the files that have secrets, keys, or anything sensitive stuff from my company's codebase in general

@salah-8 we're currently working on a new version on config.json. I'm tagging @sestinj so that he's aware of this feedback.


Split exec and target toolchain definitions

Fixes #17.

Sweet thanks! I'll start getting some miles on this at clients and if there are actual problems I'll come back with a better repro.


Deploy with tag number from git

I am sorry to write this here but I have tried to search and read but if I am not missing anything I can not run the playbok with a parameter etc relling anistrano to deploy tagxxx from git?
I have another tool where I do this
php mgt.php deploy production tag 1.0.3
The abowe deploy tag 1.0.3 to the production enviroment. It checkout from git and deploy to the servers. Can I acheve the same with anistrano?

you can do it already, as it is stated in the docs
ansistrano_git_branch: master # What version of the repository to check out. This can be the full 40-character SHA-1 hash, the literal string HEAD, a branch name, or a tag name

The name is a bit confusing, it should be sth like ansistrano_git_revision or sth similar but we kept it as it was at the beginning for BC compatibility


[WIP] conform to context interface of <MathJax.Context>

This is just like the soon-to-be-released @nteract/markdown's <MathJax.Context> except it loads using mathjax-electron. I'm prepping this PR for handling when I make the release of the packages from the nteract monorepo.
All of a sudden, we're going to have super improved LaTeX handling likely closing quite a few issues. 🎉

~In hindsight I should make this part of the monorepo too, as nteract desktop could also use this.~ Done.
TODO:

[ ] Make mathjax-electron, @nteract/mathjax dependencies for Hydrogen
[ ] Switch to @nteract/mathjax for setting up a <MathJax.Context> root element above any elements that use display area or transforms.

Tips I need -- what top level element(s) should I register this mathjax context on? It basically needs to be above any element that would render outputs (Markdown or LaTeX).

This will need to be upgraded in tandem with #1213, #1214 and use the new way I allowed <MathJax.Context /> to override how it "loads" MathJax in https://github.com/nteract/nteract/pull/2480
import { loadMathJax } from "mathjax-electron"

...

<MathJax.Context
  loader={(onLoad) => loadMathJax(document, onLoad)}
>
  {someHydrogenElementThatUsesDisplay}
</MathJax>

Woa I ignored this way too long, sorry Kyle. I'll take a look later today.

Tips I need -- what top level element(s) should I register this mathjax context on? It basically needs to be above any element that would render outputs (Markdown or LaTeX).

Right now we're registering mathjax globally on the page. This seams to have worked in the past.
mathjax-electron was handling this for us: https://github.com/nteract/mathjax-electron/blob/master/index.js#L31

That's fine. The MathJax context component can use a global version of MathJax that's already loaded -- all it does it make MathJax available on the context api to other components.


Get all problems (errors, warnings, etc.) using VSCode extensions API - 2

I am sorry, I am basically duplicating this issue: https://github.com/Microsoft/vscode/issues/30075 because I didn't understand, what is the outcome of that issue?
How can I get all problems in the workspace? I couldn't find anything related in the proposed api: https://raw.githubusercontent.com/microsoft/vscode/master/src/vs/vscode.proposed.d.ts (is the link correct?).

const diagnostics = vscode.languages.getDiagnostics();

A big Thank You, @usernamehw! Now I can see that in the API :smiley: :man_facepalming:


proposal: support pod customizing numa policy

Ⅰ. Describe what this PR does

Introduce new api for pod level numa topology policy. With the new api, users can specify numa topology policy for each
pod, so that pods that are more sensitive to latency can decide how they need to be orchestrated, rather than being passively
scheduled according to the numa topology policy on the node.
Ⅱ. Does this pull request fix one issue?

Ⅲ. Describe how to verify it
Ⅳ. Special notes for reviews
V. Checklist

[ ] I have written necessary docs and comments
[ ] I have added necessary unit tests and integration tests
[ ] All checks passed in make test

BTW, please fix DCO by git commit -s.

/lgtm

/lgtm

@hormes Hi, is there any questions for this proposal? If not we can start to implement the new feature. Please take a look if you have time, thanks.

@hormes Hi, is there any questions for this proposal? If not we can start to implement the new feature. Please take a look if you have time, thanks.

LGTM.
/approve


New Retweet Format?

This isn't necessarily a problem with STTwitter, but do you know how to post as the new quoted retweet format? Either I keep on missing it in the documentation or it's not there...

Without looking, I guess you just have to post the Tweet URL.
Can you try and let me know?

Yes that seems to be the case. Thanks! It might be good to let people know somewhere. I’ve seen some confusion around.

Though upon further testing it seems to only come through the streaming API, not the regular APIs?

There's more on this on our Twitter Community forums - quoted_status will be added in the coming weeks.https://twittercommunity.com/t/api-payloads-to-include-original-quoted-tweet-objects/38184/7

They’re being fully hydrated now. It just took a while. Ill close this issue now that it’s no longer an issue


Netflix ok, hulu not ok

I have setup the server correctly on native ipv6 and can view netflix without any issues but still face a proxy error for hulu. I was told that there are a few specific ip belonging to google that can cause this and one of them is google dns, <IP_ADDRESS>/<IP_ADDRESS>. I noticed that google dns is being used as the recursive dns. Is it possible to install unbound outside of docker (in the vps main service) and use that as the recursive dns instead of google? Which file do I need to change if I want to use unbound as my recursive dns?
Thank you

It doesn't matter which DNS server you use, if your provider's IP is blacklisted by Hulu (most are), then you won't be able to watch it.
You need to find a provider that they haven't blacklisted yet and deploy on to that. No IPv6 is required, because Hulu doesn't use it.


Out-of-the-box Auth applies middleware twice

I upgraded from 5.2.27 to 5.2.28 less than an hour ago. I noticed that all of my authentication routes have double middleware:
Login: middleware = web, guest, guest etc
My own routes are fine.
Any help?

I tried with a new install a laravel: do a php artisan make:auth, then do a php artisan route:list and voila.  double auth and guest middlewares:

Upload your routes.php.

This happens if you explicitly include the web middleware in your routes.php. The web middelware is autmatically included for these routes (https://laravel.com/docs/5.2/middleware#middleware-groups).
The issue has been discussed in https://github.com/laravel/framework/issues/12022 as well. As stated in that issue some seconds ago, the quickstart examples need to be updated since they contain the double loading as well.

I got it.
https://github.com/laravel/laravel/commit/5c30c98db96459b4cc878d085490e4677b0b67ed


GPC exclusion list doesn't have a formally documented update and maintenance process

GPC exclusion list entries should include information as to why the exclusions were added, and these entries should be reviewed on a recurring basis to see if they are still needed.
Additionally, Verizon would like to be removed from this list. Whatever they were using in the past that caused issues with GPC is probably not present anymore, and this exclusion functionality has prevented Transcend Consent Management (used on Verizon sites) from receiving GPC signals for this extension's users.

Hi @eligrey, we do include information for all privacy protection exclusions that we ship to resolve site breakage, GPC included. You can find this info in https://github.com/duckduckgo/privacy-configuration.
We also do review GPC exclusions on a regular basis and remove them when the underlying issue has been resolved. Whenever we reproduce a reported site breakage caused by the GPC signal, the first thing we do is contact the site owner to inform them of the issue and ask for resolution. If we receive a response and the site owner confirms that they have fixed the issue, we remove the exclusion immediately. Otherwise we check back every few months to see if the issue has been resolved, and if so we remove the exclusion.
In Verizon's case, we added a GPC exclusion because they weren't loading reCAPTCHAs in a few places on their website when the GPC signal was present, which was preventing users from submitting forms. This exclusion was removed earlier this week during a routine review after we confirmed the reCAPTCHA issue had been resolved, so you should be seeing GPC signals from our users on verizon.com now.

Thanks for the update. I'll let Verizon know that this is fixed and should be reaching DDG PE users shortly


Cannot edit mapping on a resource

This is an issue reported to on Helpdesk:

I just modified the Darwin Core Mapping of one of our resources which worked as expected.
Unfortunately, I cannot reopen the mapping now. As soon as I click the EDIT Button, I am redirected to the resource.do page instead of the mapping.do page.

They have two resources on their IPT (https://www.herbarien.uzh.ch/ipt/manage/), the problem concerns only one of them.

Thanks Marie for reporting this. We need their IPT logs to figure out what happened and fix it

It happens when the database source changes (add new columns/remove columns). IPT (probably) silently swallows error somewhere and just redirects back to the resource page.


Error when trying to run

It says it can't find bass.dll, I have the Plants vs. Zombies game files, where do I put them?

Hello, that is most likely because of re-plants-vs-zombies being built as x64 and the normal Plants vs. Zombies game files containing x86 binaries and causing a mismatch between the bass.dll re-plants-vs-zombies expects and the actual bass.dll.
Download the latest BASS version directly from their website here. (the x64 version of bass is located in the x64 folder inside the zip)
After that just drop bass.dll inside your game files and it should work fine.
We will make sure to, from now on, start statically linking the BASS library inside the project so such stuff doesn't happen anymore.
Best regards, Patoke.

Hi! First of all clone this repo again since I just changed the history of it to include a bunch of commits in the history that got irreparably squashed. (this will never happen again this is a one time special).
As to your actual question, this repo uses the most up to date 64 bit build of un4seen developments bass. You can download it here: https://www.un4seen.com/download.php?bass24
The current build of the game does not support reading off the main.pak due to some 64 bit shenanigans with the way the library works, this is being worked on and the plan is to give the game the ability to read from pak files. For now you'll have to undergo the fairly completely undocumented and annoying process of building tools/decrypt.c and tools/metadata.c, running the decrypt binary you just built with the main.pak file in the same directory will decrypt it (really it's just xor'd with a bunch of numbers) and then run the metadata executable which will complain at you until you've given it the right directory structure for it to unpack the game files into. ONCE you've done that, you can try running the game with it then, but actually I think there's some weird encoding stuff going on.
Honestly I'll just send you a compressed main.pak with my changes already made. I've made a fair few to this directory to get it working (including fixing a couple mistakes they made with the original files!? I'll find a way to apply a patch with a diff soon: https://1drv.ms/u/s!AgAdDsqhqeAPhmrr2HRwvZ2dS12n?e=zoAmR7
Thanks for your patience with the munted build system and if you'd like to see it fixed we're always keen on pull requests.

Aaron.

Hello, that is most likely because of re-plants-vs-zombies being built as x64 and the normal Plants vs. Zombies game files containing x86 binaries and causing a mismatch between the bass.dll re-plants-vs-zombies expects and the actual bass.dll.
Download the latest BASS version directly from their website here. (the x64 version of bass is located in the x64 folder inside the zip)
After that just drop bass.dll inside your game files and it should work fine.
We will make sure to, from now on, start statically linking the BASS library inside the project so such stuff doesn't happen anymore.
Best regards, Patoke.

Thanks, it worked!

Hi! First of all clone this repo again since I just changed the history of it to include a bunch of commits in the history that got irreparably squashed. (this will never happen again this is a one time special).
As to your actual question, this repo uses the most up to date 64 bit build of un4seen developments bass. You can download it here: https://www.un4seen.com/download.php?bass24
The current build of the game does not support reading off the main.pak due to some 64 bit shenanigans with the way the library works, this is being worked on and the plan is to give the game the ability to read from pak files. For now you'll have to undergo the fairly completely undocumented and annoying process of building tools/decrypt.c and tools/metadata.c, running the decrypt binary you just built with the main.pak file in the same directory will decrypt it (really it's just xor'd with a bunch of numbers) and then run the metadata executable which will complain at you until you've given it the right directory structure for it to unpack the game files into. ONCE you've done that, you can try running the game with it then, but actually I think there's some weird encoding stuff going on.
Honestly I'll just send you a compressed main.pak with my changes already made. I've made a fair few to this directory to get it working (including fixing a couple mistakes they made with the original files!? I'll find a way to apply a patch with a diff soon: https://1drv.ms/u/s!AgAdDsqhqeAPhmrr2HRwvZ2dS12n?e=zoAmR7
Thanks for your patience with the munted build system and if you'd like to see it fixed we're always keen on pull requests.

Aaron.

Hi Aaron,
It turns out I don't need the main.pak, because I just ended up unpacking it using a Plants vs. Zombies main.pak Patcher, and placed all the folders the game asked for in the files. But thanks anyway!


Picovoice Issue:

Make sure you have read the documentation, and have put forth a reasonable effort to find an existing answer.
Expected behaviour
In the picovoice console, after logging in it asks to enter name and  agree to privacy terms. After entering those details and submitting I should be allowed to move forward and train my custom keyword.
Actual behaviour

Aftet submitting the details, it doesn't save and shows an error response in one of the network calls in the network tab in dev tools
Steps to reproduce the behaviour
Create a personal account and then add name.
(Include enough details so that the issue can be reproduced independently.)

Sorry you've run into that issue. I'm unable to reproduce the problem on my end. Can you please retry and let me know if the issue re-occurs?
Trying to determine if this was a momentary network issue or an actual bug. Either way, the UI should respond to the error better.

Hi, thanks for the response. I tried again and the same thing is happening with me. I have tried to do it over 3 different network connections as well. Let me know if you need any more information.

Can you please email<EMAIL_ADDRESS>with the email that you used to sign up and reference the URL of this issue? We can investigate the logs and see what's up.
Also, could you please try logging out and logging back in again?

Yes sure, I will mail to them with the specifics.
I have already tried it after logging out. I even created two accounts with two different email IDs after the first one didn't work. The issue is coming in all the cases for me.

We haven't appeared to receive an email from you. Did you send one to<EMAIL_ADDRESS>If so, can you please try again and comment in this issue, or if not, let us know and we'll close this issue.

We haven't heard from you so I'm going to close the issue. Please re-open if you encounter this issue again (and send an email to that address indicating the time that the issue occurred with the email used for the account). We've also added a task internally to make the error message in this scenario appear properly.


vdc: External etcd datasource for kubernetes

[x] etcd config manager support https://github.com/threefoldtech/js-ng/pull/562
[x] etcd solution / cluster https://github.com/threefoldtech/js-sdk/pull/1858 support
[x] provision etcd cluster along with vdc deployment https://github.com/threefoldtech/js-sdk/issues/2552
[x] kubernetes vms to support external etcd https://github.com/threefoldtech/zos/issues/1190

verified.


Add monthly rate limits

https://community.openai.com/t/openai-api-proxy-to-share-limited-token-budget-with-others/535403/14

@LinqLover we support monthly spend limit now

Very nice! Would this also work with minutes? Also, monthly cost limits  are not yet combinable with hourly cost limits, are they?

Very nice! Would this also work with minutes? Also, monthly cost limits are not yet combinable with hourly cost limits, are they?

To answer your first question, minutes is not available yet but I can make it available quite easily. For the second question, they are not combinable atm.

The first one would be great! The second one, too, but not so urgent. :-)

The first one would be great! The second one, too, but not so urgent. :-)

Updated. minute sped limit should be supported in the latest version


Remove look behind to use the fast version of SublimeText indexer

Overview
Sublime Text has its own parser/tokenizer. It has an old version that handle look-behind in regexes,
and a faster on that disallow them.
I removed the look-behind in this PR trying to make as few changes as possible to the syntax.
This is a  follow up on #41.
Simple benchmark
I ran Syntax Tests - Performance on 2000 lines of Zig code,
made from custom code and hash_map.zig from the std lib.

Original syntax with look behind: average of 45.2ms over 10 runs
After this PR: average of 35.8ms over 10 runs

How to review this PR

look at the commits one-by-one.
the first commit add a lot of test documenting the behavior of the syntax before this PR.
some commit add more tests
commits that change the syntax file will also change the test only if there is a breaking change
breaking change only impact invalid Zig code being written

@emekoi Can I get a review on this please ?

@emekoi Can I get a review on this please ?

sorry about taking so long to get back to you, i was extremely busy with school but i should be able to review and merge this by friday, if not tomorrow/tonight.

sorry about taking so long to get back to you, i was extremely busy with school but i should be able to review and merge this by friday, if not tomorrow/tonight.

thanks for taking the time to do this and being so patient with me.

thanks for taking the time to do this and being so patient with me.

Thanks for the review, and maintaining this package.
I'll be sure to stick around in case people find a bug or if there is subtle behavior change that trip up users.

Thanks for the review, and maintaining this package.
I'll be sure to stick around in case people find a bug or if there is subtle behavior change that trip up users.


Razor Formatting fails when project has # in path

This issue has been moved from a ticket on Developer Community.

Since the latest visual studio update 17.3.0 (also with 17.3.1) when I format my pages using Advanced, Format Document the formatting goes mental and just keeps moving blocks of code to the right.  If I keep doing it, it keeps moving.
I have repaired and now completely reinstalled visual studio but its no better.  In the later part of my recording I am pressing the shortcode keys to repeat the Format Document.
Its also left other bits of my HTML code in a real mess with DIV and /DIVS not lining up etc.
C# Part of the code is generally ok.
Regards
Simon

Original Comments
Feedback Bot on 22/08/2022, 00:39 PM:
(private comment, text removed)
Wenwen Fan [MSFT] on 24/08/2022, 04:31 PM:
(private comment, text removed)
Simon Richardson on 24/08/2022, 05:18 PM:
(private comment, text removed)
Wenwen Fan [MSFT] on 06/09/2022, 08:05 PM:
(private comment, text removed)
Wenwen Fan [MSFT] on 06/09/2022, 08:06 PM:
(private comment, text removed)
Simon Richardson on 06/09/2022, 08:31 PM:
(private comment, text removed)
Simon Richardson on 06/09/2022, 11:19 PM:
(private comment, text removed)
Simon Richardson on 07/09/2022, 00:52 AM:
(private comment, text removed)
Wenwen Fan [MSFT] on 07/09/2022, 05:16 PM:
(private comment, text removed)

Original Solutions
(no solutions)

It's not clear from the description of the issue here, but this was caused by having a # in the users path. Looks like Roslyn and Razor language servers are both shutting down. Possibly a CLaSP issue?


Deprecated auth method in kubeconfig on GKE

Ref. https://github.com/epinio/epinio/issues/1961
Test run ok https://github.com/epinio/epinio/actions/runs/3693068424/jobs/6252638048

Codecov Report
Base: 56.99% // Head: 32.05% // Decreases project coverage by -24.93% :warning:

Coverage data is based on head (035e88e) compared to base (8196240).
Patch has no changes to coverable lines.

Additional details and impacted files
@@             Coverage Diff             @@
##             main    #1962       +/-   ##
===========================================
- Coverage   56.99%   32.05%   -24.94%
===========================================
  Files         179      179
  Lines       14523    14523
===========================================
- Hits         8278     4656     -3622
- Misses       5217     9202     +3985
+ Partials     1028      665      -363

Flag
Coverage Δ

acceptance-api
?

acceptance-apps
?

acceptance-cli
30.22% <ø> (-20.26%)
:arrow_down:

unittests
8.26% <ø> (ø)

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

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

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


[Help Text] Help text wording structure

Describe the bug
The help text for the Event Name field. should be updated to read a bit clearer.  Change FROM Name of event and displays on registration page  TO Name of the event that displays on the registration page.

Included in beta release <IP_ADDRESS>

Included in production release <IP_ADDRESS>


Junit.xml Reporter Characters

I noticed an issue with the junit reporter character encoding when used with playwright-axe. The issue seems to be exclusive to junit reporter, as opposed to perfect output from the line reporter.
Line Reporter Test Result Output:

In the xml file though, the encoding is different:

CircleCi can't parse due to this:

Is this normal behavior? I've been looking at the modules for far too long.

I'm still getting the same issue. Can this be re-opened?
`

tests/foo.spec.ts:3:1 › [chromium] is a basic test with the page =================================
Error: expect(received).toBe(expected) // Object.is equality

Expected: &quot;2&quot;
Received: &quot;Playwright&quot;

  4 |     await page.goto(&quot;https://playwright.dev/&quot;);
  5 |     const name = await page.innerText(&quot;.navbar__title&quot;);
> 6 |     expect(name).toBe(&quot;2&quot;);
    |                  ^
  7 | });

    at Spec.fn (/home/user/Desktop/dev/tdd-playwright-example/e3e/tests/foo.spec.ts:6:18)
    at WorkerRunner._runTestWithBeforeHooks (/home/user/Desktop/dev/tdd-playwright-example/e3e/node_modules/folio/out/workerRunner.js:299:13)`

Here is my config file based on the new documentation:
import { ChromiumEnv, FirefoxEnv, WebKitEnv, test, setConfig } from "@playwright/test";
import { reporters, setReporters } from "@playwright/test";

setConfig({
    testDir: __dirname,  // Search for tests in this directory.
    timeout: 30000,  // Each test is given 30 seconds.
    // timeout: 90000,  // Each test is given 90 seconds.
    retries: 0,  // Failing tests will be retried at most two times.
});

setReporters([
    // Report to the terminal with "line" reporter.
    new reporters.line(),
    // Additionally, output a JUnit XML file.
    new reporters.junit({ outputFile: 'junit.xml', stripANSIControlSequences: true }),
]);

const options = {
    // Launch options:
    headless: true,
    slowMo: 50,
    // Context options:
    viewport: { width: 800, height: 600 },
    ignoreHTTPSErrors: true,
    // Testing options:
    // video: 'retain-on-failure',
    screenshot: 'only-on-failure'
};

// Run tests in three browsers.
test.runWith(new ChromiumEnv(options), { tag: 'chromium' });
test.runWith(new FirefoxEnv(options), { tag: 'firefox' });
test.runWith(new WebKitEnv(options), { tag: 'webkit' });```

@BillyBolton What is the problem in the example you pasted? You don't like & escapes? Unfortunately, we'll have to escape at least some characters, e.g. <, otherwise it won't be a valid xml file.

@dgozman Thanks for getting back to me. Yea the &amp, &quote, &gt aren't playing well. Comparing the line and junit reporters seem to conclude that it's related to ANSI formatting. I try using the new config option,  stripANSIControlSequences: true, but it doesn't seem to make a difference.


Remove CmdLineParser since the CLI has been removed now (in #83).

Coverage increased (+2.7%) to 59.344% when pulling b052418fb61832e55dcdc30d2a3c446b715a4a91 on tomwhite:remove-command-line-parser into 05298aefa4f91998c335525fee00fc48a2559cc8 on HadoopGenomics:master.

@tomwhite: I was about to revise my PR but now you did this already which is great :-) go ahead, let's clean this up. Thanks, I completely had forgotten about this "inline" dependency.


Fix compile error on MSVC 2019

MSVC does not support initializing the rb_data_type_t structure from a pointer. Error is:
ruby_wrapper.c(359): error C2099: initializer is not a constant
ruby_wrapper.c(359): error C4047: 'initializing': 'void *' differs in levels of indirection from 'int'
Fixed by moving code to pycall_init_ruby_wrapper.

@cfis Thank you for fixing this issue!
I want to apply this change only to the case of MSVC, so I'll merge this after adding some small fixes similar to the following trick:
https://github.com/ruby/ruby/pull/3024/files#diff-779cc912d1b415b1412b73ce56c7b649R56-R65

Also, I want to add a mew CI job for testing on MSVC.
I will add some commits to this pull-request.

Sounds good to me. Interesting that CI has MSVC ruby builds, that's helpful to know.

@cfis I have trouble that we don't have MSVC-built Ruby for running CI.
Before preparing MSVC-built Ruby, I want to confirm the reason why you want to use pycall.rb on MSVC-built Ruby.
Could you please explain it?  Isn't mingw-version Ruby satisfy your requirements?

I use both mingw and mswin versions - but usually mingw ruby.  However, I use the mswin version of ruby for creating/debugging extensions on Windows. Visual Studio has some really nice debugging facilities, and also let me step through Ruby itself and Python. For example, I use mswin ruby for my work on ruby-prof, libxml-ruby, etc.
Hope that helps.

@cfis Thank you for your feedback.  I understood your situation.
I'd like to let pycall.rb support of MSVC-built Ruby.  For the purpose, I need to change our ruby/setup-ruby action so that we can install MSVC-built Ruby binary on the virtual environment.
Please give me some days to the preparation.

No problem, and thanks for looking at this. Let me know if you need any help.


Implement OKEx dealer bot

The current implementation for hedging USD balance uses FTX Perpetual Futures/Swap.
That strategy has a few documented drawbacks.
So we intend on implementing hedging using OKEx inverse futures.
Allowing for the using the most advantageous exchange/strategy, a composition of both, etc.

After thinking about it, I think one of the first task that should be done is refactoring the FTX module and splitting in 2 parts.
First we want to have a high level how much USD should be hedged/maintain. And also have some boundary to know whether it's necessary to buy/sell/hedge some USD.
ie: we should have $1,000 in our books (we have a liability of $1,000 from the user of the wallet), but we already have $998, and so no action is really needed because our holding is in the boundary of +/- 1%.
now if we get a user balance of $1,200, then we need to buy USD, and we can execute some order. we will want to have multiple provider for the USD part. so we could have OkEx/FTX/Bitfinex, stable coin, or local "real" USD in a bank account as a provider. from an architecture standpoint, this part can be different because we can have different providers with different rules.
So it's probably make sense to first start refactoring our backend in that sense.

All right re-orienting toward that split.
Will investigate the MongoDb watch for changes on USD balance,
so notification would be sent and processed to assess need for re-hedging.
Alternatively query the database on a set sustainable rate (1x / 5sec?).

moved to its own repo


JS fixes for polkadot-v0.9.37

I recently recompiled the standalone node with polkadot-v0.9.37 for testing and realized that at least some things do not work:
Tasks:

[ ] we can no longer parse the extrinsic updates after sending transactions.
[ ] next-phase call
[ ] It needs to be compatible with both pre-v0.9.37 and v0.9.37 (luckily connection works, so we can connect to the node and check its chain-id (parachain vs. solochain) and runtime_spec version to distinguish the code that needs to be called. This is how I implemented it for another case: https://github.com/encointer/encointer-wallet-flutter/pull/884/files

Note: There is already a more recent encointer-js version (merely updating the library won't resolve these issues though). However, the last time I tried to integrate it into our webView, I failed to load the JS-code in the webView. I don't understand why this is so brittle.

Might be solved by #1344


ruby-1.9.3-p392 fails to build

On Mountain Lion, with i686-apple-darwin11-llvm-gcc-4.2 installed:
Recipe: rbenv::user
  * rbenv_ruby[1.9.3-p392 (stephen)] (stephen) action installRecipe: <Dynamically Defined Resource>
  * rbenv_script[rbenv install 1.9.3-p392 (stephen)] action run

  * rbenv_script[rbenv install 1.9.3-p392 (stephen)] action nothing (up to date)
  * script[rbenv install 1.9.3-p392 (stephen)] action run
================================================================================
Error executing action `run` on resource 'script[rbenv install 1.9.3-p392 (stephen)]'
================================================================================

Mixlib::ShellOut::ShellCommandFailed
------------------------------------
Expected process to exit with [0], but received '1'
---- Begin output of "bash"  "/tmp/chef-script20130225-11972-84e9lw" ----
STDOUT:
STDERR: rbenv: version `global' not installed
Downloading yaml-0.1.4.tar.gz...
-> http://dqw8nmjcqpjn7.cloudfront.net/36c852831d02cf90508c29852361d01b
Installing yaml-0.1.4...
Installed yaml-0.1.4 to /Users/stephen/.rbenv/versions/1.9.3-p392

Downloading ruby-1.9.3-p392.tar.gz...
-> http://ftp.ruby-lang.org/pub/ruby/1.9/ruby-1.9.3-p392.tar.gz
Installing ruby-1.9.3-p392...

BUILD FAILED

Inspect or clean up the working tree at /tmp/ruby-build.20130225153220.12196
Results logged to /tmp/ruby-build.20130225153220.12196.log

Last 10 log lines:
compiling safe.c
compiling signal.c
compiling sprintf.c
compiling st.c
st.c:520:35: error: implicit conversion loses integer precision: 'st_index_t' (aka 'unsigned long') to 'int' [-Werror,-Wshorten-64-to-32]
            i = table->num_entries++;
              ~ ~~~~~~~~~~~~~~~~~~^~
1 error generated.
make: *** [st.o] Error 1
make: *** Waiting for unfinished jobs....
---- End output of "bash"  "/tmp/chef-script20130225-11972-84e9lw" ----
Ran "bash"  "/tmp/chef-script20130225-11972-84e9lw" returned 1

Resource Declaration:
---------------------
# In /Users/stephen/src/rbenv/providers/script.rb

 28:   script new_resource.name do
 29:     interpreter   "bash"
 30:     code          script_code
 31:     user          new_resource.user     if new_resource.user
 32:     creates       new_resource.creates  if new_resource.creates
 33:     cwd           new_resource.cwd      if new_resource.cwd
 34:     group         new_resource.group    if new_resource.group
 35:     path          new_resource.path     if new_resource.path
 36:     returns       new_resource.returns  if new_resource.returns
 37:     timeout       new_resource.timeout  if new_resource.timeout
 38:     umask         new_resource.umask    if new_resource.umask
 39:     environment(script_environment)
 40:   end
 41:

Compiled Resource:
------------------
# Declared in /Users/stephen/src/rbenv/providers/script.rb:28:in `block in class_from_file'

script("rbenv install 1.9.3-p392 (stephen)") do
  action "run"
  retries 0
  retry_delay 2
  command "\"bash\"  \"/tmp/chef-script20130225-11972-84e9lw\""
  backup 5
  environment {"RBENV_ROOT"=>"/Users/stephen/.rbenv", "USER"=>"stephen", "HOME"=>"/Users/stephen"}
  returns [0]
  user "stephen"
  code "export RBENV_ROOT=\"/Users/stephen/.rbenv\"\nexport PATH=\"${RBENV_ROOT}/bin:$PATH\"\neval \"$(rbenv init -)\"\n\nrbenv shell global\n\nrbenv install 1.9.3-p392\n"
  interpreter "bash"
  cookbook_name :rbenv
end

[2013-02-25T15:35:22+00:00] ERROR: Running exception handlers
[2013-02-25T15:35:22+00:00] ERROR: Exception handlers complete
Chef Client failed. 6 resources updated
[2013-02-25T15:35:22+00:00] FATAL: Stacktrace dumped to /var/chef/cache/chef-stacktrace.out
[2013-02-25T15:35:22+00:00] FATAL: Mixlib::ShellOut::ShellCommandFailed: script[rbenv install 1.9.3-p392 (stephen)] (/Users/stephen/src/rbenv/providers/script.rb line 28) had an error: Mixlib::ShellOut::ShellCommandFailed: Expected process to exit with [0], but received '1'
---- Begin output of "bash"  "/tmp/chef-script20130225-11972-84e9lw" ----
STDOUT:
STDERR: rbenv: version `global' not installed
Downloading yaml-0.1.4.tar.gz...
-> http://dqw8nmjcqpjn7.cloudfront.net/36c852831d02cf90508c29852361d01b
Installing yaml-0.1.4...
Installed yaml-0.1.4 to /Users/stephen/.rbenv/versions/1.9.3-p392

Downloading ruby-1.9.3-p392.tar.gz...
-> http://ftp.ruby-lang.org/pub/ruby/1.9/ruby-1.9.3-p392.tar.gz
Installing ruby-1.9.3-p392...

BUILD FAILED

Inspect or clean up the working tree at /tmp/ruby-build.20130225153220.12196
Results logged to /tmp/ruby-build.20130225153220.12196.log

Last 10 log lines:
compiling safe.c
compiling signal.c
compiling sprintf.c
compiling st.c
st.c:520:35: error: implicit conversion loses integer precision: 'st_index_t' (aka 'unsigned long') to 'int' [-Werror,-Wshorten-64-to-32]
            i = table->num_entries++;
              ~ ~~~~~~~~~~~~~~~~~~^~
1 error generated.
make: *** [st.o] Error 1
make: *** Waiting for unfinished jobs....
---- End output of "bash"  "/tmp/chef-script20130225-11972-84e9lw" ----
Ran "bash"  "/tmp/chef-script20130225-11972-84e9lw" returned 1

This looks very similar to: https://github.com/sstephenson/ruby-build/issues/297
I wonder if this means ruby-build is using clang rather than gcc?

1.9.3 hit end of life in february, so closing off.


Reset doesn't work with VectorEnv api: https://gymnasium.farama.org/api/vector/

Reset by pose doesn't work with VectorEnv API, should consider setting initial pose as an argument during make().

Can you provide an example?
I can reset the vectorized environment through the options. See tests in the linked pr.

I was using the instantiation used in CleanRL: here, where multiple envs can run in parallel.

Good idea, we can define reset modes like in racecar_gym (random, grid, etc) and pass them in the config

Fixed by #94


Added switch to force manual mapping

In this case, the module is not going to be added to the internal modules list, but it seems to be logical :) The new flag is "ForceRemap".
I haven't run your tests, as I'm not yet sure how they works, but I doubt it would break anything.
I'm not sure if:
pMod = std::make_shared<const ModuleData>(ldrEntry);
is leaking anything or if it's correct at all, it might be solved any better way, please check that out.
Thanks!

Merged manually after few improvements - 7a2f7b23da1e1ac205463826e4cdc66d1ad57c37


Tracking issues with generating reference docs v1.14

Opening up this issue tracking the problems generating for v1.14: https://kubernetes.io/docs/contribute/generate-ref-docs/
Federation docs, these might not even be needed, but they generated nothing new (except undoing the relative link work):
https://kubernetes.io/docs/contribute/generate-ref-docs/federation-api/
https://github.com/kubernetes/website/pull/13363
kubectl, API, and components are all failing for various reasons when following the instructions. Working with @tengqm and @kubernetes/sig-docs-en-owners to review / fix / improve the documentation.
cc/ @zacharysarah @spiffxp @steveperry-53

@jimangel i suspect https://github.com/kubernetes/website/issues/13426 has the same root cause ?

I'd like to close this before 1.15 drops, but there's still some work left to be done, such as figuring out direction on fedv1 docs and some other minor PRs I've seen come in correcting the ref-gen docs. I'm going to help where I can, but I'll look for @MAKOSCAFEE to help complete this.
/assign @MAKOSCAFEE

@makoscafee after you successfully generate the reference docs for 1.15 (and ensure documentation is exact), feel free to ping me to close.

Sure, I will ping you when is done ...

Planning on closing milestone 1.15, reference docs were generated. But not clear of the process as I was able to only generate API reference while components reference page generation is a problem still. I think I will need to work with @kbhawkey and @tengqm to update documentation of the process.

Please feel free to file issues at kubernetes-incubator/reference-docs .

There is already work in progress for this at  https://github.com/kubernetes/website/issues/14111
/milestone clear

/close
moving discussion to #14111


port automatic license update to snyk

Description
Inspired from https://github.com/dockstore/dockstore-ui2/pull/1522
Issue
n/a but all these snyk updates suddenly, probably as a result of https://ucsc-cgl.atlassian.net/browse/SEAB-4045
Please make sure that you've checked the following before submitting your pull request. Thanks!

[x] Check that you pass the basic style checks and unit tests by running mvn clean install
[x] Follow the existing JPA patterns for queries, using named parameters, to avoid SQL injection
[x] If you are changing dependencies, check the Snyk status check or the dashboard to ensure you are not introducing new high/critical vulnerabilities
[x] Assume that inputs to the API can be malicious, and sanitize and/or check for Denial of Service type values, e.g., massive sizes
[x] Do not serve user-uploaded binary images through the Dockstore API
[x] Ensure that endpoints that only allow privileged access enforce that with the @RolesAllowed annotation
[x] Do not create cookies, although this may change in the future

on the other hand, this won't work because snyk keeps on targeting the generated pom files
shall we turn this off @david4096 @coverbeck on the snyk side?

turned off on the snyk side


[material-ui] Add PropsProvider for Pigment CSS integration

Context
The current approach for customizing Material UI component's default props is using the theme.components.*.defaultProps through React context.
For Pigment CSS, it's not possible to use React Component inside the theme because of build-time extraction.
Solution
Extract the defaultProps to another context, PropsProvider, so that users can customize the default props of Material UI components with Pigment CSS.
How's it look like

Before or After migration to v6: no changes required, the defaultProps that pass to the theme will propagate to PropsProvider automatically.

Transitioned to Pigment CSS while having Emotion: no changes required as well, as long as the ThemeProvider is used on the client.

Pigment CSS without Emotion: users need to import PropsProvider from Pigment CSS and pass thetheme.components
import PropsProvider from '@mui/material/PropsProvider';

<PropsProvider
  value={{
    // copied from the existing theme.components
    MuiButton: { defaultProps: { disableRipple: true } },
    MuiSelect: { defaultProps: { IconComponent: CustomIcon } },
  }}>
  {children}
</PropsProvider>

Finally, Emotion can be removed.

Search keywords:

For Pigment CSS, it's not possible to use React Component inside the theme because of build-time extraction.

It's still no clear to me that this is true. We didn't explore https://github.com/mui/material-ui/pull/42476#discussion_r1632325822? The way Yak makes it work doesn't seem to even need the React.cache API, using a global does the same thing, since as I understand it, there no theme nesting support in Pigment CSS: https://yak.js.org/how-does-it-work#the-runtime-part.
I'm connecting the same dots with https://github.com/mui/pigment-css/issues/129. It would make sense to me to remove the <DefaultPropsProvider> API, we need this too for server components. Issue updated.


Multiple RowLacks?

Is there some trick that I'm missing that would allow me to say that a row lacks more than one label?
I encountered this when trying to do some stuff with purescript-record:
import Data.Record as Rec
import Data.Symbol (SProxy(..))
import Type.Prelude (class RowLacks)

insertA ∷ ∀ r. RowLacks "a" r ⇒ { | r } → { a ∷ Int | r }
insertA = Rec.insert (SProxy ∷ SProxy "a") 1

insertB ∷ ∀ r. RowLacks "b" r ⇒ { | r } → { b ∷ Int | r }
insertB = Rec.insert (SProxy ∷ SProxy "b") 1

insertAB ∷ ∀ r. RowLacks "a" r ⇒ RowLacks "b" r ⇒ { | r } → { a ∷ Int, b ∷ Int | r }
insertAB = insertA <<< insertB -- seems to be unimplementable, this and any combination of annotations etc. I could come up with for intermediate steps fails

As far as I can see there's no way that multiple RowLacks constraints can be satisfied, given the Union-based implementation. Maybe this is something that would be better off solved by the compiler after all?

@garyb in this case, you'll want proof of RowLacks "a" (b :: Int | r), i.e:
insertAB ∷ ∀ r. RowLacks "a" (b :: Int | r) ⇒ RowLacks "b" r ⇒ { | r } → { a ∷ Int, b ∷ Int | r }
insertAB = insertA <<< insertB

Alternatively you'd need some a way of conjuring up an instance of RowLacks "a" (b :: r) from RowLacks "a" r and "a" /= "b".  Which sounds more difficult.

RowLacks "a" (b :: Int | r)

That's probably the only thing I didn't try when attempting to work around it, thanks!


Building out Unit 4

Ready for review @rors @nmadd

@rors @nmadd

I'm merging this so i can start working on #6


uv_fs_read hangs

Version: <1.28.0>
Platform: <Linux RHEL 64Bit>

I am trying to read file with multiple threads. but uv_fs_read does gets to callback and hangs.
sample code is attached for reproducing the issue
main.txt

AFAICT, you've filled the workqueue with blocking functions, then wondered why no workqueue operations are proceeding. You need to launch your own worker threads.

There's a UV_THREADPOOL_SIZE environment variable that you can set to control the size of libuv's internal thread pool (default: 4.) See also http://docs.libuv.org/en/v1.x/threadpool.html.
I'm moving this to libuv/help.


Function list API return more message

Is your feature request related to a problem? Please describe.
Now. function list API just return functions name and it's greate if return more message:

Running [true/false]
workerId ""
Restarts (from status.numRestarts)

like this,

NAME
RUNNING
RESTARTS
WORKERID

first
true
0
c-standalone-fw-localhost-8080

second
false
10
c-standalone-fw-localhost-8080

Describe the solution you'd like
Describe alternatives you've considered
Additional context
I am not familiar with function code, but if want implement this function quickly and easily, Can process the code.
Request status API for every function name. I am not sure about that Quality and performance.
Componentlmpl#listFunctions
for (FunctionMetaData functionMetaData : functionStateList) {
            if (InstanceUtils.calculateSubjectType(functionMetaData.getFunctionDetails()).equals(componentType)) {
             *** get status ***
                retVals.add(functionMetaData.getFunctionDetails().getName());
            }
        }

The issue had no activity for 30 days, mark with Stale label.


a bug for AliOS-Things with ESP32 on Ubuntu 16.04.4

with rel_1.3.1  build esp32 has a bug. rel_1.3.1 is ok.
my script is:
1:
git clone --recursive -b rel_1.3.1 https://github.com/alibaba/AliOS-Things.git
cd AliOS-Things
aos make helloworld@esp32devkitc   //  Error   not find esptool.py
2:
git clone --recursive -b rel_1.3.0 https://github.com/alibaba/AliOS-Things.git
cd AliOS-Things
aos make helloworld@esp32devkitc  //Ok

Please attach corresponding error log. esptool.py should be able to download automatically.


Cleanup "delete LOI" related UI code

Fixes #1181

@gino-m  PTAL?

Codecov Report

Merging #1182 (176d49d) into master (d004a82) will increase coverage by 0.06%.
The diff coverage is n/a.

@@             Coverage Diff              @@
##             master    #1182      +/-   ##
============================================
+ Coverage     28.82%   28.89%   +0.06%
+ Complexity      843      840       -3
============================================
  Files           304      304
  Lines          8360     8319      -41
  Branches        368      365       -3
============================================
- Hits           2410     2404       -6
+ Misses         5819     5784      -35
  Partials        131      131

Impacted Files
Coverage Δ

...estdetails/LocationOfInterestDetailsViewModel.java
69.81% <0.00%> (-3.08%)
:arrow_down:

...gle/android/ground/ui/home/HomeScreenFragment.java
0.00% <0.00%> (ø)

...le/android/ground/ui/home/HomeScreenViewModel.java
0.00% <0.00%> (ø)

...restdetails/LocationOfInterestDetailsFragment.java
0.00% <0.00%> (ø)

Help us with your feedback. Take ten seconds to tell us how you rate us.


write ending

On audio-speaker 2.0 for browser, caught a bug with the write ending in web-audio-stream (at https://github.com/audiojs/audio-speaker/pull/44) as detailed in the PR
A few weeks back I tried using some of ScriptProcessorNode's events (documented on MDN) to solve this:

complete
onend
statechange
...
audioprocess

None of these besides audioprocess seemed to actually do anything under ctx.destination, which you had already seemed to be using.  So I'm curious how we should solve this.
Ideas? @dfcreative @connorhartley :smile:

@jamen can you please show a test-case for that? Not really following what you mean. The basic premise is that writing null should disconnect ScriptProcessorNode and end streaming if there is any. Also I am not sure why is it addressed here, not in web-audio-write, I suppose we should not use web-audio-stream in audio-speaker at all.

@dfcreative My bad.  There is still web-audio-stream/write here on the master branch, and you splitting the package had jogged my memory.  Moving.

@jamen is this issue still actual for web-audio-write? I can fix while at it


forest: Watch taxinoibai.pro

forest requests the watch of the watch_keyword taxinoibai\.pro. See the MS search here and the Stack Exchange search in text, in URLs, and in code.

taxinoibai\.pro has been seen in 1 true positive, 0 false positives, and 0 NAAs.

Approved by Ryan M in Charcoal HQ


Fix of the comment. It looks like a typo.

I think I saw a typo. And I tried to fix it but ... the script for pirates does not want to be executed.
And I don't see the reason. Sorry.


Support for switching ruby lambda literals

It'd be cool if we could switch between
-> (a, b) { a.do_stuff_with(b) }

and
lambda do |a, b|
  a.do_stuff_with(b)
end

Though there's gonna have to be some fiagling to make it not overlap with the block-syntax splitjoiner
(thanks for the awesome plugin, btw :smile:)

Sorry it took me a while to get to this.
It would be possible to transform the arrow-block into a lambda, but I'm not sure under what circumstances we want which. I guess both syntaxes are perfectly valid for both one-line and multiline formats, right? Is this a matter of coding style?
If it is, one thing I'm considering is adding an option, similar to the one in #67. Maybe something like:
" Arrow lambda when splitting, but normal "lambda" when joining
let g:splitjoin_ruby_arrow_lambdas = 'Sj'

" Arrow lambda when joining, but normal "lambda" when splitting
let g:splitjoin_ruby_arrow_lambdas = 'sJ'

Does this look like a reasonable option to you? Or do you think it should always be one or the other for some reason? Or is it different depending on circumstances (in which case, maybe it could be added as a rule in switch.vim instead, so you can switch between them on a case-by-case basis)?

This is wrong. Multi-line lambda in ruby is written like this:
lambda { |x|
...
}

Not with do end like blocks.

that's just like, your opinion, man

https://github.com/bbatsov/ruby-style-guide#single-line-blocks

https://github.com/bbatsov/rubocop/blob/master/lib/rubocop/cop/style/block_delimiters.rb#L44

Oh right, sorry, it's only when you pass multiline lambda as a parameter. For example when using scope in Rails. You need to surround it with () or use curly braces otherwise the block will be treated as a parameter for scope.
It just messed up in my head.

yeah, you're right on that.

Sorry, again, for taking so long to address this.
For starters, there was a separate issue related to this, actually: https://github.com/AndrewRadev/switch.vim/issues/27
What I implemented there was a way to toggle between the arrow style and lambdas. That said, it would still be a useful tool to have in splitjoin, I assume not many people use the lambda syntax anyway. I'll leave this open and hope I can get to this a bit sooner this time.

Small correction just in case:

I assume not many people use the lambda syntax anyway

Ruby style guide recommends using lambda for multiline blocks: https://github.com/bbatsov/ruby-style-guide#lambda-multi-line.


Broken links

Hi,
Is it possible to get access to the linked models and datasets?

Some of the models are available at https://huggingface.co/NbAiLab. But most of them are still private until we get a decent performance. Sharing the data is a different story. We'll try to publish as much as legally possible 🤷🏼


reverts use of semistandard directly (#2648)

Abstractions around ESLint are not widely supported in text editors or IDEs.  For example, I could tell WebStorm to use node_modules/eslint-config-semistandard/eslintrc.json, but there's no way for WebStorm to know about the overrides in the semistandard prop of package.json, nor the overrides specified for test files in Makefile.
semistandard is arguably easier to use on the CLI, and requires less configuration files, but the cost to users primarily interfacing with ESLint via contextual information in an IDE (like myself!) outweighs the benefit.
cc @kt3k

Worth mentioning: WebStorm supports standard, but not semistandard, nor xo.

@Flet
Can you see this issue? How do you think?

Personally I don't use editor integration for linting. I think this issue depends on people and the team who use it. If you evaluate the negative impact is bigger than the benefits here, I think it makes sense to switch back to eslint cli.

Yeah, @boneskull is right in that IDE integration can be tricky if its not supported by an existing plugin. For WebStorm, it should be possible to configure it manually similar to how it can be done with standard: https://standardjs.com/webstorm.html

be sure to turn on them semicolons in the xml :)
for .eslintrc.json use {"extends": ["semistandard"]}

semistandard and standard have some ignore paths baked-in that eslint does not (including ignoring everything in .gitignore), so to remedy that, you may also need an .eslintignore file: http://eslint.org/docs/user-guide/configuring#ignoring-files-and-directories.
There are semistandard plugins for Sublime Text and VS Code which work great, but yeah it can be a pain in other IDEs :/. The JetBrains folks might be receptive to adding semistandard too, but I don't know if anyone has asked :).
For this particular PR, I'd say adding a .eslintignore file would be needed.

For this particular PR, I'd say adding a .eslintignore file would be needed.

We have an .eslintignore file already, which was not removed.  This should be GTG.


Implement Home Page with Login/Signup Navigation

Description:
The application needs a home page that serves as the entry point for users. This page should feature clear navigation options for users to either log in or sign up. The design should be in line with our Material UI theme.
Acceptance Criteria:

Home page design is consistent with Material UI standards.
Contains buttons for Login and Signup that redirect to their respective pages.
Responsive design that looks good on both mobile and desktop.

Tasks:

[x] Design the home page layout using Material UI components.
[x] Implement routing to Login and Signup pages.
[x] Test responsiveness on different screen sizes.

Completed


[FEATURE] Debug log in vscode's output tab

something like this:

watch out for the format:
2023-05-31 11:41:45.945 [info] message
possible types (assumed):

info
warning
error
debug (not sure if this is supported by the color coding)

Code sample:
   //Create output channel
    let orange = vscode.window.createOutputChannel("Orange");

    //Write to output.
    orange.appendLine("I am a banana.");

Docs:

vscode.window.createOutputChannel
appendLine

ensure to change the working directory to the current project's root : https://stackoverflow.com/a/64726585/818732
--> keep in mind that the variable vsc returns is likely an array

just in case users have multiple project folders open, you should always match whatever path you get from a context menu execution to then be able to switch the current working directory to the right root folder!
mcdev expected to be executed in the root folder of a project because that's where it will look for the config file .mcdevrc.json and .mcdev-auth.json

as expected, running any context-menu related commands wont bother checking or changing the working directory but simply take whatever terminal is currently open, paste & execute the relevant command. that ends in the following error:


Adds Support for JSON Serialization/Deserialization

ADDITIONS

mixin MetadataKeys for string constants with field names of the Metadata object
instance method toJson() and static method fromJson(..)
getter Metadata.hasAllMetadata

CHANGES

rewrite from "old" abstract class API to the modern mixin API

FIXES

Fixes a Bug in the url getter of TwitterCardParser, JsonLdParser and HtmlMetaParser
Fixes a bug in image getter of JsonLdParser

Thanks for this, once again. I am considering reworking the strategy for the JsonLD Parser. It will cause too much runtime errors as the implementation of the schema isn't as rigid and universal across the web. This is why the image-getter for the JsonLDParser is quite buggy.
What do you think?
(Will checkout this PR in a few, Covid Madness at my work isn't giving me much time nowadays)

I totally agree. Good idea. Making this plugin as robust as possible will bring us lots benefits.

@jg-l Your commit message for the merge says version 3.0.1. I guess it's typo and you meant 0.3.1, though. 🙃

Oops 😂
Yes it's still 0.3.1. On pub.dev as well!


PropTypes no longer supported

Hey, minor error at CircularProgress.js:66 onward.

Warning: PropTypes has been moved to a separate package. Accessing React.PropTypes is no longer supported and will be removed completely in React 16. Use the prop-types package on npm instead. (https://fb.me/migrating-from-react-proptypes)

This is now fixed in the npm package.


Support and document how to configure sharp to fail when loading corrupt images

Currently, sharp will only emit a warning for corrupt images. It'd be great to add an option to sharp to support rejecting the loading of these images.
See https://github.com/lovell/sharp/issues/607#issuecomment-298678935 and https://github.com/lovell/sharp/issues/607#issuecomment-298692269 for background.
(as always, thanks for Sharp! 👍

The property to set in vips is fail:
http://jcupitt.github.io/libvips/API/current/VipsForeignSave.html#VipsForeignLoad--fail
For example:
$ head -c 10000 k2.jpg > x.jpg
$ vips copy x.jpg y.jpg

(vips:30177): VIPS-WARNING **: read gave 2 warnings

(vips:30177): VIPS-WARNING **: VipsJpeg: Premature end of JPEG file
$ vips copy y.jpg z.jpg

So y.jpg is now a valid JPEG file, but with the bottom part set to a mid grey.
If you set fail, vips will bail out on the first warning:
$ vips copy x.jpg[fail] y.jpg

(vips:30579): VIPS-WARNING **: error in tile 0 x 64

(vips:30579): VIPS-WARNING **: error in tile 0 x 72
VipsJpeg: Premature end of JPEG file
VipsJpeg: out of order read at line 64
$ vips copy y.jpg z.jpg
VipsForeignLoad: "y.jpg" is not a known file format

So y.jpg is no longer a valid file.
From C++, fail is a gboolean property, just set it to TRUE like any load option.

I'll take a crack at this now.

I sketched out a PR, would love a review: https://github.com/lovell/sharp/pull/798

OK, I'm missing something here. I picked up the PR a couple days ago, and can't get an invalid image to raise errors with VIPS, either from holes or from simple truncation, from either the command line tool, or by setting "fail".
$ for i in 1 2 4 8 16 32 ; do cp input.jpg punch-$i.jpg ; punch punch-$i.jpg $i ; done
$ for i in *punch* ; do vips copy $i[fail] /tmp/$i  ; done

No warnings are written by vips, and all files written to /tmp are valid JPGs of roughly the same size. This is with freshly-built vips 8.5.8 on linux.

You're right, jpegload doesn't seem to handle fail properly, I'll have a look.
The png loader seems to have the obvious behaviour:
john@kiwi:~/pics$ head -c 10000 k2.png > x.png
john@kiwi:~/pics$ vips copy x.png y.png
(vips:17592): VIPS-WARNING **: Read Error
....
john@kiwi:~/pics$ echo $?
0

So lots of warnings, no error return, and y.png has some pixels in, though not many. With fail set:
john@kiwi:~/pics$ vips copy x.png[fail] y.png

(vips:18274): VIPS-WARNING **: Read Error

(vips:18274): VIPS-WARNING **: error in tile 0 x 0
vipspng: libpng read error
vips2png: unable to write "y.png"
john@kiwi:~/pics$ echo $?
1
john@kiwi:~/pics$ ls -l y.png
-rw-r--r-- 1 john john 54 Aug 28 19:22 y.png

Warnings, though not so many, an error return, and the output is not a valid PNG.

Ah, it's that libjpeg treats a truncated file as a warning, not an error, and libvips fail only applies to errors.
https://github.com/jcupitt/libvips/commit/942b0446a3fb81ef1d8eff7ffdf044514e0c761e#diff-10494e2c915fa105d119105af8a50976
I wonder if this is correct. A truncated file seems rather serious to me, and should perhaps be treated as an error.

@jcupitt thanks for digging. I just added PNG tests to the PR, too, if that helps validation.

With 8.5 head I now see:
john@kiwi:~/pics$ head -c 2000 k2.jpg > x.jpg
john@kiwi:~/pics$ vips copy x.jpg y.jpg
(vips:19972): VIPS-WARNING **: read gave 2 warnings
(vips:19972): VIPS-WARNING **: VipsJpeg: Premature end of JPEG file
john@kiwi:~/pics$ echo $?
0
john@kiwi:~/pics$ vips copy x.jpg[fail] y.jpg
(vips:20028): VIPS-WARNING **: error in tile 0 x 8
VipsJpeg: Premature end of JPEG file
john@kiwi:~/pics$ echo $?
1

I'll add a test to the new libvips test suite as well.
I think you could have got an error from libjpeg with --fail, but it would have been rather difficult.


onStart (ITestListener) is executed one time before the first class in package

TestNG Version
7.10.0
Expected behavior
onStart is executed before all tests for each class in test run
Actual behavior
onStart is executed one time before the first class in test run
Is the issue reproducible on runner?

[ ] Gradle 8.0

Test case sample
Assuming we have 2 classes in one package with tests Test1 and Test2. I've implemented ITestListeners and willing to use onStart, onTestStart methods from there:
class Listener : ITestListener {
    override fun onTestStart(result: ITestResult?) {
        println("onTestStart: ${result?.name}")
    }

    override fun onStart(context: ITestContext?) {
        println("onStart: ${context?.allTestMethods?.first()?.instance?.javaClass?.name}")
    }
}

@Listeners(Listener::class)
class TestClass1 {
    @Test
    fun test1() {

    }

    @Test
    fun test2() {

    }
}

@Listeners(Listener::class)
class TestClass2 {

    @Test
    fun test1() {

    }

    @Test
    fun test2() {

    }
}

Expected output:
onStart: TestClass1
onTestStart: test1
onTestStart: test2
onStart: TestClass2
onTestStart: test1
onTestStart: test2
Actual output:
onStart: TestClass1
onTestStart: test1
onTestStart: test2
onTestStart: test1
onTestStart: test2

Kotlin 1.9.22 (Java 17)
TestNG version 7.10.0 (7.9.0)

Thank you in advance for an answer, i do believe i interprete documentation right... If not, could you, please, suggest how to achieve an expected behavior with listeners

@maksim-kanin - The onStart(ITestContext) is supposed to be executed ONLY once per <test> tag. Since you have included the same listener class on each of the 2 test classes, TestNG evaluates this to only 1 listener and executes its onStart(ITestContext) once.
If you are looking at doing something before a class, you could perhaps try implementing IClassListener. For more details please check https://testng.org/#_listening_to_class_level_invocations

Does that satisfy your ask ? Please let us know

Hi, @krmahadevan Thank you for your help, IClassListener was exactly what i've been looking for!


122 add fig anchor metadata to validations

Closes #122
I created a script that read through the fig_source.html, which I saved from browser, and found each of the elements containing the validation ID for each SBLCheck.  Then I grabbed the href and stored that in the fig_anchor in the phase_validations.py file.
I created a pytest to use the same html to loop through both the phase validation fig_anchors and the hrefs to compare to ensure each validation ID had the correct fig anchor.  I also updated the existing test_cli.py formats to include the new fig_anchor field.
Might be a good idea to store this tool somewhere, it's all local currently.  Just in case they change something, it makes it easy to loop through the py file and insert the fig_anchor instead of manually copy/pasting.
Updated the schema to include the fig_anchor in the check, which automatically carries over into the validation results.
Updated the df_to_json to include the fig_anchor in the json we send to clients.  The other df_to's automatically get the fig_anchor.
I don't think we need another story in the filing-api, the fig_anchor for each result will be in the JSON blob.
I would LOVE to figure out how to automatically pull down the FIG html in an actual usable way for the pytests.  However, because it's all javascripted, the actual hrefs don't come across as full links if you do a request.get, or curl, or wget.  The only way I've found to have both the full href link and the Validation ID associated with it is saving the page off in a browser.

I created a script that read through the fig_source.html, which I saved from browser, and found each of the elements containing the validation ID for each SBLCheck. Then I grabbed the href and stored that in the fig_anchor in the file.
I created a pytest to use the same html to loop through both the phase validation fig_anchors and the hrefs to compare to ensure each validation ID had the correct fig anchor. I also updated the existing test_cli.py formats to include the new fig_anchor field.

🤯 Very cool. 😄

Might be a good idea to store this tool somewhere, it's all local currently.

👍 We could add a utils to this repo.

Updated the schema to include the fig_anchor in the check, which automatically carries over into the validation results. Updated the df_to_json to include the fig_anchor in the json we send to clients. The other df_to's automatically get the fig_anchor.

👍

I don't think we need another story in the filing-api, the fig_anchor for each result will be in the JSON blob.

👍

I would LOVE to figure out how to automatically pull down the FIG html in an actual usable way for the pytests. However, because it's all javascripted, the actual hrefs don't come across as full links if you do a request.get, or curl, or wget. The only way I've found to have both the full href link and the Validation ID associated with it is saving the page off in a browser.

🤔 Maybe something that drives a browser like selenium?

Here's a quick, dirty, barely-tested scraper. Requires beautifulsoup4 and requests.
from bs4 import BeautifulSoup, Tag
import requests

url = "https://www.consumerfinance.gov/data-research/small-business-lending/filing-instructions-guide/2024-guide/"
response = requests.get(url)
soup = BeautifulSoup(response.content, 'html.parser')

divs: list[Tag] = soup.find_all("div", class_="o-fig_section__sub-sub")

id_to_anchor = {}

for div in divs:
    a = div.h4.a
    id = a.get('id')
    text = a.text.strip()

    prefix = "Validation ID: "

    if text.startswith(prefix):
        validation_id = text.lstrip(prefix)
        id_to_anchor[validation_id] = id

from pprint import pprint
pprint(id_to_anchor)

{'E0001': '4.1.1',
 'E0002': '4.1.2',
 'E0020': '4.1.3',
 'E0040': '4.1.4',
 'E0060': '4.1.5',
 'E0080': '4.1.6',
 'E0100': '4.1.7',
 'E0120': '4.1.8',
 'E0121': '4.1.9',
 'E0140': '4.1.10',
...

Here's a quick, dirty, barely-tested scraper. Requires beautifulsoup4 and requests.
from bs4 import BeautifulSoup, Tag
import requests

url = "https://www.consumerfinance.gov/data-research/small-business-lending/filing-instructions-guide/2024-guide/"
response = requests.get(url)
soup = BeautifulSoup(response.content, 'html.parser')

divs: list[Tag] = soup.find_all("div", class_="o-fig_section__sub-sub")

id_to_anchor = {}

for div in divs:
    a = div.h4.a
    id = a.get('id')
    text = a.text.strip()

    prefix = "Validation ID: "

    if text.startswith(prefix):
        validation_id = text.lstrip(prefix)
        id_to_anchor[validation_id] = id

from pprint import pprint
pprint(id_to_anchor)

{'E0001': '4.1.1',
'E0002': '4.1.2',
'E0020': '4.1.3',
'E0040': '4.1.4',
'E0060': '4.1.5',
'E0080': '4.1.6',
'E0100': '4.1.7',
'E0120': '4.1.8',
'E0121': '4.1.9',
'E0140': '4.1.10',
...

The problem with this is it doesn’t provide an actual navigable link, ie a full URL.  I tried this with the request content already and while you can easily get the index (like 4.2.1, etc) I thought the goal was the fig_anchor would be the full https URL path.  If so unfortunately this doesn’t work.  From what I’ve been reading Selenium would be the way to go.  If all we need to do is provide the #4.2.1 and the front end is configured to provide the rest well then yes much easier and a simple request content is doable.

Here's a quick, dirty, barely-tested scraper. Requires beautifulsoup4 and requests.
from bs4 import BeautifulSoup, Tag
import requests

url = "https://www.consumerfinance.gov/data-research/small-business-lending/filing-instructions-guide/2024-guide/"
response = requests.get(url)
soup = BeautifulSoup(response.content, 'html.parser')

divs: list[Tag] = soup.find_all("div", class_="o-fig_section__sub-sub")

id_to_anchor = {}

for div in divs:
    a = div.h4.a
    id = a.get('id')
    text = a.text.strip()

    prefix = "Validation ID: "

    if text.startswith(prefix):
        validation_id = text.lstrip(prefix)
        id_to_anchor[validation_id] = id

from pprint import pprint
pprint(id_to_anchor)

{'E0001': '4.1.1',
'E0002': '4.1.2',
'E0020': '4.1.3',
'E0040': '4.1.4',
'E0060': '4.1.5',
'E0080': '4.1.6',
'E0100': '4.1.7',
'E0120': '4.1.8',
'E0121': '4.1.9',
'E0140': '4.1.10',
...

The problem with this is it doesn’t provide an actual navigable link, ie a full URL. I tried this with the request content already and while you can easily get the index (like 4.2.1, etc) I thought the goal was the fig_anchor would be the full https URL path. If so unfortunately this doesn’t work. From what I’ve been reading Selenium would be the way to go. If all we need to do is provide the #4.2.1 and the front end is configured to provide the rest well then yes much easier and a simple request content is doable.

Actually heck the only piece that changes is that # part so the rest of the url could have just been configured in an env setting and just appending the # stuff at the end.  I was grabbing that in the first 30 minutes of playing around with this lol.  It was trying to get the full actual link that was a pain in the arse.

Sorry, misunderstood the problem.

Actually heck the only piece that changes is that # part so the rest of the url could have just been configured in an env setting and just appending the # stuff at the end. I was grabbing that in the first 30 minutes of playing around with this lol. It was trying to get the full actual link that was a pain in the arse.

Yep. Exactly. 😄

Sorry, misunderstood the problem.

Actually heck the only piece that changes is that # part so the rest of the url could have just been configured in an env setting and just appending the # stuff at the end. I was grabbing that in the first 30 minutes of playing around with this lol. It was trying to get the full actual link that was a pain in the arse.

Yep.   Ok I still have a previous copy of the script that directly used the web page so will be easy to switch back to.  Thanks!


Asynchronously Dispatch Queries via Brokers

Resolves #25.
Allows dispatching queries via brokers in an asynchronous fashion. Dispatch is done on a single thread since transactional database operations are involved. The reason for this is that Spring's transaction propagation has to be respected. Using multiple threads breaks this because transaction information is stored within thread local variables and thus cannot be shared between threads.
The goal for this implementation is that individual brokers can fail without affecting those that ran sucessfully. This gives the UI (or any other client) the possibility to request at least partial results.

Returning a 500 status code has been kept in case of an error. We can still discuss whether a 2xx code with additional error information is better suited in this case. However, this would change the API requiring a new version and client adjustments.


Make MaxBlockSizeBytesPoS part of GlobalParams

#1139  👈
feature/proof-of-stake

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

Join @lazynina and the rest of your teammates on  Graphite

Merge activity

Mar 29, 3:19 PM EDT: @lazynina started a stack merge that includes this pull request via Graphite.


cmake-3 won't install files

Not yet sure of the root cause, but we have a repro where cmake-3.8.0 on CentOS-7 does not install the built files. I * think* this is specific to the cmake version, but OS could also be a factor.

I am using cmake 3.5.2 on os x and i have noticed that when i build hello_world, only the rxt file and the package.py file get installed. Neither the bin/hello or the python/hello_world.py get installed.

Hi, yeah this has been reported before, seems to be some cmake-3 change. I'll look into this soon.
Thx, A
Sent from my iPhone

On 11 Aug 2017, at 9:30 pm, jlgerber<EMAIL_ADDRESS>wrote:
I am using cmake 3.5.2 on os x and i have noticed that when i build hello_world, only the rxt file and the package.py file get installed. Neither the bin/hello or the python/hello_world.py get installed.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or mute the thread.

Might be related to the cmake policy CMP0037. That stops you from overriding the install target. This can be disabled by setting:
cmake_policy(set CMP0037 OLD)
This is a temporary solution and my guess is that tools like Rez should probably use the cmake server functionality since v3.7.
https://cmake.org/cmake/help/v3.7/manual/cmake-server.7.html

@smaragden, unfortunately, adding this command did not fix my problem on os x.

This is what cmake spits out with version 3.9.0
CMake Error at [...]/rez-2.13.1.tbe1/lib/python2.7/site-packages/rez-2.13.1.tbe1-py2.7.egg/rezplugins/build_system/cmake_files/RezProject.cmake:21(add_custom_target):
  The target name "install" is reserved or not valid for certain CMake
  features, such as generator expressions, and may result in undefined
  behavior.
Call Stack (most recent call first):
[...]/rez-2.13.1.tbe1/lib/python2.7/site-packages/rez-2.13.1.tbe1-py2.7.egg/rezplugins/build_system/cmake_files/RezBuild.cmake:95 (rez_project)
  CMakeLists.txt:5 (include)

@smaragden 's suggestion fixes it, you just need to use SET instead of set.
cmake_policy(SET CMP0037 OLD)

I faced the same issue on osx. I figured the culprit to be add_custom_command(install) in RezProject.cmake (removing it fixed the issue for me) -https://github.com/nerdvegas/rez/blob/master/src/rezplugins/build_system/cmake_files/RezProject.cmake#L21
I'll admit I have no knowledge of cmake, so I have no idea what that line does and if I should be removing it at all. Maybe there is an elegant fix for this.

Thanks for that info Venu.
A
On Thu, Dec 21, 2017 at 3:32 AM, Venu Krishnamurthy <
<EMAIL_ADDRESS>wrote:

I faced the same issue on osx. I figured the culprit to be
add_custom_command(install) in RezProject.cmake (removing it fixed the
issue for me) -https://github.com/nerdvegas/rez/blob/master/src/
rezplugins/build_system/cmake_files/RezProject.cmake#L21
I'll admit I have no knowledge of cmake, so I have no idea what that line
does and if I should be removing it at all. Maybe there is an elegant fix
for this.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/nerdvegas/rez/issues/450#issuecomment-353112644, or mute
the thread
https://github.com/notifications/unsubscribe-auth/ABjqSjxoeeJuWn1X2K1n5wDCTn6QbL62ks5tCTaUgaJpZM4OyWtD
.

Closing as I believe this is addressed in https://github.com/nerdvegas/rez/releases/tag/2.17.1, please reopen if this is not the case.


Clean up RedHat 7 Helix queue use in our repos

Though RedHat 7 is EOL, dotnet/arcade and dotnet/aspnetcore continue to test on our redhat.7.amd64 queues in at least some branches. dotnet/runtime was a big user of these queues but has fortunately already moved to test with Docker containers for more current OS versions.

Release Note Category

[x] Feature changes/additions
[ ] Bug fixes
[ ] Internal Infrastructure Improvements

Release Note Description
Removed use of the following Helix queues: redhat.7.amd64, redhat.7.amd64.open, redhat.7.amd64.open.rt, redhat.7.amd64,open.svc, redhat.7.amd64.rt, and redhat.7.amd64.svc in the dotnet/arcade and dotnet/aspnetcore repositories due to the EOL status of the operating system.

branches to update:

[ ] release/6.0 in dotnet/arcade, https://github.com/dotnet/arcade/blob/release/6.0/tests/UnitTests.proj#L66 and https://github.com/dotnet/arcade/blob/release/6.0/tests/UnitTests.proj#L78 in particular
[ ] release/6.0 in dotnet/aspnetcore, https://github.com/dotnet/aspnetcore/blob/release/6.0/eng/targets/Helix.Common.props#L31 in particular

dotnet/aspnetcore#58561 on hold for branches to open again

done


Smart github events to slack notifications

We need following rules for notifications:

If personal message in comments github, notify user in public channel with personal tag
If PR - notification in reviewme channel

Merge PR if 2 major likes

@AndriyBaibak 1 hour spent

@vkuchyn do we actually need "If personal message in comments github, notify user in public channel with personal tag"? We have #github chanell for that.

@romach it would be great - there are lots of noise and mess. But this feature is not high priority. You can create puzzle for future, not to waste much time on it


yrd: init at 0.5.3

Motivation for this change
Added cjdns diagnostic tool.
Things done

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option build-use-sandbox in nix.conf on non-NixOS)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
[x] Fits CONTRIBUTING.md.

@GrahamcOfBorg eval

Success on x86_64-linux (full log)
Attempted: yrd
Partial log (click to expand)
writing entry points to yrd.egg-info/entry_points.txt
reading manifest file 'yrd.egg-info/SOURCES.txt'
writing manifest file 'yrd.egg-info/SOURCES.txt'
running build_ext

----------------------------------------------------------------------
Ran 0 tests in 0.000s

OK
/nix/store/sd4xwkx0v63dp60r915jd478y0ygziga-yrd-0.5.3

Success on aarch64-linux (full log)
Attempted: yrd
Partial log (click to expand)
writing entry points to yrd.egg-info/entry_points.txt
reading manifest file 'yrd.egg-info/SOURCES.txt'
writing manifest file 'yrd.egg-info/SOURCES.txt'
running build_ext

----------------------------------------------------------------------
Ran 0 tests in 0.000s

OK
/nix/store/d1jphwaxcllqg6p6zmrzy11n4cn4cdz1-yrd-0.5.3

Success on x86_64-linux (full log)
Attempted: yrd
Partial log (click to expand)
writing entry points to yrd.egg-info/entry_points.txt
reading manifest file 'yrd.egg-info/SOURCES.txt'
writing manifest file 'yrd.egg-info/SOURCES.txt'
running build_ext

----------------------------------------------------------------------
Ran 0 tests in 0.000s

OK
/nix/store/sd4xwkx0v63dp60r915jd478y0ygziga-yrd-0.5.3

Success on aarch64-linux (full log)
Attempted: yrd
Partial log (click to expand)
writing entry points to yrd.egg-info/entry_points.txt
reading manifest file 'yrd.egg-info/SOURCES.txt'
writing manifest file 'yrd.egg-info/SOURCES.txt'
running build_ext

----------------------------------------------------------------------
Ran 0 tests in 0.000s

OK
/nix/store/d1jphwaxcllqg6p6zmrzy11n4cn4cdz1-yrd-0.5.3

@akru Could you squash your commits?

@nlewo Yes, can I open another PR?

No, you basically have to rebase -i your branch and push --force. Something such as https://stackoverflow.com/questions/35703556/what-is-squashing-commits-in-git.

Success on x86_64-linux (full log)
Attempted: yrd
Partial log (click to expand)
/nix/store/sd4xwkx0v63dp60r915jd478y0ygziga-yrd-0.5.3

Success on aarch64-linux (full log)
Attempted: yrd
Partial log (click to expand)
writing entry points to yrd.egg-info/entry_points.txt
reading manifest file 'yrd.egg-info/SOURCES.txt'
writing manifest file 'yrd.egg-info/SOURCES.txt'
running build_ext

----------------------------------------------------------------------
Ran 0 tests in 0.000s

OK
/nix/store/b9f13vcmgilsx8qyx7452mpd0dhgg8ps-yrd-0.5.3


Add a solution to problem 1376

This problem can be solved by tree traversal. The problem statement guarantees that the subordination relationships have a tree structure. The idea is to first build the tree structure of the organization and then perform a traversal on it. During the traversal, we can compute the time that each employee is informed and return the latest time.

Close this PR since the whole solution has been revised with a different idea.


Operator should create Kafka topics automatically if possible

This issue has been resolved, and should not affect the upcoming 1.0.0-v2.0.0.final release.


Frrist/fx config instance

Summary by CodeRabbit

New Features

Introduced new functions for managing compute nodes, including Docker and WebAssembly support.
Added new endpoints and functionalities for node information and job management.
Enhanced content publishing with support for various platforms like IPFS, S3, and local storage.
Introduced a new router with middleware for handling HTTP requests efficiently.

Refactor

Significant restructuring of node management, server setup, and network configuration to enhance performance and scalability.
Updated and simplified the requester node functionalities, improving event handling and job scheduling.

Bug Fixes

Updated error handling in node ranking and storage configurations to prevent potential issues.

Documentation

Improved inline documentation and comments for future refactoring and understanding of the codebase.

coderabbitai review

@coderabbitai review

This was a helpful spike. Closing in favor of https://github.com/bacalhau-project/bacalhau/pull/3959 which extracted a subset of the work from this relating to configuration. A second PR will follow which will include the dependency injection via fx as was sketched out here.


Program is not inheriting from Node but from AST

Currently we have :
class Program : public AST {
private:

but rest nodes inherit from AST->Node->XX. Check why this was done and whether Program can inherit from node as well.

After some investigation I figured out that ast::Program is created by the all rule of the parser. This means that in the beginning of the parsing of a file there is an ast::Prorgam created. Then every top level block is added to the ast::Program in the public member NodeVector blocks. My guess is that it doesn't inherit from Node to distinguish it from other types of Nodes due to its special nature.
After making Program inherit from Node by editing nmodl.yaml there were no issues with the included tests of NMODL or using NMODL to build CoreNeuron.