Text and icons blurry on mesa 24.1

Hyprland Version
Hyprland, built from branch main at commit a42b984f51a00e88a13a45e1b5e9e3d4ec470254  ().
Date:
Tag:

flags: (if any)

System Information:
System name: Linux
Node name: fedora
Release: 6.6.3-413.asahi.fc39.aarch64+16k
Version: #1 SMP PREEMPT_DYNAMIC Sat Jan 27 17:19:54 UTC 2024

GPU information:

os-release: NAME="Fedora Linux Asahi Remix"
VERSION="39 (Thirty Nine)"
ID=fedora-asahi-remix
ID_LIKE=fedora
VERSION_ID=39
VERSION_CODENAME=""
PLATFORM_ID="platform:f39"
PRETTY_NAME="Fedora Linux Asahi Remix 39 (Thirty Nine)"
ANSI_COLOR="0;38;2;60;110;180"
LOGO=fedora-logo-icon
CPE_NAME="cpe:/o:fedoraproject:fedora-asahi-remix:39"
DEFAULT_HOSTNAME="fedora"
HOME_URL="https://fedora-asahi-remix.org/"
DOCUMENTATION_URL="https://docs.fedoraproject.org/en-US/fedora/f39/system-administrators-guide/"
SUPPORT_URL="https://discussion.fedoraproject.org/c/neighbors/asahi/asahi-help/94"
BUG_REPORT_URL="https://bugzilla.redhat.com/"
REDHAT_BUGZILLA_PRODUCT="Fedora"
REDHAT_BUGZILLA_PRODUCT_VERSION=39
REDHAT_SUPPORT_PRODUCT="Fedora"
REDHAT_SUPPORT_PRODUCT_VERSION=39

plugins:

Bug or Regression?
Bug
Description
Text and icons look really low resolution after updating to mesa-asahi 24.1.0, this doesn't happen on sway nor gnome/kde.
How to reproduce
Install/upgrade to 24.1.0~asahipre20240214, reboot
Crash reports, logs, images, videos
Here's a pic of the before and after:
Before:

After:

oopsie, looks like a mesa asahi bug.
Asahi is not officially supported, take it to their tracker.


Dynamic title, description and image not working.

I am setting dynamic value of title, description and image. But not working.
Template :
     <share-buttons
           [title]="fbShareTitle"
           [url]="'http://maprentalsstaging.azurewebsites.net'"
           [twitter]=false
           [google]=false
           [pinterest]=false
           [linkedIn]=false
           [tumblr]=false
           [reddit]=false
           [stumbleUpOn]=false
           [whatsApp]=false
           [description]="fbShareDescription"
           [tags]="'tags'">
    </share-buttons>

In component I am setting like this :
   fbShareTitle = '';
   fbShareDescription = "";

   ngOnInit() {
	this.fbShareDescription = 'This is a first sharing description';   // Working fine here.
	this.fbShareTitle = 'Testing';             // Working fine here

            this.propertyService.getProperyById(this.propertyId)
	     .subscribe((data: any) => {
	            	let object = Object.assign({}, data);
	            	this.fbShareDescription = object.desc;    // This is not working
	                this.fbShareTitle = object.title;     // This is not working
            });
   }

Getting error :

Please any suggestion?
Thanks

@RathodGirish facebook will always give you this on localhost, try some live URL and see..

Yeah thank @MurhafSousli
Its working fine in live URL.
Thanks a lot :)


addUserToGroup incorrect thrown error

Hi,
Just started using this api and really impressed, thanks for all the hard work so far!
I have noticed a problem with addUserToGroup feature though, I am able to successfully add a user to a group chat using the api despite an error being thrown...
ERR! ERROR in addUserToGroup -->  { __ar: 1,
ERR! ERROR in addUserToGroup -->    error: 1545052,
ERR! ERROR in addUserToGroup -->    errorSummary: 'Unable to add user.',
ERR! ERROR in addUserToGroup -->    errorDescription: 'The user could not be added to the conversation. Please try again later.',
ERR! ERROR in addUserToGroup -->    payload: null,
ERR! ERROR in addUserToGroup -->    jsmods: { require: [ [Object] ] },
ERR! ERROR in addUserToGroup -->    bootloadable: {},
ERR! ERROR in addUserToGroup -->    ixData: {},
ERR! ERROR in addUserToGroup -->    lid: '6215920767082530033' }
Are you able to look into this?
Thanks
Peter

Thanks for opening an issue about this.
We actually noticed it yesterday and it seems to be something funny on Facebook's side (since it still adds the user to the chat and you will get an event saying a user was added to the chat).  It's possible that they've changed the format of the request, so we'll keep an eye on it, but for now I guess I would say you should just ignore this error.

Okay no problems at all! Thanks for looking into that!
Peter

Definitely, thanks for bringing it to our attention :)


Help - Web agent and pushover configuration

Hi,
This is a help/ feature suggestion thread.
Is it possible to have several web agents connected to a single pushover agent (and or slack agent too) such that when an event is produced by a web agent the url sent by the pushover agent is the url of that specific web agent.
Currently the only way I see of doing this is to make a pushover agent for every web agent, this seems excessive.
Many Thanks
Spencer

You can send arbitrary data in your web agent event using the template option. If you want to set the URL dynamically to the scraped page you can use {{ _url_ }} or {{ _response_.url }} depending on your use case.

Hi, I've given that a go but I am getting an error with my config which stops it saving.
When type is html or xml, all extractions must have a css or xpath attribute (bad extraction details for "template")
My config is below, sorry about the phone screenshot.

