Deducing return types from boolean arguments does not work with JSDoc and default argument values

🔎 Search Terms

typescript template default argument
jsdoc infer template from default argument

I found this but it is closed so probably not the same issue: https://github.com/microsoft/TypeScript/issues/29401

🕗 Version & Regression Information
I don't know how to determine the exact version VSCode is using, but I just updated it...
⏯ Playground Link
https://www.typescriptlang.org/play/?jsx=0&filetype=js#code/C4TwDgpgBAKuEGEAWB7AlgYwgHhlCAHsBAHYAmAzlAEYooA2EAhiQDSwBiaAThcOzADKEDCnIA+KAF5Y+IqUpRg3AK7QA-Jx58oALljDR5ANwAoU6EhQAIiKYgAovPIVc7ABpziL2JJkwzUwB6ACoQ0ygQqAABYgBbMHomYigAb1oGZhIAXygANQAlCGAVbhIKADkVOOoIbgio6LAmbiY4tNsMeydvSmwMxhZ2QuLS8qqauvFc7lGyyura7igGmNmS+bS4SGR0LGwRjfHFuvYSE+52Pm40EgBzaYag0wAzFRIMYDQxKHWxgCEQLtMBAABR-eYTJbSJSqCAASjSESgKORKKCQRiwAoAFo0HcSChZmjfnMSFBzvR6GZsuYjDoKKgVPQyP8IFC6jCISRAcCsKDlGp4WZ6cAoIyUMzWRBBMpbncuWTeagQaCXkx6BQEWYMVAAIKfFQaqBIW7AfSi8VMll6zUoNkc7j6a7yqAAHwpF1MlolUttFHt7IuiqOyr2YOF5iAA
💻 Code
This problem occurs when trying to use JSDoc with TypeScript conditional types. The conditional type I created is the following:
type TypeChoice<T extends boolean, TFirst, TSecond> = T extends true ? TFirst : TSecond;

Application in plain javascript:
/**
 * @template {boolean} VReturnsNumber
 * @param {VReturnsNumber} returnsNumber
 * @returns {TypeChoice<VReturnsNumber, number, string>}
 */
// Error in the argument:
// Type 'boolean' is not assignable to type 'VReturnsNumber'.
//   'boolean' is assignable to the constraint of type 'VReturnsNumber',
//   but 'VReturnsNumber' could be instantiated with a different subtype of constraint 'boolean'.ts(2322)
function returnByChoice(returnsNumber = true) {

    // @ts-ignore
    return null;
}

// These two work fine and show the desired type
const shouldBeNumber = returnByChoice(true);
const shouldBeString = returnByChoice(false);
// Actual hint: const shouldAlsoBeNumber: string | number
// default argument is not applied
const shouldAlsoBeNumber = returnByChoice();

🙁 Actual behavior
Error on the default argument:
Error in the argument:
Type 'boolean' is not assignable to type 'VReturnsNumber'.
  'boolean' is assignable to the constraint of type 'VReturnsNumber',
  but 'VReturnsNumber' could be instantiated with a different subtype of constraint 'boolean'.ts(2322)

Additionally, the type is unknown if no argument is provided.
🙂 Expected behavior

The default argument should not cause an error. At the very least, it should not cause an error when the base type for the template is a primitive type
The default arguments should be used to infer the template if possible

Additional information about the issue
I also tried to use this to suppress the error, but it kills the type inference in the process (the result type is always number | string):
type DecayExtends<T, X extends T> = T;

/**
 * @template {boolean} VReturnsNumber
 * @param {DecayExtends<boolean, VReturnsNumber>} returnsNumber
 * @returns {TypeChoice<VReturnsNumber, number, string>}
 */
function returnByChoice(returnsNumber = true) {

    // @ts-ignore
    return null;
}

Looking more into the other issue I found, I can see that problem number 2 can be manually fixed by stating the default value. I also tried stricter template constraint, but the error still persists:
/**
 * @template {true|false} [VReturnsNumber=true]
 * @param {VReturnsNumber} returnsNumber
 * @returns {TypeChoice<VReturnsNumber, number, string>}
 */
function returnByChoice(returnsNumber = true) {
    // @ts-ignore
    return null;
}

The error still is:
Type 'boolean' is not assignable to type 'VReturnsNumber'.
  'boolean' is assignable to the constraint of type 'VReturnsNumber', but 'VReturnsNumber' could be instantiated with a different subtype of constraint 'boolean'.ts(2322)

In this case, it is clearly wrong, as there is no way VReturnsNumber could be instantiated to anything except true or false.

The error is correct, because generic arguments can be provided explicitly:
returnByChoice<false>() // Why is returnsNumber true, when the type can only be false?

That is what is meant with

but 'VReturnsNumber' could be instantiated with a different subtype of constraint 'boolean'

Effectively a duplicate of #58977
@MartinJohns you're my favorite "eager noobie" here btw 👍 (this is an inside joke, OP please disregard)

If this is a duplicate would it make sense for me to create more specific issue regarding the workaround with DecayExtends not working? It seems that workaround should not lose the value type but it does. Or is that also known issue/has a reason for not working?

DecayExtends isn't going to work either because, in the presence of zero candidates, the type parameter is resolved to its constraint (it's as if you had written returnByChoice<true | false>). It's more or less the same root cause.


Feature Request: Option to render to app-specific directories

At the moment kubekutr renders by resource-type (i.e. all deployments end up in a deployments directory).
It might be nice to have the option to split these by application.
i.e render to:
./app1/deployment.yaml
./app1/service.yaml
...
./app2/deployment.yaml
./app2/service.yaml

I'm not sure whether this would require a re-structuring of the config, such that all application resources can share the name value. At the moment all config definitions seem independent from one another.
i.e. another config file could look like:
applications:
- name: app1
  deployments:
  ...
  ingresses:
- name: app2
  deployments:
  ...

Generally speaking, this might help from a development point of view, since it would allow you to focus on all resources for the specific application in question (i.e. bundle them all under the same directory).

Yeah this kind of structure probably makes more sense from a UX perspective. Will add in next release :)

@nabadger I've a rough implementation ready, but how would you like to address the issue where one app can have multiple deployments/ingress resources? Would a format like {name}-{resourceType} be fine?
For example, for the given config
workloads:
  - name: app
    deployments:
      - name: app
        ...
      - name: queue
        ...
    services:
      - name: app
        ...

How structure should look like?
myapp/app/app-deployment.yml
myapp/app/queue-deployment.yml
myapp/app/app-service.yml

I'd like to think of config as a "workload" (set of resources like deployments, services, ingresses required to run your application fully) and in that case multiple resource types are definitely possible. So we need to prefix them with the name, to make it unique. This is what I could think of, if you've better ideas please let me know :smile:


Dropdown events do not fire

Hi there,
It seems as though the dropdown events (i.e. hidden.bs.dropdown. show.bs.dropdown, etc.) do not ever seem to fire. For example:
https://www.bootply.com/xVuE4LjekJ#
The alert never fires.
I suspect this is due to some namespace conflict, perhaps between popper and bootstrap.
It also does not work if the dropdown is NOT in a nav: https://www.bootply.com/BNRritTKHC#
It's of course possible that I have a code problem, but I can't see it.
Thanks.

That's because dropdown are wrapped in a container and it's this container which will trigger the events see my CodePen with our latest release (beta 2) :
https://codepen.io/Johann-S/pen/JOXqZZ?editors=1010

Hi @Johann-S ,
Thanks for that. I have confirmed in my code that the event fires on the PARENT element; in my actual code it's more complex than just nested in a single container, so I wanted to mention that's the correct place to listen for the event.
I would request one or both of the following:

The documentation be updated, as it indicates that the events fire on the element itself and not its parents, or;
This is still a bug, at least from the perspective of how you would expect to code for this and complexity. event.target will obviously refer to the parent element, which can be useful, but generally will require a second traversal back down the tree in the code to be able to do something to the element in question (for example, disabling it): https://codepen.io/anon/pen/JOKPGr?editors=1011 This is, at the very least, inefficient.

Thanks.

It's written in our documentation here : https://getbootstrap.com/docs/4.0/components/dropdowns/#events

@Johann-S : The documentation is confusing because the example and behavior is either wrong or inconsistent.
It DOES say that it fires at the parent element, but then the example just below it uses the following code:
$('#myDropdown').on('show.bs.dropdown', function () {   // do something… })
That appears to directly target the dropdown created in non-existent example HTML, NOT the dropdown's parent.
With tabs https://getbootstrap.com/docs/4.0/components/navs/#events, a very similar example exists:
$('a[data-toggle="tab"]').on('shown.bs.tab', function (e) {   e.target // newly activated tab   e.relatedTarget // previous active tab })
Which clearly targets the element that is used to activate the tab, NOT that element's parent and NOT the tab pane itself.
Ultimately this is where my confusion stems from; I have a nav that includes both tabs and a dropdown, and the behavior is inconsistent between the two types of elements (which is fine so long as it's documented clearly).
I admit that I am barely more than a junior dev so perhaps this is obvious to more senior folks, but without a complete example (or a more consistent one to the other examples on the page, NONE of which have an element with id=myDropdown which would be selected by the example code above).
Thanks very much for the good work (and help) and for replying on a Sunday :)

I'll reopen this issue and mark as a documentation one if someone want to improve this section

@Johann-S :+1: Thank you!

Hi @djryanj ,
If you include popper.js from your html page be sure to include it in its UMD version.
say, i.e.:
<script src="/libs/popper.js-1.12.6/dist/umd/popper.js"></script>
if you have all the dependencies into a root lib of the site.
--R

Haven't seen any movement here, safe to close now or is there something we can do still?


Add Vaib Extension

Description
Fixed the issues as per review on previous PR
Checklist

[x] I read the extension guidelines
[x] I read the documentation about publishing
[x] I ran npm run build and tested this distribution build in Raycast
[x] I checked that files in the assets folder are used by the extension itself
[x] I checked that assets used by the README are placed outside of the metadata folder

@pernielsentikaer

I have resolved the CI errors and added some screenshots as per the docs

🎉 🎉 🎉
Such a great contribution deserves a reward, but unfortunately we couldn't find your Raycast account based on your GitHub username (@Vaib215). Please link your GitHub account to your Raycast account to receive your credits and soon be able to exchange them for some swag.

Haven't received any credits


implement table layer for kvdb

:warning: Please install the  to ensure uploads and comments are reliably processed by Codecov.
Codecov Report
Attention: Patch coverage is 59.77961% with 146 lines in your changes missing coverage. Please review.

Project coverage is 81.25%. Comparing base (21c8821) to head (dfc265f).

Files with missing lines
Patch %
Lines

table.go
62.17%
94 Missing and 38 partials :warning:

pkg/utils/utils.go
0.00%
14 Missing :warning:

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@            Coverage Diff             @@
##             main      #11      +/-   ##
==========================================
- Coverage   88.75%   81.25%   -7.50%
==========================================
  Files          12       13       +1
  Lines        1040     1403     +363
==========================================
+ Hits          923     1140     +217
- Misses         85      193     +108
- Partials       32       70      +38

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


datadog 5: dd-agent.service misses gohai dependency

Issue description
systemctl status dd-agent.service | grep -o 'dd.collector.*gohai.*'
dd.collector | checks.collector(collector.py:793) | gohai file not found
dd.collector[13874]: INFO (collector.py:793): gohai file not found

Expected output: empty
gohai can be found here: https://github.com/DataDog/gohai
This holds for nixpkgs master.

I've submitted #41015 to fix this issue.
We've worked around it until this is released by adding the gohai derivation to our NixOS configuration repository and adding it to the unit's path, like so:
# Add gohai dependency to unit's $PATH.
# See: https://github.com/NixOS/nixpkgs/issues/39810
systemd.services.dd-agent.path = [ pkgs.gohai ];

NixOS will merge that option together with the already existing derivations on the unit's path, so it doesn't break the existing configuration.
We've been running this change on a dozen or so live-servers and it works fine.
Side-note: @coretemp if you're a frequent Datadog + NixOS user and are interested in getting Datadog agent v6 packaged (+ maybe have time to help out, it's got a very complicated build process) please let me know! :)

The fix is now in 18.03 :)


Fixed the Ordered List Issue

Pre-Submission Checklist

[x] Your pull request targets the staging branch of FreeCodeCamp.
[x] Branch starts with either fix/, feature/, or translate/ (e.g. fix/signin-issue)
[x] You have only one commit (if not, squash them into one commit).
[x] All new and existing tests pass the command npm run test-challenges. Use git commit --amend to amend any fixes.

Type of Change

[x] Small bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds new functionality)
[ ] Breaking change (fix or feature that would change existing functionality)
[ ] Add new translation (feature adding new translations)

