upgrade dyaml to 0.5.3

turns out fixing #67 is as easy as this :)

edit: I also had to upgrade vibed due to a build error on my machine :/

I explicitely set the dyaml version to 0.5.3 because the newer one doesn't compile with GDC (7073ad881e5f0cf98346610db125b8dfdd06db15). Unfortuntely the Docker images is currently built with GDC because it supports static linking and allows a very slim Docker image. The latest LDC supports static linking so that might wirth a shot. Or GDC gets a newer fromtend version :-)

.. And vibe 0.7.29 isn't even released? Which compiler do you use, I didn't have problems compiling that setup.

The latest LDC supports static linking so that might wirth a shot.

LDC is a lot more up-to-date than GDC
Their latest release is even at 2.70.2!
https://github.com/ldc-developers/ldc/release
But unfortunately it doesn't compile this project - I get a lot of error messages :/
/usr/local/bin/ld: error: cannot find -lssl
/usr/local/bin/ld: error: cannot find -lcrypto
/usr/local/bin/ld: error: cannot find -levent_pthreads
/usr/local/bin/ld: error: cannot find -levent
/usr/local/bin/ld: error: cannot find -lssl
/usr/local/bin/ld: error: cannot find -lcrypto

.. And vibe 0.7.29 isn't even released? Which compiler do you use, I didn't have problems compiling that setup.

DMD64 D Compiler v2.071.0
Do you by chance have a different dub.selections.json?
One normally shouldn't put this file to the gitignore ;-)

But unfortunately it doesn't compile this project - I get a lot of error messages :/

You need to install libevent and libssl, see https://github.com/rejectedsoftware/vibe.d#user-content-additional-setup-on-linux-debianubuntumint.

@MartinNowak maybe my message was confusing - I meant static linking with ldc doesn't work either. Compiling and dynamic linking works fine with dmd.

@wilzbach Thanks for the hint. I never thought about dub.selections.json and more suspected it being some nasty autogenerated file :-) I'll remove it from gitignore and checkin my current version.

@wilzbach I'll give the latest LDC release a try. 0.17 already supports static linking. That is really a pain to not be able to compile using the latest DMD.

@wilzbach I'll give the latest LDC release a try. 0.17 already supports static linking. That is really a pain to not be able to compile using the latest DMD.

I am not an expert in linking, but can't we use dmd to compile and then use gcc for the static linking?

I am not an expert in linking, but can't we use dmd to compile and then use gcc for the static linking?

Me neither :-) I am sure there will be a way to fix this. I'll try to find some time the next days to look into this.

I just pushed 0c8a39a which should solve all issues because LDC 0.17.1 is working just fine.

Static linking is a weird go-ism, not supported on most distributions, due to security issues (think of fixing heartbleed by updating openssl if everyone used static linking, also ASLR can only be done effectively w/ dynamic libraries)
You prolly do this b/c you want to run this in a stripped down docker base OS, and my longterm recommendation is to stop wasting your time using a crappy technology as docker (which also has several security and performance issues, and whose complexity is only worth if you run a PaaS). Rather use user friendly and simple containerization such as lxc. It also makes a lot more sense, to ship a 1MB executable rather than a 50MB OS and a 50MB static binary.
 Already wasted almost a whole week on docker before realising hows crappy it was and how much complexity it adds even to simple deployments, but maybe it's the right tool for spawning sessions.
For drepl I used selinux sandboxes and plain old user limits (cgroups alone can't avoid fork bombs). Much safer than using docker without selinux, and also avoids the slow layered fs.
That said static binaries do have some use-cases (distribution independent linux binaries), but unless you release closed source binaries, you hardly need anything to run on every platform.
See Issue 6952 - Static Linking on Linux.

I see you have execution providers, I might just make an adapter for selinux sandboxes.

Thank you for the long and interesting rant :-)

You prolly do this b/c you want to run this in a stripped down docker base OS
That's right. The dlang-tour is running in a Docker container in production and has a size of about 19MB. The base OS is busybox which is really small compared to an Ubuntu base which is more thatn 100MB.

Sorry for the long rant, but I hate seeing people being pushed into stupid technological directions for commercial reasons

I do understand your arguments but the choice for Docker was not for commercial reasons but because I have a lot of experience with Docker also when used in production environment. I also do agree that Docker has a lot of quirks and sometimes you get the feeling the software is getting worse with every version. But if you know the quirks and can live with them you get a lot of benefits (IMO):

Easy deployment: as the deploy guide shows the deployment is super easy using docker compose. Its just a matter of minutes of getting the tour ready for production on any (Linux) server. And you don't need any other orchestration system to get it up and running.
Distribution: Docker features a registry and allows simple pushing/pulling of images from everywhere
Automatic updates are very simple to implement (see deployment guide). Currently a new push creates a Docker image on Travis CI and the server checks itself whether their is a new image and restarts the service if needed. That leads to zero maintenance for regular updates.
Testability: the Docker image you create is the same everywhere. On your local box, on production etc. That facilitates testing.

Of course it might be a matter of taste but also given all its problems I currently don't know any other full-stack solution which simplifies the distribution and configuration of cloud services.

also has several security and performance issues

Which do you mean? I mean it depends on the use case but generally you shouldn't have a big difference in performance compared to bare metal. And it is certainly better than virtualized hosts.

I see you have execution providers, I might just make an adapter for selinux sandboxes.

Any other level of execution provider is very welcome! Actually Docker wasn't the first choice because of sandboxing but because of the other reasons listed above. But still it made creating a temporary sandboxed environment a no-brainer...


docs: modify ts branch setup in readme

Why
ts branch has some differences with js branch.  even the ts branch is on building, we still need to update the readme to make sure it can be setup .

With a post request like this:
POST  /sources/${source id}
{
    collector: ['Issue'],
    enricher: ['LeadTime'],
    options: {
        projects: ['ProjectName'],
        boards: ['Scrum Board Id']
    }
}

response:
{ task: 'task id' }

How does a user set their jira auth tokens?

I also don't see anything on data base migrations. If I make a new collector, where does the data get stored in the code?

How does a user set their jira auth tokens?
We should create jira source first, with the request as following

POST /sources
{
    type: 'Jira',
    options: {
        host: '',
        username: '',
        password: 'password or api token'
    }
}

The jira auth info will be stored with the source.

I also don't see anything on data base migrations. If I make a new collector, where does the data get stored in the code?

Seems database layer is not ready yet.


0.16.0 fails to build on OS X; can't find Python.h

I'm having trouble building scipy 0.16.0 for Homebrew. The Python includes directory doesn't seem to be added to the compiler search path.
Here is a transcript of a failed build: https://gist.github.com/tdsmith/2bd137bf0c502be96746#file-01-python-L4146-L4158
I've tried building this a few ways and I think I'm seeing this building within Homebrew against both system Python and Homebrew Python, with versions of numpy which both do and do not contain the patch in numpy/numpy#5866, but I may be confused.
0.15.1 is building fine. I'll keep looking but I wanted to see if y'all had any insight before I got too deep in the weeds.

It looks like fwrappers which was added in 0.16.0 in scipy/linalg.setup.py is the only lib built with add_library that needs Python.h
I can't reproduce the issue with my older Homebrew install. Can you try if this fixes it: https://github.com/rgommers/scipy/commit/92a47a38e67f13ad72151e046fbc2c72c336edfe ?

It does fix the build. Weirdly, this happens while building against system Python, which shouldn't be exposed to our distutils.cfg. This error shows up on the test bot and I can reproduce it on my machine but  we got a pull request for scipy 0.16.0 from someone who wasn't having any trouble.

-- Thanks for your quick response! Do you think this is the right solution?

I suspect something could be improved in numpy.distutils as well, but the problem is that any fixes there aren't useful - we need to support older numpy versions. So yes, I think this is the way to go. I'll submit a PR.

I'm not sure if the distutils.cfg usage in Homebrew is reversible? That would be an even faster fix, so Scipy 0.16.0 starts building again (my fix can only land in 0.16.1).

This doesn't seem related to our distutils.cfg problems; I can reproduce this with Apple's system python, which doesn't look at our distutils.cfg, and an unpatched numpy 1.9.2.
A delay until a fix lands probably isn't urgent; nobody's complained about build failures on Homebrew with a self-installed numpy because those users are all getting wheels.
Thanks again; I'll add this patch to our formula.

@tdsmith okay, sounds good then. I'm not sure then why I can't reproduce it and others haven't seen this either. But if my patch makes the problem go away, that's good enough for me right now.


Final search result is focused in continue reading

Type of installation
Portable (.zip)
Steps to reproduce

Search for any present name in continue reading.
The final search result record is focused in continue reading.

https://github.com/mienaiyami/yomikiru/assets/127277950/c3cb9c56-004d-4cbb-9ff9-05113adc5e91
✔ Expected Behavior
To not focus the final search result.
❌ Actual Behavior
Final search result is focused.

It was originally just a side effect of adding keyboard controls in search. But I kept it as an indicator that click enter in search now will open the item.
I will remove focus but keep the enter function if it looks annoying.

It's caused by same thing in #225, So I will be closing that.


 There are a lot of changes here, and there are a lot more coming after this one. The primary goal of this PR is to stop exporting Razor services as Roslyn language and workspace services. A secondary goal is to clean up the Visual Studio layer in preparation for fully separating out the legacy editor support into a separate assembly.
Related to #9515

Going to merge this, since it is blocking my next PR.


GitLab API v4 Support?

Hello,
is the GitLab API v4 now supported ?
https://about.gitlab.com/2018/06/01/api-v3-removal-impending/
Thanks
Regards,
Bjoern

Thanks for this note. I've just merged #63 into master.
If you could, please try out the latest master and let me know if you run into any issues. I will try to find some time to do more testing myself, as well. Cheers!

HI @tyler-sommer
I checked out the fresh master branch, run composer install and still getting:
Skipping Exception 'The "https://gitlab.com/api/v3/projects/..." file could not be downloaded (HTTP/1.1 410 Gone)'.

@tyler-sommer
Is there a reason composer/composer is locked at 1.4.3 and libs are replaced by this?

I tried to update the composer to the latest version and replace v3 to v4 here: https://github.com/terramar-labs/packages/blob/master/lib/Composer/Repository/Vcs/GitLabDriver.php#L244
I'm getting Skipping Exception 'Undefined index: public'. now.

@robertpustulka Check these notes for an explanation there. Nice catch, though, that definitely needs to be addressed. Unfortunately, I don't have the bandwidth to get this corrected right now.
The error you're seeing looks to be due to the "public" field in the JSON received not being there in the v4 API https://github.com/terramar-labs/packages/blob/master/lib/Composer/Repository/Vcs/GitLabDriver.php#L296
It would be best to remove the monkey patch altogether, but if thats not possible just updating Composer and then update the monkey patched files, ensuring that these 3 lines to the github driver and these 3 lines to the gitlab driver still exist afterward.

