Explore adding "actions", an easy way to add static resource methods

See #252

+1
Any chance to tackle this soon ?

+1 this would be awesome.

I'll gist you my method for doing this. Perhaps it will be helpful.

That would be great

Here you go. Let me know if you have any questions. https://gist.github.com/kentcdodds/6d4616b431a079808c6f

Initial implementation already complete in js-data + js-data-angular. Closing this in favor of the feature already being available there.


Upload failed suddenly

In the middle of my note-taking session, the uploads stopped working suddenly:

I have no idea what happened, it was working fine this morning.
I already had the "Client ID" from the last update, which is why I was able to work with the plugin before.
What could be the cause of this new issue? 🤔

UPDATE: I created a second imgur account and now seems to be working. Does imgur have a image limit now? 🤔


fix: check gamma non-zero

We found not making sense to require gamma != 0; so close this one. It is okay to query the swap with only a tiny tiny amount of token.


private repo with public issues

The feature just landed in Gogs, see:
https://github.com/gogits/gogs/issues/649#issuecomment-289144558
Commit https://github.com/gogits/gogs/commit/8196430f47842fba4f227b105cd96d4b981d077d
To be ported

duplicated with #639


Include nested/flat tests

see: https://github.com/zarr-developers/zarr-python/issues/707
Most implementations support the choice of . or / as a dimension separator. The arguments for readers and writers could be: default, ., /. Auto-detection would be a potential other option depending on what is decided on 707.
Note: this may be the beginning of introducing many other dimensions to the test matrix.

I started looking at this.
zarrita: can generate either flat or nested
zarr-python: can generate v2 files with nested format when using FSStore instead of the default DirectoryStore.
xtensor-zarr: it looks like it is possible to generate flat or nested for v3, but in the v2 spec case the '.' separator is currently hard-coded.
The n5 format does not seem to support a flat storage scheme

Sounds about right. A few notes:

The n5 Java implementation does allow a choice of schemes, so we could choose to add support to the Python implementation. I haven't done so yet.
It's still poorly tested, but https://github.com/zarr-developers/zarr-python/pull/716 should allow more directory stores to use the new metadata. Not sure if we want to use this repo to test that work, or settle that work first.
Perhaps @davidbrochart would be up for adding it to v2 as well based on https://github.com/zarr-developers/zarr-python/pull/715?

Perhaps @davidbrochart would be up for adding it to v2 as well based on zarr-developers/zarr-python#715?

Sure, that should be easy.

https://github.com/zarr-developers/zarr-python/pull/716 is now merged and released. I'd assume for all participating implementations, the behavior shoudl be:

writer uses default --> .zarray with no dimension_separator --> reader uses "."
writer uses "." --> .zarray with dimension_separator="." --> reader uses "."
writer uses "/" --> .zarray with dimension_separator="/" --> reader uses "/"

and in all three cases something other than fill_value should be returned.


UI: Add 'About Us' Section

Add 'About Us' section + title in home page to match screenshot below. Link 'Learn More' to contact page

'Welcome to Essential Piece' text color: #14212B
'About Us' text color: #6EC9BE
Company Logo Overlay:

'About Us' Image:

'Learn More' Button SVG:
<svg xmlns="http://www.w3.org/2000/svg" width="281" height="70" viewBox="0 0 281 70">   <g id="shape" transform="translate(-1046 -2053)">     <path id="Rectangle_18_copy_3" data-name="Rectangle 18 copy 3" d="M1046,2053h261l-40.542,70H1046Z" fill="#6ec9be"/>     <path id="Rectangle_23" data-name="Rectangle 23" d="M1317,2053h10l-40,70h-10Z" fill="#818285"/>   </g>   <text id="LEARN_MORE" data-name="LEARN MORE" transform="translate(43.772 41.93)" fill="#fff" font-size="20" font-family="Poppins-SemiBold, Poppins" font-weight="600" letter-spacing="0.104em"><tspan x="0" y="0">LEARN MORE</tspan></text> </svg>

'About Us' Text:
I have a strong commitment to providing excellent experiences for clients. I have always been playing one or multiple sports, starting with youth soccer and basketball. When I was 13 my first weightlifting experience was at my high school weight room for football. I’ll never forget the atmosphere, environment, and life lessons I was taught. During college, I was known for staying in the campus gym and sharing my knowledge and experiences with my friends, as well as playing Collegiate Rugby. I am now experienced in personal training and group exercise instruction, with a certification from NASM as well as a B.S. in Exercise Science. I would absolutely love to be apart of your fitness journey and your story.

Updated Image:


Adding Psalm

I added a baseline with the current errors. The errorLevel 3 was suggested by psalm --init

FYI https://github.com/guzzle/guzzle/issues/2603#issuecomment-629826442

I disagree with @Tobion here.
Psalm and PHPStan are doing the same thing. PHPStan is a bit more friendly and easy to use locally. But Psalm has way more features and discovers way more things (Just look at our psalm baseline file). Psalm also integrates with github reviews better, it actually makes a review comment on the line with issues.
So, yes, Two static analysers may be too much. I dont mind using two, but if we should remove one, then PHPStan is the one to go.

The PR is ready for a review

I've been using both in my daily work. They are reporting different things and I do thing they complement each other.


Disable CURLOPT_SSL_VERIFYHOST when building with NOVERIFY=1

This makes testing with self-signed or temporary certificates easier.

Thanks for your contribution. I will need to take this up with the other maintainers to see if we want to merge this.

What's the added value of VERIFYHOST=0 if VERIFYPEER=0 already skips the certificate check as a whole (it will still build up the TLS connection, as far as I understand VERIFYPEER)?

What's the added value of VERIFYHOST=0 if VERIFYPEER=0 already skips the certificate check as a whole (it will still build up the TLS connection, as far as I understand VERIFYPEER)?

The two checks are independent of each other, so even with VERIFYPEER disabled, the host is still checked. From the curl documentation:

You typically also want to ensure that the server is the server you mean to be talking to. Use CURLOPT_SSL_VERIFYHOST for that. The check that the host name in the certificate is valid for the hostname you are connecting to is done independently of the CURLOPT_SSL_VERIFYPEER option.

That was a problem with the certificate I was using in my tests.


how can i Turn   cb_3c_3C4x_mem_bn_vgg.prototxt   into    channel_pruning.prototxt ?

how can i Turn   cb_3c_3C4x_mem_bn_vgg.prototxt   into    channel_pruning.prototxt ?

how can i turn cb_3c_3C4x_mem_bn_vgg.prototxt into channel_pruning.prototxt ?

Duplicate of #16


Why are middleware and route callables bound different?

Hey,
why is the middleware callable bound to the container and the route callable to the app? I tried to figure out why there would be two different ways to get to the stuff, but I can't :confused:
I think it would be great if I could use them both in the same way.
By the way, good job! I like Slim 3 already :smile_cat:

@sirdiego it's mostly the same since the APP redirects stuff to the container anyway.

Hey @geggleto,
I know that it is mostly the same, but it isn't exactly. On the container you can access the stored entries by $container->get('name') and $container['name']. For the application you have to use $this->name, because the get method is already used for the routes and the array access dies with this exception:
Fatal error: Cannot use object of type Slim\App as array

That's why I started thinking about the reasoning behind the different bindings.

My personal preference would be to always bind the container to the $this.
This would mean that if you wanted to use subRequest in a route callable, you'd have to use ($app), though.

+1 what @akrabat said.

I agree with @akrabat and @codeguy

Fixed. We now bind Container everywhere!

Awesome, thanks!


Parse issue titles with code

This quick fix should allow sphinx toolbox to parse issue titles with code in them. Tests were run successfully; however, I do not know how to add new tests.
Closes #91.

Thanks for your contribution. I've changed the implementation slightly to handle quote characters and I've added some tests too.
Let me know if this still works OK for you.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

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

Totals

Change from base Build<PHONE_NUMBER>:
0.006%

Covered Lines:
3850

Relevant Lines:
4230

💛  - Coveralls

Thanks for the edits @domdfcoding! Works and looks good to me!

Hooray, the tests pass!
I will backport this to v2.18.x, as the master branch is geared up for v3 which isn't ready yet.


feat: add hlog example

What type of PR is this?
feat

What this PR does / why we need it (English/Chinese):
en: add hlog example

Which issue(s) this PR fixes:

Fixes https://github.com/cloudwego/hertz/issues/293

LGTM


Drop for Phoenix-MR.

TODO:
A sample MR job.
Junit tests for the MR support code.

The relevant parts of this are already in the code base.
Seems to mostly in PHOENIX-1454


[MRG] Fix doc build

Hello @holgern! Thanks for opening this PR. We checked the lines you've touched for PEP 8 issues, and found:

In the file examples/interruptible-optimization.py:

Line 50:1: E302 expected 2 blank lines, found 1
Line 51:80: E501 line too long (88 > 79 characters)

Merged without review as it is only a simple fix in the doc and it did  not change the library itself.


Enable test_snapshot_mounted_filesystem

Longhorn 4089
Signed-off-by: Derek Su<EMAIL_ADDRESS>

I believe this is only for testing purpose, so we will not need this anymore after resolving multipath running on our infra nodes.

Yes, if the tests pass, I will enable the test function test_snapshot_mounted_filesystem in this PR.

@derekbit Need to backport to 1.3.x and 1.2.x if the case was disabled.


Unable to find version

Describe the bug
Github action failing for basic flow with cant find version error
To Reproduce
This is my current build yml picked from basic usage
urrent runner version: '2.310.2'
Operating System
Runner Image
Runner Image Provisioner
GITHUB_TOKEN Permissions
Secret source: Actions
Prepare workflow directory
Prepare all required actions
Getting action download info
Error: Unable to resolve action mongodb/mongodb-atlas-cli@main, unable to find version main
Have also tried telling it to use 0.2.0
Expected behavior
Please explain what you expected to happen.
Screenshots

Desktop (please complete the following information):

OS:MacOS
MongoCLI version [e.g. 1.2]

Additional context
May be being stupid but help appreciated

Sorry was using this snippet from basic set up
jobs:
use-atlas-cli:
runs-on: ubuntu-latest
steps:
  - name: Setup AtlasCLI
    uses: mongodb/mongodb-atlas-cli@main
  - name: Use AtlasCLI
    shell: bash
    run: atlas --version # Print Atlas CLI Version

fixed -> use @master instead of main

You seems to still have some naming discrepancies on further investigation for me current working name is
mongodb/atlas-github-action@main
which is a combination of basic setup and https://github.com/marketplace/actions/atlas-cli-github-action
Maybe worth realligning


The 'mountPath' of the PVC is not correctly pointing to the '/home/node/.n8n' path.

