ServiceBusProcessor: AutoComplete has race condition with StopProcessingAsync

The ServiceBusProcessor.AutoComplete feature has a race condition with the StopProcessingAsync() method.  The following code does not cause the message to be completed:
var client = new ServiceBusClient(connectionString);
var processor = client.GetProcessor("test", new ServiceBusProcessorOptions
{
    AutoComplete = true
});

var tcs = new TaskCompletionSource<bool>();
processor.ProcessMessageAsync += async args =>
{
    var message = args.Message;
    Console.WriteLine(Encoding.UTF8.GetString(message.Body.ToArray()));
    tcs.SetResult(true);
};

processor.ProcessErrorAsync += args => throw args.Exception;

await processor.StartProcessingAsync();
await tcs.Task;
await processor.StopProcessingAsync();

I believe the issue is a race condition where my app code is calling StopProcessingAsync() immediately after tcs.SetResult(true) is called inside the event handler, which causes the processor to stop before it has completed the message.
I think the best fix would be for StopProcessingAsync() to wait for any pending completions, since from the app code perspective the message is complete.
Another option would be for the processor (or message) to expose some way for app code to know when the message has been auto-completed.
Without one of these changes, I don't think it's possible to use AutoComplete in a reliable way, since app code doesn't know when it's safe to call StopProcessingAsync().

Agree that StopProcessing should allow for in flight message processing to complete. Under the hood, it is actually already doing this, but the cancellation token that we use to signal that the processing should be stopped is threaded through to the Complete call used during autocompletion. A simple fix would be to not propagate this token through for the complete call.

This was fixed.


more robust coverage

This module should be able to detect all the popular formats of datestrings. Currently only a few formats are supported.

Closed via https://github.com/radiovisual/is-datestring/commit/1264e28b45972f2a16dfe83d68373a5cb83a87d6


Simple usage instructions in the doc

It will be nice to have a simple example usage in the docs for the Map and Marker. One should not have to dig through the example codes to figure out the basics. In my case I need a draggable marker

I just figured this out..
Set draggable: true in the _handle_map_click function, like so:
_handle_map_click (event) {
    const {markers} = this.state;
    markers.push({
     position: event.latLng,
     draggable: true
    });
    this.setState({ markers });
}

...and in the  component like this...
<Marker
  position={marker.position}
  key={marker.key}
  draggable={marker.draggable}
  animation={marker.animation}
  onRightclick={this._handle_marker_rightclick.bind(this, index)} />

Ok. Thanks


Fix overflow of latex in Callouts

As mentioned here: https://github.com/shuding/nextra/issues/1516

@B2o5T How? I can't see any guide in the README or contrib

pnpm changeset and follow steps


az cosmosdb check-name-exists => too many 500 error responses

az feedback auto-generates most of the information requested below, as of CLI version 2.0.62

Describe the bug
When I run the command az cosmosdb check-name-exists --name MyCosamosDBAcc
I get:
request failed: Error occurred in request., RetryError: HTTPSConnectionPool(host='management.azure.com', port=443): Max retries exceeded with url: /providers/Microsoft.DocumentDB/databaseAccountNames/MyCosamosDBAcc?api-version=2015-04-08 (Caused by ResponseError('too many 500 error responses',))
To Reproduce
az cosmosdb check-name-exists --name XXXXX
Expected behavior
true or false
Environment summary
Windows 10,
az cli  2.0.67
PowerShell 6
Additional context

Dup


Allow to provide custom URL path for calico binaries

We need to have an ability to provide custom URLs for calico
binaries to be downloaded from.
This patch allows to redefine the below variables:
CONFD_URL
BIRD_URL
BIRD6_URL
BIRDCL_URL

Why are these custom URLs needed?  In the long term it would be better to incorporate any fixes needed into our own builds - or even better into the upstream versions.

Sometime we need to test binaries (bird, confd, etc) which were built from the code that is not yet merged and is under CI, so we need to have an ability to provide the path to custom URLs. Definitely this pull-request doesn't change the default behavior.
Sure we can make wget outside of Makefile, but this not correct way from make system, since modification time of targets can be "older" and Makefile will re-build target, using hard-coded urls

LGTM


Provide API for Grapes JS plugins

Environment

Platform version: 7.1.x
Addon version: 0.4.0

Description
Provides the API for Grapes JS plugins which allow to configure enabled plugins or presets for HTML editor.

Plugins description - https://github.com/cuba-platform/grapesjs-addon#components


perf: cache latest semvers resolved remotely

It doesn't seem to introduce significant improvement as expected though.


Decide whether EventArgs to JSON Conversion is needed

Removed EventArgs to JSON conversion, since for now we only want the conversion to POCO


Translations update from Hosted Weblate

Translations update from Hosted Weblate for UseID/iOS-App (Localizable.strings).
It also includes following components:

UseID/iOS-App (Localizable.stringsdict)

Current translation status:

@ChaosCoder this one can be just closed now, right?


Add Support for Observe Only Resources

Description of your changes
This PR adds support for Observe Only resources.
I have:

[x] Run make reviewable test to ensure this PR is ready for review.

How has this code been tested
Basic Observe Scenario
I have created an Observe Only RDS instance as follows:
apiVersion: rds.aws.upbound.io/v1beta1
kind: Instance
metadata:
  annotations:
    crossplane.io/external-name: example-dbinstance-hasan
  name: rds-to-observe
spec:
  managementPolicy: ObserveOnly
  forProvider:
    region: us-west-1
  writeConnectionSecretToRef:
    name: observed-rds-out
    namespace: default

And verified that the resource was ready&synced and all the fields at status.atProvider were appropriately filled.
Upgrade Path

Installed upbound/provider-aws:v0.32.1 from the Marketplace.
Created an RDS instance and KMS Key using the example file.
Wait until both are Ready & Synced.
Upgraded the provider to my local built from this PR.
Verified that the provider upgrade was successful and reported as Installed & Healthy.
Previous resources are still Ready & Synced.
Created a controller config and referred to it from provider to enable management policies.
Previous resources are still Ready & Synced and provider runs properly with the alpha feature enabled.
Created an Observe Only RDS instance using the yaml above.
Verified that it becomes Ready & Synced with all observed fields filled appropriately.

Locally rebased this PR so that up dependency is v0.16.1, built ulucinar/provider-aws:v0.34.0-rc.0.10.gad098f2c from this PR, and successfully validated it. We can merge this after @sergenyalcin's performance regression tests.

We can merge this after @sergenyalcin's performance regression tests.

I did some performance tests by building an image from this branch. Any performance regression was not observed in these tests.


Added actionOpenClosed for contactSensors (simulated)

Allow open/close to be conditionally invoked on simulated contact sensors. Contact sensors  do not have open/close command. Simulated contact sensor with no actionable tile would be one way to update a SmartThings contact sensor through MQTT.

@stjohnjohnson this would be a nice one to merge.

Please merge this.


raise RuntimeError('Timeout during packet send')

Hello
I have followed that procedure to use TinyLora with a Raspberry Zero to send data to the Thing network TTN with LoRaWAN
https://learn.adafruit.com/lora-and-lorawan-radio-for-raspberry-pi/usage
All went well, excepted when I send the data, I always get that error even if TTN receive the message

Sending periodic data...
Traceback (most recent call last):
File "radio_lorawan.py", line 114, in
send_pi_data_periodic()
File "radio_lorawan.py", line 70, in send_pi_data_periodic
send_pi_data(CPU)
File "radio_lorawan.py", line 80, in send_pi_data
lora.send_data(data_pkt, len(data_pkt), lora.frame_counter)
File "/usr/local/lib/python3.5/dist-packages/adafruit_tinylora/adafruit_tinylora.py", line 234, in send_data
self.send_packet(lora_pkt, lora_pkt_len, timeout)
File "/usr/local/lib/python3.5/dist-packages/adafruit_tinylora/adafruit_tinylora.py", line 275, in send_packet
raise RuntimeError('Timeout during packet send')
RuntimeError: Timeout during packet send

I looked at TTN console and each time the Raspberry send a message, I can see that my devise has been seen 1 second ago.
But my Phyton script return alway the above message instead of

Data sent!

I spend about 3 hours to understand why. I have my own gateway is in my flat, at 1m of the nodes and I just totaly reinstall the gateway.
In TTN console, the gateway is connected and seen.
I am based in Switzerland then I changed that line:
ttn_config = TTN(devaddr, nwkey, app, country='EU')
Any idea how I can debug and solve it?
By the way if you can tell me how I can display the power of signal when the message is sent, that would be GREAT.
My goal is to prepare a simple node and when I start moving away from my gateway, I would like to be able to read the power of the signal until I am too far from the gateway range. At this time the node should display "gateway lost" because it can not reach the gateway anymore.
That the only objectif of my Raspberry/TinyLoRa, but first I have to make sure that when I send a message, even if the message is receive at TTN, I do not have that message any more.

File "/usr/local/lib/python3.5/dist-packages/adafruit_tinylora/adafruit_tinylora.py", line 234, in send_data
self.send_packet(lora_pkt, lora_pkt_len, timeout)
File "/usr/local/lib/python3.5/dist-packages/adafruit_tinylora/adafruit_tinylora.py", line 275, in send_packet
raise RuntimeError('Timeout during packet send')
RuntimeError: Timeout during packet send

Thank a lot and have a good evening

The problem were due to a bad configuration of the IRQ pin


Add stripe invoice / subscription items

Associated docs update: https://github.com/VulcanJS/vulcan-docs/pull/76

I wonder if the naming isn't getting a bit confusing? Because you add a plan property to create a subscription, but a subscription property to add an invoice item…?
Maybe I need to refresh my memory as to how Stripe plans/subscriptions/invoice items/etc. work but I have to confess I'm not super clear on how this all fits together right now…

Maybe you could give a simple example of how you'd use all of these together? What was your use case for adding these features?

If you look at the docs, it shows what’s required to use the different stripe features.

Charge
Subscription
Subscription Item (basically a subscription that belongs as a sub item on a subscription)
Invoice Item (1 off charge that is charged within a subscription cycle)

The required items for those are in the docs and the logic I implemented follows that logic.


CircleKnob provides unwanted feedback on changes

This issue was fixed in 0.1.3 for the dimmer widget, but persists for the CircleKnob widget

Issue still exists.


Tests failing on a clean repository clone

Replicate -

Run git clone https://github.com/uutils/coreutils && cd coreutils && cargo test
Read message returned by cargo test

Host specs -

Windows 11 64-bit Latest (as of 3rd August 2023)
AMD Ryzen 5 3600x
16GB 3800MHz RAM
Used PowerShell in VS Code

What is the message from cargo test?

Attached the message from cargo test, used
cargo test > out.txt

out.txt

Note that stderr only reports compiles, module runs and test rerun message ; hence not included in the text file.

Right, so touch is consistently 200 seconds off from the expected value?

I'm in India, which has the IST time zone. It is UTC + 5:30
Here is a screenshot from windows settings -

