[⚡ Feature]: TurboRepo + NextJS deployed to Cloudflare Pages

Description
Hey
Is there a way to deploy a nextjs app within a monorepo to Cloudflare Pages?
I tried multiple build settings but i cannot get it to work?
Should i give up or these i a possibility for this tot work?
Additional Information
No response
Would you like to help?

[ ] Would you like to help implement this feature?

Hey there,
Have you tried using a script in a package.json file to start building your application? For instance, in the Next.js app in your monorepo, you could have a script called pages:build and it could run npx @cloudflare/next-on-pages. Then, your build command on Cloudflare Pages could be something similar to pnpm --filter @apps/web pages:build, running the script in that app in your monorepo.
This strategy should build your application in the correct location. You would just need to update your output directory in your Pages project's build settings to reflect the location where your app is, e.g. /apps/web/.vercel/output/static.

@james-elicx thank you ... it worked after a few minor fixes in settings
unfortunately:
14:20:08.450 | ✘ [ERROR] Error: Pages only supports files up to 26.2 MB in size
-- | --
14:20:08.450 |
14:20:08.451 | cache/webpack/edge-server-production/0.pack.gz is 461 MB in size

too bad

Hello, it sounds like you might be trying to upload your .next folder contents.
Your output directory should be {path to app}/.vercel/output/static and it shouldn't be referring to .next at all, or trying to upload .next/cache.

@james-elicx u were again right ... but now
15:05:28.588 | Success: Assets published!
-- | --
15:05:30.872 | Error: Failed to publish your Function. Got error: Your Functions script is over the 1 MiB size limit (workers.api.error.script_too_large)

my problem is that i do not have a _worker.js file ... maybe because the build failed

are these the ones?

just deployed this to vercel and checkout all the edge functions .... none of them is above 1MB but cumulated the size for sure is above 5MB

are these the ones?

No, those are assets, not the _worker.js directory. You have to build it locally if you want to zip it and calculate the size of the worker directory to understand how much above your limit it is.

just deployed this to vercel and checkout all the edge functions .... none of them is above 1MB but cumulated the size for sure is above 5MB

What Vercel deploys is very different to what we deploy. We try to minify functions as much as we can and also combine them into one main worker.

No, those are assets, not the _worker.js directory. You have to build it locally if you want to zip it and calculate the size of the worker directory to understand how much above your limit it is.

just deployed this to vercel and checkout all the edge functions .... none of them is above 1MB but cumulated the size for sure is above 5MB

What Vercel deploys is very different to what we deploy. We try to minify functions as much as we can and also combine them into one main worker. The size Vercel tells you is not really related to the size we deploy to Pages.

@james-elicx are you specifically talking about the _worker.js directory in this comment? I'm trying to figure out exactly how to test bundle sizes locally.
It sounds like what Cloudflare validates sizes against is a zipped version of the /.vercel/output/static/_worker.js directory, is that correct?

@james-elicx are you specifically talking about the _worker.js directory in this comment? I'm trying to figure out exactly how to test bundle sizes locally.
It sounds like what Cloudflare validates sizes against is a zipped version of the /.vercel/output/static/_worker.js directory, is that correct?

If you zip the _worker.js directory, you should be able to get an idea of the zipped bundle size that is uploaded to Cloudflare, yes.

hey @james-elicx  do you know anyway on how I can deploy my hono app from my turborepo


Feature request: pass a command to render ssh as a flag

I'd love it if I could pass a flag to render ssh that runs the command as soon as it connects. It's really common to boot up an interactive REPL like the Ruby on Rails console.
So, for example:
render ssh srv-abc123 --command "bundle exec rails console"
render ssh -c srv-abc123 "bundle exec rails console"

In this commit in release 1.1.0 we allow you to pass args on to ssh, psql, pgcli, and redis-cli. You'll just need to do -- before any args you want to be passed on. For example:
render ssh <service-id> -- echo 'hello world'

Going to close this, but feel free to reopen the issue if this doesn't address your needs.

@mdbenjam thank you for adding this!
I can confirm I'm able to boot an interactive console with render ssh srv-abc123 -- "cd /rails; bin/rails c"
But I'm not seeing the normal output from the command that I'd expect:
Expected
This is how it looks when I do render ssh src-abc123 and then when the shell starts, I run: cd /rails; bin/rails c:

Actual

So it seems like there's something off with the way the connection is working or something. I'll keep experimenting but figured I'd share an early report on how it's behaving.

Could you try passing in -t. When you pass a command to ssh it doesn't allocate a pseudo-terminal. -t forces this and I think should get back your formatting.

Thank yo so much, that does it! Running render ssh srv-abc123 -- -t "cd /rails; bin/rails c" accomplishes exactly what I was hoping for. 🎉


Exclude portalocker_tests from installed packages

Coverage remained the same at 100.0% when pulling 1e33ac812d5cff732cc75d241f3c3e2bea86383f on ignatenkobrain:patch-3 into 19193c2425c48d6815ade9f3ada516a4bb39a49a on WoLpH:develop.

Good one, that's an improvement on your earlier pull request indeed :)


