Comment on pull request with a link when the build starts

Leaves a comment on the pull request notifying you when the corresponding build starts.

Can't we do this with GitHub status notifications? Like Travis CI does?

In fact the code right below this is using the status API.


data-custom-properties not picked up if option is inside an optgroup

I've been using <option data-custom-properties="somevalue"> on my multiple select to store some additional data I need. This has been working fine until today when I wrapped my <option>s in an <optgroup>. Suddenly all my customProperties are null. This looks like a bug to me?

Please may you provide a jsfiddle highlighting the issue

Here's a fiddle that highlights this, and my other two issues in one:
https://jsfiddle.net/n7yvrdzL/5/

Hi, I'm also facing this issue, there's any more info about this?

this issue persist on 9.0.1

Dirty fix for this one is find the var addGroupChoices = function (choice) { ... change the customPropeties: choice.customProperties in to customProperties: choice.getAttribute('data-custom-properties'),


ScriptEditor - Right Click Menu broken

Operating system or device - Godot version:
Linux - current master
Issue description (what happened, and what was expected):
The options when opening the right click menu in the script editor don't do what they say:
Select All deletes the line of the cursor or the current selection
Paste undos
Undo does nothing?
etc...
Steps to reproduce:
Open a script, right-click inside the text editor and choose Select All, this should delete a line.

Seem like a one-off error to me.

Can confirm actually, on the 2.2 branch.

Bug comes from tools/editor/plugins/script_text_editor.cpp:1229, await PR ;)

copy / paste not working either when using the right click menu.

@plucian Shouldn't be an issue when/if #7458 is merged. If you really want, you can test the PR for yourself :smiley:

this has been merged into the master not into the 2.2 branch. Could you fix that branch too please ?


refactor!: move MaxNumActivePoolsPerPair into params

Description
Move MaxNumActivePoolsPerPair global constant into the module parameters.
Also change the default value of 50 to 20.
Tasks

[x] Move global constant into parameters
[x] Set the parameter value on upgrade

References

Before we can merge this PR, please make sure that all the following items have been
checked off. If any of the checklist items are not applicable, please leave them but
write a little note why.

[ ] Appropriate labels applied
[ ] Targeted PR against correct branch
[ ] Linked to Github issue with discussion and accepted design OR link to spec that describes this work.
[ ] Code follows the module structure standards.
[ ] Wrote unit and integration
[ ] Updated relevant documentation (docs/) or specification (x/<module>/spec/)
[ ] Added relevant godoc comments.
[ ] Re-reviewed Files changed in the Github PR explorer
[ ] Review Codecov Report in the comment section below once CI passes

Is there any need of Import/Export parts for the param addition? No question/comments other than that.
@richard-bachman SetParams and GetParams will take care of it. Ref the line of code.


Fixing error in example code in readme

Summary
Just a small fix to the readme file.
Details and comments

@mtreinish Bumping this


[cluster-test] Initialize Vaults and genesis using management tools

Initialize Vault keys and generate genesis.blob in cluster test pod and temporarily deliver them to each validator pod, /opt/libra/etc/genesis2.blob

btw, this pr is still a draft, could you change it back?

LGTM, please address the comments

do you have any test plans?

do you have any test plans?

./scripts/cti -E RUST_LOG=debug --pr 5425 --run bench --enable-lsr --lsr-backend vault

/land

This is a bit unusual, these are Rust APIs, but you're using the CLI parser to access the commands. This is very fragile and not a very good pattern because it will fail at runtime and not compile time. We should update this to directly call into the APIs.
@andll @snakech

Gotta. I will look at how smoke test works and refactor this PR. Thanks for pointing out.

by then, please revert this change first I reckon


Support the Array of Struct

Is your feature request related to a problem? Please describe.
After https://github.com/Canner/wren-engine/pull/914, we support Array and Struct type. However, the array of struct isn't supported now.
SELECT struct_array[1].f1 from t1

Describe the solution you'd like
Because this syntax isn't supported by DataFusion, we should add this on the DataFusion side first.
Describe alternatives you've considered
A clear and concise description of any alternative solutions or features you've considered.
Additional context
Add any other context or screenshots about the feature request here.

There are some required works for this issue

https://github.com/apache/datafusion-sqlparser-rs/pull/1541
Plan the access chain for the nested type
Unparse the plan back to sql


Add tpl/site

If it's not much more complicated than https://github.com/gohugoio/hugo/issues/5467, it would be useful to also have site globally available.
It would save having to pass the Page Context to a template which only a needs few params on top of being able to use a few .Site variable/method. (ex: .Site.IsServer https://github.com/gohugoio/hugo/issues/4981#issuecomment-414711966)

I guess that makes sense. But we should probably stop with this one ...

Got it. It's all about the complexity/critical ratio.
It would makes sense that site variables are accessible globally, but again not a critical.


merge from arlac77/template-github,arlac77/template-kronos-module

package.json

chore(package): add ^3.7.0 remove ^3.6.0 (devDependencies.ava)

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


proposed documentation changes

Hi @synek, @tfiers, @ghoshm. I updated the README and CONTRIBUTING guide a bit to make it a bit more newbie friendly. Thoughts? I'm thinking the current CONTRIBUTING.md file could basically go on the main COMOB website.
Feel free to make suggestions/edits. I wrote it quickly and roughly.

I saw you mentioned "open research questions in the notebooks" and it gave me a thought. I've seen in other projects that people "ask for" an issue by commenting on it (e.g. "hey can I take this issue?"), and then the maintainer replies and gives them the go ahead to work on it. This helps make sure that only one person at a time works on each issue, and avoids conflict around who gets credit for things.
Maybe we want to have something similar for research questions?

I think the Contributing.md can also go on the JupyterBook website, next to the Introduction notebook

@synek that seems like a good way to do it and we could encourage people to do so!
@tfiers great can you make that happen?


remove the default hardcoded pull secret

Signed-off-by: myan<EMAIL_ADDRESS>fixed: https://issues.redhat.com/browse/ACM-4731

/cc @clyang82 @morvencao


Azure self-hosted CI runner for GPU tests

Closes https://github.com/groq/mlagility/issues/26
Overview
This PR adds a self-hosted CI runner for MLAgility GPU tests. The VM used is powered by Azure.
Good to understand before reviewing
How often does this workflow runs? 📅
It runs every time a PR is merged into main. You can also trigger it manually using the GitHub cli: gh workflow run 'Test MLAgility Using Self-Hosted Azure GPU' --ref azure_gpu_ci. You can see also see the results as part of the actions page.
How fast is it? ⏱
Around 8m 20s. You can find more details about it here.
What happens if the GPU test fails? 💀
The badge on README.md indicates when this test fails:
The workflow should also always turn off the Azure VM (even if the test fails).
What happens if we try to run multiple of those workflows simultaneously? 🦺
This workflow only starts once the VM has been turned off by the previous workflow. As a result, collisions will only happen if multiple workflows are launched simultaneously. This is unlikely since multiple PRs can't merge simultaneously.
Although very unlikely, issues might still occur if 3 or more PRs merge within a very small time frame (e.g. Workflow A is running, workflows B, and C are waiting and might try to start at the same time). Note that this is only possible if the GPU workflow takes significantly longer than our traditional CI that needs to run between merges.

Added tag for closing https://github.com/groq/mlagility/issues/26


[Feature request] Show app name in the log

Hi
It's good to see name of the app (and icons) requested the dns query in the log (recent queries).

Unfortunately this is extremely difficult, and perhaps impossible.  DNS queries are emitted by the system itself, not by applications, and Intra has no direct way to link each query to the application that generated it.  Thanks to the system DNS cache, many queries produced by apps will never reach Intra at all.
It may be possible to infer an indirect linkage using the contents of /proc/net/tcp, but this mapping is not precise and can be delayed.

So how does split tunneling works?!

Unfortunately this is extremely difficult, and perhaps impossible. DNS queries are emitted by the system itself, not by applications, and Intra has no direct way to link each query to the application that generated it. Thanks to the system DNS cache, many queries produced by apps will never reach Intra at all.
It may be possible to infer an indirect linkage using the contents of /proc/net/tcp, but this mapping is not precise and can be delayed.

Apps can be excluded or included in the VPN by an explicit Android API.  If an app is excluded from the VPN, its traffic is never visible to Intra at all.


Update opentelemetry-sdk-testing to 1.12.0

Updates io.opentelemetry:opentelemetry-sdk-testing from 1.11.0 to 1.12.0.
GitHub Release Notes - Version Diff
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

Ignore future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "io.opentelemetry", artifactId = "opentelemetry-sdk-testing" } ]

labels: test-library-update, early-semver-minor, semver-spec-minor, commit-count:1

Superseded by #504.


Improve check_last_exit_status function

Improve the naming of the check_last_exit_status function because it is not always the last status that is checked.
For example: check_exit_status.
This function takes a fourth parameter that does not seem to be used. Should we remove it?
In case of modification of the argument don't forget to change the name and parameters where the function is called.

As this repo is now low-maintenance, I'm closing all issues that are not bugs.


Better Watermark

Could I get some help on discord? When enabling the watermark the menu goes quite transparent


Instance segmentation labels

Is there a way we can get the instance labels?

Yes, you can access the instance label by dense version of Campus3D dataset. The labels in this version are organized with 23 semantic classes as dimension of N x 23. For each semantic class, the integer number larger than 0 for each point (each row) does stand for the instance id of the point.
However, there are some classes being not countable or difficult to identify instance (like ground or driving road), so the instance ids might not be quite meaningful. We provide the instance label mapping of C4 level in data/label_convert.yaml, where 4 labels can be used for instance segmentation.


Don't specify centroid as the only way to turn an area to a point

Different implementations may prefer different ways of turning polygons into points, and this lets them experiment with new ways. Additionally, the centroid is a particularly bad algorithm as it may lay outside the polygon.

