Unable to build a project with amplify 10.3.0

Before opening, please confirm:

[X] I have installed the latest version of the Amplify CLI (see above), and confirmed that the issue still persists.
[X] I have searched for duplicate or closed issues.
[X] I have read the guide for submitting bug reports.
[X] I have done my best to include a minimal, self-contained set of instructions for consistently reproducing the issue.
[X] I have removed any sensitive information from my code snippets and submission.

How did you install the Amplify CLI?
npm
If applicable, what version of Node.js are you using?
14.x
Amplify CLI Version
10.3.0
What operating system are you using?
macos
Did you make any manual changes to the cloud resources managed by Amplify? Please describe the changes made.
No manual changes made
Amplify Categories
Not applicable
Amplify Commands
Not applicable
Describe the bug
deploy an old project  with amplify console and amplify cli 10.3.0 will failed
more detail in codebuild log
deploy with version 10.2.x works fine.
run localy, "amplify push" with version 10.3.0  will work.
Reproduction steps
every thing works fine with a new project ...
GraphQL schema(s)

Project Identifier
ed57bb8a46476511162afa8497d78147
Log output

2022-10-18T21:35:58.642Z [INFO]: Backend environment dev found in Amplify Console app: myproject
2022-10-18T21:36:00.727Z [WARNING]: - Fetching updates to backend environment: dev from the cloud.
2022-10-18T21:36:02.485Z [WARNING]: - Building resource api/myprojectcontactapi549
2022-10-18T21:36:02.502Z [WARNING]: - Building resource api/AdminQueries
2022-10-18T21:36:02.516Z [WARNING]: - Building resource api/myprojectgraphql
2022-10-18T21:36:05.319Z [INFO]: ✅ GraphQL schema compiled successfully.
                                 Edit your schema at /codebuild/output/src490452204/src/myproject/amplify/backend/api/myprojectgraphql/schema.graphql or place .graphql files in a directory at /codebuild/output/src490452204/src/myproject/amplify/backend/api/myprojectgraphql/schema
2022-10-18T21:36:05.322Z [WARNING]: - Building resource api/myapi
....
2022-10-18T21:36:05.376Z [WARNING]: - Building resource auth/authressource
2022-10-18T21:36:05.391Z [WARNING]: - Building resource auth/userPoolGroups
2022-10-18T21:36:05.453Z [WARNING]: - Building resource storage/mystorage
2022-10-18T21:36:18.220Z [WARNING]: ✔ Successfully pulled backend environment dev from the cloud.
2022-10-18T21:36:18.433Z [WARNING]: ✖ There was an error initializing your environment.
2022-10-18T21:36:18.914Z [INFO]: 🛑 Failed to pull the backend.
2022-10-18T21:36:18.915Z [INFO]: File at path: '/codebuild/output/src490452204/src/myproject/amplify/backend/auth/myproject0e23e38c0e23e38c/build/parameters.json' does not exist
                                 Learn more at: https://docs.amplify.aws/cli/project/troubleshooting/
2022-10-18T21:36:18.915Z [WARNING]: - Creating Zip
2022-10-18T21:36:19.008Z [INFO]:
2022-10-18T21:36:19.008Z [INFO]: ✅ Report saved: /tmp/myproject/report-1666128978926.zip
2022-10-18T21:36:19.008Z [WARNING]: - Sending zip
2022-10-18T21:36:19.008Z [INFO]: File at path: '/codebuild/output/src490452204/src/myproject/amplify/backend/amplify-meta.json' does not exist
2022-10-18T21:36:19.010Z [WARNING]: ✖ Sending zip
2022-10-18T21:36:19.152Z [ERROR]: !!! Build failed
2022-10-18T21:36:19.152Z [ERROR]: !!! Non-Zero Exit Code detected

Additional information
No response

I'm having this issue as well with amplify version 10.3.0 when I clone my project onto my local machine and try to initialize the backend environment using amplify init. This also occurs when my project tries to build in the cloud whenever CD is triggered on commits to git. When I downgrade to version 10.2.0 it works on my local machine. I'm currently trying to find a way to downgrade the version of amplify the console is using online.

I confirm the same issue when building from GIT in CI. Downgrading to previously used 10.0.0 solved the issue.

I've the same error

amplify push worked just fine for me but when pushing to my Github repo it would trigger the amplify hosting to rebuild.
But it would fail with
File at path: '/codebuild/output/src031203168/src/auth/amplify/backend/auth/<resourceName1>/build/parameters.json' does not exist

I landed here and the aforementioned workaround worked for me on my main branch front-end build but only on the first try after adding the workaround to the building settings amplify.yml.
A subsequent deployment attempt resulted in a different error of
ENOENT: no such file or directory, lstat '/codebuild/output/src065924861/src/auth/amplify/backend/function/<resourceName2>/src'

Sure I could create another similar workaround here but it feels like a whack-a-mole solution for a backend with a lot of resources.
Project Identifier: df3110bc24ac6a0ad4ef8fc33826839f
Hoping for a solve soon. Happy to provide any additional information.

Just as a callout, this affected a project I am working on, is there anyway to avoid this in the future? @renebrandel

Do you have an ETA when a new amplify-cli will be released or git build is starting to work again?

10.3.1 was released earlier today. It does not solve the front-end build for our app.
Since this issue started it broke our amplify custom config we had to write to link the Cognito user migration to a lambda function – because for some reason that's not supported out of the box with amplify-cli. We then realized our amplify custom overwrote the default amplify auth triggered functions like PreSignUp and PostConfirmation that we had setup. So our amplify custom grew to fill in more blanks for lack of support in amplify-cli.
 And finding more and more settings in Cognito that need tweaking but aren't available in amplify-cli. So having to write more and more "hacky" code to fill holes this tool has – whether through amplify custom or overrides.ts.
I'm ranting into the void because it's been so frustrating using Amplify. Nothing but a headache and a battle from the start. Dropping our use of amplify-cli in favor of using Terraform - which I wish we had done from the start.
Good luck to any one that uses this in a major production app with paying customers.

Hi @zackspear can you share more details around your custom resource configuration? Specifically, does it interact with any amplify project files? Are you using a custom resource instead of the auth category or in addition to auth?


top navigation bar looks empty on (some?) error pages

Current behavior (Feb 23, 2022)
Summary
On certain error pages, the top navigation bar looks completely black and thus empty, but contains 1 (?) clickable area.
Further details
During several tests on Feb 23 that resulted in error pages, say, due to producing a URL in the browser address bar for which there is no page (cf. screenshot), the navigation bar seems missing/empty. However, at the position expected for the TIRA icon, there's still a clickable link (area) that leads to the frontpage (which is then displayed correctly).

--< screenshot starts here >--

--< screenshot ends here >--

This applies to pages that are missing as well as to pages that exist, but for which the user has no access rights.
Side remarks
FL vaguely believes that we have seen (and perhaps discussed) this behavior before, but after a bit of searching I could not find an existing issue - thus creating this one.
Expected behavior
The top navigation bar remains available on error pages.

This is how it looks on the official Discourses forum.
Notice how only the logo remains.
The logo not appearing for us on that error page is linked to #116 and will be resolved as soon as #116 is resolved.
Will close this for now.


Upgrade the Windows VM Agent

When the user follow Manual installation paragraph to manually install the VM agent with a Windows installer package,
the following description will confuse the end user in Upgrade the VM Agent paragraph:
The Azure VM Agent for Windows is automatically upgraded.
From the context understanding,
Does it means that the user need to upgrade the VM agent manually when installed VM agent manually?
If yes, It is better to clarify that this automatically upgraded process only applies to the VM initialized provision from Azure Market images. For VM Agent installed from Windows installed package, It need user to upgrade manually.
Thanks and Best Regards!
Edward Chen

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

ID: ab75f449-661d-bd1e-3ebd-61e2384a911a
Version Independent ID: e7c93a31-6076-26b8-a53d-d36df01ec6bc
Content: Azure Virtual Machine Agent Overview
Content Source: articles/virtual-machines/extensions/agent-windows.md
Service: virtual-machines-windows
GitHub Login: @mimckitt
Microsoft Alias: akjosh

@rockboyfor
Thanks for the feedback! I have assigned the issue to content author to check and update the document as appropriate.

@mimckitt
Can you please check and add your comments on this doc update request as applicable.

I have updated the doc to be more clear.
VM images deployed from the marketplace will automatically have the latest version of the agent. If you installed the agent manually or are creating custom images, you should update the agent manually as well.
Once the PR merges the changes will go live after a few hours.
#please-close


FileUtils: Add a recursive chmod_r and chown_r

Implements a recursive chown and chmod that applies to all subfiles and subdirectories at a given path.

I'm not sure if these dedicated methods make sense... it's just quite easy to get this done with the non-recursive variants.
With the new glob implementation from #5179 the overhead is really negligible.
Dir["path/**"] { |path| File.chmod(path, mode) }

@straight-shoota I'm thinking now if it stiil makes sense to have FileUtils in the stdlib - there are some non essentials methods like aliases to Dir and File, and others that can be done via the new glob implementation. But if I undestand correctly the implementation, we can't modify the root directory itself this way but the sub directories/files?

I realise there is already a precedence for using different method names for different modes in FileUtils, such as rm_r, but could we consider using either method overloads or default values which I feel would be a cleaner API.
So instead of:
def chmod_r(path, mode : Int)
  ...
end

We'd have:
def chmod(path, mode : Int, recurse : Bool = false)
  ...
end

I'd also prefer methods to be named for the conceptual action they take, rather than named after the historic way to take this action in an operating system's shell, and to use words rather than abbreviations or acronyms. So instead of rm, why not remove or delete? Instead of chmod, why not grant or something?
More generally, the use of _suffix method naming in Crystal to provide different ways of taking the same action does not seem idiomatic. Ruby may do this due to the lack of method overloading, however Crystal does not have this problem.
I feel the method name should equal the action taken, it should not equal the action taken + options applied to the action.

