Module and inverter parameters from data sheet to pvlib-python

I am trying to model a PV system using pvlib-python. The required PV modules and the inverter models are not present in any of the databases that's been used in either pvlib or SAM by NREL (updated version).
These models are:
Module   = GCL-P6/72H 330
Inverter = SG2500HV (by Sungrow)
I want to know is there any standard way or any approach to input parameters from datasheets directly?
Thanks in advance!

Briefly, yes there are standard ways but they are not coded in pvlib.
As background, a module performance model requires certain parameter values (e.g., Rshunt for the CEC model) that are not found on datasheets.  For modules, there are techniques to obtain parameters for some module performance model (e.g., De Soto, CEC) from datasheet values.  The SAM SDK includes these functions, so you may be able to use SAM to get a set of module model parameters.
For inverters, it's a bit more difficult.  The efficiency curves are needed; these could be digitized from the figure commonly found on an inverter cutsheet. The parameters for the Sandia inverter model can then be estimated by a fitting method.  The method is not difficult (I have a spreadsheet somewhere) and I believe it is also coded in SAM.
Estimation of model parameters from data is an important gap which pvlib currently lacks.  We welcome pull requests :)

cwhanse,
Could you provide step by step procedure to load a locally saved PV module file (CEC model) to Python?
Also could you provide the template file for the CEC model so that i can populate the parameters from datasheet / SAM SDK?
Thanks,

We don't yet have the CEC model implemented but you can use the Desoto model with CEC parameters, it will give you very similar results.
The Desoto model parameters are:

I_L_ref (light current at STC)
I_o_ref (dark current at STC)
a_ref  (product of diode ideality factor, number of series cells and cell thermal voltage at STC)
R_s (series resistance)
R_sh_ref (shunt resistance at STC)
alpha_sc (in A/K, temperature coefficient of Isc)