The centroid algorithm is subpar because the centroid of e.g. a C-shaped area would be outside the area.
Mapbox once implemented an algorithm better fit for such a use case called polylabel (https://github.com/mapbox/polylabel), there are various ports to different languages, some are mentioned in the readme.

Yes, pole of inaccessibility is a popular choice, but I don't think we should dictate the algorithm.

The specification currently specifies centroids because that was the only algorithm to turn polygons into points supported by Tilemaker back then (there is a second option since Tilemaker 3.0).

I agree that we should make this change. It is very unlikely to break anybodies styles and from @Nakaner s comment it i clear that there was never any intention to force centroids, so this is basically just an editorial change clearing up the meaning of the text.

I've fixed the merge conflicts. Can another steering committee member review and merge this?


VIH-8530 - add cert bot

JIRA link (if applicable)
https://tools.hmcts.net/jira/browse/VIH-8530
Change description
Add certbot implementation
Does this PR introduce a breaking change? (check one with "x")
[ ] Yes
[ X ] No

implementation changes


perf: Avoid re-renders due to PropsContext

Each Box consumes and provides an new plain object value from PropsContext, that causes pointless re-renderings. Using a single empty object to represent no additional props and component memoization might fix that.

@gabriellsh Could you please test it working on Rocket.Chat? Comparing the latest version against this branch via React DevTools profilling would be enough.

@gabriellsh Could you please test it working on Rocket.Chat? Comparing the latest version against this branch via React DevTools profilling would be enough.

tested, looks way better, thanks!


File-size data migration

Auto-detect file size from file system.
To be discussed: how to let users pass arguments?

Decision: data migration along with schema migration

Codecov Report

Merging #96 into master will increase coverage by 0.01%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##           master      #96      +/-   ##
==========================================
+ Coverage   99.56%   99.58%   +0.01%
==========================================
  Files          12       12
  Lines        1610     1676      +66
==========================================
+ Hits         1603     1669      +66
  Misses          7        7

Impacted Files
Coverage Δ

...are/alembic/versions/1921a169739b_add_file_size.py
100.00% <100.00%> (ø)

ngshare/dbutil.py
100.00% <100.00%> (ø)

ngshare/test_dbutil.py
100.00% <100.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 fa3299d...81cadfd. Read the comment docs.


[BUG] Code does not transpile

The current version of code does not transpile.
Error while TSC:
><EMAIL_ADDRESS>tsc C:\workspace\github\sidebar-sdk-js
> tsc

src/message-adapter/message-adapter.ts:59:5 - error TS2322: Type '{ init: (...args: any[]) => void; configure: (...args: any[]) => void; checkGlobal: () => { checkId: string; }; onGlobalCheckRejected: (...args: any[]) => void; invalidateRanges: (...args: any[]) => void; onVisibleRangesChanged: (...args: any[]) => void; dispose: (...args: any[]) => void; }' is not assignable to type 'AcrolinxSidebar'.
  Object literal may only specify known properties, and 'dispose' does not exist in type 'AcrolinxSidebar'.

59     dispose: _.noop
       ~~~~~~~~~~~~~~~

test/spec/async-adapter.ts:140:7 - error TS2322: Type '{ init(_initParameters: InitParameters): void; configure(_config: SidebarConfiguration): void; checkGlobal(documentContent: string, _options: CheckOptions): Check; onGlobalCheckRejected(): void; invalidateRanges(invalidCheckedDocumentRanges: CheckedDocumentPart[]): void; onVisibleRangesChanged(_checkedDocumentRanges...' is not assignable to type 'AcrolinxSidebar'.
  Object literal may only specify known properties, and 'dispose' does not exist in type 'AcrolinxSidebar'.

140       dispose(_callback: () => void) {
          ~~~~~~~

test/spec/multi-plugin.ts:150:7 - error TS2322: Type '{ init(_initParameters: InitParameters): void; configure(config: SidebarConfiguration): void; checkGlobal(documentContent: string, _options: CheckOptions): Check; onGlobalCheckRejected(): void; invalidateRanges(invalidCheckedDocumentRanges: CheckedDocumentPart[]): void; onVisibleRangesChanged(_checkedDocumentRanges:...' is not assignable to type 'AcrolinxSidebar'.
  Object literal may only specify known properties, and 'dispose' does not exist in type 'AcrolinxSidebar'.

150       dispose(_callback: () => void) {
          ~~~~~~~

Found 3 errors.

I had a look at the Code and the current Version of acrolinx/sidebar-interface does not support AcrolinxSidebar.dispose (See error above). After removing all dispose the code transpiles again.
Tested with:
OS: Win 10 Pro
Node: 8.12.0
NPM: 6.4.1

@kpalatzky Thank you for your bug report.
We’ve overseen this problem, because our test pipeline uses the exact version from package-lock.json (using "npm ci").
Probably we should test it also automatically using "npm i".
I’ve fixed the code and will also prepare a new release, updating some dependencies by the way.

@acromarco When will the release happen? Because later this day i will make a PR for a little change in the XeditorAdapter (We need this change after we released Xeditor 6.0). Would be nice if this change could be in there too!

@kpalatzky OK, I will do the release after merging your PR, maybe tomorrow.


Container build failed on downloading nodejs

I can't build the docker container using the command
docker build -t ort .
error:
> Task :reporter:compileKotlin

FAILURE: Build failed with an exception.

* What went wrong:
Execution failed for task ':kotlinNodeJsSetup'.
> Could not resolve all files for configuration ':detachedConfiguration3'.
   > Could not download node-12.14.0-linux-x64.tar.gz (org.nodejs:node:12.14.0)
      > Could not get resource 'https://nodejs.org/dist/v12.14.0/node-v12.14.0-linux-x64.tar.gz'.
         > Read timed out

I tried on two different computers on two different networks.
build.log

I might have been a temporary outage at the https://nodejs.org/ side. Please try again shortly.

Yes, it looks like the https://nodejs.org/dist/v12.14.0/node-v12.14.0-linux-x64.tar.gz is returning 500 but it sometimes works. This has been happening the last few days. Is there something we could do to make the build of the container more stable?

As a quick measure, if you're fine with the Docker build writing to the host, you could try mounting the host's Gradle directory into the container so the dependencies get cached between builds instead of being re-downloaded.

I am using gitlab to build this because I actually do not want anything to cache. If this happens more often, I'll need to consider it though. Thanks!


theano not using GPU

Hi.
I was wondering how I can use GPU for training?
I checked with tutorial theano python code and confirmed it uses GPU.
However, when I run wavenet, it does not use GPU and instead uses CPU.
Could you share how I can use GPU?
Thanks

Figured it out.
make sure tensorflow-gpu is installed..
pip list | grep tensorflow-gpu can confirm its existence.
Furthermore, refer to link below for requirements for theano 1.0.1 just in case.
http://deeplearning.net/software/theano/requirements.html
You can check if theano is using gpu with the following python code :
##################### start code
from theano import function, config, shared, tensor
import numpy
import time
vlen = 10 * 30 * 768  # 10 x #cores x # threads per core
iters = 1000
rng = numpy.random.RandomState(22)
x = shared(numpy.asarray(rng.rand(vlen), config.floatX))
f = function([], tensor.exp(x))
print(f.maker.fgraph.toposort())
t0 = time.time()
for i in range(iters):
r = f()
t1 = time.time()
print("Looping %d times took %f seconds" % (iters, t1 - t0))
print("Result is %s" % (r,))
if numpy.any([isinstance(x.op, tensor.Elemwise) and
('Gpu' not in type(x.op).name)
for x in f.maker.fgraph.toposort()]):
print('Used the cpu')
else:
print('Used the gpu')
########################### end code
And run
THEANO_FLAGS=device=cuda python test.py


Fix chart to set deletionTimeout  value actually read by config.go

What does this PR do?

[ ] Adds new functionality
[ ] Alters existing functionality
[ ] Fixes a bug
[ ] Improves documentation or testing

Please briefly describe your changes as well as the motivation behind them:

In #798, I changed the values.yaml to match the key in configmap.yaml. It turns out, values.yaml was right and configmap.yaml was wrong 🤦 . I've aligned everything to match what's actually in config.go
https://github.com/DataDog/chaos-controller/blob/d7060a8799045216f1470513f14fb4382310327c/config/config.go#L45

Code Quality Checklist

[ ] The documentation is up to date.
[ ] My code is sufficiently commented and passes continuous integration checks.
[ ] I have signed my commit (see Contributing Docs).

Testing

[ ] I leveraged continuous integration testing

[ ] by depending on existing unit tests or end-to-end tests.
[ ] by adding new unit tests or end-to-end tests.

[ ] I manually tested the following steps:

x
[ ] locally.
[ ] as a canary deployment to a cluster.

Thanks for the fix 🙏


Replace an existing assertion

I have a technical question after reading the standard briefly. How can we prevent a hacker from replacing an existing (genuine) assertion by a fake one? Because the hacker would know the hashing methods as listed in the standard.  - I am just trying to understand how the standard could protect various hacking cases.
Thanks!

The hash of the assertion is contained in the claim, and the contents of the claim are included in the payload over which the signature is generated. This means:

If an attacker alters the hash in the list of assertions in the claim, the signature will fail to validate, because the attacker does not have access to the signer's private key to regenerate the signature.
If an attacker alters the contents of an assertion, the hash will no longer match, and so the assertion will fail to validate at that point.

In other words, knowledge of the hash function is not helpful; it's the attacker not having knowledge of the signing key that prevents this attack.


Add the in-place tools to the product

Purpose
Adding the in-place tools to the bin folder of the product.
Goals
In-place tools should be available from the next product releases.
Test environment
java version "1.8.0_181"
Apache Maven 3.6.2

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.


pouchdb-fauxton blocks use of pkg

I have a project using express-pouchdb that is being packaged up as a single exe using https://github.com/zeit/pkg
However since including express-pouchdb, it is failing due to the way that pouchdb-fauxton is including its assets.
As I am not using pouchdb-fauxton I can make it build by manually removing any references to that module, but that is not a great longterm solution.
The simplest way to solve this would be to remove the dependency on pouchdb-fauxton and require the users who want it to explicitly add it.

Is there any updates on this one from the pouchdb devs?


Feature Request: Sort packages alphabetically inside each category

Hello - Thank you for creating pipenv. It is really interesting and useful. Thanks for putting all of the package and environment management into one place.
What do you think about the idea of sorting package names alphabetically inside each category?
I tried to collect some examples of where else this is done in the python community:

I see that pipenv itself added alphabetical sorting for the hashes inside the lock file in March 2018 - https://github.com/pypa/pipenv/issues/1529 .
Both pip freeze and pip list output their list of packages in a case-insensitive sorted order, and the help docs for each command specifically mention this.
Tools like isort are built specifically to sort and organize packages - https://pycqa.github.io/isort/
Tutorials like RealPython mention sorting as a best practice - "I like to sort them alphabetically because that makes it easier to find what I am looking for." - https://realpython.com/lessons/requirements-files-best-practices/

I have encountered many python projects and requirements files where the package list is sorted alphabetically for easier readability.
Personally I find package lists easier to read and diff when the names are sorted. It might be nice if pipenv took care of this automatically.
From reading through pipenv code, issues, and discussions - my understanding is that this would be only a cosmetic change, and not have any change on the functionality. It appears that pipenv does smart calculations to figure out the correct package installation sequence, and always installs packages in a good working order. Thus sorting the contents of the Pipfile package list should only affect how humans read and use the Pipfile. It would not affect the tool behaviour in any way. Is this correct?
I have a very small working prototype, which I can send a PR for shortly. Simply to start a discussion. (edit: here - https://github.com/pypa/pipenv/pull/5965 )
I realize this might be a drastic change for existing projects that use pipenv, so I thought I would start by filing this Issue to ask.
Thank you for your time.
..
From the Issue template:

Existing issues

pipenv added alphabetical sorting for the hashes inside the lock file in March 2018 - https://github.com/pypa/pipenv/issues/1529
Some discussion on ordering here, but closed in Aug 2022 as "may have improved" - https://github.com/pypa/pipenv/issues/2274

make sure you are running the latest version of Pipenv.

I just cloned the repo today to write a test PR:
$ pipenv --version
pipenv, version 2023.10.4.dev0

Is your feature request related to a problem? Please describe.
I find it faster and easier to read a list of python packages if that list is sorted alphabetically by package name.
This also results in smaller diffs when adding new packages or doing code reviews, because it is clear where any package should be added or removed in the list.
Adding sorting as a feature may cause initial pain when a project sorts their list(s) for the first time.
But hopefully after that it will always result in smaller diffs and easier, faster reading and work.
Describe the solution you'd like

pipenv creates package lists in sorted alphabetical order
Whenever a package is added to a category, it is inserted in the correct sorted location

Describe alternatives you've considered

Manually sorting Pipfiles. This is not ideal because it requires regular manual work, and you have to pay extra attention each time to make sure the package goes into the right spot. It may also results in differences across different people working on the same project, since the behaviour is not being consistently and automatically applied. Some people may sort their files. Others may not.
Doing nothing. Just living with unsorted Pipfiles. Obviously this works and the pipenv community has been living with it so far. However it seems like reading or searching through a package list takes longer than it should - I have to grep or look for longer to check whether a package is already in the Pipfile, and where.

Additional context

Please run $ pipenv --support, and paste the results here.

I ran this but it seems to contain several pieces of personally identifying information. Is it needed for feature requests? I have left it out for now.

IMHO if this is only done when running Pipenv commands that modify the Pipfile (e.g. install, uninstall), this shouldn't be disruptive. Other tools (*couch* npm *cough*) do much more to their package definition files than just sorting the packages alphabetically. If someone really wants to sort their packages differently, they can just add them to Pipfile manually and then run pipenv install instead of pipenv install <package>.


Build workflow deactivated

Cancel deactivation by

automatically commit regularly
notify administrators to reactivate the workflow

ref: https://docs.github.com/en/actions/managing-workflow-runs/disabling-and-enabling-a-workflow

This issue was resolved by https://github.com/tusvml/blog/commit/f539997905f807262077fcbf90d860268b75363d.


Cannot connect to Ollama Endpoint.

I received the following issue by using an Ollama Endpoint: http://my_server_ip:11434/api.
[Instructor:DEBUG] 2024-10-08T07:13:46.505Z: Unknown provider - cant validate options.
It is noted that:

I am using TS Lib.
The correct prefix by our server is /api, instead of /v1.

Your help is much appreciated!

I also tried prefix /v1 but it did not work.


Child add/remove animation support?

Is there a way to animate on child add/remove?

Hi @hurelhuyag!
I apologize for the delay in responding.
It is currently not possible. I need to check it out okay?


Only create nat gateways if a private subnet has been defined.

Since AWS nat gateways cost $0.045 per hour, only create them if they are needed to support private subnets.

Closed by PR #98


New Referenceable classification type called "Placeholder"

Is there an existing issue for this?

[X] I have searched the existing issues

Current Behavior
Missing specific type for describing special lineage use cases.
Expected Behavior
New classification type called "Placeholder" (similar to Memento) to designate partial, incomplete Referenceable.
Use-case: Create lineage in cases where abstract or incomplete elements are present in the external technology (i.e. Data Stage sending information for IGC virtual assets).

Suggested name "Placeholder" and Area7 since it is for lineage.
Data Engine (or Asset Lineage) OMAS is the place where this will be used. (Refinement still needed in separate issue about Data Engine); Open question: Classification assigned on asset level (and use anchors to read) or all elements in context (?)

Alternatives
No response
Any Further Information?
No response
Would you be prepared to be assigned this issue to work on?

[X] I can work on this

@lpalashevski I was wondering what the lifecycle of this classification might be in your use case; including considering how these assets are searched for, made real, deleted and audited.
It might be useful to add in properties to the classification to help with its lifecycle management. Maybe a description and something to identify the code that is creating the classification. Maybe a String property called creator :" ETL engine 1" or ETL Engine 2"; if this would help for finding, deduplicating and auditing.

Updates discussed on 21-09-2021:

Agreed name of the new classification type: "Incomplete";
At this point there is no clear requirement to add properties to help managing the lifecycle of the classification. (such as in the example above; we can always add properties later if there is more clear use-case for that).


demo 2019 build failed

I tried to build the 2019 demo with unity 2019.2.19f1 and it tells me the build failed because: UnityCompositorInterface' does not exist in the current context, I tried both by using the build setings -> build to generate a VS solution and the mrtk build window to generate an appx and both failed, tried it for ARM and ARM 64 with no susccess

A fix went in for this yesterday:
https://github.com/microsoft/MixedReality-SpectatorView/pull/389
You may need to pull the most recent master to pick up the change.
Please reopen this issue if you still experience problems after getting commits in the above pull request. :)


Use fips preview packages

at the moment we have a weird issue where usbmuxd user is added and a usbmuxd,libusbmuxd6 packages are installed.
i have no idea where this is coming from as we only installed fips related packages. so why would a usbmux deamon be installed that is used for appl/ios devices
@toabctl @rthill91 ideas

All CIS-related exclusions were still from the bionic fips branch where we used an AWS-specific kernel. Now that we have switched to a generic fips kernel the CIS spec just works.


Fixed extern crate statement for rustc_serialize

There were the following issues with your Pull Request

Commit: 6c0dc0307f2bd9a968b72461fbf73dd5362c0c87
Commits must be in the following format: %{type}(%{scope}): %{description}

Guidelines are available at https://github.com/rustless/jsonway/blob/master/CONTRIBUTING.md

This message was auto-generated by https://gitcop.com

Should have read https://github.com/rustless/jsonway/blob/master/CONTRIBUTING.md beforhand


KeyError: u'height_scale'

Hi,thank you very much for this tool.
I want to do a conversion from pytorch to caffe.
My settings are as follow:

onnx 1.6.0(pip install onnx)
pytorch 0.4.0
protobuf 2.6.1
python 2.7.17

There is no error in running test.py
root@ubuntn:/home/merz/onnx/onnx2caffe-master# python convertCaffe.py hourglass.onnx hourglass.prototxt hourglass.caffemodel
[libprotobuf WARNING google/protobuf/io/coded_stream.cc:537] Reading dangerously large protocol message.  If the message turns out to be larger than<PHONE_NUMBER> bytes, parsing will be halted for security reasons.  To increase the limit (or to disable these warnings), see CodedInputStream::SetTotalBytesLimit() in google/protobuf/io/coded_stream.h.
[libprotobuf WARNING google/protobuf/io/coded_stream.cc:78] The total number of bytes read was 765562426
Traceback (most recent call last):
  File "convertCaffe.py", line 108, in <module>
    convertToCaffe(graph, prototxt_path, caffemodel_path)
  File "convertCaffe.py", line 55, in convertToCaffe
    layer = converter_fn(node,graph,err)
  File "/home/merz/onnx/onnx2caffe-master/onnx2caffe/_operators.py", line 277, in _convert_upsample
    factor = int(node.attrs["height_scale"])
KeyError: u'height_scale'

What should I do with this error?
Please guide me.

hi, did you fix this bug?

@CangHaiQingYue 搞不定，放弃了

I meet this issue,too.

I meet the issue when run 'convertCaffe.py', and it links to " _convert_resize",
 factor = int(node.attrs["height_scale"]) KeyError: u'height_scale'
What should I do?

I think it might comes from pytorch2onnx,when I set  torch.onnx.export to opset_version=9,this error disappear. @CangHaiQingYue

Thanks a lot!@Eliza-and-black

@CangHaiQingYue 搞不定，放弃了
我直接写死了

@CangHaiQingYue 搞不定，放弃了
我直接写死了

Do you means  you let factor = 2 ?

@CangHaiQingYue 搞不定，放弃了
我直接写死了

Do you mean that you let factor=2?

@CangHaiQingYue 搞不定，放弃了
我直接写死了

Do you mean that you let factor=2?

Yes


More parameters in BezierChartScale

Hello,
Could you please include the Daily parameter in BezierChartScale ?
Thanks !

And, could you also include 15min and 30min parameters ?
Thank you very much !

There is a new scale   : BezierChartScale.HOURLY , that you could use

Hello, I saw this option but I would also need a Daily parameter.

there is a WEEKLY scale :)

Please add DAILY scale, weekly scale does not work because it sums multiple days' y-axis values of that week and coalesce into one data point.

Sorry, what I wrote is inaccurate.  In weekly, y axis values of the same day is coalesced into same day, which, for example, if a value was entered twice for the day, it becomes a sum of those values


Fixes https://github.com/jnbt/candy_check/issues/36

Coverage increased (+1.2%) to 99.927% when pulling 9e35d5c28356d22765c617f2f8613fe4ee8aa58e on sonxurxo:feature/app_store_multiproduct into 28872f98ba5c26cd2dd7f395d5c03090ecf13bea on jnbt:master.

Just force-pushed because I removed the bundle version update in .travis.yml and gemspec

@sonxurxo I moved your changed to an own PR #40, because I didn't wanted the Verification or RecipeCollection class to be changed for a specific use case.
Instead I moved most of the modifications to SubscriptionVerification


networking mode, aws-speedup from talk on 12/05/2017

tagging @tima , @dfederlein , @thedoubl3j
SQUASHHHed

Will be combing through some of this tomorrow, my apologies on being late to the game.


Create Song “2024-06-17/index”

Automatically generated by Netlify CMS

👷 Deploy Preview for subtle-kashata-e61983 processing.

Name
Link

🔨 Latest commit
1dc8bf08062c4e259594c54d601f418d1b3c9b13

🔍 Latest deploy log
https://app.netlify.com/sites/subtle-kashata-e61983/deploys/66705b035fb20f0008735662


fix readme

the current docs have a bit of a contradiction.
In the top where it shows the parameters it mentions separator but in the example it uses - uses: jungwinter/split@v1 and v1 has a typo where it expects seperator.

No worries we used it today and ran into this 😁. Thought id contribute the fix.


install.bash: automatically add lines to .bashrc / .bash_profile

hopefully you can use tput on mac, if not we can just drop that.

Wow, :+1:
Maybe we could also source the .bashrc automatically?

hmm I dont think there is an easy way to that. :)

Isn't there the source command?

yeah, but then you source it for the script that runs it and not for the interactive shell.

Ah I see.

Very nice, thank you!

hopefully you can use tput on mac

Worked fine on my osx, no need to update the PR.


Unable to use multiple XmlElement objects?

Let's say I want to pull some XML from a website:
PS C:\WINDOWS\system32> $content = Invoke-RestMethod -Uri https://www.bankofcanada.ca/valet/observations/FXCADUSD/xml -Method Get

And let's say I want to peel off some of the subelements and fool around with them, like I would with any other object:
PS C:\WINDOWS\system32> $test = $content.data.seriesDetail.series
PS C:\WINDOWS\system32> $test

id       label   description                                      dimension
--       -----   -----------                                      ---------
FXCADUSD CAD/USD Canadian dollar to US dollar daily exchange rate dimension

PS C:\WINDOWS\system32> $test.id = "12345678"
PS C:\WINDOWS\system32> $test

id       label   description                                      dimension
--       -----   -----------                                      ---------
12345678 CAD/USD Canadian dollar to US dollar daily exchange rate dimension

Why is it that the parent document ends up being affected?
PS C:\WINDOWS\system32> $content.data.seriesDetail.series

id       label   description                                      dimension
--       -----   -----------                                      ---------
12345678 CAD/USD Canadian dollar to US dollar daily exchange rate dimension

PS C:\WINDOWS\system32>

Is this intended behavior?

Not so much explicitly intended by PowerShell, but more a function of how reference types behave.
A majority of .NET types are reference types, which means that if you assign the value to another variable, only the reference to that item is assigned. This avoids what would otherwise be a significant amount of processing overhead that would be required to entirely copy the objects, and essentially means that if you assign a reference to a variable, it will still affect the original variable or the original object it was attached to.
To cover the few cases where you do want to create actual copies of items, many classes have Clone() methods which can be applied to create copies, though the degree to which they are copied and how much data can be duplicated will vary depending on the implementation for that individual class.
According to the documentation here there is a .CloneNode() method on XmlElement, so you should be able to do $test = $content.data.seriesDetail.series.CloneNode() to create a copy of the node to work with and modify without affecting the original node.


Delete footprint(s), then Undo, erases LCSC part selection

This is an issue in my case only because I often build more than one PCB for a product in the same Project/PCB file. This is because the boards mate together and share a schematic. So when I export Gerbers, I will simply select one of the boards, delete it, then export the other. After, I will Undo, delete the other one and export again.
I can also imagine a situation where a part(s) is accidentally deleted and thus Undo is used. If this happened after LCSC part selection, you'd have to search and select again. Depending on the number of parts, it could be a big problem.
Thanks for your generous work on this!

Hi,
this is a very special way of doing things ...
The problem is that the info what footprints are on the board is directly read from the *.kicad_pcb file.
The plugin stores the footprints into the project.db file when the plugin is started and deletes those from the database that are no longer on the board. So if you select a LCSC part for a footprint, that info is stored in the database as well.
Then you close the plugin, delete a footprint, start the plugin again and it will delete the just removed footprint as well as the LCSC info.
Changing the plugin in a way that supports your special workflow is a lot of work and I think I don't want to invest that time ...
If you come up with a PR that implements this and does not make the normal experiance worse I happily merge it.


How to properly use requestVerifyKey?

I'm having trouble using requestVerifyKey to verify a signed message.
const keychain = window.steem_keychain;
keychain.requestSignBuffer('username', 'msg', 'Posting', function(response) {
    console.log(response);

    keychain.requestVerifyKey('username', response.reply, 'Posting', function(response) {
        console.log(response);
    });
});

This results in a "Incomplete data or wrong format" error.
#78 suggests adding a # to the message so if I'm not mistaken it would be requestVerifyKey('username', '#' + response.reply, 'Posting'.
This solves the error but does not result in a successfull callback, but in another error
{
    data: {
        key: 'posting',
        message: '#1f716b3247b63179ed74a2a1ba7a9b098ac4e895986f475fdcfee9cb83e08f0b9e1284d6022ed072a1641cfb6268022e9dbe7c62c874433d42774a42a084c0e870',
        method: 'Posting',
        request_id: 2,
        username: 'username'
    },
    error:  'decode_error',
    message: 'Could not verify key.',
    request_id: 2
    result: null,
    success: false

}

What exactly am I doing wrong and what is the purpose of starting the message with an #?

verifyKey simply checks if you can decode a message with a given key.
Steps:

Encode whatever message using user's public key.
Ask the user to decode it using requestVerifyKey
check the value returned against the message you ve encoded.


nargo check doesn't rewrite Prover.toml and Verifier.toml

Aim
I was trying to use nargo check to generate Prover.toml and Verifier.toml files to be able to fill in witness and public inputs.
Expected Behavior
nargo check should recreate/rewrite generated files (Prover.toml and Verifier.toml)
Bug
I created new nargo/noir project and after nargo check there were two files generated - Prover.toml and Verifier.toml; Prover.toml contained witness input format for default nargo/noir circuit.
When I changed the circuit and ran nargo check again - it didn't change the files.
To Reproduce

Create new nargo project (nargo new example)
Run nargo check
Change the circuit , so it has different variables set as an input
Run nargo check again and generated files not changed

Installation Method
Binary
Nargo Version
nargo 0.6.0 (git version hash:<PHONE_NUMBER>a9e3e46c6d8c3169ad5d25971d4282, is dirty: false)
Additional Context
No response
Would you like to submit a PR for this Issue?
Maybe
Support Needs
No response

I think an important thing to keep in mind for this issue is that nargo check shouldn't delete info from the user's Prover.toml. i.e. If I remove an argument then it should still exist in the Prover.toml file but if I add a new one then it can create a skeleton value for it.


Feature Request: Link Tomorrow's Day Planner

Thanks so much for your work on this.
I often plan the next day the night before. Is there any way we can link the day planner to the date of the note, or does it always have to be done day of?

Bump. Being able to plan tomorrows tasks, would indeed be very useful.

Agreed!

+1

+1

As an added note, one use case is: I know I have to do tasks X, Y and Z on days A, B and C. I'd love to be able to create daily planner notes in advance for days A, B and C, add tasks X, Y and Z to them. So when the plugin is triggered to create a new day's note, it could check if a note already exists for that day, and just not create a new one (thus preventing overwriting the pre-created notes for days A, B and C).
I may have over-complicated the description with all the single-letter codenames for days and tasks but hopefully it makes sense :) If not, happy to provide more (and clearer) info!
And, thank you so much for all the work you've been doing here. I can only hope that you have an idea of how much easier you made lots of people's lives with this plugin.

