Is there a way to replace black color with transparent color in MTIImage?

Checklist

[X] I've read the README
[X] I've searched for existing GitHub issues

I have searched all filters but didn't find a suitable one.

MTIChromaKeyBlendFilter(https://github.com/MetalPetal/MetalPetal/blob/1f0da098b564e2730f6ca90d2bf625975960b57e/Frameworks/MetalPetal/Filters/MTIChromaKeyBlendFilter.h)?
This filter replaces color in the inputImage with the inputBackgroundImage.

Hi, thanks for the great library!
I'm trying to achieve this also (convert a mask that is black/white to transparent/white). This is my function:
func makeTransparent(mask: UIImage) -> UIImage {
    let ciMask = CIImage(cgImage: mask.cgImage!)
    let mtiImage = MTIImage(ciImage: ciMask, isOpaque: true)

    let contextOptions = MTIContextOptions()
    let context = try! MTIContext(device: MTLCreateSystemDefaultDevice()!, options: contextOptions)

    let opacityFilter = MTIChromaKeyBlendFilter()
    opacityFilter.inputImage = mtiImage
    opacityFilter.inputBackgroundImage = MTIImage(color: MTIColor.clear, sRGB: true, size: mask.size)
    opacityFilter.color = MTIColor.black

    let outputImage = try! context.makeCGImage(from: opacityFilter.outputImage!)
    return UIImage(cgImage: outputImage)
}

It takes an image, and am trying to export an image with transparency instead of black.
Any help would be appreciated, thank you.

@YuAo , I tried using your suggestion to use MTIChromaKeyBlendFilter  to replace 'background color' in MTIImage with 'clear color'. It is working when displaying the resulting MTIImage in MTIImageView, but not when saving as UIImage.  Do you have any suggestions?
Note: This is happening with my modifications made to sample app (SketchViewController).  I am trying to save only the brush strokes (and not the background).
I modified the background so that it is red.
            let background = MTIImage(color: MTIColor.init(red: 1, green: 0, blue: 0, alpha: 1), sRGB: false, size: self.brushImage.size)
            compositingFilter.inputBackgroundImage = background
            compositingFilter.layers = [MultilayerCompositingFilter.Layer(content: self.brushImage)]
            let _ = try? self.imageView.context!.startTask(toRender: compositingFilter.outputImage!)

Then I used  MTIChromaKeyBlendFilter  to make the red background transparent, and converted to UIImage so that only brush stokes (with transparent background) are written to UIImage:
    private func saveDrawing() {

        let filter = MTIChromaKeyBlendFilter()
        filter.inputBackgroundImage = MTIImage(color: MTIColor.init(red: 0, green: 0, blue: 0, alpha: 0), sRGB: true, size: imageView.image!.size)
        filter.inputImage = imageView.image
        filter.color = MTIColor.init(red: 1, green: 0, blue: 0, alpha: 1)

        let result = filter.outputImage
        imageView.image = result

        let outputImage = try! self.imageView.context!.makeCGImage(from: result!)
        let finalImage = UIImage(cgImage: outputImage)

        UIImageWriteToSavedPhotosAlbum(finalImage, nil, nil, nil)
    }

@vjdaemp UIImageWriteToSavedPhotosAlbum always saves the opaque image.  Use Photos.framework to save the image.

ah, yes I forgot about that -- thanks @YuAo


SRP271375 - SARS-CoV-2 infection of the oral cavity and saliva

Project short name:
SARS-CoV-2InfectionOralCavity
Primary Wrangler:
Ami
Secondary Wrangler:
Ida

Associated files

Google Drive:

https://drive.google.com/drive/folders/1ImHp0sxZEEXTWp26NufiI-LSa6G9WQHY
N.B. Because the Saliva data is not yet published via a publication (although it is public in their data portal), the authors have requested that we don't yet submit that data. Therefore V2 below is the main version for submission (release 21). It does not include the Saliva data.
V1 - with Saliva data
https://docs.google.com/spreadsheets/d/1EZUOzDGefmGqm4s_GGnwY6v2m5WoARCCuLYNx4n0dOo/edit#gid=1592345125
V2 - without Saliva data
https://docs.google.com/spreadsheets/d/19j9VAkPH_gGXpN0-9uCIUp_Aedv8IPQW10yeTJwAH1w/edit#gid=1139711375
Link to Ingest

https://contribute.data.humancellatlas.org/projects/detail?uuid=dcbb50d1-9acf-4f70-9fda-b1f63a948c49

Published study links

Paper: SARS-CoV-2 infection of the oral cavity and saliva

Accessioned data: SRP271375

N.B. requested a new collection protocol ontology term: https://github.com/HumanCellAtlas/ontology/issues/117
Key Events

[x] Convert published metadata to HCA spreadsheet
[x] Manually curate dataset to meet HCA metadata standard
[x] Collect any matrix and cell-type annotation files
[x] Upload sheet to validate metadata
[x] Check linking using ingest graph validator
[x] Transfer raw files to ingest to validate data files
[ ] Ask the Secondary Wrangler for an end-to-end review of the project. Ask the Expertise Wrangler to review specific tabs if needed
[ ] Submit dataset to Production
[ ] Complete the Export SOP
[ ] Convert project data to SCEA format following the SCEA conversion SOP if appropriate

Sounds to me like they are re-analysing existing data but also unpublished data. Worth a check

https://contribute.data.humancellatlas.org/projects/detail?uuid=69d798a1-80b3-4fc8-b124-e3a256422127 seems to me like a pre-print of this exact study. And it also seems like we were reached out directly by the contributor at some point.

Requested a new ontology term: https://github.com/HumanCellAtlas/ontology/issues/117

Emailed authors about obtaining gene expression matrices for the primary neutrophil data.

6e3fa1af-45a8-426b-8d8a-e5de1e257707

Stuck in "graph validation requested" state

Re-triggered graph validation

Graph valid

Hi Ami, I'm done secondary reviewing! It looks great I only have a few comments
Project

the project and study accession come from a different publication, I think that this data was re re-analysed in this paper
the cell count should be 13,824

Donor

there is additional metadata like sex age ethnicity and disease in the supplementary material table 1, tabs  1 and 2,

Enrichment

enrichment_protocol_gingiva_centrifugation is not really an enrichment step, it can be omitted

Analysis protocol

analysis_protocol_cell_type_assignment_1 can be fused with analysis_protocol_quantification_2 since they both only apply to the salivary gland data
a protocol is needed to describe how the two datasets were combined in byrd_warner_integrated.processed.h5ad

Analysis file

byrd20_gingiva.processed.h5ad should not be linked to analysis_protocol_quantification_2

Ok done, re-submitting it.

Getting an error trying to delete the old submission from ingest prod. Need to upload the updated submission.

Now getting an error importing the new submission.

submission deletion was successful, import should now work okay.

Re-import was successful.

Exported and submitted import form.

@MightyAx to investigate if this has actually been exported

waiting on issue: ebi-ait/hca-ebi-wrangler-central#926

Export Successful

Looks good.


Some EMC values above 1000000 are multiplied by 1000 during rendering

It appears that the latest updates to Project Expansion broke the way some EMC values are displayed. To be more specific, if the actual EMC value is more than 1000000, it appears to be multiplied by 1000 during rendering. For example, 1,234,567 would be shortened to "1.23 Billion" instead of "1.23 Million". If the actual EMC value was 2,147,483,648 (Yellow Fuel), it would be rendered as "2.14 Trillion" instead of "2.14 Billion".
This did not happen in older versions of Project Expansion, which correctly format the numbers with 1000000 being a million and not a billion.

 Mod list
	Mod List:
		client-1.18.2-20220404.173914-srg.jar             |Minecraft                     |minecraft                     |1.18.2              |DONE      |Manifest: a1:d4:5e:04:4f:d3:d6:e0:7b:37:97:cf:77:b0:de:ad:4a:47:ce:8c:96:49:5f:0a:cf:8c:ae:b2:6d:4b:8a:3f
		configured-1.5.3-1.18.2.jar                       |Configured                    |configured                    |1.5.3               |DONE      |Manifest: NOSIGNATURE
		forge-1.18.2-40.1.0-universal.jar                 |Forge                         |forge                         |40.1.0              |DONE      |Manifest: 84:ce:76:e8:45:35:e4:0e:63:86:df:47:59:80:0f:67:6c:c1:5f:6e:5f:4d:b3:54:47:1a:9f:7f:ed:5e:f2:90
		jei-1.18.2-<IP_ADDRESS>.jar                          |Just Enough Items             |jei                           |<IP_ADDRESS>           |DONE      |Manifest: NOSIGNATURE
		ProjectE-1.18.2-PE1.0.1.jar                       |ProjectE                      |projecte                      |1.0.1               |DONE      |Manifest: NOSIGNATURE
		ProjectExpansion-1.18.2-1.0.4.jar                 |Project Expansion             |projectexpansion              |1.18.2-1.0.4        |DONE      |Manifest: e9:fa:82:d2:7b:66:ce:37:db:52:70:63:25:68:04:a9:72:41:8f:cf:6c:9a:da:52:ab:05:5b:97:2c:13:54:37

Known issue, already fixed


Update ffmpeg extension

Description
– I added a new video file conversion option, called VideoLoop Converter. This uses FFMPEG to create a h.264 .mp4 file without sound, at a bitrate of 4000kbps. This is intended to be used as a replacement for GIFs on certain websites.
– Changed the progress view so that the feedback is in percentage, rather than a float between 0 and 1.
– Fixed an issue that prevented processing (rotation/conversion) of files that has spaces in the path.
– Changed version number to 1.1.0, from 1.0.0

Screencast

Checklist

[x] I read the extension guidelines
[x] I read the documentation about publishing
[x] I ran npm run build and tested this distribution build in Raycast
[x] I checked that files in the assets folder are used by the extension itself
[x] I checked that assets used by the README are placed outside of the metadata folder

Saw a misstake in the code, so I closed the request. Sorry. I am new to Github.


ytdlp-gui: init at 0.2.5

Description of changes

ytdlp-gui is a GUI application for yt-dlp
Things done

Built on platform(s)

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

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

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

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

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

[x] Fits CONTRIBUTING.md.

Runs fine for me

Thanks a lot for reviewing!!
But the git history got messed up with the merge that I did by accident, don't know how to solve it tbh :), will check it later

Interactive rebasing on origin/main or cherry-picking your two commits and force-pushing comes to mind

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

will create a new pull request that fixes all the issues and has good git history when I'm free
thanks for taking the time to review this PR


[ FEAT ] Add last modification date column

We can obtain this information in both stat and statx syscalls.
With this column, we can do something like:
SELECT NAME, LAST_MODIFICATION_DATE FROM . WHERE LEVEL = 1
-- RETURNS THE NAME AND THE LAST_MODIFICATION DATE FROM FILES OF THE CURRENT DIRECTORY.

Getting the time from stat structures, and then parse it.
In Linux, we can use the localtime function.
In Windows, we can use this function


Doc fix for Custom Widget Styles

In each of the Widget Styles "Create a Custom Style" sections e.g. https://ttkbootstrap.readthedocs.io/en/latest/widgets/label.html#create-a-custom-style if the Style.configure method is used the way it is written it will result in an error due to the mixing of keyword and non-keyword arguments. The addition of the "style" keyword in each of these code snippets would fix this.
Style.configure(style='TLabel', font=('Helvetica', 12))
instead of
Style.configure('TLabel', font=('Helvetica', 12))

This is a copy of the traceback. It is solved by having "style" as a keyword argument.

I have no idea. It is working now! Vorführeffekt. Thanks for taking the time.


useFocusEffect should not retrigger on re-render

Reported by in https://github.com/react-navigation/hooks/pull/43/files#r334335721 by @ArrayZoneYour
The following code is likely to produce an infinite loop, even if we make sure the dependency is stable using a ref, because useFocusEffect use "navigation" as a dependency and the object is unstable.
useFocusEffect(useCallback(() => {
  setParamsRef.current({});
},[setParamsRef]));

The focus effect might retrigger unnecessarily due to the navigation being unstable
Somehow this is related to https://github.com/react-navigation/hooks/issues/40
@satya164 what do you think about using a useNavigationRef hook as you suggested in some other issue? Didn't get a clear answer from the React team on how to handle this kind of situation but that should work. Or maybe I should depend only on isFocused and addListener which are the only 2 fn used, but if core does not guarantee stability across render the problem remains.

I think it's okay to use useNavigationRef for the navigation prop for internal implementation of useFocusEffect in this library,

@satya164 I think it will only solve half of the problem for users.
useFocusEffect(useCallback(() => {
   setParams({something: true});
},[setParams]));

The following code will trigger again and again without a stable setParams (cf https://github.com/react-navigation/hooks/issues/35)
If we don't export this API, people will have to find a solution themselves, like:
const useNavigationRef = () => {
  const navigation = useNavigation()
  const ref = useRef(navigation);
  useLayoutEffect(() => {
    ref.current = navigation;
  });
  return ref;
};

const navigationRef = useNavigationRef();
useFocusEffect(useCallback(() => {
   navigationRef.setParams({something: true});
},[navigationRef]));

So basically they end up re-implementing the internally used useNavigationRef hook

Is there any update on this issue ? I'm currently facing the problem and the previous solution (with useNavigationRef) doesn't work for me (I'm using 1.1.0).
thanx
Sami

I can confirm I have this issue as well.
I'm using react-navigation in conjunction with apollo hooks, the idea is to trigger a refetch of the data when the screen is focused. The code is basically the following:
`
const {data, refetch} = useQuery('myGraphqlQuery') //apollo hook
const refetchInACallback = useCallback(() => { refetch()}, [refetch])
useFocusEffect(refetchInACallback)
`
this will trigger an infinite loop of rerendering.


Benchmark investigation

Deep dive into benchmarks and their design

Reopened #24, as it covers this anyway. Closed this one as not planned


Auto scaling

Read about Fenzo in netflix blog. Auto scaling concept sounds interesting. I'm yet to try my hands on Fenzo. Can you please let me know context of auto scaling here? Is it like Fenzo will shutdown/bring up VMs based on the demand ?
Thanks in advance,
Mani

Fenzo provides the logic/smarts to shutdown or bring up VMs based on demand. It provides a callback mechanism. You provide the implementation of the callback inside which you will use your preferred way to bring up the VMs or terminate them. See here.

Thanks @spodila ! Is there a DL where I can reach out if I have any questions ?

I would still maybe add a bit of clarification to the front page, because

it can autoscale the execution hosts cluster based on demand for resources

is misleading, I was trying to find your AWS EC2 autoscaling engine for something like 30 minutes to no avail. Maybe

it's possible to implement cluster autoscaling via provided action callback

would be much more precise?


fix(setup): fix branches setup

Bug introduced by #47

:tada: This PR is included in version 1.10.1 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Support for Cassandra user defined types

Hey there,
Thank you for your work on this amazing library, I was wondering if it was possible to support Cassandra User Defined Types?

Hello there,
I played around with adding support for UDTs in a manual way which is not too unpleasant to work with:
import com.ringcentral.cassandra4io.cql._

object UserDefinedTypeValue {
  implicit val userDefinedTypeValueReads: Reads[UdtValue] =
    (row: Row, index: Int) => (row.getUdtValue(index), index + 1)

  implicit val userDefinedTypeValueBinder: Binder[UdtValue] =
    (statement: BoundStatement, index: Int, value: UdtValue) => (statement.setUdtValue(index, value), index + 1)

  implicit class UserDefinedTypeValueReadsOps(udtReads: Reads[UdtValue]) {
    def map[B](f: UdtValue => B): Reads[B] = (row: Row, index: Int) => {
      val (underlying, nextIndex) = udtReads.read(row, index)
      (f(underlying), nextIndex)
    }
  }

  implicit class UserDefinedTypeValueBinderOps(udtBinder: Binder[UdtValue]) {
    def contramapWithSchema[A](f: (A, UserDefinedType) => UdtValue): Binder[A] = new Binder[A] {
      override def bind(statement: BoundStatement, index: Int, value: A): (BoundStatement, Int) = {
        val udtValue = f(
          value,
          statement.getPreparedStatement.getVariableDefinitions.get(index).getType.asInstanceOf[UserDefinedType]
        )
        udtBinder.bind(statement, index, udtValue)
      }
    }
  }
}

By adding map to Reads and contramapWithSchema to Binder - this gives us a way to easily go back and forth from a UdtValue to Scala datatypes. Here is some usage:
final case class BasicInfo(height: String, weight: String)
object BasicInfo {
  import UserDefinedTypeValue._

  implicit val basicInfoCqlReads: Reads[BasicInfo] =
    Reads[UdtValue].map { udt =>
      val height = udt.getString("height")
      val weight = udt.getString("weight")
      BasicInfo(height, weight)
    }

  implicit val basicInfoBinder: Binder[BasicInfo] =
    Binder[UdtValue].contramapWithSchema[BasicInfo] { case (info, constructor) =>
      constructor
        .newValue()
        .setString("height", info.height)
        .setString("weight", info.weight)
    }
}

This is quite an improvement over the original or doing it repeatedly for UDTs without this mechanism:
  implicit val basicInfoCqlReads: Reads[BasicInfo] = (row: Row, index: Int) => {
    val udt    = row.getUdtValue(index)
    val height = udt.getString("height")
    val weight = udt.getString("weight")
    (BasicInfo(height, weight), index + 1)
  }

  implicit val basicInfoCqlBinder: Binder[BasicInfo] = (statement: BoundStatement, index: Int, value: BasicInfo) => {
    val existingSchema: UserDefinedType =
      statement.getPreparedStatement.getVariableDefinitions.get(index).getType.asInstanceOf[UserDefinedType]

    val udtValue =
      existingSchema
        .newValue()
        .setString("height", value.height)
        .setString("weight", value.weight)

    (statement.setUdtValue(index, udtValue), index + 1)
  }

Would you consider a PR to add this functionality to support UDTs?

Hello, @calvinlfer !
Thank you for you interest in our library! That's a great idea to support UDT and the implementation is good, I would do it that way myself if I had time. Please, do make a PR, it will be a very valuable contribution.

Will do sir, thank you 🙏


Take a screenshot

With a simple press of a button, the user should be able to take a screenshot of the current page they are on and have that image included in their issue report.
Taking a screenshot should not show the widget in the resulting image.

The widget currently returns a html blob and not an image. This will be okay for now, we will send the blob to the backend and there we might do some magic if we really want an image.


null method and gem dimensions

I haven't dug into this, but all of my heaps result in null dimensions in the "Allocations by Method" and "Gems by Allocation Size" pie charts.

Naïvely looking at the heap dump data, I see that many entries do include method and file entries, and that the extractGemName helper does get called with those file names and correctly returns gem names.
Just posting this in case others are experiencing it or know of a solution.

For what it's worth, I got a little deeper into this issue.
var gemDimension = objectsCrossfilter.dimension(function(obj) { return extractGemName(obj.file); });
renderPieChart("#gem-info", gemDimension, groupByAllocationSize(gemDimension));

//...

function groupByAllocationSize(dimension) {
  return dimension.group().reduceSum(function(obj) {
    return obj.memsize || 0;
  });
}

In the snippet above, dimension.group() doesn't correctly group by the dimension. I.e., dimension.group.all() returns just 1 element with a key of null.
I'm at the limit of my DC/D3 understanding, but would be happy to share a heap dump if it helps.

I'm also experiencing this issue with a heap dump that does include method and file entries.


Fixed GeometryRefiner::RefineInterior() to not cause invalid deallocations

Method GeometryRefiner::RefineInterior() was appending integration rules to its internal array even when the rule was existing. This caused consequent invalid deallocations in the destructor and unclean exits / crashes. For example GLVis was affected, when the element refinement was increased and arrows for a vector field were visualized (that is how I noticed that, where instead of normal exit it just threw nice "free(): invalid pointer Aborted (core dumped)" :smile: )

Could someone rerun that Travis CI, it just failed to download hypre.

@najlkin, thanks for tracking down and fixing the bug. Is this ready for review? If so, please add the label ready-for-review.

@v-dobrev it is ready for review (the changes are minimal), but I do not have permissions to add labels as a mere member of mfem/users. That is also the reason why I am making the pull request from my fork again.

@najlkin you are a member of mfem/collaborators now :)

Thanks @tzanio ! :sparkles:

Is the conditional check on the Append() still necessary after the other modifications?  Seems like all cases in the switch either return ir immediately (if non-null) or allocates one.  If the switch reaches the default case, it is an error.
It does not do harm, but seems to imply a null state is an expected condition there, and implies that returning null is an expected return value.
if (ir) { IntPts[Geom].Append(ir); }

Yes, I agree. It is rather internal double check, so an assert suits better.

Merged in next for testing...


Change maxSize on RequestHeaderParser

Hello!
You make a wonderful project, which we use in production. But we have a moment that has to be constantly maintained and remembered about it. This is the maximum size of the buffer, when parsing the request. Now it's 4096 characters, which is not very convenient. Is it possible to make this parameter configurable or get rid of it?
React\Http\RequestHeaderParser, line 18 on v0.7.4

Hey why is the 4096 length limit an issue for you? It is only used for the limiting the request header size and not the body. Are your headers exceeding that size and if so by how much?

Thanks for filing this! I think it makes sense to provide a way to configure this limit and we would welcome PRs for this :+1:
For the reference: Note that this limit only applies to the total maximum size of the incoming request headers, it does not limit the size of the (much bigger) message bodies or the outgoing response headers.

In some cases, we have a more than 4k headers bag. Maximum is 16k

Dang that is a lot , as @clue suggests would you mind creating a PR for it? :)

I'm willing to help here but I'm not sure how we should get the stuff to the right place.. any preferences here?
// Option#1 inject via optional parameter in __construct() OR via setter
$config = new ServerConfiguration($options);
$config->setRequestHeaderBufferSize($maxSize);

$server = new Server($callback, $config);
$server->setServerConfiguration($config);

// Option#2 inject server configuration via environment variables (default: 4096)
setenv('REACTPHP_REQUEST_HEADER_BUFFER_SIZE', $maxSize);
$server = new Server($callback);

// Option#3 setter inside the server
$server = new Server($callback);
$server->setRequestHeaderBufferSize($maxSize);

Tbh I would rather see an extra argument to the constructor with options, like:
$config = [];
$config['max_header_size'] = 1024 * 16;
$server = new Server($callback, $config);

Small status update: We've raised the hard-coded default request header size to 8 KIB via #253 for the v0.8.0 release. This means that this will be less of an issue for most consumers of this package.
That being said, I think we all agree that offering a way to configure this limit is the goal here. We will look into coordinating efforts to get this in in one of the next releases :shipit:

Hi i see this issue mark as easy pick so i would love to give it a try.
from the thing i can read the approach would be:

Adding second parameter "config" to server class constructor.
Adding third parameter "config" to react/http/src/Io/StreamingServer.php constructor.
Adding a contructor to react/http/src/Io/RequestHeaderParser.php where the header max size is pass.

or

Adding second parameter "config" to server class constructor.
Adding a setHeaderMaxSize to react/http/src/Io/StreamingServer.php.
Adding a setHeaderMaxSize to react/http/src/Io/RequestHeaderParser.php where the header max size is pass.

The second list i dont know if would work because the streamingServer constructor get the parser to listen right away, so the set would happen after. Thanks would appreciate some feedback

This doesn't appear to be a high priority issue ever since we've updated the default to 8 KiB, but I still think it makes sense to expose this configuration.
I'm not sure what this could looks like tho. The StreamingServer is marked internal and I don't think this insignificant feature warrants exposing this class as part of our public API. Likewise, the RequestHeaderParser is also entirely internal only.
Any input and or PRs welcome :+1:

This doesn't appear to be a high priority issue ever since we've updated the default to 8 KiB, but I still think it makes sense to expose this configuration.
I'm not sure what this could looks like tho. The StreamingServer is marked internal and I don't think this insignificant feature warrants exposing this class as part of our public API. Likewise, the RequestHeaderParser is also entirely internal only.
Any input and or PRs welcome :+1:

Is this issue still relevant, it looks like the original concern has been resolved, or am I overlooking something?
Reference: I am currently going through all issues and pull requests in react to see if they're still open for discussion or got resolved in the meantime.

I'm closing this for now as it hasn't received any input in a while and I believe this has been resolved in the meantime. If you feel this is still relevant, please come back with more details and we can always reopen this :+1:
Thank you for your effort nonetheless, keep it up! :+1:

We are running into the same issue over here now. Some other application sets rather large cookies (30kb+) on the domain root,  we kind of are forced to receive (running on a subdomain).
If you're ok with it, I'd submit a PR implementing the solution outlined by @Manuelacosta98 :

Adding second parameter "config" to server class constructor.
Adding third parameter "config" to react/http/src/Io/StreamingServer.php constructor.
Adding a contructor to react/http/src/Io/RequestHeaderParser.php where the header max size is pass.

@fritz-gerneth Sounds good, we're always happy about PRs. I could imagine that this involves some bigger adjustments, but if you have a good solution in your mind, I am happy to take a look 👍

We are running into the same issue over here now. Some other application sets rather large cookies (30kb+) on the domain root, we kind of are forced to receive (running on a subdomain).
If you're ok with it, I'd submit a PR implementing the solution outlined by @Manuelacosta98 :

Adding second parameter "config" to server class constructor.
Adding third parameter "config" to react/http/src/Io/StreamingServer.php constructor.
Adding a contructor to react/http/src/Io/RequestHeaderParser.php where the header max size is pass.

@clue We might have to consider configuration middleware or something at this point. Or a configuration object one can pass into the server class to configure certain things.

We might have to consider configuration middleware or something at this point. Or a configuration object one can pass into the server class to configure certain things.

Should this then be merged with the StreamingRequestMiddleware (in a BC way)?

We might have to consider configuration middleware or something at this point. Or a configuration object one can pass into the server class to configure certain things.

Should this then be merged with the StreamingRequestMiddleware (in a BC way)?

Or a new middleware perhaps


hexbinhex: init at unstable-2023-03-31

add hexbinhex as a package
Description of changes
This package contains six utility programs to convert between hex, binary, ascii-binary and the oddball NIST format for 90B testing. The hex output is more compact than hexdump or od -x.

For more information: https://github.com/dj-on-github/hexbinhex
Things done

Built on platform(s)

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

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

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

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

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

[x] Fits CONTRIBUTING.md.

This looks fine. Have you checked if it can cross compile to aarch64?

Yes nix-build -A pkgsCross.aarch64-multiplatform.hexbinhex did work for me.

@ofborg build hexbinhex pkgsCross.aarch64-multiplatform.hexbinhex

Result of nixpkgs-review pr 245844 run on x86_64-linux 1

1 package built:

hexbinhex

@ofborg build hexbinhex


Add availability group name tag

What does this PR do?
Our test environment reports resource_group_id as a readable string, so was used as the identifier string for the tag.  Based on customer reports, this column actually outputs a UUID, while another column name provides the human-readable string.
Instead of replacing the tag availability_group, this PR adds a new one named availability_group_name based on the name column.

Can you include something in the tests for this?

Can you provide more details on the availability_group tag value? If it's always UUID, it might worth replacing it

we would need to check if its being used

we would need to check if its being used

If it's a UUID:

It's definitely not great to send that as a tag.
Hardcoding a query with availability_group:12345 if 12345 is a unique value would make much sense I believe.
It will continue to work for monitors that group by availability_group.

After second thoughts, I think the thing I'm not sure I understand is how much of a UUID is this value. Does it have the same cardinality as availability_group_name (i.e is there a 1-1 mapping between the group ID and its name)? Or is it a unique identifier that is never reported twice?

Does it have the same cardinality as availability_group_name (i.e is there a 1-1 mapping between the group ID and its name)?

Assume so, yes - there should be an alignment with the group name.


Adding a pagination_info helper

It's pretty trivial, but I was thinking this library should included something like this:
iex> Scrivener.HTML.pagination_info(%Scrivener.Page{total_pages: 10, page_number: 5, total_entries: 100, page_size: 10}) |> Phoenix.HTML.safe_to_string()
"<div class="pagination_info">
  Showing 51 to 60 of 100 entries
</div>"

It should be as simple as something like:
def pagination_info(paginator, name \\ "entries") do
  text =
    with first = (paginator.page_number - 1) * paginator.page_size + 1,
         last = min(paginator.page_number * paginator.page_size, paginator.total_entries),
         total = paginator.total_entries,
         do: "Showing #{first} to #{last} of #{total} #{name}"

  content_tag(:div, safe(text), class: "pagination_info")
end

But then what if the user wants this to be different text, or international? Its so simple to create my first thought is that this could cause more problems than it solves.


Octopod Generates Tornado.access Errors when connecting to octoprint.

Octopod Generates the below error:
2019-06-04 07:51:25,292 - octoprint.server.util.sockjs - INFO - New connection from client: ::ffff:<IP_ADDRESS>
2019-06-04 07:51:25,611 - tornado.access - WARNING - 409 GET /api/printer (::ffff:<IP_ADDRESS>) 10.07ms
2019-06-04 07:51:26,080 - tornado.access - WARNING - 409 GET /api/printer (::ffff:<IP_ADDRESS>) 9.16ms
2019-06-04 07:51:26,549 - octoprint.plugins.navbartemp - INFO - Checking SoC internal temperature
2019-06-04 07:51:26,751 - octoprint.server.util.flask - INFO - Passively logging in user _api from ::ffff:<IP_ADDRESS>
2019-06-04 07:51:26,805 - octoprint.server.util.sockjs - INFO - User _api logged in on the socket from client ::ffff:<IP_ADDRESS>
Pi 3b. Octoprint 1.3.11

Hi @evanquinnalaska, could you provide more information about what you were trying to do in the app when the 409 error happened?
Based on OctoPrint's API documentation, the 409 error means that the printer is not operational. Maybe it was printing while you tried to control it? Did OctoPod present any error message indicating that the request failed?

Hi @evanquinnalaska, is this still happening with the latest version of OctoPod?
Thanks,
Gaston

Closing ticket since I think latest fix should have solved this issue. If this is still happening please open a new ticket. Thanks.
Gaston


Dissappearing HTML scripts in Project Online environment

Target SharePoint environment
SharePoint Online
What SharePoint development model, framework, SDK or API is this about?
SharePoint REST API
Developer environment
Windows
What browser(s) / client(s) have you tested

[X] 💥 Internet Explorer
[X] 💥 Microsoft Edge
[X] 💥 Google Chrome
[X] 💥 FireFox
[X] 💥 Safari
[ ] mobile (iOS/iPadOS)
[ ] mobile (Android)
[ ] not applicable
[ ] other (enter in the "Additional environment details" area below)

Additional environment details

browser version
SPFx version
Node.js version
etc

Describe the bug / error
Issue description:
From one day to another (without changing, editing or deploying something) some of our PDP’s did not work anymore, this affected all projects in the environment. Below you can see an example of one PDP.

Issue Analysis:
We are using several different JS scripts to customize the layout of several PDP’s. We found that those JS scripts did not load correctly because they could not be found and therefor the PDP’s were not displayed correctly (see screenshot below from Dev Tools).

When analyzing the issue further why the JS scripts could not be found we saw that the HTML files which are implemented in the PDP’s and referencing the JS scripts contained wrong folder URL’s --> the folder where the JS scripts were saved did not exist.
Usually the URL looks like this:
src="../SiteAssets/01_PDP/bbrDeactivateEnterpriseCustomFields.js"
But instead, the URL in all HTML files looked like this:
src="01_PDP/bbrDeactivateEnterpriseCustomFields.js"
	The reference “../SiteAssets/” was completely missing, in all HTML files which were placed in the Site Assets folder
Further information:
The issue came up already twice and there were no manual changes to the HTML files both times, the issue just came up “over night”.

first time on the 17th of September 2021 and affected all our environments (Production, QAS, Dev and Sandbox)
second time on the 6th of May 2022, affected only the Production environment

I kindly ask you to look into this issue as it already came up twice without any manual adjustments from our side and affected our Production environment extremely!
Steps to reproduce
n.a.
Expected behavior
n.a.

@Reisskocher - thanks for reporting this one!
One question - have you (or your customer where it's happening) recently enabled Office 365 CDN?

hello @AJIXuMuK , thank you very much for your prompt reply. I forwarded the question to our O365 Admin and will get back to you asap!

hello @AJIXuMuK , I received the information from our O365 Admin that Office 365 CDN isn't enabled.


nuget package not getting uploaded on successful released build

Hi,
We need to upload a package on NuGet, the build got successful & We have cross-checked everything at our end.
But, still it is not getting uploaded, below is the error:
Failed to read the package file. Ensure it is a valid NuGet package with a valid manifest.
Below is my VS version:
Visual Studio Community 2019 for Mac
Version 8.6 (build 4520)
Installation UUID: 9dd2c7ff-ef51-46d5-adbe-bfb60909615d
GTK+ 2.24.23 (Raleigh theme)
Xamarin.Mac <IP_ADDRESS> (d16-6 / 088c73638)
Package version: 610000104

Kindly provide the solution asap, it is very critical for us.

How is the package being created?

Hi Claire,
It is published now, but added some suffix to package name.
Will remove those suffixes with new release.
And, thank you for your help. :)


Need to hire a Developer to deploy and develop on Peatio

HI,
I need to hire a Developer to deploy and develop an exchange on Peatio.
email me-<EMAIL_ADDRESS>

we install all coins
https://peatio.io

Hi,
If you are planning to develop and deploy cryptocurrency exchange platform on Peatio script, we are a group of skilled and proficient cryptoexchange developers who are specialized in everything related to Peatio from architecture, hosting, MVP and CI. If you have any query, you may connect us via Skype. Here is the username –<EMAIL_ADDRESS>Website : https://antiersolutions.com/cryptocurrency-exchange-development-company.html
Email us<EMAIL_ADDRESS>


New App Initialization with PostgreSQL Database Setup and Schema Creation

Hello My Dearest Reviewer 😃
This pull request initiates the development of our new app, named "MeaningfulApp," setting up the project structure and database configurations using PostgreSQL. Here are the key components implemented:

Project Initialization: The project is initialized with Git version control for seamless collaboration and tracking changes.

PostgreSQL Database Setup: The PostgreSQL database is configured, ensuring compatibility and scalability for our app's data needs.

Schema Creation: The project schema is meticulously designed based on the provided ERD (Entity-Relationship Diagram) and naming conventions. Here are the notable schema details:

Users Table: Created with columns such as id (primary key), username, email, and photo (stored as a link to an image).
Posts Table: Includes columns for id (primary key), title, content, author_id (foreign key referencing id in the Users table), and any other required attributes following the ERD.

Migration Files: Migrations are created and executed to ensure seamless database changes without data loss or inconsistency.

Foreign Keys and Indexes: Foreign keys are established in the Posts table, with the column linking to the Users table appropriately named author_id. Corresponding indexes are created for all foreign key columns to enhance query performance.

This implementation aligns precisely with the provided specifications and maintains consistency with the ERD, ensuring a robust database structure for the app.
Additional Notes:

The schema design and table/column names strictly adhere to the guidelines and references provided in the sneak peek.
The use of Git version control ensures a traceable history of changes for future iterations and collaboration.

Please review the changes made and ensure they meet the project requirements. Any feedback or suggestions for improvement are highly appreciated.

@zeff96  Thank you so much sir...


Add tasks 171-175

Adding more datasets from SPL project

Thanks!
Now that we're adding more tasks for other languages, wondering if you can help us address #113, before we merge this PR.

Can't these MT tasks also go the other way around? For example, Spanish to English too.

Looks good to me, thanks!

After adding all tasks with English sources, I can continue adding other directions if the main authors think this is indeed a good addition.

Sure we would be happy to see them in future PRs,!

Updated Readme to resolve merge conflicts.


conda: add libgdf_cffi to notebook env

The demo uses the notebook env, which doesn't have libgdf_cffi. This results in
ImportError: No module named 'libgdf_cffi'

Thanks!


Restore Room Player Authority

ReplacePlayerForConnection strips authority so this reapplies it.

should we merge this already or wait for paul's replaceplayerforconnection pr?

should we merge this already or wait for paul's replaceplayerforconnection pr?

Merge now - This will work regardless, and I don't want another version to go to the asset store that's broken.

:tada: This PR is included in version 5.1.8 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


The toggle keymap is duplicated with macOS editor:delete-to-end-of-subword

As the title.

Welp.. Any ideas? I changed it because ctrl-shift-t launches a terminal in linux. #98

I think ctrl alt T is okay.

Er sorry, typo. I meant ctrl-alt-t launches terminal.

Actually you could change the keymap config into this:
'atom-workspace':
  'ctrl-alt-n': 'split-diff:next-diff'
  'ctrl-alt-p': 'split-diff:prev-diff'
'.split-diff':
  'ctrl-alt-,': 'split-diff:copy-to-left'
  'ctrl-alt-.': 'split-diff:copy-to-right'
'.split-diff-ui':
  'ctrl-alt-,': 'split-diff:copy-to-left'
  'ctrl-alt-.': 'split-diff:copy-to-right'
'.platform-darwin atom-workspace':
  'ctrl-alt-t': 'split-diff:toggle'
'.platform-win32 atom-workspace, .platform-linux atom-workspace':
  'ctrl-alt-d': 'split-diff:toggle'


Does it support texture ?

Hi,
We are trying to use this loader on windows. Unable to load texture with any model. Does it support texture along with the model?
Thanks,
Prakhar

https://github.com/syoyo/tinygltf/issues/291


Adding HYAS Protect App

Please ensure your pull request (PR) adheres to the following guidelines:

Please refer to our contributing documentation for any questions on submitting a pull request, link: Contribution Guide

Pull Request Checklist
Please check if your PR fulfills the following requirements:

[x] Testing of all the changes has been performed (for bug fixes / features)
[x] The readme.html has been reviewed and added / updated if needed (for bug fixes / features)
[x] Use the following format for the PR description: <App Name>: <PR Type> - <PR Description>
[x] Provide release notes as part of the PR submission which describe high level points about the changes for the upcoming GA release.
[x] Verify all checks are passing.
[x] Do NOT use the next branch of the forked repo. Create separate feature branch for raising the PR.
[x] Do NOT submit updates to dependencies unless it fixes an issue.

Pull Request Type
Please check the type of change your PR introduces:

[x] New App
[ ] Bugfix
[ ] Feature
[ ] Code style update (formatting, renaming)
[ ] Refactoring (no functional changes, no api changes)
[ ] Documentation
[ ] Other (please describe):

Security Considerations (REQUIRED)

[ ] If you are exposing any endpoints using a REST handler,
please document them in the readme.html.
[ ] If this is a new connector or you are adding new actions

[ ] Please document in the readme.html all methods (eg, OAuth) used to authenticate
with the service that the connector is integrating with.
[ ] If any actions are unable to run on SOAR Cloud, please document this in the readme.html.

[ ] Are you introducing any new cryptography modules? If yes, please elaborate their purpose:
[ ] Are you are accessing the file system? If yes, please verify that you are only accessing paths returned through
the Vault API.
[ ] Are you are marking code to be ignored by Semgrep with nosemgrep?
If yes, please provide justification in an additional comment next to the ignored code.

Release Notes (REQUIRED)

Provide release notes as part of the PR submission which describe high level points about the changes for the upcoming GA release.

What is the current behavior? (OPTIONAL)

Describe the current behavior that you are modifying.

What is the new behavior? (OPTIONAL)

Describe the behavior or changes that are being added by this PR.

Other information (OPTIONAL)

Any other information that is important to this PR such as screenshots of how the component looks before and after the change.

Pay close attention to (OPTIONAL)

Any specific code change or test case points which must be addressed/reviewed at the time of GA release.

Screenshots (if relevant)

Thanks for contributing!

Hi @RamboV, please let me know once you complete the development so I can prioritize this PR for review.

Hi @RamboV, to fix the no code const error, you can move the constants to the consts file the way it was present in your initial commit https://github.com/splunk-soar-connectors/hyasprotect/pull/1/commits/b3239112ef67b927020c0050282ae4460ca73f24

Hi @RamboV, to fix the no code const error, you can move the constants to the consts file the way it was present in your initial commit b323911

Hi @dvora-crest, thank you we have moved everything to constant file, can you kindly check and let us know what else we need to attend to make all the checks successful.

Hi @RamboV, I have addressed the static test failure. The semgrep check is expected to fail against forks.

Hi @RamboV, I have addressed the static test failure. The semgrep check is expected to fail against forks.

Hi, @dvora-crest Thank you. Can you tell us the timeline for the review to be completed?
FYI we will be one more package tomorrow.

Thanks for confirming @RamboV

The semgrep check is expected to fail against forks. Following is the result of the local semgrep run.


V1.18.1 rebase master

@theishshah we need to

drop the 3 patches: 17, 18, 19.
update the version in patches/03-setversion.patch

That should fix this PR.

/retest


Convert oqs-kem-info.md code points to hex

Please give a brief explanation of the purpose of this pull request.

Minor documentation update to use hexadecimal notation for KEM code points.

Does this PR resolve any issue?  If so, please reference it using automatic-closing keywords like "Fixes #123."

Not that I'm aware of.

Notes
The initial motivation for this PR was a small documentation change. That change uncovered an unexpected warning message. It appears that the erroneous warning message motivating this commit is due to some vestigial code special-casing x-curves. The nid_ecx_hyybrid kem field was only used to generate the duplicate ecx nid warning message, so we track duplicates in a local set instead. Additionally, we expand the warning message to all curve NIDs, not just x-curves and treat duplicates as fatal because extra curve NIDs should be unique for each KEM.
Duplicate NID detection is tested below:
$ git diff
diff --git a/oqs-template/generate.yml b/oqs-template/generate.yml
index 4865eb7..fa3ddfe 100644
--- a/oqs-template/generate.yml
+++ b/oqs-template/generate.yml
@@ -114,6 +114,8 @@ kems:
           nid: '0x2F90'
         - hybrid_group: "x25519"
           nid: '0x6399'
+        - hybrid_group: "p521"
+          nid: '0x6399'
         - hybrid_group: "p256"
           nid: '0x639A'
       old:

$ LIBOQS_SRC_DIR=$(pwd)/../liboqs python3 oqs-template/generate.py
ERROR: duplicate hybrid NID for kyber768 : 0x6399 in generate.yml. Curve NIDs may only be specified once per KEM.

$ echo $?
1

I'm not sure what's up with the CI; I don't appear to have permissions to view the build errors in CircleCI. I ran the tests locally on a linux box:
$ cmake --build build && ( cd build; ctest -V )
...
100% tests passed, 0 tests failed out of 5

Total Test time (real) =  29.85 sec

Thanks for the update: This is now perfect to retain.

some guidance on that warning's original intent

This is basically a question/reminder to @bhess to re-evaluate the original ECX logic (via an extra-YML file) he originally introduced. IMO it could be removed as #177 completely changed the build logic to permit more than a single X-hybrid for the same PQ alg.

@WillChilds-Klein Thanks again for the contribution! CI has run successfully so we'll merge this tomorrow. As this is more than a trivial change, please consider amending this PR before merge by adding yourself to the Contributors list in the README.


Improv/api initialization speed

Description
In the edge case where a dangling image could not be deleted (e.g. the
parent image failed to start but is never removed using
docker rm $(docker ps -a -q)) we would sleep for 1 second and retry
this 10 times. This would cause the orchest-api to be unresponsive and
thereby making the FE feel unresponsive for views (e.g. /projects)
that query the orchest-api.
Ideally the host machine has a clean docker environment, i.e. no
dangling images, but we can't guarantee this. Luckily, this edge case
should be rare for users and only happen for developers of Orchest.

Good exploration 👍🏻 the proposed LGTM as well.

lgtm, I have left a comment about something we are discussing in another PR, not feeling strongly about it but keen to know your opinion.


When changing a job from cron to one-off after duplication it still runs as a cron job

Describe the bug
When changing a job from cron to one-off after duplication it still runs as a cron job
Environment

OS: Linux
Browser: Brave
Version: v2021.11.11
Environment: prod, cloud

Might be related to https://github.com/orchest/orchest/issues/466.


Support for "default branch"

It would be nice if Gogs could support a repository setting like GitHub's Settings -> Branches -> Default branch.

We have that, It's under Settings -> Options -> Default Branch

Bah I'm blind, sorry for the noise.


extending NuGetV3 source detection with Artifactory feed format

extending NuGetV3 source detection with Artifactory feed format
www.jfrog.com/confluence/display/RTF/NuGet+Repositories

@matthid any chance you can release it?
Viktor Svub<EMAIL_ADDRESS>schrieb am Mo., 25. Juni 2018, 12:31:

extending NuGetV3 source detection with Artifactory feed format
www.jfrog.com/confluence/display/RTF/NuGet+Repositories
https://www.jfrog.com/confluence/display/RTF/NuGet+Repositories#NuGetRepositories-NuGetAPIv3RegistrySupport
You can view, comment on, or merge this pull request online at:
https://github.com/fsprojects/Paket/pull/3267
Commit Summary

Merge upstream master/5.171.0 from fsprojects/Paket
Merge upstream master/5.172.2 from fsprojects/Paket
extending NuGetV3 source detection with Artifactory feed format

File Changes

M src/Paket.Core/Versioning/PackageSources.fs
https://github.com/fsprojects/Paket/pull/3267/files#diff-0 (2)
M tests/Paket.Tests/Paket.Tests.fsproj
https://github.com/fsprojects/Paket/pull/3267/files#diff-1 (1)
A tests/Paket.Tests/Versioning/PackageSourceSpecs.fs
https://github.com/fsprojects/Paket/pull/3267/files#diff-2 (40)

Patch Links:

https://github.com/fsprojects/Paket/pull/3267.patch
https://github.com/fsprojects/Paket/pull/3267.diff

—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/fsprojects/Paket/pull/3267, or mute the thread
https://github.com/notifications/unsubscribe-auth/AADgNHtB45z3qA5kPmLI1EVjAiduQILMks5uALwagaJpZM4U15kN
.


nm.checkpoint: fix incorrect interval when adding a timer

According to GLib documentation the interval specified when using
timeout_add() is on miliseconds.
Signed-off-by: Fernando Fernandez Mancera<EMAIL_ADDRESS>

Coverage remained the same at ?% when pulling a7039c77de05ad1e02570b3e2b588b5adac6df14 on ffmancera:timeout_miliseconds into e7b7c6abea8fdbe3adc6260f8ad0b7d483a1a674 on nmstate:master.


Add python 3 compatibility without affecting python 2 functionality

This PR implements compatibility with python 3 using the conventions here to maintain simultaneous compatibility between python 2 and 3: http://python-future.org/compatible_idioms.html
It updates import statements, and raise, map and queue usage. It requires the "future" module on install, required to enable this. It also updates argument format to the objc library.
We hope sharing these changes can make your repo useful to others with python 3. Thanks for open sourcing in the first place :)