The template isn't extracted and thus goes outside the extract option like this:
{
  "expected_update_period_in_days": "7",
  "url": "{{ url-e }}",
  "type": "html",
  "mode": "on_change",
  "extract": {
    "title": {
      "css": "h1",
      "value": "string(.)"
    },
  "template": {
    "url": "{{ _url_ }}"
  }
}

Just managed to get that working, thank you very much!
Last quick question how would the url be embedded into a slack message
{
  "webhook_url": "xxx",
  "channel": "#123",
  "username": "Huginn",
  "message": "Site change",
  "icon": "",
  "url": "{{_url_}}"
}

This is currently how I have it and its not worked

works a dream, thanks again!

unsubsribe
Unending @.***>于2022年6月3日 周五20:03写道：

The template isn't extracted and thus goes outside the extract option
like this:
{
"expected_update_period_in_days": "7",
"url": "{{ url-e }}",
"type": "html",
"mode": "on_change",
"extract": {
"title": {
"css": "h1",
"value": "string(.)"
},
"template": {
"url": "{{ url }}"
}
}
—
Reply to this email directly, view it on GitHub
https://github.com/huginn/huginn/issues/3133#issuecomment-1145892762,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AHH7F5D2H24J5WDU5WBNKATVNHX7PANCNFSM5XTATDIQ
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>


Multiple keys

Codecov Report

Merging #10 (32b94a3) into main (413bdc2) will increase coverage by 0.89%.
The diff coverage is 62.02%.

@@            Coverage Diff             @@
##             main      #10      +/-   ##
==========================================
+ Coverage   55.41%   56.30%   +0.89%
==========================================
  Files           5        6       +1
  Lines         157      222      +65
==========================================
+ Hits           87      125      +38
- Misses         62       83      +21
- Partials        8       14       +6

Impacted Files
Coverage Δ

cmd/jwks-server/main.go
0.00% <0.00%> (ø)

internal/config/config.go
63.79% <34.37%> (-36.21%)
:arrow_down:

internal/jwks/jwks.go
65.11% <82.35%> (+1.47%)
:arrow_up:

internal/config/rsa.go
88.88% <88.88%> (ø)

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 413bdc2...32b94a3. Read the comment docs.


Change Group name from "legend-pilot" to "legend-shared"

Following completion of https://github.com/finos/legend/issues/273 (change of group-name, to match new name), I wanted to suggest we take the next step and change the group-name from "legend-pilot" to "legend-shared" in GitLab. The pilot has ended and so too, I think, has the semantic value of "legend-pilot". How about we go just with "legend-shared", which represents better what these models are -- shared, public models built in and with Legend.

@pierredebelen @beekemarie @gs-bracej -- any thoughts on this? Seems to me like "legend-shared" would be more logical name  for our collection of model repos in GitLab.

I think that would be confused with the legend-shared code repo we have. I would also be concerned about it becoming a dumping ground for models without clear ownership. Maybe the thing to do would be to have an ISDA group for CDM and similar models, a FINOS-FO group for models owned by FINOS, and a Scratch group for things that shouldn't be used in production?

Good point @gs-bracej. It can't be named "legend-shared"
I like the idea different groups for different purposes. I especially like the implication that the name and group-name could/should match the name of the overarching FINOS project (or external standards body/initiative) associated with the work. What do others think?
Also a reminder of the loosely related discussion at https://github.com/finos/community/issues/64 about on-going modeling efforts that may leverage the shared Legend instance.


Azure Key Vault TemplateSpec Solution-- DO NOT MERGE AS 1P

Change(s):
See guidance below
Reason for Change(s):
This PR is part of template spec solution migration for solution Azure Key Vault.
See guidance below
Version Updated:
2.0.0
Required only for Detections/Analytic Rule templates
See guidance below
Testing Completed:
The link for test results is - https://microsoftapc.sharepoint.com/:w:/r/teams/SentinelCATTeam/_layouts/15/Doc.aspx?sourcedoc={A5119E85-10AA-4F7B-A8C5-AAAF63E9699A}&file=Sentinel Solution Migration -TemplateSpec - Azure Key Vault.docx&action=default&mobileredirect=true
See guidance below
This is an 1P solution.

Hi @v-sabiraj
Can you please get confirmation on this PR merging or closing and act accordingly.
Thanks

@v-atulyadav Please resolve conflicts.


Some word change suggestions:

"This is a beginner-level post, just to clarify and visualize JavaScript prototype-based inheritance."
could be:
"This is a beginner level post to clarify JavaScript prototype-based inheritance via words and diagrams."

"A lot of incomplete and even wrong info can be found on Internet about JavaScript prototypal inheritance. I will just try to explain it again with help of diagrams."
could be:
"Much online information about JavaScript prototypal inheritance is scattered, wrong, confusing and misleading. This explains it as clearly, truthfully and easily as I can with clear images and words."

"JavaScript inheritance understanding is important, even if you are not going to use JavaScript OOP patterns, since many of language built-in functionality based on inheritance."
could be:
"Understanding JavaScript's inheritance approaches is important because the core tenets of its design and operation have been designed around its OOP patterns and abilities. Even if you don't plan on using these it's illuminating to understand the origin and reasoning of the language's methodologies."

"I am not advocating of using OOP patterns and "classical"-like inheritance in JavaScript at all. I personally prefer using "factory" and "mixin" instead of a "constructor" pattern. But this post is not about patterns, it is only about JavaScript prototype-based inheritance mechanics visualization."
could be:
This article is clarification of prototype-based inheritance mechanics and understanding them through visualisation. My personal preferences for use of "factory" and "mixin" over "constructor" patterns does not colour the article. I am not advocating use of OOP patterns and "classical"-like inheritance in Javascript.

@dissidently Yes, article really need some editorial assistance

It is an achievement in its own right. There is no other exhaustive and direct attempt (that I can find) to clarify all these things in anywhere near the clarity and earnestness you're doing. I "forked" and have done some more editing. But I still need to understand JavaScript more fully. I'm not completely grokking prototypes and its constructor ideas, yet.

I may need to ask you questions about how to think about aspects of JavaScript to best edit what you're saying, sometimes. Apologies in advance. I'm not a programmer. Design is my limit.

Ok, let's do it


Update Posteriors if Denominator is Zero

https://trello.com/c/KXNyFaYc/219-low-n-rt-posterior-reset-causes-red
This PR addresses issue #
Please Check

[ ] Are the JSON schemas updated?
[ ] Are the api docs updated?
[ ] Are tests passing?

Hey @jeremiq:
The reseting posterior to the initial prior is breaking county colors because the prior is so high. Can we either do this fix (just keep passing forward the previous day's posterior) or can we have a second "reset prior" that is centered around 1?


Don't show autocompletion for generic type, when the generic parameter is not matched

Consider this code:
declare global {
  /** @tsplus type Array */
  export interface Array<T> {}
}

/**
 * @tsplus fluent Array sum
 */
export function sum(self: Array<number>): number {
  return self.reduce((prev, cur) => prev + cur, 0)
}

/**
 * @tsplus fluent Array average
 */
export function average(self: Array<number>): number {
  return self.sum() / self.length
}

const arrayOfObjects = [{ a: 1 }];

arrayOfObjects.^

The extension methods sum and average need a subtype of Array<number>, and so should not be in the list of autocompletions when the type is Array<{a:1}>:

This is expected, follows the same convention as instance methods:


[SEDONA-520] Add apache-commons jar to snowflake dependencies

Did you read the Contributor Guide?

Yes, I have read Contributor Rules and Contributor Development Guide

Is this PR related to a JIRA ticket?

Yes, the URL of the associated JIRA ticket is https://issues.apache.org/jira/browse/SEDONA-520. The PR name follows the format [SEDONA-XXX] my subject.

What changes were proposed in this PR?
Added missing commons-lang3 dependency to snowflake/pom.xml.
How was this patch tested?
Passed the snowflake CI.
Did this PR include necessary documentation updates?

No, this PR does not affect any public API so no need to change the docs.

There is a separated PR for updating the snowflake CI https://github.com/wherobots/sedona-snowflake-tester/pull/21/files. This needs to be merged.

@jiayuasu Is there any repository-level settings for SNOWFLAKE_GEOTOOLS_VERSION environment variable? I've changed the environment variable in snowflake-tester.yml but the workflow job is still using the old version.

@Kontinuation No, this repo does not have any repo level setting for geotools version. See

The organization also has no org level variables at all

BTW, 2 concurrent Snowflake PR will conflict with each other since they both use the same database and stages

I have confirmed that now the snowflake tester is picking up 1.5.1-28.2

I've re-run the CI and it has passed: https://github.com/wherobots/sedona-snowflake-tester/actions/runs/8410338354/job/23028649572?pr=20. The fix seems to be valid.


Issue #465: Wrong indexing of Struct touched by Rays

Motivations:

See issue #465
Was maybe related to issue #459 ?

Main changes:

Rays.get_indStruct_computeInOut() is not a property anymore but a method
It returns 2 arrays of indices (indIn and indOut) instead of one (indStruct: can be retrieved by concatenating the two)
It can natively include only a unique StructIn (as required by Rays._prepare_inputs_kInOut())
Rays._prepare_inputs_kInOut() has been corrected:
To handle the changes mentioned above
And because the bug was coming from the wrong indexation of lists of structures

The bug is now fixed, and it does indeed also fix the so-called camera inversion mentionned in issue #459
Example:

In [1]: import tofu as tf
/home/didier/Documents/Projects/tofu/tofu/__init__.py:90: UserWarning:
The following subpackages are not available:
    - tofu.imas2tofu
    - tofu.mag
  => see print(tofu.dsub[<subpackage>]) for details.
  warnings.warn(msg)

In [2]: conf = tf.load_config('WEST-coils')

In [3]: [cc.set_color('b') for cc in conf.dStruct['dObj']['PFC'].values() if 'Casing' in cc.Id.Name or 'Thermal' in cc.Id.Name]
Out[3]: [None, None, None, None, None, None, None, None, None, None, None, None, None]

In [4]: cam = tf.geom.utils.create_CamLOS2D(P=[0.4, -3.5, 0], F=0.1, N12=1000, D12=0.1, angs=[np.pi, np.pi/5, 0], config=conf, Exp='WEST', Diag='Vis', Name='test')

In [5]: cam.plot_touch(quant='angles')
Out[5]: <tofu.utils.KeyHandler_mpl at 0x7fd4afeb6310>

In [6]: conf = tf.load_config('TCV-coils')

In [7]: conf.set_colors_random()

In [8]: cam = tf.geom.utils.create_CamLOS2D(P=[2.5, 0, 0], F=0.21944, angs=[np.pi, 0, 0], N12=[195, 487], D12=[0.03354, 0.083764], config=conf, Name='SXR', Exp='TCV', Diag='ME-SXR')

In [9]: cam.plot_touch()
Out[9]: <tofu.utils.KeyHandler_mpl at 0x7fd4ca02bc10>

Issues:

Fixes, in devel, issues #459 and #465
However, in issue #459, a separate problem is visible: some LOS have a very short length in the middle of the camera
This is probably due to the definition of the TCV wall, which is double
=> opening a separate issue

Codecov Report

Merging #466 (0158325) into devel (616375c) will decrease coverage by 0.02%.
The diff coverage is 66.66%.

@@            Coverage Diff             @@
##            devel     #466      +/-   ##
==========================================
- Coverage   51.20%   51.18%   -0.03%
==========================================
  Files          32       32
  Lines       19183    19185       +2
==========================================
- Hits         9823     9819       -4
- Misses       9360     9366       +6

Impacted Files
Coverage Δ

tofu/geom/_core.py
61.01% <66.66%> (-0.17%)
: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 616375c...0158325. Read the comment docs.


⚠ SolarOscp has degraded performance

In cf1110f, SolarOscp ($SOLAROSCP_URL) experienced degraded performance:

HTTP code: 200
Response time: 313 ms

Resolved: SolarOscp performance has improved in 47a7752 after 9 minutes.


Bug: Wrong schema detection in case of composite keys

Testing RepoDb on SQL Server and having composite key tables, i've run into the following behaviour. RepoDb cannot look out the complete composite key, using only the first field of it to determine the primary key. That results in total deletion of records which have equality on the determined field.
Here follows a sample code to replicate behaviour (TSQL + C#) :
   CREATE TABLE [dbo].[Product](
      CompanyCode VARCHAR(2) DEFAULT '' NOT NULL,
      ProductCode VARCHAR(25) DEFAULT '' NOT NULL,
      Description VARCHAR(50) DEFAULT '' NOT NULL,
      PRIMARY KEY(CompanyCode, ProductCode)
   )

    // Sample class
    public class Product
    {
        // First field of primary key
        public string CompanyCode { get; set; }
        // Second field of primary key
        public string ProductCode { get; set; }
        public string Description { get; set; }
    }

    // Code to replicate behaviour
    var p = new Product()
                {
                    CompanyCode = "CC",
                    ProductCode = "Product_01",
                    Description = "FOR TEST USE"
                };
   // The following command result in complete deletion of all records having CompanyCode = "CC", without considering ProductCode
   connection.Delete<Product>(p);

Thank you

@EmilianoMusso - thank you for this report. Yes, this is correct and it is a "known" issue as well in RepoDb even here in the organization. It can only support the non-composite Primary Key as of writing this.
Given with your sample:

var product = new Product()
{
	CompanyCode = "CC",
	ProductCode = "Product_01",
	Description = "FOR TEST USE"
};

The alternative for you is below.
connection.Delete<Product>(p =>
	p.CompanyCode == product.CompanyCode && p.ProductCode == product.ProductCode);

Or via dynamic like below.
connection.Delete<Product>(new { product.CompanyCode , product.ProductCode });

It is still fast as it using the columns of you Composite Key.
In relation to the support to Composite Key, we will discuss it internally, therefore, just leave this issue open.

@mikependon Thank you so much.
Yes, i've already tried dynamic and lambda prior submitting the issue, and they are ok, and fast - like you said.
Thank you for your answer, samples and time.

@EmilianoMusso, I have describe this composite keys scenario in our limitation page. After the further analysis, we cannot support this use-case as we are supporting the other use-case that is completely eliminating the validity of having the support to composite keys, for the Fluent-Based operations.
It is important to take note that the Non-Fluent-Based operations (i.e.: Query(TableName), Insert(TableName), etc) are completely supporting this, like the alternative solution that I recommended.

@mikependon Ok, thank you. Limitation page is clear enough, as are alternative solutions. Shall we close the issue, or you prefer to keep it active?

I created a label to tag this as limitation. Maybe just keep this one open so the community would see it still. Thank your for this report. 👍🏼


Error installing cocoapod

Hello,
I want to install but I have a problem with Cocoapods. I run the command line but still have an error. (see screenshot) Could you help me ? Thank you Sorry for my bad English I'm French;)

@Pitoux
Do you have Xcode installed ? Usually in flutter doctor -v shows below:
[!] Xcode - develop for iOS and macOS
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Xcode 12.3, Build version 12C33

Also, can you share the mkmf.log generated ?

Thanks for your feedback.
Here is the result

@Pitoux looks like CocoaPods are not installed on your machine. You can verify that by running pod --version. If CocoaPods are installed correctly you will see a similar output with the version number of the version of CocoaPods installed.

To install CocoaPods you can follow the instructions given by the  flutter doctor -v command. That is to run command sudo gem install cocoapods which is going to install the latest version of CocoaPods.

@daniil-shumko ,
when I type: "pod --version" I get the error message: "zsh: command not found: pod"
When I type "sudo gem install cocoapods" I get an error message

@Pitoux
Please file this issue in cocoapods repo, as this seems to be an issue in your setup and not an issue in flutter framework.https://github.com/CocoaPods/CocoaPods/issues
Closing from here.


send_message is not firing

send_message event is not firing:
v: 0.6.13
const GmailFactory = require("gmail-js");
const gmail = new GmailFactory.Gmail(window.$);
window.gmail = gmail;
window.addEventListener("load", function() {
   gmail.observe.on("load", () => {
      gmail.observe.on("http_event", function(params) {
            console.log("data:", params);   //working!
        });
       gmail.observe.on("compose", function(params) {
            console.log("data:", params);   //working!
        });
        gmail.observe.on("send_message", function(url, body, data, response, xhr) {
            console.log("on_send_message", "url:", url, 'body', body, 'email_data', data, 'response', response, 'xhr', xhr);   //doesn't work!
        });
        gmail.observe.after("send_message", function(url, body, data, response, xhr) {
            console.log("after_send_message", "url:", url, 'body', body, 'email_data', data, 'response', response, 'xhr', xhr);   //doesn't work!
        });
        gmail.observe.before('send_message', function(url, body, data, xhr){
            console.log("before_send_message", "url:", url, 'body', body, 'email_data', data, 'response', response, 'xhr', xhr);   //doesn't work!
        });
   });
});

I can reproduce this issue, but not sure why it fails.
Do you know if this is something which used to work for you, which is now broken in a recent release?
From what I can tell, there's no XHR request sent by gmail when a user click "Send" in the compos-window. If this is a change on Gmail's end, we will have a hard time detecting these events in the future.

It is first gmail-js version I'm working with.

Right. I've tested now with my production extension and the event fires there, just like it should.
But with your example, Gmail didn't even send the email. Based on that, I concluded that your usage of the events had to be wrong, effectively breaking Gmail in the process.
So I tried something simpler:
gmail.observe.on("send_message", console.log);
gmail.observe.after("send_message", console.log);
gmail.observe.before('send_message', console.log);

And voila, it works:

Your example includes a response parameter, and that's nowhere to be found in the definition. If you use TypeScript you will avoid issues like this and the TypeScript-compiler will tell you before you're even running the code :)
The correct signature is
private onMessageSent(url: string, body: string, data: any, xhr: XMLHttpRequest): void {
   // your code here
}

Basically, omit the response parameter, and you should be fine.

Hey there,
It does work fine, my mistake was I  wasn't building the project before testing it once it is updated.


Can't open jpegs

The app works fine for me when I try to open a directory with .png  but I get nothing when opening a directory with jpegs. All jpeg's are filtered out here if image.isNull(): in loadFile in labelImg.py
I have PyQt4 installed, on mac OS X 10.10.5, running on python 2.7.9.
Do you have any idea what might be going on? any suggestions?
Much Appreciated.

Any jpegs?  It's weird. Could you try to add log or print some messages in the bellow function.
https://github.com/tzutalin/labelImg/blob/a87ef4c70e31b468a58963c650bf1f39731bcaf0/labelImg.py#L750

hello Lin，this app is very good. one question: the Next Image button and the Prev Image button can not work ,how to fix it?
Thanks

@whcjb,
Do you mean you load the images from a folders, and next and previous button cannot work? What's your OS?

Yes, from a folder that have some images. My OS is Ubuntu 14.04-desktop

Yes,from a folder that has many images. My OS is Ubuntu-14.04-desktop, nearly same to that in your demo video


Install module from manifests without dependencies to module-manager

Description
This pull request introduces changes in the provisioning/installation workflow. It replaces module-manager dependency in favour of a new solution based on manifests rendered from the chart. The reconciler applies objects fetched from the manifests into the cluster and takes ownership of the created resources. The reconciler ensures that specs between manifests and cluster resources stay consistent and doesn't allow any difference between them.
Related issue(s)

See #73

/unhold

/retest

/assign


Not moving with keybord

DCCommentView is not moving up when it is tapped.
Code:
commentView = DCCommentView(scrollView: self.tableView, frame: self.view.bounds)
self.view.addSubview(commentView)
commentView.delegate = self
commentView.charLimit = 255
self.commentView.tintColor = GSColor.main

yeah I don't know why that is based off your code. Like I said in #2, I haven't looked or updated this code in years. That being said, I don't have any plans to look at this code in the near future, but a PR is welcomed if you find the cause. Best of luck!


RedisLabs cloud and "ReplyError: NOAUTH Authentication required" error

Issue
I am using RedisLabs cloud server to run a test project in which Redis connection looks like this:
if (process.env.REDIS_URL) {
  var client = redis.createClient(process.env.REDIS_URL);  // creates a new redisClient
} else {
  const redisHost = process.env.REDIS_HOST || '<IP_ADDRESS>';
  const redisPort = process.env.REDIS_PORT || 6379;
  const redisPassword =  process.env.REDIS_PASSWORD || '';

  const options = {
    host: redisHost,
    port: redisPort
  };

  if (process.env.NODE_ENV == 'production') {
    options.password = redisPassword;
  }

  var client = redis.createClient(options);  // creates a new redisClient
}

I've provided the Endpoint URL as REDIS_URL and it works just fine, but then spews out ReplyError: NOAUTH Authentication required errors every now and then. After some digging I found out it is an error related to the lack of password or something. There is a Default User Password field in the cloud server, so I've inserted its value as { password: "string" } options object in createClient(), but it hadn't get rid of the errors. Neither does Endpoint URL conform to the options.url format, so I don't know where to insert the password in there, if it's the right password in the first place.
Environment

Node.js Version: 12.18.4
Redis Version: 6.0.5
Platform: Windows 7 actually, that's why I am using cloud version.

@BassOfBass Any luck with this one, I've the same issue!

@ferasallaou You have to insert Database Name and Default User Password values as [user][:password@] in the redis_url as per url format, so no options object in the end, at least for the url.


Upgrades camshaft-reference to 0.8.0

Targets camshaft 0.14.0

cc @xavijam @viddo @javisantana

LGTM
On Friday, June 10, 2016, Raul Ochoa<EMAIL_ADDRESS>wrote:

Targets camshaft 0.14.0
[image: screen shot 2016-06-10 at 14 35 00]
https://cloud.githubusercontent.com/assets/45346/15964472/ffd456b4-2f18-11e6-8e5b-5a36787e6a92.png
cc @xavijam https://github.com/xavijam @viddo https://github.com/viddo
@javisantana https://github.com/javisantana
You can view, comment on, or merge this pull request online at:
https://github.com/CartoDB/cartodb/pull/7767
Commit Summary

Upgrades camshaft-reference to 0.8.0, targets camshaft 0.14.0

File Changes

M npm-shrinkwrap.json
https://github.com/CartoDB/cartodb/pull/7767/files#diff-0 (7)
M package.json
https://github.com/CartoDB/cartodb/pull/7767/files#diff-1 (2)

Patch Links:

https://github.com/CartoDB/cartodb/pull/7767.patch
https://github.com/CartoDB/cartodb/pull/7767.diff

—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/CartoDB/cartodb/pull/7767, or mute the thread
https://github.com/notifications/unsubscribe/AA7uHUSU-tA-ivMaGZ7_jBn5BOez5duaks5qKVrkgaJpZM4Iy6Wg
.

--
Nicklas Gummesson, software engineer
CartoDB — Map Your World's Data
Live sessions to learn CartoDB http://cartodb.com/webinars/


How to know whether NNI is using GPU?

Describe the issue:
i can run the example code of minst_pytorch. i guess i set NNI to use GPU, i don't know that for sure.
whatever you set in the config.yml, you can always see 'no_cuda': False in the trial.log(i put trial.log under this)
Environment:

NNI version: 2.6
Training service (local|remote|pai|aml|etc): local
Client OS: ubuntu 18.04
Server OS (for remote mode only):
Python version: 3.8
PyTorch/TensorFlow version: 1.10.1
Is conda/virtualenv/venv used?:  virtualenv in anaconda
Is running in Docker?: no

Configuration:

Experiment config (remember to remove secrets!):

searchSpaceFile: search_space.json
trialCommand: python3 mnist.py  # NOTE: change "python3" to "python" if you are using Windows
trialGpuNumber: 1
trialConcurrency: 1
maxTrialNumber: 10
maxExperimentDuration: 1h
tuner:
name: TPE
classArgs:
optimize_mode: maximize
trainingService:
platform: local
trainingService:
platform: local
useActiveGpu: true           # NOTE: Use "true" if you are using an OS with graphical interface (e.g. Windows 10, Ubuntu desktop)
maxTrialNumberPerGpu: 1      #   Reason and details:  https://nni.readthedocs.io/en/latest/reference/experiment_config.html#useactivegpu

Search space:
{
"batch_size": {"_type":"choice", "_value": [16, 32, 64, 128]},
"hidden_size":{"_type":"choice","_value":[128, 256, 512, 1024]},
"lr":{"_type":"choice","_value":[0.0001, 0.001, 0.01, 0.1]},
"momentum":{"_type":"uniform","_value":[0, 1]}
}

Log message:

nnimanager.log:
[2022-01-20 04:43:25] INFO (NNIDataStore) Datastore initialization done
[2022-01-20 04:43:25] INFO (RestServer) RestServer start
[2022-01-20 04:43:25] WARNING (NNITensorboardManager) Tensorboard may not installed, if you want to use tensorboard, please check if tensorboard installed.
[2022-01-20 04:43:25] INFO (RestServer) RestServer base port is 8080
[2022-01-20 04:43:25] INFO (main) Rest server listening on: http://<IP_ADDRESS>:8080
[2022-01-20 04:43:26] INFO (NNIManager) Starting experiment: 0zv4dxuc
[2022-01-20 04:43:26] INFO (NNIManager) Setup training service...
[2022-01-20 04:43:26] INFO (LocalTrainingService) Construct local machine training service.
[2022-01-20 04:43:26] INFO (NNIManager) Setup tuner...
[2022-01-20 04:43:26] INFO (NNIManager) Change NNIManager status from: INITIALIZED to: RUNNING
[2022-01-20 04:43:26] INFO (NNIManager) Add event listeners
[2022-01-20 04:43:26] INFO (LocalTrainingService) Run local machine training service.
[2022-01-20 04:43:26] WARNING (GPUScheduler) gpu_metrics file does not exist!
[2022-01-20 04:43:27] INFO (NNIManager) NNIManager received command from dispatcher: ID,
[2022-01-20 04:43:27] INFO (NNIManager) NNIManager received command from dispatcher: TR, {"parameter_id": 0, "parameter_source": "algorithm", "parameters": {"batch_size": 64, "hidden_size": 512, "lr": 0.0001, "momentum": 0.7760213206833312}, "parameter_index": 0}
[2022-01-20 04:43:31] INFO (NNIManager) submitTrialJob: form: {
sequenceId: 0,
hyperParameters: {
value: '{"parameter_id": 0, "parameter_source": "algorithm", "parameters": {"batch_size": 64, "hidden_size": 512, "lr": 0.0001, "momentum": 0.7760213206833312}, "parameter_index": 0}',
index: 0
},
placementConstraint: { type: 'None', gpus: [] }
}
[2022-01-20 04:43:37] INFO (NNIManager) Trial job Bqb0p status changed from WAITING to RUNNING
[2022-01-20 04:45:42] INFO (NNIManager) Trial job Bqb0p status changed from RUNNING to SUCCEEDED
[2022-01-20 04:45:42] INFO (NNIManager) NNIManager received command from dispatcher: TR, {"parameter_id": 1, "parameter_source": "algorithm", "parameters": {"batch_size": 64, "hidden_size": 512, "lr": 0.001, "momentum": 0.7308649721316198}, "parameter_index": 0}
[2022-01-20 04:45:47] INFO (NNIManager) submitTrialJob: form: {
sequenceId: 1,
hyperParameters: {
value: '{"parameter_id": 1, "parameter_source": "algorithm", "parameters": {"batch_size": 64, "hidden_size": 512, "lr": 0.001, "momentum": 0.7308649721316198}, "parameter_index": 0}',
index: 0
},
placementConstraint: { type: 'None', gpus: [] }
}
[2022-01-20 04:45:53] INFO (NNIManager) Trial job iTtJE status changed from WAITING to RUNNING
[2022-01-20 04:47:53] INFO (NNIManager) Trial job iTtJE status changed from RUNNING to SUCCEEDED
[2022-01-20 04:47:53] INFO (NNIManager) NNIManager received command from dispatcher: TR, {"parameter_id": 2, "parameter_source": "algorithm", "parameters": {"batch_size": 64, "hidden_size": 128, "lr": 0.0001, "momentum": 0.4727135784273845}, "parameter_index": 0}
[2022-01-20 04:47:58] INFO (NNIManager) submitTrialJob: form: {
sequenceId: 2,
hyperParameters: {
value: '{"parameter_id": 2, "parameter_source": "algorithm", "parameters": {"batch_size": 64, "hidden_size": 128, "lr": 0.0001, "momentum": 0.4727135784273845}, "parameter_index": 0}',
index: 0
},
placementConstraint: { type: 'None', gpus: [] }
}
[2022-01-20 04:48:04] INFO (NNIManager) Trial job Cwqkr status changed from WAITING to RUNNING
[2022-01-20 04:50:04] INFO (NNIManager) Trial job Cwqkr status changed from RUNNING to SUCCEEDED
[2022-01-20 04:50:05] INFO (NNIManager) NNIManager received command from dispatcher: TR, {"parameter_id": 3, "parameter_source": "algorithm", "parameters": {"batch_size": 64, "hidden_size": 512, "lr": 0.1, "momentum": 0.5413591322473582}, "parameter_index": 0}
[2022-01-20 04:50:10] INFO (NNIManager) submitTrialJob: form: {
sequenceId: 3,
hyperParameters: {
value: '{"parameter_id": 3, "parameter_source": "algorithm", "parameters": {"batch_size": 64, "hidden_size": 512, "lr": 0.1, "momentum": 0.5413591322473582}, "parameter_index": 0}',
index: 0
},
placementConstraint: { type: 'None', gpus: [] }
}
[2022-01-20 04:50:15] INFO (NNIManager) Trial job Nfbz9 status changed from WAITING to RUNNING
[2022-01-20 04:52:15] INFO (NNIManager) Trial job Nfbz9 status changed from RUNNING to SUCCEEDED
[2022-01-20 04:52:16] INFO (NNIManager) NNIManager received command from dispatcher: TR, {"parameter_id": 4, "parameter_source": "algorithm", "parameters": {"batch_size": 128, "hidden_size": 256, "lr": 0.0001, "momentum": 0.21466705347551573}, "parameter_index": 0}
[2022-01-20 04:52:21] INFO (NNIManager) submitTrialJob: form: {
sequenceId: 4,
hyperParameters: {
value: '{"parameter_id": 4, "parameter_source": "algorithm", "parameters": {"batch_size": 128, "hidden_size": 256, "lr": 0.0001, "momentum": 0.21466705347551573}, "parameter_index": 0}',
index: 0
},
placementConstraint: { type: 'None', gpus: [] }
}
[2022-01-20 04:52:26] INFO (NNIManager) Trial job lwoDF status changed from WAITING to RUNNING
[2022-01-20 04:54:16] INFO (NNIManager) Trial job lwoDF status changed from RUNNING to SUCCEEDED
[2022-01-20 04:54:17] INFO (NNIManager) NNIManager received command from dispatcher: TR, {"parameter_id": 5, "parameter_source": "algorithm", "parameters": {"batch_size": 32, "hidden_size": 128, "lr": 0.1, "momentum": 0.5981442137999028}, "parameter_index": 0}
[2022-01-20 04:54:22] INFO (NNIManager) submitTrialJob: form: {
sequenceId: 5,
hyperParameters: {
value: '{"parameter_id": 5, "parameter_source": "algorithm", "parameters": {"batch_size": 32, "hidden_size": 128, "lr": 0.1, "momentum": 0.5981442137999028}, "parameter_index": 0}',
index: 0
},
placementConstraint: { type: 'None', gpus: [] }
}
[2022-01-20 04:54:27] INFO (NNIManager) Trial job Ps2UV status changed from WAITING to RUNNING
[2022-01-20 04:56:47] INFO (NNIManager) Trial job Ps2UV status changed from RUNNING to SUCCEEDED
[2022-01-20 04:56:48] INFO (NNIManager) NNIManager received command from dispatcher: TR, {"parameter_id": 6, "parameter_source": "algorithm", "parameters": {"batch_size": 128, "hidden_size": 128, "lr": 0.001, "momentum": 0.9064910421608436}, "parameter_index": 0}
[2022-01-20 04:56:53] INFO (NNIManager) submitTrialJob: form: {
sequenceId: 6,
hyperParameters: {
value: '{"parameter_id": 6, "parameter_source": "algorithm", "parameters": {"batch_size": 128, "hidden_size": 128, "lr": 0.001, "momentum": 0.9064910421608436}, "parameter_index": 0}',
index: 0
},
placementConstraint: { type: 'None', gpus: [] }
}

dispatcher.log:
[2022-01-20 04:43:24] INFO (nni.experiment/MainThread) Creating experiment, Experiment ID: 0zv4dxuc
[2022-01-20 04:43:24] INFO (nni.experiment/MainThread) Starting web server...
[2022-01-20 04:43:25] WARNING (nni.experiment/MainThread) Timeout, retry...
[2022-01-20 04:43:26] INFO (nni.experiment/MainThread) Setting up...
[2022-01-20 04:43:26] INFO (nni.experiment/MainThread) Web UI URLs: http://<IP_ADDRESS>:8080 http://<IP_ADDRESS>:8080
[2022-01-20 04:43:26] INFO (nni.tools.nnictl.launcher/MainThread) To stop experiment run "nnictl stop 0zv4dxuc" or "nnictl stop --all"
[2022-01-20 04:43:26] INFO (nni.tools.nnictl.launcher/MainThread) Reference: https://nni.readthedocs.io/en/stable/Tutorial/Nnictl.html
[2022-01-20 04:43:27] INFO (nni.tuner.tpe/MainThread) Using random seed 755466869
[2022-01-20 04:43:27] INFO (nni.runtime.msg_dispatcher_base/MainThread) Dispatcher started

trial.log
[2022-01-20 04:43:37] PRINT {'data_dir': './data', 'batch_size': 64, 'batch_num': None, 'hidden_size': 512, 'lr': 0.0001, 'momentum': 0.7760213206833312, 'epochs': 10, 'seed': 1, 'no_cuda': False, 'log_interval': 1000}
[2022-01-20 04:43:40] INFO (mnist_AutoML/MainThread) Train Epoch: 1 [0/60000 (0%)]	Loss: 2.312253
[2022-01-20 04:43:51] INFO (mnist_AutoML/MainThread)
Test set: Average loss: 2.1372, Accuracy: 5719/10000 (57%)

[2022-01-20 04:43:51] INFO (mnist_AutoML/MainThread) Train Epoch: 2 [0/60000 (0%)]	Loss: 2.144817
[2022-01-20 04:44:02] INFO (mnist_AutoML/MainThread)
Test set: Average loss: 1.6174, Accuracy: 7143/10000 (71%)
[2022-01-20 04:44:02] INFO (mnist_AutoML/MainThread) Train Epoch: 3 [0/60000 (0%)]	Loss: 1.604976
[2022-01-20 04:44:14] INFO (mnist_AutoML/MainThread)
Test set: Average loss: 0.8386, Accuracy: 8141/10000 (81%)
[2022-01-20 04:44:14] INFO (mnist_AutoML/MainThread) Train Epoch: 4 [0/60000 (0%)]	Loss: 1.001007
[2022-01-20 04:44:25] INFO (mnist_AutoML/MainThread)
Test set: Average loss: 0.5419, Accuracy: 8583/10000 (86%)
[2022-01-20 04:44:25] INFO (mnist_AutoML/MainThread) Train Epoch: 5 [0/60000 (0%)]	Loss: 0.496065
[2022-01-20 04:44:37] INFO (mnist_AutoML/MainThread)
Test set: Average loss: 0.4267, Accuracy: 8808/10000 (88%)
[2022-01-20 04:44:37] INFO (mnist_AutoML/MainThread) Train Epoch: 6 [0/60000 (0%)]	Loss: 0.400071
[2022-01-20 04:44:49] INFO (mnist_AutoML/MainThread)
Test set: Average loss: 0.3682, Accuracy: 8958/10000 (90%)
[2022-01-20 04:44:49] INFO (mnist_AutoML/MainThread) Train Epoch: 7 [0/60000 (0%)]	Loss: 0.185010
[2022-01-20 04:45:01] INFO (mnist_AutoML/MainThread)
Test set: Average loss: 0.3286, Accuracy: 9064/10000 (91%)
[2022-01-20 04:45:01] INFO (mnist_AutoML/MainThread) Train Epoch: 8 [0/60000 (0%)]	Loss: 0.295494
[2022-01-20 04:45:13] INFO (mnist_AutoML/MainThread)
Test set: Average loss: 0.3036, Accuracy: 9119/10000 (91%)
[2022-01-20 04:45:13] INFO (mnist_AutoML/MainThread) Train Epoch: 9 [0/60000 (0%)]	Loss: 0.284811
[2022-01-20 04:45:25] INFO (mnist_AutoML/MainThread)
Test set: Average loss: 0.2797, Accuracy: 9184/10000 (92%)
[2022-01-20 04:45:25] INFO (mnist_AutoML/MainThread) Train Epoch: 10 [0/60000 (0%)]	Loss: 0.246093
[2022-01-20 04:45:37] INFO (mnist_AutoML/MainThread)
Test set: Average loss: 0.2610, Accuracy: 9239/10000 (92%)

no stdout no stderr

How to reproduce it?:
just copy my config.yml and paste into yours, then run nnictl create --config nni/examples/trials/mnist-pytorch/config.yml command.

You can add logging.info('CUDA available: %s', torch.cuda.is_available()) to your trial code, and check trial's log file.

You can add logging.info('CUDA available: %s', torch.cuda.is_available()) to your trial code, and check trial's log file.

thanks, now i can see the logo information, and i am sure CUDA is working.
but i have anthor question: how to set the suitable epoch_num of my experiment? i mean for different datasets the best epoch_num are different, can i use nni search that for me in condition that i do not know the search space of epoch_num(so i will not set epoch_num in my seach_space.json)

creating a new issue for the new question and closing this resolved question.


TypeError with Sphinx 5.0

When using autodoc_pydantic with the just-released Sphinix v5.0.0, we get a TypeError:
  File "/home/redacted/.venv/lib/python3.8/site-packages/sphinxcontrib/autodoc_pydantic/directives/autodocumenters.py", line 205, in add_content
    super().add_content(more_content, no_docstring)
TypeError: add_content() takes 2 positional arguments but 3 were given

This appears to be changed in sphinx's autodoc extension here: https://github.com/sphinx-doc/sphinx/commit/267954edba498dcc132f98c6a6e7fd029e8f6bbb#diff-e43bdd6f8f37a12d2536e09e57c5e8999cb8de18b9c7ba49126f90576c4328acR2666
It seems that autodoc_pydantic will also need to adjust accordingly.

@lukehsiao Thanks for reporting this issue. I can confirm that it occurs due to a changed function signature in sphinx 5.0.

@all-contributors please add @lukehsiao for bug

Bug is fixed and new version of autodoc_pydantic=1.7.1 is released to support sphinx=5.0.

Amazing, thank you!


Rename mapping/es to storage and start using elasticsearch lib

This commit renames mapping.es.py to storage.py to unify things with
other projects. This also adds initial usage of elasticsearch library
client and expands config to allow old and new code to coexist.
Closes #42
Related #37

Coverage decreased (-5.7%) to 79.63% when pulling a13cbc6d3c75d2a692f24acbe8d7aaafdfe47aa4 on use_storage into a9a6a807a4b0fa3260432e2a8321a3c65865d9f6 on master.


Getting Use model.to('Cuda') when trying to use Flash Attention

I installed all necessary drivers and packages, including nvcc to build Flash Attention, all smooth sailing. While everything works fine, when I try to use
insanely-fast-whisper --file-name audio.ogg --flash True
I get the following warning:
You are attempting to use Flash Attention 2.0 with a model not initialized on GPU. Make sure to move the model to GPU after initializing it on CPU with model.to('cuda')
However, the transcription still works, but - so I assume - without Flash.
I tried it on Ubuntu 22.04.4 LTS on 24 Cores 32GB + GTX4090 24GB and A100 80GB.
What did I miss?
PS: Maybe I should add that this is an awesome project. Thank you.

@Vaibhavs10 CC

Got the same issue...

I am also facing the same issue. Were any of you able to fix the issue? Thanks!

insanely-fast-whisper --task transcribe --flash  True  --device-id 0 --file-name  audio.mp3
You are attempting to use Flash Attention 2.0 with a model not initialized on GPU. Make sure to move the model to GPU after initializing it on CPU with model.to('cuda').
How to initialize model on GPU?


Update use-cannon

I still need one more update to use-cannon before this will build.

Waiting on https://github.com/pmndrs/use-cannon/pull/207

its up

its up

Thank you!

Something is broken, gonna revert this until we figure it out.


added count vector

Please follow the guidelines

Apologies for my initial mistake in ignoring few guidelines .  Being a first timer made it happen . The issue has been addressed I believe.
Thank you


Engine seperation of concerns

Re-organize the files that are related to the engine, this should go in the engine folder not in the data or game folder.

The core functionality typically provided by a game engine includes a rendering engine (“renderer”) for 2D or 3D graphics, a physics engine or collision detection (and collision response), sound, scripting, animation, artificial intelligence, networking, streaming, memory management, threading, localization support, scene graph, and may include video support for cinematics.

This issue was moved to danielricci/gosling-engine#4


Please update deps

The dependencies are already up-to-date.  A couple of the dev-dependencies are outdated, but that doesn't affect the functionality of the library.
Nonetheless, I'll add it to my to-do list to update the dev-dependencies soon.


Update promo code generation

Save gift card and voucher code in format ABCD-EFGH-IJKL instead of ABCDEFGHIJKL

Impact

[ ] New migrations
[ ] New/Updated API fields or mutations
[ ] Deprecated API fields or mutations
[ ] Removed API types, fields, or mutations
[ ] Documentation needs to be updated

Pull Request Checklist

[ ] Privileged queries and mutations are guarded by proper permission checks
[ ] Database queries are optimized and the number of queries is constant
[ ] Database migration files are up to date
[ ] The changes are tested
[ ] GraphQL schema and type definitions are up to date
[ ] Changes are mentioned in the changelog

Task linked: SALEOR-4115 Update gift card code

@fowczarek We don't have to, old codes will be working.


Consider adding values from some popular overhaul mods

In particular, Elden Ring Reforged

Good idea. I have no idea how easily accessible the weapon data is for those mods, so I'll have to look into it.

@ThomasJClark If you need help with figuring anything out, let me know; I work on Reforged.

Thanks @ividyon  will definitely let you know if I need help. I'm taking a look at the mod files to estimate the scope of this, but I think it should be totally viable to import the modified EquipParamWeapon/CalcCorrectGraph/ReinforceParamWeapon/etc. into the calculator

@ividyon a couple questions

I see Reforged changes how status buildup increases with weapon upgrade. Do you know which param table or field was changed to do this?  Is something different in the SpEffectParam for the status buildup effect, or is this a change to a script?
The regulation data has weapons with a "Reforged" affinity (e.g. "Gargoyle's Halberd [Reforged]") but there isn't a reinforce type or CalcCorrectGraph for these. Is this a placeholder for something, or an obtainable affinity?

@ThomasJClark
There is generally not much scripting in Souls for things like that, they're usually quite hardcoded. So params will hold all your answers. You can use Dsmapstudio to navigate them comfortably.
The Reforged affinity is not yet implemented and can be skipped.

I'm updating the TarnishedSpreadsheet's spreadsheet https://docs.google.com/spreadsheets/d/1ki0jmMkfeNdVxd3G1WggXdXrHHVWVAClRsYtsJvmTiI/copy
It's still WIP, only the items are done so far.

WIP preview link: https://err.elden-ring-weapon-calculator.pages.dev/. Pick "Elden Ring Reforged" in the dropdown on the left (or in the hamburger menu on mobile).
There are still some bugs I'm aware of and more testing to be done. In particular:

Some scaling curves aren't working, resulting in NaN attack power
Cipher Pata has 0 attack power for some reason
Missing icons for new affinities
Need to actually verify that the calculations match in-game stats

But overall it seems very close to working


Test Flowzone PR 488

Auto-generated by https://github.com/product-os/flowzone/pull/488
Will be closed automatically once the PR is merged.

Auto-closing pull request


Reverse Pop Route

Hi,
So...
Navigator.pop(context)) goes back to the previous page, however once on that page is it possible to then go back (forwards to the page that .pop was called on)? Would imagine something like a .pull(context)ish is doable as the state tree keeps history.
Pulling back from a pop would be useful for flipping in between pages when the route is unknown.
Looked around for documentation, did not find any.
Thanks

There is no way to restore the popped page, the page is disposed and no longer in memory. You have to push a new route again. What is the use cases you are trying to do?

Hi @chunhtai,
Trying to get the effect of a browser, for back and forward, to switch between views, via a back < and forward > button,  if I open a view I can go back but  once  back I cannot return to the previous view via a forward button.  Similar to the default AppBar leading but with a forward trailing.
Using this to navigate between views of charts, where once on a view a new view can be shown by tapping a chart.
Tried a solution where a view would have a wild card route like /view/:id and then caching the route name using route at index -1 to go forward, but wildcards isn't apart of the core Navigator.
Currently building for Web/Mobile/Unix :)

We are in the middle of designing a new router widget for web which will also take care of the forward and backward button for web. I am still not sure if i completely understand your use case, couldn't you cache the previous route name after you hit the back button and then push a new route with that route name when user hit forward button?

Caching works with pre-defined routes, where the route name is const, your suggestion would work.
But as routes cant be declared on the fly, a route with view/:id cannot be known before hand and therefore declared as a route.
Will keep trying. Thanks!

Maybe a little bit of example code can describe your use case better. I am not sure what do you mean by a route with view/:id

Closing this issue as inactive

Ended up using a Flutter package called Fluro. Forgot about  this, sorry for bad issue hygiene :)


