Future of requiring basisOfRecord and controlled vocabulary

A new recommendation has been put forward to modify basisOfRecord which (I believe) would result in changes to the controlled vocabulary currently in place for basisOfRecord in the IPT. In order to publish data via the IPT you are required to include basisOfRecord and the values in that field must be one of these exactly: PreservedSpecimen, FossilSpecimen, LivingSpecimen, MaterialSample, Event, HumanObservation, MachineObservation, Taxon, Occurrence, MaterialCitation (e.g. "Human Observation" will cause a dataset to be rejected).
Will the IPT continue to require basisOfRecord? Will the controlled vocabulary be updated to follow the suggested changes?
@timrobertson100

Assuming that recommendation does go through in the next edition of DwC:

Will the IPT continue to require basisOfRecord?

Yes, I would expect as the reason it was first made a required field remain valid (i.e. the need to have a basic understanding of what the record actually represents)

Will the controlled vocabulary be updated to follow the suggested changes?

Yes, although I don't imagine that this would impact things as much as one might think. I'd anticipate that:

On data publishing, the IPT would handle a spreadsheet containing preserved_specimen, PRESERVED_SPECIMEN, 'preservedSpecimen' etc and map that onto the correct term.

When the IPT produces the output archive, the value it puts out for that field might change the format slightly. I doubt many consumers would notice, as it's fairly common to consume this stuff in a case insensitive manner ignoring whitespace and _ characters.

Thanks Tim!

On data publishing, the IPT would handle a spreadsheet containing preserved_specimen, PRESERVED_SPECIMEN, preservedSpecimen etc and map that onto the correct term.

The problem is that this is not currently how the IPT works. If you have anything but "PreservedSpecimen" it will be rejected. Meaning you either have to update the file so the data in it matches that exactly or create a translation.

So will the IPT be updated to handle any of a number of input values?

For thoroughness's sake I also tried "human_observation" with the same result:

Thanks for confirming that Abby. That seems like a nuisance, and something perhaps we should just fix for any vocabulary field on import (ignore whitespace, casing and _). Agree?

just fix for any vocabulary field on import

I assume this conversion in the IPT would only be done for those fields that require a vocabulary? Or would this also be done for fields where a vocabulary is recommended?

Thanks @peterdesmet - yes, I think applying only to those fields the IPT requires makes sense.
Basically, it's to stop the IPT failing the publication just because you wrote presErved_SpeciMen  instead of preservedSpecimen (or whatever format is needed). That is just a nuisance that can be easily handled.


Feature request: add VECTOR data type support

Hi,
With 23ai version release, we now have VECTOR as a native data type. it would be great to add Vector support in Swift driver.
Reference implementation in python
Thanks!

Wow, this was quick 👍
Thank you!


Fix docstring and consistent use of a for second order orientation

Hi Nils,
just used your closures (thanks!) and fixed some docs on the way.
Changed case of first argument consistent to lower case a in closures.
Cheers Julian

Thanks Julian! 👍

Thanks Julian! 👍


Fix SSECallibrationFragment replacement logic, code and warning sanifications

Compiles fine and tested
Please tell me what do you think, what can I change and what should I remove

No problem, thanks for consider my work:)
When you are done, close the pull request


MinIO documentation is out of date