Thanks for sending the pull! I tried it out on both OSX and Linux and on OSX with homebrew python (3.6.1) I couldn't get pybojc to work.  I installed it as they instruct from pip (https://pythonhosted.org/pyobjc/install.html ) but whenever it calls into the CoreBluetooth, etc. APIs it just segfaults:
tony-imac:examples tony$ python3 list_uarts.py

Using adapter: Default Adapter

Searching for UART devices...

Press Ctrl-C to quit (will take ~30 seconds on OSX).

2017-07-24 15:54:01.622 Python[38170:8013081] *** Terminating app due to uncaught exception 'OC_PythonException', reason: '<class 'AssertionError'>: <memory at 0x10264d588>'

*** First throw call stack:

(

0   CoreFoundation                      0x00007fff95236452 __exceptionPreprocess + 178

1   libobjc.A.dylib                     0x00007fff961dcf7e objc_exception_throw + 48

2   CoreFoundation                      0x00007fff95236399 -[NSException raise] + 9

3   _objc.cpython-36m-darwin.so         0x00000001026ae49e PyObjCErr_ToObjCWithGILState + 46

4   _objc.cpython-36m-darwin.so         0x00000001026b6172 -[OC_PythonObject forwardInvocation:] + 866

5   CoreFoundation                      0x00007fff951a62d2 ___forwarding___ + 514

6   CoreFoundation                      0x00007fff951a6048 _CF_forwarding_prep_0 + 120

7   CoreBluetooth                       0x00007fff8b5ea346 -[CBCentralManager handlePeripheralDiscovered:] + 879

8   CoreBluetooth                       0x00007fff8b5eaa93 -[CBCentralManager xpcConnection:didReceiveMsg:args:] + 314

9   CoreBluetooth                       0x00007fff8b5f47de __34-[CBXpcConnection handleMsg:args:]_block_invoke + 77

10  libdispatch.dylib                   0x00007fff8f48393d _dispatch_call_block_and_release + 12

11  libdispatch.dylib                   0x00007fff8f47840b _dispatch_client_callout + 8

12  libdispatch.dylib                   0x00007fff8f48bc1c _dispatch_main_queue_callback_4CF + 1685

13  CoreFoundation                      0x00007fff951eb949 __CFRUNLOOP_IS_SERVICING_THE_MAIN_DISPATCH_QUEUE__ + 9

14  CoreFoundation                      0x00007fff951aa82d __CFRunLoopRun + 1949

15  CoreFoundation                      0x00007fff951a9e28 CFRunLoopRunSpecific + 296

16  Foundation                          0x00007fff894f7f99 -[NSRunLoop(NSRunLoop) runMode:beforeDate:] + 270

17  _objc.cpython-36m-darwin.so         0x00000001026c28c7 ffi_call_unix64 + 79

18  ???                                 0x00007fff5e0ba460 0x0 +<PHONE_NUMBER>09312

)