Bugfix/get queue attributes timestmap

Any feedback?

My apologies, dropped off my radar in midst of other crazy :-)
Looks simple and good.


[DOCS] Improvements to docker-compose getting started installation guide

Hey team, as discussed on Slack, I'm opening this issue regarding a couple of improvements to the docker-compose  installation/deployment getting started guide:

The link to the docker-compose.yaml for the wget command seems to be outdated and should point to https://raw.githubusercontent.com/argilla-io/argilla/develop/examples/deployments/docker/docker-compose.yaml
If one wants to run this locally, the docker-compose.yaml has to be amended, and in particular line 39 #- xpack.security.enabled=false should be uncommented, otherwise the argilla-server container can't communicate with the elasticsearch container.
It could be worth mentioning also that, compared to the quickstart example, the default user used here is argilla with password 1234, instead of owner and 12345678.
Finally, even though as per the docker-compose.yaml comments, one could override the password and api-key of the default used by uncommenting and setting the variables DEFAULT_USER_PASSWORD and DEFAULT_USER_API_KEY. I could not get it to work, and the default user stayed unchanged. Maybe those comments are outdated? And maybe this is not really strictly speaking related to documentation, but I thought worth mentioning anyway.

I hope this is all useful!
Which page or section is this issue related to?

https://docs.argilla.io/en/latest/getting_started/installation/deployments/docker_compose.html

