[Docs] Structure Sage Monorepo docs

What product(s) is this documentation issue for?
Sage Monorepo
Documentation issue

[ ] Reporting a typo
[ ] Reporting a documentation bug
[ ] Documentation improvement
[ ] Documentation feedback

Description
We now have a solution to render markdown files so that they are displayed on GitHub pages.
This solution requires us to structure the documentation in a way that is intuitive to the users of the doc.
The goal of this ticket is to come up with an initial structure, potentially with placeholder pages and sections, that we can then fill in. In particular, this work should results in clear guidelines on how to write new documentation content, e.g. how to format links, where to place markdown files, etc.
Is there a specific documentation page you are reporting?
No response
Anything else?
There are three places that may include relevant documentation that we may want to include in the doc site:

Workspace-level README
Project-level READMEs
Pages in the docs/ folder

Code of Conduct

[X] I agree to follow this project's Code of Conduct

First proposed change: categories
I want to first focus on the different categories the docs should have.  We currently have the following:

If I were to put myself in someone else's shoes, these categories may not be as straightforward to navigate through.  Taking inspiration from Angular, I propose we have the following instead:

where:

Home - showcases the main features of the sage-monorepo and what projects it currently supports
Getting Started - introduces the main technologies of the monorepo (Angular, Nx, Devcontainer) and how one can set up locally or on a remote host
Developers Guide - the bulk of the docs; where one can find out to monitor the stack, collect logs, update dependencies, common issues and how to debug, etc.  This category can then be split into project-level guides, e.g.:

OC: add a new Angular library, add a new Angular component, edit styles/themes, add tests, etc.
schematic: add a new API, etc.
iAtlas: ...
Agora: ...

Reference - cheat sheets
Updates and Releases - changelog, release notes, etc.
Contributors Guide: walks through how one can contribute to the monorepo (bug report, feature request, adding a feature, updating the docs, etc)

In your opinion, would this new structure be more intuitive? @tschaffter @rrchai @mdsage1

This is a great start! Here are some suggestions:

Developers Guide: I would suggest to structure it by programming languages, framework and technologies rather than projects. This structure would better support the creation of standard and the reuse of existing docs.

Contributing Guide: Could this section be included as part of the Developers Guide?

Products: Section to introduce the different products developed in the monorepo (OpenChallenges, Schematic, etc.) + group the documentation of their different components (project-level README)?

Reference: The term may be a bit too vague. Maybe Resources? (better?) What else could fall in this category in the short term? Videos? The Code Quality dashboard?

Updates and Releases: Maybe split the two to be more specific? While releases information would typically be captured by GitHub releases, this seems difficult to provide project-specific release information in the context of a monorepo. So I'm all in for placing releases information in the docs. Updates sounds a bit vague. What about "What's New?" or "Feed"?

"Reference" is a common category in other docs, at least the ones I've come across.
It's generally where one could quickly find all the available commands to run, or a list of all of the available class libraries, object models, etc.  I think for our purpose, "Reference" could be a good place to put our "cheat sheets".

While releases information would typically be captured by GitHub releases, this seems difficult to provide project-specific release information in the context of a monorepo.

You're right - I'll go ahead and remove that!

Contributing Guide: Could this section be included as part of the Developers Guide?

Updates sounds a bit vague. What about "What's New?" or "Feed"?

Hm, what do you think about removing these as categories and using them as sections instead?  Perhaps for the Home category? For example:

Added to Sprint 24.2


Geolocation capture issue

In case we want to capture current GPS location in enketo forms, then the latitude and longitude field being editable dilutes the capturing of current location as it can be modified by the person submitting the form.
Is there any way to make the GPS coordinates non editable

Not yet, but some day we will (be able to use a calculation without user input). There is a spec for it.
Until then, the best you can do is use the appearance 'hide-input'.

How to use the appearance hide input?
On Tue, May 14, 2019, 7:53 PM Martijn van de Rijdt<EMAIL_ADDRESS>wrote:

Not yet, but some day we will (be able to use a calculation without user
input). There is a spec for it.
Until then, the best you can do is use the appearance 'hide-input'.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/enketo/enketo-website/issues/80?email_source=notifications&email_token=AMCHG6KAV4FQWAOH55RSZMLPVLDPJA5CNFSM4HMZIRN2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODVLURZQ#issuecomment-492259558,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AMCHG6LFRV7XE2AHMICZNITPVLDPJANCNFSM4HMZIRNQ
.

Got it.
Thanks for the input!
On Tue, May 14, 2019, 7:53 PM Martijn van de Rijdt<EMAIL_ADDRESS>wrote:

Not yet, but some day we will (be able to use a calculation without user
input). There is a spec for it.
Until then, the best you can do is use the appearance 'hide-input'.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/enketo/enketo-website/issues/80?email_source=notifications&email_token=AMCHG6KAV4FQWAOH55RSZMLPVLDPJA5CNFSM4HMZIRN2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODVLURZQ#issuecomment-492259558,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AMCHG6LFRV7XE2AHMICZNITPVLDPJANCNFSM4HMZIRNQ
.


Supporting types missing for Typescript

What happened?
This page https://www.pulumi.com/registry/packages/azure-native/api-docs/insights/component/#supporting-types does not provide info for supporting types.

Expected Behavior
I expect to see the supporting types information.
Example:

IngestionMode | Indicates the flow of the ingestion. | 'ApplicationInsights''ApplicationInsightsWithDiagnosticSettings''LogAnalytics'