Hello, sorry if I am misunderstanding something, but when I deploy with Helm and enable persistence with the default SQLite database, the persistent volume is mounted at /root/.n8n, while all data is generated under /home/node/.n8n.
This implies that when I delete the pod, all data is lost. A workaround is to modify the deployment and map it to /home/node/.n8n, like this:
volumeMounts:
- mountPath: /home/node/.n8n
name: data
But modifying the deployment in this way results in a loss of Helm deployment capability.
Environment:
kubernetes on k3s
n8nio/n8n Tag: 1.6.1
Chart Version: n8n-0.11.0
Helm Version:"v3.12.3"
The values.yaml that I use:
https://pastebin.com/zYNE1CnF

maybe @bartlaarhoven, can help here, as he created the PR.

Correct, I ran into the same and created PR #49, which is merged into Chart version 0.12.0. So if you update your chart, it should work.

I confirm it works correctly after helm upgrade.
Thanks!!


ModuleNotFoundError: No module named 'websockets.handshake'

Traceback (most recent call last):
File "/usr/local/bin/wsstat", line 7, in
from wsstat.main import wsstat_console
File "/usr/local/lib/python3.6/dist-packages/wsstat/main.py", line 5, in
from wsstat.clients import WebsocketTestingClient
File "/usr/local/lib/python3.6/dist-packages/wsstat/clients.py", line 14, in
import websockets.handshake

I'm on 3.5 and instead of the websockets.handshake error I get this:
Traceback (most recent call last):
File "z:\anaconda3\envs\websocket\lib\runpy.py", line 193, in run_module_as_main
"main", mod_spec)
File "z:\anaconda3\envs\websocket\lib\runpy.py", line 85, in run_code
exec(code, run_globals)
File "Z:\anaconda3\envs\websocket\Scripts\wsstat.exe_main.py", line 4, in
File "z:\anaconda3\envs\websocket\lib\site-packages\wsstat\main.py", line 5, in
from wsstat.clients import WebsocketTestingClient
File "z:\anaconda3\envs\websocket\lib\site-packages\wsstat\clients.py", line 13, in
import websockets
File "z:\anaconda3\envs\websocket\lib\site-packages\websockets_init.py", line 1, in
from .imports import lazy_import
File "z:\anaconda3\envs\websocket\lib\site-packages\websockets\imports.py", line 90
f"{package}.{name} is deprecated",
^
SyntaxError: invalid syntax
Any ideas?

The error was caused by F-string being invalid syntax.  Seems like one of the dependencies is probably too high a version. Make sure you're using virtual environments. Something like this for https://uoa-eresearch.github.io/eresearch-cookbook/recipe/2014/11/20/conda/

same here on python 3.8 macOS

File "/usr/local/lib/python3.6/dist-packages/wsstat/clients.py", line 14, in
import websockets.legacy.handshake
there are many things to update beyond that


Update sbt-github-actions to 0.9.2

Updates com.codecommit:sbt-github-actions from 0.9.1 to 0.9.2.
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

Ignore future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "com.codecommit", artifactId = "sbt-github-actions" } ]

labels: sbt-plugin-update, semver-patch

It appears that MiMa is trying to be run, although I don't know why. It fails because it can't find a 0.27 artifact.


Failure cases handling

Currently we support only success path, we may need to support failure path too.
Exact scope of this issue is TDB.

hi pigmej, do we have some plan or proposal for this requirement? we want to use AppController for workflow tasks (k8s jobs), thanks.

@nkwangleiGIT Currently @gluke77 is doing research about that, how we can support that etc. We will have document for that somewhere in the middle of this week.
@nkwangleiGIT Could you describe your use case?

Basically, we have an idea to let user decide what to do with failures, and provide him with several options. What we plan for first iteration of this feature is:

retires with timeouts (configurable)
rollback
tear down on failure, e.g. entire graph will be destroyed after failure

In the future we plan to support more sophisticated scenarios, when you can attach custom graphs on different failures. This would allow more elegant tear down or alternative graph execution.
Anyway, it would be great to see your use cases @nkwangleiGIT , we could consider them in our design

@pigmej @zen  thanks so much for your reply.
About my user case, here are some basic info:

We're trying to create build-flow(CI) using k8s Jobs, so it's quite like the approach in your demo:
https://youtu.be/BXRToNV4Rdw?t=178
Thanks again for the great demo.
Basically, it'll do similar flow like shippable but using k8s Jobs for each build step.
http://docsv2.readthedocs.io/en/latest/config.html#test-and-code-coverage-visualization
So we need to handle the failure case of each step, including timeout, retry times, remove the latter steps after the failure point. Or even use job as the decision gateway, and make decision based on the output of job(low priority for now)
For long term, it'll be great if we can use k8s to do workflow or bpm.
BTW, we also expect AppController to support Deployment, so we can use the Dependency for application deployment and visualize these scenarios.

We'd like to contribute this project once we get familiar with this great tool, it'll be also great if you can send me some more documents to get start besides the ones in this repository, thanks!

@nkwangleiGIT Ptal #98. If you think that the document does not cover all your usecases, pls let me know.

@gluke77 it's sufficient for our current requirement, when we can get the 1st version implementation about this?
BTW, how we should handle the events about failure? or we have to watch related k8s API for this?
Thanks

@nkwangleiGIT As for implementation ETA, well the only answer that I could give is "we want it rather sooner then later". As for externally available events -- we don't have any plans yet.

@nkwangleiGIT or maybe you meant more or less some post deployment events ? And handling these errors? If yes then it's also being worked on currently.

@gluke77 got it, I'll keep a close eye on this issue and AppController project
@pigmej yes, events to add custom actions or integrate with other service will be great to simplify the work
Thanks all

General handling landed, specific cases moved to next milestone.

So far retries (w/o timeouts) #174 and 'on-error' conditional subgraphs #184 are implemented. Timeout support #199 is in progress. 'Ignore this' and 'ignore with children' will follow shortly.

'Ignore this" implemented #205

All items from that proposal are now implemented, closing it.


[Off Canvas] isSticky

I'm working with the Off Canvas menu and noticed this phrase in the docs:

isRevealed - Allow the offcanvas to remain open for certain breakpoints. Can be used with isSticky.

Well, I didn't see an option isStick so I thought I would check the JS and I found this:
/**
* Allow the offcanvas to be sticky while open. Does nothing if Sass option `$maincontent-prevent-scroll === true`.
* Performance in Safari OSX/iOS is not great.
*/
// isSticky: false,

I assume it's commented out because of the performance issues referenced.
Is there a timeline to add this into Off Canvas? I'd like to use it.

Until Safari can work smoothly with scroll events, I don't know that we'll get this in there. It worked great on Firefox and Chrome, hell, even IE9 was ok, but Safari was totally broken. By default, if you use a reveal-for class, it becomes position fixed when it's revealed, but for some reason making that work any other time is difficult.

@zurbchris - Thank you for the information.


⚠ Bofrost.be has degraded performance

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

HTTP code: 200
Response time: 2040 ms

Resolved: Bofrost.be performance has improved in d09da28 after 6 minutes.


Add support for python 3.13

Other changes:

Drop support for python 3.8
Update dependencies

@aminalaee Please take a look at this PR (and maybe squash and merge 😅)

look good to me, no actually code changes

actions/upload-artifact@v4 is not broken for matrix, it just different name.
for example, use wheels-${{ matrix.job-index }} and use wheels-* in download-artifact v4

for example, use wheels-${{ matrix.job-index }} and use wheels-* with merge-multiple (or another option name, I forget) in download-artifact v4

@trim21 Correct me if I am wrong, but I think you mean strategy.job-index from the docs

@trim21 can we get this reviewed and merged?

@trim21 can we get this reviewed and merged?

I don't have permission to merge PR

Oh. My bad. Thanks for the review and nudges by the way @trim21
I suppose we have to wait for @aminalaee

is there a classifiers for Python39+ ？

Not that I know of

@aminalaee Thank you. If I may ask, when can we expect the new release. This is currently blocking one of my projects from transitioning to Python 3.13

I think we should at least solve https://github.com/aminalaee/uuid-utils/issues/69 by merging https://github.com/aminalaee/uuid-utils/pull/70 or remove SafeUUID from pyi


fix: Match root weight for patterns without edges

feat: SinglePatternMatcher implements PortMatcher for weighted graphs
fix: LinePattern is never empty

How about this?


Complex members not part of complexes in SBGN output

When PC is queried to return the result in SBGN, it used to be the case that complex members were part of their parent/owner complexes (in SBGNML, member glyphs were simply listed inside their owner complex glyphs) but that no longer seems to be the case. From the layout of the returned pathway, it seems like members are properly contained / nested in their parent complexes but that is not the case!
Also see https://github.com/iVis-at-Bilkent/newt/issues/176

@ugurdogrusoz , could you please provide a query example (preferably, not too large SBGN-ML result), which e.g. could be used for the jUnit test...

I am not convinced that a PC graph query (returning SBGN, which is simply the result of converting and applying layout to the BioPAX sub-network using Paxtools' sbgn-converter) does it wrong: I can see, e.g., complexes look good (see in the target/classes folder) after the jUnit test:
        <glyph class="complex" id="Complex_2f962a315f3fb9e5f65d1ab529473d1d" compartmentRef="early_endosome_membrane">
            <label text="BMP:p-BMPR:Endofin:SMAD1/5/8"/>
            <bbox w="105.0" h="129.00002" x="136.62164" y="177.60197"/>
            <glyph class="macromolecule" id="Protein_ae756c0799a473296ae3d213c27ea5df_Complex_2f962a315f3fb9e5f65d1ab529473d1d" compartmentRef="early_endosome_membrane">
                <label text="Type II receptor"/>
                <bbox w="48.0" h="25.0" x="138.62164" y="249.60197"/>
            </glyph>
            <glyph class="macromolecule" id="Protein_33880690357c646d9f33c4462a837720_Complex_2f962a315f3fb9e5f65d1ab529473d1d" compartmentRef="early_endosome_membrane">
                <label text="BMP2"/>
                <bbox w="48.0" h="25.0" x="138.62164" y="179.60197"/>
                <glyph class="state variable" id="Protein_33880690357c646d9f33c4462a837720_Complex_2f962a315f3fb9e5f65d1ab529473d1d.info.1" compartmentRef="early_endosome_membrane">
                    <state value="x[283 - 396]"/>
                    <bbox w="35.0" h="10.0" x="145.12164" y="174.60197"/>
                </glyph>
            </glyph>
            <glyph class="macromolecule" id="Protein_cbd95b1234782d618bc2f6cd96d9d907_Complex_2f962a315f3fb9e5f65d1ab529473d1d" compartmentRef="early_endosome_membrane">
                <label text="SMAD1/5/8"/>
                <bbox w="48.0" h="25.0" x="138.62164" y="279.602"/>
            </glyph>
            <glyph class="macromolecule" id="Protein_6a3ed4bc09ead0918dd19d7370bd8df5_Complex_2f962a315f3fb9e5f65d1ab529473d1d" compartmentRef="early_endosome_membrane">
                <label text="ZFYVE16"/>
                <bbox w="48.0" h="25.0" x="138.62164" y="214.60197"/>
                <glyph class="state variable" id="Protein_6a3ed4bc09ead0918dd19d7370bd8df5_Complex_2f962a315f3fb9e5f65d1ab529473d1d.info.1" compartmentRef="early_endosome_membrane">
                    <state value="x[1 - 1539]"/>
                    <bbox w="35.0" h="10.0" x="145.12164" y="209.60197"/>
                </glyph>
            </glyph>
            <glyph class="macromolecule" id="Protein_d0ba752300c6b80a663f41fc4d04b996_Complex_2f962a315f3fb9e5f65d1ab529473d1d" compartmentRef="early_endosome_membrane">
                <label text="p-4S-BMPRI"/>
                <bbox w="48.0" h="25.0" x="191.62164" y="179.60197"/>
            </glyph>
        </glyph>

and the PC web service uses the same sbgn-converter and does not further modify the SBGN-ML results.

PS:
Refs: iVis-at-Bilkent/newt#176

@IgorRodchenkov Let's investigate this further on our side and get back to you

Looks like the way compartmentRef's are specified in paxtools is fine. As explained in the related Newt issue (https://github.com/iVis-at-Bilkent/newt/issues/176), since the nesting of objects is more flexible in SBGN but not in visualization library (e.g. Cytoscape.js), we have to ignore complex members' compartmentRefs to resolve our issue.


Error when deploying app

Hello,
I just encountered with an error. It exited deploying process at the end of it. It gives me this error:
remote: ERROR: Build failed: exec: job exited with status 22
I couldn't find any information related to status code 22.
Thanks

Did you manage to find out where to debug these build job error? Official docs aren't that helpful on these.

@bin4ryio actually what i think about current state of Flynn project is UNSTABLE. actively looking for more stable alternative. I'm commenting like this because we're long time user of Flynn and it's complete mess. Don't recommend you to use Flynn at all. Please also let me notice if you find more stable PaaS alternative.

OK noted. I should stick with Dokku til then. Appreciate ur input mate

Flynn is unmaintained and our infrastructure will shut down on June 1, 2021. See the README for details.


feat: implement OnDust to sweep reaped funds into FeeAccount

Signed-off-by: Gregory Hill<EMAIL_ADDRESS>

Good, but do we still have dust amount set to 0? Also, you think it's possible to add a test for this?

This is just a minor addition for when we adjust the existential deposits. As such, probably not worth testing right now - also because it's implementation lives in an external pallet.


Remove type alias keyword

The number one question on elm faq is:
What is the difference between type and type alias?
So what about having no type alias at all and replace this:
type alias MaybeList a = Maybe (List a)

with this:
MaybeList a = Maybe (List a)

Since variables and types are distinguish by first upper/lower case, we know if that is function or type alias.
This way elm will become even more compact.

see here https://github.com/elm/compiler/issues/2101


Internal Commands: Filtering by height has no effect

Resolved by: https://github.com/Granola-Team/mina-indexer/pull/959

This is in the Done column, but not closed. I will close. @n1tranquilla re-open if the issue is not resolved, please.


Citation file

#73

Just noting here that I think the citation button won't go into affect until develop is merged into main. An option would be to point this PR at NREL:main instead, which should not create any complications with later merging develop into main (or if it does, they will be easy to manage). @genevievestarke any opinions on that?

Sounds good to me!  I think it might be easier to push this file to main, and then merge back into develop?

Sounds good to me! I think it might be easier to push this file to main, and then merge back into develop?

Ok; I think to do that, the best way will be for me to check out this file on main; push directly to main; merge main into develop; and push develop. This PR will remain open during that process, but once that is done, I can "merge" this PR to show it's completion (although, at that point, there should be no difference between develop and this PR).

I've now directly pushed CITATION.cff to main ca4fcda, and then merged main back into develop f598a1a. To avoid mucking up the history, I'm closing this PR without merging, and will then close #86.


Update sbt to 0.13.18

Updates org.scala-sbt:sbt from 0.13.13 to 0.13.18.
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention @scala-steward in the comments below.
Have a fantastic day writing Scala!

Ignore future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [{ groupId = "org.scala-sbt", artifactId = "sbt" }]

labels: semver-patch

Closing as OBE; on newer version.


Update Docker Images

Describe the new feature you'd like
We haven't updated our docker images since the release.
We should look into automatically pushing these to DockerHub when Jenkins does a build

@nickwalton did this


Parser errors on +=, -=...

This kind of arithmetic causes parser errors:
level += 1.

should work on Cloud language version,
https://syntax.abaplint.org/?filter=move#/statement/Move

Is Cloud supposed to be Cloud ABAP or S4HANA Cloud?

try again, version 1.7.11 released, set language version to v755


Rename ordered_dict_intersection -> compat_dict_intersection

xarray does not use OrderedDicts any more, so name did not make sense. As suggested here https://github.com/pydata/xarray/pull/3877#discussion_r396620551.

[x] Tests added updated
[x] Passes isort -rc . && black . && mypy . && flake8
[x] Fully documented, including whats-new.rst

Erm, is this function not used anywhere apart from its tests? 😄

Doesn't look like it, but it seems like a thing that could be useful, e.g. for handling combining attrs dicts, so why delete it?

Agree!

Thanks @johnomotani


cutest words

hello am using your plugin with textAngular which is wisiwig text editor for angular
the plugin works correctly but it cuts words and shows ...readmore and when i click readmore it wont start continuing where it exactly at it cuts which it should be. and also other issue is when i click readmore it starts continuing in new line and readleas showed at the end of the line with some spaces left. let me show you my snap shot.

@sinteco this directive has not been tested with wysiwyg editors. But, maybe you could create an example in Codepen to demonstrate the issue? Also, if you can suggest a fix in a PR that would be appreciated.


Send RSSID with heartbeat

Return signal strength with each heartbeat -> nice visualization

Heartbeat now contains state info:
typedef struct __attribute__ ((packed)) {
  uint16_t voltage;
  int8_t rssi;
} BuzzerState;


Python 3 compatibility

Changes:
Tkinter changed name.
Print is a function now.
Encoding is only needed at the file level.

Thank you! This isn't going to break Python 2.7 compatibility, right?
I've stopped developing in Python so I'm not in the loop...

Thank YOU :) i got 5000 messages ported way easier because of this script.
It should preserve 2.7 compatibility yes.

😉

Could you please create another PR for fixing #25? sys should be imported before using it


GPU memory leak

After 80 epochs (8 hours), I got this error
Epoch 80:  87%|████████████████▌  | 750/858 [04:56<00:42,  2.53it/s, loss=0.258]Traceback (most recent call last):
  File "train.py", line 442, in <module>
    main(args)
  File "train.py", line 424, in main
    trainer.fit(forecaster, datamodule=data_module)
  File "/home/u7701783/.local/lib/python3.8/site-packages/pytorch_lightning/trainer/trainer.py", line 460, in fit
    self._run(model)
  File "/home/u7701783/.local/lib/python3.8/site-packages/pytorch_lightning/trainer/trainer.py", line 758, in _run
    self.dispatch()
  File "/home/u7701783/.local/lib/python3.8/site-packages/pytorch_lightning/trainer/trainer.py", line 799, in dispatch
    self.accelerator.start_training(self)
  File "/home/u7701783/.local/lib/python3.8/site-packages/pytorch_lightning/accelerators/accelerator.py", line 96, in start_training
    self.training_type_plugin.start_training(trainer)
  File "/home/u7701783/.local/lib/python3.8/site-packages/pytorch_lightning/plugins/training_type/training_type_plugin.py", line 144, in start_training
    self._results = trainer.run_stage()
  File "/home/u7701783/.local/lib/python3.8/site-packages/pytorch_lightning/trainer/trainer.py", line 809, in run_stage
    return self.run_train()
  File "/home/u7701783/.local/lib/python3.8/site-packages/pytorch_lightning/trainer/trainer.py", line 871, in run_train
    self.train_loop.run_training_epoch()
  File "/home/u7701783/.local/lib/python3.8/site-packages/pytorch_lightning/trainer/training_loop.py", line 569, in run_training_epoch
    self.trainer.logger_connector.log_train_epoch_end_metrics(epoch_output)
  File "/home/u7701783/.local/lib/python3.8/site-packages/pytorch_lightning/trainer/connectors/logger_connector/logger_connector.py", line 325, in log_train_epoch_end_metrics
    self.log_metrics(epoch_log_metrics, {})
  File "/home/u7701783/.local/lib/python3.8/site-packages/pytorch_lightning/trainer/connectors/logger_connector/logger_connector.py", line 208, in log_metrics
    mem_map = memory.get_memory_profile(self.log_gpu_memory)
  File "/home/u7701783/.local/lib/python3.8/site-packages/pytorch_lightning/core/memory.py", line 365, in get_memory_profile
    memory_map = get_gpu_memory_map()
  File "/home/u7701783/.local/lib/python3.8/site-packages/pytorch_lightning/core/memory.py", line 384, in get_gpu_memory_map
    result = subprocess.run(
  File "/opt/conda/lib/python3.8/subprocess.py", line 516, in run
    raise CalledProcessError(retcode, process.args,
subprocess.CalledProcessError: Command '['/usr/bin/nvidia-smi', '--query-gpu=memory.used', '--format=csv,nounits,noheader']' returned non-zero exit status 255.
Epoch 80:  87%|████████▋ | 750/858 [04:57<00:42,  2.52it/s, loss=0.258]

I think the only dataset I've ever run for that many epochs is the toy dset, but I will look into this. Is this a very small dataset, or are you using large regularization settings? If you aren't using --early_stopping you may want to try that to prevent long training times.
I'm not 100% sure if that is an issue with GPU memory itself or maybe in the logging of gpu memory stats to wandb...

Python is refcounted (see https://discuss.pytorch.org/t/cuda-out-of-memory-on-the-8th-epoch/67288). This is not an issue with GPU memory as the python process would entirely crash as OOM and often written to /var/log/syslog. In other words, you wouldn't get the returned non-zero exit status 255. error message.
This is triggered from the flag log_gpu_memory in the train.py Pytorch Lightning trainer. This is deprecated and if you still want to log the metrics, try updating to using DeviceStatsMonitor. See https://pytorch-lightning.readthedocs.io/en/stable/extensions/generated/pytorch_lightning.callbacks.DeviceStatsMonitor.html?highlight=DeviceStatsMonitor#pytorch_lightning.callbacks.DeviceStatsMonitor.
Other options include just taking out that flag and use another bash window to periodically execute nvidia-smi query yourself. I couldn't find specific examples of why nvidia-smi would trigger that error but it's likely not associated with GPU OOM.

@jhillhouse92 Thank you, that's about what I figured. I'll just remove that option in the public version, it's only in there to help me pick hparams for each dataset. The best model is usually the biggest one that fits in memory. As long as the model doesn't crash with a clear OOM error on the first backward pass it's fine for most situations... not a super important thing to log.


chore(docs): update CSV Connector Plugin documentation

chore(docs): update CSV Connector Plugin documentation

Included in !344


Add support for Google Extensible Service Proxy

Cloud Endpoints' Extensible Service Proxy only takes JWTs, not access tokens.  This
change provides the ability to trasform UserCredentials and ServiceAccountCredentals
such that they provide JWTs as bearer tokens.
CCLA is in the works.  I thought I'd get the ball rolling with the review.

This PR is for https://github.com/google/google-auth-library-java/issues/157.

I wonder if ServiceAccountCredentals needs to customize the forEsp() method since there's ServiceAccountJwtAccessCredentials.

Sorry for the delay in getting to this. Before we can review this PR, you'll need to sign one of CLA's mentioned above.

Closing as the CLA hasn't been signed yet so we can't look into the PR yet. Feel free to reopen when the CLA is signed.


Change the colors of shadow

There is no option to change the shadows colors of components. How can we change it ?

You can change it using these properties:
Positive depths: shadowLightColor/shadowDarkColor
Negative depths: shadowLightColorEmboss/shadowDarkColorEmboss
You can set them in your theme to be applied throughout the app.


OOPS in CPP

I want to provide a  description and work of all the concepts of OOPs in CPP.
I'll provide clear , well explained and optimized code for the same.

/assign

Please do assign this to me.


Fix warning when building the plugin

The following warnings appear when the project is compiled:
WARNING: cljs.core/+, all arguments must be numbers, got [string number] instead. at line 39 /home/sbauer/Documents/dev/Rainbow/src/lt/plugins/rainbow.cljs
WARNING: cljs.core/+, all arguments must be numbers, got [string any] instead. at line 45 /home/sbauer/Documents/dev/Rainbow/src/lt/plugins/rainbow.cljs

This is resolved with the merging of #6


Fixed font-size

Issue that this pull request solves
Closes: #429
Proposed changes
Brief description of what is fixed or changed
Changed font size for all forms to be the same
Types of changes
Put an x in the boxes that apply

[x] Bugfix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] Documentation update (Documentation content changed)
[ ] Other (please describe):

Checklist
Put an x in the boxes that apply

[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my own code
[x] I have commented my code, particularly in hard-to-understand areas
[x] I have made corresponding changes to the documentation
[x] My changes generate no new warnings
[x] My changes does not break the current system and it passes all the current test cases.

Screenshots

Other information
Any other information that is important to this pull request
I realized the font-size was different due to the HTML tags. There are <inputs> and <textarea> tags. <textarea> had a smaller font(14px) therefore I changed the global font-size of <inputs> to 14px too

Hey @LeeRenJie Please avoid using global css. Instead make this change in the respective component's css module

Done!


⚠ ClamAV has degraded performance

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

HTTP code: 200
Response time: 111 ms

Resolved: ClamAV performance has improved in c44c4b8.


Output folder storage

Recreating the structure of the input folder might be worthwhile.  React may have a relative path from the selected file.

Finished


would like a mirror of https://github.com/dart-lang/lints

We've recently added a new package to dart-lang - the 'lints' package. We'd like to use this in the sdk repo as a setting for static analysis, and need to be able to bring it in as a DEP.
The url is https://github.com/dart-lang/lints; thanks!
cc @pq

Done: https://dart.googlesource.com/lints

👍 , thanks!


Add rb-editor-wrapper component

TP: https://rockabox.tpondemand.com/entity/10463
Needs additional Angular work outside this PR.

Are you planning on carrying out the angular work? It is just a serious of transcluded directives. Would make sense to do it here IMO.

Are you planning on carrying out the angular work? It is just a serious of transcluded directives. Would make sense to do it here IMO.

@chrisvxd I want feedback on whether this approach is going to work, naming etc before splitting it out - it's easier to work with statics if changes are necessary. Also means someone else could pick up the Angular task if necessary.

I think it is also better to keep these static for now as there will be complexities around multi-transclude and incorporating ui-view into the directives.


Documentation and a live example needed

This is a very important library for SVG animators like me, thank you. But please add some documentation and a live example.

yes please!

Yes #1 issue. I don't know JavaScript since I'm a designer. I hope this will let me still use SMIL the way it was. :( thank you!

bump

The author of the Flash2Svg tool, that uses SMIL to export flash animations, published an usage example:
Without SMIL polyfill:
http://www.tbyrne.org/staging/smil-polyfill/tears.htm
With SMIL polyfill:
http://www.tbyrne.org/staging/smil-polyfill/tears_polyfill.htm


[SPARK-12933][SQL] Initial implementation of Count-Min sketch

This PR adds an initial implementation of count min sketch, contained in a new module spark-sketch under common/sketch. The implementation is based on the CountMinSketch class in stream-lib.
As required by the design doc, spark-sketch should have no external dependency.
Two classes, Murmur3_x86_32 and Platform are copied to spark-sketch from spark-unsafe for hashing facilities. They'll also be used in the upcoming bloom filter implementation.
The following features will be added in future follow-up PRs:

Serialization support
DataFrame API integration

do we also need to update the test runner to add this module?
cc @JoshRosen

Test build #49811 has started for PR 10851 at commit bb7435f.

@rxin Already added sketch module to sparktestsupport per Josh's suggestion.

Test build #49820 has started for PR 10851 at commit 486414d.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/49818/
Test FAILed.

Merged build finished. Test FAILed.

Test build #49824 has started for PR 10851 at commit 92d3a52.

Test build #49811 has finished for PR 10851 at commit bb7435f.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/49811/
Test PASSed.

Test build #49826 has started for PR 10851 at commit 2bf907a.

Somehow there is no timing information for the test cases in this new module. Can you take a look at that? You might need to change the sbt build file.

Test build #49827 has started for PR 10851 at commit 7ea22a9.

Oh, I forgot: you also need to update project/SparkBuild.scala to add the new submodule.

Test build #49820 has finished for PR 10851 at commit 486414d.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/49820/
Test PASSed.

Merged build finished. Test PASSed.

Test build #49824 has finished for PR 10851 at commit 92d3a52.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/49824/
Test PASSed.

Test build #49826 has finished for PR 10851 at commit 2bf907a.

This patch fails PySpark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/49826/
Test FAILed.

Merged build finished. Test FAILed.

Test build #49827 has finished for PR 10851 at commit 7ea22a9.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/49827/
Test PASSed.

Merged build finished. Test PASSed.

Test build #49848 has started for PR 10851 at commit a6e7479.

Test build #49848 has finished for PR 10851 at commit a6e7479.

This patch fails Java style tests.
This patch does not merge cleanly.
This patch adds no public classes.

Build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/49848/
Test FAILed.

Test build #49882 has started for PR 10851 at commit e06ff13.

Test build #49882 has finished for PR 10851 at commit e06ff13.

This patch fails Scala style tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/49882/
Test FAILed.

Test build #49923 has started for PR 10851 at commit 4adb57a.

Test build #49923 has finished for PR 10851 at commit 4adb57a.

This patch fails MiMa tests.
This patch merges cleanly.
This patch adds no public classes.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/49923/
Test FAILed.

Merged build finished. Test FAILed.

Test build #49925 has started for PR 10851 at commit 57a31e6.

Test build #49925 has finished for PR 10851 at commit 57a31e6.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/49925/
Test PASSed.

@liancheng can you make sure the generated javadocs look ok?

I've checked the Javadoc, it looks good.

I looked at this quickly (i.e. didn't do a detail review), but changes lgtm.

Test build #2445 has started for PR 10851 at commit 1608ec9.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/49927/
Test FAILed.

Test build #49929 has started for PR 10851 at commit 65853ad.

Test build #49929 has finished for PR 10851 at commit 65853ad.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/49929/
Test PASSed.

Going to merge this. Thanks.
Would be great if @cloud-fan can take another look at the implementation.

@rxin The constructor visibility issue would be fixed in #10893.


Create a Windows iomgr suitable for the callback-based API

What version of gRPC and what language are you using?
master, C++
What operating system (Linux, Windows, …) and version?
N/A
What runtime / compiler are you using (e.g. python version or version of gcc)
N/A
What did you do?
Used the gRPC C++ API
What did you expect to see?
A Windows iomgr suitable for running the callback-based API
What did you see instead?
Not sure
Anything else we should know about your project / environment?

@guantaol can I ask you about this issue status? Maybe there is information about the planned completion date? Looking forward to the release with this issue to start working with callback API)

@guantaol  can you provide and update or close?

Closing as a duplicate of #16504.


TypeError: Cannot read property 'split' of undefined

I've previously run git-labelmaker and it worked just fine but when I ran it today I needed to use a new token since I forgot my password. After I entered the new token and a master password I got the error [TypeError: Cannot read property 'split' of undefined]. I uninstalled and reinstalled git-labelmaker and it didn't fix the issue. I tried diving into the source to see what might be happening here but I'm honestly not too sure...
node version: 5.11.1
npm version: 3.9.5
Any thoughts?

My guess would be that the repos url format is possibly causing the issue.
https://github.com/himynameisdave/git-labelmaker/blob/master/bin/modules/readRepo.js#L14
Some github urls contain : which might not be getting picked up by this package - which is used in this project.

@dwoodiwiss thanks for looking into this, I think you may be correct in which case I will file & fix it in node-github-url-from-git

Thanks for looking into this. I went back and looked at my dev log for the day and here's my notes:
touch labels.json
sublime labels.json

# paste in labels and save
# forgot my master password...
# deleted old token from here: https://github.com/settings/tokens
# generated new token with repo access...
git-labelmaker
# Chose to create new token
# Got error `[TypeError: Cannot read property 'split' of undefined]`...

# remove git-labelmaker
npm uninstall -g git-labelmaker

# and install again...
npm i -g git-labelmaker

# enter access token and master password and got error again...

The remote repo that I was intending to add labels to had a - in the url though. I'm not sure if that could be the issue?

I have this same issue with a repo with a dash '-' in the name as well...

I try to change remote form ssh to https.

I've added some proper error handling, and as far as I can tell the node-github-url-from-git package handles all of the standard github urls. If there are anymore issues/if you are not seeing meaningful errors, let me know and we can re-investigate.
Otherwise feel free to submit tickets to node-github-url-from-git with your specific test cases/git urls that are failing 👍


More flexible sorting.

Users might expect that clicking on table headers will sort tracks in some reasonable way. Seems to be also useful in general.

Duplicate of https://github.com/KeitIG/museeks/issues/62


Improve test coverage for the application model

Right now we have tests that only run when DCP is installed. We can verify the output model outside of running the application as well

We're still at pretty low numbers here. We should get this up @mitchdenny


fix(Box): replace barrel file import

Solves problem of barrel file import with
Box.js:5 Uncaught TypeError: createTheme_default is not a function
       at Box.js:5:22

which leads to some circulars which are not so obvious to figure out at first look

[x] I have followed (at least) the PR section of the contributing guide.

@ZeeshanTamboli sorry but I cannot. In my case issue happens btw three repositories (custom shared mui components, other custom react components, project repo...). I can refer to issues like https://github.com/mui/material-ui/issues/31835 or to StackOverflow https://stackoverflow.com/questions/74542488/react-material-ui-createtheme-default-is-not-a-function where devs are suggesting changing imports of the Box component. Instead of doing hotfix in my code and disabling eslint rule to allow imports from @mui/material I just changed locally import from barrel one in my node_modules and everything started working correctly. It's a small change which I guess we can add cause it's not even a breaking change 🙏

Thank you!!!!

@Dzixxx But can you explain why does it work after doing this?

@ZeeshanTamboli I really would like to but the problem is that when I was debugging the issue from the description I only had that barrel file idea working - this barrel import might be used somewhere else in the code inside mui or during building process via vite some optimisation caused the issue becaue of tree shaking

As long as this is not a big change (I guess any from component perspective) we could just add it. I really would like to explain it better but I'm too early in actual project to provide full explanation of the flow and how it happened that this one line changer saved my a$$ 😂

@Dzixxx We shouldn't make code changes without a clear explanation. It seems to be an issue with esbuild or vite's cache, as described in #31835. Until that issue is resolved, you'll have to apply the workaround of changing the Box import.


Dockerfile: Add u-boot-tools for mkimage

This is required by the ppc32 kernel build. Note that it should not be
necessary, as we do not need the wrapped image.
I opened an issue to fix this: https://github.com/linuxppc/linux/issues/211
Signed-off-by: Joel Stanley<EMAIL_ADDRESS>

Required for https://github.com/ClangBuiltLinux/continuous-integration/pull/100


Fix: Check for remaining events in the internal X11/xcb buffers

I went to #xcb on Freenode and psychon was so kind to point me to:
https://docs.rs/x11rb/0.8.0/x11rb/event_loop_integration/index.html
It seems that while events are being handled new events can be queued up
in the internal buffers. And so the next poll() will wait even though there are
still events to be handled.
This is a race condition, so that might be why this does only occur on some systems.

I also hacked together an example that reproduces the problem and hopefully makes it clearer what is going on. Since I don't know where to leave it, I'll leave it here: https://gist.github.com/psychon/6fc6a6ed06ddfc5eea5f719aac14b7bf
The example creates a window, sends an event to the window, then does get_input_focus().get_reply(). This last part ensures that the event was already received by xcb and the FD is no longer readable.
Then, I call poll() and print the information that the FD really is not readable, but poll_for_event() still returns an event afterwards.

I also hacked together an example that reproduces the problem and hopefully makes it clearer what is going on. Since I don't know where to leave it, I'll leave it here: https://gist.github.com/psychon/6fc6a6ed06ddfc5eea5f719aac14b7bf
The example creates a window, sends an event to the window, then does get_input_focus().get_reply(). This last part ensures that the event was already received by xcb and the FD is no longer readable.
Then, I call poll() and print the information that the FD really is not readable, but poll_for_event() still returns an event afterwards.

I want to note: This is the cause of the resizing issues I've had, so this is not only theoretical.

I want to note: This is the cause of the resizing issues I've had, so this is not only theoretical.

This fix looks correct to me. To make it airtight, the logic in drain_xcb_events also needs to be restructured so that poll_for_events returning None is the last thing that happens before it returns.


Update MD file on new branch 2017-11-26_09-32-05

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

split/content1.md
:white_check_mark:Succeeded

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


Unable to Achieve Intermediate Opacity When Modifying Alpha Values in Colormap

Hello,
I’m trying to adjust the opacity of a specific region in my volume by modifying the alpha values in my colormap, but the changes are not reflected in the image.
My goal is to make the selected region fully opaque, while setting the other regions to an intermediate opacity level.
To achieve this, I first created an RGBA colormap JSON with all alpha values set to 125. However, there were no visible changes in the image’s opacity, which remains fully opaque.
I also followed the demo's example Voxel atlas Starting from my RGB colormap (my RGBA without the alpha channel), I obtained the LUT and set the fourth value (index 3) for all regions to 96, and for the selected region to 200 (lut[(index * 4) + 3]). Although I can see the LUT values changing in the console, there is no effect on the image’s opacity. However, if I adjust the R, G, and B values, the color changes, but not the opacity.
Continuing my attempts, I created a function that, when a region is selected, adds the "A" key to my RGB colormap. It uses the same number of values as "R" (and "G" and "B"), setting them to 125, except for the selected region’s alpha value, which is set to 255. Despite these changes, there was no effect on the opacity. The only changes I observe occur when setting alpha values to 0 or 255 (transparent and fully opaque), but this is not what I’m aiming for. I would like to achieve an intermediate opacity for the non-selected regions.
I also tried using an RGBAI colormap but encountered the same outcome. The opacity only changes when the alpha values are set to 0 or 255, not with intermediate values.
I don’t understand what might be going wrong. Is it possible that opacity adjustments are limited to predefined colormaps in niivue? Could you please help me with this functionality?
Thank you for your assistance.

@Appryll this is the intended behavior. You can find out more details in the colormaps developer notes.

As you found with the voxel atlas live demo](https://niivue.github.io/niivue/features/atlas.html), the alpha blending is controlled by a single opacity value, in this example setOpacity(). Therefore, the colormap's alpha value only distinguishes between transparent (0) and opaque (any other value).
The voxel alpha value influences the volume rendering. Voxels can vary from being almost completely transparent (like a cloud, that has no hard surface) all the way to being completely opaque (like the hard surface of a rock). You can see this influence in the colormaps live demo where you can compare a translucent surface:

let cmap = {
 R: [0, 255],
 G: [0, 255],
 B: [0, 255],
 A: [0, 22],
 I: [0, 255],
};

to an opaque one:
let cmap = {
 R: [0, 255],
 G: [0, 255],
 B: [0, 255],
 A: [0, 255],
 I: [0, 255],
};

You can make sub-threshold voxels translucent by using the alphaThreshold property, as shown in the alphathreshold live demo which uses the method described here.

@neurolabusc Thank you for your quick response and the information provided.
Does this mean that with the following colormap, I should get a volume with completely opaque regions?
let cmap = {
  R: [0, 255, 24, 170],
  G: [0, 76, 128, 170],
  B: [0, 62, 100, 170],
  A: [0, 125, 125, 125],
  labels: ["air", "reg_1", "reg_2", "reg_3"]
};

Because that is exactly what I am getting a volume with fully opaque regions. However, I can still adjust the volume's opacity without any issues using setOpacity() as you described.
Regarding the voxel atlas live demo, I understand that when the mouse hovers over a region, the LUT changes and modifies the alpha value to 200, which makes the region stand out from the rest. This is exactly what I'm trying to achieve, but I haven't been able to replicate it.
With the following colormap, wouldn’t I obtain regions with different colors and opacities?
let cmap = {
  R: [0, 255, 24, 170],
  G: [0, 76, 128, 170],
  B: [0, 62, 100, 170],
  A: [0, 255, 125, 50],
  labels: ["air", "reg_1", "reg_2", "reg_3"]
};

Thank you for your time.

Note that for the 2D slices, alpha compares zero versus non-zero, so making gray transparent
let cmap = {
 R: [0,   0, 120, 175],
 G: [0,  90,  60, 185],
 B: [0, 120,  60, 175],
 A: [0, 1,  0, 1],
 labels: ["air","CSF","gray","white"],
};

versus opaque (with 1..255 giving the same result), with the setOpacity slider adjusting the translucency of all regions with non-zero values.
let cmap = {
 R: [0,   0, 120, 175],
 G: [0,  90,  60, 185],
 B: [0, 120,  60, 175],
 A: [0, 1,  1, 1],
 labels: ["air","CSF","gray","white"],
};

I am still not sure I understand the question. I do think that this live demo illustrates how you can set the colormapLabel.lut color lookup table (lut) with custom alpha. Note that colormapLabels differ a little bit from traditional colormaps. One typically uses trilinear interpolation for scalar color maps (a pixel half way between voxels with brightness 5 and 7 could be reasonably estimated as having an intensity of 6; a pixel half way between Brodmann Areas 5 and 7 must use nearest neighbor as it is not area 6). Make sure your NIfTI atlas uses the NIFTI_INTENT_LABEL.

@Appryll , I'm closing this for now. Let us know if it would be better to keep the issue open.


WIP settings menu

Added menu structure and logic
Added routing in start.js to spawn menu (for debugging atm)
style changes

Let me know when can merge etc.


Using the hotkey to toggle combat with PCs does not work.

The hotkey (Default Shift C) works when selecting a group of NPCs to add them to the tracker and starting combat. Unfortunately, if you try doing this with a PC selected, it causes the following exception.

To Reproduce
Steps to reproduce the behavior:

Add a player character to the scene.
Press Shift+C
See error 😭

Desktop (please complete the following information):
OS: Windows 10
Browser: Chrome
FVTT Version: 0.7.9, DND5e Version 1.2.4
Module version: 0.1.9
Game running locally.
No other mods running.

 The error only occurs when the token's character sheets have already been deleted. Was testing on an old scene sorry!


add support for adoptium/jdk11u-fast-startup-incubator project

Hi there! We have created a fast startup project(https://github.com/adoptium/jdk11u-fast-startup-incubator) under adoptium group to explore some novel JVM startup technilogies. I also create a simple build pipeline(x64_linux) for it. This build pipeline is currently hosted on http://ci.dragonwell-jdk.io/job/build-scripts/job/jobs/job/jdk11u/job/jdk11u-linux-x64-fast_startup/, which is a fork of the official adoptium pipeline(https://github.com/adoptium/ci-jenkins-pipelines / https://ci.adoptopenjdk.net/). I want to try to move it to official adoptium CI platform(https://ci.adoptopenjdk.net/) and host on that. Please let me know your advice and thoughts, thanks in advance!
RELATE: https://github.com/adoptium/temurin-build/pull/2866

Is the project interested in the test run? I mean if the project build would run smoke tests and aqa tests? If it's yes, then probably other changes needed.

Is the project interested in the test run? I mean if the project build would run smoke tests and aqa tests? If it's yes, then probably other changes needed.

Hi @sophia-guo, this project only runs JDK jtreg tests. The purpose of this project is to explore some HotSpot techniques(Pre-initialization for CDS and static analysis based on that, etc), currently only jtreg tests are added/modified.

Related PR is now merged.


New Package: Helix 22.08.1

[x] Have you signed the Contributor License Agreement?
[x] Have you checked that there aren't other open pull requests for the same manifest update/change?

[ ] Have you tested your manifest locally with winget install --manifest <path>?
[x] Does your manifest conform to the 1.2 schema?

Note: <path> is the name of the directory containing the manifest you're submitting.

Addresses helix-editor/helix#4344.
This is a zip type installer, which does not appear to be supported until winget 1.4 is released.
Microsoft Reviewers: Open in CodeFlow

@microsoft-github-policy-service agree

@msftbot .zip

Should this be HelixEditor.Helix instead of Helix.Helix?

https://github.com/microsoft/winget-cli/releases/tag/v1.4.3531

v1.4.10173 released, so .zip packages is now supported.

v1.4.10173 released, so .zip packages is now supported, i suppose?

From the description of v1.4.10173 release:

Note: The Windows Package Manager Community Repository does not accept zip applications. They will not be accepted until after 1.4 is Generally Available and has been rolled out to the majority of Windows systems via the automatic upgrade from the Microsoft Store. Users may test with local manifests.

It seems like you can test zip packages now, but publishing isn't yet available.

@wingetbot run
[Policy] Reset Feedback

@wingetbot run

I'm not sure why, but the dependency isn't being installed. For some reason, the manifest as written does not install the dependency. However, removing the dependency lines and replacing them with seemingly-identical text does install the dependency. Maybe the issue is a line ending or other non-visible character?

Apologies, I mistakenly thought we had added dependency support.

This one can be closed I guess since it was added here: #101377?
I haven't installed it through winget yet, but how come this one couldn't be merged yet the other one was?

That version doesn't require a dependency - it passed automatic validation.


example-tutorial.pillar: Code colorisation not correct?

In the 03-fileContents-FileReference.ombu, the code after the closing bracket is also shown in green, but the other code is red.
^ self directoryInMemory / self fileName ...
Is that correct behaviour? (GT Win64-v0.8.227)

Ah, ok - I got it ;) the green one is the example code, and the red one the "instantiated" one...

That is actually a diff view. The red one is what gets replaced and the green one is the new code.


correctness of agem.py

Thanks for your comprehensive work! However, I have a question about the algorithm correctness in agem.py.
In line 54 of agem.py:
If the grad_ref and grad_batch are not obtuse, the model will update the grad of grad_ref. But according to the design of A-GEM, actually the grad_batch should be updated. Is that a bug or my fault?

I also think it is a bug, but when I modify the code to update grad_batch, the model's accuracy worsens. (It forgot almost all of past data) Did you solve the issue?


Add recordEdge to mutation payloads

If detected connection resolver, then composeWithRelay should add nodeEdge(sort: SortType) field to payload, for passing it to RANGE_ADD on the client mutation config. Right now this wrapper does not allow to do it out of the box.

Is it easy to implement this feature? Do you have plan to implement it?

No, I have not plan to implement it, cause does not use RANGE_ADD. For simplicity, I completely reload relay connection in my app.
But with pleasure share ideas how it should be implemented.
Before, let look on what is it cursor and how it works under the hood:

for every document cursor keeps values from fields, by which you make the sort (e.g. .sort({ _id: 1, age: 1}) then you should construct such cursor base64({_id: 34ef85...34, age: 28})
sort can be proceeded only by unique indexes (to avoid overlapping)
when proceeding before and after arguments with the cursor, you should parse values from it and use them via search with operators .where({ _id: {$gt: 34ef85...34}, age: {$gt: 28}})

So the most tricky part is determining correct cursor value for the nodeEdge. It can not be calculated in mutation without knowledge of how your connection (with which order/sorting) fetched your data on the client. And when you sending a mutation, you should provide sort fields as arg to nodeEdge for gathering correct cursor value.
The code will look something like this:
UserTC.get('$create'). getTypeComposer().setField('nodeEdge', {
  args: {
    sort: UserTC.get('$connection.@sort'),
  },
  type: UserTC.get('$connection.edges'),
  resolve: (source, args, context, info) => {
    // PREPARE `node`
    // console.dir(source, { depth: 4, color: true });
    const node = source.record;

    // PREPARE `cursor`
    // console.dir(args, { depth: 4, color: true });
    // see how constructed cursor https://github.com/nodkz/graphql-compose-connection/blob/master/src/connectionResolver.js#L190
    const cursor = ...;

    return {
      cursor,
      node
    };
  },
});


openssl error

Hi,
I'm getting the following error when running the step, docker-compose -f docker-compose.setup.yml run --rm certs
"E: Unable to locate package openssl"
if I comment out the "apt-get install unzip openssl -y" in setp/setup.sh, I get "/usr/share/elasticsearch/config/setup.sh: line 127: openssl: command not found"
what am I missing?
thanks

you can add apt-get update to setup/setup.sh, this should fix openssl: command not found

I'm having the same problem.
Although I have openssl installed , I get "E: Unable to locate package openssl" error.

I think I solved the error.
This is the error I got:
E: Unable to locate package openssl
ERROR: 100

After a bit of reading the files which are used to setup certificate, I realized that it is not our main host which lacks openssl package. The container in which the certificates are supposed to be generated doesn't have openssl package to generate the certs. So I decided to install the openssl package manually by changing the configurations as follows:
1.Open "docker-compose.setup.yml" and comment out the following line:
image: docker.elastic.co/elasticsearch/elasticsearch:${ELK_VERSION}

2.Add below lines right after "certs:" (line 4 I guess):
IMP: mind indentations
build:
      context: .
      dockerfile: Dockerfile

3.Create a file named "Dockerfile" and write below lines onto it:
IMP: write the ELK version explicitly (I copied it from .env-sample file)
FROM docker.elastic.co/elasticsearch/elasticsearch:7.16.0
RUN apt-get update && apt-get install -y openssl

4.Your files should look like this:
docker-compose.setup.yml:
version: '3.5'

services:
  certs:
    build:
      context: .
      dockerfile: Dockerfile
    container_name: certs
    #image: docker.elastic.co/elasticsearch/elasticsearch:${ELK_VERSION}
    command: bash ${ELASTIC_DIR}/config/setup.sh
    user: "0"
    volumes:
      - ./swag:/swag/
      - ./secrets:/secrets/
      - ./setup/setup.sh:${ELASTIC_DIR}/config/setup.sh
      - ./setup/instances.yml:${ELASTIC_DIR}/config/instances.yml:ro
    environment:
      ELASTIC_PASSWORD: ${ELASTIC_PASSWORD}
      SUBDOMAIN: ${SUBDOMAIN}
      SUBFOLDER: ${SUBFOLDER}
      STAGING: ${STAGING}
    networks:
      - elk

volumes:
  secrets:
    driver: local
  setup:
    driver: local

networks:
  elk:
    driver: bridge

Dockerfile:
FROM docker.elastic.co/elasticsearch/elasticsearch:7.16.0
RUN apt-get update && apt-get install -y openssl

5.Now you can run the following command successfully:
sudo docker-compose -f docker-compose.setup.yml run --rm certs

P.S. This is not a best-practice solution and it needs to be optimized, I just came up with this and wanted to help you as well by sharing it.
BTW, you might get an error saying the following, but it is a minor one and you can ignore it. I guess it might have been solved if I had installed apt-utils package via the "Dockerfile".
debconf: delaying package configuration, since apt-utils is not installed

Hope you got it sorted now!!!

Or I could simply add apt-get update && before apt-get install unzip openssl -y in "setup/setup.sh" file. :)))


fix: Osm username,  add escaping where necessary

Description
This PR focuses on improving the codebase in terms of security and consistency by making changes to how text content is inserted into various elements within the application. Specifically, I have replaced the use of .html() with .text() for setting text content, ensuring safer content insertion and aligning with best practices.
Fixes #1124

Hi @RitaDee! This is excellent, thank you for your work on this.
If possible, can you do some testing of these changes and show some screenshots of that testing, highlighting the areas where the username are displayed? This is good practice to show any github repo maintainer that your code works and hasn't broken anything.

One of the issues with our current codebase is that any string lifted from our core.yaml file (i.e. anything with a t() invocation or a tHtml() could be using  inline to the string for formatting.
If the code modifications ONLY cover osm username rendering, then this should be good.

Also, I approved this PR to have our automated test workflow run some unit tests (results above). It appears that the node 16.14 build failed- but this is not due to any changes made in this PR, and is likely due to a flaky test. I do not consider that test failure a blocker by any means. :)

Thank you, @Bonkles, for the review. I will write the test and attach the screenshots.

Also, I approved this PR to have our automated test workflow run some unit tests (results above). It appears that the node 16.14 build failed- but this is not due to any changes made in this PR, and is likely due to a flaky test. I do not consider that test failure a blocker by any means. :)

Noted.

Thank you, @Bonkles, for the review. I will write the test and attach the screenshots.

I ran a very quick test of your branch and saw one issue: the contributors.js code is causing the following to happen in the bottom bar of Rapid:

Yeah sorry - some context on this is that we (Rapid) pulled in a bad regression from the iD project a while ago before the code diverged, and now there are places all around the Rapid code where the string sanitization has been removed.  Every usage of .html really needs to be checked carefully because there is a lot of code that is expecting to pass span or other html tags through it.
some related issues:
https://github.com/openstreetmap/iD/issues/8813
https://github.com/openstreetmap/iD/pull/7998
https://github.com/openstreetmap/iD/pull/8817

One of the issues with our current codebase is that any string lifted from our core.yaml file (i.e. anything with a t() invocation or a tHtml() could be using inline to the string for formatting.
If the code modifications ONLY cover osm username rendering, then this should be good.

I am not so clear on this. Do I revert the changes made on the file?

Thank you, @Bonkles, for the review. I will write the test and attach the screenshots.

I ran a very quick test of your branch and saw one issue: the contributors.js code is causing the following to happen in the bottom bar of Rapid:

I have made an adjustment to this. This is what it looks like now:

Yeah sorry - some context on this is that we (Rapid) pulled in a bad regression from the iD project a while ago before the code diverged, and now there are places all around the Rapid code where the string sanitization has been removed. Every usage of .html really needs to be checked carefully because there is a lot of code that is expecting to pass span or other html tags through it.
some related issues: openstreetmap/iD#8813 openstreetmap/iD#7998 openstreetmap/iD#8817

I will check this out. Thank you

Hi @RitaDee! This is excellent, thank you for your work on this.
If possible, can you do some testing of these changes and show some screenshots of that testing, highlighting the areas where the username are displayed? This is good practice to show any github repo maintainer that your code works and hasn't broken anything.

Here is the screenshot of the test:

@Bonkles, this is ready for review.


Fix extract with out-of-bounds index

When folding a OpCompositeExtract that is fed by an
OpCompositeConstruct, we handle and out of bounds
index, but only in the case where the result of the
OpCompostiteConstruct is a struct.  This change
refactors that folding rule and then improves it to
handle an out-of-bounds access when the result of the
OpCompositeConstruct is a vector.
Fixes #4514

The clang format error is not related to my change.

I think the code format error was fixed: https://github.com/KhronosGroup/SPIRV-Tools/blob/master/include/spirv-tools/optimizer.hpp#L32
Just rebasing it would make it working.


Returns wrong interface

Environment details

OS: Ubuntu
PHP version: 8.1
Package name and version: @latest

Steps to reproduce
            $client = $this->_getGoogleClient();
            $service = new \Google\Service\Drive($client);
            $driveFile = new \Google\Service\Drive\DriveFile();
            $driveFile->setName($filename);
            $driveFile->setParents($parent);
            $driveFile->setMimeType($mime);
            $request = $service->files->create($driveFile);
            $mu = new \Google\Http\MediaFileUpload($client, $request, $mime, base64_decode($data), true, $chunkSize);

Throws the following:
Message: Google\\Http\\MediaFileUpload::__construct(): Argument #2 ($request) must be of type Psr\\Http\\Message\\RequestInterface, Google\\Service\\Drive\\DriveFile given
line: 83
file: vendor/google/apiclient/src/Http/MediaFileUpload.php
Followed as in : https://github.com/googleapis/google-api-php-client/blob/main/examples/large-file-upload.php#L92-L109

Added and it returned RequestInterface.
$client->setDefer(true);


Duplicating objects and clearing layer reverts to initial text

Any duplicated text box reverts back to it's original state when I clear layers.
So, option+drag, and then change the text to a new ID from the JSON, then import, then clear - it reverts to the oldest ID. It didn't used to work like this and it's ruined a lot of work because I copy design elements all over the place. Creating a new text box each time is inefficient.

our desired behavior is: 'clear' reverts a selected text layer to its initial {placeholder}. this means there can only be ONE placeholder per layer.
we did this because sometimes, we were quite irritated ourselves when we used an already populated layer, overwrote its data with another placeholder – but populating would always use the original placeholder, because it is stored in the layer as meta data. that's quite unpredictable.
also, we wanted a 'clear' that's really solid and works across different versions of the plugin (meta data on layers get usually linked to a specific plugin identifier). this is something we changed/added at some point.
for us, this behavior works much better and is above all predictable. no surprises.
if this makes your work more inefficient, could you please elaborate a bit more on what your desired 'clear' behavior would be?


Custom Icons?

The main maplace says 'Markers and custom icons’ but i cannot find any documentation of how to accomplish this, is it possible with the current release?

Hi,
First to say very nice plugin for maps. But im facing some issue here. I used oprtions like below:
locations        : [
{
lat: -6.210122,
lon: 106.848057
},
{
lat: -6.914863,
lon: 107.608413
},
],
map_div          : '#route_map',
type             : 'directions',
directions_panel : '#directions',
map_options      : {
zoom             : 16,
disableDefaultUI : true,
streetViewControl: false,
mapTypeControl   : false,
},
generate_controls: false,
show_markers     : false,
draggable        : true,
what i want to ask is, even i change the zoom option value its didnt take effect not zoom in and zoom out.
Second, the icon options for marker. even i setup icon option like this icon: '/assets/img/marker.png' or document.location.origin + '/assets/img/marker.png' the marker did not change.
Any clue for this issue? anything else, very nice to used.
Thank you very much,
Nucleo1985


update go_spec documentation: fix casing typo

camelCase and PascalCase are different cases, but they are often confused
https://en.wikipedia.org/wiki/Naming_convention_(programming)#Examples_of_multiple-word_identifier_formats

Thanks for your pull request. It looks like this may be your first contribution to a Google open source project (if not, look below for help). Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA).
📝 Please visit https://cla.developers.google.com/ to sign.
Once you've signed (or fixed any issues), please reply here with @googlebot I signed it! and we'll verify it.
What to do if you already signed the CLA
Individual signers

It's possible we don't have your GitHub username or you're using a different email address on your commit. Check your existing CLA data and verify that your email is set on your git commits.

Corporate signers

Your company has a Point of Contact who decides which employees are authorized to participate. Ask your POC to be added to the group of authorized contributors. If you don't know who your Point of Contact is, direct the Google project maintainer to go/cla#troubleshoot (Public version).
The email used to register you as an authorized contributor must be the email used for the Git commit. Check your existing CLA data and verify that your email is set on your git commits.
The email used to register you as an authorized contributor must also be attached to your GitHub account.

ℹ Googlers: Go here for more info.

@googlebot I signed it!

Sounds good!


I want to finetune model with a few Chinese datasets, Could you give some datasets format? thank you!

if you can provide same format datasets on huggingface, that's so great. thank you!
or you can paste some examples, O(∩_∩)O~~

OK， I find from here. thanks
https://github.com/MeetKai/functionary/issues/99


Upgrading from 0.6.5 to 0.6.7

Seems like 'Inline_edit' and 'inline_add' aren't working properly when updating from 0.6.5 to 0.6.7. It doesn't shows Edit buttons (inline) even when forced to :true.

How to reproduce, and what kind of trouble do you have?

I' had a conditional inline_edit button. It was all ok at 0.6.5, but at 0.6.7 its not rendering when needed. At the attached images an example.
V. 0.6.5:

V.0.6.7:

It seems related to to 'visible' option that is set in the config.model block.
Prior 0.6.7, the 'visible' option was hiding the model in the navigation sidebar only but would let you use the inline add/edit buttons, which was perfect.
In 0.6.7, i have tested that if you set 'visible' to false in that config, the inline add/edit button also disappear which is a problem.

+1 same thing

This is your culprit: https://github.com/sferik/rails_admin/commit/10655cbfe972a2972b6580175f8be7fe2e20572c
I got stung by something else from this commit.
It seems that visible has acquired a huge range of meaning all of a sudden. It initially just configured nav bar presence, but now it has come to mean something like "enabled".

The change in 10655cb is okay. We do not want disabled actions to show up - by default.
The problem is that enabled? was checking model's visible?. In that way, hiding a model leads to corresponding actions in-executable.
Please try with the latest master, and see if it works for you.

my issue #2330 is caused by this change, but how exclude a model (like join table) and keep his attributes accessible in a nested form.


cloud_admin modification or wrapper to record/log commands with comments.

Command should require a reason/description for the change and maintain a time stamped log of all commands issued as per Ryan T's and Randy's request..

That's a good idea. We do something like that with our batch worker nodes too. So this information would be logged to a separate file? Like a changelog for tracking incidents?

will use a separate log file, by default going to require all cloud_admin commands to include -C "comment" arg but include option to disable.


ModelGen: [BodyStream] model option is not being validated correctly

This isn't a high priority right now.  I'm going to disable the check for this API via a [NoControllerValidation] for now and come back to this later.


Still using v1beta1 crds

Upgrading to the v1.x.x version of this library should use the v1 karpenter crds. v1beta1 is hardcoded causing errors when trying to deploy nodepools into v1 clusters.

@misterjacko this should be fixed with the release of v1.0.8 and the work in #204. Closing this for now, feel free to re-open if the issue persists.


love_11: 11.1 -> 11.2

Semi-automatic update generated by https://github.com/ryantm/nixpkgs-update tools. This update was made based on information from https://repology.org/metapackage/love/versions.
meta.description for love_11 is: '"A Lua-based 2D game engine/scripting language"'.

Checks done (click to expand)

built on NixOS
/nix/store/1cjsmn6ln1zwhk0d1sikbg6s8jwblz6i-love-11.2/bin/love passed the binary check.
1 of 1 passed binary check by having a zero exit code.
0 of 1 passed binary check by having the new version present in output.
found 11.2 with grep in /nix/store/1cjsmn6ln1zwhk0d1sikbg6s8jwblz6i-love-11.2
directory tree listing: https://gist.github.com/5198144b1b16af64b545e9bda41512ab
du listing: https://gist.github.com/87829d3ab23c6bdfc5e1fe6d16fde23c

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

3 total rebuild path(s)
1 package rebuild(s)
1 x86_64-linux rebuild(s)
1 i686-linux rebuild(s)
0 x86_64-darwin rebuild(s)
1 aarch64-linux rebuild(s)
First fifty rebuilds by attrpath
love_11

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/1cjsmn6ln1zwhk0d1sikbg6s8jwblz6i-love-11.2 \
  --option binary-caches 'https://cache.nixos.org/ https://r-ryantm.cachix.org/' \
  --option trusted-public-keys '
  r-ryantm.cachix.org-1:gkUbLkouDAyvBdpBX0JOdIiD2/DP1ldF3Z3Y6Gqcc4c=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(r-ryantm's Cachix cache is only trusted for this store-path realization.)
Or, build yourself:
nix-build -A love_11 https://github.com/r-ryantm/nixpkgs/archive/24b116160450a5759f83253bb813ea6d7104804b.tar.gz

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

cc @7c6f434c for testing.

@7c6f434c @r-ryantm the hash on this update was broken, and ofborg's build checks each failed. Is the new style of ofborg check not adequate? Maybe I should revert that for failed builds.

IRC discussion context:

https://logs.nix.samueldr.com/nixos/2018-12-17#1545061202-1545061841;

@7c6f434c @r-ryantm the hash on this update was broken, and ofborg's build checks each failed. Is the new style of ofborg check not adequate? Maybe I should revert that for failed builds.

I am not sure I waited for build checks — I thought that the bot has built the update somehow… Apparently something interesting (unstable hash for VCS download?) has happenned.

Comment on failure might be nice, though

Why are these build failures tagged a "Neutral" (with a non-alarming grey color)?

Because there is no separation of of build failures by type, and many build failures are indeed neutral.

5256cb8353c120e7507aecb579ec60441fa9bfa9

It looks like the problem might be with nix-prefetch-url and not them changing the source. Can anyone else confirm this output using https://github.com/NixOS/nixpkgs/commit/5256cb8353c120e7507aecb579ec60441fa9bfa9?
[ryantm@home1:~/p/nixpkgs]$ nix-build -A love_11.src
these paths will be fetched (2.64 MiB download, 14.25 MiB unpacked):
  /nix/store/fg2hz90z5bc773gpsx4gfxn3l6fl66nw-source
copying path '/nix/store/fg2hz90z5bc773gpsx4gfxn3l6fl66nw-source' from 'https://cache.nixos.org'...
/nix/store/fg2hz90z5bc773gpsx4gfxn3l6fl66nw-source
[ryantm@home1:~/p/nixpkgs]$ nix-prefetch-url -A love_11.src
unpacking...
path is '/nix/store/5v19bw7fggsfd3jx38qdl7i0xw3gy2mg-source'
0bp22mzkjy48gncj5vm9b7whzrggcbs5pd4cnb6k8jpl9j02dhdv

[ryantm@home1:~/p/nixpkgs]$ nix-prefetch-url -A love_11.src

Wait, does nix-prefetch-url even support fetchFromBitBucket? It might have fetched whatever it could by the provided URL…

nixpkgs-update uses nix-prefetch-url to get the hash for the src attribute, so if fetchFromBitBucket doesn't work with that, I need to blacklist fetchFromBitBucket until I find a better way to get the hash.


tag_by_labels: remove the plugin

fixes osbs-7626
tag_by_labels is no longer part of any osbs-client arrangement, so
remove it from atomic-reactor.
Signed-off-by: Mark Langsdorf<EMAIL_ADDRESS>Maintainers will complete the following section:

[x] Commit messages are descriptive enough
[x] "Signed-off-by:" line is present in each commit
[x] Code coverage from testing does not decrease and new code is covered
[x] JSON/YAML configuration changes are updated in the relevant schema
[x] Changes to metadata also update the documentation for the metadata
[x] Pull request includes link to an osbs-docs PR for user documentation updates

Updated Release Notes to include tag_by_labels in the list of removed plugins


Import PlaceResult in Angular 6 and TypeScript 2.9.2

I'm converting an existing application from Angular 5 to Angular 6 and TypeScript 2.9.2 and the compiler is complaining that it doesn't like:-
import { } from 'googlemaps';

so I changed it to:-
import { google } from “@agm/core/services/google-maps-types”;

so a statement like:-
const autocomplete = new google.maps.palces.Autocompete(…);

works. But when use this:-
const place: google.maps.places.PlaceResult = autocomplete.getPlace();

and I get an error saying:-
Cannot find namespace ‘google'

I want to change that to:-
const place: PlaceResult = autocomplete.getPlace();

but I can figure out how to import PlaceResult.
Where do I import PlaceResult from? Is this the right approach or is there some other way?
I’m using the latest versions of<EMAIL_ADDRESS>and<EMAIL_ADDRESS>and Angular 6.1.10.

PlaceResult is not defined in this package. If you would like to use @types/googlemaps which I am trying to merge with #1648 you can use the following:
in your index.ts or somewhere that's going to be included in your TS build (it only needs to be declared once)
/// <reference types="@types/googlemaps" />

In your file you can now use the type definition you expected above:
const place: google.maps.places.PlaceResult = autocomplete.getPlace();

However you have to make sure the Google Maps API has been loaded before you call any methods that use google maps. This means you cannot use class constants or inline definitions, but instead must move the calls into the constructor and resolve a promise when the API has been loaded.
You can do this by importing GoogleMapsAPIWrapper and awaiting on .createMap(...) or .getNativeMap() or using the MapsAPILoader directly.

For now you probably want to try `<EMAIL_ADDRESS>since google maps types import is broken as reported in #1729
However, you probably shouldn't be importing import { google } from "@agm/core/services/google-maps-types"; regardless. I'm not sure how the TypeScript compiler emits that, but the google namespace is a global and in that file it probably should have been written as declare var google: any instead of being an exported variable.

I wouldn't use @types/googlemaps.
You need to do two things.

define var google at top of component
only initalize google maps objects after (mapReady) event


[Suggestion] It would be better if the prompt information is "Select Database" in the input box when selecting a database

OS: All
Azure Resource Extension Build Version: 20220426.4 (unified)
Azure App Database Extension Build Version: 20220422.3 (unified)
Regression: Not a regression
Repro Steps:

Click "Connect to a database" in an opened Mongo Scrapbook -> Select a subscription -> Select "Azure CosmosDB" -> Select a resource.
Check whether the prompt information is clear in the input box or not.

Actual:
The prompt information is unclear in the input box.

Suggestion:
It would be better if the prompt information is "Select Database".

Fixed by #1998


[feature-wip](partial update) support MOW partial update in stream load

Proposed changes
Issue Number: close #xxx
Problem summary
Describe your changes.
Checklist(Required)

[ ] Does it affect the original behavior
[ ] Has unit tests been added
[ ] Has document been added or modified
[ ] Does it need to update dependencies
[ ] Is this PR support rollback (If NO, please explain WHY)

Further comments
If this is a relatively large or complex change, kick off the discussion at<EMAIL_ADDRESS>by explaining why you chose the solution you did and what alternatives you considered, etc...

run buildall

run buildall

run buildall

TeamCity pipeline, clickbench performance test result:
the sum of best hot time: 34.65 seconds
stream load tsv:          450 seconds loaded<PHONE_NUMBER>9 Bytes, about 158 MB/s
stream load json:         24 seconds loaded<PHONE_NUMBER> Bytes, about 93 MB/s
stream load orc:          73 seconds loaded<PHONE_NUMBER> Bytes, about 14 MB/s
stream load parquet:          31 seconds loaded 861443392 Bytes, about 26 MB/s
https://doris-community-test-1308700295.cos.ap-hongkong.myqcloud.com/tmp/20230321201317_clickbench_pr_118130.html

run buildall

run buildall

run buildall

run buildall

run buildall

run feut

run buildall

run buildall

run buildall

run buildall

run buildall

run buildall

run buildall


Ustream

Checklist

[ ] I'm reporting a broken site support issue
[x] I've verified that I'm running youtube-dl version 2019.04.30
[x] I've checked that all provided URLs are alive and playable in a browser
[x] I've checked that all URLs and arguments with special characters are properly quoted or escaped
[x] I've searched the bugtracker for similar bug reports including closed ones
[x] I've read bugs section in FAQ

Verbose log

PASTE VERBOSE LOG HERE

Description

WRITE DESCRIPTION HERE
Could anyone tell what is the problem?
youtube-dl -v https://www.ustream.tv/embed/23240580?html5ui
[debug] System config: []
[debug] User config: []
[debug] Custom config: []
[debug] Command-line args: ['-v', 'https://www.ustream.tv/embed/23240580?html5ui']
[debug] Encodings: locale cp932, fs mbcs, out cp932, pref cp932
[debug] youtube-dl version 2019.04.30
[debug] Python version 3.4.4 (CPython) - Windows-10-10.0.17763
[debug] exe versions: ffmpeg N-92246-gc2ac3b8e6a, ffprobe N-92246-gc2ac3b8e6a
[debug] Proxy map: {}
[ustream] 23240580: Downloading webpage
ERROR: Unable to extract content video IDs; please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; type  youtube-dl -U  to update. Be sure to call youtube-dl with the --verbose flag and include its complete output.
Traceback (most recent call last):
File "C:\Users\dst\AppData\Roaming\Build archive\youtube-dl\ytdl-org\tmpfeq7l785\build\youtube_dl\YoutubeDL.py", line 796, in extract_info
File "C:\Users\dst\AppData\Roaming\Build archive\youtube-dl\ytdl-org\tmpfeq7l785\build\youtube_dl\extractor\common.py", line 529, in extract
File "C:\Users\dst\AppData\Roaming\Build archive\youtube-dl\ytdl-org\tmpfeq7l785\build\youtube_dl\extractor\ustream.py", line 178, in _real_extract
File "C:\Users\dst\AppData\Roaming\Build archive\youtube-dl\ytdl-org\tmpfeq7l785\build\youtube_dl\extractor\common.py", line 1004, in _search_regex
youtube_dl.utils.RegexNotFoundError: Unable to extract content video IDs; please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; type  youtube-dl -U  to update. Be sure to call youtube-dl with the --verbose flag and include its complete output.

I've the same issue and here is my verbose log. Thank you team for any insight.
D:\Downloads>D:\Downloads\cli\youtube-dl.exe --verbose https://www.ustream.tv/embed/20082839 [debug] System config: [] [debug] User config: [] [debug] Custom config: [] [debug] Command-line args: ['--verbose', 'https://www.ustream.tv/embed/20082839'] [debug] Encodings: locale cp1252, fs mbcs, out cp437, pref cp1252 [debug] youtube-dl version 2020.03.24 [debug] Python version 3.4.4 (CPython) - Windows-10-10.0.18362 [debug] exe versions: none [debug] Proxy map: {} [ustream] 20082839: Downloading webpage ERROR: Unable to extract content video IDs; please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; type  youtube-dl -U  to update. Be sure to call youtube-dl with the --verbose flag and include its complete output. Traceback (most recent call last):   File "C:\Users\dst\AppData\Roaming\Build archive\youtube-dl\ytdl-org\tmpjwbwqymm\build\youtube_dl\YoutubeDL.py", line 797, in extract_info   File "C:\Users\dst\AppData\Roaming\Build archive\youtube-dl\ytdl-org\tmpjwbwqymm\build\youtube_dl\extractor\common.py", line 530, in extract   File "C:\Users\dst\AppData\Roaming\Build archive\youtube-dl\ytdl-org\tmpjwbwqymm\build\youtube_dl\extractor\ustream.py", line 178, in _real_extract   File "C:\Users\dst\AppData\Roaming\Build archive\youtube-dl\ytdl-org\tmpjwbwqymm\build\youtube_dl\extractor\common.py", line 1005, in _search_regex youtube_dl.utils.RegexNotFoundError: Unable to extract content video IDs; please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; type  youtube-dl -U  to update. Be sure to call youtube-dl with the --verbose flag and include its complete output.

It happens for live-only embed streams for which ustream.vars.uumfOffAirContentVideoIds=[].
(url: https://www.ustream.tv/embed/{video_id}?html5ui=1)
I tried skipping the 2 if m.group('type') and 'https://api.ustream.tv/videos/%s.json' % video_id to go directly to _get_streams and _get_stream_info:

the first self._download_json on 'http://r%d-1-%s-recorded-lp-live.ums.ustream.tv/1/ustream' % (rnd(1e8), video_id) does work:
[{'args': [{'host': 'xxxx-lp-live.ums.services.video.ibm.com', 'connectionId': 'xxxxxxxx-xx'}], 'cmd': 'tracking'}]
but the second on 'http://%s/1/ustream?connectionId=%s' % (host, connection_id) fails: [{"args":[{"code":"INVALID_CONNECTIONID","message":"Invalid connectionId"}],"cmd":"warning"}]

Anyway, the web page seems to get all its data from a websocket at wss://rxxx-1-{video_id}-channel-wss-omega.ums.ustream.tv/1/ustream with all the cdn config host and chunks metadata (among other things, like thumbnails).

The problem still remains to be solved. Unable to extract content video IDs from web pages
https://ustream.tv/channel/24046197
https://ustream.tv/channel/24038333