I can look into preparing a PR in the next days

For another data point, I had the same experience regarding bullet point 4, and haven't yet understood how to alter the default user credentials in the server image successfully via environment variables.
Also, if you do do this work @giorgionicoli, I think https://github.com/argilla-io/argilla/issues/4712 can be closed if you address bullet point 3.

I'm not sure what changed, but in starting to debug this further I noticed that upgrading to a new server image
(in my case from
argilla/argilla-server@sha256:6d80baa6359109797c72afb316c1417234ab781a1288797043440bc9d69f75fd -> argilla/argilla-server@sha256:b1db6256bbc614946ec648ee19b637ea65a1152924e33d2519dc441db3f5d2c6)
I can no longer reproduce the issue with setting the DEFAULT_USER_PASSWORD.
So that point may be addressed already in main, depending on how the newer image was built.

Thanks for providing the correct link to the docker compose here :).
I want to highlight that the link in the docs is still broken. Would be an easy fix I think and might save some folks the headache of searching the compose file.

It should be fixed in the latest documentation: https://docs.argilla.io/latest/getting_started/how-to-deploy-argilla-with-docker/. Credentials are fixed too: https://docs.argilla.io/latest/how_to_guides/user/#initial-users-and-credentials


Trigger on multiple tabs

I'm attempting to utilize the demo, and it works fine on a single tab. I'm trying to create a trigger that fires upon insertion. However, in the case of multiple tabs, the trigger only fires on the tab where the insert operation is executed. I'm aware that I could use a SharedWorker to share the database connection across tabs. But in my case, I need to support many version of browser like safari 15 ( that doesn't support Sharedworker)