Oh very interesting, I guess it might not be timezone-related then or maybe it is, but in some more complicated way 🤔

The problem may be with chrono getting the incorrect time for whatever reason. Would you like me to run some code that would allow us to rule chrono out?

That would be great! I was thinkin the culprit could be the conversion from chrono to filetime. But yeah, let's rule chrono out first.

I'm not familiar with the crate but I ran this code -
use chrono::prelude::*;

fn main() {
    let local = Local::now();
    println!("{:?}", local);
}

Using chrono v0.4.26.
Output -

This seems to be correct

A little reading later, I think I understand how to test if chrono to filetime is an issue.
Here is the code -
use chrono::prelude::*;
use filetime::FileTime;
use std::time::{SystemTime, UNIX_EPOCH};

fn main() {
    let ftime = datetime_to_filetime(&Local::now()).unix_seconds();
    let stime = SystemTime::now()
        .duration_since(UNIX_EPOCH)
        .unwrap()
        .as_secs();
    println!("{}\t{}", ftime, stime);
}

fn datetime_to_filetime<T: TimeZone>(dt: &DateTime<T>) -> FileTime {
    FileTime::from_unix_time(dt.timestamp(), dt.timestamp_subsec_nanos())
}

Prints the following -
1691056504<PHONE_NUMBER>

Note that it is 15:25 3rd August 2023

So that's correct? Just as a sanity check, could you run one of the failing test cases manually in the terminal and check whether the time is correct? It might be an issue with the tests, instead of an issue with touch.

Executing command -
coreutils pr --pages=3:5 --column=3 -a -n ".\tests\fixtures\pr\column.log" > pr.txt

Test - test_pr::test_with_column_across_option
Times seem to be wrong.
pr.txt

Running the command again a few minutes later displayed the time as following (in the output) Aug 03 13:18 2023
Whereas it should be Aug 03 15:44 2023

Strange, we don't do much in pr, just formatting the last modified time. Could you maybe try a small program that just gets a system time, converts it to DateTime<Local> and formats it (like pr does)?
Maybe the conversion from system time to chrono is wrong?

Yes,
use chrono::prelude::*;
use filetime::FileTime;
use std::time::SystemTime;

const DATE_TIME_FORMAT: &str = "%b %d %H:%M %Y";

fn main() {
    let time: DateTime<Local> = SystemTime::now().into();
    println!("{}", time.format(DATE_TIME_FORMAT));
}

outputs
Aug 03 18:12 2023

Which is correct

Any idea why this was failing? I'd love to test this again tomorrow and close this issue.

I'm still not sure to be honest


Pokeio.SetLocation( .... ) / Error: Coords object missing

Hi, I just tried to set a new location (coords) and it doesn't work as expected.
The object I passed to the function:
var location = {
type: "coords",
name: "",
latitude: 55.654122,
longitude: 12.497218,
altitude: 0
}
Pokeio.SetLocation(location, function(error, coordinates) { .... }
Error: Coords object missing
at Pokeio.self.SetLocation (..\pokemon-go-node-api\poke.io.js:563:25)
at ..\example.js:52:4
at ..\pokemon-go-node-api\poke.io.js:176:11
at ..\pokemon-go-node-api\poke.io.js:217:14
at Request._callback (..\pokemon-go-node-api\poke.io.js:148:16)
at Request.self.callback (..\request\request.js:187:22)
at emitTwo (events.js:106:13)
at Request.emit (events.js:191:7)
at Request. (..\request\request.js:1044:10)
at emitOne (events.js:96:13)
This happens just when I pass coordinates instead of a location name.

Pass lat/ long in a coords object.
"coords": {
latitude: ,
longitude:
}


added contributor name and removed progress-bar

Fixes #5
Our project does not have this type of issue. Please see the above video -
Vedio link : https://drive.google.com/file/d/1LfNaJdHpdGXXYp2fFKrkTejsx6qOLH0Y/view?usp=sharing

Please merge this PR.

@ramya-4004 Please review the PR.

@ramya-4004 Please review the PR.

A video link has been attached to give a better idea about the issue. Refer that.


[BUG] Can't resolve '@/node_modules/next-auth/providers/email'

When running locally I'm seeing this error
./utils/app/auth/providers.ts:97:27
Module not found: Can't resolve '@/node_modules/next-auth/providers/email'
   95 |   }
   96 |   if (EMAIL_FROM) {
>  97 |     const provider = await import('@/node_modules/next-auth/providers/email');
      |                           ^
   98 |     const EmailProvider = provider.default;
   99 |     providers.push(
  100 |       EmailProvider({

https://nextjs.org/docs/messages/module-not-found

Import trace for requested module:
./app/api/auth/[...nextauth]/route.ts

I'm not quite sure why it was referencing the local @node_modules folder but after changing it to "next-auth/providers/email" it works again.

Same with "@/node_modules/next-auth/react"


Processor has no len()

Hey, I've been using deepspeed with Webdataset and I've been getting this error: TypeError: object of type 'Processor' has no len(). The script works version 0.1.62 (didn't try any other). I believe it has to do with the removal of length in wds.WebDataset and the other classes. Could this change be reverted?

The __len__ method was removed by default because some libraries use the presence of __len__ to determine whether a dataset is iterable.
Just use the with_length method:
dataset = WebDataset(...)
dataset = dataset.with_length(n)

Note that this is commonly combined with with_epoch:
dataset = WebDataset(...)
dataset = dataset.repeat().with_epoch(n).with_length(n)


Can't insert images at igniterealtime.org

Using Pade for Work. Usually i could just Ctrl-V into group chat to paste an image, now it doesn't work and i can't find a button to do this manually.

There are 2 possible issues that cause this to happen.

httpfileupload plugin does not respond to service discovery requests and converse disables uploading. When that happens, the upload icon disappears from the toolbar.

converse has a bug that causes the httpfileupload  service discovery to fail from race conditions at startup when opening bookmarked rooms. This bug is not consistent and does not not happen all the time for all rooms.

Workaround is to close room and re-open or avoid auto-join rooms. If the icon still does not happen after a re-open, the you are looking at issue no1 and you would need to reload the openfire httpfileupload plugin as it sometimes fails after long periods of usage

I was able to upload with Spark at that time though

That means it was the converse issue no2 that was in effect

I tried to restart Pade. I have this room on auto join

Restart does not work as the same thing can happen again at startup. Just leave the room and rejoin or use the refresh button

The refresh button is the simplest. It is a leave and rejoin action


feat: improve initial state of approve0/approve1 in uni-v2 deposits

Description

Pull Request Type

[ ] :bug: Bug fix (Non-breaking Change: Fixes an issue)
[ ] :hammer_and_wrench: Chore (Non-breaking Change: Doc updates, pkg upgrades, typos, etc..)
[ ] :nail_care: New Feature (Breaking/Non-breaking Change)

Issue (if applicable)

closes #
Risk

Testing

Engineering

Operations

Screenshots (if applicable)

Current dependencies on/for this PR:

develop

PR #4355

PR #4363   👈

This comment was auto-generated by Graphite.


implement graphQL API authentication using AWS Cognito User Pools

@nikgraf this addresses https://github.com/serverless/serverless-graphql-apollo/issues/96
Goal: AWS API Gateway by default is available to the world and is not secure. Hence, secure GraphQL Endpoints in Production (/graphql and /graphiql)
Few Key Points:

Authentication vs Authorization: https://dev-blog.apollodata.com/a-guide-to-authentication-in-graphql-e002a4039d1
If a user is authentication to access graphql endpoints he may or may not be authorized to access all the resources. It depends on individuals use case if he/she implements authorization logic in backend and then make API calls to grapqhl based on who is authorized to access what resource OR handle this in graphql itself (https://dev-blog.apollodata.com/auth-in-graphql-part-2-c6441bcc4302). This PR ONLY handles authentication of the user using AWS Cognito. Authentication itself can be handled in graphql (resolver) but among different disadvantages mentioned in the posts above, I would say you would not want to expose your schema to the outside world and its scary to have your endpoint available to the outside world (anyway)

Different ways to Authenticate:
AWS provides different ways to secure API Gateway endpoints mainly 1) Cognito User Pools 2) Federated Identities 3) Custom Auth.. This PR implements ONLY ONE method i.e cognito user pools.

Client Authentication: http://dev.apollodata.com/react/auth.html

Authentication to resources in resolver (like REST) can be handled by adding token in the graphql schema and passing it in the resolver which would then be added to the header of fetch call.

Basic Requirements:

Creating Cognito User Pools in AWS is a requirement to run this solution in production (to test graphQL endpoints on local via serverless offline plugin you don't need cognito)

Client will have to generate JWT tokens and pass it in the Authorization header while making API call. For now, 1) if you directly make an API call using postman (you can pass the authorization header and token value)  2) if you are making call to /graphiQL endpoint on your browser you can download google chrome plugin - ModHeader and pass Authorization Token in your web browser and /graphiQL endpoint will work (by default graphiQL endpoint will be unauthorized to access remotely).  3) Front End React Client must ideally have a mechanism to generate this token (valid for 1 hour) and then pass it during apollo client call with headers and refresh token when needed.  We need another iteration of this PR i.e V2 where client can handle this more gracefully.

Completed Functional Testing on local (serverless-offline) and remote (AWS)
Important:
This PR deals with graphQL authentication and NOT authorization. By default if the person is authenticated, he will be authorized to access graphQL resources. Authorization itself can be handled at graphQL (resolver layer) or another layer above graphQL which decides authorization of a user based on different rules
Authentication:
Involves verifying who the person says he/she is. This may involve checking a username/password or checking that a token is signed and not expired. Authentication does not say this person can access a particular resource.
Authorization:
Involves checking resources that the user is authorized to access or modify via defined roles or claims. For example, the authenticated user is authorized for read access to a database but not allowed to modify it.
Functional Testing:

Genearte Token:

yarn generate-token

Test Authentication of GraphiQL

Test Authentication of GraphQL

Test Client Authentication

Reference:
http://dev.apollodata.com/react/auth.html
Reference:

AWS Talk on API Gateway Security (Cognito User Pools skip to 23:00) - https://www.youtube.com/watch?v=n4hsWVXCuVI

http://docs.aws.amazon.com/apigateway/latest/developerguide/apigateway-integrate-with-cognito.html

https://github.com/aws/amazon-cognito-identity-js : Use Case 4 in Readme

https://medium.com/@kangzeroo/user-management-with-aws-cognito-2-3-the-core-functionality-ec15849618a4

@sid88in For some reason I'm getting an "Unauthorized" response when doing a request (same with Postman or Chrome with ModHeader).
I'm correctly getting the token (not sure if I should use the access token or the id token, anyway I tried with both).
Am I missing something?

Hey EmipC,
I don't see your comment anymore. Was this issue resolved?
thanks
Sid
On Fri, Oct 13, 2017 at 11:43 AM, EmiPC<EMAIL_ADDRESS>wrote:

@sid88in https://github.com/sid88in For some reason I'm getting an
"Unauthorized" response when doing a request (same with Postman or Chrome
with ModHeader).
I'm correctly getting the token (not sure if I should use the access token
or the id token, anyway I tried with both).
Am I missing something?
—
You are receiving this because you were mentioned.