Available in 0.7.0.


Allow custom library permissions to be set for group(s)

„We use Draw.io to create mockups for our Clients. We do have multiple Clients and Projects dealt with by different groups of people.
We have projects that use licensed icon libraries and projects that require to only be seen by people that signed the according NDA.
If we can’t restrict reading rights to Draw.io libraries we risk, that the licensed icon libraries are used in projects not allowed to use them and projects requiring an NDA can’t use libraries at all. This results in huge inconveniences and in the end would force us to look for alternatives to Draw.io.“

Allow to set custom libraries for groups per default


keep everything contained in the webpage

Fixes #2

Wait, it's not working

Wait, it's not working

Yeah, it removed all the options. Good thing netlify gave the preview so I didn't have to do any testing lol.

Wait, it's not working

Yeah, it removed all the options. Good thing netlify gave the preview so I didn't have to do any testing lol.

@TechStudent11 Also, you may need to go to this video and get embed codes from there - https://www.youtube.com/watch?v=TtY9eRayseg. The songs are timestamped, so that's much easier for you. If you don't wanna do that, I can do that but just explain the code to me. Thanks

Bruh I just updated it

@Netlify isn't giving me another deploy preview :(

@netlify isn't giving me another deploy preview :(

Oh, its actually on the same link nvm