You might (just speculating) be able to use storage events for that purpose. You could set aside a specific localStorage or sessionStorage key for that purpose, set it to some arbitrary value when you make a change, and react to that event in your other tabs. Someone in the sqlite forum suggested that idea for cross-tab notification of changes.

another option is the broadcast channel api. I'd expect this to be quicker than storage events since you don't need to write anything. https://developer.mozilla.org/en-US/docs/Web/API/Broadcast_Channel_API
I've hooked up the https://www.sqlite.org/c3ref/update_hook.html to fire events through a broadcast channel so all tabs can respond to changes made by other tabs. Note that exposing the update hook in wa_sqlite requires https://github.com/rhashimoto/wa-sqlite/pull/133

You might (just speculating) be able to use storage events for that purpose.

That can work, and might be the best choice before Safari 15.4 which OP might need here, but in general BroadcastChannel is a better signaling mechanism (as I previously mentioned here). It is delivered to all origin contexts, i.e. not just Window, can be precisely targeted (storage fires on every localStorage or sessionStorage change, potentially causing irrelevant callbacks), and can deliver any structured cloneable payload.
[Ninja-ed by @tantaman but I'll post anyway]


Historical Data from Legacy Systems- Comments from Ministry Staff on Student and Institution Accounts  ported over from legacy systems

Comments from Ministry Staff on Student and Institution Accounts  ported over from legacy systems

@HRAGANBC @CarlyCotton @Joshua-Lakusta can you please look deeper into this. Is this a requirement for the Part-Time Release and a priority for Ministry Users?

@michesmith @CarlyCotton - This has come up previously and been declined for approval into scope as it is not critical, it's just a quality of life thing for Service Delivery. SD staff will continue operating legacy systems alongside SIMS daily for the first 2 years of SIMS, hence why it was considered not a deal breaker for them to have to reference both systems to make a holistic assessment about a student.
That being said, it's never been formally explored or estimated. Is that what you're looking for? When this was talked about previously, it was not even considered for enhancement given the value case for this sunsets fairly quickly as more and more student activity is housed within SIMS and the old SFAS comments become more and more irrelevant with each year.


Move Doc struct to Doc.h

This splits out the Doc struct to its own header. The reason for doing this is to use the struct in AggregateWriteBuffer.h without introducing a direct circular dependency on P_CacheVol.h, but it seems good even as a standalone change so I'm PRing it separately. I moved some #defines that went along with the Doc struct as well.
The second commit in this PR adds a self_type alias to Doc. This might not have been necessary; I was looking for various things to clean up and did it while I was messing around.

[approve ci autest]

I heard some concerns from @cmcfarlen about putting moving this to a "public" header when it was in a "private" one before.


Observer over terminal-size values

Hi,
this issue is related to this comment in mitsuhiko/dialoguer#130.
We were trying to re-render prompts on resize of the terminal and I wasn't sure how to implement this. So I wanted to ask if someone here has an idea, how we can raise some kind of event when the terminal size changes.
My idea was to create a busy loop over these values and raise an event if some value has changed. But I have absolutely no idea how much load that would create while querying unix file descriptors or the winapi respectively. I have no experience with these kind of things.
Or maybe you have an idea for a completely different approach.
I'm definitely willing to make a PR to make this happen, but I wanted to clarify this stuff first.
I hope, someone can help :)
Best regards
Beh