If you have those values in your locally saved file, then you can pass them as keyword arguments to
(['a_ref', 'I_L_ref', 'I_o_ref', 'R_sh_ref',
305                                 'R_s', 'alpha_sc', 'EgRef', 'dEgdT'],

seems like in the future this issue might be better asked at pvlib-python Google group thanks!

Hi,
I tried using Desoto model with CEC module parameters and it's working. I am looking at system losses at the moment. So issue of working on CEC database is kinda resolved.
I was able to run pvwatts model but there I found this issue that currently the modelchain can't handle
(pvwattts modeled) system with multiple modules in series and parallel to an inverter. So that's a drawback.
It doesn't end here. There are quite a lot number of module and inverter models that we can't find in Sandia/CEC databases and how we can implement them directly having only datasheet values is a big gap.
I am still looking to possible solution and suggestions.
Regards
Jawairia

Hi,
I tried using Desoto model with CEC module parameters and it's working. I am looking at system losses at the moment. So issue of working on CEC database is kinda resolved.
I was able to run pvwatts model but there I found this issue that currently the modelchain can't handle
(pvwattts modeled) system with multiple modules in series and parallel to an inverter. So that's a drawback.
It doesn't end here. There are quite a lot number of module and inverter models that we can't find in Sandia/CEC databases and how we can implement them directly having only datasheet values is a big gap.
I am still looking to possible solution and suggestions.
Regards
Jawairia

We prefer questions about use of pvlib to appear at https://groups.google.com/forum/m/#!forum/pvlib-python
Getting to module model parameters from datasheets is not currently supported in pvlib - see my response above. You may want to look at the capability in SAM - see this thread https://sam.nrel.gov/node/69678
Best regards,
Cliff

This a combination of usage question and duplicate of other issues, so closing the issue.


Make primer work on Windows again

I set out to figure out why the changes I made in #97 caused primer to hang in typeshed's CI, but quickly realised that I couldn't run primer locally at all any more! This is because shlex.quote() does the wrong thing on Windows when you pass it a path. All you actually need to do on Windows to quote a path is to do some basic validation and then surround it with "", so there's a pretty simple workaround.

Is the issue that shlex.quote prefers single quotes instead of double quotes?

I think that's the main issue, yeah, though I don't really know much more than what the warning in the docs tells me.
This solution is derived from Eryk Sun's comments on a Stack Overflow question that suggest that this should always do the right thing if all you need to do is to quote a Windows path, and it was the quoting of paths that was causing me grief locally.
An alternative solution would be to use subprocess.list2cmdline, which I think tries harder to capture all the semantics of the Windows command line. I didn't go for that because:

Not sure we need it here
It's undocumented
There was some scepticism recently when people suggested documenting it: https://discuss.python.org/t/why-is-subprocess-list2cmdline-not-documented/25044/1


attach the remaining count and continue to the List object

add search label search.clusterpedia.io/attach-remaining-count and search.clusterpedia.io/attach-continue

label
url query

-selector  "search.clusterpedia.io/attach-remaining-count=true"
attachRemainingCount=true

--selector  "search.clusterpedia.io/attach-continue=true"
attachContinue=true

example:
curl http://<IP_ADDRESS>:8080/api/v1/namespaces/default/pods?labelSelector=search.clusterpedia.io/attach-remaining-count=true&limit=1&attachContinue=true
{
    "kind": "PodList",
    "apiVersion": "v1",
    "metadata": {
        "continue": "1",
        "remainingItemCount": 1
    },
    "items": [
    ....
    ]
}

Does not support List as ParitalObjectMetadataList with count, other cases will come with count.
https://github.com/kubernetes/kubernetes/pull/107177

LGTM

After a simple discussion, it was decided to use WithRemaningCount and WithContinue


Added note on LockState

https://github.com/MicrosoftDocs/office-docs-powershell/issues/3856

Docs Build status updates of commit faa087c:
:clock10: Full build: the average full build time is 11 min(s) 52 sec(s), this is based on the last 878 build(s) for this repository.

Docs Build status updates of commit faa087c:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

sharepoint/sharepoint-ps/sharepoint-online/Set-SPOSite.md
:white_check_mark:Succeeded
View (sharepoint-ps)

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.

Docs Build status updates of commit e1eefb3:
:clock10: Full build: the average full build time is 11 min(s) 52 sec(s), this is based on the last 882 build(s) for this repository.

Docs Build status updates of commit e1eefb3:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

sharepoint/sharepoint-ps/sharepoint-online/Set-SPOSite.md
:white_check_mark:Succeeded
View (sharepoint-ps)

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.

Docs Build status updates of commit 164c0fe:
:clock10: Full build: the average full build time is 11 min(s) 52 sec(s), this is based on the last 884 build(s) for this repository.

Docs Build status updates of commit 164c0fe:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

sharepoint/sharepoint-ps/sharepoint-online/Set-SPOSite.md
:white_check_mark:Succeeded
View (sharepoint-ps)

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


Add postinstall script to fetch data. Update doc.

I have added a postinstall step to the package.json, this way the script is run to update the database. I have also updated the docs.

This looks great! I'm a bit skeptical to running the script on postinstall though - it makes deployment and updating dependencies etc a bit weird. If you rename the npm script to something else and document the use/setup of it, I'd happily merge this.

Updated. Let me know if you want me to change anything.

:+1:


added additional badges

Current coverage is 100% (diff: 100%)

No coverage report found for master at b260fbc.

Powered by Codecov. Last update b260fbc...c23de9d


Describe how to add a custom network through plugin options

same issue， i need qq

@zenonux
The docs was updated, so as the demo

@zenonux
The docs was updated, so as the demo

awesome


Handling resources usage accros modules

I had a lot of false positives when I tried disableAndroidResources feature. This happens especially when resources are used across modules.
Imagine a module has a String resource that is used by android module. ModuleCheck will suggest to put buildFeatures.androidResources = false in android {} block. This will actually compile fine.
But then during application's assemble task and during resource linking process, build will fail because of AAPT.
Here is an example error that happens in upstream module when resources disabled in downstream:
Execution failed for task ':app:processDebugResources'.
> A failure occurred while executing com.android.build.gradle.internal.res.LinkApplicationAndroidResourcesTask$TaskAction
   > Android resource linking failed
     ERROR:/Users/redacted/app/src/main/res/xml/xml_file.xml:79: AAPT: error: resource string/some_string_resource not found.

Thanks for the report!  I was able to reproduce this.
Auto-disabling androidResources seems to work correctly, unless the module is referencing its own R (or some other module is -- but that's a strange thing to do in this case).
In order to disable androidResources:

the module can't have any res files of its own
there can be no references to the R file (since it wouldn't be generated)
no AndroidManifest.xml can reference any resources at all??
a. I would have to experiment with how this works with nonTransitiveRClass
no resValues defined in its Gradle config
the module can't be an application module

This is a very good summary for sure. When nonTransitiveRClass is not enabled, there could be another edge case:
In parent modules, developers can accidentally use the wrong R import. It can both be the merged R class with the package name of the parent module. Or it can also be the R class from any of the dependant modules.

Yeah, the most common cause I've seen for using the "wrong" R is when copy/pasting.  It happened all the time when creating Espresso robots.
I hope to be able to fix this within the next couple of days.
Thanks again for the report!


modify install_cc to intall cc for all links

Since column generation and function build_link_cost_map() in DUE, DSO, and DODE require all link's travel time, I modify install_cc to be compatible with my testing cases.

Should we remove register_links then?

Also, should we do this for install_cc_tree as well?

I think we can leave register_link() as is since we don’t need cc_tree for each link in those procedures and installing cc_tree for each link will consume too much memory.On Jun 6, 2023, at 15:59, kunhtkun @.***> wrote:
Also, should we do this for install_cc_tree as well?
—Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this because you modified the open/close state.Message ID: @.***>

Then I honestly do not like this inconsistency, but I do not have a strong feeling either. I will leave it to your decision.


Service Worker not defined while accessing PWA in development

As first reported in a PR by @cuining, this regression seems to be introduced in the move away from Parcel.
I believe this may be addressed in #3833, but some more information is still needed.
OS/Web Information

Web Browser: Chrome
Local OS: macOS
Remote OS: macOS
Remote Architecture: 64 bit
code-server --version: 3.11.0

Steps to Reproduce

Start code-server via yarn watch
Open in Chrome, add a PWA shortcut
Refresh

Expected
No console errors should occur.
Actual
Service worker registration fails due to an invalid path. However, the file is available at an different path.
Screenshot

Apologies @cuining for prematurely closing your PR and thank you for raising the issue! ♥ (and @TeffenEllis for following up on this!)
3.11
I am not able to reproduce in 3.11, which is a good sign.

Local development
I am also not able to reproduce 🤔 (macOS + Brave)

Wait, premature comment, I did not refresh

Video
https://user-images.githubusercontent.com/3806031/126562428-49941d78-f023-4f31-8e65-99d822fed199.mov
Okay so actually I believe this is a different bug. I told @code-asher about this a while ago but couldn't reproduce consistently. This happened before the migration away from Parcel.
The workaround is to close the PWA and reopen.
But the fact that you're seeing the raw JS in the browser feels odd and I wonder if these are two separate bugs or the same, manifesting differently 🤔

I have the same issue with 3.11. It looks for the serviceWorker.js at /node_modules/code-server/dist/serviceWorker.js but in fact the new location is /node_modules/code-server/out/browser/serviceWorker.js

Ahhh, I must have forgot to change it in one place. Thank you for the tip @lsoica!

Could this be a caching issue? i.e. it cached the old path. Would a hard refresh fix it?

Cleaning app cache and browser file cache didn't help. Anything else that I've missed ?

Nope! Thank you for trying. cc @code-asher in case he has any other ideas. We're going to look at this in the next cycle (starting next week 🤞)

Ack we didn't update this path!
https://github.com/cdr/code-server/blob/7b8cd25c0c48b6a1799587ebbc9b09d6af7988f3/src/browser/register.ts#L8
We probably didn't notice during development because we had the dist directory from previous builds.

Looks like we should re-open that PR as it has the right fix.

Cool! I'll do that since I closed it.

Same for me but in 'production'.
[Service Worker] registration: Failed to register a ServiceWorker for scope ('https://DOMAIN/') with script ('https://DOMAIN/static/4e8cd09ef0412dfc7b148b7639a692e20e4fd6dd/usr/lib/code-server/dist/serviceWorker.js'): A bad HTTP response code (404) was received when fetching the script. TypeError: Failed to register a ServiceWorker for scope ('https://DOMAIN/') with script ('https:/DOMAIN/static/4e8cd09ef0412dfc7b148b7639a692e20e4fd6dd/usr/lib/code-server/dist/serviceWorker.js'): A bad HTTP response code (404) was received when fetching the script.
The manifest look like this
{
  "name": "code-server",
  "short_name": "code-server",
  "start_url": "./../../../../../../../..",
  "display": "fullscreen",
  "background-color": "#fff",
  "description": "Run editors on a remote server.",
  "icons": [
    {
      "src": "./../../../../../../../../static/4e8cd09ef0412dfc7b148b7639a692e20e4fd6dd/usr/lib/code-server/src/browser/media/pwa-icon-192.png",
      "type": "image/png",
      "sizes": "192x192"
    },
    {
      "src": "./../../../../../../../../static/4e8cd09ef0412dfc7b148b7639a692e20e4fd6dd/usr/lib/code-server/src/browser/media/pwa-icon-512.png",
      "type": "image/png",
      "sizes": "512x512"
    }
  ]
}

MacOS
3.11.1

Thanks for the update! This should be fixed when we get 3.11.2 out the door.


Required view 'menu' with ID<PHONE_NUMBER> for method 'onClick' was not found

Error:
Caused by: java.lang.IllegalStateException: Required view 'menu' with ID<PHONE_NUMBER> for method 'onClick' was not found. If this view is optional add '@Nullable' (fields) or '@Optional' (methods) annotation.
this is my BaseFragment
@Optional
@Nullable
@Override
public View onCreateView(LayoutInflater inflater, @Nullable ViewGroup container, @Nullable Bundle savedInstanceState) {
View view = inflater.inflate(getLayoutId(), container, false);
ButterKnife.bind(this, view);
return view;
}
@Override
public void onActivityCreated(@Nullable Bundle savedInstanceState) {
    super.onActivityCreated(savedInstanceState);
    initData();
}

Fragment A OnClick：
@OnClick(R.id.saveContacts)
public void onClick(View view) {
    switch (view.getId()) {
        case R.id.saveContacts:
            initdate();
            break;
    }
}

What should I do？

The exception message tells you what to do. Either the view needs to always be available or you need to annotate it as being nullable.


[GSOC] integrated support for Zvbc

Sorry, why this is closed?

Sorry, why this is closed?

My GSOC mentor Zewen told me that we are not going to support some instructions with SEW=64.
So we can skip these SEW=64 instructions and I can close the PR

Okay, but the PR is good, so why not merge it? I just need an explanation about valid SEW values.

here it states that LMUL >= SEWmin/ELEN. So for a instruction that only supports SEW=64, SEWmin=64 and ELEN=64
Hence LMUL >= 1

I see, can you make another PR so we can merge this in?

sure


Add country as parameter for App Store

When the app is published in some non-us country, this can't be found using the current method.
Added a new parameter country that allows to pass a custom country in order to get the proper bundleId.

Hi there,
I wanted to create this as a draft. We found a problem related getting the version for the App Store. I tested adding this country as a parameter and it works fine, but I found it hard to keep the old one backward compatible, as adding the country parameter to the get.handler would collide with the get from the version handler without country.
So at the end I only added it as another path to the version endpoint that is the one used from checkVersion.
Maybe it would be good to add this country as a queryParameter in order to keep all the paths working like now?

Hi! Amazing work!
Indeed, a query string parameter might be good, seems a bit cleaner within the current setup.

Ok, I updated it to receive the country as a queryStringParameter, don't know if there is a better way to set the default value in  serverless, as if not parameter is provided, queryStringParameters is null.

Thanks for your contribution!
Released as<EMAIL_ADDRESS>


Update MPI wrappers to be more flexible and handle C++/MPI types correctly

Currently MPI wrappers implemented in coreneuron/mpi/mpispike.cpp have quite some limitations:

they were derived from old C style code
for different argument types there are different functions i.e. code duplication (e.g. nrnmpi_dbl_allreduce_vec for double and long argument type)
doesn't handle type like size_t correctly  (see this)
MPI Op type is passed as int 1, 2, 3  (quite unintuitive API)
When #ranks < 2, not all functions avoid calling MPI functions

We should rewrite them to avoid repeated code and address above limitations. I assume @alkino will be interested to re-write this. @olupton  and @ohm314 can provide input if these should be refactored/implemented in certain way.
P.S. the reason for avoiding calling MPI routines for #ranks < 2 is that MPI compiled library can be used in serial or parallel execution. In serial execution NEURON doesn't initialise MPI and libraries like HPE-MPT end-up with floating point exceptions.

Also, keep in mind that currently the passed operation for reductions is not the MPI_Opt code, but another user-defined opreation code that gets translated.

I don't know if this is relevant to CoreNEURON but one of the reasons for the c api to all this in NEURON is to allow dynamic loading of libmpi (if it exists). This is useful for binary distributions installed on machines where mpi may not be installed, and also be somewhat agnostic about which mpi is installed.

but one of the reasons for the c api to all this in NEURON is to allow dynamic loading of libmpi (if it exists).

@nrnhines : but even if we implement MPI wrappers in C++ with template parameters (to be type agnostic), we can still compile libnrnmpi.so and that can be dynamically loaded?

I still have a lot to learn about modern c++.  If that is the case (it wasn't true in the 1990's, and we need to replace all the elaborate macros that were needed because templates didn't work) then go for it.


Moving gradle dependencies of Dagger2Sample project

Need to export all the dependencies of the root file and sync with the standards of other projects in the repo.

It is done


[Documentation] Example in the generated docs

Move the examples from the readme to the docs

@trkohler do you want to help with this too?


CallCounter C++ version

Feedback on C++ conventions and style particularly welcome

Benchmarks in progress

Could you try benchmarking just called_without_tracing() calls, C++ vs Python?

@jonathanchu33 please feel free to leave as it is, You're not expected to work beyond your internship period. :)

@kkimdev Any update on this PR? Please. Thanks!

Status: This PR itself doesn't have much perf benefit unless _FrequentTracingDetector is also migrated to C++ together. So marking this P2 and we will revisit after we solve other tf.function overheads.

@kkimdev  Any update on this PR? Please. Thanks!

@kkimdev Any update on this PR? Please. Thanks!

@kkimdev Any update on this PR? Please. Thanks!

@kkimdev Any update on this PR? Please. Thanks!

@kkimdev Any update on this PR? Please. Thanks!

@kkimdev Any update on this PR? Please. Thanks!

@kkimdev Any update on this PR? Please. Thanks!

@kkimdev Any update on this PR? Please. Thanks!


@conda-forge-admin, please re-render

@conda-forge-admin, please re-render.

Hi! This is the friendly automated conda-forge-webservice.
I just wanted to let you know that I rerendered the recipe in conda-forge/casacore-feedstock#72.


Migrate back to a single enum for Naia events

There are now (or will be, with the next release) 17 different types of events that the Client can emit.. Initially I really enjoyed the simplicity of being able to have a separate system for each type of Event, but I think there may be benefits to just having a single system handle all Client/Server events via a large enum .. It would reduce a lot of code and make things more maintainable going forward, as well as enforce a specific ordering between event types.
Interested if anyone has thoughts on this. Are the current event handling ergonomics worth it?

I personally prefer discrete systems for code cleanliness and packing things into modules.
One example is if my auth system is especially complex, with a lot of system parameters, I may opt to move it to a module. If it all had to be one system, it's not ideal to have processed the auth events, but then those params are never released to be used by other systems, because the mutable references are still held even when not being used.


[SPARK-22291][SQL] PostgreSQL UUID[] to StringType: Conversion Error

What changes were proposed in this pull request?
This PR fixes the conversion error when reads data from a PostgreSQL table that contains columns of uuid[], inet[] and cidr[] data types.
For example, create a table with the uuid[] data type, and insert the test data.
CREATE TABLE users
(
    id smallint NOT NULL,
    name character varying(50),
    user_ids uuid[],
    PRIMARY KEY (id)
)

INSERT INTO users ("id", "name","user_ids")
VALUES (1, 'foo', ARRAY
    ['7be8aaf8-650e-4dbb-8186-0a749840ecf2'
    ,'205f9bfc-018c-4452-a605-609c0cfad228']::UUID[]
)

Then it will throw the following exceptions when trying to load the data.
java.lang.ClassCastException: [Ljava.util.UUID; cannot be cast to [Ljava.lang.String;
    at org.apache.spark.sql.execution.datasources.jdbc.JdbcUtils$$anonfun$14.apply(JdbcUtils.scala:459)
    at org.apache.spark.sql.execution.datasources.jdbc.JdbcUtils$$anonfun$14.apply(JdbcUtils.scala:458)
...

How was this patch tested?
Added test in PostgresIntegrationSuite.

cc @viirya Can you help review this? Thanks.

Can one of the admins verify this patch?

Please add tests to https://github.com/apache/spark/blob/master/external/docker-integration-tests/src/test/scala/org/apache/spark/sql/jdbc/PostgresIntegrationSuite.scala

@jmchung You can add unit test into PostgresIntegrationSuite.

Seems that we don't even have test against uuid column, when you create test for uuid[], can you also create test for uuid? Thanks.

Besides uuid, other PostgreSQL types such as  "cidr", "inet", they are treated as StringType too, will they work?

Thanks @wangyum and @viirya, I'll add the corresponding tests in PostgresIntegrationSuite.
To @viirya , I'm not sure if the other data types will work,  will consider them into tests, thanks!

gentle ping @wangyum and @viirya, the test case has been added to PostgresIntegrationSuite. As @viirya mentioned early, data types inet[] and cidr[] did not work as for instance of String, they are both mapped to Java's Object class.

@jmchung There are another StringType types "json", "jsonb" in PostgresDialect. Can you also add them into tests?  Thanks.

cc @HyukjinKwon @cloud-fan Please help trigger the jenkins test if you have time. Thanks.

ok to test

add to whitelist

Test build #83103 has started for PR 19567 at commit 040e369.

Thanks @HyukjinKwon :)

Test build #83103 has finished for PR 19567 at commit 040e369.

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

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

Merged build finished. Test PASSed.

Test build #83116 has started for PR 19567 at commit 0998a77.

Test build #83116 has finished for PR 19567 at commit 0998a77.

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

Merged build finished. Test PASSed.

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

Test build #83145 has started for PR 19567 at commit edc5f6f.

Test build #83145 has finished for PR 19567 at commit edc5f6f.

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

Merged build finished. Test PASSed.

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

LGTM except one comment, good catch!

This is not just for UUID[] for now. We should update the PR title.

Thanks @viirya, the title has been changed. Please correct me if the modified title is still inappropriate.

gentle ping @cloud-fan and @viirya, there are some feedbacks about the behavior of obj to string.
case StringType =>
  (array: Object) =>
    array.asInstanceOf[Array[java.lang.Object]]
      .map(obj => UTF8String.fromString(obj.toString))

As @HyukjinKwon mentioned before, it may unsafe to covnert the object to string directly, we'll have the java.lang.NullPointerException in test cases. StringType is different from other types use nullSafeConvert method to deal with the null, it process within UTF8String.fromString.
The UTF8String.fromString returns null and UTF8String so that we cannot use String.valueOf(obj) to avoid the NPE, it will raise another errors (because null != "null"):
- Type mapping for various types *** FAILED ***
  Array("a", "null", "b") did not equal List("a", null, "b") (PostgresIntegrationSuite.scala:120)

IMHO, if we don't want to make separation of match cases, we may remain null to meet the original design, e.g.,
case StringType =>
  (array: Object) =>
    array.asInstanceOf[Array[java.lang.Object]]
      .map(obj => UTF8String.fromString(
        if (obj == null) null else obj.toString))

Alternative is we keep the String and Object cases?
Would you mind give some comments? I can keep working on this :)

Test build #83183 has started for PR 19567 at commit f3d45d0.

Test build #83183 has finished for PR 19567 at commit f3d45d0.

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

Merged build finished. Test PASSed.

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

Test build #83190 has started for PR 19567 at commit 588902d.

LGTM

LGTM too

LGTM

Test build #83195 has started for PR 19567 at commit fae5c45.

Test build #83190 has finished for PR 19567 at commit 588902d.

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

Merged build finished. Test PASSed.

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

Test build #83195 has finished for PR 19567 at commit fae5c45.

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

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

Merged build finished. Test FAILed.

the last commit just change the test name and shouldn't break pyspark tests, I'm merging to master, thanks!

@jmchung can you send a new PR for 2.2? thanks!


fix: added meta tag viewport for GH Pages

After adding meta tag 'viewport' a layout change was required, so changes were made to the styles

:tada: This PR is included in version 3.0.0-beta.14 :tada:
The release is available on:

npm package (@beta dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Material UI Tables + React Virtualized

Current Behavior
Material tables aren't virtualized so big data tables render extremely slow.
Context
I would like Material UI tables to be virtualized by default or at least have the option to so we can render big data tables.
Right now there's not really a good option (well supported) to do this with React Virtualized and Material UI.

Dupe of #7450


I download an autorun sb application to the board and terminal stop response

I was trying to setup the VSP service and it worked (I can send a message from my phone to the board and it shows on the terminal)
Afterward, I tried to load another application, it all shows timeout.
What happened here, is there anything I can do? I tried ticking and unticking break and reset botton but none of them worked.

You probably have an autorun application loaded, if you untick DTR, then enable BREAK for 2 seconds and disable it, you will be back into interactive mode whereby you can clear the module using at&f* and load a new application


[Bug]: Problem with lightcurve fit error magnitudes (NIRSpec)

FAQ check

[X] Yes, I checked the FAQ and my question has not been addressed.

Instrument
NIRSpec (Stages 1-3)
What happened?
I’m currently using Eureka v0.5 (c.9/20/2022).  It is working, but I'm having some problems.
I've been trying to duplicate the WASP-39b transmission spectrum shown in Figure 2 of your (ERS-1366 Team) recent Nature paper (arXiv:2208.11692).
I’m starting with the full, JWST Stage 2 calints (4 segments, 21000 ints), and then running Eureka Stage 3 – 6.  It runs to completion and generates all of the various graphs and tables.
The problem is that I can get a spectrum that’s in the general ballpark, but it is significantly off.  I've attached the spectrum plot here.

The error bounds on the transit depth and other fit parameters are several orders of magnitude smaller than they should be.  Of course, given that, there are no y-error bars shown on the final spectrum plot.  The binned median values of the spectrum also don't really track to Figure 2.  For the LC fits I'm using dynesty with 1024 live pts.  I'm only considering 12 spectral channels to keep the test run time down.  I've constrained the extracted wavelength range to between 3.0 - 5.25 microns.
The S6 log file did show a WARNING: The selected meta.wave_max (5.25) is larger than the longest wavelength (5.206340312957764).  This might throw off the value in the last bin, but would it affect the others?
Any help would be appreciated.
Error traceback output
WARNING: The selected meta.wave_max (5.25) is larger than the longest wavelength (5.206340312957764).
What operating system are you using?
 Running Stage 2 yourself and setting skip_photom to be True might help alleviate some of those issues. The next thing I'd check is that you're properly fitting for the noise is Stage 5, so make sure you're fitting for something like scatter_mult which should be around 1 but a bit higher since 1 is the idealized limit. It's possible you're not fitting for that properly which is resulting in your fitted values being overly constrained by your explicit or implicit priors.
As for the wavelengths, you can increase the xwindow in Stage 3 to capture the longer wavelengths if you want. Another option is that you can change your Stage 4 wave_max to be something like 5.20 which is safely within the wavelength range you have to work with. Alternatively, leaving things as is shouldn't mess anything up too much, it's just that your last wavelength bin will be narrower than the others I believe.

Thank you very much for your response.
I made the following changes:

used actual JWST Stage 1 rateints data (4 segments, ~ 21000 ints total)
set the 'skip_photom' variable in S2 to True
changed S4ecf 'wave_max' to None
changed S5epf 'scatter_mult' to a median of 2.0 and sdev of 1.0
started Eureka from S2

The run started off smoothly.  It read the first segment rateints.fits file and started extracting the integration and applying the aperture correction.  This ran through the first segment of 6100 integrations.  Then, it generated a log file and the first x1dints.png, as well as the related calints and x1dints.fits files.  It did the same for the second segment, with another 6100 ints. Then it started the third segment and this is where things went off the rails.  Clearly the problem is in S2, and oddly somewhere during the seg 3 analysis.
I've attached a terminal dump file that hopefully will help to sort this out.  Again, I would appreciate any suggestions.
Terminal_dump_(20220930T0850).txt

Hmm, well basically the only time I've ever seen that "Killed" message is when your computer runs out of RAM. It's surprising that this would only happen on the third segment, but perhaps the STScI's Stage 2 code has a bit of a memory leak. NIRSpec is very much the instrument I'd expect this issue to be encountered on though as it's data file are very large. As the Stage 2 code isn't our own, there isn't much I can suggest other than trying to get access to a computer with more RAM or trying to break the Stage 1 output files into even smaller segments (while being sure to adjust all of the FITS header values and data extensions)

Maybe skip the extract_1d step

Thanks for the responses/suggestions.  I think that @taylorbell57 was exactly right about the RAM problem.  I monitored the RAM usage during a run, and it hit the ceiling just about the time that Eureka loaded the 3rd segment of rateints from JWST Stage 1.  Over the past week I've set myself up with a lot more compute power and memory.  I now have 28 vCPUs and 64 GB of RAM.  This appears to solve the memory problem.  I've made several runs and it works -- much faster too.
I'm now also skipping the extract_1d step in Stage 2 per @kevin218's suggestion.
I'm still having problems with the error bounds on the transit depth among other things.
I was originally using a 'scatter_mult' with a mean of 1.1 and sdev of 0.1.  This didn't work (extremely small error bounds), so I tried several other normal distributions with higher means, trying to keep the low end tail ~ above 1.0.  I also tried a uniform distribution with UL of 10 and LL of 1.  This worked -  sort of.  I got larger y-error bounds.  After some consideration, it seemed to me that a normal distribution would probably be more appropriate.  As long as I could keep the low end tail near 1.0 very small, and extend the top end tail, so I tried the case with a mean of 12 and sdev of 3.  This has a very small tail below 1.
This run ran to completion.  I've plotted the results along with the published ERS-1366 results (attached).  As you can see the mean value points in my run are consistent with the ERS-1366 results up to ~ 3.8 microns.
Issues: [1] At longer wavelengths (> 3.8 microns) my results for transit depth  appear to drift high by ~ 0.1% to 0.2%.
[2] In addition, my error bars across the spectrum are too large by about a factor of 7 or so. Finally,
[3] my longest wavelength channel center was 4.9755 microns.  I set 'wave_max' = None in the S4_ecf, so I was expecting the max wavelength center to be ~ 5.288 microns (as published).
Any suggestions that you might have regarding my issues would be appreciated.

The next step would probably be to apply group level background subtraction. Unfortunately, that pull request is still in the works. @erinmmay might be able to share a copy of her code.

I can push my Stage 1 files to my fork tomorrow for the GLBS and provide you the link to that code. @cdfortenbach, we can communicate over email for the new inputs as they're not yet documented. I did notice that you're using the rateints from MAST, so just an FYI that the GLBS will require you to run S1 yourself.
This may partially address the larger than expected uncertainties, but I don't think it can fully explain the 7x larger uncertainties. I think we'll need to see more inputs/outputs of various stages to work out the problem.

Thanks for your comments.  I thought that I would give things another try.  I downloaded the uncal files from MAST (4 segments) and set up the S1_ecf for the Stage 1 reduction.  I used the template in GitHub (for the S1_ecf) and just used the default parameters (with updated paths, etc.).  In the S4_ecf, I reduced the number of spec channels to 5 from 94 just to speed things up for a test. I also reset wave_max to 5.28.
Upon execution, the code appeared to load the 1st uncal segment and work its way through 6100 ints (the 1st segment, I think).  Then it seems to have lost track (see terminal dump attached).  I waited 45 minutes, but there was no longer any terminal activity.  It did not write the log for segment 1, and appeared to just be hung.  Any thoughts/suggestions on this would be most appreciated.
Terminal_dump_(20221010T1253).txt

I tried another Stage 1 start.  As before, I used the uncal files from MAST (4 segments) and set up the S1_ecf for the Stage 1 reduction. This time I changed the S1ecf 'ramp_fit_max_cores' from 'none' (not sure what that means) to 'half'.  The run started out more smoothly. It proceeded through Stage 1, generating the full set of outputs including the rateints files.  It also made it through Stage 2 and generated all of the calints files. It then seems to have gone partially through Stage 3 and failed.  I don't think it likes what it sees in the calints files. The only change in this run from my previous runs that went to completion is that now I'm using the JWST uncal files to start and the Eureka Stage 1 reduction to generate the rateints files.  I've attached a terminal dump that may give some clue about the failure.  Any thoughts would be appreciated.
Terminal_Dump_20221012T0929.txt

That's a new error message for me - not sure why the Stage 3 centroiding algorithm broke like that...
Thanks for letting us know what you had to do to get over that Stage 1 issue though - one of us devs should add that to our FAQ page as I'm pretty sure you're not the first one to have encountered that

It would appear that there may be some sort of flaw in the Eureka Stage 1 products (rateints) that causes this Stage 3 centroiding failure.  I decided to revert to using the JWST Stage 1 rateints and start Eureka at Stage 2.  I went for a full run of 94 spec channels again to match the ERS-1366 results (from 3.0 to 5.28 microns). Everything went well through Stage 4, and I believe I cured one of my issues.  Previously I had set wave_max to 'None' in the S4_ecf.  This had caused a strange cutoff of the Stage 4 results at 4.968 (?).  So, I reset wave_max to 5.28 and that seems to have solved the problem - at least for Stage 4;  it went through all 94 channels.  Now the run continued smoothly into Stage 5 and worked through 83 of 94 spec channels and then failed. I have attached the S4_log, the S5_log, and the terminal dump for the failure.  It appears to have been a dynesty problem. Seems like an odd place to fail.  Any thoughts would be appreciated.
S4_wasp39b.log
S5_wasp39b.log
Terminal_dump_20221013T1125.txt

I just noticed something that could help with diagnosing this problem.  The log shows that during S4 at Bandpass 82 the 'Identified outliers' jumped from ~ 1 to 21500 and stay that high through the end of the stage (Bandpass 93).  The S5 starts smoothly, but when it reaches the Bandpass/Channel 83 dynesty fails.  Seems like these events are related.  Any thoughts/suggestions would be appreciated.

Back on 10/8/22, I summarized a set of issues that I had encountered.
Issues: [1] At longer wavelengths (> 3.8 microns) my results appear to drift high by ~ 0.1% to 0.2%.
[2] In addition, my error bars across the spectrum are too large by about a factor of 5 to 10 or so. Finally,
[3] my longest wavelength channel center was ~ 4.976 microns.  I set 'wave_max' = None in the S4_ecf, so I was expecting the max wavelength to be ~ 5.294 microns (consistent with the values given in the NASA Exoplanet Archive).
I believe that I've solved one of the issues: item #3.  The strange failures that I was experiencing trying to push the long wavelength end of the analysis are the fault of not more carefully setting 'xwindow' in the S3_ecf.  The default shown in the example control files is [60,410].  Unfortunately this will trim both the low and high end of the spectrum and will not allow a clean reproduction of the published results.  After much trial and error I have reset 'xwindow' to [11,480], and it works. I get reasonable results out to ~ 5.294 microns.  I haven't tested things carefully at the short end, so I may need to move that limit back up a bit to avoid edge issues.  I guess this looks obvious in hindsight, but at the time it wasn't to me.
Regarding items #1 and #2, I've tried many different things, but so far no luck in improving the results. The 'scatter_mult' value seems to have a very strong influence on the error bounds (and probably the mean values for the data too).  The original setup of scatter_mult with mean=1.1, and sdev=1, eliminates the error bounds altogether on all of the fit parameters.  I opened up the scatter_mult prior by using a uniform distribution from 0.1 to 50.  This seems to work well.  The fit seems to work cleanly and the posterior peak for the scatter_mult is ~ 5 to 25. Now, all of the parameters have much more reasonable errors, except that the rp error is still high (by 5x to 10x). Again, any thoughts/suggestions would be appreciated.

To help avoid others encountering the first of your issues, perhaps someone can add onto that warning in Stage 4 to say to consider extending the xwindow in Stage 3 if the wave_min or wave_max are outside of the bounds of the array.
As for your other two issues, looking at Erin May's forked repository and her message above is likely the way to move forward. I believe each of the ERS teams made a fork of the Eureka repository to be able to make the rapid changes needed to handle their ERS data, and hopefully we'll see those changes get merged into the main branch soon, but for now those are all living in their respective forks.

Thanks very much for your quick response.  I'll take a look again at @erinmay's note.  It may just be time for me to take a pause and wait until 'those changes' are merged into the main branch.

Hi @cdfortenbach - I'm working on getting a variety of updates to S1 pushed to my fork here: https://github.com/erinmmay/Eureka and a PR will be opened soon. It's a combination of updates from a variety of people and includes options to run group level background subtraction (which helps make the ramp fits more accurate). You're welcome to start taking a look at my fork now and email me with questions<EMAIL_ADDRESS>while I work on documentation.
Some possible answers to other issues you've been having:
One possible reason your own S1 rateints files weren't working well is the jumpstep, for the data set you're working on there are only 5 groups and the 1/f noise results in jumps being flagged in the ramp that aren't real, so we recommend not running that step at the moment (I believe that should be in the methods of the paper you're trying to reproduce).
I've also had similar issue in the centroiding in S3 when trying to use flux calibrated data due to problems with the current flat field available for NIRSpec. I recommend not running the flat field or photom step in S2, and that may help.
For the dynesty failure, this happens to me when there are no good data points in the bin that's being fit, which is common on the edges of the detectors. I've added options to skip bad light curves but that's a longer lead time for getting on github.

Thanks very much @erinmay.  This is very helpful info.  I will study the details, and try the suggested action.  I may take a look at your fork and will email questions as you suggested.

Hi @cdfortenbach, as we've discussed this in person and over email and I was able to send you ECFs that were able to reasonably reproduce the ERS PRISM results, I am now going to close this issue.


Update docstrings to import dask.ml

Now that dask/dask#6384 is in updated user examples to import from dask.ml instead of dask_ml.
xref #687

@jacobtomlinson @jrbourbeau Are we to a point where this can be merged?

Yes, since https://github.com/dask/dask/pull/6384 was included in the dask=2021.11.0 release we should be good to go. However we'll need to update our minimum support dask version to 2021.11.0 here:
https://github.com/dask/dask-ml/blob/1e811ce48951a45a54859aca387510abdc085722/setup.py#L14


No tests

I originally built this tool for myself, since others are using it now it would be a good idea to add tests for existing commands and increase coverage.

Closed with:
9171d0753d7e75de8d6d2fa4338773f4954cee0b
94f78138b478b0b9c128cd93fddb29fb86ee70d6
508d1099500bb2ebcf85e621da6b48bc5bb7831c
71742c3169103f801f44eb75ce22b78af7290f79


addresses are not deleted when autoDeleteAddresses is true

Describe the bug
When configuring a broker to auto-delete expired addresses like this:
# ...
  addressSettings:
    addressSetting:
      - match: '/tmp/#'
        expiryDelay: 300000
        autoDeleteAddresses: true
        autoDeleteAddressesDelay: 30000
        autoDeleteQueues: true
        autoDeleteCreatedQueues: true
        autoDeleteQueuesDelay: 30000
        autoDeleteQueuesMessageCount: -1
# ...

it

correctly deletes automatically created, expired, queues
does not delete automatically created, expired, addresses (e.g. named '/tmp/abc/123')

Further information:
I use the broker as a relay for STOMP websocket messaging (spring boot) and define an acceptor like this:
  acceptors:
    - name: stomp
      port: 61613
      protocols: all

I also verified, that the addresses had queues / consumers / messages at least once.
When setting the artemis version (in broker yaml) to version: 2.26.0, it correctly deletes the queues as well as addresses.
Version info:

broker: activemq-artemis-broker-kubernetes@sha256:52f337cd5d7d5d199eebf4fc722f8c5279f652506f009efbc0f8f6de88e18718
artemiscloud: v1.2.1 (latest)

[!TIP]
Vote this issue reacting with :+1: or :-1:

I think this change in version 2.28.0 is the problem: https://github.com/apache/activemq-artemis/pull/4325/files

Since this is not directly related to artemiscloud, I have emailed<EMAIL_ADDRESS>detailing the problem.

@AlexW00 thanks for sharing your findings


invalid await (TypeScript Errors) when using detectAllFaces | drawDetections

Hi there,
great Library! I was following your Documentation here and got it working with only minor Setup-Problems. So there aren't any Runtime-Errors. But, TypeScript is complaining that Invalid 'await' of a non-Promise value. (await-promise)tslint(1)
and it's kind of confusing I must say.
The Error happens at const detections = await faceapi in the following Code-Block
Going by your Example like this:
public async detectFaces() {
  const tinyFaceOptions = new faceapi.TinyFaceDetectorOptions({
    scoreThreshold: 0.45,
  });

  const useTinyModel = true;

  const detections = await faceapi
    .detectAllFaces(this.videoElement, tinyFaceOptions)
    .withFaceLandmarks(useTinyModel)
    .withFaceDescriptors();

  const resizedDetections = faceapi.resizeResults(
    detections,
    this.videoDimensions,
  );

  // draw detections into the canvas
  faceapi.draw.drawDetections(
    this.faceApiVideoOverlay.nativeElement,
    resizedDetections,
  );
}

okay fair enough, none of the chained Methods seem to return any Kind of Promise (don't know for sure).
Removing the await however results in a different Error at resizedDetections right here:
// draw detections into the canvas
faceapi.draw.drawDetections(
  this.faceApiVideoOverlay.nativeElement,
  resizedDetections,
  // ^ right here
);

and says

with
const resizedDetections: faceapi.ComputeAllFaceDescriptorsTask<faceapi.WithFaceLandmarks<{
    detection: faceapi.FaceDetection;
}, faceapi.FaceLandmarks68>>
Argument of type 'ComputeAllFaceDescriptorsTask<WithFaceLandmarks<{ detection: FaceDetection; }, FaceLandmarks68>>' is not assignable to parameter of type 'FaceDetection | IRect | { detection: FaceDetection; } | IBoundingBox | TDrawDetectionsInput[]'.
  Type 'ComputeAllFaceDescriptorsTask<WithFaceLandmarks<{ detection: FaceDetection; }, FaceLandmarks68>>' is missing the following properties from type 'TDrawDetectionsInput[]': length, pop, push, concat, and 26 more.ts(2345)
face-api-cam.component.ts(143, 13): Did you forget to use 'await'?
Peek Problem
Quick Fix...

So using the Quickfix which is adding await to resizedResults results in the same Error as using it in detections.
Do you have any Ideas in how to avoid or fix this? I also could provide my ts-lint.json if necessary.
Thank you for your help.
Also: that's an Angular Project and the Code itself works fine, just TypeScript throws a fit.

Okay in addtion:
the Error about resizedDetections only appears directly in VSCode when using the TSLint-Plugin.
However when running tslint in a Commandline, everything is fine.
But in any case: the await isn't needed as it seems. The error displayed by VSC is still weird imo


WIP Divider implementation

Attributes to add:
showDivider
showDividerHorizontal
showDividerVertical
dividerDrawable
dividerDrawableHorizontal
dividerDrawableVertical

Still WIP.
TODO:

Add tests
Squash commits

Sent another PR to avoid resolve many conflicts in squashing commits.


docs: include mention of etcd configuration yaml in configuration.md

Configuration.md doesn't mention the etcd.conf file format or the order of precedent between flags, file, and environment variables.

@spzala
might want to pick it up?

Currently, if etcd is configured with config file, environmental values are ignored
https://github.com/coreos/etcd/blob/2f57a84ee6294e2d1e3dcda6ef2388bc582a9844/etcdmain/config.go#L247-L255

@xiang90 sure, sounds good, I will be working on it. Thanks!

WIP. I also noticed that some flags are in the Configuration.md are not in the etcd.conf (e.g. metrics, listen-metrics-urls etc) and vice-versa (e.g. log-output is missed in the Configuration.md). I will add them as part of the fix unless there was a specific reason. Also, I wonder if it makes sense to have flags ordered alphabetically (per section)? Thanks!

@spzala

Also, I wonder if it makes sense to have flags ordered alphabetically (per section)?

Some of them might be grouped again within the section. For example, we put all network related flags together again. But it seems that it is man's convention to sort them alphabetically. No strong opinion here.

@xiang90 thanks, makes sense, so let's not worry about that part under this issue. May be something we can talk about as part of future discussions.


alacritty config

there is no alacritty config

Hi @fl215,
It shouldn't be too hard to add it. All we need is to extend the build.lua with something similar to existing kitty output. PR is welcome. 👍


WiFi settings not stored after reboot

Every time i reboot/restart SmartEVSE it does not connect to the WiFi so i need to perform the procedure of starting AP and inputting WiFi details to it.
Why it doesn't store the connection details?

It does store the credentials.
Just to make sure, upgrade to latest version https://github.com/dingo35/SmartEVSE-3.5/releases/tag/v3.5.0-RC0
Goto Menu/Wifi/Setup and follow the exact instructions .
If your other settings arent stored either you might have a hardware problem...

You sure you set the Wifi option to Enabled?

That is strange, i kept it running for a while and perform the reboot again and now it works. Not sure what happened. Thank you anyway.


Add a default value to RevisionHistoryLimit if none is provided in the Certificate

Is your feature request related to a problem? Please describe.

I imagine many are upgrading cert-manager to 1.3.x mainly because of RevisionHistoryLimit feature. An awesome feature btw. However after upgrade any existing Certificates needs revisionHistoryLimit paramater on all Certificate resources in all clusters. This might not be a problem is smaller environments but it does introduce a lot of work in large deployments. To avoid this I suggest having a default value to RevisionHistoryLimit.
Describe the solution you'd like

If revisionHistoryLimit is not set then the value is defaulted to 25.
Describe alternatives you've considered

Add a flag to cert-manager-controller --default-revision-history-limit that would control the value to RevsionHistoryLimiti if none is provided in the Certificate resource
Additional context

Environment details (remove if not applicable):

Kubernetes version: 1.18.14
Cloud-provider/provisioner: Pivotal Kubernetes Service
cert-manager version: 1.3.1
Install method: e.g. helm/static manifests: Helm

/kind feature

This is also a very useful feature in my eyes. We also run a cluster for different project teams that contains thousands of certificates. With the number of certificate renewals, the number of resources increases to such an extent that Velero, for example, runs into timeouts during backup. Please include this in one of the next releases.
Thank you very much.

/remove-lifecycle rotten

This is something we would like added as well. As we are almost exclusively using ingress annotations to generate certificates we have a lot of old certificate requests in our clusters.
Is there anything I could help out with to get this moving?

+1 for this.  Our certificates are automatically generated from the Ingress resources as per https://cert-manager.io/docs/usage/ingress/, and I can't see anyway to configure the "RevisionHistoryLimit" on them to use this feature.  As a result, all the old CertificateRequests are just piling up...

+1 for this. Our certificates are automatically generated from the Ingress resources as per https://cert-manager.io/docs/usage/ingress/, and I can't see anyway to configure the "RevisionHistoryLimit" on them to use this feature. As a result, all the old CertificateRequests are just piling up...

Without a workaround to this, cert requests will pile up indefinitely, so this is still an issue.

/reopen

/reopen


Add TypeAlpine

apk is now in the purl spec: https://github.com/package-url/purl-spec/blob/master/PURL-TYPES.rst#apk

@spiffcs ran gofmt and verified tests pass locally. you can run the workflow again

It looks like the problem is downloading updated test data. make test has:
curl -L https://raw.githubusercontent.com/package-url/purl-spec/master/test-suite-data.json -o testdata/test-suite-data.json

Ironically, there was a PR that adds this test data file in, which was merged: https://github.com/package-url/packageurl-go/pull/32 but it still downloads the file when running make test.

@kzantow this should fix it https://github.com/anchore/packageurl-go/pull/10

@noqcks merged! I really would like to get our changes upstream, but this is good for moving forward quickly 👍

Added upstream PRs:

https://github.com/package-url/packageurl-go/pull/36
https://github.com/package-url/packageurl-go/pull/38

Main (master) branch is failing, too... I'm trying to sort through it.

https://github.com/anchore/packageurl-go/pull/11

Also merged 🎉

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
25 unchanged lines in 1 file lost coverage.
Overall coverage decreased (-10.8%) to 80.087%

Files with Coverage Reduction
New Missed Lines
%

packageurl.go
25
80.09%

Totals

Change from base Build<PHONE_NUMBER>:
-10.8%

Covered Lines:
185

Relevant Lines:
231

💛  - Coveralls


Reinstall VPS button

Finally I can report some progress in this matter.
Ubuntu 14.04 is almost finished and Debian 8 image is booting, need just few fixes to make it better.


depends_on with a dependency that is a cask errors as unsupported special dependency

brew config output
HOMEBREW_VERSION: 3.2.11
ORIGIN: https://github.com/Homebrew/brew
HEAD: 84f4fbf3cf5e83d33de0b28b2658fcb50aaa761e
Last commit: 5 days ago
Core tap ORIGIN: https://github.com/Homebrew/homebrew-core
Core tap HEAD: 00889339ce46b6ca17575145a908b0dc2928429a
Core tap last commit: 48 minutes ago
Core tap branch: master
HOMEBREW_PREFIX: /usr/local
HOMEBREW_CASK_OPTS: []
HOMEBREW_MAKE_JOBS: 8
Homebrew Ruby: 2.6.3 => /System/Library/Frameworks/Ruby.framework/Versions/2.6/usr/bin/ruby
CPU: octa-core 64-bit kabylake
Clang: 12.0.5 build 1205
Git: 2.33.0 => /usr/local/bin/git
Curl: 7.64.1 => /usr/bin/curl
macOS: 11.5.2-x86_64
CLT: <IP_ADDRESS>.1.1623191612
Xcode: 12.5.1

brew doctor output
Your system is ready to brew.

[X] I ran brew update and am still able to reproduce my issue.
[X] I have resolved all warnings from brew doctor and that did not fix my problem.

What were you trying to do (and why)?
Install a formula that has a depends_on: :multipass where multipass is a cask.
brew install 4d63/dockerhost/dockerhost

The formula is:
class Dockerhost < Formula
  desc "Setup a docker host on macOS in a lightweight Ubuntu vm."
  homepage "https://github.com/leighmcculloch/dockerhost"
  bottle :unneeded

  depends_on :multipass
  depends_on "docker"
  depends_on "jq"

  version "1.0.5"
  sha256 "8aad86aa7ca4b4fa599204fe51c7c3be23389c2b36a3d18df28b8377b5234228"

  url "https://raw.githubusercontent.com/leighmcculloch/dockerhost/v#{version}/dockerhost"

  def install
    bin.install "dockerhost"
  end

  test do
    system "#{bin}/dockerhost"
  end
end

What happened (include all command output)?
Brew complains that the :multipass is an unsupported special dependency.
% brew install 4d63/dockerhost/dockerhost
==> Tapping 4d63/dockerhost
Cloning into '/usr/local/Homebrew/Library/Taps/4d63/homebrew-dockerhost'...
remote: Enumerating objects: 46, done.
remote: Counting objects: 100% (46/46), done.
remote: Compressing objects: 100% (21/21), done.
remote: Total 46 (delta 18), reused 41 (delta 13), pack-reused 0
Receiving objects: 100% (46/46), 4.86 KiB | 4.86 MiB/s, done.
Resolving deltas: 100% (18/18), done.
Error: Invalid formula: /usr/local/Homebrew/Library/Taps/4d63/homebrew-dockerhost/Formula/dockerhost.rb
dockerhost: Unsupported special dependency :multipass
Error: Cannot tap 4d63/dockerhost: invalid syntax in tap!

What did you expect to happen?
I expected the multipass cask to be installed.
According to the Brew Formula Cookbook specifying a symbol (:multipass) instead of a string ("multipass") is appropriate if the dependency is a cask.
https://github.com/Homebrew/brew/blob/8db3289f15f08eea5406653141e11a911a2cd24d/docs/Formula-Cookbook.md#L141-L156
Step-by-step reproduction instructions (by running brew commands)
brew install 4d63/dockerhost/dockerhost

:multipass isn't a valid Requirement. A formula cannot simply depend on a cask. Refactor your formula to be a cask and then you can depend on other casks + formulae.
List of available Requirements in code:
https://github.com/Homebrew/brew/blob/master/Library/Homebrew/requirements.rb

To elaborate a bit on the above:
brew parses Symbol arguments to depends_on as a Requirement, where a Requirement specifies that the formula requires "one or more formulae, casks or other system-wide installed software (e.g. Xcode)".
Homebrew has some pre-specified Requirements (see link above), but you can also create your own. In particular, Homebrew does not provide Requirements for arbitrary casks.

Got it, thanks for explaining how it works. Changing my formula to a cask worked. Thank you.

you simply misunderstood the docs

👍🏻 I think the docs could be improved to clarify that when it says that symbols can be used for casks, it means a limited set of casks and not all or any casks.


Leaking goroutines

As https://github.com/cesanta/docker_auth/issues/391 noticed - docker_auth is leaking memory by leaking goroutines.
Looking at the heap profile runtime.malg is using a lot of memory, this is because of a goroutine leak:
$ go tool pprof 2024-09-07T16:13:49+1000_goroutine.out
File: auth_server
Build ID: f0023e9b8e89438ff941b55f3266cd69dfff1f88
Type: goroutine
Time: Sep 7, 2024 at 4:13pm (AEST)
Entering interactive mode (type "help" for commands, "o" for options)
(pprof) top5
Showing nodes accounting for 10539, 100% of 10542 total
Dropped 45 nodes (cum <= 52)
Showing top 5 nodes out of 9
      flat  flat%   sum%        cum   cum%
     10539   100%   100%      10539   100%  runtime.gopark
         0     0%   100%       4889 46.38%  github.com/schwarmco/go-cartesian-product.Iter.func1
         0     0%   100%       5638 53.48%  github.com/schwarmco/go-cartesian-product.iterate
         0     0%   100%       5638 53.48%  runtime.chansend
         0     0%   100%       5638 53.48%  runtime.chansend1
(pprof)

I also created an issue against https://github.com/schwarmco/go-cartesian-product/issues/6 with the potential solution, but posting it here for visibility as well:
diff --git a/auth_server/authz/acl.go b/auth_server/authz/acl.go
index 2f2e824..b0aa21c 100644
--- a/auth_server/authz/acl.go
+++ b/auth_server/authz/acl.go
@@ -1,6 +1,7 @@
 package authz

 import (
+	"context"
 	"encoding/json"
 	"fmt"
 	"net"
@@ -11,7 +12,6 @@ import (
 	"strings"

 	"github.com/cesanta/glog"
-	"github.com/schwarmco/go-cartesian-product"

 	"github.com/cesanta/docker_auth/auth_server/api"
 )
@@ -180,14 +180,17 @@ func matchStringWithLabelPermutations(pp *string, s string, vars []string, label
 			}
 		}
 		if len(labelSets) > 0 {
-			for permuation := range cartesian.Iter(labelSets...) {
+			ctx, cancel := context.WithCancel(context.Background())
+			defer cancel()
+
+			for permuation := range IterWithContext(ctx, labelSets...) {
 				var labelVars []string
 				for _, val := range permuation {
 					labelVars = append(labelVars, val.([]string)...)
 				}
 				matched = matchString(pp, s, append(vars, labelVars...))
 				if matched {
-					break
+					return matched
 				}
 			}
 		}
@@ -195,6 +198,45 @@ func matchStringWithLabelPermutations(pp *string, s string, vars []string, label
 	return matched
 }

+func IterWithContext(ctx context.Context, params ...[]interface{}) <-chan []interface{} {
+	c := make(chan []interface{})
+
+	if len(params) == 0 {
+		close(c)
+		return c
+	}
+
+	go func() {
+		defer close(c) // Ensure the channel is closed when the goroutine exits
+
+		iterate(ctx, c, params[0], []interface{}{}, params[1:]...)
+	}()
+
+	return c
+}
+
+func iterate(ctx context.Context, channel chan []interface{}, topLevel, result []interface{}, needUnpacking ...[]interface{}) {
+	if len(needUnpacking) == 0 {
+		for _, p := range topLevel {
+			select {
+			case <-ctx.Done():
+				return // Exit if the context is canceled
+			case channel <- append(append([]interface{}{}, result...), p):
+			}
+		}
+		return
+	}
+
+	for _, p := range topLevel {
+		select {
+		case <-ctx.Done():
+			return // Exit if the context is canceled
+		default:
+			iterate(ctx, channel, needUnpacking[0], append(result, p), needUnpacking[1:]...)
+		}
+	}
+}
+
 func matchIP(ipp *string, ip net.IP) bool {
 	if ipp == nil {
 		return true

I think the function is small and simple enough you can copy it into your code and not rely on a lib. Let me know if you want me to raise a PR with the above changes.
Thanks!

We're interested in fixing it.


update to nats 0.9.4

Nats 0.9.4 has been released with the announce bug fixes, update to this version


Census communal increase test coverage of downstream data

What is the context of this PR?
Dumps the answers that are going to be submitted and compares with the answers that were entered for Census_Communal
How to review
Run the integration test.

Is it worth having different answers for expected_downstream_data
Or is there no need?

@dcdarrell9 we are only verifying submission 'data' for census schemas.


Is it possible to load and show existing log with this ?

Title

Try this.
 data R_LOG type ref to ZCL_LOGGER.

  r_log ?= zcl_logger_factory=>open_log(
    object = object
    subobject = subobject
    desc = desc
    create_if_does_not_exist = create_if_does_not_exist
    settings = zcl_logger_factory=>create_settings(
      )->set_autosave( auto_save )
  ).

Try this.
 data R_LOG type ref to ZCL_LOGGER.

  r_log ?= zcl_logger_factory=>open_log(
    object = object
    subobject = subobject
    desc = desc
    create_if_does_not_exist = create_if_does_not_exist
    settings = zcl_logger_factory=>create_settings(
      )->set_autosave( auto_save )
  ).

I suppose your question has been answered. Please do not hesitate to reopen is issue if need be.


IPv6 & IPv4 use same port crash.

Describe the bug
use IPv6 [::] and IPv4 <IP_ADDRESS> and use same port(eg.21000) crash in my raspberry pi4.
To Reproduce
use salvo::prelude::*;

#[tokio::main]
async fn main() {
    let router = Router::with_path("<**path>").get(cached_proxy);
    tokio::spawn(async {
        let router = Router::with_path("<**path>").get(cached_proxy);
        let acceptor = TcpListener::new(String::from("[::]:21000")).bind().await; Server::new(acceptor).serve(router).await;
    });
    let acceptor = TcpListener::new(String::from("<IP_ADDRESS>:21000")).bind().await; Server::new(acceptor).serve(router).await;
}
#[handler]
async fn cached_proxy(_req: &mut Request, _res: &mut Response) { }

Expected behavior
support tcpv4 and tcpv6 at same time.
Screenshots
output
pi@raspbian:~/workspace/rust/cached_api$ cargo r -r
   Compiling cached_api v0.1.0 (/home/pi/workspace/rust/cached_api)
    Finished release [optimized] target(s) in 34.11s
     Running `target/release/cached_api`
thread 'tokio-runtime-worker' panicked at /home/pi/.cargo/registry/src/github.com-1ecc6299db9ec823/salvo_core-0.63.1/src/conn/mod.rs:201:31:
called `Result::unwrap()` on an `Err` value: Io(Os { code: 98, kind: AddrInUse, message: "Address already in use" })
note: run with `RUST_BACKTRACE=1` environment variable to display a backtrace
^C

Desktop (please complete the following information):

OS: Win10
Browser ...
Version salvo 0.61 & 0.63.1 rustc 1.73.

additional
It's OK in Windows 10.
It's OK use [::1] and <IP_ADDRESS> when using same port in raspberry pi4.
It's OK use [::] alone.
Backtrace
pi@raspbian:~/workspace/rust/cached_api$ RUST_BACKTRACE=full cargo r -r
    Finished release [optimized] target(s) in 0.73s
     Running `target/release/cached_api`
thread 'tokio-runtime-worker' panicked at /home/pi/.cargo/registry/src/github.com-1ecc6299db9ec823/salvo_core-0.63.1/src/conn/mod.rs:201:31:
called `Result::unwrap()` on an `Err` value: Io(Os { code: 98, kind: AddrInUse, message: "Address already in use" })
stack backtrace:
   0:     0xaaaae111a6ec - std::backtrace_rs::backtrace::libunwind::trace::heaab0e590535aeb3
                               at /rustc/cc66ad468955717ab92600c770da8c1601a4ff33/library/std/src/../../backtrace/src/backtrace/libunwind.rs:93:5
   1:     0xaaaae111a6ec - std::backtrace_rs::backtrace::trace_unsynchronized::h89cc7ae9ebb707d7
                               at /rustc/cc66ad468955717ab92600c770da8c1601a4ff33/library/std/src/../../backtrace/src/backtrace/mod.rs:66:5
   2:     0xaaaae111a6ec - std::sys_common::backtrace::_print_fmt::h08c31be18fedf422
                               at /rustc/cc66ad468955717ab92600c770da8c1601a4ff33/library/std/src/sys_common/backtrace.rs:67:5
   3:     0xaaaae111a6ec - <std::sys_common::backtrace::_print::DisplayBacktrace as core::fmt::Display>::fmt::hc38bcf44d9e857e3
                               at /rustc/cc66ad468955717ab92600c770da8c1601a4ff33/library/std/src/sys_common/backtrace.rs:44:22
   4:     0xaaaae113b27c - core::fmt::rt::Argument::fmt::ha5b752f9cd7ef4a3
                               at /rustc/cc66ad468955717ab92600c770da8c1601a4ff33/library/core/src/fmt/rt.rs:138:9
   5:     0xaaaae113b27c - core::fmt::write::h9fac187ae7486f3c
                               at /rustc/cc66ad468955717ab92600c770da8c1601a4ff33/library/core/src/fmt/mod.rs:1094:21
   6:     0xaaaae11178dc - std::io::Write::write_fmt::h239e9fb6296b3a7f
                               at /rustc/cc66ad468955717ab92600c770da8c1601a4ff33/library/std/src/io/mod.rs:1714:15
   7:     0xaaaae111a520 - std::sys_common::backtrace::_print::h52f67cfa8753b0ab
                               at /rustc/cc66ad468955717ab92600c770da8c1601a4ff33/library/std/src/sys_common/backtrace.rs:47:5
   8:     0xaaaae111a520 - std::sys_common::backtrace::print::hdea7481e2c957a93
                               at /rustc/cc66ad468955717ab92600c770da8c1601a4ff33/library/std/src/sys_common/backtrace.rs:34:9
   9:     0xaaaae111bba4 - std::panicking::default_hook::{{closure}}::h7c36fa733369c49e
                               at /rustc/cc66ad468955717ab92600c770da8c1601a4ff33/library/std/src/panicking.rs:270:22
  10:     0xaaaae111b8cc - std::panicking::default_hook::h303eee75f9a8f6a8
                               at /rustc/cc66ad468955717ab92600c770da8c1601a4ff33/library/std/src/panicking.rs:290:9
  11:     0xaaaae111c168 - std::panicking::rust_panic_with_hook::h270c94381ec34744
                               at /rustc/cc66ad468955717ab92600c770da8c1601a4ff33/library/std/src/panicking.rs:707:13
  12:     0xaaaae111c030 - std::panicking::begin_panic_handler::{{closure}}::h3653e3502bcc1625
                               at /rustc/cc66ad468955717ab92600c770da8c1601a4ff33/library/std/src/panicking.rs:599:13
  13:     0xaaaae111abd0 - std::sys_common::backtrace::__rust_end_short_backtrace::h6b8510f2f024eeeb
                               at /rustc/cc66ad468955717ab92600c770da8c1601a4ff33/library/std/src/sys_common/backtrace.rs:170:18
  14:     0xaaaae111bd80 - rust_begin_unwind
                               at /rustc/cc66ad468955717ab92600c770da8c1601a4ff33/library/std/src/panicking.rs:595:5
  15:     0xaaaae0f2c680 - core::panicking::panic_fmt::ha96945d7a1b20293
                               at /rustc/cc66ad468955717ab92600c770da8c1601a4ff33/library/core/src/panicking.rs:67:14
  16:     0xaaaae0f2c968 - core::result::unwrap_failed::h52241e4ea1d20f58
                               at /rustc/cc66ad468955717ab92600c770da8c1601a4ff33/library/core/src/result.rs:1652:5
  17:     0xaaaae0f52bf8 - salvo_core::conn::Listener::bind::{{closure}}::h89a49ff811a8e5e2
  18:     0xaaaae0f36388 - cached_api::main::{{closure}}::{{closure}}::h98b1ee5eea1fab85
  19:     0xaaaae0f34f14 - tokio::runtime::task::core::Core<T,S>::poll::hba8c5b3c48cf2b94
  20:     0xaaaae0f2f4f0 - tokio::runtime::task::harness::Harness<T,S>::poll::h46d47dc1c4f2fd1b
  21:     0xaaaae10d2668 - tokio::runtime::scheduler::multi_thread::worker::Context::run_task::ha220636d44fac3be
  22:     0xaaaae10d0030 - tokio::runtime::context::scoped::Scoped<T>::set::h1ce4a5bf264f0863
  23:     0xaaaae10d2320 - tokio::runtime::scheduler::multi_thread::worker::run::h5a57c31435d58fe0
  24:     0xaaaae10e3adc - tokio::runtime::task::core::Core<T,S>::poll::h871775fbbe59196e
  25:     0xaaaae10cb0cc - tokio::runtime::task::harness::Harness<T,S>::poll::hb3a873bee79e751c
  26:     0xaaaae10edaac - tokio::runtime::blocking::pool::Inner::run::h8403d5d4d76e7800
  27:     0xaaaae10e4760 - std::sys_common::backtrace::__rust_begin_short_backtrace::h94edcef3c32dab02
  28:     0xaaaae10e8448 - core::ops::function::FnOnce::call_once{{vtable.shim}}::hc940c9bdbc895459
  29:     0xaaaae111e970 - <alloc::boxed::Box<F,A> as core::ops::function::FnOnce<Args>>::call_once::h01303c6de09f5469
                               at /rustc/cc66ad468955717ab92600c770da8c1601a4ff33/library/alloc/src/boxed.rs:2007:9
  30:     0xaaaae111e970 - <alloc::boxed::Box<F,A> as core::ops::function::FnOnce<Args>>::call_once::h4f39e7995f973f4c
                               at /rustc/cc66ad468955717ab92600c770da8c1601a4ff33/library/alloc/src/boxed.rs:2007:9
  31:     0xaaaae111e970 - std::sys::unix::thread::Thread::new::thread_start::h9f792bc0c268271e
                               at /rustc/cc66ad468955717ab92600c770da8c1601a4ff33/library/std/src/sys/unix/thread.rs:108:17
  32:     0xffff950307e4 - start_thread
  33:     0xffff94eb559c - <unknown>
  34:                0x0 - <unknown>
^C

Only reproduced on raspberry pi4? I don't have device to test.

I run it in another raspberry pi3b+ with lastest raspbian os & ubuntu server instead of 3rd os in pi4.
The result is same. I don't know if it's a common problem in arm64 platforms, but I have no more arm64 devices for testing it.

I check it with tokio & std, same result. Maybe it's a spec problem of raspberry pi.


Update to 3.5.7 and include "--bind_ip_all" in our default command

Fixes #180

Just for future reference, if --bind_ip and --bind_ip_all are both specified, --bind_ip_all takes precedence.


Can't update a component on dashboard

Hi,
On the dashboard, i see all our components with the radio boxes behind it. If i change one, i get an popup saying "Component updated". But after a refresh it's back to the old value! Bug?

All fixed! Thanks :)


Can the Jot View be used without having to use JotViewStateProxy?

I don't want to be able to save my Jot View state to disk, but I couldn't seem to get the Jot View to work without implementing the JotViewStateProxyDelegate and loading in an initial state. Is it possible to make the Jot View work without the state proxy delegate?

It needs a JotViewStateProxy to load properly. It'd be a nice improvement to have it load a default JotViewStateProxy that just points to tmp paths. If you want to code something like that up I'll merge it in.

I'm the opposite - I only do Obj-C and haven't touched Swift.
If you're able to convert / create a sample Swift project that includes the JotUI framework, I would hugely appreciated it (and I'm sure many others would as well!)


Choosing a LICENSE

also hi :wave:

Hi! Adding MIT with #5 to start.

I chose a BSD for my class in the Fall but I'm not completely convinced that was any better. I think the third clause is interesting though: "Neither the name of the copyright holder nor the names of its
contributors may be used to endorse or promote products derived from
this software without specific prior written permission."
https://github.com/dphiffer/data-and-social-justice/blob/master/LICENSE

I'll leave this issue open for class discussion!

Reference: more discussion about the two licenses

couple more links that reminded me of this issue:
https://github.com/lerna/lerna/pull/1616
https://redislabs.com/community/licenses/

sure, yeah I'd be happy to!


Configs unclear and not working...

I am trying to replace minecraft:lime_glazed_terracotta with industrialforegoing:ether_gas
I have tried all of the following syntaxes and they all crash the game on load.
[<minecraft:lime_glazed_terracotta>=<industrialforegoing:ether_gas>]

[<item:minecraft:lime_glazed_terracotta>=<fluid:industrialforegoing:ether_gas>]

["minecraft:lime_glazed_terracotta"="industrialforegoing:ether_gas"]

It should be:
["minecraft:lime_glazed_terracotta=industrialforegoing:ether_gas"]


[Question] Docstring for Experiment class

Isn't there a docstring for the Experiment class?

Can you explain the parameters of the class to me? It would be very helpful.

Hopefully, below explanation will suffice for a docstring. I'll create a PR.
Docstring for Experiment class

"""
    This class represents an A/B experiment. It provides several methods to
    analyze and visualize the results of the experiment. The experiment is
    based on data provided as a DataFrame, with information about the numerator
    and denominator of the tested metrics and their grouping columns.

    Attributes:
        data_frame (DataFrame): DataFrame that contains the experimental data.
        numerator_column (str): Name of the DataFrame column that contains the numerator of the tested metric.
        numerator_sum_squares_column (Union[str, None]): Name of the DataFrame column that contains the sum of squares of the numerator.
        denominator_column (str): Name of the DataFrame column that contains the denominator of the tested metric.
        categorical_group_columns (Union[str, Iterable]): Column(s) that categorically group the data.
        ordinal_group_column (Union[str, None]): Column that ordinally group the data.
        interval_size (float): Size of the confidence interval. Defaults to 0.95.
        correction_method (str): Method for correction. Defaults to BONFERRONI.
        confidence_computer (ConfidenceComputerABC): ConfidenceComputerABC object to compute confidence intervals.
        confidence_grapher (ConfidenceGrapherABC): ConfidenceGrapherABC object to plot confidence intervals.
        method_column (str): Column that contains the experimental method.
        bootstrap_samples_column (str): Column that contains the bootstrap samples.
        metric_column (str): Column that contains the metric of interest.
        treatment_column (str): Column that contains the treatment groups.
        power (float): Desired statistical power. Defaults to 0.8.
        feature_column (str): Column that contains the features.
        feature_sum_squares_column (str): Column that contains the sum of squares of the features.
        feature_cross_sum_column (str): Column that contains the cross product sum of the features.
    """


Can not see memcached server

I can not find place to config site to memcached. Can you please instruct me the way to SEE memcached works on the localhost?

@crabvn
In this example, I use the official memcached repository in dockerhub.
Normally it is unnecessary for development purposes, but if you want to write the configuration file please call the following URL.
https://hub.docker.com/_/memcached
For information on how to call memcached from PHP, refer to the following commit.
https://github.com/koni/docker-php-nginx-mysql-memcached/commit/19a851fd819388a7f6282a9e7e0f090418ef77a9

@koni
I am able to connect to memcached from PHP after a few day researching :). Thing need for now is connect from Nginx.
Anyways, thanks for your support bro.


400 Error when running action

Hi,
Thanks for releasing this github action.
I am attempting to use it in my project and I get a 400 error every time it runs.
Error: AxiosError: Request failed with status code 400
I feel like either my webhook is not setup correctly or something else?
I setup the WebHook as a "Connection" on a MS Teams Channel via the GitHub Enterprise app.
Thanks!

I probably spoke too soon, 😓 It seems there are 2 ways to create a webhook, I just found the second method so I will try that when I can and revert.

Hello,
I have never used this in conjunction with GH Enterprise app in Teams and would have to point you to their documentation for any issues interfacing with that.
This particular action is expecting an MS Teams "Incoming Webhook" created from a channel's "Connectors" options.
I would try creating one of those and see if you have better luck.
It's possible that a feature to support the GH app webhooks could be added in the future but at this time the Incoming Webhook is what you need.

@mikesprague It's a bad habit I have sometimes, I speak too soon! Thanks for the quick response!

@frankrooneyjci All good... def been there and done that. Hope that info gets you moving. Here's a link to the specific docs as well: https://learn.microsoft.com/en-us/microsoftteams/platform/webhooks-and-connectors/how-to/add-incoming-webhook?tabs=dotnet
I'm going to close this but feel free to reopen or create a new issue if you have further problems.

Thats fine with me!


Repository links in README.md should point to Hyperledger urls

Is this a regression?
Yes
Description
All the repository links in README.md still point to iohk basURL and should now point to Hyperledger (https://github.com/hyperledger/*)
Please provide the exception or error you saw
all repository URLs in README.md point to https://github.com/input-output-hk/*

Please provide the environment you discovered this bug in
https://github.com/hyperledger/identus-edge-agent-sdk-kmp/

Anything else?
Creating an issue so someone (me?) can make a PR

Thank you, @coveloper, for pointing this out. I will update the links.


Adding EF 2.0 results in Exception in VS2017 React/Redux template web app

'System.Collections.Generic.Dictionary`2<System.String,System.Object> Microsoft.Extensions.Configuration.IConfigurationBuilder.get_Properties()'.
Stack Trace:
at Microsoft.Extensions.Configuration.FileConfigurationExtensions.SetFileProvider(IConfigurationBuilder builder, IFileProvider fileProvider)
at Microsoft.Extensions.Configuration.FileConfigurationExtensions.SetBasePath(IConfigurationBuilder builder, String basePath)
at Microsoft.AspNetCore.Hosting.WebHostBuilder.BuildCommonServices(AggregateException& hostingStartupErrors)
at Microsoft.AspNetCore.Hosting.WebHostBuilder.Build()
at reactredux.Program.BuildWebHost(String[] args) in C:\Users\smills\webstuff\reactredux\Program.cs:line 19
at reactredux.Program.Main(String[] args) in C:\Users\smills\webstuff\reactredux\Program.cs:line 15

Was using VS2107 preview.  Retesting with Release - problem not encountered at this point.


As a user, I'd like to have OCRs automatically created for Contacts based on relationship defined through the realtionship object so I can manage board referrals, etc.

"Is there a way for contact relationships to automatically translate into an opportunity contact role?
Our Board has a minimum give/get requirement that we have to track. Our Board members provide us with their solicitation lists, which we have uploaded into relationships in Salesforce with the Board member as "referrer" and their prospect as "referee." For every single time the prospect gives, we would like this relationship to automatically translate into a "referrer" contact role in the donation opportunity, so that we can then attribute soft credit. This would make it significantly easier to track their give/get requirement."
https://powerofus.force.com/0D58000001OrXDY

**lurch: add

Tracking W-008274

Also this Idea here: https://powerofus.force.com/hub-ideas/1040

Included in beta release 3.131 (Beta 11)

Included in production release 3.131


Add a blurb to the homepage about the raylib-game-template

This adds a short blurb as well as a prominent link to the raylib-game-template repo to encourage new users to try that out.  The link is in a style similar to other icons on the page with a 6px #888888 border that changes to #111111 on hover.

@michaelfiber Added!


Allow returning details from find/walk/glob

This is a slightly different take on #116

Closes #116
Resolves #214

Are any plans in releasing fsspec? The last release has been almost three months ago. It would be great to be able to use these improvements

Agreed, I'll put it on my list.

Thanks!


Tips for downgrading this to an older version of tree-sitter

Hi! I'm interested in forking this library and rebuilding it for use in an Atom grammar. I tried doing this with the library as is, but because it the language version is 13
https://github.com/stsewd/tree-sitter-comment/blob/674c5cbb948b596c3ed664b510dde76e1f21aee3/src/parser.c#L8
it is incompatible with the current versions of Atom (and Atom might not catch up any time soon).
 Do you think it's a simple matter of just regenerating the C code with an older version of tree-sitter-cli, but with the same grammar you have here?

Do you think it's a simple matter of just regenerating the C code with an older version of tree-sitter-cli, but with the same grammar you have here?

Yeah, you can re-generate the grammar, it should just work. You just need to download the ts cli version that you want and call tree-sitter-cli generate, you can also use this commit https://github.com/stsewd/tree-sitter-comment/tree/6b0dbd56284023617c3ea755624e55d0254f813a (that's using 0.17.3). There aren't any differences in the grammar from that commit vs master, so you can use it without problems.

Thanks! Installing from '6b0dbd5' was a quick fix.
I was wondering if you might know how to work around what seems to be a common limitation with other language grammars: that comments are parsed as single block, which makes language injection of this grammar hard/impossible.
For example, I'm trying to use the tree-sitter-vue parser to update the Atom vue grammar. But it treats the entire comment block as a comment, so I can't just feed it into your parser. The <!-- etc. isn't parsable by tree-sitter-comment.

Based on other parsers I've looked at (JS, Python, etc.) That seems to be a common pattern. I think to use your parser, you would need to actually parse the contents of the comment block as its own node (e.g. comment_line, or comment_content), unless there's another way to do this.
I've tried simply removing the parts that are not parsable by ts-comment  before passing it on, but I haven't had success with that yet...

you mean having <!-- being a different node and the comment just TODO? then you can pass the whole thing without problems <!-- TODO: something --> that would be parsed as ("text" tag (name) ":" "text") or (tag (name))

Let me know if there is anything actionable from my side.


Async events to event loop thread

@bwilless Async provides a mechanism to marshal an event from a thread to the main thread. This is exists primarily to support calling event loop functions from a thread, though there are other use cases.
Event loop functions (for example set a oneshot timer) must be called from the same thread the event loop is running on, by default the main thread. Failure to so will result in bad undefined things eventually happening.

The API is modeled after libuv async


Update autorest version to prerelease 3.0.0-alpha.20240905.2

Generated by Autorest Regen Preview build 20240905.2Triggered from pull request: https://github.com/Azure/autorest.csharp/pull/5043

API change check
API changes are not detected in this pull request.


Cannot install on staging for testing

Description
All flowforge scoped packages on staging get installed from GitHub package manager.
This package needs to add the same mechanism that other packages (e.g. launcher) to make it available to the staging environment,
@hardillb could you direct us on this one pls?
Epic/Story
No response
Have you provided an initial effort estimate for this issue?
I have provided an initial effort estimate

@Steve-Mcl I believe this has been resolved now with the devops changes made by @ppawlowski?


Cancel streaming insert?

I've tested https://github.com/ClickHouse/ch-go#stream-data and I see that if I terminate the program in the middle of the block loop, data written so far appear server side.
Is there a way to change this? Maybe a server setting to cancel query if connection is broken?
Thanks,

I think that it is server-side and we can't change it.

To cancel streaming insert, use io.EOF, but data that was already sent can be persisted by server.
Probably this can be changed by some kind of transactions, but I'm not aware of status of ACID in ClickHouse.


SHA256 checksum error for tcllib

See https://github.com/tcltk/tcllib/issues/5
Can you please update the checksum in the file https://github.com/Homebrew/homebrew-dupes/blob/master/tcl-tk.rb (line 34)?
Thanks!

Done, thanks for following up on that!


DO NOT MERGE! Fast Forward stable to b5bb32226c91905ed71aa630bb4c74c788005fbb.

This PR will be merged automatically once checks are successful.

Jenkins job auth/postsubmit passed


Critical Bug: Bot doesn't work at all!

Due to changes to the UI the bot won't work. Please fix Deku <3

what is the issue


Co-dependency on containers

This is not yet a bug, more like a request for clarification.
I am working with microservices, and it would be nice to be able to spin up the various containers for the final integration tests.
One issue I'm having a hard time understanding is how to make one container wait for another container, but I see this is already a separate issue so that I won't go too deep into that.
The other issue I have is I fail to understand how to avoid hard-coding my computer's IP for one container to see another i..e I am running one container that needs to see a MongoDB. They both start nicely, but the mongo container is invisible to the others unless I hard code my local computer's IP.
I've tried a whole range of combinations of .WithNetwork() and .WithNetworkAlias(), but I cannot seem to get them to see each other.
Some clarification in the readme should help me a lot :)
Other than that, this is a fantastic package!

I collect best practices in #438, but that does not cover Docker networks yet 😅. I can add another example soon as we figured out a solution for you.

The other issue I have is I fail to understand how to avoid hard-coding my computer's IP for one container to see another i..e I am running one container that needs to see a MongoDB.

I think this is covered by a test. You can do something like this. The test assigns a network to both containers, then they will see each other through the hostname (WithHostname). Which is not the Hostname property. OC, that's misleading and needs to be improved. I just noticed it now...

One issue I'm having a hard time understanding is how to make one container wait for another container

Something like this can be done with a custom wait strategy. Pass your own strategy to WithWaitStrategy.

@digitaldias Do you need any further help or can we close this issue?

I'll close this issue. Don't hesitate to reopen it again if you have any further questions or problems.


folder name vs table name

i would find it a very conveniant way to name my folders like my tables.
so i could call it db.myTable.myQuery
the problem is : folder can't be called like table. if a folder is named after a table name, function like save aren't accessible. is it the expected behaviour ?

Yes, that's by design. There's only one db namespace and tables are loaded in before folders/scripts, so if there are naming collisions the latter will win out. If that's how you prefer to organize your scripts, the easiest thing to do is to prefix your folder names -- so db.tablename refers to the table itself, and db._tablename or some such is a collection of scripts that operate on the table.


Include github.com in the trusted image domains list

Github Actions seem to have their Badges stored on github.com directly.
For https://github.com/conesoft/files it generated it on https://github.com/conesoft/files/actions/workflows/publish.yml/badge.svg, which doesn't render on README.md
Describe the solution you'd like
Add github.com to the trusted image domain list.

This looks like a duplicate of https://github.com/NuGet/NuGetGallery/issues/8645

Yes indeed, @augustoproiete -- thanks for connecting the dots. @davepermen, keep an eye on issue #8645 and PR #8646 for this fix.
I appreciate you including an example URL. I tested it against the new regex in the aforementioned PR and it matches:

I'll close this issue as a duplicate.


MobilenetV2 config file. Can you share the mobilenetv2 training config.

Which layer should I train in backbone net？
In pysot/core/config，only resnet50 backbone train layer provided. For Mobilenetv2, which layer should I train? Thank you.

It mainly depends on how you modified the backbone

Have you tried to train with mobilenet? After merging with pysot/core/config.py and changing the training layer of backbone, I got poor results

Have you tried to train with mobilenet? After merging with pysot/core/config.py and changing the training layer of backbone, I got poor results

Yes，I trained it. I got a not bad results. I use the defualt config in siamrpn_mobilev2_l234_dwxcorr

Have you tried to train with mobilenet? After merging with pysot/core/config.py and changing the training layer of backbone, I got poor results

Yes，I trained it. I got a not bad results. I use the defualt config in siamrpn_mobilev2_l234_dwxcorr

where you get the backbone of mobilenet, I download from website, however, it seems mismatching the model used in SiamRPN++. Thank you

Have you tried to train with mobilenet? After merging with pysot/core/config.py and changing the training layer of backbone, I got poor results

Yes，I trained it. I got a not bad results. I use the defualt config in siamrpn_mobilev2_l234_dwxcorr

where you get the backbone of mobilenet, I download from website, however, it seems mismatching the model used in SiamRPN++. Thank you

Modify it，and I also tried pretrain my own mobilenetv2 in imagenet

Right, I used the oringal mobilenetv2 to train as the experiments/siamrpn_mobilev2_l234_dwxcorr/config.yaml said.
But this config do not have a backbone layer train recommendation, it will merged with pysot/core/config.yaml, that means the the backbone train layer will be the same with resnet50, right? My question is ： Is this really okay? Thank you

It's hard to tell, you have to try it yourself

Have you tried to train with mobilenet? After merging with pysot/core/config.py and changing the training layer of backbone, I got poor results

Yes，I trained it. I got a not bad results. I use the defualt config in siamrpn_mobilev2_l234_dwxcorr

Have you tried to train with mobilenet? After merging with pysot/core/config.py and changing the training layer of backbone, I got poor results

Yes，I trained it. I got a not bad results. I use the defualt config in siamrpn_mobilev2_l234_dwxcorr

Can you share us with the result on your own pretrained  MobileNet_v2?

Have you tried to train with mobilenet? After merging with pysot/core/config.py and changing the training layer of backbone, I got poor results

Yes，I trained it. I got a not bad results. I use the defualt config in siamrpn_mobilev2_l234_dwxcorr

where you get the backbone of mobilenet, I download from website, however, it seems mismatching the model used in SiamRPN++. Thank you

Modify it，and I also tried pretrain my own mobilenetv2 in imagenet

and I have another question: how long did you take to train your own mobilenet_v2 ？ and what is your environment(eg: what type of your GPU)?

您可以在自己训练有素的MobileNet_v2上与我们分享结果吗？

the EOA of VOT2018 is 0.275

我还有一个问题：您花了多长时间训练自己的mobilenet_v2？您的环境是什么（例如：哪种类型的GPU）？

my server have 8 2080ti; I dont remember how long it spent

@SteveTsg ，thanks your reply. I have some question to consult you, and can I add your wechat， the number of my wechat is "jzb12390"

Close since the issue is resolved.
Please create a new issue should you have other questions

Have you tried to train with mobilenet? After merging with pysot/core/config.py and changing the training layer of backbone, I got poor results

Yes，I trained it. I got a not bad results. I use the defualt config in siamrpn_mobilev2_l234_dwxcorr

where you get the backbone of mobilenet, I download from website, however, it seems mismatching the model used in SiamRPN++. Thank you

Modify it，and I also tried pretrain my own mobilenetv2 in imagenet

Have you tried to train with mobilenet? After merging with pysot/core/config.py and changing the training layer of backbone, I got poor results

Yes，I trained it. I got a not bad results. I use the defualt config in siamrpn_mobilev2_l234_dwxcorr

could u tell me where did u get the mobilenetv2?  in the paper of  MobileNetv2 ?

@SteveTsg Hi！
Can you share us your pretrained mobilenet V2？Our lab does not have the conditions for pre-training. It takes a month to train on my own. Thank you very much！！


allow empty fields

due to migration, there are empty fields that are returned. Changing the openapi spec to accommodate these empty fields.

yes we should https://github.com/chanzuckerberg/corpora-data-portal/issues/684


Enable Subscriptions on the HAPI FHIR JPA Server

https://github.com/opensrp/hapi-fhir-jpaserver-starter/issues/29

Its already a configuration we can override. Why is this necessary?

@bennsimon How can we override this property, via deployment script?

We maintain our own application.yaml so we can update if needed.


[Question] Alternative for GPT4 eval?

I am still forbidden from approaching GPT4 api, so is there an alternative for evaluation?

We support many different evaluators, which we compare here. You just need to specify the desired annotator in the CLI, e.g. for chatgpt use --annotators_config chatgpt. If you don't have GPT4 I would suggest choosing between the following:

chatgpt cheaper but less human-like
text_davinci_003 pretty high human-agreement

The only issues you might encounter are:

if your model generates very long outputs given that chatgpt/text-davinci-003 have a context window 2x smaller than GPT4.
we didn't precompute the leaderboard for those.

Given that this seems to be a recurrent problem I will try to precompute the leaderboard using the 16k token chatgpt.

We've now added a chatgpt_fn annotator. You can find a minimal leaderboard using it here https://github.com/tatsu-lab/alpaca_eval#models and evaluation of its performance here https://github.com/tatsu-lab/alpaca_eval#evaluators. In general, it's not as good as gpt4 or claude, but it should be the most widely accessible one.


Update ipca.py

20201026: Added the function BS_Wdelta to test the significance of single PSF for a GB-model using the bootstrapped Wald-statistic

Closing this PR as it is updated an overruled by PR https://github.com/bkelly-lab/ipca/pull/10