Fixed bug in Array.map and Array.indexedMap

Array.map and Array.indexedMap have a bug that can most easily be seen
with the following use case:
EXPECTED: Array.empty == Array.map (\e -> e) Array.empty
ACTUAL: Array.empty != Array.map (\e -> e) Array.empty
(the same for Array.indexedMap)
This is because Array.map/Array.indexedMap initializes itself
incorrectly using:
    table: new Array(a.table)  // creates a new array with an array element

instead of
    table: new Array(a.table.length) // a correctly-sized array

This bug has been fixed in this commit

I just realized this bug has been reported in https://github.com/elm-lang/core/pull/169
However this is a better fix, especially because the bug exists in all Array.map calls
(not just the empty case). It is most visible in the empty case but the initialization is
incorrect in all cases.

Awesome, thanks for the fix!

Yay! Awesome!
By the way, I just ran elm-check on the old code and it has found the bug.
Map transforms lists and arrays equally has failed with the following input: (<function>,Array.fromList [])

I know it's a bit of a shameless plug, but I think we could look into elm-check integration with the current testing of the standard library. I'm pretty surprised by how well elm-check is currently doing.

Any chance of a 0.14.x patch release for this?

Prompted by you @pchiusano, I proposed it for next stable with #192.


Resuming training from checkpoint

Hi,
Thanks for this amazing repo and work!
Could you please guide me as to how to resume training from a checkpoint.

Hi, please try to add -c "checkpoint_pth" to the training command, e.g., NCCL_P2P_DISABLE=1 CUDA_VISIBLE_DEVICES="0,1,2,3" python -m torch.distributed.launch --nproc_per_node=4  --master_port 29502 train.py -p 29502 -d 0,1,2,3 -n "dataset_name" -c "checkpoint_pth" . Thanks.

Thanks, it works! Closing the issue


meta: remove bountysource plugin

Bountysource is enabled on these issues, which adds more noise than anything. We've never actually had any bounties on any issues. So it should be disabled.

Done


wrong branch in workflow

currently CI does not run on master as the branch name is wrong in the workflow
https://github.com/Mitsunee/modern-diacritics/blob/cd305b06cb2d3b44ea0dcb271f217314b4e3bee7/.github/workflows/test.yml#L4-L6
The current release (2.2.1) passed testing in the PR and no code changes were comitted since then, so this isn't an immediate issue.

Fixed in https://github.com/Mitsunee/modern-diacritics/commit/4f240376e5454561d848966784ca154968decb41


Show axis labels in 3-d viewers

Can we use vispy.scene.visuals.Text? (does it allow 2-d rotation?)

We could use the 'tick' parameter for Axis Visual instead of using Text Visual and setting the transform for it.
Here is an example for this:
https://github.com/vispy/vispy/blob/master/examples/basics/scene/surface_plot.py

@PennyQ - ah yes, good idea. Actually, now you mention it, it would be great to actually show ticks, tick labels, and axis labels (the latter being e.g. 'RA' or 'Mass').
Would you or @maxwelltsai like to work on this?

@astrofrog Yeah, I can do it. So do we still want to make it shown as a 'grid box' way or just a scaled 'xyz' axis? This is the Axis Visual so if we want a grid then we need to use two such visuals and place them to be a grid box I guess.

@PennyQ - I was thinking we could keep it a grid box, show the ticks on all axes but the labels only on 3 axes - what do you think?

@astrofrog I found the AxisVisual in Vispy doesn't support 3D now and I asked them about any progress on it, so far no replies. And the method they use is quite hard (from Matplotlib and other research). So do you know any easy methods that we could implement first to make the ticks/labels shown in Glue? Thanks!

@astrofrog @PennyQ Has there been any progress on this topic?

@macrocosme - @PennyQ started working on this in https://github.com/glue-viz/glue-vispy-viewers/pull/96 - will try and finish this up this week!

@macrocosme - this is now done! (in the developer version)


Correctly set LookupFlags for generated kerning lookups

ufo2ft kernFeatureWriter sets IgnoreMarks lookupflag by default on its kerning lookups, unless there are kerning pairs between glyphs classified respectively as bases and marks (or marks vs marks); the latter are split into separate lookups where IgnoreMarks lookupflag is not set.

Originally posted by @anthrotype in https://github.com/googlefonts/fontc/pull/517#discussion_r1374274796

added in #733


Espoo is currently failing with error

2020-10-04 13:45:06.412 (+03:00)        a4410309-9757-4b8f-badb-9831be0b6bbd    ERROR   Invoke Error    {"errorType":"TypeError","errorMessage":"Cannot read property 'next' of undefined","stack":["TypeError: Cannot read property 'next' of undefined","    at FuelScraper.parseStationRow (/var/task/src/fuelScraper.js:61:36)","    at Node. (/var/task/src/fuelScraper.js:46:38)","    at initialize.exports.each (/var/task/node_modules/cheerio/lib/api/traversing.js:300:24)","    at FuelScraper. (/var/task/src/fuelScraper.js:45:26)","    at Generator.next ()","    at fulfilled (/var/task/src/fuelScraper.js:5:58)","    at processTicksAndRejections (internal/process/task_queues.js:97:5)"]}

Tampere also is failing


Feature/isdb 96 backlink issue

Before creating a pull request make sure that:

[ ] commit messages are meaningful and follow good commit message guidelines
[ ] README and other documentation has been updated / added (if needed)
[ ] tests have been updated / new tests has been added (if needed)

Please remove this line and everything above and fill the following sections:
JIRA link (if applicable)
Change description
Does this PR introduce a breaking change? (check one with "x")
[ ] Yes
[ ] No

@cshiwarale Basically this file was deleted so it's showing in diff


⚠ Spryr has degraded performance