libc++abi.dylib: terminating with uncaught exception of type NSException

Abort trap: 6

With the Apple-provided pyobjc bridge in Apple's Python 2.7 it still works great though:
tony-imac:examples tony$ python list_uarts.py

Using adapter: Default Adapter

Searching for UART devices...

Press Ctrl-C to quit (will take ~30 seconds on OSX).

Found UART: Bluefruit52 [85433902-d600-4a12-b17f-ef983172355f]

So I'm not sure what's up with pyobjc and why it's failing with later versions.  I'll still pull in the change though just with the note to folks using Python 3 that I'm not sure pyobjc works well for it right now (at least in my testing on OSX El Capitan).
For the Linux side I added a few changes to work with the future library and tried it out with the latest bluez 5.46 on a Pi with CSR810 BLE dongle.  Seems to work great with python 2 and 3 now.  Thanks again!


ERI index corresponding to the two-electron integrals

After the Hartree-Fock calculation, we can get the one and two-electron integrals from
h1e = reduce(np.dot, (mf.mo_coeff.T, mf.get_hcore(), mf.mo_coeff)) # (C^T)HC
eri = ao2mo.kernel(mol, mf.mo_coeff, compact=False)

I wanted to use them to calculate the Hartree-Fock energy with the formula as
$$E_{HF} = \langle\Phi_0 | \hat{H} | \Phi_0\rangle =  \sum_i^N [i|\hat{h}|i] + \sum_{i>j}^N  \left([ii|jj] - [ij|ji]\right)$$
and my code reads
import numpy as np
from pyscf import gto, scf, ao2mo
atoms_CH4 = [ # CH4
    ['H',    0.0000,    0.9870,    0.0000],
    ['C',    1.0390,    0.9870,    0.0000],
    ['H',    1.0390,    1.9740,    0.0000],
    ['H',    1.0390,    0.0000,    0.0000],
    ['H',    2.0260,    0.9870,    0.0000]]
