chore: remove swiftype related resources

Relates to https://github.com/newrelic/docs-website/issues/4481

By removing this from the theme, you'll also be removing it from the developer site. Is that intentional? I'd prefer removing the config altogether on the docs site instead so that the related resources aren't used.
Then again, I haven't been involved in conversation, but that linked ticket @zstix linked to is for the docs site.


Initial ethernet support for LPC433x

Ethernet support for LPC433x

Can you extend the year in the license please?

Extended the year to 2015.


Question about imgui

Hey guys, anyone know how to change color of the buttons, slider etc in game? It's not working for me if I go to imgui.cpp and then go to line 741 and change code of the colors and then compile it doesn't work. Nothing have changed. I want to change colors of this gray buttons etc. I'm bored of this color. So can anyone help me?
Maybe @namazso ?

you can use imgui::pushstylevar(with popstylevar after) for specific object or set global imgui style (writing to imgui::getstyle)

@anarh1st47 I don't understand bruh :/ I don't understand how to edit imgui style so I suck in it. Can you please specify your reply for "braindeads" what to do exactly? I just want to change color of that buttons, sliders etc to a specific color.

https://github.com/namazso/nSkinz/blob/master/src/render.cpp#L228


Add a Licence to Project

Hey i wonder what type of licence i should add to this project

THns you are the best


render trials that have geolocation

renders trials that have geolocation on them on a map when the option is selected and adds geolocation to trials.
app crashes if geolocations are strings but that shouldn't be a problem when all of the geolocations are made to be GeoPoints.

Tested and works. Displays empty map if no trials were executed yet. Provides a geolocation point if there was a trial.


Change all Low Priority measures to be cost based

Closes #986

Closing as this approach won't work. See:
https://github.com/ebmdatalab/openprescribing/issues/986#issuecomment-426215975


[Feature]: Support for Azure OpenAI

Is your feature request related to a problem? Please describe.
We are currently using Azure OpenAI to make API calls, seems that GPTCache only supports using the standard OpenAI API keys and it isn't possible to route the requests through Azure
Describe the solution you'd like.
OpenAI's library allows us to specify to route the requests through Azure with the following config:
openai.api_type = "azure"
openai.api_version = "2023-05-15"
openai.api_key = "<api key>"
openai.api_base = "https://<tenant>.openai.azure.com/"
model = "gpt-35-turbo"

Including support to do this through the GPTCache OpenAI adapter would be the way to replicate the same for GPTCache
Describe an alternate solution.
N/A
Anything else? (Additional Context)
No response

@SimFG yes, and particularly the Azure OpenAI API key (which is fundamentally different from the vanilla OpenAI API Key)  should be able to be used to route the requests to api_base in the absence of a standard OpenAI API key


gdi_polygon_sc: not implemented with /gdi:sw

I'm able to reproduce it only with remmina on a special Remmna branch called libfreerdp_updates (which is itself not 100% mature): connecting to a windows 2008 server and opening "Shares and Storage Management" it seems that some areas of the screen are not filled, and on the console I get a lot of these errors:
[12:40:47:435] [29813:29832] [WARN][com.freerdp.gdi] - gdi_polygon_sc: not implemented
[12:40:47:435] [29813:29832] [WARN][com.freerdp.gdi] - gdi_polygon_sc: not implemented
[12:40:47:435] [29813:29832] [WARN][com.freerdp.gdi] - gdi_polygon_sc: not implemented
[12:40:47:435] [29813:29832] [WARN][com.freerdp.gdi] - gdi_polygon_sc: not implemented
[12:40:47:435] [29813:29832] [WARN][com.freerdp.gdi] - gdi_polygon_sc: not implemented
[12:40:47:435] [29813:29832] [WARN][com.freerdp.gdi] - gdi_polygon_sc: not implemented
[12:40:47:435] [29813:29832] [WARN][com.freerdp.gdi] - gdi_polygon_sc: not implemented
[12:40:47:435] [29813:29832] [WARN][com.freerdp.gdi] - gdi_polygon_sc: not implemented

@giox069 you need to deactivate POLYGON_SC capability in preconnect, it never was implemented.

It works, after disabling both NEG_POLYGON_SC_INDEX and NEG_POLYGON_CB_INDEX.
Thank you :)


Move all constants out of the code and into the new initializer

Constants:
Remove any hard coded values into the new config/initializers/dmproadmap.rb file.
Use the config.[category].[variable] = "[value]" syntax. Be sure to add a comment explaining what the variable is used for.
Then update the place where the variable was removed from with Rails.configuration.x.[category].[variable]
If the variable is used often, consider adding a helper method to the ApplicationService. For example Rails.configuration.x.application.name is used all over the codebase so it was simplified to ApplicationService.application_name
Credentials:
Go through any config and config/initializer files we have that are samples and move their sensitive values into the new config/credentials.yml.enc file.
Also look into switching references to these values over to ENV variables so that installations can either use the config/credentials.yml.enc or environment variables.
Here are some articles to read before starting that work. I would suggest we err towards the Rails6 approach:
https://medium.com/cedarcode/rails-5-2-credentials-9b3324851336
https://blog.capsens.eu/rails-5-2-imperfect-credentials-4ef03934aea4
https://blog.saeloun.com/2019/10/10/rails-6-adds-support-for-multi-environment-credentials.html

need to move the new Recaptcha flag and Google Analytics tracker stuff into the new config/initializers/dmproadmap.rb

related to #2036. The Dragonfly initializer also has a secret that should be moved to the credentials file

There is a sample config/credentials.yml.enc file which contains brief instructions on how to set it up. In the current set up the credentials are in the one file with a single RAILS_MASTER_KEY env variable. When we move to Rails 6 this will get split out into environments.
Because the credentials are now encrypted the devise.rb. and recaptcha.rb files can now be standard so they don't need .example version and they can be safely checked in.
There are a couple of other constants pulled out and set in dmproadmap.rb


Hiding a row of items in GridContainer messes up the layout

Godot 3.0.3 RC3
I use a GridContainer with two columns to make a small inspector, and sometimes I need to hide lines so I hide two items at a time.
But it doesn't work:

I'm aware there is an issue with hiding an item, but here I'm hiding a full row.
It seems it was fixed in master though, but 3.1 is coming in more than a month... any chance to get it fixed in 3.0.3?

Given that 3.1 was released in the meantime and 3.2 is around the corner, there's little incentive to backport a fix to 3.0. If anyone feels strongly about it, feel free to make a PR backporting the relevant changes.


Support for Mqtt server login, support for Gpio3

I added support for defining a username and password for Mqtt-servers demanding this. I also added a possibility to use Gpio3 on the Esp-01 as receiver input by disabling RX.

Hi, I guess it makes sens to split this into two pull requests, one for each topic.
As for the username and password, did you modify the example config file?  What will happen if there is no password and username given?
As for GPIO3 I guess using it os ok and the if clause to configure the serial port accordingly seems valid. However, I would not change the standard GPIO. The project is based on a self made PCB board which by uses the already given GPIO. Furthermore, people have to adapt to their needs. It does not make sens to change the GPIO, as we would soon receive another path for yet another port number.
Lets see what @puuu has to say. ;)

Thanks for your response! Should I just do new commits or shall I remove the ones I already commited?

Did you tested this PR?
Yes its been running on one of my Esp01's for a few days. I also used it to get key presses from a RF remote into Home Assistant.

Just replace/edit the commits.

@bphermansson you got somehow meshed up with the git merge.
Using git rebase is the way to edit commits. Another hint that makes the work easier: As suggested by github, pull-request should be done from separate branches.
Anyhow, thank you very much for you contribution. I merged the ESP01 commit in 90d3046ff1936e7a5ea03c72cf89ea6154003e4b and the authentication commit in 8271deca54a1b4332595eb1af01663fe818c4663 , after editing.
You can find it in the auth branch. Please test it!

Thanks! I'm not used to use Git/GitHub this way as you might have guessed :)
I will try your branch asap.

No problem, we are all here to learn more...

Thanks :)
I've tested the auth branch now and it works great!
One thing for comments or docs: Remember to remove the cable on Rx/Gpio3 after programming, before testing the circuit. Otherwise Gpio3 is pulled high be the serial converter and nothing is received.

Thanks :)
I've tested the auth branch now and it works great!
One thing for comments or docs: Remember to remove the cable on Rx/Gpio3 after programming, before testing the circuit. Otherwise Gpio3 is pulled high be the serial converter and nothing is received.

Thank you for testing.
I merged the auth branch into master.

Great!


Distribute a prebuilt binary

It is recommended to release not only the source code, but also the precompiled version to reduce the CI time

no, I will try it , thanks

Closing this as this isn't part of the scope.


Visual Studio 2017 compilation flag for python_internal bindings

To get a successful compilation of internal Python bindings with Visual Studio 2017, I've added the option /bigobj in bindings/python_internal/CMakeLists.txt.

@agodemar Thanks! Looks good... I'll merge it.


GIST Ground Truth Data Missing

I want to run the Search Performance Test on the GIST dataset. I created a new test, since current workloads don't have GIST as part of the performance test. Currently GIST and SIFT are only used in capacity test.
However, the dataset doesn't contain the ground truth data. It only downloads train.parquet and doesn't download the ground truth data (I believe that would be neighbors.parquet).

Right. We are considering opening up more datasets in the next release, as well as supporting users with their own local datasets.

Currently GIST and SIFT are only used in capacity test.
the dataset doesn't contain the ground truth data.
I believe that would be neighbors.parquet

Right. We are considering opening up more datasets in the next release, as well as supporting users with their own local datasets.

Currently GIST and SIFT are only used in capacity test.
the dataset doesn't contain the ground truth data.
I believe that would be neighbors.parquet

@alwayslove2013 Also need this! Any update here? or is there any way to generate neighbor.parquet from the origin gist and sift ground truth files? thx

@alwayslove2013 I wanted to ask how we can generate ground truth data. I am using pgvector, and when I remove the index and query the data my understanding is that I should get the GT data. Just to verify this, I tested this on the OpenAI 500K dataset (cosine distance), I found that the there are few mismatches in the GT data that I calculated and the one provided by VectorDBBench. The difference is only in the order, and the set of returned vector is the same. Usually two elements are just swapped.

This happens when there are ties in the ground truth,  there is no guarantee that any specific engine will return ties in a specific order, or even in the same order consistently,


Multisites and GPM

Hi there,
Trying to see how would maintaining a multi site configuration with the GPM.
The only way I see would be keeping the plugins and themes under user/ and symlink the needed elements in each /user/env/domain/plugins or /user/env/domain/themes
Is there another way to control/maintain each sub sites with the GPM separately ?
(with the CLI "FATAL: Must be run from ROOT directory of Grav!" obiously yields when I bin/gpm in a sub domain and from the root I see not option to select the sub domains)
Cheers
Bruno.

Matthias answered in discord. GPM manages themes and plug-ins in user/.

Closing


Add a failover flag for clusters

Is there a reason this is going in to master rather than the failover branch in the forcedotcom repo?