one more thing...

@TechStudent11 It's working. I would like to see how you did it.
Baby It's Cold Outside starts little early. I can still here the end part of the first song.

@TechStudent11 It's working. I would like to see how you did it.
Baby It's Cold Outside starts little early. I can still here the end part of the first song.

It needs to start at 3:27 not 3:26.

kk

The rest is all very good. Thanks a ton!

@TechStudent11 It's working. I would like to see how you did it.
Baby It's Cold Outside starts little early. I can still here the end part of the first song.

It needs to start at 3:27 not 3:26.

@KendallDoesCoding No prob. Should work now.

@TechStudent11 I'm adding the js part to another file and then its good to merge.

@KendallDoesCoding Alright. That's way better than my implementation.

@KendallDoesCoding Alright. That's way better than my implementation.

I just want to format everything, but you did all the hard work needed. This is exactly what I needed to get done. Now, I too know how to do it.

@TechStudent11 I just finished indenting it.

@KendallDoesCoding I just finished indenting it.

Thanks, I'll do final edits and merge tommrow.

Off-topic: I never tried Netlify PR's, I was at 11.5 minutes and now I'm on 13 after this PR lol. I only have 300 minutes deployment time, but that's too much so I'll anyway not reach that in the next 4 days. So that's all good.

Thanks a lot @TechStudent11 . Merging...


plugins/none-ls add stylelint

Adds stylelint to none-ls

Looks like the build errors might be transient?
Not sure if I can trigger them again without a commit

Looks like the build errors might be transient?
Not sure if I can trigger them again without a commit

I re-triggered it.


Improvements for Siphon materials

allow more time
remove u''
No need to import metpy in NCSS model example
Try to make things a bit more gradual with getting hands on
NCSS grid as point not shown

Siphon materials are now their own episode again as of #247 and these are solved/no longer relevant.


Add support for PSR-3

Fixes slimphp/Slim#2930

Coverage remained the same at 100.0% when pulling 8ca334706e189956a71027aa5bf8bd571dba45ea on elazar:feature/psr-log into b8c69e6a544ae75c221390ee1c00b92b92d6054c on slimphp:4.x.

Great 👍 Thank you!


Specify the correct install location for arch

Thank you @vwxyzjn. I believe that location isn't specific of Arch but, rather, it is the system-wide installation path for services menus. So I think it's a good idea to include that route but marked as system-wide or other similar label.