mol = gto.Mole()
mol.atom = atoms_CH4
mol.basis = 'sto-3g'
mol.verbose = 0
mol.build()
mf = scf.RHF(mol)
mf.kernel()
from functools import reduce
h1e = reduce(np.dot, (mf.mo_coeff.T, mf.get_hcore(), mf.mo_coeff)) # (C^T)HC
eri = ao2mo.kernel(mol, mf.mo_coeff, compact=False)
eri = eri.reshape((mol.nao,mol.nao,mol.nao,mol.nao))

hf_str = np.array([1,1,1,1,1,0,0,0,0])
first_part = np.sum(np.diag(hf_str) * h1e)
second_part = 0
for i in range(1,5):
    for j in range(0,i):
        second_part += eri[i][i][j][j]-eri[i][j][j][i]
print('HF energy    = ', mf.e_tot-mol.energy_nuc())
print('my_HF energy = ', first_part+second_part)

but the result reads
HF energy    =  -54.05508294707886
my_HF energy =  -35.712483622065726

I'm wondering if the indexes of eris I used are wrong???
Thank you very much!!!

The problem is that your formula is based on spin-orbitals (Szabo 2.111) but the MO integrals in pyscf are integrals of spatial orbitals. So you need to derive an RHF energy formula on spatial orbitals (Szabo 2.177) and adjust the code.