I don't think there is a reason to push to a separate branch in that repo, travis-ci still runs on this branch

Looks good


After file transfer the program usable/no trash can access either/ the file dose not update

Description
After file transfer the program usable/no trash can access either/ the file dose not update in the drives or in any folders after deleting a file
Steps To Reproduce
After file transfer the program usable/no trash can access either/ the file dose not update in the drives or in any folders after deleting a file
Requirements

e.g. Fix crash that occurs when...

Files Version
<IP_ADDRESS>
Windows Version
10
Log File
No log file

Thanks for the report, Please update to <IP_ADDRESS> to continue receiving support https://files.community/docs/install#:~:text=Keep Files up to date. If this is still an issue please comment here


spawn will not exit when finished if there are connections open

Describe the bug
Found as part of #354.
In a real AWS Lambda, context.callbackWaitsForEmptyEventLoop = false; can be set - this allows one to run a lambda without disconnecting from database. This is used to keep Lambda connections open on MongoDB, RethinkDB, etc.
Arc needs to simulate AWS's behavior locally.
We need to create the equivalent of callbackWaitsForEmptyEventLoop = false (ie, exit as soon as all processing is done, rather than waiting for the event loop to be empty?
To Reproduce
Here is some demo code. Change FUNCTION_NAME to 'connectOnly' or 'connectAndDisconnect' to demonstrate the behavior.
Right now 'connectAndDisconnect' works, but 'connectOnly' does not - it finishes processing, but won't ever exit node since the connection is still open.
let spawn = require('child_process').spawn
var log = console.log.bind(console)
let minify = script=> '"' + script.replace(/\n/g, '').trim() + '"'

var functions = {
    connectAndDisconnect: `const rethinkdb = require('rethinkdb');
    ;(async function(request, context) {
        var connection = await rethinkdb.connect({ host: 'localhost', port: 28015 });
        await connection.close();
        console.log('I connected successfully!');
    })();`,
    connectOnly: `const rethinkdb = require('rethinkdb');
    ;(async function(request, context) {
        var connection = await rethinkdb.connect({ host: 'localhost', port: 28015 });
        console.log('I connected successfully!');
    })();`
}

const FUNCTION_NAME = 'connectOnly'

log(`Testing ${FUNCTION_NAME}`)

var functionToTest = functions[FUNCTION_NAME]

var child = spawn('node', ['-e', minify(functionToTest)], {
    shell: true,
    stdio: [0, 'pipe', 'pipe'],
    cwd: 'C:\\Users\\mikem\\OneDrive\\Documents\\appless\\documentdb-arc-test\\src\\http\\get-index',
    env: {}
})

var completed = false;

let timeout = 5000;
// bake a timeout
setTimeout(function () {
    log(`Timed out after ${timeout} ms!`)
    child.kill()
}, timeout)

// 'close' will occur if the process finishes naturally
// 'exit' will occur if the process is killed
console.log(`What we want is to be see a 'close' event using the 'connectOnly' code above`)
;['close', 'exit'].forEach(function(event){
    child.on(event, function () {
        log(`'${event}' event happened!`)
        if ( ! completed ) {
            completed = true
            log(`Done!`)
        }
    })
})

What happens?
Unless we close the connection, the lambda will never complete and will time out
Expected behavior
Once the lambda has completed processing it should exit, even if the connection is still open.

Alternatively lambda-local simply uses require to load the lambda, instead of spawn. The lamdas execute normally, run their callback and will be considered to be finished even if there are open connections, unlike with spawn where the process never terminates:
https://github.com/ashiina/lambda-local/blob/develop/lib/lambdalocal.js#L180

We used to do that but won't fly w other runrltimes unfortunately

Yeah I notice it's used to launch Python. I hate having two code paths, but I don't see any way to make a spawned node process exit when compution is done, even if there are handles open.
Unless: we spawn a node process that does the require... 🤔

I'm gonna hack here. BRB.

Cool, worth exploring! God speed (also don't mind a bit of divergent paths as long as semantics same)

Just a quick update that I've been working on other things and haven't had the time to create a proper fix. If someone else wants to grab it be my guest,


Improving geometry displays for PS,TS and remote handling room

A suggestion to use the REve display as an alternative to geometry browsing. We currently only have a crude outline of the PS, TS and production target. To do MC beam studies we might want to improve the view and Sridhar is to start thinking about what we might want here

@sridhar130 is this something you'd still like to pursue?

Hi @sophiemiddleton, If someone else is attempting then it's alright. But, next month I can try to pursue it.

No worries, I think there's a more general action item which is more pressing, but if you still think this is something you want to add then let me know. If not then we can close the issue


Simplify NetworkChannel interface

This pull request removes the try_connect function from the NetworkChannel.
The runtime will now only tell the NetworkChannel implementation such as CoapUdpIpChannel to open or close a connection. The channel will then internally take care of connection establishment.
The runtime can get the NetworkChannel state using get_conneciton_state.

@erlingrj I am thinking about one thing: It seems to me that the runtime really only cares about one thing now: Is the channel connected or not. So maybe it is reasonable to replace that get_connection_state function with a simple boolean function: is_connected?

I have now also checked that the POSIX federated example works :) So I mark this PR as ready 🥳

Thanks for the feedback! All good points I will try to find a better solution tomorrow

@erlingrj I am thinking about one thing: It seems to me that the runtime really only cares about one thing now: Is the channel connected or not. So maybe it is reasonable to replace that get_connection_state function with a simple boolean function: is_connected?
Or do you think maybe there could be reasons why we need more information in the future?

Let's do this, I think it is better to keep the API as minimal as possible and then potentially expand later

So I hope I have addressed the parallel send and receive issue now.
I am now using a pipe between send_blocking and the worker thread.
The worker thread uses select to listen for the channel socket and the pipe.
I also fixed some issues with reset_socket causing the other federate to close the channel which is something that should only happen from within the runtime in my opinion, because it makes it impossible for the tcp ip channel to recover

I have now changed it to use an eventfd_t instead of a pipe. This way it should be compatible to zephyr OS also

The review should be addressed now :)


build(tracking_object_merger): remove no longer needed nlohmann_json dependency

Description

Remove use of nlohmann_json, it's no longer used and it being in CMakeLists.txt is causing the Debian build fail (see https://github.com/autowarefoundation/autoware-deb-packages/actions/runs/6639929281/job/18039381566?pr=41)
Tests performed

Not applicable.
Effects on system behavior

Not applicable.
Pre-review checklist for the PR author
The PR author must check the checkboxes below when creating the PR.

[ ] I've confirmed the contribution guidelines.
[ ] The PR follows the pull request guidelines.

In-review checklist for the PR reviewers
The PR reviewers must check the checkboxes below before approval.

[ ] The PR follows the pull request guidelines.

Post-review checklist for the PR author
The PR author must check the checkboxes below before merging.

[ ] There are no open discussions or they are tracked via tickets.

After all checkboxes are checked, anyone who has write access can merge the PR.

@YoshiRi @yukkysaito if in the future you want to reenable this, please also add <depend>nlohmann-json-dev</depend> to the package.xml, otherwise rosdep can't find the dependency. Thanks.


Fixed 18048 std.bigint.toDecimalString is impure

CircleCI is segfaulting. I don't see anything here that should do that, but I'm just a hairy northerner, so I could be wrong.

CircleCI is segfaulting. I don't see anything here that should do that, but I'm just a hairy northerner, so I could be wrong.

You won't believe this, but you aren't wrong. We had some issues with random segfaults of DScanner lately.

I didn’t know and find it interesting that you can just return a char[] as a string. Cool!


Remove unnecessary comments from acli.py - Resolve PR #2

Remove unnecessary comments from acli.py specifically at lines #34, #35, #45, #52 and #105

Reference issue #1

@heisendumb I think you have certain experience in programming and this issue seems rather trivial for you, hence closing this PR. Kindly find other issues in my projects that suits you better 😄


stdlib: Add direct.h to ucrt.modulemap

direct.h is an header Windows provides that provides POSIX file system APIs such as these.

@swift-ci please smoke test

I dont think that Direct belongs in the POSIX namespace.  What exactly is the module currently missing?
If it is _mkdir that you are after, can you please explain the reason for that?  You should be using CreateDirectoryExW over that; at the very least, _wmkdir

@compnerd I'm not too attached to anything from direct.h. I'm in the process of porting a big-ish set of file-system APIs. So I think the best course of action is to hold off this change and see how much I can achieve without relying on direct.

That sounds good to me.  But keeping it updated is going to be useful I think.  There’s a good possibility that I may have missed stuff that we should merge if we find.  Let’s identify the loose ends and get them resolved.

sounds good. Thanks for your timely responses. I'll close this for now. Will report any findings from the future.


Fix installation errors

It only affected python3 setup.py install installations, so this was never caught. Nice work! Merging this; thanks for the PR.


Add custom storage

This is a proof-of-concept to see whether Hypercore could support alternative storage to random-access-storage.
This PR cleans up the API of the internal Storage class so that the main hypercore does not reach into the internals of the storage API, then it adds an opts.storage option for passing an instance of a custom storage class that implements the same API as the default storage.
NB. This is not the same as opts.createStorage, which refers to the instance of random-access-storage used by the Storage class. This is for overriding the whole storage with something other than random-access-storage.
How to implement custom storage?
See hypercore-storage-level as an example storage implemented on a leveldb database.
Why?