Quick update for those interested, #98 should fix the GitLabDriver.php issues. Please give it a shot and let me know if you run into any further problems. Thanks again!

Hi, my gitlab's version is 7.X and only supports v3 api. Which packages version support gitlab v3 api? I use the latest version and cannot work

Version 3.2 should work for you, @wwulfric. Cheers

There's an undefined function call in the synchronizePackages method. I'll submit a PR asap

This seems to be resolved. If you run into further issues related to this change, please open a new issue. Thanks!


Fixed PhpDoc

According to issue https://github.com/starekrow/lockbox/issues/7, I've fixed PhpDoc for better compatibility with fig-standards

Looks good in principle, but the whitespace changes leaked all over the code. Can you resubmit with just the comments changed?

here is what this looks like after psr-2 is adopoted (#14)
https://github.com/KJLJon/lockbox/commit/601fe28bc547ce3cf0ef1509031d4f8f3e42ebec

@KJLJon LGTM! I think we can merge PSR-2 rules + Fixed PhpDoc together
@starekrow do you approve?

Yep, I'd be happy to take a PR for both. Right now I'm getting a slightly cleaner merge by starting with #8 than #14 but there's a lot of bits to fixup by hand; I'll hold off for now. Definitely want to do a change like this (touching every source line) sooner than later.

KJLJon/lockbox@601fe28 merges #8 and #14 together from the 2 separate timelines / branches.
I will submit a PR for that and if approved it should automatically close both #14 and #8 since it is pulled from both of their histories.


Refactor GHAs

Refactors a lot of our GHAs to pass inputs through the env.
A lot of other random cleanup. I have added a lot of comments on this PR to help with any confusion.
@W-16223935@

QA notes:
✅ getPreReleaseTag
✅ npmPublish
✅ create-github-release (tested with prereleases)
❌ NUTs fail:
https://github.com/salesforcecli/plugin-data/actions/runs/10492620689/job/29064652653
I think it's because the checkout happens after setup-node:
https://github.com/search?q=repo%3Aactions%2Fsetup-node %2FSupported file patterns%2F&type=code
✅  doesn't execute ls or $(ls) when printing from output via env var (also couldn't find any workflow that picks up user-content and tries to echo it (validatePr is already fixed)


Make new options and target framework accessible to nuget consumers

I've had a go at what it could look like instead. The new options bit would be a bit simpler if we changed the converter to be non-static.
I've gone for a non-breaking change instead for now but it leads to lots of overloads which wouldn't be needed if ConversionOptions was a property on the converter instead.
@hvanbakel

I had a few other tries, for example I added an option parameter in the Transform method of ITransformation. But I really like your version, it is much better if we set the options when we create the ITransformation implementations.
How do you think about adding the TargetFrameworks option to ConversionOptions? If we add this option, we could also move the creation of TargetFrameworkTransformation as a pre converter in the ProjectConverter.

@mungojam btw, thanks a lot for your pull request. I will merge it for sure.

How do you think about adding the TargetFrameworks option to ConversionOptions?

Good idea yeah!


Buildings data

Overview
This PR includes final CSV data summarized state-level building counts and key stats from NREL data. Additional scripts include exploratory OSM count data and sanity checks for Microsoft Data.
Prepared data available here.
Closes #21
Notes
Documented as well in scripts/buildings.md:
Buildings Data Notes
Sources:

Microsoft building footprint counts
National Renewable Energy Lab building energy types (see Commercial, Residential, also, direct links to ComStock meta and ResStock meta full data)

Methods:

Microsoft data taken directly, a little clean up by nothing major
ComStock and ResStock, grouping by state name, then filtering for non-electric heating, water, and cooking range sources. Pandas used to calculate percentages of buildings, nothing fancy!

Other experiments:

OSM was attempted to see if the building coverage was good, but unfortunately it was not. See scripts/osm_building_count.py for posterity
See also sanity_check_ms_data for methodology on a quick validation on MS data.

Testing Instructions
NA Script/data research based.

@nofurtherinformation awesome - thanks for this!
@swatland610 as primary implementer for the data import process, would you be able to review this PR?

@nofurtherinformation this all looks awesome! Thanks for putting this together!
There two things we need to consider here is hooking this up with the Makefile.
Connecting to the Makefile
So I think all we need is:

a function from one of your file that takes the csv file as an input
a function that returns the csv as a json object
another file in scripts/ that houses pulls in both of those functions that prints the formatted json object.

Once we have that, we can connect it to the Makefile! Here's the one I did for the initial emissions data as an example. get_emissions
Open to suggestions on this too! Just let me know!
State Name Formatting
This one it looks like you may have done but perhaps for another purpose (sorry if I'm missing that!).
For the returned json object, please make sure your state names are in slug case (illinois or new_hampshire) It just matches to emissions json structure.
Happy to connect if you have any questions!

Sounds good @swatland610! I just pushed a couple updates that work locally, seem ok. Take a look and let me know? I added an argument for the json builder util for non-timeseries data, too.

@swatland610  - re-upping this! As it is now, and this PR outputs a separate buildings.json file with the same structure as emissions. Let me know if this works on your end and anything needed!
We chatted briefly Tuesday about pros and cons on separate vs combined data files for different topics, landing generally on combined for ease of use.

Sorry for the delay! I think this works now! Thank you @nofurtherinformation !

@derekeder would you be able to gut check me on the Makefile edit. I believe that should work, but I see the deploys failing above. I wonder if that's it.

@swatland610 - were you able to run the site with this locally? It might be worth merging with main in case that helps but the errors seem to indicate the structure of the main data being used changed in a way that broke pages.

@swatland610 I ran the make command locally and the files were generated:
cd data
docker-compose up --build

When working on the data, you will need to run this command and commit the updated/added files to the repository for Gatsby to load when it deploys.
However, now that we have 2 files in the /data/final/ directory, and those two files have the same root level keys (alabama, alaska, etc), gatsby-transformer-json is getting confused and overriding the data.
I think the best way to resolve this, while keeping the files separate, would be to add a key wrapping the json object in each file to avoid the data from colliding when gatsby-transformer-json merges them. Something like:
buildings.json
{
  "buildings": {
    "minnesota": {
      "buildings": 2914016
    },
    "mississippi": {
      "buildings": 1507496
    },
   ...
  }
}

emissions.json
{
  "emissions": {
    "colorado": [
      {
        "year": 1990,
        "dirty_power": 31.4,
        "buildings": 9.4,
        "transportation": 20.1,
        "dumps_farms_industrial_other": 26.5
      },
      {
        "year": 1991,
        "dirty_power": 30.7,
        "buildings": 10.2,
        "transportation": 20.3,
        "dumps_farms_industrial_other": 27.1
      },
      ...

Once this is done, we will also need to update the queries we're using in src/pages/index.js and src/components/state-details.js. The build is failing right now because these queries are not currently valid.

Thanks for this @derekeder @swatland610! This should be building now, but cue up Thanos what did it cost? meme.
I added outer object tagging in utils.py to add additional outer tags on the json outputs as described ({emissions: {...}}, {buildings: {...}}). The big issue I'm seeing is that the graphql tree now has two edges in the array, and the first is null, the second has the data (presumably, the first edge is buildings).

One workaround here is to do a quick data filter operation, but this seems not ideal for a variety of reasons, but there is certainly a better graphql way to do this?
const filteredData = data.allFinalJson.edges.find(edge => !!edge.node.emissions).node.emissions
I wasn't able to find graphql filter in object falsy-ness, only individual props:

My instinct here might be that combining json in the data workflow might be a better solution longer term, so we could just access finalJson.emissions[${stateName}]

@nofurtherinformation ok I think I found a better way to address having separate files. According to this stack overflow thread, its the folders that count when gatsby-transformer-json processes a json file. If we put each json file in its own separate folder, they'll be loaded in as separate objects in graphQL:
Folder tree structure:

graphql browser:

If we go this route, we can keep our files in their original format and remove the outer tag. Wanna give that a shot?

Ok @derekeder  @swatland610! A couple merge conflicts here to reconcile with main, but this workflow should be good.
I updated the makefile to remove old folders and make new ones needed, and utils to fit the folder data structure. I also updated the queries, and we'll need to do a bit more for the new components!

Hi folks -- just merged main into this and updated the relevant queries and utilities.
Please note here I swapped out the hard coded calls for data.finalJson[innerProperty] to expect the state data object, since we'll be using multiple different data objects (eg. data.emissionsJson, data.buildingsJson).
I think this is good to go, let me know any additional changes needed, and looking forward to pulling in the rest of the data!


className did not match SSR with emotion - caused by styled factory

Duplicates

[X] I have searched the existing issues

Latest version

[X] I have tested the latest version

Current behavior 😯
Warning: Prop `className` did not match. Server: "MuiTypography-root MuiTypography-h6 css-16x3c2x-MuiTypography-root" Client: "MuiTypography-root MuiTypography-h6 css-16x3c2x-MuiTypography-root e1de0imv0"
    at h6

As you can see, it is adding a single class name to the client that does not exist on the server: e1de0imv0.  When I did a source search for that, the classname is generated by a call to styled.
https://github.com/mui/material-ui/blob/02939b2d7f54e7bd7a4b2f31595fb97709465b5a/packages/mui-styled-engine/src/index.js#L4

This is happening if I ever use a mui component that uses styled.  In this case, it throws the error when I call Typography.  However, for whatever reason it appears that this is not apply this last css string until we get to the client.  I am at a loss.  I have read all the docs, gone through source code, and nothing.
Expected behavior 🤔
SSR should work with @mui/material
Steps to reproduce 🕹
Steps:
I followed and have the EXACT example from the nextjs-with-typescript in terms of my MyDocument.getInitialProps and confirmed that does the job (as you can see since the rest of the classes all get populated properly).  Not sure exactly how to reproduce this one or where to even look.
Context 🔦
Errors on every page load, ssr doesnt have that last class although I haven't found that it is even used anywhere from what I can see it is worthless -- but i am guessing its added for targeting or something?
Your environment 🌎

`npx @mui/envinfo`
  System:
    OS: macOS 12.4
  Binaries:
    Node: 16.14.0 - ~/.nvm/versions/node/v16.14.0/bin/node
    Yarn: 1.22.18 - ~/.yarn/bin/yarn
    npm: 8.3.1 - ~/.nvm/versions/node/v16.14.0/bin/npm
  Browsers:
    Chrome: 100.0.4896.88 <-- used chrome
    Edge: Not Found
    Firefox: 99.0
    Safari: 15.5
  npmPackages:
    @emotion/react: ^11.7.1 => 11.7.1
    @emotion/styled: ^10.0.27 => 10.0.27
    @types/react: ^16.9.52 => 16.9.52
    react: ^16.14.0 => 16.14.0
    react-dom: ^16.14.0 => 16.14.0
    typescript: ^4.6.3 => 4.6.3

I can confirm that the classname being added and causing the error is the class generated by https://github.com/mui/material-ui/blob/02939b2d7f54e7bd7a4b2f31595fb97709465b5a/packages/mui-material/src/Typography/Typography.js#L28
I ran toString() against it and it came back as the class name causing the problem here.  So no clue why that is happening but at least some info is presenting itself.

It doesn't look like this bug report has enough info for one of us to reproduce it.
Please provide a CodeSandbox (https://material-ui.com/r/issue-template-latest), a link to a repository on GitHub, or provide a minimal code example that reproduces the problem.
Here are some tips for providing a minimal example: https://stackoverflow.com/help/mcve


On beta site, languages other than English have two Maths entries on the toolbox

Go to https://pxt.microbit.org/beta and choose French from the Language menu.

@martinwork nice catch

Also on-start block is missing.


Recommended approach for handling a vertex with many (millions) of edges

We have a case where we have many (potentially millions) of edges attached to a single graph vertex.  An example use case is that we want to be able to tag records.  To do that, we create a vertex representing the tag and link it to all records we want to tag.  The idea is that we could then traverse from the tag to get the entities with that tag.  The underlying problem is that once there are many edges associated with a vertex, operations on the vertex are very slow (even just simply trying to read a single property from the tag vertex).  Is there a recommended way to model this type of relationship in the graph db?

Hm, it means that you are using presentation of edges embedded in document. Which kind of database do you use ?

@nmcintyre2014 are you running in distributed mode?

We're currently not running in distributed mode, but we plan to if we need to scale beyond one node.  We're using 2.0.10 in vanilla graph mode.

I would create an index on the edge.
Reading about RidBags, it seems like this kind of problems have been encountered before. Maybe instead of transitioning from one RidBag structure to another based on a threshold size, an index could automatically be used.


Improve folders & files tree discovery

We should probably use the following two NPM libraries to improve our current way of reading files, without having to refactor too much code:
https://www.npmjs.com/package/glob - to discover the entire file tree for **/*.yaml files
https://www.npmjs.com/package/ignore - to ignore all files from .gitignore
https://www.npmjs.com/package/graceful-fs - Queues up open and readdir calls, and retries them once something closes if there is an EMFILE error from too many file descriptors.
https://www.npmjs.com/package/@parcel/watcher - faster watcher for files

Adding the performance label to this


Unclear teaching description and ununified usage of words ECTS, CP

The content of the teaching events sometimes does not have any descriptive text or misses it completely (s. top right box).
Also, the terms "ECTS" and "CP" are used in an alternating manner (s. top left box, bottom right box). For clarity reasons, it is favorable to only use one of them, i.e. EITHER "ECTS" OR "CP".
Besides, "Course" as a term alone is not as descriptive as e.g. "Practical Course" – which it is (s. bottom right box).

https://csbiology.github.io/#teaching

[ ] Missing descriptions added
[ ] Usage of either "ECTS" or "CP" but not both of them
[x] Replacing sole "Course" with "Practical Course"

Since the information is parsed straight out of OLAT, it makes little sense to implement a gazillion String.replaces, just because the university is not able to provide the information consistently.
*sigh*


Add UniquePtr::to_shared and SharedPtr::from_unmanaged

This adds a way to create SharedPtr's from C++ pointers created with new (or via unique_ptrs).
I'm a new contributor here, let me know if I did this in a reasonable way, and I'm happy to put some effort in if this is something that's interesting.

I fixed that error and added an additional test to cover the situation, but it needs approval to run again. Just pinging just in case you don't get commit notifications.

Freshly rebased onto master :)

Hmm I think it would be reasonable also to implement SharedPtr<T> from UniquePtr<T>, happy to add that if you agree.


Unable to use private repository harbor in webhook

Current version 1.6.30.
I try to write inside the body of the url that sends the webhook.
{
    "push_data": {
        "tag": "latest"
    },
    "repository": {
        "repo_name": "web_package_images/marketing",
        "repo_url": "https://xxx.com"
    }
}

but, update unsuccessful.
Failed to allocate instance [container:1i494]: Bad instance [container:1i494] in state [error]: Error response from daemon: pull access denied for web_package_images/marketing, repository does not exist or may require 'docker login': denied: requested access to the resource is denie

Then I try to use docker login on that server in rancher. also reported the same error.
It seems that the sent urls all go to the docker hub instead of the private warehouse harbar.

Rancher version 1.x is no longer supported.


Make Manifest

Description

This helper will create a new Manifest object, add it by reference to the items property, and return it
Function name / Parameters / Return

Function name: make_manifest()
Parameters: None
Returns: A Manifest object
Objects

[ ] Base Object (all Prezi3 Classes)
[x] Collection
[ ] Manifest
[ ] Canvas
[ ] Range
[ ] AnnotationCollection
[ ] AnnotationPage
[ ] Annotation
[ ] Content Resource - Image
[ ] Content Resource - Video
[ ] Content Resource - Text
[ ] Service
[ ] Provider
[ ] Other - please list below

Considerations

Manifests must not be embedded inline in a Collection (https://iiif.io/api/presentation/3.0/#51-collection) and therefore this helper must also create a ManifestReference and add that to the items property of the Collection object calling this helper.

Should add optional parameters of ID but also use the id generating code.
Label would need to be required as a parameter.

Potentially only a name change for this method:
https://github.com/iiif-prezi/iiif-prezi3/blob/main/iiif_prezi3/helpers/collection_add_methods.py


Site Picker doesn't return sites with title with 2 or less characters

Category

[x] Enhancement
[X] Bug
[ ] Question

Version
Please specify what version of the library you are using: [2.7.0]
Expected / Desired Behavior / Question
There are many sites with just 2 characters in the title, like TI, HR etc. I would like to find these sites typing one or two characters (maybe with component property that allow me to set minimun characters to type before run search).
Observed Behavior
These sites are missing in the result set when I type full title. The only way to retrive is typing multiple spaces (very strange, and not obvious for users)
Steps to Reproduce

Add PropertyFieldSitePicker to the Property Pane
Type search text with 2 or 1 characters
Sites with 2 or 1 characters on title cannot be found

Any updates about that issue?

Hello @joaoworkhub - that is a good suggestion.
I will look into it to provide some solution.
BTW, we encourage everyone to make PRs. So, if you want to implement that feature - just let us know.

Looks like it explicitly checks if the search term is greater than 2 characters .

The search query used to search for sites also works with 2 or less characters (tested it). I can adjust this so the component searches when at least 1 character is entered.


Documenting Plonk Merkle Tree

Updated with a usage example.

Codecov Report

Merging #201 (a7d6385) into master (3fd8035) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #201   +/-   ##
=======================================
  Coverage   26.14%   26.14%
=======================================
  Files          36       36
  Lines        2264     2264
=======================================
  Hits          592      592
  Misses       1672     1672

Impacted Files
Coverage Δ

arkworks-plonk-gadgets/src/merkle_tree.rs
0.00% <ø> (ø)

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 3fd8035...a7d6385. Read the comment docs.


Mock for $xhr

Let's provide $xhr mock that can accepts xhr expectations (instead of expecting on $browser level).
We should allow to respond custom headers as well.

Closing, as $httpBackend provides all these features.


tagged releases?

Helpful for downstream users and packagers to at least know:

What commit (source tarball) corresponds to version X
What is suitable for general use vs. the current development work.

Not a big deal, but thought I'd mention it in case it was just an oversight instead of intentional.
Thanks!

Thanks, no it is an oversight (actually just a newbie mistake). I should have a new release in the next
few days, so I will see if I can get it right. Any checklist anywhere a newbie can follow?

Thar she blows. Hope she doesn't blow too hard.  Compiles the Linux kernel.


More of a question than an issue

I absolutely love Refills and the whole Bourbon family, and I want to contribute a new component, but I am a bit confused how develop a new one using this stack.
I'm completely comfortable doing the SASS/HTML scaling and development, I'm just confused how to set up/view my development changes!
Any help is appreciated, thanks! :)

@iryanclarke any luck with the video hero unit, I was pondering on doing one as well...

I'm nearly done it, just polishing it up! :P Still need to commit it into my forked repo :V

@iryanclarke awesome, please let me know when it's available, I would love to have a look...

I'll let you know, school is bearing down hard so I'll do it when I can!


SVDF layer implementation compatible with TFLite’s SVDF operator

Although TFLite has a built-in SVDF operator (as listed here, and whose implementation can be seen here), currently, Tensorflow doesn't have a keras layer implementation for the SVDF operation.
I was wondering what would be the sequence of operations (using Tensorflow's Python library) needed so that they could be fused and recognized as an SVDF operator after converting to TFLite.

This issue originally reported by @VictorDominguite has been moved to this dedicated repository for ai-edge-torch to enhance issue tracking and prioritization. To ensure continuity, we have created this new issue on your behalf.
We appreciate your understanding and look forward to your continued involvement.

This issue originally reported by @VictorDominguite has been moved to this dedicated repository for ai-edge-torch to enhance issue tracking and prioritization. To ensure continuity, we have created this new issue on your behalf.
We appreciate your understanding and look forward to your continued involvement.


Fix case statement for Github

Other references here use just github and not github-organization.  Believe this fixes #1075

i'm hesitant to accept this as we have already swapped this once in the past via #928 due to github-organization causing panics. can you provide a test case that asserts this should be correct?

I personally have no case for such a change -- was just working from the open issue looking to help out.  If they have not provided one by now, we can probably close out this PR.

@gstotts & @jacobbednarz this is still happening

@lattwood i believe @jacobbednarz was looking for a test case that would assert this be correct.  I opened the PR in regards to an open issue but, I did not have such a test case myself.


Automatically cleanup messages for past events

Hmm... I need to add support for a different kind of history storage (S3?) for running the bot as a serverless function...


[feature request] Connect Assistant

I think this would be great if you could connect this to an openai assistant instead of just chatgpt.
I'm pretty new so if anyone can help me do this I would appreciate the help.

The Assistant API is still in its beta, so I don't have immediate plans to utilize it in this project. Once the feature is GAed, I may consider doing something with it.


There is a memory leak, How to really release graphics resources

Load the following model：
machine_tool.glb(80.2MB)
Using ar-model-viewer-java in the project，replace model.
Add the model to the arfragment page and finish，Don't exit app. The graphic resources are not released.

Reproduction steps：

Create a new entry class to jump

Run and Open the MainActivity, add the model in screen, return to  RouterActivity
Check  Profile tool -> Memory ->  Graphics

OutOfMemory will appear if repeated multiple times

I'll check this in the next view days. What I know is, that the ArSession is not released immediately and therefore it could be, that the resources are not released. You can try to release the ArSession by yourself. You can access the Session through ArSceneView and call destroySession. To be fair it is not really a SceneForm issue but a lifecycle issue. The function onDestroy can be called couple of seconds later after the view is dismissed. That's it for now, I look further into this topic the next days!

I'll check this in the next view days. What I know is, that the ArSession is not released immediately and therefore it could be, that the resources are not released. You can try to release the ArSession by yourself. You can access the Session through ArSceneView and call destroySession. To be fair it is not really a SceneForm issue but a lifecycle issue. The function onDestroy can be called couple of seconds later after the view is dismissed. That's it for now, I look further into this topic the next days!

Thank you very much for your recovery!  I'm looking for a way to really release graphics resources.
Call ArSceneView->destroySession(), It doesn't work, destrorySession() has been indirectly called in BaseArFragment:
BaseArFragment->onDestroy()->destroy()->arSceneView.destroy()->ArSceneView.destroySession()

I have the same issue. I'm using .glb object format and they are not being released at all. I found SceneView.destroyAllResources(); as one command that originally gets into the renderer.java class of the sceneform library. In the code bellow you see that two functions are called:  ResourceManager.getInstance().destroyAllResources(); and EngineInstance.destroyEngine();
The first one tries to access the registry for rendering features which is referred as "resourceHolders" and release all the resources. The second one shuts down the engine and terminates the app. I altered the code to just call the first function without destroying the engine (commented this line) to see if the graphical mem is released but unfortunately nothing changed. The graphical mem is released only when we close the app. Is this problem related to the object type, i.e., because I'm using .glb file?
public static void destroyAllResources() {
ResourceManager.getInstance().destroyAllResources();

EngineInstance.destroyEngine();
}
public void destroyAllResources() {
for (ResourceHolder resourceHolder : resourceHolders) {
resourceHolder.destroyAllResources();}

I have the same issue. I'm using .glb object format and they are not being released at all. I found SceneView.destroyAllResources(); as one command that originally gets into the renderer.java class of the sceneform library. In the code bellow you see that two functions are called: ResourceManager.getInstance().destroyAllResources(); and EngineInstance.destroyEngine(); The first one tries to access the registry for rendering features which is referred as "resourceHolders" and release all the resources. The second one shuts down the engine and terminates the app. I altered the code to just call the first function without destroying the engine (commented this line) to see if the graphical mem is released but unfortunately nothing changed. The graphical mem is released only when we close the app. Is this problem related to the object type, i.e., because I'm using .glb file?
public static void destroyAllResources() {
ResourceManager.getInstance().destroyAllResources();

EngineInstance.destroyEngine();
}
public void destroyAllResources() { for (ResourceHolder resourceHolder : resourceHolders) { resourceHolder.destroyAllResources();}

EngineInstance.destroyEngine(); It is useful

I think it may be because EngineInstance statically holds the filament.Enginereference
But there is a problem,How to reinitialize,I found EngineInstance.getEngine(); , But it didn't work,

@brady2037  How to reinitialize, are you done?

anyone? :(

Is any solution here ?

You should move to SceneView


Update ballerina home env variable

Purpose
Update KeyStore path by setting the $ ballerina home value into an environmental variable.
Related to: https://github.com/ballerina-platform/ballerina-lang/issues/20436

Closing this PR since all the temporary fixes are fixed with 1.1.0-RC1.


OpenTracing AutoConfiguration crashes with a non-brave OT implementation

OpenTracingAutoConfiguration creates a Brave Tracer if OpenTracing classes are on the classpath.
https://github.com/spring-cloud/spring-cloud-sleuth/blob/master/spring-cloud-sleuth-core/src/main/java/org/springframework/cloud/sleuth/instrument/opentracing/OpentracingAutoConfiguration.java#L40-L41
This crashes the app if a non-brave OT implementation is used. To Repro:

Launch SpringBoot2 with a Datadog JavaAgent with no Brave classes present: https://docs.datadoghq.com/tracing/setup/java/#installation-and-getting-started
App will crash on startup: java.lang.ClassNotFoundException: brave.opentracing.BraveTracer

As a workaround, the OT configuration can be disabled.
https://github.com/spring-cloud/spring-cloud-sleuth/blob/master/spring-cloud-sleuth-core/src/main/java/org/springframework/cloud/sleuth/instrument/opentracing/OpentracingAutoConfiguration.java#L39
Seems like a simple fix would be to add a check for a brave implementation to the OT autoconfiguration.
@ConditionalOnClass(Tracer.class, BraveTracer.clss)

Happy to open a PR if that helps.

This won't actually work the way you are expecting it to. If you disable the BraveTracer and create an instance of some OtherTracer then you will end up with 2 different tracing systems; sleuth (built on brave) and your OT tracer. The OT support in Sleuth is explicitly to allow using the use of OT instrumentation not to swap the tracer since sleuth is a tracer itself historically.

This bug that @realark  submited was when Spring Cloud (specifically spring-cloud-zuul) Finchley SR1 and Spring Boot 2.0.4 were used.  We would love to exclude all of Spring Sleuth and not even bring in the libraries but compilation will not work then.  The only way this works with datadog APM is to disable sleuth (spring.sleuth.enabled=false)

I'm guessing this is because datadog APM is affecting the classpath of the VM by sharing a type that might be present in the application classpath. This will also be a problem with opentracing 0.32 is release, as well 0.33 etc.
I would recommend asking DD APM to hide their types used internally so that they won't conflict in situations such as this or other unrelated opentracing upgrades. I suspect most APM agents will not have this problem because they don't leak usage of opentracing apis into user space. In fact I know of no other apm which uses OT internally. For example, I'm quite sure instana agent supports multiple opentracing api versions.
TL;DR; sooner or later datadog will sort this out.. mind asking them to start now? The isolation will solve other problems that a "spot fix" here won't

other thing.. can you confirm that setting "spring.sleuth.opentracing.enabled" to false isn't sufficient?

actually disregard my question as it looks like from the original description, you only need to set "spring.sleuth.opentracing.enabled=false" not disable all of sleuth. I was distracted by the assumption you have to disable sleuth entirely which seems a bit off.
For those wondering why we shouldn't do @ConditionalOnClass(Tracer.class, BraveTracer.class)... Sleuth itself, its support of opentracing is indeed dependent on BraveTracer. The classnotfoundexception explains what is wrong, which is that the implementation is not available.
If we step away from whether or not DD agent should be leaking opentracing classes, we can get closer to what we could or should do here. Let's assume this wasn't about a redundant tracing setup.. that's the distraction.
Let's assume someone put in some code of their own, and wondered why it wasn't working? With the current setup, it would crash, which is probably a good thing as it is relatively easy to find the missing dependency as per our guide.
Let's say instead, we just started up anyway. For example, we did the @ConditionalOnClass(Tracer.class, BraveTracer.class) bit.. that silently failed and then didn't configure the tracer etc. There's no eager crash, people will wonder why their data is missing and probably open issues or start a wandering discussion on support. This falls onto volunteers to clean up.
That's why I'm personally ok with how it is now.. in the rare case you have an agent which activates the classpath and is redundantly setup with sleuth, and that results in the crash.. disable via "spring.sleuth.opentracing.enabled=false" (or rewrite an exception message to say something more clearly.. like "did you mean to include XXX dependency?").

@adriancole Thanks for the explanation. That makes sense. Usually better to crash early than behave strangely later.
In terms of putting OT on the classpath, that's an early design decision of the dd tracer. We're working to decouple the dd tracer from OT so we don't have to inject OT classes for javaagent users.

I am facing this same issue and Adrian's suggested flag spring.sleuth.opentracing.enabled=false helped. In my case it appeared by changing Spring Boot and Sleuth release train versions.

I was running my app on Spring Boot 2.0.3 and Sleuth Finchley.RC1. The app was working fine.
I updated the versions to Spring Boot 2.0.4 and Sleuth Finchley.SR1. My app broke for a different issue - this version of Sleuth and Couchbase Java SDK (2.5.9+`) do not like each other.
Then I reverted my app to Sleuth Finchley.RC1, but retained Spring Boot version as 2.0.4. This failed with the issue that discussed in this ticket.

In all above scenarios, I was using Reactor Core and Reactor Extra 3.2.0.RELEASE.


Add progress chart

Progress chart (#200), built with Chart.js, now appears on the stats page, using backend data gathering in chart_utils.py. Features include:

Meta solves are highlighted as blue circles that can be moused over for their names/solve times.
Times of ticks along the x-axis are alwas given in a format such as Fr 4:00 pm, in the local timezone. (Not sure if I should standardize this as ET or hunt time, but I don't think that would be better or necessary.) The minute of every tick time will be a multiple of 15.
Ticks are limited in count so as to not make the chart too messy.
Grid dimensions are based on the size of the page window.
Chart extends from the start time to either the current time or the end time (if specified), whichever is earlier.
Chart creation/display is disabled if the start time isn't given or if the current time is before the start time.

(Remaining todo: make the chart and stats page as a whole update every 10 seconds or so. This might not be super necessary and would be the last remaining major step, but it would be nice.)

Very nice! Added a few nitty comments.
A few questions:

Does this chart auto-refresh? It would be cool to have a cardinality stream that shows this chart over time (auto-refreshing every minute or so)
What do you think of charting unlocks on top of solves? (with a different color)

Not yet, but Chart.js has easily accessible ways to update a chart that's already on the page (with smooth animations, too). The implementation might be tricky and I'd probably be doing this with the stats page update, so I'm thinking of including that feature in a future issue/PR if I have enough time to implement it. (At this rate, hopefully!)
Totally doable. Aesthetically, though, the unlocks might greatly outpace our solves, which might squish the solves line down greatly (and make it look pretty flat), so I'm not sure if it would look good.

I'm also thinking of recoloring the solve line green instead of making the color a reference, just to make it in line with our existing color scheme. (If we were to add the unlock line, that would be gray.)

Very nice! Added a few nitty comments.
A few questions:

Does this chart auto-refresh? It would be cool to have a cardinality stream that shows this chart over time (auto-refreshing every minute or so)
What do you think of charting unlocks on top of solves? (with a different color)

Not yet, but Chart.js has easily accessible ways to update a chart that's already on the page (with smooth animations, too). The implementation might be tricky and I'd probably be doing this with the stats page update, so I'm thinking of including that feature in a future issue/PR if I have enough time to implement it. (At this rate, hopefully!)
Totally doable. Aesthetically, though, the unlocks might greatly outpace our solves, which might squish the solves line down greatly (and make it look pretty flat), so I'm not sure if it would look good.

I'm also thinking of recoloring the solve line green instead of making the color a reference, just to make it in line with our existing color scheme. (If we were to add the unlock line, that would be gray.)

Very nice! Added a few nitty comments.
A few questions:

Does this chart auto-refresh? It would be cool to have a cardinality stream that shows this chart over time (auto-refreshing every minute or so)
What do you think of charting unlocks on top of solves? (with a different color)

Not yet, but Chart.js has easily accessible ways to update a chart that's already on the page (with smooth animations, too). The implementation might be tricky and I'd probably be doing this with the stats page update, so I'm thinking of including that feature in a future issue/PR if I have enough time to implement it. (At this rate, hopefully!)
Totally doable. Aesthetically, though, the unlocks might greatly outpace our solves, which might squish the solves line down greatly (and make it look pretty flat), so I'm not sure if it would look good.

I'm also thinking of recoloring the solve line green instead of making the color a reference, just to make it in line with our existing color scheme. (If we were to add the unlock line, that would be gray.)

Awesome.
I think for the purposes of Mystery Hunt, the unlocks shouldn't significantly outpace solves.

Very nice! Added a few nitty comments.
A few questions:

Does this chart auto-refresh? It would be cool to have a cardinality stream that shows this chart over time (auto-refreshing every minute or so)
What do you think of charting unlocks on top of solves? (with a different color)

Not yet, but Chart.js has easily accessible ways to update a chart that's already on the page (with smooth animations, too). The implementation might be tricky and I'd probably be doing this with the stats page update, so I'm thinking of including that feature in a future issue/PR if I have enough time to implement it. (At this rate, hopefully!)
Totally doable. Aesthetically, though, the unlocks might greatly outpace our solves, which might squish the solves line down greatly (and make it look pretty flat), so I'm not sure if it would look good.

I'm also thinking of recoloring the solve line green instead of making the color a reference, just to make it in line with our existing color scheme. (If we were to add the unlock line, that would be gray.)

Awesome.
I think for the purposes of Mystery Hunt, the unlocks shouldn't significantly outpace solves.

The unlocked line has been successfully added!

All tooltips are disabled for the unlocks line, and a few color changes (to lines and fill areas) have been made in line with the existing color scheme.

The unlocked line has been successfully added!

All tooltips are disabled for the unlocks line, and a few color changes (to lines and fill areas) have been made in line with the existing color scheme.


tour: automatic save of the last accessed tour page

I would like to suggest to add a new feature which is the automatic save of the last accessed tour page so the user could find himself again in the page that he left. I don't think newcomers will read all the everything in one shot.
Thx

CC @adg @campoy


Leaflet/SmPC delete not possible.

After successfully creating a product with SmPC, when we try to remove the SmPC - the Enterprise Wallet doesn't allow the removal.
Steps to recreate the issue:

Create a product

Add Leaflet and SmPC to the product

Create a batch for the product with serial number

Use the mobile to Scan the batch data matrix - you will view the SmpC and Leaflet uploaded.
*Works as expected *

Come back to product.

Remove/ Delete the SmPC inherited on the product

Update the product

Switch to the mobile app and scan this batch

Even though the SmPC was removed in the step 6 , it is still displayed.

Re open the product in edit mode -
you will observe that the SmPC that was deleted is still there.

Tested and works fine

Hi @adiganga2002
We have a new issue in the leaflet area
Can you do the following test please:
•	Create a product (load a leaflet) and create a Batch.
•	Now try to update the ePI.
--it is observed that the ePI does not get updated - instead - the ePI is deleted but new ePI that was selected is not uploaded.
Could you please fix this ASAP
@adiganga2002 @skutner  @asaccool


A few remarks (standard headers in cmp.h + size_t return for writer)

Hello there,
First of all, thank you very much for this very nice small library: it's awesome! :+1:
Just a few remarks. I've had some very minor issue since stdint.h and stddef.h were not included inside cmp.h is there a reason not to include them directly? Also, from what I see in cmp.c, you are using the writer function as if it returned bool and I was wondering why ask for a size_t here then?
Thank you again for your great library! :)
Cheers.

Thanks for the kind words :)
Yeah, I used to be opinionated about this because of some previous bad experiences, but now I think it's better for header files to be self-sufficient (i.e. @fperrad's PR #53). I think I'll go ahead and merge that.

Gonna close as I merged #53; feel free to check in again if that doesn't solve your issue.


Gift Creation

[ ] Asset missing on gift creation modal

[ ] Some external link attached to gift link

[ ] Shared gift not able to open

Not reproducible


Put Types files in @types folder

We should organize our types in a folder preferably @types

Nice idea. I don't think it makes sense since some users already expect a certain location for the types.


Updates to extension: UI, caching embeddings, temp logo

Should be simple to build and test out.
Here's a 128x128 icon I made as an alternative to the default Xenova one. I'm not entirely happy with it (too pixelated at this resolution) but it's in for now.

Nice, that really catches the spirit! We can always reiterate on it, but I really like it for now :)
Very curious about testing the new features for the extension!

I just wanted to test the extension but I'm bumping into some issues again. Seems like one icon/animation file AnimatedSquare is missing:
ERROR in ./src/popup/popup.vue?vue&type=script&lang=js (./node_modules/vue-loader/dist/index.js??ruleSet[1].rules[3].use[0]!./src/popup/popup.vue?vue&type=script&lang=js) 5:0-50
Module not found: Error: Can't resolve './AnimatedSquare.vue' in '/mnt/c/Users/dome/Downloads/SemanticFinder/extension/src/popup'
resolve './AnimatedSquare.vue' in '/mnt/c/Users/dome/Downloads/SemanticFinder/extension/src/popup'
  using description file: /mnt/c/Users/dome/Downloads/SemanticFinder/extension/package.json (relative path: ./src/popup)
    Field 'browser' doesn't contain a valid alias configuration
    using description file: /mnt/c/Users/dome/Downloads/SemanticFinder/extension/package.json (relative path: ./src/popup/AnimatedSquare.vue)
      no extension
        Field 'browser' doesn't contain a valid alias configuration
        /mnt/c/Users/dome/Downloads/SemanticFinder/extension/src/popup/AnimatedSquare.vue doesn't exist
      .js
        Field 'browser' doesn't contain a valid alias configuration
        /mnt/c/Users/dome/Downloads/SemanticFinder/extension/src/popup/AnimatedSquare.vue.js doesn't exist
      .json
        Field 'browser' doesn't contain a valid alias configuration
        /mnt/c/Users/dome/Downloads/SemanticFinder/extension/src/popup/AnimatedSquare.vue.json doesn't exist
      .wasm
        Field 'browser' doesn't contain a valid alias configuration
        /mnt/c/Users/dome/Downloads/SemanticFinder/extension/src/popup/AnimatedSquare.vue.wasm doesn't exist
      as directory
        /mnt/c/Users/dome/Downloads/SemanticFinder/extension/src/popup/AnimatedSquare.vue doesn't exist
 @ ./src/popup/popup.vue?vue&type=script&lang=js 1:0-136 1:0-136 1:137-262 1:137-262
 @ ./src/popup/popup.vue 2:0-56 3:0-51 3:0-51 8:49-55
 @ ./src/popup/popup.js 2:0-32 5:10-15

webpack 5.88.2 compiled with 1 error in 2620 ms

Do you know how to fix this?
In case, do you think the extension would we ready for the public or is it too early yet? I was thinking that it would be quite a nice occasion to get more contributors on board and I'm quite eager to advertise SemanticFinder as soon as you agree too. :)

See #37 for the fix.
Sure, let's release it to the public. I want to make the extension better but I haven't got much progress on it in the last couple weeks and it could certainly use a fresh set of eyes. Let me know when you go ahead and advertise. I'll try to boost the signal.


[WIP] Support timing function for animation. Fix #12

Timing function enum added

Add it to Animatable objects and AnimationConfiguration
Use this enum when possible instead of CAMediaTimingFunction

TODO

[x] add more type in enum (see extension CAMediaTimingFunction)
[x] add more type in enum  see CAMediaTimingFunction( contructors in code
[x] add custom type with control points in enum ?
[x] manage spring animation or not, animation default value
[ ] allow to custom timing functions in ActivityIndicator?
[ ] Add an example
[ ] Update CHANGELOG
[ ] Documentation

          1 Error

:no_entry_sign:
Any changes to library code need a summary in the Changelog.

          3 Warnings

:warning:
PR is classed as Work in Progress

:warning:
Consider adding supporting documentation to this change. Documentation can be found in the docs directory.

:warning:
Consider adding / updating the demo ap.

  Generated by :no_entry_sign: Danger

added more type using sourcery to generate switch case from static variables
and this stencil template
{% for type in types.all %}

{% for variable in type.variables %}
case {{ variable.name }}
{% endfor %}

{% for variable in type.variables %}
case .{{ variable.name }}:
return .{{ variable.name }}
{% endfor %}

{% for variable in type.variables %}
case "{{ variable.name|lowercase }}":
self = .{{ variable.name }}
{% endfor %}

{% endfor %}

default value managed using operator ?? to replace nonewith a default one at the right of the operator
I think I made a fix in ActivityIndicatorAnimationOrbit ring2OpacityAnimation
same could be done in ring1OpacityAnimation

a new one with
two translation animations
two or one timing function type menu selectors
this allow to compare two timing functions (could compare always with linear if only one menu selector)

Nice Idea !
It's a great first step.
The inconvenient of this solution is that we can see the effect of the timing function only on Translations.

Maybe I will do the two case
1 - in Animations add a button to display a new view controller to select the timing function

I am working on it today
2 - add a new entry with two elements, one animation (maybe later an animation type selector in a new view controller)
a picker for timing function

added in "Animations" a view to select the Timing Function as illustrated in the previous message
Need some help for the Documentations, where to document it?

@phimage Sorry for the delay, do you still need help for the documentation?


Fix firewalld role vars

Changes:
Fix a number of incorrectly named firewalld role variables.

I'm going to merge as I'd like to make a release


Add label-has-associated-control test for

Coverage remained the same at 99.484% when pulling 232d2350e6a38d839d4b69d920217bb454580d07 on simPod:add-test into 573d39e2e50c4025275a0ce2a4d68aca1e4466b1 on evcohen:master.

Coverage remained the same at 99.484% when pulling 232d2350e6a38d839d4b69d920217bb454580d07 on simPod:add-test into 573d39e2e50c4025275a0ce2a4d68aca1e4466b1 on evcohen:master.


Allow :headers option to send custom headers

Hello. Thank you for the useful gem!
Occasionally, I have to pass some custom headers such as X-Api-Token: .... or Authorization: Bearer .... to GraphQL servers. So I'd like to add the :headers option.

This looks great! Thank you!
The build errors are my fault, I've fixed them on master.


fix issue with persistence when recovering state

After loading a snapshot, the recover state method would then attempt to replay events from the index that the snapshot was taken at (i.e. 10). However, it would include the last event the snapshot was taken at (i.e. 10!) which isn't correct - it should replay events from the next index (i.e. 11)
example
what happened: 1,2,3,4,Snapshot,5,6
recover state: snapshot,4,5,6 < THE 4 HERE IS WRONG
should be:
what happened: 1,2,3,4,Snapshot,5,6
recover state: snapshot,5,6

@AsynkronIT/developers am I ok to merge this in?

Approved!


Installation is failing

Hi 1modm, Can you please check on this? Installation is failing while installing dependencies with Exit Code 1.

Usually happens because network or connectivity issues


Kareem branch

Finished tree intent functions and completed one comprehensive test for tree intents and getting two lists of intents

Just resolved merge conflicts between Kareem branch and main- Clean architecture changes in main preserved, and combined with business logic updates in Kareem branch.
It would be a good idea to review the changes I made in that commit (a983091) to make sure the clean architecture is all good. The business logic tests are working just fine.
Also, in the commit directly after I resolved an error relating to a redundant and outdated Tree.java file, which has been removed.


Modify required arguments for Contract.create

Allow Contract.create() with one of the following.

name+abi+address
name+abi+bin
file

We have a similar PR, so we will close this one and continue at https://github.com/q9f/eth.rb/pull/83!

Oh, sorry.

No problem! ;)


Use only immediately preceding JSDoc

Now only the immediately preceding jsdoc of a node is retrieved by getJSDoc, although it still does the correct non-local lookup for nodes like ParameterDeclaration.
This doesn't change parsing or binding, which use the per-node Node.jsdoc property directly. But it does change everything that relies on getJSDoc, which includes the checker and language service.
Fixes #32062, which contains the analysis that justifies the change.

I believe this also “fixes” #31492


Feature/cu p1btku boss basic structure

Updating develop with changes in the boss scene and boss scripts. There is still some work in progress, but this way the boss in level 2 can be updated.
Pending tasks:

Merge initial camera travel into Boss.scene.
A bug listed on clickup that is not assigned yet.
Switch should cancel critical mode
Balancing
SFX and music

Resolved Conflicts


[Monitor][Query] Update API for resource centric query

With https://github.com/Azure/azure-rest-api-specs/pull/18141 merged, the azure-monitor-query code should be regenerated using the latest specs/autorest.  We should also be setting version-tolerant: true in the swagger file to ensure that we use CADL DPG.

Label prediction was below confidence level 0.6 for Model:ServiceLabels: 'azure-spring:0.2643865,Monitor - Query:0.1414261,Monitor:0.07571463'
Label prediction was below confidence level 0.6 for Model:CategoryLabels: 'Mgmt:0.50597346,Client:0.45510432,Service:0.036323722'


[doc] not the coroutines that I was expecting for

I would kinda expect Websocket, coroutine to be a C++20 coroutine as its part of the language now. Its a bummer that its a stackful boost coroutine example.
Maybe we can :

Websocket, coroutine -> Websocket, stackful boost coroutine
Change Websocket, coroutine example code to have C++20 coroutine.

https://www.boost.org/doc/libs/1_80_0/libs/beast/doc/html/index.html

Coming in 1.81: https://www.boost.org/doc/libs/develop/libs/beast/doc/html/beast/examples.html
Note that ther eis no C++20 coroutine type, the examples ues asio::awaitable which is one possible implementation to use.

thanks


tf2ss() does not seem to work for static gains

Creating a state space object of a static gain using tf2ss results in the ValueError: A must be square.
For example: tf2ss([1.],[1.])

I just verified this for tf2ss (using slycot)--definitely looks like a bug.  Note that there are two cases that may happen when you call tf2ss, depending on whether slycot is installed.  In fixing this bug, it would be good to add tests that test both versions (with and without slycot).

The case of static gains when Slycot is installed is now handled, given changes from https://github.com/python-control/Slycot/pull/5, which will be in the next release of Slycot.

fixed in PR #129, changes from which are now in master branch and will appear in the next release.


Isabelle: 2016

Thank you for the contribution. It was merged directly as a1add23207f326ab6bae33ec6b6e0241afbd70da to keep commit history cleaner. Your contribution is still credited to you.


Mumps: add variant mumps_par with mpi support

Change log
mumps: keep mumps mostly unchanged except the optimized build flag "-O3".
mumps_par: add new varient mumps with mpiSupport, with optional parmetis(unfree) and ptscotch support.
Testing build of mumps_par with parmetis and ptscotch are passed on x86_64-linux and aarch64-linux.
Build of mumps_par failed on darwin platform due to broken dependency of scalapack.
Many downstream scientific packages need mumps_par, for exemple: mfem(to be contributed), pastix(to be contributed), petsc.
This pull request will be ready to be reviewed until the pr https://github.com/NixOS/nixpkgs/pull/350824 is merged.
Things done

Built on platform(s)

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

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[x] sandbox = relaxed
[ ] sandbox = true

[x] Tested, as applicable:

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

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
24.11 Release Notes (or backporting 23.11 and 24.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.

Add a :+1: reaction to pull requests you find important.

Result of nixpkgs-review pr 351476 run on aarch64-darwin 1

50 packages built:

aligator
aligator.doc
bonmin
casadi
crocoddyl
crocoddyl.doc
ipopt
openturns
pinocchio
pinocchio.doc
proxsuite-nlp
proxsuite-nlp.doc
python311Packages.aerosandbox
python311Packages.aerosandbox.dist
python311Packages.aligator
python311Packages.aligator.doc
python311Packages.casadi
python311Packages.crocoddyl
python311Packages.crocoddyl.doc
python311Packages.example-robot-data
python311Packages.example-robot-data.doc
python311Packages.neuralfoil
python311Packages.neuralfoil.dist
python311Packages.openturns
python311Packages.pinocchio
python311Packages.pinocchio.doc
python311Packages.proxsuite-nlp
python311Packages.proxsuite-nlp.doc
python311Packages.tsid
python311Packages.tsid.doc
python312Packages.aerosandbox
python312Packages.aerosandbox.dist
python312Packages.aligator
python312Packages.aligator.doc
python312Packages.casadi
python312Packages.crocoddyl
python312Packages.crocoddyl.doc
python312Packages.example-robot-data
python312Packages.example-robot-data.doc
python312Packages.neuralfoil
python312Packages.neuralfoil.dist
python312Packages.openturns
python312Packages.pinocchio
python312Packages.pinocchio.doc
python312Packages.proxsuite-nlp
python312Packages.proxsuite-nlp.doc
python312Packages.tsid
python312Packages.tsid.doc
tsid
tsid.doc

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

55 packages built:

aligator
aligator.doc
bonmin
casadi
crocoddyl
crocoddyl.doc
ipopt
mumps
mumps_par
openturns
pagmo2
pinocchio
pinocchio.doc
proxsuite-nlp
proxsuite-nlp.doc
python311Packages.aerosandbox
python311Packages.aerosandbox.dist
python311Packages.aligator
python311Packages.aligator.doc
python311Packages.casadi
python311Packages.crocoddyl
python311Packages.crocoddyl.doc
python311Packages.example-robot-data
python311Packages.example-robot-data.doc
python311Packages.neuralfoil
python311Packages.neuralfoil.dist
python311Packages.openturns
python311Packages.pinocchio
python311Packages.pinocchio.doc
python311Packages.proxsuite-nlp
python311Packages.proxsuite-nlp.doc
python311Packages.pygmo
python311Packages.tsid
python311Packages.tsid.doc
python312Packages.aerosandbox
python312Packages.aerosandbox.dist
python312Packages.aligator
python312Packages.aligator.doc
python312Packages.casadi
python312Packages.crocoddyl
python312Packages.crocoddyl.doc
python312Packages.example-robot-data
python312Packages.example-robot-data.doc
python312Packages.neuralfoil
python312Packages.neuralfoil.dist
python312Packages.openturns
python312Packages.pinocchio
python312Packages.pinocchio.doc
python312Packages.proxsuite-nlp
python312Packages.proxsuite-nlp.doc
python312Packages.pygmo
python312Packages.tsid
python312Packages.tsid.doc
tsid
tsid.doc

This is amazing. Thanks for the hard work, will merge this as soon as you fix the conflict, as it is tested working with https://github.com/MatthewCroughan/codedisaster/pull/1


groupWith needs sorting to make groups unique

R.groupWith(R.equals,[0,1,1,2,1,1])
//[[0], [1, 1], [2], [1, 1]]

i checked the resource that its complexity is O(n2)
  while (idx < len) {
    var nextidx = idx + 1;
    while (nextidx < len && fn(list[idx], list[nextidx])) {
      nextidx += 1;
    }
    res.push(list.slice(idx, nextidx));
    idx = nextidx;
  }

so should we make it more efficient to just return the sorted version,like below?
R.groupWith(R.equals,[0,1,1,2,1,1])
//[[0], [1, 1, 1, 1], [2]]

Why not use R.groupBy?
R.groupBy(R.identity, [1,1,2,1,2,3])
// {1: [1,1,1], 2: [2,2], 3: [3]}

groupWith is adopted from #1688.
I think groupWith useful due to reflection of adjacency.

@briefy: Three things:

That would assume that the elements can be ordered.  There is no reason to assume that.

While I haven't thought it through very deeply, that algorithm looks O(n) to me, actually faster than it would be if there was sorting involved.  The key steps are n - 1 calls to the supplied comparator, a similar number of comparisons, and some probably smaller number of calls to slice and push, as well as various assignments.  Am I missing something?

This behavior is very intentional.  The behavior of groupBy, for objects, does group all of the related properties into separate elements.  And you can do that for lists as well by sorting first if that's what you want.  But the current behavior also allows you to collect sequential runs of related elements, which can be powerful.  The function is actually changing in a way to make this a bit more flexible. (#2012, #2016)

@jeiea  thx, i adotped groupBy

@CrossEye   Based on the issue below, i think slice is O(n)
https://github.com/tj/node-thunkify/pull/12
however,i think what you said  pretty make sense,i choose to close the issue,
Thanks a lot !


memory increasing during training

when i train a model on GPU, the Memory is increasing. the dataiter i write following the fcn-xs

@li-haoran Could you help provide a small example?

https://github.com/li-haoran/mxnet/blob/master/example/pose/cpmdata.py  here is my data iter
https://github.com/li-haoran/mxnet/blob/master/example/pose/solver.py  this is solver

https://github.com/li-haoran/mxnet/blob/master/example/pose/solver.py#L80
you need to pass the initial executor as shared_mem here

There is usually no need to write a solver now. Try mx.module.bucketing_module

have a look at the bucketing example

.

This issue is closed due to lack of activity in the last 90 days. Feel free to reopen if this is still an active issue. Thanks!


feat(tasks): Add @typescript-eslint to lint_rules CI task

Part of #2020

Add @typescript-eslint plugin rules
w/ refactoring

Fix compile errors to add other plugins
Remove not intended unsupported rule in eslint

(Use #2117 for updating for a while?)

Current generated outputs
- [x] ban-ts-comment
- [x] ban-types
- [ ] no-array-constructor
- [x] no-duplicate-enum-values
- [x] no-explicit-any
- [x] no-extra-non-null-assertion
- [ ] no-loss-of-precision
- [x] no-misused-new
- [x] no-namespace
- [x] no-non-null-asserted-optional-chain
- [x] no-this-alias
- [x] no-unnecessary-type-constraint
- [x] no-unsafe-declaration-merging
- [ ] no-unused-vars
- [x] no-var-requires
- [x] prefer-as-const
- [x] triple-slash-reference

Warnings
⚠ Rule: no-empty-interface is implemented but not in their lists.
⚠ Rule: adjacent-overload-signatures is implemented but not in their lists.
⚠ Rule: consistent-type-exports is implemented but not in their lists.

Ahh ... no-array-constructor is implemented by eslint/no-array-constructor, I'll need to fix this somehow 😢


Can't find ML.NET "Machine Learning" in the ADD button on the project...

after installing ML.NET in VS2019 (and enabling it in the options), tried to add to the project the scaffold of the machine learning, but can't find "Machine Learning" button inside the ADD to the project button.

Did you also select "install ML.Net ModelBuilder" when installing VS2019?
Seems to be a dup of #969

Yes I installed ML.Net ModelBuilder (which is a preview).

thank you @LittleLittleCloud , the problem was that I have to install the ML.NET ModelBuilder in a separate process from the VS2019 Extension Manager (beside the VS2019 Installer).
once i installed via the Extension Manager the problem was solved.

#954

#978


Correcting upload (Decline cookies & Close the "split" window)

Hello,
I've added some functions to remove the cookies popup and "split" window when trying to upload video.
Feel free to test this.
Thank you for your attention.

Hey, @blackdede I am having a few issues with _remove_window.
From a software design perspective maybe splitting _remove_window into _remove_cookies_window and _remove_split_window instead of passing a string as an argument might be better.
However, more broadly, I am having issues getting uploads to work on Safari. Also, you switch to the default context (ie: moving out of the upload iframe) which can cause issues with the webdriver (you may have to switch back if I am not mistaken).
Anyways, good work through. I will definitely look more into this and hopefully fix some of the popups which have been taking place.

Let's edit this PR


Reverse Engineer HasComputedColumnSql() as needed.

Fix for #4944. For computed columns used HasComputedColumnSql() instead of HasDefaultValueSql().

:shipit:

Found a few more [CanBeNull]s on return values. Removed them too and checked in with 2acd2f1.


Animation load only once (FireFox)

live link: http://trickstemplates.com/themepr/theme3/
Animation in the 2nd tabs only once.
All other browsers are doing good ...
Thanks in advance for your reply!

@PiKkoO  you need to add classes 'animated bounceInLeft' each time you want the animation to fire.
See fiddle: http://jsfiddle.net/5pduaktb/1/

@gskalinskii
Very many thanks!!!!!!!!!!!!!!!!!!!!


Need build instructions for the source code.

I am looking to use this source code for a demonstration for a customer and the readme doesn't provide any context on how to actually build a the .dll for 2021. Some instructions would be helpful.

@dixonjoel are there specific 2021 instructions for build?  Or is it just building the sln in VisualStudio?

It should just be opening the Solution and building in Visual Studio. I use Visual Studio 2019.
I believe you also have to have NI Switch Executive installed – there is a COMReference to niSE and niseCfg that require that.
[NI]
Joel Dixon
Principal Software Engineer
Configuration-Based Software
ni.comhttps://www.ni.com/
From: Kunal Patel @.>
Sent: Wednesday, August 18, 2021 1:53 PM
To: ni/niinstrumentstudio-switchexecutive-hosted-application @.>
Cc: Joel Dixon @.>; Mention @.>
Subject: [EXTERNAL] Re: [ni/niinstrumentstudio-switchexecutive-hosted-application] Need build instructions for the source code. (#3)
@dixonjoelhttps://urldefense.com/v3/__https:/github.com/dixonjoel__;!!FbZ0ZwI3Qg!7WJ5uLgFUmZTf4Hj6T_v8lKc-wNN1jhUxHTVGejYfesUY3LJCt-sZyeTcBHfxA$ are there specific 2021 instructions for build? Or is it just building the sln in VisualStudio?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://urldefense.com/v3/__https:/github.com/ni/niinstrumentstudio-switchexecutive-hosted-application/issues/3*issuecomment-901350320__;Iw!!FbZ0ZwI3Qg!7WJ5uLgFUmZTf4Hj6T_v8lKc-wNN1jhUxHTVGejYfesUY3LJCt-sZyft2qwCww$, or unsubscribehttps://urldefense.com/v3/__https:/github.com/notifications/unsubscribe-auth/AJEUUOXYCGWWXP6E4662RSLT5P6PDANCNFSM5CMRF2PQ__;!!FbZ0ZwI3Qg!7WJ5uLgFUmZTf4Hj6T_v8lKc-wNN1jhUxHTVGejYfesUY3LJCt-sZyeYhKefQw$.
Triage notifications on the go with GitHub Mobile for iOShttps://urldefense.com/v3/__https:/apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675__;!!FbZ0ZwI3Qg!7WJ5uLgFUmZTf4Hj6T_v8lKc-wNN1jhUxHTVGejYfesUY3LJCt-sZycF_NLXQQ$ or Androidhttps://urldefense.com/v3/__https:/play.google.com/store/apps/details?id=com.github.android&utm_campaign=notification-email__;!!FbZ0ZwI3Qg!7WJ5uLgFUmZTf4Hj6T_v8lKc-wNN1jhUxHTVGejYfesUY3LJCt-sZydgIxAD1A$.

FYI:  if SwitchEeecutive isn't installed it will give you an error (as the app looks in the registry to see if its installed).


chore: adjust auto update run time

Coverage remained the same at 100.0% when pulling 24e86fd92c873d066bf6f241869d88717b63e3eb on adjust into e00f6abed85c930b250bf2b33f0cc461aaa1ae58 on main.

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

GitHub release
npm package (@latest dist-tag)

Your semantic-release bot :package::rocket:


[KYUUBI #1759] Remove duplicate close operation api

Why are the changes needed?
Remove duplicate close operation api
#1445 is duplicate with #1517 where action='close'.
/${version}/operations/${operation_identifier}
desc: apply an action for an operation based on a given session identifier and operation identifier
method: PUT
params:
action: "CANCEL" or "ADD" or "CLOSE"
returns: status code

/${version}/sessions/${session_identifier}/operations/${operation_identifier}
desc: remove operation based on a given session identifier and operation identifier
method: DELETE
params: none
returns: an instance of OperationHandle

How was this patch tested?

[ ] Add some test cases that check the changes thoroughly including negative and positive cases if possible

[ ] Add screenshots for manual tests if appropriate

[ ] Run test locally before make a pull request

Codecov Report

Merging #1760 (9af70a0) into master (ab4184d) will decrease coverage by 0.01%.
The diff coverage is n/a.

@@             Coverage Diff              @@
##             master    #1760      +/-   ##
============================================
- Coverage     59.55%   59.54%   -0.02%
+ Complexity      234      233       -1
============================================
  Files           268      268
  Lines         13315    13308       -7
  Branches       1692     1691       -1
============================================
- Hits           7930     7924       -6
+ Misses         4708     4706       -2
- Partials        677      678       +1

Impacted Files
Coverage Δ

...apache/kyuubi/server/api/v1/SessionsResource.scala
77.64% <ø> (+0.47%)
:arrow_up:

...rg/apache/kyuubi/engine/trino/TrinoStatement.scala
64.70% <0.00%> (-1.18%)
:arrow_down:

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 ab4184d...9af70a0. Read the comment docs.

thanks, merged to master


run task directly in hook

basically:
[hooks]
postinstall = {task = "build"}

this would make things a bit faster since mise wouldn't need to run a second time with mise -q build

@jdx would this inherit sources for watch_files hooks?

I'm probably going to remove [[watch_files]]. I think it overlaps too much with other things.

Does that mean we won't have a feature similar to direnv's watch_file?

I'm not sure but I think it's confusing to have task sources/outputs, mise watch and [[watch_files]] as all separate concepts

I think that's a major selling point for direnv.


Fix labels & eos_token for SFT

Fixed two issues:

Padding should be ignored in training. Their labels should be set to -100 for CrossEntropyLoss to ignore them.
Append correct eos_token to the response text. Otherwise the default <|endoftext|> will be tokenized into multiple tokens.

Just rebased and fixed code format. Could anybody merge this PR since it's been approved. I don't have merge permission.

@li-plus, apologies for the delay on this. Can you please share a bit more details on the motivation for this PR. For example, what improvements did you observe in your workloads? Thanks!

@tjruwase These are bugs that should be fixed. In master code, labels for padding are not ignored. Model will be trained to generate padding tokens, which is unnecessary. Also, eos token is not recognized as a single token. If you examine the input_ids in the training_scripts/opt/single_gpu/run_1.3b.sh example, it's tokenized into [49069, 15483, 1397, 1116, 29015, 15483, 15698], i.e. ['.<', '|', 'end', 'of', 'text', '|', '>'], which is wrong. This PR fixed these bugs.


Blotted logs

I have a queue that has jobs with repeatDelay on each Job.
The logs don't get cleared which causes:

Increased DB size.
The process running the queue memory usage gradually goes up and up.

Any advice on how to handle this?

The latest version (3.1.0) added a limitJobLogs Queue option. Is this what you need Sagiv?

Yes thanks man, thats awesome 👍 Are there any significant  breaking changes in 3.x? Is there a changelog for breaking changes?

Change Log
Are you using the error events? v3 changed the error event arguments to fix a bug.

No, glad to see you are semver compliante though 👍
Thanks @grantcarthew this solved my problem. It even truncted the logs without having to reinstate the jobs or anything. Great work as always!


BUGSNAG_CAPTURE_SESSIONS is slowing down every request. Set by default.

Describe the bug
By default when you install the dependency, BUGSNAG_CAPTURE_SESSIONS is set to true if you don't publish the vendor config file.
It's taken me weeks to be able to debug our application and why it was slower on the live site than test site. If you have this set to true, it was increasing every request by 150ms. We have all XL instances on AWS so knew it wasn't caused by infrastructure.
Playing around more, I switched this to false, immediately dropped request from 300-350ms down to 150ms. Turning it back to true immediately increased it back. I spun up another instance to check this on another config and found the same thing again.
We are checking the "Document Request Time" in chrome. In network tab, under "doc", you can see the actual document total request time.
I would expect that it should be heavily documented since this is the case, and disabled by default. The trade-off of a slow request is not worth having the session logs in Bugsnag... especially not on any medium to large application.
Steps to reproduce

Install on any application.
Don't publish the vendor config file
Since bugsnag capture sessions is set, note the request time.
Set BUGSNAG_CAPTURE_SESSIONS=false in the .env file and clear any config caches (and recache)
Run the request again, note the request time. For us, on multiple instances, it was 150ms less when disabling this.

Environment

Bugsnag version: 2.22
PHP version: 7.4.x
Composer version: 2.1
Laravel version: 8.33
Lumen version (if any): -

By default, this is set to false: https://github.com/bugsnag/bugsnag-laravel/blob/master/config/bugsnag.php#L286.

It looks like the code in https://github.com/bugsnag/bugsnag-laravel/blob/master/src/BugsnagServiceProvider.php#L453 is wrong, however.

Yeah I just found this too. When I published the config, it worked fine by default, but without it (which is what I had), was reverting to true.
Seems strange it adds so much to a request too. I'm in Aus so maybe they use a US server? But I wouldn't expect a request with such basic information to be more than 50ms. I would also expect it to be batched & queue'd as the recommended solution. What do you think? I saw the batch config, but couldn't tell if this was for reporting sessions or only for errors as it says - https://docs.bugsnag.com/platforms/php/laravel/configuration-options/#batch-sending

Hi @jonathan-bird,
You're correct, the BUGSNAG_BATCH_SENDING config option is only for errors and doesn't apply to session reporting.
BUGSNAG_CAPTURE_SESSIONS should always default to false if there is no config file - if you're seeing behaviour other than this please feel free to re-open the issue with a repro case.


[Bug] Signup and login pages say "Log in to taniarascia"

To Reproduce

Log out of TakeNote if you're already logged in.

Click Log In or Sign Up.

Note the text under Welcome.

Expected behavior
I'd get rid of the text altogether and just keep the Welcome and the button underneath. Otherwise, you could say "Log in to continue," but that would be redundant.
If you plan on adding more login options in the future, you could get rid of "Continue with GitHub" and just say "GitHub." That way, you could use "Log in to continue" or some other welcome text.
Screenshots

I haven't found a way to change this text yet. It seems to just take the account name.

I haven't found a way to change this text yet. It seems to just take the account name.

Do you mean it takes the account name of the currently logged in user or just takes your account name? Maybe I can look into it.

@danyalasif It seems to be taking Tania's GitHub username. I think it may be a problem with the authentication service's logic—maybe it's taking the name of the owner for an app, instead of the app name itself? Like, I wonder if the app were under a GitHub organization named TakeNote, would that make a difference?


Simple shared directory via config

Works well in my case.
Able to configure/reconfigure after or before default environment seed.
Sprig.configure do |config|
  config.shared_directory = 'shared'
end

Yes, ideally we need something with merge functional. But currently i found this case pretty useful.

Thanks @merqlove. Can you describe how you use this? Do you have apps where you want all your seeds to be the same for all your environments?

@ltk Yeah, currently i have app where i have 100% identical seed. Later it will be splitted into two or more, but for now i want DRY :)
Seems that next time it will fail if process depends on shared model.
But single call or call for different models works fine.
Example:
include Sprig::Helpers

Sprig.configure do | config|
  config.shared_directory = 'shared'
end

sprig [IdReservation, Genre, User, User::Profile, User::ProfileAvatar, User::Purse, SupportTicket, Page, Block,
       BlockPlace, Broadcast::Trailer, Broadcast::Trailer::Video, Broadcast, Broadcast::Background, Broadcast::Poster,
       Broadcast::Subscription, Broadcast::Ticket, Chat, SocialNetwork, SocialLink, Statistic]

# ...
# Sprig.configure do | config|
#  config.shared_directory = nil
# end

# sprig[Something]
#

Rebased with master

Coverage increased (+0.007%) to 98.967% when pulling 99be7d4e10634945b3ba4cd87e71d2defe65ad23 on merqlove:master into e09532617b05bb1064e58eccaedac17ff7495713 on vigetlabs:master.


v1.12 Backports 2024-01-02

[ ] #27187 (@pchaigno)
⚠ Conflict due to TestLoadKeys (slightly different in v1.14) and NewNodeHandler (no nodeAddressing parameter in v1.14)
[ ] #27212 (@pchaigno)
[ ] #27274 (@brb)
[ ] #29934 (@pchaigno)
[ ] #30000 (@brb)
⚠ Conflict due to the position of the encryption-ipsec.rst file (under Documentation/gettingstarted/ in v1.14)

Once this PR is merged, a GitHub action will update the labels of these PRs:
 27187 27212 27274 29934 30000

/test-backport-1.12

/test-backport-1.12

Cilium IPsec upgrade flake tracked here, rerunning.


TokenHolder RevertRedemption Function

This PR implements revert redemption function in TokenHolder.
Flow:

User signs the data using an authorized ephemeral key.
Facilitator calls TH.revertRedemption by presenting his Hashlock and pays the gas.
After verification of ephemeral key, CoGateway.revertRedemption is called.
ephemeral key nonce is consumed in this process.

As per requirement OpenST should run on a single chain also i.e. It should run by communicating with EIP20Token also. TH should not be tightly integrated with UtilityToken. Considering this in mind coGateway address is fetched from UBT while doing revertRedemption.
TH imports UtilityTokenInterface and CoGatewayRedeemInterface. The interfaces has been made it as light as possible to optimize bytecode size.
We are using controlled .call for nonce consumption in TH.revertRedemption with strict check that only CoGateway.revertRedemption can be called.
Note:
In current implementation Facilitator pays the penalty in base tokens.
However as per protocol, redeemer should be charged penalty which is 1.5 times of bounty amount in base tokens. Does this mean TokenHolder needs to hold base tokens for performing revertRedemption? There is an open discussion around it.
Fixes #85

Discussed PR feedback changes(applied to redeem PR also) has been incorporated.
Tokenholder should not be tightly binded with utility token interface. That was the case already. Integration of EIP20TokenInterface and UtilityTokenInterface separately avoids the naming confusion.
UtilityTokenInterface has been renamed to UtilityTokenRequiredInterface which has minimum function signatures required for the TH integration. setCoGateway, mint and burn functions are not needed in this interface as TH doesn't need them.
cc: @deepesh-kn


Should avoid bringing dependencies recompose and styled-components

Very nice library but sadly I won't be able to use it.
The library should be made more vanilla JS and avoid bringing dependencies such as recompose and styled-components. Not everyone want to add those to existing project.
Not really an issue for anyone okay with those choice, but maybe others would opt in if it was not because of this.

This is a good idea. I will take the time to deal with this issue. Thanks.


Disable metrics collection

Inox apparently does not report any metrics to Google but they are collected nevertheless. This is the default behavior of chromium and it cannot be changed without altering the source.
For example calls to the UMA_HISTOGRAM_* macros or UserMetricsAction are all over the place and allow tracking of a wide variety of internal or user initiated processes.
The results can be visualized at chrome://histograms and chrome://user-actions.
Disabling the collections of metrics should increase privacy. In case the browser decides to send metrics back to Google, they will be empty. Besides, I can only imagine that it would result in an improved resource usage.
As for the implementation part of such a patch, it should be possible to modify the macros/functions to skip the metric generation. I will try it myself when I find the time.
EDIT: Disabling the user metrics seems easy by returning instantly in RecordComputedAction and eventually in AddActionCallback at base/metrics/user_metrics.cc. chrome://user-actions stays empty that way.
Disabling the histograms on the other hand seems rather complicated. The UMA_HISTOGRAM_* macros in base/metrics/histogram_macros.h are just a small subset of all calls. Replacing them with empty macros results in a lower histogram utilization but the rest is still active. Apart from that histograms are referenced in many different ways which makes it difficult to disable them as a whole.
They seem to have one thing in common though: base/metrics/histogram{,_base}.cc. It should be possible to modify these files in such a way that histograms become useless.
EDIT2: Reduced the histogram amount to just a dozen by redirecting them to a single dummy histogram. The remaining ones must use a non-standed way of creation.
Now it should be possible to prevent them/it from being updated over time.

Closing in favor of #82.
Sorry for the issue spam, I didn't think I can find the solution that quickly.


ti arm compiler generated elfs have issues

Hello Daniel,
I was playing arround with the Jacinto from TI, and I wanted to write some code for an ARM R5 core. At one point, I wanted to do some calibration using a2ls, I used the "--create" option and got this:
"
evaluate_exprloc eval result is unhandled: RequiresMemory {
address: 536912364,
size: 1,
space: None,
base_type: UnitOffset(
0,
),
}
evaluate_exprloc eval result is unhandled: RequiresMemory {
address: 0,
size: 1,
space: None,
base_type: UnitOffset(
0,
),
}"
The elf I used is the one from the TI demo website: https://software-dl.ti.com/codegen/docs/tiarmclang/compiler_tools_user_guide/getting_started_guide/building_simple_application.html
build using command line, not ccs.
Any idea what could be the issue?

It looks like the TI compiler generated some complicated variable references. The DWARF debugging format is targeted at debuggers, and can express things like "use the value in memory (at runtime) to compute the location of this other value". That is what is being requested here.
A2ltool cannot do that, and the a2l format is not suited for variables whose location changes between runs anyway.
I'll take a longer look at this when I have more time, but I think the only thing that can be done is to delete the warning message.

Commit 4db32d6 removes the warning message


Add a method to clear the queue

This adds a method to clear the queue so that any pending promises won't be triggered.

Coverage decreased (-3.4%) to 96.61% when pulling d2cde6e4de8267d77c544294b9083f5389ca1a2c on heruan:patch-1 into b8c5722bbc5951ca89bd8cc5ef22bdcfdbcea339 on azproduction:master.

Sorry @heruan for that epic delay. Could you please cover your improvements with tests?

@azproduction I've moved on 😄 I'm currently out of spare time to add tests, but maybe others can dive in and complete this PR.

Why do you need to reset pendingPromises?

Probably my misunderstanding of its purpose at the time!

Are there any updates on this feature? @azproduction

Closing PR as the feature is no longer required by the PR author.


need a way to get leaflet (or google map) javscript instance

i think we need a way to get leafmap map  javascript instance inside blade template

Hi! what kind of api would you look for in this?
maybe we can refrence the instance on the window

Closing as OP did not respond.


Support for multi-chip devices

What should we add?
Epic to track Qiskit support for multi-chip devices (i.e. devices with disconnected qubit components, but withconnected real-time classical communication):
Functionality support:

[x] #9710
[x] #9802
[x] #9840

Testing:

[x] Conditional support across components

[x] Automated transpiler testing as part of #9840 and #9802

https://github.com/Qiskit/qiskit-terra/pull/9802/files#diff-df5e0c6b04d9ea56420653f11b66b75d78856b979e1e43abe8cf4e1b78ed9a4aR2171-R2204
https://github.com/Qiskit/qiskit-terra/pull/9840/files#diff-df5e0c6b04d9ea56420653f11b66b75d78856b979e1e43abe8cf4e1b78ed9a4aR2153-R2191

Bugs, performance follow ups:

[ ] #9834

#9834 is pre-existing and could happen even with a connected coupling map