Grid as tree table

There is a nice sample for nested grids.
Is it possible to define a detail template (third level rows) in the child table e.g. to show a N:M relationship in one view?
If yes than how?

Yes. You can enable the expand/collapse (detailTemplate) config option for the nested grid in order to achieve that. We  don't support tree structure for grids and the only way to achieve that is with detailTemplate for each nested level.


update Joule pin labels and add some new ones

Update pin 1 to match Joule labeling
Change Pins 100-104 to match Joule labeling
Add GPIO pins on 81-84
Add basic debian bundling

Change Pins 100-104 to match Joule labeling

Slightly tangential: which labels are you referring to?

@tingleby has been working on Debian packages for our ppa. But ur pin naming changes look interesting. Can you split the commit so the debian stuff is separate?

Sure, I can split the commit.  Pins 100-104 are labeled LED1, LED2, LED3, LED4 which is how the tuchuck board is silkscreened, but the signal names on the schematic (silkscreen directives?_ and the hardware design doc call those pins GPIO_0, GPIO_1, GPIO_2 and GPIO_3.  I thought it would be best to call the pins by the correct name.

Sorry to bug you about this, but which is the "tuchuck board"? I have the Joule and expansion board, neither of which has the word "tuchuck" on it (keep in mind that I'm a third party that does not have access to internal intel manufacturing naming).
My board has them labeled GP100, GP101, GP102, GP103

I think "Tuchuck" is internal name of the expansion board... At least that is what I have taken it to mean based on some of the documentation I have found.  I think those pins are label GPIO0, GPIO1, GPIO2 and GPIO3.  It just looks like GP100,GP101,GP102 and GP103.

GPIO0, GPIO1, GPIO2 and GPIO3

Ha! Indeed they are.
So, is there now an expansion board that has them labelled LED1, LED2, LED3, LED4?

So, is there now an expansion board that has them labelled LED1, LED2, LED3, LED4?

Maybe there was at some point, but I am not aware of one, thus my change.  The schematic for the breakout board has them labeled as in your picture so I'm not quite sure where the LED1, etc. came from.

Ok, cool—thanks for explaining.

@jewalt do you know what, that's very true, and I don't think anyone has noticed ;-) Internally we've been calling them 100-103 for ages. I don't want to change it now, that'll lead to way more confusion than I need. I'll add a note in the documentation
@rwaldron 'expansion board' == 'tuchuck'. I think 'expansion board' is prone to confusion so I don't use it.


Negative temperature values not valid in TLM frame?

There is no info in specification that negative values are not valid for TLM frames, so I think it would be logic to expand range in Eddystone Validator app(now it's 0 to 60 degrees Celsius).

I agree. Negative value temperatures should is supported according to the spec (As I read it at least) so the validator should reflect this. Only "problem" is that there is not really any dest for invalid values except if it is -128.0f (as it is really the only invalid value in the accepted range (127.0f to 128.0f).

https://github.com/google/eddystone/pull/67


Couple of problems with bined-netbeans-plugin

Unfortunately the discussion section is not acitve so I created this ticket instead. There are some problems for people of using your plugin. I will just add the links to the created issues and hope that you can see the problem somehow fast. If you need more info, I can try to add tickets for every problem and try to check them before doing this:
https://github.com/apache/netbeans/issues/3797 - md opened as binary
https://github.com/apache/netbeans/issues/4584 - A java.lang.NoClassDefFoundError exception has occurred
https://github.com/apache/netbeans/issues/4657 - Variables window show no variables

Hi, thanks for reporting.
I noticed ClassNotFoundException issue (second and third link) few weeks ago. It's nbm building issue - not passing optional dependency after gradle nbm extension was updated. Reverted to mandatory dependency in current bined plugin development build.
https://bined.exbin.org/download/?org-exbin-bined-netbeans.nbm
Plan was to probably rework building to maven, but I didn't get to it yet.
As to first issue, I blame substandard NetBeans design/architecture ;-) - after killing thousands of plugins, I don't expect they would care to improve that, so I'll try to take a look if I can find any mitigation to this issue...

Hey thx for the info. I think problem one we already had this. But I dunno why your plugin seems the only one to create such a problem? Maybe this: https://github.com/exbin/bined-netbeans-plugin/issues/3

Created new build today trying to fix the issue with md files context menu (issue with internal "-nb" filetype postfix).
It seems uninstalling event is not called by NetBeans, not sure I can do anything about that... Also optional module dependencies still doesn't seem to work for me :-/
https://bined.exbin.org/download/?org-exbin-bined-netbeans.nbm
I'll try to release plugin in next few days...

Released version 0.2.4.
It doesn't seem I can request verification in Plugin Portal thou...

Why do you think that the problem has smth todo with the postfix of the mimetype? Shouldn't that be a problem? I mean the devil is in details sometimes but what will the problem be here? Will check your new version soon.

It's my best guess and it seems to fix the issue with md files context menu for me. I didn't find any relevant documentation or section in NetBeans source code, so I guess that postfix is the case for dynamically registered mimetypes. I already spend too many hours on this, so as always, this is my best effort result...

I think it was a mistake by them. If you have a look inside the MarkdownDataObject there is also text/x-markdown and texst/x-markdown-nb so I think -nb should be removed but I dunno why Jan did this.

Requested verification of version 0.2.4. I assumed I had to wait before requesting it as the buttons were not available, but I had to drop previous verification first :-|

Unfortunately I don't know how the verification works. Will check your plugin soon and will add a comment into the initial issues.

Can you please also add your new version to your github repo? Current version is still 0.2.2.

Did you forget to add the nbm file to the releases section or don't you want to? The intention was to have the nbm file also here until there is the verification.

I didn't use github release for nbms for this project. You can download nbm on project's website, on NetBeans plugins site or in maven repository.

Forgot to close this. Assuming it's fixed in both 0.2.4 and 0.2.5.


Timeout in REST calls

Idea
We can specify timeout in the REST calls using grpc-timeout
This was proposed to a customer here, and it worked for them.
It would be great if we could add this to our documentation for two reasons.

For completion sake.
The name grpc-timeout header is a very cryptic name for rest call, and very hard for somebody to figure out on their own.

Sure, I can add this header to the docs but I have a couple questions:

Does this apply to all REST calls and if so, should I create a generic note in each REST topic that says something like "If you get a timeout error, specify a value for grpc-timeout in the header"
Do you have guidance on what values they should specify and why these values? For example, is there a good value for customers to use all the time, or are there different values for different scenarios?

Re 1: This does apply to all REST calls, but we would like to  mention this ideally for calls that can potentially take longer. Some from the top of my head are: CreateCorpus, IndexData, FileUpload, Query/Summarization.
Re 2: 30S seems to be a long enough timeout for most of the calls I mentioned above.

Re 1: This does apply to all REST calls, but we would like to mention this ideally for calls that can potentially take longer. Some from the top of my head are: CreateCorpus, IndexData, FileUpload, Query/Summarization.
Re 2: 30S seems to be a long enough timeout for most of the calls I mentioned above.

Thank you for the details! I will update the docs and attach a PR to this issue.


Clipping/mis-rendering of SVG paths

I'm trying to chase down an issue I'm seeing when using bodymovin on an AE composition with some vector shapes. The layer seems to be getting clipped and/or the stroke not properly rendered. Best described with a screenshot - notice the left-most line appears clipped as if the stroke width was not taken into account. The rounded end of the line is also squared off in the bodymovin export. The same thing is happening on the right of the graphic.

As best as I can tell, this is happening in the export/conversion process, not the player? I can have a go at a pull request if you can point me at any likely locations in the code to start looking

are you using some type of matte masking?
can you share the .aep?

A colleague has made the animations, so I'm not sure.. maybe?
icon-animations.aep.zip

It's an issue with strokes and svg filters.
Browsers don't handle well that combination.
Instead of using a fill effect on the strokes, you should instead use a stroke color in the shape layer itself.
It will be much more performant also.

Thank you for the fast response, I will relay the advice!


Cache for loaded translations

I'm submitting a ...  (check one with "x")
[ ] bug report => check the FAQ and search github for a similar issue or PR before submitting
[x] support request => check the FAQ and search github for a similar issue before submitting
[ ] feature request

Current behavior
I'm using exensively this module for my translations. Currently, I'm downloading the translations using an api call, not an static json file. It works very well and I can add new labels or change translations from a DB dinamically. But now I'm currently seeing that every component that uses the service or pipe, makes a call to the api, without caching the data, even if the language is the same. Currently, there are like 30 calls to the api at least.
Expected/desired behavior
I'm expecting that the factory only makes a call per language and then caches the data until language changes.
Reproduction of the problem
Appoint to an external api for getting non-static translations.
What is the expected behavior?
What is the motivation / use case for changing the behavior?
Performance
Please tell us about your environment:
Windows 10
Visual Studio Code

ngx-translate version: 7.2.0

Angular version: 4.3.6

Browser: [all ]

Currently,

I'm downloading the translations using an api call, not an static json file.
that's great, could you teach to me how to implement solution 🙏 pra

@Micha-kun why don't you just create a service for fetching via api, then cache it on the first fetch.


[Feature Request] Ability to mock the ID generator

Existing Component
Yes
Component Name
All
Description
I am using Jest and its snapshot feature to test some views are correctly generated.
However, as Element UI randomly generates IDs for components, the snapshot are non-deterministic, and the tests fails.
Here is an example of the html rendered by a collapse component :
          <div role="tabpanel"
                 aria-hidden="true"
                 aria-labelledby="el-collapse-head-7804"
                 id="el-collapse-content-7804"
                 class="el-collapse-item__wrap"
                 style="display: none;"
            >

The aria-labelledby, and id attributes are randomly generated, hence non-deterministic, and make the snapshot test fail.
Would it be possible to allow the overriding of the generation of id.
Or maybe, is it already possible ?

I have moved this to product backlog, so i would close this issue.

@liril-net @furybean
Is this function addition already supported?
I can't find the document, Issue, or PR and I don't know how to use it, so please let me know.

Agreed! I am running into this issue as well in Element UI plus. Snapshots are a very powerful feature for Jest and it is unfortunate that Element-ui does not support them.


strip slash from url when adding new endpoint

Victor i tried the quick fix for urls. (dev branch) got this.
Traceback (most recent call last):
              File "/home/user/.local/share/QGIS/QGIS3/profiles/default/python/plugins/stac_browser/controllers/add_edit_api_dialog.py", line 75, in on_api_success
              self.hooks['add_api'](api)
              File "/home/user/.local/share/QGIS/QGIS3/profiles/default/python/plugins/stac_browser/controllers/configure_apis_dialog.py", line 77, in add_api
              api = api.strip("/")
             AttributeError: 'API' object has no attribute 'strip'

I have noticed the same issue with most urls if trailing slash is included.

right, my fault. Thanks for the fix


lychee v0.17.0 cannot handle URL fragment

I find a regression when updating lychee from v0.16.1 to v0.17.0
Given the following two HTML files:
website/dist/index.html
<!DOCTYPE html>

    <title>Index</title>

index

    About

website/dist/about.html
<!DOCTYPE html>

    <title>About</title>

about

Lychee v0.16.1 works fine:
~/Downloads/lychee_0_16_1 --base ./website/dist/ './website/dist/**/*.html' --include-fragments
  1/1 ━━━━━━━━━━━━━━━━━━━━ Finished extracting links                                                                                🔍 1 Total (in 0s) ✅ 1 OK 🚫 0 Errors

But lychee v0.17.0 fails:
~/Downloads/lychee_0_17_0 --base ./website/dist/ './website/dist/**/*.html' --include-fragments
  1/1 ━━━━━━━━━━━━━━━━━━━━ Finished extracting links                                                                                Issues found in 1 input. Find details below.

[website/dist/index.html]:
   [ERROR] file:///about.html%23about

🔍 1 Total (in 0s) ✅ 0 OK 🚫 1 Error

My environment: macOS v15

I'm not sure what changed between 0.16.1 and 0.17.0, but I suspect it relates to --base modeling Base URL, while probably what you want in this case is the upcoming --root-dir

--root-dir in v0.18.0 works well for my case by the way. Thanks!


[Bug]: An error occurred when creating an Oracle backup database with OB version 3. x

ODC version
odc430
OB version
ob3.x
What happened?

What did you expect to happen?
Prompt for setting set obunread_consistency='WEAK '
How can we reproduce it (as minimally and precisely as possible)?
Create an Oracle link with OB version 3. x
Anything else we need to know?
No response

I have fixed this issue and there is a new one here, please notice #2643

pass


[FR] Support app module denpends on another module which all have C/C++ code.

Is your feature request related to a problem? Please describe.
I have many modules in an android studio project,  module-a is a base module whitch has C/C++ library, and other modules depend on module-a, and other modules also have C/C++ and thire build depend on module-a header files and link use libmodule-a.
Describe the solution you'd like
When module-a use prefab publish, other modules use prefab, the build will all finish.
Additional context
I need to run all modules once all, and publish them to maven center, and now both run and publish not work, because of can't find link so file.

This sounds like you're asking for an AGP feature, not a prefab feature. Prefab already supports module and package dependencies. File a bug against AGP at http://b.android.com (the C++ component, I guess).

OK, I will create a issue on issue tracker.


[QUESTION] libarchive vs ark cookbook?

Does the ark cookbook supersede the libarchive cookbook in terms of functionality?  I used to use libarchive but it does not seem supported any more.

In terms of features/functionality it does supersede libarchive. The advantage of libarchive is that it uses ffi-libarchive and as such is Pure ruby. The cookbook can be considered unmaintained so probably best to close this https://github.com/reset/libarchive-cookbook/issues/27


Determine routing strategy for react

We need to determine a routing strategy to use for this project. The approach should handle nested routes.

Going to use react router.

Turns out that the hookrouter solution is and will never be supported via typescript. There was someone else who added the type definitions, but it isn't being kept up to date. I am going to look into another solution.


[front] enh: add browse action

https://github.com/dust-tt/dust/issues/5469
https://github.com/dust-tt/dust/issues/5470
https://github.com/dust-tt/dust/issues/5471
https://github.com/dust-tt/dust/issues/5473
Description
Adds a new browse action to fetch a list of urls.
Risk

New action is hidden in feature flag,
Code can be rolled back

Deploy Plan
Deploy new table in migration, then deploy front

As discussed IRL, testing before merge requires the credentials part in prod (+ maybe some other stuff) => a separate PR for that, then test, then we can merge this one pending resolution of above coms

As discussed IRL, testing before merge requires the credentials part in prod (+ maybe some other stuff) => a separate PR for that, then test, then we can merge this one pending resolution of above coms

Here : https://github.com/dust-tt/dust/pull/5597/commits/e771b9456ab1865a7a4ecc76d11976d0649f426f
only credentials required, also added migration to create table before merging this one


[BUG] bar "offset" attribute is not supported as described in src/config.yaml

Expected Behavior
in order to support multiple bars, bar offset attributes should be supported
Current Behavior
exception:
bars:

yasb-bar:

offset:

unknown field

Possible Solution
support bar offset attribute
Steps to Reproduce

add bars into config.yaml wil offset, such:
bars:
yasb-bar:
enabled: true
screens: ['*']
class_name: "yasb-bar"
alignment:
position: "top"
center: false
blur_effect:
enabled: false
acrylic: false
dark: false
window_flags:
always_on_top: false
windows_app_bar: true
dimensions:
width: "100%"
height: 32
offset:
x: 0
y: 0
padding:
top: 0
left: 0
bottom: 0
right: 0
widgets:
#left:   ["komorebi_workspaces", "komorebi_active_layout", "active_window"]
left:   []
center: ["clock"]
right:  []
run yasb
get exception

and thank you for the great peace of software!

having the same issue did u resolved it?

No.
I stopped using yasb,
I switched to GlazeWM which has its own bar.


Why on earth these info shows here

Microsoft PowerToys version
0.71.0
Installation method
Microsoft Store
Running as admin
Yes
Area(s) with issue?
General
Steps to reproduce

✔ Expected Behavior
No response
❌ Actual Behavior
No response
Other Software
No response

Hi @sayemon10
That's really strange. Do you see the same in Control Panel > Programs and Features?

/needinfo


notebook blogs for svm

Please check if the PR fulfills these requirements

[ ] The commit message follows our guidelines or the commits will be squashed into one.
[ ] Tests for the changes have been added.
[ ] Docs have been added / updated.

What kind of change does this PR introduce?
...

Does this PR introduce a breaking change?

This PR fixes/implements the following -

[ ] Bug 1
[ ] Bug 2
[ ] Feature 1
[ ] Feature 2
[ ] Breaking changes

Closing issues
...

Rendered at http://nbviewer.jupyter.org/github/jellAIfish/fromscratchtoml/blob/62175887d32131b679a68954f99474c2a1bbd04a/docs/notebooks/svm_from_scratch.ipynb

Not now.


bugfix: jerry_get_object_native_pointer: accept NULL for out_native_info_p

With the jerry_set_object_native_pointer() the native_info_p is optional, so I think it makes sense that it is optional for the getter as well.
JerryScript-DCO-1.0-Signed-off-by: Martijn The<EMAIL_ADDRESS>

@martijnthe Please rebase to the master.

Rebased + addressed comment.


[WIP] EMD 1d

Hi there,
I started coding a specific EMD for mono-dimensional case (i.e. when sorting both arrays is enough).
Doc is missing for the moment (will do that asap), but a basic implementation that covers the non uniform weight case and tests that checks if the results are coherent with EMD are already there.
On my machine, I ran the following timing test:
>>> n = 20000
>>> m = 3000
>>> u = np.random.randn(n, 1)
>>> v = np.random.randn(m, 1)
>>> ot.tic(); ot.emd_1d([], [], u, v, metric='sqeuclidean'); ot.toc()
array([[0., 0., 0., ..., 0., 0., 0.],
       [0., 0., 0., ..., 0., 0., 0.],
       [0., 0., 0., ..., 0., 0., 0.],
       ...,
       [0., 0., 0., ..., 0., 0., 0.],
       [0., 0., 0., ..., 0., 0., 0.],
       [0., 0., 0., ..., 0., 0., 0.]])
Elapsed time : 2.3728668689727783 s
2.3728668689727783
>>> ot.tic(); M = ot.dist(u, v, metric='sqeuclidean'); ot.emd([], [], M); ot.toc()
RESULT MIGHT BE INACURATE
Max number of iteration reached, currently 100000. Sometimes iterations go on in cycle even though the solution has been reached, to check if it's the case here have a look at the minimal reduced cost. If it is very close to machine precision, you might actually have the correct solution, if not try setting the maximum number of iterations a bit higher
/Users/tavenard_r/Documents/costel/src/POT/ot/lp/__init__.py:104: UserWarning: numItermax reached before optimality. Try to increase numItermax.
  result_code_string = check_result(result_code)
array([[0., 0., 0., ..., 0., 0., 0.],
       [0., 0., 0., ..., 0., 0., 0.],
       [0., 0., 0., ..., 0., 0., 0.],
       ...,
       [0., 0., 0., ..., 0., 0., 0.],
       [0., 0., 0., ..., 0., 0., 0.],
       [0., 0., 0., ..., 0., 0., 0.]])
Elapsed time : 8.67806887626648 s

Romain

Thank you romain, this is nice.
Is it me of Cython is not particularly fast (2 sec for n=20000?)? it is probably due to the use of the dist function, you should probably implement it in cython for squared and absolute value and use the dist only for weird stuff ;)
Rémi.

If I change to the following:
        if metric == 'sqeuclidean':
            m_ij = (u[i, 0] - v[j, 0]) ** 2
        else:
            m_ij = dist(u[i].reshape((1, 1)), v[j].reshape((1, 1)),
                        metric=metric)[0, 0]

I get the same timings (in the order of 2secs)...
The slow part seems to be when we deal with G. If I remove all the stuff related to G I get:

Elapsed time : 0.0061719417572021484 s

I will check is using a sparse representation for G helps

Also, new timings (for larger problem than above) are:
>>> import ot
>>> import numpy as np
>>> from scipy.stats import wasserstein_distance
>>>
>>> n = 20000
>>> m = 30000
>>> u = np.random.randn(n)
>>> v = np.random.randn(m)
>>>
>>> ot.tic(); _ = wasserstein_distance(u, v); _ = ot.toc()
Elapsed time : 0.012831926345825195 s
>>> ot.tic(); _ = ot.emd_1d([], [], u, v, metric='euclidean', dense=False); _ = ot.toc()
Elapsed time : 0.04144096374511719 s
>>> ot.tic(); M = ot.dist(u.reshape((-1, 1)), v.reshape((-1, 1)),
...                       metric='euclidean'); _ = ot.emd([], [], M); _ = ot.toc()

RESULT MIGHT BE INACURATE
Max number of iteration reached, currently 100000. Sometimes iterations go on in cycle even though the solution has been reached, to check if it's the case here have a look at the minimal reduced cost. If it is very close to machine precision, you might actually have the correct solution, if not try setting the maximum number of iterations a bit higher
/Users/tavenard_r/Documents/costel/src/POT/ot/lp/__init__.py:106: UserWarning: numItermax reached before optimality. Try to increase numItermax.
  result_code_string = check_result(result_code)
Elapsed time :<PHONE_NUMBER>843872 s

We are a bit slower than scipy's implementation, not sure whether this is due to Cython or to the fact that scipy do not deal with G :/

@rtavenar did you run "cython -a" on the pyx file to see if it's white (no yellow slow python lines)?

@agramfort

Did not know this one, thanks for the tip
I've changed the np.abs, now the only yellow lines I get are the return, the np.zeros lines and the cdist, but I do not know how to remove these ones.

you cannot remove yellow lines of np.zeros or return.
For cdist either you can directly call blas functions from scipy or you need
to code the metrics in cython

I've had a look there, could not find obvious matches for distances, but maybe it's not the right place :/
Regarding coding the metrics in Cython, this is what I have done for Euclidean distance and Squared Euclidean distance up to now. The question is: should I code all of them even if they are unlikely to be used, or only a subset?

hello, I think those two are OK just be clear in the documentation that the others are slower and use cdist (such a slow function btw ;) )
Rémi