Limited file descriptors
Hypercore opens 6 random-access-storage instances, which each leave open a file descriptor. With multiple hypercores open (e.g. for multifeed this can quickly add up to a lot of file descriptors. Android is limited to 1024 FDs per app, and some devices limit to 512, so 86 hypercores would exceed this limit. This is quite possible if each device in a project had 2 hypercores and 50 people are participating.

Server storage
For storing Hypercore data online, being tied to random-access-storage requires storage to be on an attached disk. Allowing alternate storage would allow data to be stored in an external database, or object storage such as S3.

Recovering space
Currently, using sparse files means that you only use disk space for the hypercore blocks that you have downloaded. However, clearing these blocks does not recover disk space (see https://github.com/mafintosh/fsctl/tree/punchhole for WIP to add native "punchhole" functionality to clear a sparse file and recover disk space). Implementing a custom storage, such as a key-value store, allows disk space to be recovered when blocks are cleared.

Progress

[x] Isolated existing storage module. Currently hypercore reaches down into the internals of the storage module. Better isolation allows a stable public API for the storage module.
[x] Create a proof-of-concept leveldb storage. For performance reasons it might be better to keep some storage (e.g. bitfield and tree) as random-access storage, but for the initial POC it's using all leveldb.
[x] Get tests passing. Some of the tests reach down into storage internals, so they need to be modified.

NB. I completely understand if the team doesn't want to merge this - it exposes more API surface area to maintain, so it might not make sense, and I think there is a slight performance impact. It is super useful though for where we don't have access to the filesystem, e.g. in a browser or in a serverless environment.

@gmaclennan random-access-storage is not related to the filesystem, it is the base library/system for any kind of implementation and you should be able to create a random-access-leveldb if you wanted to. In the description, I believe you wanted to point at shortcomings of random-access-file?! It would be absolutely possible to manage the amount of file system pointers of random-access-storage by only keeping the (say) last-recent 50 file system pointers at a time open. This could be added to random-access-file or even added as a layer on top of random-access-storage.
I am thinking of a module called on-demand-random-access-storage:
const ondemand = require('on-demand-random-access-storage')
const file = ondemand(require('random-access-file'), {
  maxOpen: 50, // only open up to 50 at a time
  autoClose: 500 // closes idle instances after 500 ms
})

This way you could pass any given random-access-storage implementation to the ondemand constructor and it would open the underlying random-access-file implementation only if a read or write operation happens and close it if idle for too long. Too many requests at a time could use a queue and a LRU to close file handles that have been used "longest time ago".

Thanks @martinheidegger, that's certainly one option for solving some of the problems I outline. random-access-idb solves this for indexeddb in the browser by paging data and essentially emulating random access storage on a key-value store.
One of the motivations for experimenting with the approach in this PR is because internally within the storage class of hypercore, it is reading and writing data as if to a key-value store, essentially storage.getData(sequenceNumber). The default storage model essentially does some clever stuff to do that getData(sequenceNumber) in a really fast and efficient way over random-access-storage, but then emulating random access on a key-value store looses that efficiency.
I think either approach can work. I like that by exposing the storage API of hypercore it allows a lot of flexibility about how data is stored, but it increases the API surface area of hypercore. The changes to hypercore were less work for me that writing on-demand-random-access-storage and random-access-leveldb, and I think more efficient. I like that it creates options like storing each block of data in a hypercore as a separate file, which is useful when storing large binary data like images. It means that the data can be read without relying on hypercore, since it's just files.
But anyway, putting this out there with an open mind as to whether it makes sense to merge or not.


Alternative to priority_queue dependency

dc78d2ddb5b37bdc0ace5d886678fefdffa349fc added a new dependency on the priority-queue crate, which is LGPL licensed - unfortunately, since Rust statically links all libraries and we distribute binaries to customers, this currently likely prevents us using mysql_async past that commit. Is there a world where you'd consider an alternative to that dependency, either permanently or via a feature flag, to allow non-copyleft-compliant use of mysql-async?

Oh, thanks for pointing it out!
@cloneable, could you please come up with a replacement?

Sure! Technically, p-q is also licensed under MPL2, but I'm going to look for an alternative. Maybe the wait queue can be replaced with a tokio semaphore based queue. Re-insertion was the big issue.


chore(example): update to vitejs

Description
Updates the example/ directory to use vitejs and support node.js v18.
Detail

Bundling has migrated from create-react-app to vite

Are there any trade offs vs. other app bundlers? Or is the goal just to have something relatively straightforward to replace CRA due to its incompatibility with node LTS?

Are there any trade offs vs. other app bundlers? Or is the goal just to have something relatively straightforward to replace CRA due to its incompatibility with node LTS?

@geotrev Ultimately, it is a drop in replacement for CRA for compatibility however it does bring simplicity compared to other bundlers with some configurability if needed. Vite also uses rollup under the hood which is a common tool in our toolchain.


Add configs for apu{2,3,6}

@pkubaj Some pre-commit failures slipped through the cracks: https://results.pre-commit.ci/run/github/554712842/1710315604.E2-RLA3JShmb559hG6i0Xg
It is not from your code, but I would appreciate, if you could fix it as well.
Changes LGTM, we just need some logs proving they work fine.

I needed to switch the boot menu shortcut to F10, because APU2 uses F10.

Ah right, I tested a build from before the boot menu keys were changed.

logs.tar.gz
Here are logs for apu2. For apu3, the cable used for flashing doesn't work, so I couldn't test it there, but since @mkopec agrees to the boot key change, I assume it will be fine as well.

Logs from APU6
logs.tar.gz
APU3 wasn't tested because of no output on the serial (even after flashing the last public coreboot release).


Issues from switchback & rttc

Switchback:

Original post: https://github.com/node-machine/switchback/issues/7
RTTC

Original post: https://github.com/node-machine/rttc/issues/2

Original post: https://github.com/node-machine/rttc/issues/3

https://github.com/node-machine/rttc/issues/7

Hey everyone, we're moving these issues to the main Sails repo for greater exposure to the community.  We're able to transfer the actual posts since they are from a different org, but I've made screen-captures of them to follow-up with here in Sails.  Hope these help out and feel free to add any further developments - Thanks!

This will get addressed as part of the larger vuln/deprecation warning sweep.
2 and 3. both of these are about making the browserified build size smaller for browser-side use cases (e.g. static analysis in a web browser / in electron-based native app)
same as 1


Cytokine Production Terms

New Term:  cytokine production (and associated regulation
terms)
Definition:  The synthesis and release of a cytokine following
a stimulus, resulting in an increase in its intracellular or
extracellular levels.
Comment:  Note that this term is intended for use when a
gene product is seen to cause apparent increases in
intracellular or extracellular cytokine levels, without specific
regard as to whether the increase is due to increased
biosynthesis, increased secretion of preexisting cytokine
molecules, or increased conversion from precursor
molecules.
Parentage (DAG View):
immune response
< cytokine production ;  GO:New Term
< cytokine biosynthesis ;  GO:0042089
< cytokine secretion ;  GO:0050663
< regulation of cytokine production ;  GO:New Term
% negative regulation of cytokine production ;
GO:New Term
< negative regulation of cytokine biosynthesis ;
GO:0042036
< negative regulation of cytokine secretion ;
GO:0050710
% positive regulation of cytokine production ;
GO:New Term
< positive regulation of cytokine biosynthesis ;
GO:0042108
< positive regulation of cytokine secretion ;
GO:0050715
< regulation of cytokine biosynthesis ; GO:0042035
< regulation of cytokine secretion ;  GO:0050707
Reference:  Fundamental Immunology (ISBN 0-727-3514-9)
Cytokine production for specific cytokines may be added as
children to this term as necessary.  Such terms are useful in
the GO because they can be indicative of the course of an
immune response.
The regulation terms:
New Term:  regulation of cytokine production
Definition:  Any process that modulates the frequency, rate,
or extent of production of a cytokine.
Reference:  Fundamental Immunology (ISBN 0-727-3514-9)
New Term:  negative regulation of cytokine production
Definition: Any process that stops, prevents, or reduces the
rate of production of a cytokine.
Reference:  Fundamental Immunology (ISBN 0-727-3514-9)
New Term:  positive regulation of cytokine production
Definition:  Any process that activates or increases the rate
of production of a cytokine.
Reference:  Fundamental Immunology (ISBN 0-727-3514-9)
The cytokine biosynthesis and cytokine secretion terms and
their associated regulation terms should all retain their
original locations in the process DAG in addition to their new
relationships indicated above.
-- Alex
Reported by: addiehl
Original Ticket: "geneontology/ontology-requests/1422":https://sourceforge.net/p/geneontology/ontology-requests/1422

thanks www.movied.org


Placement of 'after_save' callback before 'after_*' callbacks causes false negatives and code duplication

I have a rails model with many callbacks - including:
after_create :trigger_created_event, unless: :skip_subscriptions
after_update :trigger_updated_event, unless: :skip_subscriptions
after_destroy :trigger_deleted_event, unless: :skip_subscriptions

Rubocop returned no errors.
I added an after_save callback to the beginning of the group:
after_save :update_status
after_create :trigger_created_event, unless: :skip_subscriptions
after_update :trigger_updated_event, unless: :skip_subscriptions
after_destroy :trigger_deleted_event, unless: :skip_subscriptions

I then ran rubocop -A, which "corrected" many errors:
434 offenses detected, 434 offenses corrected
Looking at the file, I saw that some of the callbacks (outside of the above list) had been moved, with rubocop reporting that they were in the wrong order (they were not).
Other callbacks were duplicated by rubocop up to ~50 times.
I undid those changes by rubocop, and changed the ordering of the after_* callbacks, like so:
after_create :trigger_created_event, unless: :skip_subscriptions
after_update :trigger_updated_event, unless: :skip_subscriptions
after_destroy :trigger_deleted_event, unless: :skip_subscriptions
after_save :update_status

Running rubocop -A again returned no errors.
Expected behavior
I would expect that rubocop either:
a) Returned no errors, regardless of the ordering of the after_* callbacks
b) Returned some error saying that after_save needs to be at the end of the list, if that is intended
Actual behavior
Rubocop returned >400 errors, moving some callbacks and duplicating others many times.
Steps to reproduce the problem

Take an existing rails model, with existing callbacks. The more the better.
Include a group of after_* callbacks, but not an after_save.
Ensure rubocop passes.
Add an after_save callback above the other after_* callbacks.
Run rubocop

RuboCop version
$ [bundle exec] rubocop -V
1.22.2 (using Parser <IP_ADDRESS>, rubocop-ast 1.12.0, running on ruby 2.7.3 x86_64-darwin19)
  - rubocop-rails 2.12.4
  - rubocop-rspec 2.5.0

Note: Upgrading to 1.22.3 had no effect.

Closing this issue as I realized afterwards this belongs in the rubocop-rails repo. I've re-created the issue there: https://github.com/rubocop/rubocop-rails/issues/583


General Post POC refactoring.

Add unit test coverage
Add linting and static code analysis
Normalize and simplify entity ids
Split entities into correct platforms (ie. binary_sensor and number)
Simplify _data updating process and property fetching, keeping whole returned data structure

Resolves #8

Codecov Report

:exclamation: No coverage uploaded for pull request base (main@ea370ba). Click here to learn what that means.
Patch has no changes to coverable lines.

:exclamation: Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the GitHub App Integration for your organization. Read more.
Additional details and impacted files
@@           Coverage Diff           @@
##             main       #9   +/-   ##
=======================================
  Coverage        ?   99.62%
=======================================
  Files           ?        9
  Lines           ?      269
  Branches        ?        0
=======================================
  Hits            ?      268
  Misses          ?        1
  Partials        ?        0

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


document v6.84 enhancements in the UG

Reported by ansley b manke on 26 Dec 2012 18:30 UTC
They are listed in the v6.84 release notes
Migrated-From: http://dunkel.pmel.noaa.gov/trac/ferret/ticket/2018

Modified by ansley.b.manke on 28 Feb 2013 23:16 UTC


Meta inf

CI: Security warnings found!

CI: Security warnings found!

CI: Security warnings found!


Remove need for global gulp

As it is now, localdev.js run spawns a gulp instance from the global scope. We can remove this need by relying on scripts in package.json

Closing of, localdev is dead !


[semantic-dom-diff] should strip global boolean attrs

      + expected - actual

             </button>

      -

      +

             <button data-step="2">
               2
             </button>

This test should pass under semantic-dom-diff, since, semantically hidden is equal to hidden=""

I think this was probably a user error on my part


Missing documentation for package operators and public classes

