Running a 2nd sidekiq process will overwrite schedule configuration stored in Redis

I've noticed that, if one runs a 2nd sidekiq process with a separate configuration, the recurring jobs tab gets emptied out in the web extension.
Please note that the schedule itself with rufus still continues to operate and queue the scheduled jobs on the correct intervals.
I believe this might be the cause of #293
Setup
Run a rails app with Sidekiq and sidekiq-scheduler. Mount the Sidekiq::Web UI with the sidekiq-scheduler/web required.
Include a config/sidekiq.yml with a configured schedule, e.g.,
:concurrency: 5
:queues:
  - critical
  - default
  - low
:schedule:
  do_something_every_minute:
    class: DoSomethingJob
    args: matey
    queue: :scheduler
    cron: '0 * * * * * America/Los_Angeles'
# ...

Next create another sidekiq configuration, this time without a schedule:
:concurrency: 5
:queues:
  - scheduler
  - other

Steps to reproduce

Start the rails server

Start a sidekiq process with the default configuration loaded (the one with the schedule)

Go to the Recurring Jobs tab in the Sidekiq Web UI, you should see the schedule like so:

Start the other sidekiq process loading the configuration without a schedule

Refresh the Recurring Jobs tab, you'll see the schedule disappear:

This bug is order dependent, if the process with the schedule boots last, the Recurring Jobs tab will show the schedule.
I made a reproducible app here: https://github.com/wspurgin/vanilla/tree/sidekiq-scheduler
Hypothesis
I believe it has to do with how the schedule is stored. Namely, that the key used to store and retrieve the schedules is just schedules and not namespaced to the booted Sidekiq process (e.g., by pid or full process name). Therefore the last booted sidkeiq process' "schedule" configuration wins out.

I would say that this is just not a supported feature by sidekiq-scheduler and not a bug. Why do you need to run one process with the schedule and the other one not?

Hi @marcelolx 👋 Thanks for the speedy reply!
There are scenarios where, like those hinted at in the Sidekiq Wiki, a service or set of services might have a handful of sidekiq processes for handling different queues. Only one configuration may be the definitive source of the "schedule" and its setup.
This issue arises when those Sidekiq process share the same Redis instances (which again, I feel is a some what normal practice when using Sidekiq). It feels redundant to have every configuration define the same schedule, it become very hard to do so if, for instance, the sidekiq processes in question do not share the same codebase but share the same redis infrastructure.

@wspurgin Particularly I would prefer to keep it explicit, this way who is gonna look at the config file will know that the scheduler is disabled (with dynamic schedules it would/could start disabled and then we would have to enable it when the schedule changes).
Also what I think that could help would be using https://github.com/resque/redis-namespace, this way each "schedule" would be namespaces and they wouldn't override, but I think disabling the schedule is way easier.

👍 Heard on the explicit enabled/disable flag. Maybe just a documentation enhancement then (again, if anything)
Namespaces would also solve this specific issue... but that would mean all of Sidekiq is namespaced which might not be as desirable. Disabling is definitely the easiest thing to do.

@wspurgin Documentation enhancements are welcome! If you can add the scenario described above I'll be happy to merge! (and anything else you consider valuable)
Yeah, disabling definitely is the easiest solution

Thanks @marcelolx 👍 Opened #362 for it 😄 Let me know what you think.

Hmm, quick question: the key is :enabled: at the root level, right? Would it make more sense to have it under the :schedule: key or something?
Also, would you run into this issue if using a different file for the schedule? I'm thinking not since in that case each process would load it from the same file, but wanted to double-check. https://github.com/sidekiq-scheduler/sidekiq-scheduler#load-the-schedule-from-a-different-file
Thanks!

@ibrahima Yes, it is at the root level. In this PR https://github.com/sidekiq-scheduler/sidekiq-scheduler/pull/412 we're going to move it to the scheduler key, which still is kind of root level. Moving under the schedule config doesn't "solves" the problem, since you still have to define it, right? I think what was expected was sidekiq-scheduler being disabled if no schedules key is present, but I think that is a bit dangerous (breaking change btw) since some people may be doing things with the assumption the scheduler will be enabled if not explicitly disabled.
Yes, it will because the issue is the schedules key in Redis being the same. If the docs are not clear about that, we can definitely improve them, but I think we should keep it clear that if you have to do that, you want to disable the scheduler explicitly.

Ah yeah that makes sense re: the "root" key.
So just to confirm: If I am loading the Sidekiq-scheduler config from a different file, all of my Sidekiq processes will have the same config. This is different from some processes having a config defined and some having :enabled: false set since in the latter case not all processes load the config. But will loading the config from a different file cause any problems? Since each process will end up storing the same config over and over again it should be fine right? To me this seems to be the same as having multiple Sidekiq processes all using the same sidekiq.yml which defines the same schedule, which should be pretty normal. It's just that if we have multiple sidekiq yml config files then we run into this issue? Thanks for the clarification!


Add mappings for DIG Autonomy.

Mapping file for autonomy is in dig-alignment. Ignoring this request.


GHA: different token

Task/Issue URL:
Description
Steps to test this PR
Feature 1

[ ]
[ ]

UI changes

Before
After

!(Upload before screenshot)
(Upload after screenshot)

#5248  👈
develop

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @malmstein and the rest of your teammates on  Graphite


Link to tinuous-inception in README

Closes #104.

Codecov Report

Merging #128 (f7fe0df) into master (1e6c566) will decrease coverage by 3.85%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master     #128      +/-   ##
==========================================
- Coverage   71.69%   67.84%   -3.86%
==========================================
  Files          10       10
  Lines         961      961
  Branches      142      142
==========================================
- Hits          689      652      -37
- Misses        217      256      +39
+ Partials       55       53       -2

Impacted Files
Coverage Δ

src/tinuous/appveyor.py
51.19% <0.00%> (-25.00%)
:arrow_down:

src/tinuous/travis.py
59.09% <0.00%> (-4.55%)
:arrow_down:

src/tinuous/github.py
51.27% <0.00%> (-3.39%)
:arrow_down:

src/tinuous/__main__.py
51.04% <0.00%> (-2.10%)
:arrow_down:

Continue to review full report at Codecov.

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


Missing mention of FutureVuls offering

I just discovered today, by chance, that there is a commercially supported version of Vuls, called FutureVuls, and that it has been existing for 2 years now. Congratulations! :thumbsup: I thought FutureVuls was going to be a next OSS version of Vuls under development, with some architecture and feature changes. :sweat_smile:
It would be interesting to get a mention of FutureVuls in the README documentation, with a few links, such as:
https://vuls.biz/en
https://vuls.biz/lp/
I could not found a FutureVuls organization on Linkedin. It could be useful too.
Note that I only use go-cve-dictionary from the OSS Vuls stack, but such mention of VultureVuls could also be welcomed in the other repositories.

Thank you for recommendation!

It would be interesting to get a mention of FutureVuls in the README documentation

Sounds nice.
We are considering documentation rewrites/re-organization.
It may be included at that time, and I will close this issue for now.

I could not found a FutureVuls organization on Linkedin. It could be useful too.

If you are interested, FutureVuls is the SaaS service run by Future Corporation, https://in.linkedin.com/company/future-corporation .


Display time between objc tests

Printing timestamps between tests for better troubleshooting the timeout problem.

Failed tests:
gRPC_interop_pull_requests - #9011


There is no 12.1 or 11.6 versions of pytorch-cuda for MacOS. Which should I use?

Sorry, and I just found unsloth is only working on linux/windows


[Bug]: Cypress Tests are in need of repairs

Summary of the bug:
Currently, most if not all of the cypress tests are broken and are in need of dire need of repair
Steps to reproduce the bug:
Run any of the Cypress tests to see that they break.
Expected Behavior:
The Cypress tests run (and hopefully succeed).
Current Behavior:
The Cypress tests don't run.

Issue was broken into three seperate issues and therefore this issue is no longer useful.
Refer to Issues

#119
#120
#121


Optimize some common if/else cases

if(predicate, 0, 1) or things like this are a very common operation in SQL.
The current GpuIf code will try to optimize for the case when all of the values in the predicate are true, or when all of them are false. The only time this would work as an optimization is if either the cost of computing the true or false expression and the copy_if_else is very high. High enough to offset the cost of computing the allTrue/allFalse nearly every time.
We cannot totally know the probably of everything in the predicate for this batch being true or false. It does happen and depending on the use case it could happen a lot. But generally it is not that common. Also we can know, at least to some degree the cost of the falseExpr, trueExpr, and the copy_if_else. For cases where they are literal values this is all very very low.
Because the probably of the predicate being all true or all false is low I decided to just disable this optimization, except for the case when there are side effects and the cost of doing the if/else is much higher.
In a real world user query I saw this save over 13% for some specific profiling ranges. The actual total query time saved was about 300ms. A small, but measurable difference.

build


Get the print result or status of the printer through the API

Hello,
Recently, I want to write code to get printer status and result after creating the print job. But I don't know what APIs do I need. The functionality I want to implement is as follows:

When the print job is created, I want to know the printer if start to print. If it's not printed, how can I get the reason.
When the print job is finished, how do I know if job is finished. How do I get exception information if an error occurs during printing? (Errors such as: paper jam, paper out, out of ink    )

I looked at a little code and tried to write a test code, but it didn't work for my purposes. This is the code file printer.c    that I have written. I was hoping someone could guide me.
Thank you.
                                                                                Best Regards

printer_2023_08_01_151951.zip

I thought for a moment, does CUPS only get printing success or not?