OK, and I'll also have to be clear that only strings are accepted as metrics for emd_1d

OK, so now I added proper docstrings. Let me know if something is missing or should be changed.

This is great, thank you @rtavenar for the code and optimization.
I will merge it now.


How to get the joint point coordinates

Hello, this works great. But I have a question, how to get the reconstructed joint point coordinates?

Hi @CapenZen
I'm not sure if I understand. Could you elaborate what you mean?

Sorry, maybe my expression was not very clear. What I mean is that for in-the-wild videos, can I get specific three-dimensional joint point coordinates of the human body similar to [1,2,3]?

@CapenZen As I explained in issue 4, we're dealing with an ill-posed problem. That is, a tall person away from the camera has the same 2D projection scale as as short person close to the camera. Therefore, similar to MotionBERT and LCN, the model output is (lambda × actual 3D pose) instead of directly outputting it. (The way to compute lambda is explained in issue 4. Please read the image I attached there from the LCN paper).
So for the in-the-wild videos, when the look at the model output, there are two things that you can notice:

The output is normalized and is in range [-1, 1]. It can be denormalized the same way I explained in issue #4.
The output is multiplied by a factor lambda. Unfortunately it is impossible to find the value and multiply by 1/lambda to get the actual coordinates because of dealing with an ill-posed issue.