From: https://learn.microsoft.com/en-us/azure/templates/microsoft.insights/components?pivots=deployment-language-bicep#applicationinsightscomponentproperties
Steps to reproduce
Visit the link listed above. See that there are no supporting types shown.
Output of pulumi about
N/A
Additional context
No response
Contributing
Vote on this issue by adding a 👍 reaction.
To contribute a fix for this issue, leave a comment (and link to your pull request, if you've opened one already).

just an observation - it looks like this happens for types that are listed as part of a union type.


fix(snapshot_cleaning): set ref_id to None to prevent postgresql crash

Fix bug on the clear-snapshot endpoint

We should understand why the foreign key issue did not cause an exception with sqlite, and if we can configure it to have the same behaviour as postgresql, otherwise we will have the same problem in the future.


Cannot install dependencies

When I install the plugin, all goes right, but when I start the configuration routine, after inserting the infill percentage, i click on install dependencies. Then it gives me an error like: Error: Command '/bin/bash -c 'sudo /usr/bin/apt-get update'' returned non-zero exit status 1

Thanks for your feedback. The installation dependency part can be tricky and has only been tested on OctoPi configurations.
First, please make sure you have an active connection to the internet at the time of install. If apt-get is located in another location you may need to create a branch that references the correct apt-get location for your os.


/discord-ban and /discord-unban reason params are not refined

/discord-ban reason param does not actually provide an AuditLogEntry reason
and /discord-unban does not even have said param.

Just checked seems /time-out and /discord-kick also have this lels

Will be fixed in 1.2.0


Broadcast Transaction Error:tx-sapling-active

What does the error means? And how can I solve that?
Looking forward to your reply

@Wangmmx Is this still a problem? Can we close this issue?

Closing as no responsive from @Wangmmx


Missing Build scripts for V1?

I can't see any build scripts for the v1 versions after v1.41-beta release. Why?
https://github.com/mui-org/material-ui/blob/v1.0.0-beta.41/package.json
I am trying to build from repo. Can please help?
https://github.com/mui-org/material-ui/blob/v1.5.1/package.json

Check the packages/material-ui folder out.


Default value not working in v.4.3.0

I'm having problems to get the default value in v4.3.0, the problem is I'm not able to replicate it in the codeSandbox but when I switch to v4.2.0 it's working, I have looked the changes in the lastest version and I saw you added a defaultValue prop
https://github.com/anubra266/choc-autocomplete/blob/932823f7935abe6ef0909594fe4fc58ee058bbf5/src/use-autocomplete.ts#L160
I put console.log(autoCompleteProps.defaultValue) and it's undefined so the return of useParseDefaultValue is undefined as well, as I'm using the input in controlled mode I think it should get value as default value, I think that is the way how this was working before

Hi, can I see your code. @gepd

Hi, can I see your code. @gepd

Im using RHF so the default value is given in
const methods = useForm({ defaultValues: document });

The component is declared like this:
<FormControl isInvalid={methods.formState.errors[name]} mb="3">
  {isEdit && (
    <FormLabel htmlFor={name} mb="0" fontWeight="normal">
      {label}
    </FormLabel>
  )}
  <Controller
    name={name}
    control={methods.control}
    render={({ field: { onChange, onBlur, value } }) => (
      <AutoComplete {...props} freeSolo rollNavigation suggestWhenEmpty openOnFocus isReadOnly={!isEdit}>
        <AutoCompleteInput
          onChange={onChange}
          value={value}
          onBlur={onBlur}
          variant={isEdit ? "filled" : "ghost"}
          autocomplete="off"
        />
        <AutoCompleteList>
          <AutoCompleteGroup title="Title" showDivider>
            {options.map((option, oid) => (
              <AutoCompleteItem key={`option-${oid}`} value={option} textTransform="capitalize">
                {option}
              </AutoCompleteItem>
            ))}
          </AutoCompleteGroup>
        </AutoCompleteList>
      </AutoComplete>
    )}
  />
  <FormErrorMessage>{methods.formState.errors[name]?.message}</FormErrorMessage>
</FormControl>

I know the value is been passing because when I console log the value prop I can see it, but also the same code work without problem in v4.2.0

Ah I get now. The defaultValue overrides the value you're providing to the input.

yes I think so!

I added the defaultValue so you could define tags, beforeHand, I'll change it to defaultTags.

Fixed in 4.3.1

Thanks It's working again!


Svelte - fix: npm run check

Fix:

Adding a bounty on this, for all the recent work for svelte, angular, react, etc.
Thanks @qmonmert

@pascalgrimaud thanks 🙂

After having a quick look, I think we should keep:
"@/*": ["main/webapp/app/*"],

The correct fix should be:
import App from<EMAIL_ADDRESS>
Instead of
import App from<EMAIL_ADDRESS>

Anyway I'm not a Svelte user, I'm letting you decide what's the best :)

@pascalgrimaud indeed, two solutions work, maybe your is best, I update the PR

npm t does not work with your solution @pascalgrimaud 🤔

Need probably to update the vitest config ts:
    alias: [
      { find: '@', replacement: path.resolve(__dirname, 'src/main/webapp/app') },
      { find: '@assets', replacement: path.resolve(__dirname, 'src/main/webapp/assets') },
    ],

you try on your machine, it does not work for me

So let's keep your 1st solution, we'll see later to improve this part

I think I find solution I update

@pascalgrimaud https://opencollective.com/generator-jhipster/expenses/117655 🙂

@qmonmert : approved, thanks for your work


Webview and Ionic Communication

Hello, is there any way to capture a message from the webview?
In React Native, I can do something like the code below, but I couldn't find a solution for this in your library.
const INJECTED_JAVASCRIPT = `(function() {
    window.ReactNativeWebView.postMessage(JSON.stringify({key : "value"}));
})();`;

<WebView
  source={{ uri: 'https://reactnative.dev' }}
  injectedJavaScript={INJECTED_JAVASCRIPT}
  onMessage={(event) => {
       const data = JSON.parse(event.nativeEvent.data);
       alert(data.key);
  }}
/>;

Hi.
This plugin doesn't know how to work with postMessage.
Here is a similar issue https://github.com/Cap-go/capacitor-inappbrowser/issues/84

This could be done pretty fast. You can support it here: https://console.algora.io/bounties/new

A workaround that I developed in cordova webview was reading local storage.  Now I preferred using directly a websocket (using pusher)

Hello, I needed the feature for my last project, so I made it.
You can now post messages from inappbrowser like that:
window.mobileApp.postMessage({ "detail": { "message": "myMessage" } })

And listen to it from the Capacitor webview with
InAppBrowser.addListener('messageFromWebview', (event) => {
  console.log('messageFromWebview', event)
})

Both ways were implemented.
To do the reverse from Capacitor webview
    InAppBrowser.postMessage({
      detail: {
        status: 'ok4',
      },
    })

And listen in the inappbrowser:
  window.addEventListener('messageFromNative', (event: CustomEvent) => {
    const message = event.detail
    console.log('Received message from native:', message)
    // Handle the message here
    // For example, you could update some state or trigger an action
  })


Rabbitstream API site url error

Description
I think the problem with the Sflix provider is the API link for rabbitstream, the API does not respond, the netlify site url where the API is hosted is inaccessible.

Provider
Sflix
Reproduction steps
Open App
Select Sflix provider
Choose a movie
Click "Watch now" Button
Operating system
Android 14
Logs
No response
Please confirm the following

[X] I have searched the existing issues and this is a new ticket, NOT a duplicate or related to another open issue.
[X] If related to a provider, I have checked the site and it works, but not the app.

Yes, I know, thanks
The developer who created this API said that it was removed because too many people were using it and that it was deployed with a free subscription, so it had a request limit.
When I will have news, I will try to update it

Thank you very much for reply, but if the API is Open Source, he can give us either the API link so we can do what is necessary to host it.

I don't know him, and I don't know if its OpenSource also. I saw this on Discord

Ok

Fixed on Release 1.6.12


feat: (IAC-925) Added an option in IAC AWS to use a single AZ

Improvements
The 'enable_multi_zone' flag has been implemented to enable customers to deploy EKS worker nodes across multiple availability zones in AWS. Additionally, the option to deploy EKS worker nodes into a single availability zone has been added.
Changes

When 'enable_multi_zone' is set to 'true', the EKS cluster will be deployed across multiple availability zones defined in the 'module.vpc.private_subnets' list.
When 'enable_multi_zone' is set to 'false', the EKS cluster will only be deployed in the first availability zone in the 'module.vpc.private_subnets' list.

Testing
See internal ticket for details and test artifacts:

Scenario
Provider
.tfvars file
k8s Version
Multi-Zone Flag 'false' : Results
Multi-Zone Flag 'true' : Results

1
AWS
sample-input.tfvars
1.23
Using the AWS Console - validated that all EKS nodes are deployed in the same zone.
Using the AWS Console - validated that all EKS nodes are deployed in multiple zones.

2
AWS
sample-input-connect.tfvars
1.23
Using the AWS Console - validated that all EKS nodes are deployed in the same zone.
Using the AWS Console - validated that all EKS nodes are deployed in multiple zones.

3
AWS
sample-input-custom-data.tfvars
1.23
Using the AWS Console - validated that all EKS nodes are deployed in the same zone.
Using the AWS Console - validated that all EKS nodes are deployed in multiple zones.

4
AWS
sample-input-defaults.tfvars
1.23
Using the AWS Console - validated that all EKS nodes are deployed in the same zone.
Using the AWS Console - validated that all EKS nodes are deployed in multiple zones.

5
AWS
sample-input-gpu.tfvars
1.23
Using the AWS Console - validated that all EKS nodes are deployed in the same zone.
Using the AWS Console - validated that all EKS nodes are deployed in multiple zones.

6
AWS
sample-input-ha.tfvars
1.23
Using the AWS Console - validated that all EKS nodes are deployed in the same zone.
Using the AWS Console - validated that all EKS nodes are deployed in multiple zones.

7
AWS
sample-input-minimal.tfvars
1.23
Using the AWS Console - validated that all EKS nodes are deployed in the same zone.
Using the AWS Console - validated that all EKS nodes are deployed in multiple zones.

8
AWS
sample-input-singlestore.tfvars
1.23
Using the AWS Console - validated that all EKS nodes are deployed in the same zone.
n/a

Keeping this PR open for now, but pulling the changes/diffs into a new working branch (IAC-1174) based off staging. This seems easier than trying to rebase it.

Superseded by #234


Using Rust datatypes via RPC

Hi,
I'm just wondering if you could point me in the right direction, on how to use non-standard
datatypes, such as those which are specific to Rust, such as bigints, via RPC.
Would I need to represent the data as say an array of bytes, then use
Rusts transmute feature to convert to a bigint somehow, or is there a better way?
cheers
Chris

Using the Cap'n Proto Data type sounds sensible to me. However, I would shy away from using transmute().  It looks like BigInt has functions like from_bytes_le() that would be better suited for that purpose.


no function clause matching cowboy_req:do_reply

Trying to migrate from cowboy 2.0 to 2.5.0
using cowboy middleware trying to make 302 redirect + setting cookies
-define(GOTO,         <<"/">>).
-define(RESPONSE_CACHE_CONTROL(HttpStatus, Req, LocationURL),
  cowboy_req:reply(HttpStatus, #{
    <<"Location">> => LocationURL,
    <<"Cache-Control">> => <<"no-cache, no-store, max-age=0, must-revalidate">>
  }, Req)).

{stop, ?RESPONSE_CACHE_CONTROL(302, set_cookie(SID, CowReq), ?GOTO)}

set_cookie(Sid, CowReq) ->
  {ok, [HttpOpts|_Rest]} = application:get_env(?APP, http_opts),
  cowboy_req:set_resp_cookie(?SECURITY_SESSION_ID_KEY, Sid, maps:from_list(HttpOpts), CowReq).

but i'm getting the error

