Build failing after upgrading

After updating to the newest version, my webpack build is failing. Any ideas?
Module build failed: ReferenceError: Unknown plugin "transform-object-rest-spread" specified in "/Users/satazor/Work/npms/npms-www/node_modules/react-gravatar/.babelrc" at 0, attempted to resolve relative to "/Users/satazor/Work/npms/npms-www/node_modules/react-gravatar"

You need to exclude node_modules from babel_loader.
On Mon, Jul 4, 2016 at 5:10 PM André Cruz<EMAIL_ADDRESS>wrote:

After updating to the newest version, my webpack build is failing. Any
ideas?
Module build failed: ReferenceError: Unknown plugin "transform-object-rest-spread" specified in "/Users/satazor/Work/npms/npms-www/node_modules/react-gravatar/.babelrc" at 0, attempted to resolve relative to "/Users/satazor/Work/npms/npms-www/node_modules/react-gravatar"
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/KyleAMathews/react-gravatar/issues/61, or mute the
thread
https://github.com/notifications/unsubscribe/AAEVh9scVYXT74XMTU8ODQ_533g7wTP0ks5qSaD1gaJpZM4JEslj
.

I've fixed it, thanks for the tip.

Please write  how did u fixed this?

@mtetno if you're seeing the same error, make sure your webpack is setup the same as here: https://github.com/babel/babel-loader#usage

Specifically the exclude key.


Question: Can you link a form button to the tour next button?

I have set up the bootstrap tour framework on my website which works great.
At the moment I have 2 pages which the tour needs to traverse. On page one the last step you need to submit the form which saves the information and takes you to page 2.
Is there a way to submit the form when user click next on the tour and incremented the step by 1 if the user clicks the submit button on the page.
I hope this makes sense.

You'll need to write your own JavaScript code using tour.next() to go to next step after submit.

Thanks this wast the same solution I went for.


Lost all tables after removing snapshot bucket

A long story, please bear with me.
I have a self-hosted instance of grist as a container. I had setup document snapshots with a self-hosted minio container. It worked fine but I ran into issues once I added a region identifier(just a dummy name) in the minio console. This then prevented grist from starting as it was throwing a S3Error: The authorization header is malformed; the region is wrong exception.
I then decided to remove the Minio instance to set it up from scratch and deleted the snapshots bucket as well. My assumption was I could start the snapshots in Grist afresh with the new Minio instance and a new access key. But after this action, grist did not show my  pages and tables, instead offered me a recovery option. Unfortunately, I did not this option and instead restarted my grist container. Now while the parent document exists, none of the pages and tables are shown. I just have 1 table with the default columns. I checked if disabling snapshots using the env variable would help, but no luck.
Any method of recovering my tables and data?

Eek, sorry to hear this! Snapshots are more than for history -- when set up, the long-term storage you configure (like S3 or MinIO) becomes the authoritative repository of Grist documents, and the local disk of a server becomes the transient cache. Generally that's what you want, because you can stop and start a server (or even multiple), and all your documents are there because they come from the long-term storage.
That does mean it's important to have a backup of any important documents you have on a server before you start experimenting with snapshot configuration, because you are basically telling Grist that what's on the server is an unimportant temporary cache, while what's in S3 / MinIO is what matters.
(Don't punish yourself for not using the recovery option; I don't think it would've helped anyway if Grist didn't find your data. I think Grist is a bit too willing to start with an empty document when it can't find a document, instead of showing an error... This might be inherited from sqlite's behavior.)
I don't know the details of what happened in the scenario you describe. The typical server setup persists documents in /tmp/grist. Start by checking if there is anything there. Back up anything you find. Your data may still be there. You can check any .grist file using sqlite3 (e.g. sqlite3 FILENAME.grist 'SELECT tableId FROM _grist_Tables' will show all user tables in the document; if you only see Table1, then it's probably an empty one). If you find data you care about, you can upload any document like that to any running Grist installation via the browser, by importing from your Grist home page.

Thanks for the inputs. Lesson learned!!
There is no folder grist under /tmp. But I found .grist files which have the tables and data. I think I can salvage something from this. Is there any way to get hold of the formulas or the raw python code?

The .grist files have everything, including formulas! It's the complete document. See this: https://support.getgrist.com/exports/#restoring-from-backup

Phew! The document was just a little older than the set of last major changes I made. I have gotten back 95% of my document. Thanks a lot for your help! I will close this issue.


Question: Partition Resolution

Hello,
I am new to GraphQL, so apologies if this question is missing the mark or if I am misusing any terms.
I have objects which are partitioned by a hash of the object ID. There is a service for each partition. There is a gateway service which is responsible for routing our current REST requests to the correct partitions and aggregating results.
I would like to add a GraphQL endpoint hosted on the Gateway. I would like to also host a GraphQL endpoint on each partition, and have the gateway service delegate portions of the overall GraphQL query to the correct partition, then merge the results. I am not sure if this is possible, or how to do this?
For example, if the gateway gets the query:
{
  objects(ids: [1, 2, 3, 4]) {
    name
    child_object {
      name
    }
  }
}

I would like the gateway to call the GraphQL route for each partition and combine the results:
Query to partition 1:
{
  objects(ids: [1, 2]) {
    name
    child_object {
      name
    }
  }
}

Query to partition 2:
{
  objects(ids: [3, 4]) {
    name
    child_object {
      name
    }
  }
}

I have seen documentation regarding stitching together schemas of different types, so it seems that it is possible to delegate portions of the GraphQL query to different services. However this seems to be done on the type level, but in this case the partitions have the same types.
I have also seen how I could use Resolvers and DataLoaders to batch and request the items from each partition using our existing REST calls. However this again seems to be on the type level, and I have not seen whether it is possible to "forward" the GraphQL child-query in a resolver, or merge results.
Thank you!

I do see a lot of injectable information in the resolver docs, but am not yet sure the best approach to use them, or if doing this on the resolver level is a good idea.

I guess there is not a good way to do this? The reason I would prefer to forward the graphql sub-query, as opposed to calling partition services via REST inside the gateway data loader, is the similar to the reasons to do schema stitching when types are different - fewer network calls, no need to maintain REST routes, and transmitting only the needed data.

You could write a simple middleware for this like we did with schema stitching. The middleware would have the split logic and merge the results back together.

Thanks Michael, that is good to know. I am not as familiar with the internals of the library as you are, would the middleware to look at for inspiration be DelegateToRemoteSchemaMiddleware ? Any other types to be aware of?

And would it be correct that this would be a query middleware?

Have you done our graphql-workshop already? It goes into many aspects.
https://github.com/ChilliCream/graphql-workshop

Thanks Michael, I had not done the workshop. When I have some time I will look through the workshop. Feel free to close this if you want to since it seems to have come to a short term resolution: if someone wants to do this they should create custom middleware, and the workshop can provide inspiration on how to do so.


Extended Flywheels 1.2.5-1.19.2-0.2.f crashing with Create 0.5.1a

Here I leave you the crash report.
Details:
Mod File: /C:/Users/educl/curseforge/minecraft/Instances/Twitch/mods/extendedflywheels-1.2.5-1.19.2-0.5.f.jar
Failure message: Extended Flywheels (extendedflywheels) has failed to load correctly
	java.lang.NoSuchMethodError: 'com.tterrag.registrate.util.nullness.NonNullSupplier com.simibubi.create.foundation.data.CreateRegistrate.lazy(java.lang.String)'
Mod Version: 1.2.5-1.19.2-0.5.f
Mod Issue URL: NOT PROVIDED
Exception message: java.lang.NoSuchMethodError: 'com.tterrag.registrate.util.nullness.NonNullSupplier com.simibubi.create.foundation.data.CreateRegistrate.lazy(java.lang.String)'

Stacktrace:
at com.rabbitminers.extendedflywheels.ExtendedFlywheels.<clinit>(ExtendedFlywheels.java:31) ~[extendedflywheels-1.2.5-1.19.2-0.5.f.jar%23628!/:1.2.5-1.19.2-0.5.f] {re:classloading}
at jdk.internal.reflect.NativeConstructorAccessorImpl.newInstance0(Native Method) ~[?:?] {}
at jdk.internal.reflect.NativeConstructorAccessorImpl.newInstance(NativeConstructorAccessorImpl.java:77) ~[?:?] {}
at jdk.internal.reflect.DelegatingConstructorAccessorImpl.newInstance(DelegatingConstructorAccessorImpl.java:45) ~[?:?] {}
at java.lang.reflect.Constructor.newInstanceWithCaller(Constructor.java:499) ~[?:?] {}
at java.lang.reflect.Constructor.newInstance(Constructor.java:480) ~[?:?] {}
at net.minecraftforge.fml.javafmlmod.FMLModContainer.constructMod(FMLModContainer.java:68) ~[javafmllanguage-1.19.2-43.2.8.jar%23871!/:?] {}
at net.minecraftforge.fml.ModContainer.lambda$buildTransitionHandler$10(ModContainer.java:121) ~[fmlcore-1.19.2-43.2.8.jar%23870!/:?] {re:mixin}
at java.util.concurrent.CompletableFuture$AsyncRun.run(CompletableFuture.java:1804) ~[?:?] {}
at java.util.concurrent.CompletableFuture$AsyncRun.exec(CompletableFuture.java:1796) ~[?:?] {}
at java.util.concurrent.ForkJoinTask.doExec(ForkJoinTask.java:373) ~[?:?] {}
at java.util.concurrent.ForkJoinPool$WorkQueue.topLevelExec(ForkJoinPool.java:1182) ~[?:?] {}
at java.util.concurrent.ForkJoinPool.scan(ForkJoinPool.java:1655) ~[?:?] {re:mixin,re:computing_frames}
at java.util.concurrent.ForkJoinPool.runWorker(ForkJoinPool.java:1622) ~[?:?] {re:mixin,re:computing_frames}
at java.util.concurrent.ForkJoinWorkerThread.run(ForkJoinWorkerThread.java:165) ~[?:?] {re:mixin}

all Create addons will be broken with 0.5.1 for at least a few days

all Create addons will be broken with 0.5.1 for at least a few days

I know, just wanted to post it in case no one does.

Is there a release with this fixed? Just was trying to update my pack and ran into this.

Is there a release with this fixed? Just was trying to update my pack and ran into this.

Nope, the mod didn't get an update since I post it here. We have to wait.

I thought not, it was confusing to see this issue closed as completed when the problem wasn't actually fixed

I was trying my hand at updating it and I think the major roadblock is that the Create version of the mod isn't at 0.5.1 yet


Update CM TCK tests for coordination

ref: #474

@stbischof Thanks for updating the tests; I think those two tests are good with the proposed changes. However, we also need to adjust the existing tests:

testCoordinatedConfigurationOnBeforeRegisteredManagedService is currently checking for four events to happen; however with the proposed changes, it will just be one (the first one invoking the managed service with null). During the coordination the configuration is created, updated and then deleted, resulting in no invocation of the managed service
testManagedService - similar here, the configuration is created, updated, deleted - managed service will only be invoked with "null" once
I have the feeling that we then have an overlap between those two new tests and the two that need adjustment; but that's probably ok.