Note that other works like STCFormer or PoseFormerV2 do not consider this lambda and the model outputs the actual pose coordinates without any lambda. But that means the model considers that the person has the same distance to the camera as the training data, which does not happen on in-the-wild examples.
The MotionAGFormer trained on MPI-INF-3DHP also does not consider any lambda. So after the denormalization it is what the model estimates about the actual scale (which is biased by the distances it has seen in training data and is not perfect).

Thank you for your patient explanation. My understanding of this knowledge has deepened.


Error in decoration provider treesitter/highlighter.win

I installed the "make" highlighter and I seem to be getting the following error:
Error in decoration provider treesitter/highlighter.win:
Error executing lua: /usr/share/nvim/runtime/lua/vim/treesitter/query.lua:248: Query error at 9:4. Invalid node type "heredoc_end":
  (heredoc_end) @injection.language)
   ^

stack traceback:
        [C]: in function '_ts_parse_query'
        /usr/share/nvim/runtime/lua/vim/treesitter/query.lua:248: in function 'fn'
        /usr/share/nvim/runtime/lua/vim/func/_memoize.lua:54: in function 'fn'
        /usr/share/nvim/runtime/lua/vim/func/_memoize.lua:54: in function 'get'
        /usr/share/nvim/runtime/lua/vim/treesitter/languagetree.lua:132: in function 'new'
        /usr/share/nvim/runtime/lua/vim/treesitter/languagetree.lua:503: in function 'add_child'
        /usr/share/nvim/runtime/lua/vim/treesitter/languagetree.lua:388: in function '_add_injections'
        /usr/share/nvim/runtime/lua/vim/treesitter/languagetree.lua:440: in function 'parse'
        /usr/share/nvim/runtime/lua/vim/treesitter/highlighter.lua:350: in function </usr/share/nvim/runtime/lua/vim/treesitter/highlighter.lua:345>

Please advise.

I'm having the same problem.

I had encountered this as well. In my case the fix was to install the bash plugin. See also: https://github.com/nvim-treesitter/nvim-treesitter/discussions/5690#discussioncomment-7610874


Setup project structure

Scaffold the project's initial folder structure

Project structure outline
NB: This is if we are to use Server-Side-Rendering, which in theory, should make deployment easier
banking-app/
├── backend/                          # Spring Boot application
│   ├── src/main/java/com/banking/    # Backend Java source code
│   │   ├── controller/               # REST API controllers
│   │   ├── service/                  # Business logic
│   │   ├── model/                    # Domain models (Account, Transaction, User)
│   │   ├── repository/               # Database access (e.g., JPA Repositories)
│   └── src/main/resources/           # Application resources
│       ├── static/                   # React build files (served if needed)
│       ├── application.yml           # Spring Boot configuration
│       └── templates/                # Thymeleaf (optional for fallback views)
│
├── frontend/                         # React SSR setup
│   ├── components/                   # React components
│   ├── pages/                        # React pages (SSR-rendered)
│   ├── public/                       # Static assets (images, CSS, etc.)
│   ├── server/                       # Node.js server for SSR
│   │   ├── server.js                 # Express.js server for React SSR
│   │   └── react-build/              # React build output for production
│   ├── package.json                  # React project dependencies
│   └── next.config.js (if using Next.js)
│
├── docker-compose.yml                # Container setup for easier deployment
└── pom.xml                           # Spring Boot dependencies


HDDS-9249. Intermittent failure in TestRootCARotationManager#testProcessingCheck

https://issues.apache.org/jira/browse/HDDS-9249
Test the TestRootCARotationManager in personal repo with 10 * 10 iteration.
https://github.com/ChenSammi/ozone/actions/runs/6147386344/job/16678776945

@adoroszlai , could you help to review this patch? I run it locally, the total execution of testProcessingCheck remains all most the same.

Thanks @ChenSammi for working on this.

Test the TestRootCARotationManager in personal repo with 10 * 10 iteration.
https://github.com/ChenSammi/ozone/actions/runs/6147386344/job/16678776945

TestRootCARotationManager is a unit test, integration.sh does not run it.  Thus 10x10 iterations executed 0 tests in this case.
Try replacing it with unit.sh (also need to replace job name integration with unit).

Tested it locally.
@adoroszlai , could you help take another look?

@adoroszlai , thanks.


Change dnx451 -> net451

React to the change in commit https://github.com/aspnet/Localization/commit/c6bf98c9b7be4641282fe26a13972fbd02100c4b
BTW why the changes doesn't include the samples and unit tests?
/cc @pranavkm @kirthik

This no longer needed after PR #130


Feature/allow inclusive enddates

This PR introduces the addition of a inclusiveEndDates arguments for Gantt chart definitions.
Adding this argument will add 1 day to all the end dates, effectively treating them to be inclusive.
Relevant Issues:

https://github.com/knsv/mermaid/issues/868

Tasks:

[x] write specs for inclusive end dates
[x] add inclusive end date functionality to ganttDb.js
[x] write specs for gantt.jison parser improvements
[x] add definitions to the gantt.jison parser for inclusive end dates
[ ] add e2e tests to test this new functionality

Can't get the e2e tests running on my machine so I'm not sure if I'll be able to do this last task.

Example Usage:
gantt
      title Inclusive end dates (Manual date should end on 4th)
      dateFormat YYYY-MM-DD
      axisFormat %d
      inclusiveEndDates
      section Section1
         2 Days: 1, 2019-01-01,2d
         Manual Date: 2, 2019-01-01,2019-01-03

Pull Request Test Coverage Report for Build 800

38 of 58 (65.52%)  changed or added relevant lines in 2 files are covered.
4 unchanged lines in 2 files lost coverage.
Overall coverage increased (+0.3%) to 53.164%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

src/diagrams/gantt/ganttDb.js
21
23
91.3%

src/diagrams/gantt/parser/gantt.js
17
35
48.57%

Files with Coverage Reduction
New Missed Lines
%

src/diagrams/gantt/ganttDb.js
1
70.54%

src/diagrams/gantt/parser/gantt.js
3
45.07%

Totals

Change from base Build 795:
0.3%

Covered Lines:
2366

Relevant Lines:
4411

💛  - Coveralls


sfdx force:org:create command fails

Summary
sfdx force:org:create command fails to create Scratch org.
Steps To Reproduce:
Repository to reproduce: salesforce/salesforcedx:latest-full

We are using salesforce/salesforcedx:latest-full in our CI/CD pipelines
In our pipeline we authenticate DevHub
Create Scratch Org

Expected result
Scratch org gets created and we display scratch org info
Actual result
Scratch org is not getting created and sfdx force:org:display -u (alias) command is failing to display info

when you open an issue, please provide something we can reproduce using sfdx commands and the indicated output from sfdx version --verbose.
Right now, I see some script or wrapper that's probably wrapping sfdx commands, which we have no way of verifying its correctness.
Is something wrong with your script? Your config?  Your CI environment?

Hi Shane - As we are using docker I could find version in our pipelines. I found this on the docker SALESFORCE_CLI_VERSION=7.160.0. Below is what I have it in my mac.
CLI Version :
sfdx-cli/7.160.0
Architecture:
darwin-x64
Node Version :
node-v16.16.0
Plugin Version:
@oclif/plugin-autocomplete 0.3.0 (core)
@oclif/plugin-commands 1.3.0 (core)
@oclif/plugin-help 3.3.1 (core)
@oclif/plugin-not-found 1.2.6 (core)
@oclif/plugin-plugins 1.10.11 (core)
@oclif/plugin-update 1.5.0 (core)
@oclif/plugin-warn-if-update-available 1.7.3 (core)
@oclif/plugin-which 1.0.4 (core)
@salesforce/sfdx-plugin-lwc-test 1.0.0 (core)
alias 2.1.0 (core)
apex 1.1.0 (core)
auth 2.2.2 (core)
community 2.0.0 (core)
config 1.4.14 (core)
custom-metadata 2.0.0 (core)
data 2.0.4 (core)
generator 2.0.2 (core)
info 2.0.1 (core)
limits 2.0.1 (core)
org 2.0.3 (core)
salesforce-alm 54.6.2 (core)
schema 2.1.1 (core)
sfdx-cli 7.160.0 (core)
signups 1.2.0 (core)
source 2.0.7 (core)
telemetry 2.0.0 (core)
templates 55.0.0 (core)
trust 2.0.1 (core)
user 2.1.0 (core)
OS and Version:
Darwin 21.5.0
Scripts that we have are working since long and they haven't been changed. They last worked today at 11CET and started failing after that.

Can you produce the error without your scripts, using just CLI commands?

With just CLI it works in my local VS Code.

DebugLog.txt
Also attaching logs from Pipelines if it helps.


Missing guard for duplicate class method declaration

Following class definition:
class a
  hello: ->
  there: ->
  hello: -> 2

is compiled to following Javascript code without any exception thrown:
var a;
a = (function(){
  a.displayName = 'a';
  var prototype = a.prototype, constructor = a;
  a.prototype.hello = function(){};
  a.prototype.there = function(){};
  a.prototype.hello = function(){
    return 2;
  };
  function a(){}
  return a;
}());

...which doesn't prevent us erroneous re-definition of a method. I would expect a proper error, like duplicate property definition in an object:
a =
  hello: ->
  there: ->
  hello: -> 2

# => duplicate property "hello" on line 4

Would you consider adding such a guard in class definitions?

That makes sense to me. I can't think of any reason why it should be one way for classes but another way for objects.


shouldInterceptAjaxRequest no call

initialOptions: InAppWebViewGroupOptions(
crossPlatform: InAppWebViewOptions(
debuggingEnabled: true,
javaScriptCanOpenWindowsAutomatically: true,
useShouldInterceptAjaxRequest: true,
useShouldInterceptFetchRequest: true,
                )
            ),

            shouldInterceptAjaxRequest: (InAppWebViewController controller, AjaxRequest ajaxReq) async {
              print("ajaxReq.url: ${ajaxReq.url}");

              ajaxReq.headers.setRequestHeader("token", DeviceManager.getDeviceToken());
              ajaxReq.headers.setRequestHeader("deviceid", DeviceManager.getDeviceId());

              return ajaxReq;
            },

I don't have enough information about your problem. I need a more detailed description.
An ajax example is available inside the example/test_driver folder: https://github.com/pichillilorenzo/flutter_inappwebview/blob/master/example/test_driver/in_app_webview_ajax_test.dart
The corresponding server-side (Node.js) that manages the ajax request is here:
https://github.com/pichillilorenzo/flutter_inappwebview/blob/f569e369f433d218dac78f43462c83be1c5c4b1d/nodejs_server_test_auth_basic_and_ssl/index.js#L164

This problem appears on Android phones