The problem is that your formula is based on spin-orbitals (Szabo 2.111) but the MO integrals in pyscf are integrals of spatial orbitals. So you need to derive an RHF energy formula on spatial orbitals (Szabo 2.177) and adjust the code.

It works, Thanks you very much !!! ❤❤❤


Feature request: Cliff Sensor Intensity Values

Discussed in https://github.com/iRobotEducation/create3_docs/discussions/319

Originally posted by Kola50011 January 24, 2023
Hey!
I am working on a project, where we would like to detect a black line on the floor using the 4 cliff sensors. Sadly, this information not published by the ROS API. Is there any way to access such information, other than the /hazard_detection topic? We  would greatly appreciate something similar to the /ir_intensity topic.

Hello, I saw that this feature was removed from H.1.0 - any plans when this might be added? I would really appreciate native support instead of having a workaround and mounting my own, additional sensors to the robot.

Hi @Kola50011! Not to worry, this feature is still in development. As soon as there is a planned release, I will let you know via this issue.

What is the ETA on this feature?
Thank you,

It is the team's intention to release this very soon.


cors

@EzraWeller I reoganized the endpoints and added cors here. Could you do a quick review and merge?


Add IJ EAP release to our build matrix

So that we can catch failures ealier

oops. this is a dup.


Update certification-faq.md

