PRs to include zap.store in app listings

Like awesome-android and others, for CLI as well.
Need to find them, list them and start sending PRs.
Also include other listings not on github/gitlab.
@alltheseas

talk about it in forums, reddit, discord, wherever people mention/like obtainium, fdroid etc

https://github.com/binaryshrey/Awesome-Android-Open-Source-Projects/

find, and share zapstore to android/open source resource lists

https://github.com/binaryshrey/Awesome-Android-Open-Source-Projects/
https://github.com/offa/android-foss
https://reddit.com/r/androidapps
https://albertomosconi.github.io/foss-apps/
https://apt.izzysoft.de/fdroid

https://github.com/binaryshrey/Awesome-Android-Open-Source-Projects/pull/56

https://github.com/offa/android-foss/pull/443

https://github.com/albertomosconi/foss-apps/issues/71

@franzaps are these the right instructions  for f-droid?
https://gitlab.com/fdroid/fdroiddata/blob/master/CONTRIBUTING.md#merge-requests

probably it is best to ask someone who has a reddit account with history to post on behalf of ZS

I'll ask around for people with good reddit karma

https://github.com/aljazceru/awesome-nostr/pull/471 submitted a PR to add zap.store to top of the awesome nostr list to facilitate for better app discovery on nostr, via nostr app!

@alltheseas maybe go over these links and: check they link to the new brand name and domain, and push to new ones

Please let's use "Zapstore" and no other variations (not zap.store, not Zap Store or Zap.Store)

@alltheseas Btw what you submitted to r/androidapps looks spammy. I would've taken it down too


GroupEffect.finish does not finish children.

To whoever fixes this: test that your solution works for sequences, especially pending sequences.


Support for native javascript eventListener.handleEvent()

Hi,
In javascript it is possible to use an Object as eventhandler.
See: eventListener.handleEvent() on MDN

listener
The object which receives a notification (an object that implements the Event interface) when an event of the specified type occurs. This must be an object implementing the EventListener interface, or a JavaScript function. See The event listener callback for details on the callback itself.

(Lol, a listener IS an object OR a function.... but the first is almost never explained... 🙄)
The great advantage for this that you do not need to use bind(this) etc, because when using an object as listener its context is kept. See this great article on this
But it does not work out-of-the-box here in OL. (Or do I overlook something)
Because a listener is called with call() in event/Target.js, and so this native feature breaks.
Of course an OL olObject.on(event, listener) is different then the native javascript event-listener. But I think it would be very nice to have this, and it can be done easily. 🙂
OL has handleEvent() functions in Interactions (and others?). As far as I can see, this will not conflict when the native handleEvent() would be implemented.
Example:
class MyEventHandler{
  _someProp = 'my values';
  handleEvent(e){
    alert(this._someProp);
  }
}

let oHandler = new MyEventHandler();

// initiate some interaction in OL
let oDraw = new Draw(options);
oDraw.on('drawend', oHandler);

// or on some htmlelement
let oButton = document.getElementById('myButton');
oButton.addEventListener('click',oHandler);

Nicer example
class MyInteractionWrapper{
  _someProp = 'my values';
  _oInteraction;

  handleEvent(e){
    alert(this._someProp);
  }
  constructor(){
    // initiate some interaction in OL
    this._oInteraction = new Draw(options);
    this._oInteraction.on('drawend', this);
  }
}

let oDraw = new MyInteractionWrapper();

NB!!
This ONLY works with addEventListener() and not with properties like object.onevent which only accepts a function. see mdn
But OL works only with addEventListener().
At the moment I can get this working, by adding method call(context,e) to the eventHander-object.
Like:
  call(context,e){
    // context is NOT used!
    return this.handleEvent(e);
  }

But that is not very nice.
One way to get this working with OL is changing the code in target.js like:
let fnPropagate;
for (let i = 0, ii = listeners.length; i < ii; ++i) {
     if(listeners[i].handleEvent){
        // do not enforce context for that would break the native functionality
	fnPropagate = listeners[i].handleEvent(evt);
     }else{
	fnPropagate = listeners[i].call(this, evt);
     }
     if (fnPropagate === false || evt.propagationStopped) {
         propagate = false;
          break;
     }
}

I do not know if this would create a conflict with the existing handleEvent() functions, but I don't think so.
I can do a PR, but need some help on getting the typedef right 😬
I really look forward to see your reactions on this.
Greetz, flexJoly
PS. I hope the code-examples have no typos, because I did not test these exact examples.

You could also just change a single line in target.js:
fnPropagate = listeners[i](evt);

That will make both functions and event listeners work.
I am not sure if we still need to call the listener with the target as this. But feel free to create a pull request. If no tests fail, it should be fine.

Thanks for your reply!
I am not sure if I understand you correctly. But I tried the following
        if (listeners[i](evt) === false || evt.propagationStopped) {
          propagate = false;
          break;
        }

This does not work, the this is needed as context for the listener in OL itself.
It break for example directly  this.getSource()  in handleSourcePropertyChange_.
So I tried:
      let fnPropagate;
      for (let i = 0, ii = listeners.length; i < ii; ++i) {
        if (listeners[i].handleEvent) {
          fnPropagate = listeners[i](evt);
        } else {
          fnPropagate = listeners[i].call(this, evt);
        }
        if (fnPropagate === false || evt.propagationStopped) {
          propagate = false;
          break;
        }
      }

This does not work also, for handleEvent() is only called directly by the user-agent, but not with some scripting in between.

When testing my original code, I got some typedef error.
In events.js I added:
/**
 * Listener Object. EventListener with function handleEvent()
 * As in: https://developer.mozilla.org/en-US/docs/Web/API/EventListener/handleEvent
 * When the function returns `false`, event propagation will stop.
 *
 * @typedef {Object} ListenerObject
 * @api
 */

And changed ListenerFunction typedef to:
/**
 * Listener function. This function is called with an event object as argument.
 * When the function returns `false`, event propagation will stop.
 *
 * @typedef {ListenerObject.handleEvent|function((Event|import("./events/Event.js").default)): (void|boolean)} ListenerFunction
 * @api
 */

But I am stuck here 😿 getting the error from test:
src/ol/events.js:31:14 - error TS2702: 'ListenerObject' only refers to a type, but is being used as a namespace here.

31  * @typedef {ListenerObject.handleEvent|function((Event|import("./events/Event.js").default)): (void|boolean)} ListenerFunction

Is the rationale behind your attempts only to avoid having to bind the function to this? If so, then ES6 provides a simpler solution than adding support for EventListener. Just use an arrow function, which will use the this from the function creator:
const that = this;
interaction.on('drawend', (e) => {
  console.log(that === this); // true
});

Hi,
No I like to have the eventListeners as a nice standalone blackbox object.
My background is old Cobol (1989) and very clean programming 👼
So I hate shortcuts like closures and arrow functions 😉 🤣
And as EventObjects are native javascript I thought it would be a good idea if this is possible in OL too.
Greetz, flexJoly

Ok, point taken. Since your suggested change does not add much code or overhead, we'd accept a pull request. Thanks in advance!

Only I do not know how to get past the typedef error as I mentioned above 😞

@flexjoly In events.js, just add EventListener as type to the ListenerFunction typedef:
/**
 * Listener function. This function is called with an event object as argument.
 * When the function returns `false`, event propagation will stop.
 *
 * @typedef {EventListener|function((Event|import("./events/Event.js").default)): (void|boolean)} ListenerFunction
 * @api
 */

In events.js, just add EventListener as type to the ListenerFunction typedef:
/**
 * Listener function. This function is called with an event object as argument.
 * When the function returns `false`, event propagation will stop.
 *
 * @typedef {EventListener|function((Event|import("./events/Event.js").default)): (void|boolean)} ListenerFunction
 * @api
 */

This gives errors too 😣


[OpenAI] Function calling API for Hermes-2-Pro

This PR supports Hermes-2-Pro-Llama-3 (hence also Hermes-2-Pro-Mistral) with function calling API. That is, user pass in functions in ChatCompletionRequest.tools, while the output would have tool_calls, which is parsed based on the model's string output.
Currently, we only support Hermes-2-Pro models. Internally, we populate the system message based on the official guide, and use json schema to guarantee the model's output is valid (an array of name: function_name, arguments: any, each representing a function call).
We also deprecate gorilla's support for function calling for now because we cannot use grammar to guarantee the validity of its output.
examples/function_calling is updated to reflect the usage. examples/json_schema's functionCallingExample() is still kept, as it offers users with more flexibility.

Output of examples/function_calling


MacOS - Make C64Analyser auto-populate a new GlobalConfig.json with a subfolder of ~/Documents

Make it the same as the Spectrum version


Add proper import for classified kind

This is my fault from ages ago I think, I forgot to add the import to the main index file.

You brought shame to your family, erik


cmd/vendor/github.com/google/pprof: TestSymbolizationPath failing on Plan 9

Since CL 37617, TestSymbolizationPath is failing on Plan 9.
--- FAIL: TestSymbolizationPath (0.02s)
	fetch_test.go:79: :/prod/path/binary:abcde10001, want /tmp/home918705075/pprof/binaries/abcde10001/binary, got /prod/path/binary
FAIL
FAIL	cmd/vendor/github.com/google/pprof/internal/driver	0.484s

See https://build.golang.org/log/a92f99b739e464ffb558519631cff96aca419355

See issue 100 and pull request 101 in pprof repository.


ApifyRequestQueueClient.batch_add_requests cannot handle more than 25 requests

This is an intentional limitation of the Apify API. Javascript client bypasses this, the python one doesn't.

Once this is done we can revert the https://github.com/apify/apify-sdk-python/pull/261 hotfix.


Feature Request:  Modded BIOS with Stock Voltages download option