Once you submit a job, you can monitor the job to see its state (IPP_JSTATE_PENDING when queued, IPP_JSTATE_PROCESSING when it starts printing, IPP_JSTATE_COMPLETED, ``IPP_JSTATE_CANCELED, or IPP_JSTATE_ABORTED` when the job is done.
ipp_jstate_t get_job_state(const char *printer_name, int job_id)
{
  char printer_uri[1024];
  ipp_t *request, *response;
  ipp_jstate_t job_state;

  httpAssembleURIf(HTTP_URI_CODING_ALL, printer_uri, sizeof(printer_uri), "ipp", NULL, "localhost", 631, "/printers/%s", printer_name);
  request = ippNewRequest(IPP_OP_GET_JOB_ATTRIBUTES);
  ippAddString(request, IPP_TAG_OPERATION, IPP_TAG_URI, "printer-uri", NULL, printer_uri);
  ippAddString(request, IPP_TAG_OPERATION, IPP_TAG_KEYWORD, "requested-attributes", NULL, "job-state");

  response = cupsDoRequest(CUPS_HTTP_DEFAULT, request, "/jobs");
  job_state = (ipp_jstate_t)ippGetInteger(ippFindAttribute(response, "job-state", IPP_TAG_ENUM), 0);
  ippDelete(response);

  return (job_state);
}

@michaelrsweet  Hi michaelrsweet, thank you for your reply. I changed the code a little bit and  tried to test the code which you give, But I found that, the job_state I get is 0(zero). Is there something wrong with my code? The version of  CUPS I use is 2.4.5.
Look at my code:
`ipp_jstate_t get_job_state(const char *printer_name, int job_id)
{
char printer_uri[1024];
ipp_t *request, *response;
ipp_jstate_t job_state;
//httpAssembleURIf(HTTP_URI_CODING_ALL, printer_uri, sizeof(printer_uri), "ipp", NULL, "localhost", 631,       "/printers/%s", printer_name);
httpAssembleURIf(HTTP_URI_CODING_ALL, printer_uri, sizeof(printer_uri), "ipp", NULL, cupsServer(), ippPort(),"/printers/%s", printer_name);
request = ippNewRequest(IPP_OP_GET_JOB_ATTRIBUTES);
ippAddString(request, IPP_TAG_OPERATION, IPP_TAG_URI, "printer-uri", NULL, printer_uri);
ippAddString(request, IPP_TAG_OPERATION, IPP_TAG_KEYWORD, "requested-attributes", NULL, "job-state");
response = cupsDoRequest(CUPS_HTTP_DEFAULT, request, "/jobs");
if(!ippFindAttribute(response, "job-state", IPP_TAG_ENUM))
printf(" \n\n  fun[get_job_state] attr get null     \n\n");
job_state = (ipp_jstate_t)ippGetInteger(ippFindAttribute(response, "job-state", IPP_TAG_ENUM), 0);
ippDelete(response);
return (job_state);
}`
Thank you.

The result is:
I can see the message from screen                       fun[get_job_state] attr get null         .

Thank you for your reply. @michaelrsweet . I had solved my problem.


Define etcdctl config file with SSL variables

Let's add /etc/sysconfig/etcdctl with paths to the client
server TLS files and endpoint. This will make possible to run
etcdctl command in easy way, e.g.
source /etc/sysconfig/etcdctl
etcdctl cluster-health

e2e tests are broken right now, we can ignore this error since it's irrelevant to this change.


[feature] Support output levels using Sound's AnalyserNode

Create a simple way to get the intensity/levels using a sounds analyser upon playback.

Closed by #9


CRS Error

Hello
Firtst of all sorry for my bad english. But i am a german.
Im new on QGIS, and i have a question. I downloaded the Natural Earth TIFF
https://www.naturalearthdata.com/downloads/10m-raster-data/
I import this Tiff in Qgis Version 3.16.2
Then i change the KBS ( I think in english its the name CRS) to
World_times (ESRI: 54048) or to Sphere_times (ESRI: 53048)
but when i do this, the Map disappear and i have a white blank screen instead of the Map.
Have you an idea waht i doing wrong?
Thank you very much for your Help
Best regards
Thomas

@Algarius sorry, this issue tracker is for the QGIS website only. So I'm going to close it here.
When I load your data, and then change the project crs to one of the ones you mentioned all is fine. So I cannot reproduce your issue.
If you have questions, like this (which does not seem to be a QGIS Bug), please use the mailing lists
To learn about basics of QGIS (im Deutch):
https://www.qgis.org/de/docs/index.html
There is also https://fossgis.de/ the German OpenSource GIS community with some very helpfull German speaking people.

@Algarius sorry, this issue tracker is for the QGIS website only. So I'm going to close it here.
When I load your data, and then change the project crs to one of the ones you mentioned all is fine. So I cannot reproduce your issue.
If you have questions, like this (which does not seem to be a QGIS Bug), please use the mailing lists
To learn about basics of QGIS (im Deutch):
https://www.qgis.org/de/docs/index.html
There is also https://fossgis.de/ the German OpenSource GIS community with some very helpfull German speaking people.


Add flag and command for custom pipelines

Changes

Allow custom pipeline to be applied on remote builds
Allow currently applied pipeline to be exported to a file

/kind

Fixes #791

Release Note

Added flag to specify a custom tekton pipeline to be applied from a file and command to export cluster pipeline to a file

Docs

As mentioned on #791, do you think it would make a sense to think about the solution that will mimic the approach that is being used for language templates? To add posibility to define custom pipelines with the very similar workflow.

/assign @salaboy

I think @zroubalik knows most about pipelines.

As mentioned on the team call, I think that it might be beneficial to investigate, whether using pipelines templates is not a better one. By templates I mean a similar approach that we have taken with Extensible Language Packs -> https://github.com/knative-sandbox/kn-plugin-func/blob/main/docs/language-pack-providers/language-pack-contract.md#extensible-language-packs

@shinigambit ping - any status updates on this?

I have an idea how to refactore the whole Tekton integration around the Pipeline as a Code tooling. This should bring us this functionality as well.


ci: Add tests for supported Go versions

Add supported Go versions (1.13~) to the test matrix on pre-submits.
Fixes #13

Codecov Report

Merging #19 (fd215ed) into main (2702f97) will not change coverage.
The diff coverage is n/a.

Additional details and impacted files

@@           Coverage Diff           @@
##             main      #19   +/-   ##
=======================================
  Coverage   95.94%   95.94%
=======================================
  Files           1        1
  Lines          74       74
=======================================
  Hits           71       71
  Misses          2        2
  Partials        1        1


[mysql] Fix the fetchSize parameter is not used

issues : #1582

@fuyun2024  MySqlTimezoneITCase hangs in test.

@fuyun2024 Maybe we need to add some tests to verify the bug is fixed.

If we open the useCursorFetch, the tests in the  MySqlTimezoneITCase will fail.
It seems that the date type field can not be parsed right when useCursorFetch is true.

If we open the useCursorFetch, the tests in the MySqlTimezoneITCase will fail. It seems that the date type field can not be parsed right when useCursorFetch is true.

This problem has been fixed in DBZ-3238(https://github.com/debezium/debezium/pull/2245).
So I have applied these changes to mysql cdc connector.

HI @ruanhang1993 , Thank you for your time and suggestions,  @GOODBOY008  has submitted a new PR  #1629 , and I will close this submission later.


[Open Link in Specific Browser] Automatically recognize links in Raycast and suggest to open in specific browser

Extension
https://www.raycast.com/koinzhang/open-link-in-specific-browser
Description
When I post a link in raycast, it should suggest open in a specific browser.

Who will benefit from this feature?
It will make the use of the extension more natural.
Anything else?
No response

Hi @vbshubham First of all I'm glad you're using the Open Link in Specific Browser extension.
The feature request you made is not currently available as an extension, it should be a built-in Raycast capability and you should make a feature request to Raycast officially.
Or @pernielsentikaer and @sxn can take a look at this issue.


Clarified that CompletionItem.sortText is coverted to lower case

Clarified that CompletionItem.sortText is coverted to lower case.

https://github.com/Microsoft/vscode/blob/master/src/vs/editor/contrib/suggest/suggest.ts#L53

Well, comparison happens case-insensitive at first and later both variants are used, e.g matching the casing is better. @tamuratak what's the issue here, why is this needed?

I think the latter part "matching the casing is better" is not true if items have a common prefix and users type only the following word. The order  pre_aaa, pre_abb, pre_Aaa, pre_Abb can not be returned when users type a.

It should be clarified that even if sortText is set, the order cannot be controlled because the sorting is case-insensitive.
If we require users to type the prefix, the order becomes intended one.

It should be clarified that even if sortText is set, the order cannot be controlled because the sorting is case-insensitive.

That's not how it works. The sort text is used when comparing item before taking the user prefix into account, e.g when suggestions have been requested without a prefix (pre_a or pre_A from above's sample). Once the user has typed a prefix it will be used to resort completion based on how well a completion matches the prefix - when the prefix-score of an items are equals, then  the initial, sortText-based score is used to resolve such conflict.
I agree that this needs to be documented somewhere (apart from a bunch of issues) but I don't think that vscode.d.ts is the right place. We have https://github.com/microsoft/vscode-extension-samples/tree/master/completions-sample which we can polish a bit more and we can then make a new guide in the 'extension guides' section of https://code.visualstudio.com/api.

I have created https://github.com/microsoft/vscode-docs/issues/2825 in vscode-docs, closing this PR without action


LL parser support to create protocol API review from autorest yaml

Integrate parser so APIView can take yaml as input file to generate protocol reviews.

@LibbaLawrence @kristapratico @lmazuel : This PR has changes to integrate protocol parser in APIView,

@pakrym That was so quick. Thanks for the approval.


Getting only one sub document

So I have a product document which has a nested images document. The structure is like this:
{
"_id" : ObjectId("551e617eccfff2fc193c9869"),
"currency" : "US DOllar",
"images" : [
{
"filename" : "551e617eccfff2fc193c9869_logo-big.jpg",
"height" : NumberLong(500),
"width" : NumberLong(500),
"updated_at" : ISODate("2015-04-07T10:35:10.142Z"),
"created_at" : ISODate("2015-04-03T09:53:31.168Z"),
"_id" : ObjectId("551e631bccfff278173c9869"),
"deleted_at" : ISODate("2015-04-03T09:53:43.102Z")
},
{
"filename" : "551e617eccfff2fc193c9869_sameer.jpg",
"height" : NumberLong(500),
"width" : NumberLong(500),
"updated_at" : ISODate("2015-04-03T10:00:39.969Z"),
"created_at" : ISODate("2015-04-03T10:00:39.969Z"),
"_id" : ObjectId("551e64c7ccfff200173c9869")
}
],
"is_active" : "1",
"updated_at" : ISODate("2015-04-07T08:05:15.932Z"),
"created_at" : ISODate("2015-04-03T09:46:38.340Z"),
}
Now I want only one sub document, i.e. image whose filename is "551e617eccfff2fc193c9869_logo-big.jpg" along with the whole document. How can I do that?

I know how to do it in mongo
Ie.db.products.find({"images.filename":"551e617eccfff2fc193c9869_logo-big.jpg"},{"images.$.filename":1,"currency":1}).pretty()
But how to execute it with this package?

Since a recent PR, the project() method was added. You can give it a try.


Switch prow configs to use Let's Encrypt https address.

The route from prow.tekton.dev -> cluster ingress  is configured using Google internal DNS for now.
The certificate is issued through let's encrypt using these configurations.

As part of this I noticed I had two prow config files checked in in different locations. Deleted the extra one.

/lgtm


Update azure-web-apps.md

Adding clarification for the NodeJS and Python applications when both auto and manual instrumentation are enabled.
Source:
https://learn.microsoft.com/en-us/azure/azure-monitor/app/azure-web-apps-nodejs?tabs=linux#enable-application-insights
https://learn.microsoft.com/en-us/azure/azure-monitor/app/azure-web-apps-python#enable-application-insights

@AaronMaxwell
Can you review the proposed changes?
Important: When the changes are ready for publication, adding a #sign-off comment is the best way to signal that the PR is ready for the review team to merge.
#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team


Missing KeyKpAdd mapping

Just noticed that there is a mapping missing in
ThirdParty\OpenSource\gainput\lib\source\gainput\keyboard\GainputInputDeviceKeyboardWin.h
The mapping missing is
dialect_[VK_ADD] = KeyKpAdd;
I'm attaching a patch with the small fix.
MissingMapping.zip

Thank you! This is very nice of you! We will add this to the next release. In the meantime, if you are interested in spending more time with the cutting edge of the code base, I could invite you to our internal
Gitlab server where all the magic happens. Drop me a note ...

Thanks for your answer. That invite would be great :) . I've got another fix here somewhere... I'll check if it still an issue in latest and drop you a patch in case it is.

I would need your e-mail address for this.

Sure<EMAIL_ADDRESS>

Fixed with Release 1.40


fix #1319, static link to libgcc & libstdc++ to support CentOS7

After this change, the ldd command output for libduckdb_java.so_linux_amd64 will be:
	linux-vdso.so.1 =>  (0x00007fff9b188000)
	libpthread.so.0 => /lib64/libpthread.so.0 (0x00007fbd14081000)
	libm.so.6 => /lib64/libm.so.6 (0x00007fbd13d7f000)
	libc.so.6 => /lib64/libc.so.6 (0x00007fbd139b1000)
	/lib64/ld-linux-x86-64.so.2 (0x00007fbd15093000)

instead of
./libduckdb_java.so_linux_amd64: /lib64/libstdc++.so.6: version `GLIBCXX_3.4.20' not found (required by ./libduckdb_java.so_linux_amd64)
./libduckdb_java.so_linux_amd64: /lib64/libstdc++.so.6: version `CXXABI_1.3.8' not found (required by ./libduckdb_java.so_linux_amd64)
./libduckdb_java.so_linux_amd64: /lib64/libstdc++.so.6: version `GLIBCXX_3.4.21' not found (required by ./libduckdb_java.so_linux_amd64)
	linux-vdso.so.1 =>  (0x00007ffe8af82000)
	libpthread.so.0 => /lib64/libpthread.so.0 (0x00007fe18a626000)
	libstdc++.so.6 => /lib64/libstdc++.so.6 (0x00007fe18a31e000)
	libm.so.6 => /lib64/libm.so.6 (0x00007fe18a01c000)
	libgcc_s.so.1 => /lib64/libgcc_s.so.1 (0x00007fe189e06000)
	libc.so.6 => /lib64/libc.so.6 (0x00007fe189a39000)
	/lib64/ld-linux-x86-64.so.2 (0x00007fe18b4e8000)

If this fix is fine, we may also consider to change other Command Line tools to add this linker flag to support more Linux distros.
Thanks.

Thanks, we already do this for the shell/library CI build I believe.

Thanks, we already do this for the shell/library CI build I believe.


Fix #97: split unbalanced SPI transfers into rw+r or rw+w

I think this should be OK but so far I've only tested it on equal-sized buffers and reads-bigger-than-writes as I don't have anything convenient set up for checking what happens with writes-bigger-than-reads.

Changelog added. I tested this with short reads too and it seems to work as expected.


Move mesh

Heads up @raynol-dsouza that in the process of moving RectMesh I discovered that the last PR introduced a bug into the laplacian -- it worked OK for perfectly smooth functions, but has numeric trouble for noisy fields, which is exactly our starting point in Schroedinger.
EDIT:
Sorry, I intended for this to be a pure refactor:move, but I found a bug in laplacian and got carried away. In addition to moving the laplacian bug is squashed, accuracy (i.e. points in the stencil) is handled as a class attribute, and there's a demo notebook.

Currently I see that derivative adds a new axis in the first column, so from (nx, ny, nz) it makes (3, nx, ny, nz), correct? It's super important to document it somewhere in the DocString because other people (e.g. me) might do it otherwise other way around (i.e. (nx, ny, nz, 3)).

@liamhuber
Aha. Good catch! I cannot test the branch unless it has #63 merged (because of the fenics ModuleImportError. Could you please merge it into this PR?

Once #63 is merged into master I'll pull master in. I don't want to mess around with branch-to-branch merges atm. There is no rush to get this stuff in though, so I don't mind waiting for 63.

I just realised that RectMesh is in schroedinger, but doesn't it make more sense to have it in its own thing (especially since now I'm thinking about using it for my grain boundary dynamics stuff)