In e26f4f5, Spryr (https://spryr.com) experienced degraded performance:

HTTP code: 200
Response time: 3107 ms

Resolved: Spryr performance has improved in 687d713 after 7 minutes.


feat(accounts-controller): add getNextAvailableAccountName public method

Explanation
Make this function available since the logic might be tricky when we have gaps in the account list.
This can then be used by the UI if it needs to get the next account name available during account creating (from within the extension).
References

Changelog
@metamask/accounts-controller

CHANGED: Make getNextAvailableAccountNameFor method public

Checklist

[x] I've updated the test suite for new or updated code as appropriate
[x] I've updated documentation (JSDoc, Markdown, etc.) for new or updated code as appropriate
[x] I've highlighted breaking changes using the "BREAKING" category above as appropriate

@metamaskbot publish-preview


[PE-D][Tester B] Wrong error message

Wrong error message here it should say that this supplier already exists, also point to improve why cant I have one supplier supplying more than one types of food. Quite sure this is the norm in the real world

Labels: type.FunctionalityBug severity.High
original: adeearyaa/ped#5

Intended right ? @Bahamas20

Second part is intended will specify in user guide

The command input is also invalid, no space between ABC Pte Ltd and p/


Endpoint Routing: Find The Matched Route

@rynowak while upgrading from .net core 2.2 to 3.0 these lines of code are no longer working. Am I missing anything? Perhaps, the way we are doing template matching have surely changed to a pattern matching. Let me know your thoughts:
    public RouteEndpoint GetRouteEndpoint(string path)
        {

            var routeEndpoints = _endpointDataSource.Endpoints.Cast<RouteEndpoint>();

            var matchedEndpoint = routeEndpoints.Where(e => new TemplateMatcher(
                                                                        TemplateParser.Parse(e.RoutePattern.RawText),
                                                                        new RouteValueDictionary())
                                                                .TryMatch(path, new RouteValueDictionary()))
                                                .OrderBy(c => c.Order)
                                                .FirstOrDefault();
            return matchedEndpoint;
        }

@mgolois thanks for contacting us.
@rynowak thoughts?

Looks like a dupe of https://github.com/aspnet/AspNetCore/issues/17391


fix: cap long prefix paths

This change ensures that riot does not generate prefix paths that are too long and might go beyond the limits imposed by the FS.
Fixes #198.

Is it necessary to drop 3.6 support here?

Is it necessary to drop 3.6 support here?

Not here, I can make another PR for that, but CI couldn't pull 3.6 anymore and I thought it didn't make sense to invest time in trying to fix it for such an old version.

What's FS?

macos-latest 3.6 and ubuntu-latest 3.6 are required actions. Can we remove this check?

What's FS?

File System 🙂

macos-latest 3.6 and ubuntu-latest 3.6 are required actions. Should we remove this check to unblock this PR?

Yes, I think we should. But I would like to drop support for 3.6 in another PR first. I'll try to open it today 🤞


Research Data Generation

Research of methods to generate realistic data sets used in application.

Reasearched what time of data is relevant to the application and how that data has to be presented.


Token issue?

Hello,
I'm using the version 2021.3.4 with Hacs. I'm able to use the integration and control my robot during 1 day. Then I have to restart home assistant to be able to control again the robot. Is it a token problem? I have no issue in logs.
Thank you.

Hello,
that sounds relaly like a token isssue, because the token is valid for 24h. There was change in 2021.3.3 which refreshs the token instead of aksing for a new token every time the token expires. But in my tests I had no issuses.
Can you add the debug logging to your configuration.yaml and have a look what happens 24h after restart?
logger:
  default: info
  logs:
    custom_components.husqvarna_automower: debug
    custom_components.husqvarna_automower.vacuum: debug
    custom_components.husqvarna_automower.config_flow: debug
    aioautomower: debug

I will also release a new version soon, which completely changes the token managment. It would be good, if I could fix your issue before releasing it.

The expected behaviour should look like that:
2021-04-01 02:03:28 DEBUG (MainThread) [custom_components.husqvarna_automower] Updating mower data
2021-04-01 02:03:28 DEBUG (MainThread) [custom_components.husqvarna_automower] Getting new token, because expired
2021-04-01 02:03:28 DEBUG (MainThread) [aioautomower] Resp.status: 200
2021-04-01 02:03:28 DEBUG (MainThread) [custom_components.husqvarna_automower] Token expires at<PHONE_NUMBER> UTC

I restarted Home Assistant with debug log, but I have no issue anymore :p I think sometimes the action can take 10-15 min if the robot is in sleep mode. But no issue with token so you can close the ticket.
Thank you for your work.


Update Composer dependencies (2020-11-14-00-12)

Visual regression test passed!
View the visual regression test report


More on handling Postgres environmental variables in settings.py

There are two environment issues that need to be fixed.
Issue 1.
Here is the DATABASES environment variable inside my settings.py (lines  92-108):
SELECTED_DB = ""

VALUE = os.getenv('SELECT_DB')

if VALUE == "0":
   SELECTED_DB = "HEROKU_POSTGRESQL_SILVER_URL"
elif VALUE == "1":
   SELECTED_DB = "HEROKU_POSTGRESQL_NAVY_URL"

DATABASES = {
   'default': dj_database_url.config(
       env=SELECTED_DB,
       default='sqlite:///'+os.path.join(BASE_DIR, 'db.sqlite3'),
       conn_max_age=600)
   }

print(DATABASES)

'SELECT_DB' is an environmental variable (dictionary) declared inside Heroku’s dashboard which is paired with the value specified as an integer, either 1 or 0. On the next line, if the VALUE is 0, then dj_database_url.configures the db to be HEROKU_POSTGRESQL_SILVER_URL. If the value is 1, then dj_database_url.configures the db to be, HEROKU_POSTGRESQL_NAVY_URL. These are references to Postgres URIs which are also formed as configuration variables showing inside the Heroku dashboard. The first “SILVER” db contains Lorem Ipsum placeholder content and the second “NAVY” db contains meaningful blog content posted by the end user. This is my general understanding of what is going on here.
The problem with all of the above is that Heroku Postgres supports flexible and dynamic on-the-fly provisioning for creating, spinning up, spinning down, swapping around different apps, backing up, and restoring db silos.  As you can read at that heading in the Heroku guide it says that to initialize (create) a fresh db, in the Heroku shell, you run: (local venv) $ heroku addons:create heroku-postgresql:hobby-dev
To view my current Postgres db addon composition and structure, you can use this command:
(local venv)  $ heroku pg:info
 ›   Warning: heroku update available from 7.47.11 to 7.47.12.
=== HEROKU_POSTGRESQL_NAVY_URL, DATABASE_URL
Plan:                  Hobby-dev
Status:                Available
Connections:           0/20
PG Version:            12.5
Created:               2021-01-09 20:43 UTC
Data Size:             10.0 MB
Tables:                21
Rows:                  110/10000 (In compliance)
Fork/Follow:           Unsupported
Rollback:              Unsupported
Continuous Protection: Off
Add-on:                postgresql-real-1

=== HEROKU_POSTGRESQL_ROSE_URL
Plan:                  Hobby-dev
Status:                Available
Connections:           0/20
PG Version:            13.2
Created:               2021-02-26 17:21 UTC
Data Size:             7.9 MB
Tables:                0
Rows:                  0/10000 (In compliance) - refreshing
Fork/Follow:           Unsupported
Rollback:              Unsupported
Continuous Protection: Off
Add-on:                postgresql-adjacent-58092

=== HEROKU_POSTGRESQL_SILVER_URL
Plan:                  Hobby-dev
Status:                Available
Connections:           0/20
PG Version:            12.6
Created:               2020-11-19 10:32 UTC
Data Size:             9.6 MB
Tables:                21
Rows:                  84/10000 (In compliance)
Fork/Follow:           Unsupported
Rollback:              Unsupported
Continuous Protection: Off
Add-on:                postgresql-lorem-1

To designate a primary database, you can use this command:
(local venv) $ heroku pg:promote HEROKU_POSTGRESQL_SILVER_URL (or whichever other db name is available as it appears in the list above). Here in the command line is where  I should swap out / switch between databases and not statically coded inside settings.py (like it currently is in my first snippet above).
With the newly created ROSE db instance, the settings.py is out of date. I would need to add another elif condition which specifies a SELECTED_DB (as described above) as "HEROKU_POSTGRESQL_ROSE_URL" with a value of 2, and then add that config variables inside the Heroku dashboard manually. That’s not ideal, right? The db environmental variable should allow the Heroku sysadmin to change between db’s using the (local venv) $ heroku pg:promote <db name>
Issue 2:
I should be able to import (access) remote aws Postgres db’s locally by specifying the remote URL in settings.py in this format: postgres://USER:PASSWORD@HOST:PORT/NAME. This feature is described in John Elder’s Udemy course specified here.
This feature was actually already implemented in PR #79 but was lost and removed in subsequent PR’s (#80. #81, #82). This needs to be re-implemented. I tried implementing it myself just now by removing the VALUE/SELECTED switches and replacing them with this:
DATABASE_URL = "" # since this is the default handle as you can see above in the output for the command: $ heroku pg:info

DATABASES = {
   'default': dj_database_url.config(
       env=DATABASE_URL,
       default='sqlite:///'+os.path.join(BASE_DIR, 'db.sqlite3'),
       conn_max_age=600)
   }

But this throws an OperationalError.
To summarize, here is the takeaway:
The DATABASES environmental variable needs to be reconfigured inside settings.py to essentially enable:
1. the Heroku sysadmin to swap db’s using the $ heroku pg:promote command rather than updating settings.py every time a new db is spinned up/down.
2. the web developer to reference the remote db locally

Hi @UmarGit! The OperationalError at /portal is resolved in PR #85 + #86. Django is deployed and this traceback is gone. Thank you for this. However the other component to my issue is how to refer to (by exporting) the aws ec2 instance locally. For example, when running:

(local venv) $ export DATABASE_URL='postgres://jckmeibmxypuvn:11033b956dd9cc6011e8efb7d23d56bb277bef7e9634a31ae0f0b304f34e9667@ec2-3-231-241-17.compute-1.amazonaws.com:5432/d9eds0cfpna1li'

...my local Django environment is still serving db.sqlite3.  I even tried exiting my local dev server, invoking  $ unset DATABASE_URL and then re-invoking the export command (above) and re-running the server.  My local db remains sqlite.
How do I properly export this DATABASE_URL locally?

DATABASE_URL

Brother are you using windows ?

DATABASE_URL

Brother are you using windows ?

I run Manjaro Linux locally

Hey @enoren5 , brother you can run this command to check if DATABASE_URL, properly setup.
echo $DATABASE_URL

Here it is:

(local venv) $ echo $DATABASE_URL
postgres://jckmeibmxypuvn:11033b956dd9cc6011e8efb7d23d56bb277bef7e9634a31ae0f0b304f34e9667@ec2-3-231-241-17.compute-1.amazonaws.com:5432/d9eds0cfpna1li

I even tried with other db instances like this:

(local venv) $ export DATABASE_URL='postgres://xkdbsiijoknylq:c9f9d6cdb11bb69b34ba65ce4bb845cf1d14f905f1aba539d5ecd6515f1da19d@ec2-54-84-98-18.compute-1.amazonaws.com:5432/dcjnf0h34cfsc5'
(local venv) $ echo $DATABASE_URL
postgres://xkdbsiijoknylq:c9f9d6cdb11bb69b34ba65ce4bb845cf1d14f905f1aba539d5ecd6515f1da19d@ec2-54-84-98-18.compute-1.amazonaws.com:5432/dcjnf0h34cfsc5

...yet the local db remains db.sqlite3, even after refreshing and trying to navigate other pages

I can confirm that this issue has been resolved in PR's #85 and #86. Based on @UmarGit's comments, I have consolidated them and expanded upon them in the README.md at this bookmark here. The README.md further explains how to switch remote postgres data from the cloud to appear locally.
Thank you Umar for your hard work.


Documentation around no-internal-hashes & automated GitHub Pages deployments

I tried an experiment around using no-internal-hashes for a GitHub hosted pack, whilst also generating the hashes and hosting via GitHub Pages on successful builds via Actions.
https://github.com/PepsiDogs/pepsipack/blob/main/.github/workflows/packwiz.yml
It might be nice to draft up some documentation around this process, along with a skeleton repo that can be used as a blueprint or starting point.

I've moved this to the website repo, since this is where the majority of documentation is :)
I do already have an example modpack at https://github.com/packwiz/packwiz-example-pack, though it doesn't use no-internal-hashes. We could add a no-internal-hashes variant on a branch?


Releasing this to PiPy

Hi Michael; thanks for creating this. Is there a checklist you may have around to understand what would be needed to release this package into the wild? even if it is a beta version for the time being?
Also, would you need help with that? I am happy to help.

Hi @fferegrino Published this to PyPI: https://pypi.org/project/fastapi-chameleon/ :)


Add an event callback to enable cleaning up the private data on redisAsyncContext

This change adds a cleanup event to the redisAsyncContext such that the private data can be deleted or modified as a result of the context being free'd. This way it is possible to allocate private data on the context whose lifetime is tied to that of the context.
This is essentially the only meaningful change to hiredis made by hiredis-vip to support cluster. Adding the data cleanup event to hiredis proper allows for hiredis-vip to be a pure wrapper class that depends on hiredis directly. This way hiredis-vip won't continually lag behind.

I don't have a problem with this.
@mnunberg See any downsides here?
Also you can disregard the failing Windows test, that's not related to your PR.

Out of curiosity, @michael-grunder or @mnunberg, are there larger guidelines or suggestions to contributing to hiredis other than blind submitting PRs? I would have liked to have discussed whether this change made sense beforehand but couldn't find the best place to do that.

There aren't really any explicit guidelines.  Larger scale changes often come from requirements in Redis (e.g. RESP3 support), but other than that the library is pretty low velocity.
As for other changes, I tend to try and balance the benefits with how much pain it's going to cause when backported into Redis.
That said, you're welcome to open issues to discuss changes.  I'm generally around to respond.

Merged, thanks!


Compatible with ue5

It is compatible with UE 5.1.

Parts of the incompatibiliy owes to the introduction of 16-bit skin weights in UE5.3.
Details could be found at UnrealEngine @ e2210cb. This afftects the import behavior from FSkinWeightInfo to FSkinWeightVertexBuffer, and the latter now requires a uint16 encoded weight, with the range 0 - 65535 instead of 0 - 255. The import code would throw away the lower 8 bits of the uint16 encoded weight if the actual storage type is 8 bit, which is exactly the case when running the plugin unmodified for UE5.3.

@libreliu thanks for these info. Feel free to open a PR if you have this working already.

Parts of the incompatibiliy owes to the introduction of 16-bit skin weights in UE5.3. Details could be found at UnrealEngine @ e2210cb. This afftects the import behavior from FSkinWeightInfo to FSkinWeightVertexBuffer, and the latter now requires a uint16 encoded weight, with the range 0 - 65535 instead of 0 - 255. The import code would throw away the lower 8 bits of the uint16 encoded weight if the actual storage type is 8 bit, which is exactly the case when running the plugin unmodified for UE5.3.

Thanks for the suggestion, changes have been submitted.

Thanks!

Thanks for the merger, but I haven't tested it in detail yet, I hope someone can help test it, and I will fix it.

Another bug discovered: Should add Surface.MaterialIndex = RenderSection.MaterialIndex; in DecomposeSkeletalMesh to initialize the Material, otherwise this may init to some garbage value and hence break the FMeshSurface data.
(Will file a PR if I have time..!)


Only check staged files

Only check staged files when trying to commit

Not sure if this should be in scope of this library but I've created https://github.com/okonet/lint-staged for this. Check it out and let me know what you think.

Great, this is exactly what I was looking for; I didn't realize you had another repo for this. Thanks for your quick and diligent response on this!

Just FYI I'm not the maintainer nor developer of this package... just stumbled upon this issue.

Thank you I overlooked your username originally.

I'm going to re-open and wait and see what the owner has to say.

I agree that this should not be part of this library, at least as far as I understand it.
But I would definitely get a property called onFailure (or something close) or preferably finally, so we can give a npm script name which would be run respectively when an error occurs or always, at the end of the pre-commit.
This would help us start with a script which stash everything but staged files, and unstash those files once scripts have run, whatever the result.

Actually that would be awesome! Stashing and unstashing is something I'd like to integrate into 🚫💩 lint-staged but it's not trivial at the moment.

I'm not sure though it can be done as part of the pre-commmit hook.

I've created a script that does exactly this. It uses pre-commit to start a node script that programmatically goes over all the staged files.
https://gist.github.com/bertyhell/c8ae817626e053375bad0d62efc1da7d
Improvements are welcome.


ENH: Traces dock widget

This PR follows https://github.com/mne-tools/mne-python/pull/8997#issuecomment-802891396 and brings support for docked traces:
This is how it looks by default for me on i3:

And the dock widgets can be detached/resized/reattached:

Locally, Brain testing works but I don't know how macOS CIs will react :pray:

import os
import mne
from mne.datasets import sample

data_path = sample.data_path()
sample_dir = os.path.join(data_path, 'MEG', 'sample')
subjects_dir = os.path.join(data_path, 'subjects')
fname_stc = os.path.join(sample_dir, 'sample_audvis-meg')
stc = mne.read_source_estimate(fname_stc, subject='sample')
initial_time = 0.096
brain = stc.plot(subjects_dir=subjects_dir, initial_time=initial_time,
                 clim=dict(kind='value', lims=[3, 6, 9]),
                 surface="white",
                 size=600,
                 background="white")

Maybe show_traces="separate" is not necessary anymore and should be removed.

I think the result is better when the margins are disabled for the traces widget :)