Hi there, sorry for not replying sooner.  Constantly pooling for terminal size changes is not something that terminal-size should do.  On Linux, you can listen for the WINCH signal to know when the terminal size changes.  I bet Windows has some callback you can register, but I don't know it off-hand.
The tui crate can also send events when the terminal size changes, so looking at how tui does it might be useful too.
Good luck!


Boolean toggle randomly selects the wrong value from where you click

Bug report
Required System information

Node.js version: v14
NPM version: v6
Strapi version: v4.2.2
Database: SQLite3
Operating system: Linux Mint 20.3

Describe the bug
Sometimes when selecting a boolean value it will select the opposite value then where your mouse pointer is. This appears to be random per boolean field and it looks like it depends on if the boolean had a value before and you click clear
Steps to reproduce the behavior
Loom video shared internally as it contain's customer information. See internal TID1663 or my post in the product slack channel.
Short summary:

Create a content-type with a boolean field
Try to click on the boolean false
If the value is properly selected, hit clear
Try to click on true
Notice that the false value is selected instead

Alternatively repeat this logic selecting true first, clear, then select false if you cannot reproduce.
Expected behavior
Boolean value should select the value your mouse is on
Additional context
EE Customer, Internal TID1663

Internal Slack message: https://strapihq.slack.com/archives/CEN0F9WF6/p1657299258545899
Thanks for the report, @derrickmehaffy. Overall the component is supposed to work that way: it is a toggle and in fact it doesn't matter where you click (True or False) - the value will always be toggled. I agree though, that after clicking somewhere after the initial value being null, it should probably go into "True" state instead of "False".
What do you think?

Yeah I think as long as the interaction is consistent that would be a start. Was there a specific reason we moved to this logic instead of the "normal" toggle logic?


Wallet doesn't update confirmations

v2.0.5 Win10 64 bits
I've sent some testnet coins to my wallet and after 4 confirmations the wallet still show 0 confirmations, even after refreshing the page and restarting OB.

Testnet has been very flaky. @cpacia would that be the most likely explanation here?

Got another related error. Besides not receiving the testnet coins, after some time a retry payment button appears, but I've never done any payment on this wallet.

The try buttons appears when a transaction is stuck (i.e. has been unconfirmed for a certain period of time -I believe the server has it at 6 hours). It's a way that the recipient can help it along by bumping the fee. This is functionality that the Bitcoin api offers and we're making it available to the user. I think the idea is that the transaction may be stuck because the sender did not give a sufficient fee and this is a way the recipient can aid the process. Of course, in this case it may also be stuck because Testnet has been flaky.

It doesnt make sense. It is asking for the receiver to pay a fee to receive his money. But the receiver has 0 btc in his wallet.

That's a good point. I'm pretty sure if you hit Send, it would give you an error that you have insufficient funds. I suppose it would be better to hide the Retry button if there are no funds in the wallet.

I created an issue to make the text more clear and not show the retry option if the user doesn't have enough funds at #608

Closing this issue as the action items are tracked in other issues.


Fix 120 bug firstchild of undefined

Also explain what you do in the PR and reference the associated issue.

I will invert the names of methods and explain the full request . Thanks for your review :+1:

Cool, i think this could be a great thing to point to this pull request to explain how close the pop in with a button should be done.


allow the user to have a custom config filepath

we should make it so that a user can run arbiter see --config <name>.toml instead of only having predefined paths. it should still be able to accept arbiter see --config

in arbiter/crates/cli/src/config/mod.rs you will see this is where we parse the config file. If you run arbiter see --config right now it will iterate through the specified file paths of "where to look" for a file called either Config.toml or config.toml. So in the current implementation there is a specific place where the user must put their config file and it must have a specific name. thus the --config flag represents a boolean value that the program just checks is present. What we would like to do is all a user to "optionally" (use option type for the subcommand) specify a file name and path after the --config flag such that command would look something like this: arbiter see ---config somefilepathandname.toml so that the user can specify where their config file is and what it is called. Everything for this issue is in the cli crate.

Mind assigning me to this one?

@eljhfx on it.

PR: #69


Version of trancribestreaming sdk is not updated. It generates errors about added language

https://github.com/amazon-connect/amazon-connect-realtime-transcription/blob/0d0abe7c50a819561ca94b12cf4a3c63de298351/build.gradle#L11
The newest version is 2.16.10.
2.5.0 version generates many errors related added languages (ex> japanese, korean etc..)
At the end of troubleshooting about language problem, I got to know version is not updated.
I hope other people don't encounter mistake like me, So correct this ASAP..

I'm facing the same issue.
Here is the quick fix to make it work.
I have updated the lib to 2.16.21, and force Java 8 as runtime.
Java 8 may not be necessary if you're not using Lambda.
build.gradle
apply plugin: 'java'

java {
    toolchain {
        languageVersion = JavaLanguageVersion.of(8)
    }
}
repositories {
    mavenCentral()
}

dependencies {
    compile fileTree(dir: 'jars', include: '*.jar')
    compile group: 'io.reactivex.rxjava2', name: 'rxjava', version: '2.2.2'
    compile(
            'software.amazon.awssdk:transcribestreaming:2.16.21',

            'com.amazonaws:aws-java-sdk-dynamodb:1.11.475',
            'com.amazonaws:aws-java-sdk-kinesisvideo:1.11.475',
            'com.amazonaws:aws-lambda-java-core:1.2.0',
            'com.amazonaws:aws-lambda-java-events:1.2.0',
            'com.amazonaws:aws-java-sdk-cloudwatch:1.11.480',
            'org.slf4j:slf4j-api:1.7.24',
            'org.slf4j:slf4j-log4j12:1.7.24',

            // need this for our async clients
            'software.amazon.awssdk:netty-nio-client:2.2.0',

            // need this for logging
            'org.apache.commons:commons-lang3:3.6'
    )
}

task buildZip(type: Zip) {
    from compileJava
    from processResources

    into('lib') {
        from configurations.compileClasspath
    }
}

build.dependsOn buildZip

Here is the pull request I created - can you check to see if this fixed your issue? https://github.com/amazon-connect/amazon-connect-realtime-transcription/pull/57

Thanks. It works well with #57.


Make integration

Developers should be able to configure subo to execute Make targets as part of a build. This would allow for things like building static sites before creating a bundle.
For example, if I run subo build . --make app, I'd want subo to run make app before starting the actual build.

HI, I'm taking this one

Hi, I'm working on this

This was completed by @samderlust !


[Runtime] Fix findBridgeWitness when building for the shared cache.

The static bridgeableProtocol inherits the ptrauth_struct attribute which uses the B key, and that's not allowed on global data in the shared cache. Pass the value directly as a parameter instead.

@swift-ci please test


OADP-2049: Cleanup replication source CRs if retain policy not set

/cherry-pick oadp-1.2

@shubham-pampattiwar thank you! leaving the code review to the experts


[TODO] linear dampling

You can refer to:

https://github.com/open-spaced-repetition/ts-fsrs/pull/133
https://github.com/open-spaced-repetition/go-fsrs/pull/28

https://github.com/open-spaced-repetition/swift-fsrs/pull/21


Dynamically expose new gRPC's in a C++ server

We have a requirement to expose multiple gRPC services through a single server listening in a single port. We also need to add and remove RPC's on the fly (without re-start of the server).
Platform is Linux and container is written in C++.
The implementation of the service calls is fixed for all gRPC's as this container only needs to do some generic processing on the gRPC call before handing over to the actual back end which resides outside the container. This processing would involve things like access control, logging, rate limiting. Assuming this can be pre-written and loaded to the C++ container at start-up, what is left is to setup gRPC server (which is already running) to accept calls to new gRPC's and route them to the generic request processor. Similarly, we need to be able to remove gRPC's on the fly. This is like adding new http routes to an already running http server.
We are ok to dynamically load new libraries using 'dlopen' for example.
Is this supported in gRPC? If so where could I find more information.
Thank you
I think this
One possible way to do this would be:

Implement new service
Wrap this in a

Have you looked at grpcs generic API?


[BACKEND] MS-Windows servers support

Installing open SSH on Windows as optional feature is a requirement.
But login without password (using public key) does not seem to work.
Already tried with SSH authentication agent service started, but without success.

Next try did not bring results as well, W10 ssh server does not want to enable public key authentication.

Install 3rd party ssh server as open ssh for windows ?
Do not use public key auth and stick with intercative password typing from node-ssh?
...?


Assembled statements

This PR makes it easier to get latest assembled statements of a given model in multiple ways:

At assembly time, the assembled statements will be dumped into a constantly updated 'latest_statements_{model_name}.json' file to be found easily (first file in S3 directory) and a timestamped file for keeping track of changes
Download statements button on all statements page will download (if available) the file called 'latest_statements_{model_name}.json' instead of time-stamped one
New endpoint ('/latest_statements/') allows to get statement jsons and the link to latest file programmatically through API request

Nice!


Generalize circuit breaker SPI

CircuitBreaker SPI doesn't enumerate states anymore.
It only asks if it can be called and how to handle calls.
This lets you track any state and wire in advanced CircuitBreaker libs.
For example, resilience4j defines more exotic states like DISABLED or
or FORCED_OPEN.
It also counts duration of calls and various other features.
So instead of reimplementing all possible circuit breaker features,
the SPI only asks for methods, which db-replica actually needs.

CircuitBreaker SPI doesn't enumerate states anymore.
It only asks if it can be called and how to handle calls.
This lets you track any state and wire in advanced CircuitBreaker libs.
For example, resilience4j defines more exotic states like DISABLED or
or FORCED_OPEN.
It also counts duration of calls and various other features.
So instead of reimplementing all possible circuit breaker features,
the SPI only asks for methods, which db-replica actually needs.

I like we get rid of the state. It makes SPI easier, more powerful and more stable.
I don't like the handler, as it makes SPI worse without any benefit for the end-user.

CircuitBreaker SPI doesn't enumerate states anymore.
It only asks if it can be called and how to handle calls.
This lets you track any state and wire in advanced CircuitBreaker libs.
For example, resilience4j defines more exotic states like DISABLED or
or FORCED_OPEN.
It also counts duration of calls and various other features.
So instead of reimplementing all possible circuit breaker features,
the SPI only asks for methods, which db-replica actually needs.

I like we get rid of the state. It makes SPI easier, more powerful and more stable.
I don't like the handler, as it makes SPI worse without any benefit for the end-user.

Thanks for the comments folks! I'll catch up tomorrow

If I understand correctly, one way is to remove CircuitBreaker SPI entirely and let consumers handle that via their own DatabaseCall aspects. Is that the idea @wyrzyk?

#60 removed CB from API, so reshaping it is no longer necessary


Sonoff adapter not correcly supporting dimmer control