Reduce Idle Connections from Health Checks [#1102]

Purpose
The health check endpoints are opening up too many connections against the fidesops application database.  Shoutout to @RobertKeyser and @sanders41 for spotting.
Changes

Stop creating a new engine as part of the process of running a health check.
Create an engine to be used across the application, and pass this into get_db_session when we go to get a single connection, instead of creating a new engine. (Note we are currently using the default pool_size and max_overflow parameters. This is something that may need to be configurable in the future).

Note that our celery processes intentionally create their own separate engines for managing connections for running privacy requests.

Background

The typical usage of create_engine() is once per particular database URL, held globally for the lifetime of a single application process. A single Engine manages many individual DBAPI connections on behalf of the process and is intended to be called upon in a concurrent fashion. The Engine is not synonymous to the DBAPI connect function, which represents just one connection resource - the Engine is most efficient when created just once at the module level of an application, not per-object or per-function call.

Checklist

[x] Update CHANGELOG.md file

[ ] Merge in main so the most recent CHANGELOG.md file is being appended to
[ ] Add description within the Unreleased section in an appropriate category. Add a new category from the list at the top of the file if the needed one isn't already there.
[ ] Add a link to this PR at the end of the description with the PR number as the text. example: #1

[ ] Applicable documentation updated (guides, quickstart, postman collections, tutorial, fidesdemo, database diagram.
If docs updated (select one):

[ ] documentation complete, or draft/outline provided (tag docs-team to complete/review on this branch)
[ ] documentation issue created (tag docs-team to complete issue separately)

[ ] Good unit test/integration test coverage
[ ] This PR contains a DB migration. If checked, the reviewer should confirm with the author that the down_revision correctly references the previous migration before merging
[ ] The Run Unsafe PR Checks label has been applied, and checks have passed, if this PR touches any external services

Ticket
Fixes #1102

thanks for merging @sanders41 ⭐


Anyone still using this?

This is one of the few golang routers i see that can be used for vercel platform. Anyone using this to verify it's still working, since the last update was 2 years ago.
The codes all looks alright though.

https://github.com/TBXark/g4vercel/blob/b104f2daa04275ae982d74f6913ad82665dfef53/gee.go#L82-L97
You can use any router library for go and use it by adding an adaper
func Handle(w http.ResponseWriter, req *http.Request)

This library is relatively simple and lightweight, and no bugs have been found so far.

thank u. have u tried gofiber?

gofiber is too complicated for vercel


Bulk Trust Extensions

Describe your suggested feature
I install most of the extensions from the default repo. This is great for extension diversity, that is until that time of the year when half or even all of the extensions need an update. I understand that trust should definitely not be a single button-press for everything, but at least have a repo trust system.
Other details
I thought I saw something similar to this request, but when I tried searching issues I got nothing. Maybe it was in an old Tachyomi issue or a reddit thread.
Acknowledgements

[X] I have searched the existing issues and this is a new ticket, NOT a duplicate or related to another open or closed issue.
[X] I have written a short but informative title.
[X] I have updated the app to version 0.17.0.
[X] I will fill out all of the requested information in this form.

 Mihon doesn't associate or provide any repos, much less have a default one.
And there is already is trust by repo. If all your extensions were installed from a specific repo and that repository is also added to your extension repos list they are automatically trusted. (If it's not visually updating restart the app)

Well he check the box saying he was on 0.17.0
Also duplicates of #320 #884 #487

My bad with the repo part, I probably just forgot exactly where it came from. I've used it since I've started using this app.
Either way, the automatic trust feature is not functioning on my Samsung Galaxy Note 9

That is unless there is a toggle for that that I haven't found yet hidden in the settings

Remove the repo from settings and add it back. Restart the app afterwards. If it doesn't work complain to whoever owns the repo


Configuration merging for app configuration

As ...
P.D. - Plugin Developer
I want ...
the default app configuration settings to be deeply merged with user-supplied app configuration settings
So that ...
default settings are not lost when providing user-supplied settings
I know this is done when...

I create a Nautobot App with default settings for dictionaries

default_settings = {
  "setting": {"nested_setting": True}
}

I install that App and locally configure it

PLUGINS_CONFIG = {
  "this_example_app": {
    "setting": {"other_setting": False}
  }
}

The nested_setting key does not disappear from the configuration

Optional - Feature groups this request pertains to.

[ ] Automation
[ ] Circuits
[ ] DCIM
[ ] IPAM
[ ] Misc (including Data Sources)
[ ] Organization
[X] Plugins (and other Extensibility)
[ ] Security (Secrets, etc)
[ ] Image Management
[ ] UI/UX
[ ] Documentation
[ ] Other (not directly a platform feature)

Database Changes
None
External Dependencies
Possibly something that deep-merges dictionaries.

A separate solution could be explicitly discouraging nested configuration, but since we already make heavy use of this in the ecosystem that doesn't seem to be the correct way for me.


Fix counting in GitHub workflow

The GitHub workflow test-maven-setup-action.yaml has a step that counts the occurrences  of what is expected to dynamically appear in the settings.xml (link👇)
https://github.com/navikt/bidrag-test-github-actions/blob/1b0d62e4a361dc72b3b564b035e7a8d10c0faedf/.github/workflows/test-maven-setup-action.yaml#L30
This counting only counts 1 but will not reveal the actual number of occurrences. The file is logged and it appears to be as expected, but I cannot verify using grep -c... I have been using grep many times before, and cannot see why it doesn't work.
Help finding the reason of this is highly appreciated...

corrected numbers should be
          echo check number of repos
          if [[ $COUNT_REPOS -ne 2 ]]; then
          ...
          echo check number of maven-releases
          if [[ $COUNT_NAME_MAVEN -ne 3 ]]; then
          ...
          echo check number of snapshots
          if [[ $COUNT_NAME_SNAP -ne 3 ]]; then
          ...
          echo check number of github actor
          if [[ $COUNT_ACTOR -ne 2 ]]; then
          ...
          echo check number of github tokens
          if [[ $COUNT_TOKEN -ne 2 ]]; then
          ...


Data Explorer Accessiblity

For #6019

[X] Pull request represents a single change (i.e. not fixing disparate/unrelated things in a single PR)
[X] Title summarizes what is changing
[X] Has a news entry file (remember to thank yourself!)
[X] Appropriate comments and documentation strings in the code
[ ] Has sufficient logging.
[ ] Has telemetry for enhancements.
[ ] Unit tests & system/integration tests are added/updated
[ ] Test plan is updated as appropriate
[ ] package-lock.json has been regenerated by running npm install (if dependencies have changed)
[ ] The wiki is updated with any design decisions/details.

Sorry phantom commits are back for some reason


Implement conflict_target for ON CONFLICT clause in INSERT

The conflict_target specifies the columns which are part of a primary key
constraint. According to the PostgreSQL INSERT syntax, specifying the
conflict_target is mandatory.
https://www.postgresql.org/docs/9.5/static/sql-insert.html

@mfussenegger Fixed the object column conflict target.


Comments need responsive design

Currently the comments look very ugly and are a bug on mobile.

As per 8d330e96a834a9824b32795e4490a02049397e01
Comments now readable in mobile and scrollable when too large.
Page scrolls to word where possible when clicked.


Fix intergrated tests

// Describe your PR here; eg. Fixes #issueNo
After PR#1690, the field custom_init_script was removed. So we need to change to use init_script and use prefix file: to support init scripts .

Codecov Report

Merging #1691 (d735324) into master (d64801f) will decrease coverage by 0.52%.
The diff coverage is n/a.

@@             Coverage Diff              @@
##             master    #1691      +/-   ##
============================================
- Coverage     60.54%   60.01%   -0.53%
+ Complexity     2273     2263      -10
============================================
  Files           477      477
  Lines         10109    10109
  Branches       1030     1030
============================================
- Hits           6120     6067      -53
- Misses         3481     3541      +60
+ Partials        508      501       -7

Impacted Files
Coverage Δ

...enyu/client/tars/TarsServiceBeanPostProcessor.java
25.67% <0.00%> (-32.44%)
:arrow_down:

...henyu/plugin/grpc/resolver/ShenyuNameResolver.java
45.26% <0.00%> (-17.90%)
:arrow_down:

...che/shenyu/sync/data/http/HttpSyncDataService.java
63.10% <0.00%> (-8.74%)
:arrow_down:

...henyu/plugin/httpclient/NettyHttpClientPlugin.java
57.14% <0.00%> (-2.05%)
:arrow_down:

...elimiter/handler/RateLimiterPluginDataHandler.java
69.23% <0.00%> (-1.54%)
:arrow_down:

.../plugin/grpc/loadbalance/AbstractLoadBalancer.java
68.47% <0.00%> (-1.09%)
:arrow_down:

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 d64801f...d735324. Read the comment docs.


Documentation: Homepage: fix example script

Updated the homepage example with one that will work given the changes on https://angularjs.org

Thanks! Merged as e89db5976397101c089b76536d286209d601763a


[PAPI-292] Remove Guest JWT Requirement

Removes legacy auth endpoint
Removes Authorize attribute from public endpoints
Wallet JWT have a 24h expiration time (up from 1h)
Adds auth to refresh token balance endpoint
Removes Deploy controller

cc @drmathias merging this now, you can review and comment and I'll follow up with another PR with anything.
Tried to work alongside this but its frustrating between other branches and UI work w/ conflicts etc.


chore: upgrade TSLint to v5

Resolved in https://github.com/biosustain/potion-node/commit/d2c5cb3c571c4039e44180a13e34e6e1af02e09c.


#1873 home page - separate tabs for login and register.

This is a request that simply demands separate tabs for login and register on the home page.

Should I did do it again and send a pull request?
On 18 Mar 2017 7:14 p.m., "Steve Gordon"<EMAIL_ADDRESS>wrote:

@stevejgordon requested changes on this pull request.
@babagy https://github.com/babagy Seems that some of the commit is
missing after the git work you did? The change doesn't seem to be included
any longer.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/HTBox/allReady/pull/1920#pullrequestreview-27731822,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AZPFlDNBeEWnqUopzSb_UrNBSBp6A4iIks5rnC0GgaJpZM4MhbjJ
.

@babagy There are a couple of I think unrelated commits in here. If you're able to back those out and recommit the work you did, you can force push the branch to update this PR. If you think it's easier to close the PR though we can start from a fresh one. Let me know if you need a hand at all and thanks for your patience on the Git side of things.

Please Steve, close the PR an I wish pull the current version and work
afresh then submit a new PR.
On 18 Mar 2017 7:43 p.m., "Steve Gordon"<EMAIL_ADDRESS>wrote:

@babagy https://github.com/babagy There are a couple of I think
unrelated commits in here. If you're able to back those out and recommit
the work you did, you can force push the branch to update this PR. If you
think it's easier to close the PR though we can start from a fresh one. Let
me know if you need a hand at all and thanks for your patience on the Git
side of things.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/HTBox/allReady/pull/1920#issuecomment-287569814, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AZPFlIgAA3KNZ2rFQhk89AlzCbFiYuI6ks5rnDPmgaJpZM4MhbjJ
.

Ok, thanks @babagy


Update component_youtube.tpl

1. Why is this change necessary?
label doesn't match function
modestbranding has to be 0, to disable branding, as the label says.
2. What does this change do, exactly?
it will hide the branding
3. Describe each step to reproduce the issue or behaviour.
4. Please link to the relevant issues (if any).
5. Which documentation changes (if any) need to be made because of this PR?
6. Checklist

[ ] I have written tests and verified that they fail without my change
[X] I have squashed any insignificant commits
[ ] This change has comments for package types, values, functions, and non-obvious lines of code
[X] I have read the contribution requirements and fulfil them.

missunderstood youtube-docu. Sorry.


Improved prompt to reply in the same language

I modified the prompt (English only for now) so that it generates the reply in the same language as the original email. Previously, the answers were always generated in English even though I receive emails in multiple languages.
(Good job for this extension by the way and thank you for open sourcing it.)

Simple and useful change. I will try to update the package on the store later today.
Thank you for your contribution !


Version 0.0.6

Do we need make a new binary release with 0.0.6 tag too? because it is still 0.0.5 on releases

if you don't mind :)


Community plugin: Fixed typo; corrected handle (github not twitter)

Fixed a typo and changed to github handle for my community plugin, eleventy-plugin-broken-links.

Thanks, @pdehaan --- appreciate your help. It's appearing correctly now.


WIP: Improve UI, some a11y

Shall we do fonts the same way we do them on the infrastructure site?
<!--
--
  | GET FONTS AND GET THEM FAST.
  | FROM: https://csswizardry.com/2020/05/the-fastest-google-fonts/
  | - 1. Preemptively warm up the fonts’ origin.
  | -
  | - 2. Initiate a high-priority, asynchronous fetch for the CSS file. Works in
  | -    most modern browsers.
  | -
  | - 3. Initiate a low-priority, asynchronous fetch that gets applied to the page
  | -    only after it’s arrived. Works in all browsers with JavaScript enabled.
  | -
  | - 4. In the unlikely event that a visitor has intentionally disabled
  | -    JavaScript, fall back to the original method. The good news is that,
  | -    although this is a render-blocking request, it can still make use of the
  | -    preconnect which makes it marginally faster than the default.
  | -->
  |
  | <!-- [1] -->
  | <link rel="preconnect"
  | href="https://fonts.gstatic.com"
  | crossorigin />
  |
  | <!-- [2] -->
  | <link rel="preload"
  | as="style"
  | href="https://fonts.googleapis.com/css2?family=PT+Serif:ital,wght@0,400;0,700;1,400;1,700&family=Roboto+Mono:ital,wght@0,400;1,400&family=Roboto:ital,wght@0,300;0,400;0,700;1,300;1,400;1,700&display=swap" />
  | <!-- [3] -->
  | <link rel="stylesheet"
  | href="https://fonts.googleapis.com/css2?family=PT+Serif:ital,wght@0,400;0,700;1,400;1,700&family=Roboto+Mono:ital,wght@0,400;1,400&family=Roboto:ital,wght@0,300;0,400;0,700;1,300;1,400;1,700&display=swap"
  | media="print" onload="this.media='all'" />
  |
  | <!-- [4] -->
  | <noscript>
  | <link rel="stylesheet"
  | href="https://fonts.googleapis.com/css2?family=PT+Serif:ital,wght@0,400;0,700;1,400;1,700&family=Roboto+Mono:ital,wght@0,400;1,400&family=Roboto:ital,wght@0,300;0,400;0,700;1,300;1,400;1,700&display=swap" />
  | </noscript>

Hmm well current, all additional JS + CSS are injected in the  via the Ghost UI (the CSS on this PR are redundant) -- do you think we should modify the templates or avoid doing that in case of a major update to our theme? (I think the latter might make sense, even if we did modify the templates slightly to remove the authors)

Not sure. I defer to you on what makes sense, you know the code and are much more experienced than me.
Here's some info you might need from me to make the call on what makes sense...
It looks Casper is updated almost monthly with minor updates (https://github.com/TryGhost/Casper/releases?after=3.0.6) and the 2-->3 changed happened this time last year (so, not expecting it to happen soon). The most recent update was 2 days ago (sigh). So, I'm expecting we might need to make some minor updates each month (or, at least check in each month to see if an update is worth making).
If it took ~1 hour a month to keep this aligned and very performant (aka make updates we think are good) I'd be pretty happy (after, obviously this initial period of learning / investment and one or two minor updates where we may need to find a bit of a rhythm). Hell, if some of our updates are just 'good for all' maybe we could even submit a PR to them and see what happens?

Couple of things.
Currently in default.hbs there is code to get the inter fonts. It's also in the code injection. Is that intended, seems to have in pages twice so I guess not.
Also, could we just include https://github.com/OAButton/ghost-blog-theme/pull/1#issuecomment-711173906 in that code injection? Since it's injected into the head it seems like it might have the same affect as putting it in the code directly. I hope so, as that's where analytics is 🙃

FYI I have added some more custom CSS to adjust colors on the homepage / posts so they pass accessibility tests and don't affect styling too greatly (in my opinion). Can reverse if it looks bad, but I don't think it does.

I've signed up here: https://gscan.ghost.org/?s=true. It may be worth you doing the same.

FYI I have added some more custom CSS to adjust colors on the homepage / posts so they pass accessibility tests and don't affect styling too greatly (in my opinion). Can reverse if it looks bad, but I don't think it does.

No problem; wondering if you’re using the CSS variables?
Also, I’ve been doing the styling for dark mode, I’m assuming your computer is on light mode right?

I did use the variable :)
It is in light mode, yes. I had a feeling dark mode may be an issue but figured it wouldn't for at least… a few hours ;)
Joseph McArthur
Assistant Director: Right to Research Coalition ( http://righttoresearch.org )
Director / Founder: Open Access Button ( http://openaccessbutton.org )
Twitter: @Mcarthur_Joe ( https://twitter.com/Mcarthur_Joe )
Skype: joseph_mcarthur
On Tue, Oct 20, 2020 at 2:36 PM, Sophy <<EMAIL_ADDRESS>> wrote:

FYI I have added some more custom CSS to adjust colors on the homepage /
posts so they pass accessibility tests and don't affect styling too
greatly (in my opinion). Can reverse if it looks bad, but I don't think it
does.

No problem; wondering if you’re using the CSS variables?
Also, I’ve been doing the styling for dark mode, I’m assuming your
computer is on light mode right?
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub (
https://github.com/OAButton/ghost-blog-theme/pull/1#issuecomment-712855362
) , or unsubscribe (
https://github.com/notifications/unsubscribe-auth/ABFZMZ37DDKYYZCBIER7MKDSLWG7PANCNFSM4SRG64ZA
).

Custom stuff for dark mode is good now too!

Cleaned up the header injection & re-uploaded the compressed theme files with the faster font loading script.

Cleaned up the header injection & re-uploaded the compressed theme files with the faster font loading script.


Generic package size limited to 32MiB

Description
If you try to upload a package that is larger than 32MiB to the generic package registry, the upload appears to succeed, and the package can be seen in the web ui, but downloading the package later gives a 0 byte file.
I suspect this is due to the hard-coded buffer size on this line.
Looking at references to CreateHashedBufferFromReader, the 32MiB limit also seems to apply to the other package registries as well.
Gitea Version
1.17.0
Can you reproduce the bug on the Gitea demo site?
I did not receive the confirmation email after registering, so I was unable to attempt it.
Log Gist
No response
Screenshots
No response
Git Version
No response
Operating System
No response
How are you running Gitea?
Reproduced on docker and Windows 10.
Database
SQLite

This has been fixed by #20635


MonadPlus instance for Input

Should be trivial, since Input already has instances for both Monad and Alternative.

Could you write this up into a pull request?

Sure!


Question about git tag policy and build automation

Hi --
Travis and Vagrant files for Fabric currrently have references to a "version".
https://github.com/hyperledger/fabric/blob/master/devenv/Vagrantfile#L33
https://github.com/hyperledger/fabric/blob/master/.travis.yml#L37
These are hard-coded.  Is there a way to make those variables that are based on the current git tag, so that build artifacts (currently we have a packer-based build tool-chain) can be automatically tied back to their specific Fabric tag/origin?
For example:  IBM/Fabric maintainers would create a new tag, and the new tag would trigger automatic rebuilds (well, at least dockerhub supports this).
@hyperledger/fabric-maintainers @ghaskins

+1 to making this a function of a git tag

+1

we've adopted a versioning and labeling strategy based on semver. closing this


Load the SVSM kernel from an ELF file

Implement a minimal ELF file reader
Make the Stage2 to read the SVSM kernel from an ELF (instead of raw binary) and load it according to the layout described therein.
Let the SVSM kernel determine its own memory layout to transfer over to the runtime page tables from the ELF metadata as well.
Implement address space randomization in Stage2.

Future plans include

to load the symbol table from the ELF as well and to e.g. pretty-print stacktraces and
to use the unwinding records from the ELF for unwinding, alleviating the need for the current framepointer based approach.

I really like these changes, but want to go this way in somewhat smaller steps.
The reason is that the ASLR changes drop the assumption that the the virtual base is aligned to a 2M boundary, which will hurt us down the road. So here are the steps I propose:

Submit only the ELF loader changes, they make the bulk of this PR anyway
Then we separate the kernel text+data+bss mapping from the SVSM memory mapping, best into a separate top-level page-table entry. Since the kernel image is still in the SVSM memory mapping, make sure it is mapped RO or even unmapped there.
Implement ASLR based on this change, which allows to have more bits of randomness. Even the top-level page-table index can be random within certain bounds.

I'm closing this PR, a new one with the first ELF loader part only will follow shortly.


SSE Streams only stay open for 1 minute

As far as I can tell, SSE streams from the public stellar horizon servers close (with an EOF) after 1 minute. If you run a bridge connected to these servers you will see Started listening for new payments  logged every minute, which really pollutes server logs!

It's happening because horizon.stellar.org is behind Elastic Load Balancer which kills idle connections after one minute. You can deploy your own horizon server and it shouldn't be happening.

Could we implement a heartbeat?

@bartekn see https://aws.amazon.com/blogs/aws/elb-idle-timeout-control/
I would recommend to set it to maximum - 3600 seconds.
One possible downside of a short timeout - with request retry implementation that does not take into account the cursor (last_id), the code can retry each minute with the same cursor, and read the same data over and over.

OK, let's discuss here: https://github.com/stellar/go/issues/376


Wrap error for failing ImageSize calls

This should increase the debug-ability when encountering this kind of issue.

LGTM

/approve

LGTM
On Tue, Feb 4, 2020, 08:44 OpenShift CI Robot<EMAIL_ADDRESS>wrote:

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: rhatdan
https://github.com/containers/libpod/pull/5079#issuecomment-581915326, saschagrunert
https://github.com/containers/libpod/pull/5079#
The full list of commands accepted by this bot can be found here
https://go.k8s.io/bot-commands?repo=containers%2Flibpod.
The pull request process is described here
https://git.k8s.io/community/contributors/guide/owners.md#the-code-review-process
Needs approval from an approver in each of these files:

OWNERS https://github.com/containers/libpod/blob/master/OWNERS
[rhatdan]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/containers/libpod/pull/5079?email_source=notifications&email_token=AB3AOCG4KOO7BZD2G55Z6WDRBFWKBA5CNFSM4KPW3JO2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEKXVGNY#issuecomment-581915447,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AB3AOCDOJJULBQK7U3JYNELRBFWKBANCNFSM4KPW3JOQ
.

/lgtm
/hold
hold cancel, once tests pass.

LGTM

/hold cancel


[SPARK-37310][SQL] Migrate ALTER NAMESPACE ... SET PROPERTIES to use V2 command by default

What changes were proposed in this pull request?

This PR proposes to use V2 commands as default as outlined in SPARK-36588, and this PR migrates ALTER NAMESPACE ... SET PROPERTIES to use v2 command by default.
Note that the work to tests covering both v1/v2 were done in #34842.
Why are the changes needed?

It's been a while since we introduced the v2 commands,  and it seems reasonable to use v2 commands by default even for the session catalog, with a legacy config to fall back to the v1 commands.
Does this PR introduce any user-facing change?

No
How was this patch tested?

Existing *AlterNamespaceSetPropertiesSuite tests cover this PR's change.

Test build #146163 has started for PR 34891 at commit fb6e7d3.

cc @cloud-fan

Kubernetes integration test starting
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/50636/

Kubernetes integration test status failure
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/50636/

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/50636/

Test build #146163 has finished for PR 34891 at commit fb6e7d3.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged to master. Thank you @imback82 !

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/146163/


Implemented a new 'bind-loop' mount type for shared private storage b…

…etween parent and child apps
Description
This feature allows shared private storage for apps. This involves creating one loop device for a data.img file and bind mounting that into the containers of the different apps.
Test Procedure

Make the Dobby daemon and install it on the vagrant development VM:
$ cmake -DCMAKE_BUILD_TYPE=Debug -DRDK_PLATFORM=DEV_VM -DLEGACY_COMPONENTS=ON -DCMAKE_INSTALL_PREFIX:PATH=/usr ../
$ sudo make install
Run the Dobby daemon in one terminal:
$ sudo /usr/sbin/DobbyDaemon --nofork
Create a test config.json. Example:
{ "version": "1.1", "args": [   "/bin/sleep", "60" ], "cwd": "/home", "console": {   "limit": 1048576,   "path": "/tmp/console.log" }, "env": [ ], "etc": {   "hosts": [     "<IP_ADDRESS>\tlocalhost"   ] }, "memLimit": 498073600, "network": "open", "rdkPlugins": { }, "restartOnCrash": false, "user": {   "gid": 30000,   "uid": 30015 }, "mounts": [         {            "destination": "/home/private",            "fstype": "ext4",            "options": ["nosuid", "nodev", "noexec" ],            "source": "/home/vagrant/sleepy_test/data.img",            "type": "bind-loop"       }  ] }
Launch an app using this config.json:
$ /usr/bin/DobbyTool start sleepy1 config.json
Wait 20-30s and launch another app:
$ /usr/bin/DobbyTool start sleepy2 config.json
Check only one loop device is created for both apps:
$ losetup -a
Repeat the last step to check the device loop is released only after the second app has finished running.

Type of Change

[ ] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] Other (doesn't fit into the above categories - e.g. documentation updates)

Requires Bitbake Recipe changes?

[ ] The base Bitbake recipe (meta-rdk-ext/recipes-containers/dobby/dobby.bb) must be modified to support the changes in this PR (beyond updating SRC_REV)

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.jignatius seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.


Add support for virtual host specific cookies

Session ID's for an authenticated user are shared between virtual hosts.  If a user authenticates against and receives a cookie for a virtual domain, that cookie can be replayed, and will work against any virtual host that uses the same session store.  It would improve security if the sessions are specific to the virtual host they are assigned on.  A couple of suggestions would be to either include the virtual host, on the back-end, before the session ID is written, so a session ID will not work across virtual hosts.  Another option would be to include a configurable session prefix that gets pre-pended to the session ID before it is written.  The first (prepending the virtual host) might actually be preferrable as it would allow multiple virtual hosts to be serviced out of a single config file (less apache memory used) as in
ServerAlias *.foo.com
Then cookie 123 delivered to client, if assigned on www1.foo.com would be written on the back-end as www1.foo.com-123, but set the cookie 123 on the client.  A cookied assigned on www2.foo.com would be written with a back-end session id of www2.foo.com-123 and also written on the client as 123.

yep, instead of segmenting the shared cache, cookies being replayed at a domain they weren't issued for are now blocked in 265edf58343bd8110bbc477387cb6b3b813377fe ; let me how that works

Roger that.  I'll get the latest version downloaded and run through my
cookie substitution test then post back with my results shortly.
On 10/12/2015 09:51 AM, Hans Zandbelt wrote:

yep, instead of segmenting the shared cache, cookies being replayed at
a domain they weren't issued for are now blocked in 265edf5
https://github.com/pingidentity/mod_auth_openidc/commit/265edf58343bd8110bbc477387cb6b3b813377fe
; let me how that works
—
Reply to this email directly or view it on GitHub
https://github.com/pingidentity/mod_auth_openidc/issues/98#issuecomment-147459182.

I ran through the same tests as before.

log in to foo.mydomian.com
Using cookie editor add a cookie with the values from the foo.mydomain.com domain for bar.mydomain.com.
Open the page at bar.mydomain.com, instead of allowing me access it now re-directs me to my loging discovery page.  Closing this out as fixed (if that's OK).

Ok, thanks for the quick follow-up!

Right back atcha'
On 10/12/2015 01:47 PM, Hans Zandbelt wrote:

Ok, thanks for the quick follow-up!

Reply to this email directly or view it on GitHub:
https://github.com/pingidentity/mod_auth_openidc/issues/98#issuecomment-147515310


setup: bump cryptography dependency to 1.1.2

Hopefully fix #643

Coverage decreased (-0.2%) to 81.714% when pulling 412f742250a5597d02f2d40368557c82d8656c23 on xrmx:upcrypto into 55c549d86f6c6ba6a207cee17a5595783945c1b9 on airbnb:master.

https://github.com/pyca/cryptography/issues/2503

Latest is 1.4, I'll send another PR bumping up more libs.


Shrink down circle image

Can we shrink down the circle image? It's super large, too large imho.

Hi, Pieter - that method of resizing the art doesn't work with our tools (oh, how I tried), but this did:
Do you want to replace the markdown image reference with an HTML image reference and then resubmit?
Thanks!

Hi, Pieter- I've discussed your suggestion regarding the size of the graphic with the author of the article. He points out that when we size down the graphic, the font is tough to read. We'll look at redesigning the graphic to balance legibility with size/whitespace.
Thanks for raising the issue. For now, I'm closing this pull request without merging. Thanks - Liza


No include instructions for Autolinker.js

Something seems off with Autolinker.js -- all I see is this Rebuild link and despite having rebuilt it I still can't see the include instructions.
Thank you!

I fixed rails-assets so it shows instructions for gems ending with .js. Following works for me//
//= require Autolinker.js

Much obliged!

Only //= require Autolinker.js/Autolinker.js works over here though.


How do I set Transceiver.StreamIDs to change the offer (a=msid-semantic) of HoloLens 2

Hi
I try to use web browser and HoloLens 2 for video communication, and this is the execution flow I follow:
1.- Web client enters the conversation.
2.- Hololens client enters and sends the offer.
3.- Web client receives it, sets the SDP and generates the answer.  It is sent to the Hololens.
4.- The Hololens receives it, and saves the SDP.
5.- Both clients send ICE candidates and receive them.  They complete the ICE gathering.

Now, I can see the browser video stream on HoloLens 2, but can't get Hololens 2 stream on browser. I checked the HoloLens 2's offer and the browser's answer, and found the a=msid-semantic of HoloLens 2's offer may be different. like this:
This is part of the HoloLens 2's offer:
a=group:BUNDLE 0 1
a=msid-semantic: WMS
m=video 9 UDP/TLS/RTP/SAVPF 96 97 98 99 100 101 127 125 104
a=mid:0
a=sendrecv
a=msid:- f583a0a9-2f80-4d85-b6af-ef7ba83ed13d
a=ssrc-group:FID<PHONE_NUMBER><PHONE_NUMBER>
a=ssrc:2667659048 cname:CWYsPV9tCmupNHSl
a=ssrc:2667659048 msid: f583a0a9-2f80-4d85-b6af-ef7ba83ed13d
a=ssrc:2667659048 mslabel:
a=ssrc:2667659048 label:f583a0a9-2f80-4d85-b6af-ef7ba83ed13d
m=audio 9 UDP/TLS/RTP/SAVPF 111 103 9 102 0 8 105 13 110 113 126
a=mid:1
a=sendrecv
a=msid:- a69056ce-f68f-49d0-a376-d2012b6ad755
a=ssrc:1498154044 cname:CWYsPV9tCmupNHSl
a=ssrc:1498154044 msid: a69056ce-f68f-49d0-a376-d2012b6ad755
a=ssrc:1498154044 mslabel:
a=ssrc:1498154044 label:a69056ce-f68f-49d0-a376-d2012b6ad755

This is part of the browser's answer
a=group:BUNDLE 0 1
a=msid-semantic: WMS egMOeAUXRITlOTOaE3PRZZoftadyBOOhMNxy
m=video 9 UDP/TLS/RTP/SAVPF 96 97 98 99 100 101 127 125 104
a=mid:0
a=sendrecv
a=msid:egMOeAUXRITlOTOaE3PRZZoftadyBOOhMNxy dceec5bc-426f-4869-9727-f4df48c98818
a=ssrc-group:FID 300348358 243777033
a=ssrc:300348358 cname:Sqg5iBI1faqe3I0w
a=ssrc:243777033 cname:Sqg5iBI1faqe3I0w
m=audio 9 UDP/TLS/RTP/SAVPF 111 103 9 0 8 105 13 110 113 126
a=mid:1
a=sendrecv
a=msid:egMOeAUXRITlOTOaE3PRZZoftadyBOOhMNxy c30fc5ed-e915-43d5-b189-4326ef006e91
a=ssrc:144003280 cname:Sqg5iBI1faqe3I0w

I searched some Issues (see #695 and #657), and found Transceiver.StreamIDs may be solve this problem. But I don't know how to set Transceiver.StreamIDs and in which script or function.
Any help will be very appreciated!
Thanks in advance.

Solve the problem successfully!
I changed the StartConnection function of PeerConnection.cs script. In line 407, the StreamIDs is added to the TransceiverInitSettings, like this:
var settings = new TransceiverInitSettings
{
    Name = $"mrsw#{index}",
    InitialDesiredDirection = wantsDir,
    StreamIDs = new List<string>() { "hololens2_stream" }
};
tr = _nativePeer.AddTransceiver(mediaLine.MediaKind, settings);

It worked. Now, I can get HoloLens2 stream on browser. Nice~

I will close this issue.

Plus, the StreamID should be a UUID string, otherwise there may be some bugs.


Support new guild features when we know what they are

https://github.com/Discord-Datamining/Discord-Datamining/commit/927edd4a616f0ca487a9dc49ea8975a8abf1674b#commitcomment-66424885

https://github.com/Discord-Datamining/Discord-Datamining/commit/56ca239eeaeb2c3a5b1291934f797a911dfe7690

https://github.com/Discord-Datamining/Discord-Datamining/commit/b6322df0409480434653249ba2fa3d6707c72f55#commitcomment-84413823


ci: fix risc0 test failing

Description

The risc0 test is currently failing in the CI due to:

Testing RISC Zero Go bindings...
go test ./operator/risc_zero/... -v
/tmp/go-build13719[6](https://github.com/yetanotherco/aligned_layer/actions/runs/10779337791/job/29892606443#step:7:7)0975/b001/risc_zero.test: error while loading shared libraries: librisc_zero_verifier_ffi.so: cannot open shared object file: No such file or directory
FAIL	github.com/yetanotherco/aligned_layer/operator/risc_zero	0.001s
FAIL
make: *** [Makefile:51[7](https://github.com/yetanotherco/aligned_layer/actions/runs/10779337791/job/29892606443#step:7:8): test_risc_zero_go_bindings_linux] Error 1

Solved in #953


Add more convenience APIs for common binary operators

Overview
Added convenience APIs in SyntaxFactory for binary operators such as!=, +, -, *, /, %, +=, -=, *=, /=, %=, <, >, <=, >= along with existing ==

I'm new to gyb, but can we have python file which has a list of binary operators in SwiftSyntax side so that we can generate gyb files?

What do you see the benefit of naming each one and not just use a generic version:

What do you see the benefit of naming each one and not just use a generic version:
SyntaxFactory.makeBinaryOperator("+=")

?
We'd need a generic version anyway for custom operators, and seeing the operator written out may actually be a bit more readable. We could also have this do a sanity check that the string really contains an operator.

I thought reserved binary operators can have explicit api, but generic one looks much better.

@swift-ci Please test

Build failed
Swift Test OS X Platform
Git Sha - 7e9caf21141180fec269d67c4a6537af2c8830e4


How to include background images

First, thank you so much for this fantastic work!!
Here is my question. I need to include some background images to the training data generator, because they contain things that can be easily confused with the target objects. Consider them as the hard negative samples. During the hard negative mining, SSD should be able to pick them up and decreases the false positives. I am currently using the parse_csv tool. In the label.csv, they look like the following:
image_name, class_id, xmin, ymin, xmax, ymax
image1.jpg, 0, 0, 0, 0, 0
And I set include_class = [0, 1, 2, 3] in the data generator.
Is this the right way? Could you please suggest other ways to include those images in the data generator?
Thanks!

Out of the three parsers, the CSV parser is the only one that doesn't take the possibility of images without any ground truth into account.
However, with a few workarounds it'll still work. It should actually work exactly the way you're doing it, but you need to make sure that a few parameters are set correctly.
The way you're doing it, after parsing the annotations, the list of labels for each background image will contain one box of class 0 with box coordinates (0, 0, 0, 0), i.e. a degenerate bounding box. This works out fine, because DataGenerator can automatically remove degenerate boxes from the ground truth, leaving the background images with no ground truth at all, which is exactly what you want. With the following settings, everything should work out:

In the data generator's generate() method, make sure degenerate_box_handling is set to 'remove'. This is the default value anyway, but it's important. As explained above, this makes sure that the (0, 0, 0, 0) dummy boxes are being removed.
Also in the data generator's generate() method, set keep_images_without_gt=True. As the argument name says, this makes sure that the generator keeps an image in the batch even if there is no ground truth for it.
In the constructor of SSDLoss, set n_neg_min to some reasonable number greater than zero. This could be important, because in its default settings, the loss function will only take at most 3 times as many negative boxes into account as there are positive boxes. This means in case there are 0 positive boxes in a batch, then there are also 0 negative boxes going into the loss. n_neg_min makes sure that a certain minimum number of negative boxes always goes into the loss. You could set n_neg_min=batch_size for example.

This is great. Thanks for your prompt and detailed explanation!


Don't check layer in layerof function

Codecov Report

Merging #4 (17e72d0) into main (5c36fe1) will decrease coverage by 0.55%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##             main       #4      +/-   ##
==========================================
- Coverage   71.84%   71.28%   -0.56%
==========================================
  Files           6        6
  Lines         103      101       -2
==========================================
- Hits           74       72       -2
  Misses         29       29

Impacted Files
Coverage Δ

src/AbstractLayeredArray.jl
61.53% <100.00%> (ø)

src/AdjointLayeredArray.jl
38.09% <100.00%> (-2.82%)
:arrow_down:

src/LayeredArray.jl
91.66% <100.00%> (-0.65%)
:arrow_down:

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 5c36fe1...17e72d0. Read the comment docs.


[Bug] Compiled files in run artifacts are showing incorrect schema

Is there an existing issue for this?

[X] I have searched the existing issues

Current Behavior
While doing a blue/green test, user is adding a 'staging_' prefix with the generate_schema_name macro. The compiled files within the run artifacts.

vars:

is_stg (value of 1 passed through dbt run --vars 'is_stg: 1')
bg_schemas (in dbt_project.yml)

{%- if custom_schema_name in var('bg_schemas') and var('is_stg', none) is not none -%}
    {%- set custom_schema_name = 'staging_' ~ custom_schema_name -%}
    {{ custom_schema_name | trim }}
{%- else -%}
    {{ custom_schema_name | trim }}
{%- endif -%}

Expected Behavior
'staging_' prefix should be displayed for those schemas identified in the blue/green schema var.
Steps To Reproduce

Create generate_schema_name macro with blue/green schemas and is_stg var.
Identify blue/green schemas var in dbt_project.yml.
Create job with the following run steps: dbt run --vars 'is_stg: 1 && dbt test --vars 'is_stg: 1
Run job and check the compiled file for a blue/green schema.
Should display 'staging_' prefix

Relevant log output
No response
Environment
- dbt: 0.21.0

What database are you using dbt with?
redshift
Additional Context
No response

@crystalro0 sorry for the delay in getting back to you.  Thank you for the detailed write up on what a user is experiencing.
Unfortunately, I'm having trouble replicating the experience or maybe I'm misinterpreting the problem.  I set up a (very) simple project with just 2 models, the bg_schemas var, and your custom generate_schema_name macro.  When I run it the appropriate schemas get the 'staging_' prefix.  I'm running in dbt Cloud with 0.21.0 against Redshift with a job as you indicated.
The appropriate models end up in a schema prefixed with 'staging_' and the others do not.  And when I look into target/compiled/ at the model files, the refs have been converted into the expected schema prefixed with 'staging_'.
Would you be able to take a look at the project I set up and let me know if there is something that I missed setting up?  If not, would you be able to fork it and run it yourself to see if you get the same unexpected results?
This is going to take a bit more digging to get to the bottom of but we'll figure it out!
One more note that there were multiple fixes that came out in 0.21.1 that should not be breaking and might be worth the upgrade.

hi @emmyoop, thanks for checking this out! Sorry I didn't specify which compiled files. My bad on that. The original schema is showing up in the compiled schema tests.
Under my local: compiled/jaffle_shop/models/staging/jaffle_shop/stg_jaffle_shop.yml/schema_test/accepted_values_stg_payments.sql

This looks like what will happen with dbt 0.21 and partial parsing turned on and a custom generate_schema_name macro. It is fixed in 1.0.0 (where the fix consists of forcing a full re-parse). If you add or change a custom generate_schema_name macro in 0.21 you will have to do a full-reparse by turning off partial parsing.

@crystalro0 since @gshank pointed out this is fixed in 1.0.0, I'm going to go ahead and close this issue as we will not be fixing it in 0.21.  Please reopen it if you find that it is still a big in 1.0!


Freezes when trying to load latest releases

KawAnime always freeze when it's trying to load the things in "news" section? It always says "retrying in 45 seconds" then freezes again. Is there a way to not auto-load the "new releases" section? It works fine when I remove the WiFi connection.

Hello,
I have several questions to be able to get you a better answer.
What do you mean be "The app freezes?" ? Like not responding at all? If so, how much RAM do your computer have?
You're saying "news" but according to what you're saying, I believe you are talking about the "release" section in which case this problem is a duplicate of #27 and #35.
I'm going to do something that will fix this for the next release.

My RAM currently is 4gb (laptop). Yes, it just freezes (not responding) on the tab while it's loading the releases.

It should be fixed on master branch, I'll try to make a release as soon as I can!


how to locate the ego locations in the original nuscenes map?

Very amazing project!
Maybe there is some differences between the given "singapore-onenorth" and the original  nuscence "singapore-onenorth", for example , when i use the same ego pose to locate, i get different hd bev maps from your code and the original nuscenes map explorer.

Thanks for your attention to our project~
The maps we provide are sourced from OpenStreetMap rather than directly converted from the original nuscenes map. This can lead to discrepancies, so complete alignment with the original nuscenes map may not be possible. We appreciate your understanding.


Doesn't seem to be working in v1.16.5

Version 3.1.0-Forge
Minecraft v1.16.5
Forge v36.1.7

Modlist

Due to a limitation in Forge, IKWID is unable to save changes to the options.txt at startup, and therefore only works when an options.txt has not previously been created. I can look into working around this limitation. Thank you for bringing this to my attention!

Fixed in 3.2.0


[msal-common] DeviceCodeClient not properly throwing if the server errors during polling

Core Library
MSAL Node (@azure/msal-node)
Core Library Version
latest
Wrapper Library
Not Applicable
Wrapper Library Version
not applicable
Description
The DeviceCodeClient here does not consider scenarios in which the service crashes. The executePostToTokenEndpoint method of the base client only adds telemetry of service error cases.
Error Message
No error message is shown.
Msal Logs
No response
MSAL Configuration
{}

Relevant Code Snippets
//

Reproduction Steps
This issue can be reproduced by passing a HTTP client pointing to a proxy, or mock the network requests to return an error during polling.
Expected Behavior
If the authentication service fails to respond or crashes, or the authorization-pending string is missing, polling should stop and the error should be thrown.
Identity Provider
Azure AD / MSA
Browsers Affected (Select all that apply)
None (Server)
Regression
No response
Source
Internal (Microsoft)

cc @bmahall can you please look into this?

@msftbot good bot 🙂

Marking it as bug and assigned to me. Will work on this soon.


duplicated HDD

When using one hdd image, I get both sda and sdb referencing to the same disk
[    2.175070] sd 0:0:0:0: [sda] 30720 512-byte logical blocks: (15.7 MB/15.0 MiB)
[    2.195071] sd 0:0:1:0: [sdb] 30720 512-byte logical blocks: (15.7 MB/15.0 MiB)

Yep (and they access the same underlying buffer). This is a duplicate of #14 though.


migrate-from-to-a-different-public-cloud-datacenter

Please use the form below, leaving the prefilled data to help us. Thank you.
Page link: migrate-from-to-a-different-public-cloud-datacenter
Document link: migrating-to-different-cloud.md
My Issue/Suggestion
The title is confusing; the use of the form  from to  is confusing

Thank you for reporting the issue. I've fixed the typo, the update should be published today.


Feature/GitHub models provider

GitHub Models Provider Integration
Overview
This PR adds support for GitHub Models, integrating a comprehensive suite of AI models through Azure ML endpoints. This integration enables developers to leverage powerful language models while maintaining compatibility with existing OpenAI-style interfaces.
Features Added

GitHub Models provider implementation using Azure ML endpoints
Support for multiple model series:

O1 Series (Preview and Mini)
Meta Llama Series (405B/70B/8B/Vision Instruct)
Mistral Series (Large/Medium/Small)
GPT-4O Series (Advanced/Standard)
Phi Series (Phi-2, Phi-1.5)

OpenAI-compatible interface for seamless integration

Implementation Details

Added GitHub Models configuration in environment variables (GITHUB_API_KEY)
Updated model selection interface to include GitHub Models
Added comprehensive documentation in CONTRIBUTING.md
Fixed action runner promise handling and linting issues

Setup Requirements

Obtain a GitHub API Key from https://github.com/settings/tokens
Add to .env.local:

GITHUB_API_KEY=your_key_here

looks nice good work sir <3


fix #1078 Azure AKS authentication issues

@jessesuen signed CLA and added fix to additional two locaitons.


fix duplicate file drop

this issue is being worked on in hyperdrive: https://github.com/mafintosh/hyperdrive/issues/63

what is the issue here? will this be fixed by the recent changes to hyperdrive-import-files?

I'm 50/50 on this one...on one hand it would be nice to de dupe files, on the other hand, hashing everything is too expensive and simply relying on mtimes for deduping isn't safe. I'd say we should make the stack work in a way that it doesn't matter really, even if a file is added again.

@mafintosh what do you think about this one? ^

ping @mafintosh @juliangruber any update on this?

ping @juliangruber – not sure what's the status here?

this is not for dat desktop to solve, it's a hyper* issue. closing here as we haven't settled on a solution yet.


Fixed #25616 -- Added a note regarding missing dependencies on LookupError

This  fixes #25616 by including additional information in the traceback regarding missing dependencies during an attempted migration.

Closing due to lack of follow-up.


Form text inputs not responding in go run

Not sure if this is mentioned somewhere or not but I am unable to enter text into a form input box. Whatever I type is redirected to the console where I ran go run. I ran build and the form was then usable. Is there something I missed?

package main

import (
	"fmt"
	"log"
	"net/http"
	"os"
	"runtime"

	"github.com/alexflint/gallium"
)

var html = `
<!doctype html>

<form method="post" action="/testform">
<input name="testinput" type="text"  value="" />
<button type="submit">Submit</button>
</form>

`

type app struct {
	ui     *gallium.App
	window *gallium.Window
}

func (a *app) handleIndex(w http.ResponseWriter, r *http.Request) {
	fmt.Fprint(w, html)
}

func onReady(ui *gallium.App) {
	opts := gallium.FramedWindow
	opts.Shape.Width = 500
	opts.Shape.Height = 500
	window, err := ui.OpenWindow("http://<IP_ADDRESS>:9478/", opts)
	if err != nil {
		log.Fatal(err)
	}

	app := app{
		ui:     ui,
		window: window,
	}
	http.HandleFunc("/", app.handleIndex)
	go http.ListenAndServe(":9478", nil)
}

func (a *app) handleTest(w http.ResponseWriter, r *http.Request) {
	a.window.OpenDevTools()
	r.ParseForm()
	fmt.Println("Name", r.Form["testinput"])
	fmt.Fprint(w, fmt.Sprintf("Hello %s", r.Form["testinput"]))
}
func main() {
	runtime.LockOSThread()
	gallium.RedirectStdoutStderr(os.ExpandEnv("$HOME/Library/Logs/Gallium.log"))
	gallium.Loop(os.Args, onReady)
}

Try this:
$ go install github.com/alexflint/gallium/cmd/gallium-bundle
$ go build ./example
$ gallium-bundle example
$ open example.app

Unfortunately OSX makes it exceedingly difficult to do proper UI stuff without launching the app from within a proper app bundle. I would like to fix this but I have not yet found a complete solution, so for now I recommend you stick with the above.

ok no problem, thanks


work strange in IE9

in IE9 ,use FB's react work fine,replace with react-lite work strange and throw error.
SCRIPT5007: Unable to get value of the property 'cache': object is null or undefined

SCRIPT87: Invalid argument.

WTF?

I have never seem this error message before, can you add some code for reproducing?

error SCRIPT87: Invalid argument.  in:
t.parentNode.insertBefore(e,t.parentNode.childNodes[t.index])


Refactored MetricsDefinition to use a MetricBuilder and be based off a super class

Refactored the MetricDefinition to inherit a Metric super class which can be inherited by other type of metrics (coming soon). Also refactored MetricDefinition to be immutable and created a MetricDefinitionBuilder which behaves similar to how MetricDefinition used to.
Naming is also handled using a setter instead of on creation for simplicity in upcoming metric changes.
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

All but the resolveEnvironment tests passed which didn't pass on my machine before these changes, and these changes should not affect those tests. Also tested changes using the lambda example and the metrics were logged to CWL as expected


Added Azure.SQLMI.AAD

PR Summary
Fixes #2117
Added Azure.SQLMI.AAD rule.
PR Checklist

[x] PR has a meaningful title
[x] Summarized changes
[x] Change is not breaking
[x] This PR is ready to merge and is not Work in Progress
Rule changes

[x] Unit tests created/ updated
[x] Rule documentation created/ updated
[x] Link to a filed issue
[x] Change log has been updated with change under unreleased section

Other code changes

[ ] Unit tests created/ updated
[ ] Link to a filed issue
[ ] Change log has been updated with change under unreleased section

@BernieWhite Ready for review :)

@BernieWhite fixed :)


Updates de.js

Brings de.js up-to-date with en.js.  It's been a while since I've done one of these, I hope I did it okay.

None of this needs to block the merge. This is all fine. Unless I've missed typos, in which case it's my bad.
From #997, @ValerieKenyon says:

(BTW: I was working from en.js in the google docs, updating to de.js in the google docs, and then creating a new, up-to-date branch. I copied and pasted my German from the google doc into the new branch and tweaked it in VS Code. I also noticed that the en.js in dev branch (and in germanTranslation18_11_13) has a lot more text in it than the en.js google doc, so my updated de.js doesn't actually contain this newer text.)

I can't tell exactly what you mean, but it sounds like the google doc is out of date, so someone needs to go through and update it carefully. There are about to be more changes to it, so after the next couple merges, which should happen in just a few minutes, anyone can feel free to take that on. There's actually an issue for that already.
For English, we can basically just cut and paste the local changes into the google doc. The updates to the other languages need to happen more carefully.
@ValerieKenyon : Is that something you'd be interested in doing/have time to do?


Latest DEV doesn't find pokemons?

I have the latest DEV and it looks like that it doesn't find pokemon?
Here is the log:
[10:57:53]  Starting PokemonGo Bot....
[10:57:54]  [x] Next egg incubates in 1.58 km
[10:57:54]  No lucky eggs... skipping evolve!
[10:57:55]  -- Recycled 10x Hyper Potion (keeps only 30 maximum)
[10:57:56]  -- Recycled 10x Revive (keeps only 30 maximum)
[10:57:56] Walking from (52.21115700000001, 5.9699231) to [52.21115700000001, 5.9699231] 0.00km
[10:57:58] Walking from (52.21115700000001, 5.9699231) to [52.21295700000001, 5.9699231] 0.20km
[10:58:57]  [x] Next egg incubates in 1.52 km
[10:59:35]  Server seems to be busy or offline - try again - 1/5
[11:01:01] Walking from (52.21295700000001, 5.9699231) to [52.21295700000001, 5.9717231] 0.12km
[11:02:56] Walking from (52.21295700000001, 5.9717231) to [52.21115700000001, 5.9717231] 0.20km
[11:02:59]  [x] Next egg incubates in 1.32 km
[11:06:12] Walking from (52.21115700000001, 5.9717231) to [52.209357000000004, 5.9717231] 0.20km
[11:07:04]  [x] Next egg incubates in 1.05 km
[11:09:16] Walking from (52.209357000000004, 5.9717231) to [52.21115700000001, 5.9717231] 0.20km
[11:11:01]  [x] Next egg incubates in 0.81 km
[11:12:17] Walking from (52.21115700000001, 5.9717231) to [52.21295700000001, 5.9717231] 0.20km
[11:15:10]  [x] Next egg incubates in 0.58 km
[11:15:25] Walking from (52.21295700000001, 5.9717231) to [52.21295700000001, 5.9699231] 0.12km
[11:17:28] Walking from (52.21295700000001, 5.9699231) to [52.21115700000001, 5.9699231] 0.20km
[11:19:09]  [x] Next egg incubates in 0.33 km
Can somebody explains to me what is going wrong?

Did you set up the task to catch pokemons?
Share your config file

"catch_pokemon": true,
"forts": {
  "spin": false,
  "move_to_spin": false,
  "avoid_circles": true,
  "max_circle_size": 50
},
"websocket_server": false,
"walk": 4.16,
"action_wait_min": 1,
"action_wait_max": 4,
"debug": false,
"test": false,
"health_record": true,
"location_cache": false,
"distance_unit": "km",
"reconnecting_timeout": 15,
"item_filter": {
  "1": { "keep" : 100 },
  "101": { "keep" : 10 },
  "102": { "keep" : 30 },
  "103": { "keep" : 30 },
  "201": { "keep" : 30 },
  "701": { "keep" : 100 }
},
"evolve_all": "NONE",
"evolve_speed": 20,
"evolve_cp_min": 300,
"use_lucky_egg": true,
"hatch_eggs": true,
"longer_eggs_first": true,
"evolve_captured": "all",
"release_pokemon": true,
"catch_randomize_reticle_factor": 1.0,
"catch_randomize_spin_factor": 1.0,
"catch": {
  "any": {"catch_above_cp": 0, "catch_above_iv": 0, "logic": "or"},
  "// Example of always catching Rattata:": {},
  "// Rattata": { "always_catch" : true }
},
"release": {
  "any": {"release_below_cp": 0.8, "release_below_iv": 0.0, "logic": "or"},
  "// Example of always releasing Rattata:": {},
  "// Rattata": {"always_release": true},
  "// Example of keeping 3 stronger (based on CP) Pidgey:": {},
  "// Pidgey": {"keep_best_cp": 3},
  "// Example of keeping 2 stronger (based on IV) Zubat:": {},
  "// Zubat": {"keep_best_iv": 2},
  "// Also, it is working with any": {},
  "// any": {"keep_best_iv": 3},
  "// Example of keeping the 2 strongest (based on CP) and 3 best (based on IV) Zubat:": {},
  "// Zubat": {"keep_best_cp": 2, "keep_best_iv": 3}
},
"vips" : {
     "Any pokemon put here directly force to use Berry & Best Ball to capture, to secure the capture rate!": {},
    "any": {"catch_above_cp": 1200, "catch_above_iv": 0.9, "logic": "or" },
    "Lapras": {},
    "Moltres": {},
    "Zapdos": {},
    "Articuno": {},

    "// S-Tier pokemons (if pokemon can be evolved into tier, list the representative)": {},
    "Mewtwo": {},
    "Dragonite": {},
    "Snorlax": {},
    "// Mew evolves to Mewtwo": {},
    "Mew": {},
    "Arcanine": {},
    "Vaporeon": {},
    "Gyarados": {},
    "Exeggutor": {},
    "Muk": {},
    "Weezing": {},
    "Flareon": {}

Catch pokemon: true

Please update your config up to the config.json.example, or try to put this script right after gmapkey:
    "tasks": [
      {
        "type": "HandleSoftBan"
      },
      {
        "type": "CollectLevelUpReward"
      },
      {
        "type": "IncubateEggs",
        "config": {
          "longer_eggs_first": true
        }
      },
      {
        "type": "TransferPokemon"
      },
      {
        "type": "EvolveAll",
        "config": {
          "evolve_speed": 20,
          "use_lucky_egg": true
        }
      },
      {
        "type": "RecycleItems"
      },
      {
        "type": "CatchVisiblePokemon"
      },
      {
        "type": "CatchLuredPokemon"
      },
      {
        "type": "SpinFort"
      },
      {
        "type": "MoveToFort"
      },
      {
        "type": "FollowSpiral"
      }
    ],

i have the same problem; exacly same log.
Bot walks back and forth and does not catch anything even with newest config

Your issue seems to be duplicate of #2031 .
For me, removing this from the config did the trick:
  {
    "type": "FollowSpiral"
  }

PS. Some other people said that changing Evolve_All to false did it for them, but that didn't work for me.

without FollowSpiral it just does nothing for me


discovery working

Codecov Report

Merging #21 (53da495) into master (b5621b2) will increase coverage by 3.03%.
The diff coverage is 46.05%.

@@            Coverage Diff             @@
##           master      #21      +/-   ##
==========================================
+ Coverage   52.96%   55.99%   +3.03%
==========================================
  Files           8        8
  Lines        1316     1543     +227
==========================================
+ Hits          697      864     +167
- Misses        502      536      +34
- Partials      117      143      +26

Flag
Coverage Δ

unittests
55.99% <46.05%> (+3.03%)
:arrow_up:

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

Impacted Files
Coverage Δ

internal/repo/repo.go
56.00% <28.57%> (-0.65%)
:arrow_down:

mobile/datahop.go
50.20% <41.87%> (-12.61%)
:arrow_down:

mobile/discovery.go
30.43% <45.83%> (-0.60%)
:arrow_down:

internal/replication/replication.go
52.25% <63.63%> (+1.85%)
:arrow_up:

internal/matrix/matrix.go
70.56% <100.00%> (+44.72%)
:arrow_up:

ipfs.go
66.20% <0.00%> (+0.68%)
:arrow_up:

Continue to review full report at Codecov.

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


Demo: "Show Value Under Mouse, Multiple Series" is broken

ScottPlot5 demo "Show Value Under Mouse, Multiple Series" responds unintuitively

suggested fix
https://github.com/ScottPlot/ScottPlot/blob/main/src/ScottPlot5/ScottPlot5 Demos/ScottPlot5 WinForms Demo/Demos/ShowValueOnHoverMultiple.cs
I added a line in the section where it is searching for the point which is nearest to the mouse position. This should fix the issue.
        formsPlot1.MouseMove += (s, e) =>
        {
            // determine where the mouse is
            Pixel mousePixel = new(e.Location.X, e.Location.Y);
            Coordinates mouseLocation = formsPlot1.Plot.GetCoordinates(mousePixel);

            // get the nearest point of each scatter
            Dictionary<int, DataPoint> nearestPoints = new();
            for (int i = 0; i < MyScatters.Count; i++)
            {
                DataPoint nearestPoint = MyScatters[i].Data.GetNearest(mouseLocation, formsPlot1.Plot.LastRender);
                nearestPoints.Add(i, nearestPoint);
            }

            // determine which scatter's nearest point is nearest to the mouse
            bool pointSelected = false;
            int scatterIndex = -1;
            double smallestDistance = double.MaxValue;
            for (int i = 0; i < nearestPoints.Count; i++)
            {
                if (nearestPoints[i].IsReal)
                {
                    // calculate the distance of the point to the mouse
                    double distance = nearestPoints[i].Coordinates.Distance(mouseLocation);
                    if (distance < smallestDistance)
                    {
                        // store the index
                        scatterIndex = i;
                        pointSelected = true;
                        smallestDistance = distance; //I added this line to update smallestDistance.
                    }
                }
            }

            // place the crosshair, marker and text over the selected point
            if (pointSelected)
            {
                ScottPlot.Plottables.Scatter scatter = MyScatters[scatterIndex];
                DataPoint point = nearestPoints[scatterIndex];

                MyCrosshair.IsVisible = true;
                MyCrosshair.Position = point.Coordinates;
                MyCrosshair.LineColor = scatter.MarkerStyle.Fill.Color;

                MyHighlightMarker.IsVisible = true;
                MyHighlightMarker.Location = point.Coordinates;
                MyHighlightMarker.MarkerStyle.Outline.Color = scatter.MarkerStyle.Fill.Color;

                MyHighlightText.IsVisible = true;
                MyHighlightText.Location = point.Coordinates;
                MyHighlightText.LabelText = $"{point.X:0.##}, {point.Y:0.##}";
                MyHighlightText.LabelFontColor = scatter.MarkerStyle.Fill.Color;

                formsPlot1.Refresh();
                base.Text = $"Selected Scatter={scatter.Label}, Index={point.Index}, X={point.X:0.##}, Y={point.Y:0.##}";
            }

Whoops, good catch!
This line:
smallestDistance = distance;

should be inserted between lines 52 and 53:
https://github.com/ScottPlot/ScottPlot/blob/5ed3178403e56283dfd583b05890893d1d9ab2f0/src/ScottPlot5/ScottPlot5 Demos/ScottPlot5 WinForms Demo/Demos/ShowValueOnHoverMultiple.cs#L71-L85
If someone wants to open a pull request to correct this issue, I'd be happy to merge it in this evening!

Good catch indeed! I was wondering why it was behaving weirdly sometimes, but couldn't figure out the cause. It's so obvious now :-) I made a PR #3684

resolved in #3684


feat: enable grouping broken paragraphs in partition_text

Summary
Closes #406. Adds a group_broken_paragraphs cleaning brick for grouping together sentences that have a page break for formatting purposes. This is common in .txt files. Also add a paragraph_grouper kwarg to partition_text to allow for passing in a paragraph grouping callable.
Testing
The following code should produce two NarrativeText elements. Without the paragraph grouper, partition_text produce four elements due to the line break.
from unstructured.partition.text import partition_text
from unstructured.cleaners.core import group_broken_paragraphs

text = """The big brown fox
was walking down the lane.

At the end of the lane, the
fox met a bear."""

partition_text(text=text, paragraph_grouper=group_broken_paragraphs)

Couple of comments:
I'll try this out on some real docs that tend to have blank lines between paragraphs and see there are reasonable elements that come out. It's not obvious to me they would. E.g., https://www.ietf.org/rfc/rfc1918.txt and https://www.ietf.org/rfc/rfc1918.txt .
Also, it'd be nice if parition_txt() could automatically do the right thing in most cases, allowing the user to choose a certain splitting pattern if desired. I.e., paragraphs in examples above are NarrativeText items, and in the case of a phone book (a bunch of single lines), each line is its own element (not worrying about the type of element too much). From an implementation note, it seems like it would be more straightforward to choose between to regex splitters, adding a "blank line" one after https://github.com/Unstructured-IO/unstructured/blob/3467a27/unstructured/nlp/patterns.py#L71 when partitioning, rather than relying on a cleaning brick to do so after the fact (though the cleaning brick is a nice to have regardless).


Replace inter font with ciscosans font.

Description

Delete ciscosans font family.
Add Inter font family.
The inter font family has been downloaded from here: https://rsms.me/inter/
The format of the font is woff2 which are small in size and mostly preffered for Web.

Links

Reason to switch Inter from ciscosans: Figma link
https://rsms.me/inter/

closing since not necessary yet.


jcenter is shutting down, replace by Gradle Plugins

jcenter is shutting down in less than 90 days, so we need to remove it from our build scripts. All dependencies we needed from there are available on Gradle Plugins repo as well.

Build fails on fetching the natives, but that shouldn't be caused by this change.

The Gradle repo is already present on generated project, and it is needed for Gretty. So no todo left here. :)


Cleanup code

CodeFactor found an issue: Invalid flags are not handled. Add a *) case.
It's currently on:
scripts\git\cleanupcode.sh:24-27
Commit 6d2ca2b6b4fbf211d7adebc35417aa779def3635


Update JamesServerExtension

This PR is used to test effect of changes in postgresql branch (https://github.com/apache/james-project/pull/2072) on master branch

Can you please better explain why this change is needed?

The old code of JamesServerExtension:
PER_ENCLOSING_CLASS(
            (extension, context) -> {
                if (!isNested(context)) {
                    extension.start(context);
                }
            },
            (extension, context) -> { },
            (extension, context) -> { },
            (extension, context) -> {
                if (!isNested(context)) {
                    extension.stop(context);
                }
            }),

With the old code, when running test class that have nested classes, the beforeAll and afterAll method of all registrableExtensions run several times (corresponding to number of nested classes) during the test class execution while beforeEach and afterEach method of registrableExtensions only run once at the beginning and the end -----> wrong order
With the new code:
PER_ENCLOSING_CLASS(
            (extension, context) -> {
                if (!isNested(context)) {
                    extension.setup(context);
                    extension.start(context);
                }
            },
            (extension, context) -> { },
            (extension, context) -> { },
            (extension, context) -> {
                if (!isNested(context)) {
                    extension.stop(context);
                    extension.tearDown(context);
                }
            }),

when running test class that have nested classes, beforeAll, afterAll, beforeEach and afterEach methods of all registrableExtensions only run once with the right order beforeAll -> beforeEach -> afterEach -> afterAll
In addition, I believe the new code is easier to understand than the old code. The old code really confused me.


[WIP] Attempt to fix carousel isActive

Follow up https://github.com/cds-snc/covid-alert-app/issues/890

I tried to use Observable but it doesn't help. As this is not important, I close this for now.


Add initial auth infrastructure.

Description
This is based on pac4j and pac4j-play.  It involved sorting out a diamond dependency issue with Jackson but that's solved now, with dep overrides (permanently? we'll see!).
Checklist

[x] Created tests which fail without the change (if possible)
[ ] Extended the README / documentation, if necessary

Issue(s)
@carolinedanzi primary reviewer, @hmhsu and @bion comments welcome but feel free to ignore.

Nope, in fact it only allows username / password pairs like test/test, admin/admin.  Trying to make relatively small PRs.


Android: Exception on write: android.os.Parcelable on a null object reference

Plugin version: 0.6.6
Android: 5.0.2
  var message = [
    $window.ndef.textRecord(textData),
  ];
  $window.nfc.write(message, ...)

produces the following error:
"Attempt to invoke virtual method 'android.os.Parcelable android.content.Intent.getParcelableExtra(java.lang.String)' on a null object reference"

This error occurs when there hasn't been any tag attached to the device since the app start - this error changes to 'null' once a tag is attached to the device

On Android the write method must be called from within an NDEF Event Handler. It fails if there's no tag present.


add shorter syntax for defining fact-rules

Instead of writing Fact(a, b) <- (true); one can now write Fact(a, b);

Thanks, looks great!


C++ Library Working

The cpp branch, which should be available on Github now, is integrated into Python.
The C++ code is currently in a repo at "https://github.com/r-barnes/compactnesslib".
I haven't yet set the library to compile using OpenMP (which would parallelize everything), but could do so in the future, though it may complicate the build somewhat.
The cpp branch does not yet replace any of the existing functionality of the library, but I'm raising this issue to suggest that we modify the library so that it does.
An example of how to use compactnesslib from Python is as follows:
#!/usr/bin/env python

import mander
import sys

if len(sys.argv)!=2:
  print("Syntax: {0} <GeoJSON file>".format(sys.argv[0]))
  sys.exit(-1)

with open(sys.argv[1], 'r') as fin:
  content    = fin.read()
  score_json = mander.ProcessGeoJSON(content, '')
  print(score_json)

Handing this an input GeoJSON consisting of Massachusetts's electoral districts returns, after 0.07s on my laptop, the following output:
{
  "0":{
    "perim":500138,
    "area":6.21574e+09,
    "PolsbyPopp":0.312265,
    "Schwartzbe":0.558807,
    "ConvexHull":0.74883,
    "Reock":0.410659
  },
  "1":{
    "perim":452510,
    "area":4.4387e+09,
    "PolsbyPopp":0.272402,
    "Schwartzbe":0.521921,
    "ConvexHull":0.770146,
    "Reock":0.398661
  },
  "2":{
    "perim":335527,
    "area":2.03904e+09,
    "PolsbyPopp":0.227605,
    "Schwartzbe":0.477079,
    "ConvexHull":0.675721,
    "Reock":0.31953
  },
  "3":{
    "perim":463187,
    "area":1.79412e+09,
    "PolsbyPopp":0.105087,
    "Schwartzbe":0.324171,
    "ConvexHull":0.64404,
    "Reock":0.366629
  },
  "4":{
    "perim":204721,
    "area":1.67266e+08,
    "PolsbyPopp":0.0501522,
    "Schwartzbe":0.223947,
    "ConvexHull":0.567434,
    "Reock":0.072261
  },
  "5":{
    "perim":498935,
    "area":8.67639e+08,
    "PolsbyPopp":0.0437988,
    "Schwartzbe":0.209282,
    "ConvexHull":0.567975,
    "Reock":0.191279
  },
  "6":{
    "perim":2.01532e+06,
    "area":3.3033e+09,
    "PolsbyPopp":0.0102205,
    "Schwartzbe":0.101096,
    "ConvexHull":0.551033,
    "Reock":0.0405621
  },
  "7":{
    "perim":662617,
    "area":1.41659e+09,
    "PolsbyPopp":0.0405442,
    "Schwartzbe":0.201356,
    "ConvexHull":0.713262,
    "Reock":0.174989
  },
  "8":{
    "perim":252726,
    "area":7.1762e+08,
    "PolsbyPopp":0.141191,
    "Schwartzbe":0.375753,
    "ConvexHull":0.613872,
    "Reock":0.294034
  }
}

The output ids could be selected from the incoming data, though I'm not sure what the attribute names are or how they are stored in the objects. Could someone upload an example GeoJSON?
The output could also be the fully GeoJSON multipolygons with the scores attached as attributes.
If I don't hear back from anyone in a week or so, I'll likely make an attempt at modifying the Python myself.
Hope everyone's doing well!

Hey @r-barnes, there are some GeoJSON files in the test/data directory of the repo. As for the structure of the output of metrics.scoresToGeojson(), the specified metric name and score gets added as a key-value pair to the properties attribute of the original .geojson file. Let me know if you need any more detail than that.

Thanks, I've got property extraction from GeoJSON working now.
I've also, as of today, got bounded scores working. These take in the boundaries of electoral districts and the boundaries of the states which contain them and calculate compactness scores which account for the fact that the electoral district must be contained by the state.
Since the interface uses GeoJSON or WKT to communicate with the library I've also implemented a caching scheme to reduce computation.
It's probably worth chatting about what good return values are, and what the interface should look like.
At the moment the library ingests GeoJSON and returns a JSON dictionary containing only the scores for each district queried. I could imagine it ingesting shapely items and converting them to library format internally, as well as returning JSON dictionaries, attributified GeoJSON, or straight-out Python dictionaries.
The library also includes some high-performance, bulk-processing functionality to read/write shapefiles directly from disk without involving Python. This allows for commands like:
mander.addBoundedScoresToNewShapefile(sys.argv[1], sys.argv[2], sys.argv[3], sys.argv[4])

Use cases are needed at this point.

It's merged.


How can i disable route path auto replacing?

Description
When i create /src/pages/contact_done.vue, generated path is placed {"path" : "/contact-done"}.
How can i disable this behavior?
Steps to reproduce
Create page component which contains _ on filename.
Expected result
Generated path is same as filename. e.g {"path": "/contact_done"}
Environment

Libs:
- gridsome version: 0.6.7
- @gridsome/cli version: 0.2.1

For Tooling issues:
- Node version: v10.11.0
- Platform:  Mac

Sorry for late reply.
I'm trying to replace only static pages with Gridsome from an application made with Nuxt.js which appilication's URL are separated by underscore.
I know the suitable URL for SEO, however I don't want to redirect. (Actually I don't have authority.)
So this behavior is unexpected for me.
thanks.

I also have specific path requirements. Could someone please provide information on how to control or disable path rewriting.
I have files at /docs/test.md and the paths are being altered to /docs-test/. This only happens in version 0.7.0 and not 0.6.9.


Test blues on CPUs

@limn1 reported problems running simulations on CPUs. I'll revisit this issue after #19 has been addressed.

I think we're pretty sure CPUs aren't a particular issue anymore (although they are slow to run on for sure).


[docs] DNS changes to support docs.carbonmark.com

gitbook has been setup with following URL docs.carbonmark.com
Following the same settings as docs.klimadao.finance
Need DNS updated.

Requested record to add from Chaz

@sy-zygy 1252e64f01-hosting.gitbook.io

Record has been created -- but getting a SSL err

Now I am getting a cloudflare err

@Atmosfearful any ideas?

It is working now @jabby09


gh-38: started moving over to using jsonnet

Resolves gh-38

Currently getting an error, would love another set of eyes on this. Error:
RUNTIME ERROR: multi mode: top-level object was a string, should be an object whose keys are filenames and values hold the JSON for that file.
	During manifestation
make: *** [dashboards_out] Error 1

I've largely been following the kubernetes-monitoring/kubernetes-mixin project.

Huzzah! Managed to get this working and break things down in a modular way. Most dashboards share similar metrics, but they're implemented once and then shared around.
There's a little more work to be done, but I think this is a good start.
For those unfamiliar with jsonnet reading this, I'm using jsonnet-builder which lets me pull components from other repositories. This is then passed into jsonnet with an entrypoint (currently defined in lib/) for each thing we generate (rules, alerts, dashboards, etc).

@mkw for awareness since he was asking about dashboards before I left.

Here's a preview of the dashboard for the tracker process:


feat: support evaluation of fn call

coverage: 81.946% (+0.1%) from 81.842% when pulling e5a9a1b3a246316fd7f1830493947b14d7c0bfa8 on feat/eval-fn-call into 348f2e9b21af6ebdcd2e97ae752fce3dbd50514a on master.


Missing params

Added in most parameters and endpoints
Resolves #43, #27, #17

@michaelnlindsay I agree with creating a new request_options object, but maybe in another PR (and by someone with more Python experience :P)

lgtm!


[CPDLP-3493] NPQ Internal API - Add feature flag instead of application config

Context
Ticket: https://dfedigital.atlassian.net.mcas.ms/browse/CPDLP-3493
Changes proposed in this pull request

Added feature: Feature.ecf_api_disabled? with flipper

Looks good to me too, @leandroalemao regarding toggling I was wondering if we do need toggling methods as those are given to us by Flipper so need to add a wrapper method?

Looks good to me too, @leandroalemao regarding toggling I was wondering if we do need toggling methods as those are given to us by Flipper so need to add a wrapper method?

I believe flipper has an admin UI to enable/disable features, unlikely to need the toggle:
/admin/feature_flags/features


Example of playing and pausing the video

Can someone please post an example of using the api's listed here to programmatically pause and play? Thanks!

You can check how is done in this example: https://xdae.github.io/react-plyr/?selectedKind=React Plyr&selectedStory=External Controls %26 State&full=0&addons=1&stories=1&panelRight=1&addonPanel=storybook%2Factions%2Factions-panel
https://github.com/xDae/react-plyr/blob/master/.storybook/stories/withWrapper.js

thanks!


[BUG] System.TypeLoadException

Tried to set running newest bits. When started, application fails with System.TypeLoadException.
Error

Exception or Stack Trace
System.TypeLoadException
HResult=0x80131522
Message=Method 'get_EventBehaviourForContinueAsNew' in type 'DurableTask.ServiceBus.ServiceBusOrchestrationService' from assembly 'DurableTask.ServiceBus, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=d53979610a6e89dd' does not have an implementation.
Source=
StackTrace:

Project file
<Project Sdk="Microsoft.NET.Sdk">

  <PropertyGroup>
    <OutputType>Exe</OutputType>
    <TargetFramework>netcoreapp2.2</TargetFramework>
  </PropertyGroup>

  <ItemGroup>
    <PackageReference Include="Microsoft.Azure.DurableTask.AzureStorage" Version="1.6.0" />
    <PackageReference Include="Microsoft.Azure.DurableTask.Core" Version="2.1.1" />
    <PackageReference Include="Microsoft.Azure.DurableTask.ServiceBus" Version="2.0.10" />
  </ItemGroup>

  <ItemGroup>
    <ProjectReference Include="..\SharedLib.NetCore\SharedLib.NetCore.csproj" />
  </ItemGroup>

</Project>

Sample
static void Main(string[] args)
       {

           string hubName = "DtfSampleHub2";
           IOrchestrationServiceInstanceStore instanceStore = new AzureTableInstanceStore(hubName, m_StorageConnStr);

           var orchestrationServiceAndClient = new ServiceBusOrchestrationService(m_ConnStr, hubName, instanceStore, null, null);

           orchestrationServiceAndClient.CreateIfNotExistsAsync().Wait();

           m_TaskHubWorker = new TaskHubWorker(orchestrationServiceAndClient);

           m_TaskHubWorker.AddTaskOrchestrations(typeof(TimerOrchestration));
           m_TaskHubWorker.AddTaskOrchestrations(typeof(HelloOrchestration));
           m_TaskHubWorker.AddTaskActivities(typeof(SchedulerTask), typeof(UserInputTask), typeof(PlayFileTask));

           m_TaskHubWorker.StartAsync().Wait();

           Thread.Sleep(int.MaxValue);
       }

Thank you for opening this issue! We are routing it to the appropriate team for follow up.

@ddobric Microsoft.Azure.DurableTask.ServiceBus is referencing the old client, WindowsAzure.ServiceBus. Which is not related to this client here.

This issue should be assigned to DurableTask framework (https://github.com/Azure/durabletask)

Microsoft.Azure.DurableTask.ServiceBus prior to version 2.1.3 did not support DotNetCore. Version 2.0.10 of Microsoft.Azure.DurableTask.ServiceBus has Microsoft.Azure.DurableTask.Core version 2.0.10 as a dependency, the project it set to reference a newer version explictly.
Recommend using Microsoft.Azure.DurableTask.ServiceBus 2.1.3 and not explicitly calling in a different version of Microsoft.Azure.DurableTask.Core.


Upgrading to Bevy 0.12

Not saying the changes I made are the best way to make this work with Bevy 0.12 but they do compile and run. :sweat_smile:

closed by #53


Confusion regarding EMPTY_MIDDLEWARE

Hi, I'm a little confused about how EMPTY_MIDDLEWARE works. It's empty and therefore returns nil. Since it runs last, it makes all of my stacks return nil instead of their current env.
I'm doing the following in my code to work around this:
::Middleware::Runner.const_set :EMPTY_MIDDLEWARE, lambda { |env| env }

Am I misunderstanding something? Doing the whole stack backwards?

I want to bump this since #3 has not been merged.
Also it would be great if the doc can explain what the implementation should look like for the end stack. End stack (e.g. Rails in Rack app) are technically not a middleware, but will be implemented as one here. What should it return? Should it call @app?


duperemove: 0.12 -> 0.13

Description of changes
https://github.com/markfasheh/duperemove/releases/tag/v0.13
Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[x] Tested basic functionality of all binary files (usually in ./result/bin/)
23.11 Release Notes (or backporting 23.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

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

1 package built:

duperemove

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

1 package built:

duperemove

Thanks for contributing to nixpkgs! :3


memory leaks when parsing chinese

var _grammar = new EbnfGrammar (EbnfStyle.W3c).Build ($"id ::= [a-zA-Z\u0100-\uffff_][0-9a-zA-Z\u0100-\uffff_]*", "id");
var _match = _grammar.Match ("张三李四");

Can we add more examples? About the EBNF parser method

var _grammar = new EbnfGrammar (EbnfStyle.W3c).Build (@"
/* base */
id					::= [a-zA-Z_][0-9a-zA-Z_]*
s					::= [
]+

svar				::= id
svar_op0_expr		::= '(' svar_expr ')'
svar_op1_expr		::= (('++'|'--') svar_expr) | (svar_expr ('++'|'--'))
svar_expr			::= s? (svar | svar_op0_expr | svar_op1_expr) s?
", "svar_expr");
var _match = _grammar.Match (" ++ abc");

This code will cause an endless loop and the stack overflow

Thanks for reporting the issue!  Do you know what is leaking?  Are you reusing the grammar or building it every time?

Also, interesting to note that you are using this for other languages.  I've used invariant versions of methods (e.g.  char.ToLowerInvariant()) for case sensitivity as it is faster.  Is that something that needs fixing up for your needs?

Ah, I think this is less of a memory leak vs. a memory hog.  Some of the optimizations are to change ranges into a dictionary lookup, this should have some limit to the range of characters before optimizing that.

One way to get around this for now is to turn off the GrammarOptimizations.CharacterSetAlternations optimization, for example:
myGrammar.Optimizations = Optimizations.All & ~GrammarOptimizations.CharacterSetAlternations;


SVProgressHUD is outdated

Hello there, please upgrade the podfile to use SVProgressHUD to 1.1.3 and change revelant codes in order to sample app work in ios 9+

Thanks @dreampowder!
Side note: The app worked perfectly fine in iOS 9+, the only thing was that the SVProgressHUD had dark text on a dark background because of the little sharedView hacks that were removed in #63.


Disable Responsive Navbar

Hey:
My current application is using the responsive CSS option. Is there any way or are there plans to have the ability to disable the collapsing behaviour on the navbar?
To summarise...
I want to use Bootstrap with the Responsive CSS but do not want the navbar to collapse.
Regards:
John

You can change this by editing responsive.less (or bootstrap-responsive.css) or by overriding the media queries in your own stylesheet. Look for the following (taken from responsive.less):
// TABLETS AND BELOW
// -----------------
@media (max-width: 979px) {

  // UNFIX THE TOPBAR
  // ----------------
  // Remove any padding from the body
  body {
	padding-top: 0;
  }
  // Unfix the navbar
  .navbar-fixed-top {
	position: static;
	margin-bottom: @baseLineHeight;
  }
  .navbar-fixed-top .navbar-inner {
	padding: 5px;
  }
  ......

Manually remove it for now.


Reworking of component endpoints + Open API docs

Description
This PR fixes #3048
Notes for Reviewers
Signed commits

[ ] Yes, I signed my commits.

Codecov Report

Merging #3872 (929a5d7) into master (c8dd33a) will decrease coverage by 7.01%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master    #3872      +/-   ##
==========================================
- Coverage   25.17%   18.15%   -7.02%
==========================================
  Files          63       63
  Lines        4787     4787
==========================================
- Hits         1205      869     -336
- Misses       3221     3661     +440
+ Partials      361      257     -104

Flag
Coverage Δ

gointegrationtests
?

integrationtests
∅ <ø> (?)

unittests
18.15% <ø> (ø)

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

Impacted Files
Coverage Δ

mesheryctl/internal/cli/root/system/status.go
0.00% <0.00%> (-62.83%)
:arrow_down:

mesheryctl/internal/cli/root/system/start.go
1.15% <0.00%> (-55.00%)
:arrow_down:

mesheryctl/internal/cli/root/system/stop.go
0.90% <0.00%> (-39.10%)
:arrow_down:

mesheryctl/internal/cli/root/system/restart.go
1.78% <0.00%> (-32.15%)
:arrow_down:

mesheryctl/internal/cli/root/system/check.go
1.39% <0.00%> (-22.38%)
:arrow_down:

mesheryctl/internal/cli/root/system/logs.go
0.00% <0.00%> (-21.60%)
:arrow_down:

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 c8dd33a...929a5d7. Read the comment docs.

@piyushsingariya There is a failing link check.
@DelusionalOptimist The swagger docs workflow seems to fail due to a missing token. Could you check that?

Yes @navendu-pottekkat fixing it soon for the Open API swagger .


Draft email template for monthly APT Update

Description
Beginning this sprint, the APT curator will start sending out an APT "newsletter" type of email to all users. The idea is to keep in touch with users so that know we are supporting them, provide any interface updates/improvements that have been implemented, maybe give a general heads up of upcoming improvements (we must be very cautious if we mention these), and let them know to use the feedback form if they experience issues/or email the curator).
^ Just brainstorming content ideas. This needs a team discussion.
Tasks:

[ ] discuss what type of content we should include
[ ] design an email template (e.g., design section for monthly updates)
[ ] create a group email (Brad - using NASA email)
[ ] create bi-sprint issue as a reminder to send this email

Acceptance Criteria

[ ] Completed template is ready

@ashleyriddle92 @cw0045 I will need help determining what content we should include and with creating a friendly design. We can discuss at our weekly tagup 5/15.

@bwbaker1 this is in sprint 1. Should this be moved to another sprint?

@wrynearson I moved it.


Fix Skype mixed-content and accessible Emoji warnings

Amending Skype avatar request to use HTTPS
Amending alt text for Skype avatar to "Skype avatar", to fix webpack warning
Adding accessibility fallback for confounded face emoji, to fix webpack warning

Checklist

[x] npm test returns no warnings or errors.

https://emojipedia.org/confounded-face
Take that up with the Unicode consortium 😉


Keys are lowercased in facet distribution

When asking for a facet distribution, the keys are returned lowercased. They should be returned with the case specified by the user.

Hello! Still have the issue (if it's indeed the same issue)

create an index
add:

{ "id": 2,    "title": "Pride and Prejudice", "author": "Jane Austin",  "genre": "Romance" },

update the settings to set genre as attributesForFaceting
get the documents:

[
    {
        "id": 2,
        "title": "Pride and Prejudice",
        "author": "Jane Austin",
        "genre": "Romance"
    }
]

-> OK

search:

{
    "q": "pride",
    "facetsDistribution": ["genre"]
}

{
    "hits": [
        {
            "id": 2,
            "title": "Pride and Prejudice",
            "author": "Jane Austin",
            "genre": "Romance"
        }
    ],
    "offset": 0,
    "limit": 20,
    "nbHits": 1,
    "exhaustiveNbHits": false,
    "processingTimeMs": 2,
    "query": "pride",
    "facetsDistribution": {
        "genre": {
            "romance": 1
        }
    },
    "exhaustiveFacetsCount": true
}

There was an issue when this pr was merged, #901 fixes that.