Explicilty map tag => tags on aws.autoscaling.Group

We have an awkward mapping of tag=>tags and tags=>tagsCollection on aws.autoscaling.Group.  This leads to some confusion in our tfbridge mapping.
For now, we can fix #293 by making the tag=>tags part of the mapping explicit.  Longer term, we likely need to make deeper improvements in tfbridge to handle this and/or come up with a different approach for aws.autoscaling.Group projection of tag/tags.
Fixes #293.

I can confirm that this fixes the reproduction of the issue I had from #293! Thanks!


Update to Rescript compiler v10.1?

I would like to use async/await from the 10.1 version of the compiler.
Can you provide instructions to build it or are you planning on updating the package soon?

The library works fine with a project using 10.1, what it doesn’t have is APIs that leverage async/await.
I’m not sure when I’ll have time for that, or whether to add new apis side by side rather than force people to upgrade. Contributions definitely welcome.

Apologies for not following up. A Promise submodule was added to the fs API in #14, which works with async/await:
https://github.com/TheSpyder/rescript-nodejs/blob/913a14bf229472128c21ed4c78ed83832216fae0/test/atomic/Fs.test.res#L5-L12
I think (much like node itself) this is a good path forward. I hope to add more over time - if there's a particular API you want a promise mapping for please let me know and I'll do it before others.


Review Simulation API sections

Review Simulation API sections
Creating and running simulations:

Notebook 3.5
https://gridappsd-training.readthedocs.io/en/latest/api_usage/3.5-Creating-Running-Simulation-API.html

Publishing and subscribing:

Notebook 3.6
https://gridappsd-training.readthedocs.io/en/latest/api_usage/3.6-Controlling-Simulation-API.html

3.5-Creating-Running-Simulation-API

Quiz-for-Lesson-3.5 requires content.


Mgmt: remove catch exception Samples

catch exception make playback harder to see the exception, so change