I just realised that RectMesh is in schroedinger, but doesn't it make more sense to have it in its own thing (especially since now I'm thinking about using it for my grain boundary dynamics stuff)

Take a quick peek at the PR name 😉 (I have to confess, when I looked at the GitHub diff it did not pop out to me even when searching for it!)

XD I thought this branch had already been merged. Typical mistake of mine sorry!!!


merge 2.6 from main

v2.6 (pre)

Default Search Engines that are not available by region are automatically hidden.
Fixed an issue that caused unnecessary redirects in certain cases when the default search engine is Google and CSE is set to Google.

retry


Add base styling for inline and fenced code blocks

The project doesn't currently style code blocks in any way. This adds sensible defaults that can easily be overridden through variables.
Additions

Added base styling for inline and fenced code blocks

Testing

This is difficult to test in any of our doc sites due to the existing code styling that lives in those.

Review

@Scotchester
@cfarm
@anselmbradford
@sebworks

Screenshots

Checklist

[x] Changes are limited to a single goal (no scope creep)
[x] Code can be automatically merged (no conflicts)
[x] Code follows the standards laid out in the front end playbook
[x] Passes all existing automated tests
[x] New functions include new tests
[x] New functions are documented (with a description, list of inputs, and expected output)
[x] Placeholder code is flagged
[x] Visually tested in supported browsers and devices
[x] Project documentation has been updated (including the "Unreleased" section of the CHANGELOG)

That inline example looks a lot bigger than 13px. Is that actually how it renders out?

@Scotchester I've removed all the custom code styles on the GH pages branch to show what these defaults look like, I think a few were still getting through in the above screenshots.

The text size looks correct, I'm assuming the previous screenshots were affected by the GH pages branch styles.
Agreed on the side padding, I've reduced it to 3px to match the top.
I think the bottom space is ok, the problem with adding bottom padding is it'll mess up the line-height if the inline code has a line of text below it, like in a paragraph.

:+1: :shipit:


[8.x] Add File::lines($path) method

Many of the most useful scenarios for using lazy collections involves reading files line by line (see here for more details).
This PR adds a lines method to the Filesystem class, to lazily read the lines of a file:
$collection = File::lines($path);

Example
With this in place, you can easily read Laravel's logs lazily (using the new chunkWhile method):
$pattern = '/^[\d\d\d\d-\d\d-\d\d \d\d:\d\d:\d\d]/'; // starts with e.g. [2020-09-07 00:24:11]

$logs = File::lines(storage_path('logs/laravel.log'))
    ->chunkWhile(fn ($line) => ! preg_match($pattern, $line));

The $logs variable will now be a lazy collection, from which we can pull out a single log entry at a time.

Lazy collections could be quite appropriate for paginating eloquent queries tbh. Currently there's an each method, but we could "go native" as it were, and have a way to do getLazy() or something,

@GrahamCampbell not sure why we're having this discussion on this PR :shrug:
How would getLazy be different than the current cursor method?


style: cleanup unused lll and '.' at the end of comments

What type of PR is this?
/kind cleanup
/kind failing-test
What this PR does / why we need it:
Cleaning up unused lll lint errors and periods at the end of comments errors.
Which issue(s) this PR fixes:

(https://github.com/Azure/AgentBaker/actions/runs/6399258600)
Requirements:

[x] uses conventional commit messages

[ ] includes documentation
[ ] adds unit tests
[ ] tested upgrade from previous version

Special notes for your reviewer:
Release note:
none

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 79.178%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
2236

Relevant Lines:
2824

💛  - Coveralls


[aleph-client]: Add decoders from Value into String and Option<T>.

Description
Adds decoders from ConvertibleValue into std::String and Option<T>.
Type of change
Please delete options that are not relevant.

New feature (non-breaking change which adds functionality)

Checklist:

I have created new documentation
I have bumped aleph-client version if relevant

Note
Mirror of #767 but now PR is made from not-a-fork repo.

This PR is on hold until main is merged to benjamin.


fix: should return not found

IMPORTANT: Please do not create a Pull Request without creating an issue first.
Problem:
Although we use steven to proxy all k8s resource, but we can still customize some logic with store of schema. In our custom logic, we didn't check not found error, we just returned any error with 500 error code.
Solution:
Return not found if vm doesn't exist.
Related Issue:
https://github.com/harvester/harvester/issues/4388
Test plan:
Delete a non-existing vm with this endpoint /v1/harvester/kubevirt.io.virtualmachines/{namespace}/{name}, it will return 404 not found http status code back.

I think this one should be backport to v1.2 and v1.3.

@Mergifyio backport v1.2 v1.3


Add lv_btnm_set_one_toggle/lv_btnm_get_one_toggle functions

@kisvegabor This implements the behavior we discussed in #954.

Thank you! It looks good in general. I have only a few remarks, see them in the comments.

@kisvegabor I don't see any comments... did your review get lost somehow?

I tagged you.
Here are the comments for me: https://github.com/littlevgl/lvgl/pull/966/files

@kisvegabor There are no comments there for me. Something really strange is happening. Did you go to the "Review changes" box at the top and click "submit review"? If the comments say "Pending" they won't show up for me till you do that.

Oh, I didn't... How have I missed that so far!? It explains a lot of things

@kisvegabor I think everything you mentioned is fixed now.

Looks good to me! :+1:
Thank you!


S38(28) Dev - Fund Supported Pool Subset - Part 2

https://teams.microsoft.com/l/entity/com.microsoft.teamspace.tab.planner/tt.c_19:68ee6eb15df44390b85fb02cac58153d@thread.tacv2_p_ZOb3bFXcakWu8Gl2Zd_PuGUAFIJt_h_1611165472107?tenantId=6fdb5200-3d0d-4a8a-b036-d3685e359adc&webUrl=https%3A%2F%2Ftasks.teams.microsoft.com%2Fteamsui%2FpersonalApp%2Falltasklists&context={"subEntityId"%3A"%2Fboard%2Ftask%2FZe0vNdzoI06p-eIhu-Dv42UAM2-b"%2C"channelId"%3A"19%3A68ee6eb15df44390b85fb02cac58153d%40thread.tacv2"}

Hi Paul,
Why there is no resources/sass/app.scss file included in the bug fix ? I just saw the public/css/app.css file ?
Please kindly clarify?
Thank you.

Hi Paul,
Could you please update the title from the MS Team and description on this pull request?
Thank you.

Hi Paul,
Could you please update the title from the MS Team and description on this pull request?
Thank you.

Added Thanks!

Thank you and looks good.


No consent needs to be asked during the login flow - if no requested claims are there and no claim picked as the subject

At the moment when we login to a service provider we ask the consent for the username.
No consent needs to be asked during the login flow - if no requested claims are there and no claim picked as the subject.
Please add a note to the migration guide too. If someone needs this behavior as in 5.7.0, the need to pick the http://wso2.org/claims/username claim as the subject claim under the service provider configuration.

This issue is being closed due to extended inactivity. Please feel free to reopen it if further attention is needed. Thank you for helping us keep the issue list relevant and focused!


support for http://www.mixcloud.com/

please do the support of http://www.mixcloud.com/
it would be nice if scrobbler read the tracklist from mixes.
Thx!!!

So, Mixcloud actually supports scrobbling natively, and if you have it enabled then everything gets double scrobbled :unamused:
I'm not sure how best to handle this?


propose to add jichenjc as approver

I have been involved in this project for long time and did
a lot of contributions, aslo, very active in reviewing PRs and
provide comments

What this PR does / why we need it:
Which issue this PR fixes(if applicable):
fixes #
Special notes for reviewers:

Release note:

NONE

@ramineni @lingxiankong please let me know your opinion, thanks

@jichenjc I'm ok with it.
Thanks for all the contributions so far and stepping up for more responsibility :)
/cc @lingxiankong