While reorganizing the project for packaging, I noticed the following operators and types remain undocumented:

ContextTask
CreateContext
ConfigureHarpSyncInput
ConfigureLoadTester
DeviceFactory
HarpSyncInputData
HarpSyncInputDataFrame
HubDeviceFactory
PassthroughState
StartAcquisition

We could potentially internalize some of these as implementation-specific classes (e.g. ContextTask, PassthroughState) in addition to the ones already internalized in #172. This would simplify both the API and docs effort at this point, but the operators will need to be included.

Yep on my to-do tomorrow. Thanks for pulling out the missing ones.
On Sat, Jul 27, 2024, 6:41 PM glopesdev @.***> wrote:

Assigned #173 https://github.com/neurogears/onix-refactor/issues/173 to
@jonnew https://github.com/jonnew.
—
Reply to this email directly, view it on GitHub
https://github.com/neurogears/onix-refactor/issues/173#event-13669648120,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAK47Y74DYVIKRWRH53PH7TZOQO3HAVCNFSM6AAAAABLSHIOD2VHI2DSMVQWIX3LMV45UABCJFZXG5LFIV3GK3TUJZXXI2LGNFRWC5DJN5XDWMJTGY3DSNRUHAYTEMA
.
You are receiving this because you were assigned.Message ID:
@.***>

@jonnew great, let me know if you want to merge #172 to main beforehand so you have a few less classes to worry about.

That would be helpful. Im going to fork from 172 for my working copy

Great, no need to fork, I have merged it now.


How to enable user provisioning in the new UI

[Enter feedback here]
The documentation has steps defined for the old UI, and I cannot find the 'Enable user provisioning' settings in the new UI.

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

ID: 8f6960e6-1a0c-267f-4ec4-2ff7574b1f09
Version Independent ID: b08f7ae9-2b9a-c629-74e8-71a403543b97
Content: Configure SCIM provisioning using Microsoft Microsoft Entra ID (formerly Azure Active Directory) - Azure Databricks
Content Source: databricks/administration-guide/users-groups/scim/aad.md
Service: azure-databricks
GitHub Login: @mssaperla
Microsoft Alias: saperla

wrong request

@gevorg-gh-epam
It looks like your issue is resolved since you closed this thread.
If there are further questions regarding this matter, please tag me in a comment. I will reopen it and we will gladly continue the discussion.


Allow using a specific pane/window for tests

It would be nice if it were possible to name a pane or window and then have vimux always use that window. For example, I'd like to have a separate window of my tmux session open on a second monitor with a window for test results. Then any vim windows should always use this window for vimux. It seems like it's not currently possible to guarantee that a particular window will always be used.

@michaelmior  This isn't exactly what you are asking for, but if you revert one of the commits it adds (9ec2d6), you could get very similar behavior to what you describe.

@dkrieger Thanks! I'll close this since it seems like your PR will resolve this issue.

@michaelmior let me know if it doesn't work, but so long as they share the same session I think it will work. It looks like Ben Mills hasn't been active on GH for a while, feel free to use my fork as I'll keep it up to date with upstream until/unless the PR is accepted


Vickel: Watch tamildhoollll.cc

Vickel requests the watch of the watch_keyword tamildhoollll\.cc. See the MS search here and the Stack Exchange search in text, in URLs, and in code.

tamildhoollll\.cc has been seen in 1 true positive, 0 false positives, and 0 NAAs.

Approved by Jeff Schaller in Charcoal HQ


Fix CI Integration Workflow

This PR is intended to fix the Integration CI workflow. The libxmtp binaries have been updated and failing tests have been marked @Ignored for now to focus on the failure point which was related to the presence of dynamically linked libatomic  symbols as noted in the following libxmtp PR: https://github.com/xmtp/libxmtp/pull/709

Do you mind updating this text file so we know which version of the binarys this is https://github.com/xmtp/xmtp-android/blob/main/library/src/main/java/libxmtp-version.txt


Fix tests for 2023.2 update

Fix tests for the update to support idea version 2023.2

It looks like we are failing some verification checks:
https://github.com/launchdarkly/ld-intellij/actions/runs/7342537375/job/19991903736?pr=70
Compatibility problems (12):
      #Invocation of unresolved method com.intellij.ui.dsl.builder.Cell.label$default(Cell, String, LabelPosition, int, Object) : Cell
          Method com.launchdarkly.intellij.settings.LaunchDarklyApplicationConfigurable.createPanel$1$7$4$1.invoke(com.intellij.ui.dsl.builder.Row $this$row)

It looks like we are failing some verification checks: https://github.com/launchdarkly/ld-intellij/actions/runs/7342537375/job/19991903736?pr=70
Compatibility problems (12):
      #Invocation of unresolved method com.intellij.ui.dsl.builder.Cell.label$default(Cell, String, LabelPosition, int, Object) : Cell
          Method com.launchdarkly.intellij.settings.LaunchDarklyApplicationConfigurable.createPanel$1$7$4$1.invoke(com.intellij.ui.dsl.builder.Row $this$row)

Oh. I updated that. Hopefully it should pass now


Add package sections to changelog

In the initial changelog layout I didn't think about the fact that the
three published packages are versioned independently (and we don't
necessarily release updates to all three at the same time), so it makes
sense to split them out into separate sections.

Alternatively we could have three separate files, e.g. CHANGELOG-uefi.md, CHANGELOG-uefi-macros.md,
CHANGELOG-uefi-services.md. Not sure which is clearer, I'm probably just bikeshedding myself :)

I'd say it's alright to stick to a single file for all packages. It's only the main crate which sees the most of changes, the others will probably have very small changelogs 😅


opt Optimize the conversion of numeric types to strings.

Proposed changes
before
mysql [(none)]>select sum(length(cast(number as string) ))   from numbers("number" = "100000000");
+-----------------------------------+
| sum(length(cast(number as TEXT))) |
+-----------------------------------+
|                         788888890 |
+-----------------------------------+
1 row in set (7.24 sec)

now
mysql [(none)]>select sum(length(cast(number as string) ))   from numbers("number" = "100000000");
+-----------------------------------+
| sum(length(cast(number as TEXT))) |
+-----------------------------------+
|                         788888890 |
+-----------------------------------+
1 row in set (1.40 sec)

More batch functions will be implemented in the future.

run buildall

run buildall

run buildall

run performance

run buildall

run buildall

run buildall

run buildall

run buildall

run p0

run p0

run buildall


Add colourise to the store

Signed-off-by: Finnian Anderson<EMAIL_ADDRESS>

Thank you for getting to this. Please make the changes to the existing colorisation function - we don't need to maintain multiple versions - that's going to be super confusing. https://github.com/alexellis/repaint-the-past


[Bug] Addon was loaded twice

Describe the bug
After I installed addon and listed it in main.ts, I got an error:
@storybook/addon-styling was loaded twice, this could have bad side-effects
Steps to reproduce the behavior

Install addon
Add addon to main.ts
Run storybook
See console

Expected behavior
I do not exect any error in the console.
Environment

OS: macOS Ventura 13.4.1
Node.js version: v16.17.0
NPM version: 6.14.16
Browser (if applicable): Chrome
Browser version (if applicable): 115.0.5790.170

might be duplicate of #67

@Integrayshaun hey this issue seems to be caused by the export managerEntries from src/preset.ts. In particular, It seems addon-styling is registered twice because of having both src/manager.ts and managerEntries in src/preset.ts at the same time.
Screen captures

Hey @sookmax that's a great catch!!! I'll get a patch out for this today

Hey folks! I have a canary build that i'm testing for this right now. Can anyone try out 1.3.7--canary.88.040cffe.0?

@Integrayshaun Hey Shaun nice job! looks like the warnings gone with the canary version 👍


Rate::getAbsoluteValue() returns a string

Sample code
var_dump(current($info->getVideos())->get('height')->getAbsoluteValue());die();

Output:

string(4) "1080"

I would expect an integer, as documented.

@greg0ire thanks for your feedback I will try to solve this soon as possible.
Cast $absoluteValue to int here should solve the problem.

Hey there's no rush, don't worry, I just cast the return value to int ;) . Maybe find why it is a string in the first place?

