pydoclint 0.5.2 fails to find violation in a particular folder

But pydoclint 0.4.2 was able to find a lot of violations in that folder.
This issue was originally reported by @Gabriel-p in this comment: https://github.com/jsh9/pydoclint/issues/122#issuecomment-2187265736
Created a dedicated issue here to better track this.

Hi @Gabriel-p ,
After my investigation, I've found that this is actually the intended behavior.  The old behavior (which you would get with pydoclint ≤ 0.4.2) was actually the result of a bug.
In asteca/modules/, the docstrings are either written without any arg/return/yield/... lists (in which case pydoclint will ignore them (if --skip-checking-short-docstrings is set to True), or written in the numpy style, such as: https://github.com/asteca/ASteCA/blob/09204a91ed785a00ae498a20905cbfd59ab84a03/asteca/modules/nmembers.py#L231-L243
In older pydoclint versions (≤ 0.4.2), if users specify style=sphinx, pydoclint's docstring parser would parse the docstring several times, each with a potential style (google, numpy, sphinx, ...).  Whichever style that results in the most parsed contents gets "picked".  In particular, the docstring example above was parsed as a numpy style dosctring.
But in the new version (≥ 0.5.2), if you specify style=sphinx, pydoclint won't try to parse that docstring into other style.  Instead, it attempted the specified style ("sphinx"), and as a result, the arg lists written in other styles don't register at all.

I see. Is there any way to emulate the old behaviour in the new version? It was very useful for knowing which functions were missing a valid (sphinx) docstring

Maybe there is a way: I can add a new "style" called "XXX", in which I'll use an "open docstring parser" (which means the parser will try to parse the docstring with all possible styles, and will detect which style makes the most sense).
But this will obviously slow down pydoclint's speed (because more time will be spent on parsing), so I'd suggest only using this "style" in certain use cases.
I think I need help figuring out how to name this "XXX".  Can you offer some suggestions?

Well, "openparser" seems descriptive enough but I think there's a perhaps simpler solution. Whatever style the user sets, if pydoclint can not parse the docstring raise a warning and that's it. That's enough to alert the user that something is not working as expected and there would be no need to add a new parser. Right now the default behaviour is apparently to fail silently.

I'm closing this issue because there's a duplicated issue on the same topic: https://github.com/jsh9/pydoclint/issues/169
Hi @Gabriel-p , if you read the issue that I'm linking and if you run the sample script in my reply, you'll see that the docstring parser of pydoclint can always parse out something from a string, even if the style is not expected.


[Enhancement] website test

Description
As a product owner, I want to deliver high quality code and nice examples. This repository contains a website, which can and should be tested in a meaningful way. I would like to integrate a graphical website test.
Criteria

choose a proper tool
update the cirrus pipeline
the test should check for

links are working
images are loaded
tests should run on gekko(firefox) and webkit/chrome-based browsers

Additional information

selenium seems nice and easy

obsolete


hnix-0.2.0 fails its test suite

Citing from http://hydra.cryp.to/build/945583/nixlog/1/raw:
Running 1 test suites...
Test suite hnix-tests: RUNNING...
hnix
  ParserTests
    constant int:             OK
    constant bool:            OK
    constant path:            OK
    constant uri:             OK
    simple set:               OK
    set inherit:              OK
    set scoped inherit:       OK
    set rec:                  OK
    set complex keynames:     OK
    set inherit direct:       OK
    inherit selector:         OK
    int list:                 OK
    int null list:            OK
    mixed list:               OK
    simple lambda:            OK
    lambda or uri:            OK
    lambda pattern:           OK
    lambda app int:           OK
    simple let:               OK
    nested let:               OK
    let scoped inherit:       OK
    if:                       OK
    identifier special chars: OK
    simple string:            OK
    string dollar:            OK
    string escape:            OK
    string antiquote:         OK
    select:                   OK
    select path:              OK
    fun app:                  OK
    indented string:          OK
    indented string escape:   OK
    operator fun app:         OK
    operators:                OK
    comments:                 FAIL
      Exception: data/let.nix: openBinaryFile: does not exist (No such file or directory)

1 out of 35 tests failed (0.01s)
Test suite hnix-tests: FAIL

I suppose that data file is missing from the release tarball.

I've pushed 0.2.1 that includes the necessary data files.


Interface behaviors

This pull request shifts from abstract classes to interfaces for behaviors.
Advantages:

More freedom for entity implementations.
Less code duplication, as classes can share multiple behaviors.
Less memory overhead for the same reason.

Disadvantages:

Slightly more complex loading scheme to account for class inheritance.
Possibly a small performance overhead for calling interface methods rather than abstract methods.

Codecov Report

Merging #55 into development will decrease coverage by 0.52%.
The diff coverage is 80.18%.

@@               Coverage Diff               @@
##           development      #55      +/-   ##
===============================================
- Coverage        72.09%   71.57%   -0.53%
===============================================
  Files              281      282       +1
  Lines            13905    13548     -357
===============================================
- Hits             10025     9697     -328
+ Misses            3880     3851      -29

Impacted Files
Coverage Δ

...eSharp/Simulations/Base/Time/BaseAcceptBehavior.cs
100% <ø> (ø)
:arrow_up:

...imulations/Base/Frequency/BaseFrequencyBehavior.cs
80% <ø> (-7.5%)
:arrow_down:

SpiceSharp/Parameters/ParameterSetDictionary.cs
41.11% <ø> (-1.9%)
:arrow_down:

SpiceSharp/Simulations/Behaviors/BehaviorList.cs
88.88% <ø> (ø)
:arrow_up:

SpiceSharp/Simulations/Base/BaseLoadBehavior.cs
66.66% <ø> (ø)
:arrow_up:

...omponents/Distributed/VoltageDelay/LoadBehavior.cs
90.9% <ø> (-1.4%)
:arrow_down:

...ponents/Semiconductors/MOS/Level1/NoiseBehavior.cs
85.71% <ø> (-3.18%)
:arrow_down:

...eSharp/Simulations/Base/BaseTemperatureBehavior.cs
100% <ø> (ø)
:arrow_up:

.../Simulations/Behaviors/EntityBehaviorDictionary.cs
80% <ø> (-4.62%)
:arrow_down:

...lations/Implementations/Noise/BaseNoiseBehavior.cs
33.33% <ø> (ø)
:arrow_up:

... and 96 more

Continue to review full report at Codecov.

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

Yeah sorry, it may be a little bit messy because I had to try a few things...

I understand. No worries. I will review now.

@svenboulanger  I was thinking about something different but your points are also  valid.
What is a IBaseBehavior ?
There is a following code in BaseSimulation.

The description says: 'An interface that describes behaviors for a SpiceSharp.Simulations.BaseSimulation"
There is also a code:
            _loadBehaviors = EntityBehaviors.GetBehaviorList<IBaseBehavior>();
Shouldn't be IBaseBehavior a base to to all BehaviorTypes of BaseSimuation (ITemperatureBehavior and IInitialConditionBehavior) ? It's either wrong name or something different I don't understand.

I'll find a different name. The original logic was BaseBehavior for BaseSimulation, FrequencyBehavior for FrequencySimulation, etc. but if it's too generic I can always choose another name.


Use of emult in process

In GitLab by @rkemp on Jul 14, 2016, 16:52
I am finding it impossible to tell from the vardes if the old, simple blanket option still exists that allows emult to be set manually.
There are options:
blktmodel /0/ : switch for blanket/tritium breeding model (but see lblnkt):
= 0 original simple model;
= 1 KIT model based on a helium-cooled pebble-bed blanket (HCPB) reference design
blkttype /3/ : Switch for blanket type:
= 1 WCLL; efficiency taken from WP13-DAS08-T02, EFDA_D_2M97B7
= 2 HCLL; efficiency taken from WP12-DAS08-T01, EFDA_D_2LLNBX
= 3 HCPB; efficiency taken from WP12-DAS08-T01, EFDA_D_2LLNBX
iblanket /1/ : switch for blanket model:
= 1 CCFE HCPB model;
= 2 KIT HCPB model;
= 3 CCFE HCPB model with Tritium Breeding Ratio calculation
We have multiple redundancy in this and it is not clear which options are actually valid. Could this be tidied up and could someone let me know if the original model does still exist (blktmodel = 0 does not do it)? If not, why not -- it should be preserved for future flexibility and rapid exploration of new blanket options.

Please see #1594 which Clair has just implemented in #3035.


order-of-operation and precision issues when defining a modulo axis

Reported by andrew wittenberg on 7 May 2014 23:56 UTC
I'm seeing the following problem in single-precision Ferret (v6.725), which is what we're still using at GFDL to run our automated GCM diagnostics.  Haven't been able to reproduce this in the double-precision version yet, perhaps because I haven't been able to come up with the right floating-point numbers to highlight the truncation issues.  But check this out:
 	NOAA/PMEL TMAP
 	FERRET v6.725
 	Linux 2.6.18-274.12.1.el5 64-bit - 04/10/12
 	 7-May-14 19:49

yes? def ax/edges/X/modulo=360 ct_hax =<PHONE_NUMBER> + {0-180,180}
yes? def ax/edges/X/modulo=360 ct_hax =<PHONE_NUMBER> + {-180,180}
 **ERROR: improper grid or axis definition: Axis length exceeds modulo length

The removal of the "0" leads to an error!  Why?  Does it affect the order of operations somehow?
And similarly:

yes? def ax/edges/X/modulo=360 ct_hax = {`179.9211884-180`,`179.9211884+180`}
 **ERROR: improper grid or axis definition: Axis length exceeds modulo length

Migrated-From: http://dunkel.pmel.noaa.gov/trac/ferret/ticket/2173

Comment by ansley.b.manke on 8 May 2014 18:05 UTC
The expression in the first command,<PHONE_NUMBER> + {0-180,180}
is actually
yes? list<PHONE_NUMBER> + {0-180,180}
             VARIABLE :<PHONE_NUMBER> + {0-180,180}
             SUBSET   : 2 points (X)
 1   / 1:  179.9
 2   / 2:  359.9

yes? list {0-180,180}
             VARIABLE : {0-180,180}
             SUBSET   : 2 points (X)
 1   / 1:    0.0
 2   / 2:  180.0

which surprises me - an expression inside constant-array syntax is not evaluated, but no warning is given. So that's a separate issue.
For single precision Ferret, the numbers on the command line are truncated to single-point floating point. So<PHONE_NUMBER>-180 is being truncated, then when the coordinates are stored in double precision the values in the low-order bits are such that the coordinate values cause the axis to be "too long". In double-precision Ferret the numbers given in the command are kept at full precision throughout and I don't think we'd ever see this bug.
A fix in single precision would be to compare the computed axis length and the modulo length ONLY TO SINGLE-PRECISION accuracy.  Ferret doesn't do that, but just sends the error message if the length from the coordinates is greater than the modulo length (with that comparison done in double precision).
Note that the following syntax gives the desired axis - leave off the modulo length:
        NOAA/PMEL TMAP
        FERRET v6.725
        Linux 2.6.32-431.11.2.el6.x86_64 64-bit - 05/08/14
         8-May-14 11:02

yes?  def ax/edges/X/modulo ct_hax =<PHONE_NUMBER> + {-180,180}
yes?  sh axis/x ct_hax
 name       axis              # pts   start                end
 CT_HAX    X                    1mi   179.92               179.92
   Axis span (to cell edges) = 360 (modulo length = axis span)

       I     X                   XBOX      XBOXLO
       1>  179.9212              360        -0.07881165

Comment by ansley.b.manke on 8 May 2014 18:20 UTC
My last comment above isn't strictly true - the axis can be defined without error if we take off the modulo length and just say /modulo.  But the axis edges are still slightly farther apart than 360 degrees, so the axis is not quite 360 degrees.

Comment by andrew.wittenberg on 8 May 2014 20:28 UTC
Oops, sorry for my syntax error there (unevaluated expression in the braces).  And thanks for your explanation.
But I'm puzzled -- why would converting a value from single to double result in nonzero low-order bits?  I can see why going in the other direction would alter the values.  But how can a conversion to higher precision produce a change in spacing?
By the way, if we can determine for sure that this wouldn't happen in double-precision Ferret (and I haven't yet been able to reproduce it there), then I suppose we can eventually resolve this as "wontfix".

