Implement download retry mechanism

We discussed this in a few places now:
https://github.com/nsidc/earthaccess/issues/481#issuecomment-1978680018
https://github.com/nsidc/earthaccess/issues/594#issuecomment-2161829080
Figured it's time for a dedicated issue :grin:

Any updates on this plan?
I'm trying to process a month of data using the earthaccess tool to grab 1 PACE file at a time but for some reason earthdata is giving timeout errors quite often making it difficult to actually process multiple days of files. Not sure if there is an issue with earthdata so I've reached out to them (awaiting a response), but also wondering if the retry option might be available soon. (or is there at least a way to get an error message returned from earthaccess to implement a manual retry after a 30 or 60 second wait)
Thanks!

@itcarroll
It's something as simple as
import earthaccess

start_date = '2024-05-01 00:00:00'
end_date = '2024-05-01 23:59:00'

min_lon = -120; max_lon = -100; min_lat = 20; max_lat = 40
earthaccess.login(persist=True)
results = earthaccess.search_data(short_name = 'PACE_OCI_L1B_SCI',cloud_hosted=True,temporal=(start_date,end_date),count=400,bounding_box=(min_lon,min_lat,max_lon,max_lat),version='2')

I don't think it's an OBDAAC issue though, I've been having issues with TROPOMI data as well.
The problem is, I'm trying to grab say 6 PACE granules at a time. Normally only 1 or 2 fails, but of course then I have gaps. Also, I'm trying to download the files temporarily, then delete when I'm done with them because I don't want to be storing TB's of PACE data locally.

Similar to @zfasnacht1013 , I tried using earthaccess.search_data to download multiple files (~ a month of GPM_MERGIR files in my case). The first time, 3 failed with HTTP error 500. The second time, 1 (of those 3) failed.

@zfasnacht1013, although the workaround above should do the trick, and can also serve as a basis for directly adding such functionality to earthaccess, would you mind elaborating your use case, if you can?
In general, we want to discourage fully downloading files, and instead provide advice on how to avoid such downloading, and instead on how to perform direct reads, grabbing only the parts of the files containing the data you need, assuming you don't actually need the files in their entirety.

@chuckwondo I'm developing research trace gas algorithms for PACE. I'm using the PACE L1b at 1km and use the full spectra of reflectances, so there's really not much of a way to subset the files before I use them. I need to start upscaling a produce 1-2 months to use for validation. Since it's a research product, I'm developing it in NCCS and not in any PACE sand box.
This is going to be a general theme moving forward, not only with PACE, but other instruments. We are doing something similar with TEMPO and will also need to grab large chunks of data to process and develop our products for validation.
I'm not sure this is something that has been considered much yet at Earthdata. I would assume the ideal scenario for earth data is that folks work in AWS for development to limit the network transfer, but since AWS is pay-to-play and NCCS is not, with budgets being tight, we are left to develop in NCCS.
This might be something for further discussion to solution ideas on how to move forward with this kind of use case. Feel free to reach out to me if we should have a meeting to further discuss.


Why can't find the class Landroid/os/Build;?

androguard version 3 and 4 all have this problem.

https://github.com/androguard/androguard/issues/765
Like this problem, I also want to get the static variables of the Build class, perhaps because instructions such as sget are not handled when creating xref?


chore: release v0.6.20

:robot: I have created a release beep boop
0.6.20 (2024-05-14)
What's Changed

chore(deps): bump golang.org/x/net from 0.22.0 to 0.23.0 in the go_modules group by @dependabot in https://github.com/coffeebeats/gdenv/pull/241
chore(deps): bump github.com/urfave/cli/v2 from 2.27.1 to 2.27.2 by @dependabot in https://github.com/coffeebeats/gdenv/pull/243
chore(deps): bump golangci/golangci-lint-action from 4 to 5 by @dependabot in https://github.com/coffeebeats/gdenv/pull/244
chore(ci): migrate release-please to new repository by @coffeebeats in https://github.com/coffeebeats/gdenv/pull/248
chore(deps): bump github.com/go-resty/resty/v2 from 2.12.0 to 2.13.1 by @dependabot in https://github.com/coffeebeats/gdenv/pull/246
chore(deps): bump golangci/golangci-lint-action from 5 to 6 by @dependabot in https://github.com/coffeebeats/gdenv/pull/247
fix: migrate off of deprecated linters by @coffeebeats in https://github.com/coffeebeats/gdenv/pull/249

Full Changelog: https://github.com/coffeebeats/gdenv/compare/v0.6.19...v0.6.20

This PR was generated with Release Please. See documentation.

:robot: Release is at https://github.com/coffeebeats/gdenv/releases/tag/v0.6.20 :sunflower:


[Improvement] add multi MySQL server version to CatalogMysqlIT Integrate test to improve compatibility

What would you like to be improved?
CatalogMysqlIT testing MySQL with mysql:8.0, we'd better add mysql:5.7
How should we improve?
No response

Hi, I want to take this issue,
@FANNG1,  can you help run the server, I'm using this cmd  ./bin/gravitino.sh start but it's not working, am I missing some configuration

thanks for taking this issue， you can refer https://datastrato.ai/docs/0.3.0/how-to-install/ to how to start GravitinoServer, please  and you could check logs in logs directory, please note that if you deploy Gravitino by ./gradlew compileDistribution, you should goto distribution/package directory to start Gravitino

Hi, @kumarav1nash this issue is expected to be fixed in 0.4 in this week, so I fixed it.

Thanks @FANNG1, from last two weeks i was occupied with my org release cycle so did not get the chance to work on it.


Remove --version flag from cip command.

What type of PR is this?
/kind bug
/kind cleanup
/kind api-change

What this PR does / why we need it:
This change removes the --version flag in favor of the existing version command. Currently the flag does not actually change any behavior in CIP, nor does it actually print the version. Since we already have a command responsible for producing version information, we no longer need this flag.
Such a change WILL effect our users, as it's an API change of the cip CLI. Therefore, this should be pointed out within the release notes of the next CIP major version.
Which issue(s) this PR fixes:
#345
Special notes for your reviewer:
This API change should be documented in the next release.
Does this PR introduce a user-facing change?
YES
The flag --version no longer exists.
Remove the version flag from the cobra cli.

cc: @listx @amwat @justaugustus @kubernetes-sigs/release-engineering

How can we keep track of this API change for release documentation?
@justaugustus

Thanks!
/lgtm
/approve


[Suggestion] Adding config to disable catching rain water in tank

MC 1.12.1
Forge 2460
AM 0.1.2beta
I wonder if it would be possible to add a config option which disables that the tank collects water from the rain. Reason being that I might add a mod that adds toxic rain and then it wouldn't make thematic sense anymore.

Sure. I'll add that next version

Both rainfall collection and evaporation will be configurable (with the option to disable) next version


Research Toolbar Issue on iPhones

[ ] Make it possible for other devices to access the web app from the server

[ ] The toolbar on compose is covered by Safari's browser toolbar.

table bottom not visible
not able to scroll horizontally or vertically
have to double click to open a chart


Create H33loWorld.c

A simple C program for Hello World
What the program do ?

In what programming language it is written?

This pull request do not contains files in specific folders. Please fix it and reopen


Vue Router not works on Firefox, Works perfect on Chrome and Safari

Version
3.2.0
Steps to reproduce
I just setup the Laravel 7 with Latest Vuejs. I Just setup simple sampleComponent.Vue
What is expected?
import Vue from 'vue'
import VueRouter from 'vue-router'
Vue.use(VueRouter)
const routes = [
{
path : '/example',
component : require('./components/ExampleComponent.vue').default
}
];
const router = new VueRouter({
routes
});
const app = new Vue({
el: '#app',
router : router
});
That is my code. What is wrong in that
What is actually happening?
[Vue warn]: Unknown custom element:  - did you register the component correctly? For recursive components, make sure to provide the "name" option.
(found in )
[Vue warn]: Unknown custom element:  - did you register the component correctly? For recursive components, make sure to provide the "name" option.
(found in )
In firefox, It will me that error.

Please ignore mimimal reprodoction link

We need a boiled down reproduction to check. Open a new issue I you manage to reproduce

same issue with me also. Scroll behaviour not working in firefox.


FR - Customization of json data for mqtt_json device_tracker

Home Assistant release with the issue:

0.103.5
Last working Home Assistant release (if known):
Operating environment (Hass.io/Docker/Windows/etc.):

Python VirtualEnv on CentOS 7
Integration:

Description of problem:
The device_tracker mqtt_json require the payload to be longitude/latitude/altitude/gps_accuracy but that's not always the payload you might receive, so it might be nice if it can be specified.
Ex. payload_json_latitude: "GPS.lat"
Problem-relevant configuration.yaml entries and (fill out even if it seems unimportant):

Traceback (if applicable):

Additional information:

Thank you for reaching out. We use GitHub for tracking issues, not for providing support or tracking feature requests.
If you want to suggest a feature, you should try our Community Forum: Feature Requests.
If you have additional questions, feel free to join our Discord chat server.
Thanks! 👍

Thanks @frenck I did that but this can at the same time be considered an issue because the pre-defined HA structure it is not flexible enough to work with the Tasmota GPS implementation, so the integration between those two "platforms" cannot be made unless I create a specific Rule in Tasmota which has a consequence increase the messaging to MQTT Server.
But yes, there are also some features that might be nice to have implementated, but in this case this was marked as FR while it is also an issue since tasmota guys said they're not gonna change the message payload.


Replaced log display panel on the home page(index.html) with ckeditor display box

Replaced log display panel on the home page(index.html) with ckeditor display box(read only), the old display panel was having problems while showing tables.
The middle partition on the home page used to show short descriptions by using first 40 html words of log data and extracting text words from it.
Now it will show first 20 text words. Function returnFirstXWords() no longer required now.

The URL links not opening earlier should open on click now.