@j8r Yes, the root folder must be handled separtely, but I'd still prefer that over an additional method.
Maybe it would make sense to have something like Dir.walk_tree(path) which yields each item in that path to avoid globbing.

I agree with you @lachlan for the recurse Boolean, underscores are not elegant here. We can merge most of the methods of the FileUtils Object to the File one. mkdir_p, rm_r and cp_r can be also be changed this way.
In an other hand I disagree to replace all LibC references by new names, specially chmod which use an octal number like 0o750, or chown that use uids and gids. This are very specific arguments.
For mkdir(_p), which take only one argument, and optionally an other for the permission mode, why not:
Dir.create("dir/", recurse: true)
Dir.delete("dir/")

rm doesn't exist as is in the LibC, so replacing it by delete is good!
I very like the recurse - this translates an Unix argument to a Crystal one.

Please let's not discuss these advanced ideas in this PR, open a new issue for that.

Sorry, I open an issue

Issue opened at #5231

I close this PR. FileUtils, File and Dir will certainly be cleaned in the future and then a recursive approach will be implemented for most commands in the future.


Cannot install rsync on VM

Check whether it's possible to directy install rsync or not on circleci/ruby:2.6.0-node-browsers

Fixed by https://github.com/open-education-polito/website/commit/5eb5efc373d704662e47026d05ccba4a139ee6b0, closing


encoding issue

@jgm,
There's a few instances including pandocfilters itself have the following line
encoding = sys.getfilesystemencoding()

What is the reason behind this? As pandoc only supports UTF-8, shouldn't we assumes it is always UTF-8?
It is causing some problem in the test I'm setting up, e.g. see https://github.com/jgm/pandocfilters/runs/3015950651?check_suite_focus=true
It all relates to the encoding to byte and some calls accepts str only.

This probably started with plantuml.py, which influenced ditaa.py and blockdiag.py to write in a similar way.
c.f. 60ed8b9eedb937c837a6fda080411761737aa900.


Adding support for CloudStack

I am a Cloudstack contributor. I want to add support for Kargo in Cloudstack. Where do I get started?

Hi,
You can have a look at #587  , it's still in progress but it will give quite good hint where to start to cloudstack as a 'cloud-provider' to kargo.
Or you can describe here the extra steps that are required to deploy on cloudstack and we will point where to do the changes.

Thank you I'll have a look at this.

So I have looked at the various moving pieces and have an understanding of what needs to go where. Here is my summary. Please correct me if I am wrong.
First, depending on the cloud providor the kargo-cli creates a playbook specific to that cloud provider and runs it to get the initial vms (this would be the kargo aws --instances 3 for example) and creates the inventory file. once this is done, when we run kargo deploy --aws which runs the main deployment using the inventory file from earlier.
Is this accurate?
Now if I were to add my own provider (Cloudstack in my case) I'd have to add the logic to generate the initial playbook that would create my network and VMs in Cloudstack, allocate them public IPs and then generate the inventory file correct?

Hello @syed ,
Yes that's exactly how the current python version works.
VM's are provisionned using ansible's playbooks.
We're planning to use terraform instead after writting a first version of the golang version.

Thanks @Smana If I were to contribute now, should I still go for the python-cli or do I work on the golang cli?


x-correlator in CloudsEvent notification?

