Inaccurate for OpenAI's cookbook example

Great work on this! I found one edge case where the local token calculation seems to be off by a couple tokens compared to the API response. This example is from OpenAI's cookbook example:
      messages: [
      {
        role: 'system',
        content:
          'You are a helpful, pattern-following assistant that translates corporate jargon into plain English.',
      },
      {
        role: 'system',
        name: 'example_user',
        content: 'New synergies will help drive top-line growth.',
      },
      {
        role: 'system',
        name: 'example_assistant',
        content: 'Things working well together will increase revenue.',
      },
      {
        role: 'system',
        name: 'example_user',
        content:
          "Let's circle back when we have more bandwidth to touch base on opportunities for increased leverage.",
      },
      {
        role: 'system',
        name: 'example_assistant',
        content:
          "Let's talk later when we're less busy about how to do better.",
      },
      {
        role: 'user',
        content:
          "This late pivot means we don't have time to boil the ocean for the client deliverable.",
      },
    ],
    tokens: 129,

The result is:

Expected: 129 (from examples above & validated against OpenAI's API response)
Received: 121 (from local token calculation)

Root Cause
OpenAI's algorithm shows that you're supposed to add one for each name field
If you break it down into individual test cases it seems to show that:
{
    messages: [
      {
        role: 'system',
        content:
          'You are a helpful, pattern-following assistant that translates corporate jargon into plain English.',
      },
    ],
    tokens: 25,
  },
  {
    messages: [
      {
        role: 'system',
        name: 'example_user',
        content: 'New synergies will help drive top-line growth.',
      },
    ],
    tokens: 20, // local calc 18
  },
  {
    messages: [
      {
        role: 'system',
        name: 'example_assistant',
        content: 'Things working well together will increase revenue.',
      },
    ],
    tokens: 19, // local calc 17
  },
  {
    messages: [
      {
        role: 'system',
        name: 'example_user',
        content:
          "Let's circle back when we have more bandwidth to touch base on opportunities for increased leverage.",
      },
    ],
    tokens: 28, // local calc 26
  },
  {
    messages: [
      {
        role: 'system',
        name: 'example_assistant',
        content:
          "Let's talk later when we're less busy about how to do better.",
      },
    ],
    tokens: 26, // local calc 24
  },
  {
    messages: [
      {
        role: 'user',
        content:
          "This late pivot means we don't have time to boil the ocean for the client deliverable.",
      },
    ],
    tokens: 26,
  },

Making that change (to add a token for every message.name) fixes those tests, but then breaks these ones:

Final tweaks
So all said, it seems if there's a function then you need to subtract 2:
  if (message.name) {
    tokens += 1 // matches OpenAI's cookbook
  }
  if (message.role === 'function') {
    tokens -= 2
  }
  if (message.function_call) {
    tokens += 3
  }

Good find! Thanks for explaining in so much detail. I was aware that the name parameter was one of the few cases I hadn't tried out yet so it makes sense that it was off.
Feel free to submit the tests and fix in a PR, or if you'd rather I can easily copy the changes in directly.

Happy to submit a PR as a thanks! Will do tomorrow

Thanks for the PR! Just released in v0.2.5


[question] How to use py::bool_ in nanobind?

Hi,
I need to create something like this: https://github.com/pybind/pybind11_json
for nanobind and for rapidjson.
I fail to understand how to construct a "bool" binding object with nanobind.
How to create a bool object currently?
Also as I saw "::float_" was also dropped in favor of "::int_", am I getting it right?
Thanks,

Please don't open bug tickets for questions, we have a discussion tab for this.


Migrate dart:wasm and tests to NNBD

Sub issue of #38037 .

Migration instructions here.

Done: https://dart-review.googlesource.com/c/sdk/+/131421

Hi, in that CL, I see the code was migrated, but not the tests. All of the tests at "tests/lib_2/wasm" need to by copied to "tests/lib/wasm" and also made NNBD clean.
Leaf sent out a doc with instructions on how to migrate tests. I'm working to improve that doc. In the meantime, let me know if you have any questions.

I think the tests are already NNBD clean. I'll copy them to tests/lib/wasm

SGTM, thanks!


shared_preferences cannot run in background (not registered)

Hello,
Problem
I'm running a background-service when receiving a push notification. It seems like the local_notifications-plugin is not registered when the App is running in Background:
MissingPluginException(No implementation found for method getAll on channel plugins.flutter.io/shared_preferences)
It actually works perfectly fine when running in Foreground and the App is active.
Attempt to fix the problem
I tried to register the Plugins on Application-Level in the "Application.kt":
 GeneratedPluginRegistrant.registerWith(registry);
but I receive an error when doing so:
Type mismatch: inferred type is PluginRegistry? but FlutterEngine was expected
Flutter doctor output (no errors)
Flutter doctor -v outputs no errors:
`[✓] Flutter (Channel beta, 1.19.0-4.3.pre, on Mac OS X 10.15.5 19F101, locale de-CH)
• Flutter version 1.19.0-4.3.pre at /Users/marinoschmid/Documents/DevTools/flutter
• Framework revision 8fe7655ed2 (5 days ago), 2020-07-01 14:31:18 -0700
• Engine revision 9a28c3bcf4
• Dart version 2.9.0 (build 2.9.0-14.1.beta)
[✓] Android toolchain - develop for Android devices (Android SDK version 29.0.3)
• Android SDK at /Users/marinoschmid/Library/Android/sdk
• Platform android-29, build-tools 29.0.3
• Java binary at: /Applications/Android Studio.app/Contents/jre/jdk/Contents/Home/bin/java
• Java version OpenJDK Runtime Environment (build 1.8.0_242-release-1644-b3-6222593)
• All Android licenses accepted.
[✓] Xcode - develop for iOS and macOS (Xcode 11.5)
• Xcode at /Applications/Xcode.app/Contents/Developer
• Xcode 11.5, Build version 11E608c
• CocoaPods version 1.9.1
[✓] Chrome - develop for the web
• Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome
[✓] Android Studio (version 4.0)
• Android Studio at /Applications/Android Studio.app/Contents
• Flutter plugin version 46.0.2
• Dart plugin version 193.7361
• Java version OpenJDK Runtime Environment (build 1.8.0_242-release-1644-b3-6222593)
[✓] VS Code (version 1.46.1)
• VS Code at /Applications/Visual Studio Code.app/Contents
• Flutter extension version 3.12.1
[✓] Connected device (3 available)
• SM A202F   • RF8M80Z5DBY • android-arm64  • Android 10 (API 29)
• Web Server • web-server  • web-javascript • Flutter Tools
• Chrome     • chrome      • web-javascript • Google Chrome 83.0.4103.116
• No issues found!`
Platform
Error occured on Android.
Other Ideas
Somehow register the Plugin in the Application.kt should fix the problem, but I don't achieve this action and it doesn't seem to be documented. My Application.kt (shouldn't be important, but maybe it helps):
`
class Application : FlutterApplication(), PluginRegistrantCallback {
    override fun onCreate() {
        super.onCreate()
        FlutterFirebaseMessagingService.setPluginRegistrant(this);
        FlutterMain.startInitialization(this)
    }

    override fun registerWith(registry: PluginRegistry?) {
        if (!registry!!.hasPlugin("io.flutter.plugins.firebasemessaging")) {
            FirebaseMessagingPlugin.registerWith(registry!!.registrarFor("io.flutter.plugins.firebasemessaging.FirebaseMessagingPlugin"));
        }

        if (!registry!!.hasPlugin("com.dexterous.flutterlocalnotifications")) {
            FlutterLocalNotificationsPlugin.registerWith(registry!!.registrarFor("com.dexterous.flutterlocalnotifications.FlutterLocalNotificationsPlugin"));
        }

    }
}

`
Edit: Formatting + Platform

Can be fixed by adding
"        SharedPreferencesPlugin.registerWith(registry.registrarFor("io.flutter.plugins.sharedpreferences.SharedPreferencesPlugin"));
"
to your Application.kt


Improve context logging in plain console for parallel build

Currently (as of 4.0.1) the logging in the plain console (CI service/dumping output to a file) with --info enabled doesn't provide information about context (task) on behalf which a given line is executed. It makes it really hard to determine some issues when parallel execution (for instance problems with task rebuilding with Build Cache enabled - Appending inputPropertyHash for ... are mixed up). It's especially painful using CI server executor with 32 vCPUs...
Expected Behavior
Ability to clearly determine what task (or processing context in general) a given printed line is connected to. It could be a line prefix pointing to a task or even to a thread id or a worker number. Context switching would need to be marked in the second case to see where given thread/worked started/finished processing of a given unit of work.
Current Behavior
There is no easy way to connect a printed line to a task if parallel execution is enabled.
Your Environment
Gradle 4.0.1, Jenkins to execute Gradle jobs (can be anything with non interactive console)

You think that's the same issue as https://github.com/gradle/gradle/issues/2221 which we are currently working on. @eriwen Is that a correct assessment?

It seems to be another remedy for the same problem. As I understand correctly logs will be grouped by tasks (in execution phrase) giving the whole logs from a given execution put together (not mixed up with other tasks), right @eriwen? If yes, the line prefix would be probably unnecessary.

@szpak That's correct. The output would look similar to the rich console output task grouping.

That's correct @szpak. This will be (hopefully) a part of Gradle 4.2.

Given that, it seems that we should close this as a duplicate of #2221.


Tiles are not compatible with Mapbox Vector Tile v2 specification

This breaks rendering when using mapbox-gl-native.
This is due to mojodna/tilelive-tmsource#9. After forking the repository and adding<EMAIL_ADDRESS>and corresponding mapnik version as dependencies the tiles render correctly.

This breaks rendering when using mapbox-gl-native.

Oh oh this is a big one. So as of no the vector tiles we generate are not compatible with v2?!
I thought it is backwards compatible.
This is kind of a big one since would mean we need to rerender yet again which requires that @klokan would be willing to donate another infrastructure round.
At least it is now much faster thanks to #324.
But let' s not jump ahead. We will continue with what we have now so we can deliver.

This is due to mojodna/tilelive-tmsource#9. After forking the repository and adding<EMAIL_ADDRESS>and corresponding mapnik version as dependencies the tiles render correctly.

So we need to install it from https://github.com/hannesj/tilelive-tmsource?
Perhaps you can prepare a PR since you got it working 😁 🙇  ?

They are mostly compatible, only some elements aren't rendered or look strange. This is due the more stringent validity requirements by v2. (changes are described here)

This is kind of a big one since would mean we need to rerender yet again which requires that @klokan would be willing to donate another infrastructure round.

Just discussed with @manuelroth. We don't want to render again before the bachelor thesis is finsihed (middle of June) since it is important that we don't have many more moving parts at this point in time.
Let's for sure keep this open.

<EMAIL_ADDRESS>now generates v2 vector tiles. Thanks @hannesj for the PR (and apologies for not merging it sooner).

This also causes some rendering artefacts with recent versions of mapbox-gl-js (mapbox/mapbox-gl-js#1606)

I am keen to make one more complete world rendering, with diff updates turned on - and fixed this as well as other reported issues.
Would it be doable in the first half of July at latest, @lukasmartinelli? Then we can manage the world USB sticks for FOSS4G.

I am keen to make one more complete world rendering, with diff updates turned on - and fixed this as well as other reported issues.
Would it be doable in the first half of July at latest, @lukasmartinelli? Then we can manage the world USB sticks for FOSS4G.

Happy you are willing to sponsor another run. I would love to do that. Thank you.
I want to try and first integrate those few schema changes that were proposed and then start asap.
Do we now that NOW we are compatible for sure? Or do we need to create test extract and test it again with native clients?

Great! Yes, it would be good to test properly before we start rerendering.

@lukasmartinelli this PR https://github.com/osm2vectortiles/osm2vectortiles/pull/366 needs to be merged first before we are able to create MVT v2 compatible vector tiles.

Just a small note, as I don't know what hardware the initial import is done on, but these might be great budget-friendly dedicated servers for the osm2vectortiles project, as you can rent them for one week and they have 64 GB RAM:
https://www.runabove.com/sandbox-servers.xml
Or these ones with SSDs:
https://www.runabove.com/i7-game-servers.xml

@hyperknot I can't remember off-hand how many AWS VMs they are using, but I know they are all 8-core. I think it's 4 VMs.
The biggest issue is cores, and secondly storage. This is an embarrassingly parallel problem. You can scale this as large as you want to, but it is best to have a copy of the DB local to the workers (ie, on the same machine). The DBs need to be on SSD for good performance. RAM isn't a huge concern, I find 8GB is a decent sweet-spot for a few workers as Postgres is mostly using it for cache. The workers themselves use very little RAM.
So the most ideal setup would be many machines, lots and lots of cores, low RAM, and local SSD. You could do this easily with an adhoc cluster, but propagating the database to many machines could be a pain depending on your network and automation.

Just a small note, as I don't know what hardware the initial import is done on, but these might be great budget-friendly dedicated servers for the osm2vectortiles project, as you can rent them for one week and they have 64 GB RAM:
https://www.runabove.com/sandbox-servers.xml
Or these ones with SSDs:
https://www.runabove.com/i7-game-servers.xml

We are happy that @klokan is sponsoring the VMs needed for rendering OSM2VectorTiles.
However thanks for the tip, this sounds like a great offer to try my personal data projects.

@hyperknot I can't remember off-hand how many AWS VMs they are using, but I know they are all 8-core. I think it's 4 VMs.
The biggest issue is cores, and secondly storage. This is an embarrassingly parallel problem. You can scale this as large as you want to, but it is best to have a copy of the DB local to the workers (ie, on the same machine). The DBs need to be on SSD for good performance. RAM isn't a huge concern, I find 8GB is a decent sweet-spot for a few workers as Postgres is mostly using it for cache. The workers themselves use very little RAM.
So the most ideal setup would be many machines, lots and lots of cores, low RAM, and local SSD. You could do this easily with an adhoc cluster, but propagating the database to many machines could be a pain depending on your network and automation.

All correct assumptions @stirringhalo. We are using 16core - 64 GB machines (but RAM could be lower - we just want the 16 cores). And distribute it by copying the DB (also correct assumption) to a local attached SSD (no network filesystem).
For updating we only need 1 machine - for rendering we used 4 to 8 machines (depends how fast we want it rendered). For the next run I will probably scale it up to even more since we are a bit in a hurry.

Rendering finished.


Replaced all ansible-semaphore with semaphoreui

There where alot of references back to the old repo. Even if it works, it looks ugly.
I also added "--no-optional" to npm in the taskfile so development can be done on non-MacOSx hosts too.

It is very useful PR. Thank you!

Error in latest build:

Is that due to the test no clearing the go.mod between runs?

Hi @jriberg I closing this PR because it duplicates your newer PR https://github.com/semaphoreui/semaphore/pull/2357.


Update EDAM Terms

We need a way to propose new EDAM Terms if we find missing ones. I think for now we can add them to this issue.
We can follow up on integration later. The basic concept would be:

make a comment in the dev version in web protégé: https://webprotege.stanford.edu/#projects/69591619-4eda-4f03-9e7f-65b213038fe1/edit/Classes?selection=Class(<http://edamontology.org/topic_3168>)
get the concept approved by the community
make a PR to the ontology github: https://github.com/edamontology/edamontology

EDAM: Differential Community / Abundance

EDAM: long reads


Fix typo in footer

Fix typo
React is great.. But this is GraphQL! 👍
Cheers mate!

Legend thanks @SmolinPavel


Statefulset

@heathseals does this look correct? It seems to be working fine, but I get nervous about changing the API version


How does merge work?

I have this jsfiddle https://jsfiddle.net/ejyvpbwx/12/
I merge map1 with same values that are already in map1.
why is the result false, is it a bug?

It's because merge creates a new object. They may have the same contents, but they won't be reference equal (===) 🙂


Changing default client port to 7777 for release 0.2.1

Client port for users is defaulting to 50101 when it should be 7777.  I believe this is about local tests.

I'll add those changes to the master PR branch.  We should prioritize external users over local test.

bors r+

bors r+


Bones: Fix wrong world matrix for meshes attached to bone

Let's say mesh M1 is attached to bone B, and bone B is part of a skeleton used by mesh M2.
If M1 comes before M2 in scene.meshes, then M1 will use a world matrix from B that is lagging one frame behind compared to M2, because it's when M2 is processed that the world matrices of the bones are being recomputed (in Skeleton.prepare).
Repro: https://playground.babylonjs.com/#317BIH#37
If you click on "Slash 01", the sword is moved and a screenshot of the next frame is taken and displayed in the bottom right corner. You will see that the particle system (attached to the sword) is displayed at the previous location of the sword.
Creating the PR in draft mode for discussion.

Please make sure to label your PR with "bug", "new feature" or "breaking change" label(s).
To prevent this PR from going to the changelog marked it with the "skip changelog" label.

Please make sure to label your PR with "bug", "new feature" or "breaking change" label(s).
To prevent this PR from going to the changelog marked it with the "skip changelog" label.

Snapshot stored with reference name:
refs/pull/14185/merge
Test environment:
https://babylonsnapshots.z22.web.core.windows.net/refs/pull/14185/merge/index.html
To test a playground add it to the URL, for example:
https://babylonsnapshots.z22.web.core.windows.net/refs/pull/14185/merge/index.html#WGZLGJ#4600
Links to test babylon tools with this snapshot:
https://playground.babylonjs.com/?snapshot=refs/pull/14185/merge
https://sandbox.babylonjs.com/?snapshot=refs/pull/14185/merge
https://gui.babylonjs.com/?snapshot=refs/pull/14185/merge
https://nme.babylonjs.com/?snapshot=refs/pull/14185/merge
To test the snapshot in the playground with a playground ID add it after the snapshot query string:
https://playground.babylonjs.com/?snapshot=refs/pull/14185/merge#BCU1XR#0


Upgrade package versions

Describe the feature
Could we upgrade some outdated package versions in the Test project. I am using your package but its causing vulnerability issues for packages needed to be upgraded, namely:

system.text.regularexpressions.4.3.0 -> system.text.regularexpressions.4.3.1
system.net.http.4.3.0 -> system.net.http.4.3.4

We have the same problem.

@vaibhav-goyal  Reporting Vulnerabilities, Thanks!


Add Contribution Guidelines, License and Code of Conduct in Readme File.

Hi @ayeshafalak
I would like to add contributing.md file in the repo and link it and other files in the readme file.
Please assign it to me.
Thanks

I also want to work on this issue please assign this to me , i'll do it for sure
:) thanks @ayeshafalak

@ayeshafalak
Why is it duplicate ?
It's not added in your readme file

Or you don't want to add it in your readme file. @ayeshafalak


Extension host terminated unexpectedly. Please reload the window to recover.

VSCode Version:1.11.2
OS Version: win 64

Steps to Reproduce:

#23976


linter failures in make-adopt-build-farm.sh

What are you trying to do? Submit PRs
Expected behaviour: lint checks run successfully on areas of code that aren't modified by my PRs
Observed behaviour: linker fails: e.g. https://github.com/AdoptOpenJDK/openjdk-build/pull/2541/checks?check_run_id=2155314328
Any other comments: Since this wasn't previously a problem I would imagine the linter has been updated with new rules that have caused a problem.
If we don't already have them, we should perhaps have clearer guidelines in the CONTRIBUTING guide to help people run the checks on their own system.

The new rules was probably me adding the super linter in #2513 . I'll look into this and your twin issue #2530

I was going to mention that one but I thought I'd found a change that had gone into this file since then. But it does seem the most likely option :-)


tf.function ensures type annotations and input_signature match

The input_signature argument of tf.function allows users to specify the input dtypes of function arguments.
Python type annotations for Tensors will contain the dtype of the Tensor.
These changes will allow tf.function to ensure that type annotations and input_signature match. When there is a mismatch, an error will be thrown.
This is of particular importance to ensure forward compatibility, so that in the future we can switch from input_signature to type annotations without breaking existing code.
Example
@tf.function(input_signature=[tf.TensorSpec(None, dtype=tf.int32)])
def f(x: tf.Tensor[dtypes.Int32]):
  return x

@tf.function(input_signature=[tf.TensorSpec(None, dtype=tf.int32)])
def f(x: tf.Tensor[dtypes.String]): # Error! Type annotation does not match input_signature.
  return x

This is not ready to be merged until dtypes have their own types #40696 and the Tensor class is defined as a Generic class #40921.

@rahul-kamat  Can you please resolve conflicts? Thanks!

It has been 15 days with no activity and the awaiting response label was assigned. Is this PR still valid? Assigning the stalled label. Please comment to reassure me that this is still being worked on.

The PR is still valid, but we've been delayed on the integration.

Hi @rahul-kamat I'm going to go ahead and close this PR, because it seems to have stalled. If you're still interested in pursing this (and responding to my comments), please feel free to reopen! Thank you for your contribution!

Hi @rahul-kamat, @mdanatg  I'm going to go ahead and close this PR, because it seems to have stalled. If you're still interested in pursing this (and responding to my comments), please feel free to reopen! Thank you for your contribution!


DropBox

App Name: DropBox
Package Name: com.dropbox.android

Hello! Weʼre switching to a new system for icon requests in order to be able to better manage them. From now on, icon requests can be submitted here: https://forms.gle/Fx8vZAiWdW1Tyjo57. If your icon request hasnʼt yet been fulfiled, it has been copied to our new icon request database, meaning no action is required. Consequently, this issue will be closed.


Bug: Certain characters aren't shown correctly.

Describe the bug
Certain characters aren't shown correctly in embed titles. For example, the small capital T (т) is displayed as an m. This appears to only happen in the titles, not the descriptions.
To Reproduce
This can be noticed in the following studio: https://scratch.mit.edu/studios/27244700/

This is because the font used for titles (Comfortaa) doesn't support these characters. The reason that it doesn't happen in descriptions is because the descriptions use a different font.
Possible fixes:

Change the font to something else
Only change the font for titles containing these characters

For some reason, it is specifically that character which doesn't show correctly. Other studio titles with special characters fallback onto a different font if they aren't supported by Comfortaa, but this character displays as an m instead.

Because this only happens to one character (otherwise it defaults to a fallback font), I am going to close this issue because it is very low impact. If this affects more characters in the future then it will be reopened.


Adjust with subm-operator api rename

Depends on: https://github.com/submariner-io/submariner-operator/pull/1651
Signed-Off-By: Janki Chhatbar<EMAIL_ADDRESS>

/approve

/lgtm


feat: support fish shell completion

Description
Add support for fish shell completion.
Issue reference
#879
Checklist
Please make sure you've completed the relevant tasks for this PR, out of the following list:

[x] Code compiles correctly
[ ] Created/updated tests
[ ] Extended the documentation

Hello, I do know know if this feature has a unit test for it so i haven't updated unit test. And for documentation I think I updated everywhere I could find (only within this file). If there's other places I should update this documentation, plz let me know.

LGTM. Thanks for the contribution.

The DCO is broken again @alexlai97


Enable deregister on TASK_KILLING

Marathon with Mesos 0.28 has a TASK_KILLING event that will be emitted when the
task is about to get killed. This should be the earliest indicator that the
instance should be removed from consul.

Coverage remained the same at 82.416% when pulling e761e1bd7aeae46e1beef5df744bf288961a8341 on hubrick:feature/task-killing into 4cde4737cac0858c640a4d095012af3a85fd1014 on allegro:master.


Is there any way to disable adding ng-hide class to img?

I'm using your directive to display preview of the image. For non-image types I display custom image/icon (for example for PDF). Whenever user choose image, but then change his mind and choose PDF, your directive keeps adding ng-hide class to img tag, and my PDF icon is hidden.
Is there any way to disable that functionality?

Post your code or create a jdfiddle.


Fork?

Is michaelherger's fork the official one?

I think the plan was for Paul to nominate someone else as a maintainer so that we don't lose al issues/PR's by moving to a fork

I don't know but when I took a look at Fork it seems like this master has been merged to michaelherger's

NO! I only forked it as the base to apply the few changes I need for my Spotty plugin. But I have close to zero Rust experience...

Anybody here who feels experienced enough and has time to do an official fork?

Any clear winners here?

At the moment there is nobody who wants to win. I'm new to Rust and I think many others too

How about handing it into an organization of its own and invite known contributors as members. That would enable others than @plietar to approve and merge PullRequests.

That's where I'm at too, re: just learning rust. Not a bad idea, @janLo. Is @plietar responding to any librespot noise at able to do that?

Nobody has stepped forward so there's nothing to respond to. I think that's the first step here.


Enhancement request: allow JAR to output notice_schema.json without validating anything

Feature request
It would be useful to be able to export notice_schema.json without actually validating a feed, so that we can automatically update our schema information for validation notices, for example when we upgrade validator versions.
Currently, it seems that if you are using the -n command line option to generate notice_schema.json (docs), you still have to specify -i or -u, i.e., you have to actually validate a feed.
Is your feature request related to a problem? Please describe.
We use the validator in our pipeline and it would be convenient to be able to monitor for notice schema changes without having to actually validate a feed. We would perhaps want to run a separate job on a different cadence just to check for validation schema changes when we upgrade validator versions, for example.
Proposed solution
I would like to be able to run the validator JAR from the command line using the -n option without specifying an input feed in -i or -u.
Describe alternatives you've considered
We can just define a test feed that we always use to validate when we are trying to generate notice_schema.json. This is not too onerous, it is a fine interim solution. However it would be nice to not have to do this.
Additional context
This is not a high priority, simply a nice to have.

Just to confirm the behavior you are seeing, when you run the command-line jar with options like -n -o /output-directory, are you not seeing the schema written at all?  Or are you just seeing a non-zero exit code from the application?
I can confirm the latter behavior (non-zero exit code), which we should fix, but I can't confirm the former behavior (schema not written at all).


[BEAM-14138] Revert "[BEAM-14112] Avoid storing a generator in _CustomBigQuerySource"

Reverts apache/beam#17100
Caused Python PostCommit to fail: https://ci-beam.apache.org/job/beam_PostCommit_Python38/2407/

Run Python 3.8 PostCommit

R: @tvalentyn @chunyang @KevinGG

LGTM

Thanks.


Tubnonomic automation no longer working properly

The turbnomic automation fails to deploy on both OCP 4.8 and 4.10 on IBM Cloud.
Automation: https://github.com/IBM/automation-turbonomic
The deployment never becomes healthy b/c all of the PVCs (which use the storage class created by the 202 layer) never become healthy.
The error events coming from the PVCs are similar to:
Generated from vpc.block.csi.ibm.io_ibm-vpc-block-csi-controller-0_be0c2dbc-2418-4f0a-8069-3a17a2503793
6 times in the last 1 hour
(combined from similar events): failed to provision volume with StorageClass "ibmc-vpc-block-mzr": rpc error: code = InvalidArgument desc = {RequestID: 84bf2f91-8e7d-4c93-af0b-b22daff331e5 , Code: InvalidParameters, Description: Failed to extract parameters, BackendError: <sizeRange> is an invalid parameter, Action: Please provide valid parameters}

The storage class has a volume size 10-2000Gi.  The volumes have a size of 1Gi, so I suspect that is the issue.

is there any reason why the default storage (block) or maybe ibmc-vpc-block-metro-10iops-tier since wait for first customer is specified in https://github.com/turbonomic/t8c-install/wiki/Storage-Class-Requirements ?

I was able to recover the deployment by changing the storage class to the cluster default: ibmc-vpc-block-10iops-tier
Before changing the automation, I need to understand why we created that storage class in the first place... and I do not know why.   @tcskill @mjperrins @seansund Any idea why we create the ibmc-vpc-block-mzr storage class for IBM Cloud deployments?  This is what is causing turbo to fail.

@triceam yes. That is needed on ibm cloud only when you have a vpc deployment or multi region deployment because by default Ibm cloud creates block storage with bind immediately and need it to be wait for first consumer (note only for the use case described above) when on classic cluster not needed, also when on single zone not needed

standard metro classes will work for this use case, no need for a custom storageclass

OK, well, it doesn't work anymore.  The <sizeRange> is an invalid parameter, Action: Please provide valid parameters} error is b/c turbo is requesting 1Gi volumes, and the minimum size is 10Gi.  I will update our automation to use the metro class.

Yes I see new metro has wait for first consumer so should work fine

Removed 202 layer (no custom storage class for IBM Cloud, switched to use ibmc-vpc-block-metro-10iops-tier storage class on IBM Cloud.
Fixed in:

https://github.com/cloud-native-toolkit/automation-solutions/pull/191/
https://github.com/IBM/automation-turbonomic/pull/17


[OPIK-159] Experiment delete endpoint

Details
Added endpoint to delete experiment and its experiment items.
`DELETE /v1/private/experiments/{id}'
Issues
OPIK-159
Testing

Automated component tests
Manual test via minikube
Scenarios:

Happy path (Delete created experiment)
Delete not existent experiment
Delete experiment with experiment items
Delete multiple experiments

@thiagohora im +1, get batch delete no need for single. in addition if FE passed non existing ID you can ignore no need to fail request.

I left many comments, but they're minor, so no blockers.
I also reviewed the DB models in order to validate the logic. The only thing that should be cascaded is the experiment item. We discussed to not to cascade to the traces (and therefore spans and feedback scores). Also, it's clear that it shouldn't cascade upwards dataset and dataset items.

Correct, we are only cascading to experiment items:
https://github.com/comet-ml/opik/pull/336/files#diff-b67b3ca0cf191513869cffa83beb91365f763e7fa12b424550bbd754dcbc60b9R152


variable location has never been used. Does not work for any image.

The variable (location) has never been used. However, it has got value assigned. What is the purpose of this variable. Also, the parser does not work for image.

@krishnasinha
Below is the use for the content location header. It was not useful to me when I wrote this, so nothing ever happened with it. Feel free to enter a pull request with support for this header.
What do you mean it doesn't work for images? Can you be more specific?

MHTML defines the naming of objects that are normally referred to by URLs and the means of aggregatingresources that go together. Two MIME headers, Content-Location and Content-Base, are defined in order to resolve references to other content stored locally in related body parts. Content-Base gives an absolute URL base, or "starting point",for relative URLs that appear in other MIME headers and in HTML documents that do not contain any BASE HTML elements. Content-Location specifies the URL that corresponds to the content of the body part that contains this header.

https://msdn.microsoft.com/en-us/library/ms526265(v=exchg.10).aspx

Thanks

Never got back to me. Closing.


Compose

Created a docker-compose file.
Modified Dockerfile.
Created a .env file to manage version and environment variables.

I don't know how to create a pull request with a new branch.
I guess you'll have to create this new branch manually before merging...

Thank you for the pull request. But we should not remove the simple version. I will see how to put this in a different folder.

So I included it in a separate folder. Sorry but I had to lose your commit history. I didn't know how to include it without deleting the old.
Thank you a lot for this big contribution!


[WIP] PoC – Email Verification

:tickets: Ticket(s): Closes #19

:construction_worker: Changes
Added new endpoints "/user/create_account" and "/user/verify" for verifying user emails
:flashlight: Testing Instructions

Bother @bobheadxi about giving you .env variables for the email username and password. Copy .env.sample and create .env. Fill in the blanks.

make testenv
make core
make gateway

Go to localhost:8081/user/create_account?email=<your email or temp email>, check it sends an email. Hopefully, you get "success" callback.
Using the hash given in the email go to localhost:8081/user/verify?hash=<insert your deterministic hash here>, check the response and if it added the email to the list of verified. Hopefully, you get "success" callback.

Note: I am putting up this PR to get ideas about how to do things better in Go.
:shit: TO-DO

[ ] Add tests
[ ] Clean up junk I wrote
[ ] Reflect code review

@leesw98 same package, separate file (ie move the csv-writing to a file called poc.go)


Quicker cfml

Refactored parts of the cfml interface and calculator so that running fits is significantly quicker now.
There are still two large contributors (apart of the obvious profile calculations):
1 - I/O of reading and writing temp files
2 - undo stack update between optimization steps (required for constraints)

will be addressed by applying changes from the io_serialization branch of easyCore
is a potential feature modification: https://github.com/easyScience/easyDiffractionLib/issues/60

moving to draft to look at again after the release


Focal Transformer on 1-D Data

Great Work !
My query is that : can this method be run on 1-D features ? Because the pooling operations done before flattening is done for 2-D representation !

Hi,
thanks! Yes, you can simply split the 1-D into multiple window chunks and then perform pooling for each window by directly applying a fc layer without any further flatten as I did for 2d windows.
thanks,


More than one table in a document

Hello, I am trying to convert a PDF that contains multiple nested tables, and it is not working.
I managed to convert one table when I select just it, but if I go with de default selections, that covers the entire table area, the result is wrong.
The document I am trying to convert if from Official Government Diary, that is open in Internet, for exemple the one below.
https://www.imprensaoficial.com.br/DO/BuscaDO2001Documento_11_4.aspx?link=%2F2012%2Fempresarial%2Ffevereiro%2F16%2Fpag_0005_8HBIFCNOH5JF4e90U1JNETH84LI.pdf&pagina=5&data=16/02/2012&caderno=Empresarial&paginaordenacao=100005
Is there any way of getting the entire table correct in a more automated way?
Thanks,
Mauricio

Table autodetection is not perfect. I recommend experimenting with saving templates or with scripting to solve this problem. Using one of the wrappers for Tabula in your programming language of choice, you can autodetect tables and discard the outermost box, which should give you the result you're looking for.


iPad displayCutoutPadding behaves differently by the screen orientation

Describe the bug
On iPad, displayCutoutPadding applies zero insets in landspace modes but applies some insets in portrait modes. In the normal portrait mode, there is a top inset; in the upside-down portrait mode, there is a bottom inset.
Affected platforms

iOS

Versions

Kotlin version*: 1.9.21
Compose Multiplatform version*: 1.5.11
OS version(s)* (required for Desktop and iOS issues): 16.6
OS architecture (x86 or arm64): arm64
JDK (for desktop issues): N/A

To Reproduce
Steps and/or the code snippet to reproduce the behavior:

Generate a Compose project using the Kotlin Multiplatform Wizard.
Edit composeApp/src/iosMain/kotlin/MainViewController.kt as follows. (The button in the example shows/hides the status bar)

import androidx.compose.foundation.background
import androidx.compose.foundation.layout.Box
import androidx.compose.foundation.layout.displayCutoutPadding
import androidx.compose.foundation.layout.fillMaxSize
import androidx.compose.foundation.layout.safeContentPadding
import androidx.compose.material.Button
import androidx.compose.material.MaterialTheme
import androidx.compose.material.Text
import androidx.compose.runtime.DisposableEffect
import androidx.compose.runtime.remember
import androidx.compose.ui.Modifier
import androidx.compose.ui.graphics.Color
import androidx.compose.ui.interop.LocalUIViewController
import androidx.compose.ui.window.ComposeUIViewController
import androidx.compose.ui.zIndex
import kotlinx.cinterop.ExperimentalForeignApi
import kotlinx.cinterop.readValue
import platform.CoreGraphics.CGRectZero
import platform.UIKit.UIStatusBarAnimation
import platform.UIKit.UIViewController
import platform.UIKit.addChildViewController
import platform.UIKit.didMoveToParentViewController
import platform.UIKit.willMoveToParentViewController

fun MainViewController() = ComposeUIViewController {
    val uiViewController = LocalUIViewController.current
    val rootViewController = uiViewController.view.window?.rootViewController
    val statusBarController = remember { StatusBarController() }
    DisposableEffect(rootViewController) {
        if (rootViewController != null) {
            with(statusBarController) {
                rootViewController.addChildViewController(this)
                rootViewController.view.addSubview(view)
                didMoveToParentViewController(rootViewController)
            }
        }
        onDispose {
            with(statusBarController) {
                willMoveToParentViewController(null)
                view.removeFromSuperview()
            }
        }
    }
    MaterialTheme {
        Box(Modifier.fillMaxSize()) {
            Box(Modifier.displayCutoutPadding().fillMaxSize().background(Color.Red).zIndex(0.0f))
            Box(Modifier.safeContentPadding().fillMaxSize().background(Color.Blue).zIndex(10.0f))
            Box(Modifier.safeContentPadding().zIndex(20.0f)) {
                Button(onClick = {
                    statusBarController.statusBarHidden = !statusBarController.statusBarHidden
                }) {
                    Text("Click me!")
                }
            }
        }
    }
}

@OptIn(ExperimentalForeignApi::class)
class StatusBarController : UIViewController(nibName = null, bundle = null) {
    init {
        view.setBounds(CGRectZero.readValue())
    }

    var statusBarHidden: Boolean = false
        set(value) {
            field = value
            setNeedsStatusBarAppearanceUpdate()
        }

    override fun prefersStatusBarHidden(): Boolean = statusBarHidden
    override fun preferredStatusBarUpdateAnimation(): UIStatusBarAnimation =
        UIStatusBarAnimation.UIStatusBarAnimationSlide
}

Edit iosApp/iosApp/ContentView.swift as follows. (displayCutoutPadding does not work without .ignoresSafeArea(.all))

import UIKit
import SwiftUI
import ComposeApp

struct ComposeView: UIViewControllerRepresentable {
    func makeUIViewController(context: Context) -> UIViewController {
        MainViewControllerKt.MainViewController()
    }

    func updateUIViewController(_ uiViewController: UIViewController, context: Context) {}
}

struct ContentView: View {
    var body: some View {
        ComposeView().ignoresSafeArea(.all) // Compose has own keyboard handler
    }
}

Launch the app on your iPad.

Expected behavior
With every screen orientation, displayCutoutPadding must behave the same.
Screenshots
https://github.com/JetBrains/compose-multiplatform/assets/17005454/cc08d043-df50-456a-93c3-cb5d76879a4e
Additional context
The current implementation relies on UIViewController.safeAreaInsets, and on iPadOS, safeAreaInsets does not return different values by screen orientation. So, WindowInsets.Companion.displayCutout should check for UIDevice.currentDevice.userInterfaceIdiom and use iosSafeArea.only(WindowInsetsSides.Top) regardless of the orientation on iPad.
However, that makes displayCutoutPadding always put some top insets when the status bar is present. This is because safeAreaInsets behaves differently on devices without notches, even on the iPhone SE. The only way to make the value of safeAreaInsets.top 0 is to hide the status bar, as shown in the example above.
One way to "fix" this problem would be to ignore safeAreaInsets on iPad and iPhone SE and set a manual additionalSafeAreaInsets.

Thanks! I will take a look

Maybe related to this issue: https://github.com/JetBrains/compose-multiplatform/issues/4045#issuecomment-1867391478

Please check the following ticket on YouTrack for follow-ups to this issue. GitHub issues will be closed in the coming weeks.


Support for Loading Databases from URL Path or Embedded Storage Mode

I am requesting support for a feature in ArangoDB that enables loading databases directly from a URL path or implementing an embedded storage mode similar to other databases like OrientDB's "plocal" mode.
Expected Behavior:
Ability to specify a URL path (file system path or remote URL) for loading database files directly into ArangoDB.
Implementation of an embedded storage mode where ArangoDB operates in a self-contained manner within an application environment, simplifying deployment and management.
Use Case:
Our application architecture requires seamless integration with an embedded database engine for local data storage and processing. Having ArangoDB support direct loading from URL paths or an embedded storage mode would streamline our deployment process and reduce external dependencies.
Additional Information:

Details about how OrientDB's "plocal" mode or similar embedded modes work in other databases.
Use cases or scenarios where this feature would be beneficial, such as development environments, single-server deployments, or applications requiring embedded databases.
Any technical considerations or challenges that might need to be addressed for implementing this feature in ArangoDB.
Impact and Importance:
Implementing this feature would enhance ArangoDB's versatility and appeal to developers and applications requiring embedded database capabilities, contributing to a more seamless and efficient development and deployment experience.

well, arguably ArangoDB Enterprise edition already supports similar with hot backup and rclone?

does, it supports to run DB in embedded mode?

As you probably know ArangoDB is implemented in C++, hence it will never be able to be ran inside a java process - if thats what you mean?

yes


Feature: Plugins

fix #376

[x] Create a plugin definition
[x] Reference plugin in the code
[x] Plugin define script to run with arguments
[x] Plugin define port to open
[x] SSH automatically open port defined in plugin
[x] Validate arguments
[x] Validate plugin definition returns PluginDefinition
[x] Optional name for ports if multiple
[ ] Move all custom scripts to plugin

Docker image for plugins I think can come in v2 as all of this is already required

Can the script be a url? That would fix #309 as well.

not as of now, not sure how we can make this one

This might be a good place to fix #211 as well.

docs for how to use/ what are the defaults/ etc?

Docs are in the wiki:
https://github.com/Azure/aztk/wiki/Write-a-plugin-for-aztk
https://github.com/Azure/aztk/wiki/Cluster-Configuration


[MXNET-375] Lp Pooling and Global Lp Pooling

Description
As Title
Checklist
Essentials

[x] The PR title starts with [MXNET-375]
[x] Changes are complete (i.e. I finished coding on this PR)
[x] All changes have test coverage:
Unit tests are added for small changes to verify correctness (e.g. adding a new operator)
Nightly tests are added for complicated/long-running ones (e.g. changing distributed kvstore)
Build tests will be added for build configuration changes (e.g. adding a new build option with NCCL)
[x] Code is well-documented:
For user-facing API changes, API doc string has been updated.
For new C++ functions in header files, their functionalities and arguments are documented.
For new examples, README.md is added to explain the what the example does, the source of the dataset, expected performance on test set and reference to the original paper if applicable
Check the API doc at http://mxnet-ci-doc.s3-accelerate.dualstack.amazonaws.com/PR-$PR_ID/$BUILD_ID/index.html
[x] To the my best knowledge, examples are either not affected by this change, or have been fixed to be compatible with this change

Changes

[x] Support Lp Pooling
[x] Unit tests for Lp Pooling

Comments

For bridging the gap between MXNet and ONNX operators

@reminisce @rahul003 @anirudh2290 @eric-haibin-lin Please give a review if you have time, thanks!

Where does LP pool come from? we need to add documentations for it. See describe when registering Pooling.

@piiswrong The first appearance of LpPool seems to be in this paper: Convolutional Neural Networks Applied to
House Numbers Digit Classification (https://arxiv.org/pdf/1204.3968.pdf). I'll update the doc with the link to the paper.

Merging it since it's approved by 2 committers.


leaver nerdtree buffer need 1-2 secconds.

with this change
-    autocmd bufenter * if (winnr("$") == 1 && exists("b:NERDTreeType") && b:NERDTreeType == "primary") | q | endif
+    autocmd bufenter * if (winnr("$") == 1 && exists("b:NERDTree") && b:NERDTree.isTabTree()) | q | endif

i find if i use c-w c-h to leave nerdtree buffer,it will need more time than beffore

Is this still an issue? I can't reproduce it here :-/

with mini vimrc test,this issue has gone.sorry,maybe it is due to my autocmd filetype init,so close it


Generate notifications to multiple users at the same time.

Greetings,
I'm new into web programming and I'm building a web application. The application works with a user loging system and depending on the user it will load an specific content to submit/edit. Everytime a user submits or edit data I want to generate a notification to all other users logged in the system.
I don't know how I can reach this. What I've thought is that I could create a script somehow ( I don't know how) which constantly look for changes on the database. Then, I replicate this script in all the user content page so when the script checks for changes it will generate a notification to each user. I know this is not like a "host to everyone"-message and probably not the best idea but it will gerenate a notification from their own content.
I hope you understand my idea. If anyone can help me or guide me I will apreciate it.
Sorry for my bad english.

I don't think this is a Push.JS issue and is really a design question for your application.  If you can make your application push down some information when any user submits or edit data, THEN you can use push.js to show notifications as per the examples.


jail release point?

just came across this serverside mod and wondering if you can set a command for where you set release point for the jail that they were sent too. this would be great if there is multi jail in the server like different faction

No presser because the server I'm trying to get going with my modpack is player build spawn and city and player run factions and hoping have player run economy as well and it is at the starting stage still

This was implemented with https://github.com/Caltinor/SimpleJail/commit/d37824ff66ec8802c6f103cf3870acacd8c53648
just forgot to close this issue


Simulation parameters contain the type of mutation that is going to be implemented in the simulation.

Depends on:

[ ] #117

this issue is not needed


[AutoPR compute/resource-manager] Update config for Go SDK

Created to sync https://github.com/Azure/azure-rest-api-specs/pull/2755

This PR has been merged into https://github.com/Azure/azure-sdk-for-python/pull/2276

(message created by the CI based on PR content)
Installation instruction
Package azure-mgmt-compute
You can install the package azure-mgmt-compute of this PR using the following command:
pip install "git+https://github.com/Azure/azure-sdk-for-python@restapi_auto_2755#egg=azure-mgmt-compute&subdirectory=azure-mgmt-compute"
You can build a wheel to distribute for test using the following command:
pip wheel --no-deps "git+https://github.com/Azure/azure-sdk-for-python@restapi_auto_2755#egg=azure-mgmt-compute&subdirectory=azure-mgmt-compute"
If you have a local clone of this repository, you can also do:

git checkout restapi_auto_2755
pip install -e ./azure-mgmt-compute

Or build a wheel file to distribute for testing:

git checkout restapi_auto_2755
pip wheel --no-deps ./azure-mgmt-compute

Direct download
Your files can be directly downloaded here:

azure_mgmt_compute-4.0.0rc1-py2.py3-none-any.whl
azure-mgmt-compute-4.0.0rc1.zip


What is D32 format?

Sorry for posting my question as an issue
Many quantized hexagon_nn ops has _d32 variant. What is D32 format and how it is different from "flat"?
e.g.
QuantizedAdd_8p8to8 - elementwisely adds Input A and Input B together. (flat format)
0: Input A data (quint8 tensor)
1: Input B data (quint8 tensor)
...
0: Output data (quint8 tensor)

QuantizedAdd_8p8to8_d32 - Elementwise Add; inputs and output are in d32 format
0: Input A data (qint8)
1: Input B data (qint8)
...
0: Output data (quint8)

Duplicated https://github.com/XiaoMi/nnlib/issues/11.


Separate user and dev requirements

Background
The current dependencies in requirements.txt are all development-related. They are not essential for using the package. Perform the following actions to improve the dependency management aspects of this template:
Plan

Move the current contents of requirements.in and requirements.txt to dev-requirements.in and dev-requirements.txt.
Modify some logic in the package to use a third-party package, simulating a common circumstance. Consider using pandas and/or a visualization package.
Replace the current version of requirements.in and requirements.txt with the references to the third-party dependency used during the execution of the package.
Modify the build/test GitHub Actions workflow to account for these changes.

References

Example handling of separate requirements.txt and requirements.in files https://github.com/viperior/aequus/blob/056c53bdf94599a1f386529a6b66f35537358067/.github/workflows/test.yml

The -r flag can be used within requirements files.
So there can be two files:

requirements.in
dev-requirements.in

-r requirements.txt

where dev-requirements.in has an instruction to install everything in requirements.in.
Then, developers don't have to worry about installing two sets of dependencies. A developer simply uses dev-requirements.in to get both the normal dependencies but also those needed to develop and test.

This requires #90


Support description message for entire application.

This feature allows users to add a description message for entire application. It's similar to ArgumentParser(description="...").
(c.f., https://docs.python.org/3/library/argparse.html#argparse.ArgumentParser)

A commit (6f8f7dec0766ea062817cbd6c352f7d06659432b) implements this feature. Closed.


Repo sync for protected CLA branch

The pull request is created from master637739805399739590 to master to fix git push error for protected CLA branch

@opbld15 : Thanks for your contribution! The author(s) have been notified to review your proposed change.

#label:"aq-pr-triaged"


error choosing own color palette in pl.spatial_scatter

Description
I created my own color palette as a ListedColormap and verified that it was correct via isinstance(). However when I use it as the palette argument in pl.spatial_scatter() it fails to set. I also tried using a list of colors as the error message suggested, but then I got an error that it was not a ListedColormap. Is there a different way I should be specifying my own colors?
Note that I also tried using just a dictionary as this previously worked with scanpy which also didn't work. It would be very helpful if the dictionary version worked so that I can specify cluster_name to color. Is there a way to do this?
isinstance(my_pal1, ListedColormap) returned True
sq.pl.spatial_scatter(adata, color="CellType", legend_fontsize = "small", palette = my_pal1)
Traceback

Traceback (most recent call last):

File "/home/kguion/anaconda3/envs/juliet/lib/python3.9/site-packages/squidpy/pl/_color_utils.py", line 24, in _maybe_set_colors
raise KeyError("Unable to copy the palette when there was other explicitly specified.")
**KeyError: 'Unable to copy the palette when there was other explicitly specified.'**

During handling of the above exception, another exception occurred:
Traceback (most recent call last):

File "/mnt/c/Users/Kate/Documents/KSOM/Juliet_Emamaullee/code/deconvolute_tangram.py", line 203, in <module>
squidpy_analysis( sample = "6a", scale = 0.15 )
File
"/mnt/c/Users/Kate/Documents/KSOM/Juliet_Emamaullee/code/deconvolute_tangram.py", line 154, in squidpy_analysis
sq.pl.spatial_scatter(adata, color="CellType", legend_fontsize = "small", palette = my_pal1)
File "/home/kguion/anaconda3/envs/juliet/lib/python3.9/site-packages/squidpy/pl/_spatial.py", line 418, in spatial_scatter
return _spatial_plot(adata, shape=shape, seg=None, seg_key=None, **kwargs)
File "/home/kguion/anaconda3/envs/juliet/lib/python3.9/site-packages/squidpy/pl/_spatial.py", line 168, in _spatial_plot
color_params = _prepare_args_plot(
 File
"/home/kguion/anaconda3/envs/juliet/lib/python3.9/site-packages/squidpy/pl/_spatial_utils.py", line 745, in _prepare_args_plot
_maybe_set_colors(source=adata, target=adata, key=c, palette=palette)
File
"/home/kguion/anaconda3/envs/juliet/lib/python3.9/site-packages/squidpy/pl/_color_utils.py", line 27, in _maybe_set_colors
add_colors_for_categorical_sample_annotation(target, key=key, force_update_colors=True, palette=palette)
File "/home/kguion/anaconda3/envs/juliet/lib/python3.9/site-packages/scanpy/plotting/_utils.py", line 480, in
add_colors_for_categorical_sample_annotation          _set_colors_for_categorical_obs(adata, key, palette)
File "/home/kguion/anaconda3/envs/juliet/lib/python3.9/site-packages/scanpy/plotting/_utils.py", line 419, in
_set_colors_for_categorical_obs                       raise ValueError(

**ValueError: Please check that the value of 'palette' is a valid matplotlib colormap string (eg. Set2), a  list of color names or a cycler with a 'color' key.**

Version

...

Hi @kguion1 , seems like scanpy supports only wrapped palette as a cycler. Can you try running
from cycler import cycler
my_pal1_c = cycler(colors=my_pal1.colors)

sq.pl.spatial_scatter(adata, color="CellType", legend_fontsize = "small", palette = my_pal1_c)

Hi, thanks for the suggestion. When I tried your code, I got the following error:
Traceback (most recent call last):
File "/home/kguion/anaconda3/envs/juliet/lib/python3.9/site-packages/squidpy/pl/_color_utils.py", line 24, in _maybe_set_colors  raise KeyError("Unable to copy the palette when there was other explicitly specified.")
KeyError: 'Unable to copy the palette when there was other explicitly specified.'
During handling of the above exception, another exception occurred:
Traceback (most recent call last): File "/mnt/c/Users/Kate/Documents/KSOM/Juliet_Emamaullee/code/deconvolute_tangram.py", line 245, in  squidpy_analysis( sample = "6a", scale = 0.15 )  File
"/mnt/c/Users/Kate/Documents/KSOM/Juliet_Emamaullee/code/deconvolute_tangram.py", line 157, in squidpy_analysis
sq.pl.spatial_scatter(adata, color="CellType", legend_fontsize = "small", save = pngfile, palette = my_pal1_c)               File
"/home/kguion/anaconda3/envs/juliet/lib/python3.9/site-packages/squidpy/pl/_spatial.py", line 418, in spatial_scatter
return _spatial_plot(adata, shape=shape, seg=None, seg_key=None, **kwargs) File
"/home/kguion/anaconda3/envs/juliet/lib/python3.9/site-packages/squidpy/pl/_spatial.py", line 168, in _spatial_plot
color_params = _prepare_args_plot( File
"/home/kguion/anaconda3/envs/juliet/lib/python3.9/site-packages/squidpy/pl/_spatial_utils.py", line 745, in _prepare_args_plot
_maybe_set_colors(source=adata, target=adata, key=c, palette=palette) File
"/home/kguion/anaconda3/envs/juliet/lib/python3.9/site-packages/squidpy/pl/_color_utils.py", line 27, in _maybe_set_colors
add_colors_for_categorical_sample_annotation(target, key=key, force_update_colors=True, palette=palette)  File
"/home/kguion/anaconda3/envs/juliet/lib/python3.9/site-packages/scanpy/plotting/_utils.py", line 480, in
add_colors_for_categorical_sample_annotation
_set_colors_for_categorical_obs(adata, key, palette)  File
"/home/kguion/anaconda3/envs/juliet/lib/python3.9/site-packages/scanpy/plotting/_utils.py", line 425, in
_set_colors_for_categorical_obs  raise ValueError("Please
set the palette key 'color'.")
ValueError: Please set the palette key 'color'.

Hi, thanks for the suggestion. When I tried your code, I got the following error:

Sorry, this was my mistake, should be my_pal1_c = cycler(color=my_pal1.colors), was colors=....

Using that, it now recognizes that cycler, but states it needs to be a ListedColormap
File "/mnt/c/Users/Kate/Documents/KSOM/Juliet_Emamaullee/code/deconvolute_tangram.py", line 157, in squidpy_analysis             sq.pl.spatial_scatter(adata, color="CellType", legend_fontsize = "small", save = pngfile, palette = my_pal2_c)               File "/home/kguion/anaconda3/envs/juliet/lib/python3.9/site-packages/squidpy/pl/_spatial.py", line 418, in spatial_scatter       return _spatial_plot(adata, shape=shape, seg=None, seg_key=None, **kwargs)                                                   File "/home/kguion/anaconda3/envs/juliet/lib/python3.9/site-packages/squidpy/pl/_spatial.py", line 250, in _spatial_plot         color_source_vector, color_vector, categorical = _set_color_source_vec(                                                      File "/home/kguion/anaconda3/envs/juliet/lib/python3.9/site-packages/squidpy/pl/_spatial_utils.py", line 480, in _set_color_source_vec                                                                                color_map = _get_palette(                                                                                File "/home/kguion/anaconda3/envs/juliet/lib/python3.9/site-packages/squidpy/pl/_color_utils.py", line 55, in _get_palette       raise TypeError(f"Palette is {type(palette)} but should be string or ListedColormap.")
TypeError: Palette is <class 'cycler.Cycler'> but should be string or ListedColormap.

same errors in the squidpy 1.2.2

this one worked for me in squidpy 1.5.0:
import matplotlib.pyplot as plt
color_palette = plt.colormaps["tab20c"]
sq.pl.spatial_scatter(adata, color="CellType", legend_fontsize = "small", palette = color_palette)

For anyone still having this issue like I did, I was using v 1.2.2, I updated to v 1.6.1 then I tested on a spatial dataset with 2 categories. This worked for me.
cols = ['#F0A830', '#7F1CD3']
custom_cmap = mcolors.ListedColormap(cols)
isinstance(custom_cmap, ListedColormap) # returns True

sq.pl.spatial_scatter(
    adata,
    library_id="spatial",
    shape=None,
    color='category',
    palette=custom_cmap
)


CSV widget table header misalignment

Upload a CSV, when viewed in the csv viewer widget, the headers are misaligned with the columns.

And clean up commit https://github.com/Kitware/minerva/commit/87305c282790350841d8e3aa05c6bfdc00254751

And remove the commented line from https://github.com/Kitware/minerva/commit/80f3226643f9f1ceac9ec7f9c7f8e797c8c9efe9


Re-Arrange the FORCE repo for testing

Do not merge yet please.

@PaulTalbot-INL
Hi @dylanjm,
The PR is ready for review now. Now, all the Aspen/HERON integration tests are setup and are successful.
The main full test is the: /integration_tests/Aspen_HERON_FullTest.
All the others tests are tests for intermediate steps.
The tests include running python files, comparing texts, images and xml files, running HERON and RAVEN.
For now,  The tests should work if RAVEN, HERON and FORCE are inside the same folder. Some changes to the paths are needed once we have a clear idea about the structure of the FORCE installation that includes RAVEN, HERON, HYBRID, etc.
For the tests to work, the raven libraries need to be activated: "conda activate raven_libraries"
Thank you!

I didn't perform a full review, as there's quite a bit of extra added to this PR at this point that doesn't need reviewing.

@PaulTalbot-INL
Hi Paul, let me know if more changes are needed to this PR.


Explicitly use '&' in http_build_query()

Same as PR https://github.com/J7mbo/twitter-api-php/pull/172 but to develop branch.

Some hosting environments are configured to use & rather than &. Calling the twitter api end up with error #32 / Could not authenticate you

Thanks a lot! I'll merge this into Develop now.


During functional testing against 12.1.0, trust domain test fails with 409

When running the test, test/functional/cm/test_trust_domain.py::TestTrustDomain::test_curdl, against 12.1.0, the test fails with the following message:
iControlUnexpectedHTTPError: 409 Unexpected Error: Conflict for uri: https://<IP_ADDRESS>:443/mgmt/tm/cm/trust-domain/ Text: u'{"code":409,"message":"01020066:3: The requested trust domain (/Common/test_trust) already exists in partition Common.","errorStack":[],"apiError":3}'
This may be due to improper cleanup of a previous test run. Will do more research and detail it here.

Looks like this was fixed by #722


Build a markdown Editor using React

Proposal Submission
Proposed title of article
Build a markdown Editor using React
Use keyword research to improve your article's SEO.
Proposed article introduction
In this tutorial we'll be building a markdown Editor. A markdown is a markup language for creating formatted text using plaintext. Many àre farmilar with it but it actually seems fun to build it ourselves. And as I frontend developer, I choose to do it with React
We will use this writing snippet to help us assess the overall quality before approval so ensure it has been proofread, spellchecked and grammarchecked.
For example, if the article is based on machine learning, use the following: [Machine learning] Introduction to Machine Learning.
If the article is based on developing an Android application, use the following: [Android] Developing Apps using Android.
ONLY viable topics to pick from:

API
Artificial Intelligence
Blockchain
Containers
Edge Computing
Languages
Machine Learning
Networking
Node.js
Security

Key takeaways
In this article we'll be able to:

Understanding of React
How does it rank among other frontend framework
why you should use React
Build a markdown Editor in React

Article quality
It's not just about building the markdown but understanding how the framework works, why does it stand out, what gives it the edge amongst others. This article covers that. Also If you want to add to a list of projects you done. Then this article is for you.
References
Please list links to any published content/research that you intend to use to support/guide this article.
Conclusion
Finally, remove the Pre-Submission advice section and all our blockquoted notes as you fill in the form before you submit.
We look forwarding to reviewing your topic suggestion.
Templates to use as guides

How To Guide Template
Software Review Template
Tutorial Template

Thank you for submitting your topic. @oyedeletemitope
After some careful consideration, it struck us that this topic may be a bit over-saturated throughout other blog sites and official documentation.
We believe this is the best way for students to build a great portfolio (for potential employers) is by building what does not exist and what can provide the most value.
Please feel free to suggest an alternate topic to explore. 🚀


OSX Support

Related issue https://github.com/mateidavid/zstr/issues/5

Can you confirm if this is still an issue with the current master branch? If not, I would close this PR.
Thanks!

It seems fixed, closing.


add one empty line before a markdown list

this Pull Request fix the HTML rendering for the HTML list items on the chapters:

General Metadata Recommendations
Description Classification
Credit Attribution

Thanks @joenio for the PR!


Evaluate benefits of analyzers

Currently we have four analyzers in Google.Cloud.Tools.Analyzers:

One about default literals, which is documented as probably redundant now
Optional parameters must be specified internally
Every file must have a copyright notice
We can't expose particular dependencies in our public API

We removed the analyzer for Diagnostics a while ago. It's not clear whether the above analyzer is actually running - that's testable, of course, but it probably isn't running in CI, which is where it would be useful.
The downside is that there's quite a lot of complexity in terms of project generation and build etc. Let's work out whether the pros and cons balance.

Okay, it looks like for whatever reason, the analyzers aren't running in either CI (which doesn't surprise me much) or locally (which does surprise me). Given that we haven't obviously suffered from the lack of running them, it makes sense to simplify things by removing them. Will do so in the gax-v4 branch.

Amanda noted that they probably weren't running as we detected their presence using netstandard1.3, but actually built 2.0. The argument about "we haven't suffered from not running them" still stands :)

Fixed in #8494 and #8536


Add Slovak translation

Hello, I really like this app, I have to say that this is probably the best looking and functioning weather app when we talk about FOSS weather apps. You did really nice job! I would be excited if I would have a weather radar included in this app. Good luck for future development!

Thanks for the feedback! Suggestions for new features are better to write to the issues so that I don't forget. And many thanks for the translation


Upgrade Kotlin, Gradle, Trikot.streams, Trikot.foundation

Description

Upgrade kotlin to version 1.5.31
Upgrade trikot.foundation to version 2.2.2
Upgrade trikot.streams to version 2.2.5
Upgrade gradle wrapper to version 7.2

How Has This Been Tested?
Project has been built with this new version
Types of changes

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

Changes in gradlew.bat come from the command:
./gradlew wrapper --gradle-version=7.2 --distribution-type all


Xcode is saying CoachMarksController' has no member 'start'

Describe the bug
When I use this line,
self.coachMarksController.start(in: .window(over: self))
New Version of 11.1 is showing the following error:
Value of type 'CoachMarksController' has no member 'start'
To Reproduce
In the previous version of Xcode, it was working. But not working in Xcode 11.1
Expected behavior
I have seen inside CoachMarksController class. start method actually exists. But still, Xcode is showing this error message.
Screenshots

Environment:

Xcode: 11.1
Swift 5
Device:  Any
iOS version: 11.3
Dependency Manager: CocoaPods

Same. Updated from 'master' to 1.4 in my cart file. Previously I was getting an odd message about Start(on:) being deprecated in 1.2.1 but I was not able to change it to the recommended start(in:). Not not recognizing start as a method of CoachMarksController.

I 'fixed' this for me by updating my app to Swift 5, which finally got it to recognize a method with ProcessContext as a parameter. Hacking around on my local copy my notes:

I was able to add a method that showed up when I rebuilt my app with the newly built framework (func mylittlepony(in: Bool).
when I changed the Bool to ProcessContext (enum) mylittlepony no longer showed up as method in autocomplete.
I had also tried moving around the #if #else but that didn't make a difference
I've had to update my call to .start with the enum format and that part seems to be working (i.e. it starts when loading that view controller as it used to).
Previously I had been using Xcode 11.1 but hadn't updated to Swift 5 for my code yet.

Oh and once I updated to Swift 5, it offered to change my .start(on:) to .start(in:)

Ah, thanks for the context @derekgraham. There was a bug in the Swift 4.x toolchain that happened only with CocoaPods, which is likely related to this issue, see #198.
I'll have to investigate and see if it still happens with Swift 5, but as far as I know it doesn't.

Closing this, as I'm certain it was caused by a bug in the toolchain 3 years ago. It's been working well since (as far as I know).


feat: wire-up THORChain lending aggregated positions data

Description

Pull Request Type

[ ] :bug: Bug fix (Non-breaking Change: Fixes an issue)
[ ] :hammer_and_wrench: Chore (Non-breaking Change: Doc updates, pkg upgrades, typos, etc..)
[ ] :nail_care: New Feature (Breaking/Non-breaking Change)

Issue (if applicable)

closes #
Risk

Testing

Engineering

Operations

Screenshots (if applicable)

Current dependencies on/for this PR:

develop

PR #5568

PR #5569

PR #5579

PR #5590   👈

This stack of pull requests is managed by Graphite.


Break TSM out of model architecture notebook

Separate TSM example out of the model architecture notebook in the examples directory for clarity and ease of use / documentation.

Work by @imscw95 in the IJM_NSM_v2 branch is moving this work forward, in preparation of an NSM notebook.


Should goog.DEBUG be false for builds distributed via npm?

goog.DEBUG is true in the google-protobuf.js runtime distributed in the npm package. This setting gates the various assertion code in the runtime (goog.asserts.ENABLE_ASSERTS = goog.DEBUG;).

Is this expected?
If so, should DEBUG be set to false for production builds? And should two builds be distributed, or a means be exposed to configure this?
If not, should the gulpfile that generates the runtime supply --define goog.DEBUG=false to the closure compiler invocation?

Good question. Internally, we are use to people deliberately disabling debug mode when moving to production. It's unclear what the best approach is for public distribution.

i think this is a case where we would need a new API
it would be reasonable to expect some customers to want to run with asserts and debug code disabled, but due to our releasing a bundled version of closure library this isn't possible.
consider angulars enableProdMode


Adding new logo

Hi @shiffman! This adds the new logo to the menu bar to replace "THE CODING TRAIN" characters.
If you have time, it might make sense to make a new SVG file that has both the train and the name next to each other. This will make it take up a bit more space. You should be able to just update the SVG file and the CSS should work for this as well. Happy to answer any questions!

Thank you @runemadsen, this is fantastic!! @jasontheillustrator take a look here and let us know if you have any comments before I merge! Do you want to create an SVG with the logo + name as Rune suggests?
https://deploy-preview-1343--codingtrain.netlify.app/


Device choice for implementation

Hii @holdmypotion great stuff, it behaves awesome on computer but on mobile devices it's not user friendly I mean it gives some lag to the user, any solution on implementing this feature only on desktop devices and not on mobile devices (probably using react-device-detect library) ?

You can use by window size or better way use by the device.


V3

Removes mongodb dependency (less memory usage, less complicated, easier to get running)
Adds device pairing mode
Forces all user input and driver messages through a JSON schema validator for consistency and consistent error handling
Re-organisation of the project structure
Rewrite of all the unit tests to avoid using mocks
Applies prettier and eslint to make code formatting consistent
Switch from npm to yarn

Codecov Report

Merging #49 into master will increase coverage by 22.86%.
The diff coverage is 65.37%.

@@             Coverage Diff             @@
##           master      #49       +/-   ##
===========================================
+ Coverage   44.86%   67.72%   +22.86%
===========================================
  Files          25       28        +3
  Lines         798      911      +113
  Branches       93      105       +12
===========================================
+ Hits          358      617      +259
+ Misses        384      264      -120
+ Partials       56       30       -26

Impacted Files
Coverage Δ

eventCreators.js
100% <ø> (ø)
:arrow_up:

management.js
0% <0%> (ø)
:arrow_up:

utils/jobs.js
0% <0%> (ø)

comms/index.js
0% <0%> (ø)
:arrow_up:

app.js
0% <0%> (ø)
:arrow_up:

schemas/deviceTypes/light.js
100% <100%> (ø)

schemas/event.js
100% <100%> (ø)

controllers/interface.js
100% <100%> (ø)

schemas/deviceTypes/speaker.js
100% <100%> (ø)

schemas/deviceTypes/index.js
100% <100%> (ø)

... and 31 more

Continue to review full report at Codecov.

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


Build into iframe

Instead of 'lazily' builing the app file-by-file on change using the resolver, run a full build and dump the code into an iframe.
Benefits:

isolate the app from the editor
allows other versions of ember to be used
use css without polluting parent window

modules we need to include

app/app.js
app/router.js
boot.js (boots the app)

EmberCLIService

buildGist
fileFromBlueprint
contentForAppBoot

closed in #24


Make converter optional

What does this PR do?
This PR makes the converter optional in OTel agent. It is enabled by default, but users now have an option to disable it in agent config.
Motivation

Additional Notes

Possible Drawbacks / Trade-offs

Describe how to test/QA your changes
Run the agent with the otelcollector.converter.enabled set to false (or via env DD_OTELCOLLECTOR_CONVERTER_ENABLED=false and ensure the collector config does not get enhanced)

/merge

/merge


Incorrect coefficient restriction

The implementation currently restricts the leading coefficient of a secret polynomial to be nonzero. The stated goal of this appears to be avoiding a reduction in the number of shares required to reconstruct the secret. Unfortunately, this reasoning both breaks the security guarantee of the design and does not achieve the goal in a meaningful way.
As analyzed by Ghodosi et al. (specifically section 3.1), restricting the leading coefficient in this way allows a subthreshold of colluding players to definitively identify a value that the secret cannot be, which breaks the security guarantee otherwise provided by the Shamir design.
The reason that this restriction does not meaningfully achieve its goal of protecting against a reduced-threshold condition is subtle. Suppose the intended threshold is t players. With no coefficient restriction, it is certainly the case that the leading coefficient will be zero with 1/256 probability. If a threshold of t - 1 players collude and interpolate a value, they must verify that it is the correct secret; remember, they do not know in advance if the zero-coefficient case is at play. However, if the players were able to verify that the secret is correct, each player alone (without colluding) could simply try all 256 possible secrets! Therefore there is no advantage to a subthreshold of players that cannot be realized with equal expected value by non-colluding brute force.

Please see the attached document with remarks on the trade-offs considered in the existing implementation and the decision to enforce non-zero leading coefficients. In summary, enforcing non-zero leading coefficients is considered to be a safer implementation choice than allowing zero leading coefficients and statistically preserves security for all instances.
remarks.pdf


Support for caret version numbers

Bower packages can have dependencies specified using a caret in the version, eg ^0.6.2, which allows the first non-zero digit to change only (details here).
At present these aren't converted to rubygems version constraints by rails-assets, and so the build fails:
rake aborted!
There was a Gem::Requirement::BadRequirementError while loading rails-assets-textAngular.gemspec:
Illformed requirement ["~> ^ 1.2.0"] from
  /tmp/d20150219-16574-18415eg/rails-assets-textAngular.gemspec:18:in `block in <main>'
/usr/local/rvm/gems/ruby-2.0.0-p598@global/gems/bundler-1.7.9/lib/bundler.rb:394:in `rescue in eval_gemspec'
/usr/local/rvm/gems/ruby-2.0.0-p598@global/gems/bundler-1.7.9/lib/bundler.rb:383:in `eval_gemspec'
/usr/local/rvm/gems/ruby-2.0.0-p598@global/gems/bundler-1.7.9/lib/bundler.rb:354:in `block in load_gemspec_uncached'
/usr/local/rvm/gems/ruby-2.0.0-p598@global/gems/bundler-1.7.9/lib/bundler/shared_helpers.rb:44:in `chdir'
/usr/local/rvm/gems/ruby-2.0.0-p598@global/gems/bundler-1.7.9/lib/bundler/shared_helpers.rb:44:in `block in chdir'
/usr/local/rvm/gems/ruby-2.0.0-p598@global/gems/bundler-1.7.9/lib/bundler/shared_helpers.rb:43:in `chdir'
/usr/local/rvm/gems/ruby-2.0.0-p598@global/gems/bundler-1.7.9/lib/bundler.rb:349:in `load_gemspec_uncached'
/usr/local/rvm/gems/ruby-2.0.0-p598@global/gems/bundler-1.7.9/lib/bundler.rb:339:in `load_gemspec'
/usr/local/rvm/gems/ruby-2.0.0-p598@global/gems/bundler-1.7.9/lib/bundler/gem_helper.rb:31:in `initialize'
/usr/local/rvm/gems/ruby-2.0.0-p598@global/gems/bundler-1.7.9/lib/bundler/gem_helper.rb:13:in `new'
/usr/local/rvm/gems/ruby-2.0.0-p598@global/gems/bundler-1.7.9/lib/bundler/gem_helper.rb:13:in `install_tasks'
/usr/local/rvm/gems/ruby-2.0.0-p598@global/gems/bundler-1.7.9/lib/bundler/gem_tasks.rb:2:in `<top (required)>'
/tmp/d20150219-16574-18415eg/Rakefile:1:in `require'
/tmp/d20150219-16574-18415eg/Rakefile:1:in `<top (required)>'
(See full trace by running task with --trace)

It should be straightforward to convert these to rubygems compatible version ranges. The semver readme (linked previously) gives several examples of how to do this.

We should handle it well. What version of textAngular  are you trying to build?

textAngular 1.3.11
On Fri, 27 Mar 2015 at 5:53 am, Adam Stankiewicz<EMAIL_ADDRESS>wrote:

We should handle it well. What version of textAngular are you trying to
build?
—
Reply to this email directly or view it on GitHub
https://github.com/rails-assets/rails-assets/issues/242#issuecomment-86835102
.

Thanks for reporting this issue. It should be fixed now.

Excellent, thanks - I'll try it out later today
On Fri, 27 Mar 2015 at 7:53 am, Adam Stankiewicz<EMAIL_ADDRESS>wrote:

Thanks for reporting this issue. It should be fixed now.
—
Reply to this email directly or view it on GitHub
https://github.com/rails-assets/rails-assets/issues/242#issuecomment-86861682
.

Hey I think there's a bug in this. In textAngular 1.3.11, package.json specifies these dependencies:
  "browserDependencies": {
    "angular": "^1.2.x",
    "rangy": "^1.3.0",
    "bootstrap": "^3.0.x",
    "font-awesome": "^4.0.x"
  },

But the gemspec for textAngular 1.3.11 has this:
  spec.add_dependency "rails-assets-angular", ">= 1.2.0", "< 2"
  spec.add_dependency "rails-assets-bootstrap-css-only", ">= 3.0.0", "< 4"
  spec.add_dependency "rails-assets-font-awesome", ">= 4.0.0", "< 5"
  spec.add_dependency "rails-assets-rangy", ">= 1.2.0", "< 2"

Note that for rangy, package.json specifies rangy ^1.3.0 but the gemspec specifies >= 1.2.0 < 2". By my understanding, the gemspec should instead specify >= 1.3.0 < 2".
I've tried refreshing the build via the rails-assets website and then reinstalling the gem, but it still has this.

Bower looks into bower.json, not package.json: https://github.com/fraywing/textAngular/blob/v1.3.11/bower.json


Navmenu new UI: can't see tabbar title in iphonex

This was fixed here https://github.com/raedahgroup/dcrios/pull/548


Resolve strange sourcecode include

I think this one is fairly self explanatory
https://github.com/kanflo/opendps/blob/c548a92ee5ef0424ced7c079b52a8e0ce1cbd180/dpsboot/common.c#L1-L10

Yup, that file is not one of my proudest moments ;)


iOS 9 not working

After upgrading from iOS8 to iOS9, all the calls to ydn-db are failing. I have searched the internet and found one reference to a problem: http://stackoverflow.com/questions/33081419/values-function-in-ydn-db-not-working-in-safari-and-ios9
Other than that, there is no mention anywhere on the Internet of this issue. We have multiple clients that have upgraded their iPads to iOS9, and they are now not able to use them with our product. Do you have a solution for this? You marked my previous issue report at a known issue. If it is, we would like to know what you plan to do about it. Or at least point out a link on the Internet that mentions it. This is NEW to iOS9. It has nothing to do with any issues reported prior to its release.

Please don't create an issue if you don't have solid proof about the issue.

 It fails miserably. Please don't blow off my issues without investigation.


Structure of the repository - export-ignore and assets

As we can see from https://github.com/concrete5/concrete5/pull/3809, we (finally!) are moving to a more simple and intuitive structure of the repository.
A few notes here...
a. Simplify ZIP
In this perspective, what about simplifying the repository when it's downloaded as a ZIP file?
I mean, when people download an archive (being it for a release or for a branch), what about removing all the stuff that are not needed for the core developers?
This is as easy as updating the .gitattributes file, and the result can be seen downloading this file.
I adopted this approach in https://github.com/KorvinSzanto/concrete5/pull/2, and this is great because:

TravisCI and core developers are not affected, since they use git clone to get the archive contents
End-users will have a clean archive with stuff they don't need

b. The /build folder
Ideally, it would be great to exclude the /build folder, but it's not possible with the current approach: we need this folder in order to build the CSS and JS assets since they are not under GIT.
The solution for this problem could be to add them to the repository.
Since it'll cause problems when re-building the assets locally, we could add a script that developers could run that tells git to ignore the built assets, something like this:
git update-index --assume-unchanged concrete/js/app.js
git update-index --assume-unchanged concrete/js/backstretch.js
# etc

When the assets should be committed, we could run another script like this:
git update-index --no-assume-unchanged concrete/js/app.js
git update-index --no-assume-unchanged concrete/js/backstretch.js
# etc

These two scripts could be implemented in our current Gruntfile.js (so that we wouldn't have to manually update the list of built assets).
c. The /concrete/css/build and /concrete/js/build folders
Those folders are not needed for production, so what about moving them under the /build folder?

A few lines of code explains better.
What I'm suggesting is:

Add the built assets to the repository and remove the build folder from the ZIP archive downloadable from GitHub: https://github.com/mlocati/concrete5/commit/77905cbfe7e630249cafd94cd380fea5b8b68016
Tell GIT to ignore JS/CSS assets when we're developing them: see https://github.com/mlocati/concrete5/commit/6508f60150538496867f29096a36205475b1f508
Thanks 1. the installation process is simplified - see  https://github.com/mlocati/concrete5/commit/0f4d0028a72bd4e4bb6db6bea37ef7930f7a2603 (the repository contains the built scripts and we don't risk to include their debug version thanks to 2.

The above commits can be found in this branch of mine

I'm not sure that I like the idea of having the build script modify the git state. Could we just leave these ignored and add them as needed with git add -f?

I'm not sure that I like the idea of having the build script modify the git state. Could we just leave these ignored and add them as needed with git add -f?

I tried it some time ago, and IIRC it didn't worked: the ignored status is just for files that are not under git control. Once a file is in git, any change to it will be tracked (unless we set the assume-unchanged bit - and that's what's done in the above code of mine)

Hmm, well why ignore it at all then? Why not just keep track of it every time it changes? I had originally thought that we'd need to push built versions overtop of github's auto generated zips.

I'd ignore their changes only when we do a grunt debug (at least it's so in my branch and in my mind)

Another solution could be to use the composer script features... I'll digger into this

I hadn't considered that at all, after looking at your implementation it sounds like a reasonable approach to the problem.

it sounds like a reasonable approach to the problem.

Ok, let's create a PR then :wink: (https://github.com/concrete5/concrete5/pull/3839)


[5.5] Pass along the -emit-symbol-graph flags with -emit-module-separately

Originally merged on main in https://github.com/apple/swift-driver/pull/730
Resolves rdar://83905653

@swift-ci please test


Updated open in intellij button to work with flatpaks

Hey!
I've switched over to linux as my main os for programming related things a bit ago.
Since flatpaks are cool I'm using those for intellij too, however the open in intellij button didn't account for flatpak installations.
This PR makes the button work with flatpaks.

Looks fine to me; one message maybe could be more clear for what users should do if IDEA isn't found, but I think I may be rewriting a lot of messages for the new UI that will be incoming shortly. I'll merge this; it doesn't look like it will conflict. Thanks!


UEFI disks are not supported

The software only supports bios disks and not uefi, #FIXME line 629 in basilico.py, if it is a uefi disk it should be resizing partition 3 but is resizing partition 1.

Maybe it should detect the partition type and resize the the first "Linux filesystem" one it finds, or the last one, or skip the "EFI Sytem" one (or the same thing using UUIDs).
All the data we could ever desire can be obtained with lsblk -J -o PATH,FSTYPE,PARTFLAGS,PARTLABEL,PARTN,PARTTYPE,PARTTYPENAME,PTTYPE once the image has been written, or with fdisk -l asd.img before writing anything, which is harder to parse tough

The last partition created by the latest release are still not resized

Fixed in https://github.com/WEEE-Open/pesto/commit/24f2e0b85a7875fb2ea2d6a25c517257eec68276 (possibly)


change url

@samdark http://local.xyz.com/site/default/view/?id=2&alias=x+y+z+Program  i have following url i want to change using url manager in yii2 into http://local.xyz.com/Programs/2-x-y-z-Program how to rewrite url.

'Programs/<id>-<alias>' => 'site/default/view'

Please use forums next time since issue tracker is for framework bugs and feature requests.

thanks @samdark let me know how to  remove + between x+y+z+Program

Don't specify it in URL.

http://www.yiiframework.com/doc-2.0/yii-behaviors-sluggablebehavior.html

@samdark  how to remove ?alias
http://local.advance.com/programdetail/?alias=Child+Care+Volunteering+Program


Send content ids with redirects

Required for https://github.com/alphagov/govuk-content-schemas/pull/148

Would you mind adding a test that validates against the schema and hooking it up so that it gets built whenever there is a change to the schemas as per the documentation?

@jamiecobbett Yep, we'll do that as another pull request.

It made sense to put this change on top, otherwise we'd have mutually dependent pull requests.

I don't have Jenkins permissions to complete the final steps here.
I've done as much as I can – i.e. the jenkins-schema.sh has been set up and just needs to be called form this job as per the instructions.

@tuzz Done.

@elliotcm Thanks!


Using GuildMember.ban/kick returns 403, Guild.banMember succeeds

This works:
// message received from client.on("message", message => {....
message.member.guild.banMember(message.author, 5); // bans member + deletes messages in 5 days

But this doesn't:
// message received from client.on("message", message => {....
message.member.ban(5); // bans member + deletes messages in 5 days

Catching the error from ban (formatted):
Error: cannot PUT /api/v6/guilds/[guild_id]/bans/[member_id] (403)
status: 403,
text: '{"code": 0, "message": "Privilege is too low..."}',

I have double-checked that the bot has ban/kick permissions and the person to be banned/kicked is in a lower role than the bot. Can't kick with any method, since there is no guild.kickMember. Any reason why GuildMember.ban and Guild.banMember require different privileges/permissions?

Guild.banMember isn't a feature in 9.x.x, are you using an outdated version of discord.js?

Package.json says that I'm using 9.3.0:
"discord.js": "^9.3.0"

message.guild.ban(message.mentions.users.first(), "days"); Should work

Turns out the code was trying to ban/kick the wrong member. Thanks!


Need advice for the customisation in JSQ as per my application

Hello Jesse,
I have researched over your library and found that its really amazing to use this for my application, below are my issues that I need an advice from you to move ahead.
I need to achieved UI of chat similar to whats app and few of its operation like mentioned below.

PhotoMediaItem currently have feature to show default loading and gray image background.  I need to show a blur image as the initial chat received from XMPP is having a dirty image base64 received in stanza.
On received i add the photo message object with this. So i want to override the loader with Blur image and download Button on top. On press i have to start showing a custom loader like M13ProgressRingView over this photo item.
On load download over from storage location I refresh the image back to original image from local path.

Same thing for video to show the behaviours like whatsapp IOS App.
Also there are certain parameters required for this settings that i need to pass in the datasource of JSQ array but my data is coming from core data that also needs to updated when relevant server information is received like message seen, sent, delivered, photo idNot downloaded, yet to start, pending, download active,upload active...and so on.
My question is I have to manage the JSQMessage object with this additional info that is updated in my DB object but to update the JSQMessage data object i need to reload the entire UI. Is this right approach or you can advice some optimised way so I update both database and JSQ message object.
Also Can i subclass PhotoMediaItem, VideoMediaIem and LocationMedia Item as mentioned in your doc to handle this extra parameter conditions stated above to store in the UI objects.
Also is it advisable to update / Subclass the JSQMessage Object. I tried manipulating some of the classes as per my requirement but this approach totally messes the UI and data loading as it becomes lagging and inconsistent way.
Below are the issue that have aroused due to modifying the existing classes. Summarising the requirement.

Autolayout messed , no compile errors but the Bubbles keeps shifting and spreading automatically.
Wanted to add more items and modify chat bubble as per my design of app but this needs additional parameters that I need your expertise to guide me where should i add and what is not to be touched.
Photo Video and Location are very generic how to customise then to add UI effects as per the requirement.
How to add additional Media item like Location With landmark, Audio bubble, Customised bubble extending the JSQMediaItem.

Sorry for the long email, but I felt this is more convenient library that can be customised and used by your guidance.
your response is eagerly awaited.
Regards.
Vicky

Very good question actually I want to ask the same thing and especially the customisation of media items like whatsapp loaders etc

Same here , Im experiencing the same troubles and cant figure out how to customize the bubbles ,specially the loading indicator part before and while downloading a media.

Thanks @vickydhas ! :smile:
I'm sorry, but I don't have time to answer. I suggest posting on StackOverflow, or something similar!


Doesn't work with Gsync

There is a large amount of mouse lag and stuttering,  what happens is that g-sync is trying to match the fps of an unoptimized program which has low fps, This happens because the refresh rate of the monitor becomes less than it's native refresh rate, which in turn, makes the mouse looks laggy. This is common in some hardware accelerated programs, so as you may guess, you have 2 main options for a temp fix, to make ScreenToGif fixed refresh in the control panel or disable Gsync on windowed mode altogether.

Closing this was external.
I implemented a feature to improve the cycle rate of the capture system, but I'm not sure if would help anyway with this issue.


Display photos on a map

There should be a view of a map that has points representing photos.

This is for me the missing feature of TroveBox. Does anyone have a prototype? I'd like to try to implement this, but by looking at the source code of TroveBox I finds that my PHP skills are limited ;-)
Some years ago I did that, using the Web services.

This feature requires a little bit of backend work to expose location information. The majority of the work will be in the UI/map work.
Ideally there would be an API to get high level lat/lon coordinates at the city/state level. We would also need an API to return photos within a range of a lat/lon pair. I think these are separate. To get high level lat/lon I would suggest implementing it when photos are added/updated/deleted and storing a new lat/lon only if it's not within a certain distance of one already stored. http://stackoverflow.com/questions/15736995/how-can-i-quickly-estimate-the-distance-between-two-latitude-longitude-points
To get photos within a radius we'd need to do the standard radius calculations in the SQL query.


If using the step-by-step PowerShell you will overwrite existing Group.Unified settings

[The step-by-step instruction here https://docs.microsoft.com/en-us/azure/active-directory/enterprise-users/groups-assign-sensitivity-labels#enable-sensitivity-label-support-in-powershell will cause the current information to be overwritten by the EnableMIPLabels value.
You should add that it's necessary to retrieve the current settings first, something like $Setting = Get-AzureADDirectorySetting | ? { $_.DisplayName -eq "Group.Unified"} detailed here https://docs.microsoft.com/en-us/azure/active-directory/enterprise-users/groups-settings-cmdlets#update-settings-at-the-directory-level
If not, the very common EnableGroupCreation "False" will end up as "True" so all users in the org. can create groups.]

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

ID: ef84a3e0-879e-4312-a89d-7a3bb0cd9a51
Version Independent ID: 43851f8a-b71f-0a0a-8b1f-629e3528972f
Content: Assign sensitivity labels to groups - Azure AD
Content Source: articles/active-directory/enterprise-users/groups-assign-sensitivity-labels.md
Service: active-directory
Sub-service: enterprise-users
GitHub Login: @curtand
Microsoft Alias: curtand

@ChristianJBergstrom
Thanks for your feedback! I've assigned this issue to the author who will investigate and update as appropriate.

Bump

@ChristianJBergstrom thank you for the feedback. I have created a work item for us to track internally. #please-close

@barclayn Hello! Totally forgot about this one. It’s been a year! 😃 Thanks for the update though!