/lgtm

You deserve it, welcome!
@jichenjc @ramineni since we have 3 active reviewers now, please comply with the policy that we only merge non-trivial PRs that have at least two /lgtm
/lgtm
/approve


Updated 'dcos task exec' to print better error message.

Running dcose task exec against containers launched by the docker
containerizer is currently unsupported. This commits prints a better
error message when attempting to run this command against such a
container.

bumped.


Flac downsampling

Flac downsampling would be extreemly usefull for the transcoder.

@Mechazawa I agree.

I too would love to see this.

I have created support for downsampling here https://github.com/karamanolev/WhatManager2/pull/91


Stability has no concept of deferred libraries

(Internal bug is b/62414314)
@hterkelsen wrote:

It's possible for deferred components not to be ready by the time fixture.update() completes. We should wait for deferred loads to complete.

While writing this bug, I noticed Testability (for e2e tests) suffers from a similar issue.
We have a few options, as usual, none of them a slam dunk:

Somehow integrate with a public API that both DartDevC/Dart2JS would emit for this purpose.
angular_test only: Add a DeferredLibraryStabilizer, JS-interop monkey patch "stuff".
angular_test only: In dev-mode, change loadDeferred:

// lib/src/runtime.dart
List<Future<void>> pendingDeferredLibraries = [];

// lib/src/.../app_view.dart
void loadDeferred(Future<void> Function() load1, Future<void> Function load2) {
  final waitFor = new Future.wait([load1, load2]).then((_) {
    _finishDeferLoad();
  });
  if (isDevMode) {
    pendingDeferredLibraries.add(waitFor);
  }
}

... and use pendingDeferredLibraries as a signal for stability by default in angular_test.

Add a generic trackAsyncWork function for stability:

F trackAsyncWork<F extends Future<void> Function()>(F func) {
  if (isDevMode) {
    return () async {
      knownAsyncWork++;
      try {
        return func();
      } finally {
        knownAsyncWork--;
      }
    };
  }
  return func;
}

... and use the counter of knownAsyncWork as a signal for stability. We could use trackAsyncWork by default for all calls to loadDeferred, and let users use it as well for their own events (loading from IndexedDB, XHRs, etc).
To be honest, I haven't heard folks request support for Testability, but I also know very little about it (or e2e testing, in general) - maybe there are other mechanisms like waiting for XHRs using web driver that are better suited for that purpose. In my head 3 makes the most sense, but doesn't generalize well (where 4 does).
/cc @TedSander @liuming0 for additional input.

Where did this come from? Was it from a client having a problem? or something else?
So I don't think this happens in practice today for a couple of reasons which we won't always be able to rely on:
a) For component tests DDC doesn't really defer code. So the timing isn't the same as Dart2js and might not come up there.
b) For web driver tests they typically wait for an element to be shown, or disappear before continuing on in the test. Some examples could include waiting for a progress spinner or bar to hide, or waiting for the specific element to show.
That said there is some testabilities for things such as activities which may end up catching some of these.
That said I think we shouldn't rely on this. It would be nice for angular to add this to the list of things it knows needs to be stable.
Without diving in too deep 4 sounds the best to me.

@TedSander:

Where did this come from? Was it from a client having a problem? or something else?

Actually from @hterkelsen, who was trying to test loadDeferred.

a) For component tests DDC doesn't really defer code.

That's true, but you still do need to wait for the future to complete, I think?
/cc @jmesserly @vsmenon

b) For web driver tests they typically wait for an element to be shown

Ah, then I'm not too concerned.

Let's close this, for now. I don't see this being acted on, or else we should track general instability.


Particles move relative to the emmitter and not the domain.

Particles in rParticle currently are positioned relative to the emitter. This means that whenever the emitter moves, the particles move with it. By default, particles should have a world position value which can be used to update the location of the particle.

Turns out that the spawn location of particles should be the changing factor. The current implementation seems to be fine.
Closing.


how to convert scnn pytorch model to caffe model?

Dear sir :
I have trained the scnn net ,now I want to convert the scnn pytorch model to caffe model ,I have no idea ,can you give me some advice ,thank you

See this link.

Thank you very much

Does this link work?

I have not used that code. You can have a try.

thank you
------------------ 原始邮件 ------------------
发件人: "HOU<EMAIL_ADDRESS>发送时间: 2020年6月3日(星期三) 上午9:53
收件人<EMAIL_ADDRESS>抄送: "葛恒赫"<380768194<EMAIL_ADDRESS>主题: Re: [cardwing/Codes-for-Lane-Detection] how to convert scnn pytorch model to caffe model? (#282)
I have not used that code. You can have a try.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or unsubscribe.

have you tried any network from pytorch model to caffemodel ,I have tried the tool you recommend to me last week ,but I have met many errors ,can you help me or some advice ,the detail problem I can provide ,it is so long


Fix messed up merge

Description
Voting gauge generator script got messed up on develop during a merge, this fixes that by syncing with what's on master.
Type of change

[x] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] Dependency changes
[ ] Code refactor / cleanup
[ ] Documentation or wording changes
[ ] Other

How should this be tested?
Please provide instructions so we can test. Please also list any relevant details for your test configuration.

[ ] Test A
[ ] Test B

Visual context
Please provide any relevant visual context for UI changes or additions. This could be static screenshots or a loom screencast.
Checklist:

[x] I have performed a self-review of my own code
[ ] I have requested at least 2 reviews (If the PR is significant enough, use best judgement here)
[x] I have commented my code where relevant, particularly in hard-to-understand areas
[x] If package-lock.json has changes, it was intentional.
[x] The base of this PR is master if hotfix, develop if not

@mendesfabio can you check that the script that's on master is correct? I think I messed it up during a merge and I'm not sure if it just got messed up on develop or if it's incorrect on master too.


Better control C++ and Java garbage collection

As reported elsewhere, a where that is difficult to find, the master branch with glovecrash4 will run the infinite parallel test for about 2:50 hours before Java says it is out of memory.  That is with a new Processor being created each time the 10 documents are re-processed.  If the same Processor is reused, then the program ran for 20 hours without stopping.  Why?

I published a local copy of the debug version of dynet for Windows which includes a fairly good leak detector.  I ran a shortened version of the infinite parallel test, because the leak detector slows things down.  After 45 minutes the program exited and did one last garbage collection.  No leaks were detected other that the one added on purpose to make sure it is working.  It appears that all the memory is accounted for and will be collected at some time,  That time may just be too late.

Next I modified all the Scala classes in DyNet to have a close() method which calls delete on the internal Java object which in turn calls the C++ method to free memory.  In other words, manual memory management is being used in Scala.  Now only to CluProcessor.annotate() I have added code to close the embeddings.lookupParameters and embeddings.collection.  So far, the resulting program has been running 11:30 hours and the operating system does not note any excessive memory use.  Not every Scala object is being closed, but doing these few seem to help.

Excellent!!

fwiw i enjoyed reading that
On Sunday, May 23, 2021, Mihai Surdeanu @.***> wrote:

External Email
Excellent!!
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/clulab/processors/issues/494#issuecomment-846614771,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABJCPCP64VRMW3ZVHQJXXNLTPFLRBANCNFSM45L77SZQ
.


[PE-D][Tester A] Typo in Viewing a Contact

Sorry but I think it should be 'an Own Wedding label'...

Labels: type.DocumentationBug severity.VeryLow
original: Shantaa/ped#6

can't believe this is catched


Update xmake.lua

m4 package fails on linux, no idea what it is

I have fixed it. see https://github.com/xmake-io/xmake-repo/pull/274
we should use libsodium-$(version)-msvc.zip instead of libsodium-$(version)-stable-msvc.zip
xxx-stable-xxx will be updated frequently


update-dbainstance verbose should be warn

"is already the latest available" should be a warn instead of verbose so that the person can know why the updater quit

@potatoqualitee do you mean the following in Get-SqlInstanceUpdate?
Write-Message -Message "$($currentVersion.Build) on computer [$($computer)] is already the latest available." -Level Verbose

@sirsql that is the right one.


IGAPP-1030:  Remove query_all_packages permission

This pull request belongs to an issue on our bugtracker.
You can find it there by looking for an issue with the key which is mentioned in the title of this pull request.
It starts with the keyword IGAPP.
Since the in-app-browser lib community doesn't release frequently, we can not expect to get the package permission removed in time.
Therefore i created a patch file to remove the permission in the android manifest.
I'm gonna create an additional ticket to remove this patch when the library was updated
IGAPP-1039
Testing:

build app:yarn android
check android/app/build/intermediates/merged_manifest/AndroidManifest.xml for the used-permission query_all_packages it should be removed now
check if in-app-browser is still working

Tested on android real device

I still have QUERY_ALL_PACKAGES in AndroidManifest, even after deleting build folder and running yarn install. What am I missing? It is in the subfolder debug though (android/app/build/intermediates/merged_manifest/debug/AndroidManifest.xml)

Same for me. Query all packages is still there.

@steffenkleinle  & @SteffiStuffel
i had to add some packages for yarn to execute the patch and a postinstall script. Now it should work fine.
I also added instruction for removing this in the follow up ticket
https://www.npmjs.com/package/patch-package
I'm not 100% sure about this
You can use --dev if you don't need to run yarn in production, e.g. if you're making a web frontend.

Is it fine for a native app too? Dunno if its okay to keep them in dev deps

@steffenkleinle & @SteffiStuffel i had to add some packages for yarn to execute the patch and a postinstall script. Now it should work fine. I also added instruction for removing this in the follow up ticket https://www.npmjs.com/package/patch-package I'm not 100% sure about this
You can use --dev if you don't need to run yarn in production, e.g. if you're making a web frontend.

Is it fine for a native app too? Dunno if its okay to keep them in dev deps

Installing it as dev dependency should definitely be enough since it is only needed for the postinstall script

Just tested this with a production build and the permission is not included and the inappbrowser still works :+1:

I guess there will not be an update in time. Therefore lets merge it :)


Add anyIndexed function

Hi, I'm trying to implement something like this:
R.some.idx(function(value, i, list) { return value != R.nth((i + 1) % R.length(list), list); });

But Ramda doesn't provide indexed some or any. Wouldn't it be worth to add anyIndexed function?

I don't understand what this predicate proves:
function(value, i, list) { return value != R.nth(i, list); }
as i read this, it says, "true, when the value at index i is not equal to the value at index i". Am I missing something?
we've been over the indexing thing before. I think what we would all like is a way to decorate iterators to take extra args, but default to the single arg case. What do you propose for anyIndexed? What is the API/signature?