margin=True
margin=False

It's ready for reviews @larsoner, @agramfort

From a quick test this is my experience / thoughts:

This will be a great enhancement!

The matplotlib widget has a minimum width that is too big -- I should be able to resize it smaller than this:
https://user-images.githubusercontent.com/2365790/111998156-1d1b9e00-8af2-11eb-8eab-144dc0075e61.mp4

The resize-boundary-grabber (what you can click on to actually resize the widget) once it has been removed appears to be 1 pixel. Maybe this is related to the removal of some border or padding (?) but it makes it very difficult to resize

Removing the control and matplotlib widgets is easy because they have titles. The "brain plots widget" should also have a title so it can just as easily be moved

2) should be fixed after aa23887828ca6a70d64c697d9306897196c386d8, I modified _window_ensure_minimum_sizes so that the minimum size for the mplcanvas is more flexible. I did not manage to make it work without the margins but maybe it will help with 3) in the end.
About 4), I still have to find a way to do that :)
Locally, the notebook test was failing so I fixed it in here. I don't know why it was not detected.
E   ~/source/mne-python/mne/viz/_brain/_brain.py in _configure_tool_bar(self)
E      1232     def _configure_tool_bar(self):
E      1233         self._renderer._tool_bar_load_icons()
E   -> 1234         self._renderer._tool_bar_set_theme(self.theme)
E      1235         self._renderer._tool_bar_initialize()
E      1236         self._renderer._tool_bar_add_screenshot_button(
E
E   AttributeError: '_Renderer' object has no attribute '_tool_bar_set_theme'

About 4), I still have to find a way to do that :)

Could you embed the plotter in a group box, and make that group box what's movable?

I did not manage to make it work without the margins but maybe it will help with 3) in the end.

Still 1 pixel for me, can you replicate? If so, could you try the margin stuff suggested here:
https://stackoverflow.com/questions/60279502/floating-qdockwidget-resize-change-size-of-the-resize-handler-grip-on-the-bord
If not, I can work on it, since it might actually be the fault of my Ubuntu theme:
https://forum.qt.io/topic/60122/increase-floating-qdockwidget-resize-margin-size

Could you embed the plotter in a group box

I know how to do this

make that group box what's movable?

But not that. I will make some research :sweat_smile:

Still 1 pixel for me, can you replicate?

With i3, I resize with the keyboard. I should switch to my KDE to replicate :+1:

But not that. I will make some research ￼

I'd assume that however you're currently making the matplotlib window movable, you "just" need to do that same thing with a new groupbox widget, then make the plotter a child of that group box. Hopefully it's not more complicated than this, but if it is, I apologize :) If it ends up being really complicated maybe it's not worth pursuing.

however you're currently making the matplotlib window movable

Ah this came basically for free with QDockWidget, it's part of the default features:
https://doc.qt.io/qt-5/qdockwidget.html#DockWidgetFeature-enum

So add a new GroupBox to that widget, put the plotter in that group box, and maybe it works?

And the dock widgets can be detached/resized/reattached:

Haven't tried this out yet, but hopefully one cannot "accidentally" detach the docks? Because that's something I hate :D These multi-window designs are … idk, is anyone even still using that? Not even GIMP comes with that stuff anymore by default! 😅

From trying it out, I find it works pretty well. You can move/detach/reattach easily.

I tried in 8d836cd but it was some sort of frankenstein, it's way too hacky and not stable in my opinion :sweat_smile:
https://user-images.githubusercontent.com/18143289/112320591-6fee8480-8caf-11eb-960b-ee071bcc0b31.mp4

Those interactions seem okay. Why not make the QDockWidget the central widget?

Ohhhhh I see. You need a central widget and things go around it. I can live with the brain plots being this central widget :)

In that case we should be good to go here, then?

Ready on my end. Do you want to take care of 3) in a different PR maybe?

Fixed the grip issue

Thanks @GuillaumeFavelier !


Performance drop for certain Y-strides in 2D and 3D real to hermitian interleaved transforms

Hi,
I observed a significance performance drop for certain Y-strides in 2D and 3D real to hermitian interleaved transforms. I'm using the clFFT-client application for testing.
This is my device:
OpenCL platform [ 0 ]:
    CL_PLATFORM_PROFILE:     FULL_PROFILE
    CL_PLATFORM_VERSION:     OpenCL 2.0 AMD-APP (2004.6)
    CL_PLATFORM_NAME:        AMD Accelerated Parallel Processing
    CL_PLATFORM_VENDOR:      Advanced Micro Devices, Inc.
    CL_PLATFORM_EXTENSIONS:  cl_khr_icd cl_khr_d3d10_sharing cl_khr_d3d11_sharin
g cl_khr_dx9_media_sharing cl_amd_event_callback cl_amd_offline_devices

OpenCL platform [ 0 ], device [ 0 ]:
    CL_DEVICE_NAME:                      Hawaii
    CL_DEVICE_VERSION:                   OpenCL 2.0 AMD-APP (2004.6)
    CL_DRIVER_VERSION:                   2004.6 (VM)
    CL_DEVICE_TYPE:                      GPU
    CL_DEVICE_MAX_CLOCK_FREQUENCY:       1025
    CL_DEVICE_ADDRESS_BITS:              64
    CL_DEVICE_AVAILABLE:                 TRUE
    CL_DEVICE_COMPILER_AVAILABLE:        TRUE
    CL_DEVICE_OPENCL_C_VERSION:          OpenCL C 2.0
    CL_DEVICE_MAX_WORK_GROUP_SIZE:       256
    CL_DEVICE_MAX_WORK_ITEM_DIMENSIONS:  3
                         Dimension[ 0 ]  256
                         Dimension[ 1 ]  256
                         Dimension[ 2 ]  256
    CL_DEVICE_HOST_UNIFIED_MEMORY:       FALSE
    CL_DEVICE_MAX_CONSTANT_BUFFER_SIZE:  65536 ( 64 KB )
    CL_DEVICE_LOCAL_MEM_SIZE:            32768 ( 32 KB )
    CL_DEVICE_GLOBAL_MEM_SIZE:<PHONE_NUMBER> ( 8192 MB )
    CL_DEVICE_MAX_MEM_ALLOC_SIZE:<PHONE_NUMBER> ( 4048 MB )
    CL_DEVICE_EXTENSIONS:                cl_khr_fp64 cl_amd_fp64 cl_khr_global_i
nt32_base_atomics cl_khr_global_int32_extended_atomics cl_khr_local_int32_base_a
tomics cl_khr_local_int32_extended_atomics cl_khr_int64_base_atomics cl_khr_int6
4_extended_atomics cl_khr_3d_image_writes cl_khr_byte_addressable_store cl_khr_g
l_sharing cl_khr_gl_depth_images cl_amd_device_attribute_query cl_amd_vec3 cl_am
d_printf cl_amd_media_ops cl_amd_media_ops2 cl_amd_popcnt cl_khr_d3d10_sharing c
l_khr_d3d11_sharing cl_khr_dx9_media_sharing cl_khr_image2d_from_buffer cl_khr_s
pir cl_khr_subgroups cl_khr_gl_event cl_khr_depth_images cl_khr_mipmap_image cl_
khr_mipmap_image_writes