Did you get some javascript errors from debugging (chrome://inspect/#devices)??
However, without any code to test, I can do nothing about it, because I can't reproduce your error.
As I said, the ajax example if linked to you is working.
If the ajax request you are trying to intercept has been made before the onLoadStop event (that is the onPageFinished event on Android platform), then, on Android, there isn't anyway to intercept it unfortunately!
If you can control the javascript code of the webpage your showing, then the ajax request should be made after the DOM is ready, otherwise you can't intercept it.

@pichillilorenzo  Thanks, indeed as you said at the end

@pichillilorenzo  hi,I have a problem again, sometimes it won't be called on the Android device. The URL is https://h5.jimmychat.com/active/#/pages/index/index, very sad, please help me.

Could you explain to me where are your Ajax calls in your code? Otherwise, I can't tell if it is working or not. What do you expect exactly in your use case?

@pichillilorenzo  in  init(), call  https://tchatweb.jimmychat.com/TChat/user/score/info,This ajax needs to set a specific request header, so I need to use the shouldInterceptAjaxRequest method to forward. Mainly, it is normally available on iOS devices.Oh hope you can understand what i'm talking about

@pichillilorenzo I finally found out that it cannot be called in China. Is it because part of the network is blocked, like the Chinese network cannot access google.com

Is there any Google service in flutter_inappwebview's js code that intercepts Ajax?


Updated add_view and change_view to reflect change_view signature change

... in Django 1.4b. This should work in the future if signatures changes again.
Use of kwargs only (without args) should be safe, because admin views are called using keyword arguments.

I think that's the right thing to do, but only on next. I applied my own, less intrusive fix on master.
Thanks!


configurable heading levels/depths in TOC

Discussed in https://github.com/shuding/nextra/discussions/2156

Originally posted by realfish August 13, 2023
Is there any configuration option to limit the depths of TOC (on the right sidebar)?

Duplicate https://github.com/shuding/nextra/issues/2965


Facing UnknownApiNameOrVersion error for GA4 API

Hi, I'm facing an UnknownApiNameOrVersion error when hitting the GA4 API. Below mentioned is the URL build code. Does anyone know how to give the version number?
self.service = build(serviceName='analytics',
version='v4', http=http)
error: googleapiclient.errors.UnknownApiNameOrVersion: name: analytics  version: v4

Thanks for providing the info, @apurtak !


Organize hackathon

[ ] what topic?
[ ] when? how long? (at least a week, as most people work fulltime)
[ ] how to handle submissions?
[ ] how to announce? only Discord, X/Twitter, ...?
[ ] how to rate the submissions to find the winners? community vs jury vote?

Test


Update scrivener to 2.0

I wasn't able to use Scrivener.HTML v1.1 with Scrivener.Ecto v1.0 because of a discrepancy in the Scrivener version. Bumping the version here looks appears to work in my app.
mix test is failing due to what looks like an issue with Pavlov (?), I got the same test error on master so I'm not 100% sure if this PR is green.
Error:
** (CompileError) test/scrivener/html_test.exs:7: function describe/2 imported from both Pavlov.Case and ExUnit.Case, call is ambiguous
    (elixir) src/elixir_dispatch.erl:111: :elixir_dispatch.expand_import/6
    (elixir) src/elixir_dispatch.erl:82: :elixir_dispatch.dispatch_import/5
    (stdlib) erl_eval.erl:669: :erl_eval.do_apply/6
    (elixir) lib/code.ex:363: Code.require_file/2
    (elixir) lib/kernel/parallel_require.ex:56: anonymous fn/2 in Kernel.ParallelRequire.spawn_requires/5

Welp, the tests look like they passed on semaphore so I suppose that was an issue with my environment.

FYI, as a temporary workaround.. you can define your scrivener dependency to override what's used elsewhere:
defp deps do
  ...
  {:scrivener, "~> 2.0", override: true},
  {:scrivener_ecto, "~> 1.0"},
  {:scrivener_html, "~> 1.2"},
  ...
end

I'm not following why we need to go to a v2.0. Thanks for fixing the compiler warning, but I dont see any need to bump the version (and prefer to do so myself rather than via PRs).

It's not necessary, I wasn't aware of the deps override key (thanks for the tip @jwarlander). I can remove the version bump and leave the warning fixes if you'd like.

Which you've already taken care of, so I'll close this.


Cannot import grafx, modules are missing

Description
Cannot import grafx because of missing modules, namely torchaudio and scipy.
How to reproduce

Install grafx by running pip install grafx (or, git clone https://github.com/sh-lee97/grafx then pip install -e . )
import grafx via python REPL (for the sake of minimal reproduction)
ModuleNotFoundError: No module named 'torchaudio'. Even if you have torchaudio installed, ModuleNotFoundError: No module named 'scipy' gets raised.

Expected behavior
Should be able to import grafx without manually resolving dependencies afterward.
How to fix

For the temporary fix, after installing grafx, run pip install torchaudio scipy.
Add torchaudio and scipy in pyproject.toml to permanently resolve this (recommended).

Environment
Python 3.10.12, Ubuntu 22.04

Created draft pull request
https://github.com/sh-lee97/grafx/pull/6

Oops, my bad. Sorry for the inconvenience and thanks for raising the issue! We approved your PR.
We are working on the GitHub workflows to automatically test and detect such issues - we will finalize this asap.


made a wireless access point using same wifi but my phone can't connect to it

I successfully made a wireless access point using same wifi but my phone can't connect to it. I am using an android phone.
I get the following message on terminal
Config dir: /tmp/create_ap.wlp3s0.conf.tp8RRixn
PID: 25003
Network Manager found, set ap2 as unmanaged device... DONE
wlp3s0 is already associated with channel 6 (2437 MHz), fallback to channel 6
Creating a virtual WiFi interface... ap2 created.
Sharing Internet using method: nat
hostapd command-line interface: hostapd_cli -p /tmp/create_ap.wlp3s0.conf.tp8RRixn/hostapd_ctrl
Configuration file: /tmp/create_ap.wlp3s0.conf.tp8RRixn/hostapd.conf
WARN: Low entropy detected. We recommend you to install `haveged'
Failed to update rate sets in kernel module
nl80211: sendmsg: Network is down
Using interface ap2 with hwaddr 74:e5:43:42:6d:70 and ssid 'okay'
recv: Network is down
and also how do I delete the access point that I have created?

I have the same problem,now I solved it
You may update your "hostapd" .
Now I use the version is 2.3-1,it can be work and my phone can connect to it.
About how to delete just press "ctrl+c" on terminal

Thanks a lot @frank0234 that worked for me :D


An error occurs after executing "Azure: Group by ARM Tag..." action

OS: Win10
Build Version: Azure Resources v0.5.7-alpha.0
Repro Steps:

F1 -> Click "Azure: Group by ARM Tag...".
Check whether no error occurs.

Expect:
No error occurs.
Actual:
An error occurs.

More Info:
This issue doesn't reproduce when executing commands "Azure: Group by Resource Group/ Location/ Resource Type".

Duplicate of #465


Cache::get method sometimes panics in due to an integer overflow in quanta::instant::Instant::now

This issue was reported by @BrynCooke on May 10, 2022 UTC via https://github.com/moka-rs/moka/issues/113#issuecomment-1122263388

The number of times needed to hit this is very low. Sometimes it happens within 10 invocations, sometimes 100.

I have been hitting this issue and am able to reproduce by running an integration test in a loop.
Looks like it is possibly related to this: https://github.com/metrics-rs/quanta/issues/61
The stacktrace is as follows:
2022-05-10T11:16:52.903652Z ERROR integration_tests: panicked at 'attempt to add with overflow', /rustc/7737e0b5c4103216d6fd8cf941b7ab9bdbaace7c/library/core/src/num/mod.rs:834:5

1: std::panicking::rust_panic_with_hook
             at /rustc/7737e0b5c4103216d6fd8cf941b7ab9bdbaace7c/library/std/src/panicking.rs:702:17
   2: std::panicking::begin_panic_handler::{{closure}}
             at /rustc/7737e0b5c4103216d6fd8cf941b7ab9bdbaace7c/library/std/src/panicking.rs:586:13
   3: std::sys_common::backtrace::__rust_end_short_backtrace
             at /rustc/7737e0b5c4103216d6fd8cf941b7ab9bdbaace7c/library/std/src/sys_common/backtrace.rs:138:18
   4: rust_begin_unwind
             at /rustc/7737e0b5c4103216d6fd8cf941b7ab9bdbaace7c/library/std/src/panicking.rs:584:5
   5: core::panicking::panic_fmt
             at /rustc/7737e0b5c4103216d6fd8cf941b7ab9bdbaace7c/library/core/src/panicking.rs:143:14
   6: core::panicking::panic
             at /rustc/7737e0b5c4103216d6fd8cf941b7ab9bdbaace7c/library/core/src/panicking.rs:48:5
   7: core::num::<impl u64>::next_power_of_two
             at /rustc/7737e0b5c4103216d6fd8cf941b7ab9bdbaace7c/library/core/src/num/uint_macros.rs:2188:13
   8: quanta::Calibration::adjust_cal_ratio
             at /home/bryn/.asdf/installs/rust/1.60.0/registry/src/github.com-1ecc6299db9ec823/quanta-0.9.3/src/lib.rs:272:25
   9: quanta::Calibration::calibrate
             at /home/bryn/.asdf/installs/rust/1.60.0/registry/src/github.com-1ecc6299db9ec823/quanta-0.9.3/src/lib.rs:226:13
  10: quanta::Clock::new::{{closure}}
             at /home/bryn/.asdf/installs/rust/1.60.0/registry/src/github.com-1ecc6299db9ec823/quanta-0.9.3/src/lib.rs:307:17
  11: once_cell::sync::OnceCell<T>::get_or_init::{{closure}}
             at /home/bryn/.asdf/installs/rust/1.60.0/registry/src/github.com-1ecc6299db9ec823/once_cell-1.9.0/src/lib.rs:974:57
  12: once_cell::imp::OnceCell<T>::initialize::{{closure}}
             at /home/bryn/.asdf/installs/rust/1.60.0/registry/src/github.com-1ecc6299db9ec823/once_cell-1.9.0/src/imp_std.rs:95:19
  13: once_cell::imp::initialize_inner
             at /home/bryn/.asdf/installs/rust/1.60.0/registry/src/github.com-1ecc6299db9ec823/once_cell-1.9.0/src/imp_std.rs:171:31
  14: once_cell::imp::OnceCell<T>::initialize
             at /home/bryn/.asdf/installs/rust/1.60.0/registry/src/github.com-1ecc6299db9ec823/once_cell-1.9.0/src/imp_std.rs:93:9
  15: once_cell::sync::OnceCell<T>::get_or_try_init
             at /home/bryn/.asdf/installs/rust/1.60.0/registry/src/github.com-1ecc6299db9ec823/once_cell-1.9.0/src/lib.rs:1014:13
  16: once_cell::sync::OnceCell<T>::get_or_init
             at /home/bryn/.asdf/installs/rust/1.60.0/registry/src/github.com-1ecc6299db9ec823/once_cell-1.9.0/src/lib.rs:974:19
  17: quanta::Clock::new
             at /home/bryn/.asdf/installs/rust/1.60.0/registry/src/github.com-1ecc6299db9ec823/quanta-0.9.3/src/lib.rs:305:31
  18: core::ops::function::FnOnce::call_once
             at /rustc/7737e0b5c4103216d6fd8cf941b7ab9bdbaace7c/library/core/src/ops/function.rs:227:5
  19: once_cell::sync::OnceCell<T>::get_or_init::{{closure}}
             at /home/bryn/.asdf/installs/rust/1.60.0/registry/src/github.com-1ecc6299db9ec823/once_cell-1.9.0/src/lib.rs:974:57
  20: once_cell::imp::OnceCell<T>::initialize::{{closure}}
             at /home/bryn/.asdf/installs/rust/1.60.0/registry/src/github.com-1ecc6299db9ec823/once_cell-1.9.0/src/imp_std.rs:95:19
  21: once_cell::imp::initialize_inner
             at /home/bryn/.asdf/installs/rust/1.60.0/registry/src/github.com-1ecc6299db9ec823/once_cell-1.9.0/src/imp_std.rs:171:31
  22: once_cell::imp::OnceCell<T>::initialize
             at /home/bryn/.asdf/installs/rust/1.60.0/registry/src/github.com-1ecc6299db9ec823/once_cell-1.9.0/src/imp_std.rs:93:9
  23: once_cell::sync::OnceCell<T>::get_or_try_init
             at /home/bryn/.asdf/installs/rust/1.60.0/registry/src/github.com-1ecc6299db9ec823/once_cell-1.9.0/src/lib.rs:1014:13
  24: once_cell::sync::OnceCell<T>::get_or_init
             at /home/bryn/.asdf/installs/rust/1.60.0/registry/src/github.com-1ecc6299db9ec823/once_cell-1.9.0/src/lib.rs:974:19
  25: quanta::get_now
             at /home/bryn/.asdf/installs/rust/1.60.0/registry/src/github.com-1ecc6299db9ec823/quanta-0.9.3/src/lib.rs:532:9
  26: quanta::instant::Instant::now
             at /home/bryn/.asdf/installs/rust/1.60.0/registry/src/github.com-1ecc6299db9ec823/quanta-0.9.3/src/instant.rs:25:9
  27: moka::common::time::Instant::now
             at /home/bryn/.asdf/installs/rust/1.60.0/registry/src/github.com-1ecc6299db9ec823/moka-0.8.2/src/common/time.rs:24:17
  28: moka::sync::base_cache::Inner<K,V,S>::current_time_from_expiration_clock
             at /home/bryn/.asdf/installs/rust/1.60.0/registry/src/github.com-1ecc6299db9ec823/moka-0.8.2/src/sync/base_cache.rs:760:13
  29: moka::sync::base_cache::BaseCache<K,V,S>::get_with_hash::{{closure}}
             at /home/bryn/.asdf/installs/rust/1.60.0/registry/src/github.com-1ecc6299db9ec823/moka-0.8.2/src/sync/base_cache.rs:170:23
  30: moka::cht::map::bucket_array_ref::BucketArrayRef<K,V,S>::get_key_value_and_then
             at /home/bryn/.asdf/installs/rust/1.60.0/registry/src/github.com-1ecc6299db9ec823/moka-0.8.2/src/cht/map/bucket_array_ref.rs:48:30
  31: moka::cht::segment::HashMap<K,V,S>::get_key_value_and_then
             at /home/bryn/.asdf/installs/rust/1.60.0/registry/src/github.com-1ecc6299db9ec823/moka-0.8.2/src/cht/segment.rs:306:9
  32: moka::sync::base_cache::Inner<K,V,S>::get_key_value_and_then
             at /home/bryn/.asdf/installs/rust/1.60.0/registry/src/github.com-1ecc6299db9ec823/moka-0.8.2/src/sync/base_cache.rs:630:9
  33: moka::sync::base_cache::BaseCache<K,V,S>::get_with_hash
             at /home/bryn/.asdf/installs/rust/1.60.0/registry/src/github.com-1ecc6299db9ec823/moka-0.8.2/src/sync/base_cache.rs:167:27
  34: moka::sync::cache::Cache<K,V,S>::get

The number of times needed to hit this is very low. Sometimes it happens within 10 invocations, sometimes 100.

@BrynCooke

Looks like it is possibly related to this: https://github.com/metrics-rs/quanta/issues/61

Yeah. It looks like so. From your stack trace, I see quanta::instant::Instant::now function is causing the panic.
https://github.com/metrics-rs/quanta/issues/61#issuecomment-1122296177

My Lenovo laptop is AMD based, and it happens pretty quickly.
It also occurred once on a CircleCI run.
My colleagues, who have M1 and Intel CPUs have not been able to reproduce.

As a workaround, I am going to add a Cargo feature to Moka to opt-out Quanta.
Also, when I have more time, I will try to reproduce the issue by using AMD based instance at Amazon EC2.

As a workaround, I am going to add a Cargo feature to Moka to opt-out Quanta.

Memo: I could reuse the codes here: https://github.com/moka-rs/moka/pull/39

As a workaround, I am going to add a Cargo feature to Moka to opt-out Quanta.

Working on this workaround via #121. Not quite finished yet.

Hi @BrynCooke,
I have published Moka v0.8.3 to crates.io. Now you can opt out Quanta crate by adding default-features = false to the Moka dependency in your Cargo.toml. For more details, please see this section of the README.
Hope this helps.
I will keep this GH Issue open until Quanta addresses the issue.

@tatsuya6502 Thanks for sorting this out so quickly!

Hi @BrynCooke — I have published Moka v0.8.4 to crates.io. It depends on the latest Quanta v0.10.0 that has the fix for the panic issue. Please try Moka v0.8.4 when you have time. Thanks!

I had a go and can no longer reproduce the issue. Thanks @tatsuya6502!

Glad to hear the issue is fixed. 👍🏻


Issue #748: Make popup dismissable

Based on polymer docs for paper-toast (https://elements.polymer-project.org/elements/paper-toast#property-noCancelOnOutsideClick) it seems this should make the popup dismissable by clicking outside of the element.
Unsure of how to test this error handler locally, so I couldn't verify.

To test it, run in dev mode (--dev switch on the command line), open the console with Ctrl+Shift+i, and run: require('electron').remote.getGlobal('Emitter').sendToGooglePlayMusic('error', "Test");
I gave your branch a quick test and it seemed to work.

Can confirm it works :+1:, can you squash this to one commit though @bjohnso5? No point in having 3 commits for 1 line worth of changes.

Gotta admit, I have no idea how to use rebase. I tried to squash, and now I see 5 commits in this PR. Any suggestions?

I'm taking a stab in the dark here, but try:
git status # to check and make sure you're only comitting one file.
git commit -am 'Commit Message' # recommit your changes
git push origin master -f # force push it to your repo, which will update here.

With this being a one line change the worst that happens is you delete the repo and try again, but this has been my method of squashing. I'm sure there are more efficient ways.

Here's a pretty good guide if you are still having problems. http://eli.thegreenplace.net/2014/02/19/squashing-github-pull-requests-into-a-single-commit

Thanks @jostrander, worked like a charm. I'll have to keep this example in my back pocket for the inevitable situation at work where I need to rewrite commit history. I'd managed to avoid it until today!

LGTM :+1:


General improvements

This PR includes a number of general improvements. See the commit list for details.
Some highlights include:

Disks are detected simultaneously and asynchronously, no longer blocking the main thread and reducing the complexity from O(n) to a theoretical O(1)
final and const are used wherever possible, including in the Disk constructor
getDisk works with all FileSystemEntities, and doesn't need them to actually exist

Thank you for improving to the code's structure, readability and documentation
Async disk scan will be very useful


naming consistency: require vs imports

We should either use require and import or requires and imports.  We currently have require and imports, which is hard to remember.  I propose that we use require and import only in the future, temporarily deprecating imports.

I think it would be quite simple to add both.
Just like we have priority and priorities.

@radumereuta the grammar accepts imports only at the beginning of a module, right? if so, there's really no reason not to add both, because it will not reserve any new identifiers in a way such that they cannot be used in rules.

How about for requires and ensures in rules?

require statements can only exist outside of modules, so that should not be an issue either.

Yes, of course, but I was trying to raise this other issue, since they are still keywords, and users won't know the difference.

The verification community uses requires/ensures for those (see http://rise4fun.com/Dafny/tutorial/guide), so we should probably also keep them like that.

Closed via #1405


Issue#243 shifted spt window

The spt window in the first night is shifted to the left in those files starting the recording at midnight. I have tested a couple of changes in g.sib.det and in g.part5 which solve the issue. I have also tested these changes in files starting at different times and the output is not affected.
Best,
Jairo

Codecov Report

Merging #253 into master will increase coverage by <.01%.
The diff coverage is 87.5%.

@@            Coverage Diff             @@
##           master     #253      +/-   ##
==========================================
+ Coverage   75.15%   75.15%   +<.01%
==========================================
  Files          63       63
  Lines        7998     8003       +5
==========================================
+ Hits         6011     6015       +4
- Misses       1987     1988       +1

Impacted Files
Coverage Δ

R/g.part5.R
82.31% <100%> (+0.05%)
:arrow_up:

R/g.sib.det.R
69.29% <80%> (-0.04%)
:arrow_down:

Continue to review full report at Codecov.

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

Thanks @jhmigueles, I am going to double check it, and if I do not encounter any issues I will merge these changes in the upcoming days.

Hi Jairo,
Issue #243 refers to the ggir google group post by Anne Backes, where she shared a milestone data file generated with GGIR part 1 and a plot from part 4. I have now processed that file with your latest code changes, but I am getting the same part4 visualisation as she.
Did you also test your solution with her file? If yes, did the visualisation look different for you?
Best, Vincent

Just checked this. The HDCZA algorithm identify correctly the time points for in and out bed estimations with the sptwindow_HDCZA function. However, times get wrong for the first day when being passed to "number of hours from last midnight" format (dependent on the initial recording time). To avoid so, I propose to directly look for the bed in and out times identified by the HDZCA algorithm in the "time" vector and then transform this to "number of hours" format (just in this first night when the initial recording time is different to midnight).
Let me know if anything is unclear.
Best,
Jairo

Thanks Jairo, All looks great. When the tests on Travis pass I will merge this in the Master and prepare new release.


[Bug] README.MD Error

What happened
README.MD description has some describe error about the Directory
+ ├─┬ electron
+ │ ├─┬ main
+ │ │ └── index.ts    entry of Electron-main
+ │ └─┬ preload
+ │   └── index.ts    entry of Electron-preload

Expected
Actually preload folder involve splash.ts not the index.ts like this
+ ├─┬ electron
+ │ ├─┬ main
+ │ │ └── index.ts    entry of Electron-main
+ │ └─┬ preload
+ │   └── splash.ts    entry of Electron-preload

You can submit a PR for fix it. 😅
Rename splash.ts to index.ts, and then modify corresponding options of vite.config.ts

Thanks for your advice i will fix it immediately


Proper use of consume()

Hi,
for e in take(10, my_iterator):
    print(e)

This prints the first 10 elements,
When I want to skip the first 10 elements, and get the next 10 elements, I want to write something like:
for e in take(10, consume(my_iterator, 10)):  # wrong code, this does not work
    print(e)

How would I do that? Or in other words: Why does consume() not return an iterator? (Related: Why is the ordering of arguments in take and consume different?)

Thank you 👍 That code example is exactly what I was looking for.
Thanks for your anwser.


ChatGPT generates "inline" math equations using the ( and ) delimiters but they do not render.

Could you please support specific LaTeX math mode delimiters for inline math mode?
inline math mode : \( and \)

I've discovered AutoLatex while trying to solve this exact issue! Here's my naive workaround:

Paste results into VSCode
Go to Edit > Replace
Turn on Regex mode (.*), Find \\\( , Replace with $$$, and click the Replace All icon. Image below for comparison

Repeat the process, but with \\\) in the Find section
Copy and paste In Google Docs, using "Edit > Paste from Markdown"
Use Auto-LaTeX as normal


Safari may not be displaying the message box correctly in SafeAndMine

On Phil's Iphone X, when he navigated to his Safe And Mine app, his saved message was not displayed until he physically tapped on the message box. This is a UI bug that may have something to do with Safari/Iphone X that needs to be fixed.

It looks like it's just a problem with safari. I haven't found a fix yet, but I'll keep looking


How to use for Smart Filter for customize views

OpenUI5 version: 1.120.4
Browser/version (+device/version): chrome/latest
We find there's the view management for Smart Filter and would like to set some default views by using Smart Filter. But we don't know how to implement it.
The following is the requirement:

There are 4 predefined views and every user will see it.(such as "View All Data", "View Today Data", "View Yesterday Data", "View 10days Data")
User can switch these predefined views. (Such as user select the "View Today Data")
User can create new views and set new views as default.
If new view is default view, user will see it as default.
If there's no default view, the page will show predefined view as the default view. (e.g. page will open "View All Data" if there's no default view)
User can set the predefined view as default.(e.g. user can set "View Today Data" as default)
User cannot modify/delete the predefined views.
User can open the specific view by url parameters. (e.g. urlxxx?view="View Yesterday Data", page will show "View Yesterday Data" view)

After I take research on Smart Filter, it seems not possible to implement.
Do you know how to use Smart Filter to implement these functionallity? or is there any other controls to do that?

Hi,
SmartFilterBar is part of SapUI5 not of OpenUI5. GitHub is a place for managing issues for OpenUI5 only.
Please, open a ticket in SAP ticket system for managing incidents, like
https://launchpad.support.sap.com/
Greetings,
Teodor


feat: estimate user operation gas smart account client action and how to guide

Pull Request Checklist

[ ] Did you add new tests and confirm existing tests pass? (yarn test)
[ ] Did you update relevant docs? (docs are found in the site folder, and guidelines for updating/adding docs can be found in the contribution guide)
[ ] Do your commits follow the Conventional Commits standard?
[ ] Does your PR title also follow the Conventional Commits standard?
[ ] If you have a breaking change, is it correctly reflected in your commit message? (e.g. feat!: breaking change)
[ ] Did you run lint (yarn lint:check) and fix any issues? (yarn lint:write)
[ ] Did you follow the contribution guidelines?

PR-Codex overview
The focus of this PR is to enhance gas estimation in the Smart Account Client by adding estimateUserOperationGas action and supporting stateOverride for gas estimation.
Detailed summary

Added estimateUserOperationGas action for gas estimation in Smart Account Client.
Introduced stateOverride for flexible gas estimation in transactions.
Updated buildUserOperation to use internal functions for better structure.
Improved documentation for gas estimation methods and usage.
Enhanced error handling and type definitions for better clarity.

The following files were skipped due to too many changes: packages/core/src/actions/smartAccount/internal/initUserOperation.ts, site/using-smart-accounts/estimate-gas/estimate-user-op-gas.md, site/using-smart-accounts/estimate-gas/estimate-user-op-gas-state-overrides.md, site/migration-guides/entrypoint-v7.md

✨ Ask PR-Codex anything about this PR by commenting with /codex {your question}

@moldy530 @dphilipson final look/stamp please. thanks!

I added a few more comments about delegateAndRevert in the thread above. Other than that, everything here looks good to me.

// https://etherscan.io/address/0x0000000071727de22e5e9d8baf0edac6f37da032
export const EntryPointAbi_v7 = [
  {
    inputs: [
      { internalType: "bool", name: "success", type: "bool" },
      { internalType: "bytes", name: "ret", type: "bytes" },
    ],
    name: "DelegateAndRevert",
    type: "error",
  },
  {
    inputs: [
      { internalType: "uint256", name: "opIndex", type: "uint256" },
      { internalType: "string", name: "reason", type: "string" },
    ],
    name: "FailedOp",
    type: "error",
  },

Entrypoint v7 added this to entrypoint contract, not in entrypoint v6. then, what is this update referring to?
@dphilipson

@dphilipson updated per your suggestion.
#### `stateOverride` field added to `UserOperationOverrides` for validating and estimating gas for transactions

The state overrides during transaction calls (e.g., [`eth_call`](https://docs.alchemy.com/reference/eth-call) and [`eth_estimateGas`](https://docs.alchemy.com/reference/eth-estimategas)) allow for more flexible testing and gas estimation. Specifically, `EntryPoint` v0.7 introduced the `delegateAndRevert()` helper in the `EntryPoint` contract to accommodate nodes without state overrides in `eth_call` and `eth_estimateGas` for validating and estimating gas for transactions in simulated conditions without permanently changing the on-chain state.

Although some bundlers, such as Alchemy [`rundler`](https://github.com/alchemyplatform/rundler), do support the `stateOverride` feature for user operation gas estimation for both EntryPoint v0.6 and v0.7 user operations, until Entrypoint v0.6, the [`eth_estimateUserOperationGas`](https://eips.ethereum.org/EIPS/eip-4337#-eth_estimateuseroperationgas) RPC method did not include the optional `stateOverride` parameter per specification. From EntryPoint v0.7, `stateOverride` has been officially added as an optional parameter to the gas estimation method.

To take advantage of such update, `aa-sdk` supports `stateOverride` value as part of the [`UserOperationOverrides`](/resources/types#useroperationoverrides) to be used during the gas estimation of simualted user operation transactions.


Errors during Vega template import

I created a basic Vega chart and saved it as a Deneb Template: JSON File
When I'm trying to import the template from this file I'm getting multiple error messages:
.
What is wrong? Vega code has been restored from the template correctly and when copied from the specification window and inserted again using Create New Specification->Vega->[empty] it work with no errors or warnings.

As discussed via DM, this appears to be importing a Vega template but then switching the interpreter to Vega-Lite (as evidenced by the debug pane showing Vega-Lite 5.4.0 as the interpreter). Workaround in this case would be to change to Vega in the settings pane post import but is far from ideal.
This is something that has already been resolved in the 2.0 builds, but will try to do a backport and bugfix release for 1.4.

Because version 2 is still some ways off, I'll be adding this to a version 1.5 build, to provide creators some additional bug fixes and minor enhancements in the meantime.
The work for this issue is now available in the latest 1.5 alpha build (refer here if you wish to know how to access and install alpha builds) and I will intend to submit to MS for publication and certification by the end of this week (10 Mar 2023) providing that no issues are found during testing. If you wish to validate this change, please do so and provide any further feedback before this date.

You can view the changelog for 1.5 in the staging area of the Deneb website here.


Fix/add assets to lo 2

Description

List of changes

Issue(s)
Fixes #
How to test

Go to https://deploy-preview-1810--oak-web-application.netlify.thenational.academy
Click on _______
You should see _______

Screenshots
How it used to look (delete if n/a):
{screenshots}
How it should now look:
{screenshots}
Checklist

[ ] Added or updated tests where appropriate
[ ] Manually tested across browsers / devices
[ ] Considered impact on accessibility
[ ] Design sign-off
[ ] Approved by product owner

mainly hack to display public video not on custom domain which isnt useful in production


Do not raise an error in setCWD() if the operation is not successful.

Mimic chdir() and libc conventions and return 0 uppon success or -1 otherwise.
Define those values as OK and NOTOK respectively in vm.h.

Thanks for this!


Feature Request: Add ability to filter out data based on clicking legend

It can frequently be valuable to filter down data in your chart to compare certain categories against one another. I think the natural place to do such filter on many charts is in the Legend component. By providing the option to pass a function to the Legend component dashboard creators would be given the capability to define functionality to filter their data as needed.
I've created a PR in draft state as a conversation starter as this enabling this functionality will require more than a couple changes in any component that renders the legend, and may have implications regarding the connection that's expected b/w categories and data in most chart components.

I intended to open an issue to request the feature but found this one.
it's really important to have this possibility, and in all types of charts. depending on the number of categories, it is interesting to be able to compare only specific labels, selected by the user.

https://github.com/tremorlabs/tremor/pull/711


Support batching with new Apollo Server

Instead of mangling the queries, clients which connect to Apollo server v0.2.1 and above should be able to make use of the clean batching interface that it provides. This will require adding an option to the batching network interface.

Also something to think about: https://github.com/graphql/express-graphql/pull/100

@helfer
Do we want to identify whether the server supports batching via a header or just leave it as a user-specified option?

@Poincare I think we can make it a user-defined option for the time being. Adding custom headers is a bit more work, and I think it only makes sense once this feature (and others) have settled a bit.

Sounds good. I can work on this.

Yeah I don't think we want to get into client/server feature negotiation just yet...


*: Fix documentation on how to run containerized jsonnet

Update docs after switching to pre-built jsonnet container in #100.
/cc: @brancz @metalmatze @s-urbaniak @squat

lgtm :+1:


Purchasing NFTs via mainnet fork leading to revert '815e1d64'.

We are integrating your marketplace with ours and are facing difficulties when it comes to purchasing listed NFTs.
We are using data from your API to purchase listed NFTs via a mainnet fork.
When we try to do so we are receiving the revert code 815e1d64.
Could you please provide steps explaining how to purchase listed items on Opensea via Seaport.

This revert indicates that the signature is invalid as can be seen here. Ensure that the chain id of the mainnet fork is 1 if trying to simulate mainnet orders.

Thank you arr00, testing this now.

Thank you arr00 :)
This has fixed our issue.
Have a good day.


the rationals

It was really natural and pleasant to define the rationals. I love quotients. I also love "migrate".
There is only one "sorry" in rat/basic.lean, two long expressions that are equivalent up to AC rewriting. I will wait for the simplifier for that.
@rlewis1988 @soonhokong: There are lots of apparent changes to one of Rob's files, but mostly it is the fact that my emacs deleted whitespace at the end of the line. Rob, I think you can get this by choosing "customize lean-mode" from the Lean menu and toggle the value on. Soonho, can you confirm this?

Rob, I think you can get this by choosing "customize lean-mode" from the Lean menu and toggle the value on. Soonho, can you confirm this?

@avigad and @rlewis1988: Yes, I just checked that it works. @rlewis1988, find Lean Delete Trailing Whitespace in "customize lean-mode". You can also set this option by having the following line in your .emacs file (or .emacs.d/init.el).
(setq lean-delete-trailing-whitespace t)

Ah, good to know. Thanks!


Social media share feature

Added few features

This branch has conflicts that must be resolved

This branch has conflicts resolve them

Resolved conflicts

@ANSHIKA-26 Could you please review the pull request when you get a chance? Thanks!

@aditiverma-21 kindly DO NOT IGNORE THE PR CHECKLIST , it clearly states that you should add screenshots and explain what changes you have done

Share the relevant screenshots of the features you have added

Thank you for merging the commits. I will try to add more attractive styles for the same.

@aditiverma-21 keep contributing

Yess

hello @hiteashgupta1 can you please add more labels to this ?

@ANSHIKA-26 @hiteashgupta1 I did the task but never got the labels can you please answer


Add More info functionality for each day.

The "More info" button would open up a new tab with a detailed diff between the Resource Provider JSON from the previous day, and the current day.  Then really all changes can be seen in detail.  Visualization of diff could be done using this Library: https://github.com/benjamine/jsondiffpatch, Demo: https://benjamine.github.io/jsondiffpatch/demo/index.html

More info functionality working.


Combine the normal calls from mulusing CreateSomaticPanelOfNormals

How do I generate the normals_for_pon_vcf.args when I proceed with Combine the normal calls from creating Create Somatic PanelOfNormals? Is it right to use the -V option instead of the -vcfs option after creation?

Hi @dongjuleem ,
I have detailed description of how to generate ".args" in the snakemake pipeline:
https://github.com/parklab/MosaicForecast/blob/master/Snakemake/Snakefile
Hope that helps,
Y.

@douym
Thank you for the helpful explanation:)


Cannot access SSO server

Using studip-sync fails with the following output:
Logging in...
Login failed!
Cannot access SSO server
I use the snap package with the University of Passau server.

I also have this issue, also in Passau

It seems like there have been some updates to the SSO server structure that break the current implementation.
I added some debug statements in 84f8791d9f0df72b0d703bc47f220e32bf8b6f13.
They won't solve the problem but give me enough information to fix the problem as I'm not at the University of Passau.
Please either download the commit via git (or if you are using snap, wait until snap builds and publishes a new version) and then execute studip-sync -v with the verbose flag. Please copy the full output and post it here so I can fix the issue.

Thanks for the debug statements.
They introduced a hidden csrf_token field  in the Login form, which needs to be sent in the POST reqeuest.
It can be extracted in the same way as the SAML data after the request on the "login_url"
The relevant section from the Login form page:
<form action="/idp/profile/SAML2/Redirect/SSO?execution=e4s1" method="post">
<input type="hidden" name="csrf_token" value="_a18f1b316a28577e6a97d119e6e2ee0477ef4e11" />

The POST data:
csrf_token	"_a18f1b316a28577e6a97d119e6e2ee0477ef4e11"
j_username	"xxx"
j_password	"xxx"
donotcache	"1"
_eventId_proceed	""

@cormensratio @tionis Please review if the change in #12 fixed the error.

Seems to be working for me, thanks!

Great!

Thank you for the quick response and fix. It is working for me 👍🏽.


[Feature Request] Persist information on who added a track

Currently you can see who added a track when it is first added. However once it's played and cycled to the end of the queue that is no longer visible.
It would be nice if that persisted so that when that track is played again users can tell who added it even if they weren't in the room when it was added.
Credit where credit is due, props, street cred, and all that.

---
Want to back this issue? **[Post a bounty on it!](https://www.bountysource.com/issues/11716214-feature-request-persist-information-on-who-added-a-track?utm_campaign=plugin&utm_content=tracker%2F9578624&utm_medium=issues&utm_source=github)** We accept bounties via [Bountysource](https://www.bountysource.com/?utm_campaign=plugin&utm_content=tracker%2F9578624&utm_medium=issues&utm_source=github).

Definitely intend to add this.  Related to #72

We now have an "extra details" view (click on the title of a track in the queue) - would be the perfect place for this.  Definitely still planning to add....

This is now live!  needs to show when added though, will open an issue for that.


release variable memory api ?

Click to expand!
Issue Type
Support
Source
source
Tensorflow Version
tf 2.8
Custom Code
No
OS Platform and Distribution
No response
Mobile device
No response
Python version
No response
Bazel version
No response
GCC/Compiler version
No response
CUDA/cuDNN version
No response
GPU model and memory
No response
Current Behaviour?
api can release memory variable

Standalone code to reproduce the issue
no

Relevant log output
No response

@MichoChan Sorry for the late response!
Could you provide more information on this issue reported here?  Please refer this link to know more on memory usage.
Thank you!


SQL Server: column 'auth_token' exceeds the maximum allowed for any data type (8000)

I have read the docs thoroughly before making this bug report.

[X] Yes

I have read through other open issues, and my issue is not a duplicate.

[X] Yes

What version of the library you are using?
0.2.4
Which python version are you using?
3.11
What operating system are you on?
Ubuntu
Description
Hi @MrThearMan,
When migrating on MSSQL DB encountered:
django.db.utils.ProgrammingError: ('42000', "[42000] [Microsoft][ODBC Driver 18 for SQL Server][SQL Server]The size (10000) given to the column 'auth_token' exceeds the maximum allowed for any data type (8000). (131) (SQLExecDirectW)")

is it possible to set a WebhookBase.auth_token max_length of 8000 or lower? if that is the case.
Thank you for your time.

Hi, thanks for the report!
I guess this is a MSSQL limitation. I'll look into other databases to see if other such limitations exist and lower the value appropriately. Meanwhile, you can subclass the Webhook model and change the field that way.


Rebranded from 'ferdi' to 'ferdium'

[x] Changed all occurrences of the following:

getferdi/server --> ferdium/ferdium-server
getferdi/ferdi --> ferdium/ferdium-app
getferdi.com --> ferdium.org
ferdi-server --> ferdium-server
meetfranz --> (as appropriate)
franz -->  (as appropriate)
FERDI_ -->  (as appropriate)
FRANZ_ -->  (as appropriate)
franz-server -->  (as appropriate)
ferdipw --> ferdiumpw

[x] Reviewed/changed urls to point to franz/ferdi for migration from those servers
[x] Need to still change the ascii art in docker/entrypoint.sh
[x] Need to ensure that the email addresses are correct<EMAIL_ADDRESS>and<EMAIL_ADDRESS>

@all-contributors please add @nathanaelhoun for review

@all-contributors please add @vraravam for code


Tell rn to set up on main queue to suppress warning

This would suppress the following warning.

YellowBox.js:80 Module RNGoogleSignin requires main queue setup since it overrides constantsToExport but doesn't implement requiresMainQueueSetup. In a future release React Native will default to initializing all native modules on a background thread unless explicitly opted-out of.

ref: https://github.com/wix/react-native-navigation/issues/1982

hi @mickamy , thanks for your PR! In an effort to fix this library asap, I went ahead and added a NO instead (https://github.com/react-native-community/react-native-google-signin/commit/302092200d05f6096a922092fd427cbd9718a283). Thanks!

Fair enough. Thanks for your patch!


KeyStoreManager

This structure encapsulates what used to be "rootKeyStore" "privKeyStore", and the certificate stores. These are being moved out of NotaryRepository, so that operations like listing keys, importing keys, and exporting keys aren't tied to a NotaryRepository structure.
This PR also moves the CryptoService and UnlockedCryptoService to a dedicated cryptoservice package.
Key import/export will be added in a subsequent PR.

Supersedes #54 and #55

Congrats, you now have 3 PRs with the same content :v:

GitHub wouldn't let me reopen #54 after the force-push :(


Fix PECL problem

Currently distribtest.php7_macos_x64_None is failing. This is believed to be caused by Homebrew bug which fails to install pecl correctly on MacOS.
running: make INSTALL_ROOT="/private/tmp/pear/temp/pear-build-rootoQjx91/install-grpc-1.35.0dev" install
Installing shared extensions<EMAIL_ADDRESS>running: find "/private/tmp/pear/temp/pear-build-rootoQjx91/install-grpc-1.35.0dev" | xargs ls -dils
10548148     0 drwxr-xr-x  3 root  wheel       96 Dec  4 22:53 /private/tmp/pear/temp/pear-build-rootoQjx91/install-grpc-1.35.0dev
10555091     0 drwxr-xr-x  3 root  wheel       96 Dec  4 22:53 /private/tmp/pear/temp/pear-build-rootoQjx91/install-grpc-1.35.0dev/usr
10555092     0 drwxr-xr-x  3 root  wheel       96 Dec  4 22:53 /private/tmp/pear/temp/pear-build-rootoQjx91/install-grpc-1.35.0dev/usr/local
10555093     0 drwxr-xr-x  3 root  wheel       96 Dec  4 22:53 /private/tmp/pear/temp/pear-build-rootoQjx91/install-grpc-1.35.0dev/usr/local/Cellar
10555094     0 drwxr-xr-x  3 root  wheel       96 Dec  4 22:53<EMAIL_ADDRESS>10555095     0 drwxr-xr-x  3 root  wheel       96 Dec  4 22:53<EMAIL_ADDRESS>10555096     0 drwxr-xr-x  3 root  wheel       96 Dec  4 22:53<EMAIL_ADDRESS>10555097     0 drwxr-xr-x  3 root  wheel       96 Dec  4 22:53<EMAIL_ADDRESS>10555098 16584 -rwxr-xr-x  1 root  wheel  8488844 Dec  4 22:53<EMAIL_ADDRESS>
Build process completed successfully
Installing<EMAIL_ADDRESS>
Warning: mkdir(): File exists in System.php on line 294
PHP Warning:  mkdir(): File exists in /usr/local/Cellar/php<EMAIL_ADDRESS>on line 294

Warning: mkdir(): File exists in /usr/local/Cellar/php<EMAIL_ADDRESS>on line 294
ERROR: failed to mkdir<EMAIL_ADDRESS>
Related to  https://github.com/Homebrew/homebrew-core/issues/41081

grpc_build_artifacts_multiplatform: running

Merging to fix the breakage.


Made Changes for Grammatical Typos in README.md

Issue
Made Changes for Grammatical Typos in README.md
Description
Made changes for README.md with respect to grammatical typos.
Checklist

[x] Github action build passing
[x] No merge conflicts

Sir, Shall I create an issue and make PR for the following again ?


replace OSX window title bar with custom title-bar

This pull request is intended to naively implement a custom titlebar on OSX.
I've included fallback styling to match El Capitan's titlebars, which is probably not necessary as I modified the four included themes to provide their own titlebar styling. I'll create PRs for these in their respective repositories if this PR gets considered.
I'd like to hear your thoughts on the implementation, I just banged on the code until it worked 😁
This is what it looks like by default:

This is the titlebar provided by one-dark-ui, with the oceanic-next syntax

This is one-light-ui, with the atom-light syntax

Things left to do:

[x] ✅ specs

This is a partial continuation of https://github.com/atom/atom/pull/10208 albeit without the combined title-bar-and-tabs functionality, for which this would be the groundwork. (As requested in https://github.com/atom/atom/issues/4599 for example.)

Any change to get rid of the black separation line between the title bar and the three file view? I think it would look much better.

@FezVrasta These screenshots are just a first stab at theming the title-bar and intended to show what it could look like. At any rate, you could always remove that line by tinkering with your user stylesheet :)
@simurai Would you like me to go ahead and create some PRs on https://github.com/atom/one-dark-ui/ and https://github.com/atom/one-light-ui/?

I wonder if the background should just be @base-background-color? So that "abandoned, but still in use" themes would adapt too. Then maybe make a PR with the current grey gradient for themes that try to stay close to OS X. Like unity-ui or native-ui.

I wonder if the background should just be @base-background-color?

That's a much better idea than using the grey gradient! I'll change it.

atom-dark with a title-bar gradient based on @base-background-color

What's the status of this PR? I think the top grey bar is one of the most annoying things of Atom. If I have to use it the whole day I need a pleasant environment, and that gray bar doesn't help at all 😓
In my opinion, this PR should have high priority.

I just can't stress how awesome and cool this thing looks to me. Would love to see it!

Let us know when you have the specs complete for this @brumm :grinning:

Made a few more comments.. sorry.. :innocent: I tried to make a PR to your PR, but your fork somehow didn't show up in the list to compare against.

Thanks for the comments! All good points. I'll be able to work on this in a few hours 👍

@simurai i made a couple of changes:

followed your advice and added title-bar as a headerPanel instead, which allowed me to undo the changes I made to workspace styling
moved title-bar variables into title-bars' less file and simplified the styling to a single background-color: @base-background-color. After much back-and-forth I decided against @tab-bar-background-color to preserve some contrast between title-bar and the rest of the UI, but I do like both.
cleaned up some superfluous code from title-bar-element and added a simple test

I'm for @tab-bar-background-color, it would look much better (and will allow a better integration in the existing themes)

@tab-bar-background-color

@base-background-color

As far as I can tell, the failed travis build doesn't seem to be related to my changes.
Should I squash now or after another once-over?
@lee-dohm @simurai

After much back-and-forth I decided against @tab-bar-background-color to preserve some contrast between title-bar and the rest of the UI, but I do like both.

Yeah, sounds good. For the bundled One + Atom themes, we can still thinking about customizing it. Also having a border by default should help make it clear where you can start dragging the window.
@FezVrasta In case your favorite theme doesn't look seamless, you could always make it transparent in your styles.less with:
.title-bar {
  background-color: transparent;
  border-color: transparent;
}

Should I squash now or after another once-over?

Only if you want, so far we haven't really bothered about it. Maybe one day we will enable it on the merge button, now that GitHub added that feature.

As far as I can tell, the failed travis build doesn't seem to be related to my changes.

Ya, looks like an unlreated timeout.

ok, so the styling is 👍
So hopefully we can get somebody from @atom/core to take a look at the code. And maybe @amytruong can give it another test drive?

I love this! Other than my above comment, I say it's ready for 🚀 !

Thanks for the reviews and comments, everyone! :shipit:

So, one thing that this doesn't appear to offer is the title bar icon:

This would appear to be a regression. We would either need to maintain this functionality or come up with a way to enable/disable the feature. Both are pretty complex since we don't currently have a system for either having configuration options specific to a platform or restarting when a configuration option is changed.
Is this something that you would be able to cover @brumm?

Sorry, the correct term for the thing I was calling a "title bar icon" is "proxy icon":
https://developer.apple.com/library/mac/documentation/UserExperience/Conceptual/OSXHIGuidelines/WindowAppearanceBehavior.html#//apple_ref/doc/uid/20000957-CH33-SW1

Hah, I knew I was missing something ;)
I'd hazard a guess that while it would be possible to approximate the proxy icon, it'd be hard/impossible to really get right (a cursory google search suggests that getting a file icon from node is not a solved problem for instance).
I'd be happy to work on providing a settings interface for this feature, which is something I wanted to do in a later PR anyway.
That would mean extending the config-schema with a way to

Define a platform for the setting
Indicating that this setting will need a restart/reopened window.
We could probably get away with just mentioning the fact inside the setting description, and then reacting to a setting change with a dialog.

Maybe like this:
customTitleBar:
  platform: 'darwin' # 'darwin', 'freebsd', 'linux', 'sunos' or 'win32'
  type: 'boolean'
  default: false
  description: 'Use custom window and theme-aware title-bar. Note: This currently does not include a file icon or title context menu.
This setting will require a restart to take effect.'

If 'polluting' the config namespace is a concern, we could abstract these two new use cases for the config schema to mean

Setting visibility/applicability
Setting side effects

customTitleBar:
  show: { platform: 'darwin' }
  sideEffect: ['restart-app']
  type: 'boolean'
  default: false
  description: 'Use custom window and theme-aware title-bar. Note: This currently does not include a file icon or title context menu.
This setting will require a restart to take effect.'

What's the purpose of such icon?

It's a representation (or in osx terms, proxy) of the currently opened file. Click-holding the icon allows you to drag the file out of the title bar to move/copy/alias it.
The title bar also provides a context menu of the currently opened file's parent directories on right-click.

On 27 May 2016, at 18:36, Fez Vrasta<EMAIL_ADDRESS>wrote:
What's the purpose of such icon?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.

It's a bit tricky, but couldn't Electron provide the proxy icon directly from the OS X API like it does for the window control buttons?
Probably it would be difficult/impossible to position the icon on the left of the text, but you could just put in on the right corner maybe.

@zcbenz Do you think it would be feasibly (aka not too much work) to add an API for this in Electron? And by this, I mean an API that will allow us to basically re-implement the "proxy icon" functionality. We'd ideally need access to the icon somehow, as well as being able to trigger (and position) the menu.

I can imagine positioning the native proxy icon would be pretty hard. Adding a config option, as proposed in ☝ https://github.com/atom/atom/pull/11790#issuecomment-222191019 feels more realistic. Then people can choose to use the native title bar with all its functionality or use the simple, but customizable one.
Maybe depends if platform specific configs would be beneficial for other things too? Or added just for this use-case?

Maybe depends if platform specific configs would be beneficial for other things too? Or added just for this use-case?

That's what I was wondering :)
I'll start with the simple platform version and see what others like @lee-dohm might say in the meantime.

Right, finally got around to actually doing this.
There are some wording problems in promptForRelaunch, since we're not actually relaunching Atom, for which I'll happily accept better phrasing.

I'm running into the same error message as the CI, badges.less isn't found.

Ah, seems like a bad rebase conflict resolution. Weird that the build script did not complain about this...

Looks like the cursor blink spec timed out, travis looks okay otherwise

Tried it out and works!!! 👍

There are some wording problems in promptForRelaunch, since we're not actually relaunching Atom, for which I'll happily accept better phrasing.

Should the button say "Quit Atom"? The word "Relaunch" suggest that it will quit AND re-open Atom automatically.

Should the button say "Quit Atom"? I feel like the word "Relaunch" suggests that it will quit AND re-open Atom automatically.

Thanks, I changed it to
{
  message: "You will need to relaunch Atom for this change to take effect."
  buttons: ['Quit Atom', 'Cancel']
}

We'll be able to actually relaunch Atom once we're using electron 1.2.2

Made sure that we set a sheet offset when using the custom titlebar:

Is this almost ready to merge? If so, that'd be great; I can't wait to have this.
I also think that @tab-bar-background-color looks better, fwiw, so I hope that will be configurable.
Overall, great work 👍

@simurai Hey, it'd be great if you could take a look at https://github.com/atom/atom/pull/11790/commits/3e53a03871ac25e68269f3897896fc93170ef471.
I decided to transition this instead of a hard display: none because the change happens after the fullscreen animation on OSX and is kind of abrupt.

I found a weird bug: If .tree-view-directory is overflowing .tree-view, toggling treeview visibility causes <atom-workspace> to scroll down by at least the height occluded and at most the height of the titlebar, effectively hiding the titlebar.
As far as I can tell, this is not caused by programmatically setting scrollTop of <atom-workspace>.
Screencast: http://i.imgur.com/hG6Zl2Z.gifv

@brumm Can confirm this. I observed this behaviour when building atom with @paulcbetts PR on this same feature (#10208).

Note: This currently does not include a file icon

This should read "Note: This currently does not include a proxy icon"

@brumm I found a weird bug

That feels like a deja-vu.. do you have the color-picker package installed? It looks like it's the same issue that tool-bar also has. A fix got merged, but I think it didn't get published yet?

Thanks @simurai! This was indeed the problem. I opened a new issue and asked @thomaslindstrom to release a new version which includes the fix.

@simurai @brumm Sorry, I thought I'd done this already. It's out in 2.2.0!

@simurai @brumm Just a quick update: v2.2.0 including the fix for this issue broke color-picker completely. I've published v2.2.1 that reverts the fix, but that means that this issue is still present.

Whats the latest here?

Looking forward to this

Ugh, sorry, I was trying to push a branch based on this to to atom/atom and accidentally pushed to master. I'm making some adjustments and working toward merging this, but it's not there yet.

✨  Thanks @brumm for putting this together. I'm already enjoying the more stylish title bar in my dev environment. This should go out with Atom 1.11.0-beta0 in the next release cycle. ⚡

Now to wait for theme support once this is released 😁

@onbjerg Material UI already supports it 😄

Thanks you so much @nathansobo for picking this up! I'm stoked to see this getting merged 🎉

I'm trying to implement this work in my beta (1.10.0-beta1).  When I add the below snippet to my style.less and save, restart atom, nothing happens. Is what I'm doing supposed to work?
Thanks!!
.theme-one-dark-ui {
.tree-view::before,
.tab-bar:before,
.tab.active:before {
display: none;
}
}

@maxwellzdeamon This won't land until 1.11.0, so you'll have to build from source until then if you want it.

@brumm is there any way to change the position of the window controls using CSS?

@FezVrasta The window controls are drawn by the OS and positions can't be changed indivdually.
You might get some milage out of changing titleBarStyle to hidden-inset which moves the controls a bit further down, but also increases the size of the title bar.
I always figured that we'd have to completely hide the titlebar to support the 'tabs-on-top' style proposed in #10208, and dynamically switch draggable areas on and off with css.
That's what I came up with the other day:

Thanks for the reply. If it can be helpful I posted my custom CSS on https://github.com/atom/atom/pull/10208#issuecomment-241346442

@brumm We came up with that design too - it looks great but the problem is that objects in the non-client area (i.e. the invisible bar from the traffic lights onward) cannot be dragged, so reordering tabs is busted :cry:

Couldn't we hide the title bar altogether, and create fake window buttons with CSS, as shown here? @paulcbetts @brumm

@controversial In Atom's case, window buttons are rendered by the OS and can't be changed.

@silvestreh Why couldn't that be changed? This is a PR to the atom core...

@controversial This is what the titlebar's DOM element looks like:

As you can see, the buttons aren't part of the DOM. You can't style them with CSS because the window manager, not Chrome, is rendering them.
AFAIK, you can't move, resize, nor hide them.

You can spawn a frameless window, which would hide the title bar completely, then you could emulate the buttons using custom CSS. https://github.com/electron/electron/blob/master/docs/api/frameless-window.md
Maybe this isn't available in the version of Electron atom is on though.

Maybe this isn't available in the version of Electron atom is on though.

This does indeed require a more modern Electron. We're very close to shipping #12300 though 🎉

Ah, great. I'll look forward to #12300 😃

@controversial I think it's not a good practice to replace native OS UI controls. They are a hell to maintain and avoids OS level customization and third party software control over them.

Yeah, you're probably right. Waiting for the newer Electron version will be a much better choice.

I love this addition so much! I'm just trying it out right now but I've noticed a little bit of extra black under the bar with Nuclide. Not sure if this an Atom issue or a Nuclide issue.

Not a huge issue, just thought it was worth noting.

Any plans to have something like this for other operating systems, given 57% of the Atom userbase is on Linux or Windows? 😄

You can spawn a frameless window, which would hide the title bar completely, then you could emulate the buttons using custom CSS

I agree with @sveggiani... Emulating native OS controls is tricky and very difficult to maintain. It's better to use the native APIs to render the native controls (for example, I believe the title bar of Google Chrome does it this way). This used to be reasonably common with native apps on Windows, I don't know if Electron exposes this functionality though.

@Daniel15 Electron does expose this, but only in a version of Electron that's later than the one Atom uses. I think a fake control would be good for the short term, using the native controls once we get to that version of Electron.

Now that Atom is on a modern Electron version can we do this? I really like @brumm's solution, even though the tabs can't be reordered. Hyper does this and the tabs aren't reorderable, but it never really bothers me. If this was an option that could be enabled by users, I think it'd be great to include some kind of solution in the Atom core.

I'm probably late to the party, but is there any way I can do this currently?  Is there a dev branch or beta release with this enabled?

@lacostenycoder, look for Title Bar option in latest stable Atom release

@MaximSokolov, already found that thanks!


Wraith falls over if domains have capital letters in their tag

This gsub definitely doesn't work properly under Linux, and fails to find the .pngs with capital letters in, so getting the stats falls over badly later on in the process.
Done the simplest fix of adding capital letters to the gsub.

:+1:

@darnould Can you review

LGTM :+1: