cmd/go: go env GOMOD tries to download newer toolchain

If go.mod says
module example.com

go 1.22

Then running go env GOMOD with 1.21 prints:
$ go env GOMOD
go: downloading go1.22 (linux/amd64)
go: download go1.22 for linux/amd64: toolchain not available

This is a bit silly, as go env needs to determine GOMOD in order to decide to upgrade in the first place.
cc @bcmills

I suspect in the general case this is working as intended. The Go 1.22 language isn't finalized yet, and in theory go env GOMOD in a Go 1.22 module may do something that today's go command doesn't know about and can only produce correct behavior by doing the download.
Does it work well to do go get<EMAIL_ADDRESS>in said module before go.mod if using Go 1.21 (RC 3 or so), or using a very recent gotip that knows how to work with a 1.22 module?

Yep, go env GOMOD and go env GOWORK in particular have to be valid (or at least somewhat sensible), because those are what we use to determine which toolchain to upgrade to.
I suppose it's possible that we upgrade the toolchain via the go.mod and/or go.work file that we find and then the reinvoked toolchain decides to do something else instead (perhaps based on an environment variable), but a change that affects that sequence seems unlikely in the foreseeable future.

Is this message helpful and actionable -

Just got bitten by this thing - the installed Go version is 1.21.7, the go.mod file has 1.22, and the CI script was trying to run go version to determine the installed Go version and see if it would need to be updated (and normally the new version of Go would be fetched and installed after that).  The failure (quite cryptic, IMO) was:
go: downloading go1.22 (linux/amd64)
go: download go1.22 for linux/amd64: toolchain not available

a workaround would be to cd /tmp before executing go version.  Still seems unfortunate to have to do this...

@dmitris - I tried a fix for this with local installed go version correcting mod file. Screenshot was attached.
There is one more scenario..
Do you consider is it worth to get rid of manual version update on go workspace  with go developement project shared with other go modules when ever a bump happens say 1.22 to 1.23.
For that , I am thinking of an approach is there a way to know - golang/go project do exist under workspace for respective go commands( advise user on that) and change to that version for workspace and correct other modules under that workspace if needed.

@matloob, @samthanawalla: this is a side-effect of the recent addition of toolchain-upgrade support. I think it will have a fairly straightforward fix.

@bcmills  - I already did a fix. Shall I CL that ?

Regarding OP, just to confirm, the requested change is that we don't want go env GOMOD to download the newer toolchain? You only want it to display the file name of the go.mod file and not upgrade?

Yes, that was the original intention intended by this issue (Bryan suggested I file this issue). I don't personally feel strongly about the behavior, it is just a bit of an oddity given that cmd/go needs to find go.mod in order to upgrade in the first place.


add support for custom formats

This PR adds a dialog allowing the user to specify custom a separator and delimiter before opening or saving a file.
Finally all that data I have in :hankey:SV format comes into play! :)
Closes #122

Rebased onto master to resolve conflict

This works great, apart from (on OSX at least) once you save a file with a custom format, you have to close the Custom window that spawns. The following seems to work:
Setting up an optional callback in saveFileAs:
function saveFileAs(format, window, callback) {
  if (!window) {
    var window = BrowserWindow.getFocusedWindow();
  }
  Dialog.showSaveDialog({ filters: format.filters }, function (fileName) {
    if (fileName === undefined) return;
    window.webContents.send('saveData', fileName, format);
    utils.enableSave();
    window.format = format;
    if (callback) callback();
  });
}

Then adding a callback in saveAsCustom which closes the dialog like so:
  ipc.once('formatSelected', function(event, data) {
    var format = makeCustomFormat(data.separator, data.delimiter);
    saveFileAs(format, window, function() {
      dialog.close();
    });
  });

I'm going to look at adding you as a contributor to make this sort of collaboration easier in the future. I might even move Comma Chameleon to its own organisation if the ODI folks are OK with it, given I don't work there anymore 😄

Odd - on linux calling window.close() here closes it correctly.
I will have a bash at implementing your solution and update.
Cheers

If we do it like this and move the .close() operation to the start of the start of the 'formatSelected' handler (but before we create the open/save dialog), does that solve it for you? Or is it important that we call Dialog.showSaveDialog() / Dialog.showOpenDialog() before we try to close the Custom dialog? In principle, they don't need to be open at the same time as we've already got the data we need from the custom dialog before we call Dialog.showSaveDialog() / Dialog.showOpenDialog().
Also, does the cancel button work for you (i.e: does this window.close() statement execute correctly on your platform?)

Ah yeah, that works. I missed the window.close() call in the frontend code, so that's why I had to implement the callback. I'm happy with this now, so will merge and push up a new release!


[QUESTION] EDITING AN EXCEL FILE WITH PIVOT TABLE WHEN SAVE PIVOT IS GONE

Please prefix your issue with one of the following: [BUG] [PROPOSAL] [QUESTION].
Package version, Laravel version
Package version: "maatwebsite/excel": "~2.1.0"
Laravel version: 5.4
Expected behaviour
Want to input data from sheet 3 then sheet 2 has a pivot table that will compute all datas inside the sheet 3 then sheet 1 will just show the results of sheet 2 in an organize way
Note that: I am just loading an existing file then saving a new one. So its like I have a template for computations for me to get the report every month
Actual behaviour
when i append datas inside sheet 3 sheet 2's computation or a way to get data is gone.
Can you help me with this one

I'm seeing this issue as well. Appears to apply to normal tables as well.
Also, annoyingly, when it removes normal tables, existing references to those tables are broken as wel leaving #REF! in it's place...


refactor(RELEASE-1191): use simple-signing-pipeline for irs

This commit adds our own copy of the set-env task to our internal directory. Using our own copy allows us to break apart the UMB and pyxis credentials.

Relates to https://github.com/hacbs-release/app-interface-deployments/pull/241


Avoid reparenting in date picker that potentially causes an issue wit…

…h focus handler
Description
A customer reported an issue with the date picker focus behaviour. I couldn't reproduce it on my end but here is how it looks like (confirmed with a screen share):

The date picker is initially disabled;
The checkbox is clicked to enable the component;
User enters an invalid value to the date picker input and clicks outside, the onBlur handler is called and a validation error is shown;
User enters a valid value to the date picker input and clicks outside - the onBlur handler is not being called and the validation error is not cleared.

The issue is not reproducible with a different type of input e.g time input.
The issue is not reproducible if the date picker is initially enabled.
This is related to the use of the focus tracker. When the date picker input is disabled the trigger is rendered alone and the reference used by the focus tracker is not attached.
How has this been tested?
Verified with the customer by sending an updated artefact.
Documentation changes
[Do the changes include any API documentation changes?]

[ ] Yes, this change contains documentation changes.
[x] No.

Related Links

Ticket: AWSUI-19519

Review checklist
The following items are to be evaluated by the author(s) and the reviewer(s).
Correctness

[ ] Changes are backward-compatible if not indicated, see CONTRIBUTING.md.
[ ] Changes do not include unsupported browser features, see CONTRIBUTING.md.
[ ] Changes were manually tested for accessibility, see accessibility guidelines.

Security

[ ] If the code handles URLs: all URLs are validated through the checkSafeUrl function.

Testing

[ ] Changes are covered with new/existing unit tests?
[ ] Changes are covered with new/existing integration tests?

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

But this PR does not fix any reparenting. The hierarchy is still unequal between editable and readonly states

div

Dropdown

trigger

div

trigger


Translate Bot Token Scopes & Event Subscriptions Page

Hello contributors, we are looking for volunteers to help us translate our Bot Token Scopes & Event Subscriptions Page into Portuguese.
Translate the wiki Bot Token Scopes & Event Subscriptions Page into Portugese.
Instructions:
Before you begin, please review our README file. It contains important information on our translation guidelines, standards, and how to get started. You can find it here.
If you're interested in taking on this translation task, please leave a comment on this issue. We'll assign the task to you, and you can start working on it.
Make sure to adhere to our translation guidelines provided in the README. This ensures consistency and quality throughout the translation.
Once the translation is complete, please create a pull request with your changes. We'll review it, and upon approval, merge it into our repository.

Hi, would gladly help you translate the getting started page into Portuguese. @aoscode @aosccode

take it @Lmandavela!

Hi @Lmandavela, are you still working on this?

Hi @antonio-pedro99, do you know if @Lmandavela is still working on this issue ?

Hi @antonio-pedro99, do you know if @Lmandavela is still working on this issue ?

I am not sure. Let's unsign?

Sure


[CSL-2129] HTTP-backed block retrieval

Closing in favor of https://github.com/input-output-hk/cardano-sl/pull/2193


Make sure Site class is kept up-to-date

Reviewer: @dharb

Description:
Adds the webNavigation.onCommitted event as another signal that we should update the site URL.
This fixes two problems:

not updating the popup around special sites (e.g. chrome://extensions)
not updating the popup when we get 200 from a service worker, rather than the server (see below)

Steps to test this PR:
To test the first problem:

Open a new tab and open e.g. chrome://extensions or about:addons - the popup should say "extensions"
Open e.g. duckduckgo.com in the same tab - the popup should say duckduckgo.com
Open the extension page again in the same tab - the popup should say "extensions" again
Pressing back and opening the popup should always show the correct site name

And the second (requires a Gmail account):

Log into your Gmail account
Go to googlemail.com
The popup should now say mail.google.com.

Automated tests:

[ ] Unit tests
[ ] Integration tests

Reviewer Checklist:

[ ] Ensure the PR solves the problem
[ ] Review every line of code
[ ] Ensure the PR does no harm by testing the changes thoroughly
[ ] Get help if you're uncomfortable with any of the above!
[ ] Determine if there are any quick wins that improve the implementation

PR Author Checklist:

[ ] Get advice or leverage existing code
[ ] Agree on technical approach with reviewer (if the changes are nuanced)
[ ] Ensure that there is a testing strategy (and documented non-automated tests)
[ ] Ensure there is a documented monitoring strategy (if necessary)
[ ] Consider systems implications

Hey @andrey-p, this is really good! Sorry for dropping the ball on reviewing this sooner, totally slipped my mind. I've done quite a bit of manual testing and haven't been able to break anything, but what would you say to writing a quick unit test or two for this change before we merge?

Awesome, looks good! 🎆


Make Result and ReedlineError public

Allows implementing custom History types
Closes #545

Seems reasonable, just need to fix the CI errors.

@fdncred fixed!

Thanks


[BUG?] " is not displayed correctly

Hi @suzuki-shunsuke thanks for developing such a useful tool!
I found a bug in github-comment and would like to report it.
What happend
When I posted the results of terraform validate -no-color with github-comment, the double quotes (") were not displayed correctly. Specifically, it is converted to ".
PR comment

Configuration
actions.yaml
      - name: Terraform validate
        working-directory: ${{ matrix.workdir }}
        id: validate
        run: github-comment exec -k validate -- terraform validate -no-color
        env:
          GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}

.github-comment.yaml
---
exec:
  validate:
    - when: ExitCode != 0
      template: |
        ## :x: ERROR : terraform validate
        {{template "link" .}}
        ```
        {{.CombinedOutput}}```

Expected behavior
Error: Reference to undeclared module

  on main.tf line 1118, in module "web_frontend":
1118:   alb_tg_arn = module.web_api_gateway_alb.alb_tg_arn

No module call named "web_api_gateway_alb" is declared in the root module.

Other
The GitHub Actions results appear to be correct.

Is this a bug on the github-comment side?

I guess AvoidHTMLEscape would solve the problem.
{{.CombinedOutput | AvoidHTMLEscape}}

Thank you for reply quickly!
It was a very mundane mistake. Sorry..
When I specified AvoidHTMLEspace, it was displayed as expected.


Can i get invidiual frame to process?

I want to be able to receive every frame to process and return to preview. can you help me?

@morejump
Are you trying to apply some live filter effects to the preview ?
I am trying to do the same, but unfortunately I think that there is no way CameraKit exposes individual frames......

This is coming soon, please reference #176


Fix run example

removed internal npm call to run-example from the execSync fucntion and instead added the commands directly here as env variables are not getting passed

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


This might be the fix for the upper disappearing word of issue 3871

https://github.com/tesseract-ocr/tesseract/issues/3871

This does not work for -c edges_use_new_outline_complexity=1

It might be this 'fix' revives the 'rejected parent' from the bucket, while other rejected blobs should probably still be available and are killed with the empty word as well. I think the children are mistrusted, as those blobs come by as well making the row, including the dot on the i. As the new layers of tesseract even know how to revive a parent I guess they could cope better with those mistrusted children as well. I'm trying to get to the right point in the debugger...

I changed the strategy of keeping the parent alive to just kill the parent as soon as it is known. That solves the issues on both lines. I wonder whether other examples can reveal a motivation for reviving the parent to a lesser extent.


ESLint v9

Problem
ESLint is used in the repo, but it's not enforced in the pipeline. Also, upgrading directly to ESLint v9 isn't possible due to conflicting dependencies with the @typescript-eslint libraries.
Solution
Many, many changes took place (in no particular order):

.eslintrc is replaced with eslint.config.js using the new flat config.
Most of the existing settings have been migrated over to the flat config.
All ESLint updates are grouped for dependabot.
npm run lint is added to the CI pipeline.
eslint-config-react-app & vite-plugin-eslint are replaced with @eslint/js, eslint-plugin-react, eslint-plugin-react-hooks, eslint-plugin-react-refresh, and typescript-eslint.
All enums are moved to a separate directory.
The projects list in the Projects component has been moved into a separate component to make the file smaller and easier to read. ProjectList will handle all the logic by loading the GitHub stats and styling each project list.
MongoDB is added to the list of technologies for EZ Recipes.
capitalize has been moved to a more reusable helper function.
Improved typing of Octokit.

Test Plan
npm run lint && npm test && npm run build all succeed. There are also no errors shown in the IDE. The website behaves like normal.

Still can't merge until eslint-plugin-react is compatible:

npm error Could not resolve dependency:
npm error peer eslint@"^3 || ^4 || ^5 || ^6 || ^7 || ^8" from<EMAIL_ADDRESS>npm error node_modules/eslint-plugin-react
npm error   dev<EMAIL_ADDRESS>from the root project

This config looks pretty clean though: https://github.com/facebook/react/issues/28313#issuecomment-2180984628


UCD and Use Case Diagram - Address Feedback

The comments from the prior Deliverable must be addressed.
    For any comment that was not addressed, deduct -2 points.

Update Use Case Models and Descriptions document with potential new use cases and based on the feedback given on prior Deliverable(s)

If the number of UCD were below 10 or the models/diagrams  were below 2 in Deliverable 2 and they did not change in Deliverable 3, deduct -5 points for each.

If the models/diagrams are missing entirely, an additional -5 points

Use this [use case description](https://docs.google.com/document/d/1jBaLHNLAAEPAqN7BCqZspq04X625YcIY/edit?usp=sharing&ouid=108788083057609121398&rtpof=true&sd=true) template. Here is an [example of Use Case Diagrams and Descriptions](https://drive.google.com/file/d/1_r0T_X7xg1zB9Vnt0CAREqIjXAOmi0MV/view?usp=sharing). Here is [another example](https://docs.google.com/document/u/0/d/1HsytILdXk_8f-2vJjSbXwhj9OYWIcvVP/edit?fromCopy=true).

Make the use case diagrams first, then assign individual people to making each use case’s description.

About 10 - 20 use case descriptions and 2-4 use case diagrams/models each including 3 - 5 use cases.

    If the number of use case descriptions (UCD) is below 10, deduct -3 points for any one missing. For example, if there are only 8 UCD, then deduct -6 points.

    If the number of use case diagrams/models is less than 2, deduct -10 points.

Use case diagrams/models should be correct and have the correct links.

    If they have unnecessary arrows, deduct -2 points per each mistake.

    If they are complicated, deduct -5 points per each diagram/model.

    Use cases should start from the actor. If not, deduct -2 points for each mistake.

    Only have include, extend and generalize links between the use cases.

        If the links are not correct or they do not have a type, deduct -2 points for each mistake.

Use case descriptions should follow the templates/guidelines from slides or the given template.

    If they do not match, deduct -5 points for each mistake.

The name of the UCD should match with the use cases in the diagram.

    If they do not match, deduct -1 point per each mistake.

The related use cases should be shown correctly in the UCD

    If they do not show correctly, deduct -1 point per each mistake.

For every other mistake, deduct -1 points for each.

If the quality of the UCD or diagrams varies substantially, deduct -10 for the team not internally reviewing/quality checking their work before submission. Not recoverable with regrad

Good job on these edits! I know this document is kind of a beast :P
I saw your comment on UCD 17, you may want to bring that up at our next meeting. (And yeah this is good, marking this as done!)


 I want to concatenate them and save to zarr.
I concatenated them with CDO and saved to netcdf.
Now I have a ~500GB netcdf that I want to convert to zarr.
I want to convert to zarr because:

I plan to run the analysis on a cluster, and I understand that zarr is better for that
By using float16 and lz4 compression, I believe I can reduce the size  to ~100GB and have faster access (I think the analysis will be i/o bound).

The netcdf:
<xarray.Dataset>
Dimensions:  (lat: 721, lon: 1440, time: 119330)
Coordinates:
  * time     (time) datetime64[ns] 2000-01-01T06:00:00 2000-01-01T06:00:00 ...
  * lon      (lon) float32 0.0 0.25 0.5 0.75 1.0 1.25 1.5 1.75 2.0 2.25 2.5 ...
  * lat      (lat) float32 90.0 89.75 89.5 89.25 89.0 88.75 88.5 88.25 88.0 ...
Data variables:
    mtpr     (time, lat, lon) float32 ...
Attributes:
    CDI:          Climate Data Interface version ?? (http://mpimet.mpg.de/cdi)
    Conventions:  CF-1.6
    history:      Fri Jul 27 16:35:19 2018: cdo -f nc4 mergetime 2000-01.grib...
    institution:  European Centre for Medium-Range Weather Forecasts
    CDO:          Climate Data Operators version 1.9.3 (http://mpimet.mpg.de/...

The code:
def netcdf2zarr(nc_path):
    chunks = {'time': -1, 'lat': 'auto', 'lon': 'auto'}
    encoding = {'mtpr': {'dtype': 'float16', 'compressor': zarr.Blosc(cname='lz4', clevel=9)},
                'lat': {'dtype': 'float32'},
                'lon': {'dtype': 'float32'}}
    ds = xr.open_dataset(nc_path).chunk(chunks)
    ds.to_zarr('myfile.zarr', encoding=encoding)

Problem description
I left my code to run over the weekend. After 63h of processing, the zarr store was only 1GB in size. The system monitor indicated that the Python process had 17TB worth of disk read.
At that rate, it would have taken months to finish.
Is there something that I can do to increase the processing speed?
I run Ubuntu 18.04 on a Core i7-6700 with 16GB of RAM. The disk is an HDD with a speed of ~100MB/s.
 We routinely do this sort of transcoding and it works reasonable well. Unfortunately something clearly isn't working right in your case. These things can be hard to debug, but we will try to help you.
You might want to start by reviewing the guide I wrote for Pangeo on preparing zarr datasets.
It would also be good to see a bit more detail. You posted a function netcdf2zarr that converts a single netcdf file to a single zarr file. How are you invoking that function? Are you trying to create one zarr store for each netCDF file? How many netCDF files are there? If there are many (e.g. one per timmestep), my recommendation is to create only one zarr store for the whole dataset. Open the netcdf files using open_mfdataset.
If instead you have just one big netCDF file as in the example you posted above, I think I see you problem: you are calling .chunk() after calling open_dataset(), rather calling open_dataset(nc_path, chunks=chunks). This probably means that you are loading the whole dataset in a single task and then re-chunking. That could be the source of the inefficiency.
More ideas:

explicitly specify the chunks (rather than using  'auto')
eliminate the negative number in your chunk sizes
make sure you really need clevel=9

Another useful piece of advice would be to use the dask distributed dashboard to monitor what is happening under the hood. You can do this by running
from dask.distributed import Client
client = Client()
client

In a notebook, this should provide you a link to the scheduler dashboard. Once you call ds.to_zarr(), watch the task stream in the dashboard to see what is happening.
Hopefully these ideas can help you move forward.

@rabernat Thanks for your answer.
I have one big NetCDF of ~500GB.
What I have changed:

Run in a Jupyter notebook with distributed to get the dashboard
Change the chunks to {'lat': 90, 'lon': 90}. That should be around 1GB per chunk.
Chunk from the beginning with ds = xr.open_dataset('my_netcdf.nc', chunks=chunks)
About the LZ4 compression, I did some test with a 1.5GB extract and the writing time was just 2% slower than uncompressed.

Now when I run to_zarr(), it creates a zarr store (~40kB) and all the workers start to read the disk, but they don't write anything.
The Dask dashboard looks like this:

Is this the expected behaviour? I was somehow expecting that each worker will read a chunk and then write it to zarr, streamlined. This does not seem to be the case.

Can you forget about zarr for a moment and just do a reduction on your dataset? For example:
ds.sum().load()

Keep the same chunk arguments you are currently using. This will help us understand if the problem is with reading the files.
Is it you intention to chunk the files contiguously in time? Depending on the underlying structure of the data within the netCDF file, this could amount to a complete transposition of the data, which could be very slow / expensive. This could have some parallels with #2004.

@rabernat I tried to do the sum. I have the same issue. The process just seems to read the disk endlessly, without even writing to the RAM:

I tried to lower the chunk size, but it doesn't seem to change anything. Without chunk, I logically get a MemoryError.
I plan to do time-series analysis, so I thought that having contiguous chunks in time would be more efficient. The netcdf was created with cdo -f nc4 mergetime, so it should have mostly the same structure, I guess.

Out of curiosity:

why do you chunk over lats and lons rather than time? The order of dimensions in your dataarray suggest that chunking over time could be more efficient
can you show the output of ds.mtpr and ds.mtpr.encoding ?

forget my comment about chunks - I thought this would make a difference but it's actually the opposite (to my surprise):
import dask as da
import xarray as xr

da = xr.DataArray(da.array.zeros((3000, 721, 1440), chunks=(10, 721, 1440)),
                             dims=('z', 'y', 'x'))
da.to_netcdf('da.nc')  # creates a 25GB file!

with xr.open_dataarray('da.nc', chunks={'z':10}) as da:
    %timeit da.sum()
5.55 ms ± 29.5 µs per loop (mean ± std. dev. of 7 runs, 100 loops each)

with xr.open_dataarray('da.nc', chunks={'y':10}) as da:
    %timeit da.sum()
2.05 ms ± 19.4 µs per loop (mean ± std. dev. of 7 runs, 100 loops each)

with xr.open_dataarray('da.nc', chunks={'x':10}) as da:
    %timeit da.sum()
3.03 ms ± 74.6 µs per loop (mean ± std. dev. of 7 runs, 100 loops each)

@fmaussion ds.mtpr:
<xarray.DataArray 'mtpr' (time: 119330, lat: 721, lon: 1440)>
dask.array<shape=(119330, 721, 1440), dtype=float32, chunksize=(119330, 16, 16)>
Coordinates:
  * time     (time) datetime64[ns] 2000-01-01T06:00:00 2000-01-01T06:00:00 ...
  * lon      (lon) float32 0.0 0.25 0.5 0.75 1.0 1.25 1.5 1.75 2.0 2.25 2.5 ...
  * lat      (lat) float32 90.0 89.75 89.5 89.25 89.0 88.75 88.5 88.25 88.0 ...
Attributes:
    long_name:  Mean total precipitation rate
    units:      kg m**-2 s**-1
    code:       55
    table:      235

ds.mtpr.encoding:
{'complevel': 0, 'shuffle': False, 'dtype': dtype('float32'), 'contiguous': False,
 'zlib': False, 'source': u'my_netcdf.nc', 'fletcher32': False,
 'original_shape': (119330, 721, 1440), 'chunksizes': (1, 721, 1440)}

Those chunksizes are the opposite of what I was expecting...

Those chunksizes are the opposite of what I was expecting...

chunksizes in encoding are ignored in your case, dask still uses your user provided encoding.
Can you still try to chunk along one dimension only? i.e. chunk={'time':1} or chunk={'lon':1}

Sorry for the confusion, I had an obvious mistake in my timing experiment above (forgot to do the actual computations...).
The dimension order does make a difference:
import dask as da
import xarray as xr

with xr.open_dataarray('da.nc', chunks={'z':10}) as d:
    %timeit d.sum().load()
3.94 s ± 95.5 ms per loop (mean ± std. dev. of 7 runs, 1 loop each)

with xr.open_dataarray('da.nc', chunks={'y':10}) as d:
    %timeit d.sum().load()
4.15 s ± 316 ms per loop (mean ± std. dev. of 7 runs, 1 loop each)

I did some tests with my big netcdf. The chunking indeed makes a difference.
chunks = {'time': 'auto', 'lat': 'auto', 'lon': 'auto'}
ds = xr.open_dataset('era5_precip.nc', chunks=chunks)
ds.sum().load()

real	161m37.119s
user	33m9.720s
sys	63m47.696s

chunks = {'time': 1}
ds = xr.open_dataset('era5_precip.nc', chunks=chunks)
print(ds.sum().load())

real	109m55.839s
user	303m40.665s
sys	451m30.788s

I'll do some more tests with the calculation of the mean on the time axis, it might be more representative of what I want to do.

I ran a comparison of the impact of chunk sizes with a profiler:
profiler = Profiler()
for chunks in [{'time': 30}, {'lat': 30}, {'lon': 30}]:
    print(chunks)
    profiler.start()
    with xr.open_dataset(nc_path, chunks=chunks) as ds:
        print(ds.mean(dim='time').load())
    profiler.stop()
    print(profiler.output_text(unicode=True, color=True))

I am not sure if the profiler results are useful:
    {'time': 30}
    <xarray.Dataset>
    Dimensions:  (lat: 721, lon: 1440)
    Coordinates:
      * lon      (lon) float32 0.0 0.25 0.5 0.75 1.0 1.25 1.5 1.75 2.0 2.25 2.5 ...
      * lat      (lat) float32 90.0 89.75 89.5 89.25 89.0 88.75 88.5 88.25 88.0 ...
    Data variables:
        mtpr     (lat, lon) float32 8.30159e-06 8.30159e-06 8.30159e-06 ...
    5652.770 compare_chunks  read_grib.py:281
    └─ 5652.613 load  xarray/core/dataset.py:466
       └─ 5652.613 compute  dask/base.py:349
          └─ 5652.404 get  dask/threaded.py:33
             └─ 5652.400 get_async  dask/local.py:389
                └─ 5629.663 queue_get  dask/local.py:127
                   └─ 5629.663 get  Queue.py:150
                      └─ 5629.656 wait  threading.py:309

In the case of chunks on lat or lon only, I get a MemoryError.
I don't know if this helps, but it would be great to have a solution or workaround for that. Surely I am not the only one working with dataset of that size? What would be the best practice in my case?

The dask task graph seems right (mean on the time dimension, lower number of chunks to make the visualisation practical):

If I understand well, the 'getter' are doing the actual reading of the file, but in reality, they do not seem to run in parallel.
As for the zarr writing part, I do not know how to check the task graph.

This might be worth testing with the changes from https://github.com/pydata/xarray/pull/2261, which refactors xarray's IO handling.

It seems that I managed to get something working as it should.
I first load my monthly grib files with iris, convert to xarray, then write to zarr. This uses all the CPU cores, but loads the full array into memory. Since the individual arrays are relatively small, that is not an issue.
Then I load the monthly zarr stores with xarray, concatenate them with auto_combine and write to a big zarr. The memory usage peaked just above 17GB with 32 CPU threads. The array and chunks dimensions are:
(time, latitude, longitude) float16 dask.array<shape=(113969, 721, 1440), chunksize=(113969, 20, 20)>
I guess that reducing the chunk size will lower the memory usage.
Using that big zarr storage, plotting a map of the mean values along the time axis takes around 15min, uses all the cores and around 24GB of RAM.
The strange part is: I think I tried that before and it was not working...


fix alloy metrics app catalog

Fix alloy metrics app catalog
This PR:

adds/changes/removes etc

Checklist

[ ] Update changelog in CHANGELOG.md.
[ ] Make sure values.yaml and values.schema.json are valid.

Alloy was released, I think it was an oversight as Théo was testing a dev version for metrics


Some problems with google API ?

Hi !
I use Cobot for my coworking space in France.
Your app is so great, I put it on ly Cobot account, all is going quite well but it's not possible to retrieve data from the google sheets doc.
It seems this the API now works differently.
Do you think you could fix it or give some help to solve this issue ?
Regards
Aymeric.

Hi, we've been able to finally reproduce the issue. The problem was that the wrong Google API to use in your project was noted in the instructions. They said Google Docs API, where it should've been Google Sheets API.
Sorry for the confusion, the instructions are fixed now


Add helper BodyClass for appending classes to the body tag from View …

…components.
This might be handy for support different flavours on same view/s or other handy use cases. It will also be aware to the full body of the existance of the toolbar, which will lead to better CSS to handle it.
This makes obsolete the use of Helmet in Plone-React for managing body classes. Mixing both will not work at all. React Helmet was not going to implement this, because backwards compat, so I used the same approach (using react-side-effects) and composing the body tag incrementally. It also works on SSR without problems.
I also removed all uses of Helmet for the body classes. It might be handy to have it around to manage other things (like the title) for now.
I also fixed some wrong use of Helmet (calls to .peek() that only should be used in tests) which is discouraged in Helmet docs. It turns out that we weren't even using it.
I think it's time to start a "developer guide", might be handy from now on...
What do you think @robgietema ??

Coverage increased (+0.2%) to 51.009% when pulling dcdbc927785da9ec8972f10b3f37bfb548112c49 on improved-helmet-body-classname-appendable into 3f134eb2f4cb3a98c7ad64b9bc71051850edf1af on master.


How to change the color of the TiTv plot using oncoplot.

Hi. I dont know if the maftools oncoplot can change the color of TiTv.

Hi,
I have added a new argument titv_col to the oncoplot which should enable custom color codes. Please install the tool from GitHub for the updated version.
# generate some random color codes for the 6 TiTv classes
titvcols = RColorBrewer::brewer.pal(n = 6, name = "Dark2")
names(titvcols) = c("C>T", "C>G", "C>A", "T>A", "T>C", "T>G")

# usage
luad = maftools::tcgaLoad(study = "luad")
maftools::oncoplot(maf = luad, draw_titv = TRUE, titv_col = titvcols)

Hope this helps and let me know if it works for you.


Compare texts line-wise to support CRLF newline in Windows

Fixing #1318 .

I found this is not safe if the file not ends with newline.
str::lines() does not respect the last empty line...
https://doc.rust-lang.org/std/primitive.str.html#method.lines


[C++] Fix name of interval type

This type is as yet unimplemented/unused anywhere else in the library

Yes that's what I discovered when looking for inspiration for the Go implementation.
I don't think I'm able to implement the C++ version though.
Should we leave the name as it is?

Codecov Report

Merging #3949 into master will increase coverage by 0.81%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master    #3949      +/-   ##
==========================================
+ Coverage   87.75%   88.57%   +0.81%
==========================================
  Files         727      593     -134
  Lines       89507    79924    -9583
  Branches     1252        0    -1252
==========================================
- Hits        78547    70790    -7757
+ Misses      10842     9134    -1708
+ Partials      118        0     -118

Impacted Files
Coverage Δ

cpp/src/arrow/type.h
95.94% <ø> (ø)
:arrow_up:

go/arrow/math/uint64_amd64.go

go/arrow/memory/memory_avx2_amd64.go

js/src/enum.ts

go/arrow/array/builder.go

js/src/Arrow.node.ts

js/src/schema.ts

go/arrow/type_traits_boolean.go

js/src/ipc/node/writer.ts

js/src/visitor/vectorloader.ts

... and 126 more

Continue to review full report at Codecov.

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

@alexandreyc I recommend that you do not create a Go implementation until the Format-level discussions are resolved around this. There is an active mailing list discussion

Agree with @pitrou. Merging.


Operator @rx has different flags in two engines

Describe the bug
It seems like the @rx operator has a different behavior in two engines (mod_security2 and libmodsecurity3)
mod_security2 has these PCRE flags:
PCRE2, PCRE
libmodsecurity3 has these ones:
PCRE2, PCRE
To Reproduce
https://github.com/coreruleset/coreruleset/issues/3277
We should discuss:

do we want to resolve this issue?
how?

I think this is a good idea, I mean add a build flag to libmodsecurity3 (mod_security2 is not affected) which changes the flag, and not in the next release but after that we make it mandatory (and we can add an optional build flag to keep the old (current) one).

PCRE2_DOTALL & PCRE2_DOLLAR_ENDONLY look the right way for me as we need to check multiline ARGS


What's this for?

I'm curious about this project and what's the motivation. I'm a computer vision geek and wanted to know what you two are toying around with here. What are you trying to have it do?
I'm curious about the multimodal models in general when it comes to images and text.

This is a fork of LLaVA, to work with Mistral. Objective is to push the vision LLM even further than what LLaVA team has accomplished. We'll be taking it in a different direction with many ideas in store to extend its capabilities, you're more than welcome to help contribute if you're interested.


Remove explicit workflow mode

Proposed changes
I think this one is ready to go, I just need a glance at the documentation. It's mostly just deleting things, which makes predicting the consequences a bit difficult, so I'd appreciate a good look-over and possibly some beta-testing on existing workflows, if possible.
There is now no explicit, opt-in workflow mode. Instead, behaviour is entirely driven by choice of output folder. "Workflow mode" will still trigger when the output dir is the same as the snakebids root, or in the results subfolder. This simply implies that the working directory will remain in the snakebids root, meaning the config file and .snakemake folder will be saved there. All other paths will trigger the old "bidsapp" mode.
I also added a cli flag to force outputs into new directories with contents, aka any directory without a .snakebids folder. This should hopefully prevent accidental writing into unintended directories without making it impossible, while still allowing seamless re-running of the same workflow. Note that these checks will not be triggered at all in the "workflow mode" conditions.
Question regarding the docs: there's currently a section dedicated to "workflow mode" but it mostly talks about setting your output dir to be equal to the snakemake dir. Very little about actually using --workflow-mode, and that can easily be removed. So, should I just keep references to "workflow mode" with the understanding that it's not literally a mode you turn on, but just a consequence of choosing certain output paths? It is a convenient description...
Checklist
Put an x in the boxes that apply. You can also fill these out after creating the PR. If you are unsure about any of the choices, don't hesitate to ask!

[x] Changes have been tested to ensure that fix is effective or that a feature works.
[x] Changes pass the unit tests
[x] I have included necessary documentation or comments (as necessary)
[x] Any dependent changes have been merged and published

Just realized that the deprecated files will all still appear in the CLI help, which probably isn't important. There is a way to keep the arg but prevent it from appearing in help (so that a nice error message will print if someone tries to use it). Should we go that route instead? @tkkuehn @kaitj @akhanf

There is a way to keep the arg but prevent it from appearing in help (so that a nice error message will print if someone tries to use it).

Sounds good to me.
Also, I may be wrong but I don't think we want the .benchmark directory (I assume from pytest-benchmark) checked in?

Also, I may be wrong but I don't think we want the .benchmark directory (I assume from pytest-benchmark) checked in?

Good catch, it's only .gitignored in the bids-inputs branch, so I must have let it in by accident.


AMBARI-25162 Fix Oozie Service Check

Added missing import statement
What changes were proposed in this pull request?
Fixed AMBARI-25162 by adding the messing import statement
How was this patch tested?
Manually. The same changes were used in a live cluster to fix the issue and run successful service check.

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/Ambari-Github-PullRequest-Builder/4940/
Test PASSed.

Thanks @dineshchitlangia for the fix.


Universal Clipboard

Hello, i would love to see if XClipper has this feature, which is auto copy the copied text, and sync it across devices (connected devices). What i mean by that is any text copied to the clipboard will be synced between all your devices. It will also be automatically inserted into the clipboard of connected devices.
Let say we have 3 Devices, which is Device A (Cellphone), Device B (Tablet) and Device C (Windows PC or Laptop), so when Device A copy any text, then it will sync to Device B and C, and will automatically inserted into their clipboard, so when they hit PASTE, the text that appeared should be the text that copied from Device A. This is what i mean by Universal Clipboard. Hopefully you understand it.

You can always click on "Copy" button in the notification. I don't see how this is a problem, plus your current clipboard also changes a lot so I think it is better to not make the sync clip item as your default clipboard.

Yes, the thing is the COPY button is not on the notification, i have to press Special Actions > Share Button then finally Copy Button.
You know, this is just a little suggestion, as a clipboard manager, it's very nice to have auto sync and inserted across devices that connected, so that we can work seamlessly between devices.

Yes, I'll add a copy button or a similar function.

Okay. Waiting for the update...

Ya maybe I'll revamp the notification system completely. That should also improve the productivity a lot.

Thanks for your work on this @KaustubhPatange. I look forward to the productivity improvements coming to the notification, so thanks also for that.
However, I wanted to also request the same auto sync feature, as long as the user can enable or disable it if they wish. But of course I don't want to be unreasonable - do you know if this would be a very burdensome or difficult feature to implement, test and maintain?

As I'll add here @riotrah, I find a very limited amount of time from my work to focus on such side projects which is why developing features takes a huge amount of time not because it is complex or something but because of the lack of time I get during a day or weekend.
I understand the requirement here & will try to get this feature out whenever I find time to work on it.


new user addition in tsusers group not reflected in xrdp without xrdp service restart

client:-ubuntu 16.04 with freerdp 2.1
server:-centos7_x64
xrdp version:-0.9.5
when new user added in tsusers group ,xrdp service need to be restarted to take effect of tsusers group
modification.
This should be dynamic.

#1061 turned out that the issue for new users? If so, close the either one.

@gnshkl1 Hello?

@metalefty we can close this issue

why?

@metalefty this issue seems to be solved .
users are reflected in xrdp in running service.
#1061 needs to be fixed.


timeout with reqHistoricalDataAsync

Hello,
I'm facing a weird behavior in my script when I call the reqHistoricalDataAsync function in the main thread it returns data very quickly (0.3 seconds) but calling the same function with the same inputs in a thread take exactly 60 seconds (when I checked this function in the package I found out that the default timeout is also 60 seconds so I believe it comes from it) my question how does the function wait to timeout mentioned to return data in a thread and not in the main thread:
async def get_historical_data(self, contract, end_date_time, duration, bar_size="1 min", what_to_show="TRADES"):         return await self.ib_client.reqHistoricalDataAsync(             contract,             endDateTime=end_date_time,             durationStr=duration,             barSizeSetting=bar_size,             whatToShow=what_to_show,             useRTH=True,         )
below is the call from a thread:
bars = asyncio.run(                             self.get_historical_data(                                 contract=position["contract"],                                 end_date_time=f"{opening_range_end.strftime('%Y%m%d %H:%M:%S')} {self.timezone_code}",                                 duration=f"{position['opening_range_minutes'] * 60} S",                                 bar_size="1 min",                                 what_to_show="TRADES",                             )                         )
could you please help me understand this and how to fix it?
Thanks.

One potential source of "nothing happens until the timeout:" when some of the IBKR APIs receive bad inputs, they return out-of-band error messages (check the log messages) but no return value.
So often we end up seeing things like "Error: contract id invalid" as an immediate status update in the logs, then your program will just freeze until the return fails to complete.
The best guard against it is to lower the timeout using async.wait_for(thing, timeout=seconds) to run your own faster error checking 🤷♂
Longer term we may be able to find a more stable way to bind error messages to currently running requests, so if we get an error status when a request is active, it could potentially stop the request early instead of waiting for the timeout.


Feat/3324 table

Changes

Arrow icon added
Table component added

@wagner-schwertz-poatek @jotape-poatek please review and don't need to wait for my approval.


Simplify installation using Docker / Docker-Compose

Hi there,
This project looks amazing, however having a bunch of prerequisites like Java gets in the way for me.
I would like to request the addition of a configured docker-compose.yml that would allow someone to get the entire thing running in just a couple of commands.
Has anyone already done this?
I'm surprised the live-view portion uses docker, but not the rest.
Cheers

Aha, I was able to get this going by cherry-picking parts of the liveview docker example (just the bambuweb service).

Do you think it will run in Synology Docker?

@blasirio Should work if theres enough resources :)

How would I install this on Docker?

How would I install this on Docker?

Check out https://github.com/TFyre/bambu-farm/tree/main/docker/bambu-liveview

Closing issue due to non activity


[warning] failed package validation and/or copy for commit 14a22ad0cc75c4ef064a132d49c450f8fca1e5cb

Hi @conda-forge/opentelemetry-proto! This is the friendly automated conda-forge-webservice!
It appears that one or more of your feedstock's outputs did not copy from the
staging channel (cf-staging) to the production channel (conda-forge). :(
This failure can happen for a lot of reasons, including an outdated feedstock
token. Below we have put some information about the failure to help you debug it.
Rerendering the feedstock will usually fix these problems.
If you have any issues or questions, you can find us on gitter in the
community chat room or you can bump us right here.
error messages:

invalid feedstock token

Hi @conda-forge/opentelemetry-proto! This is the friendly automated conda-forge-webservice!
It appears that one or more of your feedstock's outputs did not copy from the
staging channel (cf-staging) to the production channel (conda-forge). :(
This failure can happen for a lot of reasons, including an outdated feedstock
token. Below we have put some information about the failure to help you debug it.
Rerendering the feedstock will usually fix these problems.
If you have any issues or questions, you can find us on gitter in the
community chat room or you can bump us right here.
error messages:

invalid feedstock token

Hi @conda-forge/opentelemetry-proto! This is the friendly automated conda-forge-webservice!
It appears that one or more of your feedstock's outputs did not copy from the
staging channel (cf-staging) to the production channel (conda-forge). :(
This failure can happen for a lot of reasons, including an outdated feedstock
token. Below we have put some information about the failure to help you debug it.
Rerendering the feedstock will usually fix these problems.
If you have any issues or questions, you can find us on gitter in the
community chat room or you can bump us right here.
error messages:

invalid feedstock token

@conda-forge-admin please rerender

Hi! This is the friendly automated conda-forge-webservice.
I just wanted to let you know that I rerendered the recipe in conda-forge/opentelemetry-proto-feedstock#5.

@conda-forge-admin please rerender

Hi! This is the friendly automated conda-forge-webservice.
I just wanted to let you know that I rerendered the recipe in conda-forge/opentelemetry-proto-feedstock#7.

Hi @conda-forge/opentelemetry-proto! This is the friendly automated conda-forge-webservice!
It appears that one or more of your feedstock's outputs did not copy from the
staging channel (cf-staging) to the production channel (conda-forge). :(
This failure can happen for a lot of reasons, including an outdated feedstock
token. Below we have put some information about the failure to help you debug it.
Rerendering the feedstock will usually fix these problems.
If you have any issues or questions, you can find us on gitter in the
community chat room or you can bump us right here.
error messages:

invalid feedstock token

@conda-forge-admin please rerender

Hi! This is the friendly automated conda-forge-webservice.
I just wanted to let you know that I rerendered the recipe in conda-forge/opentelemetry-proto-feedstock#8.


macOS Java JDK pop up appears on executing brew doctor and brew config

brew config output
HOMEBREW_VERSION: 2.7.7-120-gd2430bc
ORIGIN: https://github.com/Homebrew/brew
HEAD: d2430bcf50d513807ab62320ff966d569afc40dc
Last commit: 3 hours ago
Core tap ORIGIN: https://github.com/Homebrew/homebrew-core
Core tap HEAD: 9700fddec6199c8cd3bb98429ea2a00224b3decf
Core tap last commit: 3 hours ago
Core tap branch: master
HOMEBREW_PREFIX: /usr/local
HOMEBREW_BROWSER: open
HOMEBREW_CASK_OPTS: []
HOMEBREW_CLEANUP_MAX_AGE_DAYS: 14
HOMEBREW_EDITOR: /usr/local/bin/mvim
HOMEBREW_GITHUB_API_TOKEN: set
HOMEBREW_MAKE_JOBS: 8
Homebrew Ruby: 2.6.3 => /usr/local/Homebrew/Library/Homebrew/vendor/portable-ruby/2.6.3_2/bin/ruby
CPU: octa-core 64-bit ivybridge
Clang: 12.0 build 1200
Git: 2.30.0 => /usr/local/bin/git
Curl: 7.64.1 => /usr/bin/curl
macOS: 10.15.7-x86_64
CLT: <IP_ADDRESS>
Xcode: N/A
brew doctor output
Your system is ready to brew.

[X] The brew doctor above contains no "Warning" lines.

What were you trying to do (and why)?
Run brew doctor.
What happened (include all command output)?
A pop up window that says "To use the "java" command-line tool you need to install a JDK. Click "More-info..." to visit the Java Developer Kit download website."
It appears to me that the following check is the source of the issue:
Checking check_cask_software_versions
java -version

I do not have Java installed.
What did you expect to happen?
No Java JDK pop up.
Step-by-step reproduction instructions (by running brew commands)
brew doctor

git blame points to https://github.com/Homebrew/brew/commit/447ea12d386832672c165d0fa29400bfcbb6efa8 but I didn't see the issue until a few days ago.

Still occurring in the new release (version 3).

Still seeing this on latest when running brew doctor.
Thanks brew! ❤

@JBallin what does brew --version say for you?

@carlocab I'm actually only seeing it on my Mac running Catalina, which shows Homebrew 3.0.0.
My other Mac, running Big Sur, does not have the issue and shows Homebrew 3.0.0-69-g3bcd919.
Curious why they show different versions?

Your Big Sur machine is running a dev version of brew, where this has already been fixed. Presumably you've run a dev command there, and that's why that's the version you're getting.
Your Catalina Mac will be fixed in 3.0.1.


Change directory structure from PSR-0 to PSR-4

Should try to sync this change with the next major release

Same as #35.  Closing this one.


Add ability to search and activate entities via the command dialog

Signed-off-by: Sebastian Malton<EMAIL_ADDRESS>fixes https://github.com/lensapp/lens/issues/1011

I think for this PR we should leave it as it is because I don't think we should pollute the top level of the command dialog too much.
If that becomes a problem then I think we could have a different shortcut to directly open the second dialog.
How does that sound @ixrock ?

I think for this PR we should leave it as it is because I don't think we should pollute the top level of the command dialog too much.
If that becomes a problem then I think we could have a different shortcut to directly open the second dialog.>
How does that sound @ixrock ?

Sounds good.


No debloated-pom.xml

I have used  configuration as true as shown in screenshot but it is not generating any debloated-pom.xml

Hi @harpreet86,
Thanks for reporting this issue.
The configuration of DepClean in your pom.xml is correct.
Could you please give a pointer to the repository you're using?
This way I can have a closer look and debug.
Thanks.

I'm closing this issue due to a lack of engagement.


Zone bypass at HONEYWELL panels

Is there any way to bypass zones with Honeywell panels?

There is currently no built-in way.  If Honeywell panels accept a keystroke sequence to bypass a zone then you could use this integrations alarm_keypress service call to send the appropriate key strokes.
I don't have a Honeywell panel myself so am not very familiar with them but when I looked at the EVL documentation I didn't see anything that allowed me to determine which zones are currently bypass which is why I did not expose bypass switches for the Honeywell panels.

I don't do any coding so can't help with designing this application but am very aware of the Vista HW panels.
To bypass a specific zone it is a keying sequence of  6
To view the current list of bypassed zones it is a keying sequence of  6#
There is no way to turn off individual zone bypasses (that I am aware of) and you must "disarm" the system (even though it is not armed) using the keying sequence of  1
Not sure if any of this information helps and makes this possible to add the the integration.


Fix/ 404 on events fetch

This PR takes care of #23

@zidhuss I noticed that the link PR was merged, so I rebased. Apologies.

No need to apologise it's all good 👍
The approach is nice but unfortunately only looks to be working on Google Chrome.
Firefox:

Safari:

No need to apologise it's all good 👍
The approach is nice but unfortunately only looks to be working on Google Chrome.

@zidhuss made a quick fix, but a dirty one haha. I think this might be the only way (other than creating an endpoint). I will leave it to your judgment.

@zidhuss Really great suggestions! Covered them on my last commit.


:selected does not work with html5 datetime

Environment

Ruby 2.7.5
Rails <IP_ADDRESS>
Simple Form 5.1.0
Browser chrome.

Current behavior
I have the following code:
class DateTimeInput < SimpleForm::Inputs::DateTimeInput
  private

  def use_html5_inputs?
    input_options.fetch(:html5, true)
  end
end

With a custom from like this
= simple_form_for(:files, url: files_album_path(@album), method: :get) do |f|
  = f.input :from, label: from, as: :datetime, include_blank: true, selected: @from, required: false #, input_html: { value: @from.strftime('%FT%T') }

The html input is empty. The work around is to uncomment the input_html part.
Note that even using @from.iso8601 does not work as it doesn't like the +200 at the end of the string.
Using html5: false also fixes the problem but the input is then "ugly" 😅
Expected behavior
The html input should be correctly filled when using html5.

Hey! I believe it's expected behavior. When you use html5 inputs datetime uses datetime_field from Rails and it's regular input field with datetime-local type. So it's not a select that's why selected option doesn't work since there's nothing to select. Since selected option is not something that SimpleForm is introduced (it's regular Rails's option) I don't think we should support it with html5 inputs

So we are supposed to use input_html?

yes. or alternatively you can use form object where you set your value so you don't have to set it in the view


sekurlsa::minidump from nanodump output file

i am using nanodump for dumping lsass.exe. everything is ok, but when i get to mimikatz by following command,got error:
mimikatz.exe "sekurlsa::minidump <path/to/dumpfile>" "sekurlsa::logonPasswords full" exit

mimikatz error: ERROR kuhl_m_sekurlsa_acquireLSA ; Memory opening

i use "x64 nanodump ssp dll", and AddSecurityPackage winapi for attaching to lsass
when i was testing all way's, detect that nanodump specified dump file size(default=>report.docx),is different from procmon.exe Full and Mini dump output.
my test:
procmon full = 71 MB ,procmon mini = 1.6 MB
nanodump = 11 MB

The SSP module creates a dump file with an invalid signature by default.
Have you tried to restore the signature using the scripts/restore_signature?

The SSP module creates a dump file with an invalid signature by default. Have you tried to restore the signature using the scripts/restore_signature?

very thanks, I'm not familiar with signatures, this approach solved my problem, thanks :)) @S4ntiagoP

Great!


Trying to map, but class does not conform the protocol 'Mappable'

I would like to map a JSON that I am receiving, but I am getting a message saying that:
Type 'City' does not conform to protocol 'Mappable'.
This is my class definition:
public protocol Mappable {
static func newInstance(map: Map) -> Mappable?
mutating func mapping(map: Map)
}
class City: Mappable {
var state: String?
var name: String?
var id: String?
class func newInstance(map: Map) -> Mappable? {
    return City()
}

func mapping(map: Map) {
    state   <- map["state"]
    name    <- map["name"]
    id      <- map["id"]
}

}
and I trying to map using this line:
let city = Mapper().map(data)
However, I am getting the error message I have mentioned.

To map data to an object of type City, you must do so as follows:
let city = Mapper<City>().map(data)


Fixed issue-44, view is hidden below status bar during call

Fixed issue #44.  When in-call, hotspot, or recording status bar is enabled, just after (custom) modal dismissal transition animation ends UITransitionView is removed from the hierarchy and that removal was moving rootViewController.view 20 points upwards, pushing view beneath statusbar and leaving a black gap at the bottom. The PR fixes this by pushing it 20px downwards, the work that UITransitionView should ideally have done after custom modal dismissal.
Also fixed issue #43

Thank you!


[BUG]python grpc call go-micro err：StatusCode.UNKNOWN

Describe the bug

What are you trying to do?
What did you expect to happen?
What happens instead?

How to reproduce the bug:
If possible, please include a minimal code snippet here.
Environment:
Go Version: please paste go version output here
please paste `go env` output here

Duplicate with #2347.


[QA] online docs are not updated

Seen in ocis v1.0.0-rc3 using ocis/ocis/docker-compose-eos-test.yml
I've none my test setup according to the 4 setup items listed in https://owncloud.github.io/ocis/eos/

Worked "astonishingly" well, but unfortunately this is outdated.
The source in https://github.com/owncloud/ocis/blob/master/docs/ocis/eos.md lists 6 setup items.
We should also clarify for which docker-compose.yml the docs apply.
The ocis/ocis/docer-compose-eos-test.yml should have different (simpler) instructions than ocis/ocis/docker-compose.yml

See also https://github.com/owncloud/ocis/issues/361 as a list of open items that are possibly helpful for eos setups.

The sync logic is maintained at https://github.com/owncloud/owncloud.github.io
Fixed by https://github.com/owncloud/ocis/commit/17198620acbfdf976c00ab11374162e0a82689c8


Allow to specify arguments in the adapterExecutableCommand based on the debug configuration

For a custom debug adapter, written in Java, that uses LSP4J for client/server communication, I would like to be able to specify (debug server) program arguments derived from the debug configuration.
The extension uses a adapterExecutableCommand to determine the executable path, which  is registered and implemented like this:
function registerGetDebugAdapterCommand(context: ExtensionContext) {
  context.subscriptions.push(commands.registerCommand('<command-id>', () => {
    return {
      command: '/absolve/path/to/executable',
      args: ['debug-server', '--trace']
    };
  }));
}

The --trace option shows details about the messages being sent forth and back and help during development or to track down problems in production. This option, however, I would like to set only if a corresponding setting in the debug configuration has been specified.
If the DebugConfiguration could be accessed from within the adapterExecutableCommand implementation, the arguments could be specified accordingly.
LSP4J makes it difficult to pass this information at a later point in time - the message tracing option needs to be set up when starting the listener that takes control over the communication process.
By passing this option through a custom request, the first few request/response cycles would go untraced.
VS Code version is 1.26

duplicate of #45220


Update format for semantic science blog

Description
Minor formatting update for the semantic science blog.
Issues Resolved
[List any issues this PR will resolve]
Check List

[ x] Commits are signed per the DCO using --signoff

By submitting this pull request, I confirm that my contribution is made under the terms of the BSD-3-Clause License.

@dtaivpp @nateynateynate Could you review this PR when you get a chance please?


Extension update

Source information
flixscans.org
Source language
English
Steps to reproduce
Cant open in normal view
Expected behavior
The extension expected to open in normal view but can't.
Actual behavior
This is what happened the the extension open "Couldn't reach Unable to resolve host "api.flixscans.org": No address associated with hostname"
Mihon/Tachiyomi version
0.16.4
Android version
Android 12
Other details
N/A
Acknowledgements

[X] I have searched the existing issues and this is a new ticket, NOT a duplicate or related to another open or closed issue.
[X] I have written a short but informative title.
[X] I have updated the app to version 0.15.3.
[X] I have updated all installed extensions.
[X] I have tried the troubleshooting guide.
[ ] I have admitted that I am a clown by having checked this box, as I have not read these acknowledgements.
[X] If this is an issue with the app itself, I should be opening an issue in the app repository.
[X] I will fill out all of the requested information in this form.

Working fine on mine! Check your wifi or user agent data! Its browsing fine on mine!


Match request host format to our own. Closes #167

request.get_host returns host:port without the protocol.

:+1:


Invalid data expected an object

Is there an existing issue for this?

[X] I have checked for existing issues https://github.com/getsentry/sentry-javascript/issues
[X] I have reviewed the documentation https://docs.sentry.io/
[X] I am using the latest SDK release https://github.com/getsentry/sentry-javascript/releases

How do you use Sentry?
Sentry Saas (sentry.io)
Electron SDK Version
4.1.2
Electron Version
12.2.3
What platform are you using?
MacOS
Link to Sentry event
https://stephan-petzl.sentry.io/issues/4040133110/?alert_rule_id=829169&alert_timestamp=1679923869387&alert_type=email&environment=production&project=1554144&ref_fallback=ctrl&referrer=alert_email
Steps to Reproduce
Just adding custom breadcrumb like this:
function translateArgs(args) {
  const [firstArg, ...otherArgs] = args
  const params = {}
  if (typeof firstArg === 'string') {
    params.message = firstArg
    if (otherArgs && otherArgs.length > 0) {
      params.data = { otherArgs }
    }
  } else {
    params.data = args
  }
  return params
}

 Sentry.addBreadcrumb({ category, ...translateArgs(args), level: 'debug' })

Expected Result
Most of the time custom breadcrumbs are working fine, but found strange case where it shows error in breadcrumbs data Invalid data expected an object.
Actual Result
It should show actual breadcrumb info instead of this error.

@ahmedmukhtar1133 With your current code, params.message is only conditionally being set, and data can be set to null.
This is causing the breadcrumb to be invalid. Use the JSON button to examine the event JSON and see what the SDK sent as breadcrumbs values.

I'm just tidying up the stale issues in this repository. Please open another issue if this is still causing you problems.


Zsh completion

Features:

can complete standard vagga options (--environ, --version etc)
can complete both project and standard vagga commands
hidden commands (the ones that start with an underscore) are not showing by default, but will complete after writing first underscore: _<TAB>
can complete options from the commands with options (docopt) field specified in vagga.yaml
show description for both commands and options if available
can complete special objects (FILE, DIR and PATH) when specified (e.g. --option=PATH). This is provided by zsh _arguments function.

Overall, looks too simple to be true :)
You should fix install.sh script. (moved bash completion, add zsh completion).
And I don't see the part that completes options. Is it somehow provided by zsh itself?

Refactored print_list function and added --zsh option to _list command
Fixed install.sh script for bash completion. It seems zsh uses the fpath environ variable to find completion functions, not sure where to put zsh-completion.sh during installation.

And I don't see the part that completes options. Is it somehow provided by zsh itself?

Yes, the last argument to _arguments function (--) tells zsh that it should run the command with --help and parse an output. We need to call this function twice: ones for vagga options itself (vagga --help), and once for command (vagga _help <CMD> --help).

I guess it should go into /usr/share/zsh/site-functions

Put completion script to /usr/share/zsh/site-functions in installation script, however according to this thread different Linux distributions have different default folders for completions (Ubuntu/Debian -- /usr/**local**/share/zsh/site-functions, CentOS/RHEL -- /usr/share/zsh/site-functions).
I can also make PR in oh-my-zsh with this script in case someone wants to install it in this way.

Looks good now. Thanks!
We need to wait for a release of vagga 0.7.1 before submitting it to oh-my-zsh I think.


[Bug]: unable to update to laravel 11

What happened?
I cant install library to laravel 11 project
How to reproduce the bug
composer r cheesegrits/filament-phone-numbers
Package Version
1.0.2
PHP Version
8.3.4
Laravel Version
11.0.7
Which operating systems does with happen with?
Linux
Notes
No response

Just as an FYI, I made pull request #4 to address this.

@cheesegrits can you merge it and create a release?

Bump. Please integrate this pull request.


not working properly

It worked for many websites but it is not working for small websites and newly created websites.

Can you please provide examples?

Dont know if this website is small or newly created, however it just won't work for me.
wappalyzer http://hortec-berlin.de will never finish for me (tested on different machines).
root@Wappalyzer:~# wappalyzer http://www.hortec-berlin.de --debug

wappalyzer | log | driver | Launching browser...
wappalyzer | log | page | Navigate to http://www.hortec-berlin.de/

and never does anything after.

Which version are you using? Your example works fine for me.

Huh, weird. I'm on version 6.0.12. What's your version?
root@Wappalyzer:~# npm list -g
/usr/local/lib
└─┬<EMAIL_ADDRESS>  ├──<EMAIL_ADDRESS>  └─┬<EMAIL_ADDRESS>    ├──<EMAIL_ADDRESS>    ├─┬<EMAIL_ADDRESS>    │ └──<EMAIL_ADDRESS>    ├─┬<EMAIL_ADDRESS>    │ ├─┬<EMAIL_ADDRESS>    │ │ ├──<EMAIL_ADDRESS>    │ │ ├──<EMAIL_ADDRESS>    │ │ ├─┬<EMAIL_ADDRESS>    │ │ │ ├──<EMAIL_ADDRESS>    │ │ │ ├──<EMAIL_ADDRESS>    │ │ │ ├──<EMAIL_ADDRESS>    │ │ │ ├──<EMAIL_ADDRESS>    │ │ │ ├──<EMAIL_ADDRESS>    │ │ │ └──<EMAIL_ADDRESS>    │ │ └──<EMAIL_ADDRESS>    │ ├─┬<EMAIL_ADDRESS>    │ │ └──<EMAIL_ADDRESS>    │ ├─┬<EMAIL_ADDRESS>    │ │ └──<EMAIL_ADDRESS>    │ └─┬<EMAIL_ADDRESS>    │   ├──<EMAIL_ADDRESS>    │   └─┬<EMAIL_ADDRESS>    │     └──<EMAIL_ADDRESS>    ├─┬<EMAIL_ADDRESS>    │ └──<EMAIL_ADDRESS>    ├──<EMAIL_ADDRESS>    ├─┬<EMAIL_ADDRESS>    │ └──<EMAIL_ADDRESS>    ├──<EMAIL_ADDRESS>    ├──<EMAIL_ADDRESS>    ├─┬<EMAIL_ADDRESS>    │ └─┬<EMAIL_ADDRESS>    │   ├──<EMAIL_ADDRESS>    │   ├─┬<EMAIL_ADDRESS>    │   │ └──<EMAIL_ADDRESS>    │   ├─┬<EMAIL_ADDRESS>    │   │ └─┬<EMAIL_ADDRESS>    │   │   ├──<EMAIL_ADDRESS>    │   │   └──<EMAIL_ADDRESS>    │   ├──<EMAIL_ADDRESS>    │   └──<EMAIL_ADDRESS>    └─┬<EMAIL_ADDRESS>      └──<EMAIL_ADDRESS>

I have a similar issue, http://hortec-berlin.de hangs as does https://bet365.ee for example.
Using version 6.1.1

This seems to be related to https://github.com/puppeteer/puppeteer/issues/4072. I've added a timeout to these calls so that it doesn't hang in 892fe1ac147421c50625c5ad081fb98670c49a67.

Thank you for the quick response!
As the exception is thrown inside setTimeout() my calling code in a node script cannot catch it
Does it make sense to also call .error() prior to throwing?
Tested that this makes it possible for me to catch it by using on.error handler:
  timeout() {
    return new Promise(() =>
      setTimeout(() => {
        let error = new Error('The website took too long to respond')
        this.error(error)
        throw error
      }, this.options.maxWait)
    )
  }

Fixed in 4e8cf66677f361add360d2d7004c5532472e9763.


Draft up the "New Home" Email

This what we need: give away the site to a partner who can use it actively as a tool and continue to develop it. We are an active volunteer group and need to give it away like we always wanted...
This is what the site does
This is our background
This is what we can do for you (e.g. stay on with dev help for 6months)
Let's talk


"cloudshell" appended to each line on terminal

Hello,
sorry if its trivial, but whenever I start a terminal, each line gets prefixed with "cloudshell".
Screenshot:

Hi, this issue probably should have been opened on the vscode server/gcp cloud shell side.
Since I cannot delete the issue I'll close it.
Sorry for the inconvenience


Three countries not being shaded when selected under asserted distribution.

For the countries England, Scotland and Wales, the map is not showing these three countries shaded when they are asserted as the area of distribution. (Apologies for awkwardly worded sentence). England has a choice with shape, but the shape is incorrect. Scotland and Wales do not have choices with shape. Northern Ireland does have a shape and appears to work fine.

Followup - the events meeting today noticed a related (same?) issue.


Nix: refactor packages and overlays

The code that I added about the flake assumed that the overlay was deprecated and nobody would use it. But it turns out that more people that I thought are using it, and it is very poorly written.
This can be already merged, and will require a rebase in #227
Pinging @fufexan  @Tanish2002 @Narice for a second look

Looks good to me!


Sealing keys with PCRs

It would be nice if we could also seal the keys given a set of PCR values.
Even nicer would be support for FAPI policies which could be provided in a JSON file. But I currently see no FAPI support in go-tpm. So its probably quite difficult to implement this without tpm2-tss's Fapi_Import()?

Yes, but for that I'd like a Reseal API to function. So I need to learn how this works in Go, along with probably implementing missing apis in go-tpm.
See https://github.com/Foxboron/ssh-tpm-agent/issues/15


feat: Separated the Lint Dependencies for the Plugins

Description
This PR separates the lint dependencies for each plugin and allows users to install only the packages for required linting using the following command:
make install-lint-requirements plugin=kedro-<plugin>
This will avoid all of the bulky dependencies required for running the tests from also being installed when the need is only for linting.
Fixes https://github.com/kedro-org/kedro-plugins/issues/415
Development notes
These changes have been tested my manually running the above make command for each plugin.
Checklist

[X] Opened this PR as a 'Draft Pull Request' if it is work-in-progress
[ ] Updated the documentation to reflect the code changes
[ ] Added a description of this change in the relevant RELEASE.md file
[ ] Added tests to cover my changes
[ ] Received approvals from at least half of the TSC (required for adding a new, non-experimental dataset)

Hey @astrojuanlu,
I believe I've been able to separate the lint dependencies. Let me know if I have missed any packages that are required for linting and if there is any documentation that I will need to update.

Thank you @MinuraPunchihewa this is an elegant solution.

I appreciate that, @SajidAlamQB. Thank you.


2.4 installer and Ruby-Tk

Does not install Ruby-Tk or give an option to do so

tk was removed from ruby-2.4 stdlibs but is still available as a gem. Use:
ridk exec pacman -S mingw-w64-x86_64-tk
gem install tk

I just opened a pull request which will automatically issue the pacman command while gem install in the future.

The tk gem auto-installs the tk-package now: https://github.com/ruby/tk/pull/10#issuecomment-319263301


Move Kmers and Refseq to sequences/subfolder

Codecov Report

Merging #3 into api-redux will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           api-redux       #3   +/-   ##
==========================================
  Coverage      84.05%   84.05%
==========================================
  Files             44       44
  Lines           3048     3048
==========================================
  Hits            2562     2562
  Misses           486      486

Impacted Files
Coverage Δ

src/seq_types/mutablesequences/copying.jl
82.5% <ø> (ø)

...tablesequences/site_counting/count_sites_bitpar.jl
100% <ø> (ø)

src/seq_types/mutablesequences/iteration.jl
100% <ø> (ø)

...es/mutablesequences/site_counting/site_counting.jl
83.72% <ø> (ø)

src/seq_types/mutablesequences/transformations.jl
98.58% <ø> (ø)

src/seq_types/mutablesequences/mutableseq.jl
89.65% <ø> (ø)

src/seq_types/mutablesequences/conversion.jl
66.66% <ø> (ø)

src/seq_types/mutablesequences/indexing.jl
97.77% <ø> (ø)

src/BioSequences.jl
0% <ø> (ø)
:arrow_up:

src/seq_types/biosequences/biosequence.jl
84.52% <ø> (ø)

... and 10 more

Continue to review full report at Codecov.

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


turn on "require PRS up to date" branch control

What do ppl think about adding the "require branches be up to date before merging" branch protection rule to main? I think it helps avoid problems when there are lots of overlapping changes happening, that e.g. a merge that passed tests on its last commit could still break things if the base has changed since then?

Looks like we have quorum, I've added the rule.

I don't mean to be annoying but it looks like that's a different rule? this was the one i was thinking of:

I am also fine with requiring pull request before merging, and have found 'dismiss stale approvals' to be slightly annoying in combination with 'require branches be up to date' because it was pretty common with us to review something, approve it, then before merging do a final rebase and that triggers the need for a new review. maybe since this isn't a "security-conscious" package and we're not anticipating people trying to sneak through changes we disable that? but also no strong preference

I agree, especially with a globally distributed group of volunteers, let's not be too strict.

Updated!

hi!! I'm late to the convo here, but I totally trust all the decisions that you all make on this. so we could potentially close this issue or keep it opened as you all see fit!! i don't want to slow progress down!


Get rid of crack

I'm running into this issue
Short story: crack is a bad dependency to have. It's basically abandonware. The last official release (2015) uses the safe_yaml gem, which is also abandonware. Note that this has been fixed more than 2 years ago, but they haven't bothered to make a release with that bugfix.
Making releases has basically no cost. Not releasing versions after bugs are fixed is a mistake.
Note also that the same_yaml gem also fixed the issue more than a year ago. But hasn't released it.  There's just no excuse in not releasing versions after changes.

Yeah, makes sense.
It was added at the point of 0.0.1 (aka The Prototype) "just to play with", obviously hard dependency on abandonware is not an acceptable thing.
Probably I'll drop the xml: true feature completely for now.


Decompiler

Not really an issue but i cant ask it in other place. What decompiler should i use for this to be able to recompile it

Learn programming


Changed the build type to ament_python and added setup.cfg

Fixed package to run with  ros2 run rqt_console rqt_console. Related to this issue https://github.com/ros-visualization/rqt_console/issues/20
Signed-off-by: ahcorde<EMAIL_ADDRESS>

@jacobperron This package does not have any tests. Does make sense to include copyright, flake8 and pep256? I can do it in a follow up PR.
Running CI up-to rqt_console

Linux
Linux-aarch64
macOS
Windows

This package does not have any tests. Does make sense to include copyright, flake8 and pep256? I can do it in a follow up PR.

If you like, a follow-up PR sounds good.

can we merge this?

@jacobperron CI is green, can we merge this one?

Linux
Linux-aarch64
macOS
Windows

Linux
Linux-aarch64
macOS
Windows

Linux
Linux-aarch64
macOS
Windows


CO-2076_Existing_Message_Templates_Throws_Error_on_Notifications

Added Upgrade script

Is there a reason you need to unbind relations in _ug400? Is that just copy/paste from somewhere?
Also, this appears to include the updated text for the "Test Template" button. That should probably be a separate commit.

Is there a reason you need to unbind relations in _ug400? Is that just copy/paste from somewhere?
Also, this appears to include the updated text for the "Test Template" button. That should probably be a separate commit.

Regarding the unbinding of the relations i was wrong. According to the documentation By default, updateAll() will automatically join any belongsTo association for databases that support joins. To prevent this, temporarily unbind the associations.. So i fixed the code to unbind the belongsTo bindings instead of the hasMany.
I will make the Template Text a separate commit.

@boshrin i applied the changes you proposed.


App fails to compile once i add flutter_callkit_incoming package

This error happens by simply adding the package to the pubspec package
On flutter run command,
this is what I get

FAILURE: Build failed with an exception.

What went wrong:
Execution failed for task ':flutter_callkit_incoming:compileDebugKotlin'.

'compileDebugJavaWithJavac' task (current target is 1.8) and 'compileDebugKotlin' task (current target is 20) jvm target compatibility should be set to the same Java version.
Consider using JVM toolchain: https://kotl.in/gradle/jvm/toolchain

and i believe this originates from the package's build.gradles kotlin extension.
could it be because flutter no longer uses buildscript or whatever??
I'm on java 20 and have a gradle version of 8.3 which works well for other project
until I added flutter_callkit_incoming package

No help in sight 😭

Hey @danielAsaboro, just add Add in your gralde.properties:
kotlin.jvm.target.validation.mode = IGNORE
it resolve the error for me

kotlin.jvm.target.validation.mode = IGNORE

tq it's worked but is this affect our app on future?


Yolov4 concat falls back to host

Is your feature request related to a problem? Please describe.
The concat ops are falling back to host in yolov4 end to end implementation
Describe the solution you'd like
run all ops on device including the concats. The perf excel sheet is attached for reference.
Additional context
running on GS - e150.
command to run: pytest tests/ttnn/integration_tests/yolov4/test_ttnn_yolov4.py
ops_perf_results_yolov4End2End_2024_05_01_21_43_28.csv

In the current YoloV4 implementation with new Conv API (#10068), we don't have concat falling back to host.
Can we close this issue? @dvartaniansTT


added a consulting firm to the list

Please review our Contribution Guidelines to ensure your changes abides by it.
Review your changes and complete the checklist below.

[x] Company name
[x] Company description
[x] Company location
[x] GitHub link
[x] How Elixir is used
[x] Sorted alphabetically

Digital Natives

Thank you for adding us to the list!

Thank you @kbardos! 👍


Fix CSV/RAW output header being application/json rather than plain/text

Description
When making SQL calls with csv or raw outputs the formatting would be incorrect as the header sets the format to application/json rather than plain/text like the legacy engine did.
This PR adds a format getter to both response formatters to allow it to be set correctly.
Issues Resolved
https://github.com/opensearch-project/sql/issues/1572
Check List

[x] New functionality includes testing.

[x] All tests pass, including unit test, integration test and doctest

[x] New functionality has been documented.

[x] New functionality has javadoc added
[x] New functionality has user manual doc added

[x] Commits are signed per the DCO using --signoff

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

@matthewryanwells have we determined if this fixes nested and objects in the Dashboard too?

@acarbonetto I can confirm that this fixes the nested, objects, and any other issues with output as the problem was how csv and raw were being handled, rather than the specific thing being processed.


[Bug]: On changing the database in Admin settings, and saving and restarting, page does not refresh

Is there an existing issue for this?

[X] I have searched the existing issues

Description
When I change the db from a default to an external, or from one external to another external, the page sits there forever with the loader, and does not automatically refresh. When we manually refresh we see an unauthorized access error. We then have to logout and sign up to go to the correct home page.
The page should not sit there idly, and appropriate information should be provided to the user
Steps To Reproduce

Public Sample App
No response
Environment
Deploy Preview
Severity
Medium (Frustrating UX)
Issue video log
No response
Version
Cloud

@btsgh we were not able to reproduce this can we reverify again?

@btsgh Any updates on this issue?

Not able to reproduce this any more. Seeing the following screen on changing the DB. Hence closing the issue.


FFT does not use current stream context

Description
When using cupy.fft within a Stream context, the Stream is not respected, and some Host to Device transfers happen on the default Stream.
For example, in code below, two non-default streams (15, 16) are created, and one of these streams is immediately set as the context. However, when looking at the timeline using NVIDIA Nsight, multiple host to device transfers occur outside of these two streams. There should be no activity on Default Stream 7 in this example.

I believe this issue is related to FFTPlan creation because disabling the plan cache causes additional communications (more than 3) on this Default Stream.
To Reproduce
import typing

import numpy.typing as npt
import numpy as np
import cupy as cp
import cupyx

def test_stream_cupy_fft(dtype=np.double, num_streams=2, w=128, jobs=5):
    rng = np.random.default_rng()
    streams = [cp.cuda.Stream() for _ in range(num_streams)]
    streams[0].use()

    x0 = cupyx.empty_pinned(shape=(num_streams * jobs, 128, w, w), dtype=dtype)
    rng.random(size=x0.shape, out=x0)
    y0 = cp.empty((num_streams, 128, w, w), dtype=x0.dtype)

    for i in range(num_streams * jobs):
        s = i % num_streams
        with streams[s]:
            y0[s].set(x0[i])
            y0[s] = cp.fft.ifft2(cp.fft.fft2(y0[s])).real```

if __name__ == '__main__':
    test_stream_cupy_fft()

Installation
Conda-Forge (conda install ...)
Environment
OS                           : Linux-5.14.0-284.18.1.el9_2.x86_64-x86_64-with-glibc2.34
Python Version               : 3.11.6
CuPy Version                 : 12.2.0
CuPy Platform                : NVIDIA CUDA
NumPy Version                : 1.26.0
SciPy Version                : 1.11.3
Cython Build Version         : 0.29.36
Cython Runtime Version       : None
CUDA Root                    : /home/beams/DCHING/miniconda3/envs/tike-profile
nvcc PATH                    : None
CUDA Build Version           : 12000
CUDA Driver Version          : 12000
CUDA Runtime Version         : 12000
cuBLAS Version               : (available)
cuFFT Version                : 11000
cuRAND Version               : 10301
cuSOLVER Version             : (11, 4, 2)
cuSPARSE Version             : (available)
NVRTC Version                : (12, 2)
Thrust Version               : 200001
CUB Build Version            : 200001
Jitify Build Version         : 3e2f16d
cuDNN Build Version          : None
cuDNN Version                : None
NCCL Build Version           : None
NCCL Runtime Version         : None
cuTENSOR Version             : None
cuSPARSELt Build Version     : None
Device 0 Name                : NVIDIA GeForce RTX 2080 Ti
Device 0 Compute Capability  : 75
Device 0 PCI Bus ID          : 0000:1B:00.0
Device 1 Name                : NVIDIA GeForce RTX 2080 Ti
Device 1 Compute Capability  : 75
Device 1 PCI Bus ID          : 0000:1C:00.0
Device 2 Name                : NVIDIA GeForce RTX 2080 Ti
Device 2 Compute Capability  : 75
Device 2 PCI Bus ID          : 0000:1D:00.0
Device 3 Name                : NVIDIA GeForce RTX 2080 Ti
Device 3 Compute Capability  : 75
Device 3 PCI Bus ID          : 0000:1E:00.0
Device 4 Name                : NVIDIA GeForce RTX 2080 Ti
Device 4 Compute Capability  : 75
Device 4 PCI Bus ID          : 0000:3D:00.0
Device 5 Name                : NVIDIA GeForce RTX 2080 Ti
Device 5 Compute Capability  : 75
Device 5 PCI Bus ID          : 0000:3F:00.0
Device 6 Name                : NVIDIA GeForce RTX 2080 Ti
Device 6 Compute Capability  : 75
Device 6 PCI Bus ID          : 0000:40:00.0
Device 7 Name                : NVIDIA GeForce RTX 2080 Ti
Device 7 Compute Capability  : 75
Device 7 PCI Bus ID          : 0000:41:00.0```

Additional Information
No response

Another puzzle is you used pinned memory, but the H2D copies say "pageable". Something is weird here, though may not be relevant.

For clarity, yes, the actual FFT computation is occurring in the correct stream context. This issue is about the host to device transfers occurring on the default stream before the FFT computation. Because these transfers occur in the default stream, they may cause synchronization issues in a more complex program?

Update: Confirmed via nsys that it's due to cufftMakePlanMany.

If there's already an idea of how to mitigate this issue, and I can be provided with some guidance. I'd be happy to try developing a PR.

@leofang Should we call cufftSetStream before cufftMakePlanMany?
https://github.com/cupy/cupy/blob/8ddb02d9f75f22ba0b17b83545124aed4d11d2ec/cupy/cuda/cufft.pyx#L785-L789

No. To be clear, when I said it's due to cufftMakePlanMany, I was referring to libcufft's internal implementation.
I did already reach out to the cuFFT team, and without revealing too much my current advice is don't waste your time trying any workarounds, they won't work.
Let's keep this issue open, and if I have any update I will share.

(btw, sorry I forgot to say, thank you @carterbox for bringing this to our attention)

Let's keep this issue open, and if I have any update I will share.

Tracked in nvbugs 4573129. Since this is not a CuPy bug, let me close this issue.


Pre-Shakeup Updates

Fixes #71
Fixes #53
Fixes #68
Fixes #62
Fixes #57
Fixes #44

@EbbyEbby - the issues listed above are addressed in this update to production!

We did it!!!


az appservice list-locations produces incomplete output

Description
It looks like a filed (and fixed) bug with xplat is in CLI 2.0:
https://github.com/Azure/azure-xplat-cli/issues/2021
Check out the results from az appservice list-locations -o tsv to see what the current output is (it's the US-EN localized region names.) Doesn't look like there's another way to get a locations list.

Environment summary
Install Method: Via curl
CLI Version: azure-cli (2.0.8), appservice (0.1.8)
OS Version: bash on Windows 10
Shell Type: Bash on Windows, using zsh

@sptramer, App service is a bit unique that it uses the geo-region which works for both asm and arm( the site created by CLI is actually visible under asm mode). The region name should work for you to create the plan, but let me know otherwise.

The issue isn't creating the plan, but being able to get the internal identifiers for the regions at all. For example, if I wanted to find all locations in the US, I can guess that the naming scheme is of the form *us, but without a guarantee from an API point to get the identifiers, I don't know. There's also the possibility of *us slurping in domains/locations I may not want to batch-operate on, such as USG-compliant datacenters.

We should push the webapp team to fix it by providing "canonical name"

@LukaszStem can you please check this issue, and fix if needed

Currently we return the externally visible regions' names. doining * US and * US 2, should work to get all US regions. The USGov regions will not show up unless you connect your CLI to the USGov cloud, same applies for Chinese, German, and all other Azure sovereign clouds.
Closing this issue, please reopen if you still get an incomplete list


Movable Features 位移信标

uplicant goes to conduct a survey of the building and the beacon does not appear upon completion, and those beacons that were before the update just disappeared when activated.  There may be a conflict of mods, but everything worked before the update.
handyprint 14 minutes ago
After surveying no beacon appears
Unholyrage 23 minutes ago
Why then are there 4 skills left in the construction branch? I have a dup in my colony with the “Masterworks” skill and still nothing works. I'm probably missing something.
handyprint 34 minutes ago
Unholyrage.... It now requires the "Masterworks" skill

Mod description
The mod has added a building called "Displacement Beacon" which allows you to build and move already surveyed items. When moving the item, it will be covered with Neutronium.

Anyway, thank you for your feedback here.

哥哥，steam上面的版本要不要先回退一下？？？？ 周末想玩缺氧！！！！！ release版本找不到1.12版本，只有最新版本

https://github.com/ttdly/ONI-Mods/releases/tag/v1.1.3-pack
这是上个版本


[1.3] Scenes Management Cluster Implantation has to updated as per Spec Changes.

Summary:

Below mentioned Attributes Id's has to be updated as per Spec.

[ ] LastConfiguredBy(0x0000)
[ ] SceneTableSize(0x0001)
[ ] FabricSceneInfo(0x0002)

EX, TS and  FS features has been removed from Spec, So Feature map value has to be updated as per Spec.
Some  Attributes(NameSupport) are removed and Id's has been changed in Spec,  So Attribute Id's has to be updated as per Spec in Attribute List.
EnhancedAddScene and EnhancedViewScene commands has been removed from Spec, So It should be removed from AcceptedCommandList.
EnhancedAddSceneResponse and EnhancedViewSceneResponse commands has been removed from Spec, So It should be removed from GeneratedCommandList.
Below mentioned commands has to be removed from SDK as per Spec.

[ ] EnhancedAddScene
[ ] EnhancedViewScene
[ ] EnhancedAddSceneResponse
[ ] EnhancedViewSceneResponse

Bug prevalence
Whenever I do
GitHub hash of the SDK that was being used
09f014cebf114980e6174277d8d45f301f77679f
Platform
raspi
Platform Version(s)
1.3
Anything else?
Spec References:

https://github.com/CHIP-Specifications/connectedhomeip-spec/blob/master/src/app_clusters/Scenes.adoc#4-features

Test Plan References:

https://github.com/CHIP-Specifications/chip-test-plans/blob/master/src/cluster/scenes.adoc#411-tc-s-11-global-attributes-with-dut_server

A PR is opened in the SDK for those changes here https://github.com/project-chip/connectedhomeip/pull/31178

On the spec side, those changes were merged here: https://github.com/CHIP-Specifications/connectedhomeip-spec/pull/8668

Issue Fixed