FIX: Better message when form action handler not found.

Fixes #4622 to some extent, although this fix will be most useful when
https://github.com/silverstripe/silverstripe-errorpage/issues/30 is
addressed also.

I assume you don't want this to close #4622 when it's merged?

Nah it turns out this message is shown without the error page fix so good to close.


Update CONTRIBUTING.md

fix a typo: , is omitted

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.


[FlatList] [Android] Issue with TextInput element in the bottom of a FlatList

Description:
TextInputs at the bottom of the screen in a FlatList automatically and immediately dismiss the keyboard on focus, making it impossible to write anything in it.
The issue is reproducible easily on a blank react-native init project. I tested it in Nexus 5x simulator and real device. I reproduce the bug every time on 0.61.
Reproduction Steps and Sample Code
<FlatList
          data={this.state.items}
          style={{ flex: 1 }}
          renderItem={(i) => (
            <View style={{ flexDirection: "row", borderBottomWidth: 1, borderBottomColor: "#333", padding: 24 }}>
              <Text style={{}}>{i.index}</Text>
              <TextInput style={{ flex: 1, backgroundColor: "#bbb" }} />
            </View>
          )}
        />

React Native version:
    OS: macOS 10.15.3
    CPU: (12) x64 Intel(R) Core(TM) i7-8750H CPU @ 2.20GHz
    Memory: 106.97 MB / 16.00 GB
    Shell: 5.7.1 - /bin/zsh
  Binaries:
    Node: 8.16.0 - /usr/local/bin/node
    Yarn: 1.15.2 - /usr/local/bin/yarn
    npm: 6.4.1 - /usr/local/bin/npm
  SDKs:
    iOS SDK:
      Platforms: iOS 13.1, DriverKit 19.0, macOS 10.15, tvOS 13.0, watchOS 6.0
    Android SDK:
      API Levels: 23, 26, 28
      Build Tools: 25.0.1, 27.0.3, 28.0.3, 29.0.0
      System Images: android-19 | Google APIs Intel x86 Atom, android-23 | Google APIs Intel x86 Atom, android-24 | Google APIs Intel x86 Atom, android-28 | Google APIs Intel x86 Atom_64, android-28 | Google Play Intel x86 Atom
  IDEs:
    Android Studio: 3.4 AI-183.54<IP_ADDRESS>52501
    Xcode: 11.1/11A1027 - /usr/bin/xcodebuild
  npmPackages:
    react: 16.9.0 => 16.9.0
    react-native: 0.61.4 => 0.61.4

Copy of https://github.com/facebook/react-native/issues/13745

set removeClippedSubviews={false}

set removeClippedSubviews={false}

This does not resolve it

I faced the same issue, I have a code like this:
<View>
    <FlatList
      onEndReached={handleEndReach}
      data={props.items}
      renderItem={renderRow}
      keyExtractor={getKey}
      onEndReachedThreshold={0.01}
    />
</View>

I can't use the last input from the flat list because it shows and hide immediately the keyboard, as some workarounds I fond some android manifest solutions but none of this worked for me so I changed the View for a ScrollView wrapper of the FlatList and it solved the problem. Maybe it is a problem when it scroll to make visible the input and the keyboard on a View Screen.
Solution:
<ScrollView>
    <FlatList
      onEndReached={handleEndReach}
      data={props.items}
      renderItem={renderRow}
      keyExtractor={getKey}
      onEndReachedThreshold={0.01}
    />
</ScrollView>

For what its worth, I have this same issue and https://github.com/facebook/react-native/issues/28246#issuecomment-621537950 did indeed fix it. No idea why..

@stale open
Wrapping FlatList with ScrollView is not recommended https://forums.expo.io/t/warning-virtualizedlists-should-never-be-nested-inside-plain-scrollviews-with-the-same-orientation-use-another-virtualizedlist-backed-container-instead/31361/12

Still Occurring in react native V0.64


[Feature Request] How group series?

What feature or change would you like to see made?
Hi! As shown in this example, how to group the series? each grouped series has its own number of images.

Why should we prioritize this feature?
Important information for everyone

That is an app level concern


Accessibility md-select aria-owns

Bug:
aria-owns in the DOM even though the elements it points to are not
What is the expected behavior?
aria-owns should not point to elements not in the DOM, if the elements are not in the DOM then aria-owns should be removed as well
What is the current behavior?
aria-owns shows up in the DOM when those elements are not in the DOM
What are the steps to reproduce?
http://embed.plnkr.co/cLXRzvn2uWQ5y5YmiU8L/
inspect select, notice non-empty aria-owns
What is the use-case or motivation for changing an existing behavior?
Currently throws accessibility issues
Which versions of Angular, Material, OS, TypeScript, browsers are affected?
"@angular/material": "2.0.0-beta.8"
Is there anything else we should know?

It does make sense to clear the attribute when the panel is hidden, however it looks like it causes a regression in the user experience. Previously NVDA would read out something like "Option text, selected" when selecting, but it doesn't do it anymore if the attribute is cleared. Also looking at the documentation, it seems like pointing to unexisting elements isn't disallowed explicitly.

Hmm yes reading the spec theres no specific mention of not allowing it. But if you read this part it implies it.
The value of the aria-owns attribute is a space-separated list of IDREFS that reference one or more elements in the document by ID.

It says "one or more elements IN the document by ID.
I'm ok with leaving it if thats what you think though.
Our business has flagged it as an accessibility bug we need to fix.
Let me know what you think, I'll create a PR if you think we should implement

This causes issues with axe-core when testing for aria roles and required children (listbox requires options).  If the aria-owns tag is removed when the listbox is closed, the test fails even though the tags are added when the listbox is opened.

This is also an issue when using the Chrome accessibility tools


[WIP] First version of the action

Ref: https://github.com/niteoweb/operations/issues/1090

[x] Add docker file.
[x] Add action.yml
[x] Add the python script.
[x] Test with a repository.
[x] Test with real world web apps.
[ ] Add tests.
[ ] Add docs.
[ ] publish the action.

@zupo
After our yesterday's discussion. I took a different approach to find the deployment status. I found that GitHub and Heroku communicate via GitHub Deployment API. So, I have rewritten the GitHub Action to use GitHub Deployment API and it works as expected.
Following are results with a test app:
When build fails:

When build is succeeded but if there is an error after deployment:

When everything is succeeded:

I have tested the same with EBN and KAI:
EBN:

KAI:

Please review the approach and let me know If any changes are required. In the meantime I will update the docs, logs and tests.

@zupo Any update here?

Sorry I remember approving it last night. Guess I clicked something else :S


Feature request testbit/2 or getbit/2 builtin

With great pleasure I saw that Trealla Prolog has:
	{"lsb", 1, fn_lsb_1, "+integer,-integer", false, true, BLAH},
	{"msb", 1, fn_msb_1, "+integer,-integer", false, true, BLAH},

Now I am stuck with the following problem. There is no predicate
testbit/2 and no evaluable function getbit/2:
?- float_pow_half_even(-14744, -3, X).
   error(existence_error(procedure,testbit/2),float_rat_half_even/4).
?- X is getbit(1024, 2).
   error(type_error(evaluable,getbit/2),(is)/2).

getbit/2 is the more common, its for example found in SWI-Prolog.
It has the advantage that it saves creating a large bigint, or a bigint
at all. One could bootstrap the two functions as follows:
testbit(X, N) :- X /\ (1 << N) =\= 0.
% testbit(X, N) :- (X >> N) /\ 1 =\= 0.

getbit(X, N, Y) :- testbit(X, N), !, Y = 1.
getbit(_, _, 0).

But usually bigint libraries have somewhere something like that,
that could be made available to the Prolog system.

Not needed anymore, was using X /\ (1 << N) == 0.
The show stopper is now this bug:

Error in bitwise operations or integer comparison
https://github.com/trealla-prolog/trealla/issues/69


State machine

Introducing a simple state to the Application model to keep track of the overall journey.
No special gem is used, just ActiveRecord enum, which provides the states on Application we need. The state transitions themselves should be concentrated in service classes (ie. ResolverService).
The current states are:

created
evidence_check
part_payment
processed

This PR centralises completion of Application, EvidenceCheck and PartPayment to the ResolverService. This service class is now solely responsible for completing those respective objects, creating EvidenceCheck/PartPayment as needed and also setting decision and decision_type on Application when moving to the final processed state.
I think we should also consider removing the query objects are with the state scopes, they may not be needed.
We should also probably remove EvidenceCheck -> Evidence, because that's what we use in the UI.

@colinbruce and @dotemacs please review carefully including manual testing.

Reviewed and happy to merge. :+1:


Is this still a supported scenario if you don't use Azure Monitor managed service for Prometheus?

This feature was a supported scenario to scrape some additional Prometheus metrics with the agent, and have them sent to Log Analytics Workspace.
Now the article states that

you should first ensure that you're scraping Prometheus metrics from your cluster using Azure Monitor managed service for Prometheus,

Is this just a suggestion, or a formal requirement?
The page also say:

Use the configuration described in this article only if you also want to send this same data to a Log Analytics workspace where you can analyze it using [log queries]

We don't want to do 'also' - we only want to send some additional Prometheus metrics to LAW
Is using the 'old' method, mainly - just configuring the configmap to scrape some metrics still fully supported?

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

ID: 9b2ab022-64e9-6258-8c0c-a8d37c1db151
Version Independent ID: f8e7fd6f-d8f9-7558-e881-0d9b70894e8e
Content: Collect Prometheus metrics with Container insights - Azure Monitor
Content Source: articles/azure-monitor/containers/container-insights-prometheus-logs.md
Service: azure-monitor
Sub-service: containers
GitHub Login: @bwren
Microsoft Alias: bwren

@tvdvoorde
Thanks for your feedback! We will investigate and update as appropriate.