I use the sonoff adapter (Version 2.5.1) with an idino zigbee dimmer which ist connected to a sonoff zigbee bridge. The bridge is running on Tasmota 12.1.1 and if I use the console command zbsend() to switch or dim the lights everything works as expected. The dimmer shows up correctly on the object view and the current state (on/off) or dimmer value (0..254) is updated as expected in the object <sonoff.0.zbbridge.xxx.SENSOR.ZbReceived.0x3180.power> (or .dimmer) datapoints. So far so good. The issue is, that I cannot control the dimmer through the adapter/iobroker. The ...power/.dimmer datapoints senn to be read only. If I set them to "writeable" in the datapoint properties they still don't update. It seems like there is no command implemented to react to a change in the datapoint and send the corresponding command via MQTT / zigbee to the dimmer.
Everytime I change a value i get the log entry <Client "zbbridge.xxx.SENSOR.ZbReceived.0x3180" not connected>
There are no other ojects within the object tree that would allow for control of the dimmer.
Can this be easely fixed or is there a work around?
Thanks for any help

Does no one else have the same problem? I just can't seem to get the issue solved.


Define how filters are represented

Define how filter should be represented in the MTH5 file and as an object.

@kkappler Check that the list of filters that we think we may reasonably encounter has a 1:1 map to the filters we have created in mt_metadata.  Only then can you close this issue

Most filters now have a container and dynamic response to metadata changes.


add auto reload page

To reload the page with a new random number when the user guesses the correct answer.

Hai i already implemented that auto update random number on reload🤔.
If u are new to open source let me tell u before adding any features/ fixing bugs like this u have to raise an issue then when u are assigned u can work on it!
if u remove that reload line maybe then, i can merge your name!🤔 sounds good?

there is a conflict that need ur atttention!

okay sir, i done resolve for a conflict.


utils.getMonth fails when initial date is null (v4.0.0-alpha.8)

utils.getMonth fails when initial date is null
I tracked it down to this line
https://github.com/mui-org/material-ui-pickers/blob/853e5d2653f925f51278784618beb2e86f64ef0f/lib/src/views/Calendar/MonthSelection.tsx#L42

[X] The issue is present in the latest alpha release.
[X] I have searched the issues of this repository and believe that this is not a duplicate.

Current Behavior 😯
It fails on clicking on calendar month/year (as shown in attached photo)

Expected Behavior 🤔
It should not fail and it should guard against null value

Steps to Reproduce 🕹
Steps:

Navigate to: https://codesandbox.io/s/material-ui-pickers-usage-demo-8478h?file=/src/App.jsx
Click on calendar icon, click on July 2020 list(?)
it fails miserably 🤕

Context 🔦
I am trying to initialize a date picker field from null, no known initial value.

Your Environment 🌎
Node.JS, OS and Windows

Tech
Version

@material-ui/core
5.0.0-alpha.2 and latest 4.11.0

@material-ui/pickers
v4.0.0-alpha.8

React
16.13.0

Browser
Chrome, Edge and Firefox

TypeScript
3.9.6

Fixed by #2045


document enforceRequestTimeout

A thing I found very confusing and took some time to debug out.
In documentation there is requestTimeout: 30000,
https://kafka.js.org/docs/configuration#request-timeout
But it is not applied as there is another undocumented configuration parameter
"enforceRequestTimeout" which is false by default.
So by default without even checking timeouts setting requestTimeout param has no effect at all :-)
Why not turn on timeout checks by default? Or at least mention in docs that there is another param to set

That's a fair point, we introduced this property some time ago because we had performance issues creating individual timers, but this was already solved. We should document the property and plan to make the default true in a major version bump.

FYI we have been running our production system for quite a while with this flag on (after specified PR) and have not had any issues with performance.


IP/CIDR allocation method & tracking

@jb3 has been researching here and will be sharing with us shortly

Joe has concluded his research and after a lengthy research paper was published
and after even receiving a $250,000 grant from Leon "Lemon" Sandøy himself, his
research concluded that whilst driving a Ferrari is truly enjoyable, the
correct way to solve this matter of allocating IP addresses is to self-host the
most complicated cluster scheduling software known to man, Kubernetes. We are
excited to announce that as a result to this research, the general public had
the ability to for once contribute to great and useful research bringing
forward open source applications.
Furthermore, I would like to thank Joe for mentioning me in his research paper
as his chief research assistant, which has helped me get another $50,000 grant
for typing up this exact message. It is being spent on therapy sessions for the
abuse that Joe has put me through as we speak.


Uninstaller doesn't remove C:\ERAM folder

After uninstalling, the C:\ERAM folder remains with all the contents inside.

Heh, I can't even seem to find a (the?) uninstaller.


Export values of MutableArray

For example:
pub struct MutableBinaryArray<O: Offset> {
    data_type: DataType,
    offsets: MutableBuffer<O>,
    values: MutableBuffer<u8>,
    validity: Option<MutableBitmap>,
}

When we want to add methods into MutableBinaryArray, we can't access the fields like values...

The rational is that values and offsets are coupled: we can't expose values as a mut reference because the user may truncate it, leading to UB.

We can still offer a &mut [u8] reference to it.

Ok. Get it, seems we should build values and offsets on my own then construct them into MutableArray.
We don't like TryPush method in this MutableArray, if our array is not null, just need to push the values and offsets with out option wrap.

You mean to avoid the conditional branch that checks for the validity, right? I agree.
There is a use-case here to introduce an auxiliary struct that just cares about offsets, values and its invariants, and have the MutableBinaryArray use it. This way both you and MutableBinaryArray can use it, and you do not have to worry about unsafe on your side.

Yes, that's right. Since MutableBinaryArray  is not very complex now, we can have our own NoneNullMutableBinaryArray instead.


Build broken due to bevy_embedded_assets