When I set the Y-stride to 542 (which is also the stride that is chosen automatically if I don't specify any) I get this performance:
clFFT-client -x 540 -y 540 -z 8 -p 100 --inLayout 5 --outLayou
t 3 --isY 542 --osY 271 -g

Execution wall time: 1.54014 ms
Execution gflops: 80.102

========================StdDev ( 2 )========================
clFFT[ 0 ]: Pruning 1 samples out of 100
clFFT[ 1 ]: Pruning 1 samples out of 100
clFFT[ 2 ]: Pruning 1 samples out of 100
clFFT[ 3 ]: Pruning 1 samples out of 100
clFFT[ 4 ]: Pruning 0 samples out of 100
clFFT[ 5 ]: Pruning 1 samples out of 100
clFFT[ 6 ]: Pruning 2 samples out of 100
clFFT[ 7 ]: Pruning 1 samples out of 100
clFFT[ 8 ]: Pruning 0 samples out of 100

===========================clFFT============================
        Handle:                   3
        Kernel:<PHONE_NUMBER>FF6260
     OutEvents:<PHONE_NUMBER>3AF4A0
        Length:         (540,540,8)
     Placeness:             InPlace
    Input Dist:             2341440
   Output Dist:             1170720
  Input Stride:      (1,542,292680)
 Output Stride:      (1,271,146340)
   Global Work:             (38880)
    Local Work:                (54)
        Gflops:                        454.02
     Time (ns):                                      233,188

        Handle:                   4
        Kernel:<PHONE_NUMBER>FF6800
     OutEvents:<PHONE_NUMBER>3AF320
        Length:         (271,540,8)
     Placeness:          OutOfPlace
    Input Dist:             1170720
   Output Dist:             1170720
  Input Stride:      (1,271,146340)
 Output Stride:      (1,540,146340)
   Global Work:           (144,640)
    Local Work:             (16,16)
        Gflops:                       976.024
     Time (ns):                                      102,909

        Handle:                   5
        Kernel:<PHONE_NUMBER>FF5FC0
     OutEvents:<PHONE_NUMBER>3B03A0
        Length:         (540,271,8)
     Placeness:             InPlace
    Input Dist:             1170720
   Output Dist:             1170720
  Input Stride:      (1,540,146340)
 Output Stride:      (1,540,146340)
   Global Work:             (39042)
    Local Work:                (54)
        Gflops:                       273.336
     Time (ns):                                      194,383

        Handle:                   6
        Kernel:<PHONE_NUMBER>63E9B0
     OutEvents:<PHONE_NUMBER>3AFC20
        Length:         (540,271,8)
     Placeness:          OutOfPlace
    Input Dist:             1170720
   Output Dist:             1170720
  Input Stride:      (1,540,146340)
 Output Stride:      (1,271,146340)
   Global Work:           (144,640)
    Local Work:             (16,16)
        Gflops:                       1006.24
     Time (ns):                                       99,818

        Handle:                   2
 Child Handles:           (3,4,5,6)
        Length:         (540,540,8)
     Placeness:             InPlace
    Input Dist:             2341440
   Output Dist:             1170720
  Input Stride:      (1,542,292680)
 Output Stride:      (1,271,146340)
        Gflops:                       335.942
     Time (ns):                                      630,299

        Handle:                   7
        Kernel:<PHONE_NUMBER>1D42F0
     OutEvents:<PHONE_NUMBER>3AF7A0
        Length:          (146340,8)
     Placeness:          OutOfPlace
    Input Dist:             1170720
   Output Dist:             1170720
  Input Stride:          (1,146340)
 Output Stride:               (1,8)
   Global Work:          (16,36592)
    Local Work:             (16,16)
        Gflops:                       290.197
     Time (ns):                                      406,629

        Handle:                   8
        Kernel:<PHONE_NUMBER>1D3750
     OutEvents:<PHONE_NUMBER>3AFAA0
        Length:          (8,146340)
     Placeness:             InPlace
    Input Dist:             1170720
   Output Dist:             1170720
  Input Stride:               (1,8)
 Output Stride:               (1,8)
   Global Work:            (292736)
    Local Work:                (64)
        Gflops:                       278.934
     Time (ns):                                       62,956

        Handle:                   9
        Kernel:<PHONE_NUMBER>1D34B0
     OutEvents:<PHONE_NUMBER>3AEEA0
        Length:          (8,146340)
     Placeness:          OutOfPlace
    Input Dist:             1170720
   Output Dist:             1170720
  Input Stride:               (1,8)
 Output Stride:          (1,146340)
   Global Work:          (16,36592)
    Local Work:             (16,16)
        Gflops:                       282.026
     Time (ns):                                      418,409

        Handle:                   1
 Child Handles:           (2,7,9,8)
        Length:         (540,540,8)
     Placeness:             InPlace
    Input Dist:             2341440
   Output Dist:             1170720
  Input Stride:      (1,542,292680)
 Output Stride:      (1,271,146340)
        Gflops:                       162.509
     Time (ns):                                    1,518,295

When I change the stride to 544 I get this performance (3x faster):
clFFT-client -x 540 -y 540 -z 8 -p 100 --inLayout 5 --outLayou
t 3 --isY 544 --osY 272 -g

Execution wall time: 0.479108 ms
Execution gflops: 257.495

========================StdDev ( 2 )========================
clFFT[ 0 ]: Pruning 5 samples out of 100
clFFT[ 1 ]: Pruning 5 samples out of 100
clFFT[ 2 ]: Pruning 0 samples out of 100
clFFT[ 3 ]: Pruning 1 samples out of 100
clFFT[ 4 ]: Pruning 0 samples out of 100

===========================clFFT============================
        Handle:                   3
        Kernel:<PHONE_NUMBER>206260
     OutEvents:<PHONE_NUMBER>380890
        Length:         (540,540,8)
     Placeness:             InPlace
    Input Dist:             2350080
   Output Dist:             1175040
  Input Stride:      (1,544,293760)
 Output Stride:      (1,272,146880)
   Global Work:             (38880)
    Local Work:                (54)
        Gflops:                       569.125
     Time (ns):                                      186,025

        Handle:                   4
        Kernel:<PHONE_NUMBER>206DA0
     OutEvents:<PHONE_NUMBER>381310
        Length:         (540,271,8)
     Placeness:             InPlace
    Input Dist:             1175040
   Output Dist:             1175040
  Input Stride:      (272,1,146880)
 Output Stride:      (272,1,146880)
   Global Work:             (39042)
    Local Work:                (54)
        Gflops:                       266.412
     Time (ns):                                      199,435

        Handle:                   2
 Child Handles:               (3,4)
        Length:         (540,540,8)
     Placeness:             InPlace
    Input Dist:             2350080
   Output Dist:             1175040
  Input Stride:      (1,544,293760)
 Output Stride:      (1,272,146880)
        Gflops:                       549.326
     Time (ns):                                      385,461

        Handle:                   5
        Kernel:<PHONE_NUMBER>2069E0
     OutEvents:<PHONE_NUMBER>380590
        Length:         (8,271,540)
     Placeness:             InPlace
    Input Dist:             1175040
   Output Dist:             1175040
  Input Stride:      (146880,1,272)
 Output Stride:      (146880,1,272)
   Global Work:            (292736)
    Local Work:                (64)
        Gflops:                        225.79
     Time (ns):                                       77,774

        Handle:                   1
 Child Handles:               (2,5)
        Length:         (540,540,8)
     Placeness:             InPlace
    Input Dist:             2350080
   Output Dist:             1175040
  Input Stride:      (1,544,293760)
 Output Stride:      (1,272,146880)
        Gflops:                       532.636
     Time (ns):                                      463,235

I tested this on both 2.8.0 and 2.10.1 - same behaviour. If I drop the Z-dimension similar thing happens. Furthermore this happens also for other sizes than 540, provided that the Y-stride = FFT_X + 2. For power of 2 FFT sizes the slowdown is about 2x.
Also - it is not a particular stride that triggers this behaviour (so it's probably not due to memory coalescing?). It can't be channel conflicts either, because those happen for power of two strides. If I use the same strides as above for 500 x 500 x 8 transform I get good performance, but if I now change the stride to 502 I again get the 3x slowdown.
If i use the -d option to dump the kernels I can see that in the second case there are no transposition kernels generated and overall the profile info shows fewer handles in the second case, so this is probably the reason. Does anybody know why? Is there some relationship between the FFT size and its strides that results in optimal performance? I'd like to understand it, so I can choose the optimal stride for my application.
Thanks,
Michal

Hi Michal,
Thanks for using our library. We have not done full performance characterization for all the combinations of transform sizes and strides. The presence of transposes helps in many cases and does not in others; obviously in your use case the transpose kernels are being used but is negatively affecting performance. If you can build the library from source, then I can suggest changes in the code to turn off transposes. It is always best to use strides and distances that resemble a 'packed' format.
In master branch, in file: src/library/plan.cpp; line 2496 (for 2D forward real transform) and line 2791 (for 2D inverse real transform) there are if statements. If you change the code so that they evaluate to false, you will disable transposes. Similarly for 3D real transforms, you can change lines 3402 and 3701 the same way.

Closing this for now.


Profile operations

This PR adds support for the various profile-related methods, data types, and helper functions in Blockstack.js. Specific methods:

extractProfile
wrapProfileToken
signProfileToken
verifyProfileToken
lookupProfile

All functions also have Objective-C bindings, found in Blockstack+ObjC.swift.
Thanks!
Shreyas

@larrysalibra .DS_Store files are being ignored now, and all profile methods now have tests!


Is that possible to save and restore the nerdtree status? What collapsed and what expanded.

Hi, I am using vim session vim -S. I have read all the issues related to vim session. I know I can not save nerdtree to the session file and I know I'd better quit the nerdtree before I mksession. I am OK to reopen the nerdtree everytime when I launch vim. The only problem and headache I have is how to possibly load the last nerdtree listing status? To resotre what directories are collapsed and what directories are expanded. Thank you very much.

@PhilRunninger thank you very much for the reply. I read that issue before. It was talking about bookmark. I never use bookmark before. and Which separate project do you mean? Do you mean https://github.com/jistr/vim-nerdtree-tabs? In the project readme.md, it does not mention anything about restoring status. I installed that plugin. I don't know how to make it store my nerdtree status. After the installation, I tried, the status is not restored.

Sorry. I should have been more specific. The project is https://github.com/scrooloose/nerdtree-project-plugin.

 I tried my best but I still don't figure out how to use it. After the installation:

I did touch ~/.NERDTreeProjects even if this might not be necessary.

When execute :NERDTreeProjectSave abc in NERD Tree buffer window, it reports:

Error detected while processing function 266[7]..269[13]..279:
line    3:
E716: Key not present in Dictionary: getDirChildren()
E15: Invalid expression: a:rootNode.getDirChildren()

When execute :NERDTreeProjectSave abc in non-nerdtree buffer window, it reports:

E121: Undefined variable: b:NERDTree
E116: Invalid arguments for function 266

I really need a doc on how to use it. Sorry about all the troubles and questions. Thanks a lot for your time.

I'm not surprised you're seeing these errors. A lot has changed in NERDTree since that project was last touched 3 years ago. I'll take a look at this, and see about moving it into NERDTree. I don't have write access to that project, and I'm pretty sure Marty's not interested in updating it.

@PhilRunninger thank you very much for this help. This is really important to a NerdTree plugin. I really don't understand why this is not implemented all this years. Without this functionality, I have no reason to use this plugin instead of the built-in netrw. Thank you very much.


Cannot build for prects using androidx

Is the bug present in the demo Chat SDK project?  YES

What modifications have you made to the Chat SDK?

Android Version: API 28

Steps taken to reproduce the problem: Just sync with gradle

Expected result: Failed to sync

Actual result:

Comments:

Can you port to androidx libraries ?

We've now updated the library to use Android X.


error when run action

anyone can help me?
i got this error when try run action
this is my yaml file
name: deploy
on:
  pull_request:
    branches:
      - qa
jobs:
  deploy:
    runs-on: ubuntu-latest
    steps:
    - name: Checkout Repository
      uses: actions/checkout@v2

    - name: Setup Node.js
      uses: actions/setup-node@v2
      with:
        node-version: '10.x'

    - name: Install Dependencies
      run: npm install

    - name: Build for Production
      run: npm run prod-build
    - uses: amondnet/vercel-action@v20
      with:
        vercel-token: ${{ secrets.VERCEL_TOKEN }}
        github-token: ${{ secrets.GITHUB_TOKEN }}
        vercel-org-id: ${{ secrets.ORG_ID}}
        vercel-project-id: ${{ secrets.PROJECT_ID}}
        working-directory: ./sub-directory

Try using amondnet/vercel-action@v25 so you get a newer version of node.  This will also eventually stop working because Vercel is deprecating node 16 and there's no newer version of this action yet. See #249

I am getting this issue too using v25. Is it necessary to use the setup Node step? I am just using the example action from the readme. If there's a fix for this to just update to a newer node version can someone tell me what that should be? I guess if this is not maintained anymore I can just fork it and update that?


Extra_hosts records are overwitten by some accident

Description
I have chosen a nerdctl compose in order to provision containers without docker. The configuration has extra_hosts: parameters to define static hosts. It is expected that all the hosts I pass over the configuration is turned out into --add-host arguments in nerdctl to create containers according to the compose configuration.
In practice, running a --debug argument to nerdctl, I can see that containers is created as expected with every host I provide in compose configuration, also I see that meta.json file is created with defined and expected number of hosts, but hosts file itself doesn't contain all of them.
I noticed that sometimes ExtraHosts records are overwitten.
I described steps how it can be reproduced.
To be honest, suddenly I found the second bug. It happens when the compose configurations are similar and are located in the same directory. And I suppose both bugs are related to each other. Otherwise, I've been using containers with same hostname but in different compose configuration files.
Steps to reproduce the issue
Start from scratch. We don't have any containers run on the host:
[root@k8s-cp-node1 default]# nerdctl ps -a
CONTAINER ID    IMAGE    COMMAND    CREATED    STATUS    PORTS    NAMES

Check a directory where network configuraton for default namespace is saved:
[root@k8s-cp-node1 default]# ls -l /var/lib/nerdctl/1935db59/etchosts/default
total 0

I have decided to create two similar compose configurations with a container in both of them with an identical configuration.
Prepare the first container with following configuration:
[root@k8s-cp-node1 default]# cat /srv/test/docker-compose.yml.1
networks:
  testnetwork:
    external: true
    name: "testnet"

services:
  k8scpapi:
    container_name: "test-cluster1-k8s-api-0-test1"
    hostname: "k8s-api-0"
    image: "alpine:latest"
    pull_policy: "never"
    restart: "unless-stopped"
    read_only: true
    runtime: "runc"
    labels:
      app: "host-main"
    extra_hosts:
      - "k8s-api-1:<IP_ADDRESS>"
      - "etcd-0:<IP_ADDRESS>"
      - "etcd-1:<IP_ADDRESS>"
    networks:
      testnetwork:
        ipv4_address: "<IP_ADDRESS>"
    privileged: false
    user: "root"
    command:
      - "sh"
      - "-c"
      - "sleep inf"

Prepare the second container with following configuration:
[root@k8s-cp-node1 default]# cat /srv/test/docker-compose.yml.2
networks:
  testnetwork:
    external: true
    name: "testnet"

services:
  k8scpapi:
    container_name: "test-cluster1-k8s-api-0-test2"
    hostname: "k8s-api-0"
    image: "alpine:latest"
    pull_policy: "never"
    restart: "unless-stopped"
    read_only: true
    runtime: "runc"
    labels:
      app: "host-main"
    extra_hosts:
      - "k8s-api-1:<IP_ADDRESS>"
      - "etcd-0:<IP_ADDRESS>"
      - "etcd-1:<IP_ADDRESS>"
    networks:
      testnetwork:
        ipv4_address: "<IP_ADDRESS>"
    privileged: false
    user: "root"
    command:
      - "sh"
      - "-c"
      - "sleep inf"

For now we still don't have any container running:
[root@k8s-cp-node1 default]# nerdctl ps -a
CONTAINER ID    IMAGE    COMMAND    CREATED    STATUS    PORTS    NAMES

Start the first container from nerdctl compose:
[root@k8s-cp-node1 default]# nerdctl compose -f /srv/test/docker-compose.yml.1 up -d
INFO[0000] Creating network test_default
INFO[0000] Ensuring image alpine:latest
INFO[0000] Creating container test-cluster1-k8s-api-0-test1

Now we have one container running:
[root@k8s-cp-node1 default]# nerdctl ps -a
CONTAINER ID    IMAGE                              COMMAND              CREATED           STATUS    PORTS    NAMES
b2f2d032f4f6    docker.io/library/alpine:latest    "sh -c sleep inf"    10 seconds ago    Up                 test-cluster1-k8s-api-0-test1

Current configuration of the first container looks like the following. As you can notice we have ExtraHosts records in the hosts file (it is fine for now):
[root@k8s-cp-node1 default]# cat /var/lib/nerdctl/1935db59/etchosts/default/b2f2d032f4f6b3724bf49ff4a9d714850af9f7d887dfef0c83e842bba0038be5/hosts
# <nerdctl>
<IP_ADDRESS>	localhost localhost.localdomain
::1		localhost localhost.localdomain
<IP_ADDRESS>    etcd-1
<IP_ADDRESS>    k8s-api-1
<IP_ADDRESS>    etcd-0
<IP_ADDRESS>   k8s-api-0 k8s-api-0.testnet test-cluster1-k8s-api-0-test1 test-cluster1-k8s-api-0-test1.testnet
# </nerdctl>

Also we have meta.json here that is properly defined as expected:
[root@k8s-cp-node1 default]# cat /var/lib/nerdctl/1935db59/etchosts/default/b2f2d032f4f6b3724bf49ff4a9d714850af9f7d887dfef0c83e842bba0038be5/meta.json
{"Namespace":"default","ID":"b2f2d032f4f6b3724bf49ff4a9d714850af9f7d887dfef0c83e842bba0038be5","Networks":{"testnet":{"cniVersion":"1.0.0","interfaces":[{"name":"br-test","mac":"82:7c:9d:cb:9e:94"},{"name":"veth5251e54a","mac":"8e:b1:2b:4a:5c:28"},{"name":"eth0","mac":"fe:c7:a3:45:11:b3","sandbox":"/proc/43578/ns/net"}],"ips":[{"interface":2,"address":"<IP_ADDRESS>/24","gateway":"<IP_ADDRESS>"}],"routes":[{"dst":"<IP_ADDRESS>/0"}],"dns":{}}},"Hostname":"k8s-api-0","ExtraHosts":{"etcd-0":"<IP_ADDRESS>","etcd-1":"<IP_ADDRESS>","k8s-api-1":"<IP_ADDRESS>"},"Name":"test-cluster1-k8s-api-0-test1"}

Now it is a time to start the second container from nerdctl compose (from a different compose-file):
[root@k8s-cp-node1 default]# nerdctl compose -f /srv/test/docker-compose.yml.2 up -d
INFO[0000] Ensuring image alpine:latest
INFO[0000] Creating container test-cluster1-k8s-api-0-test2

And now we have two conntainer running (it is expected)
[root@k8s-cp-node1 default]# nerdctl ps -a
CONTAINER ID    IMAGE                              COMMAND              CREATED               STATUS    PORTS    NAMES
2a4b4c251668    docker.io/library/alpine:latest    "sh -c sleep inf"    16 seconds ago        Up                 test-cluster1-k8s-api-0-test2
b2f2d032f4f6    docker.io/library/alpine:latest    "sh -c sleep inf"    About a minute ago    Up                 test-cluster1-k8s-api-0-test1

Current configuration of the second container looks fine as we had with the first container. As you can notice we have also ExtraHosts records in the hosts file (it is also fine for now):
[root@k8s-cp-node1 default]# cat /var/lib/nerdctl/1935db59/etchosts/default/2a4b4c251668a1266fae04c9bd60570320f2e207e811b7696b575f2d5195f512/hosts
# <nerdctl>
<IP_ADDRESS>	localhost localhost.localdomain
::1		localhost localhost.localdomain
<IP_ADDRESS>    k8s-api-1
<IP_ADDRESS>    etcd-0
<IP_ADDRESS>    etcd-1
<IP_ADDRESS>   k8s-api-0 k8s-api-0.testnet test-cluster1-k8s-api-0-test2 test-cluster1-k8s-api-0-test2.testnet
<IP_ADDRESS>   k8s-api-0 k8s-api-0.testnet test-cluster1-k8s-api-0-test1 test-cluster1-k8s-api-0-test1.testnet
# </nerdctl>

Also we have meta.json here that is properly defined for the second container:
[root@k8s-cp-node1 default]# cat /var/lib/nerdctl/1935db59/etchosts/default/2a4b4c251668a1266fae04c9bd60570320f2e207e811b7696b575f2d5195f512/meta.json
{"Namespace":"default","ID":"2a4b4c251668a1266fae04c9bd60570320f2e207e811b7696b575f2d5195f512","Networks":{"testnet":{"cniVersion":"1.0.0","interfaces":[{"name":"br-test","mac":"82:7c:9d:cb:9e:94"},{"name":"veth13aa28bb","mac":"ea:ac:62:ed:ca:c7"},{"name":"eth0","mac":"de:0f:06:2b:34:cc","sandbox":"/proc/43698/ns/net"}],"ips":[{"interface":2,"address":"<IP_ADDRESS>/24","gateway":"<IP_ADDRESS>"}],"routes":[{"dst":"<IP_ADDRESS>/0"}],"dns":{}}},"Hostname":"k8s-api-0","ExtraHosts":{"etcd-0":"<IP_ADDRESS>","etcd-1":"<IP_ADDRESS>","k8s-api-1":"<IP_ADDRESS>"},"Name":"test-cluster1-k8s-api-0-test2"}

Let's check again a number of containers:
[root@k8s-cp-node1 default]# nerdctl ps -a
CONTAINER ID    IMAGE                              COMMAND              CREATED               STATUS    PORTS    NAMES
2a4b4c251668    docker.io/library/alpine:latest    "sh -c sleep inf"    About a minute ago    Up                 test-cluster1-k8s-api-0-test2
b2f2d032f4f6    docker.io/library/alpine:latest    "sh -c sleep inf"    2 minutes ago         Up                 test-cluster1-k8s-api-0-test1

And it is a time to check for bugs (This is a first bug I found). Hosts file was being overwritten and now it doesn't contain any ExtraHost record already after the second container was created:
[root@k8s-cp-node1 default]# cat /var/lib/nerdctl/1935db59/etchosts/default/b2f2d032f4f6b3724bf49ff4a9d714850af9f7d887dfef0c83e842bba0038be5/hosts
# <nerdctl>
<IP_ADDRESS>	localhost localhost.localdomain
::1		localhost localhost.localdomain
<IP_ADDRESS>   k8s-api-0 k8s-api-0.testnet test-cluster1-k8s-api-0-test2 test-cluster1-k8s-api-0-test2.testnet
<IP_ADDRESS>   k8s-api-0 k8s-api-0.testnet test-cluster1-k8s-api-0-test1 test-cluster1-k8s-api-0-test1.testnet
# </nerdctl>

If we decide to remove one of the containers from the host by any of the compose configuration we will remove all of them together (I suppose it is a second bug):
[root@k8s-cp-node1 default]# nerdctl compose -f /srv/test/docker-compose.yml.1 down
INFO[0000] Removing container test-cluster1-k8s-api-0-test1
INFO[0000] Removing container test-cluster1-k8s-api-0-test2
INFO[0000] Removing network test_default

To check if both containers were being removed we run this command to be aware of it:
[root@k8s-cp-node1 default]# nerdctl ps -a
CONTAINER ID    IMAGE    COMMAND    CREATED    STATUS    PORTS    NAMES

Up to this moment we don't have both containers we created on this host:
[root@k8s-cp-node1 default]# ls -l /var/lib/nerdctl/1935db59/etchosts/default
total 0

Describe the results you received and expected

Compose configuration has to be independent and should not be removed with other similar ones.
ExtraHosts should be overwritten once created.

What version of nerdctl are you using?

@AkihiroSuda maybe we can close this?


Fix 9431

fixes issue #9431 I check to see if we have a original block and if we do if its editable and then show the link or if we don't have an original object it will show it too.
creating new PR to merge into develop as opposed to master

should I have branched from develop for this @aembler instead of master? seem to have a lot of conflicts

@gutig  it should be made from develop

thanks thats now done here; https://github.com/concrete5/concrete5/pull/9437


Feature Request: Add support for including a CSS class or inline style

An SVG sprite of many <symbol> tags still takes up space in the viewport even though the icons are not displayed (until they are used).  I can't think of a use case where this is the developer's intention.  If we could include a class name on the output SVG sprite, or inline style display: none everything should work as expected.
I'd like to avoid a post-processing step that tacks on a class name if we can accomplish this is one fell swoop.

@rhewitt22 Off the top of my head (sorry if I'm wrong), this should be done when enabling mode.<mode>.inline. Also, you should be able to add (almost) arbitrary attributes to the outermost <svg> element by using svg.rootAttributes. Can you confirm this?

@jkphl Indeed! The inline mode was what I needed. I spent quite a bit of time in the docs, the number of options can be a bit overwhelming -- sorry I missed that!
--si true did the trick by adding position: absolute; to the root element.
Thanks!

@rhewitt22 You're welcome! :)


SwiperLazy module returns en error

Hello! Since the update from 0.1.9, any Swiper components that use the module [SwiperLazy] return an error, did it become a default functionality?

Hey @luigigorlero it seems they taken out the module in favor of letting the browser handle Lazy Loading. You can read more on it here: https://swiperjs.com/swiper-api#lazy-loading

Alternative would be to loading="lazy" as an attr on the image itself.

@luigigorlero I updated README to let other users know they taken it out. Since nuxt-swiper v1.0.0 is a major change you can fallback to 0.1.9 in the meantime if you like using that module.
I will close this issue for now. Thanks for the letting me know! 💚


Cuda error during training step.

Hello I was running the training step as mentioned python main.py --mode train --para_limit 2250 --batch_size 24 --init_lr 0.1 --keep_prob 1.0 --sp_lambda 1.0
I encountered this error.  I am running this on GPU , is it possible that this is a memory allocation issue? Any idea how much memory is required to perform the training step?
Any pointers towards resolving this would be much appreciated.

how to do it?  my environment is
torch 0.3
cuda 9.0
wile my cuda version in "nvidia-smi" is 10.2,cudatookit is 10.1
i find that there is no higher cuda version for torch 0.3  while readme said they use torch=0.3
thank!


DNL: malloc error reproducer

Not to be reviewed or landed. PR is intended to share a bug reproducer.
On my machine vargo build gives:
mbm@m1p ~/D/s/t/v/source (bug-malloc-error)> vargo build
...
vargo info [0]: vstd outdated, rebuilding
    Finished dev [unoptimized + debuginfo] target(s) in 0.03s
     Running `target/debug/vstd_build target-verus/debug`
...
rust_verify(9577,0x1e2eedc40) malloc: *** error for object 0x9bf200009f03: pointer being freed was not allocated
rust_verify(9577,0x1e2eedc40) malloc: *** set a breakpoint in malloc_error_break to debug
thread 'main' panicked at vstd_build/src/main.rs:140:9:
vstd build failed with exit code None
note: run with `RUST_BACKTRACE=1` environment variable to display a backtrace
error: vstd_build returned status code Some(101)

I'll close this as it's only intended for sharing the code. Filed issue #1097 for discussion.


feat: Elevator status widget page-splitting/fitting logic

Asana task: Elevator status widget component - list view
This PR makes some small tweaks to the frontend styles to make the list view exactly match the designs, so that the backend can reliably calculate the height of each row as 100px + 48px * length(closures_at_station).
The rest of the code is the backend logic to achieve the page-splitting/fitting logic dictated by the designs.
It took a lot of test cases to cover all the possible outputs of this logic—sorry for the gigantic test file!

[ ] Needs version bump?

When this is ready for Christian's review, add him as assignee (so I know who's responsibility it is by looking at the PR list)


No mention of status: filtering for PR search

Code of Conduct

[X] I have read and agree to the GitHub Docs project's Code of Conduct

What article on docs.github.com is affected?
https://github.com/github/docs/blob/main/content/issues/tracking-your-work-with-issues/filtering-and-searching-issues-and-pull-requests.md
What part(s) of the article would you like to see updated?
There is no mention of the status: filters when browsing PRs. status:success is valid, and filters PRs down to those which have passing workflows. status:failure is also a valid search term.
I am not sure what other terms are valid.
Additional information
Note that this is explicitly not to do with review-status, status is about CI checks.

@PowerfulBacon Thank you for raising this issue! I'll get this triaged for review :sparkles: Our team will provide feedback regarding the best next steps for this issue - thanks for your patience! 💛

Hi @PowerfulBacon, thanks for raising an issue before raising a PR—we really appreciate it.
In this case, I would point you to https://docs.github.com/en/issues/tracking-your-work-with-issues/filtering-and-searching-issues-and-pull-requests, which is linked to under "Further reading" on the page you highlighted.
I do think you've highlighted a poor user experience here that should be improved. Unfortunately, given the complexity involved across multiple pages (and likely, coordination with internal stakeholders), it's not something we'd be able to coordinate as an open-source issue, and open to contributors.
I'm sorry I can't give you a better answer here, but please be assured that I'll highlight this internally. I'll leave this open temporarily in case you have any further questions.

Thank you, that's perfect.

Shahrul Luqman


Update Chart.yaml

I had to fix this elsewhere. The old bitnami charts URL used to forward, but it appears not to anymore. We need to explicitly use the new one: https://charts.bitnami.com/bitnami
https://github.com/paulczar/charts/blob/5bee3caca9c3baaf9a799be5cc63e26aa5d26e56/charts/dependencies/requirements.yaml#L4
https://github.com/paulczar/charts/blob/69f63e25c0f8c8b278fba8bae2ee0111a8c13dd3/ct.yaml#L6


Conflicting information within the documentation

The top paragraph of this document states "The plugin is enabled, by default, when deploying Kubernetes clusters with the Azure Container Service Engine (or ACS engine)."
The Constraints section states "The plugin is not enabled when deploying a Kubernetes cluster with the Azure Container Service (AKS) or ACS cluster with Kubernetes."
Please clarify which is correct.

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

ID: ef8e1342-e2fb-1f03-b0d5-10dbf6a48553
Version Independent ID: 2556003d-aae6-54bf-bc9c-6da12d2c4c2f
Content: Azure Virtual Network for containers | Microsoft Docs
Content Source: articles/virtual-network/container-networking.md
Service: virtual-network
GitHub Login: @jimdial
Microsoft Alias: jdial

@markedrichardson They're both correct. ACS and AKS (the constraint) both create Kubernetes clusters that are managed by Azure. The ACS Engine creates Azure Resource Manager templates that allow you to deploy a Kubernetes cluster that you manage. Does that help?

#in-process

@markedrichardson Since I haven't heard back from you, I'm closing this issue. Just let me know if you have further questions though. #please-close.


Generate thumbnail from rendered video

Is there a way to generate a thumbnail from a specific frame / timestamp of the created video once it is done rendering? Is there an action, I didn't see, that takes care of generating such a thumbnail and also uploads it alongside the video?

Hey @kalmicka
That sounds like a custom post-render action, that uses ffmpeg module from action-encode combined possibly with action-upload. Quite a combination I would say :D

That's why I am asking if such an action might already exist :)
Is there a good tutorial out there that might tell me how to build custom actions?

Unfortunately, no proper tutorial for that exists. There is only a tiny part of doc here: https://github.com/inlife/nexrender#custom-actions
And additionally, existing actions can be used as references.

Thanks for the help. Awesome project by the way! :)

Thank you, glad to be of help! :)


Offline Help was more accurate

Fix this document formatting

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

Didn't it occur to you that had I known how to fix it I would have done it myself? Not everyone is as knowledgeable as you are.

Furthermore, if I try to edit the part in the article where it says, "UBound returns the following values for an array with these dimensions:" in order to add the example array, I get this message, "You’re editing a file in a project you don’t have write access to.  Submitting a change to this file will write it to a new branch in your fork, so you can send a pull request. "

@grovelli - the above is the correct way to submit a change to a page. Someone from Microsoft will then review it and merge your changes to the documentation. By the way, I fixed the formatting on this page: https://docs.microsoft.com/en-us/office/vba/language/reference/user-interface-help/ubound-function. Was there something else you wanted changed?

@grovelli fair points, if you make your changes it'll post to your local copy "Fork", then you need to click the button on the left-hand side of the <>Code tab that says "new pull request", then fill in the short name and quick description of what changed, and do the green "create pull request button". At that point your fixes are in the hands of the MS team to integrate into the master version.
If your looking for more detail, I went through the tutorial a couple weeks back and that helped me a lot.
https://guides.github.com/activities/hello-world/

@grovelli also note that the MS team is in a major formatting makeover this week so, you're probably better holding off any changes till on/after Sept 5th


feat: auto-close browser window after auth success