We had a long, fairly rancorous discussion about this issue in #452.
There is a proposal, which I'm hoping to revive for version 0.10.0 in #484 to remove the .idx functions but allow you to wrap reduce, map, filter, some, etc like this:
addIdx(map)(function(val, idx, all) { /* ... */ }, list);

But I don't understand what you're trying to do in that code block.  Can explain in more detail?

Sorry guys, I made a mistake in my example, now it's ok. I would expect that index and list are extra params.

This will probably be available when we get the addIdx feature in place.  In the meantime, if I understand the code block above, all you're trying to do really is return true if there are two or more distinct values in the input list, false if the list contains all the same value.  If that's all you want, this would probably do the same:
pipe(uniq, length, gt(R.__, 1))

There are many other possibilities.
When I have a few minutes, I'll try to explain my philosophical objection to these idx functions in general.

I wanted to discuss my reasons for objecting to the indexed versions of functions like map, filter, reduce, etc.  We had a long discussion about adding them to some and any back in #452, and a pass at how to replace them with a common wrapper function in #484.  But I never really articulated one of my major reasons for not liking them.
Of course the Transducers effort is underway now, but I've had a similar dream since very early on.  I wondered if we might be able to unify various data structures under our fairly straightforward API.  I thought about it then in perhaps simpler terms than I'm about to suggest here, but I've been headed this way for some time.  Then I thought chiefly about unifying things like arrays and streams; Highland beat us to that.  We added the dynamic dispatch originally for the lazy lists, and expanded it so that we could handle the FantasyLand specification.  These are all very useful and important.  But I also would really like to extend many of our key functions to many other data types beyond these algebraic types.
Think about map for a moment.  In essence, mapping is really simple:  You accept a function and a data structure containing elements in the domain of the function.  You return an equivalent data structure with elements in the co-domain of the function.  The corresponding elements in the returned structure are the result of running the function on the elements of the original structure.  This is clear enough for a list: the new structure has a head which is the function applied to head of the original list and a tail which is the mapping of the function to the tail of the original list.  The recursion bottoms out with an empty list, which returns an empty list.  It's pretty simple.
But let's look at another data structure.  Why can't we easily map over a tree?
Say we had this tree, tree1:
               A
    +----------^----------+
    |          |          |
    B          C          D
+---^---+  +---^---+  +---^---+
|   |   |  |       |  |   |   |
1   E   4  6       7  8   F   4
  +-^-+                   |
  |   |                   |
  2   3                   6

Then if we wanted to map square over that tree, the result should be pretty obvious:
map(square, tree1); //=>

               A
    +----------^----------+
    |          |          |
    B          C          D
+---^---+  +---^---+  +---^---+
|   |   |  |       |  |   |   |
1   E  16 36      49 64   F  16
  +-^-+                   |
  |   |                   |
  4   9                  36

Similarly if we wanted to run filter over the tree:
filter(odd, tree1); //=>

         A
    +----^-----+
    |          |
    B          C
+---^---+      |
|       |      |
1       E      7
        |
        |
        3

It would be extremely useful if a number of our functions could work on a data type such as a tree.  Or a stream.  Or a monad.  Or ... there are many possibilities.  I don't really expect to get these ideas working particularly soon, but would like to keep them in mind as we work with the API.  The index API, however, ties us to array-like data structures.  It seems to tie our hands for the future.  And I'm loathe to do that.
Trees do not have indices.  Streams do not have indices.  Even lists really don't have indices, although we can find a simple-enough way to simulate them.  Vectors and arrays have indicese.  Tying ourselves down this way seems far too limiting.
In any case, that's one of my reasons.  Does this make sense?

I totally understand, I haven't thought about that this way. Thanks for the explanation.
My real world example is that I have a polygon defined by N vertices (x, y) and I want to compute its axes, vectors between two vertices (N, N + 1). Maybe I'm missing something, but to do this, I need to know current index and N:
var axes = R.map.idx(function(vertex, i, vertices) {
    return Vector(
      vertices[i],
      vertices[(i + 1) % vertices.length]
    );
});

I want to compute its axes, vectors between two vertices (N, N + 1).

treating elements of array-like objects in pairwise fashion is the argument I usually see for the index parameter. And it's true that without it, pairwise iteration is awkward without the index parameter.
So I guess this issue is another incentive to fill that gap ...

@honzabrecka, I would write it this way:
var axes = R.compose(R.map(R.apply(Vector)),
                     R.converge(R.zip,
                                R.identity,
                                R.converge(R.concat,
                                           R.skip(1),
                                           R.take(1))));

I defined the following function to allow me to test this in the REPL:
function Vector(x, y) { return [x, y]; }

@honzabrecka:
I saw what you were looking for earlier in the day, far from my computer, and came up with a solution almost identical to @davidchambers' version.  But as mine is broken down into distinct parts, it might be worth seeing independently:
var rotate = R.converge(R.append, R.head, R.tail); // [a, b, c, d] => [b, c, d, a]
var pairs = R.converge(R.zip, R.identity, rotate); // [a, b, c] => [[a, b], [b, c], [c, a]]
var axes = R.pipe(pairs, R.map(R.apply(vector)));

This is for the case as David suggested and vector is a plain binary function.  If it's a 2-arg constructor, you could use this instead:
var axes = R.pipe(pairs, R.map(R.apply(R.construct(Vector))));

Again, this is functionally no different from David's version, but obviously the code is structured a bit differently.

Naming the components is nice. I don't like using R.head, though, due to the empty list case. R.zip([], [undefined]) gives [], which is what we want, but it's cleaner to use R.take(1) and R.concat so we always apply R.zip to lists of equal length. :)


wxpython and vs 15.9WxPython

I just installed vs 15.9, and my auto complete for wxpython is still not working.  is there anything I can do to get this working again?  I need to do some serious coding w/ wxpython and not having intelisense for wxpython is really slowing me down.  I'm using vs pro 2017 15.9, python 3.7 and wxpython v4.0.3

I've just tested with https://wxpython.org/pages/overview/#hello-world
I'm not getting completions on wx. due to a timeout fetching results. I can prevent the timeout when debugging PTVS and then I see the results.
I'm getting completions on the objects I create. For example:
app = wx.App()

frm = wx.Frame(None, title="Hello World")

I get completions on both app. and frm.
Please let me know if your results are different from mine.
Performance of code analysis is being worked on, as part of Visual Studio Code for now, and we'll integrate those changes back into Visual Studio later for dev16.
I checked with Visual Studio Code Python extension October release, with "python.jediEnabled": false setting to ensure I'm using our Python language server. It feels like the results of wx. come back more quickly than in VS, though I'm not sure if it's quick enough to not trigger a timeout in Visual Studio. FYI they do not timeout in Visual Studio Code, it's all asynchronous.
We do want to switch to the asynchronous IntelliSense API which will allow us to get rid of the timeout and list results when they come back without impacting typing latency
This involves doing either https://github.com/Microsoft/PTVS/issues/4371 or https://github.com/Microsoft/PTVS/issues/4740.
In the meantime, I'd recommend you give Visual Studio Code a try.

I just tested this, and I concur with your findings as well.  I am seeing the same behavior

I accedently closed this, reopening it since wx is still not working properly.


Added Meme Maker by @Meme-Maker

[x] Project URL:
[x] Update contents.json instead of README
[x] One project per pull request
[x] Project added at the end of the list (a script sorts projects automatically)
[x] Avoid iOS or open-source in description as it is assumed
[x] Use this commit title format if applicable: Add app-name by @github-username
[x] Use approved format for your entry

Thx

TODO

add license
fix redirect


vue-query suspense always throws render errors

Describe the bug
I am using vue-query in SSR with suspense. I expected fetch errors to be serialized into error as described in https://tanstack.com/query/v5/docs/vue/guides/query-functions#handling-and-throwing-errors. Instead, a fetch error causes a render error. I set throwOnError: false according to https://tanstack.com/query/v5/docs/vue/reference/useQuery#~:text=throwOnError:~:text=non-serializable values.-,throwOnError but the composable still throws a render error.
This behavior is by design according to https://github.com/TanStack/query/issues/5920#issuecomment-1695821107, but unlike React, Vue's Suspense is not an error boundary, see https://vuejs.org/guide/built-ins/suspense.html#error-handling, so this behavior is problematic.
Edit: #5976 describes the same problem for Nuxt and a workaround is described in https://github.com/TanStack/query/discussions/5688#discussioncomment-6652179.
I created a minimal reproducible example which demonstrates that a query always triggers an uncaught error when data is loaded via suspense.
Your minimal, reproducible example
https://codesandbox.io/p/devbox/rough-sky-xlvpwl
Steps to reproduce

Open the codesandbox and wait until the app is compiled
The alert appears, showing that the component has thrown an uncaught error

Expected behavior
There should be an option to prevent errors being thrown when loading data via suspense()
How often does this bug happen?
Every time
Screenshots or Videos
No response
Platform
all platforms, see codesandbox
Tanstack Query adapter
vue-query
TanStack Query version
v4.8.4, v5.13.4, v15.15.0
TypeScript version
No response
Additional context
No response

@DamianOsipiuk can you triage this please?

I think this is still an issue, I took OP's repro and simply updated the vue-query package to 5.59.20 and it still reproduces the problem.
https://codesandbox.io/p/devbox/determined-borg-h2dmmj?file=%2Fsrc%2FPost.vue%3A10%2C33

bump


New icon

The Android icon isn't nice for an app.

here you go lmao something temporary until something better comes along

(does it bother anyone else that paint can't save with a blank background)

@TacoTheDank thank you very much! GIMP was able to correct the alpha channel.

It's a very nice icon that makes the app unique.

lol i mean its only supposed to be temporary i suppose, but if you want to actually use it for the icon then okay i guess

@TacoTheDank aybe it will change at some time, but it's great for now.

I think if you reject the pull request, abdulmalikkhan655 won't get his utopian reward. I'm sure he'd be unhappy about that. That utopian stuff is supposed to be moderated anyhow. If they are doing their job, they are gonna discover this thread (and others like it) sooner or later, and hopefully clean house.

Hey, I’m sorry for the inconvenience. I’ve just read the previous issue. It also made me feel a little bit sad. Blaming a project just for its users is… I don’t know, didn't make much sense to me.
I’m often checking GitHub issues with keywords such as “logo”, “design” etc. However, our actual “job” starts after a design contribution successfully merged into the project. Then users create their posts on Utopian so we are able to check if it’s plagiarised or not. Unfortunately, there is no other way to predict this beforehand. If you have any idea/solution, I’d like to pass those to project's owners.
As you already did your research and found out that we have quite strict rules. We are trying hard to keep this place clean and incentivize to open source as much as possible at the same time.
And yes our userbase not full of “open source minded” people, yet we are also trying to teach and make them love the OS. Our funds come from cryptocurrency so the users, we are trying to reach GitHub users as possible as we can.
Lastly, sorry my English is not the best, either. I didn’t think that Open Source is only open to native English speakers.
Regards,
Anil.

@anilkilic It's ok, I understand. There's only so much they can do to monitor it. :) (and your English is quite good btw)