In Ubuntu /usr/share/kservices5/ServiceMenus exists and contains some service menus installed for various packages.


Request.Form throws 'Unexpected end of stream` if Request.Body was read

using (var streamReader = new StreamReader(context.Request.Body))
{
    streamReader.ReadToEnd();
}
if (context.Request.HasFormContentType) {
    var form = context.Request.Form; // << exception here
}

Can we give better error message?

Was this for form or multipart form?
Recommendation?

It was multipart form. Something like "Could not read Form after body have been read" would be more descriptive I think.

Unexpected end of Stream, the content may have already been read by another component.?

Sounds good.


MinMax component prints maximum value on form build

Once I turn on min_max component, it prints out a number after the label, using bootstrap wrapper. I could band-aid it with adding an empty string before line 10 in min_max.rb to return that. However, I can't decide whether it is an adequate solution.

:bomb:
You can fix this returning nil for min_max method. Like this one
Could you please open a pull request fixing this issue and adding a text to it?

Ow Lord, you're completely right, we missed that. Your fix is basically right as well, but we are returning nil instead, like in the maxlength component. Could you please send us a pull request with this fix in place? Thanks!

@rafaelfranca lol, here we go :D


[upload] Apply naming convention to extra headers added to Http uploader

Artifact properties will be added as headers when uploading to Http. Property keys should follow a well defined format.
Proposal: X-<name-in-all-caps>
If name is camelCase then words will be separated by a -, such that
artifacts:
  - path: path/to.app-1.0.0.zip
    platform: osx
    extraProperties:
      foo: bar
      fooBar: foobar

Results in the following headers

X-PLATFORM: osx
X-FOO: bar
X-FOO-BAR: foobar

Http headers should be explicitly defined.


[Build System] Don't clean sourcekit-lsp prior to running tests

Depends on https://github.com/apple/swift/pull/41223/commits

Previously, we were always cleaning the sourcekit-lsp build directory prior to running tests and installing. That’s a waste of time. Only clean prior to the first build.
rdar://88618732

Build failed
Swift Test Linux Platform
Git Sha - fddbc285d15c28891733d3b142dea8dce6538f1f

https://github.com/apple/sourcekit-lsp/pull/456
@swift-ci Please test

@swift-ci Please smoke test


Scripts upgrades

Implements #129

[x] updated scripts.
[x] added multichain support

You can try deploying on local env first too:


[DDW-517] Implement Structured Logging for Daedalus Logs

This PR structures the Daedalus logs in the same format as the Cardano logs, which follows the structured logging specification from the cardano-sl docs.
From the cardano-sl docs: https://github.com/input-output-hk/cardano-sl/tree/develop/util
Screenshots:

Review Checklist:
Basics

[x] PR is updated to the most recent version of target branch (and there are no conflicts)
[x] PR has good description that summarizes all changes and shows some screenshots or animated GIFs of important UI changes
[x] CHANGELOG entry has been added and is linked to the correct PR on GitHub
[x] Automated tests: All acceptance tests are passing (yarn run test)
[x] Manual tests (minimum tests should cover newly added feature/fix): App works correctly in development build (yarn run dev)
[x] Manual tests (minimum tests should cover newly added feature/fix): App works correctly in production build (yarn run package / CI builds)
[x] There are no flow errors or warnings (yarn run flow:test)
[x] There are no lint errors or warnings (yarn run lint)
[x] Text changes are proofread and approved (Jane Wild)
[x] There are no missing translations (running yarn run manage:translations produces no changes)
[x] UI changes look good in all themes (Alexander Rukin)
[x] Storybook works and no stories are broken (yarn run storybook)
[x] In case of dependency changes yarn.lock file is updated

Code Quality

[x] Important parts of the code are properly documented and commented
[x] Code is properly typed with flow
[x] React components are split-up enough to avoid unnecessary re-rendering
[x] Any code that only works in Electron is neatly separated from components

Testing

[x] New feature / change is covered by acceptance tests
[x] All existing acceptance tests are still up-to-date
[x] New feature / change is covered by Daedalus Testing scenario
[x] All existing Daedalus Testing scenarios are still up-to-date

After Review:

[ ] Merge PR
[ ] Delete source branch
[ ] Move ticket to done on the Youtrack board

Hi @CodiePP! Do we have some kind of a structured-logs analyzer/checker which we could use to check if our logs are correctly formated?

Thanks @CodiePP - let's have a quick chat/call tomorrow to discuss our options 🙏

@MarcusHurney please let's also change the Daedalus.log extension from .log to .json.
Please be sure to update the "ALLOWED_LOGS": https://github.com/input-output-hk/daedalus/blob/feature/ddw-517-implement-structured-logging-for-daedalus-logs/source/main/config.js#L60 and any other mention of Daedalus.log 🙏

@MarcusHurney perhaps we should also make the logging output more condensed in NODE_ENV === 'production' environment 🙏 - please see the comparison of Cardano and Daedalus logging:
Cardano logs

Daedalus logs

I would like to keep the current nicer logging in development mode 🙏

@nikolaglumac
I've addressed all requested changes except 1, which is removing the stringifyError(error) call from the Logger.debug calls. I'm working on it now and will let you know when it can be given a final review.

@nikolaglumac
All the requested changes have been made. Here are two images showing what Daedalus.json looks like in production and how the logged errors look without stringifyError.