I add the PR is because I think it should be changed. Otherwise, I would just close it. (I don't need such PR for testing)
I don't think catch a whole exception in runSample is a good choice (for running, actually the main function has another catch).

This mainly reason is for future playback, Since playback test errors in Azure and Samples can hardly find the reason in log.

Then don't write such reason like

catch exception make playback harder to see the exception, so change

if you do not do the change because of it.
Also, generally speaking, method throw Exception type is not recommended.
Personally, for try-catch in sample, it is reasonable to do so, if the try-catch actually is in a place that potentially have error (e.g. no permission, no access, file not found, failed to create credential, etc.), with good suggestion in message, which helps user to successfully run the sample. Though I guess none of us bother to improve this at present.
If all of these are just routine catch above finally, you are OK to remove them.

My main reason is for playback (future playback in pipeline) so I write it in the first comment.
throw Exception is because I don't want to add every exception manually. Is it any fast way to add such specific exception?
All remove catch blocks are directly followed by return true (with which I write the regex)

It seem the UrlParer has problem?
Previously, I also saw another from it (cast error from HashMap#Node to HashMap#TreeNode)
Both are Java 8 on MacOS

I guess majority of the samples does not need a throws, except for a few cases that got other than RuntimeException.

changed to specific exception


Autocomplete issues on MacOs

Expected behavior
Autocomplete should work as described in https://docs.myaltimate.com/develop/autocomplete/

typing ref inside curly brackets should create a ref() placeholder and autocomplete should propose the list of models
typing source inside curly brackets should create a source placeholder and autocomplete should propose a list of databases first and then a list of source tables

Actual behavior
A few issues happen:

additional curly brackets are added when using the autocomplete inside curly brackets to add a ref our source
the ref placeholder created from the auto-complete includes single quotes by default and auto-complete is not proposed within those single quotes. When deleting the single quotes then the auto-complete works
autocomplete is not proposed for source macro regardless of the quotes being there or not

Steps To Reproduce

type "{{ ref" and select "ref" from auto-complete menu. this will result in "{{ {{ ref('') }} }} with the cursor after the first single quote
type anything inside {{ ref('<cursor_here>') }}. This won't trigger auto-complete menu to show up
type anything inside {{ source('<cursor_here>','') }}. This won't trigger auto-complete menu to show up

Log output/Screenshots

Operating System
macOs
dbt version
1.5.3
dbt Adapter
bigquery
dbt Power User version
v0.28.4
Are you willing to submit PR?

[ ] Yes I am willing to submit a PR!

The fix for
"typing source inside curly brackets should create a source placeholder and autocomplete should propose a list of databases first and then a list of source tables" was pushed in version 0.38.4


Accidental show recursion crashes compiler

show heading: it => {
  line(length: 100%)
  heading(numbering: "1:", it)
  line(length: 100%)
}

This is already being discussed in #229


Input data validation

Version: 2.0
I'm constantly running into issue with 'corrupted' data during development - like I accidentally store into "LINK" property a string instead of valid RID. It is stored as "#-1:-1" value.
This ends up with error:
SEVERE Error on unmarshalling field 'ouser' of record 'User{displayName:John Zemi}': value '#5:5"' is not a link [ORecordSerializerSchemaAware2CSV]

This also causes Studio not being able to fetch data from that class anymore with error: "The command has not been executed"
POST request:
http://localhost:2480/command/test/sql/-/20?format=rid,type,version,class,graph
select * from User
With response (notice ouser property):
{
"result":
[
{"@type":"d","@rid":"#12:3","@version":1,"@class":"User","displayName":"John Zemi","ouser":{null}}
]
}

Proposal
OrientDB validates input values so that incorrect data won't be stored but exception is raised instead.

OrientDB already validates it if you setup the property as LINK in schema.

That's weird - I'm using https://github.com/codemix/oriento adapter with binary protocol.
I still believe though, even when adapter is doing something wrong, DB should not allow this to happen.
I'll try to discuss my use-case with project maintainers, although it would be nice to get your opinion on this too and direct me right way.

@matuszeman in oriento you must explicitly mark a RID (Link) by wrapping it in an Oriento.RID object, otherwise it'd be impossible to differentiate RIDs from strings.

@phpnode Thanks, but I think I tried that already also - I will try again tonight. But question still remains - why insert like that is not rejected? Is "#-1:-1" considered being valid value for LINK properties? If so, it would be nice if it didn't brake when I query from Studio.


[Addition]: 9.2 HNT in a day, 7 day average beacon: 1,251   total witnessed 751

Hotspot b58 Addresses
11gsd5n23BZZ4k26oR6do89ayfj8faCryBUiTEJ6hLjYcunk65c
Discord Handle
Dannyjv4#4883
Reason(s)
Obviously gaming the system

https://github.com/helium/denylist/issues/2844


fix(dependencies): fix vuepress dependencies

Please check if the PR fulfills these requirements

[X] The commit message follows our guidelines
[X] Tests for the changes have been added (for bug fixes / features)
[X] Docs have been added / updated (for bug fixes / features)

What kind of change does this PR introduce? (Bug fix, feature, docs update, ...)
bugfix vuepress docs:build broken dependencies

What is the current behavior? (You can also link to an open issue here)
vuepress step  is in error in the CICD

What is the new behavior (if this is a feature change)?
vuepress step is passing : the documentation website is generated as expected

Does this PR introduce a breaking change? (What changes might users need to make in their application due to this PR?)
No

Other information:

@over-flo79 we had this issue before on Konviw and separated the Vuepress deps to a separate package.json and build workflow (and also separately upgraded to the latest Vuepress).
I would recommend checking that solution as it works well and is pretty fast to implement.

@MeStrak ok nice ! i'll do the same so


[sdl2-gfx] Update CMake build and find_package support

This allows for using sdl2-gfx easily by generating the proper files (sdl2-gfx-config.cmake and such) when building and providing it to vcpkg-cmake-wrapper.cmake to allow easy find_package() usage.
Makes it easy to just find_package(sdl2-gfx) target_link_libraries(main PRIVATE SDL2::SDL2_gfx) (as suggested by the porting guidelines) instead of having to mess with other methods.
I tried to follow a similar CMake build process to the SDL2_image port.
Just a fun note, I didn't know that vcpkg automatically generates the link usage message (The package sdl2-gfx:x64-windows provides CMake targets:) which I thought was cool.

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.:x: dhkatz sign nowYou have signed the CLA already but the status is still pending? Let us recheck it.

Seems like OSX CI has been broken for almost 2 weeks.

Sorry about that, should be good now.

Thank you everyone for the feedback, I will remember this for when I hopefully contribute more in the future.

Could you remove ignition-math4:x64-windows-static=fail , ignition-math4:x64-uwp=fail and add libqglviewer:x64-windows-static=fail from VCPKG_PATH/scritps/ci.baseline.txt ?
I will open a PR to fix libqglviewer build failure.

/azp run


"Start Debugging" is auto-saving open files even for unchanged files

Since updating today now when I try debugging visual studio codes saves the active file (even though it already is saved) this causes my gulp script to see the change and rebuild/restart node which means node does not show up in the attach to process list. I have "files.autoSave": "off" in my config.

VSCode Version: 1.10.1
OS Version: Windows 10

Steps to Reproduce:

Open a js file
Click the green play button in the debug view

@spacem yes this is how start debugging is behaving by design. We could introduce an configuraiton to not autosave on start.
@bpasero shouldn't the save operation be a no-op when there are no changes?

@isidorn I suggest you add a breakpoint here to find out why it saves, the reasons are explained in that location: https://github.com/Microsoft/vscode/blob/master/src/vs/workbench/services/textfile/common/textFileEditorModel.ts#L600

Another case where a save can cause something to happen is when a save participant is installed that wants to change the file (even if it is not dirty).

This issue has broken my incremental build feature.
Saving clean files when launching a debug session is not a good idea. Like many designs, my debugger tests whether a rebuild is necessary by comparing the file times of the source files with the file times of the output. If the source files are newer, a rebuild is necessary.
Performing a save on every debug session causes the source files to have a later timestamp, forcing a rebuild every time.
Can you revert this back to the original behaviour.

Also not sure if it's related, but now nodemon restart the server when I start to  debug. The net result is that I get a timeout error everytime now (related bug https://github.com/Microsoft/vscode/issues/21783)

@bpasero what changed is that now I am explictly calling the workbench.action.files.save due to https://github.com/Microsoft/vscode/issues/19415
In textFileEditorModel line 600 all those fields are true because the save action wants to trigger all external file watchers. And that is bascially the difference between the workbench.action.files.save and calling saveAll on the fileService as we had done before.
@adelphes @alemhnan @spacem yes, we will revert back to the original behavior and you can pick up the fix in tomorrow's vscode insiders.
For now I am not sure if this should be treated as a candidate for the recovery build. @bpasero opinion?

@bpasero sounds good to me. I can look into it later tonight.

@bpasero I introduced a new Save if Dirty action, currently I registered as a regular action so it appears in the command palette which I am not sure is the best.
@adelphes @alemhnan @spacem tomorrow's insiders will contain the fix for the issue. Can you please try it out. Thanks

Sorry I do not know what you mean by "tomorrow's insiders". I did a "check for updates" and it said none were found.
One thing which maybe is a different issue is I tried just closing all my open windows and starting debug. This didn't save any files however it gives the error "Attach to process: '${command.PickProcess}' doesn't look like a process id.".

@isidorn good point, alternatively we could introduce a command with "_" to exclude it from the F1 list until someone asks for it.

@spacem VS Code also has an insiders version which updates every day. You can get it from here http://code.visualstudio.com/docs/?dv=win&build=insiders
@bpasero I will remove the '...' from the label of the command

Tested with today's insider build...
2017-03-03T07:08:55.738Z
84e56fc527047223a6ad22c31a906d4a579680d0
...and it works. It no longer saves clean files when launching a debug session.
A slightly confusing behaviour linked to this is that if you have multiple dirty files open, only the currently active editor is saved when debugging is started. Is this expected?

@adelphes good point. Looking into that I figured out that a neater solution is to simply call 'workbench.action.files.saveAll' action that does everything correctly. Saves all and does not force a write. I have pushed this and will be available in tomorrow's insiders. It will also be availble in VSCode stable some time next week.
fyi @bpasero

Now when I have an untitled file open, it brings up a save dialog every time I start debugging. I frequently use untitled files as scratch pads, so this is annoying. How about saveAllIfDirty?

@isidorn you can use workbench.action.files.saveFiles to only save files, but I thought this feature was about being able to debug even an untitled file?

@roblourens this feature was about being able to debug untitiled files #19415. Let's wait if somebody else complains other than Rob and we can look into improving this. ATM treating this as designed behavior

Ok, I'll get someone else to complain about it :)
It's really annoying...

Verified neither the current nor another file get an updated modification timestamp unless they are dirty when starting to debug.

For users who are no relying on hot-exit - as I do - this is the old behavior right after F5:

This is the new behavior. Every time I now asked to save, although I don't intend to save any of the untitled files. This is super annoying.

Like Ben suggested, calling workbench.action.files.saveFiles at https://github.com/Microsoft/vscode/blob/master/src/vs/workbench/parts/debug/browser/debugActions.ts#L130 won't save untitled files.
We could also remove that call to not save at all when starting to debug, and assume that people will use autosave if they want it to save there.

Having talked with @Tyriar about his use of the dirty state I'd vote for not saving at all. Auto-save can be used when not interested in managing the dirty state manually. (Disclaimer: I'm using auto-save.)

@isidorn to preserve most of the behaviour we want and NOT trigger the "save even if not dirty", I suggest you bring back the action we had in the first place which is to subclass the save action and not force it to save unless the file is dirty. To make this impact minimal, I suggest to register the action in the same way we do it for SaveFilesAction in https://github.com/Microsoft/vscode/blob/master/src/vs/workbench/parts/files/browser/fileActions.contribution.ts#L215 so that it does not show up in the F1 list.

Now we are only saving the current active file if not dirty before debugging, which is a slight change as to what we were doing before but I think it makes a lot of sense overall.
@bpasero please code review and verify

After discussions we have decided to go back to the old behavior we were doing in the previous releases. We are saving all untitiled non dirty files before debug starts.
This is without a behavioral change and has the least amount of risk associated with it. The only downside is that we had to reopen #19415
@bpasero please code review and verify


HP 3 does not allow id other than "1"

Looks like the "id" logic in HP 3 does not work as I always get:
Module build failed: AssertionError: HappyPack: plugin for the loader '1' could not be found! Did you forget to add it to the plugin list
when I create a HappyPack object with any string other than 1 or null. I tested using the 2.x id loader parameter (now removed) and without - no difference

nevermind - I wasn't passing id to 'happypack/loader'


POST /clients should not require subscription

I'd like the ability to register clients against the API without having to specify a subscription.
Expected Behavior
Proxy clients created via the source attribute in checks are configured with a single client subscription, i.e. client:i-424242.
I believe this behaviour should be consistent when registering a client manually via the /clients api, giving a user the option to not specify any additional subscriptions and just having the client default automatically added as per https://github.com/sensu/sensu/blob/master/lib/sensu/api/routes/clients.rb#L22
Current Behavior
Currently registering a client via the /clients API will create a client subscription however an additional subscription must be passed as well as part of the payload:
201 Created:
{
  "name": "example",
  "address": "<IP_ADDRESS>",
  "subscriptions": ["sub"]
}

400 Bad Request:
{
  "name": "example",
  "address": "<IP_ADDRESS>"
}

Possible Solution
Client validation fails due to https://github.com/sensu/sensu-settings/blob/master/lib/sensu/settings/validators/client.rb#L9-L16
Context
Registering clients for use with proxy requests requires you to pass a subscription, even if not required.
It's only a minor thing, since you can just pass the client subscription in the payload, and it will be merged with the default, ultimately just leaving the one subscription. This did however catch me out since I assumed you could opt to leave out the subscription and just have Sensu create the client default.
Easily worked around.

Seems reasonable. Assigning to 1.3 milestone.


Make WITH_COLLECTION work with Builder's dev env (.io)

From Nacho (https://decentralandteam.slack.com/archives/C01UXV234JG/p1634667844206700):

One more thing (This is something that can wait until the festival finished). Is there any reason why we are not cheking the network and always use builder org here https://github.com/decentraland/kernel/blob/main/packages/shared/catalogs/sagas.ts#L175? THis is preventing for testing wearables in our dev environment

added BUILDER_SERVER_URL to specify a builder server url


fix compat table

Fixes #N/A
Description
How was this change tested?
Does this change impact docs?

[ ] Yes, PR includes docs updates
[ ] Yes, issue opened: #
[x] No

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Warning: This coverage report may be inaccurate.
We've detected an issue with your CI configuration that might affect the accuracy of this pull request's coverage report.
To ensure accuracy in future PRs, please see these guidelines.
A quick fix for this PR: rebase it; your next report should be accurate.

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.02%) to 82.599%

Totals

Change from base Build<PHONE_NUMBER>:
-0.02%

Covered Lines:
5003

Relevant Lines:
6057

💛  - Coveralls


Lastest update breaks the UI

It was OK in previous version, but this morning it asked me to install the new version then the UI is broken.

Actually even easier, can you hit CMD-R to refresh the page and let me know if it's still broken?

It's ok after the screen sleep. That's so strange
On Thu, Nov 22, 2018, 9:31 AM Jerrold Poh<EMAIL_ADDRESS>wrote:

Actually even easier, can you hit CMD-R to refresh the page and let me
know if it's still broken?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/danielbuechele/goofy/issues/431#issuecomment-440894136,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AEBZa9Zdc0fNx2ghgd2unDYaaGzyvuTbks5uxgxugaJpZM4YuiV8
.

No worries.  I'll close the issue for now but feel free to reopen if you run into it again 🙂


(rsocket-ts) feat: TCP client scaffolding with setup frame serialization

Establishes a baseline for patterns on implementing transports, serializer, and encoding.
Modifications:

TCP Transport w/ sendOne implementation
Setup Frame serialization implementation

Result:
Can establish a TCP connection to an RSocket TCP server and send a Setup Frame. See example.

Closing in favor of #12. TCP transport and frame serialization will be implemented on top of the foundation #12 provides.


Message retransmission with QoS 1

Describe your question
I'm studying MQTT 3.1.1 and testing MQTTnet, and I have a question about the message retransmission with QoS 1.
I've understood that it's possible to resend a pulished message for which the PUBACK is not received, using the DUP flag.
In other docs is mentioned that usually the resend mechanism is handled by the client library:
DUP flag The flag indicates that the message is a duplicate and was resent because the intended recipient (client or broker) did not acknowledge the original message. This is only relevant for QoS greater than 0. Usually, the resend/duplicate mechanism is handled by the MQTT client library or the broker as an implementation detail.
https://www.hivemq.com/blog/mqtt-essentials-part-4-mqtt-publish-subscribe-unsubscribe/
Does MQTTnet handle the retransmission?
It seems also that the DUP flag is not publicly exposed. How is it possible to resend messages with MQTTnet?
Which project is your question related to?

Client
ManagedClient
Generic

Unfortunately you cannot set it manually in this library and the client will not retry a transmit on its own. You will only get an exception. But I can expose this property so that users can set it manually.

@chkr1011, thank you for your reply.
Since the flag is not exposed, I suppose that the DUP flag and message retransmission are not very common in MQTT application, isn't it? I've not found much information in mosquitto docs, too.

Well it is used by the MQTTnet Server implementation because it has internal queues so that it knows when a message was sent and retries it so setting the dup flag is easy. The clint is a different thing because an exception is thrown when no ACK is sent for QoS 1.

The Dup flag is exposed in version 3.0.15-rc2.


keg_relocate: swap env var

[ ] Have you followed the guidelines in our Contributing document?
[ ] Have you checked to ensure there aren't other open Pull Requests for the same change?
[ ] Have you added an explanation of what your changes do and why you'd like us to include them?
[ ] Have you written new tests for your changes? Here's an example.
[ ] Have you successfully run brew style with your changes locally?
[ ] Have you successfully run brew typecheck with your changes locally?
[ ] Have you successfully run brew tests with your changes locally?

This might cause trouble for us in homebrew-core.

Review period will end on 2021-07-06 at 00:00:00 UTC.

We suspect that this will fix the error we got while bottling swig in homebrew-core for Linux:
2021-07-04T08:10:39.2645358Z ==> brew linkage --test swig
2021-07-04T08:10:40.4584391Z ==> FAILED
2021-07-04T08:10:40.4584949Z Missing libraries:
2021-07-04T08:10:40.4585497Z   unexpected (libgcc_s.so.1)
2021-07-04T08:10:40.4586085Z   unexpected (libstdc++.so.6)
2021-07-04T08:10:40.4586463Z
2021-07-04T08:10:40.4587344Z ==> brew install --only-dependencies --include-test swig
2021-07-04T08:11:14.0968351Z ==> brew test --verbose swig
2021-07-04T08:11:16.8242005Z ==> FAILED
2021-07-04T08:11:16.8244170Z ==> Testing swig
2021-07-04T08:11:16.8245327Z ==> /home/linuxbrew/.linuxbrew/Cellar/swig/4.0.2/bin/swig -ruby test.i
2021-07-04T08:11:16.8246715Z /home/linuxbrew/.linuxbrew/Cellar/swig/4.0.2/bin/swig: error while loading shared libraries: libstdc++.so.6: cannot open shared object file: No such file or directory
2021-07-04T08:11:16.8248080Z Error: swig: failed
2021-07-04T08:11:16.8248823Z An exception occurred within a child process:
2021-07-04T08:11:16.8250110Z   BuildError: Failed executing: /home/linuxbrew/.linuxbrew/Cellar/swig/4.0.2/bin/swig -ruby test.i

Review period skipped due to critical label.

It worked!


Changed teensy loader with updated command

Modified makefile target code with updated teensy loader command tyc => tycmd

Thank you!


Increase verbosity of apt-get update in build.sh

Addresses debugging needs in https://github.com/travis-ci/travis-ci/issues/9112 (maybe)

I think this change would be useful, but am wondering if we can maybe put it under some debug flag of some sorts, so it doesn't affect all builds.
@BanzaiMan is there an easy way to do this?

@bogdanap I don't think this needs to be pushed to production if there's a way for me to use a specific version of travis-build on some test jobs. Is that possible?

Looks like apt is a new command with Trusty (and not present in Precise), so we'll have to check that it exists before running. For this PR, though, it looks like we can just call apt-get uniformly.

On Precise, the command apt exists, but it's the Java Annotation Processing Tool. https://docs.oracle.com/javase/7/docs/technotes/guides/apt/ (and it exists at the same location /usr/bin/apt). Not sure what to do with it. I feel that apt-get is a safer choice.

Closing this in favor of a more general attempt of the same, #1343.


[semver:skip] Update config.yml

ERROR running force:org:create:  The certificate associated with the consumer key has expired.
I believe the sample project comes with a server.key that is now expired.


Spec EnumConstant

So far, in the spec we've been writing things like "enum of EQ, NE, GE, GT, LE, and LT" and #stablehlo<comparison_direction GE>, but this could use some further formalization.
In #840, we have specced other kinds of constants, but I didn't have the time to look into enum constants right away. It would be good to expand the "Constants" section to also include the syntax and the semantics for enum constants as well.

Update: Remain unassigned with low priority in favor of complete Spec work.

This ticket is part of Spec work, but I figured I'd unassign it because it's quite a bit of effort to formalize this (in my experience, literals always take a surprising amount of effort to handle properly) and the downside of delaying this work is relatively small (the missing formalism is well encapsulated).


Websocket Server Unidirectional packet loss

Hello, I used this library to build a websocket server to communicate with the browser. There was no problem in the local and LAN tests, but when I deployed it to the public network server, there was a serious packet loss problem.
After the connection is established, the data sent by the browser to the server is normally received, but the data sent by the server to the browser will probably be lost. Even if it is received, there will be a high delay, and the packet loss is one-way. Then the server will throw websocketnotconnectedexception.
I confirm that this is not a network or bandwidth problem, because other websocket server software on the same host is working normally
Here are some codes
`......
public class WSServer extends WebSocketServer {
public WSServer(int port) {
    super(new InetSocketAddress(port));
    setConnectionLostTimeout(10000);
}

@Override
public void onOpen(WebSocket webSocket, ClientHandshake clientHandshake) {
}

@Override
public void onClose(WebSocket webSocket, int i, String s, boolean b) {

}

@Override
public void onMessage(WebSocket webSocket, String s) {
    try {
        String[] split = s.split("@");
        int id = Integer.parseInt(split[0]);
        String key;
        String m = split[1];
        if (id == 0) {
            key = Math.md5(PanelConfig.password);
            String decrypt = Aes256.decrypt(m, key);
            AdminJson json = new Gson().fromJson(decrypt, AdminJson.class);
            new Thread(() -> {
                try {
                    Handler.execute(webSocket, json);
                } catch (WebsocketNotConnectedException e) {
                    e.printStackTrace();
                }
            }).start();
        } else {
            key = Math.md5(Daemon.getInstance().getInstanceById(id).getTags("password"));
            String decrypt = Aes256.decrypt(m, key);
            Json json = new Gson().fromJson(decrypt, Json.class);
            json.id = id;
            Instance instance = Daemon.getInstance().getInstanceById(id);
            new Thread(() -> Handler.execute(instance, webSocket, json)).start();
        }
    } catch (BadPaddingException e) {
        webSocket.send(new LoginJson(false).toString());
    } catch (Exception e) {
        webSocket.send(new LoginJson(false).toString());
    }
}

@Override
public void onMessage(WebSocket webSocket, ByteBuffer message) {

}

@Override
public void onError(WebSocket webSocket, Exception e) {

}

@Override
public void onStart() {

}

}
`
In the above code, the websocket object will be passed to the Handler.execute(WebSocket ws, Json json); and sending data to the browser is also carried out in this method. This method is blocking, but it won't last long.
I'm not a native speaker of English. I hope my expression is clear enough. Thank you

Hi! Thanks for your report.
Can you enable logging with timestamps and see when exactly the messages are sent? (Add any slf4j backend to classpath and set log level to TRACE. slf4j-simple is the easiest to set up: https://mvnrepository.com/artifact/org.slf4j/slf4j-simple/1.7.25)
What version of library are you using?

server.log

Hello, here are some logs (including logs of other modules). I use version 1.5.2

After testing, I found that the delay of each time the browser receives data is increasing, from thousands of milliseconds to hundreds of thousands of milliseconds, until it times out. The timing starts from the execution of the websocket.send() method, so it should not be the delay caused by my business code.

Hello, I did some new tests and found that the problem is not only in the public network environment, but also in the local connection on CentOS. There is no problem with the local connection on Ubuntu, and there is no problem with the public network environment on Windows system. I now suspect that this is not by org java_websocket, I'm sorry! Thank you for your help. I'll come back and close this iusse after I find out everything.
Thank you.

Hey
Does the observed behaviour change when you enable TCP_NoDelay? (https://github.com/TooTallNate/Java-WebSocket/wiki/Enable TCP_NODELAY)
Thank you very much for investigating!
Best regards
Marcel

Hello, I have found the problem. Before sending the data, I carried out aes256 encryption operation. That's the problem. Thanks for your help


feat: add process.modern

Types of changes

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

Description
Provides process.modern boolean.
Resolves #4443
Checklist:

[x] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly. (PR: #)
[ ] I have added tests to cover my changes (if not applicable, please state why)
[ ] All new and existing tests are passing.

Codecov Report

Merging #4532 into dev will increase coverage by <.01%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##              dev    #4532      +/-   ##
==========================================
+ Coverage   91.09%   91.09%   +<.01%
==========================================
  Files          61       61
  Lines        2190     2191       +1
  Branches      528      528
==========================================
+ Hits         1995     1996       +1
  Misses        177      177
  Partials       18       18

Impacted Files
Coverage Δ

packages/webpack/src/config/client.js
97.22% <ø> (ø)
:arrow_up:

packages/webpack/src/config/server.js
100% <ø> (ø)
:arrow_up:

packages/webpack/src/config/modern.js
100% <100%> (ø)
:arrow_up:

Continue to review full report at Codecov.

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


Add meta schema: a JSON Schema which matches recap schemas

What
This adds a JSON Schema describing valid recap schemas.
Why
There is a robust ecosystem around JSON Schema that provides tools like validation, example generation, code generation, etc out of the schema definitions. Defining a JSON Schema definition for Recap will allow Recap and its users to more easily hook into that ecosystem.
Closes https://github.com/recap-build/recap/issues/342
How
I manually created the JSON Schema file by reading closely the Recap Spec.
Known limitations
As mentioned in the issue, I don't know of a way to require that the alias strings are referencing defined types elsewhere in the document, so some invalid documents will pass. No documents that are valid should fail.
Validation
In order to help iterate on the JSON Schema definition (which I expect isn't perfect), I test up some unit tests. I define a few valid recap schema documents and use jsonschema to test the documents against the schema definition.
Potential future work

Define a meta schema in recap (maybe using recap converter from the json schema)
flesh out more positive or negative examples
use the JSON Schema definition for something like editor integrated validation

One other thought:
I could see an argument for being more explicit about the recap types (removing allOf/oneOf), even if it means repeating yourself. The argument for this would be that it makes the JSON schema way more readable to humans.
I'm indifferent on that argument at this point, but I could see the spec being a bit confusing to humans that are trying to read (vs. validators like python-jsonschema). I was a bit disoriented when I looked at it; I didn't actually understand until you mentioned the allOf/oneOf in the comment.

Re: the black failure, try running pdm style to get things cleaned up.

One thing I didn't catch that I think is a common gotcha: disallowing alias to be set when type reference an alias.

Did this by not repeating the alias definition for RecapAliasReference

I could see an argument for being more explicit about the recap types (removing allOf/oneOf), even if it means repeating yourself. The argument for this would be that it makes the JSON schema way more readable to humans.

I ended up doing as I was iterating on this because the validation errors were harder to understand.

Can you move unit/meta_schema/test_json_schema.py to tests/spec
And then move spec/*.json to a subdir like schemas or something?
And then update the .github/workflows/ci.yaml to have an extra job called spec-tests that runs pdm run pytest tests/spec -vv

done
moved them to spec/{in,}valid, so removed the .invalid. file sub-extension
done. I just followed the pattern of the existing jobs, but it's doing checkout+python+pdm installs for each step. That could be cached so it only has to happen once per build, or really once per new dependencies

I also realized that these tests should test that recap is validating the schemas the same way as the JSON Schema, so I added the recap validation to each.
This made me realize that there's no reason we shouldn't take every positive and negative validation example in the recap test suites and evaluate that against the jsonschema validation as well. Given the current structure of the tests, how feasible does that sound?

💥 Merged! Thanks!!
Follow-on things I found:

Migrating .json spec to recap-website
Update to pull spec from the recap-website
GH issue for logical type spec tests
(Maybe?) GH issue to do validation when translating to/from recap CST for proto/avro/JSON schema.

💥 Merged! Thanks!!
Follow-on things I found:

Migrating .json spec to recap-website
Update to pull spec from the recap-website
GH issue for logical type spec tests
(Maybe?) GH issue to do validation when translating to/from recap CST for proto/avro/JSON schema.

https://github.com/recap-build/recap/issues/348

Another thought: this test suite could be extended to read specs defined in YAML or TOML. There ¿might? be some situations where those languages handle values differently like null

Yup, totally. I think it's worth defining in YAML, TOML, and JSON, and validating the CST using the python-jsonschema lib.
One gotcha with TOML, in particular, is that it doesn't have a null AFAIK. Makes null defaults tricky. At least we added optional for the common case, which will work with TOML.

Opened these follow-on issues:
https://github.com/recap-build/recap/issues/349
https://github.com/recap-build/recap/issues/350
https://github.com/recap-build/recap/issues/351


🔌  Plugin: GitHub issues

Goal
Evaluate what’s there to find issues and opportunities for improvement & enhancement
What problem does this solve? Please describe.
A clear and concise description of what the problem is. Ex. I'm always frustrated when [...]
Use cases
A clear and concise description of persona and associated use cases that need to be supported.
Acceptance criteria
A clear and concise list of acceptance criteria, noting what will be delivered as part of this epic.  This will help identify associated testing, documentation and support needs.
Additional context
Add any other context or screenshots about the feature request here.

There is a plugin for github issues https://github.com/backstage/backstage/tree/master/plugins/github-issues
shared by @PatAKnight

Captured results/outcome here - https://docs.google.com/document/d/10J6_FUdmItT-Ko8K7-55RCR2EwbM4Dc_3s921IWLE8o/edit?usp=sharing

Plugin evaluation doc - https://docs.google.com/document/d/1X8zAeMj5MLl0UMA2VQ1yV_hSre5g108-W1PUaD5DkPc/edit?usp=sharing

We did an offline review of the spike, and with the PR getting merged in the upstream we feel the plugin is now complete for our requirements. It is also included in the showcase app already.
I am closing this issue off as complete.


How to start busy when regular http post or simple button click ?

I have tried something like this but without success:
window.onbeforeunload = function () {
    $scope.$broadcast('busy.begin', {remaining: 0});
};

Any clue?
Thanks ! ;-)

Ok... I juste have to use $rootScope instead of $scope ...which is obvious in fact. ʕ•ᴥ•ʔ

Yeah, there isn't currently a way to manually trigger it. I could add an optional binding where it could evaluate the expression to true or false, and toggle busy accordingly.

Yep I think that could be nice ! In some case, I try to let the form be submitted as usual, not via AJAX.
For now I simply use a ng-click that run $rootScope.$broadcast('busy.begin', {remaining: 0});... but it will be very clean with a simple trigger on click or on value changing.


improve workflow docs that use github as example, not code.teracy.org

so that it's easier to grasp the the workflow.

should be compatible with https://github.com/nvie/gitflow and waffle.io for naming conventions

verified


[notsure bug] ace_disposable dependency

Version: lastest develop https://github.com/LordGolias/antistasi/commit/f75228b2978b05a8d7c7898deafe161d92d2798a
Dedicated server without HeadlessClient
Modset: CBA + TFAR + ACE
21:12:38 soldier[B_soldier_AR_F]:Some of magazines weren't stored in soldier Vest or Uniform?
21:12:38 soldier[B_soldier_LAT_F]:Some of magazines weren't stored in soldier Vest or Uniform?
21:12:38 soldier[B_soldier_LAT_F]:Some of magazines weren't stored in soldier Vest or Uniform?
21:12:38 Warning Message: You cannot play/edit this mission; it is dependent on downloadable content that has been deleted.
ace_disposable
I checked the .sqm too and there are no depencencies. however that message display everytime there are certain units spawning.

Shouldn't having ACE handle the ACE_disposable dependency? I.e. do you have ACE without disposable?

probably all irrelevant now because of major patched from ACE and CBA


Feature/add banner for vote tab

TinderGov banner added for Gov tab

We need to fix the style of the banner. Currently see the following issues:

Background looks to dark, on the mockups it is blueish
I don't see it's blueish on the mockups. Do you mean gradient smoothness may be?
"x referendums" corner radius smaller on mockups
The corner radius is wrong, thanks!
there is a border on the mockups but can't see it on the screenshot
there is a border on the screenshot. It's 1p, same as on the mockups. I could do it thicker so it will be more noticable

upd


Time Picker UI breaks when on 26 minutes in Chrome

In newest chrome, it's ok.

hmmm, odd.

maybe you should debug yourself, i do not have this problem...

@ustccjw I was able to reproduce it fairly quickly. I'll try debugging a bit when I have a free moment or two.

Maybe it can be related to Codemirror? Or it's possible to reproduce it from the specs page in the main project?

I think so.. maybe Codemirror is trolling us ;)

I can reproduce it as well, not just for 26, but also 4.

So it seems to be symmetrical.

I have tried without the playground and its working. Maybe codemirror is trolling us ;)
Sent from my iPhone

On 04 Nov 2015, at 04:45, Ryan Feather<EMAIL_ADDRESS>wrote:
I can reproduce it as well, not just for 26, but also 4.
So it seems to be symmetrical.
—
Reply to this email directly or view it on GitHub.

Timer Picker's UI looks different for me

@adaojunior click on the 28 at the top to switch to the minutes selector and you should see the same thing.

@danmartinez101 Yeah, aparently it does not follow the specification https://www.google.com/design/spec/components/pickers.html#pickers-time-pickers

@adaojunior why is not ok with the spec??

@adaojunior just saw your previous comment. You have two flavors for Time Picker component. format="24hr" will render the 24 hours clock. If you want to stick to the spec (although you can find 24hr clock in Android), just change the format prop to ampm.

@javivelasco thanks

I cannot reproduce this error on Windows environment.
Guessing from Safari browser #49 and MarcBalaban picture (MacBook), I guess this glitch is exclusive to Apple software.

@TheDeveloperXYZ I'm able to reproduce a similar glitch on Windows 7 + Chrome.


asynctest fails on Linux under ia64

On Linux/ia64, the 1.1.1-pre9 test suite fails on asynctest:
../../test/recipes/90-test_async.t ....................
1..1
test_ASYNC_init_thread() failed
../../util/shlib_wrap.sh ../../test/asynctest => 1
not ok 1 - running asynctest

#   Failed test 'running asynctest'
#   at /<<PKGBUILDDIR>>/build_static/test/../../util/perl/OpenSSL/Test/Simple.pm line 77.
# Looks like you failed 1 test of 1.
Dubious, test returned 1 (wstat 256, 0x100)

(The full build log is here: https://buildd.debian.org/status/fetch.php?pkg=openssl&arch=ia64&ver=1.1.1~~pre9-1&stamp=1534883907&raw=0 )

I'm not convinced this is an openssl issue. This is most likely an
IA64 specific issue in the toolchain.
This test started breaking between 1.1.0g-2
(https://buildd.debian.org/status/fetch.php?pkg=openssl&arch=ia64&ver=1.1.0g-2&stamp=1515524924&raw=0)
and 1.1.0g-2+b1
(https://buildd.debian.org/status/fetch.php?pkg=openssl&arch=ia64&ver=1.1.0g-2%2Bb1&stamp=1516763329&raw=0)
So the source did not change, but it broke.

Appears to not be an OpenSSL issue so closing this.


add baidu-brpc3 trace plugin

Add an agent plugin to support baidu-brpc-3.x

[x] Add a test case for the new plugin, refer to the doc
[ ] Add a component id in the component-libraries.yml
[ ] Add a logo in the UI repo

[ ] If this pull request closes/resolves/fixes an existing issue, replace the issue number. Closes #.
[ ] Update the CHANGES log.

We require plugin test scenario to cover feature testing for this new plugin.

Are you going to follow our requirements? Any update?

No update for 5 days. Close for now.
If you want to continue this PR, please let's know.


Handle larger files

Part of the Figure out Book content - taking this into a new issue so that we can divide the large issue in case it is needed - and we keep doing issues by feature and not by "epics".

What do you mean with: and we keep doing issues by feature and not by "epics".
That sounds like something based on Azure's Backlog view (:

Well, there's nothing wrong in refering to some other ressource, however, I still don't know what you mean by it?

Just stating that I am taking one of the tasks in the #59 for now, so that I can keep them seperated, is all.

We could store files on disk on the API server 🤷

We can try that, the error seems to happen when the client tries to upload so I think the client needs more but, we can talk about it today.

This link here shows one way to come around it but, for now we want to focus on at least getting the e-books to show and then if there's time; change to store the actual file on the server and then have only the filePath in our SQL tables.


Add Typescript definitions/support

It would be great to add TypeScript definitions/support to Blueprint. Strong types will be a familiar warm blanket for less JS savvy C++ devs.
Being able to define interfaces and the like for component properties will help with avoiding errors at compile time (View would certainly benefit from this addition). It's inherently a little more complex to debug type mismatches and referencing undefined  vars in Blueprint.

💯 love the idea! I think View is a great place to start, and after #44 and #43 go in, the View API should be pretty stable, so sounds like great timing

Marking this as closed after #99. We're now generally moving ahead with typescript support, and as the APIs are stabilizing so too will the type defs. Huge thanks to both @JoshMarler and @rbaxter08– I really appreciate it guys!


StringSpec

Was thinking of adding a new spec, called StringSpec which just requires a single string, eg,
"haveKey should test that a map has a key" {
        val map = mapOf(Pair(1, "a"), Pair(2, "b"))
        map should haveValue("a")
}

Wdyt @helmbold

I like it! It is clean and simple, a piece of beauty.

I'll add it tonight then.

It might become my preferred way of testing actually

It looks awesome

It could become my favourite style, too!

This is in on the 1.3 branch. See changelog in that branch.

Awesome! I'll add the style to the documentation.


Remove unneeded dependencies from Dockerfiles

The dependencies in question were removed from the remote repos, and are no longer required for Chromium to build.
Removing them outright is safe to do, and does not affect any development environment.
Associated issue: https://github.com/alphagov/govuk-docker/issues/508

@kevindew This doesn't have to be pulled in, but as far as I can tell through looking, Chromium just doesn't need these anymore, nor does it affect our environment.

Just raising that I've not forgotten about this, I am snowed under with some pretty hefty work and as this isn't affecting most people I've put it to the side just for now.


https://freshandfit.locals.com/

DO NOT REMOVE OR SKIP THE ISSUE TEMPLATE

[X] I understand that I will be blocked if I intentionally remove or skip any mandatory* field

Checklist

[X] I'm reporting a new site support request
[X] I've verified that I'm running yt-dlp version 2023.07.06 (update instructions) or later (specify commit)
[X] I've checked that all provided URLs are playable in a browser with the same IP and same login details
[X] I've checked that none of provided URLs violate any copyrights or contain any DRM to the best of my knowledge
[X] I've searched known issues and the bugtracker for similar issues including closed ones. DO NOT post duplicates
[X] I've read the guidelines for opening an issue
[X] I've read about sharing account credentials and am willing to share it if required

Region
No response
Example URLs
-Single Video https://freshandfit.locals.com/post/4280045/stream-with-the-ladies
Provide a description that is worded well enough to be understood
Trying to download this video. It is opened only after registration. The issue is it is chunked into .ts files and the only m3u8 files I been able to get only produce short chunk.
How can I actually download whole video?
Provide verbose output that clearly demonstrates the problem

[X] Run your yt-dlp command with -vU flag added (yt-dlp -vU <your command line>)
[X] If using API, add 'verbose': True to YoutubeDL params instead
[X] Copy the WHOLE output (starting with [debug] Command-line config) and insert it below

Complete Verbose Output
yt-dlp -vU https://freshandfit.locals.com/api/v1/posts/content/hls/v_be4c7342-4f2d-49f2-a47b-dc94b8c5712f/source.m3u8
[debug] Command-line config: ['-vU', 'https://freshandfit.locals.com/api/v1/posts/content/hls/v_be4c7342-4f2d-49f2-a47b-dc94b8c5712f/source.m3u8']
[debug] Encodings: locale cp1252, fs utf-8, pref cp1252, out utf-8, error utf-8, screen utf-8
[debug] yt-dlp version<EMAIL_ADDRESS>[2af4eeb77] (win_exe)
[debug] Python 3.8.10 (CPython AMD64 64bit) - Windows-10-10.0.19045-SP0 (OpenSSL 1.1.1k  25 Mar 2021)
[debug] exe versions: ffmpeg N-90610-g2accdd3871, ffprobe N-90610-g2accdd3871
[debug] Optional libraries: Cryptodome-3.18.0, brotli-1.0.9, certifi-2023.05.07, mutagen-1.46.0, sqlite3-2.6.0, websockets-11.0.3
[debug] Proxy map: {}
[debug] Loaded 1856 extractors
[debug] Fetching release info: https://api.github.com/repos/yt-dlp/yt-dlp-nightly-builds/releases/latest
Available version<EMAIL_ADDRESS>Current version<EMAIL_ADDRESS>Current Build Hash: fd6e55986ae0904f69c03c9e31b85539908c6c9ba1603400eee5124f5c3d30a2
yt-dlp is up to date<EMAIL_ADDRESS>[generic] Extracting URL: https://freshandfit.locals.com/api/v1/posts/content/hls/v_be4c7342-4f2d-49f2-a47b-dc94b8c5712f/source.m3u8
[generic] source: Downloading webpage
[debug] Identified a direct video link
[generic] source: Downloading m3u8 information
[generic] source: Checking m3u8 live status
[debug] Formats sorted by: hasvid, ie_pref, lang, quality, res, fps, hdr:12(7), vcodec:vp9.2(10), channels, acodec, size, br, asr, proto, vext, aext, hasaud, source, id
[debug] Default format spec: bestvideo*+bestaudio/best
[info] source: Downloading 1 format(s): 4000
[debug] Invoking hlsnative downloader on "https://freshandfit.locals.com/api/v1/posts/content/hls/playlist/v_be4c7342-4f2d-49f2-a47b-dc94b8c5712f/1080.m3u8"
[hlsnative] Downloading m3u8 manifest
[hlsnative] Total fragments: 1
[download] Destination: source [source].mp4
[debug] File locking is not supported. Proceeding without locking
[download] 100% of    8.70MiB in 00:00:00 at 8.73MiB/s
[debug] ffprobe command line: ffprobe -hide_banner -show_format -show_streams -print_format json "file:source [source].mp4"
[debug] ffmpeg command line: ffprobe -show_streams "file:source [source].mp4"
[FixupM3u8] Fixing MPEG-TS in MP4 container of "source [source].mp4"
[debug] ffmpeg command line: ffmpeg -y -loglevel "repeat+info" -i "file:source [source].mp4" -map 0 -dn -ignore_unknown -c copy -f mp4 "-bsf:a" aac_adtstoasc -movflags "+faststart" "file:source [source].temp.mp4"

Duplicate of #1827


ButterKnife breaks CoordinatorLayout anchoring

I'm unsure if this is a bug in ButterKnife, or the support library, but when ButterKnife is used in conjunction with a CoordinatorLayout inside a RecyclerView, the call to ButterKnife.bind() causes the CoordinatorLayout anchoring to break.
Uncommenting the call to bind() results in everything working as expected.
I've filed a bug report with Google also, where you can find a sample project:
https://code.google.com/p/android/issues/detail?id=222802&thanks=222802&ts=1474246332

This is not caused by Butter Knife since its generated code only calls
findViewById which seems to be side-effecting in this case.
On Sun, Sep 18, 2016, 8:55 PM Josh Burton<EMAIL_ADDRESS>wrote:

I'm unsure if this is a bug in ButterKnife, or the support library, but
when ButterKnife is used in conjunction with a CoordinatorLayout inside a
RecyclerView, the call to ButterKnife.bind() causes the CoordinatorLayout
anchoring to break.
Uncommenting the call to bind() results in everything working as expected.
I've filed a bug report with Google also, where you can find a sample
project:
https://code.google.com/p/android/issues/detail?id=222802&thanks=222802&ts=1474246332
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/JakeWharton/butterknife/issues/747, or mute the
thread
https://github.com/notifications/unsubscribe-auth/AAEEEaF3R_ntSyMamFsmvT6P6LG0NjjNks5qrd1zgaJpZM4KAEZi
.


Plugin will hang the whole server while missing permission

When the plugin does not have SERVER MEMBERS INTENT on discord itself enabled, it will hang the server saying:
[12:31:48] [OkHttp https://discordapp.com/.../WARN]: [uk.co.angrybee.joe.dependencies.com.fasterxml.jackson.databind.ext.Java7Support] Unable to load JDK7 types (annotations, java.nio.file.Path): no Java7 support added

If you do enable SERVER MEMBERS INTENT this problem is solved, and the plugin will load correctly.
Maybe add a message saying that that needs to be enabled? to prevent confusion to others. DiscordSRV does give this alert:
[00:00:42 ERROR]: [DiscordSRV] DiscordSRV and its API hooks require these intents to function. Instructions:
[00:00:42 ERROR]: [DiscordSRV] 1. Go to https://discord.com/developers/applications
[00:00:42 ERROR]: [DiscordSRV] 2. Click on the DiscordSRV bot
[00:00:42 ERROR]: [DiscordSRV] 3. Click on "Bot" on the left
[00:00:42 ERROR]: [DiscordSRV] 4. Enable the "SERVER MEMBERS INTENT"
[00:00:42 ERROR]: [DiscordSRV] 5. Restart your server

Thanks for the info. I'll get on making an update for this.

This 9435d8610673de0d45f8c3850f9677284f92d51d should fix the hang and warn users until I come up with a more elegant solution.


Fix deprication warnings to show correct new function names.

Coverage remained the same at ?% when pulling 2e4c6f2bb935b7a03256fda06e02980cd04e4151 on jviereck:patch-2 into 07bb13b1e3a89102fce49a2ff43a447c100c3868 on stack-of-tasks:master.

@jviereck Thanks for filling this mistakes.
Next time, can you process in the same way but just putting the devel branch as merging branch. It is just because master is for us the latest release.
We will make a new release with bug fixes in the very next days.
Thanks again for your contribution. And do not hesitate to ask questions through issues to obtain rapid answers.

@jcarpent, thx for the merge (going to request against devel next time) and the open door to ask questions :)!!!
On September 21, 2018 3:52:32 AM EDT, Justin Carpentier<EMAIL_ADDRESS>wrote:

Merged #524 into devel.
--
You are receiving this because you were mentioned.
Reply to this email directly or view it on GitHub:
https://github.com/stack-of-tasks/pinocchio/pull/524#event-1859511889


Presence shown as orange dot but "Active now"

In the above image, activity text is "Active Now", but indicator color is not green. It should be green.
@zulipbot claim

I suspect the color indicator is the incorrect one 😄

We might want to get rid of the intermediate yellow state altogether and be either no indicator or a green dot.

Thanks @jainkuniya for flagging this!
In the webapp, if I

go through the people in the right sidebar with orange "idle" half-dots,
click on one of the chevrons to get a user's profile,
and hover on the orange half-dot within the profile,
the hover info consistently says "Just now".

I suspect the behavior reported here is what we're doing in the exact same situation. In the presence system's terms:
https://zulip.readthedocs.io/en/latest/subsystems/presence.html
I think what's happening is that the user has a client which is online now... but they haven't actually interacted with the client recently.
I'd say the combination of an orange "idle/inactive" indicator (AFAICT we don't have any words within the app to explain what it means) with the text "Just now" is pretty confusing in itself, and probably isn't a fully-considered product choice.
So, we should figure out how we want this system to behave as a product matter, and describe that clearly. Then the mobile app and webapp can both implement the behavior we decide we want.
@rishig , do you have a sense of how you'd like this area to work? /cc also @timabbott who may have more to add on how the underlying system is working. It looks like zulip/zulip#11607 (an ongoing open issue) is related, too.

yeah, I think https://github.com/zulip/zulip/issues/11607 has my updated thoughts. Agreed that "last seen" should have something like "N minutes ago" if they are not green; not sure if the current webapp behavior is a bug or a previous product decision.

@rishig can you collaborate with @vinitS101 on getting the design for https://github.com/zulip/zulip/pull/11882 (the PR for https://github.com/zulip/zulip/issues/11607) to something you're happy with?  I think that's probably what's blocked there.
The fundamental problem here is that "orange" is a weird state to explain, and we don't explain it.  The person has a connected client that's online now, but we don't know if they're at their computer.  We have plans involving the desktop app to know for sure whether the user is at their computer (and thus be able to turn a lot of orange -> green), but we'll likely never be able to do that in the webapp.  Maybe "Online, last active 12 minutes ago" might be a clean way to summarize orange states.


ready for 0.14.0

Codecov Report

Merging #150 (6573ee6) into master (46323eb) will decrease coverage by 2.27%.
The diff coverage is 100.00%.

@@             Coverage Diff             @@
##            master     #150      +/-   ##
===========================================
- Coverage   100.00%   97.72%   -2.28%
===========================================
  Files           22       22
  Lines         2204     2198       -6
===========================================
- Hits          2204     2148      -56
- Misses           0       50      +50

Impacted Files
Coverage Δ

src/viztracer/code_monkey.py
98.51% <ø> (-1.49%)
:arrow_down:

src/viztracer/event_base.py
100.00% <ø> (ø)

src/viztracer/vizlogging.py
100.00% <ø> (ø)

src/viztracer/viztracer.py
94.28% <ø> (-5.72%)
:arrow_down:

src/viztracer/__init__.py
100.00% <100.00%> (ø)

src/viztracer/decorator.py
100.00% <100.00%> (ø)

src/viztracer/main.py
97.32% <100.00%> (-2.68%)
:arrow_down:

src/viztracer/patch.py
80.45% <100.00%> (-19.55%)
:arrow_down:

src/viztracer/prog_snapshot.py
98.88% <100.00%> (-1.12%)
:arrow_down:

src/viztracer/report_builder.py
94.11% <100.00%> (-5.89%)
:arrow_down:

... and 10 more

Continue to review full report at Codecov.

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


Faceted search setting page

#1337
Checklist

[x] the [contributor license agreement] is signed
[x] commit message follows [commit guidelines]
[ ] tests are included
[ ] screenshots are included showing significant UI changes
[ ] documentation is changed or added

Description of change
This PR adds a new page that displays a list of Faceted search classifications and a dialog that is used to add a classification. Because back-end work is not completed yet, this PR only focuses on UI and does not have communications with the Server.
Once this page is fully implemented, tests and stories will be added.

Should we convert this to a draft PR for now then?


Rigidbody several regressions in godot 4.0

Godot version
4.0 alpha 3
System information
Linux mint
Issue description
Under certain circunstances, the body teleports upwards. Mostly noticiable when the body stop  making contact with the floor and the wall. Is hard to replicate, but i made a project that i think it shows it well
https://user-images.githubusercontent.com/64601424/160404052-6401b2f7-38a5-4aff-a92a-3828119b7c7b.mp4
.
Steps to reproduce
Create a platform with a collisionbody (Static, kinematic or Rigidbody dont alter the result) and a wall.
move a sphere towards the corner.
Minimal reproduction project
BugPhysics.zip

4.0 alpha 3

Please test this on 4.0alpha5. In the future, do not report issues on older alphas unless you've confirmed them on more recent alphas first.

4.0 alpha 3

Please test this on 4.0alpha5. In the future, do not report issues on older alphas unless you've confirmed them on the latest alpha first. (If you do reproduce the issue in both alphas, mention it in the issue text.)

Sorry, i didn't knew there was a new alpha. I will check and tell you if is reproducible in the new one.

I can confirm that this bug is still reproducible in Godot alpha 5

Can confirm that this is still reproducible in the last Godot 4.0 alpha 9. Maybe it should be marked as a milestone cause right now the jumps are measurable in meters.

Can confirm that this is still reproducible in the last Godot 4.0
alpha 9. Maybe it should be marked as a milestone cause right now
the jumps are measurable in meters.

This issue already has the 4.0 milestone 🙂

Ouch. Sorry, still learning this things u.u

Thanks for the clear minimal reproduction project. The bug in the MRP seems to be specific to a capsule shape with height == 2.0 * radius, so that it degenerates to a sphere, and GodotCapsuleShape3D::get_supports does not handle this case well, returning a segment of length zero instead of a point. I'll submit a PR to fix that (hence fixing the bug in the MRP). Can you reproduce this in any other cases?

Thanks for the clear minimal reproduction project. The bug in the MRP seems to be specific to a capsule shape with height == 2.0 * radius, so that it degenerates to a sphere, and GodotCapsuleShape3D::get_supports does not handle this case well, returning a segment of length zero instead of a point. I'll submit a PR to fix that (hence fixing the bug in the MRP). Can you reproduce this in any other cases?

Hi Rburing. Thanks for looking on it. I will try with other shapes and intersection angles in the last beta and post the results here if i notice it under other circumstances.


installing gpcrio - calculate_macosx_platform_tag doesn't parse 'base_version' correctly

What version of gRPC and what language are you using?
1.51.1
What operating system (Linux, Windows,...) and version?
Mac 12.6
What runtime / compiler are you using (e.g. python version or version of gcc)
Python 3.8.15
What did you do?
pip install grpcio
What did you expect to see?
should install correctly
What did you see instead?
File "lib/python3.8/site-packages/wheel/macosx_libfile.py", line 360, in calculate_macosx_platform_tag
  assert len(base_version) == 2
  AssertionError
  ----------------------------------------
  ERROR: Failed building wheel for grpcio

This is happening because distutils.utils.get_platform is returning:
macosx-12-x86_64 instead of macosx-12.6-x86_64 and the parsing code expects to see an 12.6 for the version and not 12, so the assert fails
Anything else we should know about your project / environment?

Hi, can you provide more logs for this error?

The problem is, distutils is reporting my mac's version as macosx-12-x86_64, the code in calculate_macosx_platform_tag splits this string on the '-'' and grabs the middle segment '12', and then splits it again on '.' because it's expecting something in the format of X.X, but in this case, there is nothing to split.  There is an assert check that expects -exactly- 2 values here (a major and a minor version number), and it fails, so the install breaks.
My ugly workaround was just to hardcode it to 12.6 to get the install to pass.

Are you building grpc from source with an M1 Mac? If so this PR (https://github.com/grpc/grpc/pull/31747) will be helpful.
And also distutils is vendored by setuptools, can you try again with newer version of setuptools?


Upgrade to next version of MqttNet

Describe the feature request
This library is no longer compatible with Mqttnet.AspnetCore due to drastic breaking changes.
Describe the solution you'd like
Upgrade to latest version of Mqttnet.AspnetCore and .Net 6.

Commenting to keep issue open as there's an open PR

Commenting to keep issue open as there's an open PR

Commenting to keep issue open as there's an open PR


[BUG] Fix flaky espionage dispatch mission test

Describe the bug
This test sometimes fails when running jobs:
  FAILED  Tests\Feature\FleetDispatch\FleetDispatchEspionageTest > dispatch…
  Target planet was not updated after espionage mission has arrived. Check target planet update logic on mission arrival.
Failed asserting that<PHONE_NUMBER> is less than<PHONE_NUMBER>.

  at tests/Feature/FleetDispatch/FleetDispatchEspionageTest.php:191

For example, see this GitHub Action run:
https://github.com/lanedirt/OGameX/actions/runs/11373489520/job/31640153417
To Reproduce
Steps to reproduce the behavior:

See GitHub action result: https://github.com/lanedirt/OGameX/actions/runs/11373489520/job/31640153417
Most likely if you run the tests locally many times it will also sometimes fail. However I have not tested or confirmed this yet. Verify this by executing php artisan test in the ogame-app docker container.

Expected behavior
Test to always succeed, especially if nothing regarding espionage missions has been changed. The problem in this case can be caused by underlying game logic that is actually faulty, or it can be caused by something else that makes it flaky (i.e. fail sometimes while it shouldn't).

This issue can be reproduced locally (not tested in Docker), by running the fleet dispatch espionage tests in loop:
while php vendor/bin/phpunit --filter FleetDispatchEspionageTest --stop-on-failure; do echo; done
1) Tests\Feature\FleetDispatch\FleetDispatchEspionageTest::testDispatchFleetDebrisEspionageReport
Target planet was not updated after espionage mission has arrived. Check target planet update logic on mission arrival.
Failed asserting that<PHONE_NUMBER> is less than<PHONE_NUMBER>.

tests/Feature/FleetDispatch/FleetDispatchEspionageTest.php:191

There is also another flaky case with the same testDispatchFleetDebrisEspionageReport that pops up when running only that test in the loop:
while php vendor/bin/phpunit --filter testDispatchFleetDebrisEspionageReport --stop-on-failure; do echo; done
1) Tests\Feature\FleetDispatch\FleetDispatchEspionageTest::testDispatchFleetDebrisEspionageReport
Debris field metal resources are not visible in espionage report.
Failed asserting that 2674 matches expected 1337.

tests/Feature/FleetDispatch/FleetDispatchEspionageTest.php:198

Hi @rautamik,
Thank you for testing this and for sharing the command to reproduce the error at will, that is very helpful! 👍

Re-opening this issue: the flaky tests still occur in new PR's.
I'm working on refactoring how the tests interact with time as certain tests are setting time to static values (2024-01-01) and other tests do not and therefore use the current time which causes conflicts. If all tests use the same starting point this problem should go away once and for all.

Fixed by #433


Annotations missing for inline components when annotate_rendered_view_with_filenames is enabled

I am using view_component in a Rails application with config.action_view.annotate_rendered_view_with_filenames set to true. For view components with templates, the HTML is annotated with the usual BEGIN and END. However for inline components (so relying on call and not having a template), the HTML doesn't contain those annotations.
Steps to reproduce

Set config.action_view.annotate_rendered_view_with_filenames set to true in your Rails config.
Render a view component with a template.
Render an inline view component.
See the BEGIN and END comments for the view component with a template, but not for the inline view component.

Example in which the <abbr> tag is an inline view component and another component with a template which renders Hello!:

Expected behavior
I expect the annotations to be added for inline view components.
Actual behavior
The annotations are not added for inline view components.
System configuration
Rails version: Rails <IP_ADDRESS>
Ruby version: ruby 3.1.1p18 (2022-02-18 revision 53f5fc4236) [x86_64-linux]
Gem version: 2.49.1

@dmarcoux thanks for filing this issue! The current implementation of template annotations hooks into the template compilation process, which isn't used for inline components.
I'd love to see this feature added to the framework, would you like to take it on? I'd be happy to pair with you to get it off the ground<EMAIL_ADDRESS>

would love to have this - would this be done via output_postamble and output_preamble?

@joevandyk yes, I think we'd do it in that area of the codebase! Happy to pair with you if you'd like, email me with your availability if you'd like<EMAIL_ADDRESS>


[SCB-260] Add third party notices

Follow this checklist to help us incorporate your contribution quickly and easily:

[x] Make sure there is a JIRA issue filed for the change (usually before you start working on it).  Trivial changes like typos do not require a JIRA issue.  Your pull request should address just this issue, without pulling in other changes.
[x] Each commit in the pull request should have a meaningful subject line and body.
[x] Format the pull request title like [SCB-XXX] Fixes bug in ApproximateQuantiles, where you replace SCB-XXX with the appropriate JIRA issue.
[x] Write a pull request description that is detailed enough to understand what the pull request does, how, and why.
[x] Run mvn clean install to make sure basic checks pass. A more thorough check will be performed on your pull request automatically.
[ ] If this contribution is large, please file an Apache Individual Contributor License Agreement.

Coverage remained the same at 95.285% when pulling 70e2d4e80e7354da95c16d7ed775351d3581a258 on yangbor:master into 541202272bdd5f5b76daa11c7b6cf1c1a7d9a505 on apache:master.


Use message metadata to determine whether issue_comment is pull request

Avoids a lot of unneeded API calls that will return 404

@DavidTanner PTAL.  Github informed me this was a major source of API calls in our systems.

Once it has been determined that it isn't a comment on a pr the function should return instead of continuing.  If it is a comment on a pr then it shouldn't continue checking to see if it is a pr

Thank you for this pull request! Please check this document for how the Jenkins project handles pull requests.

@DavidTanner Good point.  Fixed and moved.


fix: reload language server client on restart

Ensure the client watcher and output channel are disposed during a restart before the new client is creates, and also show a helper warning when certain variables are changed that require a restart.
Also catch errors starting the client, so that our restart configuration change handler is still registered in case the user fixes their serverPath or something like that. Similarly, always register the restart command in case user wants to start using language server after
not using it before.
Closes #419

Thanks @ian-h-chamberlain !


⚠ Fortnite has degraded performance

In 378378b, Fortnite (https://fortnite.com) experienced degraded performance:

HTTP code: 200
Response time: 1509 ms

Resolved: Fortnite performance has improved in 1c468b3.


Inverted scrollsToTop scrolls to bottom

Environment
Environment:
  OS: macOS High Sierra 10.13.5
  Node: 10.4.1
  Yarn: 1.7.0
  npm: 6.4.1
  Watchman: 4.9.0
  Xcode: Xcode 9.4.1 Build version 9F2000
  Android Studio: Not Found

Packages: (wanted => installed)
  react: ^16.4.2 => 16.4.2
  react-native: https://github.com/expo/react-native/archive/sdk-29.0.0.tar.gz => 0.55.4

Description
When a FlatList has both scrollsToTop and inverted enabled, tapping on the iOS status bar scrolls to the bottom of the list, which isn't a behavior I've seen in any native apps. It would be great if this was either changed or it was possible to do e.g. scrollsToBottom={true} or scrollsToTopDirection={'inverted'}.
Reproducible Demo
Expo Snack

This has been an annoying UX issue since I started using React Native ~10 months ago, would be a really great fix for being able to quickly scroll to the top of a chat screen or other inverted screens!

It seems to me that you are using Expo, so I suggest you open a dedicated issue there 🤷♂
Our can you reproduce on a new react-native init project with 0.57.2?

Tested with the same code from the snack on a new react-native init (0.57.4) and still have the issue that scrollsToTop scrolls to the bottom

Have you found any workaround while it's getting fixed?

This PR should fix it: https://github.com/facebook/react-native/pull/27574


got error message after new clone and new build

The Statistics computation is currently being reworked in the following issue: https://github.com/QuantConnect/Lean/issues/30. Closing this issue as it's a known problem and should be OBE with the new implementation. Thanks for submitting!
Also, this can be safely ignored. What's it's saying is that there was an extra data point in the benchmark array vs the performance array.


Desideratum: download XML mini-corpus from search results

Another suggestion was that when a user has serched for a term or browsed through various facets (or both) to find a subset of inscriptions in the search interface, they be given the option to download a ZIP file containing the XML files of just those inscriptions that appear in the search results list. This would enable them to create a mini corpus of just 1st century religious texts, or just texts on marble from Rome, or just pre-5th century texts mentioning Agathe Tyche, or whatever.

This would be a subtask of the general "download button for zip file of all EpiDoc files in corpus" task, which doesn't seem to be a ticket. @ajenhl shall I make a ticket for that, or is it already known about and in hand? (If not, I guess it's higher priority than this one, which may just move to "Future Work"…?

No need for such a ticket; I've just committed the general download link to the inscriptions index.

As a note about this particular issue, it would require changing the way search results are handled. At the moment, the pagination is handled by Solr, which returns a set number of results at a time. In order to have a download of all documents returned by a search, it would need to return all rows at once and the pagination then work on that subset. There's obviously a performance penalty there, though how much of one depends on a couple of factors (how many results there are in total, how much information is returned for each document).
Just something to bear in mind.

Is there a setting for how many results are returned? E.g. (20|50|100|all) with a speed caveat on all!
If so we could just say download current results list, and leave it to the user to decide if they need a longer list..?

Solr doesn't have an "all results" option. Values are set in assets/queries/solr/facet_query.xml (for the main search), where users are also directed to go when adding/removing facets, etc.
It occurs to me that the link to download a ZIP file of all the works referenced in the search results can perform its own query, with the rows parameter set to a very high value, and so nothing about the structure of the search and pagination be changed. That will be easy to implement.

@ajenhl this was never implemented for EFES 1.0, was it? Let's reassign to 2.0, unless you think it is trivial enough that you'd do it in your spare time in the interim? ;-)


How to use .env in urls

How should I call the .env MODEL_PREDICTION_URL inside of the service? Currently, I am calling the service as below:
url = "http://<IP_ADDRESS>:8020/v1/predict_image_label" in the https://github.com/Atforestry/fetch-data/blob/main/src/app/utils.py file

@borjagomez  I tried to use http://model-predict:8020/v1/predict_image_label and https://model-predict:8020/v1/predict_image_label  but it did not work. Getting the below error
raise ConnectionError(e, request=request)
requests.exceptions.ConnectionError: HTTPConnectionPool(host='model-predict', port=8020): Max retries exceeded with url: /v1/predict_image_label (Caused by NewConnectionError('<urllib3.connection.HTTPConnection object at 0x7f5c54ce18b0>: Failed to establish a new connection: [Errno 111] Connection refused'))
Not sure if we need to create an alias on the compose files

try to remove blank spaces in the .env file and try again. This is an error that I detected in my repository and I fixed in mine but didn’t fix it in the rest.
also try os.environ[‘KEY’] instead of .get. Maybe both work, but it worked for me.
Try to log.info the env variables to make sure you get them.


[Feature] print instruction for using jiq when no stdin is piped in

What if when one launches jiq without receiving anything from stdin, it would print an instruction that it expects a json structure, and finally Ctrl+D to terminate the json string?
$ jiq
Enter a json formatted string, press Enter and Ctrl+D to continue:
[1,2,3]
^D

[Filter]> .
[
  1,
  2,
  3
]

What say you?

Yes, that would be good. I am confused sometimes by the lack of response when jiq is launched without anything on stdin.


stackdriver: fix Cargo features for GcpAuthorizer

Changes
#50 regressed the support for actually using the GcpAuthorizer: it used the dep:gcp_auth syntax for enabling the dependency by default, but the actual GcpAuthorizer code was guarded by a gcp_auth feature which didn't exist anymore. Bring back a named feature for this (now called gcp-authorizer to match yup-authorizer).
This time I will start a PR for my work repository based on my branch to make sure it actually works.
Merge requirement checklist

[x] CONTRIBUTING guidelines followed
[x] Unit tests added/updated (if applicable)
[x] Appropriate CHANGELOG.md files updated for non-trivial, user-facing changes
[x] Changes in public API reviewed (if applicable)

Merging this. @djc I think you are listed as owner of https://crates.io/crates/opentelemetry-stackdriver in crates.io. Do you want to publish a new version?

Yeah, will do!

Published 0.19.1 on crates.io.

Interesting. I'd thought clippy would caught this

I think this is a weak spot in the toolchain, I think work is ongoing to fix it though.


try to fix broken links

see #298
also try to tidy up a reference to 10.2.0 - which was an Akka HTTP release

LGTM, nice. Through this PR, I understand how @apidoc works


Update respond-file-alerts.md

The feature is not in preview anymore and is already live
https://techcommunity.microsoft.com/t5/microsoft-defender-for-endpoint/download-quarantined-files-now-generally-available/ba-p/2581160

@JohanFreelancer9 Please copy review

Issue#https://github.com/MicrosoftDocs/microsoft-365-docs/issues/8562

@mjcaparas Please merge this PR. Thanks


Major Overhaul. New Version

This PR contains a major refactor of the library. It now supports multiple tokens on a paper wallet. It also has e2e tests for all use cases.

:tada: This PR is included in version 1.4.0 :tada:
The release is available on npm package (@latest dist-tag)
Your semantic-release bot :package::rocket:


Refactor

I'm sorry. I mistook your repository for my repository. I make pull request to my own repository.

No prob


[Bug]: Making links under "get all types of courses" clickable

What bug you encountered?
the links under the section "get all types of courses " do not direct to the required page. Therefore adding the paths to respective links and making necessary pages for them
Show us the trouble with screenshots

Which browsers are affected?
Chrome, Microsoft Edge
Agreement to Code of Conduct

[X] Code of Conduct

Hello @Illaf , thank you for your suggestion. Yes this should be implemented in our project, but for now we do not have those pages so this will take the user to a blank screen which we do not want. So we will look on this issue later.


Docker Hub / Full Container Support

Describe the problem
I would like to have listd as a container as such to install it with docker hub, not having to worry about updates.
Problems that need to be solved:

it seems that docker-compose.yml currently implements a volume for the database with local pathing, which won't work when both are containers.
listd with container support

Describe the proposed solution
I would like to see a solution where both the database and listd can be used with docker.
Have you checked if this issue has already been raised?

[X] I did not find any similar issues

Code of Conduct

[X] I agree to follow this project's Code of Conduct

found another problem that needs to be solved:

it seems that listd currently uses GOOGLE_CLIENT_ID and GOOGLE_CLIENT_SECRET, which are dynamic variables that need to be passed to the container

Hi @timondev, a new comer is here 👋.
Could you assign this to me !?

@kareemmahlees Feel free to work on this. I'll assign to you. Do note that I want to keep local development documentation in the contributing guides. We can also add another section that describes how to run the container locally if they want to.
Also, once we have a container, we can publish the image for others to use easily.


Clarification about Group users limit

[Enter feedback here]
Hello
I found in the documentation the upper limit of how many users can be part of group.
Implicitly it passes the message 'there is no limit'
But according to community post :
https://techcommunity.microsoft.com/t5/microsoft-365-groups/office-group-membership-limitation/m-p/170207
There is undocumented limit 2500 users per group.
Can you please document it?
It's relevant information for large scale integration my team is working on.
Thank you,
Tomasz
Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: 809bcea7-ebd6-58bc-9ff0-e764a459016d
Version Independent ID: f86ecf79-7095-772a-980e-356a995061c5
Content: Overview of Microsoft 365 Groups for administrators - Microsoft 365 admin
Content Source: microsoft-365/admin/create-groups/office-365-groups.md
Service: o365-administration
GitHub Login: @MikePlumleyMSFT
Microsoft Alias: mikeplum

@tomek-wday Thank you for your feedback. In fact, the number of users per group as per Teams limitation is 25000. I'll confirm with the author and make changes accordingly.
Thanks
Sri

@siunies Please help us in resolving this issue. Thanks

@tomek-wday I just got the confirmation that we haven’t documented it as a single limit for Groups because the limits apply differently based on the experience i.e. the limit is different when using a Group in Yammer or Teams or SharePoint or Outlook/Email.
For Teams -https://docs.microsoft.com/en-us/microsoftteams/limits-specifications-teams
@yogkumgit  I have checked with the author and there are no changes required in the document.
Hope this helps!
Thanks
Sri

@msbemba Thank you very much for the contribution and sharing this explanation. @tomek-wday Hope this comment is helpful for you. If you see a documentation update is required, please feel free to open an issue for the same. We proceed here to close it. Thanks for taking out some time to open the issue. Appreciate and encourage you to do the same in future also.


GPUParticles3D amount parameter emits wrong number of particles when using scale curve

Godot version
4.0beta4
System information
PopOS 22.04, 6.0.3-76060003-generic kernel, Ryzen 6800h, Radeon 6700m, amdgpu, Vulkan
Issue description
When setting the GPUParticles amount to 1, 3, 6, and maybe other, the particles emitted are the same as setting amount to 1 if a scale curve is applied
Steps to reproduce
Setup a GPUParticles3D with a ParticalProcessMaterial and change the amount with a scale curve applied to the ParticalProcessMaterial:
https://user-images.githubusercontent.com/15805689/202588137-ebdeb652-4cf4-4900-a585-7ef0acee6d92.mp4
https://user-images.githubusercontent.com/15805689/202586281-4a848458-54a2-4e93-a68f-85016cf6a2be.mp4
Minimal reproduction project
https://github.com/Mikeysax/example-project-godot-bug-gpu-particle-amount-bug-

@Mikeysax Please upload a minimal reproduction project to make this easier to troubleshoot.

https://github.com/Mikeysax/example-project-godot-bug-gpu-particle-amount-bug-

I can't reproduce this issue on PopOS 22.04 I attempted with beta 4 and beta 9

I can confirm this still happens in RC1 in one of my personal projects, one of my particle systems almost stops emitting altogether after a few seconds when a scale curve is used. As soon as I take away the curve it starts working correctly again.
https://user-images.githubusercontent.com/781121/219049175-0992e36e-41ed-47d9-8e71-83ca7201c50f.mp4
As can be seen in the video, it doesn't seem to stop emitting entirely, for whatever reason it's just way less then it should be with a scale curve applied.

This can be worked around by ensuring the beginning and end points of the curve are not exactly 0.0 (e.g. 0.01) works. It looks mostly the same, but the particle amount will not decrease over time.

I wonder if this is the same underlying problem as https://github.com/godotengine/godot/issues/55396. If it is, https://github.com/godotengine/godot/pull/55397 may help

@clayjohn I believe there's NaN' ing issues in those scales. I think it's possible that sometimes scales goes close to zero, then float point precision, then it's actually zeros, and there's a bunch of NaN going being fed every frame into the transform. In my own project i set the scale to never be exactly zero to work around this issue.

I also suspect that scale curves have repeat on when they're sampled, causing scale to wrap around

I also suspect that scale curves have repeat on when they're sampled, causing scale to wrap around

Repeat is already disabled on the scale texture: https://github.com/Calinou/godot/blob/91c0b62451ecff469d4857c7bb7977d3f15f7fd4/scene/resources/particle_process_material.cpp#L282


kubernetes-helmPlugins.helm-secrets: 3.8.3 -> 4.2.2

Description of changes
https://github.com/jkroepke/helm-secrets/releases
Things done

Built on platform(s)

[ ] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[x] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[x] Tested basic functionality of all binary files (usually in ./result/bin/)
23.05 Release Notes (or backporting 22.11 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[ ] Fits CONTRIBUTING.md.

This pull request has been mentioned on NixOS Discourse. There might be relevant details there:
https://discourse.nixos.org/t/prs-ready-for-review/3032/1837

The hooks: in plugin.yaml was remove in https://github.com/jkroepke/helm-secrets/commit/debdc015094734597b626d2ad8cbee273d989e1e since v3.6.0, no need to remove them now.


Searching for $/ in doc.perl6.org redirects to non existing page

When searching for $/ in the searchbox, the javascript suggests "Variable: $/". That points to http://doc.perl6.org/syntax/%24%2F (not existing) instead of http://doc.perl6.org/language/variables#The_%24%2F_Variable.
Other variables, e.g. $$, do not have this problem.

It looks like a duplicate of duplicate of #155.

Indeed it is. Closing. Thanks for reporting anyway.


b0 sincos shim was incorrect, causing crash

on B0 the sincos shim was written as a regular sincos method (i.e. expecting r2, r3 to be pointers for results) rather than returning values in r0-r3 as it should. This happened to be mostly harmless in the past, but got broken when fixes for out of range inputs were added (which meant shim was called without valid r2-r3)

fixes #172


Memory leaks when running the TPC-H benchmark repeatedly

Describe the bug
I've built Datafusion Comet using commit https://github.com/apache/datafusion-comet/commit/f7f0bb1ed68367b8d3e1c88010c1f943f480ea11 for Spark 3.5.1. I found that the memory usage keeps increasing when repeatedly running the TPC-H benchmark script on a set of parquet files. The parquet files were generated using https://github.com/databricks/spark-sql-perf with scale factor = 10. The memory usage could be as high as 20GB. Given the spark and comet configurations I'm using to run the benchmarks (see Additional context) this seems to be problematic.

I've noticed that the native memory allocated by Unsafe_AllocateMemory0 keeps increasing using jcmd VM.native_memory detail.diff | grep Unsafe -A 2. I'm not enabling offheap memory so the allocation should be initiated by the arrow RootAllocator:
Initially after setting the baseline:
[0x00000001099c98a8] Unsafe_AllocateMemory0(JNIEnv_*, _jobject*, long)+0xcc
[0x000000011a0523b4]
                             (malloc=870721KB type=Other +621478KB #6842866 +4937676)
--
[0x00000001099c98a8] Unsafe_AllocateMemory0(JNIEnv_*, _jobject*, long)+0xcc
[0x0000000119017be0]
                             (malloc=8463KB type=Other -469KB #221 -3)

After 10 minutes:
[0x00000001099c98a8] Unsafe_AllocateMemory0(JNIEnv_*, _jobject*, long)+0xcc
[0x000000011a0523b4]
                             (malloc=4349265KB type=Other +4100021KB #34671096 +32765906)
--
[0x00000001099c98a8] Unsafe_AllocateMemory0(JNIEnv_*, _jobject*, long)+0xcc
[0x0000000119017be0]
                             (malloc=8449KB type=Other -483KB #217 -7)

The leaked memory were allocated by the CometArrowAllocator. I've verified this by attaching a debugger to the Spark process and inspected CometArrowAllocator.getAllocatedMemory:

I've also deliberately disabled AQE coalesce partitions since I noticed this issue: https://github.com/apache/datafusion-comet/issues/381. Although it is fixed I still disabled it for being safe.  See Additional context section for more details.
Steps to reproduce
Run the TPC-H benchmark script with --iterations=100 and observe the RSS of the Spark process increase over time.
Expected behavior
Memory usage should not increase over time.
Additional context
I'm simply running it locally with master = local[4]. Here are my test environment and spark configurations:
Environment:

Operating System: macOS 14.6.1, arch: Apple M1 Pro
Apache Spark: 3.5.1
Datafusion Comet: commit https://github.com/apache/datafusion-comet/commit/f7f0bb1ed68367b8d3e1c88010c1f943f480ea11
JVM: 17.0.10 (Eclipse Adoptium)

Spark configurations:
spark.master                     local[4]
spark.driver.cores               4
spark.executor.cores             4
spark.driver.memory              4g
spark.executor.memory            4g
spark.comet.memory.overhead.factor 0.4

spark.jars                     /path/to/workspace/github/datafusion-comet/spark/target/comet-spark-spark3.5_2.12-0.3.0-SNAPSHOT.jar

spark.driver.extraClassPath    /path/to/workspace/github/datafusion-comet/spark/target/comet-spark-spark3.5_2.12-0.3.0-SNAPSHOT.jar
spark.executor.extraClassPath  /path/to/workspace/github/datafusion-comet/spark/target/comet-spark-spark3.5_2.12-0.3.0-SNAPSHOT.jar

spark.serializer          org.apache.spark.serializer.KryoSerializer

spark.sql.extensions         org.apache.comet.CometSparkSessionExtensions
spark.comet.enabled          true
spark.comet.exec.enabled     true
spark.comet.exec.all.enabled true
spark.comet.explainFallback.enabled false

spark.comet.exec.shuffle.enabled true
spark.comet.exec.shuffle.mode auto
spark.shuffle.manager org.apache.spark.sql.comet.execution.shuffle.CometShuffleManager

# Disable AQE coalesce partitions
spark.sql.adaptive.enabled   false
spark.sql.adaptive.coalescePartitions.enabled  false

# Enable debugging and native memory tracking
spark.driver.extraJavaOptions  -agentlib:jdwp=transport=dt_socket,server=y,suspend=n,address=5005 -XX:NativeMemoryTracking=detail

I've investigated the problem and found that the leak is caused by these 2 allocations: https://github.com/apache/datafusion-comet/blob/0.2.0/common/src/main/scala/org/apache/comet/vector/NativeUtil.scala#L65-L66
val arrowSchema = ArrowSchema.allocateNew(allocator)
val arrowArray = ArrowArray.allocateNew(allocator)

This is for constructing the Arrow C data structures for transferring Arrow batch vectors from Scala (JVM) to the native executor (Rust). The native executor will move the transferred vectors and take ownership of them, but the arrowSchema and arrowArray base structures allocated in JVM never get released. Each time we transfer a batch from JVM to the native executor, we leak 2 base structures worth of memory.
I applied a fix on my fork and the problem went away: https://github.com/Kontinuation/datafusion-comet/commit/a90f43a983d234acc7c9a1cf69336b865c5f93ac

The native memory allocated by Unsafe_AllocateMemory0 becomes pretty small and constant:
[0x00000001071c98a8] Unsafe_AllocateMemory0(JNIEnv_*, _jobject*, long)+0xcc
[0x00000001170c05b4]
                             (malloc=15386KB type=Other +1395KB #44 -123)
--
[0x00000001071c98a8] Unsafe_AllocateMemory0(JNIEnv_*, _jobject*, long)+0xcc
[0x0000000116817be0]
                             (malloc=8418KB type=Other -16KB #36 -5)

Running TPC-H benchmarks with AQE coalesce partitions enabled still has memory leak problem, I'm still investigating it.

Finally, I fixed the memory leak caused by AQE coalesce partitions: https://github.com/Kontinuation/datafusion-comet/commit/8657a8255e5d9777933eced764960d34ad7fcf5b
The ArrowStreamWriter holds arrow dictionaries internally, the native memory held by dictionaries will leak if the writer is not closed.

The release callback should do that

I'm afraid that's not the case. To my understanding, the release callback frees everything except the base structure. Please refer to the reference implementation of the release handler in the specification.

I'm afraid that's not the case. To my understanding, the release callback frees everything except the base structure. Please refer to the reference implementation of the release handler in the specification.

If you are referring the JVM instances of ArrowSchema and ArrowArray. They are not referred after exportBatch. How do they keep in JVM?

I'm afraid that's not the case. To my understanding, the release callback frees everything except the base structure. Please refer to the reference implementation of the release handler in the specification.

If you are referring the JVM instances of ArrowSchema and ArrowArray. They are not referred after exportBatch. How do they keep in JVM?

ArrowSchema and ArrowArray hold references to native memory. Garbage collecting these 2 objects does not release the native memory. The native memory has to be deallocated manually by calling the close method.

ArrowSchema and ArrowArray hold native memory. Garbage collecting these 2 objects does not release the native memory. The native memory has to be deallocated manually by calling the close method.

Ah, I took another look at JVM ArrowArray. The internal ArrowBuf should be released separately (done by close). Makes sense.


Feature/one core GitHub mix tests

Summary
This switches the mix mode Rtools 3.5 tests to only use one core on Github actions!
Checklist

[x] Math issue #2930

[x] Copyright holder: Columbia University
The copyright holder is typically you or your assignee, such as a university or company. By submitting this pull request, the copyright holder is agreeing to the license the submitted work under the following licenses:
- Code: BSD 3-clause (https://opensource.org/licenses/BSD-3-Clause)
- Documentation: CC-BY 4.0 (https://creativecommons.org/licenses/by/4.0/)

[x] the basic tests are passing

unit tests pass (to run, use: ./runTests.py test/unit)
header checks pass, (make test-headers)
dependencies checks pass, (make test-math-dependencies)
docs build, (make doxygen)
code passes the built in C++ standards checks (make cpplint)

[x] the code is written in idiomatic C++ and changes are documented in the doxygen

[x] the new changes are tested

@rok-cesnovar this is if #2336 doesn't build. If #2336 builds we can just approve and merge that (this pull is in it). If #2336 doesn't build and merge, then we can merge this separately.

Successful in 300m

My word
If we needed to split again in the future we can do test/unit/math/mix/fun/[a-l]*_test.cpp and test/unit/math/mix/fun/[^a-l]*_test.cpp, but hopefully we make it to R 4.1 before then.


PG::InsufficientPrivilege: ERROR:  must be superuser to COPY to or from a file

I am working through your readme.md to see if this is a good solution for me. From the rails console I am attempting
Customer.copy_to '/tmp/customers.csv'
but getting
PG::InsufficientPrivilege: ERROR:  must be superuser to COPY to or from a file
Is this an issue I can have a workaround for or am I just showing my inexperience?
Appreciate your help. Looks like this might be a really useful Gem that others in the community have directed me to.

This is a PostgreSQL limitation. To be able to import directly from and to files you need a database superuser.
You can work around this using something like this:
File.open('/tmp/users.csv', 'w') do |f|
  User.copy_to do |line|
    f.write line
  end
end