Description:
Please add a "modded bios with stock voltages" download option for those whose hardware doesn't properly run with the lower voltage mod, even when running at stock clock speeds.
Current behaviour:
Sometimes (not always, but often enough to get annoying) I get a blue screen whenever trying to do something that involves using the NVIDIA GPU on Windows 10 when running the Modded BIOS with the lower voltages (it also did it with Kasar's custom BIOS with the lower voltages, but not with his modded bios with stock voltages). And I've never had an issue running Dell's stock BIOS either. I suspect my hardware just isn't good enough to reliably run at stock clock speeds with the lower voltages, so a modded bios option with stock voltages would be greatly appreciated as the feature set still works as expected on both Windows 10 and Linux (technically, Qubes OS, although I primarily use the Intel video card on that and I don't think it ever accesses the NVIDIA card).
Expected behaviour:
No blue-screens doing normal tasks!
Steps to reproduce the problem:

Flash modded bios with lower voltages on my hardware (either this BIOS or Captain Kasar's versions) with stock clock speeds.
Try to do something with the NVIDIA video card (ex. watch movie, play game, etc).
Blue screen (sometimes).

Branch(es):
Master
Modded BIOS version: Happens under 1.01 and 1.03 (I never flashed 1.02)
GPU model: Geforce 540m on a DELL XPS 15 L502X
Display driver version: NVIDIA 378.92
Operating system: Windows 10 Pro 64-bit
Any related software which may cause the BIOS to act strangely:
Anything that tries to access the NVIDIA GPU in Windows 10, but other than that, the occurrence appears to be random.

Thanks for your input.
In the next release, I will remove the original Dell BIOS, and thus only offer to flash the modded BIOS (the current one with lowered voltage and a "stock" one with voltage untouched.
Of course for people who want to revert back the original Dell BIOS, they will still be able to dowload it on Dell's website, but as it contains security/privacy issues (like the Computrace's Rootkit), I don't want to support it anymore.

Original request done in 6bbfaedab53437f923ba71a743b6996bb6cdb792.


enhancement enhance aggregate funtion collect and add group_array aliases

Proposed changes
Issue Number: close #13982
Problem summary
Enhance aggregate function collect_set and collect_list to support optional max_size param, which enables to limit the number of elements in result array.
Checklist(Required)

Does it affect the original behavior:

[ ] Yes
[x] No
[ ] I don't know

Has unit tests been added:

[x] Yes
[ ] No
[ ] No Need

Has document been added or modified:

[x] Yes
[ ] No
[ ] No Need

Does it need to update dependencies:

[ ] Yes
[x] No

Are there any changes that cannot be rolled back:

[ ] Yes (If Yes, please explain WHY)
[x] No

Further comments
Regression tests will be added soon.

TeamCity pipeline, clickbench performance test result:
the sum of best hot time: 35 seconds
load time: 639 seconds
storage size:<PHONE_NUMBER>2 Bytes
https://doris-community-test-1308700295.cos.ap-hongkong.myqcloud.com/tmp/20230101122524_clickbench_pr_72329.html

run buildall

run p0

run p1

run p1

run feut

Regression .out file has not been modified, waiting for ipv6 bug in master fixed and then regression test is able to run on the local env.

run buildall

run p0

run p1

Hi, Does the case in this pr need to add some order by to ensure that the results are always consistent?
I'm running into something that looks like it's sorting related.
2023-02-28 16:58:32.363 ERROR [suite-thread-3] (ScriptContext.groovy:121) - Run test_aggregate_collect in /doris/regression-test/suites/query_p0/sql_functions/aggregate_functions/test_aggregate_collect.groovy failed
java.lang.IllegalStateException: Check tag 'select' failed:
Check tag 'select' failed, line 1, CHAR result mismatch.
Expect cell is: [1555555555,<PHONE_NUMBER>,<PHONE_NUMBER>, 255555555,<PHONE_NUMBER>5]
But real is: [1555555555,<PHONE_NUMBER>,<PHONE_NUMBER>, 255555555,<PHONE_NUMBER>5]

Hi, Does the case in this pr need to add some order by to ensure that the results are always consistent? I'm running into something that looks like it's sorting related.
2023-02-28 16:58:32.363 ERROR [suite-thread-3] (ScriptContext.groovy:121) - Run test_aggregate_collect in /doris/regression-test/suites/query_p0/sql_functions/aggregate_functions/test_aggregate_collect.groovy failed
java.lang.IllegalStateException: Check tag 'select' failed:
Check tag 'select' failed, line 1, CHAR result mismatch.
Expect cell is: [1555555555,<PHONE_NUMBER>,<PHONE_NUMBER>, 255555555,<PHONE_NUMBER>5]
But real is: [1555555555,<PHONE_NUMBER>,<PHONE_NUMBER>, 255555555,<PHONE_NUMBER>5]

It seems so, I will add order_by to ensure the stability.


[SCHEMA] add schema and spec version to object dumped by bidsschematools

These files may be passed around and have lives outside of readthedocs url. Having basic provenance info may save someone a headache in the future.

https://bids-specification--1224.org.readthedocs.build/en/1224/schema.json


Show also values from type of Date

Object values from type Date are ignored actually. This PR will fix this issue  #5.

Thanks!


Allow conflict-probability value of 0

For the use case of externally generated ids (no updates), allow
setting conflict-probability to the value 0.

Thanks @danielmitterdorfer !

@dliappis can we settle for either "bug" or "enhancement" for this PR?

@danielmitterdorfer I thought it contains a bit of both :) However, I realized there's something missing here, which are the checks in float_param so I guess I'll just label it as bug (fix) for now and make the follow up PR an enhancement.


Refactor codebase to async/await - spo site chrome set until spo site list, Closes #5104

Closes #5104

Thank you, we'll review it ASAP.

ready to merge 🚀

merged manually. Thank you for your awesome work👏


Change post auth URL template to use PROJECT_MEMBER_ID

Instead of OH_PROJECT_MEMBER_ID. This makes it consistent with the messaging template.
Once changed, check the post authorization URLs for existing projects on the site and fix if needed.

Just updated the 6 projects using OH_PROJECT_MEMBER_ID to PROJECT_MEMBER_ID.


[BUG] The definition of AppService in Swagger is inconsistent with the value returned by the Service.

API Spec link
https://github.com/Azure/azure-rest-api-specs/blob/35f8a4df47aedc1ce185c854595cba6b83fa6c71/specification/web/resource-manager/Microsoft.Web/stable/2021-02-01/CommonDefinitions.json
API Spec version
2021-02-01
Describe the bug

The return type of AnalysisType does not match.
Expcted: https://github.com/Azure/azure-rest-api-specs/blob/35f8a4df47aedc1ce185c854595cba6b83fa6c71/specification/web/resource-manager/Microsoft.Web/stable/2021-02-01/CommonDefinitions.json#L966
Actual:

The return type of DetectorInsightStatus does not match.
Expected: https://github.com/Azure/azure-rest-api-specs/blob/35f8a4df47aedc1ce185c854595cba6b83fa6c71/specification/web/resource-manager/Microsoft.Web/stable/2021-02-01/CommonDefinitions.json#L3003
Actual: 3
The return value of DetectorType does not match.
Expected: https://github.com/Azure/azure-rest-api-specs/blob/35f8a4df47aedc1ce185c854595cba6b83fa6c71/specification/web/resource-manager/Microsoft.Web/stable/2021-02-01/CommonDefinitions.json#L974
Actual : "WorkflowNode"

Expected behavior
The value and type returned by the service match those defined in Swagger.
Actual behavior
The definition of AppService in Swagger is inconsistent with the value returned by the Service.
Reproduction Steps
Invoke the GetAll() method in SiteDetectorCollection (version Azure.Resourcemanager.AppSerice1.1.0-beta.5).
Environment
.NET SDK:
Version: 8.0.200
Commit: 438cab6a9d
Workload version: 8.0.200-manifests.e575128c
Runtime Environment:
OS Name: Windows
OS Version: 10.0.22621
OS Platform: Windows
RID: win-x64
Base Path: C:\Program Files\dotnet\sdk\8.0.200\
.NET workloads installed:
There are no installed workloads to display.
Host:
Version: 8.0.2
Architecture: x64
Commit: 1381d5ebd2
.NET SDKs installed:
3.1.426 [C:\Program Files\dotnet\sdk]
5.0.416 [C:\Program Files\dotnet\sdk]
6.0.127 [C:\Program Files\dotnet\sdk]
6.0.321 [C:\Program Files\dotnet\sdk]
6.0.419 [C:\Program Files\dotnet\sdk]
7.0.406 [C:\Program Files\dotnet\sdk]
8.0.200 [C:\Program Files\dotnet\sdk]
.NET runtimes installed:
Microsoft.AspNetCore.All 2.1.30 [C:\Program Files\dotnet\shared\Microsoft.AspNetCore.All]
Microsoft.AspNetCore.App 2.1.30 [C:\Program Files\dotnet\shared\Microsoft.AspNetCore.App]
Microsoft.AspNetCore.App 3.1.32 [C:\Program Files\dotnet\shared\Microsoft.AspNetCore.App]
Microsoft.AspNetCore.App 5.0.17 [C:\Program Files\dotnet\shared\Microsoft.AspNetCore.App]
Microsoft.AspNetCore.App 6.0.26 [C:\Program Files\dotnet\shared\Microsoft.AspNetCore.App]
Microsoft.AspNetCore.App 6.0.27 [C:\Program Files\dotnet\shared\Microsoft.AspNetCore.App]
Microsoft.AspNetCore.App 7.0.16 [C:\Program Files\dotnet\shared\Microsoft.AspNetCore.App]
Microsoft.AspNetCore.App 8.0.2 [C:\Program Files\dotnet\shared\Microsoft.AspNetCore.App]
Microsoft.NETCore.App 2.1.30 [C:\Program Files\dotnet\shared\Microsoft.NETCore.App]
Microsoft.NETCore.App 3.1.32 [C:\Program Files\dotnet\shared\Microsoft.NETCore.App]
Microsoft.NETCore.App 5.0.17 [C:\Program Files\dotnet\shared\Microsoft.NETCore.App]
Microsoft.NETCore.App 6.0.26 [C:\Program Files\dotnet\shared\Microsoft.NETCore.App]
Microsoft.NETCore.App 6.0.27 [C:\Program Files\dotnet\shared\Microsoft.NETCore.App]
Microsoft.NETCore.App 7.0.16 [C:\Program Files\dotnet\shared\Microsoft.NETCore.App]
Microsoft.NETCore.App 8.0.2 [C:\Program Files\dotnet\shared\Microsoft.NETCore.App]
Microsoft.WindowsDesktop.App 3.1.32 [C:\Program Files\dotnet\shared\Microsoft.WindowsDesktop.App]
Microsoft.WindowsDesktop.App 5.0.17 [C:\Program Files\dotnet\shared\Microsoft.WindowsDesktop.App]
Microsoft.WindowsDesktop.App 6.0.26 [C:\Program Files\dotnet\shared\Microsoft.WindowsDesktop.App]
Microsoft.WindowsDesktop.App 6.0.27 [C:\Program Files\dotnet\shared\Microsoft.WindowsDesktop.App]
Microsoft.WindowsDesktop.App 7.0.16 [C:\Program Files\dotnet\shared\Microsoft.WindowsDesktop.App]
Microsoft.WindowsDesktop.App 8.0.2 [C:\Program Files\dotnet\shared\Microsoft.WindowsDesktop.App]
Other architectures found:
x86 [C:\Program Files (x86)\dotnet]
registered at [HKLM\SOFTWARE\dotnet\Setup\InstalledVersions\x86\InstallLocation]
Environment variables:
Not set
global.json file:
Not found
Learn more:
https://aka.ms/dotnet/info
Download .NET:
https://aka.ms/dotnet/download

Add another issue: https://github.com/Azure/azure-sdk-for-net/issues/37585
Request:
GET /subscriptions/<>/resourceGroups/<>/providers/Microsoft.Web/sites/<>/hybridconnection/<>?api-version=2021-02-01
Response:
{"id":"sdktest456","name":"Microsoft.Web/sites","type":"East US"}
The Type here is mismatch for resourceType. Or the whole response is incorrect.
Request:
GET /subscriptions/<>/resourceGroups/<>/providers/Microsoft.Web/sites/<>/hybridConnectionRelays?api-version=2021-02-01
Response:

The id here is null.


Mathews Story #8

@maliarich @Math-TECH-Hub @gibertgibs is the matter of the missing story solved?
I think we shall just add the pictures right?

yes of courses this can be closed


Update rldemo.js

Pretty sure this is what was intended. The graph doesn't appear to work correctly in the demo. This change makes the x axis match the age of the trained agent. Tested change and appears correct to me.

您好，我是薛文毅，已收到您的邮件，一会查收，谢谢！！


Scripts/notebooks to reproduce the results reported in the paper?

Are the scripts/notebooks to reproduce the results reported in the paper available? If yes, can they be added to this repository?
I see two functions in utils/utils.py to calculate recall/ndcg but that looks like computing for 1 query and not the aggregate score of the entire dataset.

Hi,
Thanks for your interest! I believe that the evaluation section of readme offered instructions on running the main evaluation numbers. Let us know if you actually mean something else and run into any problem!


feat: bump pg-meta (0.66.3) & studio (20230622)

https://github.com/supabase/postgres-meta/compare/v0.66.0...v0.66.3
Studio update includes omitting columns when querying tables/views/etc. - seems to work OK

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

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

Files with Coverage Reduction
New Missed Lines
%

internal/gen/keys/keys.go
7
7.94%

Totals

Change from base Build<PHONE_NUMBER>:
-0.07%

Covered Lines:
4745

Relevant Lines:
7678

💛  - Coveralls


all: create a Sonnenschein type to properly handle angle smearing

This CL creates a Sonnenschein with retained state for the smearing
angle.
This is in order to not generate the same pseudo-random value streams
for the angles' smearing.

Thanks, this type will also be useful to pass some of the option like the debug, smaring or number of iteration!

one thing to consider would also be to return a tuple of fmom.PxPyPzE instead of a slice.

Actually, at the very end, we might even want to define a new type containing all reconstructed four vectors:
type RecoKinem {
 t, tbar fmom.PxPyPzE
 b, bbar fmom.PxPyPzE
 wp, wm  fmom.PxPyPzE
 l, lbar fmom.PxPyPzE
 v, vbar fmom.PxPyPzE
}

and return it! This type could be common if other reconstruction method are implmented!


Prompt for asking permission

Coverage decreased (-0.04%) to 82.722% when pulling 99f67a86bda74cf47f9660e95e602f12f0663431 on development into e84bae4ebb62fdb27cfda386618ba07df2ba18ea on master.


notes on FastAPI and Docker

Put all notes together now that the API is working.

already included.


Unify logging made by DKG candidate members - remove obsolete space character

In almost all the logs printed by the DKG candidate members we use the [member:x]  prefix. But there's one log in which we have a space char before x:
[member: x] signature from sender [y] supports result different than preferred
Let's unify our notation and remove change the log to:
[member:x] signature from sender [y] supports result different than preferred

Actually, I grepped the files and found a couple more occurrences of the logs using [member: x] notation.
In keep-core/pkg/beacon/dkg/result/signing.go:
[member: %v] received multiple messages from sender: [%d]
[member: %v] signature from sender [%d] supports result different than preferred
[member: %v] verification of signature from sender [%d] failed: [%v]
[member: %v] sender [%d] provided invalid signature

In keep-core/pkg/tecdsa/dkg/protocol.go:
[member: %v] signature from sender [%d] supports result different than preferred
[member: %v] verification of signature from sender [%d] failed: [%v]
[member: %v] sender [%d] provided invalid signature


feat: added run number field to dataset and relative tests

Description
This PR adds the field runNumber to the high level fields of the datasets. The field is optional, but it has come up many times in our assessments with users that it makes sense to promote it to high level, making it easier to create the proper search functionalities.
Motivation
In our assessments with users, they have mention run number as a piece of information with high relevance.
So far it was included in the scientific metadata, but it had suffer from the variability introduced by the freedom provided by the field.
Promoting  run number to high level field, it will allow us to reduce such variability and provide a more consistent search functionality.
Changes:

dataset schema
dataset dtos
raw dataset tests

Tests included

[x] Included for each change/fix?
[x] Passing?

Documentation

[x] swagger documentation updated (required for API changes)
[ ] official documentation updated

:+1:


drop peer dependencies

Can you drop peer dependency on react-native in package.json and let the host project dictate which react-native version to use?
Social share is not using anything specific to any react-native version. It is safe to remove peer deps i think.

I think you are right, it should not need anything specific!
I just removed the peer dependency :+1:

Do you think I should make an update to version no + npm

Thanks. Yeah version bump is needed. maybe just call it 1.0.0 since it is stable now?

Just did a "minor" version bump :+1:

cool. thanks :+1:

on a side note: do you have plans for android support?

Nope no plans at the moment


Autoscaling node pools

How is autoscaling handled if a node pool has: instance_count: 2 and autoscaling:
autoscaling:
    enabled: true
    min_instances: 0
    max_instances: 3

Does that mean it will create 2 nodes and then autoscale to 0 if nothing is being ran on them or is instance_count the actual minimum?

If autoscaling is enabled then instance_count is ignored :)

That makes more sense, thanks!


Add some comments about LLM chunking

Used here for example:
https://arxiv.org/pdf/2410.03341
They just prompt a model to chunk a text to a certain number of tokens. To ensure that the output is consistent with this number, they fix the context length of the output to the number of tokens they desire the chunk to have

lol, again "just ask the LLM to do it"


GPU Accelerated Darknet

Hi again!
I have a bunch of edge devices with CUDA 9  (GPU) and ARM64 and I have successfully built Darknet/Yolo2 with GPU support.
Can I try out this project without modification? Will it simply use the GPU in this case if it doesn't detect NNPACk or do I need to make some changes?
Thank you,

Hi innerop,
Inference using GPU/CUDNN library is implemented in Darknet, and is controlled by GPU/CUDNN macros. You can build your Darknet in such way, or you can modify the top level Makefile in DeepThings to turn on the GPU and CUDNN options in Darknet:
 GPU=1
 CUDNN=1
 $(DARKNETLIB):
            $(MAKE) -C $(DARKNET) CUDNN=$(CUDNN) GPU=$(GPU) all

DeepThings doesn't require any specific inference features from libraries. So using GPU, NNPACK or whatever you need will still be compatible with current DeepThings implementation.


Check type annotations in CI

Fixes https://github.com/wagtail/wagtail-vector-index/issues/29
Pyright is configured to only run basic checks (typeCheckingMode = "basic"). It's not ideal, and there are a couple of places where I've added # type: ignore, but we can improve these over time.

Nice one thank you @mgax !


Fixed possible out of range byte value

Bytes need to be between 0 and 255, not 256

Coverage remained the same at 96.678% when pulling 7550a753565c0a1c899d36c2939b7b7e8039cd32 on alfredojf:byte-range-fix into ccb88785e2d4678b71304167d87dd912f8ca6b41 on lepture:master.

thanks.


[AzureMonitorExporter] prep exporter 1.3.0

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


HADOOP-11452 make rename/3 public

2735

 Quality gate: Failed
25 Issues

 1 Vulnerabilities
 24 Code issues

Details


ES Modules: Bolt Core / BoltElement Updates

Jira
N/A
Summary
Bolt Core renderer, decorator, and BoltElement-related Class updates that are all a required part of the broader ES Module work https://github.com/boltdesignsystem/bolt/pull/1579.
Details

Migrates existing component decorators over to use the new decorator spec when supported (using lit-element's approach as a starting point).
Migrates a handful of the most common Core utils over to the new @bolt/element base
Refactors withLitHtml, withEvents, withPreact base classes to directly extend classes (required otherwise blocking errors are thrown when all the ES Module-related work is switched on)
Forks (temporarily?) the wc-context package (the library used for our newest Context approach with minimal config required) in order to fix a breaking JS error encountered otherwise due to not checking if something exists first before calling super... https://github.com/boltdesignsystem/bolt/pull/1586/files#diff-cd43fc9b53340e11a3fb872caf823a6fR18

How to test
Unfortunately, like the updates that actually migrate our components over to use the new Class syntax, these updates can't be used in isolation (hence why I created https://github.com/boltdesignsystem/bolt/pull/1579 since most of the pieces required don't / can't work by themselves).
Similarly to what I mentioned in the polyfill loader PR, I'm honestly not sure we can practically ship this without proactively doing a major SEMVER bump, given that the Core / BaseElement updates must happen in conjunction with the polyfill, component Class migration, and build tool updates.
The only possibly I can think of to get around this would be to take all of the changes to core here + in the related PRs and add them to a brand new (temporary) @bolt/v3-core package that could live along-side the unmodified core package.
This approach would at least give us a guarantee that any v2.x written components would definitely work while we work and on start to ship broader v3.0-related work without any backward breaking changes...
Thoughts?

PR was released with v2.14.0


デバッグ用のecho

Description
環境変数の問題かなと思い、デバッグのために、echoしたいです。

Fixes # (issue)
Type of change

[ ] Document update or simple typo fix
[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)
[ ] This change requires a documentation update

How Has This Been Tested?

[ ] I have checked backward/forward compatibility that may cause regarding this change.

kubectl exec -it mysql-o11y-597f8cbdb4-dt8xq  /bin/sh
kubectl exec [POD] [COMMAND] is DEPRECATED and will be removed in a future version. Use kubectl exec [POD] -- [COMMAND] instead.
# mysqldump --single-transaction --quick --set-charset --no-tablespaces -u USERNAME -p'XXXXXXXXXXXX' -h dreamkast-dev-rds.c6eparu1hmbv.ap-northeast-1.rds.amazonaws.com dreamkast conference_days conferences proposal_item_configs proposal_items proposals talk_categories talk_difficulties talk_times talks talks_speakers tracks stats_of_registrants viewer_counts > $HOME/rds.dump
mysqldump: [Warning] Using a password on the command line interface can be insecure.
Warning: A partial dump from a server that has GTIDs will by default include the GTIDs of all transactions, even those that changed suppressed parts of the database. If you don't want to restore GTIDs, pass --set-gtid-purged=OFF. To make a complete dump, pass --all-databases --triggers --routines --events.
# ls
rds.dump

直接実行するとうまくいっているが、
kubectl logs mysql-o11y-27399910-5z7w4
mysqldump: [Warning] Using a password on the command line interface can be insecure.
mysqldump: Got error: 1045: Access denied for user<EMAIL_ADDRESS>(using password: YES) when trying to connect
mysql: [Warning] Using a password on the command line interface can be insecure.
ERROR 1045 (28000): Access denied for user<EMAIL_ADDRESS>(using password: YES)

CronJobの方はAccess denied

@riita10069
このechoだとだめっぽい


Disable inlining coroutines until the coroutine-splitting pass runs

Cloning the coroutine intrinsics generally yields invalid IR.

@swift-ci Please test.


withPartitionConstraint method on HiveSource doesn't return if partition doesn't exist

I have a partition key named title on a Hive/Parquet table.
I created data whose partitions are Mr and Mrs
If I use the withPartitionConstraint with the correct values then everything is ok.
However the following code doesn't return if I supply a partition value that doesn't exist:

    implicit val hadoopFileSystem = FileSystem.get(new Configuration())
    implicit val hiveMetaStoreClient = new HiveMetaStoreClient(new HiveConf())
    HiveSource("eel_test", "person")
      .withPartitionConstraint(PartitionConstraint.equals("title", "mr"))
      .toFrame().collect().foreach(row => println(row))

The above purposely sets the title all in lowecase, i.e. mr
Here's the DDL to create the table:

CREATE EXTERNAL TABLE IF NOT EXISTS `eel_test.person` (
   `NAME` string,
   `AGE` int,
   `SALARY` decimal(38,5),
   `CREATION_TIME` timestamp)
PARTITIONED BY (`title` string)
ROW FORMAT SERDE
   'org.apache.hadoop.hive.ql.io.parquet.serde.ParquetHiveSerDe'
STORED AS INPUTFORMAT
   'org.apache.hadoop.hive.ql.io.parquet.MapredParquetInputFormat'
OUTPUTFORMAT
   'org.apache.hadoop.hive.ql.io.parquet.MapredParquetOutputFormat'
LOCATION '/client/eel_test/persons';

Are you saying it just loops forever?

Yes that's what looks like is happening in my test code.

Fixed.


Thesis compiles

Coverage remained the same at 26.741% when pulling 1e24b6489dc02f5b1810aa5e9bc9db6b231a548f on nbgl:master into 5ba8da8ed6bcc6461436264a9d04dd38d7428432 on chengsoonong:master.


Include automated_signing in signing API response

r? @kumar303

I dislike this field name but it would be worse to call it something other than the real model field name :)
Can you adjust the docs? http://olympia.readthedocs.org/en/latest/topics/api/signing.html#checking-the-status-of-your-upload
r+wc

I removed the bit in the docs about needing to pass tests if unlisted also.

r+wc


[release-4.14] OCPBUGS-20710: mitigate effects of rapid reset

similar PRs for master were made in parts:

https://github.com/openshift/cluster-image-registry-operator/pull/938
https://github.com/openshift/cluster-image-registry-operator/pull/936

/retest

/retest

/retest

/test e2e-vsphere-operator

/retest

/retest

/jira refresh

/jira refresh

/label qe-approved

/label cherry-pick-approved

/label backport-risk-assessed

Still needs lgtm.

/lgtm


Added ability for Nautobot to send installation metrics.

Closes: #4047
What's Changed

Added new management command send_installation_metrics that is called by post_upgrade to send metrics to Nautobot cloud endpoint. Sample data:
{
    "deployment_id": "1de3dacf-f046-4a98-8d4a-17419080db79",
    "nautobot_version": "1.6.0b1",
    "python_version": "3.10.12",
    "installed_apps": {
        "3ffee4622af3aad6f78257e3ae12da99ca21d71d099f67f4a2e19e464453bee7": "1.0.0" # "example_plugin" hashed by sha256
    }
}

TODO

[ ] Explanation of Change(s)
[ ] Added change log fragment(s) (for more information see the documentation)
[ ] Attached Screenshots, Payload Example
[ ] Unit, Integration Tests
[ ] Documentation Updates (when adding/changing features)
[ ] Example Plugin Updates (when adding/changing features)
[ ] Outline Remaining Work, Constraints from Design

Looks like CI is failing because the base branch develop-1.6 needs the fixes merged from develop but I don't want to step on @grelleum's toes since he is already working on that merge.

I'm going to merge this but let's keep #4047 open until the code changes are in on the cloud side for us to validate the end-to-end flow against.


Invite People to sign up link, malformed link, running with reversed proxy

Describe the bug
Invite people to sign up for label-studio. Using the +Add People button under /people I get a wrong link. The link looks like  http://example.abc.com/labelstudio/labelstudio/user/signup/?token=12Csa . Meaning the route labelstudio is double.
To Reproduce
Steps to reproduce the behavior:

Go to Hamburger sign
Go to 'Organization'
Click on '+ Add People'
See the link http://example.abc.com/labelstudio/labelstudio/user/signup/?token=12Csa

Expected behavior
http://example.abc.com/labelstudio/user/signup/?token=12Csa
Screenshots

Environment (please complete the following information):

CentOS
Apache Httpd reverse proxy: Where the route /labelstudio goes to localhost:8080.
Label Studio Version [e.g. 1.0.2] within conda and the  --host "http://example.intern.com/labelstudio"

Comments
Label-studio works and when I remove the extra /labelstudio in the url, I can sign up.
On the server I see a GET /api/current-user/whoami request, when clicking on the '+ Add People' button.

@schafsam https://github.com/heartexlabs/label-studio/pull/1062 could you check this branch please, hope it's fixed there.

@makseq: I was able to test it now with the current master branch and it works.


Odd behavior when Terraform 'target' flag is used

A user has reported this issue.
User:
We've had to do some builds with -target set for terraform, and Infracost has reported cost decreases consistently, I would expect a cost decrease on the first build with a -target, but then future runs report the same decrease. I'm sure there's something I don't understand about it, and it's not a big deal. Just the only thing I could think to raise with you :)
Me:
Trying to understand the process you went through better:

you run terraform plan -target="my_resource" and you see a cost estimate comment showing a cost decrease. Were you expecting a cost decrease because of the change to my_resource or something else was being modified in the TF project?

You make changes to my_resource, and the comment is updated (or a new comment is posted?) showing the same cost decrease. But you didn’t expect to see this cost decrease comment?
Or are you seeing cost decrease comments even after the my_resource change was applied? But you expected to see no cost diff when re-running with -target=my_resource as the change has already been applied.

User:

expecting to see a cost decrease because we're now running with -target, yes
The cost decrease persist across several later PRs as well
Yes we saw the cost decrease after the change was applied with TF

The fix has now been released in v0.9.15.


bring back nose-ring

Hi, 👋
I recently noticed that the noise-ring library was deprecated because it didn't support ChaCha20Poly1305 and Aes256Gcm and was less useful. However, after going through the snow code, I found out that it was possible to support these encryption methods, so I created this PR.
Currently, Sensitive depends on noise_rust_crypto. Perhaps it could be moved into noise-protocol?
On my computer (M1 pro 10C), ring's throughput is 2.5 times that of rust-crypto. Additionally, symmetric encryption is a hot path in the noise protocol, so I believe including noise-ring is necessary.
👉👉👉 Take a look and let me know what you think! 🚀

rebase master and pass all test by test.sh

@sopium Sure, I'll make a copy first, but wouldn't it be better to move to noise-protocol?

done


Proposal: Detect Hacktoberfest PRs without requiring commit sign-off

Expected Behaviour

Possibility to check for Hacktoberfest spam PRs without requiring signed off commits
Current Behaviour

hacktoberfest feature requires commits to be signed-off.
Possible Solution

Split check for commit sign-off into a separate feature.
 The pr_description_required feature already helps. We can't use the hacktoberfest feature at the moment because we don't require commit sign-off.
Your Environment

[x] You're using the hosted Derek service

or

[ ] You host your own OpenFaaS cluster with Derek installed

Docker version docker version (e.g. Docker 17.0.05 ):

Are you using Docker Swarm or Kubernetes (FaaS-netes)?

Operating System and version (e.g. Linux, Windows, MacOS):

Can you suggest an alternative detection mechanism that would work for you?
I'm glad that the empty PR description is useful, what other kind of checks night be helpful to you?

@alexellis Thanks for getting back so fast 👍
Currently we're mainly struggling with Hacktoberfest spam. So anything which would automatically close these issues would be great.
I think the current mechanism for hacktoberfest without the signed-off check would work for us:

First time contributor
Only Markdown file changed
No description in the PR seems to also be a common thing in current spam PRs

What do you think of #164? We are testing it here:
https://github.com/openfaas/org-tester/pull/33
Existing committer: https://github.com/openfaas/org-tester/pull/34

What we did in the meantime was enabling interaction limits (first for recently created accounts, then for people without prior contributions). #164 sounds feature wise similar, but with the advantage that people still can submit legitimate PRs. So would be an improvement to lock repo completely

It's merged and available for all users. Just add no_newbies to your Derek feature list.
Would you also mind pinging an email to<EMAIL_ADDRESS>to connect?
Alex

Example of no_newbies:
https://github.com/openfaas/org-tester/pull/39
I was thinking of adding a min account creation date, but for your PRs - they are not actually from bots (I checked) they are from people with largely inactive but pre-existing accounts.


Update codeartifact-concepts.md

Request to update the package formats supported to include the recently launched "NuGet".
Issue #, if available:
Description of changes:
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Creating a linked issue https://github.com/awsdocs/aws-codeartifact-user-guide/issues/12

Thank you for your contribution, @tjohn123456 ! Nice catch.

Thank you for your contribution, @tjohn123456 ! Nice catch.


Inlay hints confuse matching bracket highligher

tldr:

Also note that its currently impossible to put the cursor right after (, for example, to add another parameter before current one. If it's not related to this issue then I'd file another one specifically for cursor movement.
Revision 66ec6bd.

This is a limitation we have due to the hints being implemented via VSCode decorations.
Before your issue, we had a number of reports (for example https://github.com/rust-analyzer/rust-analyzer/issues/1623) about the weird caret positioning, but this is the first about the highlighting.
Is it true that the Match Brackets setting and functionality is something that VS Code implements and provides?
If so, we should report the highlighting issue to their repo.
As for the caret position and improving inlay hints in general, here's the proper (long and demanding) way to do this: https://github.com/rust-analyzer/rust-analyzer/issues/2797
I've created a PR to update the docs describing the limitations: https://github.com/rust-analyzer/rust-analyzer/pull/3468

but this is the first about the highlighting.

I noticed this too, but didn't consider this important enough to create an issue.

Works fine with the native inlay hints now


Informative only, not to be merged: test pipeline block fix with a sub-flow

See #1245 for the full story.
This is just the final piece to prove how it happens without any special hacks touching job statuses. Without the fix this new test testBasicPipelineLevel1RunDisabledJobs gets stuck.
This commit is not intended to be merged as it is.
We first need to enable RemoteFlowWatcherTest and FlowRunnerPipelineTest ie. merge these two:

#1243
#1244

They're independent order doesn't matter.
Once that's done, I can rebase #1245 and add this new test (testBasicPipelineLevel1RunDisabledJobs) in it.

As expected, before fixes adding the test in 75ef7ad gave this Travis failure:
azkaban.execapp.FlowRunnerPipelineTest > testBasicPipelineLevel1RunDisabledJobs FAILED
    java.lang.AssertionError: Wrong status for [jobb:innerJobA] expected:<RUNNING> but was:<QUEUED>
        at org.junit.Assert.fail(Assert.java:88)
        at org.junit.Assert.failNotEquals(Assert.java:834)
        at org.junit.Assert.assertEquals(Assert.java:118)
        at azkaban.execapp.FlowRunnerTestBase.assertStatus(FlowRunnerTestBase.java:130)
        at azkaban.execapp.FlowRunnerPipelineTest.testBasicPipelineLevel1RunDisabledJobs(FlowRunnerPipelineTest.java:167)

Time to close this, test added in the fix PR: https://github.com/azkaban/azkaban/pull/1245/commits/a78877178406d4a1ab1ce44c2572011baad04190


added on_failure option to define behavior when the record change fails

This pull adds an on_failure option to the route53_record resource to allow the user to specify if they want the resource to "warn" or "fail" in the event that a record modification is not successful.

I'm going to close this out at this point since we've merged this cookbook into the AWS cookbook itself. If you're still interested in something like this take a peak at the new code in the aws cookbook


feat: Camera Passthrough

Hi Lukasz 🙌 Thanks for creating such a useful library!
I've reached for this as I needed to apply some affine transforms to an frame from Vision Camera and OpenCV has fantastic utilities for this. As I was getting setup through I noticed that the resulting frame data would be empty. I realised its because we just pass a pointer to the vector's data but once this vector goes out of scope it gets deallocated so we need to memcpy the data into the OpenCV Mat instead.
I've also included a few other handy changes here like some logging to report any errors when invoking the OpenCV functions and spruced up the example app so this can be extended with lots of lovely examples in the future!

@thomas-coldwell Wow, thanks for your great work! All changes will be includes in the next version :)


Add pushdown tests for partition tables

Describe This Problem
In #1150, a support_pushdown method is added, and we need to ensure analytic table and remote engine table can pushdown specific columns, analytic table tests have been added, but not for remote engine tables.
Proposal

Explain support remote table
Add pushdown check for remote table

Additional Context
No response

I'd like to give this issue a shot. Are there avenues where I can ask questions?

You can ask question here, also you can subscribe to the dev mail list, which can be found in our readme


Remove 'reason' column

Remove reason
Query only reason == MasterNode

Fixing pull request:
https://github.com/tomochain/tomomaster/pull/279


Replace CloudFoundry command line usage with use of API

CommandLineCloudFoundryDeployer is not safe for multiple concurrent use. Should reimplement the Deployer interface using the CF API.

Looking into this today and it seems unnecessarily complicated to pull this off. Just like everyone, we use the CLI to cf push. It turns out that push is a high level concept that has no direct equivalent in the more granular API. There is no documentation about how to do something equivalent in the API.
I found some hints. In this closed issue some other people are trying to do something similar to push using the API with some success.
It is a little discouraging to see this from @goehmen though:
"cf push is complex and is not something we recommend recreating via API"
So what use is the API then?
Another place mentions some steps that make up a push:

find and validate app to push - cf validate-for-push
refresh auth token - prob no demand
creates or updates app with app params -
creates/updates routes for app
uploads the app bits (could decomp diff kinds of uploading)
if svcs, binds app to svcs
restarts the app

There are various npm cloud foundry clients, but they have various states of maturity. None of them try to implement a push.
There is a java cloud foundry client, which does implement push.
But we don't have a java atomist client to use this in.
So I could create a small Java service that uses the java cloud foundry client and our ts automation uses it. That is more moving parts than I would like though.
Or I can look at the various hints above and reverse engineer a push using the API even though it is discouraged. I think I'll try this first.

It's probably better and easier to just make running cf push safe for "multiple concurrent use", whatever the issue is there. Perhaps just not used cached checkouts?

Perhaps just not used cached checkouts?

The concurrency problems here has nothing to do with cached checkouts but rather with the fact that the CF CLI stores the credentials in a global file that is being used for every innovation of commands with the CLI. This means whoever runs cf login last wins.

"global file" meaning the home directory? You could try change the value of the HOME environment variable before running cf.

I'm using cf-client to approximate what push does. I'm going to use the manifest.yml like push does. Apparently manifest.yml is not a base Cloud Foundry resource, but is created specifically for the CLI push. However, it seems like a nice convention so I'm just going to grab it and parse it myself and make use of the info in the more granular commands against the API.
I'm deploying apps manually with the CLI push, and I want to get the same result with what I do in the API. I need a small app that I can deploy over and over again. For some reason, I get errors when deploying black-duck-automation and pull-request-semantic-differ. That latter is really weird because it is running in CF now. I noticed some apps like ci-automation are running successfully, but I don't want to redeploy them. Any suggestions?

@claymccoy we can generate one quickly using create spring (running github-sdm) and then add cloud foundry manifest

I was having issues withall the automation clients that I tried to deploy over the weekend, and not getting any actionable error messages in the PCF log. I am using this now, which deploys fine with the cli push:
https://github.com/claymccoy/hello-node_js-express
I think I am close to getting it to deploy with the API, but it turns out the "push" is not trivial to recreate against the API (even once you have some educated guesses about what it should do).

Everything should be working except, after I upload the bits for the app and restart it, it doesn't build and stage the new bits. It just starts up the same old app. Why doesn't it do anything with the new bits? It isn't my test app, it deploys fine with CLI push.
I asked the guys in the #capi channel of the cloud_foundry slack. They were stumped too. The confirmed that there is no build command that I could kick off, and it can be tricky to understand why your new bits don't get built because it isn't under your control.
They had some recomendations. First, they recommended that I switch to the v3 API which does have a build command.
https://github.com/cloudfoundry/cloud_controller_ng/wiki/How-to-Create-an-App-Using-V3-of-the-CC-API
They also recommended running cf push -v to see all the api calls that it makes, and cf v3-push -v to understand what v3 is doing.
So, now I will look through the logs of cf push -v to determine if there is any quick fix I can change with my logic. But I'm likely going to have to take their recommendation and switch to the v3 calls. It means not using any of the js clients because they all use v2, but I understand the auth and API usage well enough at this point that it is not a problem.

wow. Thank you for your tenacity on this.

In my tests I have been deploying node apps, which seems to work fine. I'm now testing it with deploying Spring apps, which are a little different. So the deployer needs to know what it is deploying (I guess based on the build pack) and behave differently. Specifically, a node app can have it's source code uploaded, but a java app needs the jar uploaded.


feat: support iconify icon keys

This PR

takes advantage of superqt iconify support (https://github.com/pyapp-kit/superqt/pull/209) and allows icons to be expressed as iconify keys: https://icon-sets.iconify.designAction(id=..., title=..., icon='fa6-solid:arrow-up')

does a better job of autodetecting the current theme mode in the qt backend (i.e. whether the current theme is "dark" or "light"
allows the Application to explicit declare the active theme mode with Application.theme_mode = 'dark', which will then determine the color of icons generated thereafter (but see note below)
adds color_dark/color_light fields to the Icon type to allow an icon to override the color that should be used in light/dark modes.  (otherwise a dark gray / light gray default will be used in light / dark modes respectively)

note that svg-colored icons don't currently change if the parent style sheet changes.  This is just a generally harder thing to solve (and one of the few benefits of font-icons).  However, we could, as magicgui does, attach QPallete change events here and auto-change icon colors in a future PR.

cc @jni, @Czaki ... nothing here affects napari, since napari still doesn't use app-model for it's icons.  But plugins could conceivable in the future if anyone starts using the commands.icon spec ... this makes it easier for them to pick icons from iconify

I pulled off the color spec into #132  ... not entirely sure  about that part yet.  so this is just iconify now


Implement Simulator in Vue3

Description
Implement simulator in vue3
Implementation
Implementation should be in vue3/ts

Currently working on a bug where we you switch between basic and advanced, the graphs disappear.

Done with :

Using vue-chartjs instead of chart.js
Adding a line chart component
Adding a pie chart component

Currently working on converting the FarmingProfile class into a vue component

FarmingProfile is now converted to a vue component.
Fixes need to be done on:

Line Chart component.
ROI value isn't reflecting.

Ready for review.

simulator is added to the new playground


Remove notifications when messages are read, on Android

We have this feature in zulip-mobile, and should match it here.
We should also eventually add this feature on iOS. We don't yet have it there in zulip-mobile (https://github.com/zulip/zulip-mobile/issues/3119), so that will be tracked separately and happen later.

Mentioned in feedback here:
https://chat.zulip.org/#narrow/stream/48-mobile/topic/notifications.20from.20cold.20start/near/1749654
(Which was the first day after Android notifications started working generally.)

I would like to contribute to it @gnprice

I would like to contribute to it @gnprice

@SharmaDhruv2511 Please see our guide here: https://github.com/zulip/zulip-flutter#picking-an-issue-to-work-on

I picked this up today, and I've gotten the feature working successfully in a draft branch. There's still a fair amount of cleanup needed to turn that into something mergeable, though.
The way this feature works in zulip-mobile (which has worked quite well) is that on each notification we include in the "extras" a key lastZulipMessageId. Then when an RemoveFcmMessage comes in from the server, bearing a list of Zulip message IDs that have been read (and previously caused a notification), we look through the existing notifications in the notification UI to see which ones correspond to those Zulip message IDs and so should be removed.
To date, the way we've been interacting with the Android SDK for controlling notifications is with package:flutter_local_notifications, which has a Flutter plugin for this purpose. As discussed at #351, though, that package isn't entirely satisfactory. And implementing this feature turns up another case of that: the plugin provides a version of the getActiveNotifications method which tells us about the existing notifications in the notification UI… but it doesn't give us a way to see the extras on the notifications. So our choices are:

Find some other way of implementing the feature which works with the existing package:flutter_local_notifications API. There may not exist a clean way to do this, though we can likely find some kind of hack.
Patch package:flutter_local_notifications to provide those extras. Do so locally at first, and try to upstream it.
#351.

I decided this was a good prompt for trying out Pigeon and doing #351. It worked. Pigeon has some rough edges, but it definitely feels a lot more pleasant than trying to write by hand all the serialization, deserialization, and dispatch logic for talking between Dart and Kotlin.
My current draft branch doesn't do all of #351, in that it doesn't replace some of the other methods which aren't involved in this feature. But it already replaces by far the most complex of those methods, for showing a notification (so that we can set that value in "extras" in the first place). So I might just do #351 completely along the way to this issue.

I picked this up today, and I've gotten the feature working successfully in a draft branch. There's still a fair amount of cleanup needed to turn that into something mergeable, though.

Parts of that draft branch turned into #592, toward #351 switching to Pigeon.
I've now taken the remainder of that draft branch, rebased it atop all the other changes that have happened in our notifications code, and sent it as a draft PR #864. So @rajveermalviya with that branch you now have all the work I've done toward this issue, and you can go from there without duplicating work.


issues

I have some criticisms about the industry stance on not checking in NuGet packages.  But we'll first start with the issues with this article.
Regarding point #1.
"Distributed version control systems, such as Git, include full copies of every version of every file
within the repository. Binary files that are frequently updated lead to significant bloat and lengthens
the time it takes to clone the repository."
Who cares, we only clone once... However, this will force a package restore as part of the build process, which if you have a CI build, might run 50+ times per day or more!  Unless you can guarantee that packages won't be restored over and over again with every build.  This is a waste of resources and will ultimately cost you more money as you use more compute time in Azure Dev Ops.  The time wasted during a build is much more valuable that the one time cost of cloning a repository.
Regarding point #2.
"When packages are included in the repository, developers are liable to add references directly to
package contents on disk rather than referencing packages through NuGet, which can lead to hard-
coded path names in the project."
Once the packages have been restored, there is nothing stopping developers from doing this anyway.  So, this is a moot point in-my-fact (I don't have opinions ;) ... See what I did there... :) ).
Secondly, one would think that it would be a best practice that a solution should build on any developers machine upon pulling down the code from source control.  However, with a NuGet restore policy, this is not always the case.  For example, this assumes that NuGet will host all versions of all packages until the end-of-time.
Packages restore does not work with custom package sources when new developers clone the repository.  This means that the solution will not immediately build without manual intervention and configuration.
Forcing package restore is a bad-practice.  End of story.

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

ID: d965aea9-a6cc-06da-a347-6e8c89458c98
Version Independent ID: 6775f211-f4b6-6d0c-6f2e-6664b3e9721e
Content: NuGet Packages and Source Control
Content Source: docs/consume-packages/Packages-and-Source-Control.md
Product: nuget
GitHub Login: @karann-msft
Microsoft Alias: karann

I've also noticed that since there are machine specific paths in the project.assets.json file.  So, if we want to migrate to the new csproj format then we're getting forced to require package restore.  Which means, that I now need to migrate packages from our internal private NuGet repo to something ADO can see.

https://blog.ploeh.dk/2014/01/29/nuget-package-restore-considered-harmful/

Hello! You bring up several good points, so I'll try to address them individually.

.. this will force a package restore as part of the build process, which if you have a CI build, might run 50+ times per day or more! Unless you can guarantee that packages won't be restored over and over again with every build. This is a waste of resources and will ultimately cost you more money as you use more compute time in Azure Dev Ops.

This is indeed a problem! Azure DevOps will address with Azure Pipelines caching.

The time wasted during a build is much more valuable that the one time cost of cloning a repository.

I'm not sure that I follow. You still need to download your dependencies even if you add them to your Git repository. Your Git repository changes over time, so it's difficult to cache it. On the other hand, packages are easily cached as they are static files. I would expect better performance restoring dependencies through NuGet than Git. Also, keep in mind that Git repositories keep a history of all the files that were ever added. Adding dependencies to your Git repository will bloat the repository and dramatically increase initial Git clone times.

.. developers are liable to add references directly to
package contents on disk rather than referencing packages through NuGet, which can lead to hard-
coded path names in the project."
Once the packages have been restored, there is nothing stopping developers from doing this anyway.

You're right that it's still possible, however, it is significantly harder. Reference package contents directly is a bad practice and makes it harder to upgrade your dependencies. NuGet is a layer of abstraction that prevents your developers from making this mistake.

this assumes that NuGet will host all versions of all packages until the end-of-time.

That is correct. As a matter of policy, NuGet.org does not allow package deletes. In addition, you can guarantee repeatable builds using a lock file. If this isn't enough, you can always setup your own mirror of NuGet.org with the packages you depend on: https://docs.microsoft.com/en-us/nuget/hosting-packages/overview

Packages restore does not work with custom package sources when new developers clone the repository. This means that the solution will not immediately build without manual intervention and configuration.

You can check in a NuGet.config file with your desired NuGet sources preconfigured: https://docs.microsoft.com/en-us/nuget/reference/nuget-config-file
I hope I addressed your concerns. Please let me know if you have any additional questions!


feat: continuous deployment for docker image

Closes #96
Usage
The workflow is currently configured to be build-only mode. To enable pushing the image to Docker Hub, you'll need to:

Set the DOCKERHUB_USERNAME and DOCKERHUB_TOKEN variables in the repository secrets.
Set the PUSH_IMAGE variable to true (or remove it entirely) in .github/workflows/build-docker.yml.

Once configured, the workflow will automatically push the image to Docker Hub when:

The main branch is updated, or
A release is published.

Changes

Added a Dockerfile that builds the app using cargo-chef to optimize Docker build caching. On a clean build (e.g., the first time this workflow runs or when dependencies are updated), it takes around 15-20 minutes to complete. If dependencies are cached, the build time reduces to ~5 minutes.
Integrated cargo-zigbuild to build a multi-platform image supporting both linux/amd64 and linux/arm64. It should be straightforward to add other targets to the build process.

Limitations

The final build command cargo zigbuild --release ... seems to execute every time, and I'm not sure why this happens.
The cargo-chef Docker image currently supports only linux/amd64 and linux/arm64. To support other platforms, the Dockerfile needs to be adjusted, by using a different base image and adding cargo install cargo-chef.

Let me know if the docker usage should be included in README, I'll try to add it! :smiley:

Hello,
fails at the end of the workflow because the repo julien-cpsn/atac does not exist. In fact, I could NOT name my docker hub account with the - because it is not allowed. My dockerhub account username is juliencpsn instead.
How can I fix it?

Have a great day!

#@Julien-cpsn You can modify the varaible IMAGE_NAME to your current one on Docker Hub. It's also in .github/workflows/build-docker.yml, just below the PUSH_IMAGE variable.

https://hub.docker.com/repository/docker/juliencaposiena/atac/general

@alan910127
Fails on https://github.com/Julien-cpsn/ATAC/actions/runs/10542070583

Weird, I have already set it to not pushing on PR's. I'll check it later.

@alan910127 Fails on https://github.com/Julien-cpsn/ATAC/actions/runs/10542070583

@Julien-cpsn What was the event triggered the workflow? I think this information can help me diagnose easier.

@alan910127 It was triggered by releasing a new release. But when simply pushing to main it works fine. The problem is that the workflow cannot properly retrieve the release tag I think

Oh, I see. I thought the tag input for docker/meta-action was extending tags, but it actually overwrites them.
You can modify the tag multiline string in the with object under the docker/meta-action job in .github/workflows/build-docker.yml by adding one of the following lines:

type=semver,pattern={{version}} to tag as a.b.c when releasing vx.y.z
type=semver,pattern={{raw}} to tag with the exact version just released

Thanks!
I think I got it to work


Blockquote bugfix

Only render filtered children when special character present
Fixes bug when introducing Blockquote to research in https://github.com/carbonplan/research/pull/243

lgtm!


Irrefutable patterns in lambda

As title

Maybe we need match expressions for this 🤔

Right now I think we can simply translate them into single clause matches.


Add box so that JSON can be copied or pasted (thus able to share custom diagrams)

wont fix. dont need to see it. opting for storage instead...


Text overwritten ?

Used Shopware Version: <IP_ADDRESS>
Description: Some of my sides disapeer, textes i change are on old version again
Environment: productive
Steps to reproduce: Made update on shopware <IP_ADDRESS>
Expected result: Text shall be same as before
Actual result: Textes are on default ?

[public] Shopware automated response: This issue is linked to the internal issue: LAN-163
[created from LAN-163, comment 514120]


Fix lpc55s69 print

Remove accidentally included debug print() call in LPC55S69 support.
Also includes a tiny change to perform an up-front check of whether any pages have an unknown state prior to calling the analysis routine during flash programming.

/morph test


Streaming responses

My understanding of Lambda is that it only supports a single callback and therefore can't do streaming responses. The Node.js example uses res.end(): https://github.com/apex/up-examples/blob/master/community/node-basic/server#L8. Am I correct that whatever wrapper is being provided here is going to wait for end() to be called and send back the entire response (meaning res.write() calls will be buffered)?

@matthewp for now they're buffered, like you mentioned it's a Lambda limitation for now, stuck with it unfortunately.

OK, so I guess this should be closed since it's a bug in Lambda and not apex/up. Thanks!

So 2018 is over soon  and  aws lambda still does not support streaming for node.   However, streaming seems to be supported by the java implementation
@tj do you think it would be a feasible idea to replace the _proxy.js with a java implementation that would stream the response?

@ZJONSSON hmm I'm not sure that the underlying Lambda calls are actually streaming in that case, it might just be buffering and responding. Just a guess though, but as far as I know any streaming wouldn't work with the Invoke / InvokeAsync calls, might be special-cased for API Gateway though!


(DOC): basic usage docs

@alexeykudinkin @atsky I've tried to write some Rust code (my homework :) ) in intellij rust yesterday, and it is possible :)
So I started some documentation for the users of the plugin.
Guys, what do you think? Isn't it too early?

:+1:

Super!


[BUG] ARIMA model sarimax_kwargs doesn't exist after fit

Describe the bug

ARIMA model has **sarimax_kwargs as init parameters but after fitting the model, they don't exist in the model anymore. sarimax_kwargs just returns an empty dictionary.
To Reproduce

import numpy as np
from sktime.datasets import load_airline
from sktime.forecasting.arima import ARIMA

y = load_airline()

model = ARIMA(
    order=(5, 1, 4),
    seasonal_order=(3, 0, 3, 7),
    suppress_warnings=True,
    enforce_stationarity=False,
)
print(model.__dict__)

model.fit(y=y)
print(model.__dict__)

Expected behavior

After fitting, enforce_stationarity=False,which is an sarimax_kwargs parameter, should be included in sarimax_kwargs dictionary
Additional context

Versions

System:
python: 3.8.13
machine: Linux-5.13.0-44-generic-x86_64-with-glibc2.10
Python dependencies:
pip: 22.1.1
setuptools: 62.3.2
sklearn: 1.0
sktime: 0.11.4
statsmodels: 0.13.2
numpy: 1.21.5
scipy: 1.8.1
pandas: 1.3.5
matplotlib: 3.5.2
joblib: 1.1.0
numba: 0.55.2
pmdarima: 1.8.5
tsfresh: None

I've verified the bug.
This is due to non-compliance of ARIMA with the scikit-learn interface (the parameter access part), where all parameters in the constructor need to be written out, or otherwise will not be detected by parameter access methods and funniness may happen.
The solution is to write out all the sarimax kwargs in the constructor, instead of catching them in a kwargs.
We should also test for this, since it can lead to problems.

This should help: https://github.com/alan-turing-institute/sktime/pull/2731
What would be appreciated is a second set of eyes on the defaults for these parameters.
Moving them to the construtor did require to set defaults, and the docstrings of pmdarima are not 100% clear there.

Thank you for the quick fix, everything looks good to me now

well, thanks for checking! Will fix.

if you don't mind, due to you correcting my wrong defaults, I'd add you as an author to the file - as you may have seen, it is largely a specification of parameters and defaults.

I don't mind at all. Thanks.

So, @ilkersigirci ...
it would seem like there are nine other estimators with the same problem:
https://github.com/alan-turing-institute/sktime/issues/2733
Would you be up for helping with this for the forecasting module, i.e., AutoARIMA and AutoETS?
Just needs to follow the pattern in https://github.com/alan-turing-institute/sktime/pull/2731
I've started already and pinged authors of some estimators, but I need to go back to may day job now and will have to prioritize other items.

Currently, I am busy at work in the weekdays, hence I can only work on it in the weekends if you want.

ok - I thought, if you need this at work, you might have some slack.
I'll leave it as a "good first issue" for now, so it might be picked up by contributors; and if not, I will come back to this again, or you can look at whether it's still there if you feel like helping 😄


Performance issue

Hello, thank You for this tool!
I just want to share performance results on my project before and after this PR.
Before:
 -------------------- --------
  Report
 -------------------- --------
  Violations           0
  Skipped violations   901
  Uncovered            11857
  Allowed              106775
  Warnings             0
  Errors               0
 -------------------- --------

real    0m8,483s
user    0m7,044s
sys     0m1,216s

After:
 -------------------- --------
  Report
 -------------------- --------
  Violations           0
  Skipped violations   901
  Uncovered            11464
  Allowed              106775
  Warnings             0
  Errors               0
 -------------------- --------

real    3m51,235s
user    3m49,009s
sys     0m1,367s

How do you think, is it possible to optimize the algorythm?
Originally posted by @pincher2012 in https://github.com/qossmic/deptrac/issues/778#issuecomment-1017434903

Do you have a public project big enough to test optimizations against?

On a small project this patch povides some improvements, can you try it on your project?
diff --git a/src/DependencyEmitter/UsesDependencyEmitter.php b/src/DependencyEmitter/UsesDependencyEmitter.php
index 9a57620..fa6282a 100644
--- a/src/DependencyEmitter/UsesDependencyEmitter.php
+++ b/src/DependencyEmitter/UsesDependencyEmitter.php
@@ -18,12 +18,15 @@ class UsesDependencyEmitter implements DependencyEmitterInterface

     public function applyDependencies(AstMap $astMap, Result $dependencyResult): void
     {
+        $references = array_merge($astMap->getAstClassReferences(), $astMap->getAstFunctionReferences());
+        $referencesFQDN = array_map(function($ref) { return $ref->getTokenName()->toString(); }, $references);
+
         foreach ($astMap->getAstFileReferences() as $fileReference) {
             $dependencies = $fileReference->getDependencies();
             foreach ($fileReference->getAstClassReferences() as $astClassReference) {
                 foreach ($dependencies as $emittedDependency) {
                     if (AstDependency::USE === $emittedDependency->getType() &&
-                        $this->isFQDN($emittedDependency, $astMap)
+                        $this->isFQDN($emittedDependency, $referencesFQDN)
                     ) {
                         $dependencyResult->addDependency(
                             new Dependency(
@@ -38,25 +41,20 @@ class UsesDependencyEmitter implements DependencyEmitterInterface
         }
     }

-    protected function isFQDN(AstDependency $dependency, AstMap $astMap): bool
+    protected function isFQDN(AstDependency $dependency, array $referencesFQDN): bool
     {
         $dependencyFQDN = $dependency->getTokenName()->toString();
-        $references = array_merge($astMap->getAstClassReferences(), $astMap->getAstFunctionReferences());

         $isFQDN = false;
         $isNamespace = false;

-        foreach ($references as $reference) {
-            $referenceFQDN = $reference->getTokenName()->toString();
-
-            if ($referenceFQDN === $dependencyFQDN) {
-                $isFQDN = true;
-            }
-
-            if (0 === strpos($referenceFQDN, $dependencyFQDN, 0) &&
-                $referenceFQDN !== $dependencyFQDN
-            ) {
-                $isNamespace = true;
+        foreach ($referencesFQDN as $referenceFQDN) {
+            if (0 === strpos($referenceFQDN, $dependencyFQDN, 0)) {
+                if ($referenceFQDN === $dependencyFQDN) {
+                    $isFQDN = true;
+                } else {
+                    $isNamespace = true;
+                }
             }
         }

On my project i have such results
Before:
real    7m38,009s
user    7m32,079s
sys     0m2,228s

After:
real    2m4,886s
user    2m2,265s
sys     0m1,603s

So it's a significant improvement. What was the result with deptrac 0.18?
I can see a potential optimization but it might make the code way more complex.

With deptrac 0.18 the result was 8-9 sec. I think it's need to build an index with nested namespace levels and then just check if corresponding key exists instead of iterating over an array in a loop.

Yes that's what I was thinking.


Add snippet to README

Hey Alex, great tool you got here.
I used your app, completely trusting it without looking at the code (just a ~500K sat purchase).  Worked great.  But I realized, you could have totally injected your own BTC address.
I reviewed the app, but want to offer a suggestion that we make it easy for users to double check the app/QR code isn't compromised via a second, independent, code.

Ready to merge!


Weird usage printout

Sejoon, the following is what I see when I run:
python ./vaf_ncm.py --help

usage: vaf_ncm.py [-h] [-f] [-O output_dir] [-N output_filename] -I
                  input_dir_name [-nz]

    Ensuring Sample Identity v0.8
    Usage : python ./ngscheckmate_fastq <options> -1 fastqfile1 [-2 fastqfile2]  patternfile

        Input arguments (required)
          patternfile : a text file with sequences flanking representative snv sites, along with markers indicating the snv index and whether the sequence represents reference or alternative allele.
          fastqfile1 : see below 'Options'.

        Options
          -s, --ss <subsampling_rate> : subsampling rate (default 1.0)
          -d, --depth <desired_depth> : as an alternative to a user-defined subsampling rate, let the program compute the subsampling rate given a user-defined desired_depth and the data.
          -R, --reference_length <reference_length> : The reference length (default : 3E9) to be used for computing subsampling rate. If the data is NOT WGS from human, and if you're using the -d option, it is highly recommended to specify the reference length. For instance, if your data is human RNA-seq, the total reference length could be about 3% of the human genome, which can be set as 1E8.
          -L, --pattern_length <pattern_length> : The length of the flanking sequences being used to identify SNV sites. Default is 21bp. It is recommended not to change this value, unless you have created your own pattern file with a different pattern length.
          -p, --maxthread <number_of_threads> : number of threads to use (default : 1 )

          -O, --output_dir <out_dir> : A directory of output files. <default : ./>
          -N, --Name <result_name> : a prefix of name of result file. <default : output>
          -I, --input_file <absolute_path_fastq_files_list> : List of Absolute paths of fastq files (single file per line, if second fastq file is exist, write fastq files using tab delimited. (required)
                                                             ex) fastq_file_R1  fastq_file_R2   sample_ID
          -f, --family-considered mode : Reference correlations of family considered models
          -c, --answerset <answer_file> : Input answer file which have pair list of file names user want to examine (default : all possible combinations)

    Sejoon Lee, Soo Lee, Eunjung Lee, 2015

optional arguments:
  -h, --help            show this help message and exit
  -f, --family_cutoff   apply strict correlation threshold to remove family cases
  -O output_dir, --outdir output_dir
                        directory name for temp and output files
  -N output_filename, --outfilename output_filename
                        OutputFileName ( default : output ), -N filename
  -I input_dir_name, --inputDir input_dir_name
                        Inputdir name that contains ncm(VAF) file names, -I dirname
  -nz, --nonzero        Use non-zero mean depth of target loci as reference correlation. (default: Use mean depth of all target loci)

I don't think we need the usage for ngscheckmate_fastq here, and the usage for that program is wrong (e.g. called with python?). We just need to specify the input files are the output files of ngscheckmate_fastq, but that information is missing in the usage. We also don't need to print out author names in the middle..

Hi Soo,
I have changed weird usages in ncm_fastq.py and vaf_ncm.py as you mentioned.
Thank you so much for the detection of these weird usages :)
Best regards,
Sejoon Lee.

Great, thank you, Sejoon!


Option to print pick list for orders

What is the motivation for adding/enhancing this feature?

Within the track pickers card, there should also be an option to print pick lists for orders when they are in the open page.
This will move the order to "isPicked" and everything else that happens in fulfillment app during fulfillment process, but it will still show up in the open page because it is not packed.

Adding design for order card when this setting is enabled:

Some comments also added in Figma file

Verified the changes and are working fine in release v2.25.1. Hence, closing this ticket.


track original URLs to prevent unnecessary duplicates

use original URL as index?

add counter to original URLs that gets incremented each time the url is requested again. display the count with the shortened URL.


[BUG] Code style autoformatter and checker is missing

Describe the bug
Code style / formatting enforcer is missing. This cause unnecessary traction during code reviews.
Log and Stack trace

To Reproduce

Expected behavior

Code formatter is enabled and automatically formatting code before commit. (mvn spotless:apply)
Code formatting checkhing is enabled as PR validation step (mvn spotless:check)
If PR does not meet the code style / formatting requirements, PR is red and not a subject of PR reviews
Contributors are not getting angry 😠 and frustrated and keep their motivation. Reverting formatting changes does not seem to be an adequate use of their time in 2024.

I recommend to have a look at spotless-maven-plugin
Please complete the following information:

LangChain4j version:
LLM(s) used:
Java version:
Spring Boot version (if applicable):

Additional context

@langchain4j , please feel free to update PR according to your code-style preferences


feat:  mTLS-over-TCP buildkit

Adds flags to enable Copa to connect to a remote Buildkit listening on TCP address secured via mTLS.
This is very similar to creating a remote buildx driver as described here.
Closes https://github.com/project-copacetic/copacetic/issues/285

Guessing the test failures are unrelated to this change? As I see them here as well: https://github.com/project-copacetic/copacetic/pull/287
Please do let me know if you folks think there is a test you'd like to see. I've followed the same pattern that buildctl CLI has to set up the client.

Guessing the test failures are unrelated to this change? As I see them here as well: #287
Please do let me know if you folks think there is a test you'd like to see. I've followed the same pattern that buildctl CLI has to set up the client.

Yes it's happening across all PRs at the moment, working on addressing it

This is looking good to me - @sozercan do you have a chance to review?


Image height

I'm using fullsizable for a product preview, but I would like to change the height of the image or give it a padding of 40px; top/bottom.
When I give it padding with CSS it just go's down and the image will overflow the body/html.
How can I adjust the size to -40px?
thank you,

fixed it with:
-ms-transform: scale(0.8); /* IE 9 /
-webkit-transform: scale(0.8); / Safari */
transform: scale(0.8);
only don't think this is the best solution...

The dimensions of the images itself are set through javascript. You should adjust the padding of #jquery-fullsizable for the effect you are after.
#jquery-fullsizable {
  padding: 40px 0;
}

Sorry but it is set through javascript.. as you can see here:
http://jxerot.com/share/blup.png
and here:
if ($(window).height() / image.ratio > $(window).width()) {
$(image).width($(window).width());
$(image).height($(window).width() * image.ratio);
return $(image).css('margin-top', ($(window).height() - $(image).height()) / 2);
} else {
$(image).height($(window).height());
$(image).width($(window).height() / image.ratio);
return $(image).css('margin-top', 0);
}
Your padding solution just gave me padding top & bottom, but the bottom of the images is gone.
Here is my solution (that I hope issnt needed to fix the height/width margin)
http://jxerot.com/share/blup2.png

fixed it!  (still with the scale css)
here is a demo: (with many changes)
http://next.logic4.nl/artikel/22368/hamilton-american-classic-spirit-liberty-auto-chrono.html


[Python] Allow PyCapsule Interface in pyarrow.scalar constructor?

Describe the enhancement requested
Should the pyarrow.scalar constructor check for the PyCapsule Interface? It would allow any input with __arrow_c_array__, additionally validating that the input is a length-1 array.
In this case it's less important for the speed of sharing large data across multiple libraries than it is for the reliability of communicating Arrow data.
An existing workaround is to use pa.array(other_scalar).slice(0, 1), so it's not clear whether the pa.scalar constructor should or should not check for this.
cc @jorisvandenbossche
Component(s)
Python

take

/take

take


MSBuild: Clarify binding redirect version

Clarifying that binding redirects to MSBuild <IP_ADDRESS> do not restrict to the use of MSBuild 15 (they support 16 too).

@rainersigwald : Thanks for your contribution! The author(s) have been notified to review your proposed change.

#sign-off


Align Images problem

I've only encountered this kind of issue in low-memory Docker containers, and the following workaround fixed it:
sbas.compute_align(n_jobs=1)


Allows customizing polymorphic changeset

I need to run a different changeset function given some conditions. This allows passing a :with option to cast_polymorphic_embed/3 with an atom for the changeset function to use.
I picked the same name as cast_embed/3, but the expected type is different since there's no way to pass a polymorphic function ref as well.

I need to run a different changeset function given some conditions. This allows passing a :with option to cast_polymorphic_embed/3 with an atom for the changeset function to use.
I picked the same name as cast_embed/3, but the expected type is different since there's no way to pass a polymorphic function ref as well.

Glad to see someone else raise this issue and make this PR, as it's something we need as well. Basically, we want to be able to choose the embedded changeset based on the state of the parent schema. It seems like this is not possible in the library currently but this change appears to solve it!

Thank you for the PR @jmnsf !
I've added this feature in v1.4.0
However I did a bit differently:
changeset
|> cast_polymorphic_embed(:channel,
  with: [
    sms: {SMS, :custom_changeset, ["foo", "bar"]},
    email: &Email.custom_changeset/2
  ]
)

Specify the custom changeset per type
Pass an MFA or function

By the way, could you briefly explain me your use case? I'm curious to know the reason to use custom changesets.

Cool 👌🏼
I've different validation requirements for an embedded schema depending on whether it's a draft or a final version, so I use different changesets for each case.


Datafusion-Ray testing on K8s in CI/CD

After addressing #28, we are ready to launch datafusion-ray tests in a cluster in CI/CD on a Kubernetes cluster

At the moment we are only running one of the examples, how should we extend it ? cc @andygrove

ideally I would like to see TPC-H queries run and check that results are correct - we do this in the core datafusion project

we can do this at 1GB scale factor

@andygrove how do we do it in core-datafusion? Do we store the expected results in CSV or something in VCS?


Error: Can't run recursive splices.

VSCode Version: 1.23.1
OS Version: Window10

Steps to Reproduce:

disable all extension
start debug my extension
In the extension window, show the warn message 扩展未在 10 秒内启动，可能在第一行已停止，需要调试器才能继续。
In the main window dev tool. I can found the error stack

Does this issue occur when all extensions are disabled?: Yes
Main Window stack
  ERR Can't run recursive splices.: Error: Can't run recursive splices.
    at e.splice (file:///C:/Program Files/Microsoft VS Code/resources/app/out/vs/workbench/workbench.main.js:388:199)
    at e.splice (file:///C:/Program Files/Microsoft VS Code/resources/app/out/vs/workbench/workbench.main.js:398:655)
    at file:///C:/Program Files/Microsoft VS Code/resources/app/out/vs/workbench/workbench.main.js:417:361
    at e.bufferEvents (file:///C:/Program Files/Microsoft VS Code/resources/app/out/vs/workbench/workbench.main.js:169:659)
    at r.e.splice (file:///C:/Program Files/Microsoft VS Code/resources/app/out/vs/workbench/workbench.main.js:417:317)
    at t.onBreakpointsChange (file:///C:/Program Files/Microsoft VS Code/resources/app/out/vs/workbench/workbench.main.js:4230:348)
    at file:///C:/Program Files/Microsoft VS Code/resources/app/out/vs/workbench/workbench.main.js:4226:201
    at e.fire (file:///C:/Program Files/Microsoft VS Code/resources/app/out/vs/workbench/workbench.main.js:166:992)
    at e.removeFunctionBreakpoints (file:///C:/Program Files/Microsoft VS Code/resources/app/out/vs/workbench/workbench.main.js:2370:203)
    at t.removeFunctionBreakpoints (file:///C:/Program Files/Microsoft VS Code/resources/app/out/vs/workbench/workbench.main.js:4295:69)
    at u (file:///C:/Program Files/Microsoft VS Code/resources/app/out/vs/workbench/workbench.main.js:4236:578)
    at HTMLInputElement.<anonymous> (file:///C:/Program Files/Microsoft VS Code/resources/app/out/vs/workbench/workbench.main.js:4236:914)
    at file:///C:/Program Files/Microsoft VS Code/resources/app/out/vs/workbench/workbench.main.js:219:693
    at e.releaseRow (file:///C:/Program Files/Microsoft VS Code/resources/app/out/vs/workbench/workbench.main.js:219:719)
    at e.release (file:///C:/Program Files/Microsoft VS Code/resources/app/out/vs/workbench/workbench.main.js:219:551)
    at e.removeItemFromDOM (file:///C:/Program Files/Microsoft VS Code/resources/app/out/vs/workbench/workbench.main.js:392:128)
    at e._splice (file:///C:/Program Files/Microsoft VS Code/resources/app/out/vs/workbench/workbench.main.js:388:520)
    at e.splice (file:///C:/Program Files/Microsoft VS Code/resources/app/out/vs/workbench/workbench.main.js:388:274)
    at e.splice (file:///C:/Program Files/Microsoft VS Code/resources/app/out/vs/workbench/workbench.main.js:398:655)
    at file:///C:/Program Files/Microsoft VS Code/resources/app/out/vs/workbench/workbench.main.js:417:361
    at e.bufferEvents (file:///C:/Program Files/Microsoft VS Code/resources/app/out/vs/workbench/workbench.main.js:169:659)
    at r.e.splice (file:///C:/Program Files/Microsoft VS Code/resources/app/out/vs/workbench/workbench.main.js:417:317)
    at t.onBreakpointsChange (file:///C:/Program Files/Microsoft VS Code/resources/app/out/vs/workbench/workbench.main.js:4230:348)
    at file:///C:/Program Files/Microsoft VS Code/resources/app/out/vs/workbench/workbench.main.js:4226:201
    at e.fire (file:///C:/Program Files/Microsoft VS Code/resources/app/out/vs/workbench/workbench.main.js:166:992)
    at e.setExceptionBreakpoints (file:///C:/Program Files/Microsoft VS Code/resources/app/out/vs/workbench/workbench.main.js:2367:120)
    at file:///C:/Program Files/Microsoft VS Code/resources/app/out/vs/workbench/workbench.main.js:4301:525
    at Object.g [as _notify] (file:///C:/Program Files/Microsoft VS Code/resources/app/out/vs/workbench/workbench.main.js:146:807)
    at Object.enter (file:///C:/Program Files/Microsoft VS Code/resources/app/out/vs/workbench/workbench.main.js:150:85)
    at n.Class.derive._oncancel._run (file:///C:/Program Files/Microsoft VS Code/resources/app/out/vs/workbench/workbench.main.js:151:896)
    at n.Class.derive._oncancel._completed (file:///C:/Program Files/Microsoft VS Code/resources/app/out/vs/workbench/workbench.main.js:151:337)
    at file:///C:/Program Files/Microsoft VS Code/resources/app/out/vs/workbench/workbench.main.js:2437:107
    at t.e.acceptMessage (file:///C:/Program Files/Microsoft VS Code/resources/app/out/vs/workbench/workbench.main.js:2384:797)
    at t.handleData (file:///C:/Program Files/Microsoft VS Code/resources/app/out/vs/workbench/workbench.main.js:2385:660)
    at Socket.<anonymous> (file:///C:/Program Files/Microsoft VS Code/resources/app/out/vs/workbench/workbench.main.js:2385:156)
    at emitOne (events.js:96:13)
    at Socket.emit (events.js:191:7)
    at readableAddChunk (_stream_readable.js:178:18)
    at Socket.Readable.push (_stream_readable.js:136:10)
    at Pipe.onread (net.js:560:20)

Extension Window dev-tool log

@axetroy sorry for the slow response I was on vacation
Can you reproduce with vscode insiders https://code.visualstudio.com/insiders/
If you can can you please F1 > developer tools > check "Pause caucht exception" icon
Once exception happens take a screenshot and paste it here so we know where is the exact cause. Thanks a lot


Issue with Playing Foreground with playlist and specific index

@danielR2001
I am trying to play, play list. I am trying to store the last played audio and position and trying to seek with the following :
      final Result result = await _audioPlayer.playAll(
        urls,
        index: index,
        position: duration,
        repeatMode: false,
        respectAudioFocus: true,
        playerMode: PlayerMode.BACKGROUND,
        audioNotifications: audioNotifications,
      );

it works on the debug mode. but on release mode(apk), it fails with the following error.  Please let me know how to fix this issue. any insight will be very helpful.
17336
    Reason: Context.startForegroundService() did not then call Service.startForeground(): ServiceRecord{ad023f4 u0 com.xyz.pyx/danielr2001.audioplayer.audioplayers.ForegroundAudioPlayer}
    Load: 1.82 / 1.82 / 2.19
    CPU usage from 0ms to 6715ms later (2020-03-04 19:25:40.086 to 2020-03-04 19:25:46.801) with 99% awake:

I made an update of the starting position, it wasn't even implemented (i forgot about it). :D
but the release crash is happening to me too, but it happens with all the players even if not using the start position. That's is very strange.

I am working around the issue by letting the first audio play for a second and skipping to the index where I want as of now :-) I know it is ugly. but I am kinda in rush to fix the defect and move on. When I get sometime, I will definitely try to contribute this project. Appreciate your support..

First of all, the library is great @danielR2001 . Many thanks.
In the production version of our app which is being used by thousands of users, two main crash clusters (related to this library) were observed - (1) Null pointer exceptions in ForegroundPlayer startcommand and (2) RemoteServiceExceptions like the one mentioned above.
(1) were resolved by simply checking for null values, wherever possible and appropriately returning sticky & non_sticky in the startcommand function
(2) was a bit more complicated. We ended up over-riding the onCreate function and calling startForeground() from onCreate as well.
Would love to contribute these changes to the library.

@hrushi-m could you please create a pull request with theses changes, so others can benefit from theses improvements?
Thanks


Possibly extend trailing braced expression support in calls to rlang lambdas ~ { }

i.e. I could write this with function(x) { and it would stay as is

It might serve as a nice reminder to update to new ones

Since the creator of these lambas doesn't seem to want to special case their support I vote to close 😆