I was using the access token instead of the id token. Now it's working. Do you know what's the difference between them?
  Yes in the header you just need Authorization and its value i.e id token
In case of whats the difference : I think this might help: https://www.youtube.com/watch?v=n4hsWVXCuVI

@sid88in By the way, would you integrate the generate-token (login) within graphql or do you think it's better to build it as an external lambda function?
I think it's not possible at this moment to use the graphql endpoint because we need a token for each request.

@emipc it depends on the use case (I have updated the PR for the apollo client to generate token and pass it as auth header) reference http://dev.apollodata.com/react/auth.html
Ideally we would want to persist this token (1 hour) and then refresh/recreate this token (mechanism we would need eventually in the client side). If someone is directly making a call to graphql endpoints then they must pass auth token in the header (using postman). Although this PR needs to be carefully reviewed and vetted. There can be many ways to handle auth based on different use cases. This PR proposed one of such ways.

I never used Cognito since I always relied on Auth0 authentication. Building Cognito Authentication right into this repo seems to make an initial setup more complicated and it only covers one way.
I was thinking to have a simple core and then we could add guide. Authentication & Authorization with this solution could be one of them. What do you think?

@nikgraf I have not used Auth0 so maybe it will be good to evaluate it!
Yes Building Cognito Authentication in this app will increase complexity of initial setup.
I have not yet done evaluations between cognito and auth0. Cognito certainly might not be the best or only way to implement authentication in every use case. I just took a stab at authentication and wanted to start this discussion sooner than later ;)

Another solution can also be to implement a simple authentication with node-jsonwebtoken.

The client will call a authenticate mutation that will return a jwt.
Store the jwt in local storage.
Read local storage with apollo middleware.
The server will have to identify if the token is valid.

Hi @sid88in , how would you go about accessing the claims of a user from a Cognitio User Pool and making available via context so that you can access it in your revolvers?


Adding comparer to select decorator.

I was playing around with select and ng2-redux, it worked for all use cases, code is very clean, and it looks awesome guys.
I don't know if this is useful, but since our select method already supports this feature I thought it'd be interesting to have the comparer function as an argument for @select as well.
Comparer function to select a particular property seems useless, at least using with immutable. The default comparer already deals with it even if the same immutable slice gets updated in another sibling (which would generate a new object).
I wonder if when selecting just the slice might be a good shot. Thoughts?

@danielfigueiredo thanks - thought I had added this in already, guess I forgot or things got lost in the shuffle of merges when prepping for the final v3 release.


\usepackage{tikzfill} clash to tcolorbox

\documentclass{article}

\usepackage{tcolorbox}
\tcbuselibrary{skins}% <--
\usepackage{tikzfill}% <--

\begin{document}
TEST
\end{document}

Compilation fails when skins library of tcolorbox loaded.
I know \usetikzlibrary{fill.image} is safe.

The package forms (tikzfill.sty and all other tikzfill-xxx.sty) load tikzfill libraries in expl3 mode, i.e., inside \ExplSyntaxOn ... \ExplSyntaxOff.
https://github.com/T-F-S/tikzfill/blob/7d03f742686ff35effdc018cd9013e907bfef933/tex/latex/tikzfill/tikzfill.sty#L22-L24
When one of these libraries then loads (built-in) tikz library, such library is also loaded in expl3 mode, which is not supported. For example, fill.hexagon library in tikzfill loads patterns.meta.
https://github.com/T-F-S/tikzfill/blob/7d03f742686ff35effdc018cd9013e907bfef933/tex/latex/tikzfill/tikzlibraryfill.hexagon.code.tex#L22

IMO, it's better to always load tikzfill libraries in non-expl3 mode (either normal category codes or LaTeX2e internal category codes).
Apart from using \@usename{__tikzfill_usetikzlibrary:n} or renaming it to for example \tikzfill@usetikzlibrary, I also wonder why there's the need for \__tikzfill_usetikzlibrary:n, since tikz already provides \usetikzlibrary.

Apart from using \@usename{__tikzfill_usetikzlibrary:n} or renaming it to for example \tikzfill@usetikzlibrary, I also wonder why there's the need for \__tikzfill_usetikzlibrary:n, since tikz already provides \usetikzlibrary.

The reason is that \usetikzlibrary does not allow local path prefixes like \usetikzlibrary{../mytest/mylibrary}. I use such prefixes in my development environment.
I see the problems with the current loading implementation which somehow did not appear with my tests setups. So, I will throw away the expl3 mode implementation and replace it by some adapted copy of the TikZ loading.

Fixed with https://github.com/T-F-S/tikzfill/releases/tag/v1.0.1


Feature request: add search/filter à la jq

jless is awesome, thank you very much for your work on it!
Description of the feature:
At the moment jless search works à la Vim, searching back and forth for patterns with /?; in particular the documentations says:

The search is not performed over the original input, but over a single-line pretty formatted version of the input JSON.