@darko-mijic @daniloprates @MarcusHurney @DominikGuzei this PR MUST go into 1.5 - I have discovered and fixed a BUG with the ada redemption of the paper-vended-certificates.
We were sending a wrong API request body param :(

Docs: https://cardanodocs.com/technical/wallet/api/v1/#tag/Transactions%2Fpaths%2F~1api~1v1~1transactions~1certificates%2Fpost
Fixed in:
https://github.com/input-output-hk/daedalus/pull/1299/commits/bc880bcbd7a33584c326b45dfb393822818183d2

@CodiePP here's an example of Daedalus.json log file:
Daedalus.json.zip


Broken link on https://lightsats.com/guide/tip

The link to ln.cash on https://lightsats.com/guide/tip is broken.

Hi @GlenCooper thanks for reporting this!
Unfortunately it seems they shut down the site: https://twitter.com/lncash?lang=en
Probably it's best if we just remove that site from the guide.

Confirmed removed.  Looks good now.  👍

Also, just for the record, here's what they had on their Twitter page;
https://x.com/lncash/status/1627489555619119105 | https://archive.ph/F7hHl

Yeah, not good! thanks @GlenCooper  :scream:


Don't call cargo search and cargo audit unconditionally in makefiles

These seem to be called for rare purposes, but cargo search may hit the internet whenever it needs.
For some purposes, like benchmarking or building offline, it may not be appropriate to have that network hit happen unconditionally.
Restructure this part of the Makefile so that these cargo calls only happen when the targets that need them are invoked.

Fixed.


Expose Wallet Engine Sync Progress API

The problem that you wish to solve
I have several new wallets that I have added to my node, however they are all reporting as state 'ready' via /v2/wallets GET.
The strange thing is that the console says otherwise, cardano-wallet.cardano-engine is still syncing from the cardano node. E.g:
 Syncing (19.37% percent) Applied 8000 blocks, 0 rollbacks in the last 30.043631812s. Current tip is [point 4443f653 at slot 5293353]."
Now I am aware that my specific wallets may be ready to send/receive transactions, however I am creating a node backup process where it checks that all necessary processing has occurred so I can then save the state into an image for future quick-boot usage. It's currently not possible to determine when cardano-wallet has finished syncing. I have also checked the EKG stats endpoint.
Description
Expose a way to retrieve the wallet sync progress %.
/v2/wallets and /v2/network/information does not contain this info.
Implementation suggestions
Include a new field as part of /v2/networking/information

The strange thing is that the console says otherwise, cardano-wallet.cardano-engine is still syncing from the cardano node. E.g:

That's curious, if GET reports "status": "ready" on a wallet, then the wallet should be, well, ready. That said, there is a background process that detects stake pools, maybe that is causing the log messages.
Could you include more detailed logs? (Or contact us privately via the Cardano Foundation (Slack) if you don't want to post your logs publicly. Happy to escalate to a contact person if you like.)

These are the logs I receive while all my wallets are status ready as well as network/information endpoint is ready/in sync.
{"MESSAGE":"\u001b[34m[cardano-wallet.wallet-engine:Info:135]\u001b[0m [2023-09-24 09:53:04.74 UTC] 9e5295f3: Syncing (36.70% percent) Applied 6000 blocks, 0 rollbacks in the last 30.068146537s. Current tip is [point 3c4ac2fb at slot 10602566]."}

Running cardano-wallet-v2023-07-18-linux64.tar.gz, this line above appears just before shutdown once the 'in sync' state is determined. It's only at 36% due to the few times the archival node as been rebooted and is able to increment some bit before shutting back down.
Here are also some pool-engine messages around the same time:
{"MESSAGE":"\u001b[35m[cardano-wallet.pools-engine:Notice:27]\u001b[0m [2023-09-24 09:49:58.83 UTC] In sync! Applied 2267 blocks, 0 rollbacks in the last 30.013552159s. Current tip is [point 0ef01649 at slot 28892968]."}

Noticeably the ready state is available when the pool-engine sync completes before the wallet-engine?
So what is "cardano-wallet.wallet-engine:Info:135" doing?

These are the logs I receive while all my wallets are status ready as well as network/information endpoint is ready/in sync.
{"MESSAGE":"\u001b[34m[cardano-wallet.wallet-engine:Info:135]\u001b[0m [2023-09-24 09:53:04.74 UTC] 9e5295f3: Syncing (36.70% percent) Applied 6000 blocks, 0 rollbacks in the last 30.068146537s. Current tip is [point 3c4ac2fb at slot 10602566]."}

Hm. 🤔 This log message is created by a wallet whose walletId starts with 9e5295f3. Can you locate this wallet in one of the following listings?
GET /wallets
GET /byron-wallets
GET /shared-wallets

These listings also include the "state" field for each wallet. Could you filter the response and post here the "id", "state", and "tip" fields of the JSON object corresponding to the wallet whose "id" starts with 9e5295f3, as well as a recent log message from this wallet?

My apologies, I think this was a logic/bug error on my end where there was an edge case where it returned true anyway if the node was fully synced but the wallet wasn't.
Seems to work fine after adjusting my code with latest release, the state is not 'ready' immediately after creating the wallet anymore.
Closing.


What is this project?

I'm very curious. Can you tell us a little bit about what this project is?
Looks like a tool to configure opsmgr + tiles via externalized git config?

Going to close this, but I very much hope someone here will create a README with a little snippet on what this project is, it's intended use, use-cases, problems it solves, etc.
Thanks, take care--


Broken out for SVG with PNG fallback (Multiple background-image)

A variant of this issue was originally reported in https://github.com/jakubpawlowicz/clean-css/issues/371.
IN
foo {
  background: url("foo.png");
  background: none, url("foo.svg");
}

OUT
foo{background:0 0,0 0/auto,auto repeat,repeat none,transparent}

Switching the position of none (as reported in https://github.com/jakubpawlowicz/clean-css/issues/371) fixes the problem.
IN
foo {
  background: url("foo.png");
  background: url("foo.svg"), none;
}

OUT
foo{background:url(foo.png);background:url(foo.svg),none}

I'm using v3.2.5 the following options
{
      processImport: false,
      rebase: false,
      restructuring: false,
}

Good catch :+1:

@xzyfer It's fixed in 3.2.7.

@jakubpawlowicz thanks!


Support for binary serialization with arrays of non variable length

When a structure contains an array with more than one item, the types of the array should be exactly the same as the array from the structure.
For example, right now a structure like [StorageType.INT32] means it is an array of numbers with variable length. But a structure like [StorageType.INT32, StorageType.STRING, StorageType.BOOL] should expect an array like [1, "hello", true]. If it receives an array with a different length or different types, it should throw.
There are currently no unit tests for this case and I believe the implementation isn't working either.

The implementation is indeed broken because this fails:
Deno.test({
	name: "Array of numbers of fixed length",
	fn() {
		basicObjectToBinaryToObjectTest({
			array1: [1, 2],
			array2: [1, 2, 3],
		}, {
			nameIds: {
				array1: 1,
				array2: 2,
			},
			structure: {
				array1: [StorageType.UINT8, StorageType.UINT8],
				array2: [StorageType.UINT8, StorageType.UINT8, StorageType.UINT8],
			},
		});
	}
})


Unable to handle incorrect deprecated field marking in version 2.5.4

Summary
We have been using Android Apollo version 1.2.2 for a while , bumping version to 2.4.5 produces issue where deprecated field is not handled correectly.
Version
2.4.5
Description
We have fragment like this which gets parsed fine in version 1.2.2
"""
Attachable entities in a widget."
@deprecated Widget: To get widget details use query Widget#newWidgetDetails instead.
"""
union WidgetAttachment = File | ImageFile | VideoFile | WebImage | WebLink | WebVideo | Message

But if I change fragment to be defined as follows everything works
"""
Attachable entities in a some widget details. @deprecated(reason: 'Widget: To get widget details use query Widget#newWidgetDetails instead').
"""
union WidgetAttachment = File | ImageFile | VideoFile | WebImage | WebLink | WebVideo | Message

Seems like with version 2.4.5 we need some specific format to parse deprecated comment?
The error I get while compiling code is
warning: [dep-ann] deprecated item is not annotated with @Deprecated
Note, I have tried to use following directives in apollo sections of my gradle without any effect.
apollo {
    warnOnDeprecatedUsages.set(false) // true by default
    failOnWarnings.set(false) // true by default
}

I am not 100% sure if this is a bug, if someone can provide pointer to formal GQL schema grammer I would check.

Hi! Thanks for sending this! You can find the GraphQL grammar there: https://spec.graphql.org/draft/#UnionTypeDefinition
In this case, it looks like our SDL parser doesn't like the quote embedded in the first description.
In all cases, I'm surprised this has ever worked. Directives and deprecated annotations should be put outside the description:
Something like:
"""
Attachable entities in a widget.
"""
union WidgetAttachment @deprecated(reason = "Widget: To get widget details use query Widget#newWidgetDetails instead.") = File | ImageFile | VideoFile | WebImage | WebLink | WebVideo | Message

Also double checking, 1.2.2 was using the npm codegen, I'm not 100% sure how it was finding the schema but on 2.4.5, you should name your SDL schema schema.sdl for it to be parsed as SDL

This should work fine in latest versions, closing.


Add errors, retries and retry_rate to cvs output.

Rebased and squashed https://github.com/swiftstack/ssbench/pull/106

That looks good. Thanks for merge it.


hybrid grid in 3D cartographer

whether it is 3D grid map in 3D cartographer about the hybrid grid of submap? and scan matching occurs between the 3d laser fan and the 3d grid?but why do you display a 2d submap list in rviz subcribe a service of submap_query?

2D is easier to understand and make sense of than 3D voxels. So we flatten the submap into a single texture (akin to the X-Ray visualization) and ship that to rViz.

Can I understand that the existence of 3D grids is the need for scan matching methods in 3D cartographer,and 2D grid is for 2D laser scan matching, not for other reasons?

I do not fully parse the question.

Can I understand that the existence of 3D grids is the need for scan matching methods in 3D cartographer

that is definitively correct.

,and 2D grid is for 2D laser scan matching

that is also correct.

not for other reasons

Is this part of the question if the HybridGrid and ProbabilityGrid are used for other stuff? If so, the HybridGrid is used in X-Ray generation as well, maybe other places. But it is mostly for scan matching.


User-provided constraints for the FFAdd gate (iteration 2)

[ ] expose Expr::PolishToken as a type through the bindings.

[ ] Figure out how to get  #[cfg_attr( feature = "ocaml_types", derive(ocaml::IntoValue, ocaml::FromValue, ocaml_gen::Enum) )] to work with enums that have constructor arguments
[ ] Add decl_type!(w, env, GateType => "custom_gate_type"); to main.ml

[ ] Create ScalarsMinus
[ ] replace the flag with an Option<Expr::PolishToken>, but just use it as a flag (i.e.Option.is_some everywhere we need the boolean), and propagate it through kimchi and pickles.
[ ] write an interpreter for PolishToken in OCaml, panic on the complex stuff (caching, conditionals, etc.).
[ ] Update uses of Sc.constant_term and ft_eval0 to take an optional custom_gate_type RPN argument and internally call the interpreter to evaluate the RPN on the witness to return a field element (similar to what's happening in evaluate here https://github.com/o1-labs/proof-systems/blob/6233800e9a7dfb031145225492733fc5969e785a/kimchi/src/circuits/expr.rs )
[x] write a converter PolishToken -> Expr in rust (so that we can use it in the prover), similarly TODOs on complex stuff
[ ] get Kimchi side working with user-supplied gate defintion
[ ] Use the polish notation form in kimchi and pickles as the custom gate expression, make the test pass
[ ] demo several different gates with degree at most 4 (as in FFAdd)

Interesting (1-on-1) meeting (with Matthew) decision: include powers of alpha inside user supplied RPN gate definition.


backend error with case destructuring

This type-checks:
    String[2] tup = ["one", "two"];
    switch(tup)
    case ([a,b]) { String x = b; }

but results in
source/simple/run.ceylon:7: error: compiler bug: unhandled declaration unknown with type UnknownType at com.redhat.ceylon.compiler.java.codegen.AbstractTransformer.makeTypeTest(AbstractTransformer.java:4538)
    switch(tup)
    ^
1 error
1 warning

@gavinking I assume this is a backend bug and not a typechecker bug?

Well, yeah, I suppose. Though I'm not sure that it was really 100% intentionally intended that this would work, so I'm not surprised that it wasn't tested. Still, I see no good reason to actively prevent you from writing that.

OK, so what's the thing that makes this special? I fail to see it. Apparently I have an IsCase with a iscase.type that is an UnknownType. I assume the typechecker should infer that type to be what's in iscase.variable, no?

That's similar to what I though (https://github.com/ceylon/ceylon.ast/issues/116#issuecomment-228130549), the ast seems incomplete

Whatever, I use the type of the Variable, that works.


Failing to establish a connection to a 32 bit driver

OS - Windows 10, Powershell
DB - Actian SQL
I'm getting an error message from the ODBC Driver manager indicating that the node-odbc driver is attempting to make connections using 64 bit ODBC. Unfortunately the database driver for my database only supports 32 bit.
I did attempt running powershell and cmd as 32 bit but was still unsuccessful
Are there any examples for connecting to 32 bit?
'use strict'

const { series } = require('gulp')
const odbc = require('odbc')

const connection = async done => {
  try {
    console.log('connecting to ODBC driver')
    const conn = await odbc.connect('DSN=SOME_ODBC_NAME')
    console.log('connection established')
    // close connection
    conn.close(msg => {
      console.log('closed connection')
      done()
    })
  } catch (err) {
    console.log('error', err)
  }
}

module.exports = series(
  connection
)

PS C:\Users\alilland\Documents\dev\forecast-scraper> yarn run devGulp other:odbc                                                                                yarn run v1.19.1
$ set NODE_ENV=development& gulp other:odbc
[13:50:41] Using gulpfile ~\Documents\dev\forecast-scraper\gulpfile.js
[13:50:41] Starting 'other:odbc'...
[13:50:41] Starting 'connection'...
connecting to ODBC driver
error [Error: [odbc] Error connecting to the database] {
  odbcErrors: [
    {
      state: 'IM014',
      code: 0,
      message: '[Microsoft][ODBC Driver Manager] The specified DSN contains an architecture mismatch between the Driver and Application'
    }
  ]
}
[13:50:41] Finished 'connection' after 4.71 ms
[13:50:41] Finished 'other:odbc' after 6.71 ms
Done in 1.21s.
PS C:\Users\alilland\Documents\dev\forecast-scraper>

And you are sure your DSN is defined in the 32-bit ODBC Data Sources program (and not the 64-bit one)?
From reading, it also seems that the entire application stack needs to be in 32-bit, so are you running a 32-bit Node.js process as well? I don't know if there is anything inherent in the odbc package that is 64-bit, but if you confirm that none of the above issues are causing your problem, I can dig a little deeper.

@markdirish it looks like I was calling node-odbc with a 64 bit node stack, I downgraded to 32 bit and was able to successfully establish a connection. It looks like I have a new issue, but I will go ahead and open that up under a different thread. thanks!


Fix to gocloud/compute/ec2/region.go

In this, only one region (USEast) is defined although there are many more in AWS. More diverse regions' information (sa-east-1, eu-west-2, etc.) should be included as other variables so that it is convenient for people to use these prepared constants rather than instantiating a region with the information each time they would like to reference it.

@rehrumesh @PratikDhanave
Hello,
I would like to work on this issue.
I think we can add all the elements of this list here.

hi pranjal we are passing this as  parameter


Exclude both replies and reweets

Is there a way to exclude (eg. in t timeline username) both retweets AND replies at the same time? I could only ever get it to do one or the other and wondered if it could be done, or that feature be added?

I think you can do this easily with grep:
t timeline @username -l | cut -d ' ' -f9- | grep -v -E '^(RT|@)'

Takes in the list format, displays the tweet column, removes those that start with RT and those that start with @. You are left with only with tweets that start with something else. If you need to get the time stamp, or other columns, you can do it but then you have to use a little more complicated grep.

There used to be a --no-replies and --no-retweets pair of flags, but they seem to be non-functional atm. I would like this back in also - any plans for this @sferik ?
Trying to pipe a List to espeak and there's a lot of noise around the list from RTs/replies. Also ideally it wouldn't exclude retweets from list members, but that's just being picky!


Make clone + compile easier

Based on #52, decided to spend some time based on getting this down.
From a fresh install of Go, the main thing working against this seems to be the sqlite lib we're using. It's super slow unless you go and actually 'install' it (rather than just getting the source code as you do with everything else).
Evaluate the solution used in #52 vs adding a command line example to the readme going over install + run of this.

Getting rid of sqlite has really helped with this.

Maybe we should also add oragono as an actual binary to the bin folder, not sure what you need to do or how you need to package things to get go to do that. With something like that we could have, say... oragono start, which just runs through the basic setup stuff.

I see this is opened and things have changed a lot in the last 1.5 years.
The main README.md is outdated for the make targets due to the use of gorelease and that can be updated. Having just done a few things myself in the project it's still a pain to setup some of the vendoring which is just sort of a go-ism as it is today (mainly as it relates to the packages within this repository itself). None of this is a non-starter for someone new.
I would argue that if the README.md was updated for current build process and just indicated what was required this would be 'resolved". You already (awesomely) release builds for a majority of OSes that one could want and I don't think many people are chasing the latest version rather than devs and we can generally handle it if the steps are known :)

Nice, I hope it's a fair bit better than it was at least! Pushed a few changes today which should make it reflect reality. If there's any sorta trouble with the new instructions, feel free to yell and we can edit them up a bit more :)


Update figma.mdx with theme deprecation files content

Partially closes #3644
Updates content in various places on the website to reflect the deprecation of our Figma theme files.
*Note: Need to update various links to the "All themes" Figma file (previously known as White theme) once the file is published. It may change the URL name.

Changed
Content across these pages:

Designing > Design Kits (Figma tab) page.

I need a dev to help me to change the two images on this page.


Added elixir code

Is this fine?

I think a solution like has already been submitted :/
Please check https://github.com/NLDev/Hacktoberfest-2020-FizzBuzz/tree/master/Elixir to see if it's a duplicate.
Keep in mind: The goal is to find creative/abstract new solutions to FizzBuzz. Not normal standard solutions.
See The top 5 samples here.
Cheers!

Ok that's why i sent a draft, i will close this request, sorry I will try another method.

aight!

can you now check once?

I am learning this language xD

yep! merging now :)

Thanks :)