(See also: https://github.com/konveyor/pelorus/issues/347)
In addition to the missing script, the helm chart has been frozen. The new recommended workflow is using the MinIO operator.

[ ] Adapt the workflow to use the operator - new issue
[ ] Eventually integrate w/ Pelorus Helm Charts - new issue
[ ] Update the documentation

I am starting to get convinced that MinIO documentation should be replaced by noobaa as it's easier to setup and run as S3 provider for the project.

This is fixed and in review as part of the https://github.com/konveyor/pelorus/pull/558
We replaced minio recommendations with the noobaa.

Fixed as part of the PR #558. Removed MinIO and replaced by NooBaa


Fixing broken link to Jenkinsfile

The correct path to the Jenkinsfile is ./integrations/jenkins/Jenkinsfile
Signed-off-by: Marjan Stankovic<EMAIL_ADDRESS>Please ensure that your pull request adheres to our contribution guidelines. Thank you!

Just some general remark about your commits: I realized you use "marjanstankovic" as the author name / for the signoff. Shouldn't that rather be "Marjan Stankovic"?

Just some general remark about your commits: I realized you use "marjanstankovic" as the author name / for the signoff. Shouldn't that rather be "Marjan Stankovic"?

I changed it above in the comment. When doing the sign-off, I simply left the auto-generated text. Now I don't know how to change it now for the commits. Next time I will take care  :)

Just some general remark about your commits: I realized you use "marjanstankovic" as the author name / for the signoff. Shouldn't that rather be "Marjan Stankovic"?

I changed it above in the comment. When doing the sign-off, I simply left the auto-generated text. Now I don't know how to change it now for the commits. Next time I will take care  :)

Now I don't know how to change it now for the commits. Next time I will take care :)

As we are a bit picky about the Git history in this project, I'd like to kindly ask you to already fix it for this PR by rewriting your commits and force-pushing to this PR's branch. See it as a Git exercise 😉 You may want to follow these instructions.


Rearange PeerSelection

This PR rearanges modules under Ouroboros.Netowk.PeerSelection (for breviety,
the module prefix is skipped in what follows):

Governor.Types contains common types, it's not possible to mergeg it with the Types module due to cirular dependency with KnonwPeers
Governor.ActivePeers
Governor.EstablishedPeers
Governor.KnownPeers
Governor.RootPeers - includes only public root peers decision
Governor.Monitor   -which contains monitoring tasks for local roots, jobs and connections.

Documented Guarded data type
Added MkGuardedDecision type alias
Added job-pool package
Rename governor decisions
Split Governor into sub-modules

bors merge

bors r-

bors merge

bors merge
A strange error by check-stylish:
error: while evaluating anonymous function at /var/lib/buildkite-agent/builds/packet-ipxe-3-ci3-2/input-output-hk/ouroboros-network/shell.nix:3:1, called from undefined position:
...
getting status of '/var/lib/buildkite-agent/builds/packet-ipxe-3-ci3-2/input-output-hk/ouroboros-network/.git/objects/pack/pack-377d877718958c9fa46b056ed79134c1ac488888.pack': No such file or directory


Forcing null for empty tags hopefully fixes #3252

@rafatower or @Kartones , CR this fix for #3252, please. I haven't been able to reproduce it locally.

Frontend tests were OK :+1: (details)

:+1:

:+1:

Frontend tests were OK :+1: (details)


this project doesnt work anymore

i tried everything the code seem to be broken...

Yes same issue here.
Hi PerspectivesLab, what project do you use instead of this for tab buttons?
Thank you.

https://react-native-training.github.io/react-native-elements/docs/button_group.html
I decided to use this.

i used this simple tabs example here:
https://github.com/infinitered/ChainReactApp/blob/master/App/Containers/AboutScreen.js
function : renderTabs
i would recommend it as its low size, try ChainReactApp and see if it suits your needs


float8: remove unneeded kernel for scale generation

Summary:
The code to create a float8 scale is unnecessarily creating an extra GPU
kernel launch by calling torch.empty, removing this.
Test Plan:
// extract trace of a linear fwd+bwd with
python benchmarks/float8/profile_linear_float8.py ~/local/tmp/test
// verify that the GPU kernel creating an empty scale tensor is no longer there

// unit tests pass
./test/float8/test_everything.sh

Reviewers:
Subscribers:
Tasks:
Tags:

Stack from ghstack (oldest at bottom):

(to be filled)


Re-instantiation of the supabase auth client and memory leak?

Hello!
I am investigating an issue that is presenting itself in the following manner:

By starting my nuxt app in debugging mode and comparing two memory snapshots I was able to identify an excess amount of timers created:

Using the debugger I was able to trace this back to the startAutoRefresh function of the supabase gotrue client, which is called everytime the auth client is created.
Here's what I believe happens:

Each time I call serverSupabaseUser() (per h3 event) a new SupabaseAuthClient is created
Each time a SupabaseAuthClient is created, a new setInterval timer is started, and they are not cleaned up

My questions are:
Is the creation of a new supabase auth client per request expected?
I see that the supabaseClient is attached to the event here. Is there a way we can destroy or clean it up at the end of the request?
Do you have any insight into why it is necessary for the server side code to keep timers running for refresh tokens? Could disabling this be an option?
Thanks!
===
node 19.1.0
nuxt 3.2.0
nuxt/supabase 0.3.1
gotrue-js 2.4.2
supabase/supabase-js 2.7.1

Any update on this?

Seems to have been adressed by https://github.com/nuxt-modules/supabase/issues/189
Memory consumption is now stable:

Would love to hear what others have been observing before closing.


is there a way to abort tcp connection in the middle of handling to http request?

It seems that there was something related posted a while ago: https://github.com/nodejs/node-v0.x-archive/issues/1776
I'll describe case that I'm trying to handle: for simplicity assume that my node/express app listens for requests and then just proxies the request to another upstream server. Request takes a while to handle (e.g. seconds), and if upstream simply evaporates without finishing reply then I want to abort tcp connection so that first originating party would get error event instead of trying to process unfinished reply.
Below is a simple example for simplicity: server starts serving reply, says that it's 100-bytes long and then after serving 10 bytes simply closes the connection after 500ms. I wasn't able to find a way to abort http connection on server side so that client gets an error instead of thinking that request was properly served.
//server
var express = require('express');
var app = express();
app.use(function(req, res, next){
    res.setHeader('Content-Length', 100);
    res.writeHead(200);
    res.write('1234567890');
    setTimeout(function() { res.connection.destroy(); }, 500);
});
app.listen(8080);

and
//client
var http = require('http');
http.get('http://localhost:8080/', function(res){
   var length = 0;
   console.log('expected size: ', res.headers['content-length']);
   res.on('data', function(chunk){ length += chunk.length; });
   res.on('end', function(){ console.log('received: ', length); });
}).on('error', function(e){
    console.log('error', e);
});

at least on Windows if I throw a new Error inside that timeout then connection is reset (RST) and client gets error event. Why there is no api provided to abort connection, or the client is supposed to verify content length!? and what about chunked transfer then?
(I use node 6.6.0 on win10 and ubuntu 16.4)

Hi @pps83, I think you want to file the issue on Node.js directly: https://github.com/nodejs/node ; Express is just a routing layer on top of the Node.js server, and so doesn't provide the API for controlling TCP connections, since whatever there is just comes from the Node.js HTTP server.
If you change your server code to the following, you can confirm if Express actually has anything to do with your question, since it doesn't use Express:
var http = require('http');
var server = http.createServer(function(req, res){
    res.setHeader('Content-Length', 100);
    res.writeHead(200);
    res.write('1234567890');
    setTimeout(function() { res.connection.destroy(); }, 500);
});
server.listen(8080);

sorry, I was pretty sure I've posted it in nodejs, not sure how it ended up in expressjs :)


feat: support custom globs

tsc-esm-fix --src='src/main/ts/**/*.ts'
tsc-esm-fix --target='target/**/*.js'

:tada: This PR is included in version 2.18.0 :tada:
The release is available on:

v2.18.0
npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Disable fragmentMasking not working

Which packages are impacted by your issue?
No response
Describe the bug
Previously, when generating types from the graphql schema I would get a normal and expected type:
export type SomeMutation = { __typename: 'Mutation', myProject: { __typename: 'MyProject',... } | { __typename: 'MyProjectClosedError', message: string } | { __typename: 'MyProjectNotFoundError', message: string } | { __typename: 'SupplierNotFound', message: string } };

Now, the query contains fragments, but until now they were just a means of optimizing the queries.
After updating Codegen and MSW, I now get this:
export type SomeMutation = { __typename: 'Mutation', myProject: (
    { __typename: 'MyProject' }
    & { ' $fragmentRefs'?: { 'ProjectFirstLevelPropertiesFragmentFragment': ProjectFirstLevelPropertiesFragmentFragment;'ProjectSupplierFragmentFragment': ProjectSupplierFragmentFragment;'ProjectWorkflowStepsFragmentFragment': ProjectWorkflowStepsFragmentFragment } }
  ) | { __typename: 'MyProjectClosedError', message: string } | { __typename: 'MyProjectNotFoundError', message: string } | { __typename: 'SupplierNotFound', message: string } };

It's not just the structure that changed, but the whole model. This is not usable, any longer.
Your Example Website or App
I hope it is not needed, and it's just a configuration issue on my part.
Steps to Reproduce the Bug or Issue
...
Expected behavior
I expect that, unless I opt-in for a new behavior explicitly, that the schema is generated as previously.
Screenshots or Videos
No response
Platform

OS: Linux
NodeJS: 18.20.3
graphql version: 16.7.1
"@graphql-codegen/add": "5.0.3",
"@graphql-codegen/cli": "5.0.2",
"@graphql-codegen/client-preset": "4.3.0",
"@graphql-codegen/introspection": "4.0.3",
"@graphql-codegen/typescript": "4.0.7",
"@graphql-codegen/typescript-document-nodes": "4.0.7",
"@graphql-codegen/typescript-graphql-files-modules": "2.2.1",
"@graphql-codegen/typescript-graphql-request": "6.2.0",
"@graphql-codegen/typescript-msw": "3.0.0",
"@graphql-codegen/typescript-operations": "4.2.1",
"@graphql-codegen/typescript-react-query": "6.1.0",

Codegen Config File
import type { CodegenConfig } from '@graphql-codegen/cli';

const config: CodegenConfig = {
  overwrite: true,
  schema: 'libs/mocks/orchestrator-mocks/src/mocks/schema.graphql',
  documents: ['apps/**/*.graphql', 'apps/**/*.ts', 'libs/**/*.graphql', 'libs/**/*.ts'],
  generates: {
    'libs/mocks/orchestrator-schema/generated/': {
      preset: 'client',
      plugins: ['typescript-msw'],
      config: {
        futureProofEnums: false,
        fetcher: 'graphql-request',
        enumsAsConst: true,
        scalars: { Date: 'DateScalar', DateTime: 'Date' },
        nonOptionalTypename: true,
        fragmentMasking: false,
      },
    },
    'libs/mocks/orchestrator-schema/generated/graphql.schema.json': {
      plugins: ['introspection'],
    },
  },
};

Additional context
No response

Please disregard this. I passed the property wrong.
It should be:
generates: {
  presetConfig: {
        fragmentMasking: false
      }
}

Instead of:
generates: {
  config: {
    presetConfig: {
        fragmentMasking: false
      }
  }
}


⚠ Feedaty widget has degraded performance

In be13b03, Feedaty widget (https://widget.feedaty.com/public/js/feedaty.js?merchant=10213975&style_ver=2021&data-sku=36263) experienced degraded performance:

HTTP code: 200
Response time: 1092 ms

Resolved: Feedaty widget performance has improved in 03b2580 after .


Support gziped Liberty (.lib) files

Description
Liberty files are huge, which is not ideal when multiple PDKs are installed on a drive. If OL2 supported zipped .lib files naturally, that would be a big thing and super helpful. Plus, should not be super difficult, as OpenROAD seems to support zipped Liberty files already.
Proposal
Either unzip needed file in /tmp on request, or just make the read routines zip-capable.

See also https://github.com/iic-jku/IIC-OSIC-TOOLS/issues/88

@donn Here a small wish for 2025. Happy new year!


Support * wild card in TargetElement Attribute.

This will enable TagHelper authors to target IEnumerable<KeyValuePair> attributes (https://github.com/aspnet/Razor/issues/89).
Essentially:
[TargetElement(Attributes = "q-*")]
public class QueryParamTagHelper : TagHelper { }

Use:

Go to index

@DamianEdwards How important is this? Do we want this for beta5?

Yes, it's needed for the anchor Tag Helper when we finish the work to support binding to dictionary properties.

0e61b49881f77fb9123792ea12eb6f5e4411078d


More range and less delay to scan

Not sure of any specifics to why you posted this "bug." The scan radius and speed are the best values that they can be. Niantic changed the scan radius to 70 meters a while ago and last week changed the maximum scan time to roughly once every 10 seconds. Anything faster than that will show 0 Pokemon on the map and eventually earn the account a ban. If you need faster scanning, try opening a second instance of the map 5 minutes after the first one and log in with a different PTC account. Then minimize the second one and both should be running on the first map.

Could have a greater reach in less time, if possible, as a radius of 1km in
less than 5 seconds, a region or small town would be quickly scanned. Did
you understand what I said? But if the Niantic does not, ok, I'm enjoying
your work!
2016-08-09 15:13 GMT-03:00 mg6101<EMAIL_ADDRESS>
Not sure of any specifics to why you posted this "bug." The scan radius
and speed are the best values that they can be. Niantic changed the scan
radius to 70 meters a while ago and last week changed the maximum scan time
to roughly once every 10 seconds. Anything faster than that will show 0
Pokemon on the map and eventually earn the account a ban. If you need
faster scanning, try opening a second instance of the map 5 minutes after
the first one and log in with a different PTC account. Then minimize the
second one and both should be running on the first map.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/mchristopher/PokemonGo-DesktopMap/issues/783#issuecomment-238641126,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ATyDxkr-BKpiMNPdOcKGN69WeKKwQ-1yks5qeMM0gaJpZM4JgLTi
.


Some Prometheus metrics not reported - issue #1260

fixes: #1260

Needs a rebase

made on the wrong branch...


LEP-0005 implementation – remove uv_file

Since on Windows, HANDLEs are not the same as file descriptors,
confusing the two meant that almost all APIs in libuv had to go through
a thin compatibility layer, adding needless complication.
This removes that layer of external indirection and instead
works with the native Win32 API directly.
This is the implementation for https://github.com/libuv/leps/pull/8 which closes https://github.com/libuv/libuv/issues/856.
APIs added:

uv_get_osfhandle
uv_convert_fd_to_handle
uv_dup
UV_STDIN_FD / UV_STDOUT_FD / UV_STDERR_FD

APIs removed:

uv_poll_init_socket (now identical to uv_poll_init)

TODO list:

[x] documentation updates
[x] merge LEP 5

It's not clear to me why you removed uv_poll_init().  It's used a lot, removing it is rather disruptive.

On Windows, it's an alias for uv_poll_init_socket (on non-windows, the aliasing is reversed). In this case, I opted for removing the name uv_poll_init where the API changed on Windows (and keeping the name uv_poll_init_socket, which isn't affected by this change), to help FFI users find places they need to fix their code. But I can make this go either way, and I opened this as WIP to get feedback on that.

I'd rather keep uv_poll_init for consistency with the rest of the handles.

I've updated the PR based on your review feedback. Let me know what else needs to be done here.

Hey @vtjnash I'm ready to land this, once we have a bit of documentation. I seem to have forgotten to point that out earlier, sorry about that.
We need:

docs on the new utility function
corrections an all other functions that changed
an entry in a v1 -> v2 migration guide (this would start it) similar to this: http://docs.libuv.org/en/v1.x/migration_010_100.html

Thanks @sagful. I had documentation update in my TODO list at the top, and was just waiting for the LEP to be merged. I also added the migration guide – thanks for mentioned that too. Let me know if it looks good (or has any typos :P).

Damn, GH doesn't send notifications when you add new commits so I missed this. I'll review today @vtjnash.

Ok, I added a commit to remove uv_dup from the public API (I'm assuming you'll squash when you merge).

bump.

All comments have been addressed, should be ready to merge.

CI: https://ci.nodejs.org/view/libuv/job/libuv-test-commit/278/

Thanks - looks like I need to add the io.h header.

Alright, I've tried to cleanup the list of headers affected by this change to help avoid accidentally referencing shared state in msvcrt.dll where not intended. I think this should address the CI test failure, but would be best to re-run that to check that I didn't miss any.

CI: https://ci.nodejs.org/view/libuv/job/libuv-test-commit/299/

CI: https://ci.nodejs.org/view/libuv/job/libuv-test-commit/301/

CI: https://ci.nodejs.org/view/libuv/job/libuv-test-commit/305/

CI: https://ci.nodejs.org/view/libuv/job/libuv-test-commit/306/

CI: https://ci.nodejs.org/view/libuv/job/libuv-test-commit/314/

💚  landing!

Landed in 68dafe90, cheers @vtjnash! This was a lot of work and it took a long time to review, I'm sorry about the delay on my side and I'm grateful you stuck with it and took it over the finish line. 🍸

@libuv/collaborators I'll try to merge v1.x into master this weekend. It will be a messy one after this commit, but once done the next ones should be easier.

😀😀😀💥💥💥Thanks!


Windows-style paths result in module name / path mismatch

Passing windows-style path for the input file results in an error that the module name doesn't match the path, e.g.:
M:\> bin\mirth0.exe -o mirth.c mirth\main.mth
Compiling mirth\main.mth
Building.
src/mirth\main.mth:1:8: error: expected module name to match path
mirth/main.mth
mirth\main.mth

[The tricky bit is figuring out how to fix this without breaking the golden tests on windows. Maybe we need to add a command-line option to use unix paths regardless of the operating system, and use that in the test script.]

Is there a drawback to converting Windows-style paths to unix paths before using them?

Oh this has already been fixed.


[Accordion] Fix IconButtonProps spreading logic

The expanded class was being replaced when IconButtonProps was provided with a className.  To fix this, we revisit the application of className to IconButton.  We will continue to conditionally apply expanded, but now we'll combine it with className from IconButtonProps, which now defaults to an empty object.
Added onBlur test to improve coverage of this component.
Regenerated docs.
Closes #21744

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

@kgregory Thanks, it's great to not only fix the bug in question but to actually be a good boy-scout, leaving the place in better shape than when entering.

@oliviertassinari thanks.  Sorry about missing that act.  I won't make that mistake again.

@kgregory I think that it's simply because your local next branch is from an old version (~20 days), your build was green before I rebased it. Try pulling the latest changes :)

@oliviertassinari oh that's embarrassing.  Will do.


\rightharpoonup and \rightleftharpoons can't be recognized

\overset{\rightharpoonup}{a}=λ\overset{\rightharpoonup}{b}
\overset{\rightleftharpoons}{a}
\dfrac{3}{2}\overset{\rightarrow}{AC}

\rightharpoonup and \rightleftharpoons can't be recognizied either.
Originally posted by @yuwenjun1 in https://github.com/roniemartinez/latex2mathml/issues/299#issuecomment-1089982661

Closing this as the output is similar to MathJax output aside from \rightarrow which uses the value from unimathsymbols.txt.


chore(install): add validation and filtering for platform and platform family

This PR uses reflection over generated types to ensure that we are not sending invalid values for platform or platformFamily when retrieving recommendations from the recipe service.
cc @jbeveland27  @michaelkro

@ctrombley a side note, could we add some debug logging to the QueryWithResponseAndContext( ) calls for what goes out and back, this way we can troubleshoot more easily when things don't work?

@ctrombley a side note, could we add some debug logging to the QueryWithResponseAndContext( ) calls for what goes out and back, this way we can troubleshoot more easily when things don't work?


lxc launch from script launches container and then waits

The template below is mostly useful for bug reports and support questions.
Feel free to remove anything which doesn't apply to you and add more information where it makes sense.
Required information

Ubuntu Server 16.04 LTS
config:

  images.auto_update_cached: "false"
  images.auto_update_interval: "0"
  images.remote_cache_expiry: "28"
api_extensions:
- storage_zfs_remove_snapshots
- container_host_shutdown_timeout
- container_syscall_filtering
- auth_pki
- container_last_used_at
- etag
- patch
- usb_devices
- https_allowed_credentials
- image_compression_algorithm
- directory_manipulation
- container_cpu_time
- storage_zfs_use_refquota
- storage_lvm_mount_options
- network
- profile_usedby
- container_push
- container_exec_recording
- certificate_update
- container_exec_signal_handling
- gpu_devices
- container_image_properties
- migration_progress
- id_map
- network_firewall_filtering
- network_routes
- storage
- file_delete
- file_append
- network_dhcp_expiry
- storage_lvm_vg_rename
- storage_lvm_thinpool_rename
- network_vlan
- image_create_aliases
- container_stateless_copy
- container_only_migration
- storage_zfs_clone_copy
- unix_device_rename
- storage_lvm_use_thinpool
- storage_rsync_bwlimit
- network_vxlan_interface
- storage_btrfs_mount_options
- entity_description
- image_force_refresh
- storage_lvm_lv_resizing
- id_map_base
- file_symlinks
- container_push_target
- network_vlan_physical
- storage_images_delete
- container_edit_metadata
- container_snapshot_stateful_migration
- storage_driver_ceph
- storage_ceph_user_name
- resource_limits
- storage_volatile_initial_source
- storage_ceph_force_osd_reuse
- storage_block_filesystem_btrfs
api_status: stable
api_version: "1.0"
auth: trusted
public: false
environment:
  addresses: []
  architectures:
  - x86_64
  - i686
  certificate: |
    -----BEGIN CERTIFICATE-----
BLAH
    -----END CERTIFICATE-----
  certificate_fingerprint: 6ca563d2d177d8f85395e57bb21646239c731a6d1d9a8d652cc7feaa5195dc62
  driver: lxc
  driver_version: 2.0.8
  kernel: Linux
  kernel_architecture: x86_64
  kernel_version: 4.11.0-14-generic
  server: lxd
  server_pid: 34857
  server_version: "2.18"
  storage: zfs
  storage_version: <IP_ADDRESS>-1ubuntu2~2

Issue description
I am automating launching of containers. I have a script that launches a test container, prints out it's info and then stops and deletes that container.
lxc info
lxc profile show default
lxc launch -c security.nesting=true ubuntu:16.04 test
lxc list
lxc stop test
lxc delete test
lxc list

Unfortunately, when done this way, lxc launch just hangs. See output below:-
Creating test
Starting test
+------+---------+------+------+------------+-----------+
| NAME |  STATE  | IPV4 | IPV6 |    TYPE    | SNAPSHOTS |
+------+---------+------+------+------------+-----------+
| test | RUNNING |      |      | PERSISTENT | 0         |
+------+---------+------+------+------------+-----------+

This means that launching containers from scripts is not automatable?
There is no man page worth it's name for lxc. In addition, there is no way to start a container in daemon mode it looks like unlike docker.

Based on the output above, it's not launch that's hanging, it's "stop". That can happen when the init system is slow to setup its signal handler for SIGPWR.
If you don't care about having the container go through a clean shutdown (which is what "lxc stop" gets you), then pass "--force" to "lxc stop" which will just have it kill the container.
You could also just do "lxc delete test --force" which will both stop and delete the container.
I'm not sure what you mean about daemon mode. All LXD containers are running in the background, as I said, you're getting stuck on "lxc stop" here, not "lxc launch".
As for manpages, the help is in "lxc launch --help". A manpage is auto-generate from that and available through "man lxc.launch".

@stgraber  My bad then.
Re man pages; the problem is that the man page does not have more details than the CLI. For example,
curl --help will give me a quick list of options; however, if I want more details on what each option does then I do man curl.
However, with lxc the man page and lxc --help are equivalent. There is no additional information on valid values. An example is the -t parameter. Are there any predefined instance types? The REST API documentation seems to indicate there is by referencing a c2.micro instance type. However, there is no documentation on what it is or what that term maps to. Are you basing the instance types based off of AWS EC2 instance types?


Need the dependency for package_info_plus to be <= 8.0.0 not < 8.0.0

I'm deep down the Apple xcprivacy rabbit hole and getting this:
Because wakelock_plus >=1.2.4 depends on package_info_plus >=6.0.0 <8.0.0 and micdup_app depends on package_info_plus ^8.0.0, wakelock_plus >=1.2.4 is forbidden.
I'll downgrade package_info_plus to 7.0.0 and cross my fingers, but thought you should know.

No the constraint should be <9.0.0 to allow all 8.x.x versions.

No the constraint should be <9.0.0 to allow all 8.x.x versions.

So can someone update the constraint then

https://github.com/fluttercommunity/wakelock_plus/pull/52 someone did it already, it just needs to be reviewed, merged and released.


SetState unpauses the FlutterStoryController

Hi @devkrest ,
Thank you for creating this package. It has significantly sped up the development of a story-like feature in my company's app. However, we're currently facing an issue with state management. Whenever the controller is paused and we perform a SetState or any action that forces a redraw, it starts playing immediately, which is not ideal for our use case.
Is this behavior intended? Are we missing something, or is it actually a bug? I've attached a small example that replicates this behavior.
We would greatly appreciate your help.
Best regards,
import 'package:flutter/material.dart';
import 'package:flutter/services.dart';
import 'package:flutter_story_presenter/flutter_story_presenter.dart';

class TestStories extends StatefulWidget {
  const TestStories({super.key});

  @override
  State<TestStories> createState() => _TestStoriesState();
}

class _TestStoriesState extends State<TestStories> {
  late final FlutterStoryController _controller;
  late final List<StoryItem> _stories;
  final Color paradisePink = const Color(0xFFE64467);

  @override
  void initState() {
    _controller = FlutterStoryController();
    _stories = [
      StoryItem(storyItemType: StoryItemType.text, url: "Hello World"),
    ];
    super.initState();
  }

  @override
  Widget build(BuildContext context) {
    return FlutterStoryView(
      key: Key('Test-Stories'),
      restartOnCompleted: false,
      flutterStoryController: _controller,
      items: _stories,
      onStoryChanged: (int index) {
        print('~CHANGE');
      },
      footerWidget: TextButton(
        onPressed: () {
          HapticFeedback.lightImpact();
          _controller.pause();
          setState(() {});
        },
        child: Text('data'),
      ),
      storyViewIndicatorConfig: StoryViewIndicatorConfig(
        activeColor: paradisePink,
        backgroundCompletedColor: paradisePink,
      ),
    );
  }
}

Hi @AmauryTejada
Thanks for your feedback! We've identified the issue with the controller resuming playback on redraw and will work on a fix ASAP.
Best regards,
Devkrest Team

Hi @AmauryTejada. Checking this soon.

Hi @AmauryTejada. The issue is fixed. Can you please check running this branch? If everything looks good, we can merge to master. Thanks
https://github.com/devkrest/flutter_story_presenter/tree/fix/22-setState-unpauses-flutterstorycontroller

Hi @Kaival-Patel ,
I just tested the branch, and it works perfectly. Thank you very much for your quick response. Do you plan to merge this fix into the Master branch soon?

Yeah, we will merge the branch into the master before Diwali. Happy Diwali 🪔 @AmauryTejada

Hi everyone!
I hope you're all doing well. I was wondering if there's any chance we could get this fix into the master branch soon? Thanks a lot!

@AmauryTejada Already moved to master branch if you can test.


Disable CODEOWNERS for the time being

Unfortunately the only thing this is accomplishing at the moment is spamming the listed CODEOWNER to review PRs based on file changes with no option to disable this. This is not always great system because it assumes reviewers align perfectly to subdirectories of the mono repo, which may not always be the case.
Since we're a small team (for now), I think it's safe to assume the author of the PR can be entrusted to select the proper reviewers for their PRs.

Can you add to the PR description why? :)

@thomaseizinger Whoops, thought I had typed it in. Updated!

For small teams its not needed, agreeed


Human readable actor names

Right now in flipper our actor names look something like:

It would be nice if there was a way to specify a human readable string, eg by using the to_s method on the model. Perhaps there is a way to do this and I'm missing it? If so apologies.

I totally love this idea. I was thinking that likely the easiest way would be some sort of hydrator for actors. Given a dehydrated actor id (or ids) it invokes a block you provide to return a fully hydrated object (AR model, or whatever). Something in Flipper::UI config like actor_hydrator could maybe do this. I'd probably make it work with batches of ids by default to avoid n+1 lookups too.

Another alternative would be to store this along with the actor id but becomes a cache invalidation issue pretty quick.

I Would like to follow up on this.
It would be very helpful to have some simple information next to the actor,
because in a long list of User:1234567 it get's out of hand which customer is allowed to see the feature very quickly.
So I guess either we could store this information in the Gate Table, or (preferable) have another flipper method in the model like flipper_name to be able to delegate that information to a custom column i.e. username, email...

could it be possible to maybe use an alternative unique attribute such as user.email?
e.g.
  email

You definitely can. The problem is that people change their email addresses so you’d need to be sure to update flipper if so.

Worth mentioning I laid out a plan for this if someone has the time to make a PR.
https://github.com/jnunemaker/flipper/issues/657#issuecomment-1215365903

Hey @jnunemaker, I have PR #737 ready for this!

I’ll check it out soon! On vacation so I’ll be slower than usual. Thanks!

I’ll check it out soon! On vacation so I’ll be slower than usual. Thanks!

Hey John, hope you had a nice vacation! Let me know if you're able to get to this sometime soon.

I realized I should have mentioned your handle. There goes @jnunemaker

closed by https://github.com/flippercloud/flipper/pull/737


sudo-rs: package and use as replacement for 'sudo'

Project description
sudo-rs (a Rust-written sudo-compatible replacement) is having a stable first release: https://www.memorysafety.org/blog/sudo-first-stable-release/
It would be nice to have this in nixpkgs, and have an option to use it in place of the traditional sudo.
(this would be in addition to please that is available in Nixpkgs already, which is a sudo-alternative also written in rust)
Metadata

homepage URL: https://github.com/memorysafety/sudo-rs and more info: https://www.memorysafety.org/initiative/sudo-su/
source URL: https://github.com/memorysafety/sudo-rs
license: mit and apl2
platforms: unix, linux (maybe darwin too?)

Just dropping my 2 cents: sudo-rs only supports linux for now.

Needs an option too

@Nyabinary there's already a package option in the existing sudo module that we could use to select sudo-rs.
But it's currently not a drop-in replacement and so the module would need some adapting to cater to both sudo and sudo-rs.

On it ;3

Wouldn't work. Sudo-rs needs rust 1.70, which is in unstable.

Hi @alsvartr,
Rust 1.70 is stable at 2023, Jun 01.
https://blog.rust-lang.org/2023/06/01/Rust-1.70.0.html
Just excited about this, so subscribe to this issue.
Is it possible to upgrade the Rust version to stable?
If something I can help with, please kindly let me know.

You can just package sudo-rs for the unstable repo too.

You can just package sudo-rs for the unstable repo too.

Hmm... I think it is not really used sudo-rs even in unstable now, even with #252418
The package is built but there is no optional toggle to switch sudo to sudo-rs for now
  config = mkIf cfg.enable {
    assertions = [
      { assertion = cfg.package.pname != "sudo-rs";
        message = "The NixOS `sudo` module does not work with `sudo-rs` yet."; }
    ];

You can just package sudo-rs for the unstable repo too.

Got it, I will try unstable first.

I've created https://github.com/NixOS/nixpkgs/issues/253465 to track the remaining tasks


flake: remove package definition

A package derivation isn't very useful for this project. The deployment model assumes a Cargo based build on the remote server, orchestrated by Ansible. For local development we also prefer Cargo. Having the package derivation would be useful if we wanted to deploy with Nix, but we don't. It also makes taking Git dependencies in our Cargo.toml cumbersome, and may break builds of tools like bencher-client that aren't written with an isolated build environment in mind.

Thanks!


DOE Funding Request: OB JSON Validation Tool

Moved to Funding request list: https://docs.google.com/spreadsheets/d/1iCwTSf7F65iWITqP952Uy0Y9gdva5dTwtaahM4GRRds/edit#gid=0


UI issue : FotoToon Actitvity So Much Padding in Warning Pop Up

Hey @llaske ,
i found this issue in FotoToon activity.
If we click the clean all icon a warning Pop up shows.
the Pop up buttons elements cancel and continue  have extra  width which doesnt follow the sugarizer UI .
Current

How it should Be

It can be improved by a minor change in the CSS .
If you think it can be improved do let me know your thoughts on this.
I would love to fix it.

No, it's good like that.


mem profiler (windows only)

usage:
#mem_profiler begin E:/mem_profile.txt
#mem_profiler end

the output will just be a sorted list of procs and how much memory was allocated during their execution. there are no stack traces or anything fancy like that

pog


USA Hyundai Missing Force Refresh

USA Hyundai doesn't have force refresh implemented.
https://github.com/Hyundai-Kia-Connect/hyundai_kia_connect_api/blob/9c846174d64cf63ef755feb47e72997b2ff7706d/hyundai_kia_connect_api/HyundaiBlueLinkAPIUSA.py#L367-L368

This also needs to be updated to confirm to this new style: https://github.com/Hyundai-Kia-Connect/hyundai_kia_connect_api/commit/9b05c70ca9d3d0c348595c4cd9756f810018741a


Get the 3d coordinate output of the model

This template is for miscellaneous issues not covered by the other issue categories
For questions on how to work with MediaPipe, or support for problems that are not verified bugs in MediaPipe, please go to StackOverflow and Slack communities.
If you are reporting a vulnerability, please use the dedicated reporting process.
Compile the library inference model directly with tensorflowlite。I modified my own code to get the normalized real world x, y, z coordinates through the following official code.

But the output doesn't seem to be correct, what am I doing wrong?

Hi @Youmimzy ,
On Android Hands gradle you can find the result of https://github.com/google/mediapipe/blob/d4bb35fe5a44b3419ca86e4c7ecbc931dded2122/mediapipe/java/com/google/mediapipe/solutions/hands/HandsResult.java#:~:text=// Collection of detected/tracked hands' landmarks,}. To run on GPU you should enable runOnGput to True


tools: Allow trigger docker build manually

Summary
In some cases, we may need to manually trigger the docker image building process, for example, if the automatic build fails due to some reasons.
Make it easier for downstream users to build their docker images.
For example, I‘m maintaining the CI infrastructure for WAMR on the wamr-ci-12.4(based on release/12.4) branch and have added some additional toolchains to the NuttX Dockerfile. If I want the CI to compile the images I need, I must ensure that my modifications to the Dockerfile are applied to the wamr-ci-12.4 branch, while also keeping this PR on the master branch. Considering that I often submit code to NuttX, this will cause some inconvenience (I must have this PR on my master branch to manually trigger the Docker build process).
Impact
Minor
Testing
CI

@no1wudi please include the Summary comment in the commit log message!
I think a documentation about how to use Docker with NuttX and CI should be nice! I did some initial documentation some time ago: https://acassis.wordpress.com/2023/01/21/how-i-create-a-docker-image-for-nuttx/ , but maybe with your expertise we could improve it and add into NuttX. What do you think?

@acassis Thanks for your information, let me try other approaches first.


Allow user/reader to change render-style

Would it be possible to add a link/button/input element which enables the user (reader) to choose which render-style to use (and optionally persist this to localStorage)? If so, could anyone point me in the right direction?

Thanks!


EADDRINUSE <IP_ADDRESS>:8086 error for multiple instances

Recently started having following error when trying to start more than one instance of whs.
Error: listen EADDRINUSE <IP_ADDRESS>:8086
    at Object.exports._errnoException (util.js:1036:11)
...

whs version - 3.10.9
node version v6.6.0
console output looks like this
$ whs -p 4444
Websocket Server Listening on Port: 8086
Starting up http-server, serving ./ on: http://<IP_ADDRESS>:4444
Hit CTRL-C to stop the server
events.js:160
      throw er; // Unhandled 'error' event

As of 0.10.0 if you've not specified a specific port via the -p option, then one will be auto-selected for you by portfinder (now upgraded to 1.0.13.
That said, the example you've listed specifies port (via -p 4444), so that would still throw that error.
However, we're discussing a --find-port option in #363 and your input there would be welcome!

Comments are welcome in #363, but for now I'm going to close this since a) the latest versions of http-server should find a new port by default, and b) it is intended behavior that trying to run multiple instances on the same port is an error.


Changelogs should be linted on publish

Quilt has many packages, and it is easy for someone to miss moving the UNRELEASED section of a CHANGELOG.md into a versioned section on publish.
This has unfortunately already occurred, as documented in #1341.
Ideally, we would solve this on multiple fronts:

Update CHANGELOG.md automatically on publish
Lint CHANGELOG.md on publish, preventing invalid publication
_This could be done naively by checking for the presence of ## UNRELEASED, vs <!-- ## UNRELEASED -->.

Should we use conventional changelog? Lerna 3.0 had this feature built-in.


fix: Layer delete on drop and eviction can outlive timeline shutdown

This is a follow-up to #7051 where LayerInner::drop and LayerInner::evict_blocking were not noticed to require a gate before the file deletion. The lack of entering a gate opens up a similar possibility of deleting a layer file which a newer Timeline instance has already checked out to be resident in a similar case as #7051.

Whenever Layer needs a timeline, it should be accompanied by a guard; there is never any other situation.

Is this really a follow-up to #7051 ?
Seems to me like it's a similar issue like what was fixed in #7051, but, for the eviction rather the download path.
Would like to see the PR description adjusted if you think that's the case.

Sorry, I had the auto-merge on. "Follow-up" as in "continuation of the story" or even the google definition "pursue or investigate something further" as in these cases I was thinking of only during weekend.


DeepSpeed with Phi-3-mini-128K-instruct does not generate <|endoftext|> token

I am using Phi-3-mini-128K-instruct model for inference via DeepSpeed for high throughput and facing issue of model not generating <|endoftext|> during the generation time. For all my requests it generates all max_new_tokens size output and generation stops abruptly in the middle of the sentence due to token generation limit.
It generates <|end|> token in the text, which I am using as of now to limit the output but in some cases, it does not even generate <|end|> token as well.
Tested same model with vLLM engine and huggingface as well, and there it is able to generate <|endoftext|> token and stops token generation, so clearly issue is with DeepSpeed-MII only.
Expected Behaviour: During the inference <|endoftext|> (eos_token) should be generated and output generation should be stoped.
[2024-09-27 10:03:56,258] [INFO] [real_accelerator.py:203:get_accelerator] Setting ds_accelerator to cuda (auto detect)
DeepSpeed C++/CUDA extension op report
NOTE: Ops not installed will be just-in-time (JIT) compiled at
runtime if needed. Op compatibility means that your system
meet the required dependencies to JIT install the op.
JIT compiled ops requires ninja
ninja .................. [OKAY]
op name ................ installed .. compatible
async_io ............... [NO] ....... [OKAY]
fused_adam ............. [NO] ....... [OKAY]
cpu_adam ............... [NO] ....... [OKAY]
cpu_adagrad ............ [NO] ....... [OKAY]
cpu_lion ............... [NO] ....... [OKAY]
[WARNING]  Please specify the CUTLASS repo directory as environment variable $CUTLASS_PATH
evoformer_attn ......... [NO] ....... [NO]
fp_quantizer ........... [NO] ....... [OKAY]
fused_lamb ............. [NO] ....... [OKAY]
fused_lion ............. [NO] ....... [OKAY]
/home/ec2-user/anaconda3/envs/pytorch_p310/compiler_compat/ld: /usr/local/cuda-12.1/lib64/libcufile.so: undefined reference to dlopen' /home/ec2-user/anaconda3/envs/pytorch_p310/compiler_compat/ld: /usr/local/cuda-12.1/lib64/libcufile.so: undefined reference to dlclose'
/home/ec2-user/anaconda3/envs/pytorch_p310/compiler_compat/ld: /usr/local/cuda-12.1/lib64/libcufile.so: undefined reference to dlerror' /home/ec2-user/anaconda3/envs/pytorch_p310/compiler_compat/ld: /usr/local/cuda-12.1/lib64/libcufile.so: undefined reference to dlsym'
collect2: error: ld returned 1 exit status
gds .................... [NO] ....... [NO]
inference_core_ops ..... [NO] ....... [OKAY]
cutlass_ops ............ [NO] ....... [OKAY]
transformer_inference .. [NO] ....... [OKAY]
quantizer .............. [NO] ....... [OKAY]
ragged_device_ops ...... [NO] ....... [OKAY]
ragged_ops ............. [NO] ....... [OKAY]
random_ltd ............. [NO] ....... [OKAY]
[WARNING]  sparse_attn requires a torch version >= 1.5 and < 2.0 but detected 2.4
[WARNING]  using untested triton version (3.0.0), only 1.0.0 is known to be compatible
sparse_attn ............ [NO] ....... [NO]
spatial_inference ...... [NO] ....... [OKAY]
transformer ............ [NO] ....... [OKAY]
stochastic_transformer . [NO] ....... [OKAY]
DeepSpeed general environment info:
torch install path ............... ['/home/ec2-user/anaconda3/envs/pytorch_p310/lib/python3.10/site-packages/torch']
torch version .................... 2.4.1+cu121
deepspeed install path ........... ['/home/ec2-user/anaconda3/envs/pytorch_p310/lib/python3.10/site-packages/deepspeed']
deepspeed info ................... 0.15.1, unknown, unknown
torch cuda version ............... 12.1
torch hip version ................ None
nvcc version ..................... 12.1
deepspeed wheel compiled w. ...... torch 2.4, cuda 12.1
shared memory (/dev/shm) size .... 7.72 GB

Thanks for reporting this @shubhanshu786 - I'll try to take a look.


Add redisClustervCommandToNode and redisClustervAppendCommandToNode to library API

Closes #230.
I did not add new tests as original redisClusterv* APIs are not tested directly, and test coverage should not change. Please comment in this PR if separate specialized tests are required.
The only disadvantage of this PR that I see, is that va_start() is called in more cases. It can add some (albeit minuscule) overhead, but all such cases end in an error anyway.

Looks good, thanks!


Initial plugin and system test task

fixes: https://github.com/creek-service/creek-system-test-gradle-plugin/issues/13

Coverage remained the same at 100.0% when pulling ed2c209c6425e79dbf25e4bb4726e6f6a88ca6d0 on test_task into cd35a01cafda6cc1c27ab56ce48e3773645826b5 on main.


[BUG] api推理不支持mp3格式的参考音频

Feel free to ask any kind of questions in the issues page, but please use English since other users may find your questions valuable.
Describe the bug
如题
To Reproduce

开启api
请求api服务（使用.mp3作为参考音频格式）

Expected behavior
Screenshots / log
ERROR:    Exception in ASGI application
Traceback (most recent call last):
  File "/usr/local/lib/python3.10/site-packages/uvicorn/protocols/http/h11_impl.py", line 398, in run_asgi
    result = await app(  # type: ignore[func-returns-value]
  File "/usr/local/lib/python3.10/site-packages/uvicorn/middleware/proxy_headers.py", line 70, in __call__
    return await self.app(scope, receive, send)
  File "/usr/local/lib/python3.10/site-packages/kui/asgi/applications.py", line 163, in __call__
    await self.app(scope, receive, send)
  File "/usr/local/lib/python3.10/site-packages/kui/asgi/applications.py", line 118, in app
    return await getattr(self, scope_type)(scope, receive, send)
  File "/usr/local/lib/python3.10/site-packages/kui/asgi/applications.py", line 142, in http
    return await response(scope, receive, send)
  File "/usr/local/lib/python3.10/site-packages/baize/asgi/responses.py", line 167, in __call__
    chunk = await generator.asend(None)
  File "/usr/local/lib/python3.10/site-packages/baize/asgi/responses.py", line 191, in render_stream
    async for chunk in self.iterable:
  File "/exp/tools/api.py", line 331, in inference_async
    for chunk in inference(req):
  File "/usr/local/lib/python3.10/site-packages/torch/utils/_contextlib.py", line 35, in generator_context
    response = gen.send(None)
  File "/exp/tools/api.py", line 225, in inference
    prompt_tokens = encode_reference(
  File "/exp/tools/api.py", line 96, in encode_reference
    reference_audio_content = load_audio(
  File "/exp/tools/api.py", line 89, in load_audio
    audio, _ = librosa.load(reference_audio, sr=sr, mono=True)
  File "/usr/local/lib/python3.10/site-packages/librosa/core/audio.py", line 186, in load
    raise exc
  File "/usr/local/lib/python3.10/site-packages/librosa/core/audio.py", line 176, in load
    y, sr_native = __soundfile_load(path, offset, duration, dtype)
  File "/usr/local/lib/python3.10/site-packages/librosa/core/audio.py", line 209, in __soundfile_load
    context = sf.SoundFile(path)
  File "/usr/local/lib/python3.10/site-packages/soundfile.py", line 658, in __init__
    self._file = self._open(file, mode_int, closefd)
  File "/usr/local/lib/python3.10/site-packages/soundfile.py", line 1216, in _open
    raise LibsndfileError(err, prefix="Error opening {0!r}: ".format(self.name))
soundfile.LibsndfileError: Error opening <_io.BytesIO object at 0x7efe83e09c10>: Format not recognised.

Additional context
看样子实在解码的过程中出了问题：audio, _ = librosa.load(reference_audio, sr=sr, mono=True)

可以使用ffmpeg或者其它各种音频软件转码。

#482 已修复


Cleanup SKL LabelEncoder Converter Code

We have the original draft of the SKL LabelEncoder code uploaded in this branch. All credit here goes to @scnakandala, the original author and brains behind this.
It contains an un-edited implementation of LabelEncoder here.
There is a test file here that needs to be cleaned up and passing (and also tests added for errors).
Notes:

We had to treat strings a bit differently because of the way they are represented in tensors

We should add more expansive tests

In the numeric label encoder, there is a tricky spot here related to running on GPU vs CPU.

There may be some bugfixes/improvements that are available now. Our CI/CD pipeline is not currently GPU-enabled, but it would be helpful to be able to run this on a GPU-enabled machine when developing.

Btw, many people that are using LabelEncoder should actually be using OneHotEncoder. LabelEncoder was meant for labels i.e. targets, not features, and only supports a single column, while OneHotEncoder supports multiple columns.

Good to see you here @amueller! :) Yes we also have OneHotEncoder, it is work in progress in PR #147.

closed via #374


Issue with UI of 'MSGMessengerViewController' (iOS 13)

Issue with UI of 'MSGMessengerViewController' (iOS 13)
Steps:

push in Navigation controller instance of MSGMessengerViewController subclass
tap in text field of MSGInputView
keyboard appears
minimize the application by press of 'Home'
activate application by tap on app icon
show modally login screen (on top of a chat screen)
enter credentials and dismiss login screen
see issue with UI

Now it was fixed in my project by set up zero to 'topConstraint.constant' (KeyboardLayoutGuide) in 'viewWillAppear' of MSGMessengerViewController subclass.

Hi @AntonShapoval, thanks for opening this issue. If you would like to submit a PR, I would be happy to review it 👍


No channel layout for input 1

While attempting to record a video in the project, it throws error "No channel layout for input 1". What would cause this issue and what are the ways around it ?
for (int i = 0; i < audioFileName.Length; i++)
{
    audioInput += " -stream_loop -1 -i " + audioFileName[i];
    inputArgs += "[" + (i + 1) + ":a]";
}
var parameters = "-y -f rawvideo -vcodec rawvideo -pixel_format rgba" +
                 " -colorspace bt709" +
                 " -video_size " + width + "x" + height +
                 " -framerate " + frameRate +
                 " -loglevel warning -i -" + audioInput +
                 " -filter_complex \"" + inputArgs +
                 "amerge=inputs=" + audioFileName.Length +
                 "[a]\" -map 0:0 -map \"[a]\" -c:a aac -ac 2 -t " +
                 duration + "ms "
                 + preset.GetOptions()
                 + " \"" + outputPath + "\"";
return new FFmpegSession(parameters);

What is this code snippet? There is no such section in this project.

@keijiro That's the call to FFmpegSession with given parameters. I'm not sure what could possibly cause this error, whether it's something about the call, or the audiofile itself.

Do you have any ideas on what could cause this warning/error at all ?

FFmpeg returned with warning/error messages. See the following lines for details:
[Parsed_amerge_0 @<PHONE_NUMBER>74e800] No channel layout for input 1

Unfortunately, I have no idea -- I'm not a specialist of ffmpeg. I'd recommend asking a specialist to solve your problem.

I'm closing this issue now as the original post is not relevant to this project. Please reopen it for further problem.


upgrade jenkins-k8s-node to 1.1.0

test this please

test this please

test this please

@geojaz  Besides using tag 1.1.0 on jenkins slave. As the CloudSQL can only reuse the instance name after 1 week, I have randomized the instance name in the scripts.

@pydevops we need seperate PRs for this please.  How was this working w/o your updates??

test this please


Memory allocator tests and benchmarks.

What do these changes do?

Memory allocator tests and benchmarks.
Related issue number

Fixes #289

Hi @peilii,
Thanks for the effort. Could you please rename the directory to benchmark? bench-x looks a bit strange.

Hi @peilii,
Thanks for the effort. Could you please rename the directory to benchmark? bench-x looks a bit strange.

No problem at all!

Please fix the format & lint error.

Please fix the format & lint error.

I suppose it's done now. Should I also fix the Codacy Static Code Analysis problem?

Should I also fix the Codacy Static Code Analysis problem?

It doesn't matter.

Merged. Thanks!

Merged. Thanks!

No problem! Thanks!


Choreo migration tasks

 @xlight05 cloned issue wso2-enterprise/ballerina-registry#2342 on 2023-12-06:

Description:
Tasks related to Ballerina Central migration to Choreo

 @tharindu-nw commented on 2024-09-10:

We have successfully completed the migration of the Dev environment onto a Choreo PDP


Revert "Docs build v3"

Reverts Azure/azure-docs-sdk-dotnet#1407

Docs Build status updates of commit d1065f6:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

.openpublishing.publish.config.json
:white_check_mark:Succeeded

.openpublishing.redirection.json
:white_check_mark:Succeeded

api/overview/azure/backup.md
:white_check_mark:Succeeded
View

api/overview/azure/devices.md
:white_check_mark:Succeeded
View

docfx.json
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching in the Docs contributor and Admin GuideSee the frequently asked questionsPost your question in the Docs support channel


fix: BuildAggForUnion forgot to convert Qualifier Type.

Proposed changes
Issue Number: close #xxx
Problem summary
BuildAggForUnion forgot to convert Qualifier::Distinct -> ALL.
Checklist(Required)

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

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

run buildall


Question: When are security patches applied to azure functions?

When are security patches applied to azure functions? Is there a timeline?
When will functions get patched from dotnet 7.0.4 to 7.0.5?
Thank you!

@pavazpavaz we need to work with the platform team for that information across the various SKUs (Windows/Linux , Consumption, Elastic Premium, etc.) so we can share this information.
We'll provide an update once we have more information on public documentation on this, but is there as specific SKU you were interested in?

FYI @mattchenderson


Better bar widths for unevenly-spaced data (already done for boxplot)

Could something similar to 2246a737ecf825ac5794b55bf8c758a5af4b2094 be done for Geom.bar?  This was first raised in #596 and done for boxplot.  A simple example to produce the issue is
df = DataFrame(Year = [1999,2001,2002,1999,2002],
Customer = ["Marvel", "Marvel", "Marvel", "DC", "DC"],
Number = [12,33,1,23,26])
plot(df, Geom.bar, x="Year", y="Number", color="Customer")

Should be better now.


update() and ENUM

Hi,
I had an issue with the following code. All get and set works fine except that ENUM are not converted to the value set value. In this case, the UPDATE sql query will be type = 'sup' and not = '0'. save() works like a charm.
I use MySql, no sqlType.

$e = new Enterprise();
$e->setPrimaryKey($data['IdEnterprise']);
$e->fromArray($data);
if ($e->validate()) {
$q = new EnterpriseQuery();
$q->filterByIdEnterprise($data['IdEnterprise'])
->update($data);
}
Good day,
Fred

Hi! I have the same problem too. Did you find a solution for that?


feat: request with params

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

17 of 33 (51.52%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-1.0%) to 64.683%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

fetch.go
17
33
51.52%

Totals

Change from base Build<PHONE_NUMBER>:
-1.0%

Covered Lines:
337

Relevant Lines:
521

💛  - Coveralls


Accessors for reverse realtionships

It would be nice if the Playlist ↔ song through table would provide the reverse accessors. This way you could access the songs from a playlist directly and not using the filter. https://docs.sqlalchemy.org/en/14/orm/relationship_api.html#sqlalchemy.orm.relationship.params.back_populates

Hey,
thank you for your suggestion! To be honest the v6 Database handler is in the very early stages, but i plan to implement more accessors and a better API in general. Any suggestions/ideas are welcome!:)

thanks again for the suggestion!


ProgressBar: make every file with signal reachable through clicking

Signed-off-by: Leslie Lazzarino<EMAIL_ADDRESS>Summary of changes
All files that count as having signal in the progress bar, including the ones whose parent folder has signal, are now added to the list of files you can reach clicking on the progress bar.
Context and reason for change
When a folder has signal and no attribution his files are shown in the progress bar as having only signal. Anyway those files where unreachable by means of clicking on the progress bar.
How can the changes be tested
By changing an input file to recreate such configuration.

Wouldn't it be easier to also jump to folders with signal?
In the current implementation, if there is a folder with signal high up in the tree, there are many files without signal one visits when clicking on the progress bar.

Wouldn't it be easier to also jump to folders with signal? In the current implementation, if there is a folder with signal high up in the tree, there are many files without signal one visits when clicking on the progress bar.

Would require a bigger change in the logic, with increased complexity. Not sure if it is worth it.


Test Automation for Issue 41202: Combination of Submitter and Reader Role does not see the "Insert Rows" icon

Rationale
Automation for Issue 41202: Combination of Submitter and Reader Role does not see the "Insert Rows" icon.
This is only a draft pull request more tests are still being written. This is start an early review of the migration from a StudyBaseTest to a BaseWebDriverTest.
Related Pull Requests

https://github.com/LabKey/testAutomation/pull/672

Changes

Migrated test from StudyBaseTest to BaseWebDriverTest
Modified test to use StudySecurityForm.

Need to close an recreate it to merge into 21.3-SNAPSHOT.


Unzip Method

Custom unzip working. Suggestion from Datalad people: check out a branch, then do git annex add archive contents into that branch. TODO: Test this on PNC qsirecon for Audrey
bootstrap to create hdf5 of hdf5 files for each xcp output only with certain files unzipped with demographics csv + dictionary
INCLUDES:
ptseries, desc-qc, desc-frame

Ted Satterthwaite 10 Jan at 13:33
priority! ok to ask Yarik.

Ted Satterthwaite 18 Jan at 14:14
sounds awesome:

need docs for unzipping other files -- like ciftis-- for project user
need docs for loading hdf5 into R + pennlinckit

Matthew Cieslak 7 Feb at 11:48
Hanke's group knows how to do this. We should try to implement it next week


Account balance discrepancy and incorrect proxy contract address retrieval via api

Describe the bug
The account balance is showing as zero, however, through ArgentX I can see that the account has a balance.
The address of the proxy contract implementation obtained through the API is incorrect.
To Reproduce
The account balance is showing as zero, however, through ArgentX I can see that the account has a balance.
`address = '0x065934Da171A56d0aee41866C656b89ddbD5653C44F49331803dAAA052d1aCCE'
const ethAddress = '0x049d36570d4e46f48e99674bd3fcc84644ddd6b96f7c741b1562b82f9e004dc7'
const contractClass = await provider.getClassAt(ethAddress)
const ethAbi = json.parse(JSON.stringify(contractClass.abi))
const erc20 = new Contract(compileERC20.abi, ethAddress, provider);
const balanceAfterTransfer = await erc20.balanceOf(address);
const result =  weiEthToEth(uint256.uint256ToBN(balanceAfterTransfer.balance).toString())
console.log("balance", result);
`
The address of the proxy contract implementation obtained through the API is incorrect.
`
const contractClassHash = await provider.getClassHashAt(tokenAddr)
console.log("contractClassHash", contractClassHash)
const {abi: testAbi} = await provider.getClassByHash(contractClassHash)
const testContract = new Contract(testAbi, tokenAddr, provider);
const {address: testAddress} = await testContract.implementation()
console.log("testAddress", testAddress)
const {abi: classAbi} = await provider.getClassAt(tokenAddr)
const erc20Class = await new Contract(classAbi, tokenAddr, account);
const {address: implAddress} = await erc20Class.implementation()
console.log("implAddress", implAddress, toHex(implAddress), tokenAddr)
// get implementation address: 0x58c891da6c3b1ed062db42db3c6ca080724fabbe3d5073660d02aab8edc3e95
// real address: 0x048624e084dc68d82076582219c7ed8cb0910c01746cca3cd72a28ecfe07e42d
Expected behavior
A clear and concise description of what you expected to happen.
Screenshots
If applicable, add screenshots to help explain your problem.
Desktop (please complete the following information):

Browser & version [e.g. chrome]
Node version [e.g. 20.0.1]
Starknet.js version 5.17.1
Network [mainnet,]

Additional context
Add any other context about the problem here.

Hello,
I think that you should improve your understanding of the Starknet architecture, because you have some confusions about some definitions :
The ETH contract is behind a  proxy. So you have to understand how it works :

A class hash : https://docs.starknet.io/documentation/architecture_and_concepts/Contracts/contract-classes/
A contract address : https://docs.starknet.io/documentation/architecture_and_concepts/Contracts/contract-address/
So,

You have first to use getClassAt() to have the abi of the proxy.
Then call the function that provide the implemented class hash.
Use getClassByHash() to have the abi of the ERC20 contract.
You create the contract instance with the address of the proxy and the abi of the class implemented.
Finally, you can use call/invoke to interact with the ERC20.

Hello, I think that you should improve your understanding of the Starknet architecture, because you have some confusions about some definitions : The ETH contract is behind a proxy. So you have to understand how it works :  A class hash : https://docs.starknet.io/documentation/architecture_and_concepts/Contracts/contract-classes/ A contract address : https://docs.starknet.io/documentation/architecture_and_concepts/Contracts/contract-address/
So,

You have first to use getClassAt() to have the abi of the proxy.
Then call the function that provide the implemented class hash.
Use getClassByHash() to have the abi of the ERC20 contract.
You create the contract instance with the address of the proxy and the abi of the class implemented.
Finally, you can use call/invoke to interact with the ERC20.

In case of such problem, do not hesitate to contact us in our Discord : https://discord.com/channels/793094838509764618/927918707613786162

good work!

Adding it to the official documentation would be best @PhilippeR26

The guide explaining how to create a ArgentX account is based on this principle : https://www.starknetjs.com/docs/next/guides/create_account#create-an-argent-account

Hello, I think that you should improve your understanding of the Starknet architecture, because you have some confusions about some definitions : The ETH contract is behind a proxy. So you have to understand how it works :  A class hash : https://docs.starknet.io/documentation/architecture_and_concepts/Contracts/contract-classes/ A contract address : https://docs.starknet.io/documentation/architecture_and_concepts/Contracts/contract-address/
So,

You have first to use getClassAt() to have the abi of the proxy.
Then call the function that provide the implemented class hash.
Use getClassByHash() to have the abi of the ERC20 contract.
You create the contract instance with the address of the proxy and the abi of the class implemented.
Finally, you can use call/invoke to interact with the ERC20.

In case of such problem, do not hesitate to contact us in our Discord : https://discord.com/channels/793094838509764618/927918707613786162

Based on the code below, the address I obtained is 0x58c891da6c3b1ed062db42db3c6ca080724fabbe3d5073660d02aab8edc3e95, but the expected address is 0x48624e084dc68d82076582219c7ed8cb0910c01746cca3cd72a28ecfe07e42d. What could be the issue?
javascript
Copy code
const ethAddress = '0x049d36570d4e46f48e99674bd3fcc84644ddd6b96f7c741b1562b82f9e004dc7';
const ethHashAddress = "0x00d0e183745e9dae3e4e78a8ffedcce0903fc4900beace4e0abf192d4c202da3";
const contractClass = await provider.getClassAt(ethAddress);
const classContract = new Contract(contractClass.abi, ethAddress, provider);
const {address: implAddress} = await classContract.implementation();
console.log("impl address", implAddress.toString(16));

As it's not related to a Starknet.js bug or proposal of improvement, could you please use Discord media :

As it's not related to a Starknet.js bug or proposal of improvement, could you please use Discord media :

I don't have permission to access this link.

So, register  in Discord : https://discord.com/
Then to Starknet Discord : https://discord.gg/qypnmzkhbc

So, register in Discord : https://discord.com/ Then to Starknet Discord : https://discord.gg/qypnmzkhbc

I just registered an account and cannot accept the invitation.. Error: code: 40007  message: 该用户已被此服务器封禁.


Initial gradle plugin subproject

Still needs a bit more work/cleanup to get it where i'd like it, but here's a good first revision.
Can we diverge from libgdx version numbers? Having this library update much more frequently for minor things and non-breaking feature change would be quite nice.
Initial publish is very barebones, but functional. ./gradlew publishToMavenLocal is enough to test for now.

I don't see any reason the version needs to match libgdx if this could benefit from a different release cycle.

With the latest Android.mk/Applicaiton.mk support this should now be up to feature parity with libdgx base java based builds.
I realized I should probably ask if I should be adding new features before or after this gets merged. Things like the jnigenBuild commands which actually invoke the builds, jar packaging tasks or clean tasks.

I'm okay with adding features. I would like @Tom-Ski to take a peek at this PR before merging.

Not sure if we're going to get a review from the grey Tom-Ski here, so let me know when you want to merge.

Hey, yeah, i did a little look. But wanted to restructure some of the stuff for deployment.

I'm still not quite happy with a few things:

Jar tasks (and therefore publishing) are clunky, let alone missing IOS support.
macOS only builds are clunky, jnigenBuild should prob be smarter to only run current platform compatible targets instead of requiring you to manually run IOS/MacOS tasks.
macOS makes desktop deploy, because natives-desktop is a single archive. It requires you to manually figure out a mechanism to copy over macOS builds if you aren't cross compiling.
(The only way I can see to maybe make this better would be to split desktop like LWJGL3's new classifiers)
Clean task for jnigen artifacts. (Deleting old no longer used javah h/cpp files seems unsafe and potentially destructive if people put their main h/cpp files in jni folder)

Things which I want to try, but probably don't belong in the initial commit:

Move c/cpp includes to proper src/main/c src/main/cpp folders by default.
Move generated build scripts from the jni folder to a proper gradle build/jni or something.
Upgrade javaparser to latest now that we're allowed to use java 8.

Oh also, there's the SharedLibraryLoader subproject I want to add, but there's a limitation noted in https://github.com/MobiDevelop/gdx-jnigen/issues/2

Sounds good, I'll merge then move this repo to the libgdx org later today.


filter forwarded headers

Resolves a 404 error caused by x-forwarded-* headers set by a reverse proxy.
I didn't include all standard headers but this should cover everything important.

 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.


fix: fixes #2736

rw [id] is now elaborated as rw [@id]
Add Rewrite.Config.newGoals field for specifying how new rw goals are ordered. We use the same approach used in the apply tactic.
Add support for optParam to rw and apply tactics.

fixes #2736

The downstream failure is due to a regression from this PR that we should fix:
axiom P : Prop
axiom p : P
axiom f : P → Nat
axiom Q : Nat → Prop
axiom q : ∀ n, Q n
theorem r (p : P := p) : 0 = f p := sorry
example : Q 0 := by
  rw [r]
  apply q
  -- Used to work, but now: `unsolved goals: ⊢ optParam P p`

This is a pretty common pattern in Mathlib, where a side condition for a rewrite rule is specified with an optional value.

This is a pretty common pattern in Mathlib, where a side condition for a rewrite rule is specified with an optional value.

We have conflicting requests. In some situations, users want the implicit arguments consumed (e.g., example above). In other situations, they don't want because the implicit parameter causes a failure (e.g., associated issue).
@semorrison Any suggestions?
BTW, we can keep things as-is, and just tell users to use @ in the original issue.

I don't immediately see what #2736 has to do with optParam insertion, as there are no optParams in the example and it is instead failing with a typeclass inference failure. (Not saying there is no connection, I just don't see why these two cases are entangled.)

I don't immediately see what https://github.com/leanprover/lean4/issues/2736 has to do with optParam insertion, as there are no optParams in the example and it is instead failing with a typeclass inference failure. (Not saying there is no connection, I just don't see why these two cases are entangled.)

Common theme: implicit arguments.
 That is, both treat it as rw [@foo] and simp [@foo]. With the new behavior, implicit arguments are not processed when elaborating the tactic arguments. We have been doing that in simp, but not for rw.

@semorrison Failure above seems to be a CI issue.

No, genuine failures due to this PR. I'll start minimizing, they don't look too hard.

Many of the Mathlib failures are because the side conditions coming from rw have changed order.
e.g.
def Set (ι : Type) := ι → Prop

namespace Set

def univ : Set ι := fun _ => True
def empty : Set ι := fun _ => False

def pi {ι : Type} {α : ι → Type} (s : Set ι) (t : (i : ι) → Set (α i)) : Set ((i : ι) → α i) := sorry

theorem univ_pi_eq_empty (ht : t i = empty) : Set.pi univ t = empty := sorry

example (i : ι) (h : t i = empty) : Set.pi univ t = empty := by
  rw [univ_pi_eq_empty]
  · exact h

used to work, but on this toolchain it requires
  rw [univ_pi_eq_empty]
  · exact i
  · exact h

The Mathlib testing PI is https://github.com/leanprover-community/mathlib4/pull/8076

@semorrison Any action items for me?

@semorrison Any action items for me?

@leodemoura yes, see my comment immediately above, with a MWE showing a regression due to this PR. The order of side conditions generated by rw has changed.

@leodemoura yes, see my comment immediately above, with a MWE showing a regression due to this PR. The order of side conditions generated by rw has changed.

@semorrison I saw it, but you did not make it clear whether the new order is the wrong one or not. We have regressions all the time. Note that we currently don't have newGoals := ApplyNewGoals.nonDependentFirst in the config object for the rw tactic like we do for apply. Of course, we can add it and make nonDependentFirst the default.

I'm not sure (I haven't thought carefully about it) which ordering is the best in an absolute sense, but I know that any change to the order of generated subgoals of rw in its default mode will result in massive breakage in mathlib (estimated >4000 use sites) and will be a major task to adapt without some ad hoc migration tooling.

Ah, I see. I had thought it was clear that nonDependentFirst would be the preferred default for users and/or tactics.
That said, I don't think this is as impactful as Mario says above. Note in particular that it manages to compile about 80% of Mathlib without encountering this change, so I suspect it could be done by hand if desired.
The change that has occurred is not changing the relative order of the main goal and the introduced side goals, it is only changing the order of the side goals amongst themselves.
My preference would be that we change the behaviour to restore the ordering as before. But unless I hear otherwise I will patch Mathlib for the new behaviour, on leanprover-community/mathlib4#8076, hopefully tomorrow.

@semorrison

My preference would be that we change the behaviour to restore the ordering as before. But unless I hear otherwise I will patch Mathlib for the new behaviour, on https://github.com/leanprover-community/mathlib4/pull/8076, hopefully tomorrow.

Note that the previous order was not nonDependentFirst. It just turns out that implicit arguments are often dependent ones.
Just to clarify, given a function foo where x is the first explicit argument, rw [foo] was using the following order: all for arguments starting at x, followed by all for implicit arguments. In the example, you see the illusion of nonDependentFirst because the dependent argument is implicit.
Happy to hear you are willing to fix any breakages this change may produce.

Ah, okay, that description of the goal ordering makes sense, and I agree it's not something worth preserving as is.
Shall I fix Mathlib to work with this PR as is, and we can think about moving to dependendFirst later, or did you want to do that in this PR?

Shall I fix Mathlib to work with this PR as is, and we can think about moving to nonDependendFirst later, or did you want to do that in this PR?

I will take a look next weekend. I think it is better to try to integrate nonDependentFirst now, to avoid more breakage in the future.

@leodemoura, are you still planning on implementing nonDependentFirst here? If not we need a new plan here. (Either change the behaviour in this PR so the order of side conditions is preserved, or start the Mathlib adaptation.)

I have mathlib working again. It is not too bad, except for a very large number of rws by lemmas (mostly in the linear algebra library), where we now get errors of the form "can't synthesize Ring ?R", and which can by solved either by rw [my_lemma (R := R)] or rw [my_lemma ..].
This is because of the change so rw [my_lemma] means rw [@my_lemma]. I'm not certain yet how painful people will find this change.

The regressions in Mathlib seem very similar to the issue reported in #2736. Is this change in Lean actually an improvement?
Do you know what causes the current rw to fail to find instances, and in what situations the new rw will fail to find instances? Can we find a way to get the best of both worlds (e.g. an algorithm for rw [foo] that will succeed when either rw [@foo] or rw [foo ..] succeeds)?

@leodemoura, here is the promised minimization:
/-!
This is a minimization of an issue widely seen in Mathlib after
https://github.com/leanprover/lean4/pull/2793.

Issue in the mwe above has been fixed by #4646


[NCL-8715] Unify toolbar button positions

Key changes:

All toolbar buttons are set to align-right
Change ToolbarItem ('@patternfly/react-core') to ToolbarItem ( 'components/Toolbar/ToolbarItem' ) anywhere using Toolbar from 'components/Toolbar/Toolbar',

Hi @patrikk0123

BuildHistoryList component still uses PF ToolbarItem. LogViewer still uses both PF Toolbar and ToolbarItem. `

BuildHistoryList was changed to our ToolbarItem. But I kept LogViewer with PF Toolbar and ToolbarItem since our toolbar makes it stack into one column; I believe there are some specific changes been made for log viewer that work for the pdf toolbar only.

Is toolbar title and description unification (their presence and alignment) part of another Jira? For example, BC Dependencies and Dependants lists align them differently.

I would insist on unifying the ToolbarItem with PNC ToolbarItem in this jira unless necessary(like log viewer), because I found some inconsistent usage of ToolbarItem on the same level of page in different components, which doesn't make sense. Let me know if there is an existing ongoing JIRA that conflicts with this change.
Thanks!

@DnsZhou

I would insist on unifying the ToolbarItem..

I have no problem with that. What I meant is that some Toolbars miss title, and layout of the title and description is different on some places (BC Dependencies vs Dependants lists).

Hi @patrikk0123

I have no problem with that. What I meant is that some Toolbars miss title, and layout of the title and description is different on some places (BC Dependencies vs Dependants lists).
I have unified all toolbars to make the title and the description in same TextContent.

This PR is replaced by https://github.com/project-ncl/pnc-web-ui-react/pull/560


fix: Update build

Summary
This update adds tree-shaking to the JS build so that unused code paths are removed. This removes references to unused files. The terser plugin was also added to decrease the size of the source.

Upgraded node version in .node-version
Upgraded yarn
Removed package-lock.json
Fixed VSCode rust-analyzer settings
Updated GitHub workflows
Updated README
Updated rollup config
Added wasm-pack and @rollup/plugin-terser dependencies
Renamed package.js to sqlite3-diesel.js
Added test scripts
Added tree-shaking and terser plugin for output

Summary by CodeRabbit
Release Notes

New Features

Updated workflow configurations for better performance and clarity in CI/CD processes.
Introduced new test scripts for comprehensive browser testing, including headless modes.

Bug Fixes

Corrected paths and dependencies in the README for improved clarity and consistency.

Documentation

Enhanced README with clearer instructions and corrected examples.

Chores

Updated Node.js version and Yarn configuration for improved compatibility and performance.
Streamlined JavaScript code for better readability and maintainability.

Style

Modernized JavaScript syntax in the async proxy file for enhanced clarity.

@insipx is there anything that we need to do in this crate for sqlite3.wasm to be part of the files copied over or is that something we need to do in xtask?
with these changes, sqlite3.wasm is still referenced in the code and is required for production builds, but it's never actually accessed.

@insipx is there anything that we need to do in this crate for sqlite3.wasm to be part of the files copied over or is that something we need to do in xtask?

with these changes, sqlite3.wasm is still referenced in the code and is required for production builds, but it's never actually accessed.
@rygine need to copy sqlite3.wasm in the xtask in libxmtp for now. I'll create a ticket for a better solution sometime, maybe with a patch to the sqlite repo


M31298: Removing extra word "and"

Hello, @sptramer,
Translator has reported possible source content issue.
Description: "and" seems to be unnecessary and translator ignored it in the target text.
Please review and merge the suggested proposed file change to avoid this error. If you make related fix in another PR, then share your PR number, so we can confirm and close this PR.
Many thanks in advance.

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

azure-go-sdk-conceptual/azure-sdk-go-tools.md
:white_check_mark:Succeeded
View

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

This PR is not correct; the action vendor is a verb here, describing the process of "vendoring requirements." See https://en.wiktionary.org/wiki/vendor, Verb defn. 1. If there is a more appropriate method of stating this for translation purposes please let me know, but this is a standard English term for software management.

Thanks @sptramer !
I will let the Translator teams know.
Regards.


Documentation about OAuth2

Now that new version released, it's good to add a documentation about how to implement OAuth2, I think.
Close this issue if it is implemented in existing functions/library (Don't close this issue without leaving a comment).

There's comprehensive Spotify OAuth documentation at https://developer.spotify.com/documentation/web-api/concepts/authorization as well as our documentation at https://github.com/librespot-org/librespot/wiki/Options#oauth and https://github.com/librespot-org/librespot/wiki/Options#access-token. You can see for yourself  we use an existing "oauth2" library, it is well documented.
I consider this issue closed but if you have a concrete documentation enhancement request then feel free to make it.

Thanks.

This is command-line arguments.
I mean code, not CLI

https://docs.rs/oauth2/latest/oauth2/


Add section on continuous integration

Currently this is pretty sad.

I'm working on a tool for "local" CI stuff right now, so have been thinking about this.  Are you interested in what happens on the dev desktop (some process watches for code changes, and automatically runs some process when changes are made), or strictly on 3rd party tools like Travis?

I'm thinking about continuous integration as a process run on pull requests before they get merged.
Travis is a good example but there's many others.
For now I'd like to keep it as simple as possible, explaining why this is important, and giving example of the most common tools.


problem with bot 1

Hello,
first of all thank you for your work.
then I would like to report that I had a small problem in using the bot indeed when I run bot 1 for text spam and enter the text in sapm it displays the following error (see screen).
is this a bot bug and did I do something wrong?
Thanks in advance for your reply.

Running proxies?

Vous exécutez des proxies?

yes i use HTTP proxies

What’s your proxies format?

Quel est votre format de proxy?

i use http prixies, I put them in proxies.txt

The format should look like this.
<IP_ADDRESS>:80
<IP_ADDRESS>:80
<IP_ADDRESS>:80
<IP_ADDRESS>:80
<IP_ADDRESS>:80
<IP_ADDRESS>:80

The format should look like this.
<IP_ADDRESS>:80
<IP_ADDRESS>:80
<IP_ADDRESS>:80
<IP_ADDRESS>:80
<IP_ADDRESS>:80
<IP_ADDRESS>:80

yes I use this format well

Has anyone solved this issue? I also have the same issue.

Quelqu'un a-t-il résolu ce problème? J'ai aussi le même problème.

no we have had no solution for the moment, I hope that will be settled quickly


TypedFacilityFactory component selection does not work with interfaces

I'm trying to install an interface as an factory and then override the component selector class so that I would have separate component selector classes for different projects in my solution.
container.Register(
Component.For().ImplementedBy()
Component.For().AsFactory(f => f.SelectedWith());
And then on other project I'm trying to override the component selector:
container.Register(
Component.For().ImplementedBy());
This causes the factory instance to be null at runtime. Seems like the component selector is not resolved and needs to be a concrete class. Interface should also be supported here. If I use concrete class for the SelectedWith parameter, then it works:
container.Register(Component.For().AsFactory(f => f.SelectedWith());

I'm kind of confused by this issue because of the "pseudo" code. @Rahkola are you still around for clarifications?
I just had a quick look in the code. My first understanding is that the facility should already be resolving component selectors from the container, so this might just be a bug.
I'll try to raise a PR with a failing test for the case as originally described.

This causes the factory instance to be null at runtime.

This is weird in its own right, as the test I've just written throws an exception saying "The referenced component {ISelector} could not be resolved". To end up with null, something else must be going on in the original scenario.

Hi,
I'm still around :) This happened in somewhat more complex enterprise app that I was developing. Can't recall the actual case anymore, but I can try to re-produce the issue with more isolated environment.  It may be that the problem was caused by some other reason as you are suggesting.
I'm a bit busy with other projects at the time, but Ill try to find time to provide you isolated code which re-produces the issue. No promises though... If I don't come back to you in couple of weeks with the example code the ticket can be closed on by behalf.


Fix edge case in Cypress flaky tests "Cannot read properties of null …

…(reading required)" when the schema action has not been fulfilled yet (for some reason) and the test tries to save.
I think I've found the cause of it... let's see.
Another option would be that the reducer always return an empty object... @robgietema any reason why we opted to return null if the reducers are empty?

Hmm, now there is a different error that sounds like it has the same cause:
CypressError: Timed out retrying after 5000ms: `cy.wait()` timed out waiting `5000ms` for the 1st request to the route: `schema`. No request ever occurred.`

So maybe the problem is that something is preventing requesting the schema in the first place?

@davisagli I've seen this one too, schemas that do not load (or that take far too long in the test?) No clue why could that be happening.

LGTM


Ability to handle folder paths in BACKUP_FILENAME

It would be nice to be able to handle the BACKUP_FILENAME containing a relative folder path.
In combination with uploading to S3 this would let me upload into a folder structure.

Old issue, but this is working at least as of now: if you specify a bucket + path in the AWS_S3_BUCKET_NAME env var, it will create that folder (if it doesn't exist) and store the named backup file in it. Note: the bucket itself does have to exist! but the folder will be created.
Example:
    environment:
      AWS_S3_BUCKET_NAME: "your-bucketname/wordpress-sites"
      BACKUP_FILENAME: wp-backup-latest.tar.gz

... will create wordpress-sites folder under your-bucketname, and place your wp-backup-latest.tar.gz in it.

Thanks for pointing this out, I've now included this tip in the docs (b2ee77ce14ed61796d5cc5285dde46ddfbed8870)!


Introducing licenses.json to account for unknown/missing license names in packages

This is a list of licenses in JSON format, to be used when packages being scanned do not provide license types, only URLs.  In that case we can find license name by URL reference, if exists.  This is not a final/comprehensive list, but could be a good starting point.

@vr333dev Please sign the Contributor License Agreement!
Click here to manually synchronize the status of this Pull Request.
See the FAQ for frequently asked questions.

@xtreme-shane-lattanzio ok PR created... not sure if I should "Update branch" (above) and/or required to sign Contributor License Agreement?
Thanks

@vr333dev Thank you for signing the Contributor License Agreement!

Hey @vr333dev . The list looks good. Will there be code to actually use this list or is it intended that an application that uses license finder would have that logic?

Hello, I understood that LicenseFinder returns license names as output, in which case it would be a change in LicenseFinder logic (no one on our team is ruby "literate", unfortunately).  But if I should create a GitLab ticket in order to diagnose License-Compliance job, and see if license names are somehow derived in a GitLab template or a container, then I will need to take a different turn to resolve the issue we are experiencing.

@vr333dev What I thought we were talking about was when a nuget scan was done, the name was not found but the URL can be. In this case I would expect logic in LF to take that URL and use the newly created JSON to assign the name correctly so that the output is not unknown. I would expect that this logic would only be used for nuget at this point and then potentially expanded in the future. Either way, LF would need to use the new json in some way to fill in missing information.
Did I understand that correctly?

Our pipelines were having issue which we seemed to have fixed this morning. So your PR is currently being validated in our pipelines.

@vr333dev I am fine with merging this json but since it is not used anywhere, I am not sure of the point. Is the code being built to use this json going to be in LF or will it be in your own repos? Either way we can have this but it may be helpful to drop a note in the readme about this or it will just be forgotten

@xtreme-jason-smith this JSON is meant to be absorbed by the LF itself (if possible), since there is nothing I see us accomplishing with it when License-Compliance task runs in GitLab CI (where license names are being looked up by LF).  Unless there are other ideas?

@vr333dev Sorry for taking so long to get back to you. This makes sense yes but I am wondering if it will be part of this PR or a separate one?

Hey @vr333dev ! Im still wondering if a change should be done for this or just a readme update. If I don't hear from you in the next little while, I'm going to go ahead a close this for now so let me know what you think!

Closing due to inactivity


[jest-jasmine2]: Remove throwing an error when return value is not a promise or undefined

Summary
Fixes: #6516
Test plan
Tests should all still pass

We can merge it once an eslint rule exists for this.

@cpojer @aaronabramov the eslint rule: https://github.com/jest-community/eslint-plugin-jest/pull/136

Can you enable it inside jest tests?

What about returning a symbol?

This has stalled


Fix Power Cloud endpoint for multi-zone regions

The 'region' value contains a zone enumeration in regions with more than
one zone (e.g. 'eu-de-1'). A single API endpoint is provided for each
region regardless of zone count (e.g.
'https://eu-de.power-iaas.cloud.ibm.com').

This seems like an inconsistency in the ResourceController, it returns us-east and us-south for the region_id for those two but for this it seems to be returning a zone rather than a region right?
I'm okay with this as a workaround but would like to understand if there is a fix for the root cause

This seems like an inconsistency in the ResourceController, it returns us-east and us-south for the region_id for those two but for this it seems to be returning a zone rather than a region right?

You're right, it is inconsistent. The Resource Controller API returns only region_id, which is only "documented" in the example response. I'm guessing that the zone is provided through the region_id for lack of a zone key.

I'm okay with this as a workaround but would like to understand if there is a fix for the root cause

I'll follow up with the API team and come back with an outlook on if/when this might get fixed.

Released v0.1.2 with this change


Claude 2 Anthropic Support

It would be awesome to support claude 2! the 100k context window is amazing.
You could implement this and solve it across many LLMs: https://github.com/BerriAI/litellm

I'm the maintainer of liteLLM - happy to make a PR and add an integration😊

ChatLiteLLM() is integrated into langchain and allows you to call all models using the OpenAI I/O interface
https://python.langchain.com/docs/integrations/chat/litellm


New post: flapi.sh: a tiny command-line tool for experimenting with the Flickr API

🚀 Deployed on https://66185ebb63a0473f11be54b5--alexwlchan.netlify.app


Error: Remote operation failed.

Error:
"[GlueHomebridge] Error: Remote operation failed.
at /usr/lib/node_modules/@gluehome/homebridge-gluehome/src/lock.ts:121:19
at runMicrotasks ()
at processTicksAndRejections (internal/process/task_queues.js:93:5)"
Node.js v14.16.0
Npm v6.14.11
The HomeBridge plugin fails all times with command fails (you can see in the Error part), the same message you can see in the Glue App, but Lock and Unlock works fine

An getting this error
ERROR[Hoobs HUb 33F1@Front Door @@Lock Target State] characteristic was supplied illegal value: number 3 exceeded maximum of 1

I believe this is now resolved, in the last update. If you continue to see this issue, please reopen.


Data-preprocessing like SECOND

Hi, it was great work.
I saw that in nuscenes_util.py you changed pre-processing and evaluation methods from det3d and second.
But I need pre-processing like exactly SECOND, what is the fast and easy way to make it using your code? Maybe you have some configurations to change it ?! I just need images dict including their points and masks.
Is it possible? Will it create some problem to the rest of training/evaluation settings?

You could change the parts of data preparation to SECOND, and it will not create problems for the rest training and evaluation.
But I didn't have a configurations to generate exactly the same data with SECOND, and you need to check it by yourself.

I understand, thank you very much.


blog: sentiment-analysis-with-node-red

Description
I have created pull request for sentiment analysis with node-red blog post

Related Issue(s)

Checklist

[ ] I have read the contribution guidelines
[ ] I have considered the performance impact of these changes
[ ] Suitable unit/system level tests have been added and they pass
[ ] Documentation has been updated

Hi @ZJvandeWeg, I have created a pull request. Could you please review it and let me know if any changes are required?


images break html format in tables

best solution might be a image format.
extending html format to handle this seems tricky.

@scottdraves
I've searched a little on supporting imgs in HTMLCellRenderer and imho it's too complex to get this working:

Async rendering the HTML on canvas when the image loads is tricky see (https://github.com/twosigma/beakerx/issues/7598),
I tried to render the svg with loaded image into the canvas and the result was the empty cell. I don't event know if this is possible to render into canvas this way.

So i think we should implement the image format and render the image with drawImage canvas method.