Comment by ansley.b.manke on 8 May 2014 21:56 UTC
The low-order bits are just filled in with numbers. Not zero's. In this example the negative number at the start of the axis becomes a little more negative in the conversion to double precision, and the positive upper bound becomes a little more positive. Because the axis length is computed after up-converting, the total axis length is too long.
I am convinced that this is entirely due to conversion from single to double and so in consultation with Andrew I'm marking it as "won't fix".


How can i run this as a background task?

i am running the task in the sever.
but , how can i move it to be a background task?

I believe the best practice of do a server job is to use Docker:
https://github.com/wechaty/wechaty/wiki/Docker


My problem

Description
I am unable to get Java output in the OUTPUT panel in Visual Studio Code on macOS.
Steps to Reproduce

Install OpenJDK 17 on macOS.
Install Visual Studio Code and the Java Extension Pack.
Create a simple Java file with a main method that prints "Hello, World!".
Attempt to run the Java file using the Run button in VS Code.

Expected Behavior
The output "Hello, World!" should be displayed in the OUTPUT panel.
Actual Behavior
No output is displayed in the OUTPUT panel.
Environment

macOS Version: [your macOS version]

You might need to look in the Debug Console, not Output. See https://code.visualstudio.com/docs/languages/java for more and file an issue on the Java extension repo if you think there's a bug


Sorting works only for one column / Date

vue: 1.0.7
vue-tables: 1.4.14
webpack
component: http://pastebin.pl/view/9078880b
sorting works just for "Ilość" column
Video: https://vimeo.com/165673453
pass: tables

I think you are using illegal column names. A property cannot have a space as far a I know. Use an underscore instead.

@matfish2 - it could be true. But right know my web API serves data in that way - with spaces in json params names. Is it possible to somehow workaround it without touching the backend?

Well, I'm using vue's native orderBy filter, simply passing the variable, so I'm not sure if there is a workaround I can implement.

I guess you can also iterate over the returned data and replace the keys on the client side before assigning the data to the table.

from http://stackoverflow.com/questions/33855393/javascript-object-keys-with-spaces
http://stackoverflow.com/a/33860675/4402261 <- maybe this small change in vue-tables can help?

I tried replacing the dot notation with brackets as suggested, but my tests are failing.

@matfish2 it's my first ever approach to java script, to vue.js and to anything related to html / web aps - so I propably pass on that - not my level yet ;)
I've rebuilded my database - all fields now have underscores instead of spaces and it works fine.
Thank You :)
PS. I have some question about templates - should I ask it here or start new issue?

You can ask here

`'Data': function(row) {
var date = new Date(row['Data'].$date);
return GetDateString(date);
},
works fine. But when I try:
'Data':"{Data,$date}" a just get string {Data,$date} instead of time object (epoch style long digit)

A. When a template is passed as a string, the curly brackets should contain one of the properties in the dataset. If no match is found the string returns as it is. Clearly {Data, $date} doesn't match any property.
B. Dates should be passed as moment objects, not plain JS Date objects. The toMomentFormat option allows you to pass a format matching the one in your dataset (say 'YYYY-MM-DD'), and converts the dates for you.

 Detected Vue v1.0.21

problematic line is: dateColumns: ['Data'],

A. date columns have a unique range filter using daterangepicker, which itself depends on jquery ($) and moment. If you wish to use this filter use need to import the dependencies.
Also toMomentFormat accepts a string - moment format -  NOT a boolean.
dateFormat accepts the format you want to use to present the moment object.
If you don't want to the unique filter you can omit the toMomentFormat and dateColumns options, and manually create the moment objects, iterating over the response (which is exactly what toMomentFormat does behind the scenes).


Update specs2-cats, specs2-core to 4.10.4

Updates

org.specs2:specs2-cats
org.specs2:specs2-core

from 4.6.0 to 4.10.4.
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

Ignore future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "org.specs2" } ]

labels: library-update, semver-minor

Superseded by #115.

Superseded by #115.


Commit message starts with branch name

When opening the commit dialog, the commit message isn't empty but is preset with the branch name ("master" in my case). Is that intentional? What is it good for? The branch name is already displayed just above in the dialog.

You can change the default commit message using Tools -> Options -> TGit -> General -> Default commit message.
Next version will have an empty default commit message by default.


Parse latest release from downloads url, upgrade by default

This adds the ability to convert https://downloads.openmicroscopy.org/latest/omero into a release number. This means for normal installations the value of omero_server_release is sufficient to control whether a server is upgraded or not. The omero_server_upgrade is therefore changed to True and deprecated. The only reason for not removing it at the moment is that the IDR uses a non-standard version system where the version in the URL is not the version reported by the server. This could be remedied by https://trello.com/c/NbdI5wVn/62-release-artifacts-json-scoping since the URL version and the version in  the release-artifacts.json file could be different.

Rebased. default is now present. This complicates the testing since present can only be tested by installing an older version.

Thanks, @manics. Tagging this as m4 to get it into https://github.com/ome/ansible-examples-omero/pull/3


Fixed linking error when cmake was used, because k4arecord was not mentioned in the CMakeLists.txt

Fixed linking error when cmake was used, because k4arecord was not mentioned in the CMakeLists.txt for the simple_3d_viewer.

@anantley @yijiew Would it be possible to merge this change? It fixes the linking error when building the body tracking samples.


Incorrect license score and file lists

https://clearlydefined.io/definitions/npm/npmjs/-/svgo/1.1.1 is reported with 100 license score which is rather incorrect. A single LICENSE file is reported for the entire package which is likely the source of the issue

Probably this should be opened in the service repo, since the UI is showing what the API is actually returning.
See https://api.clearlydefined.io/definitions/npm/npmjs/-/svgo/1.1.1?expand=prs

I just reharvested this one to get the latest tools.
https://api.clearlydefined.io/definitions/npm/npmjs/-/svgo/1.1.1 showing a licensed score of 46 now

@dabutvin do we need to keep this one still opened for some reason?

no probably not, we've been doing a good job getting all the old data re-harvested.


Season ID

https://github.com/nickb1080/nba/blob/movement/test/integration/stats.js#L153
You weren't sure about the season ID. If you go to data.nba.com, it lists season_id: 12015

Nice catch, thanks!

@nickb1080 awesome project! Is it free to use this library to create a web application? I mean what about the copyrights of these stats,  do they have some limitations about the nba official stats, sportVU & synergy data while using this to create an app or software?

I've not been able to find any copyright or licensing agreement that either explicitly allows or prohibits the use of these statistics. In general, my sense is that the NBA (or some developers there) are aware that their API endpoints are being used by third party libraries and applications, but they have elected to do nothing about it so far. It's also sort of hard to tell if things like sports statistics are even subject to copyright -- a quick Google search tells me they probably aren't.
But, I'm not a lawyer.


ldeep: 1.0.57 -> 1.0.58

Diff: https://github.com/franc-pentest/ldeep/compare/refs/tags/1.0.57...1.0.58
Changelog: https://github.com/franc-pentest/ldeep/releases/tag/1.0.58
Description of changes

Things done

Built on platform(s)

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

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

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

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

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

[ ] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

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

2 packages built:

ldeep
ldeep.dist


Getting hardfault in _nx_packet_release function

Hi all,
I am using stm32f205 mcu and using threadx, netx 6.2 version,
Random time I am getting hardfault from _nx_packet_release so I did depth debugging and found where exact it is generating hardFault,
So it is generating while   / Update the links of the adjacent threads.  / in _nx_packet_release function,
/* Determine if there are any threads suspended on the block pool.  /
thread_ptr =  pool_ptr -> nx_packet_pool_suspension_list;
if (thread_ptr)
{
/ Remove the suspended thread from the list.  */
        /* See if this is the only suspended thread on the list.  */
        if (thread_ptr == thread_ptr -> tx_thread_suspended_next)
        {

            /* Yes, the only suspended thread.  */

            /* Update the head pointer.  */
            pool_ptr -> nx_packet_pool_suspension_list =  NX_NULL;
        }
        else
        {

            /* At least one more thread is on the same expiration list.  */

            /* Update the list head pointer.  */
            pool_ptr -> nx_packet_pool_suspension_list =  thread_ptr -> tx_thread_suspended_next;

----------->>>> from below line code it is generating a hardFault, <<<<------------------
            **/* Update the links of the adjacent threads.  */
            (thread_ptr -> tx_thread_suspended_next) -> tx_thread_suspended_previous =
                thread_ptr -> tx_thread_suspended_previous;**

Mostly it is generating after discover DHCP packet, Could you guys tell me why this is happening or where should I look?

Closing as duplicate with https://github.com/azure-rtos/netxduo/issues/194


Create a tutorial for volunteers of the rhythm of the week!

[ ] Make a video tutorial about how to promote and invite people to their communities
[ ] Make a document with a checklist to guide people with the channels where they should invite people

Instead of a document, how about a canvas?

closing this as stale, though probably not obsolete


README: update TSC, fix linting.

Add the new TSC to the README, manpage
Fix linting complaints about whitespace in the README

Review period will end on 2021-03-16 at 15:18:32 UTC.

Just a note: brew update-maintainers will do this if the Homebrew/tsc team has been updated before it is run.

@nandahkrishna Thanks! Have updated the team now.

Review period skipped due to critical label.

The automated workflow will also alphabetise this list, I believe (as the GitHub team returns alphabetical order). It might be preferable to do that here to avoid too much churn?

The automated workflow will also alphabetise this list, I believe (as the GitHub team returns alphabetical order). It might be preferable to do that here to avoid too much churn?

@issyl0 Good call, done!


[CI] Enable tvOS/macOS builds.

Fixes #1238. TL;DR we had to turn off the tvOS builds for a while because Xcode9b6 didn't have Apple TV simulators.
I also added macOS as it should work just fine right now (tested on this branch). It was removed a year ago because it wasn't working properly for Swift 3 - #649 for anyone interested in it.
Oh, and I'm targeting master branch - will rebase 10.0 once it is merged. We should have it on both anyways.

Thanks, @petester42!


[Debugger] When debugging a transaction debugger go through lines which are not suppose to run but program works fine.

Issue:
When debugging a transaction block it will ignore conditional blocks (ex:- IF conditions) and visit inside those conditional blocks and always visit the aborted block even when there is no abort statement in transaction block or no exception occur. But the output of the program is correct.
Sample program
import ballerina.data.sql;
import ballerina.lang.system;

function main (string[] args) {
    sql:ConnectionProperties properties = {maximumPoolSize: 1};
    sql:ClientConnector db = create sql:ClientConnector (sql:MYSQL, "localhost", 3306, "test1", "root", "password", properties);
    transaction {
        sql:Parameter[] parameters = [];
        int i = db.update("INSERT INTO `students` values (1, 'test')", parameters);
        system:println("Inserted count:" + i);
        if (i == 0) {
            abort;
        }
        } failed {
        system:println("Failed..");
        retry 3;
        } aborted {
       system:println("Transaction aborted..");
    } committed {`
        system:println("Transaction committed...");
    }
}