Checklist:

[x] Tested changes locally.
[x] Closes currently open issue (e.g. Closes #XXXX): Closes #8501

Description

Changed the tests in seed/challenges/01-front-end-development-certification/html5-and-css.json to the following:

  "assert.equal($(\"ol\").prev().text(), 'Top 3 things cats hate:', 'message: You should have an ordered list for \"Top 3 things cats hate:\"');",

   "assert.equal($(\"ul\").prev().text(), \"Things cats love:\", 'message: You should have an unordered list for \"Things cats love:\"');"

:sparkles: :sparkles: LGTM. :sparkles: :sparkles:
:tada: Thanks for your contribution & Congratulations on your first submission to Open Source :tada:


Problem with adding wind fields to the fieldset

Hi,
I'm trying to add the wind fields to my fieldset in order to implement the windage; this is the code that I'm using:
data_current = "20180101_h-CMCC--RFVL-MFSe3r1-MED-b20200901_re-sv01.00.nc"
data_wind= "CRMA-WRF_10mwinds_Adriatic_2018010100-2018010123.nc"
filenames_current = {'U': {'lon': data_current, 'lat': data_current, 'data': data_current},                                   'V': {'lon': data_current, 'lat': data_current, 'data': data_current}}
variables_current = {'U': 'u', 'V': 'v'}
dimensions_current = {'U': {'lon': 'lon', 'lat': 'lat',  'time': 'time'},                                      'V': {'lon': 'lon', 'lat': 'lat',  'time': 'time'}}
filenames_wind = {'Uwind': {'lon': data_wind, 'lat': data_wind, 'data': data_wind},                               'Vwind' :{'lon': data_wind, 'lat': data_wind, 'data': data_wind}}
variables_wind = {"Uwind": "air_u", "Vwind": "air_v"}
dimensions_wind = {'Uwind': {'lon': 'lon', 'lat': 'lat',  'time': 'time'},                                  'Vwind': {'lon': 'lon', 'lat': 'lat',  'time': 'time'}}
fieldset = FieldSet.from_netcdf(filenames_current, variables_current, dimensions_current)
fieldset_wind= FieldSet.from_netcdf(filenames_wind, variables_wind, dimensions_wind)
fieldset.add_field(fieldset_wind.Uwind)
fieldset.add_field(fieldset_wind.Vwind)
And this is the advection kernel that I'm using for my first tests:
def Advection2(particle, fieldset, time):
(uc1, vc1 )= fieldset.UV[time, particle.depth, particle.lat, particle.lon]
 u1, v1 = fieldset.Uwind[time, particle.depth, particle.lat, particle.lon],fieldset.Vwind[time, particle.depth, particle.lat, particle.lon]
ua1 =  u1+uc1
 va1 =   v1+vc1
particle.lon+= ua1 * particle.dt
 particle.lat += va1 * particle.dt
When I run the simulation I get this warning: WARNING: ParticleSet is empty on writing as array at time 7200.
I don't know what the problem could be. I've tried to read the wind fields as currents in order to see if the problem was the lecture of the netcdf file but it has been read correctly and the particles have been advected.
Anyone has an idea how I could solve this problem?
Thank you!

Hi @ClaudiaFarris, could it be that your wind fields don't have their units converted? See also the UnitConverter tutorial at https://nbviewer.org/github/OceanParcels/parcels/blob/master/parcels/examples/tutorial_unitconverters.ipynb


Add a Daml Hub deploy script

and please attend the warnings about Bash, e.g. replace all ${VAR} with "$VAR" (or "${VAR}" but that's totally unnecessary)

and please attend the warnings about Bash, e.g. replace all ${VAR} with "$VAR" (or "${VAR}" but that's totally unnecessary)

what else?

I tried:

on DABL with the uploader
locally, with Canton
locally, with Sandbox
I followed the README.


Add an API that will log ChipLogError via NSLog

Problem
https://github.com/project-chip/connectedhomeip/pull/14816#discussion_r802035535
Proposed Solution

Add a new API that will log a ChipError using NSLog

It's not clear that we need this, given that ChipLogEror is using the os_log bits....

Not required for 1.0

I have verified that ChipLogError ends up using os_log_with_type(..., OS_LOG_TYPE_ERROR, ...).


Update to docker GA V2

See #5. Changes adopted from https://github.com/docker/build-push-action/blob/master/UPGRADE.md

Superseded by #15


[KNI][4.15] fix commit check

The env var COMMITS may be empty. Handle it accordingly.
Signed-off-by: Francesco Romani<EMAIL_ADDRESS>(cherry picked from commit 19127985f7cce4a175665fafba15127c763d5aa7)

/close

/reopen

unrelated infra issue


Adding TLS support to prometheus instance

TLS support enables endpoints to be more secured.
Signed-off-by: Arun Kumar Mohan<EMAIL_ADDRESS>

@aruniiird In addition to the above comments on the code.
I don't see the needed changes to the prom CR to facilitate TLS communication. How is that done?

@nb-ohad , @bindrad , please take another look...

I'm pretty much blocked by these errors while running testcases,
• [FAILED] [3.054 seconds]
ManagedOCS controller
/home/amohan/aruns/programming/go/ocs-osd-deployer-project/src/github.com/red-hat-storage/ocs-osd-deployer/contr
ollers/managedocs_controller_test.go:30
  reconcile() converged mode
  /home/amohan/aruns/programming/go/ocs-osd-deployer-project/src/github.com/red-hat-storage/ocs-osd-deployer/con
trollers/managedocs_controller_test.go:506
    a Grafana datasources secret exists in the openshift-monitoring namespace
    /home/amohan/aruns/programming/go/ocs-osd-deployer-project/src/github.com/red-hat-storage/ocs-osd-deployer/c
ontrollers/managedocs_controller_test.go:1245
      [It] should create k8sMetricsServiceMonitorAuthSecret in primary namespace
      /home/amohan/aruns/programming/go/ocs-osd-deployer-project/src/github.com/red-hat-storage/ocs-osd-deployer
/controllers/managedocs_controller_test.go:1246

  Begin Captured GinkgoWriter Output >>
    2022-04-05T05:47:03.780+0530        INFO    controllers.ManagedOCS  Starting reconcile for ManagedOCS      {
"req.Namespace": "primary", "req.Name": "managedocs"}
    2022-04-05T05:47:03.782+0530        INFO    controllers.ManagedOCS  Non provider deployment, skipping reconc
ile for onboarding validation secret
    2022-04-05T05:47:03.782+0530        INFO    controllers.ManagedOCS  Reconciling StorageCluster
    2022-04-05T05:47:03.782+0530        INFO    controllers.ManagedOCS  Reconciling CSVs
    2022-04-05T05:47:03.782+0530        INFO    controllers.ManagedOCS  Reconciling alertRelabelConfigSecret
    2022-04-05T05:47:03.782+0530        INFO    controllers.ManagedOCS  Reconciling Prometheus Service
    2022-04-05T05:47:03.789+0530        INFO    controllers.ManagedOCS  Reconciling CA Bundle ConfigMap
    2022-04-05T05:47:03.789+0530        INFO    controllers.ManagedOCS  Reconciling Prometheus
    2022-04-05T05:47:03.796+0530        INFO    controllers.ManagedOCS  Reconciling Alertmanager
    2022-04-05T05:47:03.801+0530        ERROR   controllers.ManagedOCS  An error was encountered during reconcil
ePhases {"error": "resourceVersion should not be set on objects to be created"}
    sigs.k8s.io/controller-runtime/pkg/internal/controller.(*Controller).reconcileHandler

Search showed this link: https://github.com/cert-manager/cert-manager/issues/3615 , but this is for a full fledged environment not for a test setup. For test setup, didn't find a place to upgrade the cert-manager version.

Test issue is resolved (through PR: https://github.com/red-hat-storage/ocs-osd-deployer/pull/164). Re-based on top of latest main (which has the fix).
@nb-ohad , @bindrad , please take a look...


Routing With Time WIndows

Hi,
I'm experimenting with Time Windows.
I have a route  that i want it to start at 8:00 am  , with 80 stops without time windows and one  with time window 12:00 -14:00.
I tried setting the time window to  min =28800 (08:00) , max =61200 (17:00) to all stops except the one that I set it to min =43200 (12:00) , max = 50400 (14:00), but i seems like the time window is not working.
Any suggestions how to set the correct time windows for this route?

This should be fixed now in https://github.com/itinero/optimization/commit/e0ea72a7c63159ba897e8469d1d3160e1575f157 and https://github.com/itinero/optimization/commit/06f3757fe3fc00ec544e3ab897d22f635e264cc0


implement TransactionDownloader

Resolves L2B-1539

@adamiak I am wondering if the name should be TransactionDownloader. I did not like FeederGatewayCollector as it refers mainly to the logs (that are collected, we are rather downloading stuff).

Personally FeederGatewayCollector  sounds more meaningful to me. I know it's a collector (so it follows the collector pattern) and I know it's downloading from feeder gateway. With TransactionDownloader I don't know which transactions, where from, and how is this "Downloader" implemented.  I'll leave the comment in the code.
But I agree that your name is "cleaner". I just prioritize having the name being very meaningful and convey as much technical details as possible.


some partial earth image disapear

I downloaded the Cesium1.64, installed the npm and run “node ./server.js” to start the server, run “localhost:8080/index.html” to start HelloWorld.html step by step.
I found some partial earth image disapear when Cesium initialize earth image(As show in above). It does not happen every time,it more likely to happen when the google browser cache cleanup.
It gives us not a very good experience, I want to know how to solve this problem.
If anyone has the same experience or knowledge about this problem,I would like to discuss this very much.
Thank you.
My Email is<EMAIL_ADDRESS>

Are you able to reproduce this in the hosted CesiumJS hello world example?
https://cesium.com/downloads/cesiumjs/releases/1.64/Apps/HelloWorld.html
I'm not able to reproduce this with a clear cache and trying to refresh multiple times. Are you using the default Bing maps imagery and all default settings?
When this happens does that gap last or does it get filled after a few seconds? Are you able to share a video of this?
Closing for now since there isn't enough information in this issue to investigate or resolve. You can always use the Cesium forum for general questions or when you're still gathering information.


Error when running just build-ui

+ sed -i 's/lang="en"/lang="{{ lang }}"/g' templates/html/account.html
sed: 1: "templates/html/account. ...": undefined label 'emplates/html/account.html'
error: Recipe `build-ui` failed with exit code 1

It looks like the files were built
I can see a templates/html/account.html file but no replacement.
<!DOCTYPE html>
<html lang="en">
. . .

That's a bit weird.
There is the CONTRIBUTIND.md which should help with the dev setup.
On what kind of host are you trying to build?  The only reason I can imagine for now if you would not have BASH available somehow.

Yes, it is weird. It actually doesn't interfere with me using the service having built it from source. I do have BASH available, I'm on macOs 11.7.9

I don't own any Mac, so I cannot test stuff there.
Maybe the shebang is a problem on Mac. The current version is more portable, but let's check it anyway.
Please change the very first line of the build-ui recipe to #!/usr/bin/bash and try again.

https://github.com/sebadob/rauthy/pull/281


Added new dispatcher (sendshortcut)

Describe your PR, what does it fix/add?
Implemented the Request from https://github.com/hyprwm/Hyprland/issues/5631
bind = <MODKEYS><KEY>,sendshortcut,<NEW_MODKEYS><NEW_KEY>[,WINDOW_RULES]
Window rules are optional, if not given, the keys are sent to the current focused window.
This is a really nice feature so one wouldn't need simple key remaps aswell.
Is there anything you want to mention? (unchecked code, possible bugs, found problems, breaking compatibility, etc.)
I want to mention the problem with mapping a single keyname (e.g. "g") to a keycode (not keysym).
Needed to add a cache where I add keycodes to keys at the first time they are pressed (only those keys that are sent to a certain window, not all keys). Therefore I added a new field in the KeyManager Header. Code is commented adequately
Is it ready for merging, or does it need work?
Ready I guess, did the testing, but I don't know how it behaves in xwayland world....

What do you mean by temporary keymap?

wtype does exactly the same thing with iterating through all codes (https://github.com/atx/wtype/blob/d71be3a7b3f93b534a2823fd68cabd7ac2a02359/main.c#L171),  "caveman-like"  aswell :)
I could add the current keyboard to the map key, that  would allow differentiation between keyboards, but completely building a whole keymap from file would be 1. unnecessary 2. time consuming. Or do you know of any other alternative?

you can add the current keyboard I guess

Note: I didn't use your XWTOXW variable (checked if windows are xwayland) from pass. Can you check, if everything works in XWayland aswell? I did it, everything worked besides sending to "xev"

Can you please confirm that sending to XWayland works aswell? Now I can't send shortcuts to something like xev

doesn't look solvable to me. I think XWayland processes events separately and thus our timing tricks dont work. The question is whether we care. I would say "no".
This diff removes some redundant comments, cleans up code, and makes wl -> xwl work, leaving xwl -> xwl in a kinda broken state (will send to current)
patch.txt
test it, commit it, and if everything's ok we can ship this

Is it my fault or where is the diff?

It works
I am happy
and this feature is extremely nice


Refresh specs for Kernel.Integer

And remove our local test for a base argument that cannot be coerced into an integer, these have been accepted upstream (including version guards and a spec for the fix in Ruby 3.3).

I tried running our test suite with ruby-3.0-preview3, a few of our local specs fail. I'm trying to get those tests into the upstream specs with version guards, so we can at Ruby 3.3 to the CI as soon as it's released.


dsl bug fixed

event module's macro definition bug fixed
https://github.com/cpursley/walex/issues/34

Thank you so much! I made some minor changes and started a new branch:
https://github.com/cpursley/walex/pull/36


Running with large mod cache fills up /tmp

Hi :grin:!
I tried using lean, but it's filling up my 8G /tmp partition with files and then erroring with No space left on device errors:
cp: cp: error writing '/tmp/list-tmp375795932/listme/.git/objects/d3/f8ce0d0475fa4d2f6c2fd713e4910b817eb6f8'error writing '/tmp/list-tmp819901685/listme/.git/objects/d3/c868a16dba96e42c4fb3c61b07a2de0fe103a6': No space left on device: No space left on device

Perhaps the directory for temporary files could be configurable so it can run on a disk with more space?  I don't think mod chaches of comparable size is going to be unusual.

@johanbrandhorst Could you see if your issue persists at HEAD? I've added a CL that cleans up /tmp resources after they're used which should help this.

Unfortunately I'm still seeing the same "no space left on device" errors with latest master and a clean 8G /tmp partition.


Standardize on a storage format

I have a very similar project here: https://github.com/ionelmc/pytest-benchmark
It depends on pytest of course but what if it could load data from perf (or vice-versa)?
pytest-benchmark storage format is in files like <run_number>_<description>.json and each file contains multiple tests. Eg: https://github.com/ionelmc/pytest-benchmark/blob/master/tests/test_storage/0001_b87b9aae14ff14a7887a6bbaa9731b9a8760555d_20150814_190343_uncommitted-changes.json
The main differences I see to perf is the multiple tests in a single file and the raw data is optional (default is to only store the computed stats).
What do you think, could there be a common format to allow interoperability?

Currently, perf only supports a single benchmark. IMHO it's not enough. Most benchmark tools support multiple benchmarks. I wrote perf for the CPython Benchmark Suite http://hg.python.org/benchmarks which contains more than 30 benchmarks. I may be annoying to have 1 json file per benchmark. At least, it would be nice to be able to store multiple benchmarks in the same JSON file.
I changed the internal format of samples recently in perf from normalized timing per loop iteration to raw timing (total for all loop iterations). I made this change to be able to more easily detect if the benchmark is unstable if a single sample is too short (ex: less than 1 us):
http://perf.readthedocs.io/en/latest/perf.html#runs-samples-warmups-outter-and-inner-loops
Not only detect that the benchmark was unstable when running the benchmark, but also after when analyzing a file.
I'm not sure that having two formats for samples (raw and normalized) is a good idea.

I changed the internal format of samples recently in perf from normalized timing per loop iteration to raw timing (total for all loop iterations).

Not sure what you mean here. What's this normalization? Feel free to point me to places in code.

https://github.com/ionelmc/pytest-benchmark/blob/master/tests/test_storage/0001_b87b9aae14ff14a7887a6bbaa9731b9a8760555d_20150814_190343_uncommitted-changes.json
I looked at this example.
perf JSON perf is similar, but the main difference is that perf stores all samples instead of storing only statistics. Statistics are computed on demand. The advantage is that if perf is enhanced tomorrow, you will be able to display your benchmark result differently. For example, the first perf version didn't have histograms and only displayed mean, std dev, min and max. A new perf version may display IQR tomorrow, you will be able to analyze old benchmark results.
Another difference is that perf JSON only support one benchmark result per file, your format supports multiple results into a single file. But this difference is less important, I can enhance perf for that :-)

Another difference is that perf JSON only support one benchmark result per file, your format supports multiple results into a single file. But this difference is less important, I can enhance perf for that :-)

FYI I introduced BenchmarkSuite in the future perf 0.6. A benchmark suite is made of multiple benchmarks. A benchmark is made of multiple runs, and each run is made of samples (including warmup samples). In a suite, a benchmark is identified by its name which must be unique.

I just released perf 0.6 which now supports benchmark suites, not only individual benchmarks. A JSON file is a benchmark suite, it's the format version 2.
I don't want to modify perf format to only store statistical results, I prefer to store all samples grouped per run. If you want an interoperable format, I suggest to modify your format to use something closer to perf.
You may reopen the issue if you are interested to modify your format ;-)

Do you have an example of the multiple-suite format?

Example of a benchmark JSON file (indented for readability):

3 benchmarks (call_simple, go, telco)
each benchmark has 2 runs, each run has 1 warmup sample + 3 raw samples

{
    "benchmarks": {
        "call_simple": {
            "metadata": {
                "description": "Test the performance of simple Python-to-Python function calls",
                "cpu_model_name": "Intel(R) Core(TM) i7-3520M CPU @ 2.90GHz",
                "date": "2016-07-06T18:32:43",
                "python_implementation": "cpython",
                "python_executable": "/usr/bin/python3",
                "platform": "Linux-4.6.3-300.fc24.x86_64-x86_64-with-fedora-24-Twenty_Four",
                "python_version": "3.5.1 (64bit)",
                "perf_version": "0.6",
                "cpu_count": "4",
                "name": "call_simple",
                "aslr": "enabled",
                "timer": "clock_gettime(CLOCK_MONOTONIC), resolution: 1.00 ns",
                "hostname": "selma",
                "duration": "2.4 sec"
            },
            "warmups": 1,
            "runs": [
                [
                    0.271314807,
                    0.239580888,
                    0.237846342,
                    0.23672315
                ],
                [
                    0.271740932,
                    0.243592464,
                    0.237245481,
                    0.236748594
                ]
            ],
            "loops": 1,
            "inner_loops": 20
        },
        "go": {
            "metadata": {
                "description": "Test the performance of the Go benchmark",
                "cpu_model_name": "Intel(R) Core(TM) i7-3520M CPU @ 2.90GHz",
                "date": "2016-07-06T18:32:46",
                "python_implementation": "cpython",
                "python_executable": "/usr/bin/python3",
                "platform": "Linux-4.6.3-300.fc24.x86_64-x86_64-with-fedora-24-Twenty_Four",
                "python_version": "3.5.1 (64bit)",
                "perf_version": "0.6",
                "cpu_count": "4",
                "name": "go",
                "aslr": "enabled",
                "timer": "clock_gettime(CLOCK_MONOTONIC), resolution: 1.00 ns",
                "hostname": "selma",
                "duration": "5.5 sec"
            },
            "warmups": 1,
            "runs": [
                [
                    0.56381269,
                    0.580709772,
                    0.576461901,
                    0.658624167
                ],
                [
                    0.605411489,
                    0.601818249,
                    0.555815024,
                    0.644006216
                ]
            ],
            "loops": 1
        },
        "telco": {
            "metadata": {
                "description": "Test the performance of the Telco decimal benchmark",
                "cpu_model_name": "Intel(R) Core(TM) i7-3520M CPU @ 2.90GHz",
                "date": "2016-07-06T18:32:51",
                "python_implementation": "cpython",
                "python_executable": "/usr/bin/python3",
                "platform": "Linux-4.6.3-300.fc24.x86_64-x86_64-with-fedora-24-Twenty_Four",
                "python_version": "3.5.1 (64bit)",
                "perf_version": "0.6",
                "cpu_count": "4",
                "name": "telco",
                "aslr": "enabled",
                "timer": "clock_gettime(CLOCK_MONOTONIC), resolution: 1.00 ns",
                "hostname": "selma",
                "duration": "1.2 sec"
            },
            "warmups": 1,
            "runs": [
                [
                    0.097667996,
                    0.109760575,
                    0.10790484,
                    0.103240233
                ],
                [
                    0.109492493,
                    0.103709194,
                    0.100290215,
                    0.09819936
                ]
            ],
            "loops": 4
        }
    },
    "version": 2
}

pytest-benchmark don't store all the data by default (it's an opt-in currently) because it's designed to save runs quite often. Maybe it could be an opt out - provided there are some sensible default settings preventing 50mb output files.
Here's an example with the full data format:
{
    "benchmarks": [
        {
            "stats": {
                "mean": 2.1958498263759373e-06,
                "iqr_outliers": 3812,
                "stddev_outliers": 163,
                "iqr": 1.00000761449337e-07,
                "ld15iqr": 1.7999991541728377e-06,
                "q1": 1.8999999156221747e-06,
                "outliers": "163;3812",
                "hd15iqr": 2.1999985619913787e-06,
                "stddev": 6.292674923984151e-06,
                "max": 0.0009249150007235585,
                "iterations": 1,
                "q3": 2.0000006770715117e-06,
                "rounds": 58477,
                "data": [
                    4.300000000512227e-06,
                    2.499999027349986e-06,
                    2.099999619531445e-06,
                    1.999998858082108e-06,
                    1.8999999156221747e-06,
                    ....
                    2.70000055024866e-06,
                    2.599999788799323e-06
                ],
                "median": 2.599999788799323e-06,
                "min": 2.4000000848900527e-06
            },
            "fullname": "tests/test_lazy_object_proxy.py::test_perf[objproxies]",
            "group": null,
            "options": {
                "timer": "perf_counter",
                "max_time": 1.0,
                "warmup": false,
                "disable_gc": false,
                "min_time": 5e-06,
                "min_rounds": 5
            },
            "name": "test_perf[objproxies]"
        }
    ],
    "commit_info": {
        "id": "30e14966ebaf8f0fde100e1d43c860c42ac16541",
        "dirty": false
    },
    "version": "3.0.0",
    "machine_info": {
        "processor": "x86_64",
        "system": "Linux",
        "node": "newbox",
        "python_compiler": "GCC 4.8.4",
        "machine": "x86_64",
        "release": "3.16.0-71-generic",
        "python_version": "3.5.1",
        "python_implementation": "CPython"
    },
    "datetime": "2016-07-06T18:50:25.824890"
}

Some questions on your format ... in this:
            "runs": [
                [
                    0.271314807,
                    0.239580888,
                    0.237846342,
                    0.23672315
                ],
                [
                    0.271740932,
                    0.243592464,
                    0.237245481,
                    0.236748594
                ]
            ],
            "loops": 1,
            "inner_loops": 20

What are the inner_loops for? What's the difference to loops?
Why do you have a list of lists in runs? What does it mean?


Files left in Caskroom directories

I've noticed that many casks leave what appear to be "junk" files in their Caskroom directories.

Are these leftovers after mounting a DMG? Should they be there or should homebrew-cask be cleaning them up?

Are these leftovers after mounting a DMG?

Yes.

Should they be there or should homebrew-cask be cleaning them up?

They should be there. Sometimes you might need them, and sometimes we might (uninstall scripts provided with the apps, for example).


merge package from arlac77/npm-package-template

package.json

chore(package<EMAIL_ADDRESS>chore(scripts): cover@#overwrite c8 --temp-directory build/coverage ava && c8 report -r lcov --temp-directory build/coverage
chore(package): add nyc from template
chore(package): set $.ava.require='esm' as in template
chore(package): set $.ava.files='tests/-test.js,tests/-test.mjs' as in template
chore(package): set $.ava.extensions='js,mjs' as in template

Coverage remained the same at 47.75% when pulling c7425e28cc6216681e319f0089267a18a477abab on template-sync-1 into c0ecd12b34fab9cb96ce6c56b1c74a6b4a301aa5 on master.

:tada: This PR is included in version 8.1.24 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Not able to download udacimak

Not able to download udacimak

Hope this Helps

Me too

I think the problem is in youtube-dl itself. After doing some researches, I have found that one of the following solutions would work.

change youtube-dl to youtube-dl-exec in the dependency
use<EMAIL_ADDRESS>but I failed to execute them, if anyone can help

I think the problem is in youtube-dl itself. After doing some researches, I have found that one of the following solutions would work.

change youtube-dl to youtube-dl-exec in the dependency
use<EMAIL_ADDRESS>but I failed to execute them, if anyone can help

have you found the solution? If yes then please do share.

I got it working somehow.

Instal udacimak version 1.5.0 via npm install -g<EMAIL_ADDRESS>Locate the place udacimak is installed via npm root -g
ex: C:\Users\<user>\AppData\Roaming\npm\node_modules\udacimak
Replace the lib folder with this folder lib.zip
Replace the package.json with this package.zip
Reinstall all the packages via npm install (make sure to be in the udacimak folder)

You might have to rename the rendered file to something without spaces if you get an error about download path

I got it working somehow.

Install udacimak version 1.5.0 via npm install -g<EMAIL_ADDRESS>Locate the place udacimak is installed via npm root -g
ex: C:\Users\<user>\AppData\Roaming\npm\node_modules\udacimak
Replace the lib folder with this folder lib.zip
Replace the package.json with this package.zip
Reinstall all the packages via npm install (make sure to be in the udacimak folder)

You might have to rename the rendered folder (ex: Intro to Machine Learning with Pytorch v3.0.0 to ml) to something without spaces if you get an error about download path

Thank you so much for responding but youtube-dl is again causing an error. Now I am able to download Udacimak but when I run npm install in udacimak folder then I am getting an error (

).
Please help me out.

I got it working somehow.

Install udacimak version 1.5.0 via npm install -g<EMAIL_ADDRESS>Locate the place udacimak is installed via npm root -g
ex: C:\Users\<user>\AppData\Roaming\npm\node_modules\udacimak
Replace the lib folder with this folder lib.zip
Replace the package.json with this package.zip
Reinstall all the packages via npm install (make sure to be in the udacimak folder)

You might have to rename the rendered folder (ex: Intro to Machine Learning with Pytorch v3.0.0 to ml) to something without spaces if you get an error about download path

Thanks for the info. But since the installation fails, there is no udacimak folder in node_modules. Even if I were to create it generates these errors

Interesting, I'm not why npm install is installing<EMAIL_ADDRESS>Try installing<EMAIL_ADDRESS>from the udacimak folder via 'npm install<EMAIL_ADDRESS>You might also have to install 'valid-url' via 'npm install<EMAIL_ADDRESS>

Interesting, I'm not why npm install is installing<EMAIL_ADDRESS>Try installing<EMAIL_ADDRESS>from the udacimak folder via npm install<EMAIL_ADDRESS>You might also have to install valid-url via npm install<EMAIL_ADDRESS>
youtube-dl is no more available in npm as it got deprecated. It is replaced by youtube-dl-exec. Are you able to download? I mean it is working in your machine?

youtube-dl is no more available in npm as it got deprecated. It is replaced by youtube-dl-exec. Are you able to download? I mean it is working in your machine?

Deprecated means it will longer get supported (updates), but that doesn't mean that the code doesn't work. The fact the<EMAIL_ADDRESS>installed means that youtube-dl installed. Try installing different versions of youtube-dl like<EMAIL_ADDRESS>you can find a list of all the versions here user the versions tab. I made this colab example to show that youtube-dl installs (keep in mind I didn't update the lib folder in the colab example).

Okay, I had the same problem and found a workaround.
Use Docker. In docker downloading works without these errors.

npm install<EMAIL_ADDRESS>
I was having the same issues guys. But this guy is a life saver. I followed his 5 steps but it still showed the same error. In his next comment he asked to install different youtube versions in same directory and it worked for me. I tried this statement npm install<EMAIL_ADDRESS>

@kendreaditya thanks.

youtube-dl is no more available in npm as it got deprecated. It is replaced by youtube-dl-exec. Are you able to download? I mean it is working in your machine?

Deprecated means it will no longer get supported (updates), but that doesn't mean that the code doesn't work. The fact the<EMAIL_ADDRESS>installed means that youtube-dl installed. Try installing different versions of youtube-dl like<EMAIL_ADDRESS>you can find a list of all the versions here under the versions tab. I made this colab example to show that youtube-dl installs and udacimak runs (keep in mind I didn't update the lib folder in the colab example).

Thank you so much but now I am getting a youtube download failed error. I am sharing the error message and my package.json. Please help me out.

Thank you so much but now I am getting a youtube download failed error. I am sharing the error message and my package.json. Please help me out.

I sometimes face the same error, I'm pretty its because of youtube-dl. It looks like v3.4.0 is the most stable in terms of downloading the videos. However, if you run the same command again, it might work (that's what I did), other wise try installing different versions of youtube-dl.

Failed to validate your Udacity token. Please try again. Here's the error message:

I faced the same issue, its because of the API query is not working, this post explains that further. My lib.zip file also has the same thing, just have to find the API file. After I fixed that, I also got an error when downloading, which is also fixed in the post.

Of cpourse, the same post worked for me. Now, I have a new trouble, when I am rendering, The Udacimak app doesn't download the videos. Any Idea? Thanks

Of course, the same post worked for me. Now, I have a new trouble, when I am rendering, The Udacimak app doesn't download the videos. Any Idea? Thanks

I also facing the same problem. If you find any solution please share.

I found a possible solution, but doesn't work for me:
https://github.com/udacimak/udacimak/issues/137#issuecomment-623157930
https://github.com/udacimak/udacimak/issues/137#issuecomment-623157930

Other alternatives to download are torrent and this
https://coursedrive.org/?s=Data+architect+udacity

I've gone through downloading it after a day play around with that. Thank you @kendreaditya by referring your clear sequences, I would like to consolidate as following for Win10:

install latest nodejs, npm, git & Microsoft Visual C++ 2010 Redistributable Package (x86)
open CMD as administration
npm install<EMAIL_ADDRESS>then npm list youtube-dl
3. npm install -g<EMAIL_ADDRESS>then npm list udacimak
cd /Program Files/nodejs/node_modules/udacimak
npm install<EMAIL_ADDRESS>then npm list youtube-dl
 npm install -g<EMAIL_ADDRESS>then npm list udacimak
Replace the lib folder with this folder lib.zip
Replace the package.json with this package.zip
udacimak settoken ...
udacimak listnd
udacimak download nd880
udacimak render "AI for Trading v3.0.0"
Hope it help you guys, cheers!!

@BinhKieu82 not working, it doesn't even pass the youtube-dl part

@BinhKieu82 not working, it doesn't even pass the youtube-dl part

Hi mschief148,
Lucky me that i could install youtube-dl version 3.4.0. I got failure many times with 3.5.5 & very frustrated. I don't know how to explain for more, my thought is that just try with the lowest version of youtube-dl & udacimak, once you come to the step 12, it will automate upgrading the latest version. Sorry that's all i can help.

@BinhKieu82 not working, it doesn't even pass the youtube-dl part

Hi mschief148,
Lucky me that i could install youtube-dl version 3.4.0. I got failure many times with 3.5.5 & very frustrated. I don't know how to explain for more, my thought is that just try with the lowest version of youtube-dl & udacimak, once you come to the step 12, it will automate upgrading the latest version. Sorry that's all i can help.

Lowering version of youtube-dl to 3.4.1 solve my problem. Thank you for your time

Check out #206
Hope my solution can help you

I've gone through downloading it after a day play around with that. Thank you @kendreaditya by referring your clear sequences, I would like to consolidate as following for Win10:

install latest nodejs, npm, git & Microsoft Visual C++ 2010 Redistributable Package (x86)
open CMD as administration
npm install<EMAIL_ADDRESS>then npm list youtube-dl
 npm install -g<EMAIL_ADDRESS>then npm list udacimak
cd /Program Files/nodejs/node_modules/udacimak
npm install<EMAIL_ADDRESS>then npm list youtube-dl
 npm install -g<EMAIL_ADDRESS>then npm list udacimak
Replace the lib folder with this folder lib.zip
Replace the package.json with this package.zip
udacimak settoken ...
udacimak listnd
udacimak download nd880
udacimak render "AI for Trading v3.0.0"

Hope it help you guys, cheers!!

I followed your steps, but it only renders a root index.html within 1 second. Other subfolder index files did not create. Any help?


Task for ensuring dependency licenses

... ensuring that dependency licenses aren't restricted in such a manner that we cannot use them.

Per offline discussion, no standard way to achieve this given our wide range of dependency types/sources.


numpy.std implementation - not sa fast as I would like :-/

Still a WIP, there seems to be a performance issue on the axis version of numpy.mean / numpy.var and numpy.std as a consequence...

pbrunet : it's ok now


Add kaleido pkg to environment.yml

ref: https://2i2c.slack.com/archives/C028WU9PFBN/p1721061503987039

I'm getting dial tcp <IP_ADDRESS>:443: connect: connection refused when binder is pushing the image

I verified that this builds and works on the binderhub-ui-demo hub


kabanero pipeline to build spring-boot2 stack failing with Non-resolvable parent POM for dev.appsody:application:0.0.1-SNAPSHOT

When trying to build the java-spring-boot2 stack using a Tekton pipeline I am getting the following failure about non resolvable parent POM.  Is there a version mismatch somewhere?
Steps to reproduce:

Did an appsody init java-spring-boot2
Was able to do an appsody build locally successfully
Pushed it to a git repo (https://github.com/kvijai82/appsody-java-spring-boot2)
Ran pipeline in an OKD cluster to build the repo above and failed with the error.

``
INFO[0030] args: [-c /project/java-spring-boot2-build.sh package]
Installing parent dev.appsody:spring-boot2-stack:0.3.4

mvn install -q -f /project/appsody-boot2-pom.xml
Updating parent version
mvn -q versions:update-parent -DparentVersion=[0,1)
[ERROR] [ERROR] Some problems were encountered while processing the POMs:
[FATAL] Non-resolvable parent POM for dev.appsody:application:0.0.1-SNAPSHOT: Could not find artifact dev.appsody:spring-boot2-stack:pom:0.3.3 in central (https://repo.maven.apache.org/maven2) and 'parent.relativePath' points at no local POM @ line 6, column 11
@
[ERROR] The build could not read 1 project -> [Help 1]
[ERROR]
[ERROR]   The project dev.appsody:application:0.0.1-SNAPSHOT (/project/user-app/pom.xml) has 1 error
[ERROR]     Non-resolvable parent POM for dev.appsody:application:0.0.1-SNAPSHOT: Could not find artifact dev.appsody:spring-boot2-stack:pom:0.3.3 in central (https://repo.maven.apache.org/maven2) and 'parent.relativePath' points at no local POM @ line 6, column 11 -> [Help 2]
[ERROR]
[ERROR] To see the full stack trace of the errors, re-run Maven with the -e switch.
[ERROR] Re-run Maven using the -X switch to enable full debug logging.
[ERROR]
[ERROR] For more information about the errors and possible solutions, please read the following articles:
[ERROR] [Help 1] http://cwiki.apache.org/confluence/display/MAVEN/ProjectBuildingException
[ERROR] [Help 2] http://cwiki.apache.org/confluence/display/MAVEN/UnresolvableModelException
Package project in the foreground
mvn clean package verify
[INFO] Scanning for projects...
[ERROR] [ERROR] Some problems were encountered while processing the POMs:
[FATAL] Non-resolvable parent POM for dev.appsody:application:0.0.1-SNAPSHOT: Failure to find dev.appsody:spring-boot2-stack:pom:0.3.3 in https://repo.maven.apache.org/maven2 was cached in the local repository, resolution will not be reattempted until the update interval of central has elapsed or updates are forced and 'parent.relativePath' points at no local POM @ line 6, column 11
@
[ERROR] The build could not read 1 project -> [Help 1]
[ERROR]
[ERROR]   The project dev.appsody:application:0.0.1-SNAPSHOT (/project/user-app/pom.xml) has 1 error
[ERROR]     Non-resolvable parent POM for dev.appsody:application:0.0.1-SNAPSHOT: Failure to find dev.appsody:spring-boot2-stack:pom:0.3.3 in https://repo.maven.apache.org/maven2 was cached in the local repository, resolution will not be reattempted until the update interval of central has elapsed or updates are forced and 'parent.relativePath' points at no local POM @ line 6, column 11 -> [Help 2]
[ERROR]
[ERROR] To see the full stack trace of the errors, re-run Maven with the -e switch.
[ERROR] Re-run Maven using the -X switch to enable full debug logging.
[ERROR]
[ERROR] For more information about the errors and possible solutions, please read the following articles:
[ERROR] [Help 1] http://cwiki.apache.org/confluence/display/MAVEN/ProjectBuildingException
[ERROR] [Help 2] http://cwiki.apache.org/confluence/display/MAVEN/UnresolvableModelException
error building image: error building stage: waiting for process to exit: exit status 1
[root@kalathur-okd-dal-master common]#
``

Non-resolvable parent POM for dev.appsody:application:0.0.1-SNAPSHOT: Could not find artifact dev.appsody:spring-boot2-stack:pom:0.3.3 in central
There is a mismatch/problem: the pom.xml in the initialized project is asking for a non-present version of the parent pom.

Updated the version in the pom.xml from 0.3.3 to 0.3.4 to avoid the version mismatch and that resolved the problem.  Thanks @ebullient!


[WIP] Containerize the full RP int-like procedure

[WIP] The PR automates the full RP dev int-like procedure using dockerfile and Makefile.
Which issue this PR addresses:

Fixes ARO-9327 & ARO-9328
What this PR does / why we need it:

How to use it to create ARO RP?

run make rp-full-dev
login to Azure from the container (limitation that we can workaround using our user with az login)
run source hack/rp-dev/setupRPConfig.sh $AZURE_PREFIX # setup config file
run make pre-deploy-aks # deploy predeployment resources prior to AKS
run ./hack/rp-dev/preRPDeploy.sh
run make deploy # fully deploy all the resources for ARO RP

Which changes does it include?

Set Azure prefix (AZURE_PREFIX) and USER as optional - partially of https://github.com/Azure/ARO-RP/pull/3681
Add two bash scripts under hack/rp-dev/ for automating https://github.com/Azure/ARO-RP/blob/master/docs/deploy-full-rp-service-in-dev.md#deploying-an-int-like-development-rp and a helper script hack/devtools/rp-dev-helper.sh.
Separate deploy target into pre-deploy-full and pre-deploy-aks due to rpServiceKeyvaultDynamic which depends on aro-aks-cluster-001 that hasn't been set up ATM.
Workaround for azsec-monitor problem
Add dockerfile Dockerfile.rp-full-dev to build an image with the required packages to automate the full RP dev int-like.
Add a target to initiate the container build and enter it, and another target to cleanup 4 resource groups and 4 keyvaults which are needed per USER/AZURE_PREFIX

Test plan for issue:

Is there any documentation that needs to be updated for this PR?

How do you know this will function as expected in production?

Close this PR in favor on a new one #3764


Fundamental Windows incompatibility when accessing the file system

What happened:
Syft is at least not working properly on Windows analyzing a project in a directory (syft scan dir:...).
I used variations of SYFT_GOLANG... env variables to enable module remote access.
License information is not present in the Syft JSON SBOM output.
I saw the same symptoms like described in #1933.
I debugged syft . The problems go deeper and do not just affect licenses.go, so I create a dedicated bug for it.
In general, file (path) operations of syft do not support Windows properly.
Path operations for the local file system are done with functions coming from "path" module instead of "path/filepath".
The Go documention states clear that "path" is only for Unix/Linux, but not for Windows.
Instead, "path/filepath" has to be used, because this module support both Unix-like and Windows OS.
I also recognized that syft is hanging (at the end?) of processing on Windows, if -v is NOT set.
Another funny thing I saw is that you try to access paths that are concatenated taking the local path and typical Unix/Linux locations
[0000] TRACE searching filetree by path path=D:\Projekte\Go\Hello_world\etc\os-release
[0000] TRACE searching filetree by path path=D:\Projekte\Go\Hello_world\usr\lib\os-release
[0000] TRACE searching filetree by path path=D:\Projekte\Go\Hello_world\etc\system-release-cpe
[0000] TRACE searching filetree by path path=D:\Projekte\Go\Hello_world\etc\redhat-release
[0000] TRACE searching filetree by path path=D:\Projekte\Go\Hello_world\bin\busybox
I did not investigate what would need to be considered if the scanned object is not a directory tree.
What you expected to happen:
The readme file should have a clear note that there are issues on Windows until these are resolved.
Use "path/filepath" instead of "path" functions for file system accesses to make it compatible with Windows and Unixes.
Syft should terminate properly at the end, even if -v is not used.
Finally #1933 is resolved also.
Steps to reproduce the issue:
Execute syft on a Go project with a colorized "Hello world" (one external dependency) on WIndows.
Anything else we need to know?:

Environment:

Output of syft version:
Application: syft
Version:    0.104.0
BuildDate:  2024-02-07T20:51:36Z
GitCommit:  bd0cb916df4376a06e56ef7f3cc0da6659d7c2c6
GitDescription: v0.104.0
Platform:   windows/amd64
GoVersion:  go1.21.6
Compiler:   gc
OS (e.g: cat /etc/os-release or similar):
Windows 11

Thanks for reporting, we'll look into this.

Use "path/filepath" instead of "path" functions for file system accesses to make it compatible with Windows and Unixes.

Looking at using path vs filepath packages might be a bit of a red herring since our file.Resolver layer normalizes windows paths to unix paths so that much of the business logic early on does not have to worry about these concerns (thus we can use path).

Hi guys,
I noticed the issues at the places and command in first instance where you are NOT using you file resolver operations, but "path" functions instead of "filepath" directly! So this is absolutely not a red herring.
What I did to resolve go cataloger issues (tested on windows only):
config.go: changed "path" to "filepath" operations (goPath, g.LocalModCacheDir)
licenses.go: a glob definition must also match to the current OS. Also here Base and Join with "filepath" instead of "path"
package.go: also here you have hardcoded "/" and path.Join where file resolver is not used.
I saw that you have implemented the file resolver unindexed_directory.go, but you mix "path" and afero, and this is not platform independent and not working on Windows (so you don't normalize it properly). I did not work with afero yet, but since Windows FS is supported (I took this from their Readme), you must consider platform-independant path operations either solely with pure afero or a working mix with other platform-independent libraries like "filepath".
I hope you understood now why I wrote "fundamental" :)
BR

Hi @Joerki,
Thanks for the detailed issue report. The specific issue you reported, where Syft was incorrectly computing the path on the host to golang proxy cache, was fixed by https://github.com/anchore/syft/pull/2654.
I'm leaving this issue open because I'd like to improve the CI testing of Syft on Windows in general, and am using this issue to track that work. If you encounter any related bugs, please let us know.

Thank you very much @willmurphyscode for taking care on the issues.
I will check our build step soon and replace our temporary workaround, stay tuned for updates and keep an eye on the app :)


Boundary Condition E=(1,0,0) for Nedelec elements for tetrahedral meshes

Hi everyone,
I am working on solving the equation curl curl E + i a E=J using Nedelec finite element spaces on a tetrahedral mesh. My computational domain is a cube. I need to apply the Dirichlet BC E=(1,0,0) on one face of the cube (physically this is a current sheet flowing in the X direction). It seems that neither the examples nor the miniapps provide a direct solution for this scenario. Could anyone provide guidance for it? Thank you
M

I think ProjectBdrCoefficientTangent should do the trick, see the Gridfunction documentation.


tx event query syntax

I apologize for asking a beginner question here. I'm not sure of the best place. I'm having a hard time making tx event queries.
Based on response I receive from the server, the format should be {eventType}.{eventAttribute}={value} but when I use that format, I get an error: "grpc_message":"\nparse error near equal (line 1 symbol 20 - line 1 symbol 21):\n"="\n. This seems to indicate failure on parsing the =. I'm not sure the correct format since the format recommended by the service is not being accepted.
from cosmpy.protos.cosmos.tx.v1beta1.service_pb2_grpc import ServiceStub
from cosmpy.protos.cosmos.tx.v1beta1.service_pb2 import GetTxsEventRequest

tx = ServiceStub(channel)
event_request = GetTxsEventRequest(
    events=["delegate.validator=osmovaloper1md9f5524vtmrn64lyv2pdfn7cnkjkklf44vtjz"]
)
tx.GetTxsEvent(event_request)

Error message:
---------------------------------------------------------------------------
_InactiveRpcError                         Traceback (most recent call last)
Input In [69], in <module>
----> 1 tx.GetTxsEvent(event_request)

File ~/.local/lib/python3.10/site-packages/grpc/_channel.py:946, in _UnaryUnaryMultiCallable.__call__(self, request, timeout, metadata, credentials, wait_for_ready, compression)
    937 def __call__(self,
    938              request,
    939              timeout=None,
   (...)
    942              wait_for_ready=None,
    943              compression=None):
    944     state, call, = self._blocking(request, timeout, metadata, credentials,
    945                                   wait_for_ready, compression)
--> 946     return _end_unary_response_blocking(state, call, False, None)

File ~/.local/lib/python3.10/site-packages/grpc/_channel.py:849, in _end_unary_response_blocking(state, call, with_call, deadline)
    847         return state.response
    848 else:
--> 849     raise _InactiveRpcError(state)

_InactiveRpcError: <_InactiveRpcError of RPC that terminated with:
        status = StatusCode.UNKNOWN
        details = "
parse error near equal (line 1 symbol 20 - line 1 symbol 21):
"="
"
        debug_error_string = "{"created":"@1647842437.170417548","description":"Error received from peer ipv4:65.21.x.x:9090","file":"src/core/lib/surface/call.cc","file_line":903,"grpc_message":"\nparse error near equal (line 1 symbol 20 - line 1 symbol 21):\n"="\n","grpc_status":2}"
>

I wasn't quoteing ' the search value.


[chainerx] add basic loss functions

Add basic loss functions
Not sure how to go about writing test. Should we use prebuilt chainer functions to compare or write numpy versions of the same. I believe, first would be better option as they are well-tested already.

Thank you for the PR! I think that the first seems better too. It would be good if you use op_utils.ChainerOpTest.

Sure will try it that way.

Having the reduction argument in Chainer is mostly due to a historical reason. I briefly discussed it with the team but we thought it might be better simplifying the interfaces by simple not taking any reduction option as arguments (as long as those functions can still be defined). Could you change it that way (it should be a lot simpler)?

Having the reduction argument in Chainer is mostly due to a historical reason. I briefly discussed it with the team but we thought it might be better simplifying the interfaces by simple not taking any reduction option as arguments (as long as those functions can still be defined). Could you change it that way (it should be a lot simpler)?

What would be the default behaviour in that case? The case with default value of reduction?

Sorry my point was that these functions don't perform any reduction at all.

Sorry my point was that these functions don't perform any reduction at all.

Sorry I am a bit confused, gaussian_kl_divergence does perform reduction right?

gaussian_kl_divergence is element-wise up to this point https://github.com/chainer/chainer/blob/master/chainer/functions/loss/vae.py#L58-L61 (referencing Chainer's code but you see the algorithm). My suggestion is to skip this part entirely in the ChainerX version of it, just returning the element-wise loss. It'll be up to the caller to take the sum or the mean.

Jenkins, test this please.

@hvy do note that huber_loss is quite flaky.

Jenkins CI test (for commit fb47e85e1708f36349daa91612e6f738ae77352d, target branch master) failed with status FAILURE.

Not exactly certain but it might be related to https://github.com/chainer/chainer/issues/7241. Will try to dig into it. This week.

It was probably due to Absolute being non-differentiable at 0.
We get zeros in the input as we generate input from normal distribution centred at 0.
Realised that when MeanAbsoluteError also failed. Have added dodge_nondifferentiable = True for HuberLoss as well as MeanAbsoluteError.

I'm very sorry for a delayed input but we for now have a consensus to keep the APIs simple in terms of how these types of routines reduce. More specifically, we're not performing any reductions at all (users can perform arbitrary reductions afterwards). This sort of goes agains Mean{Squared,Absolute}Error. I discussed this with @beam2d and thought that we actually don't need to provide these two routines. MeanAbsoluteError could be replaced with AbsoluteError.
Could we hear your opinion, and would it be okay simply to omit these two reducing routines from this PR and introduce a AbsoluteError? (The other two routines should of course be there)
It might however be important to let the users find SquaredDifference from both the "loss" and the math section in the documentations. It might be beyond the scope of this PR though.

Umm honestly, I feel 50-50 on this decision. I agree the API should be minimal and sufficient. Also reducing the output from {Squared, Absolute} would be trivial and will allow user to do whatever he wants.
But also the fact that MSE is like a staple loss (especially for regression and compression)  in the machine learning community. It is always present in almost any framework you take. So I take it that everyone assumes it to be present.
Case can also be made if user really wants some other reduction output, it is easy to get since these functions are relatively trivial to implement.
So are you sure we should remove them the reduction from them?

Expanding the API in the future is always possible so I believe this is the right approach for the time being. I absolutely understand your point in that the users might expect MSE to be there so we might reconsider this again. It's reasonable by the way to e.g. in the docs of squared_error mention how MSE should be implemented. That'd make it searchable.

Jenkins, test this plese.

Jenkins, test this please.

Jenkins CI test (for commit 04b4b0701c97456f4bab8996d8333938c3915c74, target branch master) succeeded!

Thanks for the changes. It basically LGTM so let me merge after merging becomes possible (after release). There are some minor errors in the docs but I could fix them later (unless you want to of course). I think it's okay to delete squared_difference by the way but that should probably be addressed in a separate PR.

Oh. Sure.
I would be more than happy if you could fix the docs.
Shouldn't both exist? For explicitly expressing the behaviour.
We can have a single function in C++ while in Python use it to alias the other?
Thank You.

Got it. I see your point but at the same time think it'd then be better to reconsider the name 🤔

Jenkins, test this please.

Jenkins CI test (for commit 04b4b0701c97456f4bab8996d8333938c3915c74, target branch master) failed with status FAILURE.

Jenkins, test this please.

Jenkins CI test (for commit 04b4b0701c97456f4bab8996d8333938c3915c74, target branch master) failed with status FAILURE.

Failures seem unrelated. LGTM!


Be profile management

@ShayAvraham @tomerguttman @odedZaken
Hi guys, can someone please check the profile page after logging in?
it should update all the user details and upload a new image if changed.
You should register and log in before checking.
@ShayAvraham and I added the wtforms library so it should be checked too.
Thanks!

looks good !


Allow building with Gradle

Only tested with linux host build for now.

Hi @FDelporte
Not sure if you want Gradle support, but I had to make it as a stepping stone for testing something anyway.
I am going to test Windows native builds, but prefer to use gradle.
Anyway, I see the build is failing. Interesting - did not touch any of the existing files (I should think).
Cheers,
Jesper

Hi @FDelporte
Not sure if you want Gradle support, but I had to make it as a stepping stone for testing something anyway.
I am going to test Windows native builds, but prefer to use gradle.
Anyway, I see the build is failing. Interesting - did not touch any of the existing files (I should think).
Cheers,
Jesper

Thanks a lot Jesper! I'm personnaly a Maven-user, but indeed adding Gradle support as example for other users is great!
No clue yet why builds seem to be broken as indeed they should not be influenced by your commit. Will merge and check later what is happening.

Thanks a lot Jesper! I'm personnaly a Maven-user, but indeed adding Gradle support as example for other users is great!
No clue yet why builds seem to be broken as indeed they should not be influenced by your commit. Will merge and check later what is happening.

FYI @jskovjyskebankdk builds are fixed!

FYI @jskovjyskebankdk builds are fixed!

nice

nice


Don't print local ip address on 404 page, fixes #944

See #944 for discussion.
There's no reason to print it there, and it unnecessarily leaks information if the web server is sitting behind a proxy (CloudFlare for example) or a load balancer.
Unrelated question: why is it sending xml stuff here, if the page is html?

https://developer.mozilla.org/en-US/docs/Web/HTML/Element/address
It's HTML tag, though ZNC uses it incorrectly.
Also I think this should go to 1.6 branch, not to master.
2015-07-07 17:03 GMT+01:00 Pavel Djundik<EMAIL_ADDRESS>
See #944 https://github.com/znc/znc/issues/944 for discussion.
There's no reason to print it there, and it unnecessarily leaks
information if the web server is sitting behind a proxy (CloudFlare for
example) or a load balancer.
Unrelated question: why is it sending xml stuff here, if the page is
html?
You can view, comment on, or merge this pull request online at:
https://github.com/znc/znc/pull/986
Commit Summary

Don't print local ip address on 404 page, fixes #944

File Changes

M src/HTTPSock.cpp
https://github.com/znc/znc/pull/986/files#diff-0 (5)

Patch Links:

https://github.com/znc/znc/pull/986.patch
https://github.com/znc/znc/pull/986.diff

—
Reply to this email directly or view it on GitHub
https://github.com/znc/znc/pull/986.

I didn't mean address tag, I meant <?xml tag at the top, and xmlns on html. That stuff isn't needed.

Ah, I wanted to switch to XHTML.
But I had to rollback the last part of it (content-type), because of MSIE.
So now it's XHTML... except that it's not.
2015-07-07 18:01 GMT+01:00 Pavel Djundik<EMAIL_ADDRESS>
I didn't mean address tag, I meant <?xml tag at the top, and xmlns on html.
That stuff isn't needed.
—
Reply to this email directly or view it on GitHub
https://github.com/znc/znc/pull/986#issuecomment-119268557.

Is there any particular reason why you wanted it to be XHTML?

Just to make it more strict.
If XHTML parses correctly, there is a somewhat bigger chance that it
renders the same across browsers.
And in case of some error HTML is still successfully parsed, but
differently.
2015-07-07 18:07 GMT+01:00 Pavel Djundik<EMAIL_ADDRESS>
Is there any particular reason why you wanted it to be XHTML?
—
Reply to this email directly or view it on GitHub
https://github.com/znc/znc/pull/986#issuecomment-119270068.

LGTM, but please retarget it to 1.6 branch

I would need to open a new pull request for that.

Ok, commited as 3bc7b820ace3bf00b8532e23e1a88b763f758955


chore: Testing the CLA Assistant

Making sure we see the trigger working.

This Pull Request was only to test and ensure that the CLA Assistant is working, will not merge it in.


fly: 6.7.0 -> 6.7.2

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://repology.org/metapackage/fly/versions.
meta.description for fly is: "A command line interface to Concourse CI"
meta.homepage for fly is: "https://concourse-ci.org"
meta.changelog for fly is: ""
Updates performed

Golang update

To inspect upstream changes

Release on GitHub

Compare changes on GitHub

Impact

Checks done (click to expand)

built on NixOS

The tests defined in passthru.tests, if any, passed

0 of 0 passed binary check by having a zero exit code.

0 of 0 passed binary check by having the new version present in output.

found 6.7.2 with grep in /nix/store/ffncr0xxvdhbkr8ay8biimvssi12wr3d-fly-6.7.2

directory tree listing: https://gist.github.com/c885b4ce6cbf43290b87af5ff1f53c0e

du listing: https://gist.github.com/0c9f230bbb514a0fd5e7c52fe345fd7d

Rebuild report (if merged into master) (click to expand)

4 total rebuild path(s)

1 package rebuild(s)

1 x86_64-linux rebuild(s)
1 i686-linux rebuild(s)
1 x86_64-darwin rebuild(s)
1 aarch64-linux rebuild(s)

First fifty rebuilds by attrpath
fly

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/ffncr0xxvdhbkr8ay8biimvssi12wr3d-fly-6.7.2 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A fly https://github.com/r-ryantm/nixpkgs/archive/1590c5302baae3f9deece67b859f877e329c89a9.tar.gz

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/ffncr0xxvdhbkr8ay8biimvssi12wr3d-fly-6.7.2
ls -la /nix/store/ffncr0xxvdhbkr8ay8biimvssi12wr3d-fly-6.7.2/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review 1

1 package built:

fly

Maintainer pings
cc @ivanbrennan @kalbasit for testing.

Result of nixpkgs-review pr 105607 run on x86_64-darwin 1

1 package built:

fly


Unit testing docs

Contributions

[X] I have read the contribution docs, and understand what's expected of me.

Link to the page on docs.getdbt.com requiring updates
https://docs.getdbt.com/docs/build/unit-tests
What part(s) of the page would you like to see updated?
We got some feedback that suggested the following updates to our documentation:

Clarify fixture requirements for all dependencies (to avoid null values in output)
Emphasize satisfying all model logic conditions (i.e. join / where clauses to avoid nulls)
Under-the-hood code example of how unit tests compile

Additional information
No response

Thanks Doug! Let us know when you want a review on that PR.

I'm not sure if this might at all be relevant, but I found (in my opinion) a setup for using unit tests for macros that seem to work pretty well, and might be especially relevant for package maintainers.
So just putting it out there in case you'd like to modify this point slightly: We currently only support unit testing SQL **models**.
https://medium.com/p/89bdb5de8634


Hide monthly pass sections for free Logan trips

Summary of changes
Asana Ticket: Trip Planner | SL Trips from Airport to South Station - Monthly Pass Fare
Hide monthly pass sections for free Logan trips
This is specifically just for Silver Line only (no transfers) trips
starting at one of the Logan terminals.

Before getting review, please check the following:

[x] Does frontend functionality render and work correctly in IE?
[x] Have we load-tested any new pages or internal API endpoints that will receive significant traffic?
[x] Are interactive elements accessible to screen readers?
[x] Have you checked for tech debt you can address in the area you're working in?
[x] If this change involves routes, does it work correctly with pertinent "unusual" routes such as the combined Green Line, Silver Line, Foxboro commuter rail, and single-direction bus routes like the 170?
[x] Are the changes organized into self-contained commits with descriptive and well-formatted commit messages?

Sorry about the redundant comments @arkadyan -- the page state here was being all weird on me.

Thanks, @thecristen. I made all those updates.

@ingridpierre This is a visual change, but there's nothing new added with it. Are you interested in looking it over?

@arkadyan This one is fine, with Cristen's approval!


feature: use file-like objects for net saving and loading

i think it's worthwhile to use file-like objects to save networks. i'm systemizing my use of theanets and i'm going through temporary files to save networks in a db.
basically it allows for more flexibility and it's more pythonic. heck it wouldn't be too big of a deal to have users provide their file objects instead of specifying a filename to save() and load().
i'd do a PR but i'm not sure about future use of pickling in the code.

This is a good idea! I'll try to get to it in the next couple of weeks.

goes for save_progress as well.

For save_progress if I passed a file-like object, would the same object be overwritten each time the model was saved? Using seek(0)?

@lmjohns3 good question! but this 'problem' would be left to be addressed by the user. you don't know the correspondence from obj -> file.

I think this should do it, please reopen if not.


Set up a trivial AOT Halide Generator and ITK wrapper

We probably want AOT compilation to avoid JIT overhead at runtime. JIT is an option if we need to support more dynamic types, though probably out-of-scope for this project.
Refer to Generators part 1 and Generators part 2 for details. Note especially the use of Output<Buffer<void, 2>> and output.type=uint16 in my_second_generator.
https://github.com/halide/Halide/blob/main/README_cmake.md#aot-mode
This will generate a static library containing the compiled Halide code for each parameter combination declared in CMake. We then must create a thin ITK filter which links to that shared library. It should just wrap its inputs/outputs in Halide::Runtime::Buffer and invoke the compiled Halide callable.
This is also a good time to double-check python bindings. They should "just work" but we will see.
itkCastImageFilter is probably a good candidate for a truly trivial filter to test the build system and python wrapping.

Trivial generator is already configured in #8, but no wrapping logic is done yet.


Q: DependencyInjection callee

Is there anyway to identify the constructor that required the injection of a given instant/creation of the object? and/or access metadata/attributes on the constructor parameter that is to be injected at runtime?

No there isn't. We don't provide that context.

thank for your quick reply, do you think this is possible? If so what area should I look into with regards to providing some contextual information to the DI created object(s)

Nope, as part of the abstraction its hard to provide this sort of information. I'd recommend using another container that does provide this context.

Thanks, will look to research other DI systems for this


Add command for specifying a different exit status

Sometimes I would like to handle the match returned by fzf differently according to the keybinding that was used to accept it. I haven't found a way to do this easily in the current version of fzf (though I imagine writing to a temporary file with the execute command would work, but that feels like a workaround), but thought that a possible solution would be to add a command such like accept-[exit-code], that would accept the match and terminate with the exit status [exit-code], a bit like the sed command q [exit-code]. If you think that this would be a good idea, I'd be happy to work on it and submit a pull request.

--expect is exactly what you're looking for. See man page for details.


Reintegrate Plugins and remove support

With phpDocumentor 3 we have replaced Cilex with a bare version of the Symfony Console Application and PHP-DI. This breaks how plugins currently work but plugins didn’t see a high adoption rate so we can remove support for this to clean up the codebase. In a future version we might re-add plugin support if there is enough demand.
Tasks

[ ] Convert LegacyNamespaceFilter to a View Filter and remove plugin
[ ] After validation has been reworked, remove the Plugin\Core\Descriptor
[ ] Remove the Plugin and Parameter classes
[ ] Remove plugin loading from Application class

Closing the 3.0 tickets, will reconsider them after our re-alignment


Update ActiveSupport.

[x] Have you followed the guidelines in our Contributing document?
[x] Have you checked to ensure there aren't other open Pull Requests for the same change?
[x] Have you added an explanation of what your changes do and why you'd like us to include them?
[ ] Have you written new tests for your changes? Here's an example.

[x] Have you successfully run brew tests with your changes locally?

Update ActiveSupport to include the fix in https://github.com/rails/rails/pull/34037.

Nice work @reitermarkus 👏


refactor: replace go-connections tls config with stdlib

drop docker/go-connections dependency
use crypto/tls config
ignore unused options and keep default ciphers and tls version

:green_heart: Build Succeeded

Buildkite Build
Commit: 4bafa3cb3e0f3acba7cc87327a637357ac1fd05e

:green_heart: Build Succeeded

Buildkite Build
Commit: f8f66e2ff5d65e95ca974c17e8b8df019acb90a1

History

:green_heart: Build #236 succeeded 4bafa3cb3e0f3acba7cc87327a637357ac1fd05e

@kruskall Looking at the conversation between you and @leehinman, should this PR be closed unmerged?


Buggy top bar [Library]

Steps to reproduce

Disable delegated sources and integrated hentai features
Open a manga in your Library
Click one genre chip
Clear the query and press back

Expected behavior
Search button, filter button, and update library button should be present at the top bar
Actual behavior
Search button, filter button, and update library button are not present at top bar.
Only unclickable overflow button is present.

Crash logs
No response
Tachiyomi version
r412
Android version
11
Device
Redmi Note 10
Other details
No response
Acknowledgements

[X] I have searched the existing issues and this is a new ticket, NOT a duplicate or related to another open issue.
[X] I have written a short but informative title.
[X] If this is an issue with an extension, I should be opening an issue in the extensions repository.
[X] I have tried the troubleshooting guide.
[X] I have updated the app to version 1.7.0.
[X] I have updated all installed extensions.
[X] I will fill out all of the requested information in this form.

Duplcate of https://github.com/jobobby04/TachiyomiSY/issues/368


allow disable lang on Link

I am using api routes to handle auth0. When routing to those routes, it results in a 404, due to the language being added.
This adds the option to disable the addition of the language.

@aralroca The builder › files tree › should build all the necessary files for all languages test is failing, not sure why

@BjoernRave looks that via GitHub Actions all tests passed well... I'm going to take a look at this test. But your code looks good! Thank you a lot for your contribution!!

@aralroca np, just saw, that I also committed the yarn.lock file, not sure if it introduces any problems

Don't worry, I just updated all deps before the 0.12.0 release 🙂

@all-contributors add @BjoernRave for code


Evaluation error blocking nixos-unstable

An eval error is failing hydra evals of nixos-unstable (trunk-combined), preventing channel updates:
https://hydra.nixos.org/jobset/nixos/trunk-combined#tabs-errors
The last successful eval was 2 days ago, on commit 2deb07f3ac4eeb5de1c12c4ba2911a2eb1f6ed61
The error is masked by a Hydra bug (https://github.com/NixOS/hydra/issues/822) so the only error message we get is
error: unexpected EOF reading a line

The error can be reproduced locally with nix-shell -p hydra-unstable --run 'hydra-eval-jobs -I . nixos/release-combined.nix --verbose'. I am currently bisecting on this.

Dang, hydra-eval-jobs is super slow and I gotta get some sleep. Here's where I'm at if someone else wants to pick it up:
git bisect start
# bad: [c713c5d261be7849dab40f4c635abde82eab2ffd] Merge pull request #143910 from jtojnar/gpaste
git bisect bad c713c5d261be7849dab40f4c635abde82eab2ffd
# good: [2deb07f3ac4eeb5de1c12c4ba2911a2eb1f6ed61] Merge pull request #143289 from TredwellGit/electron_13
git bisect good 2deb07f3ac4eeb5de1c12c4ba2911a2eb1f6ed61
# bad: [b96ab960d3a2ef371355f149327975799d41ed67] vtun: remove
git bisect bad b96ab960d3a2ef371355f149327975799d41ed67
# bad: [28f111989d54f0bf5b8ea145ad2b8aac6b3003d7] python38Packages.scikit-hep-testdata: 0.4.9 -> 0.4.10
git bisect bad 28f111989d54f0bf5b8ea145ad2b8aac6b3003d7
# bad: [eddbc253f4dbff3a7f80dc157126b142e11fe717] Merge pull request #143304 from sheeaza/patch-1
git bisect bad eddbc253f4dbff3a7f80dc157126b142e11fe717
# bad: [19e71dfa871977759b6a3eff7479e4a07e77038b] gromacs: 2020.4 -> 2021.3, cuda, mpi cleanups, performance tunings
git bisect bad 19e71dfa871977759b6a3eff7479e4a07e77038b

@r-burns: I suppose the messages from --verbose aren't helpful for identifying the problem?

I had hoped they would be but I didn't see anything obvious

It's morning here, so let me continue 😄

The linked hydra logs include this:
error: value is a function while a set was expected

       at /nix/store/apli94zyi7haaygpj3xdy0qaiw7zx35x-source/lib/attrsets.nix:381:46:

          380|   */
          381|   zipAttrsWith = f: sets: zipAttrsWithNames (concatMap attrNames sets) f sets;
             |                                              ^
          382|   /* Like `zipAttrsWith' with `(name: values: values)' as the function.

It's somewhere in the middle of the log, so it is easily overlooked. Is there a way to run the job with the equivalent of --show-trace to see where the error comes from?

git bisect good 398a73ac980f2e89981d8d704d7e800bb7a9bfaf
Bisecting: 10 revisions left to test after this (roughly 4 steps)

It's slow as usual.  The single step took 160 minutes, and I don't think it's a weak machine.

I don't think I can do the bisection properly.  On bffe6436 it ran relatively normally for about 80 minutes and then suddenly started to consume way too much RAM.  When I killed it, it used 16G RSS + 17G swap (compressed in zram but still).
Well, I could assume that these explosions only happen in bad cases, though it's still quite annoying.  If someone can do this with more RAM, it might work better.

i can run it on @jonringer 252G machine just give me the cmd

This one should suffice:
hydra-eval-jobs -Inixpkgs=. nixos/release-combined.nix

all the commands please, i haven't used bisect much

Ah, you run all the log of git bisect commands from previous comments.  Then it depends... automatic way would be git bisect run hydra-eval-jobs -Inixpkgs=. nixos/release-combined.nix
I often like to inspect the individual failures by hand (to make sure that it's the same error), but in this case we haven't found anything useful in the logs, so it's probably OK to leave it to that automatic git bisect run.

https://github.com/NixOS/nixpkgs/blob/master/nixos/tests/ghostunnel.nix#L1 starts with
{ pkgs, ... }: import ./make-test-python.nix {
while every other test in this directory starts with something like
import ./make-test-python.nix ({ pkgs, ... }: {
Perhaps something in #140792 caused this difference to break things?
cc @roberth who is the ghostunnel test maintainer.

I can't see that path being touched in the current bisection range.

ghostunnel does seem unrelated but here's a fix #144001.

I'm getting extreme memory usage from just the nextcloud tests. I've stopped evaluation at ~23GB.

From my incomplete tests I (also) find it likely that at least one of the big problems came from PR #140792.

Good morning everyone, I tried an unattended git bisect run overnight and it points to 329a4461a7a1898b3aad13968a232a054422c4fb at the breaking commit. Seems like there is some agreement there.

Fix: https://github.com/NixOS/nixpkgs/pull/144014

Evaluated: https://hydra.nixos.org/eval/1717845


Feat/bookinfo docs optimize

What type of PR is this?
optimize: docs and code optimize

What this PR does / why we need it (en: English/zh: Chinese):

en:

Added a QuickStart section to the README doc
Support for xDS auth
Update dependencies & charts

zh:

新增快速开始章节到 README 文档
支持 xDS 认证
更新依赖和 charts

Which issue(s) this PR fixes:

Thanks. Please optimise your PR title and description.

Thanks. Please optimise your PR title and description.

done


Slow connection with reverse proxy

Hi,
We're using this package, but we keep having issues with the reverse proxy and ssl, as described in the dox. We use apache instead of nginx, but otherwise we try to do the same thing.
It seems to be working but the initial connection to wss://ourapp/app/socket_id takes a really long time, and also sometimes failes because of that.
Anyone know what to do about this? Or does anyone have an example apache config and websockets/broadcasting config?

How much time does it take for you to establish a connection? We are using nginx as reverse proxy and using multiple instances through ALB and served through CloudFront. Ours is around 1.2 second to establish connection.

it differs a lot. Sometimes it has a connection immediately, but it can take up to 30 seconds.


Update Dependency Versions

This PR updates dependency versions.

FYI: Spring integration team will do out of band release around 2021-03-04. Please don't merge this PR before we finished release.

Closing as these version upgrades were made in another PR


Expose annotations as a collection in the Property Grid

Annotations should be editable through the property grid. Preferably as an expandable collection with a [...] button to open a collection editor (so that both the annotation name and value can be changed, and so that existing annotations can be deleted, and new annotations can be added).
Annotation edits should be undoable.
Consider whether annotations used internally by Tabular Editor should remain hidden in the UI - perhaps an option to unhide them for curious users :-)

Issue fixed in release 2.5.


[AutoPR eventgrid/data-plane] Add new EventGrid event type for Azure Machine Learning Services

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

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


Format code and update subtrees

Cool, thanks for updating the formatting!
In the future, it might be good to keep the update-subtree commits separate from the commits on our own code. Then it's more clear what changed and in case we need to do a git bisect at some point it will be easier to track down where a error comes from. But I don't think it's worth re-basing this commit.

The formatting was done with the clang tool you mentioned in the other issue, right? In some places it seems to do something a bit funny, e.g., line 644 in src/test/polytope_hash_test.cc :
https://github.com/FALCONN-LIB/FALCONN/pull/53/files#diff-78
I'm not sure if this is what we want. But if the tool does something sensible in most other cases, it's probably good to use it.

It's the only tool for C++ that uses a real parser rather than just using a bunch of ad hoc regular expressions, so we should stick to it.
If you can tweak it so that you are more happy with the results, we can use these settings by default.

OK, sounds good. I like that the tool is actually a parser, it should be more robust than cpplint.py, which I used a while ago (but I don't remember any big issues actually).


Farmers-Service WebApp

Title and Issue number

Title :Kisan-Chan-WebApp
Issue No. : #319
Code Stack : html, css, js, nodejs runtime, neo css, bootstrap, pug for rendering, youtube api, translate api,
Close  #319

Video (mandatory)
https://github.com/apu52/METAVERSE/assets/110652247/2b0fbbd2-711c-43bf-9aa7-55e51b901788
by the way idk why, but dropdown list isnt showing in video, i recorded multiple times but none where visible and video preview is also not available. Please do click on that link and video will appear.
imager of other modules:

Checklist:

[x] I have mentioned the issue number in my Pull Request.
[x] I have commented my code, particularly in hard-to-understand areas
[x] I have created a helpful and easy to understand README.md
[x] I have updated the projectData.json file for my contribution
[x] I have added the img of my project in the assets/img section

Additional context (Mandatory )
project is made in nodejs, some modules like trenting news, fertilizer calculator, and maps are and entire backend are made my me, some frontend is done my calssmates, and for maps module tyhe google api has expired, so maps js logic is removed and normal map script is added and because of expired api trnslate page is not working. Just some lack of money :(
Are you contributing under any Open-source programme?
codepeak 2023

@Sami3160 Please resolve the deployment part & resolve the merge conflicts

@Sami3160 Please resolve the deployment part & resolve the merge conflicts

All merge conflicts are resolved!