Forgot to add my name as author, it is fine right?

Added it for you:
https://github.com/NLDev/Hacktoberfest-2020-FizzBuzz/blob/master/Elixir/FizzBuzz.ex#L1

Thanks!


Release v0.14.0

What this PR does / why we need it:
Special notes for your reviewer:
Does this PR introduce a user-facing change?

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Comparison is base (b9810d9) 28.74% compared to head (bfd8ba9) 28.74%.

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@           Coverage Diff           @@
##             main    #1335   +/-   ##
=======================================
  Coverage   28.74%   28.74%
=======================================
  Files          34       34
  Lines        3079     3079
=======================================
  Hits          885      885
  Misses       2077     2077
  Partials      117      117

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


strip: Parameter properties assignments are written before super()

Input code
class A extends Object {
  constructor(public a, private b) {
    super();
  }
}

Output code
class A extends Object {
    constructor(a, b){
        this.a = a;
        this.b = b;
        super();
    }
}

The constructor at runtime:
Uncaught ReferenceError: Must call super constructor in derived class before accessing 'this' or returning from derived constructor

Seen in https://github.com/denoland/deno/blob/v1.2.2/std/io/bufio.ts.
cc @bartlomieju

Fixed by publising new version of swc_ecma_utils.


QueryComplexity does not work with ES Modules (ES2020, Typescript)