@TacoTheDank why don't you do Report abuse from here? But be careful with your words. They can easily block you.
@anilkilic he @waqasanwar94 (now deleted account) has even said in dnet/adsdroid#22 that he has made the icon for MaterialFBook (it's just in root of the tree), while the commit was made by @abdulmalikkhan655 account. Such things happen when one becomes nervous (in this case that reward).

its funny, hes going around calling me a scammer in issues ive commented in (copy pasting it too lmfao)
he even pasted one in my own repository KEK

@TacoTheDank I know, that's why I suggested to use Report abuse.

thats what im doing lol, i just need to get all the links and stuff together

but theres a LOT, and i may not remember to include everything that he's been in, including his fake waqasanwar94 account

@TacoTheDank according to his Google account name, the @abdulmalikkhan655 account is the fake one, although he has deleted and re-created the other one.

🤔 this whole thing is a circus, its going to take a while to get everything lmao

@TacoTheDank https://plus.google.com/s/Waqas Anwar/top

wat

Thanks for your understanding, both accounts (waqasanwar, abdulmalikkhan as their steemit usernames) were banned.
Here is the the part will look like advertisement but this is our discord channel if you want to join and see what’s actually going on [https://discord.gg/uTyJkNm] . It’s not your duty for sure but you can also send us a ticket when you face this kind of things or about any matter [https://support.utopian.io/].

oh ok cool, sure i'll join
its certainly more real-time than github threads 😂

also i reported his account to github around 6 hours ago, so hopefully ill get something back soon

lol looks like someone else noticed too

@anilkilic im @Taco#1190 btw

@TacoTheDank all the people say that issues aren't possible to delete, but #3 has gone... has the support deleted it?

It happens when users get banned from GitHub, I suppose #3 was created by @abdulmalikkhan655

i think it was yea


[AIRFLOW-855] Replace PickleType with LargeBinary in XCom.

PickleType in XCom allows remote code execution. In order to deprecate it without changing mysql table schema, change PickleType to LargeBinary because they both maps to blob type in mysql.
Adding "enable_pickling" to XCom set, get_one, get_many signatures to allow switching  between Pickle type and JSON. Use JSON to serialize/deserialize can fix remote code execution issue.  "enable_pickling" should be added to core section of airflow.cfg as a boolean setting.
Throwing informative error msg when incompatible value passes in. For example "enable_pickling" = False but a non JSON object passes in.
Pickling should be removed in airflow 2.0 release.
Please accept this PR that addresses the following issues:

https://issues.apache.org/jira/browse/AIRFLOW-855

Testing Done:

Added tests to test basic functionalities on set, get_one, get_many.

Added test to test if non JSON obj failed to serialize when disable picking.

@aoen @saguziel

Is this postgres compatible?

LGTM (reviewed and iterated on this PR internally with amaliujia/saguziel), but I would get @jlowin's stamp since he is the XCOM expert.

@amaliujia I think you need to add the option explicitly to configuration.py so it isn't a hidden parameter. As written I believe this will crash any DAG that currently uses XComs because enable_pickling will be None and the configuration check will raise an error since the option doesn't exist.
At a minimum, you'll need to add comments to UPDATING.md that make it very clear that this is a breaking change and users coming from 1.8 to 1.9 must enable pickling by default to maintain compatibility.

Please note that we were contemplating purging pickling with @mistercrunch.

@bolkedebruin Based on Sqlalchemy documentation (http://docs.sqlalchemy.org/en/latest/core/type_basics.html):
PickleType builds upon the Binary type to apply Python’s pickle.dumps() and impl alias of LargeBinary.
Sqlalchemy should have same underlying implementation on a specific DB. I think it should be compilable on Postgres.

@bolkedebruin I am probably going to repick up this PR, just a heads up that I think the pickling you and Max wanted to remove was probably the pickled dags, not the pickled xargs (correct me if I'm wrong!).

Is there a way to get xcom to output to jsonb for postgresql?   I tried changing the datatype of value to jsonb.  What happened with this ticket?
CREATE TABLE xcom (
id integer NOT NULL,
key character varying(512),
value jsonb,
"timestamp" timestamp without time zone NOT NULL,
execution_date timestamp without time zone NOT NULL,
task_id character varying(250) NOT NULL,
dag_id character varying(250) NOT NULL
);


TextBuffer::onDidReload never fires for external changes

Prerequisites

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

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

Description
TextEditor::onDidReload appears to not be firing, at least within the 10 second timeout of package specs.
Steps to Reproduce

Check out<EMAIL_ADDRESS>Run the package specs
Note that two specs are failing.

Expected behavior:
TextEditor::onDidReload fires when the file is reloaded due to the external process changing the file.
Actual behavior:
The specs are expecting a notification that the fix was complete from ESLint (which triggers), as well as Atom declaring via TextBuffer::onDidReload that it has finished reloading the file after the external ESLint process has finished changing the file here.
Reproduces how often:
100% of the time.
Versions
Using Atom version:
Atom    : 1.19.0
Electron: 1.6.9
Chrome  : 56.0.2924.87
Node    : 7.4.0
Using APM version:
apm  1.18.2
npm  3.10.10
node 6.9.5 x64
python 2.7.11
git 2.13.0

Tested on Windows 10 x64, and Ubuntu 14.04 x64.
Additional Information
Possibly related to the problem in https://github.com/atom/atom/issues/14189, likely caused by https://github.com/atom/atom/pull/14435.
This is a regression from Atom v1.18.0, where these specs worked properly.

Ok, for posterity, the problem was only in the tests. The code that fires the onDidReload event now relies on the built-in Date.now method, which is mocked by default in tests.

If you run into this issue in your own specs, you can work around it by adding jasmine.unspy(Date, 'now') to the beforeEach of your spec to disable the mocking of Date.now. Assuming of course that your specs don't rely on that mocking. If your specs do rely on that, you should be able to get this to work still by calling advanceClock.


Allow more flexibility of umbrella headers in C++ Interop.

Description
Hello SwiftPM team,
First off, C++ introp included with Swift 5.9 is a major success, after successfully building much of the computer graphics landscape in the pursuit of building Pixar's USD for the Swift programming language with the SwiftUSD project.
This is a project I'd like to tie into the official OpenUSD project, however, due to the existing directory structure of OpenUSD, SwiftPM imposes the restructuring of the entire project as you can see in the SwiftUSD project.
The reason this restructuring is necessary is due to the rule that all Swift target headers must not be visible to other Swift targets with the publicHeadersPath field. However, I believe that even if a shared "root" include path is used, we can still uniquely identify the paths to each target's respective "contextual headers" within a shared "root" include path.
Is there any possibility of adding more flexibility to how SwiftPM accepts these headers, one that would allow for the directory structure of OpenUSD as well as other project directory structures as seen across the ASWF landscape - I believe with this feature we can see rapid (or, shall I say swift 😉) adoption of both the Swift programming language, and the power of SwiftPM across the industry.
Thank you,
Tyler Furby
Swift & OS version (output of swift --version && uname -a)
swift-driver version: 1.87.3 Apple Swift version 5.9.2 (swiftlang-<IP_ADDRESS>.56 clang-15<IP_ADDRESS>.5)
Target: arm64-apple-macosx14.0
Darwin wabitwo.local 23.2.0 Darwin Kernel Version 23.2.0: Fri Oct 13 09:27:54 PDT 2023; root:xnu-10002.60.54~14/RELEASE_ARM64_T6000 arm64

Do you have a more concrete (ideally smaller) example of which header structure currently doesn't work?

Do you have a more concrete (ideally smaller) example of which header structure currently doesn't work?

Sure, I will provide a simple example repository with a Package.swift, to replicate that Swift targets are not allowed to have a shared include directory visible to other swift targets.

@neonichu Here is an example repository, when running swift build, the following error message says the following:
error: 'helloswiftpm': target 'HelloSwiftPM' has invalid header layout: umbrella header found at '/Users/furby/Wabi/HelloSwiftPM/HelloSwiftPM/HelloSwiftPM.h', but directories exist next to it: /Users/furby/Wabi/HelloSwiftPM/HelloSwiftPM/SayHello, /Users/furby/Wabi/HelloSwiftPM/HelloSwiftPM/SayHelloAgain; consider removing them
error: 'helloswiftpm': target 'SayHello' has overlapping sources: /Users/furby/Wabi/HelloSwiftPM/HelloSwiftPM/SayHello/SayHello.cpp
error: 'helloswiftpm': target 'HelloSwiftPM' has invalid header layout: umbrella header found at '/Users/furby/Wabi/HelloSwiftPM/HelloSwiftPM/HelloSwiftPM.h', but directories exist next to it: /Users/furby/Wabi/HelloSwiftPM/HelloSwiftPM/SayHello, /Users/furby/Wabi/HelloSwiftPM/HelloSwiftPM/SayHelloAgain; consider removing them
error: 'helloswiftpm': target 'SayHello' has overlapping sources: /Users/furby/Wabi/HelloSwiftPM/HelloSwiftPM/SayHello/SayHello.cpp

Would you be able to show me a module.modulemap file that would work for this example? When I set it up I receive an error that 'helloswiftpm': target 'SayHello' has overlapping sources

I have updated the module maps with the closest I was able to get it compiles just fine and it runs, however, I am unable to refer back to the root include directory of HelloSwiftPM for the other C++ target, as the paths must all be stripped down to their own respective modules.

Here is that revision:
@neonichu https://github.com/furby-tm/HelloSwiftPM/commit/e50d8770fdda538bded5c515284d46186a684b72

As another attempt I was able to get things to work with everything sharing that root prefix, but it feels like more of a hack that requires moving all project directories into a secondary root directory of the same name (and the reason a module.modulemap file cannot go in the repository root is because SwiftPM will pick up on the other root level directories such as .git and not allow it):
@neonichu
https://github.com/furby-tm/HelloSwiftPM/commit/bf0b7a890de71f59002ad7eafd64a10130f2e773

@furby-tm you could also try to make the USD target a systemLibrary in SwiftPM, that would make SwiftPM a bit more relaxed when it comes to the paths of headers.

@egorzhdan Thanks! I'll take a look at systemLibrary as well, I'm awfully close in that last revision though to get exactly what I need, I just need to be able to refer to that top level HelloSwiftPM directory without having to create that secondary HelloSwiftPM within it, do you think I'm about at the limits with that approach?
Im thinking there's got to be a way that I can slap a module.modulemap at the repo root, because that should fix it, but I wasn't sure if you've ever seen or attempted to have a module map at the repositories root before.

@egorzhdan In perhaps the strangest of ways, I was able to bring the "root include" in by setting one of the libraries path's (with it's public headers directory) to the root of the swift package repo, now when other targets depend on it - that target includes the root.
It's real ugly! But I think this will work, any possible side effects of doing this?
https://github.com/furby-tm/HelloSwiftPM/commit/081f988f918dce3199b37ac8711064a61b90868f

Hey folks! I really appreciate your help, currently I am able to build https://github.com/wabiverse/OpenUSD, no changes from the official repo structuring, and things are building great with this Swift Package, it required the strange workarounds that I noted above, but I'm just glad to see everything working!
~ Keep up the great work with SwiftPM 🎉

Quick update with the above workaround, after spending more time with it and messing around with it a bit more, that does not seem to be a suitable fix for large projects with dependencies due to modules getting re-imported and those modules additionally getting re-imported into other dependencies' namespace blocks, sort of turning into a gigantic mess.
It appears all SwiftPM is missing here is the ability to support building CXX interop targets without a "project source directory" usually marked by src; or in Swift project convention a Sources directory.
For some reason, SwiftPM picks up on directories such as .build and .git at the root of the project repo residing next to a CXX interop target path at the root of the project repo if you attempt to build a CXX interop target with a target configuration of: path: ".", complaining that you should "consider reducing the number directories to one".

Closing in favor of #7413


Salsa type inference should find or types

Testing #2218
function a1() {
    return 5;
}

function a2() {
    if (1) {
        return 5;
    } else {
        return '5';
    }
}

Hovering over a1() shows its return type is number. Hovering over a2() shows its return type is any. IMHO it should show number|string -- maybe this inference code was written before or types were introduced?

This issue was moved to Microsoft/TypeScript#6663


[AutoDiff] NFC: Refactor TangentSpace

Refactors TangentSpace so that it uses shorter case names like "vector", "struct" and "enum" and ASTContext::getTangentSpace uses TypeBase::getAutoDiffAssociatedType to get the tangent space. I'll start another patch to move ASTContext::getTangentSpace under TypeBase.

@swift-ci please test tensorflow


Compilation Error: cannot find symbol class Int64

When generating the API from an openApiSpec that uses scalar int64 values, the following code is generated:
requestInfo.setContentFromScalarCollection(requestAdapter, "application/json", body.toArray(new Int64[0]));
Int64  does not exist in java. This should probably be new Long[]

Hi @NetzwergX
Thank you for using kiota and for reaching out.
Can you provide an example OpenAPI description here please?

@baywet The description I am using is https://esi.evetech.net/_latest/swagger.json?datasource=tranquility

Thank you for the additional information.
Apparently all instances are type integer format int64 in this description.
But according to the registry it should instead by type number (or string).
Can you try editing a local copy and generating again to see if that's the root cause?

Well, when looking at OAS 3.1.0, §4.4 Data Types, I find the following table:

type
format
Comments

integer
int32
signed 32 bits

integer
int64
signed 64 bits (a.k.a long)

number
float

number
double

string
password
A hint to UIs to obscure input.

This also seems to have been the case in OAS 3.0.

Thank you for the additional information.
Interesting, the format registry probably needs to be updated then.
The JSON schema spec would also support the case you're making
The mapping is already implemented in the core generation engine
Which would mean the issue actually comes from the writer which is missing a call to the convention service to get the actual type name like this
conventions.GetTypeString(requestParams.requestBody.Type, codeElement)
Is this something you'd like to submit a pull request for provided some guidance?


http = ok, https doesn't work

Hi
I use docker-gen (container) with the official nginx container and nextcloud.
http acces to nextcloud is fine. But https doesn't work.
 The mydomain.com.key and mydomain.com.crt are stored under this path: /mnt/docker/nginx/proxy/certs
Thanks.

The template from docker-gen does not contain the SSL part. Only the template from (nginx-proxy)[https://github.com/jwilder/nginx-proxy/blob/master/nginx.tmpl] has this.
Also, you don't need to mount the nginx conf.d volume to a persistent location. It's contents are always generated. You just need to name the volume.

This repo should no longer handle issues or discussions related to nginx-proxy / the nginx-proxy template, unless there is something we need to change / fix in docker-gen in order to allow a change / fix in nginx-proxy.


Relatinoship between SuSiE and SuSiEGLMM

Hi there,
I'm interested in using this implementation of SuSiE in my research pipeline, but I'm hesitant about using it if it's someone's side project. Is this project actively updated to keep up with SuSiE's implementation and bug fixes?
-Joe

@JosephLalli this is an experimental package and not meant for production now. I'll move it off the public radar to avoid confusions. We'll bring it back when ready. Thank you!


#1499 remove server-configuration-user-interface module

Signed-off-by: bogdan-sava<EMAIL_ADDRESS>

yes. next PR, if this one goes to master.

I think it's a good idea to split out too -- as there will be changes affecting packaging / containers too and we should merge into master close in time or will break UI in containers


Flyway doesn't pick up migrations in a native image

The following exception has been thrown on startup inside a native image:
io.micronaut.context.exceptions.BeanInstantiationException: Bean definition [javax.sql.DataSource] could not be loaded: Unable to obtain inputstream for resource: db/migration/V1__Test.sql

Cannot reproduce, the issue disappeared without any change :/


Fix error wording when @kubernetes:Service is used on an non anonymous listener of service.

Purpose

Fix error wording when @kubernetes:Service is used on an non anonymous
listener of service. Fixes https://github.com/ballerinax/kubernetes/issues/268.

Codecov Report

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

@@            Coverage Diff            @@
##             master     #286   +/-   ##
=========================================
  Coverage     77.18%   77.18%
  Complexity      655      655
=========================================
  Files            56       56
  Lines          2845     2845
  Branches        290      290
=========================================
  Hits           2196     2196
  Misses          514      514
  Partials        135      135

Impacted Files
Coverage Δ
Complexity Δ

...ernetes/processors/ServiceAnnotationProcessor.java
81.66% <ø> (ø)
14 <0> (ø)
:arrow_down:

Continue to review full report at Codecov.

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


move repo azure-sdk-pipeline to azure-sdk-tools

This PR moves the codes in https://github.com/Azure/azure-sdk-pipeline/tree/dev to this repository.

/azp run


[PE-D][Tester C] Minor UI bug: app title is still same as AB3

Don't forget to change the app title to TASsistant before the actual PE. This can be done in MainWindow.fxml.

Labels: type.FunctionalityBug severity.VeryLow
original: jingjie88/ped#14

Changed to TAssistant


chatgpt for slidev

Is your feature request related to a problem? Please describe.
sometimes chatgpt generated slide is not good enough.
Describe the solution you'd like
make a slidev GPTS in GPT store or create an extension for GitHub copilot

Feel free to build your down GPTs. If they work well, we'll be happy to link them in the docs. Closing this for now as we don't have a plan to provide that officially.


Document "tags" for compound words

What is the best way to express a compound word like "entity framework" in the tags section?

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

ID: 06a19d7d-253d-f17f-6421-624c9867473d
Version Independent ID: cc37eb17-f715-080c-7bfc-9eae868d50ba
Content: .nuspec File Reference for NuGet
Content Source: docs/reference/nuspec.md
Product: nuget
GitHub Login: @karann-msft
Microsoft Alias: karann

@SidShetye Is this question about whether spaces are OK, and/or capitalization? I'd recommend finding the official terminology and using that.
@karann-msft @nkolev92 in case I'm missing something here.

Hello, NuGet.org does not support compound words in a single tag. We recommend you create multiple tags like entity and framework. As a work around, you can add use a tag like entity_framework, entity-framework, or EntityFramework.

@Mikejo5000: This is more about what is best for discoverability. In other words, if authors want a positive hit for a "entity framework" user query, all things considered, what should be the input at the .nuspec (or equivalently, .csproj) file stage?
@loic-sharma : Are there plans to lift that limitation in the near future? If not, of the various suggestions you presented, what's the authoritative best recommendation given the current nuget.org indexing and search delivery results? Customers could speculate and experiment but I'm hoping this ticket helps document based on the actual workings.

Are there plans to lift that limitation in the near future?
I think this goes beyond nuget.org.
The NuGet pack experiences define the tags as a space delimited list.

While not saying never, a change like that would have a long tail.

@Mikejo5000: This is more about what is best for discoverability. In other words, if authors want a positive hit for a "entity framework" user query, all things considered, what should be the input at the .nuspec (or equivalently, .csproj) file stage?

You should make your csproj's tags something like entity;framework. This value will get split into entity and framework, which will match entity framework search queries.

Ok. Perhaps we can limit this to the best recommended given today’s limitations?


performance fixes for dmatrix construction using adapters

the parallel group builder isn't amenable for piece wise construction of a dmatrix
from batches of data. it relies on accumulating data on each thread and consolidating
at the end of it all. further, it also repeatedly resizes the internal vectors as and
when new data trickles in. the resizes are extremely expensive, if a row 'x' is attempted
to be added, where 'x' is large, and when there are little to no elements before 'x'
the piece-wise construction exemplifies this issue, when large batches of data are taken in.
the future batches with large row indices will assume that there are going to be data
coming in for older rows, and will try to make room for those, by carving out large
chunks of memory, only to be thrown away very soon when smaller batches of data are processed
with large row indices
this approach will control the state of the parallel builder by pre-allocating the thread
vectors and reusing them for every batch. it will further treat each batch as though it
is the entire data, and once the build is complete, it will consolidate the batch data and the
offset vectors into the dmatrix's data and offset vectors

@sriramch I will run some tests tomorrow at work, thanks for fixing so fast!


UPDATE MEMBERSHIP DETAILS

PR:
website: https://planetagro.org,
twitter: https://twitter.com/PlanetagroMX,
facebook: https://facebook.com/uniondeproductores
Contract details on etherscan: https://etherscan.io/token/0x2775f2a3c83bee1541d1d1bc308b3bb432b45151

All checks are complete and it meets metamask requirement

Inactive contract


Bump gce-proxy to 1.33.6

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


Usage Analytics MVP

Given that a user interacts with Taqueria, when they run a command via the CLI or the VS Code extension, then they should be counted as an active user.
Starting point
Read https://developers.google.com/analytics/devguides/collection/protocol/v1
BACKGROUND
Taqueria needs to implement an MVP for usage analytics to provide us with an indication of the number of Daily Active Users (DAU). The MVP will be limited in scope, defined as Daily Active Users, by CLI & VS Code Extension, and Taqueria version.
The anticipated solution for the MVP will be Google Analytics Measurement Protocol

<EMAIL_ADDRESS>https://github.com/NomicFoundation/hardhat/blob/master/packages/hardhat-core/src/internal/cli/analytics.ts
Oak HTTP library may be helpful: https://oakserver.github.io/oak/

DEFINITION
Daily Active User: A Taqueria user that runs a taq command via the CLI or command palette in a given 24hr period
TO DO

[ ] Opt-in messaging

ACCEPTANCE CRITERIA

[ ] Prompt for users to opt-in when running taq init
[ ] Ability to enable and disable app analytics using a taq command
[ ] CICD pipeline can be distinguished from a regular user or CICD pipeline opts out of analytics if it cannot be distinguished
[ ] Data tracked for:

[ ] Taqueria version
[ ] CLI or VS Code
[ ] Timestamp
[ ] User ID (to determine total and unique DAUs)

[ ] Data populates to datastore
[ ] UI shows data by different data points tracked

OUT OF SCOPE

Existing projects will be assumed to have opted out of telemetry
Opt-in persistence across projects

ADDITIONAL INFO

Environment variable can be used to disable tracking.

Also, here is a suggested HTTP library for Deno:
<EMAIL_ADDRESS>

Client ID generation example:
async function getClientId() {
  let clientId = await readAnalyticsId();

  if (clientId === undefined) {
    clientId =
      (await readSecondLegacyAnalyticsId()) ??
      (await readFirstLegacyAnalyticsId());

    if (clientId === undefined) {
      log("Client Id not found, generating a new one");
      clientId = uuid();
    }

    await writeAnalyticsId(clientId);
  }

  return clientId;
}

function getUserType(): string {
  return isRunningOnCiServer() ? "CI" : "Developer";
}

@mweichert, do you have a preference of using the Deno library mentioned above vs custom logic for building payloads and sending via standard HTTP requests?

Here's how I would go about implementing this:
Use GA4: https://developers.google.com/analytics/devguides/collection/protocol/ga4
Provide means of injecting tracker/measurement id in Taqueria

Configure setTrackerId to be a field of the SanitizedArgs type
Adjust bin/build.sh to include --setTrackerId to TAQ_TRACKER_ID env variable

Provide means of getting consent from the user

When taq is first executed, we check if consent was collected. We get this from localStorage.
If not prompted before, we prompt for consent. Consent is stored in localStorage

Add task of enabling/disabling tracking

Add two tasks, "opt-in" and "opt-out" to enable/disable tracking

Add event tracking

Emit events for when both internal and task-provided tasks are executed. We can use fetch() to do this. Include the parsedArgs / sanitizedArgs in the event parameters:
https://www.simoahava.com/analytics/implementation-guide-events-google-analytics-4/

Here's how we would use fetch for instance:
const measurementId = 'G-XXXXXXXXX';
const apiSecret = '<secret_value>';

function addCustomerTier(clientId, events) {

 // Request the customer tier from the CRM.
 const customerTier = getCustomerTier(appInstanceId);

 const queryParams = `?measurement_id=${measurementId}&api_secret=${apiSecret}`;
 fetch(`https://www.google-analytics.com/mp/collect${queryParams}`, {
   method: "POST",
   body: JSON.stringify({
     "client_id": clientId,
     "user_properties": {
       "customer_tier": {
         "value": customerTier
       }
     },
     "events": JSON.parse(events)
   })
 });
}

How will we ask for the initial consent from the user?

How will we ask for the initial consent from the user?

@russellmorton when someone runs taq, we check if consent was collected by looking up a key called "consent" in localStorage. If the value is "opt-out", we do nothing. If the value is "opt-in", we run Taqueria as usual but emit usage events. If the value is empty or doesn't exist, then we display:
"May we have your consent please for Taqueria collect anonymous usage details according to our terms & conditions: https://taquera.io/terms& conditions? [Y/n]"
If the response is "n" or "no" (case insensitve), then we set "consent" to "opt-out" in the localStorage; otherwise set it to "opt-in".

Should we loop in Neil for the prompt message?

Yes, I have given @neilmangan the heads up on this.

@russellmorton and @hu3man, this data collection will likely be governed by the ECAD Labs privacy policy. Our privacy policy is general and designed to be augmented with additional specific consents in contexts such as this.
Our request for consent will need to include or link to a short plain language description of the type of data and PII collected (eg. what kind of events? how much data about the events? do we collect data that would allow us or a third party identify the person submitting the information? etc.), a description of what we will do with the information and whether we will disclose it to any other parties.
Keeping the collected data to a minimum and providing clear up-front disclosures of what we are collecting (and not collecting?) will hopefully encourage more users to opt-in.
Do we have anything like this yet?

The implementation is still coming together but we start filling you in on what we have so far @neilmangan. There is also the standard data Google collects (IP, session, location, etc) but I'm not sure if that's relevant
For the MVP we are likely going to have 2 events:

taq_task_executed
taq_error

taq_task_executed
This will track each call to Taqueria (taq <taskName> command) and will provide metrics such as DAU/MAU
Fields collected:

timestamp (UTC)
taq_ui ('CLI' or 'VS Code')
taq_version (the version of the Taqueria binary installed)

taq_error
This event requires more discussion regarding what fields will be included. There was  talk of using the full output of --logPluginRequest for our error reporting, but @russellmorton brought up a good point that the file path would contain identifiable information so we may need to add some pre-processing of that data before sending the event
Currently, the payload is:
*** START Call to @taqueria/plugin-flextesa ***
node \
<EMAIL_ADDRESS>\
--taqRun \
proxy \
--i18n \
'{}' \
--config \
'{"language":"en","plugins":[{"name":"@taqueria/plugin-flextesa","type":"npm"}],"contractsDir":"contracts","testsDir":"tests","artifactsDir":"artifacts","environment":{"default":"development","development":{"networks":[],"sandboxes":["local"],"storage":{}}},"sandbox":{"local":{"accounts":{"default":"bob","bob":{"initialBalance":"3000000000","keys":{"alias":"bob","encryptedKey":"edpkurPsQ8eUApnLUJ9ZPDvu98E8VNj4KtJa1aZr16Cr5ow5VHKnz4","publicKeyHash":"tz1aSkwEot3L2kmUvcoxzjMomb9mvBNuzFK6","secretKey":"unencrypted:edsk3RFfvaFaxbHx8BMtEW1rKQcPtDML3LXjNqMNLCzC3wLC1bWbAt"}},"alice":{"initialBalance":"2000000000","keys":{"alias":"alice","encryptedKey":"edpkvGfYw3LyB1UcCahKQk4rF2tvbMUk8GFiTuMjL75uGXrpvKXhjn","publicKeyHash":"tz1VSUr8wwNhLAzempoch5d6hLRiTh8Cjcjb","secretKey":"unencrypted:edsk3QoqBuvdamxouPhin7swCvkQNgq4jP5KZPbwWNnwdZpSpJiEbq"}},"john":{"initialBalance":"4000000000","keys":{"alias":"john","encryptedKey":"edpktt6t2ENhxiQqun6bXPPWC6tFVvNPTDRh1gEPGX4BgDgbDnmGzP","publicKeyHash":"tz1Zwoh1QCVAvJ4sVTojMp9pLYp6Ji4NoZy6","secretKey":"unencrypted:edsk3Un2KRUUoWZufi914HQem96ejVFnkyD8GXRPUSkgqy5etsYXEN"}},"jane":{"initialBalance":"5000000000","keys":{"alias":"jane","encryptedKey":"edpkvS6TDSWcqqj3EJi3NRrCMyN7oNw1B3Hp37R19tMThqM8YNhAuS","publicKeyHash":"tz1aHUAC4oviwJuZF1EvVSvFz7cu9KMNYBph","secretKey":"unencrypted:edsk3UkdS1UBfTBkMAoFxmfdmUHmCdNhTzDJ1cSJuZhU1b6k6fZZoQ"}},"joe":{"initialBalance":"1000000000","keys":{"alias":"joe","encryptedKey":"edpkuT1QYPYbLLQz9dXhQS33ncsixxeGHbNGmntPTR4VBbWmskHPrV","publicKeyHash":"tz1MVGjgD1YtAPwohsSfk8i3ZiT1yEGM2YXB","secretKey":"unencrypted:edsk3Un2FU9Zeb4KEoATWdpAqcX5JArMUj2ew8S4SuzhPRDmGoqNx2"}}},"label":"Local Tezos Sandbox","protocol":"PtHangz2aRngywmSRGGvrcTyMbbdpWdpFKuS4uMWxg2RaH9i1qx","rpcUrl":"http://localhost:20000"}},"network":{},"configFile":{"value":"/Users/jimmychenchen/Desktop/test-flextesa/.taq/config.json"},"configDir":{"value":"./.taq"},"projectDir":{"value":"/Users/jimmychenchen/Desktop/test-flextesa"},"hash":{"value":"267ba805f53cebbcdd76a2fb1a00ad2d216c82a0327118f8a9453bdf19e0b6df"}}' \
--envVars \
'{}' \
--configDir \
'./.taq' \
--projectDir \
'./' \
--maxConcurrency \
10 \
--debug \
false \
--logPluginRequests \
true \
--fromVsCode \
false \
--setBuild \
'3c34134' \
--setVersion \
'v0.3.0' \
--help \
false \
--d \
false \
--p \
'./' \
--sandboxName \
'local' \
--sandbox \
'sandbox' \
--task \
'start sandbox'

Most of those fields seem benign to me and would be useful for error triage and investigation but there are a couple problematic fields there:

config - contains private keys and faucets for testnets
node - contains a full path containing identifiable information

The error event needs to be sanitized even further because the config file could be adjusted by the user in a number of ways, constituting additional PII, although the risk would be small. We should always adopt the approach of gathering nothing more than is required at any point.
I would like to think about the following split for errors:

Metrics to know that the error has occurred and we can log very basic, completely sanitized information for this using the GA implementation we have now.
Error Logging where we possibly request additional consent from the user to transmit potential PII to assist with debugging.

Having arbitrary config data just for counting errors creates more risk than problems it solves.

@russellmorton and @hu3man, this data collection will likely be governed by the ECAD Labs privacy policy. Our privacy policy is general and designed to be augmented with additional specific consents in contexts such as this. Our request for consent will need to include or link to a short plain language description of the type of data and PII collected (eg. what kind of events? how much data about the events? do we collect data that would allow us or a third party identify the person submitting the information? etc.), a description of what we will do with the information and whether we will disclose it to any other parties.
Keeping the collected data to a minimum and providing clear up-front disclosures of what we are collecting (and not collecting?) will hopefully encourage more users to opt-in.
Do we have anything like this yet?

We do not have content to use for this yet, I will draft something and share with the group.

@russellmorton, I was assuming we would just strip out those 2 fields (config and node)
If users reach out for support directly, we could then request their config.json file for 1 on 1 debugging or additional investigation

@hu3man, perfect. I added this to the acceptance criteria.

@jchenche, please add a property for OS - provided by Deno.build

@mweichert @jchenche @AlirezaHaghshenas how will consent be handled for the VS Code extension?

@russellmorton We can implement a GUI prompt for that.


Rule B006 cause autofix error

Ruff 0.0.287 (latest changes from main branch)
ruff  *.py --select B006 --no-cache --fix

file content(at least simple cpython script shows that this is valid python file):
def readLinkagesByLineFile(linkages_file,
                           column_mapping={
                           }):
    """
    """

error
/home/rafal/test/tmp_folder/861_IDX_0_RAND_97224242746230905011214.py:2:43: B006 Do not use mutable data structures for argument defaults
Found 1 error.

error: Autofix introduced a syntax error. Reverting all changes.

This indicates a bug in `ruff`. If you could open an issue at:

    https://github.com/astral-sh/ruff/issues/new?title=%5BAutofix%20error%5D

...quoting the contents of `/home/rafal/test/tmp_folder/861_IDX_0_RAND_97224242746230905011214.py`, the rule codes B006, along with the `pyproject.toml` settings and executed command, we'd be very appreciative!

python_compressed.zip

This occurs if there is not a newline at the end of a file after a docstring
error: Autofix introduced a syntax error in `example.py` with rule codes B006: invalid syntax. Got unexpected token ':' at byte offset 145
---
def readLinkagesByLineFile(linkages_file,
                           column_mapping=None):
    """
    """         if column_mapping is None:
        column_mapping = {}

---