Problem description
We took the decision to add x-correlator in our API which is fine but I have a doubt for notification.
Do we have to add the x-correlator as well in the POST {$request.body#/webhook/notificationUrl}? This is the notification send to the listener.
In order to stay fully compliant with CloudsEvents (I did not find any mention x-correlator in CloudsEvents) I tend to think that we must not have this x-correlator in the notification response & endpoint (for CloudsEvent compliance and to get it easy for the developer)
Expected behavior

Add this specific comment in the guidelines
Alternative solution

Additional context

From Telefonica view, the indication of the support of x-correlator header in the POST notification sent to the listener is good to be reflected in the APIs. Indeed we consistently can incorporate as well in https://github.com/camaraproject/Commonalities/blob/main/artifacts/notification-as-cloud-event.yaml.
In CloudEvents spec there is no reference to x-correlator header (as well as other HTTP headers), because it is focused in the metadata model for the event not in the use of the underlying HTTP protocol.
The indication of the support of the x-correlator header provides alignment within the CAMARA APIs Architecture:

It aims for the same purpose of troubleshooting (observability and traceability process) in one existing communication scenario
With the same considerations for App Developers as in usual API Client -> API Server (Operator) traditional flow
And being used with the same considerations as per currently indicated within Commonalities: https://github.com/camaraproject/Commonalities/blob/main/documentation/API-design-guidelines.md#9-architecture-headers

Therefore, indicating it within the POST notification, we are not adding something additional but just reflecting the possible use of this header within this communication flow, with the purpose given to this header

Thanks @PedroDiez
Got your points - we did not have any drawback to have the x-correlator in the notification as well, this is not inconsistent with CloudEvents so better to keep it.
Works for me.
@shilpa-padgaonkar @rartych as you were a lot involved in subscriptions & cloudEvents discussions no blocker for you to have the x-correlator in the notification yaml?

Therefore, indicating it within the POST notification, we are not adding something additional but just reflecting the possible use of this header within this communication flow, with the purpose given to this header

@PedroDiez @bigludo7 I have here a slightly different view: If we are indicating the X-Correlator header within the POST notification, it gets mandatory for the API Consumer (the party receiving the POST notification) to implement the logic for the X-Correlator. It is not optional for them. The question is if we want to add this additional effort to the API consumer. Until now it was optional for the API consumer to use (the non-standard) X-Correlator header.
Also on the API Provider side: if we indicate that there could be an X-Correlator header in the POST Notification and the API Consumer has to implement the logic anyway, shouldn't it be then maybe mandatory for an API Provider to send an UUID to be consistent across API implementations?
BTW: X-Correllator seem to be the correct according to the Guideline, not x-correlator

@PedroDiez @bigludo7 @hdamker I'm not sure I fully understand the use case. Do we want to correlate notifications with their subscription (inject the X-Correllator provided by the subscriber when requesting a subscription) or correlate the notification with its response?
In the case of the CloudEvent, if it is indeed a question of associating all the events with the subscription, then it seems more judicious for the subscriber to manage it by the URL of the subscription.
What do you think?

Hi,
@hdamker Understood you point, and from our view the fact that an API Consumer indicates back the X-Correlator in the response when indicated in request by API Server is not a big effort, and the indication of it in the request is not mandatory for the API Server. At the end is the same commitment as in the other way, if an API Consumer indicates it in the request it is mandatory for the API Server to send back in the response.
So, even it has some impact in API Consumers, it is needed to be able to perform the correlation in these kind of flows and give value to the purpose of this header.
@patrice-conil  It refers to correlate the notification with its response

As per commented in Commonalities Meeting 25th March, for the notifications case, the indication fo the correlator by the API Consumer will be indicated as recommended behaviour, and indicated properly in issued PR


Consider reorganize the project structure for Heroku compatibility?

I'm trying to deploy it on Heroku (https://github.com/southwolf/midori-todo-example),
but Heroku won't automatically add nodejs build pack if no package.json in project root.
I have to run heroku buildpacks:add --index 1 heroku/nodejs.
Maybe it's better to reorganize the project structure, making it more friendly to users.

Sounds reasonable. I should make it easier to deploy on Heroku as an example project.


Selected drop-down list values are not vertically aligned

Steps to reproduce:

Go to the "Users" section for the administrator
Open the filtration unit
Select values for the “Status” and “Role” drop-down lists

Actual result: The selected value is pressed to the top of the value output field (screen)
Expected result: The selected values are aligned in the middle of the vertical
Req ID: REQF_adm_user-005, REQF_adm_asss-003

@elenaustyugova Cannot reproduce bug by provided scenario. Selected item displaying with correct align.
OS: Ubuntu
Browsers: Firefox 73.0.1, Chromium 78.0.3904.332, 80.0.3987.87
I suggest to check again bug and update reproduction steps with additional information or decline bug.

@iperevozchikov please check it in Safari. It was not mentioned in the description of the bug, I updated it. It's a minor bug. If it can not be fixed, we can decline it.


Custom Fields can only be strings in the UI, but all JSON types are allowed with ORM/API

Environment

Python version: 3.8
Nautobot version: 1.0.2

Steps to Reproduce

Create a Custom Field of type Text (example will assume it has been assigned to dcim > device and has a name of my_custom_field)
In UI route to Devices > Devices > Add Device
In my_custom_field that was created in step 1, provide a list value ["<IP_ADDRESS>"] (we will name this device my_device)
Save the new Device
Launch nbshell
Get Device object that was just created (device = Device.objects.get(name="my_device"))
Inspect the my_custom_field value device.cf["my_custom_field"] and observe that is a string "['<IP_ADDRESS>']" or isinstance(device.cf["my_custom_field"], str)
Update the my_custom_field to be a list with the ORM (device.cf["my_custom_field"] = ["<IP_ADDRESS>"])
Call validated save to update the changes in the database (device.validated_save())
Refetch the device (device = Device.objects.get(name="my_device"))
Inspect the my_custom_field value device.cf["my_custom_field"] and observe that is is a list ['<IP_ADDRESS>'] or isinstance(device.cf["my_custom_field"], list)

Expected Behavior
Based on the Custom Field Type being named Text, I would expect only string data to be stored in the custom field's value. So for this example, I would either expect validated_save() to fail validation, or type conversion to a string before storing it in the database.
Having said that, I would prefer the Type to be JSON and input from UI to be stored as "json"

Observed Behavior
The UI and the ORM have different constraints on what type of data can be stored in the custom field

Long story short this inconsistency is because the UI validation is being done in the forms where as API/ORM are falling back to "default" model field validation, therefore only type-checking valid JSON or not.
We'll need to move the strict validation out of the UI forms and closer to the model.

#897 covers adding a separate "JSON" type for custom fields. For "Text" fields assigning a non-string value should be an error.

Should be a simple matter of adding appropriate logic to CustomField.validate().


Running the pretrained Model

Hello,
I hope this issue is not too basic, but is there a posibility to run th the pretrained model right away, and if so how could i do that?
Thank you for your Help,
Best regards MB

Hi, the pre-trained models for CIFAR-10, CIFAR-100, and STL-10 have been uploaded to the ./save folder.

Hi, the pre-trained models for CIFAR-10, CIFAR-100, and STL-10 have been uploaded to the ./save folder.

Thank you for your quick response,
Sadly im not too well-versed on the informatical site.
I couldn't find any executable files in the save folder, so i guess i have to run cluster.py and at some point refer to said files in the code, could you please tell me where that would be or what i otherwise would have to do.
thnk you very much for you help

You could load the specific pre-trained model in cluster.py by modifying the configurations in ./config/config.yaml

First of all thank you very much for your help,
Now that i am able to run the model, i wanted to ask wheter there is an option to get an graphical output which could be used in a presentation? i.e. is there a way to give the code a sample of pictures (not necessary in the dataset but with the same motives/classes) and get the augmented pictures, the output-Matrix of ICH/CCH and possibly a graphical representation of the clustering (like the t-SNE pictures in your paper) as output?
(the more explicit you could answer the more helpful it would be for me)
i hope i dont annoy you too much with all my question and im very grateful for your help,
best regards
MB

To visualize the original and augmented images, you could use Image.show() function in the PIL package.
The output matrices of ICH/CCH could be visualized by the matplotlib.pyplot.matshow() function in the matplotlib package.
For the t-SNE plots, please refer to https://github.com/Yunfan-Li/Contrastive-Clustering/issues/14

Thank you for those hints,
i will try to implement them.
For that, could you please point out to me how those quantities are named in the code and where to find them?
Youre help is very appreciated, thank you very much,
best regards MB


grid layout scroll triggers directly to load more - bug

I have discovered the grid layout mechanism on the line mentioned here.
There is a missing in the Grid layout configuration and it leads to on load more function triggers when the scroll touch is triggered.
   mVisibleItemCount = layoutManager.getChildCount();
   mTotalItemCount = layoutManager.getItemCount();

if (!isLoadingMore && (mTotalItemCount - mVisibleItemCount) <= mFirstVisibleItem) {
          //todo: there are some bugs needs to be adjusted for admob adapter
          onLoadMoreListener.loadMore(mRecyclerView.getAdapter().getItemCount(), lastVisibleItemPosition);
             isLoadingMore = true;
             previousTotal = mTotalItemCount;
}

We need to fix this as it triggers constantly with unnecessary rate. Lets focus also focus on grid layout support as more people will be using this for tablet version of the app. This is also mentioned in #247, #241, #125

I think in my case to solve this issue is to implement a predefined lookupspan to control everything inside the gridlayoutmanager.

    final GridLayoutManager manager = new GridLayoutManager(this, 3);
            manager.setSpanSizeLookup(new GridLayoutManager.SpanSizeLookup() {
                @Override
                public int getSpanSize(int position) {
                    if(adapter.isFooter(position)){

if(!isLoadingMore){
      onLoadMoreListener.loadMore(mRecyclerView.getAdapter().getItemCount(), lastVisibleItemPosition);
      isLoadingMore = true;
}

                        return manager.getSpanCount();
                    }
                    return  1;
                }
            });

please now read #276

#285 done


[Logs] IA restructure + adding content types

@kodster28, I can't figure out why those two files aren't appearing in the TOC - the file structure on my machine is correct but they aren't getting pulled into the build. Any help appreciated!

@crwaters16 You "skipped a folder level" -- the get-started/logpush-configuration-api/Examples folder, currently with no files, must have at least an index.md file.
Also, please rename the Examples folder to examples.

@crwaters16 You "skipped a folder level" -- the get-started/logpush-configuration-api/Examples folder, currently with no files, must have at least an index.md file.
Also, please rename the Examples folder to examples.

Pedro I JUST had this realization on my morning walk and am very glad to see it echoed in your rec! Will do, to both. Thank you!


Update rtLog.c to fix compilation warning

Reason for change: To fix warning when ENABLE_RDKLOGGER is ON
Test Procedure: Tested and verified
Risks: Low

Closed due to inactivity, please restore if still needed.


Updates needed to get the system working natively on Apple M1 hardware

Coverage remained the same at 85.862% when pulling e910eba629efab6ee34c0d416911774da272191c on monterey-updates into 4ff5575b004daf8f2a4c407b813c6a83dff21aaf on master.

Really??  I definitely prefer your way.  Sometimes I just don't understand
rubocop!  I am so sorry to have belabored the (non)issue.
On Sat, Jun 25, 2022 at 2:50 PM Nathan Wilson @.***>
wrote:

@.**** commented on this pull request.
In app/models/rss_log.rb
https://github.com/MushroomObserver/mushroom-observer/pull/1055#discussion_r906713350
:

@@ -437,7 +437,7 @@ def decode_args(args)
end

def decode_time(str)

Time.parse(str).in_time_zone

Time.parse(str + "+0000").in_time_zone

I would have agreed with you, but I just checked and it doesn't seem to
care. Neither way generates a violation and it's exactly the same number of
characters. For some reason I don't fully understand I slight prefer the
way I wrote it, but like Rubocop, I don't have strong opinion. :-)
—
Reply to this email directly, view it on GitHub
https://github.com/MushroomObserver/mushroom-observer/pull/1055#discussion_r906713350,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAYTNNO3SFIQATSSR7Y3K5TVQ5IIPANCNFSM5Z2TEFOQ
.
You are receiving this because you commented.Message ID:
@.***
com>

Maybe Rubocop doesn't care about the str + var concatenation here because Time.parse already casts the var as a string, or sanitizes it some other way...?

I'm looking forward to getting this merged to make my 2013 iMac happy on 10.14 Mojave...
just kidding!
But glad you figured it out, Nathan.


Issue setting up env, cu113

I had trouble setting up my env following the instructions in the Usage section. Specifically, I was getting : ERROR: Could not find a version that satisfies the requirement torch==1.10.2+cu113
I solved this by installing the cu112 wheel as is shown in the DIaM requirements.txt.  Is there a reason why this was removed from the requirements? Happy to make a small PR to add it back in if you want 😄

@jdilger Thank you for letting me know. Accidentally deleted. I didn't notice it. It's been added.

No problem, happy to contribute!


Heap out-of-bounds read in strtoul (68345541)

Hello HTTP Parser team,
As part of our fuzzing efforts at Google, we have identified an issue affecting
HTTP Parser (tested with revision * master b11de0f5c65bcc1b906f85f4df58883b0c133e7b).
To reproduce, we are attaching a Dockerfile which compiles the project with
LLVM, taking advantage of the sanitizers that it offers. More information about
how to use the attached Dockerfile can be found here:
https://docs.docker.com/engine/reference/builder/
TL;DR instructions:

mkdir project
cp Dockerfile.HTTP Parser /path/to/project/Dockerfile
docker build --no-cache /path/to/project
docker run --cap-add=SYS_PTRACE -it image_id_from_docker_build

From another terminal, outside the container:
docker cp /path/to/attached/reproducer running_container_hostname:/fuzzing/reproducer
(reference: https://docs.docker.com/engine/reference/commandline/cp/)
And, back inside the container:
/fuzzing/repro.sh /fuzzing/reproducer
Alternatively, and depending on the bug, you could use gcc, valgrind or other
instrumentation tools to aid in the investigation. The sanitizer error that we
encountered is here:
INFO: Seed:<PHONE_NUMBER>
INFO: Loaded 2 modules (621 guards): [0x7f77f35724a0, 0x7f77f3572e50), [0x77ae38, 0x77ae3c),
/fuzzing/fuzzer: Running 1 inputs 500 time(s) each.
Running: /tmp/poc
ASAN:DEADLYSIGNAL
=================================================================
==11==ERROR: AddressSanitizer: SEGV on unknown address 0x608000010000 (pc 0x7f77f212d2f4 bp 0x000000000000 sp 0x7ffdcc9d3300 T0)
==11==The signal is caused by a READ memory access.
    #0 0x7f77f212d2f3  (/lib/x86_64-linux-gnu/libc.so.6+0x372f3)
    #1 0x7f77f336b5bd in http_parser_parse_url /fuzzing/http-parser/http_parser.c:2371:23
    #2 0x517ae5 in LLVMFuzzerTestOneInput /fuzzing/security-research-pocs/autofuzz/http_parser_parse_url_fuzzer.cc:5:3
    #3 0x522d3e in fuzzer::Fuzzer::ExecuteCallback(unsigned char const*, unsigned long) (/fuzzing/fuzzer+0x522d3e)
    #4 0x517e8e in fuzzer::RunOneTest(fuzzer::Fuzzer*, char const*, unsigned long) (/fuzzing/fuzzer+0x517e8e)
    #5 0x51c397 in fuzzer::FuzzerDriver(int*, char***, int (*)(unsigned char const*, unsigned long)) (/fuzzing/fuzzer+0x51c397)
    #6 0x517bab in main (/fuzzing/fuzzer+0x517bab)
    #7 0x7f77f21162b0 in __libc_start_main (/lib/x86_64-linux-gnu/libc.so.6+0x202b0)
    #8 0x41f4d9 in _start (/fuzzing/fuzzer+0x41f4d9)

AddressSanitizer can not provide additional info.
SUMMARY: AddressSanitizer: SEGV (/lib/x86_64-linux-gnu/libc.so.6+0x372f3)
==11==ABORTING

We will gladly work with you so you can successfully confirm and reproduce this
issue. Do let us know if you have any feedback surrounding the documentation.
Once you have reproduced the issue, we'd appreciate to learn your expected
timeline for an update to be released. With any fix, please attribute the report
to "Google Autofuzz project". A patch which may partially address this issue is included for your consideration.
We are also pleased to inform you that your project is eligible for inclusion to
the OSS-Fuzz project, which can provide additional continuous fuzzing, and
encourage you to investigate integration options.
Don't hesitate to let us know if you have any questions!
Google AutoFuzz Team
artifacts_68345541.zip
68345541.txt

https://github.com/nodejs/http-parser/blob/b11de0f5c65bcc1b906f85f4df58883b0c133e7b/http_parser.c?utf8=✓#L2371
I think this might be because the port could be the last part of the url string, which need not be terminated, because it is passed as a buffer of explicit length.

Suggested fix: https://github.com/nodejs/http-parser/pull/409


[FLINK-3631] CodeGenerator does not check type compatibility for equa…

Adds a check on the CodeGenerator.scala code which does a check on the left and right operands so that all the comparison expressions can work now. (=,!=,>,<)

Thanks for the PR @ramkrish86
Looks good. Just one minor thing. Can you add a one line comment to each test that explains why the program should fail such as the tests testNonWorkingSubstring1() and testNonWorkingSubstring2()? Thanks!

I have done the update @fhueske .Thanks for the review.

Perfect, thanks for the update. Good to merge.

Merging


container component doesn't use the legacy SlottedItem structure

id -> item
Slot -> slot
TODO: Determine whether this affects other places and whether this was in 1.20.5 or 1.21.

bump?

If you have the time I'd appreciate some info on what's stated in this issue description, then it'd be a lot easier to handle fixing this.

In my duplicate of this issue I have some screen shots of the errors. Not quite sure what you are asking for. It just seems that the it thinks its a error/problem when its how the commands are meant to be typed. Are you wanting the minecraft commands I used to get the errors?

TODO: Determine whether this affects other places and whether this was in 1.20.5 or 1.21.

Are there any other data components or NBT locations where this change in format occured?
Did this change occur in 1.20.5 or 1.21?

No idea I only noticed in my case.
I can only confirm 1.21. (the current version of minecraft)

Here is an example of the command I used give @p chest[container=[{slot:0,item:{id:"minecraft:stone",count:1}}]] 1


Add DeploymentHelper for selective compilation based on Mix.env()

Similar to the #if preprocessor in C.
Let me know if any of you have a better name for if_compilation
With the current setup, there's cruft in the production code (which dialyzer complains about) and (a bit more pettily) shows up as lack of code coverage.

Pull Request Test Coverage Report for Build 1131

1 of 1 (100.0%)  changed or added relevant line in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.2%) to 100.0%

Totals

Change from base Build 1128:
0.2%

Covered Lines:
492

Relevant Lines:
492

💛  - Coveralls

How bout if_env (this pr closed alr tho)


We need a unified version, and using master code is an unstable thing.

The latest parallel chain should use version 0.9.23.

we are currently using nimbus which is on 0.9.10 https://github.com/PureStake/nimbus
i was worried about version conflicts if we upgraded onto 0.9.23 what did you think?

i have merged https://github.com/Diora-Network/Diora/commit/c9ffd9ab158988fd85516c2eb742b431b4dc0d03

we are currently using nimbus which is on 0.9.10 https://github.com/PureStake/nimbus
i was worried about version conflicts if we upgraded onto 0.9.23 what did you think?

Now the latest version is 23, I think this is not a big problem.


Add option to force connection via USB cable

I always connect my iPad via USB cable, because, for me at least, the wifi version of Sidecar randomly disconnects at least once per hour (a lot of people report similar issues on various discussion forums).
When I run SidecarLauncher with the USB cable connected, the iPad still often connects via wifi. I'd love to have a way to force it to connect via USB.
(I wondered if I could solve this via System Settings, without the need to change SidecarLauncher. I've not been able to find a way to do this. Some internet discussion mentions that it's necessary to have your iPad's "Trust this computer" setting enabled; I have verified that I do.)

My motivation for SidecarLauncher is for when the iPad is away from the Mac. What's your use case here? If you're on USB, I assume you're already at your Mac, in which case just start SideCar as normal through Control Center?

I connect 5-10 times per day due to moving locations and (mainly) the frequent freezing/disconnection of Sidecar when it runs on WiFi.
Given this, I really value having a simple keyboard shortcut to press when I wake my Mac from sleep, rather than an annoying three mouse clicks.
(To get a sense of "really value": I'd pay $30 / year for this.)
(On the frequent disconnections: I've never found a way to force Sidecar to run via USB, short of switching on Airplane mode on the iPad. This is pretty annoying because then I have to switch it off again when I use the iPad normally.)

There's a get/setDataLink method I see associated with a Sidecar device's config. Looks promising?
But I'm sorry to say I probably won't look too much into this right now.

I found a way that might force connection via usb cable ,you can download "Apple Configurator" app ,in the setting you can set to run the sidecarlauncher shortcuts when iPad is connected to mac  .also you might need to add 2 or 3 seconds delay to make sure that sidecar is connected via usb.   Then when you connect your ipad to your mac via cable it will enable sidecar automatically

When I connect my iPad to Sidecar wirelessly and then connect a USB cable, the iPad explicitly says "Switching to USB".
@peterhartree, can you possibly confirm this same behavior on your end?
If so, then I'll be curious to know if you experience any problems. If you do, then it'll mean that either (1) the USB connection itself can be just as unreliable as Wi-Fi (cheap cables/hubs maybe?) or (2) SideCar can silently transition from USB to Wi-Fi. If either case is true, then having a force USB option won't help as that would only apply to the initiation of the SideCar session.
Then there's the question of what connection method is chosen when the Mac wakes from sleep and automatically reconnects to SideCar.

Thanks for looking into this! I'm travelling this weekend, I'll get back to
you next week.
Le sam. 1 juin 2024, 04:05, Ocasio-J @.***> a écrit :

When I connect my iPad to Sidecar wirelessly and then connect a USB cable,
the iPad explicitly says "Switching to USB".
@peterhartree https://github.com/peterhartree, can you possibly confirm
this same behavior on your end?
If so, then I'll be curious to know if you experience any problems. If you
do, then it'll mean that either (1) the USB connection itself can be just
as unreliable as Wi-Fi (cheap cables/hubs maybe?) or (2) SideCar can
silently transition from USB to Wi-Fi. If either case is true, then having
a force USB option won't help as that would only apply to the initiation
of the SideCar session.
Then there's the question of what connection method is chosen when the Mac
wakes from sleep and automatically reconnects to SideCar.
—
Reply to this email directly, view it on GitHub
https://github.com/Ocasio-J/SidecarLauncher/issues/4#issuecomment-2143216604,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAKY3WCG43S3AU4IUT4M2R3ZFET4TAVCNFSM6AAAAABEV56MMOVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDCNBTGIYTMNRQGQ
.
You are receiving this because you were mentioned.Message ID:
@.***>

Here's the behaviour I see:

USB cable connected => Activate SidecarLauncher => Sidecar via USB
USB cable disconnected => Activate SidecarLauncher => Sidecar via Wifi
USB cable disconnected => Activate SidecarLauncher => Sidecar via Wifi => Connect USB cable => Nothing happens (continues on Sidecar via Wifi)

It's interesting that you see different behaviour. My system details:

MacBook 14-inch 2023 (M2 Pro); Sonoma 14.2.1
iPad Pro (6th generation); iOS 17.4.1.

When I created the ticket, I reported the following:

I always connect my iPad via USB cable, because, for me at least, the wifi version of Sidecar randomly disconnects at least once per hour (a lot of people report similar issues on various discussion forums).
When I run SidecarLauncher with the USB cable connected, the iPad still often connects via wifi. I'd love to have a way to force it to connect via USB.

I'm happy to say that this roughly never happened to me in May.
I'm using a different USB cable and working on a different Wifi network. I imagine there's also been some Mac / iOS updates during this period.
So—perhaps this feature is no longer needed.
I'll report back if I start seeing the issue again.

Interesting that you don't see the "Switching to USB" indicator. I'm on a M2 Mac Mini running Sonoma 14.5 and an iPad Air 5th Generation running iPadOS 17.5.1.
You also mentioned:

Some internet discussion mentions that it's necessary to have your iPad's "Trust this computer" setting enabled; I have verified that I do.)

From what I understand, both devices need to trust each other, not just one way. Although I don't recall ever doing this.
Anyways, after some poking, I managed to successfully find an ugly way to force USB (or wireless). But the limitation is that the connection method is persistent. For example, once USB is forced, disconnecting the cable will not automatically transition to a wireless connection.

Oh cool! If, given what you now know, it's fairly easy to add a "force USB"
option to the script, then I'd vote in favour. Personally I never want to
connect via WiFi.
I believe my setup is that both devices trust each other.
Thanks for looking into this, and thanks again for making this script. I
run it via keyboard shortcut several times a day!
Le lun. 3 juin 2024, 18:11, Ocasio-J @.***> a écrit :

Interesting that you don't see the "Switching to USB" indicator. I'm on a
M2 Mac Mini running Sonoma 14.5 and an iPad Air 5th Generation running
iPadOS 17.5.1.
You also mentioned:
Some internet discussion mentions that it's necessary to have your iPad's
"Trust this computer" setting enabled; I have verified that I do.)
From what I understand, both devices need to trust each other, not just
one way. Although I don't recall ever doing this.
Anyways, after some poking, I managed to successfully find an ugly way to
force USB (or wireless). But the limitation is that the connection method
is persistent. For example, once USB is forced, disconnecting the cable
will not automatically transition to a wireless connection.
—
Reply to this email directly, view it on GitHub
https://github.com/Ocasio-J/SidecarLauncher/issues/4#issuecomment-2145608563,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAKY3WFYSOS4EH2IR6DOUMLZFSIULAVCNFSM6AAAAABEV56MMOVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDCNBVGYYDQNJWGM
.
You are receiving this because you were mentioned.Message ID:
@.***>


Tagging is cumbersome.

Currently each release must be done manually.
All dependencies must be fixed throughout the project. (Container, installed software etc.)
Possible solution: Bash Release script for fixing versions.
Note: Also the volume must be tagged, in order to allow concurrent jenkins instances.

Volume is specified as jenkins home and volume in docker container.

Will be implemented in another container, will be solved when I implement the gce-version with backup.


Inconsistently works 10.0.19042

This is a bit odd but sometimes I'm able to connect to <IP_ADDRESS> and sometimes I am not. I followed the steps in https://github.com/stascorp/rdpwrap/pull/1160#issue-480711748. Everything in the config application is green as expected.
When it does not work Remote Desktop either closes without warning or I get the following message:

I found a work around - stopping the Remote Desktop Services service and then restarting my PC. But I'd like to have it working consistently. I have tried uninstall/reinstalling, running autoupdate, messing with the settings but nothing seems to do the trick. Has anyone seen this / has any suggestions?

Same Problem here!

Try to disable RemoteFX in your Group Policy, worked for me.

Still you have this problem? RDPConf screenshot?


🔴 BUG: % not working properly

Describe the bug
Well, the % doesn't behave as the normal calculators have the % function behave,
** Idea what you can do to prevent**
you can have 2 types of % behaviors based on input after the % but that would make you probably re-write the whole calculator system and end to implement BOS-MAS in order of breaking down you're instructions and make a correct  numeric operation to execute
To Reproduce
Steps to reproduce the behavior:

Go to 'the web server'
Click on '8 , * , 1,0, %'
Scroll down to 'see the console of browser'
See error

Expected behavior
8 * 10% = 0.8
this is because 10% = 0.1 in numeric form
so basically this translates to 8 * 0.1
because it takes 8 * 10% and just parses it to javascript numeric operation it will return an error because it does not do the 10 % through something, like 10 % 1 = 0 because the 1 can fully go into the 10 so there are no leftovers
Screenshots

Desktop (please complete the following information):

OS: [Windows 10 Pro]
Browser [edge]
Version [idk]

Additional context
javascript/computer operations of % are always the leftover form dividing

I would like to help you with this but as I said you will probably have to write a BOS-MAS order for you're calculator to break down the whole calculation and that way it is also easier to implement #3

Hi @jiri132 👋,
Thank you for this Bug report.
If I'm honest, what is 'BOS-MAS'? I've never heard of it. 🙁
How can I do that?
I really appreciate your extraordinary help jiri132.
Thank you kindly 🙏
Best regards,
NH

Hey @N3v1 ,
Sorry I meant BOD-MAS
a small mistake of mine!
It is the way how we define our math laws like

()
power, sin,cos,tan,root

,/

+, -

And instead of doing eval()
you could also create an algorithm to sort out the ways how it should perform these actions in that way you can precompute 10% to 0.1 so that you can type: 8 * 10% it will get run through BOD-MAS way of in what order it would get looked at the new calculation of numeric values:  8 * 0.1 thus giving the result needed
This is one way to implement the usage of the start of you're understanding of BOD-MAS in our computing laws!
Kind regards,
JB

Hi @jiri132 👋,
Thank you for your explanation. I will try to fix the problem in the next few weeks.
If I needed help I would mention you if that's ok with you.
Thank you kindly 🙏
Best regards,
NH

Hi @jiri132 👋,
the error comes because you didn't type anything after the percent sign. I don't have the 404 error at the beginning.
I'm sorry if I misunderstand you but the percentage calculation works.

Example 10% 100 = 10

Example 10 % missing Number

In addition, the calculator respects the BODMAS and/or the PEDMAS rule:
Example:

Beispiel:
20 - 5 * 3 = 5
20 - 5 * 3 != 45

But, if you have any suggestions on how I can change the code, it would be helpful if you clone the repo, create a new branch, make the changes and send them to me via a pull request. Only if you have time and desire.
Thank you kindly 🙏
Best regards,
NH

Hi @jiri132 👋, the error comes because you didn't type anything after the percent sign. I don't have the 404 error at the beginning. I'm sorry if I misunderstand you but the percentage calculation works.
Example 10% 100 = 10
Example 10 % missing Number

Hey @N3v1
That is right I didn't put anything behind the %, in normal calculators you don't do the % numeric operation like in code to use it to see what the left overs are from dividing.
So normally if you do 100% it means that it that you had a it all for example in real life
income: 10 000
federal tax: 51%
the n the calculation is 10 000 * 51%
in this case the outcome is: 5 100
so how did we come with the 5 100? well it is not that hard to understand the % way of representing this is a way to make it simpler to look at the fraction, normally this would get broken down to (% value) / 100
so for example: (51%) / 100 -> 51/100 - > 0,51
So if you look back again at the calculation you can represent it also as 10 000 * 0.51 -> 5 100
it is thee same outcome but then with a fractional display.
the current use of % isn't wrong if you aim the calculator to work just like code, but I think you wanted  an calculator that is applicable to 'normal' calculators if I am not mistaken :D
I hope this cleared up any of the confusion that I may brought you about the the error / bug
Kind Greetings,
JB

Hi @jiri132 ,
I got it now. I'm trying to revamp the calculator somehow
Thank you kindly 🙏
Best regards,
NH

Hi @N3v1 ,
I have found a way to keep the code how it is and still introduce mathematical functions and manipulate the variables so that you don't have to write BOD-MAS yourself!
Just check my recent pull request and it has all information in it to hopefully let you understand it how to transform the calculation and append the correct values each time that we would need in normal (), * /, + - way of computing
Kind regards,
JB

Hello @jiri132, would you mind taking a look at issue (#19)?

I urgently need your response ASAP. @jiri132


FP calling convention, mixed int and fp args

Krste pointed out that the FP calling convention isn't explicit about what happens when you have both int and fp args.  It only says that FP regs are still used even if the int regs are exhausted.  But doesn't say what happens if the int regs haven't been exhausted yet.  Or what happens with int args after you start using FP regs for args.  In particular, if you have 8 int args and 8 fp args (which are smaller than int/fp regs), then you can pass all of them in registers no matter what order they appear in.
There is apparently an old RISC-V calling convention which can still be found on the web which says otherwise.

It does say " Values are passed in floating-point registers whenever possible, whether or not the integer registers have been exhausted." I think it could be more explicit, but by my reading is fairly unambiguous about how registers could be used.
It might be worth considering following the lead of the AArch64 calling convention and providing pseudocode that describes how register + stack slot assignment works.

Pseudocode would be a great improvement, as would some examples to illustrate the finer points.


Freezing when moving

When you move the picture, it freezes sometimes. It's unpleasant when you want to adjust perfectly the picture on the website.

Did the problem appeared recently (is it a regression)? What version of Firefox and PP are you using?
Honza

I installed PP for 2 weeks so I don't know.
Nightly 45.0a1 (2015-12-07) macosx


Dependencies on Windows

Hi, on Windows, win I click on the GPU icon, there's an error because it can't find the "pip" executable.  I do not have it installed. Maybe it should be part of the vscode-genv install?
2024-05-24 18:02:42.388 [error] Error: Command failed: pip install genv
'pip' is not recognized as an internal or external command,
operable program or batch file.
at ChildProcess.exithandler (node:child_process:423:12)
at ChildProcess.emit (node:events:517:28)
at maybeClose (node:internal/child_process:1098:16)
at ChildProcess._handle.onexit (node:internal/child_process:303:5)
at Process.callbackTrampoline (node:internal/async_hooks:130:17) genv.installation.install {"value":"run-ai.vscode-genv","_lower":"run-ai.vscode-genv"}

hi @joshmouch thanks for reaching out!
unfortuantely, Genv does not support Windows machines at the moment, and it is not planned for any time soon

Ok. Is there a way to either say that during the install or, even better,
do the windows install but then automatically target the wsl2 Linux install
like Ai Studio for VScode (previously windows ai studio) does?
On Mon, May 27, 2024, 4:00 AM Raz Rotenberg @.***>
wrote:

hi @joshmouch https://github.com/joshmouch thanks for reaching out!
unfortuantely, Genv does not support Windows machines at the moment, and
it is not planned for any time soon
—
Reply to this email directly, view it on GitHub
https://github.com/run-ai/vscode-genv/issues/10#issuecomment-2132879948,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAXURRENGAPACCM2E4O3PIDZELRZ3AVCNFSM6AAAAABIII5DI6VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDCMZSHA3TSOJUHA
.
You are receiving this because you were mentioned.Message ID:
@.***>


how to deal between and sql when we use ComplexKeysShardingAlgorithm in 4.0.0-RC1 version

ComplexKeysShardingAlgorithm is to process the sharding case in which multiple sharding keys are used; ComplexShardingStrategy needs to be used together. It has a relatively complex logic that requires developers to deal by themselves.
FYI: https://shardingsphere.apache.org/document/current/en/features/sharding/concept/sharding/

ComplexKeysShardingAlgorithm is to process the sharding case in which multiple sharding keys are used; ComplexShardingStrategy needs to be used together. It has a relatively complex logic that requires developers to deal by themselves.
FYI: https://shardingsphere.apache.org/document/current/en/features/sharding/concept/sharding/

I know how to use it .
I mean i use ComplexShardingStrategy with two colums , and one colums is datetime type.
when i execute this sql
select * from table_a where colum_a='' and colum_b between time1 and time2
will throw  exception like this :
org.apache.shardingsphere.core.strategy.route.value.BetweenRouteValue cannot be cast to org.apache.shardingsphere.core.strategy.route.value.ListRouteValue
I find other issue the same with me , and i known it is a bug in 4.0.0-RC1 version.
I just want to avoid it in my projection.
3Q for you answer.

I use equal replace between and .and i hope this bug can be fixed next version


fix: Eiav2 spec patch

EIAv2 ServiceRequest RoutingProtocol needed to update its use of discriminator
This PR resolves that issue and allows the SDK to be usable to create an EIA service

CI failed, and IMO the failure points towards limiting the generator config change to the eiav2 package for now: https://github.com/equinix/equinix-sdk-go/pull/59/files#r1618973886
To limit the scope of the config change, I believe you could pass in the useOneOfDiscriminatorLookup setting the same way we pass in packageVersion in the codegen task.

Great catch by CI. Glad that I didn't add something that would get it all twisted going forward. I've moved it from Global to Local, re-made locally, and it's built the same as it was before. CI is passing now 👍


Remove some content from the README in favour of the documentation

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint.

Could you resolve the conflicts and then we can merge :)

This is in mergable state :) thanks @pcuenca !


Vue-slick usage

Hi, i am ajusting my vuejs app to work with Nuxt and i am having problem with vue-slick caousel library. I need help how to use it with Nuxt.
The way i use is:
import Slick from 'vue-slick';
...
data() {
// slick options
}
...
components: {
'slider': Slick
}
Thanks

This is the error i am getting:
SyntaxError: Unexpected token <
at exports.runInThisContext (vm.js:53:16)
at Module._compile (module.js:404:25)
at Object.Module._extensions..js (module.js:432:10)
at Module.load (module.js:356:32)
at Function.Module._load (module.js:311:12)
at Module.require (module.js:366:17)
at require (module.js:385:17)
at r (/Users/dinokajtez/Websites/etr-client-ssr/node_modules/vue-server-renderer/build.js:6964:16)
at Object. (webpack:/external "vue-slick":1:0)
at webpack_require (webpack:/webpack/bootstrap 89eafc9a72efc28dbf24:25:0)
at Object.exports.modules.259 (7.server-bundle.js:521:68)
at webpack_require (webpack:/webpack/bootstrap 89eafc9a72efc28dbf24:25:0)
at Object.exports.modules.260 (components/slider-big.vue:3:0)
at webpack_require (webpack:/webpack/bootstrap 89eafc9a72efc28dbf24:25:0)
at Object.exports.modules.188 (7.server-bundle.js:402:85)
at webpack_require (webpack:/webpack/bootstrap 89eafc9a72efc28dbf24:25:0)

i think that you should show your code better

This is the code:
``

use
this.products.length < 4 ? true : false
instend of
this.products.length < 4

I tried what you suggested but i get the same error.

Looks like the component might have some trouble with server-side rendering.
Try this:
const Slick = process.BROWSER_BUILD ? require('vue-slick') : null

instead of:
import Slick from 'vue-slick'

Nice one, this worked. Thanks a lot.

@dinokajtez I'm still stuck with this issue, can you please re-explain the solution to make vue-slick work with nuxtjs. I was spending days to solve frontend libaries to work with nuxtjs like slick and scrollmagic.


NestedModule processing does not respect $PSDefaultParameterValues

If a module has bad verbs, and you specify:
$PSDefaultParameterValues."Import-module:DisableNameCheck"=$true
You can import that module and not receive any warnings.  HOWEVER
If that module has a nested module which has bad verbs, then it will complain about those verbs when you import the original module.
In other words, NestedModule processing does not respect $PSDefaultParameterValues.
This turns out to be a big headache as there are a number of 3rd party modules we are using with bad names and the warnings are screwing up the signal-to-noise ratio of our logs.

Isn't this part of the design of $PSDefaultParameterValues?
A default set in global scope is not supposed to affect a module - and nested modules are process in the scope of the module, not in global scope.
If the nested modules won't fix their names - isn't it the responsibility of the module depending on the nested modules to disable the name check?  This would require explicitly import the modules in a psm1 instead of using the NestedModules property in a psd1, but it seems like this would be consistent with the design for safely setting default parameter values.


Unable to start client on Ubuntu based operating systems.

Description
After running run4client.sh client immediately crashes after trying to render a new window.
Reproduction
Try to run the client on the newest master commit.
Screenshots

Additional context
this has started to happen after ive updated my my system and still occurs on a fresh linux mint install.

Fixed in 5080920
Removed giant CM app icon.


[PORT] healthcheck feature

Port this change from botbuilder-dotnet/master branch:
https://github.com/microsoft/botbuilder-dotnet/pull/3796

healthcheck feature fixes #3313
Changed projects

Microsoft.Bot.Builder
Microsoft.Bot.Schema
Microsoft.Bot.Builder.Tests

[changes required]

issue is tracked with https://github.com/microsoft/botbuilder-js/issues/1640


Camera mode

I apologize if this was mentioned somewhere but does this only work with Orthographic camera?
I use perspective camera for the parallax effect and I was wondering if this could be modified in some way to work with perspective camera, didn't know where else to reach so I figured I'd create an issue

I checked the value differences when I change from ortho to perspective in code, screenDelta was always 0 in ortho so I hardcoded the screndelta to Vector4(0f,0f,0f,0f) and it seems to be working in perspective camera for now, but I don't know if this breaks something else

I have the same issue as well. I use perspective camera and have some grass assets with a non-zero depth (z-axis) for a parallax effect, but adding the grass or foliage shaders usually makes the grass asset become fixed to a zero depth and removes the parallax effect. I played around with shader and found that creating variations of it with different velocity strengths and wind scales made some grasses stay at their depth, but it's pretty finicky.
Hopefully this issue can be resolved, otherwise I may have to find another shader. Awesome shader though!

@Paerux I was able to make a modification to the grass_shader shader graph so that the shader now works with a Perspective camera.

Essentially, I split the object's position into x, y, and z channels and only applied the shader vertex transformations to the x-channel, which prevents the shader from modifying the object's depth (z-axis). If you want the y-channel to be modified also, make sure to apply that channel from the Split to the Add node (I recommend pairing the x and y into a Vector2).
The old shader structure is pictured below:

Notice that it transforms all channels of the object's position from a Vector3 out.
If these instructions don't work for you, I can also attach my shader graph file.

Hello, I've got the same issue. Seems like is not working on perspective camera.
If I use perspective, all good:

... but my project uses orthographic, so this is what it does:

I tried @coderEFE, but result was the same. I don't need depth, as player won't interact with things on foreground/background.
Pinging @aarthificial, since a lot of users might use Orthographic cameras. Any idea on how to fix this would be appreciated. Thanks!


⚠ Bofrost.be has degraded performance

In b025bc2, Bofrost.be (https://www.bofrost.be) experienced degraded performance:

HTTP code: 200
Response time: 1532 ms

Resolved: Bofrost.be performance has improved in a4ec3ca.


 with no task items have the 'task-list' class

The filter should leave  with no task list items alone.

This issue was moved to deckar01/task_list#8


Do you need maintenance help?

Description
@nrbnlulu I see this library as an important part of strawberry-django (and in future an important part of any django app; since once you go graph you don't go back!).
But AFAICT it's just you maintaining an ever-increasing issue list, and there's a stack of PRs dating back almost a year.
Are you in need of help/sponsorship with this project?
If so perhaps we can jump on a call and talk about whether there's a way to help you out.

The current state of the project is that I passivelly maintain it. Meaning I'll accept PR's though I am not working on featutes or bug fixes unless I need in my app.

Hey @thclark, appreciate you offering to support this repo. I just wanted to check if you were still willing to push this forwards with @nrbnlulu?

push this forwards with @nrbnlulu?
wym?

Hey @thclark, appreciate you offering to support this repo. I just wanted to check if you were still willing to push this forwards with @nrbnlulu?

I had hoped to be added as a maintainer properly but the messages above suggested that wasn't wanted.
I don't spend hours/days making PRs for libraries unless I know I can get them merged - it's fine if there's an active project, lots of maintainers and a roadmap so you can check what you're doing is inline with the plan. But for smaller libraries like this I find that work gets wasted - PRs drag on forever or never get merged. So to contribute properly you need to have the agency to resolve existing PRs, come up with a roadmap, triage enquiries and generally drive things forward.
That's what I mean by maintenance help. But to be honest since my original offer i've been erring toward doing my own thing anyhow, sorry!


⚠ ClamAV has degraded performance

In e9dd343, ClamAV (https://spamassassin.apache.org/) experienced degraded performance:

HTTP code: 200
Response time: 240 ms

Resolved: ClamAV performance has improved in 151b653.


Revert "Modify batch IT to use count instead of hash"

Reverts apache/beam#26327

Accidentally merged a PR still in review


Made use of the xlsx_export_url and the csv_export_url instead of hardcoding the urls in the template, added test case

This PR solves issue #10101
Updated the submissions_index.html file and add test case for the same in test_views.py .

@suyash5053 be sure to look at the failing CI issues
#!/bin/bash -eo pipefail
pipenv run flake8

Creating a Pipfile for this project...
./wagtail/contrib/forms/tests/test_views.py:15:1: F401 'wagtail.contrib.forms.views.SubmissionsListView' imported but unused

Exited with code exit status 1

CircleCI received exit code 1

And the failing unit test
ERROR: test_export_urls_include_filters (wagtail.contrib.forms.tests.test_views.TestFormsSubmissionsList)
----------------------------------------------------------------------
Traceback (most recent call last):
  File "/home/runner/work/wagtail/wagtail/wagtail/contrib/forms/tests/test_views.py", line 373, in test_export_urls_include_filters
    expected_csv_url = reverse('wagtailforms:export_submissions', args=[self.form_page.pk]) + '?foo=baz'
  File "/opt/hostedtoolcache/Python/3.9.16/x64/lib/python3.9/site-packages/django/urls/base.py", line 88, in reverse
    return resolver._reverse_with_prefix(view, prefix, *args, **kwargs)
  File "/opt/hostedtoolcache/Python/3.9.16/x64/lib/python3.9/site-packages/django/urls/resolvers.py", line 802, in _reverse_with_prefix
    raise NoReverseMatch(msg)
django.urls.exceptions.NoReverseMatch: Reverse for 'export_submissions' not found. 'export_submissions' is not a valid view function or pattern name.

thanks - reach out if you need a hand.

@suyash5053 test case is failing because wagtailforms doesn't have the url patterns export_submissions .
checkout file for its url pattern declaration
https://github.com/wagtail/wagtail/blob/dc2afe89b94265ea1dd592ea0e64090b05de2cf8/wagtail/contrib/forms/urls.py#L11

@salty-ivy I appreciate your help and the time you gave to help me, but I have tried create the export url, still the issue persists. will share the output log once I recreate this thing again as I manipulated the code.


add a state for generating chunks

add transitions to generate and fetch chunks from retriever

Is RAG call required in Housing FSM?

@charmilahruthika we should never generate chunks in the FSM. It's an offline activity that uses the Indexer service.
FSM is for the bot runtime.


chore: Add capabilities module to next

:tada: This PR is included in version 0.10.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


fix(backend): use internal pptr channel as service - fe channel

added throttler for sending notification from service,
used raw Runtime.evaluate and Runtime.addBinding instead of exposeFunction,

Solution on top of raw protocol is faster then current exposeFunction
and at the same time we need to send all messages from service to frontend
using the same channel to simplify throtling logic.

Murge


⚠ PrivateHD has degraded performance

In be21388, PrivateHD ($PHD) experienced degraded performance:

HTTP code: 200
Response time: 2671 ms

Resolved: PrivateHD performance has improved in 5fdee20.


Problem with hiding columns in Column selector

I am not sure if hidden and hiddenByColumnsButton are working as expected.
I need to hide a column (lets say "name" in the below example taken from the live editor in docs) from the table and ALSO the column selector. I imagine I would do the following:
function HideColFrmTbleNotFrmColsBtn() {
  return (
    <MaterialTable
      icons={TABLE_ICONS}
      options={{
        // Allow user to hide/show
        // columns from Columns Button
        columnsButton: true
      }}
      data={[
        {
          name: 'Foo',
          surname: 'Bar'
        },
        {
          name: 'Baz',
          surname: 'Fee',
        },
      ]}
      columns={[
        {
          title: 'Name',
          field: 'name',
          hidden: true,
          hiddenByColumnsButton: true
        },
        {
          title: 'Surname',
          field: 'surname',
          hiddenByColumnsButton: false
        }
      ]}
    />
  )
}

But I still see the "name" field in the column selector:

Am I using this incorrectly?
I have read the instructions here and here, and believe I have followed them proprerly. They pretty clearly say:

which is what I have done above (isn't it?!).

Hi! The way the columns button is handled has changed. See here for more info. https://material-table-core.com/docs/breaking-changes#columns-button

@oze4 I had actually read the info on that page before:

I have read the instructions here and here, and believe I have followed them proprerly.

https://material-table-core.com/docs/breaking-changes#columns-button gives me two options:

I wish to use the first option, as attempted below. I am not sure if I understand this correct. I am editing the live editor on this page to figure this out: https://material-table-core.com/demos/columns/hide/
Objective: to hide the column "name" from appearing in the drop down column selector.
When I set all columns to hiddenByColumnsButton: false
I still get both fields in the column selector:

But more appropriately, based on info in https://material-table-core.com/demos/columns/hide/,

I set hiddenByColumnsButton: true for the "name" column and keep it as hiddenByColumnsButton: false for the "surname" column:

But I still get the same output as before.
Apologies if I haven't followed the instructions properly (the docs probably needs a bit more clarification if thats the case; I would be happy to make a PR once I understand it myself), but can you show me how to update the code on the live editor such as to hide the "name" column from appearing in the column selector?

@oze4 : Could we reopen this ticket, as it seems to be unresolved?

I am facing the same problem - hiddenByColumnsButton doesn't work for me.

As far as I have tried, things don't work as per the documentation.
What works for me currently is along these lines:

hidden is used for initially (default) hiding a particular column in the table, and which you wish to alllow the user to view (Add to table) if they so desire
hiddenByColumnsButton seems (as per documentation) to be intended to be used for disabling hiding/viewing a particular column by the user
my use case was to simply hide a particular column particularly from the user, but to have the data passed on as part of rowData. What I did was to simply not specify the column (attribute) I wanted permanently hidden in the table columns parameter. I do still get access to this attribute in detailPanel.

hiddenByColumnsButton had been working but must have been broken recently.  I'll try to find out when and why.

@oze4 Will you remove the redundant return, or shall I submit a PR?  It would be preferable for #144 to be merged so this fix doesn't pile onto that.

@appukuttan-shailesh apologies for that - somehow the changes I made were reverted (to code that didn't work). I have since updated the code and published v2.3.31, which should fix this for you.

Thanks for resolving this @peacechen , @oze4

Will this property have an attribute associated with location?
for the translation of the text
for example > EN > "Add or remove columns" in ES> "Mostrar u ocultar columnas"


MultiQC: faster path lookups

The work _work_path_to_rel_final_path does is wasteful, because it
does up to thousands of isfile and isdir calls for every path
tested.
This might make sense if the files to check are small, but not with very
large analyses.
This patch changes the behavior to take into account the fact that we're
dealing with a dictionary:

If the path matches the dictionary keys, fetch it immediately (O(1))
If the path is a file, then we have our path
If it is not in dictionary, it is part of a directory: select, among
the keys, only those that contain it
Of these, check if they are directories and break if a match is found

This should make the function significantly faster because it will
reduce the number of actual filesystem lookups to almost nothing.
@chapmanb  @roryk  - Please check if this still matches your expectations of what the code should do

Luca -- this looks great, thanks for the more comprehensive fix on this. Much appreciated.


test: add GET/PATCH professional profile

Have at it 😅. I didn't do so 'cause I didn't see the need for it.
but once you need it. You can easily change it. but be careful as some fields are intensionally badly configured to test the filter body functionality (i.e., updating doctor role to be an admin, as it is an illegal move)


Fix param name

Update README to match the code in 78dc09deecd0ec61c70e13c44eabcc448028a35c so the alert examples in the docs now work :smile:

Thanks a lot for your PR


Multi attachments

The base code was downloaded from dev branch a while ago, maybe that's why it's able to merge only when compared with dev branch,
I removed class DAAttachment and class DAAttachmentList in .py as they are no longer needed in our current approach (without playground).

Per our discussion on today's meeting, below is what the download screen will look like. Do you prefer the buttons to be in the same color as the bottom one or different?

I like how that version looks! I think the different colors are okay, since they're doing different things: the green button is updating the server, and the blue buttons are opening a zip file or a yaml file in a new tab.


Add explicit nobs calculator functions

Related to #243 and #245
We should give users a function to explicitly calculate nobs_total and nobs by band information, as would be especially needed in the "lean" mode case mentioned in #245

Will take this as a pre-cursor to #245

Closed by #251


Balancer Pool Fetching

Just setting the stage for our integration test and thought we might also want to initialize pools for the orderbook.
Not meant to be merged, but could eventually go into main.

No longer necessary as his had already been done.


False positive on NoUnused.CustomTypeConstructorArgs

False positive on NoUnused.CustomTypeConstructorArgs if a type is declared in one module and its constructors are pattern-matched in another module.
SSCCE (Short Self-Contained Correct Example)
I created a gist to illustrate the issue:
https://gist.github.com/ShrykeWindgrace/6ed0fd9335cfa9e200da68674bedfccb
elm-review in this project results in
- ELM-REVIEW ERROR -------------------------------------- src/Messages.elm:5:15

NoUnused.CustomTypeConstructorArgs: Argument is never extracted and therefore
never used.

4| type Msg
5|     = Content String
                 ^^^^^^
6|     | Search String

This argument is never used. You should either use it somewhere, or remove it at
the location I pointed at.

────────────────────────────────────────────────────────── src/Messages.elm:6:14

NoUnused.CustomTypeConstructorArgs: Argument is never extracted and therefore
never used.

5|     = Content String
6|     | Search String
                ^^^^^^

This argument is never used. You should either use it somewhere, or remove it at
the location I pointed at.

I found 2 errors in 1 file.

even though we explicitly pattern-match on constructors and use the deconstructed value in the update function in the Main module. The documentation for this rule explicitly mentions == and /= operators without deconstruction, but as far as I understand we are not in this case.
Expected behavior
No error reported.
Additional context
"elm-review": "2.3.2"
"jfmengels/elm-review-unused": "1.1.2"

Thanks for the report and the SSCCE, this was very helpful!
This should be fixed in v1.1.3. The problem was due to a bug in jfmengels/elm-review, so the patch here was fixing the issue there and bumping the dependency to that new version (v2.3.3)

Thanks a lot for this prompt fix!

My pleasure 🙂
When you have the time, I'd love confirmation that the issue has been fixed for you!

I have just checked - indeed, with
            "jfmengels/elm-review": "2.3.3",
            "jfmengels/elm-review-unused": "1.1.3",

in review/elm.json I no longer see any false positives.

Awesome! Thanks for letting me know :blush:


Data Structure Writeup

http://<IP_ADDRESS>:4200/brandonso//2024/05/16/Data-Structures-Writeup_IPYNB_2_.html

1.92/2
There are great description per image and I was able to understand it properly. The images are understandable and they meet every requirement that is listed on the page. Overall you did very well and gave a nice holistic perspective on each section.


gh-207. Integrate dotenv with webpack

close #207

could you commit .env.sample file with an example of env vars ?


What's the size of OpenImagesV4 with 9m images?

I tried to download OpenImagesV4 from the official website, but it shows that the OpenImagesV4 will costs about 18TB. Does this dataset really cost so much space?

It should be around ~1-1.5TB.

It should be around ~1-1.5TB.

Thank you very much!


Added RobustScaler for feature scaling

scale training set using RobustScaler
apply the same scaling transformation to test set

I'm going to incorporate this in the next major update (coming soon). I'll add you to the contributors list.


Literature review: sensors synchronization

Sensor synchronization is common in systems which depend on multiple sensors, and we should examine if it is something we want to implement

An additional synchronization task to consider is the clock sync between the hydrophone driver and the main board. Otherwise timestamped data from the hydrophones will give erroneous results when used on the xavier.

An additional synchronization task to consider is the clock sync between the hydrophone driver and the main board. Otherwise timestamped data from the hydrophones will give erroneous results when used on the xavier.

Closing since this was never really needed, but is noted for future reference


cookbooks uploaded to community with knife cookbook site from OSX cannot be used in kitchen.

I pushed the logstash 0.9.1 cookbook to community using the knife cookbook site command from an OSX box.  When I try to run kitchen from a wrapper cookbook that requires it, the wrapper cookbook fails to run because of pax headers.
-----> Starting Kitchen (v1.2.2.dev)
-----> Converging <default-ubuntu-1204>...
       Preparing files for transfer
       Resolving cookbook dependencies with Berkshelf 3.1.4...
       Removing non-cookbook files before transfer
       Transferring files to <default-ubuntu-1204>
[2014-07-17T13:46:43+00:00] INFO: Forking chef instance to converge...
[2014-07-17T13:46:43+00:00] WARN:
* * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * *
SSL validation of HTTPS requests is disabled. HTTPS connections are still
encrypted, but chef is not able to detect forged replies or man in the middle
attacks.

To fix this issue add an entry like this to your configuration file:

  # Verify all HTTPS connections (recommended)
  ssl_verify_mode :verify_peer

  # OR, Verify only connections to chef-server
  verify_api_cert true

       To check your SSL configuration, or troubleshoot errors, you can use the
       `knife ssl check` command like so:

         knife ssl check -c /tmp/kitchen/solo.rb

       * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * *

Starting Chef Client, version 11.12.8
[2014-07-17T13:46:43+00:00] INFO: *** Chef 11.12.8 ***
[2014-07-17T13:46:43+00:00] INFO: Chef-client pid: 1367
[2014-07-17T13:46:48+00:00] INFO: Setting the run_list to ["recipe[loltest::default]"] from CLI options
[2014-07-17T13:46:48+00:00] INFO: Run List is [recipe[loltest::default]]
[2014-07-17T13:46:48+00:00] INFO: Run List expands to [loltest::default]
[2014-07-17T13:46:48+00:00] INFO: Starting Chef Run for default-ubuntu-1204
[2014-07-17T13:46:48+00:00] INFO: Running start handlers
[2014-07-17T13:46:48+00:00] INFO: Start handlers complete.
Compiling Cookbooks...
[2014-07-17T13:46:48+00:00] INFO: WindowsPackage light-weight resource already initialized -- overriding!

================================================================================
Recipe Compile Error in /tmp/kitchen/cookbooks/logstash/providers/PaxHeader/config.rb
================================================================================

SyntaxError
-----------
/tmp/kitchen/cookbooks/logstash/providers/PaxHeader/config.rb:1: syntax error, unexpected tIDENTIFIER, expecting $end
18 gid=1604020879
      ^

I bumped the version to 0.9.2 and reuploaded from an Ubuntu box and then it works fine.   Given that chefdk supports OSX this needs to be fixed so we get the same experience uploading cookbooks from either OS.
Here's a dumb test wrapper cookbook that incudes the logstash cookbook that I used to test this out - https://github.com/paulczar/logstash-debug-repo   the master branch depends on the broken version,  the 092 branch depends on the newer fixed version.

Are there any workarounds for this at the moment? I'm running into this problem with a community dependency that I can't simply re-upload to the Supermarket.

One workaround that I've found is to remove any directories called "PaxHeader" after installing cookbooks.  If you're using Policyfile, then you would do something like:
chef install
find ~/.chefdk/cache/cookbooks -type d -name PaxHeader -exec rm -r {} +
chef update

I'm not sure what the equivalent would be with Berks, but it should be pretty similar to the above.
I've also submitted a PR, referenced above.
One long-term solution would be to remove dependencies on Minitar, which can't handle anything beyond what's specified by POSIX.

What's the status on this? There doesn't appear to have been an official update since 2014.

does putting PaxHeader in a chefignore file fix this?

Unfortunately that does not fix it for me.  (I'm using Policyfile. I haven't triggered the issue with Berks.)

I also ran into this problem with the zap cookbook from the community supermarket. I would greatly appreciate a fix other than going into the cache and removing the PaxHeader files.

needs to be replicated on recent chef and submitted as a new issue, and should reference as to if chefignore'ing PaxHeader works correctly or not.

This has been fixed as far as I can tell.  The issue was resolved in the cookbook-omnifetch gem version 0.4.0.  I'm on Chef DK 3.1.0, which includes cookbook-omnifetch 0.8.0.


[High Priority] Add compliance to website for privacy policy and cookies

[ ] Cookie policy page specifying what is tracked etc. Linked in the privacy policy created below, and maybe in the cookie banner too. Use this website to create a cookie policy
[ ] Cookie banner (if accepted then track cookies, if declined then do not. Maybe this sets a global variable or something). Especially turning off Google Analytics if cookies are rejected. Find a clean way to do this in JS rather than the build etc.
[ ] Privacy policy page specifying legal things like my use of Google Analytics etc. Linked in the footer of the website and takes a user to a separate page. Use this website to create the policy

[ ] Add the actual policies to the policy pages
[ ] if the variable is checked such that user does not want tracked, turn off tracking on the website (either removing the header thing that google analytics uses, or see if it allows you to stop tracking with some variable?)

Consider not having cookie policy decline by default, since if someone goes to another page it should have the cookie policy pop up again when they go to the main page. That way they can for example read the cookie policy and then go back and accept it


Tree breadth first

[] Top-level README “Table of Contents” is updated
[] README for this challenge is complete
[] Summary, Description, Approach & Efficiency, Solution
[] Picture of whiteboard
[] Link to code
[] Feature tasks for this challenge are completed
[]Unit tests written and passing
[] “Happy Path” - Expected outcome
[] Expected failure
[] Edge Case (if applicable/obvious)


Tiny typo fix in streaming.go

I noticed while using the library that it logged "Listenning" with 2 ns.

Hm, looks like the oauth library got updated to use some stdlib additions from Go 1.6+. I'll take a look and see if we can either pin those out, or if we'll have to drop our builds for 1.4 and 1.5. So the test failures here are unrelated to the pull request.
(In any case, thanks for the patch!)

@ChimeraCoder: LGTM. Tests should be fixed, too, since we dropped 1.4 and 1.5 support now.


[Feature]: Update logo.png to the halloween version

Feature Summary
Player SUBMARINO designed a delightful halloween version of the logo. Let's replace client/js/img/logo.png with this new logo for the month of October.
Detailed Description
Here is the file:

We can just do a straight asset swap for the time being and simply rename the new file logo.png and replace the existing one so that each instance of it in the site is replaced

Hi, assign this to me.

@Chitvan14 one thing I moved with your commit is that the artboard size between the two assets doesn’t match up so a straight swap won’t work without a new asset.
@itsalaidbacklife we’ll need a new one from SUBMARINO.

@seriouslysean what are the required dimensions?

Update: The current logo's dimensions are 1222 × 1063. Here is a revised version at nearly the same size. @Chitvan14 Can you work with this one?

@Chitvan14 if this is complete, can you open a PR?

Done, Please Check.


Adapt Flutter 3.0

when run on Flutter 3.0
log show this warning
- 'WidgetsBinding' is from 'package:flutter/src/widgets/binding.dart' ('../../Program/Flutter/packages/flutter/lib/src/widgets/binding.dart').       data: MediaQueryData.fromWindow(WidgetsBinding.instance!.window),

Same in my side

@shareef-dweikat @lyellss , Hello please use latest version, that was a breaking change from flutter itself.


beamerpresenter-mupdf: 0.2.2 -> 0.2.3

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://github.com/stiglers-eponym/BeamerPresenter/releases.
meta.description for beamerpresenter-mupdf is: "Modular multi screen pdf presentation viewer"
meta.homepage for beamerpresenter-mupdf is: "https://github.com/stiglers-eponym/BeamerPresenter"
meta.changelog for beamerpresenter-mupdf is: ""
Updates performed

Version update

To inspect upstream changes

Release on GitHub

Compare changes on GitHub

Impact

Checks done (click to expand)

built on NixOS
The tests defined in passthru.tests, if any, passed
found 0.2.3 with grep in /nix/store/gc7d9x5rqhqwjiky1m9r42c4n0r2faga-beamerpresenter-0.2.3
found 0.2.3 in filename of file in /nix/store/gc7d9x5rqhqwjiky1m9r42c4n0r2faga-beamerpresenter-0.2.3

Rebuild report (if merged into master) (click to expand)

3 total rebuild path(s)

3 package rebuild(s)

First fifty rebuilds by attrpath
beamerpresenter
beamerpresenter-mupdf
beamerpresenter-poppler

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/gc7d9x5rqhqwjiky1m9r42c4n0r2faga-beamerpresenter-0.2.3 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A beamerpresenter-mupdf https://github.com/r-ryantm/nixpkgs/archive/41d183db990f690877a2df9abfd6adbe01ad48c7.tar.gz

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/gc7d9x5rqhqwjiky1m9r42c4n0r2faga-beamerpresenter-0.2.3
ls -la /nix/store/gc7d9x5rqhqwjiky1m9r42c4n0r2faga-beamerpresenter-0.2.3/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review run on x86_64-linux 1

2 packages built:

beamerpresenter
beamerpresenter-poppler

Maintainer pings
cc @pacien @dotlambda for testing.

Duplicate of #189931.


Cannot rename conversations

Current behavior:
Clicking the "rename" button on a conversation in the sidebar opens a text field to edit the conversation's name, but nothing typed in this field is saved (nor is there a clear indication of how to save the name).
Expected Behavior:
Text typed in the field can be saved by pressing the enter key (or maybe also add a "save" button to replace the "rename" button when active?)

Seems to have been fixed, can't reproduce anymore.


fix(storage): bump storage version to v0.46.4

What kind of change does this PR introduce?
Version Update Storage 0.46.3

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

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

Files with Coverage Reduction
New Missed Lines
%

internal/gen/keys/keys.go
5
11.76%

Totals

Change from base Build<PHONE_NUMBER>:
-0.03%

Covered Lines:
6141

Relevant Lines:
10457

💛  - Coveralls


support Transfer encoding chunked in Response

With this change, it's now possible to stream data via transfer-encoding:chunked.

Looks good to me, did you run any tests on it?

I didn't include any tests in this PR, because.... I didn't see any tests in the repo.  Is there a test tree that I missed?
I am using the modified library, with chunked support, in some of my own projects, and it's working well.

Alright then, I was just asking whether or not you had tested it, as we currently don't have any unit tests.
Thanks for your contribution!