A revised/updated  FAQ content is available here: https://docs.google.com/document/d/1TqOIh8RlYdtuV400XMn-LnobzQCrobNcKdt_YKsO-m8/edit. I would like to propose substituting the current help doc with the material from the document aforementioned.
Purpose of this pull request
This pull request (PR) ...
Issue number:

Select the type of change:

[ ] Minor Changes - Typos, formatting, slight revisions
[x] Update Content - Revisions and updating sections
[ ] New Content - New features, sections, pages, tutorials
[ ] Site and Tools - Updates, maintenance, and new packages for the site, Gatsby, React, etc

I would like to propose substituting the current help doc with the material from the document aforementioned.

@zazocar , were you planning to do this or were you looking for a resource to help? cc: @stacykor


Missing or mutilevel grains break rendering

By default the master will checkout a token with a vault policy for all salt minions as well as a single minion. But for the case where you have multiple clusters of machines with different roles, you may want to create a policy for the environment of each cluster and/or a specific role within that cluster. In our environment that means:
policies:
  - saltstack/minions
  - saltstack/minion/{minion}
  - saltstack/minions/{grains[ec2_tags:Environment]}/{grains[ec2_roles]}

In that last example, if a host doesn't have an ec2_roles grain, the mapping code crashes. It is also unable to deal with evaluating sub grains in the ec2_tags:Environment case.

