Default value does not save to Immutable structures

Type: Bug (I think it's a bug)
It's example of my problem.
Suppose I've received response for my items list, and in response I have empty array (no items). It's valid response (store in variable exampleDataFromResponse).
And I would like to insert data into my Immutable state:
const exampleDataFromResponse = [];
const state = Immutable.Map();
const updatedState = state.update('list', Immutable.List(), list => list.concat(exampleListFromResponse));
console.log('state.list:', updatedState.get('list'));

Then I make operations like above and I suppose to have my Immutable.Map state like:
{
    list: [],
}

Actual result:
{ }

Result will be same and for other operations: merge, mergeIn, mergeDeepIn, ...
But if you will change variable const exampleDataFromResponse = ['somedata'] all logic will work as expected.

More simple example:
const state = Immutable.Map();
updatedState = state.update('list', Immutable.List(), () => Immutable.List());

My created in callback List() does not put in my map;

While this can be confusing behavior, it's not a bug because it ensures the constraint that providing an identity function to update() results in no changes.
If your intention is to use a default value other than undefined in your update function, then I'd suggest using function argument defaults.
As an example:
const state = Immutable.Map();
updatedState = state.update('list', (list = Immutable.List()) => list.concat([]));


Save states

Ability to save different app states to Local Storage, and load them on demand.
This could reduce the need for various URL bookmarks for different app states or different factories. Instead, the zip state could simply be saved to local storage with a name and retrieved via settings panel.

Will consider if this is a requested feature, but this doesn't seem worthwhile considering state can already be saved via the url.


Failed to upload a file - FailedPrecondition: 400 Unauthorized: Missing authentication token

Description of the bug:
When I try to upload an image to Gemini, it gives me an error. I am using v0.7.2.
import google.generativeai as genai
genai.configure(api_key=GEMINI_API_KEY)

genai.upload_file("absolutepathto/image.png")

Actual vs expected behavior:
---------------------------------------------------------------------------
_InactiveRpcError                         Traceback (most recent call last)
File /opt/conda/lib/python3.10/site-packages/google/api_core/grpc_helpers.py:76, in _wrap_unary_errors.<locals>.error_remapped_callable(*args, **kwargs)
     75 try:
---> 76     return callable_(*args, **kwargs)
     77 except grpc.RpcError as exc:

File /opt/conda/lib/python3.10/site-packages/grpc/_channel.py:1181, in _UnaryUnaryMultiCallable.__call__(self, request, timeout, metadata, credentials, wait_for_ready, compression)
   1175 (
   1176     state,
   1177     call,
   1178 ) = self._blocking(
   1179     request, timeout, metadata, credentials, wait_for_ready, compression
   1180 )
-> 1181 return _end_unary_response_blocking(state, call, False, None)

File /opt/conda/lib/python3.10/site-packages/grpc/_channel.py:1006, in _end_unary_response_blocking(state, call, with_call, deadline)
   1005 else:
-> 1006     raise _InactiveRpcError(state)

_InactiveRpcError: <_InactiveRpcError of RPC that terminated with:
	status = StatusCode.FAILED_PRECONDITION
	details = "Unauthorized: Missing authentication token"
	debug_error_string = "UNKNOWN:Error received from peer ipv4:<IP_ADDRESS>:443 {created_time:"2024-09-09T13:27:28.392766886+00:00", grpc_status:9, grpc_message:"Unauthorized: Missing authentication token"}"
>

The above exception was the direct cause of the following exception:

FailedPrecondition                        Traceback (most recent call last)
Cell In[42], line 1
----> 1 genai.upload_file(image_path)

File /opt/conda/lib/python3.10/site-packages/google/generativeai/files.py:71, in upload_file(path, mime_type, name, display_name, resumable)
     68 if display_name is None:
     69     display_name = path.name
---> 71 response = client.create_file(
     72     path=path, mime_type=mime_type, name=name, display_name=display_name, resumable=resumable
     73 )
     74 return file_types.File(response)

File /opt/conda/lib/python3.10/site-packages/google/generativeai/client.py:97, in FileServiceClient.create_file(self, path, mime_type, name, display_name, resumable)
     94 request = self._discovery_api.media().upload(body={"file": file}, media_body=media)
     95 result = request.execute()
---> 97 return self.get_file({"name": result["file"]["name"]})

File /opt/conda/lib/python3.10/site-packages/google/ai/generativelanguage_v1beta/services/file_service/client.py:921, in FileServiceClient.get_file(self, request, name, retry, timeout, metadata)
    918 self._validate_universe_domain()
    920 # Send the request.
--> 921 response = rpc(
    922     request,
    923     retry=retry,
    924     timeout=timeout,
    925     metadata=metadata,
    926 )
    928 # Done; return the response.
    929 return response

File /opt/conda/lib/python3.10/site-packages/google/api_core/gapic_v1/method.py:131, in _GapicCallable.__call__(self, timeout, retry, compression, *args, **kwargs)
    128 if self._compression is not None:
    129     kwargs["compression"] = compression
--> 131 return wrapped_func(*args, **kwargs)

File /opt/conda/lib/python3.10/site-packages/google/api_core/grpc_helpers.py:78, in _wrap_unary_errors.<locals>.error_remapped_callable(*args, **kwargs)
     76     return callable_(*args, **kwargs)
     77 except grpc.RpcError as exc:
---> 78     raise exceptions.from_grpc_error(exc) from exc

FailedPrecondition: 400 Unauthorized: Missing authentication token

Any other information you'd like to share?
I've confirmed that the API key is correct by running another text-only prompt.

Have you tried uploading other files, such as PDFs or audio files?

I tried with a PDF file. Same error for PDF.

Are you sure your key is good? This Files notebook works:
https://colab.sandbox.google.com/github/google-gemini/cookbook/blob/main/quickstarts/File_API.ipynb
Is it possible it's picking up on some application-default credentials and getting confused?

@MarkDaoust  Thank you for checking. I've confirm that the key is good.
I am on Kaggle environment.

Oh, kaggle! I was just talking to them.
This might be the fix you need: https://github.com/google-gemini/generative-ai-python/pull/548
It's in v0.8.1.
Try updating to 0.8.1 and see if that works.
It if doesn't it may be a metter of waiting until Kaggle updates it to 0.8.1, shich should be soon, since they were asking me to fix this.

Thanks. I've updated the library to v0.8.1 and it works without error!


⚠ 366.ru has degraded performance

In 33a7075, 366.ru (https://366.ru) experienced degraded performance:

HTTP code: 200
Response time: 1965 ms

Resolved: 366.ru performance has improved in e9ef5b2.


chore(bridge-ui): static common services

TODO

TODO

Is this ready for review?

Closing this issue as it's getting picked up in this other one: https://github.com/taikoxyz/taiko-mono/pull/13497, where we have the changes from the latest PR, which contains a refactoring from this other PR: https://github.com/taikoxyz/taiko-mono/pull/13488


npm -i yargs doesn't work for me

When I run npm -i yargs, it shows Unknown command: "yargs". npm i yargs works for me.

fixed readme.md


Working on Beckn Onix Hackathon

Beckn Onix Hackathon
Date: 26,27,28 Spt 2024
Location:
Time:
Email Template for TSP ID link

Update 6 Spt
We are changing the Email Template based on Ravi's suggestion, and we are changing the name of the hackathon to Win Room from now on.

[ Need to send TSPs after new template is approved by Ravi ]
[ Connect NPs for Win Room ]
[Connect Dhiraj for asking lumpsum budget info]
[send email to Sujit for FIDE event budget discussion ]

Update

40 + TSPs email sent

We got 9 TSPs  RSVP for Win Room

We got a lump sum budget from Dhiraj Side 4 lac for an event for 120 -150

Next Task

Master Class PPT & Date confirmation  from @ravi-prakash-v
Kick-off call PPT

Help needed

Need Help in Marketing
Venue help  - I talked to IIM B, and they don't have a venue on our date, now I am talking with Google for a venue
We need to decide whether we need to push the final demo day date or not.
Need data to connect people from ONDC, ONEST, UEI, Belem Aberta, and Open Gambia etc. for invite


libnet80211: start an open_esplib library.

Would like to start add some functions for libraries other than libmain, and here's two I wrote recently.
Need help with how to actually get this to be linked. It seems to get built ok, just not used?

Solved the makefile problem, sorry for the noise. The *_DEFAULT_ROOT was not correct, but we have open_esplibs_ROOT so using that.

Unfortunately still got problems here, the new functions are not used in preference to the sdk functions. Have been able to get one of them to be used, buy adding a dummy function in the same object file, and referencing it from the application, but even this does not help with the other. Linker challenge I assume?

Added some hack references to demonstrate how I've seen these used. Perhaps someone with good knowledge of the linker can stop the problem? But any there is a hack workaround. Once a complete object file had open definitions I presume removing it would also address the problem, so the hack might only be temporary.

I have no idea why those functions are not linked instead of the Espressif's.
The build stage 2 that renames symbols in the Espressif's libraries are also weakens the symbols, so they should be replaced with no-weak versions.
SDK processing stage 2: Renaming symbols in SDK library build/sdklib/libnet80211_stage1.a -> build/sdklib/libnet80211.a
xtensa-lx106-elf-objcopy --redefine-syms ../../lib/allsymbols.rename --weaken build/sdklib/libnet80211_stage1.a build/sdklib/libnet80211.a

The linking workaround might get unnecessary eventually when enough of libnet80211 will be open source and it will be directly referenced from other source code.


nested result sets fail when executed in a transaction

Exception message: Invalid TDS stream: EOF or unknown token type returned

The problem is that when two or more nested result sets are used within an active transaction, the commented error occurs. If we do the same but without being in a transaction, the problem does not appear
Further technical details
SQL Server version: SQL Server 2016
Operating system: Windows Server 2019

@miguelsandro  thx for the issue. Can you please provide a code sample to reproduce the issue?

Yes, I'm executing one query with tx.Query then inside the Rows.Next() iteration block I'm making one o more calls to tx.Query.

I don't think that's a supported scenario until someone implements MARS on the driver. You'll need to fetch all the rows from the first query then iterate through the data to run the next query.

Thanks for the feedback. I will have to modify the code to adapt it to Go


Update aws-sdk-go to v1.2.5

This allows Empire itself to obtain IAM credentials from the ecs agent. CloudFormation doesn't yet support providing an IAM role when creating a task definition, so the EmpirePolicy is still assigned to the instance role in the example cloudformation stack.
Note that this PR doesn't add support for specifying an IAM role for apps to use.

👍


Fix minor typo in HSM Extension

Update 'a hierarchical topology groups' to 'hierarchical topology
groups'.
Signed-off-by: hasheddan<EMAIL_ADDRESS>

@atishp04 do you prefer GitHub PRs or sending patches to mailing list for this repo? Happy to send to mailing list if preferred :)


mpvScripts.evafast: Specify upstream branch in updateScript

Description of changes
Specified mpvScripts.evafast.updateScript's branch argument.
The upstream repo's HEAD points to branch master,
whereas the currently-packaged version comes from branch rewrite
@purrpurrn This will need updating once upstream merges rewrite into master.
Things done

[x] Ran nix-shell maintainers/scripts/update.nix --argstr package mpvScripts.evafast
confirmed the updateScript now behaves as desired
[x] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

Sorry for the missing branch name 😅. Otherwise this looks fine.


compute if present share the same code of traversing to leaf

Many operations implement traverse_to_leaf (with small variations). It's better to extract this logic into a function for better readibility.

It may slow down the fast read path


Usage reports page missing dropdown

How to get to the mailbox usage report
In the admin center, go to the Reports > Usage page.
From the Select a report drop-down, select Exchange > Mailbox usage.

The dropdown is missing in 365 admin center since several months ago.  I believe there was a UI update.  Please update documentation for viewing mailbox usage reports.

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

ID: e612a4e4-d051-27a7-e54a-bb1ac3fe0849
Version Independent ID: 653be32f-e4e8-8a72-98b3-a629238ecbd5
Content: Microsoft 365 Reports in the admin center - Mailbox usage - Microsoft 365 admin
Content Source: microsoft-365/admin/activity-reports/mailbox-usage.md
Service: o365-administration
GitHub Login: @KwekuA
Microsoft Alias: kwekua

@naps1saps Thank you for your feedback. We have made changes to the document via PR#https://github.com/MicrosoftDocs/microsoft-365-docs/pull/3440
Thanks
Sri

@msbemba Thank you very much for the contribution and updating the documentation with PR. @naps1saps Hope this update is helpful for you. Thanks for taking out some time to open the issue. Appreciate and encourage you to do the same in future also.


v0.0.5: Add option to not verify remote vault certificate

Here's a quick patch to allow this to work without verifying remote vault SSL certs.

LGTM

Actually, I think inverting the config variable clashes with vault-ruby and ENV["VAULT_SSL_VERIFY"]. I'm going to make this ssl_verify and you can set it to false instead.

And also take the opportunity to add the full suite of SSL options.


Condense all require_once statements into a foreach - for readability purposes

Hi @ArunSahadeo . Thanks for your pull request. To be honest I don't agree that this array and loop makes the code more readable, as it introduces a new layer of complexity without really removing a significant amount of code. Therefore, I will close this.


RadioButtonGroup fires select event multiple times when changing selected index

I have a radiobuttongroup with three buttons.
I want them all enabled and selectable.
But, depending on certain parameters when the user chooses a radio button I may programmatically decide that radio button should not be chosen.
In this case I would set the radio button back to the original using RadioButtonGroup.setSelectedIndex()
But, this fires the select event multiple times.
As an example see this jsbin
http://jsbin.com/gayami/edit?html,output
Selecting any radio button make the program set the selection back to index 0.
But doing this fires the select event 3 times as shown in the console.
The select event should fire one time - when the user selects the radio button.

Hi, thanks for pointing this out to us, we'll provide a fix asap.
Kind regards,
Christiane

tracked internally:<PHONE_NUMBER>

Hi,
normally when you don't want a user to select a radio button you should disable it.
However, your point is valid (the event should fire once). We will provide a fix with the next patches of the latest release versions (including 1.30 which you are using).

I was debugging this and was also trying to see if I could fix this. I looked at the code to understand it. This is what I observed.

This doesn't happen when using breakpoints.
I looked at the call stack and it seems to be coming from afterFocus.
Makes me think that it might be related to focus (afterFocus internally calls setSelectedIndex).
I will debug further to get some more clues and may be a fix.

Meanwhile, one can try if delaying the default index selection using setTimeout helps.

Fixed.
Thanks and Kind Regards,
Diana, SAPUI5 Dispatcher


chore(main): release 1.0.1

An automated release has been created for you.
1.0.1 (2024-02-07)
Miscellaneous

Sync files with stordco/common-config-elixir (#9) (4c9c41c)

This PR was generated with Release Please. See documentation.

:robot: Release is at https://github.com/stordco/lunar/releases/tag/v1.0.1 :sunflower:


Added holiday configuration for 2017 on ups provider

This PR adds a basic holiday configuration for the 2017 year on the ups provider for tests.
Required for #453 and further PR that will be done in 2017.

@benwah I've rebased and updated the dates according to the review.
Thanks !


[TypeError: Route.resource(...).middlewares is not a function]

How to use middlewares using route.resource ??

I have 3 routes :
`const Route = use('Route');
Route.get('/', 'HomeController.index');
Route.post('api/auth', 'AuthController.auth');
Route.resource('users', 'UsersController');`
and one middleware called a auth. I do not declared middleware on any route, but when I called any route the middleware is called. How I can resolve this?

Wrap it inside a group.
Route.group('web', function () {
   Route.resource('')
}).middlewares('')

that works, thanks!


Validation errors should call the callback

Input validation errors appear to be thrown rather than passed to the callback, which makes for some awkward code downstream. Right now, we need to do error checking in a try/catch and in the callback:
try {
    osrm.tile([x, y, z], function(err, result) {
        if (err) return next(err);
    });
} catch(err) {
    return next(err);
}

It would be nice to consolidate this into:
osrm.tile([x, y, z], function(err, result) {
    if (err) return next(err);
});

/cc @karenzshea

Yes. This problem exists for all plugins. What needs to happen is that: We add a Nan::TryCatch block for the parameter parsing and instead of just returning like here
we would need to call the callback with the error, like we do here.

Parameter validation throwing exceptions rather than calling the callback seems to be the norm across all native modules that I checked. To stay consistent here, we are going to leave this as is.


Add/Implement Bootstrap 4 & Font-Awesome

I appreciate you submitting this @Bartekus, but I'm not sure adding Bootstrap 4 + Font-Awesome to the core starter kit is the best way to go for ReactQL.
It adds a very opinionated design layer to what was intended to be a flexible starter kit that can feature any UI. Bootstrap's use of jQuery and Tether are also at odds with React's implicit control of the DOM; I'd prefer any UI examples to use native React.
I'm not opposed to the idea of adding some UI framework/examples, but Bootstrap adds a lot of weight and opinion to a relatively lightweight starter kit. I'd prefer native options like Grommet, Material-UI, etc.
I'll close this for now and re-visit UI later.

I've been eying Grommet for a while now, it's very concise and well structured so I think I'm going to give them a go and see how it goes. I'll open a PR once I'm done.

As a side not, I'm not a huge Material-UI fan, but I really do like React-Toolbox and highly recommend looking into it.


DataCite DOI registrations failing

Users with a custom DOI prefix from DataCite are currently getting the message "Pre-publication check failed: could not verify doi:10.xxxx/xxxx exists!" when they try to publish a resource which has a reserved DOI.
This seems to be due to a change in the DataCite API.
A workaround may be to log in to https://doi.datacite.org/ , find the DOI, and set the URL, then attempt to republish from the IPT.
I have only just reported the problem to DataCite, so I will wait to hear what they say before going any further.

DataCite have fixed the initial problem, although that has revealed a different change in their API.
I think the workaround is probably no longer necessary, but there will be problems deleting or changing DOIs.

This will be resolved with #1453


Fix a few things with Home Assistant example

Sorry about not splitting this up into multiple commits. I tinkered locally and once I got it all working I got it all moved back to upstream the fixes and changes. Included in this PR is:

Fix compile errors (global variable was not capitalized).
Add ability to disable WiFi/MQTT for calibration, useful if you want to calibrate before connecting to WiFi/MQTT.
Add auto-reboot on number of failures so sensor auto-reconnects after MQTT server downtime. I noticed when I applied updates and had to reboot my MQTT server I had to unplug/replug the kit. Auto-rebooting after a number of errors fixes this.
Update sensor.yaml example to play nicer with Home Assistant. Sorry about the block change, I think that was my git client updating line endings. If you diff with no whitespace you see that its updating to add device classes for all sensors which lets Home Assistant auto-pick the right icon and such, and rounded so that you don't get 10 digit decimals on your dashboards.
Adds a KWh intergation sensor to the sensor.yaml file. If you correctly categorized the total wattage input as "power" (which is done here) and then add an integration pointing to it (also done here), then it makes the "energy_spent" sensor available on the Energy dashboard. So I added that as an example to anyone else going the Home Assistant route so it works pretty much out of the box to get hourly/daily usage graphs.

Looks good! Thanks!


Raised Nan when training on 64643-size images

Hi ! Thx for sharing this repo!
I was using your model to train on 64 x 64 x 3 size images (celeba), it raised Nan at the very beginning. I think it might be the inv_conv's problem cuz I encounter such problem in my implementation of glow in tensorflow. Any suggestions on this issue?
Another thing is the model seems to be so big, it works with 1 batch size on one tesla v100 32G but fails with 4 batch size and more. I reduce the block num from 10 to 4 and it still runs out of memory...
Can you give me some suggestions on training 64x64 size images? Thx!!

Sorry about posting the Nan issue.. It was my mistake which cause this nan issue.


Added options for Arch Linux in the download options

Added the AUR URLs to the default download options under "Alt. Linux Build"
Before - http://hfimg.pw/i/g4py9.png
After - http://hfimg.pw/i/ohzrm.png

How is the LBRY app being added to that site?

It is the AUR (Arch User Repository).
Currently, there is only .tar.gz and .deb versions of the LBRY Application for Linux. Users of Arch Linux can use the AUR to clone and build the application with a simple (packagebuild/PKGBUILD) script.
The script installs dependencies and the application for them. Think of it like make and make install, but far more simple.

What do we have to do to upload to that site?

There are instructions on the Wiki page.
https://wiki.archlinux.org/index.php/Arch_User_Repository#Submitting_packages
Create an AUR account, add your SSH key to your profile, and use some Git commands to commit and push to the AUR repositories.

Do you know if that process can be automated?

No, not really as the AUR is user-maintained and the maintainers/author of each repository have to manually commit and push the updates.
I, along side others on the AUR would gladly keep the lbry-app-release repository up to date as new LBRY application versions are released.

Hi, I am the maintainer for the current AUR package and have been faithfully doing so for over a year. I received a merge request for your package, lbry-app-release, @kcseb .
@seanyesmunt yes it can technically be automated if I were to listen for releases on LBRY then run a script to update the package. I haven't had time to look into this. As of right now however, I manually update and test the packages locally.
I will happily add anyone from LBRY as a maintainer on the AUR for managing releases too.

@seanyesmunt I'd support adding this along with the flatpak support in a community maintained section of the readme: https://flathub.org/apps/details/io.lbry.lbry-app

Was planning on bringing that up today. Yeah, I think it would be a great idea to have another table below the current downloads table titled "Community-maintained releases" With a note describing it.

We just need to figure out how we want to support the autoupdate process for them. Right now, they would get prompted to download the deb file. Maybe we can disable this for flatpaks/aurs.
Someone also mentioned making a ppa for the deb, and the update process would also be a bit less painful.
The longer-term autoupdate process would include either appimage or snaps, as this is what electron builder currently supports.

I'll get a new community maintained section created for Flatpak/AUR builds. I'd ask @TimurKiyivinski / @kcseb to look after the AUR package and @iuyte for the flatpak (also asked @choffee as he just submitted one for 0.22.2 - https://github.com/flathub/io.lbry.lbry-app/pull/9)
I'll open a separate issue for dealing with auto update and maybe a community member can look into it.

@TimurKiyivinski if you'd like to assign me as a co-maintainer on the AUR, my handle is the same as here on GitHub (kcseb) Thank you.

@TimurKiyivinski
We've updated to

@kcseb I've added you as a co-maintainer, but please do not attempt hijacking packages any more.


Basic transfer visualization feature

Thanks for rebasing already @fno2010 :D I will get to reviewing this after tagging the 33.0.0-alpha release


Implement Sheet.Cell(string name)

The name argument should take a column name, such as A1.

It's been hard to figure out an elegant way of getting all Cell objects in all Row objects, using LINQ. Had to ask at StackOverflow:
http://stackoverflow.com/questions/28613462/linq-how-do-you-get-all-x-objects-in-all-y-objects

Seconds later, I got the answer:
IEnumerable cells = this.rows.SelectMany(r => r.Cells);


Why use RepeatDataset when train basicvsr++?

@ckkelvinchan  Hi, since training time is determined by training iteration, it seems it is no difference between using RepeatDataset or not.
Waiting for your response! Thanks.

@LuoXin-s It is just repeating the meta file, so that it will not reload the dataset, which slows down training. It does not affect the results

Closing due to inactivity, please reopen if there are any further problems.


Can 3 multi asset implementation

Add pallet_assets,
token bridge related pallets,
Staking pool pallet (No CI yet)

Thank you, apart from the comments, I have two general questions:

can we have a better name for bridge-transfer and bridge? I know it's from litentry, but I found it's easy to get mistaken - at least when I try to call extrinsics: they are so similarly named and the names don't convey enough information
can pallet-asset-handler be integrated into bridge pallet? It seems to be post-action of receiving token bridge ingress transfer

You are absolutely correct. But for this version we might have to deal with in case for the compatibility support of litentry. So I decide to not change anything about interface.
In the end, asset handler should totally replace bridge and bridge-transfer pallet, since you can easily see they get not too much storage useful.

pallet stable staking added. Still need CI tests though

Okay. Although Tests is not 100% percentage covered, I list the potential coverage list and comment.
I hope to make this PR merged and separate the Test impl in another PR.
As impl EVM precompile and front-end team testing is been blocking.


Status on drop is not loaded

Context
on turbulent drop, the minting status is not loaded

Task
Make sure that it is properly loaded

Also, first 3 sec of loading show this

Is it how it was supposed to be?
If so, would be better to show

for the drops value I changed it to<PHONE_NUMBER>0000000 in cf-d1
for unlimited icon should be fixed in #10849
fyi, in the basescan still show the number value: https://basescan.org/token/0x9789b13d485b124255f9dbf42aa7091aa22e87f8


Feature request: Pluggable commands

It would be really handy if we could extend the shell command parser out with support for additional, user-defined commands.  For my current use case, for example, it'd be great to be able to provide Deno-based drop-ins for things like gzip, tar and so on, but you probably don't want every possible command under the sun within dax itself...?
I know our vet has complained about my cats getting a little too fat, anyway...

I like that idea and I think it plays well with the CommandBuilder https://github.com/dsherret/dax#commandbuilder and custom $.

Implemented in #8.


Query regarding the distinction between scaffold and decorations

Thanks for sharing this insightful code with the community! It has been really helpful to understand how scaffold decoration works and how the pre-processing is done using the code.
When looking at the scaffold.py script, the following lines are used to distinguish between a scaffold and a decoration:
            # detect whether there is one fragment with as many attachment points as cuts (scaffold)
            # the rest are decorations
            if cuts == 1:
                sliced_mols.add(SlicedMol(fragments[0], [fragments[1]]))
                sliced_mols.add(SlicedMol(fragments[1], [fragments[0]]))
            else:
                scaffold = None
                decorations = []
                for frag in fragments:
                    num_att = len([atom for atom in frag.GetAtoms() if atom.GetSymbol() == ATTACHMENT_POINT_TOKEN])
                    if num_att == cuts and not scaffold:
                        scaffold = frag
                    else:
                        decorations.append(frag)
                if scaffold:
                    sliced_mols.add(SlicedMol(scaffold, decorations))

cuts is given a default value of 4 in the program. In the above code, num_att should be ideally equal to 4 to call a fragment as a scaffold. What would happen in cases where the num_att is lesser than 4 but greater than 1? The fragment should be still classified as a scaffold rather than a fragment right? I thought 4 is the maximum number of cuts allowed per molecule rather than the least number of attachment points required to define a scaffold. Can you elaborate on this?

The confusion was clear after looking at slice_db.py.


Password authentication allowed

Hi, the documentation states that password authentication is not used for login. This relies on the password for the 'core' user not being set.
I've deployed a new fleet and immediately received a warning from AWS support about password based authentication being enabled (regardless of possible).

Setting the password of the core user allows for password based authentication:

Could the default for /etc/ssh/sshd_config include:
PasswordAuthentication no

Tried to change it / make it a default via user-data, but /usr is ro.
Regards.

/etc/ssh/sshd_config is a symlink into /usr, but you can move a new file over the symlink. Using the write_files section of a cloud-config will work.

Closing due to inactivity.


Lambda config example should contain runtime

I am trying to run the chip-scale, it trowing the  error of Runtime
➜  chip-n-scale-queue-arranger git:(master) ✗ yarn deploy
yarn run v1.21.1
$ AWS_SDK_LOAD_CONFIG=true kes cf deploy --kes-folder config --kes-class config/kes.js
Already Uploaded: s3://test21/test21/lambdas/5af33326c1a6ce66052c3d3bc93b6ed96aa28029-DownloadAndPredict.zip
Template saved to config/cloudformation.yml
Uploaded: s3://test21/test21/cloudformation.yml
There was an error deploying the CF stack
[/Resources/DownloadAndPredictLambdaFunction/Type/Runtime] 'null' values are not allowed in templates
error Command failed with exit code 1.
info Visit https://yarnpkg.com/en/docs/cli/run for documentation about this command.
➜  chip-n-scale-queue-arranger git:(master) ✗

The Runtime value is empty in cloudformation.yml

@Rub21 you can add this to the lambda config; I'll update in the example config and thanks for flagging

Per chat with @Geoyi ,   we should add  runtime: python3.7, it works!!

Closed via #15


Switch UX to Cloudscape like solution

Switch UX to Cloudscape like solution. For example example table using cloudscape.
This will also help us separate the data from the HTML and styles.
Currently we're generating HTML and data and styles on the Python side. Instead on Python we'll only generate JSON, and we'll have a disjointed presentation system that knows how to read these jsons and load them into cloudscape components (like a table).
Requirements: Basic REACT knowledge is required.
More: Cloudscape. An open source design system for the cloud. Cloudscape offers user interface guidelines, front-end  components, design resources, and development tools for building intuitive, engaging, and inclusive user experiences at scale.
Current UX:
Suggested UX:

@gilinachum I can probably take it. Let's discuss


[FEATURE] Optimized SEO for redirecting URLs

What this feature is about?
It is common for all URL Shortening services to have best and customized SEO settings, so whenever user pastes shortened URL by our app in his posts or DMs, the receiver can overview where he will be redirected to.
Possible Solution
Since, we are restructuring our app into nextjs, the next app will handle most of the SEO settings itself. But we will use some external libraries to make customizable and platform specific SEO for our app.
Additional context
Here are some screenshots of tinyurl short links when pasted into various apps.
Twitter

Instagram DM

Please assign this issue to me!

Hey @sujeetgund, I am assigning this issue to but try to finish #15 first.
Please add short flow about it, what package you are using, etc.

Hey @sujeetgund, you can now work on this issue i have merged the changes and now it's live


max_bin documentation incorrect

@hcho3
I notice that our parameter documentation refers to the value 'max_bins' but the parameter itself within XGBoost is 'max_bin'.
I believe this has led to incorrect results in this blog article:
https://blogs.technet.microsoft.com/machinelearning/2017/07/25/lessons-learned-benchmarking-fast-machine-learning-algorithms/
I also notice that in all of our tree algorithms we initialise parameters allowing unknowns. If at all possible I think we should change this to prevent these kind of ambiguities in future.

@RAMitchell I've submitted a PR to fix the misspelling.
It would take a non-trivial amount of modification to disallow unknown parameters, as many different parameter structures (CLIParam, FastHistParam, GBTreeTrainParam and so forth) draw from the common pool of arguments. For instance, an argument that is "unknown" to CLIParam may be later accepted by GBTreeTrainParam. We'd need to keep track of which arguments have been picked up by parameter structures and which have not.
@tqchen Any idea about the unknown parameter issue? The cleanest way I can think of is to add a new interface InitConsume(&cfg) to DMLC parameter module. The new interface would physically remove arguments from the pool, so that we can tell which arguments in the list have been used or not. Just before training starts, we can check if cfg is empty or not.

@tqchen Actually, never mind. We could simply keep a (hidden) static database of used parameters. Let's try not to modify the DMLC module if possible.

It would also be nice to express more sophisticated relationships between parameters. E.g. if I design a tree_updater algorithm I could declare that it supports the subsample parameter but not monotonic constraints.

@RAMitchell I agree that it would be nice, although I suspect it would involve updating the DMLC parameter module. For now, I'd put it on a long-term wish list.
@tqchen I've submitted a provisional PR to implement detection of unused parameters. See #2571.


(doc) 'JIRA tracker' link fix in the README

Currently, the 'JIRA tracker' link points to http://tickets.puppetlabs.com/
which redirects itself to https://tickets.puppetlabs.com/ with a 301 redirection.
I just changed this to point directly to the correct https page.

Good catch! Thanks @ryayon!

Nice to hear !
Thanks


Some test cases failed in the rocketmq's store module.

BUG REPORT

Please describe the issue you observed:

I bulild the develop branch by the following mvn build commands, but failed as different unit test cases when every run!
(1). mvn clean install;
(2). mvn clean test -Pit-test;

Please tell us about your environment:

I run the test cases in mac, windows enviroments;
For example, I run the following mvn build commands;
(1). mvn clean install;
(2). mvn clean test -Pit-test;

Other information (e.g. detailed explanation, logs, related issues, suggestions how to fix, etc):

sub-task1-issue-number: sub-task1 description here,
sub-task2-issue-number: sub-task2 description here,
...

https://bugs.java.com/bugdatabase/view_bug.do?bug_id=4715154

First test case  problem
https://github.com/apache/rocketmq/issues/3102
https://github.com/apache/rocketmq/issues/2993
https://bugs.java.com/bugdatabase/view_bug.do?bug_id=4715154

Okay, maybe in next version, we can make the unit test cases in store module much more stability. @odbozhou

Good catch for quality. delivery speed improvement should not sacrifice quality degradation. I'll keep an eye on this. Break a leg~


Package wine-mono and wine-gecko into the container

Wine usually offers to download wine-mono and wine-gecko when creating a new wineprefix. However, as mentioned in the wiki https://wiki.winehq.org/Mono and https://wiki.winehq.org/Gecko, they can also be installed directly on the machine into /usr/share/wine/ so you don't need to download them for every new wineprefix. Could you add these packages into the container?

I thought about including them into the Conty's image, but unfortunately they don't compress well and take a lot of space. The image size with wine-gecko and wine-mono installed is ~250 MB larger than without them, so it's not worth it, in my opinion. It's better to let Wine download them on demand.
By the way, Wine doesn't download them for every new prefix, it downloads them once into XDG_CACHE_HOME (~/.cache/wine by default) and then uses the downloaded files for new prefixes.


Correct nullptr + offset scenario to pass ASAN check

Summary: ASAN does not allow nullptr + an offset to be present from its UndefinedBehavior category. We need to clean these places up so that ASAN can run without issues.
Differential Revision: D35919458

@majetideepak There seemed to be an s3 test timing out. What will be the easiest way to reproduce this?

@majetideepak Deepak, can you help us figure out the failure in ci/circleci: linux-parquet-s3-build  ?

@tanjialiang, @mbasmanova sorry, I missed your comments. I am taking a look now.


EHN BIDS filter

Add BIDS filter that follows the same design as fMRIPrep
also a lot of refactoring

Codecov Report

Merging #28 (633ab1e) into main (455a74c) will decrease coverage by 42.54%.
The diff coverage is 34.46%.

Additional details and impacted files

@@             Coverage Diff             @@
##             main      #28       +/-   ##
===========================================
- Coverage   89.58%   47.05%   -42.54%
===========================================
  Files           6        8        +2
  Lines         365      408       +43
===========================================
- Hits          327      192      -135
- Misses         38      216      +178

Impacted Files
Coverage Δ

giga_connectome/postprocess.py
19.31% <ø> (ø)

giga_connectome/run.py
95.00% <ø> (-5.00%)
:arrow_down:

giga_connectome/mask.py
68.62% <12.90%> (-16.97%)
:arrow_down:

giga_connectome/utils.py
41.77% <14.81%> (-58.23%)
:arrow_down:

giga_connectome/workflow.py
11.42% <16.66%> (-80.16%)
:arrow_down:

giga_connectome/denoise.py
31.81% <31.81%> (ø)

giga_connectome/atlas.py
66.03% <66.03%> (ø)

giga_connectome/__init__.py
77.77% <100.00%> (+27.77%)
:arrow_up:


Scraping of the first Primewire TV item after opening the app fails

Sometimes, all streams from the first selected Primewire item are incorrectly resolved to https://www.primewire.li#

Fixed by 9514898


Redirect to -- or provide a link to -- the login page when there is an auth problem

When a user is not logged in, the UI shows a generic permission error. Either the user should be redirected to the login page, or they should be provided a link.

Seems to be resolved


Empty results with /api/v1/cars/1/status

Hello,
I have an issue with the teslamateapi. I do not get valid responses from the /api/v1/cars/1/status, however, the response of /api/v1/cars/1 is fine. How can this be?
The logs also look fine.
Response /api/v1/cars/1/status
{
	"data": {
		"car": {
			"car_id": 1,
			"car_name": "MJ3"
		},
		"status": {
			"display_name": "",
			"state": "",
			"state_since": "0001-01-01T00:53:28+00:53",
			"odometer": 0,
			"car_status": {
				"healthy": false,
				"locked": false,
				"sentry_mode": false,
				"windows_open": false,
				"doors_open": false,
				"trunk_open": false,
				"frunk_open": false,
				"is_user_present": false
			},
			"car_details": {
				"model": "",
				"trim_badging": ""
			},
			"car_exterior": {
				"exterior_color": "",
				"spoiler_type": "",
				"wheel_type": ""
			},
			"car_geodata": {
				"geofence": "",
				"latitude": 0,
				"longitude": 0
			},
			"car_versions": {
				"version": "",
				"update_available": false,
				"update_version": ""
			},
			"driving_details": {
				"shift_state": "",
				"speed": 0,
				"heading": 0,
				"elevation": 0
			},
			"climate_details": {
				"is_climate_on": false,
				"inside_temp": 0,
				"outside_temp": 0,
				"is_preconditioning": false
			},
			"battery_details": {
				"est_battery_range": 0,
				"rated_battery_range": 0,
				"ideal_battery_range": 0,
				"battery_level": 0,
				"usable_battery_level": 0
			},
			"charging_details": {
				"plugged_in": false,
				"charge_energy_added": 0,
				"charge_limit_soc": 0,
				"charge_port_door_open": false,
				"charger_actual_current": 0,
				"charger_phases": 0,
				"charger_power": 0,
				"charger_voltage": 0,
				"scheduled_charging_start_time": "0001-01-01T00:53:28+00:53",
				"time_to_full_charge": 0
			}
		},
		"units": {
			"unit_of_length": "km",
			"unit_of_temperature": "C"
		}
	}
}

Response /api/v1/cars
{
	"data": {
		"cars": [{
			"car_id": 1,
			"name": "MJ3",
			"car_details": {
				"eid": 19437902XXXX,
				"vid": 18249217XXXX,
				"vin": 5YJ3E7EA2MFXXXXX,
				"model": "3",
				"trim_badging": null,
				"efficiency": 0.13
			},
			"car_exterior": {
				"exterior_color": "PearlWhite",
				"spoiler_type": "None",
				"wheel_type": "PinwheelRefresh18"
			},
			"car_settings": {
				"suspend_min": 12,
				"suspend_after_idle_min": 15,
				"req_not_unlocked": false,
				"free_supercharging": false,
				"use_streaming_api": true
			},
			"teslamate_details": {
				"inserted_at": "2021-03-13T12:22:57+01:00",
				"updated_at": "2021-03-15T09:54:24+01:00"
			},
			"teslamate_stats": {
				"total_charges": X,
				"total_drives": X,
				"total_updates": X
			}
		}]
	}
}

Logs
teslamateapi_1  | 2021/03/16 10:54:11.855339 [info] TeslaMateAPICarsV1 /api/v1/cars/1 executed successful.
teslamateapi_1  | [GIN] 2021/03/16 - 10:54:11 | 200 |    3.695194ms | <IP_ADDRESS> | GET      "/api/v1/cars/1"
teslamateapi_1  | 2021/03/16 10:54:14.813472 [info] TeslaMateAPICarsStatusV1 /api/v1/cars/1/status executed successful.
teslamateapi_1  | [GIN] 2021/03/16 - 10:54:14 | 200 |  775.427588ms | <IP_ADDRESS> | GET      "/api/v1/cars/1/status"

Thank you in advance!

Hi @kegelmeier,
Interesting! Looks to me that there is something related to MQTT then.
What type of setup of TeslaMate are you running.. Basic Docker or Advanced Traefik? Or something custom made?
Does MQTT container log say something similar to an attempt to connect?
Could you please share the configuration of your MQTT section in docker-compose.yml file?
It should look similar to something like this:

  mosquitto:
    image: eclipse-mosquitto:1.6
    restart: always
    ports:
      - <IP_ADDRESS>:1883:1883
    volumes:
      - mosquitto-conf:/mosquitto/config
      - mosquitto-data:/mosquitto/data

In addition, please set DEBUG_MODE environmental variable to true so that we can get more output from the log.
Regards,
Tobias

Hi @tobiasehlert
Thanks for the quick reply. I am using the Traefik setup.
Regarding the MQTT part, I also have a Home Assistant setup running on the network that serves as the broker.
Therefore the MQTT settings within the docker-compose.yml are custom as the following:
      - MQTT_HOST=<IP_ADDRESS>
      - MQTT_USERNAME=k***
      - MQTT_PASSWORD=k***

Could it be that the HA does not allow two connections from the same host?
Best,
Jens

Here additionally the output of the container logs in DEBUG_MODE
teslamateapi_1  | 2021/03/16 13:43:47.581394 [debug] initDBconnection - successfully completed (connected to postgres).
teslamateapi_1  | 2021/03/16 13:44:07.025670 TeslaMateAPICarsStatus successfully connected to mqtt.
teslamateapi_1  | 2021/03/16 13:44:07.135052 [debug] getTimeInTimeZone - UTC 0001-01-01T00:00:00Z time converted to Europe/Berlin is 0001-01-01T00:53:28+00:53
teslamateapi_1  | 2021/03/16 13:44:07.135207 [debug] getTimeInTimeZone - UTC 0001-01-01T00:00:00Z time converted to Europe/Berlin is 0001-01-01T00:53:28+00:53
teslamateapi_1  | 2021/03/16 13:44:07.135322 [debug] TeslaMateAPICarsStatusV1 /api/v1/cars/1/status returned data:
teslamateapi_1  | [GIN] 2021/03/16 - 13:44:07 | 200 |  783.093968ms | <IP_ADDRESS> | GET      "/api/v1/cars/1/status"
teslamateapi_1  | 2021/03/16 13:44:07.136668 [debug] {"data":{"car":{"car_id":1,"car_name":"MJ3"},"status":{"display_name":"","state":"","state_since":"0001-01-01T00:53:28+00:53","odometer":0,"car_status":{"healthy":false,"locked":false,"sentry_mode":false,"windows_open":false,"doors_open":false,"trunk_open":false,"frunk_open":false,"is_user_present":false},"car_details":{"model":"","trim_badging":""},"car_exterior":{"exterior_color":"","spoiler_type":"","wheel_type":""},"car_geodata":{"geofence":"","latitude":0,"longitude":0},"car_versions":{"version":"","update_available":false,"update_version":""},"driving_details":{"shift_state":"","speed":0,"heading":0,"elevation":0},"climate_details":{"is_climate_on":false,"inside_temp":0,"outside_temp":0,"is_preconditioning":false},"battery_details":{"est_battery_range":0,"rated_battery_range":0,"ideal_battery_range":0,"battery_level":0,"usable_battery_level":0},"charging_details":{"plugged_in":false,"charge_energy_added":0,"charge_limit_soc":0,"charge_port_door_open":false,"charger_actual_current":0,"charger_phases":0,"charger_power":0,"charger_voltage":0,"scheduled_charging_start_time":"0001-01-01T00:53:28+00:53","time_to_full_charge":0}},"units":{"unit_of_length":"km","unit_of_temperature":"C"}}}
teslamateapi_1  | 2021/03/16 13:44:07.136708 [info] TeslaMateAPICarsStatusV1 /api/v1/cars/1/status executed successful.

Hi @kegelmeier,
Alright, so your HA does provide the MQTT functionality in your setup?
That would mean that TeslaMateApi would need to connect to HA to get the information published by TeslaMate to HA.
Is this what you are using?
https://www.home-assistant.io/docs/mqtt/service/
When thinking of MQTT and multiple clients, that should be no problem. The question is more, why HA doesn't send back the last information published on a topic so that it can be consumed.. but it seems that you have not configured HA to retain the messages.
If you enable retain, the MQTT broker will stores the last retained message. Each client (e.g. TeslaMateApi) that subscribes to a topic pattern that matches the topic of the retained message receives the retained message immediately after they subscribe.
The broker stores only one retained message per topic and no history.
Regards,
Tobias

Hi @tobiasehlert,
I am running Mosquitto on Home Assistant with this add-on (https://github.com/home-assistant/addons/blob/master/mosquitto/DOCS.md)
I checked the set-up with an MQTT Explorer and find that the messages are already retained.

Any other idea?
Best,
Jens

I have a theory/suspicion...
https://github.com/tobiasehlert/teslamateapi/blob/9cbb2fa244610fbb01197052684059f6472a8384/src/v1_TeslaMateAPICarsStatus.go#L454
Mosquitto is not returning results within 100 milliseconds.

I am running Mosquitto on Home Assistant with this add-on (https://github.com/home-assistant/addons/blob/master/mosquitto/DOCS.md)
I checked the set-up with an MQTT Explorer and find that the messages are already retained.

Alright, then we know that it's retained already.

I have a theory/suspicion...
https://github.com/tobiasehlert/teslamateapi/blob/9cbb2fa244610fbb01197052684059f6472a8384/src/v1_TeslaMateAPICarsStatus.go#L454
Mosquitto is not returning results within 100 milliseconds.

Was thinking about the same thing @LelandSindt.. let's make it customizable to see if this can solve the problem for Jens.

@kegelmeier, created a separate branch (bug-14) and you can test the image bug-14 to see, if it works.
Set the MQTT_SLEEPTIME to 500 (default is 100) to see, if that works better for you.

Great, thanks @LelandSindt and @tobiasehlert! The image seems to work.
The first try did not work but every request after that worked out.

Awesome @kegelmeier
Then I'll go ahead and merge it with main.
Don't forget to switch image to latest to don't miss out on new stuff =D


ErrInvalidMessage is not only about CRC

Versions
Please specify real version numbers or git SHAs, not just "Latest" since that changes fairly regularly.
Sarama Version:  d65b3fd46b9f35c68df31b9884aa72dfefe31e04  ( latest )
Kafka Version: N/A
Go Version: N/A
Configuration
N/A
Logs
N/A
Problem Description
ErrInvalidMessage (Kafka error 2) returns "kafka server: Message contents does not match its CRC." as its error message. However, sending a message without a key to a log-compacted topic also results in error 2. As you can imagine, this was quite confusing.
Suggested replacement: "kafka server: Message is invalid."

The original message was taken from the wiki page which has now become https://kafka.apache.org/protocol#protocol_error_codes. That page now says the following about error 2:

This message has failed its CRC checksum, exceeds the valid size, or is otherwise corrupt.

I suppose a missing key still technically falls into the category of "otherwise corrupt", but the wording makes it sound like an internal error, not something the user has obvious control over. @edenhill should the broker really be returning error 2 in this case, or does the description of the error message need to be updated?

Encountered this issue also.
The problem was the payload size that was bigger than the kafka max size flag: message.max.bytes: 1000012 (around 1MB)
Ended up increasing the kafka flag and all works well.
Cheers


For FH users, allow for PROOF flow and non PROOF flow?

@vortexing @sitapriyamoorthi  For the version of this app that runs for FH users specifically, do you think we should allow for a non PROOF flow where users make their own Cromwell server and put in a URL and the PROOF flow? Or just the PROOF flow?
The answer to this impacts the app - only supporting one makes the code simpler. But supporting both gives it more flexibility but code is harder to maintain.
The most important question is what do think users want OR maybe what do we want users to do?
To accomplish #17 we for sure need to support a non PROOF flow, but that's of course not alongside a PROOF flow for non FH users.

Here are my two cents(and please feel free to correct any misconceptions I might have about the Cromwell functioning).

Do you think we should allow for a non-PROOF flow where users make their own Cromwell server

Here are a few use-cases that I think users could need this feature:

Users might have jobs that could support a specific Cromwell version that is different from our Cromwell version.
They might have some integrations with Cromwell, for eg: a cloud provider ?? which we do not support.

However, the larger question, is that do we want to support non-PROOF flow in our initial MVP.
@sckott : Do you think we can build this in such a way, that we can add non-PROOF flow support in future releases
Additionally we must think how must we prep PROOF now so that if we do decide in the future to support non-PROOF flow it does not impact traditional PROOF users.

Yes, we want the shiny app to interact with proof (the fred hutch implementation of Cromwell), but also anyone else shoudl be able to use the app to connect to a different cromwell deployment too.
Any cloud related integrations that are Cromwell specific have nothing to do with the app really.
If we can have a login tab that people could customize to their use case but the rest of the app is general, that would be great.

Thanks both for your feedback.
Got it on the importance of having both in the same app in the same user session.
For FH it makes sense to have both in the app, and then in instructions for others to use it we could strive to make it easy to hide/remove the login flow if they just need the I-already-have-a-cromwell-server flow


Default boxes

Continuing discussion from #126
We currently have the backend set up so that a user is not required to specify a box while uploading something. If the user doesn't specify anything, the upload goes into a new box, called "Untitled" followed by a timestamp. This isn't exactly the most intuitive behavior.
A few other options:

Require a user to explicitly specify a box when uploading
Have an undeleteable "Transport Box" that all new uploads go into if the box isn't otherwise specified
Have a "default" box for each user, which can be switched around at will. This seems like the best option so far, but there are a few decisions to make:

What if a user wants to delete the box that has "default" status? Should the "default" status automatically move to another box? If so, which one?
What happens if the user's default box is their last one, and they want to delete it?

The last option, but the user should always have at least one box. They shouldn't be able to delete their last box.

Last option, except don't let them delete the default box? They can transfer the "default" status to another box and then delete the box though.

Agreed with Raia, have a non-removable default box.

If the user deletes the default box and other boxes exist, then automatically switch their default to the next box.

How is the "next box" defined?

The first displayed non-default box in the user interface.
Let's say there are three boxes displayed in order A, B (default), and C. If the user deletes B, then A should become the default.

Update: I think we decided to drop the default box idea and just have the last box uploaded to be the one displayed first in the dropdown.

Alright, I'll re-open though, because we still need to remove the existing functionality to create a new unnamed box.
@not-an-aardvark So what we need is the call to fail if we don't send a box id.


CPU memory consumption is too large

Hello, sorry to disturb you again.I used your code to train for 3 epochs, and the cpu memory consumption reached 40G. The memory will continue to increase as the number of training increases. Eventually, the memory overflow will cause the program to terminate. I don't know why the memory consumption is so high, can you give me some suggestions? In your experiment, how much CPU memory does it take up to 150 epochs after training?

@Francisco-liu Sorry for the late reply. It may be because of this issue: https://github.com/microsoft/SpareNet/issues/12#issuecomment-888030036
The code is updated.


Switch to codecov

Closes #2181, #2182, #2183
Code Changes

Switch code climate to codecov
Changed README coverage badge to codecov

Steps to Confirm

[ ] The codecov comment shows no coverage because it uses main as the baseline for change in coverage, and because there has been no run on main yet there is nothing to compare to. The coverage for this branch can be viewed at https://app.codecov.io/github/ethyca/fides/tree/ps-codecov
[ ] Make sure references to code climate on the README.md have been replaced with codecov

Pre-Merge Checklist

[ ] All CI Pipelines Succeeded
Documentation:

[ ] documentation complete, PR opened in fidesdocs
[ ] documentation issue created in fidesdocs

[ ] Issue Requirements are Met
[ ] Relevant Follow-Up Issues Created
[ ] Update CHANGELOG.md

Description Of Changes
After #2342 and getting migrations excluded from coverage we are above the 85% coverage mark.

Thanks @SteveDMurphy for finding the extra code coverage spots I missed! 🔍


Update TargetFrameworks to net8.0 for Android and iOS (JailbreakDetector)

Updated the TargetFrameworks in the project file from net7.0-android;net7.0-ios to net8.0-android;net8.0-ios to leverage the latest features and improvements in .NET 8. This chang addresses compatibility issues with net8.0.

Hey, Thanks for updating the target frameworks. I will release a new version today / tomorrow of the packages.

Thank you!! if I can help with anything else just tell me.

Hi @banditoth! Do you know when the new version will be released? Thank you very much!

Hey, I've updated the TargetFW on other packages aswell, and I've encountered a problem with the DeviceId package compability. I will release the package you requested tomorrow. 😁

Still can't solve the DeviceId package problems, but I've released the other packages with net8.0 compatibility

Still can't solve the DeviceId package problems, but I've released the other packages with net8.0 compatibility

Thank you for your time. I have conducted some tests with the DeviceId project and have found the solutions
I will open a pull request in the other project.


Apply fixes from StyleCI

This pull request applies code style fixes from an analysis carried out by StyleCI.

For more information, click here.

is bast


feature(file-input): introducing file-input component. (closes #326) and (closes #286)

Description
Removed the file-selection part out of file-upload to create an atomic component thats able to be used stand-alone or part of file-upload.
https://github.com/Teradata/covalent/issues/326 and https://github.com/Teradata/covalent/issues/286
What's included?

td-file-input component for stand-alone usage with ngModel support.

  <md-icon>attach_file</md-icon>Choose a file...

BREAKING CHANGE feature(file-upload): td-file-upload usage of td-file-input internally and added way to set its labels so its easier to for i18n usage https://github.com/Teradata/covalent/issues/324:

    <md-icon>file_upload</md-icon>{{ singleFileUpload.files?.name }}
    <template td-file-input-label>
      <md-icon>attach_file</md-icon>Choose a file...
    </template>

docs(file-upload): updated README.md for td-file-upload and load it as part of the docs.
docs(file-upload): updated demo for td-file-upload with better examples.
feature(file-upload): add (select) event to td-file-upload component.
performance(file-upload): add OnPush change detection. https://github.com/Teradata/covalent/issues/325

Test Steps

[x] ng serve
[x] Go to http://localhost:4200/#/components/file-upload and see updated demo and docs with loaded README.md
[ ] Go to http://localhost:4200/#/components/file-input and see new demo and docs with loaded README.md

General Tests for Every PR

[x] ng serve --aot still works.
[x] npm run lint passes.
[x] npm test passes and code coverage is not lower.
[x] npm run build still works.

Screenshots or link to CodePen/Plunker/JSfiddle

text does not disappear after a file is selected

Thats an issue from material when using [value] which is fixed in their beta.2.


Why can't I +1 Christina?

Hello- I tried to "+1" a comment on a collection but an error occurred. Specifically, it took me to a new page that says "Error-- Sorry, an error occurred." This happens on multiple levels of the resource (the main resource page, inside the collection, etc.).
Thanks for looking into this!

@jbradberry This issue has the same cause as #1137 - primary key clash. After several erroneous trials, this error will go away. Assigning to you now so that we can verify the fix once you get a chance to look into it and get to the bottom of it. Feel free to reassign it to me as needed.

@hyi @mjstealey My prediction was that this was caused by destroying and then reconstructing the database during deployments.  I believe that we've fully switched over to using a persistent volume for the database at this point; have these errors gone away?

@jbradberry I think your prediction was very possible to be the cause of this issue. I am not sure whether we've fully switched over to using a persistent volume for the database at this point, I'd say it probably has not happened yet, but will defer to @mjstealey for the final words and details.

@hyi Should this be closed?

Since the issue https://github.com/hydroshare/hydroshare/issues/1137 has been closed and this issue has the same cause as that one, I am closing this as well.


[ruby] [v1.61.x] Build/test ruby 3.3 and build native gems with Ruby 3.3 support

Cherry-pick of #35399 for 1.61 since release branch appears to have been cut for pre-release builds.
It'd be good to land this for 1.61 since Ruby 3.3 has been out for a few weeks and this is one of the most popular native gems which is blocking some testing/upgrades by users who can't/don't want to build from source.
I read CONTRIBUTING.md but not entirely sure if this is needed, or whether the team have a habit of merging from master selectively prior to release. Let me know if this PR is unnecessary for consideration for 1.61 release.

Updates ruby-compiler-dock to 1.4.0 which brings Ruby 3.3 final support per https://github.com/rake-compiler/rake-compiler-dock/releases/tag/1.4.0 and starts cross-compiling for ruby 3.3.
As noted in #31991 and https://github.com/grpc/grpc/pull/35399#issuecomment-1886117733 the tests currently don't run on all ruby versions, so this is still an existing gap which this PR does not close, but also doesn't appear to make worse.
FYI @alto-ruby @apolcyn

Done in #35614


start scrubbing use case language

First pass at scrubbing the use cases to match the glossary... Probably best to view this in 'word diff' mode: https://github.com/kubernetes-incubator/service-catalog/pull/26/files?w=1

@jwforres - are we missing any personas here?

Re: User vs Developer, I think they are the same. I built the list by simply pulling "As an X" references. Happy to replace User with Developer or Application Operator (from the glossary PR #25).

LGTM

Ok, pushed persona updates. PTAL and I'll squash/merge.

LGTM


Add dynamic discovery support for consul catalog based targets

Copied from https://github.com/google/cloudprober/issues/316, originally filed by @manugarg.
Consul seems to be a popular option for service registry and discovery. Adding dynamic discovery support for consul based targets to cloudprober may be useful to users across cloud platforms.
Targets config for consul will probably look something like this:
probe {
 ...
 targets {
   rds_targets {
    resource_path: "consul://service/my-service" (or /nodes)
    filter {
      key: "name"
      value: "{{.regex}}"
    }
  }
  ...
}

Closing, as there has not been any real demand for this.


IN Condition Support

Hi,
I would like to add multiple values in filter condition with 'IN' operator similar to SQL. Is that supported?
What is the syntax for the same?
My req is to query clients 'A', 'B', 'C', 'D' etc....I can pass n number of values. With current filter support, it is allowed to provide multiple equal conditions joined with AND , OR operators. But this is not a ideal solution.
Looking for suggestions.

Do you mean something like this https://github.com/EntityGraphQL/EntityGraphQL/issues/383

Sorry, we'll need some more details. You can add the filter extension to fields and use isAny and you can read more here https://entitygraphql.github.io/docs/field-extensions/filtering


build: run tests on node.js v21 nightly

Hey everyone I tried resolving things here but encountered the upstream error reported here https://github.com/nodejs/node/issues/47614
Also, there seems to be a separate issue where the loader isn't engaged by the ava unit tests. I tried using this alternative syntax in the tests-ava package.json but the loader wasn't engaged...
{
  "ava": {
    "nodeArguments": [
      "--loader=esmock"
    ]
  }
}

Feel free to leave any input or advice for me. Thanks for pushing v20 tests @aladdin-add

the ava issue is reproduced and reported here https://github.com/avajs/ava/issues/3195

as the --loader is still experimental, I like to add a task to test it on node.js nightly build, and file an issue if it fails. WDYT?
will give it a try if no objections. 😄

looks like the node.js v20-nightly is still failing.

this branch is rebased on the latest master which handles async import.meta.resolve differently and specifies node-versions <=19.9.0

The nightly used in the tests appears to be an older nightly 20.0.0-nightly202304181c3d741cb0 as indicated by the timestamp in the name. We will want 20230514 seen here https://nodejs.org/download/nightly/

tests for all folders fail when run from the latest nightly... I won't have bandwidth to investigate further until end of week

if anyone here knows generally the way we should data should be communicated from test to loader, feel free to leave incomplete advice. If we there is a generally recommended approach, I might be able to add this this evening

let's try to implement this way, if there are no other suggestions https://github.com/nodejs/node/pull/44710#issuecomment-1368192020

I have a passing test here that uses worker thread to define the longer uri in the loader, but cannot use this machine to push up any changes so it might take some time before anything reaches this branch

I thought these changes were developed around node 21 but it looks like node 19 was used :/

When using node 21 nightly here, changes fail for ava's test folder, but they pass for the other test-runners.

ava does not work with node v20+ loaders at all https://github.com/iambumblehead/nodejs-v20-loader-ava-noop

title says enable node v20 but currently only nightly-v21 is used. That should be reflected in the title or in the code

@flovogt okay, which do you recommend?

@flovogt the title is updated

I think it's better to merge it ASAP and publish new package version. We can improve loader-userspace logic interaction later.

a new PR adding node v20 tests to the ci pipeline is created https://github.com/iambumblehead/esmock/pull/202


Colours do not come out as shown in tutorial

Using a an ANSI colour compatible terminal with VT100, I get some colour, but nothing like that shown on the tutorial pages

This is almost certainly your terminal enforcing a theme of some sort. I had the same issue with minicom inside of kitty, because I have a specific theme chosen for kitty. The only serial terminal I've tried so far that looked correct was Tabby, though I may try some other basic ones like xterm or rxvt etc.

Hello, is this issue still relevant?
I remember patching this issue by adding the option to change the color to ANSI-256

Trying to test the latest firmware to see if it's fixed, but the build fails. Going to open an issue.

I switched to using a different terminal, which works fine.


Merge back master to develop

Coverage remained the same at 91.299% when pulling f449d7ed0f2e5fb653a34eeb56c6072124237dd6 on master into 714ebc06537d3f0539244473cbf7fbfa89d0ce4b on develop.


Is it possible that some %ROUTINES are OS sensitive?

This might sound weird but I believe there is something affecting ^%G when running over Linux.
All Windows machines I've tried were just fine.
But when using Linux it hangs when trying to get out of ^%G.
I can access it, search for globals but when I type "blank return" to exit it hangs and I lose the terminal.
Points I notice:

It gives "WebSocket Read Operation timed-out" right after I hit enter.
The process is ending (I believe before it should) right after I hit enter.
It is getting stuck here: https://gist.github.com/masterbeer/f7c6be106c86befe5dd57377ec866548, with could be because the process does not exist and is not throwing error.

As I'm still making sense of the code I can not be more detailed on the possible cause. But I hope you understand.

Hello @masterbeer! Thanks for reporting. The issue is confirmed to exists.

Ok, thx!!

OK, I have found a reason causing this. This command being executed in WebTerminal under Windows does nothing:
close 0

However, on Unix machines it terminates child process...

New version is now live, please confirm that utility is fixed and everything works fine. Thanks for reporting!

You nailed it! Working fine. Thank you.


OpenAI model optional

OpenAI model optional and Update README.md

I also added a feature that allows for the deletion of inactive models when starting Streamlit.
But haven't added it to this PR yet.
If I only provide the OpenAI API key.

How do you think the results are?
from agent.research_assistant import models as enabled_models

models = {
    "OpenAI GPT-4o-mini (streaming)": "gpt-4o-mini",
    "Gemini 1.5 Flash (streaming)": "gemini-1.5-flash",
    "Claude 3 Haiku (streaming)": "claude-3-haiku",
    "llama-3.1-70b on Groq": "llama-3.1-70b",
}

unenable_model = [k for k, v in models.items() if v not in enabled_models]
for k in unenable_model:
    del models[k]

I also added a feature that allows for the deletion of inactive models when starting Streamlit.
But haven't added it to this PR yet.

It sounds convenient but I don't feel too strongly about adding it.
My suggestion: I still want to add the feature for supporting multiple agents which can be referenced by name (such as via a /path/ or body or header parameter). If we added that feature, we could have a general service metadata endpoint which returns the available agents and available models? And then the streamlit could be configured on that.
If you wanted to build it like that or in another way that would be easy to extend to also specify multiple agents, I'd be supportive. Thoughts?


Updating single document node rebuilds entire document

Branch with example: https://github.com/superlistapp/super_editor/compare/main...knopp:super_editor:single_node_rebuilds_document
https://github.com/superlistapp/super_editor/assets/96958/49d8ebc0-c012-4b27-b1a4-534a38a92a57
cc @matthew-carroll

@angelosilvestre for this ticket can you create a new example app called example_perf and configure it with a drawer that's similar to the existing example app? I'd like to move perf to it's own app so that we don't fill the shiny example app with ugly performance edge cases. I'm sure we'll add more over time.
Then, can you please add the given example to that app? This example currently demonstrates that rebuilding a single component in a document causes all other components to rebuild, too. But we don't want that to happen. We only want components to rebuild when they change. So the perf example we're adding is intended to show the opposite of what we're actually seeing.
With that example added, can you please see if you're able to adjust the component invalidating process so that identical component configurations don't cause rebuilds?

Thanks for looking into it. We had similar issue with our model.


updating old website url to current one

Small change, removes references to old website in favor of new one.

I'm not sure why this got closed. @rlmark You're listed as a maintainer on the team, so clearly you're in the org. And the workflow is using the token, so it should be able to see your org membership, even though it's listed as private.
Are your PRs and issues getting closed every time, or was it just this once?

@kytrinyx The workflow token PR hadn't been merged. I've subsequently merged it.


Added logo for AG

Not really needing a review - but to give you some visibility on this William..
These are the kind of incremental changes we might be making to the AG demo..


Force recheck causes PT to crash (0.25.0)

Try to force recheck a downloaded torrent and PT crashes each time with no error. Just poofs.
Windows 10 64 latest version

confirm🔥

I'm getting the same issue on Windows 7 64 with PicoTorrent 0.25.0 x64.


Computed paths example does not compile

Describe the bug
The Computed paths example in the Readme.md file does not compile.
There were typos with the argument names. I fixed them below and added the compiler errors.
To Reproduce
@CasePathable // 🔴 Circular reference resolving attached macro 'CasePathable'
enum Authentication { // 🔴 Circular reference
    case authenticated(accessToken: String)
    case unauthenticated
}

extension Authentication.AllCasePaths { // 🔴 'AllCasePaths' is not a member type of enum 'Authentication'
    var encrypted: AnyCasePath<Authentication, String> {
        AnyCasePath(
            embed: { decryptedToken in
                    .authenticated(accessToken: encrypt(decryptedToken))
            },
            extract: { authentication in
                guard
                    case let .authenticated(encryptedToken) = authentication,
                    let decryptedToken = decrypt(encryptedToken) // 🔴 Initializer for conditional binding must have Optional type, not 'String'
                else { return nil }
                return decryptedToken
            }
        )
    }
}

func encrypt(_ input: String) -> String {
    fatalError()
}
func decrypt(_ input: String) -> String {
    fatalError()
}

Expected behavior
The example demonstrates the computed paths properly.
Environment

swift-case-paths 1.5.4
Xcode 15.4
Swift 5.10
OS: macOS 14

Readme for easy reference
@CasePathable
enum Authentication {
  case authenticated(accessToken: String)
  case unauthenticated
}

extension Authentication.AllCasePaths {
  var encrypted: AnyCasePath<Authentication, String> {
    AnyCasePath(
      embed: { decryptedToken in
        .authenticated(token: encrypt(decryptedToken))
      },
      extract: { authentication in
        guard
          case let .authenticated(encryptedToken) = authentication,
          let decryptedToken = decrypt(token)
        else { return nil }
        return decryptedToken
      }
    )
  }
}

\Authentication.Cases.encrypted
// CaseKeyPath<Authentication, String>

@atacan This is a Swift bug that should be fixed in version 6. If you're in version 5.10 or below you need to define extensions of macro-generated types in a separate file to work around the bug.
Since this isn't an issue with the library I'm going to convert to a discussion, but let us know if we're missing something!


[AVM Module Issue]: update readme file to reflect required input names

Check for previous/existing GitHub issues

[X] I have checked for previous/existing GitHub issues

Issue Type?
I'm not sure
(Optional) Module Version
No response
(Optional) Correlation Id
No response
Description
readme file should be updated to reflect required input, from vnet_location to location.

Resolved in v0.2.0


installation error

npm notice
npm ERR! code ERESOLVE
npm ERR! ERESOLVE unable to resolve dependency tree
npm ERR!
npm ERR! While resolving<EMAIL_ADDRESS>npm ERR! Found<EMAIL_ADDRESS>npm ERR! node_modules/react
npm ERR<EMAIL_ADDRESS>from the root project
npm ERR!
npm ERR! Could not resolve dependency:
npm ERR! peer<EMAIL_ADDRESS>from<EMAIL_ADDRESS>npm ERR! node_modules/@homee/react-native-mapbox-navigation
npm ERR!   @homee/react-native-mapbox-navigation@"*" from the root project
npm ERR!
npm ERR! Fix the upstream dependency conflict, or retry
npm ERR! this command with --force, or --legacy-peer-deps
npm ERR! to accept an incorrect (and potentially broken) dependency resolution.

Have you found a way to install it?

Did anyone find a solution?
The only solution I could think of is that the author updates the package.json


Can't create references from the Cite menu

As reported by Giuliano. Currently all references need to be first created in the metadata panel before they can be cited. I agree that a quicker way of ad-hoc citing would be useful for authors.
In addition to showing available references in EditXRefTool, we could provide a link "Create new" at the top.

I honestly think you need to have the Add menu in the default view, populated by any and al element types that have a visible presence in the manuscript, and put it there. I feel like adding elements in general is a bit inconsistent at the moment.

Also support unifying that. This would create some questions though. When adding an author from the manuscript view, should we switch to metadata view automatically, or display a modal only for that action etc. I like that currently there is one place to edit for everything. Will think about this a bit more, and maybe it'd be good to move this to the next milestone and cleanup those inconsistencies.

I remember the version from very earlier this year did not have this separation View/Metadata and when I retested it I was kind of disappointed with this move.
That being said I agree with @gmaciocci that creating should be possible while in the main manuscript. I would go further and say that the metadata view should be removed.
I think that if Texture ever plans to be used for authoring and adopted by authors, this definitely has to be reviewed. We MAY hope but MUST not expect that users will change their authoring behavior.
So to answer @michael, if those are the only two options, then I personally believe that modals should be used instead of switching the user to a completely new interface.
If a third option of having the inputs inserted in a more seamlessly way like Edit-In-Place inline fields were available I would definitely vote for it.
A 4th option, which I think is more middle-ground, would be to have the whole editable sections from the metadata view moved to their respective position inside the manuscript view, pretty much exactly the same. The main difference is that they would become editable only when being edited. Once the user click outside they become their view equivalent. Something like the mokup below:
Before adding authors:

While adding authors*:
* while in this edit mode the toolbar needs to be updated to allow move up/down

After clicking outside:


Vendor check

This PR aims at enforcing always synchronized and up to date dependencies by running make vendor with a follow-up git-status check via papr. This way, we don't have to worry about the state of vendor.conf as the CI will complain if something is out of sync, or if certain packages are not needed anymore in ./vendor.
The idea to do this popped up over at CRI-O but I figured to start here, given the dependencies are already cleaned up :) If desired, I can open similar PRs over at Buildah and Skopeo.
Cc: @cevich @runcom @baude @mheon @rhatdan @mtrmac @TomSweeneyRedHat

Why isn't this wiping out the extra stuff Varlink vendors, I wonder? There are files that we're definitely not using in that vendor tree (using in the main code, at least... it builds a separate binary we use in the build phase), and I don't think we've told vndr that directory should be treated specially

make vendor has a whitelist :)
On Sun 3. Feb 2019 at 16:13, Matthew Heon<EMAIL_ADDRESS>wrote:

Why isn't this wiping out the extra stuff Varlink vendors, I wonder? There
are files that we're definitely not using in that vendor tree (using in the
main code, at least... it builds a separate binary we use in the build
phase), and I don't think we've told vndr that directory should be treated
specially
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/containers/libpod/pull/2259#issuecomment-460060071,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ALI4g0OwMIVeyKNaFitbvlrFGkHYPYedks5vJvyVgaJpZM4agCKJ
.

I would prefer to have new checks like this go into Cirrus, since PAPR is in maintenance mode.  I think what you're doing here could happen inside a container, no?  Something like:
diff --git a/.cirrus.yml b/.cirrus.yml
index 0efe7380..00927f32 100644
--- a/.cirrus.yml
+++ b/.cirrus.yml
@@ -110,6 +110,25 @@ gating_task:
         - '/usr/local/bin/entrypoint.sh lint'

+vendor_check_task:
+
+    depends_on:
+        - "gating"
+
+    env:
+        CIRRUS_WORKING_DIR: "/usr/src/libpod"
+
+    # Runs within Cirrus's "community cluster"
+    container:
+        image: "quay.io/libpod/gate:latest"
+        cpu: 4
+        memory: 12
+
+    gate_script:
+        - '/usr/local/bin/entrypoint.sh vendor'
+        - 'cd /go/src/github.com/containers/libpod && ./hack/tree_status.sh'
+
+
 build_each_commit_task:

     depends_on:
@@ -253,6 +272,7 @@ success_task:

     depends_on:  # ignores any dependent task conditions
         - "gating"
+        - "vendor_check"
         - "testing"
         - "optional_testing"
         - "cache_images"

(not tested: I think I spelled everything correctly)

Yes please; I’ve been frustrated a quite few times by repositories that immediately break when running make vendor and clearly have been in that state for months.
Keeping master branches in vendor.conf and then just not updating them has clearly not worked, so let’s at least be honest about pinning specific git commits (as I suspect this will force us to).

@vrothberg Hmm, one thing just occurred to me:  If a contributor fixes a small typo, their PR could be the unlucky one that finds vendor code is out of date.  In other words, it may not be fair to place this burden on everyone, all the time.
What do you think , if we limit this check to only running after merge?  When the post-merge testing fails, it puts an ugly red mark on the front-page: https://github.com/containers/libpod#library-and-tool-for-running-oci-based-containers-in-pods  In theory that means it will get noticed...eventually :smile:

(hint: See the build_each_commit_task section for how to limit this for post-merge/master testing)

I would prefer to have new checks like this go into Cirrus, since PAPR is in maintenance mode. I think what you're doing here could happen inside a container, no? Something like:

Sounds great! Can we get this check to run only for 1 job and not all to avoid redundant checks?

@vrothberg Hmm, one thing just occurred to me: If a contributor fixes a small typo, their PR could be the unlucky one that finds vendor code is out of date. In other words, it may not be fair to place this burden on everyone, all the time.

I don't think this could happen as we do not vendor branches anymore. The CI will only complain when the code, vendor.conf and the vendor directory are not in sync anymore and I think that's the right thing to do. This way, we can be sure that everything's in line.
To push it a bit further: with this change, we actually protect contributors from cleaning up or dealing with the vendor mess. Unfortunately, it is very common that much more things get changed when running vndr $project. When running make vendor now, only things are changed that the user actually caused.

@vrothberg oohhh, When running make vendor now, only things are changed that the user actually caused. is the key I was missing.  Yes the diff I posted will only run one task per PR, no matrix for each platform.  It'll run in parallel to the other tasks (except gating) and be fast (since it's in a container).

@cevich @vrothberg Is this ready to merge?

@rhatdan Not yet thx for asking.

@vrothberg in fact, if you want, this could be even simpler: Just tack the new commands onto the end of the existing 'gate' task's script section.  That will cause your check to gate ALL other testing if it fails.  So long as the check is quick, there's no reason for it to live in a separate task.  I'm fine either way.

@vrothberg Hehe, 36-seconds to run :smile: I thought it would take many more minutes.  Thanks for writing up what this does! Future maintainers and automation-junkies will appreciate that.
Now that you've got it working, and since it's so fast, what do you think about just including this check in the existing gating task (it's the same environment, and won't add much time)?  Also can we put your write up into the main docs?
diff --git a/.cirrus.yml b/.cirrus.yml
index dbd0e8b3..ea961a31 100644
--- a/.cirrus.yml
+++ b/.cirrus.yml
@@ -109,6 +109,12 @@ gating_task:
         - '/usr/local/bin/entrypoint.sh validate'
         - '/usr/local/bin/entrypoint.sh lint'

+    vendor_script:
+        - '/usr/local/bin/entrypoint.sh .install.vndr'
+        - '/usr/local/bin/entrypoint.sh vendor'
+        - 'cd /go/src/github.com/containers/libpod && ./hack/tree_status.sh'
+
+
 build_each_commit_task:

     depends_on:
diff --git a/contrib/cirrus/README.md b/contrib/cirrus/README.md
index e175479f..db93c6b2 100644
--- a/contrib/cirrus/README.md
+++ b/contrib/cirrus/README.md
@@ -28,6 +28,10 @@ task (pass or fail) is set based on the exit status of the last script to execut
 4. ``lint``: Execute regular `make lint` to check for any code cruft.
    Should also run for less than a few minutes.

+5. ``vendor``: runs `make vendor` followed by `./hack/tree_status.sh` to check whether the
+   git tree is clean.  The reasoning for that is to make sure that the vendor.conf,
+   the code and the vendored packages in ./vendor are in sync at all times.
+

 ### ``testing`` Task

There, now it's a perfect, polished, glossy-black finish on a race-car, and surely I best stop waxing it or it will never cross any finish-line :smiley:

/hold should-fail-but-doesnt

@vrothberg oh joy of joys.  Was just talking about that with @edsantiago the other day.  I fear it might be a royal PITA :cry:  (the dependencies are many in number from what Ed said).

@cevich my comment that day was that it was a mess because ginkgo wasn't vendored in; I think the least-awful approach is to vendor it + deps.

Done :+1:

This is getting seriously frustrating. Parts of the vendor/github.com/varlink come from different commits. The API has changed which now breaks the build. This PR will ultimately avoid such situations but it shows how easy it is to work around make vendor.

The vendored buildah had a very similar issue, so I reverted the breaking commit and cleaned up the varlink stuff.

Green :pray:

/lgtm


added laravel-9 and laravel-10 support

Thanks for adding support for new laravel versions

Thanks for adding support for new laravel versions

It is power of open source community. Thanks for accepting PR


Added ability to clear service and singleton overrides

When testing or in scripts, you sometimes want to temporarily set an override which needs to be later removed. Without a clearing function, you need to instantiate a new app that doesn't have the overrides, or resort to hacks, like replacing / re-instantiating the Locator.
This PR adds two methods: app.clearSingletonOverrides and ctx.clearServiceOverrides which call Locator#clearOverrides in the corresponding Locator.

how about singular version, clearOverride(Svc) - has there been any need for it?

@spion not that I know of. But I can add it -- it's not unreasonable to want to do that. Should I?

Maybe later when we need it.


Added place holder for actual error value

We can then do grid search base on this value

Thanks. Can you add the feature to load/save model in a separate PR ?


Bonfire: Exact Change suggestion

I suggest allowing the change array to return values in any order, or at least in reverse or original order.
In its current form, it allows only reversed order which is counter-intuitive though slightly easier to implement. Since it's already reversed from the input order presumably for convenience, it seems reasonable that the order actually is not and should not be important.

I'm on mobile, so researching this is a bit af a hassle, but if this should be implemented, I think either Chai.js' assert.deepEqual or assert.sameDeepMembers methods would be helpful.

Who is this send a pic
On Jul 3, 2015 3:30 AM, "Anton Strömkvist"<EMAIL_ADDRESS>wrote:

I'm on mobile, so researching this is a bit af a hassle, but if this
should be implemented, I think either Chai.js' assert.deepEqual or
assert.sameDeepMembers methods would be helpful.
—
Reply to this email directly or view it on GitHub
https://github.com/FreeCodeCamp/freecodecamp/issues/1186#issuecomment-118309748
.

I'm not quite sure what you're suggesting, @Chrysallis. The way it's set up now is it needs to be reversed from the original array, and only including the values being given as change. I'm sure it's set up that way as a bit of an extra challenge.

Looking at the challenge I found this line: Otherwise, return change in coin and bills, sorted in highest to lowest order. so I'm sure it's intended.
It makes sense since cashiers tend to give the change back counting larger values first, and even if you end up with an array with the correct values in the wrong order, you should just be able to make a call to Array.prototype.sort() to fix that.

Closing this as stale


[Request] Rider

Add Jet Brains Rider

For extras bucket its recommended to use toolbox. If you want Pure rider try www.github.com/Ash258/Scoop-JetBrains

Amazing, thanks


Update to impress 2.6.4 and metacom 2.0.3

Landed in https://github.com/metarhia/Example/commit/a8a31ed9d5d248ed058b1fae6e638951e15adeff


Event item - remove subtitles option on event page as does not do anything

The subtitles option does not do anything on the event screen (option below 'watch now').

estimate 30 minutes


Add orders fulfilled infobox

We're able to track the amount of orders fulfilled by checking the pvarbit 4480,
therefore we can add an infobox to display it.
the infobox will be disabled by default.

Also please rebase this PR

Fixed the comments and rebased the changes. Not sure if there's a better way than rebasing and force pushing to the branch on the forked repo but it's up to date at least :)

If you can, please fix this checkstyle violation:
https://github.com/BlueSoapTurtle/mastering-mixology/actions/runs/11095008585/job/30823122369
GH actions are not triggering in the PR because it was opened before i added them i think

Ready for review / merge
Will do a bit better with the commits in future PRs, new to github PRs

Needs rebased and feedback addressed

back home now so will update the PRs, just rebased it but will fix the comments soon

Rebased, ready for review or merge

Had to rebase so made a few changes:

renamed updateInfoboxes to refreshInfoboxes
only call refreshInfoboxes when the correct config key is changed
moved the varp check to the bottom since it's separate from the varbits
moved OrdersFulfilledInfoBox out of the ui package

Ready again for review or merge


Question - Get query of row

Data too long for column 'name' at row 27808, I get this type of error.
How to see what query at row 27808 on bulk insert?

It would be whatever was at that row in your input file.

It would be whatever was at that row in your input file.

Solved by debug it using fs, mysql.format and get the of row (line number)


Better support unhandled promise rejections

Depending the version of Node, unhandled promises rejections exit the program.
It seems some part of the program is not handling some promises correctly
I am using node 15.5.
After putting an handler (in index.js):
process.on('unhandledRejection', error => {
	console.log('unhandledRejection', error.message);
	console.trace(error);
});

here is the log:
unhandledRejection undefined
Trace: _Event {
  type: 'error',
  isTrusted: false,
  _yaeti: true,
  target: W3CWebSocket {
    _listeners: {},
    addEventListener: [Function: _addEventListener],
    removeEventListener: [Function: _removeEventListener],
    dispatchEvent: [Function: _dispatchEvent],
    _url: 'ws://<IP_ADDRESS>:14944',
    _readyState: 3,
    _protocol: undefined,
    _extensions: '',
    _bufferedAmount: 0,
    _binaryType: 'arraybuffer',
    _connection: undefined,
    _client: WebSocketClient {
      _events: [Object: null prototype] {},
      _eventsCount: 0,
      _maxListeners: undefined,
      config: [Object],
      _req: null,
      protocols: [],
      origin: undefined,
      url: [Url],
      secure: false,
      base64nonce: 'nEfONKqYabTY70pR3QGz/A==',
      [Symbol(kCapture)]: false
    },
    onclose: [Function: value],
    onerror: [Function: value],
    onmessage: [Function: value],
    onopen: [Function: value]
  },
  cancelable: true,
  stopImmediatePropagation: [Function (anonymous)]
}
    at process.<anonymous> (/projects/polkadot-launch/dist/index.js:34:11)
    at process.emit (node:events:376:20)
    at emit (node:internal/process/promises:202:22)
    at processPromiseRejections (node:internal/process/promises:223:25)
    at processTicksAndRejections (node:internal/process/task_queues:94:32)

It seems to be related to yaeti (I'm not familiar with) and websocket.

Also another error that appears a bit later:

Starting Parachain 200: 5Ec4AhPTL6nWnUnw58QzjJvFd3QATwHA3UJnvSD4GVSQ7Gop, Collator port : 15335 wsPort : 15946 rpcPort : 15846
Added  to parachain
Added --execution=wasm to collator
Registering Parachain 200
--- Submitting extrinsic to register parachain 200. (nonce: 0) ---
--- Submitting extrinsic to set balance of 5Ec4AhPTL6nWnUnw58QzjJvFd3QATwHA3UJnvSD4GVSQ7Gop to<PHONE_NUMBER>000000000000. (nonce: 1) ---
Current status is Ready
unhandledRejection undefined is not iterable (cannot read property Symbol(Symbol.iterator))
Trace: TypeError: undefined is not iterable (cannot read property Symbol(Symbol.iterator))
    at _createForOfIteratorHelper (/home/alan/projects/polkadot-launch/dist/index.js:20:104)
    at _callee$ (/home/alan/projects/polkadot-launch/dist/index.js:261:26)
    at tryCatch (/home/alan/projects/polkadot-launch/node_modules/regenerator-runtime/runtime.js:63:40)
    at Generator.invoke [as _invoke] (/home/alan/projects/polkadot-launch/node_modules/regenerator-runtime/runtime.js:293:22)
    at Generator.next (/home/alan/projects/polkadot-launch/node_modules/regenerator-runtime/runtime.js:118:21)
    at asyncGeneratorStep (/home/alan/projects/polkadot-launch/node_modules/@babel/runtime/helpers/asyncToGenerator.js:3:24)
    at _next (/home/alan/projects/polkadot-launch/node_modules/@babel/runtime/helpers/asyncToGenerator.js:25:9)
    at processTicksAndRejections (node:internal/process/task_queues:93:5)
    at process.<anonymous> (/home/alan/projects/polkadot-launch/dist/index.js:34:11)
    at process.emit (node:events:376:20)
    at emit (node:internal/process/promises:202:22)
    at processPromiseRejections (node:internal/process/promises:223:25)
    at processTicksAndRejections (node:internal/process/task_queues:94:32)

Also another error that appears a bit later:

Starting Parachain 200: 5Ec4AhPTL6nWnUnw58QzjJvFd3QATwHA3UJnvSD4GVSQ7Gop, Collator port : 15335 wsPort : 15946 rpcPort : 15846
Added  to parachain
Added --execution=wasm to collator
Registering Parachain 200
--- Submitting extrinsic to register parachain 200. (nonce: 0) ---
--- Submitting extrinsic to set balance of 5Ec4AhPTL6nWnUnw58QzjJvFd3QATwHA3UJnvSD4GVSQ7Gop to<PHONE_NUMBER>000000000000. (nonce: 1) ---
Current status is Ready
unhandledRejection undefined is not iterable (cannot read property Symbol(Symbol.iterator))
Trace: TypeError: undefined is not iterable (cannot read property Symbol(Symbol.iterator))
    at _createForOfIteratorHelper (/home/alan/projects/polkadot-launch/dist/index.js:20:104)
    at _callee$ (/home/alan/projects/polkadot-launch/dist/index.js:261:26)
    at tryCatch (/home/alan/projects/polkadot-launch/node_modules/regenerator-runtime/runtime.js:63:40)
    at Generator.invoke [as _invoke] (/home/alan/projects/polkadot-launch/node_modules/regenerator-runtime/runtime.js:293:22)
    at Generator.next (/home/alan/projects/polkadot-launch/node_modules/regenerator-runtime/runtime.js:118:21)
    at asyncGeneratorStep (/home/alan/projects/polkadot-launch/node_modules/@babel/runtime/helpers/asyncToGenerator.js:3:24)
    at _next (/home/alan/projects/polkadot-launch/node_modules/@babel/runtime/helpers/asyncToGenerator.js:25:9)
    at processTicksAndRejections (node:internal/process/task_queues:93:5)
    at process.<anonymous> (/home/alan/projects/polkadot-launch/dist/index.js:34:11)
    at process.emit (node:events:376:20)
    at emit (node:internal/process/promises:202:22)
    at processPromiseRejections (node:internal/process/promises:223:25)
    at processTicksAndRejections (node:internal/process/task_queues:94:32)

The second error is due to the config not having a "simpleParachains" field, as added recently (https://github.com/paritytech/polkadot-launch/pull/25). Btw, we should have a talk about what this adder-collator is

The second error is due to the config not having a "simpleParachains" field, as added recently (https://github.com/paritytech/polkadot-launch/pull/25). Btw, we should have a talk about what this adder-collator is


Allow splitting of backends for type

Currently, you can select one backend type and everything uses that backend.
We would like to split this to allow different backends for different types of messages, eg. have voice messages through vumi, and text messages through junebug.
Splitting the events might prove difficult, so we've decided to just have two event urls, since we can set the event url for the Junebug messages to be different than the vumi messages.

@smn Ready for review


Stop SubscribeConfiguration when client disconnects to prevent leaks and enable graceful shutdown

Fixes #6560
Clients can continue to call UnsubscribeConfiguration for backwards-compatibility. However, doing so will show a warning encouraging users to simply close the connection to the SubscribeConfiguration RPC.

/ok-to-test

/ok-to-test

@ItalyPaleAle Let's call out deprecation of gRPC unsubscribe API in the release notes and then remove it later.

Sounds good! I'll add the "Breaking change" label here so we don't forget (although this is not a breaking change)


Backdoors possibilities reduction

Shell augmentations is a nice way to introduce backdoors. Especially popular augmentations like oh-my-zsh. It consist of tonns of addons, it is nearly impossible to audit them all. Especially in such a language as a shell where there are plenty of ways to shot your leg with ok-looking code. So I think we should pick the most popular zsh addons for most popular software like git and dd and move everything else into a separate repos. This will reduce attack surfacevolume by reducing presence of unneeded and unaudited code in user's system.

I agree with the diagnostic but the solution is worse. I'd rather have the code bundled within OMZ than having it be outside the repository and outside our control.
This is a complex topic though. Closing until there's a better solution.


Settings are reset after restarting Sway config or reboot

All theme settings are correctly applied, but after restarting Sway config all reset to default Adwaita theme. The only workaround for this is using import-gsettings script https://github.com/swaywm/sway/wiki/GTK-3-settings-on-Wayland
nwg-look 0.1.3
Ubuntu 22.04 LTS
Sway 1.7

I'd say something in your system must revert them. No idea, I haven't used Ubuntu for years, and also I haven't seen your configs. Could you paste your sway config file?

Closing due to no further feedback.