Hi. I have my codebase in Typescript and had queryComplexity setup in my project some time ago. It worked well when my codebase was transpiled to commonjs and I was targetting ES2017
But recently I made the move to use ES2020 with Node.js modules (type:"module" in package.json`) and I get this error:

This is how I am importing (as in the README):
import queryComplexity, { directiveEstimator, simpleEstimator } from 'graphql-query-complexity';
And my validationRules look like this (from the README):
validationRules: [
            queryComplexity({
              estimators: [
                directiveEstimator({
                  name: 'complexity'
                }),
                simpleEstimator({
                  defaultComplexity: 1
                })
              ],
              maximumComplexity: 500,
              variables: variables,
              onComplete: (complexity: number) => {
                console.log('Query Complexity:', complexity);
              },
              createError: (max: number, actual: number) => {
                return new GraphQLError(`Query is too complex: ${actual}.
Any ideas what I might be doing wrong? Thanks.

I tried working around like this, but even this did not work:
import { directiveEstimator, QueryComplexityOptions, simpleEstimator } from 'graphql-query-complexity';
import QueryComplexity from 'graphql-query-complexity/dist/QueryComplexity.js';

function queryComplexity(options: QueryComplexityOptions): (context: ValidationContext) => QueryComplexity {
  return (context: ValidationContext): QueryComplexity => {
    return new QueryComplexity(context, options);
  };
}

It said QueryComplexity is not a constructor.

Here is a workaround:
import gqlQueryComplexity from 'graphql-query-complexity';
type gqlQueryComplexityFn = typeof gqlQueryComplexity;
const queryComplexity = (
    gqlQueryComplexity as unknown as { default: gqlQueryComplexityFn }
).default;

This should be fixed with the latest release of v0.9.0. The package now includes an ESM build as well as a commonJS build.
There is now also a named export for createComplexityRule. I updated the READMEs accordingly.


Playground: hook up more menu pinning functionality

Add ability to pin/unpin more menu items to the main navbar pinned group

https://github.com/user-attachments/assets/6b352579-8d22-418e-ad7f-b0b8dd70e2fc

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.


Return location header in created response and implemented API for promotion results

Location header will be added to artifact and promotion result created response. Other
REST APIs updated as defined in issue 95
References #77

:+1:


Proposal: Move gocat to its own repository

I have been working on some changes to gocat to support v4 and now looking at v5 and I think it might be helpful to have a separate repository just for it. Any interest in pull it out similar to the UI repository?

I'd prefer to keep it in this repo if it works for you. If you're working on a fork, just make sure all the work is in a branch named feat/update_hashcat_to_v5 and we can PR it in here when you're done.
Is there major changes in v5 that really breaks things?

Have not gotten too much into it yet so not sure. I have v4 working pretty well in my local tests and looking to use gocat to replace my current plugin for hashcat in github.com/jmmcatee/cracklord. Thanks for the consideration.

Let me think about it some more and see what it'd take. When this was an internal FireEye project, it was actually separate and would certainly make it easier to support other versions of Hashcat.

Supporting multiple versions was definitely one of the reasons as I'm not sure it will be easy to have one version that supports multiple hashcat versions, but my familiarity with cgo is limited.

@jmmcatee We will be moving gocat into it's own repository that will make it easier for contributions and use in other open source software.
It'll live in https://github.com/fireeye/gocat


Picture-glance: Consistency for entities opening more info dialog

Right now clicking entities in picture-glance can produce unexpected results. Most of toggle enabled entities trigger on/off, but some sensors open more info dialog.
However there are entities that trigger on/off even if they should not allow that, but are more suitable for more info dialog. Examples: climate, alarm, etc.
Suggestion: Either disallow entities that don't have binary state or on/off from the card, or allow for more info dialog based on domain and document this in the entity.

https://github.com/home-assistant/home-assistant-polymer/pull/1339


Server Crash with Enderio at refinedstorage.tile.TileSolderer$3.insertItem(TileSolderer.java:50)

See issue here -
https://github.com/SleepyTrousers/EnderIO/issues/3713
Version (Make sure you are on the latest version before reporting):

Minecraft: 1.10.2
Forge: 2092
Refined Storage: refinedstorage-1.0.4

Does this issue occur on a server? [yes/no]
Yes
If a (crash)log is relevant for this issue, link it here:
See Enderio github issue.

The crash log tells me nothing about what happened, so I'm not sure what I can do about this


Errors should be prioritized above warnings

Continuing from https://github.com/atom-community/linter/issues/653

Done!

:clap:


Import functions from the ilab repository

Some functions that are being called in the generate_data.py file are in this file - https://github.com/instructlab/instructlab/blob/main/src/instructlab/utils.py

@aakankshaduggal @russellb is this something we want to try and preserve commits for? if not I'm happy to take this (and the other issue I guess, seems one PR can resolve both)

ack @russellb - fine either way - AFAIK the code in question is only used in the CLI currently

ack @russellb - fine either way - AFAIK the code in question is only used in the CLI currently

question is it called from only code other than what got moved here

Okay looked into this a bit - we import from two locations within instructlab - config and utils
From config, we import the following:

DEFAULT_MULTIPROCESSING_START_METHOD

Used in lab.py but also this is just a variable

DEFAULT_API_KEY

Used in many places but also this is just a variable

DEFAULT_MODEL_OLD

Used in lab.py but also this is just a variable

get_model_family()

Used by generate_data.py but also server.py

From utils, we import the following:

chunk_document()

Only used by generate_data.py

max_seed_example_tokens()

Only used by generate_data.py

num_chars_from_tokens()

Only used by generate_data.py

read_taxonomy()

Used by generate_data.py but also lab.py

get_sysprompt()

Used by generate_data.py but also lab.py, chat.py and make_data.py

You can write rules for Ruff or PyLint to detect these types of imports and raise an error. InstructLab uses Ruff for that:
[tool.ruff.lint.flake8-tidy-imports.banned-api]
"yamllint".msg = "yamllint is for CLI usage only."

This issue is a soft blocker for #1471 in the instructlab CLI.

It looks like DEFAULT_MULTIPROCESSING_START_METHOD, DEFAULT_API_KEY, and DEFAULT_MODEL_OLD are no longer present in SDG, I'll mark those as complete for this epic.

The change to remove the last import from the main instructlab package is here: https://github.com/instructlab/sdg/pull/89


config: faster confirmation link expiration

Fixed test where endpoint was registered after the first request
Cleanup unused declarations
Updated assets build for example app
Tested expiration links for password recovery and email verification
Closes #269

This pr is cleaned up to address only the config issue on #279


Trim Fleuntd tag for Cloud Logging

Just like the changes made for Elasticsearch logging, the PR trims the tag to contain only the pod name and the container name. This tag shows up under Resource ID in the Cloud Logging console. @mr-salty

cool - LGTM

LGTM, thanks for adding the explanation!


MiniGPT-4 support?

According to the readme, MiniGPT-4 is supported. But I don't see any code/example/API for it, only for Vicuna?

OK, will try once https://github.com/jina-ai/opengpt/issues/75 is resolved.


ENYO-1382 & ENYO-1383: Updated ExpandableListItem and LabeledTextItem…

… font sizes to match UI spec.
Also did some cleanup and organization of the LESS files
Created mod-able variables to augment the values easier (useful for a fonts future update).
Enyo-DCO-1.1-Signed-off-by: Blake Stephens<EMAIL_ADDRESS>

Deleting a branch will do that - let me spin another PR...


Slack: Screensharing broken (4.33.84/4.33.90)

Describe the bug
Try to screenshare in a huddle - when clicking screensharing icon slack crashes. Applied Option B of the mesa-fix already, tried with/without hardware-acceleration, with/without --disable-gpu-flag, with/without wayland-flags.
Steps To Reproduce
Steps to reproduce the behavior:

Open Huddle in a channel
Try to screenshare
Slack will crash (because of segfault)

Expected behavior
Screensharing working
Additional context
Coredump: https://pastebin.com/FmLv4yEZ
Notify maintainers
@mmahut
@maxeaubrey
Metadata
Please run nix-shell -p nix-info --run "nix-info -m" and paste the result.
[user@system:~]$ nix-shell -p nix-info --run "nix-info -m"
 - system: `"x86_64-linux"`
 - host os: `Linux 6.5.1, NixOS, 23.11 (Tapir), 23.11.20230910.dirty`
 - multi-user?: `yes`
 - sandbox: `yes`
 - version: `nix-env (Nix) 2.17.0`
 - nixpkgs: `/nix/store/jy6rzyvpsx43x0k0vwk9faymw4aa2y0m-source`

greetings I just tried screensharing in slack  4.34.115 64-bit ( master )
worked for me
nix-info -m
 - system: `"x86_64-linux"`
 - host os: `Linux 6.1.52, NixOS, 23.11 (Tapir), 23.11.20230912.35e3b6f`
 - multi-user?: `yes`
 - sandbox: `yes`
 - version: `nix-env (Nix) 2.17.0`
 - channels(kirillvr): `""`
 - channels(root): `"nixos"`
 - nixpkgs: `/nix/var/nix/profiles/per-user/root/channels/nixos`

also this is using wayland

Can confirm, cherry-picking this commit to my nixpkgs-unstable checkout solves the issue. Would leave it open for others having problems until it lands in unstable - just side node (not relevant for me) can't open Help -> About anymore after update (when I wanted to check version)

@SebTM do you mean Help -> About Slack ?  this also works for me

@SebTM do you mean Help -> About Slack ? this also works for me

Correct, I can see the following in logs:
[09/23/23, 15:56:57:752] info: Store: CREATE_WINDOW {
  "type": "about-box"
}
[09/23/23, 15:56:57:752] info: Store: CREATE_WINDOW {
  "type": "about-box"
}
[09/23/23, 15:56:59:191] info: Store: CREATE_WINDOW {
  "type": "about-box"
}
[09/23/23, 15:56:59:191] info: Store: CREATE_WINDOW {
  "type": "about-box"
}

Tried multiple times to open it and can see this events but main windows is not blocked (still usable) and no new window can be found.

Fixed in latest release, Backport for it coming up so going to close it. #257149


ArmNN prebuild version

Could you release a prebuild version ??

Hi @jnulzl we're planning to do this by providing Debian and Ubuntu packages. The first version going through the Debian testing system can be seen here: https://piuparts.debian.org/sid/source/a/armnn.html

Hi @jnulzl we're planning to do this by providing Debian and Ubuntu packages. The first version going through the Debian testing system can be seen here: https://piuparts.debian.org/sid/source/a/armnn.html

Nice！Thanks！


Fixes minor errors in DG

Codecov Report

Merging #279 (d90b148) into master (7e616b8) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master     #279   +/-   ##
=========================================
  Coverage     66.68%   66.68%
  Complexity      676      676
=========================================
  Files           107      107
  Lines          2203     2203
  Branches        259      259
=========================================
  Hits           1469     1469
  Misses          629      629
  Partials        105      105

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 7e616b8...d90b148. Read the comment docs.


Make sure our templates compile

The best thing to do here would be to have a separate build workflow for them given they only require pulumi and should work in a clean env without any packages published locally.

Yes, also for the upcoming examples.
The question is do we want to trigger templates and examples compilation on every PR or nightly?

Depends on what resources we have for GHA I guess. Could start with a
requirement that a PR can't be merged into develop without last commit
passing that workflow, mostly because it will be quite a long running job.
On Mon 25. Sep 2023 at 09:23, Paweł Prażak @.***> wrote:

Yes, also for the upcoming examples.
The question is do we want to trigger templates and examples compilation
on every PR or nightly?
—
Reply to this email directly, view it on GitHub
https://github.com/VirtusLab/besom/issues/155#issuecomment-1733068723,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ACBVNUXXGZY32YXXN4TUVGDX4EWOVANCNFSM6AAAAAA5CXXD3I
.
You are receiving this because you authored the thread.Message ID:
@.***>

I believe we already test compilation on both, templates and examples.
What is missing is automatic addition to the test target.


Unset saved errors on a POST

If a user submits an invalid input, then tries again with an input that triggers an exit page then on going back, the previous error still exists on the session and so is displayed. Reset any saved errors on any POST to ensure that previous POST's errors are removed.

:+1:


Running erc20 script fails

Running yarn erc20 script fails with the following error:
yarn run v1.22.22
warning ../package.json: No license field
$ ts-node src/main.ts erc20
************* Onboarding user  0xe04f0C7eF0d73a935E19Db3AcB8f3Ae006478491  *************
************* Onboarded! created user key and saved into .env file *************
Running erc20 example...
/Users/daniel/projects/coti-sdk-typescript-examples/node_modules/ethers/src.ts/utils/errors.ts:694
            error = new Error(message);
                    ^
Error: could not decode result data (value="0x", info={ "method": "balanceOf", "signature": "balanceOf()" }, code=BAD_DATA, version=6.12.1)
    at makeError (/Users/daniel/projects/coti-sdk-typescript-examples/node_modules/ethers/src.ts/utils/errors.ts:694:21)
    at assert (/Users/daniel/projects/coti-sdk-typescript-examples/node_modules/ethers/src.ts/utils/errors.ts:715:25)
    at Interface.decodeFunctionResult (/Users/daniel/projects/coti-sdk-typescript-examples/node_modules/ethers/src.ts/abi/interface.ts:916:15)
    at staticCallResult (/Users/daniel/projects/coti-sdk-typescript-examples/node_modules/ethers/src.ts/contract/contract.ts:346:35)
    at processTicksAndRejections (node:internal/process/task_queues:95:5)
    at async staticCall (/Users/daniel/projects/coti-sdk-typescript-examples/node_modules/ethers/src.ts/contract/contract.ts:303:24)
    at async Proxy.balanceOf (/Users/daniel/projects/coti-sdk-typescript-examples/node_modules/ethers/src.ts/contract/contract.ts:351:41)
    at async erc20Example (/Users/daniel/projects/coti-sdk-typescript-examples/src/examples/erc20.ts:36:30)
    at async main (/Users/daniel/projects/coti-sdk-typescript-examples/src/main.ts:16:5) {
  code: 'BAD_DATA',
  value: '0x',
  info: { method: 'balanceOf', signature: 'balanceOf()' },
  shortMessage: 'could not decode result data'
}
error Command failed with exit code 1.
info Visit https://yarnpkg.com/en/docs/cli/run for documentation about this command.

cc @gmesika-coti

issue was with the address of the pre-deployed contracts, they were pointing to the wrong addresses (prior to the reset of the devnet), fixed in commit d707987158fc9b70dbec103e78e517314997435d


Automated Package Upgrade

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Revive "iterate away from official plan" script

For Nick's paper, specifically

Need to redo this as 100 x 100, once we resolve the repetition issue.


UITextField different textRect for editing and static text.

Hi, i am have issue with custom UITextfield which has different textRect and editingTextRect.
In UITextField+NUI.m found override_editingRectForBounds
  - (CGRect)override_editingRectForBounds:(CGRect)bounds {
     return [self textRectForBounds:bounds];
}

Seems to be it is a bug, i guess method implementation should be:
 - (CGRect)override_editingRectForBounds:(CGRect)bounds {
    return [self override_editingRectForBounds:bounds];
}

Good catch. If you submit a pull request we'd be happy to review it. If you could throw in some test cases as well that would be superb.


Sets default HTTP_PORT and HTTPS_PORT and bumps version to 1.9.1

Apologies.  I forgot to add the lines that set these defaults.

Thanks. Released in 1.9.1 - In general you shouldn't bump the version in PRs, thats my job :)


Add functions for dataframe: median, mean, min, max, sum

Hello @Rubtsowa! Thanks for opening this PR. We checked the lines you've touched for PEP 8 issues, and found:

In the file sdc/datatypes/hpat_pandas_dataframe_functions.py:

Line 47:5: E303 too many blank lines (2)
Line 70:121: E501 line too long (122 > 120 characters)
Line 108:121: E501 line too long (132 > 120 characters)
Line 128:5: E303 too many blank lines (2)
Line 176:5: E303 too many blank lines (2)
Line 224:5: E303 too many blank lines (2)
Line 272:5: E303 too many blank lines (2)
Line 320:5: E303 too many blank lines (2)

In the file sdc/hiframes/pd_dataframe_ext.py:

Line 1641:1: E402 module level import not at top of file

Hello @Rubtsowa! Thanks for opening this PR. We checked the lines you've touched for PEP 8 issues, and found:

In the file sdc/datatypes/hpat_pandas_dataframe_functions.py:

Line 47:5: E303 too many blank lines (2)
Line 70:121: E501 line too long (122 > 120 characters)
Line 108:121: E501 line too long (132 > 120 characters)
Line 128:5: E303 too many blank lines (2)
Line 176:5: E303 too many blank lines (2)
Line 224:5: E303 too many blank lines (2)
Line 272:5: E303 too many blank lines (2)
Line 320:5: E303 too many blank lines (2)

In the file sdc/hiframes/pd_dataframe_ext.py:

Line 1641:1: E402 module level import not at top of file

Please fix the code style issues.