Auto-close the browser window after successful authentication
To enhance QoL and user experience when using sigstore/cosign/gitsign often, this feature should automatically close the authentication window once the user successfully logs in, thereby streamlining the signing process and reducing unnecessary interactions after authenticating. Prior to this, users manually close the window, a step that can be easily become tedious, leading to a cluttered browser (I know, what a problem to have!).

I have a WIP that I've been testing locally with gitsign and it's been pretty good so far.
https://github.com/sigstore/sigstore/compare/main...stgarf:sigstore:feature/autoclose-auth-page

That sounds great! If you wanna go ahead and open a PR I can review it :)

@wlynch @imjasonh didn't one of you attempt this before and run into a road block?


pull request for e2e test at 01/02/2017 02:26:12

:white_check_mark: Validation status: passed
For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


dpo:edit commands of improvePlacement for better HPWL.

Signed-off-by: niv280<EMAIL_ADDRESS>Hi,
I change the set of command in improvePlacmentto to give better results.
Here is comparision between the new set commands ( from the left ) to the old command ( in the right):
 [INFO ODB-0128] Design: aes_cipher_top
Original HPWL           372201.2 u	Original HPWL           372201.2 u
Final HPWL              354025.3 u	Final HPWL              360418.2 u
Delta HPWL                  -4.9 %	Delta HPWL                  -3.2 %

[INFO ODB-0128] Design: gcd
Original HPWL             7709.2 u	Original HPWL             7709.2 u
Final HPWL                7154.5 u	Final HPWL                7322.4 u
Delta HPWL                  -7.2 %**	Delta HPWL                  -5.0 %

[INFO ODB-0128] Design: ibex_core
Original HPWL           652460.9 u	Original HPWL           652460.9 u
Final HPWL              636120.8 u	Final HPWL              639841.3 u
Delta HPWL                  -2.5 %	Delta HPWL                  -1.9 %

The most affect gain from increase the movePerCandidate from 20 originally to 100, even though the run time degraded I think it worth it, but that results is on small design so please advise if you think that is too much in bigger design.
Other change is change the order of the commands that gave better results.
Also I notice that the mis and reorder method don't make much different at least on the tested design so I moved them to the end, but I also can comment them to reduce some run time.

@akennings FYI for any comments

Increasing the moves per candidate seems fine if the run time does not get out of control for larger designs.  Changing the order of commands is also fine.

If the def files varies run to run with the same parameters then that is a problem that needs investigating.  We expect deterministic results.

The problem was on my side, the results are deterministic.
The format code check are failing me, it's looks like it expecting change only in c++ files, this is true?
I still need to try to route the design with the new command, I will update about it tomorrow.

The format code check is not blocking for now.  However it is helpful if you can run clang-format on the code to keep it up to date.

Hi,
I fixed the clang format error.
Also, I tried to route ibex design with the new command, here what it looked like:

Here is with the old set of command:

Unfortunately, the short and metal spacing violation was degraded with the new command from 4 to 21 but the overall wire length reduced.
I will try to reduce the moves per candidate and see if I getting better results in the shorts violations.
I think it is also important to compare the congestion but I'm don't sure were I can check it, can you guide me?

Unfortunately, the short and metal spacing violation was degraded with the new command from 4 to 21 but the overall wire length reduced.

Interesting. I was wondering how we prevent DPO from undoing both timing and congestion based global placement. Should it incorporate the STA and grt data (like gpl currently does)?

To check congestion you can look at the congestion map in the GUI or dump it out with "gui::dump_heatmap name filename"

The routing congestion looks pretty similar to me.
The old commands:

with new commands:

What do you suggest for the next steps?

Perhaps it is something more local.  The m3 shorts are new so perhaps take a look and see what is happening there.  Is there increased local congestion that might explain the problem?

@maliberty Maybe we should add a feature to compare the congestion values between two runs?

@maliberty Maybe we should add a feature to compare the congestion values between two runs?

Sure though you can dump the heat map to a csv and do it easily externally.