This is tracked with git issue - https://github.com/ballerinalang/ballerina/issues/3281

this is tracked with https://github.com/ballerinalang/ballerina/issues/3591

Reopen since it was closed pointing to the same issue.

Closing since the conditional statements debugging inside transnational blocks are working as expected in the latest SwanLake release (SLAlpha4).


remove styleC fixes #48

Failed on travis with java 8 for some reason.

Thanks for the @joprice. StyleC is on its way out but

I want to go back its motivating requirements and see if we can come up with an alternative before it's removed.
It might be better to deprecate it first before deleting it. (Although it is appealing to delete it fast because it takes Shapeless off the classpath.)

Ok no problem. Close this?

Yeah, let's close it. Thank you for the PR though.

@joprice Quasi-explanatory ramblings in #48. Let's just do it.


FR: ArrayHelper::index, remove key from array

Would this be useful? A 4th option for ArrayHelper::index which specifies the key should be removed from the array.
Helps avoid some duplication of data. I can do PR and tests if it's likely to be used.
$array = [
  ['id' => '123', 'data' => 'abc', 'device' => 'laptop'],
  ['id' => '345', 'data' => 'def', 'device' => 'tablet'],
  ['id' => '345', 'data' => 'hgi', 'device' => 'smartphone'],
];

$result = ArrayHelper::index($array, 'id');
[
 '123' => ['id' => '123', 'data' => 'abc', 'device' => 'laptop'],
 '345' => ['id' => '345', 'data' => 'hgi', 'device' => 'smartphone']
]

$result = ArrayHelper::index($array, 'id', [], true);
[
  '123' => ['data' => 'abc', 'device' => 'laptop'],
  '345' => ['data' => 'hgi', 'device' => 'smartphone']
]

Yes. It can be useful but another argument can't be added without breaking backwards compatibility.

This was how I was planning to do it.
public static function index($array, $key, $groups = [], $removeKey = false)

Then it's used like,
ArrayHelper::index($array, 'key', [], true);

I'm not sure if it would be usable with groups so could make the $groups param dual purpose?

No. Making it dual purpose is a bad idea.