SO I was trying to install the application by cloning the git repo and running cargo install --path . I also tried cargo build --release and cargo r --release all of them show the same errror behaviour.
First it asks to define BEVY_ASSET_PATH and once defined as assets the build fails with
❯ cargo r --release
   Compiling bevy_embedded_assets v0.7.0 (https://github.com/StaffEngineer/bevy_embedded_assets?rev=d9dac8d#d9dac8de)
   Compiling tantivy v0.19.2
   Compiling bevy_pkv v0.7.0 (https://github.com/StaffEngineer/bevy_pkv?rev=bf79a1a#bf79a1ae)
   Compiling bevy_markdown v0.2.0 (/export/home/toast/workspace/rust/velo/crates/bevy_markdown)
   Compiling bevy_smud v0.5.0 (https://github.com/StaffEngineer/bevy_smud?rev=3a7e902#3a7e9023)
   Compiling toml v0.7.4
   Compiling linkify v0.9.0
error: couldn't read /export/home/toast/.cargo/target/release/build/bevy_embedded_assets-c856f64c88a722f0/out/assets/example_asset: No such file or directory (os error 2)
 --> /export/home/toast/.cargo/target/release/build/bevy_embedded_assets-c856f64c88a722f0/out/include_all_assets.rs:3:59
  |
3 | embedded.add_asset(std::path::Path::new("example_asset"), include_bytes!("assets/example_asset"));
  |                                                           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  |
  = note: this error originates in the macro `include_bytes` (in Nightly builds, run with -Z macro-backtrace for more info)

error: couldn't read /export/home/toast/.cargo/target/release/build/bevy_embedded_assets-c856f64c88a722f0/out/assets/subdir/other_asset: No such file or directory (os error 2)
 --> /export/home/toast/.cargo/target/release/build/bevy_embedded_assets-c856f64c88a722f0/out/include_all_assets.rs:4:64
  |
4 | embedded.add_asset(std::path::Path::new("subdir/other_asset"), include_bytes!("assets/subdir/other_asset"));
  |                                                                ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  |
  = note: this error originates in the macro `include_bytes` (in Nightly builds, run with -Z macro-backtrace for more info)

error: couldn't read /export/home/toast/.cargo/target/release/build/bevy_embedded_assets-c856f64c88a722f0/out/assets/açèt: No such file or directory (os error 2)
 --> /export/home/toast/.cargo/target/release/build/bevy_embedded_assets-c856f64c88a722f0/out/include_all_assets.rs:5:50
  |
5 | embedded.add_asset(std::path::Path::new("açèt"), include_bytes!("assets/açèt"));
  |                                                  ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  |
  = note: this error originates in the macro `include_bytes` (in Nightly builds, run with -Z macro-backtrace for more info)

The following warnings were emitted during compilation:

warning: Asset folder found: assets

error: could not compile `bevy_embedded_assets` (lib) due to 3 previous errors
warning: build failed, waiting for other jobs to finish...

It looks to me to be the dependency that is broken but may be the setup to build aswell.
This happens with the master branch and the 0.7.0 tag

It also happens to me with 0.5.0

It looks to me the library treis to embed it's own assets https://github.com/StaffEngineer/bevy_embedded_assets/tree/main/assets which is a completely bogus behaviour imho.

Do you have an unusual .cargo/config.toml?
For me, I had to run it using
BEVY_ASSET_PATH="path\to\velo\assets" cargo run

I did set BEVY_ASSET_PATH and cleaned my cargo config from that settings. I had to set this variable to get this far.
Also the assets that it wants to include are inside bevy_embedded_assets which must not be included by the final app and are not present in the velo assets folder

I did set BEVY_ASSET_PATH and cleaned my cargo config from that settings. I had to set this variable to get this far.
Also the assets that it wants to include are inside bevy_embedded_assets which must not be included by the final app and are not present in the velo assets folder

Try to set it as absolute path to velo's assets folder.

@StaffEngineer thanks! that worked. the Fullpath made it compile.
I never had the asset path issue with other beavy projects before and I did try with different configurations in cargo build config. But to no help. I still think it's a bug in bevy_embedded_assets that it tries to package it's own assets if it doesn't find any assets to package though.

What bevy_embedded_assets does is:

First, check BEVY_ASSET_PATH environment variable, then just assumes it's correct and uses it (you've given it assets which just assumes relative path to itself).

If it isn't set (most people), then it looks for the target folder, then looks for the assets folder within the shared parent of the target folder, and for most people that, but for us, who've done cargo config. This feels hacky to me but that's how it's implemented.

If you have the absolute path to the assets folder set in the BEVY_ASSET_PATH environment variable, it will work.
Or, if you have a normal cargo config (so build target is within the root of the project), it will also work with the hack above.

It's actually not for caching but to reduce the size of my workspace folder to manageable sizes. I had situations before where it would grow above 100GB of artifacts in target. If sccache can prevent that then I am happy to start uign that instead. What I wanted to note though is that the target dir is a cargo modifiable feature and build scripts are asked to not rely on the target directory being inside the source dir.
As I have ZFS I can also apply compression and block deduplication to the cargo target dir if it is shared. :)
I would find it very nice if velo had a build.rs that simply sets the BEVY_ASSET_PATH to a full path and bevy_embedded_assets asks the developer to setup the build rather than assuming things not considered stable whcih it cannot get from the env.

I would find it very nice if velo had a build.rs that simply sets the BEVY_ASSET_PATH

It can be simply noted in the readme to set it as an absolute path. If anyone experiences any issues, feel free to create a pull request.

bevy_embedded_assets asks the developer to setup the build rather than assuming things not considered stable which it cannot get from the env

feel free to create an issue on bevy_embedded_assets repo.


(backport) Github CI fixes

We have to backport the set of Github CI fixes from last week because the feature is going to be deprecated. As such, our PR testing on this branch could work.
This brings all of the fixes, GH cleanups and related changes I implemented last week.
Note: there's 3 commits on core + components that are here with the intention of testing the patch set.

Components tested:
Total 	Tested 	Failed ❌ 	Passed ✅
2 	2 	1 	2

Awesome. It worked. I'll remove the cherry-picked cleanups and we should be good to merge.


Setup: User Testing 1 & 2

Page numbers following scenario: Non-admin user/Admin has set up secrets/Creating new test

[ ] P1. Specify what the tool does for the user.

[ ]  "This tool helps developers or organizations set up automated integrated testing for docassemble packages that have a GitHub repository. First, it will set up GitHub secrets, if necessary. Next, it will create a testing branch for your docassemble package on your Github account that you may merge with the main branch”

[x] P1. Change time on task (up to 1 hour)
[x] P1. Change text in "GitHub Secrets" alt-text at top of page from "bottom of the page" to "bottom of this page"
[ ] (P1. Consider adding instruction: "Before you start, contact your administrator to see if you have GitHub Secrets set up"?)
[x]  P2. Change first line ("You indicated that..." is not in line with what the users believe they have answered previously)
[x] P2. Change word "package" to "interview" to keep in line with internal terminology
[ ] P3; P5. Consider whether hyperlinks at top are appropriate
[ ] P3. Further consider whether output files section is helpful or distracting to userbase
[ ] P3. Clarify instructions:

[ ] "Complete the following instructions" header
[ ] Tell them to use their GitHub account ("Log into your GitHub account")
[ ] 1. "Read the instructions at this link under "Create Personal Access Token"
[ ] 2. "Create your Personal Access Token: Tap the checkbox for "workflow" permissions. Ensure that all "Repo" options are also selected."
[ ] 3. "Finish creating Personal Access Token"
[ ] 4. "Copy the token and paste below"

[ ] P3. Specify that they are looking for the URL for their interview's repo
[ ] P5. Instructions should be reorganized and made more specific

[ ] Review actions first, then follow other checks

[ ] P5. Second list may be moved elsewhere (user may close window before doing this optional task)
[ ] P5. Second list, item 7: "File" should be made more specific

This says 1/2, but we discussed both. Is this just the notes from the first user? Can we add the notes about the second user? Sorry I haven't gotten a chance to look through it more thoroughly. Hope I can do that soon.
Notes from an additional user. This user was setting org secrets:

[ ] Assumed the url could be the url of any server. We might want to break out the url into its own question with a question preceding it asking if they have a window where they are logged into the test user account
[ ] 'Restart' made them think the server would be restarted.

User 1

[ ] Pointed out AL in title is shortened, but explained below
[ ] Confused about why “organization” was in quotes
[ ] Cued by question about admin to feel like they needed to find an administrator
[ ] Unsure whether secrets were set up

[ ] Requires them to check with administrator?

[ ] Clicks every link

[ ] Check whether each link is necessary or hinders flow

[ ] Unsure initially where to get PAT, but finds page without issue

[ ] Flips back and forth between interview page with instructions and github documentation

[ ] Unsure what account they should use to get the PAT

[ ] Chose 7 day expiration

[ ] Unsure which URL to use for their package

[ ] Follows the patterning of provided URL
[ ] Could use more clear instructions on which URL they should use

[ ] Appreciates page after inputting PAT--verifies that they did previous step correctly
[ ] P5-expected pull request link to go to documentation, not repo
[ ] Confusion on red x appearing in action page (due to coding flow on our end, known issue)
[ ] Unsure whether to request review or merge branch (due to our unclear scenario)

[ ] They do not seem to have clear instructions on their end about which option they should do outside of our test

[ ] P5-Unsure whether “step 1” was a header or an instruction
[ ] Did not delete PAT (read the step but did not follow the instructions)
[ ] Did not know what YAML file looked like, but correctly identified without help
[ ] Believed the “restart” button meant to do another test setup
[ ] Confused by “at github secrets” at end of test.

User 2

[ ] Unsure whether they were an admin, confused by line “depends on your situation”
[ ] P2-did not like first sentence (says because of repeated word “repo” but user 1 identified same sentence was weird due to “you identified that...”)
[ ] Didn’t click alt-text “about github secrets”

[ ] Would have emailed admin to ask about secrets

[ ] Unsure how to get PAT at first.

[ ] Appears skeptical that the github documentation link is what they need at this step.

[ ] Unsure what account they should use to get the PAT
[ ] Github did not show their account picture when page was not full-screened (not something we can correct, but something to keep in mind!)
[ ] Choose 7 day expiration
[ ] Kept continuing on to the “use on command line”

[ ] Appeared to understand that it wasn’t relevant to them after spending a minute or two flipping back between interview, documentation, and create PAT page

[ ] Did not struggle to find the URL for their package
[ ] Did not properly set up PAT permissions

[ ] Was confused by the wording of the sentence (“select workflow. This will also select Repo.” Needed to read several times
[ ] Initially did not set up workflow permissions
[ ] Correctly interpreted error page, verbally explained why they got the error, but still struggled to translate that knowledge back to generating PAT
[ ] Used back button on browser to return

[ ] Appeared to have difficulty with terminology (merge/pull/review)
[ ] Did not delete either PAT generated
[ ] Instructions did not specify to go to docassemble, but knew because the line “playground” or by general understanding of the sentence
[ ] Unsure which YAML they were testing, but correctly replaced the blank spaces


EntityManager and Storage connection

Changes:

refactored add method to set
refactored the constraint of a component to only accept mutable fields
added the Assoc Array of StorageInfo[TypeInfo] searchable by the Component type to have access to the Storage through the EntityManager
added the following methods to manipulate entities and components:

get - fetches a component from an entity, returns a pointer to the component or null if it fails
set - associates an entity with a component, returns true if successful or false if it fails
remove - removes a component from an entity, returns true if successful or false if it fails
removeAll - removes all components from an entity, returns true if successful or false if it fails
gen - now an entity can be generated with components on the fly, returns the newly generated entity
getOrSet - set the component if the entity is no associated with one and/or returns it

A component can be associated to an entity the following ways:
@Component struct Foo {}
@Component struct Bar {}
auto em = new EntityManager!size_t();

// with set
em.set!Foo(em.gen());
em.set(Foo, Bar)(em.gen());
em.set(em.gen(), Foo());
em.set(em.gen(), Foo(), Bar());

// with gen
em.gen!Foo;
em.gen!(Foo, Bar);
em.gen(Foo());
em.gen(Foo(), Bar());

// with getOrSet
em.getOrSet!Foo(em.gen());
em.getOrSet!Foo(em.gen(), Foo());

// gen was also updated to work with EntityBuilder
em.entityBuilder
    .gen!(Foo)
    .gen!(Foo, Bar)
    .gen(Foo())
    .gen(Foo(), Bar());

Components can be fetched the following ways:
@Component struct Foo { int x, y; }
auto em = new EntityManager!size_t();

// with get
Foo* foo = em.get!Foo(em.gen!Foo);

// with getOrSet
Foo* foo = em.getOrSet!Foo(e);
Foo* foo = em.getOrSet(e, Foo(2, 3));

Components can be removed using:
@Component struct Foo {}
@Component struct Bar {}
auto em = new EntityManager!size_t();

em.remove!Foo(em.gen!Foo);
em.remove!Bar(em.gen!Foo); // does not remove any, returns false
em.remove!(Foo, Bar)(em.gen!Foo); // only removes Foo, returns true
em.remove!(Foo, Bar)(em.gen()); // does not remove any, returns true
em.removeAll(em.gen!(Foo, Bar)); // removes all, returns true

The only aspect I'm not realy sure about is the remove logic when it comes to removing more than one component at the same time.

The only aspect I'm not realy sure about is the remove logic when it comes to removing more than one component at the same time.

(...) if no Component is passed, like the current behavior, all components are removed?

I guess that shouldn't happen. It could lead to silent errors and not readable for outside people. removeAll is good enough to remove all entities. It's clear and not exaggeratedly long.

(...) if no Component is passed, like the current behavior, all components are removed?

I guess that shouldn't happen. It could lead to silent errors and not readable for outside people. removeAll is good enough to remove all entities. It's clear and not exaggeratedly long.

(...) if no Component is passed, like the current behavior, all components are removed?

I guess that shouldn't happen. It could lead to silent errors and not readable for outside people. removeAll is good enough to remove all entities. It's clear and not exaggeratedly long.

remove does not remove entities. It removes components associated to an entity. The tricky thing here is:

remove!(A)(e) - removes A if associated with e,  returns true if successful
remove!(A,B,...)(e) - removes only the associated components ignoring the non associated, only returns false if the entity is invalid meaning as long as the entity is valid it always returns true even if no component passed was removed

(...) if no Component is passed, like the current behavior, all components are removed?

I guess that shouldn't happen. It could lead to silent errors and not readable for outside people. removeAll is good enough to remove all entities. It's clear and not exaggeratedly long.

remove does not remove entities. It removes components associated to an entity. The tricky thing here is:

remove!(A)(e) - removes A if associated with e,  returns true if successful
remove!(A,B,...)(e) - removes only the associated components ignoring the non associated, only returns false if the entity is invalid meaning as long as the entity is valid it always returns true even if no component passed was removed


Is this partial order alignment procedure correct?

For example, to implement edit distance, set the parameters to:
_matScore = 0 ;  _misScore = -1 ; _insScore = -1 ; _delScore = _delStartScore = _delEndScore = -1 ;
Then, set the program's input to:

char s1[] = "ACGTGTACAGTTGAC" ;
char s2[] = "AGGTACACGTTAC" ;
char s3[] = "AGTGTCACGTTGAC" ;
char s4[] = "ACGTGTACATTGAC" ;
char test[] = "AGGAGTGCATTTACG";

Use s1-s4 to create a poa diagram：

poa.Init(s1, strlen(s1)) ;
printf( "%d\n", poa.Add(s2, strlen(s2))) ;
printf( "%d\n", poa.Add(s3, strlen(s3))) ;
printf( "%d\n", poa.Add(s4, strlen(s4))) ;
poa.VisualizePOA() ;

So far, I think everything is correct.
But when the test sequence is compared with the poa diagram, the score and path output by the program do not match, and I think the score is wrong.
As shown below：

the output is:

Obviously, the scores and comparison paths are not consistent
I would greatly appreciate your response.
Thank you very much.

Thank you for the testing. There are some bugs when initializing the score matrix. Your example should work now. Please let me know if you find any other issues.


action update should know which attribute to change

not just give it an Action object, so the backend can be optimized.

Would you please elaborate on that?


add job-dsl for the github-issue plugin

example
...
publishers {
  issueRepo('https://github.com/fsimmend/github-issues-plugin')
  issueTitle('job $JOB_NAME failed')
  issueText('some log output')
  issueReopen()
  issueAppend()
  issueLabel('teamX buildjobs')
}
...

This is already supported by the Automatically Generated DSL:
job(String name) {
  publishers {
    gitHubIssueNotifier {
      // Title to use for the GitHub issues created when builds start to fail.
      issueTitle(String value)
      // Body text to use for the GitHub issues created when builds start to fail.
      issueBody(String value)
      // If specified, this label will be applied to all tasks created through this plugin.
      issueLabel(String value)
      // Repo to use for the GitHub issues to create when builds start to fail.
      issueRepo(String value)
      // Check this to change the behavior when a job fails a second time and previously created issue exists, if checked, this issue get reopened instead of creating a new one.
      issueReopen(boolean value)
      // If checked, and a job is continuously failing, every additional failure adds a new comment.
      issueAppend(boolean value)
    }
  }
}


Fix typos in doc comments

Seems odd that CI is failing due to a comment change...

Any updates on this?