15:00:40.385 [error] CRASH REPORT Process <0.19142.0> with 0 neighbours crashed with reason: no function clause matching cowboy_req:do_reply(302, #{<<"Location">> => <<"/">>,<<"content-length">> => <<"0">>}, <<>>, #{http_only => true,path => "/",resp_cookies => #{<<"skey">> => [<<"skey">>,<<"=">>,<<"2JP...">>,...]}}) line 771
15:00:40.397 [error] Lager event handler error_logger_lager_h exited with reason {'EXIT',{{case_clause,[web_clients_listener,<0.19141.0>,1,<0.19142.0>,function_clause,[{cowboy_req,do_reply,[302,#{<<"Location">> => <<"/">>,<<"content-length">> => <<"0">>},<<>>,#{http_only => true,path => "/",resp_cookies => #{<<"skey">> => [<<"skey">>,<<"=">>,<<"2JPZ50AU90V6_153986440378369">>,<<"; Version=1">>,<<>>,<<>>,[<<"; Path=">>,<<"/api/p24">>],<<>>,<<>>]}}],[{file,"/home/jeka///test/****/_build/default/lib/cowboy/src/cowboy_req.erl"},{line,771}]},{web_middleware,...},...]]},...}}

Could you please show me the right direction to resolve this issue ?
p/s: everything is working fine on cowboy 2.0

It was working fine before the final 2.0 release you mean, surely? The position of the Opts argument of set_resp_cookie was changed before 2.0, it's now Req, Opts instead of Opts, Req.

whoa! you're tottaly right
thank you
the issue was fixed


Attempt to troubleshoot pytype issue by disabling cache on examples.

Trying to debug mysterious (and locally unreproducible) pytype failures in our CI tests.

Codecov Report

Merging #2628 (5f9f503) into main (722838c) will decrease coverage by 0.09%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             main    #2628      +/-   ##
==========================================
- Coverage   79.57%   79.48%   -0.10%
==========================================
  Files          50       50
  Lines        5313     5264      -49
==========================================
- Hits         4228     4184      -44
+ Misses       1085     1080       -5

Impacted Files
Coverage Δ

flax/core/nn/linear.py
42.34% <0.00%> (-0.52%)
:arrow_down:

flax/struct.py
78.12% <0.00%> (-0.34%)
:arrow_down:

flax/core/scope.py
89.77% <0.00%> (-0.07%)
:arrow_down:

flax/core/lift.py
95.81% <0.00%> (-0.07%)
:arrow_down:

flax/linen/normalization.py
97.08% <0.00%> (-0.05%)
:arrow_down:

flax/core/frozen_dict.py
96.19% <0.00%> (-0.04%)
:arrow_down:

flax/linen/summary.py
99.00% <0.00%> (-0.01%)
:arrow_down:

flax/traverse_util.py
99.01% <0.00%> (-0.01%)
:arrow_down:

flax/linen/recurrent.py
100.00% <0.00%> (ø)

flax/training/train_state.py
0.00% <0.00%> (ø)

... and 4 more

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

Its so weird, I created this PR #2612 with a dummy change and it passed, but main is currently failing.
Seems to be random?

yeah it's driving me crazy.  I'm trying one long-shot fix here


Autos should display returned results of last function

Currently autos window seems to be totally empty on NTVS. In many languages usually it displays return values of last called functions and some autos/locals when you step over.
See this C++ sample screenshot

@digeff Does this work for the inspector protocol?

We should consider implementing this in the TypeScript LS. i.e. implement IVsLanguageDebugInfo.GetProximityExpressions


For component data, allow widget data to be entered directly

Currently, the widget data is obtained through query. The page data entry should allow widget data to be entered directly, like the page show direct.
/pages/show/direct?composition=theone&widgetname=BlackbootHeader&widgetdata={"title":"CMS","mainLinks":[{"link":"/","text":"HOME"}],"sideLinks":[{"link":"/contact","text":"CONTACT"},{"link":"/about","text":"ABOUT"}]}

User can enter widget data by value or by query


Fix object replacement values being cast as strings

Description
New regression uncovered today:
With text content:
+  "To create this market, countries will be removed from your [object Object],, ,[object Object],, and ,[object Object] markets, and those markets will be deleted. Any settings for these markets will also be permanently deleted."
To contain string:
-  "countries will be removed from your Canada, United States, and United Kingdom markets"

I assumed all objects were going to be valid React elements when I refactored the translation code, but I was wrong as it could be an array of React nodes.
So I fixed the tests and the replacement logic, then I made sure tests were passing with both the old and the new implementation.
I also added a bunch of tests for the available format regexes, to make sure we do not break them if we ever update them.
Type of change

[x] @shopify/react-i18n Patch: Bug (non-breaking change which fixes an issue)

Checklist

[x] I have added a changelog entry, prefixed by the type of change noted above (Documentation fix and Test update does not need a changelog as we do not publish new version)

Is it me or is the CLA status glitched out?

Is it me or is the CLA status glitched out?

Is it me or is the CLA status glitched out?

Yeah it does that sometimes. I know you're a shopify employee so I'll use my admin-ness to merge this.


Backspace not available in Search Bar anymore

Hi,
thanks for the updated script for Nautilus 47!
Like the title says, with the updated version of your script the backspace button is not available in search bar anymore.
Any chance to fix this?

Thanks for the observation, I had not found that error.
I hope I can fix it before the final version of fedora is released, if you can help me find a solution, it would be perfect.
Thanks

Steps to reproduce the error:

Click on search bar.
Search something.
Press backspace.
Instead of deleting the last character, search closes, and the main window goes one level up.

Another error:

Create a new folder.
Write something.
Press backspace.
Instead of deleting the last character, the main window goes one level up, meanwhile the new folder pop-up keeps open.

@juanmah  I hope I can fix it before the final version of fedora is released, if you can help me find a solution, it would be perfect.
Thanks


Update topical-software.rst

Add battery modeling software "PyBaMM" to the list of topical software

Merged, thanks @tinosulzer


Question:

is there a 2.8.1 of the Project afternight bloxstrap fork? (And idk how to fix the system collection glitch)

This is the system collection thing

Can you send the log file please?

sure might take some time for me to get back sorry about it lol
On Thu, Oct 31, 2024 at 6:23 AM pizzaboxer @.***> wrote:

Can you send the log file please?
—
Reply to this email directly, view it on GitHub
https://github.com/bloxstraplabs/bloxstrap/issues/3576#issuecomment-2449515581,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/BMRY73QMLUDKAUFNFKBMGFTZ6IAJNAVCNFSM6AAAAABQ5GCAJ2VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDINBZGUYTKNJYGE
.
You are receiving this because you authored the thread.Message ID:
@.***>

@pizzaboxer  is there a FastFlag to bring back the old HP bar? this new one sucks, since the Roblox menu function was removed.
Thank you


Add tests to functions

Create tests for individual functions. Check that profile pulls the correct data, and raises appropriate errors.

Added tests for fetch function with PR #91. Still working on setting up codecov to report coverage of the tests.


Fixes + Enhancements

IMPORTANT: Where possible all PRs must be linked to a Github issue

Fixes Incorrect Password Flagged as Internet Connection Issue , Fixes #2535
Enhance Config Sync - Implement Batch Syncing, Closes #2529

Engineer Checklist

[x] I have written Unit tests for any new feature(s) and edge cases for bug fixes
[x] I have added any strings visible on UI components to the strings.xml file
[x] I have updated the  CHANGELOG.md file for any notable changes to the codebase
[x] I have run ./gradlew spotlessApply and ./gradlew spotlessCheck to check my code follows the project's style guide
[x] I have built and run the FHIRCore app to verify my change fixes the issue and/or does not break the app
[x] I have checked that this PR does NOT introduce breaking changes that require an update to Content and/or Configs? If it does add a sample here or a link to exactly what changes need to be made to the content.

Code Reviewer Checklist

[ ] I have verified Unit tests have been written for any new feature(s) and edge cases
[ ] I have verified any strings visible on UI components are in the strings.xml file
[ ] I have verifed the CHANGELOG.md file has any notable changes to the codebase
[ ] I have verified the solution has been implemented in a configurable and generic way for reuseable components
[ ] I have built and run the FHIRCore app to verify the change fixes the issue and/or does not break the app

Counterpart to 0.2.4 PR https://github.com/opensrp/fhircore/pull/2538


Add completed page / order number

Start thinking about how the school can implement the order number system like big restaurant chains.

Switched to name basis


Site Timezone Selection

Issue type: Feature Request

When adding a site it would be nice to have a Timezone field that could be selected. This would then document the local Timezone for the site. Additionally if selected below it could display the sites current local time as calculated from the servers off-set.
For example, the server is set to PST and the time is 0700 and the date is June-16-2017. Site timezone is selected as EST.
The selection field shows:
US/Eastern
Below the selection field is actual time:
10:00 US/Eastern June-16-2017

I like it, but we need to figure out how to cultivate a list of timezones for a user to choose from. Something like django-timezone-field should work, although it would introduce a new dependency, and I'm not sure how we'd maintain timezone choices over time.

What about if on the Django admin portal you exposed all the Timezones as tick boxes. Then as an admin, you select the ones you want to apply to you. In our case, as a global company we really only operate in about 14 timezones world wide. So I'd "tick" those 14 which would then be presented to me in the drop-down list on the add site front end?

I'm might be totally off base here but what's wrong with setting it to UTC
which should be a gold standard for local times worldwide. I don't see any
practical use case here other than a convenience.
On Fri, Jun 16, 2017 at 07:56 Tim Franklin<EMAIL_ADDRESS>wrote:

What about if on the Django admin portal you exposed all the Timezones as
tick boxes. Then as an admin, you select the ones you want to apply to you.
In our case, as a global company we really only operate in about 14
timezones world wide. So I'd "tick" those 14 which would then be presented
to me in the drop-down list on the add site front end?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/digitalocean/netbox/issues/1283#issuecomment-309048361,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ARj55enpZY8EY37vqKg1aVTYvXthpgRQks5sEpe0gaJpZM4N8iZ0
.

--
Roman N. Kutsel

I think the proposal is simply to track the local time for each site, so that it can be referenced in maintenance notices and whatnot.

That exactly correct. Nothing is more fun then to have an operations team do a page out to on-call only to find out it's for a site that is past business hours with no contacts.

Makes sense. This feature will definitely come in handy.
On Mon, Jun 19, 2017 at 07:01 Tim Franklin<EMAIL_ADDRESS>wrote:

That exactly correct. Nothing is more fun then to have an operations team
do a page out to on-call only to find out it's for a site that is past
business hours with no contacts.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/digitalocean/netbox/issues/1283#issuecomment-309448725,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ARj55RW82MlyY6O6y22rO8fImbvRl9Fnks5sFn9DgaJpZM4N8iZ0
.

--
Roman N. Kutsel

It would nice to add "maintenance window" field for a site model.
For example 11 PM - 6AM.

Implemented in b20258c

Hi.
First, give thanks for the contribution you have made with Netbox.
We are implementing it in the company and we would like to change the TIME_ZONE in the configuration.py file to adjust it to our time use. We are in the Canary Islands - Spain and our time use is UTC +01: 00.
I have placed TIME_ZONe = 'UTP +01: 00' but it does not work. Can you tell me what is the correct way to assign this parameter?
Greetings and thank you.

Resolved.
TIME_ZONE = "Atlantic/Canary"
Thank you.


[AutoPR servicefabric/data-plane/servicefabric/resource-manager] chore: jsonfmt servicefabric

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

(message created by the CI based on PR content)


Missing dependency - file

Missing dependency
Missing the package file.
That cause to the function import_helper to fail
To Reproduce
Steps to reproduce the behavior:

Create VM of Ubuntu Server 22.04.3 LTS with 50GB and 16GB Ram
2.run the following commands:

 git clone --recursive -j 8 https://github.com/e-m-b-a/embark.git
cd embark
sudo git submodule update --remote --merge
cd emba
git checkout master && git pull
cd ..
sudo ./installer.sh -d

Expected behavior
Install everything
Desktop (please complete the following information):

OS: [e.g. Ubuntu Server 22.04.03 Lts]

Additional context
Also the installation still doesn't work in clean installation on clean VM.

This issue should be already resolved from EMBA side. If not, please re-open it


Option to exclude types of materializations from documentation rules

Describe the feature
The documentation rules, such as fct_undocumented_models, apply to all models including ephemeral models. As ephemeral models are used for lightweight transformation or selections, typically directly from source tables, their documentation is less of a priority than other materialization types.
Would create a project level var than can be referenced in fct_undocumented_models, e.g.
final as (

    select
        resource_name,
        model_type

    from all_resources
    where not is_described and resource_type = 'model' and not materialized = '{{ var("excluded_materialization") }}'

)

Describe alternatives you've considered
Either switching off fct_undocumented_models or going through all the ephemeral models and adding documentation by hand.
Additional context
Unsure if there are other rules it may be sensible to add an option to exclude ephemeral models from.
Who will this benefit?
Users with significant numbers of undocumented ephemeral tables, but otherwise good documentation coverage.
Are you interested in contributing this feature?
Yes, I can contribute if there is community interest in this feature. Would allow any type of materialization to be specified for exclusion.

Hi @evanodell . Thanks for raising this issue. I had a chat with the team but we're not convinced that excluding ephemeral models would fit every use case. Some people might want to make sure that they are documented as well.
As it seems that it might quite specific to your situtation I would recommend:

deactivating the fct_undocumented_models model in the project evaluator package
creating your own version of fct_undocumented_models (or with an other name), as a slight change from the OOTB version.


ARROW-8609: [C++] fix orc jni crash

check if arrow buffer is null before passing to the constructor
Signed-off-by: Yuan Zhou<EMAIL_ADDRESS>

Thanks for working on this. Could you please enable the ORC tests in the jni build (just use java/adapter/orc)?

@github-actions autotune everything

We also need to re-enable the disabled ORC test case.

The lint failure is unrelated and should be fixed on master.

The ORC test is passing. Thanks @zhouyuan for fixing it!


Intel aero platform error

Describe the bug
Current intel aero board uploads *.elf file instead *.px4.
To Reproduce
Steps to reproduce the behavior:
make (intel platform def) upload.
Expected behavior
uploads *.px4 file and flashing.
Log Files and Screenshots
Always provide a link to the flight log file:

Download the flight log file from the vehicle (tutorial).
Share the link to a log showing the problem on PX4 Flight Review.

Add screenshots to help explain your problem.
Drone (please complete the following information):
Intel aero RTF.
Additional context
Add any other context about the problem here.

https://github.com/PX4/Firmware/blob/master/boards/intel/aerofc-v1/CMakeLists.txt#L43


[SPARK-29957][TEST][test-java11] Reset MiniKDC's default enctypes to fit jdk8/jdk11

What changes were proposed in this pull request?
Hadoop jira: https://issues.apache.org/jira/browse/HADOOP-12911
In this jira, the author said to replace origin Apache Directory project which is not maintained (but not said it won't work well in jdk11) to Apache Kerby which is java binding(fit java version).
And in Flink: https://github.com/apache/flink/pull/9622
Author show the reason why hadoop-2.7.2's  MminiKdc failed with jdk11.
Because new encryption types of es128-cts-hmac-sha256-128 and aes256-cts-hmac-sha384-192 (for Kerberos 5) enabled by default were added in Java 11.
Spark with hadoop-2.7's MiniKdcdoes not support these encryption types and does not work well when these encryption types are enabled, which results in the authentication failure.
And when I test hadoop-2.7.2's minikdc in local, the kerberos 's debug error message is  read message stream failed, message can't match.
Why are the changes needed?
Support jdk11 with hadoop-2.7
Does this PR introduce any user-facing change?
NO
How was this patch tested?
Existed UT

cc @wangyum @dongjoon-hyun
This way can fix error happened in https://github.com/apache/spark/pull/26533#issuecomment-555266324

Can one of the admins verify this patch?

Thank you @AngersZhuuuu

ok to test

Can one of the admins verify this patch?

ok to test

Test build #114072 has started for PR 26594 at commit e9ef3ba.

Merged build finished. Test PASSed.

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

Test build #114072 has finished for PR 26594 at commit e9ef3ba.

This patch fails due to an unknown error code, -9.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

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

Test build #114083 has started for PR 26594 at commit e9ef3ba.

Merged build finished. Test PASSed.

Test build #114083 has finished for PR 26594 at commit e9ef3ba.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

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

strange..I can run these failed UT in local with jdk8/jdk11 success. But failed in Jenkines...

Retest this please.

Test build #114327 has started for PR 26594 at commit e9ef3ba.

Merged build finished. Test PASSed.

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

Retest this please.

For this. I can pass failed UT by run it alone, but will failed when run all UT together.
Seems unit tests affect each other in terms of environment variables. I am checking on this.

@AngersZhuuuu . I also tested your PR locally.
That's the reason I re-trigger this. If this fails on my local environment, I didn't trigger an useless run. 😄

BTW, if you can find a corresponding Apache Hadoop JIRA issue, that will be more persuasive
for this PR.

BTW, if you can find a corresponding Apache Hadoop JIRA issue, that will be more persuasive
for this PR.

Hadoop jira: https://issues.apache.org/jira/browse/HADOOP-12911
In this jira, the author said to replace origin Apache Directory project which is not maintained (but not said it won't work well in jdk11) to Apache kerby which is java binding(fit all java version).
And in Flink: https://github.com/apache/flink/pull/9622
Author show the reason why hadoop-2.7.2's MminiKdcfailed with jdk11. Because new encryption types ofes128-cts-hmac-sha256-128andaes256-cts-hmac-sha384-192` (for Kerberos 5) enabled by default were added in Java 11.
And when I test hadoop-2.7.2's minikdc in local, the kerberos 's debug error message is  read message stream failed, message can't match.

Please summarize them into the PR description~

Please summarize them into the PR description~

Done

Merged build finished. Test PASSed.

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

Test build #114329 has started for PR 26594 at commit 3ec2891.

Hmm. It seems that KafkaDelegationTokenSuite fails again in Jenkins.
[info] org.apache.spark.sql.kafka010.KafkaDelegationTokenSuite *** ABORTED *** (189 milliseconds)
[info]   java.io.IOException: Could not configure server because SASL configuration did not allow the  ZooKeeper server to authenticate itself properly: javax.security.auth.login.LoginException: kerberos.example.com: Name or service not known

I tested this only at Mac with AdoptOpenJDK8 and 11. How about your environment? I'm wondering if we need to validate this in the linux or not.

Test build #114327 has finished for PR 26594 at commit e9ef3ba.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

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

I tested this only at Mac with AdoptOpenJDK8 and 11. How about your environment? I'm wondering if we need to validate this in the linux or not because of the following error message.

I test this in Mac with Oracle Jdk8 and jdk11. I can pass this UT when run it alone.
I can test this in Debian.
My guess is that there were other tests that affected the environment variables

Ya. Let me know if you found the root cause. Thanks.

Test build #114329 has finished for PR 26594 at commit 3ec2891.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

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

Test build #114339 has started for PR 26594 at commit 2e282f2.

Merged build finished. Test PASSed.

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

Test build #114340 has started for PR 26594 at commit a224fad.

Merged build finished. Test PASSed.

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

Test build #114339 has finished for PR 26594 at commit 2e282f2.

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

Merged build finished. Test PASSed.

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

Test build #114340 has finished for PR 26594 at commit a224fad.

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

Merged build finished. Test PASSed.

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

@dongjoon-hyun
@wangyum fix this in https://github.com/apache/spark/pull/26533/commits/50a11ecc445bc0398b86193efc2ade7e22498a34
And i checked the code , in hadoop-2.7 MiniKdc, after start all minikdc server, it will refresh Config which is related to kerberos.
https://github.com/apache/hadoop/blob/b165c4fe8a74265c792ce23f546c64604acf0e41/hadoop-common-project/hadoop-minikdc/src/main/java/org/apache/hadoop/minikdc/MiniKdc.java#L474-L480
But in Hadoop-3.2.0 's MiniKDC, when start minikdc, it won't call refresh() method.
Since our all UT are tested in one process. So other UT about kerberos impact Kafka's UT about  Kerberos.
We have two way to fix this.

Add config to refresh Krb5Conf in jaas config refreshKrb5Config
Call refresh method in KafkaTestUtils.setUpMiniKdc() like

    // refresh the config
    val classRef: Class[_] = if (System.getProperty("java.vendor").contains("IBM")) {
      Utils.classForName("com.ibm.security.krb5.internal.Config")
    } else {
      Utils.classForName("sun.security.krb5.Config")
    }
    val refreshMethod = classRef.getMethod("refresh", Array.empty[Class[_]]: _*)
    refreshMethod.invoke(classRef, Array.empty[AnyRef]: _*)

Build finished. Test PASSed.

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

Test build #114443 has started for PR 26594 at commit dfdf589.

Merged build finished. Test PASSed.

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

Test build #114443 has finished for PR 26594 at commit dfdf589.

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

Merged build finished. Test PASSed.

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

As a general suggestion the mentioned delegation token test would be good to get tested with mvn, sbt and with a local machine (and still it's possible that it won't work on some environments).

Test build #114570 has started for PR 26594 at commit 7ab6192.

Merged build finished. Test PASSed.

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

Test build #114570 has finished for PR 26594 at commit 7ab6192.

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

Merged build finished. Test PASSed.

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

retest this please

Test build #114578 has started for PR 26594 at commit 7ab6192.

Merged build finished. Test PASSed.

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

Test build #114578 has finished for PR 26594 at commit 7ab6192.

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

Merged build finished. Test PASSed.

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

Is this only relevant for Kafka?

Currently only kafka UT meet these problem.

Is this only relevant for Kafka?

I've added this initially used only by Kafka which can be re-used at any other places when needed.

I've added this initially used only by Kafka which can be re-used at any other places when needed.

I am working on enable spark thrift server support proxy client user. Will use MiniKDC when build UT.

Couple of comments not yet resolved, can you either resolve them or add your points?

Couple of comments not yet resolved, can you either resolve them or add your points?

Solve this, any more advise?

This and this for example.

This and this for example.

// scalastyle:on has been added.
For second problem, I looked at the kerberos about API， don't have direct api to check this.
Customize this is safe , I think.
https://www.oracle.com/technetwork/java/javase/11-relnote-issues-5012449.html#JDK-8014628
https://issues.apache.org/jira/browse/KAFKA-7338

I've asked // scalastyle:off println...
The second case I think it's fine, thanks.

I've asked // scalastyle:off println...
The second case I think it's fine, thanks.

==  sorry ....., updated

Not yet seen execution with [test-maven], is it not possible?

Test build #114627 has started for PR 26594 at commit 476d60e.

Merged build finished. Test PASSed.

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

Not yet seen execution with [test-maven], is it not possible?

Will do this after current build test.

Test build #114627 has finished for PR 26594 at commit 476d60e.

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

Merged build finished. Test PASSed.

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

Not yet seen execution with [test-maven], is it not possible?

Could you help to trigger reresr?

retest this please

Test build #114642 has started for PR 26594 at commit 476d60e.

Merged build finished. Test PASSed.

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

Test build #114642 has finished for PR 26594 at commit 476d60e.

This patch fails due to an unknown error code, -9.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

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

retest this please

Merged build finished. Test PASSed.

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

Test build #114658 has started for PR 26594 at commit 476d60e.

Test build #114658 has finished for PR 26594 at commit 476d60e.

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

Merged build finished. Test PASSed.

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

retest this please

Test build #114677 has started for PR 26594 at commit 476d60e.

Merged build finished. Test PASSed.

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

Test build #114677 has finished for PR 26594 at commit 476d60e.

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

Merged build finished. Test PASSed.

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

I think the test comment is not yet resolved: https://github.com/apache/spark/pull/26594/files#r350680194

Test build #114848 has started for PR 26594 at commit b782e99.

Merged build finished. Test PASSed.

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

Test build #114850 has started for PR 26594 at commit 10a0006.

Merged build finished. Test PASSed.

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

@gaborgsomogyi @wangyum  How about current way ?
If don't have a libdefaults section , we add one at krb5.conf's header

Test build #114851 has started for PR 26594 at commit 03f7fb1.

Test build #114848 has finished for PR 26594 at commit b782e99.

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

Merged build finished. Test PASSed.

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

Merged build finished. Test PASSed.

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

Test build #114850 has finished for PR 26594 at commit 10a0006.

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

Merged build finished. Test PASSed.

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

Test build #114851 has finished for PR 26594 at commit 03f7fb1.

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

Merged build finished. Test PASSed.

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

The approach looks good, there are still no tests...

The approach looks good, there are still no tests...

Here is in UT ..., I really don't know how to add UT for this, so handle it in if condition when there are no libdefaults in krb5.conf

Test build #114884 has started for PR 26594 at commit e252dd0.

Merged build finished. Test PASSed.

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

Test build #114884 has finished for PR 26594 at commit e252dd0.

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

Merged build finished. Test PASSed.

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

Test build #114926 has started for PR 26594 at commit 223533f.

Merged build finished. Test PASSed.

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

Test build #114926 has finished for PR 26594 at commit 223533f.

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

Merged build finished. Test PASSed.

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

Thanks, everyone! This is a great fix!


Exported svg is too large 1.3MB, while 2x png is just 70+KB

Expected Behavior
svg should be reasonabely small than png, jpgs.
Actual Behavior

Browser:

 Google Chrome Version 72.0.3626.96 (Official Build) (64-bit)
OS: macOS 10.14.2

Code Snippet (If Applicable)

code here

Shall I take up this issue? @mfix22

@Joel-hanson go for it! 👍
I just don't know how much there is to do here. We use dom-to-image to create PNGs and SVGs, but curious as to what you come up with.

#683

Closed with #683


fix attribute key import

make sure we use the correct namespace when an attribute category was installed by a package

Fixed in 80935590a040912bf71bde30e908a67f9ce52bdd


🐞 [Node SDK] withExec removes whitespace of arguments

What is the issue?
When calling withExec and passing a single argument that contains whitespace, all whitespace is stripped.
For example, .withExec(['echo', 'foo bar']) will print foobar.
The expected output is foo bar.
Log output
N/A
Steps to reproduce

Run the following

import Client, { connect } from<EMAIL_ADDRESS>
connect(async (client: Client) => {
    await client.container().from('alpine:latest')
        .withExec(['echo', 'foo bar'])
}, { LogOutput: process.stdout })

observe that the output is foobar rather than the expected output

Dagger version
dagger v0.2.36 (ea275a3ba) darwin/arm64
OS version
macOS 12.1

Hi, thanks for reporting!
I'm gonna investigate this and see what's causing this.


BUG: sticky note grows EXTRA large when pasting a longer paragraph of text

https://user-images.githubusercontent.com/14358394/200132626-10538a7e-c736-410e-8a60-4015a430a758.mp4

@zsviczian @DanielJGeiger I have pushed a fix for this, could you test it in https://excalidraw-git-aakansha-fix-paste-bound-text-excalidraw.vercel.app/ ?

Thanks. Seems to be fixed!


Content Suggestion: Bitcoin Standard translated in Korean

Link to Content: Bitcoin Standard 달러는 왜 비트코인을 싫어하는가
Briefly Describe Content: Bitcoin Standard by Saifedean
Suggested Section: Books

Bitcoin Standard: 달러는 왜 비트코인을 싫어하는가
Sold in Yes24, Kyobo, Interpark

there are other books translated in korean from the Books section.
Programming Bitcoin & Inventing Bitcoin & The Little Bitcoin Book
But not sure if those needs to be listed other than the Bitcoin Standard.

okay can you make a PR?

Yessir


Quantum Programming Studio website down

Maybe Website domain has expired so it doesn't access this platform.

QPS is online - no issues. Could be some temporary network error.

@mirsakhawathossain are you able to access it now?

Hi there, QPS works perfectly fine. Thank you very much.
On Wed, 9 Aug 2023, 12:47 pm Petar Korponaić, @.***>
wrote:

@mirsakhawathossain https://github.com/mirsakhawathossain are you able
to access it now?
—
Reply to this email directly, view it on GitHub
https://github.com/quantastica/quantum-circuit/issues/81#issuecomment-1670765538,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AFTB6USUNZS7CIQIU42ZDP3XUMW7TANCNFSM6AAAAAA3JPYBXA
.
You are receiving this because you were mentioned.Message ID:
@.***>

Thank you for reporting issue. Closing now.


feat/refactor(meta): support consistent hash based parallelized chain

What's changed and what's your intention?

Support consistent hash in merge and batch query node. When creating a materialized view, stream manager will map downstream chain actor to upstream materialize actor that belongs to the same parallel unit. Batch query node will filter data with distribution_keys and hash_mapping. So chain node now has the same distribution with upstream materialize node.
Refactored some code in fragmenter and stream manager.

Note:

StreamTableScan's distribution in optimizer is still AnyShard, because we fill distribution_keys in meta. We should find a way to eliminate redundant exchange after chain node.
Because now we map chain's upstream in stream manager instead of fragmenter, resolve_chain_node logic in fragmenter actually can be removed. We still leave it here for compatibility with Java frontend.

Checklist

[x] I have written necessary docs and comments
[x] I have added necessary unit tests and integration tests

Refer to a related PR or issue link (optional)
Resolve #1875
Resolve #1188

Good job, Generally LGTM


Schema Orchestration

Goals for Version 12:
Schema stitching should be configurable
Our view here is that we want to be able to configure the external schema through an easy to use UI. There should be no need to code. Schemas should be created from their micro-service via simple drag&drop and deployed by clicking a publish button.
Stitching does not mean GraphQL only
We not only want to stitch together GraphQL schemas. We want to be able to consume gRPC, Swagger (REST) and OData as schema sources. They should be treated as if they were GraphQL schemas.
Stitching does not just mean merging two schemas
We think in schema representations. We want to be able to have representations of our data without the need of reshaping our micro-services. The GraphQL sub-services or micro-services should be clean and without considerations of the overall schema. The gateway should be able to completely rewrite how a schema is represented. So, there could be cases where the representation of the inner schemas is completely different to the external representation.
Everything is just GraphQL
You can express your gateway completely with a GraphQL file just like today.
Monitoring
The schema registry will keep track on performance data off all the orchestrated services and will have notification mechanisms so that it is easy to keep track of the health status of the system.
CI/CD
We want to have a change preview API. This basically means that you can push a schema to the schema registry from your build and the schema registry will give you a report that tells you which changes are breaking. This helps you avoid breaking your public apis and build powerful CI/CD pipelines.
Resilience
The gateway will not go down when one server is down. GraphQL is great since it allows to provide errors on resolver basis, this mean that if one server is down not the the whole system is down.
Aliases and Conversions
When configuring a schema, we will have tooling available so that you can merge scalars from different server platforms. Also, you will be able through the UI to add small functions that normalize the scalar output. For instance, a DateTime scalar will not always mean the same thing on every platform, with Hot Chocolate schema orchestration, you will be able to normalize the values and we will apply these converters to your schema data. This makes the external schema predictable and homogenous.
Configuring a gateway should be dead simple.

With version 9.1 we will add the first big block for that with our execution plans.
With that we will divert away from the default GraphQL execution algorithms and will have an optimized execution algorithm per query.
https://github.com/ChilliCream/hotchocolate/issues/677
https://github.com/ChilliCream/hotchocolate/issues/642
We still will be 100% spec compliant:

... and hence the server has the freedom to execute the field entries in whatever order it deems optimal.

The spec defines default algorithms but the servers are free to optimize the algorithms.
So, far as I know only GraphQL-Java and HC are exploring this at the moment.

Subscription Stitching

Also, what we want to do with version 10 is introducing a new more efficient server to server protocol. So, we will support the standard protocols but we also will have an optimized GraphQL protocol between HC servers that we will make available as spec for others.

while i really like your ideas about the tooling there's one concern with where the stitching rules are stored. while your proposal with the drag&drop UI sounds nice and easy to use, I'm not sure that this should be the primary way to keep the stitched schema up to date.

Schema stitching should be configurable
Our view here is that we want to be able to configure the external schema through an easy to use UI. There should be no need to code. Schemas should be created from their micro-service via simple drag&drop and deployed by clicking a publish button.

my main concern with this is that any project of a larger size is working on multiple microservices at any time. to keep the CI/CD of those microservices in sync with the required changes on the central schema stitching is getting very hard very soon. you basically need to have two changes (the graphql-schema in the microservice and the logic how this should be stitched in the schema registry) coordinated.
what about if we could define these rules in the same code where we have the (microservice-) schema? we could have CI/CD pipelines which could check for consistency. there's no coordination needed for releases. we have a single source of truth and basically inversion of control for schemas.

What do we do with things where you want multiple schema representations? Then you would need to redeploy all the micro services in order to get a new schema representation in.
This is one of my main concerns. Also I think why should the micro service be bothered with logic about the bigger picture? And this becomes an even bigger question when you take OData and rest into account. How does a legacy rest service tell the gateway its structure.
That does not mean I am completely opposed to your Idea here, I mean that is what Apollo federation does.
There must be a better solution out there. I mean we have two more month to figure out how this is going to work :) Then I would like to start implementing this one.

yes, there's not one way which solves all problems. i guess there are two competing solutions which fundamentally define how a company is organized.
for things like REST you can't annotate somwhere else than a central place (or you are required to have some kind of proxy microservice just for this, which is not great as well). So REST can only be stitched at registry level.
for multiple schemas/representations. i see there could be some use cases here. but for our company this is one of the design goals to prevent from happening. we come from multiple representations (REST) which wasn't that great. now with graphql we have the possibility to just have one graph which spans the whole company. using different graphql schemas would put as back to the REST days where we have to optimize for different representations which makes the developement a lot more complex.
but yes it's hard and that's probably why apollo is still changing directions as there is no obvious solution. i guess it all comes down to organization/processes rather than technology.


BUG: Building tests fails on github workflow machines

Packaging works, but building tests fails due to bug in itk build scripts.


Unable to start Axon with Single Node

Contact Details
No response
Current Behavior
Description
I am unable to start Axon on the main branch .
ubuntu@ip-172-31-95-221:~/axon$ cargo run -- run -c devtools/chain/config.toml -g devtools/chain/genesis_single_node.json
    Finished dev [unoptimized + debuginfo] target(s) in 0.78s
     Running `target/debug/axon run -c devtools/chain/config.toml -g devtools/chain/genesis_single_node.json`
Error [ProtocolError] Kind: Codec Error: Rlp("RlpExpectedToBeList")

Expected Behavior
Expected Behavior:
Axon starts up
OS
Ubuntu 22.04
Axon version
main 680e53ef8b7f7294d424347c32b5245efecafd90
Kernel
Linux ip-172-31-95-221 5.19.0-1029-aws #30~22.04.1-Ubuntu SMP Thu Jul 13 17:17:32 UTC 2023 x86_64 x86_64 x86_64 GNU/Linux
Relevant log output
No response
Anything else?
No response

Notice that the #1270 #1302 and #1319 are breaking change, so it is not compatible with the old data. So is there any old data in the environment?

Notice that the #1270 #1302 and #1319 are breaking change, so it is not compatible with the old data. So is there any old data in the environment?

Hi, @samtvlabs
Would you try cleaning up  the old chain data in devtools/chain/data and then restart Axon?

Thanks this resolves this


Cli should suggest plugin install if subcommand is not found

Something like:
$ kontena vagrant master create
The plugin 'vagrant' is currently not installed. You can install it by typing: 'kontena plugin install vagrant'

Would it have a list of known plugins or would it suggest The plugin 'masster' is currently not installed... ?

It can have a predefined list of known plugins.


Issue with sample code: Invalid response 303: See Other

If I use your sample code in .Net MVC with my real URL/L/P:
var user = "someuser";
var pass = "somepassword";
var uri = new Uri("https://someserver/api/");
var auth = new AutodetectAuthenticator(user, pass);
var client = new HttpAsyncClient(auth, uri);
HGrid result = await client.CallAsync("about", HGrid.InstanceEmpty, "text/zinc");

I get "Invalid response 303: See Other" every time, I had to add this code :
var user = "someuser";
var pass = "somepassword";
var uri = new Uri("https://someserver/api/");
var auth = new AutodetectAuthenticator(user, pass);
var client = new HttpAsyncClient(auth, uri);
await client.OpenAsync();
HGrid result = await client.CallAsync("about", HGrid.InstanceEmpty, "text/zinc");

And then the calls started working.  Am I doing something wrong or does that line need to be added to the documentation?

You are correct, that step was missing from the documentation.
Fixed in #51


MEny Entry Swapper Teleport Item on Construction Cape

Currently with the feature on, the cape left click becomes "Teleport", which brings up the portal interface. Could this be expanded to a picklist to choose between the "Teleport" and "Tele to POH" options?

Duplicate of #6227


[PrometheusCheck] change name of method shadowing one in the base class

What does this PR do?
Rename _submit_metric to _submit.
Motivation
The AgentCheck base class shipped with v6 is slightly different from the one shipped with v5, one of the differences is there's already a method called _submit_metric but the purpose is different.

Sister PR: https://github.com/DataDog/dd-agent/pull/3600


To make transition smooth

Check it out.

@sinss  thanks for the tip.
actually im working on a different approach and would upload as soon as possible.


Django 1.7 and Python3 compatibility

@Luthaf 's https://github.com/lilspikey/django-background-task/pull/12 on the original repo (use with compat #1)

Do you have plans to host a stable fork here ? It would be easier to drop the 1.6 compatibility, as this version is deprecated by the Django team.

Hi @Luthaf
Yes, we will maintain and extend this fork of django-background-task.
We will provide support for two Django LTS versions (currently 1.4.x and 1.8.x) and what officially is supported by Django. This will be done using django-compat. This works great in production (we are using both LTS versions and Django 1.7 at the moment in our projects.
What's already done:

We cherry picked @weijia 's completed task mechanism that logs what was done
To avoid conflicts on PyPI/Github we renamed the repository to django-background-tasks (plural). For an easy upgrade from django-background-task to django-background-tasks, the internal module structure were left untouched.
The latest version including completed task mechanism is on PyPI for production use. Currently Django 1.4 and 1.7.

After integrating compat and updating the test we will review the PR you made and cherry pick what is still not covered.
We would appreciate your help to improve django-background-tasks.

@Luthaf your pr has been merged into this repo.

Hi all,
I'm very happy that I contributed to the project. Could you please
help to remove
the line: author = 'q19420' in models_completed.py? That's my internal
id in my company. And it was just added by the IDE automatically.
Regards,
Richard
2015-05-04 23:11 GMT+08:00 Yannik Ammann<EMAIL_ADDRESS>
Closed #2 https://github.com/arteria/django-background-tasks/issues/2.
—
Reply to this email directly or view it on GitHub
https://github.com/arteria/django-background-tasks/issues/2#event-296339374
.

Sure @weijia


Update test_getting-started.mdx

changed the overview graph

🚀 Deployed on https://652efaba1989fd027400db13--cozy-fox-0defec.netlify.app


Remove broken schema code links

http://www.openmicroscopy.org/site/support/contributing/data-model-schema.html needs rewriting in light of the decoupling of the schema from the BF repo. This just removes the links to the moved code for now as likely the process will be revised further during the 5.3.0 development phase.
Should make fix https://ci.openmicroscopy.org/view/Docs/job/CONTRIBUTING-merge-docs/907/warnings3Result/ and make the build green.
I realise it's not pretty but it's a temporary fix until one of the dev team has time to re-write it properly.

Build green again https://ci.openmicroscopy.org/view/Docs/job/CONTRIBUTING-merge-docs/908/
I could add a note to the top of the page explaining that the process is being updated given the decoupling and the page will be updated soon? Not sure if we are in a position to describe the final replacement workflow as yet @sbesson ?

As mentioned in the description, this page needs to be reviewed/updated to describe the new development workflow now the data model source code is decoupled to its own repository. I would propose to do this as we work through ome-model 5.3.1 and the integration into OME Files C++ 0.3.0 /cc @rleigh-codelibre @mtbc  In the meantime, this PR can be marked on hold.

The only problem with leaving this on hold is that https://ci.openmicroscopy.org/view/Docs/job/CONTRIBUTING-latest-docs/ is going to be unstable until it's merged.

Understood. If the unstability is an issue, then having a top-level warning indicating that this page is restructured following the migration of ome-model is sensible.

I'll push a further commit to add a warning box at the top of the page this afternoon.

@sbesson ok to merge now? Do we have a card for updating this page? Not sure which board it belongs on...

Let's put a card on the OME Files 0.3.0 board in the model column. Now these components are decoupled, OME Files should be the overarching project.

https://trello.com/c/iiefFrlI/34-update-https-www-openmicroscopy-org-site-support-contributing-data-model-schema-html-in-light-of-model-decoupling

Merging and releasing as an interim measure until content is re-written.


Add JS (lib/) subsystem labels

Depends on https://github.com/nodejs-github-bot/github-bot/pull/41
Does some.. fancy.. mapping for some of the outliers.
Also limits the labels to 4, and just uses lib / src at 5.
cc @phillipj

Rebased on master.

LGTM 👍

Updated a bit, condensed some of the oddities a bit.

@Fishrock123 deployed
We're now running v6 in production 👯


[AutoPR graphrbac/data-plane] Added OAuth2 GET and POST to GraphRBAC.json spec

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

(message created by the CI based on PR content)

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

@lmazuel can we release a new package with this change, so https://github.com/Azure/azure-cli/pull/6975 can be merged
CC @shanepeckham

I can due diligence and double check Shane's change works, if you needs quality assurance


Chapter-2 Custom Transformers

Hello @ageron
So, after setting up the class CombinedAttributesAdder, when I create the object "attr_adder", I get the following error:

TypeError                                 Traceback (most recent call last)
 in ()
18
19
---> 20 attr_adder = CombinedAttributesAdder(add_bedrooms_per_room=False)
21 housing_extra_attribs = attr_adder.transform(housing.values)
TypeError: object() takes no parameters
I am not sure what is the main cause of this error. Kindly help.
Also, great book. :)

Thanks @Aimech ,
It looks like the CombinedAttributesAdder is just an instance of the object class? Could you please ensure you are using exactly the same code as in the notebook? In particular, make sure the indentation is correct.
from sklearn.base import BaseEstimator, TransformerMixin

# get the right column indices: safer than hard-coding indices 3, 4, 5, 6
rooms_ix, bedrooms_ix, population_ix, household_ix = [
    list(housing.columns).index(col)
    for col in ("total_rooms", "total_bedrooms", "population", "households")]

class CombinedAttributesAdder(BaseEstimator, TransformerMixin):
    def __init__(self, add_bedrooms_per_room = True): # no *args or **kwargs
        self.add_bedrooms_per_room = add_bedrooms_per_room
    def fit(self, X, y=None):
        return self  # nothing else to do
    def transform(self, X, y=None):
        rooms_per_household = X[:, rooms_ix] / X[:, household_ix]
        population_per_household = X[:, population_ix] / X[:, household_ix]
        if self.add_bedrooms_per_room:
            bedrooms_per_room = X[:, bedrooms_ix] / X[:, rooms_ix]
            return np.c_[X, rooms_per_household, population_per_household,
                         bedrooms_per_room]
        else:
            return np.c_[X, rooms_per_household, population_per_household]

attr_adder = CombinedAttributesAdder(add_bedrooms_per_room=False)

Thanks, @ageron for the help.
I was being silly while defining the class constructor. It's working now.
Thanks once again.

Are you taking suresh sir's classes for bda


4-basics_js-creation of a new branch and a new PR

created a new pr because the old one that was created on Sunday had conflicts that he could not fix, and created a new branch

Замінив всі id на data атрибути. Робив це перший раз, тому буду вдячний якщо вкажиш на недоліки


Add generic export script based on template

This adds a generic export script that can be used for producing various
configuration files based on data from NIPAP.
In addition to the script, I have provided a basic template for ISC DHCP
config (not tested).
Fixes #815.

that would be awesome. Ill test asap to populate my nipap with my current ipam data, ill keep posted

Nice addition! Just fix the default-issue I mentioned in the diff comment and I'll happily merge this :smile:

I've removed the default for username, password and port now!

Cool, merging!


After "./configure --header-only-version" the folder "header_only_version/BaseMatrixOps" is empty

MacOS 13.0.1

I found this module later, maybe I haven't read the docs carefully enough before the installation, but I don't remember the reference to the BaseMatrixOps submodule. Thanks for the library, anyway!

# clone optim into the current directory
git clone https://github.com/kthohr/optim ./optim

# change directory
cd ./optim

# clone necessary submodules
git submodule update --init

Please note that the second option does not have this instruction:
clone necessary submodules
git submodule update --init
Maybe it will be better to add some?
"Installation Method 2: Header-only Library
OptimLib is also available as a header-only library (i.e., without the need
to compile a shared library). Simply run configure with the
--header-only-version option:
./configure --header-only-version
"
Thanks!
ср, 8 лют. 2023 р. о 18:03 kthohr @.***> пише:

Closed #62 https://github.com/kthohr/optim/issues/62 as completed.
—
Reply to this email directly, view it on GitHub
https://github.com/kthohr/optim/issues/62#event-8471642417, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/ABNW63I3QWU2NYKJQKTNR4TWWPNYJANCNFSM6AAAAAAUVMXDXM
.
You are receiving this because you authored the thread.Message ID:
@.***>

Cloning submodules is now such a basic concept/requirement that I assume it to be known by all who can invoke a compiler.


Orientation issue for small meshes

For models that have small meshes, I was seeing the issue Error: mesh volume is too small...
I'm able to make this issue go away by adding the following line to the body containing the meshes: <inertial pos="0 0 0" mass="0" diaginertia="0 0 0"/>
Adding this line resolves the error, but the offending small mesh is oriented incorrectly:

I've noted that the orientation issue goes away when I increase the scale parameter of the meshes to a large enough value:

Attaching a zip file with minimal models here:
small_model_2.zip
To reproduce: python -m mujoco.viewer --mjcf small_model_2/model_small_scale.xml
I believe this is a bug.

System information:

Operating system: both Mac (M1), Ubuntu
MuJoCo version: 2.3.7

Add shellinertia=true to the hood light geoms and your problem should go away.

Why do you need to make it false? My intention was that shellinertia should only have been dependent on the mesh topology, not its size.


Cannot tick distance manager while unloading playerchunks

Timings link
No response
Expected behavior
No errors?
Observed/Actual behavior
Console Error https://nekobin.com/wububumiro

Steps/models to reproduce
I am not sure, I think the dev of quick shop is just sifting the blame?
Purpur version
patched_1.17.1.jar:git-Purpur-1355
Agreements

[X] I am running the latest version of Purpur available from https://purpur.pl3x.net/downloads.
[X] I have searched for and ensured there isn't already an open issue regarding this.
[X] My version of Minecraft is supported by Purpur.

Other
No response

Paper recently added this check into the server. Falls on the plugin dev to fix their plugin.
https://github.com/PaperMC/Paper/blob/master/patches/server/0753-Do-not-allow-ticket-level-changes-while-unloading-pl.patch#L26


Update README.md

Correct the example where err is being used instead of errors

Coverage remained the same at 100.0% when pulling 698e3ea0aa6cc3f70a03e26b36a34c3ffb56e8b9 on joturako:patch-1 into d79e33a2abba2a2a774b6c545dc82522d5c1eaeb on ctavan:master.


Allow custom content-type header

By using Dict.put_new, it won't overrider the content-type if it's already included in the header map. This allows custom content-types.

thanks @werbitzky! If you could release a new version with this included that would be awesome!

np, @mgartner
I'm about to release a new version, but I would like to update the README.md to include custom headers and make a few small code changes regarding the custom headers, I'll wait a bit for a recent answer here:
https://github.com/werbitzky/elastix/pull/14
Consider using a hashcode in your Mixfile for now and I'll release 0.4 soon
cheers


Support for radio buttons

Support for a radio button group, where every radio button has the same name, currently seems broken. The first radio button is always selected instead of the actual one (or none).

Not sure actually if this addon shouldn't be deprecated? 😬
I believe "native" rehydration with the latest Fastboot and Glimmer should be possible now. But I wasn't able to try this out yet...
PRs certainly welcome, but I will probably not put much time into this myself!

Ya, I took a look at the native rehydration (which was further along than I thought) shortly after I encountered this. It doesn't seem selected radio buttons are transferred over there either so in the mean time I've opted for a "loading" form while not rehydrated.
Maybe a notice at the top of the README that experimental native rehydration can be used would be good. I found the info about it a bit hard to find/scattered.

Maybe a notice at the top of the README that experimental native rehydration can be used would be good. I found the info about it a bit hard to find/scattered.

Sure. Given that you have found it, do you have some links at hand? 😆

Hmmm I might have missed the easy link at first. The important bits seem to be in the README: https://github.com/ember-fastboot/ember-cli-fastboot/blob/efbebad3f9278e4b8179f720579f6eeee1fe8457/README.md#rehydration


fix(storagenode): fix concurrency bugs of settings for storage and executor

What this PR does
This patch fixed concurrency bugs in the storage setting and executor setting. Issue #262 describes
it concretely.
Which issue(s) this PR resolves
Resolves #262

Codecov Report
Base: 63.21% // Head: 63.23% // Increases project coverage by +0.01% :tada:

Coverage data is based on head (fdd1781) compared to base (6f36b0d).
Patch coverage: 100.00% of modified lines in pull request are covered.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #263      +/-   ##
==========================================
+ Coverage   63.21%   63.23%   +0.01%
==========================================
  Files         130      130
  Lines       17355    17360       +5
==========================================
+ Hits        10971    10977       +6
+ Misses       5835     5831       -4
- Partials      549      552       +3

Impacted Files
Coverage Δ

internal/storagenode/storagenode.go
69.79% <100.00%> (+0.51%)
:arrow_up:

internal/metarepos/raft_logger.go
33.33% <0.00%> (-33.34%)
:arrow_down:

internal/metarepos/report_collector.go
77.00% <0.00%> (-0.43%)
:arrow_down:

internal/metarepos/storage.go
79.22% <0.00%> (-0.23%)
:arrow_down:

internal/metarepos/raft.go
79.88% <0.00%> (+0.28%)
:arrow_up:

internal/storagenode/logstream/committer.go
81.03% <0.00%> (+2.41%)
:arrow_up:

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

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


Resolve imports

This PR adds eslint-import-resolver-webpack so AirBnB's config can resolve imports.
Just wondering, do we want import to not require the .vue extension as well?
I could commit this as well:
    {{#if_eq lintConfig "airbnb"}}
    // don't require .vue extension when importing
    'import/extensions': ['error', 'always', {
      'js': 'never',
      'vue': 'never'
    }],
    {{/if_eq}}

It sounds like you're adding this to your other PR, so we can leave it out here.


chore(main): release 1.240.0

When this PR is merged, a new tagged release will be created with the notes below.
1.240.0 (2023-09-12)
Features

Add session trace metadata params (#676) (882b21c)
Add SR to pro and spa builds (only runs for LP customers) (#677) (6d8ddb4)
Centralize web vitals timings (#635) (d912e94)
Origin of agent webpack chunks now changeable (#659) (739e2dd)

This PR was generated with Release Please. See documentation.

:robot: Release is at https://github.com/newrelic/newrelic-browser-agent/releases/tag/v1.240.0 :sunflower:

Ran:
WDIO all browsers: https://github.com/newrelic/newrelic-browser-agent/actions/runs/6164974944
JIL all browsers: https://github.com/newrelic/newrelic-browser-agent/actions/runs/6164975728


I think ZonePreferenceServerListFilter  is redundant

version=1.3.1.RELEASE
groupId=org.springframework.cloud
artifactId=spring-cloud-starter-ribbon
com.netflix.ribbon#ribbon-loadbalancer;2.2.2
I think ZonePreferenceServerListFilter  is redundant and wrong 。When the zone is down , you should not select server from this zone ,but you do .

This module has entered maintenance mode. This means that the Spring Cloud team will no longer be adding new features to the module. We will fix blocker bugs and security issues, and we will also consider and review small pull requests from the community.


Resuming Trainer with ckpt_path disrupts logging by TensorBoardLogger

🐛 Bug
When I fit a model that uses a TensorBoardLogger and then save a checkpoint with Trainer.save_checkpoint, the TensorBoardLogger of the Trainer that resumes from this checkpoint loses the global step count and is not able to log to the same event files. This causes only the data logged by the second logger to show up in TensorBoard, with the global step count starting at zero.
To Reproduce
Run the CoLab notebook I developed for BoringModel. Note that the logged epochs in TensorBoard go from 10 to 20, starting at step zero.
Expected behavior
I would prefer for the logger to resume logging from the global step count of the Trainer that saved the checkpoint, and for the entire training curves from both trainers to be visible in TensorBoard.
Environment

CUDA:

GPU:

Tesla T4

available:         True
version:           11.3

Packages:

numpy:             1.21.6
pyTorch_debug:     False
pyTorch_version:   1.11.0+cu113
pytorch-lightning: 1.6.3
tqdm:              4.64.0

System:

OS:                Linux
architecture:

64bit

processor:         x86_64
python:            3.7.13
version:           # 1 SMP Sun Apr 24 10:03:06 PDT 2022

Additional context
I am using PyTorch Lightning to run Population-Based Training, so I am often saving and resuming models. I'd prefer to have the continuous TensorBoard logs for each worker so I don't end up having to keep track of n_workers * n_generations different traces in TensorBoard.
cc @awaelchli @ananthsub @ninginthecloud @rohitgr7 @carmocca @edward-io

BTW, the bug_report_model.ipynb on CoLab is a fantastic idea and well executed! Made it super easy for me to replicate this bug.

I think this is exactly this issue: #12274

Agreed! Do you think the issue of overwriting / deleting the TB data from the first training session is related to the step number or could that be a separate issue?

On May 10, 2022, at 4:28 AM, Rohit Gupta @.***> wrote:

I think this is exactly this issue: #12274
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you authored the thread.

it's different. The logging step used in the previous run is not restored and is instantiating from 0 in the new run.


How to read a gds file data and export it as PNG?

I would like to find ways to convert the read gds file into some readable format in LabVIEW. If you are not familiar with LabVIEW, please help me to convert it into PNG or formats like JSON, TXT.

@Shivamanikandan-AS You can use write_svg to export an SVG drawing of any cell in the GDSII file, but there's no such thing as a "conversion". For JSON or TXT you'd have to define the schema yourself and manually export all the cells and their elements.


Bug: auto_commit=False and retry=True in confluent-kafka doesn't result in retries

Describe the bug
Retries do not work for the confluent-kafka library as far as I can tell.
Seems related to https://github.com/airtai/faststream/issues/1001 in some way.
How to reproduce
Include source code:
# app.py
from collections import defaultdict

from faststream import FastStream, Logger
from faststream.confluent.annotations import KafkaMessage
from faststream.confluent import KafkaBroker

broker = KafkaBroker("localhost:30100")
body_to_times_to_die = defaultdict(int, {"hello": 3})

@broker.subscriber(
    "hello_world",
    group_id="foo",
    auto_commit=False,
    retry=True
)
async def async_subscriber(body: str, logger: Logger, msg: KafkaMessage):
    global body_to_times_to_die
    logger.info(body)
    body_to_times_to_die[body] -= 1
    if body_to_times_to_die[body] >= 0:
        raise Exception()
    await msg.ack()

async def main():
    app = FastStream(broker)
    await app.run()  # blocking method

if __name__ == "__main__":
    import asyncio
    asyncio.run(main())

And/Or steps to reproduce the behavior:

Run kafka.
Run the consumer.
Use kafka ui to produce a single "hello" message on the topic.
Use kafka ui to produce a single "hi" message on the topic.

Expected behavior
I expect the "hello" message to be processed 4 times. The first three times should raise the exception, and the 4th should succeed. I then expect the "hi" message to be processed without exception.
Observed behavior
The "hello" message is processed once and raises an exception. Then the "hi" message is processed once without exception.
This is my console output (I forgot to run step (3) for this run)
python -m app
2024-05-14 15:59:12,852 INFO     - FastStream app starting...
%3|1715727552.863|FAIL|faststream-0.5.5#producer-1| [thrd:localhost:30100/bootstrap]: localhost:30100/bootstrap: Connect to ipv6#[::1]:30100 failed: Connection refused (after 0ms in state CONNECT)
%3|1715727553.869|FAIL|faststream-0.5.5#producer-1| [thrd:localhost:30100/bootstrap]: localhost:30100/bootstrap: Connect to ipv6#[::1]:30100 failed: Connection refused (after 2ms in state CONNECT, 1 identical error(s) suppressed)
2024-05-14 15:59:14,891 INFO     - hello_world | foo |            - `AsyncSubscriber` waiting for messages
%3|1715727554.893|FAIL|faststream-0.5.5#producer-2| [thrd:localhost:30100/bootstrap]: localhost:30100/bootstrap: Connect to ipv6#[::1]:30100 failed: Connection refused (after 1ms in state CONNECT)
2024-05-14 15:59:15,956 INFO     - Topic `hello_world` created.
2024-05-14 15:59:15,960 INFO     - FastStream app started successfully! To exit, press CTRL+C
2024-05-14 15:59:29,786 INFO     - hello_world | foo | 0-17157275 - Received
2024-05-14 15:59:29,793 INFO     - hello_world | foo | 0-17157275 - hello
2024-05-14 15:59:29,793 ERROR    - hello_world | foo | 0-17157275 - Exception:
Traceback (most recent call last):
  File "lib/python3.11/site-packages/faststream/broker/subscriber/usecase.py", line 331, in consume
    result_msg = await h.call(
                 ^^^^^^^^^^^^^
  File "lib/python3.11/site-packages/faststream/broker/subscriber/call_item.py", line 172, in call
    raise e
  File "lib/python3.11/site-packages/faststream/broker/subscriber/call_item.py", line 164, in call
    result = await call(message)
             ^^^^^^^^^^^^^^^^^^^
  File "lib/python3.11/site-packages/faststream/broker/middlewares/base.py", line 73, in consume_scope
    await self.after_consume(err)
  File "lib/python3.11/site-packages/faststream/broker/middlewares/base.py", line 54, in after_consume
    raise err
  File "lib/python3.11/site-packages/faststream/broker/middlewares/base.py", line 64, in consume_scope
    result = await call_next(await self.on_consume(msg))
             ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "lib/python3.11/site-packages/faststream/broker/wrapper/call.py", line 201, in decode_wrapper
    return await func(msg)
           ^^^^^^^^^^^^^^^
  File "lib/python3.11/site-packages/fast_depends/use.py", line 146, in injected_wrapper
    r = await real_model.asolve(
        ^^^^^^^^^^^^^^^^^^^^^^^^
  File "lib/python3.11/site-packages/fast_depends/core/model.py", line 529, in asolve
    response = await run_async(call, *final_args, **final_kwargs)
               ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "lib/python3.11/site-packages/fast_depends/utils.py", line 48, in run_async
    return await cast(Callable[P, Awaitable[T]], func)(*args, **kwargs)
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "lib/python3.11/site-packages/faststream/utils/functions.py", line 53, in to_async_wrapper
    return await call_or_await(func, *args, **kwargs)
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "lib/python3.11/site-packages/fast_depends/utils.py", line 48, in run_async
    return await cast(Callable[P, Awaitable[T]], func)(*args, **kwargs)
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "app.py", line 22, in async_subscriber
    raise Exception()
Exception
2024-05-14 15:59:29,804 INFO     - hello_world | foo | 0-17157275 - Processed

Screenshots
Not much else to share.
Environment
Running FastStream 0.5.5 with CPython 3.11.8 on Darwin
Additional context
This seems to be a workaround if I do it right before I raise the exception:
            raw_message = msg.raw_message[0]
            topic_partition = TopicPartition(
                raw_message.topic(),
                raw_message.partition(),
                raw_message.offset()
            )
            msg.consumer.consumer.seek(topic_partition)

Hello @andreaimprovised,
This bug has been fixed as part of https://github.com/airtai/faststream/releases/tag/0.5.18 release.
Following is a sample code

from typing import Any, Dict
from faststream import FastStream
from faststream.confluent import KafkaBroker
from faststream.confluent.annotations import (
    KafkaMessage,
    Logger
)
import json
import asyncio

broker = KafkaBroker("localhost:9092")
topic = "confluent-nack-test"

app = FastStream(broker)

@broker.subscriber(topic, group_id="foo", auto_commit=False, auto_offset_reset="earliest")
async def async_subscriber(body: Dict[str,Any], logger: Logger, msg: KafkaMessage):
    logger.info(body)
    await msg.nack()

@app.after_startup
async def publish_something() -> None:
    async def _publish_something() -> None:
        i = 10
        print(f"Sleeping for {i} seconds")
        await asyncio.sleep(i)
        message = {"hi": "there"}
        await broker.publish(message, topic=topic)
        print("Published message" + json.dumps(message))

    asyncio.create_task(_publish_something())


Add option to change acme contact email

At current time if you issue cert, you will not have option to change acme contact. To change it you need issue new cert.
Contact used in expiration emails: https://letsencrypt.org/docs/expiration-emails/
Cross-link of https://github.com/mailcow/mailcow-dockerized/issues/4086

Is this not already possible with the --contact<EMAIL_ADDRESS>?

Hi @diafygi yeah, mailcow go the way of dropping existing certs manually to force issuing new one in case of changing contact person. Idea was to check existing contact and in case contact was changed compared to existing one update the contact :) to not reissue certs when this isn't actually needed. As I know let's encrypt support such flow.


undefined method `working_dir' for LWRP resource docker_container from cookbook docker

 ================================================================================
 Error executing action `run_if_missing` on resource 'docker_container[zookeeper]'
 ================================================================================
 NoMethodError
 -------------
 undefined method `working_dir' for LWRP resource docker_container from cookbook docker
 Cookbook Trace:
 ---------------
 /var/chef/cache/cookbooks/docker/libraries/provider_docker_container.rb:175:in `block in <class:DockerContainer>'
 /var/chef/cache/cookbooks/docker/libraries/provider_docker_container.rb:264:in `block in <class:DockerContainer>'
 /var/chef/cache/cookbooks/docker/libraries/provider_docker_container.rb:271:in `block in <class:DockerContainer>'
 Resource Declaration:
 ---------------------
 # In /var/chef/cache/cookbooks/cookbook/recipes/zookeeper.rb

  31: docker_container "zookeeper" do
  32: #  name "zookeeper"
  33:   repo zoo_image
  34:   tag node['TAG']
  35:   binds ["#{data_dir}:/home/app/data"]
  36:   port "2181:2181"
  37:   restart_policy 'always'
  38: end
  39:

 Compiled Resource:
 ------------------
 # Declared in /var/chef/cache/cookbooks/cookbook/recipes/zookeeper.rb:31:in `from_file'

 docker_container("image") do
   action [:run_if_missing]
   retries 0
   retry_delay 2
   default_guard_interpreter :default
   declared_type :docker_container
   cookbook_name "perf-env"
   recipe_name "zookeeper"
   repo "repo:5000/var-zookeeper"
   tag "latest"
   binds ["/srv/docker/image/data:/home/app/data"]
   port "2181:2181"
   restart_policy "always"
   container_name "image"
 end

Any idea?

Seems caused by bad parameter...

Can you elaborate on what parameter caused the issue?
On Oct 4, 2015 10:13 AM, "Kanstantsin Shautsou"<EMAIL_ADDRESS>wrote:

Seems caused by bad parameter...
—
Reply to this email directly or view it on GitHub
https://github.com/bflad/chef-docker/issues/475#issuecomment-145367250.

I copied generated block and removed additional attributes and it works. Sounds like comment, but i not sure...


Fix "it's" typo in log message

What this PR does / why we need it:
This fixes a typo (conflation of "it's/its") in a log message.
Special notes for your reviewer:
Release note:
NONE

/assign @munnerz