I think that I misunderstood the short in the design.
I used that command for routing:
detailed_route -output_drc dpo_tests/ibex_drc -output_maze dpo_tests/ibex.output.maze.log -output_guide_coverage dpo_tests/ibex.coverage.csv
and the output_drc file is empty, but in the last message in the tool I see the short that I upload above.
Maybe the last drc report in the detailed_route is not the final one?
I tired also the check_drc command but it keep failed to me.

I checked it again the drc file is empty so as I understand it there is no drc in both of the command and the route succeed with the new command also there is small improvement in the power ( in aes design) .
Please advise for the next steps here.

@maliberty Do you have some advice?

Would you include more of the router log.  I wonder if you are looking at the right message or not.

I attached log of 2 runs of ibex_sky130hd test, one with old command and one with new commands.
To the test  ( in flow.tcl) I added the improve_placement and check_placement proc to review the diff.
Let me know if that not enough and you need me to add other command to the test.
Withe new commands:
ibex_sky130hd.log
With old commands:
ibex_sky130hd_old_commands.log

@niv280 I don't see actual detailed routing in either of the logs. It looks like pin access is running, but not detailed routing itself. To get the violation count, you should see something in the form of
[INFO DRT-0199]   Number of violations = ###.

I think it was the verbose=0, here is the update logs:
With new commands:
ibex_sky130hd.log
With old commands:
ibex_sky130hd_old_commands.log
Sorry for the multiple comments.

From the logs you posted, both versions converge to 0 violations, but the new one takes 11 iterations to do so compared to 10 previously.

Yes, this is the downside of the new commands change but we are getting better HPWL
@maliberty  what do you think?

So long as we can still get to 0 drvs that's ok.  It is expected that denser designs will take longer to route.

So, it required anything else to be approved?
I'm just won't be available for the next 2 months from next week, so I want to close it.

@niv280 One thing for sure is that you need to adhere to the formatting rules. Please run clang-format on your code to pass the required formatting check.

I ran ORFS with this change and I see some degradation of results that need investigating (there are some more minor changes that we could just accept):
nangate45/bp_multi - fails in grt due to congestion
nangate45/jpeg - [ERROR] finish__timing__setup__ws fail test: -0.09 >= 0.0

Maybe there should be less ambitious optimization e.g. 50 moves per candidate instead of 100? This is a good candidate for a sensitivity study / possibly throw at the autotuner.


add lists of figures,tables,listings adoc that can be reused in specs

relates to https://github.com/riscv/riscv-isa-manual/issues/1532

Closing as it was decided not to add these lists.


Pre-training model cannot be downloaded

Pre-training model cannot be downloaded.
Thank you for your work, I can't download the pre-training model from the link you gave, can you give other download links, such as Google Driver, Baidu Wangpan, etc. Thank you very much.

I got it from your link. Thanks

@taoshiqian Hi, can you share a pre-trained model? The link fails to download now.


internal/dag: rebuild should not trigger on unrelated services

Inside a k8s cluster there are many service objects, only some of them may be referenced by a valid ingress object.
Because contour must watch all service objects it may see an unrelated service changing and this will trigger a rebuild of the dag. The end result should be a noop, but it would be better to avoid this activity in the first place.
Contour should only trigger a dag rebuild if the service is referenced from a valid ingress object.

Unassigned myself since @davecheney was going to reimplement.


[refactor]: Modify APIs of patch-db to minimize non-canonical ordering issues without hyper-aggressive locking

Motivation
The current behavior of using patch-db locking to manage concurrency across tasks. This is hyper aggressive, and full of danger of deadlocks.
Proposed Solution
Modify the API of patch-db to take all locks as a single batch request. it should not be able to acquire new locks until it drops all locks that it has.

Investigate possibility of using predicate filters and how it maps onto data tree cardinality changes.


Content losing margin-top after coming back from a view where the keyboard was opened.

Ionic version:  (check one with "x")
(For Ionic 1.x issues, please use https://github.com/ionic-team/ionic-v1)
[ ] 2.x
[x] 3.x
[ ] 4.x
I'm submitting a ...  (check one with "x")
[x] bug report
[ ] feature request
Please do not submit support requests or "How to" questions here. Instead, please use one of these channels: https://forum.ionicframework.com/ or http://ionicworldwide.herokuapp.com/
Current behavior:
Updated the angular and ionic packages to the latest version and whenever I use the keyboard on Android and go back to he previous view using the phone back button, the fixed-content and scroll-content divs loses the margin-top attribute, making the content shifts below the header.
Expected behavior:
Keep the margin-top with the height of the ion-header.
Steps to reproduce:
Create a new Ionic app, add a ion-textarea or ion-input to a view that you'll push to, type whatever or nothing at all (just let the keyboard slides in) and use the phone back button, not the app back button (this is important to reproduce the bug).
Ionic info: (run ionic info from a terminal/cmd prompt and paste output below):
cli packages: (/home/rafael/.nvm/versions/node/v8.7.0/lib/node_modules)

    @ionic/cli-utils  : 1.13.1
    ionic (Ionic CLI) : 3.13.1

global packages:

    cordova (Cordova CLI) : 7.1.0

local packages:

    @ionic/app-scripts : 3.0.0
    Cordova Platforms  : android 6.3.0
    Ionic Framework    : ionic-angular 3.7.1

System:

    Android SDK Tools : 26.1.1
    Node              : v8.7.0
    npm               : 4.6.1
    OS                : Linux 4.10

Misc:

    backend : pro

Hello! Thank you for opening an issue with us! I am closing this issue because it is a duplicate of Issue #13028. Please track that issue for further updates. Also please comment on that issue with any new information you may have to share.  Thank you for using Ionic


CLI: Reduce Binary Size & Add Windows Build

This adds a Windows build for the CLI, and also reduces the resulting binary size by removing debug symbols.  The yield is about 30% reduction in size.

@ilithanos @yitsushi what do you think?

Removing the binary size is a good addition, but adding windows build can cause some issues.
We have no mechanisms to test windows builds, and we (at least I don't) have a place to test it manually to make sure it's working properly as a result I can't say we can support it. If it doesn't work we can't even debug it. Adding a windows build would indicate we support it, but we have no capacity and resources to support it. At least that's my opinion. It can be a good addition, but is no one can guarantee we can support it that may result removing it later if it's broken at any points in time. If we add docs around it like "the windows build is not 100% supported, if something is broken with it, we may not be able to debug and fix it", but that would give a bad taste too.

The binary size reduction is good.
Regarding the windows build i completely agree with your assessment. We don't have a way to debug it, test it or maintain it. The idea is good, but at the same time i have a hard time seeing the need for it. Since the linux build could be run on a windows machine using WSL without any changes.
Having released versions that we can't support seems like a complete no go for me. I would rather document how to run the linux binary in WSL on windows, or throw a docker container on Windows considering the current state of the project, and the current amount of resources available for maintaining it.

Understood.  I'll keep the Windows binaries on my fork.  Macs are a rarity for devs around here unless you are special (large enterprise).

I have seen windows used by developers in some cases, but mostly in Azure/DotNet/c# based companies. And in those cases they always had WSL, Docker or a Linux VM in use as well.
At the current place only C level management, support, sales etc are running windows, everybody in the development related departments are offered the choice between Mac or Linux machines ;)
@bdwyertech is there a reason a Docker based or WSL based solution isn't an option? If for nothing else then just to expand my horison :)

I have seen windows used by developers in some cases, but mostly in Azure/DotNet/c# based companies. And in those cases they always had WSL, Docker or a Linux VM in use as well.
At the current place only C level management, support, sales etc are running windows, everybody in the development related departments are offered the choice between Mac or Linux machines ;)
@bdwyertech is there a reason a Docker based or WSL based solution isn't an option? If for nothing else then just to expand my horison :)

Proxies, VPNs etc...  I own the CLI tool internally and I've had to add a doctor function to check registry entries, proxy configs, env vars, docker configs, etc. to make sure everything looks good.
15k employee enterprise, I think I am one of 10 dev's with a Mac, about 100 macs total - the rest are executives.

Oh, yeah i can see that being an issue. I would say exposing tfctl to developers without understanding of basic k8s etc might be a risk :)
If using github branch planner might also be an option. I remember they days of using AnyConnect with dread. It works, but is very finicky about the environment it runs in :)
Would you mind either cleaning up the PR by removing the windows specific build, but leaving the binary size optimization? then i'm sure @chanwit will merge it when he gets around to it.
Thanks for helping out with the project and giving me some more info on the landscape of the userbase.

@yitsushi is also a maintainer who could also help review this issue in my absence.

Oh, yeah i can see that being an issue. I would say exposing tfctl to developers without understanding of basic k8s etc might be a risk :)
If using github branch planner might also be an option. I remember they days of using AnyConnect with dread. It works, but is very finicky about the environment it runs in :)
Would you mind either cleaning up the PR by removing the windows specific build, but leaving the binary size optimization? then i'm sure @chanwit will merge it when he gets around to it.
Thanks for helping out with the project and giving me some more info on the landscape of the userbase.

Sure, no prob.  I’ll clean it up later this afternoon.
100% agree with the risk aspect.  This tool feels more geared to a platform engineer than something Joe dev needs.
I’m exploring possibilities here more than anything else.  This project looks great and I love the execution model where per-project pods get stood up to scale tf execution horizontally (and offering unique permissions by way of IRSA annotation).  I’ve contributed a few features to the tf crossplane provider, quick glance says this project is more in line with our plans.  Should this be a path we take, you’ve got another contributor for sure.

Done.  Didn't rename the branch to avoid opening a new PR and losing this convo.


Port to ClassGraph from FastClasspathScanner

Resolves https://github.com/orbit/orbit/issues/310

@lukehutch PTAL :tada:

Why not just use the class name as the key?

@lukehutch what section of code are you referring to?

The same part of code as my previous comment. Sorry to not clarify.

@lukehutch I think you need to click submit review, I suspect your previous comment is a draft.

Math.abs(classId) too? That's the second chance of hash collision (and doubles the likelihood of collision...)
If it's out of scope, you might want to make another PR that gets rid of all the class id stuff. This is really bad.

Available in 1.12.0
https://github.com/orbit/orbit/releases/tag/v1.12.0

@JoeHegarty I looked at the Orbit 2.6 class CapabilitiesScanner.kt that you linked to. You need to encapsulate the ScanResult in a use { ... } block in Kotlin: https://github.com/classgraph/classgraph/wiki/API:-ScanResult#scanresult-lifecycle

@JoeHegarty I looked at the Orbit 2.6 class CapabilitiesScanner.kt that you linked to. You need to encapsulate the ScanResult in a use { ... } block in Kotlin: https://github.com/classgraph/classgraph/wiki/API:-ScanResult#scanresult-lifecycle

Thanks for catching that, fixed.


Create uvic.txt

http://www.uvic.cat/

@pueyo5 please excuse the delay - I'm adding the domain.

Can you please approve the pull request #1243 for adding the domain www.kluniversity.in
https://github.com/JetBrains/swot/pull/1243


refactor Transaction view

tracking issue: #6
Summary
Transaction view is a mess right now and needs to be cleaned up.
Description

[ ] The /transaction/HASH should be updated with the following:

[x] The transaction JSON view should be expanded by default, rather than hiding the react-json-view component
[x] The block height should link to the block page
[x] it shows a table of all of the raw ABCI events occurring in the context of the transaction. Currently, some of them (the note commitments) are handled specially, but at this stage, we should just show the raw events, and not attempt to parse or interpret them.

Closed by #73