libtool: unknown option character `X' in: -Xlinker

Wow, thanks for sharing this swift poker hand evaluator, and for the quick response on that last issue! That got rid of the compiler errors, but I still get a linker error. I'm using 6.3.1, the full release, not the beta. Not sure what could be different about my setup than the norm, but perhaps there is something out of whack. Surprised we didn't both see this issue, but I'm sure you would have done something about this if you'd seen it.
error: /Applications/Xcode.app/Contents/Developer/Toolchains/XcodeDefault.xctoolchain/usr/bin/libtool: unknown option character `X' in: -Xlinker
Usage: /Applications/Xcode.app/Contents/Developer/Toolchains/XcodeDefault.xctoolchain/usr/bin/libtool -static [-] file [...] [-filelist listfile[,dirname]] [-arch_only arch] [-sacLT] [-no_warning_for_no_symbols]
Usage: /Applications/Xcode.app/Contents/Developer/Toolchains/XcodeDefault.xctoolchain/usr/bin/libtool -dynamic [-] file [...] [-filelist listfile[,dirname]] [-arch_only arch] [-o output] [-install_name name] [-compatibility_version #] [-current_version #] [-seg1addr 0x#] [-segs_read_only_addr 0x#] [-segs_read_write_addr 0x#] [-seg_addr_table ] [-seg_addr_table_filename <file_system_path>] [-all_load] [-noall_load]
And from up above this:
Libtool /Users/Mitchell/Library/Developer/Xcode/DerivedData/PokerHandEvaluator-gknwerskfubtczhcsjkmpwfkpgrt/Build/Products/Debug-iphonesimulator/libPokerHandEvaluator.a normal x86_64
cd /Users/Mitchell/Documents/programming/swift-poker-hand-evaluator-master
export IPHONEOS_DEPLOYMENT_TARGET=8.0
export PATH="/Applications/Xcode.app/Contents/Developer/Platforms/iPhoneSimulator.platform/Developer/usr/bin:/Applications/Xcode.app/Contents/Developer/usr/bin:/usr/bin:/bin:/usr/sbin:/sbin"
/Applications/Xcode.app/Contents/Developer/Toolchains/XcodeDefault.xctoolchain/usr/bin/libtool -static -arch_only x86_64 -syslibroot /Applications/Xcode.app/Contents/Developer/Platforms/iPhoneSimulator.platform/Developer/SDKs/iPhoneSimulator8.3.sdk -L/Users/Mitchell/Library/Developer/Xcode/DerivedData/PokerHandEvaluator-gknwerskfubtczhcsjkmpwfkpgrt/Build/Products/Debug-iphonesimulator -filelist /Users/Mitchell/Library/Developer/Xcode/DerivedData/PokerHandEvaluator-gknwerskfubtczhcsjkmpwfkpgrt/Build/Intermediates/PokerHandEvaluator.build/Debug-iphonesimulator/PokerHandEvaluator.build/Objects-normal/x86_64/PokerHandEvaluator.LinkFileList -L/Applications/Xcode.app/Contents/Developer/Toolchains/XcodeDefault.xctoolchain/usr/lib/swift/iphonesimulator -Xlinker -add_ast_path -Xlinker /Users/Mitchell/Library/Developer/Xcode/DerivedData/PokerHandEvaluator-gknwerskfubtczhcsjkmpwfkpgrt/Build/Intermediates/PokerHandEvaluator.build/Debug-iphonesimulator/PokerHandEvaluator.build/Objects-normal/x86_64/PokerHandEvaluator.swiftmodule -o /Users/Mitchell/Library/Developer/Xcode/DerivedData/PokerHandEvaluator-gknwerskfubtczhcsjkmpwfkpgrt/Build/Products/Debug-iphonesimulator/libPokerHandEvaluator.a

When are you getting this error exactly? Are you trying to link the evaluator to some other code?
I'm also using Xcode 6.3.1 and can't reproduce the error. I just open the project, run all the tests (⌘+U) and it works fine.

I'm getting this when I compile the project exactly as downloaded from GitHub without any modifications. In order to make sure that I hadn't changed a thing, I unzipped the project fresh after closing Xcode and rebooting my computer. I opened and compiled and the same error was there. Are there compiler settings outside of the project settings? That is the only thing I can think of, unless there is some bug in Xcode 6.3.1 having to do with the migration path from beta to the final release of 6.3.1. Any other thoughts or ideas?
  From: Ivan Sanchez<EMAIL_ADDRESS>
To: s4nchez/swift-poker-hand-evaluator<EMAIL_ADDRESS>Cc: JPVan<EMAIL_ADDRESS>Sent: Monday, April 27, 2015 8:41 PM
Subject: Re: [swift-poker-hand-evaluator] libtool: unknown option character `X' in: -Xlinker (#3)
When are you getting this error exactly? Are you trying to link the evaluator to some other code?I'm also using Xcode 6.3.1 and can't reproduce the error. I just open the project, run all the tests (⌘+U) and it works fine. —
Reply to this email directly or view it on GitHub.

Apparently, according to what I've found, "Xcode does not support building static libraries that include Swift code", which is causing that error. I've fixed by changing the main target to dynamic library.
I'm not sure about the implications of this change since I haven't tried to reuse this code as part of another project (I was just using tests to check it was working). I hope it solves your problem though.

Awesome, thanks!!

—
Reply to this email directly or view it on GitHub.


Add gemini/gemini-pro model to fetched models

This commit adds the gemini/gemini-pro model to the list of models fetched from the server in the fetchModels function.

This issue is already fixed. See this: https://github.com/OpenDevin/OpenDevin/issues/591#issuecomment-2034662809
Moreover hardcoding model ids in OpenDevin is a bad idea, could cause divergence from what is actually supported in Litellm.

Closing thanks to the alternate fix


Only basic functionality work by fact on MiniDSP DDRC24 and 2x4HD

I use latest build + DDRC24/2x4HD (MCU 1.61, DSP 1.6) + WifiDG and only basic settings can be changed like (Master Settings, Gain, mute, etc) using cli. All other changes like outputs/inputs settings dont work by fact.
Works only = "minidsp gain", "minidsp mute", etc
Dont work = "minidsp output 3 gain -- -20", and everything that is related to inpust/output
What can be wrong?

The official applications don't read state from the device, so the settings get changed, but they don't get reflected in their user interface. That's a limitation of the way the firmware is designed.


Bug: Attempting to disable dragging conditionally on an element in a list of items results in the entire list being non-draggable.

Attempting to re-work the draggable example in the docs. You can see the reproduction on the feature/no-drag branch

Facing the same issue. Basically:

draggable does allow to exclude additional elements inside the parent to not be draggable
but it does not allow to disabled one of the value list items as otherwise it will throw this console error

Is this intended or a bug? The use case is a dynamic list of object as values where a property of the object can define whether this specific object is sortable or not.

@distantnative Your bullet points break down the issue well. I wouldn't consider it a bug but more of a poorly thought out API. If I had the opportunity to do it over again, I think I would have a separate callback function for the first bullet point, and draggable would be a callback function for the second. Even then, there is some more granularity to consider: what if the developer also wants to prevent the given element from being sorted as well?
In the meanwhile while I mull this over, my suggestion would be to override event handlers and write your own logic to determine whether a given node should be draggable or sortable. I've created a small playground on the docs you can view here playground (should be more viewable in light-mode). Here's the link to the code: (https://github.com/formkit/drag-and-drop/blob/main/docs/pages/issue-12.vue)

@sashamilenkovic thanks for the quick reply and even more for providing a workaround

@sashamilenkovic thanks for the quick reply and even more for providing a workaround

No problem at all! Let me know if you have any questions about the implementation/see any issues with it.

@sashamilenkovic Any updates on this? I'm facing the same issue and while your work-around does the job, it feels a bit messy.


rewrite read from csv to support directed vertices

my bad, just rewrote graph to adjList and it's done


fix(navigation): render long titles correctly

fix https://github.com/jgroth/kompendium/issues/110

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


New command: get tenant app catalog URL

Get URL of the tenant app catalog

command: spo tenant appcatalog get
requires connection to the tenant admin site

For more information see: https://github.com/SharePoint/PnP-PowerShell/blob/master/Commands/Admin/GetTenantAppCatalogUrl.cs

Can I start looking into this?

Absolutely! Appreciate your help!


[QUESTION] i want to create airgrapped update on 1.3.0 to 1.3.1

Hello, when running kubectl create -f i receive the following error saying
Ouput
kubectl create -f http://<IP_ADDRESS>/v1.3.1/version.yaml
Error from server (AlreadyExists): error when creating "http://<IP_ADDRESS>/v1.3.1/version.yaml": versions.harvesterhci.io "v1.3.1" already exists
is possible to remove current file and add airgrapped update?

@AhBun9 You can get the current versions with:
kubectl get versions -n harvester-system

and delete one version or edit it with kubectl delete/kubectl edit.


Show the number of views per content item in the table

This is no longer ready for review due to the recent merged changes from #39

Following chats from #38, moving Services::GoogleAnalytics from lib to app
Also changing metric from ga:pageviews to ga:uniquePageviews, According to Performance analysts.. "ga:uniquePageviews is a better measure of times the content was viewed in a session""

Trello card
Description
For every content item, content designers want to see the number of page views for a content item belonging to an organisation.
Here a new column/attribute (i.e number_of_views) has been added to the
content_item(s).
The number_of_views attribute’s value is obtained from the Google Analytics Reporting API V4 [1].
This response from the API is returned as an instance of [1]. when this is converted to JSON it returns [3].
For more on query, see [2], [4] and [5]
For obtaining the page views a piece of content_item, this endpoint [1] is queried with the base_path supplied.
This PR also updates the content items view table to show the page views.
[1] https://analyticsreporting.googleapis.com/v4/reports:batchGet
[2] https://developers.google.com/analytics/devguides/reporting/core/v4/parameters
[3] https://developers.google.com/analytics/devguides/reporting/core/v4/rest/v4/reports/batchGet#ReportRequest.FIELDS.date_ranges
[4] API explorer request for just one
[5] API explorer request for more than one

Superseded by #45


Inconsistency among autograd and finite-difference gradients

Hello,
I am trying to implement an optimization whose FOM mixes three differentiaded sub-FOMs, among which the Q-factor; he >problem I am facing is an inconsistency between the autograd and the finite-difference gradient for this quantity:

def of_daisy(params):
DbottomINT = params[0]
DmiddleINT = params[1]
DtopINT = params[2]
DgratingINT = params[3]
DspacerINT = params[4]
fillAINT = params[5]
fillBINT = params[6]
phc = struttura(DbottomINT, DmiddleINT,DtopINT, DgratingINT,DspacerINT,fillAINT,fillBINT)
#path = phc.lattice.bz_path(['G', 0.01*np.array([np.pi, 0])], [100])
neig = 20# number of Bloch bands to store
gmax = 10 # truncation of reciprocal lattice vectors
gme = legume.GuidedModeExp(phc, gmax=gmax)
options = {'gmode_inds': [1],
       'numeig': neig,
       'verbose': False,
       'gradients': 'exact',
       'compute_im': False
        }
gme.run(kpoints=np.array([[0], [0]]), **options)
(freq_im, _, _) = gme.compute_rad(0, [1])
Qinv = 2*freq_im[0]/gme.freqs[0, 1]
 ....
 ....
 ....
 return npa.divide(Qinv,FOM2)

The starting point autograd gradient is:
Autograd gradient :    [-4.79306617e-17 -6.15112936e-18 -4.55829281e-17 -1.78071921e-17
-5.44515292e-17 -7.66094519e-18  1.08829952e-17]

The finite-difference gradient is:
Numerical gradient:   [-3.30385247e-16 -1.63853199e-16 -2.85469258e-18 -6.53521882e-16
-1.30974041e-16 -2.60724278e-16  1.13700734e-16]

Could you help somehow?

This is hard to debug without having the full code, and even with having the full code it will require some work. Could you start simple and for example check if the gradient of Qinv is correct and same with the gradient of FOM2? If the latter is the problem, could you provide that function?

This is hard to debug without having the full code, and even with having the full code it will require some work. Could you start simple and for example check if the gradient of Qinv is correct and same with the gradient of FOM2? If the latter is the problem, could you provide that function?

Sorry, right I missed to mention this: I actually tested each figure of merit alone; the only one producing the error is Q (or Qinv, it doesn't make any difference if I invert before or in the return); FOM2 is already made of 2 other figures of merit and does produce matching autograd and finite-difference gradients.. Q produces mismatching gradients both alone or multiplied by FOM2.
Thanks!

I see. Nothing comes off the top of my head as what could be wrong. I would need to try out your full code (if you could prepare a script?) and find some time to work on it. Alternatively you could try dissecting the problem even further - we know that Q works in some cases as in some of the examples, so can you simplify your problem to a case in which it works, and then figure out which component exactly breaks it?

I see. Nothing comes off the top of my head as what could be wrong. I would need to try out your full code (if you could prepare a script?) and find some time to work on it. Alternatively you could try dissecting the problem even further - we know that Q works in some cases as in some of the examples, so can you simplify your problem to a case in which it works, and then figure out which component exactly breaks it?

Thank you so much! Here the code:
import numpy as np
import matplotlib.pyplot as plt
import time
import autograd.numpy as npa
from autograd import grad, value_and_grad
import legume
from legume.minimize import Minimize
from legume.utils import grad_num
%load_ext autoreload
%autoreload 2
c_speed = 3 * 10**14
Grating parameters as for Fig. 3 (b)
nsio2 = 1.448
ntio2 = 2.13
#nman2400 = 1.4967
nman2400 = 1.4967
a = 2 * 0.7
lambda0 = 0.6165
ymax = 0.05      # ficticious supercell length in y-direction, should be smaller than 1/gmax below
fillA0 =  0.277 / 2   # width of dielectric
fillB0 = 0.124      # width of dielectric rods
Dbottom0 = 0.140 / a
Dmiddle0 = 0.042 / a
Dtop0 = 0.155 /a
Dspacer0 = 0.001 /a
Dgrating0 = 0.149 /a
epsSIO2 = nsio22
epsTIO2 = ntio22
epsMAN2400 = nman2400**2
legume.set_backend('autograd')
lattice = legume.Lattice([1, 0], [0, ymax])
def struttura(Dbottom, Dmiddle,Dtop, Dgrating,Dspacer,fillA,fillB):
phc = legume.PhotCryst(lattice, eps_l=epsSIO2, eps_u=1.)
gap = (0.5 - fillA - fillB)/2
phc.add_layer(d=Dbottom, eps_b=epsTIO2)
phc.add_layer(d=Dmiddle, eps_b=epsSIO2)
phc.add_layer(d=Dtop, eps_b=epsTIO2)
phc.add_layer(d=Dgrating, eps_b=1.)
phc.add_shape(legume.Poly(eps=epsMAN2400, x_edges= [-0.5, -0.5, -0.5 + fillA, -0.5 + fillA], y_edges=[element * ymax for
element in [0.5, -0.5, -0.5, 0.5]]))
phc.add_shape(legume.Poly(eps=epsMAN2400, x_edges= [-0.5 + fillA + gap, -0.5 + fillA + gap,-0.5 + fillA + gap + fillB, -0.5 +
fillA + gap + fillB] ,y_edges=[element * ymax for element in [0.5, -0.5, -0.5, 0.5]]))
phc.add_shape(legume.Poly(eps=epsMAN2400, x_edges= [-0.5 + fillA + 2gap + fillB, -0.5 + fillA + 2gap + fillB,-0.5 + 2fillA
+ 2gap + fillB, -0.5 + 2fillA + 2gap + fillB] ,y_edges=[element * ymax for element in [0.5, -0.5, -0.5, 0.5]]))
phc.add_layer(d=Dspacer, eps_b=epsSIO2)
return phc
def of_daisy(params):
DbottomINT = params[0]
DmiddleINT = params[1]
DtopINT = params[2]
DgratingINT = params[3]
DspacerINT = params[4]
fillAINT = params[5]
fillBINT = params[6]
phc = struttura(DbottomINT, DmiddleINT,DtopINT, DgratingINT,DspacerINT,fillAINT,fillBINT)
neig = 20# number of Bloch bands to store
gmax = 12 # truncation of reciprocal lattice vectors
gme = legume.GuidedModeExp(phc, gmax=gmax)
options = {'gmode_inds': [1],
       'numeig': neig,
       'verbose': False,
       'gradients': 'exact',
       'compute_im': False
        }
gme.run(kpoints=np.array([[1], [0]]), **options)
(freq_im, _, _) = gme.compute_rad(0, [1])
#Qinv = 2*freq_im[0]/gme.freqs[0, 1]
Q = gme.freqs[0, 1]/2/freq_im[0]
gapFOM = npa.square(npa.subtract(gme.freqs[0, 3],gme.freqs[0, 2]))
QFOM = npa.multiply(npa.arctan(npa.divide(Q,10**21)),100)
Ez = gme.get_field_xy('e', kind=0, mind=3, z=DbottomINT + DmiddleINT/2, component='z', Nx=100, Ny=100)[0]['z']
fieldFOM = npa.arctan(npa.divide(npa.square(npa.amax(npa.abs(Ez))),50)) * 10**3
FOM = npa.multiply(fieldFOM,QFOM)
FOM2 = npa.multiply(FOM,gapFOM)
return npa.divide(1,FOM2)

def of_d_r(params):
p_daisy = npa.array(list(params))
return of_daisy(p_daisy)
 obj_grad = value_and_grad(of_daisy)

 pstart = npa.array([Dbottom0, Dmiddle0,Dtop0, Dgrating0,Dspacer0,fillA0,fillB0])

 grad_a = obj_grad(pstart)[1]
 print("Autograd gradient:   ", grad_a)

 grad_n = grad_num(of_daisy, pstart)
 print("Numerical gradient:  ", grad_n)


Automated Package Upgrade

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Cross Platform Path Combination

Good Day,
In light of the recent conversation and development towards cross platform support this PR gets ahead of the curve and ensures the paths are generated correctly, not depending on the \ character which is windows specific. One side effect is that the strings that are generated must be static readonly and instead of instanced constants.
Two further changes to discuss/consider:

I would like to include removing the @ verbatim strings, if that is something you are interested in. with the \s removed they are wholly unnecessary.
It does not appear to be necessary to cache these values. We could convert them to Properties and pass them back in real time.

Both changes are pretty straight forward, the later has more opportunity for impacting change across the framework/platform and so we may want to put that in a second PR, later.
I have updated my game to run on this build, created a handful of new projects and did not run across any issues these changes would create. I did not see any test cases for this block and given that you would like them to be created I can put them together for you as well.
I have synced up the formatting as best I could manage, I hope it complies. If not I will gladly correct any issues that present themselves.
Best wishes.

Looks good to me. Assigning the community team for review to keep them in the loop, but I'd say this is a merge.

It does not appear to be necessary to cache these values. We could convert them to Properties and pass them back in real time.

Don't see a reason why those values should change at runtime, so I'd keep them as static readonly fields for now.

I would like to include removing the @ verbatim strings, if that is something you are interested in. with the \s removed they are wholly unnecessary.

Agreed, go ahead. Leaving the PR open for the removal commit, will merge afterwards.
Thanks for the PR!

Merged. Congrats to your first PR, and thanks for the contribution 🙂


Add rule to drop size definition in casting to VARCHAR

On spark, when casting to VARCHAR(n) in a query, it doesn't actually truncate the string to the defined length.
But spark doesn't allow t cast to varchar witouth a length definition.
If we blindly copy this to DuneSQL, the string will get truncated resulting in a different query output.
This rule will drop all size definitions from a VARCHAR cast.
Spark example: https://dune.com/queries/2601383
DuneSQL example: https://dune.com/queries/2601384

Oh sheesh, I hadn't seen your PRs, I'm so sorry! I thought I had turned on notifications for everything. Sorry!

no worries @vegarsti!
I completely agree with @GeorgeSittas comments, doing this at parse time for the spark dialect is the correct approach.
Do I move the logic to v2_transforms in custom_transforms.py to achieve that?

Oh sheesh, I hadn't seen your PRs, I'm so sorry! I thought I had turned on notifications for everything. Sorry!

no worries @vegarsti!
I completely agree with @GeorgeSittas comments, doing this at parse time for the spark dialect is the correct approach. Do I move the logic to v2_transforms() in custom_transforms.py to achieve that?

Exactly! Thanks!

The fix has been merged upstream, so this issue should be resolved with the next SQLGlot release.

@0xRobin I created #74 to track this, should be fixed very soon, since SQLGlot releases often. Thanks for raising this!

great to see this implemented upstream. :+1:


Added configuration for SSL keystores

Code was compiled & tested successfully in my use case.

I hereby accept the CLA associated with this repository.

Please resolve the conflicts so it's ready to merge.

Done.


Added the scroll to top with enhanced ui and fixed someone has putted the css directly on button which is bad habbit fixed that

Pull Request Format
PR Title
#74 Adding "Scroll top down" feature to the webpage
Type of PR

Add X in the box to specify the improvement type.
[X] Bug fix
[X ] Feature enhancement
[ ] Documentation update
[ ] Other (specify): ___________

Description
I have created the scroll to bottom button with enhanced ui and user experience looking very amazing and fixed some isses by inexperienced people directly applied styles to button it took me 3 min to find that someone has already applied the styles directly to the button tag which is not good.
Screenshots / Videos (if applicable)
Before:
https://github.com/user-attachments/assets/c1b48029-cb37-4bf2-a108-3c03e2cbc00c
After:
https://github.com/user-attachments/assets/63d8059f-d642-4c30-892d-f156924695ae
Checklist

Add X in the box to specify.
[X] I have performed a self-review of my code.
[ X] I have tested the changes thoroughly before submitting this pull request.
[ X] I have provided relevant issue numbers, screenshots, and videos after making the changes.
[ X] I have commented my code, particularly in hard-to-understand areas.

Thank you for reviewing my pull request!

@swaraj-das Check this and try to check on people who are doing like this applying styles directly on tag and so okay it is not good to do so. Merge it when you have checked this.

@swaraj-das and @AnitSarkar123 If you have reviewed then you can add labels and assign it to me and merge it.

@swaraj-das and @AnitSarkar123 Please merge this PR if you have no issue i have by mistake added the new changes in the same PR which is not good can we talk over this please another issue i rasied and solved is merged into this PR only respond asp if you can

issue #110 is also done in this only by mistake i pushed it and it is committed here now

@swaraj-das and @AnitSarkar123 If you get time then you can tell me the conflicts are increasing with the the increase time so i will have to do more work in that case.

varcel can't deployed it so we can't marge it at this time ...also make this pr again with an another branch because 2 issues solved in one you can't get the points of the tow issues

will create another PR for the scroll


Updated docker build process

Hi!
This PR contains changes that working on my Mac to build docker containers.
I cannot figure out why script need envsubst.
Renamed dockerfile, removed -f option in build execution
Added ARG variable, this allow to support Docker Desktop and more clear way to push values to build context.

Much more intuitive, thanks.


swc-loader does not expose filename to plugins

Describe the bug
While reading the node-swc code comment here  it says that the filename is exposed to plugins, but while testing and reading the docs, I see no way or example to access the filename that is currently being passed to the plugin when using swc-loader with webpack.
Input code
No response
Config
No response
Playground link
No response
Expected behavior
The filename should be exposed to plugins when using the swc-loader. There should also be an example in the docs for this.
Actual behavior
The filename is not exposed to plugins when using the swc-loader and there is no example on how to access the filename
Version
0.1.15
Additional context
No response

Moving to the https://github.com/swc-project/swc-loader repo


getmd5() after uploading the file causing a warning.

Let's go strait to it! A code like this one:
try{     $file->upload();     $file->getMd5(); // This line }catch (...) {...}
Is causing a warning:

warning: md5_file(...\tmp\phpA6C9.tmp): failed to open stream: No such file or directory

I have the same problem too :/


Move context value provider from MediaProvider to local/useContextValueProvider

Summary
Move the body of MediaProvider that generates the context value to media/local/provideContextValue.
As per: docs.google.com/document/d/1S9jTh6Lz3UEhj9Wkb7S7x_kJGvRVpEaNOqk2aCyb4y8.
Testing Instructions
Open the media pane to verify that it appears.

LGTM on provideContextValue name change

👍


Numeric input field accept the text input

Describe the bug
A Numeric input field is accepting other input as well instead of numbers only in the emulator like Text.
Component
SDC library
To Reproduce
Steps to reproduce the behavior:

Go to any questionnaire.
Click on any numeric field.
Then type any letter.

Expected behavior
A numeric field should only accept the numeric input either on an emulator or a physical device
Screenshots
https://user-images.githubusercontent.com/62104757/225644437-f1611087-14a6-4920-b8d7-38ceb218f351.mp4
Smartphone (please complete the following information):

Device: emulator
Android version: 11
Build number: RSR1.201013.001

@owais-vd  I think this is working as intended. The user can type in anything, but it wont get saved into the QuestionnaireResponse if it is invalid. Are you seeing the invalid text saved in the QuestionnaireResponse?

Hi @omarismail94 Yes you are right the end answer would be saved as an empty result in DB, so the idea is just to prevent the user input if getting invalid. here you can see my comment https://github.com/opensrp/fhir-resources/issues/934#issuecomment-1463656473
cc: @dubdabasoduba

Im not sure if the link is invalid or leads to a private thread, but I cannot see your comment!

oh the repo is private but the whole idea is could we set a DigitsKeyListener instance in the edit-text field so that the user can only enter provide digits in text-field.

Hi, @omarismail94 is there any update on https://github.com/google/android-fhir/issues/1915#issuecomment-1477574956 ?


MultiSelect not diplayed if f.value is empty

In the PanelSettings, the MultiSelect Component is not displayed is his value is "".

Is this not normal? I seem to remember that this field expects an array as input.
@jf-viguier can you confirm ?

Nope, not normal, i've fixed it in https://github.com/PrestaSafe/prettyblocks/tree/remove-lang-table
I've some improuvements to add (for settings) and it's gonna be OK.


Cannot use generic values in defineStore with setup stores

Reproduction
https://stackblitz.com/edit/github-wn9aqn-1ldb7t
Steps to reproduce the bug

Run npm i
Go to store/error.ts to see the typescript error

Expected behavior
Should have no typescript errors and be able to useStore normally after specifying generic with setup stores
Actual behavior
Getting typescript error, maybe it is unable to unwrap types because of the generic
Additional information
No response

This is indeed a limitation Vue refs due to unwrapping, note it has been mentioned in a couple issues before. Moving to a discussion


plasma-integration: Fix font style name bug with Qt >= 5.8

Motivation
Qt 5.8 corrects a bug in font style name handling. Unfortunately, KDE relies on that bug. As a result, some font styles (bold, italic, etc.) will be unavailable for UI fonts. This is a patch from upstream and a configuration file update.
Testing

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

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

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

@GrahamcOfBorg test plasma5

Failure on aarch64-linux (full log)
Partial log (click to expand)
error: while evaluating anonymous function at /var/lib/gc-of-borg/nix-test-rs-2/repo/38dca4e3aa6bca43ea96d2fcc04e8229/builder/grahamc-aarch64-community-2/nixos/release.nix:19:72, called from /var/lib/gc-of-borg/nix-test-rs-2/repo/38dca4e3aa6bca43ea96d2fcc04e8229/builder/grahamc-aarch64-community-2/lib/attrsets.nix:282:43:
while evaluating 'hydraJob' at /var/lib/gc-of-borg/nix-test-rs-2/repo/38dca4e3aa6bca43ea96d2fcc04e8229/builder/grahamc-aarch64-community-2/lib/customisation.nix:168:14, called from /var/lib/gc-of-borg/nix-test-rs-2/repo/38dca4e3aa6bca43ea96d2fcc04e8229/builder/grahamc-aarch64-community-2/nixos/release.nix:19:80:
while evaluating 'importTest' at /var/lib/gc-of-borg/nix-test-rs-2/repo/38dca4e3aa6bca43ea96d2fcc04e8229/builder/grahamc-aarch64-community-2/nixos/release.nix:15:26, called from /var/lib/gc-of-borg/nix-test-rs-2/repo/38dca4e3aa6bca43ea96d2fcc04e8229/builder/grahamc-aarch64-community-2/nixos/release.nix:19:90:
while evaluating anonymous function at /var/lib/gc-of-borg/nix-test-rs-2/repo/38dca4e3aa6bca43ea96d2fcc04e8229/builder/grahamc-aarch64-community-2/nixos/tests/make-test.nix:1:4, called from /var/lib/gc-of-borg/nix-test-rs-2/repo/38dca4e3aa6bca43ea96d2fcc04e8229/builder/grahamc-aarch64-community-2/nixos/release.nix:15:34:
undefined variable 'lib' at /var/lib/gc-of-borg/nix-test-rs-2/repo/38dca4e3aa6bca43ea96d2fcc04e8229/builder/grahamc-aarch64-community-2/nixos/tests/make-test.nix:5:14

Failure on x86_64-linux (full log)
Partial log (click to expand)
error: while evaluating anonymous function at /var/lib/gc-of-borg/.nix-test-rs/repo/38dca4e3aa6bca43ea96d2fcc04e8229/builder/grahamc-zoidberg/nixos/release.nix:19:72, called from /var/lib/gc-of-borg/.nix-test-rs/repo/38dca4e3aa6bca43ea96d2fcc04e8229/builder/grahamc-zoidberg/lib/attrsets.nix:282:43:
while evaluating ‘hydraJob’ at /var/lib/gc-of-borg/.nix-test-rs/repo/38dca4e3aa6bca43ea96d2fcc04e8229/builder/grahamc-zoidberg/lib/customisation.nix:168:14, called from /var/lib/gc-of-borg/.nix-test-rs/repo/38dca4e3aa6bca43ea96d2fcc04e8229/builder/grahamc-zoidberg/nixos/release.nix:19:80:
while evaluating ‘importTest’ at /var/lib/gc-of-borg/.nix-test-rs/repo/38dca4e3aa6bca43ea96d2fcc04e8229/builder/grahamc-zoidberg/nixos/release.nix:15:26, called from /var/lib/gc-of-borg/.nix-test-rs/repo/38dca4e3aa6bca43ea96d2fcc04e8229/builder/grahamc-zoidberg/nixos/release.nix:19:90:
while evaluating anonymous function at /var/lib/gc-of-borg/.nix-test-rs/repo/38dca4e3aa6bca43ea96d2fcc04e8229/builder/grahamc-zoidberg/nixos/tests/make-test.nix:1:4, called from /var/lib/gc-of-borg/.nix-test-rs/repo/38dca4e3aa6bca43ea96d2fcc04e8229/builder/grahamc-zoidberg/nixos/release.nix:15:34:
undefined variable ‘lib’ at /var/lib/gc-of-borg/.nix-test-rs/repo/38dca4e3aa6bca43ea96d2fcc04e8229/builder/grahamc-zoidberg/nixos/tests/make-test.nix:5:14

@ttuegel do you know if this is a bug in ofborg, or nixos, or the kde test?

@grahamc The test passes locally on x86_64-linux. I presume it is an ofborg problem, but I don't have positive evidence of that.

I think it may have been fixed in https://github.com/NixOS/nixpkgs/commit/2a09b4275c353bde6ee72d126cd0e99834dcf66e?

I think you're right. Let's try this again:
@GrahamcOfBorg test plasma5

Failure on x86_64-linux (full log)
Partial log (click to expand)
building path(s) ‘/nix/store/z3nim83965cpap3j4xlp0znqmq6jkgmi-desktops’
building path(s) ‘/nix/store/40gwn8qjklck15y54czrz3w62a62056w-sddm.conf’
cannot build derivation ‘/nix/store/3g32mcj8k13923zw5nfbg6jwafnxxd76-etc.drv’: 4 dependencies couldn't be built
cannot build derivation ‘/nix/store/nf5dxks8xy69djmx4hmb76ymzc501g8r-nixos-system-machine-18.03.git.7b6618b.drv’: 2 dependencies couldn't be built
cannot build derivation ‘/nix/store/mmcln1539xm2mkwzry6yfac00911nqrn-closure-info.drv’: 1 dependencies couldn't be built
cannot build derivation ‘/nix/store/43yi0cy9arvyyxqsbkfwc4dycv8dsrh8-run-nixos-vm.drv’: 2 dependencies couldn't be built
cannot build derivation ‘/nix/store/yb99ggwy8wiqc2s9hhv73rh9mcj76b88-nixos-vm.drv’: 2 dependencies couldn't be built
cannot build derivation ‘/nix/store/zlqy87fda5bmgq68cmhpqkyh1y9m7mcg-nixos-test-driver-plasma5.drv’: 1 dependencies couldn't be built
cannot build derivation ‘/nix/store/yzcjln4df0glx0byj7vm916is71c66w3-vm-test-run-plasma5.drv’: 1 dependencies couldn't be built
error: build of ‘/nix/store/yzcjln4df0glx0byj7vm916is71c66w3-vm-test-run-plasma5.drv’ failed

Failure on aarch64-linux (full log)
Partial log (click to expand)
cannot build derivation '/nix/store/2fi499hyfgfmkixff23kg0yhydm85gyb-user-units.drv': 1 dependencies couldn't be built
building '/nix/store/jpqliw0h1m7mr4ni51aj2divffrgh3v3-sddm.conf.drv'...
cannot build derivation '/nix/store/g1j51fww66d7nx86s06220h7vk7rsqq5-etc.drv': 4 dependencies couldn't be built
cannot build derivation '/nix/store/jv29iz8dmzj45yq1lbjvwv8iz51ysn0y-nixos-system-machine-18.03.git.96559f2.drv': 2 dependencies couldn't be built
cannot build derivation '/nix/store/nnkyzx8crv4b1jwdimnlm44vdjhc2jc6-closure-info.drv': 1 dependencies couldn't be built
cannot build derivation '/nix/store/hw5pcil5zl2v7sq585wikw2wdnmfgjbr-run-nixos-vm.drv': 2 dependencies couldn't be built
cannot build derivation '/nix/store/axif7a08l9y73wvgw6bh217p7fmb6nfp-nixos-vm.drv': 2 dependencies couldn't be built
cannot build derivation '/nix/store/c188a5qmlv5rdk1kf7f2003vfv680iab-nixos-test-driver-plasma5.drv': 1 dependencies couldn't be built
cannot build derivation '/nix/store/yz8w60b86g8xwzki0qn7j1dx1v9x6pkc-vm-test-run-plasma5.drv': 1 dependencies couldn't be built
error: build of '/nix/store/yz8w60b86g8xwzki0qn7j1dx1v9x6pkc-vm-test-run-plasma5.drv' failed

This seems to be the core of the issue:
trying https://phabricator.kde.org/file/data/nr6lcghxw2xejj7pftfh/PHID-FILE-dexapcoktxshi5dtwhk7/D9070.diff
  % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
                                 Dload  Upload   Total   Spent    Left  Speed

  0     0    0     0    0     0      0      0 --:--:-- --:--:-- --:--:--     0
  0     0    0     0    0     0      0      0 --:--:-- --:--:-- --:--:--     0
curl: (22) The requested URL returned error: 404 Not Found
error: cannot download D9070.diff from any mirror
builder for ‘/nix/store/9kz4hdn0mhh045bcqplfaggfn3y9b2gj-D9070.diff.drv’ failed with exit code 1

It seems like Phabricator might randomly generate patch URLs, so let's try downloading it from the interface instead.

Failure on aarch64-linux (full log)
Partial log (click to expand)
copying path '/nix/store/868calwv6klw7sk1v7awj6v5vw85zkny-breeze-qt5-5.11.5-bin' from 'https://cache.nixos.org'...
copying path '/nix/store/j6g2s4kylb4chzrqrfyfdi8wsy0vvk9h-kactivities-5.42.0-dev' from 'https://cache.nixos.org'...
copying path '/nix/store/0vhs2fyjw9xyailfzwagm66cg5y8k8ks-kdeclarative-5.42.0-dev' from 'https://cache.nixos.org'...
copying path '/nix/store/l0ywx0pncw63z5905lrbf2ybik4k64wp-knewstuff-5.42.0-dev' from 'https://cache.nixos.org'...
copying path '/nix/store/7phdg1mbrlhihf4azmviwdsi4lb10ip1-plasma-framework-5.42.0-dev' from 'https://cache.nixos.org'...
copying path '/nix/store/wh1kxkx3kfmn6m3iafqr4vspszr876l8-kcmutils-5.42.0-dev' from 'https://cache.nixos.org'...
copying path '/nix/store/7b40n5dxmwj7dn5qi244mqqlbznd8r15-frameworkintegration-5.42.0-dev' from 'https://cache.nixos.org'...
copying path '/nix/store/yqvgh6z1k8k3y753gchr5n9xdhg1mkz5-breeze-qt5-5.11.5-dev' from 'https://cache.nixos.org'...
cannot build derivation '/nix/store/38bh8jv3lzgxxyvq2nyhpmhg23jrm1mb-plasma-integration-5.11.5.drv': 1 dependencies couldn't be built
error: build of '/nix/store/38bh8jv3lzgxxyvq2nyhpmhg23jrm1mb-plasma-integration-5.11.5.drv' failed

Failure on x86_64-linux (full log)
Partial log (click to expand)
fetching path ‘/nix/store/mikskfygfv5lw9fjiq6dxjdvh9apqn55-breeze-qt5-5.11.5-dev’...
could not download ‘http://<IP_ADDRESS>:32062/mikskfygfv5lw9fjiq6dxjdvh9apqn55.narinfo’ (Curl error 7)

*** Downloading ‘http://cache.nixos.org/nar/1hvl41rc38q20p13vkr2zfvfgqashmvpi7zcdqj3v3451lpbz537.nar.xz’ (signed by ‘cache.nixos.org-1’) to ‘/nix/store/mikskfygfv5lw9fjiq6dxjdvh9apqn55-breeze-qt5-5.11.5-dev’...
  % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
                                 Dload  Upload   Total   Spent    Left  Speed

  0     0    0     0    0     0      0      0 --:--:-- --:--:-- --:--:--     0
100  3400  100  3400    0     0   3400      0  0:00:01 --:--:--  0:00:01 11147

cannot build derivation ‘/nix/store/qqnjq068wibckf345smc7q26vmj1r3cx-plasma-integration-5.11.5.drv’: 1 dependencies couldn't be built
error: build of ‘/nix/store/qqnjq068wibckf345smc7q26vmj1r3cx-plasma-integration-5.11.5.drv’ failed

@ttuegel ^ was this auto-building helpful, or annoying?

Helpful, since I was just going to request the build again anyway. 😄
I can't figure out why the hash is failing now, but that's an unrelated issue.

Let's try checking the patch in to Nixpkgs, since I can't get a stable link upstream. 🙄

Success on x86_64-linux (full log)
Partial log (click to expand)
glibPreFixupPhase
/nix/store/nrs5d9a43il7ybbd2mfncxi68k2mh300-plasma-integration-5.11.5
post-installation fixup
shrinking RPATHs of ELF executables and libraries in /nix/store/nrs5d9a43il7ybbd2mfncxi68k2mh300-plasma-integration-5.11.5
shrinking /nix/store/nrs5d9a43il7ybbd2mfncxi68k2mh300-plasma-integration-5.11.5/lib/qt-5.10/plugins/platformthemes/KDEPlasmaPlatformTheme.so
strip is /nix/store/5qj61lcvzlap87rf6blvf8p577d482bv-binutils-2.28.1/bin/strip
stripping (with command strip and flags -S) in /nix/store/nrs5d9a43il7ybbd2mfncxi68k2mh300-plasma-integration-5.11.5/lib
patching script interpreter paths in /nix/store/nrs5d9a43il7ybbd2mfncxi68k2mh300-plasma-integration-5.11.5
checking for references to /tmp/nix-build-plasma-integration-5.11.5.drv-0 in /nix/store/nrs5d9a43il7ybbd2mfncxi68k2mh300-plasma-integration-5.11.5...
postPatchMkspecs

Success on aarch64-linux (full log)
Partial log (click to expand)
glibPreFixupPhase
post-installation fixup
shrinking RPATHs of ELF executables and libraries in /nix/store/vg3zh6yl90cylr2caq7hx5l79b2gxc1r-plasma-integration-5.11.5
shrinking /nix/store/vg3zh6yl90cylr2caq7hx5l79b2gxc1r-plasma-integration-5.11.5/lib/qt-5.10/plugins/platformthemes/KDEPlasmaPlatformTheme.so
strip is /nix/store/xmpjypwjmp2qi1chs5kr0hacnh161ls4-binutils-2.28.1/bin/strip
stripping (with command strip and flags -S) in /nix/store/vg3zh6yl90cylr2caq7hx5l79b2gxc1r-plasma-integration-5.11.5/lib
patching script interpreter paths in /nix/store/vg3zh6yl90cylr2caq7hx5l79b2gxc1r-plasma-integration-5.11.5
checking for references to /build in /nix/store/vg3zh6yl90cylr2caq7hx5l79b2gxc1r-plasma-integration-5.11.5...
postPatchMkspecs
/nix/store/vg3zh6yl90cylr2caq7hx5l79b2gxc1r-plasma-integration-5.11.5

@GrahamcOfBorg test plasma5

Failure on aarch64-linux (full log)
Partial log (click to expand)
machine# [    0.639765] clocksource: Switched to clocksource arch_sys_counter
machine# [    0.673799] VFS: Disk quotas dquot_6.6.0
error: OCR failed with exit code 1 at /nix/store/gka8j27d6g1h1naf0dnmdnxvr3p7angk-nixos-test-driver/lib/perl5/site_perl/Machine.pm line 586.
OCR failed with exit code 1 at /nix/store/gka8j27d6g1h1naf0dnmdnxvr3p7angk-nixos-test-driver/lib/perl5/site_perl/Machine.pm line 586.
cleaning up
killing machine (pid 627)
vde_switch: EOF on stdin, cleaning up and exiting
vde_switch: Could not remove ctl dir '/build/vde1.ctl': Directory not empty
builder for '/nix/store/3p7mfnyky3kyqwhshcim68wsfssisv6s-vm-test-run-plasma5.drv' failed with exit code 2
error: build of '/nix/store/3p7mfnyky3kyqwhshcim68wsfssisv6s-vm-test-run-plasma5.drv' failed

Success on x86_64-linux (full log)
Partial log (click to expand)
machine: exit status 1
syncing
machine: running command: sync
machine: exit status 0
test script finished in 1420.05s
cleaning up
killing machine (pid 596)
vde_switch: EOF on stdin, cleaning up and exiting
vde_switch: Could not remove ctl dir '/tmp/nix-build-vm-test-run-plasma5.drv-0/vde1.ctl': Directory not empty
/nix/store/kf3fbwjfsxfb5a3nwls7dpx9dlffrk0j-vm-test-run-plasma5

@grahamc
The test fails on aarch64 due to a problem with OCR; the root of the problem seems to be:
machine: waiting for (?^:Alice Foobar) to appear on the screen
machine: performing optical character recognition
machine: sending monitor command: screendump /build/ocrin.ppm
sh: command: not found

and later:
convert: unable to open image `/build/ocrin.ppm': No such file or directory @ error/blob.c/OpenBlob/2735.
convert: no images defined `tiff:-' @ error/convert.c/ConvertImageCommand/3258.

Is there a known issue with OCR on aarch64?

cc @dezgeg
For now I'd not worry too much about the aarch64 failure.


contact us form does not look good

contact us form does not look good , plss assign me so that i can make a better contact us form

Go ahead


How to train model to extract feature for quality assessment?

For example, I want to train iResNet 50 on MS1M to extract feature for quality assessment. How to set label and losses?

I understand.


Boost Source library with references on all source libraries.

It will require to rename the src NuGet packages. For example:
boost_math => boost_math-src

Fixed


[shared_preferences] Add macOS support

This is already implemented in FDE, and should be moving shortly.

FAILURE: Build failed with an exception.

What went wrong:
Could not determine the dependencies of task ':shared_preferences:compileDebugAidl'.

Could not resolve all task dependencies for configuration ':shared_preferences:debugCompileClasspath'.
Could not resolve project :shared_preferences_macos.
Required by:
project :shared_preferences
Unable to find a matching configuration of project :shared_preferences_macos:
- None of the consumable configurations have attributes.
Could not resolve project :shared_preferences_web.
Required by:
project :shared_preferences
Unable to find a matching configuration of project :shared_preferences_web:
- None of the consumable configurations have attributes.

Try:
Run with --stacktrace option to get the stack trace. Run with --info or --debug option to get more log output. Run with --scan to get full insights.

Get more help at https://help.gradle.org

BUILD FAILED in 6s
Finished with error: Gradle task assembleDebug failed with exit code 1

@goldenplan This issue is about porting the macOS plugin to the flutter repo. For errors on the plugin please file another issue with a copy of the results on flutter doctor -v.
Please try and  delete your Podfile.lock file, and update your Flutter version. Also make sure you are using shared_preferences: ^0.5.6


Image cropped when Scale and Layout set to 125%

When you set Scale and Layout to 125%, the image is "zoomed in" by default. When the snapshot is taken, then only the part that was visible in VideoCaptureElement is captured, the rest is cropped and you can see a black space that should be rest of your picture.

Solved by wrapping VideoCaptureElement in ViewBox.


fix fix dead cloud cluster status empty

Proposed changes
Issue Number: close #xxx

Further comments
If this is a relatively large or complex change, kick off the discussion at<EMAIL_ADDRESS>by explaining why you chose the solution you did and what alternatives you considered, etc...

run buildall

run buildall

run p0

run buildall

run buildall

run buildall


What are the advantages and disadvantages of the argo rollouts and the kruise rollouts

The biggest difference between Kruise-Rollout and Argo-Rollout is that Argo-Rollout itself is a Workload, which was designed to replace Deployment; Kruise-Rollout is a bypass architecture designed to enhance the capability of Deployment.
For instance:

When you want to use Argo-Rollout:  you MUST find a way to migrate your Workloads and Pods to Argo-Rollout; and when you no longer want it, you have to also find a way to migrate back;

When you want to use Kruise-Rollout: you only need to apply the Rollout configuration WITHOUT migrating your Workloads and Pods. When you no longer need it, you can just delete it at any time (even during the publishing process);

In general, Kruise-Rollout is to enhance the ability of Workload, not replace your Workload !!!

@veophi Thank you very much for your answer. I think I already know what to choose.


Unify most standalone documentation & move to Read The Docs or Zulip.org

Zulip's standalone (not in-line) documentation is in three places, mainly:

Zulip.org (staged at https://github.com/zulip/zulip.github.io )
Zulip's Read The Docs site (staged at https://github.com/zulip/zulip/tree/master/docs )
the Zulip GitHub repo's README and other help files in the repo outside the docs/ directory (for example, the API examples)

It'll be easier for people to find what they want if it's in one site or directory. I suggest either Read The Docs for its ease of reading, searching, and document structure, or Zulip.org for its official branding.
I figure that this task will also include

copying/editing the README into a better index/intro page for the Read the Docs site, one that mentions the mobile/desktop apps and links to the major integration/API-related pages
resolving the duplication of zulip.com/hello, zulip.com/features and the zulip.org content
creating better navigation on Zulip.org to make content easier to discover

[Any individual Zulip installation (such as Zulip.com) or client (iOS, Android, web) has inline help or pages like /api and /integrations that need to be host-sensitive or context-sensitive. I'm not suggesting pulling those out of where they are now; I'm only talking about standalone docs. Similarly I am not suggesting moving inline comments from code files. And since the api/ directory is distributed as the "API bindings tarball" on http://zulip.com/api , as @timabbott has pointed out, that bundle should probably continue to be a coherent bundle with some documentation inside, and we shouldn't remove the comment docs we have there.]

Read The Docs also has an "edit on GitHub" link. Right now our docs are a mix of reStructured Text and Markdown, and as @timabbott mentioned, one annoying issue is involved with moving everything to docs/ is that the "Edit on GitHub" links to readthedocs don't work with a mixed Markdown+rST project. So if we move to RtD and we want to lower the barrier to improvements and make that link work, that would require losing the little bit of formatting flexibility rST gives us and switching to just Markdown. I'm fine with that.

I'm working on this now.

@timabbott The Project Clearwater folks had a similar situation to ours and have a sort of complicated setup that allows them to write Markdown -- and then every time a Markdown file gets checked in, their CI sees that, uses pandoc to generate rST, and checks it in. This is because there is fairly basic stuff, like tables of contents and linking from one file or section to another, that is not currently supported if you use Markdown with Read the Docs. And it seems like the Read the Docs people actually foresee that people will be mixing rST and Markdown for the foreseeable future:

We imagine that API reference documentation will continue to be authored in RST for quite some time. Also index pages and other reference heavy content will continue to be RST. FAQ’s, blog posts, and other less reference heavy content is a great candidate for writing in Commonmark.

The real issue I think we want to fix here more urgently is [that people can't use the Edit On GitHub button]("Edit on GitHub" links broken for markdown files in a mixed rST/markdown project). After a bit of investigation by me and a friend, I think we might be able to persist in our mixed rST/Markdown state by changing our requirements.readthedocs.txt file to read
sphinx>=1.3.6
as per https://github.com/spatialaudio/nbsphinx/issues/26 . (Basically, our conf.py configuration does mention .rst and .md as file suffixes, so if we make that change, the "Edit on Github" button should work for both rST and Markdown documents.)
Can we try that?

Yep, that worked!  Thanks for tracking that down @brainwane!

I guess that didn't fix this overall issue, reopening.

I'm realizing that the improvements @christi3k is making in #748 and #747 will go some distance towards #669, and that resolving #669 will be a big part of the de-duplication that points the way forward for zulip.com/zulip.org deduplication/symlinking as well.
Also: do we have control over zulip.com, in case we'd like to move stuff off zulip.com/hello and zulip.com/features?
So, until those two questions are settled, I believe it makes sense for me to confine myself to addressing these parts of the task:

edit the README into a better index/intro page for the Read the Docs site, one that mentions the mobile/desktop apps and links to the major integration/API-related pages
create better navigation on Zulip.org to make content easier to discover
rearrange the docs that are currently on Read the Docs into a better structure

We're hoping to migrate documentation off depending on zulip.com.  Most of the content on zulip.com, aside from zulip.com/integrations, is already on zulip.org (certainly /hellos is now the homepage of zulip.org and /features is now zulip.org/features.html).
Focusing on those areas sounds good.

It's working fine to symlink the main README.md file into the docs/ directory, but Sphinx seems to be choking when I try to symlink /api/README.md into docs/ -- maybe something about directory traversal? I'd welcome any hints, in case you've seen this sort of thing before.

Haven't seen this before, but.  We could put a cp command in the docs
makefile maybe?
-Tim Abbott (mobile)
On May 15, 2016 11:11 AM, "Sumana Harihareswara"<EMAIL_ADDRESS>wrote:

It's working fine to symlink the main README.md file into the docs/
directory, but Sphinx seems to be choking when I try to symlink
/api/README.md into docs/ -- maybe something about directory traversal?
I'd welcome any hints, in case you've seen this sort of thing before.
—
You are receiving this because you modified the open/close state.
Reply to this email directly or view it on GitHub
https://github.com/zulip/zulip/issues/668#issuecomment-219301216

I tried messing with the Makefile for about 10-15 minutes (added $ cp ../api/README.md api-readme.md to the dirhtml section, tried variations involving $BUILDDIR) and wasn't able to get the built HTML to include it. This was my first time trying to fiddle with Makefiles in this way and someone who has worked with them a lot more may have more luck. Leaving it alone for now.

What if we put the real file under docs/ and made the root README.md a symlink to that -- would GitHub still display it?  If so, that would surely work...


In WebpackDevMiddleware, support array-style Webpack configs

As per comments in #132, Webpack itself allows multiple simultaneous configurations (the webpack.config.js file just needs to export an array of configs).
WebpackDevMiddleware should support it (just because there's no reason why not, as long as each config specifies a distinct publicPath). Need to have the JS code return an array of publicPath values to .NET rather than just a single one.

I'm glad you took this even if nobody was explicitly requesting it :')


Voip : decline call when room opened freeze riot

To reproduce:

Open a 1to1 room
Receive a call (audio or video) from an other user from this room
Decline the call

The calling layout appears with the status "Call ended" and it's not possible to close it. The application is frozen.

Note: this bug when the device the call while being in an other room. An iPhone 6+ was used for this test.
Version: 0.2.3 develop #1257

Done

Validated in 0.3.4 develop #1374


1.19.3 installtion on openSUSE: Signature verification failed [6-File is unsigned]

Prerequisites

[ X] Put an X between the brackets on this line if you have done all of the following:

Reproduced the problem in Safe Mode: http://flight-manual.atom.io/hacking-atom/sections/debugging/#using-safe-mode
Followed all applicable steps in the debugging guide: http://flight-manual.atom.io/hacking-atom/sections/debugging/
Checked the FAQs on the message board for common solutions: https://discuss.atom.io/c/faq
Checked that your issue isn't already filed: https://github.com/issues?utf8=✓&q=is%3Aissue+user%3Aatom
Checked that there is not already an Atom package that provides the described functionality: https://atom.io/packages

Description
openSUSE Tumbleweed 64 bit
1.19.3 64 bit rpm (https://github.com/atom/atom/releases/download/v1.19.3/atom.x86_64.rpm) install error: Signature verification failed [6-File is unsigned]
Only when installing with zypper. rpm -i succeeded without errors.
Steps to Reproduce

Download https://github.com/atom/atom/releases/download/v1.19.3/atom.x86_64.rpm
Try to install it on openSUSE with zypper zypper in atom.x86_64.rpm

Expected behavior: successful installation
Actual behavior:
atom.x86_64.rpm:
    Package is not signed!

atom-1.19.3-0.1.x86_64 (Plain RPM files cache): Signature verification failed [6-File is unsigned]

Reproduces how often: always
Versions
openSUSE Tumbleweed latest updates (23.08.2017)
RPM version <IP_ADDRESS>
zypper 1.13.31

@olegantonyan Do other releases of Atom install correctly?
/cc: @ungb Since you have been dealing with install errors on Linux before. Do you know anything about this?

All prior 1.19.3 releases were ok

@olegantonyan, We've updated the README in https://github.com/atom/atom/pull/15380.  I believe that the best way to install atom is as you suggested:  sudo rpm -i and the README are now updated with those steps.
I'm going to close this issue, as I believe this issue is actually from the OS and Zypper maybe having a more strict policy on packages being signed.

@ungb well, if you say so...
It doesn't look like a good idea to me (almost as disabling SSL cert verification when you have problems with certificates)

@olegantonyan +1 following the spec is more secure and future-proof.

@olegantonyan Thanks for the feedback!
I've created issue https://github.com/atom/atom/issues/15509 to add signing for linux package and added it to the backlog.  Currently we’ve never signed our linux packages. The fact that you're seeing the warning now is a change in the OS


2.0.x is not able to create targets automatically

Moving from the 1.0.4 to the 2.0.x (both 2.0.0 and 2.0.1) I have some problems with creating targets automatically when a new user asks for a feature flag. The proxy reply telling that no feature flags are enabled. The /client/auth request returns with status code 200.
Rolling back to the 1.0.4 or bypassing the ff-proxy everything works like a charm.
If i can provide you something more, just ask!

Maybe related to #285?

Hi @ilpianista
I'm really sorry for the delayed response, I missed this issue and it's only just come to my attention.

Moving from the 1.0.4 to the 2.0.x (both 2.0.0 and 2.0.1) I have some problems with creating targets automatically when a new user asks for a feature flag. The proxy reply telling that no feature flags are enabled. The /client/auth request returns with status code 200. Rolling back to the 1.0.4 or bypassing the ff-proxy everything works like a charm.
If i can provide you something more, just ask!

Just to make sure I'm understanding the question, when you say 2.0.x is not able to create targets automatically do you mean if you use 2.0.x you don't see the targets appear in the UI? But when you use 1.x you do see targets appear in the UI?
If what I'm asking above is correct then yes it would be related to this change you mentioned. In v2 of the Proxy we made the decision to not forward targets on to Harness Saas by default, the Proxy still creates them and adds them to its redis cache but because they aren't forwarded on to Harness Saas you won't see them in the UI. If you want to see the Targets in the UI you can enable this functionality by setting the environment variable FORWARD_TARGETS=true

Maybe related to #285?

However I've just realised we only added support for forwarding targets on to the Primary Proxy in V2 and not the read replicas. If you're running v2 as a single Primary the same way you would with v1 then it should all work ok as long as the FORWARD_TARGETS=true environment variable is set. However if you're planning on running the V2 Proxy in HA Mode then we'll need to do some work so the replicas support forwarding the targets on.

Sorry for the late reply, setting FORWARD_TARGETS=true solved my issue. Thanks!

No worries, glad that helped resolve the issue!


Allow XrootD stage-out.

Fixes #161.
Briefly tested with a simple task. People wanting to test are welcome!
For context, at present, the configuration below will just skip XRootD for stage-out with some message like:
>>>> skipping unhandled stage-out method: root://deepthought.crc.nd.edu//store/user/... @ Sat Feb 23 08:50:36 2019

and go directly to gsiftp when hdfs is not available. This PR should fix that.
    output=[
        "hdfs://eddie.crc.nd.edu:19000/store/user/$USER/lobster_test_" + version,
        "root://deepthought.crc.nd.edu//store/user/$USER/lobster_test_" + version,
        "gsiftp://T3_US_NotreDame/store/user/$USER/lobster_test_" + version,
    ]

I don't have time to test this, but if you did at least basic testing, I'm happy to merge this pull request.  Just to verify: you actually tested this code by running some jobs and staging out the output via XRootD?

@klannon Yes, I ran some jobs on crcfe02 and verified the stage out happened via XRootD


Issue accessing @@color-@{name} properties outside the library

The issue is stated at http://stackoverflow.com/q/31475629/881286.
Can you help with it?
Thanks

Already solved.
.color-@{color}-@{step} {
    @buffer: 'color-@{step}';
    @color: @@buffer;

    color: @@color;
}


Add Dapr to PATH when using the MSI installer

Describe the proposal
User can install the Dapr CLI (dapr.exe) in a custom path using the MSI installer. This proposal is to use the installer to also update the PATH variable and point to the Dapr CLI installation.
Documentation reference: https://wixtoolset.org/documentation/manual/v3/xsd/wix/environment.html
Something like this can be used to append the new directory to $env:PATH.
<Environment Action="set" Name="PATH" System="yes" Id="dapr_path" Part="last" Value="$(var.DaprEXEDirectory)" />

Release Note

RELEASE NOTE: UPDATE MSI installer to update the path after Dapr CLI installation

For now can be mitigated with a docs line to add installed cli directory to path

/assign

Closed by #1081


Attribute maps updated

Attribute-maps/saml_uri.py updated with CN and SCHAC OIDS.
Shibboleth.py and base.py was removed because not useful anymore, basic.py refers to mace:dir and probably this is quite useless in our SAML2 definitions.

Thanks and welcome :-)


Deno Not Authorized

bro i cant deploy the bots pls help

That's intended, since the web server it creates is only accessible to Discord with correctly signed data. Just add the URL as Interactions Endpoint in Developer Portal.

still not work

still same

What does not work? Can you give more info?

I've entered the endpoint into the discord developer portal but it's still Not Authorize
and I have also tried entering the redirect url too
but everything is still the same and error
with the words Not Authorized

You don't have to open the site at all... just invite & use the bot in server now.

I've invited the bot but it's still offline

fixed!
I just changed a bit in the mod.ts file section
also thanks for help :)


RFP - houdoku

Checklist

[x] The package I am requesting does not already exist on https://chocolatey.org/packages;
[x] There is no open issue for this package;
[x] The issue title starts with 'RFP - ';
[x] The download URL is public and not locked behind a paywall / login;

Package Details
Software project URL :https://houdoku.org/
Direct download URL for the software / installer : https://github.com/xgi/houdoku/releases/download/v2.14.0/Houdoku-Setup-2.14.0.exe
Software summary / short description:
Houdoku is a free and open source manga reader for the desktop.

Package added.
Install version: https://community.chocolatey.org/packages/houdoku
Portable version: https://community.chocolatey.org/packages/houdoku.portable

Both packages are approved.


After running this tool, OTS warns: WARNING: Layout: tags aren't arranged alphabetically.

I'm new to this tool, perhaps an issue on https://github.com/kojiishi/east_asian_spacing is more appropriate?
It seems add-chws doesn't properly sort the GPOS feature tags when building the features, which gets noticed by opentype-sanitizer:
WARNING: Layout: tags aren't arranged alphabetically.

Fixed in v1.4.1, thank you for reporting this.


[AutoPR blueprint/resource-manager] Update lock modes

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

(message created by the CI based on PR content)

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