would it be possible to introduce some sort of jq grammar where we can filter via, say
.key <-- get values of nested key
[] <--- get array elements
select(.key|test("<regex>") <--- search for regex and return key or value

and similar? Or is this already present and I overlooked it?
Thank you again for the great work!

Options available for this:

xq - pure rust implementation of jq, it's still under development.
jq_rs - rust bindings for libjq via C api.

This could be an alternate implementation of #80. Or maybe #80 could be a hot-key for this filter.


process stops progressing after reaching "generating idea 2/2

Is the process stopping because I requested only 2 ideas to be generated?
I'm also curious about how to obtain the full paper.
I've been waiting for an hour, and the GPT API usage has been stuck at $0.5. Additionally, the server isn't utilizing any GPU resources.
Below is all the output I received in the terminal after running the command. Please refer to it.
(ai_scientist) root@ubuntu:~/AI_Scientist/AI-Scientist$ python launch_scientist.py --model "gpt-4o-2024-05-13" --experiment nanoGPT_lite --num-ideas 2
Using GPUs: [0, 1, 2]
Using OpenAI API with model gpt-4o-2024-05-13.
Generating idea 1/2
Iteration 1/3
{'Name': 'heterogeneous_learning_strategies', 'Title': 'Exploring Heterogeneous Learning Strategies in Multi-Agent Systems', 'Experiment': 'This experiment will involve modifying the training loop to support different learning strategies for individual agents within a multi-agent system. We will implement variations such as different exploration-exploitation balances, distinct learning rates, and alternative reward structures. The performance of these heterogeneous systems will be compared against baseline systems with homogeneous learning strategies. Key metrics will include collective performance, convergence speed, and robustness to varying tasks.', 'Interestingness': 8, 'Feasibility': 7, 'Novelty': 8}
Iteration 2/3
{'Name': 'heterogeneous_learning_strategies', 'Title': 'Exploring Heterogeneous Learning Strategies in Multi-Agent Systems', 'Experiment': 'This experiment will involve modifying the training loop to support different learning strategies for individual agents within a multi-agent system. Specifically, we will implement variations such as different exploration-exploitation balances (e.g., epsilon-greedy vs. softmax), distinct learning rates, and alternative reward structures. The performance of these heterogeneous systems will be compared against baseline systems with homogeneous learning strategies using metrics like average reward, convergence time, and robustness to varying tasks. Key functions to modify include the training loop and the agent initialization process to support multiple strategies.', 'Interestingness': 8, 'Feasibility': 7, 'Novelty': 8}
Iteration 3/3
{'Name': 'heterogeneous_learning_strategies', 'Title': 'Exploring Heterogeneous Learning Strategies in Multi-Agent Systems', 'Experiment': 'This experiment will involve modifying the training loop to support different learning strategies for individual agents within a multi-agent system. Specifically, we will implement variations such as different exploration-exploitation balances (e.g., epsilon-greedy vs. softmax), distinct learning rates, and alternative reward structures. The performance of these heterogeneous systems will be compared against baseline systems with homogeneous learning strategies using metrics like average reward, convergence time, and robustness to varying tasks. Key functions to modify include the training loop and the agent initialization process to support multiple strategies. Multiple experimental runs will be conducted to account for variability in learning outcomes, and detailed documentation will be provided to ensure reproducibility of the results.', 'Interestingness': 8, 'Feasibility': 7, 'Novelty': 8}
Idea generation converged after 3 iterations.
Generating idea 2/2
Iteration 1/3
{'Name': 'communication_strategies', 'Title': 'Impact of Communication Strategies on Multi-Agent System Performance', 'Experiment': 'This experiment will involve modifying the training loop to include communication phases where agents can share information. We will implement different communication strategies such as synchronous vs. asynchronous communication, and centralized vs. decentralized communication. The performance of these strategies will be evaluated using metrics like convergence rate, final performance, and robustness to varying tasks. Key functions to modify include the training loop to add communication phases and the agent initialization to support different communication protocols. Multiple experimental runs will be conducted to account for variability in learning outcomes, and detailed documentation will be provided to ensure reproducibility of the results.', 'Interestingness': 8, 'Feasibility': 7, 'Novelty': 8}
Iteration 2/3
{'Name': 'communication_strategies', 'Title': 'Impact of Communication Strategies on Multi-Agent System Performance', 'Experiment': 'This experiment will involve modifying the training loop to include communication phases where agents can share information. We will implement different communication strategies such as: (1) Synchronous communication where agents share information at fixed intervals. (2) Asynchronous communication where agents share information independently. (3) Centralized communication where a central node aggregates and redistributes information. (4) Decentralized communication where agents communicate directly with each other. The performance of these strategies will be evaluated using metrics like convergence rate, final performance, and robustness to varying tasks. Key functions to modify include the training loop to add communication phases and the agent initialization to support different communication protocols. Performance metrics will be collected during and after training and analyzed to determine the effectiveness of each communication strategy. Multiple experimental runs will be conducted to account for variability in learning outcomes, and detailed documentation will be provided to ensure reproducibility of the results.', 'Interestingness': 8, 'Feasibility': 7, 'Novelty': 8}
Idea generation converged after 2 iterations.
Checking novelty of idea 0: Fundamental Principle
Response Status Code: 200
Response Content: {"total": 538, "offset": 0, "next": 10, "data": [{"paperId": "09498b36bb66169f05e6be614a5493576123b6ed", "title": "Large Language Models to the Rescue: Deadlock Resolution in Multi-Robot Systems", "abstract": "Multi-agent robotic systems are prone to deadlocks in an obstacle environment where the system can get stuck away from its desired location under a smooth low-level control policy. Without an external intervention, often in terms of a high-level command, it is not possible to guarantee tha
Response Status Code: 429
Response Content: {"message": "Too Many Requests. Please wait and try again or apply for a key for higher rate limits. https://www.semanticscholar.org/product/api#api-key-form", "code": "429"}
Backing off 0.9 seconds after 1 tries calling function search_for_papers at 00:34:35
Response Status Code: 200
Response Content: {"total": 147, "offset": 0, "next": 10, "data": [{"paperId": "bf8d94a808bcd41062d1d61bab9045d61c610241", "title": "Co-Evolving Multi-Agent Transfer Reinforcement Learning via Scenario Independent Representation", "abstract": "Multi-Agent Reinforcement Learning (MARL) is extensively utilized for addressing intricate tasks that involve cooperation and competition among agents in Multi-Agent Systems (MAS). However, learning such tasks from scratch is challenging and often unfeasible, especially for
Response Status Code: 200
Response Content: {"total": 1079, "offset": 0, "next": 10, "data": [{"paperId": "a4391af54bea44c52956822fa83d1fc8ed6f8251", "title": "Multi-Agent Deep Reinforcement Learning Based Transmission Latency Minimization for Delay-Sensitive Cognitive Satellite-UAV Networks", "abstract": "With the ubiquitous deployment of a massive number of Internet-of-Things (IoT) devices, the satellite-aerial networks are becoming a promising candidate to provide flexible and seamless service for IoT applications. Concerning about the
Response Status Code: 200
Response Content: {"total": 153, "offset": 0, "next": 10, "data": [{"paperId": "0b41a6899c29a04e1217e6cc80a3d915ea18e2d8", "title": "FinCon: A Synthesized LLM Multi-Agent System with Conceptual Verbal Reinforcement for Enhanced Financial Decision Making", "abstract": "Large language models (LLMs) have demonstrated notable potential in conducting complex tasks and are increasingly utilized in various financial applications. However, high-quality sequential financial investment decision-making remains challenging.

It looks like you have stopped the process after the novelty check, keep it running for it start experiments. It will run all ideas that are deemed novel.
And yes, only 2 ideas will be generated if you specify 2 ideas!

Keep it running for it start experiments. It will run all ideas that are deemed novel. It could be the case that the two initial ideas are not novel.
And yes, only 2 ideas will be generated if you specify 2 ideas!

Thank you for reply directly!!
As you mentioned, when I force-stopped the process using ctrl+c, I encountered the following error message. It seems that the GPT-4o limit is 30,000 TPM, but I can't control this value; I can only manage the usage limit! The GPT-4o-mini model has a limit of 200,000 TPM, so I'll try running it with that model instead.
openai.RateLimitError: Error code: 429 - {'error': {'message': 'Request too large for gpt-4o in organization org-v70GRNvNZutI6mTdyL5gd2Qu on tokens per min (TPM): Limit 30000, Requested 30052. The input or output tokens must be reduced in order to run successfully. Visit https://platform.openai.com/account/rate-limits to learn more.', 'type': 'tokens', 'param': None, 'code': 'rate_limit_exceeded'}}

We would advise sticking to GPT-4o but trying to increase your OpenAI account tier.


Add changes related to PAT support in APICTL

Purpose
Add changes related to https://github.com/wso2/api-manager/issues/2754

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.Naduni Pamudika seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.


Ensure that checkstyle rules are enforced by Travis

@nerdyverde for external review (accept your team invite!)


Fouc in SPA mode with tailwindcss in production mode

Reproduction
I built a site with tailwindcss in remix SPA mode, and it's deployed on cloudflare page here https://venft-lookup.pages.dev/
As you can see in the below screenshot, there's a flash of unstyled content:
https://github.com/remix-run/remix/assets/126733611/342f33bc-2c12-4454-b4db-58e43953cace
Code is hosted on https://github.com/oxSaturn/venft-lookup, but I can't reproduce it on dev env.
System Info
System:
    OS: macOS 14.2.1
    Shell: 3.6.1 - /opt/local/bin/fish
  Binaries:
    Node: 18.17.1 - /usr/local/bin/node
    Yarn: 1.22.19 - /usr/local/bin/yarn
    npm: 9.6.7 - /usr/local/bin/npm
    pnpm: 8.14.1 - /usr/local/bin/pnpm
    bun: 1.0.21 - ~/.bun/bin/bun
  Browsers:
    Edge: 120.0.2210.133
    Safari: 17.2.1
  npmPackages:
    @remix-run/dev: ^2.5.0 => 2.5.0
    @remix-run/node: ^2.5.0 => 2.5.0
    @remix-run/react: ^2.5.0 => 2.5.0
    vite: ^5.0.0 => 5.0.11

Used Package Manager
pnpm
Expected Behavior
No fouc in production.
Actual Behavior
Fouc in production.

This looks like it's 2 underlying issues:

The <Meta> component rendering on the server is rendering matches past the root and including the meta information from the _index.tsx route.  This causes a hydration issue because we hydrate only the root route.  This is a bug we'll get fixed.
Even with (1) fixed, Cloudflare doesn't seem to be adding any caching headers on on the root stylesheet, so when react flips from the root HydrateFallback to the root component, and renders a new <Links> component, react seems to remove the old stylesheet and replace it with a new link tag (which is not render blocking because we've already rendered the document).  It feels like react should be able to use the same link tag vi vdom diffing because it doesn't change, but for now the solution there is to add cache-control headers on your static assets in cloudflare.

Fix by #8538 and will be available in 2.5.1 when released 👍

This looks like it's 2 underlying issues:

The <Meta> component rendering on the server is rendering matches past the root and including the meta information from the _index.tsx route.  This causes a hydration issue because we hydrate only the root route.  This is a bug we'll get fixed.
Even with (1) fixed, Cloudflare doesn't seem to be adding any caching headers on on the root stylesheet, so when react flips from the root HydrateFallback to the root component, and renders a new <Links> component, react seems to remove the old stylesheet and replace it with a new link tag (which is not render blocking because we've already rendered the document).  It feels like react should be able to use the same link tag vi vdom diffing because it doesn't change, but for now the solution there is to add cache-control headers on your static assets in cloudflare.

I've updated to 2.5.1-pre.1 to test
seems cloudflare did cache it now (didn't check it before), but still fouc problem, am I misunderstanding something?

That's not a cached version - a 304 means the request went to cloudflare and cloudflare responded with "nothing has changed" - but I think since it's still a net new request/response the old styles are removed and the new ones appliued causing a brief FOUC.
Instead you want to make sure you have a proper Cache-Control header on your static assets so that the browser never even makes a request (and you don't even see the 304)

That's not a cached version - a 304 means the request went to cloudflare and cloudflare responded with "nothing has changed" - but I think since it's still a net new request/response the old styles are removed and the new ones appliued causing a brief FOUC.
Instead you want to make sure you have a proper Cache-Control header on your static assets so that the browser never even makes a request (and you don't even see the 304)

Thanks, pushed a header of Cache-Control https://github.com/oxSaturn/venft-lookup/blob/main/public/_headers, so far it works well.


feat: CreateFile

Fixes #93
Blocked by #171 (it has to be fixed first, otherwise the CI will fail)

@Mergifyio update

@Mergifyio update


feat: implement graceful shutdown on sigterm

What kind of change does this PR introduce?
Bug fix https://github.com/supabase/postgres-meta/issues/663
What is the new behavior?
Uses closes-with-grace package from fastify authors. Shuts down the server on the following events

process.once('SIGTERM')
process.once('SIGINT')
process.once('uncaughtException')
process.once('unhandledRejection')

Additional context
Tested with docker-compose.yml
$ docker compose up --build
...
[+] Running 1/1
 ✔ Container postgres-meta-meta-1  Recreated                                           0.1s
Attaching to postgres-meta-meta-1
postgres-meta-meta-1  | (node:1) ExperimentalWarning: Importing JSON modules is an experimental feature and might change at any time
postgres-meta-meta-1  | (Use `node --trace-warnings ...` to show where the warning was created)
postgres-meta-meta-1  | {"level":"info","time":"2024-02-21T10:03:46.750Z","pid":1,"hostname":"dce592e0de3a","msg":"Server listening at http://<IP_ADDRESS>:8080"}
postgres-meta-meta-1  | {"level":"info","time":"2024-02-21T10:03:46.757Z","pid":1,"hostname":"dce592e0de3a","msg":"Server listening at http://<IP_ADDRESS>:8081"}
^CGracefully stopping... (press Ctrl+C again to force)
Aborting on container exit...
[+] Running 1/1
 ✔ Container postgres-meta-meta-1  Stopped                                             0.2s
canceled

Sweet! Would've thought this is built-in


Shapekey order is ruined after running script.

The order of shapekeys are completely changed after running the script. This messes with stuff once imported back into Unity.
https://user-images.githubusercontent.com/37721153/154396128-8985a61c-95e5-4e24-8208-2dccf7e62332.mp4

Just checked and this happens in cats during the apply as rest pose, since my code just calls directly into cats. I'll take a look once the height stuff is sorted out.

Fixed by #16


Missing POM in Maven Central

Hi, I am trying to use the geodb as a dependency and I am seeing
Downloading from central: https://repo.maven.apache.org/maven2/org/opengeo/geodb/0.9/geodb-0.9.pom
[WARNING] The POM for org.opengeo:geodb:jar:0.9 is missing, no dependency information available

and my build fails because of this. Comparing with one of my own projects to geodb there seems to be some files missing?

This turned out to be files available on a custom repo: repo.boundlessgeo.com


How can I find more faster FFT algoritm using your alphatensor ?

Hi, @saran-t @google-admin @matejbalog @googlebot @AlisonBell
This is great work!
How can I find more faster FFT algoritm using your alphatensor ?
FFT = Fast Fourier Transform.
Please teach me ~!
Thanks in advance.
Best,
@bemoregt.

That's the entire point of their releasing this amazing work. To inspire.


Unable to build image with RUN statement in Dockerfile

resin-cli version: 7.6.0
Operating system and architecture: Ubuntu 18.04

Hi, I am having trouble building a images from Dockerfiles. I cannot use the RUN command in a Dockerfile while using resin build. For some reason the the command and its parameters are treated like they're scripts.
When I commit my changes and run git push resin master, the build does succeed.

The following example is enough to make it fail:
FROM resin/raspberry-pi2-node

RUN true

When I then run resin build, the following error appears:
$ resin build --deviceType raspberrypi2 --arch armhf
[Info]    Compose file detected
[Info]    Building for armv7hf/raspberry-pi2
[Build]   Built 1 service in 0:01
[Build]   test [==========>          ]  50% Step 2/4: standard_init_linux.go:190: exec user process caused "no such file or directory"
[Error]   Build failed
The command '/bin/sh -c true' returned a non-zero code: 1

If you need help, don't hesitate in contacting us at:

  GitHub: https://github.com/resin-io/resin-cli/issues/new
  Forums: https://forums.resin.io

P.S. I do make use of a docker-compose-file, but I don't think this is the reason.

@Vinno97 the problem here is that you're building an ARM base image, filled with ARM binaries, but running them on an x86 machine. Lots of things on this base image will implode by default, since they're not built to run on your architecture.
You can fix this by passing the --emulated/--e argument to resin build, which will emulate the target architecture for your RUN commands, so they work correctly on your local machine. That should solve this immediately, but feel free to shout if it doesn't.


dunno filter out available uids while sampling

when we set sample size big, we often get the following error
sample larger than population or is negative

this happens when total number of available uids is smaller than the sample size

this is already tested on testnet (72) and is intended to give more workload to miners for response time diversity


xarray.concat for datasets fails when object is a child class of Dataset

What happened?
The following error is thrown:
>       result = type(datasets[0])(result_vars, attrs=result_attrs)
E       TypeError: __init__() got an unexpected keyword argument 'attrs'

This is coming from the change merged as part of this PR:
https://github.com/pydata/xarray/pull/6784/files (line 593)
When "datasets" is a list of class objects which are the child of Dataset, type(datasets[0]) will return that class as the type. This class however, doesn't necessary has attrs in its own init, hence the code breaks.
I'm not convinced that this is a correct behaviour, and if you want to initialize the Dataset class, you should revert that line of change.
What did you expect to happen?
Code to run without any errors.
Minimal Complete Verifiable Example
import xarray

class MyDataset(xarray.Dataset):
    __slots__ = ()

    def __init__(self, a, b, c, d=None):
        super().__init__()

        attrs = {
            'at1': b,
            'at2': d
        }

        super().__init__(a, coords=c, attrs=attrs)

if __name__ == '__main__':
    a = {
        "x": 1,
        "y": 3
    }
    b = ["x", "y"]
    c = {
        "coord1": 0,
        "coord2": 1
    }
    ds1 = MyDataset(a, b, c)
    ds2 = MyDataset(a, b, c)
    ds3 = MyDataset(a, b, c)

    xarray.concat([ds1, ds2, ds3], dim="coord1")

MVCE confirmation

[X] Minimal example — the example is as focused as reasonably possible to demonstrate the underlying issue in xarray.
[X] Complete example — the example is self-contained, including all data and the text of any traceback.
[X] Verifiable example — the example copy & pastes into an IPython prompt or Binder notebook, returning the result.
[X] New issue — a search of GitHub Issues suggests this is not a duplicate.

Relevant log output
No response
Anything else we need to know?
No response
Environment
xarray==2022.6.0
python: 3.9.12 | packaged by conda-forge | (main, Mar 24 2022, 23:27:05)

In the example the naming  and meaning of the arguments of the __init__ method are different from the Dataset base class.
Subclassing is often more reliable when it adheres to the Liskov Substitution Principle which states that objects of a superclass should be replaceable with objects of its subclasses and in other words: a subclass should only add functionality and not reduce it.
There are some other workarounds though:
If all you need is a convenient custom constructor then you could add a classmethod factory method of your subclass or even just use a plain function which returns a Dataset.
Another option is to convert your MyDataset instances to plain Dataset instances before passing them into xarray.concat, e.g.
xarray.concat([ds.to_dataset() for ds in [ds1, ds2, ds3]], dim="coord1")
where your subclass may define as a convenience method:
def to_dataset(self):
    """Convert to plain Dataset."""
    return xarray.Dataset(self, attrs=self.attrs)

Because we want to get same type of Dataset (subclassed or not) as we input.  That's what the T_Dataset type implies. And since we forced a Dataset at the end that wasn't the case.

Shall we close? It seems like the current code is what we want.

I'll leave that with you guys to decide. I've reported it, it's been looked at, and it has a record of it. So if the code is as you intended, I don't mind if the issue is closed.

@gabicca given that we do say in our docs that we don't recommend subclassing xarray objects, I'm going to close this issue as a downstream "subclass at your own risk" problem. If you would like it to be easier to subclass, we would welcome your input here.


Add more details for packages that need to support old Swift versions

Packages that support older Swift versions cannot use swiftLanguageMode(s), unless they add another manifest for pre 6.0 (or only want to use the top level swiftLanguageVersions and not the per-target).

To be honest I'm not sure this is worth adding to the guide, or maybe it should be a separate section on its own. I started writing after receiving questions about this, but the end result is a little confusing. Up for discussion regardless.


[TASK] Better handling for common HERON data.

Issue Description
From notes on a meeting for this:
The basic problems that we discussed were how distribute the workshop materials and the associated ROMs that are used by them.
Some questions related to this are

Where to distribute this?

How to find the ROMs if someone moves or copies an input file? (Note that the relationship between the heron executable and data files is not constant for different install methods)

These need to work for three different cases:

Installation via a git clone

Installation via pip install

Installation via the standalone install (one stop)

For the git clone, the plan is that the ROMs will continue to be available in the clone.  For the pip install and the standalone install these will need to be added.  Since the current ROMs used by the workshop are significantly bigger than the HERON pip install, they will be a separate install. For the standalone install, the workshop materials and ROMs will be included.
In order to find the ROM data, we plan on first of all, moving the ROM data from various places in HERON/tests to HERON/data and adding a variable HERON_DATA (similar to the existing BASE_WORKING_DIR) that will point to HERON/data. If the environment variable HERON_DATA exists, it will use this for HERON_DATA.  For git clones and the standalone install we will put logic in HERON so it can automatically find HERON/data.  For pip installs, the user will have to manually set a HERON_DATA environment variable.

For Change Control Board: Issue Review
This review should occur before any development is performed as a response to this issue.

[ ] 1. Is it tagged with a type: defect or task?
[ ] 2. Is it tagged with a priority: critical, normal or minor?
[ ] 3. If it will impact requirements or requirements tests, is it tagged with requirements?
[ ] 4. If it is a defect, can it cause wrong results for users? If so an email needs to be sent to the users.
[ ] 5. Is a rationale provided? (Such as explaining why the improvement is needed or why current code is wrong.)

For Change Control Board: Issue Closure
This review should occur when the issue is imminently going to be closed.

[x] 1. If the issue is a defect, is the defect fixed?
[x] 2. If the issue is a defect, is the defect tested for in the regression test system?  (If not explain why not.)
[x] 3. If the issue can impact users, has an email to the users group been written (the email should specify if the defect impacts stable or master)?
[x] 4. If the issue is a defect, does it impact the latest release branch? If yes, is there any issue tagged with release (create if needed)?
[x] 5. If the issue is being closed without a pull request, has an explanation of why it is being closed been provided?

Good to close via #351.


Readme updates

More readme updates

@wbenoit26 I added shells of README's for most projects - at a minimum I described how to build each projects environment.
Let me know any feedback.


Multiple issues with the app

First off - this is THE APP I have been wanting to have (and had thought of even writing one some time in future) to encrypt and decrypt files across multiple Operating Systems (I use macOS, Windows and Linux). I am glad that you built it. However I noticed a few glitches.

When trying to open a file, in the file selector window, a single click is registered as two clicks on the same place (so I get two levels deep, depending on which folder would appear under the mouse pointer where my first folder was). I am not sure if the error is in the Fyne base framework or not, but this is happening. I am on Apple Silicon MacBook with Sonoma (in case that matters).
When decrypting, if I use wrong password, the resulting file (empty one) is created anyway and then the prompt of wrong password is shown to me. Shouldn't that prompt be shown first and the resulting file not be created if the password was wrong?

Thanks!

Thanks for the kind words!

Is definitely a bug in Fyne, I have reported it: https://github.com/fyne-io/fyne/issues/5053

Good catch! I hadn't thought about this, but have now fixed it so that the password is checked before the output file is created: 9c3d402ea8faf203d6088441b75760891d65f52f. Unfortunately the file has to be selected again after the password check failed; this is currently a limitation of the Fyne framework.

PS: Personally I'm not a huge fan of the file selector in Fyne, so I usually just drag and drop files into spind.

Hi, vaibhav-kaushal

2. When decrypting, if I use wrong password, the resulting file (empty one) is created anyway and then the prompt of wrong password is shown to me. Shouldn't that prompt be shown first and the resulting file not be created if the password was wrong?

I have the same issue as you on Android, but I think it's a restriction of the operating system. I also think it's because the file is created when it's opened via the 'dialog.NewFileSave(callback func(fyne.URIWriteCloser, error), parent fyne.Window) *dialog.FileDialog' since a 'fyne.URIWriteCloser' is opened, resulting in a file being created. If the confirmation is canceled, the file remains even if it's empty. This makes sense on Android, but on Windows or Linux, it should be deleted if it isn't written to afterward.

Both issues should now be resolved with the latest release: https://github.com/codesoap/spind/releases/tag/v0.2.1
Let me know if the fixes work for you!

Closing due to inactivity.


Language switching

The question may seem funny, but: Is there a way to switch to another language on the site?
https://people.djangoproject.com/es/ links to Django people from Spain, https://es.people.djangoproject.com/ doesn't work. I am afraid translations aren't implemented, can anyone confirm it?

The function that Django uses internally to get the language from a request (get_language_from_request) checks the path, the session, the cookies, and the language headers. I was hoping there would be a GET fallback, but there does not appear to be.
Because the site's urls.py hijacks the normal /es/-style language specification, as you mentioned, I don't see how you could change your language as a user, short of clearing your cookies and then having your browser send a different language header. I couldn't find anything in the urls to suggest they've reimplemented this some other way, though it's entirely possible I missed something.
This is a bit confusing, considering there are translations in the code for 17 non-English languages with no easy way to change between them.

After having read your comment, I started to investigate. It occurs that LANGUAGES setting was missing 10 languages. I've opened #83. Its merge will solve my concerns. Thanks @phu.

Whether this helps really depends on your system language, I think; as I understand it, this should enable those extra languages for users whose browser sends the appropriate headers, but it still won't be possible to manually switch languages (since the normal url patterns for that are overridden by the project).
Am I mistaken, or am I misunderstanding your problem?

I was wrongly concerned that site is working only in English, without any option to change the language. That's because I thought my native language was properly added to site, but it wasn't.
So my original motivation to express the need of adding language switching will be gone soon.

OK. So your concern isn't actually switching languages, it's that your native language had a translation but hadn't been added to the settings?

Yes, it is.
We can keep the issue opened nonetheless. Suggested solution would be letting the user choose a language from a list of available languages in account's settings.


POC: use default stage when no stage has been set

Q
A

Bug fix?
Yes

New feature?
No

BC breaks?
?

Deprecations?
?

Fixed tickets
#1109

has status 'works for me'

Fixed in #1092


fix ASPxperienceDemos (close #395)

For fix tests I just replaced (https://github.com/DevExpress/testcafe/pull/396/files#diff-27aea5773c389e89e84c3a8e9953640dR448)
new iframe.contentWindow[automation.AUTOMATION_RUNNERS].ClickAutomation(element, clickOptions) :

to
new iframe.contentWindow[AUTOMATIONS].ClickAutomation(element, clickOptions) :

I didn't refactor actions.js, but I added this file in 'lint-client' and fixed all ESlint error in it.
\cc @AlexanderMoskovkin

lgtm
Can merge after tests are succeed

@testcafe-build-bot \retest

@testcafe-build-bot \retest

@testcafe-build-bot \retest

@AlexanderMoskovkin  Can we leave version without fix ESLint errors in this PR?

Yes, we can. We should fix the tests first. Is this ok now?

Yes, I checked it on our CC task 'TESTCafe ASPxperienceDemos tests - Get Rid Of JQuery '

lgtm then


Title: Water Quality Prediction

Is your proposal related to a problem? Please describe.
Water Quality Prediction using supervised learning
Add any other context or screenshots about the proposal request here.
No response

@Soumya-Kushwaha Do  not add half description about your project be professional .


[ACR] Update to 2017-10-01 api-version

All resource management now points to 2017-10-01 api-version.
Add support for registry geo-replications.
Removed getting storage account access key.
Rename Basic SKU to Classic, but Basic is still accepted with a deprecation warning.
Various improvements and fixes.

Fixes https://github.com/Azure/azure-cli/issues/4517, https://github.com/Azure/azure-cli/issues/4050, https://github.com/Azure/acr/issues/65.
Contains https://github.com/Azure/azure-cli/pull/4195.

This checklist is used to make sure that common guidelines for a pull request are followed.
General Guidelines

[x] The PR has modified HISTORY.rst describing any customer-facing, functional changes. Note that this does not include changes only to help content. (see Modifying change log).

Command Guidelines

[x] Each command and parameter has a meaningful description.
[x] Each new command has a test.

(see Authoring Command Modules)

@troydai Thank you for the review comments. The CI is failing because I need to re-record tests for appservice. Please comment if you have more suggestions. /cc @SajayAntony

Looks good. But make sure CI passes.

/cc @yugangw-msft for updating acr SDK in appservice module.

@djyou still want this to not be merged right now?

Still waiting for the api-version open to public ...

@troydai As discussed offline, close this PR for now.


Fix for AudioPlayer causing a crash on voiceConnection#closing

Looks like some leftover from commit 9eaf145
TypeError: this.destroyAllStreams is not a function

Should be this.destroyCurrentStream()

Hydar plz


Cleanup

Pull Request Test Coverage Report for Build 1596

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

Totals

Change from base Build 1592:
0.0%

Covered Lines:
1175

Relevant Lines:
4029

💛  - Coveralls


Clarified software versioning language.

(Still need to clarify hw versioning language.)
Fixes #98

Pending issues related to this change. Do not merge until resolved.

Discarding this because the general submission rules now have more precise language.


Set parameters fails when base-revision branch name and a directory in the root directory share the same name

Orb version: path-filtering<EMAIL_ADDRESS>
What happened:
I have a directory named master within the root of my repository. And coincidentally, my base revision branch is the master branch.
The path-filtering/set-parameters step always fails when I have base-revision set to master because that directory exists in the root of my repository. The Git checkout command has no clarity on if this should be a local file (pathspec) checkout or a tracking branch checkout, so that step fails and terminates the pipeline.
fatal: 'master' could be both a local file and a tracking branch.
Please use -- (and optionally --no-guess) to disambiguate
Traceback (most recent call last):
  File "<string>", line 23, in <module>
  File "<string>", line 15, in checkout
  File "/usr/lib/python3.8/subprocess.py", line 516, in run
    raise CalledProcessError(retcode, process.args,
subprocess.CalledProcessError: Command '['git', 'checkout', 'master']' returned non-zero exit status 128.

There is no workaround for this since the base-revision branch name is being passed in as an argument into the python command. I've tried to bypass for example:
error: pathspec 'master --' did not match any file(s) known to git
Traceback (most recent call last):
  File "<string>", line 23, in <module>
  File "<string>", line 15, in checkout
  File "/usr/lib/python3.8/subprocess.py", line 516, in run
    raise CalledProcessError(retcode, process.args,
subprocess.CalledProcessError: Command '['git', 'checkout', 'master --']' returned non-zero exit status 1

Expected behavior:
The path filtering orb might need to offer a new disambiguate parameter which makes the checkout process dynamic.
If the parameter is specified as true, then the disambiguate (--) flag should be passed in during the checkout process. Otherwise, it can be left as is.

Additional Information:

This issue is very specific and we won't have the bandwidth to address this. A possible workaround could be changing the directory name.


[Feature request] Add NVMe drive support

It should be possible to connect nvme drive to "e key m.2 x1 pcie" port designed as wifi card slot via this adapter:
http://www.aliexpress.com/item/M-2-NGFF-Key-M-to-Key-A-E-Extension-Cable-Adapter-Card-with-High-Speed/32995476307.html
However this also need support via software
More info here: https://devtalk.nvidia.com/default/topic/1049706/jetson-nano/can-this-board-use-m-2-ssd-nvme/

https://github.com/jetsonhacks/rootOnNVMe.git


add a collector to get ceph usage stats from ceph cluster

Total Cluster values aswell as per pool values are taken into account here.

This repo has moved to https://github.com/python-diamond/Diamond
Please open a new pull request there if this issue is not resolved with the current code there.
Thanks!


Jsonplaceholder

added development section and json endpoints to jsonplaceholder

Hey @stephan2307 please make an individual pull request for each suggestion. Closing this, please resubmit. =)


Fix sepBy throws when max==1

Thank you for merging #14!
This pull request fixes sepBy throws an error when max == 1. (maybe sepBy with max==1 is meaningless through)

Oh, good catch! I'm working on a few updates over here: https://github.com/wavebeem/bread-n-butter/pull/16


A&R -> Ability for Rec Services/ Admin to add sub areas

Description:
Context:
Might have many new sub areas coming up (traffic counters discovered) - Access to south coast data map. 49 counters installed in south coast. So how to incorporate them into the system?
Acceptance Criteria:
[Note: Use 'Given/When/Then' format if it makes sense to. Otherwise, a simple checklist that can be tested.]
Development Checklist:

[ ] ...
[ ] ...
[ ] ...

Dependencies

Blocked by
Blocking

Relevant documentation as reference
Definition of Ready

[ ] Acceptance criteria are included
[ ] Wireframes are included (if applicable)
[ ] Design / Solution is accepted by Product Owner (if applicable)
[ ] Dependencies are identified (technical, business, regulatory/policy)
[ ] Story has been estimated (under 13 pts)

Definition of Done

In progress:

[ ] Acceptance criteria are tested (Functionality meets the acceptance criteria defined in the ticket)
[ ] UI meets accessibility requirements
[ ] Unit tests are written
[ ] Work is traceable in GitHub
[ ] PR linked to ticket number
[ ] If needed/required - Dev adds flag/label to highlight any migration steps necessary prior to PROD deployment

Code review:

[ ] Code is peer reviewed and has passed CI/CD tests

QA:

[ ] Acceptance criteria are tested (Functionality meets the acceptance criteria defined in the ticket)
[ ] Code is potentially shippable to the production environment
[ ] Functional features have been tested and passed by QA
[ ] UI components tested by designer
[ ] Code is deployed to PROD when moved to 'done' column (unless requested otherwise by PO)

PO Review:

[ ] Acceptance criteria are tested (Functionality meets the acceptance criteria defined in the ticket)
[ ] Reviewed and approved by Product Owner

Notes:

@sallyhaynes - what activity will traffic counters fall under? Are they for day use?

@meyerdarcie Yes, they will all be under day use, and are a mix of traffic and trail counters (the majority being trail). So it would affect data under "vehicle count" and "trail count".

October 11, 2024 Prioritization session - discussion notes:

Regions have said they don't have time to enter the data and information, which would mean it would fall to Sally and Nicole to administer and they have limited capacity.
This would help automate the process
Can start with South Coast as a pilot.

Todo:

designers work with devs and to see what the design could be - feedback from admin users
get some concrete user needs down prior to discussion.
so, need to setup session. Create a ticket to do the exploratory work

Deadlines?

depends on the level of complexity of this ticket and how much work it would be
Season launch happens in April, so if we want to incorporate these for next season (again, not a hard deadline), we would need to have it done before then.

this one ties into #333 for the notes field to be added for the park, i dont want it to get lost!


Large memory usage in very sparse programs

Describe the bug
When generating large sparse programs, CVXPY spends a huge amount of memory canonicalizing the expression and passing it to a solver. In particular, a quadratic program with at most ~60k nonzeros in its equivalent, handcrafted affine conic map takes up < 10Mb, while CVXPY's memory usage shoots up to around 50Gb.
To Reproduce
Clone this repo and run test.py after replacing n = 1000 with n = 10000.
Version

OS: Mac OS Catalina
CVXPY Version: 1.0.25

Initially reported by @ludi1001

@angeris did you vectorize the code?

@SteveDiamond their model is not vectorized. Most of the Constraint objects are over scalars, and if I understand correctly, the model generated by their test script will have n Constraint objects, where they recommended testing with n=10000.
I don't think this model should be used as a benchmark for compile time. However, I do think this model raises an interesting question of memory pressure. Why is it that so much memory is allocated at once? What's going on with garbage collection, either at the C++ or Python level?

This is not vectorized, but either way it's still a relatively small number of constraints (as discussed with @akshayka )—the main reason for no vectorization is that the code was supposed to be clear and follow the exact derivation in the paper.
I can add a simple vectorized version for testing purposes as well, but it would be surprising if looping over maybe 40k-80k variables (even in Python) would yield such a large slowdown?/large memory consumption?
I haven't benchmarked anything yet since I haven't gotten the chance to dig a little deeper into the code.

@rileyjmurray Duplication of the same sparse matrix across multiple atoms is very likely an issue. As @angeris said, I think our Python-to-C++ interface probably copies data.
In fact you can see the copy in cvxcore/src/LinOp.hpp:set_sparse_data:127
    for (int idx = 0; idx < data_len; idx++) {
      tripletList.push_back(
          Triplet(int(row_idxs[idx]), int(col_idxs[idx]), data[idx]));
    }

The binding from Python to C++ is not pretty. It was written many years ago, and the authors are no longer associated with the lab/cvxgroup.
At some point, we should probably drop the swig wrapper and use PyBind11, which makes it easy to convert scipy matrices / numpy arrays to Eigen matrices without copies (https://pybind11.readthedocs.io/en/stable/advanced/cast/eigen.html).

We likely won't switch to PyBind11 anytime soon.
In the meantime, we could rewrite cvxcore/src/LinOp.hpp:set_sparse_data to map the data directly into a SparseMatrix, in order to avoid a copy (this is what set_dense_data does). See https://eigen.tuxfamily.org/dox/group__SparseQuickRefPage.html, "Mapping external buffers".
In principle, this should be easy: all you'd need to do is pass a buffer holding the data, and buffers for what SciPy calls indices and indptr (Eigen calls these InnerIndices and outerIndexPtr, respectively). We'd also need to modify the signature of set_sparse_data and pass in the indices/indptr, & modify the swig binding (cvxcore/python/cvxcore.i).
I wouldn't be surprised if there were lots more wasteful copies hiding in the code. But fixing this function is an important first step. Contributions are welcome!

I did some of the initial work in this branch: https://github.com/cvxgrp/cvxpy/tree/fewer_copies_in_cvxcore
If anyone does want to take a go at reducing memory consumption (@angeris ?), you could work off that branch as a starting point.
(The change looks bigger than it is, because it updates Eigen.)
The relevant changes in are LinOp.hpp and cvxcore.i. The change maps sparse data held by atoms directly into cvxcore, reusing the SciPy sparse matrix buffer. There are many more copies in LinOpOperations.cpp, so this change alone won't get you any memory savings (the functions get_constant_data, sparse_reshape_to_vec, and get_mul_mat, for example, do unnecessary copies). I wouldn't be surprised if there were copies earlier on in Python as well.
It would be a good idea to make similar changes for dense data, at some point.

@angeris , I looked into this a bit more. I ran the second lower bound problem in your test script at master, with n=10000 (the one that runs after the call to complex_to_real). A few observations.

Canonicalization was very slow (around 15 minutes).
Memory consumption didn't spike to 50 GB (I only have 16 GB of RAM -- how much does your Mac have?). Python seemed to use about 3 GB of RAM at most, but I wasn't looking at top for the whole 16 minutes / I didn't profile memory consumption.
The objective is very large. The expression dual_tree has 630,006 nodes in it. I counted nodes via

    def count_nodes(root):
        if not root.args:
            return 1
        return 1 + sum(count_nodes(arg) for arg in root.args)
    num_nodes = count_nodes(dual_obj)
    print('num nodes: ', num_nodes)

After the dcp2cone canonicalization runs, your problem is going to be very very large (as measured by the number of expressions). Meaning the description of your problem is very large, even though the size of the numerical data is not that large.
You're probably going to need to vectorize the construction of your problem so that its description isn't so large.


Error Cannot inherit from non-open class 'CropViewController' outside of its defining module

i am getting this error when inheriting CropViewController

@hoangnam714
Which version are you using?

i am using CropViewController latest version, i using spm to install

@hoangnam714
For some reason, SPM still shows 1.7.5

Can you use exact version 2.19.0 instead for now

I tested with SPM exact version and it worked for me.
Let me know if it solved your issue.

@hoangnam714
I haven't heard feedback from you for a while so I just close this ticket for now. You can reopen it anytime if you still have the issue.

@hoangnam714 For some reason, SPM still shows 1.7.5
Can you use exact version 2.19.0 instead for now

Thanks, that's the solution


staging: bypass nuke reset resolution

Base version 3.16.9-quad-1.12.0 with added https://github.com/quadproduction/OpenPype/pull/743

staging released, closing.


refactor(app): Simplify upgrade/downgrade modal render logic

overview
This PR addresses #2401 by breaking up the modal render logic into more digestible components. This allowed for a more sane first pass at [not now] button closing vs ignoring a robot update, and popup behavior.
changelog

break up UpdateRobotModal into AppUpdate, RobotSync, and Reinstall modals
only ignore update when an available robot update is presented
remove AvailableAppUpdate info when robot sync is the preferred option

review requests
Please test:

(robot < app, app < latest)
(robot = app, app < latest)
(robot > app, app < latest)
(robot < app, app = latest)
(robot = app, app = latest)
(robot > app, app = latest)

TODO: (not in this PR)
There are still some outstanding items to be discussed later today for a follow up PR

[not now] button behavior (ignore update version vs close)

this PR takes a first pass at this
rather than hit the ignore endpoint to store on robot, we should be storing this in state or config

auto pop up behavior when robot or app update is available and not seen yet

Discussed flow with @pantslakz

If app update available when a user opens the app, redirect to app update screen in "More"

Clicking not now on app page sets that app update to seen in state (existing behavior)

A user then navigates to Robot setting page for a robot and sees auto pop up app update available modal, this is now in context of an app update to get the latest robot server update

Clicking not now sets robot app update seen in state (not the same seen as in app page) and sets robot update ignored (server call on robot)
Clicking [click here] syncs the robot (spinner) if its a reinstall or downgrade
Clicking [click here] shows release notes modal if its an upgrade

At this point no more auto pop ups for this app version for this session, no more robot update auto popups for this robot cause the version is ignored on robot

other notes

Heading for app update available always shows available app update version
Heading for sync robot modal always shows available robot wheel version (aka current app version)

Tested on wandering-👩:
robot < app, app < latest
3.5.1 | 3.6.0-beta1 | 3.6.0

[ ]

redirect to menu/app/update on load
[ ]
always show UpdateAppModal first when there is appUpdate.available,regardless of upgrade downgrade/reinstall action
[ ]
VersionList shows:

[ ]
current app version
[ ] current robot version

[ ] available app update version
[ ] [view app update] links to menu/app/update
[ ] [click here] shows available upgrade release notes (for robot, not app)
[ ] [not now] closes modal, sets current available wheel version to ignored

robot = app, app < latest
3.6.0-beta1 | 3.6.0-beta1 | 3.6.0

[ ] redirect to menu/app/update on load
[ ] always show UpdateAppModal first when there is appUpdate.available, regardless of upgrade downgrade/reinstall action
[ ] VersionList shows:

[ ] current app version
[ ] current robot version
[ ] available app update version

[ ] [click here] installs current available wheel (spinner modal screen)
[ ] [view app update] links to menu/app/update
[ ] [not now] closes modal, does not ignore this api version

robot > app, app < latest
3.7.0-beta1 | 3.6.0-beta1 | 3.6.0

[ ] redirect to menu/app/update on load
[ ] always show UpdateAppModal first when there is appUpdate.available, regardless of upgrade downgrade/reinstall action
[ ] VersionList shows:

[ ] current app version
[ ] current robot version
[ ] available app update version

[ ] [click here] installs current available wheel (spinner modal screen)
[ ] [view app update] links to menu/app/update
[ ] [not now] closes modal, does not ignore this api version


Form not submitting when using drizzle-zod to create schemes to use in zodResolver

Describe the bug
When using drizzle-zod to generate schemas from database schemas and using them in the useForm resolvers the onSubmit won't fire at all for some reason,
So for example:
export const userSchema = createInsertSchema(users);

and then using it in the useForm
const { handleSubmit, register, control, getValues } = useForm<User>({
    resolver: zodResolver(userSchema),
    defaultValues: async () => {
      const res = await fetch(`/api/users/`)
      const user = (await res.json()) as User;
      return user;
    },
  });

  const onSubmit: SubmitHandler<User> = async (data) => {
    console.log(data)

    await fetch('/api/users/updateUser', {
      method: "PUT",
      body: body: JSON.stringify(data),
    })
  };

in this example when you click the Submit from the form it doesn't work at all.
however if you build your own schema using zod it works fine.
For reference this was working about a about a week ago.
No errors show in the browser console or terminal, so i don't know what is up there
To Reproduce
Steps to reproduce the behavior:

create a schema using zod
create a zod schema using drizzle-zod and createInsertSchema
import that schema into a useForm resolver
try submitting the form (should not work)

Codesandbox link (Required)
very simple application that doesn't seem to submit:
https://codesandbox.io/p/sandbox/react-hook-form-zod-resolver-not-working-9c7qn9
Expected behavior
I expected that validation would work out of the box as usual

Desktop (please complete the following information):

OS: Mac ventura 13.5.2, Windows 10, Windows 11
Browser Chrome
Version latest

Additional context
References:

drizzle-zod
My app that i'm building

zodSchemas / db
profile settings page just commented out the resolver for it to work

how?


boto fix

Coverage remained the same at 39.511% when pulling 484c82f22fce2300fced1f4310d9c6a6746a7534 on akhon:ci-fix into 7840f8d5df003dba9943f218b95369ed5a8b485d on bninja:master.


 IF YOU DO NOT FOLLOW THE FORMAT OR MAKE A GENERIC TOKEN REQUEST YOUR ISSUE WILL BE DELETED WITHOUT COMMENT
YOUR JUSTLIST MUST FOLLOW THE JSON SPECIFICATION
https://github.com/justswaporg/justlists/blob/main/example.justlists.ts
Checklist

[ ] I understand that this is not the place to request a token listing.
[ ] I have tested that my JustList is compatible by pasting the URL into the add a list UI at justswap.org.
[ ] I understand that filing an issue or adding liquidity does not guarantee addition to the justlists website.

Please provide the following information for your token.
JustList URL must be HTTPS.
JustList URL: https://github.com/c-exinsco/CXTron/blob/main/lgflist.json
JustList Name: CXTron
Link to the official homepage of the JustList manager:
Slack: https://cexinscocrypt-dod6095.slack.com/
Facebook: https://facebook.com/cexinsco
LinkedIn: https://linkedin.com/in/cexinsco
Twitter: https://twitter.com/cexinsco
Instagram: https://www.instagram.com/cexinsco
Discord: https://discord.new/QQy9fskxavaD
Pinterest: https://pinterest.com/cexinsco
Reddit: https://www.reddit.com/user/cexinsco
Tiktok: https://tiktok.com/@cexinsco?lang=en
Telegram: https://t.me/cexinsco
Medium: https://c-exins-co.medium.com/
Youtube: https://www.youtube.com/channel/UCjtssoExd5cf9NNKooyvW4g
Website: https://www.c-exins.co/
Email<EMAIL_ADDRESS>Saturday, 29 May, 2022
C-EXINS Blockchain Department, Odeonus.com

Hello,c-exinsco.I hope you have a good day.
The CXT token can not find in SunSwap V1 ,V1.5 and V2, you must first create createExchange in SunSwap by youself. Specific methods can be referred to https://www.sunswap.com/docs/sunswap-interfaces_en.pdf.

This issue will be closed, if you want to submit a new JustList, please resubmit.


Make sure to have an annotated tag on each release

What should we do?
Make sure to have an annotated tag on each release.
Why should we do it?
When I run npm run release for Zeebe Modeler 0.9.1 release, lerna created just an ordinary git tag instead of an annotated one. This caused Travis CI to skip the commit in the distro stage. I had to replace tag manually. @pinussilvestrus confirmed that he experienced the same issue.

What is an ordinary vs. an annotated tag? :thinking:

git tag v0.9.1 (that's what probably lerna run with npm run release, outcome)  v.  git tag -a v0.9.1 (outcome. In the second case you get a commit which contains only the commit message plus reference to tag.
I guess we could just fix it on CI. 🤷

I do not see the relevant difference.
In both cases, CI will pick up the new tag and build it. Right?

Nope, that's the thing here. In the first scenario, CI does not trigger the distro builds. Maciej had to annotate the tag manually to properly work

OK let's clarify this issue. The problem was that I had to manually create a tag commit to have the release picked up by CI.

I updated the issue description.


Replace #2603's (Rust Symbol Mangling) "v2" with "v0".

When #2603 was implemented, we decided on these names (https://github.com/rust-lang/rust/pull/57967#issuecomment-463596992):

legacy, for the pre-#2603 mangling based on the Itanium C++ one
v0, for the #2603 mangling

That differs from the RFC, so I've renamed away the "v2" in the RFC title (https://github.com/rust-lang/rfcs/pull/2603#event-3127804967) and the tracking issue (https://github.com/rust-lang/rust/issues/60705#event-3127801831), and that leaves the file itself.
cc @rust-lang/compiler

Seems fine.

This will impact the URL of https://rust-lang.github.io/rfcs/2603-symbol-name-mangling-v2.html -- but I don't know to what extent we expect those links to be truly "permanent". (I've always wished those links only had the RFC number, so that they were more "guessable" if nothing else.)

I'm just going to merge it =)


WIP, do not merge: update for Scala 2.12.0-RC1

I made these changes for the sake of the Scala 2.12 community build.
I'm submitting them as a WIP PR in case someone decides to resurrect
this project and finds the work I did here useful.

oh, ha, I didn't realize there was a newer fork over at https://github.com/sbt/sxr . I'll attempt to change the community build to use that instead. it's https://github.com/scala/community-builds/issues/300 now


dffml: Removed unused imports

Removes unused imports  throughout the library
Used the following command:$  git ls-tree -r remove_unused_imports --full-name  | awk '{print$4}'|grep "\.py" | xargs autoflake --in-place --remove-all-unused-imports --ignore-init-module-imports

Codecov Report

Merging #1209 (d0a295c) into master (38390a4) will decrease coverage by 0.15%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##           master    #1209      +/-   ##
==========================================
- Coverage   85.29%   85.13%   -0.16%
==========================================
  Files         171      171
  Lines       10887    10805      -82
  Branches     1788     1788
==========================================
- Hits         9286     9199      -87
- Misses       1245     1250       +5
  Partials      356      356

Impacted Files
Coverage Δ

dffml/cli/cli.py
62.35% <ø> (-0.42%)
:arrow_down:

dffml/configloader/json.py
100.00% <ø> (ø)

dffml/df/base.py
87.77% <ø> (+0.23%)
:arrow_up:

dffml/df/memory.py
79.74% <ø> (-0.06%)
:arrow_down:

dffml/df/types.py
82.48% <ø> (-0.05%)
:arrow_down:

dffml/model/slr.py
96.42% <ø> (-0.07%)
:arrow_down:

dffml/optimizer/parameter_grid.py
45.45% <ø> (-1.22%)
:arrow_down:

dffml/plugins.py
91.42% <ø> (-1.08%)
:arrow_down:

dffml/util/cli/parser.py
100.00% <ø> (ø)

dffml/util/net.py
87.80% <ø> (-0.71%)
:arrow_down:

... and 44 more

Continue to review full report at Codecov.

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


FAQ block controller did not call LinkAbstractor when editing rich te…

If you dropped an image into the rich text description of an FAQ entry, when you went back to edit the entry, the image didn't show up. Added LinkAbstractor::translateFromEditMode() on each description which is what the content block does.

Thanks!


on search suggestions for the pipeline is not coming up

Currently when users search for something we are getting suggestions for different entities except pipelines.

Closing this as it is working after adding pipeline index in suggestion api.


Voltage mismatch issue

I am not 100% sure but ain't the GPIO pins of esp32 only outputs 3.3V? So won't that create a problem with a4988?

You don't get the 5v from the ESP but from the 5V line of your powersupply

Oh. Hello. I didn't mean that. The dir step and enable pins of A4988 to require 5V signals if I am not wrong. So if the esp outputs 3.3V. Won't that be a problem?

That is not a problem I am using this config quite a long time without any problems.


The Extraction mkMapping and decompose methods have code to determine…

… if a tuple is being used and whether this should be considered as formatted as array. The tuple_? method is a Seq that contains the classes for the 22 varieties of tuples and the tuple_? method does a O(n) scan on this frequently during the case class serialization / deserialization process. Currently, the code is performing contains first and then checking whether Tuples are formatted as arrays (where the default is false). This change reverses the order to short circuit this check.
Mailing List thread:
https://groups.google.com/forum/#!topic/liftweb/E8tHa_5MWt8

✅  @chriswebster has signed the contributors file
✅  There is a ML thread where a committer has asked for this fix.
This is golden for merge. Thank you for the contribution!

(Going to wait a beat on the merge since we're having some issues cutting the 3.2.0-M1 build, however.

Oh, JK, 3.2.0-M1 is out! Huzzah! Merging!


Inverted display causes offset in image

It seems part of the screen is not drawn on when using PortraitInverted or LandscapeInverted, as if the image is outside the screen. Though it's fine using Portrait or Landscape.
I am using st7789_240x240 method since that is the resolution of the display.
I will contact our provider and try to get the exact part number and datasheet.
It's supposed to be a st789v2 with 240x240 pixels.
Here are images of the behavior

This seems like an offset from 320 vs 240. Can you try using the vanilla st7789 constructor and see if that works? I suspect your display has the full 320x240 framebuffer and inverting makes it "scrolled" into the off area.

That is what I also suspect, though the "visible" display is actually 240x240. Using the 320x240 results in the same behavior. Maybe my device has a 320x240 frame buffer but a 240x240 display? Could that even be possible?

That is what I also suspect, though the "visible" display is actually 240x240. Using the 320x240 results in the same behavior. Maybe my device has a 320x240 frame buffer but a 240x240 display? Could that even be possible?

Oh yes, that's what most ST7789s are. I'm just wondering where the logical error with the offset is. I'm guessing setting the MADCTL to inverted will simply render "from the other side" so it's all shifted by the hw scroll buffer area (the 80 pixels off screen) so when using Portrait or Lanscape we go from "top to bottom" rendering. But when using inverted we go "bottom to top" where bottom is 320, but the display shows 0, 0 to 240, 240 so we render 80 x 240 pixels "out of view".

After testing out a bit, I can confirm that it's rendering out of view by 80 pixels. Maybe this could be fixed with the partial mode display area (PTLAR, 0x30)? I'm not sure though since I just started to work with the device.

After testing out a bit, I can confirm that it's rendering out of view by 80 pixels. Maybe this could be fixed with the partial mode display area (PTLAR, 0x30)? I'm not sure though since I just started to work with the device.

I think the confusion here is in the interplay of inverting and "visible area". Inverting using the MADCTL bit will simply make it consider "0, 0" to be on the other side of the framebuffer, but the visible area is still on the same physical pixels.
I'll think about how to best support those orientations and keep things simple. One workaround for now you can do is to create your own ModelOptions where you provide a custom window offset handler function that gives 80 for the right axis.
This is not correct of course, the driver should handle this internally but until I fix this, you can get things done that way.


zbm-builder.sh: Add support for running on hosts with SELinux enabled

This commit updates zbm-builder.sh to include a new -M option for specifying podman/docker volume mount flags. On systems with SELinux enabled, -M z must be specified so that the volume will be relabeled with the container_file_t SELinux type. Otherwise, the directory contents are not accessible within the container.
In addition, when using -l <source dir>, -O -e=DRACUT_NO_XATTR=1 must be specified. By default, when dracut copies files to its temporary directory, it runs:
cp --preserve=mode,xattr,timestamps,ownership <...>

This environment variable makes dracut disable the xattr flag. This is necessary because cp would try to preserve the security.selinux xattr, which is never allowed inside a container. This should have no impact on the generated initramfs image because ZFSBootMenu does not use any custom xattrs and cpio is incapable of storing them anyway. mkinitcpio does not try to set xattrs, so no workaround is needed for that.

I did tested with mkinitcpio. It doesn't need anything like DRACUT_NO_XATTR because it never tries to set xattrs in the first place.

I think this is a bit too specific. We're already half-way there with the -O flag, so I'd propose adding an -M flag that can be used to specify volume mount options for the /build and /zbm mountpoints.  If not empty, it can be added to the volume flags the way you are doing SELINUX_FLAG, but also lets other users to other volume weirdness with their containers. (The /zbm mount should still have ro forcefully inserted before any MOUNT_FLAGS that might be appended.)
As for the DRACUT_NO_XATTR, this can already be accomplished with -O --env=DRACUT_NO_XATTR=1.
Yes, this requires two flags instead of your one, but it is totally generic and can be simplified by either

Just aliasing the zbm-builder.sh command to add the -M z -O --env=DRACUT_NO_XATTR=1 option, or
Creating a zbm-builder.conf file with the contents

MOUNT_FLAGS="z"
RUNTIME_ARGS+=( --env=DRACUT_NO_XATTR=1 )

and making sure zbm-builder.sh sees the config.

Thanks for the feedback. I've updated the PR with your -M suggestion and updated the documentation accordingly.

Looks good, although I propose a few changes to the documentation in 00e500f33ce09cc9e843fccbeaede32f3bcbb4e5. Please edit the changes as appropriate or, if you approve, I'll squash and merge.

Thanks, @ahesford! I like the new wording you have in 00e500f33ce09cc9e843fccbeaede32f3bcbb4e5 as-is.

Thanks again!


Delaboration and pretty printing

What question should the reference manual answer?
How does Lean produce user-visible output? How can this be customized? How does it interact with syntax extensions?

Duplicate of #122


add feature to apply custom operation

Description
This PR adds the feature to support applying custom operations.
Tests Done:
All the tests were done when adapter was running within the container

[x] Installing linkerd
[x] Uninstalling linkerd
[x] Applying custom yaml for deployments and services
[x] Applying custom yaml for deployments and services in custom namespace
[x] Applying custom yaml for deployments and services in a custom namespace with annotation linkerd.io/inject: enabled. Deployments were meshed successfully.
[x] Deleting the deployments by applying custom yaml

Signed-off-by: Utkarsh Srivastava<EMAIL_ADDRESS>

Thank you for running these tests from within a container.


Activation checkpointing leads to significant throughput drop

It looks like activation checkpointing leads to tremendous throughput drops (factor  >100).
For reproduction, I attached config_example_mem_map_dataset_max.txt. Note that while the model has the same parameters in both experiments, their overall size is rather small. Could it be that AC is quite inefficient for smaller models?

I ran some benchmarks and it looks like, the throughput not only is significantly less ... but also even declines more over time.


Coercion improvements

closes #61 (removes  explicit LazyArray  check as it's not needed anymore)
closes #62, allows to do coerce(X, Count=>Continuous, Unknown=>Multiclass), deals with Missing too.

Codecov Report

Merging #63 into dev will increase coverage by 0.67%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##              dev      #63      +/-   ##
==========================================
+ Coverage   92.07%   92.75%   +0.67%
==========================================
  Files           8        8
  Lines         202      207       +5
==========================================
+ Hits          186      192       +6
+ Misses         16       15       -1

Impacted Files
Coverage Δ

src/coerce.jl
100% <100%> (+4.16%)
:arrow_up:

src/conventions/mlj/mlj.jl
92.59% <0%> (-0.52%)
:arrow_down:

Continue to review full report at Codecov.

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