Fix center mode (in non-infinite mode) cannot go to the last slide group

when groupsToShow > 1

fixed in v0.3.1


Route [voyager.prdatatable.index] not defined. (View: ... vendor\tcg\voyager\resources\views\tools\bread\index.blade.php)

Version information

Laravel: v8.12
Voyager: v1.4
PHP: 7.3.26
Database: 5.0.4

Description

Symfony\Component\Routing\Exception\RouteNotFoundException
Route [voyager.prdatatable.index] not defined. (View: C:\xampp\htdocs\test\vendor\tcg\voyager\resources\views\tools\bread\index.blade.php)
Steps To Reproduce
Steps to reproduce the behavior:

Go to '...'
Click on '....'
Scroll down to '....'
See error

Expected behavior
A clear and concise description of what you expected to happen.
Screenshots

Additional context
Add any other context about the problem here.

Issue template is not optional.  Please provide more information or we cannot help you

Closing because no feedback was provided


BackHandler: specify function return type for handler

Summary
Following the comment https://github.com/DefinitelyTyped/DefinitelyTyped/pull/42618#discussion_r384584159
Modify the flowtype of BackHandler function to have more specific return type.

Changelog

[General] [Changed] - Changed type of BackHandler to be more specific.
Test Plan
No flow error in RNTester

@TheSavior Can you merge this? 🙏


New SKU for ER

This checklist is used to make sure that common issues in a pull request are addressed. This will expedite the process of getting your pull request merged and avoid extra work on your part to fix issues discovered during the review process.
PR information

[ ] The title of the PR is clear and informative.
[ ] There are a small number of commits, each of which have an informative message. This means that previously merged commits do not appear in the history of the PR. For information on cleaning up the commits in your pull request, see this page.
[ ] Except for special cases involving multiple contributors, the PR is started from a fork of the main repository, not a branch.
[ ] If applicable, the PR references the bug/issue that it fixes.
[ ] Swagger files are correctly named (e.g. the api-version in the path should match the api-version in the spec).

Quality of Swagger

[ ] I have read the contribution guidelines.
[ ] My spec meets the review criteria:

[ ] The spec conforms to the Swagger 2.0 specification.
[ ] The spec follows the guidelines described in the Swagger checklist document.
[ ] Validation tools were run on swagger spec(s) and have all been fixed in this PR.

Automation for azure-sdk-for-js

Can one of the admins verify this patch?

Automation for azure-sdk-for-python

Automation for azure-sdk-for-java

Automation for azure-sdk-for-go

@jianghaolu I'm merging the PR to unlock the main PR to be merged. Tell me if anything here is a concern for you :)
Thanks!


Website and µblock

$ is undefined when I activate µblock. This makes the whole animation on the right never appear.
I've never had any issue with jQuery + µblock before,  so I might be missing something here.
Here are my filters (in French):

I'm using µblock as well and I'm not having the issue. Which page did you test?

The homepage with the scroll animation. The failing script is home.js. And I think it really depends on the activated lists.

I have more and more webpage failing with ublock recently, wonder if we need to dig this one

And I think it really depends on the activated lists.

Maybe first set the default lists, I am not sure a lot of users have tuned their lists

I have ublock and it works, dont mess too much with all those lists :D


docs: update readme with multiple scope syntax

Description
Update the README with the required syntax for multiple scopes
Related Issue
https://github.com/adobe/aio-apps-action/issues/39
Motivation and Context
How Has This Been Tested?
Screenshots (if appropriate):
Types of changes

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

[x] I have signed the Adobe Open Source CLA.
[x] My code follows the code style of this project.
[x] My change requires a change to the documentation.
[x] I have updated the documentation accordingly.
[x] I have read the CONTRIBUTING document.
[ ] I have added tests to cover my changes.
[ ] All new and existing tests passed.

@fe-lix-
Good point. Updated the README to reference "meta scopes"


Spaceships example: change Without to With in Visual Interp setup

I noticed replication on the client adds Position before it adds Confirmed, causing the Confirmed entities to also get the visual interpolation components.
I changed the filters to make more sense, and not bothering with interp on the server.

Hm that's strange; or is it because of pre-spawning?

on main at the moment, if you open the egui inspector, you can see that as well as the players, all the balls have the visual interp components on their Confirmed entities as well as on the Predicted. so it's nothing to do with predictive spawning – all those entities are server authoritative anyway.
only explanation i can think of it that Position and Rotation are inserted before the Confirmed component, causing the Without<Confirmed> filter to be useless.

from a quick look, perhaps because ShouldBePredicted is used to trigger spawning the Predicted entity, and only then is the Confirmed component added to the confirmed entity.. not really a bug, just caught me out.


Support EditorConfig as inputs

I would like to see EditorConfig be used as a fallback for the default spacing when no format values are specified.
Advantages: keep indentation config in one place; no desync in config/execution; no need to pass arguments or override; ability to specify different values for different files/directories if needed for vendored third-party files
Disadvantages: requires parsing .editorconfig, a third-party format & would incur additional dependencies; minor endorsement of EditorConfig as a project since it would get special support
See response: https://github.com/kdl-org/kdl-rs/issues/96#issuecomment-2525061644

Good idea!
#55 added support for specifying indentation level using a custom kdlfmt config file. Adding fallback support for editorconfig should be pretty straightforward 😄


src: end va_list before returning

Explicitly end the va_list before returning in the ffStrbufSetF() function as va_lists are not ended implicitly.

Thanks


Limit mouse-over zone for creature tokens

Currently, hovering the mouse over a creature token or its threat zone displays status information about that creature. In close quarters, this causes misidentification of creatures too often. Instead, just consider a hover over the token itself, not the threat zone, too.

Now pops up window with status info when mouse hovers only over creature token


cURLUploader add -T, unpossuble to use -F

I think we should have an option to decide how to uplod file, -T or if i need multipart form use -F instead

@ghostinside : The cURLUploader task is a convenience task for uploading files, hence -T is passed. If you want to use cURL with -F option, you can write a script and run it using the Shell script task or the Batch script task .
Thanks,
Madhuri


<RichText isEditMode={false}

[? ] Bug
[? ] Question
Version
Please specify what version of the library you are using: [  3.13.0   and   3.14.0-beta.4277292  ]
Expected / Desired Behavior / Question
RichText isEditMode={false} should show the full width of the text the same as isEditMode={true}
Observed Behavior
The lines are broken as if there is a limit to the line length

it should look like so

Steps to Reproduce
      <div style={{ width: '100%', }}>                     <RichText isEditMode={false} value={importantNotes} />                     <RichText isEditMode={false} value={detailNotes} />                     {hasUserInput &&                         <RichText isEditMode={true} value={userInput} onChange={getUserInput} />                     }      </div>
Is there a settings or class to override?
Thanks!

@Ofer-Gal , I can't reproduce that behaviour , can you check or give more details please...

Thats all I have. I will use another control instead.
Thank you.

Now with 3.14.0 I have similar issue:
With isEditMode={ture}

with isEditMode={false}

I am glad the read only has no border but why is the text broken at the start?
Is there any style I can apply?

Hi @Ofer-Gal,
I tried with latest version (3.15.0) and couldn't repro. Did you have the chance to try too? If it's the same behavior, do you have a code snippet or how the content is saved / loaded?


Support for VanillaPlus

Is there a way to use FlightCore with VanillaPlus?

Not at the moment, no. I'd be fine to take a PR for it though ^^


chinese support

i have see same problemhttps://github.com/microsoft/vscode/issues/78621

i have see same problemChinese support in remote ssh

i have see same problemChinese support in remote ssh

chinese extension document😄


Change localhost into IP address

I want to use the IP address instead of localhost, so that another PC can be connected to and join the blockchain network?

Hey,
You can call naivecoin with the option -a <IP_ADDRESS> to publish the service.
Best

Thank you!

Interesting.
What happens when you access the web UI on the same machine but using the IP instead of localhost or <IP_ADDRESS>?

no, that's not my mean. I mean, when I run this blockchain using my ip (example: <IP_ADDRESS>) and when I use another computer in same LAN network (example: <IP_ADDRESS>) access to it and it not response anything. If it can access to from another computer, so why we can demonstrate the feature peer-to-peer of the blockchain ?

Yes, I got your question right. I asked to try to access from the same machine using the IP to check if the service is responding to the correct interface (IP exposed to the LAN). If it is responding it means that the problem is not which interface selection in the service. However if you try to access from the same machine using the IP and the service is not responding, it could mean that there is a bug somewhere.
Another test that you can do is to start the naivecoin using the exposed IP instead of <IP_ADDRESS> (Probably this is the problem)
Meanwhile, I'm gonna test this setup here. OK?

ya, if access from the same machine using the IP and the service is responding good. But use another machine to access to it and it can't recognize the service.
Hope you can figure out it soon to see this blockchain for multiple clients in the network, so it can realistic instead of run all in one machine.

Hey, how about this problem? is that true that's I just can only run the blockchain in one computer?

@conradoqg help me this problem, please?

Oh wait....
I do same to you and it works! But, in the x.x.x.x:port/api-docs page, it return an errors in the right bottom corner. When I click into it, it open new window and said "{"schemaValidationMessages":[{"level":"error","message":"Can't read from file http://<IP_ADDRESS>:3002"}]}"

Hey,
That link opens a validation page. Since your service is not published online (I'mean, it's not a public IP) the online validator doesn't work.
Best,

So, how to fix this error? I try to change validatorUrl from "null" to "false" but it still get an errors?

Hey. This is not an error. That link will only work if you expose the naivecoin to the internet, using a public IP.


UTF-8

I've run the db_seed on collation latin1_swedish (my fault), so all tables with text inherit the collation from db. Would be great to have possibility (in config/seed file/environment) to set collation for created tables to avoid similar issues. Is it possible to do with SQL Alchemy?

Maybe encoding in Engine Creation will help. I'm going to check and test it.
http://docs.sqlalchemy.org/en/latest/core/engines.html?highlight=encoding#sqlalchemy.create_engine.params.encoding

I have ride around the docs, and see only one place where this may be relevant:
http://docs.sqlalchemy.org/en/latest/core/type_basics.html
To set a collation type on column definition.

I tried it with column definition but without success.
When I used encoding=latin1 in connection url, it ends with : Failed processing pyformat-parameters; 'latin-1' codec can't encode character '\u013e' in position 4: ordinal not in range(256).
Maybe easiest way in this century is using same encoding as python3 uses :)
IMHO it isn't good pattern mixing-up encodings.

Might be. Would be great to have that info in README / Install file.


remove cedric

Note: /catalogs/**.yaml content should be generated automatically and not be modified manually.
Also note that in this repository we are not creating releases. Customers are encouraged to use the main branch and will receive any changes immediately.
What does this PR do?
(Please set a descriptive PR title. Use this space for additional explanations.)
Any background context you can provide?
(Please link public issues or summarize if not public.)
Do the docs (README or /docs) need to be updated?

[ ] README.md has been updated
[ ] /docs has been updated

Should this change be mentioned in the release notes?

[ ] CHANGELOG.md has been updated

Sorry for the extra work. I have just updated the file myself. No more need for this PR.

awesome thanks. closing this PR


8946 background sms broadcasts

Depends on https://github.com/thecartercenter/elmo/pull/519

Going to wait on reviewing this until the other one is merged and this is rebased since there are quite a lot of changes to the other one.