I've created #505 which superceeds (includes) this PR


TC Login 07 - The email can be pasted in the "Email" field

Description : In the searching filed "Email", when logging in, the users can paste the e-mail .
Preconditions : The website validator (https://admin.peviitor.ro/login) is up and running

Step 1 : Open the validator.
Expected results : The window "Conectare" and the  field "Email" can be seen

Step 2 : Copy your email adress and paste it inside the "Email" field.
Expected results  : The adress have been pasted.


Exclude does not work correctly on constrained type parameters

TypeScript Version:
<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>
Search Terms:
exclude extends omit
Code
type Primate = 'monkey' | 'human';

function zee<P extends Primate>() {
    const a: Exclude<P, 'human'> = 'monkey';
}

Expected behavior:
No error
Actual behavior:
Type '"monkey"' is not assignable to type 'Exclude<P, "human">'.
Playground Link:
https://www.typescriptlang.org/play/#src=type Primate %3D 'monkey' | 'human'%3B

function zee<P extends Primate>() {
    const a%3A Exclude<P%2C 'human'> %3D 'monkey'%3B
}

Related Issues:

I believe this is working correctly.  If P is never or "human", then indeed "monkey" is not assignable to Exclude<P, "human">.


Wrong file extension in the example in "usage instructions"

Hey, this is superb.
I spotted that the example command uses the song extension .xrns rather than .xrnc.
python generate_theme.py /path/to/image/file --saturation 1.5 --theme_style base --output custom_theme.xrns

Thanks for the catch! I'll make a correction shortly.


chore: update issue templates according to mo

Signed-off-by: Aylei<EMAIL_ADDRESS>This PR update the issue templates according to matrixorigin/matrixone.
Another option is to use an org default template according to https://docs.github.com/en/communities/setting-up-your-project-for-healthy-contributions/creating-a-default-community-health-file, but I'd copy and modify these now since I don't know how general they are.

/merge


Revisit outcomes in Pinned Dependencies probes

https://github.com/ossf/scorecard/pull/3829 needs a follow-up to get the outcomes and the evaluation scores/errors precise as mentioned by @spencerschrock here: https://github.com/ossf/scorecard/pull/3829#discussion_r1472000487.

[ ] https://github.com/ossf/scorecard/pull/3829#discussion_r1472000487
[ ] https://github.com/ossf/scorecard/pull/3829#discussion_r1496616001
[ ] https://github.com/ossf/scorecard/pull/3829#discussion_r1496604686

At least one more TODO: https://github.com/ossf/scorecard/pull/3961#discussion_r1543793323


Drag and drop from export history

Fixes #602
I can not get https://electronjs.org/docs/api/native-image#nativeimagecreatefromdataurldataurl to work with the kapture image, any ideas why or how to get the correct image when dragging?
Do we want the screenshot or a static image for all dragged files?

Do we want the screenshot or a static image for all dragged files?

Screenshot

Closing in favor of #641.


UnstructuredFileLoader loads wrong "page number" metadata of word documents

Checked other resources

[X] I added a very descriptive title to this issue.
[X] I searched the LangChain documentation with the integrated search.
[X] I used the GitHub search to find a similar question and didn't find it.
[X] I am sure that this is a bug in LangChain rather than my code.
[X] The bug is not resolved by updating to the latest stable version of LangChain (or the specific integration package).

Example Code
from langchain_community.document_loaders import UnstructuredFileLoader
import json
import os
msFiles = ["BV24-006-0_BV_Laufbahnmodell.docx"]
for filename in msFiles:
loader = UnstructuredFileLoader(filename, mode='elements')  # elements | single
docs = loader.load() # Document
Convert each document to a dictionary
data = [doc.dict() for doc in docs]
print(f"document will be serialized with {len(data)} elements!")
with open(f"{filename}.txt", 'w', encoding="utf-8") as f:
json.dump(data, f, ensure_ascii=False)
Error Message and Stack Trace (if applicable)
No response
Description
The page number of returned list of documents is wrong:
View of DOC File - page 55 of 70

View of parsed Document:  page 16

System Info
windows
python 11

I am reinsured: it's Langchain's bug rather than the code

This is a question of semantics about what a page number means.
In most extraction applications, one keeps track of the index of the page (which is what the page number you're seeing likely corresponds to) and there's a separate metadata field for the page number as is visible on the page.
The former is needed by code (e.g., to be able to scroll to the correct page in the PDF), the latter is needed by humans.

semantics ?.  what a page number means ?.    The page number is clear in context of the ms word document. The Document has 70 pages and the user can navigate in Word by pages. If he goes to the page number 12, the editor show him the page number 12 and show also the number there. if the user navigates to the last page, word show  him the page number 70...and so on. If the user wants to reuse the page_number as source (I think it's the sense of that flag), then the page_number must be selecteed properly for all "loaded/parsed pages".
the UnstructuredFileLoader class selects properly the page_number metadata for the first 30 pages (which are equivalent to the ms word pages). After the number page 30 or 33 the page_number metadata will not be more incremented.  So my guest is that UnstructuredFileLoader has a bug by parsing the Word Document.
Best Regards,


Issue/4337 scenario4 (#4537)

Pull request opened by the merge tool on behalf of #4537

Merged into branches iso5 in ffa4beb6bb425ac99908e9e2dca3f4f3b8f22533


Bump to 9.0.0 for all projects

Fixes #693

Ack @christianacca about this

Hi @tomascassidy, what's the motivation for bumping all the libraries? I'm sure there are good reasons, but technically I don't think it's required.

There was a breaking change in the MembershipReboot package for 9.0.0 changing GetCurentPrincipal to GetCurrentPrincipal.  The other packages just need to be recompiled to be compatible with that naming change and bumping the version number seemed the easiest option.
The current situation is described in #693 where the MembershipReboot 9.0.0 package is not compatible with MembershipReboot.Webhost 7.1.0.

Ah ok -- sorry, I had forgotten about the rename issue. I'll try to get these to nuget soon.

Just checking when BrockAllen.MembershipReboot.Ef version 9.0.0 nuget will be published. Thanks

I think I just successfully pushed them -- please test? Thx

I see a new version on nuget. Thanks for a quick turnaround!


Fix tests

This fixes the tests and updates a Django 1.10 incompatibility.
Two tests (test_logout and test_logout_service_local) are still failing due to, what I think is a bug in pysaml2 (See https://github.com/rohe/pysaml2/pull/363)

First of all, Thank you for doing this. This project has not seen working tests in a long time which is a huge problem, especially now when people start to send PRs more. So I'm happy that you took this challange :-)
Now, as I said, I have never seen the tests work in this project and did not try to run them since I joined. So, could you please provide also some requirements and steps how to run them? Just executing tox in the directory does not seem to work for me, but maybe I'm doing something wrong...

Could you provide the error you are getting? Except for the above-mentioned tests running 'tox'  should work. I'm not at the machine which I developed this on right now, so I can' t verify if I missed a dependency or a file.

Arghh, seems my tox cache was just broken. Alright, works now (except for those two tests). Thanks flupzor!

Awesome! And thank you for your fast review :)


Regressions, confusion, and docs

I've been avoiding upgrading due to the syntax changes and finally did today and have noticed quite a few issues.

746b35b7034f9ed74d0b3fa839480c3dac0d50fa commit message has radius is obsolete. Looking though the code of kwm-overlay radius is still very much consumed by kwm-overlay. Why was this removed??
The cycle-focus screen bug is back. It cycles across displays again.

The new docs are very confusing, and even incomplete in some areas.
 Manage window-tree
        kwmc tree <opt>
        <opt>: rotate <arg1> | save <arg2> | restore <arg2>
        <arg1>: 90 | 180 | 270
        <arg2>: filename

This syntax for the docs is incredibly difficult to parse. I can't see any reason to organize it like this. arg1 and arg2 are very confusing, since they are always actually arg2
 Rotate the windows in the current focused space clockwise
        kwmc tree rotate <arg>
            <arg>: 90 | 180 | 270
Save the current window size layout of the focused space
       kwmc tree save <arg>
            <arg>: filename
Restore the saved window size layout to the focused space
        kwmc tree restore <arg>
            <arg>: filename

Something like that is much easier to read and parse as an end user of the tool. I understand it's more verbose, but Manage window-tree doesn't mean a whole lot to an end user.
swap -window is missing mark
node -window is confusing. An end user does not understand what a "node" is or that it relates to a root node of a binary space partition tree.
I don't understand the need for the - anymore in the commands (for example focus -window). Before it made a bit more sense, as they visually appeared as flags. Is there something that makes the - necessary and not focus window?
node -window vs focus -sub-window. Again, "node" is confusing with out any explanation. "node" is pretty ambiguous. Then the node is referred to as a -sub-window` with no indication that they are the same thing.
Still going though my kwmrc file and updating it to the new syntax I'll report anything else I find here. It's been a little bit frustrating upgrade process so far.

The new command syntax is very much still in progress. I'm in the first iteration, where I simply wanted to port the existing functionality to a different format. The interpreter as it has been used previously will be removed, as I'm thinking going to be writing a proper parser/pre processor.

The radius value has not been affecting how the border was drawn, ever since that commit that added InsetRect.

This is not a true statement.
Top screenshot is a radius of 8. Bottom screenshot is a radius of 18. I reverted to the commit right before you removed support for the radius. I urge you to please revert that change.

I would suggest you remain at the 1.1.5 release as well, unless there has been a fix since that you really need.

I really don't mind living on the bleeding edge, it allows me to catch things like the radius issue and help report bugs during refactors. I'm just reporting my findings from after the syntax change from an end user perspective in hopes to help shape the kwm user experience.

The cycle-focus regression I am referring to is #233

Top screenshot is a radius of 8. Bottom screenshot is a radius of 18. To test this I reverted to the commit right before you removed support for the radius. I urge you to please revert that change.

Pretty sure that it made no impact for me. I will however, revert that change.

I really don't mind living on the bleeding edge, it allows me to catch things like the radius issue and help report bugs during refactors. I'm just reporting my findings from after the syntax change from an end user perspective in hopes to help shape the kwm user experience.

This is appreciated, however there are huge changes coming, just figured I should let people know.

there are huge changes coming

:+1:  excited to test them :)

Regarding kwmc commands, I want to have something like the following in the end:
kwmc [-flags] <opt> <val> | kwmc <opt> <val> [-flags]

Does that imply that flags would be able to be used together? Currently no flags can be used together, correct?
If that is the goal, do you have examples of what flags might be able to be used in a single command?

Does that imply that flags would be able to be used together? Currently no flags can be used together, correct?

That's one of the things I'm hoping to achieve, yeah.

do you have examples of what flags might be able to be used in a single command?

There are tons of use cases really, but a specific case could be to focus some window and move it to a space on some display.

That sounds awesome! Excited to see that develop.

Currently I'm thinking something like this:
e.g:
kwmc f -w $win_id focus a window
kwmc f -s $space_id focus a space
kwmc fmm -wds $win_id $display_id $space_id focus a window, move it to diff display and space
opt:
f: focus
m: move
s: swap
z: zoom
r: refresh
flags:
w: window
d: display
s: space
Again, I'm really not sure of the best approach myself yet.

I'm kinda curious, is chaining operations like the above example a bad idea?
e.g: kwmc fmm -wds $win_id $display_id $space_id
The command is chaining three operations:

perform a focus command of type -w | --window
perform a move command of type -d | --display
perform another move command of type -s | --space

Is this a bad approach or something I can keep iterating on?

is chaining operations like the above example a bad idea? ... Is this a bad approach or something I can keep iterating on?

I most certainly think that it is not a bad idea, I do however think it would make kwm a lot more complicated when it is already having a little trouble with regressions.
I am curious what the style of chaining you suggest might offer that custom shell commands are not able to offer.
For example a
mycommand.sh
that contains
marked=$(kwmc query current)
kwmc focus --window $marked
kwmc move --display $1
kwmc move --space $2

kwmc bind prefix-v sys mycommand 1 3
It's much more verbose, but keeps the chaining in terms of simple kwmc commands, reducing the complexity.

I am curious what the style of chaining you suggest might offer that custom shell commands are not able to offer.

No difference really that I can think of. It's just less verbose, and doesn't require a shellscript every time.

Yes, the old syntax did look a little nicer for sequential commands. My opinion is that streamlining the shell script system would offer a little more power, while keeping the  overall complexity relatively low.
If we could somehow figure out an easier way to supply kwmc to the shell scripts, or even better the users $PATH with out having to resort to asking users to set up a launch service to set it that would make the shell scripts even more useful. Right now there is a bit of boilerplate that could be removed that might make the shell scripts more useful.

Yes, the old syntax did look a little nicer for sequential commands. My opinion is that streamlining the shell script system would offer a little more power, while keeping the overall complexity relatively low.

Guess I'll just go for this for now, and revert back to the old command style, as that gives me more time to work on the window rule system.

Commands now follow the old pattern, but the provided flags should make more sense.
There have changes to the kwmc config focus commands as well.
I'll leave it like this and if/when I decide to change things I'll do them on a separate branch at the start.


Couldn't find Enum for XMLHttpRequest.ReadyState

I expected an Enum for ReadyState and couldn't find it. Can you please help me on this ?
This also brings me to a question that "Do we have any api reference ?" How do i know which Namespaces, Types, Classes, Interfaces and Enums are available Out-Of-Box, what do they do and how they are related to the JS ?

@RyanCavanaugh I mean to say "If not XMLHttpRequest.readyState exists, please add it to the in-built types. Also please create an API reference document, so far i can see only a HandBook available to get started".


Variable names which are not yet in CF

How best to test for model variable names not yet in CF tables

These can be just added to variables.py with the netCDF variable name (and units, etc.).


Fix root cause for special result row set not updating when flags are updated

As described in #1375, we need to properly maintain the result set. Go through the API endpoints and ensure the following. Described for flags, but also applies for comments and acmg ratings.

on creation of flag

ensure that an according entry exists in the result set
override the result set entry flag-related properties with flag, save

on update of flag

ensure that an according entry exists in the result set
override the result set entry flag-related properties with flag, save

on deletion of

fetch according entry of result set
check if there are any acmg annotations or comments
if this is not the case, delete the entry
otherwise, clear all flag-related properties and save the entry

RCA is not correct. The actual root cause it that the flags, comments and acmg ratings for a case were hold in the variant store and variants in the case result set were matched to the flags/comments/acmg ratings by release and coordinates. The matching was broken because genomeBuild was used instead of release.


how to set postion of the window?

Is there a way to change the position of the window. For example, I'd like to make it in the center of the screen. Thank you!

Thanks for your feedback. But unfortunately our upstream project, webview, does not support this feature, so we can't do that. This repo is only a Node.js binding for webview. If you need new features, you could send you feedback to webview. I would be happy to try making API bindings after webview adding them.

You may see this feature request in  webview: https://github.com/webview/webview/issues/642, in which it mentioned a method to center the window using C++, webview-nodejs support setting the target window handle. So if you can create a centered window, and pass the handle to webview-nodejs by using target parameter in new Webview(debug?: boolean, libPath?: string, target?: null), it may work.


Object Detection: Treat FPN post-NMS-top-N as parameter scalable by batch size

The reference merges all proposed bounding boxes from all feature levels and images in the batch into a single tensor before applying a top-N. As the batch-size per GPU changes, the ratio of kept vs. discarded proposed boxes changes (Example: batch 2, 20k proposals over all feature maps and images, take 2k = 10% kept. Batch 1, 10k proposals over all feature maps, 2k kept = 20%. Similarly for b4 we'd keep 2k of 40k = 5%).
Changing this parameter to be a function of batch size per GPU would allow these ratios to remain constant.
Relevant code from the reference: https://github.com/ddkang/Detectron/blob/benchmarking/lib/ops/collect_and_distribute_fpn_rpn_proposals.py#L69-L86
I propose that we allow scaling this number of merged proposals taken by batch size (using 2000 proposals kept at batch 2 as the reference). Thus batch would keep 1k proposals, batch 4 would keep 4k)

This has long been a confusing trick in detectron. In maskrcnn-benchmark they kept the same behavior. I think what you propose here requires the deep learning system have the capability to store and pass these RoIs between batches? As the RoIs get accumulated, isn't it a overflow since every RoI would produce a RoI align feature map?

Look at it offline.

Duplicate.


Hero - Add variation on product pages

Add a variation "has-product" on all product pages (Except calcium) for both languages.
Reference: https://main--fonterra-fernleaf--hlxsites.hlx.live/en/products/yogurt

Updated all product pages. Please validate.

Looks good, closing this card


kvm: Add Machine field to the template of qemuSwtpm

Machine field is now used to choose the QEMU's vtpm driver, so it must be added to the qemuSwtpm template. It fixes the following error:
"failed to build domain config: can't write to config file /run/domainmgr/xen/xen1.cfg (template: qemuSwtpm:11:10: executing "qemuSwtpm" at <.Machine>: can't evaluate field Machine in type struct { CtrlSocket string })"

Sorry for merging the buggy code before =D

All good @OhmSpectator , this was my fault, I forgot to put this code when suggesting to @shjala the checkings through template....


Non linear registration

Investigate FNIRT (or ANTs) for nonlinear MR registration

or, better yet, also compare https://github.com/neurodata/ndreg

@dmannan and @loftusa consider this.
first, make note of all the registrations that we perform.
then, get input from @ebridge2 as to which he believes we should replace, and why.
then, try the ones he thinks we should replace, and see how it goes.


Program chairs that are also authors can see their reviewer assignment

I logged on as Pascal Bercher and looked at one of his papers - he can see the "ICAPS 2019 Workshop Hierarchical Planning Paper Assignment" for his paper which is displayed right below the anonymized reviews.

Are you talking about the matching browser?

No paper assignments show up on the forum page itself if you are a program chair.

I see, yes, that is data that comes from the matching system. May be we need to set the Author group as nonreaders but if he decides to run the matching, this papers won't get any assignment because the matching system is being run with his permissions.

Can we just not show it on the forum page?

may be, but he can still see them in the matching browser


14 build action

Note: I use assembleDebug here rather than buildDebug because linting fails. A future iteration should use buildDebug once linting issues are solved.


[Do not merge] Debug build for appveyor

Coverage remained the same at 81.783% when pulling 9c03e27d314a7650985e48908dfec1a1201dd542 on barrbrain:appveyor-debug into 3dcb26667f98a014d582dac40b7dcb65291f3425 on xiph:master.


Dev Test: Number Play 1.1.0-dev.10

Dev Test
Mentions: @amanda-phet @zepumph @chrisklus @Nancy-Salpepi @kathy-phet @KatieWoe
This is the first dev test since 1.0 was published in spring of 2022, but please note that https://github.com/phetsims/qa/issues/917 has some overlap with Number Play. Thanks all!
Simulation links

github repository for issues
phet top-level directory
sim: all_phet.html

Test Matrix
Please include all (non-screen reader) feature testing in these records if applicable.

[x] Latest macOS, Safari (Time = 3.5hrs)
[ ] Windows 10, Chrome (Time = 1.5)
[x] Windows 10/11, Firefox (Time = 1.5)
[x] Latest iOS, Safari (Time = 3.5)
[x] Windows 11, Chrome (Time =4.5)

Light testing, or optionally skip if time crunch:

[x] Latest macOS, Chrome (Time =1.75hrs)
[x] Latest Chrome OS, Chrome (Time = 1hr)

Features included

Speech Synthesis (not voicing, but still speaks to you).

Focus and Special Instructions

Please test the locale settings and number speaking quite extensively, as we have made made changes here even since the number-compare dev.6 dev test.
Please note that there are lots of great cases and examples of imperfection with ordering, layering, and hiding objects especially on the lab screen. Feel free to make GitHub issues for this kind of thing, but please don't spend an exorbitant amount of time here, as there is a good chance that, unless it poses serious concern pedagogically, that we do not have the bandwidth to fix it.
The Lab screen is the exact same code as Number Compare except that this Lab screen doesn't have compare symbol cards.
The Game screen has not changed much since 1.0 - the main change is that there is a reward node now.

Issues to Verify
These issues should have the "status:ready-for-review" label. Unless an issue says to close after verifying, assign the
issue back to the developer.

[x] https://github.com/phetsims/number-compare/issues/32

For QA...

General features

What to Test

Click every single button.
If there is sound, make sure it works.
Make sure you can't lose anything.
Play with the sim normally.
Try to break the sim.
Try to include browser version numbers
If there is a console available, check for errors and include them in the Problem Description.
Run through the string tests on at least one platform, especially if it is about to go to rc.
Check the Game Up harness on one platform.

Accessibility features

What to Test

Specific instructions can be found above.

Make sure the accessibility (a11y) feature that is being tested doesn't negatively affect the sim in any way. Here is
a list of features that may be supported in this test:

Alternative Input
Interactive Description
Sound and Sonification
Pan and Zoom
Mobile Description
Voicing

Test all possible forms of input.

Test all mouse/trackpad inputs.
Test all touchscreen inputs.
Test all keyboard navigation inputs (if applicable).
Test all forms of input with a screen reader (if applicable).

Magnification
This sim supports magnification with pinch and drag gestures on touch screens, keyboard shortcuts, and mouse/wheel controls. Please test magnfication and make sure
it is working as intended and well with the use cases of the simulation. Due to the way screen readers handle user input, magnification is NOT expected to
work while using a screen reader so there is no need to test this case.

FAQs for QA Members

There are multiple tests in this issue... Which test should I do first?
Test in order! Test the first thing first, the second thing second, and so on.

How should I format my issue?

We typically assign the developer who opened the issue in the QA repository.

My question isn't in here... What should I do?
You should:

Consult the QA Book.
Google it.
Ask Katie.
Ask a developer.
Google it again.
Cry.

Calling testing here

Thanks you all!! I'm going to go ahead and close.


API updates to the AnimationPlayer

Objective
Added AnimationPlayer API UX improvements.

Succestor to https://github.com/bevyengine/bevy/pull/5912
Fixes https://github.com/bevyengine/bevy/issues/5848

Solution

Created RepeatAnimation enum to describe an animation repetition behavior.
Added is_finished(), set_repeat(), and is_playback_reversed() methods to the animation player.

To determine if an animation is finished, we use the number of times the animation has completed and the repetition behavior. If the animation is playing in reverse then elapsed < 0.0 counts as a completion. Otherwise, elapsed > animation.duration counts as a completion. This is what I would expect, personally. If there's any ambiguity, perhaps we could add some AnimationCompletionBehavior, to specify that kind of completion behavior to use.
Migration Guide

No breaking changes.

@DevinLeamy the conflicts aren't trivial; could you resolve them so we can re-review and merge?

@DevinLeamy the conflicts aren't trivial; could you resolve them so we can re-review and merge?

@alice-i-cecile I'm not seeing any conflicts?

Updated to incorporate the changes from https://github.com/bevyengine/bevy/pull/5912.

Made the animation clip optional as per the comment from added some simple functions to AnimationPlayer #5912

I don't think the handle should be an option

Unsure why CI is failing. Seems unrelated to the changes, as alien_cake_addict.rs does not use the AnimationPlayer.


Bump to miniconda version 23.10.0

Summary

This PR does two things:

Update the miniconda version to 23.10.0
Update the configuration so that the script can be properly run

The install script has now a new check that tests that the first parameter ends in .sh to ensure it explicitly gets called through bash/zsh/sh/etc.
Fixes #262
Use Cases

Use of the latest version of miniconda available at this time
Checklist

[x] I have viewed, signed, and submitted the Contributor License Agreement.
[x] I have linked issue(s) that this PR should close using keywords or the Github UI (See docs)
[x] I have added an integration test, if necessary.
[x] I have reviewed the styleguide for guidance on my code quality.
[x] I'm happy with the commit history on this PR (I have rebased/squashed as needed).

Since #497 was merged with the work from this PR, I am closing it.


Calling len() on the scipy.spatial.transform.rotation.Rotation object with one rotation should return 1

When calling the len() function on the scipy.spatial.transform.rotation.Rotation object with exactly one rotation, it currently raises a TypeError instead of returning 1.
(Link to source code)
Reproducing code example:
from scipy.spatial.transform import Rotation as R

r = R.from_quat([1, 0, 0, 0])
print(len(r))    # raises TypeError

Error message:
TypeError                                 Traceback (most recent call last)
<ipython-input-227-c5f861eb1f8a> in <module>
      4 try:
----> 5     print(len(r))
      6 except Exception as e:

rotation.pyx in scipy.spatial.transform.rotation.Rotation.__len__()

TypeError: Single rotation has no len().

Calling len() on an object with 2 rotations returns 2 as expected:
r = R.from_quat([[1, 0, 0, 0],
                 [1, 0, 0, 0]])
print(len(r))    # returns 2

Moreover, when creating a rotation object using the from_euler() class method, it correctly returns 1 for one rotation around a single coordinate axis, but raises the above error again for a rotation around more than one coordinate axes:
r = R.from_euler('x', [0.1])
print(len(r))    # returns 1

r = R.from_euler('xz', [0.1, 0.1])
print(len(r))    # raises TypeError

Scipy/Numpy/Python version information:
import sys, scipy, numpy; print(scipy.__version__, numpy.__version__, sys.version_info)
1.6.0 1.19.5 sys.version_info(major=3, minor=9, micro=1, releaselevel='final', serial=0)

I think the answer to this issue is in https://github.com/scipy/scipy/pull/12398

I think the answer to this issue is in https://github.com/scipy/scipy/pull/12398

This still works this way and seems inconsistent. I don't immediately see how gh-12398 answers whether this is intended or not - looks like a bug to me.

Based on the comments in https://github.com/scipy/scipy/issues/16663 (https://github.com/scipy/scipy/issues/16663#issuecomment-1194117213 in particular) I think this is intended behavior.  r = Rotation([1, 0, 0, 0]) is like a scalar rotation, so the object has no length, and len(r) should raise a TypeError.  r2 = Rotation([[1, 0, 0, 0], [0, 1, 0, 0]]) is like a sequence of two rotations, and len(r2) is 2.
If my interpretation is correct, we can close this issue.


Porting the circular buffer challenge.

You might want to make an entry for this problem in config.json as well.

Thanks k4rtik! I added that change.

Thanks! Would you mind squashing your commits into a single commit?
I'm leaving this for @LegalizeAdulthood to review and merge.

Squashed as per kytrinyx. Thanks for taking a look.

Can you rebase onto the latest and resolve the conflicts?  I suspect it is because I accepted a PR that reordered the exercises: https://github.com/exercism/xcpp/pull/82


WIth the limitation of homes, allow the first home to be named "bed" if it's available.

This home does not impact the number of homes that you have available but instead is a built-in home that works only when a bed has been placed by you. You can also implement a maximum of one bed and whether or not other players can use or destroy that bed. This would be a very useful feature for friendly survival servers like mine that just want the players be able to spawn at their base without being liable for any damages to their bed.

User places the bed down.
Home named bed is created in Essence homes list. Nobody can interact with the bed placed by the user, nor can it be destroyed by grief.
User destroys the bed.
Home named bed is removed.

The functionality to prevent bed griefing can be a configurable feature along with whatever else you find configurable.

I'll have to have a look into how this will work, could take some time. Thanks!


ARROW-4501: Fix out-of-bounds read in DoubleCrcHash

Not if the memory allocation is larger than the actual string, I suppose.

possibly what happened here, if there's an extra null byte.

Rebased.


About the uspto50k P2R model checkpoint

Dear authors,
Thanks for sharing the code, and some checkpoints about this great work. l am wondering how you trained these checkpoint (i.e., models/P2R/USPTO_50K_P2R.pt), via 1) pretrain-finetune, or 2) train from scratch?

I noticed that in the Readme, there are two modes available for users for training a model checkpoint.

This checkpoint is obtained by pretrain-fintune formula.
However, if obtained by train from scratch, you should be able to get a close prediction result.

Thank you! may l ask which pretrained checkpoint you were using? since I observed that the file size of the shared pretrained checkpoints is ~500MB, but the fine-tuned checkpoint here is only ~200MB, which is not consistent with the checkpoint size

Thank you! may l ask which pretrained checkpoint you were using? since I observed that the file size of the shared pretrained checkpoints is ~500MB, but the fine-tuned checkpoint here is only ~200MB, which is not consistent with the checkpoint size.

I apologize for the confusion.
The checkpoints in the "pretrained" folder are all pre-trained, while the remaining checkpoints are trained from scratch.
If you are looking to obtain fine-tuned checkpoints, it may require you to perform the fine-tuning by yourself.
Upon checking my local repository and disk storage, it appears that the fine-tuned checkpoints have been inadvertently lost or misplaced.
I apologize for any inconvenience this may have caused you.

Thank you for your detailed reply, and now l know where the checkpoints come from exactly. May I ask one more question about the role of pretraining in R-SMILES? If the models by "training from scratch" are very close to that by "pretrain-fintune formula", why we need pretraining?

Thank you for your detailed reply; now I understand how the checkpoints are trained in detail. May I ask one more question regarding the role of pretraining in R-SMILES? If the models trained from scratch are very close to those trained using the pretrain-finetune formula, what is the point of pretrainig models here?
Thanks

This is a good question. It can be attributed to two main points:
(1) It does indeed bring about certain performance improvements. In our local experimental environment, the model trained from scratch achieved top-1 and top-10 accuracy rates of 55.0% and 89.8% respectively, while the model fine-tuned from pretraining achieved top-1 and top-10 accuracy rates of 56.3% and 91.0% respectively;
(2) It significantly reduces the training time on large datasets. Training a model from scratch on large datasets like uspto-full is very time-consuming, whereas the training time for fine-tuning based on a pretrained model is much less.

Since the questioner has not asked any further questions for a long time, I believe it is appropriate to close this issue.


Add testing of EpiAware to CI

This PR adds unit testing as a CI action. It also reorganises the .toml files to follow the recommended Julia style of having a nested test environment for all testing dependencies.
This PR has been locally tested using act and passes all checks.
_Note: The use of a special temporary environment when tests are called may not work correctly with caching. However, I think resolving this is probably not worth the lift if we can instead wait until julia-actions are enabled.

act is failing for me.
So the first failure running act locally from bash with act --container-architecture linux/amd64
[Test EpiAware/test-EpiAware] ⭐ Run Main actions/cache@v3
[Test EpiAware/test-EpiAware]   🐳  docker cp src=/Users/samandfi/.cache/act/actions-cache@v3/ dst=/var/run/act/actions/actions-cache@v3/
[pre-commit/pre-commit      ]   🐳  docker exec cmd=[node /var/run/act/workflow/hashfiles/index.js] user= workdir=
[Test EpiAware/test-EpiAware]   🐳  docker exec cmd=[node<EMAIL_ADDRESS>user= workdir=
[Test EpiAware/test-EpiAware]   💬  ::debug::Resolved Keys:
[Test EpiAware/test-EpiAware]   💬  ::debug::["julia-1.10.0-310dd69f39ec4ba5882e8507dd218d1d143ca697aa9af89d76e91f0e8ce4ce4f"]
[Test EpiAware/test-EpiAware]   💬  ::debug::Checking zstd --quiet --version
[Test EpiAware/test-EpiAware]   💬  ::debug::1.4.8
[Test EpiAware/test-EpiAware]   💬  ::debug::zstd version: 1.4.8
[Test EpiAware/test-EpiAware]   💬  ::debug::Resource Url: http://<IP_ADDRESS>:50477/_apis/artifactcache/cache?keys=julia-1.10.0-310dd69f39ec4ba5882e8507dd218d1d143ca697aa9af89d76e91f0e8ce4ce4f&version=3971121c6fff6d6a8e35c7a73fb7ab8e8701d96e94026f2b0cd51c32618257da
[Test EpiAware/test-EpiAware]   💬  ::debug::Failed to delete archive: Error: ENOENT: no such file or directory, unlink ''
| Cache not found for input keys: julia-1.10.0-310dd69f39ec4ba5882e8507dd218d1d143ca697aa9af89d76e91f0e8ce4ce4f

then we get some SEGFAULTS
[Test EpiAware/test-EpiAware] ⭐ Run Main actions/cache@v3
[Test EpiAware/test-EpiAware]   🐳  docker cp src=/Users/samandfi/.cache/act/actions-cache@v3/ dst=/var/run/act/actions/actions-cache@v3/
[pre-commit/pre-commit      ]   🐳  docker exec cmd=[node /var/run/act/workflow/hashfiles/index.js] user= workdir=
[Test EpiAware/test-EpiAware]   🐳  docker exec cmd=[node<EMAIL_ADDRESS>user= workdir=
[Test EpiAware/test-EpiAware]   💬  ::debug::Resolved Keys:
[Test EpiAware/test-EpiAware]   💬  ::debug::["julia-1.10.0-310dd69f39ec4ba5882e8507dd218d1d143ca697aa9af89d76e91f0e8ce4ce4f"]
[Test EpiAware/test-EpiAware]   💬  ::debug::Checking zstd --quiet --version
[Test EpiAware/test-EpiAware]   💬  ::debug::1.4.8
[Test EpiAware/test-EpiAware]   💬  ::debug::zstd version: 1.4.8
[Test EpiAware/test-EpiAware]   💬  ::debug::Resource Url: http://<IP_ADDRESS>:50477/_apis/artifactcache/cache?keys=julia-1.10.0-310dd69f39ec4ba5882e8507dd218d1d143ca697aa9af89d76e91f0e8ce4ce4f&version=3971121c6fff6d6a8e35c7a73fb7ab8e8701d96e94026f2b0cd51c32618257da
[Test EpiAware/test-EpiAware]   💬  ::debug::Failed to delete archive: Error: ENOENT: no such file or directory, unlink ''
| Cache not found for input keys: julia-1.10.0-310dd69f39ec4ba5882e8507dd218d1d143ca697aa9af89d76e91f0e8ce4ce4f
[Test EpiAware/test-EpiAware]   ✅  Success - Main actions/cache@v3
[Test EpiAware/test-EpiAware]   ✅  Success - Main ./.github/actions/install-julia
[Test EpiAware/test-EpiAware] ⭐ Run Main Run unit tests for EpiAware
[Test EpiAware/test-EpiAware]   🐳  docker exec cmd=[bash --noprofile --norc -e -o pipefail /var/run/act/workflow/2] user= workdir=
[pre-commit/pre-commit      ] ⭐ Run Main actions/cache@v3
[pre-commit/pre-commit      ]   🐳  docker cp src=/Users/samandfi/.cache/act/actions-cache@v3/ dst=/var/run/act/actions/actions-cache@v3/
[pre-commit/pre-commit      ]   🐳  docker exec cmd=[node<EMAIL_ADDRESS>user= workdir=
[pre-commit/pre-commit      ]   💬  ::debug::Resolved Keys:
[pre-commit/pre-commit      ]   💬  ::debug::["julia-1.10.0-310dd69f39ec4ba5882e8507dd218d1d143ca697aa9af89d76e91f0e8ce4ce4f"]
[pre-commit/pre-commit      ]   💬  ::debug::Checking zstd --quiet --version
|
| [111] signal (11.1): Segmentation fault
| in expression starting at none:0
[pre-commit/pre-commit      ]   💬  ::debug::1.4.8
[pre-commit/pre-commit      ]   💬  ::debug::zstd version: 1.4.8
[pre-commit/pre-commit      ]   💬  ::debug::Resource Url: http://<IP_ADDRESS>:50477/_apis/artifactcache/cache?keys=julia-1.10.0-310dd69f39ec4ba5882e8507dd218d1d143ca697aa9af89d76e91f0e8ce4ce4f&version=3971121c6fff6d6a8e35c7a73fb7ab8e8701d96e94026f2b0cd51c32618257da
| get_item_for_reloc at /cache/build/builder-amdci4-6/julialang/julia-release-1-dot-10/src/staticdata.c:1730 [inlined]
| jl_read_reloclist at /cache/build/builder-amdci4-6/julialang/julia-release-1-dot-10/src/staticdata.c:1874
| jl_restore_system_image_from_stream_ at /cache/build/builder-amdci4-6/julialang/julia-release-1-dot-10/src/staticdata.c:2996
| jl_restore_system_image_from_stream at /cache/build/builder-amdci4-6/julialang/julia-release-1-dot-10/src/staticdata.c:3458 [inlined]
| ijl_restore_system_image_data at /cache/build/builder-amdci4-6/julialang/julia-release-1-dot-10/src/staticdata.c:3523
| jl_load_sysimg_so at /cache/build/builder-amdci4-6/julialang/julia-release-1-dot-10/src/staticdata.c:586 [inlined]
| ijl_restore_system_image at /cache/build/builder-amdci4-6/julialang/julia-release-1-dot-10/src/staticdata.c:3494
| _finish_julia_init at /cache/build/builder-amdci4-6/julialang/julia-release-1-dot-10/src/init.c:859
| julia_init at /cache/build/builder-amdci4-6/julialang/julia-release-1-dot-10/src/init.c:844
| jl_repl_entrypoint at /cache/build/builder-amdci4-6/julialang/julia-release-1-dot-10/src/jlapi.c:725
| main at julia (unknown line)
| unknown function (ip: 0x400087bd8f)
| __libc_start_main at /lib/x86_64-linux-gnu/libc.so.6 (unknown line)
| unknown function (ip: 0x4010b8)
| Allocations: 0 (Pool: 0; Big: 0); GC: 0
|
| [111] signal (11.1): Segmentation fault
| in expression starting at none:0
| get_item_for_reloc at /cache/build/builder-amdci4-6/julialang/julia-release-1-dot-10/src/staticdata.c:1730 [inlined]
| jl_read_reloclist at /cache/build/builder-amdci4-6/julialang/julia-release-1-dot-10/src/staticdata.c:1874
| jl_restore_system_image_from_stream_ at /cache/build/builder-amdci4-6/julialang/julia-release-1-dot-10/src/staticdata.c:2996
| jl_restore_system_image_from_stream at /cache/build/builder-amdci4-6/julialang/julia-release-1-dot-10/src/staticdata.c:3458 [inlined]
| ijl_restore_system_image_data at /cache/build/builder-amdci4-6/julialang/julia-release-1-dot-10/src/staticdata.c:3523
| jl_load_sysimg_so at /cache/build/builder-amdci4-6/julialang/julia-release-1-dot-10/src/staticdata.c:586 [inlined]
| ijl_restore_system_image at /cache/build/builder-amdci4-6/julialang/julia-release-1-dot-10/src/staticdata.c:3494
| _finish_julia_init at /cache/build/builder-amdci4-6/julialang/julia-release-1-dot-10/src/init.c:859
| julia_init at /cache/build/builder-amdci4-6/julialang/julia-release-1-dot-10/src/init.c:844
| jl_repl_entrypoint at /cache/build/builder-amdci4-6/julialang/julia-release-1-dot-10/src/jlapi.c:725
| main at julia (unknown line)
| unknown function (ip: 0x400087bd8f)
| __libc_start_main at /lib/x86_64-linux-gnu/libc.so.6 (unknown line)
| unknown function (ip: 0x4010b8)
| Allocations: 0 (Pool: 0; Big: 0); GC: 0
| qemu: uncaught target signal 11 (Segmentation fault) - core dumped
[pre-commit/pre-commit      ]   💬  ::debug::Failed to delete archive: Error: ENOENT: no such file or directory, unlink ''
| Cache not found for input keys: julia-1.10.0-310dd69f39ec4ba5882e8507dd218d1d143ca697aa9af89d76e91f0e8ce4ce4f
| /var/run/act/workflow/2: line 2:   111 Segmentation fault      julia --project=EpiAware -e 'using Pkg; Pkg.test()'
[Test EpiAware/test-EpiAware]   ❌  Failure - Main Run unit tests for EpiAware

Not ideal but I think the thing we likely want to review here is if it works on CI which it appears to? I imagine the issue here is that the CI is targeting a version of Julia your computer can't install because of the different silicon?


Short URL for duxesrg...arm11.bin link

That link is a pain to tap in on the 3ds. A QR code would be nice except 2.1 doesn't seem to have a QR reader. So I made this short link and confirmed it works from 2 systems: http://goo.gl/UCsZgi
Analytics: https://goo.gl/#analytics/goo.gl/UCsZgi/all_time

On a system with known browser based exploits, it is probably not a good idea to have users trained to input shortened or redirected URLs that could do any number of things (especially on 2.1 with 2xrsa that gives full arm9 access from browser!).


Development

Merge updated README to main

No one to approve my own code so I have to break the rules


Function return_diff_list doesn't return the results list

The 'return_diff_list' function never returns the 'results' list, so weak ciphers, KEX, MACs, and HostKey algorithms are not reported.

Submitted PR https://github.com/evict/SSHScan/pull/22

Thanks!


fix: add HD keyring check to persistAllKeyrings

Explanation

This PR adds a check to the persistAllKeyrings method to avoid removing the HD Key Tree keyring.
References

Related to https://github.com/MetaMask/mobile-planning/issues/1507
Changelog

@metamask/keyring-controller

CHANGED: Add check to avoid removing HD Key Tree keyring during execution of persistAllKeyrings

Checklist

[x] I've updated the test suite for new or updated code as appropriate
[x] I've updated documentation (JSDoc, Markdown, etc.) for new or updated code as appropriate
[x] I've highlighted breaking changes using the "BREAKING" category above as appropriate

Looks great! I was about to approve, but I noticed that there are now conflicts


Rearrange code to be compliant with the Arduino libraries folder

Rearranged code and added examples.

All tests passed.


Header and intro html home page

Conflicted. Please resolve it first

Not working very well

Not responsive for HD screen https://gyazo.com/c7717d322a4a37f534fc23755f8d579f
Mobile menu doesn't work https://gyazo.com/63f5d63c60c475bf25cea2ae69d64dbf
Intro block if ugly, Did you follow the design? https://gyazo.com/95c5ca6fa68c100d2eaa2b2a4238fa25


docs: remove redundant trailing slashes from links

🔗 Linked issue

❓ Type of change

[x] 📖 Documentation (updates to the documentation, readme or JSdoc annotations)
[ ] 🐞 Bug fix (a non-breaking change that fixes an issue)
[ ] 👌 Enhancement (improving an existing functionality like performance)
[ ] ✨ New feature (a non-breaking change that adds functionality)
[ ] 🧹 Chore (updates to the build process or auxiliary tools and libraries)
[ ] ⚠ Breaking change (fix or feature that would cause existing functionality to change)

📚 Description
Removes redundant trailing slashes from links across the docs for consistency.

📝 Checklist

[ ] I have linked an issue or discussion.
[x] I have updated the documentation accordingly.

Would be nice to see if we could create a lint rule to catch this in future.


[Question] Is it possible to generate the cobertura file from workspace

I would like to know if is possible to generate the cobertura.xml file running from the workspace, or in order to generate the xml file it needs to run directly from the project directory.
For instance I'm trying to run from the workspace root dir the following command, and it doesn't generates the cobertura.xml file
cargo tarpaulin --all-features --workspace --tests --doc --out Xml

Is that correct, the cobertura.xml file only is generated if you run on the project dir? thanks in advance

It should work fine for a workspace, do you have a public project I can try to see what's going on?

Isn't public right now, I'll try to reproduce the error on a public project

Well, figure out that there was some errors happening on the test on some packages, removing --no-fail-fast  helped me to find the spot, after fix the issues the file started to generated correctly. Thanks for the support.


Device ID appears to be necessary for logging events

Last week, our back-end using the Amplitude Node package stopped logging events. Our events have a user_id, but no device_id. While trying to debug, we added a dummy device_id and logged started working again. Removing it and using just a user_id does not work.
In the case of using just a user_id, we see the following response:
Amplitude request completed, status code: 200, status: success, nr ingested: 0
When including a dummy device_id, we get:
Amplitude request completed, status code: 200, status: success, nr ingested: 1
Our js looks approximately like this:
        client.logEvent({
            event_type: "test-dummy",
            // device_id: "test_device",
            user_id: user_id,
            ip: req_data.ip
        }).then((res) => {
            console.log("Amplitude request completed, status code: " +
                res.statusCode +
                ", status: " + res.status +
                ", nr ingested: " + res.body.eventsIngested );
        });

Hi @mjohnsullivan,
In amplitude 1.0.x there is an issue where we silently throw away events that have neither device or user id - I was wondering if there are cases where user Id is undefined as well?
We plan on deploying a fix soon for 1.1.x to log this more readily and send a SKIPPED response instead so it's easier to debug.

Thanks for fixing that up; during testing we checked that we're sending user_ids.


Please update for vue-resource newer versions

vue-table is broken with newer version of vue-resouce (0.9.0+)
Please fix it.
anyway, thank you

@pyaehein Thanks. Didn't notice there was an update. Will look into that though.

@pyaehein I've updated vue-resource to 0.9.3 and everything still works fine. What is the error that you has encountered?

Sorry, i used old vue-table.js with new example. Now, problem solved :)


Proposals Admin/Timeline Configurator

User Story
As an Infrastructure Committee Admin, I want to have a way to configure the timeline and other relevant settings for all proposals.
Acceptance Criteria

[ ] Verify that the UI implementation corresponds to the Figma design
[ ] Once an admin clicks on the configurator icon next to the timeline of any proposal, they get into edit mode, where they can update the fields below. During edit mode, they can click cancel to exit without saving changes. Or save to apply the changes.

[ ] Proposal Status: Admin can select the appropriate stage (Draft, Review, Approved, Approved – Conditional, Rejected, Canceled, Payment Processing, Funded).
[ ] Payment Link: Admin can optionally add multiple payment links, which will appear under the “5) Funded” section.
[ ] GitHub Project URL: Admin can optionally add a URL for GitHub project. If nothing is entered, the “GitHub Project” section will not be visible on the proposal.
[ ] Project Coordinator: Admin is required to enter Project Coordinator once they get to the decision stage. Until that point, if no coordinator is entered, the “Project Coordinator” section will not be visible on the proposal.

[ ] Logging: Logs are created every time there is a new edit or activity on the RFP. The log includes the admin who made the change (link to their profile), the event, and timestamp.

Attachments
See mock up:

Updated AC. Removed GitHub Project URL field.


[feature-request] embulk extra args options.

Hello.
I would like to execute embulk run -c diff.yml config.yml command.
But I can't add -c diff.yml option in embulk operator.
So Now I'm using sh operator. It's not smart way.
Please add extra args option for embulk operator.
Digdag version: 0.8.1
Thanks.

How about adding config_diff command like this?
+load:
  embulk>: data/load.yml
  config_diff: true

If this feature is enabled, embulk operator

Reads a config diff stored in the database (h2/PostgreSQL) and generates a temporary config diff file if it exists
Executes embulk command with -c option specifying the config diff file
Reads a generated config diff file and stores it in database (h2 / PostgreSQL)

@komamitsu Thank you for the reply.
I think config_diff option is useful.
Someone want to use needs more options.
(ex. Now I'm using -r option)
How about those options?
I'll ask another Embulk user about this option.
embulk>:
config_diff:   # -c option
bundle:        # -b option
log_level:     # -l
system_config: # -X
- page_size:
jvm_opts:      # -J
resume_state:  # -r (Don't needs?)

-J option support file path. It's simple. But It does not work on Windows platform.
Usage: embulk run <config.yml>
  Options:
    -r, --resume-state PATH          Path to a file to write or read resume state
    -o, --output PATH                (deprecated)
    -c, --config-diff PATH           Path to a file to read & write the next configuration diff

  Plugin load options:
    -L, --load PATH                  Add a local plugin path
    -I, --load-path PATH             Add ruby script directory path ($LOAD_PATH)
    -C, --classpath PATH             Add java classpath separated by : (CLASSPATH)
    -b, --bundle BUNDLE_DIR          Path to a Gemfile directory (create one using "embulk mkbundle" command)

  Other options:
        --log PATH                   Output log messages to a file (default: -)
    -l, --log-level LEVEL            Log level (error, warn, info, debug or trace)
    -X KEY=VALUE                     Add a performance system config

I asked the Embulk users about this option.
This is the summary.
I think it is better to discuss embulk> operator feature before implement config_diff
The reason is the following.

An Embulk user use sh> operator instead of embulk> operator.

A) It support all of the embulk options. (Include config_diff)
B) It support Liquid template engine.

config_diff option solve problem A). But It does not solve the problem B).
So If this option implement, those of users can't use embulk> operator.
I asked Mr. Furuhashi about this issue at Embulk meetup.
(I created this issue before the meetup.)
He wants to implement ${} syntax in Embulk. But this feature has not implemented yet.
Reference Twitter (Japanese text).

embulk operator had been deprecated. So close.


Molecule 4.0.4 Missing Lint

Issue Type

Bug report

Molecule and Ansible details
$ ansible --version && molecule --version
ansible --version && molecule --version
ansible [core 2.15.2]
  config file = /playbook/ansible.cfg
  configured module search path = ['/home/cli/.ansible/plugins/modules', '/usr/share/ansible/plugins/modules']
  ansible python module location = /usr/local/lib/python3.11/site-packages/ansible
  ansible collection location = /playbook/collections
  executable location = /usr/local/bin/ansible
  python version = 3.11.4 (main, Jun 15 2023, 06:10:48) [GCC 12.2.1 20220924] (/usr/local/bin/python)
  jinja version = 3.1.2
  libyaml = False
molecule 4.0.4 using python 3.11
    ansible:2.15.2
    azure:23.0.0 from molecule_plugins
    containers:23.0.0 from molecule_plugins requiring collections: ansible.posix>=1.3.0 community.docker>=1.9.1 containers.podman>=1.8.1
    delegated:4.0.4 from molecule
    docker:23.0.0 from molecule_plugins requiring collections: community.docker>=3.0.2 ansible.posix>=1.4.0
    ec2:23.0.0 from molecule_plugins
    gce:23.0.0 from molecule_plugins requiring collections: google.cloud>=1.0.2 community.crypto>=1.8.0
    podman:23.0.0 from molecule_plugins requiring collections: containers.podman>=1.7.0 ansible.posix>=1.3.0
    vagrant:23.0.0 from molecule_plugins

Molecule installation method (one of):

pip

Ansible installation method (one of):

pip

Desired Behavior
Molecule 4 should still support lint
Actual Behaviour
aa458c73cbbe:/playbook$ molecule test
CRITICAL Failed to validate /playbook/molecule/default/molecule.yml

["Additional properties are not allowed ('lint' was unexpected)"]

@ssbarnea Is this still an issue after after the release of Molecule 6?

FYI seeing the same with molecule 5.1.0 and, alas, going to 6.0.2 isn't an option 'cos driver supported is borked as per this 😿


Track Current (IPC-2152) Calculator Scrolls Slowly When In full-screen and Intermediate Variables are Expanded

Closing as porting to web app.


DocC Version Switcher

Fixes #1834
Feature gated for development/staging at the moment until the other tickets are also implemented.

how exciting

Beautiful!

One last visual change before merging. I had removed the "Switch to…" from the server-side HTML so it was easier to check for empty arrays as an indicator of no other versions, intending to put it back in CSS but had never got around to doing that. It now looks like this:


Avoid scanning tokens past end of input

This PR adds bounds checks and tests for a set of partial citation inputs that currently raise IndexError from get_citations --
'123 supra.'
', supra.'
', 1 U.S., at 1'
'1 U.S., at 1'
'v. Bar, 1 U.S. 1'
'1 U.S. 1 ('

The one functional change is that get_citations('supra.') now returns a SupraCitation with no antecedent, where before it would have returned nothing. I think it's less surprising to have get_citations('supra.') do the same thing as get_citations('no cite supra.') instead of being a special case.

Looks fine code-wise. Would it help future-us to have some comments in there? The index adding/subtracting is pretty confusing without something like that?

I added a few comments, and tweaked extract_id_citation to do the same thing with a little less + 1. Looks like tests have a false failure and want to be rerun.
I didn't add specific comments for all the if index > 0:, because it ends up being ten different copies of "Bounds checks to ensure we don't scan before start or end of words" and started to feel like noise. Basically you have to never access words[i] without also deciding what makes sense to do if i < 0 or i > len(words), or clamping i so that can't happen, and the comment ended up feeling like it was restating what the code was doing rather than illuminating it.

I went ahead and merged this. I think our venv cache got screwy again, but there's a 95% chance the next PR will be from you, so I imagine you'll catch any failures on that one, if they're real.
Thanks for this fix.


es6 generation fails under certain conditions

Im not sure if this is an openfl issue, or an haxe issue (or possibly both?), but the following minimal repro fails to run when using es6 generation and -final:
package;

class Main extends openfl.display.Sprite {
    public function new() {
        super();

        var something = new SomeSubClass();
        var x = SomeSubClass.FOO;
        trace(something, x);
    }
}

enum abstract ClassType<T:SomeClass>(String) to String from String {
}

class ClassTypeBuilder {
    public static function name<T:SomeClass>(name:String):ClassType<T> {
        return cast name;
    }
}

class SomeClass {
    public function new() {
    }
}

class SomeSubClass extends SomeClass {
    public static final FOO:ClassType<SomeSubClass> = ClassTypeBuilder.name("someSubClass");
}

fails with (at runtime):
Main.js:1320 Uncaught TypeError: tb.name is not a function
    at Main.js:1320:215
    at H.lime.$scripts.Main (Main.js:1342:11)
    at H.lime.embed (Main.js:1342:309)
    at (index):40:8

Must use -final and -D js-es=6 in order to trigger it

I can't reproduce if I convert it to pure Haxe with static function main() and -dce=full and -D js-es=6. I believe that OpenFL's -final flag uses -dce=full, but it also uses Closure Compiler to minify the code. I wonder if Closure is renaming or removing something that it shouldn't.

Yeah, i figured it would probably be down to something in the minification process, but I didnt / dont know enough about it to be sure. Didnt think to test in a pure haxe app either - my bad :)

Any news on this?

I think Josh added Terser as an optional minification pipeline

Any news on this?

Yes, just to further verify this. You can try using lime build html5 -minify -terser if you have this issue and hopefully it with remedy the situation.

Sadly, that didn't work either. Same issue.


Do not error out if a service instance status cannot be determined

Mark it as Unknown instead.
Was the change discussed in an issue?
No.

@metacosm Can you please add the steps for how to test

Appveyor test passed, we just exceeded artifact storage quota :-(

@kadel  can we clean up the old artifacts?


External selection mgmt

This works pretty well. I've been comparing this package to a bunch of other react trees, and I am most hopeful this will work for my use case.
One thing I find strange is I can only access selection state from inside the Node renderer.  Why not expose onSelect the same as you do with onEdit and onToggle, to make it easier for syncing with the selection state of the tree?
Related, you added an onClick prop (that isn't listed on the documentation page) and I'm wondering if that was meant to make it possible to configure external selection state synching? It fires the synthetic click event and not some computed state object so I'd be we wary using that to try and set up outside mgmt.
Am I missing something?

ah, nm. I realized you can access those methods on the ref handler. Thanks for making this package!

@markwatabe I pondered this same question. In the end, I decided to make the methods available on the ref as you discovered. Here is a piece of code I wrote for the brimdata/brim app that selects an item in the tree whenever the external selection changes.
https://github.com/brimdata/brim/blob/79640b32f3b74d5dc49dd7b3e5546efaebc17791/src/app/features/sidebar/queries-section/index.tsx#L135-L142
Here is a video showing that piece of code in action.  The tree component selects the correct item when the tab switches.
https://user-images.githubusercontent.com/3460638/160152953-fc71977a-d275-4ed5-b48d-45439a1b8812.mov

I need to document the Tree's onClick and onContextMenu props. Those are attached to the tree container so that you can take action when the user clicks or rightclicks on an area of the tree that has no items. Maybe you clear the selection, or provide a different context menu.
About a year ago I did the same search for a tree library and couldn't find one flexible enough, hence react-arborist. If you don't mind sharing, I'd love to hear what your use case is.

@markwatabe Was there anything in the dev console when your other drag and drop functionality died? I'll make an issue.

Also @markwatabe what kind of library (or not) are you using for the other drag and drop functionality in your app?

It seemed like the presence of this library seemed to kill the default behavior of other components that had onDrag, onDrop, etc events. That is to say I toggled on and off importing this library and that seemed to fix the problem. That being said, I might have had other libraries loaded that could have created a complex situation?
I can try and reproduce in the next couple of days and post the isolated code.


Tambah Dynamic Open Graph

Tujuan: user boleh terus nampak detail event melalui preview image di shared link di social media (dynamic OpenGraph)
Tutorial: https://cruip.com/generate-dynamic-open-graph-and-twitter-images-in-next-js/
Contoh:

nice feature. good idea ni


Company name changes - update in Orgbook test required

Hi,
MOTI has completed some test company name changes on the following BC corporations which have had name changes completed in COLIN:
Company 1 new name: 0879839 B.C. LTD.
Company 1 Incorporation number: BC0879839
Company 2 new name: 0879833 B.C. LTD.
Company 2 Incorporation number: BC0879833
This is a high priority for us as we're testing a major bug in our public facing application.

Orgs have been queued for processing in test

Thank you @amanji

Closing as completed: See screenshots below

Thanks!!
Jonathan Argue
Consultant
J.S. Argue and Associates Ltd.
Email: @.***
Cell:<PHONE_NUMBER>
www.jsargue.ca
www.imaginarytable.ca
I acknowledge that I have the privilege of living and working on  the
traditional and unceded territory of the lək̓ʷəŋən speaking peoples, today
known as the Esquimalt and Songhees Nations.
On Tue., Sep. 26, 2023, 15:05 Akiff Manji, @.***> wrote:

Closing as completed: See screenshots below
[image: OrgBook_Test_0879833]
https://user-images.githubusercontent.com/5605464/270807538-4ed31772-8a5e-47c9-b8f9-4e0a5b8617df.PNG [image:
OrgBook_Test_0879839]
https://user-images.githubusercontent.com/5605464/270807541-7da7bbf8-d8b5-48ce-8fa1-84fbc61451d8.PNG
—
Reply to this email directly, view it on GitHub
https://github.com/bcgov/DITP/issues/71#issuecomment-1736364347, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AAKRTPUMVZHZZMJF2C4YTZ3X4NGSRANCNFSM6AAAAAA5IEZ474
.
You are receiving this because you authored the thread.Message ID:
@.***>

--
IMPORTANT: This transmission is sent on behalf of J.S. Argue & Associates
Ltd. and it may be privileged, proprietary or confidential. If you are not
the intended recipient or a person responsible for delivering this
transmission to the intended recipient, you may not disclose, copy or
distribute this transmission or take any action in reliance on it. If you
received this transmission in error, please notify us immediately by
telephone at<PHONE_NUMBER> and please dispose of and delete this
transmission. Thank you.


Bug: setting reviewers for multiple file patterns is not working properly

Hi your action suits very well for my use case, so thanks for your work on this project.
I think I might bump into a bug today. Could you please provide some help? @shufo
My action is:
name: "Auto Assign"
on:
  pull_request:
    branches:
      - master
    types: [opened]

jobs:
  assign_reviewer:
    runs-on: ubuntu-latest
    steps:
    - uses<EMAIL_ADDRESS>      with:
        config: '.github/assign-by-files.yml'
        token: ${{ secrets.GITHUB_TOKEN }}

My assign-by-files.yml file is like:
---
'*.md':
  - AAA

'media/':
  - DDD

'.github/':
  - BBB
  - CCC

My PR changed 2 yml files under .github/ folder so the bot should assign only BBB and CCC as reviewers. And the github action runs with no error but the bot requested AAA, BBB, CCC, DDD as reviewers. Could you please check on this?

@yikeke
Thanks for reporting issue!
It seems bugs around minimatch and Fixed in this PR.

Minimatcher was used incorrectly and was matching every rule

Could you try latest version that merged this PR?
name: "Auto Assign"
on:
  - pull_request

jobs:
  assign_reviewer:
    runs-on: ubuntu-latest
    steps:
    - uses<EMAIL_ADDRESS>      with:
        config: '.github/assign-by-files.yml'
        token: ${{ secrets.GITHUB_TOKEN }}

I tried in my working repo and I think it works now, thanks so much!


Small features for pirat war and quest druid

I have got some values for minions (like minion race) and heroes.
Also I fixed weapons health, now it is work correct.

Great job bro.


Kuiper manager docker images v1.0.0 and 1.0.1 are not working

The web interface of the kuiper manager is not loading in v1.0.0 and v1.0.1 of the docker images. It says page not found.

sudo docker run --name kuiperManager -d -p 9082:9082 emqx/kuiper-manager:1.0.1

Docker logs don't show any errors.

@ysfscream Could you please take a look at the problem?

@zhanghongtong Could you please repack the kuiper-manager:1.0.1? Thanks.

Any update?
@jinfahua  @zhanghongtong  @ysfscream

@untilbit Hello, sorry for the late reply. This is a bug that has been fixed, please re-pull the docker image and run it, thanks for your feedback.

Issue fixed.


add sgx-ansible script (Ubuntu 20.04)

Adds the sgx-ansible script for Ubuntu 20.04 with an Example User ssh key and and examplehost server. It's up the up to date script with our current, unpublished version.
I have skimmed through all files but didn't find any personal information left. What I did find however, is that some scripts (which we have not written by ourselves, but rather imported from geerlinguy) are outdated and will most likely not work for Ubuntu 20.04. These scripts are unrelated to sgx itself.
Should we maintain these scripts, leave as is, or delete them? (example: keyboard, munin-node, netplan...)
The README will be added with a separate key.
Related to issue #1

If the basic installation works as-is, I'd suggest to merge this one and just file an issue to update the outdated dependencies


[#127] update gitignore file

Resolves #127
I add some ignore for IntelliJ, MacOS and CMake by using this, https://github.com/github/gitignore/blob/master/Global/JetBrains.gitignore
And I think we need to discuss what to add or update.
I'm not sure which one is better between using
.idea/**/workspace.xml
.idea/**/tasks.xml
.idea/**/usage.statistics.xml
.idea/**/dictionaries
.idea/**/shelf

to ignore some of .idea
or just using
.idea/ to ignore all of .idea
If you have any opinion of this issue, feel free to review. :)

Coverage remained the same at 85.229% when pulling fa58647cb897e4c12c3f7e8446690a06c00fc551 on RulLu16:feature/127_update_gitignore into 085c7b193ca3c8fe8c2d6d328c8a4521aa53c7cf on ztkmkoo:develop.


Lamda function failed

The lamda function failed with this error. Do you have any suggestions as of why?
Thank you so much!
2017-08-10T09:13:51.892Z	39847bd5-7dac-11e7-bf61-c376ad7ff20a	SyntaxError: Unexpected token C in JSON at position 0
at Object.parse (native)
at Request._callback (/var/task/index.js:78:29)
at Request.self.callback (/var/task/node_modules/request/request.js:188:22)
at emitTwo (events.js:106:13)
at Request.emit (events.js:191:7)
at Request.<anonymous> (/var/task/node_modules/request/request.js:1171:10)
at emitOne (events.js:96:13)
at Request.emit (events.js:188:7)
at IncomingMessage.<anonymous> (/var/task/node_modules/request/request.js:1091:12)
at IncomingMessage.g (events.js:292:16)
at emitNone (events.js:91:20)
at IncomingMessage.emit (events.js:185:7)
at endReadableNT (_stream_readable.js:974:12)
at _combinedTickCallback (internal/process/next_tick.js:80:11)
at process._tickDomainCallback (internal/process/next_tick.js:128:9)

I'm going to close this issue here and move it to the skill repository. The Heroku server doesn't require a Lambda machine.


Template for Pull Request

Added pull_request_template.md in .github . Template for pull request which will be used when a pull request in made
Type of change

[x] Added Template

Sorry, I understand the motivation of contribution here, but pull request templates are meant to be to make reviews by maintainers / core contributors easier, hence they're created by core contributors. This just isn't what we'd want from a template, if we need one. As it stands we also don't get a lot of discussions that lead to PRs. We either "just get a PR" without notice and that often doesn't lead to a merge, or leads to some back and forth, or we get a PR that has been discussed prior, where the author and reviewers are already aware of what's happening.


replace xyce with ngspice

replace xyce with ngspice since all available generators support ngspice. This will make the CI of ldo-gen to pass.

@lulu9312 the ldo-gen is passing now


Enable -msave-restore at -Oz

get this to land: https://reviews.llvm.org/D134129

(i actually don't think we want this, but we should keep the bug open so that if that change does go in, we can turn it off again for Android. at the very least we'll want to benchmark to see the negative impact.)


Use local filesystem images for TileOverlay

The plugin used to be able to handle local images for TileOverlays. When trying to upgrade my app I couldn't get this working again.
I'm not sure when the change was introduced, but PluginTileProvider.java now explicitly limits the tileUrlFormat to starting with "http://" and "https://". URLs starting with "file://" will never work.
Its fairly straight forward to get this working again for Android, but I don't know enough to do the iOS coding.
This would be really useful for caching map tiles for use when a device is not online.

Android or iOS?

It seems I did not implement file:// for Android

Any way to do it using Local Tiles? Thank you!

I've put in a pull request to fix this on Android (it already works on iOS).  Hopefully they'll merge it.
If you need it working before that, you can see my suggested changes at my fork of this repo at
https://github.com/webalchemist/phonegap-googlemaps-plugin
This version is working fine for me.

Thank you for long-time waiting.
The v2-beta version supports local tiles both Android and iOS.
map.addTileOverlay({
    // Load image files from the local file path
    tileUrlFormat: "../images/map-for-free/<zoom>_<x>-<y>.gif"
}, function(tileOverlay) {

});

https://github.com/mapsplugin/v2.0-demo/blob/master/www/TileOverlay/addTileOverlay.html#L42-L47
The v2-beta is available.
https://github.com/mapsplugin/v2.0-demo


Feature request: Desktop support

Hello! Thanks for the work on this project!
I'd like to request desktop support (windows, MacOS and Linux) in this project. I'm not sure if it's in the scope of this project to capture from video streams or not on these platforms, but I think just being able to provide an image buffer and scan it for a QR code would be very useful. I can't help with development, but I can help with testing if needed!

Hi @dainnilsson
This project uses the official camera  plugin, which does not support desktop platforms at the moment.
But in the future, when possible, I plan to add support for desktop platforms.
Track Linux status
Track MacOS status
Track image stream support on Windows
Thanks!

Got it, thanks! Would it be possible to just provide an API for detecting and decoding a QR code from an image on desktop? That would be sufficient for my use case at least, and would probably be required anyway once the camera plugin supports desktop.

I think it will be possible, I will work on it.

Hi @dainnilsson
Desktop platforms are now supported in the v1.0.0 beta version.
You can use this version to detect and decode QR codes from images on desktop platforms.


Add Action types

Motivation and Context

Added in new action types for CE Estab Individual print  files

What has changed

Added new Action types

How to test?

Run the tests
Create action rules with new action types

Links

Trello
Screenshots (if appropriate):

 (IC05/IC06/IC08) Generate CE ESTAB Individual Addressed Print Files - NI (5)


Fix datum in tx and ref scripts

(Couldn't reopen https://github.com/input-output-hk/cardano-node/pull/3881, so created this one)
:snowflake: Add a roundtrip property TxBodyContent -> TxBody -> TxBodyContent
This helped in fixing the :bug: and uncover the two additional gaps in the code. I'm not 100% happy with the current implementation of the property though! I needed to accept two exceptions to the general ===:

SimpleScriptV1 reference scripts may become SimpleScriptV2
A TxOutDatumHash + a matching ScriptData may become a TxOutDatumTx

:snowflake: Resolve datum hash + matching datum in transaction to TxOutDatumInTx, fixes #3866
:snowflake: Add missing script languages to scriptLanguageSupportedInEra for BabbageEra
:snowflake: Allow scripts in any language as refeference scripts

@Jimbo4350 squashed commits correctly and added type signatures to all functions in where clauses. Might want to have another look if this is good to go.

bors merge

bors r+

bors merge


Make proper version for Linux and Windows

When I add
// *** Code injected for MathJax support
    // See: https://github.com/fsavje/math-with-slack

    var mathjax_inject_script = `
      var mathjax_config = document.createElement("script");
      mathjax_config.type = "text/x-mathjax-config";
      mathjax_config.text = \`
        MathJax.Hub.Config({
          messageStyle: "none",
          extensions: ["tex2jax.js"],
          jax: ["input/TeX", "output/HTML-CSS"],
          tex2jax: {
            skipTags: ["script","noscript","style","textarea","pre","code"],
            inlineMath: [ ['\$','\$'] ],
            displayMath: [ ['\$\$','\$\$'] ],
            processEscapes: true
          },
          TeX: {
            extensions: ["noErrors.js", "noUndefined.js"]
          }
        });
        \`;
      var mathjax_script = document.createElement("script");
      mathjax_script.type = "text/javascript";
      mathjax_script.src = "https://cdn.mathjax.org/mathjax/latest/MathJax.js";
      document.getElementsByTagName("head")[0].appendChild(mathjax_config);
      document.getElementsByTagName("head")[0].appendChild(mathjax_script);

      var render = function (records, observer) {
          MathJax.Hub.Queue(["Typeset", MathJax.Hub]);
      };
      var target = document.querySelector('#msgs_div');
      var observer = new MutationObserver(render);
      var config = { attributes: false, childList: true, characterData: true, subtree: true };
      observer.observe(target, config);
    `;

    window.webviews = document.querySelectorAll(".TeamView webview");
    setTimeout(function() {
      for(var i = 0; i < webviews.length; i++) {
        webviews[i].executeJavaScript(mathjax_inject_script);
      }
    }, 10000);

    // *** End injected MathJax

below startup(); call in index.js it works just fine under Windows. Only the bash script wouldn't work.
Note, that I had to change the bash escapes of the grave accent.

Thank you @crstnbr. I suspect that would be the case. Excellent that you could confirm that for me. I'll try to draft up a batch script as soon as possible. Thanks again!

Youre welcome. Thanks for coming up with this in the first place.
Btw, the index.js file was located here: C:\Users<USERNAME>\AppData\Local\slack\app-2.5.1\resources\app.asar.unpacked\src\static\index.js

Hey @crstnbr I've added a windows version. If you want, it would be great if you could try it out. Just follow the instructions here to install it.
I've added some safety checks so that it doesn't overwrites the wrong file. If you've edited index.js by hand, you'll probably need to restore it to the original version for the script to work. If you didn't take a backup, here's the original version: index.js.zip
Thanks for helping out!

Worked for me!

Excellent, thanks!


Garage door opener doesn't update open/closed status

Describe the bug
SmartLife Garage door accessory always shows "closed" in homebridge and apple Home.  It accurately shows open/close status in SmartLife app.  Device is this - meross/smartlife.  It's also successfully opening/closing the garage door, but it's entirely open loop - no feedback.
Steps To Reproduce

Add garage door opener device to SmartLife app on account connected to TuyaWebPlatform
That's it.  Open/closed status doesn't update.

Plugin Version
v1.1.5
Homebridge Config
{
   "bridge": {
       "name": "Homebridge 8A80",
       "username": "0E:BA:B5:59:8A:80",
       "port": 51296,
       "pin": "801-58-869",
       "advertiser": "avahi"
   },
   "accessories": [],
   "platforms": [
       {
           "name": "Config",
           "port": 8581,
           "platform": "config"
       },
       {
           "name": "TuyaWebPlatform",
           "options": {
               "username": "",
               "password": "",
               "countryCode": "1",
               "platform": "smart_life"
           },
           "defaults": [
               {
                   "id": "eb1057ce47bb30fee4v3sp",
                   "device_type": "garage"
               }
           ],
           "scenes": false,
           "platform": "TuyaWebPlatform"
       }
   ]
}

Device Config
Getting credentials
Got credentials
{   'access_token': 'AZv051hf5af6f30az1704168657387TAv8dBioccThGmv',
    'expires_in': 864000,
    'refresh_token': 'AZv051hb4a99e69az1704168657387TAv8d87UHJYRDZ6',
    'token_type': 'bearer'}
Getting devices
Got devices
{   'header': {'code': 'SUCCESS', 'payloadVersion': 1},
    'payload': {   'devices': [   {   'data': {'online': True, 'state': False},
                                      'dev_type': 'switch',
                                      'ha_type': 'switch',
                                      'icon': 'https://images.tuyaus.com/smart/icon/ay15773419281994CtTi/1583392220ed539fa5927.png',
                                      'id': 'eb1057ce47bb30fee4v3sp',
                                      'name': 'Smart WiFi Garage Door Opener'}],
                   'scenes': []}}

Additional Context

@milo526


docs: fix typo on engineering_levels.md

This looks, walks and quacks like a typo. 🤔

Not fixing the his on line 89 because https://github.com/ockam-network/website/pull/468 already covers it

Thank you for fixing that 🙏


Add code gathering for single cell

It looks like the functional tests aren't working either. You might have to add more things to our document mocking in the reactHelpers.ts

You can run them locally with 'npm run test:functional'. That should repro the bug

It looks like the functional tests aren't working either. You might have to add more things to our document mocking in the reactHelpers.ts

Yep fixed them here and here. Probably related to https://github.com/palantir/blueprint/issues/397?


Add "cookies" alias for the cookie command

Is your feature request related to a problem? Please describe.
Instinctively I wanted to wrote down cookies instead of cookie.
I think that we can clearly add cookies as a shortcut to handle this case.
Describe the solution you'd like
Allow to use cookies as an alias for the cookie command.


docs: In the description of Fat Bits, indicate where it zooms in on and how to control it

Hi there,
I started playing around with Decker this weekend and have been enjoying myself - one point of confusion was how a user can choose what part of the current card to zoom in on with Fat Bits; the docs weren't super clear on this so I had to skim the source to figure it out.  This patch simply adds the behaviour to the reference doc.
If memory serves, HyperCard's behaviour was to remember the place most recently clicked on by a draw tool (so, for instance, one could pencil some drawing and then zoom in for details using Fat Bits without having to explicitly indicate where Fat Bits should go); I wonder if Decker users would benefit from this behaviour too?
Thanks!

Thanks for the clarification.
I will point out one other detail noted later in Decker's manual:

Holding control or command while clicking will enter Fat Bits mode centered on the position you click, or exit Fat Bits mode.


Feedback for "Creating a Project from the Terminal"

https://help.sap.com/webcomponents/products/SAP Business Application Studio/9d1db9835307451daa8c930fbd9ab264/c29e1a3a652242e089af453c2177249a.html?locale=en-US
In Markdown it looks good:

but not on SAP Help Portal:

while at it, why not mention the shortcut:
CTRL+`
it works better on the US keyboard than it reads as an instruction since it can easily be confused with:
CTRL+'
which does 'nothing'.

Hi,
Thank you both for your feedback!
@iwonahahn There is already an internal thread open with this issue. You are welcome to follow the progress there,  https://sapjira.wdf.sap.corp/browse/DPS-11558
@greg2git  We prefer not to confuse the users with several ways of doing the same action, so for the moment we will keep the menu instructions.  For the users who prefer to use the keyboard, there is a link to the list of the shortcuts on the SAP Business Application Studio Welcome page.


unable to resize .webp format

Microsoft PowerToys version
0.87.0
Installation method
Microsoft Store
Running as admin
No
Area(s) with issue?
Image Resizer
Steps to reproduce
features that were working fine just a week or two ago suddenly disappeared
running as administrator didnt work
reinstalling powertoys didnt work
im new to git and cant find button to upload image
so ill just leave a image link of problem
https://imgur.com/Ta88WvR (.webp)
sorry for the foreign language but you get the idea
image resizer doesnt exist in context menu
it looks fine in .png file it seems to only happen in webp format
https://imgur.com/a/mFIhfly (.png showing image resizer menu)
✔ Expected Behavior
No response
❌ Actual Behavior
No response
Other Software
No response

contents of https://github.com/microsoft/PowerToys/issues/1929 didnt helped

ImageResizer never worked for .webp format, as far as I know. /needinfo

ImageResizer never worked for .webp format, as far as I know. /needinfo

nah it worked fine before and recent update seems to have fixed the prob so its working fine rn


Fix for Issue #1387.

Check the return value of 'ecma_op_put_value_lex_env_base' function.
JerryScript-DCO-1.0-Signed-off-by: Robert Sipka<EMAIL_ADDRESS>

Please add the testcase too.

Sorry, I've added the test case too.

LGTM

Thanks, I've rebased it with master.


[AutoML] Clone AutoML feature branch into docs/samples/automl

@natke to review if you have a moment --
Thank you for all the help!!!!

@shauheen to check on whether we are ok with the shared infrastructure.

Need to confirm this doesn't create any coupling between ML.NET and AutoML document generation, document index, and monikers.