Aha, good catch. I'll add some error handling there.
For the nested grains, it gets a bit tricky - that is actually just a salt helper syntax, which is also configurable, I believe. So it might be hard to implement correctly. But since it is just nested dictionaries, you can use this (although less compact):
{grains[ec2_tags][Environment]}

Super! I'd expect that a missing gran gets mapped to a "missing!" or some not likely to be already used string.
For the substring, if that syntax works, that's great.

Hm, I'm a bit wary of assigning a policy which "should never exist". Those things tend to bit back eventually ;)
Just pushed a PR, #13, which logs an error. Are there cases where this is insufficient?

Works for me

This came back after the tokenization code.
With a policy like: saltstack/minion/{grains[ec2_tags][Environment]}/{grains[ec2_roles]} where Environment is undefined
 salt `hostname -f` vault.read_secret secret/salt-master/jenkins
  salt-master:
    ERROR: {'error': "'Environment'"}

Hm, but ec2_tags is defined? I'm having problems reproducing this, but I'll add a few test cases and see if I can catch it.

Could you share those grains? I've tested with this, and it looks fine here:
ec2_tags:
  Not-Environment: unit-test
ec2_roles:
  - test-runner
  - hamster

Here's the reported error:
Exception occurred in runner vault.show_policies: Traceback (most recent call last):
  File "/usr/lib/python2.7/site-packages/salt/client/mixins.py", line 395, in _low
    data['return'] = self.functions[fun](*args, **kwargs)
  File "/var/cache/salt/master/extmods/runners/vault.py", line 50, in show_policies
    return _get_policies(minion_id, config)
  File "/var/cache/salt/master/extmods/runners/vault.py", line 77, in _get_policies
    for expanded_pattern in _expand_pattern_lists(pattern, **mappings):
  File "/var/cache/salt/master/extmods/runners/vault.py", line 113, in _expand_pattern_lists
    (value, _) = f.get_field(field_name, None, mappings)
  File "/usr/lib64/python2.7/string.py", line 640, in get_field
    obj = obj[i]
KeyError: 'Environment'

From the policy:
saltstack/minion/{grains[ec2_tags][Environment]}/{grains[ec2_roles]}

For a host that does have an ec2_tags grain, but not an ec2_tags:Environment grain.
ec2_tags:
        ----------
        Name:
            SaltMaster
        Roles:
            SaltMaster
        aws:cloudformation:stack-name:
            Saltstack-Master
        cost_allocation:
            Saltstack

If the above policy is removed from the list, vault.show_policies is then able to render properly
# salt-run vault.show_policies $(hostname -f)
- saltstack/minions
- saltstack/minion/ip-10-241-208-27.ec2.internal
- saltstack/minion/roles/SaltMaster

Argh! Nevermind. Once again, I had an older checkout. Current master doesn't have this issue.
Thanks again!