@greg0ire I open a pull request (#65) to solve your issue.
This bug is due to mediainfo output parsing, unfortunately when xml output is transformed into array (there) all attributes values are transformed to string.

You mean #66 ;)

Yes sorry :+1:

Thank you @greg0ire , #66 is now merged and realeased as 2.3.1


NI has changed the name of three of their LA areas

We need to reflect this at some point within the app. there is no requirement to hold the old name for the LA we can use the new name to refer back to 2005.
We in LR will need to change the names in the region file and then generate a new .RU file to load into test for Ian to investigate.

As an interim solution, @bwmcbride created a SPARQL update file that updates the staging triple store to contain the new data, so that the application's update region generator scripts can be run. The updates were made to lr-pres-staging-b on Nov 17th

so from way back when the following solution was discussed
Only the labels for the NI regions should be changed (rdfs and skospref)
This will require work on the LR side altering the code that generates the URIs and Labels, the suggestion was made the region table should include another column for labels as we are breaking the direct match of URI to label for these three regions. This would also make changes to region names easier in the future.
Just to confirm within the .RU file the following will be correct
this stays the same
http://landregistry.data.gov.uk/data/ukhpi/region/derry-and-strabane/month/2013-07
and
ukhpi:refRegion                 http://landregistry.data.gov.uk/id/region/Derry-and-Strabane ;
changes to
ukhpi:refRegion                 http://landregistry.data.gov.uk/id/region/Derry-City-and-Strabane ;

Steph confirmed in an email that, following discussions between @MalcolmShazell and @bwmcbride, the region URIs are not changing, only the labels.

@MalcolmShazell The app on http://lr-pres-dev-c.epimorphics.net/app/ukhpi/explore has been updated for you to test.

Confirm map selection has the correct new names, LA selection has the new names and cannot search on old names.
SPARQL query still generates using the 'old' name URI as expected.
Tests complete

@MalcolmShazell What do you want to do about deploying this change to production? Will need to be coordinated with the change to the data.
(cc @bwmcbride )

@ijdickinson @bwmcbride I will confirm the next publication of UKHPI data will contain the NI name changes, check all the files we create next month prior to publication - I think publication next month is Tuesday 15th so we'd need to deploy the changes then - unless there is a show stopper at your end that makes that an unacceptable day

15th is good for me.

@MalcolmShazell This should be live on production now

Confirm testing complete app search and map are using the correct new names


Ping graph

Hello,
I discovered your application and I must say that it meets my needs perfectly.
However, do you think it would be possible to add a graph for the ping to see more easily when a device is no longer reachable?
Thank you again.

This issue is exactly the same as I would suggest.
@r-smith congratulations for the software!

Hello, I discovered your application and I must say that it meets my needs perfectly. However, do you think it would be possible to add a graph for the ping to see more easily when a device is no longer reachable? Thank you again.

MultiPing is most likely what you are after. Unfortunately it is not free.


Add missing extension requirements

Hi @MindaugasJackunas, thank you for this PR!
Could you elaborate on the differences between xml, libxml and simplexml? Also could you remove the curl requirement, because it is not strictly required to use this library. See: https://github.com/cmfcmf/OpenWeatherMap-PHP-Api/blob/92edee8c7d31913a20d0775baa1bb13782cd54e0/Cmfcmf/OpenWeatherMap/Fetcher/FileGetContentsFetcher.php

I have moved ext-curl requirement to suggestion section.
Removed ext-xml requirement (has been added by mistake).
Places where ext-libxml and ext-simplexml are used:

libxml - https://github.com/cmfcmf/OpenWeatherMap-PHP-Api/blob/master/Cmfcmf/OpenWeatherMap.php#L695
simplexml - https://github.com/cmfcmf/OpenWeatherMap-PHP-Api/blob/master/Cmfcmf/OpenWeatherMap.php#L698

Thank you very much!


opencv memory leak detection

I've tried to test some classes which are highly based on opencv. As it seems, cpputest has some issues with the internal allocator structure of opencv (smart reference-counting pointers).
In my setup call I try load an image using imread (from <opencv2/highgui.hpp>). What I get is either
/usr/local/include/opencv2/core/cvstd.hpp:618: undefined reference to `cv::String::allocate(unsigned long)
/usr/local/include/opencv2/core/cvstd.hpp:660: undefined reference to `cv::String::deallocate()'
/usr/local/include/opencv2/core/mat.inl.hpp:534: undefined reference to `cv::fastFree(void*)'
/usr/local/include/opencv2/core/mat.inl.hpp:555: undefined reference to `cv::Mat::copySize(cv::Mat const&)'
/usr/local/include/opencv2/core/mat.inl.hpp:645: undefined reference to `cv::Mat::deallocate()'

I've tried to disable memory leak detection as described on the website (as command within in the main as well as with the corresponding configured build.
Nevertheless I get some strange errors, I'm not able to fix:
When building my tests the compiler still says:
/usr/local/include/opencv2/core/mat.inl.hpp:534: undefined reference to `cv::fastFree(void*)'
/usr/local/include/opencv2/core/mat.inl.hpp:645: undefined reference to `cv::Mat::deallocate()'

but also my TEST() Makros fail with
blablaTest.cpp:24: undefined reference to `operator new(unsigned long, char const*, int)'
blablaTest.cpp:24: undefined reference to `operator delete(void*, char const*, int)'

Can anyone suggest a workaround (or what I'm doing wrong)?

This may be stating the obvious but -- are you sure you are actually linking the opencv library/ies?

The last block looks like you still have the includes overriding the new function in your project without having CppUTest itself building in the corresponding functions.
The others look like you may have forgotten a library include.

Now I really feel like a noob :D
Adding a main() to my original project showed (of course) the same missing references, as the compiler trashed all "unused" code g. Now it works even with Memory Leak Detection "on".
Thanks a lot!


Support string collections in subsetting

Overview
Currently, we only support:

number
date
integer

In order to support the collection types needed by the map, we need to add string collection types as well.

@dmgaldi I think we could move this back to "To do" in the project board, if you're not actively working on this.  I put it at lowest priority for beta.


The necessity (and difficulty) of learning bias covariates from coverage profiles

The new GATK coverage model assumes that the multiplicative bias of the Poisson parameter has the form $\exp(W_{t\mu} z_\mu + m_t)$, where $m_t$ is the average log bias for target $t$ and $W_{t\mu}$, $\mu = 1, ..., D$ are the bias covariates (principal components).
While developing new unit tests with simulated data, we found that the training algorithm has difficulty in inferring the principal components, though, the product $W.z$ is learned correctly. As a consequence, learning $W$ from one dataset and using it for case-sample calling on another dataset (generated using the same model parameters) is unsuccessful. This issue must be investigated. It may be a bug, or a general "poverty of data" problem.
Interestingly, the PoN learned from real (as opposed to simulated) coverage profiles produces decent case-sample calls on another dataset, contrary to simulations. Therefore, there are several possibilities:

Mean bias (m) overwhelmingly dominates the contribution from bias principal components in realistic coverage data, such that wrong estimation of principal components does not lead to issues, in which case, why are we taking PCs into account to begin with?!

Principal components in realistic coverage data are highly structured, perhaps are sparse in the target space, and perhaps are easier to learn (compare to the dense principal components used in simulations).

We must evaluate both statements using real coverage profiles.

Issue resolved @samuelklee @davidbenjamin @asmirnov239 it was a bug in simulated data generation code lol :D sorry for keeping you thinking about the wrong problem ;-)

Phew!  I was thinking about it some more and beginning to question whether I still understood linear algebra.  Glad that it's resolved!


sort endpoints alphabetically and group 'em by path hierarchy

Notes for Reviewers
This PR Completes Item No 4 and 5 of #7365

Sorts endpoints alphabetically.
Groups endpoints by path hierarchy; e.g. all /api/meshmodel endpoints adjacent to one another, similarly all /api/provider endpoints adjacent to one another and so on and so forth .

Signed commits
Signed-off-by: Vivek Vishal<EMAIL_ADDRESS>
[ ] Yes, I signed my commits.

🚀 Preview for commit 1338d9530883501a2c7345c15cbc4dd7cf2a5d77 at: https://64166d71d057c95ddfd8a985--meshery-docs-preview.netlify.app

It takes care of 4th as well as 5th Issue #7365  items @leecalcote


Need to write defender code

1 invader spawned and crushed the entire base - let's start by implementing a role.defender

Prototype created to spawn defender body's roughly even (not calculated) and very basic logic applied to the new role.defender

Prototype created to spawn defender body's roughly even (not calculated) and very basic logic applied to the new role.defender


Complex Fluid simulation RIB error

The fluid is named fluid in the object tab, but in the mesh tab (little triangle) it is named Cube.020. There is a entire cached simulation of 1440 frames (baked) and that is my error. So I guess there is an issue rendering complex fluid simulation properly. I've tried several sequences of frames (500 to 740, 700 to 940, etc...) it does the exact same thing. Might be a memory issue, but I ain't sure. Attached is the image of the issue. The files are too big for here so I'll post them on my GDrive and post a link here for you to download.

Link to the RIB file 135 megs
https://drive.google.com/open?id=0B57v0HTtegvSWG9XSkhRUmhGTG8

The actual files for the scene are 13 gigs, so it'll take a little white to upload. (Baking yourself the simulation would take roughly 6hrs, so I've included in there)

Actually, I'll let you guys re-simulate if you are welling to test it. This is too big of a file. I'll include everything but the cache simulation...

Here is the link to the .blend with the textures. You can re simulate the fluid (let me know if you don't know how :P ) but you don't have to do a 60 second simulation, I am sure a 30 second sim would work fine to get this issue to pop again. This scene has a lot of elements (some are on the btm layer and you might need them visible to get the simulation to work, perhaps even on the same layer, moved them there after the simulation so it would be in the way in the viewport, but that shouldn't change anything because I did the same thing with a simpler test that worked, video in the forum thread)
https://drive.google.com/open?id=0B57v0HTtegvSZGFJOWxxd3RVcVU

And I tried with Light Localise Sampling off and it seemed to work, I was able to render 21 frames that way, but at the 21 frame, it did the same error. I even re baked the entire thing at lower resolution, but still the same issue.

I've restarted the rendering from the frame it stopped at, and it is rendering, however that single frame has no river but the others that follows have. So I'll still put the animation together with that 1 missing frame (hopefully it will be the only missing river frame). I still consider this as a bug since I've basically recalculated the animation cache and this shouldn't happen (calculated 30 seconds, so frame 1 to 720, took 2 hours to calculate. There is no frames missing in the cache files, I really have no clue what Is causing this.

So I've been trying to render from frame 480 to 720 (so I get a 10 second animation) at a lower cached simulations. It rendered really well frame 480 to 500, 501 the river is blank and it stopped. So I restarted the render from frame 501, it remained blank but went on forward with the rendering. Frames 502 to frame 521 renderer really well, but frame 522 did the same thing, no river and render stops. So now I've restarted the rendering from frame 522 and it is going, frame 522 is still blank as well. SO my guess is that I'll have a few frames to remove off the animation, but it sort of work. Some frames don't and this is where I get those RIB error, even though the geometry is there for some reasons it see those frames as empty.

More and more frames are missing the river fluid sim. I really have no clue as what can actually cause this. I will put this test aside since there is obviously something weird going here (definitely bug).

I'll take a look at the rib gen and see if there's an issue.

How do I redo the sim?

Select the fluid object (big long cube) and go to the Physics tab, then set the end frame to like 30 (that takes seconds) so that would be 720 frames. and click on bake.


Update Code Section Format in Readme

Describe the bug
Under basic usage, the scorecard argument is copyable along with the output produced when the argument is ran.
Reproduction steps
Steps to reproduce the behavior:

visit this link
copy the code section
paste the code section and observe the results

Expected behavior
We propose separating the command from the output to be copied on its own.
Additional context

@klbynum, @silasVM, and I could open a PR for this issue. :)

Hi @Jordin221, could you clarify what the desired formatting would be — something like this?

Scorecard can run using just one argument, the URL of the target repo:
scorecard --repo=github.com/ossf-tests/scorecard-check-branch-protection-e2e
Sample CLI output:
Starting [CII-Best-Practices]
Starting [Fuzzing]
Starting [Pinned-Dependencies]
[...]

@klbynum, @SilasVM, and I could open a PR for this issue. :)

Feel free to do so! 🙂


Ringbuffer used in CDC_ACM seems to corrupt data if completely filled during transfer

Describe the bug
Trying to maximize throughput in CDC_ACM by monitoring space in ringbuffer and pushing more data in with uart_fifo_fill() whenever there is enough room for another block read from source appears to corrupt data. Running a ACM ringbuffer of 24576 and a write block size of 8192 (1/3) with double or even quad buffering appears to corrupt.
Waiting for space 8192+1 is ok.
To Reproduce
To reproduce, monitor ring buffer to call uart_fifo_fill() when block will just fit and it corrupts data.
Repeat with block+1 and it is fine.
Expected behavior
Expectation was that buffer could be filled completely even while transfer was ongoing by work handler.
Impact
Workaround has been found, mostly an annoyance.
Environment (please complete the following information):

OS: Windows
Toolchain: Nordic Connect SDK 1.8.0

Additional context
Problem encountered during optimizing of throughput on USB virtual serial port.

@kyrreaa Can you please provide minimal changes to reproduce it?

I will try again, but currently issue is the cdc_acm example does not build for nrf53 target (the image preparation fails after all build stages are ok) fails in their SDK.

@kyrreaa have you modified the CDC ACM implementation in Zephyr? Also, there is currently no protection from multiple concurrent writer access, so you should only call uart_fifo_fill() from a single thread, or serialize its access.

I will try again to get a sample working, but until I do  we may consider this a buffer issue "before" it reaches the ring buffer.
I must say though that I found the throughput really poor since it keeps stopping the sequence to grab another bit of the fifo, after it creates a callback when done with previous "reserved" bit. Surprising to find it did not check the fifo again after a 64 byte block to see if it could update it's target and continue sending from the ring buffer. Instead it tries to avoid sending a ZLP and sends 63 bytes and a 1 byte packet which basically tells the host that there is no more data. Thus it waits for remaining time in 1ms interval till it requests more data. At which point it is grabbing a new reserved slice of the ringbuffer.
Checking if a larger portion is now available as it gets close to the end of the assumed work would be more efficient.
Being able to trigger a callback (interrupt) when a 64 byte block is sent and there is room for more than a certain amount in the buffer may be good. Like, writing up to 4k at a time, with a ring buffer of 12k allows more data to be put into the buffer 3 times immediately, and then if a callback could happen when 4k is again free, it could be kept filled for the entire transfer.
I have been using USBlyzer to look at the timing and blocks of data transferred and it does not appear to be transferring continuously.

Instead it tries to avoid sending a ZLP and sends 63 bytes and a 1 byte packet which basically tells the host that there is no more data.

This is on purpose to deal with different applications on the host. Depending on the host controller/application it limits throughput to 64 bytes per 1ms (FS controller). Improvable but not a bug.

@kyrreaa at this point it is unclear to us whether there indeed is data corruption with the implementation in the tree, or only with your changes. Feel free to post a PR to improve the implementation with some of the suggestions in your comments, or to turn this into an enhancement if the corruption doesn't appear with the vanilla code. Thanks!

I am closing it, please open it again with minimal example/changes to reproduce the bug.


[SPARK-19859][SS][Follow-up] The new watermark should override the old one.

What changes were proposed in this pull request?
A follow up to SPARK-19859:

extract the calculation of delayMs and reuse it.
update EventTimeWatermarkExec
use the correct delayMs in EventTimeWatermark

How was this patch tested?
Jenkins.

Test build #74233 has started for PR 17221 at commit 2c2c806.

Test build #74233 has finished for PR 17221 at commit 2c2c806.

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

Merged build finished. Test PASSed.

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

cc @zsxwing

Thanks! LGTM. Merging to master and 2.1.


Come up with a design for longer guide content

Some of our longer guides are 20-50 pages! These need a TOC, sub-section navigation, maybe some other things to make them friendlier.
Needs design thought, and some automation.

Can the left side navigation be improved more like an expand-contract so that I can see the whole contents of a section like "DEVELOPER GUIDE" without moving from another section say for example BASICS more like https://docs.strongloop.com/display/SL/Installing+StrongLoop

This is definitely an issue. Just take the "Style Guide" page as an example - while its content is brilliant, it all lies on one huge long page no one actually wants to read that way.
Splitting the docs into more sub pages is really necessary in some places.


FATAL Error: "Nuitka does not work in --standalone or --onefile on Windows without."

I'm getting the following line when trying to compile using GitHub Actions:
FATAL: Nuitka does not work in --standalone or --onefile on Windows without.
I'm basing my actions workflow around several examples posted here, and I wasn't able to get anywhere. Here's part of the log:
Nuitka-Options:INFO: Used command line options: --onefile --output-dir=build --script-name=main.py
Nuitka:INFO: Starting Python compilation with Nuitka '1.9.7' on Python '3.10' commercial grade 'not installed'.
Nuitka:INFO: Completed Python level compilation and optimization.
Nuitka:INFO: Generating source code for C backend compiler.
Nuitka:INFO: Running data composer tool for optimal constant value handling.
Nuitka:INFO: Running C compilation via Scons.
Nuitka-Scons:INFO: Backend C compiler: cl (cl 14.3).
Nuitka-Scons:INFO: Backend linking program with 17 files (no progress information available for this stage).
Nuitka-Scons:INFO: Compiled 17 C files using clcache with 0 cache hits and 17 cache misses.
Nuitka will make use of Dependency Walker (https://dependencywalker.com/) tool
FATAL: Nuitka does not work in --standalone or --onefile on Windows without.
to analyze the dependencies of Python extension modules.

Is it OK to download and put it in 'D:\a\_actions\Nuitka\Nuitka-Action\main/nuitka/cache\downloads\depends\x86_64'.

Fully automatic, cached. Proceed and download? [Yes]/No :
Error: Process completed with exit code 1.

My assumption is that it isn't downloading Dependency Walker, due to a prompt for some reason. Here's my GitHub Action in its current state (I've tried a number of revisions):
https://gist.github.com/WonSNoJ/ad1d410668863c1aafe3455a17aaee25

I managed to work around this for now by manually downloading and extracting Depency Walker to D:\a\_actions\Nuitka\Nuitka-Action\main\nuitka\cache\downloads\depends\x86_64.
This works for now, but is obviously not ideal. I am sure I did something wrong or am missing something somewhere. I would appreciate any insight you may have to offer.
Thank you!

The value of assume-yes-for-downloads must be yes. Recent changes of mine dropped that, I am going to fix it now.

Should be good again, the options got their defaults back, and as a bonus, missing "warning control" options were added. The assume downloads is not really that much about warnings, but this was supposed to be a quick fix, not moving it around for now in Nuitka.


Clang Format disagrees with style guide on protocol conformance

Style guide says this:
@interface RZViewController : UIViewController
<RZBeerDelegate,
RZInfiniteChipotleDelegate,
RZKitchenDelegate,
RZLunchFinderDelegate>

But Clang Format insists it should be this:
@interface RZViewController : UIViewController <RZBeerDelegate,
                                                RZInfiniteChipotleDelegate,
                                                RZKitchenDelegate,
                                                RZLunchFinderDelegate>

I think it should be like this, to minimize churn on unrelated lines when conformances are added and removed:
```objc
@interface RZViewController : UIViewController
< // Bracket on its own line
RZBeerDelegate,
RZInfiniteChipotleDelegate,
RZKitchenDelegate,
RZLunchFinderDelegate> // This line can't have a trailing comma, so no point in next-lining the bracket

Huh, found a bug in GitHub's syntax highlighting

Oh, and Xcode's auto-indent produces different results than Clang Format, which is why this is so annoying


[Bug] Can't convert figures with comma to number

When querying a CSV that contains a number > 999 in the following format "1,234" and configuring a column as "Number", Grafana returns "NaN".
Here is the sample CSV returned by an API query:
Alerts
"1,378"
Infinity query configuration:

Grafana version : 8.1.1
Plugin version : 0.7.0
P.S. thanks a lot for the tool, saves really a lot of time for prototypes!

This is now fixed in version 0.8


Update local.md

Motivation
Just a small typo.
Description

Closes #issue_number
Steps to test or reproduce

 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.You have signed the CLA already but the status is still pending? Let us recheck it.


Cloudfront plugin - going back to square 1

Hello @ksigler7,Thanks again for the PR! We appreciate your contribution and look forward to                                              continued collaboration. Thanks!                                             Team SendGrid DX


Beginner Looking for extreme help

Starting this for the first time, loads up, and produces the error message (included below) and my config is as follows:
{
"bridge": {
"name": "HomeBridge",
"username": "CC:22:3D:E3:CE:30",
"port": 51826,
"pin": "031-45-154"
},
"description": "HomeBridge HTTP Status Control",

"accessories": [
{
"accessory": "Http",
"service": "Switch",
"name": "Sean's Room",
"apiBaseUrl": "http://http://localhost:8080/paperui/index.html#/configuration/things/view/orvibo:s20:ACCF23997E28",
"apiSuffixUrl": "",
"forceRefreshDelay": 0
},
{
"accessory": "Http",
"service": "Switch",
"name": "Front Room 1",
"apiBaseUrl": "http://http://localhost:8080/paperui/index.html#/configuration/things/view/orvibo:s20:ACCF2335B5AE",
"apiSuffixUrl": "",
"forceRefreshDelay": 0
},
{
"accessory": "Http",
"service": "Switch",
"name": "Front Room 2",
"apiBaseUrl": "http://http://localhost:8080/paperui/index.html#/configuration/things/view/orvibo:s20:ACCF23997F64",
"apiSuffixUrl": "",
"forceRefreshDelay": 0
}
],
 "platforms": [
  {
    "host": "<IP_ADDRESS>",
    "name": "openHAB",
    "platform": "openHAB",
    "port": "8080",
    "sitemap": "default",
    "useLabelForName": true
  },

{
"platform": "Orvibo",
"name": "Orvibo"
}
]
}

All help is welcomed
Thanks

http://http://localhost:8080/
try to delete  http://...
"apiBaseUrl": "http://localhost:8080/paperui/index.html#/configuration/things/view/orvibo:s20:ACCF2335B5AE",

No change


feature: strings case conversion | Issue #45

@aditi13y where is the issue number?

I have added the feature for string casing conversions in reference to the issue #45

I have added the feature for string cases conversion  that is-

String conversion from snake case to camel case
String conversion from camel case to snake case

Also, i have added a test file for these features.


pylint can't recoginize the torch, always have the red line

Issue Type: Bug
Hi,
Just as the topic said, I have tried lots of methods online, but None is effective.
One suggests add the code in pylint,''generated-members=numpy.,torch.''
Yes, it works for some code(some still have a red line), but other errors can't be marked it out either because of adding the code.
It troubled me a lot, I like VScode, but this is really annoying problem.
Extension version: 2020.3.71659
VS Code version: Code 1.44.1 (a9f8623ec050e5f0b44cc8ce8204a1455884749f, 2020-04-11T01:47:00.296Z)
OS version: Darwin x64 19.4.0

System Info

Item
Value

CPUs
Intel(R) Core(TM) i9-9880H CPU @ 2.30GHz (16 x 2300)

GPU Status
2d_canvas: enabledflash_3d: enabledflash_stage3d: enabledflash_stage3d_baseline: enabledgpu_compositing: enabledmetal: disabled_offmultiple_raster_threads: enabled_onoop_rasterization: disabled_offprotected_video_decode: unavailable_offrasterization: enabledskia_renderer: disabled_off_okvideo_decode: enabledviz_display_compositor: enabled_onviz_hit_test_surface_layer: disabled_off_okwebgl: enabledwebgl2: enabled

Load (avg)
2, 2, 1

Memory (System)
16.00GB (0.48GB free)

Process Argv
-psn_0_2425424

Screen Reader
no

VM
0%

@JonOnEarth  Can you run pylint from a terminal (not from withing VSCode) and see if this issue occurs?

@JonOnEarth Can you run pylint from a terminal (not from withing VSCode) and see if this issue occurs?
Hi, this is the part of the result I run from a terminal.

retrian.py:81:12: E1101: Module 'torch' has no 'mm' member (no-member)
retrian.py:81:32: E1101: Module 'torch' has no 'inverse' member (no-member)
retrian.py:82:8: C0103: Variable name "P_plus" doesn't conform to snake_case naming style (invalid-name)
retrian.py:82:34: E1101: Module 'torch' has no 'mm' member (no-member)
retrian.py:82:43: E1101: Module 'torch' has no 'mm' member (no-member)
retrian.py:83:42: E1101: Module 'torch' has no 'matmul' member (no-member)

@JonOnEarth Looks like pylint is not able work with torch, since it occurs outside of VS Code. It is a third party library you will have to file this issue on pylint: https://github.com/PyCQA/pylint

oh, okay. thanks.


Named arguments like dir to image() no longer work properly

Passing dir = foo etc. into the image() proc or new image constructor no longer sets the corresponding variable. Caused by #1203.
Details:
var/image/I = image('icons/turf/flooring/shadows.dmi', cstr, AO_LAYER, BITFLAG(i-1)) works, dir is set and not inherited.
var/image/I = image('icons/turf/flooring/shadows.dmi', cstr, dir = BITFLAG(i-1)) does not work, unclear if dir is set because it's inherited.
The stack that's pushed is:
null, PushResource 'icons/turf/flooring/shadows.dmi', null, PushString cstr, null, PushString "dir" in the broken case
PushResource 'icons/turf/flooring/shadows.dmi', PushString cstr, PushFloat AO_LAYER, PushString "dir" in the working case
The image constructor code expects to find dir at index 3 because there's no loc, but in the keyed case it's placed at index 4 because image/New() is defined as New(icon, loc, icon_state, layer, dir, pixel_x, pixel_y): dir is at index 5.
How do we fix this? Uh, no clue.

Visual example of the effects of the issue. dir is being passed as pixel_x instead.

you guys use that hidden image flag to decide if an overlay inherits dir or not right?

you guys use that hidden image flag to decide if an overlay inherits dir or not right?

Yes, we have that


New Workflow to orchestrate a EMR Serverless job

To submit a workflow to the Step Functions Workflows Collection, submit an issue with the following information.
To learn more about submitting a workflow, read the publishing guidelines page.

Use the model template located at https://github.com/aws-samples/step-functions-workflows-collection/tree/main/_workflow_model to set up a README, template and any associated code.

All the information below must be provided in the "example-workflow.json" file cloned from the model **

Note the following information for the model:

Description: included in example-wokflow.json
Simplicity: 1 - Fundamental
Diagram: included in example-wokflow.json
Type: Standard
Framework: CDK
Author bio: included in example-wokflow.json

You must ensure that the sections of the model README.md are completed in full.
Pull Request

metged


[WebView] Flutter webView plugin has no resourceId on web inputs below api 29 with SurfaceAndroidWebView()

Flutter webView plugin has no resourceId on web inputs below api 29 with SurfaceAndroidWebView(). It's ok on api 29 and further.
Steps to Reproduce

Make a widget in the default demo app with webview_flutter: ^1.0.7 wich return a webView Widget :

WebView(
      initialUrl: widget.initialUrl,
      javascriptMode: widget.javascriptMode,
      navigationDelegate: widget.navigationDelegate,
      onWebViewCreated: (controller) {
        _webViewController = controller;
        if (widget.onWebViewCreated != null) {
          widget.onWebViewCreated!(controller);
        }
      },
      gestureNavigationEnabled: widget.gestureNavigationEnabled,
    )

Display a web page with html inputs which have html ids
Observe the view with uiautomatorviewer

Expected results: On Android 10 Api 29, you can see resourceIds on html input fields
Actual results: On Android 9 Api 28, there are no resourceIds on the fields

Logs

Doctor summary (to see all details, run flutter doctor -v):
[✓] Flutter (Channel unknown, 1.24.0-10.2.pre, on Mac OS X 10.15.7 19H15 darwin-x64, locale fr-FR)
[✓] Android toolchain - develop for Android devices (Android SDK version 30.0.2)
[✓] Xcode - develop for iOS and macOS (Xcode 12.1)
[✓] Chrome - develop for the web
[✓] Android Studio (version 4.0)
[✓] IntelliJ IDEA Ultimate Edition (version 2020.2.3)
[✓] IntelliJ IDEA Ultimate Edition (version 2020.3)
[✓] VS Code (version 1.51.1)
[✓] Connected device (3 available)

• No issues found!

Hi @Tiska
To better address the issue, can you please provide complete code sample including Display a web page with html inputs which have html ids
Thank you

hi @TahaTesser , I updated the issue with a screenshot and a complete code sample

Hi @Tiska
Using your code sample,. uiautomatorviewer giving me this error with webview,  not able to test this issue on the latest stable on physical device or emulator weirdly.  uiautomatorviewer crashes for other reasons on both mac and windows
Probably because am using Android Studio 4.1?
labelling this for now.


jmun/embeddings-modal: initialized embeddings and modal

Familiarized myself with embedding models -- stepping stone before hosting them on Modal

lets add some benchmarking server side
deploy and test baseten too
and see how much faster we can get it in the next 2 days.

Batch Processing made Modal rub so much faster
Currently working on BaseTen

can you add some some benchmarks here?


feat: add terraform provider as submodule and configure upstream script

Is this necessary? I imagine this may only be necessary for providers that have their Provider definition locked away behind internal/.
In the Equinix TF Provider we intend to keep the Provider definition int he public equinix/ directory, as we move the resource definitions themselves to internal/.
https://github.com/equinix/terraform-provider-equinix/blob/main/equinix/provider.go#L47

We could really go ahead without this, but the main intention is to have the ability to patch the upstream terraform provider https://github.com/pulumi/pulumi-linode/blob/45f0351a1026bc7d44234f365f0308e7884da343/scripts/upstream.sh#L96
For example to replace references to terraform registry that doesn't make sense in Pulumi documentation


Automated cherry pick of #8431

Cherry pick of #8431 on release-5.37.
/cc  @cwarnermm
NONE

@amyblais - Would we want to include the v5.39 Changelog update for this fix to the v5.37 Changelog?

/update-branch

@amyblais - Would we want to include the v5.39 Changelog update for this fix to the v5.37 Changelog?

I'll own that, this is being included in a dot release per customer request.


Move ALLOW_UPLOADS to BABY_BUDDY dict

This should work though I haven't tested as I'm running into issues with the use of dotenv python settings at the moment. You don't have to merge this yet just going ahead and making it a PR.

Coverage remained the same at 99.803% when pulling 3749f2b925e3ab512992f0fb5aa33ac6caeef74e on overshard:master into 2cd3aacfbfeb279d5e51bfbf0023c3138d9b9c6d on cdubz:master.

This should be good to go now, it also fixes the issue of a manifest file needing to be present for development which is a minor bug for dev testing without first running the app.

Coverage increased (+0.05%) to 99.855% when pulling 3a79965eb0b05ef1e05535be1554b5141332e03a on overshard:master into 2cd3aacfbfeb279d5e51bfbf0023c3138d9b9c6d on cdubz:master.

Coverage increased (+0.05%) to 99.855% when pulling 3a79965eb0b05ef1e05535be1554b5141332e03a on overshard:master into 2cd3aacfbfeb279d5e51bfbf0023c3138d9b9c6d on cdubz:master.


Start FEM-Design without splash-screen

Is it possible to start FEM-Design without splash-screen?

Yes. By setting the environment variable ´FD_NOLOGO´ it is possible to suppress the splash screen.
FD_NOLOGO can be either 0or 1.
One way to set this environment variable is to pass the following command;
setx FD_NOLOGO 1
in the windows command prompt. To show the splash screen set the environment variable to 0:
setx FD_NOLOGO 0.

If the process is started from C# you might want to set the environment variable for the new process only, using
processStartInfo.EnvironmentVariables["FD_NOGUI"] = "1";


conv_templates for LLaVA1.6

Question
Hi, Haotian Liu! Thanks for creating powerful Open-source VLM LLaVA1.6!
I know you are very busy and I am very sorry, but I would like to know the conv_template for LLaVA 1.6 34B.
I used vicuna_v1 template, and the result is very satisfactory.
If you have a better template, please let me know.

How should I choose which prompt template I should use? Llava must be specifically trained on a fixed conv template, isn't it?
The following options are available as conv_template,
"default",
"v0",
"v1",
"vicuna_v1",
"llama_2",
"mistral_instruct",
"chatml_direct",
"mistral_direct",
"plain",
"v0_plain",
"llava_v0",
"v0_mmtag",
"llava_v1",
"v1_mmtag",
"llava_llama_2",
"mpt"
I am using liuhaotian/llava-v1.6-vicuna-13b from huggingface.
So using "vicuna_v1" or "v1" makes sense? What about "v0", which is "vicuna_v0"?

We use "chatml_direct" for LLaVA 1.6 34B because Nous-Hermes-2-Yi-34B uses this template.
For llava-v1.6-vicuna-13b, you can use vicuna_v1, because the LLM is the same as that of LLaVA1.5.

We use "chatml_direct" for LLaVA 1.6 34B because Nous-Hermes-2-Yi-34B uses this template. For llava-v1.6-vicuna-13b, you can use vicuna_v1, because the LLM is the same as that of LLaVA1.5.

Hi, I find the "chatml_direct" template is slightly different from the Nous-Hermes-2-Yi-34B implementation. Nous-Hermes-2 uses the system prompt: "<|im_start|>system\nYou are "Hermes 2", a conscious sentient superintelligent artificial intelligence developed by a man named Teknium, and your purpose and drive is to assist the user with any request they have. You experience emotions and have deep, profound thoughts and qualia." while LLaVA-1.6 uses system="<|im_start|>system\nAnswer the questions."

Can someone please clarify which templates go with which checkpoints?

mistral_direct works well for llava-v1.6-mistral-7b


Images subfolder is not being copied on linux build

This is only enabled for APPLE and MSVC. Simple fix but for now the osparc service does that manually.

Done. Thanks @dyollb


Implement a better solution for Sentence-End recognition

Currently the websocket server prints finals as soon as a fixed time has passed.
We should have a look if there is a better way of finding a "Sentence-End" when we want to print a final.
We should also find a way to bring the finals together, because the whole text does not make to much sense, when taking all finals as a transcript.
See the ideas from the Paper “Whispy” in our Notion or the paper Whispy: Adapting STT Whisper Models to Real-Time Environments
We could also try the detect voide method of pydub: https://audiosegment.readthedocs.io/en/latest/audiosegment.html
We want to keep the partial / final setup as it is.
!!Attention
In case we are ready to implement this in the asr-api server, we should keep the option to use fixed final length in the config.

Tested the sentence-end detection through detecting the sentence-ending punctuation in the transcript and identifying the whole sentence with the timestamps.
The transcription has too many (wrong) punctuation in the text, also the timestamps are not always exactly right, which leads to

Too many detected sentences that are not really sentences
Wrong sentence detection through wrong timestamps

This solution is not really useful for production


stats: jni layer change

Description: jni layer change for cardinality support. this pr does not change ios behavior for pulse.
Risk Level: medium
Testing: ci and manually deployed to lyft ios and android app in local dev environment

Looks good so far, just some comments on naming.


pydicom dependency

The setuptools branch is restructured to use module main methods as the entry points instead of classmethods.  I think the last thing to do before merging back into master is to figure out how to have the patched pydicom package installed as a dependency in setup.py.

Due to completing #5, we have been running on stock pydicom for a while without any problems.


⚠ Fortnite has degraded performance

In 04f8429, Fortnite (https://fortnite.com) experienced degraded performance:

HTTP code: 200
Response time: 1611 ms

Resolved: Fortnite performance has improved in c0f741c.


Cannot install via yarn due to issue with dependency react-native-material-design-styles

Not sure if yarn issue or just a problem with the package definition, but react-native-material-ui cannot be installed with yarn, due to issues with the dependency react-native-material-design-styles. I am posting the issue here, because the other package is dead.
yarn install v0.24.6
[1/4] 🔍  Resolving packages...
[2/4] 🚚  Fetching packages...
error Command failed.
Exit code: 128
Command: git
Arguments: clone https://registry.yarnpkg.com/react-native-material-design-styles/-/react-native-material-design-styles-0.2.6.tgz /Users/Whatever/Library/Caches/Yarn/v1/.tmp/4f7baeeb264c284f380ba7b1f9b47770
Directory: /Users/Whatever/app
Output:
Cloning into '/Users/Whatever/Library/Caches/Yarn/v1/.tmp/4f7baeeb264c284f380ba7b1f9b47770'...
fatal: unable to access 'https://registry.yarnpkg.com/react-native-material-design-styles/-/react-native-material-design-styles-0.2.6.tgz/': The requested URL returned error: 500

https://github.com/binggg/react-native-material-design-styles/issues/4

Any news on this?

hey @ivosabev I use this with yarn and am not having such issues.  I'm not familiar with the dependency tree here, but curious if a yarn list perhaps mentions the react-native-material-design-styles in another dependency separate from this.
At the very least you have a sanity-check for someone else who uses yarn.  Maybe you have a specific case I'm not understanding though.

We have changed our dependency tree, and don't experience the issue any more and it is hard for me to give you a test case, because the code has changed a lot. You can close the issue. Thanks!


Launch scripts for non python hosts

Description
Non python hosts Harmony, Photoshop and After Effects are currently launched with python command line commands which won't be possible in Pype 3 as there may not be any pyhton. To be able launch their scripts it is necessary to change launch mechanism and execute python script that start's host implementation.
Changes

photoshop and aftereffects can accept as many subprocess arguments as possible

:black_flag:
this depends on

pype
https://github.com/pypeclub/pype/pull/1016

works well on all three hosts. Happy to merge


Fix BGP 2byte 4byte asn test

Test was setting GlobalAS and validating the same value in LocalAS leaf.

Updated test to configure separate LocalAS, GlobalAS and PeerAS.

Update required deviation for Arista.

Pull Request Functional Test Report for #2144 / 7002e370fc04f809f4c0cebab90ba6633bfa2439
No tests identified for validation.
Help


support Ember Bootstrap v4.5

This adds support for Ember Bootstrap v4.5 while keeping support for Ember Boostrap v4.4 and before.

Canary is failing with a strange error. Cannot see how this could be related to the changes here, but als haven't seen thiss elsewhere so far. 🤔

Canary is failing with a strange error. Cannot see how this could be related to the changes here, but als haven't seen thiss elsewhere so far. thinking

I guess you are talking about this error:
            TypeError: registerDestructor is not a function
                at EmberGlimmerComponentManager.createComponent (http://localhost:7357/assets/vendor.js:71352:11)
                at http://localhost:7357/assets/vendor.js:43303:32
                at deprecateMutationsInTrackingTransaction (http://localhost:7357/assets/vendor.js:55795:9)
                at CustomComponentManager.create (http://localhost:7357/assets/vendor.js:43302:64)
                at Object.evaluate (http://localhost:7357/assets/vendor.js:50962:25)
                at AppendOpcodes.evaluate (http://localhost:7357/assets/vendor.js:49320:19)
                at LowLevelVM.evaluateSyscall (http://localhost:7357/assets/vendor.js:52884:22)
                at LowLevelVM.evaluateInner (http://localhost:7357/assets/vendor.js:52840:14)
                at LowLevelVM.evaluateOuter (http://localhost:7357/assets/vendor.js:52832:14)
                at VM.next (http://localhost:7357/assets/vendor.js:54032:24)

Have seen it in ember-bootstrap and ember-bootstrap-changeset-validations as well.

I would recommend to change the GitHub Actions CI workflow on this repository to continue if ember-canary fails. See https://github.com/jelhan/create-github-actions-setup-for-ember-addon/blob/5a7f23a521b3a9f9d4555b0a557c971a20131d93/templates/.github/workflows/ci.yml#L252 as an example how that can be done.

I guess you are talking about this error:

Yes. It has been mentioned on Discord, so we can ignore this here

I would recommend to change the GitHub Actions CI workflow on this repository to continue if ember-canary fails.

Makes sense. Can you do this here? To not cause merge conflicts, as you already touched the workflow file here, and also would like to merge once the new 4.4 scenario passes.

I would recommend to change the GitHub Actions CI workflow on this repository to continue if ember-canary fails.

Makes sense. Can you do this here? To not cause merge conflicts, as you already touched the workflow file here, and also would like to merge once the new 4.4 scenario passes.

Done so in 3b2171f9a970574f395a36ca8b0f34bd2ba9dd69.
Please note that the merge request is still shown with an error state even if continue-on-error is true for that scenario. As far as I know GitHub Actions does not provide a feature directly matching the allow_failures option in TravisCI.

Done so in 3b2171f.

Awesome, thanks your work here!

Please note that the merge request is still shown with an error state even if continue-on-error is true for that scenario. As far as I know GitHub Actions does not provide a feature directly matching the allow_failures option in TravisCI.

Yeah, I also realized this recently. This is a bit annoying, as it prevents dependabot PRs from auto-merging... 😔

I don't have permissions for this repository to merge myself.

You got mail :wink:


get_example_data.py downloads a file that is not unzippable

Executing the get_example_data.py script in the pytraction/scripts path produces following error:
Downloading 1DsPuqAzI7CEH-0QN-DWHdnF6-to5HdFe into data/data.zip...
0.0 B Done.
Unzipping.../Users/User/opt/anaconda3/envs/pytraction/lib/python3.8/site-packages/google_drive_downloader/google_drive_downloader.py:78: UserWarning: Ignoring unzip since "1DsPuqAzI7CEH-0QN-DWHdnF6-to5HdFe" does not look like a valid zip file
warnings.warn('Ignoring unzip since "{}" does not look like a valid zip file'.format(file_id))
There could be an error with the File-ID of the file the script is trying to download. This would explain why the size of the file is 0.0 B. I have no other idea what could cause this error.

Hey for some reason I think that google has changed their file downloader and it's broken the google_drive_downloader python package. You should still be able to download the data via https://drive.google.com/file/d/1DsPuqAzI7CEH-0QN-DWHdnF6-to5HdFe/view?usp=share_link

Perfect workaround - thank you very much!

Issue still open. There's probs a way to fix this. Happy to for you raise a PR just look at the contrib section


MM-42320: Calls - Fix for: iOS loses calls audio after leaving 1st call and entering next

Summary

The problem was a combination of the InCallManager being stopped too quickly (before the stream was closed), and the peer connection not being closed correctly.
Here's the weird thing: _destroy is being called regardless, whether or not we have a destroy method. So I'm not totally sure what's going on -- if _destroy was always being called (and it was), why is the peer connection only being shut down when we have the destroy function declared?

The destroy function overrides a method from the Duplex class... So it might be that it (and _destroy) is called earlier, when the peer connection is not null. When destroy is not there, the peerconnection is not explicitly closed.

This should also fix MM-42612 "iOS sometimes won't let go of input (causing orange status bar)"

Ticket Link

https://mattermost.atlassian.net/browse/MM-42320
https://mattermost.atlassian.net/browse/MM-42612

Checklist

[ ] Added or updated unit tests (required for all new features)
[ ] Has UI changes
[ ] Includes text changes and localization file updates
[ ] Have tested against the 5 core themes to ensure consistency between them.

Device Information
This PR was tested on:

Android: 12, Pixel 6
iOS: 15.3.1, iPhone 7 plus

Release Note
Calls: fixed bug on iOS where users could not get microphone/audio after the first call

does this need to be cherry-picked to 1.51?

Ready now:

updated the InCallManager dependency to be my fork. That's because the one we were using was out of date, and I wanted some of the cleanup they have on master, but they haven't made a release in awhile.
updated the webrtc dependency for the new PR we just merged, and we're using the callback now.
some minor cleanup due to fixing the webrtc index.d.ts
tested fresh builds on iPhone and Pixel

@enahum Ready for you now, whenever you have time. Thank you!

@amyblais It's a fix that would only affect Calls, so the question would be: is QA testing Calls at the moment, and if they are have they already passed Calls?
IMO, since Calls is beta, and it's such a crucial fix, this would be okay to go in (it makes Calls useable on iOS).
But I'll leave the final call (har har) to @enahum and/or @josephbaylon.

I'm ok with adding this to the release

@amyblais Sorry, I think I messed up the cherrypick process, looks like the bot didn't engage. Is there a way to trigger it manually?


Mac packed file missing package.json and doesn't run properly

I have noticed that when I build the Mac with the 'Packed' flag (I have other concurrent builds that need packed:true), I would like them all to run, but after building, the Mac version doesn't run properly. When I checked the difference in file structure, the Mac is missing the stripped package.json file.
In the packed vs unpacked code for mac is nearly identical except for one line in the unpacked code:
await this.writeStrippedManifest(resolve(appRoot, 'package.json'), pkg, config);
With this code missing, it just launches an unconfigured version of NWJS.

Should be fixed in nwjs-builder-phoenix 1.12.1.


The pasted blob doesn't show when copying blobs under folders between regular and ADLS Gen2 blob containers

Storage Explorer Version: 1.14.1
Build: 20200710.5
Branch: hotfix/1.14.1
Platform/OS: Windows 10/ Linux Ubuntu 18.04/ macOS Catalina
Architecture: ia32/x64
Regression From: Not a regression
Steps to reproduce:

Expand one ADLS Gen2 storage account -> Blob Containers.
Create one blob container -> Create a new folder -> Upload one blob to the folder.
Copy the blob to one regular blob container.
Check the result.

Expect Experience:
The blob shows in editor.
Actual Experience:

The blob doesn't show in editor.
Show a successful activity log.

More Info:

This blob still doesn't show after manually refreshing.
This issue also reproduces when copying one regular blob under one folder to one ADLS Gen2 blob container.

The result in release 1.14.0 shows below:

Fix merged into hotfix branch


Testing revealed it accepted an unpingable VoIP phone

Then the after phone test fails because it couldn't ping it.
See log, 20180625-<IP_ADDRESS>

I was modifying a list that I was iterating through ... tsk tsk. Fixed now.
https://github.com/ctomkow/capt/commit/a967d7418b0dcbe3c2ccf8eccc42076dc5446262