How to use intersection observer for highlighting active heading

Hi XD
I have my own toc workflow for a sveltkit app i'm making and I have everything taken care of up until the intersection observer to highlight the current on screen headers in the toc...trying to figure out how to do it, but have no knowledge of intersection observers or using them with svelte...looking through your code and I'm not sure where that part begins...
In my toc I am looping through an array objects:
[
  { level: 1, heading: 'Frequencies' },
  { level: 1, heading: 'Notes' },
  { level: 2, heading: 'Equivalence' },
  { level: 2, heading: 'Pitch Classes' },
  { level: 1, heading: 'The Keyboard' },
  { level: 1, heading: 'Two Zones' },
  { level: 2, heading: 'Zone 1' },
  { level: 2, heading: 'Zone 2' },
  { level: 2, heading: 'Zone 1 + Zone 2' },
  // ...etc...
]

that is generated by a custom rehype plugin traversing some markdown...
I then loop through it in my toc.svelte file:

  {#each headings as {level, heading}}

      <a class='toc-link'
        href={`/words/${slugify(title)}#${slugify(heading)}`}
      >
        {removeTags(heading)}

  {/each}

...and everything is almost all good when the slugified href matches the current hash and I can click around and go to hash links and all that...still not perfect and prolly should use
's and
's, making my own slugify() function and it doesn't quite work right yet, but these are other issues...
The last big piece of the toc puzzle is highlighting the toc headers that are currently in the viewport...just not sure how to go about it...
...I'm looking through your code and I'm not sure where that part begins and how to adapt it to my array of objects. I also have a REPL I've put together by lifting a codepen that works and trying to make it work with svelte, but can't figure it out...
Svelte REPL:
https://svelte.dev/repl/7006d7890b964928b8f72622a3d067ac?version=3.42.5
Original Codepen:
https://codepen.io/bramus/pen/ExaEqMJ
If you don't have time or interest for this no worries, totally understand, but if you have any thoughts that you would be willing to share I am all ears :-)
Thank you!

Hey @rchrdnsh, identifying the currently viewport-intersecting heading in the ToC happens here:
https://github.com/janosh/svelte-toc/blob/56ba0cb25faa6b7628185e12091fbb4dd5d4d0dd/src/lib/Toc.svelte#L50-L52
The matching heading node is then assigned a class of active here:
https://github.com/janosh/svelte-toc/blob/56ba0cb25faa6b7628185e12091fbb4dd5d4d0dd/src/lib/Toc.svelte#L96
and styled here
https://github.com/janosh/svelte-toc/blob/56ba0cb25faa6b7628185e12091fbb4dd5d4d0dd/src/lib/Toc.svelte#L123-L125
Does that help/answer your question?

Closing for now. Please reopen if questions remain.


feat(svelte-query): Simplify types

WritableOrVal is now applied to the options in types.ts directly, which reduces type complexity both within the library and when using it in your application. Might be breaking if you were previously using something like WritableOrVal<CreateQueryOptions> to define your options store, but this was introduced in the v5 alpha so it doesn't need a major version bump.

Codecov Report
Patch coverage has no change and project coverage change: -30.15 :warning:

Comparison is base (cd84ed8) 88.97% compared to head (6315386) 58.82%.

:exclamation: Current head 6315386 differs from pull request most recent head 60f7dd6. Consider uploading reports for the commit 60f7dd6 to get more accurate results

:exclamation: Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.
Additional details and impacted files
@@             Coverage Diff             @@
##            alpha    #5442       +/-   ##
===========================================
- Coverage   88.97%   58.82%   -30.15%
===========================================
  Files          71       14       -57
  Lines        2385      102     -2283
  Branches      624       15      -609
===========================================
- Hits         2122       60     -2062
+ Misses        218       36      -182
+ Partials       45        6       -39

Impacted Files
Coverage Δ

packages/svelte-query/src/createBaseQuery.ts
94.44% <ø> (ø)

packages/svelte-query/src/createInfiniteQuery.ts
0.00% <ø> (ø)

packages/svelte-query/src/createMutation.ts
92.30% <ø> (ø)

packages/svelte-query/src/createQueries.ts
100.00% <ø> (ø)

packages/svelte-query/src/createQuery.ts
100.00% <ø> (ø)

... and 57 files with indirect coverage changes

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


Trying to host ProxyKernel within a .NET Framework assembly results in an InvalidOperationException

I am running into the following exception when I send any SubmitCode command to ProxyKernel from within a net472 process using the new netsandard2.0 (MS.DN.Interactive.VS) library.
I was not running into this when the same SubmitCode command was being sent to a ProxyKernel from within a net6 process using the corresponding netstandard2.1 (MS.DN.Interactive) library.
I will update this issue with more learnings after I try a few more things that we discussed offline.
System.InvalidOperationException
  HResult=0x80131509
  Message=Cannot apply a context that has been marshaled across AppDomains, that was not acquired through a Capture operation or that has already been the argument to a Set call.


Allow scatter plots with more legend items than number of classes

This line forces the number of items in the legend to be the same as the number of classes.
https://github.com/facebookresearch/visdom/blob/master/py/visdom/__init__.py#L1140
In my case, not all the classes can be sampled in a particular batch. len(opts['legend']) becomes greater than K.
Describe the solution you'd like
Allow number of legend items at least the number of classes
Describe alternatives you've considered
I can manually remove legend items not used in this particular batch, but this is cumbersome.

Seems like a reasonable change to me, I'd be happy to accept a PR that implements a solution that makes sense to you, though I imagine it'd need to take in a list of mappings, considering this line just indexes into the legend items directly. You could make an optional legendmap opt that has to be the length of K, mapping entries in order in K to their values in the legend, and then default that to the array [0 ... K-1] if it's not provided.

Ah right right - Y is supposed to serve this purpose in scatter


[builder] Overlapping components don't get merged in static ttf

As explained in: https://github.com/googlefonts/fontmake/issues/813
It looks like the overlapping components are currently decomposed after merging overlapping outlines.

I see we have an issue open in FB for this, https://github.com/googlefonts/fontbakery/issues/2999.
do you know which environments render overlapping components badly? I thought we only disliked transformed components.

this https://github.com/googlefonts/fontmake/issues/253#issuecomment-318044968 and following discussion might be relevant

Thanks Cosimo and apologies for being dense.
In Rosa's example, the Pesata could cause the even-odd-winding rendering issue (I reckon it won't, it only seems to happen on glyphs like Oslash where the bar doesn't penetrate the bowl of the O). If it does, the components should be decomposed and the outlines merged. The same will also need to happen if it wasn't constructed from components just outlines. However, is this worth the effort? afaik this issue only happens in El Capitan which was released in 2015. We do get some complaints on google/fonts regarding this issue but they seem to be decreasing.
I agree with decomposing flipped components because FreeType renders these badly.

I used to lean towards the latter, but now I think ufo2ft DecomposeComponents filter should be updated to do the right thing, that is, check if two components overlap and if either has a reversing transform decompose both.
In https://github.com/fonttools/fonttools/blob/main/Lib/fontTools/ttLib/removeOverlaps.py I used skia-pathops to check if the component's contours intersect one another.

Sounds fairly sensible to me if we think it's worth the effort to save El Cap users.
Reason why I asked about VFs is that the gftools builder can instantiate static fonts using the varLib instancer. If you instantiate static fonts from a VF, you want the outlines and components to be compatible so you can transfer vtt hints.

Thanks for your responses Cosimo. Picture is much clearer now.

now I think ufo2ft DecomposeComponents filter should be updated to do the right thing, that is, check if two components overlap and if either has a reversing transform decompose both.

Surely just decomposing the components isn't enough. Don't you also need to merge the outlines otherwise you could end up with the even-odd-winding issue in OS X El Cap?
Perhaps this issue is better placed in ufo2ft since it sounds like we could fix this in a filter.

It looks like the source file is constructed from a stretched macron.

This means we're decomposing the components because it's a transformed component. However, it should still merge these outlines. I'll make a pr soon.


Add Cytoscape related functions

Update the name of the function of exporting the graph to Cytoscape JSON to indicate the JSON is for Cytoscape JS library.
Add a new function to export the graph to Cytoscape compatible GraphML format along with styles.xml files for styling in Cytoscape.
Add function to load the graph into a live instance of Cytoscape

closes #46


feature request : add an option to display a DAG

it will be nice to have an easy to use option something like to_dag, that automatically show a diagram with all the tables dependencies.
it will something similar to duckdb.sql(" select 42").explain() that automatically generate query plan

users seems to like this kind of easy to use functionalities

Thanks @djouallah for the request and I agree users would like that. Currently it feels a bit outside of the scope of SQLFrame and I would prefer if I could leverage some other library to generate it. Currently you can generate column-level lineage using the new lineage method but it relies on SQLGlot's implementation which isn't very user friendly. Context: https://github.com/eakmanrq/sqlframe/issues/138

One thing to clarify on this request, you want to see a DAG view of a given DataFrame correct? So if the DataFrame joined Table A and Table B then you would want to see a graph of Table A and Table B being joined together? Do you care about column-level lineage in this example?

no, I don't care about column lineage

you can leverage sqlglot's logical planner
https://github.com/tobymao/sqlglot/blob/main/sqlglot/planner.py


how to get data from different model ?

hello
i have in my table Questions a field called language_id  which is the id of the language in the Languages table
i want to get the name of that language through the id
$table->addColumn('language_id', 'Language', function() {
return Language::find(1)['Name'];
});
so how i can pass the language_id to the find or any other way to get this =(
thank you

$table->addColumn('language_id', 'Language', function($model) {
      return Questions::find($model->id)->language->Name;
  });

i try this but is there a better way ?

In case anybody else wants to know, here is how it's done:
$table->addColumn('{put anything here, it wont even be used}', 'Column Label', function ($model)
{
    // Anything you return in this function will be used as the content of the column

    return $model->some_attribute;
});


Allow use of net-ssh 5.2.0 gem

Description
host-key error when using net-ssh-4.2.0
Update the gem specification to use net-ssh-5.2.0.  net-ssh-5.2.0 fixes the following bug:
https://github.com/net-ssh/net-ssh/pull/601
Which occurs when running test kitchen:
[2019-04-16T11:18:22.977009 #85258] ERROR -- default-Centos7: Message: SSH command failed (host key algorithm mismatch for signature 'ssh-rsa' !=<EMAIL_ADDRESS>The latest chef gem requires net-ssh-4.2.0.
Gem Version
14.12.3
Platform Version
Mac OSX. 10.14.4
Replication Case
Using a bastion host and remote host with the following kex and host_key:

kex: diffie-hellman-group1-sha1
host_key<EMAIL_ADDRESS>
Attempt to run kitchen converge.
This can be fixed by updating the chef.gemspec and global specification
chef-<new-version>.gemspec files to allow the most recent version.

This is actively being worked here for Chef 15: #8380

net-ssh requires the following gems for ed25519 support:

rbnacl (>= 3.2, < 5.0)
rbnacl-libsodium, if your system doesn't have libsodium installed.
bcrypt_pbkdf (>= 1.0, < 2.0)
See https://github.com/net-ssh/net-ssh/issues/478 for more information

You closed issue about net-ssh and rbnacl without actually fixing them.
This is bad work, chef is not reliable anymore.
Lets go ansible.

rb-nacl is required for net-ssh 4.x not net-ssh 5.x and rbnacl and other methods of using libsodium for ed25519 support are not compatible with all operating systems supported by Chef. We've removed several blockers to using net-ssh 5.x including getting the upstream to release versions of the other net-* gems we depend on in Chef and DK/Workstation. That has unblocked net-ssh 5.x for Chef 15 and allowed us to add the ed25519 gem to our gemspec as well as train/test-kitchen. As I noted in the comment above it's in the works. Please assume good intentions. There's real people working to support these projects and things are often more complex than they appear from the surface, especially when it comes to large dependency trees.

I'm closing this out at this point since we are allowing net-ssh 5 on master https://github.com/chef/chef/blob/master/chef.gemspec#L29 and we're shipping the ed25519 gems as well. They've also been added to Test Kitchen 2.1 and the upcoming train 2.0 release.


Document the current implementation

Fixes #45

@PKief could you look into https://github.com/marketplace/actions/publish-to-github-wiki and add it to this PR?

@PKief could you look into https://github.com/marketplace/actions/publish-to-github-wiki and add it to this PR?


Update or remove docs/source/guides/develop-iroha-module.md

This doc is heavily outdated
Who can help
@6r1d @outoftardis

There was a comment from Aleksandr a few months back:

This guide should be revised. Preferably once we have decided the shape in which modules will be implemented.

Was such a decision made?


Custom server-side error handler

Describe the problem
I'm using the adapter-node, and ship my logs to a log aggregator. I try to send all logs in a similar format, to be able to reuse log parsers. Currently, the default error handler (here and here) print the whole stack trace on multiple lines, which comes into the aggregator as separate log messages.
Describe the proposed solution
I'd prefer if I were able to catch the errors and handle them myself, so that I can format and print them consistent with other logs in my projects.
Alternatives considered
Make a new hook that can catch the error.
Importance
would make my life easier
Additional Information
No response

duplicate of https://github.com/sveltejs/kit/issues/1857

Thank you. I did look for a similar error first, but didn't see this one.


Enhancement map marker

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

2 of 2 (100.0%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 99.511%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
11127

Relevant Lines:
11131

💛  - Coveralls


Remove Dierckx, move to Interpolations

@szy21 can you please have a look at the buildkite output and check if everything is still looking good?

Yes, the plots look good.


Introduce query alias option.  Provides a possible fix for #29.

Putting up a solution for introducing a query alias.

Closing for the reason mentioned here:
https://github.com/marcusolsson/grafana-json-datasource/issues/29#issuecomment-762212980


bug: drawer-end cascades for nested drawers

What version of daisyUI are you using?
v4.12.13
Which browsers are you seeing the problem on?
All browsers
Reproduction URL
https://play.tailwindcss.com/C3gVcWnOy2?size=762x720
Describe your issue
When using nested drawers (one for the right and one for the left), if you try to put your drawer-end as the outside drawer, it cascades down and makes both of the drawers be a "right" drawer. You can see the behavior in the Tailwind Play up above. It's modified from an existing Tailwind Play that @saadeghi posted here https://github.com/saadeghi/daisyui/discussions/2278#discussioncomment-6927619.
That Play exhibits the correct behavior because the left drawer is the outside drawer and the right (drawer-end) drawer is the inside drawer.

https://github.com/saadeghi/daisyui/blob/dfec9ea0f91552ff348b1e7dc0c26432ae2064d3/src/components/unstyled/drawer.css#L36-L55
It looks like it's this styling. I think it could be fixed by adding a direct descendant selector to the .drawer-toggle class
& > .drawer-toggle {


ROX-14560: stop adding is_org_admin and org_id claims to ROX tokens

Description
as part of ROX-14560, stop adding is_org_admin and org_id claims to ROX tokens

/retest

To my previous point: we have customer case where  org_id claim was used, which gives us more motivation to do a proper migration
https://github.com/stackrox/stackrox/issues/6641

Thank you @dhaus67 @stehessel @ivan-degtiarenko , I'll starting working on the migration


⚠ AZ-Launcher has degraded performance

In 1815609, AZ-Launcher (https://az-launcher.nz/) experienced degraded performance:

HTTP code: 200
Response time: 2787 ms

Resolved: AZ-Launcher performance has improved in 0024ffc after 46 minutes.


Add config option for model owner key

Option to set owner key in config for one to many relationship  when  polymorphic is not set to true for cases where user may have different key/table structure for their model.

Coverage increased (+0.02%) to 90.419% when pulling e8c5a1ddb35d2de6d1ee1c973efbdd99d0b87171 on wizston:model-owner-key-config into 0d536e63ec00984353dda6ec8ea3d524e42cd822 on fenos:master.

@Gummibeer That doesn't address the purpose of this PR. This PR is to provide the option to set the foreign key in your own Model.

@wizston and this is what this method returns. The method will return the key name of the related model - so in general id or any other value if you have changed this. Putting this directly in the config is, for me, overhead cause it will duplicate the configuration work.

@Gummibeer  How would you advice in a situation where my model uses "user_id" column name instead of "id"? The the method returns the key name not set the key name. You wouldn't advice that i change my model relationship column name to id cause of the library, while there could be someone out there that communicated with some external data with any structure. I think adding it in the config would solve lots of these issues.

Ok, what we need is the primary key of the other "user" model - this is by default id but can be changed to whatever you like - on the model and in database. The primary key of the user model is returned by his getKeyName() method.
So yes, this could be a case where some work would be great - but I don't see the point why to introduce a new config value if it is possible to solve it by native laravel methods!?

Thanks @Gummibeer, I just adjusted the commit.

Coverage increased (+0.06%) to 90.457% when pulling 8767a15d6b99a250733dde7c859e7c25fc15d664 on wizston:model-owner-key-config into 0d536e63ec00984353dda6ec8ea3d524e42cd822 on fenos:master.

Can you please fix the failed checks and in best case add an unittest for it!?

https://travis-ci.org/fenos/Notifynder/jobs/232078189
Needs a review


Update and remove unneeded dependencies

Fixes high severity security issues noted by snyk.
Removals

Apparently unused marked dependency.

Changes

Make async gulp tasks return that weren't properly.
Update is-reachable, localtunnel, and snyk.

Testing

npm install
gulp audit:perf should pass and show no error at the end.

No errors, assuming the output is expected?
Threshold of 70 not met with score of 51

No errors, assuming the output is expected?

Yup! All good.

@anselmbradford The merging of this PR started failing Jenkins builds:
> node scripts/npm/apps-install

./cfgov/unprocessed/apps/analytics-gtm's npm output: added 4 packages in 5.354s

./cfgov/unprocessed/apps/owning-a-home's npm output: added 50 packages in 10.14s

./cfgov/unprocessed/apps/regulations3k's npm output: added 142 packages in 6.147s

npm WARN optional SKIPPING OPTIONAL DEPENDENCY<EMAIL_ADDRESS>(node_modules/fsevents):
npm WARN notsup SKIPPING OPTIONAL DEPENDENCY: Unsupported platform for<EMAIL_ADDRESS>wanted {"os":"darwin","arch":"any"} (current: {"os":"linux","arch":"x64"})

added 1584 packages in 106.157s
Building project…
module.js:549
    throw err;
    ^

Error: Cannot find module 'ansi-colors'
    at Function.Module._resolveFilename (module.js:547:15)
    at Function.Module._load (module.js:474:25)
    at Module.require (module.js:596:17)
    at require (internal/module.js:11:18)
    at Object.<anonymous> (/var/lib/jenkins/jobs/cf.gov-ansible-build/workspace/cfgov/gulp/tasks/test.js:1:82)
    at Module._compile (module.js:652:30)
    at Object.Module._extensions..js (module.js:663:10)
    at Module.load (module.js:565:32)
    at tryModuleLoad (module.js:505:12)
    at Function.Module._load (module.js:497:3)
    at Module.require (module.js:596:17)
    at require (internal/module.js:11:18)
    at requireDir (/var/lib/jenkins/jobs/cf.gov-ansible-build/workspace/cfgov/node_modules/require-dir/index.js:126:33)
    at Object.<anonymous> (/var/lib/jenkins/jobs/cf.gov-ansible-build/workspace/cfgov/gulpfile.js:15:1)
    at Module._compile (module.js:652:30)
    at Object.Module._extensions..js (module.js:663:10)

Huh, uh okay. I might just remove that altogether and inline the red color. It's only used in one place. I'll open a PR.


update package-lock.json

Coverage remained the same at 65.863% when pulling beaa2d67ae0f310111f0252f7ea67ee3a4cf2758 on update_dependencies into a417fa29cf56e7f17b32ba3ade5992f01b80bdf9 on master.


Add the possibility to delete a full table from values

We had an where users wanted to delete some default configuration from the values.yaml, but it wasn't possible, this fix overcomes that issue, of course, I'm not 100%-ly sure in this implementation, but it is a good topic starter.

@bonifaido would you mind adding unit tests and docs here to cover this use case? Thanks!


Update change log about connecting to Azure AD via proxy

As title. Update change log about connecting to Azure AD via proxy.

Hi, @saragluna , please help to review this PR when you have time.


Add local build optimizations via gradle.properties

These optimizations will be more apparent once #180 is implemented.

Hey @notjuanortiz,
Your changes look good to me!
View build log
TravisBuddy Request Identifier: a706af00-0cd2-11ea-9489-798921b627f8


 But is my contribution :smile:

Hi @ivomarsan. It looks like something strange is happening with this PR. It's coming from "unknown repository" and seems to include some commits that are already in the repo. 😕


[FancyZones] Allow a different shortcut key for moving windows between zones (allow both FZ and Windows Snap)

Summary of the new feature/enhancement
I want to be able to move windows between zones while also being able to use windows snap. Something like Win+Shift+Left and Win+Shift+Right for moving between zones.
Proposed technical implementation details (optional)
Add a Shift modifier for the shortcut key allowing users to move windows between configured zones.
Do not use Win+Ctrl+Left and right because that is already used by windows to swap between virtual desktops

Hi @kylelawrence
Win+Shift+Arrow is already used by Windows to move windows to another monitor.

Looks like Win+Alt+Left/Right are still unused. There's also Alt+Ctrl+Left/Right.

We also need to reserve a shortcut key to move windows to virtual desktops.

Added to https://github.com/microsoft/PowerToys/issues/6611

I'd also love to see this feature as I constantly find myself having to touch the mouse during an otherwise "keyboard focused work" only to move a window to some fancy zone which can be really distracting at times.


ol.style.RegularShape#getPoints may return double amount of points given to the constructor

The method getPoints of ol.style.RegularShape returns double the amount of points given to the constructor if a radius2 was provided.
I think this follows the logic that the points given to the constructor are applied to each circle, thus resulting in the double amount.
But this means that the constructor option points and the points returned from getPoints are different from each other, which is not comprehensible by distinguished names or the documentation.
I can think of the following solutions:

Match the meanings by having to specify the total amount of points for both circles in the constructor.
Match the meanings by letting getPoints only return the number of points on one circle.
Distinguish the meanings by renaming the methods name to getTotalPoints or something similar.
Distinguish the meanings by documentation only.

I can create a PR for one of these solutions.
And btw i think it would be nicer to have this doubling of points in the constructor, where it can be found easily and not in both the methods draw_ and drawHitCanvas_

There is also similar unobvious behaviour with radius2 option: if you don't pass it, than it will be set eaual to primary radius, so getRadius2 will return a number;

Pull request welcome, thanks in advance!


Peer info during handshake

It would be very useful to applications like crawler to have the following info during peer connect/handshake

bee version
uptime
db-capacity (configured/ used)

Of-course this can also be a protocol after connection. Also it can be like an optional one so that people who dont want to share this info with crawler can block it.

Note that all values can be lied about. So, you could never trust this

I would personally not convolute the discovery with this, but can imagine it would make sense as a separate protocol

Note that all values can be lied about. So, you could never trust this

Yes. I agree. But so is every other information you get from a peer in any of the protocols (ex: connected peers in gossip).

I would personally not convolute the discovery with this, but can imagine it would make sense as a separate protocol

Agree. I have the same thinking. May be it should be a separate protocol with options to enable or disable during node startup. So that it is optional for users to report this to his peers. very similar to '--ethstats' flag in go-ethereum.
With this we would get a very important stat from the network (total disk capacity of the network). Even some approximation is a good thing to have.
Let me know of this is okay and I can PR this sometime if you guys have no issues.


test: remove parallel flag from unit test

Removed parallel flag from go test command to run unit-tests.
Closes #62
Signed-off-by: Rolson Quadras<EMAIL_ADDRESS>

Codecov Report

Merging #63 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master      #63   +/-   ##
=======================================
  Coverage   92.43%   92.43%
=======================================
  Files          17       17
  Lines         555      555
=======================================
  Hits          513      513
  Misses         24       24
  Partials       18       18

Continue to review full report at Codecov.

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

Thoughts about the golang test cache?

Thoughts about the golang test cache?
count=1 disables it.

@troyronda looks like the default value is 1.
https://golang.org/cmd/go/#hdr-Testing_flags
-count n
Run each test and benchmark n times (default 1).
If -cpu is set, run n times for each GOMAXPROCS value.
Examples are always run once.

Thoughts about the golang test cache?
count=1 disables it.

@troyronda looks like the default value is 1.
https://golang.org/cmd/go/#hdr-Testing_flags
-count n
Run each test and benchmark n times (default 1).
If -cpu is set, run n times for each GOMAXPROCS value.
Examples are always run once.

@rolsonquadras there's no need to set the default value

@rolsonquadras @Baha-sk @llorllale To
disable test caching, use any test flag or argument other than the
cacheable flags. The idiomatic way to disable test caching explicitly
is to use -count=1.
https://golang.org/pkg/cmd/go/internal/test/


An error occurred while starting the application.

After successfully deploying on Azure this error appear when y try to reach app. In Teams app i have the same error.
An error occurred while starting the application.
.NET Core 4.6.29812.02 X86 v<IP_ADDRESS>    |   Microsoft.AspNetCore.Hosting version 2.1.1-rtm-30846    |    Microsoft Windows 10.0.14393

Hello! Any ideas or support?
How i can check is my app running?


weird nesting of paginated posts insights data

fb api zrejme vracia nekonzistentne odpovede pri paginacii posts insights dat pre niektore pages, prejavuje sa to tym ze vysledny podstrom sa zanoruje do predchadzajuceho podstromu namiesto toho aby sa zaradil vedla neho ako sibling.
priklad query:
posts{insights.metric(post_negative_feedback, post_engaged_users, post_consumptions, post_impressions_fan, post_impressions_paid, post_impressions, page_posts_impressions_organic, page_posts_impressions_paid, page_posts_impressions)}

vrati zanalyzovanu strukturu:
Writing output to csv files. Analyzed structure: {:parent-types {page_posts_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights 9, page_posts_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights 9, page_posts_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights 9, page_posts_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights 9, page_posts_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights 9, page_posts_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_insights_

posts insights pri paginacii vracali redeundantne data, vyriesilo sa to tak ze paginacia insights sa berie ze zacina od root node.
https://github.com/keboola/ex-facebook-graph-api/commit/877a2e8562cc37076840dd17ad4727577605859d#diff-114d365b2d62639561b79167c32ca633R58


Fix testsuite #6097

Fixes #6097, Closes #6101, Closes #6071

Passes for me too, didn't check though

But remember, as it might be important to find later:

Works, but should change the commit message though. We fixed the code, not just the test suite


fix(index): missing export runAfterBootstrapEffects

missing export runAfterBootstrapEffects was causing AOT to fail
https://github.com/ngrx/effects/issues/115

:+1:

+1

@MikeRyan52


Retrieving Guilds

Hello!
I'm trying to retrieve the guilds the user is currently in. I can pass in 'guilds' as a scope and I can see that Discord is requesting access from the user in the login screen, but the guilds are not present in the hash returned from omniauth. Am I missing something?
Thank you in advanced!
omniauth.rb
Rails.application.config.middleware.use OmniAuth::Builder do
    provider :discord, ENV['DISCORD_CLIENT_ID'], ENV['DISCORD_CLIENT_SECRET'], scope: 'guilds identify'
end

sessions_controller.rb
...
def create
  auth_hash = request.env['omniauth.auth']
  render plain: auth_hash.to_yaml
end
...

Nevermind.
For anyone else who needs this, request access with guilds scope then do a GET request with the token to get the guilds:
response = HTTParty.get('http://discordapp.com/api/users/@me/guilds', {
  headers: { "Authorization": "Bearer #{self.token}" }
})


Add library "database"

As per a discussion @campaul and I had OOB, the library "database" could just be a symlinked blob storage directory under .cache.
Eg. if you import some raw files on your desktop, they would be stored in the library as symlinkes like so: $LIBRARY/.cache/blobs/hash -> /home/user/Desktop/somefiles.cr2.
This would be true of all files, even if they're actually in the $LIBRARY directory.

Fixed in 3f38ea7011975015a5138900a6b80e544178cd4c

Fixed in 3f38ea7011975015a5138900a6b80e544178cd4c


Feature Enhancement: CLI Flag to output Correct Predictions

Rasa NLU version: 0.13.8
We're trying to compare a current production model with a model that we're building with Rasa NLU. As part of our migration we want to compare how the two models handle the same input. The evaluate CLI tells us when our model has a different prediction, but we'd like to know if they have the same prediction to evaluate for false positives.
It would be awesome to add a CLI flag like --successes successes.json to get the output of all the correct predictions.
My team and I are willing to add this feature, if it will be accepted upstream.
Thanks!

this has been merged already


runtime stack must be "custom" to run docker images

The docs should say "custom" instead of ""  in the following line:
az functionapp create --name <app_name> --storage-account <storage_name> --resource-group AzureFunctionsContainers-rg --plan myPremiumPlan --runtime  --deployment-container-image-name <docker_id>/azurefunctionsimage:v1.0.0
Otherwise people will never figure out what to put in here espacially when working with an outdated cli which doesn't give a hint and says that only dotnet, node, pyhton and java is supported on linux.
Thanks

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

ID: 5d505ab9-d351-6650-6d0c-bb65d2daa229
Version Independent ID: 2a1d3487-a93d-2206-7ef9-a91aa275ec4e
Content: Create Azure Functions on Linux using a custom image
Content Source: articles/azure-functions/functions-create-function-linux-custom-image.md
Service: azure-functions
GitHub Login: @ggailey777
Microsoft Alias: glenga

Hello @DanielMeixner,
Thank you for your feedback! We will review and update as appropriate.

Hello @DanielMeixner,
Thank you for your feedback! We will review and update as appropriate.

Hello @DanielMeixner, Thank you for reaching out.
In the "Other" section of the this page, I see that the az functionapp create command has --runtime custom as shown below

For different supported languages  I see that the az functionapp create command has --runtime <functions runtime stack> depending on the language used.

Please let me know if there are any concerns. Thank you!

Hello @DanielMeixner, Thank you for reaching out.
In the "Other" section of the this page, I see that the az functionapp create command has --runtime custom as shown below

For different supported languages  I see that the az functionapp create command has --runtime <functions runtime stack> depending on the language used.

Please let me know if there are any concerns. Thank you!

Hello @DanielMeixner, Since we have not heard back from you we will now proceed to close this thread. If there are further questions regarding this matter, please tag me in your reply. We will gladly reopen the issue and continue the discussion.

Hello @DanielMeixner, Since we have not heard back from you we will now proceed to close this thread. If there are further questions regarding this matter, please tag me in your reply. We will gladly reopen the issue and continue the discussion.


Issue installing the library

After cloning the repository, there is no OBD2Swift.xcodeproj in the OBD2-Swift folder.
How can I manually install the library?

Hi check is today. in OBD2-Swift folder contains only library classes. To try the library you can run example project in Example folder. Also library can be installed via cocoa pods, just add pod 'OBD2-Swift' to your pod file

Great! I was following the instructions in the README, but the cocoa pods method is exactly what I was looking for.


Angular 13 support (+ Ivy partial compilation)

Hello,
I did read the comment saying that we shouldn't ask for "Angular update".
I just have some error/warning when using ngx-popperjs, so this issue is more about a reminder/list of things I detected using ngx-popperjs with Angular 13 (I used --force to for the install) and that will be cool to take a look when you'll have time switching to Angular 13 support.

Generating browser application bundles (phase: setup)...Warning: Entry point 'ngx-popperjs' contains deep imports into 'D:/Git/XXX/node_modules/@popperjs/core/lib/types'. This is probably not a problem, but may cause the compilation of entry points to be out of order.

Compiling the lib (need partial Ivy compilation):
Processing legacy "View Engine" libraries:
- ngx-popperjs [es2015/esm2015] (https://github.com/tonysamperi/ngx-popperjs)
Encourage the library authors to publish an Ivy distribution.

Like I said I'm not asking for anything, it's just a list of things I detected.
Thank you for this lib.

Hi @Yberion thanks for the post, but others had already shared something similar...
Anyways I think I won't get any errors when I update dependencies...
I will publish the update in the next few days (26-27-28 december).
Happy holidays

Hi @Yberion this should be out tomorrow!

Hello @tonysamperi thank you for the answer and the fast reaction, this is appreciated!
Keep the good work and have a good holidays!

Done 🎉


feat: remove uniqueness from sequelize storage

…tries in case of rerun

Thanks. Could you add a test that fails without this change?

Since this was reverted, how do we work around --rerun ALLOW now?

I solved it on my end, I didn't have time to create a better fix. Sorry...


Update readme

Hi there, I've been playing with your config to learn the ways of NixOS for the past days. First times I tried to run the script as it was it kept getting stuck at compiling dependencies and I had to restart my PC to unfreeze it. After some digging I come up to this educated guess: my RAM was not enough for how many parallel processes my CPU could spawn. So my 16 core AMD CPU would eat up all the 16GB of RAM I had and it got into a starvation scenario.
Managed to get past that with adding --cores 10 to the switch rebuild command in your install.sh. Thought it could help others in my situation with this PR.

Thanks for your help :)


Plutus plugin should automatically mark all functions and instance methods as inlinable

Describe the feature you'd like
Most of the functions used in Plutus program need to be market as INLINABLE to be used outside the module.
This is inconvenient. Moreover it is insufficient in some cases.
For example, automatically derived class instances cannot be fixed this way.
Describe alternatives you've considered

Defining all instances manually is a bit silly for large ADTs:

instance Eq Bool where
  {-# INLINABLE #-}
  (==) = ...

Marking all functions as inlinable also adds unnecessary code when the code is already processed by Plutus plugin.
We could add plugin with GHC_OPTION that marks all functions, including derived instances as inlinable.

Describe the approach you would take to implement this
I would love to talk with Plutus plugin author and get instructions on how to best add it.

s/may/should?

I believe we can't do this with plugins, but I might be wrong. Plus, you would need the INLINABLE-adding plugin to run on the modules you use, not just the ones where you compile the final PLC, which is much more difficult to set up.

I think the streamly plugin does this FWIW. See https://github.com/composewell/fusion-plugin

I think that sets INLINE so long as the binding already has an unfolding. I don't think we can create an unfolding if one doesn't exist. And we'd still have to run the plugin on the upstream libraries to achieve that also.

The current Plutus setup compiles libraries or brings them from IOHK cache anyway. Can we connect this plugin and simply rebuild entire library once? Or mark certain modules as used by Plutus and rebuild them with GHC having new compiler option?

@michaelpj has something changed since the issue was opened? Does -fexpose-all-unfoldings now help us alleviate the problem? Is there anything we could do here?

I don't know of a way to improve the current situation. My best ideas all involve substantial rework of the entire system :(

I don't know of a way to improve the current situation. My best ideas all involve substantial rework of the entire system :(

I see. So this is something that would be great to have, but requires a huge amount of effort. Marking it with "Low priority" then.

-fexpose-all-unfoldings exposes optimized unfoldings, which are often unusable because GHC's optimization pipeline performs optimizations that assume e.g. the target platform, stuff about foreign pointers and so on. We're fairly reliant on getting pre-optimization Core.
Ultimately this is our big problem: how do we get the core for bindings in other modules? We are getting by with the current approach, but it's fragile and many things can make it go wrong. We never got very far with trying to get something into GHC to help. So: a big idea is needed IMO.

@michaelpj Can't you just disable the optimisations then?

a) not enough, and b) because of the way we Plutus Tx works, that also means turning off optimization for normal Haskell code compiled in that compilation, which is not what you want.
There's also the fact that many things in base have non-elementary definitions, e.g. there is stuff that uses dataToTag instead of having a pattern-matching definition.

It doesn't seem like we can do much within the context of this issue, so I'm closing it. Feel free to reopen if you disagree.


This will validate the input.

User Story:

AS AN employee with access to sensitive data
I WANT to randomly generate a password that meets certain criteria
SO THAT I can create a strong password that provides greater security

Acceptance Criteria:
GIVEN I need a new, secure password

WHEN I answer each prompt
THEN my input should be validated and at least one character type should be selected

Every code block related to this issue has been checked. The pull request is approved


Add JQuery click event

Please add a on-click date event.

@dipenspaceo I could use this aswell, have you managed to get something working?


Allow excluding certain types in payload output.

This PR add support for excluding certain payload types from output both in the vrps command and the HTTP server.
For the vrps command, new command line options --no-route-origins, --no-router-keys, and --no-aspas have been added. For the payload output endpoints of the HTTP server, a new excludes query parameter can be used to exclude routeOrigins, routerKeys, or aspas.
These flags are available even for formats that don’t actually support outputting all these payload types.
When router keys or aspa has not been disabled, those payload types are now properly excluded rather than adding empty sections for them.
Fixes #852.

Come to think of it, now that the object members for excluded payload types disappear completely, we can add router keys and ASPA to regular JSON – they will not be present unless router keys and ASPA is enabled explicitly.


sync.yml for _Quizzes repository needs to have its own documentation

Right now we only talk about OTTR_Template's sync.yml in the docs so it leaves people guessing about the sync.yml in OTTR_Quizzes.
Docs docs and more docs.

This has been specified in the docs now


Make the Gatekeeper test accept status messages in different order

The order sometimes varies, but it doesn't happen often. This accounts for this.

/hold

Kind test failed?!

I had addressed this flake in my PR, but I've pulled this commit over since it's a better solution: https://github.com/open-cluster-management-io/governance-policy-framework-addon/pull/49

/remove-hold


Importer: Refactor for more accuracy

Keep track of succeeded, skipped, and errored imports in the main loop, and report on them at the end of it.

What's a sample output from this?

If this is too much work feel free to ignore, but I think it would be really clean to hide all logs etc and just show a summary at the end:
Sample:
Found 10 unimported resources.
Imported 6 resources:

google_project
google_project_service["foo"]
...

Skipped 2 resources:

google_project_lien
...

Failed to import 2 resources:

google_service_account: reason for error
...

Then in -verbose mode it will show detailed logs etc. Or maybe we can use https://github.com/google/glog and just log at different verbosities.

If this is too much work feel free to ignore, but I think it would be really clean to hide all logs etc and just show a summary at the end:
Sample:
Found 10 unimported resources.

Imported 6 resources:
- google_project
- google_project_service["foo"]
...

Skipped 2 resources:
- google_project_lien
- ...

Failed to import 2 resources:
- google_service_account: reason for error
- ...

Then in -verbose mode it will show detailed logs etc. Or maybe we can use https://github.com/google/glog and just log at different verbosities.

Done. Used a template for this.


puppet: Swap the one use of the cron resource for an /etc/cron.d file.

The cron resource places its contents in the user's crontab, which
makes it unlike every other cron job that Zulip installs.
Switch to using /etc/cron.d files, like all other cron jobs.
Testing plan: Test-applied on a host.

Thanks for cleaning this up; I definitely did not anticipate the cron resource placed it in that user's crontab.


Add pallet allowing users to stake and receive rewards

Re: #156
Some questions @nikoladimitroff

Should we have a minimum stake amount? Power users may make frequent staking TXNs, but I don't want to gatekeep poorer users.
What should the staking lock period be? I set it to 30 days for now, but we can change it.

Small TODOs before submitting:

[ ] Minimum stake amount
[x] Stake for multiple durations.
[X] Investigate using ReservableCurrency instead of our own holding account.
[ ] Increment amount instead of replacing if multiple stakes for a user in the same block.

@shelbyd

the avg daily income right now is > 4 FCL so I think a minimal stake amount of 10 FCL is reasonable (~3 days worth of current data capture, ~5 days worth of future KYC + data capture)
Let's have a couple of locking periods which increase the number of shares you get from the rewards (just like we wanted the holder program to reward people with more coindays). Let's have 30d, 60d, 90d as options for now

@nikoladimitroff Do we want to automatically withdraw the staked amount (plus new tokens) when the lock period expires, or allow it to stay staked, but unlocked, until the user chooses to withdraw?
With manual withdrawals, the user could have a significant balance currently unlocked and be receiving a large share of rewards. For example, if they locked for the maximum duration and haven't withdrawn.
With automatic withdrawals, users need to re-stake tokens and actively maintain this locked stake.
There are some more benefits / drawbacks, but I wanted to start the conversation. We can also VC about it with Julio and/or Antoni if we think that's necessary.

@nikoladimitroff Do we want to automatically withdraw the staked amount (plus new tokens) when the lock period expires, or allow it to stay staked, but unlocked, until the user chooses to withdraw?
With manual withdrawals, the user could have a significant balance currently unlocked and be receiving a large share of rewards. For example, if they locked for the maximum duration and haven't withdrawn. Additionally, users will need to pay a TXN fee to withdraw their stake.
With automatic withdrawals, users need to re-stake tokens and actively maintain this locked stake.
There are some more benefits / drawbacks, but I wanted to start the conversation. We can also VC about it with Julio and/or Antoni if we think that's necessary.

@shelbyd I propose a third one - similar to how bank deposit work.

You lock some currency for a certain amount of time
After that time expires, you get access to your funds + the interest
You could choose to re-deposit. You could choose to withdraw. You could just to do nothing and the funds just stay there without earning anything further.

This approach:

forces people to actively manage their staked funds (which increases engagement)
doesn't reward people who haven't manually withdrawn their funds as in your first proposal
since people are incentivized to either restake or withdraw, this gives us a better understanding of the amount of FCL currently locked and the liquid amount
allows users to restake without paying a TXN fee

This seems effectively identical to automatic withdrawals, but with more explanatory and implementation complexity. I think that we should go with automatic withdrawals for all the reasons listed.
Note: Users would still need to pay a TXN fee to restake/withdraw. Users need to pay TXN fee for any user-initiated on-chain activity.


SNI Object routes sidebar doesn't link to corresponding section

The SNI Object routes link in the sidebar of https://getkong.org/docs/ does not link to the SNI Objects section of the docs.

@chrisjoywebb can't reproduce this, seems fixed.

indeed, see https://github.com/Mashape/getkong.org/pull/397/files


Default Background/Foreground found in the schema doesn't match with what shows up in Terminal.

Environment
Windows build number: Microsoft Windows [Version 10.0.19013.1102]
Windows Terminal version (if applicable): 0.6

Steps to reproduce
Create a color scheme with foreground as #ffffff and background as #000000 to represent white and black respectively. Create two profiles of cmd, assign one the custom color scheme, and the other with no color scheme to give it the default background and foreground. The default profile has a slightly lighter black background, and a slightly darker white foreground.
Expected behavior
The schema found here: https://aka.ms/terminal-profiles-schema says that the default background is #000000 (black) and the default foreground is #ffffff (white).

Maybe its because of the acrylic/transparancy effect?

Hmm, this is a bit confusing. The trick here is that the default value of "colorScheme" is "Campbell".
That documentation should probably be updated to not give a default value for those settings, since they're unset by default 😄

not give a default value

So, the default in schema is actually the value that shows up as a suggestion or gets automatically input in a compliant editor— it’s apparently less about what the default is and more about user education?

Ah, the default scheme being Campbell was tripping me up a little. 😅 I was in the mindset of thinking of default as "what the fg/bg color should be when I don't set it in either the profile or the color scheme".
Looks like in the schema, background/foreground block doesn't have a default property, and so when adding fg/bg to the settings json, it simply auto fills a "#" and lets you put the hex in yourself. But, hovering over the key for the description tells you this: "Default #000000 (black).".


Fix infinite loop in exception formatting when exceptions have cyclic references

The GetFormattedExceptionMessage method in ExceptionHelper.cs recursively traverses the inner exceptions of an exception to build a formatted error message string. However, it didn't handle the case where the inner exceptions have a cyclic reference, causing an infinite loop.
This change fixes that issue by keeping track of the exceptions seen during traversal in a HashSet. If an exception is encountered again, it breaks out of the loop and appends a "[Cyclic Exception Reference]" message to the output, avoiding the infinite recursion.
The key changes are:

Introduce a HashSet seenExceptions to keep track of exceptions encountered so far.
Check if the current exception has already been seen. If so, append a "[Cyclic Exception Reference]" message and break out of the loop.
If the current exception hasn't been seen, add it to the seenExceptions set and continue processing as before.
A new test class ExceptionHelperTests has also been added with two test methods:

GetFormattedExceptionMessage_WithCyclicExceptions_ShouldNotLoopInfinitely - Verifies that cyclic exceptions are handled correctly and don't cause infinite loops.
GetFormattedExceptionMessage_WithNoCyclicExceptions_ShouldFormatNormally - Ensures that the normal formatting still works as expected for non-cyclic exceptions.

This fixes #3971

@microsoft-github-policy-service agree

I still don't understand this. An exception can't have a cyclical inner exception reference without using reflection to hack it (unless someone can give me source code to prove me wrong?) And that would hang on the throw anyway so this wouldn't solve that?

Good point! I assumed that the throw was good and it was hanging inside MSTest but that's the throw that's causing exception so nothing to do on MSTest side.
Thanks @thomhurst


If a polyline is one coordinate, it is not updated.

I have some code like this in my render() function.
const {poyline} = currentResult;
if (polyline) {
  polyline = <MapView.Polyline
  coordinates={polyline}
  strokeWidth={4}
  strokeColor={'#000'}/>;
}

As the current result is updated, the polyline is meant to be redrawn on the map. However, this does not happen if the polyline has exactly one coordinate (i.e., polyline = [{lat: 30, lng: 30]} and the old polyline remains on the map.

Can you help me to draw polyline on android
I am having this issue :
https://github.com/airbnb/react-native-maps/issues/1095
Please Help me


disable alternative URLs from useHead, SEO plugin?

Details
I have the SEO plugin that adds alternative URLs to the pages meta via useHead, I am not using the autoI18n plugin.
I noticed that the alternative URLs are being added to the sitemap.
I want these disabled, so I tried adding:
autoI18n: false to the sitemap config, but they are still being added.
How can I disable the alternative URLs in the sitemap that are getting added from the useHead SEO plugin?

Hey, can you provide a reproduction? This module and all child modules do not add any alternative tags using useHead.
autoI18n: false should disable any alternatives

autoI18n: false doesnt disable the alternatives in this case. Also Nuxt I18n isn't even being used at all.
We are manually adding the hreflag to via useHead() like soo....
{rel: 'alternate', href: 'https://www.domain.ca' + route.path, hreflang: 'en-CA'},
{rel: 'alternate', href: 'https://www.domain.eu' + route.path, hreflang: 'en-BE'},

that is where it is getting them from, but there is no way to tell it NOT to include them in the sitemap output.

Can you provide a reproduction?

not really sorry. But there is nothing unusual/custom about how we use it.

Okay but I can't help without one so I'll close this until there's one available

the bug it still there, so you should keep it open until it is fixed.

It's not clear how to reproduce the big so I can't fix it


Recommended resources for a kubei container

Hello Team and thanks for sharing this project with the open source community !
I'd like to deploy kubei on my cluster and couldn't see any resources limitations in the deployment object.  What are the minimum RAM and CPU a kubie container needs in order to operate well?
Thanks!

Hey @chencivalue, thanks a lot for pointing out the issue. We added the required resources and limits as we measured it across several deployments. Thanks!!

Appreciate it, thanks!


Hyperparamaters

Sorry, I'm new to Deep Learning.
May I know what are the hyperparameters you used to get VGG19  of Test_acc： 94.646% and Resnet18 of Test_acc： 94.040%
Mine seem to always be getting an average accuracy of 88% after 10 folds for both.
I used lr 0.01, bs 128, epoch 40, learning_rate_decay_start = 20, learning_rate_decay_every = 1 and learning_rate_decay_rate = 0.8

please try again~I will make it reproducible via random seed if i have time.


Update NativeIO name to Foundation Storage API

This PR updates the README and security questionnaire with our new name. The examples and IDL have not been updated, to avoid confusion until the change lands in Chrome.
I'll rename the repos and create redirects after this PR is merged.

Is it possible to link to the discussion that led to the new name (if an online discussion) or provide a synopsis (if an offline discussion)?

yea, the new name is confusing.  wasn't the whole point to provide/virtualize"native", system-level file-api's for emscripten/wasm-sqlite (something wicg's filesystem-api could not do)?

Hello @asutherland, @kaizhu256, thanks for reaching out,
The reason the name was changed is two-fold:

There is an overall effort in Chrome to make our codebase more inclusive (for an example in public documentation, Native is a discouraged term here)
We received feedback that the name was unclear (both internally, and in our early TAG review, where Storage Foundation API was received positively)

After a long-running internal discussion, we settled on the new name because it nicely conveys the idea that this is a low-level API, and that one of its main use cases is to act as the basis for more complex storage semantics. Of course, we are open to feedback and change, especially if there are strong reasons on why we shouldn’t go for Storage Foundation API.

are there use-cases outside of database/streaming-media? if not how about something along lines of "dbfs" or "streamfs"?
"foundation-storage-api" doesn't help in educating ppl why wicg-filesystem-access-api is inappropriate for above scenarios (and reason for this proposal).

Thank you for the context and links!  The new name seems to nicely address both concerns!


Improvement proposal

Hi, dear Razikus!
I'm currently working with openBalena and implement parts of its solutions by myself.
Your dashboard is stunning and must be clear - so helpful for me.
Currently, I`m trying to solve the problem of pushing data to openBalena registry via GitHub actions using the custom scenario.
OpenBalenaCLI GithubAction
But for this moment have no idea about the next steps.
If you need so - I have an idea of how to ensure the main openBalena to use with your dashboard builtin.
So underline my main proposal:

Integrate your dashboard as part of openBalena server
Automate development of apps, and image creation using Github Actions
Customize Quzar with new styles to make it more user-friendly

Waiting for your response!

Hello Mark!
It's very good idea to make something like this.
I would prefer to make more "agnostic" balena builder schema
Something like Git Hook (https://git-scm.com/book/en/v2/Customizing-Git-Git-Hooks)
So scenario here can be like balena cloud - push to special open-balena-git repository -> hook executing balena build -> logs streamed to terminal
But then we are losing the power of Github Actions and build-in server. But anyway it can be still integrated to Open Balena.
We have working solution with GitlabCI that is building and pushing data over registry. It can even pull images from private repo.
It uses also my other repo that is logging in before any action specified
https://github.com/Razikus/balena-dockerized-cli
It's very simple here
docker-deploy-master:
  image: razikus/balena-dockerized-cli:11.35.5
  services:
    - docker:dind
  script:
    - >
      echo -e "'registry.gitlab.com':" >> /registry-secrets.yml
    - >
      echo -e "    username: '$BASEIMAGELOGIN'" >> /registry-secrets.yml
    - >
      echo -e "    password: '$BASEIMAGEPASSWORD'" >> /registry-secrets.yml
    - DEBUG=1 balena deploy blabla --source . --emulated --build --nocache -h docker -p 2375 --logs --registry-secrets /registry-secrets.yml --buildArg BLABLA=$BLABLA"

Hello Mark!
It's very good idea to make something like this.
I would prefer to make more "agnostic" balena builder schema
Something like Git Hook (https://git-scm.com/book/en/v2/Customizing-Git-Git-Hooks)
So scenario here can be like balena cloud - push to special open-balena-git repository -> hook executing balena build -> logs streamed to terminal
But then we are losing the power of Github Actions and build-in server. But anyway it can be still integrated to Open Balena.
We have working solution with GitlabCI that is building and pushing data over registry. It can even pull images from private repo.
It uses also my other repo that is logging in before any action specified
https://github.com/Razikus/balena-dockerized-cli
It's very simple here
docker-deploy-master:
  image: razikus/balena-dockerized-cli:11.35.5
  services:
    - docker:dind
  script:
    - >
      echo -e "'registry.gitlab.com':" >> /registry-secrets.yml
    - >
      echo -e "    username: '$BASEIMAGELOGIN'" >> /registry-secrets.yml
    - >
      echo -e "    password: '$BASEIMAGEPASSWORD'" >> /registry-secrets.yml
    - DEBUG=1 balena deploy blabla --source . --emulated --build --nocache -h docker -p 2375 --logs --registry-secrets /registry-secrets.yml --buildArg BLABLA=$BLABLA"

We should think about it and make some decision

It is here https://docs.solar-control.tech/openbalena/open-balena-push-analog.
Commit in master repo is actions which is signalize that your code is need to be builded and push widely to all devices.
Dividing by profiles/branches is also good practice in development flow, like:
master -> production
develop -> develop application
feature/* -> your feature envs.
Using my action you are have possibility to execute os configure, or preload command, to upload your customized images  as artifacts directly in repo with project.

Wow it looks very nice.
Sorry i'm not very familiar with Github actions but it looks like Gitlab CICD
Is it 100% free? Any running worker is free or there are some free running hours available?
With super-nice scenario we can even integrate open-balena-dashboard with automatically creation of your actions within it
https://docs.github.com/en/rest/reference/actions#enable-a-selected-repository-for-github-actions-in-an-organization

@Razikus
It has default worker limits, like 72 hours for one build, and 1000 API requests in an hour - so limits so low.
It is 100% free of the cause of self-hosting.
And it is possible to add SHA to the application variables for checking release versions too)
Github allows to use API, so we can create it directly into the dashboard to provide BalenaCloud functionality in OpenSouce project :)

Hi! Can you provide a forking possibility, please?
Now I'm working with improving the UI side, and cannot create a fork from your repo :(


[FLINK-4334] [quickstarts] Correctly exclude hadoop1 in quickstart POMs

Thanks for contributing to Apache Flink. Before you open your pull request, please take the following check list into consideration.
If your changes take all of the items into account, feel free to open your pull request. For more information and/or questions please refer to the How To Contribute guide.
In addition to going through the list, please provide a meaningful description of your changes.

[x] General

The pull request references the related JIRA issue ("[FLINK-XXX] Jira title text")
The pull request addresses only one issue
Each commit in the PR has a meaningful commit message (including the JIRA id)

[x] Documentation

Documentation has been added for new functionality
Old documentation affected by the pull request has been updated
JavaDoc for public methods has been added

[ ] Tests & Build

Functionality added by the pull request is covered by tests
mvn clean verify has been executed successfully locally or a Travis build has passed

Thanks, this makes a lot of sense.
We should merge this for Flink 1.1.1
For Flink 1.2.0, we should actually fix the root issue, which is that the hadoop1 artifact has a scala suffix even though it does not need one. I think this was introduced by accident.

I merged this into the 1.1 release branch.
For the 1.2 releases, we should use this fix: #2343

@rehevkor5 Since the Apache bot does not automatically close this pull request, could you close it manually?

You got it @StephanEwen


Add job_id to worker

{ ...work.environment, INPUT_DIR, OUTPUT_DIR, CHECKPOINT_DIR }
{ ...work.environment, INPUT_DIR, OUTPUT_DIR, CHECKPOINT_DIR, SALAD_JOB_ID: work.id }
Can we add the salad job_id to the worker has access to it for logging purposes?

This will be included in the next release 0.5.0. There's a draft PR in #8 if you want to track the status.


Weighted class selection

Appeared to work after initial testing, will update later to fully remove the debug logs.
Unfortunately this approach doesn't work unless the Use custom settings checkbox is ticked.
https://github.com/Favid/ResistanceOperationOverhaul/blob/6542ed6cb893823791c6dd4f5b728bcbb30f392c/Src/ResistanceOperationOverhaul/Classes/UITLE_LadderModeMenu_Override.uc#L1273-L1278
Is there some specific reason why InitSquad() is only called with custom settings enabled? Moved the call locally outside the if() and everything still seemed  to work, though did not test further than starting the first mission.

Appeared to work after initial testing, will update later to fully remove the debug logs.
Unfortunately this approach doesn't work unless the Use custom settings checkbox is ticked.
https://github.com/Favid/ResistanceOperationOverhaul/blob/6542ed6cb893823791c6dd4f5b728bcbb30f392c/Src/ResistanceOperationOverhaul/Classes/UITLE_LadderModeMenu_Override.uc#L1273-L1278
Is there some specific reason why InitSquad() is only called with custom settings enabled? Moved the call locally outside the if() and everything still seemed to work, though did not test further than starting the first mission.

This is intentional - if Enable Custom Settings is not checked, then everything should work as if this mod is not even installed. So it should use vanilla behavior for generating the squad, you shouldn't get credits and science, there won't be rewards/upgrade/choose mission screens, etc.

Hmm, I may have missed a memo somewhere, but I thought that option would only allow for changing the default option, not enable/disable the whole mod. I would change the naming to reflect that.

It's mentioned in the readme but I agree that it's not very clear. I'm panning on renaming the label to Enable Resistance Operation Overhaul and maybe adding a tooltip


Potential Bug: Formatting hangs while formatting file.

Hey folks,
Did not know how to classify this since I can't guarantee it's a bug.
As I was trying to format the following js code:
https://cdn.shopify.com/shopifycloud/shopify/assets/checkout_2021-01-25-ca23aa375cfbe97f7ea054a0e4816dc8adc01895e1301ccd89148d508f60df5e.js
The formatting never completed even after waiting for 30 minutes for the process to end. The same file can be beautified in the browser in a couple of seconds.

The code right now is not optimized for formatting minified files. It seems to take a long time in the playground too, so not a wasmer issue.
Would be good if I implemented https://github.com/dprint/dprint/issues/263 to catch these more easily, but yeah there should be something done here to make this faster.

Thanks @dsherret, really appreciate that.

@aancw no, not yet. dprint-plugin-typescript is not very good at minified files at the moment and it would require maybe a day of investigation and work to fix it. I recommend excluding these files for now.

@dsherret Noted. Thank you for your dedication in this lib :)


[MERGE] merge facl branch to master

@gpang merge facl branch to master, PTAL.

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/Alluxio-Pull-Request-Builder/20642/
Test FAILed.

jenkins, test this please

Jenkins, test this please

@gpang PTAL, conflict-free merge

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/20643/
Test PASSed.

Merged build finished. Test PASSed.

I think @calvinjia should also take a look.